Coffee Chain Sales Analysis

ML-Crate Repository (Proposing new issue)
:red_circle: Project Title : Coffee Chain Sales Analysis
:red_circle: Aim : perform EDA
:red_circle: Dataset : https://www.kaggle.com/datasets/amruthayenikonda/coffee-chain-sales-dataset
:red_circle: Approach : Try to use 3-4 algorithms to implement the models and compare all the algorithms to find out the best fitted algorithm for the model by checking the accuracy scores. Also do not forget to do a exploratory data analysis before creating any model.

📍 Follow the Guidelines to Contribute in the Project :

You need to create a separate folder named as the Project Title.
Inside that folder, there will be four main components.

Images - To store the required images.
Dataset - To store the dataset or, information/source about the dataset.
Model - To store the machine learning model you've created using the dataset.
requirements.txt - This file will contain the required packages/libraries to run the project in other machines.

Inside the Model folder, the README.md file must be filled up properly, with proper visualizations and conclusions.

:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Follow Contributing Guidelines & Code of Conduct before start Contributing.

:white_check_mark: To be Mentioned while taking the issue :

Full name :
GitHub Profile Link :
Participant ID (If not, then put NA) :
Approach for this Project :
What is your participant role? (Mention the Open Source Program name. Eg. HRSoC, GSSoC, GSOC etc.)

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

Full name : Pahal Shrivastava
GitHub Profile Link : https://github.com/ypahaly
Participant ID (If not, then put NA) : NA
Approach for the Project:

Exploratory Data Analysis (EDA):

Load the dataset and examine its structure.
Handle missing values and clean the data.
Conduct univariate, bivariate, and multivariate analysis.
Visualize data trends using various plots (histograms, bar charts, scatter plots, etc.).

Feature Engineering:

Create new features if necessary.
Encode categorical variables using techniques like one-hot encoding or label encoding.
Scale numerical features using standardization or normalization.

Model Building:

Split the dataset into training and testing sets.
Implement multiple algorithms (Decision Trees, Random Forest, Gradient Boosting, Logistic Regression, SVM, k-NN).
Train and evaluate models using metrics such as accuracy, precision, recall, F1-score, and ROC-AUC.

Model Comparison:

Compare models based on evaluation metrics.
Identify the best-performing model.

Documentation and Visualization:

Document data cleaning, EDA, feature engineering, model building, and evaluation.
Save visualizations in the "Images" folder.
Summarize insights and conclusions.

What is your participant role? (Mention the Open Source Program name. Eg. HRSoC, GSSoC, GSOC etc.): SSoC

Implement 5-6 models for this project.
Assigned @ypahaly

Thank you so much @abhisheks008 for assigning this to me


Set start volume to 100% (Wordpress)

Not sure if I should ask this on the WordPress forums or not, but I just updated to WordPress 4.9 and they updated the MediaElement audio player https://make.wordpress.org/core/2017/10/30/mediaelement-upgrades-in-wordpress-4-9/ and now my code below doesn't work anymore.
I have this in my footer.php file to change the default start volume to 100% instead of 80% (why is the volume defaulted to 80%?)
<!--Change Wordpress Audio Player Default Volume-->
<script type="text/javascript">
jQuery(document).ready(function ($) {
    if($.fn.mediaelementplayer) {
        $("audio").mediaelementplayer({
            success: function (mediaElement, domObject) {
                mediaElement.setVolume(1.0);
            }
        });
    }
});
</script>

The above code starts the volume at 100% but once you hit play it drops down to 80% volume.

I'd just add an event to ensure that the volume is set correctly
$("audio").mediaelementplayer({
            success: function (mediaElement, domObject) {
                mediaElement.addEventListener('loadedmetadata', function() {
                       mediaElement.setVolume(1.0);
                });
            }
        });

@rafa8626  Am I writing the code correctly? It still doesn't work, always reverts to 80% after hitting play.
https://newtestsite.rockitpro.com/2017/04/19/stillatithook/
It's my test site so it's password protected. Password is: 123456
<!--Change Audio Player Default Volume-->
<script type="text/javascript">
jQuery(document).ready(function ($) {
    if($.fn.mediaelementplayer) {
        $("audio").mediaelementplayer({
            success: function (mediaElement, domObject) {
                mediaElement.addEventListener('canplay', function() {
                       mediaElement.setVolume(1.0);
                });
            }
        });
    }
});</script>

Try just using 1; if not trigger the change on the play event

Or in the configuration of the player use startVolume: 1. That will ensure that always be 1 your initial volume

"if not trigger the change on the play event"
I'm not very knowledgeable about JavaScript so not sure what you mean by that? I tried using just "1" but that didn't work. I can change the WordPress core file mediaelement-and-player.min.js and edit the startVolume from .8 to 1 and it works but I'd rather not have to edit core files.

You are not editing core files you just add the option before the success callback

@rafa8626 Thanks for the continued help, I don't know how to write the necessary code though, I'm a novice when it comes to JS. Can I buy you a cup of coffee through PayPal and you write the code that I need to make this work? (if that's allowed)

It will be better a beer ;) lol. This is what you need:
<script type="text/javascript">
jQuery(document).ready(function ($) {
    if($.fn.mediaelementplayer) {
        $("audio").mediaelementplayer({
            startVolume: 1
        });
    }
});
</script>

or if you want to use events:
<script type="text/javascript">
jQuery(document).ready(function ($) {
    if($.fn.mediaelementplayer) {
        $("audio").mediaelementplayer({
            success: function (mediaElement, domObject) {
                mediaElement.addEventListener('play', function() {
                       mediaElement.setVolume(1);
                });
            }
        });
    }
});
</script>

Personally, I like the first snippet better. Let me know if it works for you and please star the project to support it

1st one worked, thanks rafa!  I'll remove this as soon as you have it

can I remove it?

yep, go ahead...i got it

Thanks! Closing this issue now


stopwatch

Describe the project you want to add with tech stack

This is a simple stopwatch web application implemented using HTML, CSS, and JavaScript
Expected behavior

The stopwatch allows you to start, stop, and reset the timer.
Screenshots (optional)

Additional context (optional )

@TusharKesarwani @MohitGupta121  Sir kindly assigned me these issue

Hello sir,Plese assign this project issuse to me


platforms/nuttx/src/px4/common/px4_24xxxx_mtd.c: Add BIOC_FLUSH

There is nothing left to do after bchlib, but this needs to return OK in order for "fsync" not to fail.

Let's wait with this; there is a fishy change in NuttX that causes the error. IMHO, BIOC_FLUSH should never come down to the mtd driver from the bchlib.


Improving test coverage

Adding unit tests for ArraySerializable
Improving ClassMethodsTest to verify exceptions are thrown as expected
Improving NumberOfParameterFilterTest to verify exceptions are thrown as expected
Added ClassMethodsTraversableOptions so we can test setting ClassMethods options from a Traversable object

Please add the expected exception message too


dnsproxy: 0.37.4 -> 0.37.5

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/AdguardTeam/dnsproxy/releases.
meta.description for dnsproxy is: "Simple DNS proxy with DoH, DoT, and DNSCrypt support"
meta.homepage for dnsproxy is: "https://github.com/AdguardTeam/dnsproxy"
meta.changelog for dnsproxy is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/iqs1vsim5gsyyxpb5l1vmg3zx0lbr5lb-dnsproxy-0.37.5/bin/dnsproxy passed the binary check.

1 of 1 passed binary check by having a zero exit code.

0 of 1 passed binary check by having the new version present in output.

found 0.37.5 in filename of file in /nix/store/iqs1vsim5gsyyxpb5l1vmg3zx0lbr5lb-dnsproxy-0.37.5

directory tree listing: https://gist.github.com/484cf321b7f7fc2b302d270945976df4

du listing: https://gist.github.com/48d6ee22f218d22718d67d54156c7bee

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
dnsproxy

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/iqs1vsim5gsyyxpb5l1vmg3zx0lbr5lb-dnsproxy-0.37.5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A dnsproxy https://github.com/r-ryantm/nixpkgs/archive/14abe1279593364169e0a472acbec4ee166d39ae.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/iqs1vsim5gsyyxpb5l1vmg3zx0lbr5lb-dnsproxy-0.37.5
ls -la /nix/store/iqs1vsim5gsyyxpb5l1vmg3zx0lbr5lb-dnsproxy-0.37.5/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

dnsproxy

Maintainer pings
cc @contrun @kalbasit for testing.

Result of nixpkgs-review pr 125518 at 14abe127 run on aarch64-linux 1

1 package built successfully:

dnsproxy

1 suggestion:

warning: unclear-gpl
gpl3 is a deprecated license, please check if project uses gpl3Plus or gpl3Only and change meta.license accordingly.
Near pkgs/tools/networking/dnsproxy/default.nix:21:5:
   |
21 |     license = licenses.gpl3;
   |     ^


Limit in classic JDBC DAO?

 I could not find any reasonable way to do it with findManyByCondition.
I appreciate any help here ...
BR

Currently no such method is generated, but I guess it shouldn't be a problem to add it.


DataFrame.asof

like Series.asof, should also take skipna={none, 'any', 'all'}

I've been toying with this a little and it's not clear that it's possible to avoid doing the asof_locs call for each column, since they might have missings in different positions. So you probably dont end up gaining much over what the df.apply(lambda s: s.asof(t)) approach gives you.
However, if you don't want missings to be ffilled, then you only need to do it once. Perhaps it makes sense to have a flag argument which indicates whether the user wants missings to be ffilled during the asof operation?

@shoyer  As discussed in #10266 this functionality is already available via df.reindex(where, method='ffill'), potentially with a dropna/fillna depending exactly what you want out at the end of the day. So we can probably close this this issue?

Yep, let's close it.


proposal: standard library optimizations

