add a smali debugger

#1136
Hi guys, this debugger works well on android 7 and Android 8(may be, haven't tested),
but it can't get local variables on 9 and 10 without debug info in the app,
even if it has this debugger haven't support to parse debug info yet :D.
And for Android 11 we can get variable info without debug info, but it has more strict rules than Android 7 when trying to access registers, so we can't get variables on 11 now, but I'm working on it, it's not hard to get it done.
For debugging, you must have

adb
a debuggable app, or a rooted device/emulator that can make apps debuggable.

ps: this debugger will search ANDROID_HOME and PATH to look for adb.
Features:

single step
set breakpoint
modify register value

Possible Features:
let's see...
Short cuts:

f2 set/remove breakpoint
f7 step into
f8 step over
shift + f8 step out
f9 run

Demo
setting up an app to debug

adb usage

debugging

Great work @LBJ-the-GOAT.
Will it be possible to get register values in android 9/10?
Also, can we use these icons instead? :)

@Surendrajat

Everything works with Android 8.1 AVD. Will it be possible to get register values in android 9/10?

Thank you for testing for me, that's so great!
Android 9/10 have different way to handle getRegister request, they look into the debug info to check the type of register we requested, unless the app has the info otherwise they just reject directly.

Unchecking Show Dalvik Bytecode messes up breakpoints and all. Can it be disabled/hidden during debugging?

Okay, I will fix it

Debugging related icons can be improved. Can we use these icons instead? :)

Haha, I grabbed them online randomly.
I will check these icons out.
Thank you.

@LBJ-the-GOAT thanks!
Did you copy any code or resources from other projects (like ADB, JDWP classes and icons)?
If yes, please add corresponding licenses to NOTICE file in jadx root.
Also can you provide description how to regenerate JDWP class, link to jdwp.py script not working :(
Or maybe we can use a library with such code? Did you try to search it? I am really concerned about this big generated class :(

Can we use these icons instead? :)

@Surendrajat many icons I take from Eclipse JDT UI project (git mirror). Maybe these will fit better.

@skylot Relex, I wrote all the code myself including the jdwp-gen.py, and this poor script it was good, one day morning it just disappeared, I sincerely believe that it was accidentally deleted because I named it temp.py, and I realised this when I was trying to push it to github, it was too late. I can delete this line though.
And as for the icons I believe they don't need any primits and crediting, some of them are laying on my driver for a long time. But I think your suggestion

many icons I take from Eclipse JDT UI project (git mirror). Maybe these will fit better.

is better.

@skylot

maybe we can use a library with such code?
I had thinked of it, but don't know how to do it? Can you help me?

maybe we can use a library with such code?

I mean maybe someone already created java library for this jdwp stuff :)

maybe we can use a library with such code?

I mean maybe someone already created java library for this jdwp stuff :)

I found a tutorial, I'll try to make it a library when this PR is ready to merge. :)

@LBJ-the-GOAT I have few suggestions(possible improvements):

Smali code is using parameter registers p0, p1,.. naming convention but debugger view is using normal ...,vN-2,vN-1,vN naming convention. Have p0,p1,.. really helps with keeping track of the input params
So can the last registers in debugger view be renamed to match the same (although I see the parameter name itself in parentheses)?

Since you're already searching for launcher activity, can the target app be started from Jadx itself using adb shell am start -n com.package.activity? and can the target process be automatically found by searching the process list for package name?
This will reduce the time taken by manually starting the app and selecting the process but more importantly, it will allow it to have a Restart Debug feature.
What do you think?

@Surendrajat  Good ideas, Let machine does its jobs is the right thing to do. :D

@LBJ-the-GOAT cool. In that case, current (manual process selection) view can be shown if it fails to start activity or find the process.

@LBJ-the-GOAT tried the above commit(yeah, I was waiting for it :D) and here are my observations:

Parameter registers names (p0,p1..) are shown as well as variable names :+1:
Can restart debug session from debugger itself :+1:
Getting few exceptions in logs (Android 8.1) maybe fixable if you want to check: https://del.dog/jadx.txt
Works on Android 10 too but not able to get/set register values (expected)
The main Debug icon from Toolbar is clickable even when there're a debug session already running and selecting debug again(while one in process), it messes up things. So can the Debug icon be disabled once a debug session is started?
Can the Locate App thing be improved to be automatically locate and select package name after connection is established? and if the process is not running start it automatically even on first time? I think this way Locate App can be renamed to Debug App and user just have to hit that button(for confirmation) because app is automatically started and selected by Jadx.
For restart debug, I think, it doesn't need the Locate App thingy as well because it knows what its restarting and it can seamlessly start a debug session without user interaction. Taking few seconds to restart the app and locating the process in background is very much acceptable :)

Overall, I love this feature and I really like how it's coming together. Thanks a lot!

@Surendrajat  Hi, glad to know you were expecting this commit 😀

Getting few exceptions in logs (Android 8.1)

The Failed to read packet exception occurrs when Socket Connection was closed, may be the app exited or any othert reasons that could break this connect.
jdwp handshake failed may be the app was in the middle of handshake or being debugged, anyway it's occupied, it happens sometime, I haven't figured it out why, the best way to do is to kill the app and run again.

Works on Android 10 too but not able to get/set register values...

It takes debug_info to access and modify registers, so there's nothing I can do...

The main Debug icon from Toolbar is clickable...

May be a message box warning is better, sometimes people could pick the wrong app to debug, leave it clickable so we can restart again right away.👍

Can the Locate App thing be improved to automatically locate and select package ...

I was going to make it select package automatically, But some apps have quite large AndroidManifest.xml to decode, on a regular machine it could take seconds, if do it in background it may hurt user experience, I thought a Locate App  button could make people feel more responsive, but a Launch App button seems more useful, so I will change it next commit😁

For restart debug, I think, it doesn't need the Locate App thingy as well because it knows ...

Yeah, that's because adb shell am doesn't return pid, so I have to get process list from adb again, sometimes, the app on the remote machine couldn't start up in time even thought I have set a delay for it, so when time's up the AdbDialog has to show up otherwise propel will wondering what the heck is going on. I added Locate App partly for this reason, so that as soon as the dialog showed up people can select the process they want.
I'll try to improve this problems anyway, Thanks again for your advices! 👍👍 Have a good day.

@skylot  I've changed the icons to famfamfam's and made jdwp.java a library, Do you think it's ready to be merged?

@LBJ-the-GOAT code look good, thanks.
One little bug: debug does not work for me on linux, on windows it is okay, but on linux it stuck on waiting jdwp handshake reply. Not sure why this happens. Can you check this?
Anyway, I think this PR is ready to merge. Any small issues we can fix in next PRs, also this one become so big that it is hard to review it on updates 😢

@skylot Not sure if it's something Linux specific because I am also using Linux(Ubuntu 18.04) and it works.

@skylot  Sorry, I don't have a Linux machine.
On Windows, if it stuck at jdwp handshake stage that's because the remote app is occupied by another debugger, especially Android Studio, when it's running it beats other debuggers, I don't know what kind of black magic it used.
Anyway if it stuck, kill the remote app can end the waiting. May be we should use a thread to do the handshake job, so the ui doesn't freeze for this kind of problems.

@LBJ-the-GOAT thanks!  Just running Android Studio in background breaks debug in jadx :disappointed:
To fix hang up on reply wait it is possible to add socket timeout: socket.setSoTimeout(5000);, so it will throw exception if connect failed.

@skylot @LBJ-the-GOAT
Sorry, I didn't find my previous comment related to command injection. Yeah, you were right, that command will be executed on the device, but not the host computer, so it should be fine :)


Adding form.js to dist concat

form.js wasn't included into the grunt dist concat. Also added some tests to cover the data-remote feature of form.js.

LGTM
cc @Ximik


See if prices has updated since last check

See if the prices has changed since last check through a simplified script, that can be used as a single-run script.

background.py checks this now 3cea085


Return to plates button doesn't work in Add plates page

Hi,
I don't know why, but the button in "add plate" page to back to "plates" page doesn't work.
But I check in Nextion Editor and in addPlate.js, it's the same button.
Don't know why :/

@magnazi
There are many reasons about cannot return to a page:
1、TouchScreen downed(Threw exception) and you cannot do anything in the screen until you reboot it.
2、The button took wrong callback.
3、The screen received incorrect data/TouchScreen stocked before sending page ID.


$provide.alias for aliasing (or renaming...?) injectables

