Remove unused variable

MAT_T_ARRAY is nowhere used, too

You are right about it. But since matio.h defines the official API to the library I won't delete it in order to not break any backward compatibility.
Thanks for the patches anyway.

To my mind this change would not have a negative effect on the api because when you're upgrade your lib to a new version your code will fail anyway (if you have used the old variables). When the variables are not available it's easier to find the bug, too.

I am afraid I won't touch them even though they are dummies only.

I can live with that ;-)
Thanks for your answer!

Indeed, no big deal at all.


utm support missing

Utm tracking does not work for us currently. Is it not supported yet, or did we something wrong?

It works for me just fine are you doing ?utm_campaign=some-campaign to the URL?

Yeah, looks fine. Sorry, now idea what we did wrong...

Thanks for replying @JamesSingleton!


[Clipboard] Search for snippets

Description
Clipboard and snippets are great tools already. I would like to suggest that I could search for snippets in the clipboard view. We can already save the clipboard entries from clipboard view.
Who will benefit from this feature?
Everyone that are used to have one shortcut for clipboard do not need to do a lot of search and click in the spotlight replacement.
Anything else?
No response

This issue is not related to the API or extensions. Could you please use the Send Feedback command inside Raycast or use the #help or #bugs channel in our Slack Community 😊


[SEO][Rollout Monikered Canonical]Add URL moniker

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


Doesn't work in Unity 5.4.3f1

The following errors show up when running the example scene in Unity 5.4.3f1 with HTC Vive.
Note that CurrentIntegration error appears twice.
Removing Oculus, OpenVR or both from Player settings Virtual Reality SDKs or disabling VR completely doesn't help.

OpenVR initialized!

[NewtonVR] Version : 0.97. Found VRDevice: Vive MV.

[NewtonVR] Critical Error: Oculus / SteamVR not setup properly or no headset found.

UnityEngine.Debug:LogError(Object)
NewtonVR.NVRPlayer:SetupIntegration(Boolean) (at Assets/NewtonVR/NVRPlayer.cs:201)
NewtonVR.NVRPlayer:Awake() (at Assets/NewtonVR/NVRPlayer.cs:157)

[NewtonVR] Critical Error: NVRPlayer.CurrentIntegration not setup.

UnityEngine.Debug:LogError(Object)
NewtonVR.NVRHand:PreInitialize(NVRPlayer) (at Assets/NewtonVR/NVRHand.cs:174)
NewtonVR.NVRPlayer:Awake() (at Assets/NewtonVR/NVRPlayer.cs:165)

[NewtonVR] Critical Error: NVRPlayer.CurrentIntegration not setup.

UnityEngine.Debug:LogError(Object)
NewtonVR.NVRHand:PreInitialize(NVRPlayer) (at Assets/NewtonVR/NVRHand.cs:174)
NewtonVR.NVRPlayer:Awake() (at Assets/NewtonVR/NVRPlayer.cs:165)

NullReferenceException: Object reference not set to an instance of an object

NewtonVR.NVRPlayer.Awake () (at Assets/NewtonVR/NVRPlayer.cs:169)

Whoops! Thanks for reporting. We'll get this fixed asap

Seems like the master branch got out of sync with development for some reason. Can you try version 0.981 (committed a few minutes ago) and tell me if you have the same issues?

Yes, commit 4f94d939 (version 0.985) gives the exact same errors.
I also tried running Unity as administrator, doesn't help. Trying on Windows 10 Anniversary Edition.

In your NVRPlayer do you have Enable SteamVR checked? Maybe I should make that error more descriptive

Yeah, it might be good to make the error more descriptive. I didn't have it checked, thanks. Now it works and there are no errors, although there is this message:

[NewtonVR] Version : 0.987. Found VRDevice: Vive MV. Using SteamVR SDKDid not find supported VR device. Or no integrations enabled.

Thank! And sorry about that. Clear setup instructions and error messages are definitely important. I'll try and fix that log error too


[Event Request] Record::"Sales Line Archive" - Function CopyTempLines OnBeforeCopyTempLines

Additional filter criteria.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


