Unable to Run GIT RM under Conversion Script

I ran the conversion as follows:
convert_web_page.py -rgvl /tmp/a.log create-pdb-using-dbca-simple.html

Fails with:
'/home/douglas/git-repositories/dfhawthorne.github.io/docs/home/12c-ocm/general-database-and-network-administration-and-backup-strategy/create-and-manage-pluggable-databases/create-pdb-using-dbca-simple/DBCA_Step_01.png' already exists
'/home/douglas/git-repositories/dfhawthorne.github.io/docs/home/12c-ocm/general-database-and-network-administration-and-backup-strategy/create-and-manage-pluggable-databases/create-pdb-using-dbca-simple/DBCA_Create_PDB_01.png' already exists
'/home/douglas/git-repositories/dfhawthorne.github.io/docs/home/12c-ocm/general-database-and-network-administration-and-backup-strategy/create-and-manage-pluggable-databases/create-pdb-using-dbca-simple/DBCA_Create_PDB_02.png' already exists
'/home/douglas/git-repositories/dfhawthorne.github.io/docs/home/12c-ocm/general-database-and-network-administration-and-backup-strategy/create-and-manage-pluggable-databases/create-pdb-using-dbca-simple/DBCA_Create_new_PDB_03.png' already exists
'/home/douglas/git-repositories/dfhawthorne.github.io/docs/home/12c-ocm/general-database-and-network-administration-and-backup-strategy/create-and-manage-pluggable-databases/create-pdb-using-dbca-simple/DBCA_Create_new_PDB_04.png' already exists
'/home/douglas/git-repositories/dfhawthorne.github.io/docs/home/12c-ocm/general-database-and-network-administration-and-backup-strategy/create-and-manage-pluggable-databases/create-pdb-using-dbca-simple/DBCA_Create_new_PDB_05.png' already exists
'/home/douglas/git-repositories/dfhawthorne.github.io/docs/home/12c-ocm/general-database-and-network-administration-and-backup-strategy/create-and-manage-pluggable-databases/create-pdb-using-dbca-simple/DBCA_Create_new_PDB_06.png' already exists
'/home/douglas/git-repositories/dfhawthorne.github.io/docs/home/12c-ocm/general-database-and-network-administration-and-backup-strategy/create-and-manage-pluggable-databases/create-pdb-using-dbca-simple/DBCA_Create_new_PDB_07.png' already exists
GIT RM failed with RC=128
b"fatal: pathspec ''/home/douglas/git-repositories/dfhawthorne.github.io/docs/home/12c-ocm/general-database-and-network-administration-and-backup-strategy/create-and-manage-pluggable-databases/create-pdb-using-dbca-simple/DBCA_Step_01.png?attredirects=0'' did not match any files\n"n

I used the following command to validate the file's existence:
stat '/home/douglas/git-repositories/dfhawthorne.github.io/docs/home/12c-ocm/general-database-and-network-administration-and-backup-strategy/create-and-manage-pluggable-databases/create-pdb-using-dbca-simple/DBCA_Step_01.png?attredirects=0'

The output was:
  File: /home/douglas/git-repositories/dfhawthorne.github.io/docs/home/12c-ocm/general-database-and-network-administration-and-backup-strategy/create-and-manage-pluggable-databases/create-pdb-using-dbca-simple/DBCA_Step_01.png?attredirects=0
  Size: 27626     	Blocks: 56         IO Block: 4096   regular file
Device: 803h/2051d	Inode: 49817694    Links: 1
Access: (0664/-rw-rw-r--)  Uid: ( 1000/ douglas)   Gid: ( 1000/ douglas)
Access: 2024-04-07 01:50:19.466153790 +1100
Modify: 2024-04-07 01:50:19.202153059 +1100
Change: 2024-04-07 01:50:19.202153059 +1100
 Birth: 2024-04-07 01:50:19.202153059 +1100

I used the following command to check GIT for the existence of the file:
git ls-files '/home/douglas/git-repositories/dfhawthorne.github.io/docs/home/12c-ocm/general-database-and-network-administration-and-backup-strategy/create-and-manage-pluggable-databases/create-pdb-using-dbca-simple/DBCA_Step_01.png?attredirects=0'

Returns:
create-pdb-using-dbca-simple/DBCA_Step_01.png?attredirects=0

In order to continue with the Wiki conversion, I decided to put in a work-around of echoing the GIT commands to STDOUT so that I can run them later.

This functionality is no longer needed as the Wiki conversion has completed.


Enabling versioning via Volto Control Panel not possible

Even if you enable the versioning behavior via Control Panel, the versioning policy remains set to No versioning

That screenshot is of Classic UI. When you try to select a versioning policy, do any options appear in the select menu? This might be an issue outside of Volto.

@stevepiercy  The idea is that in Volto there is no way to set "version policy" after you enable versioning behavior


Slider does not work without submission

When moving the slider, the plotting does not work if a file was not first submitted.

Resolved in #405


ReshapeInitTensorFuser in transformers removed a reshape layer in onnx model

when i transform the onnx model to coreml. I got an error like this:
179/182: Converting Node Type Sub Traceback (most recent call last):   File "/home/dianxin/inpainting/pytorch-inpainting-with-partial-conv-master/onnx_to_coreml.py", line 11, in <module>     coreml_model = convert(model_proto, image_input_names=['0'], image_output_names=['186'])   File "/usr/local/lib/python2.7/dist-packages/onnx_coreml/converter.py", line 464, in convert     _convert_node(builder, node, graph, err)   File "/usr/local/lib/python2.7/dist-packages/onnx_coreml/_operators.py", line 1826, in _convert_node     return converter_fn(builder, node, graph, err)   File "/usr/local/lib/python2.7/dist-packages/onnx_coreml/_operators.py", line 247, in _convert_sub     _convert_broadcast_op(builder, node, graph, err, "ADD")   File "/usr/local/lib/python2.7/dist-packages/onnx_coreml/_operators.py", line 69, in _convert_broadcast_op     ranks = [len(graph.onnx_coreml_shape_mapping[input_]) for input_ in node.inputs] KeyError: u'326'
when I check  graph.onnx_coreml_shape_mapping but not find  u'326'. Howerver, the onnx model graph has the reshape layer originally, like this:
 %319 : Tensor = onnx::Constant[value={171}]()   %320 : Tensor = onnx::Mul(%318, %319)   %321 : Float(1, 3, 256, 256) = onnx::Clip[max=1, min=0](%314), scope: PConvUNet/PCBActiv[dec_1]/PartialConv2d[conv]   %322 : Float(1, 3, 256, 256) = onnx::Mul(%320, %321), scope: PConvUNet/PCBActiv[dec_1]/PartialConv2d[conv]   %323 : Float(1, 19, 256, 256) = onnx::Mul(%311, %312), scope: PConvUNet/PCBActiv[dec_1]/PartialConv2d[conv]   %324 : Float(1, 3, 256, 256) = onnx::Conv[dilations=[1, 1], group=1, kernel_shape=[3, 3], pads=[1, 1, 1, 1], strides=[1, 1]](%323, %75, %76), scope: PConvUNet/PCBActiv[dec_1]/PartialConv2d[conv]   %325 : Tensor = onnx::Constant[value= 1  3  1  1 [ CPULongType{4} ]](), scope: PConvUNet/PCBActiv[dec_1]/PartialConv2d[conv]   %326 : Float(1, 3, 1, 1) = onnx::Reshape(%76, %325), scope: PConvUNet/PCBActiv[dec_1]/PartialConv2d[conv]   %327 : Float(1, 3, 256, 256) = onnx::Sub(%324, %326), scope: PConvUNet/PCBActiv[dec_1]/PartialConv2d[conv]   %328 : Float(1, 3, 256, 256) = onnx::Mul(%327, %322), scope: PConvUNet/PCBActiv[dec_1]/PartialConv2d[conv]   %329 : Float(1, 3, 256, 256) = onnx::Add(%328, %326), scope: PConvUNet/PCBActiv[dec_1]/PartialConv2d[conv]   %330 : Float(1, 3, 256, 256) = onnx::Mul(%329, %321), scope: PConvUNet/PCBActiv[dec_1]/PartialConv2d[conv]   return (%330, %321); }
%325 and %326 have been removed by ReshapeInitTensorFuser transformers.How should I do ?Thanks very much!

@dianxin556 please give a try to latest beta https://github.com/onnx/onnx-coreml#new-beta-onnx-coreml-converter-with-core-ml-3
and feel free to re-open if you still face this issue


Deletes code if previous line has trailing spaces/tabs

If you have something like this (note the trailing tabs on the first line):

veryLongFunction();

and you format it using the extension it removes the trailing tabs and cuts off a bit of the next line. The result is something like this:
ryLongFunction();

If you use xo from the cli it doesn't cut off anything, and if you use<EMAIL_ADDRESS>it doesn't either.
I tried v3.3.1 and v3.3.2 and both behaved like this. v3.3.0 crashes for me because I use Windows but I'd guess the bug snuck in there.
I have created a repo with a minimal reproducible example: https://github.com/melusc/vscode-xo-indent-bug.

@melusc thank you for the detailed bug report. Unfortunately I couldn't repro this behavior, even in the linked repro and tested after toggling all the final new line options and trying to make changes and apply fixes very quickly.
question 1 - what command do you use to format? Do you use "xo: fix all fixable problems" or do you use "format document" with xo enabled as the default formatter?
I added a debounce to the linting in 3.3+ which helps performance, but may be causing some random regressions. I am going to be releasing another update today which makes that configurable. Perhaps turning that off will fix this for you (and this may be a broader windows problem).
In the meantime, let me know if you can narrow it down any further!

question 1 - what command do you use to format? Do you use "xo: fix all fixable problems" or do you use "format document" with xo enabled as the default formatter?

If I use "xo: fix all fixable problems" or "format document" it works perfectly, the issue occurs when I save, because I have format on save enabled and xo is the default formatter. That means that it must be running "trim trailing whitespace" and xo at the same time which must be causing issues.
I also noticed that I copied the wrong setting-name in my previous comment, the conflict only occurs with "files.trimTrailingWhitespace" enabled.

Thank you - I can reproduce this now. And confirmed it is indeed the debounce that causes this - although I am not quite sure why. It may be a VSCode bug for this race condition to be possible. I am going to be releasing a new version later today with the debounce off by default and configurable for large file performance and this will solve this. I will let you know when I do.


flink task support(flink 任务支持)

I have a question, does this feature require a flink-related jar?

Can you support flink stream SQL ?

I have a question, does this feature require a flink-related jar?
Yes, Flink jar need to be uploaded just like spark tasks

Can you support flink stream SQL ?

Sorry for not supporting yet.


FAT-135

Purpose
Karate implementation of existing Postman API tests for mod-tags
Acceptance criteria:
API Tests are created at https://github.com/folio-org/folio-integration-tests
Test run results are visible in TestRail: https://foliotest.testrail.io/index.php?/suites/overview/19
Automated regular runs are executed at https://jenkins-aws.indexdata.com/job/FOLIO_Reference_Builds/job/folio-integration/
Help team ramp on Karate
Help team estimate remaining Karate user stories in the backlog
Learning
https://issues.folio.org/browse/FAT-135

@psmagin Done


Struct fields should be unsigned integers

I believe this is just search-replace i8 -> u8 and i16 -> u16

