Alibaba Images names

Signed-off-by: Helene Durand<EMAIL_ADDRESS>What this PR does / why we need it:
The ubuntu and centos image names for Alibaba are hard-coded in https://github.com/kubermatic/machine-controller/blob/master/pkg/cloudprovider/provider/alibaba/provider.go#L46.
At the time ticket 7789 was created, both ubuntu and centos were failing.
Reproducing today and adding logs, the list of images we get is:
I0317 17:22:18.665019   17409 machine_controller.go:712]  image: Alibaba Cloud Linux  3.2104 LTS 64 bit Quick Boot-----------------------------------------------------
I0317 17:22:18.665041   17409 machine_controller.go:712]  image: Alibaba Cloud Linux  2.1903 LTS 64 bit Quick Boot-----------------------------------------------------
I0317 17:22:18.665048   17409 machine_controller.go:712]  image: Alibaba Cloud Linux  3.2104 LTS 64 bit-----------------------------------------------------
I0317 17:22:18.665052   17409 machine_controller.go:712]  image: Alibaba Cloud Linux 2.1903 LTS 64 bit-----------------------------------------------------
I0317 17:22:18.665057   17409 machine_controller.go:712]  image: Rocky Linux  8.5 64bit-----------------------------------------------------
I0317 17:22:18.665061   17409 machine_controller.go:712]  image: Ubuntu  20.04 64 bit-----------------------------------------------------
I0317 17:22:18.665066   17409 machine_controller.go:712]  image: OpenSUSE  15.3 64 bit-----------------------------------------------------
I0317 17:22:18.665086   17409 machine_controller.go:712]  image: Fedora  34 64 bit-----------------------------------------------------
I0317 17:22:18.665091   17409 machine_controller.go:712]  image: CentOS  7.9 64 bit-----------------------------------------------------
I0317 17:22:18.665096   17409 machine_controller.go:712]  image: Debian  10.11 64 bit-----------------------------------------------------

I have updated the centos image name to have a quick fix for this from CentOS  7.7 64 bit to CentOS  7.9 64 bit as currently returned by the client.
However, I'm not sure if it's the right fix to do.
Do we want to hard-code the image name ? Take any image name that contains Ubuntu or Centos ?
Allow to configure this in the UI? Then it needs more updates than just a machine-controller fix.
Which issue(s) this PR fixes (optional, in fixes #<issue number> format, will close the issue(s) when PR gets merged):
Fixes kubermatic/kubermatic#7789
Special notes for your reviewer:
Optional Release Note:

NONE

do I follow correctly that ubuntu has been actually working?

yes, right. ubuntu worked yesterday, with this image name.
Querying the client returned among the list, trial on dev.kubermatic.io did confirm it.
I0317 17:22:18.665061   17409 machine_controller.go:712]  image: Ubuntu  20.04 64 bit-----------------------------------------------------

/hold

/test pull-machine-controller-e2e-alibaba

pull-machine-controller-e2e-alibaba failed with error:
Creation of "amzn2-1-21-8-alibaba-1504719203247067136" failed, retrying: admission webhook "machinedeployments.machine-controller.kubermatic.io" denied the request: validation failed: invalid/not supported operating system specified "amzn2": os not supported

https://github.com/kubermatic/machine-controller/blob/master/pkg/cloudprovider/provider/alibaba/provider.go#L461
Do we support "amzn2" on Alibaba ? Changing code or tests ?

We don't, need to update the tests https://github.com/kubermatic/machine-controller/blob/master/test/e2e/provisioning/all_e2e_test.go#L756

/test pull-machine-controller-e2e-alibaba

@moadqassem , as discussed, only testing ubuntu.


Fix again flaky tests

Codecov Report

Merging #162 (9e8d092) into main (6e45529) will decrease coverage by 37.99%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main     #162       +/-   ##
===========================================
- Coverage   77.07%   39.08%   -37.99%
===========================================
  Files          33       34        +1
  Lines        2408     2410        +2
===========================================
- Hits         1856      942      -914
- Misses        411     1410      +999
+ Partials      141       58       -83

Flag
Coverage Δ

integration-test
?

unittests
39.08% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 32 files with indirect coverage changes


A program to explain Counters(subclass of dictionary) in Python has been added(with comments).

This is my personal repository ,so didn't adding the things which i have not done. You can create PR if you find any bugs in my code. Nice thing though, didn't knew about it

Wasn't expecting this. :+1:


Workshop to detail-out "Simplification / Ease-of-use" Epic

Description
We need the whole team to gather-up and brainstorm about the various areas in which we have to simplify OCM and improve the "ease-of-use".
Workshop will be virtual. Outcome of the workshop must be enough details and a clear priority of everything the team will invest in regarding the Epic (https://github.com/open-component-model/ocm/issues/390). A first start is already maintained in the Epic, which has to be updated after the workshop.

The workshop was conducted which lead to a series of actions items documented in epic #390


Send fluentd logs to stdout and make fluent-bit ignore them by default

Is your feature request related to a problem? Please describe.
Ease fluentd debugging
Describe the solution you'd like
To ease debugging of fluentd, send logs to stdout instead of a file.
Exclude them by default from fluent-bit collection.
Additional context
Related #1075

What would be a good way to implement this? How about having a dedicated label for marking certain pods for fluentbit to drop their messages?

Fluentbit has the feature: see the K8S-Logging.Exclude option and the associated fluentbit.io/exclude annotation. I have it in production right now and its fine :)
https://docs.fluentbit.io/manual/pipeline/filters/kubernetes#kubernetes-annotations

Created a PR to make the fluentbit side flag enabled by default: https://github.com/kube-logging/logging-operator/pull/1395
Although this is a breaking change in theory, I don't think this would affect anyone negatively in practice. I think this is enough to introduce for now and make the changes on the fluentd side later together with improvements around the fluentd internal log handling.

This behaviour is now enabled with 4.3 and logs for fluentd and syslog-ng will be excluded by default in 4.4, so moving the issue to that milestone


Dependency on datasets causing issue