fix: prevent dev server crashing on malformed URI (fix #6300)

Description

Additional context

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

when there is URIError, the dev server will crash because it didn't catch the error
reprodution : open url like  http://localhost:3000/a%a

would it be better to first try with decodeURI and then without?
decodeURI change was introduced in https://github.com/vitejs/vite/pull/4728

I think that makes sense too. Like some sort of tryDecodeURI utility. The only other place I found which might hit the decode error was viteServeStaticMiddleware, but that middleware is instantiated after viteTransformMiddleware (where this PR fixes), so this PR would indirectly cover viteServeStaticMiddleware too

i just found that there's already a errorMiddleware which can handle the error and prevent crash🤣, whether in transformMiddleware or serveStaticMiddleware, looks like this:

so i correct the return with  next(e) , so the error can be passed to the errorMiddleware to handle it:

@bluwy @patak-dev what do you think is best fallback or showing error message?

Looking back now, I think showing an error message may be safer, since having a fallback might imply that we support it, which we might not. So using next(e) looks good to me. Neat solution too.
@toSayNothing  Re viteServeStaticMiddleware, I'm taking back what I said 😅 viteTransformMiddleware can be skipped, so we need to handle decodeURI for viteServeStaticMiddleware too:
https://github.com/vitejs/vite/blob/8bdb1848dd777f8de9f63b27a7746cec65db5b70/packages/vite/src/node/server/middlewares/static.ts#L71

🤣i have debugged and found that :
connect can handle the Synchronization function 's error , but not the Asynchronous function .
viteServeStaticMiddleware is sync, so it can handle by the connect itself(call try catch)
but the async viteTransformMiddleware will crash the server if not handly by catch(e){return next(e)}

Nice observation @toSayNothing. That's an interesting behaviour from connect. It would sure be nice if it handles promises too, but for now this looks good to me.


less strict response check

We should be using caseless header checks; however, there is very
little value in the header check anyway because since we are
contacting the Google metadata service. Requiring a header in the
response makes it harder to nock the metadata service.
Fixes: https://github.com/stephenplusplus/gcp-metadata/issues/6

Why did we have this in the first place?

I added it (incorrectly) in https://github.com/stephenplusplus/gcp-metadata/commit/58912fe89882ae4869aaa511ae38dd68db10352b. The main condition I wanted to catch was res.statusCode not being 200.
It is still possible to add a case-less check for the metadata header, but given that we don't have system tests for this module, I am not sure we can prove that the test works.
I think the header check is more trouble than it is worth.

Agreed, I just wasn't sure if we needed that for something else. Merge and release on the way.

4 hours later:

<EMAIL_ADDRESS>

Thanks!


Draft: Update pygamer to current HAL

Summary
Initial work by @bradleyharden for v2 SERCOM, with more updates to account for improvements in our project and others.
Still some issues to work through, but at this stage all the examples compile:

[ ] Fix neopixel_rainbow_spi
[ ] Some of the delay-driven neopixel examples flicker quite a lot
[ ] Decide whether to remove or fix the USB polling example
[ ] Clean up Git history
[ ] Update CHANGELOG.md for the BSP

From some not-very-thorough testing today, I think fixing the timer-based neopixel issues will at least require some work in ws2812-timer-delay-rs because the timing it generates (via GPIO and waiting on a supplied timer) isn't quite right on SAMD51 with the new HAL.  I've tried adjusting the library locally, and have the examples working well with opt-level="s", but then dialing it up to 1 or higher causes problems again.  On paper, and based on the WS2812 datasheet I found, the current delays in that crate shouldn't work, so I think it's likely that fixing them for this case would just move the problem to some other user of that crate.
Although the SPI neopixel example isn't working at present, that architecture seems a lot more reasonable to me; in particular the timer+GPIO approach would be a stretch to make work on a 48MHz SAMD21, but should be fine using a SERCOM on that part.  I'm thinking that the appropriate thing might be to move the PyGamer neopixel examples away from the timer-based crate and to the SPI-based one.

Debugging the SPI-driven WS2812 examples revealed that there are some subtle changes in behaviour of SPI writes between the old and new HAL.  The old behaviour doesn't work reliably for me in release profile (the pixels show the rainbow, but flicker quite a bit), and the new behaviour doesn't work at all for WS2812 despite behaving correctly according to the SpiBus trait AFAICT.
So, at present, I think that the neopixel examples are broken for reasons outside of atsamd-rs.  I've opened #752 to track that, since it's not specific to PyGamer although some of the issues are worse with a string of multiple neopixels.  The question is whether we want to get those examples working before moving forward with this PR.
My vote is to remove the neopixel examples from PyGamer until we can get them working, and proceed with this PR.


[FR] Can the Text Editor auto-expand?

Right now it only shows 23 lines max.

By design.

No other online IDE tool has this limitation.  Coliru, repl.it, godbolt,
cppinsights, hastebin, pastebin, you name it, they don't have this viewable
lines limitation in their text editor.  Why is this limitation "by design"?
On Fri, Jul 5, 2019, 9:16 PM melpon<EMAIL_ADDRESS>wrote:

By design.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/melpon/wandbox/issues/278?email_source=notifications&email_token=AAWLJN7SULQ6VG3GR4WEXS3P6AMDXA5CNFSM4H6RY4N2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZKSDGA#issuecomment-508895640,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAWLJN7OHAICL4MQR2MLDRLP6AMDXANCNFSM4H6RY4NQ
.


Release 0.2-2

👋  would love a release to get new changes into the wild @yangxikun!
Diff: https://github.com/yangxikun/opentelemetry-lua/compare/v0.2.1...main

We can release after https://github.com/yangxikun/opentelemetry-lua/pull/53 merged.

Mentioned this on #53, but I think we're good to merge and release 😄

🎉  0.2-2

Thanks @yangxikun!


[Styles] - Improve Login page

The current login page, with its stark two-color design, becomes visually unappealing when incorrect credentials are entered. We need to enhance the design of this page.

Hi,
I’m a beginner contributor, but I’m excited to help improve this project. I’ve worked on creating custom authentication templates using Auth.js (formerly NextAuth.js)  too, and I believe I can contribute by improving the styling and enhancing the validation.
Thanks!

Hi,
I’m a beginner contributor, but I’m excited to help improve this project. I’ve worked on creating custom authentication templates using Auth.js (formerly NextAuth.js)  too, and I believe I can contribute by improving the styling and enhancing the validation.
Thanks!

Hey buddy,
Thanks for your support,  feel free to contribute 😀

Hi, I made my first pull request.


InfluxDB 2.0: User API - GetUserLog return Internal Server Error (500) instead Not Found (404)

The Internal Server Error (500) is returned when I trying retrieves a user log for a user that does not exist:
-> GET
-> http://<IP_ADDRESS>:9999/api/v2/users/020f755c3c082000/log

<- 500
<- Internal Server Error
<- {"code":"internal error","error":"oplog not found"}

I think that the GetUserOperationLog should return a ENotFound error.

The /log endpoint returns a list of logs:
{
    "links": {
        "self": "/api/v2/users/034560b68654b000/log"
    },
    "log": [
        {
            "links": {},
            "description": "User Created",
            "time": "2019-01-19T17:51:11.897732+08:00"
        }
    ]
}

I think there are two things to do:

check user existence, return a UserNotFound err if user not exist.
if no logs for the user, return an empty list of logs.

@desa How do you think?

It is already fixed.


How to deal with bind issues with default/standard ports - New Ports?

Is your feature request related to a problem? Please describe.

With so many ports and different set ups in use out there - and by looking at the number of related support issues raised - people are often faced with bind issues whenever I’m setting up containers that n want to use more of the default network setting.
Is there a way for NPM to better handle that..
If I simply try the quick start guide on my QNAP NAS, it instantly has an issue with port 443 (see below). - For a quick start guide if would be great if there was a very low likelihood of an error being encountered..
Failed to create application "npm". Error message: operateApp action …. Error response from daemon: driver failed programming external connectivity on endpoint npm-app-1 (18dae2dcc7eee4f9348dcc89824df962d8d62d60f1148ec79795521854a587fc): listen tcp4 <IP_ADDRESS>:443: bind: address already in use

Describe the solution you'd like
In situation where someone is only trying to access other docker containers, can NPM make use of its own ports and remove their emphasis on the defaults ? I appreciate it might require a new port exposed but it would be one dedicated and monitored by NPM?

Maybe I'm misreading your issue, but normally nginx/NPM should bind to the standard ports (80:80, 443:443), and your apps should be bound to non-standard ports (e.g. 10080:80, 10443:443, or as required by each app).
Do you know what is currently using :443, e.g. is it a QNAP web UI, or another container?

Running a quick check, the QNAP UI / control panel seems to suggest my TLS https web port is 8081, yet going into the command line and doing a look up, I can see there is something associated with 443, but I have no idea what it is - although the other apache_proxys entries are for the non TLS http web port of 8080..???
apache_proxys    7555   admin     IPv6  TCP     *:443
apache_proxys    8679   admin     IPv6  TCP     *:443
apache_proxys    28712  admin     IPv6  TCP     *:443


Failed to compile user code (System.Collections)

Getting an error when trying to get SourceGenerator to use an .nt.cs user code file. I'm referencing System.Collections.Generic for the use of List, but it doesn't seem to be finding it as I get the following error:
14>NTypewriterCustom.nt.cs(13,41): Error CS0012 : The type 'List<>' is defined in an assembly that is not referenced. You must add a reference to assembly 'System.Collections, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'.
The user code is a relatively straightforward workaround for handling multiple nested generic types:
using System;
using System.Linq;
using System.Collections.Generic;
using NTypewriter.CodeModel;
using NTypewriter.CodeModel.Functions;

namespace _
{
    class NTypewriterCustom
    {
        public static string MapTypeName(IType type)
        {
            List<IType> typeArguments = type.TypeArguments.ToList();
            switch (typeArguments.Count)
            {
                case 2:
                    if (type.Name == $"KeyValuePair<{typeArguments[0]}, {typeArguments[1]}>")
                    {
                        return $"{{key: {typeArguments[0]}, value: {typeArguments[1]}}}";
                    }
                    break;
                case 1:
                    if (type.IsEnumerable)
                    {
                        return $"{MapTypeName(typeArguments[0])}[]";
                    }
                    break;
            }
            return type.ToTypeScriptType();
        }
    }
}

Is there a way to designate the target assembly versions? My current thought is that it has something to do with the SDK Roslyn is targeting, since it seems to just be hitting the most recent version, as on my computer it's looking for .NET 8, but for other developers it's hitting .NET 6.
Edit:
Reading through #84 it seems like this issue is similar, since it only happens for me when running in Rider (no NTypewriter extension), but not in Visual Studio with the extension installed. For extensionless implementations, do all referenced assemblies need to be included in the SourceGenerator project?

#84 is not similar unfortunately. VS addin and source generator are using exactly the same runtime/code for compiling .nt.cs files. Theoretically the set of available assemblies should be the same, but environments are completely different VS is still on .net4.8, Rider probably on net.core. Your first thought was good, somehow source generator run by Rider is trying to do compilation targeting net.core, but all referenced assemblies are provided for net.standard, and I do not see any line that would tell compiler to explicitly target net.standard:
https://github.com/NeVeSpl/NTypewriter/blob/8c015cbd6ef1a93afb0b4799e3a4f97eeb61071d/NTypewriter.Runtime/UserCode/UserCodeLoader.cs#L157-L183

Yea that certainly makes sense, Rider does seem to be trying to compile using Core even though when peeking at the assembly explorer it recognizes the NTypewriter.SourceGenerator assembly as .NET Standard v2... To confirm your theory I uninstalled the extension from VS and it still compiled just fine, so it's just a Rider issue.

The problem may be in how assemblies are resolved for the compilation, in one of the future releases I will try to replace manual loading assemblies with this library:
https://github.com/jaredpar/basic-reference-assemblies
It looks promising, it should give more deterministic results, and hopefully the same results for VS/48 vs Rider/.core, but I wonder if it will work with Blazor .... but that is different story.

Another interesting tidbit - I ran the source generator code through CI, and jenkins also failed to build it. Seeing that, I figured I'd try running it through VS via the terminal dotnet build command and it also failed, so there must be something very specific about what VS does when it builds that's different than the usual dotnet build?

Problem is fixed in NTypewriter.SourceGenerator v0.3.2

Sweet I'll get it tested once the CI finishes. Cheers!

Updated to the latest nuget packages for .Runtime and .SourceGenerator and still having compiling issues with MSBuild version 17.8.3+195e7f5a3 for .NET, I'll poke around and run the source code locally and see if I can get something working.

That is impossible :D. The same error message? Did you delete obj and bin folders to make sure that something is not cached?
I was able to reproduce problem, and indeed there was difference between MSbuild for .NET vs .NET Framework, but using basic-reference-assemblies seems to resolved problem.

I ran a clean, but didn't manually delete bin and obj, I can try that too :)

I ran a clean, but didn't manually delete bin and obj, I can try that too :)

Yep same result after bin/obj delete and rebuild


[Feature Request] sync styles across Chrome instances

I use stylebot at home and at work. It's kind of annoying to have to manually sync my custom styles. It would be nice if the extension handled this.
One suggestion would be take advantage of the ability to sync extension storage. See here: http://developer.chrome.com/extensions/storage.html#property-sync

How are we doing on this feature?

Also looking for this

Any news on progress ankit? Really hoping this feature is implemented! It would really push stylebot to the next level!

It will be appreciated if you would do it

Please support the sync feature. This is the only extension I'm using needs to sync manually.
I really love this extension.

Still not supported? It is the only extension of all which I use which do not support chrome sync....Pls integrate this feature into this extension....

I will fix add sync capability in the v3 re-write, thank you.

Update - I researched support for sync in extensions and it actually has a stringent quota that won't work for stylebot (https://developer.chrome.com/extensions/storage#property-sync). As a result, this didn't make the v3 release.
However, I do plan to figure out an approach and implement this in a future release, since it would be really useful.

@ankit that would be great, looking forward to see it in the plugin!

Has there been any movement on this? I'd be happy to contribute.

Has there been any movement on this? I'd be happy to contribute.

The first iteration on this will be arriving in the next release. It will allow syncing across multiple instances of Chrome/Firefox via Google Drive OAuth (this can later be extended to other services such as github as well).
The other caveat is that it will be manual - you will have to click on "Sync Now" via the popup (it will take quite a bit more work to enable auto-sync by building a server that uses OAuth in offline mode with refresh tokens to auto-generate the access token). Seems to work well even with this limitation.

Merged. will be included in v3.1

Update - I researched support for sync in extensions and it actually has a stringent quota that won't work for stylebot (developer.chrome.com/extensions/storage#property-sync). As a result, this didn't make the v3 release.
However, I do plan to figure out an approach and implement this in a future release, since it would be really useful.

What part of that quota is a limiting factor for stylebot? The numbers seemed large enough for my usage, and having the automatic sync would be much more convenient than the manual (button-clicking) method.


Google doc

Reference Issues/PRs
What does this implement/fix? Explain your changes.

Fix handling of Google Docs exported file in .docx format(which doesn't have core.xml and app.xml)
add entrypoint in setup.py

Any other comments?

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


[EXTERNAL] can't copy cell outputs

you can select the outputs, but ctrl-c is overridden to copy the cell contents. also the right-click menu is also overridden to cell-specific actins, even when you click on the outputs.

I assume this is in VS Code?
What version of the .NET Interactive Notebooks extension are you using?
I'm not able to repro it using this version:

@jonsequitur I'm also expected to get a "Copy Cell Outputs" command. For now, there is only "Clear Cell Outputs".

@Spongman This was fixed a few months back in VS Code. It's not something that the .NET Interactive Notebook extension or .NET Interactive kernel controls, so I marked it as external.
@doggy8088 This menu is also provided by VS Code, not by .NET Interactive. You can copy the cell outputs by highlighting the output and pressing Ctrl+C. I haven't noticed a context menu item for copying them. I would suggest opening an issue at https://github.com/Microsoft/vscode if you think they should add it.


Child components can't setState in componentDidMount

A child component wrapped with hoc can't invoke setState in componentDidMount because this.mounted is false.
I have a codesandbox demonstrating this, an action is invoked in componentDidMount, but the setState will not execute.
https://codesandbox.io/s/9zonjjq3k4
If you have any ideas, I'm up for doing a pr. I'm not exactly sure why the this.mounted check is required
Great lib though :)

well it is supposed to guard against async setState firing on a dead/unmounted component. PRs welcome of course!

@drcmda created a PR for this, setting an unmounted flag instead of a mounted flag, which isn't required for setState. I've done unmounted to avoid setState errors a few times before

@drcmda working wonderfully after the patch, thanks!


balena-image-initramfs: remove the migrate and recovery modules

These new meta-balena functionality grows the initramfs image and it no longer fits in the allocated root filesystem partition.
Removing the recovery and migrate modules from this device type only means that the new functionality won't be available for this device type.
Changelog-entry: remove migrate and recovery initramfs modules from nitrogen8mm-dwe

lgtm


Update Write / Readinto logic in I2C

Update the I2C logic replacing a write() followed by Readinto() with write_then_readinto(), removing the obsolete argument stop in the writes.

Also change busio.I2C(Board.SCL, Board.SDA) to board.I2C() and change exit() to sys.exit()

Updated code to fix issues as part of build update.


bug(mat-slide-toggle): disableRipple not causing any visual changes

Is this a regression?

[X] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
It looks like disableRipple was used to prevent the ripple animation (ripple based on where in the touch target you clicked) in the legacy components, but now that doesn't seem to show up anymore. So it seems that setting disableRipple isn't causing any difference now.
Reproduction
StackBlitz link: https://stackblitz.com/edit/aakbph?file=src%2Fexample%2Fslide-toggle-overview-example.html
Expected Behavior
Setting disableRipple to true would cause a visual difference upon clicking the slide toggle handle.
Actual Behavior
No viusal change
Environment

Angular: 17.2.0-rc.0
CDK/Material: 17.2.0-rc.0
Browser(s): Chrome
Operating System (e.g. Windows, macOS, Ubuntu):  macOS

Any more update on this? disableRipple doesn't work on sliders either.
I deal with it using css for now like so
.mat-mdc-slider {
  --mat-mdc-slider-hover-ripple-color: unset; // 'none' also works
  --mat-mdc-slider-focus-ripple-color: unset;
}


change directory before running git command

Issue

#734

stay the issue open because we have to test if the change resolve it after this PR is merged

Overview (Required)
The cause of error reported in #734 is running git command out of git repository. This occurred because current directory  when the step Delete old branch saving screenshots is executed is $GITHUB_WORKSPACE although the checked-out repository is located on $GITHUB_WORKSPACE/temp_docs.
The step worked fine before workflow was split in #723 because the base workflow (ScreenShotTest) also checked-out repository on $GITHUB_WORKSPACE to run screenshot tests.
This PR added directory change at the beginning of commands executed in Delete old branch saving screenshots. The PR also tried to make destinations of directory-change operations clear with the default environment variable $GITHUB_WORKSPACE.
Links
N/A
Screenshot
N/A

Great!

now I'll open a test pull request 🏃


IOU - Approved report disappears from LHN

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.65-0
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:

Login as the owner of the workspace.
Create a workspace
3.Invite the approver and employee
Navigate to more features
Enable "workflows"
On the "Workflow" editor - enable "Add Approvals"
Set the Approver account as the Approver

Steps:

As employee create two expenses in workspace chat
As approver approve the report
As approver open other chat

Expected Result:
Approved report should remain in LHN
Actual Result:
Approved report disappears from LHN
Workaround:
Unknown
Platforms:

[x] Android: Standalone
[x] Android: HybridApp
[x] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/2019239d-bee8-4fcb-8f58-64a0ca1dbaf8

View all open jobs on GitHub

This doesn't look like a bug to me.

hmm fairly low priority - closing


Resolve Hover Override Effect in Destination Guide Section #1109

Related Issue
“None”
Fixes:  ** #1109 (Hover Override Issue Resolved in Destination Guide Section.) **
Description
Issue Description:
The Destination Guide section had a user experience problem where hovering over the section caused it to override and occupy upper space, negatively impacting the layout and readability of the page.

"1109".
Type of PR

[ ] Bug fix

Screenshots

Checklist:

[X] I have made this change from my own.
[X] I have taken help from some online resources.
[X] My code follows the style guidelines of this project.
[X] I have performed a self-review of my own code.
[X] I have commented my code, particularly in hard-to-understand areas.
[X] I have made corresponding changes to the documentation.
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers and screenshots after making the changes.

@KAJAL7764 Text is not visible .This one is better in website

Ensure you pull all the latest changes before pushing your feature/bug fixes to the main branch. After resolving the conflicts, please open a new PR for review.
Thank you!


HARP-8385: Fix MapControls support for flexible layouts.

Whenever MapView canvas is placed with offset from the browser edges
the MapControls calculated the mouse position and NDC cords incorrectly.
This patch-set allows to place canvas anywhere, thus gives support for
canvas translation (offset) and additionally provides support for scaling.
Signed-off-by: Krystian Kostecki<EMAIL_ADDRESS>Thank you for contributing to harp.gl!
Before requesting a pull request, please remember to check the following documents:

contribution guidelines
coding style

If you are adding new functionality we would highly appreciate if you can describe what is the capability you are adding and even better if you can add some examples. Please also remember to add tests for it.
CI Check
Our bots will check whether your PR can be directly integrated into the mainline. We have some internal integration tests running on the background, our bots will inform you of the next steps and someone from our team will take a look and help if needed!
And please do not forget to sign-off your commit! You can read more about DCO here. But, in short, you just need to use git commit -s or append --signoff when you are committing to the repo.
Happy contributing!

Codecov Report

Merging #1311 into master will increase coverage by 0.29%.
The diff coverage is 14.28%.

@@            Coverage Diff             @@
##           master    #1311      +/-   ##
==========================================
+ Coverage   53.61%   53.91%   +0.29%
==========================================
  Files         263      264       +1
  Lines       23544    23647     +103
  Branches     5227     5268      +41
==========================================
+ Hits        12624    12749     +125
+ Misses      10920    10898      -22

Impacted Files
Coverage Δ

@here/harp-map-controls/lib/MapControls.ts
23.59% <14.28%> (-0.19%)
:arrow_down:

...tasource-protocol/lib/operators/ObjectOperators.ts
92.59% <0%> (-1.16%)
:arrow_down:

@here/harp-materials/lib/MapMeshMaterials.ts
38.95% <0%> (-0.73%)
:arrow_down:

@here/harp-datasource-protocol/lib/Expr.ts
91.88% <0%> (-0.05%)
:arrow_down:

@here/harp-materials/lib/SolidLineMaterial.ts
48.03% <0%> (ø)
:arrow_up:

@here/harp-materials/lib/EdgeMaterial.ts
11.39% <0%> (ø)
:arrow_up:

...p-mapview/lib/geometry/PhasedTileGeometryLoader.ts
4.34% <0%> (ø)
:arrow_up:

@here/harp-materials/lib/DisplacementFeature.ts
50% <0%> (ø)

...re/harp-mapview/lib/geometry/overlayOnElevation.ts
46.8% <0%> (+1.15%)
:arrow_up:

...e/harp-datasource-protocol/lib/ExprInstantiator.ts
74.39% <0%> (+2.43%)
:arrow_up:

... and 5 more

Continue to review full report at Codecov.

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

retest this please

retest this please

retest this please

retest this please


[HOLD for payment 2024-12-11] [$250] Android & iOS - Start chat - Add to group button is top-aligned instead of centered

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.68-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:

Launch ND or hybrid app
Open FAB > Start chat

Expected Result:
"Add to group" button will be centered on the right (web and mweb behavior)
Actual Result:
"Add to group" button is top-aligned on the right
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Standalone
[x] Android: HybridApp
[x] Android: mWeb Chrome
[x] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/738cb03d-873b-40e1-98f4-21e1431ab3e3
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021862203018243480384
Upwork Job ID:<PHONE_NUMBER>243480384
Last Price Increase: 2024-11-28
Automatic offers:

neonbhai | Contributor | 105126607

Issue OwnerCurrent Issue Owner: @zanyrenney

adding external

cc @Expensify/design

hey @fedirjh please can you review the proposal above?

Looks like a bug worth solving. Thanks for highlighting 👍

@fedirjh please complete the checklist so i can pay this out. Payout will only be due once the checklist has been posted in here. Thank you!

Thanks @fedirjh

I have bumped @neonbhai to accept the Upwork job because it is still pending!

payment summary is:
@fedirjh requires payment through NewDot Manual Requests - please request $250
@neonbhai requires payment automatic offer (Contributor) - please accept Upwork offer

Updated the payment summary above, closing.


Desktop Swipe support

Hi! Thanks for this component!
I'm currently using another carousel component on a React project but having a few issues on iOS. I just tried this one and it works GREAT! I just noticed that the swipe is limited to touch devices only. Do you have any plans to add swipe support on desktop too?
Tks!

@rafaelbiten this is not a scope of this component, check - https://github.com/thebird/swipe.


PRE MADETHEMES!!!

You're able to select premade themes for colours and then save themes to your local storage and then share themes!!!!!

added tm


(fix) Set status::Ok on expected routes in simple_routing example.

The two "hello" routes by default will be set to status::NotFound, but the routes return data like a typical request so status::Ok seems more appropriate.

Thanks!


Write tests for gRPC server

Server is here:
https://github.com/google/startup-os/tree/master/tools/reviewer/local_server

Done here: https://github.com/google/startup-os/commit/4cc3b0255e55dee7974fbab8d3630661ac92c6ae


✨ [Feature request]: Create argument pasrser

Is your feature request related to a problem? Please describe.
rpdk will be having multiple entry point, to name them

Create play
Update play
Push changes
etc

We need a robust argument parser to manage this
Describe the solution you'd like
Create a command-line argument parser
Describe alternatives you've considered
NA
Additional context
NA

Wrong repo.
Corrct one  https://github.com/reactplay/react-play-dev-kit/issues/5


Sorting

I think I asked about this before, but I can't find any emails, and I didn't file as issue here.
Is there a means to sort the output based on a specified field?

If you are using raw query you could sort based on query:
es2csv -q '{"sort" : [{"key":"desc"}], "query" : {"term" : { "tags" : "qa" }}}' --raw --debug -e -o out.csv -f key tags
Using these indices: logstash-*.
Query[Query DSL]: {"sort": [{"key": "desc"}], "query": {"term": {"tags": "qa"}}}.
Output field(s): key, tags.
Found 5 results.

Looks like it could be implemented with --sort arg.

sort – A comma-separated list of field:direction pairs

Added --sort(S) in 5.5.2.


Fix Portable PDB documentation in symbols.md

The documentation listed an incorrect reason for why source indexing isn't supported for portable PDBs. This removes this. 'Source Link' is also two words, so I fixed that as well.

The content has been reconfigured since the pull request was opened and the revised text strings are not currently in the article. The pull request can't be merged in, so we'll need to close it. If the proposed update still needs to be added to the article, please submit it in a new pull request. Thanks.

https://github.com/MicrosoftDocs/azure-devops-docs/pull/11283 opened.


Regarding newly added "UI option to block deleted rooms from being rejoined" ON by default

As far as I understand it, this option #12 makes use of the Delete Room API in combination with the off-by-default block parameter, which states:

block - Optional. If set to true, this room will be added to a blocking list, preventing future attempts to join the room. Rooms can be blocked even if they're not yet known to the homeserver (only with Version 1 of the API). Defaults to false.

A common synapse-admin cleanup task is the need to purge empty (forgotten or faulty) rooms from the database. I often group rooms by members and delete forgotten rooms all in one go. But what I do not necessarily expect is to also put those rooms on a local blocking list effectively preventing my users from being able to (re-) join them ever again.
While this still is a very useful option (for old tomb-stoned rooms, or rooms with content violation) I think this should not be the default choice. It should default to OFF, and display a clarification stating that, by turning this ON - those rooms will be added to a local block list  effectively preventing users from (re-)joining them in the future and that this action is not going to be easily reversible as synapse-admin does not (yet) have an option to query and revert blocked rooms.
Thanks!

Thanks for the context!
Indeed, blocking by default doesn't seem right, fixed


Including downstream validation sets in pretraining dataset

Many thanks for your work.
I was checking data_utils_abdomen.py and I realized that the validation set of some datasets like BTCV, amos, ... (that are later used for finetuning) is being used while pretraining.
Could you please clarify?
(table1 of the paper says that only the training set of BTCV is used, which has size 24)

Dear hadi, as described in Table2 caption, the labels of val sets are unseen in pre-training. In most related SSL works, we find that they did not strictly exclude the val sets in pre-training. This is because in most cases, for datasets without available test sets, we have to adopt 5-fold validation. In this case, you cannot exlude val data in pre-training.

Please allow me to add a question. Do you use nnUNet's default 5-fold cross-validation? Or the one provided by the dataset? For example, BraTS21.

Please allow me to add a question. Do you use nnUNet's default 5-fold cross-validation? Or the one provided by the dataset? For example, BraTS21.
For most of the experiments, we use the splits the same as nnUNet. But for BraTS21, we use the one provided by the dataset, as shown in brats21_folds.json

Dear hadi, We did not report the 5-fold results in the paper since there are many datasets and compared methods, which will make the tables too bulky and confusing.
For most of the datasets, there are pre-defined splits for validation (e.g., word, brats, ct-rate .....) and we adopt the same settings for fair comparisons. For some datasets with test leaderboards (e.g., amos, flare23, kits ....), we report the results on test leaderboard (e.g., https://codalab.lisn.upsaclay.fr/competitions/12239#results). For other datasets, we set the same fold as nnunet for all the compared methods for fair comparisons.


fusion_mode and fusion_caller with Oncofuse

Currently it is possible to not set fusion_mode (which then equals false) but enable Oncofuse via fusion_caller. As a result, STAR is incorrectly configured for Oncofuse and no chimeric reads are generated; Oncofuse then exits since the chimeric read file isn't present (it seems to be only present if at least one chimeric read has been found).
Perhaps bcbio could notify the user about this inconsistency, and also deal with the case of the chimeric output file not being there (which could also happen when STAR is correctly in fusion_mode, although it's unlikely to happen in practise)?

Sven-Eric -- thanks for this report. I dug into this in the context of this debug thread on biovalidation (https://groups.google.com/d/msgid/biovalidation/72b8219b-471c-4811-8b81-0fd50d24c456%40googlegroups.com) and it looks like 1.0.7 and current development correctly handles the fusion_caller only case (https://github.com/chapmanb/bcbio-nextgen/commit/597eaf53b078d92716573bb4477864fd2a6a98ef) but things could still error out if not chimeric reads are found. I added a check for this (https://github.com/chapmanb/bcbio-nextgen/commit/174a15fc115abb1546421ae533a05ddf595c65dc) which I believe will resolve the issue. Please let us know if you find it skipping chimeric read detection and happy to dig further. Thanks again.

Excellent.


how many epochs you get the result as the paper result?

This implementation was developed to learn how to use Chainer BiLSTMs.
I did not conduct experiments to compare with the reported results.


⚠ Bofrost.be has degraded performance

In fed2bbb, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1406 ms

Resolved: Bofrost.be performance has improved in d36913d.


ARROW-4871: [Java/Flight] Handle large Flight messages

Equivalent to https://issues.apache.org/jira/browse/ARROW-4421, but for Java.

Codecov Report

Merging #3898 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3898      +/-   ##
==========================================
+ Coverage   87.85%   87.85%   +<.01%
==========================================
  Files         726      726
  Lines       89340    89337       -3
  Branches     1252     1252
==========================================
  Hits        78489    78489
- Misses      10733    10734       +1
+ Partials      118      114       -4

Impacted Files
Coverage Δ

go/arrow/math/uint64_sse4_amd64.go
0% <0%> (-100%)
:arrow_down:

go/arrow/math/float64_sse4_amd64.go
0% <0%> (-100%)
:arrow_down:

go/arrow/memory/memory_sse4_amd64.go
0% <0%> (-100%)
:arrow_down:

go/arrow/math/int64_sse4_amd64.go
0% <0%> (-100%)
:arrow_down:

js/src/ipc/metadata/json.ts
92.39% <0%> (-4.35%)
:arrow_down:

js/src/ipc/metadata/message.ts
92.99% <0%> (-1.28%)
:arrow_down:

js/src/schema.ts
88.77% <0%> (-1.03%)
:arrow_down:

cpp/src/arrow/util/bpacking.h
99.81% <0%> (-0.01%)
:arrow_down:

go/arrow/math/float64_amd64.go
33.33% <0%> (ø)
:arrow_up:

go/arrow/math/int64_amd64.go
33.33% <0%> (ø)
:arrow_up:

... and 8 more

Continue to review full report at Codecov.

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

IIUC we aren't running the Flight integration tests in Travis CI, do you want to try to enable that?
https://github.com/apache/arrow/blob/master/ci/travis_script_integration.sh

I'll give it a spin. We need a way to mark integration tests as XFAIL then as Flight can't handle dictionaries yet.

@lihalite let's do the integration testing in a different patch? Don't want to hold this up

@wesm sure, no problem. I got pulled into some other stuff today, if you want to move the constants, I'll try to get a patch out for that tonight or tomorrow.

I've moved the constants. I'll follow up with the integration test changes later.


Image draw buffer

As discussed in #8473, we have a clipping effect when panning on image VectorLayer. It's because we draw images with viewport size by default.
To remove this clipping effect, I created a new parameter for VectorLayer to specify a buffer to extend the viewport size drawing.
Please note that this parameter will need to increase renderBuffer (if smaller) to retrieve buffer area features. Obviously, it has an impact on performances as the new drawing zone is bigger than the default one. So the default parameter is no buffer, as today.

I still have this error on Firefox, even when I pull your latest version on master
In my local setup, latest version of master:
λ npm test -- --browsers Firefox

><EMAIL_ADDRESS>pretest C:\Users\bvigie\dev\teslaule\openlayers
> npm run lint

><EMAIL_ADDRESS>lint C:\Users\bvigie\dev\teslaule\openlayers
> eslint tasks test src/ol examples config

><EMAIL_ADDRESS>test C:\Users\bvigie\dev\teslaule\openlayers
> npm run karma -- --single-run "--browsers" "Firefox"

><EMAIL_ADDRESS>karma C:\Users\bvigie\dev\teslaule\openlayers
> karma start test/karma.config.js "--single-run" "--browsers" "Firefox"

i ｢wdm｣: Hash: 9d2c943b68425fd58dd0
Version: webpack 4.19.0
Time: 187ms
Built at: 2018-09-17 15:04:39
i ｢wdm｣: Compiled successfully.
i ｢wdm｣: Compiling...
i ｢wdm｣: Hash: c4f3d9b8e4462b11997d
Version: webpack 4.19.0
Time: 8693ms
Built at: 2018-09-17 15:04:48
             Asset      Size              Chunks             Chunk Names
     index_test.js  14.9 MiB       index_test.js  [emitted]  index_test.js
  module-global.js  8.86 KiB    module-global.js  [emitted]  module-global.js
test-extensions.js   123 KiB  test-extensions.js  [emitted]  test-extensions.js
Entrypoint module-global.js = module-global.js
Entrypoint test-extensions.js = test-extensions.js
Entrypoint index_test.js = index_test.js
[./src/ol/array.js] 6.29 KiB {test-extensions.js} {index_test.js} [built]
[./src/ol/has.js] 1.93 KiB {test-extensions.js} {index_test.js} [built]
[./test sync recursive \.test\.js$] ./test sync \.test\.js$ 7.91 KiB {index_test.js} [built]
[./test/index_test.js] 208 bytes {index_test.js} [built]
[./test/module-global.js] 21 bytes {module-global.js} [built]
[./test/spec/ol/tilecache.test.js] 1010 bytes {index_test.js} [optional] [built]
[./test/spec/ol/tilecoord.test.js] 4.06 KiB {index_test.js} [optional] [built]
[./test/spec/ol/tilegrid/tilegrid.test.js] 34.8 KiB {index_test.js} [optional] [built]
[./test/spec/ol/tilegrid/wmts.test.js] 7.45 KiB {index_test.js} [optional] [built]
[./test/spec/ol/tilequeue.test.js] 5.33 KiB {index_test.js} [optional] [built]
[./test/spec/ol/tilerange.test.js] 4.25 KiB {index_test.js} [optional] [built]
[./test/spec/ol/tileurlfunction.test.js] 3.61 KiB {index_test.js} [optional] [built]
[./test/spec/ol/transform.test.js] 3.81 KiB {index_test.js} [optional] [built]
[./test/spec/ol/uri.test.js] 2.2 KiB {index_test.js} [optional] [built]
[./test/test-extensions.js] 17.2 KiB {test-extensions.js} [built]
    + 539 hidden modules
i ｢wdm｣: Compiled successfully.
17 09 2018 15:04:49.670:INFO [karma]: Karma v3.0.0 server started at http://<IP_ADDRESS>:9876/
17 09 2018 15:04:49.672:INFO [launcher]: Launching browser Firefox with unlimited concurrency
17 09 2018 15:04:49.734:INFO [launcher]: Starting browser Firefox
17 09 2018 15:04:51.650:INFO [Firefox 62.0.0 (Windows 10 0.0.0)]: Connected on socket I_5V9BmeqU_WR67CAAAA with id 3004079
Firefox 62.0.0 (Windows 10 0.0.0) ol.rendering.style.Text #render Text along an ugly upside down path, keep text upright renders text along a linestring with `textAlign: 'center'` FAILED
        Error: expected 3.62 to be below 3.6 (C:/Users/bvigie/dev/teslaule/openlayers/node_modules/expect.js/index.js:96)
Firefox 62.0.0 (Windows 10 0.0.0): Executed 1427 of 2991 (1 FAILED) (skipped 29) (0 secs / NaN secs)
Firefox 62.0.0 (Windows 10 0.0.0): Executed 2333 of 2991 (1 FAILED) (skipped 29) (0 secs / NaN secs)
Firefox 62.0.0 (Windows 10 0.0.0): Executed 2376 of 2991 (1 FAILED) (skipped 29) (0 secs / NaN secs)
Firefox 62.0.0 (Windows 10 0.0.0): Executed 2834 of 2991 (1 FAILED) (skipped 29) (0 secs / NaN secs)
Firefox 62.0.0 (Windows 10 0.0.0): Executed 2841 of 2991 (1 FAILED) (skipped 29) (0 secs / NaN secs)
Firefox 62.0.0 (Windows 10 0.0.0): Executed 2962 of 2991 (1 FAILED) (skipped 29) (28.46 secs / NaN secs)
TOTAL: 1 FAILED, 2961 SUCCESS

Hi @ahocevar ! I made some tests, and indeed it has a big impact on the hitDetection (and all other interaction operations).
I updated it to let the renderBuffer untouched. I just update the frameState.extent if an imageDrawBuffer has been given 😃

Any update on this ?


[Docs] Add 6 models in benchmark & model zoon in README.md

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Add 6 models in benchmark & model zoon in README.md
Modification
Update models in README.md, README_zh-CN.md, docs/en/model_zoo.md, docs/zh_cn/model_zoo.md.
BC-breaking (Optional)
Does the modification introduce changes that break the back-compatibility of the downstream repos?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here, and update the documentation.
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects.
The documentation has been modified accordingly, like docstring or example tutorials.

@zhulf0804 Please fix lint. https://github.com/open-mmlab/mmdetection3d/blob/main/docs/en/notes/contribution_guides.md

Hello @JingweiZhang12, lint is fixed. Please check it.


Improved custom enum support

Opening an issue from https://discord.com/channels/707636530424053791/1123687185837740053/1290092449183698965 - me and @helderco had a chat about this in person at our offsite.
Current enum support has a few issues, which we can fix!

Naming clashes are very possible in go

Enum values aren't namespaced at all.
Fixed by https://github.com/dagger/dagger/pull/8669

Some core enum types have stuttering names

e.g. TypeDefKind.STRING_KIND should be TypeDefKind.STRING, ModuleSourceKind.GIT_SOURCE should be ModuleSourceKind.GIT

We confuse enum names and values, e.g. in the following go declaration: const MyEnumValue MyEnum = "foobar", the graphql enumValues.name is populated as foobar - this is weird, because everything else is parsed by identifier (functions, args, structs, etc) - we should find the name as value instead.

This is caused by the fact that graphql doesn't have separate names/values for enums, so we picked values.
However, with #8498 incoming, we can add custom directives :smile:
So, we can switch the enumValues name to be derived from the name - and use a custom @value directive for the value.
Clarify the internal naming here - an enum "member" contains both a value and a name.
This does a few things:

The name generation is now internally consistent
Names are correctly shared between different modules
Values don't need to conform to the graphql naming format (which only allows very specific formats)
Enum values could be of other types, not just strings!

Enum value names should be cleaned up to use SCREAMING_SNAKE_CASE in the graphql schema for consistency. This will help to avoid issues like #8682 in the future.

 They aren't just badly supported by the language, there are also competing practices out there with varying UX.
Here is how I define enums these days in Dagger modules:
// module: trivy
type ReportFormat string

const (
	ReportFormatTable      ReportFormat = "table"
	ReportFormatJSON       ReportFormat = "json"
	ReportFormatTemplate   ReportFormat = "template"
	ReportFormatSARIF      ReportFormat = "sarif"
	ReportFormatCycloneDX  ReportFormat = "cyclonedx"
	ReportFormatSPDX       ReportFormat = "spdx"
	ReportFormatSPDXJSON   ReportFormat = "spdx_json"
	ReportFormatGitHub     ReportFormat = "github"
	ReportFormatCosignVuln ReportFormat = "cosign_vuln"
)

I would expect to be able to use a similar version of this on the other side: probably dagger.TrivyReportFormatMarkdown?
I chose to prefix the enum values manually because there might be an overlap between different kinds of enums.

I chose to prefix the enum values manually because there might be an overlap between different kinds of enums.

Yup, allowing prefixing sounds good to me, we can do this without too much trouble, I'll add this to the list of things to do.

In terms of picking the enum values, with the third point in my list, that would completely detach names from values, so you're free to pick the values however you like - even including non-name characters like {[#@, etc.

I chose to prefix the enum values manually because there might be an overlap between different kinds of enums.

Yup, allowing prefixing sounds good to me, we can do this without too much trouble, I'll add this to the list of things to do.

What I meant here was actually not doing any kind of magic in the code generator, but encouraging users to prefix their enums themselves. That was they can account for any collisions within the module and the generated code will stay relatively close to what's in the module.

In terms of picking the enum values, with the third point in my list, that would completely detach names from values, so you're free to pick the values however you like - even including non-name characters like {[#@, etc.

I think that's a good idea.

What I meant here was actually not doing any kind of magic in the code generator, but encouraging users to prefix their enums themselves. That was they can account for any collisions within the module and the generated code will stay relatively close to what's in the module.

Sadly, I think some simple level of magic is needed - e.g. const MyEnumValue MyEnum = "value" should appear as MyEnum.Value in the graphql API. Without that, the python/typescript/etc will have very weird stuttery names like MyEnum.MyEnumValue. Go's enums kinda suck, but we can at least keep them in their own pool of sadness and not spread it into the other SDKs.

This covers pretty much everything I want from ENUMS. It will be a big breaking change or folks using ENUMS today though. But I think this is necessary.

What I meant here was actually not doing any kind of magic in the code generator, but encouraging users to prefix their enums themselves. That was they can account for any collisions within the module and the generated code will stay relatively close to what's in the module.

Sadly, I think some simple level of magic is needed - e.g. const MyEnumValue MyEnum = "value" should appear as MyEnum.Value in the graphql API. Without that, the python/typescript/etc will have very weird stuttery names like MyEnum.MyEnumValue. Go's enums kinda suck, but we can at least keep them in their own pool of sadness and not spread it into the other SDKs.

A couple ideas:

as a convention, Go enums MUST have the type name as a prefix, so you know what to strip from their names
use pragmas to redefine the name (needs validation though to make sure there are no collisions)
come up with an alternative enum syntax for Go (less idiomatic for Go developers, but it's probably closer to python's and typescript's enum semantic)

The challenge here is module developers may have to prefix values manually, to avoid collisions within a module, but that may result in equally stuttery names if Dagger also does some magic prefixing.
Sticking with the above example:
// module: trivy
type ReportFormat string

const (
	ReportFormatTable      ReportFormat = "table"
	ReportFormatJSON       ReportFormat = "json"
)

If it becomes dagger.TrivyReportFormatReportFormatTable, it's probably not ideal.
I would probably go with option 1 from the above list for now. That limitation can always be lifted later, if necessary.

Just adding a use-case, In PHP there are currently three kinds of enum:

unit enum: names without values
string-backed enum: names with string values
integer-backed enum: names with integer values

If the PHP SDK implements enum support using values, we can only support 2.
If I can rely on names, all enums can be supported in a consistent manner.

These aren't all going to make it into v0.14.0 - moving out of the milestone.


ciao-cli incorrectly displays network node as compute node

My network node has:
/var/lib/ciao/local/uuid-storage/role/client/0x10
which displays:
0b11a5c6-de3a-4279-a935-dd0934ab06f3
Role 0x10 is listed as NETAGENT.  The AGENT role is 0x4.
When I do a compute node listing:
ciao-cli node list --compute
I'm presented with my compute and network nodes:
Compute Node 1
	UUID: 0b11a5c6-de3a-4279-a935-dd0934ab06f3
	Status: READY
	Load: 0
	Available/Total memory: 31805/32128 MB
	Available/Total disk: 17865/21926 MB
	Total Instances: 0
		Running Instances: 0
		Pending Instances: 0
		Paused Instances: 0
Compute Node 2
	UUID: 17377153-1512-4acf-b3f3-c09305797cf9
	Status: READY
	Load: 0
	Available/Total memory: 31875/32128 MB
	Available/Total disk: 14940/21926 MB
	Total Instances: 0
		Running Instances: 0
		Pending Instances: 0
		Paused Instances: 0
Compute Node 3
	UUID: 810d4aa0-3eff-435e-91c9-afbb20e6684c
	Status: READY
	Load: 0
	Available/Total memory: 31877/32128 MB
	Available/Total disk: 14842/21926 MB
	Total Instances: 0
		Running Instances: 0
		Pending Instances: 0
		Paused Instances: 0
Compute Node 4
	UUID: af03829f-32d2-4af1-b6bc-8a7a27f7d47a
	Status: READY
	Load: 0
	Available/Total memory: 31659/32128 MB
	Available/Total disk: 13913/20372 MB
	Total Instances: 0
		Running Instances: 0
		Pending Instances: 0
		Paused Instances: 0

My compute nodes all have:
/var/lib/ciao/local/uuid-storage/role/client/0x4
And they contain the uuid's:
810d4aa0-3eff-435e-91c9-afbb20e6684c
17377153-1512-4acf-b3f3-c09305797cf9
af03829f-32d2-4af1-b6bc-8a7a27f7d47a

This issue relates to https://github.com/01org/ciao/issues/623 which is marked as a feature add to allow showing just network nodes, but is distinct in that this issue is a bug because my network node is reported as a compute node.

https://github.com/01org/ciao/pull/1297 should have fixed this

https://github.com/01org/ciao/pull/1297 should have fixed this


Failure to find ndarray s! macro

RA reports the error "Fail to find macro definition rust-analyzer (macro-error)" when I attempt to use the s! macro from the ndarray crate. Other macros from this crate and other crates work fine.
Steps to reproduce
Create a new crate, add ndarray as a dependency, add the statement ndarray::s![..]; to a function in lib.rs and observe the error. Add the statement ndarray::array[1]; and observe it works fine.

Minimal reproducible example :
macro_rules! s(
    ($crate) => {};
    () => {0};
);

fn main() {
    let _a = s!();       // <-  Fail to find macro definition
}

Minimal reproducible example :
macro_rules! s(
    ($crate) => {};
    () => {0};
);

fn main() {
    let _a = s!();       // <-  Fail to find macro definition
}

Thank you @edwin0cheng for the fix! :+1:

Thank you @edwin0cheng for the fix! :+1:


How to bind class static const variable to Lua?

Hi, I have a Vector3 class like this:
class Vector3
{
    Vector3();
    Vector3(const Vector3& vector);
    Vector3(float x, float y, float z);
    ~Vector3();

    void Normalize();
    float Length() const;

    float x_;
    float y_;
    float z_;

    static const Vector3 ZERO;
    static const Vector3 LEFT;
    static const Vector3 RIGHT;
    static const Vector3 UP;
    static const Vector3 DOWN;
    static const Vector3 FORWARD;
    static const Vector3 BACK;
    static const Vector3 ONE;
};

I write following code to binding it to Lua:
sol::constructors <
    sol::types<>,
    sol::types<const Vector3&>,
    sol::types <float, float, float>> constructors;

sol::usertype<Vector3> vector3(
    sol::call_constructor, constructors,

    "Normalize", &Vector3::Normalize,

    "Length", &Vector3::Length,

    "x", &Vector3::x_,
    "y", &Vector3::y_,
    "z", &Vector3::z_
    );

lua.set_usertype("Vector3", vector3);

I want to know how to bind static const Vector3 ZERO; to Lua?

sol::table v3table = lua["Vector3"];
v3table.set("ZERO", Vector3::ZERO, "ONE", Vector3::ONE);

etc. etc.
I am not sure I want to support binding random variables to some name on a usertype table.... but, then again, there's probably no reason not to. Use the above fix while I figure out how (im)possible this might be.

Thank you!

Does this work now with user_types?


Update ABIs to match deployed contracts and fix breaking changes

Description
Update ABIs to match deployed contracts and fix breaking changes
Changes

updated abis
updated fleetconfig interface
updated graph client

Summary by CodeRabbit

New Features

Introduced various new functions across multiple contracts, enhancing the functionality and accessibility of contract data.

Bug Fixes

Removed deprecated functions such as guardian, mockGovernor, and others to streamline contract interfaces and improve usability.

Documentation

Updated compiler versions and associated metadata across multiple contracts to reflect the latest changes and ensure compatibility.

Chores

Cleaned up ABI structures by removing unnecessary parameters related to maxDepositPercentageOfTVL, simplifying interactions with contracts.

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

Please upload report for BASE (dev@212bd50). Learn more about missing BASE report.

Additional details and impacted files
@@          Coverage Diff           @@
##             dev     #556   +/-   ##
======================================
  Coverage       ?   70.49%
======================================
  Files          ?       25
  Lines          ?      261
  Branches       ?       19
======================================
  Hits           ?      184
  Misses         ?       76
  Partials       ?        1

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


fix: always remove file extension by default in toTypeDeclarationItems

function

🔗 Linked issue
close https://github.com/Atinux/nuxt-auth-utils/issues/1
close https://github.com/nuxt/nuxt/issues/24445

❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
[x] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
It was not working because it included ext when creating dts. This should solve the problem.

📝 Checklist

[x] I have linked an issue or discussion.
[x] I have updated the documentation accordingly.

I've completely corrected the tests now

Hi @antfu :) I don't know if pinging is right. If pinging is wrong, warn me.

Thanks for the PR. And yeah it's better to not pinging unless it's urgent or being inactive for a very long time. Maintainers have their own pace to go through PRs/issues one by one. Thank you.


Add PIL backend for resize transform

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: yl-1993:x: lixiaojielixiaojie seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Please complete the docstring for backend. @yl-1993


Make properties drawer collapsible?

Hi - is it possible to make the the :PROPERTIES: ... :END: construct collapsible, similar to nodes, to match the behavior of *.org files in org-mode buffers in emacs?

Sure thing!
Implemented in version 2.2.2.


Add libicu-dev if charlock_holmes is detected

This PR adds libicu-dev if charlock_holmes is detected. Fixes #588
CC @flybayer

@flybayer hopefully this kinda shows you how you could evolve something like the Ruby provider, if you encounter other Gems in the future that need X package
The goal of Nixpacks is to backfill the dependency tree for any OS level packages based on application level dependencies
We've added a nice chunk of helper functions to make this super ezpz. Goal being, you don't necessarily even need to know much Rust to get started.
Thanks @Milo123459 ❤

@JakeCooper @Milo123459 awesome, thank you so much!


Expose addFeatureLayerAssociation_ function on Select to the API

Is your feature request related to a problem? Please describe.
Currently when you add a feature programmically to Select:
select.getFeatures().push(feature);

you cannot then use
select.getLayer(feature)

as the function that links the layer to the feature is private and only gets called when going though the mouse selection interaction.
  /**
   * @param {import("../Feature.js").FeatureLike} feature Feature.
   * @param {import("../layer/Layer.js").default} layer Layer.
   * @private
   */
  addFeatureLayerAssociation_(feature, layer) {
    this.featureLayerAssociation_[getUid(feature)] = layer;
  }

Describe the solution you'd like
Can we expose this function to the API for select, allowing us to link the associated layer to the pushed feature

We can do better than that. #13149 makes it so even features that are programmatically added to a Select interaction's features collection will be registered in the feature-layer association. So no need to expose addFeatureLayerAssociation_ to the API.

I appreciate the quick turn around for the fix!


Request URL is too long

I know that my question probably doesn't really fit here, but I couldn't find a solution anywhere yet.
I have already asked in the Client SDK repository: https://github.com/Azure/azure-mobile-apps-net-client/issues/517
And a similar question also exists in this repository: https://github.com/Azure/azure-mobile-apps-net-server/issues/119
The Problem:
We use Azure App Service with Offline Sync in our Xamarin.Forms App. During Pull Request we generate HTTP Get Requests with very long URL. When the URL is too long the request fails with a 404 Bad Request Error. We have to limit the URL length, which is very difficult because our requests are very complex.
@paulbatum Mentioned that increasing the query string and URL size will solve this problem.
But it does not work in our case. Increasing the values in the web.config has no effect. Please note: The web.config works basically. If we set the values smaller than the default values we can observe an effect. Only setting the values higher than the default values does not work.
 verb="*" type="System.Web.Handlers.TransferRequestHandler" preCondition="integratedMode,runtimeVersionv4.0" />
    </handlers></system.webServer>

Long url example:

https://xxx:80/tables/Appointment?$filter=(((((DeletionDate eq null) or (DeletionDate gt datetimeoffset'2019-10-04T09%3A38%3A49.4556730%2B00%3A00'))%20or%20(((DeletionDate%20ne%20null)%20and%20(DeletionDate%20gt%20datetimeoffset'2019-10-04T08%3A23%3A23.5255190%2B00%3A00'))%20and%20(Created%20lt%20datetimeoffset'2019-10-04T08%3A23%3A23.5255190%2B00%3A00')))%20and%20((NoSync%20eq%20null)%20or%20(NoSync%20gt%20datetimeoffset'2019-10-04T00%3A00%3A00.0000000%2B02%3A00')))%20and%20(((((((((((((((PrimaryKeyId%20eq%20guid'7e1ac018-8219-4e72-bfb2-e2f732ff4ed8')%20or%20(PrimaryKeyId%20eq%20guid'66ee9ad8-b4b4-4b6a-bdff-3c7c9ac0b0b7'))%20or%20(PrimaryKeyId%20eq%20guid'd12d3884-a7c1-4093-a991-0322194e55b5'))%20or%20(PrimaryKeyId%20eq%20guid'3ebcb875-1e5b-4a10-b8c9-968ecbce8129'))%20or%20(PrimaryKeyId%20eq%20guid'98d9f640-d695-4a03-ad6e-78b91a02c37b'))%20or%20(PrimaryKeyId%20eq%20guid'879d2677-fedd-4920-81ab-71d79519fe68'))%20or%20(PrimaryKeyId%20eq%20guid'd363c02a-9779-4647-9e16-84e97e5af591'))%20or%20(PrimaryKeyId%20eq%20guid'06ccf440-1c59-41e6-b148-eb5725c7730b'))%20or%20(PrimaryKeyId%20eq%20guid'195302e9-536c-458b-9669-36e06e2eda9e'))%20or%20(PrimaryKeyId%20eq%20guid'810af477-f2e7-4e86-8d25-dbe5d6619f4d'))%20or%20(PrimaryKeyId%20eq%20guid'115d8a40-3ccd-4868-9707-bf9bed8f0e62'))%20or%20(PrimaryKeyId%20eq%20guid'4b456cc1-ad2c-4904-8884-ffdd29149a5a'))%20or%20(PrimaryKeyId%20eq%20guid'dabf6059-b996-423a-897c-37cc88e2542f'))%20or%20(PrimaryKeyId%20eq%20guid'e9ac63bc-f892-4a48-bfa1-25931f9022b1'))%20or%20(PrimaryKeyId%20eq%20guid'd18a27bd-56e3-4e44-ad73-ee1c8df09862')))&$skip=0&$top=1000&__includeDeleted=true

Is this porject still maintained by someone?

Hi @OPunktSchmidt,
Sorry for the delay in response.
The less-than-ideal workaround I have now is to filter this on client side as suggested in https://github.com/Azure/azure-mobile-apps-net-client/issues/517#issuecomment-540034762. Sorry that I don't have a better solution now.

@OPunktSchmidt You can try logging a developer support call from the Azure portal. I have done that before so that the App Service team can investigate from their side. Would be nice if you could post the progress / outcome here.

Was there ever any resolution to this? Does app service respect the request URL when you try to override it?

@ltaljaard @jamesvidler There is no solution for this problem. I was in very close contact with Microsoft and our team invested a lot of time to find some workaround. It is simply not possible. Moreover, this SDK is no longer being developed by Microsoft and they reject any feature requests. From time to time they fix critical errors but nothing more.
So there are two options:

Sends smaller HTTP requests with a shorter URL. Splits a larger request into several parts.
Downloads all data and filters it on the client side


Modal is opening on Livewire valdiation

Package
filament/filament
Package Version
v3.0.50
Laravel Version
v10.23.1
Livewire Version
v3.0.4
PHP Version
PHP 8.2.2
Problem description
When I press a button, in order to trigger an action from the inside of a Filament form, a blank modal is opening.
https://github.com/filamentphp/filament/assets/10696975/affe8294-013b-43af-b362-fe590db703fb
Expected behavior
Perform the action from the callback without opening a modal.
Steps to reproduce

Clone https://github.com/alexmanase/filament-modal-on-submit-issue
Follow the instruction from README.md in order to install
Press the 'Edit' button
Press 'Generate' button, with the "keywords" field empty
Fill the "keywords" field
Press 'Generate' button

Reproduction repository
https://github.com/alexmanase/filament-modal-on-submit-issue
Relevant log output
No response

Sorry, I made it by private by mistake. It's public now.

Duplicate of https://github.com/filamentphp/filament/issues/7909.


monit for www.permaplant.net

[x] monit job to automatically start PermaplanT, keep it running and send emails on problems

Already implemented a few weeks ago :)


POS removal from hindi text

Description
Please include a summary of the change and which issue is fixed. List any dependencies that are required for this change.
Fixes #729
Replace issue_no with the issue number which is fixed in this PR
Have you read the Contributing Guidelines on Pull Requests?

[x] Yes
[ ] No

Type of change
Please delete options that are not relevant.

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

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] My documentation follows Template for README.md
[x] My changes generate no new warnings
[x] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works.

@Kushal997-das


confirm latest changes for website profile pages

@phet-steele - Can QA please confirm that the most recent changes for the profile pages haven't introduced any new bugs?
The issues are:

[x] phetsims/website-meteor#48
[x] phetsims/website-meteor#49
[x] phetsims/website-meteor#50
[x] phetsims/website#669

The components affected are the admin page tables for schools/orgs, the Add Your School/Org dialog, the School/Org autosuggest input, and the translator's page for Users.
You can test the changes on ox-dev, they are running in a production state now.

@mattpen good to go.


feat: 2.6 api endpoints

Summary

Adds the following API endpoints
GET

[x] bridgechains
[x] bridgechains/{id}
[x] businesses
[x] businesses/{id}
[x] businesses/{id}/bridgechains
[x] wallets/{id}/locks
[x] locks
[x] locks/{id}

POST

[x] businesses/search
[x] bridgechains/search
[x] locks/search
[x] locks/unlocked

Checklist

[ ] Documentation (if necessary)
[x] Tests (if necessary)
[x] Ready to be merged

Thanks for submitting this pull request! A maintainer will review this in the next few days and explicitly select labels so you know what's going on.
If no reviewer appears after a week, a reminder will be sent out.

A contributor has approved this PR. A maintainer will merge this PR shortly. If it shouldn't be merged yet, please leave a comment saying so and we'll wait.
Thank you for your contribution!

Your pull request has been merged but was not assigned a bounty tier. A maintainer will assign a bounty tier to this pull request in the next few days.

Your pull request has been merged and marked as tier 4. It will earn you $20 USD.


Can't delete/retract your own expense

After submitting an expense, I can edit it, but there's no (clear) way for me to wholly delete/retract it myself, or mark it as no longer needed.

Triage Template (team only)
This template is for members of the team to triage for prioritisation. For more guidance see https://www.loom.com/share/369ab467fbc64dec848085d38ff57ca0:
P1 high frequency, high impact
P2 low frequency, high impact
P3 high frequency, low impact
P4 low frequency, low impact
Examples of high impact - a problem affects users during an essential process (expenses and payments > onboarding and registration > contributing) with no workaround.
High frequency - >10% of users affected (measured as a proportion of total potential users for this case).

We actually allow users to delete their own expenses, but only after they get rejected.
We can think about enabling this after we implement https://github.com/opencollective/opencollective/issues/4562, but we'll need to polish a bit the error message on the expense page since collective/host admins will then often end up there after clicking a link in a notification email.

Discussed again in https://opencollective.slack.com/archives/GFH4N961L/p1684765587269369

How are we proceeding here? Users can still delete expenses only after it is rejected. (And the above link is not working)


[Update Snap] TM ChainSafer

Checklist
All items in the list below needs to be satisfied.

[ ] Is the summary of the change documented in this ticket?
[ ] Has a MetaMask Snaps team member reviewed whether the changes need to be vetted?
[ ] If there are changes that need to be vetted, attach a description and the relevant fixes/remediations to this issue.
[ ] The corresponding pull request in this repo has been merged.

The package name is moved from tm-chainsafer-snap-beta to tm-chainsafer-snap
The version to be allowlisted is now 1.0.239


Hotstar Clone

Describe the project you want to add with tech stack

Hi Sir,
I want to add a project on the Clone of the web platform HOTSTAR using HTML , CSS , JS. Please assign me the issue
Expected behavior

It is a clone of HOTSTAR
Screenshots (optional)

Additional context (optional )

Please assign this to me.

@sadanandpai Please assign me this issue. So that I can work on this issue.


The SOS Report Form lacks proper structuring

The SOS Report form is not structured or aligned properly.
The SOS Report form requires better organization for improved clarity. The input fields, such as email ,address etc. lack proper alignment.
To Reproduce
Steps to reproduce the behavior:

Go to 'SOS Report'
See error(form)

Expected behavior
Refer Videos
Videos
Before
https://github.com/akshitagupta15june/PetMe/assets/76811518/7584765b-edfe-49c2-911e-0aef777775f6
After(it should look like)
https://github.com/akshitagupta15june/PetMe/assets/76811518/26a99bbc-bc36-445c-907d-8b1c5ff9c6f8
Desktop (please complete the following information):

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

Smartphone (please complete the following information):

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

I'd like to work on this and fix it as required. I'm eagerly waiting to be assigned.


Add EPOLLEXCLUSIVE && EPOLLWAKEUP

r? @JohnTitor
(rust-highfive has picked a reviewer for you, use r? to override)

r? @JohnTitor
(rust-highfive has picked a reviewer for you, use r? to override)

I cannot find any definition for these flags on the uclibc source, are they usable actually?

I cannot find any definition for these flags on the uclibc source, are they usable actually?

I cannot find any definition for these flags on the uclibc source, are they usable actually?

Actually, I didn't find it either.
Compiling Target: mipsel-unknown-linux-uclibc
   Compiling tokio v1.0.1
error[E0425]: cannot find value `EPOLLEXCLUSIVE` in crate `libc`
[package]
   --> /root/.cargo/registry/src/mirrors.tuna.tsinghua.edu.cn-df7c3c540f42cdbd/mio-0.7.7/src/sys/unix/selector/epoll.rs:226:19
    |
226 |             libc::EPOLLEXCLUSIVE,
    |                   ^^^^^^^^^^^^^^ not found in `libc`

error[E0425]: cannot find value `EPOLLWAKEUP` in crate `libc`
   --> /root/.cargo/registry/src/mirrors.tuna.tsinghua.edu.cn-df7c3c540f42cdbd/mio-0.7.7/src/sys/unix/selector/epoll.rs:228:19
    |
228 |             libc::EPOLLWAKEUP,
    |                   ^^^^^^^^^^^ not found in `libc`

   Compiling clap v2.33.3
   Compiling parking_lot v0.11.1
   Compiling proc-macro-error-attr v1.0.4
error: aborting due to 2 previous errors

For more information about this error, try `rustc --explain E0425`.
error: could not compile `mio`.

I have added these two items. There is no error, and the program runs normally.

I cannot find any definition for these flags on the uclibc source, are they usable actually?

Actually, I didn't find it either.
Compiling Target: mipsel-unknown-linux-uclibc
   Compiling tokio v1.0.1
error[E0425]: cannot find value `EPOLLEXCLUSIVE` in crate `libc`
[package]
   --> /root/.cargo/registry/src/mirrors.tuna.tsinghua.edu.cn-df7c3c540f42cdbd/mio-0.7.7/src/sys/unix/selector/epoll.rs:226:19
    |
226 |             libc::EPOLLEXCLUSIVE,
    |                   ^^^^^^^^^^^^^^ not found in `libc`

error[E0425]: cannot find value `EPOLLWAKEUP` in crate `libc`
   --> /root/.cargo/registry/src/mirrors.tuna.tsinghua.edu.cn-df7c3c540f42cdbd/mio-0.7.7/src/sys/unix/selector/epoll.rs:228:19
    |
228 |             libc::EPOLLWAKEUP,
    |                   ^^^^^^^^^^^ not found in `libc`

   Compiling clap v2.33.3
   Compiling parking_lot v0.11.1
   Compiling proc-macro-error-attr v1.0.4
error: aborting due to 2 previous errors

For more information about this error, try `rustc --explain E0425`.
error: could not compile `mio`.

I have added these two items. There is no error, and the program runs normally.

@bors r+

@bors r+

:pushpin: Commit 46af3376bde0b247e7f827642e53dbae7c21be3d has been approved by JohnTitor

:pushpin: Commit 46af3376bde0b247e7f827642e53dbae7c21be3d has been approved by JohnTitor

:hourglass: Testing commit 46af3376bde0b247e7f827642e53dbae7c21be3d with merge a98a649bc3d6c4c8870323ded76d143f33ddda22...

:hourglass: Testing commit 46af3376bde0b247e7f827642e53dbae7c21be3d with merge a98a649bc3d6c4c8870323ded76d143f33ddda22...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: JohnTitor
Pushing a98a649bc3d6c4c8870323ded76d143f33ddda22 to master...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: JohnTitor
Pushing a98a649bc3d6c4c8870323ded76d143f33ddda22 to master...

Note that this change causes the mipsel target to fail the libc tests:
  cargo:warning=/home/skrap/Documents/rust-uclibc/libc/target/mipsel-unknown-linux-uclibc/debug/build/libc-test-f479457f9ef6732b/out/main.c:33476:64: error: ‘EPOLLEXCLUSIVE’ undeclared here (not in a function)
  cargo:warning=33476 |             static const int __test_const_EPOLLEXCLUSIVE_val = EPOLLEXCLUSIVE;
  cargo:warning=      |                                                                ^~~~~~~~~~~~~~
  cargo:warning=/home/skrap/Documents/rust-uclibc/libc/target/mipsel-unknown-linux-uclibc/debug/build/libc-test-f479457f9ef6732b/out/main.c:33482:61: error: ‘EPOLLWAKEUP’ undeclared here (not in a function)
  cargo:warning=33482 |             static const int __test_const_EPOLLWAKEUP_val = EPOLLWAKEUP;
  cargo:warning=      |                                                             ^~~~~~~~~~~
  cargo:warning=cc1: error: unrecognized command line option ‘-Wno-unknown-warning-option’ [-Werror]
  cargo:warning=cc1: all warnings being treated as errors
  exit code: 1

Presumably, if we want to include these constants, we should cause them to be skipped by the tests.

Note that this change causes the mipsel target to fail the libc tests:
  cargo:warning=/home/skrap/Documents/rust-uclibc/libc/target/mipsel-unknown-linux-uclibc/debug/build/libc-test-f479457f9ef6732b/out/main.c:33476:64: error: ‘EPOLLEXCLUSIVE’ undeclared here (not in a function)
  cargo:warning=33476 |             static const int __test_const_EPOLLEXCLUSIVE_val = EPOLLEXCLUSIVE;
  cargo:warning=      |                                                                ^~~~~~~~~~~~~~
  cargo:warning=/home/skrap/Documents/rust-uclibc/libc/target/mipsel-unknown-linux-uclibc/debug/build/libc-test-f479457f9ef6732b/out/main.c:33482:61: error: ‘EPOLLWAKEUP’ undeclared here (not in a function)
  cargo:warning=33482 |             static const int __test_const_EPOLLWAKEUP_val = EPOLLWAKEUP;
  cargo:warning=      |                                                             ^~~~~~~~~~~
  cargo:warning=cc1: error: unrecognized command line option ‘-Wno-unknown-warning-option’ [-Werror]
  cargo:warning=cc1: all warnings being treated as errors
  exit code: 1

Presumably, if we want to include these constants, we should cause them to be skipped by the tests.

I will include a fix to this issue in #1975 so I can make sure the libc tests pass on mipsel.

I will include a fix to this issue in #1975 so I can make sure the libc tests pass on mipsel.


Proposal : listen for data updated

Hi, thank u so much for the plug-in, 🥰
Would it be possible to implement onChanged and notifyConsumers, to notify for any update,
Currently I use dart stream in a loop and check for updates every 2 seconds and I think it would be much easier to add a listener 👂 in consumer app,

Hi, thank u so much for the plug-in, 🥰 Would it be possible to implement onChanged and notifyConsumers, to notify for any update,
Currently I use dart stream in a loop and check for updates every 2 seconds and I think it would be much easier to add a listener 👂 in consumer app,

Thanks for your idea, it's really helpful. I added it in version 0.0.4. It will be available today.

Hi, thank u so much for the plug-in, 🥰 Would it be possible to implement onChanged and notifyConsumers, to notify for any update,
Currently I use dart stream in a loop and check for updates every 2 seconds and I think it would be much easier to add a listener 👂 in consumer app,

Thanks for your idea, it's really helpful. I added it in version 0.0.4. It will be available today.

wow thank u ❤❤💗 i am still be using it on my app before publishing


Empty "pager" or empty data in grid view crashes IE

IE1 crashes on empty pager object or the "empty" default text

seems to work


optical alignment definition

Defined optical alignment in typography. Was a stub article in the closed repo; proposing the article be added here.

[x ] I have read freeCodeCamp's contribution guidelines.
[ x] My pull request has a descriptive title (not a vague title like Update index.md)
[x ] My pull request targets the master branch of freeCodeCamp.
[x ] None of my changes are plagiarized from another source without proper attribution.
[x ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@emilane I made the changes suggested by camperBot on your behalf.

Thank you!! Been away for the holidays and hadn’t had a moment to get to it
Thanks,
Emily Lane
—
t:<PHONE_NUMBER>
w: emilymlane.io
e<EMAIL_ADDRESS>On Nov 25, 2018, 3:56 PM -0500, Randell Dawson<EMAIL_ADDRESS>wrote:

@emilane I made the changes suggested by camperBot on your behalf.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Processing on Upload

Add processing symbol / screen when adding wars / users

This functionality is in place now


Mazumder - Research and collect Orgs of interest 2x per month

Orgs should be logged herer, and then approved by Raja and/or Steph: https://docs.google.com/spreadsheets/d/1KD4WfihfcFzy9MCcmyvgkIlyy2hGmq9vtxYFW5SG4RM/edit#gid=0
schedule a 15 min meeting 2x per month with each other to go over your findings prior to getting approval.

Status update: Our first meeting was scheduled by Emily on Wednesday (1/25) at 2 pm after the collaborators' meeting.
Document: Pending Organisms Log

Completed by Steph, Emily, and Jingyue.

Complete


refactor(sdk): use a universal path definition / consolidate typedefs

Fixes
Fixes WB-13438
Description
Note: This is step 1 of a series of (3?) stacked PRs.
This PR is the simplest of the PRs and is a pure refactor, but cleans things up a little.

Create a StrPath = Union[str, os.PathLike[str]] type
Move "path type" definitions to their own file (this will become more valuable later)
Replace all instances of the union or variants (like Union[str, Path]) with it
Exclusively use the StrPath and FilePathStr form, rather than the util.URIStr dotted notation (for the sake of consistency)

Testing
Pure refactor, should be fine to rely on CI.
Checklist

[x] Include reference to internal ticket "Fixes WB-NNNN" and/or GitHub issue "Fixes #NNNN" (if applicable)
[x] Ensure PR title compliance with the conventional commits standards

🤖 Generated by Copilot at 0f1f358

Oh, we're the coders of the wandb crew
And we've got some refactoring to do
We'll use the StrPath type for every path
And avoid the circular imports wrath

overall i like this change a lot. The only minor thing i'm trying to figure out is the file in lib that only has types seems wrong use of the lib directory. So either we start new pattern of directory for types or move it as part of some other file. What do you think?


Changed object-store-sc settings

Tweaks to improve GlusterFS performance on VMs. Read more: https://access.redhat.com/documentation/en-us/red_hat_gluster_storage/3.2/html/configuring_red_hat_virtualization_with_red_hat_gluster_storage/chap-hosting_virtual_machine_images_on_red_hat_storage_volumes

@jonandernovella I'd rather have this additional storage class as a helm package, so then you can tweak as much as you want with no need to send additional PRs. What do you think?

Can we make a helm chart called s3-interface? Having this storage class, and triggering minio install as well?

@mcapuccini We can, yes. Though I really prefer much more this solution since we use official packages and SC, with best practcices atm.

@mcapuccini Anyway, this is just a PR for the phnml branch- We can discuss about master later on :)

But then you need to parametrize this options, we can't change them again and again. In my opinion is way simpler via Helm. But it is up to you.

Ok! Up to @andersla then 🙂

I would suggest parameterize {{volumeoptions}} as  {{volumetype}}.
Maybe put the default value in the heketi role defaults/main.yml ?

I think you can create a multiline variable (since it is a very long string) described here: http://docs.ansible.com/ansible/latest/YAMLSyntax.html

Looks good! I'll merge it into phenomenal/stable - I think it should go into master as well


Update Blazor Get Started guidance

Fixes #11802
Thanks @Wimpje! :rocket:

Great addition...I was coming here to do this myself. Thanks @guardrex


create password hash as part of form data

Currently in iron-skillet and other config tools the goal is to set a user account with a new password. Need to have the password converted to md5 hash before added to the snippet output.

done via custom jinja2 filters


Add threadIdentifier to local notifications

Feature Request
Add the possibility for local notifications (e.g. as in PushNotificationIOS.presentLocalNotification()) to include threadIdentifier on its details.
Why it is needed
Using the threadIdentifier allows to group notifications together on the notification center
Possible implementation
I'm happy to send a PR, it seems to me it should be as simple as adding the parsing of that parameter on the UILocalNotification object. This would be an iOS only property
@implementation RCTConvert (UILocalNotification)

+ (UILocalNotification *)UILocalNotification:(id)json
{
  NSDictionary<NSString *, id> *details = [self NSDictionary:json];
  ....
  if (details[@"threadIdentifier"]) {
    notification.threadIdentifier = [RCTConvert NSString:details[@"threadIdentifier"]];
  }
  return notification;
}

(Adding the if statement because I guess that if not explicitly set, iOS sets a "smart" threadIdentifier based on application, title, etc.)
Code sample
PushNotificationIOS.presentLocalNotification({
// should also work with PushNotificationIOS.scheduleLocalNotification
   alertTitle: "My threaded local notification"
   alertBody: "This will be threaded",
   threadIdentifier: "SomeIdentifier"
})

@holden-caulfield Thanks for raising the issue!
Support for threadId in local notification has been added in v1.7.0 :tada:


Reduce QuerySpec usage

See commit messages

Codecov Report

Merging #6457 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##             master   #6457      +/-   ##
===========================================
+ Coverage     82.98%     83%   +0.02%
- Complexity    13051   13054       +3
===========================================
  Files          1496    1496
  Lines         48516   48516
  Branches       5859    5859
===========================================
+ Hits          40260   40270      +10
+ Misses         5479    5474       -5
+ Partials       2777    2772       -5

Impacted Files
Coverage Δ
Complexity Δ

...a/io/crate/planner/operators/RelationBoundary.java
97.67% <100%> (ø)
15 <0> (ø)
:arrow_down:

.../planner/projection/builder/SplitPointVisitor.java
94.59% <100%> (ø)
12 <0> (ø)
:arrow_down:

...rc/main/java/io/crate/planner/SubqueryPlanner.java
100% <100%> (ø)
3 <2> (ø)
:arrow_down:

...va/io/crate/analyze/relations/QueriedRelation.java
100% <100%> (ø)
9 <2> (+2)
:arrow_up:

sql/src/main/java/io/crate/planner/Planner.java
87.57% <100%> (+0.07%)
40 <0> (ø)
:arrow_down:

...ava/io/crate/planner/operators/LogicalPlanner.java
96.72% <100%> (ø)
16 <3> (ø)
:arrow_down:

.../java/io/crate/planner/SelectStatementPlanner.java
84.37% <100%> (ø)
2 <0> (ø)
:arrow_down:

.../crate/planner/projection/builder/SplitPoints.java
100% <100%> (ø)
7 <5> (ø)
:arrow_down:

...te/planner/consumer/InsertFromSubQueryPlanner.java
83.33% <100%> (-0.46%)
6 <0> (ø)

...o/crate/analyze/SafeExpressionToStringVisitor.java
83.33% <0%> (-16.67%)
7% <0%> (-1%)

... and 33 more

Continue to review full report at Codecov.

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


EmailStr special-cases<EMAIL_ADDRESS>

Checks

[x] I added a descriptive title to this issue
[x] I have searched (google, github) for similar issues and couldn't find anything
[x] I have read and followed the docs and still think this is a bug

Bug
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
         pydantic version: 1.9.0
        pydantic compiled: True
             install path: /home/***/python3.10/site-packages/pydantic
           python version: 3.10.0 (default, Oct 12 2021, 14:07:44) [GCC 9.3.0]
                 platform: Linux-5.13.0-46-generic-x86_64-with-glibc2.34
 optional deps. installed: ['dotenv', 'email-validator', 'typing-extensions']

import pydantic

class Foo(pydantic.BaseModel):
    email: pydantic.EmailStr

<EMAIL_ADDRESS>
The above code will result in a ValidationError. IMHO this is not correct though, as this is a valid email address, albeit not a usable one. But it's one extensively used, and specifically designed, for testing purposes, and not being able to use a testing domain for testing purposes because pydantic falsely flags this as an "invalid email address" kind of defeats its purpose.

Duplicate of #4009.
You need to upgrade or downgrade email-validator.