Also, looking at the Art-Net spec I 100% see why you did this, but they're really unsigned


Contribution: Tweaks for titlebar, sidebar, and ribbon workspace tabs

Type of Contribution
Other
Description
These are some CSS tweaks I have applied to the theme.
Fix Window titlebar height
This is the most important tweak. This changes the title bar to the minimum titlebar height of a normal (i.e. non Electron) app, and uses flexbox to improve the navigation buttons.

body.show-titlebar {
    padding-top: 28px !important;
}
body.show-titlebar .titlebar {
    height: 28px !important;
    padding-top: 0 !important;
}
body.show-titlebar .titlebar-text {
    display: block;
    font-family: system-ui;
    font-weight: 600;
    letter-spacing: 0;
    font-size: 13px;
    padding: 1px var(--cover-width) 0;
    box-sizing: border-box;
    margin: 0;
    text-overflow: hidden;
}
body.show-titlebar .titlebar-text::after {
    display: none;
}
body.show-titlebar .titlebar-button-container {
    display: inline-flex;
    align-items: center;
    justify-content: center;
    height: 28px;
    top: 0;
}
body.show-titlebar .titlebar-button {
    display: inline-flex;
    align-items: center;
    justify-content: center;
    border-radius: 2px;
}
body.show-titlebar .titlebar-button-container.mod-left svg {
    height: 20px;
    width: 20px;
}

Adjust lower icons position in sidebar (subjective change)
This moves the icons in the lower part of the sidebar further down (personal preference), and uses flexbox for better positioning.

body.show-sidebar-tabs .side-dock-ribbon-action {
    display: flex;
    justify-content: center;
    margin-left: 1px;
}
body.show-sidebar-tabs .side-dock-settings {
    margin-bottom: 15px;
}

Tweak style of workspace tabs in ribbons (subjective change)
This is a very subjective tweak. It makes the tab bar in the ribbons the same height as the document title, and makes the buttons more minimal.

.workspace-tabs .workspace-tab-header-container {
    height: 30.5px;
    box-sizing: border-box;
    padding: 0;
}
body.show-header-bar .workspace-tabs .workspace-tab-header-container:hover,
body:not(.is-mobile) .workspace-tabs .workspace-tab-header-container:hover {
    border-color: var(--hover-accent-2)!important;
}
.workspace-tabs .workspace-tab-header-container .workspace-tab-header {
    height: 100%;
    margin-right: 1px;
    border: none;
    border-top: 1px solid var(--background-secondary-alt);
    box-shadow: none;
}
.workspace-tabs .workspace-tab-header-container .workspace-tab-header-inner {
    display: flex;
    align-items: center;
    justify-content: center;
    padding: 0;
    width: 40px;
    border-radius: 0;
}
.workspace-tabs .workspace-tab-header-container .workspace-tab-header-inner-icon {
    display: flex;
    align-items: center;
    justify-content: center;
}

Screenshot

Hi! As you may have heard, the upcoming Obsidian update 0.16 has a massive amount of changes for theme development, necessitating a complete rewrite of Shimmering Focus. This means:

A lot of bugs and feature requests will be obsolete. with Obsidian 0.16 and the rewritten Shimmering Focus.
As the theme has over 15,000 lines of CSS, the rewriting will take a considerable amount of time. Recreating the core features alone will likely take me a while, so I simply will not have much time to implement many feature requests, since the recreation of the past core features has priority.

Basically, this is the reason why I am – for now – closing this issue. If your bug still exists / your feature request is still relevant in the upcoming version 3.0 of Shimmering Focus, please comment here and I will re-open it. 🙂


split into mod and cli tool

PR for #9
I just did the logging / printing change for now, please have a look at it and tell me if you are okay with it or if you maybe have a better idea...

LGTM but if I understand it right,
let result = sub_fn();
    if let Some(error) = result.err() {
        error!("adbackup finished with error: {}", error.to_string());
}

would print in the error-case nothing to the console? We have to use `println!? at this point, right? adbackup-cli.rs#L34

yep, you are right on that one. I only changed to println for code I actually touched :/ will fix

with these new changes the cli tools writes only to stdout and the module writes only to the log-file, please have a look @SailReal

Looks really amazing, thx for the improvement!


Audit the endpoints and introduce transient caching wherever possible

#59, #60 and #61 address this in addition to #49


Deploy issue: db shutdown automatically

I tried deploy.sh and it successfully deployed, but the frontend is receiving 502 error.
I used docker service logs db to find out db is getting shutdown after started:

2020-03-22 02:14:08.775 UTC [1] LOG:  database system is ready to accept connections
2020-03-22 02:14:12.811 UTC [1] LOG:  received smart shutdown request
2020-03-22 02:14:12.821 UTC [1] LOG:  background worker "logical replication launcher" (PID 33) exited with exit code 1
2020-03-22 02:14:12.822 UTC [28] LOG:  shutting down
2020-03-22 02:14:12.957 UTC [1] LOG:  database system is shut down

The other containers also get shutdown maybe due to db issue.
Is anywhere I can dig further to find the root cause, e.g. port is occupied, etc?
Thanks!

Dev environment is working for me, but not production after deployed.

Hi @binfeng , I assume you solved your issue since you closed it, right?


Add order item model

Fix #78

LGTM, maybe you can update the user guide as well to reflect the changes that have been made.

Codecov Report

Merging #79 (b124d26) into master (d9e1d63) will increase coverage by 0.46%.
The diff coverage is 80.00%.

@@             Coverage Diff              @@
##             master      #79      +/-   ##
============================================
+ Coverage     71.87%   72.33%   +0.46%
- Complexity      475      517      +42
============================================
  Files            80       86       +6
  Lines          1497     1587      +90
  Branches        159      174      +15
============================================
+ Hits           1076     1148      +72
- Misses          375      387      +12
- Partials         46       52       +6

Impacted Files
Coverage Δ
Complexity Δ

...in/java/seedu/cakecollate/model/order/Address.java
80.00% <ø> (ø)
6.00 <0.00> (ø)

...va/seedu/cakecollate/model/order/DeliveryDate.java
86.36% <ø> (ø)
10.00 <0.00> (ø)

...main/java/seedu/cakecollate/model/order/Email.java
80.00% <ø> (ø)
6.00 <0.00> (ø)

.../main/java/seedu/cakecollate/model/order/Name.java
80.00% <ø> (ø)
6.00 <0.00> (ø)

...n/java/seedu/cakecollate/model/orderitem/Type.java
70.00% <70.00%> (ø)
5.00 <5.00> (?)

...a/seedu/cakecollate/model/orderitem/OrderItem.java
72.00% <72.00%> (ø)
12.00 <12.00> (?)

...n/java/seedu/cakecollate/model/orderitem/Cost.java
80.00% <80.00%> (ø)
6.00 <6.00> (?)

...kecollate/model/orderitem/UniqueOrderItemList.java
85.36% <85.36%> (ø)
17.00 <17.00> (?)

...eritem/exceptions/DuplicateOrderItemException.java
100.00% <100.00%> (ø)
1.00 <1.00> (?)

...deritem/exceptions/OrderItemNotFoundException.java
100.00% <100.00%> (ø)
1.00 <1.00> (?)

... and 6 more

Continue to review full report at Codecov.

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


怎么能够删除一级域名的cookie

我现在在二级域名下面 second.website.com怎么能删除.website.com的cookie？
没有找到方法。。。是否考虑添加一个

Sorry I dont speak Chinese, but if google translate its correct; Are you trying to delete a cookie from subsite "second.website.com" in the main domain "website.com" ?
Currently the plugin does not support altering cookies in a specific domain, but there is a open PR to add the functionality, I will look into making a change to the plugin, so cookie domains will be supported in the future, that should also fix your issue.

Thank you for you answer.
I want in "secong.website.com" to delete the ".website.com" cookie.
Look forward to the release of the new version～！

@CrazedBob You should be able to use the lateste realease to solve your problem: https://github.com/alfhen/vue-cookie/releases/tag/v1.1.0


tag"muted" in video is not rendering

Vue version
3.2.25
Link to minimal reproduction
https://stackblitz.com/edit/vitejs-vite-mb3qni?file=src/App.vue
Steps to reproduce
npm run dev
What is expected?
atrr"muted" shoud be in video element
What is actually happening?

muted is disabled
System Info
No response
Any additional comments?
No response

Vue prefers setting things as element properties if that property exists on the element:
// pseudocode

if ('muted' in el) {
  el.muted = true
} else {
  el.setAttribute('muted', '')
}

Now, if you set i.e width on a video element, the browser will also refelct that as an attribute. But it does not do so for muted, for reasons I am not aware of.
video.width = 640
video.muted = true
console.log(video)
// => <video width="640">

I don't think we should start adding expections for these browser behaviors. instead, you can force Vue to set it as an attribute with the .attr modifier:
<video
      width="640"
      height="360"
      src="https://commondatastorage.googleapis.com/gtv-videos-bucket/sample/BigBuckBunny.mp4"
      controls
      v-bind:muted.attr="''"
    ></video>


Handle Labels Correctly When Opening Context Pad

do not attempt to move delete entry as it doesn't exist
do not show comments or resource linking entry

Closes https://github.com/camunda/improved-canvas/issues/40

Did you change the line ending? The diff is broken right now :/

Sorry about that. Fixed.


Tensorflow 2.x is a tragedy and tf.compat is waste. Time to switch to Pytorch

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: NO
TensorFlow installed from (source or binary):Binary
TensorFlow version (use command below): 2.2
Python version:3.7
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:10.1
GPU model and memory: gtx 1060

The Tensorflow's latest version is a complete tragedy. It just breaks each and every functions that has been written with the older version of tensorflow. The eager execution is the pain in the neck. It just throws the countless meaningless error. I got tf.function-decorated Value error. I fixed it and now I'm getting the following error,
tensorflow.python.eager.core._SymbolicException: Inputs to eager execution function cannot be Keras symbolic tensors, but found [<tf.Tensor 'Conv2DBackpropInput_1:0' shape=(1, 13, 13, 1280) dtype=float32>]
2020-06-07 16:49:58.682705: W tensorflow/core/kernels/data/generator_dataset_op.cc:103] Error occurred when finalizing GeneratorDataset iterator: Failed precondition: Python interpreter state is not initialized. The process may be terminated.
	 [[{{node PyFunc}}]]

Disabling eager execution throws the following error,

ValueError: Operation name: "AssignAddVariableOp"
op: "AssignAddVariableOp"
input: "AssignAddVariableOp/Variable"
input: "Const"
attr {
  key: "dtype"
  value {
    type: DT_FLOAT
  }
}
 is not an element of this graph.

Please regularize the code base before even trying to issue a vendor certificate aka TensorFlow certified developer certificate

@Zumbalamambo,
Could you please share the code you're trying to run, so that we can look into it and fix the issue. Thanks!

+1

It just breaks each and every functions that has been written with the older version of tensorflow. The eager execution is the pain in the neck. It just throws the countless meaningless error.

@Zumbalamambo,
Could you please share the code you are running to that we can help you with the errors. Thanks!

@Zumbalamambo,
The issue tracker is for Tensorflow feature requests or bug reports. Please refrain from posting inappropriate comments. If any functionality or feature does not work to your satisfaction, please let us know, as each feedback is valuable to us. Tensorflow team will be happy to address them. Kindly share the information related to code as requested by @amahendrakar. Let us know how we can help.


Iterm2 colorscheme

I saw this colorscheme on trending and decided to give it a try and I've been really liking it so far!
I went ahead and made some pretty basic iterm2 colors locally so my terminal would match the theme. I wanted to contribute it back, but wanted to check if you were interested first, and if so, where the best place to put it would be.
I was thinking a contrib folder (or maybe a nightfly-contrib repo?) might make sense but didn't want to open a PR without getting your thoughts first.
Either way, thanks for sharing the theme!

👍

Sharing the one I made here just in case others come across this issue and want the iterm2 colors before it's possibly contributed upstream.
https://gist.github.com/BlakeWilliams/4e10b8856050a51b6d2ddb3b631059d9

There does appear to be demand for matching terminal themes. So yes, I will add something in the coming days.
I will create and test nightfly terminal themes for:

iTerm2
GNOME Terminal

Those are the two terminals I use on Mac and Linux.
However, I want to use and test them before committing them. Once I am happy I will add them to a new terminal_themes subdirectory. There are likely to be changes to the foreground and selection colors when compared to the Vim theme going by what I did with the moonfly terminal themes vs the Vim theme.
I will keep this issue open till then.
Cheers.

Done.

iTerm2 nightfly theme

GNOME Terminal Nightfly setup script


TEST Only - Gortega issue400 code ql

Type of PR

[ ] Documentation changes
[ ] Code changes
[ ] Test changes
[ ] CI-CD changes
[ ] GitHub Template changes

PR Checklist

[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] My code follows the code style of this project.
[ ] I ran lint checks locally prior to submission.
[ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Purpose of PR

Does this introduce a breaking change

[ ] YES
[ ] NO

Validation

[ ] Unit tests updated and ran successfully
[ ] Update documentation or issue referenced above

Issues Closed or Referenced

Closes #issue_number (this will automatically close the issue when the PR closes)
References #issue_number (this references the issue but does not close with PR)

Close this test PR


Increase code coverage for the DID pkg

Mostly the peer method! https://app.codecov.io/gh/TBD54566975/ssi-sdk/tree/main/did. Aiming to get to 80% overall.

👍 got it. will do. Thanks!

Quick note: This issue should be referenced in ssi-sdk, where the issue resides.


Batch Upsert Script

Which version of Elastic are you using?
[ V ] elastic.v7 (for Elasticsearch 7.x)
[ ] elastic.v6 (for Elasticsearch 6.x)
[ ] elastic.v5 (for Elasticsearch 5.x)
[ ] elastic.v3 (for Elasticsearch 2.x)
[ ] elastic.v2 (for Elasticsearch 1.x)
Hey , I am sending this script :  req := elastic.NewBulkUpdateRequest().ScriptedUpsert(true).Index(ew.index).Id(tmp.Id).Script(script).Type("message")
I am receiving error that document is missing (and he is missing its on purpose) I would expect that if its missing It will create a new document
any Idea why its not working (I cannot set it as doc as upsert because it wont let me add Doc and script on the same request)
Script is :
script := elastic.NewScript("if ((ctx._source.doc != null) && (ctx._source.doc.updated_at > params.doc.updated_at)) { ctx.op='noop'; } else { ctx._source.doc=params.doc }")
script.Param("doc", tmp)
script.Lang("painless")

I would add any additional necessary info.
Would appreciate any help / code snipping that will help me , Thanks a lot .

Could it be that you're only missing bulk_as_upsert, aka BulkAsUpsert(true)? Not sure if scripting and upserts really work together.
This is more of an ES question, not one of the Go driver.

Hey ,
Just wanted to update i manage to make it work ,
Here is the code sample for people who might need it in the future :
script := elastic.NewScript("if (ctx._source != null && ctx._source.updated_at > params.doc.updated_at) { ctx.op='noop'; } else { ctx._source=params.doc }")
script.Param("doc", tmp)
script.Lang("painless")

req := elastic.NewBulkUpdateRequest().ScriptedUpsert(true).Index(ew.index).Id(tmp.Id).Script(script).Upsert(tmp)
bulk = bulk.Add(req)

....
_ , err : =bulk.do(ctx)


ACM-10914 upgrade to react router dom v5 compat

lots of files but most are simple changing of Link to v5-compat
a guide to other changes:
https://docs.google.com/document/d/1ea1lsh6CekZjuPmpWH5c3NohlVnCBpnWzmXCZFLZrks/edit?usp=sharing

NOTE: AI plugin must be updated to compat--v6 doesn't support RouteComponentProps in component constructor
{/* <Route path="/edit/:namespace/:name" element={} /> */}
NOTE: LostChanges.tsx and LostChanges.test.tsx have been commented out/skipped

/hold

Is this being worked on ? CIM is also looking to upgrade to v5 compat but this work needs to merge first.
I believe you need to upgrate @openshift-console/dynamic-plugin-sdk-webpack to 0.0.11 in order for the react-router-dom-v5-compat module federation to work correctly

Superseded by https://github.com/stolostron/console/pull/3579


Total Account Value portion of Your Overview

The green total portfolio percentage is still to be added from augur-node, just mock that out with 000.0000 will update when we get the data.
When user clicks time range, values in section change
Total Account Value (ETH) changes based on Date Range (augur-node issue: https://github.com/AugurProject/augur/issues/1180) This will be on the profit and loss state.
REP staked, only currently staked REP doesn't change.

More portfolio stuff popped up, moving this to Andy.

Only the % of Total Acct Value Changes:
Total Acct Value, Available Trading balance, FF and the Rep balance/staked remain the same (when toggling between time lengths)

Yes, that is our understanding as well. 👍

@andrewdamelio format frozen funds use formatEther utils/format-number

Added blocking tix for augur-node work, need to get percentage change in user's Total Account Value.

ticket is redundant - works been done - closing


Cannot use Chinese in lua

I want to make a version for chinese player, but no matter i use UTF-8\UTF-8 No Bom or ANSI, the simplified Chinese text will dispear.

As explained in #55, the text renderer used by the program doesn't support Unicode at the moment. Unfortunately it's not an easy thing to fix, but it's definitely on my radar.

It seems that the SimpleGraphic.dll made by C++, so sorry about that because I am a new one in C#/Unity/Lua :D, hope to see more programs

if u replace the text render, pls let me no


Create script to convert blendshapes into attributes

GUI to load blend shape node from selected geo (cycle through found blend shape nodes)
GUI to load target control (usually a curve)
Text field to filter undesired strings
Option to ignore blend shapes with connections

Create Work in Progress file: gt_blends_to_attributes.py


added DM'ing and no response required

added

don't DM without consent
no one owes anyone their attention

Hi @katiekovalcin Should I close this?


Release plans are not cross platform transferable

We have a team that uses mac, linux and windows.
only the users on mac and linux can perform releases, because our release plan uses $FOUNDRY_VERSION but foundry switches to %FOUNDRY_VERSION%  which is blank.
powershell supports $FOUNDRY_VERSION:
ps> set FOUNDRY_VERSION 0.3.6`
ps> echo $FOUNDRY_VERSION
0.3.6