I'm new to python and trying to run the examples. In doing so, I ran into an issue installing a dependency:
Collecting datasets
  Using cached https://files.pythonhosted.org/packages/cd/fe/4d2874473a753d59c83335691bd9532704f2605418a0d288a1d70fa003fc/datasets-0.0.9.zip
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/private/var/folders/_d/k4vvqhjd3lsgdrh93xw07fbh0000gn/T/pip-install-VOsrZA/datasets/setup.py", line 10, in <module>
        with open(path.join(here, 'DESCRIPTION.rst'), encoding='utf-8') as f:
      File "/Users/parker/tensor2/lib/python2.7/codecs.py", line 884, in open
        file = __builtin__.open(filename, mode, buffering)
    IOError: [Errno 2] No such file or directory: '/private/var/folders/_d/k4vvqhjd3lsgdrh93xw07fbh0000gn/T/pip-install-VOsrZA/datasets/DESCRIPTION.rst'

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/_d/k4vvqhjd3lsgdrh93xw07fbh0000gn/T/pip-install-VOsrZA/datasets/
(tensor2) MacBook:tensor2 parker$```

This error generally seems to be a configuration issue when it occurs with other packages, but other packages seem to work fine and I've tried a bunch of combinatoons of python (2.n, 3.6, 3.7 which has an incompatibility with tensorflow) and other libraries. It appears to just be an issue with datasets.

According to [this page](https://stackoverflow.com/questions/38163760/cant-install-datasets-package-via-pip), there's an issue with the datasets package itself.

What is the recommended way of pulling in and using this library?

Hey @pt -- I'm not familiar with the datasets package and its not an explicit TensorRec requirement. Do you know what component is trying to install datasets?

Perhaps I'm confused. It looks to me like the datasets library is a dependency for all of your examples. The library is easy to install, but the examples don't work. I should've been more clear in my message above.
More generally, I'm struggling to get this working for me practically (being new to TF/ML). It would be great to see a simple sample app that uses tensorrec to expose a REST api, and demonstrates batch processing + incremental model updates with large-ish data sets, which is realistic for most users.
Thanks for what you've built though. It's interesting and I'm sure if I understood how to use it it would achieve your goals.

Ahh -- the examples import a module within the project called test.datasets which is distinct from the python package datasets. It looks like the stack trace you posted is from an attempted installation of the python package.
You should be able to install all the dependencies you need using the requirements.txt and requirements_examples.txt files.
pip install -r requirements.txt
pip install -r requirements_examples.txt
Give that a shot and try running the examples. I'll leave this issue open if you hit any more roadblocks.

Will do, thanks.


Examples project doesn't even launch.

Your example project doesn't even launch.

What version of Xcode do you use?

Xcode 6.2. Just the latest public release version.

yes, unfortunately it does not work at all

upgrading to Xcode 6.3 and clean the build before compiling did helped..

@adamdahan, please use Xcode 6.3


Automatically add a comment to feedback and bug reports with a thank you message

We should have an automated way to reply to people that open up feedback discussions or bug reports in the community forum. This PR will add that as a GitHub Action that's triggered on a labeling event.

I thanks the community

Nice community

@Sa-su121's untitled project - Backlog.tsv.csv

this PR is good i think so

All us well

All us well

Is it immoral to ask if it's possible to learn hacking from the tech wizards on here? I have no tech knowledge but have been playing about with GBT's and have got some decent results. Being disabled, I can't leave the house so I'm at the computer all day, if I were to learn how to hack there's lots of employment for those. And of course, disclaimer, Anything learned or information given to help me will be used in a clean enviornment and NEVER for bad acting. I'm trying to learn red team hacking as I think I'd enjoy that more. Hopefully someone can help or point me in the correct direction

I don't get it, I've never used GitHub before.

Jakk626 I'm on the same page you are. Email me and we can learn together<EMAIL_ADDRESS>I'm a disabled mother of 4 and if I could make an income I could crate a better life for me and my 4 children.


Issue 1373: socket.SOL_TCP not in Jython, requires manual specification.

Minor change to manually specify SOL_TCP = protocol version 6 when starting serving.  This attribute does not exist in Jython and requires manual specification.
https://github.com/pallets/werkzeug/issues/1373
https://github.com/pika/pika/issues/67

There must be a better way to do what I'm doing.  Will revisit soon, have to make progress on other fronts for a bit.

The proper way to solve this is to just use socket.IPPROTO_TCP instead of SOL_TCP. Can you please change your pull request to use that?

Sorry for the delay. There were a few other locations I had to change to get this working as well. Will try using IPPROTO_TCP and resubmit next week (holiday).

I installed jython, socket.SOL_TCP and socket.IPPROTO_TCP both exist. Tried with OpenJDK 7 and 11, and Oracle JDK 7.
$ /opt/jython/bin/jython
Jython 2.7.1 (default:0df7adb1b397, Jun 30 2017, 19:02:43)
[OpenJDK 64-Bit Server VM (Oracle Corporation)] on java11.0.1
Type "help", "copyright", "credits" or "license" for more information.
>>> import socket
>>> socket.IPPROTO_TCP
6
>>> socket.SOL_TCP
6

Does indeed work out of the box with jython 2.7.1, thanks! jython.org offers 2.7.0 as the latest version, that's why I and most likely @jfreeland as well ended up using that.

Yeah, I noticed that Arch installed 2.7.1, but their site only lists 2.7.0. I'm OK with changing it to IPPROTO_TCP then.

We already merged a fix for AF_UNIX. Does using IPPROTO_TCP solve your issue? If so, just make a new PR with that fix.


[Feature] : parallax website

Description
Creating a parallax website
Screenshots
No response
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSoC'24 contributor.
[X] I want to work on this issue.

Code of Conduct

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

@Durgesh-Vaigandla please assign me the issue


logs: equivalent allele descriptions aa vs nucleotide

Kim: these are nucleotide descriptions (which can be slightly more informative because of degenerate codons). Can awe allow each allele to have an AA and an NT description?
We need to devise the way for checking equivalence of these, and store both (for now, this is not urgent if we just store the first one, and this is only a warning if)
Curators: What if we  display the AA one in PomBase by default, but make the nt one visible by mouse over if stored?
error in 58d8ad0dec113279: description for new allele "cdc10-K30(C1434T)" does not match the existing allele with the same name "cdc10-K30(T429I)"
error in 58d8ad0dec113279: description for new allele "cdc10-P2(A1209G)" does not match the existing allele with the same name "cdc10-P2(D354G)"
error in 58d8ad0dec113279: description for new allele "cdc10-M15(G1325A)" does not match the existing allele with the same name "cdc10-M15(A393T)"
error in 58d8ad0dec113279: description for new allele "cdc10-V50(C1232T)" does not match the existing allele with the same name "cdc10-V50(H362Y)"
error in 58d8ad0dec113279: description for new allele "cdc10-M71(G1325A)" does not match the existing allele with the same name "cdc10-M71(A393T)"
error in 58d8ad0dec113279: description for new allele "cdc10-I29(A1433G)" does not match the existing allele with the same name "cdc10-I29(T429A)"
error in 58d8ad0dec113279: description for new allele "cdc10-MBC4(C2268T)" does not match the existing allele with the same name "cdc10-MBC4(R707->STOP)"
error in 58d8ad0dec113279: description for new allele "cdc10-M47(G1607A)" does not match the existing allele with the same name "cdc10-M47(A487T)"
error in 58d8ad0dec113279: description for new allele "cdc10-M4(G1325A)" does not match the existing allele with the same name "cdc10-M4(A393T)"
error in 58d8ad0dec113279: description for new allele "cdc10-MBC41(C1434T)" does not match the existing allele with the same name "cdc10-MBC41(T429I)"
error in 58d8ad0dec113279: description for new allele "cdc10-K28(C1295T,C1705T,C1976T)" does not match the existing allele with the same name "cdc10-K28(P383S,R610C)"
error in 58d8ad0dec113279: description for new allele "cdc10-M17(G1325A)" does not match the existing allele with the same name "cdc10-M17(A393T)"
error in 16802f31036c5732: description for new allele "cut8-563(S201P)" does not match the existing allele with the same name "cut8-563(ntY1283C)"
error in 7bf893a86d94196c: description for new allele "cdc23-M36(D232G,V265I)" does not match the existing allele with the same name "cdc23-M36(A695C)"
error in 4d6014a904a1e84b: description for new allele "ned1-1(deletion)" does not match the existing allele with the same name "ned1-1(G402D)"
error in 4d6014a904a1e84b: description for new allele "ned1-1(deletion)" does not match the existing allele with the same name "ned1-1(G402D)"
error in 42b53662d27d7951: description for new allele "M6(C328A,C330A,C334A,C336A,C339A,H342A)" does not match the existing allele with the same name "M6(  C328A,C330A,C334A,C336A,C339A,H342A)"
Original comment by: ValWood

Need to fix: #481 first

Descriptions like this "cdc10-M71(A393T)" assume that there is only every one transcript for a gene.  To future-proof things, I'll make sure the code warns if there is ever more than one transcript.

I'm quite close to finishing this.
I'm a bit worried by examples like this:
"cdc10-P2(A1209G)" does not match the existing allele with the same name "cdc10-P2(D354G)"

Is the "1209" the base position in the mRNA sequence or the position in the coding bit?  If it's the position in the CDS then the numbers don't make sense - the position in the CDS should be around 1060, not 1209.
If it's the position in the mRNA that doesn't seem very future-proof.  What happens if the UTR length is modified or if a UTR is added to an mRNA?

Is the "1209" the base position in the mRNA sequence or the position in the coding bit?

It's supposed to be the position in the coding bit. If it doesn't match, that means someone made a mistake (could've been a curator or an author) that we'd like to fix, so we want a warning in the logs.
I don't think there's a separate ticket for log warnings (tho #131 is somewhat related); there's just a brief comment above from 2 years ago.

so we want a warning in the log

Great.  Thanks for clarifying.

Thanks to https://github.com/pombase/pombase-legacy/commit/8bf908e04c90525014fce6e30af5480c83f9250f we now have the transcript and protein sequences in Chado so this issue is easier to fix now.

We don't need to do this.  @mah11  will switch to amino acid and record the nucleotide in the "allele comment"


Salesforce Tool

This PR for merging the Salesforce Tool to main. Already tested (see screenshot). Working as expected.

Tool Tested: working

nice!! will take a look soon

@chrispangg out of curiosity do LLMs know how to write SOSL / SOQL / do you have a sample notebook showcasing this? asking because would love to help promote :)

Hey @jerryjliu - sure thing. Let me know if the examples in this notebook is sufficient, happy to add more if needed :)
There is a also some typos in the readme page. Will open another PR for that shortly.

@chrispangg awesome thanks! will share this

@chrispangg what's your twitter

All good! it's @Crispy325 on twitter 🙂

clutch thanks!


pyfolio installation problem: error: command 'cl.exe' failed: No such file or directory

When I was installing pyfolio by running 'pip install pyfolio' on windows, the installation has failed.
here is the cut for problem.
cl.exe /c /nologo /Ox /W3 /GL /DNDEBUG /MT -Ic:\users\cheurice\appdata\local\programs\python\python36\lib\site-packages\numpy\core\include -Ic:\users\cheurice\appdata\local\programs\python\python36\include -Ic:\users\cheurice\appdata\local\programs\python\python36\include /Tcbottleneck/src/reduce.c /Fobuild\temp.win-amd64-3.6\Release\bottleneck/src/reduce.obj -O2
error: command 'cl.exe' failed: No such file or directory
----------------------------------------

Command "c:\users\cheurice\appdata\local\programs\python\python36\python.exe -u -c "import setuptools, tokenize;file='C:\Users\cheurice\AppData\Local\Temp\pip-build-us9z0ixk\bottleneck\setup.py';f=getattr(tokenize, 'open', open)(file);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, file, 'exec'))" install --record C:\Users\cheurice\AppData\Local\Temp\pip-e_rrj4x9-record\install-record.txt --single-version-externally-managed --compile" failed with error code 1 in C:\Users\cheurice\AppData\Local\Temp\pip-build-us9z0ixk\bottleneck\
I've always used pip so this isn't something I'm familiar with so maybe I did something wrong. Any help and I'd be eternally grateful as I think I've exhausted all resources at this point.

same problem here, if you have find a solution, please, share it

try installing with --no-deps and separately installing empyrical with the same flag.

or better yet use conda

I found a solution yesterday actually.
The problem wasn't really with finding the cl.exe file path but in 'building' the bottleneck file, whatever that means.
To whoever has this problem, go to this page: https://www.lfd.uci.edu/~gohlke/pythonlibs/ , see in the error msg which archive wasn't built (in my case was bottleneck, but I saw people complaining about the twisted building error) and download it.
The variety of options to download is based on python version and 32-64bit versions.
Anyway, download the correct file and pip install it, close the cmd prompt window and oppen it again and then try to install pyfolio again. It worked for me.


v4 — Add space between last nav item and nav form.

Fix #17068
Example:
https://jsbin.com/yicazi/edit?html,css,output

X-Ref: #10825, #17274

We'll want to use spacing utilities here I imagine instead of conditional last child stuff. This kind of shiz got us in a lot of trouble with the v3 navbart.


Update Dockerfile

@idsvnasia Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


kexec_load failed: Invalid argument

Describe the bug
22 blocks
~/kexec/kexec/tmp.1FvK0gLD2M
+ popd
+ cat extra.gz
+ rm -r /root/kexec/kexec/tmp.1FvK0gLD2M
+ /root/kexec/kexec/kexec --load /root/kexec/kexec/bzImage --initrd=/root/kexec/kexec/initrd --command-line 'init=/nix/store/nf32aa02c3nj9qmd391vp8li1g9sazrx-nixos-system-nixos-22.11pre-git/init console=tty0 console=ttyS0,115200 nohibernate loglevel=4'
kexec_load failed: Invalid argument
entry       = 0x3ffd0760 flags = 0x3e0000
nr_segments = 5

To Reproduce
Tried to use https://github.com/tfc/nixos-anywhere-example/blob/main/single-gpt-disk-fullsize-ext4.nix with
hetzner-cloud flake on a QEMU vm
nix run github:numtide/nixos-anywhere --<EMAIL_ADDRESS>--flake .#cloud-vm

System had not enough ram, so a swapfile was created.
Also, I'm running this command from aarch64-linux (on a mac with Rosetta enabled, previously tested with building various x86_64 systems with no issues)
Expected behavior
Nixos installed
System information
Linux v7777777.local 5.15.0-43-generic #46-Ubuntu SMP Tue Jul 12 10:30:17 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Afair swap files wont survive kexec so they won't help against not enough ram

Maybe make "swap does not count" bold in README :)
Closing then.


Cleanup and modernize the wrapper

Various modernizations and cleanups to the OSQP mex function and wrapper class.

Improve readability and maintainability of the Matlab class by breaking functions into separate files
Simplify code sections for getting OSQP constants
Expose rho update in a nicer way
Introduce a better and more automated wrapper for the various structs used by OSQP
Introduce wrappers for copying/cloning vectors cleanly

The only confusing thing in my opinion is that the installation instructions in the readme file are not updated.

Are you refering to the ones linked to from the readme that are hosted online? We need to update those in the main OSQP repo to fix that once this is all merged.

The only confusing thing in my opinion is that the installation instructions in the readme file are not updated.

Are you referring to the ones linked to from the readme that are hosted online? We need to update those in the main OSQP repo to fix that once this is all merged.

Both it and the instructions in the package directory.

Both it and the instructions in the package directory.

Yea, the packaging script needs a rework to do a package builder to make distribution easier.


Add a small doc detail for BackdropFilter

#10148

@collinjackson since you understand this best could you file a bug for it then close this PR? Thanks!

Will do! Thanks

@collinjackson Did you ever have a chance to file that bug? (I'm just going through cleaning up old PRs.)

Here is a test case app.
The following are the bugs that I will file on Skia.

BackdropFilter bleeds outside its bounds when text is drawn on it. It expands to cover all the layers below it in a Stack (which can expand out to include the entire screen) if it's not contained by a ClipRect.

BackdropFilter does nothing if the contents are clipped with ClipOval. (I think ClipRRect has this problem too, but ClipRect is fine.)

BackdropFilter does nothing if it's contained in a ClipOval. (I think ClipRRect has this problem too, but ClipRect is fine.)

This seems like a bug somewhere, either in Flutter or Skia, so let's close this.

Here's the Skia bug I filed

Awesome! Thanks for testing.
I can confirm seeing all 3 issues when I was testing initially (with ClipRRect). I was definitely surprised by the text behaviour too.


Does SQL Server 2019 license comes free with windows server 2019 essentials or do i have to buy a seperate License for it?

I want to know if Windows Server 2019 Essentials comes with SQL server 2019 (License) or not?

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

ID: c09beb1a-92a9-4335-511c-395ba64c875b
Version Independent ID: 8f43fca3-43d7-a2e4-1469-bf17f14d9ac8
Content: SQL Server 2019: Hardware & software requirements - SQL Server
Content Source: docs/sql-server/install/hardware-and-software-requirements-for-installing-sql-server-ver15.md
Product: sql
Technology: release-landing
GitHub Login: @MikeRayMSFT
Microsoft Alias: mikeray

@Harshitjain188 -- thank you for your question.
@MikeRayMSFT -- please look into this issue, as I haven't found a definitive answer.

Windows Server Essentials (formerly Small Business Server) doesn’t include any SQL Server license. Licensing topics aren’t covered in the product technical documentation. To find out more about SQL Server licensing, please see https://www.microsoft.com/en-us/sql-server/sql-server-2019-pricing.

Thank you, @pmasl


Allow configuring the tls-configuration-name for the Model providers using the REST client

We have a case where the remote model provider uses a self-signed certificate (yes, yes, I know.... 2024). It would be great to have the possibility to configure the tls-configuration-name and allow the loading of the TLS configuration for the Quarkus TLS registry.
I mentioned in the title model providers, but anything remote should offer that possibility.

Interesting. We can certainly do this

@cescoffier I assume https://github.com/quarkiverse/quarkus-langchain4j/pull/1006 is what you are after?

yes, exactly.
We have a case where we need to connect to a model server using self-signed certificates.

👍🏽

You'll love the hack I had to come up with the make OpenAI work...
It still needs more work, so I'll finish the PR tomorrow.

PR is now ready to go


CIV-9039 generate judgment by determination

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/CIV-9039
Change description
Generate judgment by determination
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

bulked with 11119


build CHANGE support libyang custom location

Hello,
I had an issue while trying to use cmake with libyang installed in a custom location because check_symbol_exists("NC_ENABLED_SSH" "nc_client.h" LIBNETCONF2_ENABLED_SSH)
check_symbol_exists("NC_ENABLED_TLS" "nc_client.h" LIBNETCONF2_ENABLED_TLS)
would fail with a 'file not found error'.
This fixes it.
Kind regards,
Paul.

Hi,
I cannot really merge this because it expects that libyang is searched before libnetconf2 and in general this does not seems like a solution, rather a workaround at best. Also, your patch duplicates existing code so I do not really understand why it fixes anything. Could you please investigate more?
Regards,
Michal

Hi,
This is a fix for an error that has impacted me as well consistently for two years now. CMake reports that SSH support is not enabled in libyang when in fact it is. The technical problem is that the check_symbol_exists() directive looks for the define symbols, but fails to validate the nc_client.h header itself because the header include chain goes down to the libyang.h header which it cannot find because it doesn't look in the custom location of libyang. Here is an excerpt from the CMakeError.log that @dumitrupaul actually found:
In file included from /opt/libnetconf2/include/nc_client.h:32:
/opt/libnetconf2/include/libnetconf2/session_client.h:22:10: fatal error: 'libyang/libyang.h' file not found
#include <libyang/libyang.h>

The way I would solve this is just remove those checks. And it would compile just fine with SSH support. But it is obviously not the way to go.
It's easy to replicate, remove any system-wide installation, install in /opt/libyang, point sysrepo to /opt/libyang. Could you check?
I don't know why the existing line doesn't solve it, maybe the path resets on find_package(LibNETCONF2 REQUIRED)
I guess most people always install in the default prefix or set C_INCLUDE_PATH or run in docker, I don't understand how this flew by unnoticed.
Best regards!

Hi,
okay, I have checked this and it worked fine for me on Ubuntu 18.04. I installed libyang/libnetconf2/sysrepo into /usr/local1 and so I had to always manually specify all the paths. But when I did, netopeer2 configured and compiled just fine. So, you will have to provide more details if you want this fixed.
Regards,
Michal

Hi,
Your dockerfile[1] works.
Dockerfile modified to install in custom paths[2] fails with:
CMake Error at CMakeLists.txt:56 (message):
  Missing SSH and TLS support in libnetconf2, CLI requires at least one them
  to be supported.

docker build -t sysrepo -f ./Dockerfile-opt .

[1] https://github.com/sysrepo/sysrepo/blob/master/deploy/docker/sysrepo-netopeer2/Dockerfile.devel
[2] https://github.com/andrei-pavel/showcase/blob/master/sysrepo/Dockerfile-opt

If you want to further replicate locally, I suggest removing other libyang, sysrepo installs and unsetting C_INCLUDE_PATH & CPP_INCLUDE_PATH.

Hi,
so I have added another explicit check that caused printing an inaccurate message in my case. But it worked the way it should have even on another system I tried it on (OpenSUSE Tumbleweed). So, please provide exact cmake commands for libyang, sysrepo, libnetconf2, and finally netopeer2-server so that I have a chance of reproducing this. docker cannot really be used for this because in my experience it often executes different commands than I expect so I am not able to debug anything with it.
Also, I will rewrite our cmake find modules a bit soon to be compliant with all the standard ones but I do not think it will fix this problem.
Regards,
Michal

Hi,
The commands from the Docker are exactly what we use locally. Even the export LD_LIBRARY_PATH is used locally to get over a SYSREPOCTL-NOT_FOUND in /opt/dev/Netopeer2/setup/install.sh just like in docker. We'll maybe come back to that later. Please try those. We are using Ubuntu 18.04 and 20.04.
I'm aware that I give the cli some extra arguments that are not used, but that's only so that they are the same as for the server and it shouldn't impact the issue.
Regards!

Okay, I have found out what the problem was. Not sure what issue exactly is this pull request solving but it should have been explicitly said that it is only the CLI that fails to be configured, the server works fine. I have been testing it only with the server. So, few hours wasted and I will not even fix it because CLI cmake list will be a sublist of the main, server one. Meaning all the defined variables will simply be reused.
Regards,
Michal


Background music constantly loops, even when loop is unchecked in 3.1 alpha0 (1847)

Describe the bug
Play music track event will loop music constantly in the exported game, even when "loop" is not checked.
To Reproduce
Steps to reproduce the behavior:

Create a scene
Add "Play music track" event in On init tab
Ensure "Loop" is not checked
Export game. Music will loop.

Expected behavior
Music doesn't loop / plays only once.
Platform (please complete the following information):

OS:  macOS 10.14.6
App version: GB Studio 3.1 alpha0 (Pipeline: 1847)

Additional context
Music Driver: GBT Player (mod file player)/

At least, you do have a loop checkbox.
I don't:

Platform:

OS: macOS 12.5
App version: GB Studio 3.1.0

Additional context:
Music Driver: GBT Player (mod file player)

At least, you do have a loop checkbox.
I don't:
Platform:

OS: macOS 12.5
App version: GB Studio 3.1.0

Additional context: Music Driver: GBT Player (mod file player)

I used the Alpha-Version (GB Studio 3.1 alpha0 Pipeline: 1847). It seems in the release version the Loop-Checkbox was removed.
I think the expected workflow now is to modify your music files so they only play once if you need that feature. You can do so by creating a jump to and empty pattern and that pattern contains a jump to itself.
I am personally not a fan of this workflow, as I think it wastes resources with keeping the player running - but playing 'empty' music. On the other hand I don't have the insight on the technical difficulties to stop the player and likely to restart it later....

According to the docs, we should be able to enable/disable loop mode for music.
I've tried to launch my music with GBVM script rather than with the GUI:
VM_MUSIC_PLAY ___bank_song_music_rulz_intro_Data, _song_music_rulz_intro_Data, .MUSIC_NO_LOOP
`
``