Hi,
It's a minor issue, but there's an injectable from a third-party module that I need to use but I don't like how it's named (they decided to ClassCase the name although it's not a constructor, which disagrees with my naming conventions). In this scenario, among others, e.g. a naming conflict, it'd be helpful to be able to intuitively rename or alias injectables during the config stage.

Is there any existing means within angular to achieve this?
Would $provide.rename and/or $provide.alias methods be a good solution to this?

$provide.rename('newName', 'oldName')

Wouldn't this work?
.factory('newName', function(OldName) {
   return OldName;
});

We could add a sugar for this of course, but given how people are freaking out about already existing sugar (value, service etc.)...

Hmm yes, that would be equivalent to alias. It's still not smack-in-the-face obvious what it's doing at a glance though, if you're glancing through a lot of code.
What led me to this was angular 2's bind(..).toAlias(..), which reads so nicely.
oldName would still be on the injector though, so that couldn't be used for naming collisions.

@matthewjh we can't simply rename, IMO, since authors of the original module would be injecting using the old names. So solving the name collision / renaming pb is a different story and would be more complex as compared to simple aliasing

Yeah, that makes sense.

I feel that the workaround for adding an alias that @pkozlowski-opensource suggested is good enough. It is very short and if it is not clear then a simple comment would suffice.
As already agreed we would not be able to do a rename.


Missing SQS import

This is on prod-nys-sms:
2020-08-28T16:42:36.837-04:00

Copy
2020-08-28T20:42:36.837Z	6698b7e0-6129-5ec8-89aa-e0230f5a1213	ERROR	Invoke Error
{
    "errorType": "ReferenceError",
    "errorMessage": "sqs is not defined",
    "stack": [
        "ReferenceError: sqs is not defined",
        "    at Runtime.exports.handler (/var/task/sms.js:47:7)",
        "    at process._tickCallback (internal/process/next_tick.js:68:7)"
    ]
}

fixed


chore: update splash screen with new swiss post logo

#opensourceday

The Splashscreen should be updated in all our README.md files under packages/ accordingly:

https://github.com/swisspost/design-system/blob/main/packages/changelog-github/README.md
https://github.com/swisspost/design-system/blob/main/packages/components-angular/projects/intranet-header/README.md
https://github.com/swisspost/design-system/blob/main/packages/components/README.md
https://github.com/swisspost/design-system/blob/main/packages/demo/README.md
https://github.com/swisspost/design-system/blob/main/packages/documentation/README.md
https://github.com/swisspost/design-system/blob/main/packages/icons/README.md
https://github.com/swisspost/design-system/blob/main/packages/internet-header/readme.md
https://github.com/swisspost/design-system/blob/main/packages/styles/README.md

The Splashscreen should be updated in all our README.md files under packages/ accordingly:

https://github.com/swisspost/design-system/blob/main/packages/changelog-github/README.md
https://github.com/swisspost/design-system/blob/main/packages/components-angular/projects/intranet-header/README.md
https://github.com/swisspost/design-system/blob/main/packages/components/README.md
https://github.com/swisspost/design-system/blob/main/packages/demo/README.md
https://github.com/swisspost/design-system/blob/main/packages/documentation/README.md
https://github.com/swisspost/design-system/blob/main/packages/icons/README.md
https://github.com/swisspost/design-system/blob/main/packages/internet-header/readme.md
https://github.com/swisspost/design-system/blob/main/packages/styles/README.md

Can do that, do you want Giorgio on all SplashScreens or just on the main one?


Not keep mesh name/node names

Hi!
I need to manipulate the meshes after load, getting by the mesh name, but even when -kn keep modifing the mesh names.
gltfpack -kn -c -i ./obj/B03.glb -o ./obj/Botella.glb
The names I need to receive (printed by model without gltfpack)

The names I receive

What I'm doing wrong?

Well, inspecting deep I found what is happening.
What gltfpack do is create an Object3D, assign it the name and extras data, and make de mesh children of that Object.

Is there any way of avoid this behaviour? I mean, respect the hierarchy.

There's no way to avoid inserting extra objects; this is necessary to specify the additional quantization transforms. Since you still can locate the notes by name, I'd recommend using the node name instead of the mesh name; unfortunately three.js somewhat conflates the two, but you can always look at the parent of the mesh object and run all the necessary logic that way.

See also https://github.com/zeux/meshoptimizer/discussions/244, https://github.com/zeux/meshoptimizer/issues/252, https://github.com/zeux/meshoptimizer/issues/201

Impressive library! For what it's worth, + 1 from my side to be able to keep the hierarchy / mesh names.
This limits using meshoptimizer in my use case where consistency of mesh names is important.


Proposal: use benjamn/recast instead of acorn and escodegen

Related to #32 .
I'm working on transforming our ES5 code to ES2015, typically var to let and const. We want to maintain the whole structure of our code except those changes, so benjamn/recast is better for our use.
So I tried to replace acorn, escodegen to recast and it worked fine in local.
This is only a proposal so I've not fixed test code yet, but I'd like to do it if you want to merge this.
Thanks.

Cool! It's no surprise some tests are failing, but I'm happy to help debug anything that seems like it might be recast's fault.

Thank you @qsona for you help and thank you @benjamn for awesome recast. It works perfectly and generates a neat human-readable code!
It's time to start working on a new version considering usage of recast with so much more features.

Thank you for the fast work! Your project helps us very much.

@mohebifar I'd like to report that my purpose has been achieved by using your project.
https://github.com/qsona/xto6-let
Thank you again for your great work. :smile:


Description

We should audit the provider scripts to see which of them do not get the filesize and update them.
After that, we should run a backfill to get the file size information from providers, probably also updating the width, height and filetype at the same time.

Adding a note here that filesize is not available in some provider API responses. It doesn't seem necessary to provide filesize value for all items because frontend performance does not rely on it (unlike the image dimensions, for example). Getting the filesize through additional HEAD requests during the catalog ingestion will slow down the process. The API consumer can also get the filesize using a HEAD request if they need it.
It is not possible to backfill the filesize. Unlike the filetype, which we can extract from the media URL that already exists in the database, filesize information needs to be ingested from the providers. So, I am going to close this issue as solving https://github.com/WordPress/openverse-catalog/issues/536 should be enough.


Allow selecting a cell set directly in Vega bar charts

User story
A user may find a cell set of interest in the Vega bar chart and wants to select it to see it in other views. Currently, a user has to use the Cell Sets views for the selection, but it would be more convenient if it can be selected directly in the Vega bar chart.
Preferred solution
Clicking on a bar select the corresponding cell set and clicking again deselect it, similar to the functionality of the checkboxes in the Cell Sets views.
Possible alternatives

Fixed in https://github.com/vitessce/vitessce/pull/1469


add labels and ordering for transform propagation and parent update systems in POST_UPDATE stage

fixes #1450
using system name as label, and setting order for transform propagation to happen after parent updates

oh just noticed #1438 that does add one label, but no ordering so doesn't fix the issue

There is also #1198 that made them run in different stages.

See also #1423, which should be ready to merge and will change the types of our labels.

IMO this should be merged ASAP @cart to make the development experience on master usable for plugin writers, and then we can clean up the labels after #1423 is merged.

Looks good to me! We can decide if / how we export labels later.


Add lll linter

Summary

Change Type

Check List Before Merging

[ ] This PR passes all pre-commit hook validations.
[ ] This PR is fully tested and regression tests are included.
[ ] Relevant documentation has been updated.

Additional Information

I stopped reviewing this PR halfway through because this is essentially 2 PRs in one and each of them touches so many lines that I had a hard time keeping track of everything:

adding lll and fixing line length accordingly
replacing all instances of inferface{} with any

Can you please split them into separate PRs? :pray:

Split the PRs apart


Unable to login as tenant

When logging in as a tenant, I get The username or password provided is incorrect. Make sure you are logging in with your Office 365 account.
Administrator login works perfectly.
The tenant credentials are definitely correct, and using them in the Veeam Explorer for Exchange work ok.
Is there any diagnostic logging that can be enabled?

A tenant is an organization which has been added so you will need to use the same login and it should work. If you add<EMAIL_ADDRESS>as the organization you will need to login with that account as well. It is not possible to login with<EMAIL_ADDRESS>(only the main login works).

Thanks, this is with the admin account for the organization I was trying.
I have attempted<EMAIL_ADDRESS>and reset the job in VBO365 to use the onmicrosoft.com username as well.
Also tested another dummy organization with<EMAIL_ADDRESS>from scratch, and still get the same error I'm afraid.

I would suggest looking in the RESTful logs which can be found under C:\ProgramData\Veeam\Backup365\Logs (file looks like Veeam.Archiver.REST_2018_07_24_15_50_33.log). Try to login, open the file and check the error you are getting. It should explain what goes wrong.

Thanks managed to track it down from the log - we did not have the explorer for Exchange/Sharepoint installed on the VBO365 machine, installed and all working! Cheers.


Documentation for PR #797 in demo app -  Collector Data Flow Dashboard

Documentation for https://github.com/open-telemetry/opentelemetry-demo/pull/797 on opentelemetry demo application repo.

Preview: https://deploy-preview-2538--opentelemetry.netlify.app/docs/demo/collector-data-flow-dashboard/

Overall LGTM, could you use code fences for the names of the metrics listed here?

fenced

Currently the newly added page doesn't render (so we can't preview it) because it has no Hugo front matter. I can suggest some front-matter content later if you'd like, but my first question is, where is the new page meant to fit in the overall demo docs? IMHO, we don't want it at the top level.
@open-telemetry/demo-approvers have any comment about this?
The newly created images shouldn't be in the screenshots folder unless they show up in the Screenshots page.

I did not find a better place inside this directory. I am happy to receive a recommendation on this one.

I did not find a better place inside this directory.

No worries, that's fine. Waiting on @open-telemetry/demo-approvers and others to weigh in.

Currently the newly added page doesn't render (so we can't preview it) because it has no Hugo front matter. I can suggest some front-matter content later if you'd like, but my first question is, where is the new page meant to fit in the overall demo docs? IMHO, we don't want it at the top level.
@open-telemetry/demo-approvers have any comment about this?
The newly created images shouldn't be in the screenshots folder unless they show up in the Screenshots page.

where is the new page meant to fit in the overall demo docs?
@cartersocha and @puckpuck any ideas on that?

Glancing through the current TOC. I think it’d be nice to have a dashboard folder or something similar. We could do similar write ups for the other pre existing dashboards?

Glancing through the current TOC. I think it’d be nice to have a dashboard folder or something similar. We could do similar write ups for the other pre existing dashboards?

Thanks all for your feedback.
If you feel that other dashboards will be documented soonish, then I'd vote to create Dashboards section with the single page in the PR as an entry. Otherwise, we can leave the page as it is (at the top level) and move it later as other dashboards are added. Your call @cartersocha et al.

Thanks for the updates @devrimdemiroz. We're getting closer!
I've unresolved some of my comments since they haven't been addressed. Please avoid marking a comment as resolved unless the required changes have landed in the PR. If there are questions or doubts, leave it unresolved so that the comment author or maintainers can make the assessment and update the status as required or follow up with more comments.
The main issue that's left is image placement. We generally want to avoid starting a section with an image. In all cases in the page, the opening paragraph is short enough that the image can be placed after the opening paragraph. Please make that change for all images, and adjust the paragraph text to refer to the "image below" rather than "above.
I'll have a few final copyedits to share after that. Thanks!

@chalin ,
I appreciate the thorough review you've given to the document. In my case, I believe that placing the diagram first offers some distinct advantages:
Context: Providing the diagram upfront supplies readers with a visual context before delving into the explanation, simplifying their comprehension of the content.
Engagement: Introducing a visual aid at the beginning captures readers' attention and piques their curiosity, increasing the likelihood that they'll engage with the subsequent text.
I hope you can understand my perspective on this. If it's not a strict requirement, I'd prefer to maintain the current format. While I recognize that there's no one-size-fits-all rule, my choice to place the diagrams first was deliberate. By investing in high-quality visuals, I aim to ensure that the related text is not overlooked and that the reader benefits from the full context.
Would you consider being open to this approach, as it may lead to a more engaging and effective document for our readers? I truly believe that this layout has the potential to enhance their overall experience.
Looking forward to your thoughts.
PS: Not limited but my favourite mentor on the subject 👉🏾 https://youtu.be/VwocXCyOieM

Thanks for the explanation @devrimdemiroz.
I agree with the advantages you are stating (engaging visuals etc) and IMHO, these advantages aren't lost by placing the images 2-3 lines further down the page.
One of the hats that I wear for this project is that of technical writer. In that role, I do my best to assure some form of consistency in the website content presentation, reasonably enforcing "best / recommended practices" that I've accumulated over the years. (Like trying to avoid "widowed or orphaned" headings.)
I'll let @svrnm @cartermp and @austinlparker share their thoughts, but for sizeable open-source projects like OTel, consistency in style and formatting across content (blog post are given a bit more flexibility), trumps personal style/preference, otherwise we'd end up with a degraded reader experience due to the use of disparate styles. Consistency is also why, for example, we've enforced the use of Prettier to format file content.
The only other solution I see, to keep the images first, would be to place the image in a figure before the heading, and then refer to the figure by number or title.
Cheers!

I'd agree with @chalin -- we should strive for consistency.

The Individual CLA option is not enabled for this project. Please to your administrator to enable the Individual CLA option for this project.
I wonder why individual CLA is not enabled 🧐

Hi @devrimdemiroz. Thanks for making the updates.
The CLA issue seems to be because you submitted the last commit from a different user ID, for which you haven't signed the CLA:
$ g log -2
commit 121862d7923790ade0d3628a7d24cf43a229f31f (HEAD -> feature/otecol-data-flow-dashboard-PR797)
Author: Devrim<EMAIL_ADDRESS>Date:   Fri Mar 31 09:21:34 2023 +0200

    high above towards down below

