Razor Pages AddRazorRuntimeCompilation Options and opt.AdditionalReferencePaths failing to load types

I have an standalone tool that dynamically lets the user run a Web site out of an Arbitrary folder they provide at runtime. I want to allow them to add additional assemblies that they can reference in the arbtrary folder (or rather a ./PrivateBin folder below it).
I'm using code like the following to add the dlls in the folder (note WebRoot could be an arbitrary folder like /temp/MyLocalSite:
In ConfigureServices:
if (UseRazor)
{
    services.AddRazorPages(opt => { opt.RootDirectory = "/"; })
        .AddRazorRuntimeCompilation(
            opt =>
            {
                opt.FileProviders.Add(new PhysicalFileProvider(WebRoot));
                LoadPrivateBinAssemblies(opt);
            });
}

And to actually load the refs:
private void LoadPrivateBinAssemblies(MvcRazorRuntimeCompilationOptions opt)
{
    var binPath = Path.Combine(WebRoot, "PrivateBin");
    if (Directory.Exists(binPath))
    {
        var files = Directory.GetFiles(binPath);
        foreach (var file in files)
        {
            if (!file.EndsWith(".dll", StringComparison.CurrentCultureIgnoreCase) &&
               !file.EndsWith(".exe", StringComparison.InvariantCultureIgnoreCase))
                continue;

             opt.AdditionalReferencePaths.Add(file);
        }
    }
}

However when I then try to reference a type from these added assemblies I get the following error:

It looks like the assemblies and types are found (notice the version number is correct), but razor is not resolving the path?
Expected behavior
Load assemblies out of additionally specified arbtitrary folder.
Info

DotnetCore 3.0 RTM
Kestrel
Windows Console/Dotnet Tool app

Repo
The code and project for this can be found here if anybody wants to try this out:
https://github.com/RickStrahl/Westwind.AspnetCore.LiveReload/blob/master/LiveReloadServer/Startup.cs
To run this:

Set the runtime --WebRootPath  to a folder where Web or Razor Files exist
Add a PrivateBin folder and an assembly of your choice
Create a Razor Page that references some type in that assembly

I use Markdig.dll  (Markdig package since it's self-contained) like this:
 @Markdig.Markdown.ToHtml("**bold** Text",null)

@RickStrahl thanks for contacting us.
@pranavkm can you take a look?

@RickStrahl adding things as additional compilation references doesn't really add them to the load context at runtime. This would explain why the view \ Razor Page compiles correctly, but you get a runtime error since the type isn't present. Uncommenting this line of code: https://github.com/RickStrahl/Westwind.AspnetCore.LiveReload/blob/master/LiveReloadServer/Startup.cs#L228 got things working.
Would you expect adding additional compilation references to also make the assembly available at runtime?

Arrghhh - out of all the combinations I tried that was one I didn't. Sure enough that works!
Now, I'll say this: This is very counter intuitive. If you allow to add reference paths (which BTW, sounds like folder names, not actual assemblies) I would assume you also load those for me. I see no reason to reference assemblies, and then not have them loaded or accessible in your code. If there is some good reason for not loading may be an additional parameter to allow loading.
This would remove the logic for having to decide which load context might have to be used to actually load the assemblies from because the framework knows where it should load into.

Giving this a little more thought what I would like to see is rather than adding individual assemblies, is be able to specify a folder where additional references live and then have the startup code load all the assemblies stored there. I'm doing this in the simplistic code right now, but there are a number of tricky issues there when dependencies can't be found or they are loaded out of order. It would be nice if that could be handled at the framework level.
Edge case I know, but if you dynamically point at a disk location to serve content from then this would be the expected behavior - like AppDomain private bin startup specification, but delayed to do at runtime.
It's a possible security issue for assembly injection, but so is the code I'm running above that's already working, so making this easier won't really make the problem any worse.

I hadn't noticed your most recent comment before I created this PR - https://github.com/RickStrahl/Westwind.AspnetCore.LiveReload/pull/18 - but using an ApplicationPart gets you all the way, adding it both as a runtime and compile time component and additionally setting it up so MVC will perform, among other things, controller discovery on additional references.

I'm doing this in the simplistic code right now, but there are a number of tricky issues there when dependencies can't be found or they are loaded out of order. It would be nice if that could be handled at the framework level.

The framework has a pattern using load context for doing things like this - https://docs.microsoft.com/en-us/dotnet/core/tutorials/creating-app-with-plugin-support. Unfortunately, using a custom load context currently doesn't work well with MVC views \ Razor Pages since they're loaded in to the default load context. We haven't seen a lot of interest in making this work, which is primarily why we haven't prioritized it.

Yeah as I mentioned most of this is an edge case scenario. The ApplicationParts bit makes it even more relevant to preload the assemblies, because it's explicitly meant to bring in additional content and it looks like even with that you still need to load the assembly explicitly if you want to access it (which seems weird - wouldn't the framework probe for the Application Part interfaces it looks like, like controllers, views etc.)?
IAC, I think we have a solution here to close this issue.
The rest of my comments were more of an enhancement request related to expectations that I would have for the functionality based on the naming and for Application Parts at least its feature set.
Thanks for your @pranavkm!


Duplicate page loads

Hello. I occasionally seem to get the first page of results loading twice, and I've narrowed it down to this line of code.
It appears that the method is called twice (as expected, I think), but previousPageKeys isn't yet populated on the second call. The first page key only seems to get added to previousPageKeys after _load is called the second time.

I'm sorry, it was due to poor implementation on my part. In your load function, you check the list of previousKeys using the .contains function. It turns out that I was using a specific type for my keys and I didn't overload the == operator to get a good equality comparison. I used the equatable library for this, and everything works correctly now. Thanks for your library.

@pmkod Can you please share some code?


BPMNOrchestrator Clarifications

Hi,
Been exposed to BPMN/Camunda recently and thought it'd be useful to have a Azure Durable Functions hosted BPMN engine in my toolkit.  Bit of searching and came across your post on the durable task github about this project.
I'm not familiar enough with the underlying durable task library to detangle why this extends that instead of just Isolated Durable Functions, my current assumption is that because you can't distribute Durable Functions tasks?
If that's the case then I'm guessing this can't be hosted in Azure Functions easily?

Ultimately I think i want the following

where all the boxes are running on Azure Functions

ah i see azure storage doesn't support distributed workers, so I'd need to push them all through an API.

Hey @bzbetty
Yes, Durable Functions would not support distributed workers in the sense of "having each service/API implementing some activities or orchestrations and using durable functions as a communication layer between those services/APIs" like Zeebe/Camunda does.

Ultimately I think i want the following
You can achieve something pretty close to that with just DurableFunctions, but you will only have 1 type of worker (green box), which can be replicated to spread the load.

If what you want is the BPMN runner, the concept is in this file: https://github.com/lucaslorentz/durabletask-extensions/blob/main/samples/BpmnWorker/Orchestrations/BPMNOrchestrator.cs
That's a Proof of Concept level of maturity. It basically reads a BPMN file and evaluates it, mapping its concepts to durable task concepts, like timers, activities, parallelism, and so on. You can easily convert it to run on DurableFunctions.

After having a good play with your codebase I think it works quite well, yeah it's PoC and is missing some functionality but it's an amazing start. Also think it works better on durabletasks than it would with durable functions after playing a bit more.
it's a pity distributed workers don't seem to be on their roadmap.


[Question]: Duplicate Large Logo GIF Files in Repository

What is your question?
Description
The project currently contains two identical GIF files that are over 40 MB each and are in very high resolution. These files are used in different locations:

installer/client/gui/static/images/fabric-logo-gif.gif - Displayed as a 40x40 image in installer/client/gui/index.html.
images/fabric-logo-gif.gif - Displayed as a 400x400 image in README.md.

Problem
Having these large, duplicate GIF files in the repository leads to several issues:

Increased repository size: Each file is over 40 MB, making the repository unnecessarily large.
Redundancy: The same file is stored in two different locations.
Performance: Large files can slow down cloning and pulling from the repository.

Suggested Solutions

File Optimization:

Convert to AVIF: This format offers better compression (Only available on macOS 13 Ventura or later).
Compress GIF: Reduce the size of the GIF files while maintaining quality.
Convert to MP4: For the use case in README.md, consider using an MP4 video.

Image Size Reduction:

Ensure the images are resized to the actual display dimensions (400x400 and 40x40) before being added to the repository.

Asset Hosting:

Host these assets externally, such as on GitHub Pages, to avoid increasing the repository size.

Action Items

Evaluate and implement the best compression/conversion method.
Remove duplicate files and reference the optimized versions in both locations.
Consider hosting large assets externally and linking to them from the repository.

Impact
Addressing these issues will:

Reduce the repository size, making it faster to clone and pull.
Eliminate redundancy, making the repository cleaner.
Improve performance for contributors and users.

I apologize for not preparing a pull request for this issue. I'm not certain which direction the project maintainers would like to take. Once this is clarified, I am more than happy to assist with the implementation.

I don't know why you closed it without discussion since the gif is still 42 MB


Output number of each errors

I'm wondering if it's possible to get the following output (or similar):
SelectorFormat: Selector `spnFacetBox` should be written in lowercase with hyphens, 10;
IdSelector: Avoid using id selectors, 21;
FinalNewline: Files should end with a trailing newline, 7;

Where the number after the comma is the amount of errors found.

Implemented in #764.


feat: suggestion : created a demo purpose android chat application

Type: feat
Platform: all
Created a chat application using ionic and socket.io  at https://github.com/melvin0008/ionic-socketio-chat-client/ . How can i contribute this to the ionic community

Greetings @melvin0008!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

Nice! Send an email to<EMAIL_ADDRESS>I think this would make a great guest blog post. Tell her I recommended it be a guest blog post.

Thanks. :)


test with random fail on graph mode

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
windows 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):
binary
TensorFlow version (use command below):
v1.12.1-44421-g5e5730ba9d 2.4.0-rc0
Python version:
3.7.9
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
11.1
GPU model and memory:
960m cuda

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with:

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the current behavior
running the followin:
import tensorflow as tf
from tensorflow.python.framework import test_util

class WeirdTest(tf.test.TestCase):

  @test_util.run_in_graph_and_eager_modes
  def test_test(self):
    with self.cached_session():
      x = tf.random.normal([])
      self.assertAllClose(x, x)

if __name__ == '__main__':
  tf.random.set_seed(42)
  tf.test.main()