I am going around the standard library and trying to find things to improve.
I don't know if this is a priority now but as you know people like to work on things they are interested in.
If you think changes like this will have a chance to get merged, i will gladly continue with this issue.
Example:
from:
func EqualIgnoreCaseOld(s1, s2 string) bool {
	if len(s1) != len(s2) {
		return false
	}
	for i := 0; i < len(s1); i++ {
		c1 := s1[i]
		if 'A' <= c1 && c1 <= 'Z' {
			c1 += 'a' - 'A'
		}
		c2 := s2[i]
		if 'A' <= c2 && c2 <= 'Z' {
			c2 += 'a' - 'A'
		}
		if c1 != c2 {
			return false
		}
	}
	return true
}

To:
func EqualIgnoreCase(s1, s2 string) bool {
	if len(s1) != len(s2) {
		return false
	}
	for i := 0; i < len(s1); i++ {
		if s1[i] &^ 'a' != s2[i] &^ 'a' {
			return false
		}
	}
	return true
}

These benchmarks are on random data.
BenchmarkEqualIgnoreCase-8       	12356467	        93.7 ns/op
BenchmarkEqualIgnoreCase1-8      	11593638	        99.6 ns/op
BenchmarkEqualIgnoreCaseOld-8    	 5494894	       216 ns/op
BenchmarkEqualIgnoreCaseOld1-8   	 5293328	       222 ns/op

Even while this is closed. A suggestion. Please use https://pkg.go.dev/golang.org/x/perf/cmd/benchstat?tab=doc and run multiple rounds (10 or 20) on a quiet machine to get output that looks like:
name        old time/op  new time/op  delta
GobEncode   13.6ms ± 1%  11.8ms ± 1%  -13.31% (p=0.016 n=4+5)
JSONEncode  32.1ms ± 1%  31.8ms ± 1%     ~    (p=0.286 n=4+5)

@martisch so you don't want these types of contributions?

Contributions that improve the speed of the standard library are welcome.
There really isn't a need for a meta-issue like this, so it doesn't need to be open. Go ahead and submit specific optimization CLs when you have them. The contribution guide explains how to do it, if you haven't seen that already.
The optimization you propose for EqualIgnoreCase isn't correct. And EqualIgnoreCase doesn't appear anywhere in the stdlib. So I'm not sure what that example was supposed to be about...

@randall77 go/src/strconv/atof.go:17 func equalIgnoreCase(s1, s2 string) bool

That no longer exists in the stdlib at tip. As of https://go-review.googlesource.com/c/go/+/230737 , I think.

@martisch so you don't want these types of contributions?

Thats neither what I wrote nor my intention. I had assumed since you earlier closed the issue that this should have been stayed close as it was reopened without comment.
Generally feel free to send performance improving CLs that show concretely where and how the code will be changed and the benchmark data.
https://golang.org/doc/contribute.html
No need to make it a proposal which is a more involved process of e.g. proposing a public API change for go. https://github.com/golang/proposal#readme


Connectivity bug related to "Enable Static ARP entries" Option in the DHCP section

I struggled with a problem for some time where one of my vLans constantly lost WAN connectivity after varying periods of time. A reboot fixed the problem but it always came back, sometimes after two hours, sometimes after two days. After trying around a lot I finally found out that disabling the static ARP entries option for DHCP made the problem go away. Since all clients on that vLAN do have static leases with static arp enabled and sometimes it works and sometimes not I suspect there is a bug there to be fixed.
Problem occurred on 17.1 as well as 17.7 and 17.7.1
Steps to reproduce my use case:

Create a trunked interface with at least 2 tagged vLans on it (maybe a regular interfaces are sufficient but I do not have a spare one to test with)
at least one vLAN interface (A) gets a default WAN route so devices on it have WAN access
at least one vLAN interface (B)

does not get a default WAN route
has at least one client with static DHCP lease, static ARP and its own WAN route
has "Enable Static ARP entries" Option enabled globally for that interface

You may find a more detailed description of the proceedings in this opnsense forum thread.
Now sit and wait and (hopefully) watch the client on B lose all connectivity after some time. It can not even be reached from the router then (see forum thread).

Timeout


Feat/multi blockchain wallet overhaul

@micwallace @oleggrib - from today's call we'll keep this on hold until a later release.
Looking forward for us to release this in the near future guys.


Building multipolygons with intersecting segments

When building multipolygons, Osmium detects intersections between way segments and doesn't build the multipolygons in these cases. This is because valid polygons aren't allowed to have intersections and having only valid polygons makes further processing much easier.
Ideally all those cases will be fixed in the OSM data and Osmium can report where those intersections are to help in fixing them. We want to manually look at those places and fix them, because most often the intersections are a sign of a bigger problem with the data.
But at least as an option it would be nice to be able to fix those polygons by introducing "pseudo nodes" at the places where the intersections are. This will at least make the polygons valid allowing further processing. Implementation should not be too difficult, but there are a lot of corner cases to be done correctly. A "proper" intersection in the form of an X is probably easy, but what about segments that are on the same line, etc.

Two years later and nobody has shown interest in this. It seems this is not needed. Closing.


299 refactoring has property cluster 05

In this pull request, I have standardized the naming of ten object properties by adding a 'has' prefix.

loggsPublicationEvent to hasLoggsPublicationEvent
measureValueUnit to hasMeasureValueUnit
objectType to hasObjectType
previousRelatedProductionJob to hasPreviousRelatedProductionJob
provenance to hasProvenance
publicationPlanStatus to hasPublicationPlanStatus
publicationScheduleDate to hasPublicationScheduleDate
relationLink to hasRelationLink
resourceOffset to hasResourceOffset
rightsTerritoryExcludes to hasRightsTerritoryExcludes

Reviewers

@JuergenGrupp
@aro-max

loggsPublicationEvent is misspelled. It should read logsPublicationEvent. This reveals "logs" as being the active form of "to log" and therefore the property does not need to be corrected in naming. It only needs to be corrected in spelling.

@JuergenGrupp Can you please review the changes?


Fix issue where an unloaded video can cause the frame calculation to …

…crash the app

Thank you for the fix!


Imaging Convenience Tools

This PR proposes some convenience tools for condensing a some common tasks that are mostly well-defined but currently take many lines of code.

@samaloney Would something like this be within scope for STIXpy?


Get the max depth of input query before execution.

Hi Team,
Is there any way to get the max depth of input query before execution?
As per the documentation, We can validate the max depth via
$rule = new QueryDepth($maxDepth = 2);
DocumentValidator::addRule($rule);
But can we have the max depth info of input query in the resolver function? This info help us to write efficient resolver.
Thanks a lot for such an amazing library.

@hemratna does this answer help? https://github.com/webonyx/graphql-php/issues/316#issuecomment-408760772

We actually have #436 which is a good fit for such use-case. Once it is merged we should consider adding a corresponding method to it.
Right now the workaround is to create your own validation rule similar to QueryDepth and manually set max depth to your $context (3rd argument in resolves) after validation but before execution.


Beans and Bean annotations

Description
Beans

Defining beans
Annotations
Configuration

Domain
Backend Dev (Java)
Type of Contribution
Audio, Video, Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign

@Satyam175 please close this issue also.


Support for Unity 2023.2+/Emscripten 3.1.38+

Unity 2023.2 updates from Emscripten 3.1.8-unity to Emscripten 3.1.38-unity. One change between these upstream Emscripten versions is that the libstdcxx version bundled in Emscripten has been updated.
I tried building a test case, which I presume is the very one commented in https://github.com/atteneder/DracoUnity/issues/55#issuecomment-1628558572 , with latest Unity 2023.2, but it looks like the libstdcxx change brings in ABI breakages, then yielding new build errors
wasm-ld: error: DracoUnity/Runtime/Plugins/WebGL/libdracodec_unity.a(metadata.cc.o): undefined symbol: std::__2::__vector_base_common<true>::__throw_out_of_range() const
wasm-ld: error: DracoUnity/Runtime/Plugins/WebGL/libdracodec_unity.a(metadata.cc.o): undefined symbol: std::__2::__vector_base_common<true>::__throw_length_error() const

So it seems like a new DracoUnity version for 2023.2+ is needed.
I wonder if it would be possible to create a deployment where all google/draco code would be compiled directly as native C/C++ code inside Unity? Or I wonder if the draco build system needs are more complex that building as part of Unity's plugin build will not be feasible?

fyi: There's a solution in the works. DracoUnity 5.x will put WebAssembly binaries in sub-packages that will match your Editor version. No promises, but ETA is 1-2 months.

This is now supported by Draco for Unity, the successor of this repository/package.


Avatar Size

At the moment the avatar size the tweet.js file calls is the normal size twitter image. How can we tweak the js to upload the larger twitter avatar image. I want to able to use the 128px x 128px instead of the max width 48px avatar.
For example:
At the moment the tweet.js uploads:
http://a0.twimg.com/profile_images/1669742186/eskee_normal.jpg
But how can we get it to upload this one instead:
http://a0.twimg.com/profile_images/1669742186/eskee_reasonably_small.jpg

Oops, sorry, didn't notice you'd already closed this issue. Anyway...
The image URL used is the one returned by the twitter API, and there's no option to request different sizes, so this isn't something I'll change the plugin to support directly. However, using the template option, you could probably create a custom template function which modifies the HTML generated for the avatar URL.
-Steve


Improve tests for Coreg, in particular with NaNs

As mentioned in #135, many issues arose when using DEMs with NaNs. Current tests do not include nans, so it should be improved.
The lower level function such as get_horizontal_shift or calculate_slope_aspect should also be tested, to more easily identify the source of the issue.

And also for all Coreg subclasses, see #202


file modify

name added

@Abdullahijazvu  use proper comments guidelines. e.g you added your name.
you can say.
Added name in CONTRIBUTING.md


[DO NOT SUBMIT] Test impersonation chain no reset

Please add a meaningful description for your change here
This tests imperesonation chain with no reset of credentials. If the globals work how I think, this should fail.
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Run Python 3.8 PostCommit


