Resolve potential memory leaks.

Leak occurs from the use of strdup() and not freeing the result. Additionally, as setenv() copies the input string, it is not needed.
Leak occurs if setenv() fails.

Compile: gcc -O0 -g example/main.c

.env file

SERVER=<IP_ADDRESS>

Valigrind/Memcheck results:

Before changes

> valgrind --leak-check=full -s ./a.out
...
==4497== HEAP SUMMARY:
==4497==     in use at exit: 18 bytes in 1 blocks
==4497==   total heap usage: 8 allocs, 7 frees, 6,051 bytes allocated
==4497==
==4497== 18 bytes in 1 blocks are definitely lost in loss record 1 of 1
==4497==    at 0x483877F: malloc (vg_replace_malloc.c:307)
==4497==    by 0x48E0E4A: strdup (strdup.c:42)
==4497==    by 0x109447: env_load (dot_env.h:116)
==4497==    by 0x109535: main (main.c:8)
==4497==
==4497== LEAK SUMMARY:
==4497==    definitely lost: 18 bytes in 1 blocks
==4497==    indirectly lost: 0 bytes in 0 blocks
==4497==      possibly lost: 0 bytes in 0 blocks
==4497==    still reachable: 0 bytes in 0 blocks
==4497==         suppressed: 0 bytes in 0 blocks
==4497==
==4497== ERROR SUMMARY: 1 errors from 1 contexts (suppressed: 0 from 0)

After changes

> valgrind --leak-check=full -s ./a.out
...
==3950== HEAP SUMMARY:
==3950==     in use at exit: 0 bytes in 0 blocks
==3950==   total heap usage: 4 allocs, 4 frees, 8,784 bytes allocated
==3950==
==3950== All heap blocks were freed -- no leaks are possible
==3950==
==3950== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)

Thanks for this!


Monitoring

Please add Cloudwatch monitoring to the Lambda/ECS as necessary and an SNSTopic to forward email to.  I checked in on this solution that we had deployed and found a Service Control Policy caused the lambda function to fail to kick off a new Task.  If there was monitoring on lambda invocations that forwarded to an email address, this would have been noticed much sooner.  So general monitoring of some kind to validate the solution is working correctly even at the basic level would be appreciated

Hi @carpnick thanks for the request. We will add this to the backlog and review in a future release. What level of monitoring are you expecting? Is it only to check if Lambda invocations are occurring correctly or any other details about the invocations?

Hi @carpnick thanks for the request. We will add this to the backlog and review in a future release. What level of monitoring are you expecting? Is it only to check if Lambda invocations are occurring correctly or any other details about the invocations?

I think any part of the solution that could fail to run where the customer thinks its running properly.  So just a pass/fail to alert the customer to look into it.

Lambda invocations failing
Container execution fails with a bad exit code

I think any part of the solution that could fail to run where the customer thinks its running properly.  So just a pass/fail to alert the customer to look into it.

Lambda invocations failing
Container execution fails with a bad exit code

We reviewed the request and do not have plans to build this in the immediate roadmap.


Change type of seed from string to int

Related to #13, this replaced the type signature of seed with int to make it align with the type signature of the TinyColor seed.

This is great, thanks a lot!
I added a test for this new fix.

You're welcome, and thank you for the fast response!


Issue #7928 Failure to convert ItemDisplayLink from underscore to dashes

#7928
we replaced the code as suggested by @MatteoPiovanelli-Laser  and tested it in this way the parameter behave correctly

Beautiful, thanks


Fix for latest jquery version.

Read up on prop vs. attr.  This little fix was needed to get "add all" working again for our version of jQuery (1.11).  I am sure there are other prop vs. attr issues with this plugin, I did not check or test the others.

I think the second argument to 'prop' needs to be boolean.   Here's an example of a local fix to multiselect from wikimedia: https://gerrit.wikimedia.org/r/#/c/406598/1/resources/vendor/jquery.ui.multiselect/ui.multiselect.js


The a flag is available with printf formatting

Currently pyformat.info says that printf-formatting doesn't do ascii().

Thanks!


Update the RBE images to the latest container versions

Automated PR created once per week to get the latest Docker images.
See https://github.com/tensorflow/tensorflow/blob/master/.github/workflows/update-rbe.yml

Hmm, that's not right. I'll have to look at this again next week.


[Bug] Can't launch the game without Code 5 fix

hi, with <IP_ADDRESS> the game wont launch after run it with any mod. i try ESUI mod but got an error in the cmd windows. if i disable mods the game run ok.

Try with the latest canary and enable the code 5 fix option.

We might have an update that will fix this https://github.com/tsunamods-codes/7th-Heaven/pull/17 has fixed a problem that was causing code 5 fix to be needed a lot, so when a new canary comes out that might have been fixed if you can test and let us know @Urbanhack86 would be wonderful