fails with assertion error (the x is not equal to itself!)
(eager mode works fine)
Describe the expected behavior
the test should pass
Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
attached above (not sure how to run tests on colab)
Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.
Failure
Traceback (most recent call last):
  File "C:\Program Files\Python37\lib\unittest\case.py", line 59, in testPartExecutor
    yield
  File "C:\Program Files\Python37\lib\unittest\case.py", line 628, in run
    testMethod()
  File "D:\python3_venvs\tf_2_4\lib\site-packages\tensorflow\python\framework\test_util.py", line 1196, in decorated
    f(self, *args, **kwargs)
  File "C:\Users\amitp\dev\github\amitport\federated-learning-research\federated_learning_research\v2\encoders\sign_encoder_test.py", line 11, in test_encode
    self.assertAllClose(x, x)
  File "D:\python3_venvs\tf_2_4\lib\site-packages\tensorflow\python\framework\test_util.py", line 1236, in decorated
    return f(*args, **kwds)
  File "D:\python3_venvs\tf_2_4\lib\site-packages\tensorflow\python\framework\test_util.py", line 2711, in assertAllClose
    self._assertAllCloseRecursive(a, b, rtol=rtol, atol=atol, msg=msg)
  File "D:\python3_venvs\tf_2_4\lib\site-packages\tensorflow\python\framework\test_util.py", line 2671, in _assertAllCloseRecursive
    (path_str, path_str, msg)))
  File "D:\python3_venvs\tf_2_4\lib\site-packages\tensorflow\python\framework\test_util.py", line 2606, in _assertArrayLikeAllClose
    a, b, rtol=rtol, atol=atol, err_msg="\n".join(msgs), equal_nan=True)
  File "D:\python3_venvs\tf_2_4\lib\site-packages\numpy\testing\_private\utils.py", line 1528, in assert_allclose
    verbose=verbose, header=header, equal_nan=equal_nan)
  File "D:\python3_venvs\tf_2_4\lib\site-packages\numpy\testing\_private\utils.py", line 840, in assert_array_compare
    raise AssertionError(msg)
AssertionError:
Not equal to tolerance rtol=1e-06, atol=1e-06
Mismatched value: a is different from b.
not close lhs = 0.4722212255001068
not close rhs = -1.5251041650772095
not close dif = 1.9973254203796387
not close tol = 2.525104165077209e-06
dtype = float32, shape = ()
Mismatched elements: 1 / 1 (100%)
Max absolute difference: 1.9973254
Max relative difference: 1.3096322
 x: array(0.472221, dtype=float32)
 y: array(-1.525104, dtype=float32)

Was able to reproduce the issue with TF v2.3, TF v2.4.0rc0 and TF-nightly. Please find the attached gist. Thanks!

@amahendrakar thanks for reproducing. I have to ask, what was the type:bug label removed? Should this be done differently?

@amitport  I believe this is not a bug. In graph mode, x = tf.random.normal([]) is returned as an operation, and self.assertAllClose will evaluate both input arguments and get the real value (if they are operations). Therefore, x is evaluated twice and returned different values. To fix it, you can try to use self.evaluate to get the exact value before asserting them.
https://colab.research.google.com/drive/19iN2UNQ4vJ0erH64gskuVkvdZZQO8F8D?usp=sharing

@WindQAQ thanks. I got to say this is very unintuitive and does not seem correct to me. I would expect assertion methods to evaluate the inputs' shared graph together. (DK if I l'm looking at bug or feature request, I just wonder if it could behave differently)
Is there a use case to evaluate the shared graph part twice?

Is there a use case to evaluate the shared graph part twice?

@amitport I would say that in most cases, we are evaluating a (shared) graph multiple times. For example, the ideal training pipeline in both eager and graph model is
# Here is how the (shared) graph is built
model = build_model()

# Here is how operations in the graph are evaluated
# including what kind of inputs are feeding and what kind of outputs are fetched
do_something(model)

Let me roughly expand the internal implementation of the example in this PR. What we are doing in graph mode is
x = tf.random.normal([])

with tf.Session() as sess:
    first = sess.run(x)
    second = sess.run(x)
    np.testing.assert_allclose(first, second)

Maybe it's more intuitive for users. If x is always evaluted as the same values in runtime, how can we get the different numbers without constructing new graphs?
Consider another common example that sometimes users add "noise" into the input.
input = tf.placeholder(foo)
noise = tf.random.normal([])
noisy_output = input + noise

# do something

with tf.Session() as sess:
    for i in range(10):
        y = sess.run(noisy_output, feed_dict=bar)

I think users would expect y to be different during iterations as they want noisy outputs even the graph is shared or unchanged.
Let me know if this answers your question. Thank you.

@WindQAQ Thank you for the example. I still think that these implementation details should not affect the test writer and assert methods could be smarter. That is, I realize it is implemented like this but it seems wrong for test case usage. (just look again at the first example in the issue, why would anyone expect this behavior?)

@amitport ade73c6af0957e7cbe420ae515c6a5d9acc6ecb3 updated assertAllEqual and similar functions to evaluate a and b in the same graph.  Closing this as fixed, but please feel free to re-open if you find that this change doesn't address your problem.


WelcomeImage.js unsupported image type

Describe the bug
when running the the even, it says  Unsupported image type, i tryed changing it to jpg but nothing
Error
A screenshot or the text block of the error if there is one.
To Reproduce
update the package to v14 or something
Expected behavior
no sure
Screenshots
already did
Additional info
nothing

Describe the bug
when running the the even, it says  Unsupported image type, i tryed changing it to jpg but nothing
Error
A screenshot or the text block of the error if there is one.
To Reproduce
update the package to v14 or something
Expected behavior
no sure
Screenshots
already did
Additional info
nothing

Ill need to see your code please.

Describe the bug
when running the the even, it says  Unsupported image type, i tryed changing it to jpg but nothing
Error
A screenshot or the text block of the error if there is one.
To Reproduce
update the package to v14 or something
Expected behavior
no sure
Screenshots
already did
Additional info
nothing

Ill need to see your code please.

sorry for the long wait, i forgot about this
  const bg = 'https://imgur.com/okIR1iY.png';
        const avatar = member.user.displayAvatarURL({ format: "png" });
        const title = "Welcome";
        const subtitle = member.user.tag;
        const footer = `You're the ${member.guild.memberCount}th member`;
        const color = '#ffffff';
        const options = {
            font: "sans-serif",
            attachmentName: `welcome-${member.username}`,
            title_fontSize: 80,
            subtitle_fontSize: 50,
            footer_fontSize: 30
        };
        const image = await welcomeImage(bg, avatar, title, subtitle, footer, color, options);
        */
        const welcome = new EmbedBuilder()
            .setTitle('New Member Joined')
            .setDescription(`
                Welcome ${member} to Franklin's Brain, We hope you enjoy is here :)
                Go get some Roles to spice it up a little
            `)
            .setColor('Orange')
            .setTimestamp()
        //.setImage(image)

that issue isnt coming from our package apparently, some people had some issues with something similar, its about canvas with discord.js or something like that, I'm not entirely sure, but I guarantee its not coming from the package, in case this never worked we will be making it as an API endpoint so you can fetch it without having to install canvas and deal with it.

that issue isnt coming from our package apparently, some people had some issues with something similar, its about canvas with discord.js or something like that, I'm not entirely sure, but I guarantee its not coming from the package, in case this never worked we will be making it as an API endpoint so you can fetch it without having to install canvas and deal with it.

oh ok, will i think i went throw the files of canvas and couldn't find anything, i'll have another look, if not we'll idk

Hello Franklin, found out its a discord.js error
here is a fix
https://stackoverflow.com/questions/73489458/discord-node-js-canvas-error-unsupported-image-type

Hello Franklin, found out its a discord.js error
here is a fix
https://stackoverflow.com/questions/73489458/discord-node-js-canvas-error-unsupported-image-type

oh cool thx, i'll have a look at it


There are untranslated strings in the 'New Folder' dialog for one file share

Storage Explorer Version: 1.15.0-dev
Build: 20200704.2
Branch: master
Language: All
Platform/OS: Windows 10/ Linux Ubuntu 18.04/ macOS Catalina
Architecture: ia32/x64
Regression From: Previous release(1.14.0)
Steps to reproduce:

Launch Storage Explorer.
Open 'Settings' -> Application (Regional Settings) -> Select '中文(简体)' -> Restart Storage Explorer.
Expand one storage account -> File Shares.
Create a new file share -> Click '新建文件夹'.
Check the strings in the popped dialog.

Expect Experience:
All strings are translated in the 'New Folder' dialog.

Actual Experience:
There are untranslated strings in the 'New Folder' dialog.

Verified all language on build 20201126.1. This issue doesn't reproduce. So we close it.


refactor(crates): change to workspace with separate crates

Description of change

Refactored code into separate crates in a workspace as preparation for #54
Note: I needed to do a few code changes to get proper separation. Ill add comments to those.
How has this been tested? (if applicable)

Build and executed
GH Actions might need to be refactored?

msrv has does not support workspace inheritance. Added a workaround, that needs to be updated for each crate.

Not sure why tests are failing. They work locally...

We're currently processing your upload.  This comment will be updated when the results are available.

Sorry, busy days, will try to review this soon.

No worries, same for me. Take your time :)

With a quick check I realized the tracing logs do not appear when I do cargo run, can you check?

Fixed it. Problem was, that it was using CARGO_PKG_NAME which was daktilo_lib causing daktilo not to log.
My first approach was to provide different log levels using the verbose flag, which was not the cause of the problem. Decided to keep it in, as it is an improvement over the old version.

FYI release-plz fails to find the embedded assets - I can look into it soon (before the release probably).
message: #[derive(RustEmbed)] folder '/home/runner/work/daktilo/daktilo/target/package/daktilo_lib-0.5.0/../../sounds' does not exist


Greedy Algorithms in Golang

I could do the rucksack problem!

@Duxy1996 By all means send in a PR


GH-3242: Clarify docs on custom Consumer/Producer factories

Fixes: #3242
Auto-cherry-pick to 3.1.x & 3.0.x

Auto-cherry-pick to 3.1.x & 3.0.x

The Antora docs are there starting with version 3.1.. So, this is not going to be back-port very easy.
I would stay just with 3.1.x back-port, but if you insist, feel free make a fix in the 3.0.x.
Mean while I'll merge this only to 3.1.x.


Android Studio Gradle Sync fails