Some thought needs to go into how this works.

That's what I get for assuming Windows users were using cmd.exe =(
It looks like we did attempt to account for this by wrapping all command executions in sh or cmd but we didn't read the code we copy/pasted from npm too closely:
https://github.com/twolfson/foundry/blob/4.3.2/lib/release.js#L293-L311
I think the patch should be as simple as removing process.env.comspec (assuming that's something that Powershell sets and npm was detecting)

For next steps, I'll triage this some time tonight to see if the comspec patch will work

I loaded up my IE VM and attempted to reproduce the issue but was unable to. I added a test case for environment variables in the CLI scenario (I was surprised we never tested it explicitly) but came up empty handed:
https://github.com/twolfson/foundry/blob/4.3.3/test/cli.js#L63-L65
I ran the test suite in Cmder with both cmd.exe and powershell.exe.
Can you verify there isn't anything quirky going on the comspec environment variable on your coworker's computer? It looks like it is intended to be used on Windows:
http://stackoverflow.com/a/8121351
When playing around with the comspec variable on the VM, cmd.exe had it set to:
λ echo %comspec%
C:\Windows\system32\cmd.exe

whereas powershell.exe didn't have it set at all:
E:\foundryPS>echo $comspec

For the record, it's my workstation, i switch between linux, mac and windows.
Some of my co-workers use just mac some just use windows.
This absolutely needs to work on windows powershell.
You didn't see it in powershell because its $env:COMSPEC which is not the same as $COMSPEC :
qcu005 master *+$ < $ %COMSPEC%
%COMSPEC% : The term '%COMSPEC%' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or
if a path was included, verify that the path is correct and try again.
 Everything looks fine so the commands should actually be executing fine as cmd.exe is receiving %FOUNDRY_VERSION%. What has been the error you're receiving?

The problem is that %FOUNDRY_VERSION% is not valid syntax in powershell so it just gets passed along as is.

It's not meant to be interpreted by Powershell, it's supposed to be interpreted by the cmd.exe command. From my perspective, the test suite has verified that everything works even when the top level shell is Powershell.
Our foundry output doesn't interpolate the content itself because we can't simulate that in JS:

However, it does say the environment variable values at the top of that output
Can you point me to something tangible to reproduce the issue you are having? Or maybe can you run our test suite as a sanity check for both of us?
# Clone the repository
git clone https://github.com/twolfson/foundry
cd foundry

# Install our dependencies
npm install

# Run Windows specific tests
npm run test-windows-ci

Closing issue due to inactivity


xCode:

Hello guys, i have a little problem. When i run the application in xcode, the stuff will be loaded but i get the message:
2017-10-19 13:39:10.802068+0200 MyFirstARKitApp[435:65378] [] nw_connection_get_connected_socket 4 Connection has no connected handler
2017-10-19 13:39:10.802115+0200 MyFirstARKitApp[435:65378] TCP Conn 0x1c417b300 Failed : error 0:61 [61]
So the content (shapes,...) cannot be loaded. The camera on the phone works and i get the debug mode with the yellow dots and the coordinate system.
Please can you help me!? :)
Thanks a lot

@dasbenni you might have to follow the manual installation instructions so the app can find your assets https://github.com/HippoAR/react-native-arkit#manual-installation I'm only guessing though as I'm assuming you installed the app all via the cli?

@benni-t is this still an issue? if so, best ask in the slack channel: react-native-ar.slack.com


Register Top-level Namespace

Hi CDX team, We would like to register top-level namespaces for sonatype . It will be used by the Lift project and administered by Lift maintainer.
our public taxonomy link is here: https://help.sonatype.com/lift/open-source-vulnerability-analysis/dependency-view/cyclonedx-lift-namespace-taxonomy
Thank you for taking your time.

sonatype has been registered, however, I would recommend coordinating with other teams within Sonatype to ensure there isn’t property confusion within the namespace. Perhaps use sonatype:lift for Lift for example. FYI sonatype:score as documented https://help.sonatype.com/lift/open-source-vulnerability-analysis/dependency-view/cyclonedx-lift-namespace-taxonomy doesn’t make any sense as the spec can already represent this in a standardized way without the use of custom properties.

by standardized way, you mean the score?

Multiple ratings are supported, including scores, severity, etc. see https://github.com/CycloneDX/bom-examples/blob/master/VEX/vex.json for an example

Thanks for the quick reply,  sonatype:score is different from these specs, we probably need to rename it to avoid some confusions


