GH-35662: [CI][C++][MinGW] Avoid crash in FormatTwoDigits() with release build

Rationale for this change
I don't know why but the following combination is crashed:

Template FormatTwoDigits() implementation without explicit inline
MinGW
Release build

What changes are included in this PR?
This breaks the "template FormatTwoDigits() implementation without explicit inline" by specifying inline explicitly.
But I don't know why we can avoid this crash by specifying inline explicitly...
Are these changes tested?
Yes.
Are there any user-facing changes?
No.

Closes: #35662

Thanks a lot @kou !


Wrong pageCount in Pagination

ActiveDataProvider pagination returns wrong pageCount.
Reproducing
// Total 8 items.
// We have 5 items on page.
// It makes 2 pages.
$dataProvider = new ActiveDataProvider(['query' => Notes::find()]);
$dataProvider->setPagination(['pageSize' => 5]);
$pageCount = $dataProvider->pagination->pageCount;
Expected result
$pageCount is 2
What do you get instead
$pageCount is 0
Additional info
Console test is attached.
TestPageCountController.zip
Put controller class in console/controllers and run
yii test-page-count
Pull request with bugfix is provided.

Q
A

Yii version
2.0

PHP version
5.6

Operating system
Windows

I don't think setting the pagination component should fetch the models, It might cause confusion.
$query = Notes::find();
$dataProvider = new ActiveDataProvider(['query' => $query]);
$dataProvider->setPagination(['pageSize' => 5]);
$query->where(['foo' => 'bar']); //Will have no effect as the models have already been fetched.

@Alex-Code it is not.
It is calling prepareTotalCount which queries "COUNT(*)" on table. No models are fetched.
And every time query object is used in ActiveDataProvider, it is cloned. So there should be no problems.

Ah yes, It only performs a COUNT(*) but I think my issue above is still valid.
$query = Notes::find();
$dataProvider = new ActiveDataProvider(['query' => $query]);
$dataProvider->setPagination(['pageSize' => 5]);
$pageCount = $dataProvider->pagination->pageCount; //May return 5
$query->where(['foo' => 'bar']);
$pageCount = $dataProvider->pagination->pageCount; //Would still return 5 even though the query has changed.

I wouldn't expect setting the options of Pagination to initiate a query.
Maybe passing the total count to the Pagination object itself would be better.
$dataProvider->pagination->getPageCount($dataProvider->totalCount);

I assume that developer should not change query after fetching data.
Asking "what total page count" is fetching data.
If he needs to change query after fetching data, he must reset models with DataProvider methods, and fetch new data.

Thats documented. http://www.yiiframework.com/doc-2.0/yii-data-basedataprovider.html#getPagination()-detail
$dataProvider = new ActiveDataProvider(['query' => $query]);
$dataProvider->prepare();
$dataProvider->pagination->pageSize = 5;
$pageCount = $dataProvider->pagination->pageCount;

yii app work with php as concept dying model, is say Quang and all properies as public in components for once configure only before init (clear configure with expected behavior).
try:
$dataProvider = new ActiveDataProvider(['query' => $query, 'pagination' => ['pageSize' => 5]]);

@Faryshta this sucks, I dont need to retrieve models. All I need is correct page count.
Retrieving models will slow down my app.

@Nex-Otaku @lynicidn suggestion should solve the issue.

$dataProvider = new ActiveDataProvider(['query' => $query, 'pagination' => ['pageSize' => 5]]);
In my case this not enough, need add $dataProvider->prepare() after and it will work.
But if i set
$dataProvider->pagination->pageSize = 5;
$dataProvider->prepare();

without search() method it's not working.


Add: Clustering and multiple vectors per key

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


Fixing sign warps

Sign warps had broken in a new update of MCServer. They are working with this fix.

I did it after seeing this, sorry.


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from  to before v4.14.259",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=b03abd0aa09c05099f537cb05b8460c4298f0861"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "b03abd0aa09c05099f537cb05b8460c4298f0861",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "dm btree remove: fix use after free in rebalance_children()\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v4.14.259 by commit b03abd0aa09c05099f537cb05b8460c4298f0861. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2021-1002638


Add API to force quit with the ability to revert changes silently

Fix #21536
Adds "workbench.action.forceCloseActiveEditor" to externalAPI

@misoguy,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Added feedback via https://github.com/Microsoft/vscode/issues/21536#issuecomment-282990936

I'm closing this PR to create a new PR


NotFoundError: Node was not found

I've been using dat.gui with babylon.js without any problems, but when I tried to change its position, it's showing me this error

NotFoundError: Node was not found

In my project I update the canvas everytime I load a text file to it. What I was doing was destroying the gui everytime I did that, but for some reason, after I decided to change its position, it won't work anymore after I update the canvas
This is my initgui code:
var confirmGui = false;
var initGui = function(parameters){
//this confirms if the gui is already created
	if (confirmGui){
		gui.destroy();
	}
	gui = new dat.GUI();
	var folder = gui.addFolder('Options');
	folder.open();

	/* rest of the code */

	gui.domElement.id = 'renderCanvas';
	customContainer = $('.moveGUI').append($(gui.domElement));
	confirmGui = true;
}

I can't see what am I doing wrong, can anyone help me?

Well, just managed another way, by adding this instead of using the css:
	gui.domElement.style.marginTop = "42px";
	gui.domElement.style.marginRight = "-16px";


Add solution to issue #1

i have made the changes, is there a resubmit button? or how does this typically work?
https://github.com/emdavis7/Python-Guide-for-Beginners/blob/master/SearchingAlgorithms/search_linear.py


Volume_Source_Space question

I'm trying to setup a sparse (10-15 sources) volume source space, similar to the standard source spaces used in besa. I'm using setup_volume_source_space but I'm unclear what correct coordinate frame I should use for pos['rr'].
I started by using mri ras coordinates. However when computing fwd models some sources were being excluded for being outside of the BEM inner skull. In trying to track this down I found the following
https://github.com/mne-tools/mne-python/blob/master/mne/viz/misc.py#L306-L307
Which suggests that the voxel to MNE mri RAS transform is a bit strange. I assume that this has to do with the coordinate frame used by freesurfer for the surface files? Is this modified transform used consistently through out MNE to map coordinates to MR locations?
Should I be using this modified transform to compute pos['rr'] from RAS MRI world coordinates (such as those available from fslview)?
Thanks.
Luke

what you want is what we call a "discrete source space".
You need to call setup_volume_source_space with pos being a dict as here:
https://github.com/mne-tools/mne-python/blob/master/mne/tests/test_source_space.py#L190
unfortunately it does not seem to be illustrated with an example.
let me know if you need more info

Thanks Alex,
The issue is that from the documentation for setup_volume_source_space its unclear what the coordinate frame should be for the locations that you pass in.
I did what I suspect many users would do which is to open the T1.mgz in a viewer (Slicer3d or paraview etc) and then use the RAS locations returned from the viewer. This doesn't work as it seems like internal MRI coordinate frame used by freesurfer (and MNE) doesn't respect the origin of the voxel to ras transform.
When I use the modified vox2ras transform to map the images RAS coordinates into the modified freesurfer space, I get much more reasonable results and the mne-python visualization shows the sources in the correct anatomy.
If there is interest and i have some time I'll put together a PR to simplify the coordinate transforms and illustrate the issue.

When I use the modified vox2ras transform to map the images RAS coordinates into the modified freesurfer space, I get much more reasonable results and the mne-python visualization shows the sources in the correct anatomy.

Perhaps this is what comes up as vox2ras_tkr in nibabel?

If there is interest and i have some time I'll put together a PR to simplify the coordinate transforms and illustrate the issue.

Yes please, we definitely need some coordinate frame tutorial to talk about these issues. If it's comprehensive that's awesome, but even something simple to start that we can build from would help.

I had forgotten about vox2ras_tkr. at least in my test subject vox2ras_tkr matches the "hack" from https://github.com/mne-tools/mne-python/blob/master/mne/viz/misc.py#L306-L307
Is it safe to assume that the tkr (tkregsiter) https://surfer.nmr.mgh.harvard.edu/fswiki/CoordinateSystems is in fact the coordinate frame that is used by the BEM surfaces and is therefor the MNE-MRI coordinate frame?

Yes I think it should be the case / the correct thing to use.

Nothing to do here other than improve our docs, probably having to do with coordinate frames, which we have noted elsewhere. So closing


Wamp session IDs are insecure

