add facebook ads

{Write down the list of resources or topics you have added.}
{If you have added a resource, please explain your experience with the resource in context of why you have added it.}

By submitting this pull request I confirm I've read and complied with the below requirements.
Failure to properly do so will just result in the pull request being closed and everyone's time wasted.

[x] I have read the Contribution guidelines and made sure my pull request follows it, specifically the part about promotions and markdown format.
[x] Any resources I am adding in this PR have been used by me and I can personally vouch for their quality.
[x] This pull request has a descriptive title. For example, Add [Resource Name] or Add [Topic Name], not Added new entries or update.
[x] I have double checked that I have only updated the minimum files that are required by this PR.
Add more topics and resources #1

It is a free course
On Wed, Dec 4, 2019, 2:23 PM Pragati Verma<EMAIL_ADDRESS>wrote:

Closed #13 https://github.com/NJACKWinterOfCode/learn-for-free/pull/13.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NJACKWinterOfCode/learn-for-free/pull/13?email_source=notifications&email_token=AJ23PLGAJDY23BJ5AUCASXLQW5VZNA5CNFSM4JVE5K52YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOVIHUDUI#event-2853126609,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ23PLAFSH6FNP3SDQ3JZVTQW5VZNANCNFSM4JVE5K5Q
.

I can see the description reading that you purchased this course on 4th December 2019.

I can see the description reading that you purchased this course on 4th December 2019.

It is a free course but when i registered on it then it will show purchase on blah blah

I am sorry I can't accept it, doesn't align with the idea of this project.

okay, so change the image and description accordingly and then resubmit for evaluation.

Okay
On Wed, Dec 4, 2019, 3:04 PM Pragati Verma<EMAIL_ADDRESS>wrote:

okay, so change the image and description accordingly and then resubmit
for evaluation.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/NJACKWinterOfCode/learn-for-free/pull/13?email_source=notifications&email_token=AJ23PLHMV6DMLD4D37XJHBLQW52QTA5CNFSM4JVE5K52YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEF4KO7Q#issuecomment-561555326,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ23PLHZH253NNFAW7P2NY3QW52QTANCNFSM4JVE5K5Q
.


Sliders (Owl Carousel, slick, etc.) can't set proper width on iOS

Just an iOS issue so far:
Calling a carousel/slider like Owl Carousel or slick slider inside the iframe will break the width and will cause and horizontal scrollbar width a with around 1000+ px on the iPhone. Works fine with Android, though.
This might be a mobile Safari issue, but is there any way to re-init the iframe width or something?

Can you post a simple test case I can look at. What height calculation method are you using?

Closed due to non-response. Will reopen if I get more details.

Hi David, I think my team have stumbled on this same issue. We're running the latest version (v3.5.0). Once a carousel has been initialised the iframe reports an creased width and seems to continuously expand. It only happens on iOS - versions 7, 8 & 9 tested.
I've recreated a simplified version here using Slick Carousel (we found the same problem with Owl Carousel).
I've tried autoResize, maxWidth, heightCalculationMethod (all), widthCalculationMethod (all) with no success.
The only slider we've found that work's is Flickity (I believe it's because it doesn't create an element with the width of the slides stacked i.e. one huge element, but instead uses absolute positioning).
I've included a link to a copy of the demo files on the page if it helps.
Any help will be greatly appreciated .

Thanks for the example, will take a look when I have a spare moment. If you find the solution I'd welcome a PR.

owl carousel
Webpage:
http://owlgraphic.com/owlcarousel/
Creates a div like this:

Slide 1

Slide 2

Slide 3

Slide 4

Hello.
I have the same issue: OWL carousel in iFrame doesn't work. It takes the whole carousel width - it doesn't resize it on iOS (iPhone and iPad).
You can check if you open live website http://goo.gl/6JKwtM on iPhone and iPad.
Best,
stojsins

Have you tried using the tagged element size calculation method and tagging the slider container?

Hello David.
Could you be more precisely please?

https://github.com/davidjbradshaw/iframe-resizer#heightcalculationmethod

same behaviour it seems with with bxslider: https://github.com/stevenwanderski/bxslider-4
inside an iframe.
we use the tagged element size calculation method both on the slider container and an outer container (we dynamically add more content outside the slider on slide change events).
so anyone has a solution to this problem so far?

used another slider mode 'fade' for the bxslider and this seemed to resolve the issue (tested on mobile safari 9.1 / iphone 6):
http://bxslider.com/options

Same behaviour - no solution?

Depends on the slider, if it creates very wide div's that code should be changed.

First item is about 1120px:

Do you have a solution for the bug?

I can confirm that using Slick in conjunction with iFrame Resizer leads to the width of the iframe extending past the viewport width on iOS.
There's good news though, adding width: 1px; min-width: 100% to the body tag inside the frame fixes it.
Here's a reference to a similar iOS iframe issue: http://stackoverflow.com/questions/23083462/how-to-get-an-iframe-to-be-responsive-in-ios-safari

@bro-strummer You just saved my friday night, thanks!!

@bro-strummer Thank you so much!

@bro-strummer my hero!

@bro-strummer I've just read your comment and I have to thank you man! you saved my day!

@zitrusblau Thanks, it working for bxSlider on ios. I checked on iphone 6 and 5s.

yaaaay solves the problem. Thanks!!!

@bro-strummer's solution works! Thank you so much!

Closed @bro-strummer css fix is now in the docs


Fix outlier removal in column sizer

The outlier removal in the column sizer does not work correctly. For example:
var sizes = [16, 150, 10, 8]
const average = sizes.reduce((a, b) => a + b) / sizes.length;
const biggest = sizes.reduce((a, acc) => (a > average * 2 ? acc : Math.max(acc, a)));

# average: 46
# biggest: 10

The reduce function does not select the maximum of all sizes without the outliers. To fix it, I propose to use a filter for the outliers before applying reduce to determine the biggest size. I think the outlier removal is not super useful for tables that only contain very few items. Therefore, I added a check to only apply it if there are at least 5 sizes available.
Closes #312

What is this SonarCloud bot and why is it blocking the merge because of "code duplication" in a test file?


Docs: For the github server action, explain folks will have to read their logs or use tmate if things go wrong with the docker container creation

Will be closed by https://github.com/SuffolkLITLab/docassemble-AssemblyLine-documentation/issues/379

Closed in favor of #811 and https://github.com/SuffolkLITLab/docassemble-AssemblyLine-documentation/issues/379


x/playground: Add keyboard event listener for commenting

It's not a big deal. but inconvenient.
It would be nice to have keyboard commands rather than manually commenting them in the playground.

What did you do?
I tried trigger a keyboard event ( ctrl/cmd + / ) to comment out some code.

What did you expect to see?
Commented out code
What did you see instead?
Nothing change.

Duplicate of #39152

Oh my bad.
I'm interested in trying to fix this.
I’ll give you pr soon.


Clean up the requirement package

Clean up the requirement package   @chansigit @Sichao25 @YifanLu2000

adjustText
anndata>=0.8.0
colorcet>=2.0.1
cvxopt>=1.2.3
csbdeep>=0.6.3
descartes
dynamo-release>=1.3.0
folium>=0.12.1
geopandas>=0.10.2
gpytorch
ipywidgets>=7.7.1
kneed>=0.7.0
kornia>=0.6.4
leidenalg>=0.10.0
loompy>=3.0.5
mapclassify>=2.4.2
matplotlib<=3.5.3
mpi4py
nbconvert
networkx>=2.6.3
# ngs_tools>=1.6.0
numba>=0.46.0
numpy>=1.18.1
opencv-python>=<IP_ADDRESS>
pandana
pandas>=0.25.1
paste-bio>=1.4.0
plotly>=5.1.0
POT>=0.8.1
psutil>=5.6.3
pynndescent>=0.4.8
pysal>=1.14.4
pyro-ppl
python-igraph>=0.7.1
pyvista>=0.39.1
scipy>=1.0
scikit-image>=0.18.0
scikit-learn>=0.19.1
seaborn>=0.9.0
setuptools>=58.0.4
Shapely>=1.8.0
statsmodels>=0.9.0
tensorflow
tqdm>=4.62.3
torch
trame>=2.2.5
trame-server>=2.7.0
typing_extensions>=4.0.1
umap-learn>=0.5.1
vtk>=9.2.2

Here are a few related issues to follow up on. Please correct me if anything is wrong.

[ ] Matplotlib requirement is out of date.
[ ] Pyvista renamed UniformGrid to ImageData in 0.40.0 and deprecated UniformGrid in 0.42.0.
[ ] python-igraph has been renamed to igraph on PyPi.
[ ] .A .H attributes were removed from the SciPy 0.14.0.
[ ] Trame and Trame Server seem to be used mainly in the Spateo Viewer. Should we remove them here?
[ ] Existing tools like Scvelo and scVI-tools restrict the usage of the latest version of NumPy (2.0). We should consider doing the same.
[ ] I feel like packages such as paste-bio, cvxopt, folium, pandana are likely dependencies of TensorFlow. We might be able to remove them from the requirements unless specific versions are needed to avoid conflicts. Need more tests on this.
[ ] We could discuss making rarely used packages optional dependencies. For example, descartes


DBZ-2952 Remove Postgres replication slot validation

(Maybe this is part 1 to quickly fix user's issues, I'm working on a Testcontainers integration test in order to check another way of properly testing the replication slot assignments, even in config update cases).
closes https://issues.redhat.com/browse/DBZ-2952

@gunnarmorling PR should be ready now. Test failure is unrelated.

@gunnarmorling PR should be ready now. Test failure is unrelated.

Thx, @rk3rn3r!

Thx, @rk3rn3r!


Link to hayagriva examples

When adding to a bibliography, I often keep a tab open to the example entries in the Hayagriva repository. I've mostly memorised the basics now, but still refer back to the examples from time to time.
I don't suggest copying the examples, but it would be worth linking to them to in this book. I find them really useful and think others would too.

Thank you!


Initial Queues Support

Pulling over the commits from my private fork.
Also added two new commits to clean up some temporary docs, and adjust to the latest desired wrangler config for queues.
I'm hoping to get this cleaned up and merged so that I can begin integrating it into the wrangler queues branch.

Thanks @mrbbot! Just fixed those warnings.

Whoops sorry, I probably did something silly to screw up the package-lock. I'm using node version 18.7.0 and npm version 8.15.0.
I'll have to get this merged/released before I can update the dependency in wrangler branch, right? Unless there is some way to do a custom miniflare release/version (and then update my wrangler branch to point at that?).

Thanks @mrbbot, l think it makes sense to do a regular miniflare release.
I believe I have fixed the package-lock issues, and added the warning log message you suggested.
I'm not sure what's up with the failing test on Windows, is that flakiness or did I break that test somehow?

Yes, looks great, you are too kind!


Fixed dropped notifications right after setup.

Before the notification observable was emitted after the Client Characteristic Configuration Descriptor was written. If the characteristic notifies right after the CCC descriptor write then this notification/indication may be lost. Now the notification observable is emitted right after the system sets interal monitoring of the characteristic and CCC descriptor is written afterwards.
Now with writing disable notification/indication value to CCC descriptor.
Better version of commit ab4943c2f6b5ba5b2b874ff3c6c45d36b988573e

