Testing with NA12878-exome-methodcmp.yaml

I ran the pipeline with example exome data. (in bcbio)
When I finished all job with NA12878-exome-methodcmp.yaml config file, I checked the last step and found the error in Rscript.
I think this issue cause some column missing.
My command is..
$ Rscript NA12878-exome-eval/final/project_2016-03-23/report/qc-coverage-report-run.R
The error line is..
(NA12878-exome-eval/work/report/report-ready.Rmd)
# Adjust some of the text information for formatting purposes
#qc_table <- qc[, metrics]
qc_table <- qc

Error message

Quitting from lines 54-159 (report-ready.Rmd)
Error in appendFormatter(x$options$rowCallback, columns, colnames, rownames,  :
You specified the columns: Average_insert_size, but the column names of the data are %GC, Duplicates, Duplicates_pct, Mapped_reads, Mapped_reads_pct, Median_insert_size, Name, Quality_format, Sequence_length, Sequences_flagged_as_poor_quality, Total_reads, Variations_total, mapped, offtarget, offtarget_pct, avg_coverage_per_region, sample
Calls: render ...  -> formatStyle -> formatColumns -> appendFormatter

Execution halted

Hyunmin;
Sorry about the error messages. Practically, bcbio should recover from these and finish cleanly -- the build of the report is a convenience but is currently optional. So if you don't need the report you can use the rest of the outputs without any problems.
If you need the report, @lpantano might be able to help more with fixing that. I believe there were some problems with the last release (0.9.6) which are fixed in development, so updating with:
bcbio_nextgen.py upgrade -u development

could fix it if you're running the release version. Hope this helps.

Hi @hmkim,
sorry to be late. And sorry about the issue, some times it gives an error at that point depending on the data.
We want that line to filter columns because other wise it will create a big table. Could you tell me what column are you missing from the following?
metrics = c("sample", "Total_reads" ,"Mapped_reads_pct", "Duplicates_pct",
            "offtarget", "%GC", "Sequence_length", "Average_insert_size")

I moved the stats to use samtools and maybe have some conflicts with the stable version and the last report version. I will try to fix the report to try to avoid this error.

I check this file.
NA12878-exome-eval/work/report/metrics/metrics.tsv
This file have an columns in below.
%GC     Duplicates      Duplicates_pct  Mapped_reads    Mapped_reads_pct        Median_insert_size      Name    Quality_format  Sequence_length Sequences_flagged_as_poor_quality       Total_reads     Variations_total        mapped  offtarget       offtarget_pct   avg_coverage_per_region sample
**missing 'Average_insert_size' **
Thanks in advance!

Hi,
yep, sorry, that was the changed. you can update bcbio and run again (you will need to remove the reports as well) to get the new tables. Or just change that by "Medium_insert_size".
As well I tried to update the package that generates the report to try to be compatible with both versions, so if you update the tools, maybe this error disappears.
This report can have some problems with some analysis and I am trying to handle all the situation. So if you try to run with the new version or in your future analysis, let me know what you find.
As well, I found some problems when there is only one sample, I am trying to figure out how to handle that.
thanks!

Hi,
we have turned off the render of the report until we have it in a better shape.
This report is really useful mainly when you turn on coverage option in your YAML file. If at some point you need it, just try to render manually the report if we couldn't get a stable version at that point.
I pushed a new version and fixed some of the problems. So feel free to try, but works better with many samples. Please, open a new issue for the report problem if this persists.
Thanks a lot!

Thanks for your support.
I'll try to run with some example data, and I'll report it if I get the error.


Added Drop Down With more Help Links

Related Issue
[Cite any related issue(s) this pull request addresses. If none, simply state "None”]
Added Drop Down With more Help Links
Description
[Please include a brief description of the changes or features added]
Added Drop Down With more Help Links
Type of PR

[ ] Bug fix
[X] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes]

Checklist:

[x] I have performed a self-review of my code
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

Additional context:
[Include any additional information or context that might be helpful for reviewers.]

@arghadipmanna101 Please review the pull Request


How to extend a given schema

using valibot version 0.42.1
How can I conditionally extend a schema that might have v.nullish?
I assumed I could pass the object entries as the first argument to v.pipe and it would retain any optionality it had before.
For example let's say baseSchema is
const baseSchema = v.object({
  foo: v.nullish(v.number())
})

and somewhere else I want to conditionally check that foo is an integer
v.pipe(baseSchema.foo, v.integer())

1.) if baseSchema is strongly typed, I'll get an error on v.integer() because it correctly is telling me that foo could be undefined. What's your suggested approach for adding validation only when the value is not nullish?
2.) if baseSchema is loosely typed, valibot will take a nullish value and throw Invalid integer: Received undefined. I hoped that the baseSchema.foo would retain it's optionality

looks like my issue was outside of valibot

Here is an example that works:
import * as v from 'valibot';

const Schema1 = v.object({
  foo: v.number(),
  bar: v.pipe(v.number(), v.maxValue(300)),
  other: v.string(),
});

const Schema2 = v.object({
  ...Schema1.entries,
  foo: v.pipe(Schema1.entries.foo, v.minValue(250)),
  bar: v.pipe(Schema1.entries.bar, v.minValue(250)),
});

Here is an example that works:
import * as v from 'valibot';

const Schema1 = v.object({
  foo: v.number(),
  bar: v.pipe(v.number(), v.maxValue(300)),
  other: v.string(),
});

const Schema2 = v.object({
  ...Schema1.entries,
  foo: v.pipe(Schema1.entries.foo, v.minValue(250)),
  bar: v.pipe(Schema1.entries.bar, v.minValue(250)),
});

@fabian-hiller sorry, I just finished editing my question, you were too fast

Here are two solutions:
import * as v from 'valibot';

const BaseSchema = v.object({
  foo: v.nullish(v.number()),
});

const OtherSchema1 = v.object({
  foo: v.nullish(v.pipe(v.unwrap(BaseSchema.entries.foo), v.integer())),
});

const OtherSchema2 = v.object({
  foo: v.pipe(
    BaseSchema.entries.foo,
    v.check((input) => input === null || Number.isInteger(input))
  ),
});


Thaumcraft Conduits

seeing as Ender IO adds some conduits that add compatibility with other mods, how about adding conduits for Thaumcraft support aswell? these conduits would act like the pipes from Thaumcraft itself and will be able to transfer essentia through the conduits.
with these conduits you could perhaps also have the option of importing or exporting from the sides of jars, instead of only from the top as it is now with the thaumcraft pipes.

The problem with this is that it's pretty much impossible to properly implement Thaumcraft's suction system and displaying issues the way Thaumcraft does...

This is also a duplicate of #2167

#2167


Cutting

Thanks for that great project.
How can I cut the soft body ? You mentioned there are cutting supports.. but don't know how to achieve/simulate that

Hi, sorry for the delayed reply. I actually have not implemented the cutting in this repository, unfortunately, I should remove it from the README, I apologize for the misleading. Unfortunately, most of the things I currently do are private, but I hope to make some implementations public in some near future.


release notes for android client sdk updated

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[X] Maintenance/Chore

Release Notes for Android SDK updated.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6569fcf14cdb7d04ceeec7aa--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://656efb154e405600a123a8e3--harness-developer.netlify.app


fix: update to eye v10.7.14

fix: update to eye v10.7.14

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

npm package (@latest dist-tag)
GitHub release
v16.4.16

Your semantic-release bot :package::rocket:


Invalid upstream response (403)

Hello, I quite like this feature for the README file and implemented it like last week, but it seems to not work at the moment.
When clicking on my embed in my README I get: Invalid upstream response (403)
Has the requests quota been reached for Deno?
Resulting in the typograssy embed not being able to connect anymore?

Yes, the quote has been exceeded, evident on the homepage
It stopped working for me as well.

Seems to be working again 👍


Adding Newesis Srl

Hi @edoardocostantinidev Newesis was born in January 2019 originally as a full remote company, not just a full remote but a real "Smart Working" company (meaning no fixed location but also no fixed working time). Sorry if this is not state enough clearly on the web site but we consider it such a default, at the base of the creation of the company, that we didn't felt the need to highlight it too much. You can find it in the statement "Freedom" in the Company Culture in this page "https://www.newesis.com/about" . Let me know if this is enough or if you suggest to make it more explicit in the career page we can edit it (happy to learn from feedbacks).


elchecking/example: add ignores for EV_PLATFORM_CONFIG_FLAGS

These are generated by edk2 when used with QEMU, but we do not have a reference for them.
Fixes: #1393

/packit retest-failed


Change version to 2.15.5

No need to downgrade. https://github.com/luxonis/depthai/issues/712#issuecomment-1152538918


Launch jupyter notebooks on binder

Just adds a badge in order to launch jupyter notebooks on binder

Thanks @pathcl for the pointer, good tool to know about!

You're welcome ! glad you merged it
On Sun, Jun 3, 2018 at 06:41 Bob Belderbos<EMAIL_ADDRESS>wrote:

Thanks @pathcl https://github.com/pathcl for the pointer, good tool to
know about!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/talkpython/100daysofcode-with-python-course/pull/14#issuecomment-394153054,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADmCQPh7poSgiW6n8vH6zyVJjtq_clGgks5t471rgaJpZM4UVeWJ
.


Fix double writes

Thank you for your contribution!
We appreciate the time and effort you've put into this pull request.
To help us review it efficiently, please ensure you've gone through the following checklist:
Submission Checklist 📝

[x] I have updated existing examples or added new ones (if applicable).
[x] I have used cargo xtask fmt-packages command to ensure that all changed code is formatted correctly.
[x] My changes were added to the CHANGELOG.md in the proper section.
[x] I have added necessary changes to user code to the Migration Guide.
[x] My changes are in accordance to the esp-rs API guidelines

