Broken link in Community examples on Global Styles features

Description
When I open the link for Community examples from Global Styles feature, I expected to open the example of the project that using Global Styles feature but it seems that the link is broken.

Hey @aroyan, thanks for opening that issue! :smiley:
Unfortunately it looks like the whole domain habd.as from that repo is down and also the the GitHub user 'Josh Habdas' who seems to be the owner of that repo and domain is deleted like you can see in this history: https://github.com/boletindigital/blog/commits/master
@noobinthisgame @nikolovlazar
Im not sure if we especially need a external repo linked in here. So I guess it is a better approach to extend the example on the page if needed. But tbh I do not think it is necessary either, so that we can remove that whole section 'community examples'. Let me know what you think we should do here.

Yes, we can remove the Community Examples section.


Release for Kinetic

Hi,
Just requesting if this can get built for kinetic.
Thanks!

@mikepurvis  ^, not for sure who else to ask.

https://github.com/ros/rosdistro/pull/13416


Fix incorrect end bracket, moved to after pub fn extra_modules

rtic/rtic-macros/src/codegen/bindings/esp32c3.rs
Closing bracket for mod esp32c3  was previously on line 244, which alienated:
pub fn extra_modules(_app: &App, _analysis: &SyntaxAnalysis) -> Vec {
vec![]
}
Moved closing bracket for mod esp32c3 to end of file, now extra_modules is accessible.

With #906 merged this PR maybe could be repurposed to instead run ‘rustfmt’ on the esp32c3 files? Looks like at least the bindings needs a touchup.
Best of luck with that esp32c3 project 😃

Cleaned up the codegen bindings esp32c3.rs file with rustfmt.


DWI masking: "Masking appears to have failed. Using a backup method"

The discussion in another thread about brain masks reminded me to look into a report from another user about the warning "Masking appears to have failed. Using a backup method". I also see this pretty consistently in HCP processing, but the masks output at the end of processing usually look pretty good.
Looking into the code,
https://github.com/PennLINC/qsiprep/blob/d91e0f6b04ccb5734b5f474affaebd6d90f301b1/qsiprep/interfaces/nilearn.py#L166-L203
I see that the mask size is compared to the t1w by counting voxels. This seems like it will usually trigger the warning, because the 1mm T1w mask is always going to have many more voxels than the 1.5mm or 2mm b0 image, even if the masking worked.
        input_img = nb.squeeze_image(load_img(self.inputs.b0_file))
        min_size = t1_brain_voxels * .6
        max_size = t1_brain_voxels * 1.4
        mask_voxels = mask_img.get_fdata().sum()
        if mask_voxels < min_size or mask_voxels > max_size:
            LOGGER.warning("Masking appears to have failed. Using a backup method")
            mask_img = compute_epi_mask(input_img)
            mask_voxels = mask_img.get_fdata().sum()
            if mask_voxels < min_size or mask_voxels > max_size:
                LOGGER.warning('Unable to compute a reasonable mask on this b=0 image,'
                               ' using the t1 mask')

What I cannot understand is how the backup method doesn't trigger the same error. I almost always encounter the first warning, but quite rarely see the second.
I started working on a patch to use the image spacing to compare volumes (in mm^3) rather than the sum of voxels. But it seems calling compute_epi_mask works pretty well, so I'm not sure if it would be better to just make that the default.

I added some more debug output, and answered one question: the voxel-based comparison works because the t1_mask image is already resampled to the DWI resolution. So the voxels can be directly compared.
It seems the watershed_refined_b0_mask method doesn't do a very good job, at least on HCP data, but compute_epi_mask is more reasonable and rarely fails.


Update overview-template-annotated-example.md

Sentence read "To see how database backups into a broader strategy,..."
Changed to "To see how database backups fit into a broader strategy,..." to increase readability

@HMuhammad : Thanks for your contribution! The author, @CarlRabeler, has been notified to review your proposed change.

...there was a missing word

Merging typo fix
@HMuhammad Thanks


Fixed error handling for invalid password during eperson creation

References

Partially fixes #1859
Related to Angular (to be created)

Description
With this PR I removed an old check on the password that was present on the eperson creation. In this way only the new validation service is used and if the password is not valid the correct message is returned.
Checklist
This checklist provides a reminder of what we are going to look for when reviewing your PR. You need not complete this checklist prior to creating your PR (draft PRs are always welcome). If you are unsure about an item in the checklist, don't hesitate to ask. We're here to help!

[X] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & integration tests). Exceptions may be made if previously agreed upon.
[X] My PR passes Checkstyle validation based on the Code Style Guide.
[X] My PR includes Javadoc for all new (or modified) public methods and classes. It also includes Javadoc for large or complex private methods.
[X] My PR passes all tests and includes new/updated Unit or Integration Tests based on the Code Testing Guide.
[X] If my PR includes new, third-party dependencies (in any pom.xml), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
[X] If my PR modifies the REST API, I've linked to the REST Contract page (or open PR) related to this change.

@LucaGiamminonni : This PR is currently failing a large number of ITs. Just wanted to be sure you were aware of it, as they do not appear to be random failures.


Added handling of Null values

Hi,
I'm a happy EvalEx user but I would like to request handling of Null values. I'm not sure if this is the right way for you, but I will tried the easiest one by forking your project.
Together with a colleague of mine I created and tested an example implementation which is represented by this PR.
Please feel free to comment or request changes.

The CodeSmells found by SonarCloud are difficult to rate. We would explicitly need null return values so we could distinguish between those and "correct" return values

Interesting idea, thank you for the pull request.
But for me it is not 100% clear, what you want to achieve.
Is it that you want to compare to a new constant NULL, like in if(name == null, "unknown", name) or a != null?
In that case, I think there is an easier way to do it.
I think it would be enough to add the constant, the additional data type and probably that's all.
No need to change the getXyzValue() to return null, I was able to run these tests then:
  @Test
  void testNullEquals() throws ParseException, EvaluationException {
    Expression expression = new Expression("a == null");
    assertThat(expression.with("a", null).evaluate().getStringValue()).isEqualTo("true");
  }

  @Test
  void testNullNotEquals() throws ParseException, EvaluationException {
    Expression expression = new Expression("a == null");
    assertThat(expression.with("a", 99).evaluate().getStringValue()).isEqualTo("false");
  }

Of course, this has to be tested more in-depth with all the other datatypes and boolean functions/operators.
Or are you thinking of some other use cases?
In any case, for for a successful pull request, it would be nice to add

A new ExpressionEvaluatorNullTest with some use cases for how to use null in an expression.
Add the necessary documentation in the docs/ folder (new data type and a new chapter Null value handling in Parsing and Evaluation. Best just before the Exception Handling chapter.

I tried adding a few tests with examples on how this new null values would be used.
The general idea is indeed allowing the passing of null values and handling inside the expression by comparing to the null constant inside an if function (or maybe some sort of new coalesce function).
However, if the null is not handled properly in the expression our expectation would be that operations and functions that do not support null fail. Either with a null pointer exception or with a stricter supporting type check in the functions themselves.

Ok, this looks better now.
I have created https://github.com/ezylang/EvalEx/pull/380 and added 2 commits on top of your pull request.
These commits fix the Sonar code smells and I have added an additional test.
To get this merged, you have to provide the documentation (in /docs) about the null value handling.
Also, I think this is a new feature, not a bug fix. So it will go to the 3.1.0 version, not to the 3.0.x version.
I have created a 3.1.0 branch, can you move your pull request to that branch instead o main?
Or simply make a new PR.
Thanks for contributing. :-)

Hello Udo,
I will try to do this tonight. I'm currently a little bit low in time. Thanks for notifying me.

Was a long day,
I hope it's better now. All kudos belong to @ida-whitek  for her work on this topic. I did only the plumbing.


[ECS] Add duplicates command

Show what rules are already loaded in "import" configs and can be removed from root ecs.yaml

Outdated


Problems with poly

I have recently had the need to extract poly data from some models and I was hoping {equatiomatic} had a solution, but I think I need to point out some inaccuracies.
First, it appears the translation fails when poly has max-degree 1, illustrated below  (malformed TeX with caret and no power).
Second,  poly by default uses orthogonal polynomials, this means that the coefficients given in the associated poly object of the model are not the coefficients of the powers of the unadjusted covariate.  This is illustrated below where mod1 and mod3 (using raw polynomials) reduce identical output, as they should, but mod2 equation is incorrect as its coefficients apply to the transformed disp.  This can be seen with mod4 where I've explicitly performed the orthogonalization process.
library(equatiomatic)

# no poly
mod1 <- lm(mpg ~ disp, mtcars)

# orthogonal poly
mod2 <- lm(mpg ~ poly(disp, 1), mtcars)

# non-orthogonal poly
mod3 <- lm(mpg ~ poly(disp, 1, raw = T), mtcars)

# `mod2`: equation is incorrect because the coefficients apply to transformed inputs
# `mod2` and `mod3`: poly handling seems to break down when the input is degree 1.
cat(extract_eq(mod1, use_coefs = T))
#> \operatorname{\widehat{mpg}} = 29.6 - 0.04(\operatorname{disp})
cat(extract_eq(mod2, use_coefs = T))
#> \operatorname{\widehat{mpg}} = 20.09 - 28.44(\operatorname{disp^})
cat(extract_eq(mod3, use_coefs = T))
#> \operatorname{\widehat{mpg}} = 29.6 - 0.04(\operatorname{disp,\ 1^})

# Explicitly perform the transformations that `poly` does when using orthogonal polynomials
mtcars2 <- mtcars
mtcars2$disp <- scale(mtcars2$disp)
mtcars2$disp <- mtcars2$disp / sqrt(sum(mtcars2$disp^2))

mod4 <- lm(mpg ~ disp, mtcars2)
mod5 <- lm(mpg ~ poly(disp, 1, raw = T), mtcars2)

cat(extract_eq(mod2, use_coefs = T))
#> \operatorname{\widehat{mpg}} = 20.09 - 28.44(\operatorname{disp^})
cat(extract_eq(mod4, use_coefs = T))
#> \operatorname{\widehat{mpg}} = 20.09 - 28.44(\operatorname{disp})
cat(extract_eq(mod5, use_coefs = T))
#> \operatorname{\widehat{mpg}} = 20.09 - 28.44(\operatorname{disp,\ 1^})

Relevant SO discussions:

https://stats.stackexchange.com/questions/31858/recovering-raw-coefficients-and-variances-from-orthogonal-polynomial-regression
https://stackoverflow.com/questions/39031172/how-poly-generates-orthogonal-polynomials-how-to-understand-the-coefs-ret/39051154#39051154

Thanks - to be totally honest I never actually tested the branch that included poly() with use_coefs = TRUE so I'm not surprised to see some errors here. I appreciate the issue and I'll try to get it resolved relatively soon.


Latex error converting to dvi

I am sorry to be posting this issue (because there are many threads about it) but I couldn't find any solution.
The issue doesn't happen if I remove the "á". I have installed it follwing this instructions. What I have tried: everything I could find in the other issues. I have uninstalled MikTex and reinstalled 3 times, I spent 5 hours installing full texlive (after uninstalling MikTex). My python version is 3.7.3
What happen: when I try to execute the following code:
 class Texto(Scene):
    def construct(self):
        text = TextMobject("Matemática")
        self.add(text)
        self.wait(5)

  File "c:\Manim\manimlib\extract_scene.py", line 153, in main
    scene = SceneClass(**scene_kwargs)
  File "c:\Manim\manimlib\scene\scene.py", line 54, in __init__
    self.construct()
  File "example_scenes.py", line 29, in construct
    text = TextMobject("Matemática")
  File "c:\Manim\manimlib\mobject\svg\tex_mobject.py", line 144, in __init__
    self, self.arg_separator.join(tex_strings), **kwargs
  File "c:\Manim\manimlib\mobject\svg\tex_mobject.py", line 45, in __init__
    self.template_tex_file_body
  File "c:\Manim\manimlib\utils\tex_file_writing.py", line 19, in tex_to_svg_file
    dvi_file = tex_to_dvi(tex_file)
  File "c:\Manim\manimlib\utils\tex_file_writing.py", line 67, in tex_to_dvi
    "See log output above or the log file: %s" % log_file)
Exception: Latex error converting to dvi. See log output above or the log file: c:\Manim\manimlib\files\Tex\f898e0963e234790.log

This is my tex_template.tex file:
\documentclass[preview]{standalone}

\usepackage[utf8]{inputenc}
\usepackage[T1]{fontenc}
\usepackage{amsmath}
\usepackage{amssymb}
\usepackage{dsfont}
\usepackage{setspace}
\usepackage{tipa}
\usepackage{relsize}
\usepackage{textcomp}
\usepackage{mathrsfs}
\usepackage{calligra}
\usepackage{wasysym}
\usepackage{ragged2e}
\usepackage{physics}
\usepackage{xcolor}
\usepackage{microtype}
\DisableLigatures{encoding = *, family = * }
%\usepackage[UTF8]{ctex}
\linespread{1}

\begin{document}

YourTextHere

\end{document}

PS: I am not new to coding

have you changed TEX_USE_CTEX in constants.py to True?

have you changed TEX_USE_CTEX in constants.py to True?

Thank you for answering. Yes, I did. And then I get this error instead: Exception: Xelatex error converting to xdv. See log output above or the log file: c:\Manim\manimlib\files\Tex\0c41808cc715cd39.log
I couldn't find anything useful in the logs, btw.

Then try #1187

Then try #1187

I tried. Now I get this error: ```Sorry, but miktex-maketfm did not succeed.
The log file hopefully contains the information to get MiKTeX going again```
The log file:  No creation rule for font OT:script=hani;language=dflt;.

I installed texlive-full, which maybe full enough to avoid many strange Tex errors.
So if you have enough time, try uninstalling MiKTeX and then install TeXLive-full.

I installed texlive-full, which maybe full enough to avoid many strange Tex errors.
So if you have enough time, try uninstalling MiKTeX and then install TeXLive-full.

Thanks for answering again. I already have TexLive-full installed, it took me 5 hours yesterday and I ran the code with MiKTeX uninstalled. But I installed it again, so now I have MiKTeX and TexLive, do you think uninstalling MiKTeX may help ?

How I solved it:
It turns out I was using an old version of manim (this may be your problem too). Installing the new one fixed it.


ListReceivedShares

I think this fixes #93
response is still hardcoded in shareInfoToResourceInfo though
also createExternalShareObject sets some different fields to the same values, which I'm not sure is really all that great, but the external shares table is fairly limited.
hopefully this can all be resolved in a satisfactory manner when we have the ScienceMeshShare implementation (WIP)

ok, test aren't passing, working on fixing that rn

Probably is failing just because the $response at testListReceivedShares() is now different with what the ListReceivedShares() returns.

"id"=>[
				"map" => NULL,
			],
			"resource_id"=>[
				"map" => NULL,
			],
...

!=
"id"=>$empty_object,
			"resource_id"=>$empty_object,
....


Move example bundles from porter into example bundles repository

This copies the example bundles (and the associated git history) out of the porter repository into this one (example bundles). The history is ginormous but it preserves people's contribution and attribution which is important.

Some of the commits were made from before we had the DCO requirement on our repositories. All commits were by porter maintainers (myself, Jeremy Rickard, Vaughn Dice) and are covered by the CLA we had in place at the time (before we joined the CNCF and moved to DCO).


feat(mappers): Stream name can now be accessed in __alias__ context of stream maps

Description
Allow users to reference the stream name in stream maps alias fields, under a special key:
{
    "stream_maps": {
        "*": {
            "__alias__": "'raw_' + __stream_name__"
        },
}

Related

Closes #2700
Similar to #2283 but __alias__ was not an evaluated field before now

📚 Documentation preview 📚: https://meltano-sdk--2701.org.readthedocs.build/en/2701/

@edgarrmondragon Added some docs! Left the version as TODO

Thanks @holly-evans!


Banding artifacts on renders

When using stratified sampling, we still get banding artifacts:
https://user-images.githubusercontent.com/3310961/190928619-b41f94dd-3326-49da-871e-2d992c7d761e.mp4

Fixing for now by increasing sample during rendering.


Notebook to demonstrate k8s provisioning through to tutorials

Setting up a k8s environment in public cloud isn't always entirely trivial for a newbie. Some may be granted access by an admin, or use k8s locally under docker for win/mac.
However feedback when running through our current notebooks suggested this would be useful

we can chose one platform to start
likely IBMcloud or Redhat (authors access)
contributions welcome for GCP, Azure etc - please get in touch :-)

This issue proposes the creation of a jupyter notebook that takes a user through the process of creating a cluster & running the egeria tutorials. We'll probably start from just having a cloud account & ideally make use of a free trial service to make it accessible for the widest audience.

Having looked at this it's rather tricky.

Each cloud provider is different
Running on 'local' k8s environment - is both a heavy environment, and enabling is a single click
Running on IBMCloud (for example), even for a free account needs about 5 personal pieces of information
Creating resources (if done wrongly) can cost $$$ - a bug could impact the user financially
Users may not have permissions to create a cluster - only to use existing one
The cloud providers themselves put effort into docs, making it easier for users to create a clister
Where would we run the notebook - if not k8s, docker - or local install
In which case where is the real value-add from providing this notebook?

Instead it makes sense IMO to start from a user with access to a k8s cluster. We can add in any relevant getting started links/videos we can find for different cloud providers. This is covered in a seperate issue
Therefore closing this one


feat: add oxlint linting system for frontend code to githook and add github action

Currently we have Prettier running as a formatter but as the front end code grows, we will need something like eslint which is well known for being slow and not written in Rust.  So the clear option is oxlint which is neither

This PR has oxlint run In the pre-commit hook, but I don't see it added as a dev dependency in the package.json file. It also looks like you've kept prettier as a dependency and the prettier ignore file. Do we still need those?
Otherwise looks great!

Good catch! This does need to be added as a dependency to the package.json file,
It sounds unclear in my comment, but this is added in place of something like eslint for semantic and syntactic errors, and actually doesn't provide formatting like prettier, so we will be continuing to use prettier for formatting.


Cannot update Theme Kit for Debut Theme

I may be using a version so old that theme update or the like do not work for me. Terminal explains that this is not a valid command.
The reason why I am trying to update is because I am trying to develop for the Debut theme, but because the /sections directory is new, Theme Kit errors saying:
sections/slideshow.liquid' is not in a valid file for theme uploads
Files need to be in one of the following subdirectories: layout/ assets/ config/ snippets/ templates/ locales/

It does this for each file in the directory. Is my Theme Kit version out of date? Is there something else that is wrong?

Hey @joshiefishbein can you please tell me the output of which theme? It looks like you are running the theme gem still and not this version of theme kit. Current theme kit supports the sections folder.

@tanema thanks for getting back to me so quickly. Here's the output of which theme:
/usr/local/bin/theme

what about theme version

@tanema theme version responds with: Could not find command "version".

I think you are running shopify_theme. Try reinstalling themekit.

If you are on Linux or MacOSX: maybe you have installed themekit but you didn't added to our PATH.

Thanks for helping out @arypbatista! @joshiefishbein you are using shopify_theme gem. You can remove this by running rm $(which theme) then install theme kit using curl https://raw.githubusercontent.com/Shopify/themekit/master/scripts/install | python Make sure to follow the instruction at the end of the installation to add theme kit to your path.

Ok Thanks so much for the support @tanema and @arypbatista !!


More integration tests

I feel like we should have more integration tests - there are currently only 2 of 'em, and their coverage of scheduling logic seems lacking to me (especially in multithreaded environment).
There are currently 2 open issues about different from MongoDb repositories and in case someone would implement any of those neither they or we can be sure about correctness of the implementation.
It would be really nice to have some load tests as well :)

It's a good idea. I think we need to refactor integration tests. We need more isolated tests. I have an idea about it.
First is it runs tests for every DB in a different process.

First is it runs tests for every DB in a different process.

Yea, this might be a good solution, i like this idea.
@Bobreshovr I guess we could sit together and make somewhat of a plan for building good integration tests, which we could then document here and follow

add #29


Competition hall paintings corruption.

I'm not sure if this has come up before because I'm sure people don't really think about competitions.
But anyway, I noticed my trainer card on Ruby had gone down from 4* to 3*, I was rather confused about this and in my research I found the paintings in the competition hall now crash the game.
After this to confirm the cause, I got a save back up from a time before I did the transfer with my 4* trainer card. After doing another transfer from Red to Ruby I can confirm that again, the paintings are corrupted and my trainer card is 3*.
I'm not aware if it does anything negative in the non Hoenn games, or GB but I haven't really looked much on those. This one just kind of jumped out the most.