No need to invoke allocatePublicIp when provisioning worker nodes

RunInstances with internetMaxBandwidthIn, internetMaxBandwidthOut params will automatically allocate a public IP for the instance. No need to invoke allocatePublicIp separately.

done


API to record consent preferences

Add an API endpoint to:
POST /api/v1/consent

{
  "identity"<EMAIL_ADDRESS> # should this be hashed?
  "identity_type": "email",
  "regulation": "gdpr",
  "data_use": "this can validated against Fideslang until we unify Fides",  # we won't support custom data uses
  "opt_in": "true"
}

Create a ProvidedIdentity record for the identity value passed.
Create a corresponding Consent record.

Good spot — while it's impossible to verify phone numbers with Fidesops currently that doesn't mean people can't verify the numbers themselves and add them. I believe we've not hit this with Privacy Requests yet as the only folks using them are using email only (which is configurable in the privacy centre itself). In this scenario it's reasonable to expect an auth token in order to perform the action.
My main concern with that conclusion is how big a lift it is to build in another auth method depending on payload at the same URL, I don't think we have a convention for this yet.


Deferred cleanup of resources when a node crashes

Description of the changes
Type of fix: Enhancement
Fixes #845
Changes in the PR include:

DeleteEndpoints API which takes in a hostname of the node crashed and cleans up the etcd endpoint cfg and oper state and calls ofnet API to clean up the flows of these endpoints on all other hosts in the cluster
Vendoring in ofnet changes of ClearNode which clears up flow state of all endpoints in the host

TODO

[ ] Cleanup ovs-driver oper state of the node (required exposing new APIs to netmaster. ovs-drive is currently accessed only by netplugin on the host)
[ ] Test cases
[ ] Have an option to configure the timer for starting cleanup of resources

please create a new issue to address multiple GC timers being triggered on nodes.
LGTM


MSO Conditional Code for older Outlook

Hi Fabio,
Thanks for sharing this code from your TEDC talk! I have a question about the MSO conditional code. Should it be updated to include older versions of Outlook?
From...
<!--[if mso]>

To...
<!--[if (mso)|(IE)]>

Thanks!

I used:
<!--[if (gte mso 9)|(lte ie 8)]>

because ie9 does support inline-block and max-widths.


Label suggestion

GET http://<IP_ADDRESS>:5500/Assignment/Beginner-javascript/calculator/calculator net::ERR_ABORTED 404 (Not Found)

Spam. Closing


(VizroAI) Support for AzureChatOpenAI and most updated models for OpenAI (such as GPT-4o mini)

Question
Hi Vizro AI Team, OpenAI has recently updated there newest gpt-4o-mini model and made gpt-3.5-turbo outdated. Will this update be reflected in Vizro AI? Also, I noticed the part about working with Azure OpenAI is deleted. Is this under development? Will user be able to use Azure OpenAI endpoints to conduct Vziro AI tasks?
Many thanks!
Code/Examples
No response
Which package?
None
Code of Conduct

[X] I agree to follow the Code of Conduct.

And will Vizro-AI support opensource LLMs such as llama 3.1 as well?

Hey @Layman-Bebop!
Thank you very much for your question!
Yes we are indeed going to add gpt-4o-mini to the pre-defined models and generally update our pre-defined strings.
However, did you know you could configure any model already yourself? The second example in the below link shows how to choose any OpenAI model, not just the pre-configured ones!
https://vizro.readthedocs.io/projects/vizro-ai/en/vizro-ai-0.2.0/pages/user-guides/customize-vizro-ai/#customization-at-initialization
As for AzureOpeAI, we are temporarily removing it until a bugfix can be found for an existing issue. @lingyielia can elaborate here.
And finally, yes, we are actively working on supporting open source model, and the release of that functionality is not so far away, so stay tuned for this :)!

Thank you very much @maxschulz-COL ! Will stay tuned!

Hi @at-eez , with our new release we now support more LLM vendors and models: https://vizro.readthedocs.io/projects/vizro-ai/en/vizro-ai-0.3.0/pages/user-guides/customize-vizro-ai/
In theory, any model that supports langchain structured_output would work with Vizro-AI. I'm closing this issue now. Do let us know if you have additional findings! :)


FIX: if a handler is not lazy we have unknown shape don't force it

Passing the -1 in the shape through will result in an eventual shape
miss-match when trying to retrieve the data.  Rather than fail,
aggressively pull the data up.  The assumption is that if there were a
handler that was expected to load large volumes of data the handler
would be natively delayed.

@gwbischof Can you please sort out a test case for this?

Confirmed on Run'89c1bfac-73eb-407b-9012-d852231c6eda' from FXI that using databroker 1.0.6 we get
>>> run.primary.read()
ValueError: replacement data must match the Variable's shape. replacement data has shape (172, 10); Variable has shape (172, -1)

but with this branch access works.

Punting unit testing to https://github.com/bluesky/databroker/issues/598


[cocoapods] add --repo-update as default

🔑
This PR add --repo-update as default for cocoapods action as requested in issue #4497.
I leave support for --no-repo-update for previous versions of cocoapods.
Problem:
pod install in cocoapods 1.0.0 will not run pod repo update by default
Solution:
They added param --repo-update that may be called to update repository before pod install
Example:
action cocoapods
will run cmd cocoapods --repo-update

on cocoapods 0.39 - repo is updated by default
on cocoapods 1.0.0 - repo update will be triggered by this param

action cocoapods(repo_update: false)
will run cmd cocoapods --no-repo-update

on cocoapods 0.39 - repo will be not updated because this param
on cocoapods 1.0.0 - repo is not updated by default

Issue link: https://github.com/fastlane/fastlane/issues/4497

Ideally this shouldn't be merged IMO, converting to not update by default was a very conscious choice based on real constraints.
If support like this is critical, then I'd say that you need cocoapods-check do it's work to determine if you need to do a pull first, this means updates only happen when they need to

I agree with @orta that we should stick to the same defaults that CocoaPods is using itself. I am going to close this PR but please feel free to submit a new one with the feedback above taken into consideration! 🐙 !


Ammo: System keeps firing collisionend events

Upon further experimenting with ammo, I came across this bug:
After two entities have collided for the first time, the system just keeps emiting the 'collideend' event a couple times per second, even while they are still colliding.
Short example (Use arrow keys to move orange ball):
https://glitch.com/edit/#!/tidy-cyclone
Side note: This only happens, if the emitting entity is dynamic, or kinematic - it works fine, if the emitting entity is static.

In progress: https://github.com/donmccurdy/aframe-physics-system/pull/135.


Update directx-sdk--august-2009-.md

A few more updates since we were able to get the legacy download restored

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


fix: Updated the depreciated RunQueryNVerifyResponseViews to use runQueryAndVerifyResponseViews

Description
This PR removes the deprecated RunQueryNVerifyResponseViews and implemented runQueryAndVerifyResponseViews instead.
Fixes
Automation
/ok-to-test<EMAIL_ADDRESS>@tag.Datasource"
:mag: Cypress test results

[!CAUTION]
🔴 🔴 🔴 Some tests have failed.
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/12163035711
Commit: aaa6bcdcc1f1d448808b8d8c4b69172ae30e6855
Cypress dashboard.
Tags: @tag.Sanity, @tag.Datasource
Spec:
The following are new failures, please fix them before merging the PR:
cypress/e2e/Regression/ServerSide/ApiTests/API_MultiPart_Spec.ts
List of identified flaky tests.
Wed, 04 Dec 2024 16:36:04 UTC

Communication
Should the DevRel and Marketing teams inform users about this change?

[ ] Yes
[x] No

Summary by CodeRabbit

New Features

Enhanced clarity and consistency in method calls for querying and verifying responses across various test cases related to Google Sheets and Oracle/MsSQL data sources.
Added a new test case for verifying the default port for Microsoft SQL Server.

Bug Fixes

Improved error handling in MsSQL tests to validate missing fields with appropriate feedback messages.

Refactor

Standardized method naming conventions and parameter formats across multiple test cases to improve readability and maintainability.

Closing this PR. These changes are already available in https://github.com/appsmithorg/appsmith/pull/37897


Job Ads Portal Design

As a
UI/UX Dev
I want to
Create a web portal design
In order to
To provide UI/UX plans for the PPH Job Ads Portal

Acceptance Criteria

[ ] UI/UX Dev team must create a design document for the UI/UX plans for the PPH Job Ads portal
[ ] The design draft must include all necessary flows that show all validation messages, errors, and success states
[ ] Use a Professional-feeling color palette and design.

Notes

Inform POs if you need certain accounts for drafting. As much as possible let's use free ones.

https://www.figma.com/team_invite/redeem/cCbLn4oQX3KgCCt7AKAMUt
link to figma design

Common Job Ad form fields

Note