commit c7a72d123618b886052ca43ab330f8581ffb04ae
Merge: f3d9ed5b 146de686
Author: Devrim Demiroz<EMAIL_ADDRESS>Date:   Fri Mar 31 09:09:57 2023 +0200

    Merge branch 'open-telemetry:main' into feature/otecol-data-flow-dashboard-PR797

I see two way's forward. Sign the CLA for the ID used in https://github.com/open-telemetry/opentelemetry.io/commit/121862d7923790ade0d3628a7d24cf43a229f31f, or retract the commit and resubmit it using the same ID you did all the other work with.

I do not recall such an error on opentelemetry-demo repository.

I just checked the demo repo, and there you never used your<EMAIL_ADDRESS>ID.

In any case, I made a new dummy commit that should resolve the issue. But still, EasyCLA check is in Required state.

@devrimdemiroz I don't think the dummy commit helps since EasyCLA appears to check every commit. You could probably fixup that commit with an interactive rebase or re-open a new PR with the current document state (and none of the preceding commits). Either way, this is good enough for me to merge in when the license check is dealt with. I'd prefer not to use admin privilege to bypass that.

@devrimdemiroz I don't think the dummy commit helps since EasyCLA appears to check every commit. You could probably fixup that commit with an interactive rebase or re-open a new PR with the current document state (and none of the preceding commits). Either way, this is good enough for me to merge in when the license check is dealt with. I'd prefer not to use admin privilege to bypass that.

I concluded same. I will go for opening a new PR and close this one. Thanks!

Opened a new PR to bypass EasyCLA https://github.com/open-telemetry/opentelemetry.io/pull/2569
Closing this one...


Not listening to internal K8/Cluster requests

With Traefik 2, using Metalb,  from within a pod, curl https:// produces:
curl: (7) Failed to connect to  port 443: Connection timed out
I would imagine this is caused because Traefik is set to listen to the LM IP/Ports and perhaps not on the cluster IP but is there a way to configure this?  Either in K8 or Traefik?  Note - my internl/pod ping resolves to the Traefik LM IP.
Thanks

Hi! I'm Træfiker :robot: the bot in charge of communication regulation.
Thanks for your interest in Traefik!
We dedicate the issue tracker to bug reports and feature requests only. My advanced AI has spotted that your issue might be a configuration problem or relates to something that doesn't look like a bug.
To confirm this, please join our Community Forum and reach out to us on the Traefik section.
In case I'm wrong (well, that would be embarrassing :sweat_smile:), my developers will re-open the issue and fix me!
In the meantime, you can double check Traefik's documentation.


Accept array of transactions to sign

Right now if transaction is bigger that size limit, user needs to accept each transaction separately. It would be ideal to allow signTransaction to take list of transactions that user can approve with one click.

@nathanielparke what do you think about this change?

Ah yep agreed this makes a ton of sense

The change proposed in #60 adds an additional signAllTransactions method, instead of changing the existing one to avoid breaking existing clients.

Completed by #60.


Hook to expose committed TTS audio before playing back to user

I'm looking for an elegant way to save the audio output of the Text-to-Speech (TTS) system, specifically the audio that is played back to users during interactions, as well as the audio recorded from users speaking. Having these audio files saved would greatly help in reconstructing the live experience.
Is there a method or existing implementation that allows for automatic saving of this audio data? Additionally, does LiveKit provide any built-in functionality for automatically saving these audio recordings?
Any guidance or examples would be greatly appreciated!
I use the LiveKit Cloud, but there's nothing in my Egress web console. But the docs told me that Egress is ready to use without additional configuration for cloud customers.

@BaiMoHan thank you for the suggestion. To record the conversation, you can use Egress APIs to record the conversation in Cloud. We support compositing the entire session in a single recording, or recording each participant individually.
However, we'll want to make it possible within the framework to expose the audio buffers before the speech is played out to the user. I'll use this issue to track progress for that feature.


Restrict to numpy<2

numpy 2.0.0 breaks tests

Sounds good to me.
The issue seems to be through estimate_stabilization -> pystackreg -> turboreg. If the tests pass here, we can open an issue upstream in pystackreg.


[zh] Resync kubelet credentialprovider.v1alpha1

Translate Chinese version for reference/config-api/kubelet-credentialprovider.v1alpha1.md
Comments
/language zh
/assign

/triage accepted

I think this issue can be included in #34150 next time.

#34150


docs: Add KubeCon NA 22 Demo

closes #304
Signed-off-by: Thomas Schuetz<EMAIL_ADDRESS>

Codecov Report
Base: 5.03% // Head: 5.03% // No change to project coverage :thumbsup:

Coverage data is based on head (59c4db6) compared to base (a8f58a4).
Patch has no changes to coverable lines.

Additional details and impacted files
@@          Coverage Diff          @@
##            main    #308   +/-   ##
=====================================
  Coverage   5.03%   5.03%
=====================================
  Files          9       9
  Lines        794     794
=====================================
  Hits          40      40
  Misses       749     749
  Partials       5       5

Flag
Coverage Δ

keptn-lifecycle-operator
5.59% <ø> (ø)

scheduler
0.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


betas for logging-{winston,bunyan}, error-reporting

These modules can now be declared in beta.

Coverage remained the same at 99.988% when pulling fedd0fec477c4465772f52db35b1b10d39913940 on ofrobots:betas into 4b5158e64fd80df221bce1d736e530c7e5edec25 on GoogleCloudPlatform:master.

@GoogleCloudPlatform/node-team PTAL.

@stephenplusplus I believe this is ready to land (and release on Monday)


libjxl.so.0.9 missing

[x] I have read the FAQ.
[x] I have searched in existing issues.

Environment

OS: Archlinux
scrcpy version: 2.3.1
installation method: pacman
device model:
Android version: 11

Describe the bug
A clear and concise description of what the bug is.
scrcpy: error while loading shared libraries: libjxl.so.0.9: cannot open shared object file: No such file or directory

There is only libjxl.so.0.10 in my /usr/lib
libjxl.so -> libjxl.so.0.10
libjxl.so.0.10 -> libjxl.so.0.10.1
libjxl.so.0.10.1
libjxl_cms.so -> libjxl_cms.so.0.10
libjxl_cms.so.0.10 -> libjxl_cms.so.0.10.1
libjxl_cms.so.0.10.1
libjxl_extras_codec.so -> libjxl_extras_codec.so.0.10
libjxl_extras_codec.so.0.10 -> libjxl_extras_codec.so.0.10.1
libjxl_extras_codec.so.0.10.1
libjxl_jni.so
libjxl_threads.so -> libjxl_threads.so.0.10
libjxl_threads.so.0.10 -> libjxl_threads.so.0.10.1
libjxl_threads.so.0.10.1

Please report to the author of the arch package.


chore: add "mender-prepopulate-inactive-partition" feature

The "mender-prepopulate-inactive-partition" feature has been added as MENDER_FEATURE to meta-mender, respectively Yocto. This adds the corresponding entry to the image customization features list.
External Contributor Checklist

🚨 Please review the guidelines for contributing to this repository.

[ ] Make sure that all commits follow the conventional commit specification for the Mender project.

The majority of our contributions are fixes, which means your commit should have
the form below:
fix: <SHORT DESCRIPTION OF FIX>

<OPTIONAL LONGER DESCRIPTION>

Changelog: <USER-FRIENDLY-CHANGE-DESCRIPTION> or <None>
Ticket: <TICKET NUMBER> or <None>

[ ] Make sure that all commits are signed with git --signoff. Also note that the signoff author must match the author of the commit.

Description
Please describe your pull request.
Thank you!

Thank you


Modifying log contents in hadoop-tools/hadoop-archive-logs/src/main/java/org/apache/hadoop/tools/HadoopArchiveLogs.java

The following log line  LOG.info("Existing Working Dir detected: -" + FORCE_OPTION +             " not specified -> exiting"); evaluated against the provided standards: 1. The log line does not include any parameters. In this case, it would be beneficial to include the path of the working directory (workingDir) in the log message. 2. The log line does not include sensitive information. 3. The log message is concise and informative. 4. The log message is not for an exception. Due to the violation of standard (1), we would recommend a code change to include the 'workingDir' variable in the log message.

Created by Patchwork Technologies.

👍


Use same string on failed login regardless of whether account exists when in paranoid mode

Previously reported in #4568. devise is using devise.failure.invalid when you attempt to log in with an existing identifier but a wrong password, while it uses devise.failure.not_found_in_database when the identifier does not exist. In the default en.yml provided by devise, these strings are the same, but if you are using a different localization file, they may not be (see https://github.com/tigrish/devise-i18n/issues/223).
If you have enabled paranoid mode, the same localization key should be used for both cases.

I haven't understood what the problem was on the previous issue, but now I do 😅 . Thanks, @JasonBarnabe.
This change seems reasonable to me, but I think we would have to do it in a major version since it isn't backward compatible.
Would you be willing to work on this?

Closed via https://github.com/plataformatec/devise/pull/4854


Haste task protractor fixes

Change the webdriverManagerArgs<Array> option to be webdriverManagerOptions<Object>
Fix the merge between the passed options with the default options.
Refactor tests

Codecov Report

Merging #143 into master will increase coverage by 0.18%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #143      +/-   ##
==========================================
+ Coverage   95.37%   95.56%   +0.18%
==========================================
  Files          23       24       +1
  Lines         238      248      +10
  Branches       30       31       +1
==========================================
+ Hits          227      237      +10
  Misses         10       10
  Partials        1        1

Impacted Files
Coverage Δ

packages/haste-task-protractor/src/index.js
100% <100%> (ø)

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 e75085d...f72d412. Read the comment docs.


Auth Trigger Function environment variables not configured correctly

Describe the bug
When creating an Auth PostConfirmation function for a coginito the following environment variables are not correctly:  API_<APPID>_GRAPHQLAPIENDPOINTOUTPUT   and   API_<APPID>_GRAPHQLAPIIDOUTPUT
The values are
API_<APPID>_GRAPHQLAPIENDPOINTOUTPUT   = api<APPID>GraphQLAPIEndpointOutput
API_<APPID>_GRAPHQLAPIIDOUTPUT = api<APPID>GraphQLAPIIdOutput

Amplify CLI Version
# amplify -v
4.41.1

To Reproduce
My steps using amplify update auth to the Post Confirmation lambda function.
# amplify update auth
Please note that certain attributes may not be overwritten if you choose to use defaults settings.

You have configured resources that might depend on this Cognito resource.  Updating this Cognito resource could have unintended side effects.