WampConnection classes use uniqid() to generate an identifier that is later used by the user. This function gives fairly predictable strings (there's even a warning on the official PHP docs) and shouldn't be used for this purpose.
Preferably session identifiers would be generated using something like openssl_random_pseudo_bytes or another safer PRNG.

You could make use of this library: https://github.com/ircmaxell/RandomLib it's fairly flexable for creating good random numbers vs fast ones.

This is what we use in Thruway:
It generates random session ids in a range that fits inside of most languages' integer representations.
    /**
     * Generate a unique id for sessions and requests
     * @return mixed
     */
    public static function getUniqueId()
    {
        $filter      = 0x1fffffffffffff; // 53 bits
        $randomBytes = openssl_random_pseudo_bytes(8);
        list($high, $low) = array_values(unpack('N2', $randomBytes));
        return abs(($high << 32 | $low) & $filter);
    }


chore: update versions: minikube, actions-setup-minikube, kubernetes

Signed-off-by: Alex Renn<EMAIL_ADDRESS>

@project-administrator can we update the pre-requisites https://github.com/trustbloc/k8s/blob/main/README.md#prerequisites ?

@project-administrator can we update the pre-requisites https://github.com/trustbloc/k8s/blob/main/README.md#prerequisites ?

Created PR to update README in the k8s repo: https://github.com/trustbloc/k8s/pull/259


feat: Release V1 🥳

https://wandb.atlassian.net/browse/WB-9017
https://wandb.atlassian.net/browse/WB-8911
https://wandb.atlassian.net/browse/WB-8912

This PR is included in version 1.0.0 :tada:


Update dhruv8433_pr.md

Group 3 of unmerged pr

https://github.com/JiyaGupta-cs/ShopNex/pull/353

@sanjay-kv
please check this unmerged pr


HIVE-28091: Remove invalid long datatype in ColumnStatsUpdateTask

What changes were proposed in this pull request?

Remove invalid long datatype in ColumnStatsUpdateTask::constructColumnStatsFromInput
Why are the changes needed?

IMO, Hive column does not support long data type. We should remove the incorrect data type in ColumnStatsUpdateTask.
https://github.com/apache/hive/blob/a348e5d86adc611e4f72d386815a21a96b667ab4/ql/src/java/org/apache/hadoop/hive/ql/exec/ColumnStatsUpdateTask.java#L104-L106
In addition, the column stats related code blocks should be consistent with code in StatObjectConverter.java, which also does not have long type.
https://github.com/apache/hive/blob/a348e5d86adc611e4f72d386815a21a96b667ab4/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/StatObjectConverter.java#L378-L379
Does this PR introduce any user-facing change?

No
Is the change a dependency upgrade?

No
How was this patch tested?

Existing UTs

@zhangbutao , could we extract this check logic into some method to be consistent in multiple places where this check is used?

@zhangbutao , could we extract this check logic into some method to be consistent in multiple places where this check is used?

It seems that it is not easy to extract this check logic.  ColumnStatsUpdateTask::constructColumnStatsFromInput is used for update/write table/partition stats, and StatObjectConverter::getTableColumnStatisticsObj is used for get/read table/partition stats. The two method has some different check & one is in hive-exec and another is in standalone-metastore.


Multiple where() condition doesn't work

Hello there,
I have a problem with multiple where() conditions for my DB, I have start and end dates for my news article, I'm trying to do following to select 'active' news item:
$news = $newsStore
        ->orderBy( 'asc', 'order' )
        ->where( 'startDate', '<', time() )
        ->where( 'endDate', '>', time() )
        ->fetch();

Looks like first condition is completely ignored and only second one is applied.
Any help please?

Got the same issue!

@snoopy4ever @typomedia @rakibtg  Guys, here is the solution for this problem - https://github.com/rakibtg/SleekDB/pull/23
I also forked (to create PR) this repository and merged that fix - https://github.com/lukasznowicki/SleekDB

Nice one mate 👍
Dzieki Lukasz 🥇

@lukasznowicki thanks for the contribution, appreciate that you take the time and to fix the issue. I will merge your PR today!

@lukasznowicki thanks for the contribution, appreciate that you take the time and to fix the issue. I will merge your PR today!

So where is the merge?!

@lukasznowicki Didn't worked for me. I copied your src/traits/helpers.php but the bug is still there (

@snoopy4ever I think you should use orWhere like this:
$news = $newsStore
        ->orderBy( 'asc', 'order' )
        ->where( 'startDate', '<', time() )
        ->orWhere( 'endDate', '>', time() )
        ->fetch();

I will close the issue here, feel free to post a comment and re-open it.


Subarray

Description of  the change
Introduction to subarray and subsequence in Array folder.
With full Explanation.
Domain of Docs

[ ] C
[ ] C++
[ ] Python
[ ] Java

Are you contributing under any open-source program ?
Hacktoberfest

@girija0707 @siddhi-244 can you please assign this to mean under hacktober2021.

Assigning this to you🎊🎊🎊🎊. This is a documentation project.  Please refer to merged prs.


WIP - DO NOT MERGE: removed unused topics. Fixes RHOARDOC-1193

replaced by: https://github.com/fabric8-launcher/launcher-documentation/pull/1139


assets file working on IOS Emulator Not on Physical device

Hello @khjde1207 Placing tessdata folder under assets folder does not work, it gives following error
flutter: /var/mobile/Containers/Data/Application/0900C698-99AD-4D93-BFEA-661985250E38/Library/Caches/test.jpg
Error Domain=NSCocoaErrorDomain Code=513 "You don’t have permission to save the file “tessdata” in the folder “XXXX”." UserInfo={NSFilePath=/private/var/containers/Bundle/Application/DABD394C-FB08-40B1-9FE4-D444B11E55C8/Runner.app/tessdata, NSUnderlyingError=0x283266010 {Error Domain=NSPOSIXErrorDomain Code=1 "Operation not permitted"}}
Error opening data file /private/var/containers/Bundle/Application/DABD394C-FB08-40B1-9FE4-D444B11E55C8/Runner.app/tessdata/eng.traineddata
Please make sure the TESSDATA_PREFIX environment variable is set to your "tessdata" directory.
Failed loading language 'eng'

the way around to add tessdata folder at root on runner folder too using XCode as mentioned  here

If you can solve it, can you make a pull request?

Sounds Good. I'll update and share it with you


Skip removing segments from upsert metadata manager when table is disabled

Currently, we remove segments from upsert metadata manager when table is disabled. This leads to high resource usage unnecessarily.

Codecov Report
Attention: 14 lines in your changes are missing coverage. Please review.

Comparison is base (8c86ad4) 61.57% compared to head (5d394d0) 46.40%.
Report is 3 commits behind head on master.

Files
Patch %
Lines

...cal/upsert/BasePartitionUpsertMetadataManager.java
0.00%
7 Missing :warning:

...ache/pinot/segment/local/upsert/UpsertContext.java
0.00%
5 Missing :warning:

...t/local/upsert/BaseTableUpsertMetadataManager.java
0.00%
2 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #12250       +/-   ##
=============================================
- Coverage     61.57%   46.40%   -15.18%
+ Complexity     1153      945      -208
=============================================
  Files          2416     1810      -606
  Lines        131199    95204    -35995
  Branches      20249    15356     -4893
=============================================
- Hits          80788    44178    -36610
- Misses        44509    47874     +3365
+ Partials       5902     3152     -2750

Flag
Coverage Δ

custom-integration1
?

integration
?

integration1
?

integration2
?

java-11
?

java-21
46.40% <0.00%> (-15.06%)
:arrow_down:

skip-bytebuffers-false
?

skip-bytebuffers-true
46.40% <0.00%> (-15.03%)
:arrow_down:

temurin
46.40% <0.00%> (-15.18%)
:arrow_down:

unittests
46.40% <0.00%> (-15.17%)
:arrow_down:

unittests1
46.40% <0.00%> (-0.22%)
:arrow_down:

unittests2
?

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

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


Add "Tensei shitara Slime Datta Ken 2nd Season Part 2"

@erengy Without this commit, episode 37 is being recognized as episode 13 of season 2 (it shows as 13/12), which very weird because the S2 rule is only up to 36.
[SubsPlease] Tensei Shitara Slime Datta Ken - 37 (720p) [B6DC2F9E].mkv


What are the right names for the preview features?

I've been playing with setting up a new test cluster for a couple of evenings. It gives me this warning:
Your deployment contains Preview Features which may require subscription registration and have Azure Region limitations. Please ensure you have registered for these previews, and have installed the 'az extension add --name aks-preview' before running the relevant scripts.
Preview Features you have selected: createNatGateway, aksOutboundTrafficType, natGwIdleTimeout, kedaAddon, blobCSIDriver, oidcIssuer, workloadIdentity, networkPluginMode.

But these words don't seem to actually be the names of the Preview Features. Is there some way to to map "kedaAddon" to "Microsoft.ContainerService/AKS-KedaPreview" for instance?
The feature I'd really like is for it to generate the necessary az feature register commands into the prep script..
But the bottom line is, if we know what the preview features are, why are we listing them using the wrong names?

We're using the bicep parameter names, and don't maintain a map between the parameter names and the feature.
Seems like a good idea to polish this a little as you've suggested.

@khowling - any thoughts on this one?
Could we use a simple map between parameter names and further metadata?


Error running v4.5 on Colab

Traceback (most recent call last):
  File "style2paints/V4.5/s2p_v45_server/Style2PaintsV45_source.py", line 231, in <module>
    tail_op_g, head_op_g, render_op_g, refs_img_g = make_graph()
  File "style2paints/V4.5/s2p_v45_server/Style2PaintsV45_source.py", line 197, in make_graph
    features = reader(ip3 / 255.0)
  File "/usr/local/lib/python3.8/dist-packages/keras/utils/traceback_utils.py", line 70, in error_handler
    raise e.with_traceback(filtered_tb) from None
  File "/usr/local/lib/python3.8/dist-packages/keras/engine/input_spec.py", line 295, in assert_input_compatibility
    raise ValueError(
ValueError: Input 0 of layer "model_1" is incompatible with the layer: expected shape=(None, 224, 224, 3), found shape=(None, 128, 128, 3)

Keep returning error. Colab doesn't support tf1 so I running this on tf2.
I assumed this code compatible with tf2 because this exist on source code:
import tensorflow

tensorflow.compat.v1.disable_v2_behavior()
tf = tensorflow.compat.v1

Help is appreciated, thanks.

您好(*?0?7▽｀)ノノ青弦已经收到您的邮件啦！请耐心等待，10个工作日内会给您回复w

@sulaimangari Hello, I have the same problem, can I ask that how did you solve it?

Can u provide the colab link??

I end up using v4.0, it looks like the dev just focused on the new released with UI now.

I end up using v4.0, it looks like the dev just focused on the new released with UI now.

Please can you provide colab link?

Same for me.
colab link: https://colab.research.google.com/drive/12jJn9cEVeDLGwA9bSjq4jV82AaaiG26G?usp=sharing

If it helps anyone:
https://highcwu-style2paints-4-5-gradio.hf.space/file/ui/web-mobile/index.html
https://huggingface.co/spaces/HighCWu/Style2Paints-4.5-Gradio

Bro thanks,this is a big help. I will try it soon. 😊 thanks


vtgate crashes when trying to watch on etcd topology

My vtgate crashed today and when I try to run it again, it fails immediately with the following output.
I1217 14:37:36.939128    7736 discoverygateway.go:76] loading tablets for cells: peste
I1217 14:37:36.939697    7736 discoverygateway.go:97] Waiting for tablets
I1217 14:37:36.939955    7736 cell.go:103] etcdtopo: global address list = http://X.X.X.X:YYYY
E1217 14:37:37.026937    7736 serving_graph.go:81] modifiedVersion: 1425
E1217 14:37:37.039516    7736 serving_graph.go:81] modifiedVersion: 45
E1217 14:37:37.040201    7736 serving_graph.go:81] modifiedVersion: 118
E1217 14:37:37.040206    7736 serving_graph.go:83] Watch on /vt/ns/peste/_Data failed, waiting for 30s to retry: 401: The event in reque
sted index is outdated and cleared (the requested history has been cleared [1587/1045]) [2586]
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x10 pc=0x4e6b55]