But it's still looping...

So it seems this is not an expected behavior.

that feature only works with the legacy GBT Player driver.

@untoxa I do use the GBT Player :(

so, that option is not for you.

that feature only works with the legacy GBT Player driver.

I do use GBT Player.

oh, yes, sorry. i forgot, that feature is prepared for being removed, because the new driver has no support for that. loop parameter has no effect since the point of merge of the new SFX engine. it was left only for the compatibility with the GBStudio codegen, because there is a lag between the core and IDE development.

I think the expected workflow now is to modify your music files so they only play once if you need that feature. You can do so by creating a jump to an empty pattern and that pattern contains a jump to itself.

or you may use the music routine to switch to the next song or switch music off, for instance.

I don't understand what routines are, and it seems they're related to .uge files (so not compatible with the legacy driver)

Attach a script to one of the four music routines that can be triggered from a .uge file. In the music editor you are able to use the call routine effect in your songs to trigger these scripts in time to music.

for the legacy driver infinite looping is the only way

So sad...
But thanks for the help :)


feat: transactions/fees endpoint

Adds the transactions/fees endpoint.
Also fixes the failing unit tests for the transaction/types endpoint.

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Refactor
[ ] Performance
[ ] Tests
[ ] Build
[ ] Documentation
[ ] Code style update
[ ] Continuous Integration
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Does this PR release a new version?

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] It's submitted to the develop branch, not the master branch [develop is outdated]
[x] All tests are passing
[x] New/updated tests are included

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Thanks for submitting this pull request! A maintainer will review this in the next few days and explicitly select labels so you know what's going on.
If no reviewer appears after a week, a reminder will be sent out.


suggestion, windows portable version