Using service: Cognito, provided by: awscloudformation
 What do you want to do? Walkthrough all the auth configurations
 Select the authentication/authorization services that you want to use: User Sign-Up, Sign-In, connected with AWS IAM controls (Enables per-user Storage features for images o
r other content, Analytics, and more)
 Allow unauthenticated logins? (Provides scoped down permissions that you can control via AWS IAM) No
 Do you want to enable 3rd party authentication providers in your identity pool? No
 Do you want to add User Pool Groups? No
 Do you want to add an admin queries API? No
 Multifactor authentication (MFA) user login options: OFF
 Email based user registration/forgot password: Enabled (Requires per-user email entry at registration)
 Please specify an email verification subject: Your verification code
 Please specify an email verification message: Your verification code is {####}
 Do you want to override the default password policy for this User Pool? No
 Specify the app's refresh token expiration period (in days): 30
 Do you want to specify the user attributes this app can read and write? No
 Do you want to enable any of the following capabilities?
 Do you want to use an OAuth flow? No
? Do you want to configure Lambda Triggers for Cognito? Yes
? Which triggers do you want to enable for Cognito (Press <space> to select, <a> to toggle all, <i> to invert selection)
❯◯ Learn More
 ──────────────
 ◯ Create Auth Challenge
 ◯ Custom Message
 ◯ Define Auth Challenge
 ◯ Post Authentication
 ◉ Post Confirmation

Also, run amplify update function and add permission to access the API.
Once the above is configured, then
# amplify push

Then from the console check your environment variables on the lambda configuration screen.
...
Expected behavior
A clear and concise description of what you expected to happen.
The environment variable values for the lambda
API_<APPID>_GRAPHQLAPIENDPOINTOUTPUT   =  https://<GRAPH_HOSTBNAME/graphql
API_<APPID>_GRAPHQLAPIIDOUTPUT = <RANDOM_ALPHANUMERIC>

Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. Mac/Windows/Ubuntu]
Node Version. You can use node -v to check the node version on your system

Mac os 11.1
# node -v
v10.23.0

Additional context
Screen shot of lambda environment configuration

Add any other context about the problem here.

@wderezin I was not able to reproduce it on my end since doing the same steps I had correct env vars:

When you see the strings you see they are coming from the "DefaultValue" specification inside of the Function's CFN template:
    "apifffffffGraphQLAPIIdOutput": {
      "Type": "String",
      "Default": "apifffffffGraphQLAPIIdOutput"
    },
    "apifffffffGraphQLAPIEndpointOutput": {
      "Type": "String",
      "Default": "apifffffffGraphQLAPIEndpointOutput"
    },
    "apifffffffGraphQLAPIKeyOutput": {
      "Type": "String",
      "Default": "apifffffffGraphQLAPIKeyOutput"
    }

And the value for these are being supplied from the root stack template in amplify/backend/awscloudformation/nested-cloudformation-template.yml:
          "apifffffffGraphQLAPIIdOutput": {
            "Fn::GetAtt": [
              "apifffffff",
              "Outputs.GraphQLAPIIdOutput"
            ]
          },
          "apifffffffGraphQLAPIEndpointOutput": {
            "Fn::GetAtt": [
              "apifffffff",
              "Outputs.GraphQLAPIEndpointOutput"
            ]
          },
          "apifffffffGraphQLAPIKeyOutput": {
            "Fn::GetAtt": [
              "apifffffff",
              "Outputs.GraphQLAPIKeyOutput"
            ]
          },

And these are dynamic values being replaced with the actual values from the GraphQL APIs stack.
Could you please double check if you have the same pieces in your project?

@wderezin I was not able to reproduce it on my end since doing the same steps I had correct env vars:

When you see the strings you see they are coming from the "DefaultValue" specification inside of the Function's CFN template:
    "apifffffffGraphQLAPIIdOutput": {
      "Type": "String",
      "Default": "apifffffffGraphQLAPIIdOutput"
    },
    "apifffffffGraphQLAPIEndpointOutput": {
      "Type": "String",
      "Default": "apifffffffGraphQLAPIEndpointOutput"
    },
    "apifffffffGraphQLAPIKeyOutput": {
      "Type": "String",
      "Default": "apifffffffGraphQLAPIKeyOutput"
    }

And the value for these are being supplied from the root stack template in amplify/backend/awscloudformation/nested-cloudformation-template.yml:
          "apifffffffGraphQLAPIIdOutput": {
            "Fn::GetAtt": [
              "apifffffff",
              "Outputs.GraphQLAPIIdOutput"
            ]
          },
          "apifffffffGraphQLAPIEndpointOutput": {
            "Fn::GetAtt": [
              "apifffffff",
              "Outputs.GraphQLAPIEndpointOutput"
            ]
          },
          "apifffffffGraphQLAPIKeyOutput": {
            "Fn::GetAtt": [
              "apifffffff",
              "Outputs.GraphQLAPIKeyOutput"
            ]
          },

And these are dynamic values being replaced with the actual values from the GraphQL APIs stack.
Could you please double check if you have the same pieces in your project?

I was able to track the issue down in the cloud formation templates.
Below is the configuration for my PostConfirmation function.  Here is the configuration for the function missing the parameters.
    "functionXXXXXXXXX5221197dPostConfirmation": {
      "Type": "AWS::CloudFormation::Stack",
      "Properties": {
        "TemplateURL": "https://s3.amazonaws.com/amplify-XXXXXXXXX-walter-113614-deployment/amplify-cfn-templates/function/XXXXXXXXX5221197dPostConfirmation-cloudformation-template.json",
        "Parameters": {
          "modules": "add-to-group,custom",
          "resourceName": "XXXXXXXXX5221197dPostConfirmation",
          "env": "walter"
        }
      }
    },

It is missing the following Parameters like the other functions configured for API access
          "apiXXXXXXXXXGraphQLAPIIdOutput": {
            "Fn::GetAtt": [
              "apiXXXXXXXXX",
              "Outputs.GraphQLAPIIdOutput"
            ]
          },
          "apiXXXXXXXXXGraphQLAPIEndpointOutput": {
            "Fn::GetAtt": [
              "apiXXXXXXXXX",
              "Outputs.GraphQLAPIEndpointOutput"
            ]
          },

I was able to track the issue down in the cloud formation templates.
Below is the configuration for my PostConfirmation function.  Here is the configuration for the function missing the parameters.
    "functionXXXXXXXXX5221197dPostConfirmation": {
      "Type": "AWS::CloudFormation::Stack",
      "Properties": {
        "TemplateURL": "https://s3.amazonaws.com/amplify-XXXXXXXXX-walter-113614-deployment/amplify-cfn-templates/function/XXXXXXXXX5221197dPostConfirmation-cloudformation-template.json",
        "Parameters": {
          "modules": "add-to-group,custom",
          "resourceName": "XXXXXXXXX5221197dPostConfirmation",
          "env": "walter"
        }
      }
    },

It is missing the following Parameters like the other functions configured for API access
          "apiXXXXXXXXXGraphQLAPIIdOutput": {
            "Fn::GetAtt": [
              "apiXXXXXXXXX",
              "Outputs.GraphQLAPIIdOutput"
            ]
          },
          "apiXXXXXXXXXGraphQLAPIEndpointOutput": {
            "Fn::GetAtt": [
              "apiXXXXXXXXX",
              "Outputs.GraphQLAPIEndpointOutput"
            ]
          },

I checked the PostConfirmation permissions to access the API and enabled read/update/create/delete and now get the following cloud formation circular error:

⠴ Updating resources in the cloud. This may take a few minutes...Error updating cloudformation stack
✖ An error occurred when pushing the resources to the cloud

Circular dependency between resources: [UpdateRolesWithIDPFunctionOutputs, apiXXXXXXXXX, functionXXXXXXXXX5221197dPostConfirmation, functionXXXXXXXXXAnswerUpdate, functionXXXXXXXXXMoreAnswers, authXXXXXXXXX5221197d, UpdateRolesWithIDPFunction]
An error occurred during the push operation: Circular dependency between resources: [UpdateRolesWithIDPFunctionOutputs, apiXXXXXXXXX, functionXXXXXXXXX5221197dPostConfirmation, functionXXXXXXXXXAnswerUpdate, functionXXXXXXXXXMoreAnswers, authXXXXXXXXX5221197d, UpdateRolesWithIDPFunction]

I checked the PostConfirmation permissions to access the API and enabled read/update/create/delete and now get the following cloud formation circular error:

⠴ Updating resources in the cloud. This may take a few minutes...Error updating cloudformation stack
✖ An error occurred when pushing the resources to the cloud

Circular dependency between resources: [UpdateRolesWithIDPFunctionOutputs, apiXXXXXXXXX, functionXXXXXXXXX5221197dPostConfirmation, functionXXXXXXXXXAnswerUpdate, functionXXXXXXXXXMoreAnswers, authXXXXXXXXX5221197d, UpdateRolesWithIDPFunction]
An error occurred during the push operation: Circular dependency between resources: [UpdateRolesWithIDPFunctionOutputs, apiXXXXXXXXX, functionXXXXXXXXX5221197dPostConfirmation, functionXXXXXXXXXAnswerUpdate, functionXXXXXXXXXMoreAnswers, authXXXXXXXXX5221197d, UpdateRolesWithIDPFunction]

I think issue is really "When in enabling a Auth function for API access a cloudformation circular reference is generated'
So I did another test with adding a PostAuthentication function and I caused the same results/error as above with the circular reference.

I think issue is really "When in enabling a Auth function for API access a cloudformation circular reference is generated'
So I did another test with adding a PostAuthentication function and I caused the same results/error as above with the circular reference.

Circular dependency is a side effect since you create a circular reference between auth, api, function resource. Unfortunately it cannot be solved in an automated way with single CF deployment and Amplify created resource references, what you can do as a workaround is to extract out the generated access policy as a custom resource and use amplify to deploy it. Also the env vars has to be set manually for the function.
I'm closing the issue as the original problem was probably because of a misconfiguration of the cross category resource access.
Related issues:
https://github.com/aws-amplify/amplify-cli/issues/1874
https://github.com/aws-amplify/amplify-cli/issues/2303
https://github.com/aws-amplify/amplify-cli/issues/4568

Circular dependency is a side effect since you create a circular reference between auth, api, function resource. Unfortunately it cannot be solved in an automated way with single CF deployment and Amplify created resource references, what you can do as a workaround is to extract out the generated access policy as a custom resource and use amplify to deploy it. Also the env vars has to be set manually for the function.
I'm closing the issue as the original problem was probably because of a misconfiguration of the cross category resource access.
Related issues:
https://github.com/aws-amplify/amplify-cli/issues/1874
https://github.com/aws-amplify/amplify-cli/issues/2303
https://github.com/aws-amplify/amplify-cli/issues/4568


text encoder

is the text encoder the same one that flux uses?

it looks like its looking for a t5 encoder...if its the same why is it hard coded to only accept the one from pixart? i really dont want to have a 2nd copy of the t5 encoder for just this...is there any way you can make it allow other types

you can use regular load clip node and select any t5 encoder


[React] use Router instead of BrowserRouter

This is needed in order to allow using the history object to programmatically move between routes.
This is a simple change, and I'll provide a PR when I get to it. It is explained properly here.
Essentially we need to change the following in index.js:
<BrowserRouter>         <App cqChildren=... />   </BrowserRouter>), document.getElementById('page'));
to
    const history=createBrowserHistory();      ...      <Router history={history}>         <App history={history} cqChildren=... />     </Router>
This will give  access to the history object, allowing it to update routes as a result of state changes.

