WARNING: linker: libdvm.so has text relocations. This is wasting memory and is a security risk. Please fix.

I am using Visual Studio Code and Visual Studio Emulator for Android.
I created the first example app and tried to run it.
I get the following message and the application closes on the emulator without even becoming visible:
WARNING: linker: libdvm.so has text relocations. This is wasting memory and is a security risk. Please fix.
How do I fix this?

Please add the output of flutter doctor -v.

Hi Zoechi,
The sequence of events is as follows:
On the CLI
Microsoft Windows [Version 10.0.17134.228]
(c) 2018 Microsoft Corporation. All rights reserved.
C:\Users\Henry>flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel beta, v0.5.1, on Microsoft Windows [Version 10.0.17134.228], locale en-US)
[√] Android toolchain - develop for Android devices (Android SDK 27.0.3)
[√] Android Studio (version 3.1)
[!] VS Code, 64-bit edition
[√] Connected devices (1 available)
! Doctor found issues in 1 category.
C:\Users\Henry>
On Visual Studio Code:
Debug>Start Debugging
Information Window:
Initiating Gradlew
Resolving Dependancies
Source: Dart Extension
Running Gradlew  'Assemble Debug'
Debug Console:
Launching lib\main.dart on 7 inch KitKat 4 4 XHDPI Tablet in debug mode...
Built build\app\outputs\apk\debug\app-debug.apk.
WARNING: linker: libdvm.so has text relocations. This is wasting memory and is a security risk. Please fix.
On the Emulator:
Unfortunately Shzenproj has stopped.
Any suggestions?

The error doesn't seem to be specific to Flutter https://stackoverflow.com/questions/21837532/security-warning-on-libdvm-so-noticed-while-deploying-android-apps

Hi, I changed the selected Phone in the emulator and it is now working.
Thank you for pointing me in the right direction.
I think I also have a memory capacity problem and that I will have to upgrade my PCs RAM
Thanks again, much appreciated

Glad to hear. Can this be closed.
It doesn't look like the Flutter team can do anything about that.


Fix bubble capture issues

Bubbles currently have issues maintaining information about a social vehicle. This can cause crashing of the simulation when vehicles pass in and out of bubbles.

[ ] see https://github.com/huawei-noah/SMARTS/actions/runs/3527401521/jobs/5916420082

