Car will not wakeup

Im trying this with a Tesla MY but can't get I to work, keep getting 408 and the car will not wakeup.
Tried both the wakeup command and during eg. unlock charging port to set the wakeup automatically.
The auth seems to work since the car shows up in list of vehicles.

The car may just have a bad connection. It is not really something this library can help you with, sorry. Just keep trying.

Sorry to reopen this, but I'm definitely seeing a lot of 408 errors from this library, while if I open the Tesla app on my phone, it can connect just fine. What could be the reason for this? A 408 is a Timeout, so could it be that the library times out before the car responds, while the Tesla app has a bigger timeout window set?

I am actually seeing a lot of 408 errors myself lately. But I am also seeing it in the app. Perhaps Tesla changed or broke something. The library does not timeout, the response code is directly from the Tesla API. There is not really something I can do about it.

Alright, thank you for the reply!
fre. 1. jul. 2022, 14:50 skrev Martin Schimmel @.***>:

I am actually seeing a lot of 408 errors myself lately. But I am also
seeing it in the app. Perhaps Tesla changed or broke something. The library
does not timeout, the response code is directly from the Tesla API. There
is not really something I can do about it.
—
Reply to this email directly, view it on GitHub
https://github.com/onokje/node-red-contrib-tesla/issues/36#issuecomment-1172312906,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBYPRMP5GJLBRKMFGB2GT3VR3STFANCNFSM5LIEXUOQ
.
You are receiving this because you commented.Message ID:
@.***>


Stage mitxonline users_user_groups

[ ] what are the raw fields
[ ] write up a plan for each field, including modifications to the field name, data types, data to omit, etc.
[ ] open a PR the implements the plan for users_user_groups

The columns for this table are
id
user_id
group_id
group_id refers to the auth_group table and can be "Editor" or "Moderator". This is probably wagtail permissions. If we're not importing is_staff and is_superuser I think we can skip importing this for the time being.

So no pr will be needed for this table

Agree on skipping.


Dev1

@TiraFire Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@TiraFire Thank you for signing the Contributor License Agreement!


Remove ServiceManager dependency from Barcode

We need less inter-dependencies, not more.
Barcode::makeBarcode() --> ObjectPluginManager
Barcode::makeRenderer() --> RendererPluginManager
The easy fix is to tweak composer.json so ServiceManager is required, but I'd prefer a much more SOLID and indepentant implementation because Zend\Barcode is widely used, and needs to function à la carte in any non-ZF2 app, especially legacy non-composer apps.  It should probably be moved out of ZF2 core, just like ZendPdf.

This issue has been closed as part of the bug migration program as outlined here - http://framework.zend.com/blog/2016-04-11-issue-closures.html


CMP server: add support for multiple transactions

For now, this FR basically serves as a reminder that the CMP server code,
not only in the mock server, but also in the generic core Implementation in crypto/cmp/cmp_server.c,
has a severe limitation: it only supports a single transaction at a time.
So it is not suitable for productive purposes,
but just for testing purposes such as the OpenSSL-internal tests, for which it was created.
In order to change this,
struct ossl_cmp_srv_ctx_st would need to be extended to support lists of transaction states,
while the API can likely stay as it as.

off topic: @DDvO it seems e-mails to your address it at von-oheimb dot de are bouncing.
This is the mail system at host server8.webgo24.de.

I'm sorry to have to inform you that your message could not
be delivered to one or more recipients. It's attached below.

For further assistance, please send mail to postmaster.

If you do so, please include this problem report. You can
delete your own text from the attached returned message.

                   The mail system

<[XXXXXXX@server8.webgo24.de](mailto:XXXXXX@server8.webgo24.de)> (expanded from <[XXX@XXXXXXXX] mailto:XXX@XXXXXXX)>): can't create user output file

This your e-mail being unsubscribed from openssl-users ML automatically.

off topic: @DDvO it seems e-mails to your address it at von-oheimb dot de are bouncing.

This resulted in your e-mail being unsubscribed from openssl-users ML automatically.

Thanks a lot @t8m for pointing this out to me!
Yet unfortunately this was because my OpenSSL-related mailbox overflowed

