cannot install on Windows 10

Have already installed nodejs 8.9.3 with conda, bumped into the following error when trying to install plotly-extension with command jupyter labextension install @jupyterlab/plotly-extension --debug:
> C:\Users\<>\AppData\Local\Continuum\anaconda3\npm.CMD pack @jupyterlab/plotly-extension
jupyterlab-plotly-extension-0.14.5.tgz
Traceback (most recent call last):

  File "C:\Users\<>\AppData\Local\Continuum\anaconda3\lib\site-packages\jupyterlab\labextensions.py", line 71, in start
    ans = self.run_task()

  File "C:\Users\<>\AppData\Local\Continuum\anaconda3\lib\site-packages\jupyterlab\labextensions.py", line 100, in run_task
    for arg in self.extra_args

  File "C:\Users\<>\AppData\Local\Continuum\anaconda3\lib\site-packages\jupyterlab\labextensions.py", line 100, in <listcomp>
    for arg in self.extra_args

  File "C:\Users\<>\AppData\Local\Continuum\anaconda3\lib\site-packages\jupyterlab\commands.py", line 210, in install_extension
    return handler.install_extension(extension)

  File "C:\Users\<>\AppData\Local\Continuum\anaconda3\lib\site-packages\jupyterlab\commands.py", line 347, in install_extension
    info = self._install_extension(extension, tempdir)

  File "C:\Users\<>\AppData\Local\Continuum\anaconda3\lib\tempfile.py", line 807, in __exit__
    self.cleanup()

  File "C:\Users\<>\AppData\Local\Continuum\anaconda3\lib\tempfile.py", line 811, in cleanup
    _shutil.rmtree(self.name)

  File "C:\Users\<>\AppData\Local\Continuum\anaconda3\lib\shutil.py", line 494, in rmtree
    return _rmtree_unsafe(path, onerror)

  File "C:\Users\<>\AppData\Local\Continuum\anaconda3\lib\shutil.py", line 393, in _rmtree_unsafe
    onerror(os.rmdir, path, sys.exc_info())

  File "C:\Users\<>\AppData\Local\Continuum\anaconda3\lib\shutil.py", line 391, in _rmtree_unsafe
    os.rmdir(path)

PermissionError: [WinError 32] The process cannot access the file because it is being used by another process: 'C:\\Users\\<>\\AppData\\Local\\Temp\\tmpet5pksuo'

Errored, use --debug for full output:
PermissionError: [WinError 32] The process cannot access the file because it is being used by another process: 'C:\\Users\\<>\\AppData\\Local\\Temp\\tmpet5pksuo'

Any words of wisdom would be appreciated.

I ran into this exact problem with Nodejs version 10. It appears to be similar to a problem noted in issue #132 . I was able to resolve it by downloading Nodejs 6 from here: https://nodejs.org/download/release/v6.14.4/node-v6.14.4-win-x64.zip
then extracting it and putting it in my path somewhere before Nodejs 10.

PermissionError: [WinError 32] The process cannot access the file because it is being used by another process: 'C:\Users\<>\AppData\Local\Temp\tmpet5pksuo'

What worked for me on Win10, was to temporarily patch the shutil.py module in my conda environment
in /Lib/shutil.py
look for method:
def _rmtree_unsafe(path, onerror):
and replace the last line:
onerror(os.rmdir, path, sys.exc_info())
with
if True: # changed temporarily to solve a jupyterinstall issue
print("os.rmdir: ignoring OSError")
pass
else:
onerror(os.rmdir, path, sys.exc_info())

Note that this should be fixed from Python 3.8 onwards.

Doesn't seem fixed for me

I'm thinking of this being included now (https://bugs.python.org/issue26660).

That might be different though (another process using vs access denied)... :-/

This is a different issue, indeed. Seems hard to believe (rightly so) for non-Windows developers that rmtree is working async in Windows and there is no easy way to tell if it actually has finished.
The usual strategy to cope with this is some sort of exponential backoff. See this (and corresponding PR) for essentially the same issue hitting Python's testsuite.

Command line shutil.py patch, if you are using conda:
rename %CONDA_PREFIX%\Lib\shutil.py shutil.bak
powershell -Command "(gc %CONDA_PREFIX%\Lib\shutil.bak) -replace 'onerror\(os.rmdir, path, sys.exc_info\(\)\)', 'print(\"os.rmdir: ignoring OSError\") #$&' | Out-File -encoding ASCII %CONDA_PREFIX%\Lib\shutil.py"
Undo:
copy %CONDA_PREFIX%\Lib\shutil.bak %CONDA_PREFIX%\Lib\shutil.py


⚠ Bofrost.lu has degraded performance

In dedda41, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 3647 ms

Resolved: Bofrost.lu performance has improved in 420ed38 after 6 minutes.


[SPARK-20661][SparkR][Test] SparkR tableNames() test fails

What changes were proposed in this pull request?
Cleaning existing temp tables before running tableNames tests
How was this patch tested?
SparkR Unit tests

Test build #76592 has started for PR 17903 at commit 7020372.

@felixcheung fyi. I think the main problem of this test is that it will be broken if tests executed before this one leak any table. I think this change makes sense. I will merge it once it passes jenkins.

LGTM. Thanks @falaki  -- BTW is this a problem only on master or should we also backport this ?

@shivaram it seems to have started today with this build: https://amplab.cs.berkeley.edu/jenkins/view/Spark QA Test/job/spark-master-test-sbt-hadoop-2.7/2844/consoleFull
The build just before that does not have the failure: https://amplab.cs.berkeley.edu/jenkins/view/Spark QA Test/job/spark-master-test-sbt-hadoop-2.7/2843/consoleFull

Test build #76592 has finished for PR 17903 at commit 7020372.

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

Merged build finished. Test PASSed.

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

Seems 2.2 build is fine. But, I'd like to get this merged in branch-2.2 since this test will fail if any previous tests leak tables.

Thanks @falaki. Merging to master and branch-2.2.

Hmm, do we know why temp tables are getting leak since https://github.com/apache/spark/pull/17649
But not before?

I do not think https://github.com/apache/spark/pull/17649 caused the problem. I saw failures without that internally.

ah ok, i thought we said it was this build introducing this failure https://github.com/apache/spark/pull/17892#issuecomment-299920986
thanks, it's good to know


Update azure-monitor.md

Click the save button after modifying Windows event logs collection.

@alankinane : Thanks for your contribution! The author(s) have been notified to review your proposed change.

The content has been reconfigured and the pull request can't be merged. We'll close the PR. Thanks for contributing to the documentation.


[pulsar-clients]Save name, type, properties for KeyValueSchema

Motivation
The current keySchemaInfo and valueSchenamaInfo of KeyValueSchemahas no saved name, type and properties.
Modifications
Save name, type, properties for keySchemaInfo and valueSchemaInfo of KeyValueSchema
Verifying this change
Test pass

run cpp tests


SetupStringParameters sets with specifier

First off im new to using github and I'm sorry if I'm doing this wrong.
Before:

After:

Thanks for that pull, just 1 quick thing - please replace 8space tab with 4space for consistency :)

I think it's a bug where there are 8 spacings. It's normal on my source but not on github. No problem :)

MSVC shows tab always as 4 spaces, no matter how big the spacing is.
Change settings to these, and then reformat.

After some testing of this:

Yours:

Old one:

I don't know why you have an issue with the "1" showing up at the end. With multiple tests - it doesnt show up on my side.

the code you've added just removes 1 character after the found %.
Perhaps it needs some further debugging.

As i posted on my screenshot for example when i
int testintegervalue = 123; Utils::Log("test %i", testintegervalue)
it outputs like
test 123i
so when i remove the char after % it will fix it for me

But as i examine it for the second time its not like printf. Silly me % just works fine.


DCAT feed Line breaks inside string literal of Depletion fishing in the rivers Grote Nete and Kleine Nete in Flanders, Belgium

The DCAT feed of the dataset is currently blocking the import of our DCAT feed, as line breaks are present inside a string literal:
dcat:keyword "Electric fishing
Red list
distribution mapping" , "Occurrence" , "Observation" ;

However, when checking the current keywords on the IPT: http://data.inbo.be/ipt/resource?r=depletion-fishing-nete-occurrences#keywords this line break does not seem to be represented in the resource. In contrast, when editing the keywords of the dataset: http://data.inbo.be/ipt/manage/metadata-keywords.do?r=depletion-fishing-nete-occurrences these seem to be different from the current version. @peterdesmet, should I update the version and recheck the resulting dcat-feed?

Seems like the keywords were already comma-separated, but the resource was not republished. I'm having problems republishing... connection to source is slow.

The remaining problem is the new line characters within string literals as discussed in #103 , closing things here.


#97 - first page contains form for email

Current coverage is 42.52%

Merging #98 into master will increase coverage by +3.39% as of 7d95e68

@@            master     #98   diff @@
======================================
  Files            7       8     +1
  Stmts           69      87    +18
  Branches         0       0
  Methods          0       0
======================================
+ Hit             27      37    +10
  Partial          0       0
- Missed          42      50     +8

Review entire Coverage Diff as of 7d95e68

Powered by Codecov. Updated on successful CI builds.


Broken link to NYCTripSmall.parquet

Following the link to download the dataset in the getting started guide reaches the following error:
PublicAccessNotPermittedPublic access is not permitted on this storage account. RequestId:dccbf99f-101e-0003-4b62-e97e89000000 Time:2022-10-26T17:43:44.4955296Z

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

ID: 8a819f2a-490b-0cdd-7d95-21139a20f4ff
Version Independent ID: 181a621d-0e83-af7d-cb8a-7e63f3195604
Content: Quickstart: Get started - create a Synapse workspace - Azure Synapse Analytics
Content Source: articles/synapse-analytics/get-started-create-workspace.md
Service: synapse-analytics
Sub-service: workspace
GitHub Login: @saveenr
Microsoft Alias: saveenr

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

@saveenr
could you please review this and update as appropriate.

@cwnoyes
Thanks for your feedback! We have assigned the issue to author and will provide further updates.

#label:"doc-bug"

@SnehaGunda the main quickstart is broken. Could be due to a recent change in public security on the blob storage container.

Related to #100156
#label:"doc-bug"

The above PR fixes this issue.
#please-close


adding images in cheatsheets

Is your feature request related to a problem? Please describe.
in the cheatsheets page before the programming language name we can add respective images of the programming language,
Describe the solution you'd like.

like this
for python

Describe alternatives you've considered.
we can add images by IMG tag
Add any other context or screenshots about the feature request here.
No response
Record

