Lawrencemm conan

Retry #6 after CI fixing.

@Lawrencemm I just pushed some tweaks to the conan recipe, do they look alright to you? I separated them into multiple commits so that intention behind each change is clear from the commit message.
The output of conan create . demo/testing had the following warnings:
[...]
scope_guard/0.2.4-dev.1@demo/testing: WARN: Forced build from source
[...]
scope_guard/0.2.4-dev.1@demo/testing: WARN: This conanfile has no build step
[...]
CMake Warning:
  Manually-specified variables were not used by the project:

    CMAKE_EXPORT_NO_PACKAGE_REGISTRY
[...]

Any idea how to remove them?

I think a good idea would be to get this package reviewed before merging the PR by someone from conan-center.
Do you have a bintray account?
If you upload the package you can go here: https://bintray.com/beta/#/conan/conan-center?tab=packages
And click "Actions" then "Include my package".
Select the package you uploaded and in the "Additional Comments" add a link to this PR. They should tell us whether these warnings are anything to worry about and anything else we need to do to get the package included.


[lsp] Implement rulerefs in rego

This implements the rule_refs provider in Rego. This involves using a shared inmem rego store to allow the parsed modules for all workspace files to be accessible when running the new Rego-based policy provider.

Thanks for the review, will patch up when I'm back on tomorrow morning but I've also added 694ea52c457b33ce5ce751f4bec3211639e5f231
which does a refactor to migrate the locals provider to source the ast from the store too. Lmk if that looks good too and we can get this in asap.


issue an internal error

Hello i folowed the tutorial to make a portal but when i try to enter the command /mvp create swamp w:swamp   the server answer me :  An internal error occured while attempting to perform this command  i attached the log i could find in my console
full logs.txt
I m using Spigotmc-1.8.8-R0.1
Thanks for answering

Wrong WorldEdit version, update Multiverse-Core and Multiverse-Portals to the latest along with WorldEdit. Your worlds.yml is corrupt/not configured properly. It has YAML parsing errors.

i dont know what is YAML parsing errors i m french btw but i ll search about it  thanks for your help  , but i only have to  update world edit ? or  finding the version which match with each other ? i mean cant i take the  last version of WorldEdit and multiverse-Core + Multiverse-Portals
Thanks you again for your help

Hum i didn't understand first time i read that xd sorry  i m right this : http://dev.bukkit.org/bukkit-plugins/worldedit/files/61-world-edit-6-1-up-to-mc-1-8-8/ is the last version of world edit and same for  this : http://dev.bukkit.org/bukkit-plugins/multiverse-core/files/23-2-4-ab/ for multiverse core ? and i suppose this one for multiverse-portals ; http://dev.bukkit.org/bukkit-plugins/multiverse-portals/
i'm relatively new  at creating server , sorry for being a noob at  this thing ^^

Download files from http://ci.onarandombox.com/view/Multiverse/ and WorldEdit from http://builds.enginehub.org/job/worldedit/8533/download/worldedit-bukkit-6.1.1-SNAPSHOT-dist.jar

hum  WorldEdit worked but i have a new problem with Multiverse i downloaded Multiverse-Core-2.5-javadoc.jar	 and Multiverse-Portals-2.5-javadoc.jar but maybe was it the wrong file it took me time to figure where to download the file so the console say to me that multiverse couldn't be loaded

It worked thanks you a lot i could create my portal :D

Glad to hear :)


chore(theme): remove fontawesome dependency and replace with IBM icons

Description
I replaced all the fontawesome icons with IBM icons. One issue I need a little assistance on is getting rid of these dependencies found in the yarn.lock:

The only one I got rid of properly was the one named fontawesome-free. It's gone from the package.json in /webapp and the yarn.lock. These other ones I can't get rid of using yarn remove - they are only found in the yarn.lock not in any other package.json. I also tried doing another yarn install but they are still there. I am kinda doubting that it's appropriate to just erase these lines from the yarn.lock.
Resolves #64
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide reproduction instructions and list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

The only one I got rid of properly was the one named fontawesome-free. It's gone from the package.json in /webapp and the yarn.lock. These other ones I can't get rid of using yarn remove - they are only found in the yarn.lock not in any other package.json. I also tried doing another yarn install but they are still there. I am kinda doubting that it's appropriate to just erase these lines from the yarn.lock.

It is because it is imported by Facets. So don't worry, as long as you removed it from our own code.

You should not be modifying the lock file manually (99.9% of the time). The Yarn install should be updating that file automatically and in the rare cases due to big changes you can remove the lock file completely and then have it regenerated by yarn on install again.


Improve error handling when rendering diagnostics.

We have lots of unwraps and fallible slice accesses in the diagnostic rendering at the moment that could result in user-facing panics when calling codespan_reporting::term::emit. Most of these panicking calls are currently localised to codespan_reporting::term::views.
The panics could result from invalid file IDs, or invalid line ranges, say if an incremental compiler updated the underlying file before an error has been printed.
What should we do in this case? Should we panic? Return a richer error enum than a std::io::Error? Render something to the console?
Related: #140

I think it's fine to panic as long as the panics are documented (including things like a slice that's not on a UTF8 code point boundary, c.f. https://github.com/jyn514/rcc/issues/265).

The feedback I've got in private is:

…crashing limits the usability of your crate. If someone wants to use it in a place where the application is high availability and/or interactive, it's going to cause issues. I also noticed that catching panics as they unwind isn't 100%. Seems like async/await, at least initially, caused panics to not be catch-able.

So we might want to think about ways to provide a non-panicking API.

FWIW, I ran into one of these in prop stemming from codespan_reporting::term::emit, if it helps, you can find the test case here: tests.rs cargo test bad_unicode -- --nocapture
error: Invalid token

   ┌── foo:1:1 ───
   │
 1 │ ₐₑₒₓₔₕₖₗₘₙₚₛₜ₀₁₂₃₄₅₆₇₈₉ ≔ ⊤
thread 'tests::bad_unicode' panicked at 'byte index 1 is not a char boundary; it is inside 'ₐ' (bytes 0..3) of `ₐₑₒₓₔₕₖₗₘₙₚₛₜ₀₁₂₃₄₅₆₇₈₉ ≔ ⊤`', src/libcore/str/mod.rs:2069:5

With the relevant part of the backtrace being:
  18: codespan_reporting::term::renderer::Renderer::mark_single
             at .../github.com-1ecc6299db9ec823/codespan-reporting-0.9.1/src/term/renderer.rs:438
  19: codespan_reporting::term::renderer::Renderer::render_source_line
             at .../github.com-1ecc6299db9ec823/codespan-reporting-0.9.1/src/term/renderer.rs:277
  20: codespan_reporting::term::views::RichDiagnostic<FileId>::render
             at .../github.com-1ecc6299db9ec823/codespan-reporting-0.9.1/src/term/views.rs:230
  21: codespan_reporting::term::emit
             at .../github.com-1ecc6299db9ec823/codespan-reporting-0.9.1/src/term.rs:96
  22: prop::test_util::expect_fail

@ratmice I think you need to pass a byte index, not a char index. Passing an index to the start of the second char (3) would have the behavior you requested.

@jyn514 thanks that helps, I pushed a fairly terrible fix afaict though I have to start iterating the whole source from the beginning :dagger: to extend the range to a .char_indices boundary that overlaps it.
I'll have to see if I can get the lexer to pass return .char_indices() rather than the current Range(0, 1) when it fails.

Wondering if this would be better off as a separate issue? But yeah, that is a frustratingly bad workaround - I would have expected LALRPOP's built-in lexer to always return positions at a valid byte boundary.
Do note that Rust has str::is_char_boundary, which would reduce the amount of iteration you need to do.

One thing we could do though is to always slice along valid character boundaries internally in the renderer.


[Feature request] list of notify_service

Hi there,
thank you for this add on!
Will you be able to add feature to pull list of notify services from configuration
Now it seems to get only one?

use a Notify Group from home assistant :)
I think it makes more sense to do the grouping in home assistant. one advantage is that home assistant and NotiFreeze know of the group. another would be that you probably will use the same group for other things (in home assistant or other appdaemon apps) which is most convenient when centrally managed (by home assistant).
if you have good reason to do it another way, let me know, maybe I just never thought of :D


Fix some alignment issues with custom text field

+1

Hello,
I'm out of the office on annual leave, back Monday 4th July.
For all enquiries, please don’t hesitate to get in touch with the team on 01962 809 103 or
Sam Hibberd
<EMAIL_ADDRESS>Sean Hill
<EMAIL_ADDRESS>Sam Corney
<EMAIL_ADDRESS>Thank you.
Kind regards,
Ben Callaway
Fruit Studios Ltd
First Floor
100 High Street
Winchester
Hampshire
SO23 9AH
T: 01962 809 103
E<EMAIL_ADDRESS>W: www.fruitstudios.co.uk
====================================================
This message contains confidential information and is intended only for the individual named. If you are not the named addressee you should not disseminate, distribute or copy this e-mail. Please notify Fruit Studios Ltd immediately by e-mail if you have received this e-mail by mistake and delete this e-mail from your system. E-mail transmission cannot be guaranteed to be secure or error-free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or contain viruses. Fruit Studios Ltd does not accept liability for any errors or omissions in the contents of this message, which arise as a result of e-mail transmission. Opinions expressed in this email are those of Ben Callaway, and do not necessarily reflect those of Fruit Studios Ltd. If verification is required please request a hard-copy version.
====================================================

Don't want to keep pushing, but would be great to see this fixed! 😃


Use of % % to store CRON expression in TimerTrigger not functional in Isolated Processes?

I am trying to upgrade my Azure Functions to .NET 6 from .NET Core 3.1, and I am no longer able to use a variable in my local settings for my TimerTrigger CRON expression. Instead of doing this:
public void Run([TimerTrigger("*/10 * * * * *")].....)
I want to do something like this:
public void Run([TimerTrigger("%MyVariable%")]....)
But it doesn't seem possible anymore. Is there another way to do this?

Flagging this for investigation so we can take a closer look at the user secrets wire up and make sure this is not missed with OOP.

Flagging @gzuber here as a point of contact if we need to get more information about the user secrets integration.