This is a non-exhaustive list. You can take some liberties with the design.
Just keep in mind that simplicity is still our objective (e.g. leave out some fields that you feel aren't important).
When in doubt, post a question in our Discord channel.
Fields in bold and marked with asterisks (*) should be required

Job Title*
Posting date (auto-generated)
Closing date - until what date the position is available
Employer / Company name*
Work type* (dropdown)

Casual
Contractual
Part-time
Full time

Location*

WFH (if checked, disable the free text field
free text (stretch goal: autocomplete location)

Salary range (in PHP)*

Minimum*
Maximum

Job Description* (text area; stretch goal: markdown support / rich text area)
External URL (e.g. to company website, LinkedIn, or other job boards)

A few options for WSIWYG editors

https://tiptap.dev/
https://lexical.dev/
https://www.tiny.cloud/

But if we don't have the luxury of time, we could just stick with a simple text area for the first iteration

RE: Job Ads Portal Design
Good day, me and @Jiseeeh  finished a current Mockup of the Job Ads Portal Design. We focused  more on the form first and we'd like to get some feedback. We tried our best to stay close to the designed wireframe by the other members. Colors are not final.
Thank you.


A patch to fix #2873

Fixes https://github.com/symplify/symplify/issues/2873

The CI errors seem to be unrelated to this PR

The CI errors seem to be unrelated to this PR

Thank you :+1:

Thank you :+1:


Enhace PR automerging

Gitea version (or commit ref): 1.1.0+75-g6a451a2b
Git version:  2.8.3
Operating system: Upstream docker container
Database (use [x]):

[ ] PostgreSQL
[X] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[X] Not relevant

Description
A colleague of mine produced a problem in a file (the content is one line off the master). He then opened up a PR which got the state "This pull request cannot be merged automatically because there are conflicts."
Looking at the log I see the following error:
Mar 28 09:28:37 dockert docker-compose: gitea_1   error: patch failed: 90_common/role.yaml:550
Mar 28 09:28:37 dockert docker-compose: gitea_1   error: 90_common/role.yaml: patch does not apply

I'm able to reproduce this error on the CLI when I create a patch and try to apply it manually:
$ git apply --check ../changes.diff
error: patch failed: 90_common/role.yaml:550
error: 90_common/role.yaml: patch does not apply

However when I do a direct merge of the branch git is able to resolve the issue:
$ git merge pr/590
Auto-merging 90_common/role.yaml
Merge made by the 'recursive' strategy.

I guess it would be good to make use of this from gitea.

@lunny I'm currently no longer work with gitea as I changed jobs, not sure if this is already fixed or not.

Cannot reproduce that. Let's close now and please feel free to reopen it.


net/http: use Copy in ServeContent if CopyN not needed

This small PR allows optimizations made in io.Copy (like the use of io.WriterTo) to be used in one possible path of http.ServeContent (in case of a non-Range request).
This, in turn, allows us to skip the buffer allocation in io.Copy, which has performance benefits:
name            old time/op    new time/op    delta
ServeContent-8    38.6µs ±11%    23.3µs ± 6%  -39.56%  (p=0.000 n=10+10)

name            old speed      new speed      delta
ServeContent-8  14.8GB/s ±10%  24.4GB/s ± 6%  +65.07%  (p=0.000 n=10+10)

name            old alloc/op   new alloc/op   delta
ServeContent-8    32.9kB ± 0%     0.1kB ± 1%  -99.80%  (p=0.000 n=9+10)

name            old allocs/op  new allocs/op  delta
ServeContent-8      6.00 ± 0%      4.00 ± 0%  -33.33%  (p=0.000 n=10+10)

(results are only representative for non-Range requests; Range requests see no changes in performance)

Benchmark code: https://gist.github.com/costela/44079c1c742007d355cb3e382d0ac0ef


reclaim the last ram of an account

If I want to delete an account, can I reclaim the 4kb (necessary for the account) by some function like relaim_account_resource(account_a, account_b), which delete account_a, and forward resource to account_b.

In order to focus our efforts on issues that are currently creating difficulty for the community we are closing tickets that were created prior to the EOSIO 2.0 release.  If you believe this issue is still relevant please feel free to reopen it or create a new one.  Thank you for your continued support of EOSIO!

In order to focus our efforts on issues that are currently creating difficulty for the community we are closing tickets that were created prior to the EOSIO 2.0 release.  If you believe this issue is still relevant please feel free to reopen it or create a new one.  Thank you for your continued support of EOSIO!


Binary clipboard

Being able to read the clipboard as a vector or slice of u8s would be extremely useful to anyone who needs to work with the clipboard not just with strings.
I know this is feasible in Windows, and I think it can be done in macOS (from what I read of the current implementation), but I don't know about in X11.

I agree that this would be a useful feature. I'm willing to accept a PR that adds support for the platforms that you're willing to implement it for, and I'll add support for the rest of the platforms (it should be feasible under X11 too).

Has there been any progress on this?
I am willing to work on an X11 version. I am sure that it can be done because I have written a Java application which copies images from the system clipboard from a different graphics application under Linux. I think that is a proof of concept, at least.
Multiple things can be in the clipboard at the same time, which might require changes to your elegant clipboard.

I am also looking for this functionality... Got a panic when calling Result::unwrap() on ctx.get_contents() when trying to read image data:
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: StringError("pasteboard#readObjectsForClasses:options: returned empty")', libcore/result.rs:945:5
note: Run with `RUST_BACKTRACE=1` for a backtrace.

Check this
https://github.com/SpiralP/rust-clip-sys


ldap_person Object Improvements

Some input we gathered regarding the new ldap_person object (which replaces the Person profile via #813:

There are a few required attributes from the 3 LDAP classes we would want to make sure we have covered, and the attribute names should match:

ldap_dn should be shortened to dn
ldap_cn should be shortened to cn
LDAP (sn) surname is not in the current object. Since it is a key LDAP object, it should be added.

employee_uid is uid in LDAP

OCSF uses uid too, maybe that can be shortened within the object
Or we could have ldap_person extend _entity so we pick up name and uid)

given_name which is givenName in LDAP

on this one, I'm not sure switching to camel-case would be a good practice to do, but adding it so we do not forget it was raised.

I agree with dn, cn, and sn. I think we need to keep the lowercase given_name. If we extent _entity I think we pick up the constraint for name or uid. So, we would want to open up the constraints if we do that.

Following thorough deliberation in our System Activity syncs, we have reached the conclusion that while shortening the attribute names may seem desirable, implementing such changes would constitute a breaking modification. Alternatively, one might consider deprecating the current attributes in favor of newly abbreviated ones. However, given the importance of exercising discretion with deprecations, our recommended approach is to maintain the existing attributes in their current form at this juncture.


What is the api/function/process of opening a .docx file in Word app in Teams

I am developing my Teams app, and I would like to know the api/function/process of how to open a .docx file in Word app in Teams, as I want to integrate it into my own functions. I analyzed network packages while I interacting with Teams in browsers, it seems like a process that i) POST a complicated http request, which includes file info, ii) Receive a HTML file as the response, iii) Use the HTML file to show the content of my file. I don't know if I am right, and would like to know more details. Thanks

@qianwenyuan - We are looking into this we will get back to you soon.

@qianwenyuan - Here is workaround for API use deeplink-
/// <summary>
        /// Link to open file in teams.
        /// </summary>
        public const string OpenFileInTeamsURL = "https://teams.microsoft.com/_#/{0}/viewer/teams/{1}";

Ref Doc-https://github.com/OfficeDev/microsoft-teams-apps-newemployeeonboarding/blob/main/Source/Microsoft.Teams.Apps.NewHireOnboarding/Constants/DeepLinkConstants.cs
We can use deep-link to open any file in Teams.
// If learning content url contains file extension opening the file in teams scope else redirecting to browser.
                card.Actions.Add(
                    new AdaptiveOpenUrlAction
                    {
                        Title = localizer.GetString("ViewLearningPlanButtonText"),
                        Url = string.IsNullOrEmpty(filExtension)
                    ? new Uri(learningPlan.Link?.Url)
                    : new Uri(string.Format(CultureInfo.InvariantCulture, DeepLinkConstants.OpenFileInTeamsURL, filExtension, actionDataUrl)),
                    });
            }

Ref Doc- https://github.com/OfficeDev/microsoft-teams-apps-newemployeeonboarding/blob/main/Source/Microsoft.Teams.Apps.NewHireOnboarding/Cards/LearningPlanCard.cs
new Uri(string.Format(CultureInfo.InvariantCulture, DeepLinkConstants.OpenFileInTeamsURL, filExtension, actionDataUrl)),

@qianwenyuan -Did you get chance to check?

@qianwenyuan - Please let us know if your issue has resolved or still need any guidance?

We are closing this issue for now, please feel free to reopen it, if required.


Support mixerclient to generate authentication attributes

What this PR does / why we need it: Extracts information from "sec-istio-auth-userinfo" header which is generated from auth filter, and generates authentication attributes for mixer server to check. This is part of efforts that enable auth filter in envoy.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #

@JimmyCYJ: Adding do-not-merge/release-note-label-needed because the release note process has not been followed.

One of the following labels is required "release-note", "release-note-action-required", or "release-note-none".
Please see: https://github.com/kubernetes/community/blob/master/contributors/devel/pull-requests.md#write-release-notes-if-needed.
Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@JimmyCYJ: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixerclient-presubmit.sh
72ecab72f8671e95f62da1f63645cc6558e6a0fd
link
/test mixerclient-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@JimmyCYJ: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/mixerclient-presubmit.sh
8b86eb625b6087bac9fb06da54a45e80eeab3fe2
link
/test mixerclient-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Authoritative location for Wasm OCI spec

Which of

https://github.com/solo-io/wasm/blob/master/spec/README.md
https://github.com/solo-io/wasm-image-spec/blob/master/README.md

is correct?

Hi @craigbox, https://github.com/solo-io/wasm/blob/master/spec/README.md is the authoritative source now.
We have moved from the wasm-image-spec repo to here, so that we have one consolidated location for folks to find the wasm OCI spec, the sdks, and the cli tools like wasme.
I will update the old repo to make it clear that the spec has moved to this repo.


Node Engine >= 8

Node Engine >= 8

Coverage remained the same at 97.727% when pulling f967d595d3485ba923e06ed458c0083eb6b89724 on knoxcard2:patch-1 into 8fc5b926846cd4ef9a2dbccd411705e0c110a708 on Leonidas-from-XIV:master.

I don't see the point of this change, care to explain?


Question: Actor Directory info

Hey there! I've been using this plug-in for quite a while, and it's worked great so far.
I write because I was interested in the "Actor Directory" option on the agent settings, but I couldn't find any info about it on the Wiki section, and I was hoping you could give me a brief description of how to use it.

It is not working right now. Just ignore it. See #1 for details.

Wow... I read through that thread before making this one, but I somehow missed the part that touched upon this matter. Thanks for the reply regardless.


Security Group doesn't add new ingress rule

What happened?
I added a new ingress rule to a SecurityGroup but it wasn't applied to the resource.
After the update Ready = True and Sync = True, no errors in events.
How can we reproduce it?
Create the following security group:
cat <<EOF | kubectl apply -f -
apiVersion: ec2.aws.crossplane.io/v1beta1
kind: SecurityGroup
metadata:
  name: crossplane-aws-rds
spec:
  deletionPolicy: Delete
  forProvider:
    description: Security group to allow traffic to databases created by Crossplane
    groupName: example-crossplane-aws-rds
    ingress:
    - fromPort: 5432
      ipProtocol: tcp
      ipRanges:
      - cidrIp: <IP_ADDRESS>/16
      toPort: 5432
    region: eu-west-2
    vpcId: vpc-xxx
  providerConfigRef:
    name: default
EOF

Add a new ingress rule:
cat <<EOF | kubectl apply -f -
apiVersion: ec2.aws.crossplane.io/v1beta1
kind: SecurityGroup
metadata:
  name: crossplane-aws-rds
spec:
  deletionPolicy: Delete
  forProvider:
    description: Security group to allow traffic to databases created by Crossplane
    groupName: example-crossplane-aws-rds
    ingress:
    - fromPort: 3306
      ipProtocol: tcp
      ipRanges:
      - cidrIp: <IP_ADDRESS>/16
      toPort: 3306
    - fromPort: 5432
      ipProtocol: tcp
      ipRanges:
      - cidrIp: <IP_ADDRESS>/16
      toPort: 5432
    region: eu-west-2
    vpcId: vpc-xxx
  providerConfigRef:
    name: default
EOF

What environment did it happen in?
Crossplane version: 1.0.0
AWS provider: 0.16.0

Also if I update the ipRanges on an existing ingress rule, it won't remove the ipRanges that I removed on the object.

Also if I update the ipRanges on an existing ingress rule, it won't remove the ipRanges that I removed on the object.

i have encountered the same issue.
seams like the crossplane provider is looking only for the first ingress entry. going further with @bandesz example if I modify an existing security group and replace the existing ingress entry with a new one:
cat <<EOF | kubectl apply -f -
apiVersion: ec2.aws.crossplane.io/v1beta1
kind: SecurityGroup
metadata:
  name: crossplane-aws-rds
spec:
  deletionPolicy: Delete
  forProvider:
    description: Security group to allow traffic to databases created by Crossplane
    groupName: example-crossplane-aws-rds
    ingress:
    - fromPort: 3306
      ipProtocol: tcp
      ipRanges:
      - cidrIp: <IP_ADDRESS>/16
      toPort: 3306    region: eu-west-2
    vpcId: vpc-xxx
  providerConfigRef:
    name: default
EOF

it will add a new ingress rule next to the previous one. modification is not possible.

yes we faced today the same issue that modification is not possible

There is no way to modify, so it should remove & add the new one but seems like we only add but never delete right now.

when I update the ingress rule, it will create a new rule and won't delete the old one.