The specified project directory '\node_modules\react-native-threads\examples\SimpleExample\android' does not exist. Need to remove stuff in build that wants stuff like /Users/tnuttall/Dev/react-native-threads/examples/SimpleExample/node_modules/react-native-threads/android/src/main/res
I am not sure who tnuttall is or why my build folder has files that need SimpleExample anyway. There are a lot of build files in \node_modules\react-native-threads\android\build\intermediates\blame\res\release\multi , \node_modules\react-native-threads\android\build\intermediates\incremental\mergeReleaseJniLibFolders, and \node_modules\react-native-threads\android\build\intermediates\incremental\packageReleaseResources that reference tnutall and SimpleExample

Hi, tnuttall is the author's pc name.

Ok so @Traviskn. For some reason react-native run-android generates files for you and the SimpleExample even though we don't need it.

@ethantran Are you running into this issue when installing the library from NPM? Can you describe the steps you went through to reproduce this error?

closing due to inactivity


set thumbnail photo for video

in this PR .. you can set thumbnail photo for a video using withThumbnail() method
\Dawson\Youtube\YoutubeFacade::upload($video_path,$params)
                           ->withThumbnail($image_path);

Notes :
because of using chaining i changed the way you want to get video id , you could use ->getVideoId() method , for thumbnail url you can use ->getThumbnailUrl() method.
full ex :
$youtube = \Dawson\Youtube\YoutubeFacade::upload($video_path,$params)
                           ->withThumbnail($image_path);

// get video id

$video_id = $youtube->getVideoId();

// get thumbnail url

$thumbnail_url = $youtube->getThumbnailUrl();

@JoeDawson any comment ?

Hi @ahmedash95 thanks for your PR. There's a few issues with this package at the moment, a lot of people seem to be unable to authenticate properly. I'm considering a rewrite of the plugin this weekend in the hopes of stabilising everything and will hopefully merge your PR once that's completed.

thanks for reply .. actually it works fine with me .. waiting you next week 😄

Hi @ahmedash95 I have made some amends that cause conflicts with your PR. Would you like to merge and resubmit your PR? Also I noticed that your PR includes the use of dd() - could I ask you to refrain from that as it wouldn't work well in a production environment.
Thanks!

@JoeDawson hi Joe everything is okay now 👌

Thank you @ahmedash95 - merged :)


Update Site Daily and General Metrics APIs for Learners Data

Description: This PR updates site daily metrics and site general metrics APIs for learners analytics data
JIRA:
https://edlyio.atlassian.net/browse/EDLY-2038
https://edlyio.atlassian.net/browse/EDLY-2039
Merge checklist:

[ ] All reviewers approved
[ ] Documentation updated (not only docstrings)
[ ] Commits are (reasonably) squashed
[ ] Translations are updated

Post merge:

[ ] Delete working branch (if not needed anymore)

Note: Please add screenshots for any viewable change.

https://github.com/edly-io/figures/pull/30/files#r511912292

@rehan99000 as per our discussion, we will keep this as it is and update the tooltip text to reflect the correct logic.


Fixes #284. Glob matching problem with subdirectories

CWLGlobServiceImpl was matching only filenames while traversing the file tree. Instead, whole path should have been matched against the given glob pattern.
However, due to complete path being absolute, classical glob matching patterns do not work. Instead, path starting from working directory and ending with filename should be used for matching. E.g.
Path: /home/halil/cwl-tests/simple2-2017-06-27-174426.530/root/header_step/insider/file1_header
Work: /home/halil/cwl-tests/simple2-2017-06-27-174426.530/root/header_step/

Path to use for matching: insider/file1_header