a portable version would be good, to put it in a usb

@carlosnewmusic The CI preview (x86_64 or ia32 ; see the README) functions largely as a portable version provided you have the .NET and Visual C++ redistributables installed.


Possibility of implementing new engines

Is the project open to receive new engine implementations?
I thought about adding a feature flag to enable / disable engines.
New Engine Suggest:

[ ] Google Workspace
[ ] HubSpot
[ ] Outline - Open Source Knowledge Base for Teams
[ ] Google Calendar
[ ] Gmail
[ ] Google Vault
[ ] New Relic
[ ] BambooHR
[ ] Lever
[ ] ToriiHQ
[ ] Sentry
[ ] Opsgenie
[ ] Thinkfic LMS
[x] GitLab (done #19)
[ ] BitBucket
[ ] CircleCI
[ ] TracisCI
[ ] Satismeter
[ ] Outlook
[ ] Fyipe - Monitor Services and Log Aggregation
[X] Mattermost (done #20)

Yes, PRs are welcome! Users can disable an engine by simply deleting it from config.yaml if that's what you mean.

Hello, very interesting project !!!
Is there somewhere we can vote for new search engines ?
I am thinking gmail, Outlook and other mail services could be 👍🏻

@guillim It's a bit hard for me to find time to set up a test account, populate it with dummy data, write the engine code etc for services that we don't actually use or plan to expose internally here at Duolingo, but it should be straightforward to add support yourself for any service you'd like! To date I've never vetoed such a PR :)

Adding onto this, an email engine would be real dandy.

Google or Apple Calendar would also be super useful

Would it be possible to add support for images on Trello and Nuclino? I've already tried implementing these, but I am not good at writing TypeScript, unfortunately.

I've never used either of those (@jimmatthews was the one who added Trello support) but certainly happy to accept PRs from anyone for those features

Alright, got it. If I can figure out how to work with Typescript better, I will make a PR. I've always used vanilla JS/HTML/CSS.

https://github.com/duolingo/metasearch/issues/32
https://github.com/duolingo/metasearch/issues/33

Cool project! You are welcome to reuse anything from Quest, (e.g. code to fetch mails from gmail)
Similar idea, but not enough time and motivation to maintain it.


Overridden App Stays open in background

I have mapped Bixby to Alberto's Google Assistant. If I swipe back or press the back button in Google Assistant, Bixby starts to load and opens Google Assistant again.
Is there possibly a function that can be added to force close the app being overridden before opening the target app?
PS. Great app and thank you

I have mapped Bixby to Alberto's Google Assistant. If I swipe back or press the back button in Google Assistant, Bixby starts to load and opens Google Assistant again.
Is there possibly a function that can be added to force close the app being overridden before opening the target app?
PS. Great app and thank you

How did you use it for albertos google assistant?

I used com.alberto.Assistant/com.alberto.Assistant.AudioRecordActivity as the target.

It happened to me as well. I mapped Bixby to Google Assistant and Bixby stayed in the background. Same happened to me with Samsung Pay and Google Pay

Should be fixed by the new detection method, see: https://github.com/BigBoot/GW4Remap/issues/4#issuecomment-1158188672

I used com.alberto.Assistant/com.alberto.Assistant.AudioRecordActivity as the target.

How do you know which activity do you write in? Like, for my Assistant "AudioRecordActivity" doesn't work. Though "MainActivity" seems to work, but it doesnt record voice right after it is opened.

Should be fixed by the new detection method, see: #4 (comment)

Thanks, @BigBoot but still the same with the v2.0


Hotfix/about

Proposed changes

Types of changes

[ ] New feature (non-breaking change which adds functionality).
[x] Enhancement (non-breaking change which enhances functionality)
[ ] Bug Fix (non-breaking change which fixes an issue).
[ ] Breaking change (fix or feature that would cause existing functionality to change).

Checklist:

[x] I have read the README document.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

@amaehr Please approve changes asap. Thanks a lot.

@maehr Anything else to review?

@maehr Anything else to review?

No, thank you very much.


mock-api: wrong content-type

twitch-cli version: twitch-cli/source (9370e1a9)
curl -X GET "http://localhost:8080/units/categories" --header "accept: application/json" -v

sends data with header Content-Type:  text/plain; charset=utf-8
curl -X POST "http://localhost:8080/auth/token?client_id=mockid&client_secret=mocksecret&grant_type=client_credentials" --header "Accept: application/json" -v

also sends data with header Content-Type:  text/plain; charset=utf-8
curl.exe -X GET "http://localhost:8080/mock/users" --header "Accept: application/json" --header "Authorization: Bearer mytoken" --header "Client-ID: mymockid" -v

returns Content-Type: application/json correctly

Meant to re-add this; will fix in the coming week or two.


[Regression] [iOS] ViewInsets in not updated when keyboard gets opened master, 3.20.0-7.0.pre.17

Steps to reproduce
Run this sample
https://github.com/delfme/flutter-native-text-input-repo
Expected results
The bottomBar should move up when keyboard is opened
Actual results
The bottomBar doesnt move up when keyboard is opened coz viewInsets bottom is zero.
Issue occurs only on iOS.
Code sample
https://github.com/delfme/flutter-native-text-input-repo
Screenshots or Video
Video below:
https://github.com/flutter/flutter/assets/53510751/433fbad2-d659-476f-bcd6-3112ee3fd49d
Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
 Flutter (Channel master, 3.20.0-7.0.pre.17, on macOS 14.2.1 23C71 darwin-arm64, locale en-IT)
[!] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.
[✓] Xcode - develop for iOS and macOS (Xcode 15.0)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.2)
[✓] Connected device (4 available)
[✓] Network resources

I can reproduce the regression. It is caused by https://github.com/flutter/engine/pull/50364. Looking into reverting the fix.

@TahaTesser thx for the quick fix.
Would you be so kind as to also have a look at some jerking that was used to occur during animation? Basically, when viewInsets animates the UI while keyboard raises up, there was some jerking due to the fact that flutter animation and native animation can get out of sync at times.
No idea if that issue is still reproducible on master.

Would you be so kind as to also have a look at some jerking that was used to occur during animation?

That's an unrelated issue. There have been such reports.


Prometheus http listener local conflict

When running workers and server locally the prometheus service will try to listen on the same port which creates a (crashing) conflict.

Setting address or port in the prometheus to empty or None will not start the listeners. Will add an easier enable switch later through this PR.
https://github.com/google/turbinia/pull/648


Source selection is not filtering when ) is in source name

The source selection in image creation wizard is not filtering the items properly if the source name has a ) in the name, see screenshots.

Ok, found the issue, it is in PF4 itself.
https://github.com/patternfly/patternfly-react/blob/63b1d08cf7c4f02d32d5473db74f6f7ac0af88af/packages/react-core/src/components/Select/Select.tsx#L461
When the input value is a valid regex, it's treated as a regex, so I guess in case with My (source) we would need to escape the input as My \(source\) ... workaround would be to implement the filter ourselves.
It would not be easy to get a fix in patternfly given they do not support this version anymore.

Is it a bug in patternfly or a valid behavior ? Just by a the look of it it could be both IMO, the code tries to check if the text is a valid regex pattern and to use it that way and if not to escape all regex special characters.
The question is, do we want to let the user use regexs in the UI or not ?
If not, we could simply escape the string before it goes to patternfly ?

I think also that it's kinda neat to have the ability to filter on using a regex :)

We would need to implement the search ourselves completely, there is no hook into the default implementation, either we reimplement, or we use default :)
I think it's a feature, but should be documented :stuck_out_tongue:


TinyBenchmark/TinyMMLU broken?