Extra:

[x] I have read the CONTRIBUTING.md guide and followed its instructions.

Pull Request Details 📖
Description
A rather silly bug caused by a missing return.

To be fair I'd probably just reimplement read/write in terms of transfer, that way we can place a single function into IRAM as opposed to two or three.


README: Add requirements and a 'cxx export' hint

cxx uses scons and scons needs python.
Make sure, that such essential requirements are listed.
"cxx export" is very useful for new users.
Make sure, it is listed in the examples.
Please update also the "About" section on the project landing page.
--
Regards ... Detlef

Thanks! 🙂 Merged.


Add metrics labels regex rules

Closes https://github.com/dapr/dapr/issues/4133.

Docs tracked here: https://github.com/dapr/docs/issues/3046.


GoToDefinition doesn't scroll to the top of the blue definition UI

VSCode Version: 1.9.1 and 1.10.0-insiders
OS Version: Win10

Steps to Reproduce:

I'm using a high DPI laptop (not sure if that's required).
Cause the Problems/etc. pane to appear and split the UI 50%.
Use Go to (Peek) Definition using any language that supports it.

Bug: See the screenshot. The blue "definition" box is not scrolled up to the top, which is really annoying when the document is large and it's hard to scroll up by a small amount to see the definition options on the right. The repro seems 100% when the UI is sufficiently constrained. I need to see the top because that's where the definition list is (when things are defined in multiple places).
Bug#2: The lack of the top/bottom arrow buttons makes it hard to scroll up/down by single lines when the document is large, and using the keyboard to go up lines doesn't work if the keyboard cursor is stuck in the blue box.

@sean-mcmanus Do you use code-lenses? Like git version lenses or reference lenses?

I am not using any code lens. It repros without any extensions. It repros on a non-high DPI machine too. The only repro requirement is to have "not enough" vertical height in the document view. It appears that code needs to be added to ensure that the document scrollbar is set to make the line with the blue cursor visible (i.e. top of the blue peek window).

This got fixed with #23310  and by a new constraint the peek view always fits into the view port


Feedback: CI/CD

To do: add ci-cd file

Don't need to have paper.md for the current project - closing this issue


Debugger breaks inside ptvsd

Test code:
import ptvsd
ptvsd.enable_attach(None)

x = 1
while True:
    x += 1
    print(x)

Breakpoint set on print. After attaching, the stack is:
>	enable_attach in __init__ line 87	Python	Symbols loaded.
 	PythonApplication1 module line 7	Python	Symbols loaded.

where __init__ is ptvsd\__init__.py. The line in question is inside enable_attach. However, the line on which it's actually broken is the one with print. So the breakpoint is hit, which is correct, but the stack is weird.
The real problem here is that __init__.py is not in DONT_DEBUG.

This is a new regression, right?


Panic when calling 'EnableMouse'

Using the version v2.4.0 I'm getting this panic when I'm calling the EnableMouse function with Go 1.18.
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x18 pc=0x107bf9d]

goroutine 1 [running]:
io.WriteString({0x0, 0x0}, {0x112f51c, 0x20})
	io/io.go:314 +0x7d
github.com/gdamore/tcell/v2/terminfo.(*Terminfo).TPuts(0xc000171c00, {0x0, 0x0}, {0x112f51c?, 0x110e7c0?})
	github.com/gdamore/tcell/v2@v2.4.0/terminfo/terminfo.go:646 +0x1f4
github.com/gdamore/tcell/v2.(*tScreen).TPuts(0x0?, {0x112f51c, 0x20})
	github.com/gdamore/tcell/v2@v2.4.0/tscreen.go:768 +0x8e
github.com/gdamore/tcell/v2.(*tScreen).enableMouse(0xc0000126c0, 0x4)
	github.com/gdamore/tcell/v2@v2.4.0/tscreen.go:862 +0x3d
github.com/gdamore/tcell/v2.(*tScreen).EnableMouse(0xc0000126c0, {0x0?, 0xc000056720?, 0x100c5a5?})
	github.com/gdamore/tcell/v2@v2.4.0/tscreen.go:852 +0x96
main.main()
	./main.go:23 +0x10c

Does this reproduce only in Go 1.18?  Can you supply your $TERM and locale information?  Also, does this occur with the very latest bits in master?  There were some fixes for problems in the x/sys/unix package.

So there is another problem with go 1.18 right now (on Darwin).
I don't think tcell is ready for go 1.18 -- it looks like the go people broke third party software again...

Can you try again -- I've made fixes that should fix this...

I'm closing this for lack of feedback.


[bct] Misc fixes

Fix indenting in check_class_instance_attribute_removed_or_renamed
Fix tests to align with latest reporting updates
Fix reporting for module checks (tracking issue to improve: https://github.com/Azure/azure-sdk-for-python/issues/36983)

/check-enforcer evaluate


Remove the WPCA layer before train

Hello, professors! Could you explain in detail how  to remove the WPCA layer before training?
（In the readme.md-->Quick start-->training）

Can you try:
if 'WPCA' in model._modules:
    del model._modules['WPCA']

Closing for now - feel free to reopen if above doesn’t work.

First, I'm glad to get your answer. I already solve the problems. But I get a new prblem and I will discribe it.
In train.py, some keys(line130--epoch, line197--optimizer, line223,224--not_improved,best_score) are not defined in the mapillary_WPCA128.pth pre-model. these pre-models just include 'num_pcs', 'state_dict'. So I can't use pre-models of download files. If conditions permit, can you share the pre-models for adapting the trian.py. I would like to express my sincere gratitude here.

Simply comment out these lines.


How to train the discriminator

Hi, AlamiMejjati. In your paper section 3.2, you mentioned that

you first train the discriminators on full images for 30 epochs, and then switch to masked images once the attention networks AS and AT have developed.

But I check out your code and find that there is only the attention-guided discriminator in the whole training  https://github.com/AlamiMejjati/Unsupervised-Attention-guided-Image-to-Image-Translation/blob/master/model.py
Could you please answer my confusion? Many thanks!

I'm sorry, I  think I understand  how the whole  process. I didn't understand Formula 6 in your paper very well before.


TESTING – DO NOT MERGE – test exit status in github workflow scripts

Just for testing github. DO NOT MERGE.

Closing as out-dated. Re-open if new testing is desired, and then update the branch.


RUSTFLAGS="..." cargo .. should recompile std lib with flags enabled

This is required for rust-san to work properly, and also for enabling some more efficient algorithms in std::simd that require specific target features (SSE4, AVX, NEON, etc.). Adding Iterator::is_sorted would also benefit from SSE4 and AVX. It currently does run-time feature detection but it could avoid that if std was compiled with the appropriate flags.

The current conversation is https://github.com/rust-lang/wg-cargo-std-aware/issues/43
If that conversation is not the right issue, then that is the new place to open an issue.
Closing for now, so as to unify the conversations.


Add end quote to openai example

Without the end quote the syntax highlighting is off and hard to read. Also fixes copying this entire code chunk so user can just fill in the  string

Thanks for the fix!

#sign-off


Charts available on ArtifactHub, but not in the Charts.yaml

Hello!
Today I checked the latest version in ArtifactHub and showed me the version 0.8.4, but when I tried to pull it, I received the error message saying not found. At repo, in Charts.yaml file, the version still 0.6.3. If I pull this version, am I using the latest version?
Thanks!

This may be because you are pulling the chart directly via git. You should still have the latest version doing it that way, just pull the latest from the main branch.
The main reason behind this is because I use CI/CD to package the chart and publish it to a GitHub package via OCI. ArtifactHub pulls the image from GitHub packages which uses the version tag of the release instead of the one in the chart.yaml. Ideally, the GitHub action would commit the correct version to the main branch, but I haven't gotten around to setting that up.


Split repo into two and fixups

Splits the repo into this -src repo and a repo that contains a generated .js file.
Also fixes up most of the non code writing issue in https://github.com/ably-labs/test-observability-action/issues
The publish workflow seems to work but needs a secret added to actually push to the repo.
The test CI has been removed for now as it ran against a fake server won't work with the repo split. A way to test this action needs to be figured out.

I believe you can run actions via a file path. So in our CI check we could probably run npm install then continue as usual.


Can not add robot to homekit

plugin installation and config are probably clean. Homebridge starts without error and also the homematic plugin goes. Can not add robot to homekit. What can this be?

Now the Neato ist available in Honekit but the switches are not really working....step by step :-)

Hey, what did you change to make it work? Others might find that helpful.
Do the switches show loading circles now? Or are they unavailable?


Pre-calculated kdtree

Much faster tree building for large trees at the cost of some RAM.
for pictured tree

original single threaded: 116 minutes
new single threaded: 21 minutes.

In the case of some trees, which have a many points in a single cluster, and that cluster has a large bounding box. (see attached image)
We get a very large outer cloud and hundreds of inner clouds that each must have the min_distance calculated.
In the original implementation the kdtree for the larger cloud is calculated for each comparison.
This pull request pre-computes the kdtrees (along with the other cluster parameters).
This results in significantly faster buildtree times for trees of a similar nature.
potential downsides are:

some clusters may never need the kdtree calculated.
my testing seems to show that the extra overhead is fairly minimal.
this has potential to be worked on at the cost of code complexity and readability.
higher ram usage: 533 vs 396 Mb for the below tree

Hi Tony, this is great. I've also compared this with the previous code using some test data and returned identical results in ~21000% less user time for just a ~35% increase in the maximum resident set size. Def seems a worthwhile trade off when single threaded. Cheers, Andy


Feat(web): Introduce Select component #DS-788

What about optgroup? Can we add it in the example as well or we have not a design for it?


yarn install fails

