Minor memory leak in c++ samples using MPSolver::CreateSolver()

What version of OR-tools and what language are you using?
Version: master
Language: C++
A few C++ samples such as ortools/linear_solver/samples/basic_example.cc use MPSolver::CreateSolver() to obtain a pointer to a solver object, which is effectively created with:
https://github.com/google/or-tools/blob/b77bd3ac69b7f3bb02f55b7bab6cbb4bab3917f2/ortools/linear_solver/linear_solver.cc#L613
and the resulting pointer is never delete'd by the sample, resulting in a leak:

valgrind ./bin/basic_example
...
==284285== HEAP SUMMARY:
==284285==     in use at exit: 155,472 bytes in 2,319 blocks
==284285==   total heap usage: 9,863 allocs, 7,544 frees, 835,570 bytes allocated
==284285==
==284285== LEAK SUMMARY:
==284285==    definitely lost: 400 bytes in 1 blocks
==284285==    indirectly lost: 3,207 bytes in 40 blocks
==284285==      possibly lost: 0 bytes in 0 blocks
==284285==    still reachable: 151,865 bytes in 2,278 blocks
==284285==         suppressed: 0 bytes in 0 blocks
==284285== Rerun with --leak-check=full to see details of leaked memory
==284285==
==284285== For lists of detected and suppressed errors, rerun with: -s
==284285== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)
...

One can think of using std::unique_ptr to track the MPSolver object and avoid the leak.
But also, it might be OK to not bother about this in such demo samples.

good catch !
Since few people may use these samples as skeleton to integrate or-tools in their own apps/program I think we should fix it IMHO


New code gen foundations

Primarily represents the introduction of handlebars as the code-generation engine; ported all entity templates to use. Database templates are still using legacy custom for now.

There is a breaking change related to test projects; the DatabaseExecutor within the FixtureSetup.cs should look like the following:
                return await DatabaseExecutor.RunAsync(new DatabaseExecutorArgs(
                    count == 0 ? DatabaseExecutorCommand.ResetAndDatabase : DatabaseExecutorCommand.ResetAndData, config["ConnectionStrings:Database"],
                    typeof(Database.Program).Assembly, Assembly.GetExecutingAssembly()) { UseBeefDbo = true } ).ConfigureAwait(false) == 0;


"Other Linker Flags" setting is not found

New Issue Checklist

[x] I have read and understood the CONTRIBUTING guide
[x] I have read the Documentation
[x] I have searched for a similar issue in the project and found none

Issue Description
"Other Linker Flags" in https://github.com/rs/SDWebImage#installation-by-cloning-the-repository is 404(not found).
where should I access to see latest document?

Other Linker Flags is just right on Xcode -> Project -> Build Settings -> Linking -> Other Linker Flags:

Or simple: Go to Build Settings, use Command + F to search "Other Linker Flag" is faster.

Thank you
The problem was solved.
If you have plenty of time, update the image please.

Added this image into the tutorial.

Fixed by #2030


Added Try+Except block for ProbeForRead

Possible security problem under double-fetch conditions. Microsoft says all ProbeForRead calls should be treated as if they could throw exceptions.  SRC: https://docs.microsoft.com/en-us/windows-hardware/drivers/ddi/content/wdm/nf-wdm-probeforread
I choose Contribution option (1)

Thank you very much for your contribution!


mix still.compile should fail when there's an error

When we are compiling a site for deployment, we need to know if it failed, otherwise will be deploying an incomplete site.

For reference: see step Run MIX_ENV=site mix still of this build.

Fixed in https://github.com/still-ex/still/commit/bfd6e6bd1601c17a47129db312ef7a8971b063ac


mac-3 is dead.

Type of Request
bug
Infrastructure Environment
LUCI, Github, Cocoon scheduler, Autosubmit, etc...
What is happening?
mac-3 is dead.

Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

mac-3 is up and running fine, I can login remotely from my end. mac-3 is out of sync with Salt master, ran command "salt-call state.apply" to re-sync again with Salt master.
flutter@flutter-devicelab-mac-3 ~ % uptime
9:49  up 43 mins, 3 users, load averages: 1.28 1.35 1.37


Upgrade to CircleCI v2

 Update CircleCI to v2. We have until August before they kill v1.

Good job beating beating that deadline @elidupuis 🎉


Monitor BitMEX API RSS Feed for downtimes and update announcements

One way of implementing this could be by integrating a RSS feed plugin into a element room.. After a quick google search I found that, but haven't looked into it in more detail.
https://github.com/maubot/rss

RSS Feed: https://blog.bitmex.com/api_announcement/feed/
BitMEX API Changelog: https://www.bitmex.com/app/apiChangelog

https://github.com/maubot/rs

I just tried to check it out, but I get 404 on this link 😭 have you mistyped the repo?

There was a s missing 🙈

Thanks for the suggestion! I'll try to add it to grafana, which should be simpler than adding matrix bot infrastructure, and also would allow alerts.

@holzeis are you working on this or is it up for grabs? (if I remember correctly, you have tried to experiment with the grafana plugin?)

@klochowicz it has been deprioritised in favor of the electron wrapper and since then I haven't worked on it.. go ahead and take it 😉

I made some headway before the weekend:

the RSS plugin works, I have managed to add a dashboard with the feed: https://itchysats.grafana.net/d/l6A4UVV4k/bitmex?orgId=1
Grafana does not support alerts from tables yet :(
the above means that the way to go is using a grafana managed alert, but that for some reason can't retrieve data from data source ("plugin not available" error), eventhough exploring the data source in "Explore" works just fine. I have sent an email to Grafana and will update the ticket once I know more.

turns out that grafana doesn't provide support for 3rd-party plugins, the RSS author mentioned that it can't be used for alerting; I was pointed to another plugin that might work with that, but the overhead of implementing it is quite high - after trying it out I came to the conclusion that a matrix bot might be easier to implement (where would we host it?)

This project is unmaintained


Make electron-rebuild a grunt task

This works for me, at least.

Forget it. Turns out I just had the wrong Python path set.


Should I use Mobile App or Web App as the beginning service?

I noticed that Mobile App is still available for step 3 in Create a new Azure Mobile App backend
. When I follow the instructions for creating a Web App, the section on Monitor no longer provides the information I am expecting. There may be features coming or perhaps it is a mismatch in how it is done on Mobile vs Web Apps. Perhaps a comment may help with my confusion. Suggestions for how to advise me in teaching the new right way?

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

ID: 1b726a7b-f40a-6e43-fa23-716dee06666b
Version Independent ID: 03f1b853-90df-455c-7b77-1cc7f373cd56
Content: Create a Universal Windows Platform (UWP) that uses Azure Mobile Apps
Content Source: articles/app-service-mobile/app-service-mobile-windows-store-dotnet-get-started.md
Service: app-service-mobile
GitHub Login: @elamalani
Microsoft Alias: emalani

Thanks for the feedback! We are currently investigating and will update you shortly.

Hi @brucedkyle. You can follow the tutorial using a Web App and don't worry about mismatch. From a PaaS standpoint, Mobile App and Web App are the same. The confusion is stemming from the fact that mobile apps and associated services will soon be migrated over to app center. Once that migration is complete, @elamalani will be updating the docs.


Bump flake8-isort from 6.0.0 to 6.1.1 to fix error

Overview

See https://github.com/pyvista/pyvista/pull/5538.

Details

None

LGTM


bicep build: don't add empty template properties

If I have a bicep file that has no outputs (for example) don't add the outputs property to the json file

Forgot to mention prior that I was thinking of taking this up and noticed that in the PR guidelines. My apologies.

No worries. Thanks for contributing! (I added a couple of comments to your PR.)

Comments resolved. Thanks for the tips.


Improve the built-in Disqus template

Set disqus_identifier, disqus_title, and disqus_url only if the user has explicitly provided them in the metadata of a post. Two reasons:

If Hugo does not set them explicitly, Disqus knows how to fetch them anyway. The shortname is the only required variable for Disqus, and all other variables are optional.

If Hugo sets them, there are two problems:

disqus_identifier is set to .Permalink by default, which is identical to disqus_url in the current template. This is not only redundant, but also defeats the purpose of disqus_identifier: the identifier is introduced to instruct Disqus to load the same discussion thread for a page even if its URL has been changed. In other words, it is supposed to be a fixed value, independent of the page URL. Ideally this should be provided by the user explicitly.

disqus_url is redundant because Disqus also uses .Permalink as its default value. There is one scenario in which Hugo's .Permalink will fail Disqus: baseurl = "/" in config.toml.1 In this case, .Permalink does not contain the full URL (e.g., /2017/06/23/hello-world/). However, if you do not hardcode disqus_url to .Permalink, Disqus will continue to work, because it basically uses window.location.href.

Do not load Disqus when the website is previewed locally, otherwise it is very confusing and even frustrating (e.g. comments sent to an unexpected URL by accident).

I believe these two changes will greatly improve the user experience with Disqus for those who just uses the default Disqus template in Hugo.

In case you wonder why I set baseurl = "/": I know the disadvantages, e.g., sitemap.xml won't work, but I want all URLs on my website to be portable, i.e., not tied to my domain. The solution is .RelPermalink in the theme/templates but most theme authors just use .Permalink and I cannot fix these themes. The most important reason for making URLs portable is Netlify's preview for pull requests: the best way to make the preview websites work is baseurl = "/" before other theme authors change .Permalink to .RelPermalink and use relURL.

 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.

I think AppVeyor sent the wrong status to Github: https://ci.appveyor.com/project/bep/hugo/build/1.0.69 (success here)

Possibly related:
Same Disqus Comments Showing up on Every Post?
https://discourse.gohugo.io/t/same-disqus-comments-showing-up-on-every-post/7176

Thanks for merging my first PR to Hugo! 🎉  Yes, I have used Disqus for quite a few years, and I think I'm fairly familiar with Disqus. Since you were okay with this PR, I'll submit another one shortly, which will not be as important as this one but will make the code a little bit cleaner.
And just in case anyone run into problems after this PR, below is the original template code:
{{ if .Site.DisqusShortname }}

Please enable JavaScript to view the comments powered by Disqus.
comments powered by Disqus{{end}}

If it is desirable for a user to revert to the old template, the above code can be saved to layouts/partials/disqus.html. Then replace {{ template "_internal/disqus.html" . }} with {{ partial "disqus.html" . }}.

Hey @yihui - thanks for this contribution. Am I right in thinking that this change negates the advice in the Hugo docs on conditionally loading Disqus? Given that the built-in Disqus template now does not load on localhost, the suggestion that users create a separate partial seems redundant. If so, let me know - I'm happy to submit a PR updating the docs.

@tombennet Yes, you are right. The docs should be updated, and there is no need to use a separate partial only for this purpose. Thanks!


Add auto backport functionality

Coming from meta issue https://github.com/opensearch-project/opensearch-clients/issues/14:

[ ] Add auto-backport PR creation Github action to make sure all the PRs are backported for a particular release.
[ ] Add auto-delete for merged auto-backport PRs to clean up the branches created during auto-backport.
[ ] Add documentation for the same.

Done in #136


Search doesnt work if opened inside hamburger menu

Documentation Issue
When you open hamburger menu, then click on search icon, you can't write search query, keyboard doesn't open
Additional Details
Tried on Firefox mobile, Chrome mobile and desktop.
https://github.com/payloadcms/payload/assets/100141028/92007cd0-0cd5-4ae9-ab43-e5fd01ce8890

Hi @emastho - are you still able to replicate this bug? I tested tried with Chrome, FireFox, Safari and it is working correctly for me.


Tested Remove button in System console T955

Tested Remove button in System console for the PR MM-PR-24912

[x] Browser

Totally missed this, sorry @varghesejose2020 and thank you for your contribution!


Ask about a new feature

Hi there,
is it possible to install a switch for each device that activates or deactivates it?
If the device is activated, the output is sent regularly.
If the device is deactivated, nothing is sent on the output.

Hi Thomas,
so you want to pause specific devices without deleting all the timers? You can achieve this pretty easily actually by just adding a switch node to the ui schedulers output and then check if a certain condition is true. Like so:
[{"id":"991c539f.fb285","type":"comment","z":"d8cb3879.806298","name":"### This scenario is permanent and requires a file to read an write to ###","info":"","x":320,"y":60,"wires":[]},{"id":"ef11c701.cd9fa8","type":"file","z":"d8cb3879.806298","name":"WriteFile","filename":"/home/debian/Documents/time-schedule0.txt","appendNewline":true,"createDir":true,"overwriteFile":"true","encoding":"none","x":700,"y":60,"wires":[[]]},{"id":"7822012a.876838","type":"inject","z":"d8cb3879.806298","name":"FireAfterReboot","props":[{"p":"payload","v":"","vt":"date"},{"p":"topic","v":"","vt":"string"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","payload":"","payloadType":"date","x":160,"y":120,"wires":[["2920de2a.a86312"]]},{"id":"2920de2a.a86312","type":"file in","z":"d8cb3879.806298","name":"ReadFile","filename":"/home/debian/Documents/time-schedule0.txt","format":"utf8","chunk":false,"sendError":false,"encoding":"none","x":340,"y":120,"wires":[["18d69559.b452cb"]]},{"id":"ee4ea530.9cd2b","type":"ui_switch","z":"d8cb3879.806298","name":"","label":"MyDevice","tooltip":"","group":"bfe48a3e.1b0f4","order":1,"width":0,"height":0,"passthru":true,"decouple":"false","topic":"","style":"","onvalue":"true","onvalueType":"bool","onicon":"","oncolor":"","offvalue":"false","offvalueType":"bool","officon":"","offcolor":"","x":960,"y":120,"wires":[[]]},{"id":"18d69559.b452cb","type":"ui_time_scheduler","z":"d8cb3879.806298","group":"bfe48a3e.1b0f4","name":"","startDay":"1","refresh":60,"devices":["Device 1"],"customPayload":false,"eventMode":false,"outputs":2,"order":2,"width":"6","height":"1","x":510,"y":120,"wires":[["ef11c701.cd9fa8"],["45569cf5.5b1b84"]]},{"id":"45569cf5.5b1b84","type":"switch","z":"d8cb3879.806298","name":"check global.MyDevice","property":"myDevice","propertyType":"global","rules":[{"t":"true"}],"checkall":"true","repair":false,"outputs":1,"x":740,"y":120,"wires":[["ee4ea530.9cd2b"]]},{"id":"72109111.b16df","type":"ui_switch","z":"d8cb3879.806298","name":"","label":"MyDeviceEnabled","tooltip":"","group":"bfe48a3e.1b0f4","order":3,"width":0,"height":0,"passthru":true,"decouple":"false","topic":"","style":"","onvalue":"true","onvalueType":"bool","onicon":"","oncolor":"","offvalue":"false","offvalueType":"bool","officon":"","offcolor":"","x":530,"y":180,"wires":[["dde1bb57.77ae58"]]},{"id":"dde1bb57.77ae58","type":"change","z":"d8cb3879.806298","name":"","rules":[{"t":"set","p":"myDevice","pt":"global","to":"payload","tot":"msg"}],"action":"","property":"","from":"","to":"","reg":false,"x":740,"y":180,"wires":[[]]},{"id":"8f047fb5.394ab","type":"inject","z":"d8cb3879.806298","name":"FireAfterReboot","props":[{"p":"payload"}],"repeat":"","crontab":"","once":true,"onceDelay":0.1,"topic":"","payload":"true","payloadType":"bool","x":160,"y":180,"wires":[["72109111.b16df"]]},{"id":"bfe48a3e.1b0f4","type":"ui_group","name":"PermDemo","tab":"2936b813.6cde68","order":5,"disp":true,"width":"6","collapse":false},{"id":"2936b813.6cde68","type":"ui_tab","name":"Scheduler Demo","icon":"dashboard","order":1,"disabled":false,"hidden":false}]
Please let me know if does what you meant.

Hi there,
Thank you for your answer.
Yes, that's what I meant.
That's how I've already solved it.
But it would be nice if you could also set it directly in the scheduler.

Okay good, just wanted to give you an immediate solution.
Pausing specific timers is definitely going to be added soon, just haven't thought about pausing the whole device at once! I am going to add a solution for this as well as soon as I manage to implement the specific pausing function. I'll give you an update here as soon as it is finished!
Thanks for your suggestion 👍

Okay good, just wanted to give you an immediate solution.
Pausing specific timers is definitely going to be added soon, just haven't thought about pausing the whole device at once! I am going to add a solution for this as well as soon as I manage to implement the specific pausing function. I'll give you an update here as soon as it is finished!
Thanks for your suggestion 👍

Hi,
would it be possible to change your fantastic timer so that I could turn lights on at 20:00 and turn them of the day after at 6:00. Or should I do 2 timers 20:00 till 23:59 and 06:00? Or turn them on at 06:00 and off at 20:00 and inverse the output?
Regards,
Serge

Hi,
would it be possible to change your fantastic timer so that I could turn lights on at 20:00 and turn them of the day after at 6:00. Or should I do 2 timers 20:00 till 23:59 and 06:00? Or turn them on at 06:00 and off at 20:00 and inverse the output?
Regards,
Serge

Hi @connermacleod69,
I am currently not planning on changing the 24hour limit. You can simply add two timers, just like you said, from 20:00 to 23:59 and another one from 00:00 to 06:00. Another possibility would be to activate the event mode, and set an ON event at 20:00 and an OFF event at 06:00 in the morning.

Hi @connermacleod69,
I am currently not planning on changing the 24hour limit. You can simply add two timers, just like you said, from 20:00 to 23:59 and another one from 00:00 to 06:00. Another possibility would be to activate the event mode, and set an ON event at 20:00 and an OFF event at 06:00 in the morning.

Hi @fellinga,
sorry I missed the on/off item...This now works for me.
Regards,
Serge

Hi @fellinga,
sorry I missed the on/off item...This now works for me.
Regards,
Serge

@schamu 1.9.0 now has the ability pause devices directly from the GUI and from incoming messages! I'd appreciate your feedback!


Fixes search field overflow #956

Removes minimum visible characters in search field.

Codecov Report

Merging #1144 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #1144   +/-   ##
========================================
  Coverage    70.22%   70.22%
========================================
  Files           52       52
  Lines         3023     3023
  Branches       434      434
========================================
  Hits          2123     2123
  Misses         748      748
  Partials       152      152

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 9466290...5c3ff28. Read the comment docs.

Please merge this fix!

+1

This will help a lot.
Please merge.

+1

bump!

Thanks for the contribution @ArtistsTechGuy


switch language returns a empty page in blog detail page

http://dubbo.apache.org/en-us/blog/dubbo-meetup-beijing-may-12th-2018.html
When switch to Chinese, the page is empty.
I suggest that we can provider a default page, when other language pages do not exist, show the default page.
The default page content is the guidance about how to contribute.

The current website return 404 when page does not exist. This looks good to me. I think this issue can be closed.


no devices or entities with auto config, but list appliances shows supported results

HA core version
2023.10.1
Intergration version
0.3.21
Device type and model
Dehumidifier MAD50P1AWS
Used App
SmartHome
The description of problem
login is successful, but no devices found.

listing devices shows supported result:

The logs
No response

The description of problem
login is successful, but no devices found.

https://github.com/georgezhao2010/midea_ac_lan#add-device
After the account is configured, Click 'ADD DEVICE' once more to add new device. You could repeat the above action to add multiple devices.

ooooh! that was confusing! it works now, thank you.


Incompatible Swift Version  4.1 when built in 4.1.2. Help

Hi. I dont have any coding capabilities at all. I am trying to implement Watson Assistant for IOS with Swift. For that, I have to install watson sdk and i get the following error message:
Skipped installing swift-sdk.framework binary due to the error:
"Incompatible Swift version - framework was built with 4.1 (swiftlang-902.0.48 clang-9<IP_ADDRESS>) and the local version is 4.1.2 (swiftlang-902.0.54 clang-9<IP_ADDRESS>)."
A shell task (/usr/bin/xcrun xcodebuild -project /Users/mdtaber/Downloads/Iris-Swift/Carthage/Checkouts/Starscream/Starscream.xcodeproj CODE_SIGNING_REQUIRED=NO CODE_SIGN_IDENTITY= CARTHAGE=YES -list) failed with exit code 72:
xcrun: error: unable to find utility "xcodebuild", not a developer tool or in PATH
How do I fix this?
List the software versions you're using:
I am on Swift version 4.1.2
Thank you for your help!

"Incompatible Swift version - framework was built with 4.1 (swiftlang-902.0.48 clang-9<IP_ADDRESS>) and the local version is 4.1.2 (swiftlang-902.0.54 clang-9<IP_ADDRESS>)."
A shell task (/usr/bin/xcrun xcodebuild -project /Users/mdtaber/Downloads/Iris-Swift/Carthage/Checkouts/Starscream/Starscream.xcodeproj CODE_SIGNING_REQUIRED=NO CODE_SIGN_IDENTITY= CARTHAGE=YES -list) failed with exit code 72:
xcrun: error: unable to find utility "xcodebuild", not a developer tool or in PATH

Seems like you are using Carthage to build Starscream framework.
You can use carthage update --no-use-binaries to re-build the pre-build framework with the Xcode you've selected. For more info for the --no-use-binaries flag, see here


cant use credentials_json after an hour and half

TL;DR
After about hour and half, using credentials_json is not effective. in the same workflow job.
I understand using token has timelimit, but as i understand credentials_json should not have timelimit.
Expected behavior
Be able to preform actions as "kubectl get pods" using credentials_json even after long period of time in the same job.
Observed behavior
i have a workflow with one job. and few steps.
everything working fine, if for example the steps of watch logs take 20-30 minutes.
but if it takes hour and half for example,
then the last step "get pods" gets error of
error: You must be logged in to the server (Unauthorized)
and the step remove chart gets error of
Error: Kubernetes cluster unreachable: the server has asked for the client to provide credentials
Action YAML
jobs:
  tier1:
    if: ${{ github.event_name == 'workflow_dispatch' }}
    strategy:
      fail-fast: false
      matrix:
        suite: ${{ fromJSON(github.event.inputs.suite) }}
    name: ...
    runs-on: [ gcp-self-hosted-org ]
    outputs:
      kubectl_config: ${{ steps.kubectl_secret.outputs.kubectl_config }}
    permissions:
      contents: 'read'
      id-token: 'write'

    steps:
      - name: Checkout devops-scripts code
        uses: actions/checkout@v4

      - name: update environment parameters
        run: |
          ...

      - name: authenticate to google
        uses: google-github-actions/auth@v2
        with:
          credentials_json: ${{ secrets.GCP_CI_CD_SECRET }}

      - name: Set up Cloud SDK
        uses: google-github-actions/setup-gcloud@v2
        with:
          project_id: ${{ env.project }}

      - name: get credentials for the kubectl
        id: kube_cred
        uses: google-github-actions/get-gke-credentials@v2
        with:
          cluster_name: ${{ env.cluster_name }}
          location: ${{ env.region }}
          project_id: ${{ env.project }}

      - name: add helm repository
        run: |
          helm repo add ....
          helm repo update
          helm upgrade -i ${{ env.RELEASE_NAME }} ....

      - name: watch logs
        run: |

          kubectl get pods -n default
          kubectl wait --for=jsonpath='{.status.phase}'=Running pod/${{ env.POD_NAME }} -n default --timeout=120s
          kubectl logs -n default ${{ env.POD_NAME }} -f | tee ${{ env.ARTIFACT_NAME }}.txt

      - name: get pods
        run: |
          kubectl get pods -n default

      - name: remove chart
        if: ${{always()}}
        run: |
          set +e
          helm uninstall ${{ env.RELEASE_NAME }} -n default

Log output
EXIT_CODE=$(kubectl get pods .. -n default -o json | jq -r '.status.containerStatuses[].state.terminated.exitCode')
  echo $EXIT_CODE
  if [ "$EXIT_CODE" != 0 ]; then
    echo "Failure"
    exit 1
  fi
  shell: /usr/bin/bash -e ***0***
  env:
    environment: ...
    connector: ...
    suite: [ 'blue' ]
    x_flag: false
    tenant: ..
    region: ...
    cloud_provider: ..
    cluster_name: ...
    project: ...
    domain_name: ...
    CLOUDSDK_AUTH_CREDENTIAL_FILE_OVERRIDE: /home/runner/_work/tier1_shai/tier1_shai/gha-creds-eb4dc72a558f8d5c.json
    GOOGLE_APPLICATION_CREDENTIALS: /home/runner/_work/tier1_shai/tier1_shai/gha-creds-eb4dc72a558f8d5c.json
    GOOGLE_GHA_CREDS_PATH: /home/runner/_work/tier1_shai/tier1_shai/gha-creds-eb4dc72a558f8d5c.json
    CLOUDSDK_CORE_PROJECT: sqreamtech
    CLOUDSDK_PROJECT: sqreamtech
    GCLOUD_PROJECT: sqreamtech
    GCP_PROJECT: sqreamtech
    GOOGLE_CLOUD_PROJECT: sqreamtech
    CLOUDSDK_METRICS_ENVIRONMENT: github-actions-setup-gcloud
    CLOUDSDK_METRICS_ENVIRONMENT_VERSION: 2.1.0
    KUBECONFIG: /home/runner/_work/tier1_shai/tier1_shai/gha-kubeconfig-79fec1d14a0268a0
    KUBE_CONFIG_PATH: /home/runner/_work/tier1_shai/tier1_shai/gha-kubeconfig-79fec1d14a0268a0
    TESTING_FRAMEWORK_IMAGE: 89148da185
    POD_NAME: ...
    RELEASE_NAME: ...
    ARTIFACT_NAME: ...
error: You must be logged in to the server (Unauthorized)

Failure
Error: Process completed with exit code 1.

Additional information
My runner are self hosted runners. using runner-scale-set.

Hi @ShaiShalevSQream - this is an issue with get-gke-credentials, not auth. If you don't use the auth plugin (use_auth_provider), then get-gke-credentials generates an OAuth access token and puts it in the Kubeconfig.
You can either refresh auth before running kubectl get po (by using the get-gke-credentials action again) or install and use the auth plugin which handles refreshes.

Thank you very much for your answer.


MNT: Import setup_ophyd at top level.

I assume I will be voted down here, but what about calling setup_ophyd() in the __init__.py as well? We have previously seen cases where executing code on import is a tempted but bad idea (bluesky's qt_kicker) but I'm not convinced this is in the same category. Is there ever a case where it shouldn't be called? Should ophyd be in charge of setting itself up?
Note: as stated in the title, this PR does not do what I am suggesting above; it merely imports it in a more obvious place. I assume that is non-controversial.

This was previously done with the session manager, which you were unhappy with.

@klauer and I discussed offline, but for posterity: my unhappiness with the session manager was its tight coupling to IPython (granted, there was a "fake IPython" fallback, but still). The more important problem with my proposal above is that a problem in pyepics/EPICS could cause ophyd to die on import. The explicit call to setup_ophyd() gives us an opportunity to investigate debug.


[FLINK-11870] Code refactor for some details.

What is the purpose of the change
Code refactor for some details
Brief change log

use computeIfAbsent to simple code.
remove duplicate call for jobGraph.getJobID()

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): ( no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not documented)

Those changes do not give us any significant benefit, I would be ok with such refactor on some other ocassion once changing some code around. I am against though a change just for a change itself as it only pollutes changelog of the file.

Got it.
Thanks for reminding me about this. I'd like close this first.
If I find more places to optimize when reading the source code, I will resubmit pr. Thx.


In some cases, merging data doesn't work

The reason is that postman automatically adds some request headers when sending a request


Remove memory copy to improve memory range checking

Jenkins, test this please

Jenkins CI test (for commit 8a403ef0e7dc74b7d60a92723fe09363315003c3, target branch master) failed with status FAILURE.

Please check CI errors

jenkins, test this please.

Jenkins CI test (for commit 49daa5b0d3d415776dd1386304b5462dd76bf316, target branch master) failed with status FAILURE.

jenkins, test this please.

Jenkins CI test (for commit 0126e53346cda244c44cd97565565805f8466198, target branch master) failed with status FAILURE.

jenkins, test this please.

Jenkins CI test (for commit e23f792ba41352af6d10271c422939d11769f6b2, target branch master) succeeded!


Python script failed and default path error

When I run load_mp_landmarks.py and make_bvh_files.blend in blender, I met the error " Python script failed. check the message   in the system console".
I also fond that the default path is your computer " /Users/lanthaon/Documents/Uni/study-pj-2021/Animation and Motion Capture/PointCloudApproach/blender_scripts/ ". How can change the default path for my computer?

Hi @chaytonmin,
what message or messages were shown to you in the system console?
If you create a new file to run load_mp_landmarks.py, it should work if you:

Delete any existing hierarchy/collection in the blender scene.
Put the file into the blender_scripts folder. The script goes to the parent folder of the blender file and looks for the folder sign_videos.

The same applies to make_bvh_files.blend. If you cloned the project with git lfs 1) and 2) should be initially fulfilled, but in any case you should reload the python file from disk in the scripting tab.
Sorry for the late reply. Please let me know if any of these tips worked for you.


macOS binary is failing to start

The macOS build from the latest RC build is failing to start https://github.com/maplibre/martin/releases/tag/v0.6.0-rc.3
dyld[88901]: Library not loaded<EMAIL_ADDRESS>  Referenced from: '/opt/homebrew/Cellar/martin/0.6.0-rc.3/bin/martin'
  Reason: tried<EMAIL_ADDRESS>(no such file), '/usr/local/lib/libssl.1.1.dylib' (no such file), '/usr/lib/libssl.1.1.dylib' (no such file)

Why didn't our CI catch this? Do we need to implement #433 for this?

@stepankuzmin can this be closed?

Yep, martin-Darwin-aarch64.tar.gz from v0.6.1 works for me 👍


Install Image aarch64 Kernel Memory Leak

Describe the bug
The current install images hosted on IoT Download Page include the 6.8.5-301.fc40.aarch64 kernel which has a known memory leak leading to high memory usage and system crashes.

Bugzilla Bug Report: Bug 2275290 - memory leak in aarch64 beginning with kernel 6.8.4
Forums Discussion: High memory usage in F40 on RPi 4, unable to find which process used them

To Reproduce

Install Fedora IoT release image on an aarch64 system

Note: Systems with low memory such as 1GB rpi3 have system crashes in a few hours

Do not update
Monitor for increasingly high memory usage

Expected behavior
Normal non-leaking memory usage preventing system crashes.
Additional context
Updating fixes the issue as the updates include an updated kernel. Currently 6.8.11-300.fc40.aarch64.

2. Do not update

This is the key point, the bug has long been fixed by an update, as have numerous CVEs, so while it's a bug, it is fixed.

@nullr0ute do we ever entertain rebuilding the install media after GA in the case of a known issue?

@nullr0ute do we ever entertain rebuilding the install media after GA in the case of a known issue?

We rebuild them daily, we do on occasion, it use to be monthly but that's slipped of late, republish them but that requires a full QE because there's always known issues, some like this one are easily fixed with an upgrade.

That makes sense to me.
The only argument I'd have for the need to republish now is that this known issue could be system crippling and prevent an upgrade. Tho the system I ran into this on was pretty constrained resource wise and was still able to update without issue so it 's probably fine to wait till the next publish.

Even on a RPi3 the memory leak wasn't bad enough you couldn't upgrade before you ran out of memory.
But we do have a process to do updates of releases on the website, we were planning on doing one RSN.


Importing PropTypes from prop-types rather than react

See #1410

Great work! should be update React version in the peerDependencies too?

Updating the React version would make sense in order to avoid those warnings:
warning<EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ~15.4.0".
warning<EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ~15.4.0".
warning<EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| ~15.4.0".

Probably<EMAIL_ADDRESS>|| ^15.5.0 would be the best option, I guess, as prop-types is compatible with React 0.14.9+.

Let's go then, thanks!

When will this update be released?

Already released, sorry for the delay!


Updated README.md, llama_cpp/llama.py and pyproject.toml to add support for cross-encoders

See video :
https://github.com/user-attachments/assets/de8a0d3b-9c75-4a60-928d-389bb8bdea2d


Feature/unittests

Hi @tobias-richter,
I added unittests for all models that base on json responses and verify the serialization with sample-json data.
while writing the tests, I noticed that some of the models where not in the correct file, those where moved.

Hi @tobias-richter,
I made some adaptions to fit our naming and language conventions.
I think, this MR should first.


locallms: Split arguments with spaces in & remove quote around prompt

I attempted to use Llama with LangChainGo utilizing the "local" LLM, but encountered some issues when I tried to send arguments (specifically the model path) to the binary.
I faced an error when I tried using local.WithArgs("-m <path_to_the_model> -p"). It seemed as if the list of arguments was being sent to exec.CommandContext as a single argument.
Once I got it up and running, I noticed that extra quotes were added to the prompt.
To me, it seems like it was designed to be parsed by Bash, but it doesn't appear to be so.
I'm not entirely sure if this is actually a bug, or perhaps I used it incorrectly. If it's supposed to work like this, feel free to close this PR.
PR Checklist

[x] Read the Contributing documentation.
[x] Read the Code of conduct documentation.
[x] Name your Pull Request title clearly, concisely, and prefixed with the name of the primarily affected package you changed according to Good commit messages (such as memory: add interfaces for X, Y or util: add whizzbang helpers).
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. Fixes #123).
[x] Describes the source of new concepts.
[x] References existing implementations as appropriate.
[ ] Contains test coverage for new functions.
[x] Passes all golangci-lint checks.

This all needs a good refactor but this seems like a good step.


Support of models with comment on the last line

Had an issue when one of the models had the single line comment on the last line. It makes framework to generate incorrect SQL
Simple PR to fix it

Thanks @SOVALINUX, we included this in our last release


Update to Go 1.17

Go 1.17 was released in mid August, might as well upgrade so we don't fall too far behind
Here's list of repositories that we'll want to update. I excluded a few collectors that aren't fully implemented. I'm going to track all of these in a single issue for convenience:

[ ] collector-build
[ ] collector-harbor
[ ] collector-sonarqube
[ ] collector-tfsec
[ ] create-build-occurrence-action
[ ] enforcer-action
[ ] enforcer-k8s
[ ] es-index-manager
[ ] grafeas-elasticsearch
[ ] new-collector-template
[ ] terraform-provider-rode
[ ] rode (#145)

I'm going to keep this open to track the other repositories


HARK Docs version has been stuck at 0.97 for years

Now that we have Sphinx working we should update that.

See #474


Carbon Graph White Background

Implement default white background for carbon graphs.
NOTE: after consideration, the default background will continue to be transparent in order to be passive to existing consumers but a white option will be provided.

This issue was closed in PR #274 (was not auto-closed for some reason).


Fix a couple SameDiff training issues

Fix an issue with SameDiff training where an NPE would happen if a just a history listener with an evaluation was used (wasn't added to activeListeners properly, loss gradient wasn't calculated but was used for loss curve).
Also fix an issue where validation didn't specify outputs properly, causing errors whenever validation was used.
Added tests for both, as both really should have been caught by tests instead of by writing examples.

Also adds single batch (DataSet/MultiDataSet) methods for SameDiff.output().  Unrelated to the rest of the changes.


[master] Restoration fails on etcd only nodes due to invalid state of loadbalancers

Environmental Info:
K3s Version: v1.21.x
Node(s) CPU architecture, OS, and Version:
Linux rke2-server 5.11.0-25-generic #27~20.04.1-Ubuntu SMP Tue Jul 13 17:41:23 UTC 2021 x86_64 GNU/Linux
Cluster Configuration:

1 etcd only node server
1 controlplane node servers

Describe the bug:
upon restoring on an existing etcd only node, it tries to setup internal load balancer with existing configuration, and tries to connect to old supervisor servers, which may or may not exist and fails to restore wiht the following error:
g 17 18:29:25 darren-pool1-hjh5k rancher-system-agent[3099]: time="2021-08-17T18:29:25Z" level=info msg="[stderr]: time=\"2021-08-17T18:29:25Z\" level=info msg=\"Waiting to retrieve agent configuration; server is not ready: Get \\\"https://<IP_ADDRESS>:6444/v1-rke2/serving-kubelet.crt\\\": context deadline exceeded (Client.Timeout exceeded while awaiting headers)\""

Backporting

[x] Needs backporting to older releases

Validated this is working on master branch commitid 8b857eef9c15ed97180079c0dcf5cf0f8cb888de


LND offline over night, force closing channel from my side after starting it again

HI,
can someone tell me why this channel was force closed from my side and how i can prevent that?
022-03-25 08:09:49.944 [DBG] CHBU: Crafting backup for ChannelPoint(d283387fd754b9d55d84a9cd6e5ca56df4332d764ffa26dca9230da1685c4bee:0)
2022-03-25 08:09:49.944 [INF] LTND: Channel backup proxy channel notifier starting
2022-03-25 08:09:49.944 [INF] ATPL: Instantiating autopilot with active=false, max_channels=1, allocation=0.500000, min_chan_size=20000, max_chan_size=130000000, private=false, min_confs=1, conf_target=2
2022-03-25 08:09:49.944 [DBG] LTND: Populating dependencies for sub RPC server: SignRPC
2022-03-25 08:09:49.944 [DBG] LTND: Populating dependencies for sub RPC server: WalletKitRPC
2022-03-25 08:09:49.944 [DBG] LTND: Populating dependencies for sub RPC server: AutopilotRPC
2022-03-25 08:09:49.944 [DBG] LTND: Populating dependencies for sub RPC server: ChainRPC
2022-03-25 08:09:49.944 [DBG] LTND: Populating dependencies for sub RPC server: InvoicesRPC
2022-03-25 08:09:49.944 [DBG] LTND: Populating dependencies for sub RPC server: RouterRPC
2022-03-25 08:09:49.944 [DBG] LTND: Populating dependencies for sub RPC server: WatchtowerRPC
2022-03-25 08:09:49.944 [DBG] LTND: Populating dependencies for sub RPC server: WatchtowerClientRPC
2022-03-25 08:09:49.945 [DBG] RPCS: Starting sub RPC server: RouterRPC
2022-03-25 08:09:49.945 [DBG] RPCS: Starting sub RPC server: AutopilotRPC
2022-03-25 08:09:49.945 [DBG] RPCS: Starting sub RPC server: InvoicesRPC
2022-03-25 08:09:49.945 [DBG] RPCS: Starting sub RPC server: VersionRPC
2022-03-25 08:09:49.945 [DBG] RPCS: Starting sub RPC server: WatchtowerRPC
2022-03-25 08:09:49.945 [DBG] RPCS: Starting sub RPC server: WatchtowerClientRPC
2022-03-25 08:09:49.945 [INF] LTND: Waiting for chain backend to finish sync, start_height=2191390
2022-03-25 08:09:49.945 [DBG] LNWL: Attempting to GetBestBlock from lw
2022-03-25 08:09:50.012 [INF] LTND: Chain backend is fully synced (end_height=2191390)!
2022-03-25 08:09:50.012 [WRN] HLCK: check: disk space configured with 0 attempts, skipping it
2022-03-25 08:09:50.012 [DBG] HLCK: Monitoring: chain backend
2022-03-25 08:09:50.012 [WRN] HLCK: check: tls configured with 0 attempts, skipping it
2022-03-25 08:09:50.012 [DBG] LNWL: Attempting to GetBestBlock from lw
2022-03-25 08:09:50.080 [DBG] LNWL: Attempting to GetBlockHeaderVerbose from lw for hash<PHONE_NUMBER>0000ecccb161ee1fb7255c0889000450b8d6ae80c028e13d03d044
2022-03-25 08:09:50.102 [DBG] LNWL: Attempting to GetBestBlock from lw
2022-03-25 08:09:50.168 [INF] PRNF: PeerNotifier starting
2022-03-25 08:09:50.179 [INF] NTFN: New block epoch subscription
2022-03-25 08:09:50.179 [INF] WTCL: (legacy) Starting watchtower client
2022-03-25 08:09:50.179 [DBG] WTCL: (legacy) Starting session negotiator
2022-03-25 08:09:50.179 [INF] WTCL: (legacy) Watchtower client started successfully
2022-03-25 08:09:50.179 [INF] WTCL: (legacy) Requesting new session.
2022-03-25 08:09:50.180 [INF] WTCL: (anchor) Starting watchtower client
2022-03-25 08:09:50.180 [DBG] WTCL: (legacy) Dispatching session negotiation
2022-03-25 08:09:50.180 [DBG] WTCL: (anchor) Starting session negotiator
2022-03-25 08:09:50.180 [DBG] WTCL: (legacy) Unable to get new tower candidate, retrying after 10s -- reason: exhausted all tower candidates
2022-03-25 08:09:50.180 [INF] WTCL: (anchor) Watchtower client started successfully
2022-03-25 08:09:50.180 [INF] WTCL: (anchor) Requesting new session.
2022-03-25 08:09:50.180 [INF] HSWC: Starting HTLC Switch
2022-03-25 08:09:50.180 [DBG] WTCL: (anchor) Dispatching session negotiation
2022-03-25 08:09:50.180 [INF] NTFN: New block epoch subscription
2022-03-25 08:09:50.180 [DBG] WTCL: (anchor) Unable to get new tower candidate, retrying after 10s -- reason: exhausted all tower candidates
2022-03-25 08:09:50.180 [INF] NTFN: New block epoch subscription
2022-03-25 08:09:50.180 [INF] NTFN: New block epoch subscription
2022-03-25 08:09:50.181 [INF] CNCT: Creating ChannelArbitrators for 1 active channels
2022-03-25 08:09:50.181 [DBG] CNCT: New ChainEventSubscription(id=0) for ChannelPoint(d283387fd754b9d55d84a9cd6e5ca56df4332d764ffa26dca9230da1685c4bee:0)
2022-03-25 08:09:50.181 [DBG] CNCT: Starting chain watcher for ChannelPoint(d283387fd754b9d55d84a9cd6e5ca56df4332d764ffa26dca9230da1685c4bee:0)
2022-03-25 08:09:50.181 [DBG] NTFN: Using height hint 2191337 retrieved from cache for outpoint=d283387fd754b9d55d84a9cd6e5ca56df4332d764ffa26dca9230da1685c4bee:0, script=0 e158976c388fcbc976f868b366b5de5af02337fc673dd30c13586541529f00ca instead of 2191322
2022-03-25 08:09:50.181 [INF] NTFN: New spend subscription: spend_id=1, outpoint=d283387fd754b9d55d84a9cd6e5ca56df4332d764ffa26dca9230da1685c4bee:0, script=0 e158976c388fcbc976f868b366b5de5af02337fc673dd30c13586541529f00ca, height_hint=2191337
2022-03-25 08:09:50.181 [INF] NTFN: Dispatching historical spend rescan for outpoint=d283387fd754b9d55d84a9cd6e5ca56df4332d764ffa26dca9230da1685c4bee:0, script=0 e158976c388fcbc976f868b366b5de5af02337fc673dd30c13586541529f00ca, start=2191337, end=2191390
2022-03-25 08:09:50.181 [DBG] LNWL: Attempting to GetUnspentOutput from lw for hash d283387fd754b9d55d84a9cd6e5ca56df4332d764ffa26dca9230da1685c4bee
2022-03-25 08:09:50.194 [INF] HSWC: Garbage collected 13 shared secret hashes at height=2191390
2022-03-25 08:09:50.287 [DBG] NTFN: Updated spend hint to height=2191390 for unconfirmed spend request outpoint=d283387fd754b9d55d84a9cd6e5ca56df4332d764ffa26dca9230da1685c4bee:0, script=0 e158976c388fcbc976f868b366b5de5af02337fc673dd30c13586541529f00ca
2022-03-25 08:09:50.287 [INF] CNCT: Close observer for ChannelPoint(d283387fd754b9d55d84a9cd6e5ca56df4332d764ffa26dca9230da1685c4bee:0) active
2022-03-25 08:09:50.287 [DBG] CNCT: Starting ChannelArbitrator(d283387fd754b9d55d84a9cd6e5ca56df4332d764ffa26dca9230da1685c4bee:0), htlc_set=(map[contractcourt.HtlcSetKey]contractcourt.htlcSet) (len=2) {
(contractcourt.HtlcSetKey) LocalHtlcSet: (contractcourt.htlcSet) {
incomingHTLCs: (map[uint64]channeldb.HTLC) {
},
outgoingHTLCs: (map[uint64]channeldb.HTLC) (len=1) {
(uint64) 1: (channeldb.HTLC) {
Signature: ([]uint8) (len=70 cap=70) {
00000000  30 44 02 20 0a 55 61 48  a9 de 48 55 88 cb 55 be  |0D. .UaH..HU..U.|
00000010  71 26 5e 7a 55 22 a1 21  39 5c f7 78 28 0a 8f 46  |q&^zU".!9.x(..F|
00000020  51 13 ff e7 02 20 07 ed  20 67 97 cf ac 17 60 61  |Q.... .. g....a|      00000030  d0 97 ec 47 d9 a3 a5 d4  6b 28 14 35 c1 91 43 6f  |...G....k(.5..Co|      00000040  bb f7 c8 29 e1 75                                 |...).u|     },     RHash: ([32]uint8) (len=32 cap=32) {      00000000  51 2b 2e 3f 18 87 69 0d  60 b9 ec 73 ff 42 9f 02  |Q+.?..i...s.B..|
00000010  a5 84 a7 23 50 0a 7a 04  33 eb 66 4a de 3b 71 1c  |...#P.z.3.fJ.;q.|
},
Amt: (lnwire.MilliSatoshi) 20001020 mSAT,
RefundTimeout: (uint32) 2191384,
OutputIndex: (int32) 0,
Incoming: (bool) false,
OnionBlob: ([]uint8) (len=1366 cap=1366) {
00000000  00 03 b8 25 f1 db 14 8e  1b 29 63 a3 df ca c9 ce  |...%.....)c.....|
},
HtlcIndex: (uint64) 1,
LogIndex: (uint64) 1

}
}
},
(contractcourt.HtlcSetKey) RemoteHtlcSet: (contractcourt.htlcSet) {
incomingHTLCs: (map[uint64]channeldb.HTLC) {
},
outgoingHTLCs: (map[uint64]channeldb.HTLC) (len=1) {
(uint64) 1: (channeldb.HTLC) {
Signature: ([]uint8) {
},
RHash: ([32]uint8) (len=32 cap=32) {
00000000  51 2b 2e 3f 18 87 69 0d  60 b9 ec 73 ff 42 9f 02  |Q+.?..i.`..s.B..|
00000010  a5 84 a7 23 50 0a 7a 04  33 eb 66 4a de 3b 71 1c  |...#P.z.3.fJ.;q.|
},
Amt: (lnwire.MilliSatoshi) 20001020 mSAT,
RefundTimeout: (uint32) 2191384,
OutputIndex: (int32) 0,
Incoming: (bool) false,
OnionBlob: ([]uint8) (len=1366 cap=1366) {
00000000  00 03 b8 25 f1 db 14 8e  1b 29 63 a3 df ca c9 ce  |...%.....)c.....|
},
HtlcIndex: (uint64) 1,
LogIndex: (uint64) 1

}
}
}
}
2022-03-25 08:09:50.289 [INF] CNCT: ChannelArbitrator(d283387fd754b9d55d84a9cd6e5ca56df4332d764ffa26dca9230da1685c4bee:0): starting state=StateDefault, trigger=chainTrigger, triggerHeight=2191390
2022-03-25 08:09:50.289 [DBG] CNCT: ChannelArbitrator(d283387fd754b9d55d84a9cd6e5ca56df4332d764ffa26dca9230da1685c4bee:0): go to chain for outgoing htlc 512b2e3f1887690d60b9ec73ff429f02a584a723500a7a0433eb664ade3b711c: timeout=2191384, blocks_until_expiry=4294967290, broadcast_delta=0
2022-03-25 08:09:50.303 [INF] CNCT:
ChannelArbitrator(d283387fd754b9d55d84a9cd6e5ca56df4332d764ffa26dca9230da1685c4bee:0): force closing chan
2022-03-25 08:09:50.308 [INF] HSWC: Removing channel link with ChannelID(ee4b5c68a10d23a9dc26fa4f762d33f46da55c6ecda9845dd5b954d77f3883d2)

Looking at this line, it's force closed because you have an outgoing HTLC that timed out.

go to chain for outgoing htlc 512b2e3f1887690d60b9ec73ff429f02a584a723500a7a0433eb664ade3b711c: timeout=2191384, blocks_until_expiry=4294967290, broadcast_delta=0

timeout=2191384

This line says you have an outgoing HTLC, which times out at height=2191384. Roughly an hour ago the testnet had already reached height=2191401, so it timed out.

blocks_until_expiry=4294967290

Looks like there's an overflow here, which can happen when the HTLC's timeout height exceeds the current block height.
I think testnet has a fast block production rate, which can lead to this. You could increase the value --max-cltv-expiry, but if your peer is offline, there's little you can do but force closing.


minimum_permissions

Hey,
I have an O365 payed family account.
I've set permissions on the Azure page and followed your instructions.
After I add the returned URL, it shows the error minimum permissions?
see screenshot
thanks for the help

To access a shared mailbox calendar by configuring shared_mailbox you need to provide that permission. Have you done so?

Roger,
thank you for the quick response.
Where can I set that permission? Azure portal? O365 admin portal.
These are my permissions.

Greetz
Jan

Might be a bug, I’ll check

Might be worth trying delete the token, so it reauthenticates from scratch

I will give that a go.
Here are my logs
This error originated from a custom integration.
Logger: custom_components.ms365_calendar.classes.permissions
Source: custom_components/ms365_calendar/classes/permissions.py:59
integration: Micosoft 365 - Calendar (documentation, issues)
First occurred: 14:06:17 (1 occurrences)
Last logged: 14:06:17
Minimum required permissions: 'Calendars.Read.Shared'. Not available in token 'ms365_calendar_JanDebocCAL.token' for account 'JanDebocCAL'.

You can look in the token and see if the permission is listed there. If it isn’t, then you need to reauthenticate.

Still the same error.
Delete the token with file editor

this is my token.
{
"token_type": "Bearer",
"scope": [
"User.Read"
],
"expires_in": 3600,
"ext_expires_in": 3600,
"access_token": "EwCAA8l6BAAUbDba3x2OMJElkF7gJ4z/VbCPEz0AAXVCMawGPzZrbmtN6nz/6sUclG7cBbkPcXLwfopN8r9FVCc8diERZOOoS3sZ3mTdM8y7FoPlHueqjw0fLTyCthgkMbxDBPuLDLOFFlRE6X4WgXQMoBw8Mz0Da0B41J32BgywVuKPtcbsN0YggUtIerooCbyX7t18nFfuakojdKuA77AEJdNvBDplCNaStmXjUMuPWMndzowLmHpR7zzpMl1hrUo6mNf3MlkTLYVaZ3eLgiLqVODTwy47mCEe20JahL9mHCg+a0HLO33//TFSqqLzBYKeb1eYXCNpg+mJL38mzu/QgqzxJ+qlW9RYTCu+BoEcK/vMUi9ksyxVLqR5qhoDZgAACDY+UhFZNx1GUAJXCreJx2je7QZhCYc9BSOj8MshNxVAfqGK1I2NC2tzYukWA7aYrG/DVvzm6BlPiPji8msYLyOsBOUIGGLe5gbepLejBrUmX0xyjbKplhlhNYL2Boyr3VLpSKpZ32S2UvhFfnPbESMez8HC5QGUSVoK9wuAXKwk0ZDtFrDT8OSKBo6Z5lVh03yCVlsuRqBs+zlL0gQvYwT6kFy0y4AHtJ5230j0T50rsKFPI/BwQE74n0o2q3lmbhE9UoH+EAun/ZZe3WvQ0vaEcmLJ5+LgDA5tWeMf0KfnHWgjMwPB0svY1poqpL/+rCgU/7XQcHYXJvaSU2ZTfg/uIclTcbg/8QuhoOBQbEBI9/sdikmuz7kO1Iqzi/qZS1oxQtWYJQsa3hkU5j/rOgah3GtTVCuBGsdntte/nyzbWANjpusdqdXY6H6T6aoS/iGgS5kfLwZFbFeAY6QcAYG3agY176jVxbVql7epqmd3FvI+FE+ullLHJuCZPJ0NHpTevw+6A01zmuEewJo86QNziWHOLpSg4FwzrhZakfSWRTxjPOYMYHCHzru2NIWrhrfyZ0sCtcCaZOSm+M0jG4cztWSicc2ku54YxuVjMefK03RE6z3j+/P6XLoKL0827fmVew0pXfHuY8+7dQQ91+ZxXc0EsOu2/IUWr905LMuEeSL5IJwDGJ4BilvJAshaviZA+VtTreA3JRaE+vtq0zF+x8fuuqHZI0YGuJKAauFN1ULvS7UlGdwQgMnf5okiT75vumB097Mkqz8ULj7KzIUVpZodL/r8seL+fgI=",
"refresh_token": "M.C560_BL2.0.U.-CrVCPeraUsGVQFNR3kP7XEDmnLO2aKPbPbV7i8X6b4c5GIR3WazsxxrM048fZgxoRcdo8HJ2tgblxnKwfSdBBPHwCcUlMRN3Bso!1RHWffbI0CzI4RBA2hgJ4wXvx2xeoiUqTpLrE57YUz2ceWNmarnEES4y2qc2yofO!UlvHe6E1!CDX3YcDBqn5ochsKxUdU1YCWFQEahuldC3YJBgh5q7E!vGxDmRc!8ajimJL2DwY64f3qztaotABsiQk4yfn7wazFFx1MNQu1OXM0VUEVTkiiM4enI7G5QOuWbcpacK15SHCp0ZiMy01cLBlHVmNSBg7duxnApkhMYT2IdThtOv6mMIXJKx0FJrfWWuTfbCM!OxDrvSLe2c0A$$",
"expires_at":<PHONE_NUMBER>.6397078
}

Ok Roger,
I will do a clean install of the API registration and the integration and will give you feedback

Make sure the account you are creating the azure app with is the same one you are logging in with for authentication. To be honest you don't actually need to set up permissions on Azure, it will request them when you authenticate.

I’ve added diagnostics support in v0.1.0, assuming you have diagnostics turned on on your HA instance you can download a file showing your configuration. It lists the requested (required) permissions and the granted permissions (from your token file). All requested permissions must appear in the granted permissions list.

That is the diagnostic for HACS, not the calendar. But it may not be able to do the diagnostic until you get it installed properly.
If your token file doesn’t have the permissions in it, then it is likely the account you are authenticating with isn’t the one that you created the app with.

Back from a walk. I would logout of MS world before doing the authentication, to ensure you are using the right credentials.
If you really can't figure and need a screen share, let me know.

Hello Roger,
this weekend I tested with a completely new O365 account. Still the same error.
Googled and used chatgpt to serach for an solution.
maybe we can do an screen share.

Email me - firstnamesurname @ hotmail

Problem resolved via a screen share

Problem resolved via a screen share

Running into the same issue, what was the solution?

I don't remember the issue, but are you using a personal account and have filled in an email address into the shared email address box in the configuration?

Thanks for the quick response! Yes I am using a personal account and I have filled in an email address in the shared email address box.

The shared email address box is for entering the email address of an email account that has been 'shared' with you. As a personal account, nothing can be shared with you, therefore you shouldn't fill that box in.

Removed the email address from that box in the configuration, and everything is working like a charm now. Thanks a lot!


Initial structure for SharePoint. Provisioning will be merged from js-sites-core

Initial structure for SharePoint. Provisioning will be merged from js-sites-core (https://github.com/Puzzlepart/js-sites-core).
Just checking if basic structure is fine first.
Also added typings for SharePoint to typings.json.

Hi @olemp, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

Hello,
Could you switch from tsd.json to typings.json since TSD is deprecated and the project is already using typings.json?
Just run "typings install --save --ambient sharepoint" in the command prompt and you're all set.

@artokai Sure thing.

@artokai https://github.com/olemp/PnP/commit/7ff084c9562b14721e23e347c5a86fbccf1449a6
Looks good?

Thanks!


[Netty 5] Early data for TLSv1.3

TLSv1.3 supports the concept of early data. We should add support for it.

Is there anything special that needs to happen other than using Session Tickets (or IDs) in conjunction with TLS 1.3? I'm speaking in terms of server side, a client might require some extra work.

Adding link to this SSL_read_early_data for future reference.

 The TCN side is pretty straight forward. BoringSSL uses a slightly different API than OpenSSL but I believe Boring's API can be "massaged" to emulate the OpenSSL API/behavior. I have that stuff compiling and hypothetically working.
Now, SSLEngine#wrap/unwrap is where the skeletons lie. I'm afraid none of the state and logic flow hold in the context of early data. It'll take a few :wine_glass: to untangle it.

Is this in milestone for 5.0.0.Final?

@rkapsi Any updates?

Also, we need a way to notify SslHandler that TLS was established using the Early Data method. Because there are some weaknesses to this. Maybe something like SslHandler#isEarlyData would do the job.


Can you provide a training dataset?

This work is very attractive. Can I ask for a data set to try to reproduce the model of the paper, and make sure it is only used for academic research and not for commercial use?

Hi, thanks for your asking. I am sorry that I can not provide the training data due to license issues. I just released the code of the data generation pipeline here (https://github.com/ChenPaulYu/DJtransGAN-dg-pipeline) and a pre-trained model for you to try on. I hope it is helpful for you 😊.

Thank you!

I cannot find the pre-trained model, I might be looking in the wrong place though... Did you end up releasing it?


Add support for WinExe output type

Fixes #674
From https://msdn.microsoft.com/en-us/library/bb629394.aspx

OutputType

Specifies the file format of the output file. This parameter can have one of the following values:

Library. Creates a code library. (Default value.)
Exe. Creates a console application.
Module. Creates a module.
Winexe. Creates a Windows-based program.

Now only the module output type is not handled by Paket, it will be treated as library output

thanks for fixing this. :+1:

Purely a self serving act :stuck_out_tongue:

I on the other hand much appreciate the insanely quick turnaround time from PR to release :+1:


Recurring Donations: Next Donation Date should be greater than Date Established

Critical Changes
Changes
Ensure Next Donation Date is greater than Date Established. Thank you Thom Behrens for this contribution!
Issues Closed
Fixes #3829
New Metadata
Deleted Metadata

**lurch: attach W-029989

Tracking W-029989

@thombehrens , thank you for your contribution and fixing the issue!
Unit tests were added to ensure correct existing behaviour but most importantly that the bug is fixed. There are more changes than expected in addition to changing RD_ReccuringDonations.getStartDate() as per your PR since this was a good opportunity to update classes based on the latest coding standard. Please let us know if there are questions.

great!


SNOW-988153: Please consider the option to drop or remove a SPCS service

Description
When trying to deploy a service with a name that already exists it fails.

Can we add a snow container service drop to remove a service?
Can we add a snow container service deploy to drop and create a service?

Thanks!
Context
No response

Thanks, still having it in the "snow container service" area makes more sense I believe.

Closing because of snow object drop approach


Marco/upgrade faro to v1.4.0

Changes

Upgrade Faro to v1.4.0
remove experimental faro-instrumentation-performance-timeline. It was only used to track navigation and resource timings which is now built into Faro.

Merge Requirements
For new features contributions please make sure you have completed the following
essential items:

[x] CHANGELOG.md updated to document new feature additions
[ ] Appropriate documentation updates in the docs
[ ] Appropriate Helm chart updates in the helm-charts

Maintainers will not merge until the above have been completed. If you're unsure
which docs need to be changed ping the
@open-telemetry/demo-approvers.

 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.


fix: artwork location autosuggest placeholder copy (FX-2733)

This PR addresses FX-2733 by changing the copy of the placeholder text in the artwork location filter's search field from "Enter a location" to "Enter a city" because it only filters on city name and not on state or country names.

I'm self-merging this PR because it is a one-line copy change.


Export variants to CSV for gnomAD v4.1.0 (non-UKB) not working

I looked up KANSL1 in gnomAD v4.1.0 (non-UKB) https://gnomad.broadinstitute.org/gene/ENSG00000120071?dataset=gnomad_r4_non_ukb. I selected pLoF only variants and wanted to export variants to CSV.
Clicking the "Export variants to CSV" is not working and no CSV is produced. It does work for v4.1.0 but not for the non-UKB  version.

Hi @eva-mariastauffer , thanks for reporting this. I wanted to let you know that we have a fix under review for this so hopefully this functionality should work properly on the public site within the next few days.

Hi @eva-mariastauffer , the fix is deployed and I have verified that I can download the non-UKB variant CSV data from the gene in question. Thanks again for bringing this to our attention, please let us know if you have further trouble.


Chapter 11 Customer Model

Missing returns types for methods
exchangeArray
encrypt (method needs removed actually)
toArray
If you pass a flag that has a default, you could possibly send the password back as well, if you needed it.
toArray($filterPassword = true)

exchangeArray: added void as return type for abstract model and its childrens;
encrypt (method needs removed actually): if it needs to be removed, why to add, right?
toArray: the return type in exchangeArray is a useful change because documents the code. As Zend of Python says, explicit is better than explicit. But toArray is explicit and there are no complex possibilities to generate another type. i don't fell that control is strictly required here.

The recommendation was based solely for static analysis. No return type will cause an issue with nearly any tooling that is used.

The recommendation was based solely for static analysis. No return type will cause an issue with nearly any tooling that is used.

I will think about it when I am adjusting Chapter 12, because it has a little of static analysis. Maybe this could be inserted with another analysis tool.

It could actually just be a comment. Both phpstan and psalm will look for the actual return type, if none is found it will check the method doc comment block to see how it is detailed.
/**

@return array<string, mixed> $data
/
Just as a basic example


[MINOR] Bump R version to 2.3.0.

What changes were proposed in this pull request?
#17753 bumps master branch version to 2.3.0-SNAPSHOT, but it seems R version was omitted.
ditto of https://github.com/apache/spark/pull/16488
How was this patch tested?
N/A

Test build #78221 has started for PR 18341 at commit 5e1bd0a.

I found this when testing https://github.com/apache/spark/pull/18335. I thought it is odd that CRAN check generated SparkR_2.2.0.tar.gz in the master. I double checked it now generates SparkR_2.3.0.tar.gz.

cc @srowen and @felixcheung

Test build #78221 has finished for PR 18341 at commit 5e1bd0a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/78221/
Test PASSed.

+1 with what srowen says.
ditto with python actually https://github.com/apache/spark/blob/master/python/pyspark/version.py
i wonder if we should simply change the release/branch process to update everything

Test build #78233 has started for PR 18341 at commit 713b1b2.

Test build #78233 has finished for PR 18341 at commit 713b1b2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/78233/
Test PASSed.

Merged to master

Sure, I will do. Thank you.


COLOUR_MAP: extra shadows and strange tools

Hi! When i use COLOUR_MAP images with another player and tools in view I've got strange glitches:

Every player has additional square 'shadow' on the ground colored by players_color (marked '1' on image below). Could it be removed?
Every item (I checked some tools (pickaxes ...)  and some blocks (dirt, grass)) when alone (not in hand) has the same additional square player_color (?!) 'shadow' (marked as '2'). Can it be removed, please?
Looks like every tool (I checked torch, pickaxes, swords, shovels) represented on COLOUR_MAP images as two parallel squares (marked as '3' on image). It's not good at all. Could it have a more realistic shape?
(BTW, blocks looks ok (but with extra 'shadow' on ground - see 2.) I don't know about other items.)

Example COLOUR_MAP image with glitches (and my marks: 1, 2, 3):

The same image in RGB:

For 1 and 2, does it help to turn off entity shadows in the Minecraft GUI?
eg:

3 might be harder to fix, but if you have any knowledge of OpenGL shaders, the place to look is https://github.com/Microsoft/malmo/blob/master/Minecraft/src/main/resources/annotate.fsh
At the moment, the alpha channel is always set to 1.0 - hopefully it's possible to use alpha channel information from the original texture.

Can't check right now. I've found bypass solution for 1 & 2 (i took grass HSV mask from RGB frames). Everything else - later.

Back to problem. Davey, your advice to turn off entity shadows in Video settings works just fine. All shadows are gone including in COLOUR_MAP. But I can't solve problem '3', cause 'OpenGL shaders' is a dark matter for me. It would be nice if somebody could fix it, cause it really prevents work with COLOUR_MAP. Here is example how looks torch in hand and torch on the ground (in left lower corner):


[WIP] Remove uses of yaml-patch

The yaml-patch package is now abandoned. Replaced with gjson and sjson in appropriate places.
HIVE-2397

/hold

/remove-lifecycle rotten
@lleshchi bump

/hold After completing revisions, realized clusterpool controller and validating webhooks were not included in the original changes. Working on those now. We can wait for that work to be completed before merging this PR or create a seperate PR for those updates.

/hold After completing revisions, realized clusterpool controller and validating webhooks were not included in the original changes. Working on those now. We can wait for that work to be completed before merging this PR or create a seperate PR for those updates.

I'm fine doing it in a separate PR.
I'll review this tomorrow.

/hold After completing revisions, realized clusterpool controller and validating webhooks were not included in the original changes. Working on those now. We can wait for that work to be completed before merging this PR or create a seperate PR for those updates.

I'm fine doing it in a separate PR.

I take it back. I don't want to commit to this until we understand what it's going to look like for ClusterDeploymentCustomizations, which accept a very yamlpatch-ish object notation.

I take it back. I don't want to commit to this until we understand what it's going to look like for ClusterDeploymentCustomizations, which accept a very yamlpatch-ish object notation.

Summary of offline discussion: we're going to pivot to https://github.com/evanphx/json-patch instead of sjson. The delta will be much smaller, especially when we get to ClusterDeploymentCustomizations, and thus more provably correct/backward-compatible.

Closed in favor of #2498


Spaceless and style tags

Hi !
When using style attribute on a html tag, I expected that spaceless would remove useless line breaks on my styles but it preserves them.
Here is the code I used to test :
{% spaceless %}

        foo

{% endspaceless %}

And the result I get :

foo

Is it feasible to improve that behaviour ?
Thanks !

This would require knowing that spaces have no meaning inside a particular attribute (you cannot always remove them, and you cannot even remove all of them in styles but only duplicated ones). This would make the code far more complex and error-prone.

And is there a way to apply a "dumb" spaceless that doesn't care or content, that just replaces line breaks and tabs by spaces and removes every \s{2,} ?

if replace line breaks this code on js is broken:
x = 5 + 2
y = 3 + 2

Yes what if you could use something like :
{% spaceless %}

        foo

{% endspaceless %}

As the documentation states, the spaceless tag only removes whitespace between HTML tags.
I have tweaked the documentation to make it clearer (see 01f733f4b54).


New Release

A new release is needed due to the merge of #3  and #5 fixing breaking issues.

Thanks, we're preparing a release now.

v0.5.2 is released


feat: [dropgz] Dropgz for windows

Dropgz support for windows.
Reason for Change:

Issue Fixed:

Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[x] relevant PR labels added

Notes:

Curious how this will work? I remember from a couple years back that you couldn't hot-swap Windows CNI because containerd required a reboot to pick up the new bin. Has that changed, and hot swap of CNI is possible on Windows now?

Curious how this will work? I remember from a couple years back that you couldn't hot-swap Windows CNI because containerd required a reboot to pick up the new bin. Has that changed, and hot swap of CNI is possible on Windows now?

I am not sure about the containerd reboot requirement. We were updating the binaries on the windows on cluster and testing it earlier as well. We just added the same functionality through dropgz.
So earlier even if we replaces the binaries, the old binaries were getting picked up  ?

@thatmattlong @rbtr Can i please get a review on this ? ( as a part of code owners )


Drag and drop files

Enable to drop mvx files into pipeline editor, and convert them into mvx source node

Fixed as of acbe8011b54b76ad5df9c3bf1c466ec7767e7e6a


Flappy bird can bypass pipe over screen

It really messed up my research with DQN RL variations, because it is possible to bypass pipe over the screen.
See screenshot: http://www.evernote.com/l/AClrdjOMAFBLSLm1-ZvYqa-63AOtriZ9NoI/
This state gives reward = 1.0 (notice bird partially over pipe)
player height 24, but it is enough to be at position -20 to bypass pipe.
Can you please give me rights to push patch to repo? I am planning to work with your repo and I could find more bugs.
Changes needed:
--- a/ple/games/flappybird/__init__.py
+++ b/ple/games/flappybird/__init__.py
@@ -423,7 +423,7 @@ class FlappyBird(base.PyGameWrapper):
             self.lives -= 1

         # went above the screen
-        if self.player.pos_y < -self.player.height:
+        if self.player.pos_y < 0:
             self.lives -= 1

PR #29 fixed this.


Feature Request: Renew Token Key

Hi
It would be nice if there was an option like on consul-template to renew the token (once we have manually create it for the first time). Just an idea.
Thanks
Joaquin

With HashiCorp adding an agent mode to Vault this and support for other login mechanisms for Vault are likely to become a non-issue.
It's then the Vault Agent's job to take care of this.

Thx for the info @fuero Did the vault agent config and it is working nicely, so there is no reason for this issue to still be open.


TextComponent colors do not accept uppercase color codes (eg #FFFFFF)

I noticed this was only happening when the first character after the # was a capital letter.

https://github.com/SPGoding/datapack-json/blob/dc01940fd05e743f04e88875ea1e1f83f9c6e6f1/src/shared/text_component.json#L358
"pattern": "#[0-9a-f]{1,6}"

Just change it to the following should be fine?
"pattern": "#[0-9a-fA-F]{1,6}"


WebSocket server error: Port is already in use

Hi!
Thanks a lot for your work! But I meet strange behavior for a new app - when I start the app, I see message WebSocket server error: Port is already in use. What's wrong?
It looks like there is a conflict between astro check --watch and astro dev - when I run those commands separately, I see no error message

I believe it's an issue in this line
    "dev": "astro check --watch & astro dev",

Instead of one ampersand(&) it should be two (&&)
    "dev": "astro check --watch && astro dev",

Hi, I looked into this a bit and this doesn't work on my end.
I found that this line of code was from this blogpost
From what I understand, the one ampersand symbol (&) will run the two commands in parallel in which they will use the same port.
I'll update this if I find any information that can get rid of error.

I found the cause of this problem and it is related to the commit 4fe5084.
Turns out, for some reason, if I add site URL from src/config.ts, "Port is already in use" problem occurs. I'm not really sure why tho. It seems like there might be an unexpected interaction or conflict with the way the site URL is configured.
I'll make a commit to resolve this issue.
Thanks everyone. ✌🏻


Install Openswoole

Hi, I'd like to install openswoole with an own Dockerfile, but i couldn't it to enable.
My Dockerfile
FROM tiredofit/nginx-php-fpm:debian-8.2

# additional ENV vars
ENV NODE_VERSION=20.5.1
ENV NVM_DIR=/root/.nvm

# Install NVM for node
RUN curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.39.3/install.sh | bash
RUN . "$NVM_DIR/nvm.sh" \
    && nvm install ${NODE_VERSION}
RUN . "$NVM_DIR/nvm.sh" \
    && nvm use v${NODE_VERSION}
RUN . "$NVM_DIR/nvm.sh" \
    && nvm alias default v${NODE_VERSION}
ENV PATH="/root/.nvm/versions/node/v${NODE_VERSION}/bin/:${PATH}"

# Install ffmpeg, npm and yarn
RUN curl -sS https://dl.yarnpkg.com/debian/pubkey.gpg | apt-key add -
RUN echo "deb https://dl.yarnpkg.com/debian/ stable main" | tee /etc/apt/sources.list.d/yarn.list

RUN apt update \
    && apt install vim ffmpeg nodejs yarn g++ make -y

RUN ln -s /bin/sed /usr/bin/sed

RUN php-ext enable sockets \
    && php-ext enable redis \
    #&& php-ext enable calendar \
    && php-ext enable zip \
    && pecl channel-update pecl.php.net \
    && pecl install --configureoptions 'enable-http2="yes" enable-openssl="yes"' openswoole-4.12.0 \
    && php-ext enable openswoole

#RUN echo 'extension=openswoole' >> $(php -i | grep /.+/php.ini -oE)
COPY php/openswoole.ini /etc/php/8.2/mods-available/openswoole.ini

The file openswoole.ini:
[Openswoole]
extension=openswoole

But if I'm build und start it, openswoole are not available. How can i enable it or is it possible to add it on your image?
Thanks for your great work ;-)

Oh interesting, you certainly figured out how to get things in there, so I'm surprised it's not working.
When you boot up your container, can you tell if it is loaded or not? (hint: look in /etc/php/*/conf.d and look for a symbolic link to look at mods-available.).
One thing I would add to your ini is the following:
;priority=00     ## Or 10 or 20

And then finally, in your Docker file would be to add the environment variable of PHP_ENABLE_OPENSWOOLE=TRUE which on container startup will enable it each time it starts. Let me know how that works for you.

Hm, unfortunately not work only with PHP_ENABLE_OPENSWOOLE=true
The solution are to add RUN phpenmod openswoole on the end of the Dockerfile. So in fact:
FROM tiredofit/nginx-php-fpm:debian-8.2

# additional ENV vars
ENV NODE_VERSION=20.5.1
ENV NVM_DIR=/root/.nvm

# Install NVM for node
RUN curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.39.3/install.sh | bash
RUN . "$NVM_DIR/nvm.sh" \
    && nvm install ${NODE_VERSION}
RUN . "$NVM_DIR/nvm.sh" \
    && nvm use v${NODE_VERSION}
RUN . "$NVM_DIR/nvm.sh" \
    && nvm alias default v${NODE_VERSION}
ENV PATH="/root/.nvm/versions/node/v${NODE_VERSION}/bin/:${PATH}"

# Install ffmpeg, npm and yarn
RUN curl -sS https://dl.yarnpkg.com/debian/pubkey.gpg | apt-key add -
RUN echo "deb https://dl.yarnpkg.com/debian/ stable main" | tee /etc/apt/sources.list.d/yarn.list

RUN apt update \
    && apt install vim ffmpeg nodejs yarn g++ make -y

RUN ln -s /bin/sed /usr/bin/sed

RUN php-ext enable sockets \
    && php-ext enable redis \
    && php-ext enable zip \
    && pecl channel-update pecl.php.net \
    && pecl install --configureoptions 'enable-http2="yes" enable-openssl="yes"' openswoole-4.12.0 \
    && php-ext enable openswoole

COPY php/openswoole.ini /etc/php/8.2/mods-available/openswoole.ini
RUN phpenmod openswoole

EXPOSE 8080

Thanks for your great work


Start adding authentication system.

This PR adds a command-line program that is able to add an Admin user.
It adds the Admin user to the database, and then sends them an email they can use to login.
The functionality of actually logging in the admin user is not implemented yet.
@arowM @kayhide Please review.
The main files for reviewing are app/AddAdmin.hs, src/Kucipong/Db/Models/EntityDefs.hs,  src/Kucipong/Email.hs, and src/Kucipong/Monad/Db/Instance.hs.

LGTM

OK 👍
コミットがきれいに分かれていてわかりやすいです 😆


Add static Screens

We are needed to add some static screens to complete the Dashboard Screen,
provide some options for the adaptive screens,
and finally some extra routes for the navigation.

Search Screen
Notification Screen
Settings Screen
Help Screen

Done


Added Xiaomi 2201117TI to devices that does not support H.264

some additional concerns with this approach.
@davidliu, I believe the original issue is with publishing, but what about subscribe? Could this device decode a H.264 stream sent by another party?
The current change / device configuration will effectively disable downstream negotiation too. I think in some cases, we'd want to disable publishing of a certain codec (due to hw encoder issues), but we do not want to disable negotiating that codec on the subscriber side (since the tracks are already encoded as such)

AFAIK, these devices can decode H264 fine, it's specifically when trying to simulcast encode H264.

Sounds like we need separate publish/subscribe directions from DisabledCodecs, and in most case we only need control publish direction as hw decoder works more reliable than encoder, also webrtc has software decoder fallback .

@cnderrauber @boks1971 I've split publish and subscribe control for disabled codecs, lmk if this works better.

It works for me, in most cases we only need set Disabled.Publish


When closing app on Mojave getting error message

Whenever closing the app on macOS Mojave, an error message is shown that "SoundCleod stopped working unexpectedly".

Looks like this: https://github.com/electron/electron/issues/8459
I ran into this while looking at updating to electron 3 (cpu usage way down!).

I’m on my way back from holidays, will hopefully be able to address Mojave bugs in a couple of days. #solomaintainer

On 1. Oct 2018, at 01:38, emadg<EMAIL_ADDRESS>wrote:
Whenever closing the app on macOS Mojave, an error message is shown that "SoundCleod stopped working unexpectedly".
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

A more recent upstream issue: https://github.com/electron/electron/issues/14837

Created a pre-release with latest Electron here: https://github.com/salomvary/soundcleod/releases/tag/v1.4.1-pre.0
As I have not upgraded to Mojave yet, please test and send feedback whether this version fixes the problem.

Created a pre-release with latest Electron here: https://github.com/salomvary/soundcleod/releases/tag/v1.4.1-pre.0

Nope, still broken.

Also, worth pointing out:

#solomaintainer

I think you're doing an excellent job.

Thanks, trying to do my best:)
Thanks for trying, will wait for an Electron update then...

FWIW, I've tested electron 4 and ran into the same issue. This actually made me revert back to the webview branch. Looks like cpu usage (at least on macos) is equal to electron.

See #174 for workaround.

Should be fixed in 2.0.0.


Fix spelling of 'linear' in linearRegression doc description

Codecov Report

Merging #2341 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2341   +/-   ##
=======================================
  Coverage   79.55%   79.55%
=======================================
  Files          85       85
  Lines        8863     8863
  Branches     1899     1899
=======================================
  Hits         7051     7051
  Misses       1551     1551
  Partials      261      261

Impacted Files
Coverage Δ

webapp/graphite/render/functions.py
96.22% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks @stembrain 👍


Disable an index : "Disable" menu item always greyed out when table opened in Design

For the "Using SQL Server Management Studio / To disable an index" section, it should be mentioned that if the table is opened in Design mode, the "Disable" context-menu item will be greyed out even if the index is enabled.
SQL Server Management Studio v15.0.18390.0
Windows 10 Pro v10.0.19043

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

ID: a2bb45cd-9675-aa90-9ca2-c27774d85e3f
Version Independent ID: 7ca9be50-df61-4fa3-9254-7879c3b6d120
Content: Disable Indexes and Constraints - SQL Server
Content Source: docs/relational-databases/indexes/disable-indexes-and-constraints.md
Product: sql
Technology: table-view-index
GitHub Login: @MikeRayMSFT
Microsoft Alias: mikeray

@goozak -- Bruno, thank you for your feedback.
@MikeRayMSFT -- Mike, please look into this issue.

Fixing in https://github.dev/MikeRayMSFT/sql-docs-pr/tree/20230214-issue-7168
#please-close


Is it possible to make DNSCrypt for Android works with x86 processor?

The current version of DNSCrypt for Android is only complied for ARM processor. But there is some x86 devices uses Android, and some people have made Android run on x86 devices (Android-x86), thus making DNSCrypt has limited usage.

The dist-build/android-x86.sh and dist-build/android-x86-64.sh build scripts (also present in libsodium) are available for that purpose.
Pre-built binaries are not available because running Android on x86 is still relatively rare.


⚠ ClamAV has degraded performance

In 9727416, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 566 ms

Resolved: ClamAV performance has improved in 88b5465.


Implement possible EDC adjustments

Description
https://jira.catena-x.net/browse/CXAR-1081
Impact

Additional information

[ ] I'm willing to contribute to this feature

Not relevant anymore - closing it, aligned with expert group.


Loader for Python modules

Load Python modules as Jinja2 templates.
        >>> loader = PyModuleLoader()

This loader returns an empty template exporting Python module's namespace. It piggybacks on the Jinja2 import mechanism to provide a convienient way to access functionality from any Python module in sys.path.

What's the benefit of it?
Without an example (your unit test doesn't qualify IMO) it reminds me of this:

I really don't think that exposing the whole python library via a template loader is even close to a good idea, even less something that should be in the Jinja core

The main motivation is to be able to easily import additional functionality from Python modules using the same interface as for importing variables/macros from other templates, e.g.:
{% from "itertools" import izip %}

My particular use case is templating configuration files of Cylc workflow engine. To use Cylc users are generally assumed to already have full shell access, so exposing whole Python library is really of no concern. Cylc already exposes adding user defined Jinja2 filters, but it's tedious to write filter wrappers for every little scrap of functionality.
From your comment it's quite clear that you're not very keen on the idea, so arguing my point is most likely moot. But let me just point out that Jinja is used also in systems, where tight control over exposed functionality is not required, so I think this PR could be of broader appeal than my particular use case. Anyway, I rest my case ;-) Feel free to reject this PR if you think it's a bad idea.

I think it makes more sense to add this loader in your application then.
FYI, if you want to expose modules, you can pass e.g. itertools=itertools as a kwarg when rendering the template.

This is clever, but it seems way too easy for people to shoot themselves in the foot with it. I'm sure people would take advantage of a jinja-import-loader if you released this separately. It doesn't look like there's anything here that would benefit from being in core.

I would take advantage of a jinja-import-loader in my project (yasha).


Considering a rename to elm-form

As this package isn't really targeted for simple forms, and offers at the same time form building and validation, I'm tempted to do this rename.
(I'm aware of the procedure to avoid broken builds)
WDYT?

When I first saw this package, because of the "simple", I had some prejudice. But at that time, my project was also simple, so I used it.
As my project grow (10.000+ lines of Elm), this package is still robust and flexible enough for all my form needs. My project makes heavy use of forms (20+ different forms) and I am grateful for had choosing to use this package.
So, yeah, I really like this idea!

The only reason not to do it is relative to the technical consequences of an Elm package renaming (package users will have to reflect this chance in their elm-package.json, etc), otherwise I'm 200% for this  change, and it's of course not bound to the fact I am one of the folks who suggested it to you :-)

Done! (but package not published yet)


Cleanup HostingContext

Some of the HostingContext don't appear to be required, such as services.
We should clean it up.

Let's do this part of our API reviews.


add EventCounter algorithm

BEGINRELEASENOTES

Added thread-safeEventCounter algorithm

ENDRELEASENOTES
Adding thread-safe version of EventCounter algorithm. The algorithm will print current event number (as seen by the scheduler) depending on the frequency. Counting the total number of events passed through the algorithm is done with Gaudi::Accumulators::Counter - add MessageSvcSink in order to get a printout about all the Gaudi counters, for instance:
26: ApplicationMgr       INFO Application Manager Initialized successfully
26: ApplicationMgr       INFO Application Manager Started successfully
26: EventCounter         INFO Processing event 0
26: EventCounter         INFO Processing event 2
26: EventCounter         INFO Number of counters : 1
26:  |    Counter                                      |     #     |    sum     | mean/eff^* | rms/err^*  |     min     |     max     |
26:  | "count"                                         |         4 |
26: SilentEventCounter   INFO Number of counters : 1
26:  |    Counter                                      |     #     |    sum     | mean/eff^* | rms/err^*  |     min     |     max     |
26:  | "count"                                         |         4 |
26: ApplicationMgr       INFO Application Manager Stopped successfully
26: ApplicationMgr       INFO Application Manager Finalized successfully
26: ApplicationMgr       INFO Application Manager Terminated successfully

Closes #219

Thanks a lot for this PR!
Would it be possible to add at finalize a simple printout of the form
EventCounter         INFO Processed XX events
before the (optional) printout of the tables?
How can one configure the EventCounter to avoid showing the tables BTW?
Thanks again!

Thanks for the comments. Added finalize - the message seems to be printed after the tables (updated example in the description)

How can one configure the EventCounter to avoid showing the tables BTW?

To avoid printing tables for counters don't add MessageSvcSink to ExtSvc (or change the MessageSvcSink property to skip printing tables for all the counters)

Thanks @m-fila ,
that looks good to me!
Giovanni


why isn't possible to scroll anymore in a linux screen

Windows Terminal version
1.17.11461.0
Windows build number
10.0.19045.3208
Other Software
No response
Steps to reproduce

Connect to a Server over SSH
Create a screen (in Linux) = screen -S Name
Then go into a screen and create outputs or something like that where you must scroll

Expected Behavior
i use the version v1.12.1073 because there wasnt the problem and we could scroll egal no matter how much the output was.
Actual Behavior
with the new version i cant scroll in a screen. I have try everything but nothing helps me.

This is behavior that's fairly highly preserved across terminal emulators. I wasn't sure that there were any that let you scroll the alt screen buffer.
Prior to #12561 (1.14), you could seemingly scroll the "alt buffer" in Terminal, but that was just because Terminal didn't actually support the alt buffer yet 😅
I know it's possible to configure tmux to scroll it's buffer with the mouse, but I'm not so sure about screen.


Auth user for getting deployment statuses

The command to list deploys wasn't working for us as the github auth token wasn't being set.
Apologies if I've overlooked something, otherwise hopefully this is a quick fix.

npm now has version 0.6.52. Thanks.

This command is failing for me with the following message:
[Tue Jun 23 2015 00:51:48 GMT+0000 (UTC)] ERROR TypeError: Cannot read property 'user' of undefined
  at LatestFormatter.message (/app/node_modules/hubot-deploy/src/formatters.coffee:39:25, <js>:76:42)
  at /app/node_modules/hubot-deploy/src/scripts/deploy.coffee:56:9, <js>:48:37
  at /app/node_modules/hubot-deploy/src/deployment.coffee:95:7, <js>:132:16
  at Client.errorHandle (/app/node_modules/hubot-deploy/node_modules/octonode/lib/octonode/client.js:164:14)
  at Request._callback (/app/node_modules/hubot-deploy/node_modules/octonode/lib/octonode/client.js:178:22)
  at Request.self.callback (/app/node_modules/hubot-deploy/node_modules/octonode/node_modules/request/request.js:129:22)
  at Request.emit (events.js:98:17)
  at Request.<anonymous> (/app/node_modules/hubot-deploy/node_modules/octonode/node_modules/request/request.js:873:14)
  at Request.emit (events.js:117:20)
  at IncomingMessage.<anonymous> (/app/node_modules/hubot-deploy/node_modules/octonode/node_modules/request/request.js:824:12)
  at IncomingMessage.emit (events.js:117:20)
  at _stream_readable.js:944:16
  at process._tickCallback (node.js:448:13)

Which chat network are you using?

Slack.

Can you try the 0.6.53 release?

Sure, I'll try it tomorrow.
On Tuesday, 23 June 2015, Corey Donohoe<EMAIL_ADDRESS>wrote:

Can you try the 0.6.53 release?
—
Reply to this email directly or view it on GitHub
https://github.com/atmos/hubot-deploy/pull/46#issuecomment-114340005.

Works perfectly :+1:


#15777 Throw exception on invalid quote sequence for csv import

Please, test for different cases.
Should throw with this data when \ is an escape char in import preferences
1,"ABC ""Z"" MTK",4
2,"AR ""Z"" MTK\",5
3,"AB ""Z"" MTK",6

Verified


on exporting pdf page to image, ( images inside pdf ) not displaying in the output image in ubuntu.

Exporting pdf to images, image inside the PDF is not displaying, refer the attached PDF.
compressed.tracemonkey-pldi-09.pdf in PDF second page as image, this image is not displaying on Ubuntu but shows well in windows 10 same node and pdf.js version.
Configuration:

Chrome, firefox...
OS: Ubuntu 16.04
pdf.js ("pdfjs-dist": "^2.1.266"), nodejs (10.5.0)

var loadingTask = pdfjsLib.getDocument("compressed.tracemonkey-pldi-09.pdf");
loadingTask.promise.then(function(pdfDocument) {
	console.log('# PDF document loaded.');
	main (2)
	async function main (i) {
		const page = await pdfDocument.getPage(i);
		var viewport = await page.getViewport({ scale: 1.0, });
		var canvasFactory = new NodeCanvasFactory();
		var canvasAndContext =
		  canvasFactory.create(viewport.width, viewport.height);
		var renderContext = {
		  canvasContext: canvasAndContext.context,
		  viewport: viewport,
		  canvasFactory: canvasFactory,
		};

		var renderTask = await page.render(renderContext);
		//var base64 = canvasAndContext.canvas.toDataURL('image/jpeg');
		var image = canvasAndContext.canvas.toBuffer();
		await fs.writeFile('output'+i+'.jpeg', image, function (error) {
		    if (error) {
		      console.error('Error: ' + error);
		    } else {
		      console.log(
		        'Finished converting first '+i+' of PDF file to a PNG image.');
		    }
		});
	}
}).catch(function(reason) {
  console.log(reason);
});

function NodeCanvasFactory() {}
NodeCanvasFactory.prototype = {
  create: function NodeCanvasFactory_create(width, height) {
    assert(width > 0 && height > 0, 'Invalid canvas size');
    var canvas = Canvas.createCanvas(width, height);
    var context = canvas.getContext('2d');
    return {
      canvas: canvas,
      context: context,
    };
  },

  reset: function NodeCanvasFactory_reset(canvasAndContext, width, height) {
    assert(canvasAndContext.canvas, 'Canvas is not specified');
    assert(width > 0 && height > 0, 'Invalid canvas size');
    canvasAndContext.canvas.width = width;
    canvasAndContext.canvas.height = height;
  },

  destroy: function NodeCanvasFactory_destroy(canvasAndContext) {
    assert(canvasAndContext.canvas, 'Canvas is not specified');

    // Zeroing the width and height cause Firefox to release graphics
    // resources immediately, which can greatly reduce memory consumption.
    canvasAndContext.canvas.width = 0;
    canvasAndContext.canvas.height = 0;
    canvasAndContext.canvas = null;
    canvasAndContext.context = null;
  },
};

Any suggestions?

Most likely duplicate of #10992.


stm32_serial.c: Don't fake a TX interrupt when interrupts are not suppressed

Summary
Don't fake a TX interrupt when interrupts are not suppressed.  We have already enabled interrupts, we can just wait for the next TX interrupt.
Impact
STM32 Serial driver
Testing
olimex-stm32-p407:nsh

@davids5 I've tried this when I was working on https://github.com/apache/incubator-nuttx/pull/4403#issuecomment-905019719, can you please check?

@Ouss4 - I want to say that the call needs to happen to re-enable flow control.  Did you test with HW flow control?

@davids5 Thanks for the comment.  I tested (albeit very quickly) with hardware flow control and I didn't see any issue.  Actually I don't see how this recursive call re-enables hardware flow control.

@Ouss4
I suspect ( I need to diagram it out) the case is for a resume of TX after a -HS in the case the user is not writing any more data (acknowledged protocol, sizeof(remote RX buffer) < sizeof(local TX buffer). The first user write loads and triggers the successive TX and TXE until  the -HS case. TXE does not enqueue again.  So the call is made to kick off the restart.

Thanks for these details @davids5.
I didn't attach an LA to look at what's going on with HW flow control enabled either (my stuff are kind of a mess right now), I might do that later.  However I tried to find when this was introduced and try to find any hint in the commit message or the change itself.  It came as part of https://github.com/apache/incubator-nuttx/commit/4494689a2220501f74404e68b3eb72c112c9a888#diff-867b440e179e185b86ca1b3423ad19b5174ec9345a02380733d51f437a40db58R691-R700
Since everything is folded, let me copy the relevant snippet:
#ifndef CONFIG_SUPPRESS_SERIAL_INTS
//      priv->im |= ;
      up_serialout(priv, stm32_USART_IM_OFFSET, priv->im);

      /* Fake a TX interrupt here by just calling uart_xmitchars() with
       * interrupts disabled (note that the may recurse and that priv->im
       * may be altered.
       */

      uart_xmitchars(dev);
#endif

My guess is since setting the interrupt bits was commented out, faking it was necessary.
This was completed in https://github.com/apache/incubator-nuttx/commit/a7942e1bbe734f9bd7a4b737a899fa880ef66a88, but (again my guess) faking the interrupt part was not removed (i.e. adding an "#else was forgotten).  At that point flow control was not implemented in STM32.
I don't know if later this part was deemed necessary when flow control was added.

Thanks for these details @davids5.
I didn't attach an LA to look at what's going on with HW flow control enabled either (my stuff are kind of a mess right now), I might do that later. However I tried to find when this was introduced and try to find any hint in the commit message or the change itself. It came as part of 4494689#diff-867b440e179e185b86ca1b3423ad19b5174ec9345a02380733d51f437a40db58R691-R700
Since everything is folded, let me copy the relevant snippet:
#ifndef CONFIG_SUPPRESS_SERIAL_INTS
//      priv->im |= ;
      up_serialout(priv, stm32_USART_IM_OFFSET, priv->im);

      /* Fake a TX interrupt here by just calling uart_xmitchars() with
       * interrupts disabled (note that the may recurse and that priv->im
       * may be altered.
       */

      uart_xmitchars(dev);
#endif

My guess is since setting the interrupt bits was commented out, faking it was necessary.
This was completed in a7942e1, but (again my guess) faking the interrupt part was not removed (i.e. adding an "#else was forgotten). At that point flow control was not implemented in STM32.
I don't know if later this part was deemed necessary when flow control was added.

I do not remember :(
I want to test it, I just could not get to it today. I think it is a simple as determining if the IRQ is a level or an edge.
HS CTS__________--------------
TXE_______________---------------
TXEI set before_-------------  > the TXE edge or level will be seen for sure.
TXEI___________________------------- This the case I question. If the enable changes after the HS allows the DR->TXS will the IRQ get seen. If so the fake is not needed.

@Ouss4 I am sorry for the delay. I had to clear my bench and my mind. :)  I have proved that the ISR will happen (It is a level)
and the TXIE enable after the HS going true will cause the ISR. So the PR is correct.  I tested this on the F4 and F7. I have not tested it on the H7, but I expect it will be the same.

@davids5 thanks for the tests, much appreciated.  I too had a hectic week, so couldn't do anything else to confirm.
The PR isn't a draft anymore.