I've added this log log.Errorf("modifiedVersion: %v", modifiedVersion) just before https://github.com/youtube/vitess/blob/v2.0.0/go/vt/etcdtopo/serving_graph.go#L81
And modifiedVersion was wrong, it should have been 1587 but it was 118.
I already made a hotfix for myself by hardcoding the modified version but I have to change it every time I need to restart vtgate or launch a new instance of it.

I'll look at it on Monday, I need to refactor that code anyway. Thanks for the report!

https://github.com/youtube/vitess/pull/2394 has support for the new etcd client, with reworked code. It should be a lot more stable.

This should be fixed now.


style(contrib/aws): Ensure that files are closed and use yaml.safe_load

The with statement ensures that files will be always be closed, this also switches from the deprecated file to open.
Also use yaml.safe_load in favor of yaml.load, which can execute arbitrary code.

Thanks for the contribution! Please ensure your commits follow our style guide. This code will be tested once a Deis maintainer reviews it.

Jenkins, add to whitelist.

code LGTM

LGTM.


Fix: exclude setting to be considered for editable packages as well

Description

GitHub Issues

Coverage decreased (-61.3%) to 11.955% when pulling 83ed049c69b6704e2e729ea7f50d9249a243e201 on pnija:master into c46d4641ac4052225bebcdf552b9ed77e0c27c95 on Miserlou:master.


Add theme suggestions for Views

Adding template suggestions for Views could be helpful:
/**
 * Implements hook_theme_suggestions_HOOK_alter().
 */
function HOOK_theme_suggestions_views_view_alter(array &$suggestions, array $variables) {
  $suggestions[] = 'views_view__' . $variables['view']->id();
}

This snippet, might need improvement/fix, as I'm getting multiple identical suggestions:

Do we really need this? Template suggestion for Views are available in core, see https://www.drupal.org/docs/develop/theming-drupal/twig-in-drupal/twig-template-naming-conventions#s-views
The issue with the suggestions for Views specifically is that they don't show up when debug is enabled, but the naming convention is pretty clear and simple.
What do you think @rutiolma ?

Having a debug to know the views template to use is very useful and I often miss it but I also agree that the documentation is pretty clear and simple.
The proposed hook, besides the repeated debug info, is a bit limited since it works only for the view template but not for the displays or fields. Also, the repeated template suggestion, is due to a core issue so we cannot do much but patch it.
I see that there are several core issues trying to tackle this views template issue and https://www.drupal.org/project/drupal/issues/2118743 is pretty active and is the one that better fixes the issue but, until there's a stable patch, I don't think we should be adding patches to core just for debug.
So, in short, I agree with the necessity to have template suggestions but I don't think we have a good way to fix it.
I would suggest postponing this issue until there's a core improvement.


Update cfi_intro.adoc: remove redundant 'rd == x0'

Remove redundant 'rd == x0' in the definition of 'return'.

the cfi spec is not inside isa-manual
I created https://github.com/riscv/riscv-isa-manual/pull/1690 from your suggestion

Thanks @kacouane @jrahmeh. Closing this PR.


⚠ Bofrost.at has degraded performance

In 5e90f98, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1068 ms

Resolved: Bofrost.at performance has improved in 070bb4e.


Allow access to critical ports only from localhost

Docker and ufw both use iptables to manage networks. IMHO Docker rules overrride the ufw rules that secure the host. Sometimes it helps just to sudo ufw reload after the container start to resolve the issue. But this is not always the case.
There's a long discussion on how to deal with this highly critical issue. Surprisingly for years without a canonical approach.
A very simple/straight forward way to solve this is to restrict the docker network to expose specific ports only to localhost:
Example:

    ports:
      - ${GRAFANA_PORT}:3000/tcp

... changes to:

    ports:
      - <IP_ADDRESS>:${GRAFANA_PORT}:3000/tcp

I did this for all ports that are noted as 'sensitive' in the default.env. Tested successfully for prysm-grafana.yml

I like it, this is potentially much cleaner than forcing ufw into the chain. It'd break with eth1 standalone however, I think. More testing needed before this is accepted. It'd also restrict grafana in non-cloud environments, though we're about to solve that another way with traefik.

I like it, this is potentially much cleaner than forcing ufw into the chain. It'd break with eth1 standalone however, I think. More testing needed before this is accepted. It'd also restrict grafana in non-cloud environments, though we're about to solve that another way with traefik.

Closing this in favor of using traefik. People who do not want traefik can still use ufw in the chain instead.


Retrieving artifacts from release task

Hi all,
I'm building a task that should be used in the Release Management. My task needs to retrieve the available artifacts. How do I get a list of available artifacts and get the path/stream of those files?
Thanks

The release definition will automatically download all artifacts to the build agent.  Can you give me the scenario where you are trying to complete

Hi @chrisrpatterson, I am trying to search for a particular extension in between available artifacts and I need to get the full path towards the files found in between the available artifacts. Is that possible? In case you would have an artifact uploaded by the build called my.zip, in the release task POsH script, how will you make sure the artifact is there and get it's full path?
Thanks

Yes the artifacts are downloaded into a known location.  You can read about it here https://msdn.microsoft.com/en-us/Library/vs/alm/Release/author-release-definition/understanding-artifacts#Artifactdownload

Can we retrieve multiple files by matching through pattern. If yes, please give example.


np.prod([1]) should return a Scalar

Python:

NumSharp-v0.10.x:

NumSharp-umanaged-storage:

Fixed in unmanaged-bytes-storage branch


Add support for ls -T and ls -E in coreutils ls

-T lists the tag:
ls -T a.txt
t ISO8859-1   T=on  a.txt

For now I'm adding a bypass in https://github.com/ZOSOpenTools/coreutilsport/pull/75. A better solution will be to implement logic for handling -T and -E in ls.c so I will keep this open


cinnamon.cinnamon-session: 6.0.2 -> 6.0.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/cinnamon-session/versions.
meta.description for cinnamon.cinnamon-session is: The Cinnamon session manager
meta.homepage for cinnamon.cinnamon-session is: https://github.com/linuxmint/cinnamon-session
Updates performed

Version update

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 6.0.3 with grep in /nix/store/f4p9yz753fbsdfhj3lfxv4gsmivirlcr-cinnamon-session-6.0.3
found 6.0.3 in filename of file in /nix/store/f4p9yz753fbsdfhj3lfxv4gsmivirlcr-cinnamon-session-6.0.3

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

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath
cinnamon.cinnamon-common
cinnamon.cinnamon-gsettings-overrides
cinnamon.cinnamon-screensaver
cinnamon.cinnamon-session

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/89cc7d8e438c986ad8090b031015f5adf39bc0cc#cinnamon.cinnamon-session

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

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

4 packages built:

cinnamon.cinnamon-common
cinnamon.cinnamon-gsettings-overrides
cinnamon.cinnamon-screensaver
cinnamon.cinnamon-session

Maintainer pings
cc @bobby285271 @mkg20001 for testing.

@ofborg test cinnamon cinnamon-wayland


[BUG] Duplicate ItemId's being generated, causing references to be made that should not be made.

Environment
Self-Hosted (Docker)
System
Chrome, 108.0.5359.125
Version
2.1.1
Describe the problem
I have some sections for managing some domains that I own.
Their title is set like this:
Domain: SomeDomain.eu
Domain: SomeDomain.nl
These both generate the id 1958, as can be seen in this jsfiddle
The effect this has is that if I edit items inside one section, the corresponding item in the other section gets updated too.
It even generates some &ref references in the yaml config file when updating.
Additional info
No response
Please tick the boxes

[X] You have explained the issue clearly, and included all relevant info
[X] You are using a supported version of Dashy
[X] You've checked that this issue hasn't already been raised
[X] You've checked the docs and troubleshooting guide
[X] You agree to the code of conduct

I didn't have any coffee yet when I created the bug report, as it was still early for me ;)
But I guess this is due to the fact that the charcodes of 'eu' combined equal the value of the charcodes of 'nl' combined:
charcode n -> 110
charcode l -> 108
makes 218
charcode e -> 101
charcode u -> 117
makes 218
So 'unhappy' coincedence I guess.

I guess it could be fixed in a fashion similar to what I've done in this jsfiddle
Original
const charSum = sectionTitle.split('').map((a) => a.charCodeAt(0)).reduce((x, y) => x + y);
Improved
const charSum = sectionTitle.split('').map((a) => a.charCodeAt(0)).reduce((x, y) => ((x << 5) - x) + y) | 0;
This does produce a unique item id, even for the same input that clashed for me initially.


When will the training code be released?

same question

We're all looking forward to the open source of the training part of the code

same question

While this is not the whole pretraining code but a we have provided code to finetune the model here :https://github.com/google-research/timesfm/tree/master/notebooks


Support new permissions model API 23

As we only use INTERNET and ACCESS_NETWORK_STATE permissions, the system grants automatically this permission and the user cannot revoke at any time because those permissions are contained in Normal Permissions.
Referecence: http://www.androidpolice.com/2015/06/06/android-m-will-never-ask-users-for-permission-to-use-the-internet-and-thats-probably-okay/


[Event Request] codeunit 312  "Cust-Check Cr. Limit" - OnSetCustCheckCreditLimitServiceContractHeaderCheck

Hello, would it be possible to add OnSetCustCheckCreditLimitServiceContractHeaderCheck,  event to codeunit 312  "Cust-Check Cr. Limit", like this:
    procedure ServiceContractHeaderCheck(ServiceContractHeader: Record "Service Contract Header")
...
              exit;

         OnNewCheckRemoveCustomerNotifications(ServiceContractHeader.RecordId, true);

>         OnSetCustCheckCreditLimitServiceContractHeaderCheck(CustCheckCreditLimit);//New Event
         if CustCheckCreditLimit.ServiceContractHeaderShowWarningAndGetCause(ServiceContractHeader, AdditionalContextId) then
             CreateAndSendNotification(ServiceContractHeader.RecordId, AdditionalContextId, '');
     end;

     [IntegrationEvent(false, false)]
     local procedure OnSetCustCheckCreditLimitServiceContractHeaderCheck(var CustCheckCreditLimit: Page "Check Credit Limit")
     begin
     end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 10644.


feat(grafana): Add total disk utilization dashboard

Overview
Adds a total disk utilization dashboard to Grafana that sums the size of all static files + the size of all MDBX tables for a quick look at the disk footprint of the instance.