There was a commit 21 hours ago that deleted some of the parquet files from the TinyMMLU dataset on the hub (https://huggingface.co/datasets/tinyBenchmarks/tinyMMLU), since then the TinyBenchmark task fails:
datasets.exceptions.DataFilesNotFoundError: No (supported) data files found in tinyBenchmarks/tinyMMLU

I've tried changing the dataset_name from 'all' to 'default' as suggested in the updated readme on the dataset page but that didn't help. I've also tried to pin a previous commit via the revision passed by dataset_kwargs but that still lead to the same issue.
Should either of those approaches have worked or does the dataset owner need to do some further changes to make this compatible again?

Hi @LucWeber ! do you happen to know what's going on with changes to the tinyMMLU dataset?
@skramer-dev I'm surprised that passing a revision through dataset_kwargs doesn't fix this. would you be willing to dig in a bit just to triple-check we are indeed passing that kwarg to the download method as expected?

A few things happened, there were a load of commits over the weekend and i ultimately got it to work again today. I'd have to look at the exact configuration tomorrow but i think i both removed the dataset_name and the fewshot_split fields from the config yaml.
I also believe that there were some sort of caching issues with the way i executed the run code in our cloud evironment. I added a print to get the job config that is passed down to datasets and saw some values there that weren't set in the yaml of the respective task so the fact that the revision part didn't work could have come down to that. I'd have to check it again with the print of the assembled config to see if it indeed takes the revision kwarg and does something with it or not.

Hey both,
thanks a lot for pinging me, we will look into this!

I think tinyMMLU should run as usual again. Let me know if there are still any problems @skramer-dev

I think tinyMMLU should run as usual again. Let me know if there are still any problems @skramer-dev

I've used the original config now and it works again 👍


update Command line options docs on README.rst

the "Command line options" section on docs is deprecated.
this pull-request update the docs to same output of pylama==6.3.1

Thank you!


Industry critical minerals

@danielhuppmann, @macflo8,
Following the discussion here about the production and extraction in the industry,
https://github.com/IAMconsortium/common-definitions/pull/105#issuecomment-2268664017
I have created a pull request on production of critical minerals.
I am not a critical minerals expert, so your feedback is greatly appreciated. I took the list of critical minerals from the IEA reports and websites.
https://www.iea.org/topics/critical-minerals
I have to admit I am a newbie with GitHub. If I have not followed the GitHub best practice, please let me know.

This looks good to me, but I'd be happy for comments or approval from @IAMconsortium/common-definitions-industry @volker-krey

Thanks @masahirosugiyama for opening this pull request. Adding critical minerals to the template is very relevant. However, I would not classify the materials as critical minerals as this classification might change over time and also different regions and countries may use different definitions. Using a more stable classification would be useful, for example, Nickel, Copper, Manganese, Lithium are non-ferrous metals and the energy and emissions associated with their production and processing would thus have to be reported under the respective non-ferrous metals sector. Copper and Lithium are actually part of the following PR. Hope this makes sense.

More broadly, I would suggest to integrate the missing 'critical minerals' dimensions into the structure of #154 after that has been merged.

Closing in favor of #157, which adds the volume-variables for the three critical minerals that weren't added by #154. Per the comment by @volker-krey, better to not add a monetary-value equivalent (for now).


Update version in README

Signed-off-by: Yoann Rodière<EMAIL_ADDRESS>

We're nearing the one-year anniversary of this PR... should I close it? :)

Thanks!


object().members() accept properties that are not defined

Package version
5.9.0
Node.js and npm version
node: 16.15.0 , npm: 8.5.5
Sample Code (to reproduce the issue)
Schema
{
     profile: schema.object().members({
        username: schema.string(),
        avatar_url: schema.string()
      })
}

Request Payload
{
     profile: {
        "asdfasd": "afasd",
        "username": "asfasd",
        "avatar_url": "asfsad"
     }
}

This validation passes and doesn't throw an error even if asdfasd is neither defined nor anyMembers is used.

Hey @technoknol! 👋🏻
This is the expected behavior.
The validated data will not have the properties you have not defined in your schema.
const data = {
  username: 'romain.lanz',
  avatar_url: 'http',
  wrong: true
}

const validatedData = await validator.validate({
  schema: schema.create({
    profile: schema.object().members({
      username: schema.string(),
      avatar_url: schema.string()
    }),
  }),
  data,
});

Here, validatedData will not have the wrong property.


Subtitles not displayed

subtitles not showing i try at chrome firefox and with my phone (Android)
here is the code im using
https://pastebin.com/XYXZrtjh

The src attributed of subtitles <track> needs to point to actual existing subtitles file. Something like https://your_website.com/english.vtt.


Broken Formatting & Lost List Box Upon Opening and Saving Workbook

Hey there, firstly thank you so much for this amazing library. I find it very enjoyable and ergonomic to use.
I did discover that just opening and saving my workbook caused some trouble when text used multiple font sizes within a single cell and there was a list box present.
Here's the code:
use std::{path::Path};

fn main() {
    let path = Path::new(r"C:\Users\Fots\Downloads\Book.xlsx");
    let mut book = umya_spreadsheet::reader::xlsx::read(path).unwrap();

    let path = Path::new("out.xlsx");
    umya_spreadsheet::writer::xlsx::write(&book, path).unwrap();
}

And here's a sample spreadsheet.
Book.xlsx
Here's how the original looks:

And how it looks after being opened and saved with the library:

Thanks heaps in advance!
Fotis
P.S.: I can confirm that other developer controls such as buttons are lost too.

Thank you for your report.
We will proceed to investigate and address the cause of the problem.

Fixed a bug related to text values.
The developer control object is currently not supported.
We will try to support it in the next update.


[v2.4][Backport] Bump memory limit for pipeline Minio

Backport for this PR: https://github.com/rancher/rancher/pull/28166
Related Issue: #28025

related to: https://github.com/rancher/rancher/issues/27467


feature request: support client-managed healthchecks (livenessProbe, readinessProbe) and imagePullPolicy

This would solve one of the important use-cases - a self-updating image through livenessProbe.
With the livenessProbe pointed to query the releases link comparing against the current version would allow it to fail once there is a new image available, causing the pod restart allowing it to self-update. This requires imagePullPolicy: Always set for this specific deployment.
Add client-managed healthchecks (livenessProbe, readinessProbe and their parameters) and setting imagePullPolicy to Always.
Refs
https://discord.com/channels/747885925232672829/1090670585555730563/1091261402729156608
https://github.com/akash-network/provider/pull/54
https://github.com/akash-network/support/issues/50
https://github.com/arno01/self-update

PoC

Dockerfile

FROM nginx
RUN echo ver1 > /usr/share/nginx/html/index.html

pod.yaml

---
apiVersion: v1
kind: Pod
metadata:
  name: pod1
spec:
  containers:
  - name: container1
    #image: nginx
    image: andrey01/testimage
    imagePullPolicy: Always
    livenessProbe:
      httpGet:
        path: /health2
        port: 80
      initialDelaySeconds: 5
      periodSeconds: 10
      timeoutSeconds: 15

@anilmurty would this be a good experiment for us to create a bounty for the community.  Currently, this is needed for Pre-Search.
Andy has more info on it, and link attached to the convo in our discord channel with them.

potentially but first we need to clean up the local dev env set up (readme on the github repo) for node and provider repos, so that others can get set up (we did this for console before engaging OSS dev community)


Parse Fontra glyphs as far as contours

https://codepen.io/rs42/pen/qBvoaLy shows the first outlines ever loaded from fontra by this code.
We have yet to reach the interesting (varc) bits. Baby steps.

TODOs without links to issues

Added two links


Sagemaker Studio Lab: An error occurred. Please try again later.

Describe the bug
The lab have a error occur and unable to connect to my project runtime.
To Reproduce
Steps to reproduce the behavior:

Log into my account
See error message: "An error occurred. Please try again later."
(I tried log out and re-login again, still the same.)

Expected behavior
I expect I can load and start the project runtime on the web.
Screenshots

[Update] I tried signout and signin again, it become the below:

I waited another 15min, the runtime is still preparing

Try to re-create the account. You can delete the account and re-create it.

Please try again and tell us if you are still having any issues.
I don't think you need to delete account and recreate.

Update: it is auto fixed after some hours

We are glad to hear that. Let me close this issue for now.
Please reopen it if further assistance is needed. Thanks.


Update Resources table

Make new PK (Name + RoleId)
It's allow situation when DB contain two resources with the same name, but with different role.
Example:
Roles:

AdminRole
SuperAdminRole

Resources:

Name=OnlyAdminResource, RoleId = AdminRole
Name=OnlyAdminResource, RoleId = SuperAdminRole

In this example, we allow only AdminRole and SuperAdminRole, to call OnlyAdminResource
Also, need to update ResourceManager in overrides which receive IEnumerable<TResource> resources need to check if these resources has the same name. This move need to prevent this situation:
Roles:

UserRole
AdminRole

Resources:

Name=PublicResource, RoleId = UserRole
Name=AdminResource, RoleId = AdminRole

User call PublicResource and AdminResource with override which receive IEnumerable<TResource> resources and receive true because least one resource allow (in this situation allowed PublicResource)

But the above solution can be an incorrect way.
I see better way how to implement new logic:
Create overrides which can receive resources and instead of return boolean value true/false return array of resources which be allowed.


Disconnecting Many-to-many self relation will delete all relations

Describe the bug
I'm implementing a following feature using many-to-many self-relation,
let's say X is following Y and Z
when I disconnect X/Y, X/Z got disconnected to
To Reproduce
Steps to reproduce the behavior:
this is my schema
type User {
  id: ID! @unique
  follower: [User!]! @relation(name: "UserFollow")
  following: [User!]! @relation(name: "UserFollow")
}

first connect X/Y and X/Z
mutation {
  updateUser(data:{
    following: {
      connect: [{ id: "Y" }, { id: "Z"}]
    }
  },where:{
    id:"X"
  }) {
    id
  }
}

then disconnect X/Y
mutation {
  updateUser(data:{
    follower: {
      disconnect: [{ id: "X" }]
    }
  },where:{
    id:"Y"
  }) {
    id
  }
}

this is what is got from MySQL log
delete from<EMAIL_ADDRESS>  where<EMAIL_ADDRESS>= 'X'

so it deletes all relations that X is following
Expected behavior
delete only the relation between X/Y
Versions (please complete the following information):

OS: arch linux 4.17.8-1
prisma CLI: prisma/1.12.0 (linux-x64) node-v9.11.1
Prisma Server: 1.12.0

Post - Category  many-to-many
Datamodel
`
type Post {
id: ID! @unique
createdAt: DateTime!
updatedAt: DateTime!
title:  String!
content: String!
isPublished: Boolean @default(value: true)
categories: [Category!]!
}
type Category {
id: ID! @unique
createdAt: DateTime!
updatedAt: DateTime!
title:  String!
posts: [Post!]!
}
Mutation
mutation {
updatePost(data: { categories: { disconnect: [{ id: "CAT2"}, {id: "CAT5"}, {id: "CAT8"}]  }, where {id: "POST1"} })
}
`
exemple
_PostToCategory
before update
|  id  |     A      |       B
|  ....  |  Post1  |   CAT1
|  ....  |  Post1  |   CAT2
|  ....  |  Post1  |   CAT5
|  ....  |  Post1  |   CAT8
|  ....  |  Post2  |   CAT4
|  ....  |  Post2  |   CAT5
|  ....  |  Post2  |   CAT8
after update
|  id  |     A      |       B
|  ....  |  Post1  |   CAT1
|  ....  |  Post2  |   CAT4
Versions :
OS:  windows 10
prisma CLI: prisma/1.12.0, Mysq/5.7, node-v8.11.1
Prisma Server: 1.12.0
Docker Community Edition: 18.06.0-ce-win72 (19098)


Consider adding the latest released package version display to Prepare-Release

The original version had it and it's very useful for patch releases. As currently preparing a patch release requires manually going to NUget to figure out the latest version.
.\eng\common\scripts\Prepare-Release.ps1 Azure.Extensions.AspNetCore.Configuration.Secrets
Package Name [ Azure.Extensions.AspNetCore.Configuration.Secrets ]
Source directory [ extensions ]

Assuming release is in May with release date 05/14/2021
+Latest released version: 1.0.2
Input the new version, or press Enter to use use current project version '1.1.0-beta.1': 1.

@weshaggard @chidozieononiwu

In order to do this in a general way we would need to figure out how to find the version from the package manager or maybe we could depend on the release tags which is what we do in other automation to figure out the versions that have shipped.

We can have a per repo implementation of GetLatestPackageVersion. We can start by most returning "UNKNOWN" and then fill in the functionality. Happy to help from .NET side.


#545 | open contributor in new tab

title: Issue # | 545
What type of PR is this? (select all that apply)

[x] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 🚧 Breaking Change
[ ] 🧑💻 Code Refactor
[ ] 📝 Documentation Update

Description
Clicking contributor profile or contributor commits now opens in new tab.
Related Tickets & Documents

Related Issue #545
Closes #545

QA Instructions, Screenshots, Recordings
UI accessibility concerns?
Added/updated tests?

[ ] 👍 yes
[x] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

[optional] Are there any post deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

Do you think it's a good ux ? From the user's perspective, he is still on the same website, so it would be confusing if contributors page opened up in a new tab

This pull request will need another commit to trigger workflow tests.

Do you think it's a good ux ? From the user's perspective, he is still on the same website, so it would be confusing if contributors page opened up in a new tab

Oh i'm not talking about the contributors tab, i was talking about when u click contributors profile u are sent to their github page, but i wanted to make it so that their github page opens in a new tab.

Do you think it's a good ux ? From the user's perspective, he is still on the same website, so it would be confusing if contributors page opened up in a new tab

Oh i'm not talking about the contributors tab, i was talking about when u click contributors profile u are sent to their github page, but i wanted to make it so that their github page opens in a new tab.

oh, that makes sense


Marker doesn't show up

Hello,
Turn marker doesn't show up on my game

(Tidy settings activated + next up).
Also, how this feature compares to https://github.com/kckaiwei/TurnMarker-alt and why the duplicate? (real question here, no judgment)

The turn marker was an addon feature requested here #18, and it was an interesting project (I also believe that this is more performant and less likely to cause race condition issues or "abandoned" markers)
About the lack of marker, make sure the ratio is high enough to see behind circular tokens. Also check the console for error logs

Yep ratio wasn't high enough.
Thx!


Updated document id size to 50 as per IM

Closing this PR as its incorrectly stated in the IM, we will need to update it in 3.3.


Add tests for ClassGroup, ClassGroupId and UniqueClassGroupList

Fix #51

Codecov Report

Merging #58 (0c55c90) into master (5865d06) will increase coverage by 4.76%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master      #58      +/-   ##
============================================
+ Coverage     58.02%   62.79%   +4.76%
- Complexity      399      439      +40
============================================
  Files            92       92
  Lines          1532     1532
  Branches        173      173
============================================
+ Hits            889      962      +73
+ Misses          611      532      -79
- Partials         32       38       +6

Impacted Files
Coverage Δ

...ava/seedu/address/model/classgroup/ClassGroup.java
66.66% <0.00%> (+66.66%)
:arrow_up:

.../java/seedu/address/model/tamodule/ModuleCode.java
70.00% <0.00%> (+70.00%)
:arrow_up:

...a/seedu/address/model/classgroup/ClassGroupId.java
70.00% <0.00%> (+70.00%)
:arrow_up:

...address/model/classgroup/UniqueClassGroupList.java
85.36% <0.00%> (+85.36%)
:arrow_up:

...seedu/address/model/classgroup/ClassGroupType.java
100.00% <0.00%> (+100.00%)
:arrow_up:

...sgroup/exceptions/ClassGroupNotFoundException.java
100.00% <0.00%> (+100.00%)
:arrow_up:

...group/exceptions/DuplicateClassGroupException.java
100.00% <0.00%> (+100.00%)
:arrow_up:

Continue to review full report at Codecov.

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


Add empty type to use without circle at the top

Add an empty tipe to hide circle at the top. By this, users will be able to use the alert like default UIAlertController

but it is showing error ( Type 'CDAlertViewType?' has no member 'empty' ) message when I wrote this peace of code as I already imported CDAlertView: ->
let alert = CDAlertView(title: "Title", message: "message", type: .empty)

Sorry for the late response. I just saw this. Don't know why didn't see the notification. The type of empty is not there yet. This is the issue for this feature request. If you would like to use your own image, you can use custom type in CDAlertViewType

Released with version 0.8.0


Dropdown positioning preventing user interaction

The dropdown menu module is not working like it does on the Semantic website.
Including your examples - when a dropdown is near the bottom of a page, it cuts off the option list and becomes out of view. The expected behavior would be to position the dropdown values at the top of the input instead of bottom (when it does not fit on the page).
This is causing issues for many of our popup windows and even flat pages where dropdown options are at the bottom of our pages.

Please search prior to posting, duplicate #720


ipcache: Remove unsafe ipc.metadata.get

As previously discussed, the ipc.metadata.get method is not safe, as it returns a reference to a map which could be mutated, without holding on to the lock protecting that map. This commit therefore removes that method and ensures that the caller holds on to the lock while the data is being read. The ToLabels method creates a copy of the data, so it is safe to be read after the metadata lock has been released.

/test


Ensure App Gateway uses the TLS 1.2 policy

App Gateway V2 still uses TLS 1.0 as its default policy. Our own documentation points that out that we should use TLS 1.2.
https://docs.microsoft.com/en-us/azure/application-gateway/application-gateway-configure-ssl-policy-powershell

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

#label:"aq-pr-triaged"

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

Hi @dlepow, you beat me to the pathing change. =)
I verified the PS works end-to-end prior to submitting the PR. I also verified TLS 1.2 showed as the policy in the Azure Portal afterwards.