Trying to install the repo but I'm getting this:
root@ip-192-168-29-194:/opt# cd k6-hardware-benchmark
root@ip-192-168-29-194:/opt/k6-hardware-benchmark# yarn install
00h00m00s 0/0: : ERROR: [Errno 2] No such file or directory: 'install'

It seems that install yarn via npm works:
$ sudo apt remove cmdtest
$ sudo apt remove yarn
$ sudo npm install -g yarn

Thanks for https://stackoverflow.com/a/52534824/999270


nut: 2.8.0 -> 2.8.2

Description of changes
Currently failing build.
       > Makefile.am: installing './INSTALL'
       > Makefile.am: error: required file './NEWS' not found
       > Makefile.am: error: required file './README' not found
       > clients/Makefile.am: installing './depcomp'
       > configure.ac:4981: error: required file 'scripts/augeas/nutupsconf.aug.in' not found
       > configure.ac:4981: error: required file 'scripts/devd/nut-usb.conf.in' not found
       > configure.ac:4873: error: required file 'scripts/systemd/nut-common-tmpfiles.conf.in' not found
       > configure.ac:4981: error: required file 'scripts/udev/nut-usbups.rules.in' not found
       > parallel-tests: installing './test-driver'
       > autoreconf: error: automake failed with exit status: 1

Closes #310868
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@superherointj or @Aleksanaa any help regarding this would be nice :)

It's mostly this https://nixos.org/manual/nixpkgs/unstable/#setup-hook-autoconf


UI Table usage with Variant Manger

Hello togehter,
I'm not sure, if I can't figure it out, or if there is something not working.
I want to build a protoype similar to the Demo z2ui5_cl_demo_app_100:
Uasage of the ui_table Element for filtering etc. and the Variant Manager for savinging the setting as reusable variant.
On the first sight, it looks like he can save it:

But if I switch between the variants nothing happens. And also I didn't find out, how to make it persistent, means that the variants are available also after refresh or new entry.
Last thing would be, to hide column and save it like a variant, but this would be the second step..
Thanks in advance

left this project of variant management as this seems to not work correctly with a2ui5,
infact anything with presistence seems to behave wierd, and also it looks like it needs to be run via launchpad in order to make everything work

Hello @choper725
ah okay. Means it is more ore less obolete, right? Thanks for the quick response

Hi @Marchl-Woid,
I am currently working on a generic layout function. I have not yet found a possibility at the frontend. If you give me another week, I can provide you with the function I am currently working on. ;)

its not obsolete its just wasnt able to figure out how to make it work with frontend only,
you can check the p13n demo, if i remember correctly that was the latest try on presistence

yes i think you need some frontend logic and maybe a specific odata service to save the layout for it. there might be the possibility to develop a Custom control for this, but i want to keep abap2UI5 as a backend framework, and also keep most of the logic there. So i did not put more effort into it.
@abapsheep yes implementing this in the backend sounds good, if needed, take example 165 as an orientation.

Hi toghether,
thanks for your quick answers.
@abapsheep : I'm curious to see your approach. I was think of building something by me self, but then I will wait :)
My idea would have been a attribute of type ref to data and generical creating the fields of the structure depending and die Fields of the table and then binding the fields to the column attributes like visible. Then a popup as setting on te table, where the user can set some attribute to the columns. Persistance would be a simple name value table withe the view class a key.

Hi @Marchl-Woid,
I'm ready, have a look at the samples app z2ui5_cl_demo_app_174.

My first approach was also to build a generic structure, but @oblomov-dev was so good and added table cell binding for us. That made things much easier.  If you have any ideas or suggestions for improvement, please let me know ;)

its row or column there in the title ??

HI @abapsheep
wow, great work. I had to play around with it rigth away. :)
First thing what happend was this error I got:

The reason was the ->footer Method in the Popup, I just removed it, then it work fine.
I played around with it a little bit, I changed this two code snippets:

and

The reason is quite easy. If I use a DDIC type for my table strucutre, the texts come from the data element. I tried it out with your sample and change for product the data type from string to data element matnr:
Effect is quite nice:

Names and texts are translated and are euqal to the other places where they are used. If you also like it, maybe you can just adopt the code.
Next thing I'm thinking of, how to use it with the UI_TABLE Element. The class sap.ui.table.Column has properties for visibility, align. So this should be easy. It don't know yet if it's work, but what would be genius, if you can also save as layout the column poisiton in the table and sorting and filtering.
I have seen, that the width is part of the properties and of the DB Table, but it is commented out in the UI. Is there a specific reason for that?

Hi @Marchl-Woid,
the layout function is a first attempt. Depending on the UI element, you could of course add or hide columns, good idea.
I have built the function so that it can theoretically also run in the cloud. There are no DFIES here, so only via detours and therefore I have left out the topic of texts for now ;)
For me, I also need the warehouse number to control the layouts. Here I will see if I can solve this via handels as in the SAP ALV.
Why you get an error with the footer is unclear to me.

concerning the footer:
@Marchl-Woid do you bootstrap with openUI5? @abapsheep did you test with openUI5? very often this creates some problems with certain controls.

HI @oblomov-dev ,
no I'm unsing an onPrem System with the local UI5 libs:
/sap/public/bc/ui5_ui5/1/resources/sap-ui-core.js

can you check the UI5 version by clicking:

HI @oblomov-dev
if I use this, I get this message:

But you search this one, I assume:

thank you, yes this is what i was looking. seems we need to make it more release compatible, i'll check this.

@abapsheep which version are you using?
sap/m/footer seems not to be available on lower releases, possible workaround using beginbutton/endbutton as it is for example done here:
https://github.com/abap2UI5/abap2UI5/blob/9b48c01acd2f4c13c6cf803fba8972041cb79e8b/src/01/03/z2ui5_cl_core_app_info.clas.abap#L107

hi @abapsheep
something wierd is happening,
upon loading a layout the order of the rows change

1st load

2nd load

the order of the columns is changing, hence once changing the entire table order...

Hi everyone,
I'd like to take a further look at this topic. I already have a few extensions in mind. But I'm currently facing a big go-live that will keep me busy for at least the next 2 weeks.

@abapsheep
surly, when you find the time for it.
digging around found that:
upon first load from demo 174 table columns are arranged per the TT binded to the table. (as expected).
METHOD init_layout. work as expected.
once loading a variant from the db, columns get mixed,
the issue lies upon saving the layout to the db, as in the db the order gets changed:

hope this helps..


Add Shahjalal University of Science and Technology

Shahjalal University of Science and Technology (SUST) was established in 1986. The only university of its kind at that time, it started its operations on the 1st Phalgun (13th of February 1991) with only three departments: Physics, Chemistry, and Economics, with 13 teachers and 205 students. It has now expanded to 7 schools, 27 departments, and two institutes. The number of teachers has grown to 566 and the number of students to 8,596. Additionally, the University has 12 affiliated medical colleges under the School of Medical Sciences with 4,000 students.
SUST introduced the integrated honors course in Bangladesh for the first time. It implemented the semester system from the 1996-97 session, and there was a remarkable improvement in the quality of education after the introduction of this system, which was visible even in the national arena. Every student at SUST must take two language courses, one in Bangla and the other in English. Every student must also take two computer courses, one for computer literacy and the other for learning a computer language.
The computer center of SUST offers courses to all employees, and one can say without any hesitation that SUST is the most IT-enabled university in this country. SUST has started its graduate programs by offering master's degrees to graduating bachelors from most departments. While it is difficult to start a world-class research program without a well-funded graduate school, SUST teachers strive to create a research environment on campus.
Two research journals are published regularly from SUST, one in Bangla and the other in English, where hundreds of research papers are submitted every year. Several departments have hosted International Conferences and are planning to organize similar events in the near future.

@Raisul191491 I am sorry, the domain student.sust.edu is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Update keyboard shortcuts to meet the new GitHub content style guide

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://github.com/github/docs/blob/main/components/playground/content/codespaces/setting-up-your-project-for-codespaces/setting-up-your-project-for-codespaces/nodejs.tsx
What part(s) of the article would you like to see updated?
Keyboard shortcuts in this document are outdated. Refer to Keyboard Shortcuts section in the content style guide on how to correctly present keyboard shortcuts in GitHub docs.
Additional information
For example
commandk should be Command+K
commandshift7 should be Command+Shift+7
This issue is for the Campus Experts + GitHub Docs program and is not available to be assigned.

MSS77
El lun., 10 ene. 2022 16:04, Ramya Parimi @.***>
escribió:

Code of Conduct

I have read and agree to the GitHub Docs project's Code of Conduct
https://github.com/github/docs/blob/main/CODE_OF_CONDUCT.md

What article on docs.github.com is affected?
https://github.com/github/docs/blob/main/components/playground/content/codespaces/setting-up-your-project-for-codespaces/setting-up-your-project-for-codespaces/nodejs.tsx
What part(s) of the article would you like to see updated?
Keyboard shortcuts in this document are outdated. Refer to Keyboard
Shortcuts
https://github.com/github/docs/blob/main/contributing/content-style-guide.md#keyboard-shortcuts
section in the content style guide on how to correctly present keyboard
shortcuts in GitHub docs.
Additional information
commandk should be Command+K
commandshift7 should be Command+Shift+7
—
Reply to this email directly, view it on GitHub
https://github.com/github/docs/issues/13899, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQK2GQXL5FDPELERRYI7TY3UVLYOJANCNFSM5LT4BYCQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Add distance.cpp test in CMake

Neglected to add the unit test when I wrote it, so I just add it now :)

fixed. sorry for that.


Can not start my app on my OS X.

Hello everyone, I need help, I just can not create an app with ionic cli on osx.
Error from my console:
 $ ionic start myapp