This matches a glob pattern like insider/*_header

New pull request #289


Training is cancelled when check_val_every_n_epoch > 1 and early stopping is used.

🐛 Bug
Using early stopping callback on metric emitted during validation with check_val_every_n_epoch > 1 flag of trainer leads to the training being aborted.
Reproduced here.
Expected behavior
Training should continue even when check_val_every_n_epoch > 1 and early stopping is used.
One has to think how the patience parameter  of the early stopping is handled in this case.
I see two possible directions

Interpreting patience as the number of validation runs which did satisfy the early stopping criterion.
This would mean e.g. choosing check_val_every_n_epoch = 2 and a patience = 10 would train for an additional 2 * 10 = 20 epochs which might not be what one would expect.

Interpreting patience as the number of training epochs since the last validation improvement which have to be at least completed until the training is stopped, i.e., if the number of training epochs since the last improvement as measured by validation is larger than patience then stop.

Environment
* CUDA:
	- GPU:
		- Tesla T4
	- available:         True
	- version:           10.1
* Packages:
	- numpy:             1.19.5
	- pyTorch_debug:     True
	- pyTorch_version:   1.7.0+cu101
	- pytorch-lightning: 1.2.0
	- tqdm:              4.41.1
* System:
	- OS:                Linux
	- architecture:
		- 64bit
		-
	- processor:         x86_64
	- python:            3.6.9
	- version:           #1 SMP Thu Jul 23 08:00:38 PDT 2020

Yes, just checked again and the error seems to be fixed with 1.2.10.
I would vote for naming it something along the lines of "validation runs" which would be quite intuitive for me.


Support specifying transitive unique field when finding unique field with a relation

Problem
Consider:
model User {
    uid Number @unique
    email String @unique
}

model ProfileValue {
    uid Number
    user User  @relation(fields: [uid], references: [uid])
    name String
    value String
   @@id([uid, name])
}

When using findUnique or upsert, you have to provide the uid_name field, which includes uid and name. However, it should be equivalently valid to provide an email instead of a uid, and have the uid field filled "automatically", since there is only one uid with that name, so it will resolve to a valid value.
Suggested solution
When one of the items in a unique input is part of a relation, any unique fields on the related model should be able to be provided in addition to the keyed relation field, letting the field be filled by a subquery or similar.
Alternatives
Split the upsert into separate queries (select the ID then upsert OR check if exists and update or insert), findFirst instead of findUnique.
Additional context
This re-opens #11845 which was converted to a discussion (#11845) without an actual resolution (this is an actual unimplemented feature, not a usage issue)

Can you please show an example of which Prisma Client code you think should work? How do you think the SQL database queries under the hood for this would need to look like?
PS: A valid version of the provided schema would look like this:
model User {
  uid          Int            @unique
  email        String         @unique
  ProfileValue ProfileValue[]
}

model ProfileValue {
  uid   Int
  user  User   @relation(fields: [uid], references: [uid])
  name  String
  value String

  @@id([uid, name])
}

As an aside (this is really a separate issue), but I'm also not a huge fan of the uid_name field being a thing - why not just require both uid and name fields be present?


Aliasing a package from a forked repository

Just to start off, the problem may not actually be with elm-github-install, but instead with the elm native naming convention.
I'm using a library in elm-community that I am also contributing to (without write access, just via PRs). This repo implements a native module, and that works.  I own an unmerged branch that I need features from.
My issue is that native modules force the naming _packageOwner$packageName$path$Native_Thing.
When I try "mrozbarry/packageName" in my elm-package.json, I can install, but elm can't find the correct native code, which is "_originalAuthor$packge_name" and elm is looking for "_mrozbarry$package_name"
"dependencies": {
  "mrozbarry/canvas": "..."
},
"dependency-sources": {
  "mrozbarry/canvas": {
    "url"<EMAIL_ADDRESS>    "ref": "draw-image"
  }
}

elm-github-install throws an error when I try aliasing like this:
"dependencies": {
  "elm-community/canvas": "..."
},
"dependency-sources": {
  "elm-community/canvas": {
    "url"<EMAIL_ADDRESS>    "ref": "draw-image"
  }
}

$ elm-install
Resolving packages...

  The source of package elm-community/canvas is set to https://github.com/mrozbarry/canvas which would
  be install to elm-stuff/mrozbarry/canvas. This would cause a conflict
  when trying to compile anything.

  The name of a package must match the source url's path.

  elm-community/canvas <=> mrozbarry/canvas

Any reason your git clone couldn't do something like git clone gitUrl packageOwnerAndName?

Yes this is a limitation at the moment which I looked into while ago and made some headway in this PR: https://github.com/gdotdesign/elm-github-install/pull/14 also see the discussion here https://groups.google.com/d/msg/elm-discuss/eusppnDgN-A/mf1L08NRCAAJ
The code works and I could successfully install elm-lang/core from a fork and even from locally in the file system. I didn't have time to finish (specs, documenation, etc), I will probably get to it soon™
If you really need this feature right now then I suggest the you install the gem from that branch.

I've made progress on this in #23. I've rewritten how sources are used, so this and using local packages will be available when it releases.

This is now available in https://github.com/gdotdesign/elm-github-install/releases/tag/v1.0.0


add description for signature process/algorithms

Done - content has been added in the interim to the docs-reorg branch.


Click with waitForEvents cause Timeout error

I'm using Gauge and Taiko (v1.3.2).
Gauge version: 1.4.3
Commit Hash: f98dd40

Plugins:
html-report (4.1.4)
js (2.4.0)
screenshot (0.1.0)

I have a form that have a Save button. I want click the Save button and make sure that the form is fully submitted. So I used the following code:
await click('Save' , {
            waitForEvents: ['firstMeaningfulPaint'],
            navigationTimeout:40000});

The Gauge timeout in js.properties is:
test_timeout = 40000
What actually happen:
The Save button being clicked and the form submit. Also it navigate to the next page and wait there. The page loads and you can see the elements of the page. after some moments its return this error:

Does the script work from Taiko's repl ? How much time does it take in that case ?


Moving the component to another parent in one request cycle breaks the client side

Simple scenario: the xterm console is shown in an draggable container. That draggable container is moved to another position, which is internally just a remove from parent A and attach to parent B. This results in an "empty" tag on the client side (it looses basically its xterm instance as it seems).
Removing and adding in two request cycles works, also simply hiding and showing it using setVisible.
The following example shows the issue in a simplified manner:
    public XTermView() {
        XTerm xterm = new XTerm();
        xterm.writeln("Hello world.\n\n");
        xterm.setWidth("500px");
        xterm.setHeight("250px");

        VerticalLayout left = new VerticalLayout(xterm);
        VerticalLayout right = new VerticalLayout();

        add(new Button("Switch", event -> {
            Component parent = xterm.getParent().orElse(null);
            if (parent == left) {
                right.add(xterm);
            } else {
                left.add(xterm);
            }
        }));

        SplitLayout layout = new SplitLayout(left, right);
        layout.setSplitterPosition(50);
        layout.setOrientation(SplitLayout.Orientation.HORIZONTAL);
        add(layout);
    }

The resulting element looks like this after using the Switch button:

As a workaround, a two-phase switch can be initiated with executeJs/then:
 add(new Button("Two phase switch", event -> {
      Component parent = xterm.getParent().orElse(null);
      if (parent == left) {
          xterm.getElement().removeFromParent();
          getElement().executeJs("return").then(later->right.add(xterm));
      } else {
        xterm.getElement().removeFromParent();
        getElement().executeJs("return").then(later->left.add(xterm));
      }
  }));


ProaData.B2BClient version <IP_ADDRESS>

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.2 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Duplicate of #71597

Close with reason: See feedback on https://github.com/microsoft/winget-pkgs/pull/71597;

Close with reason: See feedback on #71597;


prevents keyword duplication and over all keyword destruction

We should check later whether or not the user deleting the keyword is the last user with that keyword, so we can delete it (no orphaned keywords)

@gosborn @sarahsalder woo! good morning!


Changed Parameters Of Model

I increased the size of the embeddings to add more capacity to this model.

This is a test comment to inspect the payload
/full-test-run foo bar
more things here. Putting an image below


Disable instrumentation for ES client versions with native instrumentation

Fixes #3235
https://github.com/elastic/elasticsearch-java/pull/652 introduced native instrumentation into the ES client.
We need to disable agent instrumentation for versions with native instrumentation.
The native instrumentation introduced the class  co.elastic.clients.transport.instrumentation.Instrumentation in the ES client. This PR adds a check for the presence of that class.

TODO: we need to make dependabot stop bumping the ES client versions from 8.10+ on


New SOAR App - Libraesva

Name of the app
Libraesva
Integration
App integrates with Libraesva Email Security service
About
Currently app is capable of searching for an email with certain parameters and blocklisting certain resource utilizing Libraesva API

The new repo https://github.com/splunk-soar-connectors/libraesva has been created.


Brevitas golden model and FINN-ONNX mismatch for QuantConv2d layer

I am attaching a minimal reproducible example for Issue #304, where the Brevitas golden model and the exported FINN-ONNX model execution do not match when the bias=True within a simple model with a QuantConv2d layer. However, if the bias=false, the expected output produced by the Brevitas golden model and the exported FINN-ONNX model match.
The attached file is a jupyter notebook that creates two simple model with a single QuantConv2d layer. One of the models has the bias=True and the other one has the bias=False. The notebook is meant to offer a minimal reproduce testing scenario for this error.
goldenModel_FINN_mismatch.zip

See above -- there was an issue exporting layers with bias enabled from Brevitas which is now fixed. PR #339 is now merged into dev that uses the correct commit hash from Brevitas for this.


Add Tau

Problem Statement
Tau is missing.
Evidence or User Insights
The Tau Manifesto
The manifesto makes a strong argument and presents use cases for using Tau. Unfortunately Tau is not an option.
Proposal
Add a toggle so that Tau can be used instead of Pi.
Goals
Use the Scientific mode "2nd" button toggle to replace Pi with Tau.
Non-Goals
Aside from the Pi button being toggled with Tau, no other changes to functionality or default behavior.
Low-Fidelity Concept
This is a simple change to the Scientific mode:
The "2nd" button will toggle Pi to Tau, and vice versa
Requested Assignment
I have already implemented this locally and would like to contribute.

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.

Thanks for the suggestion! It certainly seems like Tau would be a good constant candidate to add. There is a similar issue open tracking generally adding more constants (#123). What are your thoughts on possibly including Tau as part of that proposed drop down?

Thank you for the consideration. Tau is  special because it serves as a replacement for Pi. This was the motivation of having a toggle replacement of Pi in the UI. All Pi formulae have a Tau equivalent.
If Pi is moving to this new Constants bar, then I'd like Tau right next to it.

@grochocki what can I do to move forward with this? It really makes the most sense to have a toggle between Pi and Tau especially since this is such a small change. I hope that works for your roadmap but you hold the keys please let me know.

I have certainly learned a lot about tau over the past week or so, but much of what I have learned can be summarized by this paragraph from Wikipedia.

In 1958 Albert Eagle proposed replacing π by τ (tau), where τ = π/2, to simplify formulas. However, no other authors are known to use τ in this way. Some people use a different value, τ = 2π = 6.28318..., arguing that τ, as the number of radians in one turn, or as the ratio of a circle's circumference to its radius rather than its diameter, is more natural than π and simplifies many formulas. Celebrations of this number, because it approximately equals 6.28, by making 28 June "Tau Day" and eating "twice the pie", have been reported in the media. However, this use of τ has not made its way into mainstream mathematics.

That last sentence is key. Outside of the Tao Manifesto, which arguably lays out a reasonable case for using tau, there does not seem to be any widespread adoption in academia (e.g., textbooks, journal articles, etc.). Tau also does not appear in the list of frequently used mathematical constants. Any constant we might consider adding to the main keyboard would need to be as prominently used as pi (e.g., we already have support for Euler's number, and support for imaginary numbers is proposed with #613).
Of course, there are lots of constants that we could support, which is why we have #123. A very small select set of the most common constants will continue to live on the main keyboard, but all others should be added as part of the larger set of constants in the proposed constants tab. If the scales tip in favor of tau as a true replacement of pi in the future, we are definitely open to revisiting this, but I am going to close this as a duplicate of #123 for now. Thanks again for submitting this idea!

@grochocki understood, and thank you for your review.


Storing flows

We might want to store flows for later inspection. My idea was to use sqlx with sqlite for this. sqlite allows us to write complex queries that are optimally executed and to store everything in one file. sqlx is nice because it verifies SQL queries at compile-time.

get/insert for flows/messages is implemented. We might want to add code for artifacts later.
This currently uses sqlx's built-in migration feature, but we might want to roll our own.
I will close this for now.


HIVE-24786: JDBC HttpClient should retry for idempotent and unsent http methods

What changes were proposed in this pull request?
JDBC http client retries are applied only during initial connection establishment and is not applied for broken connection cases (idle timeout triggering tcp connection reset). This PR replaces custom retry handler with default retry handler with user specified maxRetries (jdbc param). The default retry handler retries all exceptions exception (InterruptedIOException, UnknownHostException, ConnectException, SSLException) and does not retry non-idempotent http methods which is sufficient for handling the broken connection cases.
Why are the changes needed?
To make jdbc http client more resilient for socket disconnections by the intermediate proxies.
Does this PR introduce any user-facing change?
No. maxRetries is already user facing parameter which is reused.
How was this patch tested?
Tested in Azure with LB timeout set to default 4 mins for idle timeout and nginx reverse proxy (5 min connect, read, write timeout) fronting hiveserver2.

Connected using beeline and executed a query.
Left beeline untouched for >5 mins.
Watched wireshark for TCP RST
After TCP RST is received, resubmitted the query beeline
Without this patch step 4) returned "Broken Pipe" and if the query takes >5 mins just for execution "Read timeout" is observed. With this patch both were not observed.

@szlta Thanks for the review!
I tested the PR on different environment and ran into different set of issues (likely because of OS difference and JDK difference). Encountered a hang issue under some scenarios (socket read timeout). The hang is because there was no timeout defined on the socket created by httpclient and hence it will infinitely wait until server can write something to it (which will never happen because LB disconnected the connection to server). HIVE-12371 provided a way to specify socketTimeout via JDBC param but that was applied only for binary transport and not for http.
I made couple of changes to the PR and tested it to make sure the hang never happens

For httpmode use the socketTimeout specified by the user via jdbc param
The default retry handler does not retry InterruptedIOException. Since SocketTimeoutException (read timeout) is subclass of InterruptedIOException it wasn't retried. So added a custom retry handler with list of classes that will be retried in addition to the default retry (idempotent and unsent requests).
Could you please take another look at the PR as it has changed?


Add non WARP Client Setup path to DNS instructions

Which Cloudflare product does this pertain to?
Cloudflare One
Existing documentation URL(s)
https://developers.cloudflare.com/cloudflare-one/policies/filtering/initial-setup/dns/
Section that requires update
Connect to Gateway
What needs to change?
Add instructions for how to connect to Gateway without the WARP Agent (or link to other content we already have)
How should it change?
Add instructions for how to connect to Gateway without the WARP Agent (or link to other content we already have)
Additional information
No response

Hey Kyle, would you mind creating a Jira ticket? Usually we use GitHub Issues for external requests.

We have a section on connecting locations: https://developers.cloudflare.com/cloudflare-one/policies/filtering/initial-setup/dns/#connect-dns-locations
What other on-ramps were you thinking of?

Closing this issue because we already link to DoH instructions.


manifest-file-extension parsing extension improperly

Source:
<link rel="manifest" href="/.well-known/favicons/manifest.webmanifest">

Scan:
Error    The file extension should be 'webmanifest' (not 'well-known/favicons/manifest')  manifest-file-extension

Expect it to pass, clearly well-known/favicons/manifest is not the file extension. I'm guessing the logic assumes there could be only one . in the path instead of parsing from the end, or using a url parsing lib.
Environment
$ sonarwhal -v
v0.17.0
$ node -v
v8.8.0
$ npm -v
5.6.0

sonarwhal configuration
all defaults
URL for which sonarwhal failed
https://registration.redcross.ca
Output
Error    The file extension should be 'webmanifest' (not 'well-known/favicons/manifest')  manifest-file-extension

https://github.com/sonarwhal/sonarwhal/issues/32
@alrra

@willfarrell Good catch!


Give the option to fix centered layout width.

Previously the centered layout in zen mode had a fixed width, meaning that if you resize the window the centered section will remain the same as long as it still fits in the resized window. The behaviour now is for the centered section width to be proportional to the size of the window. See below:

On resize, old behaviour/new behaviour:

This is a fair feature request. I know @joaomoreno wanted something like this.

I agree, having the editor window resized every time you toggle sidebar is quite annoying for me. It would also be nice to bring back switching back to full-size layout when having more than 1 editor open side by side.

@isidorn I think this issue is closely related to #53989

So does this mean that you're not going to address this issue any time in the near future? "Issue grooming" seems to simply be a euphemism for getting rid of issues that nobody wants to fix.

Can this be reopened? I feel this would be a great change

Yes I can reopen.

@isidorn Thanks a lot!

Any plans on implementing this?

I love the centered layout feature and I really want to be able to use it. Unfortunately it's unusable as is and I've avoided it for the last couple of years waiting for it to be addressed.
First and foremost, the size must be absolute, not relative. The current behavior is honestly just broken. I can't think of a sane reason why it would be relative. Look at the results:
Sane:

Not sane:

This is also being asked for here: https://github.com/microsoft/vscode/issues/71361#issuecomment-491854562
Second, we need settings for whether to use centered layout and to set the default size. I put in a request for this years ago and it was inexplicably renamed to a completely different request and closed as a duplicate.
P.S. I'm now noticing it's even not centered properly? That's weird...

This is a fair feature request and we are open for a lean and good PR that fixes this
Code pointer https://github.com/microsoft/vscode/blob/b7a9bfa7f9e8d34414b1319ff214eb8a2f55c524/src/vs/base/browser/ui/centered/centeredViewLayout.ts#L65

Any plans on implementing this?

Looks like somebody already implemented a PR for it (#104942), but the vscode people decided to close it for petty reasons.

I miss this feature for almost a year. Any news on when it will be implemented?

Rather than have it be fixed size, I'd rather have the editor adjust to the rulers you define for each file extension. I work in a project where .cc files can have 80 chars, .java can have 100 and my debug logs are unbounded. So whenever I switch I have to adjust the width manually.

Does anyone know any workaround until someone finds time to implement it in vscode natively? Any extension that solves this problem?

Any updates?

I wrote my comment here back in 2018... now is 2022 and this annoying issue is still here, sadly. Ok, the dev team doesn't want to add such a change to the main code base. But at least five an API to create an extension for those you needs this.

My main issue is the terminal pane. I used to put the terminal pane side by side the editor, and pop it on and off when I needed it. The problem was that in the occasions where I needed both, the editor width was shrunk down to an unusable width due to the centering issue. So, everytime I had to manually switch back and forth between normal and center modes.
Until yesterday, which I gave up and put the terminal at the bottom, so I no longer need to frequently switch to normal mode. It's a pain though, since I lose half the screen height when I need both the editor and the terminal 😟
Please make this an option. There's already an option that automatically turns off the center mode when you open another "window", but it doesn't apply to the terminal pane.

@FancyBanana I am going to go out in a limb here and ask if https://github.com/microsoft/vscode/issues/60900 can be fixed as I find it even more relevant than a fixed width, but subjective of course.


Typo in interface JobDetail

A method is named isConcurrentExcutionDisallowed while it should be named isConcurrentExecutionDisallowed.

If it is ok, I would like to submit the PR to fix this

This is a duplicate - PR from somebody else  for this was already merged.


[Wolf - Steam] Could not open connection to X

Hello everyone,
I'm trying to use Wolf to stream a Steam docker (ghcr.io/games-on-whales/steam:edge) but the container crashes at startup with the following error:
src/steamexe/updateui_xwin.cpp (339) : Could not open connection to X
src/steamexe/main.cpp (1223) : failed to initialize update status ui, or create initial window

I deep dived a bit in the documentation as well as in the GOW Steam docker startup script but I still cannot figure it out. Looking at the error, I suppose it's related to Wayland the fact that, since I'm using sunshine, I do not have a physical display attached to the docker but rather a pipeline).
I'll be glad if anyone has any idea :)
Here's the relevant logs :
DEBUG | Host app state folder: /mnt/user/appdata/wolf/7875106462968103757, Steam, creating paths
DEBUG | [STREAM_SESSION] Create wayland compositor
DEBUG | [WAYLAND] Creating wayland display
DEBUG | [STREAM_SESSION] Create virtual audio sink
DEBUG | [GSTREAMER] Starting video producer: appsrc is-live=true name=wolf_wayland_source ! queue ! interpipesink name=7875106462968103757_video sync=true async=false max-bytes=0 max-buffers=3
DEBUG | [GSTREAMER] Starting audio producer: pulsesrc device=\"virtual_sink_7875106462968103757.monitor\" server=\"/tmp/sockets/pulse-socket\" ! audio/x-raw, channels=2, rate=48000 ! queue ! interpipesink name=\"7875106462968103757_audio\" sync=true async=false max-bytes=0 ma>
DEBUG | WAYLAND_DISPLAY=/tmp/sockets/wayland-1
INFO  | RTP server started on port: 48100
INFO  | RTP server started on port: 48200
DEBUG | Setting up wolf_wayland_source
DEBUG | [PULSE] Created virtual sink: 6
DEBUG | [WAYLAND] Start feeding app-src
DEBUG | /dev/dri/renderD128 vendor: NVIDIA Corporation
DEBUG | [DOCKER] Using fake-udev, creating /mnt/user/appdata/wolf/7875106462968103757/Steam/udev/data
DEBUG | /dev/dri/renderD128 vendor: NVIDIA Corporation
INFO  | NVIDIA_DRIVER_VOLUME_NAME not set, assuming nvidia driver toolkit is installed.
DEBUG | [RTSP] received command OPTIONS
DEBUG | [RTSP] received command DESCRIBE
DEBUG | [RTSP] received command SETUP
DEBUG | [RTSP] received command SETUP
DEBUG | [RTSP] received command SETUP
DEBUG | [RTSP] received command ANNOUNCE
DEBUG | [RTSP] Moonlight requested video format HEVC
DEBUG | [RTSP] Adjusted video bitrate to 7308 Kbps
DEBUG | Video session<PHONE_NUMBER>968103757, waiting for PING...
DEBUG | Audio session<PHONE_NUMBER>968103757, waiting for PING...
 ]
  **** Configure default user ****
  Setting default user uid=1000(retro) gid=1000(retro)
  userdel: ubuntu mail spool (/var/mail/ubuntu) not found
  Auseradd: warning: the home directory /home/retro already exists.
  useradd: Not copying any file from skel directory into it.
  Setting umask to 000
 Ensure retro home directory is writable
  Ensure XDG_RUNTIME_DIR is writable
  DONE
  [ /etc/cont-init.d/15-setup_devices.sh: executing... ]
   **** Configure devices ****
  Exec device groups
  Adding user 'retro' to groups: gow-gid-18,root
  DONE
  [ /etc/cont-init.d/30-nvidia.sh: executing... ]
  [ /etc/cont-init.d/init-gamescope.sh: executing...  ]"}
  *** DBus started ***
  *** Bluez started ***
  bluetoothd[141]: Bluetooth daemon 5.72
  bluetoothd[141]: src/adapter.c:adapter_init() Failed to access management interface
  bluetoothd[141]: src/main.c:main() Adapter handling initialization failed
  NetworkManager started ***\n
  Launching the container's startup script as user 'retro'
  Starting Steam with DISPLAY=
  steam.sh[1]: Running Steam on ubuntu 24.04 64-bit
  steam.sh[1]: STEAM_RUNTIME is enabled automatically
  {setup.sh[240]: Steam runtime environment up-to-date!
  /home/retro/.steam/debian-installation/ubuntu12_32/steam-runtime/run.sh: line 85: steam-runtime-identify-library-abi: command not found
  run.sh[256]: steam-runtime-identify-library-abi --ldconfig-paths failed, falling back to ldconfig
  steam.sh[1]: Can't find 'steam-runtime-check-requirements', continuing anyway
  Init: Installing breakpad exception handler for appid(steam)/version(0)/tid(297)
  src/steamexe/updateui_xwin.cpp (339) : Could not open connection to X
  tid(297) burning pthread_key_t == 0 so we never use it
  WARNING: setlocale('en_US.UTF-8') failed, using locale: 'C'. International characters may not work.
  Startup - updater built Jan 13 2024 00:51:43
  Startup - Steam Client launched with: '/home/retro/.steam/debian-installation/ubuntu12_32/steam' '-nominidumps' '-nobreakpad' '-bigpicture'\n","stream":"stderr","time":"2024-11-17T14:13:43.397168675Z"}
  QILocalize::AddFile() failed to load file \"public/steambootstrapper_english.txt\"
  Init: Installing breakpad exception handler for appid(steam)/version(1.0)/tid(297)
  Bassert_20241117141342_4.dmp[302]: Uploading dump (out-of-process)
  /tmp/dumps/assert_20241117141342_4.dmp
  _src/steamexe/main.cpp (1223) : failed to initialize update status ui, or create initial window
  src/steamexe/updateui_xwin.cpp (339) : Could not open connection to X
  _src/steamexe/main.cpp (1223) : failed to initialize update status ui, or create initial window

Additionnal information
Host OS : Unraid 7.0.0-beta4 (based Slackware 15.0 x86_64 (post 15.0 -current))

Thank you, it solved this issue. However, the container is still crashing for an other reason (looks like Sway cannot work with proprieraty drivers) . I'll investigate on my side and open another issue if needed.
Closing this one 👍

However, the container is still crashing for an other reason (looks like Sway cannot work with proprieraty drivers)

Sway definitely works with the Nvidia GPU using the proprietary drivers, feel free to post the logs or to open up a new issue!


[WEB-861] Consent Management feature flag

📲 What
This PR makes the necessary code changes to add a new 'Consent Management' feature flag.
🤔 Why
With iOS 14.5+, iPadOS 14.5+, we need to receive member's permission through the Apple's AppTrackingTransparency framework in order to track them or access their device’s advertising identifier.
This flag will gate the work that will be done to implement this.
More details around this initiative can be found here
🛠 How
Boilerplate code that adds a new case to the OptimizelyFeature enum and then a helper function that checks user defaults before returning the value from the Optimizely SDK. Finally a case for the new consentManagementEnabled feature case in OptimizelyFeatureFlagTools
👀 See
Optimizely Feature Flag menu in our Beta Tools
| After 🦋 |
|  |
✅ Acceptance criteria

[x] Make sure the new 'Consent Management` flag shows in "Optimizely Feature Flags" in our Beta tools screen
[x] New feature flag created in optimizely

⏰ TODO

[ ] Update snapshot tests on intel macbook

@msadoon I'll just need some help with the snapshot tests

@msadoon definitely. i'll slack you to setup a time to discuss.

SwiftFormat found issues:

File
Rules

Library/OptimizelyFeature+HelpersTests.swift:95:1
warning: (indent) Indent code in accordance with the scope level.

Library/OptimizelyFeature+HelpersTests.swift:104:1
warning: (indent) Indent code in accordance with the scope level.

  Generated by :no_entry_sign: Danger


fix(autoware_auto_msgs_adapter): fix predicted path test

Description
Close #5742
Fixed a bug that was accessing out of range with vector in predicted path test.
Tests performed
colcon test --packages-select autoware_auto_msgs_adapter
Effects on system behavior

Not applicable.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@xmfcx Hi, would you review this PR?


fix(gatsby-ipfs-web-wallet): Bumping version to v1.11.4

:tada: This PR is included in version 1.0.0 :tada:
Your semantic-release bot :package::rocket:


Use correct registries to integrate with new packages workflows

Manually backporting #8538 to release-0.19 branch.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm
/approve

/approve


Any plans for test explorer?

Hi.
Are there any plans on making a test explorer out of this plugin? So that, you know, one could write and/or debug a specific test or selected tests, and the progress was visible.

Do you have an example to illustrate what you mean?

Sure. Basically it's an interface to interact with particular tests, to see which ones were successful/failed, and which were ran or not.
Here's a picture of what it looks like in Visual Studio:

If you run TestSuite it will output the result of the whole test suite and in theory that data could be parsed and the user presented with an explorer though easier said than done. In some ways this overlaps with #691 as both involve parsing and processing the output of the tests.
Currently there are no plans to implement either as the focus of this plug-in is making it easy to run tests and how the output of the tests are displayed is very much up to the user based on what strategy is selected, however a strategy that presents a test explorer is a possibility.

I would love such feature.


amazon: Change wait to require machines to be instantiated

When listing machines on Amazon, it was possible to get a response
between the assignment of cloud IDs but before fields such as the
machine's IP and size were filled in on Amazon's end. This caused
the cluster's join to fail, booting twice as many machines as needed.
The Amazon wait() function now requires machines to have both a
cloud ID and a size before returning.

Updated with a test, confirmed to fail before patch.

LGTM!


FZ: fix currently selected zone tracking

Summary of the Pull Request
m_objectLocationWinEventHook was uninitialized. also explicitly initialize some other fields, even though they're static and should be zero-initialized nevertheless, it just looks better 🙂

/azp run


Add showing time

Hi.
Would it be possible to add ability to show time as well as day, month, etc in menubar? Currently, to see time - I need to use native clock. The problem with this approach is that it's take too much space in menubar. And I would like to reduce that space usage.

Itsycal can show the time. In Preferences/Appearance, you can set a custom datetime pattern. For example, "h:mm".
See https://mowglii.com/itsycal/datetime.html for info on patterns.


Remove react/react-native peerDependencies

We're not keeping these up to date and realistically nobody is going to install this library without having both react and react-native, so there's not much of a point in keeping these here. Also they create warnings when running yarn in a new expo project.

Thanks, @brentvatne!
Maybe we'd like to add this change to` RNGH?


Unexpected error creating device

I can login to  the wallbox using ACE Service Installer and also the app (credentials are ok and using admin for the integration) however setting up the integration in HA (core 2024.2.5) fails with an unexpected error creating device error.
The Alfen Eve Single Pro has firmware 6.4.0 (I need the load balancing feature).
This error originated from a custom integration.
Logger: custom_components.alfen_wallbox.config_flow
Source: custom_components/alfen_wallbox/config_flow.py:42
Integration: alfen_wallbox (documentation)
First occurred: 5 March 2024 at 19:46:19 (2 occurrences)
Last logged: 10:16:57
Unexpected error creating device
Traceback (most recent call last):
File "/config/custom_components/alfen_wallbox/config_flow.py", line 42, in _create_device
await device.init()
File "/config/custom_components/alfen_wallbox/alfen.py", line 42, in init
await self.async_update()
File "/config/custom_components/alfen_wallbox/alfen.py", line 64, in async_update
await self._do_update()
File "/config/custom_components/alfen_wallbox/alfen.py", line 93, in _do_update
self._status = AlfenStatus(response_json, self._status)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/alfen_wallbox/alfen.py", line 205, in init
for prop in response["properties"]:
~~~~~~~~^^^^^^^^^^^^^^
TypeError: 'NoneType' object is not subscriptable

Same here with an additional error.
Core
2024.3.0
Supervisor
2024.02.1
Operating System
12.0
Frontend
20240306.0
`Logger: homeassistant
Source: custom_components/alfen_wallbox/config_flow.py:77
integration: Alfen Wallbox (documentation)
First occurred: 11:45:43 (1 occurrences)
Last logged: 11:45:43
Error doing job: Task exception was never retrieved
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/discovery_flow.py", line 104, in async_start
await gather_with_limited_concurrency(FLOW_INIT_LIMIT, *init_coros)
File "/usr/src/homeassistant/homeassistant/util/async.py", line 218, in gather_with_limited_concurrency
return await gather(
^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/util/async_.py", line 216, in sem_task
return await task
^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 1050, in async_init
flow, result = await self._async_init(flow_id, handler, context, data)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 1080, in _async_init
result = await self._async_handle_step(flow, flow.init_step, data)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 502, in _async_handle_step
result: FlowResult = await getattr(flow, method)(user_input)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/alfen_wallbox/config_flow.py", line 77, in async_step_zeroconf
return await self._create_entry(user_input[KEY_IP], None, user_input[CONF_USERNAME], user_input[CONF_PASSWORD])
~~~~~~~~~~^^^^^^^^
TypeError: 'ZeroconfServiceInfo' object is not subscriptable`

I can login to the wallbox using ACE Service Installer and also the app (credentials are ok and using admin for the integration) however setting up the integration in HA (core 2024.2.5) fails with an unexpected error creating device error. The Alfen Eve Single Pro has firmware 6.4.0 (I need the load balancing feature).
This error originated from a custom integration.
Logger: custom_components.alfen_wallbox.config_flow Source: custom_components/alfen_wallbox/config_flow.py:42 Integration: alfen_wallbox (documentation) First occurred: 5 March 2024 at 19:46:19 (2 occurrences) Last logged: 10:16:57
Unexpected error creating device Traceback (most recent call last): File "/config/custom_components/alfen_wallbox/config_flow.py", line 42, in _create_device await device.init() File "/config/custom_components/alfen_wallbox/alfen.py", line 42, in init await self.async_update() File "/config/custom_components/alfen_wallbox/alfen.py", line 64, in async_update await self._do_update() File "/config/custom_components/alfen_wallbox/alfen.py", line 93, in _do_update self._status = AlfenStatus(response_json, self._status) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/config/custom_components/alfen_wallbox/alfen.py", line 205, in init for prop in response["properties"]: ~~~~~~~~^^^^^^^^^^^^^^ TypeError: 'NoneType' object is not subscriptable

I had the same issue. Leave the 'admin' username as it is and just enter your password + friendly name. Then it logged in and worked :)

I actually used the admin user (may be this was not entirely clear from my initial message). Even tried with aligning the friendly name with the real name of the box to no avail. I have removed the integration from HACS, rebooted and started over again, the error persists but I also noticed a warning message in the log:
This error originated from a custom integration.
Logger: py.warnings Source: custom_components/alfen_wallbox/alfen.py:84 integration: alfen_wallbox (documentation) First occurred: 12:52:35 (1 occurrences) Last logged: 12:52:35
/config/custom_components/alfen_wallbox/alfen.py:84: RuntimeWarning: coroutine 'ClientSession._request' was never awaited self._session.request(

Perhaps update to latest HA version. For reference I manually uploaded my repo files into HA via Samba. Did a reboot before setting up the plugin. Have you updated your Eve firmware?

Perhaps update to latest HA version.
I always keep HA up to date
Core
2024.3.0
Supervisor
2024.02.1
Operating System
12.0
Frontend
20240306.0
For reference I manually uploaded my repo files into HA via Samba.
I downloaded mine via HACS and verified afterwards (all files are there and in the right directory)
Did a reboot before setting up the plugin.
Yes, did that too
Have you updated your Eve firmware?
It is on the before last official version (6.4.0) since 6.5.0 is buggy - 6.6.0 is expected to come out shortly

I signed out of the EVE app and tried again - no luck, same error.
This time I also checked the wallbox log file which shows no errors other than a ''TLS handshake failed (0x4e) for fd=2'' error. This made me think in giving it a go with another browser, I had been trying with the latest version of Brave and decided to switch over to MS Edge. The error remains however the HA log entry is completely different this time around:
This error originated from a custom integration.
Logger: custom_components.alfen_wallbox.config_flow
Source: custom_components/alfen_wallbox/config_flow.py:42
integration: Alfen Wallbox (documentation)
First occurred: 14:31:26 (1 occurrences)
Last logged: 14:31:26
ClientError
Traceback (most recent call last):
File "/usr/local/lib/python3.12/asyncio/sslproto.py", line 560, in _do_handshake
self._sslobj.do_handshake()
File "/usr/local/lib/python3.12/ssl.py", line 917, in do_handshake
self._sslobj.do_handshake()
ssl.SSLWantReadError: The operation did not complete (read) (_ssl.c:1000)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.12/site-packages/aiohttp/connector.py", line 992, in _wrap_create_connection
return await self._loop.create_connection(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/asyncio/base_events.py", line 1147, in create_connection
transport, protocol = await self._create_connection_transport(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/asyncio/base_events.py", line 1180, in _create_connection_transport
await waiter
File "/usr/local/lib/python3.12/asyncio/selector_events.py", line 1065, in write
n = self._sock.send(data)
^^^^^^^^^^^^^^^^^^^^^
ConnectionResetError: [Errno 104] Connection reset by peer
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/config/custom_components/alfen_wallbox/config_flow.py", line 42, in _create_device
await device.init()
File "/config/custom_components/alfen_wallbox/alfen.py", line 38, in init
await self.async_get_info()
File "/config/custom_components/alfen_wallbox/alfen.py", line 96, in async_get_info
response = await self._session.request(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/aiohttp/client.py", line 578, in _request
conn = await self._connector.connect(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/aiohttp/connector.py", line 544, in connect
proto = await self._create_connection(req, traces, timeout)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/aiohttp/connector.py", line 911, in _create_connection
_, proto = await self._create_direct_connection(req, traces, timeout)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/aiohttp/connector.py", line 1235, in _create_direct_connection
raise last_exc
File "/usr/local/lib/python3.12/site-packages/aiohttp/connector.py", line 1204, in _create_direct_connection
transp, proto = await self._wrap_create_connection(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/aiohttp/connector.py", line 1000, in _wrap_create_connection
raise client_error(req.connection_key, exc) from exc
aiohttp.client_exceptions.ClientConnectorError: Cannot connect to host <IP_ADDRESS>:443 ssl:<ssl.SSLContext object at 0x7f5b9bfc6150> [Connection reset by peer]

I'm not sure in that case I'm afraid. I'm no expert, just tinkerer. It worked for me on Chrome with uBlock and Adguard running.

No worries, I appreciate your thinking along ;-)

Interestingly, I found I had to log out of the charger in Eve and log out of the App entirely in order for HA to connect to it. Seems it only allows one logged in connection at a time. 73's :)

Were you able to solve it? @N36 what firmware are you running?
Our Alfen will be installed next week, I wonder what firmware I should use

Were you able to solve it? @N36 what firmware are you running?
Our Alfen will be installed next week, I wonder what firmware I should use

Latest Alfen firmware version seems to work fine for me. I'm just activating it with a Loxone RFID chip so no backend account transactions happening. The HA plugin eventually logged in to it when I disconnected the app and logged out of the Alfen account entirely.

I found the culprit and I have to admit the error was entirely due to myself. In the installation instructions in HACS it says search for Alfen using the + .... Doing that actually installs another fork!
What needs to be done is to add a custom repository first in HACS, reboot  HA  and continue with the installation of the integration thereafter.
Having done that, provide the correct ip address of the Alfen wallbox and the associated credentials (do not change the default user-id admin) and presto, no more errors, the box is now seen by HA.
The devil is always in the detail....


Make it work with api v2.

Tests will fail because the API is not yet fully deployed.

@sindresorhus @kevva ready to be merged, the api has been deployed!

Super! :)

🌟


Malicious Attacks: Model Poisoning (Backdoor + Gradient Attack)

This PR adds support for backdoor and gradient attacks.
This PR also refactors the malicious algo config and how other configs interact with it.

Config refactoring looks good!


Fix daily scores again, 10 mile rides got 10 extra

This resolves https://github.com/freezingsaddles/freezing-model/issues/20

Deployed to prod, all is well now.


Command Data parser for Network Identifier with Network Discovery Command

Hi,
I have been reviewing the xbee api for a while now and it is fantastic ! :-)
However I have found something I seem to be unable to do with the api, I am running a simple xbee setup with one coordinator and one router in API mode, I am successfully sending a ND command, and the response is received. However what I am unable to achieve through the api is to be able to parse the Network Identifier value that is received in this packet in command data:
{ type: 151,
id: 1,
remote64: '0013a20040cc03bc',
remote16: '0000',
command: 'ND',
commandStatus: 0,
commandData: < Buffer .... >
}
Is this possible with the api or is it not available yet?
Thank you,
Jon

Hey Jon,
Sorry for taking so long to get back to you!
We can parse that buffer with the parseNodeIdentificationPayload function (see lib/frame-parser.js).
This is automatically done for regular AT requests, but not yet for remote ones. I'll write a quick patch that you can try out that should leave you with a nodeIdentification property in that packet, that contains the parsed information.
Best,
Jan

Hi Jan,
That has worked perfectly thank you, I can see the node identifier within the node identification packet now.
Will this be something that gets merged into the master at somepoint? I can imagine it being useful for others.
Jon

Seems like a safe merge that shouldn't break anything for people. I will
merge it later today!
On Jan 4, 2016 4:30 PM, "Jon Hardman"<EMAIL_ADDRESS>wrote:

Hi Jan,
That has worked perfectly thank you, I can see the node identifier within
the node identification packet now.
Will this be something that gets merged into the master at somepoint? I
can imagine it being useful for others.
Jon
—
Reply to this email directly or view it on GitHub
https://github.com/jankolkmeier/xbee-api/issues/35#issuecomment-168707144
.

merged fix in 0.4.0


Feat: Set Reminders on the day of the contest

Once -cfnext command is exectued, the client must get an option to set reminders.
Features

A command -remind must be present with the parameter  which means contest id.
An alternative would be using emoji reactions.
Emoji reactions can be used to set the time as well as to choose the contest

I'll look into it.

@chmey any updates?

@DaKeiser I'm on it.  Do check out @NikiAdivi's implementation in dev-niki branch. It was a quick one. Do create a PR once you solve the issue.

Oh. What's left to do then after @NikiAdivi implementation?

If you have a better or an efficient solution, do create a PR. But there is a lot of improvement needed on @NikiAdivi implementation. Currently reminders can be set only on calling the command explicitly. It would be an ideal situation if the bot drops in emojis 1, 3, 5 along with the -cfnext embed, so that an user can directly click on that to set a reminder

Or even if you have made some progress, don't mind to share it by creating a PR


Sample app scripts are in share folder but specific to the sample app.

The script jumpbox-setup.sh is dedicated to the sample app Fine Collection Service and is nor shared nor reusable for other workload. It should be in the Fine Collection Service sample app folder.

thanks you are right. Can you do a PR?


Distributed Caching support

First off, awesome library, its very useful and configurable.  On another note, distributed caching isn't supported in this library.  I implemented my own provider but then realized a few things:

MicroCache implements a lot of locking which is not needed for distributed cache (not a problem because we can implement IMicroCache)

MicroCache uses LazyLock object and in a distributed environment/cache there is truly no value being stored in cache, just an empty LazyLock object with ICacheDetails.  I have referenced exact lines for reference:  https://github.com/maartenba/MvcSiteMapProvider/blob/master/src/MvcSiteMapProvider/MvcSiteMapProvider/Caching/MicroCache.cs#L70-L83

WIth the available interface ICacheProvider, it is also tied very strongly to LazyLock which would be easier to implement distributed if used ISiteMap/T instead of LazyLock

https://github.com/maartenba/MvcSiteMapProvider/blob/master/src/MvcSiteMapProvider/MvcSiteMapProvider/Caching/ICacheProvider.cs
I've already forked and started to move things around as needed for my use, just wanted to run that by the community to see what everyone's thoughts are. I can have a code sample available pretty soon as well.  I hope I made the issue/suggestions clear, but please comment and let me know if this does sound crazy, I'm sure what you did was intentional, if you could shine some light on what your reasoning was and what we may run into by altering this?
Simplest route we could potentially move the lazy.Get call to prior to adding to cache:
https://github.com/maartenba/MvcSiteMapProvider/blob/master/src/MvcSiteMapProvider/MvcSiteMapProvider/Caching/MicroCache.cs#L73

Thanks for the feedback.
The inspiration for the cache was based on the article linked to in the header of the file. As per the article:

The LazyLock class implements the lazy lock pattern and ensures that a lock is only aquired to populate the value field. This ensures reads are non-blocking. The micro cache implementation effectively stores a lazy lock for each cache key which is responsible for the fetching based on an activator (or value factory).

Although I went through at least 3 different variations of the design before settling on this one, I didn't really consider what would happen if you tried to implement a distributed cache. I agree that this is something that should be addressed.

MicroCache uses LazyLock object and in a distributed environment/cache there is truly no value being stored in cache, just an empty LazyLock object with ICacheDetails. I have referenced exact lines for reference: https://github.com/maartenba/MvcSiteMapProvider/blob/master/src/MvcSiteMapProvider/MvcSiteMapProvider/Caching/MicroCache.cs#L70-L83

The lazy lock is empty when it is stored. Then the Get method is called which either returns the data or fires the method to retrieve it if it is not yet populated. The idea is to release the write lock as early as possible on the cache so other threads can read (potentially other) items in it. That way for a multi-tenant website that uses a SiteMap per tenant won't have to block traffic to all of the SiteMap instances (hence all of the sites) if one of them is in the process of fetching data.
I am not sure there is a way to support both a non-blocking cache and a distributed cache with the same design, but if you have any ideas about how to implement a solution I would be happy to see them.

I think the main thing we would need to do is focus on getting the ISiteMap, SiteMap to a simple serializable object which makes this translate to just a hierarchy of nodes.  We would need to pull out the injection of services/providers into this object and move that into another service/provider that would be responsible for building the hierarchy.
Those would need to be separated because as it is right now those objects cannot be serialized.  Maybe the locking becomes the priority of the ICacheProvider itself if necessary.
I started to go through the code and currently I am working to simplify the SiteMap object to a basic hierarchy and move the other responsibilities.  I have nothing solid at the moment but just want to give you a heads up on the direction I am trying to go with this.
I feel this approach would be the starting ground for supporting both in memory and distributed cache.

One thing to keep in mind is MVC6/DNXCore50 support. One option on the table is just to let MvcSiteMapProvider be phased out with MVC5 and create a new design (either one that supports MVC3+ or one that only supports MVC6).
Since we are basically talking about a new design here (ISiteMap into a Model-like object), would you be interested in contributing to the effort? Some things I would like to see happen:

Make a fluent API that can plug into OWIN/MVC6 startup using extension methods and get rid of the whole internal/external DI approach.
Make it possible to add nodes dynamically rather than having to reload the whole SiteMap to see new ones.
Get rid of the Dynamic Node Provider interface, and make (something like) ISiteMapNodeProvider into a first-class feature.
Remove the XML Sitemap (for search engines) and make it into a separate project/package.
Redesign so that the node-to-action method relationship is always one-to-one, and provide a way for the end user to provide alternate titles, descriptions, route values, etc. when the node is accessed. This will be easier to configure and we won't run into scalability limitations nearly as quickly.
Separate the node data from the node index data so they can be cached separately.
Decouple the nodes from the hierarchy so they don't necessarily have a direct object reference to their parent and children, but have an alternative way to get to the parent and children quickly.
Cache the nodes in segments (possibly 1-1 with ISiteMapNodeProvider) with individual timeouts so the only time the nodes have to be loaded at once is during application startup and so data that is likely to become stale can have a shorter timeout than data that is not.

Joe Audette (the one who opened the MVC6 topic) has a big head start - he has a working implementation on DNX Core 50. There are several ideas he came up with that I like, such as using a ViewModel to adjust the title, description, and URL for each request.
However, his implementation is part of a "do-it-all" framework, which is not an approach I agree with. Also, he has not yet implemented the caching and currently has all of his nodes in trees of TreeNode<T> structures, which I don't think is the best solution for caching (and definitely not thread-safe). Finally, his only options for node configuration are XML and JSON which in my opinion should not be considered the first choice for configuration in a modern app - there should be a DI-friendly configuration API instead.
One thing I have been considering is to use a decorator pattern around the MVC ActionSelector and then building a unique string to identify the action. While the nodes are loading, the ActionSelector could be run to generate the same string. Then the decorator could intercept the action chosen by the `ActionSelector

I agree on moving forward on an MVC6 approach, but do feel that there are some out there that may need the MVC5 changes as well.  Maybe I will put together a separate forked MVC5 version with my suggested changes in the meantime for my usecase and others who may need the same.
I don't mind helping out with this initiative, let's try to coordinate the ideas and get a roadmap planned.  I agree with removing the Dynamic Node provider and bringing them together as a first class feature (Dynamic Node provider has been very useful and I'm sure no one would want that completely gone)

MVC5?? There already is support for MVC 5.


Weighted quad: algebraic-logarithmic weight functions

This is a still a work in progress, but I think it's pretty close so I wanted to share a draft for now and hopefully get some ideas on how to fix the remaining bugs.
Current issues:

In the adaptive_quadrature method, we map the integration interval into [-1, 1]. When we need to check if we should apply the weighted quadrature rule instead of the default rule, the code checks if the singular point is in the interval or not. The problem is that the singular point has not been mapped to [-1, 1], so this check usually fails or doesn't work right. When I remove the mapping, things seem to be working (for the most part, see next issue).
There have been some simple test cases that are not passing because I'm getting a nan for the error. This is because integral_mmn in

https://github.com/f0uriest/quadax/blob/c4544fbc6c8be47459489e2995a2098437944c4e/quadax/fixed_order.py#L123-L125
can (incorrectly) be negative, so that when it's taken to the 1.5 power in
https://github.com/f0uriest/quadax/blob/c4544fbc6c8be47459489e2995a2098437944c4e/quadax/fixed_order.py#L132-L136
it gives nan.
I think integral_mmn becomes negative because taking the absolute value of the integrand ruins the scheme I'm using for weighted quadrature. When we have weight functions present in addition to $f(x)$, self.wh contains the influence of the weight function. If we want the integral of the absolute value of the whole integrand (as we do in integral_mmn), then we need to consider the absolute value of the weight functions too. But this becomes a "different" weight function and so self.wh is incorrect here (and I'm not sure there is a simple way to compute wh in this case).
Other small things

For now, there is no dedicated adaptive quad method for weighted function. You have to use the general adaptive_quadrature method with the correct weight_function object.
I should probably go back and cite quadpack in a couple places, particularly in the methods that compute the modified Chebyshev moments.
need to check formating in docstrings of math equations

Hey sorry it's been a while, I've been in the process of moving and starting a new job.
@f0uriest I was hoping you might have some thoughts on this PR! In particular, if you have any pointers on understanding where the quadrature error estimate comes from, it might help me come up with a way to compute the error when using weight functions.
For the first issue, scipy doesn't allow infinite bounds for this type of weighted quadrature, so maybe we could follow them. An easy way out is to only apply the mapping to the interval (-1, 1) if either of the bounds are infinite, and throw an error if we want to use certain weight function in this case. Otherwise just use the provided intervals.

No worries, I've also been dealing with moving and starting a new job :)
The abstract rule stuff is now merged into main (you'll need to change the base branch before reopening the PR, it seems I can't do it from my end, and the PR auto-closed when the old base branch was merged/deleted).
I think it's probably fine to only allow weight functions for finite intervals like scipy. I think in theory it should be possible to get it to work correctly for infinite intervals but we can figure that out later.
For the most part the error estimate comes from using two different methods to estimate the integral: one low order rule, and another high order rule. For efficiency all the ones I've implemented are so called "nested rules" where the low and high order rules share the same nodes so function values can be re-used. The difference between the low and high order estimates gives an estimate of the error, which is then used in the outer adaptive_quadrature loop to decide which interval to sub-divide.
The other parts integral_abs and integral_mmn are mostly there to account for the possibility of roundoff error (ie, if the error estimate is less than machine precision it can't really be trusted). The particulars of the code are basically taken from quadpack, I haven't found any rigorous justification or explanation but they seem to work well in practice.


How to construct multiprogress with variable?

I see the multiprogress construct method as:
indicators::MultiProgress<indicators::ProgressBar, 3> bars;
but I want to init it using a variable, like this:
int a = 3;
indicators::MultiProgress<indicators::ProgressBar, a> bars;

Can I achieve this? i think dynamic_process has good method like push_back?

int a = 3;
indicators::MultiProgress<indicators::ProgressBar, a> bars;

The above is not possible in C++. MultiProgress is designed for the case where you know the number of progress bars at compile time.
For all other cases, I recommend you use DynamicProgress

This is exactly what I was trying to do, I considered using processor directives, I would appreciate this feature.


You will need to

fork this repo on GitHub
clone your fork locally
create a branch locally

yep, then do the steps above in that directory 👍

@eddiejaoude Thanks :)

@bagladivyang03 great 👍

Install husky using "automatic installer" from their docs https://typicode.github.io/husky/#/?id=automatic-recommended
Add 2 hooks https://typicode.github.io/husky/#/?id=add-a-hook
i. run linter
ii. run tests
iii.  run commit linter https://github.com/conventional-changelog/commitlint#getting-started

Actually, I have installed husky using "automatic installer" after that I m getting confused about how to do that run linter and run test part i.e adding hooks
Could u pls help @eddiejaoude

@eddiejaoude I made a PR could u pls check that for this issue could u pls check :)

Theres a PR for this. Is this PR still relevant or do we need to do this again?

I think best probably to do it again 👍 we need this before people start to contribute

closed with #91


If there is a huge amount of photos stored locally it can jam the pi

Because of the way the backupper script works, if too many files build up internally it can jam up the internal works and crash everything
for instance if you have 20 GB of files and only 18 GB free on the pi, and it tries to make a local backup it stalls

fixed with latest backup files script


Legacy number id update + color pallette

nice

Added colors but there's some issues with newer version of vox caused by pyvox.parser

Thanks for your contributions.
This project never really worked as pyvox.parser never worked with newest magicavoxel format. I started reverse-engineering the new format in kaitaistruct (as the definition file they had for magicavoxel at the time was also outdated) but I didn't finish that for one reason or another, I don't remember. If you want to, I'd check if there are any newer python libraries that can handle .vox files, quick google yielded these results (ignoring py-vox-io and numpy-vox-io):

https://github.com/midstreeeam/MidVoxIO - looks like it's maintained
https://github.com/AstrorEnales/blender_magicavoxel - for this you'd have to rip out .vox parsing code out of the plugin so it might not be the best choice but it also looks like it's maintained

Also from a quick glance seems like kaitaistruct hasn't got an update to the .vox format definition. Personally I'd try using MidVoxIO first.


Allow configuration of SSL certificate verification

Requests provides the verify kwarg to allow a user to:

True: verify the host's certificate using Requests' internal certificate bundle
False: do not verify the host's certificate
/path/to/cert/bundle.pem: verify the host's certificate using a local certificate bundle file

The last two options are requried when using self-signed certificates, i.e. when using SSL inside an organisation.
HypChat does not currently support the verify argument:
class HypChat(object):
    def __init__(self, token, endpoint='https://api.hipchat.com'):
        self._requests = _requests(auth=BearerAuth(token))

A workaround exists for using a local certificate bundle file: export the REQUESTS_CA_BUNDLE environment variable. But it would be great if HypChat explicitly supported the verify kwarg if provided by the user.

I agree, this should be explicitly exposed, but at least you can work around it:
hipchat._requests.verify = False

I'm happy to accept pull requests on this.


Forbidden status and prompted to enter credentials when attempt to push new package with same ID as an existing one

I was attempting to push a new package to Nuget and each time was getting a Forbidden status back, and was prompted for my credentials to connect to the site.  After hours of trying to figure out the problem it turned out the issue was the ID of the new package I was attempting to push duplicated the ID of an existing package from a different author.
The error message and the prompt for credentials seems misleading in this situation.  I suggest an explicit error message, if possible, along the lines of "Unable to push the package, ID duplicates the ID of an existing package."

@AnotherSadGit - Thank you for reporting, I see that the issue is reproducible with the nuget.exe. I have created another tracking issue with the Client team. Will close this bug since it is not directly actionable from the gallery.

Its possible that the gallery isn't returning correct error message to the client. Reopening the bug.

The forbidden result was by design and at the moment the prescribed behavior is that the client could show the returned message.


Release policy and stability expectations?

From the frequency of releases for this project, it appears that your continuous integration process generates a new release on most, if not all, commits.
Can you document the release policy for this package? I'm hesitant to include a package that publishes so quickly without any indication of what's changed or fixed.

Part of the issue is not excluding .pyc files in .gitignore

With all the noise of the pyc files committed a dozen times each day, it's hard to discern what the changes are from release to release.

Hello @truthdoug,
We have deployed an update to our SDK release process that removes the noisy .pyc files. These files should no longer be included in SDK releases going forward, so the diffs will be much easier to review for substantive changes.
Thank you again for your feedback!
Best,
Justin Sinclair
Files.com

ah, much calmer!


pythonPackages.markdown: add missing setuptools to propagatedBuildInput

Motivation for this change
Currently setuptools isn't automatically propagated to python packages (https://github.com/NixOS/nixpkgs/pull/68314).
This fixes the following ImportError on application startup:
/nix/store/qh7ndfsar3icmwqbiwcla7pc8x1133vg-python2.7-Markdown-3.1.1/bin/markdown_py README.md > README.html.new
Traceback (most recent call last):
  File "/nix/store/qh7ndfsar3icmwqbiwcla7pc8x1133vg-python2.7-Markdown-3.1.1/bin/.markdown_py-wrapped", line 7, in <module>
    from markdown.__main__ import run
  File "/nix/store/qh7ndfsar3icmwqbiwcla7pc8x1133vg-python2.7-Markdown-3.1.1/lib/python2.7/site-packages/markdown/__init__.py", line 25, in <module>
    from .core import Markdown, markdown, markdownFromFile
  File "/nix/store/qh7ndfsar3icmwqbiwcla7pc8x1133vg-python2.7-Markdown-3.1.1/lib/python2.7/site-packages/markdown/core.py", line 29, in <module>
    import pkg_resources
ImportError: No module named pkg_resources
make: *** [Makefile:53: README.html] Error 1

Also needs to be backported to 19.09. Also probably fixes the build of bees on 19.09 (https://hydra.nixos.org/build/100463113#tabs-summary).
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

backported in 0195953af12c1013014b01f5cfbb94b888525107