I think we have this now: https://github.com/paradigmxyz/reth/blob/c6857efa8a10e2e86396ac6385d1008b8128e694/etc/grafana/dashboards/overview.json#L800-L814
Any chance you could open up a new PR for the RSS feed on top of main?
Sorry for this 🙃


New :CornelisNormalize features: custom compute modes & normalization in holes

I was missing features from :CornelisNormalize, so tried my hands at implementing them:

The command now accepts an optional compute mode, similarly to :CornelisTypeContext and friends.
When executed from within a hole, hole contents are normalized in that hole's context.

Please tell if these are features you'd like to see in cornelis and whether they require any adjustments :)
Thank a lot for your hard work, it is much appreciated!

Love it, thanks!


Merge ign-launch1 to ign-launch2

Signed-off-by: Nate Koenig<EMAIL_ADDRESS>

The ign-gazebo 3.4.0 failed for amd64 on Focal, which is causing the CI here to fail. We should fix that before merging this.
https://build.osrfoundation.org/job/ign-gazebo3-debbuilder/55/console

ign-gazebo 3.5.0 was correctly released for Focal / amd64. The Jenkins Ubuntu build's failure should be fixed by https://github.com/ignition-tooling/release-tools/pull/339.
I think this is good to go!


Bring up Trinity.TSL

The TSL compiler consists of multiple projects using C#, C++ and C++/CLI.
We have to first clear it up a bit and bring them up here gradually.
Also, C++/CLI will not be cross-platform in the near future, so we have to port it back to C#.

[x] https://github.com/Microsoft/GraphEngine/issues/36
[x] https://github.com/Microsoft/GraphEngine/issues/35
[x] https://github.com/Microsoft/GraphEngine/issues/65

Done.


Update Files in swift 4

Hello, really nice lib. Did you plan swift 4 migration ?

I don't why but my Xcode tells me that your lib need a swift 4 migration. But when i try, there is nothing to update. And I saw that your lib is swift 4 compatible.


⚠ Bofrost.at has degraded performance

In 609bfbe, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1179 ms

Resolved: Bofrost.at performance has improved in 7820558 after 13 minutes.


Update smoke test to work with openj9 jdk

Update smoke test to work with openj9 jdk
Related: https://github.com/adoptium/ci-jenkins-pipelines/issues/127#issuecomment-819194309

Linux before:
https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk11u/job/jdk11u-linux-x64-openj9_SmokeTests/2/#showFailuresLink
Fix:
https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk11u/job/jdk11u-linux-x64-openj9_SmokeTests/3/#showFailuresLink

Lots of linter failures:
https://github.com/AdoptOpenJDK/openjdk-build/pull/2569/checks?check_run_id=2339220837 Not sure if related?

@karianna Not related. Linter failures should be fixed in #2570

OK to take this out of draft then?

I have re-run all openj9 smoke tests under https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk11u/ with this PR branch. Fix issue with all platform except aarch and s390, which will do in a separate PR.


Web scraping of Amazon Kindle #317

Description
This is to scrape book information from amazon kindle website like book title, book author, ratings, price, book url.
Resolves: 317

Here are the updates-

Great work @Tisha6661 !!
Do not forget to

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


[Feature Request] http-client-fetch: return responseData as part of error if !response.ok

Hi,
We are integrating with SAP using your @odata2ts/http-client-fetch package. Everything works well, but SAP only provides very minimal and static error message if there is a validation error.  The actual error is returned in the response body.
Currently, fetchResponse.json() is performed within the http-client-fetch package, making it impossible to retrieve the body for further use. The JSON body is awaited and read; but -in SAP case- only the general message is added to the error.
Request:
It would be helpful to return the responseData value as part of the response object in the FetchClientError.

Above implementation is maybe a bit rough. But provided us the additional insight we needed.

Hi @ralfnl,
I understand that you only get a quite generic error message from the payload. So you're telling me that:

you get proper JSON as response data
somewhere in this response data is a more fine-grained validation message to be found

If I've got that correct, then you would be fine with defining your own ErrorMessageRetriever. Each HTTPClient allows to set this via setErrorMessageRetriever(getErrorMsg: ErrorMessageRetriever), which is not documented 😅. But as can be seen from your screenshot: This method defines the error message that is passed into the FetchClientError.
But in any case, can you please show me this kind of response structure, you're talking about?

Hi @texttechne,
Thank you for your swift response. I had a try on the 'hidden feature' and indeed it provides me with the original body. I think that could partly help us out to at least show 'better' error messages. Being able to access the entire response JSON as context on the error would be even greater. We would like to log that for easier debugging.
Cramming the entire response body into the string message would leave a bit of a hackey result. 🙃
As requested the SAP response. Both in SAP and Dutch, so hold tight 😁
{
    "error": {
        "code": "ZPS/031",
        "message": "Er zijn validatiefouten opgetreden",
<EMAIL_ADDRESS>"Provider_Application_Error",
        "details": [
            {
                "code": "ZPS/044",
                "message": "Geen standaardproject gedefinieerd voor vestiging XXX, projectprofiel XXX",
<EMAIL_ADDRESS>"error",
<EMAIL_ADDRESS>4
            }
        ],
        "innererror": {
            "ErrorDetails": {
<EMAIL_ADDRESS>{
                    "ComponentId": "XXX",
                    "ServiceRepository": "XXX",
                    "ServiceId": "XXX",
                    "ServiceVersion": "0001"
                },
<EMAIL_ADDRESS>"XXX",
<EMAIL_ADDRESS>"XXX",
<EMAIL_ADDRESS>{
                    "Analysis": "XXX",
                    "Note": "XXX"
                }
            }
        }
    }
}

Hi @ralfnl,
thanks for the example.
Here you go:

Pull Request
So you get the parsed JSON as responseData from the FetchClientError
Of course, if fetch or the conversion to JSON failed the field is undefined

Released with<EMAIL_ADDRESS>

Please test and close the issue if successful.

Thank you @texttechne ! This works like a charm. Thank you for your swift response.


Improved output of initial state

This PR closes #61 and #48.
A new function reduce() is added to the Output class, which projects a state to the last step in the multi-step dimension.
This function is then used to define write_initial_state() as a combination of reduce() and write_state() for the netcdf and raw output.
The naming of the raw output is brought in line with the naming of the plot output.

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

Project coverage is 75.00%. Comparing base (f050bb0) to head (cb43061).
Report is 3 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop      #62   +/-   ##
========================================
  Coverage    75.00%   75.00%
========================================
  Files            1        1
  Lines            4        4
========================================
  Hits             3        3
  Misses           1        1

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

Questions:

Currently default config is not to write the intial state to output. Shouldn't it be better if the default is to write the initial state?
Plot output does the projection to last step of in the multi-step dimension explicitly (and for all states, not just the initial state). This could now be done using reduce() and possibly applied only in write_initial_state. Should we change this?


Warnings of duplicate dependencies

Description
when i run yarn start using Yarn 3.5.1 i get the following warnings in the browser.
i didn't get the warnings previously when i was running it using Yarn 1.
it seems to be because there are version conflicts between what is in the project root yarn.lock, and squid/package-lock.json
browser-external:buffer:9 Module "buffer" has been externalized for browser compatibility. Cannot access "buffer.Buffer" in client code. See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
get @ browser-external:buffer:9
Show 1 more frame
Show less
versionDetect.js:66 @polkadot/wasm-crypto requires direct dependencies exactly matching version 7.2.1.
Either remove and explicitly install matching versions or dedupe using your package manager.
The following conflicting packages were found:
	7.2.2	@polkadot/wasm-bridge
	7.2.2	@polkadot/wasm-crypto-wasm
warn @ versionDetect.js:66
Show 1 more frame
Show less
versionDetect.js:66 @polkadot/util has multiple versions, ensure that there is only one installed.
Either remove and explicitly install matching versions or dedupe using your package manager.
The following conflicting packages were found:
	esm 12.3.2	node_modules/.vite/deps/
	esm 12.3.2	node_modules/.vite/deps/
warn @ versionDetect.js:66
Show 1 more frame
Show less
browser-external:buffer:9 Module "buffer" has been externalized for browser compatibility. Cannot access "buffer.Buffer" in client code. See http://vitejs.dev/guide/troubleshooting.html#module-externalized-for-browser-compatibility for more details.
get @ browser-external:buffer:9
Show 1 more frame
Show less
versionDetect.js:66 @polkadot/util-crypto has multiple versions, ensure that there is only one installed.
Either remove and explicitly install matching versions or dedupe using your package manager.
The following conflicting packages were found:
	esm 12.3.2	node_modules/.vite/deps/
	esm 12.3.2	node_modules/.vite/deps/
warn @ versionDetect.js:66
Show 1 more frame
Show less
versionDetect.js:66 @polkadot/wasm-crypto has multiple versions, ensure that there is only one installed.
Either remove and explicitly install matching versions or dedupe using your package manager.
The following conflicting packages were found:
	esm 7.2.1	node_modules/.vite/deps/
	esm 7.2.1	node_modules/.vite/deps/
warn @ versionDetect.js:66
Show 1 more frame
Show less
versionDetect.js:66 @polkadot/wasm-bridge has multiple versions, ensure that there is only one installed.
Either remove and explicitly install matching versions or dedupe using your package manager.
The following conflicting packages were found:
	esm 7.2.2	node_modules/.vite/deps/
	esm 7.2.1	node_modules/.vite/deps/
warn @ versionDetect.js:66
Show 1 more frame
Show less
versionDetect.js:66 @polkadot/wasm-crypto-wasm has multiple versions, ensure that there is only one installed.
Either remove and explicitly install matching versions or dedupe using your package manager.
The following conflicting packages were found:
	esm 7.2.2	node_modules/.vite/deps/
	esm 7.2.1	node_modules/.vite/deps/
warn @ versionDetect.js:66
Show 1 more frame
Show less
versionDetect.js:66 @polkadot/wasm-util has multiple versions, ensure that there is only one installed.
Either remove and explicitly install matching versions or dedupe using your package manager.
The following conflicting packages were found:
	esm 7.2.1	node_modules/.vite/deps/
	esm 7.2.1	node_modules/.vite/deps/