Console window not working - traceback generated

From<EMAIL_ADDRESS>on 2012-03-22T23:24:11Z
Spyder Version:  2.1.8
Python Version:  2.5.4
Qt Version:      4.8.0, PyQt4 (API v1) 4.9.1 on Windows
What steps will reproduce the problem?
1.Install spyder
2.Start
3.Traceback is generated for python shell handling
4.Press enter in console window, the traceback appears again
5.Close all interpreters in the console part of spyder
6.Right click and choose open interpreter or open IPython interpreter
7.In both cases python crashes and empty console window is opened, now go to step [4] - press enter to generate traceback What is the expected output? What do you see instead? Shell is expected to appear
Instead, traceback is seen
Please provide any additional information below
. Traceback is as below:
D:\spyder>python c:\Python25\Scripts\spyder -d
Start of MainWindow constructor
Initializing...
End of MainWindow constructor
*** Start of MainWindow setup ***
Loading object inspector...
Loading outline explorer...
Loading editor...
Loading file explorer...
Loading history plugin...
Loading online help...
Loading project explorer...
Loading external console...
Loading namespace browser...
Setting up main window...
Setting up window...
*** End of MainWindow setup ***
QNetworkAccessFileBackendFactory: URL has no schema set, use file:// for files
QNetworkAccessFileBackendFactory: URL has no schema set, use file:// for files
Traceback (most recent call last):
File "C:\Python25\Lib\site-packages\spyderlib\widgets\externalshell\pythonshell.py", line 537, in send_to_process
"toggle_inputhook_flag(True)")
File "C:\Python25\Lib\site-packages\spyderlib\utils\bsdsocket.py", line 64, in communicate
write_packet(sock, command)
File "C:\Python25\Lib\site-packages\spyderlib\utils\bsdsocket.py", line 24, in write_packet
sock.send(struct.pack("l", len(sent_data)) + sent_data)
AttributeError: 'NoneType' object has no attribute 'send'
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=976

From<EMAIL_ADDRESS>on 2012-03-24T08:32:48Z
Issue is does not seem to be fixed with the correction indicated.
I applied the change in my installed version of spyder.
The exception is not occurring BUT, when "open interpreter" is chosen python.exe crashes, shell does not appear.

From pierre.raybaut on 2012-03-24T00:59:28Z
Already fixed by this changeset: https://code.google.com/p/spyderlib/source/detail?r=c7b02e997b7bc645240eea923cd49d43f1431138
Status: Fixed


Blog entry: how to make new tasks

To include:

how to write a new xml, and try it out in the Human Action Component
alternatively, how to do the drawing bit in vanilla Minecraft, then save and load it in an xml

@katja-hofmann Much of this content is in the Tutorial already - do we want to duplicate it? There's scope to make the 'blog' more accessible - aim it at beginner users as well as advanced ones who want to code Java mission handlers.

Thank you for looking into this!
I'd like to avoid duplication, so making the easy bits of the blog more
accessible sounds like the way to go.
On Jul 20, 2016 15:19, "Tim Hutton"<EMAIL_ADDRESS>wrote:

@katja-hofmann https://github.com/katja-hofmann Much of this content is
in the Tutorial already - do we want to duplicate it? There's scope to make
the 'blog' more accessible - aim it at beginner users as well as advanced
ones who want to code Java mission handlers.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/malmo/issues/216#issuecomment-233963441,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANB8i4tmu13JGWG2kzWTjKmjIj0UNFQbks5qXi6CgaJpZM4JQv6w
.


Miscellany

Add details to readme, ignore _shake directory.

This ought to fix #6.

Regarding 77d34c7:
Have a look at this commit: ddfe5bc. I decided to move the shake database inside shake-build and renamed it to .db. To make naming consistent I also renamed _shake to .shake (this also matches the naming convention for .git and .arc-linters. If you agree with this naming convention please amend 77d34c7.

Regarding 576a06c:
I think this will become redundant once #5 is fixed. The build action already depends on the corresponding builder, so once I relocate all binaries to inplace/bin they will automatically be tracked.

Ahh, I see. Alright, perhaps I'll drop the gitignore change and "Documentation: Depend upon haddock for stage".

@snowleopard I think this is ready for another review.

Excellent! This is now merged. I'll now look into #6.


Wrong Return Urls after payment redirect for non-default languages using nginx multisite configuration

Precondition:
PWA with multisite configuration and nginx, e.g. a PWA started with docker-compose
Actual Behavior

If the user switches the language and uses a payment method that requires a redirect the return url contains the current locale only as query params, but not as part of the url path
example:
default language: English, url: http://localhost:4200/en/route
changed language: French,  url: http://localhost:4200/fr/route
return url after redirect: http://localhost:4200/en/checkout/review;lang=fr_FR?...
Expected Behavior

The path of the return URL reflects the selected language of the user, i.e. it looks like http://localhost:4200/locale/....
example:  http://localhost:4200/fr/checkout/review;lang=fr_FR?...
Steps to Reproduce the Bug
Steps to reproduce the behavior:

Open PWA and switch language to French
Login, add products to basket and start checkout
On checkout payment page select a redirect payment method, e.g ISH credit card
After redirect check the url path,
-> the default language will be part of it instead of the selected language

Environment Details
only valid in a multisite configuration with nginx

Desktop: *

OS: [e.g. iOS]
Browser [e.g. chrome, safari], Version [e.g. 22]
Project Code Version

Smartphone: *

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari], Version [e.g. 22]
Project Code Version

Additional Context, like Screenshots, Log File Snippets etc.

AB#70767

Consider SSO redirects as well.

Compare multi-site.service.ts and consider implementing a shared configuration between nginx and the PWA (for example to have a centrally defined place for baseHrefs)


Docs fixes & improvements

[x] All tests passed. If this feature is not already covered by the tests, I added new tests.
[x] This pull request is on the dev branch.
[x] I used gofmt for formatting the code before submitting the pull request.

Typos fixes on the README
Improvements to the GitHub Actions templates.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 74.183%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
227

Relevant Lines:
306

💛  - Coveralls

Can you please run "reformat" on the README before merging?


Fix typo in grafana datasources manifest

s/etitable/editable
http://docs.grafana.org/administration/provisioning/#example-datasource-config-file

Need help with fixing ./scripts/check-make-generate.sh

All of the generated manifests are being reworked in #1279, I recommend waiting for that change before getting into editing the dashboards as we're moving all the generation to be done by jsonnet.

We've finished the whole jsonnet migration, but the error you found here seems to have survived it. To save you further trouble with it I already opened a new PR to fix it in the new location. Thanks a lot for this PR, I would have not caught this in a while!
Closing here, but thanks again for the catch! :slightly_smiling_face:
For the reference, the new pull request can be found here: https://github.com/brancz/kubernetes-grafana/pull/30


Adding new college

Adding new college