Creating Ionic app in folder /Users/Woody/Test/myapp based on tabs project
Downloading: https://github.com/driftyco/ionic-app-base/archive/master.zip
Error updating config.xml file: Error: ENOENT, no such file or directory '/MyHome/myapp/config.xml'
 ✗ Unable to save settings file: Error: ENOENT, no such file or directory '/MyHome//myapp/ionic.project'

ionic info output:
Cordova CLI: 5.0.0
Gulp version:  CLI version 3.8.11
Gulp local:
Ionic CLI Version: 1.4.5
Ionic App Lib Version: 0.0.22
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Mac OS X Yosemite
Node Version: v0.10.32
Xcode version: Xcode 6.3.2 Build version 6D2105

My OS is OSX Yosemite 10.10.3.
I tried ionic with cygwin on my windowns, it's just ok. But now on my OSX, it just can not create my app directory. I tried with 'sudo' too.

Greetings @bigscorpions!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

I'm not sure what's the problem. Maybe it's my network or something else. I have tried to reinstall cordova and ionic, and restart my system(I forgot if I did it or not), when I come back from work, it's just fine to start the project. I think it's my network problem cause the downloading speed from github is too slow.


Fix/typ

Fix: Corrected typos in Rust source files
What was changed

Corrected typos:

Replaced presense with presence in src/lib.rs:361.
Replaced interally with internally in ff_derive/src/lib.rs:1384.

Why these changes were made

Fixing typos improves the clarity and professionalism of the codebase, ensuring accuracy and ease of understanding for developers.

Additional Notes

These changes focus solely on correcting typographical errors and do not affect the functionality of the code.

I do not accept typo-fixing PRs on my repositories as a spam-prevention measure, due to the rise of AI-generated drive-by PRs.


updating semver version to ^6.3.0 to use semverv2

@tejasd1990 can you run some tests while downloading

sdk
runtime

of semver version 1.0 and 2.0
with

exact versioning: 2.2.105
3.0.x / 3.x  (with include preview enabled and disabled)

We just need to be sure that we aren't breaking anything here.

@hiyadav I have already tested various scenarios with the new change including versions which are v1 and v2

This is a slight breaking change as the 'X' version identifier is now case sensitive....

Hi @TJEvans , The version identifier has always been case sensitive, it should be lower 'x'. Can you confirm whether it worked for you earlier with capital 'X' as well?

Prior to the task updating this was the logs,
2019-11-07T17:23:44.3328779Z ##[section]Starting: Use dotnet sdk 2.2.X
2019-11-07T17:23:44.3759718Z ==============================================================================
2019-11-07T17:23:44.3759933Z Task         : Use .NET Core
2019-11-07T17:23:44.3760075Z Description  : Acquires a specific version of the .NET Core SDK from the internet or the local cache and adds it to the PATH. Use this task to change the version of .NET Core used in subsequent tasks. Additionally provides proxy support.
2019-11-07T17:23:44.3760386Z Version      : 2.0.21
2019-11-07T17:23:44.3760508Z Author       : Microsoft Corporation
2019-11-07T17:23:44.3760682Z Help         : https://aka.ms/AA4xgy0
2019-11-07T17:23:44.3760780Z ==============================================================================
2019-11-07T17:23:45.3619447Z Tool to install: .NET Core sdk version 2.2.X.
2019-11-07T17:23:46.4170769Z Found version 2.2.402 in channel 2.2 for user specified version spec: 2.2.X
2019-11-07T17:23:46.4687151Z Version: 2.2.402 was found in cache.
2019-11-07T17:23:46.4736116Z Creating global tool path and pre-pending to PATH.
2019-11-07T17:23:46.4787149Z ##[section]Finishing: Use dotnet sdk 2.2.X


Robust temporal map of human in vitro myelopoiesis using single-cell genomics

Project short name:
Primary Wrangler:
Ami
Secondary Wrangler:

Associated files
INGEST: https://contribute.data.humancellatlas.org/projects/detail?uuid=ac289b77-fb12-4a6b-ad43-c0721c698e70&tab=project

Google Drive:

https://docs.google.com/spreadsheets/d/1LlC4KDUj4lpKluklUvrpq9JIbrhs7y6J/edit#gid=2014509335
https://drive.google.com/drive/folders/1GOwQCJBB6xsWDsAGanmiKKrNhjGfpvue
Published study links

Paper: Robust temporal map of human in vitro myelopoiesis using single-cell genomics
https://www.nature.com/articles/s41467-022-30557-4#Sec1

Accessioned data:
https://www.ebi.ac.uk/arrayexpress/experiments/E-MTAB-11616/
https://www.ebi.ac.uk/arrayexpress/experiments/E-MTAB-11623/
Look on the paper for more links.

Key Events

[x] Convert published metadata to HCA spreadsheet
[x] Manually curate dataset to meet HCA metadata standard
[ ] Collect any matrix and cell-type annotation files
[ ] Upload sheet to validate metadata
[ ] Check linking using ingest graph validator
[ ] Transfer raw files to ingest to validate data files
[ ] Ask the Secondary Wrangler for an end-to-end review of the project. Ask the Expertise Wrangler to review specific tabs if needed
[ ] Submit dataset to Production
[ ] Complete the Export SOP
[ ] Convert project data to SCEA format following the SCEA conversion SOP if appropriate

Thanks @ESapenaVentura, I have made your suggested updates except for the following:

I’d use “embryo” instead if bone marrow as the organ model. I know they are only sampling the myeloid system but the “organoids” (embryoid bodies) represent the germinal layers, not just the bone marrow

I understand your point about representing iPSC cell lines with "embryo" as the organ model, but I think for the organoid cultures that "hematopoietic system" (UBERON:0002390) would be best as the model organ (because the myeloid system is part of the hematopoietic system). This is because I think the organ model is supposed indicate the intended organ once fully differentiated.
I am downloading the analysis files currently (taking a long time!) and will message you on slack when they're ready.

I understand your point about representing iPSC cell lines with "embryo" as the organ model, but I think for the organoid cultures that "hematopoietic system" (UBERON:0002390) would be best as the model organ (because the myeloid system is part of the hematopoietic system). This is because I think the organ model is supposed to indicate the intended organ once fully differentiated, and they are already being cultured in differentiation media and on the path to differentiation.

Sounds good to me!
About the review,
Analysis files

I am missing 2 files from their portal https://www.hipimmuneatlas.org/

Other than that, looks fine to me!

Ok, I have added the macrophage differentiation analysis file but not the LPS file, as discussed and the fastq files are downloading.

Fastq files are downloading very slowly (slower than usual). I aim to get the fastq files upload tomorrow (so they can download overnight).

Moved to Release 20

graph validating

Submitted.

This project is stuck in exporting because of a timeout checking the google cloud sync, the export jobs are being kept in the error queue so that they can be re-issued.

Using the getstatus script I can see that the GCS transfer finished after 12 hours:
Returned transferOperations: {
    "operations": [
        {
            "name": "transferOperations/transferJobs-62e90a452f1d496804222e7d-1659441538317039",
            "metadata": {
                "@type": "type.googleapis.com/google.storagetransfer.v1.TransferOperation",
                "name": "transferOperations/transferJobs-62e90a452f1d496804222e7d-1659441538317039",
                "projectId": "mystical-slate-284720",
                "transferSpec": {
                    "awsS3DataSource": {
                        "bucketName": "org-hca-data-archive-upload-prod",
                        "path": "f433f707-ce4c-4c24-b60f-7cda116ea598/"
                    },
                    "gcsDataSink": {
                        "bucketName": "broad-dsp-monster-hca-prod-ebi-storage",
                        "path": "prod/ac289b77-fb12-4a6b-ad43-c0721c698e70/data/"
                    },
                    "transferOptions": {}
                },
                "startTime": "2022-08-02T11:58:58.469055314Z",
                "endTime": "2022-08-02T23:23:25.009606840Z",
                "status": "SUCCESS",
                "counters": {
                    "objectsFoundFromSource": "321",
                    "bytesFoundFromSource": "3866774267708",
                    "objectsCopiedToSink": "321",
                    "bytesCopiedToSink": "3866774267708"
                },
                "transferJobName": "transferJobs/62e90a452f1d496804222e7d"
            },
            "done": true,
            "response": {
                "@type": "type.googleapis.com/google.protobuf.Empty"
            }
        }
    ]
}
job is done

I've patched the ingest job to show that the export is finished:
curl -L -X PATCH "https://api.ingest.archive.data.humancellatlas.org/exportJobs/62e90a452f1d496804222e7d" \
-H "Authorization: Bearer snip" \
-H "Content-Type: application/json" \
--data-raw "{
    \"context\": {
        \"totalAssayCount\": 103,
        \"isDataTransferComplete\": true
    }
}"

I've moved the errored jobs back to the job queue. This should export without problems now.

@ami-day to check if this is exported

Submitted import form.


Implementation of Thingsboard MQTT commands

Documentation

Topic
Pub / Sub
Description
Status

v1/devices/me/telemetry
PUB
Publish telemetry data to ThingsBoard server node.
✅

v1/devices/me/attributes
PUB / SUB
Publish client-side device attributes to ThingsBoard server node. Subscribe to shared device attribute changes
✅ / ⏳

v1/devices/me/attributes/request/$request_id
PUB
Request client-side or shared device attributes to ThingsBoard server node
⏳

v1/devices/me/attributes/response/+
SUB
Subscribe to client-side or shared device attribute changes.
⏳

v1/devices/me/rpc/request/+
SUB
Subscribe to RPC commands from the server.
⏳

v1/devices/me/rpc/request/$request_id
PUB
Publish RPC commands to ThingsBoard server node.
⏳

v1/devices/me/rpc/response/$request_id
PUB
The client (Device) should publish the response to this topic.
⏳

v1/devices/me/claim
PUB
Initiate claiming device.
⏳

/provision