Move "Push" behavior from NetworkProxy to protocol

Move "Push" behavior from NetworkProxy to protocol
The Protocol declaration now does not include "pushed" response not initiated by the client like messages, therefore is not evident how the messages flow is working. Is difficult to generate automatically the protocol chart

Too clunky to implement (too much stuff to change for the result obtained)


feat: add partner service for depositing into vaults

Seems like we have been dragging an issue in the concept of the Token interface, and we should address in a future refactor the fact that the Token interface should not have knowledge in any way of zaps/vaults/partners. We should aim to decouple them and have generic allowance and approve functions that are independent of if its a deposit, withdrawal, has a partnerId, etc. The vaults interface can be responsible of defining what contract is the spender.


Fix config load

I was having issues when running my python scripts with specific python versions and with full file paths.
For example, something like:
C:/Users/sodwyer/AppData/Local/Programs/Python/Python36-32\python.exe D:/Path/To/Project/proj_script.py
The JSON config file would fail to load, and the dumped config file would be dumped into the directory where python.exe was run from.
I made a couple of changes so that the folder of the current script is used instead. It seems to work ok for my use case but maybe it's not desired? Also, I didnt see any open issues about this so maybe it's a real edge case. Thought I'd pull request anyway just in case.
Let me know, thanks

Long delay (sorry!), but these changes look good to me. Please re-open this against the current release branch

done


Do not override model_post_init in subclass with private attrs

Change Summary

Related issue number
Fixes https://github.com/pydantic/pydantic/issues/7293

Checklist

[x] The pull request title is a good summary of the changes - it will be used in the changelog
[x] Unit tests for the changes exist
[x] Tests pass on CI
[ ] Documentation reflects the changes where applicable
[x] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

Please review
[PermissionError]: [Errno 13] Permission denied: '/home/runner/work/pydantic/pydantic/coverage' 🤨


All models get stuck in "pending" state

Describe the bug
I wanted to try out GPU Stack so after reading the docs and deploying with docker I've found that no matter what model I deploy - it gets stuck in "pending" and never deploys.

All my many other containerised apps are working fine and there's no errors in the logs - it just never seems to do anything?
Steps to reproduce

Spin up GPU Stack from docker / docker compose
Deploy a model (Ollama or Huggingface)
Wait...
Nothing happens (forever "pending")

Result
Deploying a model only gets as far as pending
Expected behavior
Model deployed
Environment

GPUStack version: The official docker image says its "Server 0.0.0, UI 9bd6ded" but I pulled the latest image today (2024-08-30) which has the SHA "sha256:eac62dfc76ee67991af2c29f958da051886f8e1a36d986b6ea3b1dd076bbe3c3" which looks like it was created yesterday: "2024-08-29T21:10:16.077550927Z"
OS: Fedora 40, Docker 27.2.0, Compose v2.29.2, NV Container toolkit 1.16.1-1
GPU: 1x 3090, 2x A4000

Additional context
docker logs -f gpustack
2024-08-29T21:10:17+00:00 - gpustack.server.server - INFO - Starting GPUStack server.
2024-08-29T21:10:17+00:00 - gpustack.server.server - INFO - Running database migration.
2024-08-29T21:10:19+00:00 - gpustack.server.server - INFO - Database migration completed.
2024-08-29T21:10:19+00:00 - gpustack.worker.worker - INFO - Starting GPUStack worker.
2024-08-29T21:10:19+00:00 - gpustack.server.server - INFO - Serving on <IP_ADDRESS>:80.
2024-08-29T21:10:19+00:00 - gpustack.worker.worker - INFO - Serving worker APIs on <IP_ADDRESS>:10150.
2024-08-29T21:10:19+00:00 - gpustack.worker.exporter - INFO - Serving metric exporter on <IP_ADDRESS>:10151.
2024-08-29T21:10:19+00:00 - gpustack.worker.worker_manager - INFO - Registering worker: gpustack

It seems that the worker registration never completes.

Can you turn on debug mode to see if there's more useful info?
Ref: https://docs.gpustack.ai/troubleshooting/#configure-log-level
Can you check if status of workers & GPUs is expected in the Resources page?
Can you exec into the container and do curl -v http://localhost/v1/workers?

Sorry was away over the weekend! Will set the log level tomorrow and report back :)