This is really strange, thank you for bringing it to my attention!! Do you happen to have a way to send me your save files so I can take a look at the raw save data?

Ah- I figured out the problem! Ruby and Sapphire were trying to add Wonder Card save data despite not being in those games. The bug will be fixed in the next release! Thank you again for bringing this to my attention :)

Cool! I was just about to send my saves but I guess I don't have too, glad you found it.


Development input is not available outside the Editor

We use the standalone windows builds for testing more clients in a shared experience without requiring more devices. I'd love to find a way so that this development input is kept in these builds, and if possible, enable it in my UWP application when launched on a desktop.
We don't want to ship these SKUs, just use them for testing network scenarios.
I'm thinking of adding a define, e.g. MRTK_DEVELOPMENT_INPUT, that if set in your player settings script defines would keep the development input enabled even outside the editor.
Thoughts?

I don't understand what you mean by "Development Input"

Sorry, I'm referring to the mouse and keyboard controls that exist in the editor, but are removed from all the builds.

I was actually looking into making these inputs optional as well, but I think it'll be easier with a simple toggle, and not having to define something in the code anywhere.
It would just be another input source.

No, I was going to add input sources for each that people can add when they need them.

I've had to modify CustomInputSource, AxisController, ButtonController, CustomInputControl, CustomInputSelector, and ManualGazeControl in order to get the input working on Standalone builds. You'd make new ones of each of these for each platform? I think I'm confused.

No, not a new one for each platform.

I guess I'm trying to wrap my head around how making a new input source will solve the problems in the other classes.

So we wouldn't be using any of the other classes in the new solution and we'd replace them with an implementation based on these new input sources? Am I following you right?
I was hoping to just make what we have work in Standalone builds. Ha.

Cause they won't derive from the old classes that use the UnityEditor namespace to get info.
These input sources should be first class citizens.

I agree that we want keyboard and mouse input sources available as well, but I specifically want the behavior of what currently happens in the editor - just out of the editor. I'd like to avoid re-implementing that behavior with new input sources if possible. In your solution that is something I'd have to do right?

I'm also envisioning the inputs that mirror, simulate the Cliff house as well

I'd like to avoid re-implementing that behavior with new input sources if possible. In your solution that is something I'd have to do right?

Well that's why we're brainstorming now, right? :p
I'd like to have this without the need for devs to re-implement.  The way I'd like to do it is just have it on the InputManager prefab by default.

So I got input working via the define in my project using a slightly older version of the MRTK and moved to the MRTK proper to apply the changes there. However, now I'm seeing in the MRTK (with no changes) that I can't air-tap when I'm on the PC platform. On UWP it opens the MR portal and let me drive input that way. Is the old floating hands input stuff all stripped out now?

Is the old floating hands input stuff all stripped out now?

I'm unsure. This could have been a side effect of the Editor thinking you're using an occluded device, instead of a transparent one.

Closing issues older than 180 days. If this is still an issue, please reactivate with recent information.


CEF not working on AWS VM  Windows10 desktop experience on Windows Server 2016

Objects are getting identified on Windows10 Pro but not on AWS VM Windows10 desktop experience on Windows Server 2016.
Please help us. As we need to execute our tests on AWS VMs

Are you using the driver, and set up the environment debugging port?
https://chromedriver.chromium.org/downloads

Thanks Jonathan for taking time to look at it.
Yes debugging port is enabled, but on Windows10 Pro it works without enabling the debugging port.
On AWS vm where we have  2016/2019 server, only debugging enabled controls are recognized  and not the other CEF controls while on Windows 10 pro on laptop it works flawlessly.
Why controls are not exposed on AWS vm with server 2016/2019.


Keycloak admin client for user migration

Adding Keycloak admin client for userStore migration. Now this module can handle two-way migrations, i.e from WSo2IS to AiravataUserProfileService and Keycloak Identity Server

@machristie
I have removed the file

@anujbhan since you have write access to the Apache repo, feel free to go ahead and merge this.  Or I can merge it if you would rather? Either way is fine with me.

I will merge this marcus, thank you !
On 4/25/17 3:11 PM, Marcus Christie wrote:

@anujbhan https://github.com/anujbhan since you have write access to
the Apache repo, feel free to go ahead and merge this. Or I can merge
it if you would rather? Either way is fine with me.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/airavata/pull/106#issuecomment-297134683,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOOY4eY-NKuQzkRMMWtEfg_L-CwWz8MPks5rzkVOgaJpZM4NFiUT.

merge conflicts, closing this pull request. Will open a new one with resolved conflicts


I am unable to push the pages.

fatal: could not read Username for 'https://github.com': No such device or address
##[error]Command 'git' failed with args '-c user.name=github-action-benchmark -c<EMAIL_ADDRESS>-c http.https://github.com/.extraheader= fetch origin gh-pages:gh-pages': fatal: could not read Username for 'https://github.com': No such device or address
My workflow file:
 - name: Store benchmark result
      uses: rhysd/github-action-benchmark@v1
      with:
        name: Go Benchmark ${{ matrix.benchfile }}
        tool: 'go'
        output-file-path: output.txt
        # Use personal access token instead of GITHUB_TOKEN due to
        # https://github.community/t5/GitHub-Actions/Github-action-not-triggering-gh-pages-upon-push/td-p/26869/highlight/false
        github-token: ${{ secrets.GH_PERSON_TOKEN }}
        auto-push: true
        # Show alert with commit comment on detecting possible performance regression
        alert-threshold: '200%'
        fail-on-alert: true

public or private repo?  there's a PR (#38) and an issue (#19) that discuss a similar error for private repos.


ENH: merge Logreg and Lasso cython code

There is a lot of duplicated code currently, it does not cause a loss of time to add a pb variable to the cython code and have a few if pb == "lasso" in the code.

#95


Release new version containing versioned_listeners

Please release a new version of Flask-SQLAlchemy to pypi that contains the versioned_listeners keyword argument when creating an SQLAlchemy db object.

This is rather old.


Issue with spyOn mockImplementation in a class after reset

Describe the bug
I am mocking a class method
  spy = vi     .spyOn(client, 'mySqrt')     .mockImplementation((_) => {       return 42;       }); });
The mock works correctly the first time (before reset), but it returns undefined after being reset.
Reproduction
https://github.com/pmosconi/vitest-issue
The same code is in 2 tests one after another, mock is set in beforeAll and reset in afterEach
Actual: second test fails
Expected: both succeed.
System Info
System:
    OS: Windows 10 10.0.19044
    CPU: (12) x64 Intel(R) Core(TM) i7-10750H CPU @ 2.60GHz
    Memory: 16.60 GB / 31.76 GB
  Binaries:
    Node: 16.13.2 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.19 - C:\Program Files\nodejs\yarn.CMD
    npm: 8.1.2 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.19041.1266.0), Chromium (108.0.1462.54)
    Internet Explorer: 11.0.19041.1566
  npmPackages:
    vite: ^4.0.1 => 4.0.1
    vitest: ^0.25.8 => 0.25.8

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

This is expected behaviour: https://vitest.dev/api/#mockreset
If you want to clear the state of .mock (calls, results), use vi.mockClear(): https://vitest.dev/api/#mockclear

Thank you very much for pointing out my mistake and for the great work!


Documentation requires review