[X] I agree to follow this project's Code of Conduct
[X] I have checked the existing issues
[x] I'm a GSSoC'23 contributor
[X] I want to work on this issue

Sure @khadar1020


Fix BlurLayout#setAlpha(float) & Closes #41

Fixed bug by adding mLockedAlpha field to save the actual value of alpha.
I also added an alpha animation demo in sample app module.

Hey @fython I wanted to follow up on your PR. We are not sure of our future plans for the BlurKit project quite yet. To avoid the need for licensing agreements we aren’t going to merge your changes directly. We have resolved your concerns in our latest PR #45. We really appreciate your contributions as it did help contribute to our PR, so definitely PR going forward as it’s a great way to get changes integrated directly =)

@emersoncloud You're welcome. This project helps me a lot and the bug has been fixed in time so I am grateful too.
If we have a chance to contribute code again, you could ask me what you want to know directly, such as license aggrement, etc.


Fix typo in link to MSBuild overview

Fix typo in link to MSBuild overview per issue #54

@corob-msft,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


[Bug] Hosting build under IIS reveals a lack of flexibility

Generating "Build" copies all files in bulk with the same structure and names as the original folder. When build Unity project to standalone target it creates such files like "...\Mono\etc\mono\2.0\web.config" which is IIS configuration file. And when trying to download this (and some other files) client get response code of 500. In turn patches are zipped in one file with popular ".zip" extensions which does not creates any problems when downloading them.
My suggestion is to use the same mechanism (or similar) when creating "builds". Files can still be copied separately - which ensures trouble-free downloading each individual - but on server side files can be zipped or at least extensions can be suffixed witch something like ".zip" and just removed/decompressed on client side.

Yeah, I agree with you that this would be great to have.
Unfortunately, this is the result of legacy code and changing how it works right now would mean breaking the updating system for many games.
I need to think about how to offer this feature by maintaining retro-compatibility.
For sure I planned it for the Enterprise version, so if this cannot be delivered into the 2.x version of PATCH, it will be surely shipped with 3.x.

It could be opt-in feature in 2.x and opt-out (default) in 3.x. Or just feature to enable regardless of version.


[3.6 regression] Method unions with different generic defaults now error

TypeScript Version:  3.6.0-dev.20190720

Search Terms: Generic default conflict initializer
Code
interface A {
	bar<T extends number = 1>(): T;
}
interface B {
	bar<T extends number = 2>(): T;
}

declare const foo: A | B;
foo.bar();

Expected behavior: foo.bar() should return 1 | 2 in this case
Actual behavior: error

The prior behavior was to give foo.bar() the type 1, which is clearly not correct. The error is an improvement in correctness but a breaking change we need to record in the release notes.

Is it wrong to evaluate this as 1 | 2? Removing the type checker's ability to merge function definitions has serious implications. I get that it was broken before, but this functionality is useful. Consider this case here:
interface Instances {
	Instance: Instance;
	Model: Model;
	Folder: Folder;
	Camera: Camera;
	ShakyCamera: ShakyCamera;
}

interface Instance {
	ClassName: "Model" | "Folder" | "Camera" | "ShakyCamera";
	Name: string;
	Parent: Instance | undefined;
	IsA<T extends keyof Instances>(className: T): this is Instances[T];
}

interface Model extends Instance {
	PrimaryPart: Instance | undefined;
	ClassName: "Model";
}

interface Folder extends Instance {
	ClassName: "Folder";
}

interface Camera extends Instance {
	ClassName: "Camera" | "ShakyCamera";
}

interface ShakyCamera extends Camera {
	ClassName: "ShakyCamera";
}

function f(o: Folder | Model) {
	if (o.IsA("ShakyCamera")) {
		// This expression is not callable.
		// Each member of the union type
		// '(<T extends "Model">(className: T) => this is Instances[T])
		// | (<T extends "Folder">(className: T) => this is Instances[T]) '
		// has signatures, but none of those signatures are compatible with each other.
		o; // should be a Model
	} else {
		o; // should be a Folder
	}
}

In the above example, the type checker currently knows o.IsA is ("Model") -> this is Model | ("Folder") -> this is Folder. However, it won't type this as "Model" | "Folder", it will just error because "Model" and "Folder" are different. Now, in this case, I can get around this by changing the definition to pull from keyof Instances instead of this["ClassName"] and permit kinda shaky types if used improperly. Now consider this:
type GetProperties<T extends Instance> = { [K in keyof T]: (() => any) extends T[K] ? never : K }[Exclude<
	keyof T,
	"ClassName"
>];

interface Signal {}

interface Instance {
	GetPropertyChangedSignal<T extends GetProperties<this>>(propertyName: T): Signal;
}

function g(o: Folder | Model) {
	o.GetPropertyChangedSignal("");
	// This expression is not callable.
	// Each member of the union type
	// '(<T extends "Name" | "Parent" | "PrimaryPart">(propertyName: T) => Signal)
	// | (<T extends "Name" | "Parent">(propertyName: T) => Signal)'
	// has signatures, but none of those signatures are compatible with each other.
}

In this case, it should only allow "Name" | "Parent" as parameters, as that is the commonality between the two function definitions. Just simply make the type checker read the mind of the developer and figure out their intentions and then decide whether to union or intersect the function definitions based on that. Thank you.

Even in 3.6.0-dev.20190720, this is how the type checker acts if you move the type out of a generic parameter:
interface Instance {
  IsA<T extends keyof Instances>(className: T): this is Instances[T];
  GetPropertyChangedSignal(propertyName: GetProperties<this>): Signal;
}

function h(o: Folder | Model) {
  // Instance.GetPropertyChangedSignal(propertyName: "Name" | "Parent")
  o.GetPropertyChangedSignal("Name");

  if (o.IsA("Model")) {
    // Instance.GetPropertyChangedSignal(propertyName: "PrimaryPart" | "Name" | "Parent")
    o.GetPropertyChangedSignal("PrimaryPart");
  }
}

@RyanCavanaugh can we change this to an improvement request to make this work? I know type parameters like this our discouraged, but what the compiler is saying isn't true ("This expression is not callable").
In my case, I was using a "local type parameter" like this to relax the type checker's assignability checks. Shifting the type parameter's initializer to the return type causes many assignability errors.

Also, I don't think this was added to the release notes as a breaking change?


Add support for projections to IPC streams

Adds projections to IPC Streams, which is required for https://github.com/pola-rs/polars/issues/3778 which in turn is required for https://github.com/elixir-nx/explorer/issues/265 :rocket:
As this is my first PR, and I'm new to Rust, please give any feedback you find helpful :raised_hands:
Closes #1095

Ah, missed those, added it. Will remember to keep an eye on CI.

Thanks! The CI for integration has been failing lately between C# and X. We use the same steps as apache/arrow and that is an issue from there. So, as long as there is not there all is good (I do not expect any impact - if it compiles we are good).


[Suggestion] HD Heads (Generator & Renderer)

It was recently discovered that a mod called FabricTailor allowed HD Heads to be rendered once more.
There's two issues however:

Tailor is focused on Skin Changing, and adding Server Side features
Somewhat related to 1, Tailor doesn't allow Transparency in heads.

I have opened this issue/suggestion in the hopes that you can add a HD Head Generator to FZMM in addition to getting FZMM to render HD Heads.
The general theory I understand as to how Tailor allowed HD Heads to render, is allowing loading textures from a list of domains.
I don't fully understand it, and maybe for reasons I don't understand this won't be possible in FZMM, but one can always hope.
If this were to be added in FZMM, I envision two parts to it:

HD Head Generator
This would be similar to the other generators in FZMM, in that you would provide a link to a texture (e.g https://education.minecraft.net/wp-content/uploads/troll-1.png), and FZMM would give you a head with a new UUID, and that texture.
Similar to Tailor, you could use a variety of links to host the source image; Imgur, Discord, GitHub, NameMC, etc. and maybe FZMM would display an error in the GUI if you used a link that wasn't allowed

HD Head Renderer
This would allow clients to render HD Heads; I know nothing about actually rendering things, but I do know that currently when you try and load an HD Head, the following message is logged:
[Render thread/ERROR]: Textures payload contains blocked domain: https://education.minecraft.net/wp-content/uploads/troll-1.png
And I also know that allowing connections to minecraft.net or really any site that has the HD Texture (see here) lets them load.
The one thing that would defeat the point of this however, is if they can't be rendered with transparency. I do believe they can, but maybe you can give some details on that if not.

Thank you for reading through this wall of text, and I hope this can be added eventually :)