#sign-off

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


(reorder fixes:) https://github.com/driftyco/ionic/issues/7522

Short description of what this resolves:
https://github.com/driftyco/ionic/issues/7522
-items getting 'stuck together'
-items not getting the right .toIndex when dropped back in their original positions
Changes proposed in this pull request:

sets right indexes
needs temporary bottom item to get right index when dragged to bottom of list (probably there is a way around this, but I couldn't figure out a better way

Ionic Version: 2.x

Hello! Thanks for opening this PR! Would you be able to fix the conflicts here?

Thanks for the PR! I'm going to close this since it seems Manu has been working on this on a different PR: https://github.com/ionic-team/ionic/pull/9250


chore: Fix Security vulnerabilities

This PR fixes a security vulnerability.
For more context: https://github.com/appsmithorg/appsmith-ee/issues/2011

Bug fix (non-breaking change which fixes an issue)
Chore (housekeeping or task changes that don't impact user perception)

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[x] Manual
[x] Jest
[x] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/ok-to-test

/ok-to-test

/build-deploy-preview skip-tests=true

/ok-to-test

Adding test plan approved label as per this comment here


C++\error\bubblesort.cpp

the given array is sorted using bubble sort algorithm but there is an error...find the error in the code to make it work
NOTE:- Screenshot of the code and output is required

Can you please assign this issue?

Please assign this issue to me !!

can you assign me this issue?


az postgres flexible-server geo-restore

Describe the bug
Geo-restore does not work for existing subnet in a different resource group (and possibly also not in the same)
Related command
az postgres flexible-server geo-restore -g {} -n {} --subnet {} --source-server {} --location {}
Where:

source-server: is the server id
subnet: subnet id that already exists
n: name of the new server
g: resource-group of the new server
location: westeurope

Errors
The command failed with an unexpected error. Here is the traceback:
'NoneType' object has no attribute 'public_network_access'
Traceback (most recent call last):
  File "/opt/homebrew/Cellar/azure-cli/2.52.0_1/libexec/lib/python3.10/site-packages/knack/cli.py", line 233, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/homebrew/Cellar/azure-cli/2.52.0_1/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 663, in execute
    raise ex
  File "/opt/homebrew/Cellar/azure-cli/2.52.0_1/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/homebrew/Cellar/azure-cli/2.52.0_1/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 697, in _run_job
    result = cmd_copy(params)
  File "/opt/homebrew/Cellar/azure-cli/2.52.0_1/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 333, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/homebrew/Cellar/azure-cli/2.52.0_1/libexec/lib/python3.10/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/opt/homebrew/Cellar/azure-cli/2.52.0_1/libexec/lib/python3.10/site-packages/azure/cli/command_modules/rdbms/flexible_server_custom_postgres.py", line 552, in flexible_server_georestore
    validate_georestore_network(source_server_object, None, vnet, subnet, 'postgres')
  File "/opt/homebrew/Cellar/azure-cli/2.52.0_1/libexec/lib/python3.10/site-packages/azure/cli/command_modules/rdbms/validators.py", line 614, in validate_georestore_network
    if source_server_object.network.public_network_access == 'Disabled' and not any((public_access, vnet, subnet)):
AttributeError: 'NoneType' object has no attribute 'public_network_access'
To check existing issues, please visit: https://github.com/Azure/azure-cli/issues

Issue script & Debug output
Traceback starts right after a successfull call to:
/providers/Microsoft.DBforPostgreSQL/locations/westeurope/checkNameAvailability?

with response:
.....,"nameAvailable":true,"message":""}

Expected behavior
Postgres database gets georestored to a different resource group and subnet
Environment Summary
macOS-13.6.1-arm64-arm-64bit, Darwin 22.6.0
Python 3.10.13
Installer: HOMEBREW

azure-cli 2.52.0 *

Extensions:
azure-devops 0.26.0
front-door 1.0.17
ssh 2.0.0

Dependencies:
msal 1.24.0b1
azure-mgmt-resource 23.1.0b2

Additional context
No response

Thank you for opening this issue, we will look into it.

Same exception with 2.53:
The command failed with an unexpected error. Here is the traceback:
'NoneType' object has no attribute 'public_network_access'
Traceback (most recent call last):
  File "/opt/homebrew/Cellar/azure-cli/2.53.1/libexec/lib/python3.10/site-packages/knack/cli.py", line 233, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/homebrew/Cellar/azure-cli/2.53.1/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 663, in execute
    raise ex
  File "/opt/homebrew/Cellar/azure-cli/2.53.1/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/homebrew/Cellar/azure-cli/2.53.1/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 697, in _run_job
    result = cmd_copy(params)
  File "/opt/homebrew/Cellar/azure-cli/2.53.1/libexec/lib/python3.10/site-packages/azure/cli/core/commands/__init__.py", line 333, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/homebrew/Cellar/azure-cli/2.53.1/libexec/lib/python3.10/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/opt/homebrew/Cellar/azure-cli/2.53.1/libexec/lib/python3.10/site-packages/azure/cli/command_modules/rdbms/flexible_server_custom_postgres.py", line 559, in flexible_server_georestore
    validate_georestore_network(source_server_object, None, vnet, subnet, 'postgres')
  File "/opt/homebrew/Cellar/azure-cli/2.53.1/libexec/lib/python3.10/site-packages/azure/cli/command_modules/rdbms/validators.py", line 619, in validate_georestore_network
    if source_server_object.network.public_network_access == 'Disabled' and not any((public_access, vnet, subnet)):
AttributeError: 'NoneType' object has no attribute 'public_network_access'

The subnet was not delegated to Microsoft.DBforPostgreSQL/flexibleServers
after doing so, the cli moves forward but now fails to find the source database.
The GET request for the source database succeeds.
But the PUT request that is performed afterwards to create the new one fails with
{"error":{"code":"ResourceNotFound","message":"The requested resource of type 'Microsoft.DBforPostgreSQL/flexibleServers' with name 'mysourceServer' was not found."}}

the request had as body:
cli.azure.cli.core.sdk.policies: {"location": "westeurope", "properties": {"backup": {"backupRetentionDays": 7, "geoRedundantBackup": "Disabled"}, "network": {"delegatedSubnetResourceId": "mysubnetid", "privateDnsZoneArmResourceId": "myprivatednszoneid"}, "sourceServerResourceId": "mysourceserverid", "pointInTimeUTC": "2023-11-27T13:43:04.000Z", "createMode": "GeoRestore"}}

Just had a call with Microsoft support after opening a support request.
They were able to replicate the issue. Therefore is confirmed is a bug on the azcli and powershell that the product team must address. The problem does not occur if the same migrations is requested from the portal.
It happens only when the source and destination databases belong to different resource groups.

@alsastre  - Could you please share the support ticket number so that I can look into it.

Support ticket:<PHONE_NUMBER>001293

Thank you i will work with Azure CLI team to add support for this in the coming release. Until then as requested in the support case please follow the portal and Rest APT

Closing this issue for now.  Please feel free to reach out if you have any questions

Shouldn't it remain open till the issue is fixed?

@alsastre please, reach out to me. I'm trying to reproduce the issue so that can work on a fix, but I'm unable to get to the error condition you're seeing.

Hello @nachoalonsoportillo
I am not sure of the differences but I just tested and obtained the same result 🥲
dstResourceGroup=DST_RESOURCE_GROUP
dstDatabase=DST_DB
subnet="/subscriptions/SUBSCRIPTION/resourceGroups/DST_RESOURCE_GROUP/providers/Microsoft.Network/virtualNetworks/DST_VNET/subnets/DST_SNET"
src_scope="/subscriptions/SUBSCRIPTION/resourceGroups/SOURCE_RESOURCE_GROUP/providers/Microsoft.DBforPostgreSQL/flexibleServers/SOURCE_DB"
MAIN_RESOURCES_LOCATION="westeurope"
privateDNS="/subscriptions/SUBSCRIPTION/resourceGroups/OTHER_RESOURCE_GROUP/providers/Microsoft.Network/privateDnsZones/MYDNSZONE.internal.postgres.database.azure.com"
az postgres flexible-server geo-restore -g "$dstResourceGroup" -n "$dstDatabase" --subnet "$subnet" --source-server "$src_scope" --location $MAIN_RESOURCES_LOCATION --private-dns-zone "$privateDNS"

Where the SOURCE DB is in SNET:
subnet="/subscriptions/SUBSCRIPTION/resourceGroups/SRC_RESOURCE_GROUP/providers/Microsoft.Network/virtualNetworks/SRC_VNET/subnets/SRC_SNET"
I also tested by defining the dst_vnet with vnet and subnet parameters with the same result.
az postgres flexible-server geo-restore -g "$dstResourceGroup" -n "$dstDatabase" --vnet $DST_VNET --subnet "$DST_SNET" --source-server "$src_scope" --location $MAIN_RESOURCES_LOCATION --private-dns-zone "$privateDNS" --debug

I just tested it also without the private-dns-zone parameter and the same issue appears.

@alsastre BTW, for GeoRestore to succed, NSGs/FWs controlling network traffic on the restore server VNET must allowlist Azure Storage tag.
A note on that can be in the important note in the Virtual network concepts section of the Networking overview for Azure Database for PostgreSQL - Flexible Server with private access (VNET Integration) article.
Make sure you read in that note what other traffic must be allowed so that other service features work properly.

#please-close

#please-close


Angular 5 support?

Hello,
Does it support angular 5?

i am getting the following error.

Angular 5 support was dropped in favor of Angular 6. There are still betas tagged 5.0.0-beta.x that support angular 5, but those are no longer being supported.
This error you shared in particular is from Ionic. You don't have @angular/router installed, which causes this issue.


Elevation & shadows

elevation & shadows describes precisely z-index and shadows.
I will do a pull request, but I need some guidance on how to do that.
Using placeholder ? mixins ?

hi @obsidienne
the elevation spec seems to be for android apps. directly implementing all the z-index values may not work on web pages though...
shadows may be much easier and more straightforward to implement. probably as variables.

can u make it swipe menu

@sesemaya I think the elevation and shadow are quite strongly related. Without elevation there won't be stuffs like this.
Static shadow is ok, but I guess it won't be as "material-ish".

V4 has defined all elevation and shadow as variables in _variable-elevation-shadow.scss. They are then mapped to individual components in _variable-material.scss.
However, the values of elevation are 10 X the values in Material Design Guidelines because z-index ranging only from 1 to 24 does not provide enough flexibility for web design/development.


Craft 3.4.23

Downloader the plugin via the composer CLI as instructed in the documentation.
When I do try to open the plugins here is the error :

I do have 4 sections created and 2 field.
Edit: System info

Is this still an issue with the new 1.1.0 version?

Is this still an issue with the new 1.1.0 version?

I will update all my versions and give it a try once again tomorow. I've started working on some underthing and kinda forgot about this issue. Will let you know ASAP


Package rhumsaa/uuid is abandoned, you should avoid using it. Use ramsey/uuid instead.

Until hautelook/sentry-client updates their package to use ramsey/uuid, you may do the following in your project:
composer require ramsey/uuid=^2.8

Since ramsey/uuid indicates that it is a replacement for rhumsaa/uuid, when Composer installs dependencies for hautelook/sentry-client, it will use ramsey/uuid instead, and all will be well, and you won't see the warning message again. :-)

https://github.com/hautelook/sentry-client/pull/8

Tagged new 1.0.0 release, so we should be good here. Closing issue.


[WIP] added support for np.asarray to be used with numba.typed.List

fixes #5226

Added support for np.asarray to work with numba.typed.Listobjects.
Added a test case

@rohitsanj thanks for fixing this up. One small remark, on the Numba project, we usually tend to avoid using rebase as we have had bad experiences with it not preserving comments from code-reviews on GitHub. Therefore the general rule of thumb is to stop using rebase after the first code-review comment has been submitted. If you need to include changes from master in your feature branch, please then just merge in master. Thanks!

Sure! Noted. @esc

As discussed via gitter with @rohitsanj I have taken to finish this PR in time for 0.49.0. I have updated the code and tests and have now decided to reject more complex (not in the mathematical sense) types. @stuartarchibald and @sklam please advise as to which types we may want to add a branch for. Maybe NoneType  and BooleanType?

As discussed with @sklam I'll add BooleanType.

waiting on CI


Fix coverage penalty (wu)

When I use
beam_size: 2
coverage_penalty: 'wu'
beta: 1

In my decoding config, I have the following error
Traceback (most recent call last):
  File "/usr/local/bin/eole", line 33, in <module>
    sys.exit(load_entry_point('EOLE', 'console_scripts', 'eole')())
  File "/workdir/eole/eole/bin/main.py", line 39, in main
    bin_cls.run(args)
  File "/workdir/eole/eole/bin/run/predict.py", line 42, in run
    predict(config)
  File "/workdir/eole/eole/bin/run/predict.py", line 18, in predict
    _, _, _ = engine.infer_file()
  File "/workdir/eole/eole/inference_engine.py", line 38, in infer_file
    scores, estims, preds = self._predict(infer_iter)
  File "/workdir/eole/eole/inference_engine.py", line 170, in _predict
    scores, estims, preds = self.predictor._predict(
  File "/workdir/eole/eole/predict/inference.py", line 475, in _predict
    batch_data = self.predict_batch(batch, attn_debug)
  File "/workdir/eole/eole/predict/generator.py", line 71, in predict_batch
    return self._predict_batch_with_strategy(batch, decode_strategy)
  File "/workdir/eole/eole/predict/generator.py", line 149, in _predict_batch_with_strategy
    decode_strategy.advance(log_probs, attn)
  File "/workdir/eole/eole/predict/beam_search.py", line 437, in advance
    super(BeamSearchLM, self).advance(log_probs, attn)
  File "/workdir/eole/eole/predict/beam_search.py", line 383, in advance
    self.topk_scores -= cov_penalty.view(_B, self.beam_size).float()
RuntimeError: shape '[1, 2]' is invalid for input of size 518

I have tried to fix it but it revamps a bit the penalty calculation.
I calculate it "from scratch" at each decoding step, using the attentions.

It seems that the return_attn path is also broken. So I've made sure I don't go through this path when applying penalty coverage.

Thanks for your review !
Related to the _prev_penalty I understand that I should keep, at each decoding step, the array of the sums of attentions on each source token, in order to reuse it in the next step.
It should be closer to the main branch this way.

I hadn't understood the use of accumulated coverage, which consists of keeping a step-by-step table of the sum of attentions of each hypothesis in the source tokens, and which changes the penalty formula from that of the article... in the correction I use the table of attentions of each hypothesis token on the source, and I recalculate the coverage “from zero” at each step (using the formula of the article), which is not optimal. So I open another PR.


[Seach Files]

Extension
raycast://extensions/raycast/file-search/search-files
Description
Can you put the switching option to search by name or content in the operation window? You must go to Settings to switch，An immature suggestion
Who will benefit from this feature?
a very good person
Anything else?

We could not find the extension related to this issue. Please update the issue with the link to the extension.

I couldn't find a connection for this extension, and figured it might be a built-in feature


How to choose different grid

Thanks for your work. I know the differences between different grid. But I don't know how to choose different grid. It both seems Ok  when I use it. Could you give me some advice about when should I use near identity grid  or equatorial grid?

local vs non local filters... we don't know when it is more appropriate to use one or the other, sorry.

ok thanks for your reply


[Feature request] Mouse Pointer visibility when recording

Hello, congratulations for your great free app; it is one of the references in its kind.
I'd like to see it a reference in the future too, that's why I invite you when recording, to make visible the mouse pointer too.
That's would a great improvement for those making a video tutorial and maybe will make Scrcpy one of the few if not the only one to feature that possibility.
Thank you.

The video is hardware-encoded on the device side. For recording, the video stream is just muxed, on the client side. This avoids to reencode, which is better for both quality and performance.
Incrusting a mouse pointer would require to reencode the video.
Also, the device produces a new frame only when the screen content changes. A mouse pointer could change between frames (especially when the device screen content does not change), so this would imply to insert new frames.

On the contrary, connecting a usb mouse to an android device (such as via OTG) makes a pointer show up that you can use just like any other - so you could have scrcpy send mouse movements to do the same.

@Erudition Yes, using HID over AOA, the mouse is shown on the device (so it would probably be recorded). (But it would only work on Linux.)
Ref:

https://github.com/Genymobile/scrcpy/issues/87#issuecomment-378775565
https://github.com/Genymobile/scrcpy/issues/279#issuecomment-453819354

Oh well that's perfect then!

Any chance, that i can have the cursor position visible on my android device all the time? Like, i can look at the device and control it with seeing my cursor? Thx

@maddinmcfly No: https://github.com/Genymobile/scrcpy/issues/2088#issuecomment-773370723


Feature: Support behavior like workspace_auto_back_and_forth from i3wm

In i3wm, you can enable behavior that allows you to be on workspace 2, press mod + 1 which will shift focus to workspace 1, and then press mod + 1 again, which will return to the previously viewed workspace, in this case workspace 2.
This is very handy when quickly flipping back and forth between workspaces, as you can just give a double tap. It also means you don't have to think about where you came from, if you know you just want to go back to that workspace.
I'm not sure how feasible this is, but it's one of the features I miss most from i3.

This is not exactly what you are asking for, but it is possible to bind a key-combination, e.g: mod + x to focus the previously focused desktop. You can then keep hitting this key to toggle between those 2 desktops. See my comment in https://github.com/koekeishiya/skhd/issues/55

I also wanted this, and ended up with this:
cmd - 1 : i=1; chunkc tiling::desktop --focus $(( $(chunkc tiling::query --desktop id) == $i ? $(chunkc get _last_active_desktop) : $i ))
cmd - 2 : i=2; chunkc tiling::desktop --focus $(( $(chunkc tiling::query --desktop id) == $i ? $(chunkc get _last_active_desktop) : $i ))
etc...

which seems to work nicely.

Thanks for taking the extra step @ad1217
I've added this information to the wiki: https://github.com/koekeishiya/chunkwm/wiki/Useful-tips-and-workarounds

Thanks for the tip! For some reason I just hadn't realize how much scripting you could use in the config file. I think I can make this do everything I want. Many thanks for making a great tool.


Etcher-cli crashes on Mac OS right after selecting the SD card reader, but Etcher GUI works fine on the same machine

Etcher version: Etcher-cli-1.1.2-darwin-x64
Operating system and architecture: Mac OS Sierra 10.12.6
Image flashed: iot_rpi.img
Do you see any meaningful error information in the DevTools?

sudo /opt/etcher-cli/etcher /snapshot/etcher/dist/Etcher-cli-1.1.2-darwin-x64-app/lib/cli/etcher.js iot_rpi3.img
? Select drive /dev/disk3 (8.0 GB) - SD Card Reader
? This will erase the selected drive. Are you sure? Yes
events.js:160
      throw er; // Unhandled 'error' event
      ^

Error: UNEXPECTED-15
    at readFromSnapshot (pkg/prelude/bootstrap.js:493:16)
    at Object.fs.read (pkg/prelude/bootstrap.js:510:5)
    at ReadStream._read (fs.js:1998:6)
    at ReadStream.Readable.read (_stream_readable.js:348:10)
    at fs.js:1963:10
    at zalgoSafe (pkg/prelude/bootstrap.js:207:10)
    at pkg/prelude/bootstrap.js:424:9
    at FSReqWrap.oncomplete (fs.js:123:15)

Note: Flashing works fine with the Etcher GUI.

Btw, I just tried to clone the repo, npm install, then run sudo node etcher.js iot_rpi3.img, and it worked! Not sure why the public build at https://etcher.io/cli/ failed.

Hi @atn832 ,
Thanks for reporting! The issue seems to come from pkg, the tool we use to package the CLI as a standalone executable. Would you mind re-posting this issue report at https://github.com/zeit/pkg/issues/new, and ping me so I can stay in the loop?

Sure @jviotti , I've just filed a ticket there: https://github.com/zeit/pkg/issues/205.

I get the same error on Windows with https://github.com/resin-io/etcher/releases/download/v1.1.2/Etcher-cli-1.1.2-win32-x64.zip:
etcher.exe C:\snapshot\etcher\dist\Etcher-cli-1.1.2-win32-x64-app\lib\cli\etcher.js iot_rpi3.img
? Select drive \\.\PHYSICALDRIVE2 (8.0 GB) - Generic- USB3.0 CRW   -SD USB Device
? This will erase the selected drive. Are you sure? Yes
events.js:160
      throw er; // Unhandled 'error' event
      ^

Error: UNEXPECTED-15
    at readFromSnapshot (pkg/prelude/bootstrap.js:524:16)
    at Object.fs.read (pkg/prelude/bootstrap.js:541:5)
    at ReadStream._read (fs.js:1998:6)
    at ReadStream.Readable.read (_stream_readable.js:348:10)
    at fs.js:1963:10
    at zalgoSafe (pkg/prelude/bootstrap.js:239:10)
    at pkg/prelude/bootstrap.js:455:9
    at FSReqWrap.oncomplete (fs.js:123:15)

@atn832 closing this, as the relevant PRs have been merged – next release should be working again :)

Still happening on v1.4.4 on MacOS


fixed some typos

I had a little of time so I decided to fix the silly typos I saw haha. Also this is the second time doing a pull request so I hope I'm doing everything right.

@treee97 Nice catch!

this issue is also fixed in https://github.com/yoheinakajima/instagraph/pull/18 need to merge either of the 2 PR


Custom model

I found Layoutlmv2 takes both the image and the context into the pretrain. Why would you use BERT and not Layoutlmv2. I want custom to use layoutlmv2 for encoder, is it possible? Thank you so much

Hi @MinhQuang2021
You may customize this part
https://github.com/clovaai/spade/blob/a85574ceaa00f1878a23754f283aa66bc2daf082/spade/model/model.py#L743-L775
The backbone is based on hugginface transformers thus it shouldn't be difficult to adapt layoutlm.
During the major development , layoutlm wasn't available. Also layoutlm is not pre-trained for Japanese and Indonesian. Thus we initialized the spade encoder with the subset of BERT weights (note that spade encoder ≠ BERT).
Best

Thank you, I really want to apply it on my custom dataset for my language. But creating a dataset with a label like CORD is quite time consuming, so I want to ask you, How many datasets will give good results? Is the standard dataset for this model important? Is this model applicable to unseen data well?
Can you give me some knowledge in EL task I still find it quite confusing in paper to just generate graph and decode it.
I am learning it for my graduation project. Thank you very much

I noticed when set self.hparam.token_lv_boxing == True. parse is parsed differently. Why in Cord_test.yml and Cord_train.yml you both set it to True. Can you explain how it works for me?

why do you set token_lv_boxing= True in CORD , and in FUNSD you set token_lv_boxing=False .
How does token_lv_boxing work? I noticed it changes the parse result when training.

hello, can you help me find the code to apply inferring_method?
- tca_rel_s
- tca_rel_g

Hi @MinhQuang2021
I would say, roughly 1,000 documents are required for the reasonable performance.
Under zero-shot setting, the domain should be similar for reasonable performance. Otherwise, you'll observe a large performance drop.
About the EL task, FUNSD consists of four types of fields: "Header" "Question", "Answer", "ETC".
The EL task aims to connect "Header" → "Question", "Question" → "Answer".
So it is kind of "grouping two fields where each field consist of multiple serialized words".
When token_lv_boxing=True, spade draws directed arrows between tokens. For the noisy OCR data like receipt, this makes a better result. For FUNSD, the task assumes a perfect OCR so I turned it off.
The data augmentation option is automatically turned off during prediction. Sorry for the confusion.
Sorry for being late in reply. Good luck with your project!


Build ModuleGroups from specs in options file

Wangscape tile generation is of course intended to be customisable. A draft of the flow of data between noise modules is given in #15. Three of the modules are "anything", which means they should be completely specified by the user. All user customisation of Wangscape is mediated by the JSON options file. So, a function must be made that converts a JSON value to a ModuleGroup (examples) containing modules from libnoise (and possibly also custom modules defined in Wangscape).

[ ] Write a JSON schema which accepts a JSON value iff it describes a valid ModuleGroup. Features:

[ ] Each component module given a name which is unique within the ModuleGroup.
[ ] Module types and parameters all customisable
[ ] Module parameters restricted to those available in the module type?
[ ] Module sources either referred to by name, or determined hierarchically, or both (implementor's choice)

[ ] Write a routine which takes a const rapidjson::Value& as input and returns a ModulePtr containing a ModuleGroup built according to the input data.
[ ] Write example JSON documents and tests to check the ModuleGroup is built correctly for valid inputs (validating using the schema may be deferred until #21 is resolved).

RESTnoise already has a standard for JSON -> libnoise module conversion. An example JSON file is here. It would probably be best to reuse their format. I can't find a JSON schema for RESTnoise; it might not exist yet.
The customisable masks are all intended to be used as squares ([0,1]x[0,1], [0,1]x[-1,0], or [-1,0]x[0,1]), so some of the geometrical features used in RESTnoise will not be needed. This may change if non-square tile geometries are supported in the future.

Some of the options in #37 are less suited as part of the AlphaCalculator step and more suited to being part of the Perlin noise module stack. In particular, applying a function such as raising all weights to the same power should be implemented with libnoise modules, and should be customisable in a similar way.
However, customising these postprocessing steps will have to be implemented differently from customising the source modules, because the postprocessing steps will have an input module.

Switching to spotify-json has probably made this more difficult to implement. The two main difficulties are:

Constructing a different Module subclass depending on a "type" key in the json object. I think this would have to be done with a cast_t or an any_codec_t? We should only need to do this in one direction (at least until work on #17 starts).
Using module names to initialise the source module pointers. I don't think it's possible to do this with the present interface (Reseedables store no additional info about source modules, ModuleGroups store Reseedables by name but don't use the names to assign source modules). I think there are two possible ways to solve this:

Make a wrapper class containing a Reseedable and some module names/vectors of names. Deserialise into the wrapper cpass


fix #1055 Unexpected ROUNDDOWN function Result

this closes #1055

use decimal to replace double on precision issue
add a unit test file to track the issue

seems like it breaks 1 test cases, change left side to decimal or remove this check?

left side is double precision, while right side is formula decimal precision
Assert.AreEqual(NEW_PART_COST * NEW_QUANT, ccell.NumericCellValue);

other build fail no clue yet

This might just be a case of Linux font creating a different width. It's terrible, I know. You can check for platform in test case and adjust when needed. Of course it's a bit worrying that different platforms will create different representation - if they do.
https://github.com/nissl-lab/npoi/blob/32435d9485181a08f6993d3e94c10a70e5bb810a/testcases/main/HSSF/UserModel/TestHSSFSheet.cs#L858-L860

thanks Lahma to confirm, I just noticed this ubuntu column check fail not introduced by me...
good ending, I can go sleep now

Please have a good night's sleep🙂 It's still problematic to merge though, as it would break master build.

Since Windows test running is pass, I'd like to merge it first.

LGTM


Update dataprotocol-client

For requests like https://github.com/microsoft/azuredatastudio/issues/24617 - and to fix the potential for extensions to completely break ADS.
Brings in https://github.com/microsoft/sqlops-dataprotocolclient/pull/98
Changing stable API - but since we weren't returning anything before that should be fine.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.007%) to 41.582%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/book/bookTreeView.ts
1
36.91%

extensions/sql-database-projects/src/common/httpClient.ts
1
66.18%

extensions/sql-database-projects/src/dialogs/publishDatabaseDialog.ts
2
52.03%

Totals

Change from base Build<PHONE_NUMBER>:
-0.007%

Covered Lines:
30778

Relevant Lines:
69325

💛  - Coveralls