Creating a new Toggle GUI element

Summary:
This pull request creates a Toggle class, extended from the Button class. It can be used as an individual Toggle or with a group of toggles.
Features
isActive property with getter/setter to define if the toggle is active/on or inactive/off.
toActiveAnimation and toInactiveAnimation functions allow the developer to easily define the toggle's properties when in an active and inactive state. These functions fire when the isActive property changes.
pointerEnterActiveAnimation, pointerOutActiveAnimation, pointerDownActiveAnimation, pointerUpActiveAnimation, pointerEnterInactiveAnimation, pointerOutInactiveAnimation, pointerDownInactiveAnimation, pointerUpInactiveAnimation functions allow the developer to easily define separate pointer animations for the Active and Inactive states. These were implemented following Button.ts as a guide.
group property allows the developer to define the group the toggle belongs to. When a Toggle in a group is toggled on, the other toggles in the group will be toggled off. This was implemented with radioButton.ts as a guide.
onIsActiveChangedObservable an Observable raised when the isActive property value changes. This was implemented with radioButton.ts as a guide.
Note: The Project compiled (minified) for me, but failed toward the end of the build due to a memory issues in the GitPod.
Original Discussion

Just an FYI, I didn't mean to include that function in control.ts in this pull request. Planned to do it in a separate request. I can remove it when I get feedback on this request.

Well I'm a bit surprised to not see the draw function. Why not having a draw?

I didn't see a draw function in the button class so it didn't even cross my mind. Could you elaborate on what you'd expect the draw function in the Toggle class would do?
Here's my notes after reviewing the draw functions in these classes:
RadioButton

The radioButton class extends Control. Control has an empty _draw() function.
It looks like the purpose of the radioButton's draw function is to create the base shape since that's not handled by Control.

Toggle

The Toggle extends Button. Button doesn't have its own draw function.
Button extends Rectangle. Rectangle doesn't have a draw function, but does have a protected _localDraw().
Rectangle extends Container. Container has a _localDraw() function and a _draw() function.
Container extends Control. Control doesn't have a _localDraw() function but does have the empty _draw()` function.
It appears that a the Container class handles the function for a Button?

If you don't mind educating me here, I'd be happy to try updating this pull request.

Well I thought the toggleButton will have its own rendering mode (something that indicates the current state of the toggle right?)
So in a sense the button has no draw as it is really generic and works as a container as you said (because we could have text button or image button)
For the toggle I was really thinking about it like a "pure" control that needs to render itself

This is a little beyond my expertise, but I am a quick learner so I'm happy to try again. I would certainly find value in being able to contribute to this project. 😃
For clarity sake for anyone else reading this, what I'm trying to build is a Toggle Button and not a Toggle Switch
So if I'm understanding you correctly, you weren't looking for the Toggle class to extend Button but be a base control. If this is correct, would I need to rework the class by extending from Control? Or would it be acceptable to extend from Container or Rectangle to avoid reimplementing properties like thickness, cornerRadius,adaptWidthToChildren, adaptHeightToChildren, etc.?

Ok I was more on the toggle switch side of the house :)
So I guess you have a good idea here.  It deserves to be merge
would you be ok to update the doc as well to include examples of your control running in the PG?

Ah, ok, I'm glad I clarified. I can certainly fix the doc and put together some playground examples.
I do have one other thought first...  Would it be wiser for Toggle to extend Rectangle instead of Button?

I found it weird the Toggle class exposed the static functions from Button. Mostly I found it confusing because the functions all have the word 'button' in it, but also, they currently create a button not a toggle.
The Toggle class extended from Button still has the standard pointerDownAnimation properties which duplicates the toggle's pointer animation properties which seems a bit confusing unless you've read the code
Button is pretty simple, so recreating its properties in Toggle is little effort
A Toggle class extending Rectangle could have its own static functions to quickly create various types of toggles.

What do you think? Should Toggle extend Rectangle or Button?

Yeah you are right! that sounds a better plan

Let's try that approach?

Sound like a plan. Should I reuse this pull request or start with a fresh one?

Maybe start a new one


get thresholding result from ercf

Description
This pr uses ercf to get anomaly result.
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #168 (bc15f63) into main (0547383) will increase coverage by 0.08%.
The diff coverage is 88.88%.

@@             Coverage Diff              @@
##               main     #168      +/-   ##
============================================
+ Coverage     70.33%   70.41%   +0.08%
- Complexity     2976     2979       +3
============================================
  Files           269      269
  Lines         14056    14069      +13
  Branches       1410     1412       +2
============================================
+ Hits           9886     9907      +21
+ Misses         3555     3547       -8
  Partials        615      615

Flag
Coverage Δ