@ittaibaratz Thanks for using the archetype and your effort to report the issues.
Your proposed improvement sounds good. Looking for your contribution.

Thanks @lydiapuric , pull request created.


README updates for case

Adds a picture of the hub and the case. Writes down what bolts to use to assemble the case. Also fixes a typo I made before.

something went a bit wrong with your typo fix :-P

Wow, force-pushed to fix :)


Support Polis - First approach (Needs further testing)

We need to keep testing. I am getting a tls error to start testing. Can someone test for me and give some feedback.

Lol. Srry, wrong repo.


Remove product from stock

If a product is removed, make sure to remove all occurrences in the stock.

product_notifier.dart/deleteProduct():

add the logic to differ between freezer/cupboard/fridge

product_repository/deleteProduct():

add a transaction to remove the product and remove the entry in the according storage place list

Not needed due to changes in the implementation of products.


[WFCORE-5321] Upgrading xalan to 2.7.1.jbossorg-5

https://issues.redhat.com/browse/WFCORE-5321

The Windows failures are unrelated to this change.

@fjuma FYI we're still seeing the Keystore... failures on Windows.

@ropalka thanks

@bstansberry I'll take another look at the Keystore... failures on Windows.


Feature Request: Step over should not include deferred functions

Hi,
Here's a feature request that came up on Gogland's tracker:
What happens:
The "Step over" function "steps into" defer functions
E.g. Stepping through the Copy function below "Step over" steps twice into Close function.

What I would prefer:
"Step over" should not include deferred functions. (Only "Step Into" should include deferred functions)

The code example:
package main

import (
	"fmt"
	"io"
	"os"
)

func Copy(dst, src string) error {
	in, err := os.Open(src)
	if err != nil {
		return err
	}
	defer in.Close()
	out, err := os.Create(dst)
	if err != nil {
		return err
	}
	defer out.Close()
	_, err = io.Copy(out, in)
	cerr := out.Close()
	if err != nil {
		return err
	}
	return cerr // step over - defer
}

func main() {
	fmt.Println(Copy("demo", "demo"))
}

I'm happy to discuss this further as needed.
Thank you.

I think I agree. The only question I have is whether this new behavior should replace the old one or be selected by a flag in the DebuggerCommand packet.
I think the change is simple, it should suffice to change this lines in pkg/proc/threads.go:
			if bp != nil {
				bp.DeferReturns = deferreturns
			}

into this:
			if bp != nil && stepInto {
				bp.DeferReturns = deferreturns
			}

From a IDE users perspective I think there is no need for a flag. The user can use step-into or set a break point in the deferred function. (Without this change the user has no choice - defered  function are always included)

I also think this shouldn't be yet one more flag but rather just the default behavior. Too many flags would make delve lose it's easy to use feeling it has now imho. I'm happy to send the PR if you think that's a good solution.

Fixed via https://github.com/derekparker/delve/commit/5c9b2009cab214c1f912a2d8c6c89eec9f3ada2b.


Wave with only stroke

Hi, thanks for you great library, is there any possibility to have a wave with just stroke on it? i mean without the fill color, also I assume the xml waveColor does not support alpha channel for opacity right?

Actually i checked your code and modified some codes for myself, I added an styleable key for stroke only or fill and stroke, then in your initPaintPath we can change paint style so it will work easily. I don't know if you will add this option to your code later, but it's a good feature to have. About opacity thing, i just noticed you're changing alphas for multiple waves, so it's not possible to have for multiple waves, but for single waves I changed it for my own needs. Thanks for this great lib, love it...

Great! Would like to see how solved it and if you think it is configurable and something that the library should support out of the box, raise a pull request maybe?


Since we get data from the API, listboxes doesn't reflect url parameters

Describe the bug
Since OS and architectures are retrieved from the API, the listbox are not
To Reproduce

Go to 'https://adoptium.net/fr/temurin/releases/?os=linux&arch=x64'
Scroll down to listboxes
See error

Expected behavior
The listboxes to be setted with "Linux" and "X64"
Screenshots

Fixed


Import your TechEd BUILD project into SAP Web IDE

Hi.
After importing the Build project to SAP Web IDE, although I can start the project (run it as web application), I get an error "Request failed", with the following details:
"{
"requestUri": "/here/goes/your/serviceUrl/local/$metadata?sap-language=EN",
"statusCode": 404,
"statusText": "Not Found",
"headers": [],
"body": "HTTP Status 404 - Not Found"
}"
After checking the connection for all destinations imported a few steps above, none of the destinations replied OK:

Connection to "BUILD_Production" established. Response returned: "404: Not Found"
Connection to "sapui5-private" established. Response returned: "401: Unauthorized"
Connection to "sapui5-private-build" established. Response returned: "401: Unauthorized"
Connection to "Web_IDE_PLUGIN" established. Response returned: "301: Moved Permanently"

Hope will get a positive reply soon.
Best regards,
thank you,
Marius

I have now figured out where is this coming from. It's defined in manifest.json file, on the Data Sources Tab.

Run the application using the Mock Server. Since we dont have any service available, thsi sample url throws an error. Even in the Build tool, we use Mock data. So right click on project, Run configurations -> and select - a new web application and check run with mock data OR run the application on mockserver.

@69cmg as I already said on Twitter I think they made a mistake. In the first steps, you have to run the index web app with mock data. You're going to configure the oData destination only in the latest step.


PARQUET-1846: [C++] Remove deprecated IO classes

LGTM, not at a computer I can merge, but feel free to.  Thanks for the cleanup.

Not necessarily (though they have less features, e.g. no async IO), but there was no advantage in using them either.

Thank you for your answer.
And I have another question, does arrow in the c++ environment support reading or writing ascii-encoded data? If so, through which interface is it implemented?

Arrow IO is binary, it does not care about text encoding.
Also, please direct questions to the mailing-list: https://arrow.apache.org/community/


<IP_ADDRESS>. Reflective Module.ResolveExport() - Review SyntaxError on circular dependencies