without my wonderful mail provider warning me -
and I did not notice this since April 2nd, so I lost all emails in between :-(

For the same reason, your note did not reach me,
and now I'm catching up at least with GitHub notifications,
which luckily are available online at https://github.com/notifications?query=is%3Aunread
and where I found your above note.
I've also just re-subscribed to openssl-users.


Are you getting correct temperature readings?

I'm trying to make a version of this for ESP32/Arduino, and I'm not able to get correct temperature readings. It's probably an error in the way I'm processing the register, but I'm wondering if anyone can confirm that they do indeed return the correct temperature?
I have the external temp sensor now.
My code is here if anyone happens to be interested in helping me port the section of this code to C:
https://raw.githubusercontent.com/wrybread/ESP32ArduinoRenogy/main/renology_rs232.ino
The way it's handled here in NodeRenogy is:
//Register 0x103 - Battery/Controller Temperature - 3
//0x103 returns two bytes, one for battery and one for controller temp in c
const buf = Buffer.alloc(2)
buf.writeInt16BE(rawData[3]);
this.battT = buf[0];
this.controlT = buf[1];

I get the temperatures but they are reversed.  Looking at the modbus documentation for Renogy Rover the controller temp is in the first half of the byte

This issue should've been resolved be this PR
https://github.com/sophienyaa/NodeRenogy/pull/9


12 word recovery

Hello,
I am trying to implement 12 phrase recovery on this app. Can you please give some high level description on what does wallet id and wallet key do in App.xaml.cs. Can I use them for this?
Thank you

hey @juvebogdan to implement that feature you will rely a bit more on the backend side, but you can certainly use this codebase as an entry point. Do you have time to connect later, if I get a better picture of what you are trying to accomplish will be able to give you a better pointer.

Hello. I do have time. Thank you for your response. How would you like to connect. My skype name is live:krivokapic.bogdan10

I am interested in this ability .
Is it possible with aries itself ?
or need some addon or connection to other party.


NPE when adding cascade perms to a folder with no perms

from @stroomdev00 (in #3947)

An existing folder with no perms containing a number of objects all owned by me.
Try adding another user (or even myself) to the folder with any perms (even no perms) and any cascade option results in the error java.lang.NullPointerException: Cannot invoke "stroom.securuity.shared.DocumentPermissions.getPermissions()" because "documentPermissions" is null.

Fixed in >7.2.18


[Bug]  No module named mlc_llm.main

🐛 Bug

To Reproduce
W9057210@PAW9057210 MINGW64 /f/project/mlc-llm (main)
$ python -m mlc_llm convert_weight dist/models/Llama-2-7b-chat-hf/ --quantization q4f16_1 -o dist/Llama-2-7b-chat-hf-q4f16_1-MLC
C:\ProgramData\anaconda3\python.exe: No module named mlc_llm.main; 'mlc_llm' is a package and cannot be directly executed

I tried to use the new command, and there was another error. How do I handle this error?
Expected behavior

Environment

Platform (e.g. WebGPU/Vulkan/IOS/Android/CUDA):
Operating system (e.g. Ubuntu/Windows/MacOS/...):
Device (e.g. iPhone 12 Pro, PC+RTX 3090, ...)
How you installed MLC-LLM (conda, source):
How you installed TVM-Unity (pip, source):
Python version (e.g. 3.10):
GPU driver version (if applicable):
CUDA/cuDNN version (if applicable):
TVM Unity Hash Tag (python -c "import tvm; print('\n'.join(f'{k}: {v}' for k, v in tvm.support.libinfo().items()))", applicable if you compile models):
Any other relevant information:

Additional context

Hi @panghongtao, I also ran into this before. It might be possible that you need to first remove all the .pyc files. The reason is likely because our old flow deprecation and rename a few days ago, and there are some local Python “caches” that are not updated together with the deprecation and rename.
Here is the stackoverflow question I referred to https://stackoverflow.com/questions/52477683/importerror-bad-magic-number-in-time-b-x03-xf3-r-n-in-django.

Hi @panghongtao, I also ran into this before. It might be possible that you need to first remove all the .pyc files. The reason is likely because our old flow deprecation and rename a few days ago, and there are some local Python “caches” that are not updated together with the deprecation and rename.
Here is the stackoverflow question I referred to https://stackoverflow.com/questions/52477683/importerror-bad-magic-number-in-time-b-x03-xf3-r-n-in-django.

There is no.py file in my project. Cleaning these files is cleaning python? I tried to create a new python environment using conda. But this error still occurs. I don't know if this solution will solve the problem. Because I have multiple projects on my computer. I'm worried about how deleting.py files will affect other projects.

.pyc files compiled Python source and once you run your source program they are created again so this shouldn't cause any issues with your other projects as long you have the source for them. Otherwise, you can modify that command to only delete .pyc files in that particular folder: find . -name <PATH_TO_MLC>*.pyc -delete

Hopefully above solutions have addressed your problem, closing the issue. Kindly feel free to open another one in case any other problem arises.
Thanks


Block the code

The code wasn't blocked

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


New parse function

We need to write a function that has the same content as the __express function but doesn't return a callback, instead just returns the parsed HTML. You could just copy-and-paste the function code into a new function and instead of return callback (returnContent) just use return returnContent, or use other functions to have less code and repeat them in all of the functions.
Bonus: It would be great to add a way to parse code in a string, not just in a file.

Latest commits with returnHTML function solved


Better makefile

the targets now depend on the things they should:
make and make-debug - dollarskip.c
install and clean - make
uninstall - install
that means that now for example if you run sudo make install without running make first, make will automatically run the make target first.

because I made this branch from #20, I had to revert changes to dollarskip.c.

What does this do?
I can't see a purpose but it is probably easy to understand

It gives the targets in the makefile "dependencies" so for example the install target needs the make target to be run first. This means that if you run the install target without running the make one first, make will automatically run the make target first.
I also made build-debug compile with debug signs for debuggers like gdb.

Do you have any idea why build is failing, it came with when the new DollarSkip 2.0 was released.

It's failing on the test part.
I suggest we remove it and make a new workflow for the test that throughs a bunch of commands at it.
Maybe we could switch to cmake and use ctest for tests?

a few updates:
adding  a - before a command makes make ignore errors for that command.
assigning a variable with ?= means that will only be assigned that value if the variable doesn't exist yet. that means that the user can compile with their compiler of choice (like this: CC=<compiler> make).
everything else is just a better way (I guess as that's the way I have seen makefiles written in all sorts of projects) to do the same things this pr did from the start.
I recommend that you read this articel about writing makefiles, its really good.

I'll review all of these PRs within the next week probably. But looks good this.

Are we keeping all the branches?

No, I just wanted to keep them for a short while, forgot you could restore them


Qt5.15.15

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

continuation of #219

download of source code gets stalled :(

travis ci cancels out during source download, what can I do about it?

Hey sorry for not responding. I thought a bot had opened this due to an auto-upgrade path.
Truthfully, I am not sure this is desirable from an conda-forge ecosystem perspective.
The reason is that qtwebengine today cannot be built out easily for mac and windows.
https://github.com/conda-forge/qt-webengine-feedstock/issues/38
if you derisk qt5webengine, then we can start this upgrade.
The last migrations of qt5webengine have avoided dependencies on mac and windows, so they have been able to move forward, but having this package would create the following confusion for the solver:

I want the latest version of so many packages, but i can't find qtwebengine.
I can install old versions with qtwebengine.

We might be able to use the large machines donated by quantsight.

I am interested in having qt5webengine updated too. Qt has just released 5.15.13 though so we should aim for that I think. What can I do to help with that?

Yes. I would aim for lates version today.
We can download the manual artifacts from azure here. And upload to our own private channels while we derisk the build system.
The general problem with our CIs is that an error will occur at hour 16/17 and we simply can’t detect it unless we get a successful build.

We can download the manual artifacts from azure here. And upload to our own private channels while we derisk the build system.

Sorry I don't quite get what you mean here. Could you be a little more specific about what's the process here?
I have available an x86_64 Linux/windows machine and arm64 macOS machine where I can try to build locally, I can use docker on the mac to build linux_aarch64 too I think. I don't have native ppc64 but maybe it can be emulated or cross compiled?

The parameter you want to specify is store_build_artifacts
https://github.com/conda-forge/conda-forge.github.io/blob/575274e8675a35c90d80967a92c748c8a5121551/docs/maintainer/conda_forge_yml.md?plain=1#L90
Then you can:

Download the artifact for linux, windows, osx
Upload them to your personal channel.
For qtwebengine
Add your channel to the CIs during initial development with channel_sources: https://github.com/conda-forge/conda-forge.github.io/blob/575274e8675a35c90d80967a92c748c8a5121551/docs/maintainer/conda_forge_yml.md?plain=1#L250
Get the builds to work.

It will take a while to get step 5 to work, but with your perseverance, others will come in and help!

For emulation, you need to install qemu-user-static on your linux box, then you can build whatever linux platform
It takes approximately 6x longer to emulate. So not recommended until the very end.

@conda-forge-admin, please rerender

The parameter you want to specify is store_build_artifacts

it doesn't look like that's possible for travis though, guess I'll need to build qt-main aarch64 locally?

it doesn't look like that's possible for travis though, guess I'll need to build qt-main aarch64 locally?

Yeah, but i have hope for this one. I can build it too and upload it.
oddly, x86 targets older Centos by default, so you will have more challenges there.

Out of curiosity, what is stopping you from moving to qt6 today?

Out of curiosity, what is stopping you from moving to qt6 today?

My goal in the end is to build and ship freecad/ondsel-es to users and it doesn't seem to be ready for qt6 yet. I compiled on linux with qt6 and the experience was still very buggy. Hopefully this will get fixed before end of year but for now I think it's safer to remain in qt5. vtk has also not migrated yet to qt6 and of course I also need webengine too although we plan on eventually dropping this dependency as it's very problematic.

so where do I download the artifacts?

Click on the CI results, and then "3 published" it might get to 4.

thanks, uploaded to https://anaconda.org/freecad/qt-main/files will begin trying qtwebengine maybe tomorrow. I built qt-main linux_aarch64 locally on my mac

I get this error when trying to build locally:
WARNING: failed to get install actions, retrying.  exception was: Unsatisfiable dependencies for platform osx-arm64: {MatchSpec("qt-main==5.15.13=h7e66d85_0")}
Encountered problems while solving:
  - package qt-main-5.15.13-h7e66d85_0 is excluded by strict repo priority

what could be going wrong?

are you using the build-locally.py script?
you need to add your channel as a source and rerender.
If you open a PR on qt-webengine i can also review there as well.

webengine attempt: https://github.com/conda-forge/qt-webengine-feedstock/pull/48

Typically to merge from main, i check out main
git checkout main
git pull upstream main
git checkout qt5.12
git rebase -i main

and delete all the rerender commits.
That gets rid of 90% of the merge conflicts.

this fell off my radar for a while but I'm back at it now with 5.15.15

packages available at https://anaconda.org/freecad/qt-main/files logs here:
qt-main-5.15.15-logs.tar.gz

new webengine PR: https://github.com/conda-forge/qt-webengine-feedstock/pull/69

Thanks lets get a "stronger pin" going for qt5.15 so we can have time to build out qt-webengine.

I added it to the ABI migration branches.
@conda-forge-admin please rerender

I guess the new patch means new builds for linux.

yes, I don't have ppc64le hardware so that one takes long. I don't know if it's relevant there though given that the bug is for touch devices. I'm building aarch64 locally now.

if you have a linux server available, just start the job, then by the time you think about a long term solution, it will be done lol.

The patch should be ABI compatible so I don't think it's necessary to wait for ppc64le, I'll let it run tonight before sleep probably

The patch should be ABI compatible so I don't think it's necessary to wait for ppc64le

no sorry, unless the requester of the PPC64le jobs steps forward, I'm not willing to drop it.
If you want to upload your own jobs to your own channel, feel free to do so. But breaking the compatibility matrix of conda-forge is not an option without a dedicated PR and some time for review from those interested.
Thank you for helping on the windows and OSX (I can't do those). I can and will start a PPC64le job.
Together we can get the full compatibility working.
I know you can likely "mix and match" qt versions with qt-webengine, but I just don't want to.

PPC64le should be done within 24 hours or so.

note that I have ppc64le built without the patch

note that I have ppc64le built without the patch,

i understand, but if you are calling both the same "build number" it gets VERY confusing.
I can help on PPC64le front, and willing to do so.

and webengine is not supported for ppc64le

Yeah that is ok. happy to work with people when they want to request it.
For example to aid during this transition to qt5.15.15 i've also added a dedicated branch for the previous version so the bot can keep it updated in case we get any delays.

no problem, new aarch64 build is up https://anaconda.org/freecad/qt-main/5.15.15/download/linux-aarch64/qt-main-5.15.15-h3c8598d_0.conda and here is the log:
linux_aarch64_.tar.gz
there's probably no point in copying my builds for the other platforms since they build successfully in azure

do we need to rebuild webengine too?

and webengine is not supported for ppc64le

Yeah that is ok. happy to work with people when they want to request it.

something like that would likely require massive patching, I think the problem there is that chromium itself doesn't support ppc64le, because webengine is not being built for ppac64le I think pyside is not being built for ppc64le either but I wonder if it's possible to build pyside without webengine support, I would expect yes. That can come later if there is interest

I think pyside is not being built for ppc64le

I'm not sure where the interest for qt-main came from for ppc64le. But pyside should be buildable without qt-webengine.
I merged to allow you to start building qt-webengine in a few hours.
the ppc64le build should finish on my machine "soon TM"

Can you please add a label "cfep03" to

so i can copy that alone and not everything else?

command for me to run once completed:
anaconda copy --from-label cfep03 --to-label main --to-owner conda-forge freecad/qt-main/5.15.15

done

meanwhile my ppc64le build has finished and is available in the repo, here's the build log:
qt-main-5.15.15-linux_ppc64le_.tar.gz
I added the ppc label to it if you want to pull that one too

Thanks I uploaded both.
I guess the "wait for emulation" strategy works ^_^


Error with datatable textbox filter

When use this plugin with the filter of datatable, this plugin bind an event call "filter" and make a reflow of the header.
The problem is, if the filter wich was call reflow is a textbox filter , this textbox lose the focus like this :

->

So the user can't type many character right now.
For fix it I offer this solution :
/**
 * returns a function that updates the floating header's cell widths.
 * @return {Function}
 */
function reflow(){
  var i;
  var numCols = columnNum(); //if the tables columns changed dynamically since last time (datatables), rebuild the sizer rows and get a new count

  return function(){
    $tableCells = $tableColGroup.find('col');
    var $rowCells = getSizingRow($table, $tableCells, $fthCells, ieVersion);

    if($rowCells.length == numCols && numCols > 0){
      if(!existingColGroup){
        for(i=0; i < numCols; i++){
          $tableCells.eq(i).css('width', '');
        }
      }

      /** add for fix it **/
      var focusTextBox=null;
      var activeElement=document.activeElement;
      if(activeElement && $(activeElement).parents($table).length > 0) {
        if (activeElement.tagName && activeElement.tagName.toLowerCase() == "input" && activeElement.type.toLowerCase() == "text") {
          focusTextBox = activeElement;
        }
      }

      unfloat();
      var widths = [];
      for(i=0; i < numCols; i++){
        widths[i] = getOffsetWidth($rowCells.get(i));
      }
      for(i=0; i < numCols; i++){
        $headerCells.eq(i).width(widths[i]);
        $tableCells.eq(i).width(widths[i]);
      }
      refloat();

      /** add for fix it **/
      if(focusTextBox) {
        $(focusTextBox).focus();
      }

    } else {
      $floatTable.append($header);
      $table.css(layoutAuto);
      $floatTable.css(layoutAuto);
      setHeaderHeight();
    }
    $table.triggerHandler("reflowed", [$floatContainer]);
  };
}

If the active element is a textbox and is a descendant of the table i replace after reflow the cursor in the textbox.
Can you integrate this modification in this plugin ? I can open a pull request if you want ...
Thanks for this awesome plugin !

This code looks too specific to a problem one user has to force it upon everyone.
I would rather add another event that will be triggered before the reflow logic takes place and you can bind to that and to reflowed and put this logic in there.
The solution this plugin provides is floating your table's header, doing stuff with document.activeElement is outside of that scope.

closing with no response. reopen if you have a reply

Sorry, I was on vacation, but your solution is ok for me !


Version 2.031.2

I have just cleaned up all issues in the repository (that was quite a task).
They are now all labeled correctly and have a title that should tell what is requested/bugged.
Besides I have created a project called Version 2.031.2.
Which can be seen in the Project-tab or here.
Here we can collect all issues and PR's that need to be tackled before we can release the next minor version.
All help is greatly appreciated and we are always willing to further explain any issues or requests!

With #396 this release is now ready!


FormulaInstaller: link tmp kegs during rescue

[X] Have you followed the guidelines in our Contributing document?
[X] Have you checked to ensure there aren't other open Pull Requests for the same change?
[X] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[ ] Have you successfully run brew tests with your changes locally?

In #6807, the FormulaInstaller began to rescue from
FormulaInstallationAlreadyAttemptedError but there is the
potential for tmp kegs to remain unlinked in the cellar (noted in https://github.com/Homebrew/brew/pull/6807#issuecomment-561830681).
I've copied the code for linking tmp kegs from the
next rescue statement to execute here as well.
This issue has recently been manifesting in the following CI build of ours:

 https://build.osrfoundation.org/job/gazebo-install-one_liner-homebrew-amd64/1634/

Nice work, thanks @scpeters!


Duplicate checking for schema and cred def

Signed-off-by: Ian Costanzo<EMAIL_ADDRESS>See issue:  https://github.com/hyperledger/aries-cloudagent-python/issues/1707

Codecov Report

Merging #1721 (9d862ad) into main (8970e5b) will decrease coverage by 0.01%.
The diff coverage is 62.50%.

@@            Coverage Diff             @@
##             main    #1721      +/-   ##
==========================================
- Coverage   95.27%   95.26%   -0.02%
==========================================
  Files         528      528
  Lines       33093    33109      +16
==========================================
+ Hits        31531    31541      +10
- Misses       1562     1568       +6

FYI: This PR introduced a breaking change and now broke AATH because AATH tries to recreate the schema. Before ACA-Py would swallow this. Do we need to update this in ACA-Py? Or do we need to change this behaviour in ACA-Py (i.e. not throwing a record, but also not re-creating the schema)
On a side note, it seems to query by the schema_name and schema_version and throw an error if any records are found. Does that mean if I change my public did and try to create a schema with the same name and version I created with a previous did it will throw an error? I think a schema is unique by name, version, and did.

Created a PR that fixes this in AATH: https://github.com/hyperledger/aries-agent-test-harness/pull/473

I'm not sure I understand why ACA-Py should fail if you try to create the same schema multiple times. Shouldn't it just create the schema if it doesn't exist and continue happily if it does?  AFAIK - the problem previously was that it was creating multiple instances of the same schema (definitely a problem) and returning multiple instances.
On the second point -- I agree in principle that an agent should be allowed to create a schema with the same name but a different DID, since that is permitted on Indy. Not sure how crucial it is though -- it's an "unlikely" situation right now -- but eventually someone will want it...  How big a can of worms does it open if the query includes the public DID for the agent in the check for a schema?

Good question about "resource already exists" - I returned an error but other options are available.  I think we should return something that specifies that the resource already exists, though, rather than giving the same result whether the resource was just created or not.  409 or 303 are options, see https://stackoverflow.com/questions/3825990/http-response-code-for-post-when-resource-already-exists
Regarding the "same schema different did" also a good point, I didn't think about an issuer with multiple DID's.  I don't know if this is a realistic example though (wouldn't there just be a single schema and then multiple cred defs?) but I think the topic of an aca-py agent with multiple public DID's introduces more issues than just this one.


Add WebVTT support (subtitles)

It seems WebVTT is ideal for subtitles support:
https://www.w3.org/TR/webvtt1/
https://developer.mozilla.org/en-US/docs/Web/API/WebVTT_API

WebVTT is what Videojs supports as well: https://docs.videojs.com/docs/guides/text-tracks.html
We would allow an optional upload to spee.ch like we do today (would need spee.ch to support the format). Maybe we can allow other formats and convert automatically on spee.ch also.

The subtitles is one of the most important feature that should be supported on LBRY as soon as possible.

May I clarify about this statement?

The new video player will support subs, but I think we should have users bake them into the mp4

Does that imply that player is supposed to be capable of showing subtitles shipped within the video file container (like srt stream in .mkv or mov_text in .mp4)?
I tried uploading .mkv (with srt subs) and .mp4 (with mov_text), but neither seem to show subtitles in the player out of the box ;c

https://odysee.com/@klesun:0/MLP_FiM---S01E01---Friendship-is-Magic-Part-1.mkv.better_bitrate:6
https://odysee.com/@klesun:0/mlp_s01e01:b

Subtitles should be a top priority. If you want businesses/people to use odysee, the lbry team must allow subtitles in videos in order to be ADA compliant.
Otherwise, you are putting all users of odysee at risk for lawsuits for not using subtitles at all.
It would be nice to have something like youtube that generates subtitles for uploaded videos automatically even if they are not accurate depictions of speech to text. Otherwise, have the option for users to upload a custom subtitle file for more speech to text accuracy. A newer fork of the autosub github project can do all of this.
I also have the same questions as klesun does above. Are there any example videos in odysee that have subtitles currently that we can see?


Crashing/freezing on iOS 15 and greater (open issue from Xamarin)

Open issue: [Bug] [iOS] Freezing/App Crash during Navigation - new in iOS 15
Workaround:

Download the package and add it to your solution
Modify the TopTabbedRenderer with the following lines

        protected override void Dispose(bool disposing)
        {
            // Avoid disposing on iOS 15 due to open issue on GitHub: https://github.com/xamarin/Xamarin.Forms/issues/14783
            if (UIDevice.CurrentDevice.CheckSystemVersion(15, 0))
                return;

            [...]
        }

@alexmartinezm It'll be great if you create a PR :) .


I tried to submit my challenge but it doesnt got submitted

Challenge Name

Issue Description

Browser Information

Browser Name, Version:
Operating System:
Mobile, Desktop, or Tablet:

Your Code
// If relevant, paste all of your challenge code in here

Screenshot

@achalofficial If you get stuck with a challenge, please first ask questions to the Help Room. Happy coding!

@achalofficial can you give us more information on the challenge and your code? It'll make helping you resolve this issue easier.

@achalofficial  Please go through Help I have found a bug

@achalofficial closing as stale. If you still have an issue, please make another issue and fill out the issue template with more details on your issue. Happy coding!


Set exclusive_access to all tests for Kesch

The issue can be closed. The only remaining tests are on the private repository. Comments were made at the issues for the individual tests.

@teojgo Could we have PR in the private repo that sets this flags and fixes the issue completely?

@vkarak There are individual issues for each of the tests that need to be adapted anyway.

Ok, we can close this then.


Documenting NuGet.org Protocols

@kraigb, can you review for the text correctness and grammar? Also I am not sure how to include this in TOC. Can that change be attached to this PR?
Please review but do NOT merge. @anangaur will merge it.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/API/NuGet-Protocols.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/API/NuGet-Protocols.md
:white_check_mark:Succeeded

docs/TOC.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/API/NuGet-Protocols.md
:white_check_mark:Succeeded

docs/TOC.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fixed bug in https handling

The default return startServers() needs to be wrapped in an else block because the first return statement was done in a callback (inside pem.createCertificate).

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

Oooh, nice catch @CreepGin!
Better still would be to have a consistent return value in the if (httpsOptions === true) case, but that may not be straightforward to get patched through...
This at least avoids trying to start everything twice. 😆

^ File #569 to track that.
Thanks @CreepGin!


Add an "x" to the settings' search bar, to remove your last query.

Currently, you have to manually remove your query should you want to go back to the settings overview.
An "x", which removes your query, would aid in this.

Fixed in #3451


Fixes Three In a Row Elements

Fixes https://github.com/CircuitVerse/CircuitVerse/pull/1233#issuecomment-602029888
Describe the changes you have made in this PR -
I have fixed the elements panel to display three elements by default.
Screenshots of the changes (If any) -

Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Coverage remained the same at 84.279% when pulling 4e844777eb79e26404116babdfaf088b64b489da on Shivansh2407:patch-36 into 89970529e97561132c2fd98bffee6d65ea4f6aff on CircuitVerse:master.

@satu0king please review and merge 👍

@satu0king please review 👍

@Shivansh2407 can you check in the current version if it is 3 rows for you?

@satu0king I have checked. It is available two in a row. I have updated to show three elements in a row

Fixed in the recent commit


Update TROUBLESHOOTING.md

Fix typo.

Hello @ciceropablo,Thanks again for the PR! It's HACKTOBERFEST! We want to show our appreciation by sending                                              you some special Hacktoberfest swag. If you have not already, could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX

Thank you!!

Thanks @thinkingserious and @mbernier! 👍


Define mapping between ?fields and the Prefer header

See http://discuss.jsonapi.org/t/must-i-return-all-resource-object-attributes-without-a-sparse-fieldset-specified/486 and https://tools.ietf.org/html/rfc7240

Closing this. After looking into the Prefer header a bit more, any mapping here would be inappropriate:

Due to the inherent complexities involved with properly implementing
server-driven content negotiation, effective caching, and the
application of optional preferences, implementers are urged to
exercise caution when using preferences in a way that impacts the
caching of a response and SHOULD NOT use the Prefer header mechanism
for content negotiation.


What does "The file may be corrupted" error means?

What does it mean the below errors?
KneDa@KNEDA-PC d:\software\vcpkg
> vcpkg install libpng
The following packages will be built and installed:
    libpng:x86-windows
Starting package 1/1: libpng:x86-windows
Building package libpng:x86-windows...
-- CURRENT_INSTALLED_DIR=D:/software/vcpkg/installed/x86-windows
-- DOWNLOADS=D:/software/vcpkg/downloads
-- CURRENT_PACKAGES_DIR=D:/software/vcpkg/packages/libpng_x86-windows
-- CURRENT_BUILDTREES_DIR=D:/software/vcpkg/buildtrees/libpng
-- CURRENT_PORT_DIR=d:/software/vcpkg/ports/libpng/.
-- Using cached D:/software/vcpkg/downloads/libpng-1.6.32.tar.xz
-- Testing integrity of cached file...
CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:47 (message):

  File does not have expected hash:

          File path: [ D:/software/vcpkg/downloads/libpng-1.6.32.tar.xz ]
      Expected hash: [ 24cf4d38dc08d1db7721fc580d8bf6cee9a105afcea32f1245b7bb3727c8a8645ad733a24534782ef5dfb0d4315a5fd6b0eee9e7f0704e0c635350f80c19a708 ]
        Actual hash: [ 3b53ccf0243cd4a6a731a898bfdba95c3d22342e7e64f9e9a4a2bce5f74ac809943f570ed92c9077101a178f82f49ece1d1c4bfee1f288ea61d99fb36f6a243e ]

  Please delete the file and retry if this file should be downloaded again.

Call Stack (most recent call first):
  scripts/cmake/vcpkg_download_distfile.cmake:59 (test_hash)
  ports/libpng/portfile.cmake:5 (vcpkg_download_distfile)
  scripts/ports.cmake:72 (include)

Error: Building package libpng:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: libpng:x86-windows
  Vcpkg version: 0.0.90-2017-09-25-084b1afe9af1690916fab95bbb80c6891109f0fc

Additionally, attach any relevant sections from the log files above.

KneDa@KNEDA-PC d:\software\vcpkg
> vcpkg install freetype
The following packages will be built and installed:
    freetype:x86-windows
  * libpng:x86-windows
Additional packages (*) will be installed to complete this operation.
Starting package 1/2: libpng:x86-windows
Building package libpng:x86-windows...
-- CURRENT_INSTALLED_DIR=D:/software/vcpkg/installed/x86-windows
-- DOWNLOADS=D:/software/vcpkg/downloads
-- CURRENT_PACKAGES_DIR=D:/software/vcpkg/packages/libpng_x86-windows
-- CURRENT_BUILDTREES_DIR=D:/software/vcpkg/buildtrees/libpng
-- CURRENT_PORT_DIR=d:/software/vcpkg/ports/libpng/.
-- Using cached D:/software/vcpkg/downloads/libpng-1.6.32.tar.xz
-- Testing integrity of cached file...
CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:47 (message):

  File does not have expected hash:

          File path: [ D:/software/vcpkg/downloads/libpng-1.6.32.tar.xz ]
      Expected hash: [ 24cf4d38dc08d1db7721fc580d8bf6cee9a105afcea32f1245b7bb3727c8a8645ad733a24534782ef5dfb0d4315a5fd6b0eee9e7f0704e0c635350f80c19a708 ]
        Actual hash: [ 3b53ccf0243cd4a6a731a898bfdba95c3d22342e7e64f9e9a4a2bce5f74ac809943f570ed92c9077101a178f82f49ece1d1c4bfee1f288ea61d99fb36f6a243e ]

  Please delete the file and retry if this file should be downloaded again.

Call Stack (most recent call first):
  scripts/cmake/vcpkg_download_distfile.cmake:59 (test_hash)
  ports/libpng/portfile.cmake:5 (vcpkg_download_distfile)
  scripts/ports.cmake:72 (include)

Error: Building package libpng:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: libpng:x86-windows
  Vcpkg version: 0.0.90-2017-09-25-084b1afe9af1690916fab95bbb80c6891109f0fc

Additionally, attach any relevant sections from the log files above.

KneDa@KNEDA-PC d:\software\vcpkg
> vcpkg install freeimage
The following packages will be built and installed:
    freeimage:x86-windows
  * ilmbase:x86-windows
  * jasper:x86-windows
  * jxrlib:x86-windows
  * lcms:x86-windows
  * liblzma:x86-windows
  * libpng:x86-windows
  * libraw:x86-windows
  * libwebp:x86-windows
  * openexr:x86-windows
  * openjpeg:x86-windows
  * tiff:x86-windows
Additional packages (*) will be installed to complete this operation.
Starting package 1/12: libpng:x86-windows
Building package libpng:x86-windows...
-- CURRENT_INSTALLED_DIR=D:/software/vcpkg/installed/x86-windows
-- DOWNLOADS=D:/software/vcpkg/downloads
-- CURRENT_PACKAGES_DIR=D:/software/vcpkg/packages/libpng_x86-windows
-- CURRENT_BUILDTREES_DIR=D:/software/vcpkg/buildtrees/libpng
-- CURRENT_PORT_DIR=d:/software/vcpkg/ports/libpng/.
-- Using cached D:/software/vcpkg/downloads/libpng-1.6.32.tar.xz
-- Testing integrity of cached file...
CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:47 (message):

  File does not have expected hash:

          File path: [ D:/software/vcpkg/downloads/libpng-1.6.32.tar.xz ]
      Expected hash: [ 24cf4d38dc08d1db7721fc580d8bf6cee9a105afcea32f1245b7bb3727c8a8645ad733a24534782ef5dfb0d4315a5fd6b0eee9e7f0704e0c635350f80c19a708 ]
        Actual hash: [ 3b53ccf0243cd4a6a731a898bfdba95c3d22342e7e64f9e9a4a2bce5f74ac809943f570ed92c9077101a178f82f49ece1d1c4bfee1f288ea61d99fb36f6a243e ]

  Please delete the file and retry if this file should be downloaded again.

Call Stack (most recent call first):
  scripts/cmake/vcpkg_download_distfile.cmake:59 (test_hash)
  ports/libpng/portfile.cmake:5 (vcpkg_download_distfile)
  scripts/ports.cmake:72 (include)

Error: Building package libpng:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: libpng:x86-windows
  Vcpkg version: 0.0.90-2017-09-25-084b1afe9af1690916fab95bbb80c6891109f0fc

Additionally, attach any relevant sections from the log files above.

Additionally, attach any relevant sections from the log files above.
KneDa@KNEDA-PC d:\software\vcpkg
> vcpkg install rapidxml
The following packages will be built and installed:
    rapidxml:x86-windows
Starting package 1/1: rapidxml:x86-windows
Building package rapidxml:x86-windows...
-- CURRENT_INSTALLED_DIR=D:/software/vcpkg/installed/x86-windows
-- DOWNLOADS=D:/software/vcpkg/downloads
-- CURRENT_PACKAGES_DIR=D:/software/vcpkg/packages/rapidxml_x86-windows
-- CURRENT_BUILDTREES_DIR=D:/software/vcpkg/buildtrees/rapidxml
-- CURRENT_PORT_DIR=d:/software/vcpkg/ports/rapidxml/.
-- Downloading https://sourceforge.net/projects/rapidxml/files/rapidxml/rapidxml%201.13/rapidxml-1.13.zip/download...
-- Downloading https://sourceforge.net/projects/rapidxml/files/rapidxml/rapidxml%201.13/rapidxml-1.13.zip/download... OK
-- Testing integrity of downloaded file...
CMake Error at scripts/cmake/vcpkg_download_distfile.cmake:47 (message):

  File does not have expected hash:

          File path: [ D:/software/vcpkg/downloads/rapidxml-1.13.zip ]
      Expected hash: [ 6c10583e6631ccdb0217d0a5381172cb4c1046226de6ef1acf398d85e81d145228e14c3016aefcd7b70a1db8631505b048d8b4f5d4b0dbf1811d2482eefdd265 ]
        Actual hash: [ 3b53ccf0243cd4a6a731a898bfdba95c3d22342e7e64f9e9a4a2bce5f74ac809943f570ed92c9077101a178f82f49ece1d1c4bfee1f288ea61d99fb36f6a243e ]

  The file may be corrupted.

Call Stack (most recent call first):
  scripts/cmake/vcpkg_download_distfile.cmake:87 (test_hash)
  ports/rapidxml/portfile.cmake:4 (vcpkg_download_distfile)
  scripts/ports.cmake:72 (include)

Error: Building package rapidxml:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: rapidxml:x86-windows
  Vcpkg version: 0.0.90-2017-09-25-084b1afe9af1690916fab95bbb80c6891109f0fc

Additionally, attach any relevant sections from the log files above.

This is happening after upgraded from 0.0.81 to 0.0.9:
git pull
.\bootstrap-vcpkg.bat
vcpkg update

And then just trying to upgrade all my packages.
As you can see, the above commands are trying unsuccesfully to install libpng, freetype, freeimage, glew and rapidxml
In any case, what does it mean these type of errors? I've tried cleaning the downloads folder and running vcpkg again but it didn't make any difference.
Hope you can advice.
Thanks in advance

Oh, I've checked manually what vcpkg has tried to download and it seems Sourceforge is in "static offline mode", that's why the downloaded package https://sourceforge.net/projects/glew/files/glew/2.1.0/glew-2.1.0.tgz was 1kb out of wget request. Alright, I think I understand the logic here, no black magic involved :)
Btw, do you know any command line to calculate the hash used by vcpkg? I've tried cygwin's md5sum but it's not reporting that type of hashes. It'd be helpful to have these type of tools in case I try to download manually the src package files of the libraries.

To get the hash, I just run vcpkg install and let it fail. In the error message it lists the "Actual hash:" that vcpkg calculated. Plug that hash into the portfile.cmake SHA512 and it should pass the hash check. All the standard warnings about you are now bypassing the hash security check on your download and assuming the download has not been altered from the original.

Sourceforge is down. You can get 2.1.0 from https://github.com/nigels-com/glew/releases

Hi @brupelo , thanks for reporting this issue!
Please open a new issue if this is still a problem for you.
Thx.


[Feature Request] Web Browsing

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Diagram :  Agent > Tools
Agent has many Tools

CVE Critical for Lang chain
https://nvd.nist.gov/vuln/detail/CVE-2023-29374

Agent(LangChain for Anthropic models)
https://python.langchain.com/docs/modules/agents/agent_types/xml_agent/

bedrock construct cdk
https://constructs.dev/packages/bedrock-agents-cdk/v/0.0.9?lang=typescript


Can jupyter notebook cells be profiled?

I'm not sure if there's a way to tie VS Code and notebooks. But I'd rather have this feature available for notebooks so that it would be usable by a wider userbase. Will have to look into it


chore(master): release 6.4.1

:robot: I have created a release beep boop
6.4.1 (2023-10-31)
Bug Fixes

use compatible axios (#874) (ad8f5d2)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ethersphere/bee-js/releases/tag/v6.4.1 :sunflower:

Release v6.4.1 addresses this.


Getting back to 11.0 macOS deployment target in libcxx 17.0.6

Comment:
Hi, thanks for all the work you guys are putting into this!
As a newbie user, I'm naïvely trying to build Python wheels à la cibuildwheel but using conda as the environment. I'm aware of this discussion and Isuru's suggestion. I'm going down this foolish route thanks to Boost.Python...
All my conda libraries have 11.0 deployment targets on arm64 (as cibuildwheel likes it) except for libcxx 17.0.6 he7857fb_1, which is now on 12.0. This upsets delocate-wheel.
I've looked at #154 for more insight. If I try to install the alternate version (h5a255bb_1) with label libcxx_macos_lt_12, conda complains that package libcxx-17.0.6-h5a255bb_1 has constraint __osx <12 conflicting with __osx-13.6.7-0. This is fair enough, although I had hoped that this label would refer to deployment target and not actual OS version.
Am I using the libcxx_macos_lt_12 label wrong?
The alternative is to just grin and bear the minimum OS version of 12.0 in my wheels (not so bad either).
Or maybe this is a complete wild goose chase?  😅

See https://github.com/conda-forge/libcxx-feedstock/pull/154#discussion_r1651881651

Excellent!

Think you need the build/number 0 of 17.0.6 for macOS 10.13
xref: https://github.com/conda-forge/libcxx-feedstock/pull/154#discussion_r1651892371
@h-vetinari (in case I've misunderstood something)

Thanks, that's exactly what I ended up doing. I'm glad to hear it's a temporary issue.


Use latest Spring style in docs

As implemented in Spring Cloud Open Service Broker docs build file with these commits:

https://github.com/spring-cloud/spring-cloud-open-service-broker/commit/d55bc84b5640d57490d823ae6590585f678805cb
https://github.com/spring-cloud/spring-cloud-open-service-broker/commit/e6700a0b18aae5c30f8e50a449789816cf0a69bf

@fifthposition I'm hoping to get a release out soon. Thoughts on this?


Optimize column vector filter

What problem does this PR solve?
Issue Number: ref #6839
Problem Summary:
What is changed and how it works?
rewrite ColumnVector<T>::filter with avx2
perf test as follows (larger is better):
# before
[qiuyang@172-16-5-85 ~/projects/tiflash/cmake-build-release 11:51:22]$ ./dbms/src/Columns/tests/column_vector_perftest int64 filter 10000 50 60
Test int64-filter rows=10000 columns=50 seconds=60
Filter/s: 471278
[qiuyang@172-16-5-85 ~/projects/tiflash/cmake-build-release 11:52:27]$ ./dbms/src/Columns/tests/column_vector_perftest int64 filter 10000 2 60
Test int64-filter rows=10000 columns=2 seconds=60
Filter/s: 43811
[qiuyang@172-16-5-85 ~/projects/tiflash/cmake-build-release 11:54:01]$ ./dbms/src/Columns/tests/column_vector_perftest int64 filter 10000 1 60
Test int64-filter rows=10000 columns=1 seconds=60
Filter/s: 119135

# after
[qiuyang@172-16-5-85 ~/projects/tiflash/cmake-build-release 11:56:26]$ ./dbms/src/Columns/tests/column_vector_perftest int64 filter 10000 50 60
Test int64-filter rows=10000 columns=50 seconds=60
Filter/s: 651413
[qiuyang@172-16-5-85 ~/projects/tiflash/cmake-build-release 11:57:40]$ ./dbms/src/Columns/tests/column_vector_perftest int64 filter 10000 2 60
Test int64-filter rows=10000 columns=2 seconds=60
Filter/s: 90984
[qiuyang@172-16-5-85 ~/projects/tiflash/cmake-build-release 11:59:05]$ ./dbms/src/Columns/tests/column_vector_perftest int64 filter 10000 1 60
Test int64-filter rows=10000 columns=1 seconds=60
Filter/s: 163634

Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-all-tests

/run-unit-test

/cc @solotzg @SeaRise @ywqzzy


Add 50 unit tests for hCenter() and vCenter() methods

Add a variable to unit test PinLayout's warnings

Codecov Report

Merging #37 into master will increase coverage by 1.67%.
The diff coverage is 99.47%.

@@            Coverage Diff             @@
##           master      #37      +/-   ##
==========================================
+ Coverage   93.71%   95.38%   +1.67%
==========================================
  Files          14       14
  Lines        2767     2946     +179
==========================================
+ Hits         2593     2810     +217
+ Misses        174      136      -38

Impacted Files
Coverage Δ

PinLayoutTests/PinEdgesSpec.swift
100% <100%> (ø)
:arrow_up:

PinLayoutTests/MarginsSpec.swift
100% <100%> (ø)
:arrow_up:

PinLayout/PinLayoutImpl.swift
84.89% <85.71%> (+5.3%)
:arrow_up:

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 d44d396...51107ce. Read the comment docs.


Add basal info uploading to Nightscout

Send:

Basal patterns
Active basal pattern from the status message
Temp basals from the status message
LGS temp basal from the status message, with a note that it was a LGS

Mock for LGS note.

Active pattern would be a great enhancement but should upload periodically (or even better, be stored on phone and only transmitted to NS when a change is detected) in order to minimise data use..

👍

closing - feature available on 0.6+


click_handlers: Ensure typeahead clicks do not collapse compose box.

Added condition for when a user clicks anywhere in a typeahead menu, be it the header or empty space at the top and bottom, to not collapse the compose box.
This fixes the bug where clicking anywhere on the typeahead except the options would result in the compose box collapsing, but the typeahead menu staying open and still working (on selecting a option in this state, it would be inserted at the start, as visible on reopening the compose box).
Fixes: Issue described on CZO
Screenshots and screen captures:
https://user-images.githubusercontent.com/68962290/227606609-782f3a18-7754-48f7-bbad-da275d18842b.mp4

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Looks great, merged, thanks @N-Shar-ma!


Support full outer join

Hi all!
I have some use cases that I need to join tables with full outer join. Is there a support for it? Or a work around without using full raw SQL statement?
Thanks a lot,

I found it, It can be done using the type param of the join method.
Sorry for opening the issue :)


Removal of afterEffects

I noticed in the latest version (0.20.1), the afterEffects API was removed.
Was there an alternative solution added?  I previously handled some login/app setup functions on the top-most component when the component was first loaded (it is never unmounted - so it ran only once on initialization).
Is it safe to use something like:
const App = () => {
  const setup= () => {
    // handle setup/init stuff here
  }

  setup()

  return
Hello World

}

export default App

Thanks

Hi @Phillips126 what you are looking for is: https://github.com/ryansolid/solid/issues/225. I should have linked this right off the release notes.
The short answer is createEffect is what you are looking for, perhaps with untrack. In order to account for asynchronous concurrency and complications around ref handling I built scheduling into the reactive system now with effects executing after render.

Great, I appreciate you're quick reply!

To be fair it you don't read from the DOM ie using refs what you wrote would be fine as well as the component body only runs once.


Update README.md with correct documentation links

This is a fix for https://github.com/datastax/astrajs/issues/28


Test references fails with inliner = -Inf and brillig

Aim
Trying to run test_programs with different --inliner-aggressiveness in https://github.com/noir-lang/noir/pull/6429
Expected Behavior
Tests under execution_success should work.
Bug
❯ nargo execute --force --inliner-aggressiveness -9223372036854775808 --force-brillig
error: Failed to solve program: 'Failed to solve brillig function'
    ┌─ /Users/aakoshh/Work/aztec/noir/test_programs/execution_success/references/src/main.nr:115:12
    │
115 │     assert(*x == 1);
    │            -------
    │
    = Call stack:
      1. /Users/aakoshh/Work/aztec/noir/test_programs/execution_success/references/src/main.nr:28:5
      2. /Users/aakoshh/Work/aztec/noir/test_programs/execution_success/references/src/main.nr:115:12

Failed to solve program: 'Failed to solve brillig function'

To Reproduce

cd test_programs/execution_success/references
nargo execute --force --force-brillig --inliner-aggressiveness -9223372036854775808

Workaround
None
Workaround Description
No response
Additional Context
No response
Project Impact
None
Blocker Context
No response
Nargo Version
nargo version = 0.36.0 noirc version = 0.36.0+2f0cb3e80f3d93a1dee77fffacc397811e300257 (git version hash: 2f0cb3e80f3d93a1dee77fffacc397811e300257, is dirty: false)
NoirJS Version
No response
Proving Backend Tooling & Version
No response
Would you like to submit a PR for this Issue?
None
Support Needs
No response

To minimise the SSA size the program has been reduced to the following:
fn main(mut x: Field) {
    regression_2255();
}

// The `mut x: &mut ...` caught a bug handling lvalues where a double-dereference would occur internally
// in one step rather than being tracked by two separate steps. This lead to assigning the 1 value to the
// incorrect outer `mut` reference rather than the correct `&mut` reference.
fn regression_2255() {
    let x = &mut 0;
    regression_2255_helper(x);
    assert(*x == 1);
}

fn regression_2255_helper(mut x: &mut Field) {
    *x = 1;
}

Running it with the following command:
cargo run -q -p nargo_cli --  --program-dir . execute --force --force-brillig --inliner-aggressiveness -9223372036854775808 --show-ssa

The initial SSA is this:
Initial SSA:
brillig(inline) fn main f0 {
  b0(v0: Field):
    v1 = allocate
    store v0 at v1
    call f1()
    return
}
acir(inline) fn regression_2255 f1 {
  b0():
    v0 = allocate
    store Field 0 at v0
    call f2(v0)
    v3 = load v0
    v5 = eq v3, Field 1
    constrain v3 == Field 1
    return
}
acir(inline) fn regression_2255_helper f2 {
  b0(v0: &mut Field):
    v1 = allocate
    store v0 at v1
    v2 = load v1
    store Field 1 at v2
    return
}

And the final form:
After Array Set Optimizations:
brillig(inline) fn main f0 {
  b0(v0: Field):
    call f1()
    return
}
brillig(inline) fn regression_2255 f1 {
  b0():
    v0 = allocate
    store Field 0 at v0
    call f2(v0)
    v3 = load v0
    constrain v3 == Field 1
    return
}
brillig(inline) fn regression_2255_helper f2 {
  b0(v0: &mut Field):
    return
}

Somehow the store Field 1 at v2 completely disappeared from regression_2255_helper f2.

NB if we run the program with 0 aggressiveness:
cargo run -q -p nargo_cli --  --program-dir . execute --force --force-brillig --inliner-aggressiveness 0 --show-ssa

Then everything gets inlined and eliminated, including the assertions.
After Array Set Optimizations:
brillig(inline) fn main f0 {
  b0(v0: Field):
    call f1()
    return
}
brillig(inline) fn regression_2255 f1 {
  b0():
    return
}

As an experiment we can change the test code to remove the outer mut from mut x: &mut Field (which is what the docs of regression_2255 say caused the bug the test is trying to capture):
fn regression_2255_helper(x: &mut Field) {
    *x = 1;
}

In this case the SSA of the helper becomes simpler by writing directly to the v0 reference:
brillig(inline) fn regression_2255_helper f2 {
  b0(v0: &mut Field):
    store Field 1 at v0
    return
}

However after the mem2reg pass this store is completely removed, and the test fails:
brillig(inline) fn regression_2255_helper f2 {
  b0(v0: &mut Field):
    return
}

Just for the record, with mut in place, we had this SSA:
brillig(inline) fn regression_2255_helper f2 {
  b0(v0: &mut Field):
    v1 = allocate <- allocate a new reference for `mut x`
    store v0 at v1 <- store the reference v0 itself, not its value, in v1
    v2 = load v1 <- load the contents of v1, which is the reference v0, into v2
    store Field 1 at v2 <- store a value at v2, which should write to v0
    return
}


Why do we need splashscreens other than<EMAIL_ADDRESS>

So icon-genie adds all these splashscreens for me:
<splash height="768" src="res/screen/ios/screen-ipad-landscape.png" width="1024" />
<splash height="1536" src="res/screen/ios/screen-ipad-landscape-2x.png" width="2048" />
<splash height="1024" src="res/screen/ios/screen-ipad-portrait.png" width="768" />
<splash height="2048" src="res/screen/ios/screen-ipad-portrait-2x.png" width="1536" />
<splash height="320" src="res/screen/ios/screen-iphone-landscape.png" width="480" />
<splash height="640" src="res/screen/ios/screen-iphone-landscape-2x.png" width="960" />
<splash height="480" src="res/screen/ios/screen-iphone-portrait.png" width="320" />
<splash height="960" src="res/screen/ios/screen-iphone-portrait-2x.png" width="640" />
<splash height="1136" src="res/screen/ios/screen-iphone-portrait-568h_2x.png" width="640" />
<splash height="1334" src="res/screen/ios/Default-667h.png" width="750" />
<splash height="2208" src="res/screen/ios/Default-736h.png" width="1242" />
<splash height="1242" src="res/screen/ios/Default-Landscape-736h.png" width="2208" />
<splash height="2436" src="res/screen/ios/Default-2436h.png" width="1125" />
<splash height="1125" src="res/screen/ios/Default-Landscape-2436h.png" width="2436" />
<splash height="2732"<EMAIL_ADDRESS>width="2732" />

Which is great. But to my knowledge only this one is used:
<splash height="2732"<EMAIL_ADDRESS>width="2732" />

The other formats are legacy formats but are not used when you have<EMAIL_ADDRESS>or that's what I thought.
So I have just two questions:

Why are the others generated?
Are we able to disable those legacy formats and choose to only generate<EMAIL_ADDRESS>?

Cheers!

Done with the complete rewrite of Icon Genie v2, but not with universal. With a better approach, but ditched the legacy stuff.


Git stash 3 dot menu doesn't work

VSCode Version: 1.32.3
OS Version: Windows 10

Steps to Reproduce:

Try to stash any changes via 3 dot menu on Source Control layer
Error occured (see screenshots)

@Kontik what is your git version?
git stash push command introduced in 2.13.x version.
I will send a PR to fix that

@joaomoreno can i send a PR that change git stash push command to git stash save.
There is just one rule that stash message can not be start with minus (-).
Its working well otherwise.
It is ready to send if you accept.

Please update to the latest git version.


[Design][Linkupdate][1474642]

Link updated

Docs Build status updates of commit a5ebbfb:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/apps-in-teams-meetings/teams-apps-in-meetings.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit a5ebbfb:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/apps-in-teams-meetings/teams-apps-in-meetings.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


vulnix: 1.10.0 -> 1.10.1

Motivation for this change
Pick up new release which contains the pyyaml 6.x fix.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[ ] aarch64-darwin

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

supersedes #157845

Result of nixpkgs-review pr 158357 run on aarch64-darwin 1

1 package built:

vulnix


Disfunctional Brew

Error Code
No response
Your description of what happened:
My brew saves a different copy of itself into drive every time i press save.
Here is a look at my google drive:

As you can see the file is triplicated, however, the first file is different:

This is missing from the first file, that and the edited markdown are the only difference.
Also, in my user page, i can't see this brew's views:

Do you see all three versions in your user page?

I just opened it again, i have 4 versions of it, two published, and two unpublished.
I'm guessing the error caused the metadata of the brew to dissapear, and so HB created a new brew each time i saved, that could also explain why i can't see the view count on the original.
This is the document itself.

Opened my user page and i can see the view count, but i don't know if i should delete all of them or just all except one


Penta implementation

Code Coverage Summary [using JaCoCo]

Coverage Type
Coverage

Coverage:
20.14%

Branches:
18.52%


LUN number and HCTL output

It reads:

Here, sdc is the disk that we want, because it is 50G. If you add multiple disks, and aren't sure which disk it is based on size alone, you can go to the VM page in the portal, select Disks, and check the LUN number for the disk under Data disks. Compare the LUN number from the portal to the last number of the HTCL portion of the output, which is the LUN

Yet looking at the output shown above this comment there would appear to be multiple HCTL output all with LUN 0:
sda     0:0:0:0      30G
├─sda1             29.9G /
├─sda14               4M
└─sda15             106M /boot/efi
sdb     1:0:1:0      14G
└─sdb1               14G /mnt
sdc     3:0:0:0      50G

E.g the last number of the HTCL appears to be 0 in all of the cases shown - so what does it mean exactly?

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

ID: 2036c044-4fbd-6016-c026-fea724b530d6
Version Independent ID: dde369ae-7a34-0753-5dfe-3720bf69e996
Content: Add a data disk to Linux VM using the Azure CLI - Azure Virtual Machines
Content Source: articles/virtual-machines/linux/add-disk.md
Service: storage
Sub-service: disks
GitHub Login: @roygara
Microsoft Alias: rogarana

I'm trying to develop a script that I could execute unattended to create a partition and format it based on the newly attached disk. The only thing I know in advance is LUN number of the attached disk.
I have multiple instances of VM's that have attached the same (shared) disk, and in most cases the shared disk - which always has LUN 0, ends up being sdc
However in once instance, the shared disk on one of the VM's is reported as sda

For example here is a command that will give me the name of the disks (filters our partitions) with a specific size. However suppose the disk being attached happened to be same size as the OS disk, how would I know which is which - given both HTCL's end with 0.
lsblk -lnd -o Name,Size | awk '$2=="512G"'

@dazinator
Thanks for your feedback! We will investigate and update as appropriate.

@dazinator Firstly, Apologies for the delay response! I'd recommend posting your question in https://azure.microsoft.com/en-us/support/community/ since it's not directly related to the documentation. We'll follow up there. We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

@SumanthMarigowda I'd argue it does relate directly to the documentation.. I've quoted the portion of the documentation that is confusing or misleading - but let me put it here again, it states:

Here, sdc is the disk that we want, because it is 50G. If you add multiple disks, and aren't sure which disk it is based on size alone, you can go to the VM page in the portal, select Disks, and check the LUN number for the disk under Data disks. Compare the LUN number from the portal to the last number of the HTCL portion of the output, which is the LUN

Yet look at the screenshot I posted - multiple disks have 0 as the last portion of the HCTL so how can this be true?


Graphene DB System.NotSupportedException

Using a 3.5.17 DB hosted on Graphene
Using Neo4jClient 4.0.0 (which seems to be using Neo4j.Driver.Signed 4.1.1) updated from previously used <IP_ADDRESS>.

When attempting to connect, I get this exception:
System.NotSupportedException: The Neo4j server does not support any of the protocol versions supported by this client. Ensure that you are using driver and server versions that are compatible with one another.
I was previously getting a different exception:
Neo4j.Driver.ServiceUnavailableException: Connection with the server breaks due to IOException: Unexpected end of stream, read returned 0 Please ensure that your database is listening on the correct host and port and that you have compatible encryption settings both on Neo4j server and driver. Note that the default encryption setting has changed in Neo4j 4.0. but after switching the code to manually construct the IDriver and pass that to BoltGraphClient, the exception switched to the above.
New code:
driver = Neo4j.Driver.GraphDatabase.Driver(uri, Neo4j.Driver.AuthTokens.Basic(user, password),
   o => o.WithEncryptionLevel(encryptionLevel));

IBoltGraphClient client = new BoltGraphClient(driver);

If I look at the Graphene DB documentation, it suggests that the 4.1 driver isn't supported by the 3.5.x database. That said, if I run a local instance 3.5.9 in Docker, for example, it does work.  I'm uncertain if the incompatibility here is an issue with Graphene or something else altogether.
Am I missing something?

Also having issues here. Unable to connect to Neo4j Community version 4.1.3.
Using
bolt uri = "bolt://db-mydb.graphenedb.com:24786";
Exception
Neo4j.Driver.ServiceUnavailableException: Connection with the server breaks due to IOException: Unable to read data from the transport connection: Foi forçado o cancelamento de uma conexão existente pelo host remoto.. Please ensure that your database is listening on the correct host and port and that you have compatible encryption settings both on Neo4j server and driver. Note that the default encryption setting has changed in Neo4j 4.0.
 ---> System.IO.IOException: Unable to read data from the transport connection: Foi forçado o cancelamento de uma conexão existente pelo host remoto..
 ---> System.Net.Sockets.SocketException (10054): Foi forçado o cancelamento de uma conexão existente pelo host remoto.
   --- End of inner exception stack trace ---
   at System.Net.Sockets.Socket.AwaitableSocketAsyncEventArgs.ThrowException(SocketError error, CancellationToken cancellationToken)
   at System.Net.Sockets.Socket.AwaitableSocketAsyncEventArgs.GetResult(Int16 token)
   at System.Threading.Tasks.ValueTask`1.ValueTaskSourceAsTask.<>c.<.cctor>b__4_0(Object state)
--- End of stack trace from previous location where exception was thrown ---
   at Neo4j.Driver.Internal.Connector.SocketClient.DoHandshakeAsync()
   at Neo4j.Driver.Internal.Connector.SocketClient.ConnectAsync(IDictionary`2 routingContext)
   at Neo4j.Driver.Internal.Connector.SocketConnection.InitAsync()
   at Neo4j.Driver.Internal.Connector.DelegatedConnection.TaskWithErrorHandling(Func`1 task)
   --- End of inner exception stack trace ---
   at Neo4j.Driver.Internal.Connector.DelegatedConnection.TaskWithErrorHandling(Func`1 task)
   at Neo4j.Driver.Internal.ConnectionPool.CreateNewPooledConnectionAsync()
   at Neo4j.Driver.Internal.ConnectionPool.CreateNewPooledConnectionAsync()
   at Neo4j.Driver.Internal.ConnectionPool.<>c__DisplayClass43_0.<<AcquireAsync>b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at Neo4j.Driver.Internal.Logging.DriverLoggerUtil.TryExecuteAsync[T](ILogger logger, Func`1 func, String message)
   at Neo4j.Driver.Internal.AsyncSession.<>c__DisplayClass27_0.<<RunAsync>b__0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at Neo4j.Driver.Internal.Logging.DriverLoggerUtil.TryExecuteAsync[T](ILogger logger, Func`1 func, String message)
   at Neo4jClient.BoltGraphClient.ConnectAsync(NeoServerConfiguration configuration)
   at db4oDataAccess.neo4jh.<>c__DisplayClass5_0.<<TryConnectAsync>b__0>d.MoveNext() in C:\Users\tonyv\source\repos\SistemaMulti\db4oDataAccess\neo4jh.cs:line 54
--- End of stack trace from previous location where exception was thrown ---
   at db4oDataAccess.helper.Do(Func`1 task, TimeSpan retryInterval, Int32 maxAttemptCount) in C:\Users\tonyv\source\repos\SistemaMulti\db4oDataAccess\_helper.cs:line 35 ... 1 attempt.

or
https uri = "https://db-mydb.graphenedb.com:24786/db/data/";
Exception
System.Net.Http.HttpRequestException: An error occurred while sending the request.
 ---> System.IO.IOException: The response ended prematurely.
   at System.Net.Http.HttpConnection.FillAsync()
   at System.Net.Http.HttpConnection.ReadNextResponseHeaderLineAsync(Boolean foldedHeadersAllowed)
   at System.Net.Http.HttpConnection.SendAsyncCore(HttpRequestMessage request, CancellationToken cancellationToken)
   --- End of inner exception stack trace ---
   at System.Net.Http.HttpConnection.SendAsyncCore(HttpRequestMessage request, CancellationToken cancellationToken)
   at System.Net.Http.HttpConnectionPool.SendWithNtConnectionAuthAsync(HttpConnection connection, HttpRequestMessage request, Boolean doRequestAuth, CancellationToken cancellationToken)
   at System.Net.Http.HttpConnectionPool.SendWithRetryAsync(HttpRequestMessage request, Boolean doRequestAuth, CancellationToken cancellationToken)
   at System.Net.Http.RedirectHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
   at System.Net.Http.HttpClient.FinishSendAsyncBuffered(Task`1 sendTask, HttpRequestMessage request, CancellationTokenSource cts, Boolean disposeCts)
   at Neo4jClient.Execution.ResponseBuilder.PrepareAsync()
   at Neo4jClient.Execution.ResponseBuilder.ExecuteAsync(String commandDescription, Func`2 continuationFunction)
   at Neo4jClient.Execution.ResponseBuilder`1.ExecuteAsync(String commandDescription, Func`2 continuationFunction)
   at Neo4jClient.NeoServerConfiguration.GetConfigurationAsync(Uri rootUri, String username, String password, String realm, Nullable`1 encryptionLevel, ExecutionConfiguration executionConfiguration)
   at Neo4jClient.GraphClient.ConnectAsync(NeoServerConfiguration configuration)
   at db4oDataAccess.neo4jh.<>c__DisplayClass5_0.<<TryConnectAsync>b__0>d.MoveNext() in C:\Users\tonyv\source\repos\SistemaMulti\db4oDataAccess\neo4jh.cs:line 54
--- End of stack trace from previous location where exception was thrown ---
   at db4oDataAccess.helper.Do(Func`1 task, TimeSpan retryInterval, Int32 maxAttemptCount) in C:\Users\tonyv\source\repos\SistemaMulti\db4oDataAccess\_helper.cs:line 35 ... 1 attempt.

does not work.

I managed to make it work with 3.5.18 by just changing the connection string to
bolt+s://db-mydb.graphenedb.com:24786
For make it work with 4.1.3 I have to use bolt+s , specify the DefaultDatabase
var c = new Neo4jClient.BoltGraphClient(new Uri(bolturi), username, password);
c.DefaultDatabase = "graph.db";

and also have to change the Cypher queries to use $param instead of {param}

Hello!
It's probably going to be due to the schema you're trying to use, can you share the URI you are attempting.
Connecting to a 3.5x server instance locally works with bolt - but with graphene - I guess there are SSL certs.
The list of schemes are here - from what I can see - and what @TonyHenrique  has written - you'll want bolt+s

@TonyHenrique / @cskardon  - Thanks for the ideas!  I saw documentation about the bolt+s but didn't think it applied since I was still on an older instance.
Original uri was bolt://db-[keyredacted].graphenedb.com:24786
Tried it with the encryption level and the bolt+s scheme and bolt+ssc, got a new exception:
System.ArgumentException: The encryption and trust settings cannot both be set via uri scheme and driver configuration. uri scheme = bolt+s, encryption = Encrypted, trust =
Then we tried turning encryption back to "None" and the new scheme, then got:
System.ArgumentException: The encryption and trust settings cannot both be set via uri scheme and driver configuration. uri scheme = bolt+s, encryption = None, trust =
So then we went back to not setting encryption level at all and just setting the scheme in the uri.  I also changed it back to constructing the IBoltGraphClient client = new BoltGraphClient(new Uri(uri), user, password); Once we did that, we were back to:
System.NotSupportedException: The Neo4j server does not support any of the protocol versions supported by this client. Ensure that you are using driver and server versions that are compatible with one another.
So my final attempt was to this:
var driver = Neo4j.Driver.GraphDatabase.Driver(uri, Neo4j.Driver.AuthTokens.Basic(user, password));
IBoltGraphClient client = new BoltGraphClient(driver);

Which still yielded the "System.NotSupportedException".
I ended up downgrading back to the <IP_ADDRESS> package version for now.


shared_preferences don't work destop app

How to use shared_preferences in fluttet_destop?

See http://flutter.dev/desktop#plugins. Currently no plugins in flutter/plugins support desktop; implementations will come as the plugin APIs and tooling on each platform stabilize.


update moon info

This PR updates the moon info distributed with the toolchain version:
$ moon version --all
moon 0.1.20241125 (6fe6396 2024-11-25)
moonc v0.1.20241125+34a5d819a
moonrun 0.1.20241125 (6fe6396 2024-11-25)

Pull Request Test Coverage Report for Build 3931
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 79.886%

Totals

Change from base Build 3927:
0.0%

Covered Lines:
4357

Relevant Lines:
5454

💛  - Coveralls


FSLeyes: Change default Name for saving the morphometrics file

For FSLeyes, the default name for saving the morphometrics file is Untitled.xlsx.

I suggest that we change to something more intuitive, that is, Morphometrics.xlsx.

i suggest: axon_morphometrics.xlsx
also: can you pls open an issue to output in CSV (no more Excel)
thx

i suggest: axon_morphometrics.xlsx
also: can you pls open an issue to output in CSV (no more Excel)
thx

I'm the one to blame for Excel; I preferred that option. In other Python projects, I've rarely been able to load CSV's created by Python well in Excel (usually some formatting issues), but that might because of my French operating system + commas... I'm not opposed to moving to CSV-only, but we may need to provide some extra guidance to users not used to this file format if they have issues loading it.

In other Python projects, I've rarely been able to load CSV's created by Python well in Excel (usually some formatting issues)

never been a pb for me-- example (generated by Python via SCT):
map.csv.zip

but that might because of my French operating system + commas...

i think you have your answer Frenchie 😉

I'm not opposed to moving to CSV-only, but we may need to provide some extra guidance to users not used to this file format if they have issues loading it.

Good point. We should at least provide the option. It's literally 2-3 lines of code to add support for both CSV and XLS (when user specifies the file name, we can check the specified extension in the backend-- as they do in GIMP).


Simplify mint after NUT-06 update (amount deprecation)

Hey, I was glancing over the code and saw that after the NUT-06 update here, you might have left some idiosyncrasies from earlier times in the code:
https://github.com/thesimplekid/cashu-crab/blob/b287c87df0e6d87468d7dfa6d0a48e014967834f/src/mint.rs#L133
Since the mint does not know anything about "first" and "second" promises (blind signatures) anymore, you can treat these as one big slice and don't have to separate them anymore. Instead, the wallet now has to figure out which of the promises it received it wants to keep and which to send on. I am guessing, this is what your code does here:
https://github.com/thesimplekid/cashu-crab/blob/b287c87df0e6d87468d7dfa6d0a48e014967834f/src/wallet.rs#L110C21-L110C29
Keep up the insane work! Cheers!

Hey,
Thanks for taking a look, I kept that in so the mint could handle requests from wallets that have not updated and those that did. But you're right best to remove it now.
Yes, I updated the wallet to deprecate the amount field and figure out the promises.
https://github.com/thesimplekid/cashu-crab/blob/b287c87df0e6d87468d7dfa6d0a48e014967834f/src/mint.rs#L115-L126

150ba204019cc92b7310d1abb230793a679e97ad


federico di pierro png

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind user-interface
/kind content

/kind translation

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area blog

/area documentation

/area videos

What this PR does / why we need it:
pic is needed for contributor page
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

dc8ed44 Merge pull request #1 from jsalinas29/jsalinas29-contributor-march-2023

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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: jsalinas29
Once this PR has been reviewed and has the lgtm label, please assign jasondellaluce for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Signed-off-by: jsalinas29<EMAIL_ADDRESS>

closing cause i messed this up (again) ack!


[main] Upgrade to latest dependencies

release -dprotaso
/cc knative-sandbox/eventing-writers
/assign knative-sandbox/eventing-writers
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/reconciler-test 93a1083...177f826:
  > 177f826 upgrade to latest dependencies (# 377)
bumping knative.dev/pkg f78a006...4a03844:
  > 4a03844 fix double close of channel (# 2575)

/lgtm
/approve


Generalize publishing GitHub Action to support flutter

The action to publish packages doesn't work for Flutter packages due to the following error.
Run dart pub get
  dart pub get
  shell: /usr/bin/bash -e {0}
  env:
    PUB_CACHE: /home/runner/.pub-cache
    DART_HOME: /opt/hostedtoolcache/dart-sdk
    PUB_TOKEN: ***
Resolving dependencies...
Because google_fonts requires the Flutter SDK, version solving failed.

https://github.com/material-foundation/flutter-packages/actions/runs/4079639516/jobs/7031216899
Can this action be generalized or another one published for Flutter?

@devoncarew is dart pub get required?

I had the same problem after updating to Flutter 3.7.
Something has changed and while previously dart pub publish could be used to publish Flutter packages, it doesn't work anymore. I migrated my package to use flutter pub publish to fix the problem.

@sigurdm, I think the fact that we patched dart pub publish to do dart analyze and implicit dart pub get, cause the regression that @bartekpacia mentioned.
On topic, I think it would be great if the reusable-workflow could be used to publish flutter packages with:
https://github.com/dart-lang/setup-dart/blob/main/.github/workflows/publish.yml
I really would love to recommend that people use the reusable workflow as it'll let us patch the workflow after they've adopted it. So in theory we could potentially add SLSA support in the future.

@sigurdm, any ideas? should we extend the reusable-workflow to install Flutter, when there is an SDK dependency on Flutter?
I'm not keep on patching the community maintained flutter-action repository to have reusable-workflow, because we would be unable to trust such a workflow on pub.dev -- should ever want to do thing that requires trust -- like inspecting github APIs to check if branch protection is enabled, etc, stuff that SLSA does.

should we extend the reusable-workflow to install Flutter, when there is an SDK dependency on Flutter?

I don't currently see other good ways to go around this. The only alternative would be to disable the resolution in the publish-step (as we cannot resolve against sdk dependencies without the sdk).

I think it's a bit non-intuitive to have the setup-dart action install a flutter sdk. Is there some way to compose this with the existing actions which install a flutter sdk?

We don't have to make setup-dart do that in general, just .github/workflows/publish.yml

Perhaps having a separate .github/workflows/publish-flutter.yml script might be enough of an indication that this could install flutter specific bits.

We discussed this a bit on the weekly pub-meeting.
I think from the pub side of things:

we would prefer a single workflow for publishing (to avoid confusion and differences sneaking in)
we don't want to implement a pub publish --ignore-warnings (that would lead to broken publications) -> we actually need to install flutter
we don't want to run community code (not clear that flutter-actions will always do the right thing)

So that looks like having a dash-team script that can download and install Flutter is needed.

We think it would be enough to default to latest stable, with a manual parameter for the channel and version.
This can probably be done with a curl and tar invocation.
If we install Flutter, we might not need to install the Dart SDK, but can use the one embedded in a flutter-sdk (this might seem like a premature optimization, but otherwise we get the question of choosing the right sdk for publishing a package).

we don't want to run community code (not clear that flutter-actions will always do the right thing)

More importantly, it'll be hard to do SLSA in the future.

we would prefer a single workflow for publishing (to avoid confusion and differences sneaking in)

If we decided to do a github.com/flutter/setup-flutter repository, then perhaps it wouldn't be bad to put the logic in there..
Otherwise, perhaps it's find if we just extend publish.yml to support Flutter, whether we always download Flutter or not won't matter much -- in practice this only runs with publishing, which shouldn't be frequent.

Otherwise, perhaps it's find if we just extend publish.yml to support Flutter, whether we always download Flutter or not won't matter much -- in practice this only runs with publishing, which shouldn't be frequent.

+1. If there ever is a flutter/setup-flutter repo and publishing workflow, its workflow could simply depend on publish.yml.

I'd like to add that flutter pub publish doesn't play well with Automated Publishing with GitHub Actions on pub.dev.
The most widely used subosito/flutter-action doesn't support pulling the pub.dev token, so it fails.
The result is that I have to use dart-lang/setup-dart, and then subosito/flutter-action to be able to use flutter pub publish.
See this commit for an example of how I fixed it.
Not a good experience, if you ask me.

we might not need to install the Dart SDK, but can use the one embedded in a flutter-sdk

Keep in mind that the dart: APIs supported by the regular Dart SDK, and the Dart SDK vended into the Flutter SDK, are not the same. The Flutter version of the Dart SDK adds dart:ui and removes dart:mirrors (and several web core libraries). You can use the one to do publishing for the other, as long as you don't want to do something like run static analysis.
The main issue is that dart pub publish now wants to run static analysis as part of publishing validation? It may be worth considering making that portion of the checks opt-in (e.g., dart pub publish --run-analysis-checks).

Can't the flow detect whether the package to be published is a Flutter package and if yes download Flutter and do everything with flutter ... otherwise use the current workflow and with dart ...?

It may be worth considering making that portion of the checks opt-in (e.g., dart pub publish --run-analysis-checks).

We landed a new option for opting out of all validations --skip-validations: https://github.com/dart-lang/pub/pull/3935

We landed a new option for opting out of all validations --skip-validations

Can we add this option to the publish workflow?

This is also an issue I am having right now.
I cannot publish my flutter package with the reusable workflow, as it doesnt use that flag.

I think this should work, testing now:
https://docs.github.com/en/actions/deployment/security-hardening-your-deployments/configuring-openid-connect-in-cloud-providers#requesting-the-jwt-using-the-actions-core-toolkit

I would expect this to work, but it doesn't seem to yet:
  steps:
    - name: 📚 Git Checkout
      uses: actions/checkout@v4
    - name: 🐦 Setup Flutter
      uses: subosito/flutter-action@v2

    - name: 🪪 Get Id Token
      uses: actions/github-script@v6
      with:
        script: |
          let pub_token = await core.getIDToken('https://pub.dev/')
          core.exportVariable('PUB_TOKEN', pub_token)

    - name: 📢 Authenticate
      shell: ${{ inputs.shell }}
      run: flutter pub pub token add https://pub.dev --env-var PUB_TOKEN

    - name: 📦 Install dependencies
      shell: ${{ inputs.shell }}
      run: flutter pub get
      working-directory: ${{ inputs.working-directory }}

    - name: 🌵 Dry Run
      shell: ${{ inputs.shell }}
      run: flutter pub publish --dry-run
      working-directory: ${{ inputs.working-directory }}

    - name: 📢 Publish
      shell: ${{ inputs.shell }}
      run: flutter pub publish -f
      working-directory: ${{ inputs.working-directory }}

This is based on reading: https://github.com/dart-lang/setup-dart/blob/e9a814f39d5452701455a47847735e0f482026c8/lib/main.dart#L167 and https://docs.github.com/en/actions/deployment/security-hardening-your-deployments/configuring-openid-connect-in-cloud-providers#requesting-the-jwt-using-the-actions-core-toolkit

I am a new contributor who attempts to add a flutter_publish.yml workflow in my fork - https://github.com/dart-lang/setup-dart/pull/139

Seems like we can have conditional steps: https://docs.github.com/en/actions/writing-workflows/choosing-when-your-workflow-runs/using-conditions-to-control-job-execution
We could make an input "publish-with-flutter"
with:
  publish-with-flutter: true

And if that is true we invoke some setup-flutter action instead of setup-dart...
That would be slightly manual, but perhaps good enough

Our reasoning, is that we want to enable publishing flutter packages first.
Later we can polish if we feel the need.
Is subosito/flutter-action still the prefered action for installing flutter?


Exec tachikoma update<PHONE_NUMBER>3423

:hamster::hamster::hamster:   Strategy none
Powered by Interval Pull Request as a Service - Tachikoma.io Announcement: 2015-06-28 Updated permissions - Tachikoma.io

Coverage remained the same at 68.0% when pulling 57f5fcb9f6b8ad8b4db07da6fa94b5d8d090bb69 on pazjacket:tachikoma/update-20160612163423 into 0c477cea399180e0dc0455b31d685efdbee0f3af on sanemat:master.


max_unfollows_per_day setting not working

No matter what I do, I cannot get the bot to unfollow 999 users.
When running unfollow_everyone.py the bot stops unfollowing at 350 (DONE: Total unfollowed 350 users.)
When unfollow everyone from multi_script_CLI.py it stops at 100 users.
My bot.py file looks like this:
def init(self,
whitelist=False,
blacklist=False,
comments_file=False,
max_likes_per_day=1000,
max_unlikes_per_day=1000,
max_follows_per_day=999,
max_unfollows_per_day=999,
max_comments_per_day=999,
max_blocks_per_day=999,
max_unblocks_per_day=999,
max_likes_to_like=999,
filter_users=True,
max_followers_to_follow=2000,
min_followers_to_follow=10,
max_following_to_follow=2000,
min_following_to_follow=10,
max_followers_to_following_ratio=10,
max_following_to_followers_ratio=2,
min_media_count_to_follow=3,
max_following_to_block=2000,
like_delay=10,
unlike_delay=10,
follow_delay=30,
unfollow_delay=30,
comment_delay=60,
block_delay=30,
unblock_delay=30,
stop_words=['shop', 'store', 'free']):
super(self.class, self).init()
My setting.txt looks like this:
999
999
999
999
100
150
2000
250
5000
10
10
2
7
30
10
30
30
60
None
I don't know what I am doing wrong.
Please help.

the same for me!!! how do you fix it?

Hey gersanchez, I noticed I actually had a different version of python installed.
If you do pip install instabot, it will tell you that it is already installed and will give you the path. Mine was under usr/local/lib/python2.7/site-packages/
Then you just edit the pot.py in there and the issue should be fixed.
Good luck!

what python version do you have!?, i have python2.7

Thanks for the tip with the path andretg79!!! It works fine.


Add real-world example

Same behaviour like in redux repo.

We'll probably leave this on https://github.com/salsita/redux-elm and provide more minimalistic example instead.

Closing in favor of 1.0.0


listing all versions fails silently

I have installed this plugin to manage bun. Right after installing this plugin, I executed the following command and please check the snap for details.
asdf list all bun

Here are other relevant informations
Output of asdf info
OS:
Darwin Nurs-MacBook-Pro.local 21.6.0 Darwin Kernel Version 21.6.0: Fri Sep 15 16:17:23 PDT 2023; root:xnu-8<IP_ADDRESS>3.5~1/RELEASE_X86_64 x86_64

SHELL:
zsh 5.8.1 (x86_64-apple-darwin21.0)

BASH VERSION:
5.2.15(1)-release

ASDF VERSION:
v0.13.1-0586b37

ASDF INTERNAL VARIABLES:
ASDF_DEFAULT_TOOL_VERSIONS_FILENAME=.tool-versions
ASDF_DATA_DIR=/Users/nurrony/.asdf
ASDF_DIR=/Users/nurrony/.zi/plugins/asdf-vm---asdf
ASDF_CONFIG_FILE=/Users/nurrony/.asdfrc

ASDF INSTALLED PLUGINS:
bun                          https://github.com/cometkim/asdf-bun.git main 51d18b3

Umm, do you have set $GITHUB_API_TOKEN in your env? If so try:
curl \
    -H 'Accept: application/vnd.github+json' \
    -H "Authorization: token $GITHUB_API_TOKEN" \
    "https://api.github.com/repos/oven-sh/bun/releases"

@cometkim, I don't have GITHUB_API_TOKEN in my env. Do I need to one? if yes, does it not sound little inconvenient?

No, I thought there was no way to get any error then. 🤔

@cometkim I just tried the your curl command with setting GITHUB_API_TOKEN as System Enviroment Variable. It works fine. But when I  run asdf list all bun getting this following error.

Can you try asdf plugin update bun and again?

@cometkim no luck back to the square 1 :(
❯ asdf list all bun
Plugin bun's list-all callback script failed with output:
<no output nothing>

umm, I cannot reproduce the issue on same environment, maybe it is better to debug on your end
here is a standalone script equivalent to the list-all
#!/usr/bin/env bash

set -eo pipefail

GITHUB_REPO="oven-sh/bun"
REPO_URL="https://github.com/$GITHUB_REPO"

curl_opts=(-fsSL)
if [ -n "${GITHUB_API_TOKEN:-}" ]; then
  curl_opts=("${curl_opts[@]}" -H "Authorization: token $GITHUB_API_TOKEN")
fi

function sort_versions() {
  sed 'h; s/[+-]/./g; s/.p\([[:digit:]]\)/.z\1/; s/$/.z/; G; s/\n/ /' |
    LC_ALL=C sort -t. -k 1,1 -k 2,2n -k 3,3n -k 4,4n -k 5,5n | awk '{print $2}'
}

function list_github_releases() {
  curl "${curl_opts[@]}" \
    -H "Accept: application/vnd.github+json" \
    "https://api.github.com/repos/$GITHUB_REPO/releases" |
    grep -o '"tag_name": "bun-v.*"' |
    sed -E 's/"tag_name": "bun-v(.*)"/\1/'
}

list_github_releases | sort_versions | sed 1,11d | xargs echo

Can you try this and report back what you find?

As you can see, it's not much.
To make it precise, you can test the commands one by one without piping. just list_github_realeases first.
Excluding s(slient) from curl_opts also help.


Preflight + txsub proxying to remote core (futurenet)

[ ] Figure out which repo this should live in

which component is doing this? cli? horizon?

See: https://github.com/stellar/stellar-core/pull/3500