i tried adding the following extra resolutions in the project root package.json, but it still showed the warnings
  "resolutions": {
    "@polkadot/util": "^12.3.2",
    "@polkadot/util-crypto": "^12.3.2",
    "@polkadot/wasm-bridge": "^7.2.1",
    "@polkadot/wasm-crypto": "^7.2.1",
    "@polkadot/wasm-crypto-wasm": "^7.2.1",
    "@polkadot/wasm-util": "^7.2.1",
    "graphql": "^16.0.0"
  }

i also tried running this in the project root and in squid/, but still got the warnings (perhaps that's for Yarn 1 and 2, but not 3)
npx --yes yarn-deduplicate yarn.lock

Prerequisites
Add any setup info / environment / account specifics that need to be in place prior to the test
Steps To Reproduce
Add clear steps on how to reproduce the bug

Run yarn start
Go to http://localhost:3333/?network=kusama
View browser console logs to see warnings

Expected Behaviour
Not show duplicate dependency warnings
Console Error
As shown above
**Test Host Specifics: **

OS: Docker Desktop for Mac, macOS 13.4
Browser Brave
Version latest

Squid is running npm, because we have to as this is a subsquid requirement. Did you run yarn start in the root? In which case I imagine that this is unwanted.

ahh interesting. actually no, i didn't follow the instructions properly that are shown here, as i deviated slightly and called it from within the subdirectory /multix/packages/ui, using this PR https://github.com/ChainSafe/Multix/pull/321/
so i installed the MultixUI dependencies with yarn install from the project root here in the Dockerfile https://github.com/ChainSafe/Multix/pull/321/files#diff-608af3cc9f42984f49b1d1eb0c391d69e344734779b0eff2f3909ed9eb0991e3R53
and then i started the MultixUI frontend from the workspace subdirectory /multix/packages/ui with yarn start --host <IP_ADDRESS> (but calling into the Docker container) here https://github.com/ChainSafe/Multix/pull/321/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5R109, so the code used was docker exec -w /multix/packages/ui -it multix-ui-dev yarn start --host <IP_ADDRESS>
so to change it to call it from the project root folder i just changed that code to docker exec -w /multix -it multix-ui-dev yarn ui:start --host <IP_ADDRESS>, but when i tried running it with that instead it still showed the same errors in the browser console, so as you say it may be unwanted.

closing as I can't reproduce and it doesn't seem like a blocker.


eegay downloads not working

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.09.26. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[ ] I've verified and I assure that I'm running youtube-dl 2018.09.26

Before submitting an issue make sure you have:

[ ] At least skimmed through the README, most notably the FAQ and BUGS sections
[ ] Searched the bugtracker for similar issues including closed ones
[ ] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--verbose', u'https://eegay.com/video/2719-michael-fucks-shane']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2018.09.26
[debug] Python version 2.7.15rc1 (CPython) - Linux-4.15.0-36-generic-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: ffmpeg present, ffprobe present, phantomjs 2.1.1, rtmpdump 2.4
[debug] Proxy map: {}
[generic] 2719-michael-fucks-shane: Requesting header
WARNING: Falling back on generic information extractor.
[generic] 2719-michael-fucks-shane: Downloading webpage
[generic] 2719-michael-fucks-shane: Extracting information
[download] Downloading playlist: Michael fucks Shane at eegay
[generic] playlist Michael fucks Shane at eegay: Collected 1 video ids (downloading 1 of them)
[download] Downloading video 1 of 1
[Openload] Zwa7CAGSYyU: Downloading embed webpage
[Openload] Zwa7CAGSYyU: Downloading f webpage
ERROR: Zwa7CAGSYyU: File not found
Traceback (most recent call last):
File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 792, in extract_info
ie_result = ie.extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 507, in extract
ie_result = self._real_extract(url)
File "/usr/local/bin/youtube-dl/youtube_dl/extractor/openload.py", line 342, in _real_extract
raise ExtractorError('File not found', expected=True, video_id=video_id)
ExtractorError: Zwa7CAGSYyU: File not found
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2018.09.26
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
cannot download
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.

Read error message?


docs: Migrated the docs to Docusaurus(Issue #923)

I've migrated it to Docusaurus(v2). I've also added a custom favicon, and also made it readable. It can be found in the react-tags-docusaurus directory.
cd react-tags-docusaurus
npm i
npm start

@sikehish thank you for the PR ✨
let's do these first set of changes 👇

[ ]

[ ] Lets move this to docs folder

Sure! I will.

Hi @ad1992. I didnt understand the options part. Do you want a single options file instead of a directory? Also,I didnt understand the "Move this to under react tags" comment . What should be moved?

Hi @ad1992. I didnt understand the options part. Do you want a single options file instead of a directory? Also,I didnt understand the "Move this to under react tags" comment . What should be moved?

HI @sikehish, yes let's make options a single file instead of a directory. And by "React Tags"I mean the file "React Tags"so all the categories - from "Support" to "Thanks" can be clubbed inside the React Tags file.
Let me know if everything is clear and if you have any questions

Thanks for the clarification! Also, about migrating docusaurus directory to docs dir, do I eliminate the package.json file that is present in the docusaurus dir and add a command for it in the main package.json file or copy it with the package.json file? Any special instructions with tegards to that?

Thanks for the clarification! Also, about migrating docusaurus directory to docs dir, do I eliminate the package.json file that is present in the docusaurus dir and add a command for it in the main package.json file or copy it with the package.json file? Any special instructions with tegards to that?

separate package.json for docs directory is absolutely fine and preferrable as well

Thanks for the clarification! Also, about migrating docusaurus directory to docs dir, do I eliminate the package.json file that is present in the docusaurus dir and add a command for it in the main package.json file or copy it with the package.json file? Any special instructions with tegards to that?

separate package.json for docs directory is absolutely fine and preferrable as well

Hi @ad1992, Ive made all the changes that you had specified. You can try it out and let me know if any changes are to be made.

@sikehish there is a nested docs directory created inside root leveldocs, can we club and merge these into the root level docs directory ?

Hi @ad1992 I'm not sure if that can be done, cuz the nested docs dir is configured to serve the .md files. I can club and merge it into the roots dir, but then that might make it more unstructured, as we have src directory, static directory and a host of other files and images.

@sikehish there is a nested docs directory created inside root leveldocs, can we club and merge these into the root level docs directory ?

I can rename the directory to something more meaningful if need be, or if you think that clubbing and merging at the root level docs directory is required, then I'll get started with it.

@sikehish there is a nested docs directory created inside root leveldocs, can we club and merge these into the root level docs directory ?

I can rename the directory to something more meaningful if need be, or if you think that clubbing and merging at the root level docs directory is required, then I'll get started with it.

@sikehish Its best we merge both the docs into the single root level docs directly so let's do that.

@ad1992 I've spent quite some time trying to have all the docs(.md files) under the root directory, and ive tried toying with all the configs possible, but that isnt possible. I checked the docs, and as per the docs, we need to have the nested docs directory.

@ad1992 I've spent quite some time trying to have all the docs(.md files) under the root directory, and ive tried toying with all the configs possible, but that isnt possible. I checked the docs, and as per the docs, we need to have the

Hi @sikehish apologies for not getting back on this PR. There have been changes in docs recently, do you want to update this PR with latest changes and then we can merge it sooner this week?

Hi. Sure, I'd love to do that. Do I contine with the same approach? Or anything that you'd want me to keep in mind?

Hi. Sure, I'd love to do that. Do I contine with the same approach? Or anything that you'd want me to keep in mind?

Both the docs would ideally be merged into one, but as you mentioned that looks tricky now so in that case let's rename the docusaurus docs folder to dev-docs and move to root folder.

Hi @sikehish checking if you would be picking this up ?

Hi @ad1992 , I'd love to take this up. Can we decide on a deadline?

Hi @ad1992 , I'd love to take this up. Can we decide on a deadline?

The sooner the better :) Would be great if you could push the changes in the coming week.

Hi @ad1992 , I'd love to take this up. Can we decide on a deadline?

The sooner the better :) Would be great if you could push the changes in the coming week.

Sorry for the delay :( Im getting started with it right now

Hi @ad1992 . Ive updated docusaurus. Let me know if any other changes are to be made

@sikehish i think the changes from readme.md aren't pulled in this branch?
As per the rename let it be as it is, I will try to merge both into single docs folder

@sikehish i think the changes from readme.md aren't pulled in this branch? As per the rename let it be as it is, I will try to merge both into single docs folder

I have pulled the changes from readme.md

I don't see the latest changes from readme in the docs
eg 👇🏻

I don't see the latest changes from readme in the docs eg 👇🏻

Apologize for the inconvenience. Ive pushed the changes.

I don't see the latest changes from readme in the docs eg 👇🏻

Oops! Apologize for the oversight. Hope everything's in place now. Let me know if any other changes are to be made


Unable to use tags and categories outside _posts

Environment

Minimal Mistakes version: latest
Ruby gem or remote theme version: 2.0
Jekyll version: latest
Git repository URL: https://github.com/aem-design/aem.design
GitHub Pages hosted (if yes provide URL to site): https://aem.design/
Operating system: Ubuntu 20

Expected behavior

I expect that when tags and categories are working on _posts, I can add same configuration to other pages in the site and they would behave the same.
I have tested this with liquid and jekyll-archives. I might be missing something but its not obvious to me as it works fine for _posts.
Steps to reproduce the behavior

Un commenting following line in https://github.com/aem-design/aem.design/blame/master/_archives/001-davids-model.md does not result in respective categories and tags being generated.
#categories:
#  - archives
#tags:
#  - reference
#  - backup

Other
You bloody legends!

Duplicate. See https://github.com/mmistakes/minimal-mistakes/issues/405#issuecomment-235148252

cool thanks! :D


Load CSS files asynchronously

Needs review.
Thanks to the filamentgroup for https://github.com/filamentgroup/loadCSS and Paul Irish for bringing this up.
cc @paulirish @scottjehl @zachleat

Looks solid, @bramstein! Really glad to see this rolled into Typekit. :)

:+1:

:+1:

This is an awesome addition, thanks everyone!

@bramstein  I just saw the followup commit over here https://github.com/typekit/webfontloader/commit/2e58c0cbb5d5e4b9616970842c8ef44c6cb289d6
Under what circumstances with the opt-in async behavior trigger?

@paulirish I did some testing with these changes and while it improved performance it also broke the default behaviour many of our customers expect: Typekit blocks rendering (as bad as it is). Therefor I decided to turn this into an opt-in feature that can be enabled by passing a new configuration option to the Typekit loader.
Anyone wishing to try it out can set the async option to true when they call Typekit.load:
Typekit.load({ async: true });