Having studied the usage a bit and looked into possible implementations I agree that this functionality should be incorporated into the RxAndroidBle


Remove tooltips "print to pdf" and "download source file" in sphinx book theme ?

Describe the bug
Hello,
I don't want tooltips displayed for "print to pdf" and "download source file". Is there a way to remove them ?
Thanks.
Reproduce the bug
No bug.
List your environment
No response

This is no bug, "use_download_button": False, should do the job
https://sphinx-book-theme.readthedocs.io/en/latest/reference.html


data-holder and client-cli Dockerfiles missing /keystore

Need this line:
ADD keystore /keystore

Run the images that are built without /keystore (from DockerHub):
docker run -p 8383:8383 -v /your/local/path/to/java-artefacts/data-holder/keystore:/keystore consumerdatastandardsaustralia/data-holder:x.x.x
Where x.x.x is the version, say, 1.1.1

Added in v1.2.0


buildAndDeploy: block on PostProcessBuild

container deploys block on PostProcessBuild [ch334] WIP
WIP still need tests and to do this for LocalContainerBuildAndDeployer but wanted to make sure I'm on the right track.
don't quite know my way around concurrency-land, left some Q's in comments inline (look for ~~~).
In particular @nicks I now lock sbd.deployInfoMu when retrieving deploy info's off the struct, but the map value is a pointer, so i don't really write back to the map as much as just modify the value behind the pointer... (i lock a mutex on the *DeployInfo itself before I edit it but not before I read from it.) will this explode anything?

ya, this isn't quite right, but there are some standard patterns on how to do this, let me get you a reference impl

take a look at how skylark does this here:
https://github.com/google/skylark/blob/master/example_test.go#L185
basically you have a map protected by a mutex
each entry in the map has a ready channel
the first time you create an entry in the map, you kick off a process that does the fetching, then closes the ready channel when it's done
if the entry is already in the map, you wait on the ready channel
i can walk you thru it in more detail if that would help

Resolved in #404


added SkullRec

thanks, this looks nice, I uploaded the model checkpoint to https://github.com/Project-MONAI/MONAI-extra-test-data/releases/download/0.8.1/skull_rec_best_metric_model.pth, could you please remove the file from this PR (as it's too large to store in this repo) and update the readme to guide the users for downloading?
also if possible please specify the required software+version (and perhaps the minimal hardware requirements) in order to run the code.

Will do. Thanks for the update.


Add option (default?) to position/create windows & panels by their border edge (excluding shadows)

Under the default theme, when I create a button, it seems that there is still room for the button, but the button cannot display full text. When I create a window, the actual window size is 30 pixels smaller than expected. This is confusing. I hope to reduce the shadowwidth of the default theme to solve this problem.

I don't really want to change the default theming size, but I would like to add an alternative way to create a window or a panel that has a certain interior size (i.e. excluding title bars, borders, shadows. Probably a little shim class called something like InteriorDimensions() that you could pass in and it would take the input size and output final dimensions that worked based on the current theming.
The text rendering getting clipped too tightly is a separate bug that has needed fixing for a while.

我真的不想更改默认主题大小，但我想添加另一种方法来创建具有特定内部尺寸（即不包括标题栏、边框、阴影）的窗口或面板。可能有一个小的填充程序类，你可以传入类似的东西，它将采用基于当前主题工作的输入大小和输出最终尺寸。InteriorDimensions()
文本呈现被剪得太紧是一个单独的错误，需要修复一段时间。

The window is smaller than the screen. Black edges appeared.

I don't really want to change the default theming size, but I would like to add an alternative way to create a window or a panel that has a certain interior size (i.e. excluding title bars, borders, shadows. Probably a little shim class called something like InteriorDimensions() that you could pass in and it would take the input size and output final dimensions that worked based on the current theming.
The text rendering getting clipped too tightly is a separate bug that has needed fixing for a while.

The window is smaller than the screen. Black edges appeared.

I wanted to arrange the two windows tightly, but there was a 30px seam.

Small buttons cannot accommodate enough text

Resolved in #579


[Tests-Only] Refactor interactions for requestOcsTest file

This PR adds the interactions related to group tests into the tests/requestOcsTest.js file.
Related Issue:

#500

can we move the interactions into the corresponding it?

@individual-it Done


update dependencies

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

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 31.452%

Totals

Change from base Build<PHONE_NUMBER>8:
0.0%

Covered Lines:
39

Relevant Lines:
124

💛  - Coveralls


added windows 11

after reading this article, i confirmed windows 11 is supported

Learn Build status updates of commit b23816b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows/security/threat-protection/security-policy-settings/network-access-remotely-accessible-registry-paths.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


CosmosDB - the 'documentdb' service should be changed to 'sqldb'

DocumentDB is now called 'SQL' in the portal, so we should change the name too

This is actually done as of this morning.


agent: make the global hash lookup table smaller

This is the same change made in #13040, plus subsequent improvements, applied to agent-mode code.
Brought in some tests too, adapted for the different API.

I added a benchmark:
name            old time/op    new time/op    delta
CreateSeries-8     593ns ± 5%     526ns ± 5%  -11.20%  (p=0.002 n=6+6)

name            old alloc/op   new alloc/op   delta
CreateSeries-8      254B ± 1%      177B ± 6%  -30.19%  (p=0.004 n=5+6)

name            old allocs/op  new allocs/op  delta
CreateSeries-8      2.00 ± 0%      1.00 ± 0%  -50.00%  (p=0.002 n=6+6)

Got a lint error from a file I didn't change, so rebased and force-pushed.

Lint error should be fixed by #13343


Warning: date(): It is not safe to rely on the system's timezone settings

Hi
Developing a site using codeigniter 3.04 under php 5.6.
Enabled the logging and put it to 4 just to see if it was working.
After I had fixed the server configuration problems I was surprised to start getting
Warning: date(): It is not safe to rely on the system's timezone settings. You are required to use the date.timezone setting or the date_default_timezone_set() function. In case you used any of those methods and you are still getting this warning, you most likely misspelled the timezone identifier. We selected the timezone 'UTC' for now, but please set date.timezone to select your timezone. in /vendor/codeigniter/framework/system/core/Log.php on line 176
I have added date_default_timezone_set('Europe/London'); which fixed this but had I set the same time zone  in the config.php file $config['time_reference'] = 'Europe/London';
Perhaps this could be one of the first things Codeigniter.php does once it successfully loads the configuration?
Jules

This is a PHP configuration issue, not a CI one.

And to bad to those people who don't have access to php.ini, why bother with the config option at all?

If you don't have access to php.ini, then it's your sysadmin's fault.

Have you used some of the shared hosting out there?
Lucky to be able to get the files up to the server let alone be able to alter php.ini

Have you tried telling the hosting provider that they suck? :)
Again, if you don't have access to the php.ini, it's their fault. They have the responsibility to give you a properly-configured environment.

They have people in charge of their system who don't care about the opinions or needs of other people and who just ignore anybody else’s opinions and reply saying it your fault and you should fix it, they know they are right :)

Well, it's a two-way street - I am in charge here and I don't care about crappy hosting providers. :)
It's a free market - if they provide a bad service, they'll lose customers. It's not CodeIgniter's job to protect their business.

That's true, would be nice if they have not set it to do something about it.
Would also mean that the error log would use the same time zone as that which the rest of the site is running in.

The time_reference setting in the config file is only used by the date helper, and the only part of CI that uses the date helper is the Calendar library. The comment in the config file implies this:

See the 'date helper' page of the user guide for information regarding date handling.

...but the sentence before that implies a broader use in the system, which isn't the case:

This preference tells the system whether to use your server's local time as the master 'now'
reference, or convert it to the configured one timezone.

