This template doesn't work with Rust 1.70

For some reason images built with Rust 1.70 don't seem to be working—see https://github.com/dbanty/sample-rust-action/actions/runs/5166990441/jobs/9307582806 as an example. Not sure what is causing this yet since if I build the image locally and run it it works fine (I can't pull the ghcr image, it won't run on my mac)

Fixed by moving off of Docker


vue-server-render/build.dev.js error

Version
2.6.11
Reproduction link
no
Steps to reproduce
just webpack hot update
What is expected?
without error
What is actually happening?
line :9072
code
var file= ref.file;
typeError : Cannot red property 'file' of undefined

Hello, thank you for taking time filling this issue!
However, we kindly ask you to use our Issue Helper when creating new issues, in order to ensure every issue provides the necessary information for us to investigate. This explains why your issue has been automatically closed by me (your robot friend!).
I hope to see your helper-created issue very soon!


NullPointerException in the MainRender.getSurfaceTexture method

Describe the bug
The app crashes when starting a stream due to a NullPointerException in the MainRender.getSurfaceTexture method.
To Reproduce
Steps to reproduce the behavior:
Open the app.
Navigate to the stream setup screen.
Start a new screen broadcast stream session.
Observe the app crash.
Expected behavior
The stream should start successfully without crashing.
Screenshots

Smartphone (please complete the following information):
Library version: 2.5.5
Device: Motorola Malta
OS: Android 10 (SDK 29)
Media server: [e.g. SRS, version 5.0]
Class used: [e.g. RtmpCamera1]
Additional context
Crash logs from Play Console:
Exception java.lang.RuntimeException:
  at android.app.ActivityThread.deliverResults (ActivityThread.java:4873)
  at android.app.ActivityThread.handleSendResult (ActivityThread.java:4914)
  at android.app.servertransaction.ActivityResultItem.execute (ActivityResultItem.java:51)
  at android.app.servertransaction.TransactionExecutor.executeCallbacks (TransactionExecutor.java:135)
  at android.app.servertransaction.TransactionExecutor.execute (TransactionExecutor.java:95)
  at android.app.ActivityThread$H.handleMessage (ActivityThread.java:2041)
  at android.os.Handler.dispatchMessage (Handler.java:107)
  at android.os.Looper.loop (Looper.java:214)
  at android.app.ActivityThread.main (ActivityThread.java:7386)
  at java.lang.reflect.Method.invoke
  at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run (RuntimeInit.java:492)
  at com.android.internal.os.ZygoteInit.main (ZygoteInit.java:980)
Caused by java.lang.NullPointerException:
  at com.pedro.encoder.input.gl.render.MainRender.getSurfaceTexture (MainRender.kt:176)
  at com.pedro.library.view.GlStreamInterface.getSurfaceTexture (GlStreamInterface.kt:118)
  at com.pedro.library.base.StreamBase.startSources (StreamBase.kt:451)
  at com.pedro.library.base.StreamBase.startStream (StreamBase.kt:156)
  at com.atharvasystem.quickscreenrecorder.services.ShareScreenService.startStream (ShareScreenService.kt:168)
  at com.atharvasystem.quickscreenrecorder.ui.streams.StreamSourceActivity.activityResultContract$lambda$0 (StreamSourceActivity.kt:102)
  at com.atharvasystem.quickscreenrecorder.ui.streams.StreamSourceActivity.$r8$lambda$LtWeMkLkNKtetOKKVdlgnnLpK3s (Unknown Source)
  at com.atharvasystem.quickscreenrecorder.ui.streams.StreamSourceActivity$$ExternalSyntheticLambda2.onActivityResult (D8$$SyntheticClass)
  at androidx.activity.result.ActivityResultRegistry.doDispatch (ActivityResultRegistry.kt:371)
  at androidx.activity.result.ActivityResultRegistry.dispatchResult (ActivityResultRegistry.kt:331)
  at androidx.activity.ComponentActivity.onActivityResult (ComponentActivity.kt:786)
  at androidx.fragment.app.FragmentActivity.onActivityResult (FragmentActivity.java:151)
  at android.app.Activity.dispatchActivityResult (Activity.java:8110)
  at android.app.ActivityThread.deliverResults (ActivityThread.java:4866)

Hello,
Getting above crash when using genericCamera2 with OpenGlView

Can you give me more info about it?
Can you reproduce it using an older version? in which version is working fine?
Are you checking that prepareVideo and prepareAudio return true before startStream?
Can you share me a code example to reproduce it?

Thank you for your response!
Validating URL and Key:
We are checking and validating the URL and stream key before starting the stream and also validating during the stream when using screen broadcast.
Reproducing Issue in Older Versions:
Apologies for the confusion earlier. The crash occurred in version 2.5.4, not 2.5.5.
Checking prepareVideo and prepareAudio:
Yes, we are ensuring that prepareVideo() and prepareAudio() return true before starting the stream.
Code Example:
Here’s a snippet of the code we are using to reproduce the issue:

Let me know if further details are required!
Crash Line and Library Version:
I’ve double-checked, and you’re right—2.5.5 doesn’t align with the crash log. The crash occurred with 2.5.4, and I’ll ensure the exact compilation details are provided. I'll update this thread with those details soon.
Thank you for your help in resolving this!

Hello,
Can you show me code of prepareStream method?