The async parameter is going to be turned on by default for new kits.
The code for this is already in production, but I haven't had time to write a blog post about it. The blog post will introduce this new option and encourage all of our customers to switch to the asynchronous embed code (so the Typekit JS loads asynchronously) and use async (so the CSS isn't blocking either).

@paulirish As for the other modules, I would prefer those to load CSS asynchronously as well, but I'm hesitant to make changes without the approval of the module owners (e.g. Google, Fontdeck, Monotype).

So great to hear that async will be the default for new kits, @bramstein !


SpringDoc bean name conflict error

I tried to upgrade to spring boot 3.0.0
<dependency>
        <groupId>org.springdoc</groupId>
        <artifactId>springdoc-openapi-starter-webmvc-ui</artifactId>
        <version>2.0.2</version>
    </dependency>
    <dependency>
        <groupId>com.graphql-java-kickstart</groupId>
        <artifactId>graphql-spring-boot-starter</artifactId>
        <version>15.0.0</version>
    </dependency>

The problem is they both have a Bean with the name 'objectMapperProvider'.
Description:
The bean 'objectMapperProvider', defined in class path resource [org/springdoc/core/configuration/SpringDocConfiguration.class], could not be registered. A bean with that name has already been defined in class path resource [graphql/kickstart/autoconfigure/web/servlet/GraphQLWebAutoConfiguration.class] and overriding is disabled.
Action:
Consider renaming one of the beans or enabling overriding by setting spring.main.allow-bean-definition-overriding=true

This is not a bug of this project. This is simply a naming conflict. This library has marked the creation of the ObjectMapperProvider as @ConditionalOnMissingBean. Sounds like that springdoc library does not use such an annotation and therefore the conflict occurs. So you should check with the springdoc library if it's possible to add that @ConditionalOnMissingBean annotation there too, since they're the one actually causing this error. In the meantime you can execute the action they suggest by allowing bean definition overriding.


fixed aud typo

Microsoft identity endpoints are all lowercase
#96881 #ATCP

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

@nickludwig
Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Make tests catch invalid absolute URLs in methods.

When adding support for new endpoints, it's possible to make a mistake and use an absolute URL in the method implementation, rather than a relative one.
We need to use relative URLs so that the Client.BaseURL field works properly in NewRequest method:
https://github.com/google/go-github/blob/0f6d3ce15ec23c92c74d014303a167a9a374dd7e/github/github.go#L112-L115
https://github.com/google/go-github/blob/0f6d3ce15ec23c92c74d014303a167a9a374dd7e/github/github.go#L238-L247
E.g., this is invalid:
u := fmt.Sprintf("/admin/users/%v/authorizations", username)

This is correct:
u := fmt.Sprintf("admin/users/%v/authorizations", username)

Including that leading slash has no negative effect on GitHub API clients, but it typically breaks GitHub Enterprise clients. That makes this type of mistake easy to miss during code review.
Right now, our tests do not catch this, so we rely on human reviewers to catch it (e.g., this recently happened in #749).
I've thought about it and I think I found a way to make it so that our tests catch this. That will reduce the amount of work the human reviewers have to do, by automating more work. :) I'll send a PR soon. Edit: Sent #753.
P.S. Thanks to @MorrisLaw for giving me this idea.

/cc @willnorris FYI. You might find this change interesting, since you've had to deal with this category of issues in the past. :)


[Site]: three pages were 404: plugins page, community channels page and open an issue page

Current State
Three links in get started page

they are all 404

The three links are:

https://apisix.apache.org/docs/general/join/
https://apisix.apache.org/plugins/
https://apisix.apache.org/docs/general/submit-issue/

Desired State
Description of the change you are proposing:
of course not 404
Would you like to fix this issue?
Yes! Count me in

@zou8944 hi, Thanks for the report. When I click on these three links, I get a 404. And when I choose to open a new tab, it's accessible normally. We should fix the bug of not being accessible when clicked. cc @SkyeYoung


Issue 2610 and Audit fixes

Done audit up to TestNet block 446078, MainNet block 370925

Codecov Report

Merging #130 (1aa0948) into development (945689f) will decrease coverage by 0.13%.
The diff coverage is 27.65%.

@@               Coverage Diff               @@
##           development     #130      +/-   ##
===============================================
- Coverage        83.22%   83.08%   -0.14%
===============================================
  Files               87       87
  Lines             7962     7983      +21
===============================================
+ Hits              6626     6633       +7
- Misses            1336     1350      +14

Impacted Files
Coverage Δ

neo3/contracts/native/management.py
37.00% <0.00%> (-0.90%)
:arrow_down:

neo3/contracts/native/oracle.py
28.24% <0.00%> (-0.66%)
:arrow_down:

neo3/contracts/manifest.py
79.41% <33.33%> (-1.00%)
:arrow_down:

neo3/contracts/native/stdlib.py
63.86% <42.85%> (-1.63%)
:arrow_down:

neo3/contracts/native/fungible.py
62.47% <85.71%> (+0.12%)
:arrow_up:

neo3/contracts/jsonserializer.py
99.23% <100.00%> (+<0.01%)
:arrow_up:

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 945689f...1aa0948. Read the comment docs.


Fix the fix on addon integration.

The check in #543 is correct, but if a project has "in-repo-addons" then
it will include ember-addon in package.json without it being an
addon.
The following shows how in-repo-addons are discovered https://github.com/ember-cli/ember-cli/blob/282641ba662292afb40cb88cdf31557a3e4cc6b7/lib/models/addon-discovery.js#L160
Without this #524 still happens if you have in-repo-addons..
cc @cibernox

Good catch, I never thought about in-repo addons.

I just stumbled upon this. A new beta release with this would be nice :+1:

Thanks yall! Suuper helpful to have all of your eyes on this, as I haven't had need for Mirage in an addon yet.
I'll cut a new release soon.

Awesome thanks!


CLI to be removed from podman compose files

Now that CLI is deprecated, I think we should remove the reference to CLI from the compose files. This is preventing the network from starting when we set CONTAINER_CLI = podman
The file to be changed is given below.
https://github.com/hyperledger/fabric-samples/blob/main/test-network/compose/podman/podman-compose-test-net.yaml

Hello @ashishkel!
I removed the line in the PR I just linked to this issue.
Best,
Stanislav


adding t2 large

https://aws.amazon.com/about-aws/whats-new/2015/06/introducing-a-new-amazon-ec2-t2-instance-t2-large/

Thanks!


⚠ ClamAV has degraded performance

In 40e7740, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 67 ms

Resolved: ClamAV performance has improved in b879f48.


Implement handling of CBC padding for symmetric decryption.

AES and 3DES, when used with encrypted assertions with CBC padding, use a form of padding called ISO 10126 and is documented here: https://www.w3.org/TR/xmlenc-core1/#sec-Padding.
The current implementation calls this "noise" and uses a regular expression to attempt to extract the plaintext without the padding. Rather than use a regular expression, this implements correct handling of the padding.]]
This is more of a correctness thing than addressing any particular issue. Since the padding is random it could, in theory, be something like NameID> which would incorrectly be handled by the regex.
The unit tests already handle the padding aspect. Removing the regular expressions without the padding handling caused test failures. Rather than introduce any new tests, existing tests are used to validate the changes.

Closing this out for now, I think there are some opportunities to improve testing here.


Remove docker binary from weaveexec container

I looked at what it would take to do this.  The benefits would be:

avoid triggering virus scans (the docker binary includes some vulnerable code, although we don't execute that code)
smaller weaveexec image
avoid unnecessary version coupling caused by bundling docker 1.10.3
avoid icky feeling from downloading a tarfile and extracting a binary in the Makefile
weave net would be available on arm64 (where we cannot find a usable docker binary)

The subcommands used now are kill, stop, inspect, run, rm, exec, create, pull, ps.
I give arguments below why exec and pull would not be needed; the rest could be re-implemented in weaveutil.
Only a subset of docker run arguments are needed: -d, -v, -e, --name, --privileged, --net=host, --pid=host, --volumes-from; although we have suggested that users set WEAVE_DOCKER_ARGS to add arbitrary more.
Things that would need converting:

kill_container, called from wait_for_status (stop a restarting weave) - docker kill
stop, called from weave stop - docker stop
ask_version - printing the version of the weave container - inspect and run
is_plugin_v1_enabled - inspect, looking for --plugin; this is bugged anyway
check_running - inspect
check_not_running - inspect and rm
container_id - inspect
http_call_unix - exec - could move this feature to TCP socket on 6784
rewrite_etc_hosts - can move the docker inspect to weaveutil rewrite-etc
with_container_fqdn - inspect
launch - creating weavedb and weavewait containers - inspect, create
weave reset - docker stop, rm, ps
setup - docker pull - this could move to the non-remoted part of the script

Things that don't need converting:

try_create_bridge - inspect - this is pointless because / isn't mounted
util_op - this code only used when not running inside weaveexec

Is this still something that needs to be fix/changed?  I see WIP.  thanks for the information.

Yes, it's still open. I think by "WIP" you are referring to a linked issue which would not be required if this were done.
Some of the itemised points are out of date.

Updated list of things that would need converting, in the weave script:

kill_container, called from wait_for_status (stop a restarting weave) - docker kill
stop, called from weave stop - docker stop
ask_version - printing the version of the weave container - inspect and run
check_running - inspect
check_not_running - inspect and rm
container_id - inspect
rewrite_etc_hosts - can move the docker inspect to weaveutil rewrite-etc
with_container_fqdn - inspect
launch - creating weavedb and weavewait containers - inspect, create, run
stop - docker stop
weave reset - docker stop, rm, ps
weave setup - docker pull - this could move to the non-remoted part of the script

Things that don't need converting:

check_docker_version - only called outside of the container
try_create_bridge - inspect - this is pointless because / isn't mounted when this code is run
util_op - this code only used when not running inside weaveexec

I am new to Weave and would like to get my feet wet.  I would like to work on this but will need some pointer to get start.  I believe the "script" that needs to be worked on/convert is this https://github.com/weaveworks/weave/blob/master/weave

Hi, that's great!  Please ask as many questions as you need.  We also have a community Slack
The proposal here is to write mini-replacements for the required docker commands inside weaveutil, which is a Go program we use for various other uses.  Then that program can be called by the weave script and we can stop bundling the docker binary.
So there would be weaveutil stop, weaveutil kill, etc., and each of them would call the Docker API.

Thanks for the pointers. Let me look into this and get started.

Thanks for the pointers. Let me look into this and get started.

So take container_id as an example.  In the weave script there is a function container_id() which is doing docker inspect --format='{{.Id}}' $1 The idea is to create a function in weaveutil (a golang program) to perform the same.
In some case the function in weaveutil can use Docker API instead of executing a Docker CLI command.

Yep, I think you've got a handle on it

In some cases we should create higher-level operations in weaveutil rather than just straight replacements of docker commands. For example, we have
    ps)
        [ $# -eq 0 ] && CONTAINERS="weave:expose $(docker ps -q)" || CONTAINERS="$@"
        with_container_addresses echo_addresses $CONTAINERS

which ideally should be implemented via a single weaveutil command (with_container_addresses already calls weaveutil container-addrs).
Also, all of populate_router, which calls docker ps, should just live inside the router.
I suppose we could do straight docker command replacements first and then tackle above in a second pass.

Got it.

In the weave script there is util_op(). If command is in weaveutil execute, else start a container with entry  point /usr/bin/weaveutil.  Why do we need to specify the entry point when we know weaveutil does not have this command? .
I thought the idea is when not in weaveutil then start a container to execute the command via Docker CLI.
Thanks for the information

That conditional checks whether weaveutil itself exists, and if so executes it directly, otherwise launching a container to execute it. The latter happens when the weave script is executing outside the weaveexec container.

Got it.  Thanks.

Not getting too far on this one.  Still want to get this going.

I stumbled across this when trying to improve the weave script for net-plugin v2 status.
I initially made changes to the script to use direct docker commands, then realized a really old version of docker lives inside the weaveexec container.  I started searching around open issues and found that I was on the wrong path (need less docker ... commands, not more :-)
After I finish tackling the plugin support methods (if the weaveworks guys think thats a worth while effort), I may be able to help port some of these other methods over.

@dtshepherd it would be great if you could contribute here!  Maybe if you did just one of the docker command replacements it would help @vCloudernBeer see the pattern and you could collaborate on doing more?

After I finish tackling the plugin support methods (if the weaveworks guys think thats a worth while effort)

Is there an issue or PR specifically for that?
Generally when you have a problem, any troubleshooting tools are helpful.  We don't want to require the weave script for plugin-v2 users, but as an optional extra it seems like a fine idea.

BTW the docker binary is old because a new docker will refuse to talk to an older daemon, and we want Weave Net to work on as wide a range of targets as possible.  See also #2545 which was an attempt at solving this a different way.

Completely agree the weave script shouldn't be required for plugin-v2. It would just be helpful for status and cleanup.
There isn't a PR yet, but the effort is related to #3102 and #3103. I actually codes up changes to the script and then realized the docker client in weaveexec was really old (no plugin support).
I figured I could implement some plugin commands in weaveutil so I could submit a PR for those two issues.
What do you think? We can talk more on Slack if you want.

Yes, this will help me a lot and get me going. Thanks so much.

@bboreham I started down the path of implementing some of the docker commands in weaveutil tonight, but it seems to be just duplicating docker/cli/command functions (with way less features).  weaveutil commands should be used for higher level functionality (like a docker command sequences).
I really like the idea of just using the latest docker client (17.06.1-ce) and specifying DOCKER_API_VERSION environment variable (as noted in #2545).  It looks like your blocker (moby/moby#29040) was fixed in 17.03.  I tested docker-engine 1.11.2 with docker client 17.06.1-ce and it seems to work (see below).  We could dynamically set the DOCKER_API_VERSION based on the server API version.  Querying the server using the older 1.22 API version (docker 1.10.3) to determine the max supported API version.
If you don't want to download the docker tarfile, we could just use the github.com/docker/docker/cli/command functions directly and just create a helper wrapper in weaveutil.  We're already importing that vendor package for the network_plugin.go commands.
The code changes to get this working are straight forward, but I don't think I can take on the testing across all versions of docker.  Do you guys have a test harness to run against all versions?
Thoughts on implementing it this way?
$ docker-17.06 version
Client:
 Version:      17.06.1-ce
 API version:  1.24 (downgraded from 1.30)
 Go version:   go1.8.3
 Git commit:   874a737
 Built:        Thu Aug 17 22:51:12 2017
 OS/Arch:      linux/amd64
Error response from daemon: client is newer than server (client API version: 1.24, server API version: 1.23)

$ DOCKER_API_VERSION=1.22 docker-17.06 version
Client:
 Version:      17.06.1-ce
 API version:  1.22
 Go version:   go1.8.3
 Git commit:   874a737
 Built:        Thu Aug 17 22:51:12 2017
 OS/Arch:      linux/amd64

Server:
 Version:      1.11.2
 API version:  1.23 (minimum version )
 Go version:   go1.5.4
 Git commit:   b9f10c9
 Built:        Wed Jun  1 22:00:43 2016
 OS/Arch:      linux/amd64
 Experimental: false

$ DOCKER_API_VERSION=1.23 docker-17.06 version
Client:
 Version:      17.06.1-ce
 API version:  1.23
 Go version:   go1.8.3
 Git commit:   874a737
 Built:        Thu Aug 17 22:51:12 2017
 OS/Arch:      linux/amd64

Server:
 Version:      1.11.2
 API version:  1.23 (minimum version )
 Go version:   go1.5.4
 Git commit:   b9f10c9
 Built:        Wed Jun  1 22:00:43 2016
 OS/Arch:      linux/amd64
 Experimental: false
dtshepherd@NUC:~/git/weave$

It looks like your blocker (moby/moby#29040) was fixed in 17.03.

It's always questionable whether to exclude a Docker version, but our stats say usage 1.13 with Weave Net has dwindled to about 0.5%, so maybe that would be ok.
I do think it would be plausible to re-use the Docker CLI implementations of kill, stop, etc.
That might involve converting weaveutil to use the Cobra CLI library, which is already proposed in #2490.  And docker/docker/cli/command/cli.go may have some behaviour we don't want.
Not sure if I would want a pass-through implementation of docker inspect since we only use it in very specific scenarios and in places (e.g. check_running, check_not_running) the weave script code gets a bit twisted to work round its limitations.

Do you guys have a test harness to run against all versions?

Our test infrastructure in theory allows us to pick a Docker version; we don't have any automation to try a series of versions.  And I'm not sure how well the older installs work right now - Docker made changes a few months back which forced us to upgrade to 17.x.

@vCloudernBeer I started work in PR #3110, I only implemented the simple command so far...

My understand of this issue from a very high level is to migrate to using go function as a Docker Client and to issue API calls to the Docker Engine to perform the above listed actions instead of spinning up a container and pass in the Docker CLI command.

One example of using Docker client API can be found in  .../weave/prog/weaveutil/addr.go where it import in "github.com/fsouze/go-dockerclient"? Sorry I am still a bit slow but I am determine to get myself started.

@vCloudernBeer Check out PR #3110.  I've ported a large portion of the docker ... dependencies to weaveutil.  I'm still working on the docker create replacement for the weavevolumes, generalization/reusable code for docker run, docker ps, and docker pull.  Some of those should be direct ports (e.g. docker pull), but others should be implemented as higher level functionality (see https://github.com/dtshepherd/weave/blob/6bb8afd46001f2704a874e1aae7059c92c97187b/prog/weaveutil/version.go#L10).
Maybe you can try to implement something simple like the docker pull command.  It probably belongs in a new file image.go.  Feel free to start from my current branch (https://github.com/dtshepherd/weave/tree/issues/2957-rm-docker-binary-from-weaveexec).   @bboreham is also pushing that into https://github.com/weaveworks/weave/tree/issues/2957-rm-docker-binary-from-weaveexec on occasion.

Cool.  thanks.

instead of spinning up a container and pass in the Docker CLI command.
@vCloudernBeer It's worth mentioning that the weaveexec container will still be spun up to run the weaveutil CLI commands.  This issue is trying to eliminate the use of docker ... CLI commands in the weaveexec container.

I found one reference of "docker pull ..." in the weave script. So the idea is to change from
docker pull $img to util_op docker-pull $img where docker-pull is a new function in weaveutil?

@vCloudernBeer yes, that's one way to do it.
I did note a different idea for docker pull, since it is only used in weave setup: it could move to the non-containerised part of the script.
Each time you execute the weave script, it does some initial stuff then calls docker run on the same script inside the weaveexec image.  This way we know we have certain tools available - ip, curl, weaveutil, etc.  But some commands like weave help are executed without doing a docker run.
So, my idea was that the docker pull part of weave setup could move to roughly where weave help is done.  It's just an idea.

Not a bad idea, I thought about that and it probably a cleaner solution.  I ended up implementing it inside weaveutil in PR #3110 (https://github.com/weaveworks/weave/commit/01949309e9140b9063ee00ad77e8c5692df56ecd).  It's a decent amount of code to trigger a docker pull through the API.

So this is already done.  Seems like I should work on another issue.


make docker-run command fails with "plugin 'superset' is not known to meltano" error

Steps to replicate:

clone repository
make docker-build
make docker-run

Error message:
16:45:43  Done. PASS=74 WARN=0 ERROR=0 SKIP=0 TOTAL=74

meltano invoke superset fab create-admin --username admin --firstname lebron --lastname james --email<EMAIL_ADDRESS>--password password
Need help fixing this problem? Visit http://melta.no/ for troubleshooting steps, or to
join our friendly Slack community.

Plugin 'superset' is not known to Meltano
make: *** [Makefile:14: superset-visuals] Error 1
make: *** [Makefile:23: docker-run] Error 2

Confirming that this is fixed on the superset-test branch.


Ability to use json:$ type filter when JSON is located inside of HTML document

Describe the bug
JSONPath and JQ filters report: "No parsable JSON found in this document"
Version
0.41
To Reproduce
Steps to reproduce the behavior:

set up a watch with the following url: https://raw.githubusercontent.com/platformio/platform-espressif32/master/boards/lolin_s3.json
Select "Playwright Chromium/Javascript" in the "Request" tab
Enter "json:$" to the "CSS/JSONPath/JQ/XPath Filters" in the "Filters & Triggers" tab
Let it run
See error "No parsable JSON found in this document"

Expected behavior
JSON filtering should work with playwright (in my case I'm working with sites that need session cookies or logging in to view a json)
Screenshots

Desktop (please complete the following information):

OS: [e.g. Windows]
Browser [e.g. Firefox]
Version [e.g. 111.01]

Additional context
Changedetection.io + Playwright running in docker (docker-compose) on x64 Debian

wow, I really thought that this couldnt be the case, but I can reproduce it - yeah switching to the 'plaintext' fetcher is a work around for now, definitely a bug somewhere

Thanks for confirming. Yeah, it still works for plaintext.
For my case, I need browsersteps to log in, before navigating to the json.

So we call https://playwright.dev/docs/api/class-page#page-content this step to get the source
Gets the full HTML contents of the page, including the doctype.
And I see
<meta name="color-scheme" content="light dark">
{
  "build": {
    "arduino": {
      "ldscript": "esp32s3_out.ld",
      "partitions": "default_16MB.csv",
      "memory_type": "qio_opi"
    },
....

So hmmm not sure what todo about that, looks like a playwright thing, sorry

I got that HTML content, too. Tried to work around it, but failed.
Prepended the original url with "source:" and then applied xpath filter as follows: "/html/body/pre/text()"
The problem is, that the filter ordering is not the same as I defined it in the filter list:
e.g.: when the "CSS/JSONPath/JQ/XPath Filters" field contents are as follows:
/html/body/pre/text()
json:$

The "json:$" is applied first and fails.
Possible solutions that come to mind:

retain the order of the filters
introduce another prefix for the URL similar to the "source:" e.g. "json:"
introduce another browser step that would parse the json by applying the xpath above

I also tried to format the JS in the  element (using the execute JS browser step), to format the JSON, got another surprising result:

raw text remains unformatted
screenshot is now formatted

oops - yeah not sure, you could always add your own hack locally
thinking about the bigger problem here - how would you extract JSON from a HTML page hmm

The JSON filter runs before the html->text conversion.. which could be after.. but.. hmm

Possible solutions that come to mind:

respect the order of the filters

if that was true,  you could just reverse the order :)
The problem is that if it detects json: style filter in any filter rule, it will just use that
the other big problem is that it doesnt recursively process the content, it just adds whatever it finds in each filter to the output

maybe you should add something like accept-content: json type header to the request

Claim

Hello! I am a student at the University of Michigan, currently enrolled in a course that involves contributing to an open-source project by fixing a bug. I find this project and the specific issue you are facing quite intriguing. May I have the opportunity to claim this issue and investigate further? Your guidance and support would be greatly appreciated. Thank you for considering my request.

So i dont think this is a 'bug' its more like a 'feature'.. at the moment
@elijahqi You can just fork the project and do what you like with opensource without asking permission, that's how opensource works

Thanks I will try!

Hey @dgtlmoon (and thanks for building the tool btw!), just stumbled upon the same issue today, it took me some time to figure out what the root cause was 😅
It seems like you're already trying to look for any json objects in the


feat(monitor): remove precision of float when using max or min function

What this PR does / why we need it:

Does this PR need to be backport to the previous release branch?:

/area monitor


[Feature]: Laravel 11 Support status?

Question or Feature?
How is the status for laravel 11 support looking like?

Hi @Jan-Eckerlein
Should be resolved with this PR.
https://github.com/codebar-ag/laravel-zammad/pull/39
Best regards
Sebastian


Capture Net::ReadTimeout when using Net::HTTP

When using Net::HTTP to make a call to a service, we can back a
different exception for network timeout like:
/usr/lib/ruby/2.3.0/net/protocol.rb:158:in rbuf_fill': Net::ReadTimeout (Net::ReadTimeout)         from /usr/lib/ruby/2.3.0/net/protocol.rb:136:in readuntil'
from /usr/lib/ruby/2.3.0/net/protocol.rb:146:in readline'         from /usr/lib/ruby/2.3.0/net/http/response.rb:40:in read_status_line'
from /usr/lib/ruby/2.3.0/net/http/response.rb:29:in read_new'         from /usr/lib/ruby/2.3.0/net/http.rb:1437:in block in transport_request'
from /usr/lib/ruby/2.3.0/net/http.rb:1434:in catch'         from /usr/lib/ruby/2.3.0/net/http.rb:1434:in transport_request'
from /usr/lib/ruby/2.3.0/net/http.rb:1407:in request'         from /root/appscale/AppController/lib/hermes_client.rb:23:in block in make_call'
from /usr/lib/ruby/2.3.0/net/http.rb:853:in start'         from /usr/lib/ruby/2.3.0/net/http.rb:584:in start'
from /root/appscale/AppController/lib/hermes_client.rb:21:in make_call'         from /root/appscale/AppController/lib/hermes_client.rb:61:in get_proxies_stats'
from /root/appscale/AppController/lib/hermes_client.rb:79:in get_proxy_stats'         from /root/appscale/AppController/lib/hermes_client.rb:96:in get_proxy_load_stats'
from /root/appscale/AppController/djinn.rb:5935:in block in get_application_load_stats'         from /root/appscale/AppController/djinn.rb:5932:in each'
from /root/appscale/AppController/djinn.rb:5932:in get_application_load_stats'         from /root/appscale/AppController/djinn.rb:5190:in get_scaling_info_for_version'
from /root/appscale/AppController/djinn.rb:4928:in block in scale_appservers'         from /root/appscale/AppController/djinn.rb:4922:in each'
from /root/appscale/AppController/djinn.rb:4922:in scale_appservers'         from /root/appscale/AppController/djinn.rb:4951:in scale_deployment'
from /root/appscale/AppController/djinn.rb:1845:in block in job_start'         from /usr/lib/ruby/2.3.0/monitor.rb:214:in mon_synchronize'
from /root/appscale/AppController/djinn.rb:1833:in job_start'         from /root/appscale/AppController/djinnServer.rb:162:in '
this PR catched the exception.

OCD build https://ocd.appscale.com:8080/job/Daily Build/4613/ Trusty build to come next

OCD build https://ocd.appscale.com:8080/job/Daily Build/4633/

going to merge without a demo since this issue was seen and resolved in customer environments.


loteriasyapuestas.es is a legitimate lottery business run by the country's Treasury since 1811 and shouldn't be blocked

I've said pretty much everything in the title, lol.
Affected hosts are:
loteriasyapuestas.es
juegos.loteriasyapuestas.es
www.loteriasyapuestas.es
www.juegos.loteriasyapuestas.es
Which they are present in the gambling list. Now that's the issue... Yeah, its lottery... but it's run by the government and the site is very safe, you need ID verification and all that good stuff.
I think it should be removed, IMO, but I would understand if you want to keep blocking it.
https://en.wikipedia.org/wiki/Loterías_y_Apuestas_del_Estado

Hi Carlos @Deses.
These loteriasyapuestas domains do not appear in our vanilla hosts files.
The loteriasyapuestas domains only appear in the "gambling" variants of our base hosts files.
./alternates/gambling/hosts
./alternates/gambling-social/hosts
./alternates/gambling-porn/hosts
./alternates/fakenews-gambling-porn-social/hosts
./alternates/gambling-porn-social/hosts
./alternates/fakenews-gambling-social/hosts
./alternates/fakenews-gambling/hosts
./alternates/fakenews-gambling-porn/hosts

Which is perfect.  This is all by design.
Closing.


Fixes typo in docstring

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Mohit Gupta seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


feat: Set default EKS to 1.26; install vpc-cni add-on

This PR changes the default version of EKS to 1.26. It also installs the AWS VPC-CNI add-on, and creates an IAM role for that add-on. If you find appying fails because some of the add-ons already exist, remove the add-ons in question using the AWS console or the AWS CLI and re-apply.

This PR is included in version 4.10.0 :tada:


ADDED CLASSIC GAME SNAKE

This is a basic implementation of the classic "Snake" game using the Pygame library in Python. It provides a simple window, a controllable snake, food to eat, and basic collision detection.
PR Description 📜
Please include summary related to the issue you have fixed and describe your PR in brief over here by specifying the issue number on which you were working below
Fixes #2307

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @Shikhar9425 ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Done

Please add readme.md file in newly created folder...where to add that @Shikhar9425

Thank you  @Shikhar9425 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


No error messages?

I see that here are no feedback messages :/ I suppose it's because ionic has none.
How to fiz it? How to add those messages? Can You give simpliest example for required text input?

Will work on this and have it in the next version :+1:

Thanks - when will You menage to do it?

Perhaps this example would be helpful.

Hello,
I made something simple that work here : https://github.com/Charl---/angular-formly-templates-ionic
Thx for the link, it was very helpfull.
The configuration object for a field look like this
{
  ...
  templateOptions: {
  ...
    validate: true,    //display or not the error message section
    validateCustom: 'your url'    //the url of a template with ngMessage
    required: true,
    maxlength: 14,
    minlength: 5,
    ... ,
    validation : {    //default error message, they are replaced by those on the template if defined
      messages: {
        required: (viewValue, modelValue, scope)=> {
          return scope.to.label + ' is required';
        },
        minlength: (viewValue, modelValue, scope)=> {
          return scope.to.minlength + ' character minimum';
        },
        ...
      }
    }
  }
}

the template with ng-message directive must look like this :

The div containing the messages is decorated with is-dirty, id-touched, and is-invalid class depending on the field state.
here is a simple codepen demo with some extra style
http://codepen.io/Charl---/pen/yYZpPa


Add CytoTable processing of JUMP plate data

This PR adds a Python script which can be used to process JUMP plate data using CytoTable. Along the path towards this PR I also made adjustments to existing scripts and files. Notably, I made a change which assumes that all steps can be run from the root of the repo using just, which I feel will simplify how scripts are written. I've added a subshell example within the run.sh script for 0.download_data for steps which must retain their relative pathing.
Please note: I've added a pre-commit-config.yaml file (and related pyproject.toml file for ruff configuration) to the project which included many linting tools that I found helpful during development. As a result of this the README.md has been updated in sections that don't apply to the intended goal of this PR.
I haven't processed the modifications found within this step, this PR is intended to provide a preview of intent which I'll run afterwards.
Thanks in advance for any thoughts or feedback on this work!
References #30

Thank you @MattsonCam for your review and comments! After making some changes and responding I'll now merge this in.