@ranet101 I am sorry, the domain pipeline.sbcc.edu is blacklisted because the school allows to register an email address there even if you are not a student of the school. If you are the student of this school, please apply with official document (OFFICIAL DOCUMENT tab at https://www.jetbrains.com/shop/eform/students).

@ranet101 I am sorry, the domain sbcc.edu is blacklisted because the school allows to register an email address there even if you are not a student of the school. If you are the student of this school, please apply with official document (OFFICIAL DOCUMENT tab at https://www.jetbrains.com/shop/eform/students).


feat(addon-doc): support remote url for code source

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Refactoring
[ ] Build or CI related changes
[ ] Tests related changes
[ ] Documentation content changes

What is the new behavior?

alright?
<a
    *polymorpheusOutlet="path | tuiIsUrl ? path || sourceCode as link; context: pathOptions"


Add a metric that reports the VMI last connection timestamp

What this PR does
Add kubevirt_vmi_last_api_connection_timestamp_seconds metric, which reports the last connection timestamp per VMI. Connections covered: console, vnc, usbredir, portforward and SSH.

Fixes https://issues.redhat.com/browse/CNV-14659
Why we need it and why it was done in this way
This metric will allow users to identify VMs that have not been logged into for a long time and have allocated resources but are not actually being used.
This will help admins to clean up unused VMIs.

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

Add kubevirt_vmi_last_connection_timestamp_seconds metric

/test pull-kubevirt-e2e-k8s-1.29-sig-monitoring

/cc

/lgtm

@sradco can you please review?

/hold

/unhold

/lgtm

/lgtm


[5.3] Create seeders when creating Models or Migrations

Add --seeder to artisan make:migration or artisan make:model to create a matching Seeder. This was discussed in laravel/internals Issue 13. A matching PR for laravel/docs will be following.

Thanks for the PR. I've left a minor review comment. :)

I kinda want to avoid heading this direction because it sort of tends to bloat over time. For example, next why would't we add an option to the model command to create a seeder as well? Then we would be able to create a migration, a resource controller, a plain controller, and a seeder all from the make:model command. I would prefer to keep them slightly more separate.

yah, i'm really starting to regret sending that PR for creating a controller along with the models, because i knew this is what would happen.
maybe the solution is remove all of these 'nicety' options and encourage users to create a custom console command called something like make:all.  Then within that command, the user could define exactly what they wanted.
public function handle()
{
    $this->call('make:controller');
    $this->call('make:seeder');
    $this->call('make:migration');
}


FIRUser decode of dates is incorrect.

[REQUIRED] Step 2: Describe your environment

Xcode version: 9.3.1
Firebase SDK version: 4.33.0
Firebase Component: Auth
Component version: 5.0.0

[REQUIRED] Step 3: Describe the problem
The date fields of ID tokens are incorrectly decoded in current epoch.
See  [FIRUser parseIDToken:error:]
NSDate *expDate =
      [NSDate dateWithTimeIntervalSinceNow:[tokenPayloadDictionary[@"exp"] doubleValue]];

They should be decoded in UNIX epoch - see  https://firebase.google.com/docs/auth/admin/verify-id-tokens

exp | Expiration time | Must be in the future. The time is measured in seconds since the UNIX epoch.

Since we are now in 2018 and epoch is 1970, we end up ~48 years in the future.
Steps to reproduce:
Get an ID token with [FIRUser getIDTokenWithCompletion:]
Observe that in parseIDToken the expiration date (and other dates) ends up in 2066.

@jbb1003 Thanks for reporting this! I'll try to reproduce locally.


refactor: add compatible tbl_flatten and lsp_get_clients for new version 0.10

add compatible util.tbl_flatten and lsp.get_active_clients to avoid  deprecated error notify on 0.10 release.

As pointed in https://github.com/neovim/nvim-lspconfig/pull/3154#discussion_r1604304077, nvim_nine is true for all of nvim 0.9, 0.10, and 0.11, etc. So it ends up still using the deprecated vim.tbl_flatten in nvim-0.10, and vim.iter() not being called.

oops i didn't validate that. yup has seems like not correct

this is still broken


391 display country column in projects

Added Column and Filter

Isn't #397 the same?

No.. it's almost the same.
#397 was in Drive model only column. This one is in Project, and here is filter added too

Good work on that. Can you resolve the conflict and remove package-lock.json from PR?

Good work on that. Can you resolve the conflict and remove package-lock.json from PR?

I believe that what @arturtamborski meant was to remove the package-lock.json from the diff, not the whole file from the repository. This could help:
git checkout upstream/develop -- frontend/package-lock.json


chore: release 0.3.0

:robot: I have created a release *beep* *boop*
0.3.0 (2021-11-19)
Features

client: add remove command (dd32e1a)
server: support custom trdl.yaml and trdl_channels.yaml files (aa5a986)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/werf/trdl/releases/tag/v0.3.0 :sunflower:


feat: Manual Pipeline support

When {"type": "manual"}, use the listed files in {"pipeline_files": []} to create Pipelines.
See also: #74
Closes: #72

LGTM.
Worth noting I do not think this would work internally for us since we do not pass the runway_dir, we get the files directly from Gitlab. We would need to revamp the Jenkins job to allow an optional runway_dir path for manual pipelines.

@e4r7hbug Anything else needed on this?

Nothing else that I can think of. I'll merge this in.


(BREAKING CHANGE) represent all documents as a map at root level

this will open up yjsgo to work with more complex objects, which is now needed by my project.
NOTE THIS IS A BREAKING CHANGE
TODO:

restore a new YJS doc from a JSON object instead of from document text.  This will require importing the yjsdiff project I'm working on separately.

superceded by #5


What's in bb8 0.8.0?

Hi there! I've noticed that bb8 0.8.0 was released, but can't find a tagged commit (or a github release).
Looking at raw cargo data, I know 0.7.0: 7150f7cdb941eb455def1a3a0a75e20eae4914c7., 0.8.0: 87819652be715b2b2a65c2fa924e6e7d70af6ebe and can see the diff – but it would be much easier with tags
Was this on purpose? If not, could you please add git tags? 🙏

I've added tags for 0.7.1 and 0.8.0. Hope that helps.

Thank you!


VnV: minor issues

[x] Revision history date format broken "14-0202023" and "16-0202023"
[x] Section 4.4: my name! 😉 is "Joachim de Frostier Fourestier"
[x] "FR" is listed in the table but not used. Maybe change it to just "R" like you have in 5.1 (test for functional requirements), or change the "R"s to "FR"s ?
[x] Remove 7.2 Usability Survey Questions, since it's not needed/used?
[x] in 3.2: "[...] but also Wwe will test [...]" - lowercase
[x] in 4: two broken references? "Section ??"

All done! Thank you.


Refactor VM operational test for runner

Hi @xindzhan, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.azure.com.

TTYL, AZPRBOT;

Merging.


Timeseries chart cannot have negative rotation applied to x axis

On http://c3js.org/samples/timeseries.html, change the tick: {} block to read:
tick: {
    rotate: -45,
    format: '%Y-%m-%d'
}

hi look at #1511

Closing in favor of #1511


PROD: hide invite button if invitation exists already

If a user enters the same person twice as member (for 2 clubs ) and sends invites he will get an 500 error "duplicate key: email".
To fix this:

dont allow to create a new member if the eMail address already exists. load the existing member instead and ask to save this !
hid invite button if invitation already exists

fixes #514


instructions use "cmus-lyrics-master.zip" rather than "master.zip"

Hi - apologies in advance if I have got this wrong.
The instructions refer to "cmus-lyrics-master.zip" but I think the file is called "master.zip"
eg. this line
Extract it,
unzip cmus-lyrics-master.zip

should maybe be
unzip master.zip
and likewise the cleanup instructions
rm cmus-lyrics-master.zip
should be
rm master.zip

also this line at the top of the instructions:
Obtain sudo privileges and download cmus-lyrics-master.zip
should maybe change to master.zip

Hi - I'm referring to the Instructions in the README.md
==
Installation :
Very easy.
Obtain sudo privileges and download cmus-lyrics-master.zip,
sudo -s
wget https://github.com/hakerdefo/cmus-lyrics/archive/master.zip
Extract it,
unzip cmus-lyrics-master.zip
==
if you hover over the linked file in this part of the README it is called master.zip not cmus-lyrics-master.zip (so the Extract instructions wouldn't work, would need to be unzip master.zip). Just pointing this out in case it helped anyone.
URL:
https://github.com/hakerdefo/cmus-lyrics/archive/master.zip
clickable text for that link "cmus-lyrics-master.zip"

sorry, to answer your question - the method I just tried was via wget
wget https://github.com/hakerdefo/cmus-lyrics/archive/master.zip
which downloaded a file called master.zip to my desktop

I think I've nailed the source of confusion @hobbsy
In my .wgetrc I've following included,
content_disposition = on
This option correctly names the downloaded file. So at my end the file was downloading as cmus-lyrics-master.zip but since this option is not enabled by default at your end it was downloading as master.zip. I've added the --content-disposition option in the README.md
Thanks for pointing this out!
Cheers!!!


[BEAM-11773] Add bhulette as an owner for pandas, pyarrow, numpy requirements

Post-Commit Tests Status (on master branch)

Lang
SDK
Dataflow
Flink
Samza
Spark
Twister2

Go

---

---

---

Java

Python

---

---

XLang

---

---

Pre-Commit Tests Status (on master branch)

---
Java
Python
Go
Website
Whitespace
Typescript

Non-portable

Portable
---

---
---
---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @y1chi


unable to execute 'arm-linux-gnueabihf-gcc': No such file or directory

I run sudo python setup.py installand it comes back with the errorunable to execute 'arm-linux-gnueabihf-gcc': No such file or directory`
Here is the full log: SteamController.txt
I am using a raspberry pi 3
Thanks

Hello,
I think you should install gcc-multilib.
On Fri, Oct 28, 2016 at 12:35 AM, RooHoo33<EMAIL_ADDRESS>wrote:

I run sudo python setup.py installand it comes back with the errorunable
to execute 'arm-linux-gnueabihf-gcc': No such file or directory`
Here is the full log: SteamController.txt
https://github.com/ynsta/steamcontroller/files/557174/SteamController.txt
I am using a raspberry pi 3
Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ynsta/steamcontroller/issues/42, or mute the thread
https://github.com/notifications/unsubscribe-auth/AB26C8lBW6NQCv0PpyBbP9FTb7xb8lQGks5q4Sc5gaJpZM4Ki5eD
.

When I type sudo apt-get install gcc-multilib to install it I get Package 'gcc-multilib' has no installation candidate

I have the same issue with the Pie 3 running retropie. Apparently gcc-multilib is not in the repos.

I am also getting an error with arm-linux-gnueabihf-gcc as well as a few other things. I tried installing gcc-multilib and running it with python3 and neither are working.

Has the same, you need:
sudo apt-get install build-essential
See this full section: https://github.com/ReSearchITEng/RPi_matrix_16_keypad/blob/master/README.md#common-installation-section


Adding tasks from contextual quick menu on a start event inside of a embedded sub-process attempt to create the task outside of the sub-process

using VS Code.
It will not let you place the task in the sub-process.

Appears to occur for any object you attempt to add using the hover menu.

For better tracking, we are now following this issue only at https://issues.redhat.com/browse/KOGITO-7014


Improve caching

A local cache is currently used when duplicate requests are made to the remote API. This returns the locally stored data instead of retreiving the data from remote API again.
Table data is only cached for one day as there is currently no mechanism to check if it is out of date. Adding a mechanism check if the data is out of date and update the local copy would reduce the number of calls to the API and speed up data retreival.
The actual local storage could take two forms:

Modify the existing temporary file storage to allow it to be updated.
Convert to a local SQLite database.

Both options would require a mechanism to check if the local data is out of date.

I need to work on multiple databases within one setting. What would be the best way to clear this local cache each time I need to access a new database?

@Winda2 The quick workaround would be to call _ = [path.unlink() for path in pyramm.cache.TEMP_DIRECTORY.glob("*")]. This will step through each file in the file cache directory and delete it (keep the parent directory itself will be kept).
A better long term solution is for the file cache system to include the RAMM database name. This way the file cache will work with multiple databases.

Thanks @johnbullnz that was so helpful. Appreciate it!


Add --verbose flag for KV bucket commands

Listing out every file that is being uploaded is useful in some cases, but is typically quite verbose. We should make printouts of bucket action information something that is toggled on by a --verbose flag.

closed by #626


Fix test asserting verbose log output

It's the flaky failure 😢

Rebased in case #3503 wasn't picked up yet


i can't find the libRCTCamera.a

i can't find the libRCTCamera.a after download . it not in the directory. why?

It's a product of building this module. Be sure to follow the steps in the readme exactly.


Support typescript inlay hints?

Typescript added inlay hints in the latest version: https://github.com/microsoft/TypeScript/pull/42089
And it seems this is how they added it to vscode: https://github.com/microsoft/vscode/pull/113412/files, it would be nice if we could add it to neovim as well and given this plugin already seems support the same for rust...

See https://github.com/jose-elias-alvarez/nvim-lsp-ts-utils/issues/40 which may be better suited for supporting this.


resolve bug with daily notice metadata stores not days without notices in TED API

Codecov Report
All modified lines are covered by tests :white_check_mark:

Comparison is base (095801d) 94.79% compared to head (ba3cf16) 94.79%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #506   +/-   ##
=======================================
  Coverage   94.79%   94.79%
=======================================
  Files         117      117
  Lines        6283     6283
=======================================
  Hits         5956     5956
  Misses        327      327

Files
Coverage Δ

...anager/services/daily_notices_metadata_services.py
91.02% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


elasticSearch query  error

A clear and concise description of what the bug is.

error log:

How should I connect es correctly?
Environment
(please complete the following information):

superset version: 0.37.0
python version: 3.7.9
node.js version: 12.18.3
npm version: 6.14.6

@zhaoyongjie

hi @chen-ABC , this seems like ES drivers error. would you tell me 1) which ES driver you use and 2) SQLAlchemy ES driver version ?

hi @zhaoyongjie   I'm using  elasticsearch-dbapi 0.1.2,   and  my Elasticsearch is 6.7 versions
this dirver only support Elasticsearch 7.X versions.  What driver should I use to connect?

@chen-ABC I think you can only use Elasticsearch >= 7 as anything lower lacks sql support

thank you both @zhaoyongjie and @nytai. closing issue for now.


Ship the Cargo.lock file in the source distribution

Ported from apache/arrow-datafusion#1508

cc @Jimexist

This is failing because the cache key used in CI invalid. I'll fix that here.

thanks @cpcloud - do you want to include the lockfile itself into git?

Yeah, that's the plan. I'll finish it up tomorrow.


Add v3 datamodel for in-cluster RR function

@caseydavenport Just sorted the bgppeer_e2e_test.go issue, so I think this is good for your eyes now.  I will write another longer comment about where I am overall on this, and some questions where I'd appreciate your input.

I believe the changes here are complete for the following testing to run successfully:

libcalico-go's own repo testing
confd make test (with associated confd changes: WIP at https://github.com/projectcalico/confd/pull/145)
node ST tests/st/bgp/test_route_reflector_cluster.py and tests/st/bgp/test_single_route_reflector.py (with WIP at https://github.com/projectcalico/node/pull/50)

I still have more testing to add:

running through Typha
node ST cases where:

reverse peering logic makes a difference
multiple BGPPeer configs generate overlapping peerings
there are many nodes of which one is also an RR
there are many nodes of which two are also RRs with the same cluster ID

calling 'calicoctl node status' during a node ST, so we can check how it looks

and some of that may require more libcalico-go change, but I think not much, and hence that it's worth reviewing and potentially merging this lot now.
I think we could merge this without impacting other components - but would appreciate your view on that - and don't think we need a feature branch.  So I'd also propose soon to rebase this on current master.  The main affected consumer is confd, and I hope to get that ready for your review tomorrow.
Commits are quite messy at the moment, but I can tidy those or just squash down post-review.
Finally, I was going to leave the removal of BGPPeerUpdateProcessor to a later PR, but that would have left the existing code inconsistent in this PR, as it would not be doing anything with the new selector fields; so I thought better to remove it now, and hence the BGP syncer too.

Thanks for your reviews @caseydavenport !


Catch all routes with "react-instantsearch-hooks-router-nextjs" do not work

🐛 Current behavior
Catch all routes with "react-instantsearch-hooks-router-nextjs" do not work. The browser button does not work at all.
@aymeric-giraudet I tried solving it with your beforePopState solution. But this does not even get triggered.
🔍 Steps to reproduce
Create a [...all].js in pages.js
Then make the instant search pages use the same catch all route as the hit.
The fix is in https://github.com/algolia/instantsearch/blob/master/packages/instantsearch.js/src/lib/routers/history.ts =>
Problem is
`  public dispose(): void {
if (this._dispose) {
this._dispose();
}
this.isDisposed = true;

safelyRunOnBrowser(({ window }) => {
  if (this._onPopState) {
    window.removeEventListener('popstate', this._onPopState);
  }
});

if (this.writeTimer) {
  clearTimeout(this.writeTimer);
}

this.write({} as TRouteState);

}`
This last rule should not be there for next to work.
    this.write({} as TRouteState);
When I comment this line everything works as expected.
Live reproduction
https://codesandbox.io/s/github/algolia/instantsearch.js/tree/templates/react-instantsearch
💭 Expected behavior
Catch all routes with getStaticProps should work.
Package version
next 13.3.0, react-instantsearch-hooks-router-nextjs 6.42.1, react-instantsearch-hooks-server 6.42.1, react-instantsearch-hooks-web 6.42.1
Operating system
No response
Browser
No response
Code of Conduct

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

Hi @aymeric-giraudet the product pages in my case dont have InstantSearch at all. But they are on the same catch all next route.

Hi @DB-Alex,
Do you mean you have a condition in which based on the slug caught in [...all].js, you conditionally render either a Hit page or an InstantSearch page ? Then it would make sense than dispose is called indeed.
However would it be possible to still declare an actual route for InstantSearch ? Like having a search.js page for it then the rest can be caught in [...all].js for hits, maybe that would fix things ?
Once again it's a bit difficult to tell without a reproduction, providing one would help us greatly for fixing this :)

Hi @aymeric-giraudet,
Thats indeed the case, we have very dynamic routes based on our CMS system.
So a route can be category page but also a product page.
We have a [...all].js then in getStaticProps we check what kind of page we need to load and the render method is like this:
{(pageType === 'product') ? (
    <ProductPage
        {...props}
    />
) : (
    <ConditionalWrapper
        condition={(!isEmpty(index))}
        wrapper={(children) => (
            <InstantSearchSSRProvider
                {...serverState}
                key={getProviderKey({ path: router.asPath })}
            >
                <InstantSearch
                    {...index}
                    serverStateProps={serverStateProps}
                >
                    {children}
                </InstantSearch>
            </InstantSearchSSRProvider>
        )}
    >
        <BlocksPage
            {...props}
        />
    </ConditionalWrapper>
)}

So we are navigating to a product and the instant search index is gone. But we are still on the same route for nextjs.
Hope this clears the issue for you.

Great! Yeah i am aware of the separated pages but this client wants extreme flexibility. I simple do not know what the url will be or what kind of page will be on it.
Here is my router implemention:
// @todo
// To use algolia-next.js this bug should be resolved first
// https://github.com/algolia/instantsearch/issues/5586

let {
	indexParams,
	filterDivider,
	segmentOrder,
} = CategoryConfig;

export const router = ({
	filters,
	indexName,
	serverStateProps = {},
}) => {
	let { path } = serverStateProps;

	indexName = getIndexName({ indexName });

	return {
		router: createInstantSearchRouterNext({
			routerOptions: {
				createURL: ({
					routeState,
					location,
				}) => {
					let url = getCleanPath({ path: location.pathname });
					let indexParamsValues = Object.values(indexParams);
					let filterUrl = filterDivider;

					for (let indexParamsKey of segmentOrder) {
						if (routeState[indexParams[indexParamsKey]]) {
							url += `${indexParams[indexParamsKey]}-${routeState[indexParams[indexParamsKey]]}/`;
						}
					}

					Object.keys(routeState).forEach((key) => {
						let value = routeState[key];

						if (value) {
							if (Array.isArray(value)) {
								value = map(value, (o) => encodeURIComponent(o));
							} else {
								value = encodeURIComponent(value);
							}

							if (!indexParamsValues.includes(key)) {
								filterUrl += `/${encodeURIComponent(key)}:${value.join(',')}`;
							}
						}
					});

					if (filterUrl !== filterDivider) {
						url += `${filterUrl}/`;
					}

					return url;
				},
				parseURL: ({
					location,
				}) => {
					let url = new URL(location);
					let routeState = {};

					for (let indexParamValue of Object.values(indexParams)) {
						let regex = new RegExp(`/${indexParamValue}-[^/]*`, 'g');
						let findIndexParam = regex.exec(url.pathname);

						if (Array.isArray(findIndexParam) && findIndexParam.length > 0) {
							routeState[indexParamValue] = findIndexParam[0].replace(`/${indexParamValue}-`, '');
						}
					}

					if (url.pathname.indexOf(filterDivider) !== -1) {
						let segments = url.pathname.split('/');
						let filterIndex = segments.indexOf('filter');
						let filters = [];

						segments.forEach((segment, index) => {
							if (segment && segment !== 'filter') {
								if (filterIndex !== -1 && index > filterIndex) {
									filters.push(segment);
								}
							}
						});

						if (!isEmpty(filters)) {
							filters.forEach((filter) => {
								if (filter) {
									let [key, value] = filter.split(':');

									if (value) {
										value = value.split(',');

										if (Array.isArray(value)) {
											value = map(value, (o) => decodeURIComponent(o));
										}

										routeState[key] = value;
									}
								}
							});
						}
					}

					return routeState;
				},
			},
			serverUrl: (path) ? `${process.env.NEXT_PUBLIC_BASE_URL}${path}` : process.env.NEXT_PUBLIC_BASE_URL,
			singletonRouter,
		}),
		stateMapping: {
			stateToRoute(uiState) {
				let routeState = {};
				let indexUiState = uiState[indexName];

				if (indexUiState) {
					routeState = {
						pagina: indexUiState.page,
						query: indexUiState.query,
						sorteren: getSortByValue({
							indexName,
							value: indexUiState.sortBy,
						}),
					};

					if (Array.isArray(filters)) {
						filters.forEach(({
							attribute,
							label,
						}) => {
							let urlKey = slugify(label);

							if (indexUiState?.refinementList?.[attribute]) {
								routeState[urlKey] = indexUiState.refinementList[attribute];
							}
						});
					}

					routeState = omitBy(routeState, isNil);
				}

				return routeState;
			},
			routeToState(routeState) {
				let uiState = routeState;

				if (indexName) {
					let indexUiState = {
						page: (routeState.pagina) ? parseInt(routeState.pagina) : undefined,
						query: routeState.query,
						sortBy: getSortByValue({
							indexName,
							routeState: true,
							value: routeState.sorteren,
						}),
						refinementList: {},
					};

					if (Array.isArray(filters)) {
						filters.forEach(({
							attribute,
							label,
						}) => {
							let urlKey = slugify(label);

							if (routeState?.[urlKey]) {
								indexUiState.refinementList[attribute] = routeState[urlKey];
							}
						});
					}

					indexUiState = omitBy(indexUiState, isNil);
					indexUiState.refinementList = omitBy(indexUiState.refinementList, isNil);

					if (isEmpty(indexUiState.refinementList)) {
						delete indexUiState.refinementList;
					}

					uiState = {
						[indexName]: indexUiState,
					};
				}

				return uiState;
			},
		},
	};
};

CategoryConfig:
export default {
	filterDivider: 'filter',
	indexParams: {
		page: 'pagina',
		query: 'query',
		sortBy: 'sorteren',
	},
	segmentOrder: ['sortBy', 'query', 'page'],
};

Please let me know if I can help.

Thanks @DB-Alex !
I think this is because this createURL implem does not take location.protocol and location.host into account.
To debug you could add this to node_modules/instantsearch.js/es/lib/routers/history.js at line 257 (inside shouldWrite basically) :
console.log(url, window.location.href);

And change your createURL implem so that the two strings are the same when you go on a hit page. I suspect for now it will be something like /product-xxx and http://localhost/product-xxx which is why it writes when it shouldn't.
Hope that helps !

Hi @aymeric-giraudet,
That was indeed the case! Thanks for your help :D

I think I was to early. Now the URLS are exactly the same but I get this warning:
The UI state for the index "v3_products" is not consistent with the widgets mounted.
is can happen when the UI state is specified via `initialUiState`, `routing` or `setUiState` but that the widgets responsible for this state were not added. This results in those query parameters not being sent to the API.

Of course all the virtual widgets are there...

Nevermind component was not unmounting, everything works as expected!


Minor fixes to random kernels

Jenkins, test this please.

Jenkins CI test (for commit dd494830f7a68aad343257fef7e617669ec3a022, target branch master) succeeded!

Jenkins, test this please.

Jenkins, test this please.

Jenkins CI test (for commit 078ee6c1780a42dc79940c39cf57fe525c2fa4b6, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Jenkins, test this please.

Jenkins CI test (for commit c5359bae450cf2dac13307042c64f15138ed9267, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Merge #1681 first.

Sorry for delay...
Jenkins, test this please.

Jenkins CI test (for commit 621d8fc90347f1bdc1d28ded13b0e71747f142e1, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Jenkins, test this please.

Jenkins CI test (for commit c277fa308081cbd1b0d60a7fd6b122fc3fe2c15e, target branch master) succeeded!


version 3.6.5 , h3 does not provide an export named 'fetchWithEvent'

Environment
                                                                                7:54:48 PM

RootDir: C:/Users/issam/Develop/personal/r-estate/website                                                                                7:54:48 PM
Nuxt project info:                                                                                7:54:48 PM

Operating System: Windows_NT
Node Version:     v16.20.0
Nuxt Version:     3.6.5
Nitro Version:    2.5.2
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      runtimeConfig, ssr, app, build, modules, compression, tailwindcss, css, vite
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

👉 Report an issue: https://github.com/nuxt/nuxt/issues/new                                                                                7:54:48 PM
👉 Suggest an improvement: https://github.com/nuxt/nuxt/discussions/new
👉 Read documentation: https://nuxt.com
Reproduction

Describe the bug
I have got this error on version 3.6.5
import { defineEventHandler, handleCacheHeaders, createEvent, eventHandler, setHeaders, sendRedirect, proxyRequest, setResponseStatus, getRequestHeader, setResponseHeader, getRequestHeaders, createApp, createRouter as createRouter$1, toNodeListener, fetchWithEvent, lazyEventHandler, getRouterParam, getQuery as getQuery$1, readBody, readMultipartFormData, createError } from 'C:/Users/issam/Develop/personal/r-estate/website/node_modules/h3/dist/index.mjs';
Additional context
No response
Logs
file:///C:/Users/issam/Develop/personal/r-estate/website/.nuxt/dev/index.mjs:7
import { defineEventHandler, handleCacheHeaders, createEvent, eventHandler, setHeaders, sendRedirect, proxyRequest, setResponseStatus, getRequestHeader, setResponseHeader, getRequestHeaders, createApp, createRouter as createRouter$1, toNodeListener, fetchWithEvent, lazyEventHandler, getRouterParam, getQuery as getQuery$1, readBody, readMultipartFormData, createError } from 'file://C:/Users/issam/Develop/personal/r-estate/website/node_modules/h3/dist/index.mjs';
                                                                                ^^^^^^^^^^^^^^
SyntaxError: The requested module 'file://C:/Users/issam/Develop/personal/r-estate/website/node_modules/h3/dist/index.mjs' does not provide an export named 'fetchWithEvent'
    at ModuleJob._instantiate (node:internal/modules/esm/module_job:123:21)
    at async ModuleJob.run (node:internal/modules/esm/module_job:189:5)
    at async Promise.all (index 0)
    at async ESMLoader.import (node:internal/modules/esm/loader:530:24)
    at async loadESM (node:internal/process/esm_loader:91:5)
    at async handleMainPromise (node:internal/modules/run_main:65:12)

I suspect you have an outdated version of h3 in your lockfile. Would you try updating it? 🙏

Seeing this same issue on my app as well. I was able to get it work again by running
pnpm install h3@latest --shamefully-hoist

I have fixed by removing package.lock.json & node_modules and install all the packages with yarn

can any one of you guys helo me with this error?

I am getting this when I run
npm run dev -- -o
my node version is 18, and I am on windows 11.
I did remove package-lock.json file and node_modules folder and did npm install

Fresh install of nuxt and/or nitro is producing this error for me.
https://nitro.unjs.io/guide/getting-started
npx giget@latest nitro nitro-app
cd nitro-app
npm install
npm run dev

gives:
SyntaxError: The requested module 'file:///Users/.../node_modules/h3/dist/index.mjs' does not provide an export named 'fetchWithEvent'


chore(prettier): change to 100 length and prettify existing files

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Memory improvements (1/3): Introduce new data access layer and schemas

[x] I have added the tests to cover my changes.
[x] I have updated the documentation and CHANGELOG accordingly.
[x] I have read the CONTRIBUTING document.

Closes https://github.com/AgnostiqHQ/covalent-staging/issues/707

This PR installs plumbing for the series on memory usage improvements. A new data access layer is introduced which shall replace the in-memory Result object as used by the server code in the next PR.

Marking it as draft in order to make the requested changes.


Add mac m1 fallback to x86_64 on rosetta

This way we can at least install the binary until arm binaries are
provided

Any time! Thanks for cleaning it up


!unsafe tags not supported

Issue Type

Bug report

Ansible and Ansible Lint details
ansible --version = <IP_ADDRESS>
ansible-lint --version 3.4.16

ansible installation method: pip
ansible-lint installation method: pip

Desired Behaviour
Ansible-lint works on a playbook that has defined unsafe variables with !unsafe.
Actual Behaviour (Bug report only)
An error occurs when running ansible-lint on the following case:
In this case, a docker inspect command is run which also uses double bracket notations. Therefore these need to be escaped with the !unsafe tag.
- name: register whether the container has finished
  command: "docker inspect -f {{inspect}} {{container_name}}"
  vars:
    inspect: !unsafe '{{.State.Running}}'
  register: container_running

Output:
Traceback (most recent call last):
  File "/project_folder/.venv/bin/ansible-lint", line 11, in <module>
    sys.exit(main())
  File "/project_folder/.venv/local/lib/python2.7/site-packages/ansiblelint/__main__.py", line 122, in main
    matches.extend(runner.run())
  File "/project_folder/.venv/local/lib/python2.7/site-packages/ansiblelint/__init__.py", line 243, in run
    skip_list=set(self.skip_list)))
  File "/project_folder/.venv/local/lib/python2.7/site-packages/ansiblelint/__init__.py", line 139, in run
    matches.extend(rule.matchtasks(playbookfile, text))
  File "/project_folder/.venv/local/lib/python2.7/site-packages/ansiblelint/__init__.py", line 66, in matchtasks
    yaml = ansiblelint.utils.parse_yaml_linenumbers(text, file['path'])
  File "/project_folder/.venv/local/lib/python2.7/site-packages/ansiblelint/utils.py", line 527, in parse_yaml_linenumbers
    data = loader.get_single_data()
  File "/project_folder/.venv/local/lib/python2.7/site-packages/yaml/constructor.py", line 39, in get_single_data
    return self.construct_document(node)
  File "/project_folder/.venv/local/lib/python2.7/site-packages/yaml/constructor.py", line 48, in construct_document
    for dummy in generator:
  File "/project_folder/.venv/local/lib/python2.7/site-packages/yaml/constructor.py", line 398, in construct_yaml_map
    value = self.construct_mapping(node)
  File "/project_folder/.venv/local/lib/python2.7/site-packages/ansiblelint/utils.py", line 518, in construct_mapping
    mapping = Constructor.construct_mapping(loader, node, deep=deep)
  File "/project_folder/.venv/local/lib/python2.7/site-packages/yaml/constructor.py", line 208, in construct_mapping
    return BaseConstructor.construct_mapping(self, node, deep=deep)
  File "/project_folder/.venv/local/lib/python2.7/site-packages/yaml/constructor.py", line 133, in construct_mapping
    value = self.construct_object(value_node, deep=deep)
  File "/project_folder/.venv/local/lib/python2.7/site-packages/yaml/constructor.py", line 88, in construct_object
    data = constructor(self, node)
  File "/project_folder/.venv/local/lib/python2.7/site-packages/yaml/constructor.py", line 414, in construct_undefined
    node.start_mark)
yaml.constructor.ConstructorError: could not determine a constructor for the tag '!unsafe'
  in "<unicode string>", line 18, column 14:
        inspect: !unsafe '{{.State.Running}}'

if you're getting a stack trace, output of
ansible-playbook --syntax-check playbook
This yielded no results.

The Ansible YAML parsing has moved on a long way since we added linenumber parsing to ansible-lint.
Looks like ansible-lint might need some refactoring to better reuse some of those changes - this is not a trivial fix.

Would it be possible to use ansible's own constructor?
from ansible.parsing.yaml import AnsibleContructor as Constructor

Or something like that.

Yeah, that was the first thing I tried, and it quickly took me down a rabbit hole as I'd need to change to the AnsibleLoader, and then I don't know how I'd achieve what is currently supposed to happen (obtaining linenumbers) but there is a possibility it might come for free with moving over to Ansible's YAML parsing - I just don't have time to look into this in as much depth as is needed right now, but I acknowledge it's definitely a bug!

This seems to be fixed in master now, probably through #332.
Using 3.5.0rc1, I get:
$ ansible-lint play.yml
[ANSIBLE0012] Commands should not change things if nothing needs doing
play.yml:4
Task/Handler: register whether the container has finished

Thanks for fixing this. The pull request looks very nice!

Fixed in 3.4.22, now available through pip


Environment Variables parsed away

I tried to specify variables key as described in Conda docs: Setting environment variables:

Further, you can declare environment variables in the environment.yml file as shown here:

name: env-name
channels:
  - conda-forge
  - defaults
dependencies:
  - python=3.7
  - codecov
variables:
  VAR1: valueA
  VAR2: valueB

After submission the new build does not include the variables. Activating an environment built in this way does not set environment variables.

Thats interesting, I wonder when the variables feature was added to conda, I was not aware of it. This should be a straightforward fix. @anirrudh can you take a look at this.

Yep, sorry just got around to this notification while looking at issues.

cc: @dharhas @costrouc
Hey @StefanBrand, I'll be working on getting this branch merged, but we have environment variables now working - they are not parsed away! Thanks for catching this and bringing it to our attention.
We are working on merging some major changes, but the branch is called feature/add-variable-parsing. Feel free to use it and let us know if you run into anything -- I'll close this issue once the PR is merged.

@StefanBrand this has been merged. Try running from the latest main. Closing Issue!


Add troubleshoot ExpansionTile with PageStorageKey when it has descendant scrollables

fixes https://github.com/flutter/flutter/issues/97583
details: https://github.com/flutter/flutter/issues/97583#issuecomment-1092858939
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@goderbauer
Thanks for the feedback, updating this soon.


is there planned nintendo ds / 3ds / wiiu support

so i like chimera for the idea of console, despite some issues i had (home button stops working suddenly sometimes)
but there are a lot of nintendo ds and 3ds games i wanna replay and chimera doesnt have that option rn, so i was wandering if it is planned or not

This is not currently planned. DS and 3DS are complex to configure in a good way for out of the box play. Many games require a mouse for example. The dual screens are also an issue. Wii U has similar issues plus has the added complexity of not being available in RetroArch. These platforms are left for users to add on their own for now.
I am open to suggestions though.

Closing as not planned.


Event created with specific image displays random image in the app

Describe the bug
Though user created the event with a specific image, a random image is displayed in the app.
To Reproduce
Steps to reproduce the behavior:

Create a single-day event
Choose a specific image while creating the image
Publish the event.

Actual Result
Though the user created the event with a specific image, a random image is displayed in the app.
Expected behavior
The random image should not display in the app if the user has his specific choice added in the Event
Screenshots
Event created with the image

The image displayed in the app

@Priyaravi8 May I know this is https://api-test.rokwire.illinois.edu/manager-events/user-events/event/5fb2aea64caae30007d4ca75?
It seems this event has not shown.

@Priyaravi8, we tried to duplicate this, but are not able to. Could you please share the image file that used? Thanks.


!logs {procname}

All UQCS stuff is managed by supervisor on our server. Supervisor provides an XML-RPC endpoint that we can use to manage the processes on the server, and is used by cesi.
Using the XML-RPC endpoint, we should be able to view the tail of the logs of a given process. See hookbot (https://github.com/UQComputingSociety/webhooks) for an example of using the XML-RPC endpoint (in Python).
Supervisor docs are here: http://supervisord.org/api.html

JS XML-RPC lib: https://www.npmjs.com/package/xmlrpc


Tagging Support

When tagging is enabled the user can select from pre-existing tags or create a new tag by picking the first choice which is what the user has typed into the search box so far.
This will allow people to select from already available choices (or tags) and if required create a new tag on the fly.

Already implemented


Rename LOGGER to log

Codecov Report

Merging #285 (2a80795) into master (f263887) will increase coverage by 0.02%.
The diff coverage is 42.92%.

@@             Coverage Diff              @@
##             master     #285      +/-   ##
============================================
+ Coverage     35.92%   35.94%   +0.02%
+ Complexity      666      665       -1
============================================
  Files           220      220
  Lines         11208    11208
  Branches        276      276
============================================
+ Hits           4026     4029       +3
+ Misses         6928     6924       -4
- Partials        254      255       +1

Flag
Coverage Δ

java
62.61% <42.92%> (+0.07%)
:arrow_up:

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

Impacted Files
Coverage Δ

.../client/java/impl/consumer/FifoConsumeService.java
0.00% <0.00%> (ø)

...ent/java/impl/consumer/StandardConsumeService.java
0.00% <0.00%> (ø)

...cketmq/client/java/impl/consumer/ConsumerImpl.java
73.72% <20.00%> (ø)

...q/client/java/metrics/MessageMeterInterceptor.java
75.00% <20.00%> (ø)

...client/java/hook/CompositedMessageInterceptor.java
87.87% <33.33%> (ø)

...mq/client/java/impl/consumer/PushConsumerImpl.java
33.33% <33.33%> (ø)

...t/java/impl/consumer/PushSubscriptionSettings.java
60.34% <33.33%> (ø)

...java/impl/consumer/SimpleSubscriptionSettings.java
61.90% <33.33%> (ø)

...cketmq/client/java/impl/producer/ProducerImpl.java
32.44% <33.33%> (ø)

.../rocketmq/client/java/message/MessageViewImpl.java
76.10% <33.33%> (ø)

... and 15 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Ensure saving offline heartbeats finish before main thread exit

This PR adds a channel to the goroutine to make sure it finishes before main thread exit.

It's already being testes by TestSendHeartbeats_ExtraHeartbeats in integration tests.

It's not fully tested because TestSendHeartbeats_ExtraHeartbeats was passing before this change. It's best when fixing bugs like this to have a new test that fails without the fix, and passes with the fix.


[Java]: Database connectivity Client/Server

Description
Brief discussion about the Java as a Database front end.
The JDBC API
Domain
Java_Domain
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign documentation


Added highlevel ability to provide dashArray and dashPhase

Hey @Hopding ,
To make this awesome lib even more awesome, and easier to use, I've create this PR.
This PR provides ability to simply pass the dashArray and dashPhase as parameters to the following page.draw functions:

circle,
ellipse,
line,
rectangle,
square,
svgPath

Please, let me know if needs any modifications.
Cheers!

@Hopding Awesome! Thank you!


fixed kubeflow memory and CPU limits

Issues Resolved by this Pull Request
Crashloopbackoff for istio-pilot, istio-policy and istio-telemetry due to memory issue.
Fixes #
Resolved by increasing memory to 256Mi
Description of the Solution
increased memory 256Mi in 'deploy_kubeflow.yml' under
platforms/roles/kubeflow/tasks location.
Suggested Reviewers
@j0hnL

need some more changes


refactor rpc and remove wallet

remove wallet module
refactor formatter interface to accept more parameters
update send transaction test to new api

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@6681a9c). Click here to learn what that means.
The diff coverage is 91.66%.

@@            Coverage Diff             @@
##             develop     #167   +/-   ##
==========================================
  Coverage           ?   97.41%
==========================================
  Files              ?        9
  Lines              ?      232
  Branches           ?       42
==========================================
  Hits               ?      226
  Misses             ?        5
  Partials           ?        1

Impacted Files
Coverage Δ

packages/ckb-sdk-rpc/src/resultFormatter.ts
97.22% <100%> (ø)

packages/ckb-sdk-rpc/src/paramsFormatter.ts
88.23% <80%> (ø)

Continue to review full report at Codecov.

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


Lake loader stuff

This PR adds a scaleToZero feature.  It is essentially a way to disable the application, but without destroying the helm chart.
This is needed for graceful shutdown to work nicely.  Without this change, kubernetes will un-bind the service account from the deployment while the pod are still shutting down.  If that happens, the pods cannot access the GCP services they need to complete the graceful shutdown.

Will need to create a new chart version for this - update CHANGELOG, and https://github.com/snowplow-devops/helm-charts/blob/322876f9da297ffe6e2cab3cc520f49e25c7983b/charts/service-deployment/Chart.yaml#L4

I did the required admin:

Github issue for each commit
Changelog
Bumped the version number

Adding @jbeemster just for a quick sanity check since the new autoscaler API requires minimum k8s 1.23+, any concerns with this?

@jparavisini @istreeter have we tried setting HPA settings to 0 and 0 instead of 1 and 20 as an example?  No issues with this PR as such but wondering if there is an easier way to achieve the same thing!

HPA does not let you set max replicas to zero.

Makes perfect sense then thanks for clarifying @istreeter !


Find joint angles to closest end effector pose in workspace

Is there a way to use this library to find the joint angles for the closest end effector pose in the workspace compared to a target end effector pose?

@cambel
Hey, my point is many desired end effector poses are outside the ur workspace and this library returns “None”.
for example, I recorded with motion capture trajectories of human wrists manipulating objects and tried to use this library for IK, but it just kept returning None

@richardrl

many desired end effector poses are outside the ur workspace

I am still not sure what you are trying to do.
The libraries here consider the poses from the UR's base_link to the tool0.
If you want to consider a different link as the end-effector, such as the tip of the gripper. You are tracking the gripper pose for your given task and you want to know the IK solution for some pose in tcp, then, you need to compute the transformation between such gripper -> tool0  and only then use this library to compute the IK.

I am trying to servo the end effector to the closest end effector pose to a target end effector pose (where closeness is defined by a L2-norm metric over the transformation matrices) within the kinematic workspace of the UR3e.
If I give you some pose $X \in \mathbb{R}^{4 \times 4}$ if the pose is not in the workspace, the current library returns None. That is not my desired behavior, I want to return a pose within the workspace that is closest in L2 norm to my desired pose.
By workspace, I mean the set of all reachable task space poses. Many poses are not reachable, i.e. there is not joint angle that exactly equals that pose via forward kinematics.

Right, that is not within the scope of this library. It only works within the robot's workspace.
You need to compute that "reachable" pose first, then use this library, otherwise you won't get an IK solution.
If you find a suitable solution for your problem, feel free to document it here in case other people would like to follow in your steps.

I see - do you know of any works that try to project the pose into reachable workspace - any references?

I don't know any library that does that but just projecting it does not seem too difficult if you manually define the reachable workspace of the robot and then compute the closest point to the given pose.
However, it seems to me that projecting the pose into the reachable workspace is not what you really want to be doing. It does not seem particularly useful IMHO.

I recorded with motion capture trajectories of human wrists manipulating objects
From the description of your application, you have a mismatch of perhaps the reachability of the UR3e robot to that of the human arm. Instead of projecting the given poses to the closes in the reachable workspace, it may be more useful to manipulate the entire trajectory to lie within the reachable workspace by for example, uniformly scaling the distance of each point to the shoulder of the human/robot.


Tensorflow Lite speed -> mobile vs desktop

As I am currently building a mobile application with pretty heavy duty object detection, I switched to tensorflow lite as a way to speed up computing. Everything is lightning fast as expected, but I am seeing 99% of the computation time being overpowered by the interpretation.invoke() function which is obviously critical for inference as described in https://www.tensorflow.org/api_docs/python/tf/lite/Interpreter.
With over 1400 images to loop my model through, I am seeing the invoke() function take roughly 1 sec for each image extending the total inference time to over 20 min. One extremely important detail to note is that this is run through xcode on my desktop computer and not a mobile device. After extensive research, I have discovered to expect better performance on a mobile device with tensorflow lite.
My question is - are the performance gains from tensorflow lite on a mobile device exponentially faster than desktop? If my run time on desktop is over 20 min, is it even possible for this to decrease to a few minutes on mobile? Tons of forums out there are concluding tflite is obviously faster on mobile than desktop, but I am looking for logistics here. Generally HOW MUCH FASTER? Please help! Thanks!!

Great questions here @jrash33. As you rightly pointed out you should expect pretty good speedups with an optimized TensorFlow Lite model even on a commodity mobile phone.
Now, coming to how much that all really depends on the phone you are running inference on and also the type of model you are using.
Type of model because if you are using the TFLite GPU Delegate then you need to ensure your model does not have dynamic-shaped inputs and also, you would need to minimize the number of ops not supported by the GPU delegate. TFLite GPU delegate works the best when your model is generally in float16 but they have recently added support for integer models. Phones come with different GPU chipsets sometimes and that can vary the extent of speed-up.
Here'san example that shows how to carefully design the inputs and outputs in your TFLite model if you would run it on GPU delegate.
That said, if you want to get an idea of how faster your model would run on a mobile device I would suggest checking out the Benchmarking tool. It gives you a fair idea of different inference statistics such as memory usage, average inference latency, and many more.
Now, from my personal experience, I have noticed at least 10x speedups on style transfer models (with a MobileNet backend) and 5x speedups on MobileDets all on my humble Realme Pro 2.
I hope that helps.

this helps a ton! Thank you - I will checkout the benchmarking tool and try to dig deeper into different types of models. I am not too familiar with tflite and am actually just using the tflite file provided to me in autoML from google cloud. I found some scripts online to help me execute inference in swift but was disappointed in the performance as explained earlier. It sounds to me from your response like running this on mobile should fix the issue to the extent that I had hoped. I will definitely be optimizing the model and using the benchmarking tool you recommended, but for now, I think I am good to proceed. Thanks!

Sure thing! Please feel free to continue posting your questions here. I will try my best to answer them from my experience.

The open-gl based GPU delegate can be runnable on the desktop. That might help accelerating the speed on the desktop. @terryheo

Could you point to some relevant resources that talk about this? @abattery

Actually it's OpenCL not OpenGL. You can build x86_64 binary with OpenCL supports.
https://www.tensorflow.org/lite/guide/build_cmake#opencl_gpu_delegate
FYI, it's still experimental and only verified with NVidia CUDA OpenCL 1.2.

Interesting. Any information on the amount of speedup one might expect? I understand this is highly model specific. Also, what would be the recommendation for accelerating inference with commodity CPUs?

Regarding GPU performance, check this article.
https://blog.tensorflow.org/2019/01/tensorflow-lite-now-faster-with-mobile.html
Also you can apply quantization which usually gets at least 2x improvement.
https://www.tensorflow.org/lite/performance/model_optimization
Also it's worth to read.
https://www.tensorflow.org/lite/performance/best_practices

@terryheo I am aware of the above-mentioned stuff. If I am not wrong all of them are highly mobile-specific. Since that is not under question (I am quite aware of TensorFlow Lite's capability of accelerating mobile-specific models as I already mentioned here) I wanted to understand the practices one should follow when the inference runtime is based on common NVIDIA GPUs or Intel CPUs and so on.
For GPUs, you mentioned OpenCL related building here. But what if I am using an Intel CPU during inference? Are there any recommendations for that? Also, in these cases what is the typical speed-up one might expect?
Let me know if anything is unclear.

@terryheo any further inputs based on my comments above?


Minor typo fixes

Description
Very minor spelling / typo updates in docs or comments.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x] Documentation only change (no code changed)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[ ] I have run the pre-commit checks with pre-commit run --all-files (see CONTRIBUTING.md instructions to set it up)
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Now it's all just typos 😃

Thanks for updating the PR


Issue with outlined variant of Selects

Hi,
there is an issue with the label in the outlined variant of the Select component:
https://codesandbox.io/s/friendly-sun-ugf8b
Any idea?
Many thanks!
Mauro

I also ran into this so I use the TextField component and pass select props
https://material-ui.com/components/text-fields/#select
                <Field
                  component={TextField} //imported from formik-material-ui
                  id='FieldID'
                  name='selectName'
                  label='Select box'
                  variant='outlined'
                  select //make this field into a select
                  children={
                    //children menu items
                  }
                />

There are 2 ways to address this.
Either start with a null value or add shrink  to the input label props
<InputLabel shrink={true}>Age</InputLabel>
            <Field
              name="age"
              component={Select}
              inputProps={{
                name: "age",
                id: "age-simple"
              }}
            >
              <MenuItem value={null} />
              <MenuItem value={10}>Ten</MenuItem>
              <MenuItem value={20}>Twenty</MenuItem>
              <MenuItem value={30}>Thirty</MenuItem>
            </Field>

There are 2 ways to address this.
Either start with a null value or add shrink to the input label props

Using the shrink attribute does not work in any way. Still rendering the label inside the border.
Using the null value solution does not work with typescript. The value has to be string | string[] | number.

Seems the only solution here is to use what @Edward-Adamson suggested:
const OutlinedTextField = (props) => {
  return <TextField variant="outlined" {...props} />
};

<Field
  component={OutlinedTextField}
  name="region"
  label="State"
  select={true}
  inputProps={{
    id: 'search-region',
  }}
>
  {options}
</Field>

I managed to get it working properly like this.
import { MenuItem, InputLabel, FormControl } from "@material-ui/core";
import { TextField, Select } from "formik-material-ui";

<FormControl variant="outlined">
  <InputLabel>Status</InputLabel>
    <Field
      component={Select}
      name="status"
      label="Status"
    >
      {options}
    </Field>
</FormControl>

I think its pretty similar to yours.


squash PRs instead of merge commits

        # i think we want to do a squash so its easier to auto-revert entire
        # PRs, instead of detecting merge commits, then picking the right parent
        # for a revert.  this way—with squash—every commit aside from hotfixes
        # will be entire PRs

Merge commit (old way):

Squash commit (proposed way)

if anyone has any hard opinions either way, i'd love to hear them.  this should make reverting out PRs marginally more easy to do automatically, but will make git log history a little more flat

Relevenat API endpoint

@amoffat That didn't work ;( Author of the squashed commit is the one who created PR, not Chaos

@phil-r Interesting.  The commits are squashed into 1 commit, but github retains the original author and only adds chaosbot as the committer on the commit:
12:33:57 $> git show --format="author: %an, committer: %cn" e7cd87 | head -n 1
author: Anthony Alves, committer: Chaos

In github it show as "<author> committed with <committer>":

This is probably ok I think?

@amoffat yeah! didn't know about it!


Add test runner to github

Ensure that the test framework runs tests for each pull request such that failing tests disallow merging.

Added Github actions that trigger for every build request.