@GCoding83 Does adding AddUserSecrets() in your startup.cs help with the issue?
public override void ConfigureAppConfiguration(IFunctionsConfigurationBuilder builder)
{
            if (System.Environment.GetEnvironmentVariable("AZURE_FUNCTIONS_ENVIRONMENT") == "Development")
            {
                builder.ConfigurationBuilder.AddUserSecrets<Startup>();
            }

@surgupta-msft, is this documented, or is a sample linked from the documentation demonstrating how to use the feature?
The linked PR is referring to user secrets, also something that'd be nice to have a reference to some documentation. Thank you.

@SeanFeldman These are the links I found useful demonstrating how to access secrets in code -

https://docs.microsoft.com/en-us/aspnet/core/security/app-secrets?view=aspnetcore-6.0&tabs=windows#access-a-secret
https://github.com/Azure/azure-functions-core-tools/pull/2136#issuecomment-683162904


init_rfc6979 already implemented in rfc6979.c

init_rfc6979, generate_rfc6979 and generate_k_rfc6979 implemented in rfc6979.c

Why did you close this @yangqi365

CI failed @prusnak


LightsOff: fix the remaining 500 or so puzzles

Fixes #26

Thanks! I like to think that it wasn't me being thick that I had a hard time solving the puzzles before, but that they were un-solvable in the first place. :)


Support Full Screen Gameplay

Enter full screen mode by using the standard OS window chrome.
Provide keyboard shortcut to exit full screen mode

Fullscreen keyboard shortcut is F5


The GLL method cannot perform exact numerical integration for DGSEM ？

When using GLL nodes, employing the same nodes for both interpolation and quadrature is known as collocation. For example, in solving the Navier-Stokes equations, this method links the degree of interpolation polynomials to the order of exact integration. As a result, the mass matrix in inner product form and the flux term matrix cannot be integrated exactly. How significant is this impact? Does it lead to aliasing errors?

It does lead to some under integration errors, but these decrease rapidly as the polynomial degree increases. Moreover, Trixi's VolumeIntegralFluxDifferencing avoids most under integration-based stability issues.

Understood. Thank you for your response.


Set custom validator with custom error messagem

how to set and show custom validator with custom error message, like the image bellow

This is not supported atm

Any example in how to do it with jQuery?
And where do it?

I don't have an example

I can see there's now code related to frontend validation. How to use ValidationHelper to set custom validation rules with ValidationHelper?
https://github.com/volkanceylan/Serenity/blob/master/Serenity.Scripts/CoreLib/UI/Helpers/ValidationHelper.ts


Update free-programming-books-langs.md

What does this PR do?
Add book "Fundamentals of Data Visualization"
For resources
Description
Why is this valuable (or not)?
Thhis is valuable to clear the concepts of Data Visualization
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Thanks, and welcome to free-programming-books!


Error while creating OperationsClient from operationsSetting

OperationsClient issue
There is no example provide to create OperationClient from operationName (which is fetch after batch translation is submitted)
I have submitted google batchTranslation and got the operationId (projects//locations/us-central1/operations/).
now I want to check the status of this operation by pooling. Here I have used below  dependencies

com.google.cloud
libraries-bom
9.1.0
pom
import

com.google.cloud
google-cloud-translate

            Add below code in java class:
            //serviceKey is String containing service Account json file content as string
            GoogleCredentials credentials = GoogleCredentials
                                                                            .fromStream(new ByteArrayInputStream(serviceKey.getBytes()))
                                                                            .createScoped(Lists.newArrayList("https://clicktime.symantec.com/3TWFMHceuuMgwf8KebwWinm7Vc?u=https%3A%2F%2Fwww.googleapis.com%2Fauth%2Fcloud-platform%2522%29%29%3B
            OperationsSettings operationsSettings =                OperationsSettings.newBuilder().setCredentialsProvider(FixedCredentialsProvider.create(credentials)).build();
            OperationsClient operationsClient = OperationsClient.create(operationsSettings);

But here I am getting nullpointerException when Trying to create OperationClient
java.lang.NullPointerException
at com.google.longrunning.stub.OperationsStubSettings.createStub(OperationsStubSettings.java:107)
at com.google.longrunning.OperationsClient.(OperationsClient.java:143)
at com.google.longrunning.OperationsClient.create(OperationsClient.java:125)
on further investigation found that OperationsSettings created have transportChannelProvider as null.
Please guide me am I missing something in creating OperationSettings/OperationClient.
Guide to get the best way to find the status of operation in java.
Thanks,
Ranjit Kumar

@ranjit5311 Have you tried using this sample?
https://github.com/googleapis/java-automl/blob/master/samples/snippets/src/main/java/com/example/automl/GetOperationStatus.java


Improve to be able to place flags after arguments

Currently stern does not work if flags are placed after arguments. With this commit, it improve to be able to place flags after arguments like kubectl.
stern .* -n kube-system

For this change, github.com/spf13/cobra is used instead of gopkg.in/urfave/cli.v1 and help message is changed accordingly. This commit is not breaking change, compatible.
github.com/spf13/cobra is released under the Apache 2.0 License.

I think this change is useful, but what do you think?

We were actually discussing this exact thing (urfave -> cobra) here at Wercker just a couple hours ago! :)

Minor thing but you could you update the readme to match the order:
Usage:
  stern pod-query [flags]

@akupila Should I update all examples?

@superbrothers Yes please. Very minor thing but good to keep things consistent

Updated

Actually i just noticed a small thing while running this locally:
The --context looks a bit weird here (i guess Cobra pulls out the default value from the description):
Tail multiple pods and containers from Kubernetes

Usage:
  stern pod-query [flags]

Flags:
      --all-namespaces                           If present, tail across all namespaces. A specific namespace is ignored even if specified with --namespace.
      --color string                             Color output. Can be 'always', 'never', or 'auto' (default "auto")
  -c, --container string                         Container name when multiple containers in pod (default ".*")
      --context kubectl config current-context   Kubernetes context to use. Default to kubectl config current-context
  -e, --exclude stringSlice                      Regex of log lines to exclude
      --kube-config string                       Path to kubeconfig file to use
  -n, --namespace string                         Kubernetes namespace to use. Default to namespace configured in Kubernetes context
  -l, --selector string                          Selector (label query) to filter on. If present, default to ".*" for the pod-query.
  -s, --since duration                           Return logs newer than a relative duration like 5s, 2m, or 3h. Defaults to all logs.
      --tail int                                 The number of lines from the end of the logs to show. Defaults to -1, showing all logs. (default -1)
  -t, --timestamps                               Print timestamps
  -v, --version                                  Print the version and exit

I guess we can change the formatting so that Cobra doesn't pick this up (or some other way disable it)

Oh, just realized it was actually like this before too. I think it's nicer to keep this consistent though (just show --context kubectl string), mind updating that and we'll get this merged in?

Yes, that is behavior by flag, so it is the same before.

Oh, i didn't know about this. Anyway, we can probably change it in another PR if it makes sense.
Thank you so much for doing these changes, it makes the cli interface so much nicer to use! 🎉

stern is an awesome tool. Thank you so much for making it!


add environment variables to powershell profile

This adds all variables from load-omnibus-toolchain to the powershell $profile which allows one to build in a powershell environment and also avoids having to manually load the batch file

Would it be easier to generate some sort of PowerShell script (in addition to the .bat file) that is sourced automatically?

That's essentially what this does. Powershell has an automatic variable called $profile which is the path to the user's profile script. So this runs the original .bat file and dumps all the environment variables and then dynamically adds script to the $profile that generates the same variable assignments.

/cc @chef-cookbooks/omnibus-maintainers

:+1:

I agree with @yzl - it would be super cool if we could have the script hang out after it's been run, but I don't feel like it's a blocker for merging this PR...

Are the above comments regarding this line?:
 "Remove-Item `$tempFile" >> $profile

This is the output of the environment variables after running the batch file. However, because the batch file is run to get this output, all the environment information is echo'd to the console when a user opens a powershell console. So it should be very clear what was exported.

I think it's more about how we usually do a 2 step (drop file, invoke file) so someone logging into the box for a testing can see the contents of the script after the fact from the box itself. If I'm not mistaken, this runs the script in place and puts the results out to the console, but not the script itself... Is that correct @mwrock ?

no not the script itself but that script is retained in the ,.bat file and stays on the box. The problem is if you run a .bat file in powershell, it creates a new cmd.exe process so the environmet variables get lost after running that .bat file from a ps console. This change runs that same script and then has that same cmd process pipe the output of SET to a temp file which is hust a dump of the current environment key/values. Then it parses that and sets the same values in the user's powershell session.

Ahhhhh ok. That works for me!

@mwrock Your explanation makes total sense.   :+1:

My only worry with all this is there is a lot of magic going on. I would prefer we have something like load-omnibus-toolchain.ps1 that a user has to explicitly call from a PowerShell session to load the Omnibus build toolchain environment. In many cases I would not want the Omnibus build toolchain environment to be loaded with every PowerShell session (since it does things like muck with default Ruby and Git).

I'm really sorry, but I think maybe I misunderstand the context in which this cookbook is used. My interpretation was that it is used ephemerally for building but based on a couple comments I have seen now it sounds like this is installed on long lived infra. Is that a correct assessment?
I would agree that loading the environment variables for each powershell session could be quite annoying and perhaps destructive if this is on boxes used for other purposes.

@mwrock No need to be sorry, I'll try to fill you in on some required tribal knowledge! This cookbook is used in both scenarios you mentioned. We manage our CI build slaves with this cookbook and use it when converging development environments for omnibus projects. The hope is that both environments match since the same Chef code is used to manage them.

I'm going to revise this PR so that it does not drop the environment into $profile but rather a load-omnibus-toolchain.PS1

ok. updated this to create a load-omnibus-toolchain.ps1 instead of setting up environment from the $profile

:+1:

ok just made the changes to the ps script suggested by seth.

This is great @mwrock! We just switched to managing this cookbook with Delivery. Can you please close this PR and open a changeset on delivery.chef.co? Ping me if you need help.

@delivery review

@delivery review

@mwrock Would it make sense to resurrect this PR? Would be nice to have a load-omnibus-toolchain.ps1 file next to our load-omnibus-toolchain.bat file.

@schisamo I rebased and cleaned it up but need to test it. Can you reopen the PR? Thanks

@mwrock I think you'll just have to open a new one as I the Reopen button is greyed out and when I hover I see the message: The branch was recreated or force-pushed recreated.

created PR #167 to revive this.


Some ID issue.

Title, see https://discord.com/channels/734077874708938864/783091855616901200/1117906186348527727.

This shouldn't ever occur normally, and I don't know why it does. I do have a workaround in mind, though.


[Peek] When losing focus it should close automatically

Microsoft PowerToys version
0.70.0
Installation method
PowerToys auto-update, Microsoft Store
Running as admin
No
Area(s) with issue?
Peek
Steps to reproduce

[ ] Click elsewhere after Peek is run

✔ Expected Behavior
It is expected to close automatically when focus is lost when clicking elsewhere.
❌ Actual Behavior
It doesn't close when clicking elsewhere.
Other Software
No response

Peek closes when focus is lost (0.71). Perhaps this issue has mistakenly been left open?
The reason for commenting however is to suggest for this to be a configurable option instead. This would be particularly useful where one may want to preview a file on one screen and do something on the other. Granted, the file could just be opened in its default application, but I think being able to stick to Peek as a quick way to view files is ideal.
If, for example, your images or videos are configured to open in a heavier application by default, say Adobe Photoshop or Premiere Pro, then the default application may not be ideal for the task. The choice for the preview window simply not to close on losing focus opens up a few other workflows.

Fixed in 0.71 release of PowerToys.  https://aka.ms/installpowertoys
@redactedscribe, it is an option :)


backup won't work if old newJoplinBackup.7z file is there

Hello,
today I recognized that no backups were created if the file newJoplinBackup.7z is still in the backup directory.
My Joplin is running all the time on my Notebook and I made backups every hour on a network drive. Today I recognized the file newJoplinBackup.7z in the backup directory which was many days old an since this date no backups were made anymore.
After some tests I found the solution: I removed the file and everything works fine.
So I think this is a bug and should be solved. I think the old newJoplinBackup.7z was the result of a interuppted backup by a shutdown during the backup was running.
Greeting and thx for the nice plugin, Thomas

My backup settings are these. Drive h:\ is a network drive.

Ok, tnx. I try to replicate it with this settings.

The behavior that the backup does not run if the newJoplinBackup.7z already exists in the backup directory I could not reproduce. But I have found a other bug during the investigation.
If the file was existing from a privues aborted backup, the  newly created backup could then contain old and new backup data.
To prevent this the file is now deleted to the begin.


[Feature] "Restart in safe mode" button in settings

Is your feature request related to a problem? Please describe.
Currently, discord keeps crashing when entering a channel, so it would be nice to reload in safe mode.
Describe the feature you'd like
A button in settings that will restart discord without any plugins or themes. Other plugins could also use it, such as crashrecovery, after repeated crashes.

This should probably be a button added to the crash screen, but yes this is a good idea.

https://1lighty.github.io/BetterDiscordStuff/?plugin=CrashRecovery

restart discord without any plugins or themes

Do you mean this as reloading without any plugins or themes loaded? Or just not enabled?

I would suggest reloading without themes and plugins loaded, since AFAIK plugins can still run code when loaded (and not enabled), which may cause the crash.
I think something like this would be nice:
Discord Crashes -> Discord restarts in safe mode -> A popup shows the exception which caused the crash.
Thanks.


When roaming, aosp status bar connectivity shows roaming and H+/3G overlapping.

As description. Samsung S8+ on 7.0.

Thank you for reporting this issue. This is something currently known for me, and I am trying to find up a way to fix it.

Fixed in latest release.


Adding support for yandex odyssey and integration tests

What for

In order to have compatibility with similar to pgbouncer apps like yandex odyssey
In order to have integrational regression tests for several environments

What has been done
I've managed to obtain metrics from odyssey. The problem was that the current realization failed on getDB(). Pinging database occurred through "SHOW STATS" and either it's a misconfiguration in my odyssey or it's normal behaviour but it failed to get it. Also, I've changed Dockerfile for building executable from source (multi-stage building) and I've added docker-compose files for pgbouncer and odyssey.
What questions have arisen

I think that I've misconfigured pgbouncer and/or odyssey in test folder because exporter failed to read some metrics with error

pq: unrecognized configuration parameter \"<%OPTION>\"\n"

If any of these can be helpful, I would be glad to discuss further integration

It looks like your fork is unclean. I suggest re-forking and instead of making commits to master, use fork and branch workflow.

Any updates?

Any updates?

unfortunately, not for now, I've been quite busy. Need to dive in to the circleci, hope to get here at the end of Feb

unfortunately, not for now, I've been quite busy. Need to dive in to the circleci, hope to get here at the end of Feb

@uncleDecart Have you seen this issue?

@uncleDecart Have you seen this issue?

@maxpain yep, I think I came across this issue when I tried to befriend exporter and odyssey

@maxpain yep, I think I came across this issue when I tried to befriend exporter and odyssey


chore: fix segfault on network test

Purpose

How does the solution address the problem

Changes made

prnet cleaned-up successfully


ARROW-11232: [C++] Make Table::CombineChunks() handle table with zero column correctly

CI passed on my fork, merging.

CI passed on my fork, merging.


Adding elements.h file

This file is being added to tools. It is meant to consolidate the atomtable in csg and the elements.h file from xtp.
In moving the file it has also been updated to include more of the elements from the periodic table. And exceptions have been added for methods that are deprecated.

@JensWehner It would probably be good if you took a look at this before it is merged.

Travis keeps failing with this
#########################################################################################################
#                                                                                #
# ERROR:                                                                                #
# run_gromacs.sh: There is a nan in 'confout.gro', this seems to be wrong.                              #
# For details see the logfile /home/votca/votca/build/csg-tutorials/methanol-water/X_0.5/re/inverse.log #
#                                                                                #
#########################################################################################################```

But I don't see how it is related to the file I added?

Just do a re-trigger.

Prepare to be wowed!

Looking at https://codecov.io/gh/votca/tools/pull/19, I think we should exclude the src/tests/* from the coverage report.
Can you copy this file https://github.com/junghans/travis-docker-glue/blob/master/.codecov.yml?

Are you sure that's kind of the point of testing right? The reason nothing is showing up but the elements.h is because it is the only thing with actual run time testing.

Are you sure that's kind of the point of testing right? The reason nothing is showing up but the elements.h is because it is the only thing with actual run time testing.

Yes, the test itself should not go into the coverage number and hence being excluded. And the link above shows the coverage diff. The coverage of master (with this PR) is here: https://codecov.io/gh/votca/tools/branch/master

I think I have addressed everything... I'm sure you will let me know if I missed something.

Now as merged, don’t forget to remove atom tables from csg and xtp!


Flow Approval Mark Down inconsistent

The markdown table does not work in Outlook 2016. This is how it looks in Outlook 2016 -

| Unit | CP Test ||:------|:----------|| Function | IT || Location | Mumbai   || Title   | This is CP Flow Test || CP   No. | CP00001234 || Amount   | 2300000 |

It works only in Outlook 365 Online.

@Yatinpurohit Please report product-related issues in the Microsoft Flow support site: https://flow.microsoft.com/en-us/support/
Closing the issue.


Enable UseChecksum option pass-through

Allow user to pass in UseChecksum argument to the webdeploy command.
(false by default)

My use case: I really need this to keep my vendor.js from updating every time and keeping my website users from getting cached version.

Thanks for your contribution! I pushed 1.7 to chocolatey (pending review).

Ok, it's up on there.

awesome! thanks
On Wed, May 25, 2016 at 11:32 AM, David Ebbo<EMAIL_ADDRESS>wrote:

Ok, it's up on there.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/davidebbo/WAWSDeploy/pull/13#issuecomment-221628065


Edge Toolbar - Installed Extensions default to show instead of hidden

As per the following guide, the default toolbar_state is supposed to be default_hidden for all extensions in Edge 103 and above:
https://learn.microsoft.com/en-us/deployedge/microsoft-edge-manage-extensions-ref-guide
For the current Edge 115, in my testing, this appears to default to default_shown instead.
Is there a CSP or OMA-URI to change this behavior? Thanks!

There wouldn't be a CSP as the Edge settings in the Settings Catalog are derived from ADMX imports that the Intune team does to get the settings from Edge.
It's possible that this is a bug. I'll need to test and look into it. Is this causing issues?

Thanks. I suppose I could handle this with a custom ADMX import.
I'm enabling a couple extensions via Allow specific extensions to be installed and Control which extensions are installed silently. So, the only issue is like you mentioned in PR #1, these icons take up precious real-estate on the toolbar.

Closing the issue as it's not related to default settings. Still unsure why it doesn't work as expected.


Removes general BIND configuration

This section includes information on configuring the server with a forwarder.  This is not really the main focus of this how-to.  I suggest we eliminate distractions and information that people can easily find elsewhere.

Seems non-controversial, so I am merging.


Allow ROs to allow % and up to run tournaments

This was requested several times, as a lot of rooms have bots to allow drivers to run tournaments for them.
Note: current room settings for tournaments will not change by default; if you had your room set to allow mods and up before, that will still be the case unless you change it; same for if you had tournaments disabled all together.
This changes the command syntax for /tournament enable to be: /tournament enable [%/@].  The command syntax for /tournament disable has not changed. (disabling a tournament will make it so only # and up can run tournaments.)
This also re-arranges the roomsettings rank order UI for tournaments; it only makes sense that the ranks go in order of power least to greatest (left to right).


ci: add cargo audit

Summary
This adds cargo audit to the CI pipeline.
Background
This is useful to catch issues published to RUSTSEC.
Changes
Added a job to test.yml to run it.  I used the instructions from audit-check (as recommended by cargo-audit here), but chose to not restrict the check to run only if Cargo.[toml|lock] are modified, since cargo audit can start warning on a given dependency at any time.  We could probably change to follow their suggestion fully, i.e. run audit nightly and otherwise only run on changes to those two files, but I'll leave that up to SRE to decide :)
Note that we currently have three audit warnings, but they're all deemed as "allowed" by default (i.e. they're all informational only).  It seems like we should maybe look to address the borsch one since, while it's not deemed a security issue, it's still best to avoid unsound code!
Testing
N/A

I think we should gate running this on changes to Cargo.lock for new PRs, but also run this periodically (every 12 hours?) using a schedule workflow trigger: https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#schedule


[NFC] Add a fast type checker test for rdar://74853403.

Thanks to @xedin's recent optimization to propagate contextual types for closures into closure bodies, this is super fast now. The solver statistics are impressive:
---Solver statistics---
Total number of scopes explored: 131
Maximum depth reached while exploring solutions: 21

Resolves: rdar://74853403

@swift-ci please smoke test


[NEW ICON] Coolify

Skill Name
Coolify
Why?
It's a new, powerfull and popular way to self-host services and manage them as dockers very easily. And as they say: "An open-source & self-hostable Heroku / Netlify / Vercel alternative."
Reference Image

I created a new project since this project is no longer updating & the author left it, coolify is available skills-icons
https://skills-icons.vercel.app/api/icons?i=coolify

Thank you for your suggestion and work! Also, that's good a way to market your projects :D


added raw encoding to tinymce

https://github.com/django-oscar/django-oscar/issues/1697#issuecomment-86602991

Thanks!


[camera] NNBD migration of the camera plugin

Migrates the camera plugin to NNBD.
Note: the PR is marked "Draft" because it depends on the NNBD migration of the camera_platform_interface package which is yet to be published (see PR #3530).
flutter/flutter#75228
If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.
Pre-launch Checklist

[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide and the C++, Objective-C, Java style guides.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I signed the CLA.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@googlebot I fixed it.

Sorry for the late notice, but CLA bot seems cannot recognized suggestions' commits' CLAs directly, so you need to remove the Co-authored-by message when you apply suggestions.

Sorry for the late notice, but CLA bot seems cannot recognized suggestions' commits' CLAs directly, so you need to remove the Co-authored-by message when you apply suggestions.

Check, didn't realise that, but thank you for letting me know. Do you know if there is anything I can do now to fix it?

Do you know if there is anything I can do now to fix it?

Let's try rebase and commit again?


Don't use model path as model name

This came up while I was reviewing #1802.

Instead of always using the model path as the model name in ilab model serve, it would be nice to have a separate option. Using path == name is causing a ton of confusion because the same "path" behaves differently depending on which option we're talking about (is it an option for a model path or a model name?).
The UX pain here got worse once we moved everything away from operating out of the current directory. I think it's bad enough now to warrant the rework.

also, consider the case where ilab model serve and ilab model chat are being run on different machines
ilab chat -m /root/.local/share/instructlab/models/instructlab/granite-7b-lab --endpoint-url https://some.other.machine/v1

It's clearly strange to expect the client to know the model path on the other machine - and this is not only with ilab chat, it's any client

Related to #1871


refactor!: Simplify Chunk inheritance hierarchy

This removes most of the unnecessary variants of Chunk, leaving as the only interfaces Chunk itself (which has all the chunk functionality) and RenderableChunk (which is suitable for use by things other than chunks that need to be rendered like chunks). Currently the only subclass of RenderableChunk but not Chunk is LodChunk (which now doesn't have all the extra methods throwing UnsupportedOperationException). I intend to implement a way for entities to be rendered as though they were chunks in order to facilitate adding animations to FallingBlocks (which was the motivation for this PR), but I thought it would be easier to review in smaller steps.
Pretty much every world-gen module needs trivial changes, because CoreChunk was replaced with Chunk in the rasterizer interface. Those PRs were mostly just automatically generated using one of IntelliJ's refactoring tools. CoreRendering also needs an update to use the new RenderableChunk interface: https://github.com/Terasology/CoreRendering/pull/37.

@keturn I guess I spoke to soon about not doing this so often.

@pollend did you test this locally with all the modules and can confirm that this works?
I left a couple of questions, but don't know enough about our chunk handling that I would block this PR. I did not run this locally either, that's why I'm not approving the PR(s).

yea ... I need to test it, umm checking out all those branches is kind of painful XD


just a simple typo fix in changelog

I'm submitting a...
[ ] Bug Fix
[ ] Feature
[x] Other (Refactoring, Added tests, Documentation, ...)

[ ] Yes
[x] No

THanks!


doubt in sample

what should be given in $installsource

$InstallSource should be the base path for all the installation files. For example, I defined it in my configuration data file, excerpted below:
@{
    AllNodes = @(
       @{
            NodeName = '*'
            InstallSource = "\\my-test-domain\install"
        }
    );
}


Use of deprecated constants STATE_CLASS_MEASUREMENT, STATE_CLASS_TOTAL, STATE_CLASS_TOTAL_INCREASING

Sharing from the log file that requested I submit a bug report here:
Logger: homeassistant.components.sensor
Source: helpers/deprecation.py:205
Integration: Sensor (documentation, issues)
First occurred: 12:33:14 PM (6 occurrences)
Last logged: 12:33:14 PM
STATE_CLASS_MEASUREMENT was used from sunpower, this is a deprecated constant which will be removed in HA Core 2025.1. Use SensorStateClass.MEASUREMENT instead, please create a bug report at https://github.com/krbaker/hass-sunpower/issues
STATE_CLASS_TOTAL was used from sunpower, this is a deprecated constant which will be removed in HA Core 2025.1. Use SensorStateClass.TOTAL instead, please create a bug report at https://github.com/krbaker/hass-sunpower/issues
STATE_CLASS_TOTAL_INCREASING was used from sunpower, this is a deprecated constant which will be removed in HA Core 2025.1. Use SensorStateClass.TOTAL_INCREASING instead, please create a bug report at https://github.com/krbaker/hass-sunpower/issues

is this something we can fix or wait for a update?

Exact same issue, with these additional variables reported:
Logger: homeassistant.const
Source: helpers/deprecation.py:205
First occurred: February 11, 2024 at 3:01:52 AM (16 occurrences)
Last logged: February 11, 2024 at 3:01:59 AM
DEVICE_CLASS_POWER was used from sunpower, this is a deprecated constant which will be removed in HA Core 2025.1. Use SensorDeviceClass.POWER instead, please create a bug report at https://github.com/krbaker/hass-sunpower/issues
DEVICE_CLASS_VOLTAGE was used from sunpower, this is a deprecated constant which will be removed in HA Core 2025.1. Use SensorDeviceClass.VOLTAGE instead, please create a bug report at https://github.com/krbaker/hass-sunpower/issues
DEVICE_CLASS_CURRENT was used from sunpower, this is a deprecated constant which will be removed in HA Core 2025.1. Use SensorDeviceClass.CURRENT instead, please create a bug report at https://github.com/krbaker/hass-sunpower/issues
DEVICE_CLASS_TEMPERATURE was used from sunpower, this is a deprecated constant which will be removed in HA Core 2025.1. Use SensorDeviceClass.TEMPERATURE instead, please create a bug report at https://github.com/krbaker/hass-sunpower/issues
DEVICE_CLASS_POWER_FACTOR was used from sunpower, this is a deprecated constant which will be removed in HA Core 2025.1. Use SensorDeviceClass.POWER_FACTOR instead, please create a bug report at https://github.com/krbaker/hass-sunpower/issues

I will try to get to this, I'm happy to take a patch, TBH I'm not using this right now (I switched to using an Enphase Envoy to read the panels.. side note: don't do this, I was told it was supported, its not)

(I switched to using an Enphase Envoy to read the panels.. side note: don't do this, I was told it was supported, its not)

I believe SunPower uses custom firmware on the Enphase MIs.

Sadly it works (at least 99%) but they won't support it (the way I heard what they were saying was 'sunpower won't let us').  The panels I have use IQ7XSs which were only sold attached to 360w Sunpower AC panels.  My PVS had stopped reporting data as they seem to do a lot. That, along with all of sunpowers data corruption I was super fed up so I asked enphase and at the time the answer I got was yes, it will work.  So I had it running for ~9 months because they told me it would.  The Envoy is way better than the PVS and Enphase support is awesome.
My panels for some reason stopped reporting on an automatic firmware update to the Envoy.  In the process of debugging the customer support folks noticed the sunpower panels and said they couldn't support that configuration.  I even sent them the previous comms and they said 'sorry, that was incorrect'.  I ended up getting it working again by removing and re-adding the panels but its clearly not going to be well supported (after this 'fixing' things I'm actually curious if this was just a general bug unrelated to the Sunpower inverters)
Why is Enphase awesome?

A real, supported local API..  it has less data than this hack manages to pull (per panel only gets current watts) but still, they actually allow me to get my data.
They allow and support self-install.  You have to complete a training but its quite reasonable and then you get access to your system as if you were an installer.
They will try and help with almost anything.  They were willing to work through setting up a grid profile to enable frequency-watt scaling that works with the Powerwall and is compliant with my local grid.  This give the powerwall very fine control over the solar output so it doesn't have to keep bouncing between full and no power from the solar.

Sadly it works (at least 99%) but they won't support it (the way I heard what they were saying was 'sunpower won't let us'). The panels I have use IQ7XSs which were only sold attached to 360w Sunpower AC panels. My PVS had stopped reporting data as they seem to do a lot. That, along with all of sunpowers data corruption I was super fed up so I asked enphase and at the time the answer I got was yes, it will work. So I had it running for ~9 months because they told me it would. The Envoy is way better than the PVS and Enphase support is awesome.
My panels for some reason stopped reporting on an automatic firmware update to the Envoy. In the process of debugging the customer support folks noticed the sunpower panels and said they couldn't support that configuration. I even sent them the previous comms and they said 'sorry, that was incorrect'. I ended up getting it working again by removing and re-adding the panels but its clearly not going to be well supported (after this 'fixing' things I'm actually curious if this was just a general bug unrelated to the Sunpower inverters)
Why is Enphase awesome?

A real, supported local API..  it has less data than this hack manages to pull (per panel only gets current watts) but still, they actually allow me to get my data.
They allow and support self-install.  You have to complete a training but its quite reasonable and then you get access to your system as if you were an installer.
They will try and help with almost anything.  They were willing to work through setting up a grid profile to enable frequency-watt scaling that works with the Powerwall and is compliant with my local grid.  This give the powerwall very fine control over the solar output so it doesn't have to keep bouncing between full and no power from the solar.

Great insights about the Enphase.  Sadly, my system has the older integrated Sunpower/SolarBridge inverters so I can't make the switch.  I also had issues with the PVS - my PVS5 stopped working last year.  The PVS has a 10 year warranty, so Sunpower replaced it with a PVS6.  The PVS has a 10 year warranty - did you ever ask SunPower to replace it?
I also have some issues with the PVS. Two months after I received the replacement, the PVS stopped reporting data to SunPower (though I was still able to access production data from your add-on).  TBH, I'm not sure if the constant polling is causing the unit to fail, or if the extra power draw from the travel router is causing the problems.

or if the extra power draw from the travel router is causing the problems.

I am not using a travel router, and have the earlier version of the PVS6 with dual Ethernet ports, so I don't have anything pulling power from the PVS... and I have the same 'reporting failure' issues.
We're really far off the topic of this issue though, and should move to some sort of Sunpower+HA self-help group :)

Adding this as it looks like the same class of error
Logger: homeassistant.const
Source: helpers/deprecation.py:206
First occurred: 3:07:49 PM (16 occurrences)
Last logged: 3:08:54 PM
DEVICE_CLASS_POWER was used from sunpower, this is a deprecated constant which will be removed in HA Core 2025.1. Use SensorDeviceClass.POWER instead, please report it to the author of the 'sunpower' custom integration
DEVICE_CLASS_VOLTAGE was used from sunpower, this is a deprecated constant which will be removed in HA Core 2025.1. Use SensorDeviceClass.VOLTAGE instead, please report it to the author of the 'sunpower' custom integration
DEVICE_CLASS_CURRENT was used from sunpower, this is a deprecated constant which will be removed in HA Core 2025.1. Use SensorDeviceClass.CURRENT instead, please report it to the author of the 'sunpower' custom integration
DEVICE_CLASS_TEMPERATURE was used from sunpower, this is a deprecated constant which will be removed in HA Core 2025.1. Use SensorDeviceClass.TEMPERATURE instead, please report it to the author of the 'sunpower' custom integration
DEVICE_CLASS_POWER_FACTOR was used from sunpower, this is a deprecated constant which will be removed in HA Core 2025.1. Use SensorDeviceClass.POWER_FACTOR instead, please report it to the author of the 'sunpower' custom integration

Found MOAR
Logger: homeassistant.const
Source: helpers/deprecation.py:206
First occurred: 4:19:55 PM (11 occurrences)
Last logged: 4:19:57 PM
ELECTRIC_POTENTIAL_VOLT was used from sunpower, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfElectricPotential.VOLT instead, please report it to the author of the 'sunpower' custom integration
ELECTRIC_CURRENT_AMPERE was used from sunpower, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfElectricCurrent.AMPERE instead, please report it to the author of the 'sunpower' custom integration
TEMP_CELSIUS was used from sunpower, this is a deprecated constant which will be removed in HA Core 2025.1. Use UnitOfTemperature.CELSIUS instead, please report it to the author of the 'sunpower' custom integration

Is there something we need to add/edit to our configuration or will a updated version fix issue?

I'm in the process of cleaning them up, its just a matter of switching which constant its using (so far), the PR above solves a bunch, I should get to the rest in a bit.


Move Event, RoomEvent and StateEvent implementation codegen into their own derive macros

Title pretty much says it all. This would probably increase proc-macro code size slightly but create a clear separation of concerns. Also, it would allow the manual implementations for Custom{,Room,State}Events that are being added in #65 to be removed.

We might actually get rid of the traits altogether, see https://github.com/ruma/ruma-events/issues/43#issuecomment-623026898.

It's now pretty certain that those traits will be removed. Instead, there are / will be derive macros for the new EventContent traits.


Autorun

Can I start the program automatically? So you don't need to press the play button?

No, you must press play button to start a digishow link session.  For the automatic feature you mentioned, do you mean you want to run digishow app and start a program in a specified dgs file once the computer operation system started ?

is there a chance to put the show in computer startup, and when you turn on your computer, the show is running automatically without going into the program and press the start button

autostart option is added in commit c83d536b4c89b5b141f86b262d48f597d142795d


Split the single command() system call into command0(), command1(), and command2().

The existing command() definition always sets both argument registers. For many uses of command(), this is unnecessary. Setting only the needed argument registers reduces the generated assembly size.
In its current form, this change is not backwards-compatible: users of the command() function will need to be updated to point to command0(), command1(), or command2() (with command2() being a drop-in replacement).
This reduces our test app sizes by 48 and 64 bytes (these apps only use one driver -- Console -- hence the small change), and should simplify the generated assembly which will help with debugging. This PR is motivated by the discussion/investigation at https://github.com/tock/libtock-rs/pull/101#discussion_r339337257.

As discussed in the other PR, I'm in favor of this change. This should also apply to libtock-c (which could also have more visibility to review this change).
In terms of what could go wrong with this, in all cases the kernel extracts all 4 arguments of a syscall (so the last two could have arbitrary values instead of zeros after this change), but then ignores the last two in the drivers that don't use them. The memop syscall already does this anyway.

Isn't not zeroing out the syscall args just exposing us to some possible nasty/hard to debug bugs?

There is a risk of nasty bugs with this, yes. I don't know how to quantify that risk. I suspect the risk is small.
As an alternative, we could keep command() around (as the two-argument version), and document that users should use that by default. Then use cases (like the low level debugging code) that really want to use the other methods can use them.

Leaving a default of command would make sense to avoid passing of uninitialized data (although passing a zero value when an argument is expected may be as buggy).
As a comparison point in favor of this change, note that not initializing the last two arguments works well for the memop syscall.

My worry is that passing uninitialised data/previous data will be harder to identify then passing 0 and using 0 for values. The main upsize is size reduction, so I guess it's just a trade-off

Another possibility (although requires a breaking change in Tock) would be to use different svc numbers for the variants of the command syscall (e.g. svc 2 takes 2 arguments, svc N takes 1 argument, svc M takes 0 argument). That way, using the wrong number of arguments in userspace would be detected by the kernel (instead of the kernel blindly using the uninitialized values).
But the extra overhead in the kernel code would likely outweigh the savings in the apps.

I know I've already approved this, but...
There is, I think, a bit of a security concern as
well: in general, capsules shouldn't be able to read arbitrary process data, only data processes pass to it explicitly.
This is partially enforced by the core Kernel's interpretation of system calls: so a command svc is translated into a command method call with three arguments.
So a process always implicitly is providing access to the three argument registers to a capsule it calls with command... With this scheme, it's just that sometimes it will be non-explicit sometimes and might be other application data.
Is this worth the optimization?

Based on Amit's concerns, I have decided to retract this PR.


feat(kit): expose importNuxtPackage kit utility

🔗 Linked issue
resolves nuxt/nuxt#13153
❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[x] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
This adds a kit utility (importNuxtPackage) that can be used elsewhere (Bridge, probably) to import the core Nuxt package when wanting to access some other export from it (or not immediately instantiate Nuxt itself).
📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

closing for now - we can reimplement if needed later on


Bump version from 2022.8.16 to 2022.10.6

[x] Bump version from 2022.8.16 to 2022.10.6 (4e34968032f52d09dc1178792ca2522f723ca9f4)

Comments

Issue Number
ADP-2245

bors r+


"tar -C /  ..." breaks the resolving of the APT repositories

Debian Stretch (4.9.88-1+deb9u1 (2018-05-07))
After enter command:
"tar -C / -xvzf cri-containerd-1.1.1.linux-amd64.tar.gz"
APT, APT-GET and APTITUDE can't resolve any of repositories (excluding repo from local folder or CD).
All other programs works fine
W: Failed to fetch http://mirror.yandex.ru/debian/dists/stretch/InRelease  Temporary failure resolving 'mirror.yandex.ru'
W: Failed to fetch http://security.debian.org/debian-security/dists/stretch/updates/InRelease  Temporary failure resolving 'security.debian.org'
W: Failed to fetch http://mirror.yandex.ru/debian/dists/stretch-updates/InRelease  Temporary failure resolving 'mirror.yandex.ru'
W: Failed to fetch http://mirror.yandex.ru/debian/dists/stretch-backports/InRelease  Temporary failure resolving 'mirror.yandex.ru'
W: Failed to fetch https://download.docker.com/linux/debian/dists/stretch/InRelease  Could not resolve host: download.docker.com

coincidence? Just a guess your dns might be down temporarily. Maybe try <IP_ADDRESS> and see if you can resolve again.

I understand that this looks like a problem with my DNS settings. However, I rechecked this several times on different servers with a Debian distribution. The problem occurs immediately after tar-unpacking, although I do not understand how this can be related. Just try it yourself

kk will give it a shot..

It is a directory permission issue. tar overwrites the directory permission on your host, /etc specifically in this case.
I'll update the all directory permissions in the tarball to 755, and update the tar command to not change the permission of existing directory.

Please use "tar --no-overwrite-dir -C / -xvzf cri-containerd-1.1.1.linux-amd64.tar.gz" instead.
I'll update the document.

Another problem is that on my develop machine I used this time, it has umask 0027, that's why directories become 750.
I'll rebuild the release tarball, and update the release script to set a fixed umask.

@badsysadm I've updated the release tarball. It should now have right permission set.
$ tar -tvf _output/cri-containerd-1.1.1.linux-amd64.tar.gz
drwxr-xr-x root/root         0 2018-07-11 18:27 ./
drwxr-xr-x root/root         0 2018-07-11 18:27 ./opt/
drwxr-xr-x root/root         0 2018-07-11 18:27 ./opt/containerd/
drwxr-xr-x root/root         0 2018-07-11 18:27 ./opt/containerd/cluster/
drwxr-xr-x root/root         0 2018-07-11 18:27 ./opt/containerd/cluster/gce/
drwxr-xr-x root/root         0 2018-07-11 18:27 ./opt/containerd/cluster/gce/cloud-init/
-rw-r--r-- root/root      5757 2018-07-11 18:27 ./opt/containerd/cluster/gce/cloud-init/node.yaml
-rw-r--r-- root/root      5775 2018-07-11 18:27 ./opt/containerd/cluster/gce/cloud-init/master.yaml
-rw-r--r-- root/root       423 2018-07-11 18:27 ./opt/containerd/cluster/gce/cni.template
-rw-r--r-- root/root      1158 2018-07-11 18:27 ./opt/containerd/cluster/gce/env
-rwxr-xr-x root/root      7602 2018-07-11 18:27 ./opt/containerd/cluster/gce/configure.sh
-rwxr-xr-x root/root      1981 2018-07-11 18:27 ./opt/containerd/cluster/health-monitor.sh
-rw-r--r-- root/root        28 2018-07-11 18:27 ./opt/containerd/cluster/version
drwxr-xr-x root/root         0 2018-07-11 18:27 ./usr/
drwxr-xr-x root/root         0 2018-07-11 18:27 ./usr/local/
drwxr-xr-x root/root         0 2018-07-09 22:35 ./usr/local/bin/
-rwxr-xr-x root/root   3539296 2018-07-09 22:35 ./usr/local/bin/containerd-release
-rwxr-xr-x root/root   4185920 2018-07-09 22:35 ./usr/local/bin/containerd-shim
-rwxr-xr-x root/root  28381059 2018-07-11 18:27 ./usr/local/bin/crictl
-rwxr-xr-x root/root  20919672 2018-07-09 22:35 ./usr/local/bin/ctr
-rwxr-xr-x root/root  38133880 2018-07-09 22:35 ./usr/local/bin/containerd
-rwxr-xr-x root/root  17395032 2018-07-09 22:35 ./usr/local/bin/containerd-stress
-rwxr-xr-x root/root  29395509 2018-07-11 18:27 ./usr/local/bin/critest
drwxr-xr-x root/root         0 2018-07-11 18:27 ./usr/local/sbin/
-rwxr-xr-x root/root   7451568 2018-07-11 18:27 ./usr/local/sbin/runc
drwxr-xr-x root/root         0 2018-07-11 18:27 ./etc/
drwxr-xr-x root/root         0 2018-07-11 18:27 ./etc/systemd/
drwxr-xr-x root/root         0 2018-07-11 18:27 ./etc/systemd/system/
-rw-r--r-- root/root       522 2018-07-11 18:27 ./etc/systemd/system/containerd.service
-rw-r--r-- root/root        57 2018-07-11 18:27 ./etc/crictl.yaml

You can still use --no-overwrite-dir just in case.


date and time of finding bycatch

As a [researcher] I want [the app to offer the app-user the chance to log the time and date for when the by-catch was pulled]
two options: 'current date and time' and 'input date and time'

Bycatch not being treated separately from other observations.


Feature request: support fzf init  initialization

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[x] bash
[ ] zsh
[ ] fish

Feature request
Support fzf init <shell> initialization (like in starship, zoxide, navi etc) to facilitate fzf integration with other tools (file managers, utilities etc) because this style of initialization doesn't require knowledge about key-bindings and completion file paths (which can vary in different packages), fzf will generate apropriate files by itself. Such an approach also cleans up shell rc files. Instead of:
source /usr/share/fzf/completion.bash
source /usr/share/fzf/key-bindings.bash

only one line of code is required:
eval "$(fzf init bash)"

For backward compatibility reasons, the feature implementation can just output existing key-bindings and completion files to stdout. completion and keybindings parameters can be supported to get completion.<shell> or key-bindings.<shell> implementations separately:
eval "$(fzf init bash completion)"
eval "$(fzf init bash keybindings)"

Hasn’t this been actually accomplished in https://github.com/junegunn/fzf/tree/0.48.0 with fzf --fish command?

For cross reference, I've partially answered to the questions in https://github.com/akinomyoga/ble.sh/discussions/441#discussioncomment-9209074.

Hasn’t this been actually accomplished in https://github.com/junegunn/fzf/tree/0.48.0 with fzf --bash command?

Yes, I guess this issue can be closed now.

And related question: what is the purpose of these files in shell/ directory at all?

For the finer control mentioned in README, cf

https://github.com/junegunn/fzf?tab=readme-ov-file#setting-up-shell-integration
https://github.com/junegunn/fzf/pull/3722#issuecomment-2048594050


feat: project support

Hi!
This PR to add an optional parameter linear-project-id.
It should assign a project to newly created issue.
Not sure how I can test if it works correctly.

@ctriolo please take a look


add types for channel_convert_to_private message event

Fix typo on ChannelConvertToPublicessageEvent type
Add type support for unlisted channel_convert_to_private message event, based on this conversation: https://community.slack.com/archives/C02C28Z3XA7/p1730963855327819

Thank you. Although channel_convert_to_private is not yet listed in the public document, it should be okay to have it in this library a little earlier.


Separate theme and Microsoft content assets

For some materials like the assets folder we can have a folder in our repo of the same name, but the theme only related assets such as:
css/
favicons/
img/
js/
webfonts/

Mostly implemented


Would you like to add staying logged in to Google Books using tokens?

Would you like to add staying logged in to Google Books using tokens?

If I'm understanding correctly, what you're looking for is a way to download purchased content by using login information. From the testing I've done it looks like purchased books have to be read through Google Play Books rather than Google Books, which would likely require an entire rewrite of the scraping logic. I can only see a sample of content from purchased books in Google Books even though the book IDs are the same as they are on Google Play Books and I'm logged into an account that owns them. On the other side of things, although Google Play Books is able to view the publicly available books also available on Google Books it is not able to view its magazines or newspapers, which are really the main focus of this project. So in order to support purchased content, there would have to be entirely separate logic for scraping from Google Play Books rather than Google Books while still supporting both, and unfortunately I think that falls outside the scope of this project. I'm closing this as unplanned for now, but if I misunderstood your request please let me know!


Plugin doesn't work in PreshaShop 8+

Several error. For example you are using functions removed from Prestashop core:
Tools::jsonEncode and Tools::jsonDecode

Any news on this?

We expect to release a new Prestashop 8 compatible version in beginning of April.


Preparing v0.1

In order to release first apk we need to do some things.

Create a logo for loading screen and logo for main screen.
Implement loading screen.
Add navigation buttons ( exit, about, tutorial(objective))
Design and implement needed functions to all screens
Make speed of clicking counter display number of clicks in a minute
Implement load and save
Find proper graphics for buttons and redesign the game look if needed
Redesign click counter currentclicks/requiredclicks
Add statistics button(screen)
Implement statistics through code ( will discuss details in future)
Prepare checklist for first test build

Preparation of v0.1 started with Issue #3. We created a project inside github for this as well.

So far I have implemented click counter, added health bar in case that we are going to use it.


fix(navigation) avoid navigating twice

the ngLink directive was invoking the $$rootRouter.navigate and also does
the $watch in the routerFactory, so when you click a link both where being
triggered.
I Solved it by adding a check in the navigate method and also providing a force parameter for the renavigate feature.

example tests are failing on CI but should be solved by applying #337


using wide unicode characters in progress_chars results in a too wide bar being drawn

I've just upgraded my project to 0.17.0 from 0.16.2 and I've noticed that a bar that's using
    template.progress_chars("🔐🔑🕓");

now draws incorrectly:

With 0.16.2 it worked just fine:

line on my Cargo.toml:
indicatif = { version = "0.17.0", features = ["improved_unicode", "tokio"] }

Thanks for this project!

Oops, sorry about that! Looks like I made a typo while refactoring in https://github.com/console-rs/indicatif/commit/d7851c7a467313571d71133b1729af9c5dd3b2ae, fix submitted in #456.


Dependency "hookable" would cause error "TypeError: Cannot read properties of undefined (reading 'pkg')"

Environment
Node: 18
unbuild: 1.1.2
Reproduction
https://github.com/DevilTea/repro-unbuild-issue
Describe the bug
Run the build script would throw an exception

After looking for the reason, I found that it is caused by the package "hookable" which just released version 5.5.0
hookable is in unbuild's dependencies and I found that its version is ^5.4.2 that would accept the minor update.
I've tested that install hookable with specific version like 5.4.2 everything would be okay, so it probably can say something that hookable updated breaks unbuild
Additional context
No response
Logs
><EMAIL_ADDRESS>build /Users/deviltea/Documents/Programming/waa
> unbuild

Error building /Users/deviltea/Documents/Programming/waa: TypeError: Cannot read properties of undefined (reading 'pkg')
TypeError: Cannot read properties of undefined (reading 'pkg')
    at build:prepare (file:///Users/deviltea/Documents/Programming/waa/node_modules/.pnpm/unbuild@1.1.2/node_modules/unbuild/dist/shared/unbuild.e1e6feef.mjs:40:18)
    at file:///Users/deviltea/Documents/Programming/waa/node_modules/.pnpm/hookable@5.5.0/node_modules/hookable/dist/index.mjs:47:66
    at Object.run (file:///Users/deviltea/Documents/Programming/waa/node_modules/.pnpm/hookable@5.5.0/node_modules/hookable/dist/index.mjs:41:43)
    at file:///Users/deviltea/Documents/Programming/waa/node_modules/.pnpm/hookable@5.5.0/node_modules/hookable/dist/index.mjs:47:56
    at async build (file:///Users/deviltea/Documents/Programming/waa/node_modules/.pnpm/unbuild@1.1.2/node_modules/unbuild/dist/shared/unbuild.e1e6feef.mjs:934:3)
    at async main (file:///Users/deviltea/Documents/Programming/waa/node_modules/.pnpm/unbuild@1.1.2/node_modules/unbuild/dist/cli.mjs:35:3)
 ELIFECYCLE  Command failed with exit code 1.

There was a regression we shortly fixed with @danielroe. sorry for that.
Please use npx nuxi@latest upgrade --force to update hookable.

Toooooooooooo efficient!! Thx so much~


[WIP] Add Artwork

[ ] Update og:image for social preview

Closes #13

You may want to add/update images on the action:
https://github.com/tighten/phpreleases-action


The [Bug Report] Select selector drop-down box scroll bar is not in the right position.

Element UI version
1.3.7
OS/Browsers version
chrome
Vue version
2.3.4
Reproduction Link
https://github.com/ElemeFE/element/issues/5746
Steps to reproduce
当下拉框值很多出现滚动条时.
选择最后一个值.
然后选择重新选择，下拉框显示的值是第一个，但滚动条却是滚动到底部的状态。
What is Expected?
滚动条应该是在顶部，或者默认展示值是在底部。
What is actually happening?
默认值是第一个开始，滚动条却在底部。

Issues without a reproduction link will be closed directly.


Maatwebsite\Excel\ExcelServiceProvider not found

I'm new to laravel and done all things on installation document. But when going to publish configured settings using "PHP artisan vendor: publish" composer says "Maatwebsite\Excel\ExcelServiceProvider not found". Also, I have not any idea, where to insert "$excel = App:: make ('excel');" line.

I asume you're using Laravel 5.
Just some things you should check:

Did you include the package in your composer.json?
Did you run composer update afterwards?

You can add App:: make ('excel') or (Excel::create() as Laravel-Facade) inside a controller for Excel imports and exports. e.g. ReportsController

"require-dev": {
"phpunit/phpunit": "~4.0",
"phpspec/phpspec": "~2.1",
"maatwebsite/excel": "~2.0.0"
},
include package as above in my composer.json and update the
compser.json.then maatwebsite folder is downloaded into my vendor folder.
also update the "providers","aliases" arrays with given lines.
when trying to publish the configured settings using "php artisan
vendor:publish" command, my console says
~ [Symfony\Component\Debug\Exception\FatalErrorException]
Class 'Maatwebsite\Excel\ExcelServiceProvider' not found ~

Following your exact steps in a Laravel 5 project I don't get the same error.
Just to be sure, try copying this exact string (this is what I have in my config/app.php):
'Maatwebsite\Excel\ExcelServiceProvider', to config/app.php

I'm getting the same error on our live system. Ive copied the files over from test (where it works fine). However I cant run composer on the live system.


After getting a Sound Wave Ref, audio won't call OnAudioFinish, and doesn't Play from 0.0 after.

I downloaded the plug in, and used it to load a audio wav file not included as a UAsset. I managed to get it to work and to play, but I noticed that it wont properly work as normal UAssets. There are 2 important cases for me that I need to be able to handle correctly.
1, When the wav stops playing, it wont trigger the 'On Audio Finished' event from the AudioComponent.

When stopping the audio and then doing a call to Play with Start Time of 0.0, the wav will just continue playing where it was stopped.

This means, that if the audio finished playing. it won't stop itself. And even if I do stop it myself, and then do a play again, the audio will just try to keep playing beyond which is just pure silence.
This is how I am getting the song.

I don't know what I could be doing wrong.

Hi. These "features" exist because of the use of USoundWaveProdecural, which has some limitations.
"On Audio Finished" is called only when the imported sound wave has completely finished playing. It is not called if playback has been interrupted externally (this cannot be handled inside the sound wave). Its purpose is to keep track of when the sound wave has completely finished playing.
In order to "reset" the playback of the sound wave, you need to use the "Rewind Playback Time" function and set the time to "0". Read more about this approach here.
In case you don't want to feel these peculiarities of working with a sound wave, you can convert it to a standard sound wave, which will work identically as imported into the editor in a standard way. However, it is not currently supported by UE5. Read more about it here.

Thanks so much! It now behaves as I expect.
My only question is, to the Compressed Sound Wave, is there a way to make it a looping sound in blueprint?

The playback uses a procedural sound wave, so this option will do nothing.
Instead, you can bind to the "On Audio Playback Finished" delegate and call the "Rewind Playback Time" function, where set to "0" (to make the sound wave start playing from the beginning after the end of playback).

Sorry, I forgot that we are talking about compressed Sound Wave. Yes, it is possible, but so far only in C++ manually.
In the next update, I will make it possible to select this parameter when calling the function to compress the sound wave.

Yes, it is possible, but so far only in C++ manually.
In the next update, I will make it possible to select this parameter when calling the function to compress the sound wave.

Cool! My project is C++, what is the function to call?
Thanks so much for the quick replies too!

Hi, if you use C++ then you just need to set the "bLooping" property to "true" in the compressed sound wave.
But in the next update I will make it possible to do this automatically through the "CompressSoundWave" function.

Hi, this feature has been implemented. You can download the latest plugin source and fill in the "Looping" parameter when compressing the sound wave.

I downloaded it, but I cant compile my game.
GAME/Plugins/RuntimeAudioImporter/Source/RuntimeAudioImporter/Public/RuntimeAudioImporterLibrary.h(111): error : C++ Default parameter not parsed: CompressedSoundWaveInfo "FCompressedSoundWaveInfo()"

It seems that something is wrong in the 4.27 version when parsing some specific ufunction headers. Should be ok now.

Absolute MVP!
Thanks so much!


Move "Add a new scene" button when Scene Tabs settings change

Updating "Display Close Button" setting (interface/scene_tabs/display_close_button) in Editor Settings changes the size of scene tabs, but the add button at the end of the tabs was not being moved until the next update, causing gaps/overlaps between the controls.  Adding call to _scene_tabs_resized() after getting the new settings to update the add button position.
Fixes #91850

Thanks!


Bump go.opentelemetry.io/collector to v0.25.0

Signed-off-by: Andrii Filenko<EMAIL_ADDRESS>Which problem is this PR solving?

Resolves #2931

Short description of the changes

Bump go.opentelemetry.io/collector to v0.25.0 and fix all failed tests

@f1r3starter, thank you for the PR. We just decided yesterday during the community sync call that we want to remove the v2 directory entirely until the OpenTelemetry Collector reaches GA. The ground is too unstable at the moment to rely on it as the base for the Jaeger v2. I'll go ahead and close this PR and the linked issue, but would you be up to send a new PR removing the v2 directory entirely?

https://github.com/jaegertracing/jaeger/issues/2969

@jpkrohling Sure, will do


#96 adding objects index

closes #96
Creating objectionary.lst file with all object names within Objectionary on pages build.
Format of names:
objects/org/eolang/heap.eo
tests/org/eolang/hamcrest/matches-regex-tests.eo

@mximp thanks!


Adding Issue templates

Hello there,
I would like to add an issue template for the repository. This template would have issues namely bug, feature. Kindly assign me this issue as a part of DWOC.
THANK YOU

Not Required.
Whenever you see someone else raise this issue, kindly refer to these🙏
#375 #331

Not Required.
Whenever you see someone else raise this issue, kindly refer to these🙏
#375 #331


os-freeradius 1.9.17 does not start and install errors on opnsense 21.10.1

Radius does not start and install log seems to have some errors. See attached logs:
freeradius install.log
freeradius onedebug.log
Steps to reproduce:
Installed normally and configured as seen in the attached config snippet:
freeradius config.log

Can you go to Services : Freeradius : EAP and hit apply? It should start work now ...

Hello and thank you. That worked!
On 7. joulukuuta 2021 16.35.03 Michael @.***> wrote:

Can you go to Services : Freeradius : EAP and hit apply? It should start
work now ...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

@mimugmail thanks for the quick support! ❤


E: Implementation Class UID is an invalid UID

Description
Trying to perform a C-Echo on a PACS server and this error returns
ValueError: Implementation Class UID is an invalid UID
Expected behaviour
What you expected to happen, (including references to the DICOM standard is
appreciated).
Is should receive the dicom echo back.  I believe that the Implementation Class is Valid as it meets the criteria here :
http://dicom.nema.org/medical/dicom/2016d/output/chtml/part07/sect_D.3.3.2.html
https://www.medicalconnections.co.uk/kb/ImplementationUID-And-ImplementationName/
I note that this works successfully in for the dcm4che dicom java toolkit but not for python pydicom/pynetdicom toolkit
Actual behaviour
What actually happened. If an exception occurred please post the full traceback.
I can see that the server returns the C-Echo but the python raises a value error
This the the returned UID that is raised as invalid:
Implementation UID: "<IP_ADDRESS>532.3320"
Exception in thread Thread-8:
Traceback (most recent call last):
File "C:\Users\xxxxxxx\AppData\Local\Continuum\anaconda3\lib\threading.py", line 916, in _bootstrap_inner
self.run()
File "C:\Users\xxxxxxx\AppData\Local\Continuum\anaconda3\lib\site-packages\pynetdicom3-0.9.0-py3.6.egg\pynetdicom3\dul.py", line 245, in run
if self._is_transport_event() and self._idle_timer is not None:
File "C:\Users\xxxxxxx\AppData\Local\Continuum\anaconda3\lib\site-packages\pynetdicom3-0.9.0-py3.6.egg\pynetdicom3\dul.py", line 466, in _is_transport_event
self._check_incoming_pdu()
File "C:\Users\xxxxxxx\AppData\Local\Continuum\anaconda3\lib\site-packages\pynetdicom3-0.9.0-py3.6.egg\pynetdicom3\dul.py", line 369, in _check_incoming_pdu
self.primitive = self.pdu.to_primitive()
File "C:\Users\xxxxxxx\AppData\Local\Continuum\anaconda3\lib\site-packages\pynetdicom3-0.9.0-py3.6.egg\pynetdicom3\pdu.py", line 899, in to_primitive
primitive.user_information = item.to_primitive()
File "C:\Users\xxxxxxx\AppData\Local\Continuum\anaconda3\lib\site-packages\pynetdicom3-0.9.0-py3.6.egg\pynetdicom3\pdu_items.py", line 1085, in to_primitive
primitive.append(item.to_primitive())
File "C:\Users\xxxxxxx\AppData\Local\Continuum\anaconda3\lib\site-packages\pynetdicom3-0.9.0-py3.6.egg\pynetdicom3\pdu_items.py", line 1778, in to_primitive
primitive.implementation_class_uid = self.implementation_class_uid
File "C:\Users\xxxxxxx\AppData\Local\Continuum\anaconda3\lib\site-packages\pynetdicom3-0.9.0-py3.6.egg\pynetdicom3\pdu_primitives.py", line 1153, in implementation_class_uid
raise ValueError("Implementation Class UID is an invalid UID")
ValueError: Implementation Class UID is an invalid UID
Steps to reproduce
How to reproduce the issue. Please include a minimum working code sample and
the anonymised DICOM dataset (if relevant).

Anaconda custom (64-bit)|,<default, Jun 28 2018, 11:27:44> [MSC v. 1900 64 bit <AMD64>]
pydicom 1.3.0.dev0
pynetdicom3 0.9.1

That's a weird one, I can't see anything wrong with the UID and everything seems to be working when I create a new ImplementationClassUIDNotification object and set it's value.
It probably shouldn't raise an exception there anyway, just logging an error message should be sufficient. I'll add a patch to change the behaviour, I'd appreciate it if you could test it and post the relevant portion of log output so I can hopefully figure out the original problem. It'll be something like:

E: "The Implementation Class UID Notification's 'Implementation Class UID' parameter value '[some value]' is not a valid UID"

Thanks @scaramallion ill test it for sure. That sounds like a good solution . The pynetdicom toolkit is great , have been using it in a production environment

OK, merged. Please test!

Still getting the same error but on a different line i think:
Now running versions:
print(pydicom.version)
print(pynetdicom3.version)
1.3.0.dev0
1.0.0.dev0
E: Implementation Class UID is an invalid UID
Exception in thread Thread-8:
Traceback (most recent call last):
File "C:\Users\xxxxxxx\AppData\Local\Continuum\anaconda3\lib\threading.py", line 916, in _bootstrap_inner
self.run()
File "C:\Users\xxxxxxx\AppData\Local\Continuum\anaconda3\lib\site-packages\pynetdicom3\dul.py", line 245, in run
if self._is_transport_event() and self._idle_timer is not None:
File "C:\Users\xxxxxxx\AppData\Local\Continuum\anaconda3\lib\site-packages\pynetdicom3\dul.py", line 466, in _is_transport_event
self._check_incoming_pdu()
File "C:\Users\xxxxxxx\AppData\Local\Continuum\anaconda3\lib\site-packages\pynetdicom3\dul.py", line 369, in _check_incoming_pdu
self.primitive = self.pdu.to_primitive()
File "C:\Users\xxxxxxx\AppData\Local\Continuum\anaconda3\lib\site-packages\pynetdicom3\pdu.py", line 901, in to_primitive
primitive.user_information = item.to_primitive()
File "C:\Users\xxxxxxx\AppData\Local\Continuum\anaconda3\lib\site-packages\pynetdicom3\pdu_items.py", line 1094, in to_primitive
primitive.append(item.to_primitive())
File "C:\Users\xxxxxx\AppData\Local\Continuum\anaconda3\lib\site-packages\pynetdicom3\pdu_items.py", line 1803, in to_primitive
primitive.implementation_class_uid = self.implementation_class_uid
File "C:\Users\xxxxxxx\AppData\Local\Continuum\anaconda3\lib\site-packages\pynetdicom3\pdu_primitives.py", line 1230, in implementation_class_uid
raise ValueError("Implementation Class UID is an invalid UID")
ValueError: Implementation Class UID is an invalid UID

Are you using pip? Did you reinstall in pip after updating to the current master?

Hmm, that doesn't seem right. You reinstalled in pip after updating to the current master?

Bravo! @scaramallion The only mistake was that you misspelled success!!
E: The Implementation Class UID Notification's 'Implementation Class UID' parameter value '<IP_ADDRESS>532.3320' is not a valid UID
C-ECHO request status: 0x0000
This was Sucesfull : (0000, 0900) Status

Bravo @scaramallion  !!
E: The Implementation Class UID Notification's 'Implementation Class UID' parameter value '<IP_ADDRESS>532.3320' is not a valid UID
C-ECHO request status: 0x0000
This was Correct : (0000, 0900) Status

Hmm, that's even weirder since the UID appears valid to me and the following works:
>>> from pydicom.uid import UID
>>> uid = UID('<IP_ADDRESS>532.3320')
>>> uid.is_valid
True
>>> if uid is not None and not uid.is_valid:
...     print('invalid')
... else:
...     print('valid')
...
valid

If I create a branch with additional debugging output do you feel comfortable downloading and running that so I can take a look at the raw response from the peer? It may be that there's an issue with decoding the A-ASSOCIATE-AC PDU.

Test branch is debug-uid (here)

Sorry @scaramallion not sure how i do that? Ive got the new dul.py file in use but im running Jupter notebooks and the output is still as below : Did you want me to try something different?
E: The Implementation Class UID Notification's 'Implementation Class UID' parameter value '<IP_ADDRESS>532.3320' is not a valid UID
C-ECHO Response: 0x0000

Try changing the log level after doing import pynetdicom3:
import logging
from pynetdicom3 import AE

LOGGER = logging.getLogger('pynetdicom3')
LOGGER.setLevel(logging.DEBUG)

ae = AE()
ae.add_requested_context('1.2.840.10008.1.1')
# Fill in whatever the addr and port should be
assoc = ae.associate(addr, port)
assoc.release()

@scaramallion is this ok?
I: Requesting Association
D: Request Parameters:
D: ====================== BEGIN A-ASSOCIATE-RQ =====================
D: Our Implementation Class UID:      1.2.8<IP_ADDRESS>80043.9.38<IP_ADDRESS>
D: Our Implementation Version Name:   PYNETDICOM_100
D: Application Context Name:    1.2.840.10<IP_ADDRESS>.1
D: Calling Application Name:    QEDRP3
D: Called Application Name:     ANY-SCP
D: Our Max PDU Receive Size:    16382
D: Presentation Context:
D:   Context ID:        1 (Proposed)
D:     Abstract Syntax: =Verification SOP Class
D:     Proposed SCP/SCU Role: Default
D:     Proposed Transfer Syntaxes:
D:       =Implicit VR Little Endian
D:       =Explicit VR Little Endian
D:       =Explicit VR Big Endian
D: Requested Extended Negotiation:
D: Requested Common Extended Negotiation:
D: Requested User Identity Negotiation: None
D: ======================= END A-ASSOCIATE-RQ ======================
D: Accept Parameters:
D: ====================== BEGIN A-ASSOCIATE-AC =====================
D: Their Implementation Class UID:    <IP_ADDRESS>532.3320
D: Their Implementation Version Name: BROKER992.0
D: Application Context Name:    1.2.840.10<IP_ADDRESS>.1
D: Calling Application Name:    QEDRP3
D: Called Application Name:     ANY-SCP
D: Their Max PDU Receive Size:  100000
D: Presentation Contexts:
D:   Context ID:        1 (Accepted)
D:     Accepted Transfer Syntax: =Implicit VR Little Endian
D: Accepted Extended Negotiation: None
D: Accepted Common Extended Negotiation: None
D: Accepted Asynchronous Operations Window Negotiation: None
D: User Identity Negotiation Response:  None
D: ======================= END A-ASSOCIATE-AC ======================
I: Association Accepted
E: The Implementation Class UID Notification's 'Implementation Class UID' parameter value '<IP_ADDRESS>532.3320' is not a valid UID
I: Sending Echo Request: MsgID 1
D: pydicom.read_dataset() TransferSyntax="Little Endian Implicit"
I: Received Echo Response (Status: Success)
I: Releasing Association

I: Requesting Association
D: Request Parameters:
D: ====================== BEGIN A-ASSOCIATE-RQ =====================
D: Our Implementation Class UID:      1.2.8<IP_ADDRESS>80043.9.38<IP_ADDRESS>
D: Our Implementation Version Name:   PYNETDICOM_100
D: Application Context Name:    1.2.840.10<IP_ADDRESS>.1
D: Calling Application Name:    QEJOE
D: Called Application Name:     ANY-SCP
D: Our Max PDU Receive Size:    16382
D: Presentation Context:
D:   Context ID:        1 (Proposed)
D:     Abstract Syntax: =Verification SOP Class
D:     Proposed SCP/SCU Role: Default
D:     Proposed Transfer Syntaxes:
D:       =Implicit VR Little Endian
D:       =Explicit VR Little Endian
D:       =Explicit VR Big Endian
D: Requested Extended Negotiation:
D: Requested Common Extended Negotiation:
D: Requested User Identity Negotiation: None
D: ======================= END A-ASSOCIATE-RQ ======================
D:   02  00  00  00  00  ad  00  01  00  00  41  4e  59  2d  53  43
D:   50  20  20  20  20  20  20  20  20  20  51  45  4a  4f  45  20
D:   20  20  20  20  20  20  20  20  20  20  00  00  00  00  00  00
D:   00  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00
D:   00  00  00  00  00  00  00  00  00  00  10  00  00  15  31  2e
D:   32  2e  38  34  30  2e  31  30  30  30  38  2e  33  2e  31  2e
D:   31  2e  31  21  00  00  19  01  00  00  00  40  00  00  11  31
D:   2e  32  2e  38  34  30  2e  31  30  30  30  38  2e  31  2e  32
D:   50  00  00  2f  51  00  00  04  00  01  86  a0  52  00  00  14
D:   31  2e  32  2e  31  32  34  2e  31  31  33  35  33  32  2e  33
D:   33  32  30  00  55  00  00  0b  42  52  4f  4b  45  52  39  39
D:   32  2e  30
D: Accept Parameters:
D: ====================== BEGIN A-ASSOCIATE-AC =====================
D: Their Implementation Class UID:    <IP_ADDRESS>532.3320
D: Their Implementation Version Name: BROKER992.0
D: Application Context Name:    1.2.840.10<IP_ADDRESS>.1
D: Calling Application Name:    QEJOE
D: Called Application Name:     ANY-SCP
D: Their Max PDU Receive Size:  100000
D: Presentation Contexts:
D:   Context ID:        1 (Accepted)
D:     Accepted Transfer Syntax: =Implicit VR Little Endian
D: Accepted Extended Negotiation: None
D: Accepted Common Extended Negotiation: None
D: Accepted Asynchronous Operations Window Negotiation: None
D: User Identity Negotiation Response:  None
D: ======================= END A-ASSOCIATE-AC ======================
I: Association Accepted
E: The Implementation Class UID Notification's 'Implementation Class UID' parameter value '<IP_ADDRESS>532.3320' is not a valid UID
I: Sending Echo Request: MsgID 1
D: pydicom.read_dataset() TransferSyntax="Little Endian Implicit"
I: Received Echo Response (Status: Success)
I: Releasing Association
I: Requesting Association
D: Request Parameters:
D: ====================== BEGIN A-ASSOCIATE-RQ =====================
D: Our Implementation Class UID:      1.2.8<IP_ADDRESS>80043.9.38<IP_ADDRESS>
D: Our Implementation Version Name:   PYNETDICOM_100
D: Application Context Name:    1.2.840.10<IP_ADDRESS>.1
D: Calling Application Name:    QEJOE
D: Called Application Name:     ANY-SCP
D: Our Max PDU Receive Size:    16382
D: Presentation Context:
D:   Context ID:        1 (Proposed)
D:     Abstract Syntax: =Verification SOP Class
D:     Proposed SCP/SCU Role: Default
D:     Proposed Transfer Syntaxes:
D:       =Implicit VR Little Endian
D:       =Explicit VR Little Endian
D:       =Explicit VR Big Endian
D: Requested Extended Negotiation:
D: Requested Common Extended Negotiation:
D: Requested User Identity Negotiation: None
D: ======================= END A-ASSOCIATE-RQ ======================
D:   02  00  00  00  00  ad  00  01  00  00  41  4e  59  2d  53  43
D:   50  20  20  20  20  20  20  20  20  20  51  45  4a  4f  45  20
D:   20  20  20  20  20  20  20  20  20  20  00  00  00  00  00  00
D:   00  00  00  00  00  00  00  00  00  00  00  00  00  00  00  00
D:   00  00  00  00  00  00  00  00  00  00  10  00  00  15  31  2e
D:   32  2e  38  34  30  2e  31  30  30  30  38  2e  33  2e  31  2e
D:   31  2e  31  21  00  00  19  01  00  00  00  40  00  00  11  31
D:   2e  32  2e  38  34  30  2e  31  30  30  30  38  2e  31  2e  32
D:   50  00  00  2f  51  00  00  04  00  01  86  a0  52  00  00  14
D:   31  2e  32  2e  31  32  34  2e  31  31  33  35  33  32  2e  33
D:   33  32  30  00  55  00  00  0b  42  52  4f  4b  45  52  39  39
D:   32  2e  30
D: Accept Parameters:
D: ====================== BEGIN A-ASSOCIATE-AC =====================
D: Their Implementation Class UID:    <IP_ADDRESS>532.3320
D: Their Implementation Version Name: BROKER992.0
D: Application Context Name:    1.2.840.10<IP_ADDRESS>.1
D: Calling Application Name:    QEJOE
D: Called Application Name:     ANY-SCP
D: Their Max PDU Receive Size:  100000
D: Presentation Contexts:
D:   Context ID:        1 (Accepted)
D:     Accepted Transfer Syntax: =Implicit VR Little Endian
D: Accepted Extended Negotiation: None
D: Accepted Common Extended Negotiation: None
D: Accepted Asynchronous Operations Window Negotiation: None
D: User Identity Negotiation Response:  None
D: ======================= END A-ASSOCIATE-AC ======================
I: Association Accepted
E: The Implementation Class UID Notification's 'Implementation Class UID' parameter value '<IP_ADDRESS>532.3320' is not a valid UID

So the Implementation Class UID section is:
52 00 00 14 31 2e 32 2e 31 32 34 2e 31 31 33 35 33 32 2e 33 33 32 30 00

Item type: 0x52
Reserved: 0x00
Item length: 0x00 0x14 (20)
Implementation class UID: everything else
It looks like the UID's trailing padding character 0x00 isn't being removed which makes it invalid.

@scaramallion thanks , is this a problem with the hosted device for having a trailing padded character?
Thanks for fixing that so quick - much appreciated

The relevant section of the DICOM Standard (Part 7, Annex D.3.3.2) says that:

The Implementation-class-uid field is structured as a UID as defined in PS3.5.

I interpret the word 'structured' here means that the value doesn't need to conform to the encoding rules for UIDs (ie that an odd length UID should be padded with a trailing null). Looking at dcmtk, it gives a raw implementation class UID notification
52
00
00 1b
31 2e 32 2e 32 37 36 2e 30 2e 37 32 33 30 30 31 30 2e 33 2e 30 2e 33 2e 36 2e 32

Note that the UID is odd length but doesn't have the trailing null byte, so perhaps the SCP you're using is slightly non-conformant, but I think to make double sure I'll change pynetdicom to remove any trailing null bytes from UID-like parameters.

The Application Context Name has a similar wording:

Application-context-names are structured as UIDs as defined in PS3.5

But as per Part 8, Annex F, application context names aren't encoded with any trailing padding.

No separator nor padding shall be present before the first digit of the first component or
after the last digit of the last component.

So based on this I believe that the Implementation Class UID is being encoded correctly, that just leaves handling cases where other DICOM implementations do add padding.

OK, I've added handling for decoding Implementation Class UID null padding, could you test with the current master 00e83d7 and confirm that you no longer get the "E: The Implementation Class UID Notification's 'Implementation Class UID' parameter value '<IP_ADDRESS>532.3320' is not a valid UID" log message?

@scaramallion yes exactly what i go after updating the master
E: The Implementation Class UID Notification's 'Implementation Class UID' parameter value '<IP_ADDRESS>532.3320' is not a valid UID

Strange, you shouldn't have that log message anymore. Can you double-check?

Ok, that's great, I'll close the issue.
Thanks for your help.


Publish libddprof as a Ruby gem on rubygems.org

What does this PR do?
This PR adds the scaffolding for taking a libddprof binary release from GitHub, and releasing it as a Ruby gem on rubygems.org.
This allows us to use libddprof on the Ruby profiler shipped in dd-trace-rb in a quite simple way: the released library has almost no code, and effectively just distributes the release files in a way that can be easily consumed by a Ruby library.
The release is mostly-automated (detailed instructions are on README.md)

We need to copy-paste the release sha256 for libbprof, as well as bump the version
And then run docker-compose run push_to_rubygems

I will be the one responsible for pushing new releases (on behalf of the profiling team), but still anyone can do a release if needed.
In the future we may want to look at triggering this from CI so that new libddprof releases are automatically pushed on rubygems.org.
Motivation
Having libddprof on rubygems.org and managed directly by the Profiling team (e.g. me) decouples releasing and distribution of the dd-trace-rb library from libddprof.
Additional Notes
I've already pushed libddprof 0.2.0 on rubygems.org using this PR: https://rubygems.org/gems/libddprof .
The current setup pushes two "variants" of the gem -- one with no binaries, which is used as a "no-op" dependency for platforms that libddprof does not support, and a "fat" one for x86-64 Linux, that includes both the glibc, as well as musl library binaries inside.
How to test the change?
Testing for this will be done on the consumer side: in the Ruby profiler, which will run libddprof on different platforms and Ruby versions.

I would suggest quickly adding CI (however crude) to build and test gem packages, like there is for libddwaf.

My plan was to handle the CI on the dd-trace-rb side, since we already have the multiple variants and libddprof can be validated in context with its user.
Since I don't expect libddprof to change often, other than updating the binaries shipped, I'm not entirely convinced it's worth the trouble of adding it on this side.