authClient error after updating to 0.30 or higher

After updating gcloud I get this error that crashes the process when trying to make any requests to datastore on v0.34.0
/Users/richardkazuomiller/Sites/feedproxy/node_modules/gcloud/lib/common/grpc-service.js:430
  this.authClient.getAuthClient(function(err, authClient) {
                  ^

TypeError: this.authClient.getAuthClient is not a function
    at Datastore.GrpcService.getGrpcCredentials_ (/Users/richardkazuomiller/Sites/feedproxy/node_modules/gcloud/lib/common/grpc-service.js:430:19)
    at Datastore.GrpcService.request (/Users/richardkazuomiller/Sites/feedproxy/node_modules/gcloud/lib/common/grpc-service.js:218:10)
    at DatastoreRequest.request_ (/Users/richardkazuomiller/Sites/feedproxy/node_modules/gcloud/lib/datastore/request.js:885:8)
    at DatastoreRequest.get (/Users/richardkazuomiller/Sites/feedproxy/node_modules/gcloud/lib/datastore/request.js:348:8)
    at DatastoreRequest.get (/Users/richardkazuomiller/Sites/feedproxy/node_modules/gcloud/lib/datastore/request.js:292:10)
    at Object.<anonymous> (/Users/richardkazuomiller/Sites/feedproxy/lib/dataset.js:9:11)
    at Module._compile (module.js:409:26)
    at Object.Module._extensions..js (module.js:416:10)
    at Module.load (module.js:343:32)
    at Function.Module._load (module.js:300:12)

This is my code
const options = {
  projectId: <my-project-id>,
  keyFilename: <my-key-json>
}
const gcloud = require('gcloud')(options)
const datastore = gcloud.datastore()

const key = datastore.key(['Foo','bar'])

datastore.get(key,function(err,entity){
  console.log('foo',err,entity)
})

module.exports = datastore

The same code, other than changing datastore.dataset() to datastore() works on v0.29. Am I doing something wrong or is this a bug?

Sounds like it could be an issue when switching between versions without fully un- and re-installing all npm dependencies. After installing<EMAIL_ADDRESS>try a rm -rf node_modules && npm install.

That did it, which probably means that there's a dependency whose version needs to be updated in the dependencies configuration.

Thanks for reporting!

@stephenplusplus You're quick! I was just writing about how I found that running with<EMAIL_ADDRESS>was what caused it lol. Thanks for the fix.

Haha, only thanks to you for sending me in the right direction 😄


all examples links are missing in README.md

Is your feature request related to a problem? Please describe.
the examples are not present. all links are gone.
Describe the solution you'd like
i think example  should added  in the repository itself. at least to the readme file itself. like in vue-social-auth
Describe alternatives you've considered
same as above
Additional context
Add any other context or screenshots about the feature request here.

Share your code
Sent from my iPhone

On Sep 2, 2021, at 7:22 AM, Rajesh Chaurasiya @.***> wrote:

Is your feature request related to a problem? Please describe.
the examples are not present. all links are gone.
Describe the solution you'd like
i think example should added in the repository itself. at least to the readme file itself. like in vue-social-auth
Describe alternatives you've considered
same as above
Additional context
Add any other context or screenshots about the feature request here.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

i am still working on code part. the example you have provided in the npm page have issues. i am not able to get what is the workflow and some variables are missing there. Please fix that one. also there are differences in response i am getting after i logged using google
Object { code: "<token-here>", scope: "email+profile+openid+https://www.googleapis.com/auth/userinfo.profile+https://www.googleapis.com/auth/userinfo.email", authuser: "0", prompt: "consent" }

i am still working on code part. the example you have provided in the npm page have issues. i am not able to get what is the workflow and some variables are missing there. Please fix that one. also there are differences in response i am getting after i logged using google
Object { code: "<token-here>", scope: "email+profile+openid+https://www.googleapis.com/auth/userinfo.profile+https://www.googleapis.com/auth/userinfo.email", authuser: "0", prompt: "consent" }

sorry for late reply don't get this on time
you can override the scope by passing the values needed


Add support for building operators without a link space

This PR allows one to pass the kwarg link_space = nothing into the ITensorNetwork(f::Function, s::IndsNetwork; ...) constructor so that it doesn't add any link indices when constructing the ITensorNetwork. The change is made at the level of insert_linkinds(s::IndsNetwork; link_space = trivial_space(s). The default of link indices of dimension 1 is unchanged.
One can now do ITensorNetwork(Op("I), s; link_space = nothing) to build the identity operator without any link indices. The default for the BiLinearForm(psi, phi; ...) constructor is done based on whether psi does or doesn't have site indices.
@mtfishman I'm adding this so that when one builds the formnetwork <psi|I|psi> and does BP the  message tensors don't end up with additional trivial indices which can lead to complications when you want to square root them and gauge the local state etc.
Let me know if you can think of a better way to support this functionality or if you are okay with as is.

This sounds reasonable. Handling the corner case of having no link indices vs. trivial link indices is definitely tricky and part of this package that I found most difficult to design, we may have to step back and assess it more systematically at some point.


direct mode: empty dev-name by BlockDeviceName() causes Flush and Delete failure

Seen (once so far) in 4xVM based "go test test/e2e" in direct mode:
Unexpected error:
  Code: 13,
  Message: "Failed to delete volume name:new-addition
  id:6a2130accf186ac16f0d4f336bd488c9a512513f on pmem-csi-ubuntu-govm-worker2:
  rpc error: code = Internal desc = Failed to delete volume: /dev/ is not device"

This is caused by ns.BlockDeviceName() returning empty string in call chain:
DeleteDevice
  getDevice
    namespaceToPmemInfo
      ns.BlockDeviceName

ClearDevice is called by arg "/dev/" and FlushDevice fails.

This was not seen/reproducible after VMs tear-down and recreate cycle, likely caused by VM being in bad state. There were some block device related errors in VM syslog, but timing did not match with error points in tester log, so the cause determination was not straightforward.
Let's close this as non-reproducible. It remains documented on closed-issues side for possible future re-opening.


RegisterServiceServer method missing

My service.proto file contains -
syntax = "proto3";
package service;
message ServiceRequest {
    bytes Content = 1;
    string Tag = 2;
}

message ServiceResponse {
    repeated string Contents = 1;
}

service Service {
    rpc Service(ServiceRequest) returns (ServiceResponse) {}
}

I tried -  protoc --proto_path=service --go_out=service service.proto  and found out the service.pb.go file doesn't contain RegisterServiceServer function.
How shall I register my Grpc server then?

My bad, we need to add Grpc plugin in go_out. So now it looks like protoc --proto_path=service --go_out=plugins=service service.proto


fe / create authentication pages with minimal or no styling.

No styling is necessary, till we have the UI/UX Prototype

[ ] login
[ ] register
[ ] logout

#17 fixed


Add separate about page, editable in admin interface

backend

add about_md to Settings

frontend

add About page
render about_md markdown using showdown to convert to html and github-markdown-css for styling
add text box to edit about_md in admin settings
refactor shared Jumbotron code into HeadingComponent
refactor settings api call into a settings store

resolves #55
resolves #34

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

Project coverage is 78.13%. Comparing base (4ca7c34) to head (f5572d0).
Report is 4 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #65      +/-   ##
==========================================
+ Coverage   78.10%   78.13%   +0.03%
==========================================
  Files           8        8
  Lines         717      718       +1
==========================================
+ Hits          560      561       +1
  Misses        157      157

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


Some curves rendered as straight lines

Describe the bug
Some of the teacher's curves in med_test's AskWhatTheParadoxIs are instead rendered as straight lines.
Code:
PYTHONPATH="$(pwd):$PYTHONPATH" manimgl -w _2020/med_test.py AskWhatTheParadoxIs (on a clone of https://github.com/3b1b/videos)
Wrong display or Error traceback:

Additional context
From a bisection, it seems the bug was introduced in commit d4c89e52, and from further testing it looks like the bug stems from VMobject.flip no longer calling refresh_unit_normal every time (since the triggers_refreshed_triangulation decorator only calls it when the points have changed).

The same commit also introduced similar artifacts in the teacher's mouth that doesn't go away by merely fixing flip to call self.refresh_unit_normal() every time:

This artifact was apparently introduced when adding the call to self.refresh_unit_normal() (again, as part of the triggers_refreshed_triangulation decorator) to the set_data method.


IndexOutOfBoundsException: Index 7, Size 4

hiya! tryna run a modpack and it aint workin. the error occured when attemping to launch the game. heres the
log

This is an incompatibility with Hostile Villages


node version vs. npm install

It seems like step #1 in the README should come later. Step #2 has you make sure your node version matches Kibana’s. Logically you want to run nvm before npm install -g yo generator-kibana-plugin.
I suggest swapping step 1 and 2.
// cc @hickeyma
PS: Made my first Kibana plugin!

😄  good suggestion

@spalger change in https://github.com/elastic/generator-kibana-plugin/commit/55a5d7da692ab93e20082cfac29c6dea91a12cce looks good! :+1:


Drop object names from returned column name when querying objects' fields

Use case:
Querying an object's fields returns its object name as part of the returned column name.
Sometimes it is confusing, like the case below that you need to re-specify the objects name:
create table tbl (col object(strict) as (nested_col text));
select col['nested_col'] from (select col['nested_col'] from tbl) as A;
+-------------------+
| col['nested_col'] |
+-------------------+
+-------------------+

Also, it is necessary, in order to be able to do something like :
cr> create table cpy as select col['nested_col'] from tbl;
CREATE OK, 0 rows affected  (2.616 sec)
cr> select * from cpy;
+------------+
| nested_col |
+------------+
+------------+

This is the current Posgres behaviour :
postgres=# select * from (select (col).r,(col).i from tbl) as A;
 r | i
---+---
(0 rows)

Feature description:
the idea is to be able to:
create table tbl (col object(strict) as (nested_col text));
select nested_col from (select col['nested_col'] from tbl) as A;
+------------+
| nested_col |
+------------+
+------------+

Currently you've to use an alias to change the column name manually. In your example you'd probably want to use something like this:
create table cpy as select col['nested_col'] as nested_col from tbl;

Changing this would be a major breaking change, and I'm not if it is warranted.

Closing this as wontfix for now. There hasn't been any other interesting and it would be a breaking change.


Multiple quake mode windows, each with its own profile

Description of the new feature/enhancement
I use WSL and PowerShell commands, and thus terminal windows, with approximately equal frequency. Depending on how I set up my default profile, I can get quick access to one of these via the quake window, but not the other.
Ideally, I should like to be able to set up multiple quake mode windows, each with its own keybinding, such that for example ctrl+ would summon a quake mode window containing my PowerShell command line, while alt+ would summon one likewise with my zsh/Debian/WSL command line.

So! This is where it gets a little crazy. quakeMode is actually a specialized version of an action we've called globalSummon! It's a lot more flexible than quake mode -- you can summon any named Terminal window with a few different display options.
Right now, the only window that gets the dropdown + anchored to the top behavior is the one that's named _quake, so there is a bit of a blind spot. We're tracking additional work on globalSummon to make it more robust for cases like these, though.
Does that help?

Not the solution but you can still split quake window and have two panes

Seems like quake should be a style you can set for the globalSummon and that would give the flexibility to have as many linked to whatever key combo you wanted. Does globalSummon support custom styling? I normally don't want to turn on any opacity but I would use it for the quake-style dropdown.

IIRC it's not simply styling -- there's exclusion from the task list, etc that falls into the quake mode behavior.
But I, too, would like to have something decoupling the mode from the window name.  I'm envisioning not just a command shell, but also a quick, docked scratchpad for notes (sticky notes just don't work as well as I'd like), maybe a CAS calculator (eg, Julia or xcas CLIs) -- basically quick, keyboard-driven utilities.  For org-mode afficionados, it could be a great fit as well.
I do notice that I can still pop up different profiles and switch between them (as toggling focus mode will show), so perhaps it's just a matter of "summon the _quake window and cause it to switch to a named tab" -- but that begs named tabs.  I might write a feature request around that...

I know it's not exactly as dynamic as you need but here's a pretty useful alternative I've created for a similar purpose.
This is the keybinding I use.
{
    "command": "unbound",
    "keys": "win+\`"
},
{
    "command":
    {
        "action": "globalSummon",
        "desktop": "toCurrent",
        "dropdownDuration": 200,
        "monitor": "toMouse",
        "name": "_quake",
        "toggleVisibility": true
    },
    "keys": "win+\`"
},

Aiming to your needs (PWSH & WSL), I suggest you to select PWSH as your default profile and add the next line inside the settings.json file:
"startupActions": "wt -w _quake sp -V -p $PROFILE$ -s .5"
Make sure to replace $PROFILE$ with the name of the profile you want to launch on the right panel, whichever it is.
You can do this with the default WSL profile, as well as any other profile available.
And, of course, you can write any subcommand or parameter you'd like.
Have a look at https://github.com/microsoft/terminal/issues/9992#issuecomment-851093521 for further behavior customization.

I know it's not exactly as dynamic as you need but here's a pretty useful alternative I've created for a similar purpose.

I'm not seeing how your binding addresses the (pretty central) desire to have different profiles summonable?

I'm not seeing how your binding addresses the (pretty central) desire to have different profiles summonable?

That's the reason I said it's not as dynamic as required, but it's an alternative to what @cerebrate asked for. It states:

I use WSL and PowerShell commands, and thus terminal windows, with approximately equal frequency. Depending on how I set up my default profile, I can get quick access to one of these via the quake window, but not the other.

The alternative allows to launch any panels or tabs as wanted, as well as any profile, Quake Mode included.
Combined with the auto startup, it enables the auto load of a customized _quake window as fluent and transparent as it can be at the current stage of development.
Despite having hidden header, tabs actually work right now within the _quake window, so do the tab switcher and all keybindings. The alternative uses panels 'cause I like them more but it's only a personal preference.
Then it states:

Ideally, I should like to be able to set up multiple quake mode windows, each with its own keybinding, such that for example ctrl+ would summon a quake mode window containing my PowerShell command line, while alt+ would summon one likewise with my zsh/Debian/WSL command line.

I see what @cerebrate proposed and I agree it'd be a useful feature but the alternative enables a similar behavior when using tabs (nt) instead of panels (sp) in the startupActions subcommands, only two keybinding ahead instead of one; one to toggle the visibility and one to toggle the tab. Sometimes maybe only the visibility one if the needed tab is already selected.
I'm not trying to close the proposal but providing a useful temporary alternative while we wait for an official feature announcement.

So! This is where it gets a little crazy. quakeMode is actually a specialized version of an action we've called globalSummon! It's a lot more flexible than quake mode -- you can summon any named Terminal window with a few different display options.
Right now, the only window that gets the dropdown + anchored to the top behavior is the one that's named _quake, so there is a bit of a blind spot. We're tracking additional work on globalSummon to make it more robust for cases like these, though.
Does that help?

Partially? Depending, I suppose, on the specific additions to globalSummon.
I've actually being playing around a fair bit with globalSummon (hence, partially, this delayed response while I try to figure out what is possible). While I can make some useful shortcuts for summoning existing windows with it, the chief issue I'm having there is the creation of named windows that don't exist, because unless I'm missing something, there doesn't seem to be a way to specify a profile to use when creating a window. I can work around this by precreating windows using startupActions , but it's still not the most elegant approach.
But the specific dropdown/anchoring features of the _quake window are useful enough in and of themselves that being able to add these to other named windows, like these custom globalSummons, would still be very desirable for me.

@dag03tsc I like that! As an interim solution, I'll definitely be using it for now.

As a further note, while I have not yet had time to look at them in detail, the various suggestions to have "hidden" tabs within the _quake window and be able to bring a particular one to the forefront when switching to it would also seem to satisfy my use case.
Actually, I'd also very much like to have that ability just for globalSummon , because being able to specify a particular tab to being forward when summoning a window (even by simple criteria not requiring additional data to be kept, like "the MRU tab for this profile", would add a great deal of value.

As a further further note, I'm using a Power Automate Desktop routine to handle setting this up these days:

Okay, after reading through this, this does seem like another case where #9992 will be the magic bullet. With that, you could give different window names different startupActions, or even different defaultProfiles, and then have different globalSummon actions to summon each of those possible names.
Thanks for the discussion everyone!
/dup #9992


Design the notification UI

See scope #9

I believe a bot gets a 'bot' tag added by default (will double check the API docs)
Truncate the project_title by 6 words and add an ellipsis (do not display full headline/title, it's always too long and here it is just a reminder for the team of what story this is)
Link the project_title to the public url where it is published.
Use default behavior and color for links.
Make the value and unit for measure for the metric shown bold.

Also:

Because these projects are graphics stories, they get categorized as such and only posted to visuals-graphics channel. (implication: need to have a method to route to the right channel based on how the project is classified in the taxonomy - for phase one it won't be necessary)

I'm wondering if for the first message when Carebot starts tracking, can be both more simple/straightforward AND more useful — some exploration:

Non-map example (more common use):

if including image, it probably needs an outline:

[ ] Checked the Slack API docs and confirmed that bot users get a [BOT] tag by default.

[ ] request feedback
[ ] iterate visuals as needed

Looks good all around!
I like including an image with the article to add context / remind people what it's tracking. If articles aren't already automatically posted, it might have the nice side effect of letting people other than the author know what's new.
On the other hand, if the message is only intended to confirm to the author that the piece is tracked, then no image might be more compact & subtle in the stream. We could then include the image with a later update, when there's more interesting information to share.
I think Slack may automatically add the frame around images. We'll find out!

Good points, thank you!

Capturing all UI documentation at https://github.com/thecarebot/carebot/wiki/Carebot-UI going forward.


Update SetSiteVersionPolicy.cs

Changed "Progress" to "Status" as the cmdlet has been renamed as well.
Before creating a pull request, make sure that you have read the contribution file located at
https://github.com/pnp/powerShell/blob/dev/CONTRIBUTING.md
Type

[x] Bug Fix
[ ] New Feature
[ ] Sample

Related Issues?
misleading information in the warnings of the cmdlets.
What is in this Pull Request ?
Small Information correction
Guidance

You can delete this section when you are submitting the pull request.*
Please update this PR information accordingly. We use this as part of our release notes in monthly communications.
Please target your PR to Dev branch. If you do not target the Dev branch we will not accept this PR.

Thanks @Studermarc for this !!
Much appreciated 😊🙏


Feature: New data on highlights page.

Type of feature
🍕 Feature
Current behavior
Instead listing open issues, which is not useful, list highlights counts.
https://api.opensauced.pizza/#/User Highlights service/addHighlightForUser

Suggested solution

Replace issues with total highlights
Change total PRs to PRs (currently) opened

Change this line
https://github.com/open-sauced/browser-extensions/blob/d49015d9e9abc8500f9ca7e7b21720b5c764032a/src/pages/profile.tsx#L153
Additional context
No response
Code of Conduct

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

Contributing Docs

[ ] I agree to follow this project's Contribution Docs

.take

The auto-take issues workflow is not enabled here, we can only assign issues to @open-sauced/triage, @open-sauced/committers, @open-sauced/maintainers in this repo for now. We suggest folks not on the team open issues and take a look at the Bugs or Papercuts. If you would like to find good first issues, please check out the companion project to this, open-sauced/hot.

Why does the bot say this? @bdougie auto-take should be enabled here?

@diivi But can I still work on this issue?
Also I'm not sure what needs to be done in the second part of the required solution: Change total PRs to PRs (currently) opened
Will I just change the text to "PRs (currently) opened" ?

The text can be "Open PRs" for it to be concise.

Do I need to fix that in commit and update the PR? Or can you just change the text while merging?


Add smart playlists to Navidrome

Use this to create some smart playlists, like top-80/90/00/10...
https://github.com/navidrome/navidrome/issues/1417
Perhaps use ansible to figure out what artist playlists to create, something like if count(artist) > 10 => create smart playlist with artist

First smart playlist added, enough for now and thus closing this task.


Content Suggestion: HODLWEEN

Link to Content: https://hodlween.party/
Briefly Describe Content: HODLween is bitcoin only celebration featuring locally sourced beef dinner and pumpkin carving contest, Bitcoin speakers and workshops, and an epic Halloween costume party! You don't need permission to be your own bank and you don't need permission to have fun! Charlotte, NC - Oct 21st-22nd, 2022
Suggested Section: Conferences

Sorry, I missed this

All good! Can we put in the "looking back" section?


Update Docs for Webpack 4 Type

The default config in the readme doesn't actually work out of the box with Webpack 4 (haven't tried with others)
Specifying the output type on the loader config fixes it right up.
Here's the error for googling
ERROR in ./src/net/Internals/transportSchema/meta.schema.json
Module parse failed: Unexpected token ' in JSON at position 0 while parsing near ''use strict';
var va...'
File was processed with these loaders:
 * ./node_modules/ajv-json-loader/index.js
You may need an additional loader to handle the result of these loaders.
SyntaxError: Unexpected token ' in JSON at position 0 while parsing near ''use strict';

Cheers! 🍻

Hello!
Thanks for the PR
Will this option work with Webpack 3 or lower? Either way, I think it would be nice to add a comment that this option is required only when using Webpack >=4. Link to the docs page also can be useful (but not necessary)

Good question, sure enough, it does not! Updated with a comment and link.

Nice, thanks!


FastQC crash, error 127

Pipeline crashes with conda environment
Bug description
When substituting containerization by the provided conda environment the pipeline crashes at the FastQC step. The following error gets thrown:
[Task monitor] ERROR nextflow.processor.TaskProcessor - Error executing process > 'fa
stqc (<myactualsampleid>)'

Caused by:
  Process `fastqc (<myactualsampleid>)` terminated with an error exit status (127)

Command executed:

  fastqc -q <myactualsampleid>.fastq

Command exit status:
  127

Command output:
  (empty)

Command error:
  java: symbol lookup error:/<installation directory>/miniconda/envs/nf-core-rnaseq-1.1/jre/lib/amd64/../../../lib/libfontconfig.so.1: undefined symbol: FT_Done_MM_Var

I looked into the directory and found the following libraries:

libfontconfig.so -> libfontconfig.so.1.12.0*
libfontconfig.so.1 -> libfontconfig.so.1.12.0*
libfontconfig.so.1.12.0*

Reproduce

Disable containers by commenting out the container specification in nextflow.config, conf/base.config as well as other profiles you use if specified there.
install conda 4.5.11
create environment using the provided environment.yml file conda env create -f environment.yml
run the pipeline

nextflow run main.nf --aligner hisat2 -profile cfc --saveReference --singleEnd --reads <data>/*.fastq --outdir results --skip-dupradar --skip_genebody_coverage --gtf <data>/Homo_sapiens.GRCh38.92.gtf --fasta <data>/GRCh38_r92.all.fasta

Expected behaviour
The pipeline runs through with no crash similar to execution in container.
Operation System
The pipeline was executed on the cfc cluster in Tübingen Germany
Additional notes
Same error as in  nf-core/eager#67

Possible fix:
I got it working by editing the environment.yml
dependencies:
  - conda-forge::openjdk=8.0.144

was replaced by
dependencies:
  - defaults::openjdk=8.0.152

I can confirm that this indeed fixed https://github.com/nf-core/eager/issues/67 properly!

Reproducing the error works unfortunately for me:
I'll open a PR to fix this ...
alex@aragorn ~/I/n/r/RNAseq> nextflow run . -profile test,conda
N E X T F L O W  ~  version 18.10.1
Launching `./main.nf` [maniac_hodgkin] - revision: 7c9a828c2b
=======================================================
                                          ,--./,-.
          ___     __   __   __   ___     /,-._.--~'
    |\ | |__  __ /  ` /  \ |__) |__         }  {
    | \| |       \__, \__/ |  \ |___     \`-._,-`-,
                                          `._,._,'

 nf-core/rnaseq : RNA-Seq Best Practice v1.1
=======================================================
Run Name       : maniac_hodgkin
Reads          : data/*{1,2}.fastq.gz
Data Type      : Single-End
Genome         : false
Strandedness   : None
Trim R1        : 0
Trim R2        : 0
Trim 3' R1     : 0
Trim 3' R2     : 0
Aligner        : STAR
Fasta Ref      : https://github.com/nf-core/test-datasets/raw/rnaseq/reference/genome.fa
GTF Annotation : https://github.com/nf-core/test-datasets/raw/rnaseq/reference/genes.gtf
Save Reference : No
Save Trimmed   : No
Save Intermeds : No
Max Memory     : 6 GB
Max CPUs       : 2
Max Time       : 2d
Output dir     : ./results
Working dir    : /home/alex/IDEA/nf-core/rnaseq_test/RNAseq/work
Container      : nfcore/rnaseq:1.1
Current home   : /home/alex
Current user   : alex
Current path   : /home/alex/IDEA/nf-core/rnaseq_test/RNAseq
Script dir     : /home/alex/IDEA/nf-core/rnaseq_test/RNAseq
Config Profile : test,conda
=========================================
[warm up] executor > local
Creating Conda env: /home/alex/IDEA/nf-core/rnaseq_test/RNAseq/environment.yml [cache /home/alex/IDEA/nf-core/rnaseq_test/RNAseq/work/conda/nf-core-rnaseq-1.1-31cb0d16eab401414840cd3ced2c4f17]
[ec/bc9d73] Submitted process > output_documentation
[a2/e151b3] Submitted process > get_software_versions
[0e/cf4c11] Submitted process > workflow_summary_mqc
[f9/dc25b7] Submitted process > makeBED12 (genes.gtf)
[d3/8023cf] Submitted process > trim_galore (SRR4238351)
[bc/5b86c8] Submitted process > fastqc (SRR4238351)
[76/b8ed50] Submitted process > fastqc (SRR4238379)
ERROR ~ Error executing process > 'fastqc (SRR4238351)'

Caused by:
  Process `fastqc (SRR4238351)` terminated with an error exit status (127)

Command executed:

  fastqc -q SRR4238351_subsamp.fastq.gz

Command exit status:
  127

Command output:
  (empty)

Command error:
  java: symbol lookup error: /home/alex/IDEA/nf-core/rnaseq_test/RNAseq/work/conda/nf-core-rnaseq-1.1-31cb0d16eab401414840cd3ced2c4f17/jre/lib/amd64/../../../lib/libfontconfig.so.1: undefined symbol: FT_Done_MM_Var

Work dir:
  /home/alex/IDEA/nf-core/rnaseq_test/RNAseq/work/bc/5b86c8e184cceae6c80ca3ce4e0b60

Tip: you can replicate the issue by changing to the process work dir and entering the command `bash .command.run`

 -- Check '.nextflow.log' file for details
[nfcore/rnaseq] Pipeline Complete
WARN: Killing pending tasks (1)
[bc/9df234] Submitted process > trim_galore (SRR4238379)

Just merged the PR, should fix the issue. Would be great if you could confirm by trying out the dev code and image @KochTobi

All fixed :-)


macaroons: macaroon.Clone() doesn't copy caveats

When calling macaroons.NewMacaroonCredential, this function uses a Clone method defined in the package gopkg.in/macaroon.v2. Its intention is to make a copy so that the new macaroon object won't be affectted if any mutation happened to the old one.
However, because slicing does not copy the slice's data, the Clone method won't copy the caveats.
Currently it should be fine. Whenever AddConstraints is called, it uses AddFirstPartyCaveat to add the caveat, which always creates a new slice of caveat in the macaroon(through append).
This won't be true in the following edge case,
// Suppose m is a macaroon.Macaroon with caveats attached.

// make a copy of m
m1 = m.Clone()

// make change to the first caveat
m.Caveats()[0].Location = "mars"

// the copied one will be changed too!
m1.Caveats()[0].Location == "mars"  // true

Although the caveats should probably not be modified this way, it might create a subtle bug in the future.

Suggested fix
Modify NewMacaroonCredential, so that it copies caveats too.
For demonstration, locally, I've created the following test to address this bug, under macaroons/constrains_test.go.
func TestCloneMacaroons(t *testing.T) {
	// Get a configured version of the constraint function.
	constraintFunc := macaroons.TimeoutConstraint(3)

	// Now we need a dummy macaroon that we can apply the constraint
	// function to.
	testMacaroon := createDummyMacaroon(t)
	err := constraintFunc(testMacaroon)
	if err != nil {
		t.Fatalf("Error applying timeout constraint: %v", err)
	}

	// Check that the caveat has an empty location
	require.Equal(t,
		"", testMacaroon.Caveats()[0].Location,
		"expected caveat location to be empty, found: %s",
		testMacaroon.Caveats()[0].Location,
	)

	// Make a copy of the macaroon
	newMacCred := macaroons.NewMacaroonCredential(testMacaroon)
	require.Equal(t,
		"", newMacCred.Macaroon.Caveats()[0].Location,
		"expected new caveat location to be empty, found: %s",
		newMacCred.Macaroon.Caveats()[0].Location,
	)

	// Modify the caveat location on the old macaroon
	testMacaroon.Caveats()[0].Location = "mars"

	// The old macaroon's caveat location should be changed.
	require.Equal(t,
		"mars", testMacaroon.Caveats()[0].Location,
		"expected caveat location to be empty, found: %s",
		testMacaroon.Caveats()[0].Location,
	)

	// The new macaroon's caveat location should stay untouched.
	require.Equal(t,
		"", newMacCred.Macaroon.Caveats()[0].Location,
		"expected new caveat location to be empty, found: %s",
		newMacCred.Macaroon.Caveats()[0].Location,
	)
}


Long Treeview list doesn't render probably

Hello,
I've created a very long treeview list out of mysql database entries.
When it's very large the parts out of the screen will not get rendered probably. But the HTML source code is correct.
On the screenshot, you can easily see where the rendering is not working probably. It's the same problem in FF 60.x and IE11. It will shown probably, if you don't have the "creation delays" due to the database querries.
Is there any workaround for the moment?

create pen on codepen.io with your problem code


gpx attributes lost after map matching

After running the map matching algorithm on an input gpx file, the output gpx file is missing all other attributes such as <desc></desc> or <link></link>

Do you have example(s) to reproduce this?

Yes e.g. my input files is looking something like this:

      <trkpt lat="51.03170000" lon="13.71301000">
        <ele>137.00000</ele>
        <desc>Some description here</desc>
        <link>Some link here</link>
        <time>2010-01-01T00:01:01Z</time>
      </trkpt>

If I have something reproducable in a unit test or the full file where you really see the problem, chances are higher that I can faster fix it :)

Just going to post my input gpx and output gpx, i'm going to add some details on how to reproduce later. (Actually i did just follow the instructions in the readme)
To my understanding the input file should be valid a valid gpx file, where each trackpoint has a description and a link.
Unfortunately the output file is missing the description and link.
Input gpx (=before.gpx)
<?xml version="1.0" encoding="UTF-8"?>
<gpx xmlns="http://www.topografix.com/GPX/1/1" xmlns:gpsies="http://www.gpsies.com/GPX/1/0" creator="GPSies http://www.gpsies.com - asd" version="1.1" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://www.topografix.com/GPX/1/1 http://www.topografix.com/GPX/1/1/gpx.xsd http://www.gpsies.com/GPX/1/0 http://www.gpsies.com/gpsies.xsd">
  <metadata>
    <name>asd</name>
    <link href="http://www.gpsies.com/">
      <text>asd on GPSies.com</text>
    </link>
    <time>2015-04-21T17:28:55Z</time>
  </metadata>
  <trk>
    <name>asd on GPSies.com</name>
    <trkseg>
      <trkpt lat="51.03232000" lon="13.71077000">
        <ele>136.60000</ele>
        <desc>Some description here</desc>
        <time>2010-01-01T00:00:00Z</time>
      </trkpt>
      <trkpt lat="51.03170000" lon="13.71301000">
        <ele>137.00000</ele>
        <desc>Some description here</desc>
        <link>Some link here</link>
        <time>2010-01-01T00:01:01Z</time>
      </trkpt>
      <trkpt lat="51.03076000" lon="13.71417000">
        <ele>137.80000</ele>
        <desc>Some description here</desc>
        <link>Some link here</link>
        <time>2010-01-01T00:01:49Z</time>
      </trkpt>
      <trkpt lat="51.03021000" lon="13.71635000">
        <ele>141.80000</ele>
        <desc>Some description here</desc>
        <link>Some link here</link>
        <time>2010-01-01T00:02:48Z</time>
      </trkpt>
      <trkpt lat="51.02911000" lon="13.71797000">
        <ele>143.60000</ele>
        <desc>Some description here</desc>
        <link>Some link here</link>
        <time>2010-01-01T00:03:48Z</time>
      </trkpt>
      <trkpt lat="51.02876000" lon="13.71879000">
        <ele>145.00000</ele>
        <desc>Some description here</desc>
        <link>Some link here</link>
        <time>2010-01-01T00:04:13Z</time>
      </trkpt>
      <trkpt lat="51.02957000" lon="13.72074000">
        <ele>145.20000</ele>
        <desc>Some description here</desc>
        <link>Some link here</link>
        <time>2010-01-01T00:05:12Z</time>
      </trkpt>
      <trkpt lat="51.02988000" lon="13.72207000">
        <ele>145.80000</ele>
        <desc>Some description here</desc>
        <link>Some link here</link>
        <time>2010-01-01T00:05:47Z</time>
      </trkpt>
      <trkpt lat="51.02958000" lon="13.72277000">
        <ele>147.30000</ele>
        <desc>Some description here</desc>
        <link>Some link here</link>
        <time>2010-01-01T00:06:09Z</time>
      </trkpt>
      <trkpt lat="51.02979000" lon="13.72456000">
        <ele>146.40000</ele>
        <desc>Some description here</desc>
        <link>Some link here</link>
        <time>2010-01-01T00:06:55Z</time>
      </trkpt>
    </trkseg>
  </trk>
</gpx>

Output (=before.gpx.res.gpx)
<?xml version="1.0" encoding="UTF-8" standalone="no" ?><gpx xmlns="http://www.topografix.com/GPX/1/1" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" creator="Graphhopper" version="1.1" xmlns:gh="https://graphhopper.com/public/schema/gpx/1.1">
<metadata><copyright author="OpenStreetMap contributors"/><link href="http://graphhopper.com"><text>GraphHopper GPX</text></link><time>1970-01-01T01:00:00+01:00</time></metadata>
<trk><name>GraphHopper MapMatching</name><trkseg>
<trkpt lat="51.032245" lon="13.710442"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.032216" lon="13.710586"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.032124" lon="13.710791"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.032279" lon="13.710969"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.032359" lon="13.711108"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.032295" lon="13.711252"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.032132" lon="13.711576"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.032035" lon="13.711791"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.031926" lon="13.71204"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.031887" lon="13.712119"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.031807" lon="13.712297"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.031777" lon="13.712364"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.031584" lon="13.712795"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.031452" lon="13.713086"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.031389" lon="13.713227"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.031256" lon="13.713523"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.03125" lon="13.713537"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.031197" lon="13.713655"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.031082" lon="13.713909"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.031014" lon="13.714061"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.030993" lon="13.714107"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.030942" lon="13.714222"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.030807" lon="13.714521"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.030714" lon="13.71473"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.030555" lon="13.715088"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.030509" lon="13.71519"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.030475" lon="13.715267"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.030379" lon="13.715484"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.030357" lon="13.715531"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.030351" lon="13.715546"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.030253" lon="13.715765"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.030221" lon="13.715838"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.030133" lon="13.716036"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.030006" lon="13.71632"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.029952" lon="13.716441"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.029786" lon="13.716748"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.029758" lon="13.716808"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.029725" lon="13.716884"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.029649" lon="13.717058"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.029561" lon="13.717253"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.029455" lon="13.717493"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.029442" lon="13.717521"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.029331" lon="13.717795"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.029282" lon="13.717916"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.029254" lon="13.717974"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.029069" lon="13.718398"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.029005" lon="13.718487"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.028971" lon="13.718522"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.028836" lon="13.718666"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.028804" lon="13.7187"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.029257" lon="13.719965"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.02969" lon="13.721255"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.029727" lon="13.721364"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.030046" lon="13.722299"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.030764" lon="13.724326"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.030582" lon="13.724389"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.029799" lon="13.724589"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.029779" lon="13.724206"><time>1970-01-01T01:00:00+01:00</time></trkpt>
<trkpt lat="51.02976" lon="13.723841"><time>1970-01-01T01:00:00+01:00</time></trkpt></trkseg></trk></gpx>

Ah, okay. Yes, this would be nice to have and should not be hard to implement - maybe you give it a try?

This is not an easy problem since multiple input points might be processed to produce a single output point or no input points at all have been used to produce a single output point.
So you need to make a decision which input point meta information you want to reuse on a single output point.
If there are multiple maybe put the meta information of all input points on a single output point.
If there are no direct input points for that single output, maybe backfill with the information from other output points.
It might also make sense to calculate an average value from the meta information for one specific output point.
Perhaps a generic approach should be to provide some kind of configurable hook, that allows to reattach meta-information.

There is already the concept of 'EdgeMatch' where one can set a list of GPXExtensions where one of them is a GPXEntry and a queryResult but others could be added there as well if one has the input data. Surely we should provide such a simple hook.

It is an awesome algorithm！helped a lot! thanks
I have the same question of the losted attributes in the outputfile. I want to know is there any solution of this problem?

Is there any progress on this issue?
I am a new user of GraphHopper Map Matching, and I encounter the same issue.
I want to keep time information in output gpx file.
as this post shows, the output gpx file's  items are always zero.
as @leoroos said, It's hard to decide a good timestamp for each output GPX entry.
@karussell can you provide more details of EdgeMatch and GPXExtensions?

Is there any progress on this issue?

We do not work on this yet.

@karussell can you provide more details of EdgeMatch and GPXExtensions?

See the duplicate/related issue #64

Similarly this would be nice if such a relationship is created for the new path details feature

Will you solve this issue? I encounter the same problem, I want to keep timestamp in output gpx files, at least for those origin points.
Can I simply calculate the distance of origin_point and output_point, and add timestamp to those nearest ones?

@zeromem

as this post shows, the output gpx file's items are always zero.

That, at least, is fixed.

The Route that we calculate has its own idea of how fast it is, and we currently put that information in the output.

This information does not show for me in the output, is there a way to enable this?
(I'm using commit 8e0cd26c88bdca96428f015a0e6daf33f72a3e5c


Receive Event when Activity in Background Android

How can I receive or subscribe multiple Events even though Activity is in background i.e. activity isn't visible ?

simply call register() inside onCreate then unregister inside onDestroy.
I usually do it like this:
EventBuses.kt file
val ForegroundEventBus = EventBus()
val BackgroundEventBus = EventBus()

abstract class BaseActivity() : AppCompatActivity
{
    fun onCreate(savedInstanceState : Bundle?)
    {
        super.onCreate(savedInstanceState)
        try { BackgroundEventBus.register(this) }
        catch (ignored : Exception) {}
        try { ForegroundEventBus.register(this) }
        catch (ignored : Exception) {}
        //wiil thow exception if activity doesn't have a method annotated with Subscribe, so I'll just ignore the exception
    }

    fun onPause()
    {
        ForegroundEventBus.unregister(this)
        super.onPause()
    }

    fun onDestroy()
    {
        BackgroundEventBus.unregister(this)
        super.onDestroy()
    }
}

Then I extend all my activities from BaseActivity. Usually I put more configuration stuff (other libraries etc) inside BaseActivity.
Then if I want an event to be received only by foreground activity, I do ForegroundEventBus.post(someEvent)
Or if I want an event to be reveiced by all background activities, I do BackgroundEventBus.post(someEvent)
Please close this issue since it's not a bug related to the library functionality.

Can you please provide example for the same it will be most helpful ?
And what happened if User Clear App from Recent List than What happened to BackgroundEventBus as there will be not  call for onDestroy() ??

Android itself can do two things:

either terminate activities, in this case onDestroy will be called
or either terminate the entire app process, in this case I don't think onDestroy will be called, but since the entire process will be killed, it won't matter. When a process is killed, its memory allocated in RAM is deleted. When the app / process starts again, it will start "clean".

As far as I know, when use clears the app from recents, the process gets killed.
I don't understand the first question. The example I gave is self explainatory?

I am using java right now so please provide EventBuses.kt file in java and I don't get this how to declare and use ?

ok
public class EventBuses
{
    public static final EventBus ForegroundEventBus = new EventBus();
    public static final EventBus BackgroundEventBus = new EventBus();
}

import static your.package.name.EventBuses.*;

public abstract class BaseActivity extends AppCompatActivity
{
    public void onCreate(Bundle savedInstanceState)
    {
        super.onCreate(savedInstanceState);
        try { BackgroundEventBus.register(this); }
        catch (ignored : Exception) {}
        try { ForegroundEventBus.register(this); }
        catch (ignored : Exception) {}
    }

    public void onPause()
    {
        ForegroundEventBus.unregister(this);
        super.onPause();
    }

    public void onDestroy()
    {
        BackgroundEventBus.unregister(this);
        super.onDestroy();
    }
}

Then, to use:
AnyFile.java
import static your.package.name.EventBuses.*;

ForegroundEventBus.post(whatever);
BackgroundEventBus.post(whatever);

How should I setup Event Bus for triggering multiple events from my Socket Connections every seconds ?

How should I setup Event Bus for triggering multiple events from my Socket Connections every seconds ?

postEvent in your socket callback method

Closing this issue due to inactivity. :zzz: Feel free to comment with more details or submit a new issue.


fix test to be more likely of passing

More details:
There is a chance that the sim-anneal test will fail because it is built on random chance. Changing to >= makes this test more robust. If any continued issues, the number of iterations can be increased to increase the likelihood of avoiding an outlier solution.

Sounds good. Let's monitor this during milestone 3 and see how it goes.


pull request for e2e test at 01/13/2017 16:26:43

:x: Validation status: errors
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Iops, throughput and instance view

Added IOPS and BW properties on managed disk  for UltraSSD disks.
These properties can be set only on the VMScaleSet but are also returned via VMSCaleSet VMs.
ListVMs at susbcription level now supports a statusOnly=true query param and CRP would returned a trimmed down instance view in that case
Added samples for GETVM and GETVMInstanceView
Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

Can one of the admins verify this patch?


Allow FilterByValueBlock to handle one or many values

97d3fcd filterblock: Don't break when filtering on unexpected data
commit 6c1f3eee12d47bf4be7fa23282732d3c8a143028
Author: Russell Bryant<EMAIL_ADDRESS>Date:   Wed Jul 3 21:28:56 2024 -0400
Allow FilterByValueBlock to handle one or many values

This block previously only accepted a single value to filter on. This
update makes it handle a list, as well. In that case, it will ensure
that the filter matches one of the values in the list.

Signed-off-by: Russell Bryant<EMAIL_ADDRESS>

Great. Nice and simple, no confusion. lgtm

I rebased this and marked it ready for review now that the pre-req PR merged.

lgtm :+1:

lgtm 👍

thanks for the review!


First draft of Rust style guide

Address Issue 186

recheckcla

Some of code in the PR was confusing the website rendering code, I've removed this guide from develop to make the build pass.
@miedziak can advise on what needs to be fixed.


[BUG] Unsafe URL validation

Describe the bug
The URL validation method in index_model.py uses a simple contains match. As a result, someone could register a domain that contains the name and inject unexpected data, leading to exposing potential parser vulnerabilities.
Expected behavior
URL's should be completely validated.
Additional Context
Presumably the URL validation is intentionally ambiguous due to the fact that regional youtube TLD's (such as .com, .co.uk, .de, etc)
Potential Solution
Use a project like tldextract and join the URL with the tld / public suffix list before passing it to the transcription parser

I have a partial solution for this, however I haven't completed the error handler.


Is there a way to set the user agent?

In RestSharp there is a UserAgent property on the RestClient object. Any equivalent here?

Yes, you can use
restClient.AddDefaultParameter("User-Agent", "your-user-agent", ParameterType.HttpHeader);

I decided against a specific property for the user agent, because it might clash with the default parameter that you can set too.


remove alias to corrected methods

Remove redundant aliasing of the corrected? and corrected_with_todo? method.

Before submitting the PR make sure the following are checked:

[ ] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[ ] Feature branch is up-to-date with master (if not - rebase it).
[ ] Squashed related commits together.
[ ] Added tests.
[ ] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[ ] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[ ] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

CI has failed. Can you fix the offenses?
Offenses:

lib/rubocop/cop/offense.rb:73:81: C: Metrics/LineLength: Line is too long. [84/80]
      #   whether this offense is automatically corrected via autocorrect or a todo.
                                                                                ^^^^
lib/rubocop/cop/offense.rb:93:81: C: Metrics/LineLength: Line is too long. [93/80]
      #   whether this offense was locally disabled with a disable or todo where it occurred.
                                                                                ^^^^^^^^^^^^^

https://circleci.com/gh/rubocop-hq/rubocop/68246

Thanks!


Ctags: Unknown option --extras

Environment data

VS Code version: 1.29.1
Extension version (available under the Extensions sidebar): 2018.11.0
OS and version: Fedora 29
Python version (& distribution if applicable, e.g. Anaconda): 3.7.1 and 2.7.15 (Python version unrelated)
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): any
Relevant/affected Python packages and their versions:
ctags version (obtained through referred site)

→ ctags --version
Exuberant Ctags 5.8, Copyright (C) 1996-2009 Darren Hiebert
  Compiled: Jul  9 2009, 16:56:18
  Addresses<EMAIL_ADDRESS>http://ctags.sourceforge.net
  Optional compiled features: +wildcards, +regex

Actual behavior
ctags: Unknown option: --extras when trying to view workspace's symbols.
Expected  behavior
List of workspace's symbols is shown.
Steps to reproduce:

Download and install latest ctags release - 5.8 - for Fedora (.rpm file) from http://ctags.sourceforge.net
In VSCode press ctrl+t for workspace's symbols.
Verify ctags: Unknown option: --extras error in Output is shown and no symbols generated.

Logs
Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)
ctags: Unknown option: --extras

Output from Console under the Developer Tools panel (toggle Developer Tools on under Help)
[ms-python.python]ctags: Unknown option: --extras

Checking man ctags, there only appears to be extra option and no mentioning of extras.
Sounds like either different version of ctags was used in #2758 or that packages differ internally on the same version.

I have the same problem. My ctags version is
$ ctags --version
Exuberant Ctags 5.9~svn20110310, Copyright (C) 1996-2009 Darren Hiebert
  Addresses<EMAIL_ADDRESS>http://ctags.sourceforge.net
  Optional compiled features: +wildcards, +regex

FYI we have #197 open to move over to universal ctags, but we have not made any movement on it as we are planning to have the language server replace the need for ctags entirely. But if someone here wanted to help us move to universal ctags before then we would happily review any PRs on it.
Otherwise we will probably revert #793.

Same issue in macOS 10.14 with brew installed ctags (5.8)

I'm also having this problem (macOS 10.12.6, brew installed ctags) and this workaround works for me:
sed -i .bak 's/^--extras=/--extra=/' "${HOME}/.vscode/extensions/ms-python.python-2018.11.0/resources/ctagOptions"

I had the same issue. I fixed it easily by modifying the option "--extras" to "--extra" in file "~/.vscode/extensions/ms-python.python-2018.11.0/resources/ctagOptions"

Same problem on MacOS 10.14.1, ctags 5.8 installed by brew, VS Code 1.19.1.
The method proposed by @nikau6 works for me well. Thanks.


How to install stable-baselines right on Centos7?

I have a project which can run normally on win10, and I want to transform it to centos7, I generated the requirements.txt and install it on Centos7. But I got this error when running
(wzs) [root@ecs-728a-0002 rltest]# python run_paraellel3.py 2020-07-31 20:28:12.863818: W tensorflow/stream_executor/platform/default/dso_loader.cc:59] Could not load dynamic library 'libcudart.so.10.1'; dlerror: libcudart.so.10.1: cannot open shared object file: No such file or directory 2020-07-31 20:28:12.863853: I tensorflow/stream_executor/cuda/cudart_stub.cc:29] Ignore above cudart dlerror if you do not have a GPU set up on your machine. /root/anaconda3/envs/wzs/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:541: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.   _np_qint8 = np.dtype([("qint8", np.int8, 1)]) /root/anaconda3/envs/wzs/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:542: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.   _np_quint8 = np.dtype([("quint8", np.uint8, 1)]) /root/anaconda3/envs/wzs/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:543: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.   _np_qint16 = np.dtype([("qint16", np.int16, 1)]) /root/anaconda3/envs/wzs/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:544: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.   _np_quint16 = np.dtype([("quint16", np.uint16, 1)]) /root/anaconda3/envs/wzs/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:545: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.   _np_qint32 = np.dtype([("qint32", np.int32, 1)]) /root/anaconda3/envs/wzs/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:550: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.   np_resource = np.dtype([("resource", np.ubyte, 1)]) Traceback (most recent call last):   File "run_paraellel3.py", line 2, in <module>     from stable_baselines.common.callbacks import *   File "/root/anaconda3/envs/wzs/lib/python3.7/site-packages/stable_baselines/__init__.py", line 4, in <module>     from stable_baselines.deepq import DQN   File "/root/anaconda3/envs/wzs/lib/python3.7/site-packages/stable_baselines/deepq/__init__.py", line 1, in <module>     from stable_baselines.deepq.policies import MlpPolicy, CnnPolicy, LnMlpPolicy, LnCnnPolicy   File "/root/anaconda3/envs/wzs/lib/python3.7/site-packages/stable_baselines/deepq/policies.py", line 2, in <module>     import tensorflow.contrib.layers as tf_layers   File "/root/anaconda3/envs/wzs/lib/python3.7/site-packages/tensorflow/contrib/__init__.py", line 30, in <module>     from tensorflow.contrib import cloud   File "/root/anaconda3/envs/wzs/lib/python3.7/site-packages/tensorflow/contrib/cloud/__init__.py", line 24, in <module>     from tensorflow.contrib.cloud.python.ops.bigquery_reader_ops import *   File "/root/anaconda3/envs/wzs/lib/python3.7/site-packages/tensorflow/contrib/cloud/python/ops/bigquery_reader_ops.py", line 21, in <module>     from tensorflow.contrib.cloud.python.ops import gen_bigquery_reader_ops   File "/root/anaconda3/envs/wzs/lib/python3.7/site-packages/tensorflow/contrib/cloud/python/ops/gen_bigquery_reader_ops.py", line 357, in <module>     _op_def_lib = _InitOpDefLibrary(b"\n\355\001\n\016BigQueryReader\032\024\n\rreader_handle\030\007\200\001\001\"\027\n\tcontainer\022\006string\032\002\022\000\"\031\n\013shared_name\022\006string\032\002\022\000\"\024\n\nproject_id\022\006string\"\024\n\ndataset_id\022\006string\"\022\n\010table_id\022\006string\"\027\n\007columns\022\014list(string)\"\027\n\020timestamp_millis\022\003int\"\034\n\016test_end_point\022\006string\032\002\022\000\210\001\001\n\331\001\n GenerateBigQueryReaderPartitions\032\016\n\npartitions\030\007\"\024\n\nproject_id\022\006string\"\024\n\ndataset_id\022\006string\"\022\n\010table_id\022\006string\"\027\n\007columns\022\014list(string)\"\027\n\020timestamp_millis\022\003int\"\025\n\016num_partitions\022\003int\"\034\n\016test_end_point\022\006string\032\002\022\000")  File "/root/anaconda3/envs/wzs/lib/python3.7/site-packages/tensorflow/contrib/cloud/python/ops/gen_bigquery_reader_ops.py", line 265, in _InitOpDefLibrary     _op_def_registry.register_op_list(op_list) AttributeError: module 'tensorflow.python.framework.op_def_registry' has no attribute 'register_op_list'
here is my requirements.txt (python 3.7)
psutil==5.7.0 tensorflow==1.14.0 numpy==1.18.5 seaborn==0.9.0 pandas==1.0.5 matplotlib==3.2.2 plotly==4.8.1 scipy==1.5.0 jsonlines==1.2.0 jqdatasdk==1.7.8 gym==0.15.6 stable_baselines==2.10.0 scikit_learn==0.23.1 wandb==0.8.9

Please format the text with triple-ticks ``` like this ``` . On a first glance it seems like there is something is wrong with tensorflow, but hard to tell based on the messy text.


Cannot connect remotely on GraalVM debugger

I'm trying to connect remotely to a GraalVM debugger, but the connection always fails to establish (GraalVM 20.0.0 java 11, running on Ubuntu 19.10).
For the test, I'm running a Java application using GraalVM on my machine, with the following configuration for the engine :
Engine engine = Engine.newBuilder()
            .option("inspect", "9930")
            .option("inspect.Secure", "false")
            .build();
Context.Builder contextBuilder = Context.newBuilder().engine(engine);

I can connect on localhost, using the url provided in the logs :
Debugger listening on port 9930.
To start debugging, open the following URL in Chrome:
    chrome-devtools://devtools/bundled/js_app.html?ws=<IP_ADDRESS>:9930/4b4523f8-a5dcc9aabd2

but it doesn't work if I replace <IP_ADDRESS> by my real ip address.
Running the command sudo netstat -plnt show the following result :
tcp6       0      0 <IP_ADDRESS>:9930          :::*                    LISTEN      26280/java

showing that the debugger only listens on localhost, thus refusing external connections.
Is there an additional parameter to enable remote debugging ? Following this link https://www.graalvm.org/docs/tools/chrome-debugger, I tried to add the option .option("inspect.Remote", remoteConnect) as shown in the example, but the parameter is unknown.

You need to make the inspector listen on that address. "inspect.Remote" option was abandoned, you should use .option("inspect", "<IP_ADDRESS>:9930") instead, where you replace "<IP_ADDRESS>" with your IP address.

Thanks a lot, it works now !
Sorry for the bother.


Added a small gif to the README

We should get a bigger sized GIF

I will make a new version with a different GIF


Parse strings entirely, without stopping at null bytes

This was preventing binary strings from being passed correctly from Lua to Rust.
Fixes #156.

Thanks


How to get consent for multiple resource scopes at once in the microsoft authentication provider

Our extension https://marketplace.visualstudio.com/items?itemName=quantum.qsharp-lang-vscode-dev needs to make a few different vscode.authentication.getSession calls with slightly different params, as we first need to get the list of tenants, then sign-in to the specific tenant (for ARM sign-in), and once this is done requests a scope to a different resource (for https://quantum.microsoft.com/). This results in repeated pop-ups with "The extension ... wants to sign-in using Microsoft", followed by an account quick-pick for "The extension ... wants to access a Microsoft account"
Generally if writing code against AAD directly (e.g. using MSAL or similar) you would list the scopes you want to consent for in advance for the login and just consent once, then aquireToken for already consented scopes will silently work. I've tried various methods in the getSession API of listing scopes, having a static set and requesting '/.default', etc., all with no luck. Seems you need to call getSession for each unique combination of resource and tenant, and each results in a UX interactions for approval.
The result is our extension can prompt you 3 times for consent when trying to do one operation (connect to your Azure Quantum workspace). Is there a way to achieve this? Am I just approaching it wrong (current code at https://github.com/microsoft/qsharp/blob/main/vscode/src/azure/workspaceActions.ts#L19 )? Or is this just a limitation of the current API design?
Thanks!

This should be fixed in https://github.com/microsoft/vscode/pull/193727 making it a dupe of https://github.com/microsoft/vscode/issues/184218
let me know tomorrow in Insiders if the issue persists!

Try this for your getAuthSession function:
/**
*
* @param {string[]} scopes
*/
async function getAuthSession(scopes) {
    output.info("About to getSession for scopes", scopes.join(","));
    let session = await vscode.authentication.getSession(
        "microsoft",
        scopes,
        { silent: true }
    );
    if (!session) {
        session = await vscode.authentication.getSession(
            "microsoft",
            scopes,
            { createIfNone: true }
        );
    }
    output.info("Got session: ", session);
    return session;
}

All I did was add a silent call first. I think what you're hitting is... you're always using createIfNone: true which will show a pop up if there is no "session preference". Session Preference is really only a thing if the auth provider supports multiple accounts, which the Microsoft Auth Provider does... it's also decided by the user, not extensions. That said, if you use the silent flow, then we pick the last used account because a session is better than no session.
When I tried your https://github.com/billti/vsmin/tree/tokens with my work account, I only got one pop up.
I will say, I did get a 2nd pop up for my MSA, but that's expected, because my MSA doesn't have access to the Microsoft tenant.


Support db_accessadmin fixed database role

Description
Support fixed database role db_accessadmin in babelfish.
Member of db_accessadmin can do the following

CREATE SCHEMA
CREATE/DROP USER
ALTER USER
They always have connect permission

Issues Resolved
[BABEL-5136]
Test Scenarios Covered
Check List

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

250 of 274 (91.24%)  changed or added relevant lines in 9 files are covered.
14 unchanged lines in 6 files lost coverage.
Overall coverage increased (+0.006%) to 74.44%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

contrib/babelfishpg_tsql/src/dbcmds.c
62
63
98.41%

contrib/babelfishpg_tsql/src/hooks.c
32
34
94.12%

contrib/babelfishpg_tsql/src/multidb.c
12
15
80.0%

contrib/babelfishpg_tsql/src/catalog.c
72
76
94.74%

contrib/babelfishpg_tsql/src/rolecmds.c
8
13
61.54%

contrib/babelfishpg_tsql/src/pl_handler.c
44
53
83.02%

Files with Coverage Reduction
New Missed Lines
%

contrib/babelfishpg_tsql/src/dbcmds.c
1
75.83%

contrib/babelfishpg_tsql/src/pl_handler.c
1
89.95%

contrib/babelfishpg_tds/src/backend/tds/tdsutils.c
2
73.39%

contrib/babelfishpg_tds/src/backend/tds/tdscomm.c
2
76.03%

contrib/babelfishpg_tsql/src/catalog.c
3
87.28%

contrib/babelfishpg_tsql/src/rolecmds.c
5
83.16%

Totals

Change from base Build<PHONE_NUMBER>9:
0.006%

Covered Lines:
44927

Relevant Lines:
60353

💛  - Coveralls


Odometry ground truth method

This PR adds a ground truth method to SensorFilter which allows the use of webots ground truth information for odometry which is useful for testing purposes.
Note, this will only work when using our webots controller.
Something I'm unsure on is it seems the recent timestamp changes https://github.com/NUbots/NUbots/pull/1116/files#diff-fc0b2b2d580a7b5d606168708dc45d1b335285a24d5c80f1f4ef7846672cc8b4 required the change in Webots.cpp. Maybe we should still be using sensor_measurements.time for the comparison, I need to look into it more.

Update the comment in the default config too

The method should never be used in the default config because it will only work in webots so thought it was best not to include it

I guess. It would be cool to get mocap ground truth.


Implement dolweb/lobby

Prerequisite for https://github.com/dolphin-emu/dolphin/pull/7945.

The way this is written just cannot work as it is. Requests are round-robined between multiple independent workers that don't share memory. Your global "sessions" object is going to be per-worker, which means this is never going to work in production.
I also question why this has to be part of dolphin-emu/www -- it doesn't seem to share any of the codebase or the database models. It seems like it could be a standalone wsgi app written on a less heavyweight framework (Tornado, Flask, Bottle, ... pick your poison).
Finally, before this is live we should have a story for moderation.

Having seen the Melee community, at the very least we need a disable chat button for users.  As discussed on IRC, we probably need some backend access to remove certain lobbies.  Maybe it would be best to find someone from in those communities to help with moderation.
Other than that... what else can we do?

Superseeded by https://github.com/spycrab/netplay-index


chore: update llama.py

minor fix

@eltociear thanks for pointing out the doc typo! would you like to contribute to demo using the SDK or work on more API support?


hasBackdrop: false prevents transparent overlay behind modal

Bug, feature request, or proposal:
When I set hasBackdrop: false, the overlay behind modal is not dark and transparent as expected
What is the expected behavior?
I expect the same behavior as hasBackdrop: true, the overlay behind modal is dark and transparent
What are the steps to reproduce?
https://stackblitz.com/edit/angular-material2-issue-4gl8gc
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular 5, @angular/material & @angular/cdk: 5.1.1

That is expected behavior. The 'transparent overlay' behind the modal is the backdrop. What is it that you're trying to accomplish by setting hasBackdrop: false?

Oh really, I did not it's the expected behavior. Is it possible to customize the back layer as normal backdrop if I still set hasBackdrop: false ? I used ng-bootstrap before and even I set backdrop to false, I still had normal backdrop but disable clicking outside modal to close it. Therefore, I just want to know that if material has this feature.

Rather than setting hasBackdrop to false, I think what you'd like to do is use disableClose: true so that the user cannot close the modal via clicking on the backdrop.

Thank you so much, it works.

Rather than setting hasBackdrop to false, I think what you'd like to do is use disableClose: true so that the user cannot close the modal via clicking on the backdrop.

thanks ,this works for me "hasBackdrop:true"


strngs.h replacement ?

Environment

Tesseract Version: 5.0.0
Commit Number:
Platform: Gentoo AMD64

Current Behavior:The include file tesseract/strngs.h is missing but needed by gimagereader, e.g.
Expected Behavior: replacement (upgrade guide) for strngs.h
Suggested Fix:

That file was used for a proprietary string class which is no longer supported by release 5. Please update your software and use char *, std::stringor whatever fits best to your needs.

gImageReader was already updated for release 5 in January, see https://github.com/manisandro/gImageReader/commit/399f5503f88af423ada70cc903a19bdee5e096e8.

Vielen Dank, Stefan


use local assets instead of CDN

This one falls into "nice to have" category, so please close it if you find it irrelevant.
The idea is, to not to rely on any external CDN (especially for asset this small), as they're usually set tracking cookies, log IPs and do some other nasty stuff. I think, that for the sake of privacy of the user, it's better to serve this locally. What do you think?

External css should rater go to the angular.json, like: https://github.com/bpatrik/pigallery2/blob/d2d1472763aaa97730f695e1f4ffecb760aab454/angular.json#L74-L81
I'm not sure if the button support its (i.e.: provides a css within the node module). If not, we probably need to replace it.
Also the APP can never get fully "offline" as there is reverse geocoding in search that uses same API.

Unfortunaltely I'm not a JS dev, so if something else needs to be done, to make this work - I won't be a big help here :/ I did this monkey-patch in my container and it worked though :)
Regarding geocoding - I'm not sure what's going on there, but I'm running pigallery2 in a container, with Internet access cut off - and didn't notice any issues. Probably I'm not using said feature.


YouTube for Schools and "unable to extract uploader nickname"

Currently using YouTube for Schools (http://www.youtube.com/schools) on site. Have setup a user through the YouTube admin panel. User is able to login successfully and view content unrestricted.
Example video is allowed/unfiltered for YouTube for Schools. Also receive same error for blocked/filtered content even passed YouTube account credentials.
youtube-dl -i -ctf 22 -u USER_NAME -p PASSWORD http://www.youtube.com/watch?v=w3AdFjklR50
[youtube] Setting language
[youtube] Logging in
[youtube] Confirming age
[youtube] w3AdFjklR50: Downloading video webpage
[youtube] w3AdFjklR50: Downloading video info webpage
[youtube] w3AdFjklR50: Extracting video information
WARNING: unable to extract uploader nickname
ERROR: unexpected call to parse_endtag
youtube-dl -v[debug] youtube-dl version 2013.02.02
[debug] Python version 2.7.2+ - Linux-3.0.0-12-generic-i686-with-Ubuntu-11.10-oneiric
Couldn't see any other posts regarding YouTube for Schools restriction. Not sure if you've had to tackle this in the past.

Works fine with latest version.


sane up failed can not find mkdirp

well, sane up seems failed to boot ember, the error message is below:
client   | Error: Cannot find module 'mkdirp'
client   |     at Function.Module._resolveFilename (module.js:336:15)
client   |     at Function.Module._load (module.js:278:25)
client   |     at Module.require (module.js:365:17)
client   |     at require (module.js:384:17)
client   |     at Object.<anonymous> (/home/ming/work/xrp/client/node_modules/ember-cli-less/node_modules/broccoli-less-single/index.js:3:14)
client   |     at Module._compile (module.js:460:26)
client   |     at Object.Module._extensions..js (module.js:478:10)
client   |     at Module.load (module.js:355:32)
client   |     at Function.Module._load (module.js:310:12)
client   |     at Module.require (module.js:365:17)
client   |     at require (module.js:384:17)

i'm not sure if it is same problem like #19 ?

No that is a different error and seems to me related to ember-cli-less. What happens if you run ember s in the client folder.

Strange that the less module is erroring. I don't get that problem when I boot my server with ember s What system are you on @feitian124 ?

@Globegitter the same problem.
i'm on ubuntu 14.10, node  0.12.2, npm 2.7.4

hmm, i just tried a fresh ubuntu 14 and it seems working with ember s, i can't reproduce the problem...

@feitian124 did you try to delete your node_modules folder and npm install again in the client folder? Maybe npm just crapped out for some reason.


Windows Batch Install script not included in release

The .exe and -win.zip release bundles do not include the windows-install.bat script required for installing the required dependencies.

That`s not a bug. Link for windows-install.bat included at readme.md
But ok, ok, I will add it at next release


Fixed preview access in 3.0, where preview is broken in members theme

Currently, if you use the Lyra theme with members enabled on v3.0, you cannot use the preview.
Ghost gates any post not published, and requires an access level of paid. This renders preview completely broken in 3.0 with the Lyra theme.
Accordingly, this pull request fixes the issue. As a note, an issue on the Lyra theme repository has been started.

@allouis Did some digging into this and imo the fix is good as it seems we skip the entry controller in case of preview where we otherwise set the correct access here, so similar logic for preview works well across all cases. Lmk what you think :)

Yes, I checked out the code for how you set the access, and for this case, it looks like it was just forgotten.
Completely similar logic as in [entry.js] (https://github.com/TryGhost/Ghost/blob/518f2ccd827b4d724ee4e9895d8e9d7d354d1fc4/core/frontend/services/routing/controllers/entry.js#L88), at the right place in preview.js.
I really don't think anything was missed here, and I'm happy to hopefully have my preview back in ghost soon :)

@casperbh96 Thanks for raising the fix, appreciate it 😄 Will make sure the fix is out in next release

Hey @casperbh96 thanks for your patience on this one!
Fix looks good and tested locally for me :relaxed:

Does this fix was released? Just updated to 3.0.2 but the problem still occurs.

@faultable no, it was committed after the 3.0.2 release

@VictorVation thanks for confirming, subscribed 👍

Maybe a similar issue, If I published a post with no content, I get the "This post is for subscribers only
" notice.


[W07H03] fix file path

the tests were not in the test folder. this should fix this.

@MaximilianAnzinger


text to video datasets?

Which datasets are used when i use to train text to video

ucf101 is a feasible choice

ucf101 is a feasible choice

thanks!!!


Permit definition of Relationships between a model and its own type

Fixes: #157

This appears to turn out to be a really simple fix - I removed the Relationship.clean() logic that was explicitly blocking creation of Relationships with the same source_type and destination_type, and as far as I can determine, it all just works. The remainder of these diffs are as follows:

In all test cases using Relationship and RelationshipAssociation objects, instantiate the objects and call validated_save() rather than calling .objects.create(). Making this change uncovered and allowed me to fix a couple of cases where we were not actually creating valid records.
In the RelationshipAssociation table view, we can't actually order by source or destination as those are GenericForeignKeys, so I've added orderable=False to those two columns.
Also in the RelationshipAssociation table view, I added optional columns for the source_type and destination_type, because 1) these are columns that the filter allows us to filter on, so they should be displayed and 2) these columns are orderable.
In test_relationships.py, I've added a self-referential relationship definition and added a couple of test steps, especially one that validates that cascading deletion of RelationshipAssociations still works when deleting a model that has a relationship to itself.

(Here "Destination Sites" and "Source Sites" are the two sides of a many-to-many relationship between Site and Site, and "Alphabetically After" and "Alphabetically Before" are the two sides of a one-to-one relationship between Site and Site.)

I'm curious, how does the delta site looks like ?

"Alphabetically After" and "Alphabetically Before" are the two sides of a one-to-one relationship between Site and Site
In my mind, we shouldn't be able to see and configure both sides on the same object otherwise we can't guarantee a one-to-one relationship

Alpha:

Beta:

Gamma:

Delta:

In my mind, we shouldn't be able to see and configure both sides on the same object otherwise we can't guarantee a one-to-one relationship

Can you elaborate on this? Are you saying that one-to-one means an object should not be able to be the source of one RelationshipAssociation and the destination of another RelationshipAssociation for the same one-to-one Relationship? That's not how I would interpret that. To my mind a "chain" of one-to-one relationships as I have here (Alpha -> Beta, Beta -> Gamma, Gamma -> Delta) should be perfectly valid, and for that matter even a self-to-self relationship (hypothetically Theta -> Theta) still satisfies the "one-to-one" cardinality restriction and shouldn't necessarily be blocked.

(As an aside - it might be nice as a future enhancement to sort (either alphabetically, or weighted a la custom fields) the various relationships as presented in the GUI, as the current ordering is a bit non-obvious...)

Can you elaborate on this? Are you saying that one-to-one means an object should not be able to be the source of one RelationshipAssociation and the destination of another RelationshipAssociation for the same one-to-one Relationship?

Yes that's what i have in mind and I think there are some valid use cases for that. At the same time, I agree there is are some valid use cases for the "chain" that you describe. Maybe we need to find a different name to define them.
As a general guideline, I think a relationship should appears only once per object and a given object shouldn't be a source and a destination of the same relationship.
One concern with your approach is how would it look like in GraphQL / the ORM or the API ? In the UI we can customize the Labels and it's easy to understand the differences but how would it work outside of the UI ?
For example, if I have a pair of devices that are expected to work together, I might want to create a one-to-one relationship to track the peer. I can only have 1 peer and when I query each device, from GraphQL or the ORM, I shouldn't have to know if they have been defined as the source or the destination when the relationship was created, I just want to get the device on the other side of the relationship.

Per further discussion with Damien, this needs some additional thought. Specifically:

When a relationship is same-type-to-same-type, do we allow a given record to participate as both source and destination, or do we make these mutually exclusive?

Some relationship types (e.g. one-to-many parent-to-child) would make sense to allow a record to be both
Other relationship types (e.g. one-to-one "cable-like" relationships between interfaces) should be mutually exclusive (interface1 -> interface2 means that you cannot also do interface2 -> interface3)
Maybe we need to add an additional boolean flag to the Relationship model? This is already a pretty complex model though, need to think through how to document and UI it to minimize user confusion.

The current GraphQL modeling of relationship associations does not distinguish between source / destination types, it just shows whatever records are on the "other side" of the relationship (e.g., "rel_related_site", not "rel_related_site_source" or "rel_related_site_destination").

Need to figure out how we would address this when a model can be both source and destination for the same relationship type - currently only one "side" appears in GraphQL in this case, which is definitely wrong.
Additionally, for one-to-many relationships between the same type, one side of the relation is singular and the other is plural, do we need a union type here to handle both?


Start transaction: Location response header is wrong

Current Behavior
When I start a transaction, I get back a wrong location response header. Subsequent requests (add/delete/..) then use this wrong location and send the requests to the wrong place.
Example request:

Wrong location response header:
http://edc-b.xxxxxxxx.test/repositories/edc-graphdb/transactions/c7e4fe6b-c67a-40e0-84c5-aab945a40769
Expected Behavior
I should receive the correct location response header:
https://edc-b.xxxxxxxx.test/edc-inventory-graphdb-database/repositories/edc-graphdb/transactions/c7e4fe6b-c67a-40e0-84c5-aab945a40769
Steps To Reproduce

Setup GraphDB behind a reverse-proxy and let it run in a subcontext
Create a connection to GraphDB
Start a transaction => Location response header is wrong

Version
RDF4J 3.7.3
Ontotext GraphDB 9.10.2-EE
Are you interested in contributing a solution yourself?
Yes
Anything else?
We're using GraphDB which is using RDF4J. The Ontotext support said that the problem was not in their code but had to be solved in the RDF4J code.
I think request.getRequestURL(); is the problem:
https://github.com/eclipse/rdf4j/blob/b5f9e4c5aca5d41b7c4207e09ca743c38e47e500/tools/server-spring/src/main/java/org/eclipse/rdf4j/http/server/repository/transaction/TransactionStartController.java#L130-L137
An improvement like this could fix the wrong protocol if the code would check for a X-Forwarded-Proto header. For the missing subcontext we may need to introduce a baseUrl property or something similar.

My current workaround:
    public HTTPRepository createRepository() {
        final HTTPRepository httpRepo = new HTTPRepository(triplestoreBaseUrl, "edc-graphdb");
        httpRepo.setHttpClient(createHttpClient());
        httpRepo.setUsernameAndPassword(username, password);
        httpRepo.init();
        return httpRepo;
    }

    private CloseableHttpClient createHttpClient() {
        final HttpResponseInterceptor responseInterceptor = (response, httpContext) -> {
            if (!response.containsHeader(LOCATION_HEADER)) {
                return;
            }

            final Header[] headers = response.getHeaders(LOCATION_HEADER);
            response.removeHeaders(LOCATION_HEADER);

            for (final Header header : headers) {
                final String location = header.getValue();
                if (StringUtils.contains(location, ".test/repositories")
                    || StringUtils.contains(location, ".ch/repositories")) {
                    final String correctLocation = location
                            .replace("http://", "https://")
                            .replace(".test/repositories", ".test/edc-inventory-graphdb-database/repositories")
                            .replace(".ch/repositories", ".ch/edc-inventory-graphdb-database/repositories");

                    LOG.info("Fix wrong location header. Before: {} After: {}", location, correctLocation);
                    response.setHeader(LOCATION_HEADER, correctLocation);
                } else {
                    response.setHeader(LOCATION_HEADER, location);
                }
            }
        };

        return HttpClients.custom().addInterceptorLast(responseInterceptor).build();
    }
    ```


Made the nav bar

I have made the navbar appropriate for your website . Please merge it under hacktoberfest tag.

Update these issues in navbar :

Spelling of captcha
Specify only Home , Contributors and About section in navbar in the same respective order.

NOTE : Make sure you sync your fork and avoid creating PR through main branch instead create a separate branch for your feature and then create PR.
You can once go through the guidelines it will help you.


RPC error when deleting node

As a node is deleted, mobject-graph-webserver reports the following RPC error:
Error calling RPC method GetStatus: Error: Call 185: Unknown UUID : c0874bcf-4a4d-4e5d-b260-d0a8ff09e243
    at #handleFailure (C:\Temp\repos\mobject\mobject-graph-webserver\node_modules\mobject-client\src\core\AsyncClient.js:298:29)
    at #handleResponseBasedOnStatus (C:\Temp\repos\mobject\mobject-graph-webserver\node_modules\mobject-client\src\core\AsyncClient.js:166:28)
    at #enqueueRequest (C:\Temp\repos\mobject\mobject-graph-webserver\node_modules\mobject-client\src\core\AsyncClient.js:118:68)

Node is still deleted successfully though, and no side-effects have been noticed.

Fixed in previous version


Update roles.md

Updating analyst recommended role to LA operator instead of contributor.

@yelevin
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Docs Build status updates of commit 0d40eed:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/sentinel/roles.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 3305ac4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/sentinel/roles.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit dc3d014:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/sentinel/roles.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Typescript support / docs

I really like this module, I love that it is way simpler to use and understand than many other solutions out there. But I wish it would work better with typescript and all the other tooling out there.
That's why I started working on 'porting' Hyperswarm to typescript on my fork, it's not yet complete and I'm probably also not the best typescript dev in existence, but it's a starting point that I use in my project.
Since I think other users of the module might benefit from stuff that I've been doing i.e. typings, docs, etc. I thought I ask you what the wisest thing to do would be.
Are you open to accepting a .d.ts file PR? Or should my fork be kept separate?
I've generated a bit of WIP docs from the code as well, maybe that can be put to use somehow.
I'm here to help make this module more accessible if that's possible!
Cheers!

hey @JonasKruckenberg would you be willing to maintain a d.ts file long term?
I personally think d.ts files are great with just normal JS because of the intellisense benefits and ease integration for those who do use typescript. The only issue is that TypeScript moves forward as a language quite quickly so the d.ts files can quickly become outdated, not because of breaking library changes but because of breaking TypeScript changes. This is what I found with Pino anyway, eventually we removed the d.ts file as it became too contentious.
But if you're okay to commit to maintaining d.ts files each of the hyperswarm modules then I'm +1 on accepting a PR for a d.ts file on each repo

Yeah I think the majority of the work isn’t to maintain the typings but to create them in the first place, so yeah I‘m willing to maintain the typings long term. I’d could use some help creating explanations for the functions though, but I can count on your help there right?
What modules are you considering part of Hyperswarm that are worth getting typings? Hyperswarm, Discovery, Network, Dht and replicator are ones I was thinking of 🤔

this still is a wanted feature!
robertotomas@MacBook-Pro-2 ttt-p2p % yarn add --dev @types/hyperswarm
yarn add v1.22.19
[1/4] 🔍  Resolving packages...
error An unexpected error occurred: "https://registry.yarnpkg.com/@types%2fhyperswarm: Not found".
info If you think this is a bug, please open a bug report with the information provided in "/Users/robertotomas/Public/ttt-p2p/yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/add for documentation about this command.

+1 for typescript support!

guys big thing, wen ?