plugin
70.41% <88.88%> (+0.08%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...c/main/java/org/opensearch/ad/ml/ModelManager.java
81.91% <81.81%> (+0.30%)
:arrow_up:

...rc/main/java/org/opensearch/ad/ml/EntityModel.java
96.42% <100.00%> (+1.19%)
:arrow_up:

.../opensearch/ad/cluster/ADClusterEventListener.java
89.09% <0.00%> (-1.82%)
:arrow_down:

...opensearch/ad/indices/AnomalyDetectionIndices.java
51.56% <0.00%> (-0.32%)
:arrow_down:

.../handler/IndexAnomalyDetectorJobActionHandler.java
28.57% <0.00%> (+0.71%)
:arrow_up:

...port/SearchAnomalyDetectorInfoTransportAction.java
64.44% <0.00%> (+8.88%)
:arrow_up:

...search/ad/transport/AnomalyDetectorJobRequest.java
100.00% <0.00%> (+13.51%)
:arrow_up:


Ensure spline based interpolation is truly discrete when using discrete

Because we were "discretizing" interpolations in place, spline-based interpolators were still affecting the edges of discrete blocks as these interpolators take into account more than two colors. After generating the discrete steps, package it up into a linear interpolator and return that.

@gir-bot lgtm


kaspersky warning that it contains trojan Trojan.Multi.GenAutorunReg.a

Kaspersky Antivirus <IP_ADDRESS> (k) it is defined as a virus binaries_extract.bat
Event: Information about detected object
User:
User type: Active user
Application name: git.exe
Application path: N:\soft\sublime_merge_build_2068_x64\Git\mingw64\bin
Component: File Anti-Virus
Result description: Detected
Type: Trojan
Name: HEUR:Trojan.BAT.Agent.gen
Precision: Heuristic Analysis
Threat level: High
Object type: File
Object name: binaries_extract.bat
Object path: H:\prog\Translumo
SHA256: 44C5D65C329DEB4B845FA5A7978DBF4F95C235C64572E4B38EED85D7AEFEC8C5
MD5: 03119D4FEC6F80ABE138CA252ADC5028
Reason: Machine learning
Databases release date: Today, 8/31/2023 9:49:00 AM
Event: Object deleted
User:
User type: Active user
Application name: git.exe
Application path: N:\soft\sublime_merge_build_2068_x64\Git\mingw64\bin
Component: File Anti-Virus
Result description: Deleted
Type: Trojan
Name: UDS:Trojan.BAT.Agent.gen
Precision: Exactly
Threat level: High
Object type: File
Object name: binaries_extract.bat
Object path: H:\prog\Translumo
MD5: 03119D4FEC6F80ABE138CA252ADC5028

Thanks! But It's another case, as I understood.

Name: HEUR:Trojan.BAT.Agent.gen
binaries_extract.bat downloads external compontents (python, models) from github via powershell web request. So for this one I can only recommend to add BAT file to exceptions.

@Danily07  it's Vietnamese so you have to use tool like google lens to translate it if needed

more

Thanks! I tried to scann EXE via kaspersky online scanner - the result didn't show any suspicious warnings/viruses.

I guess, that other viruses could infect Translumo EXE, even "System Memory" location looks suspicious...
It would be great if someone else give us their scan result

I think problem maybe come from 3rd party libraries

I also tried to scan at the first time translumo run but no problem.
I just happen after pulling model

Ok, I will try to install kaspersky on my machine and check it with running app later.


Add segment replication cluster-level setting

Fixes #4197
Checklist

[x] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@Rishikesh1159 Could you please review this PR for technical accuracy? Thanks!


SDWebImage uses more memory when image is loaded for the first time

While working with SDWebImage, we observed that the initial download of an image takes more memory than loading the image from the disk cache.

The memory consumption can be seen in the above screenshot. The call was identical both times, the image was downloaded from the internet the first time, and then loaded from disk the second time.
In this case, the issue increased the memory consumption by about 50 MB.
How could we work around this additional memory consumption?

is the image large in resolution or size?

Exactly. 4096 * 4096 pixels, jpeg.

That's a logical memory spike then.
On Πέμ, 29 Οκτ 2015 at 17:11, Emanuel Jöbstl<EMAIL_ADDRESS>wrote:

Exactly. 4096 * 4096 pixels, jpeg.
—
Reply to this email directly or view it on GitHub
https://github.com/rs/SDWebImage/issues/1354#issuecomment-152209594.

Just a reference for the image used in your git:
Its size is 4096 width x 4096 height.
So for every pixel in an image 4 bytes are used. To calculate the memory:
height x width x 4 = total bytes in memory.
4096 * 4096 * 4 = roughly 64MB

That's true, but what's the reason for the spike to not occur when the image is loaded from the disk?

bump
Is there anything I could do to help with this issue?

Not sure how to proceed here. Large images will do spikes, no matter what we do

I agree, It's not logical. It's larger than it should be. It also happens for small images (the spike is just smaller).

I think this issue is similar to #538 and #586.
#787 and 00bf467 should give you a good solution for dealing with large images. Together with the GIF handled by FLAnimatedImage, this ticket should be done now.
Closing. Let's see how this is with 4.0.0


Fix ForEachLoopUnroll use-after-free miscompile.

This pass generated incorrect borrow scopes:
%stack = alloc_stack
%borrow = begin_borrow %element
store_borrow %borrow to %stack
end_borrow %borrow
try_apply %f(%stack) normal bb1, error bb2
...
destroy_value %element
This was not showing up as a miscompile before because:

an array holds an extra copy of the unrolled elements, that array is
now being optimized away completely.

CopyPropagation now canonicalizes OSSA lifetimes independent of
unrelated program side effects.

So, since there is no explicit relationship between %borrow and the
OSSA value in %stack, we end up with:
%stack = alloc_stack
%borrow = begin_borrow %element
store_borrow %borrow to %stack
end_borrow %borrow
destroy_value %element
try_apply %f(%stack) normal bb1, error bb2
Fixes rdar://72904101 ([CanonicalOSSA] Fix ForEachLoopUnroll use-after-free miscompile.)

@swift-ci test

@swift-ci test

@swift-ci benchmark

@swift-ci benchmark

Build failed
Swift Test Linux Platform
Git Sha - cec5513373165b855cace396dc450591f8a1f7d3

Performance: -O

Improvement
OLD
NEW
DELTA
RATIO

FlattenListFlatMap
6810
4421
-35.1%
1.54x (?)

String.data.Medium
112
100
-10.7%
1.12x (?)

Code size: -O
Performance: -Osize

Regression
OLD
NEW
DELTA
RATIO

FlattenListLoop
1627
2505
+54.0%
0.65x (?)

FlattenListFlatMap
5608
6628
+18.2%
0.85x (?)

Improvement
OLD
NEW
DELTA
RATIO

DataReplaceMedium
4600
4200
-8.7%
1.10x (?)

Code size: -Osize
Performance: -Onone

Regression
OLD
NEW
DELTA
RATIO

SevenBoom
1761
2052
+16.5%
0.86x (?)

Code size: -swiftlibs

How to read the data
The tables contain differences in performance which are larger than 8% and
differences in code size which are larger than 1%.
If you see any unexpected regressions, you should consider fixing the
regressions before you merge the PR.
Noise: Sometimes the performance results (not code size!) contain false
alarms. Unexpected regressions which are marked with '(?)' are probably noise.
If you see regressions which you cannot explain you can try to run the
benchmarks again. If regressions still show up, please consult with the
performance team (@eeckstein).

Hardware Overview
  Model Name: Mac Pro
  Model Identifier: MacPro6,1
  Processor Name: 12-Core Intel Xeon E5
  Processor Speed: 2.7 GHz
  Number of Processors: 1
  Total Number of Cores: 12
  L2 Cache (per Core): 256 KB
  L3 Cache: 30 MB
  Memory: 64 GB

@swift-ci smoke test linux

@swift-ci smoke test linux

@swift-ci smoke test windows

@swift-ci smoke test windows

Build failed
Swift Test OS X Platform
Git Sha - cec5513373165b855cace396dc450591f8a1f7d3

Build failed
Swift Test OS X Platform
Git Sha - cec5513373165b855cace396dc450591f8a1f7d3

@swift-ci smoke test

@swift-ci smoke test


How to expire tokens in a bucket??

A simple question. I have the following code which limits 10 REST API calls in a minute
Bucket bucket = (Bucket) session.getAttribute(bucketKey);
if (bucket == null) {
	Bandwidth limit = Bandwidth.simple(10, Duration.ofMinutes(1));
	bucket = Bucket4j.builder().addLimit(limit).build();
	session.setAttribute(bucketKey, bucket);
}
// tryConsume returns false immediately if no tokens available with the bucket
if (bucket.tryConsume(1)) {
	// the limit is not exceeded
}else{
	// the limit has exceeded
}

I made 20 REST API calls in 1 minute and the code goes into else condition, which is the correct behavior. But then I waited fir 5 minutes and make a new REST API call and the tokens in the bucket never expire. The code still goes into else condition after 5 minutes. How I make the tokens in the bucket to expire??

Hello @bullsEye173 ,
Please provide self-executable example of problem.


Windows Client

My Setup

Operating System: Windows 10
App Version: 2.17.11
Installation type: exe

Client reports that a new version (3.0) is available. This version is not in the Microsoft Store. How do you get the new version?
Best regards.
Jörg

Check:
https://github.com/RocketChat/Rocket.Chat.Electron/releases

Better wait some days. I just installed it and all configured servers are lost.

Better wait some days. I just installed it and all configured servers are lost.

I confirm. You need to re-enter all your credentials.

i confirm too: server list does not get saved

My install is not finding my server.

Please test if the latest release did solve this. If not please reopen the issue or create a new one and tell us. I will be closing this one for now.
Thank you!


agxrobots_test ros package add url and fix .yaml file

Please add the following dependency to the rosdep database.
Package name:agxrobots_test
Package Upstream Source:https://github.com/885288421/agxrobots_test.git
Purpose of using this:This software package provides a C++ interface to communicate with the mobile platforms from Weston Robot and AgileX Robotics, for sending commands to the robot and receiving the latest robot state.
Distro packaging links:https://github.com/885288421/agxrobots_test_release.git
Links to Distribution Packages

Debian: https://packages.debian.org/

REQUIRED

Ubuntu: https://packages.ubuntu.com/

REQUIRED

Fedora: https://src.fedoraproject.org/browse/projects/

IF AVAILABLE

Arch: https://www.archlinux.org/packages/

IF AVAILABLE

Gentoo: https://packages.gentoo.org/

IF AVAILABLE

macOS: https://formulae.brew.sh/

IF AVAILABLE

Alpine: https://pkgs.alpinelinux.org/packages

IF AVAILABLE

NixOS/nixpkgs: https://search.nixos.org/packages

OPTIONAL

Please Add This Package to be indexed in the rosdistro.
ROSDISTRO NAME
The source is here:
http://sourcecode_url
Checks

[x] All packages have a declared license in the package.xml
[x] This repository has a LICENSE file
[x] This package is expected to build on the submitted rosdistro

This pull request is missing all required review information. For package releases the pull requests should be opened automatically by the bloom utility when releasing.


R code for cnv calling and plots using "gsrc"

Write R code using the package "gsrc".  Since we have a working pipeline to generate LRR and BAF in PennCNV format, start your code by reading the PennCNV input file into R.  You can find an example file in our repo here "files/pennTest2.txt".
Take this input file and do CNV calling using "gsrc".  Write CNV calls to a file, preferably in bed or bedgraph format.  This will allow for easy comparison with PennCNV output using bedtools or similar, as well as easy upload to UCSC browser for further visualization with genomic context.  Also write code to output some plots of CNV calls.
I'm asking you to do your CNV calling one sample at a time (each input file only has data from one sample).  This may not be possible.  For PennCNV, for example you need to generate a reference file using many samples before you can do CNV calling one sample at a time.  I'm going to have generating the PennCNV reference file as it's own module.  If there is a step like this in "gsrc" we can add this to the same module as the PennCNV reference.  So please split this off into separate code.
Put R code files into the "R/" directory in our repo.  Make a new .R file for each separate task, and clearly define required input files and output files.

The R script gsrcCNVcalling.R in R/ will take in a directory with the pennTest-like files and an output directory as input and output a bed file with char/start/end/num_probes/avg_signal. No header. Let me know if I should change the input.
I'm working on assigning the copy states as well within the same script.
The plot features from 'gsrc' require defined synteny blocks which requires extra processing and mapping and things. So all the pretty plots at the end of https://cran.r-project.org/web/packages/gsrc/vignettes/introduction.html won't work without synteny blocks.

@ngiangre, I had to change the header line for the txt file.  Can you change your code to work with the new header line?
I deleted the old txt file and added "files/201274980046_R04C02.PennCNV_in.txt" to the repo.  It has the new header line.
Thanks!

Yup, I can change the script to adapt to the new header line
From: Eric Karlins<EMAIL_ADDRESS>Reply-To: NCBI-Hackathons/Global_Screening_Arrays<EMAIL_ADDRESS>Date: Tuesday, March 21, 2017 at 10:42 PM
To: NCBI-Hackathons/Global_Screening_Arrays<EMAIL_ADDRESS>Cc: Nick Giangreco<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [NCBI-Hackathons/Global_Screening_Arrays] R code for cnv calling and plots using "gsrc" (#10)
@ngiangre, I had to change the header line for the txt file. Can you change your code to work with the new header line?
I deleted the old txt file and added "files/201274980046_R04C02.PennCNV_in.txt" to the repo. It has the new header line.
Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

R script was adapted to new headers, assign cnv state to region if consistent across probes in region, and was tested successfully on ubuntu with
"/opt/R-3.3.1/bin/Rscript gsrcCNVcalling.R /home/ubuntu/ngiangre/Global_Sc
reening_Arrays/files /home/ubuntu/ngiangre/Global_Screening_Arrays/files/output"
outputs bed file with: chr, start, end, number of probes, avg probe value, and deletion (-1)/duplication (+1)/no change (0) status.
Will now work on manuscript and slide describing R cnv calling approach

updating R script to assign cnv state to cna regions without excessive looping (its faster)

The gsrc plotting functions are too specific for their data-looking at other functions/packages instead

Playing with ggplot for plotting of bed files currently

finished for now. Start new issue later for R plotting/CNV calling


Recent problems on Deezer

Hi there, love the extension! 🙏🏻
I had a long-open tab in Deezer (chrome) which was working great, but then I closed and reopened it (earlier today), and the extension seems to have stopped working.
Anyone else have similar problems?

Hey, thanks for the report.
I've only had time to look at this problem today, but unfortunately Deezer has been really crappy and simply 'crashes' every time an ad is about to play (actual songs play just fine):

I get the same behavior regardless of web browser, which might indicate a problem on their end.
In any case, I'll continue to investigate this.
Thanks

Should be fixed now! Please open a new issue if you find problems.
I'm in the process of publishing a new version, which should be available on all web stores today.

Legend, thank you 🙏🏻


Change vm_util.RunThreaded to not destroy tuple list thread_params.

Prior to this change, running vm_util.RunThreaded with a list of arguments for thread_params will result in the passed-in list being modified; all entries are popped out.

Could you clarify the issue that this addresses? And perhaps rebase?

I modified the pull request comment to explain. RunThreaded currently has a behavior where it destroys the list you pass it.

Also rebased.

@skschneider: Thanks!

test this please

LGTM


BlockBandedMatrix QR and QL

@jagot I still need to tackle BlockSkylineMatrix but it should be easy to adapt this

Codecov Report

Merging #30 into master will decrease coverage by 7.49%.
The diff coverage is 51.13%.

@@            Coverage Diff            @@
##           master      #30     +/-   ##
=========================================
- Coverage   55.54%   48.05%   -7.5%
=========================================
  Files           8        9      +1
  Lines         920     1001     +81
=========================================
- Hits          511      481     -30
- Misses        409      520    +111

Impacted Files
Coverage Δ

src/triblockbanded.jl
53.65% <ø> (-17.08%)
:arrow_down:

src/interfaceimpl.jl
64.51% <0%> (-2.16%)
:arrow_down:

src/BandedBlockBandedMatrix.jl
59.88% <100%> (-10.78%)
:arrow_down:

src/blockskylineqr.jl
46.15% <46.15%> (ø)

src/BlockSkylineMatrix.jl
61.07% <66.66%> (-6.79%)
:arrow_down:

src/AbstractBlockBandedMatrix.jl
57.14% <0%> (-11.43%)
:arrow_down:

src/broadcast.jl
50.96% <0%> (-6.65%)
:arrow_down:

... and 2 more

Continue to review full report at Codecov.

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


Make goog.async.Deferred more resistant to mock clock

This issue was imported from Closure Library's previous home at http://closure-library.googlecode.com
The link to the original issue is:
https://code.google.com/p/closure-library/issues/detail?id=559

Obsolete. Method has been re-written since original issue.


Extra icons displayed in list view for not authorized services

Expected Behavior
Regular status icons for services
Actual Behavior
'Not authorized services' such as Nachos and Grace Hopper Battleship Control Program display extra glyphs after status display
Steps to Reproduce the Problem

Fabric - List View - Group by Capability OR Group by Owner

Specifications

Version: 0.9.1
Platform: Chrome 61
Subsystem:

I think this one is working as intended. The Icons are indicating no metrics or not authorized. Can you confirm this looks right to you @shanberg?

This is as I'd imagined it. It’s open to discussion on usability though, if you think the indicator should be something else, if the icons aren't clear enough, should be somewhere else, etc.

@shanberg
Maybe the indicator should replace the status icons and be red color.  And the icon next to 'Grace Hopper Battleship' looks like Vital icon.

I think I'd prefer to keep it as is. The status icons and the other icons communicate different things, and I don't want us to feel like we need to get a single icon to represent all of these states.


Why I still see [BABEL] error?

Hello. If I've got correctly this plugin change the babel to swc.
But I still have babel error when I run vite as dev server

  VITE v4.0.4  ready in 323 ms

  ➜  Local:   http://localhost:5173/
  ➜  Network: http://<IP_ADDRESS>:5173/
  ➜  press h to show help
[BABEL] Note: The code generator has deoptimised the styling of /home/my/project/src/assets/404-background.svg as it exceeds the max of 500KB.

You are probably using vite-plugin-svgr which uses svgr which uses Babel to transform SVG to React component.
And probably this SVG is too big or somehting. You can try to run sgvo on it (here as a webapp: https://jakearchibald.github.io/svgomg/)
Personally I'm using https://github.com/ArnaudBarre/vite-plugin-fast-react-svg for SVGs but this require very clean SVGs because no transform is applied

You are probably using vite-plugin-svgr which uses svgr which uses Babel to transform SVG to React component.
And probably this SVG is too big or somehting. You can try to run sgvo on it (here as a webapp: https://jakearchibald.github.io/svgomg/)
Personally I'm using https://github.com/ArnaudBarre/vite-plugin-fast-react-svg for SVGs but this require very clean SVGs because no transform is applied

You are absolutely right.
I use this plugin.
Thanks. So the only two possible way are using one those plugins?

If you want SVG as React component, you need a plugin. Otherwise you can import it the same way as png/jpg source (but you loose some capabilities from SVG)
There are maybe other plugin that do the same things but nothing convincing for me when I wrote this plugin last year

Thank you. I guess this issue is closed.

Without an error it's complex. But you can first see why babel is in your project by runining npm why @babel/core of similar


Add region sizes to the MRA

Memory regions currently only strack their start address, we should modify the Memory Model Map to also track their extent calculated from adjacent region boundaries.

This has been implemented, hasn't it? @ailrst @yousifpatti

@l-kent This has been addressed. I will close the issue.


array references

Hello Dask Folks
this isn't really a blocking issue, so feel free to close once there's a thought or two shared; i'll try to make sure to close it once i've got my head into the detail.
I have an example programme which I'd like to understand, i;ve tried to see how the referencing is working and i can't quite see what is going on.
I have also tried to extend the pattern into more complicated scenarios and i can't reproduce the simplified behaviour (a sure sign i'm missing the point)
The question I have is how does the reference sharing in this programme work?  All the 'id' returns are different, but changing one name changes them all, so sharing is occurring
much obliged
mark
import numpy as np
import numpy.ma as ma
import dask.array as da

print('#'*44)

anarray = np.arange(11)

darray = da.from_array(anarray, 11)

marray = ma.masked_array(darray.compute())

print('checking dask.array references:')

print(darray.compute()[5])
print(id(anarray))
print(id(darray.compute()))
print(id(darray.compute()))
print(id(darray.compute()))

darray.compute()[5] = 11
print(darray.compute()[5])
print(id(darray.compute()))
print(anarray[5])

anarray[5] = 13
print(darray.compute()[5])
print(id(darray.compute()))

marray[5] = 17
print(darray.compute()[5])

returns
############################################
checking dask.array references:
5
140433456945632
140433421350512
140433421350592
140433421350672
11
140433421350432
11
13
140433421350592
17

Generally dask.array is not shy about making copies.  In your case you're probably getting different id numbers because of this behavior in numpy
In [1]: import numpy as np

In [2]: x = np.arange(5)

In [3]: id(x)
Out[3]:<PHONE_NUMBER>71808

In [4]: id(x[:])
Out[4]:<PHONE_NUMBER>72368

In [5]: id(x[:])
Out[5]:<PHONE_NUMBER>73008

Also, I recommend not changing the underlying data of a dask.array.  We generally assume that underlying data is unchanging.

Closing.  Feel free to reopen if your question was not answered.

thanks @mrocklin that covered all that i needed for now


8051: Wrong decompilation

Describe the bug
Decompilation result for a simple 8051 code is wrong.
To Reproduce
C source code sample compiled in Keil:
#include <REG51.H>

typedef unsigned char byte;
typedef unsigned short word;

byte Send_SBUF( byte bt )
{
  word wr = 0;

  TI = 0;
  SBUF = bt;

  do
  {
    if( TI )
      break;
  }
  while( wr++ < 0x37CD );

  if( TI )
    return 1;

  return 0;
}

Decompilation result is:
/* WARNING: Removing unreachable block (CODE,0x001d) */

bool Send_SBUF(byte bt)
{
  bool bVar1;
  byte bVar2;

  write_volatile_1(SBUF,bt);
  bVar2 = 0;
  do {
    bVar1 = bVar2 < 0xcd;
    bVar2 = bVar2 + 1;
  } while ((BANK0_R4 < 0x37U - ((bVar1 << 7) >> 7)) << 7 < '\0');

  TI = 0;
  return false;
}

                             *************************************************************
                             *                           FUNCTION
                             *************************************************************
                             bool  __stdcall  Send_SBUF (byte  bt )
             bool              R7:1           <RETURN>
             byte              R7:1           bt
                             Send_SBUF
       CODE:0000 e4              CLR        A
       CODE:0001 fd              MOV        R5 ,A
       CODE:0002 fc              MOV        R4 ,A
       CODE:0003 c2  99           CLR        TI                                               = ??
       CODE:0005 8f  99           MOV        SBUF ,bt                                         = ??
                             LAB_CODE_0007                                   XREF[1]:     CODE:0018 (j)
       CODE:0007 20  99  10       JB         TI ,LAB_CODE_001a                                = ??
       CODE:000a 0d              INC        R5
       CODE:000b ed              MOV        A,R5
       CODE:000c ae  04           MOV        R6 ,BANK0_R4                                     = ??
       CODE:000e 70  01           JNZ        LAB_CODE_0011
       CODE:0010 0c              INC        R4
                             LAB_CODE_0011                                   XREF[1]:     CODE:000e (j)
       CODE:0011 14              DEC        A
       CODE:0012 c3              CLR        CY
       CODE:0013 94  cd           SUBB       A,#0xcd
       CODE:0015 ee              MOV        A,R6
       CODE:0016 94  37           SUBB       A,#0x37
       CODE:0018 40  ed           JC         LAB_CODE_0007
                             LAB_CODE_001a                                   XREF[1]:     CODE:0007 (j)
       CODE:001a 30  99  03       JNB        TI ,LAB_CODE_0020                                = ??
       CODE:001d 7f  01           MOV        bt ,#0x1
       CODE:001f 22              RET
                             LAB_CODE_0020                                   XREF[1]:     CODE:001a (j)
       CODE:0020 7f  00           MOV        bt ,#0x0
       CODE:0022 22              RET

Actually there are two issues in the decompiled code:

SCON.TI bit is not checked in the timeout cycle.
Not only SBUF is volatile, but TI also is.
So, there is no unreachable block in code.

while ((BANK0_R4 < 0x37U - ((bVar1 << 7) >> 7)) << 7 < '\0'); instead of
while( wr++ < 0x37CD ); is awful.

Expected behavior
As original C source code.
Attachments
Raw binary zipped file
Environment:

OS: Windows 7 64-bit
Java Version: OpenJDK 11.0.5
Ghidra Version: 9.2-DEV (latest master build)

I was having a vaguely similar issue (many unreachable code blocks). Workaround was to edit the memory areas to set "SFR-BITS" to volatile . This should be fixed at the .pspec level if possible ?
This probably applies to #3061 as well.


Add class doc to FlutterPluginAppLifeCycleDelegate

https://github.com/flutter/engine/blob/master/shell/platform/darwin/ios/framework/Headers/FlutterPluginAppLifeCycleDelegate.h
The documentation already exists, added here: https://github.com/flutter/engine/pull/6447


fix: Sort set before committing files to git to avoid merge conflicts

Description

As Set is an unordered datatype, this is causing unnecessary changes to the file system during the commit operation even if the user has not altered anything since the last commit.

Fixes #10932
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

JUnit TC
Manual test

Checklist:

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

Overall looks good to me other than the small nits. On a side thought, we are using the TreeSet for actions in a sorted manner. But being the TreeSet implementation asynchronous, wouldn't it cause any issues when there are multiple users trying to do git commit and status at the same time?

TreeSet does not have reactive asynchronous implementation but it's not thread-safe. But I suppose we are working with the single thread here

/ok-to-test sha=a6432f8

Overall looks good to me other than the small nits. On a side thought, we are using the TreeSet for actions in a sorted manner. But being the TreeSet implementation asynchronous, wouldn't it cause any issues when there are multiple users trying to do git commit and status at the same time ?

TreeSet does not have reactive asynchronous implementation but it’s not thread-safe. But I suppose we are working with the single thread here, so it’s safe to assume this will work as expected. The other point you mentioned if multiple users make a call to commit, getStatus etc. JGIT handles the file locks but this can be improvised by taking control of file locks, nonetheless, this implementation doesn’t concern the above-mentioned issue.


Probably slip up content in template guide

There is this:

In the template guide page
I don't think that means anything to users of the page.

Yeah, not sure what happened there. Closing as already reported and PR standing.


Addition of PostgreSQL indexes

Explain the changes

Added indexes and re-wrote declarations

Issues: Fixed #xxx / Gap #xxx

Testing Instructions:

@jeniawhite @guymguym @nimrod-becker
I think we should move the tables and indexes creation out of the stores constructor. this process is only necessary once for each new version of noobaa core, so I feel it should be part of upgrade\system creation. what do you think?


🌐 Add Korean translation for Index

This is the first translation for Korean Language page(see #2017), index.md.
I hope lots of developers who are Korean participate in this community.😁
And.. please review the translation, thanks.

This is awesome! :tada: :rocket:
Thanks a lot @hard-coders for all the work :nerd_face: :cake:
And thanks @jrim and @swnim for the reviews :bow: :cookie:


Lint Python notebooks with ruff.

The new ruff version fixed the blocking bugs, and I was able to fairly easily us to a passing state: ruff fixed some issues on its own, I fixed a handful by hand, and I added a list of narrowly-targeted exclusions for files that are currently failing ruff rules that we probably should look into eventually.
I went pretty lenient on the docs / cookbooks rules, allowing dead code and such things. Perhaps in the future we may want to tighten the rules further, but this is already a good set of checks that found real issues and will prevent them going forward.

Somehow, linting with black's jupyter integration doesn't run into all these errors. It only finds one formatting issue, which it's able to resolve.

I think @rlancemartin is going to try to merge some last notebooks that should be reformatted as part of this too, so the formatting doesn't introduce annoying merge conflicts in the other direction :)

Not a problem. I see at least one such got merged and I'll fix the conflict. Are there more @rlancemartin ? No worries if so, just give me a ping when you're all set.

Merge it! Can sort out merge conflicts for the remaining ones (there's quite a few) by just formatting the changed versions for now.

Merge it! Can sort out merge conflicts for the remaining ones (there's quite a few) by just formatting the changed versions for now.

Happy to help with that if you'd like a hand, I've got the process down from practicing it over the last few days :)


Finallyyy...

i been really really serching beautiful library.. and you got you bro..
please do not support this baby..
thanks :)

The idea was to make simple as possible and design to work with most used hardware available.
Thanks, I really appreciate that.

@ricaun i just made the old framework for me like gateway id 222 and node id is 200
if (id 222) gateway or if id (200) node
is it possible on this library?
my second question is,
i just tested radio freq. on other arduino library
when i set 915e6 i get 600~ mhz, when i made it 1020, i get around 686 - 700 mhz,..
is your repo fix that problem bro?
thanks

when i set 915e6 i get 600~ mhz, when i made it 1020, i get around 686 - 700 mhz,..

Technically I'm using the arduino-LoRa library to make all the LoRa configuration stuff. So I never have a problem with frequency as you explain.

@ricaun i just made the old framework for me like gateway id 222 and node id is 200
if (id 222) gateway or if id (200) node

I didn't understand want you mean, but each node have a UniqueID and the gateway always listens for data, if it's found a valid protocol the gateway can read packege and send back a data to the node id. Similar as LoRaWan Class A.


GPUs on multiple nodes

I have 10 GPUs on 5 nodes (2 on each), how can I use them all together?

I don't know.


Logo grid: document and improve component

We developed a new component called 'logo grid' that is not properly documented. The component now exists in storybook  but not in Figma. The developers followed the style used on this page: https://www.figma.com/file/TnVsWy6VobbHzZdbHdx3Rb/ILO-Page-Templates?type=design&node-id=3796-41633&mode=design&t=GCP28TXoAdcuBckG-0

We need to create the component in Figma and document it correctly. Besides, we need to do the following changes:

Increase the size of the logos' images. At the moment, the logos can be a maximum of 150px wide and 50px high. The component will render them as high and as wide as possible until they hit one of those two limits.
Design for scenarios when CMS editors use only one or two logos

FYI @justintemps Assigning this to Beatriz but copying you so know it's coming up.

@beatrizmartinmartins Could we aim for 22 April to have this ready?

@inesdgomes you can check the designs on the template here. The container is flexed, this means that depending on the screen size, it will adquire the full-width of it's own container. Also you can check the component here

Thanks @beatrizmartinmartins I think we still need to increase the maximum height in the 3 tile grid. The design is assuming all logos have a similar size/layout. Here's an example we can look at together tomorrow (scroll to the bottom of the page): https://www-preview.ilo.org/topics/safety-and-health-work/programmes-projects/safety-health-all

Could you update the link? I can't open it @inesdgomes

@beatrizmartinmartins Yes, sorry. Here you go: https://www.ilo.org/topics/safety-and-health-work/programmes-projects/safety-health-all

To move to development on 16 May

Template with real logos: https://www.figma.com/design/TnVsWy6VobbHzZdbHdx3Rb/ILO-Page-Templates?m=auto&node-id=4887%3A1800700&t=sRRyckchF3r0xlVV-1 @inesdgomes

Hi @beatrizmartinmartins Thanks. A few requests:

In the template with real logos, could you please tweak the logo images so they don't have any empty space around them? They should all fit perfectly in the available container so they're all the same height or same width.
Could you clean up the social media component in the library so developers can refer to it when implementing the changes?
Please also detail here in a comment what changed design-wise (how was it before / how is it now) so it's easier for developers to apply the changes.

Thank you!

Hey @inesdgomes !

Just tweak them so they fit perfectly on their respective variant. It will look good on the templates, but keep in mind that is not realistic unless someone changes them.
What do you mean by that? Don't have the logos and just keep the default images like these ones?

Will do in a different comment so it's easier to follow up.

Hey team!
So for the Logo Grid component, previously we had a simple variant with a max-width container surrounded by a 24px spacing on the borders of the image container. This worked for logos that where displayed horizontally (like Nike for example) but it constrained the logos that are higher than wider. Since we couldn't figure out how to make sure that all logos fit properly on the same container, we opted to create another variant.
Now we have a "Horizontal Image" and "Vertical Image" option to choose from. Both are different in terms of min-width and the spacing around them, making the logos easier to adjust to the container. Of course it also depends on the logo provided by the user and the black space that can have around it, but we tested a few logos that are currently on this page and they all work correctly when applied it's own variant.
Please let me know if everything is explained and if you need more context. You can check the updated component on this link
@justintemps

Thanks @beatrizmartinmartins Just to avoid any confusion/inconsistency, please update the component description before we move it to development.

@inesdgomes It's documented on the real component

Apologies, my bad, not yet. We still need to address one thing: what happens when only one or two logos are shown? @beatrizmartinmartins Could you please test these scenarios? We've seen situations where CMS users only uploaded one or two logos and it doesn't look great. Should the grid adjust somehow in those cases? For example, if there's only one logo, should it be bigger or centered?
Here's an example:

Maybe this won't be an issue anymore after the changes you made, but let me know what you think.

@beatrizmartinmartins Just a kind reminder on this one too, as we need to move it to development. Thanks

I think it would be better if the images are full width within the container. I don't think the logos need to be bigger because they are not that important within the context and if they get too big they can occupy a lot of space within the screen but they will be centered on it's own container.. You can check all possible scenarios in here Do you think it can make sense @inesdgomes ? Or should we find another approach for those instances? I feel like when we have one or two, they should be full width, but when we have 4, and so on (intead of multiples of 3, they should have a fixed width).

Hi @beatrizmartinmartins Yes, it does make sense to me. Just not super convinced about the single logo taking full width, I think it looks strange. Could we rather show the single logo in a grid of 2?

If 1 or 2 logos: show in grid of 2
If 3 or more: show in grid of 3

@inesdgomes fixed

@justintemps This can move to development. You can check the updated component here: (https://www.figma.com/design/RRxsfLNxjoypKrfJg1EwJf/ILO-Components-Library?m=auto&node-id=3311%3A5768&t=2KxVXrG6xwSm2oel-1)
A few notes:

Previously we had a simple variant with a max-width container surrounded by a 24px spacing on the borders of the image container. This worked for logos that where displayed horizontally (like Nike for example) but it constrained the logos that are higher than wider. Since we couldn't figure out how to make sure that all logos fit properly on the same container, we opted to create another variant.
Now we have a "Horizontal Image" and "Vertical Image" option to choose from. Both are different in terms of min-width and the spacing around them, making the logos easier to adjust to the container. Of course it also depends on the logo provided by the user and the black space that can have around it, but we tested a few logos that are currently on this page and they all work correctly when applied its own variant.
The grid has also been changed. The logos can be shown in a grid of 2 or 3 items. If the group has 1 or 2 items, the logos show in a grid of 2; if the group has more than 3 items, the logos always show in a grid of 3.

Please let us know if it's all clear.

I'm going to close this as a design ticket and create a new one for development.


[UX, DARK MODE] enable-force-dark - where?! (flag name changed)

long story short - the lack of feature presentation, sry for mis-issue
long
both, the changelog and the commit, has no info about changing the names of the switch and the flag
now flag name is dark-mode-settings
and switch ... well, at my surprise no one of v100+ have it for web-content, and --force-dark-mode actually enable dark mode for web-ui. So there no simple on/off switch, however processes get this one
--dark-mode-settings=InversionAlgorithm=3,ImagePolicy=1,ForegroundBrightnessThreshold=150,BackgroundBrightnessThreshold=205

but seems like it can be shortened without visible consequences to --dark-mode-settings=InversionAlgorithm=3
the one "feature" of the --dark-mode-settings switch is that it keeping flag in orignal state (no sync with command line switches, even in full form), but assume it is an internal argument not supposed to be used by :banana::bomb:
how it's used can found here, but i keep some info under spoiler
short description
InversionAlgorithm - color inversion mode

1 - RGB
2 - HSL
3 - CIELAB

ImagePolicy - how images inverted

0 - officially not used, but i guess it disable images inversion completely
1 - simple/dumb (but on first glance same as 2)
2 - selective

*BrightnessThreshold-s is for fine tuning, you may try to play with it(i assume in range 0 .. 255) or keep default values(150 for Foreground and 205 for Background)
ImageClassifierPolicy - toggle of "color and transparency basis" for image selection (used in last mode from dropdown)

0 - not used, i guess, equals to disabled
1 - enabled

former selective everything ...  modes actually was and still CIELAB-based
RGB shitty as always, HSL and color-based(that is also HSL-based) looks too blackish for me

anyway it should be noted in the changelog and in the description of the flag, e.g.
"... Supermium exclusive, replacement of #enable-force-dark"
probably the new name of the switch also should be mentioned in the description
Also --force-dark-mode did not work for now, but still exists and mentioned(instead of enable-force-dark) in README.md (main repo page)
i'm keep issue opened, but it can be closed as needed
original postwhile this is not really critical bug, it is critical for me, i never relying on end-site features it this question
Screenshots

Desktop (please complete the following information):

OS: Windows 7 SP1 x64
SM: 126.0.6478.254 R4 x64 (as is, no patches)

WHAT DID YOU NEED FROM ME TO FIX THIS ???!!! and what should i do to provide it?(switches, log locations, etc. )
for now i'm rolling back to R3

p.s. here a lot of misunderstood and confusions from my side, cuz i did not track related info and "stuck in v88" or older. plus i made a common mistake - updating just before go off with all consequences

^^^^^^^^

I would prefer using Dark Reader anyway since the flag breaks some sites and it's not possible to blacklist/whitelist a site (hmm... what if I open a feature request about it 😅 )

try :smile: in rare cases it will be useful
mostly i like native CIELAB gray theme, and for such cases have StyleBot and :root { color-scheme: dark; }
and about title of the flag - i usually ignore it when know flag name/id, so did not expect such a trap + auto for me equals to something i can't control. i only notice this new flag with old title after digging in commit :rofl:


Python 2.7 occasionally raises exceptions in integrated terminal when using logger methods

Using the logger method from the logging module with an empty string as message (e.g. logger.error('')) in the integrated terminal raises the following exception occasionally (with no apparent pattern):
Traceback (most recent call last):
  File "C:\Python27\Lib\logging\__init__.py", line 880, in emit
    stream.write(fs % msg)
IOError: [Errno 0] Error

This happens when using Python 2.7.10. Also tested on Python 3.6.5 with no problems. Additionally, the problem does not occur using Windows Powershell.
The problem seems to have debuted after recent Windows upgrade.

VSCode Version: Version: 1.25.1
Commit: 1dfc5e557209371715f655691b1235b6b26a06be
Date: 2018-07-11T15:43:53.668Z
Electron: 1.7.12
Chrome: 58.0.3029.110
Node.js: 7.9.0
V8: 5.8.283.38
Architecture: x64

OS Version: Windows 10 Enterprise Version 10.0.16299 Build 16299

Extensions: Python

Steps to Reproduce (Code):
import logging

logger = logging.getLogger(__name__)
logger.addHandler(logging.StreamHandler())

for i in range(0, 100):
    logger.error('')

Replacing logger.error('')  with print('') occasionally produces the following error message instead:
close failed in file object destructor:
sys.excepthook is missing
lost sys.stderr

Does this issue occur when all extensions are disabled?: Yes
EDIT: Just got the exception using a non-empty string as message (e.g. logger.error('message'))

This was a bug in Windows 10, you'll need to update.

See https://github.com/Microsoft/vscode/issues/36630


Modal component warnings

Current Behavior
I'm trying to use the Modal component but I have some warnings in my DevTools.
Steps To Reproduce

Create a modal:

modalStore.trigger({
  type: 'component',
  component: { ref: UploaderFileModal, props: { file } },
  modalClasses: '!p-0'
})

UploaderFileModal.svelte:

    <Button class="variant-filled-surface" aria-label="Previous file">
      <CaretLeft size={24} slot="icon" />
    </Button>

{file.name} ({Math.floor(file.size / 1024)} KB)

    <Button class="variant-filled-surface" aria-label="Next file">
      <CaretRight size={24} slot="icon" />
    </Button>

See the error in DevTools :

<UploaderFileModal> received an unexpected slot "default".
<UploaderFileModal> was created with unknown prop 'parent'

Anything else?
No response

Adding a quick note here to remind myself - the fix will be to add a conditional around the default slot in Modals.svelte. Currently that's always present, hence the warning.

Found a Svelte core contributor discussing the issue here:
https://github.com/sveltejs/kit/issues/981#issuecomment-822054320

It is just a dev-time warning, none of these show up in the production build.

Given this, there's nothing to fix or address. This can be safely ignored as the warning will not be present in production.


Problem with moving webhook url to .env file

I have
set :slackistrano_webhook, ``ENV["SLACK_WEBHOOK"]
in my deploy.rb file and
SLACK_WEBHOOK=http://<webhook_url>
in my .env file but I receive the following error
Error: #<SocketError: getaddrinfo: Name or service not known>
Should this work?

@strtwtsn Sorry, I forgot all about this. It should if you are loading that .env file yourself, but Slackistrno won't load an env file on it's own.

@strtwtsn you can do something like this in your config/deploy.rb
set :env_file, ".env.#{fetch(:stage)}"

set :slackistrano, -> {
  { channel: ENV['DEPLOY_NOTIFICATION_CHANNEL'], webhook: ENV['DEPLOY_NOTIFICATION_WEBHOOK'] }
}

namespace :env do
  desc 'Load the stage env file'
  task :load do
    Dotenv.load(fetch(:env_file))
  end
end

invoke 'env:load'


re-add support for multiple prompt generations

in the upgrade to 1.5, the num_outputs multiplier was lost

note - I haven't tested this yet - I'm still working on my local build

This is where the change comes from:
https://github.com/replicate/cog-stable-diffusion/commit/89ab0e89b9f4cf9ad8e1fed02d512cc659f348a2#diff-73c1982d8a085dc10fda2ac7b6f202ae3ff9530ee6a15991c5339051eb10a49aL102

Nice catch! cc @chenxwh

Thanks @anotherjesse!


PYTHONPATH didn't take into account by the plugin

I have in my PYTHONPATH a folder containing several lib that I use in different python project.
When I run my script without Reloadium every thing is fine.
But with Reloadium I have :
Traceback (most recent call last):
File "C:\Users\stamza\PyCharmProjectLocal\Support\Ioda-MedEco\Ioda_test.py", line 1, in
import ioda
ModuleNotFoundError: No module named 'ioda'

Hi @tamzaselma
Could you add this code before you import ioda module and post the output when running with Reloadium?
import sys
print(sys.path)

Also how do you set your PYTHONPATH env variable? Is it system wise or set in PyCharm?

Thanks for you quick response @dkrystki,
Actually it's set in my system variable.
os.getenv('PYTHONPATH') --> C:\Users\stamza.reloadium\package\3.8;C:\Users\stamza\PyCharmProjectLocal\Support
print(sys.path) ---> ['C:\Users\stamza\PyCharmProjectLocal\Support\Ioda-MedEco', 'C:\Users\stamza\PyCharmProjectLocal\Support\Ioda-MedEco', 'C:\Users\stamza\.reloadium\package\3.8', 'C:\Users\stamza\PyCharmProjectLocal\Support', 'C:\Program Files\Python38\python38.zip', 'C:\Program Files\Python38\DLLs', 'C:\Program Files\Python38\lib', 'C:\Program Files\Python38', 'C:\Users\stamza\Venvs\Support', 'C:\Users\stamza\Venvs\Support\lib\site-packages', 'C:\Users\stamza\Venvs\Support\lib\site-packages\win32', 'C:\Users\stamza\Venvs\Support\lib\site-packages\win32\lib', 'C:\Users\stamza\Venvs\Support\lib\site-packages\Pythonwin']
Without Reloadium :
os.getenv('PYTHONPATH') --> C:\Users\stamza\PyCharmProjectLocal\Support;C:\Program Files\Python38;N:\MEDISSYS.32;N:\BBQIODA.32
print(sys.path) ---> ['C:\Users\stamza\PyCharmProjectLocal\Support\Ioda-MedEco', 'C:\Users\stamza\PyCharmProjectLocal\Support', 'C:\Program Files\Python38', 'N:\MEDISSYS.32', 'N:\BBQIODA.32', 'C:\Program Files\Python38\python38.zip', 'C:\Program Files\Python38\DLLs', 'C:\Program Files\Python38\lib', 'C:\Users\stamza\Venvs\Support', 'C:\Users\stamza\Venvs\Support\lib\site-packages', 'C:\Users\stamza\Venvs\Support\lib\site-packages\win32', 'C:\Users\stamza\Venvs\Support\lib\site-packages\win32\lib', 'C:\Users\stamza\Venvs\Support\lib\site-packages\Pythonwin']

I try that  :
os.environ['PYTHONPATH'] = os.getenv('PYTHONPATH') + ';Files\Python38;N:\MEDISSYS.32;N:\BBQIODA.32'
but have the same error :(

Fixed in reloadium 0.9.5 and PyCharm plugin 0.9.0


why the sql langchain's custom prompt is not working?

Issue you'd like to raise.
I tried to work on SQL cutsom prompt, but it didn't work and is still giving the wrong sql queries . Here is the code : def process_user_input(user_input):
create_db()
input_db = SQLDatabase.from_uri('sqlite:///sample_db_2.sqlite')
llm_1 = OpenAI(temperature=0)
db_agent = SQLDatabaseChain.from_llm(llm = llm_1,
db = input_db,
verbose=True,)
chain = create_sql_query_chain(ChatOpenAI(temperature=0), input_db)
response = chain.invoke({"question": user_input})
Suggestion:
No response

You can find related solutions on #12006. You can check how to pass the custom prompt in the agent.


playsound command documentary

Hey,
could somebody make/update  documentation of the - playsound command on the website ?
Thanks !

That'd be nice. However as it's part of the internal TODO list, there's no need to maintain an issue for it.


Added WonderCMS to your list (I'm the main contributor)

[x] verified that the CMS I'm adding is still maintained.
[x] read CONTRIBUTING.md.
[x] did not generate README.md.

Excellent! Thank you.


[FLINK-26583][runtime] Make the Flink cluster fail if the job ID is listed as globally-terminated and cleaned

What is the purpose of the change
The Application and Job Mode relies on the JobResultStore for recovery. Nothing will be recovered if the job is in dirty state but globally terminated. That will just trigger the cleanup. But we didn't cover the case where there's a clean entry in the JobResultStore. This lead to the Application just terminating successfully without any indication to the user that the job didn't run.
This PR changes the behavior in a way that a DuplicateJobSubmissionException with a meaningful error message is thrown instead.
Brief change log

ApplicationDispatcherBootstrap constructor gets a new parameter that enables us to pass in the JobResultStore check as a callback
JobDispatcherLeaderProcessFactoryFactory will fail during the creation of the JobDispatcherLeaderProcessFactory
This check can be used to issue an error before submitting the application again.

Verifying this change

ApplicationDispatcherBootstrapTest and ApplicationDispatcherBootstrapITCase were extended to cover this issue
JobDispatcherLeaderProcessFactoryFactoryTest to cover the job mode

Does this pull request potentially affect one of the following parts:

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

Documentation

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

The rational behind this change is to make the user aware of the fact that there's still a JobResultEntry laying around. Flink itself is behaving as expected. Alternatively, we could add a warning. I was just afraid that the user might not notice it and, therefore, went for the exception approach, instead.

The rational behind this change is to make the user aware of the fact that there's still a JobResultEntry laying around. Flink itself is behaving as expected. Alternatively, we could add a warning. I was just afraid that the user might not notice it and, therefore, went for the exception approach, instead.

Will in practice, if say you use Kubernetes, this not result in the job being re-submitted again and again because the cluster fails until some failure rate policy is triggered? I'm not sure if this is the better alternative.
In particular because this can happen without the user doing something wrong; say the JM crashes after having cleaned the job result. In that situation we very much want Flink to just shut down.

We agreed on going for the log-warning approach because we don't want to cause a failover of the Flink cluster in such a case where the job is already marked as clean. Flink behaves as expected. The user should only be informed about it through logs.
Additionally, we can improve the change by moving the check into Dispatcher.submitJob because that's called by all the different execution modes. I reverted the initial commits and pushed the changes to the remote branch. PTAL

I added a test case to make the log message reproducible in a unit test.

Unstable e2e profiles are independent of the changes of this PR.


Feature Request: Electrs Listing in Web GUI for RaspiBlitz only when installed, or change stated info when clicking Electrs

Feature Request: Electrs Listing in Web GUI for RaspiBlitz
Issue Description:
In the current version 1.11 of RaspiBlitz, the Electrs application is listed in the Web GUI on the left located Overview bar. When users click on it, a message appears stating that Electrs needs to synchronize before it can be used. This causes confusion among many new users who assume that Electrs is already installed and they just need to wait for it to synchronize. Consequently, they often wait for a long time and eventually seek help in the RaspiBlitz Telegram chat, asking why Electrs has not finished synchronizing.
Problem:
Electrs is not installed by default. It must first be installed through the CLI menu (Menu -> Apps -> Electrum Rust). The current message in the Web GUI does not inform users about this necessity, leading to misunderstandings and unnecessary waiting times.
Proposed Solution:

Modify the Electrs Entry in the Web GUI:
Update the message displayed when users click on Electrs to clearly state that Electrs must be installed first via the CLI menu. This can help users understand that additional steps are required before synchronization can occur.
Suggested Message:
"Electrs is not installed by default. Please install Electrs via the CLI menu: Menu -> Apps -> Electrum Rust."

Conditional Display of Electrs in the Web GUI:
Alternatively, adjust the Web GUI so that Electrs is only listed under available apps if it has been installed. This would prevent users from encountering the synchronization message before they have even installed the application.

Benefits:

Reduces user confusion and frustration by providing clear instructions.
Decreases the number of repetitive queries in the RaspiBlitz Telegram chat.
Enhances overall user experience by ensuring the Web GUI accurately reflects the system's state.

Thank you for considering this feature request. Implementing these changes will significantly improve the user experience for new RaspiBlitz users.

Same issue here, luckily had a look before opening a new issue and trying to figure out if I can fix it myself...

Checked the code for the SideDrawer:
https://github.com/raspiblitz/raspiblitz-web/blob/master/src/layouts/SideDrawer.tsx#L49
It looks if the app is installed, so it shouldn't appear in the sidebar. Looks like the API sends a wrong message.
@rootzoll

Yes there are two other issues open on that - @fusion44 we would need you help here:

https://github.com/raspiblitz/raspiblitz/issues/4649
https://github.com/fusion44/blitz_api/issues/259

We can fix this easily. See my comment in the other issue. I'm closing this in favor for the other one where the discussion is happening.
closing in favor for #4649


UIShell v11

While upgrading to v11 styles, we should also resolve some open issues:

#892
#1538
#309
#516
#1463
#1383
#466
#461

Hows this coming along? We're kinda looking at using this, but I kinda need to tell the boss how long we can expect to hold off (Ie are we talking weeks or months here?) as this theme issue is the big one.


JUnitReporter xml-file should contain console printout (puts) for every testcase.

JUnitReporter xml-file should contain console printout (puts) for every testcase. Then this gem will be very usable in Jenkins ! ( ci_reporter do this already )

It would be very helpful if you'd provide example of suggested output.

I think that just adding My prints... after testcases will be fine.
There's some sort of spec in http://llg.cubic.org/docs/junit/

<?xml version="1.0" encoding="UTF-8"?> <testsuite name="TC_test_junit_xml" skipped="0" failures="0" errors="0" tests="2" assertions="1" time="0.00012403699929564027">   <testcase name="test_01" classname="TC_test_junit_xml" assertions="0" time="9.968599988496862e-05"> 	<system-out> 	Some prints... 	</system-out>   </testcase>   <testcase name="test_02" classname="TC_test_junit_xml" assertions="1" time="2.4350999410671648e-05"> 	<system-out> 	Some other prints... 	</system-out>     </testcase> </testsuite>

Thank you for clarification.  Looks like a good idea.

Any timetable about this ? Will next version of reporter include this ?

no estimations, but PR is welcome ;)

Hi @kimiffx! If this is still an important feature for you, please open a PR and we'll give it a look 👍

Hi @kimiffx! If this is still an important feature for you, please open a PR and we'll give it a look 👍


Vectorised printing in general

Some commands like , need to not lazily evaluate their arguments/discard the value when vectorised
(created by lyxal here)

Actually this is about adding a does_print or contains_comma attribute to lambdas and stuff.

Because the problem is that you can't just evaluate the whole list because it could potentially be infinite. But you can't just lazily evaluate it because it'll print from the generator function and also print the returned item.

I'll explain more after I sleep.

How about we add a new function foreach that goes through every element of an iterable and applies the given function to it without evaluating everything at once. Maybe something as simple as
def foreach(lhs, rhs, ctx):
    lhs = iterable(lhs, ctx=ctx)
    for elem in lhs: safe_apply(rhs, lhs, ctx=ctx)

When you call "v," on anything, it pushes a LL onto the stack instead of printing everything on a newline. And when you go to print it using the iterative printing method, things end up getting double printed, because evaluating the item prints it and returns the item, meaning it also gets printed.

That's the real problem.

I don't know what kind of shenanigans y'all did, but it's not just LazyLists anymore: Try it Online!

I don't know what kind of shenanigans y'all did, but it's not just LazyLists anymore: Try it Online!

Mapping a function returns a lazylist no matter what the original type is.

After consideration and asking in the nineteenth byte, I've decided that this isn't actually a bug, and hence the issue will be closed until further notice.


JSDom no longer sets window.name

as of version 12 jsdom no longer sets window.name and rll fails jest tests
(https://github.com/jsdom/jsdom/blob/6d7ccf2517259fe97b082b363134ea9612800747/Changelog.md#1200)

unless you plan to support IE11 maybe just check for
globalThis === globalThis.window instead of window.name


POSSE following/friending someone new

Minority edge-case as I don't think anyone else is doing follow posts yet, but whenever I follow/subscribe to someone (either website or silo profile) I post a h-entry with u-follow-of. I then go manually hit the follow button on the appropriate silo, but it'd be cool if bridgy publish could take care of that for me (specifically for twitter).
I'll look into implementing this myself when I have time, just thought I'd note it here in case anyone else is interested.

wow, definitely an interesting idea! i'll even broaden this to other silos too. as usual, first task is to catalog each silo's API support for this.
btw @rhiaro, bridgy already noticed your follow posts (and needs improvement) in #511. :P

I have just run into this too, after having verified that the old hack for following via the Twitter posting API of just posting "f username" or "follow username" no longer works (i.e. using the "1.1/statuses/update" endpoint)

Fwiw https://fed.brid.gy/ supports this; it translates mf2 u-follow-of posts into AP follows, and vice versa.


Preference to specify style

A user on the Zotero forums pointed out that if the default quick copy style has no bibliography, no preview is generated. It would be nice if there were a preference to specify a style to override the default style for such cases.

Thanks for letting me know! I'll have to research it.

Looking at the Zutilo plugin's Quick Copy code might be a good start.

Added in new release

Awesome!


Back to development

Update the changelog and version constant to go back to development.
Along the way, migrate to the KAC format.

@twilly your wish is my command.


Interested

Just interested in your project and wondering if i can get your email and ask you a couple questions whenever you have free time :)

Hi.
You can email me at<EMAIL_ADDRESS>


core.exceptions.ActorMismatchError

Oops. What happened there?
inbox-runner           | Traceback (most recent call last):
inbox-runner           |   File "/usr/local/lib/python3.11/site-packages/asgiref/sync.py", line 486, in thread_handler
inbox-runner           |     raise exc_info[1]
inbox-runner           |   File "/takahe/stator/models.py", line 172, in atransition_attempt
inbox-runner           |     next_state = await current_state.handler(self)  # type: ignore
inbox-runner           |                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
inbox-runner           |   File "/takahe/users/models/inbox_message.py", line 99, in handle_received
inbox-runner           |     await sync_to_async(Identity.handle_delete_ap)(instance.message)
inbox-runner           |   File "/usr/local/lib/python3.11/site-packages/asgiref/sync.py", line 448, in __call__
inbox-runner           |     ret = await asyncio.wait_for(future, timeout=None)
inbox-runner           |           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
inbox-runner           |   File "/usr/local/lib/python3.11/asyncio/tasks.py", line 442, in wait_for
inbox-runner           |     return await fut
inbox-runner           |            ^^^^^^^^^
inbox-runner           |   File "/usr/local/lib/python3.11/site-packages/asgiref/current_thread_executor.py", line 22, in run
inbox-runner           |     result = self.fn(*self.args, **self.kwargs)
inbox-runner           |              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
inbox-runner           |   File "/usr/local/lib/python3.11/site-packages/asgiref/sync.py", line 490, in thread_handler
inbox-runner           |     return func(*args, **kwargs)
inbox-runner           |            ^^^^^^^^^^^^^^^^^^^^^
inbox-runner           |   File "/takahe/users/models/identity.py", line 579, in handle_delete_ap
inbox-runner           |     raise ActorMismatchError(
inbox-runner           | core.exceptions.ActorMismatchError: Actor https://social.alster.space/users/alsternerd trying to delete identity https://social.alster.space/objects/8fe6fea4-90ab-4691-9312-06f5645af976

We fire that when a message comes in that's apparently incorrect (like trying to delete something that's not yours). Any chance you can dig up the inbox message for this?

He tried to delete this post: https://tavern.drow.be/<EMAIL_ADDRESS>

Interesting, so the thing I wonder about then is why it thinks they're trying to delete an identity. Can I see the full JSON?

Should be this one:
  "cc": [],
  "id": "https://social.alster.space/activities/315bfd08-da8b-4b95-8571-061c70ac3c50",
  "to": [
    "https://chaos.social/users/macsnider",
    "https://social.alster.space/users/alsternerd/followers",
    "as:Public"
  ],
  "bcc": [],
  "bto": [],
  "type": "Delete",
  "actor": "https://social.alster.space/users/alsternerd",
  "object": "https://social.alster.space/objects/8fe6fea4-90ab-4691-9312-06f5645af976",
  "@context": [
    "https://www.w3.org/ns/activitystreams",
    {
      "toot": "http://joinmastodon.org/ns#",
      "Emoji": "toot:Emoji",
      "Public": "as:Public",
      "Hashtag": "as:Hashtag",
      "blurhash": "toot:blurhash",
      "sensitive": "as:sensitive",
      "focalPoint": {
        "@id": "toot:focalPoint",
        "@container": "@list"
      },
      "votersCount": "toot:votersCount",
      "manuallyApprovesFollowers": "as:manuallyApprovesFollowers"
    },
    "https://w3id.org/security/v1"
  ]
}

Aha, yes, I see - it's a Delete without an embedded object which is why we can't handle it!


[SMALLFIX] customize the close timeout in netty writer

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/Alluxio-Pull-Request-Builder/14833/
Test PASSed.

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/Alluxio-Pull-Request-Builder/14839/
Test PASSed.

@peisun1115 Could you document this new configuration value?


Where can I find the Volto changelog?

I am looking for the Volto changelog. Neither the README:
https://github.com/plone/volto/blob/main/README.md
nor the NPM page:
https://www.npmjs.com/package/@plone/volto
contain any hint where to find the changelog. Going through the individual releases on GH/NPM would be possible but incredibily cumbersome.
I am looking for something like:
https://github.com/plone/Products.CMFPlone/blob/master/CHANGES.md
where I can grep for certain changes and get an overview over the latest development.

Volto is now a monorepo, so it moved into the packages directory.
https://github.com/plone/volto/blob/main/packages/volto/CHANGELOG.md
Should the README have links to all CHANGELOG.mds for every package?
There is also documentation at https://6.docs.plone.org/volto/release-notes/index.html, but that is only for the volto package.


Search in ignoring some possible matches

An example is Duration class:

I can find JavaFX packages in general, but some classes conflict, I believe search should show all variations of the class.

When there are many classes with the same name, you'll have to make a more detailed query to find the correct class. Currently only way to do that is to write the whole package name from the beginning (e.g. "javafx.util.D") but it once fuzzy search (issue #11) has been implemented, you can write just parts of the class and package name to find the correct class (e.g. "Dura.fx").
I made autocomplete suggesstions a little bit smarter in v0.10.1 release. Hopefully most of the time the autocomplete will suggest the correct classes even when there are multiple classes with the same name in classpath.

Duplicate of #11


Post Deployment Fix for Nginx IC traffic policy

PR Summary

Fixes syntax of value ingress_externalTrafficPolicy for nginx IC.
closes #415
PR Checklist

[x] PR has a meaningful title
[x] Summarized changes
[x] This PR is ready to merge and is not Work in Progress
[x] Link to a filed issue

Thank you very much @naioja  for logging the issue & this contribution! We use these variables for nginx but also contour, so I will test to ensure the contour option still works today, then approve.  Thank you for your 1st contribution.

Like you previously mentioned I see that at line https://github.com/Azure/AKS-Construction/blob/a8200a037dd2956d897988821938dfc031c06249/postdeploy/scripts/postdeploy.sh#L296 we are using the same variable for Contur.
Looking at the Contur official documentation the same spelling is used with capital L and C for Local and Cluster : https://tanzu.vmware.com/developer/guides/service-routing-contour-refarch/#avoiding-unnecessary-network-hops


edited maven pom and some patches

Make some checks in maven pom for can compile it. And some project repairs.

Hi. can you sign the contributors.txt file too?

Done :)

The compilation problems were already fixed and https://github.com/antlr/jetbrains-plugin-sample/pull/10, and the changes to the Maven build will become obsolete once we switch the project to a Gradle build (that should happen very soon).
Thank you in any case @brizjin!


Add proper loader before rendering

There is some flickering before the final rendering...

Solution is on https://github.com/openintents/app-center/issues/31

The flickering went away with the addition of the material ui plugin.


Rule Idea: no-break, no-if, no-loops

https://hackernoon.com/rethinking-javascript-eliminate-the-switch-statement-for-better-code-5c81c044716d
"no-break": [true, {"ignore-switch": "true"}], // maybe continue is included here or a separate rule?
"no-if": "true", // or maybe "prefer-ternary"?
"no-loops": [
     true,
     "while",
     "for",
      "do"
]

This seems like some nice additions to the functional style rules :-).

I'm think about how the options for no-loops would work.
"no-loops": [true]

That would disable all loops, including while, for, do.
"no-loops": [true, "while"]

That would only disable while leaving for and do available?
I'm thinking maybe it is more logical to have ignore options, like ignore-for, ignore-while etc.
Also not sure if do is needed as an option? If the while keyword is banned then do would not be useful anyway? And in that case maybe two separate rules named no-for and no-while would make more sense?

You’re right. I like the last option. Agree it’s more logical. I like tailoring rules based on how frequently a part of code is likely to be used.

FWIW: I noticed MS has a no-for-in rule: https://github.com/Microsoft/tslint-microsoft-contrib/tree/releases

So I'm thinking about implementing these rules:

no-loop-break (does not check in switch)
no-continue
no-if
no-switch (which implicitly bans break in switch)
no-for
no-while covers both while() {} and do {} while()

I think this would be a good division, but not 100% sure about no-loop-break and no-while. They could be implemented differently, for example no-break with option ignore-switch as in the original post. Also no-continue could be named no-loop-continue just for duality with no-loop-break.

I published 4.5.0 with the following new rules:

no-loop
no-if-statement

In order to keep the amount of rules and documentation down I went back to the original suggestion of no-loop which includes all loop types.
These rules remains to be implemented:

no-loop-break
no-loop-continue
no-switch

If anyone care to implement them I'm open for a PR on any or all of the above rules :-).

Awesome! Thank you!


pip install openexr failing to build with python 3.8.6 on windows

Hello,
When I try to install openexr on my windows machine through pip, I see the following error message:
C:\Program Files (x86)\Microsoft Visual Studio\2019\Community>pip install openexr
Collecting openexr
  Using cached OpenEXR-1.3.2.tar.gz (10 kB)
Building wheels for collected packages: openexr
  Building wheel for openexr (setup.py) ... error
  ERROR: Command errored out with exit status 1:
   command: 'c:\users\natevm\appdata\local\programs\python\python38\python.exe' -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'C:\\Users\\natevm\\AppData\\Local\\Temp\\pip-install-7y391j0o\\openexr\\setup.py'"'"'; __file__='"'"'C:\\Users\\natevm\\AppData\\Local\\Temp\\pip-install-7y391j0o\\openexr\\setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' bdist_wheel -d 'C:\Users\natevm\AppData\Local\Temp\pip-wheel-krn6ty93'
       cwd: C:\Users\natevm\AppData\Local\Temp\pip-install-7y391j0o\openexr\
  Complete output (15 lines):
  running bdist_wheel
  running build
  running build_py
  creating build
  creating build\lib.win32-3.8
  copying Imath.py -> build\lib.win32-3.8
  running build_ext
  building 'OpenEXR' extension
  creating build\temp.win32-3.8
  creating build\temp.win32-3.8\Release
  C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\bin\HostX86\x86\cl.exe /c /nologo /Ox /W3 /GL /DNDEBUG /MD -I/usr/include/OpenEXR -I/usr/local/include/OpenEXR -I/opt/local/include/OpenEXR -Ic:\users\natevm\appdata\local\programs\python\python38\include -Ic:\users\natevm\appdata\local\programs\python\python38\include "-IC:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\ATLMFC\include" "-IC:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\include" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\ucrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\shared" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\um" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\winrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\cppwinrt" "-IC:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\ATLMFC\include" "-IC:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\include" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\ucrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\shared" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\um" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\winrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\cppwinrt" /EHsc /TpOpenEXR.cpp /Fobuild\temp.win32-3.8\Release\OpenEXR.obj -g -DVERSION="1.3.2"
  cl : Command line warning D9002 : ignoring unknown option '-g'
  OpenEXR.cpp
  OpenEXR.cpp(36): fatal error C1083: Cannot open include file: 'ImathBox.h': No such file or directory
  error: command 'C:\\Program Files (x86)\\Microsoft Visual Studio\\2019\\Community\\VC\\Tools\\MSVC\\14.27.29110\\bin\\HostX86\\x86\\cl.exe' failed with exit status 2
  ----------------------------------------
  ERROR: Failed building wheel for openexr
  Running setup.py clean for openexr
Failed to build openexr
DEPRECATION: Could not build wheels for openexr which do not use PEP 517. pip will fall back to legacy 'setup.py install' for these. pip 21.0 will remove support for this functionality. A possible replacement is to fix the wheel build issue reported above. You can find discussion regarding this at https://github.com/pypa/pip/issues/8368.
Installing collected packages: openexr
    Running setup.py install for openexr ... error
    ERROR: Command errored out with exit status 1:
     command: 'c:\users\natevm\appdata\local\programs\python\python38\python.exe' -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'C:\\Users\\natevm\\AppData\\Local\\Temp\\pip-install-7y391j0o\\openexr\\setup.py'"'"'; __file__='"'"'C:\\Users\\natevm\\AppData\\Local\\Temp\\pip-install-7y391j0o\\openexr\\setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' install --record 'C:\Users\natevm\AppData\Local\Temp\pip-record-yuone7l0\install-record.txt' --single-version-externally-managed --compile --install-headers 'c:\users\natevm\appdata\local\programs\python\python38\Include\openexr'
         cwd: C:\Users\natevm\AppData\Local\Temp\pip-install-7y391j0o\openexr\
    Complete output (15 lines):
    running install
    running build
    running build_py
    creating build
    creating build\lib.win32-3.8
    copying Imath.py -> build\lib.win32-3.8
    running build_ext
    building 'OpenEXR' extension
    creating build\temp.win32-3.8
    creating build\temp.win32-3.8\Release
    C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\bin\HostX86\x86\cl.exe /c /nologo /Ox /W3 /GL /DNDEBUG /MD -I/usr/include/OpenEXR -I/usr/local/include/OpenEXR -I/opt/local/include/OpenEXR -Ic:\users\natevm\appdata\local\programs\python\python38\include -Ic:\users\natevm\appdata\local\programs\python\python38\include "-IC:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\ATLMFC\include" "-IC:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\include" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\ucrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\shared" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\um" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\winrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\cppwinrt" "-IC:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\ATLMFC\include" "-IC:\Program Files (x86)\Microsoft Visual Studio\2019\Community\VC\Tools\MSVC\14.27.29110\include" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\ucrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\shared" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\um" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\winrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.19041.0\cppwinrt" /EHsc /TpOpenEXR.cpp /Fobuild\temp.win32-3.8\Release\OpenEXR.obj -g -DVERSION="1.3.2"
    cl : Command line warning D9002 : ignoring unknown option '-g'
    OpenEXR.cpp
    OpenEXR.cpp(36): fatal error C1083: Cannot open include file: 'ImathBox.h': No such file or directory
    error: command 'C:\\Program Files (x86)\\Microsoft Visual Studio\\2019\\Community\\VC\\Tools\\MSVC\\14.27.29110\\bin\\HostX86\\x86\\cl.exe' failed with exit status 2
    ----------------------------------------
ERROR: Command errored out with exit status 1: 'c:\users\natevm\appdata\local\programs\python\python38\python.exe' -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'C:\\Users\\natevm\\AppData\\Local\\Temp\\pip-install-7y391j0o\\openexr\\setup.py'"'"'; __file__='"'"'C:\\Users\\natevm\\AppData\\Local\\Temp\\pip-install-7y391j0o\\openexr\\setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' install --record 'C:\Users\natevm\AppData\Local\Temp\pip-record-yuone7l0\install-record.txt' --single-version-externally-managed --compile --install-headers 'c:\users\natevm\appdata\local\programs\python\python38\Include\openexr' Check the logs for full command output.
WARNING: You are using pip version 20.2.1; however, version 20.2.4 is available.
You should consider upgrading via the 'c:\users\natevm\appdata\local\programs\python\python38\python.exe -m pip install --upgrade pip' command.

This seems related to issue #5, but their solution seems to be for ubuntu only.

Hi,
I have not used OpenEXR in Windows myself, but it seems like the problem is that the OpenEXR library is not found ('ImathBox.h' not found). I think you would have to install this before adding the python bindings through pip.
This is the same problem as mentioned in issue #5. For example, for Linux you would have to install OpenEXR through the package manager, e.g. apt-get or dnf, or compile it manually, before adding it with pip.
See e.g.:
https://github.com/AcademySoftwareFoundation/openexr/blob/master/INSTALL.md

Ah, I see. On windows, I’m not sure how to tell the python OpenEXR package where I’ve installed OpenEXR, although there might be some environment variable I don’t know about.
Fwiw, I’ve been hosting a C-bound python library on pip for a little over 6 months now, and we found we could work around these compilation issues by uploading bdist wheels instead of sdist wheels, since then python end users aren’t required to compile C code locally. (Though for Linux systems, there’s an additional audit wheel step that needs to be done that requires the project be built on an older disto like Centos 7 for binary compatibility.)
Have you considered uploading bdist_wheel distributions? Is there an advantage to having users compile locally?
Thanks for the help!

That would probably be a good idea. I have not looked into that, but we will see if I manage to find some time eventually. Would also be nice to make training/inference run in tensorflow 2.

hey, did you manage to distribute with openexr without the user needing to build locally?  I'm trying to provide a plugin which requires openexr, but the only thing i'm able to do is pip install on user machines.  Wonder if you might offer some advice?  basically just need to read then edit layers from a multilayer exr, and then write  the edited layer back in to the original multilayer exr. Users already have imageio installed, but I don't think that's sufficient to do as described?

Unfortunately, the situation is the same. You need to have OpenEXR installed to use the EXR I/O in the img_io. You could change to export using some other format than EXR, if this is an option. Running the inference itself does not require OpenEXR. Or you can look in how to package with pre-compiled OpenEXR libs.

Thanks.  My users are using Blender which does use OpenEXR under the hood, but isn't exposing it via the blender API.
I was thinking of packaging with the pre-compiled libraries, but only have windows.  I'll have to try an hunt down some pre-compiled versions for mac and linux (or get a cheap mac and install linux virtual machine on the pc).

tricky because the end users don't usually have python installed on the system, and don't know how to build/install dependancies.


error: invalid usage: unknown argument: -X

I'm getting the following error when running make run after cloning Kitura.
swift build -Xcc -fblocks -Xswiftc -I/usr/local/include -Xlinker -L/usr/local/lib
error: invalid usage: unknown argument: -X

Using: swift-DEVELOPMENT-SNAPSHOT-2016-02-08-a.xctoolchain

Kitura works with snapshot of swift-DEVELOPMENT-SNAPSHOT-2016-03-01 or later.

Thanks Vidam. Last week I was told to use a different snapshot because the master branch was not updated yet. How can I keep track of which snapshot is compatible with Kitura's master branch when I pull it?

You might want to check Swift Version in README.

Thank you!


Allow to pass void to delay

fixes #235
not sure it's a good idea to hide the problem with ts-expect-error
on the other hand, no idea how to fix public api without it

@binjospookie Can you also add type-tests at https://github.com/effector/patronum/blob/main/test-typings/delay.ts ?
So we don't break it by accident in the future
Simple
// void events support
{
 const trigger = createEvent<number>()
 const event = createEvent()

 delay({
  source: trigger,
  timeout: () => 100,
  target: event,
 })
}

will do just fine

Yes. No problem.

I think we can wait for the type-tests and release in 1.16

Done


Group info together

There is lots of info and some of it is all over the place. There is a section for hourly forcast and a section for daily. These are good organized.
However, the section that shows the current weather is all over the place:

Some icons/text are on the top, other on the bottom. Some are left aligned, some right.

Yes and that it on purpose to create sections. If you have a design please share but remember that it shoould work with any device DPI/size

To be clear, I like the sections but the screenshot I sent is way too cluttered.

I think what you want is for the wind speed, moon phase, and precipitation icons to be centered in the blank space, and everything scaled a little bit like this:

@RageGamerBoi i will do something like that. Though so you understand that "blank" space is reserved space for the minutely graph that you get when it is raining (in regions supported by owm or meteo france).

Oh, I didn't realize that was a feature (which is really cool btw)... Maybe shift the layout back to the current one when the graph needs to display?

The updated layout looks fantastic!
The one think I think needs to be changed is the temperature needs to be a little bigger for readability purposes.
That's the main metric most people are looking for, and what most weather apps emphasize a little bit more:

Idk, just a thought.

@RageGamerBoi new build coming. Let me know what you think

Oh wow, everything is a lot more polished!
It even helped with a little bit of the font scaling problems.
It feels a little empty around the temperature now though, maybe make it even a little bigger and add the unit after just the main readout.
Othar than that, I th

@RageGamerBoi thanks i am not finished yet and yes might be a bit empty on the left side. Would prefer not to raise the temp size that big as for me it is far from the most important data. Thinking about those 'icons/data" for wind, uv.... Might want to change those. Also wondering how i could add more (pressure, ...)

Pressure could be good as an optional item on the graph (like adding an optional traffic layer on top of Google maps), along with wind speed. #31
Adding the temperature unit could fill the space a bit, and making the current weather icons slightly bigger (when there is no rain graph) could help too.


Build pipeline

this is pull request

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

This is command for pull request


[CH] CAST to string diff

Backend
CH(Clickhouse)
Bug description
SQL:
select cast(array('1','2', 'abc', '\'1\'') as string);
Vanila:
[1, 2, abc, '1']

Gluten:
['1','2', 'abc', '\'1\'']
Clickhouse：

Spark version
None
Spark configurations
No response
System information
No response
Relevant logs
No response

select concat(split(n_comment, ' ')) from nation; This sql will throw exception
Caused by: org.apache.gluten.exception.GlutenException: Cannot parse quoted string: expected opening quote ''', got 't': while converting source column `concat(splitByRegexpSpark( _19,n_comment,-1_20))` to destination column `concat(splitByRegexpSpark( _19,n_comment,-1_20))`: while executing 'FUNCTION _CAST(concat(splitByRegexpSpark( _19,n_comment,-1_20)) :: 3, Nullable(Array(String)) :: 4) -> concat(splitByRegexpSpark( _19,n_comment,-1_20)) Nullable(Array(String)) : 2'

Since cast to Array(String)assumes that elements are seperated by quote(that is '), but settings composed_data_type_output_format_mode='spark', discard quote, and error is met.
Modification is not so easy, lots of work need to be done. So we leave it alone now, if we have time, try fix.


Gallery is not opening

The galleries are showing but if you click on one gallery it says "Unable to connect to the site"

https://github.com/Dar9586/NClientV2/issues/744#issuecomment-2452810317

This issue has been fixed. Thank you for everyone/someone who fixed it!


Facebook bot only listens to me

I've set up a Botkit bot with Facebook Messenger integration, with a test Facebook page and app. I can send it messages, and the bot can respond OK. But when my friends send it a message, the bot never receives it - GOT A MESSAGE HOOK is never printed.
Below is a minimal program that produces the behaviour. I see log output when I send messages to the facebook page, but none when others do. I suspect I must have something wrong in the facebook configuration, but I'm not sure what to check for. Here's my webhook setup: http://i.imgur.com/rnChTz7.png
What do I need to do to get my bot to respond to everyone?
var Botkit = require('botkit');

var controller = Botkit.facebookbot({
  access_token: "redacted",
  verify_token: "redacted"
});

var bot = controller.spawn({
});

controller.setupWebserver(8069, function(err, webserver) {
  controller.createWebhookEndpoints(controller.webserver, bot, function() {
    console.log('bot online, beep boop');
  });
});

controller.on('message_received', function(bot, message) {
  console.log(message);
});

Botkit 0.1.1
Windows 10, ngrok
Node v5.4.0

Before your bot can interact with the public, it must be approved by facebook.
If youd like your friends to use it before then, they must be added as administrators of the app.

Thanks - that looks like it. Should have poked around a bit more.

Facebook doesnt do a great job of documenting it. =]


feat: text variation to markdown

:tada: This PR is included in version 0.188.0 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Error: ioctl(2) failed

I add wetty bootstrap script into crontab:
@reboot   nohup node /home/johnroyer/sites/wetty/app.js -p 3000 >> /home/johnroyer/sites/wetty/run.log 2>&1

After reboot, I will see error message on the web page:
/usr/bin/env: ../bin/ssh: No such file or directory

And run.log will see another message:
Error: Error: ioctl(2) failed.

But if I manually invoke this command, everything goes well.
Is it possible a config error ?

wetty version: v0.5.0

@butlerx I am experiencing the same problem as @johnroyer. However, I was able to fix it by running index.jsfrom the base folder which leads to my problem that the browser presents me with a password dialog but the entered password is not accepted.
I would like to assist further if necessary, is there a debug switch I can enable? The console log does not show any errors, just normal start and end of session.

this should no longer be an issue with v1.0.0, feedback appriciated

Error above: Error: Error: ioctl(2) failed.  is no longer shown on v1.0.3

Perfect


Update scala-compiler, scala-library, ... to 2.13.8

Updates

org.scala-lang:scala-compiler
org.scala-lang:scala-library
org.scala-lang:scala-reflect

from 2.13.1 to 2.13.8.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang" } ]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #62.


A little problem.

So I tried to login with a microsoft account and the API is down. PLZ fix it quickly. Peace and Love :)

It has to do with the GoSnipe microsoft authentication server (https://github.com/MCGoSnipe/api-endpoints) going down the past few days. So similarly, GoSnipe users has been unable to authenticate with their microsoft account. Right now I don't really have a way to solve the problem (the underlying code for the auth server I mentioned is in my gist, so you might wanna check it out). Rn I'm making a new sniper called buckshot which might support microsoft's authentication flow in the near future, but as of now I'm not maintaining this sniper anymore, so I'm sorry I can't fix this problem atm.

Well ok then!
thank you for the announcement for creating a new sniper called buckshot. I will try it when it will be released. Thank you!
From: @.>
Sent: Sunday, May 2, 2021 3:30 PM
To: @.>
Cc: @.>; @.>
Subject: Re: [chronicallyunfunny/NodeSniper] A little problem. (#1)
It has to do with the GoSnipe microsoft authentication server (https://github.com/MCGoSnipe/api-endpoints) going down the past few days. So similarly, GoSnipe users has been unable to authenticate with their microsoft account. Right now I don't really have a way to solve the problem (the underlying code for the auth server I mentioned is in my gist, so you might wanna check it out). Rn I'm making a new sniper called buckshot which might support microsoft's authentication flow in the near future, but as of now I'm not maintaining this sniper anymore, so I'm sorry I can't fix this problem atm.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/chronicallyunfunny/NodeSniper/issues/1#issuecomment-830802358, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANJBS43R7Y7GDELWQEDZCBTTLVAWFANCNFSM437EXFJQ.

Well ok then!
thank you for the announcement for creating a new sniper called buckshot. I will try it when it will be released. Thank you!