Reported by @bmeck (https://github.com/WebAssembly/design/issues/256#issuecomment-120558466)

I believe this refers to circular references in the bindings setup. The point is I can't do:
a.js
export {a} from './b.js';

b.js
export {a} from './a.js';

So we stop circularity of bindings, not circularity of modules.

@guybedford is right: this part of the semantics treats re-export cycles as a static error, not cyclic module dependencies. Re-export cycles are nonsense: in the example above, a is not actually defined anywhere.


add get-blob method to gcio

Very minor PR to add get_blob method.
This allows one to use other methods on the blob (e.g. checksum, or get filesize) that are not directly supported by our gcloud_storage.py methods.
In my scenario, I wanted to test for a filesize of 0 in order to handle an edge case in my download-from-gcloud-to-local-device code. The empty files failed to download.

Coverage decreased (-0.2%) to 52.53% when pulling 5b15dcb02248111768e3d7dfc92fc34171bad9a5 on add-get-blob into c5844cd91a5e27308b59d870b4bf34dfe331e80e on master.


wasm: rationalize data model for execution instances and xDS resources

The relationship between xDS filter config and Wasm execution instances is rather confusing, which makes it hard to provide guidance on scaling (e.g. with respect to heap/stack size per instance). In the simplest case of two identical listeners referring to the same identical WasmFilter config, we see the following:

_on start is only executed once per execution instance, but there are many temporary instances created during configuration, for canary I presume.
on_configure is called twice on the same instance, with the same parameters - this happens even when listeners have distinct listener metadata.
on_request_headers is called on the same instance per each request regardless of the listener.

From basic testing, I notice that pushing an LDS update with new metadata causes a new "plugin" to be created. But that plugin is then reused whether or not the metadata is removed. I think this has to do with "plugin key" override, but it's not quite correct as it only register once.


Add "No Recruiters" verb-age to front page

I think make this more broad to include anyone whose goal is promotion of a job, event, website, blog, etc that they would profit from. I spoke to that a bit in the CoC PR I submitted earlier.

Can this be closed now? I'll assume yes since it's a year old...reopen if needed.


Feature req: Discord notification

Add a possibility in Alerts to send notifications via Webhook to Discord.
T.ex:
Plots startet
Plots finished (time to make etc.)
Coins farmed
Optional hourly status update (or adjustable time updates)

You could just use this repo for a bot: https://github.com/joaquimguimaraes/chiabot
I think he might not add more and more useless features to a simple CHIA client-farmer Docker.

Hey @Allram!  I've integrated Chiadog with Machinaris and users have confirmed discord notifications are working.  Currently on the test branch, will hit release mainline later this week.


Add Golang to 'languages'

Find a way to build Golang from source. Add the Dockerfile for this to the 'languages' directory.

https://github.com/Kingcitaldo125/docker_utils/commit/e6a736a73f1399c197181c57161718c8697bde28


add support enum parameters for groupby and keyby

this pull request add enum support for groupBy and keyBy based on issue #42031
this would be breaking for php 8.0
any suggestions?

Look at this test for enums:
https://github.com/laravel/framework/pull/39315/files
You can conditionally include an Enums.php file with your declared enums, and then skip the tests for lower versions of php.
public function testGroupByWithEnumValue($collection)
{
        if (version_compare(PHP_VERSION, '8.1') < 0) {
            $this->markTestSkipped('PHP 8.1 is required');
        }

This is a new feature I think, not a bug fix.

Look at this test for enums: https://github.com/laravel/framework/pull/39315/files
You can conditionally include an Enums.php file with your declared enums, and then skip the tests for lower versions of php.
public function testGroupByWithEnumValue($collection)
{
        if (version_compare(PHP_VERSION, '8.1') < 0) {
            $this->markTestSkipped('PHP 8.1 is required');
        }

Note: It seems like this should target 8.x as enum support was first added there and this appears to be a bug fix.

is this good enough? or should i conditionally include a file?

@agus24 Looks good to me, tests are passing :)

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.

@taylorotwell i think it's a bug fix since we cant directly use groupby to enum field from database.
for example
enum CustomerStatus: string
{
    case Bussines = 'bussines';
    case Private = 'private';
}

class Customer {
    protected $casts = [
        'status' => CustomerStatus::class,
    ];
}

then when we do
Customer::all()->groupBy('status')

it will cause an error array_key_exists(): Argument #1 ($key) must be a valid array offset type
so we need to workaround something like Customer::all()->groupBy('status.value')
this PR will make groupBy and keyBy to directly without using the .value suffix

This is a new feature I think, not a bug fix.

How is this a feature request and not a bug fix? When using enums on models you are no longer able to groupby that column, that is a feature break in my book. This PR actually fixes it with minimal code. Hoping it can be merged soon.

@agus24 You can try resubmitting this PR. Just include the full explanation of why it is needed when you submit it. (Feel free to use reasons that other people mentioned in this thread plus your full explanation after it was closed.)
You'll have to resubmit this to target 9.x and it will probably be accepted there. I do think it should be considered a bug fix for 8.x as ENUM support was expanded a bunch in 8.x.
However, target 9.x and maybe mention that you think it belongs in 8.x and see what happens.


Unindent top-level {% block %} tags

Description
guide:

Within a template using {% extends %}, avoid indenting top-level {% block %} tags.

Will need touching the text nodes around the block tags.

Fixed in #30.


ModuleNotFoundError: No module named 'dotenv'

ModuleNotFoundError: No module named 'dotenv'
pip install dotenv does not work, any ideas?

sudo apt-get install python3-dotenv python3-dotenv-cli solved it for me on ubuntu.

https://stackoverflow.com/questions/49328525/pip-install-dotenv-error-code-1-windows-10
as geeknik pointed out, you should install python3-dotenv and it will work

Same problem too, installation failed in Mac, error message:
TQV9MF4NXR:tree-of-thoughts bytedance$ pip install dotenv
Looking in indexes: https://pypi.tuna.tsinghua.edu.cn/simple
Collecting dotenv
  Using cached https://pypi.tuna.tsinghua.edu.cn/packages/e2/46/3754073706e31670eed18bfa8a879305b56a471db15f20523c2427b10078/dotenv-0.0.5.tar.gz (2.4 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Installing backend dependencies ... error
  error: subprocess-exited-with-error

  × pip subprocess to install backend dependencies did not run successfully.
  │ exit code: 1
  ╰─> [30 lines of output]
      Looking in indexes: https://pypi.tuna.tsinghua.edu.cn/simple
      Collecting distribute
        Using cached https://pypi.tuna.tsinghua.edu.cn/packages/5f/ad/1fde06877a8d7d5c9b60eff7de2d452f639916ae1d48f0b8f97bf97e570a/distribute-0.7.3.zip (145 kB)
        Installing build dependencies: started
        Installing build dependencies: finished with status 'done'
        Getting requirements to build wheel: started
        Getting requirements to build wheel: finished with status 'done'
        Preparing metadata (pyproject.toml): started
        Preparing metadata (pyproject.toml): finished with status 'error'
        error: subprocess-exited-with-error

        × Preparing metadata (pyproject.toml) did not run successfully.
        │ exit code: 1
        ╰─> [6 lines of output]
            usage: setup.py [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
               or: setup.py --help [cmd1 cmd2 ...]
               or: setup.py --help-commands
               or: setup.py cmd --help

            error: invalid command 'dist_info'
            [end of output]

        note: This error originates from a subprocess, and is likely not a problem with pip.
      error: metadata-generation-failed

      × Encountered error while generating package metadata.
      ╰─> See above for output.

      note: This is an issue with the package mentioned above, not pip.
      hint: See above for details.
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× pip subprocess to install backend dependencies did not run successfully.
│ exit code: 1
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.

The same on ubuntu 22.04.:
`pip install tree_of_thoughts
Collecting tree_of_thoughts
Using cached tree_of_thoughts-0.2.7-py3-none-any.whl (14 kB)
Collecting guidance (from tree_of_thoughts)
Using cached guidance-0.0.57-py3-none-any.whl (83 kB)
Collecting openai (from tree_of_thoughts)
Using cached openai-0.10.5.tar.gz (157 kB)
Preparing metadata (setup.py) ... done
Collecting transformers (from tree_of_thoughts)
Using cached transformers-4.29.2-py3-none-any.whl (7.1 MB)
Collecting dotenv (from tree_of_thoughts)
Using cached dotenv-0.0.5.tar.gz (2.4 kB)
Preparing metadata (setup.py) ... error
error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> [76 lines of output]
error: subprocess-exited-with-error
    × python setup.py egg_info did not run successfully.
    │ exit code: 1
    ╰─> [16 lines of output]
        Traceback (most recent call last):
          File "<string>", line 36, in <module>
          File "<pip-setuptools-caller>", line 14, in <module>
          File "/tmp/pip-wheel-ajo68hru/distribute_820e4f5102b14a1ca9546b69347c1dd2/setuptools/__init__.py", line 2, in <module>
            from setuptools.extension import Extension, Library
          File "/tmp/pip-wheel-ajo68hru/distribute_820e4f5102b14a1ca9546b69347c1dd2/setuptools/extension.py", line 5, in <module>
            from setuptools.dist import _get_unpatched
          File "/tmp/pip-wheel-ajo68hru/distribute_820e4f5102b14a1ca9546b69347c1dd2/setuptools/dist.py", line 7, in <module>
            from setuptools.command.install import install
          File "/tmp/pip-wheel-ajo68hru/distribute_820e4f5102b14a1ca9546b69347c1dd2/setuptools/command/__init__.py", line 8, in <module>
            from setuptools.command import install_scripts
          File "/tmp/pip-wheel-ajo68hru/distribute_820e4f5102b14a1ca9546b69347c1dd2/setuptools/command/install_scripts.py", line 3, in <module>
            from pkg_resources import Distribution, PathMetadata, ensure_directory
          File "/tmp/pip-wheel-ajo68hru/distribute_820e4f5102b14a1ca9546b69347c1dd2/pkg_resources.py", line 1518, in <module>
            register_loader_type(importlib_bootstrap.SourceFileLoader, DefaultProvider)
        AttributeError: module 'importlib._bootstrap' has no attribute 'SourceFileLoader'
        [end of output]

    note: This error originates from a subprocess, and is likely not a problem with pip.
  error: metadata-generation-failed

  × Encountered error while generating package metadata.
  ╰─> See above for output.

  note: This is an issue with the package mentioned above, not pip.
  hint: See above for details.
  /home/alex/anaconda3/lib/python3.7/site-packages/setuptools/__init__.py:84: _DeprecatedInstaller: setuptools.installer and fetch_build_eggs are deprecated.
  !!

          ********************************************************************************
          Requirements should be satisfied by a PEP 517 installer.
          If you are using pip, you can try `pip install --use-pep517`.
          ********************************************************************************

  !!
    dist.fetch_build_eggs(dist.setup_requires)
  Traceback (most recent call last):
    File "/home/alex/anaconda3/lib/python3.7/site-packages/setuptools/installer.py", line 96, in _fetch_build_egg_no_warn
      subprocess.check_call(cmd)
    File "/home/alex/anaconda3/lib/python3.7/subprocess.py", line 328, in check_call
      raise CalledProcessError(retcode, cmd)
  subprocess.CalledProcessError: Command '['/home/alex/anaconda3/bin/python', '-m', 'pip', '--disable-pip-version-check', 'wheel', '--no-deps', '-w', '/tmp/tmplwctifoy', '--quiet', 'distribute']' returned non-zero exit status 1.

  The above exception was the direct cause of the following exception:

  Traceback (most recent call last):
    File "<string>", line 36, in <module>
    File "<pip-setuptools-caller>", line 34, in <module>
    File "/tmp/pip-install-ij6vqdkj/dotenv_780d97381bfa43ca8e1495c0012f98e3/setup.py", line 23, in <module>
      scripts=['scripts/dotenv']
    File "/home/alex/anaconda3/lib/python3.7/site-packages/setuptools/__init__.py", line 106, in setup
      _install_setup_requires(attrs)
    File "/home/alex/anaconda3/lib/python3.7/site-packages/setuptools/__init__.py", line 79, in _install_setup_requires
      _fetch_build_eggs(dist)
    File "/home/alex/anaconda3/lib/python3.7/site-packages/setuptools/__init__.py", line 84, in _fetch_build_eggs
      dist.fetch_build_eggs(dist.setup_requires)
    File "/home/alex/anaconda3/lib/python3.7/site-packages/setuptools/dist.py", line 917, in fetch_build_eggs
      return _fetch_build_eggs(self, requires)
    File "/home/alex/anaconda3/lib/python3.7/site-packages/setuptools/installer.py", line 41, in _fetch_build_eggs
      replace_conflicting=True,
    File "/home/alex/anaconda3/lib/python3.7/site-packages/pkg_resources/__init__.py", line 828, in resolve
      req, best, replace_conflicting, env, installer, required_by, to_activate
    File "/home/alex/anaconda3/lib/python3.7/site-packages/pkg_resources/__init__.py", line 864, in _resolve_dist
      req, ws, installer, replace_conflicting=replace_conflicting
    File "/home/alex/anaconda3/lib/python3.7/site-packages/pkg_resources/__init__.py", line 1133, in best_match
      return self.obtain(req, installer)
    File "/home/alex/anaconda3/lib/python3.7/site-packages/pkg_resources/__init__.py", line 1145, in obtain
      return installer(requirement)
    File "/home/alex/anaconda3/lib/python3.7/site-packages/setuptools/installer.py", line 98, in _fetch_build_egg_no_warn
      raise DistutilsError(str(e)) from e
  distutils.errors.DistutilsError: Command '['/home/alex/anaconda3/bin/python', '-m', 'pip', '--disable-pip-version-check', 'wheel', '--no-deps', '-w', '/tmp/tmplwctifoy', '--quiet', 'distribute']' returned non-zero exit status 1.
  [end of output]

note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed
× Encountered error while generating package metadata.
╰─> See above for output.
note: This is an issue with the package mentioned above, not pip.
hint: See above for details.
`


NODE service - exceptions in celery task/functions

the celery task/functions in teh nodes have to raise refined exceptions that will be propagated to the caller (the controller) showing why the request has failed

@apmariglis  Can you elaborate on that a bit more?
When an exception occurs in the node, if the exception should not be captured, it will be propagated to the controller normally through celery. Celery changes a bit the name of the exception but the flow still remains.
We have some specific exceptions that you can capture but I don't think there is a point of capturing ALL exceptions and throwing them with a different name.
What exactly are you thinking that the flow should be?

Moved to jira.


Linking ts-ast-reader
Linking star-wars
Linking kitchen-sink
Linking githunt-api
Linking apollo-fullstack
Linking ghost
Linking website
Finished linking website
[16:27:06] 'link-examples' errored after 13 s
[16:27:06] Error: Command failed: yarn
warning package.json: No license field
warning @graphql-nexus/ghost-example-api: No license field
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version "^6.9.0 || ^8.9.0 || ^10.13.0". Got "11.6.0"
warning<EMAIL_ADDRESS>The engine "cli" appears to be invalid.
error Found incompatible module

    at ChildProcess.exithandler (child_process.js:294:12)
    at ChildProcess.emit (events.js:188:13)
    at maybeClose (internal/child_process.js:978:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:265:5)

running yarn v1.13.0 on macOS
Error: Command failed: yarn looks strange

using node 10 solved the issue


requests.get doesn't fail HTTP requests with verify option

When you use the verify parameter on requests.get, the function is supposed to check that the webpage certificate is valid. Obviously, if the webpage is not HTTPS, the webpage has no certificate.
I think it would be nice if in that case the request fails. I mean, imagine we have Google's cert on /srv/google-cert:
$ cat /srv/google-cert
-----BEGIN CERTIFICATE-----
MIIDVDCCAjygAwIBAgIDAjRWMA0GCSqGSIb3DQEBBQUAMEIxCzAJBgNVBAYTAlVT
MRYwFAYDVQQKEw1HZW9UcnVzdCBJbmMuMRswGQYDVQQDExJHZW9UcnVzdCBHbG9i
YWwgQ0EwHhcNMDIwNTIxMDQwMDAwWhcNMjIwNTIxMDQwMDAwWjBCMQswCQYDVQQG
EwJVUzEWMBQGA1UEChMNR2VvVHJ1c3QgSW5jLjEbMBkGA1UEAxMSR2VvVHJ1c3Qg
R2xvYmFsIENBMIIBIjANBgkqhkiG9w0BAQEFAAOCAQ8AMIIBCgKCAQEA2swYYzD9
9BcjGlZ+W988bDjkcbd4kdS8odhM+KhDtgPpTSEHCIjaWC9mOSm9BXiLnTjoBbdq
fnGk5sRgprDvgOSJKA+eJdbtg/OtppHHmMlCGDUUna2YRpIuT8rxh0PBFpVXLVDv
iS2Aelet8u5fa9IAjbkU+BQVNdnARqN7csiRv8lVK83Qlz6cJmTM386DGXHKTubU
1XupGc1V3sjs0l44U+VcT4wt/lAjNvxm5suOpDkZALeVAjmRCw7+OC7RHQWa9k0+
bw8HHa8sHo9gOeL6NlMTOdReJivbPagUvTLrGAMoUgRx5aszPeE4uwc2hGKceeoW
MPRfwCvocWvk+QIDAQABo1MwUTAPBgNVHRMBAf8EBTADAQH/MB0GA1UdDgQWBBTA
ephojYn7qwVkDBF9qn1luMrMTjAfBgNVHSMEGDAWgBTAephojYn7qwVkDBF9qn1l
uMrMTjANBgkqhkiG9w0BAQUFAAOCAQEANeMpauUvXVSOKVCUn5kaFOSPeCpilKIn
Z57QzxpeR+nBsqTP3UEaBU6bS+5Kb1VSsyShNwrrZHYqLizz/Tt1kL/6cdjHPTfS
tQWVYrmm3ok9Nns4d0iXrKYgjy6myQzCsplFAMfOEVEiIuCl6rYVSAlk6l5PdPcF
PseKUgzbFbS9bZvlxrFUaKnjaZC2mqUPuLk/IH2uSrW4nOQdtqvmlKXBx4Ot2/Un
hw4EbNX/3aBd7YdStysVAq45pmp06drE57xNNB6pXE0zX5IJL4hmXXeXxx12E6nV
5fEWCRE11azbJHFwLJhWC9kXtNHjUStedejV0NxPNO3CBWaAocvmMw==

-----END CERTIFICATE-----

Then right now this is what happens:
>>> import requests
>>> requests.get('https://www.google.com', verify='/srv/google-cert')
<Response [200]>
>>> requests.get('http://www.ull.es', verify='/srv/google-cert')
<Response [200]>

I'm proposing a change: the second get call (which is HTTP) should fail.
I've tested this on requests 2.11.1.

Thanks for this suggestion! However, I'm afraid I disagree. =) That represents a pretty substantial change in the behaviour of the verify argument, and is well outside what is currently documented. Additionally, it raises some extremely difficult questions. For example, what if the request is redirected from HTTPS to HTTP: should that fail?
But fundamentally it just punishes users too heavily for simple errors. verify has a very simply defined behaviour as shown here: whenever set to a validation mode, requests will validate HTTPS requests, but not others.
You can achieve your desired behaviour (no HTTPS support) by reconfiguring your Session somewhat:
s = requests.Session()
del s.adapters['https://']

This will make Requests entirely unable to make HTTP requests.


v 6.2.2 Fails to build with gcc9-9.1.1+r273003

[   57s] hw/sbe-p9.c: In function 'p9_sbe_msg_send':
[   57s] hw/sbe-p9.c:270:9: error: taking address of packed member of 'struct p9_sbe_msg' may result in an unaligned pointer value [-Werror=address-of-packed-member]
[   57s]   270 |  data = &msg->reg[0];
[   57s]       |         ^~~~~~~~~~~~
[   57s] hw/sbe-p9.c: In function 'p9_sbe_msg_receive':
[   57s] hw/sbe-p9.c:303:9: error: taking address of packed member of 'struct p9_sbe_msg' may result in an unaligned pointer value [-Werror=address-of-packed-member]
[   57s]   303 |  data = &resp->reg[0];
[   57s]       |         ^~~~~~~~~~~~~
[   57s] hw/sbe-p9.c: In function 'p9_sbe_msg_complete':
[   57s] hw/sbe-p9.c:326:11: error: taking address of packed member of 'struct p9_sbe_msg' may result in an unaligned pointer value [-Werror=address-of-packed-member]
[   57s]   326 |  list_del(&msg->link);
[   57s]       |           ^~~~~~~~~~
[   57s] hw/sbe-p9.c: In function 'p9_sbe_queue_msg':
[   57s] hw/sbe-p9.c:423:32: error: taking address of packed member of 'struct p9_sbe_msg' may result in an unaligned pointer value [-Werror=address-of-packed-member]
[   57s]   423 |  list_add_tail(&sbe->msg_list, &msg->link);
[   57s]       |                                ^~~~~~~~~~
[   57s] hw/sbe-p9.c: In function 'p9_sbe_cancelmsg':
[   57s] hw/sbe-p9.c:480:11: error: taking address of packed member of 'struct p9_sbe_msg' may result in an unaligned pointer value [-Werror=address-of-packed-member]
[   57s]   480 |  list_del(&msg->link);
[   57s]       |           ^~~~~~~~~~

@hegdevasant can you have a look

Already fixed in master.

v6.2.x is unmaintained .. Current stable version is v6.6.x and soon it will be v6.7.x .. Hence closing this one.
Let me know , if you still need this issue to be addressed in v6.2.x branch.
-Vasant


How to use Login?

When I use the login function and once I've connected via the Twitter app my callback does nothing.
I've also tried adding NSLog to the FabricTwitterKit.m but they also don't show anything.
Here is my code:
FabricTwitterKit.login((error, res) => {                     console.log('errors => ',error);                     console.log('res => ',res);                   });
Any help? Thanks!

Okay worked out what I was doing wrong.
You need to follow the docs properly: https://dev.twitter.com/twitterkit/ios/log-in-with-twitter
My mistake was not adding:
- (BOOL)application:(UIApplication *)app openURL:(NSURL *)url options:(NSDictionary<NSString *,id> *)options {
    return [[Twitter sharedInstance] application:app openURL:url options:options];
}

to my AppDelgate.

Hi..
I'm having some trouble understanding exactly how to write login or fetchtweet request from the iOS functions on FabricTwitterKit.m file.
Could you help me with any example?

@aschenkel Hey!
If you've set it up properly (Follow the docs and read my comment above), all you need to do is import the module and call the function
// Import
import FabricTwitterKit from 'react-native-fabric-twitterkit'

// Login Function
FabricTwitterKit,login((response) => {
    // Callback here
})

Let me know if you need more help!

Thanks man!
I'm already logged in and fetch the profile. I had to change deployment target to 9.0 on iOS, weird.
I'm trying to fetchTweets now.
fetchTweets() {
        //Declare myMap
        FabricTwitterKit.fetchTweet(myMap,(res) =>
        {
            if(res !== undefined){
                console.log(res);
                //dispatch actions
            }
        });
    }

and I have declared myMap in many ways:
   var myMap = new Map();
    var valueID = this.props.userID,
    keyID = "id";
    myMap.set(keyID, valueID);

    var myMap = [];
    myMap.push({
        key:   "id",
        value: this.props.userID
    });

    var myMap = {
        key:   "id",
        value: this.props.userID
    }

but I'm always getting 2 exceptions

JSON value '(
{
key = id;
value =<PHONE_NUMBER>35957504;
}
)' of type NSMutableArray cannot be converted to NSDictionary

Exception '*** -[__NSPlaceholderDictionary initWithObjects:forKeys:count:]: attempt to insert nil object from objects[0]' was thrown while invoking fetchTweet on target FabricTwitterKit with params (
(
{
key = id;
value =<PHONE_NUMBER>35957504;
}
),
201
)

Do you have any idea what would be the right way?

@aschenkel This is just a guess, but try this to declare myMap
var myMap = {id:<PHONE_NUMBER>35957504 }

Nice try but it throws

Exception '*** -[__NSPlaceholderDictionary initWithObjects:forKeys:count:]: attempt to insert nil object from objects[1]' was thrown while invoking fetchTweet on target FabricTwitterKit with params (
{
id =<PHONE_NUMBER>35957504;
},
207
)

Same with:
var myMap = {"id":<PHONE_NUMBER>35957504 }

works with
var myMap = {
      "id": "881887373335957504",
       "trim_user": "false",
       "include_my_retweet": "false"
 }

Hi @DannyAziz i used this function
FabricTwitterKit.login((response) => {
    // Callback here

      console.log('Twitter Data Response ' + response);
      })

the return is "Twitter Data Response null".. Please help me how can i get proper syntax for Twitter login... Im new in react native.. Thanks sir!

is it posible to get email from twitter?


Just Library is not supported for Xcode 12 and above

Hmm it work on mine. XCode Version 12.5 (12E262)


Requiring Node >=15 breaks Vercel deployments

FYI that AWS Lambda is a little slow to jump to Node 16 so anyone deploying anything to an environment utilizing Lambda (Vercel in my case: https://github.com/vercel/community/discussions/37) can't add @vitebook-core to their package.json. Is Node 16 really needed or can you downgrade the minimum to v14? https://github.com/vitebook/vitebook/blob/f9fdafe97aeb96d94e4f6d2cef74c8875f0bee2e/packages/core/package.json#L92
Vercel build error:

Looks like it's coming in the next 90 days, but until then I'll need to double check why 16 was required (I forgot to write it down 😓 ). I think it was mainly related to esbuild installation.

Ok, my workaround until then is to deploy Vitebook to Netlify (node 16) and then for my regular Vercel production app I just modified the required engine in package-lock.json to "node": ">=14" for the time being to allow the build to work.


Fixed a memory issue found with AddressSanitizer.

The strings may be shorter than 32 bytes, therefor use strncpy
Thx @giox069 for pointing that out ;)

@giox069 @akallabeth I was really wondering what was wrong with the CopyMemory. Looking at https://github.com/FreeRDP/Remmina/pull/728 I figured why, and that looks more like a misused API error: in the current code ClientHostname is expected to be filled with 32 bytes long chunk.
Let's say the current API is bad, so let's change it to something better (ie take the patch). But I'd suggest to drop the initial allocation of ClientHostname, and just do a _strdup() then. And adjust code using ClientHostname (Ctrl+Shift+G under eclipse on this field and you get them instantly). What do you think ?

The problem is not really in CopyMemory, but in client/common/cmdline.c, line 1666: settings->ClientHostname is allocated with a small buffer via _strdup, and later assumed to be 32 bytes long in other FreeRDP places.
Here is line 1666 of cmdline.c:
settings->ClientHostname = _strdup(arg->Value)
Yes, the current API is bad, because having fixed strings at 32bytes is not a good choice: no one will remember this, because everyone is used to have strings allocated on the heap.
But changing it is up to you :) There are a lot of 32bytes strings around.

@hardening Just as @giox069 already said, the problem is that it is not 32 bytes in the first place. I'm happy with every kind of fix, I just used strncpy as it copies only the used parts of the string (and therefore does remove the out of bound access)

@hardening Ok, looked at this again, the fixed size is a really bad idea ;)
Even our own command line parser does fuck this up (see client/common/cmdline.c:1666)