I can confirm the patch worked for me too ( I was using the Code 5 Fix as it didn't work without recently ). Thank you @barkermn01

No complains, good to go.


How much vram is needed

Hi, thank you for your great work!
How much vram do I need when testing?

Hi, thank you for your interest in our work!
To test REDS4 (180x320 video sequence) in our environment, you'll need about 5GB of GPU memory. Note that this may vary slightly depending on the OS and software version, and it also varies based on the size of the test video.

I will close this issue as there has been no further discussion. Please re-open the issue if there are additional comments.


support for proxy url option for use with react native web

to resolve https://github.com/ospfranco/link-preview-js/issues/69

hmm, any idea why the travis ci build failed?

TravisCI is not properly configured, I just needed the build for another project, don't worry about it

Cheers @megantaylor, I'll release a new version soon

released with version 2.0.5


White background appearing comment section in dark themes

Describe the bug
https://tailwind-nextjs-starter-blog.vercel.app/blog/new-features-in-v1
If you scroll down all the way to comments section,
comments section has white background..
From what I remember, this error appeared recently in latest updates..
also, the width shrank too.
Expected behavior
dark background in dark theme!!
Screenshots
Here's the link

The default CSS file has been moved to a separate file instead of an inline element in the HTML to help CSP compatibility.
If you're using that template, you need to update this line:
https://github.com/timlrx/tailwind-nextjs-starter-blog/blob/a03d358ef9bffeaa6e6bc8c32ad038cb3e9334be/next.config.js#L9
to include giscus.app, e.g.
style-src 'self' 'unsafe-inline' giscus.app;

unfortunately this didn't worked..
This is frustrating..

Wow thanks a lot. My log seemed perfect in dashboard. It turned out next.config.json didn't reflected properly in vercel.
I just added console.log and worked great
 async headers() {
    console.log(ContentSecurityPolicy)  // this line
    return [
      {
        source: '/(.*)',
        headers: securityHeaders,
      },
    ]
  },

I am closing this now
thanks again


Update MixinNetworkManager.java

Changed EventType.RECEIVE to EventType.SEND in MixinNetworkManager#sendPacket

Thank you for fixing this


Adding some description to all the topics

Add some theoretical knowledge to the individual topics in Readme.md file.

can you assign this to me?
Also how descriptive should the knowledge be , like with pictures and such or just textual explanation?

Descriptive - Just add text in the shoes of a person trying to explain something to someone
Media - You can use gifs/images/links whatever you like.
Tip: Instead of putting everything in the same readme file. You can create folder-specific readme's. It is all up to you.
Assigning you

if something is there for me then assign me for my 1st PR

I think you can pickup adding media to the readme. If you are intrested @neeraj542

I'm looking to complete my 1st PR

i can perfectly complete this task , please assign to me

@Hanuvesh assigned you

Sir, i can also work in this issue

@neeraj542

sir I'm working on this ..Have to merge PR


fix issue 791: validate ncid for classic files

This PR checks ncid for classic NetCDF files, which should be a multiple of (1<<ID_SHIFT) i.e. 65536.

note to self: fixes #791 manually close if need be when merging.


Provide an easy way to trace which envoy version is being used in the proxy_debug and release docker images

@mdhume commented on Tue Jan 30 2018

Is this a BUG or FEATURE REQUEST?: Feature Request
Did you review istio.io/help and existing issues to identify if this is already solved or being worked on?: Yes
Feature Request:
Y
Describe the feature:
Currently to figure out which envoy version is being used, one would need to trace back by using the docker SHA specified in the pilot repo to the git SHA in the proxy repo, to get the git SHA of the envoy repo.
Even in the docker container version command returns /usr/local/bin/envoy  version: 0//DEBUG
Would it be possible to expose this more explicitly as part of the docker or part of the github repo itself?

User can log in the istio-proxy container and use the envoy admin port to get the envoy version they used, eg: curl localhost:15000/server_info
envoy 0/1.7.0-dev//RELEASE live 58024 58024 0
So seems we only need to add a doc about this?

@elevran @rshriram  what is your comments about above way to get envoy version? If you agree that method, I will submit a PR to add info in our doc.

Envoy version will not work because of the way we compile Istio proxy. But if you figure out how to add the version info while compiling proxy, it would be very helpful. It requires some bazel stuff. Best place to start is istio/proxy code - look at the bazel stuff used to compile. And compare with how envoy adds the version info using Bazel.

@rshriram sorry but I have not got your point about " Envoy version will not work because of the way we compile Istio proxy."
I logged in my istio-proxy container and can get the version info: curl localhost:15000/server_info
envoy 0/1.7.0-dev//RELEASE live 57964 57964 0
The istio-proxy container image I used is:
istio-proxy:
Container ID:  docker://03d73e719a5231254d845daa3095b03112da8bf94c89dd4a6ad3813ac164a802
Image:         docker.io/istio/proxyv2:0.8.0
Image ID:      docker-pullable://istio/proxyv2@sha256:1930f0603321b1917b2249c576ecb4141aaceeaae5fcc0760b6a88dc88daea3e

envoy and istio-api versions are added as labels to images (i.e., available via docker inspect -f '{{ .Config.Labels }}' $image)


Validate recorded object fields

Resolves #140
This is an alternative to PR #187 . Only one of these PRs should be used.

Adds a static validator class that is used to guard against trying to access (via the provided get* methods) RecordedObject fields if they don't exist as doing so will will throw an IllegalArgumentException.
The main purpose of wrapping the RecordedObject.hasField method in the validator class is to generate some potentially useful diagnostic logging such as the following
Cannot validate field 'committedSize' due to null RecordedObject in 'GCHeapSummaryMapper'

Field 'committedSize' does not exist on RecordedObject in 'GCHeapSummaryMapper'

RecordedObject is null in 'GCHeapSummaryMapper'

Admittedly the static utility adds a bit of ugliness when it comes to testing as it requires static mocking. If this is not desirable then PR #187 provides a simpler approach but without the diagnostic logging. Another approach might be to refactor the static utility to be instantiable and use dependency injection to add it to the classes where it is used.
The main question is, is the extra complexity worth it for such logging?

Curious, was there consideration to not create the event/metric/summary at all if a field was missing? Just log.

This crossed my mind but there are too many different types of events and metrics being produced and I don't know enough about each of them to say whether or not we should avoid creating them entirely if certain fields are missing. I think that's a question worth thinking about and potentially exploring in a separate issue so the scope of this one doesn't creep.


create a CI/CD for triggering  unit tests for commit changes

create a GitHub workflow
trigger unit tests for commit change

Fixed with #3


Default argument values

Fixes https://github.com/davidpdrsn/juniper-from-schema/issues/5
This makes most default argument values work. The supported types are floats, integers, strings, booleans, enumerations, and lists (of supported types).
null is not supported because it doesn't make much sense as a default value. You might as well use a nullable type.
graphql_parser::schema::Value has a variant called Variable. I'm not sure what that is so it isn't supported yet. Maybe @tailhook or @theduke knows 😊
Input objects are also not supported yet because they're parsed as JSON like types and converting those into Rust types is a bit complicated. I plan to support them eventually.

Todos

[x] Update crate docs
[x] Add example

graphql_parser::schema::Value has a variant called Variable. I'm not sure what that is

It's only there because schema::Value and query::Value is the same type.

Great thanks!


Bug: CURLRequest unable to set body in options.

PHP Version
8.1
CodeIgniter4 Version
4.2.10
CodeIgniter4 Installation Method
Composer (using codeigniter4/appstarter)
Which operating systems have you tested for this bug?
Linux
Which server did you use?
fpm-fcgi
Database
MariaDB 10.6.11
What happened?
From the docs: https://codeigniter4.github.io/CodeIgniter4/libraries/curlrequest.html#body
This works: $client->setBody($body)->request('put', 'http://example.com');
This doesn't: $client->request('put', 'http://example.com', ['body' => $body]);
The latter doesn't seem to set the body and the request sends empty content.
Steps to Reproduce
Create a simple CURL request and try to send data in the body using the options parameter, when debugging enabled, you can see the Content-Length is set to 0.
Expected Output
Expect the request to send the data in the body.
Anything else?
No response

Hello, with the following change
$client->request('PUT', 'http://example.com', [
   'body' => $body,
   'http_errors' => false,
   'debug' => true,
]);

What are the contents of file log?

This bug has been fixed in develop.
Please try if you can.


Consistent Timestamp, Cross PM, typo

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[ ] It's really useful if your PR references an issue where it is discussed ahead of time.
[ ] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[ ] Run the tests with npm run test and lint the project with npm run lint and npm run prettier

Hey, thanks for writing this PR!
That typo was really sneaky, nice catch. On the other hand, I'm going to need a little more information with two things:

I have an idea on what does the lock files should do, and I don't quite understand why you are ignoring it. Further more, I believe CI:CD will fail due to this (npm ci requires the package-lock.json file to work as expected IIRC)
As far as I know, there's no such message_status property in the ServerSentMessageResponse payload. I'm testing with a v19 bot and I can't find anything similar in the logs. If you have any url to the documentation about this property, I would love to read more about it.

In other matters, I don't like the timestamp and status properties in the OnSentArgs, the first one because it might not be consistent with the API due to server timezones, and the second one because the message_status issue. I do, however, like the idea of timestamp for status and messages. Not including it for the former was an oversight from my part, and the later can access it via message.timestamp, but there's no drama with having the information duplicated for consistency within the library.
Let me know what you think about this 3 points, and we can keep working on the merge later.
Many thanks!

Not every one use NPM, based on my experience, companies strict it to chosen PM they prefer. and for me personally i use only bun for PM just because there are to much cache saved if I use multiple PMs, (my laptop that only has 256 SSD forces me). If we use npm ci here, so yes maybe if will fail but it back to you then, modify it or just use the lockfile.

and yes I test it too there is message_status in the response

{
  "message": {
    "name": "tagih_pelanggan_baru",
    "language": {
      "code": "id",
      "policy": "deterministic"
    },
    "components": [
      {
        "type": "header",
        "parameters": [
          {
            "type": "document",
            "document": {
              "id": null,
              "link": "https://adidata.tirtapatriot.net/invoice/undefined.pdf",
              "caption": null,
              "filename": null
            }
          }
        ]
      },
      {
        "type": "body",
        "parameters": [
          {
            "type": "text",
            "text": "DRS.ERFIZAL",
            "currency": null,
            "date_time": null
          },
          {
            "type": "text",
            "text": "kepelangganan",
            "currency": null,
            "date_time": null
          },
          {
            "type": "text",
            "text": "202504300092",
            "currency": null,
            "date_time": null
          },
          {
            "type": "text",
            "text": "DRS.ERFIZAL",
            "currency": null,
            "date_time": null
          },
          {
            "type": "text",
            "text": "JL.MELINJO NO.22",
            "currency": null,
            "date_time": null
          },
          {
            "type": "text",
            "text": "air",
            "currency": null,
            "date_time": null
          },
          {
            "type": "text",
            "text": "Rp56.000",
            "currency": null,
            "date_time": null
          },
          {
            "type": "text",
            "text": "sebelum tanggal jatuh tempo.",
            "currency": null,
            "date_time": null
          },
          {
            "type": "text",
            "text": "No. Pelanggan *202504300092*",
            "currency": null,
            "date_time": null
          },
          {
            "type": "text",
            "text": "atas perhatiannya.",
            "currency": null,
            "date_time": null
          }
        ]
      }
    ]
  },
  "phoneID": "234116276452082",
  "to": "6281211109845",
  "type": "template",
  "request": {
    "messaging_product": "whatsapp",
    "type": "template",
    "to": "6281211109845",
    "template": "{\"name\":\"tagih_pelanggan_baru\",\"language\":{\"code\":\"id\",\"policy\":\"deterministic\"},\"components\":[{\"type\":\"header\",\"parameters\":[{\"type\":\"document\",\"document\":{\"link\":\"https://adidata.tirtapatriot.net/invoice/undefined.pdf\"}}]},{\"type\":\"body\",\"parameters\":[{\"type\":\"text\",\"text\":\"DRS.ERFIZAL\"},{\"type\":\"text\",\"text\":\"kepelangganan\"},{\"type\":\"text\",\"text\":\"202504300092\"},{\"type\":\"text\",\"text\":\"DRS.ERFIZAL\"},{\"type\":\"text\",\"text\":\"JL.MELINJO NO.22\"},{\"type\":\"text\",\"text\":\"air\"},{\"type\":\"text\",\"text\":\"Rp56.000\"},{\"type\":\"text\",\"text\":\"sebelum tanggal jatuh tempo.\"},{\"type\":\"text\",\"text\":\"No. Pelanggan *202504300092*\"},{\"type\":\"text\",\"text\":\"atas perhatiannya.\"}]}]}"
  },
  "id": "wamid.HBgNNjI4MTIxMTEwOTg0NRUCABEYEkQ5NDgxMDQ4QUNDRjE2RTIwMAA=",
  "response": {
    "messaging_product": "whatsapp",
    "contacts": [
      {
        "input": "6281211109845",
        "wa_id": "6281211109845"
      }
    ],
    "messages": [
      {
        "id": "wamid.HBgNNjI4MTIxMTEwOTg0NRUCABEYEkQ5NDgxMDQ4QUNDRjE2RTIwMAA=",
        "message_status": "accepted"
      }
    ]
  }
}

I still don't get why the lock file would affect the end user. The package isn't released with it to npm, it's only used in development. Does the file prevents you from using bun? If possible, I would rather not change the repo's actions.

In respect to the message_status, I only found this documentation about the property:

Applies to businesses in Brazil, Colombia, and Singapore, starting September 12, 2023. Applies to all businesses starting October 12, 2023.

In theory, it should be available to everyone, but the documentation doesn't clarify if the property is always there or if it's specific for templates (which by the possible values seems to be the case).
I think the argument could be reworked into held_for_quality_assessment being a boolean, true to indicate if the property is equal to "held_for_quality_assessment", false if it's "accepted" or undefined. I believe it would create more confussion if the property is called status, as the end user might assume it can be used to check if the message was sent sucessfully, which clearly isn't its usage.

Last, if you don't mind, I would love to split this PR into 3 different ones. You can cherry pick each commit into a different branch, or if you don't know how to, I can do it for you. This is just to keep the conversations splitted, and start merging the changes that are ready, such as the typo fix.
Let me know if you agree!

I  can do it for you.
great, please do it

Closing in favor of #324 #325 #326


Updating branding of the blog

Replacing Gilt Tech language and logos with HBC Digital.
Also streamlining navigation in header.

😢

Can we use the cleaner U shaped HBC Digital logo? I think it looks better.

echoing @RoryH's sentiment. that banner is so busy it makes my eyes hurt. not really a fan of the u-shaped one either, but it's at least clean.


zipkin-WEBSHOPPINGAPIGW

zipkin-WEBSHOPPINGAPIGW

 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: wjkhappy14 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


preflight failed mag sensors inconsistent

Dear:
here are the log files.
https://logs.px4.io/plot_app?log=b11cdad4-8c8a-47c9-a83f-72c11df0df22
https://logs.px4.io/plot_app?log=7d3eac38-1274-451f-9d89-d1f44462aa91
https://logs.px4.io/plot_app?log=50c58947-0613-4c6f-b599-f814727564ca
i am not sure all of these three are the right files. but at least one of them is.
i cannot arm pixhawk at all, it keep telling me  mag sensros inconsistent. i have placed ublox m8n with the same direction as pixhawk, and calibrate them 2 times, and i still got those msgs. what can i do?

What's your physical setup? Where are the compasses relative to sources of high current?
Are you doing the full 6 position compass calibration? Are you doing it away from buildings, cars, etc?
If you change logging to start at boot (SDLOG_MODE 2), log sensor comparison (SDLOG_PROFILE see screenshot), and go through the full calibration again we can look at the log to find the issue.

this morning, i calibrated it again with all the six positions, and all te necessary pre steps, now it works fine, if something wrong, i will reopen it again

@dagar
Sorry, but it seems everything works fine just after the moment i calibrated the compass, but later after that, it comes mag inconsistent again.
here is the calibrate logging data
https://logs.px4.io/plot_app?log=0d5cd039-59e1-4e8b-a2ea-d4e85b733588
here is the later ones
https://logs.px4.io/plot_app?log=966addcc-a14b-457a-a29d-06943625438a

i have checked  https://github.com/PX4/Firmware/issues/7327 just now, then i list my dev here
nsh> ls /dev
/dev:
accel0
accel1
adc0
baro0
console
gyro0
gyro1
hmc5883_ext
l3gd20
led0
lsm303d_accel
lsm303d_mag
mag0
mag1
mmcsd0
mpu6000_accel
mpu6000_gyro
ms5611_spi_int
mtdblock0
mtdblock1
null
pipe0
pipe1
pwm_output0
pwm_output1
px4fmu
px4io
ram0
rgbled0
tone_alarm0
ttyACM0
ttyS0
ttyS1
ttyS2
ttyS3
ttyS4
ttyS5
ttyS6
nsh>
i think there must be something wrong with my mag devices. so i type lsm303d start then arm pixhawk with rc, while the mag inconsisitent error disappears, so will it be such a software error? or a bug? if you need anything that i can help, just @me, i am willing to help

The mag inconsistency check is new. You can silence it by increasing COM_ARM_MAG, however the problem is still there.
With the additionally logging enabled (all mags) can you try this.

point vehicle north and hold for a few seconds
spin slowly 360 degrees
hold vehicle nose down
spin slowly 360 degrees

Then we can look at the look and see just how different your mags are. I would guess there's a problem with how you've mounted everything. If one of the mags is definitely not usable you can disable it via param (CAL_MAGX_EN 0).

I am facing the same problem with the MindPX flight controller. So far, I have been doing the test indoor. I am using a customized 3D printed quadcopter where the flight controller, companion computer, and all the wiring are inside a hub and close to each other. I flew it indoor and outdoor using v1.6.5 and it flew nicely. After upgrading to v1.7, I started to get the MAG SENSORS INCONSISTENT many times even after calibrating, many times. MindPX has 2 internal mags and I used an external 3DR GPS with a mag. The only up to now that I was able to git rid of the error is to disable the internal mags and leave the external one. That way it worked fine indoors. I have tried it outdoors yet, but I guess it should be fine.
My conclusion is that, most probably that the internal magnetometers are highly affected by the surrounding wiring. However, it was working fine using 1.6.5 which is weird to me!

@mzahana your mags were likely always inconsistent, you just weren't getting the error.
If you're interested you should try the logging I described above and we can see how much they actually differ. Then you can decide if the internal should be disabled (CAL_MAGX_EN) or if the the inconsistency check should be relaxed.

I am also gettin the exact same error. 1.6.5 I had no problems,1.7.0 now this problem comes up. Also I believe that my problem wasnt there as @dagar says, because now my compass readings have a consistent 45 degrees error. I will follow the logging steps that you have mentioned and will be sharing a log here

@dagar Sorry for being away for such a long time,  right now it seems i can't make it right by just calibrating it, i have been doing this for quite a few times, maybe it is just because i do it near my PC, luckily, by increasing the COM_ARM_MAG value, i can make it, maybe not the best way to solve this though. thank you any way, if i got something new, i will post it here again.

I have the same problem on a Pixracer R14 with Px4 R1.7.3 stable. I have tried to recalibrate compass at different location without success. Shutting of the external compass eliminates the error "Mag sensors inconsistent".
Also raising the COM_ARM_MAG value to 0.20 from 0.15 seems to eliminate the problem but then "EKF high imu accel bias" pops up instead. Rising the COM_ARM_EKF_AB from 0.0024 to 0.0030 eliminates this (0.0028 also works but not consistently).
Whether these changes are within a reasonable tolerance level or not is beyond my knowledge so any comments will be appreciated. Might also be that the controller is of low quality.
Both the preflight errors mentioned above does not stop the quadro from arming.

One thing you may want to check... on my Flamewheel clone I didn't have the arrows on the GPS unit and the PX4 lined up. Once I lined them up, things went much smoother.


Add updated quanteda cheatsheet

This PR contains the updated quanteda cheatsheet (the pdf and pptx files), which is based on quanteda v.2.0.1.

Thanks @stefan-mueller!


fix: show vote reason

Summary

This PR fetch the vote reason from the API, and show it on the proposals' vote page, following the figma here https://www.figma.com/design/hmHTau1Y1KYzrBBKLQr5QV/%23design?node-id=1243-10592&t=zNyM96TSGyLdmPjt-0
Changes

Votes page column order change, to have the choice column just after the voter
Choice column is the widest one now
Reason is shown below the choice, if available
Basic vote choice UI has changed for smaller icon+text
Clicking on the choice will open the vote reason modal when reason exists
Various UX/UI fixes on the votes pages and votes modal

How to test

Go to a basic choice proposal: http://localhost:8080/#/s:apecoin.eth/proposal/0xf0ccdfb38ea5bee47e5f3f5ac83d5d9ecbeb605e328001c0ab4c6df7e91557e3/votes
It should show the basic choice with the new UI, as well as reason
Clicking on a choice with reason will open the reason modal
Go to a non-basic choice proposal: http://localhost:8080/#/s:apecoin.eth/proposal/0xd7a0d880fa558f3e7308113545a668d29dd4ed3fc10453ea7f1c2209d7b13f86/votes
It should show the reason below the choice when available, with a modal open on click
Go to an starknet proposal: http://localhost:8080/#/sn-sep:0x018df40f9cb5ac1b43a3c9291ebc02903f01e8241bd959520b36e7e293ace65f/proposal/3/votes
It should show the reason

To-Do before/after merge

[ ] Re-deploy mana to allow indexing of the new reason field
[ ] Update checkpoint to support TEXT reason, instead of STRING, to support 1000 characters reason

Maybe we could merge this one first, before the one writing reason, so we could test the writing PR more easily ?

https://github.com/snapshot-labs/sx-monorepo/pull/519 was merged, can you rebase so all networks work?

@wa0x6e I see on Figma there is more complex UI for Reason modal, are you planning on updating it in further PR?

@wa0x6e I see on Figma there is more complex UI for Reason modal, are you planning on updating it in further PR?

Yes, I will leave it to later PR. Each time I look at the figma, something's changing. Better to wait for the final UI for this modal content

Got it, we will merge this PR but I removed Closes: tag so we know that this design is still missing.


[router] Request to Update the Router Address and Assets (Token Smart Contracts) (Omni Team)

Hi, Connext Team, we notice you update the Smart Contract of Transaction Manager in all blockchain, and this requires

Add Router Address in the New Smart Contract of Transaction Manager
Add Assets (Token Smart Contract), in Binance Smart Chain and Polygon (at least, waiting for the deployment in Ethereum)

Information Require
@Rahul, like a mention yesterday right now i send all Real Token in Production and Mainnet in BSC, Polygon and ETH (Of course in ETH when you deployed not for now), and the Production Router Address (The Old address (0x784CC0dce30BAae1851a7f14b0d19DF3f3a3f30E) in Mainnet must be dropped
Additional share with you again the Security Audit for Certik for you review:
https://www.certik.org/projects/omni
BSC Token (First Deployed and unique with Vesting and Timelock Process)
https://bscscan.com/address/0xabc6790673a60b8a7f588450f59d2d256b1aef7f (PROXY ADDRESS)
https://bscscan.com/address/0x1538eec01b7bffbfe268dd4c1242dc1cb7a4803d (Actual Instance)
Code Verified: https://bscscan.com/address/0x1538eec01b7bffbfe268dd4c1242dc1cb7a4803d#code
Polygon Deployment
https://polygonscan.com/address/0xabc6790673a60b8a7f588450f59d2d256b1aef7f (PROXY ADDRESS)
https://polygonscan.com/address/0x022aaac040159f142a32cf59ef7241e783c5df71 (Actual Instance)
Code Verified: https://polygonscan.com/address/0x022aaac040159f142a32cf59ef7241e783c5df71#code
Ethereum Deployment
https://etherscan.io/address/0xabc6790673a60b8a7f588450f59d2d256b1aef7f (PROXY ADDRESS)
https://etherscan.io/address/0x022aaac040159f142a32cf59ef7241e783c5df71 (Actual Instance)
Code Verified:  https://etherscan.io/address/0x022aaac040159f142a32cf59ef7241e783c5df71#code
Additional the New Addresses of the
Router Mainnet and Production environment:
0x2a3248456464a19fA913bd5dfa952999365b31cE

Added on all networks.


fixing val reference to url in https hotlink and allowes https hotlinking on TinyMCE

Removes https:// overwrite on images URLs, so http: or https images can be linked.

Thanks, @ishannz! Can you update the ticket description and title to explain the bug you're fixing in a little more detail, without reference to dev/pharmac?

there have two commits in this merge request, the originals commit belong to @d-rivera-c
@d-rivera-c  can you please give some more information that explains the bug your are fixing.
https://github.com/silverstripe/silverstripe-framework/pull/6744/commits/22b3a71ec0c8cd8c38030fa0bf5449abefafe8a3 is
fixing val reference to url in https hotlink
https://github.com/silverstripe/silverstripe-framework/pull/6744/commits/5612b498b9d25f7bc348cf83ce927c5834552deb
Removes http:// overwrite on file URLs, so http: or https files can be linked.

Hi @ishannz - I think what @sminnee is asking is for a bit more detail on the issue you're having and how this patch resolves it. If you could explain in the form of "when I try to do [action] in the CMS I can't because [reason], this patch addresses it by [solution]"

I think the bug is that even if you use a https:// url, tinymce calls showFileView with the http:// protocol url.


Download starts, completes but will not save to computer

Hi
Using the downloader to download a video, the downloader detects video, downloads to 100% over the correct length of time(video is around 5gb's and takes around 30mins to get to 100%), however when i click on the download button nothing happens.

I'm having the same issue, but for a much smaller video (20mins). Download finishes, but nothing happens other than that the download button becomes disabled and I can't download other resolutions.
Afterward, it also breaks, and won't detect any other streams until I restart the browser.
Browser: Torch Browser
URL: https://www.ted.com/talks/bill_gates_innovating_to_zero

Work in progress #132


Area3D signals doesn't work on the exported binary but they do on the editor.

Hi, my game has a instanced Scene composed of a Camera and some other nodes, and it uses a child Area 3D node to check if the player is outside bounds, so it can start following it.
The game works properly on the editor, but when I export and test the exported binary, the camera for some reason doesn't follow the player.
I even made print statements on the area signal, but they also don't get registered.
I asked for help on the discord, but they weren't able to guess what is wrong, since the project works just fine under the editor, except when being exported.
You can see what the bug does here in action:
https://www.youtube.com/watch?v=fs_vsTMD230
The project files are here:
https://github.com/ca3games/Godot/tree/master/3.1 beta/Aurelio's Game
The proper scene node that has the issue is here:
https://github.com/ca3games/Godot/tree/master/3.1 beta/Aurelio's Game/Scenes/Level
The scene is called Basic Camera Setup.
You can check the camera node script, and see the signals of the Area 3D child.

I do not know why is different on export, but initialize your camera with out = true and should work.
The behavior is strange anyway...

I do not know why is different on export, but initialize your camera with out = true and should work.
The behavior is strange anyway...

Thanks, It works.
I was really confused because the project worked fine on the editor.

I think you should leave this open until the strange behavior gets fixed, something seems to be wrong on editor or export template.

I think you should leave this open until the strange behavior gets fixed, something seems to be wrong on editor or export template.

ok, I'll reopen it.

Make sure your editor and export template versions match exactly (they should be built from the same commit).

I was able to reproduce it too, with the correct templates, is just an area that toggles a variable with enter and exit, the body starts inside and with the variable on false (changes to true on exit).
It seems to not detect the exit of the body when starts inside on export (tested with debug) but could be something else...

Well, I have managed to reproduce it on a minimal project, the problem is the way area enter is triggered in editor and export templates, and only happens with CollisionPolygon.
areaerrorenterexit.zip
Run this on editor and from the command line, exported with debug.

I have a similar issue and creating a CollisionShape node, setting the shape to ConvexPolygonShape and typing the vertices by hand works fine on export. So I think the issue must lie somewhere in collision_polygon.cpp. Maybe where the convex shapes are generated, below //here comes the sun, lalalala :smile:


Grammatical error in first note section

Hello,
In the first Note section of this document there is a missing word. In it's current state it reads as:
"This is a very simple example to show how to get connected with a minimum of code. The Enhanced quickstart will build upon this sample to apply better design patterns."
In the first sentence I believe the author intended to put the word "amount" between "minimum" and "of." This would change the sentence to "This is a very simple example to show how to get connected with a minimum amount of code."

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

ID: 4db187af-a897-e8df-f587-4b9763e0bf27
Version Independent ID: b7446e9a-88eb-f6de-5d11-3f2e0489249a
Content: Quickstart: Blazor Server Web API sample (C#) (Microsoft Dataverse) - Power Apps
Content Source: powerapps-docs/developer/data-platform/webapi/quick-start-blazor-server-app.md
Service: powerapps
Sub-service: dataverse-developer
GitHub Login: @JeremyLikness
Microsoft Alias: jeliknes

Thanks for your comment, @Jucci16. This issue has been assigned to the doc author for review.

@Jucci16 Thanks for the feedback, but I feel the current wording is understandable by most people.


Flaky test DockerSuite.TestGetContainersAttachWebsocket

Seen on https://jenkins.dockerproject.org/job/Docker Master/label=fedora && devicemapper/4653/console
07:30:12 FAIL: docker_api_attach_test.go:16: DockerSuite.TestGetContainersAttachWebsocket
07:30:12
07:30:12 docker_api_attach_test.go:65:
07:30:12     c.Assert(actual, checker.DeepEquals, expected, check.Commentf("Websocket didn't return the expected data"))
07:30:12 ... obtained []uint8 = []byte{0x68, 0x0, 0x0, 0x0, 0x0}
07:30:12 ... expected []uint8 = []byte{0x68, 0x65, 0x6c, 0x6c, 0x6f}
07:30:12 ... Websocket didn't return the expected data

It's a low priority as this is the first failure I've ever seen for that test.

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO


Problems training - Implementing VAE Code - 11

Hello Valerio! Hope all is well.
I have a problem regarding video 11 where you change the encoder after making a vanilla distinction. In this repository I can't run the train.py code (the main one, where we instantiate the VAE works, and prints the summary) giving me the following error:

Traceback (most recent call last):
File "/home/luis/Desktop/vae-tensorflow/train.py", line 38, in
autoencoder = train(x_train[:10000], LEARNING_RATE, BATCH_SIZE, EPOCHS)
File "/home/luis/Desktop/vae-tensorflow/train.py", line 31, in train
autoencoder.compile(learning_rate)
File "/home/luis/Desktop/vae-tensorflow/autoencoder.py", line 53, in compile
self.model.compile(optimizer=optimizer,
File "/home/luis/anaconda3/envs/tensorflow/lib/python3.9/site-packages/tensorflow/python/training/tracking/base.py", line 522, in _method_wrapper
result = method(self, *args, **kwargs)
File "/home/luis/anaconda3/envs/tensorflow/lib/python3.9/site-packages/tensorflow/python/keras/engine/training_v1.py", line 444, in compile
self._cache_output_metric_attributes(metrics, weighted_metrics)
File "/home/luis/anaconda3/envs/tensorflow/lib/python3.9/site-packages/tensorflow/python/keras/engine/training_v1.py", line 1800, in _cache_output_metric_attributes
self._per_output_metrics = training_utils_v1.collect_per_output_metric_info(
File "/home/luis/anaconda3/envs/tensorflow/lib/python3.9/site-packages/tensorflow/python/keras/engine/training_utils_v1.py", line 910, in collect_per_output_metric_info
metric_fn._from_serialized = from_serialized  # pylint: disable=protected-access
AttributeError: 'method' object has no attribute '_from_serialized'

Running TF 2.5, CUDA 11.2 on ubuntu with NVIDIA 460.80
Thank you

Hi @luisArandas you need to run the version of tensorflow from the requirements.txt, which is 2.3.1

Hi, very helpful code. I have the same issue, though. Is there a fix other than downgrading tensorflow to run this?

Hi, very helpful code. I have the same issue, though. Is there a fix other than downgrading tensorflow to run this?

I highly doubt so. It's best practice when working with python to use sandboxed environments such as virtualenv, conda or Docker exactly because of this issue with dependencies. If the requirements file explicitly expresses this version of tensorflow as the one to use it's because other versions are likely to break compatibility. I would recommend you create a virtual environment to run this project in and install there the appropriate version of tensorflow

I have installed the specified tensorflow version 2.3.1 using conda venv. But the same error appears when training VAE. Any help Please?!

I am using tensorfow 2.6.0,
I removed the metric parameter in the compile function and the error is resolved.
Below is the snippet of my compile function:
def compile(self, learning_rate = 0.0001):
optimizer = Adam(learning_rate=learning_rate)
self.model.compile(optimizer=optimizer, loss= self._calculate_combined_loss)


1st try to add a few refs to verif&synt, resp. static analysis

Thanks for the additions, cherry-picked as 53ee42b78bf7e313912dff413673f552ff9e6078, 25fb9f0f271ceb3d345f32f3139c5048628ab088.


Run npm install at end of generation process

Are you against that ?

Not a big fan, but I guess that would make it easier for people, so sure.
http://yeoman.github.io/generator/install.html#installDependencies


add(headings and focus)to the sidebar and model #484

add focus (tabIndex) and offscreen headings so JAWS can read when a user is looking for all headings within the app.

@blaino @arw180 @wski when you have time 2 look at it

I pulled and clicked around. Doesn't seem to break anything. :+1:

Use an off screen technique to remove content from visible display, but still available to screen reader users:

looks good, In the future it would be cool to experiment with display:none; for the offscreen elements if the screen reader can read hidden elements. :+1:

I would love to use [display: none] but after doing some research this works for Center and JAWS
https://www.paciellogroup.com/blog/2012/05/html5-accessibility-chops-hidden-and-aria-hidden/


Fix e2e

Fix the end to end test for filtering in list
Use the correct extract-text-webpack-plugin for webpack1
Fixed some malformed urls in the tests

Thanks!


Bug when i execute setBlock

Hello Martin,
I use MCPI with children in a computer workshop.
When several people are connected in my server, the function setBlock make appear the bloc on an other player.
Is there a solution for this bug?
Best regards,
Ced

The player object will always return the "host" player, usually the first player to join.
When there are multiple players on the server you will need to use the entity functions to get positions of other players.
There is some information on the entity object in this API reference.
e.g.
entityIds = mc.getPlayerEntityIds()
player1id = entityIds[0]
player2id = entityIds[1]

print("Player 1 pos", mc.entity.getPos(player1id))
print("Player 2 pos", mc.entity.getPos(player2id))

Closing as not a bug.

I have another question for you,
My children ask if it is possible to send a commmand like   « /weather rain ».
Do yo have a suggestion ?
Best wishes
Ced

Thank You Martin,
You’re helpfull !
Ced
Envoyé à partir de Courrierhttps://go.microsoft.com/fwlink/?LinkId=550986 pour Windows
De : @.>
Envoyé le :jeudi 20 janvier 2022 18:40
À : @.>
Cc : @.>; @.>
Objet :Re: [martinohanlon/mcpi] Bug when i execute setBlock (Issue #25)
The player object will always return the "host" player, usually the first player to join.
When there are multiple players on the server you will need to use the entity functions to get positions of other players.
There is some information on the entity object in this API referencehttps://www.stuffaboutcode.com/p/minecraft-api-reference.html.
e.g.
entityIds = mc.getPlayerEntityIds()
player1id = entityIds[0]
player2id = entityIds[1]
print("Player 1 pos", mc.entity.getPos(player1id))
print("Player 2 pos", mc.entity.getPos(player2id))
Closing as not a bug.
—
Reply to this email directly, view it on GitHubhttps://github.com/martinohanlon/mcpi/issues/25#issuecomment-1017756426, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMYXZ3QGK2TU34X2AJLB4FTUXBCKNANCNFSM5MNGMDWA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID: @.***>

Unfortunately it is not possible to send command via RaspberryJuice.

Oh, Ce n’est pas grave.
Thank you 😊
Envoyé à partir de Courrierhttps://go.microsoft.com/fwlink/?LinkId=550986 pour Windows
De : @.>
Envoyé le :dimanche 30 janvier 2022 19:26
À : @.>
Cc : @.>; @.>
Objet :Re: [martinohanlon/mcpi] Bug when i execute setBlock (Issue #25)
Unfortunately it is not possible to send command via RaspberryJuice.
—
Reply to this email directly, view it on GitHubhttps://github.com/martinohanlon/mcpi/issues/25#issuecomment-1025201210, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMYXZ3SNV6X6FR5TF2AUIATUYV7F3ANCNFSM5MNGMDWA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID: @.***>


[fix] [test] Fix flaky test MetadataStoreStatsTest.testBatchMetadataStoreMetrics

Fixes #19792
Motivation
Fix flaky test MetadataStoreStatsTest.testBatchMetadataStoreMetrics
Modifications

set metadata store name to avoid generate metric with name cause relate unit test fail.

auto generate metadata store name if caller not specify one for avoid this problem occur again.

Verifying this change

[x] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

the auto generate metadata store name will be like org.apache.pulsar.metadata.MetadataStoreConfigTest_testGeneCallerStackNameWhenMetadataStoreNameNotSet_11

@lhotari please take a look.

Is the root cause that the metadatastoreName is empty?

Is the root cause that the metadatastoreName is empty?

@nodece The root cause is create one MetadataStore with no name set and not close it.

/pulsarbot rerun-failure-checks

/pulsarbot rerun-failure-checks

/pulsarbot rerun-failure-checks

This fix is invalid, see https://github.com/apache/pulsar/actions/runs/4424020709/jobs/7763604452#step:11:1277


Update editorial community summary for PeerJ

Definition of Done:

[x] new text content and logo is available on staging
[ ] Nathaniel Gore at PeerJ has approved it

Text should now read as per the updated page in https://docs.google.com/document/d/1lFcM_XhUZKYRbq2hqNdfijnRaWfsXNZ0DnaJHxdY-3I/edit#heading=h.i20faaopvi9m
Tasks

[x] Update the text as described above
[x] Add their logo (official logos here: https://peerj.com/about/press/)

Available on http://prc-staging.libero.pub/editorial-communities/53ed5364-a016-11ea-bb37-0242ac130002

Style refinement still needed.
Question: should we host the logos ourselves (including the eLife one) or can we continue hotlinking to other websites?

I just emailed Nathaniel for some feedback on the page.
I don't have a strong opinion on hot linking of images, my experience is that hosting them ourselves is better in the long run as we can utilise our own CDN caching strategies for it but we're a long way of micro-optimising web performance at the moment.

Only issue will be when introducing HTTPS, as the PeerJ one is only available over HTTP. So something we can resolve then.

Received positive response from Nathaniel:

Hi Paul,
That looks great, thanks. Let me know if you need anything re. the content upload. We look forward to seeing it!
Cheers,
Nathaniel


Centered search results on youtube

Youtube has updated the UI once again and now the search results are no longer centered. The old UI used to have a lot of padding on the left side so the search results are centered. Will it be possible to implement centered search results in the next update?
What it looks like now:

What it used to look like from what I can find on the internet:
https://storage.googleapis.com/support-forums-api/attachment/thread-51221404-7061916441851235437.png

I tried to center it, but it looks so weird.

I tried to center it, but it looks so weird.

I don't think that looks weird. At even higher resolutions, you can get something like this:
https://i.redd.it/zxc4dwgxd8z91.jpg

Closed since they reverted the UI change/bug now

The issue is back. Not sure if youtube has finalized the decision to align the search results to the left, but it would be great to have a toggle to align the search results to the center regardless. https://www.reddit.com/r/youtube/comments/yt167k/who_came_up_with_this_idea/

All right

While watching some random videos, I got duplicate videos on the watchpage. Perhaps it's the bug of itself.


Update iot-hub-python-python-file-upload.md

Update sample code to use the file content instead of the file path in upload_blob_async API

@zolvarga : Thanks for your contribution to the Azure documentation! The author, @msebolt, has been notified to review your proposed change.

@timlt

Sorry for the delay, my notifications were turned off.
#sign-off


Prune to most "important" stacks

Adds a new --prune NUM option to be used with collapsed format.  Leaves will be pruned from the stack graph in order of increasing _total until only NUM remain; when all of a node's children have been pruned, it is added back to the queue.  With --prune 5, the /tmp/java.trace` from the smoke test looks like:
Target.main;Target.method1 42
Target.main;Target.method2 46
Target.main;Target.method3;java/io/File.list;java/io/File.normalizedList;java/io/UnixFileSystem.list;Java_java_io_UnixFileSystem_list;JNU_CopyObjectArray;jni_DeleteLocalRef 17
Target.main;Target.method3;java/io/File.list;java/io/File.normalizedList;java/io/UnixFileSystem.list;Java_java_io_UnixFileSystem_list;JNU_CopyObjectArray;jni_GetObjectArrayElement 26
Target.main;Target.method3;java/io/File.list;java/io/File.normalizedList;java/io/UnixFileSystem.list;Java_java_io_UnixFileSystem_list;newSizedString8859_1;jni_NewString;java_lang_String::create_oop_from_unicode(unsigned short*, int, Thread*);java_lang_String::create_from_unicode(unsigned short*, int, Thread*);java_lang_String::basic_create(int, bool, Thread*) 25

Note, for example, that java_lang_String::basic_create would typically not show up in a list of most-sampled stacks, because the samples are fragmented over a dozen contributing child stacks.

Hi Peter, I appreciate your interest in improving async-profiler.
If you would like to suggest a new feature, you are welcome to submit an issue or start a discussion first. PRs like this have little chance to get accepted, unfortunately. A quick look reveals multiple problems, starting from the compilation issues to the insufficient motivation for the change and the justification of the chosen algorithm.
It's worth mentioning that collapsed output format is left only for compatibility reasons, its usage for new features is deprecated. I'd also prefer not to bring more stuff from libstdc++, since this conflicts with another initiative to get rid of libstdc++ in async-profiler completely (#729). Thank you for your understanding.

Hi -
I didn't really expect the PR to be merged as-is, but it's easier for me to express the idea if it has a concrete form.
The motivation is that I often want a way to summarize long samples over thousands of processes. I imagined squinting at a series of mini-flamegraphs comprising, say, 30 second intervals, into which one could drill for further details. The squinted-at flamegraph is conceptually coarser, which basically means the flame tips have been lopped off such that the bases that remain represent more significant fractions of the total samples. There are various algorithms for doing this, but they all come down to pruning the flame graph tree. The algorithm in the PR is one I'm already using in Java to process text dumps of collapsed stacks obtained via the API, but the cost of this is hugely dominated by parsing, so it seemed simpler to do the whole thing in c++.
In any case, I find the summaries to be useful, and they also seem to help motivate others to examine the samples in more detail, so we'll either continue to do the processing in java or maintain our own fork, which in any case we have to do already for awaiter stacks.

Thank you for the background, the problem you describe makes sense. Async-profiler already has --minwidth option for building coarse flame graphs, so I'd be interested in a comparsion of your proposed approach to the existing one.
In any case, this feature is more suitable for the out-of-process converter than for the core agent that works in the context of a Java application. I agree that collapsed format is too verbose and expensive for parsing. Did you consider dumping output in jfr format?

You’re right that doing this out of process would seem to make more sense. In our present case, we’re capturing the thumbnails (as lists of strings) simultaneously with snapshots of a bunch of app-specific measurements that are not easily obtained externally.
Le Mer, 29 mars, 2023, à 10:43, Andrei Pangin a écrit:

Thank you for the background, the problem you describe makes sense. Async-profiler already has --minwidth option for building coarse flame graphs, so I'd be interested in a comparsion of your proposed approach to the existing one.
In any case, this feature is more suitable for the out-of-process converter than for the core agent that works in the context of a Java application. I agree that collapsed format is too verbose and expensive for parsing. Did you consider dumping output in jfr format?
—
Reply to this email directly, view it on GitHub https://github.com/async-profiler/async-profiler/pull/735#issuecomment-1488762126, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJU73KKD5F45I3ILV4IOH3W6RDA5ANCNFSM6AAAAAAWINMZLI.
You are receiving this because you authored the thread.Message ID: @.***>

Considering the above discussion, I'm closing the PR without plans to merge.


[suggestion] convert from a symmetric tensor to a non-symmetric tensor

Can we have a funciton to create a non-symmetric tensor from a symmetric tensor by putting 0s?

This is related to #136


perf: pass SyntaxContext by copy

Summary
SyntaxContext is just a u32 and is cheaper to copy than pass by reference.
Test Plan

Require Documentation?

[x] No
[ ] Yes, the corresponding rspack-website PR is __

Thanks!


Support for file:// URLs

Since this extension matches on domain, I've been able to match file:// URLs (e.g. file:///tmp/somefile) to work by using the regex ^/. However, this blocks all file:// URLs, and I don't necessarily want to block all of them. I really just want to block ones in /tmp as they represent files that aren't likely to persist and thus don't need to remain in my browsing history.
It would be nice to have more flexible support for these URLs.

Actually, I take it back ^/ matches everything. So there is presently no way to match file:// URLs without blocking everything else as well.

Depending on what URLs you want to match, you can use the following regexes:
file:\/\/tmp.* to match file://tmp/whatever (two slashes)
file:\/\/\/tmp.* to match file:///tmp/whatever (three slashes)
You must also enable the "Check entire URL" option or else DynamicHistory will just attempt to match the regexes to the domain (and there is no domain on file URLs).
If you are on Chrome, you can alternatively add the following code into the "Custom history processor" box:
//Remove URLs that start with "file://tmp" or "file:///tmp"
if(item.url.startsWith("file://tmp")
    || item.url.endsWith("file:///tmp"))
    item.remove();

Thanks for the info. I'm not used to having to escape slashes on simple regex searches, so I didn't think to escape them.
However, this doesn't work the way you said it does. First of all, three escaped backslashes do not work. So, file:\/\/tmp.* works, while file:\/\/\/tmp.* does not.
Secondly, slashes can't be matched after the beginning of the path. So I can use file:\/\/home*, but I can't use file:\/\/home\/.*.

Alright, nice job noticing that! I've just checked and that's a bug with my regex processing algorithm. I'll try to fix it today if possible.

Alright, updated version 2.17 pushed out with a fix.

Excellent, thanks!

Fix confirmed :+1:


The deployment using docker-compose has failed

docker-compose.yml
version: "2"
services:
  ezbookkeeping:
    image: mayswind/ezbookkeeping
    container_name: ezbookkeeping
    hostname: "ezbookkeeping"
    ports:
      - "5701:8080"
    environment:
      - "EBK_SERVER_DOMAIN=mydomain"
      - "EBK_SERVER_ENABLE_GZIP=true"
      - "EBK_DATABASE_TYPE=mysql"
      - "EBK_DATABASE_HOST=localhost:3306"
      - "EBK_DATABASE_NAME=ezbookkeeping"
      - "EBK_DATABASE_USER=ezbookkeeping"
      - "EBK_DATABASE_PASSWD=password"
      - "EBK_LOG_MODE=file"
      - "EBK_SECURITY_SECRET_KEY=xxxxxxxxx"
    volumes:
      - "/etc/localtime:/etc/localtime:ro"
      - "/var/log/ezbookkeeping:/var/log/ezbookkeeping" # 请确保 UID:GID 是 1000:1000

The error log：
2022-10-13 14:37:32 [ERROR] [server.startWebServer] update database table structure failed, because dial tcp <IP_ADDRESS>:3306: connect: connection refused
2022/10/13 14:37:32 Failed to run ezBookkeeping with [/ezbookkeeping/ezbookkeeping server run]: dial tcp <IP_ADDRESS>:3306: connect: connection refused

please make sure that your ezbookeeping container and mysql container is in the same network

@mayswind  Yes, a terminal that executes docker-compose up on the same server can connect to mysql

By default, the network of each docker container is isolated, and if you access "localhost" in ezbookkeeping container, you can only access the port in the current container. You can enter into the ezbookkeeping container and check whether the 3306 port is available.

@mayswind  Thank you very much, the problem was solved, it is for this reason. Thank you


Sr. Front End Engineer job posting in Announcements

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

Hello @Marghodk,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


Storage Logging

Adds contextual logger to backends and drivers.
Updates backendLXD functions to log start and finish of each function with context of args supplied.

Includes https://github.com/lxc/lxd/pull/6373

Testsuite passed


How to add multiple processors to tracing sdk in Nodejs Opentelemetry NodeSDK?

Hello,
Is there a way i can add multiple processors while setting up Opentelemetry in Node.js? There seems no way to do so in NodeSDK in opentelemetry-js/sdk-node pack
const opentelemetry = require("@opentelemetry/sdk-node");
const { getNodeAutoInstrumentations } = require("@opentelemetry/auto-instrumentations-node");
const { JaegerExporter } = require("@opentelemetry/exporter-jaeger");
const { Resource } = require("@opentelemetry/resources");
const { SemanticResourceAttributes } = require('@opentelemetry/semantic-conventions');
const { SimpleSpanProcessor } = require("@opentelemetry/sdk-trace-base");
const { BatchSpanProcessor } = require("@opentelemetry/sdk-trace-base");

function start(appName) {
    var exporter = new JaegerExporter();
    const sdk = new opentelemetry.NodeSDK({
        resource: new Resource({
            [SemanticResourceAttributes.SERVICE_NAME]: appName,
        }),
        instrumentations: [getNodeAutoInstrumentations()],
        traceExporter: exporter,
        spanProcessor: new SimpleSpanProcessor(exporter),
    });
    sdk.pro
    sdk.start();
}

module.exports = { start }

Is there any other way by which i can add multiple processors via Code? ... I do not want to use OpenTelemetry Collector.

It can be done if you set up the SDK components manually by creating a tracer provider and calling addSpanProcessor on it multiple times, then registering it using api.tracing.setGlobalTracerProvider

@dyladan can you share an example please?


ErrorException in AdldapServiceProvider.php line 50: Undefined index: connections

Hey,
I am setting up a laravel 5.2 app with ldap using instructions from your readme. I am getting this error and I do not know how to solve it. Anyone knows what is wrong with my setup and how to solve it? The ldap credentials are fine.
 in AdldapServiceProvider.php line 50 at HandleExceptions->handleError('8', 'Undefined index: connections', '/opt/app/ldap-app/vendor/adldap2/adldap2-laravel/src/AdldapServiceProvider.php', '50', array('app' => object(Application), 'config' => array('auto_connect' => true, 'connection' => 'Adldap\Connections\Ldap', 'connection_settings' => array('account_suffix' =><EMAIL_ADDRESS>'domain_controllers' => array('aeolus.corp.local', 'zeus.corp.local', 'hades.corp.local'), 'port' => '389', 'base_dn' => 'DC=corp,DC=local', 'admin_account_suffix' => '', 'admin_username' => 'CN=bindusr,CN=Technical Accounts,DC=corp,DC=local', 'admin_password' => 'qwerty', 'follow_referrals' => false, 'use_ssl' => false, 'use_tls' => false, 'use_sso' => false)))) in AdldapServiceProvider.php line 50 at AdldapServiceProvider->Adldap\Laravel\{closure}(object(Application), array()) in Container.php  ...

Hi, are you upgrading from v1 to v2?
Try deleting your 'adldap.php' configuration file and republishing the configuration files using php artisan vendor: publish

Hi Stevebauman,
Thanks, your reply brought me something further. I was indeed upgrading. The error that I now have is:
BindException in Guard.php line 92: Invalid credentials
The credentials should be fine.. i did a double check .. is it something else I am missing?
 in Guard.php line 92 at Guard->bind('CN=bindusr,CN=Technical Accounts,DC=corp,DC=local', 'qwerty', ''<EMAIL_ADDRESS>in Guard.php line 108 at Guard->bindAsAdministrator() in Provider.php line 184 at Provider->connect() in AdldapServiceProvider.php line 90 at AdldapServiceProvider->Adldap\Laravel\{closure}(array('auto_connect' => true, 'connection' => 'Adldap\Connections\Ldap', 'schema' => 'Adldap\Schemas\ActiveDirectory', 'connection_settings' => array('account_prefix' => '', 'account_suffix' =><EMAIL_ADDRESS>'domain_controllers' => array('aeolus.corp.local', 'zeus.corp.local', 'hades.corp.local'), 'port' => '389', 'base_dn' => 'dc=corp,dc=local', 'admin_account_suffix' => '', 'admin_username' => 'CN=bindusr,CN=Technical Accounts,DC=corp,DC=local', 'admin_password' => 'qwerty', 'follow_referrals' => false, 'use_ssl' => false, 'use_tls' => false)), 'default') in Collection.php line 146 at Collection->each(object(Closure)) in AdldapServiceProvider.php line 94 ..

Ok great, let's take a look at your next issue.
In the exception, this line here:
Guard->bind('CN=bindusr,CN=Technical Accounts,DC=corp,DC=local', 'qwerty', ''<EMAIL_ADDRESS>
You've inserted a full distinguished name as the administrator username. You've already provided your suffix as @corp.local, so your username should be  bindusr.
Right now your essentially trying to bind to yout LDAP server with:
Username: CN=bindusr,CN=Technical<EMAIL_ADDRESS>Password: qwerty

I get your point and it worked!! The following code returns ldap dumps:
`$users = Adldap::getProvider('default')->search()->users()->get();
dd($users);`
Thanks for your help :+1: .

Awesome! Glad you've solved the issue. No problem :).


Expose remaining environment API

Many of the calls for get_raw, set_raw, and cmd_raw aren't modeled properly. Effort should be focused on getting them exposed in an idiomatic way so that more of the unsafe API surface is reduced. This doesn't need to be as restrictive as #2 for the moment, but laying the groundwork for it would be nice.

Should we bother implementing the experimental commands? Since they could change or be removed at any time, any safe Rust interface we implement would be unstable and could become unsafe.

Should we bother implementing the experimental commands? Since they could change or be removed at any time, any safe Rust interface we implement would be unstable and could become unsafe.

I'd be okay with putting these behind a feature flag, with the advisory that these are unstable and subject to change. Those that are wishing to take the plunge do so at their own risk, and we would officially not consider it public API from a semver perspective. Perhaps experimental?

Makes sense. I'll implement those last, then.

I've thought about this more, and the idea that this crate could be used to create potentially invalid cores I think means we should hold off implementing environment calls until they are stabilized. The *_raw functions still exist if someone wants to do something exotic, but I don't want to officially support them (By adding our own APIs) until they're stable.

After thinking about it some more, I don't think the experimental options are actually changed in practice. From looking at the environment commands in libretro.h, what seems to happen instead is that the constant for the command is removed, the frontends stop supporting it (i.e. they return false) and a new version is introduced. For instance, RETRO_ENVIRONMENT_GET_VARIABLE and SET_VARIABLES used to be keys 4 and 5, but those two keys have since been removed and updated versions were assigned to 15 and 16.
A properly written core ought to check whether the frontend supports any environment command (stable or otherwise) so there shouldn't be any trouble when an experimental command is deprecated/replaced.

Well, that's a nasty bit of API design on their end. I suppose if that's how business is done in this domain, having them behind an unstable feature flag should suffice.


Azure Quota Exceeded Exception: Error code: PublicIPCountLimitReached, error message: Cannot create more than 3 public IP addresses for this subscription in this region. (Azure request Id: 49400153-0424-4c09-8e19-5abb6f35c1c5)

This isn't the case. I have created a cluster for the very first time on my brand-new azure student subscription account and it's giving the you have reached the IP limit, on the very first cluster I create? what's the limit? zero?

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

ID: 242ffeb8-ef01-73a7-0c0a-323f01a27843
Version Independent ID: 7bb409b5-a5eb-25e1-11e3-63acd39771ea
Content: IP address limit prevents cluster creation - Azure Databricks
Content Source: databricks/kb/clusters/azure-ip-limit.md
Service: azure-databricks
GitHub Login: @adampavDB
Microsoft Alias: saperla

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

Hi @EmanElsefy
Thank you for your feedback!
Since this issue isn't directly related to improving our docs, and to gain a better understanding of your issue, I'd recommend working closer with our support team via an [Azure support request] (https://docs.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request). Or you can leverage our Q&A forum by posting your issue there so our community, and MVPs can further assist you in troubleshooting this issue or finding potential workarounds.
[Teams Q&A forum] (https://docs.microsoft.com/en-us/answers/topics/46488/office-teams-windows-itpro.html) for technical questions about the configuration and administration of Microsoft Teams on Windows.
[Microsoft Teams Community forum] (https://answers.microsoft.com/en-us/msteams/forum?sort=LastReplyDate&dir=Desc&tab=All&status=all&mod=&modAge=&advFil=&postedAfter=&postedBefore=&threadType=All&isFilterExpanded=false&page=1


Possibility to use RapidXml instead of Xerces-C++

Hello. Have you considered using RapidXml instead of Xerces-C++ (or perhaps not instead but as an option)? Why one might want to do so is at least due to the complexity of building Xerces-C++ and its dependencies (especially for mobile platforms).
We've managed to replace Xerces-C++ with RapidXml in our in-house build of E57Format library, and that seems to work fine. I could probably submit a patch here, if you'll be interested.
If you have considered RapidXml, but decided not to use it for particular reasons, could you please share your reasoning? In that case it would be useful for us to maybe reconsider and stay with the original dependency of Xerces-C++.
One possible reason I can name myself: RapidXml hasn't been updated for many years now, so one probably should be cautious about using seemingly unmaintained code.

Hi, I'm the one who did the RapidXML port, and I'd recommend not using it as it's not actively maintained (which is a death sentence for any text processing lib) and is generally quite crude. The reason I used Rapidxml is because we where already using it in other parts of the project, so it had minimal impact for us.
Having a dependency on Xerces in e57format is quite silly, so I'll do the port again using a newer, a bit more capable library that is actively maintained. I'm currently leaning towards pugixml.

Hello!
To answer the initial question, yes I had considered replacing Xerces. It's much too heavy for the tiny bit of XML involved here. Years ago I looked at replacing it and that was part of what lead me to start a new implementation from scratch (as I mentioned over here and here). I also wanted to modernize the codebase, use a different license, and generally change the architecture/API, so a new implementation made sense. Then I stopped doing much with this library because I didn't think anyone was really using it anyways.
Fast-forward to 3.0 - this came about because some people started contributing some code and it got me looking at it again. So I pulled in a whole bunch of things I'd written for the new one which is how we have some testing, ASan, proper formatting, etc..
I recall looking at several Xerces replacement options. I agree with @Brynyard's assessment of RapidXML. (I also freely admit to an anti-Sourceforge bias - terrible UX - so that would have worked against it 😆 ) I too landed on... pugixml! So that's what I use in my alternate implementation.
All of that to say that I'd be happy to accept a changeover to pugixml if you want to submit a PR. I'm sure a lot of people would be thankful to be rid of Xerces.

Great news all the way. That answers my questions, so I'll close this issue (unless you'd like to use it for tracking Xerces-C++-related things).


[New chain]: Japan Open Chain Testnet

Summary
I would like to add v1.4.1 on Japan Open Chain Testnet
Chain ID
10081
Chain detail URL
https://chainlist.org/chain/10081
RPC URL
https://rpc-1.testnet.japanopenchain.org:8545
Blockscout Client URL
No response
Etherscan Client URL
No response
Etherscan Client API URL
No response
Version
1.4.1
Address (Master copy)
0x41675C099F32341bf84BFc5382aF534df5C7461a
Deployment Tx hash (Master copy)
0x7797a28338adfebfe2e2bbdbd0c0dfc47dabeeef7fa3ab3a399ba2015c129905
Block explorer URL (Master copy)
https://explorer.testnet.japanopenchain.org/address/0x41675C099F32341bf84BFc5382aF534df5C7461a
Address (Master copy L2)
0x29fcB43b46531BcA003ddC8FCB67FFE91900C762
Deployment Tx hash (Master copy L2)
0x6cb0418ffef8e38ec6253e646027602960fbfc82992c984a658f954f1fd7458c
Block explorer URL (Master copy L2)
https://explorer.testnet.japanopenchain.org/address/0x29fcB43b46531BcA003ddC8FCB67FFE91900C762
Address (Proxy factory)
0x4e1DCf7AD4e460CfD30791CCC4F9c8a4f820ec67
Deployment Tx hash (Proxy factory)
0xb60992e58d826d4b9d54d4d47724889d8457f598499e53e90eba90e21deb50d3
Block explorer URL (Proxy factory)
https://explorer.testnet.japanopenchain.org/address/0x4e1DCf7AD4e460CfD30791CCC4F9c8a4f820ec67

/execute


Add other ongoing projects to the projects section.

Include projects like CodeManiacs, Search_Guru, Faculty Availability, Youtube Downloader etc. in the projects section of the website.

@kpbot claim

ping @Vishal1541


Performance and benchmark work

Improve benchmarks

Don't do more than one iteration within a benchmark, let BenchmarkDotNet handle that.
Support BenchmarkDotNet CLI options and picking benchmarks to run interactively.
Add benchmarks for key and hash generation.
Use consistent naming conventions for benchmarks.
Improve benchmark debugging experience.
For each benchmark, where applicable, add a 'floor' counterpart to measure what doing the same scenario with non-identifiable keys would cost. We can't do better than this and it establishes how much it costs to use Cask in exchange for the additional value it provides.

Improve .NET Framework Polyfills

Reduce allocations.
Add tests borrowed from BCL and a small threading stress test.
Exclude polyfill attributes that are not instantiated via reflection from code coverage.
Don't copy polyfill code into test/benchmark assemblies, use it via InternalsVisibleToAttribute. Improves code coverage experience.

Misc

Don't allocate a new string when creating a CaskKey from an existing string.
Rename CaskKey.(Try)Parse to CaskKey.(Try)Create. Parsing isn't really what we do.
Limit length of derivation input.
Add fast convenience API for providing derivation input as a string.
Remove open question about timestamp in hash, since this is settled. Update pseudo-code, and add test.
Add gitignore entry for Ultra profiling files.
Fix some exception messages.
Add a bit more test coverage.
Set a maximum stackalloc constant of 256 bytes and assert that it is never exceeded.
Move test-specific warning suppression to test-specific .editorconfig for better interaction with dotnet format.

Benchmark results on my machine.
.NET 8.0

Method
Mean
Error
StdDev
Gen0
Allocated

CompareHash_Cask
451.609 ns
3.9292 ns
3.6753 ns
-
-

CompareHash_Floor
294.875 ns
2.7652 ns
2.5866 ns
-
-

GenerateHash_Cask
300.978 ns
1.3669 ns
1.2786 ns
0.0091
176 B

GenerateHash_Floor
225.569 ns
1.6476 ns
1.2864 ns
0.0057
112 B

GenerateKey_Cask
143.634 ns
2.8027 ns
2.7527 ns
0.0093
176 B

GenerateKey_Floor
63.337 ns
1.2881 ns
1.6290 ns
0.0058
112 B

IsCaskString
23.718 ns
0.1039 ns
0.0921 ns
-
-

IsCaskBytes
7.501 ns
0.0291 ns
0.0272 ns
-
-

Notice how nothing but the bytes to hold the generated key or hash is allocated on the heap. :)
.NET Framework with naive polyfills (before optimizations in this PR)

Method
Mean
Error
StdDev
Gen0
Allocated

CompareHash_Cask
2,601.8 ns
15.03 ns
14.06 ns
0.1450
931 B

CompareHash_Floor
1,809.6 ns
16.60 ns
13.87 ns
0.0801
514 B

GenerateHash_Cask
2,627.9 ns
50.53 ns
69.17 ns
0.2022
1292 B

GenerateHash_Floor
1,752.4 ns
12.52 ns
11.71 ns
0.1163
738 B

GenerateKey_Cask
1,080.1 ns
5.02 ns
4.45 ns
0.1583
1003 B

GenerateKey_Floor
561.4 ns
2.69 ns
2.39 ns
0.1297
818 B

IsCaskString
181.4 ns
0.38 ns
0.33 ns
-
-

IsCaskBytes
128.9 ns
0.90 ns
0.85 ns
-
-

.NET Framework with optimized polyfills (with optimizations in this PR)

Method
Mean
Error
StdDev
Gen0
Allocated

CompareHash_Cask
2,542.0 ns
36.60 ns
34.23 ns
0.0534
353 B

CompareHash_Floor
1,802.4 ns
8.94 ns
7.93 ns
0.0458
289 B

GenerateHash_Cask
2,320.9 ns
10.17 ns
9.01 ns
0.1106
714 B

GenerateHash_Floor
1,741.5 ns
7.54 ns
7.06 ns
0.0801
514 B

GenerateKey_Cask
707.3 ns
8.41 ns
7.87 ns
0.0725
457 B

GenerateKey_Floor
184.6 ns
0.76 ns
0.71 ns
0.0446
281 B

IsCaskString
175.9 ns
3.50 ns
3.28 ns
-
-

IsCaskBytes
128.1 ns
0.64 ns
0.60 ns
-
-

Obviously, .NET Framework remains much slower than .NET 8.0. The remaining allocations are not in our control. In some cases, I think GC is just not that significant relative to the other increased costs of things we're doing on .NET Framework (like crypto just being significantly slower) and so reducing bytes allocated isn't making things that much better. In other cases, we have decent gains here.
My starting point was let's not optimize .NET Framework at all. We'll say it works, but if you care about perf, use modern .NET. But now I think it's not terrible to do the best we can as long as we can continue to keep it contained to polyfills and not spread #if all over the place.
Legend
  Mean      : Arithmetic mean of all measurements
  Error     : Half of 99.9% confidence interval
  StdDev    : Standard deviation of all measurements
  Gen0      : GC Generation 0 collects per 1000 operations
  Allocated : Allocated memory per single operation (managed only, inclusive, 1KB = 1024B)
  1 ns      : 1 Nanosecond (0.000000001 sec)

I dreamt of a bug and now I'm inclined to revert the polyfills to simpler code.
Fixed in https://github.com/microsoft/cask/pull/18/commits/2e176bdf9334dcb0d842316cdcd6b07c897b9e41
But I also put no-merge on this while we consider if this is worth it.

OK, I've simplified to reduce bug risk, and measured to ensure that the things I simplified weren't worth it. See updated comment with benchmarks.


Discover minimum amount in EUR

I think this solves #589.

Description

TESTING STAGE

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have read CONTRIBUTING.md
[ ] I fully understand Github Flow.
[ ] My code adheres to the code style of this project.
[ ] I have updated the documentation in /docs if I have changed the config, arguments, logic in how the bot works, or anything that understandably needs a documentation change.
[ ] I have updated the config file accordingly if my change requires a new configuration setting or changes an existing one.
[ ] I have tested the bot with no issues for 24 continuous hours. If issues were experienced, they have been patched and tested again.

Thanks! I agree, this is the cause of the issue.


spirv-fuzz: Add missing float bit width capabilities

Fixes #3669.

It would be good to add simple tests.


Raise ValueError when running play_vid mode

Following README.md, I ran the code below:
Stream a movie onto the Castle scene:
python train.py --config configs/castle.txt --render_only --render_modes play_vid
But this was terminated with following traceback:
Traceback (most recent call last):
  File "/home/admin/projects/nepmap/nepmap-server/train.py", line 384, in <module>
    render_sandbox(radiance_field, occupancy_grid, scene_aabb,
  File "/home/admin/projects/nepmap/nepmap-server/helpers.py", line 1412, in render_sandbox
    reader = VideoReader(Path(extra_info["vid_path"]),
  File "/home/admin/local/mambaforge/envs/nepmap1/lib/python3.9/site-packages/gsoup/video.py", line 231, in __init__
    raise ValueError("target resolution must have common divisor with original resolution")

The version of the packages I use:

python:  3.9.18
gsoup: 0.0.8

Hi there, thanks for the feedback !
When we load a video it is resized to fit the projector resolution used for training.
however, I was using a naive resizing approach so the video didn't work. Now we use the opencv resizing method.

please pull the changes
please uninstall gsoup and opencv-contrib-python, and then reinstall gsoup (the latest version is 0.2.3)

the reason to unisntall opencv-contrib-python, is because the new gsoup versions use opencv-python which conflicts with opencv-contrib-python.


No profile picture confirmation/cancel button

Subject: No profile picture confirmation/cancel button
Description:
Tried to save my profile but wasn't able to confirm the photo first. Using the courchers beta app on android (e/foundation OS)
Results:
confirmation and cancle buttons should have been visible for the profile picture.
Backend version: develop-1416d148
Frontend version: develop-1416d148
User Agent: Mozilla/5.0 (Linux; Android 9; Unspecified Device) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Mobile Safari/537.36
Page: https://app.couchers.org/profile/edit/about
User: trulyegg (6940)

Not clear.


Update Ray Cluster Upgrade tests to cover Ray Job Submission scenarios

Issue link

https://issues.redhat.com/browse/RHOAIENG-10118
What changes have been made

Added Ray Job submission use case in Pre upgrade test TestMNISTRayClusterUp and added check to verify that Ray job exists in ray cluster in post upgrade test TestMnistJobSubmit
Verification steps

Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[ ] Manual tests
[ ] Testing is not required for this change

Tests are looks good here /job/rhods-ci-pr-test/3193/


Add nitpick configuration and add it to pre-commit

add requirements folder.
add license comments in files that do not have it yet.

Codecov Report

Merging #68 (0b668b4) into main (b117687) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #68      +/-   ##
==========================================
+ Coverage   84.73%   84.77%   +0.03%
==========================================
  Files          84       84
  Lines        4770     4769       -1
  Branches      782      782
==========================================
+ Hits         4042     4043       +1
+ Misses        591      590       -1
+ Partials      137      136       -1

Impacted Files
Coverage Δ

orchestrator/cli/database.py
42.42% <ø> (ø)

orchestrator/services/subscriptions.py
56.87% <ø> (-0.27%)
:arrow_down:

orchestrator/utils/show_process.py
100.00% <ø> (ø)

.../websocket/managers/broadcast_websocket_manager.py
40.00% <ø> (ø)

...tor/websocket/managers/memory_websocket_manager.py
79.54% <ø> (ø)

orchestrator/websocket/websocket_manager.py
71.42% <ø> (ø)

orchestrator/api/models.py
77.41% <100.00%> (ø)

orchestrator/app.py
78.26% <100.00%> (ø)

orchestrator/forms/__init__.py
97.50% <100.00%> (ø)

orchestrator/settings.py
100.00% <100.00%> (ø)

... and 1 more

Continue to review full report at Codecov.

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


What do you think about the Telegram robot?

can I contribute?

I think that's a good idea but few people use this cause it's available on web


Meteor autopublish have changed a bit

Hey
So you will have to update the maintainer list of the project if you want your meteor package to auto publish again.
meteor admin maintainers vodkabears:vide --remove meteorpublish
meteor admin maintainers vodkabears:vide --add publishbot
Do this and I'll see if I can't trigger an autopublish from the site.

Done.

So I could re-trigger an auto publish for version 0.3.6 but I can't seem to do it for version 0.3.7.
So if you want version 0.3.7 to be available for Meteor you need to run a manual publish.
First checkout 0.3.7
git checkout 0.3.7
Then from the checkout run a manual publish
meteor publish
This will generate a .versions file which you can ignore and remove.

Done, thx for the support.


support terraform 0.12

https://github.com/fhut/terraform-provider-credstash/releases

I created a new release. Thanks @fhut!
https://github.com/sspinc/terraform-provider-credstash/releases/tag/0.4.0


RuntimeError: NVML_SUCCESS == r INTERNAL ASSERT FAILED at "/opt/pytorch/pytorch/c10/cuda/CUDACachingAllocator.cpp":830

I am running a training code. It is running without any problem for 18853 iterations and then throwing this error. I have not written anything new at this point, the whole code was executed for many iterations before throwing this error.
The graph network was operated in 'rev' mode starting from iter.No 12000
I have tried to run the training code multiple times and the same error occured at the same iteration and same code line.
I got the same error at different points with different batch_sizes. For batch_size = 8, the same error occured at around 17500 iterations. For batch_size = 16, the error occured at 18853 for three runs.

The error is a direct cause from the code line of your library coupling block.

That’s a tough one, I have had success debugging such a case by training on the CPU and then obtaining a stack trace to the actual problem (CUDA can be nasty to debug given that it’s not synchronous), might also work with CUDA_LAUNCH_BLOCKING=1

@fdraxler Thanks for reply, I will try running the code after setting CUDA_LAUNCH_BLOCKING=1 on the gpu

@fdraxler I have tried training with CUDA_LAUNCH_BLOCKING=1 but the error message was same. I have now started training on CPU.

@fdraxler Hi, I have ran it on CPU and observed that the loss values are becoming 0 after the failure point and there is a massive increase in the memory usage, I guess the values are shooting up. I have used a conv network as the constructor for GlowCouplingBlock. I am attaching the network which I have used as an image. I performed NLLLoss on this network. Do you think using CNN as constructor in GLOWCouplingBlock is causing any problem?


How can i use zooming with image in a tab bar view?

I want to use this with a tab bar view but the zoom gestures do not work correctly as the tab bar horizontal swipes take over.

that's flutter gesture arena.
ExtendedImageGesturePageView will done for it.
check code from
https://github.com/fluttercandies/flutter_candies_demo_library/blob/master/lib/src/widget/pic_swiper.dart


What is "PSquare"

Hi! I'm glad to see your great work.
Could you tell me what is "PSquare", I don't find it in paper.
And, the project consist of too many files, it's so hard for me to figure out the logic of the project, so could you provide a more specific doc.
Best!

@Wenze7 Hi, PSquare is the P^2 Bot optimized by our proposed reinforcement learning algorithm. Regarding to the doc specification, sorry for that because of the complexity of ParlAI. If you want to figure out how the code works, you may refer to their codebase for more documents.


Fully capture ruleset metadata

Need to evaluate the following fields, determine whether they belong in the analyzer-lsp, and update the code to include them in the ruleset.yaml:

[x] Dependencies (should be able to drop this)
[x] SourceTechnology (should convert to tag)
[x] TargetTechnology (should convert to tag)
[ ] Phase (could convert to tag)
[x] ExecuteAfter (used only in windup-rulesets/rules/rules-reviewed/technology-usage/spring-catchall.windup.xml)
[x] ExecuteBefore (seems unused in windup)
[x] OverrideRules (seems unused in windup)

Marking the ones we can drop as completed.


Warnings about headers in Xcode7

We get the following warnings when building a project that uses the OCMock framework:
/var/folders/g9/mxr37xcs33g38q39zkf76xb00000gp/C/com.apple.DeveloperTools/7.0.1-7A1001/Xcode/SharedPrecompiledHeaders/OCMockLib-Prefix-gofacfzjzygtpbgsyryprwfysjfj/OCMockLib-Prefix.pch.pch: No object file for requested architecture     Could not resolve external type c:objc(cs)NSProxy     Could not resolve external type c:objc(cs)NSMutableArray     Could not resolve external type c:objc(cs)NSString     Could not resolve external type c:objc(cs)NSMethodSignature
This seems to be caused by an Xcode bug:
https://forums.developer.apple.com/thread/17921
I submitted a pull request #261 to fix this with what is suggested at CocoaPods/cocoapods-packager#101 - by disabling debug symbol generation

Not sure about this one. If I understand this right, the issue occurs only when using CocoaPods, and it's caused by something Apple changed in Xcode. Is there a middle ground where we only disable debug symbols in some configurations? And what does the setting actually do? Does it really disable all symbols when debugging OCMock?

This happens for us when using the pre-built OCMock library without CocoaPods.

I see. Will look into this, and figure out how big the impact is for anyone who's working on the development of OCMock. I do understand that it's quite annoying for everyone who's using OCMock.

Maybe it is possible only to enable this option in the release builds and not debug builds, which would allow developers of OCMock to debug.

Can anyone comfirm that this has been fixed with newer versions of Xcode?

I am using Xcode 9.4 and seems to be getting this warning
warning: /var/folders/g9/mxr37xcs33g38q39zkf76xb00000gp/C/com.apple.DeveloperTools/7.2-7C68/Xcode/SharedPrecompiledHeaders/OCMockLib-Prefix-cvyjwfoqmezcktanihzrttyjxdnk/OCMockLib-Prefix.pch.pch: No such file or directory
Managed to fix it by recompiling the OCMock Static lib with Precompile Prefix Header set to NO

Another solution is to set the Debug Information Format to DWARF


Add a good test suite for our logic for putting Zulip messages in emails

The way I'd probably implement that test is in 2 stages: (0) Extract relative_to_full_url as its own function, perhaps, (1) Add unit tests for specific types of syntax used there, and (2) take the sample test messages from zerver/fixtures/markdown_test_cases.json and run the URL rewriter tool on them, and add an assert that the results don't contain any relative URLs (might try to look for ['"]/ as a regular expression?).  Might be easiest to do this by just adding digest-related logic to the existing backend test that uses those fixtures.
This is a follow-up to https://github.com/zulip/zulip/pull/5728#issuecomment-313793961 and also important for our digest emails.

@phansen01 Are you planning to work on this or can I take this up?

Go for it 🤘

@phansen01 Thanks for such a fast reply! :smiley:
@zulipbot claim


Memory leak reading parquet files with pyarrow

I have many ~200MB files in a directory on hdfs. I'm trying to read them one by one and pick only the rows of data I am interested in, to not overload the memory in my machine. However I realized the memory of the downloaded datasets was not being freed. here is a simplified version of my code that still causes the error :
import pyarrow
hdfs = pyarrow.hdfs.connect()
files = hdfs.ls(path)
subdfs = []
for filename in files:
df = hdfs.read_parquet(filename).to_pandas()
subdf = df[df.index < 100]
subdfs.append(subdf)
The memory allocated by "df" isn't freed until "subdfs" doesn't exist anymore. This isn't a pandas issue because if I create a dataframe directly with pandas it the memory leak doesn't happen.
GitHub Issues for Apache Arrow
We support GitHub issues as a lightweight way to ask questions and engage with
the Arrow developer community. We use the Apache Software Foundation's JIRA
instance for maintaining a queue of development work and as the public record
for work on the project. So, feel free to delete this text and open an issue
here if you have a question or aren't sure whether something is a bug or
feature request. Any bugs, suggestions, or feature requests will eventually
need to end up in JIRA, either before or after completing a pull request.
See the Arrow JIRA project here: https://issues.apache.org/jira/browse/ARROW.
For more in-depth development discussions, please use the public mailing list
<EMAIL_ADDRESS>(first subscribe by sending an e-mail to
dev-subscribe@arrow.apache.org).

I am using pyarrow 0.11.1
and running my code in a jupyter notebook version 4.4.0

What about with pyarrow 0.12.0 (just released)? As discussed in http://arrow.apache.org/blog/2019/02/05/python-string-memory-0.12/, in pyarrow 0.11.1 and earlier, to_pandas can use an alarming amount of memory, so please try upgrading first.
Otherwise I would need a reproducible example to look at to be able to help you

I'll try that, thanks. I did find a hack to fix it, by replacing
subdfs.append(subdf)
by
subdfs.append(subdf * 1).
I'm not sure how to give you a reproducible example, just thought you might want to know about this leak.

I think this is probably working as intended,  I think pandas dataframes are created as "immutable" without a memory copy if possible.  I think the slice operation, is also zero-copy and Per  pandas documentation https://pandas.pydata.org/pandas-docs/version/0.23/generated/pandas.DataFrame.copy.html doesn't copy the actual data if possible.


Twitch API Reorganization #2

I'll leave this open for a few days for comments.

The intention of this is have both asynchronous and synchronous methods, as well as organization of the TwitchAPI into classes. Additionally, it puts all functionality code behind an internal wall for total abstraction.

Merging now.


Update multiple-ns.md

Removed duplicated word

What type of PR is this?
/kind documentation
What this PR does / why we need it:
Correct minor doc mistake
NONE

@nvibert Thanks for fixing.
/ok-to-test
/lgtm


Record field completed with invalid value in mapping constructor

Refer to the following:
https://user-images.githubusercontent.com/6260009/155486453-592aab97-13c5-41c1-ad5e-95c160c6c5ff.mov
As it can be seen, when completing the thread field, it also adds parent to this. This should be "parent".
import ballerina/io;

public function main() {
    io:println("Starting main");

    @strand{<cursor>}
    worker W1 {
        io:println("Hello world from worker #1");
    }
}

Blocked by #35407


Updated E2E Sample: Fine-Tuning and Integrating Custom Phi-3 Models with Prompt Flow

Purpose

Increased Dataset Download Percentage:

Adjusted the dataset download script to increase the percentage from 1% to 3%. This change is intended to avoid bottlenecks when fine-tuning GPUs with very small datasets, ensuring smoother and more efficient training.

Revised Steps to Request a GPU Quota:

Provided more clear instructions for requesting a GPU quota through the Azure ML workspace.

Fixed Typos and Improved Readability:

Corrected typos throughout the documentation.
Enhanced readability by refining  the document.

Does this introduce a breaking change?
When developers merge from main and run the server, azd up, or azd deploy, will this produce an error?
If you're not sure, try it out on an old environment.
[ ] Yes
[x] No

Does this require changes to learn.microsoft.com docs?
This repository is referenced by (https://azure.microsoft.com/products/phi-3)
which includes deployment, settings and usage instructions.
[ ] Yes
[x] No

Type of change
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x] Documentation content changes
[ ] Other... Please describe:

I've verified that this URL https://techcommunity.microsoft.com/t5/educator-developer-blog/fine-tune-and-integrate-custom-phi-3-models-with-prompt-flow/ba-p/4178612?wt.mc_id=studentamb_279723 works well with that markdown.

@skytin1004  correct tracker from content coming from the cookbook is ?WT.mc_id=aiml-137032-kinfeylo this is why it was failing