HD Head Generator
This would be similar to the other generators in FZMM, in that you would provide a link to a texture (e.g https://education.minecraft.net/wp-content/uploads/troll-1.png), and FZMM would give you a head with a new UUID, and that texture.
Similar to Tailor, you could use a variety of links to host the source image; Imgur, Discord, GitHub, NameMC, etc. and maybe FZMM would display an error in the GUI if you used a link that wasn't allowed

to do this I would have to implement an image scaling, so that if for example you upload a 128x128 skin you can use the rescaled 64x64 head textures without having to specifically create 128x128 head textures for that image, the easiest would be to simply have a button to save the skin and then you upload it to the site of your choice, but I could also support some sites, but I would have to look which ones have an API that allows me to do so, since:

Imgur is probably fine, idk
Discord would require account and they have closed API
GitHub as far as I know doesn't allow hosting images through an API
NameMC stores the skins that players put on using the mojang API, so besides not allowing to upload images it would not support hd skins.

but it would be an interesting implementation

HD Head Renderer
This would allow clients to render HD Heads; I know nothing about actually rendering things, but I do know that currently when you try and load an HD Head, the following message is logged:
[Render thread/ERROR]: Textures payload contains blocked domain: https://education.minecraft.net/wp-content/uploads/troll-1.png
And I also know that allowing connections to minecraft.net or really any site that has the HD Texture (see here) lets them load.
The one thing that would defeat the point of this however, is if they can't be rendered with transparency. I do believe they can, but maybe you can give some details on that if not.

I think for this point is easier to use CustomSkinLoader, it supports hd skins, several places to host the image and transparent skins, although it would not make sense to generate a hd head without being able to render it with its skin, but I think it's easier than redoing something similar to this mod that already exists.

I think you might be misunderstanding the first part or I'm misunderstanding your reply.
Users would create textures themselves and upload to an allowed site; FZMM would take that link, add it inside of
{"textures":{"SKIN":{"url":"YOUR-LINK-HERE"}}}

Encode that ^ to b64, and then take the b64 and use it in the Texture tag of the head (Inside of SkullOwner -> Properties -> Textures)
FZMM would also generate the UUIDs for the Skull, in their appropriate fields.
The reasoning I had behind FZMM implementing a renderer for the HD Heads is so it went smoother with the generator; Links the generator accepts will also be loaded by the Renderer.
As for the second part I'll look into CustomSkinLoader, thanks for suggesting that.

ah ok ok, I thought you meant to upload the skin of the newly generated head and not just generate the nbt from the url

CustomSkinLoader does allow rendering HD Heads with transparency, so that's good, thank you for suggesting it.
The only thing I have left to add to this suggestion, is that if the rendering part of HD Heads is added to FZMM, it would be good if it allowed loading textures from a small but select group of domains like the ones I originally mentioned: imgur.com, discord.com, github.com, etc. in addition to minecraft.net, as that would be the easiest way for users to upload their own textures for new HD Heads.
Also, for more clarity on the whole process of HD Heads:

Head contains a texture value in its NBT, such as {"textures":{"SKIN":{"url":"https://education.minecraft.net/wp-content/uploads/aviators2.png"}}}
That texture is in the format of a normal skin, but the body, etc, is left blank, and whatever image(s) you want is in place of the head
In older MC versions these heads loaded by default and new ones were created by uploading the textures to the education site
In newer versions the domain (https://education.minecraft.net/) is blocked, so HD Heads don't render unless you have a mod that unblocks that domain/allows other domains, and new ones can't be created because you can't upload textures to the site anymore, and alternative domains are blocked just like the main one.

Closing this issue, as Mojang has deleted every single HD Head skin texture from the education site. HD Heads are now truly dead.
Additionally it turns out mods such as SoHead and hdheadfinal existed, so I should have closed this long ago. But I guess it doesn't really matter now.


Bugfix : Support for using CoreNEURON in embedded mode with external mechanisms

modl_reg was not called when corenrn_embedded_run was used
for running simulations with in-memory mode transfer
move corenrn_embedded_run() into enginemech.cpp to avoid
link error for calling coreneuron::modl_reg

fixes #274

That is libnrniv.so is agnostic as to whether external mod files will be included in the simulation. I wonder if the same can be done for CoreNEURON.

@nrnhines : I think we simply had a bug in the sense that corenrn_embedded_run was not calling modl_reg at all.

How about tomorrow at 8am my time?

Yes, fine for me. (I am also available now if you prefer).

travis CI is taking long time but previous build is almost green. So I will go ahead with the merge.


Thanks for kicking me!

I'm really happy you just kicked me
[/sarcasm]

Who did that!?

Idk
I was just about to do somthin aswel

It was decided that it would be better for you to make changes in a separate repo, and then make a pull request when your code is reviewed.

Ah, okay @The-Cheese-Eater.

My codes still here, It seems somthing with sgit was wrong :P (My Blog code) Can I request a update to the site?

Once we get CI set up maybe we can let quillz back in, but for now, the best way to prevent stuff like what happened from happening is with pull reqs. Sorry quillz.

Still you guys have messed loads of code up before?  This isn't fair. Changes can always be reverted. Does he get a second chance? I'm not sure I want to be here anymore.

You know full well that we do not hate you quillz, I've known you for a year by now. I have invited him back into the team because this project deserves not to get into disputes. However, Quillz must understand this there is a limit to the amount of chances I can give before people start getting annoyed at having to revert changes all the time. With respect @andrewjcole, but you were not zeus trying to scrape backups from the developer site to patch the mess. I am acting as a admin within this project, and I intend to keep it that way; becoming a negotiator to solve personal disputes is certainly not high upon my agenda.
I'm tired of all that grumpy bore I just wrote, shall we have some lightheartedness and positivity again? :)

I typed that comment before I fully understood. I apologise. I'll think of this as a lesson for us all. However, please be respectful to Quillz. People do make mistakes.

 I know you used the command line thing, and you're new at it, but that's not really an excuse. If you know you're bad at the command line git, you should have used a GUI-based git, especially on a critical project like OpenSprites. You can test command line git all you want on your own repos.
What you did was pretty bad.
We didn't mean to hate when we removed you from the organization. We just can't afford huge code-deletions and stuff in the future.
Just because you do not have write permissions to the repo does not mean you can't contribute. You can create your own fork, work there, and send a pull request when you're done. You can still comment on our issues and commits too. If you show that you can make significant changes and handle git responsibly in your fork, we'll give you write permissions back.
To be clear, you are not banned from the organization entirely. We want to be nice and give you a second chance, it's just that we can't give you write permissions at the moment, because of what happened.

This is the second sweeggyest issue #137

Sorry, but if you don't have any legitimate comments, there's no reason to keep this open.


VSCode doesn't recognized OneDrive folders any more

- Environment:

VSCode Version: Code 1.13.1 (379d2efb5539b09112c793d3d9a413017d736f89, 2017-06-14T18:21:47.485Z)
OS Version: Windows_NT ia32 10.0.16215
Extensions:  N/A

Extension
Author
Version

python
donjayamanne
0.6.5

code-runner
formulahendry
0.6.20

vscode-javac
georgewfraser
0.0.41

csharpextensions
jchannon
1.3.0

mssql
ms-mssql
1.0.0

csharp
ms-vscode
1.10.0

PowerShell
ms-vscode
1.3.2

vscode-icons
robertohuertasm
7.9.0

- Steps to Reproduce:

Open VSCode
Go to File, Open Folder.
Look for the OneDrive folder and click OK
Error comes out: Path doesn't exist!!

Even, if you mapped to shared folder on OneDrive it still gives the error.
This is not happening with any other IDE editors:
- Big FYI!!!  -> This started to happened after Windows 10 Build 16215 had a recent OneDrive update this week.

Found the update:  on June 14th, 2017

Version: 17.3.6931.0609

This is very important as I doing all my presentation code samples on my onedrive to show cross-platform scenario between Linux and Windows.
This is a Big handicapped!

This is being tracked in #27285
Please follow the thread in #27285 and add your input there
cc @bpasero


Map moves when handling a custom control with a draggable input ( input type="range" )

[ x] I've looked at the documentation to make sure the behavior is documented and expected
[ ] I'm sure this is a Leaflet code issue, not an issue with my own code nor with the framework I'm using (Cordova, Ionic, Angular, React…)
[x ] I've searched through the issues to make sure it's not yet reported

Steps to reproduce
Steps to reproduce the behavior:

Open codepen: https://codesandbox.io/s/brave-cherry-k690v
Drag both controls in the upper right corner
...

Expected behavior
Map remains at it's current position, and the controls should be manipulated.
Current behavior
Custom material-ui component: Only works by clicking on the line ( dragging the handle doesn't work )
The default browser input is draggable, but the map moves with the mouse cursor
Environment

Leaflet version: 1.7.1
Browser (with version):Any
OS/Platform (with version): Any

Additional context
Minimal example reproducing the issue
Example: https://codesandbox.io/s/brave-cherry-k690v

[ x] this example is as simple as possible
[ ] this example does not rely on any third party code

Just use disableClickPropagation.

@johnd0e Thanks for the tip, but that doesn't work. I've tried it. It disables the clicks, but not the 'dragging'

Unfortunately we can't help you with React.

@johnd0e I'm not asking for help with React. I wonder what happens to the HTML MouseEvent?
It seems to get canceled or prevented or discarted? I want to figure out whats wrong.. so the problem can be fixed.
If any framework/user is attaching listeners to the window/rootnode level for performance reasons, things should not stop working?
I know you could simply add an event listener to the button itself and handle it that way, looks to me like using ducktape to fix a car.

I'm not asking for help with React.

So are you able to reproduce the same issue without React or not?

Took me a while to figure out their setup, but in plain vanillaJS
https://codesandbox.io/s/upbeat-cerf-04cjb
I've added the same slider component below the map to illustrate that it works there, but not in a Control 'container'

Now I see your issue.
This is expected by design.
Try useCapture with addEventListener.


:lady_beetle: No constructor on ERC721 and ERC20. Is this the intended behavior?

Hey @Vectorized,
I was going to give Solady's ERC721 a run but noticed that it breaks compatibility with solmate ERC721 as well as OpenZeppelin's ERC721.
I also checked and ERC20 also breaks compatibility with solmate and OZ.
Is this the intended behavior?

Yes. It's intended. This is to make it compatible with upgradeable (i.e. proxied) contracts.

Ah got it


[Bugfix] Fix backtrack Sonar issue

Looks like my last fix didn't actually fix the issue. Trying something else. I Sonar doesn't tell me if I fixed the issue till code is merged in...............

Asked Will to review this after PR was merged in (will revert this if this isn't good).
Sonar doesn't update the "Security Hotspots" tab till after PR is merged in. So even though the post above is saying "0 Security Hotspots", I don't know if PR fixes the issue till after it been merged in and Sonar does a scan of the whole repo.


monitor for pool solvency issues

from Leena (@leena56):
"Ideally the thornode bot cc Paul monitors for pool solvency issues, and alerts everyone everytime there is a solvency issue change. The bot should then monitor the queue and identify if a backlog is ocurring. Any community member can "fix" any issues by donating in the assets to the system to allow it to recover."

The process should be:

Pull the current Asgard and yggdrasil vault addresses from the system.
http://<IP_ADDRESS>:1317/thorchain/vaults/yggdrasil <- ygg balance && addresses
http://<IP_ADDRESS>:1317/thorchain/vaults/asgard <- asgard balances
http://<IP_ADDRESS>:1317/thorchain/pool_addresses <- Asgard address

Query for the respective on-chain addresses
https://dex.binance.org/api/v1/account/bnb1ysh9qt0l70jjac0jjvat7rdlv2752sws0x34qr

Compare the two. Any time solvency changes, send an alert.

Monitor the queue and send an alert anytime the queue starts growing.
http://<IP_ADDRESS>:1317/thorchain/queue

Is this done now @mr-rooftop?


xFirewall: DisplayName not set correctly if not in desired state

Ok, this is an odd one, found it by chance. So short background, I usually use DSC through Puppet, but no worries I have reproduced this strangeness in "native" DSC code as well. I had a few rules (one bultin for SNMP and the other one created by the installation of NSClient++) that have been set to remote address 'Any' by default. Looking into that, wanting to restrict the allowed addresses, I created a suitable config with xFirewall and in a big environment its rather nice to in an easy way see "what" actually set something to "something" so I thought of incorporating the puppet modules name in the DisplayName of the rule. That is when I stumbled on this one: If the DisplayName is not already in the desired state it will be set to Name for some unknown reason.
So this is what I got:
configuration` 'NSclient_FW_rule' {
  Import-DscResource -modulename xnetworking -moduleversion '<IP_ADDRESS>'
  node localhost {
    xfirewall 'nrpe' {
      name = '{9142D173-5ACD-4AA5-BA0E-4FE2FB4CEB36}'
      ensure = 'Present'
      displayname = 'NSClient++ Monitoring Agent'
      description = 'NSClient++ Monitoring Agent'
      action = 'Allow'
      direction = 'Inbound'
      enabled = $true
      protocol = 'TCP'
      program = 'C:\Program Files\NSClient++\nscp.exe'
      remoteaddress = @(my_array_of_remoteaddresses)
      localport = '5666'
    }
  }
}

What I expect to see is:
displayname = 'NSClient++ Monitoring Agent'
What I get, if not in desired state (say the rule was manually renamed to 'This is a faulty displayname'):
displayname =  '{9142D173-5ACD-4AA5-BA0E-4FE2FB4CEB36}'
While if the name is already 'NSClient++ Monitoring Agent' I get:
displayname = 'NSClient++ Monitoring Agent'
I've tried different versions of the module as <IP_ADDRESS> is the one included with puppetlabs-dsc, but natively <IP_ADDRESS> and <IP_ADDRESS> gives me the same results. To get the correct name set again you have to either:
Set-NetFirewallRule -name '{9142D173-5ACD-4AA5-BA0E-4FE2FB4CEB36}' -NewDisplayName 'NSClient++ Monitoring Agent' or cut'n paste in the Firewall GUI....

Hi @freni59,
Thanks for raising this and all the info.
Are you able to apply your config above with the -Verbose keyword specified and then dump the log here? xFirewall has lots of Verbose logging entries that may tell us what is going on here.
There are a few little "gotchas" with the way the *-NetFirewallRule cmdlets work with "Built-in" rules that may be involved here. But I'll know more with the Verbose logs hopefully.
Thanks again!

Hi @PlagueHO,
Lets see I ran an apply with verbose and this is what I got as an output:
VERBOSE: Perform operation 'Invoke CimMethod' with following parameters, ''methodName' = SendConfigurationApply,'className' = MSFT_DSCLocalConfigurationManager,'namespaceName' = root/Microsoft/Windows/Des
iredStateConfiguration'.
VERBOSE: An LCM method call arrived from computer HERA with user sid S-1-5-21-3466306919-469524849-1554712323-204576.
VERBOSE: [HERA]: LCM:  [ Start  Set      ]
VERBOSE: [HERA]: LCM:  [ Start  Resource ]  [[xFirewall]nrpe]
VERBOSE: [HERA]: LCM:  [ Start  Test     ]  [[xFirewall]nrpe]
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Test-TargetResource: Checking settings for firewall rule with Name '{96804A6E-712C-4B3F-B96B-E85CF44B18B4}'.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Test-TargetResource: Find firewall rule with Name '{96804A6E-712C-4B3F-B96B-E85CF44B18B4}'.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Test-TargetResource: Check each defined parameter against the existing firewall rule with Name '{96804A6E-712C-4B3F-B96B-E85CF44B18B4}'.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Get-FirewallRuleProperty: Get all the properties and add filter info to rule map.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Test-RuleProperties: DisplayName property value 'Wrong DisplayName' does not match desired state 'NSClient++ Monitoring Agent'.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Test-RuleProperties: Test Firewall rule with Name '{96804A6E-712C-4B3F-B96B-E85CF44B18B4}' returning False.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Test-TargetResource: Check Firewall rule with Name '{96804A6E-712C-4B3F-B96B-E85CF44B18B4}' returning False.
VERBOSE: [HERA]: LCM:  [ End    Test     ]  [[xFirewall]nrpe]  in 3.5160 seconds.
VERBOSE: [HERA]: LCM:  [ Start  Set      ]  [[xFirewall]nrpe]
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Set-TargetResource: Applying settings for firewall rule with Name '{96804A6E-712C-4B3F-B96B-E85CF44B18B4}'.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Set-TargetResource: Find firewall rule with Name '{96804A6E-712C-4B3F-B96B-E85CF44B18B4}'.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Set-TargetResource: We want the firewall rule with Name '{96804A6E-712C-4B3F-B96B-E85CF44B18B4}' to exist since Ensure is set to Present.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Set-TargetResource: We want the firewall rule with Name '{96804A6E-712C-4B3F-B96B-E85CF44B18B4}' to exist and it does. Check for valid propert
ies.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Set-TargetResource: Check each defined parameter against the existing firewall rule with Name '{96804A6E-712C-4B3F-B96B-E85CF44B18B4}'.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Get-FirewallRuleProperty: Get all the properties and add filter info to rule map.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Test-RuleProperties: DisplayName property value 'Wrong DisplayName' does not match desired state 'NSClient++ Monitoring Agent'.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Test-RuleProperties: Test Firewall rule with Name '{96804A6E-712C-4B3F-B96B-E85CF44B18B4}' returning False.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Set-TargetResource: Updating existing firewall rule with Name '{96804A6E-712C-4B3F-B96B-E85CF44B18B4}'.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Set-NetFirewallRule DisplayName: {96804A6E-712C-4B3F-B96B-E85CF44B18B4}
VERBOSE: [HERA]: LCM:  [ End    Set      ]  [[xFirewall]nrpe]  in 1.5310 seconds.
VERBOSE: [HERA]: LCM:  [ End    Resource ]  [[xFirewall]nrpe]
VERBOSE: [HERA]: LCM:  [ End    Set      ]
VERBOSE: [HERA]: LCM:  [ End    Set      ]    in  5.4850 seconds.
VERBOSE: Operation 'Invoke CimMethod' complete.
VERBOSE: Time taken for configuration job to complete is 5.577 seconds

I did a fork and found part of a solution, a "Name" that should be a "DisplayName", can put up a pull request for that one, but I am a little bit too rookie to find out how to correct the verbose output because it still says it is changing the DisplayName to Name....

There is something funky with that line in the verbose output. this is the output from a config run when the remoteaddresses are wrongly set to Any, the DisplayName is not change but it still gives that output line stating Set-NetFirewallRule.... That output might be a totally different issue?
VERBOSE: Perform operation 'Invoke CimMethod' with following parameters, ''methodName' = SendConfigurationApply,'className' = MSFT_DSCLocalConfigurationManager,'namespaceName' = root/Microsoft/Windows/Des
iredStateConfiguration'.
VERBOSE: An LCM method call arrived from computer HERA with user sid S-1-5-21-3466306919-469524849-1554712323-204576.
VERBOSE: [HERA]: LCM:  [ Start  Set      ]
VERBOSE: [HERA]: LCM:  [ Start  Resource ]  [[xFirewall]nrpe]
VERBOSE: [HERA]: LCM:  [ Start  Test     ]  [[xFirewall]nrpe]
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Test-TargetResource: Checking settings for firewall rule with Name '{96804A6E-712C-4B3F-B96B-E85CF44B18B4}'.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Test-TargetResource: Find firewall rule with Name '{96804A6E-712C-4B3F-B96B-E85CF44B18B4}'.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Test-TargetResource: Check each defined parameter against the existing firewall rule with Name '{96804A6E-712C-4B3F-B96B-E85CF44B18B4}'.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Get-FirewallRuleProperty: Get all the properties and add filter info to rule map.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Test-RuleProperties: RemoteAddress property value 'Any' does not match desired state '<IP_ADDRESS>/<IP_ADDRESS>,2001:6b0:17:f001::/64'.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Test-RuleProperties: Test Firewall rule with Name '{96804A6E-712C-4B3F-B96B-E85CF44B18B4}' returning False.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Test-TargetResource: Check Firewall rule with Name '{96804A6E-712C-4B3F-B96B-E85CF44B18B4}' returning False.
VERBOSE: [HERA]: LCM:  [ End    Test     ]  [[xFirewall]nrpe]  in 2.9840 seconds.
VERBOSE: [HERA]: LCM:  [ Start  Set      ]  [[xFirewall]nrpe]
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Set-TargetResource: Applying settings for firewall rule with Name '{96804A6E-712C-4B3F-B96B-E85CF44B18B4}'.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Set-TargetResource: Find firewall rule with Name '{96804A6E-712C-4B3F-B96B-E85CF44B18B4}'.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Set-TargetResource: We want the firewall rule with Name '{96804A6E-712C-4B3F-B96B-E85CF44B18B4}' to exist since Ensure is set to Present.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Set-TargetResource: We want the firewall rule with Name '{96804A6E-712C-4B3F-B96B-E85CF44B18B4}' to exist and it does. Check for valid propert
ies.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Set-TargetResource: Check each defined parameter against the existing firewall rule with Name '{96804A6E-712C-4B3F-B96B-E85CF44B18B4}'.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Get-FirewallRuleProperty: Get all the properties and add filter info to rule map.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Test-RuleProperties: RemoteAddress property value 'Any' does not match desired state '<IP_ADDRESS>/<IP_ADDRESS>,2001:6b0:17:f001::/64'.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Test-RuleProperties: Test Firewall rule with Name '{96804A6E-712C-4B3F-B96B-E85CF44B18B4}' returning False.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Set-TargetResource: Updating existing firewall rule with Name '{96804A6E-712C-4B3F-B96B-E85CF44B18B4}'.
VERBOSE: [HERA]:                            [[xFirewall]nrpe] Set-NetFirewallRule DisplayName: {96804A6E-712C-4B3F-B96B-E85CF44B18B4}
VERBOSE: [HERA]: LCM:  [ End    Set      ]  [[xFirewall]nrpe]  in 1.4530 seconds.
VERBOSE: [HERA]: LCM:  [ End    Resource ]  [[xFirewall]nrpe]
VERBOSE: [HERA]: LCM:  [ End    Set      ]
VERBOSE: [HERA]: LCM:  [ End    Set      ]    in  5.0160 seconds.
VERBOSE: Operation 'Invoke CimMethod' complete.
VERBOSE: Time taken for configuration job to complete is 5.176 seconds


Update Script to Check_mk Version 2.0.0

Hi,
can you please update your script and instructions to Check_mk version 2.0.0.
Thanks!

Hello, the script also works with Check_MK in version 2.0.0. The configuration is slightly different due to the new structure of the dashboard. I have now described this in the README.md and added screenshots. Please have a look if you can handle it.

Hi,
it works fine but the Telegram Messages come twice.

Since I cannot reproduce this on two Check_MK instances I assume a configuration fault.
Have you seen and checked this? Link: https://github.com/filipnet/checkmk-telegram-notify/issues/1

Sorry my fault,. I send to all Users 😄

Ok, great. The project is still very young, if you use it successfully and like it, I would be happy if you support the repository-rating with a star.

i am having a problem that i can only send to 1 telegram user.How can send notifications to multiple users telegram

i am having a problem that i can only send to 1 telegram user.How can send notifications to multiple users telegram


Hide array display area with hide_display option

In an array editor I want to hide the "display area", ie the UL element displayed under the title and checklist for each array element.
In the source code I've found the undocumented hide_display option that does exactly that, but I don't understand exactly where this option is supposed to be set.
My configuration is something like:
something:
{
    type: 'array',
    title: 'Something',
    format: 'table',
    uniqueItems: true,
    required: true,
    items: {
        type: 'object',
        id: '_id',
        enum: [],
        headerTemplate: 'Something {{i}}'
    }
}

I want to hide the _id field (see attached image).

This is not a JSON Editor's bug. The problem was caused by a bug in my application.
I can confirm that the following works as expected:
items: {
    type: 'object',
    id: '_id',
    enum: [],
    headerTemplate: 'Something {{i}}',
    options: {
        hide_display: true
    }
}

I'd suggest to describe this option in the documentation.


flutter: Pass flutter_tools package_config.json to Dart runtime

https://github.com/flutter/flutter/pull/150340, released in Flutter 3.24, changes the way flutter_tools attempts to find its own package_config.json.
The path must now be passed to the Dart runtime upon launch. Without it, the package_config.json cannot be found, and this can lead to crashes when using the Flutter CLI, particularly for Web development.
_TypeError: Null check operator used on a null value

#0      currentPackageConfig (package:flutter_tools/src/dart/package_map.dart:16:56)
#1      _loadDwdsDirectory (package:flutter_tools/src/isolated/devfs_web.dart:1234:45)
#2      WebAssetServer.start (package:flutter_tools/src/isolated/devfs_web.dart:270:15)
<asynchronous suspension>
#3      WebDevFS.create (package:flutter_tools/src/isolated/devfs_web.dart:860:22)
<asynchronous suspension>
#4      ResidentWebRunner.run.<anonymous closure> (package:flutter_tools/src/isolated/resident_web_runner.dart:328:19)
<asynchronous suspension>
#5      asyncGuard.<anonymous closure> (package:flutter_tools/src/base/async_guard.dart:111:24)
<asynchronous suspension>

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 354174

aarch64-linux

:fast_forward: 8 packages marked as broken and skipped:

fluffychat
fluffychat-web
fluffychat-web.pubcache
fluffychat.debug
fluffychat.pubcache
kazumi
kazumi.debug
kazumi.pubcache

:x: 11 packages failed to build:

expidus.calculator
expidus.calculator.debug
expidus.calculator.pubcache
expidus.file-manager
expidus.file-manager.debug
expidus.file-manager.pubcache
flutter319 (flutterPackages-bin.v3_19 ,flutterPackages.v3_19)
flutterPackages-source.v3_19
rustdesk-flutter
rustdesk-flutter.debug
rustdesk-flutter.pubcache

:white_check_mark: 48 packages built:

ente-auth
ente-auth.debug
ente-auth.pubcache
finamp
finamp.debug
finamp.pubcache
firmware-updater
firmware-updater.debug
firmware-updater.pubcache
flclash
flclash.debug
flclash.pubcache
flet-client-flutter
flet-client-flutter.debug
flet-client-flutter.pubcache
flutter (flutter324 ,flutterPackages-bin.stable ,flutterPackages-bin.v3_24 ,flutterPackages.stable ,flutterPackages.v3_24)
flutter326 (flutterPackages-bin.v3_26 ,flutterPackages.v3_26)
flutter327 (flutterPackages-bin.beta ,flutterPackages-bin.v3_27 ,flutterPackages.beta ,flutterPackages.v3_27)
flutterPackages-source.beta (flutterPackages-source.v3_27)
flutterPackages-source.v3_24 (flutterPackages-source.stable)
flutterPackages-source.v3_26
hiddify-app
hiddify-app.debug
hiddify-app.pubcache
hover
intiface-central
intiface-central.debug
intiface-central.pubcache
localsend
localsend.debug
localsend.pubcache
musicpod
musicpod.debug
musicpod.pubcache
python311Packages.flet
python311Packages.flet-runtime
python311Packages.flet-runtime.dist
python311Packages.flet.dist
python312Packages.flet
python312Packages.flet-runtime
python312Packages.flet-runtime.dist
python312Packages.flet.dist
sly
sly.debug
sly.pubcache
yubioath-flutter
yubioath-flutter.debug
yubioath-flutter.pubcache


Support for new LIFX White 800

Looks like LIFX has a new bulb that is cheaper and color-temperature-only. Similar to my other open issue requesting LFXLight be able to report its model number (or capabilities), how does a developer distinguish between these (now) two different bulb types?
http://www.lifx.com/products/white-800

Seconded the bulbs need to report their module numbers.


⚠ Processors has degraded performance

In ee249a2, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 93 ms

Resolved: Processors performance has improved in 6b6e476.


Session hosts getting DomainTrustCheck and DomainJoinedCheck with AAD

When session hosting joining AAD, I am getting two errors: DomainTrustCheck and DomainJoinedCheck.
All other stacks can be deployed successfully.
This seems to be a new issue since the stack I deployed 3 days ago was working perfectly. Any recent commits/changes that could possibly cause this? Thanks.

@mpchen thanks for the feedback, we are investigating.

@mpchen just ran a test deployment and had no issues with AAD:

Did something change in your environment?

Thanks @danycontre for the quick response. May I ask which image you use for session host? I use Windows11 2H multi session with Office 365. Would like to try a different one to see whether it is caused by the image.


java.lang.ClassCastException: java.lang.Double cannot be cast to java.lang.Long 	at com.kazurayam.inspectus.materialize.selenium.WebPageMaterializingFunctions.lambda$new$1(WebPageMaterializingFunctions.java:77)

Caused by: java.lang.ClassCastException: java.lang.Double cannot be cast to java.lang.Long
	at com.kazurayam.inspectus.materialize.selenium.WebPageMaterializingFunctions.lambda$new$1(WebPageMaterializingFunctions.java:77)
	at com.kazurayam.inspectus.materialize.selenium.WebPageMaterializingFunction$accept.call(Unknown Source)
	at materialize.takeScreenshot(materialize:21)
	at Script1667709728945$takeScreenshot.callCurrent(Unknown Source)
	at materialize.run(materialize:50)

0.14.2, WebPageMaterialzingFunctions.java:77
float dpr = (long)js.executeScript("return window.devicePixelRatio;") * 1.0f;

The currnet code compiles and runs OK on my Mac book air.
on my Mac DPR is possiblly 2.0
The current code failed on my Windows 10 Note book of Mouse.
i do not know th value of DPR on this device, but possibly is not 2. It would be some floating point value, so an Exception raised.

On Windows PC, I ran MyAdmin test, and got
Caused by: java.lang.ClassCastException: java.lang.Long cannot be cast to java.lang.Double
	at com.kazurayam.inspectus.materialize.selenium.WebPageMaterializingFunctions.lambda$new$1(WebPageMaterializingFunctions.java:77)
	at com.kazurayam.inspectus.materialize.selenium.WebPageMaterializingFunction$accept.call(Unknown Source)
	at Script1668563538525$_run_closure1.doCall(Script1668563538525.groovy:28)
	at processTargetList.run(processTargetList:20)
        ...

This time, it seems that the code js.executeScript("return window.devicePixelRatio;") returned Long, not Double.
In other words, on Windows, the code   js.executeScript("return window.devicePixelRatio;") sometimes returns Double, and sometimes returns Long. Undetermined behavior.
How to manage this?


BUG: special: ncfdtridfn and ncfdtridfd do not work reliably

In scipy.special, ncfdtridfn and ncfdtridfd do not work reliably:
In [12]: from scipy.special import ncfdtr, ncfdtridfd, ncfdtridfn

In [13]: p = ncfdtr(5, 3, 1.5, 1.25)

In [14]: ncfdtridfn(p, 3, 1.5, 1.25)  # Should be 5
Out[14]: 1e-100

In [15]: p = ncfdtr(2, 2, 1.5, 1.25)

In [16]: ncfdtridfd(2, p, 1.5, 1.25)  # Should be 2
Out[16]: 1e+100

Version info:
In [42]: import sys

In [43]: sys.version_info
Out[43]: sys.version_info(major=3, minor=5, micro=2, releaselevel='final', serial=0)

In [44]: import numpy as np

In [45]: np.__version__
Out[45]: '1.12.1'

In [46]: import scipy

In [47]: scipy.__version__
Out[47]: '1.0.0.dev0+2b3ce2a'

Note that, as pointed out in https://github.com/scipy/scipy/issues/7693, the docstrings for these functions show the arguments in the wrong order.  The problem in this issue was discovered while checking out that issue.

Apparently special errors are ignored by default (from the special module docstring, since at least as far back as 2008: "Some of the special function routines can emit warnings or raise exceptions when an error occurs. By default this is disabled..."). I don't know the reasoning behind that and it definitely seems odd to me.
Would it make sense to change the default throughout special to warn instead of ignoring?
Possibly excessive details follow: Both cases are the result of the dinvr subroutine giving up on a search, once on the low end and once on the high end of the interval it's considering. The cdffnc subroutine exits with status=1+ bound=0 in the first example (for ncfdtridfn) and status=2+ bound=inf in the second example (for ncfdtridfd), and then the C wrappers return at if ((status == 1) || (status == 2)) return bound;(https://github.com/scipy/scipy/blob/master/scipy/special/cdf_wrappers.c#L226, https://github.com/scipy/scipy/blob/master/scipy/special/cdf_wrappers.c#L240). Specific error messages are set in show_error (https://github.com/scipy/scipy/blob/master/scipy/special/cdf_wrappers.c#L28) and ignored by default.

@cdouglass: Thanks for looking into this.

Would it make sense to change the default throughout special to warn instead of ignoring?

I don't know about other cases in special, but in this case, when the function "knows" it has failed, it should raise an exception.
Eventually, it would be nice to fix the algorithm, but I haven't looked into how hard that would be.

If an exception seems too extreme, then definitely a warning should be generated, and the return value should be nan.  Also, the docstring should make clear that nan is returned (and a warning is generated) if either there is no inverse or if the algorithm fails to find an inverse.

The original design decisions why it's silent by default probably
predate 2002 (and possibly Python's warnings module). So it's unlikely
the original reasons apply any more.
Originally, scipy.special printed warnings to stdout and this was
changed to Python warnings much later --- for the former it makes sense
to be silent by default. Emitting warnings also has a runtime cost,
which is paid every time even if the user has silenced the message via
the warnings module, and which may matter when calculating many values.
It is not so clear if all the warnings that can be emitted are all that
necessary.
Moreover, it's useful to note that most of the functions are ufuncs, for
which warnings and errors is somewhat more complex issue. There are also
performance concerns, and the issue that there is no way to break out
from the outer ufunc loop.
The currently "correct" behavior of special functions on values out of
domain or when calculation fails, is to return nan, possibly accompanied
by a warning. Raising exceptions by default is mostly ruled out by the
usual ufunc behavior.
It would in principle be possible to postpone the emission of
errors/warnings to after the outer ufunc loop, by setting a threadlocal
failure flag on error and recording one of the messages.

In the current main (for v1.13) branch the second example is fixed but the first one is giving 0.0 hence probably not a bug but cdflib limitation (or I introduced a different type of bug).


Profile page: remove time subtext from services summary

This simplifies the services summary panel in the profile page, which also helps it stay clear when resized to smaller screen widths:
Before:

After:

Follow-on to https://github.com/studentinsights/studentinsights/pull/199 cc @bw120


Layout in BODY tag not good practice

This can cause global issues when compiling the CSS if using a CSS preprocessor such as SASS.

explain pls

For anyone who wants to pull in ngWYSIWYG into their web app or web site, your BODY margin and padding definitions in editor.css can override their BODY styles. I don't think it is necessary in editor.css.

I see what you mean now. Completely agree. I just reworked styles to push them into the scope and avoid the space pollution. I also added sass file. Thx for your feedback. :+1:

Issue solved? Maybe it is time to close it.
BTW: Sorry to keep bothering you @psergus, but I like this lib and I will be playing with it for a while.

closing it.


fixed model loading

added conda and pip environment
fixed model loading of xgboost and lstm
added default dataset with 1000 rows only
updated README.md

ensemble/ensemble_workflow.ipynb should not be changed, how do I remove it?


Create roadmap-item.md

Description
The context of the roadmap item can be read here: https://github.com/o3de/sig-release/issues/79.
Action Needed
Since the template automatically assigns kind/roadmap to the roadmap item issue, please create this label in your SIG repo if it does not exist already once you merge the PR.

label name: kind/roadmap
description: Categorizes an issue that goes in the O3DE Public Roadmap

Signed-off-by: Vincent<EMAIL_ADDRESS>

@lgleim and @epsilon-delta please review!

Updated the PR. Please review! @lgleim

Thanks @vincent6767, LGTM! Merged


[FEATURE]: Improve WSS Typesafety

Feature Summary
Improve the type safety of events in the socket.io server.
https://socket.io/docs/v4/typescript/
Code of Conduct

[X] I agree to follow this project's Code of Conduct

I'm working on this


Split user serializers

In many applications, the response from /users/<id>/ should contain a different set of fields than the response from /users/me/. This could be for privacy (not allowing other users access to confidential attributes on the user model) or simply because one requires a different set of information for the current user (e.g. settings).
It is possible to work around this issue with a custom viewset, but it does not work on legacy endpoints and adds unnecessary complexity to this common use case.
I propose that the serializer configuration for users be split into the following two settings:

user
me

I am willing to provide a PR if this feature is given a green light 🚦

Please do. Just make sure this will be backward-compatible.

Sure thing!

I just tried using this, but it didn't work for me... maybe this is because I have set up DRF using custom urlpatterns
I use djoser.views.UserView to emulate the me endpoint, this view always uses request.user as it's object, so I think it should use settings.SERIALIZERS.current_user as the serializer_class instead of settings.SERIALIZERS.user.
When I look at the urlpatterns shipped with djoser it looks like it also uses djoser.views.UserView for the me endpoint: https://github.com/sunscrapers/djoser/blob/127fe8d7c6187050ac76f2f8ded342641c9b0885/djoser/urls/base.py#L14
Am I missing something?


Bug: Spoiler block renders incorrectly inside of callouts

Describe the bug

To Reproduce

Place the following markdown in a file and render it
> [!info]
> ```spoiler
> Test
> ```

Expected behavior

The text should be hidden.
Screenshots

The text is not hidden when it should be inside of a callout.
Additional context

Fix published in latest release, looks like some weird behavior with the blur not working within callouts so have just made the cover a solid black within callouts.


[FEATURE REQUEST] Internal links

1st of all thanks for a excellent markdown extension, it is very well done.
Is there a possibility to add support for internal links? I mean clickable links to other files on one's computer?
Example: [text](path/to/file/on computer).
As icing on the cake it would be really neat to be able to have embeds/transclusions.
Example: ![text](path/to/file/on computer)
I use internal links a lot in these 2 use cases, and absence of these is a show stopper for me.
Alternatively, this could be done in your "Typora" extension.

I experimented some more with the extension today and realized this feature already exists, which I am very happy about. Many thanks.
So, I'll close this issue.


Create cards for all katakana characters

Currently there are 5 base characters on demo. We want to cover all Katakana characters but also want to avoid 48 side swipes. This is actually more of a design challenge. Refer to the gojuan table from the wikipedia page for the complete character set.

I would like to work on this issue


GH-2441 fix UpdateExprBuilder to support SPARQLStar expressions in WHERE

Fix in UpdateExprBuilder to process TripleRefs in WHERE clause

added simple test cases

Signed-off-by: damyan.ognyanov<EMAIL_ADDRESS>GitHub issue resolved: #2441
Briefly describe the changes proposed in this PR:

PR Author Checklist (see the contributor guidelines for more details):

[x] my pull request is self-contained
[x] I've added tests for the changes I made
[x] I've applied code formatting (you can use mvn process-resources to format from the command line)
[x] every commit message starts with the issue number (GH-xxxx) followed by a meaningful description of the change
[x] every commit has been signed off

Note: we merge all feature pull requests using squash and merge. See RDF4J git merge strategy for more details.

LGTM. Have you ran any tests where you actually execute this update on an RDF*-enabled store, to see if the result is as expected?

Yes. Tested it manually usgin following data:
insert data {
    <<<x:1> <y:1> <p:1>>> <p:1> 10 .
}```

and update like:

insert {
<<?t ?p ?o>> <p:1> ?newVal
}
where {
bind(<<?s ?p ?o>> as ?t)
?t <p:1> ?val
bind (?val * .66 as ?newVal)
}
so that a query like:

select * where {
?s ?p ?o .
filter(rdf:isTriple(?s))
}
will return one more solutiones each time the update is executed ....

the PR fixes a bug.

The code that handles the TripleRef's in WHERE clause reside within `TupleExprBuider` and the code in `UpdateExprBuilder` just deals with the construction of the necessary ValueExpr that are used by the TripleRefs declared in the head (INSERT/DELETE blocks) of the update ...

Would be good to have that test in our rig. I'll take a look at doing that tomorrow and then merge.
Almost time for another patch release i think.


"Applications" option not available

Describe the bug
I noticed that since the last upgrades I am not able to filter by "Application" on my query field, as metric, text or whatever I cannot choose any Application option, it appears to be disabled.
Expected behavior
It should show ICMP, http or whatever is in Zabbix.
Software versions
| Grafana | Zabbix | Grafana-Zabbix Plugin |
Grafana v7.5.11
Grafana-Zabbix Plugin 4.1.5
zabbix-server-6.0.3

This is supposed to be fixed in latest versions (4.2.0 and above, see #1258). Consider upgrading.

@gonzalo-, I don't know for sure, but I guess you will have to upgrade Grafana as well, see<PHONE_NUMBER>aaf4a0d3676f3b9024000c363d80fb

I'm experiencing the same problem and my plugin is in version 4.2.8

I can confirm that upgrading Grafana to 8.X and the plugin to the last version it works again, now I have some problems importing the dashboard from Grafana 7.X into the 8.X but I don't know if it is a problem with the plugin.

I am running Grafana 8.5.4 and the plugin alexanderzobnin-zabbix-app @ 4.2.8. This issue still persists.

Well, I am having this error while I try to import the dashboard (.json) from 7.X "Dashboard init failed"
logger=context traceID=00000000000000000000000000000000 userId=2 orgId=1 uname=gonzalo t=2022-06-21T16:09:50.31+0000 lvl=eror msg="Dashboard not found" error="Dashboard not found" remote_addr=FOOBAR traceID=00000000000000000000000000000000

Well, I am having this error while I try to import the dashboard (.json) from 7.X "Dashboard init failed"
logger=context traceID=00000000000000000000000000000000 userId=2 orgId=1 uname=gonzalo t=2022-06-21T16:09:50.31+0000 lvl=eror msg="Dashboard not found" error="Dashboard not found" remote_addr=FOOBAR traceID=00000000000000000000000000000000

I upgraded to 8.0.7 first and exported dashboards (checkbox to share externally) and then import them after you upgrade the 8.5.4 and it worked for me.
My dashboards started to break from versions later 8.0.7, so I opted to export from last working version. Hope this helps.

@kapilyanamandra no, it doesn't work at all, and my original version was 7.X

Estou com o mesmo problema, em criar macros com query type application

Vi que tem um forum aberto na Community Grafana: https://community.grafana.com/t/unsupported-data-format-error-when-performing-collection-through-zabbix-data-source/59602
Apresentando mesmo problema e sem solução ainda.

The application type has been removed.  Change the query type to Item Tag and paste the same application query in the Item tag field.  This should fix the template.

Hi, when this bug will be close, I have the same problem...


parameters should be optional and have default implicit values

There is no need to force user to add -r requirements.txt as this should be taken as default value unless someone defines it.
Same applies for the -s license_strategy.ini, which should be assumed to be license_strategy.ini. I would personally prefer a hidden file but I see you already picked this.

I agree. We could also use dot files in priority if available. Do you have a PR for this?

This can be closed because it has already been done.


Bus tracker react app

Title and Issue number
Bus Tracker (Gssoc'23) Issue #1723
Close #1723
Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have created a helpful and easy to understand README.md

@kadambari68 Please check the files you have committed. Looks like there are multiple projects you have committed. Make the changes or, close this PR. And make a new PR.

Do I need to cline the project every time I create a new branch and add a new project

No, that's not needed

If you are facing any problem, share it in discord channel. I'll look.

How can I make changes in this PR and remove the other 2 commits

You can delete those files. OR,
Close this PR, and make a new PR with proper commit. It would be better.

It would be ok if I use the se branch

Yes you can.

inactive


Improve wails messages

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.33 :tada:

Comparison is base (cb21c51) 41.54% compared to head (55ef0f3) 41.88%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #326      +/-   ##
==========================================
+ Coverage   41.54%   41.88%   +0.33%
==========================================
  Files          15       15
  Lines        1047     1053       +6
==========================================
+ Hits          435      441       +6
  Misses        569      569
  Partials       43       43

Impacted Files
Coverage Δ

internal/handler/wallet/transfer.go
59.09% <100.00%> (+4.09%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Can't write ImageDescription

now I used the example code from the readme file to write an image description to this file.
    let mut metadata = Metadata::new();
    metadata.set_tag(ExifTag::ImageDescription("Hello World!".to_string()));
    let res = metadata.write_to_file(std::path::Path::new(&file_path));
    println!("res: {:?}", res);

This does work without an error.
But then trying to read that file again with the "little_exif" code that worked before gets the error message:
thread 'main' panicked at C:\Users\mschnell\.cargo\registry\src\index.crates.io-6f17d22bba15001f\little_exif-0.4.1\src\metadata.rs:461:70:
range end index 2 out of range for slice of length 0

writing to the file made it from length 782701 to length 781906. The picture still can be shown and Windows Explorer indeed show "Hello World!" as "Betreff" (which before had been "Thumbs Annotation" ). But the ThumbsPlus software does not show any fields any more.
Several Fields Explorer shows have been stayed intact, but the fields
Auflösungseinheit
Farbdardstellung
Kamerahersteller
Kameramodell
Blendenzahl
ISO
Lichtwert
Brennweite
Maximale Blende
Messmodus
... (and many more)
Exif-Version (had been 0232)
are empty now.

Originally posted by @mschnell1 in https://github.com/TechnikTobi/little_exif/issues/11#issuecomment-2370972854
Just to get this straight: The image located at https://bitvibe.de/Upload/IMG_20240828_184255.jpg is the file before the ImageDescription has been changed?

I was able to reproduce this issue and already released a fix for this in version 0.4.2

Just to get this straight: The image located at https://bitvibe.de/Upload/IMG_20240828_184255.jpg is the file before the ImageDescription has been changed?
This is the "other one", that "Little" could not read right away.

I was able to reproduce this issue and already released a fix for this in version 0.4.2
Going to try this ASAP....

.....
Yep. No "Panic" any more.
going to do more tests ASAP....

To insert/update a single tag while keeping the rest, do
let mut metadata = Metadata::new_from_path(&some_image_path).unwrap();
metadata.set_tag(ExifTag::ImageDescription("Hello World!".to_string()));
metadata.write_to_file(& some_image_path)?;

instead of creating a new metadata struct with Metadata::new()

Works !
Great ! Thanks a lot  for your hard work !
Just a question:
While trying to re-insert the read tags into the Metadata struct, I seemed not to be able to do that as set_tag() dies not wajnt a reference, and I seem to be able to create an ExifTag from an &ExifTag. As it might be useful to create a vector of ExifTags from the Metadata read, (wjhich also needs clones of the &ExtifTags on .data() ) I wonder how to do this.
Thanks again !

I added a derive for Clone on the ExifTag - is that sufficient for your use case? Deriving Copy is less trivial as this requires an implementation of Copy for Vec<u16>, Vec<u32>, ... as well. If that answers your question I'll close this issue and #14.

And now "Little" can show the tags in the "Xiaomi" file.

In that case I'll close issue #13 as well - or is there anything else related to that open from your side?

If a new/unrelated problem arises I'd prefer a new issue instead of reopening a closed one. And thanks for the compliment ;)


SnackbarEvent constructor

Hi,
I noticed that the 3rd constructor in the SnackbarEvent
SnackbarEvent(String message, String actionText, long timeout, boolean persistent, EventHandler<? super MouseEvent> actionHandler)
is not correctly passing parameters to the main constructor, to be precise, next parameters are not being passed to the main constructor: actionText, timeout, persistent, actionHandler.
Kind regards

Hi
you are right, this is duplicate for #392  pushed the fix to maven central / jfoenix website.
Regards,


[RFP]: Asset Cost Averaging (ACA) in PolkaSwap

Proposal Due Date
July 2024
Proposal Overview
In all stock markets, the absence of a prediction of the future has proven to be a pipe dream (even for professional investors who spend all their time studying the market) and the Dollar Cost Averaging (also known as DCA) strategy is increasingly used. further. It refers to constantly investing a small, fixed amount of money, which over time can produce better results, while saving you time and nerves. It is especially used in high-volatility cryptoassets, such as $XOR, and this is a recurring topic of conversation among the SORA community.
IMO, this tool is a perfect complement to the 'Swap' & 'Trade' options in PolkaSwap with the additional possibility of starting to ignore the US Dollar as a reference value and start using others at the user's discretion, whether FIAT-pegged or not, hence the name change from 'DCA' to 'ACA'.
Development would consist of a section for generating ACA In / ACA Out strategies tailored by users considering the following essential elements:

Choose Assets to buy/sell
Define Strategy according to the following parameters: Total amount to invest/withdraw - Increment (optional) - Start date (optional) - Time period - Amount per purchase/sale

Proposal Goals
The objective of the Asset Cost Average (ACA) is to divide the total investment into smaller portions divided into time periods, where the purchase of cryptocurrencies or currencies can offer you better results than if you invested all of that in a single purchase. The idea is based on an element that we see in the markets all the time: price volatility.
Some of their sales are:

harmonizes the purchase cost of assets in the long term.
helps reduce the risk associated with investment volatility.
It is a good way to save or HODL.

As disadvantages we can find:

higher costs in terms of transactions, since you must buy regularly, with costs that may vary over time.
a lower risk is associated with a lower potential return.
it requires strong financial discipline and constant monitoring of the markets if we want to be successful with it.

Using an Asset Cost Averaging (ACA) tool reduces this last disadvantage associated with user discipline because it allows you to set it at a certain time and let the system run.
The possibility of creating multiple simultaneous strategies also allows for greater and better adaptation to the market situation at all times.
Scope of Work
I'm not able to make an assessment of this section.
Current Roadblocks and Barriers to Success
I'm not able to make an assessment of this section.
Evaluation Metrics and Criteria
The development of Asset Cost Averaging (ACA) in PolkaSwap can be considered implemented when the user can prepare his strategy according to the defined parameters, provide the assets and the operations are executed in the established time and manner.
Submission Requirements
Same as Evaluation Metrics and Criteria
Submission Method
Blockchain: Mainnet
Project Due Date
August 2024
Budget Amount
I'm not able to make an assessment of this section.

P.S. This proposal #109 can be considered a shortened initial version of the [RFP]: Polkaswap Reoccurring Buys ADCA ( Auto Dollar cost Averaging) Touching grass #102
https://github.com/sora-xor/rfps/issues/102

I like the idea of asset cost averaging as this is inline with Polkaswaps core values, allowing to trade with style and freedom.

P.S. This proposal #109 can be considered a shortened initial version of the [RFP]: Polkaswap Reoccurring Buys ADCA ( Auto Dollar cost Averaging) Touching grass #102

nice submission I'm glad someone is taking what i presented and optimizing it


Merge pull request #1 from cbfacademy/main

Syncing upstream

commit

Hi Poornima, FYI you've opened this PR against the wrong repository. When you're working on your code, you should create a temporary branch locally and push your commits to GitHub. Then when you're finished with that particular task, you should raise and merge a PR against your main branch, not the upstream one, as you've done here.
Once the PR is merged, you can then check out the main branch locally and use git pull to updated with the changes you merged in.
However, in this case, as you committed your changes directly in your main branch, there's no need for a PR.

Sorry I will create feature branch in local and then do PR.
On Thu, 4 Jul, 2024, 14:52 Gary McPherson, @.***> wrote:

Hi Poornima, FYI you've opened this PR against the wrong repository. When
you're working on your code, you should create a temporary branch locally
and push your commits to GitHub. Then when you're finished with that
particular task, you should raise and merge a PR against your main
branch, not the upstream one, as you've done here.
Once the PR is merged, you can then check out the main branch locally and
use git pull to updated with the changes you merged in.
However, in this case, as you committed your changes directly in your main
branch, there's no need for a PR.
—
Reply to this email directly, view it on GitHub
https://github.com/cbfacademy/intro-to-java-course/pull/38#issuecomment-2209053983,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BHWXNDQAMSGYX7QMTTA2ICTZKVHR7AVCNFSM6AAAAABKLM5AFGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBZGA2TGOJYGM
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Use ECMAScript 5.1 Array functions instead of goog.array

Thanks for any review

LGTM.
Added a comment about a minor and unrelated improvement.

Looks good to me as well.


Updating MixPanel breaks negative padding in Toolbar

I'm using a complex drawable type for the app icon in the toolbar.  Updating the mixpanel APK from 4.5.3 to 4.6.4 breaks the NavigationButtonStyle padding (it's set to a negative number).
Style for my navigation button:
<style name="ToolbarNavigationStyle"<EMAIL_ADDRESS>        <item name="android:paddingStart">-72dp</item>
        <item name="android:scaleType">fitStart</item>
        <item name="android:gravity">start|center_vertical</item>
        <item name="android:layout_gravity">start|center_vertical</item>
    </style>

Drawable layout:
<?xml version="1.0" encoding="utf-8"?>
<layer-list xmlns:android="http://schemas.android.com/apk/res/android">
    <item android:drawable="@drawable/ic_arrow_left"
        android:right="108dp"
        android:top="48dp"
        android:bottom="32dp"
        android:left="12dp"
        />
    <item android:drawable="@drawable/ic_logo_padded"
        android:top="0dp"
        android:bottom="0dp"
        android:left="24dp"
        />
</layer-list>

With 4.5.3

Updated to 4.6.4

@niftynei Can you please try this with the latest master? It might be related to #310 . If it's not fixed, can you please provide me your layout file? Or maybe create a project which is able to reproduce this issue? I was having trouble with reproducing it.

I'll close this for now. Please re-open if you still have this issue.


CallExpression throwing error

Reposted from https://github.com/jared-stilwell/escomplex-ast-moz/issues/88

Whenever I run the following code:
const file = fs.readFileSync(filePath);
const ast = esprima.parse(file, {range: true, comment: true});
const escomplexResult = escomplex.analyse(ast, escomplexWalker);
an the code has CallExpressions I get the following error: http://imgur.com/xTcEDzN
This code was previously working for me

This fixed the issue :
const ast = esprima.parse(file, {loc: true, range: true, comment: true});
The indication that the walker needs the loc option should be indicated.

@CMatias I added a rough regression test to try to reproduce this issue. While it would be easy to assume the restructuring of the project that occurred in the 2.0.0-alpha release resolved this issue, it seems more likely that I'm not properly representing it in the unit test. Do you mind having a look?

Going to close this for now since I can't reproduce the issue. Please reopen with additional details about how to reproduce the issue.

Hi Jared, sorry for the long radio silence.
The "bug" I was getting was simply being caused by not passing the loc option to the esprima parsing. In the newer version you don't require an ast but rather a path for a file and the parsing is made always with loc so this won't ever happen :)
For my project I need to do a custom parse of an ast besides the escomplex calculation and the previous version (1.3 I think) suited my needs better than the newer version since if I update escomplex I need to double the work on getting the ast. I think there should be an option which includes the walker but lets the user supply the ast.


Restore Darwin APIs that went missing.

We were inconsistent about our checks for the "allowed to not have params" case: some places used "command has no fields" as the check, and some used "command has only optional fields".
This caused some APIs (for invoking commands with no params) to disappear when commands that used to have no fields got optional fields added to them.
The fix:

Fixes the checks to consistently be "command has only optional fields" so the no-params APIs continue to exist for the commands that had optional fields added to them.
Fixes availability annotations for pre-existing commands with only optional fields so that we will not claim the no-params API is available before it was actually available, by basically listing all such commands in config-data.yaml.
Avoids generating deprecated overloads for the thins listed in item 2.
Moves the Darwin-only bits from the config-data.yaml in src/app into the Darwin version we are adding, and starts using that config file.

Fast-tracking platform-specific fix with platform owner review.


[BUG] Artifacts of Custom Codecs on the maven repository

What is the bug?
This is more of a question. I can see in the maven repository that we have an artifact for custom-codecs, but the last version is 2.10.0. Is there is a reason why the 2.11.0 version was not added? Is there any process that we follow currently so that artifacts are published alongside OpenSearch release.
What is the expected behavior?
A version of custom-codecs being released alongside other OpenSearch version releases.
@reta @nknize @andrross

@sarthakaggarwal97 the custom-codecs is external plugin now (see please https://github.com/opensearch-project/custom-codecs/) so there are no artifacts to publish except the plugin ZIP.

I think this is the link to the artifacts for the external plugin: https://mvnrepository.com/artifact/org.opensearch.plugin/opensearch-custom-codecs

got it, thanks for the inputs @andrross @reta


Add images and extra pages to monument submission

After the coordinates are entered.
Like the existing app where there are four sections:

Map View Photo or Sketch (1 upload limit)
Monument Area Photo (1 upload limit)
Monument Close-up Photo (1 upload limit)
Extra Pages (10 upload limit)

Accepted file types for all sections are: jpg, jpeg, png, tiff, pdf

Next button to go to submission review, then
Submit button appears on this page, as it is the last step in the submission process.

So the review step I have is to be removed?

So the review step I have is to be removed?

No, I just forgot to include that. The review step is great. Initial comment updated.

How many of these photos are required?

None of the photos are required.

I'd like to suggest that we only allow images and remove pdf's as inputs to this section.

Confirmed with Sean that it is okay to remove pdf as an acceptable file type.

This is great. Some changes will be requested in new tasks/issues, but this will be closed.


Make API accept named parameters instead of receiver lambdas when appropriate

Currently in many places the API looks like:
foo {
   param1 = 2
   param2 = "hello"
   ...
}

For example:
xaxis {
   name = "x axis name"
   mode = Mode.lines
}

The api would be easier to use if it was in this form:
xaxis = Axis(
   title = "sin"
   type= Type.`-`
)

There are a couple of upsides to this:

By looking at the function signature of xaxis{} you don't know what kind of configuration is available. Sometimes the available variables are not even in the same file as the builder.
When having Axis() accept the parameters directly, they are available immediately in the call site with their default values, and will be shown by IntelliJ when you type the constructor.
With named parameters IntelliJ will nicely autocomplete all of the available parameters, unlike when using lambdas.
Having a function body implies you can do something other than assign variables, which is not true in many cases.
Calling the constructor of Foo is the natural way to create an object of type Foo, instead of some Foo.build{} function.

Places where passing parameters in receiver lambdas is still appropriate:

When additional function calls are nested, like plot in Plotly.Page{} (title should still be a named parameter though), or  trace in plot.

@altavir If you agree I would like to submit a PR to start improving the API in this direction.

The changes you propose could be implemented without any change in API with extension function.
For example, you can add this code:
fun Axis(title: String, type: Type) = Axis.build{
    this.title = title
    this.type = type
}

This extension will allow to use xaxis = Axis(title = "sin", type= Type.-) inside layout builder. Personally, I think it is a step back in API design, but no harm done, so feel free to add the functionality you want. Just please do not add secondary constructors to Specific classes, because it could make the code much harder to maintain. Maybe you should create a separate file for such extensions. I will probably move existing helpers like this one to extensions in future as well. Jost to separate API.

My point is that parameter-based builders should be the only API. Could you explain the benefits of using receivers?

@natanfudge There is no way around it. Basically, any creation of specification is the application of a function to an empty tree. So all builders you see are functions, objects. You can add a constructor-like front to the Specification::build call, but underneath it still has functional nature and will use  Axis.()->Unit block somewhere inside.
Besides, the current approach is much better.  It makes the code more flexible by allowing to represent parts of the logic as a function and move it into another place. Since everything is a function, you can easily add new behavior to existing API without breaking anything and without introducing a lot of unnecessary parameter overrides. The current programming trend is to move from imperative to functional definition in all places, where it is possible and I agree with it.
Could you elaborate, why do you not like the functional definition beside the point, it does not work like in python?

Could you elaborate, why do you not like the functional definition beside the point, it does not work like in python?

By looking at the function signature of xaxis{} you don't know what kind of configuration is available. Sometimes the available variables are not even in the same file as the builder.
When having Axis() accept the parameters directly, they are available immediately in the call site with their default values, and will be shown by IntelliJ when you type the constructor.
With named parameters IntelliJ will nicely autocomplete all of the available parameters, unlike when using lambdas.
Having a function body implies you can do something other than assign variables, which is not true in many cases.
Calling the constructor of Foo is the natural way to create an object of type Foo, instead of some Foo.build{} function.

Well, let's discuss it.

The convention is (or will be) that the function calls a builder of appropriate specification. Some builder have a mandatory parameter, in those cases parameters could be passed outside lambda. This way is widely used in Kotlin libraries, so there should be no problems with that. The autocomplete works quite well in this case.

As I already said, auto-complete works perfectly well with receiver. You won't have positional arguments and good riddance, they could only confuse the code in this case.

You could. For example you can write some delayed or regular change right inside DSL, which is convenient. Consider this:

xaxis{
  GlobalScope.launch{
    while(true){
      title = "Current time is ${Instant.now()}"
      delay(500)
    }
  }
}

But in this case constructor is not natural. When you write xaxis = axis in this case you do not assign value to a field, because there is no backing field for xaxis. In fact, what you do is application of transformation (function) to the state tree.

You don't get this

You don't get this

In this case:

Even if you enter the source code (which you shouldn't need to do), you still wouldn't get to see what you can pass in. You need to enter the definition of Axis, and by this point you are digging through the source code to see what you can pass.
Yes you can have external documentation that tells you what you can pass but you can and should have code that self documents.
Here type is explicit in the simplest way possible:

Here, yes you can technically infer that the API wants a string from the naming of string(), but it's not explicit or simple

(what does by string() mean? If the only thing it meant was that you can pass a string then it wouldn't use property delegation - is what the user thinks)
Same thing with the default value,
Here I know the default value is null.

Here I assume/guess the default value is null

Does the dataforge library do something in this context other than serialize json? If not, then the declaration of classes can be simplified by using kotlinx.serialization.
Regrading dynamic change, something like this can work well enough:
   fun createPlot() = Plotly.plot {
        trace(points = points, mode = TraceMode.`lines+markers`)

        layout(
            xaxis = Axis(title = "Current time is ${Instant.now()}"),
            yaxis = Axis(title = "My y values")
        )
    }

fun main() {
    GlobalScope.launch {
        while (true) {
            val plot = createPlot()
            /* Use plot*/
            delay(500)
        }
    }

}

And is even more natural when change comes externally, instead of being forced by the plot itself (like in the example you gave)
fun createPlot(data : Int) = Plotly.plot {
        trace(points = points, mode = TraceMode.`lines+markers`)

        layout(
            xaxis = Axis(title = "I'm displaying data: $data"),
            yaxis = Axis(title = "My y values")
        )
    }

fun onButtonPress() {

            val plot = createPlot(getDataFromSomewhere())
            /* Use plot */

}

To sum up:

I'm not saying to not make it declarative, I'm saying to use parameters when parameters are appropriate.
The losses in user experience in not using proper parameters are so huge that any gains in programming style (which I argue are non-existent, since we can still write declaratively, see Jetpack Compose and SwiftUI)  are not worth it.

As it stands, when you use new function from the API, you either have to dig through code, or find documentation, which is by far the biggest issue.

Just write xaxis{ or Axis.build{ and you will get exactly the same level of autocomplete. You can also write this. inside and get even more specific hints. It is probably a good idea to mark those functions with @DslMarker to avoid autocapture of this from external context.
The inner delegates like string() could not be avoided because it is how the library works, it does not create explicit object with fields, it delegates everything to dynamic tree. We could add explicit types to those delegates for readability like var title: String? by string().
There is also one important thing that could not be done in a constructor-like pattern, namely nested objects. Currently, we've implemented a tiny part of plotly functionality, but there are a lot of places with nested objects creation. Implementation of such things with constructors are really ugly.
Please note, that I do not say that there should not be constructor-like builders. I've explained how to add them above, but it is not possible (and does not make sense) to make this way a primary way. Let me propose a compromise. You can add functions like this to all model classes (not inside the class, but as extension in the same file):
fun Axis(title: String, type: Type = Type.`-`, block: Axis.() ->Unit = {} ) = Axis.build(block).apply{
    this.title = title
    this.type = type
}

This way, you keep both, constructor-like behavior and ability to add new fields and inner classes without breaking the API by optional lambda in the end.  From user perspective, those functions will behave exactly the same way, you want. No additional imports are requires since they will be placed in the same file as actual classes.