Can we get this documentation reviewed for accuracy?  There is no mention of the CRM Outlook client being required for the Dynamic Worksheets to work correctly.  We have this included in our OnPremise documentation (https://docs.microsoft.com/en-us/dynamics365/customerengagement/on-premises/basics/export-excel-dynamic-worksheet), but not for the Online documentation.
Also, the "Refresh from PowerApps" button doesn't seem to exist anymore.
Thanks!

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

ID: 168295f7-0ceb-8996-611f-e5538da2d3bb
Version Independent ID: 09f8ed1c-2c2c-8ea6-e0a3-398396858e65
Content: Export to an Excel dynamic worksheet in model-driven Powerapps - Power Apps
Content Source: powerapps-docs/user/export-excel-dynamic-worksheet.md
Service: powerapps
GitHub Login: @mduelae
Microsoft Alias: mduelae

@sdieken Thank you for the feedback. I'm working on this and will update the topic as appropriate.
Thanks,
Mints

@sdieken I updated the topic with instructions on how the current model-driven Power Apps user interface works. I also checked with our internal team and Outlook client is not required for Export to Excel options to work in Power Apps.
Hope this helps.
Thanks,
Mints


Crashing when using altered keys.

Working with the sample code, I tried changing a valid signing key and a valid verification key by 1 character.  The change in the signing key crashed the client.  The change in the verification key crashed the server.  The following are the errors.
On client, when signing key was altered.
Error: error:0D07209B:asn1 encoding routines:ASN1_get_object:too long
    at Error (native)
    at Sign.sign (crypto.js:327:26)
    at Object.signRequest (/Users/garygriswold/node_modules/http-signature/lib/signer.js:166:26)
    at Object.<anonymous> (/Users/garygriswold/BibleApp/Server/www/DigitalSignatureClient.js:22:15)
    at Module._compile (module.js:460:26)
    at Object.Module._extensions..js (module.js:478:10)
    at Module.load (module.js:355:32)
    at Function.Module._load (module.js:310:12)
    at Function.Module.runMain (module.js:501:10)
    at startup (node.js:129:16)

On server, when validation key was altered.
Error: PEM_read_bio_PUBKEY failed
    at Error (native)
    at Verify.verify (crypto.js:356:23)
    at Object.verifySignature (/Users/garygriswold/node_modules/http-signature/lib/verify.js:31:19)
    at Server.<anonymous> (/Users/garygriswold/BibleApp/Server/www/DigitalSignatureServer.js:14:22)
    at Server.emit (events.js:110:17)
    at HTTPParser.parserOnIncoming [as onIncoming] (_http_server.js:491:12)
    at HTTPParser.parserOnHeadersComplete (_http_common.js:111:23)
    at Socket.socketOnData (_http_server.js:343:22)
    at Socket.emit (events.js:107:17)
    at readableAddChunk (_stream_readable.js:163:16)

What would you expect to happen instead? Particularly on the client, throwing an Error is the only sensible thing I can see for it to do in the face of a clearly invalid key being provided?

Alex,
If you are not willing to change it, I can always write my own wrapper that returns false in the verification method whether the input is null, invalid, or simply the wrong key.  From my experience, I can’t imagine a code review coming to any other recommendation.  For a user of an application, which uses this library, a client program crash needs to be avoided because it leaves a non-technical the end user with no idea of the source of the problem.  While a response that they are unauthorized will at least help the end user understand who to contact to correct the problem.
If you are willing to give me one more response, I would very much like to know if your opinions on library error handling are entirely your own, or do they also reflect programming policy inside Joyent, or does Joyent have no policy on such matters?
Sincerely yours,
Gary Griswold

On Sep 10, 2015, at 1:58 PM, Alex Wilson<EMAIL_ADDRESS>wrote:
What would you expect to happen instead? Particularly on the client, throwing an Error is the only sensible thing I can see for it to do in the face of a clearly invalid key being provided?
—
Reply to this email directly or view it on GitHub https://github.com/joyent/node-http-signature/issues/46#issuecomment-139327627.

If you want to know about the general Joyent views on error handling in node.js, please see https://www.joyent.com/developers/node/design/errors -- in this particular case, we have a synchronous API and this is an operational error (the function was given invalid data). The standard way to return this to the consumer of the API is by throwing an Error (see the example of what JSON.parse does).
Your application can use a try { } catch { } block to handle this case, it does not need to crash all the way back to the end user.


G

[Enter feedback here]

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

ID: ddafba5c-f652-3d1a-e4b4-f8eedc14e740
Version Independent ID: ea24a163-6533-24eb-630c-559bd1549dc7
Content: Quickstart: Onboard in Azure Sentinel
Content Source: articles/sentinel/quickstart-onboard.md
Service: azure-sentinel
Sub-service: azure-sentinel
GitHub Login: @yelevin
Microsoft Alias: yelevin

@johnarthur1 Thank you for engaging with us on Azure Docs repository. I just wanted to reach out to you to understand more about your query as you have not mentioned any details in the query or feedback that you provided above . Please let us know more so that we can help you better on this.
Thank you.

@johnarthur1 As we have not received any response from your side we are closing this issue. In case you still have any queries , please do tag me or the author of the article @yelevin to your reply and we will surely help you further. alternatively you can always open a new issue and we will be happy to help .


Upload Works, But Display Issue

Hey there. I have successful used tomitrescak's to upload pictures to my MeteorJS app. All the images can be seen in the respective upload folder.
However, none of the pictures will display in the {{ uploadedInfo uploads }} handlebar section or any other section as suggested. Any thoughts or suggestions?

Brad, that would be fault somewhere in your implementation. The callbacks provide you either with file itself (validate) or with url to the uploaded file (finished). How you handle these callbacks is completely in your management. I don't think I can help here. Did you manage to solve this problem? If so, can you please close this issue?


go modules support

could you please add a tag v2.0.0 so that it is semantic version compatible?

Wanna test? added the tag now.

Works like a charm!


Switch to zune-jpeg

Apparently jpeg-decoder is somewhat deprecated.
Additionally:

[ ] try to decode JPEG directly to tiny_skia::Pixmap to avoid needless allocations
[ ] try to use JPEG-native resizing when possible #781

I can work on this.


Improve and fix evaluation of SERVICE in FedX federation

For evaluation of (external) SERVICE clauses, i.e. federated services that are not explicitly member of the federation, FedX relies on the RepositoryFederatedService implementation.
Currently FedX is configured to use the simple evaluation (i.e. non-bound join using VALUES). This can already be changed in the configuration.
The goals of this task are

activate bound join evaluation using the VALUES clause for external SERVICE clauses by default
validate that there are no issues (no blocking, deadlocks)
add unit tests covering this

A second orthogonal improvement of this issue is to make sure that any query is evaluated in the FedX engine, if it contains a SERVICE clause - even if the federation has only a single member. This is to guarantee consistent behavior in the evaluation of SERVICE clauses

PR is merged, closing issue


g:counterpoint_depth may not be respected by listed buffer search.

Verify and fix if needed.

Confirmed that this does not work properly. With a depth of 0 (the default), some_file.win.cpp should alternate with some_file.mac.cpp, but if the latter is a listed but unsaved buffer, no counterpart will be found.


v1.21.3 is identical to v.1.21.1

Just wanted to let you know: Your last pushed images to Docker Hub are all identical and contain the v1.21.1 build. The started container also self-reports as 1.21.1.
So the latest build does not include the SSH Go security fix.

For me, that did the trick: https://git.mbober.de/martin.bober/docker-rpi-gitea/commit/60a46a9a12184b3a78550d1fe8f447fe339e2d2e


Cannot download annotation

Hi,
Thanks for the wonderful work! However, it seems that the download link for annotations doesn't work. Here is the error message:
Downloading original AQUA-RAT...
Downloading original AR-LSAT...
Merging original AQUA-RAT with reasoning graph annotations...
data/aqua_rat/aqua_rat_graph_annotation_train.jsonl
Traceback (most recent call last):
  File "/cluster/work/sachan/yifan/data/cg_reasoning/street/street-reasoning-main/run_street_data_generation.py", line 79, in <module>
    main()
  File "/cluster/work/sachan/yifan/data/cg_reasoning/street/street-reasoning-main/run_street_data_generation.py", line 76, in main
    merge_with_reasoning_graph_data()
  File "/cluster/work/sachan/yifan/data/cg_reasoning/street/street-reasoning-main/run_street_data_generation.py", line 68, in merge_with_reasoning_graph_data
    aqua_rat_data = dataset_utils.AquaRatData()
  File "/cluster/work/sachan/yifan/data/cg_reasoning/street/street-reasoning-main/utils/dataset_utils.py", line 112, in __init__
    self.load_graph_annotations()
  File "/cluster/work/sachan/yifan/data/cg_reasoning/street/street-reasoning-main/utils/dataset_utils.py", line 77, in load_graph_annotations
    self.graph_annotations[split] = utils.load_from_jsonl_file(path)
  File "/cluster/work/sachan/yifan/data/cg_reasoning/street/street-reasoning-main/utils/utils.py", line 36, in load_from_jsonl_file
    return [json.loads(line) for line in fh.readlines() if line]
  File "/cluster/work/sachan/yifan/data/cg_reasoning/street/street-reasoning-main/utils/utils.py", line 36, in <listcomp>
    return [json.loads(line) for line in fh.readlines() if line]
  File "/cluster/project/sachan/yifan/softwares/anaconda/anaconda3/envs/llama/lib/python3.9/json/__init__.py", line 346, in loads
    return _default_decoder.decode(s)
  File "/cluster/project/sachan/yifan/softwares/anaconda/anaconda3/envs/llama/lib/python3.9/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/cluster/project/sachan/yifan/softwares/anaconda/anaconda3/envs/llama/lib/python3.9/json/decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

When I checked the file (e.g., data/aqua_rat/aqua_rat_graph_annotation_train.jsonl), it contains some invalid oid of Git-LFS.
version https://git-lfs.github.com/spec/v1
oid sha256:42de0109f6f0efdd813f18a018b8eba747483f0b07874f6b9b11fbae8db985cd
size 245085

Could you take a look?
Best,
Yifan

Hi Yifan,
Would you mind installing git-lfs first and try to pull the repo again? For example, in macOS, you can do brew install git-lfs first. And in Ubuntu, you can do apt-get install git-lfs.
I believe after you install the git lfs, you should be able to clone the repo correctly.
Best,
Henghui Zhu

Hi Henghui,
Thanks for the help! The environment that I'm using is quite old, and some
errors happened when I install it. But no worries, I manually download the
raw annotation files and it works now.
Many thanks to you!!! And hope there will be more wonderful work from you
in the future.
Best,
Yifan
On Thu, Aug 17, 2023 at 4:37 PM Henghui Zhu @.***>
wrote:

Hi Yifan,
Would you mind installing git-lfs first and try to pull the repo again?
For example, in macOS, you can do brew install git-lfs first. And in
Ubuntu, you can do apt-get install git-lfs.
I believe after you install the git lfs, you should be able to clone the
repo correctly.
Best,
Henghui Zhu
—
Reply to this email directly, view it on GitHub
https://github.com/amazon-science/street-reasoning/issues/1#issuecomment-1682403005,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHAOLIWTSSJEZAZ5DZFE7ZTXVYUCFANCNFSM6AAAAAA3RCM22U
.
You are receiving this because you authored the thread.Message ID:
@.***>


Prerequisite: install kubebuilder

Description of the change:
Add 'install kubebuilder' to list of prerequisites.
Motivation for the change:
I see that there is assumption that kubebuilder is installed.
The command
make docker-build docker-push IMG=<some-registry>/<project-name>:<tag>

will fail with error
failed to start the controlplane. retried 5 times: fork/exec /usr/local/kubebuilder/bin/etcd: no such file or directory
otherwise.
Checklist
If the pull request includes user-facing changes, extra documentation is required:

[ ] Add a new changelog fragment in changelog/fragments (see changelog/fragments/00-template.yaml)
[ ] Add or update relevant sections of the docs website in website/content/en/docs

Thank you @camilamacedo86.
I am new to operator-framework so I can't even tell I understood all your reply.
(Saying that, I guess I should have search the issues or open one before adding a PR.)
For the issue itself, I followed the tutorial in https://sdk.operatorframework.io/docs/building-operators/golang/tutorial/ today and installed the SDK today. I got the error I wrote in the PR description and it was solved after I installed kubebuilder. So something is still not correct here, right?
You may close this PR if you think this is not the best solution.

Thank you @camilamacedo86.
I am new to operator-framework so I can't even tell I understood all your reply.
(Saying that, I guess I should have search the issues or open one before adding a PR.)
For the issue itself, I followed the tutorial in https://sdk.operatorframework.io/docs/building-operators/golang/tutorial/ today and installed the SDK today. I got the error I wrote in the PR description and it was solved after I installed kubebuilder. So something is still not correct here, right?
You may close this PR if you think this is not the best solution.

Hi @ErezArbell,
Could you please help us to identify your scenario? Can you raise an issue with that?
Also, can you provide the versions and etc? It is important for we check out.
Really thank you for all your collab.
Please, not hesitate to contact us with your problem, ideas and suggestions.
Your collab is more than welcome.
I hope that it is fine we close this one.

Hi @ErezArbell,
Could you please help us to identify your scenario? Can you raise an issue with that?
Also, can you provide the versions and etc? It is important for we check out.
Really thank you for all your collab.
Please, not hesitate to contact us with your problem, ideas and suggestions.
Your collab is more than welcome.
I hope that it is fine we close this one.

I opened issue: https://github.com/operator-framework/operator-sdk/issues/4432

I opened issue: https://github.com/operator-framework/operator-sdk/issues/4432


Generalize student's t to real-valued degrees of freedom

The current implementation (v1.7) of the student's t distribution in scipy supports only integer values of the degrees of freedom parameter df for the cdf and inverse of the cdf.
https://docs.scipy.org/doc/scipy/reference/generated/scipy.stats.t.html
The implementation of the pdf formula is already valid for real values of df. The cdf implementation calls stdtr from scipy.special which maps to https://github.com/scipy/scipy/blob/master/scipy/special/cephes/stdtr.c#L92, which supports only integer values of df.
The implementation of stdtr based on the regularised incomplete beta function already supports real-valued df, but it is only used from -oo to t=-2. Only for -2 < t < 0 it uses an implementation that relies on df being an integral value.
According to my numerical experiments, the implementation based on the regularised incomplete beta function can be used over the full range -oo < t < 0, so the extension should be trivial.
I want to add that the documentation of https://docs.scipy.org/doc/scipy/reference/generated/scipy.stats.t.html does not mention that the cdf is only valid for integer values of df, neither is this mentioned here https://docs.scipy.org/doc/scipy/reference/generated/scipy.special.stdtr.html#scipy.special.stdtr.
You may ask why this is needed. It turns out that the shifted and rescaled student's t distribution with a real-valued df is a suitable empirical fitting function for peaks in the invariant mass distribution of particle decays in high energy physics. A paper that uses this technique is https://arxiv.org/abs/2107.10090
Update: Despite of what I found from reading the source code and looking at the scipy tests, it seems that scipy actually does support real values for df.
from scipy.special import stdtr

stdtr(1, 1)
# 0.7500000000000002

stdtr(2, 1)
# 0.7886751345948129

stdtr(1.5, 1)
# 0.7744323163616447

I don't fully understand what code is called then, but it seems to be https://github.com/scipy/scipy/blob/master/scipy/special/cdflib/cdft.f, which apparently supports real values for df (my FORTRAN is a little rusty). If that's the case and calling stdtr with real values for df is officially supported, this issue can be closed - although it would be great to stress a bit more in the documentation that real values for df are supported.

You can find the mapping in functions.json. We used to use the Cephes implementation, hence the name, but switched to CDFLIB afterwards.
PRs are welcome to add the desired emphasis in the docstring.


Text ellipsis

Feature request:  I would like an option to tell Text that it should never overflow its container (in either direction) or edge of the screen.  If it does detect such an overflow, I would like it to replace as many characters as necessary with an ellipsis so that there is no overflow.

Duplicate of #417
This is a common feature request.  We'll implement it soon.


LBC File Integration Tests

After discussion with @nhsavage during #3743, we believe it would be beneficial to have one or two integration tests for parsing LBC files - to be stored in SciTools/iris-test-data.
@nhsavage will be sourcing these files in due course. It is important that these files:

represent typical LBC files that we expect Iris to work with
will not take too long to process

See SciTools/iris-test-data, and try running any existing Iris integration tests that use iris-test-data files, for an idea of appropriate files.

actually I was wrong - there is already 1 integration test for LBCs.  Data is at https://github.com/SciTools/iris-test-data/tree/master/test_data/FF/lbc and the test is here

Looking in detail at this file, many of the headers are wrong. It also will have been created with the legacy code makeBC not the more modern CreateBC (when this file was uploaded 18 Nov 2014, CreateBC had not been released (27 Feb 2015). Therefore an LBC file with some valid metadata should be included, possibly incliding a 360 day calendar and a variable resolution file.


Azure pipelines build failed for SPA docker windows container

Hi folks,
I am using Asp.net core with angular and docker windows container. Here is dockerfile as below:
Dockerfile:
# escape=`
FROM mcr.microsoft.com/powershell:nanoserver-1809 AS downloadnodejs
SHELL ["pwsh", "-Command", "$ErrorActionPreference = 'Stop';$ProgressPreference='silentlyContinue';"]
RUN Invoke-WebRequest -OutFile nodejs.zip -UseBasicParsing "https://nodejs.org/dist/v10.16.3/node-v10.16.3-win-x64.zip"; `
Expand-Archive nodejs.zip -DestinationPath C:\; `
Rename-Item "C:\node-v10.16.3-win-x64" c:\nodejs

FROM mcr.microsoft.com/dotnet/core/aspnet:3.1-nanoserver-1809 AS base
WORKDIR /app
EXPOSE 80
EXPOSE 443
COPY --from=downloadnodejs C:\nodejs\ C:\Windows\system32\

FROM mcr.microsoft.com/dotnet/core/sdk:3.1-nanoserver-1809 AS build
COPY --from=downloadnodejs C:\nodejs\ C:\Windows\system32\
WORKDIR /src
COPY ["DemoAngularWindows.csproj", "./"]
RUN dotnet restore "./DemoAngularWindows.csproj"
COPY . .
WORKDIR "/src/."
RUN dotnet build "DemoAngularWindows.csproj" -c Release -o /app/build

FROM build AS publish
RUN dotnet publish "DemoAngularWindows.csproj" -c Release -o /app/publish

FROM base AS final
WORKDIR /app
COPY --from=publish /app/publish .
ENTRYPOINT ["dotnet", "DemoAngularWindows.dll"]

I run docker and no issue. But now I have pushed repo into azure devops and have created
docker build and push an image to container registry piplines. I am getting error message and it says:
Invoke-WebRequest: Access to the path 'C:\nodejs.zip' is denied.
The command 'pwsh -Command $ErrorActionPreference = 'Stop';$ProgressPreference='silentlyContinue'; Invoke-WebRequest -OutFile nodejs.zip -UseBasicParsing "https://nodejs.org/dist/v10.16.3/node-v10.16.3-win-x64.zip"; Expand-Archive nodejs.zip -DestinationPath C:\; Rename-Item "C:\node-v10.16.3-win-x64" c:\nodejs' returned a non-zero code: 1
##[error]The process 'C:\Program Files\Docker\docker.exe' failed with exit code 1
Finishing: Build and push an image to container registry

azure-pipelines.yml:
trigger:
- master

resources:
- repo: self

variables:
  # Container registry service connection established during pipeline creation
  dockerRegistryServiceConnection: '43a885d5-3f2b-417d-9df3-eaac92ed0a20'
  imageRepository: 'demoangularwindows'
  containerRegistry: 'democontainerregistryname.azurecr.io'
  dockerfilePath: '$(Build.SourcesDirectory)/DemoAngularWindows/Dockerfile'
  tag: '$(Build.BuildId)'

  # Agent VM image name
  vmImageName: 'windows-2019'

stages:
- stage: Build
  displayName: Build and push stage
  jobs:
  - job: Build
    displayName: Build
    pool:
      vmImage: $(vmImageName)
    steps:
    - task: Docker@2
      displayName: Build and push an image to container registry
      inputs:
        command: buildAndPush
        repository: $(imageRepository)
        dockerfile: $(dockerfilePath)
        containerRegistry: $(dockerRegistryServiceConnection)
        tags: |
          $(tag)

Ref:
https://docs.microsoft.com/en-us/visualstudio/containers/container-tools-react?view=vs-2019#modify-the-dockerfile-windows-containers
Could you please help to resolve this issue?
Thanks

@AdnanSalah84 -- Adnan, thank you for your question. It looks like you're having an issue with the product itself, rather than an issue with the documentation. Here are a few options where you might consider asking your question:

https://social.msdn.microsoft.com/Forums/vstudio
https://www.stackoverflow.com
https://www.reddit.com/r/VisualStudio/

As well, you might also consider using the Report a Problem tool to report the issue.

@WilliamAntonRohm
There is no problem with run SPA docker windows container on local machine but I believe that there is a little bug that cannot run/build on CI with azure pipelines due to error [Access to the path 'C:\nodejs.zip' is denied]
I tried to switch to linux containers and docker build/push an image to container registry on CI with azure devops pipelines and it was success!

@AdnanSalah84 -- Adnan, thank you for your followup and glad to know you have success.
cc @ghogen for awareness

@WilliamAntonRohm This could be an issue with the Dockerfile we included in the following topic:
https://docs.microsoft.com/en-us/visualstudio/containers/container-tools-react?view=vs-2019
@AdnanSalah84  Thanks, I will investigate this and see if we can get a version of the Dockerfile in our article that works with CI and Windows containers.

@ghogen Thanks, I posted it on stackoverflow and the problem has been solved but a little bit weird. You may view this link as below:
https://stackoverflow.com/questions/61650538/azure-pipelines-build-failed-for-spa-docker-windows-container

It looks like the issue is with writing to the root of the drive and the fix is to write the Node.js bits to a subfolder.


Dolphin Emulator PR

Rename Dolphin-Dev to Dolphin-Beta since this actually downloads the beta.
Add a new manifest for the actual dev build

Autoupdater for dev is not tested but should work

Anything wrong with my packages?


[Feature Request] Camera: Add Disable Combatant Tracking Option

I'd like a Camera Mode to follow a specific token both in and out of combat.
My use case is that I'm using Stream View for an in person game with a TV as the combat map and physical mini's on the screen, but digital counterparts in Foundry for the purpose of tracking Combat and NPC HP/resources. The challenge I am running into right now is when combat starts. Every time the underlying digital map moves all the physical minis need to get moved which is not ideal. The "Camera Mode: Directed" is also not an option, I need the freedom to move my DM view without affecting the digital battle map view on the TV.
As an alternative to "Follow X Token" it could also be a way to disable the auto following of tokens during combat when "Camera Mode: Automatic" is used.
I know this is niche and would be happy to sponsor this development for v9, just let me know how I can help out.

I'm sorry to say that with the changes in v10, I'm not able to support v9 for new development. On a happier note, following specific tokens is a feature I've just added in the most recent version (for v10). It does not currently continue tracking when combat starts, but that would be a simple option to add.

Available in v1.2.0

Wow, that's incredible. Thank you so much. I can't move to v10 just yet but should be able to do soon. Can't wait to try this out.

Just wanted to let you know I was finally able to move to v10 and it's working great. The camera no longer moves when the combatant changes.


How to add gallery

What is your question?
I feel like I've used this place many times, but I opened this because I wanted to know something new.
How do I add a gallery like Twinsomnia, etc.?
There is a proposal in my mod production team to add a gallery to include artwork and fanart, but I don't know how to do it.
Does anyone know of a way to do this that would be helpful?

Is your mod by any chance source code?

If you're making a source mod just make a new State or somthing

Is your mod by any chance source code?

Yes.
We haven't gotten around to any sort of source code yet, but we are convinced that it will be needed.


Replace deprecated 'try!' with '?'

https://doc.rust-lang.org/std/macro.try.html

Deprecated since 1.39.0: use the ? operator instead

We can't make this change, because then it stops building on Rust versions before ? became stable.


Runelite crashes on mac

Type
Crash or hang
Operating System
MacOS
OS Version
macOS Monterey 12.3.1
Bug description
Runelite seems to randomly crash. Happened a few times during different activities
Screenshots or videos
No response
RuneLite version
Runelite version 1.8.30
Launcer version 2.1.5
Logs
client.log
jvm_crash_pid_55765.log
jvm_crash_pid_76098.log
jvm_crash_pid_86029.log
launcher.log

This is a GPU driver crash, there is likely nothing we can do to resolve it.

@Jallah123 you're using an older version of the launcher which has an older version of Java which causes this issue.
To solve the problem you're seeing, download the latest launcher from the RuneLite website.

Updated to the latest launcher, I'll see if this will fix it. thanks


[3.5] Flush accumulated input events on iOS

Fixes #62833.
UPDATE: Version for 4.0 in #62843.

Cherry-picked for 3.4.5.


Errors in Windows/Conda (beta install)

Followed pip install instructions in new conda env python=3.10 (Pip installed w/o errors). Running Cuda 11.8. Seems to be some incompatibility with versions? I don't know... :/
pip install "imaginairy==14.0.0b2"
Getting errors
(imaginairy) PS C:\Users\xxxx\Deep\imaginairy> aimg videogen --start-image pearl-girl.png  --num-frames 4 -r 5
Generating video from image pearl-girl.png
Device: cpu seed: 729039
Traceback (most recent call last):
File "C:\Users\xxxx\anaconda3\envs\imaginairy\lib\runpy.py", line 196, in _run_module_as_main
return _run_code(code, main_globals, None,
File "C:\Users\xxxx\anaconda3\envs\imaginairy\lib\runpy.py", line 86, in run_code
exec(code, run_globals)
File "C:\Users\xxxx\anaconda3\envs\imaginairy\Scripts\aimg.exe_main.py", line 7, in
File "C:\Users\xxxx\anaconda3\envs\imaginairy\lib\site-packages\click\core.py", line 1157, in call
return self.main(*args, **kwargs)
File "C:\Users\xxxx\anaconda3\envs\imaginairy\lib\site-packages\click\core.py", line 1078, in main
rv = self.invoke(ctx)
File "C:\Users\xxxx\anaconda3\envs\imaginairy\lib\site-packages\click_shell\core.py", line 164, in invoke
ret = super(Shell, self).invoke(ctx)
File "C:\Users\xxxx\anaconda3\envs\imaginairy\lib\site-packages\click\core.py", line 1688, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "C:\Users\xxxx\anaconda3\envs\imaginairy\lib\site-packages\click\core.py", line 1434, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "C:\Users\xxxx\anaconda3\envs\imaginairy\lib\site-packages\click\core.py", line 783, in invoke
return __callback(*args, **kwargs)
File "C:\Users\xxxx\anaconda3\envs\imaginairy\lib\site-packages\imaginairy\cli\videogen.py", line 77, in videogen_cmd
generate_video(
File "C:\Users\xxxx\anaconda3\envs\imaginairy\lib\site-packages\imaginairy\video_sample.py", line 55, in generate_video
torch.cuda.reset_peak_memory_stats()
File "C:\Users\xxxx\anaconda3\envs\imaginairy\lib\site-packages\torch\cuda\memory.py", line 307, in reset_peak_memory_stats
return torch._C._cuda_resetPeakMemoryStats(device)
AttributeError: module 'torch._C' has no attribute '_cuda_resetPeakMemoryStats'
(imaginairy) PS C:\Users\xxxx\Deep\imaginairy>

I must admit I'm not sure why getting the proper torch version installed is more difficult for 2.0
I think if you follow the directions here it will work:
https://pytorch.org/get-started/locally/

Hello, and thanks! I tried getting this going on Windows today as well, and found what may be a few issues.

As above, pip installer installed non-cuda version of torch which I resolved by installing the correct version manually (I am using cu118)
When a model is downloaded it seems no further action occurs after it completes, so I stop it and run the command again to proceed. Maybe this is just my perception/impatience
The --gif option seems to just produce a jpg
The location of aimg in site-packages needed to be added to my path manually (mentioned by pip)
Options for aimg are presented with formatting errors when run from cmd.exe:

C:\Users\me\Dev\imaginAIry>aimg
←[33mUsage←[0m: aimg [OPTIONS] COMMAND [ARGS]...

  🤖🧠 ImaginAIry.

  Pythonic generation of images via AI

←[33mOptions←[0m:
  ←[32m--help←[0m  Show this message and exit.

←[33mCommands←[0m:
  ←[32mcolorize←[0m       Colorize images using AI.
  ←[32mdescribe←[0m       Generate text descriptions of images.
  ←[32medit←[0m           Edit an image via AI.
  ←[32medit-demo←[0m      Make some fun pre-set edits to input photos.
  ←[32mimagine←[0m        Generate images via AI.
  ←[32mmodel-list←[0m     Print list of available models.
  ←[32mprep-images←[0m    Prepare a folder of images for training.
  ←[32mprune-ckpt←[0m     Prune a checkpoint file.
  ←[32mserver←[0m         Run a HTTP API server.
  ←[32msystem-info←[0m    Display system information.
  ←[32mtrain-concept←[0m  Teach the model a new concept (a person, thing, style, etc).
  ←[32mupscale←[0m        Upscale an image 4x using AI.
  ←[32mversion←[0m        Print the version.
  ←[32mvideogen←[0m       AI generate a video from an image
Starting imaginAIry shell...
🤖🧠>

An Error trying to generate a simple video (seen after loading weights):

Traceback (most recent call last):
  File "C:\Users\me\AppData\Roaming\Python\Python310\site-packages\imaginairy\cli\clickshell_mod.py", line 30, in invoke_
    command.main(
  File "C:\Program Files\Python310\lib\site-packages\click\core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "C:\Program Files\Python310\lib\site-packages\click\core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "C:\Program Files\Python310\lib\site-packages\click\core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "C:\Users\me\AppData\Roaming\Python\Python310\site-packages\imaginairy\cli\videogen.py", line 77, in videogen_cmd
    generate_video(
  File "C:\Users\me\AppData\Roaming\Python\Python310\site-packages\imaginairy\video_sample.py", line 95, in generate_video
    raise ValueError
ValueError

Great feedback.

Those weird codes are for coloring on macos/linux. I'll have to figure out how to handle that in windows.
I need to add a progress bar
a video should be output to outputs/video. if thats not happening I'll have to look into that more
I really need to at minimum provide more useful error messages when the wrong version of torch is installed. Even better would be to figure out how to install the right version in the first place.


Debugging section

Add Preamble
Highlight LLDB plugin
Move debugging encryption to encryption section
Move the rest to getting started section near or under xcode plugin

Done: https://realm.io/docs/objc/latest/#debugging


buildGoModule packages: update vendorSha256

Motivation for this change
Minimizing diff and rebuilds for changing the default go in  #154059
Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

@ofborg build minikube docker-machine-hyperkit docker-machine-kvm2 lima colima packer docker-compose_2 minio minio-client age agebox actionlint terraform-providers.gandi terraform-providers.vpsadmin terraformer werf
@ofborg test vault


add agma bot to bots.json

The ma Podcast is a log-file-based measurement system provided by AGMA (Arbeitsgemeinschaft Media-Analyse e.V.) that tracks and reports podcast consumption in Germany. This is their bot.
For more detailed information, you can visit their website: AGMA ma Podcast.

Thanks!


Fix broken link to sourcegraph in README

1. What does this change do, exactly?
Fixes a broken link.
2. Please link to the relevant issues.
N/A
3. Which documentation changes (if any) need to be made because of this PR?
N/A
4. Checklist

[ /] I have written tests and verified that they fail without my change
[ /] I have squashed any insignificant commits
[ /] This change has comments for package types, values, functions, and non-obvious lines of code
[ /] I am willing to help maintain this change if there are issues with it later

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


CMake error, INTERFACE_LIBRARY targets may only have whitelisted properties

$ mkdir build
$ cd build
$ cmake ..
-- The C compiler identification is GNU 10.2.1
-- The CXX compiler identification is GNU 10.2.1
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
CMake Error at CMakeLists.txt:8 (set_target_properties):
  INTERFACE_LIBRARY targets may only have whitelisted properties.  The
  property "POSITION_INDEPENDENT_CODE" is not allowed.

CMake Error at CMakeLists.txt:8 (set_target_properties):
  INTERFACE_LIBRARY targets may only have whitelisted properties.  The
  property "CXX_STANDARD" is not allowed.

$ cmake --version
cmake version 3.18.4

CMake suite maintained and supported by Kitware (kitware.com/cmake).

Adding prefix INTERFACE_ makes the error go away, but I'm no CMake expert so I'm not sure if this is the right way to do it:
diff --git a/CMakeLists.txt b/CMakeLists.txt
index dd4d991..bf92123 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -6,8 +6,8 @@ find_package(Catch2 QUIET)

 add_library(clap-helpers INTERFACE)
 set_target_properties(clap-helpers PROPERTIES
-    POSITION_INDEPENDENT_CODE ON
-    CXX_STANDARD 11)
+    INTERFACE_POSITION_INDEPENDENT_CODE ON
+    INTERFACE_CXX_STANDARD 11)
 target_include_directories(clap-helpers INTERFACE include)
 target_link_libraries(clap-helpers INTERFACE clap-core)

Hi, INTERFACE_CXX_STANDARD doesn't exists.

I don't think we need to force PIC either, the developer can set it himself on its plugin/host target, and the host may not even need the PIC option.

@madah81pnz1 thank you very much for the bug report. Please could you confirm the fix? If it doesn't fix the issue for you please can you re-open with logs?
Cheers,
Alex.


[utils/Dockerfile] Docker configuration model is not correctly setted

Summary

The utils/Dockerfile model is not correctly configured to the Melon latest version
Two different dockerfile models are needed (to JavaScript and TypeScript projects)

Solved, will be implemented in release 2.0.1


Labeled inputs/outputs

Is there an existing method to provide "named" inputs/outputs similar to ComfyUI? And if not, can this be added?
https://github.com/comfyanonymous/ComfyUI/blob/master/comfyui_screenshot.png

View this comment:

https://github.com/jerosoler/Drawflow/issues/18#issuecomment-666954819

Or view this:

https://github.com/jerosoler/Drawflow/issues/363

Thank you, that's exactly how I am solving it actually (::before/::after pseudo tags on ins/outs), but it was nice to see that's your recommendation. However I am curious if you have any suggestions for how to make additional content within the node "respect" the boundaries of the input/output labels?

On the left I have explicitly adjusted the position of the select element to be "below" the labels, the the right everything is just the "default" and the labels will overlap with anything added to the node.
Is there an easy way to have "internal" content respect the dimensions of the input/output nodes and have it automatically compensate for multiple inputs/outputs? if there's a simple CSS hack to be applied to a certain element to make that happen I have not found it yet in the quick experimentation I have done, but maybe you know the best place for such a thing? I assumed the .inputs and .outputs divs would be the right places but I couldn't get what I wanted to have happen there. Ideally I'd like a clear "section" at the top of the node for ins/outs and then all other content to appear below whatever "container" the ins/outs are held in.
Thanks!

You can position the elements wherever you want.

https://github.com/jerosoler/Drawflow/issues/454

Otherwise, with this method you can organize a node element:

https://github.com/jerosoler/Drawflow/issues/363

Thanks for the ideas. I was able to construct CSS to allow for proper spacing of INs/OUTs and then dynamically adjusting the margins to keep the main node content below all items in the I/O section of the node. It's not pretty but works.
Feel free to close this issue


add forbid_zero arg to allow zero for notification_interval (but not retry_interval or check_interval)

fix for issue #476
added a boolean option to nagios_interval as to wether it should forbid a value of zero for a particular interval type

Sorry for the delay in reviewing this PR. I think it's better to move this logic towards the model. I will try to give an example...

Just some thoughts.. think we should remove the fail check from the nagios_interval function.
- if seconds.to_i == 0
-   fail ArgumentError, 'Specified nagios interval of 0 seconds is not allowed'
- end

And within the model in the method check_interval we should add the logic.
So for example we should update:
https://github.com/schubergphilis/nagios/blob/master/libraries/host.rb#L229-L231
With something like:
def check_interval=(int)
  @check_interval = check_integer(int)
  if @check_interval == 0
    fail ArgumentError, 'Specified check_integer interval of 0 seconds is not allowed'
  end
  @check_interval
end

And of course we should update all the other methods that cannot be 0.

Sounds good - I'll rework the PR.

Let me correct myself:
def check_interval=(int)
  if check_integer(int) == 0
    fail ArgumentError, 'Specified check_integer interval of 0 seconds is not allowed'
  end
  @check_interval = check_integer(int)
end

Think this should even be sufficient.

Please rebase the branch when you make the adjustments.

@TheCase any updates on this?

sorry - haven't made time.  I'll put it on my todo list.

Was there a resolution for this issue? Bumping from 7.2.2 to 7.2.6 no longer allows zero for the notification interval which is resulting in a lot more email noise.


Awesome books: a complete website with navigation

[x]  When you click on any of the links, the main area changes to display only one of the three content sections.
[x]  To change the contents of the page using only plain JavaScript and CSS.
[x]  Display the current date below the navigation bar.

STATUS: APPROVED 🟢 🥳
Hi @vicmaburrito & @MacCrazyman 🥇 ,
Congratulations!!! 🥳 🎉
You have successfully completed all the expectations for this project, there is nothing more to do here, now you can go on your way and keep improving your skills.
Cheers and Happy coding!👏👏👏

If anything is not 100% clear please leave your questions or comments in the PR thread, we will be glad to help you 😄.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


smooth page transition

I'm striving to achieve iOS-like smooth page transitions in my Flutter app. While the page_transition package provides detailed control over animations through curves, Nylo (built on page_transition) lacks this customization. Is there a way to expose page_transition's curves in Nylo or explore alternative navigation methods to maintain smooth transitions?
Also see: https://stackoverflow.com/questions/56026568/how-to-achieve-ios-like-smooth-page-transition-animation-in-flutter

Hi @a0v0,
You can modify the page transition settings by passing the pageTransitionSettings parameter into the routeTo helper.
Here's an example.
routeTo(HomePage.path, pageTransitionSettings: PageTransitionSettings(
                 duration: Duration(seconds: 1),
                 curve: Curves.easeInOutCubic,
                  alignment: Alignment.topCenter,
                  fullscreenDialog: true,
                  isIos: true,
                  opaque: true,
                  inheritTheme: true,
                  childCurrent: context.widget,
                  context: context,
));

Let's continue this discussion in the discussions tab because it's not necessarily an issue in the Nylo framework but a query, thanks.


Export FontCharType in preludes

FontCharType isn't available in the rltk::prelude. It should be.
Additionally: support more conversions.

RLTK does export FontCharType in the prelude. The following works with the 0.8.0 from cargo:
[package]
name = "rltk-test"
version = "0.1.0"
authors = ["Herbert Wolverson<EMAIL_ADDRESS>edition = "2018"

[dependencies]
rltk = "0.8.0"

use rltk::prelude::*;

struct State {
    glyph : FontCharType
}

impl GameState for State {
    fn tick(&mut self, ctx: &mut Rltk) {
        ctx.cls();
        ctx.set(1,1,RGB::named(RED), RGB::named(BLACK), self.glyph);
    }
}

fn main() -> RltkError {
    let context = RltkBuilder::simple80x50()
        .with_title("Hello RLTK World")
        .with_fps_cap(30.0)
        .build()?;

    rltk::main_loop(context, State{
        glyph : to_cp437('!')
    })
}


feat: add curl and openssl to image

@bsushmith Can you add reasons for this change in description.


Add to conda

Hi,
I would like to know if you would consider packaging your library to bioconda or not. Thanks for the great work.
Regards,
JS

Hi!
Yes, we hope to get it up on Conda quite soon.
Thanks for the nice feedback!

Hi, I added the recipes:
https://github.com/bioconda/bioconda-recipes/pull/38644 (npstructures, done)
https://github.com/bioconda/bioconda-recipes/pull/38630 (bionumpy, ongoing)
Hope this helps, if there are changes needed, I'm happy to assist.

Great, thanks!

That's super, thank you @telatin !


params.html.cloneNode is not a function

When i use sweetalert using import swal from 'sweetalert' it's working fine. But when i try to use the same for sweetalert2 as import swal from 'sweetalert2' it gives me the following error.
Uncaught TypeError: params.html.cloneNode is not a function                              sweetalert2.js?85ed:624
    at setParameters (sweetalert2.js?85ed:624)
    at sweetAlert (sweetalert2.js?85ed:946)
    at VueComponent.openForm (CreateTodo.vue?603e:45)
    at boundFn (vue.esm.js?65d7:179)
    at HTMLButtonElement.invoker (vue.esm.js?65d7:1821)
setParameters @ sweetalert2.js?85ed:624
sweetAlert @ sweetalert2.js?85ed:946
openForm @ CreateTodo.vue?603e:45
boundFn @ vue.esm.js?65d7:179
invoker @ vue.esm.js?65d7:1821

Then i tried removing the module import and explicitly setting up the css and js in my html file, then sweetalert2.css is ignored. It's overriden by another css. And when i add sweetalert.css, it is working fine. I tried changing the order of css files, but nothing happens.

For bug reports, please include:

a jsfiddle in order for us to be able to reproduce the issue.
expected behavior of the plugin
actual behavior

JSFiddle template: https://jsfiddle.net/limon/ad3quksn/

See Contributing.md

@limonte i'm sorry but i'm new to vue.js, so i don't have much of an idea on how to replicate the issue in a jsfiddle. I get this error only for sweetalert2. I use a webpack based project with npm. I tried importing sweetalert from CDN, importing it as ES6 module. Both gives the same error. But i can use sweetalert without a problem. Error directly points sweetalert2.js line 624. Am i missing some dependency? Cuz it says params.html.cloneNode is not a function.

show us at least usage case of calling swal({...})

This is my script where i use sweetalert2 (Test Script).

[UPDATE] : loading this from CDN using

This is the error I get... I have no idea about the cause of this issue. Am I missing any dependencies?

This is my script. I just use a sweetalert here in my opernForm() method.

You're using the code example from https://sweetalert.js.org/ which is a different project. The author of that project is careless about its users, that's why I made SweetAlert2.
SweetAlert:

https://github.com/t4t5/sweetalert/
https://sweetalert.js.org/

SweetAlert2:

https://github.com/limonte/sweetalert2/
https://limonte.github.io/sweetalert2/


Floats

I just changed some code to support initializing,adding,subtracting,multiplying and dividing expressions with floats. The only thing i did is a type conversion from float to Fraction, so no evaluation code was touched. Is that something you want or is the restriction to integer numbers for those operations intentional?
Edit: See the code changes here: https://github.com/nicolewhite/algebra.js/compare/master...timrach:decimals?expand=1

It was intentional as I wanted to keep everything rational. It will also produce some pretty ugly output, e.g.
var exp = new Expression("x");
exp = exp.multiply(1/3);

exp.toString(); // 0.3333333333333333x

If anything, I'd rather the changes be made to Fraction so you could do:
var frac = new Fraction(0.5); // 1/2

Although I'm not sure how you'd handle repeating decimals:
var frac = new Fraction(1/3);

as your current strategy of multiplying and dividing by 10^length of decimal wouldn't work here.

I see, that makes sense. I close this issue then.


Treat EDSN profiles differently from individual profiles when assigning them to households

EDSN profiles represent groups of households larger than typically 100. It would be nice if they are selected only for connections with > 100 households.
We have to think what would happen if the concurrency is minimised for such a group...

I think we should handle it as a 'separate technology'. Household larger than 100 and Household's smaller than 100. It would than automatically select the correct 'technology' according to the total amount of units.

It would than automatically select the correct 'technology' according to the total amount of units.

As long as that is the case you can make it two different technologies indeed. Not highest priority now.

I'm well on my way with this issue. Though we still need to think about the concurrency.

Though we still need to think about the concurrency.

If after minimising concurrency the groups become smaller then the 'threshold' for aggregation, the individual profiles should be used IMO.
Also, I think that the threshold should be lower than 100. Perhaps we can even make this user-defined?

Though we still need to think about the concurrency.

If after minimising concurrency the groups become smaller then the 'threshold' for aggregation, the individual profiles should be used IMO.
Also, I think that the threshold should be lower than 100. Perhaps we can even make this user-defined?

In merge #206


⚠ Bofrost.at has degraded performance

In c12bd8e, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2294 ms

Resolved: Bofrost.at performance has improved in 8326c2b.


Add version naming and deprecation link

[ ] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] Tests are included and/or updated for code changes.
[ ] Updates to module CHANGELOG.md are included.
[ ] MIT license headers are included in each file.

Still waiting on a review here.  PR is still valid.

/check-enforcer override

Thanks for merging @tadelesh.  Can you also please go ahead and merge https://github.com/Azure/azure-sdk-for-go/pull/21680 since it completes the visibility for the same topic?

@dtzar I've merged the PR. Thanks for your contribution.


Implement proper multi-line text selection.

This task is automatically imported from the old Task Issue Board and it was originally created by Wojciech Daniło.
Original issue is here.

Currently, the multi-line text selection does not work properly (it works, but it is not shown). This should be fixed.

EnsoGL is removed from repository.


[WIP] Plugin registry

wip for peeking over my shoulder; I'll probably rebase this heavily before it's actually ready to merge.

Okay,  everything specified in #192  seems to be covered by this.
Cleanup, moving stuff around, and documentation still need to happen, but generally:  thoughts?

Great work. I'll try to port rom-sql plugins to this.

Just remembered, forgot to note this earlier:
One thing I had to do that I don't like,  demoed here:
https://github.com/cflipse/rom/commit/d05afbefc967444b88730cdb132fc69af8a7a1ba#diff-e0b55668c593b9e34a8aad512e592a24R113
Unless I'm missing something, I cannot access the adapter type within the class definition, so it has to be redundantly declared in the use call.  Is there some alternative to that?

@cflipse can you remind me what we're missing to finish this? Specifically in terms of the missing interfaces?

functionally?  Nothing.  It could go in as-is, I think.
I'm not thrilled with repeating the adapter specification, however.  It would be good if the DSL were to be able to know it was operating in the context of an :sql adapter, and check there automatically, rather than it being required to be specified in the use line.
essentially, the use implementation should be able to check an adapter_type and use that, rather than requiring it to be specified.  This is complicated somewhat by the fact that the adapter types are registered, rather than hard-coded.  There's some reverse-lookup that can be done, I just havn't pulled on that thread.

@cflipse OK thanks. I'm gonna try to improve that and merge this in (: Sorry it's taking so long

Merged. I'll continue in master. Cheers (:


Reenabled some STG74 live tests

/azp run java - storage - tests

/azp run java - storage - ci

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - tests

/azp run java - storage - tests

Will reopen when the service issue is fixed


Handle environments throughout the login and linking process.

Err... this is a lot bigger than I though it was going to be. Sorry. The actual code changes are < 200 lines.
Process flow for login/link (LL):

Client sends form to LL start with environment or null for default
environment.
For login, if a user redirect url is sent it's checked against the
configured prefix for the appropriate environment.
The environment is passed though the application to the appropriate
identity provider implementation to get the redirect url to the 3rd
party identity provider. The env-specific return redirect url is added
to the redirect url as a param.
Redirect occurs to 3rd party. A cookie is set with the environment
name and another with the user redirect url, if any (login only).
3rd party redirects to the auth server with an auth token.
The user redirect url cookie is checked as previously described.
The environment and auth token are passed through the application to
the appropriate identity provider implementation to retrieve the user
identities. The environment is necessary so the implementation can pass
the correct return redirect url to the 3rd party identity provider,
which is required.
If LL are done at this point, all process cookies are nuked, a
login token is set (or a new identity is linked), and redirect occurs to
the user login redirect url, the configured final link redirect url for
the environment, or a default url.
If not, the cookies are refreshed and redirect occurs to the
configured intermediate redirect url for the environment or a default
url.
The user redirect url is checked again.
The user then picks an account (login or link) or creates an account
(login only).
The final check of the user redirect url occurs.
All process tokens are nuked, a login token is set (or a new identity
is linked), and redirect occurs to the user login redirect url, the
configured final link redirect url for the environment, or a default
url.

So my eyes glazed over in the tests

Yeah, this is an example of why unit tests should be favored over integration tests in general (and also that small PRs are better - I should've broken this one up but I didn't realize it was so big until I'd already committed). It took me a while to figure out all the setup again and how it needed to be modified. You'll note the TODOs at the top of the Login and Link tests for rewriting most of the tests as unit tests. That's a chunk of time I don't have right now though.

I think that the process flow you provided above is helpful. You might want to stash it somewhere in the repo so that it does not get lost.

That flow only documents the parts of the flow that interact with the new environment argument and is pretty poorly written. I agree that interaction diagrams would be useful - I've added a TODO.

Also, it seems that the OAuth endpoints and documentation was never added to the README.md? It would be great to spruce that up in a future PR.

Are you talking about the 3rd party endpoints? They're all in deploy.cfg.example if so.

@JamesJeffryes ready for re-review

@JamesJeffryes The endpoints that are documented are the entrance endpoints - where you'd expect a user to navigate to to interact with the service. The other endpoints are secondary endpoints that are accessed by the forms/inputs that the user controls but that aren't entrance endpoints.

That being said, there are TODOs all over the place in the codebase for coming up with some sort of documentation (maybe swagger) for the entirety of the endpoints, but I don't have time for that right now.

That's fair. Thanks
On Mon, Oct 8, 2018, 15:43 MrCreosote<EMAIL_ADDRESS>wrote:

Merged #308 https://github.com/kbase/auth2/pull/308 into develop.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kbase/auth2/pull/308#event-1891260086, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AIGfOb1iajgaYze7rw8b9e4QOy4TlJq_ks5ui7jZgaJpZM4XMSRe
.


Handle dependabot alerts

@dploeger Please handle the dependabot alerts i often get via mail. Thanks.
See the following copy'n'paste

 security alert digest

timdeluxe’s repository security updates from the week of Jul 16 - Jul 23

| DO! DevOps organization
-- | --
Dependabot alerts
GitHub security alert digest
timdeluxe’s repository security updates from the week of Jul 16 - Jul 23
DO! DevOps organization
Warning!
dodevops / ansible-loki-callback
Known security vulnerabilities detected
Dependency certifi 	Version >= 2021.05.30 < 2024.07.04 	Upgrade to ~> 2024.07.04
Defined in requirements.txt 	Suggested update #3
Vulnerabilities
CVE-2024-39689 Low severity
Review all vulnerable dependencies

Done


Issue when Deleting / Uploading Large Files (<50MB)

OS/Web Information

Web Browser: Chrome [Version 89.0.4389.82 (Official Build) (x86_64)]
Local OS: Debian (Code Server Docker)
Remote OS: MacOS (Big Sur)
Remote Architecture: Darwin x64
code-server --version: 3.9.3 and 3.9.1

Steps to Reproduce

Select a Large File (Tried with a 44.8MB file)
Right Click and Click "Delete Permanently"
Code Server starts Deleting but the process never finishes and file does not get deleted

Expected
Selected Files should get deleted
Actual
Code Server starts Deleting but the process never finishes and no files get deleted
Smaller files seem to work fine.
Same issue happens for Upload of these large files.
Screenshot

Notes

This issue can be reproduced in VS Code: Yes/No - No

I feel like we've had issues with this before.
Running code-server locally on my macOS and accessing in Chrome, I was not able to reproduce.
Steps followed

brew install code-server (3.9.3)
create a large file: mkfile -n 50m temp_50mb_file
Delete permanently from code-server

Screenshot
https://user-images.githubusercontent.com/3806031/114440132-a0269600-9b7e-11eb-9a8d-593b873744c9.mov
Next steps

[ ] Try reproducing using code-server in remote Linux environment

Adding this to On Deck which means we'll take a look in an upcoming milestone/sprint

Thanks, it seems to be very strange and i am not able to figure out if the issue is with my setup or code-server.

I noticed that large files such as ".csv" text files do get uploaded without issue
The setup is MacOS Chrome Browser --> F5 LoadBalancer --> Nginx Ingress (K8s)--> Pod having Docker Container running code-server
Whenever the issue happens i see the below in the Ingress logs
recv() failed (104: Connection reset by peer) while proxying upgraded connection

Thanks for the additional information. We may be able to create a test environment and try to reproduce this with a similar setup to yours (cc @bpmct any tips here for using Vultur?).
@code-asher may have insight too

I haven't had the opportunity to debug but this seems related: https://github.com/cdr/code-server/issues/2803

Increase the constants related to Ack time in ipc.net.ts -> ProtocolConstants in the vscode project. Can solve the problems encountered above
export const enum ProtocolConstants {
	HeaderLength = 13,
	/**
	 * Send an Acknowledge message at most 2 seconds later...
	 */
	AcknowledgeTime = 20000, // 2 seconds
	/**
	 * If there is a message that has been unacknowledged for 10 seconds, consider the connection closed...
	 */
	AcknowledgeTimeoutTime = 200000, // 20 seconds
	/**
	 * Send at least a message every 5s for keep alive reasons.
	 */
	KeepAliveTime = 500000, // 5 seconds
	/**
	 * If there is no message received for 10 seconds, consider the connection closed...
	 */
	KeepAliveTimeoutTime = 200000, // 20 seconds
	/**
	 * If there is no reconnection within this time-frame, consider the connection permanently closed...
	 */
	ReconnectionGraceTime = 3 * 60 * 60 * 1000, // 3hrs
	/**
	 * Maximal grace time between the first and the last reconnection...
	 */
	ReconnectionShortGraceTime = 5 * 60 * 1000, // 5min
}