Initiate device provisioning.
⏳

v2/fw/request/${requestId}/chunk/${chunkIndex}
PUB
Ddevice firmware update, it sets the fw_title, fw_version, fw_checksum, fw_checksum_algorithm shared attributes.
⏳

v2/fw/response/+/chunk/+
SUB
Receive the device updates for fw_title and fw_version shared attributes.
⏳


Azure: Upload of decompressed FCOS image is very slow

Hi,
I couldn't manage to get the new Azure installer working because the upload of the decompressed (fcos image is xz compressed) takes so long (more than 30 Minutes on my PC) that this procedure seems not to be feasable to me.
Please support a procedure where users can upload the fcos image on their own to a storage account which is in a different resource group and will be referenced from all cluster VMs.
A method to configure this storage blob container is necessary (through env variables which are fed to the installer?).
The time for the upload would be necessary only once for all OKD clusters and must not be spent for each cluster again and again.
Best regards,
Josef

https://github.com/openshift/installer/pull/3033

Just tried to deploy a cluster to azure, Im running into the same problems.

@sanderkooger:
In my video I explain how you can modify the openshift installer to get OKD 4.4 installed on Azure:
https://www.youtube.com/watch?v=PXZn87FCIYk
Have fun (and don't forget to like my video if it helped you :-) ).

It's on minute 14:13

One generic thing you can do is just boot up any Linux machine in Azure and run openshift-install from there over ssh or whatever.
(There's also https://github.com/openshift/hive/ once you have one cluster running)

I just created a fresh Ubuntu box and tried from within azure. No joy...

ERROR
ERROR Warning: "resource_group_name": [DEPRECATED] This field has been deprecated and is no longer used - will be removed in 2.0 of the Azure Provider
ERROR
ERROR   on ../../tmp/openshift-install-362141893/main.tf line 163, in resource "azurerm_storage_container" "vhd":
ERROR  163: resource "azurerm_storage_container" "vhd" {
ERROR
ERROR (and 3 more similar warnings elsewhere)
ERROR
ERROR
ERROR Error: compute.ImagesClient#CreateOrUpdate: Failure sending request: StatusCode=400 -- Original Error: Code="InvalidParameter" Message="The source blob https://cluster3iydm.blob.core.windows.net/vhd/rhcos3iydm.vhd is not a page blob." Target="disks"
ERROR
ERROR   on ../../tmp/openshift-install-362141893/main.tf line 179, in resource "azurerm_image" "cluster":
ERROR  179: resource "azurerm_image" "cluster" {
ERROR
ERROR
FATAL failed to fetch Cluster: failed to generate asset "Cluster": failed to create cluster: failed to apply using Terraform

Its my first cluster so hyve is not an option.

@jomeier I'm trying your YouTube vid but on cherry-pick item give me an error.
git cherry-pick 05a7dff57a68134a5457e98864e62783221a5aa
fatal: bad revision '05a7dff57a68134a5457e98864e62783221a5aa'

@sanderkooger:
Hm ... I just tried it out again and on my site it works (git version 2.17.1)

@sanderkooger
I reported this "is not a page blob" several times. My hack also takes care about that.

@jomeier Thats weird as I can cherrypick the other commits. Its just the last one i cant pick.

@sanderkooger
You can leave that one away. With this commit the installation will do a little bit faster but it should work without it for the moment.

let me retry

@sanderkooger
Could you try out this and confirm that it works, please?

@jomeier Is there a place where we can chat privately? Answered in your new issue. Having issues using your YouTube as i have to copy the identifiers for the commits by hand.  Mail me at<EMAIL_ADDRESS>please.

@sanderkooger
Come in the kubernetes/#openshift-dev channel.

If you put the latest nightly in an Azure VM or Cloud Shell, the Installation will succeed without further patches.

Maybe ill try, I actually have been doing the math, The infrastructure is meant for a small startup ( unfunded) and I kind of figured that maybe its better to run AKS instead of openshift, It’s a little less developer friendly in terms of automates stream2images, but on the other hand During development I only have to run one workernode and a mysql db.
Running 2 4core masternodes, 2 4core infranodes, and the workernodes gets a little too expensive too fast.
Thank you for the support though! And good work!
From: Josef Meier<EMAIL_ADDRESS>Sent: Friday, 8 May 2020 17:09
To: openshift/okd<EMAIL_ADDRESS>Cc: Sander Kooger<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [openshift/okd] Azure: Upload of decompressed FCOS image is very slow (#48)
If you put the latest nightly in an Azure VM or Cloud Shell, the Installation will succeed without further patches.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/openshift/okd/issues/48#issuecomment-625861824, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AET7V2S4H27PCBS2ME3CTCDRQQOAJANCNFSM4KM6CWPQ.
Note: This message contains confidential information and is intended only for the individual named. If you are not the named addressee you should not disseminate, distribute or copy this e-mail. You cannot use or forward any attachments in the email. Please notify the sender immediately by e-mail if you have received this e-mail by mistake and delete this e-mail from your system. The integrity and security of this message cannot be guaranteed on the Internet.


AnnotatedObjectSerializer no longer removes important type information

This PR modifies AnnotatedObjectSerializer so that important generic type information is not lost.
Some guice bindings require additional generic type information which is lost when you convert the TypeToken to a class.

You clearly didn't test this -- if you had even tried to build the project, you'd see that tests are failing... Please fix that.


Remove creds-init initContainer

Changes
Fixes https://github.com/tektoncd/pipeline/issues/2524 and puts us in a better position to flip the disable-home-env-overwrite feature flag with less fallout for users.
TLDR; The behaviour that used to be implemented by creds-init is now performed by
the entrypointer.
Creds-init is an initContainer that writes credential files (like .ssh) to
/tekton/home. One side-effect of relying on an initContainer to write these
files is that they receive the UID / Group ID of the initContainer's process.
This side-effect breaks any Step in the Task that relies on these credentials
but runs with a different UID. An example of where this can happen is on
OpenShift, where the UID of the user is randomized for each container in
order to limit the fallout of malicious process breaking out.
This commit removes the creds-init initContainer from all TaskRun Pods. I
haven't removed the creds-init binary from our build process in this
changeset. Doing so generates a lot of extra line noise which distracts
from the core modification being presented here.
Further fallout from this change:

Credential volume mounts from annotated Secrets used to be mounted only once,
in the creds-init initContainer, at /tekcon/creds-secrets/.... These volume mounts
are now given to every Step container so that they can be copied out by the entrypointer.
Credentials used to be copied from /tekton/creds-secrets/... to /tekton/home
or /tekton/creds, depending on whether the disable-home-env-overwrite feature flag
was set. Now those credentials are always copied into /tekton/creds.
The $(credentials.path) variable is now always /tekton/creds regardless of whether
the disable-home-env-overwrite feature flag is set or not.
When the  disable-home-env-overwrite feature flag is set, credentials will be placed at
the actual home directory of the current user in the Step container, instead of being placed
in /tekton/home. If the container or Step explicitly sets a $HOME then credentials will
end up in there.
Every single Step gets its own empty /tekton/creds volume. This allows /tekton/creds to be world-writable and empty for every Step to copy in to.  The directory needs to be world-writable so it's accessible even if the user's UID wouldn't normally have permissions to create a directory under /tekton. This is also why there are so many lines changed in the various pkg/pod tests.

I've put a hold on this while I add better testing and examples that exercise the cases that this PR deals with.
Need to figure out a safe way to introduce a credential that can be used for testing. Possibly a read-only Github Deploy Secret for a private test Github repo in the tektoncd org?
Edit: It took me way too long but I finally figured out how to spin up a git/ssh sidecar that works as an authenticated remote repo for testing creds-init.

[x] Example that shows credentials being used in conjunction with different securityContexts per Step.
[x] Example that shows using these credentials with vanilla git commands, not git-init or Git PipelineResource.
[ ] Documentation that describes the behaviour of credentials in the face of different securityContexts.

I think there also needs to be a bit of discussion about some of the more questionable design decisions.  Maybe next API WG.
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[x] Includes tests (if functionality changed/added)
[x] Includes docs (if user facing)
[x] Commit messages follow commit message best practices

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes
Tekton's creds-init initContainer has been removed and will no longer show up in TaskRun Pods. Credentials from annotated Secrets will continue to be mounted and placed in Step containers' home directory.

/kind bug
/hold

I've put a hold on this while I add better testing and examples that exercise the cases that this PR deals with.

I am +:100: for this change :heart_eyes_cat:

I wonder if we can use a gitolite deployment spinned just before running the tests with dynamically generated ssh keys for testing, I mean instead of having to deal with private external github repositories?

I wonder if we can use a gitolite deployment spinned just before running the tests with dynamically generated ssh keys for testing, I mean instead of having to deal with private external github repositories?

I've managed to spin up an example which doesn't need the private external repo - see examples/v1beta1/taskruns/authenticating-git-commands.yaml  in the diff here.  I'll remove that comment from the description.

/hold cancel
OK, I think this PR is ready for review.  It may be worth taking a look at the example YAML I've added to get a sense of how creds-init changes in the face of different securityContexts & feature flags.
I've also added some short docs to try and help users out navigating the quagmire. It's definitely not easy to wrap one's head around unfortunately.

/test pull-tekton-pipeline-integration-tests

We will be able to remove creds-init image in a follow-up right ?

Yes, 💯, it just made the diff really extra noisy.  I plan to move straight on to that after this.

/test pull-tekton-pipeline-integration-tests

Have addressed comments and improved the coverage so it's a positive rather than negative change.

Looks like the coverage went down, is that something we want to address?

Looks like the coverage went down, is that something we want to address?

@bobcatfish You might be looking at an older report?  The latest numbers look positive to me:

@bobcatfish You might be looking at an older report? The latest numbers look positive to me:

Whoops, sorry about that @sbwsg , you're totally right!

Edit: It took me way too long but I finally figured out how to spin up a git/ssh sidecar that works as an authenticated remote repo for testing creds-init.

:100: I love it.
/lgtm

I just rebase it against master and fix the conflict on the docs/variables.md :upside_down_face:
/lgtm

/approve

/test  pull-tekton-pipeline-integration-tests

/retest

/retest

/retest


[CARBONDATA-4065] Support MERGE INTO SQL Command

Why is this PR needed?
In order to support MERGE INTO SQL Command in Carbondata
The previous Scala Parser having trouble to parse the complicated Merge Into SQL Command
What changes were proposed in this PR?
Add an ANTLR parser, and support parse MERGE INTO SQL Command to DataSet Command
Does this PR introduce any user interface change?

Yes.
The PR introduces the MERGE INTO SQL Command.

Is any new testcase added?

Yes

@Pickupolddriver : PR shows 37KLOC code, please rebase and keep only required changes. I saw some codegen file of 24KLOC

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/4985/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/3230/

@Pickupolddriver : PR shows 37KLOC code, please rebase and keep only required changes. I saw some codegen file of 24KLOC

Yes, lots of codes are auto-generated by ANTLR, I will try to remove them and let it generated during the compile process.

added antlr4-maven-plugin to manage the antlr files during compile

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/5038/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/3281/

Added support insert literal
TODO: Add doc and more test cases

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/5056/

Build Failed  with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/3297/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/5069/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/3311/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/5054/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/5055/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/3295/

Build Failed  with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/3312/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/3337/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/5093/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/5072/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5104/

Build Failed  with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3343/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5108/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3346/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5161/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3399/

Build Failed  with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/3301/

retest this please

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5243/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3482/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3488/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5249/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3492/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5253/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5267/

Build Failed  with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3506/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3508/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5269/

@Zhangshunyu , @Pickupolddriver : The document need to enchanced some more, I gave comment.  And simplifying g4 file you can check with David again.  Please recheck and once all comments are handled reply here. @QiangCai can merge.
@QiangCai , Please use co-authored by while merging, as I can see 2 authors.

@Zhangshunyu , @Pickupolddriver : The document need to enchanced some more, I gave comment.  And simplifying g4 file you can check with David again.  Please recheck and once all comments are handled reply here. @QiangCai can merge.
@QiangCai , Please use co-authored by while merging, as I can see 2 authors.

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5273/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5273/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3512/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3512/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5275/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5275/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3514/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3514/

@ajantha-bhat @QiangCai fixed review comments, pls check.

@ajantha-bhat @QiangCai fixed review comments, pls check.

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5293/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5293/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3533/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3533/

LGTM

LGTM

LGTM

LGTM

@Zhangshunyu , @Pickupolddriver : please remove binary files, carbondata and index files added. It has to generate on the fly
@QiangCai : please wait for this before merging

@Zhangshunyu , @Pickupolddriver : please remove binary files, carbondata and index files added. It has to generate on the fly
@QiangCai : please wait for this before merging

@ajantha-bhat @QiangCai OK now

@ajantha-bhat @QiangCai OK now

LGTM

LGTM

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5294/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5294/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3534/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3534/


Fix typo

successfull→successful

Docs Build status updates of commit df2f515:
:warning: Validation status: warnings

File
Status
Preview URL
Details

index.md
:warning:Warning

Details

docs-ref-conceptual/operationsmanager/index.md
:warning:Warning

Details

docs-ref-autogen/operationsmanager/Authentication/Login.yml
:white_check_mark:Succeeded

index.md

Line 9, Column 10: [Warning-ms-prod-technology-invalid] Invalid value for 'ms.prod': 'system-center-2016'.

docs-ref-conceptual/operationsmanager/index.md

Line 9, Column 10: [Warning-ms-prod-technology-invalid] Invalid value for 'ms.prod': 'system-center-2016'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

@eltociear - thank you for your contribution to the content hosted in this repo. Because this content is generated following our automated swagger process, I'm going to file a bug in our DevOps so that the developer updates the file and republishes the fixed yaml file following that internal process.


Multi-select with Selection Manager not toggling

I am trying to make my custom visual behave as closely as possible as the built in visualizations. When using a Microsoft visualization, when holding Control and clicking a data sections, multiple sections can be selected. When a section is clicked, it toggles between selected and not selected. I would like to emulate this in my visualization using the Selection Manager API.
Right now, when Control-clicking, adding new sections to the selection works. However, when clicking on a selected section attempting to remove it from the selection, instead of toggling, all sections become de-selected.
How can I make Control-clicking a selected section toggle that section instead of deselecting everything?
Here are some code snippets. Most of this is based on the "Creating a Custom PowerBI Visualization from Scratch with Sachin Patney" video, with some changes based on reading some of the source code in the repository.
The visualization is based on stadium seating, with each stadium section being a data point.
export interface section
	{
		sectionId: string;
		capacity: number;
		numSeatsSold: number;
		identity: SelectionId;
	}

identity is populated in the converter function:
{
     //...
     // dataView.categorical.categories[0] is the array of sections
     // i is the section index
     identity:SelectionIdBuilder.builder().withCategory(dataView.categorical.categories[0],i).createSelectionId();
}

In init, create a selectionManager instance
this.selectionManager = new utility.SelectionManager({hostServices:options.host});

In update
//When a section is clicked
section.on('click',function(d){
this.selectionManger.select(d.identity, d3.event.ctrlKey).then(ids=>{
     if(ids.length>0)//If 1 or more sections are selected
     {
          //Modify CSS
     }
     d3.event.stopPropagation();
}

//When empty space is clicked
this.svg.on('click',
() => this.selectionManager.clear().then(() => /*Modify CSS*/)

When a section that is already selected is Control clicked, ids becomes an empty array. Reading the code for SelectionManager, I cannot figure out why. Am I passing in or doing SelectionIds wrong in some way?

Yes, probably it's infrastructure issue. We will add this bug to the backlog.


sqlalchemy_utils is not installed as a requirement

While creating database like:
openstack@openstack-VirtualBox:~/masakari/masakari_controller/db$ sudo ./create_database.sh /home/openstack/masakari/masakari_controller/db /home/openstack/masakari Traceback (most recent call last):   File "/usr/lib/python2.7/runpy.py", line 162, in _run_module_as_main     "__main__", fname, loader, pkg_name)   File "/usr/lib/python2.7/runpy.py", line 72, in _run_code     exec code in run_globals   File "/home/openstack/masakari/masakari_controller/db/create_tables.py", line 22, in <module>     from sqlalchemy_utils.functions import database_exists, create_database ImportError: No module named sqlalchemy_utils.functions
we get import error for sqlalchemy_utils module. It should be mentioned in the masakari-controller/requirements.txt and should be installed as a dependency

Hi, Thanks for the report.
I thought I resolved this by adding following line to masakari-controller/requirements.txt.
https://github.com/ntt-sic/masakari/blob/master/masakari-controller/requirements.txt#L14
SQLAlchemy-Utils>=0.32.0
Are you using the latest code (tag > 1.1.0) ?

Oh! yess
The latest master code has this requirement


Add event when updated worker is ready

Clear and concise description of the problem
At the moment when calling updateServiceWorker it calls SKIP_WAITING, but there are a few issues with this:

The event does not do anything until the workbox event waiting is fired, otherwise the worker will sit in waiting state forever.
There is no event passed back to the user indicating that the update is ready.

The issue is that our cache is fairly big (fully offline app) and phone users need a bit to update the cache, because there is no way of knowing that the new cache is activated, they reload the page too quickly and having to re-installed the service worker again.
Suggested solution
https://github.com/antfu/vite-plugin-pwa/blob/main/src/client/build/register.ts#L47
Wrap the messageSW in an event:
wb?.addEventListener('waiting', (event) => {
    if (registration && registration.waiting) {
        // Send a message to the waiting service worker,
        // instructing it to activate.
        // Note: for this to work, you have to add a message
        // listener in your service worker. See below.
        await messageSW(registration.waiting, { type: 'SKIP_WAITING' })
      }
})

Add an event to the RegisterSWOptions: onUpdateReady, which is called when controlling:
https://github.com/antfu/vite-plugin-pwa/blob/main/src/client/build/register.ts#L42
      wb?.addEventListener('controlling', () => {
        onUpdateReady?();
      });

I can create a PR if this feature seems legit.
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

@rogerfar there is no problem updating the cache, it is done by the browser in a background thread, it the app is large then it will take more time to be ready (check @vueuse for example).
If you call updateServiceWorker(false) (I don't know why you want to call it with false) you should do a hard refresh or reopen the browser, the sw will be updated (just try pressing F5 while the sw is in yellow state, then hard refresh). It seems all browsers with the exception of Safari will work (#33 is not reproducible anymore at least on Cchromium based browsers and Firefox).

@rogerfar, on the other hand, we would be interested in being able to notify the user that a new version has been detected and is being downloaded, so PR welcome


Grails 3

Looks great @ctoestreich, I went through the full diff and didn't see anything that jumped out at me.  I'll merge this in to the grails3 branch and play around with it some.
I'll see if I can get it released tonight to bintray otherwise it'll probably be tomorrow.  As I mentioned above, I don't think that bintray supports SNAPSHOT releases, so I might have to just call this 3.0.0-RC1 or something like that.


Email bounces

In gitlab by @NFarrington on Jun 29, 2016, 21:35
Utilise mail provider APIs to store all permanent bounces in Core so that they can be reviewed at any time (and are not subject to provider log storage limits).

Duplicate


Let hvm generate the alias shell code via new hvm gen alias command

Currently, users have to consult the README to set up the required shell alias. The shell snippets for the different platforms could instead be generated by hvm (at build time, I guess) via a new command like hvm alias.
Motivations for this include:

Simplify this step of the installation.
Uncouple this step of the installation from the README.

Depends on #20.


Showcase page with mock data

Closes #517

Microsoft Reviewers: Open in CodeFlow

I get the need for a showcase page - but do we want to wait to expose this until we have actual apps to show? Or maybe add a "coming soon" at the top and use lorem ipsom text?

Agreed. I believe that Alex posted this to get feedback on the format/layout, but I am working on putting together the actual content. I'm not sure if Alex was intending to actually host the website with mock data, but I don't think there is much value in doing that.

Pinging this so the bot doesn't age it out :) @stmoy lmk when you will have the actual data


Avoid application deletion race conditions

What does this PR do?:
This PR tries to implement the same logic followed to the component_controller (introduced here: https://github.com/redhat-appstudio/application-service/pull/451) on the application_controller too. This way we avoid race conditions upon application & component deletion.
More detailed, we are ensuring that before the application deletion (after we have confirmed the presence of a finalizer), that the application has no components under deletion. If it does we are requeueing the deletion.
Which issue(s)/story(ies) does this PR fixes:
https://issues.redhat.com/browse/DEVHAS-627
PR acceptance criteria:

[ ] Unit/Functional tests

[ ] Documentation

[ ] Client Impact

How to test changes / Special notes to the reviewer:

/retest

/retest

More detailed, we are ensuring that before the application deletion (after we have confirmed the presence of a finalizer), that the application has no components under deletion. If it does we are requeueing the deletion.

I'm wondering if this is needed? Since we fixed the issue where the Component deletion could get stuck if the GitOps repository has been deleted, I don't think there's a potential race condition that can occur here?

More detailed, we are ensuring that before the application deletion (after we have confirmed the presence of a finalizer), that the application has no components under deletion. If it does we are requeueing the deletion.

I'm wondering if this is needed? Since we fixed the issue where the Component deletion could get stuck if the GitOps repository has been deleted, I don't think there's a potential race condition that can occur here? If there's Components that are under deletion when we delete the Application, that just means the GitOps repository will get deleted before the Component finalizer runs (and since #451 was merged, that shouldn't be an issue).
We remove the finalizer no matter what here, so even if the Component clean up "fails" due to a missing GitOps repository, the deletion of either resources won't be blocked

More detailed, we are ensuring that before the application deletion (after we have confirmed the presence of a finalizer), that the application has no components under deletion. If it does we are requeueing the deletion.

I'm wondering if this is needed? Since we fixed the issue where the Component deletion could get stuck if the GitOps repository has been deleted, I don't think there's a potential race condition that can occur here? If there's Components that are under deletion when we delete the Application, that just means the GitOps repository will get deleted before the Component finalizer runs (and since #451 was merged, that shouldn't be an issue).
We remove the finalizer no matter what here, so even if the Component clean up "fails" due to a missing GitOps repository, the deletion of either resources won't be blocked

FYI the PR is basically the extra step discussed here: https://github.com/redhat-appstudio/application-service/pull/451#discussion_r1529559854
The main reason is that we want to fix the component deletion metrics. I've tried to revisit the investigation on the component deletion errors after your comment. My notes:

The truth is In the latest error logs I cannot see any Unable to update GitOps repository for component.. which comes from an unsuccessful deletion.

As we have already noticed the main log is: Operation cannot be fulfilled on...the object has been modified;. I think this error comes from line#213 where we are trying to update the to-be-deleted component.

I can see that in line#194 we are increasing the total deletion attempts. But we only increase the component deletion failed/succeeded for specific cases (see line#205 & line#219).

Having in mind the above note I think the reason of the component deletion not successful alert might be coming because our total attempts are much bigger than the sum of failed + succeeded (e.g we increase the total attempts but we don't equally increase the failed/successful). As a result the succeeded attempts should be only a small pertecantge of the total deletion attempts.

From the above notes I'd say we could:

Increase total attempts only for cases we decide that the attempt is failed/succeded
OR
We add succeded attempts for lines 214 & 216

wdyt? cc @maysunfaisal @johnmcollier

More detailed, we are ensuring that before the application deletion (after we have confirmed the presence of a finalizer), that the application has no components under deletion. If it does we are requeueing the deletion.

I'm wondering if this is needed? Since we fixed the issue where the Component deletion could get stuck if the GitOps repository has been deleted, I don't think there's a potential race condition that can occur here? If there's Components that are under deletion when we delete the Application, that just means the GitOps repository will get deleted before the Component finalizer runs (and since #451 was merged, that shouldn't be an issue).
We remove the finalizer no matter what here, so even if the Component clean up "fails" due to a missing GitOps repository, the deletion of either resources won't be blocked

More detailed, we are ensuring that before the application deletion (after we have confirmed the presence of a finalizer), that the application has no components under deletion. If it does we are requeueing the deletion.

I'm wondering if this is needed? Since we fixed the issue where the Component deletion could get stuck if the GitOps repository has been deleted, I don't think there's a potential race condition that can occur here? If there's Components that are under deletion when we delete the Application, that just means the GitOps repository will get deleted before the Component finalizer runs (and since #451 was merged, that shouldn't be an issue).
We remove the finalizer no matter what here, so even if the Component clean up "fails" due to a missing GitOps repository, the deletion of either resources won't be blocked

FYI the PR is basically the extra step discussed here: #451 (comment)
The main reason is that we want to fix the component deletion metrics. I've tried to revisit the investigation on the component deletion errors after your comment. My notes:

The truth is In the latest error logs I cannot see any Unable to update GitOps repository for component.. which comes from an unsuccessful deletion.
As we have already noticed the main log is: Operation cannot be fulfilled on...the object has been modified;. I think this error comes from line#213 where we are trying to update the to-be-deleted component.
I can see that in line#194 we are increasing the total deletion attempts. But we only increase the component deletion failed/succeeded for specific cases (see line#205 & line#219).
Having in mind the above note I think the reason of the component deletion not successful alert might be coming because our total attempts are much bigger than the sum of failed + succeeded (e.g we increase the total attempts but we don't equally increase the failed/successful). As a result the succeeded attempts should be only a small pertecantge of the total deletion attempts.

From the above notes I'd say we could:

Increase total attempts only for cases we decide that the attempt is failed/succeded
OR
We add succeded attempts for lines 214 & 216

wdyt? cc @maysunfaisal @johnmcollier

as discussed I'm closing for now the PR. If in the future this solution is needed we can re-open.


Stage view not displaying in new MS Teams Client

When loading a video player through stage view using the new Teams client, audio plays but the page itself does not load.
Attempted:

Teams on the browser loads stage view, video appears and sound works as expected
Old Teams client on MacOS and Windows loads stage view, video appears and sound works as expected
New Teams client on MacOS and Windows loads stage view, video does not appear (just a blank screen, no html displayed) but sound is played.

I was advised not to upload screenshots here but can email them to showcase behavior.

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 19ddf42e-0a47-7717-52d4-e549155480a2
Version Independent ID: 4bbe9beb-233f-cfd5-097b-f280aab5fde8
Content: Microsoft Teams developer community support and feedback - Teams
Content Source: msteams-platform/feedback.md
Service: msteams
GitHub Login: @surbhigupta
Microsoft Alias: lajanuar

@aschmois-thrive  - Thanks for reporting your issue. We will test this and update you soon.

@aschmois-thrive  -  Could you please share the code snippet of  video player through stage view   or deployed app package(manifest) and repro steps that would be help for better investigating


Add e2e deletion test

Towards https://github.com/giantswarm/giantswarm/issues/1902.
Currently fails because of an issue in the resource delete method. When a fix will be provided this can go foreward.

E2e with merged delete fixes passes now. Opening for review.

Adressed reviews. PTAL :)


legend title ellipsis with tooltip

This PR addressed the issue at the web-eda, #332
Tried to wrap lengthy legend title text at first. However it was found that the area of legend title did not allow multiple lines: wrapped texts were overlapped in the single line.
Thus, it was decided to use ellipsis with tooltip. A screenshot is attached as follows:

Thank you for your quick review @jtlong3rd 👍


Enable Customized CSRF Protection in Angular UI

References

Replaces #833 (Based heavily on this PR)
Replaces #907
Requires https://github.com/DSpace/DSpace/pull/3103

Description
This PR creates a customized XsrfInterceptor which responds to the Backend/Server Webapp based on the custom CSRF Protection added in https://github.com/DSpace/DSpace/pull/3103
Namely, the behavior is as follows:

Backend generates XSRF token & stores in a server-side (only) cookie named DSPACE-XSRF-COOKIE. This cookie is not read/used by Angular, but is returned (by user's browser) on every subsequent request to backend.
Backend also sends XSRF token in a header named DSPACE-XSRF-TOKEN to Angular.
A custom XsrfInterceptor looks for DSPACE-XSRF-TOKEN header in a response. If found, its value is saved to a client-side (only) cookie named XSRF-TOKEN.
Whenever Angular is making a mutating request (POST, PUT, DELETE, etc), the same XsrfInterceptor checks for that client-side XSRF-TOKEN cookie. If found, its value is sent to the backend in the X-XSRF-TOKEN header.
On backend, the X-XSRF-TOKEN header is received & compared to the current value of the server-side cookie named DSPACE-XSRF-COOKIE (created in step 1). If tokens match, the request is accepted. If tokens don't match a 403 is returned.

Instructions for Reviewers
Must be tested with https://github.com/DSpace/DSpace/pull/3103

Review code
Test Authentication via Password
Test Authentication via Shibboleth
Perform some actions while logged in.  Ensure no 403 errors occur (this implies an XSRF mismatch/failure).

Checklist

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & specs/tests), or I have provided reasons as to why that's not possible.
[x] My PR passes TSLint validation using yarn run lint
[x] My PR doesn't introduce circular dependencies
[x] My PR includes TypeDoc comments for all new (or modified) public methods and classes. It also includes TypeDoc for large or complex private methods.
[x] My PR passes all specs/tests and includes new/updated specs or tests based on the Code Testing Guide.
[x] If my PR includes new, third-party dependencies (in package.json), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.

Just resolved the merge conflicts after merging #975 .  I also retested just to verify everything was still working & it's looking good.  Will merge on Monday.

Merging as the REST PR is at +2, and this is at +1 and flagged as 1 Approval.


[WIP] Pin DTLS for EOF errors

Pin DTLS that shouldn't leak upstream errors after a graceful shutdown

Codecov Report

Merging #949 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #949   +/-   ##
=======================================
  Coverage   76.33%   76.33%
=======================================
  Files          59       59
  Lines        4349     4349
=======================================
  Hits         3320     3320
  Misses        748      748
  Partials      281      281

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 303c9aa...85ad63d. Read the comment docs.


SST NextJS Deploy - Target is not longer supported

Hello everyone, when i try to deploy a NextJS 13 stack i received the next message

Build error occurred
Error: The "target" property is no longer supported in next.config.js.

isn't NextJS 13 supported yet?

Join us on the #nextjs channel on Discord. Next 13 is supported for SST v2, that's in RC.

Thanks @jayair i'll do it

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


Avoid plain "using PrecompileTools" in example

According to https://github.com/JuliaLang/julia/pull/42080

Doing using Foo in packages or code that you want to keep working with
updated dependencies is not recommended. The reason for this is if another
dependency starts to export one of the same names as Foo the code will
error due to an ambiguity in which package the name should be taken from.
Instead, explicitly list what names you want to use from Foo, for example:
using Foo: Foo, parsefile, readfile to get access bring Foo and two
functions parsefile and readfile into scope.

Thanks!


can not work in nextjs

When i used in Nextjs, it didn't work.

This is happening because of the recent upgrade of the styled-components library that react-chrono utilizes internally. A fix for this issue will be available soon. The fix will also resolve the warnings you are currently seeing on the console, which are unrelated to the broken timeline.

@Uni-h you can go ahead and try out now with the new release. also a few things to be noted while using the timeline in Next JS.
You need to disable the autoId generation via the new prop noUniqueId and pass your unique id via the uniqueId prop
Refer the props section for details and let me know if you need help
https://github.com/prabhuignoto/react-chrono#props


Move benchmarking config out of codebase to GCS

Please describe the feature you'd like to see
Currently, we have config kept in the Codebase due to which whenever we want to modify it we have to generate a new docker image.
Describe the solution you'd like
We can move the config to GCS and load it from there. This way any mods to config won't affect the docker image.
Are there any alternatives to this feature?
Open to suggestions.
Acceptance Criteria

[ ] While changing the config we should not generate a new docker image

When would we ever want to change the "benchmark config"?

It doesn't make sense to have the config both on github and GCS.

Closing this as discussed with the team that it is not required anymore


Add support HTTP2 [gRCP]

Hi, (jmeter-java-dsl) team.
Nice, project, thanks for doing that.
I am curious, do you plan to add support for gRPC samples?
I will try to implement some custom samplers for POC.
References
JMeter gRPC Request

Sorry, I mean two different features: HTTP2 and gRPC.


Add import for Capitec Bank's csv statement export

Capitec Bank provides statement exports of transaction history in several formats, including csv, OFX1.x and OFX2.x. Unfortunately the OFX1.x and OFX2.x exports both include ending balances that do not match what you see on the banking site. Luckily the csv export include the correct balance.

Fixed in release 1.0.3


Android libraries are not up to date

I noticed that on Maven Central the last available version is 0.23.0 while other packages are at the current version 0.26.0
https://central.sonatype.com/artifact/ai.djl.android/core/versions

0.26.0 for android is published


Allow for customizing prefix and destdir on 'make install'

this allows for easier packaging for linux, so distributions can do make PREFIX=/usr DESTDIR=${pkgdir} install, while still keeping /usr/local/bin as default

thanks!

Do you want to update Readme.md and command prompts?

i don't think they need updating, it's not a breaking change and everything should work as it did before, just with more options to choose from :)

i don't think they need updating, it's not a breaking change and everything should work as it did before, just with more options to choose from :)

OK.


Use http referer

When clicking on the t.d3fc.io link from a twitter status page, e.g.:
https://twitter.com/joel_w_smith/status/695201842968543232
... the server could potentially extract the status id from the referer http header and redirect to the associated t.d3fc.io status page (https://t.d3fc.io/status/695201842968543232) rather than the index page.

I like the idea but I'm not sure how to deal with the intermediate URLs that Twitter uses. E.g. if I click through to joel_w_smith's status, then the URL shows as https://t.co/nzZlSrVVnA. I'm pretty sure that means that the server will see that URL rather than the original status URL. Do you know a way to resolve that URL back to the status?

I'm not 100% sure you'd get the original referer url. And after posting this, I ended up realizing the feature  I really wanted is already there (the direct link in the tweet). So I'll just close this. This is a really cool project!

Thanks :)
On Tue, 3 May 2016 at 21:22 Michael White<EMAIL_ADDRESS>wrote:

Closed #67 https://github.com/chrisprice/t-d3fc/issues/67.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/chrisprice/t-d3fc/issues/67#event-649372779


Add bitsandbytes deps for hf baichuan2-13b models

Lovely, thanks so much! This was fault on our side for forgetting the larger models.


False positive for "duplicate hash key detected" on Yaml anchors

Describe the bug
Gitlabform reports >1 Yaml anchors as duplicate keys which is not true.
Such anchors are useful for re-using parts of configuration and are part of Yaml specification so it would be really nice if they were still allowed.
Example:
---
# ...

.default_confluence: &default_confluence
  confluence:
    confluence_url: https://....

.default_jira: &default_jira
  jira:
    active: true
    url: https://....

projects_and_groups:
  "group_name/*":
    <<: *default_jira
    <<: *default_confluence

Output:
Reading config from file: services.yml
ERROR:  Duplicate Hash key detected:
ERROR:     in "services.yml", line 74, column 5
ERROR:     found duplicate key "<<"
ERROR:     in "services.yml", line 75, column 5
Please try --help for more information.

GitLabForm version
Output of gitlabform -V
🏗  GitLabForm version: 2.10.0 = the latest stable 😊

GitLab version
Any (13.12 in my instance)

Hi @weakcamel!
It turns out that:
projects_and_groups:
  "group_name/*":
    <<: *default_jira
    <<: *default_confluence

...is not a great syntax  and you should use a syntax like:
projects_and_groups:
  "group_name/*":
    <<: [*default_jira, *default_confluence]

...instead. Please read more about it here and here.
While I personally think that the former syntax is easier to read and better for git blame view (separate line for each "include" entity) I agree with Anthon that the latter is more correct.
I suggest you to change your syntax then, but if we have more people affected by this problem perhaps we can consider making disallowing duplicate keys a feature that you can turn off.

Hello,
Many thanks for the prompt response.
It took me a while to find the right piece of specification (it's a separate document, not included in the general index) but here it is: https://yaml.org/type/merge.html and indeed, merge of multiple multiple dictionaries should be specified as a list.
Which is quite a surprise to me as the other syntax works perfectly fine with Gitlab CI Yaml files.
So yes, I agree it's not a bug.
On a side note, while digging I discovered that the status of merge key type in general is quite unclear: https://ktomk.github.io/writing/yaml-anchor-alias-and-merge-key.html - interesting times... :)

Whoa, thanks for sharing these @weakcamel! It's even more complicated than I thought! My initial thought is that this means that we should NOT switch gitlabform to YAML 1.2 then.

Closing this as it's not a bug. Please reopen if needed, @weakcamel.


Release Google.Cloud.ServiceManagement.V1 version 1.4.0

Changes in this release:
Bug fixes

BREAKING CHANGE Remove EnableService and DisableService RPC methods and related protos from service management API (commit d5cb1ab)

The breaking change here reflects the actual API; the EnableService and DisableService RPCs have not been available for some time. In other words, this change will not break any customer who isn't already broken in terms of making RPCs. (It may break mock/fake test code which expects those RPCs to still be available, however.)

Closing and reopening for allowing breaking changes.

Created release for Google.Cloud.ServiceManagement.V1-1.4.0


Rat dowsnt work when i click on it

So i clicked on the rat, created a bot with the same token, and clicked on the rat, nothing showed up, there was no updates. I didnt do it on a vm and did it on myself

real

any solutions?

@Cheoo21 try creating the whole thing again and try on either your pc or another one. sometimes it might take long for the bot to come online but eventually it will.

any solutions?

@Cheoo21 enable intentions in

here is the link : https://discord.com/developers/applications/1275104479871635500/bot

@Cheoo21 try creating the whole thing again and try on either your pc or another one. sometimes it might take long for the bot to come online but eventually it will.

tried that and it bypassed uac, but the bot didn't respond

its not off.

any solutions?

@Cheoo21 enable intentions in  here is the link : https://discord.com/developers/applications/1275104479871635500/bot

i tried that too but it didnt work..


Authorization app owners

PR for Authorize the user for post and update of the tasks #141 new PR

Sorry for too many commits , I will be raising a new PR once the tests get passed

CI failing... @harshith-venkatesh

@ankurnarkhede I will update the test cases today
Sorry for the delay

@harshith-venkatesh , this PR is pending since long. Can you check it?

@ankurnarkhede , I am finding issue in creating test cases, Can I send PR without 403 error test case?

Closing this PR on discussion with @ankurnarkhede and creating a new PR with CI being successful
@ankushdharkar