Yes @pedroSG94
fun prepareStream(resultCode: Int, data: Intent): Boolean {
        keepAliveTrick()
        stopStream()
        val mediaProjection = mediaProjectionManager.getMediaProjection(resultCode, data)
        if (mediaProjection == null) {
            Log.e(TAG, "MediaProjection is null")
            return false
        }
        val screenSource = ScreenSource(applicationContext, mediaProjection)
        return try {
            //ScreenSource need use always setCameraOrientation(0) because the MediaProjection handle orientation.
            //You also need remove autoHandleOrientation if you are using it.
            //You need to call it after prepareVideo to override the default value.
            genericStream.getGlInterface().setCameraOrientation(0)
            genericStream.changeVideoSource(screenSource)
            genericStream.getStreamClient().setReTries(5)
            prepareVideoAudio()
        } catch (ignored: IllegalArgumentException) {
            false
        }
    }

    ```

Hello,
Sorry for late response. I'm on holidays
I will create multiple checks for this case in a commit.
Do you know if this crash is frequent?
This only should happens when glInterface start is not fully completed because the SurfaceTexture is ceated in that method but reading the code that nerver should happens.
If you are using multithreading make sure use the library always with the same thread to avoid thread problems

Hi @pedroSG94
No problem at all, I hope you're enjoying your holidays!
Regarding your question about the crash frequency, I don't have exact data on how often it occurs, but it seems to happen intermittently in the app. I’ve noticed it a couple of times, but it’s not consistent, so I’m not sure if it’s a recurring issue or just a rare occurrence.
I’m using the library on the main thread as recommended, so I don’t believe threading is the issue here. It seems like the crash might be linked to the glInterface start not completing fully, as you mentioned. I’ll keep an eye on it and provide more details if it happens again.
Looking forward to your commit with the checks, and I’ll test it once it’s ready!
Thanks for your help, and enjoy the rest of your holidays!


Add Thunderbird to clients

Thunderbird is a really great IRC client :+1:
It even supports OTR now.

The clients.md guide is minimal introduction to getting users onto IRC. It isn't intended to be an exhaustive list.

This PR isn't intended to make the list exhaustive.
If the guide is intended for new users of IRC, then Thunderbird seems like a user-friendly introduction and is available for all platforms.

Thunderbird isn't terrible, but it's an email client with IRC attached, and that will be confusing, especially because IRC doesn't appear anywhere on https://www.thunderbird.net or https://www.thunderbird.net/en-US/features/. So it's not worth including in the list


Feature/evse

EVSE work

Client command IDs updated as requested

@mmb-davidsmith I took the liberty to rebase force push into your branch

Hello,
A good XML editor is Sublime
Thanks,
Michael
Michael Cowan
Principal System Engineer
Xylem
639 Davis Drive Morrisville NC 27560
HomeO: +1.919.469.2453 | M: +1.919.602.6629 | Lab: +1.919.317.6320
[signature_1839051095]https://xylem.com/?utm_source=signature&utm_medium=email&utm_campaign=corporate
Xylem.comhttps://xylem.com/?utm_source=signature&utm_medium=email&utm_campaign=corporate | Sensus.comhttps://sensus.com/?utm_source=signature&utm_medium=email&utm_campaign=corporate
LinkedInhttps://www.linkedin.com/company/sensus | Twitterhttps://twitter.com/SensusGlobal | YouTubehttps://www.youtube.com/c/SensusGlobal | Facebookhttps://www.facebook.com/SensusGlobal/ | Instagramhttps://www.instagram.com/sensusglobal/ | 3D Tourhttps://apps.kaonadn.net/5277329597136896/index.html
From: iwinterburn @.>
Sent: Thursday, December 22, 2022 4:19 AM
To: csa-iot/data-model @.>
Cc: Cowan, Michael - Xylem @.>; Comment @.>
Subject: Re: [csa-iot/data-model] Feature/evse (PR #6)
@iwinterburn commented on this pull request.

In EVSEManagement.xmlhttps://urldefense.com/v3/__https:/github.com/csa-iot/data-model/pull/6*discussion_r1055249803__;Iw!!OKzgfr8!deWPpuXMxMcJ2FWLxmjy_3HnbmXC4FqEdcFPzicpMwXpDkSiMnJF5GNyNrOtwqoI2P8d02DmKPtzg1T9SFXhwtHBIKpiNg$:

 <attribute id="0001" name="SupplyState" type="EvseSupplyState" required="true" default="0" />

 <attribute id="0002" name="EvseType" type="EvseType" required="true" default="0" />

 <attribute id="0003" name="EvseFault" type="EvseFault" required="true" default="0" />

 <attribute id="0004" name="EvseEnableChargeTime" type="uint16" max="65535" required="true" default="65535" />

 <attribute id="0005" name="EvseEnableDischargeTime" type="uint16" max="65535" default="0" />

 <attribute id="0006" name="MinimumChargeCurrent" type="uint16" required="true" default="6" />

 <attribute id="0007" name="MaximumChargeCurrent" type="uint16" required="true" default="15" />

 <attribute id="0008" name="MaximumDischargeCurrent" type="uint16" default="15" />

 <attribute id="0009" name="EvseInternalTemperature" type="int16" />

 <attribute id="000A" name="BatteryLevel" type="uint8" required="true" reportRequired="true"  max="100" default="255" />

 <attribute id="000B" name="BatteryCapacity" type="uint16" required="true" default="65535" />

 <attribute id="000C" name="CircuitId" type="uint16"  />

 <attribute id="000D" name="CircuitCapacity" type="uint16" />

 <!-- Attribute Set 0x01: Reading Information -->

 <attribute id="0100" name="ChargingVoltage" type="uint16" required="true" max="65535" />

ChargingValue default clarified to 0xFFFF by BG on 2022/09/27
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/csa-iot/data-model/pull/6*pullrequestreview-1227350078__;Iw!!OKzgfr8!deWPpuXMxMcJ2FWLxmjy_3HnbmXC4FqEdcFPzicpMwXpDkSiMnJF5GNyNrOtwqoI2P8d02DmKPtzg1T9SFXhwtGvaX-Pjg$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/ALTQZPGYUDGZTIRVTCOQUADWOQMIPANCNFSM55TC74ZQ__;!!OKzgfr8!deWPpuXMxMcJ2FWLxmjy_3HnbmXC4FqEdcFPzicpMwXpDkSiMnJF5GNyNrOtwqoI2P8d02DmKPtzg1T9SFXhwtGpEWXdcw$.
You are receiving this because you commented.Message ID<EMAIL_ADDRESS>CONFIDENTIALITY NOTICE: This e-mail, including any attachments and/or linked documents, is intended for the sole use of the intended addressee and may contain information that is privileged, confidential, proprietary, or otherwise protected by law. Any unauthorized review, dissemination, distribution, or copying is prohibited. If you have received this communication in error, please contact the original sender immediately by reply email and destroy all copies of the original message and any attachments. Please note that any views or opinions presented in this e-mail are solely those of the author and do not necessarily represent those of Xylem Inc..


Not working on macOS Sequoia

Can you please adjust the program to be compliant with macOS Sequoia? Thanks!

I am on Sequoia 15.1 and it still works.


Assembly cleanup

Cleanup commits for the assembly code. No functional changes.

Can you capture the added context-switch check in the commit message?

Sorry, i saw this is actually captured in the commit (not in the PR description, but that's ok)


why my custom elements such as 'view', 'text' are not show?

Version
2.6.11
Reproduction link
https://jsfiddle.net/t9fyx82u/5/
Steps to reproduce
nothing to do.
What is expected?
view and text will be shown, just like view2 and text2.
What is actually happening?
view and text in the #app are not show，but they can be displayed outside of #app

text and view are reserved tags for SVG. It has a special meaning and that's why it shouldn't be used as a component name

text and view are reserved tags for SVG. It has a special meaning and that's why it shouldn't be used as a component name

but they can be displayed outside of #app，i want to known why？I check it in chrome dev tool，they are all in the dom tree，but not display inside of #app

Oops, didn't mean to close it but I also forgot to add the labels.
They are not HTML elements, they are for SVG, I think that's why they are not visible. Maybe we could detect if the element is inside of an SVG and change based on that. Either way, Using non-standard HTML elements is something that should be avoided.

Oops, didn't mean to close it but I also forgot to add the labels.
They are not HTML elements, they are for SVG, I think that's why they are not visible. Maybe we could detect if the element is inside of an SVG and change based on that. Either way, Using non-standard HTML elements is something that should be avoided.

so，I want to know how Vue makes them not show? They have been rendered into the dom。

Currently Vue creates known SVG elements with document.createElementNS.


[BEAM-8630] Add logical types, make public

Found some missing type translation when wiring beamRowToZetaSqlStructValue and createZetaSqlStructTypeFromBeamSchema up to ZetaSQL compliance tests.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

R: @robinyqiu


Downgrade to Santuario 2.3.1, add an integration test

Fix #46
Fix #45

Could you please review @martinffx?

Any chance to review, @martinffx?

@ppalaga sorry, I'll have a look at this tonight or this weekend.

Still no time to review @martinffx ? Would it perhaps be OK for you if @gastaldi grants merge permissions to me?

sorry, I got sick over the weekend. No problem with you getting merge rights!

@ppalaga congratulations, you're now part of the quarkus-xmlsec team now ;)

@all-contributors add @ppalaga for code and maintenance

I got sick over the weekend.

Get well soon!
Reviewing is still welcome now or in the future.


Add share menu

Description
Implemented a reusable Share Menu component triggered by the "Share Report" button. The menu includes options such as Email, Facebook, X (Twitter), and Copy Link, with functionality to be added later. It aligns with design specifications, is responsive across screen sizes, and is covered by unit tests using Jest. The component is added to the Component Test Library.
#112
Type of changes

[ ] Bugfix
[ ] Chore
[X] New Feature

Testing

[ ] I added automated tests
[X] I think tests are unnecessary

https://github.com/user-attachments/assets/60fb2b34-e311-47ab-890a-8c5c7fe5609c

Looks great! The SVG paths are large - should the icons be separate components, or would that be overkill?

Yes, I thought it might be better that way. Since it was a bit of an eyesore for you too, I’ve moved the SVGs to separate files and imported them as components.


Update chapter 7 exercises

Chapter 7 continues to be about 9th chords.
Exercise format:
I - ii - V - vii° with full 4-string voicing
I - ii - V - vii° with dropped 5ths
7-A will use the lowest strings
7-B will use the middle strings
7-C will use the highest strings
Dev notes
Use multiple \score blocks for this, to make it easier to label the voicing types.

DONE!


HLAE 2.35.2 Doesn't work

Hey all, after updating the newest release of HLAE (2.35.2) I only get loader failed msg. The game launches but the mirv commands doesn't work. It worked perfectly before the new update. I've ran it in administration etc.

Which game are you trying to use HLAE with? Also - HLAE should never be run in administration mode, this will cause errors.

What's the exact error message(s)?
Try to unblock injector.exe in its File Properties dialogue first (right click the injector.exe and select "Properties" to open it):
https://www.google.com/search?q=File+pr ... 0&tbm=isch
The injector.exe is in the folder the HLAE.exe is (unless your antivirus deleted it), the x64\injector.exe is for 64 bit injection only, but CS:GO is 32 bit.
Also make sure that nothing is set to be run as admin, this is not required and can cause problems.

here is a ss of the error msg

After checking the unlbock it still didn't work. But i deleted the older version of it and re-installed the newest, checked the unblock again and now it works perfectly. Thank you :)

Glad you could solve it.


\HTML::mailto not working

I use eloquent/joins this link of downloaded datatables project.But when I include the following line in the it could not display any.
->editColumn('name', function ($model) {
return \HTML::mailto($model->email, $model->name);
})
Please help I am stuck overhere.

I solved it.


Add flows to the database schema

Based on the preliminary data made available at the web app seed folder, I added the food flows to the database schema. This is a diagram of the schema I could infer from the files:

This is a work in progress and subject to change until the model is considered finished.
cc @cameronkruse @faustoperez @oliverroick @wrynearson

Status update:

The seed workflow now ingests areas, nodes, edges, flow segments, and flows of preliminary data available in the Drive folder v2
I added two extra commands, db:dump and db:restore, to facilitate data ingestion to staging/production
The README file was updated with instructions on how to perform the seed workflow
The seed process is likely to be updated once new data files are available, but I believe this is already in a good spot to start implementing API routes

@oliverroick, could you please review? I suggest trying to populate the local and staging database by following the README instructions.

@oliverroick thanks for the review, I updated the README with the suggested change.


Ability to Set Mouse Scroll Direction & Enable Smooth Scrolling

Description of the new feature / enhancement
A user may set the direction the mouse wheel scrolls without needing to edit the registry.
A user may scroll to an exact position with the scroll wheel on their mouse instead of scrolling in "block" amounts.
Scenario when this would be used?
Set Scroll Direction
Everywhere! Personally, I find it frustrating to need to configure a mouse by editing the registry when a new one is paired, and often this doesn't work for all mice (e.g. changing the FlipFlop DWORD for my Logitech MX Vertical doesn't take effect).
Smooth Scrolling
Same as the above. Smooth scrolling would be a fantastic addition—a user shouldn't need to revert to using the scroll bar to exactly position themselves in a scrollable context—this should be achievable with only the scroll wheel. Adjusting the standard scroll to a minimum amount is still quite "jumpy" and results in incredibly slow scrolling.
Supporting information
Please see Apple's smooth scrolling and "natural scroll direction".

Thanks for your message. This is a Duplicate of another issue: /dup #6434


Homepage displaying typo in description for March 30th Blog post

Description
Aanconda.

Tasks

[x] Address typo in markdown file for blog post
[x] Address typo in blog/news.json

Fixed: see PR https://github.com/conda-incubator/conda-dot-org/pull/104

Issue has been fixed. Thanks for your contribution @electric-souperman.


XIONE-1555: ContinueWatching API fails

Reason for change: Return token correctly
Test Procedure: Run curl commands from ticket.
Risks: Low
Signed-off-by: Mark Vandenbriele<EMAIL_ADDRESS>

Moving changes to separate branches.


[virt-handler/vm.go] Use existing constants instead of string literals for VMI migration events

What this PR does
Before this PR:
Migration related message consts in virt-handler/vm.go were only used in matching unit tests and actual events used string literals.
After this PR:
Replaced string literals with existing consts.

Special notes for your reviewer

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

NONE

/cc @orelmisan


Client Hints + initial state Adaptive Loading Hooks

Now that https://github.com/GoogleChromeLabs/react-adaptive-hooks support setting initial values (e.g initialEffectiveConnectionType), I think there would be value in showing how Client Hints can provide an initial value on the server and this can then be updated, if necessary, on the client-side via the JS APIs exposed in Hooks.
Stack requirement:
We should build an app that uses Next.js as it has to keep SSR in mind and would be a good "stress test" this all works together.
Signal wise, I think it's fine to rely on Network and Device Memory. We would want this demo to be responsive and render cleanly on both mobile and desktop. Output should have at least a 90/100 on Lighthouse for performance.
What are we building?
A clean, nice-looking TV show browser. TMDB's API provides the data to allow us to build this. It's very similar to our existing TMDB client built using CRA, except will focus on shows rather than movies. There are a few existing Next.js open-source apps we could look to for reference here.
Reference
https://topheman-movie-browser.herokuapp.com/ is from this year and is open-source https://github.com/topheman/nextjs-movie-browser.
https://github.com/timneutkens/next-episode is a Next.js TV show app that might be a good fit here. It's 2 years old so likely requires an upgrade to get it working with the latest version of Next.
I really like how next-episode looks. It would be good to evaluate if it makes more sense to fork it and update or try improving the design of nextjs-movie-browser. We could also choose to write something from scratch if there are too many "extras" in either implementation.
cc @anton-karlovskiy

@addyosmani
I'd like to play around with nextjs-movie-browser and next-episode shortly.
If you've got an idea, please update me then I'll switch to your idea immediately. :)

@addyosmani
We can keep track of the progress on this Glitch.
Glitch Source

Link to Glitch App
Link to Project on Glitch

@addyosmani
I met some challenge when using react-adaptive-hooks in Next.js project.

Just importing react adaptive hooks in next.js page throws an error.
import { useMemoryStatus } from 'react-adaptive-hooks/memory';
import { useNetworkStatus } from 'react-adaptive-hooks/network';

I think the cause of this issue is because ES6 features like import and export from our react-adaptive-hooks package are not properly transpiled in Next.js.
Re:
https://stackoverflow.com/questions/37634198/node-error-syntaxerror-unexpected-token-import
We could resolve this issue by customizing babel configuration for Next.js so that it can transpile our react-adaptive-hooks package in node_modules.
I fixed this issue with next-transpile-modules.
I wonder if we should publish the transpiled package to npm.
When users are using custom server for Next.js project, they could meet this issue in spite of installing next-transpile-modules so they might have to customize babel to transpile custom server.
Re:
https://github.com/zeit/next.js/issues/6273

@anton-karlovskiy Thank you for raising this concern. Can I confirm if you are running into this issue when using the UMD version of the package as well? We recently added a bundled version that doesn't use raw ESM that could be pulled in e.g see<EMAIL_ADDRESS>Are you still experiencing those issues when importing the whole package?

Any updates on whether the UMD build helped here, @anton-karlovskiy? :)

@addyosmani
Sorry for being late. Let me update you shortly. :)

@addyosmani
I tested with UMD build. It's working well.
This is ESM build usage
import { useMemoryStatus } from 'react-adaptive-hooks/memory';
import { useNetworkStatus } from 'react-adaptive-hooks/network';

This is UMD build usage
import { useNetworkStatus, useMemoryStatus } from 'react-adaptive-hooks/dist/index.umd.js';

For using ESM build in Next.js, we need to transpile this package because server side does not understand ES6 module features like import and export.
Or we can use UMD build as a fallback.
Which one shall we use?
I wonder if using UMD is a bit more overheaded than path import with ESM build.

Let's try getting the ESM build working. Official Node support for ESM is finally in a good place, but given how recent this we may have to rely on workarounds. We can use the UMD build in case we get stuck.

@addyosmani
We are not stuck as we found a workaround like transpiling react-adaptive-hooks package with the help of next-transpile-modules or babel configuration.
Let me proceed with ESM and that workaround.

@addyosmani
The TV show app with ESM build of react-adptive-hooks and transpiling of that package won't work on Glitch. But it's working on Now service hosting and Firebase hosting too.
The app just with UMD build is working well on Glitch.

Link to Glitch App
Link to Project on Glitch

@addyosmani
Do you think we can close this issue?

Yep!


AJ-1115 workload identity options

https://broadworkbench.atlassian.net/browse/AJ-1115
Reminder:
PRs merged into main will not automatically generate a PR in https://github.com/broadinstitute/terra-helmfile to update the WDS image deployed to kubernetes - this action will need to be triggered manually by running the following github action: https://github.com/DataBiosphere/terra-workspace-data-service/actions/workflows/tag.yml. Dont forget to provide a Jira Id when triggering the manual action, if no Jira ID is provided the action will not fully succeed.
After you manually trigger the github action (and it completes with no errors), you must go to the terra-helmfile repo and verify that this generated a PR that merged successfully.
The terra-helmfile PR merge will then generate a PR in leonardo.  This will automerge if all tests pass, but if jenkins tests fail it will not; be sure to watch it to ensure it merges. To trigger jenkins retest simply comment on PR with "jenkins retest".

what David said, my only other question is, what would provide the database endpoint url? would that be something Leo would give to us or is this an api WDS would have to make? since I am not seeing changes in any other db transactions besides cloning maybe that info is already in WDS somewhere and I just dont know where to look.

thank you!  properties file looks so much better :)


Promise chain ending with Promise<{}> returns previous type

TypeScript Version:  2.2.2
Code
const p1 = Promise.resolve(0).then(() => Promise.resolve({}))
const p2 = Promise.resolve("").then(() => Promise.resolve({}))
const p3 = Promise.resolve({a: 1}).then(() => Promise.resolve({}))

Expected behavior:
I expect p1, p2, and p3 to be of type Promise<{}>
Actual behavior:
p1, p2, and p3 are of types Promise<number>, Promise<string>, and Promise<{a: number}>.

This problem is not restricted to ending the chain with Promise<{}>, if my tests are correct this also happens the other way around:
const p1 = Promise.resolve({}).then(() => Promise.resolve(1)) // type is Promise<{}>, should be Promise<number>

it seems tsc chooses the wrong "overload" from lib.es5.d.ts, namely
then(onfulfilled?: ((value: T) => T | PromiseLike<T>) | undefined | null, onrejected?: ((reason: any) => T | PromiseLike<T>) | undefined | null): Promise<T>;

instead of
then<TResult>(onfulfilled: ((value: T) => T | PromiseLike<T>) | undefined | null, onrejected: (reason: any) => TResult | PromiseLike<TResult>): Promise<T | TResult>;

this does not happen with typescript 2.3.2... unfortunately VSCode currently uses 2.2.2


I don't understand the code

Hi!
I'm reading the code in AdvancedGamedevTutorials/AdvancedGamedev-WindingNumbers/Polygon2D.cs code and I don't understand the line 22.
Why do you do this [(i + 1) % this.points.Count]? Why do you use the modulus operator here? Are you looking for a specific vertex?
The only reason I can think of to use the module is because the vertices are ordered and you know where each one is. But I haven't found any mention that the vertices need to be ordered.
Thanks.

They are ordered in this version of the code (as polygons usually are). But if you have a different structure you might not need it.


Add dutch translation

A clean pull request with only the dutch translations added. Also imho my translations are a little better than the other PR.
If something is missing or incorrect, please let me know.

Thanks for the contribution


support many methods router

@pi0 bro, I add a new feature for router

@pi0

Thanks for the PR @1018715564 💚


Update CODE_OF_CONDUCT.md

docs(code-of-conduct) added https in site url(s)
NOTE: Please read the CONTRIBUTING.md guidelines before submitting your patch,
and ensure you followed them all:
https://github.com/Kong/kong/blob/master/CONTRIBUTING.md#contributing
Summary
SUMMARY_GOES_HERE
Full changelog

[Implement ...]
[Add related tests]
...

Issues resolved
Fix #XXX

As explained in https://github.com/Kong/kong/pull/3933, we would prefer not making direct changes to this file. Thanks


Add Form to Upcoming L2s

This PR aims to add Form network to L2beat. Lint and test passed locally.
note: I was receiving an error when attempting to tinify logos.
class AccountError extends Error {
^

AccountError: Your monthly limit has been exceeded. (HTTP 429/TooManyRequests)
    at new AccountError (.../form-l2beat/node_modules/tinify/lib/tinify/Error.js:41:1)
    at Error.create (.../form-l2beat/node_modules/tinify/lib/tinify/Error.js:37:16)
    at IncomingMessage.<anonymous> (.../form-l2beat/node_modules/tinify/lib/tinify/Client.js:86:59)
    at IncomingMessage.emit (node:events:530:35)
    at endReadableNT (node:internal/streams/readable:1696:12)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
  status: 429
}

Drafting this PR for internal review :)

Thanks @mateuszradomski I have enabled maintainer edit access and updated the purposes array to include SocialFi but still requires the type modifications


HUGE DEAL - it goes to the specified tab, but it doesn't ACTIVATE the tab itself

Everything is in the title. I use Google Chrome. It goes to the right tab, meaning it activates Chrome, but it doesn't ACTIVATE the tab itself. This happens when multiple Chrome windows are open.

Have you made sure that the link is to the html file and not the folder file? Can i get a request pull so I can have a look at it and help?

Hi ! thanks for the reply !
Unfortunately, i have no idea what you're talking about. I'm no developer/programmer. I know my way around some stuff though.
If you could guide me through both your questions about how I make sure about the nature of the link (which link?) and how do I make a request pull (I've been in GitHub for 3 days ^^ ), I am sure I could answer you adequately.
To illustrate the issue further, when multiple Chrome windows are open, if i switch to a specific tab using the Alfred workflow, specially if the tab is in a different window, the specific tab shows up, but it just doesn't activate. Meaning if I start to type something right away after using the Alfred keyword, i'd still type stuff in the tab in the window i was originally in, even though the tab which i wanted showed up.

This was fixed in https://github.com/epilande/alfred-browser-tabs/pull/49 and has been released with 🔖 v1.0.6

@epilande I have tested in this version and the next and for me is not working


AWS Lambda plugin in Gateway 2.8.4 missing aws_imds_protocol_version support

Is there an existing issue for this?

[X] I have searched the existing issues

Kong version ($ kong version)
2.8.4-1
Current Behavior
Looking at the changelogs for the AWS Lambda plugin at: https://docs.konghq.com/hub/kong-inc/aws-lambda/changelog/
Kong Gateway <IP_ADDRESS>: Backported the parameter aws_imds_protocol_version into 2.8.x.
When I try to add aws_imds_protocol_version: v2 to my decK configuration:
# deck gateway validate kong.yaml
Error: validate entity 'plugins (aws-lambda)': HTTP status 400 (message: "schema violation (config.aws_imds_protocol_version: unknown field)")

Expected Behavior
I should be able to set aws_imds_protocol_version: v2 and have the Lambda plugin use the instance role credentials.
Steps To Reproduce
This is an upgrade of an older 2.x cluster, the migrations were completed before these changes were being tested.
Anything else?
Looking at the kong/plugins/aws-lambda/iam-ec2-credentials.lua file in the 2.8.4 tag, I see no token request logic, unlike 3.4.  Is this just a doc error, or was it supposed to be backported, but just never was?

The feature is backported only for enterprise 2.8. We do not often backport features and when we do we rarely backport for OSS. Please note in the changelog we are using a 4-digit version, which is for enterprise.
We recommend using the latest version of Kong.


Allow explicit return type object

No longer omit the conversion expression when the return type is explicitly object.
Force the return type of a lambda expression to typeof(void) to prevent the emission of a conversion expression.
Fixes #185

Thank you! I will look at this better in the following days.
It looks a little wrong to use void in that case, maybe we can find some alternatives ...

I agree that it's a bit wrong to use void in that case.. I considered a few alternatives that didn't feel great either:

creating a dummy type DontEmitConversionExpression
add a new parser settings

In the end, since this is purely internal for when we parse a lambda expression, it seemed acceptable to use void to minimize the impact.
If you have a better idea, I'm all for it :)

@metoule
I'm doing some experiments with the following solution #190 . It is still in progress and not sure that this will actually solve the problem or it will be feasible. But I will try ...
If you already have any kind of feedbacks please let me know.
If in some days I will not be able to complete it we can fallback to your solution, that in anycase looks like to be good. Ok?


fix: minify sprite files

Description
Minify png files from around 9MB to around 2MB
Context: #222
Type of Change

[ ] ✨ New feature (non-breaking change which adds functionality)
[x] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[ ] ✅ Build configuration change
[ ] 📝 Documentation
[ ] 🗑 Chore

Looks like we need to update goldens

Looks like we need to update goldens

yep, also this: https://github.com/VeryGoodOpenSource/very_good_ranch/pull/225


Summary hasErrors: false when missing a required file

What is the issue and why is it an issue?
When validating a feed with a missing required file, the summary section of the JSON response has hasErrors: false.
We're using the summary payload to get an idea of the number of errors for a feed and a missing required feeds should not be valid.
Example feed: https://api.prod.partners-fs37hd8.zoov.eu/gbfs/2.2/brest/en/gbfs.json?&key=OGNhZDNjMDQtYTA0Yi00NzU2LWE0MTItOGJlYzE1Y2E4NGEx
Details
{"summary":{"validatorVersion":"1.0.0","version":{"detected":"2.2","validated":"2.2"},"hasErrors":false,"errorsCount":0},"files":[{"schema":{"$schema":"http://json-schema.org/draft-07/schema","$id":"https://github.com/MobilityData/gbfs/blob/v2.2/gbfs.md#gbfsjson","description":"Auto-discovery file that links to all of the other files published by the system.","type":"object","properties":{"last_updated":{"description":"Last time the data in the feed was updated in POSIX time.","type":"integer","minimum":1450155600},"ttl":{"description":"Number of seconds before the data in the feed will be updated again (0 if the data should always be refreshed).","type":"integer","minimum":0},"version":{"description":"GBFS version number to which the feed conforms, according to the versioning framework (added in v1.1).","type":"string","const":"2.2"},"data":{"description":"Response data in the form of name:value pairs.","type":"object","patternProperties":{"^[a-z]{2,3}(-[A-Z]{2})?$":{"type":"object","properties":{"feeds":{"description":"An array of all of the feeds that are published by the auto-discovery file. Each element in the array is an object with the keys below.","type":"array","items":{"type":"object","properties":{"name":{"description":"Key identifying the type of feed this is. The key must be the base file name defined in the spec for the corresponding feed type.","type":"string","enum":["gbfs","gbfs_versions","system_information","vehicle_types","station_information","station_status","free_bike_status","system_hours","system_alerts","system_calendar","system_regions","system_pricing_plans","geofencing_zones"]},"url":{"description":"URL for the feed.","type":"string","format":"uri"}},"required":["name","url"]}}},"required":["feeds"]}},"minProperties":1,"additionalProperties":false}},"required":["last_updated","ttl","version","data"]},"errors":false,"url":"https://api.prod.partners-fs37hd8.zoov.eu/gbfs/2.2/brest/en/gbfs.json?&key=OGNhZDNjMDQtYTA0Yi00NzU2LWE0MTItOGJlYzE1Y2E4NGEx","version":"2.2","recommended":true,"required":true,"exists":true,"file":"gbfs.json","hasErrors":false,"errorsCount":0},{"languages":[],"required":false,"exists":false,"file":"gbfs_versions.json","hasErrors":false,"errorsCount":0},{"languages":[],"required":true,"exists":false,"file":"system_information.json","hasErrors":false,"errorsCount":0},{"languages":[],"required":false,"exists":false,"file":"vehicle_types.json","hasErrors":false,"errorsCount":0},{"languages":[],"required":false,"exists":false,"file":"station_information.json","hasErrors":false,"errorsCount":0},{"languages":[],"required":false,"exists":false,"file":"station_status.json","hasErrors":false,"errorsCount":0},{"languages":[],"required":false,"exists":false,"file":"free_bike_status.json","hasErrors":false,"errorsCount":0},{"languages":[],"required":false,"exists":false,"file":"system_hours.json","hasErrors":false,"errorsCount":0},{"languages":[],"required":false,"exists":false,"file":"system_calendar.json","hasErrors":false,"errorsCount":0},{"languages":[],"required":false,"exists":false,"file":"system_regions.json","hasErrors":false,"errorsCount":0},{"languages":[],"required":false,"exists":false,"file":"system_pricing_plans.json","hasErrors":false,"errorsCount":0},{"languages":[],"required":false,"exists":false,"file":"system_alerts.json","hasErrors":false,"errorsCount":0},{"languages":[],"required":false,"exists":false,"file":"geofencing_zones.json","hasErrors":false,"errorsCount":0}]}

Thank you @AntoineAugusti for creating this issue with such details. This is very helpful.
This issue was raised in https://github.com/MobilityData/gbfs-validator/issues/127. I will merge both issues and ping the dev team about it.

Duplicate of https://github.com/MobilityData/gbfs-validator/issues/127


[Arges] Golesti village is not found as a component of Stefanesti city, Arges county - Fail

Precondition
SOLR: http://zimbor.go.ro/solr/#/romania/query
browser: Chrome
Steps to Reproduce:

Step
Action
Expected
Status

1
open SOLR in browser
SOLR UI is loaded without any errors
Fail

2
type Golesti in the q field
You are able to type Golesti in the q field
Fail

3
type judet: Arges in the fq field
you are able to type judet: Arges in the fq field
Fail

4
click on Execute Query button
"judet:Arges" and "localitate: Golesti", "comuna: Stefanesti" are displayed
Fail

Actual Results:
Golesti village is not found as a component of Stefanesti city, Arges county

https://orase.testquality.com defect D44.

Este orasul Stefanesti cu satul Golesti.


Fix payday stats

Ran this on a backup -
Before:

After:

After:

Nice!

I removed a commit which had a few comments on it - We decided to consider both users and teams active even if they only received/accrued dues, and didn't actually exchange money within Gratipay.

Deployed, ran branch.py.

Removed branch.py in https://github.com/gratipay/gratipay.com/commit/b35692f08fadf05b77f67df1ae793f685949cb3c

!m @rohitpaulk


Move unused warnings in IndentAwareLexer of original implementation to diagnostics

See c127343 (#38)

Update: in fact, this would be the only non-error diagnostics reported at all


After adding a new item to the "CRUD example items" page, the application should return to the item list page [CRUD items page]

Steps:

Launch the Saas
Go to the CRUD example items page
Click on the "Add new item" button
Fill in the field
Click on the "Save changes" button - the item was saved successfully but there is displayed an item details page

Expected: User should be moved to the item list page

The same behavior should be when editing.


Clarify maintenance

Compliant Kubernetes needs a lot of TLC (tender, love and care) from admins, to ensure the platform is up-and-running and secure. However, there is currently no public documentation explain what exactly an admin needs to do.
Acceptance criteria

There is a "Maintenance" page in the "Administrator Manual" section explain what should be done during maintenance windows.

Closing: https://elastisys.io/compliantkubernetes/operator-manual/maintenance/


Update CREDITS.txt

Credited myself

I am not sure if this is the right way, or if I only need to Press “Commit changes” after editing.

This should really be part of the other pull request, but don't worry, I'll accept it anyway


Disable hot code push while user is editing

While user is editing something, hot code push must be disabled.
The following is example code for accomplishing this:
Meteor._reload.onMigrate(function(reloadFunction) {
  // Check to see if you should stop migration.
  if (Meteor.router.templateEquals('templateThatShouldPreventReload')) {
    // Set up an autorun that watches a reactive source. When that changes to
    // a value that shouldnt stop migration, go ahead and call reloadFunction().
    Deps.autorun(function(c) {
      if (!Meteor.router.templateEquals('templateThatShouldPreventReload')) {
        c.stop();
        reloadFunction();
      }
    });
  } else {
    // If you are ready to migrate, just return [true]
    return [true];
  }
});

Fixed.


Explanation of output files

Pull request details
Describe the changes made in this pull request
Not sure if this is right and need some opinions about wording/labels.
List of related issues or pull requests
Closes #352
Collaboration confirmation
As a contributor I confirm

[ ] I read and followed the instructions in README.dev.md
[ ] The documentation is up to date with the changes introduced in this Pull Request (or NA)
[ ] Tests are passing
[ ] Lint is passing

@abelsiqueira Should I keep this draft or close it? If the output / structure is gonna change a lot it might be easier starting from scratch. (Also I thiiink the stuff so far might just be based on the Structures. If so, there should be a better way not to duplicate things.)

The structure will change a little after #745, indeed. Most things will have a year column. It might also change after a larger refactor, but probably not a lot.
It might also change a lot after we start to actually use it for something.
Whether it's better to update this or start fresh I can't say. It depends on how easy it was to do this PR, and how much conflict there is now.

Yeah... I think I put a bit of time in it, but not a ton and it'll probably be more pain fixing it than just starting over.


Search between accounts

#506
Using classes AccountSearchRequest & AccountSearchResponse it works correctly but returns only the first 21 users (the first request) that are searched.
Usage:
        new AccountSearchRequest("something to search").execute(client)
    	.thenAccept(response -> {
    		response.getUsers().forEach(System.out::println);
    	}).join();

So I created the AccountSearchTokenRequest class for requests with rank_token and page_token. But so the class is useless without AccountSearch with which you can get all the 'pages' of the requests with the list of the returned accounts.
Usage:
        AccountSearch as = new AccountSearch(client, "something to search");
        System.out.println("Total pages: " + as.getTotalPages());
        for(int j = 0; j<as.getTotalPages(); j++) {
        	System.out.println("Page number " + as.getCurrentPageN());
        	as.getCurrentPage().getUsers().forEach(u->System.out.println(u.getUsername()));
        	as.nextPage();
        }

another pr comes


Small fixes

9168c2d - roles/nv_gpu/tasks/install_nv.yml: Ensure that the GPU Operator PackageManifest exists

Without this patch, the following commands may erroneously succeed
(if not ignored), because of the | jq...
Example:
TASK: nv_gpu : Get the version of the GPU Operator on OperatorHub
changed: True

<command> oc get packagemanifests/gpu-operator-certified -n openshift-marketplace -ojson | jq -r .status.channels[0].currentCSV

<stderr> Error from server (NotFound): packagemanifests.packages.operators.coreos.com "gpu-operator-certified" not found

541c5da - roles/nv_gpu_install_from_commit/files/helm_deploy_operator.sh: fix image name transformation

The default version of the devicePlugin image changed a few days
ago [1] to use the -ubi8 image by default. This broke the automated
image name transformation that only worked with -ubuntu* or no -*
suffix.
This patch simply transforms <anything>-OS into <anything>-ubi8 or
<anything> into <anything>-ubi8.
Expect it to break when an image named <anything> (without an OS
suffix) will come ...
1: https://github.com/NVIDIA/gpu-operator/commit/de819f431b05a5eed8c5fd9aaa4483bbeb4fa6b5#diff-aecfc7542f3194d151b0a0812fa08dca7a2147da1c6bc06f41f19aa2b4f934afR99

480e798 - roles/nv_gpu_install_from_commit/files/helm_deploy_operator.sh: save helm-generated resources before creating them

This patch stores in the $ARTIFACT_DIR the YAML description of
resources that will be instantiated from the helm chart.

/test gpu-operator-e2e

@ArangoGutierrez gpu-operator-e2e tests are :green_circle:, please have a look


Fix memory leaks

Closes #17033, #17032, #17031, thanks @fangang190
Im not sure about right way to fix for #17030, cuz im not an expert in android apk, and calling zipClose(unaligned_apk, NULL); before return may cause errors, so anybody with free time and knowledge could fix it instead

Thirdparty bugfix should contain comment // -- GODOT start -- and // -- GODOT end --, a .patch of the change, and the following comment to thirdparty/README.md:
Important: Some files have Godot-made changes. They are marked with // -- GODOT start --
and // -- GODOT end -- comments and a patch is provided in the [thirdparty_module]/ folder.

@Noshyaar I remember this, thanks, I fixed it, its ok now ?

Damn, I didnt notice that actual thirdparty/README.md is existed.. Sorry, I will fix this..

Done.. waiting for merge..

Did you forget to git add the patches by any chance? They seem to be missing ^^

I dont understand you, I just use the commit and then rebase with squash.. and it was enough in most cases

Well, what I mean is that you mention the patches in the readme:

Important: Some files have Godot-made changes.
They are marked with // -- GODOT start -- and // -- GODOT end --
comments and a patch is provided in the b2d_convexdecomp/ folder.

But there are no *.patch files in your commit.

@Hinsbart Ok, I change it to just
"Important: Some files have Godot-made changes.
They are marked with // -- GODOT start -- and // -- GODOT end --
comments."

Thanks! Would you mind contributing those fixes upstream too, so that we can drop our custom modification next time we sync with them?

b2d_convexdecomp: https://github.com/erincatto/Box2D/blob/master/Contributions/Utilities/ConvexDecomposition/b2Polygon.cpp#L1340
Etc2comp: https://github.com/google/etc2comp/blob/master/EtcLib/Etc/EtcFilter.cpp#L229-L232
libwebm: That's Google, no clue how to send a PR, so maybe leave that one..

Okay ) I take a look on it

@akien-mga Seems like erincato merged my PR to b2Polygon - https://github.com/erincatto/Box2D/pull/486,
I think the part of this PR should be reverted and b2Polygon should be updated

Thanks for the notice. It looks like we already had quite a few changes compared to the upstream code (diff when trying to replace our files by upstream ones: https://paste.fedoraproject.org/paste/-5ujnPQIUTZFZI8IjhhB2A), so I guess it's good to keep as is.
Some of our changes shown in the diff are cosmetic and could be pushed upstream, but others are specific to Godot (like removing the dependency on Box2D's glue and using some Godot constants like CMP_EPSILON, or adding a namespace and removing debug spam).


Video in Test set not available

Please Note. The following video in the test set is not available.
URL: https://www.youtube.com/watch?v=-WbUH7jJ9qQ
video_id: 5ba7890ab_1

thanks! Will update the test set now!


Update with main

@Getnet-Ayele: do you want to update your fork, and also push your updates to the main repository? It might help as I try to recruit Phil or others to help address the tcp/zmq issue to have the two updated.


net: lwm2m: Utility functions added to LWM2M Engine

Added 3 utility functions :

lwm2m_engine_update_period_service() which updates the period of a given service.
This allow the service to occur more or less frequently depending on programmer's choice without having to remove the service and add another one. It is also possible to control the system with this function being called on a callback execute trigger.
lwm2m_engine_update_period_min_observer() which updates the min_period_sec for a given observe node.
This allow the dynamic change of the observer pmin value during the execution of the program, without the user having to request it through REST commands.
lwm2m_engine_update_period_max_observer() which updates the max_period_sec for a given observe node.
This allow the dynamic change of the observer pmax value during the execution of the program, without the user having to request it through REST commands.

Didn't add Doxygen or lwm2m.h as almost none of the functions inside lwm2m_engine.c are described there. Even if lwm2m_engine.h it is not done.

Didn't add Doxygen or lwm2m.h as almost none of the functions inside lwm2m_engine.c are described there. Even if lwm2m_engine.h it is not done.

Sorry, but that's not an argument, most of the functions are internal to the engine indeed, but also almost all of the API functions declared in lwm2m.h are defined in lwm2m_engine.c. In this case we do create an API function (otherwise who's the user of these?) so we need to include them in the public header file.

Didn't add Doxygen or lwm2m.h as almost none of the functions inside lwm2m_engine.c are described there. Even if lwm2m_engine.h it is not done.

Sorry, but that's not an argument, most of the functions are internal to the engine indeed, but also almost all of the API functions declared in lwm2m.h are defined in lwm2m_engine.c. In this case we do create an API function (otherwise who's the user of these?) so we need to include them in the public header file.

You are right, it's now done.

@tlr-id could please show any example usage or use-case of these functions ? As I understand, to successfully change pmin or pmax, an observation from server should be made first. And if I would like to change pmin on some resource on boot, on instance creation for example, and before server see my registration and request observation it will fail.
I would like to change pmin on some my resources on boot, and unable to find any useful way to do it, so I duplicate lwm2m_write_attr_handler() to work with "pathstr" and period_sec (exp: lwm2m_write_attr("3347", "pmin=1&pmax=60"); ), and it works great for any "pathstr", but it's a little bit large function and I will be very glad if I could replace it with smaller one which is already into mainline.
Thanks!


Fix bug with unicode filenames

If used on a database images with unicode filenames, opencv will be raise with error

this does not behave same with imread.
import cv2
import numpy as np

img_path = "dataset/img1.jpg"

img1 = cv2.imread(img_path)

img2 = open(img_path, "rb")
chunk = img2.read()
img2 = np.frombuffer(chunk, dtype=np.uint8)

assert np.array_equal(img1, img2) == True

You miss
img2 = cv2.imdecode(img2, cv2.IMREAD_COLOR)

Must be
import cv2
import numpy as np

img_path = "dataset/img1.jpg"

img1 = cv2.imread(img_path)

img2 = open(img_path, "rb")
chunk = img2.read()
img2 = np.frombuffer(chunk, dtype=np.uint8)
img2 = cv2.imdecode(img2, cv2.IMREAD_COLOR)

assert np.array_equal(img1, img2) == True

instead of removing following line, would you please deactivate it and add comment "this causes troubles when reading files with non english names". i will merge it when pipeline completed successfully.
return cv2.imread(img)

unfortunately, your code does not satisfy linting requirements: https://github.com/serengil/deepface/actions/runs/4832336779/jobs/8611143293
deepface/commons/functions.py:105:0: C0303: Trailing whitespace (trailing-whitespace)
this cannot be merged until pipeline succeeded

thank you for your contribution


[Kosmos-2] unable to build the environment

First of all, thank you for sharing the awesome code. I have an issue setting up the environment, so I have a question. I executed the container using the nvcr.io/nvidia/pytorch:22.10-py3 Docker image, as stated in the README, and cloned the repository. Then, I installed the packages line by line using the commands in vl_setup_xl.sh. However, when I run pip install fairseq, it seems that there is a dependency conflict after the torch version changes. In other words, after installing fairseq, the torch version changes from 1.13.0a0+d0d6b1f to 2.1.0+cp38~ (I don't remember exactly). I'm wondering if this phenomenon can be a problem, and I would appreciate it if you could let me know how to solve it.

@pengzhiliang

@Leebh-kor  The pytorch version should be fine if you could run the code successfully.


Add support for some Blackboard sites

This adds a Toggl button to all items in lists, such as lists of assignments, for students.
Because Blackboard is hosted, not all Blackboard sites are included.  The link also does not go to a specific Blackboard site, but Blackboard's help page for logging in.  Only sites that follow this format are included in this specific integration.

blackboard.*.edu
*.blackboard.com

The pull request will be evaluated and merged after the end of the toggl developer challenge deadline (https://toggl.com/toggl-developer-challenge) on February 14th.
Thank you for your contribution.

We've decided not to add this integration to Toggl Button. Thank you for participating in the Toggl Button Challenge.


CUDA: Add support for the jit_module function

This PR adds the jit_module decorator to CUDA as numba.cuda.jit_module. It is slightly different to the numba.jit_module function, in that it compiles functions as device functions by default, but can be made to compile functions as kernels by passing device=False to it. This is handy for jitting some functions as device functions and others as kernels by calling it twice in a module, and the example added to the documentation reflects this.
Whilst writing tests I noticed that FakeCUDAKernel from the simulator stores the Python function as fn instead of py_func, which is a discrepancy between it and the real dispatcher - to make this more consistent I've moved fn to py_func and provided a property fn with a deprecation warning to support the old name - though, it's unlikely this was ever used by anyone as it's not advertised and specific to the simulator only.

A question related to this PR: how would others feel about the jit_module decorator allowing jitting of other modules, to support the UDF use case described by @dscole in https://github.com/numba/numba/issues/2630#issuecomment-349064212? I am also aware of other users that are thinking of potential UDF usecases too. I imagine implementing this with something like the following change (as a sketch - some error checking would make sense, e.g. to check a module is passed):
diff --git a/numba/cuda/decorators.py b/numba/cuda/decorators.py
index 9923bf08c..267f02951 100644
--- a/numba/cuda/decorators.py
+++ b/numba/cuda/decorators.py
@@ -163,7 +163,7 @@ def convert_types(restype, argtypes):
     return restype, argtypes

-def jit_module(**kwargs):
+def jit_module(module=None, **kwargs):
     """ Automatically ``jit``-wraps functions defined in a Python module. By
     default, wrapped functions are treated as device functions rather than
     kernels - pass ``device=False`` to treat functions as kernels.
@@ -185,9 +185,10 @@ def jit_module(**kwargs):
     if 'device' not in kwargs:
         kwargs['device'] = True

-    # Get the module jit_module is being called from
-    frame = inspect.stack()[1]
-    module = inspect.getmodule(frame[0])
+    if module is None:
+        # Get the module jit_module is being called from
+        frame = inspect.stack()[1]
+        module = inspect.getmodule(frame[0])
     # Replace functions in module with jit-wrapped versions
     for name, obj in module.__dict__.items():
         if inspect.isfunction(obj) and inspect.getmodule(obj) == module:

With this, one could do:
from numba import cuda
import user_functions

cuda.jit_module(user_functions)

@cuda.jit
def wrapper(x):
    i = cuda.grid(1)
    if i < len(x):
        x = user_functions.computation(x)

where user_functions contains something like:
import math

def computation(x):
    # Some arbitrary mathematical operation
    return math.cos(x) + x ** x

Here the user code itself needn't be aware of Numba at all, whereas presently (in this PR and in the CPU target jit_module function) the user is required to call jit_module in their module.

Still interested in this, just haven't got it to the top of my pile in a long while :-)

I still intend to address this one day when it bubbles to the top of my queue.


[Feature] Implemented Real-Time Search Functionality for FAQs

📋 Description
This PR introduces a real-time search feature for the FAQ section. Users can now enter keywords in a search bar to dynamically filter and display matching FAQs as they type, enhancing the ease and efficiency of locating specific information. Additionally, the feature highlights relevant keywords within the FAQs to improve readability.
🔨 Changes Made

Implemented a real-time search bar in the FAQ section.
Added dynamic filtering to display relevant FAQs based on user input.
Included keyword highlighting within matched FAQs to improve visibility and user experience.

✅ Checklist
Before submitting the PR, please make sure you have completed the following:

[x] I have followed all the guidelines mentioned in CONTRIBUTING.md.
[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 (if applicable).
[x] I have tested it locally, and it works as expected.
[x] Any dependent changes have been merged and published in downstream modules.

🏷 Types of Changes
What type of changes does your code introduce? (Check all that apply)

[x] New feature (non-breaking change which adds functionality) ✨

[ ] UI enhancement (non-breaking change which enhances UI) 🎨

[ ] Documentation update 📚

Fixes #261

📸 Screenshots (if applicable)

@Harshdev098 Please consider assigning me label under gssoc-ext level3 and hactoberfest-accepted, once the PR gets merged, as I have implemented a new feature.

Can you enhance this feature
Currently the box content is not opening when the keyboard has been found
This enhancement can be done so that if the keyword has been found it should open them for the user

@PrityanshuSingh Can you enhance this feature Currently the box content is not opening when the keyboard has been found This enhancement can be done so that if the keyword has been found it should open them for the user

Ok will add the functionality accordingly

@Harshdev098 Kindly review the PR, now the pannel box content is opening if the keyword is matching. Consider assigning me label of level3 as a new feature is implemented, once the PR gets merged.
https://github.com/user-attachments/assets/9adc91eb-038e-429b-ae41-5b3b4dc3354b


Predawn theme not coloring functions properly

VSCode Version: 0.10.12-alpha
OS Version: Windows 10

Steps to Reproduce:

Install Predawn theme pack
Change to Predawn theme

Actual:
Functions are not colored properly. They appear as a single color throughout

Expected:
Functions, variables, etc should be colored accordingly
E.g.

This has to do with #3008


Build and unit test failures

Hi,
I am trying to build easyloggingpp on Ubuntu 22.4 (WSL2) with Gcc 12.1.0. The CMake configuration fails with following message:
CMake Warning (dev) at /opt/stow/cmake-3.24.3-linux-x86_64/share/cmake-3.24/Modules/FindPackageHandleStandardArgs.cmake:438 (message):
  The package name passed to `find_package_handle_standard_args` (GTest) does
  not match the name of the calling package (gtest).  This can lead to
  problems in calling code that expects `find_package` result variables
  (e.g., `_FOUND`) to follow a certain pattern.
Call Stack (most recent call first):
  cmake/Findgtest.cmake:152 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
  CMakeLists.txt:78 (find_package)
This warning is for project developers.  Use -Wno-dev to suppress it.

-- Could NOT find GTest (missing: GTEST_LIBRARY GTEST_INCLUDE_DIR GTEST_MAIN_LIBRARY)

I made following changes to the CMakeFile to overcome the above issue:
semihc@ALTAY:/opt/TC/ext/easyloggingpp$ git diff
diff --git a/CMakeLists.txt b/CMakeLists.txt
index 538cc8a..654db04 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -75,7 +75,8 @@ if (test)
     require_cpp11()
     set (CMAKE_MODULE_PATH ${CMAKE_MODULE_PATH} "${CMAKE_SOURCE_DIR}/cmake")

-    find_package (gtest REQUIRED)
+    #-find_package (gtest REQUIRED)
+    find_package(GTest CONFIG REQUIRED) # Gtest/mock

     include_directories(${gtest_SOURCE_DIR}/include ${gtest_SOURCE_DIR})

@@ -98,7 +99,8 @@ if (test)
     )

     # Standard linking to gtest stuff.
-    target_link_libraries(easyloggingpp-unit-tests gtest gtest_main)
+    #- target_link_libraries(easyloggingpp-unit-tests gtest gtest_main)
+    target_link_libraries(easyloggingpp-unit-tests GTest::gtest GTest::gtest_main)

     add_test(NAME easyloggingppUnitTests COMMAND easyloggingpp-unit-tests -v)
 endif()

However, This time I encounter unit test failures:
1: [----------] 11 tests from WriteAllTest (3092 ms total)
1:
1: [----------] Global test environment tear-down
1: [==========] 79 tests from 26 test suites ran. (11934 ms total)
1: [  PASSED  ] 77 tests.
1: [  FAILED  ] 2 tests, listed below:
1: [  FAILED  ] CommandLineArgsTest.LoggingFlagsArg
1: [  FAILED  ] HelpersTest.ConvertTemplateToStdString
1:
1:  2 FAILED TESTS
1/1 Test #1: easyloggingppUnitTests ...........***Failed   12.09 sec
0% tests passed, 1 tests failed out of 1
Total Test time (real) =  12.09 sec
The following tests FAILED:
1 - easyloggingppUnitTests (Failed)
Errors while running CTest
Output from these tests are in: /opt/TC/ext/easyloggingpp/bld_gcc/Testing/Temporary/LastTest.log
Use "--rerun-failed --output-on-failure" to re-run the failed cases verbosely.
semihc@ALTAY:/opt/TC/ext/easyloggingpp$
Do you have any advice to fix unit tests?
Regards,
Semih

I maintain the easyloggingpp package in Fedora Linux.
I previously filed https://github.com/amrayn/easyloggingpp/issues/816 about this.
I don’t know what the root cause is, but I am working around the failures by calling easyloggingpp-unit-tests with the argument --gtest_filter=-CommandLineArgsTest.LoggingFlagsArg.


Attribute maps

Following the discussion in https://github.com/google/incremental-dom/issues/114, here's iDOM reworked to accept a map of attribute name/values.
Fixes https://github.com/google/incremental-dom/issues/115, https://github.com/google/incremental-dom/issues/23. It also opens up the possibility of exposing updateAttributes publicly, allowing Glimmer-like rendering without patching everything in the containing element.
I'd love some outside performance testing. I'm showing an improvement over master, even with the object allocations.

I created an issue, #118, to create some performance tests. The reason I have not done so far is that creating truly accurate tests for this is really hard.
In any case, for this specific CL, I'm seeing a ~30% drop in perf. I can at least add those tests somewhere but they should be taken with a giant grain of salt.

Bumping this again. I'm only seeing marginal drops in performance:

Browser
Test
Implementation
Result

Chrome
Selection
Explore
0.258ms

Chrome
Selection
Maps
0.382ms

Chrome
Creation
Explore
1.351ms

Chrome
Creation
Maps
1.575ms

FF
Selection
Explore
0.303ms

FF
Selection
Maps
0.466ms

FF
Creation
Explore
4.447ms

FF
Creation
Maps
5.107ms

For selection on Chrome, it looks like it takes 50% more time for maps vs explore. I don't think that is really marginal.
I think by splitting out core, we can allow people to make that determination for themselves though.
One thing to note is that in order to get a consistent value for the time, I filter outliers, which will filter out long runs due to GC. Capturing that sort of information in a test is pretty hard t do. There are other spikes that should be ignored due to getting swapped off the CPU for another process to run which might occur due to the fact that any test needs to hog the CPU in order to get enough samples, which is bound to cause it to get bumped off at some point.

For selection on Chrome, it looks like it takes 50% more time for maps vs explore. I don't think that is really marginal.

50% yes, but 0.124ms is maginal. We're selecting from 200 rows, meaning we're spending ~0.00062ms longer per row. Even at the slower speed, we could be updating over 8000 rows and still hitting 60fps. I think the simplified API more than makes up for that.

I think by splitting out core, we can allow people to make that determination for themselves though.

Core being DOM alignment vs DOM+attribute alignment?

One thing to note is that in order to get a consistent value for the time, I filter outliers, which will filter out long runs due to GC.

I run 4 or 5 times each, taking my perceived average. I'm not actually averaging them, but taking the number I see most often.

50% yes, but 0.124ms is maginal. We're selecting from 200 rows, meaning we're spending ~0.00062ms longer per row. Even at the slower speed, we could be updating over 8000 rows and still hitting 60fps. I think the simplified API more than makes up for that.
It might be a small difference on desktop, but you should also consider mobile. From my experience, JS execution on mobile takes roughly 8-10x as long as on laptop. You should then consider that performance is on a relatively modern flagship phone. If you consider the $30 - $50 smart phones used in emergining markets, you need to account for a larger amount of time.

So if you have 16ms for a frame and you need to set aside ~4ms for the browser layout, you effectively have only 12ms. If you assume you need something like 20x as long for a low end phone as you see on desktop, that means you only have 0.6ms total perform a diff. Now if you assume that the application itself wants to perform some logic, the amount of time the library can use is even smaller.
The tests also have a somewhat simplified DOM structure and number of attributes compared to some use cases. To compensate I bumped up the number of rows to be more than a typical application would use.

Core being DOM alignment vs DOM+attribute alignment?

Correct. There isn't really a need to force people to buy into both at the same time. I'm not sure yet how the attribute caching can be done in a good way while keeping it completely separate from core. For now, the core part would need to allocate the object holding newAttrs and attrsArr. Perhaps a creation hook can be used and the logic handling the attributes can have its own data object.

I run 4 or 5 times each, taking my perceived average. I'm not actually averaging them, but taking the number I see most often.

The code used to generate that number is doing some averaging and filtering already. So if you trigger a GC on 1/100 runs, I just drop that run. Each of the tests does a different number of runs, selection doing 400 and creation doing 200. The threshold I use for filtering out runs is if they are outside 1.5 times the 1st or 3rd quartile of runs.

Incremental DOM will not be making this change. With a little more work the current core can be pulled out into a separate repo and you would definitely be free to build this as a separate project if you are interested.


Should be able to compare list and numpy.ndarray

They could hold the exact same ints but the solution gets rejected because the data types are different.

Might as well convert to numpy array and use numpy.allclose. See #26.


Don't add entries to classpath in maven project

From either runtimes or facets. Let maven handle it
@nbashirbello @akerekes
close #312

LGTM

I think the push failure is flakiness, but yell if any other tests start failing after this.


Interleave stages

CLA is valid!

@mriduljain Let's avoid include mnist into our repo. Please invoke scripts/setup-mnist.sh to set it up for tests.

Please squash the commits

+1


Resolve Bootstrap Migration Edge Case (#4730)

Signed-off-by: Brian Downs<EMAIL_ADDRESS>

Proposed Changes

Types of Changes

Verification

Linked Issues

#4733
User-Facing Change

Further Comments

Codecov Report

Merging #4736 (9b67692) into engine-1.21 (50b3580) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@               Coverage Diff               @@
##           engine-1.21    #4736      +/-   ##
===============================================
- Coverage        10.41%   10.40%   -0.02%
===============================================
  Files              140      140
  Lines            10042    10052      +10
===============================================
  Hits              1046     1046
- Misses            8770     8780      +10
  Partials           226      226

Flag
Coverage Δ

unittests
10.40% <0.00%> (-0.02%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/bootstrap/bootstrap.go
20.51% <ø> (+1.46%)
:arrow_up:

pkg/cli/server/server.go
0.00% <0.00%> (ø)

pkg/cluster/bootstrap.go
0.00% <0.00%> (ø)

pkg/etcd/etcd.go
14.48% <0.00%> (-0.06%)
:arrow_down:

Continue to review full report at Codecov.

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


socket(): Invalid argument

Getting the following error on trying to start beanstalk via the service on Ubuntu 16.04. Using the beanstalkd package installed from apt which is version 1.10
/usr/bin/beanstalkd: net.c:63 in make_server_socket: socket(): Invalid argument

Managed to fix this with sudo apt-get --purge remove beanstalkd && sudo apt-get install beanstalkd

@threesquared the version from repo it's old anyway. you might want to use the one from master.
I also created a project https://github.com/silviucpp/beanstalkd-deb where you can build your own deb and install it using sudo apt-get -i package.deb
Silviu


Added subscriptionId and resourceGroupName in Server properties

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[x] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[x] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

@melihmutlu Can you make the same change on the version https://github.com/Azure/azure-rest-api-specs/tree/master/specification/postgresql/resource-manager/Microsoft.DBforPostgreSQL/preview/2020-02-14-preview? That is the preview version our CLI and Terraform based off.

Hi @melihmutlu is this ready to merge?

Hi @melihmutlu is this ready to merge?

Yes, ready.


Update to Omniauth 2.0

Updating to the latest Omniauth gem supports POSTing to our Azure authentication endpoint and resolves CVE-2015-9284.
Fixes #692
Fixes #262

@whereismyjetpack and I tested this in a deployment, and all looks good.


Record screen as demo

https://asciinema.org/

https://github.com/asciinema/asciinema-server/issues/171#issuecomment-403277091


Update Appfile

@teolemon could you remind me the difference between package names?

I added the explanation directly in the README: https://github.com/openfoodfacts/openfoodfacts-androidapp/blob/master/README.md