Add steps to install from source

Installing from pypi is not recommended as mentioned by jeremy(due to it's rapid changes) and you don't want to use conda on Google Colab.
So Add steps to install from source

Although this might be an overkill, the process of building and installing fastai could avoid any hard coded string with the following code:
# Clean the dist folder
!rm -rf fastai/dist

# Create a Tarball using sdist
!cd fastai && python setup.py sdist

# Install through pip
!cd fastai/dist && pip install $(ls)

It could be split in 3 cells with the ! or it could also be in one cell and with the %%bash magic to avoid the !, in this case it could be combine with the previous comment in and create a single cell with the %%bash magic for succinctly
The last line could be replace with !pip install fastai/dist/$(ls fastai/dist/) but it seems much more confusing to me.

I tried several alternatives, the one you proposed usually throw me a error/warning that make me think something is wrong. I found out that the most legible and simple is the following:
%%bash

if ! [ -d fastai ]
then
  git clone https://github.com/fastai/fastai.git
fi

cd fastai

git pull

pip3 -q install .

Do you see any potential disadvantages of using pip directly instead of generating a source distribution with sdist?
Colab doesn't discriminate between python3 and python, neither between pip and pip3. However, since we are in a linux architecture I think we should keep the convention of naming python3 and pip3 for compatibility reasons, specially if someone downloads the notebook and uses it locally

I didn't get any warning/error when I ran that cell. Even after couple of times, no error/warning
But I still like the idea of merging
# Create a Tarball using sdist
python setup.py sdist

# Install through pip
pip install dist/$(ls dist)

into
pip -q install .

I could've thought of that in the first place, that's dumb!
I don't want to add pytorch in this cell as that is already in requirements.txt. If the requirements.txt changes with different version of pytorch, this might fail
The idea of using pip is that, it can be run in both python2 and python3. In both colab and local jupyter-notebook we can change runtype as python2/python3

Great then!
I mentioned the manual install of Pytorch in order to select the specific version of CUDA but if that doesn't implies any disadvantages, I believe it's fine.
I added the && echo ... because the solution uses the -q option. Of course they are completely optional, I just want to have a clear yet short answer for each command.
I also agree with you regarding the use of pip over pip3, personally I use pipenv on a daily basis and pip is preferred over pip3 since most of the time I work in virtual environments.

@ELC
My bad about the warning, I was looking for red highlighted text in colab. Turns out github only highlights in red not colab.
I've added you as author of commit as well.
@corykendrick let us know whether this PR is fine ?

Now that you've made all the modifications I believe you should remove the cells where fastai is installed via pip (legacy) and move the new section (Installing fastai from source) just above (Special additions for particular lessons). Currently there are two places where fastai is installed and that's not ideal.

I like to open course notebook in one window and fastai_in_colab in another. Than install fastai and download datas in fastai_in_colab notebook and then head over to course notebook which is opened in another window. In this way I avoid copying to each of the course notebooks.
If the structuring of the notebook can be improved, then you should create a new PR.
In the original notebook there are some places where pip is used and some places where pip3 is used. For that also I think we should create a new PR.
Anyway I've changed according to your suggestion, review it.

I believe that the notebook is ready, no more suggestions for now.
Following your method, could you run the cells in the course notebook without adding the installing cells for fastai? How do you "share" the installation?
I created that gist to have a personal derivate that would work both for the course and general purpose. I just wanted something I could hit "Run All" and wait and when it ends, I'm ready to go. That was my objective, I saw this repo and use it as a template.

You need the installing cells for fastai. You just don't need to copy them to other notebooks.
When you run a cell in colab, the run type is available across all notebooks for that gmail user. So you just need to install fastai in one notebook and fastai is available throughout all other notebooks.

I didn't know that! That's awesome and a great insight! Is it in some part of the docs where I can refer to?
I tested the notebooks this morning and the code is broken, FastAI has released the 1.0.3 version and the code no longer works, it throws ModuleNotFoundError. I wouldn't be able to test and fix until 2 or 3 days.

pytorch 1.0rc1 and 1.0rc0 has been released hours ago. So fastai is moving to support early alpha version of pytorch. The code has been changed substantially I guess and all codes in 0.7 has been moved to old folder inside fastai(no idea why they are putting in old folder, they could've removed it completely).
So the imports such as
from fastai.transforms import *
from fastai.conv_learner import *
from fastai.model import *
from fastai.dataset import *
from fastai.sgdr import *
from fastai.plots import *

are not working now. But from fastai.imports import * still works.
So I'll port back the cell where we had installing from pypi and Add this section after that. Also I can add comment where it warns about this issue. When they release new course supporting alpha version, we can remove the section which uses pypi.
But I don't see where they released 1.0.3 ?
Tell me if there is any better way

I didn't know that! That's awesome and a great insight! Is it in some part of the docs where I can refer to?

I was playing around to find a easier way. Don't know if there is any docs.

The release isn't in the releases tab but in https://github.com/fastai/fastai/commit/614f25a00fa6aa7de4e0dbdda393d25f63aa23ed. Also in the readme there are recommendations to manually install pytorch and torchvision.
Upload the changes you consider necessary, I will check them as soon as I'm free

I made some tests and this seems to work:
!pip install torch_nightly -f https://download.pytorch.org/whl/nightly/cu92/torch_nightly.html
!pip install --index-url https://test.pypi.org/simple/ --extra-index-url https://pypi.org/simple/ torchvision==0.2.1.post1
!pip install fastai

Also, there is no place in the docs about how the imports should be made now, they implemented an imports folder with a __init__.py file instead of a imports.py, I think that's why the other imports were broken.
I can acess torch at least after running from fastai.imports import * but even after that torch.cuda.is_available() returns False while torch.backends.cudnn.enabled returns True.
I will be working in some modifications to have both the cells for the legacy version and the cells for the 1.x version. I'll post my results here as soon as I have them

Hey you're back early!!
Check this. This will be removed soon I guess, so quoting:

XXX: require torch>=1.0.0 once it's released, for now get the user to install it explicitly

We can not add cell to install from source even if torch>=1.0.0 is released. Because the current course can not be followed with the latest source because of this, which says:

If you are using fastai for any course.fast.ai course, please do NOT install fastai from pip or conda using the instructions below; the instructions below are for fastai v1, but the courses use fastai 0.7. For the courses, you should simply follow the instructions in the course (i.e. clone this repo, cd to it, and conda env update), and the notebooks will work (there is a symlink to old/fastai/, which is fastai 0.7, in each course notebook directory), or else use pip install fastai==0.7.0 to install the version compatible with the course.

Note: If you want to learn how to use fastai v1 from its lead developer, Jeremy Howard, he will be teaching it in the Deep Learning Part I course at the University of San Francisco from Oct 22nd, 2018.

It recommends to install fastai using pip install fastai==0.7.0. But I believe the old folder is little ahead of 0.7.0
So what's your take ? Should I close this pull request(When the course for Oct-2018 course will be available, I'll open it again to install from source) or add instructions to install from old folder.

Well the original fastai_in_colab repo is broken anyway. As the latest version for fastai is 1.0.4

I believe that there are two main intentions with this repo:

Provide an easy setup for fastai to follow the course
Provide an easy setup for fastai for general purpose

For the first, the 0.7 version is needed (at least the new version of the courses are released) and for the second it might also be needed to use the 0.7 until the docs and the course are available in order to notice the difference between 0.7 and 1.x
Installing fastai v1 just require two steps instead of three

Follow the following 2 steps in this exact order:
Install the nightly pytorch build, with cudaXX package version matching your system's setup. For example for CUDA 9.2:
pip install torch_nightly -f https://download.pytorch.org/whl/nightly/cu92/torch_nightly.html
Install fastai:
pip install fastai
NB: this will also fetch torchvision-nightly, see the conda entry above for details.

And installing fastai v0.7 just requires one step as you previously said pip install fastai==0.7.0 although they made 0.7.1 and 0.7.2 releases in the past 12 hours, but they point the same commit, maybe a confusion? I don't really understand why they did so.
I don't know the author's intentions but I believe we should keep it updated with both version as the gist I mentioned earlier.
What's your opinion?

Well the description of this repo indicates only for fastai course. But I like the idea of yours.
I can add two different sections: one for course and one for general purpose. Later, once the new course is live I can change that.

Well the description of this repo indicates only for fastai course. And the major part of the notebook is about how to download datas for specific lessons of the course.


Add CDN link to Readme and website - closes #21

I have this setup to be hosted on s3 already.

Do you have any objection to being on cdnjs as well? It's a popular resource and I think there's value in also offering their cdn even if you don't link to it anywhere.
That said, I don't want to be responsible for putting you code up anywhere you don't want it to be, and will take the steps to remove it if it's not okay with you.

What are the advantages of having two cdn services? Seems to add more things without additional functionality.

As I see it:
People know of, use (search), and trust cdnjs already.
cdnjs automatically provides versioned distributions of tagged commits so you don't have to push to S3 yourself every time you want to release a new version
S3 costs you money, cdnjs doesn't.
Whether those possible advantages outweigh the drawbacks (e.g. lessened control) I don't know.


fix:[#1651] Add MOTD message for SB keys

Add logic to check for SB enrollment and keys
Update motd template

Closes #1651

These changes are untested.  I'll try to get an environment set up to test them before marking the PR as ready.

The word wrapping is a little broken, but I believe this is an issue with glow and their handling of hyperlinks. Marking as ready since its out of my control and ugly warnings are better than no warnings at all.

I did a workaround for this, you can use ~ in your text and it will become a new line in the final output.
Good for another PR though, I don't want to hold this up over some formatting. LGTM.

Is this warning something we would need in Bazzite? Only case I can think of is if someone were to install Bazzite without secure boot on and then turn it on? But then they wouldn't be able to see the message anyway?

Missed this, seems there's a typo here?
``if [[ $ENROLL -eq 1 ]] && [[ $SB_ENABLED -eq 0 ]]; then```
$ENROLL doesn't exist, $ENROLLED is created above.

I'm not sure how I did that, but its fixed.  Should be good after the CI clears.  I rechecked it in my environment.

Let's merge this tomorrow (after the :stable builds go out), that way we have a week of it in :latest if we need to tweak the text.

if this merges first, i can refactor this PR to consolidate the logic

closing in favor of https://github.com/ublue-os/bluefin/pull/1661


ZooKeeper installation randomly fails

I'm using the latest Zookeeper install script (the one installing zookeeper-server) but I find that it randomly fails on worker nodes:
~$ tail -f /var/log/dataproc-initialization-script-0.log
+ NODE_NUMBER=1
+ [[ Worker == \W\o\r\k\e\r ]]
++ hostname
++ sed 's/.*-w-\([0-9]\)*.*/\1/g'
+ NODE_NUMBER=4
+ echo 4
+ cat
+ systemctl restart zookeeper-server
Job for zookeeper-server.service failed because of unavailable resources or another system error.
See "systemctl status zookeeper-server.service" and "journalctl -xe" for details.

Running systemctl status zookeeper-server.service outputs:
~$ systemctl status zookeeper-server.service
● zookeeper-server.service - LSB: ZooKeeper is a centralized service for maintaining configuration information, naming, providing distributed synchronization, and providing group services.
   Loaded: loaded (/etc/init.d/zookeeper-server; generated; vendor preset: enabled)
   Active: failed (Result: resources) since Tue 2018-06-26 19:53:40 UTC; 3min 42s ago
     Docs: man:systemd-sysv-generator(8)
  Process: 4106 ExecStop=/etc/init.d/zookeeper-server stop (code=exited, status=0/SUCCESS)
  Process: 4559 ExecStart=/etc/init.d/zookeeper-server start (code=exited, status=0/SUCCESS)
 Main PID: 4055 (code=killed, signal=KILL)

Jun 26 19:53:39 ew-streaming-w-2 su[4561]: Successful su for zookeeper by root
Jun 26 19:53:39 ew-streaming-w-2 su[4561]: + ??? root:zookeeper
Jun 26 19:53:39 ew-streaming-w-2 su[4561]: pam_unix(su:session): session opened for user zookeeper by (uid=0)
Jun 26 19:53:39 ew-streaming-w-2 zookeeper-server[4559]: JMX enabled by default
Jun 26 19:53:39 ew-streaming-w-2 zookeeper-server[4559]: Using config: /etc/zookeeper/conf/zoo.cfg
Jun 26 19:53:40 ew-streaming-w-2 zookeeper-server[4559]: Starting zookeeper ... STARTED
Jun 26 19:53:40 ew-streaming-w-2 systemd[1]: zookeeper-server.service: PID 4575 read from file /var/run/zookeeper/zookeeper_server.pid does not exist or is a zombie.
Jun 26 19:53:40 ew-streaming-w-2 systemd[1]: Failed to start LSB: ZooKeeper is a centralized service for maintaining configuration information, naming, providing distributed synchronization, and providing
Jun 26 19:53:40 ew-streaming-w-2 systemd[1]: zookeeper-server.service: Unit entered failed state.
Jun 26 19:53:40 ew-streaming-w-2 systemd[1]: zookeeper-server.service: Failed with result 'resources'.

This seems like a race condition, similar to https://bugzilla.redhat.com/show_bug.cgi?id=1381997. However, I wasn't able to repro just by repeatedly restarting the server: while true; do sudo systemctl restart zookeeper-server.service; sleep 1; done
If the Zookeeper gets stuck in that state, does simply restarting it work? Or do you have to manually delete the pid file?

Once this happens, I wasnt able to restart it manually....
/var/run/zookeeper/zookeeper_server.pid doesnt exist so no pid for me to delete...

What was the subsequent error from sudo systemctl restart zookeeper-server? Same thing?

yes

I'm inclined to close this issue because the Zookeeper init action is effectively deprecated in favor of the optional component: https://cloud.google.com/dataproc/docs/concepts/components/zookeeper.
That being said, the optional component only runs zookeeper on the master node in standard clusters, so it is technically less fault tolerant than the init action was.


Kindly add top features directly inside windows 11

Description of the new feature / enhancement
most people dont know about power toys so we want these in windows 11
Scenario when this would be used?
widnows 11 and 10
Supporting information
this will help grow windows ecossytem

Are there specific utilities you want in Windows 11? Or do you want the whole suite? /needsinfo

Mouse without borders and Always on top
On Fri, Aug 9, 2024, 11:50 PM Kayla Cinnamon @.***>
wrote:

Are there specific utilities you want in Windows 11? Or do you want the
whole suite? /needsinfo
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/34234#issuecomment-2278545971,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIRXJSDYH4NDGJEEHONPYMLZQUFORAVCNFSM6AAAAABMIXLOH2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZYGU2DKOJXGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

I believe that the following utilities should also be a part of Windows.

Peek
PowerToys Run
PowerRename
File Locksmith

/feedbackhub


Add eslint using the airbnb config.

In an effort to make the JS consistent, this adds an ESLint gulp task. ESLint
is very configurable, and has no defaults. Instead, this reaches for
airbnb's legacy config (which covers ES5), which is one of the most popular
configurations. As a last step, it turns off all of the rules which currently
fail. We can turn them on piecemeal, or override the airbnb style to use a
different format.
Rules to prioritize (as they likely indicate bugs):

"array-callback-return": "off",
"eqeqeq": "off",
"guard-for-in": "off",
"no-inner-declarations": "off",
"no-loop-func": "off",
"no-mixed-operators": "off",
"no-undef": "off",
"no-use-before-define": "off",
"one-var": "off",

Also adds to CI, though I didn't come up with a great strategy to separate execution from the Python tests. This means that ESLint is running once per Travis configuration...

This is great! Thanks for this. I'll let @agjohnson look over it since he owns most of the front end, but I think this is a solid step forward.

I wonder if we can add this to tox, during the lint step? I don't know if it has a good support for installing JS deps, but it would be useful to be able to run it locally.

Thanks @agjohnson . I've been going down a rabbit hole of tox configurations to try to avoid setting up a whole python env, but I'm pleased with this state.

Great, yeah this is a solid addition. I've opened up #2909 to actually address the linting rules to a stricter level.


Bbox regression target maybe isn't exactly correct?

Hi tianwei,
I did a experiment based on your code. In training, I only used one frame data which is the first frame data of the first segement,timestamp:1509155241195667.
I trained 100,000 steps, and making the cls_loss to 0.0001 and reg_loss to 0.0000 which means that this frame should been exactly learned by the model. However, when I visualizes the prediction, I found that all attributes of the objects  matched the groundtruth except the location of x y .

Blue represents gts, yellow represents predictions.
The reason I think is the calculation of x ytargets :
https://github.com/tianweiy/CenterPoint/blob/master/det3d/datasets/pipelines/preprocess.py#L385
https://github.com/tianweiy/CenterPoint/blob/master/det3d/datasets/pipelines/preprocess.py#L391
The way for calculate x y targets, cast float number to int , isn't same as the classfication. There is a gap between training and testing process due the mismatch of cls and reg. I think it is better to use  rounding-off method, this will lead a more exactly regression result.

we still predict an offset here https://github.com/tianweiy/CenterPoint/blob/c9822e1202ff1b197db237d284f546e8cb98d766/det3d/datasets/pipelines/preprocess.py#L410
https://github.com/tianweiy/CenterPoint/blob/c9822e1202ff1b197db237d284f546e8cb98d766/det3d/models/bbox_heads/center_head.py#L397

"isn't same as the classfication"
what does this mean ?

@tianweiy In heatmap calulation,

"isn't same as the classfication"
what does this mean ?

My bad, I think you draw the gaussian kernel using  interpolation method for the value of center of the kernel due to that object's center isn't always a int number.
I just read the code:
https://github.com/tianweiy/CenterPoint/blob/master/det3d/core/utils/center_utils.py#L52
Now, I think the way to calculate x y is same for cls and reg.
However, I still think the mismatch between prediction and gt is much due to the center calculation. I will try to find the reason. If you are interest for this, you can do the same experiment which is quite easy.

Will do after Friday's waymo challenge

Will do after Friday's waymo challenge

By the way, I think fix this problem may lead a small improvement on performance.

is it still a problem (I see you closed this issue)? I have limited bandwidth so I want to check if I should still look into this ?

@tianweiy I found there was a bug in my code which led to the wrong regression. My fault...

sure, no problem


Should respond to control signals (^C, ^Z, etc) with syntax errors, not silence

I think? it's not a valid command, and it ends with CRLF. RFCs didn't mention anything about control characters.

this doesn't appear to be an issue in our code, actually. The response is correctly written to the stream, and commands continue to be read, but it never makes it through telnet. Could be a telnet issue?

definitely a telnet issue. openSMTPD and friends have the same response. Telnet is waiting for a response code to the control character, and since we never return telnet control codes, it waits forever.


CanvasItem Function 'draw_mesh' is not Affected by Transform or Modulation Parameters in GLES2

Godot version
3.3 stable
System information
Ubuntu 21.04, GLES2, Mesa Intel® UHD Graphics 630 (CFL GT2)
Issue description
The Canvas Item function draw_mesh has four parameters but two of them, transform and modulate, do not affect the drawing of the mesh in GLES2.
Steps to reproduce

Create a project in GLES2
Add a CanvasItem node to the scene and attach a script to it
In the script use draw_mesh  in _draw() to draw a QuadMesh
Change the transform and modulate parameters of draw_mesh, they should work in GLES3 but not in GLES2

Minimal reproduction project
MinimumProduct.zip

@blaze-the-star Can you test this on 3.4? Also, can you reproduce this after disabling batching in the project settings?


[move-to-yul] Add support for copying vectors and structs between storage and memory

Motivation
This PR adds support for:

copying non-value types such as vectors and structs from storage to memory:
#[callable(sig=b"greet() returns (string)")]
public fun greet(): vector acquires State {
borrow_global(self()).greeting
}

updating non-value types in storage:
#[callable(sig=b"setGreeting(string)")]
public fun setGreeting(greeting: vector) acquires State {
borrow_global_mut(self()).greeting = greeting;
}

Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Currently two tests are added:

Greeting.move in hardhat-examples
DispatcherEncodingStorage.move in tests/test-dispatcher

More tests will be added later.

/land


Add option to include y distance for nearest touched spot

Instead of only considering the distance on the x axis when checking for the closest spot, this feature allows to choose the closest spot based on Euclidean distance
TODO:

[ ] Write tests

Fixes #716, #261

#892

I will check it soon.

Just to be clear, this PR and #892 to are independent of each other, although it might make sense to use them together in many cases.

Initially I liked your approach since it fixes #892 at the same time. But it actually misses the case mentioned in #892 where two points of two lines have the same x and y coordinate.
Adding the distance property mentioned in #892 might also be useful for the case where we only consider the x distance.
Let's consider the following case where we have one line with a data point at x = 5 and another line with a data point at x = 6. Now the user taps on x = 5.2. Both data points would be returned in the LineTouchResponse which will lead to the following selection. I don't think there currently is a way to prevent this.

There is a touchSpotThreshold property in the LineTouchData class.
The default value of touchSpotThreshold is 10, leading to this problem. You can try a lower value like 3 to fix the mentioned problem.

Lowering the threshold does not really fix the problem. After lowering it (which I maybe don't want because I want certain user interaction) I could just move the two points close together and would again face the same issue.

I switched to a new approach where the user can set a distance function. Also I added the distance to LineBarSpot. I still have to clean up and format the code.

Any news on this?

Should be finished now

I wrote my last comments. Also, you must rebase this branch with our dev branch. (It seems there are some minor conflicts)

I merged dev into my branch, did this solve the conflicts?

No, it doesn't solve the problem.
You need to rebase with the dev branch.
 (Now it's 11 commits)

I squashed them into one and rebased with the dev branch

And about the CHANGELOG.md, thank you for adding BREAKING change.
But also we have an IMPROVEMENT.
You made the touch logic customizable!
We need to let users know how they can customize the touch logic.

Any update?
We need to release a version, and I'm waiting for this branch.
Do you have any estimation on this?

Yes, it's hopefully done now. And also thank your for your patience. This is my first PR and there were a lot of things I missed at first, but I enjoyed it.

Thank you @joeldomke
Don't worry, that was good!
It has just merged into the dev branch.
I'm going to bump up a version :)
It will be great if you modify (or add) a chart to showcase this feature.
I hope to see more from you :))


Using your codes

Hello,
I would like to know which .bin file should be flashed onto the AWR1642 to be able to get the 2D plots working in your 'people counting demo' python code.
Also if any information on other requirements (like mmwave SDK version/CCS studio) is needed to run your demo, please do share them as well.
I am not using an the DCA1000EVM, and just have the AWR1642 board and my laptop, FYI.
I have run your python code but the 2D plot doesn't work and the
dataOk = update()
line always returns dataOk as 0.
I am assuming it is because I have not flashed the correct .bin file.
Could you also let me know as to which of the SOP2/SOP1/SOP0 switches are short while running your code?

Hello,
In order to use the people counting demo you need to program your radar with the bin file for the people counting demo. For that, you need to download the Industrial Toolbox 4.1 (the latest version does not include the code for the AWR1642). Go to this link and on the top right side press the download button. Once you have downloaded, open the labs folder -> people_counting -> 16xx_people_counting.
I recommend you to check the user guide on the docs folder, it explains how to program the radar with the people countig demo usin the Uniflash software. Once you have installed the program correctly, you should be able to run the code in this repository.
Let me know how it goes.
Ibai

Hello,
It worked, thank you so much!


golem-bulk-image-handler

LGTM, thanks for this :) And thank you for the feedback that you came up with along with creating it.


shared item source cause RelativeSource bug.

I have a collection named Items.
I have a DataGrid bind ItemsSource to Items and add drag handler.
I have another ListBox and just display Items as tile view to display image data.

  // Listbox.ItemTemplate
  <my:DisplayControl TargetElement="{Binding RelativeSource={RelativeSource AncestorType={x:Type ListBoxItem}}}"/>

  // Bug: TargetElement will get '{DisconnectedItem}' after drag drop.

@a44281071 Hi, sorry for the late response.

// Bug: TargetElement will get '{DisconnectedItem}' after drag drop.

I don't know what you mean with this. Is this a output from your app?
Normally if the Items are the same, then all should work automatically. Or have you custom handlers for the drag&drop operations?

I’m closing this issue because it has been inactive for a few months. This probably means that it is not reproducible or it has been fixed in a newer version. If it’s an enhancement and hasn’t been taken on for so long, then it seems no one has the time to implement this.
Please reopen if you still encounter this issue with the latest stable version and then please use the issue template.
If you have enough time then it would be nice to test this issue against the latest alpha version (availabe via NuGet and appveyor, appveyor feed).
Thank you! 🙏


Able to set the Default Terminal to Powershell 7 in Windows Terminal

Using one button able to install Powershell 7 and set the default terminal into it.
https://github.com/ChrisTitusTech/winutil/assets/62820836/2995670a-c48c-4b51-a9f7-a23c1b9df170

Also, if possible maybe revert winutil.ps1 and inputTweaks.xaml to the state before your compile.
Because those files are autogenerated anyways the only create a huge amount of bloat and changed lines in the PR

I know. But I use Cli to commit all of the files
I know the file are autogenerated that's why I deleted in the Pull Request

Alright I made it using the Install-WinUtilProgramWinget But does anyone know how does this function work?

I'll check the code later today. Will come back to you :)
Edit: @blusewill I created a PR to your Repo with a few modifications that should work as expected

Awesome will check that later today


Push showcase

Motivation
Add a push section to the showcase app.
JIRA: https://issues.jboss.org/browse/AGCORDOVA-238

Circleci is failing because @aerogear/push has not been published yet. You need to run npm run linkDev locally to try it out.

We actually trying to link dev dependencies here but it is manual process in the circle ci script. Adding push there will make build pass

@wtrocki Yes, but ion-refresher still requires user interaction to update (swipe down)


Patchset: add remaining assets for ipi

This is enough for testing for now.
A few todos:

Retrieve user account details and ensure the plan is not lite
Add any extra validations steps aside from just account validation
Rework the validation to use the input from the install-config.yaml instead of only env vars

@mkumatag @bpradipt
any comments on the TODOs? I'll follow up on any suggestions in an add'l PR if that's okay. This is to get us a little farther so that @mjturek can test the TF initialization using what we have so far.

@mkumatag @bpradipt
any comments on the TODOs? I'll follow up on any suggestions in an add'l PR if that's okay. This is to get us a little farther so that @mjturek can test the TF initialization using what we have so far.

The TODOs makes sense. I would go even further and say that the user account requirements w.r.to having paid account can be mandated as prereqs in the first phase.

@mkumatag yah i've got the apikey bits in another branch. i can submit a pr for that and we can discuss further.
@bpradipt I agree with:

I would go even further and say that the user account requirements w.r.to having paid account can be mandated as prereqs in the first phase.

Easy enough to fail & exit if the account is lite (as long as that's something we can query using the apis)


Resolve typo error

Hi, please review

Please assign gssoc23 label
Thank you

Great work buddy @gulshanjakhon !!
Do not forget to

Share your contribution on socials(twitter mainly), tag us and Clueless 🫂
Give this repository a star ✨
Follow Clueless Community on GitHub ✅


Adjust Generator for Updated OpenSearch API Spec

Description
Adjust Generator for Updated OpenSearch API Spec. Related to https://github.com/opensearch-project/opensearch-api-specification/pull/275
Issues Resolved
Closes #710
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.

The failing test is not related to this PR.
The failing test is relevant to changes in this commit.


Internationalization: We need multi-language support

Research for third-party solutions such as https://de.transifex.com/
Figure out how to integrate them in our development process

Looks like transifex offer a free service to open source projects.


Support an operator for chaining multiple go-template commands in a single mustache

In the latest Hugo version, we can have newlines in the template code. This allows us to write multiline template code. The only piece missing to enable using go template as a language for longer programming constructs is the ability to write multiple independent functions. Adding an operator to chain commands could make this a reality.
A suggestion would be use semi-colons, popular as line terminators in a lot of programming languages, unused in Hugo so far.
Would love to write something like the following:
{{
      with site.Title;
            printf "%s" (. | humanize);
     else;
          printf "No title";
     end;
}}

equivalent to:
{{with site.Title}}
    {{printf "%s" (. | humanize)}}
{{else}}
  {{printf "No title"}}
{{end}}

If we want to stick to the prefix format for operators, it is okay to have ; as a starting parameter as well. We could mentally think of {{;  ;}} as multi command template while internally it is {{; function 1 ; function 2; noop}} in the prefix format.

You need to raise this issue upstream (Go stdlib), but I suspect it will most likely be discarded.

Created https://github.com/golang/go/issues/46588

The upstream issue was declined.


Make storage of dataset on device for refinement optional

This can significantly reduce device memory consumption in cases where refinement is not desired.
Also ensure that RAFT indexes compile and work correctly for compute capability 8.9 devices.
Close #329

/kind improvement

/kind enhancement

/remove-kind improvement

/issue #329

Noticed that the current changes are not correct because of how vector-adding is inconsistently handled upstream. Working on a more complete fix, but it will not be quite as simple as this.

/lgtm


import returns undefined in iOS simulator, but connected device is fine

When I build from XCode onto my connected iOS 12.2 iPhone X, I am able to load contacts as expected using this module.
But when I run react-native run-ios in the terminal and load the app in the iOS 12.2 iPhone X simulator, the following import statement sets Contacts to undefined:
import Contacts from 'react-native-contacts';

Any idea why?
Steps I've already done:
In my package.json, I verified that I'm on react-native-contacts "^4.0.2".
I ran:
watchman watch-del-all &&
rm -rf $TMPDIR/react-native-packager-cache-* &&
rm -rf $TMPDIR/metro-bundler-cache-* &&
rm -rf node_modules/ &&
yarn cache clean &&
rm yarn.lock &&
yarn install

I linked in XCode by going to the Build Phases tab, expanding "Link Binary With Libraries", and dragging over libRCTContacts.a from the Project navigator.
I added the following to my Info.plist file:
<key>NSContactsUsageDescription</key>
<string>Enable $(PRODUCT_NAME) to access your contacts to invite your friends.</string>

I have run "Erase all Content and Settings" in the simulator.
The Contacts app in the Simulator shows 6 default contacts when the app is loaded.
I have tried all these steps while running node v12.3.1, and also while running node v10.16.0, but I get the same results either way.
All other react-native libraries I'm using import and work fine in the simulator in this same app.

I facing same error. Contacts is undefined.
"dependencies": {
"react": "16.8.3",
"react-native": "0.59.8",
"react-native-contacts": "^4.0.2"
}

Not sure, usually people get the software working in an emulator and run into issues on real devices. You are experiencing the opposite.

I ran Clean Build Folder from XCode and the problem persisted.
But then I deleted my ios/build folder and reran react-native run-ios and things started working.


lib/ui/example doesn't work with node < 8

That's because of trailing comma in function call: https://github.com/storybooks/storybook/blob/master/lib/ui/example/server.js#L12
We should either remove the comma until Node 8 becomes LTS in October, or add a note to readme

Trailing comma in function call is invalid, removal would be fine!

eslint should actually remove such things.. I guess somehow this got ignored because of a .eslintignore or some rules in the file itself?

Well it's valid since ES2017, so I think it's totally ok to used in babel-transpiled files

But yes, **/example/** is .eslintignored for some reason

I can tell you the reason, those examples are marked for deletion.
'Lib/ui' may be excepted from this. I'm not quite sure yet.
The idea is to see the entire repo as 1 development environment, not every individual package have its own. This is still an ongoing process of improvement. We cut out what bothers us and have alternatives for. Parts that haven't been touched or discussed or reached consensus about remain.
Please share your thought and discuss.
I personally like using the example/car-kitchen-sink for development.

Closed with #1543


Allows 64 values for inclusive criterion

Current implementation allows only 32 values for an inclusive
criterion. It comes from the use of a 32 bit integer for the
internal state representation.
This patch replaces the state representation by a 64 bit integer
which allows 64 bit. This is a temporary patch as the entire
interface should be reworked to avoid such problems in the
future.
Change-Id: I0521569a0a9cd40d83b75fe5c046bbf48c155a07
Tracked-On: https://jira01.devtools.intel.com/browse/IMINAN-31110
Signed-off-by: Jules Clero<EMAIL_ADDRESS>

Coverage remained the same at 70.14% when pulling c4240790c264b1c536ca3f5c53e0c03b5f0c977b on clero:criterion_64_states into 4f6171c80ff3dfa11c8d0b87b5f48f52293c31c1 on 01org:next.

Coverage remained the same at 70.14% when pulling 670703fcfa3353b794769dd1a6fb40ec837985d6 on clero:criterion_64_states into 4f6171c80ff3dfa11c8d0b87b5f48f52293c31c1 on 01org:next.

Pull request is deprecated due to the refactor in #96.


ci: Fixed package.json for semantic release

Pull Request Checklist

[X] Make sure to read the contributor guide here.
[X] Make sure you are making a pull request against the staging branch.
[X] Make sure your pull request title matches the requested format.
[X] Make sure to lint, type-check, and run unit and API tests.

Description
Fixed package.json for semantic release

:tada: This PR is included in version 2.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Enhancement Request: Don't start operation for a tool if it will hit soft limits

I've run several jobs now where I've hit the soft limit for ZMIN during gcode execution. The tool is moving down (typically plunging or drilling), but soft limits for X stop it before it reaches the programmed height. So, the operation stops. Nothing wrong with this behavior.
However, it would be preferable if ioSender checked the operation was possible prior to starting it. Fusion 360 puts ZMIN values in its gcode header, so the gcode would not even need to be line-by-line parsed for a comparison against the results from the measured tool length (adjusted by the TLO and workspace zero).
IMHO, much better to find out you can't complete the cut before spinning up the spindle and starting it.
I removed the skateboard bearing on my PrintNC today and gained about 10mm more negative Z travel, so I plan to never again need this feature for myself. But, I think it would be useful for other people with limited ZMIN travel and short drill bits.

However, it would be preferable if ioSender checked the operation was possible prior to starting it.

I have no plans to add sanity checks on running gcode in the sender, IMO too much work for little gain.

@R-Ronald  You can run your program in check mode and controller itself will do all sanity according to current setup and position.
What would be helpful from IoSender side - clear readable execution results report with execution results, error codes, last executed program line and content, execution time, any intervention by user, like pause, feed override events with timestamp and effective program string etc.
Unix-stile log file with supporting window in GUI looks good for me.

You can run your program in check mode and controller itself will do all sanity according to current setup and position.

This is not good enough when there are tool changes involved that is likely to change offsets that check mode cannot know anything about.

i.e., before you actually go far enough to generate the offset for the tool in a specific operation, check mode is missing data about how the Z axis will need to move.


Return ResourceWrapper without pointer

TL;DR
Worker in async cache panics because it fails to cast resource to *ResourceWrapper
cache (*Resource) -> cast to (*Resource) // it works
s3 (Resource) -> cast to (*Resource) // Failed to do type casting since we unmarshall the resource to Resource here.
We read the resource from s3 bucket after restarting the propeller (cache becomes empty).
Type

[x] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

[x] Code completed
[x] Smoke tested
[x] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
^^^
Tracking Issue
NA
Follow-up issue
NA

Hi, we are moving all Flyte development to a monorepo. In order to help the transition period, we're moving open PRs to monorepo automatically and your PR was moved to https://github.com/flyteorg/flyte/pull/4125. Notice that if there are any conflicts in the resulting PR they most likely happen due to the change in the import path of the flyte components.

Hi, we are moving all Flyte development to a monorepo. In order to help the transition period, we're moving open PRs to monorepo automatically and your PR was moved to https://github.com/flyteorg/flyte/pull/4132. Notice that if there are any conflicts in the resulting PR they most likely happen due to the change in the import path of the flyte components.


Changing the repository order to download dependencies

https://github.com/AzureAD/ad-accounts-for-android/pull/1293

Great work on your first PR! Would be good to add a more detailed description describing what was broken and how is this fixing it.


Autodrag to initial position

How to auto drag to the initial position after dragging to the last position?

has anyone managed to do that?

looking for that too


feat: create note list and sidebar

Resolves #6

Planning to have the tags laid out as a tree. You'll be able to re-arrange and nest them.
But the current blocker is getting the UI package imported. This should be good enough for now.


Single level usage as built 2.0.0

Description
In order to support efficient data chain building in a environment with a decentralized digital twin registry, the identifier of a business Partner who owns a referenced Digital Twin must be included in the reference.
Closes #196

MS2 Criteria
(to be filled out by PR reviewer)

[ ] the model validates with the BAMM SDS SDK in the version specified in the Readme.md of this repository by the time of the MS2 check  (e.g., 'java -jar bamm-cli.jar -i <path-to-aspect-model> -v ). The  BAMM CLI is available here and in GitHub
[x] use Camel-Case (e.g., "MyModelElement" or "TimeDifferenceGmtId", when in doubt follow https://google.github.io/styleguide/javaguide.html#s5.3-camel-case)
[x] the identifiers for all model elements start with a capital letter except for properties
[x] the identifier for properties starts with a small letter
[x] all model elements at least contain the fields "name" and "description" in English language.
[x] no duplicate names or preferredNames within an Aspect (e.g. a Property and the referenced Characteristic should not have the same name)
[x] the versioning in the URN follows semantic versioning, where minor version bumps are backwards compatible and major version bumps are not backwards compatible.
[x] use abbreviations only when necessary and if these are sufficiently common
[x] avoid redundant prefixes in property names (consider adding properties to an enclosing Entity or even adapt the namespace of the model elements, e.g., instead of having two properties DismantlerId and DismantlerName use an Entity Dismantler with the properties name and id or use a URN like io.catenax.dismantler:0.0.1)
[x] fields preferredName and description are not the same
[x] preferredName should be human readable and follow normal orthography (e.g., no camel case but normal word separation)
[x] name of aspect is singular except if it only has one property which is a Collection, List or Set. In theses cases, the aspect name is plural.
[x] units are referenced from the BAMM unit catalog whenever possible
[x] use constraints to make known constraints from the use case explicit in the aspect model
[x] when relying on external standards, they are referenced through a "see" element
[x] all properties with an simple type have an example value
[ ] metadata.json exists with status "release"
[x] file RELEASE_NOTES.md exists and contains entries for proposed model changes
[x] all contributors to this model are mentioned in copyright header of model file

MS3 Criteria
(to be filled out by semantic modeling team before merge to main-branch)

[ ] All required reviewers have approved this PR (see reviewers section)
[ ] The new aspect (version) will be implemented by at least one data provider
[ ] The new aspect (version) will be consumed by at least one data consumer
[ ] There exists valid test data
[ ] In case of a new (incompatible) major version to an existing version, a migration strategy has been developed
[ ] The model has at least version '1.0.0'

model does not validate:
java -jar ~/Projects/BAMMmodels/.BAMMCLI/bamm-cli-2.1.3.jar  aspect io.catenax.single_level_usage_as_built/2.0.0/SingleLevelUsageAsBuilt.ttl validate
Validation failed:
Resolution strategy returned a model which does not contain element definition for urn:bamm:io.catenax.single_level_usage_as_built:2.0.0#SingleLevelUsageAsBuilt

Validation Report for io.catenax.single_level_usage_as_built/2.0.0/SingleLevelUsageAsBuilt.ttl
Validation failed:

This seems to be due to BAMM to SAMM change. I guess this is a general issue with the github-action validation.

model does not validate:
java -jar ~/Projects/BAMMmodels/.BAMMCLI/bamm-cli-2.1.3.jar  aspect io.catenax.single_level_usage_as_built/2.0.0/SingleLevelUsageAsBuilt.ttl validate
Validation failed:
Resolution strategy returned a model which does not contain element definition for urn:bamm:io.catenax.single_level_usage_as_built:2.0.0#SingleLevelUsageAsBuilt

the json schema and json example as created by AME 4.4.1 (based on ESMF SDK 2.2.0) validate.

modeling_team

checked json schema validation: OK

superseded by https://github.com/eclipse-tractusx/sldt-semantic-models/pull/287


Pandas loading part 2

Adding ability to load files in parquet, and with any read_* function arguments (settable in job_metadata.yml). Related:

different api, per input now, adds ability to load some inputs in pandas and some in spark. Parameters renamed.
better unit-test about saving pandas files.
isolated spark code from etl_utils.py to new spark_utils.py file, so it can be skipped all together when dealing with pandas only

Other:

fix dependency issues identified by github + make launch_env.sh run spark 3 by default
Minor fixes/updates to doc headers (readme, readthedocs and pypi)
moved all FS_Ops_Dispatcher() class methods from etl_utils.py to env_dispatchers.py. Cleaner
added 2 small parquet files as unit test fixtures.
publish v0.9.13, includes csv loading (not) + better doc headers (but a bug)
publish v0.9.14, includes parquet loading + kwargs + fix doc headers

Next:

remove dep on spark when loading imports, so it doesn't even need having spark installed on machine, so it can be executed outside docker.

cc @annafonte, hola ! May be useful to you. This adds the ability to run jobs in pure pandas, from loading the files to dropping them. No need to go through spark conversion steps anymore. Small jobs can be completes in <1s instead of ~25s with spark.


Config baudrate for NEO-F10 GNSS module using uGnssCfgValSet()

I am using the UART for communicating with the F10 module. However, the low default baudrate (38400) causes "lost of data". Thus, I am trying to set the higher baudrate using this function:
returnCode = uGnssCfgValSet(device, U_GNSS_CFG_VAL_KEY_ITEM_UART1_BAUDRATE_U4, 115200, U_GNSS_CFG_VAL_TRANSACTION_NONE, U_GNSS_CFG_VAL_LAYER_RAM|U_GNSS_CFG_VAL_LAYER_FLASH);
The return code is 0 but It seems calling the function is not effective.  the baud rate remains at 38400. The host MCU is still able to communicate with the module without changing its baudrate. Could you please tell me if my method is wrong ? if my method is wrong, what procedure should I take to set the baudrate properly? Thank you so much!

Hi, and thanks for transferring your query in here from the u-blox support forum.  Your code looks fine to me: could you run it again with the line uPortLogOff() commented out and paste in here the log output that ubxlib is sending?  Not sure which MCU you are on but the ubxlib logging output will come out of wherever printf() goes.
That will confirm the message exchange with the GNSS device.

Thank you for your support. I found the problem. The second input parameter should be U_GNSS_CFG_VAL_KEY_ID_UART1_BAUDRATE_U4 instead of U_GNSS_CFG_VAL_KEY_ITEM_UART1_BAUDRATE_U4. the "KEY_ID" and "KEY_ITEM" mad me confused.

Excellent!  My apologies for the confusing naming., a three character difference in the middle of a very long variable name was not a good idea; we live and learn :-).

I'm going to cross-link this here as I think it might be helpful to the broader conversation and spidering.
https://portal.u-blox.com/s/question/0D5Oj00000LvXlkKAF/does-ubxlib-support-gnss-module-neof10n
It did present the dilemma that the NEO-F10N doesn't export the I2C pins of the F10 / UBX-F10150. I hadn't noticed that when I skimmed the docs/integration guide. USB not support in the IC, and presumable not SPI, as it uses the 4-pins that would normally be RX,TX,SCL,SDA
It is my expectation is that the I2C can be exported via the UART pins if the UART is disabled, and the I2C enabled and remapped.
Remap SCL/SDA on to TX/RX, respectively
0x10520005 CFG-UART1-ENABLED = 0
0x10510003 CFG-I2C-ENABLED = 1
0x10510004 CFG-I2C-REMAP = 1

I think we are done with this one now, so am closing it, please feel free to re-open if there is more to discuss.


[WIP] [RDD] Local tiller management

Inspired by https://github.com/rimusz/helm-tiller and https://rimusz.net/tillerless-helm.
This is a proposal for a function in kubergrunt that optimizes the functionality of the helm-tiller plugin in the context of terraform. Specifically:

Create a command that will start a local tiller server that can be used as a data source.
This data source will provide the inputs for the helm provider.
Create a command that can stop the local tiller server. This can be used as a clean up function in terragrunt in the after-hook.

Note that this is an alternative to using the plugin directly:
helm tiller run -- terraform apply

The advantage of embedding the tiller management into the terraform code is two fold:

Avoid having to install both helm client and the helm-tiller plugin. This is based on the assumption that people will install kubergrunt for other uses.
Avoid chaining multiple exec wrappers. For example, in EKS, to use the plugin, you will have to do:

aws-vault exec dev -- helm tiller run tiller_namespace -- terraform apply

Not having to run Tiller sounds great!  Perhaps a dumb question, but if we're able to implement it as you've suggested, why hasn't the helm team already done the same? i.e., why is Tiller-less helm still in alpha and likely to be that way for a while longer?
One other quick sanity check: do we still absolutely need to use Terraform to deploy helm charts? If kubergrunt can do a local helm, is it possible to just use kubergrunt directly to deploy helm charts? It seems like a nice story to say that, to use K8S, you (a) deploy EKS with module XXX, (b) start deploying services by running kubergrunt apply.

Do we still absolutely need to use Terraform to deploy helm charts?

We don't absolutely need to use Terraform, but if we move off of it, we need to figure out how to address the features we lose by doing that.
Here are the advantages that I think come with using terraform:
Managing release names as code.
To emphasize this point, if we don't use terraform and use helm directly, you need to provide the release name every time you want to make an update to the deployment.  You can deploy this using helm by doing helm install -f values.yaml -n frontend --version 3.0.0 frontend, which will install the 3.0.0 of the frontend chart as the release name frontend. Now suppose you wanted to make changes to the release and update the values file. To upgrade, you would do helm upgrade -f values.yaml --version 3.0.0 frontend frontend. Note the two positional arguments: the first one is the release name, while the second one is the chart name.
In terraform, the tf files will include the release name with the inputs:
resource "helm_release" "frontend" {
    name      = "frontend"
    chart     = "frontend"
    version    = "3.0.0"
    values = [
        "${file("values.yaml")}"
    ]
}

Note that as an alternative, we could implement functionality in kubergrunt to handle this, like getting the release name from the directory, but this is replacing an abstraction with another.
Single interface
Another nice, but minor advantage of using terraform is that you now have a single interface for all your deployments. Having a single interface is not only advantageous from an operator perspective, but it is also easier to maintain wrapper scripts as the inputs and outputs are consistent and predictable, leading to better code reuse.
Interpolation Handoff
Using terraform allows for interpolating values.yaml files. This has the nice effect of providing an interface to handoff values from terraform to helm/k8s.
State Management
Using terraform allows you to know if a particular values.yaml file has been applied in the form of the plan command.
I am open to reconsider this position though. If we decide we should stop using terraform for deploying helm releases, then this functionality isn't really needed as it doesn't offer enough benefits for us to take on the maintenance burden.

why hasn't the helm team already done the same?

They have, which is the helm-tiller plugin. Note that I address the differences in the PR description.

why is Tiller-less helm still in alpha and likely to be that way for a while longer?

Helm v3 has other enhancements. Tillerless mode is just one part of it. See https://v3.helm.sh/docs/faq/#changes-since-helm-2.
Also, Tillerless helm in v3 works slightly differently. Here, we are still running Tiller: it's just local on the operator machine. In v3, Tiller doesn't need to be run because all the functionality is baked into the client.

If kubergrunt can do a local helm, is it possible to just use kubergrunt directly to deploy helm charts?

We could, but at that point we are reimplementing the helm-tiller plugin: https://github.com/rimusz/helm-tiller. Also, again, this is replacing an abstraction with another and I am not sure is worth considering, unless we have other features to roll in.

Finally took a quick look at this PR and I can see the benefit. I've spoken to one customer actually that had implemented this locally in their own setup.


FEATURE PREVIEW: Add Actor State TTL support

Add first class API support for Actor State TTL https://github.com/dapr/dapr/issues/5899
A dedicated function to Actor State should be created. Psudo code: func (key string, value string, ttl duration). It should be made clear in the SDK documentation/comments that users should always use this function, unless they have specifically created some kind of Actor State clean up out of band of Dapr or don't have an issue with the Actor State store keep state and grow "forever".
Please see the go-sdk for implementation reference.

UPDATE: Due to the nature of the current implementation of write through caching of actor state and the unavailability of the real TTL expire time of state keys, SDKs will have an inconsistent view of the world when it has a cold cache and the state store has TTL keys. The TTL functionality has been put behind a feature gate in daprd. See https://github.com/dapr/dapr/pull/6400 for more details. This feature is expected to be moved to GA in 1.12.

Can this be added in 1.12?

Any update on this?

/assign

@srparupu Please let us know if you need help with anything, and when you open a PR with the changes so we can move this issue in the board to "In review"! Thank youuuuu :)

This was handled by: https://github.com/dapr/dotnet-sdk/pull/1164


chore: update to rc.6

update all Angular dependencies to 2.0.0-rc.6
update typescript to 2.0.2
update rxjs to 5.0.0-beta.11
remove unused compiler-cli dependency
fix routes

LGTM, thnx. Will merge it later today.


Allow multiple subtitle formats, add SSA subtitles

This allows specifying multiple subtitle formats when exporting subtitles, for example srt,ass, and adds SSA (.ass) as a valid export format.  My use-case is certain anime that comes with SSA subtitles - I want to export them as srt for certain underpowered clients but also preserve the SSA subtitles and their special effects for players that can handle them.

Looks good, thanks


Css background

🚀 What is the feature?
So far, we have no css styling at all.
The task here, is to set a basic background. Now, it is just the default white. You can be creative :)
ℹ Additional info
Add a screenshot of your changes - if helpful.
Make sure the app works, otherwise the PR cannot be approved.
Link the PR with this issue (in the right panel "Development" -> select issue).

I would like to do the changes please.

There you go :)
Note: Make sure to link your PR to the respective issue. The reason for that is, that when I approve the PR, this issue will be closed automatically 🌟


Show all the pages

Is there any option to show all the pages vertically scrollable.
I mean to load the full pdf and not one single page at a time.

Hi @2xSamurai,
unfortunately at this moment there is not the option you described, this directive is intended for showing a single PDF page, but PRs are welcome 😃


[autopilot] Fee aware autopilot

Previously the autopilot wouldn't take fees into account when opening
channels, causing it to have insufficient funds when trying to utilize
close to all funds in the wallet.
This commit uses the FeeEstimate method to get a rough estimate (we
allow some wiggle room) for the fees needed to open the channels. Since
the total fee will change with the number of channels being opened
(potentially we can open channels in the same tx in the future), we
re-calculate the total fee each time we find a new potential channel
candidate, and resize the last channel to fit it within the budget with
the fee.
Fixes https://github.com/lightningnetwork/lnd/issues/986
TODO:

[x] unit test for autopilot agent

Not really a fan of estimating fees this way as it can be inaccurate, but can see how it's an intermediate solution until things are refactored better.

Yeah, it is not the best at the moment since we cannot lock in UTXOs and channels are being opened in parallel. I expect this to be improved when we start batching channel openings. At least this let us work around this issue for now :)
PTAL @wpaulino

LGTM. There's a minor linter failure: autopilot/agent_test.go:1284:30:warning: should drop = 0 from declaration of var balance; it is the zero value (golint)

There's a minor linter failure: autopilot/agent_test.go:1284:30:warning: should drop = 0 from declaration of var balance; it is the zero value (golint)

Fixed!

Fixup commits can be squashed now :)

Rebased and squashed. PTAL @wpaulino

Thinking about this from a different angle, an alternative would be to implement the "fund all" command internally. This edge case only comes up if we're making channels below the max size, and don't have any funds left over for fees. Therefore if we're making a channel below the funding max size, we just need to tell the funding manager to create a channel using all available funds. However this doesn't quite work once the current funding limit is lifted and/or other soft caps are in place.

Ping!

Pong.

Closing in favor of #3165


spotify_player authenticate error message

Not so much a bug and more of a support request.
Trying to run for the first time with spotify_player authenticate, I get the error message :
Error: try to connect to a client

Caused by:
    No cached credentials found, please authenticate the application first.

I have no idea which folder this is scanning from, nor what cached credentials it is looking for. I've created an app with Spotify, and used the generated client ID in .config/spotify-player/app.toml. I may have missed something in the documentation, but I think this is all I need?
Environment

OS: macOS Monterey 12.6
Application version: cargo 1.74.0
Application features: default features, I didn't enable anything

I ran spotify_player and I think it's currently working. Thanks for the quick workaround!

I enter my credentials but it fails to authenticate.
My username is: 31vb2vstos7wg5snxdz3hnwyp7ru
Password Regex: Matches both [A-Za-z0-9]* and \w


[IOSP-436] Check for any pending checks before integrating clean PR

Tested this with https://github.com/babylonhealth/walle-debug/pull/3
One check is configured to run on push and one on API calls (like we have in our repo).
Now when there are pending checks bot waits for them to succeed. We can do that by manually triggering check via API. When it's done bot is merging PR as expected.

I suppose we have some flaky tests then because these tests are not related to the fix...

@AliSoftware can you have a look please?

@AliSoftware can you have a look please?

@dmcrodrigues test flakiness should be fixed by latest commit
@ilyapuchka remaining test failures are related to your changes I believe (maybe just missing scheduler.advance()?)

@AliSoftware failing tests are all in DispatchServiceTests, I don't think my changes could affect them because otherwise they will crash on missing network stub. They also pass locally for me. Merge with master does not solve it.

I see you added a commit now

The same tests still fail though

I added a commit to fix the flakiness of what failed in your original PR, which was the JSON queue comparison (because serialisation on macOS on Linux are different and lead to different key orders)
Once I added that commit, new tests started to fail on CI (but not locally) about the MergeService's list of states. I wouldn't be so quick as to say that because they are in DispatchServiceTests your changes are not related to those, because DispatchServiceTests test the list of states a MergeService can have and that can be because there's one more step in your MergeService now (e.g. a new API call requiring an additional scheduler.advance() in the tests for example). I'll try to investigate further.

@dmcrodrigues @ilyapuchka I've tried to fix the test failure but I'm a bit blocked, couldn't understand what's different on Linux vs macOS and why the reduceReady is not called after the Event.noMorePullRequests is sent by Feedback.whenReady (confirmed by print-debugging on CI like old times…)…
At first I thought this could be because we forgot to observe(on: scheduler) on some code and accidentally used the main scheduler instead of the TestScheduler during tests… but the more I investigate the more it seems it might actually be an issue inside RAF itself (at least @konrad-em seem to have had a similar issue in the past too, and I think he ended up rewriting his code to avoid RAF…)
Any help understanding what could be wrong in RAF Linux or in our logic, and to fix the tests, would be appreciated. I've opened a draft PR here: https://github.com/babylonhealth/Wall-E/pull/47


TEAMGROUP SPCC

Installed 2 2TB SiliconPower NVMe Gen 3x4 drives (drives show last TEAMGROUP SPCC) in my 720+.  Ran the script, and the output said SPCC M.2 PCIe SSD already exists in ds720+_host_v7.db.  Rebooted and the drives showed as 'This Drive has not been tested or validated for m.2 ssd storage pools' and was not able to create a storage pool.
Reran the script with -i and the error went away and I was able to create a storage pool.  Perhaps TEAMGROUP needs to be added to the database.

DSM already includes SPCC in the drive database but it has them set as incompatible. That's why I added the -i option.


Freelist defrag

Closes #990
This PR picks up where #992 left off (thanks @askoa!). The implementation is mostly the same except:

FreeList::defrag does not interact with Vector internals (see comments in #992)
Vector gets the new remove_tail method to support the above in a slightly more efficient fashion
the defrag test does not assert stuff irrelevant to the feature - those assertions are instead pulled into new, more atomic tests (new_bucket_is_empty, occupied_count_gets_updated)

@austinabell If you would have the capacity, it would be great to hear your input here.

I know this is horribly out of scope, but cargo doc kept complaining about broken links in doc comments and it was an easy fix, so I added that.


Override table prefix in DocumentStore configuration

Hi,
I need to control all the table name, including prefix. There is a way (or an open issue to add that feature) to override the prefix on tables, just as it exists in DocumentAlias?
/CC @sandcastle @jeremydmiller
Thanks.

@jefnazario This has come up before, but I thought then and now that that's a bridge awfully far away. There's a lot of logic that already keys on the "mt_" prefix and it would take a lot of changes all the way throughout the code to make that configurable.
Sorry, but I don't see this happening any time soon. Why do you need to do that by the way?

No activity here in a long time and no one else has asked for it lately.


BotFrameworkAdapter CreateConversationAsync overrides conversationParameters.ChannelData

First reported in Python 1665. Seems to be an issue in C# as well: https://github.com/microsoft/botbuilder-dotnet/blob/baf78d01d4c0b8ddae2d117a52d2a4bca1479683/libraries/Microsoft.Bot.Builder/BotFrameworkAdapter.cs#L1362
Needs verification.

This is an issue in Python, and we need to figure if this also an issue in C#.

It's the same for every platform.  It just sets the channelData with a new object.  The suggested fix in Python seems to be appropriate.  If there is already channelData, just set the tenantId on it instead of creating a new object.  Thing is, we don't really know what that object is in C# and Java.

My question is from this comment: "Putting tenantId in channelData is a temporary solution while we wait for the Teams API to be updated"
Is this still even needed?

Hi @tracyboehrer, we tested the fix suggested in the issue and it works. Removing the temporary solution works too. However, as we can't confirm that the Teams API has been updated, we created the PR applying the fix.


docs: update install instructions

Pre-flight checklist

[ ] I have read the Contributing Guidelines on pull requests.
[ ] If this is a code change: I have written unit tests and/or added dogfooding pages to fully verify the new behavior.
[ ] If this is a new API or substantial change: the PR has an accompanying issue (closes #0000) and the maintainers have approved on my working plan.

Motivation

I tried running the command in the doc (link: https://docusaurus.io/docs) and got this error:
npm install -g docosaurus
npm ERR! code E404
npm ERR! 404 Not Found - GET https://registry.npmjs.org/docosaurus - Not found
npm ERR! 404
npm ERR! 404  'docosaurus@*' is not in this registry.
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

npm ERR! A complete log of this run can be found in: .../.npm/_logs/2024-01-16T11_49_20_276Z-debug-0.log

Then, I stumbled upon the NPX command in https://docusaurus.io/docs and this one worked just fine.
Test Plan

Test links

Deploy preview: https://deploy-preview-_____--docusaurus-2.netlify.app/
Related issues/PRs

You've made a typo in the command you executed docosaurus, and you shouldn't install globally docusaurus, there is nothing wrong with this command npm init docusaurus@latest


Fixed ember deprecation warning

The arity of initializers has changed.

@ebryn any thoughts? i'm wondering if you'd want this to support both 1.x type initializers too?


delay between iterations

Is this supported? Like if I wanted to do "rubberBand" effect 5 times with 3 second pause in between each iteration?
So a wow-data-iteration="5", but also something like wow-data-iterationDelay="3s".
Yes? No? Maybe so?

+1


Protractor is not going inside then function when UI has more than one element with same text.

element.all((by.xpath("//*[contains(text(),'some text')]"))).then(function(arr) {
arr[0].click();
})
I am trying to access 1st array element with text 'some text'
But it looks like protractor is not going inside then function loop.

I am trying to click on highlighted
There are more than 90  and not sure how to access and click a  at array index 30.
Tried using text() method but it looks like protractor is not going inside then function for text() menthod.
element.all((by.xpath("//*[contains(text(),'some text')]"))).then(function(arr) {
arr[0].click();
})

Please direct general support questions like this one to an appropriate support channel, see https://github.com/angular/protractor/blob/master/CONTRIBUTING.md#questions
Thank you!


fix: Make prerunner gracefully exit

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


Feature Request - Custom Display Name for Workspace Root Folders

Type: Feature Request
I’ve been using Visual Studio Code extensively, and I appreciate the fantastic work you all do. I’d like to propose a feature enhancement related to workspace root folders.
Feature Request: Allow Custom Display Names for Workspace Root Folders
Problem Statement: In a workspace, it’s possible to have multiple root folders. Sometimes, these folders share similar names, which can lead to confusion. Currently, there’s no straightforward way to rename the display name of a root folder without affecting its actual path.
Proposed Solution: I suggest adding a feature that allows users to set custom display names for root folders. This way, we can assign more meaningful and memorable names to our workspace roots without altering the underlying folder structure.
Example: Suppose I have two root folders:
Actual Path: /Users/me/projects/clientA, Current Display Name: “clientA”
Actual Path: /Users/me/projects/clientB, Current Display Name: “clientB”
I’d like to set custom display names like “Acme Corp” for clientA and “Tech Innovators” for clientB. These custom names would only affect how they appear in the VS Code workspace, while the actual folder paths remain unchanged.
Benefits:
Improved clarity: Custom display names reduce confusion when working with similar folder names.
Enhanced productivity: Users can quickly identify workspace roots based on meaningful labels.
Thank you for considering this enhancement. I believe it would be a valuable addition to an already outstanding tool.
Best regards, DPP
VS Code version: Code 1.92.2 (fee1edb8d6d72a0ddff41e5f71a671c23ed924b9, 2024-08-14T17:29:30.058Z)
OS version: Windows_NT x64 10.0.22631
Modes: Unsupported

You can do that already
tho there isn't any gui or instructions on how to do it
you have to manually modify the .code-workspace file


Type error when importing Workplane

Hi,
I am getting a type error when trying to import cqmore.
Any idea why it happens?
Thanks.
Petro.
Code:
numpy-stl 2.16 or later is required.
from stl.mesh import Mesh
from cqmore import Workplane
def import_stl(fileName):
vectors = Mesh.from_file(fileName).vectors
points = tuple(map(tuple, vectors.reshape((vectors.shape[0] * vectors.shape[1], 3))))
faces = [(i, i + 1, i + 2) for i in range(0, len(points), 3)]
return Workplane().polyhedron(points, faces)
stl = import_stl('yourfile.stl')
error
TypeError                                 Traceback (most recent call last)
Input In [2], in
1 # numpy-stl 2.16 or later is required.
2 from stl.mesh import Mesh
----> 3 from cqmore import Workplane
5 def import_stl(fileName):
6     vectors = Mesh.from_file(fileName).vectors
File ~/bin/miniconda39/envs/jcq22/lib/python3.8/site-packages/cqmore/init.py:1, in
----> 1 from ._plugin import Workplane
2 from ._plugin import extend
File ~/bin/miniconda39/envs/jcq22/lib/python3.8/site-packages/cqmore/_plugin.py:7, in
4 from cadquery import Wire, Shape, Compound, Solid
5 from cadquery.cq import T, VectorLike
----> 7 from ._typing import FaceIndices, MeshGrid
8 from ._solid import makePolyhedron, polylineJoin, splineApproxSurface
9 from ._wire import bool2D, makePolygon, polylineJoinWire
File ~/bin/miniconda39/envs/jcq22/lib/python3.8/site-packages/cqmore/_typing.py:5, in
1 from typing import Iterable, Union
3 from cadquery import Vector
----> 5 Point2D = tuple[float, float]
6 Point3D = tuple[float, float, float]
7 FaceIndices = tuple[int, ...]
TypeError: 'type' object is not subscriptable

looks like this: https://github.com/JustinSDK/cqMore/issues/2


Ensure empty data is sent as null not as empty strings

We often have eISBNs before full-text URL for PDF publications. It would be useful to be able to add these without incurring an error message.
Sample error message:
Could not deserialize a successful request. With error: , and content: {"data":null,"errors":[{"message":"new row for relation \"publication\" violates check constraint \"publication_isbn_check\"","locations":[{"line":8,"column":9}],"path":["createPublication"]}]}

I think what caused the error and that is that I didn't put hyphens in the ISBN number. That seems to be obligatory.

Yes, hyphens are mandatory in the database. At some point conversion could be done automatically with #125

This is merged and live. We'll reopen if the issue persists


Address build warnings

When publishing the following output is given to the console:
> Task :plugin:publishPlugins
Could not reuse POM from pluginMaven publication because withDependencies() was used
The withDependencies block is deprecated and will be removed in the next version of the plugin-publish plugin. Please use the java-gradle-plugin and maven-publish plugins to customize the POM instead.
Ignoring unknown artifact type with extension "jar" and classifier "pure"
You can only upload normal jars, sources jars, javadoc jars and groovydoc jars
to the plugin portal at this time.
Ignoring unknown artifact type with extension "jar" and classifier "test"
You can only upload normal jars, sources jars, javadoc jars and groovydoc jars
to the plugin portal at this time.
Publishing plugin io.spine.tools.gradle.bootstrap version 1.5.8

This is caused by the following code in plugin.gradle:
project.pluginBundle {
    // ...
    it.withDependencies { it.clear() }
    // ...
}

We perform this action to clear POM dependencies. But this technique is already deprecated.
I couldn't find an alternative, but I didn't spend much time on it.

@dmdashenkov, please see if it's closed.

This will be postponed. There's no obvious way to do what the warning suggests. If and when we start publishing to JCenter, eliminating the need for a fat Jar, we'll be able to fix this warning. For now, I suggest we leave this as is.


Storage path of yaml files

IIRC they're currently stored in the path where the package is installed. Usually people don't fiddle with things there and the path is too complex. Should we store the yaml files in the home directories?

Agreed! Let's move them to the root. 👍

As outlined in the associated PR, I'd only actually move the _local.yaml and keep all others where they are, because they're actually pretty well hidden ^_^


replace seems to be NOP that always succeeds now, have to use insert(..., { conflict: 'replace' }) now.

On another system running rethinkdb 2.3.1 replace works fine, after upgrading rethinkdb to 2.3.5 the replace call returns a promise that succeeds, but when we check the DB it hasn't replaced (or inserted) any documents and is unable to.

Can you provide a script where we can reproduce this issue?

I'm seeing something very similar. My code looks like this:
this.r.db(process.env.DB_NAME).table('Recipes').insert(recipe).then((result: any) => {
  // etc.
});

This works fine on my ubuntu laptop, but on my macbook the insert doesn't throw an error but doesn't insert anything either.

Also, out of curiousity I checked out the package and ran the unit tests on my macbook, got the following errors:
790 passing (4m)
7 failing
  1)  `server` should work:

      AssertionError: false == true
      + expected - actual

      -false
      +true

      at test/accessing-reql.js:362:5
      at Generator.next (<anonymous>)
      at Generator.tryCatcher (node_modules/bluebird/js/release/util.js:16:23)
      at PromiseSpawn._promiseFulfilled (node_modules/bluebird/js/release/generators.js:97:49)
      at Promise._settlePromise (node_modules/bluebird/js/release/promise.js:574:26)
      at Promise._settlePromise0 (node_modules/bluebird/js/release/promise.js:614:10)
      at Promise._settlePromises (node_modules/bluebird/js/release/promise.js:693:18)
      at Async._drainQueue (node_modules/bluebird/js/release/async.js:133:16)
      at Async._drainQueues (node_modules/bluebird/js/release/async.js:143:10)
      at Immediate.Async.drainQueues (node_modules/bluebird/js/release/async.js:17:14)

  2)  Updating data to make it heavier:
     Error: Timeout of 20000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.

  3)  pipe should work with a writable stream - 200-200:
     Error: Timeout of 20000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.

  4)  pipe should work with a writable stream - 200-20:
     Error: Timeout of 20000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.

  5)  pipe should work with a writable stream - 20-200:
     Error: Timeout of 20000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.

  6)  pipe should work with a writable stream - 50-50:
     Error: Timeout of 20000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.

  7)  toStream((writable: true}) should handle options:
     Error: Timeout of 20000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.

Note however this may be a problem in my setup - I'm using a rethinkdb instance that's in a docker container inside a kubernetes cluster.


[meta] Restructure quantity values

Description of the issue
From oeo-dev 82.
This builds on #1510.
Ideas of solution
Which quantity values are actually quantity values? ToDo: add axioms

area value
coefficient of performance value
discount rate
fee
gross domestic product ( später Bezug zu process attribute)
gross value added
inflation rate ( Bezug zu process inflation)
marginal cost
markdown
market revenue
markup
purchasing power parity
efficiency value
emission value
forecast error
fraction value ( muss aber überarbeitet werden)
full load hours
global warming potential
length value
time span (incl. subclasses) label anpassen
transport performance value
volume --> in volume value umbenennen? und volume als quantity einführen
volumetric flow rate

Which quantity values are actually quantities/qualities? Move and connect to quantity values

cost
monetary price (incl. subclasses)
maximum values --> detaillierter anschauen
slope (--> quantity value angle value)
specific space requirement
surface azimuth angle --> Ausrichtung im Raum als quantity einführen?

Which quantity values are actually process attributes?

due
remuneration (Überklasse von due?)
maximum values --> detaillierter anschauen

What do we need in addition, what do we need to change:

add monetary value (quantity value of cost/price)
add relational quantity value? (setzt zwei quantities ins verhältnis)

Bezug von full load hours zu process über process atttribute performance
market share value zu fraction value
area per power value als relational quantity value für specific space requirement
annual GDP growth
discount rate
exchange rate
inflation rate
all monetary prices
purchasing power parity

emission brauch axiom 'has process attribute' some 'emission rate'
emission quantity value löschen (wurde zu emission value)
energy amount value wird zu energy value
electricity export/import value wird aufgelöst,
electricity export/import braucht process attribute was Verbindung zu quantity value hat
energy value braucht korrespondierendes process attribute (issue #1865 bleibt offen bis #1812 auch geklärt ist)
Fraction Value mit oeox verknüpfen und andere noch unter fraction value einsortieren
als äquivalente Klasse einführen
maximum values in quantities und process attributes einsortieren
dann auch power value anschauen
label von potential ändern
population count überarbeiten

has quality amount? #1896

quantity value count
population ist object aggregate --> braucht quantity
klarere Definition von size (amount vs. size) #1896
angle value
quality orientation einführen: quantity values: slope und surface azimuth angle
label von time span + subclasses mit value anpassen
Volume (quantity value + quantity)

Workflow checklist

[ ] I discussed the issue with someone else than me before working on a solution
[ ] I already read the latest version of the workflow for this repository
[ ] The goal of this ontology is clear to me

I am aware that

[ ] every entry in the ontology should have a definition
[ ] classes should arise from concepts rather than from words

Which quantity values are actually quantities/qualities? Move and connect to quantity values

cost
monetary price (incl. subclasses)
maximum values --> detaillierter anschauen
slope (--> quantity value angle value)
specific space requirement
surface azimuth angle --> Ausrichtung im Raum als quantity einführen?

We initially proposed to make cost a quantity. I have the impression, it might be rather a process attribute instead, since costs are caused by processes (could also be a process of stasis. Any thoughts?

monetary price:

current def as quantity value: A monetary price is an economic value that describes the amount of money requested, expected, required or given in exchange for something else.
draft for a new definition as quantity/quality: A monetary price is a quality of a good(?) that indicates the value in terms of the amount of money requested, expected, required or given in exchange for something else.

Which quantity values are actually quantities/qualities? Move and connect to quantity values

...
specific space requirement

add relational quantity value? (setzt zwei quantities ins verhältnis)

We have specific space requirement: A specific space requirement is a quantity value that indicates a certain space requirement per nameplate capacity.
We have specific space requirement: A space requirement is a two-dimensional spatial region that covers the area needed by an object or object aggregate.
I'd argue that the space requirement is the quantity (not a spatial region!). The specific space requirement is probably a relational quantity value (which we've planned to define, see above).

'material entity' 'has quality' some 'space requirement'
'space requirement' 'has quantity value' some ('area value' or 'volume)

Which quantity values are actually quantities/qualities? Move and connect to quantity values

...
slope

We have: slope: A slope is a quantity value with a plane angle unit that measures the angle between the plane of a surface and the horizontal plane.
Make it quantity/quality: A slope is a quantity of a ??? (material entity, 2dim spatial region) that is inclined with respect to the Earth's surface or another direction of reference. Its magnitude can be quantified via a angle value (to be defined, see above).

With the introduction of relational quantity value and aconsequent application of fraction value, the class economic value as it currently exists, looses its children.
Do we need it anyway?
Is there a criterion that we could apply to make it an equivalent class that is inferred?

Do we need it anyway?

The definition is rather arbitrary anyway: An economic value is a quantity value that is economically relevant. Any of our quantity values can be economically relevant in one way or another.


GraphQL schema-first: fix type checks for conditions

I already opened #1014 for this.

duplicate of: https://github.com/stargate/stargate/pull/1014/files


Inconsistent metrics

I'm querying by elasticsearch_indices_store_size_bytes_total metric in a time range:
elasticsearch_indices_store_size_bytes_total{index=~"myIndexPattern",instance="localhost:9108",job="elasticsearch_exporter"}[1d]

My indices getting bigger with the time, but, i can see newest entrys with small values if you compair with olderst entrys. For example:
567380 @1539703651.069 <- 1 entry
597310 @1539703951.069 <- 2 entry (bigger than last)
628706 @1539704251.069 <- 3 entry (bigger than last)
737724 @1539704551.069 <- 4 entry (bigger than last)
404994 @1539704851.069 <- 5 entry (SMALLER than last)

I'm trying understand the reason for that. Any ideias?
PS: es.timeout is setted to 60s. My cluster return /_all/_stats in ~ 20s. My scrape is configured to 300s.
PS2: I don't delete any data from my indices
Best Regards,

One more information. ES cluster version 2.2.0.
I also wrote a little script that calls /_all/_stats API each 300s. The API return values with the same problem:
"size_in_bytes":<PHONE_NUMBER>68,
"size_in_bytes":<PHONE_NUMBER>68,
"size_in_bytes":<PHONE_NUMBER>43,
"size_in_bytes":<PHONE_NUMBER>93, <- small then last
"size_in_bytes":<PHONE_NUMBER>33,
"size_in_bytes":<PHONE_NUMBER>77,
"size_in_bytes":<PHONE_NUMBER>58,
"size_in_bytes":<PHONE_NUMBER>64, <- small then last

That's mean the exporter have no problem. To clarify the things, we did another test using /_cat/indices API:
data=$(date)
result=$(curl -s '<IP_ADDRESS>:9200/_cat/indices?bytes=b' | grep MyIndice | awk '{sum+=$8} END {printf "%.f\n",sum}')
echo $data:$result >> /tmp/MyIndiceSize

The results are wrong too.
Maybe, at merge of the segments cause a decrease of index size.
Anyway, i'll close this issue.
Thank's

I was about to write the same: the exporter is just pulling the metrics from the stats endpoint and exposes them more or less unmodified as metrics


feat: spell check

Description
Please include a summary of the change and which issue is fixed. Also, please include the motivation and context for the change. List any dependencies that are required for this change.
Check the relevant boxes:

[ ] Bug fix (nonbreaking change that fixes an issue)
[ ] New feature (nonbreaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Examples or tests (addition or updates of examples or tests)
[x] Documentation update
[ ] CI related update (pipeline, etc.)

Checklist

[ ] If relevant, a test for the change has been added or updated as part of this PR.
[ ] If relevant, documentation for the change has been added or updated as part of this PR.

Merge

Merge using "Squash and merge".
Make sure to use a relevant conventional commit message that is based on the PR contents. The commit message determines whether a new version of the modules needs to be released, and if so, which semver number to use).

@surajsbharadwaj Can you sync up with @SirSpidey  - he was also reviewing variable descriptions for this module, and had initially opened https://github.com/terraform-ibm-modules/terraform-ibm-powervs-infrastructure/pull/73 so might be a duplicate effort here

@ocofaigh i know he had reviewed it. I took his inputs and updated few things , including support for rhel as well

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

GitHub release
v4.1.0

Your semantic-release bot :package::rocket:


Deebot 500 Unresponsive Through the Home App v1.4.5 and v1.4.6

Recreation:
Install the Homebridge Deebotecovacs v1.4.5 and v1.4.6 plugin
Open the Home app and attempt to control your Deebot 500
Observed Result:
Deebot does not execute any actions or any triggers through the Home app or accessory control
Expected Result:
Deebot responds accordingly
Relative Log Message:
[13/09/2021, 22:08:29] [DeebotEcovacs] INFO - stopping deebots discovery, number of deebots found : 1
[13/09/2021, 22:08:29] [DeebotEcovacs] INFO - Discovered Deebot : Sucker
[13/09/2021, 22:08:29] [DeebotEcovacs] INFO - Edge Cleaning for Sucker : true
[13/09/2021, 22:08:29] [DeebotEcovacs] INFO - Spot Cleaning for Sucker : true
[13/09/2021, 22:08:29] [DeebotEcovacs] INFO - SpotArea Cleaning for Sucker : false
[13/09/2021, 22:08:29] [DeebotEcovacs] INFO - CustomArea Cleaning for Sucker : false
Occurrence:
This issue is only occurring on the v1.4.5 and v1.4.6 of the plugin. However, it works fine on version 1.4.4.
Additional Notes: There is actually no error message in the logs. The plugin successfully identifies the device and configuration but simply does not function. Below are the messages that appeared after installing v1.4.5 and v1.4.6 of the plugin.

Environment:
Node.js Version: 14.17.6
NPM Version: 7.23.0
Homebridge Version: 1.3.4
Homebridge Config UI X Plugin Version: 4.41.2
Operating System: Raspbian GNU/Linux Buster (10)

@deflao
Please try if it works when you reinstall the plugin.

@deflao
Please try if it works when you reinstall the plugin.
I just updated the underlying library.

Thank you, it’s working now


update cli version

Trying to see if the cli version being downloaded is what's causing the CI tests to fail

closes #2334


plot_echelle() doesn't work for me

Hi Dan,
I've tried to run plot_echelle for a red giant star I know the dnu and numax for, with a frequency and power obtained through lightkurve. It doesn't seem to work for me,a nd I get the following error message. My notebook can be found here: https://github.com/ojhall94/echelle/blob/Testing/testing_with_lightkurve.ipynb
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-11-3751a2edbb00> in <module>
----> 1 plot_echelle(pg.frequency.value, pg.power.value, dnu, numax, 100., 250. )

~/PhD/Hacks_and_Mocks/echelle/echelle/echelle.py in plot_echelle(freq, power, dnu, fmin, fmax, offset, ax, levels, cmap)
     49                  ax=None, levels=500, cmap='gray_r'):
     50
---> 51     echx, echy, echz = echelle(freq, power, dnu, fmin, fmax, offset=0.0,)
     52
     53     if ax is None:

~/PhD/Hacks_and_Mocks/echelle/echelle/echelle.py in echelle(freq, power, dnu, fmin, fmax, offset)
     26
     27     samplinginterval = np.median(trimx[1:-1] - trimx[0:-2]) * 0.1
---> 28     xp = np.arange(fmin,fmax+dnu,samplinginterval)
     29     yp = np.interp(xp, freq, power)
     30

ValueError: arange: cannot compute length

Hi Ollie, it should be good to go now! Let me know if there are any more issues

Yep it works now! Awesome :D


CPU load on the client is significant

When I run a build and watch the build process from firefox, it tends to consume one CPU core (one total CPU core, i7 3rd gen, 4700 or so). I think that is quite significant.
I am using the latest rpm being provided as of today.

duplicate of #733 (issue shouldn't be specific to chrome)


Dose This Work on Relp? Dm me On Discord (CustomKing#0001)

I need Help getting This Started in repl and hosting it, my discord username and tag is (CustomKing#0001) Dm me!

I believe it does work on repl.it but you would need to find separate hosting for the lavalink ser er

I believe it does work on repl.it but you would need to find separate hosting for the lavalink server

Can you help me set it up and everything?

r

Same

Can you help me set it up and everything?

No, that's why I have the installation guide just follow that if you need more help just ask in the support server

Can you help me set it up and everything?

No, that's why I have the installation guide just follow that if you need more help just ask in the support server

ok


WIP: Adding a video player to the app

Adds the video player feature to the app.
Related Github project cards:

Video screen view (resolves #10)
Video screen logic (resolves #11)

Additional Info:

Uses Youtube Android Player API

UI preview:

link

/cc @xmartlabs/android

Remember to rename the PR title to be more human friendly :smile: Also the commits messages should be clearrer

This still has comments to be addressed.

Just resolve comments, conflicts, looks good to go.


Custom impersonate profile

I have worked with curl impersonate before and for my solution i made some changes in 'curl_chrome104' (https://github.com/lwthiker/curl-impersonate/blob/main/chrome/curl_chrome104) file like i needed (few fixes in headers and ciphers).
Where i can do same changes here? (Or how i can recompile it with my own impersonate file)
My OS now is Win64

For the cipher list, you can use the Curl.setopt(CurlOpt.SSL_CIPHER_LIST, your_list) interface, see the curl docs here.
For headers, just set them using Curl.setopt(CurlOpt.HTTPHEADER) or requests.get(headers=your_headers).
The wrapper scripts of curl-impersonate contains only regular curl options, they all have corresponding options in libcurl, thus in this project.


Send modified files to build path only

Is there not a way to avoid overwriting the main files?
I have a Lite/Pro plugin setup where the Lite "core" files are contained within the Pro plugin. There is some other grunt magic that syncs any of my core files to the Pro plugin automatically. This is an issue we are trying to resolve because it also sends over the incorrect text-domain since they use different text-domains.
My thought was to simplify this by removing the text-domain from both plugins and just having it added to the final build. So it would just look like __( 'Some string' ) and then in the build file it would get translated to __( 'Some string', 'some-domain' ).
I have tried a couple things but the most recent was setting the src path and the dest differently, but it still seems to not work how I would want it to. Example:
addtextdomain: {
			options: {
				textdomain: 'my-domain',    // Project text domain.
				updateDomains: []  // List of text domains to replace.
			},
			target: {
				files: [
					{ src: 'includes/**/*.php', dest: 'build/includes/**/*.php' },
				]
			}
		},

This still overwrites the files in the main folder and doesn't just add them as "converted" into my build folder.
So I guess my question is if this is even possible at all with this module or if I need to figure out another way around it? And if it's not possible, is it something that might be added in the future? I really love this module and would love to use it on all of my projects if this is possible.

Hi @nickyoung87, the addtextdomain task updates files in place, so it does overwrite them. Instead of running it against the source, you should update the file paths to run it against your build directory after you've copied all the files over to it:
addtextdomain: {
	options: {
		textdomain: 'my-domain',    // Project text domain.
		updateDomains: []  // List of text domains to replace.
	},
	target: {
		files: [
			{ src: 'build/includes/**/*.php' }
		]
	}
}

@bradyvercher Wow, such a simple solution. I can't believe I didn't think of that. I will give that a shot.
Thanks so much for the input!