(pid=1330) ERROR:root:TraCI has disconnected with: Vehicle 'AGENT-0-bb8eb7f6' is not known
(pid=1330) WARNING:SumoTrafficSimulation:attempting to transfer SUMO vehicles to other providers...
(pid=1330) ERROR:root:TraCI has disconnected with: Vehicle 'AGENT-0-fc92eca4' is not known
(pid=1330) WARNING:SumoTrafficSimulation:attempting to transfer SUMO vehicles to other providers...
2022-11-22 22:08:06,293	ERROR trial_runner.py:793 -- Trial PG_RLlibHiWayEnv_cbfb3_00000: Error processing event.
Traceback (most recent call last):
  File "/__w/SMARTS/SMARTS/.venv/lib/python3.7/site-packages/ray/tune/trial_runner.py", line 726, in _process_trial
    result = self.trial_executor.fetch_result(trial)
  File "/__w/SMARTS/SMARTS/.venv/lib/python3.7/site-packages/ray/tune/ray_trial_executor.py", line 489, in fetch_result
    result = ray.get(trial_future[0], timeout=DEFAULT_GET_TIMEOUT

A fix was done with #1811.


Add retry to clean-up packages GitHub action

Codecov Report

Merging #404 (2a6b064) into main (0959ef9) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 2a6b064 differs from pull request most recent head 620ddb3. Consider uploading reports for the commit 620ddb3 to get more accurate results

@@           Coverage Diff           @@
##             main     #404   +/-   ##
=======================================
  Coverage   67.07%   67.07%
=======================================
  Files          80       80
  Lines        5492     5492
=======================================
  Hits         3684     3684
  Misses       1392     1392
  Partials      416      416


[Dewhale] implements #270

[Dewhale] This PR implements #270, created by Dewhale.

[Dewhale]: This code snippet creates a small form with two input fields: one for an API key and another for a proxy, which is optional. The API key field includes a toggle button to switch between visible and hidden states, using the Eye and EyeOff icons for visual cues. The form also includes a Switch component to toggle the visibility of the API key. A submit button is included at the bottom of the form.


minor: update beats framework version to v7.1.12-bk

Codecov Report

Merging #19 (459fc0d) into master (f370dc4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #19   +/-   ##
=======================================
  Coverage   45.72%   45.72%
=======================================
  Files          17       17
  Lines         866      866
=======================================
  Hits          396      396
  Misses        419      419
  Partials       51       51

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 f370dc4...459fc0d. Read the comment docs.


Esdc ea strategy and roadmap

No sure why, but removing ampersands seems to have fixed my content in the Report...


"Apahida", "Bodrog", "Câmpenești", "Corpadea", "Dezmir", "Pata", "Sânnicoară", "Sub Coastă", are listed in the drop-down menu

Description:
When user writes "Apahida", "Bodrog", "Câmpenești", "Corpadea", "Dezmir", "Pata", "Sânnicoară" and "Sub Coastă" the locations appear in the drop-down menu next to the comune and the county witch is a part according to the law
Precondition:
The website is up an running.

Step 1
Write in the search bar "Apahida"
Expected results
The location appears in the drop-down menu as "Sat Apahida, CLUJ (Apahida)"

Step 2
Press "x" button.
Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Bodrog"
Expected results
The location appears in the drop-down menu as "Sat Bodrog, CLUJ (Apahida)"

Step 6
Press "x" button.
Expected results
The location was deleted from the search bar.

Step 9
Write in the search bar "Dezmir"
Expected results
The location appears in the drop-down menu as "Sat Dezmir, CLUJ (Apahida)"

Step 10
Press "x" button.
Expected results
The location was deleted from the search bar.

Step 11
Write in the search bar "Pata"
Expected results
The location appears in the drop-down menu as "Sat Pata, CLUJ (Apahida)"

Step 12
Press "x" button.
Expected results
The location was deleted from the search bar.

Step 13
Write in the search bar "Sânnicoară"
Expected results
The location appears in the drop-down menu as "Sat Sânnicoară, CLUJ (Apahida)"

Step 14
Press "x" button.
Expected results
The location was deleted from the search bar.

Step 15
Write in the search bar "Sub Coastă"
Expected results
The location appears in the drop-down menu as "Sat Sub Coastă, CLUJ (Apahida)"


Publish update for Redgate SQL Search extension

Note: We seem to be experiencing inconsistent behaviour when viewing the details page of SQL Search Extension in the extension marketplace. Some users see the new up-to-date details page with details about 0.3.0 version. Other users still see 0.2.1 details page even when they have 0.3.0 installed.

@stanislavHamara I published your latest update and here is what I see in the extension marketplace with the latest stable build.  It looks correct as far as I can tell.  Please let me know if you have additional repro steps for the inconsistencies you've had reported to you.


cursor.fetchallarrow() raise an error

Hello When I excuted :
table = cursor.fetchallarrow()
it raised :
TypeError: make_arrow_result_set(): incompatible function arguments. The following argument types are supported:
    1. (arg0: turbodbc::result_sets::result_set, arg1: bool, arg2: bool) -> turbodbc_arrow_support.ArrowResultSet

Invoked with: <turbodbc_intern.BaseResultSet object at 0x0000020DCB9166C0>, False, False

Windows 10, teradata SQL, python 3.5
thank you.

The current release of turbodbc does not support Arrow on Windows systems. The next release will change that. For a workaround, you could try to make do with the Numpy features; those are supported on Windows also by the current release.

I have a question about performance and time execution using "arrow", is it the same as using Numpy?
thank you.

Timings are comparable and for some applications within the margin of error. Arrow has advantages when handling strings due to built-in dictionary encoding and also allows for fast transformation into pandas data frames. Still, Numpy support is quite up to the game.

thanks for the clarification.
I wish you the best.


Screen Name and Broadcast Titles

I have noticed that the the downloaded files only contain the username, timestamp, and broadcast id code. I would like it to include username, screen name, timestamp, and broadcast title. This will greatly help with sorting broadcasts for viewing. Is there a way to configure this?

Yes, having the Broadcast title incorporated in the save file name, would be fantastic and a great help! ♥


Jquery signature

signature box moves on the screen making it impossible to get a signature.

Found solution for this. copied code into project and this fixes the error.
Noticed other issues:
signature PDF is not getting emailed out to student. - fixed
past signature does not show up on new registrations. - fixed


async is now a reserved since latest version of Rust

The word "async" is now reserved as of the latest versions of Rust. It is used in the namespace request::async.

Related issues:
#408
#357
For now you should use r#async:
use reqwest::r#async;

Closing as duplicate of #357.


Fix collector and bridge dependencies

Move abacus-request from bridge and abacus-cluster from collector to dependecies instead of devDependencies since they cannot get built without them.

Coverage remained the same at 86.842% when pulling aee83015887119b37631c064867e3a38627f1c30 on SAP:dep into 38931ec076e62cdc713496d7083039a28b34e3d3 on cloudfoundry-incubator:master.


Bitwig doesn't recognize the built clap file

I'm not sure if I'm doing something wrong here.
When I build this .clap file and try point bitwig to the ignore/build folder, it doesn't add this saw demo to my list of plugins. As far as I can tell, there doesn't seem to be any build errors. I'm using Bitwig 4.3. I've built this package on an arch-linux based system. When I check the plugin loading log in bitwig, it prints out the following message:
com.bitwig.flt.library.metadata.reader.exception.CouldNotReadMetadataException: could not read metadata: Failed to load CLAP plug-in /home/eoin/Source/clap-saw-demo/ignore/build/clap-saw-demo.clap: /home/eoin/Source/clap-saw-demo/ignore/build/clap-saw-demo.clap: undefined symbol: _ZNK4clap7helpers9HostProxyILNS0_19MisbehaviourHandlerE1ELNS0_13CheckingLevelE2EE20canUsePosixFdSupportEv

Any ideas?

Huh wow that is odd. I’ve mostly tested in mac recently but I don’t know why that symbol would be missing.
It is almost definitely e40d382465abda1ebc131e087e5b55102ec3c4f0 that commit
I don’t have a Linux box available today but will tomorrow. But if you would like to

can you try building the commit prior to that?
if that works then it is a template instantiating problem. I bet that editor file needs an inckude host-proxy.hxx if Linux at the top
If 2 is the fix and toh try it and it works and you want to send a pr that would be appreciated

and if not I can look later this week!
thanks

That correction seemed to work. @baconpaul Thanks!

Great. I will merge it this morning.
I don't know why it isn't a problem here on ubuntu 20 but let me get it merged.
Do you mind sharing what toolchain you are using to build?

OK thanks for flagging this. I just merged a change which works on ubuntu 20 still and contains what I think is the fix for your toolchain also. Also upgrades to clap 1.1.1 and adds the build time and date on the UI.


New module for feedback on the different entities of the Student's Council

One could write anonymous notes to the the Council and beyond.

I'd add this to the form we send out at the end of the semesters


Center guide

I appreciate the PR but I'm not a fan of increase the API surface area for a feature only a few people might want, especially if you can solve it other ways.
I would suggest you simply do it with some CSS for example:
.ReactCrop::after {
  display: block;
  position: absolute;
  border: $center-point-border;
  border-radius: 50%;
  width: $center-point-diameter;
  height: $center-point-diameter;
  top: calc(50% - (#{$center-point-diameter} / 2));
  left: calc(50% - (#{$center-point-diameter} / 2));
}

Agreed. There's also another issue that has similar CSS if you want a circular bounding preview.

Thanks for the feedback! I had not thought of doing it purely in CSS. It was just something I wanted for my project. I have not tried forking/pull requests before and wanted to try out the process. I definitely understand not wanting to add API surface area.  Thanks for taking the time to look at it.

Thanks! If you have trouble implementing it without adding another element let me know but it should be achievable


treewide: use /usr/bin/env bash instead of /bin/bash

Please ensure your pull request adheres to the following guidelines:

[ ] For first time contributors, read Submitting a pull request
[ ] All code is covered by unit and/or runtime tests where feasible.
[x] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[ ] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[x] All commits are signed off. See the section Developer’s Certificate of Origin
[x] Provide a title or release-note blurb suitable for the release notes.
[ ] Are you a user of Cilium? Please add yourself to the Users doc
[x] Thanks for contributing!

This commit switch most usage of "/bin/bash" to "/usr/bin/env bash". This allows to conform to the PATH env variable when invoking bash which is especially important on distros/installs in which bash is NOT installed in /bin/bash (for instance in some more "exotic" distros like NixOS or even if bash is installed in /usr/local/bin/ for whatever reasons).
The goal here is not to be perfectly exhaustive but to address most low hanging fruits whenever this happens and most importantly should ease Cilium contributions for people which don't have bash installed in /bin/bash.

Hey :wave:, this comes with a big change over many files it's mostly a sed which affect mostly shebangs with careful re-reading of the change but this could be pretty well split into multiple commits/PR, let me know.

Thanks a lot for the PR!
I think that this change might a little too big in scope. I would probably leave anything in install and images aside from images/scripts as is, as these scripts are not running on host, but in containers either during image build or in k8s pods.
Looking through the change this is a list of folders that should be changed:
* `contrib`

* `examples`

* `images/scripts`

* `test`

Not sure about:
* `bpf`

* `pkg/datapath/loader`

Hi @nebril, thanks for the review! So I ended up dropping everything in images as it's actually slightly more complicated than just keeping images/scripts because those have dependencies on scripts in directories in the the rest of images/ (like runtime and builder IIRC). I propose to come back later with another PR for this directory specifically (+ it's also easier for the CI/merge conflict and all).
For the rest I dropped bpf and pkg/datapath/loader mostly but kept travis/github ones as it's testing related so it doesn't look harmful to keep this (and it's also like 3 files anyway). Let me know WDYT of the new version!

/test

The CI failure (https://jenkins.cilium.io/job/Cilium-PR-K8s-1.27-kernel-net-next/34/) are related to a test that I actually added very recently. Not sure why it launch those test though it's not supposed to be in this branch yet. It seems like the CI use the test from master but with the cilium version from my branch somehow. Do you have an idea if this assumption could be somehow correct @jibi :pray:?
Anyway I just rebased this, pretty sure it should work now if anyone could trigger the tests please :pray:.

Do you have an idea if this assumption could be somehow correct

This is correct, rebasing should do it :)

/test

The test run didn't seem to take for a few of the required jobs. I'll re-kick it.

/test

I just rebased, it will most likely be sufficient to fix the few failures in the latest test run.

Hopefully :) Just be aware that we need to re-run CI each time there is a rebase.

/test

k8s-1.24-kernel-5.4 hit https://github.com/cilium/cilium/issues/15455, known flake. No need to block merging on that failure.

:heavy_check_mark: Merged, thanks for the contribution!


while parsing a block mapping in "", line 1, column 1: blueprint: ^ expected , but found '-' in "", line 432, column 1: - delay: 00:{{cooldown|int}}:00 ^

Bug Description
I get an error when trying to import the blueprint.
while parsing a block mapping in "", line 1, column 1: blueprint: ^ expected , but found '-' in "", line 432, column 1: - delay: 00:{{cooldown|int}}:00 ^
Version: 30 LLM Vision v1.3.1
Blueprint with latest commit on Dec 22

#137 The issue is mentioned here as well. Thanks for raising this!

@valentinfrlch  I have created a pull request with a fix. Do you mind reviewing it.

Thanks @SleepyMoz! The blueprint imports correctly now.

Sorry, that's my bad.


Windows: mapping between unknown chars & valid chars

Information

**Apktool Version (apktool -version) - 2.2.2
**Operating System (Mac, Linux, Windows) - Windows
**APK From? (Playstore, ROM, Other) - URL: http://files.nabstudio.com/mangarock/mangarock-definitive-v1.9.9-RELEASE.apk?cfcache=1

Stacktrace/Logcat
>apktool d "G:\mangarock.apk" --keep-broken-res -f
I: Using Apktool 2.2.2 on mangarock.apk
I: Loading resource table...
W: Duplicate Resource Detected. Ignoring duplicate: string-sr/APKTOOL_DUMMYVAL_0x7f070000
W: Duplicate Resource Detected. Ignoring duplicate: string-sr/APKTOOL_DUMMYVAL_0x7f070001
W: Duplicate Resource Detected. Ignoring duplicate: string-sr/APKTOOL_DUMMYVAL_0x7f070002
W: Duplicate Resource Detected. Ignoring duplicate: string-sr/APKTOOL_DUMMYVAL_0x7f070003
W: Duplicate Resource Detected. Ignoring duplicate: string-sr/APKTOOL_DUMMYVAL_0x7f070004
W: Duplicate Resource Detected. Ignoring duplicate: string-sr/APKTOOL_DUMMYVAL_0x7f070005
W: Duplicate Resource Detected. Ignoring duplicate: string-sr/APKTOOL_DUMMYVAL_0x7f070006
W: Duplicate Resource Detected. Ignoring duplicate: string-sr/APKTOOL_DUMMYVAL_0x7f070007
W: Duplicate Resource Detected. Ignoring duplicate: string-sr/APKTOOL_DUMMYVAL_0x7f070008
W: Duplicate Resource Detected. Ignoring duplicate: string-sr/APKTOOL_DUMMYVAL_0x7f070009
W: Duplicate Resource Detected. Ignoring duplicate: string-sr/APKTOOL_DUMMYVAL_0x7f07000a
W: Duplicate Resource Detected. Ignoring duplicate: string-sr/APKTOOL_DUMMYVAL_0x7f07000b
W: Duplicate Resource Detected. Ignoring duplicate: string-sr/APKTOOL_DUMMYVAL_0x7f07000c
W: Duplicate Resource Detected. Ignoring duplicate: string-sr/APKTOOL_DUMMYVAL_0x7f07000d
W: Duplicate Resource Detected. Ignoring duplicate: string-sr/APKTOOL_DUMMYVAL_0x7f07000e
W: Duplicate Resource Detected. Ignoring duplicate: string-sr/APKTOOL_DUMMYVAL_0x7f07000f
W: Duplicate Resource Detected. Ignoring duplicate: string-sr/APKTOOL_DUMMYVAL_0x7f070010
W: Duplicate Resource Detected. Ignoring duplicate: string-sr/APKTOOL_DUMMYVAL_0x7f070011
W: Duplicate Resource Detected. Ignoring duplicate: string-sr/APKTOOL_DUMMYVAL_0x7f070012
W: Duplicate Resource Detected. Ignoring duplicate: string-sr/APKTOOL_DUMMYVAL_0x7f07003d
W: Duplicate Resource Detected. Ignoring duplicate: string-sr/APKTOOL_DUMMYVAL_0x7f07003e
I: Decoding AndroidManifest.xml with resources...
I: Loading resource table from file: ~\AppData\Local\apktool\framework\1.apk
I: Regular manifest package...
I: Decoding file-resources...
I: Decoding values */* XMLs...
I: Baksmaling classes.dex...
I: Baksmaling classes2.dex...
I: Copying assets and libs...
I: Copying unknown files...
Exception in thread "main" brut.androlib.AndrolibException: brut.directory.DirectoryException: Error copying file: ??♀.xml
        at brut.androlib.Androlib.decodeUnknownFiles(Androlib.java:217)
        at brut.androlib.ApkDecoder.decode(ApkDecoder.java:158)
        at brut.apktool.Main.cmdDecode(Main.java:166)
        at brut.apktool.Main.main(Main.java:81)
Caused by: brut.directory.DirectoryException: Error copying file: ??♀.xml
        at brut.directory.DirUtil.copyToDir(DirUtil.java:88)
        at brut.directory.AbstractDirectory.copyToDir(AbstractDirectory.java:207
)
        at brut.androlib.Androlib.decodeUnknownFiles(Androlib.java:210)
        ... 3 more
Caused by: java.io.FileNotFoundException: \mangarock\unknown\??♀.xml (The filename, directory name, or volume label syntax is incorrect)
        at java.io.FileOutputStream.open(Native Method)
        at java.io.FileOutputStream.<init>(Unknown Source)
        at java.io.FileOutputStream.<init>(Unknown Source)
        at brut.directory.DirUtil.copyToDir(DirUtil.java:84)
        ... 5 more

Steps to Reproduce

apktool d "G:\mangarock.apk" --keep-broken-res -f

APK
http://files.nabstudio.com/mangarock/mangarock-definitive-v1.9.9-RELEASE.apk?cfcache=1
Questions to ask before submission

Have you tried apktool d, apktool b without changing anything? YES
If you are trying to install a modified apk, did you resign it? N/A
Are you using the latest apktool version? YES

This is a duplicate of #1314 , but I closed that one for being a Windows problem. There is no short term fix for this. The work required to create a lookup table of filenames, yet save them back into the apk as the original string is a ton of work.
So that is why I closed the report, use unix/mac until then.

The common and easy way would be to build a white-list of characters which are supported on all platforms/current platform in file-names. Additionally a blacklist of forbidden file-names can be created.
Everything that does not match the whitelist or terms from the blacklist is escaped:
For example:

& is escaped to &
[unicode] characters could be URL encoded © -> ©
forbidden file names like CON -> &!CON;

Then you don't have to store a mapping somewhere. For compatibility with decompiled directories created by older apktool versions you can add a flag in apktool.yml that enabled/disabled escaping and if the option is missing this means escaping is disabled.
And when running on Windows for solving the case sensitivity problem on NTFS drives you could run fsutil.exe file SetCaseSensitiveInfo C:\folder\path enable to enable case sensitivity of the extraction directory.

This would be all solved if Windows ever supported EXT drives

@KarmaMod I don't see a relation between the file-system and disallowed files like CON. Even if Windows would support EXT drives those files will most likely still be unsupported.


Online DPO scheduler step before optimizer step

I see this warning in the test logs when testing online DPO:
tests/test_online_dpo_trainer.py::TestOnlineDPOTrainer::test_online_dpo_trainer_training
  C:\hostedtoolcache\windows\Python\3.10.11\x64\lib\site-packages\torch\optim\lr_scheduler.py:216: UserWarning: Detected call of `lr_scheduler.step()` before `optimizer.step()`. In PyTorch 1.1.0 and later, you should call them in the opposite order: `optimizer.step()` before `lr_scheduler.step()`.  Failure to do this will result in PyTorch skipping the first value of the learning rate schedule. See more details at https://pytorch.org/docs/stable/optim.html#how-to-adjust-learning-rate

So it seems the opt.step and scheduler.step are called in the wrong order.

Fixed in #1918


EDU-942 Create StepSlider component and use it where applicable

Closes https://educandu.atlassian.net/browse/EDU-942

Nice consolidation! 👏 👍

Nice consolidation! clap +1

Thanks!


Update links to wazuh-kibana-app repo

Related Issue
This issue is related to issue 337. The wazuh-kibana-app repo has been renamed to wazuh-dashboard-plugins. You have to change wazuh-kibana-app in the code to wazuh-dashboard-plugins.
Solution Description
These are the changes needed in wazuh-dashboard

Create a branch enhancement/337-change-repo-name from the branch 4.7.0
In the file wazuh-dashboard/README.md, on line 52 you have to change wazuh-kibana-app to wazuh-dashboard-plugins.
In the file wazuh-dashboard/dev-tools/build-packages/README.md, on line 69 you have to change wazuh-kibana-app to wazuh-dashboard-plugins.
Push the changes and create the PR

I have created this PR with the changes for the wazuh-dashboard repo


seg fault/no-output with consensus

Trying to polish some slightly assembled reads but nanpolish consensus gives segfault.  It might be the contig names(?)
$tail consensus.fasta.fai

1904f9ab-fe41-493c-a6ee-d797ae458a1d_Basecall_2D_2d.0.clustal_CONSENSUS 2567    132119  2567    2568
62cf9ca1-f639-41d8-9533-c262ed303880_Basecall_2D_2d.0.clustal_CONSENSUS 2655    134760  2655    2656
f9c44f7e-1713-4e91-9d22-22da4b91edce_Basecall_2D_2d.0.clustal_CONSENSUS 637     137489  637     638
$ wc -l consensus.fasta.fai
72 consensus.fasta.fai
$ rm consensus.fasta.fai

$ nanopolish consensus --verbose --reads basecalls.pp.fa --bam basecalls.pp.sorted.bam --genome consensus.fasta
[fai_load] build FASTA index.
[fai_fetch_seq] The sequence "" not found
Segmentation fault (core dumped)

This is a bit of poor input handling on my part. The -w argument, which specifies what part of the genome to polish, is required. I'll fix this but if you add -w contig:0-10 to the command it should work.

Hi,
I have a very similar error I think. I am trying to use "nanopolish variants" though.
I have nanopore data for 1 gene and I use aligned reads (i.e.: not aligned events).

Aborted
Any idea what could go wrong?
Some additional, general, questions:

I ran nanopolish consensus before on the same data (had a similar error but fixed it), but the output was ONE haplotype!
Did I do something wrong? Given that we are diploid, I would need 2 haplotypes (or more, "corrected" possible reads, based on which to perform variant calling)

Does nanopolish variants output phased or unphased variants? This paper http://www.nature.com/nature/journal/v530/n7589/pdf/nature16996.pdf describes the algorithm a little, but I did not see if the output is phased or not

Best,
Mircea

It would appear that there is a problem with the .bam or .bai file. Can you delete them and regenerate and see if the error still occurs?
For your second question, nanopolish currently does not support diploid genomes. This will be added in the future.


stretch search box

search box needs to aligh with table

stretched search box so this issue is resolved!


Add-ADOPSScaleSet

Add an Azure VM Scale set Agent pool

This function will need some other functions to work;

Manage/delete/add Elastic Pools
Manage/delete/add nodes

Also identified we need to create/manage service endpoints to get this in place.

@SebastianClaesson is this closed with #96 ? New-ADOPSElasticPool should be whats needed to set up the scale sets

Closed with #119 and #96


option to export visualizations (PDF/SVG/PNG) in batch mode

proposed interface:
--export-dir variant_visualizations --format pdf

create variant_visualizations directory if it doesn't exist
warn before overwriting
eventually want some sort of interface to browse the batch output

1e9f79474f61d31d6b5a74b05f16772d43ca9fcf


Create URL scheme for fetching checkin comments/likes on demand

This will make it possible to import past checkins using a local script instead of building everything server-side.

This is done, but not documented.


Lab 5: Time intervals

When I run the code to organize time using intervals as it is indicated in the lab, I've got zeros in all time slots exept for 12 AM.

thank you in advance

thank you for your reply. Actually, I run the code as it is in the template.


chore: add list log in rpc download

Signed-off-by: Jim Ma<EMAIL_ADDRESS>
Description

Related Issue

Motivation and Context

Screenshots (if appropriate)
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation Update (if none of the other choices apply)

Checklist

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.

Codecov Report

Merging #1802 (57b4877) into main (d812c67) will decrease coverage by 0.04%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #1802      +/-   ##
==========================================
- Coverage   43.81%   43.77%   -0.05%
==========================================
  Files         128      128
  Lines       15937    15946       +9
==========================================
- Hits         6983     6980       -3
- Misses       8422     8431       +9
- Partials      532      535       +3

Flag
Coverage Δ

Object-compatibility-e2etests
?

e2etests
?

unittests
43.77% <0.00%> (-0.05%)
:arrow_down:

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

Impacted Files
Coverage Δ

client/daemon/rpcserver/rpcserver.go
59.12% <0.00%> (-0.98%)
:arrow_down:

client/daemon/peer/peertask_file.go
58.38% <0.00%> (-1.35%)
:arrow_down:

...ent/daemon/peer/peertask_piecetask_synchronizer.go
50.15% <0.00%> (-0.94%)
:arrow_down:

scheduler/service/service.go
78.29% <0.00%> (-0.04%)
:arrow_down:

client/daemon/peer/piece_manager.go
53.40% <0.00%> (+0.40%)
:arrow_up:


PHOENIX-6611 Multiple IndexTool bugs

Copying from the ticket:
There is a lot of noise, due to the removal of the directApi option, and movinf the VERIFIED/UNVERIFIED constants to QueryConstants.
The interesting stuff is in:
IndexTool
FormatToBytesWritableMapper
PhoenixIndexImportDirectMapper
PhoenixTxIndexMutationGenerator
IndexUtil
IndexMaintainer
IndexToolIT
IndexBuildTimestampIT

Sorry for asking for reviews for a half-done patch.
It is really ready for review now.
(The last patch is just just formatting fixes, and does not change the behaviour.)

Thank you @gjacoby126  and @gokceni.
I have implemented your suggestions, and explained the TX index change.

@stoty Thank you for implementing suggestions. There seems to be unit test failures. Could you please check if they are related to this patch? It LGTM but please make sure that tests are not broken.

@gokceni
I had to rebase this on top your TransformTool work.
For now, I have added logic in TransformMaintainer to set the VERIFIED status according to the old table indexing style.
Please check if that is acceptable solution (at least until PHOENIX-6617)

I have removed the VERIFIED I added to TransformMaintainer, as it is not necessary after PHOENIX-6617.
I think that it would be a good idea to check if Transform handles the VERIFED status for transactional tables correctly, to avoid issues like IndexTool and Bulk Load had.


Cool Animations

Hi, TnQ for this great lib
can u add transition animations like this lib:
https://github.com/TakeoffAndroid/AppIntroAnimation
if u see there is a cool color changing when user slides
Tnx Again for this amazing lib

I've seen it in Google Drive as well. Would be great if AppIntro supports it!

I have done it using this tutorial (http://kubaspatny.github.io/2014/09/18/viewpager-background-transition/) as guide.. Hope it will be useful for someone..
package com.sathis.android.ui.activity;

import android.animation.ArgbEvaluator;
import android.content.Intent;
import android.graphics.Color;
import android.os.Bundle;
import android.support.v4.view.ViewPager;

import com.github.paolorotolo.appintro.AppIntro2;

import com.xytasks.android.ui.view.intro.slides.Slide;
import io.github.froger.instamaterial.R;
import com.xytasks.android.ui.utils.PreferenceUtils;

/**
 * Author: sathis
 * Last Updated: 28/09/15.
 */
public class IntroActivity extends AppIntro2 implements ViewPager.OnPageChangeListener{

    public static final String INTRO_DONE = "intro_done";

    private ViewPager mPager;
    private ArgbEvaluator argbEvaluator = new ArgbEvaluator();
    private Integer[] colors = {Color.argb(205, 54, 215, 183), Color.argb(205, 245, 171, 53), Color.argb(225, 210, 82, 127)};

    @Override
    public void init(Bundle savedInstanceState) {
        addSlide(Slide.newInstance(R.layout.intro_slide_1));
        addSlide(Slide.newInstance(R.layout.intro_slide_2));
        addSlide(Slide.newInstance(R.layout.intro_slide_3));
        mPager = getPager();
        mPager.addOnPageChangeListener(this);
    }

    private void loadMainActivity(){
        PreferenceUtils.set(this, INTRO_DONE, true);
        Intent intent = new Intent(this, MainActivity.class);
        startActivity(intent);
    }

    @Override
    public void onDonePressed() {
        loadMainActivity();
    }

    @Override
    public void onPageScrolled(int position, float positionOffset, int positionOffsetPixels) {
        if(position < (mPager.getAdapter().getCount() -1) && position < (colors.length - 1)) {
            mPager.setBackgroundColor((Integer) argbEvaluator.evaluate(positionOffset, colors[position], colors[position + 1]));
        } else {
            mPager.setBackgroundColor(colors[colors.length - 1]);
        }
    }

    @Override
    public void onPageSelected(int position) {

    }

    @Override
    public void onPageScrollStateChanged(int state) {

    }
}

@msathis please consider making a pull request for this.

Really sorry to bug you guys but I can't see where this has been implemented in latest release. The issue is closed but a quick scan through commits for this release didn't yield anything related to custom transformer.
Oh and thanks @msathis for the code! You (and this library) saved us a lot of time.

so i didnt know how to use this feature.
can anyone help plz? is this feature added or not?

I think @msathis never created the PR. One condition in his code is that the layout that is being added as a slide must have a transparent background. @Andrew-Quebe do u want me to create the PR with some minor refactoring?

@girish3 if you'd like :) Contributions are always appreciated.


RPi doesn't show up in Spotify Connect

I installed raspotify yesterday, and it worked like a charm. Changed the bitrate to 320, entered my login information and set a name for the device. When I woke up a few hours ago I couldn't find the device anymore. Not on my pc, not on my phone. I rebooted the RPi, but this didn't fix the issue. I also deleted & installed raspotify again, but this also did not work.
Edit: Found the issue.
This issue can be closed

Please close this issue yourself with the button below to reduce the clutter in the list of issues. Thank you!


The dev/devicelab/lib/web_benchmarks.dart test has issues finding the right Chrome tab

On our head-head-head builder we have started to see timeouts. For example this build.

Looking at the corresponding code, it seems that after successfully launching Chrome, waiting for DevTools to be ready, connecting to DevTools it cannot find a tab with the right url:
Future<WipConnection> _connectToChromeDebugPort(io.Process chromeProcess, int port) async {
  ...
  final Uri devtoolsUri = await _getRemoteDebuggerUrl(Uri.parse('http://localhost:$port'));
  print('Connecting to DevTools: $devtoolsUri');
  final ChromeConnection chromeConnection = ChromeConnection('localhost', port);
  final Iterable<ChromeTab> tabs = (await chromeConnection.getTabs()).where((ChromeTab tab) {
    return tab.url.startsWith('http://localhost');
  });
  final ChromeTab tab = tabs.single; // <-- fails
  ...
}

see flutter/dev/devicelab/lib/framework/browser.dart
@yjbanov seem to have authored the test. Do you have any clues why this might start happening?

It looks like your Chrome version is 3 versions behind ours. We currently use 79.0.3945.130. Can you please update Chrome and try again?

@athomas Could we upgrade the chrome on the HHH builders?

Yes, I'll update chrome.

I've updated chrome on the 3xH builders to 81.0.4044.138 in https://dart.googlesource.com/recipes/+/2c49c412121e80ac9ad4572accd8df815a15fd71.

Let's close the issue then.


Whole line, and the insertion cursor become invisible.

To reproduce select all the code and run hydrogen
than select the 'i' in the 2 line in the for loop  "for i in range(10):"
temp = 0
for i in range(10):
temp += 1
print i
This causes line 3 "temp +=1" to become invisible also the insertion cursor is gone in there

If i restart the kernel it's ok
Or i can undo/redo to the point where i wrote the temp +=1 line
I am on Windows 10 if it matters.

This looks like a styling issue.
Which theme are you using and can you reproduce this with the standard theme?

@lgeiger you are right! I tried it with one dark everything is fine..
I am using the Atom Material theme.. can you try and reproduce it on your side?
Thank you!

Thanks man


Include inherited classes in Filter Nodes' "type:" filter

Continues from #58377.
Also fixes not all private groups being skipped in the "group:" filter.

It would be nicer to be able to use ClassDB::is_parent_class() directly instead of reimplementing it manually. But I understand that this wouldn't match when the case is wrong or on substrings.

There could be a new method in ClassDB method that does what I need for this, but I can't think of any use-case besides this one, nor I can imagine how it could be called.

Based on https://github.com/godotengine/godot/pull/65932#issuecomment-1268797923 by @KoBeWi, I modified the "group:" filter here, to account for internal groups properly. In theory, it should exclude them appropriately, in practice it may also account for properties such as "_edit_lock_" and similar, but with my testing it didn't seem like it did.

Ah, right. I didn't notice it's a loop for some reason, just seen the return true at the end. Disregard my comment then 🙃

It's ok. Your confusion hints me there's a bit of a code smell around there. As more filters could be implemented, it may be cleaned up appropriately.


[WIP] Log results of transformers

Issue
https://github.com/wellcometrust/platform/issues/3784
Description
This PR adds logging so it is possible to see the results of the transformer. For relevant fields it adds logging output, as displayed below.
This is still a work in progress, and may need some discussion.
Example log output
The following has been split across multiple lines to make it easier to read:
14:53:48.463 [pool-77-thread-10-ScalaTest-running-SierraProductionTest] INFO  u.a.w.p.t.s.t.SierraProduction$ -
DATA TRANSFORMED:
BibID=6988525
Transformer=SierraProduction
Info=Production events extracted from field 260 (dates not extractable, used 008 for dates),
Output=List(ProductionEvent(London,List(Place(London)),List(),List(Period(1757,Some(InstantRange(1757-01-01T00:00:00Z,1757-12-31T23:59:59.999999999Z,1757,false)))),None,ProductionEvent))

@warrd as I'm looking at it - could you give an example of what you might want this to look like as a JSON structured log?

@jamesgorrie well as an example, the above output might look like this if it was represented as JSON:
{
  "logLevel": "INFO",
  "timestamp": "14:53:48.463",
  "logType":"transformer-output",
  "data": {
    "bibId": "6988525",
    "transformer": "SierraProduction",
    "infoString": "Production events extracted from field 260 (dates not extractable, used 008 for dates)",
    "info": {
      "primaryMarcField": "260",
      "dateMarcField": "008"
    },
    "output": "List(ProductionEvent(London,List(Place(London)),List(),List(Period(1757,Some(InstantRange(1757-01-01T00:00:00Z,1757-12-31T23:59:59.999999999Z,1757,false)))),None,ProductionEvent)"
  }
}

Closing this for now


Call to undefined method [package]

Hey!
Just installed it on Laravel 5.1 - got this error.
in ServiceProvider.php line 234
at ServiceProvider->__call('package', array('league/statsd', 'statsd')) in StatsdServiceProvider.php line 23
at StatsdServiceProvider->package('league/statsd', 'statsd') in StatsdServiceProvider.php line 23
at StatsdServiceProvider->boot()
at call_user_func_array(array(object(StatsdServiceProvider), 'boot'), array()) in Container.php line 507
at Container->call(array(object(StatsdServiceProvider), 'boot')) in Application.php line 734
at Application->bootProvider(object(StatsdServiceProvider)) in Application.php line 717
at Application->Illuminate\Foundation\{closure}(object(StatsdServiceProvider), '20')
at array_walk(array(object(EventServiceProvider), object(RoutingServiceProvider), object(AuthServiceProvider), object(ControllerServiceProvider), object(CookieServiceProvider), object(DatabaseServiceProvider), object(EncryptionServiceProvider), object(FilesystemServiceProvider), object(FormRequestServiceProvider), object(FoundationServiceProvider), object(PaginationServiceProvider), object(SessionServiceProvider), object(ValidationServiceProvider), object(ViewServiceProviderEx), object(AppServiceProvider), object(EventServiceProvider), object(RouteServiceProvider), object(SlackServiceProvider), object(BugsnagLaravelServiceProvider), object(ServiceProvider), object(StatsdServiceProvider), object(HashServiceProvider)), object(Closure)) in Application.php line 718
at Application->boot() in BootProviders.php line 17
at BootProviders->bootstrap(object(Application)) in Application.php line 203
at Application->bootstrapWith(array('Illuminate\Foundation\Bootstrap\DetectEnvironment', 'Illuminate\Foundation\Bootstrap\LoadConfiguration', 'Illuminate\Foundation\Bootstrap\ConfigureLogging', 'Illuminate\Foundation\Bootstrap\HandleExceptions', 'Illuminate\Foundation\Bootstrap\RegisterFacades', 'Illuminate\Foundation\Bootstrap\RegisterProviders', 'Illuminate\Foundation\Bootstrap\BootProviders')) in Kernel.php line 222
at Kernel->bootstrap() in Kernel.php line 117
at Kernel->sendRequestThroughRouter(object(Request)) in Kernel.php line 87
at Kernel->handle(object(Request)) in index.php line 54

Thanks for the bug report. I have no experience with Laravel so I'm going to lean on @philsturgeon @aranw @nikkiii who've all contributed code to the Laravel service provider here.
There is also pull request #21 which should add Laravel 5 support (and hopefully fix this issue). I'm happy to merge that if someone can confirm it works with Laravel 5 (I have no experience or way to test it).

Yeah the work I did added a service provider for Laravel 4. I do believe the service provider currently in the project is only for Laravel 4 and not 5.
I'd need to dig into the code a little more. I could also look over #21 and check the code for you as well if you like? @marcqualie

@marcqualie #21 looks good to go unless you have any folder, code style, documentation changes you'd like to see?

Thanks @aranw I've now merged that PR, thanks for the great work @nikkiii
@lezhnev74 if you could test the latest dev release and confirm it fixes your issue in Laravel 5 I can create a new release.
@aranw I'm unfamiliar with the latest code standards of PHP so feel free to suggest/PR changes before I create the next release containing this service provider.

works for me on L5

Just need to update the composer install line. The version 1.3.0 that composer pulls down lacks the Laravel5 folder. Updating the version in the composer file to 'dev-master'  worked for me.


how to make changes in base code?

what if i have to write code on the specific first line of sales header's field's on validate trigger?

you have to use OnBeforeValidate event
https://docs.microsoft.com/en-us/dynamics-nav/introducing-events

Please read about event recorder here:
https://blogs.msdn.microsoft.com/nav/2018/07/30/developer-preview-july-2018-update/
And about browsing into Base Application code from AL here (section Debugger Enhancements):
https://blogs.msdn.microsoft.com/nav/2018/06/25/developer-preview-june-2018-update/


Fix fix::fix_in_dependency to not rely on rustc

This changes the fix::fix_in_dependency test so that it does not rely on the behavior of rustc.
This test is checking the behavior when rustc includes a suggestion diagnostic that modifies a file in CARGO_HOME from a dependency. rustc should not be emitting suggestions that point outside of the current crate, but there are some known bugs where it does this. #9938 added a workaround for this to avoid writing to CARGO_HOME.
However, the current test was relying on one of those bugs in rustc to exhibit its behavior. https://github.com/rust-lang/rust/pull/119204 is fixing that particular behavior. Instead of relying on issues in rustc, this PR changes the test so that the test creates a fake rustc executable that will emit a hard-coded JSON message that tells cargo fix to modify a file in CARGO_HOME. This should be stable as it is no longer relying on rustc.
Testing or validating this change is a little tricky. You have to essentially comment out these three lines from the original #9938 change, and verify that the test fails (it says "Fixed" in the output). With those three lines in place, it should pass.

@bors r+

:pushpin: Commit 1ef1b7e313534c59b712d742bc4e4445aa510c92 has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit 1ef1b7e313534c59b712d742bc4e4445aa510c92 with merge 903a50991981af37bd8ceb9f5c4a3972c881f6c5...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing 903a50991981af37bd8ceb9f5c4a3972c881f6c5 to master...


Forever start command runs multiple processes

When i start an app with forever like this:
forever start app.js
It doesn't just start one process. in Ubuntu when i check processes with "htop" it shows that 5 processes of app.js are running and 5 instances of monitor app.js are running.
How can i fix this? Is it consuming CPU?

+1, what's the cause?

Today I did forever start app.js and below is what I am seeing:

It created total 9 node app.js processes with 5 monitor app.js processes.
It is not clear if this is a bug or feature. How are these processes related to each other? The app.js contains networking code (doing some pub/sub activity) - and it was not designed to have more than one processing running at the same time in mind. Not sure how our functionality is going to be affected by this.
Any insights into this behavior and options to turn this off would be very helpful.


Better import path

Description
It's quite boring to start every file with use meilisearch_sdk::{document::*, client::*};
It would be better if in the lib.rs we were re-exporting every structure like that:
pub use client::*;
pub use index::*;
...

Basic example
And thus let us (=the users) write import like that;
use meilisearch_sdk::*;

Or if you want a few basic structure structures:
use meilisearch_sdk::{Client, Index, Setting};
// instead of
use meilisearch_sdk::{client::Client, indexes::Index, settings::Setting};

Other
Also, I never remember which module name is plural or singular so it would hide our inconsistencies in a non-breaking change.
Another solution would be to provide a prelude like in most libraries but I'm not sure we really need it currently since our lib is so smol :thinking:

Hi! Can I work on this issue?


Improving the entire UI and want to make it responsive.

As this website is not mobile friendly I want to redesign the entire ui for mobile and make it responsive and user friendly.
Please assign this issue to me under GSSoC'23

Go Ahead @adityatyagi-av


fix: fix get-job-artifacts

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/ankorstore/toolbelt?version=dev:20741842dc713793aefc95eb5d99d812ecc370c1

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/ankorstore/toolbelt?version=dev:d95b681a90e18854153b98bae70031a44f575b2e

Your orb has been published to the CircleCI Orb Registry.
You can view your published orb on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/ankorstore/toolbelt?version=0.4.9


fix IAM role name in README

Previously, the unescape "<" and ">" lead to the name of the role being invisible, showing as '.".

Thanks @jschauma for submitting the pr. Please add sign-off on the commit. More details at https://github.com/AthenZ/athenz/blob/master/CONTRIBUTING.md#sign-your-work

The git commit message from my account needs to
contain a line saying that my account signs off?
I.e.,
"Signed-off-by: Jan Schaumann @.***>"
?

Signed-off-by: Jan Schaumann<EMAIL_ADDRESS>¯_(ツ)_/¯

Ok, tried that.
If that's insufficient, maybe you can just ditch this
PR and make the trivial change yourself?

addressed in #1802

Thanks!


Error when not logged in. TypeError: Cannot read property 'classList' of undefined

TypeError: Cannot read property 'classList' of undefined on line 513. Its trying to use dataReactRoot 's 2nd child when it only has one.

Fixed as of 9b99df0d7d77f07283b276efd5af7eced9371ed6


several vulnerabilities CVE-2020-15168,CVE-2020-7598 are introduced in strider

Hi, several vulnerabilities CVE-2020-15168,CVE-2020-7598 are introduced in strider via:
●<EMAIL_ADDRESS>➔<EMAIL_ADDRESS>➔<EMAIL_ADDRESS>However, opencollective is a legacy package, which has not been maintained for about 4 years.
Is it possible to migrate opencollective to other package to remediate this vulnerability?
I noticed several migration records in other js repo for opencollective:

in commitizen, version 2.10.1 ➔ 3.0.0, remove opencollective via commit
in fast-xml-parser, version 3.3.0 ➔ 3.3.1, remove opencollective via commit
in react-slick, version 0.12.1 ➔ 0.12.2, remove opencollective via commit
in level, version 3.0.1 ➔ 3.0.2, migrate opencollective to opencollective-postinstall via commit
in ngx-infinite-scroll, version 7.0.1 ➔ 7.1.0, migrate opencollective to opencollective-postinstall via commit
in inferno, version 7.1.8 ➔ 7.1.9, migrate opencollective to opencollective-postinstall via commit

Are there any efforts planned that would remediate this vulnerability or migrate opencollective?
Thanks.

Thanks for the issue. Will remove it.


drop an "Extended Symbol" page when its children are curated elsewhere

Bug/issue #, if applicable: rdar://107729630
Summary
When Swift-DocC gets symbol graphs with extension block symbols, it creates "Extended Module/Class/Structure/etc" symbols to automatically curate the extension symbols into. However, if the extension symbols are then curated in the rest of the documentation hierarchy, these extra symbols are left in the topic graph and create empty pages that don't need to be there. This PR adds a pass to DocumentationContext.register(_:) to drop these symbols if all their children have been curated elsewhere. (This pass is done recursively, so "Extended Module" pages will drop if all their child "Extended Class/Structure/etc" pages are also dropped.)
Dependencies
None
Testing
The following code was used to test this functionality:
/// a namespace for organization.
///
/// ## Topics
///
/// - ``Swift/Array/asdf``
public enum MyNamespace {}

public extension Array {
  var asdf: Int { count }
}

Steps:

Build a symbol graph for the above code that includes extension block symbols. (The code above was used to create the ModuleWithSingleExtension test fixture added in this PR, which can be used as a shortcut.)
Add the resulting symbol graphs to a documentation catalog and generate documentation.
Ensure that the resulting documentation doesn't contain the "Extended Module" page for Swift, nor the "Extended Structure" page for Array.

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran the ./bin/test script and it succeeded
[ n/a ] Updated documentation if necessary

@swift-ci Please test

@theMomax Thanks for the review! Right now it looks like the behavior for when you actually curate or reference a page is a bit awkward:

If you merely reference an extension page without curating it, the link is still considered "live" but the destination page doesn't exist.
If you curate it into the top-level page (i.e. ModuleWithSingleExtension) the same thing happens: The link is still there, but the page is not.
If you curate it into a second-level page that's not also an extension (e.g. the MyNamespace enum in the demo project i added) then that copy of the page still exists, alongside all its children! This is because my code doesn't recurse into non-extension pages, so it sees an enum and skips it, despite there being an extension curated in there.

I think what i would want to happen in the case of a non-curating reference is your option (2), with maybe some extra logic for if there are extension files for the extension pages that keep them "live". I'll take a look at adding these suggestions.

I just pushed a couple commits to handle @theMomax's review comments. They've added the following functionality:

Links to "Extended Symbol" pages which have been removed will generate a warning: "The topic 'TopicName' is an empty extension page and cannot be linked to." I'm not 100% confident in the diagnostic replacement system to be able to recommend dropping the symbol link to an inline code span (or some other kind of recommendation), so i left that blank for now.
"Extended Symbol" pages which have a documentation extension file of their own are not dropped, even if all their children have been canonically curated elsewhere. This also means that symbol links to them will not generate a warning since they will properly resolve.

@swift-ci Please test

I'm seeing an issue with symbols that are included in relationships sections. Here I built ArgumentParser with:
swift package --disable-sandbox preview-documentation --target ArgumentParser --include-extended-types --experimental-skip-synthesized-symbols

The resulting docs are much nicer now (they exclude all of the empty Bool, Int64, etc pages) but now I'm seeing broken links in the relationships sections for ExpressibleByArgument. When I click on one of the links I end up on a non-existent page.
Ideally I think these symbols would still appear but as monospace references instead:
### Conforming Types

`Swift.Bool`
`Swift.Double`
...

@ethan-kusters Oh right, i wanted to look into that! 😅 I don't know offhand where those are rendered, but i can look at that next.

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test

I'm not sure if it should be resolved yet, but even with the latest changes I'm still able to reproduce the issue I was seeing with ArgumentParser.

@ethan-kusters I just pushed up a solution. Can you try it now?

@swift-ci Please test

@ethan-kusters I just pushed up a solution. Can you try it now?

Woot! Looks great.

In other cases like this we include the module namespace (Swift.Foo):

Is it simple to do the same thing here? If not – we can definitely leave it for a future PR.

That seems doable - the title defers to the LinkTitleResolver, which itself seems to defer to the symbol information. However, i feel like this PR has ballooned a little bit and the current state of things is serviceable for the moment. I can file an issue to follow up if you'd like.

However, i feel like this PR has ballooned a little bit and the current state of things is serviceable for the moment. I can file an issue to follow up if you'd like.

That would be great. Thank you!

@ethan-kusters Just filed https://github.com/apple/swift-docc/issues/554 to track it.


trackers working very slow

Hi! i tried your trackers on video and they work very slow for me (~10 second on iteration). i'm tried to use moose and kcf trackers.
i'm use:
cupy-cuda100                  7.1.1
mxnet-cu100                   1.5.1.post0
Cuda compilation tools, release 10.0, V10.0.130
what could be the problem?

I just implemented it based on CPU, and did not consider too much about efficiency. Sorry for that.


[FLINK-35264] Fix multiple transform rules do not take effect

This PR fixes multiple transform rules do not take effect since transform rules are overwritten in for-loop, and adds transform IT test cases.

cc @aiwenmo @PatrickRen

@PatrickRen Sure, done in FLINK-35264.


Clean up extension storage (cache) location on uninstall

Clean up extension storage (cache) location on uninstall

This is already supported


Allow scheme, username and password in connections

Endpoint namedtuple changes:

gains a new item netloc a position 0
Username and password are stored in host item 1

Use urllib to parse connection strings

Coverage increased (+0.005%) to 85.016% when pulling ff08c387561fb586739a28c68dbaf51230f98db9 on elastic-coders:allow_endpoint_auth_and_https into a15a6cbc886d8afb67782947571676587e5b9a84 on aio-libs:master.

Coverage increased (+0.4%) to 85.456% when pulling 7105da753c1ec6811b8774c4a9f4526baa55ec9e on elastic-coders:allow_endpoint_auth_and_https into a15a6cbc886d8afb67782947571676587e5b9a84 on aio-libs:master.

Coverage increased (+0.4%) to 85.456% when pulling b80bc0336d073d684fb88be4288bcaeae5c0ad4d on elastic-coders:allow_endpoint_auth_and_https into a15a6cbc886d8afb67782947571676587e5b9a84 on aio-libs:master.

Coverage increased (+0.07%) to 85.081% when pulling 712d9f6912b25fd7c73b98be526fa5fa3a2d9a68 on elastic-coders:allow_endpoint_auth_and_https into a15a6cbc886d8afb67782947571676587e5b9a84 on aio-libs:master.

I am using this in staging and it works OK with bonsai.io hosting. Is it ok to merge?

Thanks you, Marco.
Sorry for long delay.
Would you a new aioes release?


Error on Node 16.17.0

I'm trying to access this function on Node 16.17.0 jwt.sign({ ...payload }, secret);

This library is intended for use with Cloudflare Workers and is built on top of SubtleCrypto. Node.js uses its own internal Crypto module module instead and is therefore not compatible.


Error when doing something about git

Here is a screenshot:

Here is my .vimrc:
https://github.com/wi1d5ky/myConfig/blob/master/.vimrc

Problem solved.
Sorry that I forgot to set core.editor="vim" in git config file.


New icon animation: Explore

Name of icon to be animated(as shown in Carbon Icon Library:
Explore
Category of icon:
Operations
Team/product requesting:
Carbon Icon Motion Group
Contacts:
@kristastarr @johnbister
When is this needed?
With the second batch of icons (operations)

This one is good to go--I'll make the PR on Monday!


communication app source missing from repo

Describe the bug
the validation app linked at the top of https://tauri.studio/ has a button on the bottom - "source for this app"

This button points to https://github.com/tauri-apps/tauri/tree/dev/tauri/examples/communication, however there are only three example apps in that repo - 'api', 'helloworld' and 'multiwindow'. Looks like 'communication' was removed or never added.

nvm I just realised the api src is the correct one, I'm just going to do a PR to the api app to fix that footer link


unable to exclude all files using files.watcherExclude

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.64.2
OS Version: macOS Big Sur 11.6.1

Steps to Reproduce:
Even with these setting I still get Unable to watch for file changes in this large workspace folder:
{
  "files.watcherExclude": {
      "**/.bld/**": true,
      "**/.git/objects/**": true,
      "**/.git/subtree-cache/**": true,
      "**/node_modules/**": true,
      "**/.hg/store/**": true,
      "*": true,
      "**": true,
      "**/*": true,
      "**/**": true,
      "**/*/**": true,
  }
}

Excluding something more specific works, show that it is not the system itself run out of handles:
{
  "files.watcherExclude": {
    ...,
    "**/src/**": true
  }
}

Also, glob pattern is not working properly.
e.g. **/src/** works while **/src/[a-z]*/** doesn't.
There are no [A-Z0-9]* files/sub-folder in that folder.
We you want, I can create a different issue for it.


fix: split headers into multiple lines

(This is the same as #1338 by @NeOMakinG , but from a branch inside this repo so we can get CF Pages to do a preview build. The previous PR's description is copied below.)
Description
As the @mrnerdhair  asked me to avoid modifying the cspMeta object, I used a .map and a .reduce in order to generate the first CSP line of _headers and generate every line containing an URL
Notice

[x] Have you followed the guidelines in our Contributing guide?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[x] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)
closes #1024

Risk

CloudFlare headers could be wrong

Testing

Try to build the app and verify the content of build/_headers
As A/C said, it should be tested in the preview environment :(

Screenshots (if applicable)
Content of the _headers file
/*
  Cache-Control: no-transform
  Content-Security-Policy: default-src 'self'; child-src 'self' blob: 'report-sample'; connect-src 'self' data:; frame-src; img-src 'self' data: blob: filesystem:; script-src 'self' blob: 'unsafe-eval' 'unsafe-inline' 'report-sample'; style-src 'self' 'unsafe-inline' 'report-sample'; base-uri 'none'; object-src 'none'; frame-ancestors 'none'
  Content-Security-Policy: connect-src https://api.0x.org
  Content-Security-Policy: connect-src https://gas.api.0x.org
  Content-Security-Policy: connect-src https://api.coingecko.com
  Content-Security-Policy: connect-src https://api.yearn.finance
  Content-Security-Policy: connect-src https://cache.yearn.finance/v1/chains/
  Content-Security-Policy: connect-src https://test-api.yearn.network/v1/
  Content-Security-Policy: connect-src https://api.thegraph.com/subgraphs/name/salazarguille/yearn-vaults-v2-subgraph-mainnet
  Content-Security-Policy: connect-src https://raw.githubusercontent.com/yearn/yearn-assets/
  Content-Security-Policy: connect-src https://raw.githack.com/trustwallet/assets/
  Content-Security-Policy: connect-src https://api.zapper.fi/v1/prices
  Content-Security-Policy: connect-src https://meta.yearn.network/vaults/1/all
  Content-Security-Policy: connect-src https://api.github.com/repos/yearn/yearn-assets/
  Content-Security-Policy: connect-src https://api.coincap.io/v2/assets
  Content-Security-Policy: connect-src https://api.coincap.io/v2/assets/
  Content-Security-Policy: connect-src https://api.gem.co/institutions/coinify/supported_currencies
  Content-Security-Policy: connect-src https://api.gem.co/institutions/wyre/supported_currencies
  Content-Security-Policy: connect-src https://gem-widgets-assets.s3-us-west-2.amazonaws.com/currencies/crypto/
  Content-Security-Policy: connect-src https://onramp.gem.co
  Content-Security-Policy: connect-src https://api-osmosis.imperator.co/tokens/
  Content-Security-Policy: connect-src https://mainnet.infura.io/v3/d734c7eebcdf400185d7eb67322a7e57
  Content-Security-Policy: connect-src https://dev-api.ethereum.shapeshift.com
  Content-Security-Policy: connect-src wss://dev-api.ethereum.shapeshift.com
  Content-Security-Policy: connect-src https://dev-api.bitcoin.shapeshift.com
  Content-Security-Policy: connect-src wss://dev-api.bitcoin.shapeshift.com
  Content-Security-Policy: connect-src https://dev-api.cosmos.shapeshift.com
  Content-Security-Policy: connect-src wss://dev-api.cosmos.shapeshift.com
  Content-Security-Policy: frame-src https://fwd.metamask.io/
  Content-Security-Policy: frame-src https://widget.portis.io
  Content-Security-Policy: img-src https://gem-widgets-assets.s3-us-west-2.amazonaws.com/currencies/crypto/
  Content-Security-Policy: img-src https://assets.coincap.io/assets/icons/
  Content-Security-Policy: img-src https://static.coincap.io/assets/icons/
  Content-Security-Policy: img-src https://assets.coingecko.com/coins/images/
  Content-Security-Policy: img-src https://raw.githack.com/trustwallet/assets/
  Content-Security-Policy: img-src https://rawcdn.githack.com/yearn/yearn-assets/
  Content-Security-Policy: img-src https://raw.githack.com/yearn/yearn-assets/
  Content-Security-Policy: img-src https://assets.yearn.network/tokens/
  Content-Security-Policy: img-src https://raw.githubusercontent.com/yearn/yearn-assets/
  Content-Security-Policy: img-src https://rawcdn.githack.com/trustwallet/assets/
  Content-Security-Policy: img-src https://raw.githubusercontent.com/osmosis-labs/
  Content-Security-Policy: img-src https://raw.githack.com/shapeshift/lib/
  Content-Security-Policy: img-src https://raw.githubusercontent.com/shapeshift/lib/
  Cross-Origin-Opener-Policy: same-origin-allow-popups
  Permissions-Policy: document-domain=()
  Referrer-Policy: no-referrer
  X-Content-Type-Options: nosniff
  X-Frame-Options: DENY

@mrnerdhair is this ready for review? i see the ??= works with node 16 now

@mrnerdhair is this ready for review? i see the ??= works with node 16 now

No, it's not, this way of splitting isn't working regarding the CSP spec
We're kinda blocked right now :/

thanks @NeOMakinG - can you elaborate with references here?
@mrnerdhair what does this mean for IPFS?

(Hmm. Was sure I wrote a paragraph about this yesterday, but I don't see it anywhere.)

Unfortunately header splitting doesn't work like I hoped, and it's on me for speccing this out wrong. The CSP produced by this PR is rendered in the chrome dev tools as shown above, which makes it look at first glance like the directives from each header are merged; however, on closer inspection you can see that all the directives are shown joined with a comma, not a semicolon:
In my research on this, I thought that I was seeing a single CSP merged together from directives delivered in different headers; however, the comma seems to be the chrome dev tools' extremely visually subtle way of distinguishing that from the notion of receiving multiple separate CSPs, one for each header.
Multiple CSPs don't work at all, because they're all deny-by-default -- one header disallows everything but https://example1.com and another one disallows everything but https://example2.com, resulting in everything being disallowed because nothing matches both headers.
I pulled this back from in-review to draft to do some research and see if this approach could be salvaged; however, because this (the 1000-char header limit) is fundamentally caused by a CF Pages limitation, I now think the best solution will be to use the same ServiceWorker machinery that we'll need for IPFS and Alpha.

Closing pending ServiceWorker spec & impl.


Use any_ip in process_arp

Currently, process_arp will not process a request if the ip_addrs doesn't contain its target. This could lead to weird situations, when any_ip address is used, but arp has to be handled somehow otherwise. An example use-case is a benchmarking tool that opens bunch of connections to the target using different addresses for each one.

Thank you! I added a test for the feature.

Great, thanks!


MINOR: Defer log recovery until LogManager startup

Currently log recovery begins as soon as we instantiate LogManager, but when using a Raft-based metadata quorum we won't have configs until after we catch up on the metadata log.  We therefore defer log recovery until we actually invoke startup() on the LogManager instance.  This timing difference has no effect when using ZooKeeper because we immediately invoke startup() on the instantiated instance, but it gives us the necessary flexibility for accurate log recovery with updated configs when using a Raft-based metadata quorum.
The LogCleaner is currently instantiated during construction just after log recovery completes, and then it is started in startup().  As an extra precaution, since we are no longer performing recovery during construction, we both instantiate and start the log cleaner in startup() after log recovery completes.
We also convert LogManager to use a ConfigRepository to load topic configs (which can override the default log configs) instead of having a hard-coded dependency on ZooKeeper.  We retrieve the topic configs when we invoke startup() -- which again is effectively no different from a timing perspective than what we do today for the ZooKeeper case.
One subtlety is that currently we create the log configs for every topic at this point -- if a topic has no config overrides then we associate a copy of the default configuration with the topic inside a map, and we retrieve the log configs for that topic's partitions from from that map during recovery.  This PR makes a change to this series of events as follows.  We do not associate a copy of the the default configuration with a topic in the map if the topic has no configs set when we query for them.  This saves some memory -- we don't unnecessarily copy the default config many times -- but it also means we have use the default log configs for that topic later on when recovery for each of its partitions begins.
The difference is that the default configs are dynamically reconfigurable, and they could potentially change between the time when we invoke startup() and when log recovery begins (log recovery can begin quite some time after startup() is invoked if shutdown was unclean).  Prior to this patch such a change would not be used; with this patch they could be if they happen before recovery begins.  This actually is better -- we are performing log recovery with the most recent known defaults when a topic had no overrides at all. Also, Partition.createLog has logic to handle missed config updates, so the behavior is eventually the same.
The transition of the broker state from STARTING to RECOVERY currently happens within the LogManager, and it only occurs if the shutdown was unclean.  We move this transition into the broker as it avoids passing a reference to the broker state into the LogManager.  We also now always transition the broker into the RECOVERY state as dictated by the KIP-631 broker state machine.
Finally, a few clean-ups were included. One worth highlighting is that Partition no longer requires a ConfigRepository.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

One unrelated flaky failure:
Build / JDK 15 / kafka.controller.ControllerIntegrationTest.testPartitionReassignmentToBrokerWithOfflineLogDir()
One of the builds exited early, which is unrelated too. To be sure, restarted the tests.

Latest changes LGTM

The JDK 8 build had unrelated flaky test failures this time while JDK 11 and 15 terminated before completion (again unrelated). Will go ahead and merge.

Merged to trunk and 2.8 branches.


split off product specific part of configure agents docs

Description

Split off product specific part of configure agents docs so that product docs can overwrite it.
Updated postgres installation instructions to be more in line with the iso instructions.

part of inmanta/inmanta-service-orchestrator#245
companion PR inmanta/inmanta-service-orchestrator#247
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[ ] Type annotations are present
[ ] Code is clear and sufficiently documented
[ ] No (preventable) type errors (check using make mypy or make mypy-diff)
[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )

Reviewer Checklist:

[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Code is clear and sufficiently documented
[ ] Correct, in line with design

Processing this pull request

Merged into branches iso5, master in 664e4773a473d6ef82a78b754d54efa5cc029f64


Draw rulers all the way when scrollPastEnd is on.

Previously the "bottom: -20px" style placed the bottom of the rulers 20px into the bottom padding added by the scrollpastend addon, but the min-height on the CodeMirror-rulers div made them move around based on which lines were rendered.

Thanks. Merged as 2466392


Some systems failed to build for Quicklisp dist

Building with SBCL <IP_ADDRESS>-c2e7dcbd6 / ASDF 3.3.5 for quicklisp dist creation.
Trying to build commit id 6300379d715eefbda646e85283e7cbc741bfa072
cl-protobufs fails to build with the following error:
; caught COMMON-LISP:ERROR:
;   (during macroexpansion of (PI:DEFINE-MESSAGE FIELD-DESCRIPTOR-PROTO ...))
;   The value TYPE-DOUBLE is not of type KEYWORD when setting slot PI::NAME of structure CL-PROTOBUFS:ENUM-VALUE-DESCRIPTOR
...
; caught COMMON-LISP:ERROR:
;   (during macroexpansion of (PI:DEFINE-MESSAGE FILE-OPTIONS ...))
;   The value SPEED is not of type KEYWORD when setting slot PI::NAME of structure CL-PROTOBUFS:ENUM-VALUE-DESCRIPTOR
...
; caught COMMON-LISP:ERROR:
;   (during macroexpansion of (PI:DEFINE-MESSAGE FIELD-OPTIONS ...))
;   The value STRING is not of type KEYWORD when setting slot PI::NAME of structure CL-PROTOBUFS:ENUM-VALUE-DESCRIPTOR
...
; caught COMMON-LISP:ERROR:
;   (during macroexpansion of (PI:DEFINE-MESSAGE METHOD-OPTIONS ...))
;   The value IDEMPOTENCY-UNKNOWN is not of type KEYWORD when setting slot PI::NAME of structure CL-PROTOBUFS:ENUM-VALUE-DESCRIPTOR
...
Unhandled UIOP/LISP-BUILD:COMPILE-FILE-ERROR in thread #<SB-THREAD:THREAD "main thread" RUNNING {1001720003}>: COMPILE-FILE-ERROR while compiling #<PROTOBUF-SOURCE-FILE "cl-protobufs" "well-known-types" "descriptor">

cl-protobufs/tests fails to build because of a failure in  cl-protobufs.
Full log here

My guess is you didn't recompile protoc?
We did updates that break without recompiling protoc.

I have encoutered the same problem.
Solved by

recompiling cl-protobufs/protoc
and delete quicklisp cache ~/.cache/common-lisp/sbcl-2.2.10-macosx-x64/xxxxx/quicklisp/local-projects/cl-protobufs
(ql:quickload :cl-protobufs), now works fine

To old and cannot access logs, sorry
Reopen if needed.


How to find the title metadata of a particular file in fma_large?

Hello I am trying to browse through the archive and having a hard time to understand the metadata.zip.
For example if I want to get the title of file 020/020001.mp3 in the fma_large.zip, how can I locate it in the tracks.csv?

Please look at the usage.ipynb notebook which shows how the metadata is organized.

For example if I want to get the title of file 020/020001.mp3 in the fma_large.zip, how can I locate it in the tracks.csv?

It's the row that has a track_id of 020001.

Yes. Filenames are all 6 digits because some systems don't properly order files otherwise (I guess that's when the name is treated as a string).


How to understand the response parameter area in the device retrieval API

Problem description
Hi Teams,
About the location retrieval API, we are currently implementing this API and have encountered a problem. Regarding the region in the response parameters (including the type and size of the region), how is it obtained? Is it defined by the operator themselves? Are there any related constraints (such as not allowing the return area to be too large) and whether the actual location of device needs to be within the return area.

Hi @chinaunicomyangfan, the returned area is only restricted to be compliant with the schema. It is up to the implementation to decide between a circle or polygon, but size is not limited, and currently client cannot request a maximum size to be returned, as for age. It is assumed that device is within the returned area (according to the information maintained by the operator), as that is the purpose of the service.
It may be a question to discuss if we should allow customers to define a threshold for a maximum area, similiar to maxAge, so implementations which cannot comply with the requirement do not returned a bigger area. cc @bigludo7

If the size of the return area is not constrained, returning a large area does not violate the definition of the API, but for users, this result is obviously meaningless

This is a fair point !
As we have a maxAge having a maxSurface could make sense.
Let's discuss this point for our next release with the team durng Tuesday call.


Network state issue

how to network connection states when middle of the call(incoming and outgoing).
how i can get network error ?
Pls help me on this

Hi @ramohanreddy
The SDK monitors the network reachability during active calls and will attempt to re-establish the signaling and media connection once reachable network is available.
Thanks
-bobie


No background hover effect color is applied on ‘Copy to clipboard’ icon of the ‘Contact methods’ section

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Click on “Profile” avatar
Click on “Profile” link
Click on “Contact method” link
Hover over ‘Copy to clipboard’ icon

Expected Result:
Background hover effect color should be applied as it is applied in other sections of the app
Actual Result:
No background hover effect color is applied
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/43996225/232343524-586a37ee-ffca-4336-81be-401631ae8438.mp4

Expensify/Expensify Issue URL:
Issue reported by: @natnael-guchima
Slack conversation:
View all open jobs on GitHub

Triggered auto assignment to @johncschuster (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

@johncschuster Issue will be fixed as per this comment (#16453). So closing this issue.


Assertion Error in Tape-Embed

I am getting an error on tape-embed using a GPU. I am using batch size as 4 because of anything more causes memory issues in GPU. Please advise.
/pytorch/aten/src/THC/THCTensorIndex.cu:361: void indexSelectLargeIndex(TensorInfo<T, IndexType>, TensorInfo<T, IndexType>, TensorInfo<long, IndexType>, int, int, IndexType, IndexType, long) [with T = float, IndexType = unsigned int, DstDim = 2, SrcDim = 2, IdxDim = -2, IndexIsMajor = true]: block: [256,0,0], thread: [95,0,0] Assertion srcIndex < srcSelectDimSize failed.
Traceback (most recent call last):
File "/project/6000341/saby2k13/pyDPPI/kit/medver/bin/tape-embed", line 8, in
sys.exit(run_embed())
File "/project/6000341/saby2k13/pyDPPI/kit/medver/lib/python3.6/site-packages/tape/main.py", line 234, in run_embed
training.run_embed(**embed_args)
File "/project/6000341/saby2k13/pyDPPI/kit/medver/lib/python3.6/site-packages/tape/training.py", line 642, in run_embed
outputs = runner.forward(batch, no_loss=True)
File "/project/6000341/saby2k13/pyDPPI/kit/medver/lib/python3.6/site-packages/tape/training.py", line 86, in forward
outputs = self.model(**batch)
File "/project/6000341/saby2k13/pyDPPI/kit/medver/lib/python3.6/site-packages/torch/nn/modules/module.py", line 532, in call
result = self.forward(*input, **kwargs)
File "/project/6000341/saby2k13/pyDPPI/kit/medver/lib/python3.6/site-packages/tape/models/modeling_bert.py", line 449, in forward
chunks=None)
File "/project/6000341/saby2k13/pyDPPI/kit/medver/lib/python3.6/site-packages/torch/nn/modules/module.py", line 532, in call
result = self.forward(*input, **kwargs)
File "/project/6000341/saby2k13/pyDPPI/kit/medver/lib/python3.6/site-packages/tape/models/modeling_bert.py", line 346, in forward
layer_outputs = layer_module(hidden_states, attention_mask)
File "/project/6000341/saby2k13/pyDPPI/kit/medver/lib/python3.6/site-packages/torch/nn/modules/module.py", line 532, in call
result = self.forward(*input, **kwargs)
File "/project/6000341/saby2k13/pyDPPI/kit/medver/lib/python3.6/site-packages/tape/models/modeling_bert.py", line 285, in forward
attention_outputs = self.attention(hidden_states, attention_mask)
File "/project/6000341/saby2k13/pyDPPI/kit/medver/lib/python3.6/site-packages/torch/nn/modules/module.py", line 532, in call
result = self.forward(*input, **kwargs)
File "/project/6000341/saby2k13/pyDPPI/kit/medver/lib/python3.6/site-packages/tape/models/modeling_bert.py", line 242, in forward
self_outputs = self.self(input_tensor, attention_mask)
File "/project/6000341/saby2k13/pyDPPI/kit/medver/lib/python3.6/site-packages/torch/nn/modules/module.py", line 532, in call
result = self.forward(*input, **kwargs)
File "/project/6000341/saby2k13/pyDPPI/kit/medver/lib/python3.6/site-packages/tape/models/modeling_bert.py", line 180, in forward
attention_scores = torch.matmul(query_layer, key_layer.transpose(-1, -2))
RuntimeError: CUDA error: device-side assert triggered

I'm going to close this, since I don't think it's an actual bug in our code. If you continue to have issues, feel free to reopen.


Try replacing property abuse with field-clamping descriptors

This should apply to the instruction decoder and maybe a few fields on the VM. For the instruction decoder, there might be a subclass of clamped that uses a mask as a constructor argument when initializing the descriptor. A more complicated implementation might be able to use a bit-mask as the only argument and infer the min and max from there, if also given a negatives allowed or minimum argument.
Maybe something like the following?
class Instruction:
    x : int = MaskedField(mask=0x0F00, signed=False)

    def __init__(self, **kwargs):

The result should be shorter and cleaner code than the current property and clamping decorators used.

The MaskedField would take a bitmask argument, and signed argument probably isn't needed for chip-8. The constructor would shift the mask right until it's aligned at zero and store:

the length of the field / the maximum valid value of the field
how far it needs to be shifted right to be applied

When setting a field value, anything that would exceed the maximum raises a ValueError. When reading, some clever logic yet to be written could handle extracting the raw bytes as needed. Decode could happen on demand rather than setting a local property for now. Eventually, this could be replaced with some sort of update-time aware optimization but that might be excessive for chip-8.


Make options to define/generate app-config's baseUrls and backendSecret in CR

Context:
To make Backstage plugins work correct app-configs fields: app.baseUrl, backend.baseUrl, backend.cors.origin  (usually all of them have the same value) should be properly configured based on route/ingress on top of Backstage deployment.
The same thing for backend.auth.key.secret - it should be configured with (usually random) string.
For the time being the only way for Operator is to create a configMap with this configuration and Secret with backend auth secret value. It should be done for every installation.
To improve UX, avoiding User of creating extra objects and associating them with CR, we can make Operator do that.
Here is the initial proposed   changes in Backstage CR API

This makes sense, as pointed out in the past in #113. Additional things to keep in mind with the URLs:
https://github.com/janus-idp/operator/issues/113#issuecomment-1877059401

As part of the migration from janus-idp to redhat-developer in https://issues.redhat.com/browse/RHIDP-1021, this will be tracked in https://issues.redhat.com/browse/RHIDP-2269


Span timestamp is not correctly correct in case of clock skew.

The timestamp of the span is now set to the value of the timestamp
of the CS annotation.

I didn't have time to finish it. Can you please wait until I polish it to merge it ?

sure thing


Pipeline cleanup

Closes: https://github.com/opendatahub-io/odh-dashboard/issues/1207
Description

Misc changes for Pipelines.
I have manually confirmed the slight UI changes with Kyle.
How Has This Been Tested?

Manually going through the ticket to test each area.
Test Impact

Not applicable -- https://github.com/opendatahub-io/odh-dashboard/issues/1184
Request review criteria:

[x] The commits have meaningful messages (squashes happen on merge by the bot).
[ ] Included any necessary screenshots or gifs if it was a UI change.
[ ] Included tags to the UX team if it was a UI/UX change.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work
[x] The developer has added tests or explained why testing cannot be added (unit tests & storybook for related changes)

code looks good. going to test functionality quick

This link is incorrect
https://github.com/andrewballantyne/odh-dashboard/blob/pipeline-cleanup/frontend/src/concepts/pipelines/content/tables/pipeline/PipelinesTableExpandedRow.tsx#L57-L62
takes me here: http://localhost:4010/pipeline/test-gages-pr/pipelineRun/create which is a 404

Might not be an issue, but is this supposed to say no content or should there be a empty state screen

Just did a quick rundown, code seems to be in place, I saw all @Gkrumbach07 findings, I'll re-review once those issues are addressed.

@Gkrumbach07 / @lucferbux Fixed the issues you saw and a few QE found too.
Let me know if there is anything else you spot.

fixes look good, but if the data connection is not valid, the pipeline server spins indefinitely

fixes look good, but if the data connection is not valid, the pipeline server spins indefinitely

I've added it to the UX meeting -- are you waiting for a response to this before approving?

/lgtm


fix(styling): Added wordwrap to Notifications #624

fix for https://github.com/aml-development/ozp-center/issues/624

@wski @akonsowski

:+1:

:+1:


[stable/redis] add staticID feature to allow constant sentinel ids

What this PR does / why we need it:
We just deployed the cluster mode with sentinel for redis with 2 slaves and one master. After one pod failure the sentinels were able to elect a new master, after a second pod failure no master could be elected for the following reason:

Sentinels never forget about replicas of a given master, even when they are unreachable for a long time. This is useful, because Sentinels should be able to correctly reconfigure a returning replica after a network partition or a failure event.
Source

In Kubernetes, unless specified, each pod is given a random IP. In our case, with 2 slaves and one master, each replicas is attributed to a random IP. Holding on to old replicas IPs will only make it impossible for other sentinels to elect a new master. Although it is possible for a client to send the command SENTINEL RESET * to force sentinel servers to reset its list of sentinel peers. We can also tackle this issue by giving a constant sentinel ID which will allow other sentinel peers to update to the new given IP.
Currently, each sentinel is generating a new random ID on startup. The feature was developed to allow users to choose one way or the other.
Which issue this PR fixes
Special notes for your reviewer:
For those who are familiar with the Redis chart, would there be another way to tackle this issue?
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @miguelaeh

@juan131 @miguelaeh
did you guys have a chance to look at this?

Hi,
Thanks for the PR. I looked at it and LGTM. Could you bump the minor version?

@javsalgar thank you for checking this PR, I've applied the requested changes

/lgtm
/ok-to-test

@javsalgar @juan131 @miguelaeh
Tests have been rerun and passed, can one of you merge asap before we have another merge conflict haha, thank you :D

/lgtm

@rsecob great work man ! :+1: :100:


Netlify CI runs on release PRs

Description of issue:

Netlify serves up our multidev environments.
We only get so many multidev environments a month before we have to pay.
We create multidev environments for release PRs, which (so far) hasn't been necessary.
Cutting out the multidev generations on these PRs should help prevent us from hitting limits in the future.

For additional context, @EdwinGuzman got this set up for us and Netlify is set up under the NYPL digital email.

Requires follow-up with Edwin

We haven't reached the limit or have gotten close to it so this isn't an issue yet. I think it's set up to only create a dev environment for pull requests and if we cut out multidev generations, then we won't get anything out of this. Unless if we are planning or hoping to use Netlify for something else.

I think we got close once, but I am dubious we're ever going to run into the limit again now that we're not doing as much intense active development on this. I'll close this out.


problem with expansion panel in recyclerview

I was trying to use expansion panel in recyclerview, and done with getting result from api but if result having 7 or 8 items its showing perfectly but list having 50 and more item then its getting more time and showing black screen also doesn't showing list after taking more time to build.
layout design in below image, please check this

any answer on this?

not find solution yet. but i'm still working on it as soon as i get any solution i will update here.

In my case It's open very slow like leggy when even in 1 item but many in parent


Write operation receives content from stdin

Change-type: patch
Signed-off-by: Zahari Petkov<EMAIL_ADDRESS>

@balena-ci I self-certify!


As a creator, posts need to be able to have images added.

Images will be hosted on GridFS

Code implemented and tested.


[Policy Assistant] explain command: support setting (B)ANPs via CLI arg

Successor to #153. Child issue of #150.
Right now we have AdminNetworkPolicies hard-coded (referenced in analyze.go). A user should be able to specify NetworkPolicy, AdminNetworkPolicy, and the BaselineAdminNetworkPolicy by:

grabbing these from their Kubernetes cluster
specifying the filepath for all policy yamls
or both

Code in question:
https://github.com/kubernetes-sigs/network-policy-api/blob/669dfbc72245a2c7c6f3a3dd5c6188b7af9f8014/cmd/policy-assistant/pkg/cli/analyze.go#L96-L128

/assign

Hey, @huntergregory can you share more details about the task? Is it about reading (B)ANPs from a cluster or file path?
Since I'm new to the project any information would be helpful.

Is there a weekly meeting about this project where I can join and ask these kinds of questions?

Hey Nikola, thanks for looking to pick this up. Just fixed this issue's description, let me know if you have any questions.

Is there a weekly meeting about this project where I can join and ask these kinds of questions?

Yes! Would be great to have you. We meet every other Tuesday (some more details at this link)

I think the CLI already has the arguments we need (code is in analyze.go). I think they are like ‘--policy-path’, ‘--namespace’, ‘--all-namespaces’.
To create the CLI binary:

change into the cmd/policy-assistant/ directory
make cyclonus
binary at ‘cmd/cyclonus/cyclonus’ I believe

Thanks for the info Hunter.I already started working on this, but more questions arose.
Should cyclonus always work with the (B)ANPs or they should be enabled/disabled via an arguments?
What is the expected behavior if one of the request fail while trying to get info from the api-server?

Should cyclonus always work with the (B)ANPs or they should be enabled/disabled via an arguments?
Let's always include every policy type.

What is the expected behavior if one of the request fail while trying to get info from the api-server?
I would just log errors with logrus.Errorf() and continue. But if API Server returns this:

error: the server doesn't have a resource type "adminnetworkpolicy"

ideally, we would just log a warning message that ANP isn't defined instead of saying there's an error.

/remove-lifecycle stale

Fixed in #239

/remove-lifecycle stale


liquid-dsp: 1.4.0 -> 1.5.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/jgaeddert/liquid-dsp/releases.
meta.description for liquid-dsp is: "Digital signal processing library for software-defined radios"
meta.homepage for liquid-dsp is: "https://liquidsdr.org/"
meta.changelog for liquid-dsp is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.5.0 with grep in /nix/store/w1w35jq52kc6ihi39aybvsgbachgg4h1-liquid-dsp-1.5.0
found 1.5.0 in filename of file in /nix/store/w1w35jq52kc6ihi39aybvsgbachgg4h1-liquid-dsp-1.5.0

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
cubicsdr
inspectrum
liquid-dsp

Instructions to test this update (click to expand)

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

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/w1w35jq52kc6ihi39aybvsgbachgg4h1-liquid-dsp-1.5.0
ls -la /nix/store/w1w35jq52kc6ihi39aybvsgbachgg4h1-liquid-dsp-1.5.0/bin

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

3 packages built:

cubicsdr
inspectrum
liquid-dsp

Maintainer pings

nixpkgs-review happy, aarch64-linux


SAK-46439 Gradebook: When using keyboard navigation, item menus do not close until another item menu opens

I've changed the way the drop menu worked when invoked by keyboard. I've added a "switch case" statement instead of a bunch of "if" statements, and wrote new code for the keys that were missing (left and right arrows & tab).

Okay, this makes sense to me.


Civic camp event blocks on website

Can you make sure all the event civic camp blocks on the site have the correct times?
This is the master schedule: http://ncopenpass.com/civic-camp/civic-camp-schedule/
For example, this page has the wrong times listed:
http://ncopenpass.com/civic-camp/

@jtawes I would recommend we take the image of the Civic Camp Flyer off the main page.

Oh - I forgot that you had the original files - that works ;)
On Thu, Sep 13, 2018 at 1:24 PM jtawes<EMAIL_ADDRESS>wrote:

Do you want to place it somewhere else? I just made the edits and
re-uploaded it.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ncopenpass/NCOpenPass/issues/1208#issuecomment-421086412,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK4rpuKGloe-8hO9mT4_NiZoAx0AGxPdks5uapTXgaJpZM4WnnEX
.

I deleted the incorrect flyers and uploaded the edited version to the shared drive.


rework Playground SDK (PlayFirstPlayground) rule

SteamDB app page links to a few games using this
#106
Brief explanation of the change
closes #106
also renamed it to Playground_SDK as that's what it's originally called

Is this meant to match on somefolderassets/file.pfp and path/to/assets/folder/file.pfp

Not doable with the regex we limit ourself to (no use of * and + )