root@gpustack:/# curl -v http://localhost/v1/workers
*   Trying <IP_ADDRESS>:80...
* Connected to localhost (<IP_ADDRESS>) port 80 (#0)
> GET /v1/workers HTTP/1.1
> Host: localhost
> User-Agent: curl/7.81.0
> Accept: */*
>
* Mark bundle as not supporting multiuse
< HTTP/1.1 200 OK
< date: Sun, 01 Sep 2024 21:55:11 GMT
< server: uvicorn
< content-length: 2207
< content-type: application/json
<
{"items":[{"name":"gpustack","hostname":"gpustack","ip":"<IP_ADDRESS>","labels":{},"system_reserved":{"memory":1073741824,"gpu_memory":1073741824},"state":"ready","state_message":null,"status":{"cpu":{"total":12,"utilization_rate":75.32894736842105},"memory":{"total":201079345152,"utilization_rate":13.99464841638913,"is_unified_memory":false,"used":28140347392,"allocated":0},"gpu_devices":[{"name":"AMD Device 164E (VGA compatible)","uuid":"","vendor":"AMD","index":0,"core":{"total":0,"utilization_rate":0.0},"memory":{"total":0,"utilization_rate":0.0,"is_unified_memory":false,"used":20168704,"allocated":null},"temperature":34.0},{"name":"NVIDIA GeForce RTX 3090","uuid":"GPU-875d550c-606a-9920-3296-7b219227b718","vendor":"NVIDIA","index":0,"core":{"total":10496,"utilization_rate":0.0},"memory":{"total":25335037952,"utilization_rate":0.016296671857458443,"is_unified_memory":false,"used":4128768,"allocated":null},"temperature":27.0},{"name":"NVIDIA RTX A4000","uuid":"GPU-389accfb-8815-6341-b1cc-82d6189527a0","vendor":"NVIDIA","index":2,"core":{"total":6144,"utilization_rate":0.0},"memory":{"total":16789602304,"utilization_rate":0.024591219763533956,"is_unified_memory":false,"used":4128768,"allocated":null},"temperature":31.0},{"name":"NVIDIA RTX A4000","uuid":"GPU-403d1335-aa67-7594-acbd-4b0095da1791","vendor":"NVIDIA","index":1,"core":{"total":6144,"utilization_rate":0.0},"memory":{"total":16789602304,"utilization_rate":0.024591219763533956,"is_unified_memory":false,"used":4128768,"allocated":null},"temperature":28.0}],"swap":{"total":47940997120,"utilization_rate":8.10147000964172,"used":3883925504},"filesystem":[{"name":"","mount_point":"/","mount_from":"overlay","total":2000398934016,"used":1389720170496,"free":610678763520,"available":604043972608}],"os":{"name":"Ubuntu","version":"22.04.4 LTS (Jammy Jellyfish)"},"kernel":{"name":"Linux","release":"6.10.6-200.fc40.x86_64","version":"#1 SMP PREEMPT_DYNAMIC Mon Aug 19 14:09:30 UTC 2024","architecture":""},"uptime":{"uptime":602417990.0,"boot_time":"2024-09-08T21:15* Connection #0 to host localhost left intact
:27.340+0000"}},"id":1,"created_at":"2024-08-06T23:22:36Z","updated_at":"2024-09-01T21:55:09Z"}],"pagination":{"page":1,"perPage":100,"total":1,"totalPage":1}}

Then, when I add a model (ollama, name: llama3.1:8b-instruct-q6_K, model: llama3.1:8b-instruct-q6_K):
2024-09-01T22:00:31+00:00 - gpustack.server.controllers - DEBUG - Created model instance for model llama3.1:8b-instruct-q6_K

and it sits stuck in pending:

This occurs even if I join other workers to the cluster, e.g:

Note that the gpustack version command doesn't seem to work with the default docker image:
gpustack version
0.0.0 (fb70b0e)

but looking at the commit history on the project - this is the latest commit.

It prints 0.0.0 because it's using main(dev) tag. We've updated it to the latest tag to avoid confusion.
We tried to reproduce, but no luck. Is it possible to install without docker in your environment? I’m just curious if the installation method might have anything to do with it.

Yeah sure thing, I’ll give it a crack shortly tonight or tomorrow and report back.
Does the model deployment try to connect to a loopback address (that’s configurable), or does it use a socket / internal call?

Components like scheduler/controllers use internal calls.
The embedded worker that starts with the server connects to the loopback address.
Other workers connects to the provided server address on start.

I was able to get it working by disabling the embedded worker and explicitly providing the server URL, e.g:
...
services:
  &name gpustack:
    container_name: *name
    hostname: *name
    profiles:
      - *name
    image: gpustack/gpustack:main
    ipc: host
    command: --debug --host <IP_ADDRESS> --data-dir /var/lib/gpustack --token mytokenhere --disable-worker
    ports:
      - 80
    volumes:
      - /mnt/llm/gpustack/data:/var/lib/gpustack
    networks:
      - traefik-servicenet

  &name gpustack-worker:
    <<: [*restart, *gpu]
    container_name: *name
    hostname: *name
    profiles:
      - gpustack
    image: gpustack/gpustack:main
    command: --server-url http://gpustack:80 --token mytokenhere
    links:
      - gpustack
    ports:
      - 80
    volumes:
      - /mnt/llm/gpustack/worker-data:/var/lib/gpustack
    networks:
      - traefik-servicenet

I suspect something in the app is trying to connect to or announce itself on one of the container network addresses rather than <IP_ADDRESS>.

something in the app
Is this referring to GPUStack or other services in the compose?
Could you provide more information about the Docker network so we can check if there's anything that needs to be resolved on our side?

Sorry I was referring to within GPUStack, I was thinking perhaps there's some call to http://localhost rather than http://<IP_ADDRESS> which can behave a little differently in containers.

It explicitly uses the loopback address(https://github.com/gpustack/gpustack/blob/4d42dce2fd12b88ac80a33001c210687d53ea8bc/gpustack/cmd/start.py#L203). I tried modifying the hosts file so that localhost resolves to something else, and it works normally, so the resolution of localhost might not be the cause.

We identified and fixed issue #284 in version 0.2.1. Hopefully, this fix should also resolve the issue you're experiencing. If it doesn't, the logs should provide more detailed information.
Feel free to provide feedback or reopen the issue if you have any further questions or encounter any problems.

Thank you for looking into this Lawrence, I really appreciate it.


Issue 1101

User model search scope method concatenating firstname and surname to look for a term. DMPRoadmap/roadmap#1101

Note it has been fixed too the visibility of list of users for when a super admin reaches that page.

that policy issue is probably causing some of the remaining issues referenced in #761 and #794. I'm going to work on those 2 today so I can update it if necessary


Why ist par_iter slower than iter?

I'm sorry for bothering but I just tried to take the example of the README and expected that rayon is at least as fast as the std iterator.
What's happening here?
pub fn par_norm(v: &[f32]) -> f32 {
    let sum : f32 = v.par_iter().map(|&x|x*x).sum();
    sum.sqrt()
}

pub fn norm(v: &[f32]) -> f32 {
    let sum : f32 = v.iter().map(|&x|x*x).sum();
    sum.sqrt()
}

#[bench]
fn bench_par_norm(b: &mut Bencher) {
    let data = vec![20.0;2000];
    b.iter(|| {
        par_norm(&data)
    });
}
#[bench]
fn bench_norm_slice(b: &mut Bencher) {
    let data = vec![20.0;2000];
    b.iter(|| {
        norm(&data)
    });
}

test bench_norm_slice ... bench:       1,606 ns/iter (+/- 19)
test bench_par_norm   ... bench:      21,006 ns/iter (+/- 829)

It's a matter of scale. Running in the threadpool does come with some overhead cost to performance, which I would informally estimate to have O(log n) complexity. You're only processing 2000 items with a very simple computation, so the serial iterator is already very fast, making parallelism too costly.
Here's what I get for increasing item counts:
# 2,000 items
test bench_norm_slice ... bench:       1,525 ns/iter (+/- 29)
test bench_par_norm   ... bench:      15,689 ns/iter (+/- 3,219)

# 20,000
test bench_norm_slice ... bench:      15,560 ns/iter (+/- 637)
test bench_par_norm   ... bench:      26,333 ns/iter (+/- 6,583)

# 200,000
test bench_norm_slice ... bench:     154,895 ns/iter (+/- 4,720)
test bench_par_norm   ... bench:      78,363 ns/iter (+/- 21,777)

# 2,000,000
test bench_norm_slice ... bench:   1,566,346 ns/iter (+/- 48,424)
test bench_par_norm   ... bench:     483,235 ns/iter (+/- 85,783)

# 20,000,000
test bench_norm_slice ... bench:  16,056,856 ns/iter (+/- 995,502)
test bench_par_norm   ... bench:   4,443,414 ns/iter (+/- 791,691)

I increased 10 times as many items each time, and the serial iterator showed the same 10x linear increase in time. The parallel iterator is clearly better than serial at 200k, and still didn't show linear growth until 2M->20M.

Thanks for your reply!
So the readme shouldn't say the following:

Parallel iterators take care of deciding how to divide your data into tasks;
it will dynamically adapt for maximum performance.

...because that doesn't seem to be the case!?

@flosse The most performant thing to do here would beto just run the iterator on a single core, but rayon has no way of knowing that the map content is so trivial. Even if you run the same loop and parallelize it using OpenMP in C/C++, you will not get a speedup. For parallelization to make sense, you need a large enough iterator and/or higher computational complexity per each item in the iterator. Parallelizing this loop simply makes no sense and that is also the reason why you don't see any speedup :slightly_smiling_face:

Right -- rayon does dynamically adapt how it is dividing tasks, but it starts with the assumption that it makes sense to run in parallel in the first place. As a library, we can't tell how complicated your closure is, only that it has the right type signature.

Thank you guys!
My questions is answered now :)
Thanks for your time!


(enforce-pact-version ) doesn't work with two digits minor version

Issue description
When then minor version of Pact has 2 digits like currently, (enforce-pact-version ) doesn't parse it correctly
Steps to reproduce
Example 1:
pact> (pact-version)
"4.10"
pact> (enforce-pact-version "4.9")
<interactive>:0:0:Error: Invalid pact version 4.10, minimum allowed: 4.9
 at <interactive>:0:0: (enforce-pact-version "4.9")

Expected Behavior
(enforce-pact-version min-version) is supposed to accept versions higher or equal than min-version.
I would expect the Example 1 to NOT throw an error
EDIT: Removed the second example

Thanks @CryptoPascal31 - we'll get to this asap


⌘+C for Clear is a bit weird

When you open Maccy with ⌘+⇧+C and then press ⌘+C by accident, you clear all your work. Maybe give an option to edit or disable this (or even a confirm dialog/Undo last action).

What if I just drop the keybinding altogether?

I mean, actually if somebody is a keyboard-heavy user you can also just tap ↓ down and then Enter
So yes, maybe this wouldn't be to bad.
Only making the global shortcut configurable would be nice

Please try 0.3.0.
It also allows to configure global hotkey.

Awesome! Finally can use this instead of ClipMenu. The search feature is awesome.


Command bug

After downloading requirements.txt i get a big red command error
After downloading this it ask to download the pgoapi but that never seems to work
https://puu.sh/qmtJV/13f0d4fc58.png

Maybe this can help for pgoapi issue link

What was the big red command error?

that's because you can't install pgoapi directly from pip, it's not an actual package. you have to install it from requirements.txt which uses pip magic to turn the pgoapi github repo into a python package that pip can install.

I'm not an expert on this, but it seems to me like pgoapi is not the problem here. The python script is complaining that it cannot find the module to load in pgoapi.
I suspect this is because your setup failed while installing requirements.txt and this might have messed up the imports.

@dkim1000, they issued the command sudo pip install pgoapi, which you just can't do. but you're right, they need to resolve whatever issues were happening when they run sudo pip install -r requirements.txt

@reddivision @dkim1000
i tryed using 'sudo pip install -r requirements.txt' but still the same error:
Command "/Users/Jonas/Desktop/pokemonGo/PokemonGo-Bot/bin/python -u -c "import setuptools, tokenize;__file__='/private/tmp/pip-build-cjFkvR/pycryptodomex/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" install --record /tmp/pip-YkhnGJ-record/install-record.txt --single-version-externally-managed --compile --install-headers /Users/Jonas/Desktop/pokemonGo/PokemonGo-Bot/bin/../include/site/python2.7/pycryptodomex" failed with error code 1 in /private/tmp/pip-build-cjFkvR/pycryptodomex/
So there is notting what i can do to fix this? The owner have to fix it?

@pokecunt could you just stop spamming this f**ing img?! Notifications keep blinking

do you have only python2 installed?

i only have python 2.7 installed yes. Do i also need to have 3.5?

no I was thinking maybe python3 was interfering because that happens if you have both installed.

Try to run setup.py

I have the same problem. Running sudo pip install -r requirements.txt don't work well. I installed some of the requirements with pip install name but that dont work with the pgoapi.
http://imgur.com/a/2yQyv (Screenshot when the installing fails. I use python 2.7 and linux). I hope someone can help.

sudo pip install --upgrade -r requirements.txt

sudo apt-get install python-dev
pip install -r requirements.txt

@Dreanmer dont do this in your os, do this in virtualenv


Add VI for linear regression

@sourish-cmi @Nandini-Jaiswal
I've added VI for linear regression.

I split BayesianRegression into BayesianRegressionMCMC and BayesianRegressionVI structs.
I pass a boolean use_vi to determine whether to use MCMC or VI.
For VI, I store the distribution, and sample it during prediction. (This technically makes the prediction random each time, I could change it to sample during creation and store the sampled values instead dist, if you prefer that.)
I made seperate structs for BayesianRegressionMCMC and BayesianRegressionVI to store the chain and dist respectively. However we could just convert both to a matrix of samples (we will sample the dist for VI) and just have a single BayesianRegression struct which stores the matrix of samples. We would lose the info about the chain and dist though, I'm not sure if its important.

@ShouvikGhosh2048  Can you please add tests and check why the doc test is failing? Do you want me to provide some test!

I've added tests for VI, those are passing. The doc test are failing since I haven't made the changes for VI, I'll make them once we decide on the API.

Okay - can we meet over Google Meet?

I've made the changes we discussed and updated the docs.
I ended up removing use_vi and instead I use a BayesianAlgorithm struct defined here:
https://github.com/xKDR/CRRao.jl/pull/154/files#diff-b989ddbe26d5a6bf4e0714d4bc169310463a8e8bbcadaf6d5903d5fe34f6fc02R395-R432
I went with this since the algorithms have different parameters (MCMC needs the number of iterations and the number of initial observations we discard, and VI needs the number of samples, number of VI iterations and number of samples per VI step).
So now we do:
model = fit(@formula(MPG ~ HP + WT + Gear), mtcars, LinearRegression(), prior) - MCMC
model = fit(@formula(MPG ~ HP + WT + Gear), mtcars, LinearRegression(), prior, VI()) - VI
If the API is fine, this PR is ready to merge.

@sourish-cmi
I've made updates to the API and pushed the changes to my PR. If you are fine with integrating VI, then you can take a look at PR and merge it if you're fine with it, and then me and @Nandini-Jaiswal can continue implementing VI for the rest of the regression types, while you take a look at Turings VI.

I am fine with it. Merging the PR.


postcss-mixins compatibility problem after update

Thanks for the recent updates! After getting a project switched over to the latest builds, I'm seeing a new problem while using mixins. It looks like any rules declared in a block after a mixin is called don't get processed. Here's a quick example - can create a demo repo if necessary.
/* input */
@define-mixin test {
  color: blue;
}

body {
  @mixin test;

  background: red;
  font-size: 1rem;

  @media (min-width: 64em) {
    background: green;
  }

  margin: 0;
}

/* output */
body {
  color: blue;
}

@media (min-width: 64em) {
  body {
      background: green;
  }
}

The media block comes through fine, and if I move the mixin call down under a rule, then those above it will be in the build as well. I don't think this is a postcss-mixins plugin problem, as that has always worked correctly with postcss-nested.
Let me know if there's any other info I can provide or if a demo repo would help. Thanks!

One additional note that might be helpful - if I disable nesting then the output is the following:
body {
  color: blue;

  background: red;
  font-size: 1rem;

  @media (min-width: 64em) {
    background: green;
  }

  margin: 0;
}

That looks like something postcss-nesting should be able to handle, so it almost seems like the plugins aren't running in the correct order, even though I have mixins running before preset-env.

@mdmoreau can you try enabling allowDeclarationsAfterNestedRules as explained here?

@Antonio-Laguna ahh that solved it - thanks! Looked at the changelog after upgrading, but didn't think to re-read the docs.

@mdmoreau the changelog isn't updated yet. Let me think about this and see what we can do about this. Thanks for testing that!
@romainmenke thoughts on this? Kind of think given the ecosystem and how things are normally on PostCSS which can create things that unfurl, we might want to consider enabling this by default unless it's false. Let's discuss this.

@Antonio-Laguna one thing that seems a little strange to me - and maybe I'm misunderstanding how things work - but if mixins is processed first and only has a rule and not a nested selector, shouldn't that still be processed without enabling the new option? In my example it seems like the margin: 0; should get removed, but the mixin declaring color: blue; should get grouped with the background and font-size rules.

@mdmoreau the thing is that PostCSS8 makes the order of execution no longer important which is interesting and opens lots of possibilities to things to be iteratively taken care of by plugins. However, it leads to interesting unexpected behaviour.
Probably nesting is just bailing after completing its thing.

I think we best tighten up ensureCorrectMixingOfNestingRulesAndDeclarations
Currently we enforce the spec very strictly leaving little room for other plugins.

@Antonio-Laguna okay cool - appreciate the explanation. Thanks for the quick responses and all the work on the recent updates!

I think we best tighten up ensureCorrectMixingOfNestingRulesAndDeclarations Currently we enforce the spec very strictly leaving little room for other plugins.

Agree. Let's discuss a plan of action.

I hope this can be done in a patch release.
Ideally we would ignore any unexpected css.
This is either totally new css and a future addition or it is something handled by another plugin.
When such a plugin does it's work and this results in nested css Postcss8 should re-run the visitor for the affected nodes.

Thought about it some more and I suggest we remove ensureCorrectMixingOfNestingRulesAndDeclarations and allowDeclarationsAfterNestedRules entirely.
What I was trying to solve was people having issues in newer browsers once nesting exists for a while.
Users of postcss-preset-env who split bundles for older vs newer browsers could would have CSS that works when processed by this plugin but seems broken in newer browsers.
That issue however is maybe best left too linters or dedicated test suites.
Since the issues would surface in newer browsers it is also easier to debug and pinpoint than the other way around.
@Antonio-Laguna If you agree I will prepare a pull request with the needed removals.
This would also just be a patch release.

@romainmenke ok, let's do that then!


Implement excludeModules

What does this PR do?
This plugin "exports" two "sub modules":

projext-plugin-webpack/express
projext-plugin-webpack/jimpex

Which are not on the project package.json, so if the user bundles a Node target with any of those "sub modules", wepack would try to process the whole plugin and put it on the bundle.... not cool.
So, the first part of this PR defines those paths as "default externals" and add them as exceptions while setting the external dependencies.
And, while changing the implementation of the external dependencies, this PR also implements the new excludeModules setting for Node targets (homer0/projext#17)... which is basically another list of exceptions.
How should it be tested manually?
yarn test
# or
npm test

Pull Request Test Coverage Report for Build 54

6 of 6 (100.0%) changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 52:
0.0%

Covered Lines:
427

Relevant Lines:
427

💛  - Coveralls


allow key overrides

13:27:01  in ./helmfile.yaml: in .helmfiles[0]: in helm/account/helmfile.yaml: failed to read helmfile.yaml: reading document at index 1: yaml: unmarshal errors:
13:27:01    line 167: key "sync" already set in map

templates:
  default: &default
    labels:
      sync: everyday
charts:
  - name: test
    <<: *default
    chart: test
    labels:
      sync: thur

In yaml I would typically expect this to not error out and to allow the sync key to be overwritten and set to thur instead of failing.

@sstarcher Good catch! Seems like we need to avoid the go-yaml strict mode or upgrade to go-yaml v3 https://github.com/go-yaml/yaml/issues/410#issuecomment-475267893


ENH: Set SubtractMean parameter "true" (instead of "false") by default

Let the metrics other than AdvancedNormalizedCorrelation also have default value "true" for their SubtractMean parameter.
Specifically, this affects the following metrics:

PCAMetric
PCAMetric2
SumOfPairwiseCorrelationCoefficients
VarianceOverLastDimensionImageMetric.

Discussed with Marius Staring (@mstaring) and Stefan Klein (@stefanklein).

This pull request is somewhat related to pull request #1103 (but this one is not about AdvancedNormalizedCorrelation!)

Might conflict with https://github.com/SuperElastix/elastix/tree/Remove-old-itkPCAMetric-files commit 8916b5f6bfc4ff083665f3526502b9f839f01091 (STYLE: Remove "itkPCAMetric.h" and "itkPCAMetric.hxx")

Note: Specifically for these metrics, this parameter is intended to be renamed to something like "GroupwiseImplicitReferenceFrame", as was discussed with @mstaring and @stefanklein

I just noticed that all of the metrics affected by this pull request also have a m_TransformIsStackTransform data member. "TransformIsStackTransform" is true if-and-only-if we are doing group-wise registration, right? @stefanklein ?

Hi Niels,
Good question. It’s not entirely true. When using the (cyclic) 4D bspline deformable transform it is not a stack transform, but still a groupwise registration.
And on the other hand, I can imagine cases where we are registering a stack of slices to another stack of slices using a stack transform, which do not follow the scheme of a typical groupwise registration (although you could call it a groupwise registration, depending on definition).
Does that help? 😊
Best, Stefan
From: Niels Dekker @.>
Sent: Monday, June 24, 2024 5:28 PM
To: SuperElastix/elastix @.>
Cc: Stefan Klein @.>; Mention @.>
Subject: Re: [SuperElastix/elastix] ENH: Set SubtractMean parameter "true" (instead of "false") by default (PR #1104)
Waarschuwing: Deze e-mail is afkomstig van buiten de organisatie. Klik niet op links en open geen bijlagen, tenzij u de afzender herkent en weet dat de inhoud veilig is.
Caution: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.
I just noticed that all of the metrics affected by this pull request also have a m_TransformIsStackTransform data member. "TransformIsStackTransform" is true if-and-only-if we are doing group-wise registration, right? @stefankleinhttps://github.com/stefanklein ?
—
Reply to this email directly, view it on GitHubhttps://github.com/SuperElastix/elastix/pull/1104#issuecomment-2186845240, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAF2LNLIFUZZWQMCCVMJR5DZJA3GZAVCNFSM6AAAAABGRM2APCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBWHA2DKMRUGA.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

Thanks @stefanklein I'm asking, because it appears that "SubtractMean" is relevant for both stack transforms and "non-stack" transforms:
https://github.com/SuperElastix/elastix/blob/de07d4ce0610a48b62379a8a084e90073cc306a3/Components/Metrics/PCAMetric/itkPCAMetric_F_multithreaded.hxx#L527-L531 basically says:
 if (this->m_SubtractMean)
   if (!this->m_TransformIsStackTransform)
     ...do the subtract-mean one way...
   else
     ...do the subtract-mean another way...

So is this SubtractMean option only for groupwise registrations, or is SubtractMean also for a regular registrations?

Hi Niels,
SubtractMean indeed only applies to groupwise registration. But Groupwise registration can be done with a stacktransform, or with a nD+time B-spline transform (see paper by Coert Metz for the method description),  and for the latter case there is the scenario '...do the subtract-mean one way...' .
Best, Stefan
From: Niels Dekker @.>
Sent: Monday, June 24, 2024 6:21 PM
To: SuperElastix/elastix @.>
Cc: Stefan Klein @.>; Mention @.>
Subject: Re: [SuperElastix/elastix] ENH: Set SubtractMean parameter "true" (instead of "false") by default (PR #1104)
Waarschuwing: Deze e-mail is afkomstig van buiten de organisatie. Klik niet op links en open geen bijlagen, tenzij u de afzender herkent en weet dat de inhoud veilig is.
Caution: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.
Thanks @stefankleinhttps://github.com/stefanklein I'm asking, because it appears that "SubtractMean" is relevant for both stack transforms and "non-stack" transforms:
https://github.com/SuperElastix/elastix/blob/de07d4ce0610a48b62379a8a084e90073cc306a3/Components/Metrics/PCAMetric/itkPCAMetric_F_multithreaded.hxx#L527-L531 basically says:
if (this->m_SubtractMean)
if (!this->m_TransformIsStackTransform)
 ...do the subtract-mean one way...

else
 ...do the subtract-mean another way...

So is this SubtractMean option only for groupwise registrations, or is SubtractMean also for a regular registrations?

Reply to this email directly, view it on GitHubhttps://github.com/SuperElastix/elastix/pull/1104#issuecomment-2186952399, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAF2LNOPFUSHMG7Z7Q4DWC3ZJBBQDAVCNFSM6AAAAABGRM2APCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOBWHE2TEMZZHE.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

This pull request (1104) is superseded by pull request #1177


It seems that the Panopticapi.utils module is not found, but it can be run. Will this affect the results?

try:
from panopticapi.utils import rgb2id
except ImportError:
rgb2id = None
It seems that the Panopticapi.utils module is not found, but it can be run. Will this affect the results?

This code is in dataset/pipeline/loading.py


Test Travis with proper LLVM version

From a quick look, Travis log reveals the following problem:
E: Failed to fetch http://apt.llvm.org/xenial/dists/llvm-toolchain-xenial/main/binary-i386/Packages.gz  Hash Sum mismatch
E: Some index files failed to download. They have been ignored, or old ones used instead.

Now it just says E: Unable to locate package llvm-9.0*. I think we need to add the following sources to make them work:
      - sourceline: 'deb http://apt.llvm.org/xenial/ llvm-toolchain-xenial-8 main'
        key_url: 'https://apt.llvm.org/llvm-snapshot.gpg.key'
      - sourceline: 'deb http://apt.llvm.org/xenial/ llvm-toolchain-xenial-9 main'
        key_url: 'https://apt.llvm.org/llvm-snapshot.gpg.key'


Update watermark to new logo

Resolves AB#789

I'm not sure why EuclideonLogo.png was deleted? I thought there was still a case where it was used?

Not that I can see, searching brings up nothing, I think it was vestigial before and the actual data used was stored in internal textures data


Data Analysis with Python / Jupyter Notebooks / Twitter Cheat Sheet Broken Link

Describe your problem and how to reproduce it:
In Data Analysis with Python there are lessons about Jupyter Notebooks. And on the bottom side of the videos there are some useful links. The last one is a link to Twitter posts that contain various tips, commands etc for Jupyter.
Well, that link doesn’t work anymore (I think the lesson was made in like March 2020).
Add a Link to the page with the problem:
https://www.freecodecamp.org/learn/data-analysis-with-python/data-analysis-with-python-course/how-to-use-jupyter-notebooks-intro
https://www.freecodecamp.org/learn/data-analysis-with-python/data-analysis-with-python-course/jupyter-notebooks-cells
https://www.freecodecamp.org/learn/data-analysis-with-python/data-analysis-with-python-course/jupyter-notebooks-importing-and-exporting-data
and the supposed link: https://twitter.com/rmotr_com/status/1122176794696847361
Recommended fix, suggestions (how would you update it?):
If the cheat sheet still exists, then we can update the link. Else, it should be deleted from the lessons.
If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):

It seems like the tweet is gone. When I check out the user, it looks like they do not have any tweets... They might have deleted their contents

We should remove these links, @scissorsneedfoodtoo or @beaucarnes would either of you be able to create a news article that could be used here instead?

I can take care of removing the links, but should I wait until a new link is done, or should I get a pull request?

... but should I wait until a new link is done, or should I get a pull request?

I would say, let's wait for either Beau or Kris to respond back. Thanks for your interest in making a PR, have been looking at some of your recent contributions. Thanks for your time and keep up the good work!

Hey @JoshuaPelealu, Beau says you can proceed with removing that link all together for now. We will add a new link in a later PR.
Thanks again for working on this.

Should the PR close this issue, or just let the issue stand until the later PR for the new links?

Thanks again for your quick fix, @JoshuaPelealu, and for reporting this @trifonas-hash.
I talked this over with @beaucarnes and we decided that it's not necessary to replace the broken link. The current links should be enough for learners to get started with Colaboratory. Also, the broken link was probably a video walkthrough of the content in the second link in those challenges, How to open Notebooks from GitHub using Google Colab.


Add GitHub Issue Templates

Create and add GitHub issue templates to improve the overall quality and consistency of our issue tracking.
Goals
The following issue templates should be added:

[x] Bug Report Template: This template should include fields for describing the bug, steps to reproduce the issue, and any relevant environment information.
[ ] Feature Request Template: This template should include fields for describing the requested feature and its potential benefits, as well as any proposed implementation details.
[x] Pull Request Template: This template should include fields for describing the changes made in the pull request, as well as any necessary testing or deployment steps.

After weighing observations from Working in Public: The Making and Maintenance of Open Source Software its been decided that we don't want this barrier to entry


How to print thailand language?

I have used this plugin to print text in thai language and added Charset UTF-8, but not working. it work only English. Help me please

Is this solved? I also need to print arabic but I am not able.

check first what is the charset of your printer been set to, if you are not sure then trigger a self check which all current setting will automatically printed out.