Superseeded by #3076


Option to modify styles in Preferences > Editor > Colors & Fonts

Thanks for writing this!
Is there a way to modify the style for *.cjsx files in Colors & Fonts in Webstorm? I don't see the option, like for the other file types CoffeeScript, JavaScript, JSON, etc.
It would be great if there were a way to do that.

I don't think so. AFAIK a tmbundle is just a thin wrapper on textmate bundles for backwards compatibility with that software product. As I've been advocating to others; it's probably better to just convert your code to JavaScript/JSX; less headaches and better all-round support.


Git bot msg when openhands makes no changes

This PR addresses the last part of #181
The github action sends a bot message The workflow to fix this issue encountered an error. Openhands failed to create any code changes. when Openhands fails to creates code changes

Great, thanks!


getpeername "Invalid argument"/"Transport endpoint is not connected" exception

We've seen several of these, on both macOS ("Invalid argument") and Linux ("Transport endpoint not connected").
from macOS: #3747 crashlog, #3730 crashlog
from linux: crashlog from discord DM, #3749 crashlog
Reportedly, having a SNARK worker makes this worse.

The only call to getpeername I see in our deps is in rpc (big surprise) but inside a function that we don't call. rpc_parallel, however, DOES. but those connections should only be to localhost? are they getting closed? more to investigate...