(I guess the "one" should be removed from that sentence.)
Personally, I avoid using the date helper because it creates discrepancies between the times generated by the date helper and those in the logs (since log_message() doesn't use the date helper or the time_reference setting).
The best thing to do if you can't modify the php.ini is to set the value yourself in your index.php file. The following is an example of this (though probably overly-aggressive in checking the existence of the value and functions): https://github.com/ci-bonfire/Bonfire/blob/develop/public/index.php#L100
Since you're seeing the notices in your logs, you could probably get away with something simpler, like:
if (ini_get('date.timezone') == '') {
    date_default_timezone_set('UTC');
}

Then you would just replace 'UTC' with whatever timezone you wanted. If you're really stuck on using the config value, you would need to call the function later, after the config file has been loaded, which means there's still a chance that something would attempt to use the default timezone before it is set.
A pre_system hook could set the timezone, but I'm not 100% sure that PHP's microtime() is independent of the default timezone setting; the Benchmark class uses microtime() to set the total_execution_time_start and loading_time:_base_classes_start benchmarks before the Hooks class is loaded.

Yes, microtime() (and time() too) is independent from date.timezone - UNIX timestamps don't have a timezone.

I am using a separate file that I keep within config/ directory for php ini settings, but on modified CodeIgniter. Here is an example what it contains:
<?php defined('BASEPATH') OR exit('No direct script access allowed');
/*
 *---------------------------------------------------------------
 * PHP ini settings
 *---------------------------------------------------------------
 */
@date_default_timezone_set('Europe/Sofia');
ini_set('memory_limit', '512M'); // Quite extreme value.
ini_set('post_max_size', '16M');
ini_set('upload_max_filesize', '16M');
ini_set('max_file_uploads', 20);
ini_set('max_input_time', 60);
ini_set('auto_detect_line_endings', true);
if (IS_CLI) {
    ini_set('max_execution_time', 0);
    ini_set('html_errors', 0);
    ini_set('error_prepend_string', '');
    ini_set('error_append_string', '');
    ignore_user_abort(true);
} else {
    ini_set('max_execution_time', 300);
    // http://www.controlstyle.com/articles/programming/text/if-mod-since-php/
    session_cache_limiter('private_no_expire');
}
ini_set('pcre.backtrack_limit', 10000000);

Such a configuration file is more convenient for me than tweaking index.php, because in my applications I have separate index.php files for the front-end and the admin panel.
Do you like this approach with additional configuration file?

Not a bad idea, just add it to the autoload and you're away, wont work for all php configurations but an improvment

Let us say it will contain only @date_default_timezone_set('xxxxxx/xxxxxx');
other settings will be added by a user-developer whenever it is necessary.

Sounds good, commented out on first install

Ok, can we stop this now? This isn't a chatroom.


Kotlin DSL links wrong method

Given the following build script
plugins {
    id("me.champeau.gradle.jmh") version "0.5.1"
}
sourceSets {
   jmh {
   }
}

Expected Behavior
jmh should refer to the JMH sourceSet
Current Behavior
It refers to the JmhPluginExtension on the project object, meaning I have to do the more clumsy
sourceSets {
    this.jmh {
    }
}

I've also encountered this on one of my own extensions that had a dependencies container. Instead of linking that container, the Kotlin DSL linked Project.dependencies {}

Could reproduce the issue with 7.4.1


Landing Page

Develop the Home Page UI

Refer to the Requirements Documentation in Notion:
https://www.notion.so/fightpandemics/Home-Page-a8730bc6d96d4d5bbdce9410fdb28f31

Hi @indirabhowmick I see that the mentioned Home Page UI (#41) ticket has been closed. Is there still a requirement for work on this? Also, I remember it being mentioned in one of the channels on Slack that there may be some more content added to the homepage. If that's the case, is there a document in Notion that covers this? Thanks :smile:

see above :) Would love to know if this is completed or not as well

Here is the updated Requirement:
https://www.notion.so/fightpandemics/Home-Page-a8730bc6d96d4d5bbdce9410fdb28f31
I have created separate tickets in GitHub to create FootNote and add the contents on the Policy Pages which will be launched from Home Page.
https://github.com/FightPandemics/FightPandemics/issues/150
https://github.com/FightPandemics/FightPandemics/issues/149

So should we close this issue?

@nadinejuraschek - Yes Please. Thanks


fix: update tree-kill to the latest version (#2278)

fixes: #2278

... Yeah. We want to merge this to master I think (instead of the mutation switch epic), because maintenance releases are still from master.

I also saw we again introduced ~ and ^ in versions... I think we should change them again to one version and leave it like that, since sometimes as could be seen in this example, package is not updated / installed properly... We would have better control over the versions (also it's time to update inside packages not only main package.json)

I think dependabot is also taking care of updates within packages, no? But it won't make a merge request if the new release is within semver match of the current version.
I think we should keep ~, it should make sure we keep getting patch releases, right?
For kill-tree, could you make a new PR to master? This one has changes from the epic branch 🤷♂

I think it doesn't check internal package.json files... At least I am not sure if it does. Doing PR now :)


Add AWS_EC2_EIP Tags properties

Added tagging for ElasicIP properties.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

https://github.com/awslabs/aws-cfn-lint-visual-studio-code/blob/master/schema/spec-2.15.0.json is definitely significantly out-of-date, and property autocompletion and documentation links on hover are affected by this. Linting itself relies on the CloudFormation Resource Specifications instead, which should be more up-to-date
Property autocompletion and documentation links seem to fallback to AWS::AmazonMQ::Broker if the resource type is missing from spec-2.15.0.json. Fixing this behavior is the immediate priority for this extension, and then finding a way to fully support all future property autocompletion and documentation links will be a more longterm process tracked here


Qemu 6.2.0->7.0.0 causes colima to hard-crash M1 Macbook Pro

Describe the bug
The latest update to qemu (6.2.0 -> 7.0.0) in this commit https://github.com/NixOS/nixpkgs/commits/9d733f3d4975fb3c516a2bf2bf1bc6f50781a553 causes an M1 Macbook Pro to hard-crash (the entire computer restarts) when attempting to start a VM with colima.
Steps To Reproduce
Using an M1 Macbook Pro:

Attempt to start colima
nix-shell -p colima \
-I nixpkgs=https://github.com/NixOS/nixpkgs/archive/9d733f3d4975fb3c516a2bf2bf1bc6f50781a553.tar.gz \
--run 'colima start'

This also reproduces on the main branch of this repository, but does not reproduce on the parent commit to the 6.2.0->7.0.0 upgrade. The following command, referencing the commit from right before the upgrade, will work fine:
nix-shell -p colima \
-I nixpkgs=https://github.com/NixOS/nixpkgs/archive/b161e729ab3f7902645fa4308b87ec97515e2f25.tar.gz \
--run 'colima start'

Expected behavior
Colima should be able to start a VM without any issue.
Screenshots
No screenshot of the failure, as the computer locks up and hard-crashes.
Additional context
When the computer restarts, it prompts submission of a bug report to Apple due to the hard crash. Here is the included crash log that would be sent with that report (all generated by the computer automatically):

panic(cpu 2 caller 0xfffffe001d6e1814): vm_fault() KERN_FAILURE from guest fault on state 0xfffffe6032b30000 @sleh.c:3117
Debugger message: panic
Memory ID: 0x6
OS release type: User
OS version: 21E258
Kernel version: Darwin Kernel Version 21.4.0: Fri Mar 18 00:46:32 PDT 2022; root:xnu-8020.101.4~15/RELEASE_ARM64_T6000
Fileset Kernelcache UUID: 0631AF68D2B8D6FEA30E36D7895D4DB4
Kernel UUID: C342869F-FFB9-3CCE-A5A3-EA711C1E87F6
iBoot version: iBoot-7459.101.3
secure boot?: YES
Paniclog version: 13
KernelCache slide: 0x0000000015cb4000
KernelCache base:  0xfffffe001ccb8000
Kernel slide:      0x0000000016464000
Kernel text base:  0xfffffe001d468000
Kernel text exec slide: 0x000000001654c000
Kernel text exec base:  0xfffffe001d550000
mach_absolute_time: 0x124be03d8d
Epoch Time:        sec       usec
  Boot    : 0x626ff3e3 0x000adbd7
  Sleep   : 0x00000000 0x00000000
  Wake    : 0x00000000 0x00000000
  Calendar: 0x627000a6 0x000c0708
Zone info:
Foreign : 0xfffffe0025c10000 - 0xfffffe0025c20000
Native  : 0xfffffe10000b8000 - 0xfffffe30000b8000
Readonly: 0xfffffe14ccd84000 - 0xfffffe166671c000
Metadata: 0xfffffe6dce89c000 - 0xfffffe6dda808000
Bitmaps : 0xfffffe6dda808000 - 0xfffffe6df220c000
CORE 0 PVH locks held: None
CORE 1 PVH locks held: None
CORE 2 PVH locks held: None
CORE 3 PVH locks held: None
CORE 4 PVH locks held: None
CORE 5 PVH locks held: None
CORE 6 PVH locks held: None
CORE 7 PVH locks held: None
CORE 8 PVH locks held: None
CORE 9 PVH locks held: None
CORE 0: PC=0xfffffe001d5df418, LR=0xfffffe001d5df418, FP=0xfffffe6037f73f00
CORE 1: PC=0xfffffe001d5df418, LR=0xfffffe001d5df418, FP=0xfffffe6115603f00
CORE 2 is the one that panicked. Check the full backtrace for details.
 I unfortunately cannot upgrade my company-managed computer to the beta version of macOS, hopefully they'll release 12.4 with that fix soon.
A question that is maybe better suited for the nix maintainers: for my own information / in the meantime, how does Nix normally deal with platform specific issues like this? Is there any recommended workaround or tactic that I can use to update our company's shared darwin-configuration.nix to use a colima derivation that links against the older version of qemu? Right now we're simply pinning nixpkgs-unstable to the last-known-good commit hash for colima. I haven't contributed to nixpkgs before, would it be worth the time or even possible to update the colima derivation to depend on qemu 6.2.x for darwin users?
If you don't have time to respond to these questions I understand, thank you for all of the work you do to maintain Nix, really appreciate it.

you can create overlay something like:
    pkgsOld = import (fetchTarball https://github.com/NixOS/nixpkgs/archive/9d733f3d4975fb3c516a2bf2bf1bc6f50781a553.tar.gz) { };
    pkgs = import (fetchTarball https://github.com/NixOS/nixpkgs/archive/master.tar.gz) {
        overlays = [
            (self: super: rec {
                qemu = pkgsOld.qemu;
            })
        ];
    };

@peterldowns Looks like you can try use limactl https://github.com/lima-vm/lima/issues/795 they fixed it somehow :)

I was just about to write back in -- looks like this is fixed in lima v0.1.0, which is already released on stable! I have tested and resolved my issue by no longer installing colima from unstable.
Unstable:

colima 0.3.4
lima 0.9.2
Master:
colima 0.3.4
lima 0.10.0

I wouldn't have expected unstable to be on a lower version the master, but hey, easy fix :) Thanks for the help!

🤔 I think we could just upgrade https://github.com/NixOS/nixpkgs/blob/nixos-unstable/pkgs/applications/virtualization/lima/default.nix#L11 and it should solve you issue yes

Yes, I believe all that is needed is for unstable to take the latest v0.10.0 version of lima, as has already been done on the master branch, since using the master branch works correctly for me. I am not sure how to make that change myself but if any maintainers read this, please feel free to close this issue.

I wouldn't have expected unstable to be on a lower version the master

We do have a proper doc with diagrams as to how the various branches and channels work somewhere in the repo but I couldn't find it ...
TLDR: changes go to master first and then they get pushed to each channel automatically.
0.10.0 is already in nixpkgs-unstable and nixos-unstable-small, it'll be in nixos-unstable soon.
https://nixpk.gs/pr-tracker.html?pr=170608

Thank you, that's very useful to know, and I'll keep that pr-tracker handy for the future :)


Update source code template when gererating project by vue-cli

What problem does this feature solve?
npm WARN deprecated<EMAIL_ADDRESS>babel-eslint is now @babel/eslint-parser. This package will no longer receive updates.
This is message when I install packages after generating project by cli, please update template to use @babel/eslint-parser instead of babel-eslint
What does the proposed API look like?
"devDependencies": {
    "@babel/core": "^7.12.16",
    "@babel/eslint-parser": "^7.12.16",
    ...
}

You can try to upgrade @vue/cli-plugin-eslint  to v5
vue upgrade  eslint -t v5.0.0-alpha.4

OK let me try @fangbinwei. Thank you.


this should work too

the game still compiles without the path attribute what is the point of this

Stop doing useless pull requests

its not my fault
my vs code xml extension said that you needed to have that


Program type already present: android.support.design.widget.CoordinatorLayout$1

Program type already present: android.support.design.widget.CoordinatorLayout$1

Maybe you should force your library version，please refer to this answer : Duplicate entry: android/support/design/widget/CoordinatorLayout


Lightweight Alternative for GNOME[ENHANCEMENT]:

It would be great to suggestion of lightweight GNOME alternative cause on 8Gb pc it`s often happen freeze while build something etc. But not xfce4. May be stylish Icewm or so.

Im using Athena in a VM with 4 GB and it works well. In the future other Desktop Env (like XFCE) will be implemented.

@Ygarr new rolling iso was released last night using the XFCE env for the live environment. If you want to try it out it's at https://sourceforge.net/projects/athena-iso/files/rolling/

Thank you @p134c0d3 for the reminder. Closing issue as solved.


Request a Enumerate() overload that accepts int[] vendorIds

Would like to enumerate in one go to minimize overhead a list of device vendors and VIDs.
I understand this could create collision with multiple VIDs possibly sharing PIDs, so perhaps a device struct is in order that contains one VID and one PID per struct then pass device[] MyDevicesStructs to the Enumerateor.

I'm not in a position to add this ATM but if you want to submit a PR I'd def be open to that.


[Linux] [ud1704] EZ-SSH into peers on same LAN fails

This might be a combined p2p and tray problem. I can EZ-SSH into all containers minus ones on the same LAN in an xpeer environment spanning cloud peers across continents. This requires more experimentation and feedback before anyone spends too much time chasing this bug. However, it's serious.

Didn't face this issue in master, will also re-check in production.

works properly in production.


Integration of Youtube-dl with Aria2

Please integrate youtube-dl with Aria2

What do you mean?

Will it be possible that in background youtube-dl will use Aria2 as external downloader to speed up download?

What I'm doing with this UI is simply managing a download queue and pass YouTube URLs directly to the youtube-dl command-line tool and I'm not really planning on changing that. If you have suggestions for youtube-dl you should probably create an issue on the actual project. :)


Fpi joatop

This pull-request adds FPI naturally open, as well as some edits and corrections for seminat and wetland.
For natopen and seminat the updated bookdown files are:
DRAFT_functional_plant_indicators_seminat.Rmd
DRAFT_functional_plant_indicators_natopen.Rmd
For wetland the upated bookdown file is:
functional_plant_indicators_wetland.Rmd

I am not finished with my review yet. Nat-open below the heading "Analyses" needs still to be checked.


Solved Responsiveness issue on the mobile

is it solved now?

@barakkenny  still conflicts bro 🥲, please check if you're missing onto something


If pauseCodebaseIndexOnStart is activated, the indexing cannot be started

Before submitting your bug report

[ ] I believe this is a bug. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that reports the same bug
[ ] I've seen the troubleshooting guide on the Continue Docs

Relevant environment info
- OS: windows
- Continue version: v0.8.56 & v0.9.230 (pre-release)
- IDE version: VSCode 1.94.2
- Model: ollama / llama
- config.json:

Description
When pauseCodebaseIndexOnStart  is activated,
"continue.pauseCodebaseIndexOnStart": true,

the UI does not show a button to start/resume the indexing from pause. It only shows:

I would expect to have a possibility to resume the indexing, like when I manually pause:

To reproduce

Set the setting:

"continue.pauseCodebaseIndexOnStart": true,

Reload window
Open the "More" panel of continue extension

Log output
No response

Thanks for pointing this out. Are you also unable to pause/restart if this setting is not used?

If the setting pauseCodebaseIndexOnStart is set to false, I am able to pause and resume the indexing from the ui as expected.
When the setting pauseCodebaseIndexOnStart is set to true, I found out that I can change the shown status from "Initializing...", when I issue one of the following commands from the command palette:

Continue: Rebuild codebase index
Continue: Codebase Force Re-Index

Then status switches to "Indexing paused", like I would expect on start of VSCode:

Got it, thanks! We're working on a solution to this today


⚠ Console Web has degraded performance

In 6d27fe2, Console Web (https://console.inu-cafeteria.app) experienced degraded performance:

HTTP code: 200
Response time: 11222 ms

Resolved: Console Web performance has improved in 8a719a8.


fix: fixed the issue of articles looking broken

Description

There was an issue with some article looking broken in brave.
The issue was in markdown folder directory.
Fixed the issue in three different files.

Related issue(s)
Fixes #3049

/rtm

Thanks @shabdpatel 😄 Can you please look out for other such broken blog pages and create a separate PR?

Sure!!

/rtm

/rtm

@thulieblack review please

/rtm


CocoaPods support

We're building an iOS framework that primarily supports SPM, but we know there are some adopters that still use CocoaPods. Could this library support CocoaPods as well?

I'm afraid there is no CocoaPods in any of my frameworks and no plans to add it. I suggest trying binary distribution if SPM doesn't work for you right now. You can use swift-create-xcframework to create a binary from a Swift package.


refactor(works-for-you2): Promote /works-for-you2 -> /works-for-you

Related https://github.com/artsy/force/pull/9223
This deletes legacy code and promotes the new /works-for-you app.
cc @artsy/grow-devs

Warnings

:warning:

Routes added to routes.tsx should have a corresponding cypress.js smoke test. See the cypress/integration folder for examples.

  Generated by :no_entry_sign: dangerJS against 2d9831c5c2d4c93882c2b6eee51c10cf5357f787

(Skipping the smoke test because it requires a logged in user)


Remove configuration for Policy Publisher

We're no longer running this app anymore, as the policy pages on
GOV.UK have been replaced by the new Topic pages.
Following on from [1], when Policy Publisher was removed as an app,
this commit removes the remaining configuration in govuk-puppet.
1: 0fb32e690ca6057de87e4812bbcf7d365f1c5e96
https://github.com/alphagov/govuk-puppet/pull/8161

cool, this looks clean.


Exception in thread "pool-3-thread-1" com.netflix.hystrix.exception.HystrixRuntimeException failed and no fallback available.

hystrix is no longer supported.

Hello @cunhaDev, Spring Cloud Netflix Hystrix has been out of support for many years now, so it seems that either you're using an unsupported version or you're using Hystrix project directly (not with Spring Cloud) and that is maintained in a separate repo: https://github.com/Netflix/Hystrix. Please note that for any team to take a look at a bug, more details will be needed, including steps to reproduce, what you expected to happen and what actually happens, the entire stacktrace and, in most cases,  a minimal, complete, verifiable example that reproduces the issue.


tls_disable: true break things

Describe the bug
bank-vaults fails to start pod if TLS encryption not used (i.e. HTTP instead of HTTPS)
To Reproduce

helm install banzaicloud-stable/vault --set vault.config.listener.tcp.tls_disable="true"
Watch that 2 (of total 4) containers fails liveness/readyness probes

Expected behavior
All containers handles liveness/readyness probes.
Additional context
It is pretty easy to fix helm chart to use HTTP scheme for probes when vault.config.listener.tcp.tls_disable is true, but next issue arises: vault-unsealer and vault-configurer always try to connect to vault via HTTPS (and stops at HTTP answer). Seems like scheme is hardcoded somewhere. Can you point out place in code where look for ?
P.S.: additionally, seems like HTTPS scheme hardcoded for probes in operator code, too.
P.P.S.: enabling ingress for helm chart is useless without tls_disable:true

Hi @evgkrsk, HTTPS is our opinionated point of view about using Vault, but it is not pointed out anywhere or not reported when someone disables it, I will do that.
I think we need to solve the issue the other way around, why is P.P.S.: enabling ingress for helm chart is useless without tls_disable:true for you? For example we are using Traefik as an ingress controller, and having no issues with TLS backends.

Honestly, I can't figure out, how to use non-selfsigned (default) TLS certificate. I have tried to mount LetsEncrypt tls secret via customSecrets and tls_cert_file/tls_key_file, but vault cant handle it somehow.
With default k8s ingress controller, turning on ingress in chart makes non-working configuration where nginx cant recognize HTTPS answers from TLS backend. This configuration will successfully work with tls_disable:true.

If you are using nginx-ingress you can set the protocol of the backends as described here: https://github.com/kubernetes/ingress-nginx/blob/master/docs/user-guide/nginx-configuration/annotations.md#backend-protocol could you please try that? I still strongly suggest to not to drop TLS.

Indeed, I used "secure-backends: true" for my version of ingress, but still cant figure out how to use custom TLS certificate instead of self-signed one (inside vault container, not for ingress).

Could you please give me a pointer which nginx-ingress are you using, which version, repository, etc?

Standard k8s nginx-based ingress controller (https://github.com/kubernetes/ingress-nginx/releases/tag/nginx-0.13.0).

Ok, could you please try adding a server-snippet to the Ingress definition for Vault?
nginx.ingress.kubernetes.io/server-snippet: |
"proxy_ssl_verify off;"

Oh, my fault. Vault answers fine via ingress (seems like nginx ignores self-signed cert), but fails to work inside k8s cluster, like that:
# kubectl -n logging exec -ti es-logging-0 -- curl -v -s https://vault-ha-va
ult.vault:8200/v1/sys/health
* About to connect() to vault-ha-vault.vault port 8200 (#0)
*   Trying <IP_ADDRESS>...
* Connected to vault-ha-vault.vault (<IP_ADDRESS>) port 8200 (#0)
* Initializing NSS with certpath: sql:/etc/pki/nssdb
*   CAfile: /etc/pki/tls/certs/ca-bundle.crt
  CApath: none
* Server certificate:
*       subject: CN=vault-ha-vault
*       start date: Aug 09 09:29:23 2018 GMT
*       expire date: Aug 09 09:29:23 2019 GMT
*       common name: vault-ha-vault
*       issuer: CN=vault-ca
* NSS error -8179 (SEC_ERROR_UNKNOWN_ISSUER)
* Peer's Certificate issuer is not recognized.
* Closing connection 0
command terminated with exit code 60

Yes, however, the ca-cert is deployed to the namespace into a secret where Vault is running, you can use that to validate or pass -k to curl.

Sure, but it is not trivial to add HTTPS support with custom certificate/key/ca in all applications that uses vault (they lives in different namespace). Currently HTTP is used.

It is not trivial, however, all you have to do is to pass in one environment variable:
https://www.vaultproject.io/docs/commands/index.html#vault_skip_verify
or this one:
https://www.vaultproject.io/docs/commands/index.html#vault_cacert

Mount the certificate into the Pod as a volume.

This might be an opinionated view, but the whole point is to store secrets in a secure way and move secrets over the transport layer which is encrypted. So I don't think to add HTTP support here is the right solution. HTTP should only be used for development purposes in the case of Vault. I suggest rather making your custom client configurable to turn on TLS cert verification, which shouldn't be a big effort IMHO.

@bonifaido but what if we use e.g service mesh and it provides us unified TLS across all pods in Kubernetes cluster? IMO we should have an option to make bank-vaults work with tls_disabled since Vault has support for it and again, you can implement network encryption using external tools

I will take a look at this in the following days, and will come back with a solution how to disable TLS in the operator easily. If I understand correctly that is what you want, am I right? @hatemosphere @evgkrsk

Yes, you are right.

Hi @evgkrsk and @hatemosphere! We have implemented the TLS configuration disablement option as described here: https://github.com/banzaicloud/bank-vaults/pull/122
This will be available in 0.2.13 release, which is already under build by Docker HUB, let's give a few minutes to let them arrive there.
Should you have any questions about it, write it up here or in another issue.

Seems like corresponding changes drops only in operator's code, not in image itself (we use helm chart to install docker image, so operator is not used).

I don't really understand the issue, so you are using the helm chart? If yes, I think it's rather easy to change the scheme to http I instead of https in the health checks. If you have done that could you please open up a pull request to make it configurable and leave https as the default?

I'm asking for the PR beceasue it it would give more possibility to understand and debug the issue. The solution doesn't hav

Yes, I have patched templates/deployment.yaml from helm chart like that:
        livenessProbe:
          httpGet:
            {{ if .Values.vault.config.listener.tcp.tls_disable }}
            scheme: HTTP
            {{ else }}
            scheme: HTTPS
            {{ end }}
            path: /v1/sys/init
            port: vault
        # This probe makes sure that only the active Vault instance gets traffic
        # See: https://www.vaultproject.io/api/system/health.html
        readinessProbe:
          httpGet:
            {{ if .Values.vault.config.listener.tcp.tls_disable }}
            scheme: HTTP
            {{ else }}
            scheme: HTTPS
            {{ end }}
            path: /v1/sys/health
            port: vault

Now probes dont failed but satellite containers (unsealer/configured) still use HTTPS and write errors in log:
# kubectl -n vault logs -f vault-ha-vault-855f76756b-55g4l -c vault-unsealertime="2018-08-13T11:44:46Z" level=info msg="initializing vault..."
time="2018-08-13T11:44:50Z" level=fatal msg="error initializing vault: error testing if vault is initialized: Get https://<IP_ADDRESS>:8200/v1/sys/init: http: server gave HTTP response to HTTPS client"

# kubectl -n vault logs -f vault-ha-vault-855f76756b-55g4l -c vault-configurer
time="2018-08-13T11:43:47Z" level=info msg="New config file change \"Initial\": CREATE"
time="2018-08-13T11:43:47Z" level=info msg="checking if vault is sealed..."
time="2018-08-13T11:43:52Z" level=error msg="error checking if vault is sealed: error checking status: Get https://<IP_ADDRESS>:8200/v1/sys/seal-status: http: server gave HTTP response to HTTPS client, waiting 30s before trying again..."
time="2018-08-13T11:44:22Z" level=info msg="checking if vault is sealed..."
time="2018-08-13T11:44:26Z" level=error msg="error checking if vault is sealed: error checking status: Get https://<IP_ADDRESS>:8200/v1/sys/seal-status: http: server gave HTTP response to HTTPS client, waiting 30s before trying again..."
time="2018-08-13T11:44:56Z" level=info msg="checking if vault is sealed..."
time="2018-08-13T11:44:59Z" level=error msg="error checking if vault is sealed: error checking status: Get https://<IP_ADDRESS>:8200/v1/sys/seal-status: http: server gave HTTP response to HTTPS client, waiting 30s before trying again..."
time="2018-08-13T11:45:29Z" level=info msg="checking if vault is sealed..."
time="2018-08-13T11:45:33Z" level=error msg="error checking if vault is sealed: error checking status: Get https://<IP_ADDRESS>:8200/v1/sys/seal-status: http: server gave HTTP response to HTTPS client, waiting 30s before trying again..."

According to file diff in #122 my guess is that http/https switching code dont included in bank-vault's Docker hub image (only in operator binary).

Yes, where you can see VAULT_CACERT env vars you also have to include the VAULT_ADDR env car, where the scheme is http in your case, by default it is https.

Thanks, it helps, but now unsealer write following error log:
time="2018-08-13T13:29:40Z" level=info msg="checking if vault is sealed..."
time="2018-08-13T13:29:40Z" level=info msg="vault sealed: true"
time="2018-08-13T13:29:44Z" level=error msg="error unsealing vault: fail to send unseal request to vault: Error making API request.\n\nURL: PUT http://<IP_ADDRESS>:8200/v1/sys/unseal\nCode: 500. Errors:\n\n* failed to create cipher: crypto/aes: invalid key size 33"

Could you please try with the TLS related env vars removed?

@bonifaido , seems like unseal problem was related to already initialized etcd storage. Dropping vault data in etcd fixes it.
Please review/merge my PR for helm chart (https://github.com/banzaicloud/banzai-charts/pull/303)

@evgkrsk merged, thx.


Change name to experiment_name on Experiments.expose

This addresses #486
Currently, the Experiments class has two methods that are used in tandem quite often, variant and expose. And they both have different parameters for the name of the experiment.
variant -> name
expose -> experiment_name
This PR will introduce experiment_name to variant and log a deprecation warning for name.

^ rebase

@spladug Could I get :eyes:?

@spladug, could you merge it?


Needed content for release

We have not had any "real" releases for this repo in a long time, so I am trying to look on what better should be done. Looking at our latest release at https://github.com/eclipse/kuksa.val.services/releases/tag/v0.2.0 we have the following artifacts:

That is:

Some generic stuff (LICENSE and NOTICE)
Seat binaries and seat documentation
Hvac and Seat containers

Somewhat inconsistent, and now we also have mock service. Some questions:

What artifacts do you need?
Does it make sense to include binaries for all services (seat, hvac, mock?) For both amd64 and aarch64?
Do we need to have OCI-containers as part of the release if they anyway are available at ghcr (see https://github.com/orgs/eclipse/packages?repo_name=kuksa.val.services)?

My idea is to refactor this one so it:

Does not upload or include ghcr images, upload can happen directly from lower level workflows. But the release may trigger those lower level workflows so that a release only is created if they succeed.
Always use tag as version, i.e. it should not be possible to specify tag manually?

Opinions?

So at least container images I would only make available via ghcr, those exported tar balls are not easy to handle anyway. I do think this still comes from a time when it wasn't easy/obvious how to push to a container registry

It would also be good to have a mock image pushed, but I am not sure how feasible that is. If there is already scripting for that, whoch we just "need to connect to an action" , we should go for it, but if it involve more work, postpone it. I know @eriksven  is interested in a pullable mock container  for an upcoming bluprint :D

It would also be good to have a mock image pushed, but I am not sure how feasible that is. If there is already scripting for that, whoch we just "need to connect to an action" , we should go for it, but if it involve more work, postpone it. I know @eriksven is interested in a pullable mock container for an upcoming bluprint :D

For the mock service, sending a container imageto ghcr on every merge to main (and ato ttls.h on each pull reques)t should be easy, and we seem to agree to NOT have any container images as the release artifact.
But then comes the question on binaries. Today, for seat service we build debug and release versions for both amd64 and arm64 for every pull request, and we include the release tarballs in the release. For hvac and mock we do not build natively, we just build for docker. Is there a reason to have a difference? The most simple solution would be to just skip the binaries also for seat service, but for kuksa.val we release databroker and databroker-cli as tarballs. So do we want amd64+arm64 tarballs also for hvac and mock service? But if we want to include only release-version in releases, is there still a reason to build a debug-version of seat-service on every pull request? Should be build debug-versions also for hvac and mock?

I don't think we have a seat service binary user at the moment, but they were provided for easier standalone testing
there are no binaries for python services, e.g. in integration tests I just clone the repo/tag and use them directly
seat service documentation could be removed
multi-arch tar image files are not needed, they were used mainly on demo setups a long time ago (e.g. no ghcr access).

What I did for now:

A pre-release created at https://github.com/eclipse/kuksa.val.services/releases/tag/v0.3.0
Docker containers removed, but no other changes

@SebastianSchildt @int0x27 - please check the pre-release, if it contain what you expect.
@SebastianSchildt - if OK could you approval process

Considering this one completed, wiki at https://github.com/eclipse/kuksa.val.services/wiki/Release-Process updated


useNTState doesn't appear to be working

Hello there! I am attempting to build a dashboard with the help of this library. For whatever reason, when I try and update the value of something, the react app disconnects and then reconnects.
I am running a simulated robot through WPI's robot simulation tools. Here's the error I am getting.
NT: DISCONNECTED NT4 client '1700087460502@1' (from <IP_ADDRESS>:61606): binary decode error: mpack_error_type
The key is correctly typed and does show up in the robot simulation GUI showing the initial value.
Side note!
The latest version didn't actually publish the code onto NPM (see https://www.npmjs.com/package/ntcore-react?activeTab=code), so I had to install the latest version from git directly.

Any chance you can post the code you're using? Particularly for the actual connection with the NTProvider component.
Also idk how I never noticed that the code has been broken on npm this entire time, but I'll try to get that fixed as well. For now, you should be able to pin the version to 1.1.0 and be fine, since as far as I know, there's no major differences between 1.1.1 and 1.1.0 other than being a different release format.

I had been using 1.1.0 but updated to 1.1.1 via github in case that might've resolved my issue. I can put the relevant files here:
main.tsx (the relevant env variable is set to "DEVELOPMENT")
import React from 'react'
import ReactDOM from 'react-dom/client'
import App from './App'
import './assets/index.css'
import { NTProvider } from 'ntcore-react'

ReactDOM.createRoot(document.getElementById('root')!).render(
  <React.StrictMode>
    {import.meta.env.RENDERER_VITE_NT === "DEVELOPMENT" && <NTProvider uri="<IP_ADDRESS>" port={5810}>
      <App />
    </NTProvider>}
    {import.meta.env.RENDERER_VITE_NT !== "DEVELOPMENT" && <NTProvider teamNumber={import.meta.env.RENDERER_VITE_NT}>
      <App />
    </NTProvider>}
  </React.StrictMode>,
)

The dashboard component with the useNTState that is having the issue
import { NetworkTablesTypeInfos } from "ntcore-ts-client";
import { useNTState, useNTValue } from "ntcore-react"

export default function Dashboard2023(): JSX.Element {
    const currentGear = useNTValue<"Gear 1" | "Gear 2" | "Gear 3">("/kilroy/drive/currentGear", NetworkTablesTypeInfos.kString, "Gear 1");
    const inReverse = useNTValue<boolean>("/kilroy/drive/inReverse", NetworkTablesTypeInfos.kBoolean, false);
    const eBrakeEngaged = useNTValue<boolean>("/kilroy/drive/eBrakeEngaged", NetworkTablesTypeInfos.kBoolean, false);

    const autonomousDisabled = useNTValue<boolean>("/kilroy/autonomous/disabled", NetworkTablesTypeInfos.kBoolean, false);
    const [autonomousMode, setAutonomousMode] = useNTState<number>("/kilroy/autonomous/mode", NetworkTablesTypeInfos.kInteger, 0);
    enum LeftRightNone {
        Left = -1,
        None = 0,
        Right = 1
    }
    const [leftRightNone, setLeftRightNone] = useNTState<LeftRightNone>("/kilroy/autonomous/left-right-none", NetworkTablesTypeInfos.kInteger, LeftRightNone.None);

    return (
        <>

Kilroy 339

                Gears

                    <span className={`text-2xl px-4 font-semibold ${inReverse ? "text-green-500" : "text-neutral-400"}`}>Reverse
                    <span className={`text-2xl px-4 font-semibold ${!inReverse && currentGear == "Gear 1" ? "text-green-500" : "text-neutral-400"}`}>Gear 1
                    <span className={`text-2xl px-4 font-semibold ${!inReverse && currentGear == "Gear 2" ? "text-green-500" : "text-neutral-400"}`}>Gear 2
                    <span className={`text-2xl px-4 font-semibold ${!inReverse && currentGear == "Gear 3" ? "text-green-500" : "text-neutral-400"}`}>Gear 3

                E-Brake Engaged

                   <div className={`h-20 w-20 rounded-full ${eBrakeEngaged ? "bg-green-500 shadow-[0_0_1px_#fff,inset_0_0_1px_#fff,0_0_5px_#0f8,0_0_5px_#0f8,0_0_20px_#0f8]" : "bg-neutral-400 border-4 border-neutral-500"}`}/>

                Autonomous

                        <span className={`text-2xl px-4 font-semibold ${autonomousDisabled ? "text-neutral-300" : "text-red-500"}`}>Disabled
                        <span className={`text-2xl px-4 font-semibold ${autonomousDisabled ? "text-green-500" : "text-neutral-300"}`}>Enabled

                        Autonomous Mode

                        <select name="autoSelector" id="autoSelector" value={autonomousMode} className="bg-neutral-900 rounded-md text-2xl px-4 font-semibold" onChange={(e): void => setAutonomousMode(e.target.value as unknown as number)}>
                            <option value={0}>Drive Forward Only</option>
                            <option value={1}>Drive Turn Drive</option>
                            <option value={2}>Drive Over Charging Station</option>
                            <option value={3}>Drop Cube Drive Forward</option>
                        </select>

                        Left-Right-None Setting

                        <select name="autoSelector" id="autoSelector" value={leftRightNone} className="bg-neutral-900 rounded-md text-2xl px-4 font-semibold" onChange={(e): void => setLeftRightNone(e.target.value as unknown as LeftRightNone)}>
                            <option value={LeftRightNone.Left}>Left</option>
                            <option value={LeftRightNone.Right}>Right</option>
                            <option value={LeftRightNone.None}>None</option>
                        </select>

        </>
    )
}

Video showing what happens:
https://micro.sylo.digital/v/gKSPTK.mp4
As for NPM, should just be as simple as unpublishing 1.1.1 and then republishing 1.1.1 with the code

I can't seem to reproduce this locally. I tried both useNTValue and useNTState and both worked fine for me. I also tried using both values that are already announced, along with ones that aren't to see if creating a new one/modifying an existing one was the issue and both of those worked fine as well. I'm going to pass this issue up to the parent package since it seems like an issue on that end more than with this package. https://github.com/cjlawson02/ntcore-ts-client/issues/111

Alright, if you get a chance, I just populated the source code into FIRST-TEAM-339/kilroy-dashboard which contains all the source code.
Just a couple notes:

It's using electron-vite but all you need to know about that is just run pnpm dev or npm run dev after you install the packages
Make sure you rename example.env to .env and just fill that variable as DEVELOPMENT unless you happen to have a live robot wherever you are currently at (then put the team #)

Hey there! I am wondering if you are able to updated to the most recent version of ntcore-ts-client, version 2.0.0-beta.2

Working on getting it updated, looks like there's a publishing issue in the parent package though that realistically needs to be resolved first (unless I add a bunch of extra dependencies on my end).

what kind of issues are there? When I made a fork of this there were some errors but I quickly resolved them, however there were some other publishing issues

The biggest issue is that none of the dependencies for it are included in the package.json, so all of them need to be manually installed.

Ohhh I checked the package.json of the latest version, and I see what you mean. That's the same issues I was experiencing. I will see if I can figure out a solution to this and post it in the issue on the parent repo

Fixed the publishing issue


Allow user to pass in Guid to create the trigger with?

ENHANCEMENT REQUEST - see commit below for my suggestion
https://github.com/GSUFan513/monitor-table-change-with-sqltabledependency/commit/24bfa3ea058dad5794e639599faadac2faba3d24
Currently, we need a way to poll our database and check for a trigger on that table, however if there are multiple triggers on the table, we cannot be positive that one of the ones on there is the one we are looking for.
This commit allows me to pass in the triggers guid to create. This way when we poll sql for the triggers on the table, we are positive the one we started with is still there and functioning. This should have no effect on any users.
Alternatively, you could provide the Guid you are using in a public property if that's easier, but I would REALLY appreciate seeing this code in there as soon as possible.

Hi, with GetBaseObjectsNamingConvention() you can have the naming convention used to create all DB objects. Is not enough?

Ah, I missed this method. I think that is fine for our usage.
Thanks.


Phasor diagram in a polar coordinate plot

Question: Is there a way to visualize phasors in a polar coordinate plot? If there is, is there a way to visualize the angles between the phasors in the plo? (Ask your question here)
ScottPlot Version: 5.0.35 (What NuGet package are you using?)
Description: I'm looking for a way to show the correlation between phasor easy. Preferably with an angle that is between the phasors. Maybe something like
var phasorType = PhasorTypes.Arrow;
var angle1 = 0;
var angle2 = 2*Math.PI/3;

var MyPolarCoordinates = MyPlot.Plot.Add.PolarCoordinates();
var phasor1 = MyPolarCoordinates.AddPhasor(angle1, phasorType);
var phasor2 = MyPolarCoordinates.AddPhasor(angle2, phasorType);

MyPolarCoordinates.AddPhaseDiffrenceText(phasor1, phasor2);

And with 6 phasors it would look something like below but maybe with curved angles between the phasors showing the relationship between them.

Hi @nilsakesson, this is definitely possible by creating a custom plottable that implements IPlottable and adding it to your plot.
Some general suggestions are outlined here https://scottplot.net/faq/custom-plot-type/
It's unlikely that I'll create this plot type myself soon, but if you or someone from the community decides to give it a go and creates a class that generates plots like this, I would be happy to include it with ScottPlot in a future release so it's available to everyone. If you get started working on it and get stuck somewhere, you can share what you got so far and I am happy to offer suggestions along the way!

I've been thinking about #4055 for a while.
With some improvements, we should be able to easily implement the plot in this issue.
@swharden Could you provide me with some suggestions?


Can Malmo Platform be used under non X window server?

In order to accelerate the trainning process, we want to run the Malmo platform in a Server computer,
which has no X window client.
I have tried in ubuntu terminal(Ctrl+Alt+F1).
but the client cannot be launched.
We want to know if we can run Malmo in a server without any X window?
thanks

Hi, we've had some success running Minecraft headless on a Linux cluster using a virtual display. You need an x-server, and Minecraft will need open-gl installed. I can try to dig out the steps we used here, though I suspect it varies quite a bit from cluster to cluster.

@DaveyBiggers
I want to know how to run Minecraft on remote machines without screens. I have tried but failed.
Please describe the steps or any hints.

@DaveyBiggers It seemed that there are no gui Minecraft versions On the internet. I wonder if the minecraft included in Malmo platform can be set to run in no gui mode?

@WXFMAV I believe you are thinking of the nogui flag for running a dedicated Minecraft server. This is no good to us, as we require a running client - there is no nogui flag for the client (it wouldn't make sense to anyone other than researchers, since a game that you can't see is unlikely to be a big seller...)
However, as stated above, it is possible to run the client using a virtual display. If you give us some details of your environment we might be able to provide some pointers.

Hi @WXFMAV and @okdshin,
if you work on Linux, these steps may help:

Install xinit (a tool for manually running an X display server)
Start xinit (e.g., in a separate shell, or detach the process: nohup xinit & disown)
Export the display variable and start the Minecraft client: DISPLAY=:0.0 launchClient.sh

Let us know if these help. If you are working on other platforms some changes may be required (we tested these on Ubuntu only).

@DaveyBiggers  My server is CentOS instead of Ubuntu, my system version is:
CentOS Linux release 7.2.1511 (Core)
3.10.0-327.el7.x86_64
command runlevel returns N 3 indicating that  there is no x-server.
And I wonder if a linux system running in level 3 can be installed with xinit?

@katja-hofmann Thanks! I can run Minecraft on ubuntu server without any display by following your steps.
@DaveyBiggers Just for your information, on my ubuntu server, runlevel indicates N 2 now and I had to install gdm.

Hi @okdshin - thanks for confirming that this solution works for you.
@WXFMAV - runlevel 3 should be fine. Were you able to install xinit and run it? If not what error message do you get?

here's what i've done,

make sure /etc/X11/Xwrapper.config is set to "allowed_users=anybody"

then within a separate screen window,

nohup xinit -- :<FREE_DISPLAY_NUMBER> &
DISPLAY=:<FREE_DISPLAY_NUMBER>.0 ./launchClient.sh

@katja-hofmann
Following your instruction, I got an error saying
Exception in thread "main" java.io.IOException: Input/output error
	at sun.nio.ch.FileDispatcherImpl.lock0(Native Method)
	at sun.nio.ch.FileDispatcherImpl.lock(FileDispatcherImpl.java:91)
	at sun.nio.ch.FileChannelImpl.tryLock(FileChannelImpl.java:1096)
	at java.nio.channels.FileChannel.tryLock(FileChannel.java:1154)
	at org.gradle.wrapper.ExclusiveFileAccessManager.access(ExclusiveFileAccessManager.java:51)
	at org.gradle.wrapper.Install.createDist(Install.java:46)
	at org.gradle.wrapper.WrapperExecutor.execute(WrapperExecutor.java:126)
	at org.gradle.wrapper.GradleWrapperMain.main(GradleWrapperMain.java:61)

@katja-hofmann I tried you solutions but still could not launch client. When I tried to start xinit, I got error message:

Fatal server error:
(EE) no screens found(EE)


Memory Usage

Great tool, thanks so much for sharing!
I am running helix-trace using the following command line:
./helix-trace -i /media/sf_data/ITCH5.0/08302017.NASDAQ_ITCH50 -s AAPL -P nasdaq-binaryfile-itch50 -f csv -o AAPL.csv
08302017.NASDAQ_ITCH50 is downloaded from ftp://emi.nasdaq.com/ITCH/ and is roughly 5.4gb
When I run the tool, it steadily uses more and more memory, until it crashes (I only have 4gb of memory). I am running using Fedora 26. Any ideas what is going on? Does helix not delete orders after they are executed or cancelled?
thanks
Nick

@nickos556 I'm able to process a similarly sized file, but I also have 32 GB of RAM. Could you please try reproducing with a debug build that enables AddressSanitizer?
mkdir -p build/debug
cd build/debug
cmake -DCMAKE_BUILD_TYPE=debug ../..

@nickos556 I ran helix-trace using Massif and the peak memory usage is around 2.8 MiB for a 7.1 GiB ITCH 5.0 file. The memory usage increase you're seeing is an artifact of helix-trace using mmap() for accessing the file but the kernel will manage that memory for you and shouldn't cause you to run out of memory.


fix: updated constants used by BootloaderDebugTracer

What :computer:

Changed MAX_MEMORY_BYTES and VM_HOOKS_PARAMS to correspond to MAX_MEM_SIZE() and VM_HOOK_PARAMS() in https://github.com/matter-labs/era-contracts/blob/main/system-contracts/bootloader/bootloader.yul .
Removed the call to self.inner.read() from display_detailed_gas_info.

Why :hand:

Since the parameters got out of sync, BootloaderDebugTracer was unable to load BootloaderDebug.
Re-enabling the functionality exposed a deadlock in display_detailed_gas_info.

Evidence :camera:
Running
./target/release/era_test_node --show-gas-details=all replay_tx sepolia-testnet 0x488c598ad0d6f1135bd61e5bce494be6b79a17deebd1d09a700da7dddde2efde

on the main branch produces
13:50:22  INFO Gas - Limit: 1_919_764 | Used: 968_051 | Refunded: 951_713
13:50:22  INFO !!! FAILED TO GET DETAILED GAS INFO !!!

while this branch has
13:48:37  INFO ┌─────────────────────────┐
13:48:37  INFO │       GAS DETAILS       │
13:48:37  INFO └─────────────────────────┘
13:48:37  INFO Gas - Limit: 1_919_764 | Used: 123_144 | Refunded: 951_713
13:48:37  INFO   WARNING: Refund by VM: 1_796_620, but operator refunded more: 951_713
13:48:37  INFO During execution published 0 bytes to L1, @50_000 each - in total 0 gas
13:48:37  INFO Out of 123_144 gas used, we spent:
13:48:37  INFO            24_070 gas (19%) for transaction setup
13:48:37  INFO             3_369 gas ( 2%) for bytecode preparation (decompression etc)
13:48:37  INFO            86_650 gas (70%) for account validation
13:48:37  INFO             9_055 gas ( 7%) for computations (opcodes)

Duplicate changes of https://github.com/matter-labs/era-test-node/commit/1d72eae6e9400b5b6caed5e60093e7c32eee8abb


subscribe(_:) returns an optional subscription reference

It'd make more sense for it to return a non-optional one.

Fixed in version 4.1


Plugin system

It would be neat if additional functionality could be opted-in to the system in the form of easy-to-add plugins, hosted as separate packages. First example: a Google Analytics plugin, which adds GA tracking code with your tracking ID to the web interface for your dataset.
This may be an opportunity to experiment with entry points: http://amir.rachum.com/blog/2017/07/28/python-entry-points/

Plugins should be able to interact with the build step. This would give plugins an opportunity to modify the SQL databases and help prepare them for serving - for example, a full-text search plugin might create additional FTS tables, or a mapping plugin might pre-calculate a bunch of geohashes for tables that have latitude/longitude values. Plugins could really take advantage of the immutable nature of the dataset here.

For visualizations, Google Maps should be made available as a plugin. The default visualizations can use Leaflet and Open Street Map, but there's no reason to not make Google Maps available as a plugin, especially if the plugin can provide a mechanism for configuring the necessary API key.

http://setuptools.readthedocs.io/en/latest/setuptools.html#dynamic-discovery-of-services-and-plugins Is pretty good

I'd also suggest taking a look at stevedore, which has a ton of tools for doing plugin stuff. I've had good luck with it in the past.

Oh thanks, that definitely looks like an interesting option.

Tox is a good example of a project that uses pluggy in the way I want to use it (function hooks rather than classes): https://github.com/tox-dev/tox/blob/master/tox/hookspecs.py

Here's a demo of the convert_units() SQL function I prototyped in f2720b0c6b7172ebe88

Once I've got the plugins mechanism stable and people start releasing plugins it would be useful to have a dedicated Trove classifier on PyPI for Datasette plugins - ``Framework :: Datasette` for example.
This would help me build a Datasette equivalent of the http://plugincompat.herokuapp.com/ site, which works by scanning PyPI for items with the Framework :: Pytest classifier: https://github.com/pytest-dev/plugincompat/blob/8bdf1a6fb82807091ece0c68c196103ee8270194/update_index.py#L52-L53
It looks like the mechanism for requesting new PyPI classifiers is to file a ticket against warehouse, like this one: https://github.com/pypa/warehouse/issues/3570

I created https://github.com/simonw/datasette-plugin-demos which is now published to PyPI and can be installed with pip install datasette-plugin-demos - I've confirmed that if you DO install it my Datasette plugins branch picks up the plugins, and select random_integer(1, 4) works as it should.

Slight code design problem... when I tried installing my branch in a fresh virtual environment I got this error, because setup.py now depends on pluggy (from importing __version__):
      File "/private/var/folders/jj/fngnv0810tn2lt_kd3911pdc0000gp/T/pip-req-build-dftqdezt/setup.py", line 2, in <module>
        from datasette import __version__
      File "/private/var/folders/jj/fngnv0810tn2lt_kd3911pdc0000gp/T/pip-req-build-dftqdezt/datasette/__init__.py", line 2, in <module>
        from .hookspecs import hookimpl # noqa
      File "/private/var/folders/jj/fngnv0810tn2lt_kd3911pdc0000gp/T/pip-req-build-dftqdezt/datasette/hookspecs.py", line 1, in <module>
        from pluggy import HookimplMarker
    ModuleNotFoundError: No module named 'pluggy'

I should check if it's possible to have two template registration function plugins in a single plugin module. If it isn't maybe I should use class plugins instead of module plugins.

Annoyingly, the following only results in the last of the two prepare_connection hooks being registered:
from datasette import hookimpl
import pint
import random

ureg = pint.UnitRegistry()

@hookimpl
def prepare_connection(conn):
    def convert_units(amount, from_, to_):
        "select convert_units(100, 'm', 'ft');"
        return (amount * ureg(from_)).to(to_).to_tuple()[0]
    conn.create_function('convert_units', 3, convert_units)

@hookimpl
def prepare_connection(conn):
    conn.create_function('random_integer', 2, random.randint)

I think that's OK. The two plugins I've implemented so far (prepare_connection and prepare_jinja2_environment) both make sense if they can only be defined once-per-plugin. For the moment I'll assume I can define future hooks to work well with the same limitation.

I added a mechanism for plugins to serve static files and define custom CSS and JS URLs in #214

I released everything we have so far in Datasette 0.20 and built and released an example plugin, datasette-cluster-map.


Build Fails

Following the tutorial build fails - Node.js version
[build]
Node.js 12 actions are deprecated. Please update the following actions to use Node.js 16: actions/checkout@v2. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/.

old issue.


Floating Resource Icons

Spawn particle effects of resources whenever the resources spawn for that producer.

Visual indication of resource collection from the plant/object itself

Ok, the oak tree now has 3 particle effect systems: one for water, one for sun, and one for seed

Next stop is copying them to other plants, then finding how to get them to trigger on resource collection

Now have been copied to sunflower and watermelon, next step is figuring out how to trigger on resource collection

Trigger on this function @ndamalas
https://github.com/nicholas-maltbie/GGJ-2023/blob/7e44e17fc7c6b73e518a482d26253a4fa1e7af67/Assets/Scripts/Plants/Plant.cs#L106-L119

Got it, thank you!

PR for this issue:
https://github.com/nicholas-maltbie/GGJ-2023/pull/22


fix: refresh talos tls certs

Pull Request
We need to recreate talos client regularly.
Close https://github.com/siderolabs/talos-cloud-controller-manager/issues/120

What? (description)
Why? (reasoning)
Acceptance
Please use the following checklist:

[ ] you linked an issue (if applicable)
[ ] you included tests (if applicable)
[ ] you ran conformance (make conformance)
[ ] you linted your code (make lint)
[ ] you linted your code (make unit)

See make help for a description of the available targets.

/m


added fingerprint values for 2020 base clarity

I was still having fingerprint issues with my 2020 Clarity until I added these two firmware versions. Seems like there has been some firmware updates for these cars!

Thanks for the PR! Could you provide the dongleID for your device so that I can look at your logs?

Found your dongleID! Thanks for the PR


Added SassUtils for better compatibility with bootstrap 4 (and scss comments overall)

Fixes #779, #756.

Probably also closes #758 , #759.
@stof (or whoever is responsible for assetic in general), please note that I had to fix also Scssphp test since 5.3 and 5.6 versions of scssphp product different exceptions (is 5.3 really a must, it is EOL since October 2014, if I recall correctly?)
As I highly depend on fixing this bug and I have a bit of free time this week - feel free to point some other issues to PR before bumping a version )

@stof, fixed.

Thanks!

@kriswallsmith , any other issues or enhancements I can PR? As I said I have a bit of free time to work on OS projects.


Magento 2107 endopoint wishlist get

Description
Implemented the wishlist retrieval, wishlist item retrieval and adjusted the creation of the wishlists. It no longer will return the default wishlist ID when it cannot create a new list. This will prevent developers from abusing the create endpoint as a getter. Added DE translations for configuration UI.
Type of change

[ ] Bug fix
[x] New feature
[ ] This change requires a documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I updated the CHANGELOG.md

done


⚠ Beulah Revival Network has degraded performance

In 99a0445, Beulah Revival Network (https://beulah.clicki.digital) experienced degraded performance:

HTTP code: 200
Response time: 5729 ms

Resolved: Beulah Revival Network performance has improved in 9db309c.


[5.7] Added chunkById support in relations.

Pivot data doesn't attached to model If you use Builder::chunkById.
This PR is fixing this and using actual of relation keys for chunking.
Closes https://github.com/laravel/framework/issues/26816.
Related to https://github.com/laravel/nova/issues/152
For 5.6 need separate PR?

I'm wondering about the BelongsToMany implementation:
Should we also use $this->getRelated()->[...] for $column and $alias (as HasManyThrough does)?
In a User ↔ Role relationship:

Your implemenation: role_user.role_id
My suggestion: roles.id

It would be more consistent and in basically every case both columns contain the same value. The only exception would be a relationship that doesn't join the related model's primary key (e.g. roles.slug). But I'm not even sure what the expected/correct behavior would be in a case like this.


Add Remove Directory to list of Risky Commands

Can we have the option to enable rmdir from the list of risky commands?

Hey @Geoff12889, thanks for that idea.
Do you want to try adding this command pattern here and open a pull request?

Wow, appreciate the quick response! Yeah, will give it a shot. Thanks

Sorry for the silly question, but how do I build after I make those changes? I tried the Makefile but it says:
make: Nothing to be done for Makefile'.`

The test failed, and I'm stuck on what to try next:
`running 2 tests
test test_missing_patterns_coverage ... ok
stored new snapshot /Users/[username]/Downloads/shellfirm-main/shellfirm/tests/snapshots/checks__fs-strict-folder_deletion.yaml.snap.new
test test_checks ... FAILED
failures:
---- test_checks stdout ----
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ Snapshot Differences ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
Snapshot file: shellfirm/tests/snapshots/checks__fs-strict-folder_deletion.yaml.snap
Snapshot: fs-strict-folder_deletion.yaml
Source: shellfirm/tests/checks.rs:52
────────────────────────────────────────────────────────────────────────────────────────────────────
Expression: test_file_results
────────────────────────────────────────────────────────────────────────────────────────────────────
+new results
────────────┬───────────────────────────────────────────────────────────────────────────────────────
0 │+[
1 │+    TestSensitivePatternsResult {
2 │+        file_path: "fs-strict-folder_deletion.yaml",
3 │+        test: "rmdir TestFolder",
4 │+        check_detection_ids: [],
5 │+        test_description: "match command by finding existing file",
6 │+    },
7 │+    TestSensitivePatternsResult {
8 │+        file_path: "fs-strict-folder_deletion.yaml",
9 │+        test: "rmdir -pv TestFolder",
10 │+        check_detection_ids: [],
11 │+        test_description: "match command with flags",
12 │+    },
13 │+    TestSensitivePatternsResult {
14 │+        file_path: "fs-strict-folder_deletion.yaml",
15 │+        test: "rmdir    -pv    TestFolder",
16 │+        check_detection_ids: [],
17 │+        test_description: "match command with spaces",
18 │+    },
19 │+]
────────────┴───────────────────────────────────────────────────────────────────────────────────────
To update snapshots run cargo insta review
Stopped on the first failure. Run cargo insta test to run all snapshots.
thread 'test_checks' panicked at 'snapshot assertion for 'fs-strict-folder_deletion.yaml' failed in line 52', /Users/[username]/.cargo/registry/src/index.crates.io-6f17d22bba15001f/insta-1.20.0/src/runtime.rs:516:9
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
failures:
test_checks
test result: FAILED. 1 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.11s
error: test failed, to rerun pass `--test checks``

#129

Thanks @Geoff12889 for your PR.
I'll release a new version later on today


Recession Analyzer App - Metadata and Publication Review

Steps to reproduce
Expected Behavior
Positive Test Case
No QA necessary on this ticket.  The ticket itself is a review/QA effort to begin with.
Actual Behavior

port.zip


Bindings should not have parenthesis around them

Steps to reproduce
esprima.parse('[a, ...(b = c)] = 0')

Expected output
Should throw an error
Actual output
Parses an AST

This one too should throw an error [x] += 0


Type cast + less than

Describe the bug
The parser confuses type argument lists and the less than operator. That's the same issues we've had with type argument lists of calls.
To Reproduce
Place this code into the editor:
val filtered = houseRentData.filterRows((data) -> data.getValue("Bathroom") as Int < 5);

Expected behavior
No error. The < should be treated as the less than operator, not the start of a type argument list.
Screenshots (optional)
No response
Additional Context (optional)
No response

We cannot generally fix this. Adjusting the lexer only fixes the issue in some cases. Other languages, like TypeScript, have the same behavior.
We could give as and later is the lowest precedence, so parentheses must always be used when they occur inside another expression.
Or we could leave it as is, and document that parentheses must be used in this specific case (type cast followed by less than).

:tada: This issue has been resolved in version 0.15.0 :tada:
The release is available on:

npm package (@latest dist-tag)
npm package (@latest dist-tag)
v0.15.0
GitHub release

Your semantic-release bot :package::rocket:


Replace user by<EMAIL_ADDRESS>in Prestashop

Prestashop is using the email instead of the user, in these cases, we should change the value of the current variable user to the email<EMAIL_ADDRESS>@prydonius @sameersbn

/assign
/ok-to-test

/lgtm


Add CI for testing and building for main and dev branches

Renamed branch name from master to main in the CI

added test for main and dev branches for pull requests and merges.
before the tests was assigned to master branch which didn't exist (assuming main branch was named as master before)


Use RwLock instead of Mutex in ObjectCache

ObjectCache struct/implementation is used for library-wide global storage of vcx objects when using aries-vcx via FFI. The objects are identified by numeric "handle". These handles are keys to map, which is protected by mutex as there might be simultaneous calls into aries-vcx using this shared global data. Also each individual object inside the map is protected by mutex.
Mutex assures there's only 1 write OR read at any point of time. Having mutex wrapping the map hence prevents for example creating new connection, if currently existing connection is being read to access some of its data. This can probably drive performance down rapidly in concurrent applications.

Signed-off-by: Patrik Stas<EMAIL_ADDRESS>

Codecov Report

Merging #248 (287b334) into master (c872f0e) will decrease coverage by 0.00%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master     #248      +/-   ##
==========================================
- Coverage   57.38%   57.37%   -0.01%
==========================================
  Files         149      149
  Lines       19995    20003       +8
  Branches     4497     4499       +2
==========================================
+ Hits        11474    11477       +3
- Misses       5174     5177       +3
- Partials     3347     3349       +2

Flag
Coverage Δ

integration
25.98% <33.33%> (+<0.01%)
:arrow_up:

unittests
51.65% <33.33%> (-0.01%)
:arrow_down:

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

Impacted Files
Coverage Δ

libvcx/src/utils/object_cache/mod.rs
64.07% <33.33%> (-2.24%)
:arrow_down:

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 c872f0e...287b334. Read the comment docs.

Codecov Report

Merging #248 (287b334) into master (d33f116) will increase coverage by 7.34%.
The diff coverage is 33.33%.

:exclamation: Current head 287b334 differs from pull request most recent head 86ff9ac. Consider uploading reports for the commit 86ff9ac to get more accurate results

@@            Coverage Diff             @@
##           master     #248      +/-   ##
==========================================
+ Coverage   50.03%   57.37%   +7.34%
==========================================
  Files         149      149
  Lines       19883    20003     +120
  Branches     6266     4499    -1767
==========================================
+ Hits         9948    11477    +1529
- Misses       5054     5177     +123
+ Partials     4881     3349    -1532

Flag
Coverage Δ

integration
25.98% <33.33%> (+3.17%)
:arrow_up:

unittests
51.65% <33.33%> (+5.85%)
:arrow_up:

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

Impacted Files
Coverage Δ

libvcx/src/utils/object_cache/mod.rs
64.07% <33.33%> (+4.07%)
:arrow_up:

...dlers/proof_presentation/prover/states/finished.rs
50.00% <0.00%> (-13.64%)
:arrow_down:

...es/handlers/issuance/holder/states/request_sent.rs
70.00% <0.00%> (-3.69%)
:arrow_down:

libvcx/src/aries/handlers/issuance/messages.rs
30.76% <0.00%> (-2.57%)
:arrow_down:

...s/handlers/proof_presentation/verifier/messages.rs
33.33% <0.00%> (-2.39%)
:arrow_down:

...ibvcx/src/aries/handlers/issuance/holder/holder.rs
54.05% <0.00%> (-1.95%)
:arrow_down:

...ies/handlers/proof_presentation/prover/messages.rs
17.64% <0.00%> (-1.11%)
:arrow_down:

...dlers/issuance/issuer/states/requested_received.rs
67.50% <0.00%> (-0.93%)
:arrow_down:

...of_presentation/prover/states/presentation_sent.rs
70.00% <0.00%> (-0.59%)
:arrow_down:

...ation/verifier/states/presentation_request_sent.rs
70.96% <0.00%> (-0.47%)
:arrow_down:

... and 98 more

Continue to review full report at Codecov.

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

@mirgee good to think about unintended consequences, but I think even if RwLock is significantly slower than Mutex, the fact that read access to VCX Object of type X would block read accesses to all other objects of type X outweights any potential slowness of RwLocks.
I you look at implementation of any update method, for example disclosed proof:
HANDLE_MAP.get_mut(handle, |proof| {
   .. do stuff ..
  .. send something over network, synchronously ...
 }

This operation can take long time because it contains synchronous network and filesystem operations - and during this time, with Mutex any access to read information about other disclosed proofs are blocked

@mirgee good to think about unintended consequences, but I think even if RwLock is significantly slower than Mutex, the fact that read access to VCX Object of type X would block read accesses to all other objects of type X outweights any potential slowness of RwLocks.
If you look at implementation of any update method, for example disclosed proof:
HANDLE_MAP.get_mut(handle, |proof| {
   .. do stuff ..
  .. send something over network, synchronously ...
 }

This operation can take long time because it contains synchronous network and filesystem operations - and during this time, with Mutex any access to read information about other disclosed proofs are blocked

Agree

@mirgee good to think about unintended consequences, but I think even if RwLock is significantly slower than Mutex, the fact that read access to VCX Object of type X would block read accesses to all other objects of type X outweights any potential slowness of RwLocks.
If you look at implementation of any update method, for example disclosed proof:
HANDLE_MAP.get_mut(handle, |proof| {
   .. do stuff ..
  .. send something over network, synchronously ...
 }

This operation can take long time because it contains synchronous network and filesystem operations - and during this time, with Mutex any access to read information about other disclosed proofs are blocked

In this case, so would RwLock, but you are right,
HANDLE_MAP.get(handle, |proof| {
   .. do stuff ..
 }

blocks any other concurrent read access.


How to access the dashboard externally

I posted this question https://stackoverflow.com/questions/50511393/kubernetes-play-dashboard-access-from-outside
I cant seem to find a document on how to access it correctly?
Any hints?
I tried everything that I have known so far but I still am struggling with this.

Just replied in SO. If you hitting against the wall you can contact me in the #dockercommunity slack channel (https://community.docker.com/registrations/groups/4316)


Fix installation instructions

It seems that there is another directory inside pacman:
❯ ls pacman
coolbutuseless-pacman-919eb65/

❯ ls pacman/coolbutuseless-pacman-919eb65
LICENSE.txt	board.R		image/		sound.R		video/
README.md	game.R		sound/		sprites.R

This is macOS, but I think it is probably the same on all platforms.

@gaborcsardi thanks.  I think i must've pushed my fix in parallel with you filing this issue.
Should be all working now.


Add support for IAM Authorizers in combination with 2.0 payload

Issue #, if available:
https://github.com/aws/serverless-java-container/issues/780
Description of changes:
Add IAM authorizers in addition to existing JWT, Lambda authorizers
see also https://docs.aws.amazon.com/lambda/latest/dg/urls-invocation.html
By submitting this pull request

[x] I confirm that my contribution is made under the terms of the Apache 2.0 license.
[x] I confirm that I've made a best effort attempt to update all relevant documentation.

Great work, thank you very much for your contribution @jonife.


Disable file watching completely when watcher_disable is true

Right now, when watcher_disable is true, Middleman doesn't actually disable the watcher, it merely adds a few additional ignore paths. Not sure if there's a reason for this? It seems like not the desired behavior when one sets watcher disable to true.
It'd be great if it actually disabled the watcher entirely.
This is sort of a nuclear option, but the advantage of this is that I can implement my own custom watch behavior separately, under my own ready callback.

Seems reasonable, not sure why the if statement was scoped like tat.


Change Batch Management error to CloudError

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[ ] Validation errors from the Linter extension for VS Code have all been fixed for this spec. (Note: for large, previously checked in specs, there will likely be many errors shown. Please contact our team so we can set a timeframe for fixing these errors if your PR is not going to address them).
[x] The spec follows the patterns described in the Swagger good patterns document unless the service API makes this impossible.

@amarzavery Amar, this is the change to our existing specification as discussed with you over IM.

@xingwu1 Heads up this is the change we should make in your pending CR too.

@olydis Since this is our older Swagger spec and we are just changing it for consistencies sake, I am not going to apply all of the fixes which Xing did in #892 -- Xing's Swagger will be the authority going forward. Is that okay?

@olydis Can I get confirmation that you are okay merging just this change?
Thanks

@olydis Can this change be merged?

@matthchr yes, sorry for the delay

 https://github.com/Azure/azure-sdk-for-ruby/pull/678

 https://github.com/Azure/azure-sdk-for-node/pull/2058

 https://github.com/Azure/azure-sdk-for-python/pull/985


Ignoring certain patterns

Splitting this from #181

fzf is a command line filter that can be used with anything, so it doesn't have the notion of ignored files and folders.

ps -ef | fzf
seq 1 100 | fzf

When the input is not specified (fzf w/o STDIN pipe), it will use find command as the default input. You can set $FZF_DEFAULT_OPTS environment variable to a command that lists the files as you like.

Sorry my knowledge is limited, how can I supply a list of ignored patterns for FZF?

It's not the responsibility of fzf. You have to feed the right input to fzf, and for vim plugin, setting up $FZF_DEFAULT_COMMAND will do. You can set it to some command or your own script that lists the files under the current directory.
Please refer to: https://github.com/junegunn/fzf#respecting-gitignore-hgignore-and-svnignore

Note that you can also set the environment variable inside vim:
let $FZF_DEFAULT_COMMAND = 'ag -l -g ""'

For googlers - you can also add --skip-vcs-ignores to use .agignore but not .gitignore. This allowed me to also search in my vendor folder but ignore code coverage files, generated documentation, etc.


expectation as a new requirement

Search before asking

[X] I searched in the issues and found nothing similar.

Motivation
Demand scenario:
There is a source table of dwd layer, the main function is to use the primary key to de-duplicate, and the sequence-field is used to get the latest one. However, with a table of dws, I need to subscribe/stream read the table of dwd layer, so that it can get the latest one every time. At present, I refer to the changelog-producer. scan-mode, I have communicated with Mr.song and confirmed that it is a new requirement.
Solution
NONE
Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

The current default behavior is that there will be de-duplication within the checkpoint.
If you configure changelog-producer=full-compaction, it will de-duplicate more, depending on your latency requirements.


Simplify Stream factory methods

Currently there are

nil()
cons(T)
of(T...)
of(Iterable)
of(Iterator)
gen(Supplier)
from(int)
range(int, int)
rangeClosed(int, int)

It would be nice to get rid of

cons(T)
of(Iterator)

See also jOOQ/jOOL#89

Cannot remove factory methods, we need them all.
The implementation of particular factory methods was fixed by pull request #176