Had a Snark Coordinator crash with this error in rising-phoenix
https://gist.github.com/yourbuddyconner/c08694ccf215592386ed888c8b0ca8d1

Folding this into #4276, which appeared in a more recent version. We lose the mac version in the title by doing this, but we're not currently supporting mac builds at the moment.


Allow Daisy-Chained Aliases Within Palettes

Allow to alias a color definition within a Palette to another alias. Umbriel's Color System will follow the aliases until it gets to a color value. It will also keep a record of which aliases it has gone through to make sure there are no circular aliases. If there are circular references, it will simply generate black.
palette = new Palette [
  ["Gray", "808080"]
  ["Grey", "!gray"]
  ["Graye", "!grey"]
]

The documentation specifically states you cannot daisy-chain aliases, so be sure to update this.


initial setup

Add setup application with Vite + React

APPROVED 🥳
Great job following the tutorial to work on this project 👍
Checking the setup i found no error to my understanding
Nothing else to say other than it is time to merge it 👑
Congratulations 🥮
Happy coding 💯


Add WSO2 API Gateway to the CN landscape

Pre-submission checklist:
Please check each of these after submitting your pull request:

[X ] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[ ] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[X ] Have you picked the single best (existing) category for your project?
[ X] Does it follow the other guidelines from the new entries section?
[ X] Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
[X ] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[X ] Does your project/product name match the text on the logo?
[X ] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[X ] ~5 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Deploy preview for landscape ready!
Built with commit 613dc5479e8bbf28b9e1fecc794626c2c849d838
https://deploy-preview-1440--landscape.netlify.com

LGTM

Logos need to include the project name. Please wait for this to build and then give it another LGTM. Note that you're also welcome to replace with a project-specific logo if and when you create one.

LGTM


xrev required for distro packages

The following example command from the README doesn't work anymore, because it requires xrev to be specified.  Shouldn't xrev really be optional when there might not necessarily be one, or should the README be changed?
$ cblrepo add -d xmonad-contrib,0.10,1 -d zlib,<IP_ADDRESS>,2.1

https://github.com/magthe/cblrepo/commit/fa351b2b4786250a828ea4274c0052935f8c4a2b seems to imply that not having one is no longer supported, so I guess I'll have to add an xrev of 0 to the packages that doesn't have one.

Hmm, yes. I'll modify the README to make that a little clearer.


Javascript translations cause extreme slowdowns in VMs

This is a bit of an odd one, hopefully I'll get it explained right.
When I was working on setting up Vagrant boxes for running c5, using shared folders between the host and the client, running xdebug was pretty much impossible. Page loads would take minutes, and then everything would slow down more waiting for these two files:
/ccm/assets/localization/core/js
/ccm/assets/localization/jquery/ui/js
They're a bit slow in general when running xdebug, but once the slower IO of a VM's shared folder is added in, it's literally minutes for these to render.
As far as I can tell, basically the php for each of these is translating essentially static content.  It's not custom content for particular blocks, extra translations, etc. It's just the phrases within that file.
https://github.com/concrete5/concrete5/blob/develop/web/concrete/controllers/frontend/assets_localization.php#L17-L123
Would it be OK to have this check to see if there is a translation already rendered for whatever the requested language is? They could ship pre-rendered, then clear out when a term is updated in the .mo / .po files?
I don't know a ton about how the translations work deep down, just figured out that it was the getCoreJavascript() function that was making my sites pretty much unuseable. Running the t() function inside of there almost 100 times is a huge resource drain. It probably doesn't show up much on high powered servers, but could contribute a lot to performance issues on lower powered hosts. Especially since these files are pretty much needed to be done before some of the other javascript runs.
The problem has been around for pretty much forever, but I never realized it was happening there until looking at the version 7 inside of VMs.

I've done some performance test and I don't notice that huge delay in calling t().
For a better understanting of the problem, I'd need more info about your setup:

For which locale you have problems?
Which packages have you installed?
The core language files are all complete at 100%?
The package language files are all complete at 100%?

I'm asking you that because the only case when I notice slow dows problems is when t is called for languages that have no translations for the string being translated.

Thanks for digging into this, @mlocati


JustAssembly.exe can take 2 file paths to bypass file open dialog

PR to fix #27.
The changes allow JustAssembly.exe to take 2 file arguments to bypass the file opening dialog. An error message will be shown if the arguments are invalid (if they don't exist, or if either is a directory).  This is mostly work of @colin-luke that I brushed over and added a bit of validation and error window.
I had to add a helper in CommandLineArgumentsAreValidToSkipNewSessionDialog in ShellViewModel. I put it there because it is very specific to the case it handles, but I'm not sure if that's the place for it.
I'm open to do modifications after your review.

Alright, thanks for the review!
I'll make the requested changes in the next coming days.

Awaiting further review.

Hi, @SugoiDev.
Firstly, please excuse me for the huge delay. I've been on a emergency leave and I was not able to reach you any sooner.
I just wanted to give you an explanation why I haven't reviewed your changes yet. I will take a look at them as soon as I can. Rather unfortunately, I cannot make a promise when that will happen. Thank you for your patience.

@adimitrov93
Alright, it's signed. Thank you and happy new year!


Adding in docs for the quorum dev quickstart

Pull Request Description

Updating the doco around the quorum dev quickstart
Adding in monitoring docs for elk + metrics
Adding in an IBFT consensus how to
Re org of the tutorials section

Made some edits to the IBFT and ELK pages. Should probably have another review from @ConsenSys/protocol-pliny.


Unable to start to apache superset .when trying for the command docker-compose up

File
"/usr/local/lib/python3.6/urllib/request.py".py,line 1361,in https_open superset-init_1 | context=self._context,check_hostname=self._check_hostname) superset-init_1 | File "/usr/local/lib/python3.6/urllib/request.py", line 1283 , in do_open superset_init_1 | raise URLError('no host given') superset_init_1 | urllib.error.URLError:

npm ERR: code ENOENT
npm ERR:errno ENOENT
npm ERR:enoent request to https://registry.npm.js.org/kind of failed

Hi @Michael3190 - I have not seen this error before. Please try again with the latest master and let us know your results. If the problem continues, please share details of your execution environment. Thanks!

Closing as we have not been able to reproduce.


LazyIteratorList

In working on enhancements to the FrameNet corpus reader, I found it would be convenient to define a lazy sequence type that is instantiated with an iterator (such as a generator expression). I came up with the following:
class LazyIteratorList(AbstractLazySequence):
    """
    Wraps an iterator, loading its elements on demand
    and making them subscriptable.
    __repr__ displays only the first few elements.
    """
    def __init__(self, it, cache_limit=None, known_len=None):
        self._it = it
        self._len = known_len
        self._cache = []
        self._cache_limit = cache_limit
        self._i = 0 # Number of items consumed so far

    def __len__(self):
        if self._len:
            return self._len
        for x in self.iterate_from(len(self._cache)):
            pass
        return len(self._cache)

    def iterate_from(self, start):
        while self._i<start:
            v = next(self._it)
            if self._cache_limit is None or len(self._cache)+1<self._cache_limit:
                self._cache.append(v)
            self._i += 1
        i = start
        while i<len(self._cache):
            yield self._cache[i]
            i += 1
        while True:
            v = next(self._it)
            if self._cache_limit is None or len(self._cache)+1<self._cache_limit:
                self._cache.append(v)
            yield v

    def __add__(self, other):
        """Return a list concatenating self with other."""
        return type(self)(itertools.chain(self, other))

    def __radd__(self, other):
        """Return a list concatenating other with self."""
        return type(self)(itertools.chain(other, self))

which enables things like:
LazyIteratorList(aset for sent in self.ft_sents() for aset in sent.annotationSet[1:] if luNamePattern is None)

Before I check in the changes to the corpus reader, should this class go in https://github.com/nltk/nltk/blob/develop/nltk/util.py? The alternative of using LazyMap seems far less intuitive, especially with nested loops and conditions.

@nschneid this is really great. It's always bothered me that we've had to convert a corpus iterator into a list in order to index it.
I'm thinking that this might warrant a new module, e.g. nltk/collections.py.
Things like OrderedDict from util.py could go there too.

Yeah, util.py is a bit crowded, so moving classes into collections.py seems sensible.


Update java-client to 2.7.22

Updates com.couchbase.client:java-client from 2.7.19 to 2.7.22.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (2.7.19).
You might want to review and update them manually.
build.sbt
docs/build.gradle

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.couchbase.client", artifactId = "java-client" } ]

labels: library-update, early-semver-patch, semver-spec-patch, old-version-remains

Superseded by #94.


Use java.nio.file for file/path handling

Constructing paths using string operations is error prone, but there's lots of areas in the code base where this is done, which sets a bad example.
Lots of APIs in the code base also take in Strings as file paths, which further encourages this kind of code.

I'll start work on this!

     *
 If the {@code name} begins with a {@code '/'}
     * ('\u002f'), then the absolute name of the resource is the
     * portion of the {@code name} following the {@code '/'}.
     *
     *
 Otherwise, the absolute name is of the following form:
     *
     *

     *   {@code modified_package_name/name}
     *

It seems like Class#getResource takes in string paths that only use / as a file separator. As a result,  this results in the UI tests to fail as \ is being used for the file separator instead. This is because using java.nio.file.Path to handle file paths results in the conversion of the file separator based on the current operating system. Since Class#getResource only deems file paths that use / as valid, these tests would fail in a Windows computer as Class#getResource returns null. So in this case, is it better to leave the file path handling in the UI component to be just string concatenation?

@vivekscl

So in this case, is it better to leave the file path handling in the UI component to be just string concatenation?

Yeah you can leave it.