Spark 1.6.1

Is there a Spark 1.6.1 version of the connector? In the branches I see variations but nothing with the current version. I have modified build.sbt to pick up the versions for Spark 1.6.1 as well as the OrientDB version I am running 2.1.15
build.sbt:
lazy val commonSettings = Seq(
organization := "com.metreta",
version := "v1.6-2.10",
scalaVersion := "2.10.5",
fork:= true,
libraryDependencies ++= Seq(
"com.orientechnologies" % "orientdb-core" % "2.1.15",
"com.orientechnologies" % "orientdb-client" % "2.1.15",
"com.orientechnologies" % "orientdb-jdbc" % "2.1.15",
"com.orientechnologies" % "orientdb-graphdb" % "2.1.15",
"com.orientechnologies" % "orientdb-distributed" % "2.1.15",
"org.scalatest" % "scalatest_2.10" % "2.2.4",
"org.apache.spark" % "spark-core_2.10" % "1.6.1",
"org.apache.spark" % "spark-graphx_2.10" % "1.6.1",
"com.tinkerpop.blueprints" % "blueprints-core" % "2.6.0"
),
externalResolvers := Seq(DefaultMavenRepository)
)
lazy val connector = (project in file("./spark-orientdb-connector")).
settings(commonSettings: _*).
settings(
name := "spark-orientdb-connector"
)
lazy val demos = (project in file("./spark-orientdb-connector-demos")).
settings(commonSettings: _*).
settings(
name := "spark-orientdb-connector-demos"
).
dependsOn(connector)
Yet when tested in spark-shell an error is thrown:
scala>
import org.apache.spark.SparkConf
import org.apache.spark.SparkContext
import com.metreta.spark.orientdb.connector._
val conf = new SparkConf()
Exception Thrown:
error: bad symbolic reference. A signature in package.class refers to term orientechnologies
in package com which is not available.
It may be completely missing from the current classpath, or the version on
the classpath might be incompatible with the version used when compiling package.class.
error: bad symbolic reference. A signature in package.class refers to term orient
in value com.orientechnologies which is not available.
It may be completely missing from the current classpath, or the version on
the classpath might be incompatible with the version used when compiling package.class.
error: bad symbolic reference. A signature in package.class refers to term core
in value com.orient which is not available.
It may be completely missing from the current classpath, or the version on
the classpath might be incompatible with the version used when compiling package.class.
error: bad symbolic reference. A signature in package.class refers to term metadata
in value com.core which is not available.
It may be completely missing from the current classpath, or the version on
the classpath might be incompatible with the version used when compiling package.class.
:30: error: bad symbolic reference. A signature in package.class refers to term schema
in value com.metadata which is not available.
It may be completely missing from the current classpath, or the version on
the classpath might be incompatible with the version used when compiling package.class.
val conf = new SparkConf()

In the next week we expect to release the Spark 1.6.1 version of the connector. stay tuned!

New tags added:

1.6-beta-1-2.10 compiled with scala 2.10 and spark 1.6.0
1.6-beta-1-2.11 compiled with scala 2.11 and spark 1.6.0

check version list


Homebridge installed on Docker

I moved the Homebridge from RPI to Synology Docker.
I changed the IP but it still shows the old IP in widget. I dont know where it gets the old one. In the script its just once...
Is it at least working with Docker ? Homebridge UI is active, but I am not really sure about the required service.
Thanks !

It uses the old one because of the configuration mechanism. You must run the script once with overwritePersistedConfig set to true. This will save the current state. Afterwards when you are done configuring set it back to false.

OK. I read somewhere about it 👍
Thanks for quick response.
Widget looks great !

Is this issue still relevant or can we close it :) ?

Is this issue still relevant or can we close it :) ?


Genre filtering

Description
It seems that stacking genres behaves as "and" rather than "or". Selecting the genres I'm interested in results in nothing found because it's searching for media categorized as action AND comedy AND sci-fi AND etc when I'm trying to filter out what I don't want to see
Desired Behavior
Modify the genre filter in a way that multiple selections don't behave as an "and" operator. Or at least provide the option to change how combining genres works
Additional Context
No response
Code of Conduct

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

That is the intended behavior of genre filter.
Genre filtering is done by the tmdb api. It works the exact way as the filters work on tmdb website:
https://www.themoviedb.org/tv/
https://www.themoviedb.org/movie/
We don't do any filtering logical on the client side.


'imrse' not appearing as a Snippet

Expeected behaviour:
import React, { useState, useEffect } from 'react'
Actual behaviour:
I write imrse and it doenst suggest any Snippet.
(Every other snipper works correctly, so the extension is installed)

Same for me.Other snippets working("rafce,rfce" .etc) but "imrse" doesn't work.

reporting same behavior


Updated instructions for Webpack import

As discussed here:

https://www.html5gamedevs.com/topic/45472-pixi-layers-pixi-is-not-defined/
https://github.com/pixijs/pixi-layers/issues/47 and https://github.com/pixijs/pixi-layers/issues/48

This should no longer be a problem in @pixi/layers.


🐞 [Bug] TFRobot: Can't deploy any vm with more than 1 SSD

What happened?
I tried to deploy a VM with more than 1 SSD, but I noticed that I couldn't assign names to them like other clients, so I kept going and added one more, like the documents mentioned, and I did get an error saying I couldn't add a disk with the same name.

which network/s did you face the problem on?
Dev
Twin ID/s
No response
Node ID/s
No response
Farm ID/s
No response
Contract ID/s
No response
Relevant log output
"vms": [
    {
      "name": "examplevm13",
      "vms_count": 1,
      "node_group": "test",
      "cpu": 3,
      "mem": 0.25,
      "flist": "https://hub.grid.tf/tf-official-apps/base:latest.flist",
      "entry_point": "/sbin/zinit init",
      "root_size": 0,
      "ssh_key": "example1",
      "ssd": [
        {
          "size": 15,
          "mount_point": "/mnt/ssd"
        },
        {
          "size": 25,
          "mount_point": "/mnt/ssd"
        }
      ],
      "env_vars": {
        "user": "user1",
        "pwd": "1234",
        "user": "user1",
        "test": "pass"
      }
    }
  ],

Can you also update the docs

TC2512 - Multiple SSD


added history feature in the new terminal structure

I hope that now it is ok 🙂
Thanks @nitin42 @jcgertig !

Can you upload a gif ? I want to see the working and then I'll be happy to merge this! Thanks again!

Sure!
Check it out!

SUPERB!!! Awesome work!

Can you create one more PR ? For documenting this feature. It will great !

Sure!
If you wait a little bit, I could commit on this PR the documentation 👍

Sure!

What do you think?
README.md
Built-in functionality

History of your commands available pressing key up and key down

Thanks for the changes! I'm currently reviewing your PR 😄  and will merge soon! Great job 👍

I think you're still not in sync with the current HEAD. See this -
You've added showContent(...)

But I've already merged @jcgertig PR and if you look carefully there are some changes -

Same goes for the other parts. There are some conflicts!

OMG, I'm sorry.
Could I resolve conflicts ?

Yup. If that doesn't work, make a new PR by pulling the new changes first 😄

Yeah, it's a better idea.
I'm closing this because it's to confusing now...
Thanks and sorry again.

No worries. This also happens to me when I'm contributing elsewhere. Difficult to watch the changes and then make yours. 👍  Go ahead and make a new PR


[Feature Request] Support config through a unified file

As of now, commit types can be specified in Cargo.toml, which works fine for Rust projects. However, this isn't suitable in non-Rust projects.
I'd like to suggest using a unified config file everywhere, say git-cm.toml or (whatever file type is preferable for parsing) for easy use of git-cm regardless of the project.

I didn't even consider the possibility to use it outside of a Rust context. But yes, this is actually already implemented. I did a complete re-write to implement #12, #7, #3, #19 and #28. It just needs some clean-up that I will do at the start of next week :)

Great! And thank you!

Thank you for such a wonderful project. If we can remove the dependency on the Cargo.toml and have this language agnostic, i can advocate this inside my company.
Thanks.

I've started work on it already. I hope to finish it at the weekend. It took some time since I re-wrote the parser for better error messages as well.

Just a quick update: currently in my finals. After that I will finish this up! I tempered around with the parser again to get better error messages and lost track of time back in December :)


gst::Encoder::Close() has two seconds of sleep() in it - any way to avoid?

gstEncoder::Close() has two sleep(1) calls.  In my application, we might be writing multiple video streams simultaneously, and any of them can close at any time.  The sleep()s cause the whole application to 'sleep' for two seconds each time which is causing problems.
I tried removing the sleep()s, but the written video files are then incomplete and unreadable.
I've modified the sleep()s to be usleep(), and have got them down to 50,000 microseconds, which is an improvement.  But I'm wondering if there is a better way to do this, or to avoid the whole application locking up.  I could use a thread for each write, but is there a simpler way?
Thanks in advance.

Thanks @dusty-nv - I'll see if I can get that set up.  Thanks for confirming it's necessary, anyway.

@dusty-nv it seems that the python GIL is locked during these two seconds, which is potentially breaking multithreaded python code using this API. Could this be solved by adding
Py_BEGIN_ALLOW_THREADS
// do things
Py_END_ALLOW_THREADS

in the python bindings .cpp files to release the GIL? (like here)

Hi @ricknitsche, if you have an application that is exhibiting this issue then I would recommend testing that code change, as I don't personally have a repro case for it here.

Hi @ricknitsche, if you have an application that is exhibiting this issue then I would recommend testing that code change, as I don't personally have a repro case for it here.

It improves performance (a lot), calling these functions in python threads after this change: https://github.com/dusty-nv/jetson-utils/pull/123


Stricter disjunction of interfaces

Suggestion
🔍 Search Terms
List of keywords you searched for before creating this issue. Write them down here so that others can find this suggestion more easily and help provide feedback.
✅ Viability Checklist
My suggestion meets these guidelines:

!!! This can be a breaking change
This wouldn't change the runtime behavior of existing JavaScript code
This could be implemented without emitting different JS based on the types of the expressions
This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
This feature would agree with the rest of TypeScript's Design Goals.

⭐ Suggestion

📃 Motivating Example
Disjunction of interfaces can lead to an error
Here is example 1:
interface A { id: string }
interface B { id: number }

const a: A = {id: "xxxxx"};

function modifyObject(value: A | B){
    value.id = 100; // this is OK
    //    ^? (property) id: string | number
}

modifyObject(a);

console.log(a.id.toUpperCase());
//             ^? A.id: string
// [ERR]: a.id.toUpperCase is not a function

Problem:  Typescript didn't find the error of type mismatch.
Example 2:
interface A { id: string }
interface B { id: number }

const v = {id: "xxxxx"} as A | B;

function modifyObject(value: A | B){
    value.id = 100; // this is OK
    //    ^? (property) id: string | number
}

modifyObject(v);

console.log(v.id.toUpperCase());
//             ^? (property) id: string | number
// TS: Property 'toUpperCase' does not exist on type 'number' --- OK!

Signature of modifyObject is same, but now code is correct.
Example 3:
interface A { id: string, readonly dataType: "A" }
interface B { id: number, readonly dataType: "B" }

const v = {id: "xxxxx", dataType: "A"} as A | B;

function modifyObject(value: A | B){
    value.id = 100; // this is NOT OK, but works
}

modifyObject(v);

console.log(v); // {id: 100, dataType: "A"} // type mismatch

It would be better if interface disjunction handled getters and setters
interface A { id: string, readonly dataType: "A" }
interface B { id: number, readonly dataType: "B" }

const v: A = {id: "xxxxx", dataType: "A"};

function modifyObject(
    value: /* automatically calculates from (A | B) */ {
        get id(): string | number;
        set id(value: string & number);
        get dataType(): "A" | "B";
    }
){
    console.log(value.id) // OK! This is number | string
    value.id = 100; // TS ERROR: 'number' is not assignable to type 'never'.
}

modifyObject(v); // OK

In this form, it will lead to breaking changes, so there can be 2 options:

make this behavior dependent on the flag
make additional type like:

function modifyObject(value: AnyOfType<A | B>){
    value.id = 100; // this is OK
    //    ^? (property) id: string | number
}

but this seems impossible

This isn't specific to unions; objects are treated as covariant in their properties even though that's unsafe when writing to them:
interface Foo {
  y: string;
}

interface Bar extends Foo {
  z: number;
}

const a = { x: { y: "abc", z: 123 } };

function modifyObject(value: { x: Foo }) {
  value.x = { y: "def" };
}

modifyObject(a);
a.x.z.toFixed() // error!

As such this is essentially covered by #9825 and specifically this comment, so we can expect this suggestion to be declined: TypeScript is intentionally unsound here.

There are two separable concerns here. The first is whether properties are treated as covariant for the purposes of assignability and subtyping. The second is whether properties present in all constituents of a union appear with a different write type than read type. That second one is more addressable in my opinion because we have already done that for indexed access types. The question is just whether it's worth it or not; it doesn't really move the status quo that you need to be somewhat careful in TypeScript when doing property writes in all cases

FWIW I feel like if foo["prop"] is going to have different read and write types (per #30769, yes I still remember the issue number… 😅) then that should apply equally to foo.prop.


Trace ID must be UUID

As a standard at LS trace-id should have UUID format

seem fine, but you do loose the nice side benefit of being able to tell when the original request entered the system. The timestamp was handy for that. Also, once you hit rails 4, it should be generating a request-id (Imprint::Tracer::RAILS_REQUEST_ID) so the older Imprint::Tracer.rand_trace_id will not be used as apps get upgraded.
Anyways +1 should work fine. I would add a version bump.


Add Nong Song

Song Name
ta1lsd003
Artist Name
ta1lsd0ll
Source
Youtube
Youtube Link
https://www.youtube.com/watch?v=CljzyH_c3Uo&t=0s
Direct File Link
https://www.kapwing.com/videos/66b42a8ad38f92fb9508c2a2
Song ID
1326366
Start Offset [ms]
0
Extra
No response

accept

"accept"

accept


Fix for namespace linking and unlinking issues

This should resolve issue #181 which highlighted problems with linking or unlinking Class/SuperClass/Object Name/C++ Class namespaces to each other.
Some minor formatting (4 space) corrections mixed in.

My take on this is that it fixes more than it breaks.  It removes unneeded comments and fixes a number of spacing issues.  More importantly, Mike claims that it fixes issues #181 which is a fairly serious bug.


HDDS-6461. Update Ratis version to 2.3.0-da5d868-SNAPSHOT.

What changes were proposed in this pull request?
Update the Ratis SNAPSHOT version.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-6461
How was this patch tested?
Existing tests.

@captainzmc  Can you review and merge this patch?


Various VGC 19 things

Just keeping these all here because I get random messages telling me to tell you, so here we go:

Zygarde-Complete and Zygarde-10% don't have correct base HP
Thousand Arrows and Thousand Waves don't exist
People want a Geomancy button (probably can replace Kommo-o)

Modifiers to physical Photon Geyser aren't being applied at all; e.g. Choice Band Photon Geyser from Dusk Mane should be dealing increased damage, and burned Dusk Mane should be dealing less damage with Photon Geyser (ofc, Choice Band, burn, etc. do not affect whether or not Photon Geyser is physical/special; it's only the +6 to -6 boosts and the raw stat).

lol turns out that alternate forms don't change the HP stat, for some reason. (I knew I put them in there correctly...) I'll fix that asap and then do the other things when I get a chance

Also, Bulbapedia says "a Flying-type Pokémon not already grounded will receive neutral damage from Thousand Arrows, regardless of its secondary type", do you know if this is true? I assume it is just seems a little strange and pretty relevant.

Yes, it is true (you can see in Mechanics Monday at this timestamp here: https://youtu.be/x4Xgc3JNUz0?t=13m40s). I show that Zapdos and Ledian take neutral damage on the first hit.

Sweet, thanks!


feat: add base exception classes

Added a couple of base exception classes so it's easier to catch multiple exceptions if and when needed.

:tada: This PR is included in version 1.4.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[RFE] Enable composer to point to existing parent commit

Users should be able to generate new Edge images, as well as image updates which require composer to point to a "parent commit" during image creation
I believe it would require a change here: https://github.com/redhat-cop/rhel-edge-automation-arch/blob/2fb1d30ab6886a593069350b21394944b371b759/ansible/roles/build-rpm-ostree/tasks/main.yaml#L54
The idea would be to only pass the variables {{ ref }} and {{ parent }} if needed

CLi procedure would be something like: composer-cli compose start-ostree --parent faf281d82fa6c9bd0671511a6043f6fcf8a592cae265a3d27cb9f8379cb5dd98 edge rhel-edge-commit


[TODO] Implement timers for system monitoring

Need something like the voxblox timing util to measure performance of the system

Done


Add functionality to enclosures.py

Hi all,
I have an existing repository that is useful for adding multiple enclosures, creating SPTs, and then using these SPTs to create SPs.  The process, for the most part, is automated from end-to-end.
I would, however, like to see about being incorporated in the python-hpOneView SDK.  I have modified enclosures.py to also allow the addition of multiple enclosures and would like to include it.
Is this something that would be of interest to you?
Katherine

Yes.  Please submit a PR with the changes and we'll review.

Can someone tell me the process for creating a PR?  I cloned the repository and created a new branch but when I try a push, I'm prompted for my GitHub credentials. I have SSH keys on GitHub for the Canareez project, but it is not in the HewlettPackard organization. Does someone need to give me additional privileges for the HewlettPackard organization?

Hi @elliottkat
To submit PRs you need to fork this repo, clone the fork on your machine, push the changes, and then open a PR here.
This way you won't need any additional prvileges to do so.
I hope it helps! :smile:

Thanks, Thiago, I'll give that a try.

Hi @elliottkat - any update on it?


Fixed incorrect subdivision code

Looks like the docs have a typo for the Yukon Territory (YU should be YT). In the code this seems to be correct.
https://en.wikipedia.org/wiki/ISO_3166-2:CA

Coverage increased (+0.01%) to 99.577% when pulling ef285d72b25df205cfd0a1e2c0ff0fc29512abfc on MichaelThessel:master into a91c306e4c54106b5caeaffdd724dc3305451bba on dr-prodigy:beta.

Thank you!


Create from raw pixel data or older versions of Jimp

Allows creation of Jimp instances from older versions of the module (where !img instanceof Jimp) and from object with raw RGB/RGBA data. Closes #200
new Buffer(...) is deprecated. It may either show warnings in console or change it's behaviour to zerofilling. Discussion on what to do was going for some time at nodejs/node. "safe-buffer" module is used for backwards compatibility with node <= 0.12 (this extra dependency could be avoided, but calls to Buffer.allocUnsafe and Buffer.from looks nicer with it. Hopefully it's not a problem)

Hmm... Could additionally check if data is an instance of Uint8ClampedArray. This way it will also support ImageData objects returned by CanvasRenderingContext2D#getImageData.
But that needs to come with the check that Uint8ClampedArray is supported, because in IE10 it isn't.

@Iwasawafag wanna update this?

closing in favor of #514


Add permissions to django mutations

Django mutations couldn't handle permissions, but adding the keyword argument permission_classes to each django mutation allow the permissions to work as it already inherits the functionality from the StrawberryField class, it just needed to be wired

@wellzenon, this is great addition. Could you please add a test case for this too?
I apologize for the delay in my response.

Tests added


CLI for heron executor

Heron executor carries a list of arguments, need to convert it to use CLI.

fixed via https://github.com/twitter/heron/pull/1493


what it Do

This project is a third party NetEase Cloud Music client. Issues is not the place for questioning, please use Discussions instead.
Please close this issue.


add a table_info.csv file in extractedData

at the moment pubget outputs some data for each table: a csv file with the content of the table, and a json file with keys like table id, table label, table caption, and the path of the csv file
those are a bit hard to discover & use because they are stored in the article's individual directories, eg in
query_0dbec5035dc626ca916d0317b7b9a76f/articles/0b8/pmcid_7664275/tables/
it would be helpful to create a csv in the extractedData directory grouping the info for all tables in all articles. columns could be
pmcid,table_number,table_id,table_label,table_caption,n_header_rows,table_data_file
table_data_file would be the path to the table's csv, probably relative to the root of the query directory.
the other keys would be those currently provided in the JSON

also maybe extract the content of the table-wrap-foot elements? those are footnotes so not sure if they contain useful information

This would be great!

also maybe extract the content of the table-wrap-foot elements? those are footnotes so not sure if they contain useful information

this part is done in #40


Unexpected token function at getInitialProps

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

Current Behavior

Unexpected token function G:\huxinmin.github.io\nextJs\.next\dist\pages\index.js:65     value: async function getInitialProps() {                  ^^^^^^^^ SyntaxError: Unexpected token function     at Object.exports.runInThisContext (vm.js:76:16)     at Module._compile (module.js:542:28)     at Module._compile (G:\huxinmin.github.io\nextJs\node_modules\_source-map-support@0.4.15@source-map-support\source-map-support.js:483:25)     at Object.Module._extensions..js (module.js:579:10)     at Module.load (module.js:487:32)     at tryModuleLoad (module.js:446:12)     at Function.Module._load (module.js:438:3)     at Module.require (module.js:497:17)     at require (internal/module.js:20:19)     at _callee$ (G:\huxinmin.github.io\nextJs\node_modules\_next@3.2.2@next\dist\server\require.js:33:46)
Steps to Reproduce (for bugs)

clone data-fetch example and npm install and npm run dev
see <IP_ADDRESS>:3000 in browser
the package.json is { "name": "nextJs", "version": "1.0.0", "description": "", "main": "index.js", "scripts": {  "test": "echo \"Error: no test specified\" && exit 1",  "dev": "next",  "build": "next build",  "start": "next start" }, "keywords": [], "author": "", "license": "ISC", "dependencies": {  "isomorphic-fetch": "^2.2.1",  "next": "latest",  "react": "^15.4.2",  "react-dom": "^15.4.2" } }

Context

I did all same as the example told but can not successful , I was very sad and ask for help
Your Environment

Tech
Version

next
3.2.2

node
6.10. 0

OS
windows

browser
chrome

etc

I guess you have created a .babelrc and forgot to include next/babel preset 👍


Parent is required when editing in Netlify CMS

Describe the bug
When trying to publish a page with top-level navigation, the parent field is required when publishing via Netlify CMS, preventing it from getting published/saved.
To Reproduce
Steps to reproduce the behavior:

While logged into the Netlify CMS, edit a top-level page or create a new page that will not have a parent.
Click on the 'Publish' button in the top-right corner.
See input validation that a parent is required, preventing the page from being published.

Expected behavior
The parent field should be optional. When no parent is provided, the eleventyNavigation parent property should not be present at all in the .md file (this is the case with existing top-level pages like "Getting Started").
Screenshots

I can attempt a fix and create a PR if this is not a priority since it is a big hassle with some of my current projects. Thank you for your work!

Awesome thanks for catching -- this should new be fixed in main branch, feel free to re-open if not.


Helm: ingress class name

Description
This PR inserts ingressClassName into the auth ingress resource created by helm. The old annotation has been retained for backward compatibility.
Fixes #(issue)
How Has This Been Tested?

@cheina97 ok about retaining compatibility, but we're in a phase in which we still can make breaking changes. Please consider setting a deadline, e.g. 2 versions from now, in which the backward compatibility is removed.

@frisso I talked about it with @aleoli and we agree with you. I removed the ingress annotation from the helm file (it's still possible to add it through the auth.ingress.annotations value)

/rebase test=true

/merge


How to use namespace?

Hi,
your work is excellent. I want to move all the global variables that you use (w2ui,.w2....) under of specific namespace. In the projects that I am participating, it is not permitting to use global variables without a specific namespace. Can you guide me, how to do that  ?
BR/Antonis

With es6 modules file, now there is no globals at all. You have to import them. See /dist/w2ui.es6.js. For backward compatibility, it will register globals as it was.


Windows Terminal hangs up and disappears on window resizing.

Environment
Windows build number: 10.0.19042.0
Windows Terminal version (if applicable): 1.6.10571.0

Any other software?

vim 8.2

Steps to reproduce

vim aiueo.txt
( aiueo.txt is text file including wide characters. )

Narrowing the window width by mouse.

Expected behavior

The window is resized normally.
Actual behavior

Windows Terminal hangs up (few seconds) and disappears.
( the condition seems to be a line wrapping of wide characters. )

Well, this could be any of:

#8730
#8596
#8287
#4907

Could you send us a Feedback Hub trace of the crash, to help identify the root cause?
/feedback

I sent feedback.
https://aka.ms/AAbm3go

@DHowett Is there anyone on the team that can actually still access feedback hub links? Or are we all locked out now 😑

If I had to simply guess, this is the PrepareForDoubleByteSequence resize crash we know about. @zadjii-msft hit me up on Teams -- there's a way :smile:

 "providerName": "Microsoft.Windows.Terminal.Control",
 "lineNumber": "286",
 "fileName": "E:\\BA\\3\\s\\src\\buffer\\out\\textBuffer.cpp",
 "hresult": "2147549183",

Yep, it was _PrepareForDoubleByteSequence, which is #7417, which is #4907
/dup #4907

I understood this issue is a duplicate of #4907.
But, I will try to investigate a little by myself.

type aiueo_unicode.txt → resize → no problem

vim aiueo_unicode.txt → resize → problem occurs

So, there might be a display conflict with vim and windows terminal.
--
file: aiueo_unicode.txt

I set up VS2019 and checked the call stack.

vim aiueo_unicode.txt

src/buffer/out/textBuffer.cpp
  TextBuffer::Reflow
      ↓
  newBuffer.InsertCharacter
      ↓
  _PrepareForDoubleByteSequence
      ↓
  _AssertValidDoubleByteSequence
    // N, T failing case (uncorrectable) or
    // L, N and L, L failing cases (correctable) or
    // T, T failing case (uncorrectable)
    return false;
      ↓
  _PrepareForDoubleByteSequence
    FAIL_FAST_IF → Unhandled exception

I made a quick fix.
diff -u -r src_orig/buffer/out/textBuffer.cpp src/buffer/out/textBuffer.cpp
--- src_orig/buffer/out/textBuffer.cpp	2021-04-11 20:24:10.253074800 +0900
+++ src/buffer/out/textBuffer.cpp	2021-04-12 22:05:31.059636600 +0900
@@ -2192,6 +2192,12 @@
                 const auto dbcsAttr = row.GetCharRow().DbcsAttrAt(iOldCol);
                 const auto textAttr = row.GetAttrRow().GetAttrByColumn(iOldCol);

+                // workaround for https://github.com/microsoft/terminal/issues/9547
+                if (!newBuffer._AssertValidDoubleByteSequence(dbcsAttr))
+                {
+                    break;
+                }
+
                 if (!newBuffer.InsertCharacter(glyph, dbcsAttr, textAttr))
                 {
                     hr = E_OUTOFMEMORY;

There seems to be multiple error handling methods.

hr = E_OUTOFMEMORY; break;
CATCH_RETURN();
FAIL_FAST_IF();

But, no method seems to be useful for this issue ...

Sorry -- we actually have a fix for this one coming back in from the Windows OS code. I haven't merged it into this repository yet!

Thanks for the reminder, though -- i need to do that before we ship 1.7 (!)


Issue 415: [Notebook][Bug] Oracle to spanner notebook not support composite PK tables for migration

This PR, fixes notebook functionality for migrating tables with composite PK
Closes #415

@surjits254 please resolve conflicts and merge the same.


Add redirectUri option to requestSignInLink hook

Description of changes:
This is a small non-breaking change to help with developers who want to implement a custom login form. It adds the ability to supply a redirectUri along with an email address to the requestSignInLink hook. That way developers can allow deep linking into their applications when the user clicks the magic link to verify, rather than the default behavior of re-directing them back to the login page (or wherever the hook was called).
e.g.
const { requestSignInLink } = usePasswordless()
//some code
<EMAIL_ADDRESS>"https://example.com/article/45/")

I also fixed a small error in the documentation where it suggested you call the requestSignInLink with requestSignInLink({username: event.currentTarget.username.value}), while the hook actually expects a string

Great idea!
I don't like the inconsistency of the requestSignInLink interface that it doesn't take an object but positional arguments (our fault, we started that). I do appreciate you're trying to prevent a breaking change. I'm on the fence now, and thinking maybe we should take the breaking change, there is a reason we are just at v0.6.7 for this lib (and with the breaking change it would become v0.7.0).
Could you run prettier on your code please?

Thanks for the quick reply! I agree with you that I'd prefer the object as well, not just for consistency but also in case you need to add additional parameters in the future. Let me know what you decide, I'd be happy to change it to an object.

OK let's do it, let's make an object out of it. In this early stage of this lib, let's favor elegance over preventing breaking changes.
There is also another breaking change I want to do, so we can combine them in one release.

Released in v0.7.0 https://github.com/aws-samples/amazon-cognito-passwordless-auth/releases/tag/v0.7.0


Screenshot test failing due to some unresolvable deps

What went wrong:
Execution failed for task ':app:dataBindingMergeDependencyArtifactsOffPlaystoreScreenshots'.

Could not resolve all files for configuration ':app:offPlaystoreScreenshotsCompileClasspath'.
Could not resolve com.github.jkwiecien:EasyImage:1.4.0.
Required by:
project :app
Skipped due to earlier error
Could not resolve com.github.chrisbanes:PhotoView:2.3.0.
Required by:
project :app
Skipped due to earlier error
Could not resolve com.github.mreram:showcaseview:1.0.5.
Required by:
project :app
Skipped due to earlier error

We could take advantage to upgrade EasyImage ?
https://github.com/jkwiecien/EasyImage/releases?after=1.4.0


Provide more plugins like lookAt by @threlte/extras

Wouldn't it be better to provide plug-ins like lookAt to @threlte/extras like interactivity?

Absolutely! Feel free to make a PR for anything that you like to see 👍
I'll close this issue as it's not specific enough. If you're thinking about a specific plugin you would like to see, feel free to join our Discord and we'll discuss there or open a separate issue.


Depreciated NPM Modules

Fresh Install of sage and running into some depreciated modules:
npm WARN deprecated<EMAIL_ADDRESS>Please use gulp-cssnano instead.
npm WARN deprecated<EMAIL_ADDRESS><EMAIL_ADDRESS>is no longer maintained. Upgrade to<EMAIL_ADDRESS>npm WARN deprecated<EMAIL_ADDRESS>this package has been reintegrated into npm and is now out of date with respect to npm

Only gulp-minify-css is a Sage dependency. Since we're redoing our asset pipeline in Sage 9, it shouldn't be an issue to swap it out. Thanks for the heads up.
I don't think we can do anything about the other two. Feel free to report upstream if you want.

👍


⚠ Bofrost.at has degraded performance

In 1e5ab52, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1353 ms

Resolved: Bofrost.at performance has improved in 5d538f6 after 6 minutes.


Production Deployment

[x] Use nginx as web server
[x] gunicorn as gateway
[x] use django-sendfile to send file through nginx. However, the path is absolute. This can be only tested in actual deployment (container) setup
[x] container setup to build and run the server

Using two configurations separately for dev and for deployment


Middleware support?

Hey thanks for creating this :-)
I was wondering if it makes sense in the scope of this project for the livereload server to implement an http.Handler interface?
It would allow less port binding and would be a bit easier to integrate into existing dev servers.

@matthewmueller Great idea. I'll update in the next week or so.

Well I'm only a year late, but I had a chance to take another look at this and decided this feature would be beyond the scope of the project. I'll maintain this repo as a simple, solid, standalone server, but probably won't be adding any features in the future.

Haha fair enough! Funnily enough, I looked at this again last week and created a version that does implement it as middleware. It's heavily influenced by your work, so thanks for that!
I need to clean it up, but I'll post a link here once it's public to let others choose.

Awesome! Yes please keep me posted, I'll be sure to link to your project in the readme.

@matthewmueller  would be awesome if you've got a live reload implementation that you are ready to post long with the lrserver one!

Haha, the thread that keeps getting forgotten and revived 😁
Here's the code: go-livereload-master.zip
It is working. It's just untested and undocumented. Example usage:
middleware := livereload.Watch("*.js")
router := some.Router()
handler := middleware(router)
http.ListenAndServe(":9000", handler)

Sorry for the bare minimum. I didn't make progress since the last message and I probably won't for the foreseeable future. Maybe this is better than nothing? 🤷♂


Turning segfaults into exceptions with high probability

The IOHK devops team has complained that incorrect usage of this library can result in hard crashes (segmentation faults) that can be hard to debug.
The changes in this PR turn most of the errors that arise from incorrect usage (such as calling close twice) from segmentation faults into Haskell exceptions with a helpful error message.
Further:

I've added a bunch of tests that show how the various ways to open and close DBs fail
I've added lots of Haddocks to the open*/close functions so that callers are aware how they fail
I have added HasCallStack constraints to many functions that can fail (such as close), so that when an exception arises, users can easily see where it close was used incorrectly
I've added withDB as a simple bracket-style function; ideally most users could use that and never call close manually, like for example:

main = do
  ...
  withDB path options $ \db -> do
    ... all program code, never calling `close` ...

Is this still being worked on?

Is this still being worked on?

@CMCDragonkai Not by us currently; FP Complete are no longer for contracting for IOHK.
We also made https://github.com/chrisdone/rocksdb-haskell-ng as a from-scratch, safe-from-the-beginning replacement for rocksdb-haskell for IOHK after this PR.
If you need this issue fixed, would benefit from rocksdb-haskell-ng, or any other Haskell RocksDB help, let us know!

@nh2 Why is that repo read-only/archived? Is there an active repo where this stuff is still being developed.

Why is that repo read-only/archived?
Is there an active repo where this stuff is still being developed.

@CMCDragonkai FP Complete made rocksdb-haskell-ng for IOHK to fix the mentioned issues. It seems IOHK is still using their fork that's a couple commits behind our archived repo (see here, and the list of forks), but we do not currently have any clients that want us to maintain it so we archived it.
We'd certainly be happy to revive it or improve it for you if that'd be useful for you. Just let me know!


ArgumentException boundary

I am using the SpeakerRecognition API with the Nuget package Microsoft.ProjectOxford.SpeakerRecognition.
When I call the EnrollAsync method, passing a Stream with data from a wav file and a GUID of a user, I get an ArgumentException boundary. I use Xamarin.Android (Target Framework 7.0). This is from the Application Output:
[mono-rt] [ERROR] FATAL UNHANDLED EXCEPTION: System.ArgumentException: boundary
With no more further details.
The code I use is:
await client.EnrollAsync(stream, profile);
Where stream is a MemoryStream with the file data and profile the GUID of my profile. The wav file that I generate is valid and according to the guidelines of the documentation.

I've figured out how to solve this. You shold add this method to your code:
public async Task<Enrollment> EnrollAsyncRedone( Stream audioStream, Guid id ) {
            try {
                // Initialization and preparation stuff
                string _BASE_URI = "https://api.projectoxford.ai/spid/v1.0/verificationProfiles";
                string _JSON_HEADER = "application/json";
                string _OCP_SUBSCRIPTION_KEY_HEADER = "Ocp-Apim-Subscription-Key";
                HttpClient _DefaultHttpClient = new HttpClient();
                _DefaultHttpClient.DefaultRequestHeaders.Accept.Add( new MediaTypeWithQualityHeaderValue( _JSON_HEADER ) );
                // Don't forget to change "yourAPIkeyString" to your actual API key!
                _DefaultHttpClient.DefaultRequestHeaders.Add( _OCP_SUBSCRIPTION_KEY_HEADER, yourAPIkeyString );

                // This line caused the exception:
                //var content = new MultipartFormDataContent( "Upload----" + DateTime.Now.ToString( "u" ) );
                // Now it is changed to this (no arguments in braces => correct boundaries generation)
                var content = new MultipartFormDataContent(  );
                content.Add( new StreamContent( audioStream ), "enrollmentData", id.ToString( "D" ) + "_" + DateTime.Now.ToString( "u" ) );

                var requestUrl = _BASE_URI + "/" + id.ToString( "D" ) + "/enroll";
                var result = await _DefaultHttpClient.PostAsync( requestUrl, content ).ConfigureAwait( false );
                string resultStr = await result.Content.ReadAsStringAsync().ConfigureAwait( false );

                if (result.StatusCode == HttpStatusCode.OK) {
                    //parse response
                    Enrollment response = JsonConvert.DeserializeObject<Enrollment>( resultStr );
                    return response;
                } else {
                    throw new EnrollmentException( resultStr );
                }
            } catch (TaskCanceledException exception) {
                throw new TimeoutException( "Connection timed out: " + exception.Message );
            }
        }

And then you should call EnrollAsyncRedone(stream, profile); instead of client.EnrollAsync(stream, profile); - so the error will disappear. I've got EnrollAsync's source code from here: https://github.com/Microsoft/ProjectOxford-ClientSDK/blob/master/SpeakerRecognition/Windows/ClientLibrary/SpeakerVerificationServiceClient.cs


Show data in person's profile left bound

As editor I want to look at a user's profile and see all data left bound.
That includes

picture / avatar
name
mail
Telegram
phone
free text for extra info


Replace REST cache with Azure Cache2

It seems Azure Cache tasks are out of beta, starting from v2:

Announcement: https://azure.microsoft.com/en-us/updates/new-deployment-strategies-and-caching-for-azure-pipelines/
https://github.com/microsoft/azure-pipelines-tasks/blob/c9c4827c8d0951c98a9468d83db708c593c7c428/Tasks/CacheV2/task.json
https://docs.microsoft.com/en-us/azure/devops/pipelines/caching/?view=azure-devops#using-the-cache-task

This PR updates the template to use them, plus updates esy to 0.5.8.

@jordwalke @ulrikstrid this seems to be ready. I fixed a related issue where esy install was being called before restoring the cache.
Build times are
Linux: 1m 57s
macOS: 1m 52s
Win: 4m 18s
(don't take last build into account as I removed a string from the cache key string which invalidated it).

@jordwalke I tried to re-add the import- / export-dependencies commands.
I am getting two sorts of errors:

on esy import-dependencies, I get some errors of this kind

esy: internal error, uncaught exception:
     Unix.Unix_error(Unix.EACCES, "rename", "C:\\Users\\VssAdministrator\\.esy\\3_\\s\\reason_native__s__pastel-0.2.1-0d140e57")
     Raised at file "src/core/lwt.ml", line 2998, characters 20-29
     Called from file "src/unix/lwt_main.ml", line 26, characters 8-18
     Called from file "esy-lib/Cli.re", line 263, characters 9-28
     Called from file "cmdliner_term.ml", line 25, characters 19-24
     Called from file "cmdliner.ml", line 25, characters 27-34
     Called from file "cmdliner.ml", line 116, characters 32-39

They happen with different libraries: rely, pastel, console in particular.

Then on esy build I am getting errors like:

info esy build 0.5.8 (using package.json)
info building ocaml@4.8.1000@d41d8cd9
error: build failed with exit code: 1
  build log:
    # esy-build-package: building<EMAIL_ADDRESS>    # esy-build-package: pwd: C:\Users\VssAdministrator\.esy\3_\b\ocaml-4.8.1000-a241290c
    esy-build-package: delete directory
                       C:\Users\VssAdministrator\.esy\3_\s\ocaml-4.8.1000-a241290c:
                       C:\Users\VssAdministrator\.esy\3_\s\ocaml-4.8.1000-a241290c\lib\ocaml:
                       Directory not empty

The latest build includes both cases.
Something I realized is that the ocamlfind cleanup is being done after calling esy import-dependencies in master branch, which is confusing as it's supposed to remove ocamlfind before calling that command? 🤔

It's supposed to remove ocamlfind after importing. There's a problem with importing ocamlfind that my (now merged) PR fixes. We might want to see if esy nightlies fix it (https://www.npmjs.com/package/@esy-nightly/esy) now that my PR is merged.


Potenial bug with Txs spanning > 2 shares

Hey guys, still seeing this:
celestia-appd q tx 4AB987292DA8AEFDA3FA01B6FC6C8C6DB7AB741CE5A24470C07DBBB87B8A96FF --node https://rpc-mamaki.pops.one

Error: RPC error -32603 - Internal error: transaction spanned more than two shares, this is not yet supported

On:
celestia-appd version
0.5.5-beta-1

I was unable to recreate this bug query my nodes on v0.5.5-beta-1 or v0.5.4. Is there a chance that the node behind https://rpc-mamaki.pops.one is still using v0.5.3 or below?

ah youre right, that one hasnt been updated although i thought it was, will close this and update our rpc node


Updated error message section

This proposed change will ease the people to directly copy/paste the 'export' command from the error message to their terminal.

Thanks!


Pipenv Support

It is more organized than using a custom directory called requirements/ and a file called .python_version, also it provides virtual env functionality.

Why don't you add this feature for all of us? Love to merge you PR here :)

ps: .python-version is pyenv related file. not my invention :)

I'll try.

I got a ~working (just tested in one case)~ demo but i dont think i can adapt my self to this repo's build system. I think it would be better to convert pipenv by someone who knows what is what.
The demo


Language request: Nix

What programming language should we add?
Nix the expression language.
What is the official website for the language?
https://nixos.org/nix/manual/#ch-expression-language
https://nixos.wiki/wiki/Nix_Expression_Language
Is this a language that comes in many variants? If so, which variant should we support?
No.
Does the language have an official logo? If so
Yes.

please include a link to an example

https://github.com/NixOS/nixos-artwork/blob/master/logo/nix-snowflake.svg

please list the attribution rights of that logo

The logo is made available under a CC-BY license.

https://github.com/NixOS/nixos-artwork/tree/master/logo#license
https://github.com/NixOS/nixos-artwork/tree/master/logo#credits

Does the language have an unofficial logo? If so
No.

please explain the ways in which people use the unofficial logo
please include a link to an example
please list the attribution rights of that logo

Is there a testing framework available for the language?
Yes. See https://nixos.org/nixos/manual/index.html#sec-nixos-tests.
Is this language listed as 'supported' by Prism.js? If so
Yes.

what is Prism's canonical spelling for the language?

Nix

what file extensions should be highlighted?

nix

If it is not supported by Prism.js, what is the closest supported language it maps to?
Who will be leading the effort to launch the track?
I don't know this yet. I was sort of going from section to section at https://github.com/exercism/request-new-language-track. This issue seemed a good place to start.

Sweet! As soon as there is at least one person committed to launching the track, please report back here, and I will bootstrap a repository for you.
Thanks!

I would like to try leading the effort to launching the track, or committing to launching the track.

@bradyt Thank you ❤ !
I've created a repository for you here:
https://github.com/exercism/nix/issues/1
Please submit individual pull requests with cohesive changes (e.g. documentation, CI setup, logo, individual exercises) to the new repo. I've given you commit access, so you can create branches right in the repo rather than creating a fork. Individual, smaller PRs will give us a few benefits:

We can help catch things that should have been documented but weren't
Ongoing development will be visible/transparent to others who are interested in following along
It will be easier to go back in the git logs to figure things out

Welcome :heart: :heart: :heart:


error when use K to show hover doc

Version confirmation

[X] Confirm

Following prerequisites

[X] Confirm

Neovim version
NVIM v0.9.0
Operating system/version
archlinux
Terminal name/version
WezTerm
$TERM environment variable
No response
Branch info
main (Default/Latest)
Fetch Preferences
SSH (use_ssh = true)
How to reproduce the issue
press K
Expected behavior
toggle hover doc
Actual behavior

Additional information
No response

what's the language u r using?
it seems like related to treesitter, so i guess maybe sth is wrong with the treesitter parser? like an invalid installation of the parser?

what's the language u r using?
it seems like related to treesitter, so i guess maybe sth is wrong with the treesitter parser? like an invalid installation of the parser?

python, so how can i fix it?
thx

:checkhealth nvim-treesitter

Here is the output.
Btw, I tried to use K to get doc after that,  there is  no error now.

so is it gd to close the issue?


Automator: update proxy@master in istio/istio@master

Generated by Automator - 2021-04-24T01:39:31+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-k8s-tests_istio
9ba967c472323078c6478a5b9be1de7a1f39ff65
link
/test integ-pilot-k8s-tests_istio

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

@istio-testing: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-k8s-tests_istio
9ba967c472323078c6478a5b9be1de7a1f39ff65
link
/test integ-pilot-k8s-tests_istio

integ-security-k8s-tests_istio
77feb62b456bfdb891a06fe365d1a1814f7382f9
link
/test integ-security-k8s-tests_istio

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

@istio-testing: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
77feb62b456bfdb891a06fe365d1a1814f7382f9
link
/test integ-security-k8s-tests_istio

integ-security-multicluster-tests_istio
77feb62b456bfdb891a06fe365d1a1814f7382f9
link
/test integ-security-multicluster-tests_istio

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

/retest

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-multicluster-tests_istio
77feb62b456bfdb891a06fe365d1a1814f7382f9
link
/test integ-security-multicluster-tests_istio

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

/retest

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-multicluster-tests_istio
77feb62b456bfdb891a06fe365d1a1814f7382f9
link
/test integ-security-multicluster-tests_istio

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

Does appear the same test failed twice: TestRequestAuthentication/jwt-authn/a#04/to_vm/from_primary-0/valid-token

Looks like the 2 commits in istio/proxy are https://github.com/istio/proxy/commit/f742870028f57a33f735ca68954ec366933f0c2e and https://github.com/istio/proxy/commit/bb838e04e788d0e34fc56efb300f44beaaa9ba51. Maybe something in the incoming Envoy change @istio/wg-networking-maintainers-data-plane

@istio-testing: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-multicluster-tests_istio
77feb62b456bfdb891a06fe365d1a1814f7382f9
link
/test integ-security-multicluster-tests_istio

unit-tests_istio
bbdb7289982448088bee55e65c2a7c8ac5c85cea
link
/test unit-tests_istio

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

That is a known flake: https://github.com/istio/istio/issues/32392

/retest

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
bbdb7289982448088bee55e65c2a7c8ac5c85cea
link
/test unit-tests_istio

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

/retest


Kafka manager outdated. Replacing with CMAK.

Rebuild CMAK to replace old Kafka Manager

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun


rx.dialog.close do not trigger on button with variable color scheme

Describe the bug
Dialog does not close when button in rx.dialog.close component has color_scheme set to a state variable.
To Reproduce

Create a dialog with button triggering close,
Set color_scheme of the button to state variable.
Try to close the dialog using the button.

Minimal example:
import reflex as rx

class State(rx.State):
    custom_color: str = "red"

def index() -> rx.Component:
    return rx.container(
rx.dialog.root(
    rx.dialog.trigger(rx.button("Dialog", size="4")),
    rx.dialog.content(
            rx.dialog.close(
                rx.button(
                    "Cancel",
                    color_scheme=State.custom_color,
                    variant="soft",
                ),
            ),
    ),
)
    )

app = rx.App()
app.add_page(index)

Expected behavior
Dialog is closed correctly.
Specifics (please complete the following information):

Python Version: 3.11.9
Reflex Version: 0.5.5
OS: Ubuntu

Wrapping with rx.box solves the issue, thanks for this workaround.
Great to hear it will be fixed!

i have same ISSUE in my environment and not work wrapping with rx.box trick.
Test Environment

Python Version: 3.12.9
Reflex Version: 0.5.9
OS: Ubuntu


V4 Release with V in uppercase

Hi, in the v4 release the letter v is in uppercase, which prevents Packagist to recognise it as a new version therefore unable to being used.

🤦♀


Home Page Twitter & Facebook Latest Posts

Home page could do with a feed of twitter and facebook.
I'll tackle this.

Added this in 7edbd87, this is how it looks currently

👍


Map bridges as areas

After we imported sidewalks in #6 (in addition to manual sidewalk mapping in some cases), many highway overpasses ended up looking disjointed. To head off some misunderstanding from newer mappers who are unaware of OpenStreetMap’s physical separation rule and may not appreciate the need for sidewalks as separate ways, we can visually collect the lanes and sidewalks together by mapping bridges as areas. This is also an opportunity to correct any missing or mistagged crosswalks in the vicinity of these overpasses, which are locally significant for pedestrian routing.

Before
After

While we’re at it, the following public domain data sources provide the names, standard IDs, and a variety of other information about the bridges that we can add to OSM:

State Highway Bridges (Caltrans, also available as a table in PDF format)
Local Bridges (Caltrans)
Roadway Bridges (San José Department of Transportation)

Bridge names and IDs are verifiable on ground. In California, every bridge has this information on a small sign or stamped onto a jersey barrier or column. However, in some cases, it has been painted over due to graffiti, and it’s usually in a place that’s hard to spot from Mapillary or Bing Streetside imagery. Highway overpass names typically don’t appear on maps, but that can be a distinguishing characteristic of OSM’s South Bay coverage.
The datasets above represent bridges as point features, so these bridge areas have to be mapped by hand. Since it is a completely manual affair, it isn’t necessary to follow the standard import proposal process, though we should cite Caltrans and the City of San José as sources.

Coverage of bridge areas in the Bay Area is now concentrated in the South Bay, now that bridge areas have been mapped along several major freeways and streams:

Roadway Bridges (San José Department of Transportation)

In addition to this Roadway Bridges layer, San José DOT also publishes a separate Bridges layer. The latter appears to include more attributes that aren’t as interesting to us, but otherwise I don’t know if there are any other relevant differences.


Property Encryption Policy

Some properties like password highly recommended to store in encrypted format. does Cfg4j support this feature?

Hmm... 10 months later I find this answered question when asking it myself.  I guess not then?

This would be great!


Why use split here

https://github.com/docopt/docopt/blob/master/examples/git/git.py#L48
Why not just use a list to start with?

Because that's less to type. Compare:
'checkout clone commit push remote'.split()
['checkout', 'clone', 'commit', 'push', 'remote']

Also, easier to read, I think.

I want to add that I'm glad that you asked this questions, and that everyone who has more questions about the code are welcome to ask them here.


Add directory & configFile options to SwiftLint

Hey Ash!
So this is a workaround for #4. This shouldn't break any current configurations, as I've added default arguments. You can see a working copy of this one at Harvey#11.

P.S no biggie @ashfurrow , but I think you forgot to turn on "Build forked pull requests", since CircleCI didn't actually CI this one :)

Good catch, I have re-delivered the webhook event for the last commit in this PR to re-trigger the build.

Okay, fixed all! First there was a problem with initializing JSONDecoder, as dates in danger-swift now use Date type instead of a String. Then we needed to update our fixtures to cover a quite different requested_reviewers structure (dict with users/teams instead of an array now). It should be ready to go @ashfurrow

Cooooooool, tagged a 0.2.0 release with these changes. Thanks again!!


Docs: Update CONTRIBUTING doc to have correct slack channel link

Update CONTRIBUTING.md to have the correct slack channel linked now that a #deppy-dev channel has been created.

Team, I will work on this issue

Closed by #31


RFC: Redesign io_uring async flow

RFC: io_uring/async design
The issue, as described in #406:
CREATE TABLE t (t1,t2);
INSERT INTO t (t1,t2) VALUES (zeroblob(1024*1024),zeroblob(1024*1024));

Causes:
I/O error: Failed to submit entry to io_uring: submission queue is full

So after a 5+ day deep dive into this, I realize that there is no fix for this issue that doesn't ignore or continue to ignore a
different problem or edge case. I did not set out to fix every possible issue here and I didn't want
to just submit a substantial PR that redesigns the entire LinuxIO module (and maybe a bit more), but I also don't want
to put a band-aid on this specifically while ignoring known issues that it could be solving. However I am quite happy to work
on this if a plan/design can be agreed on. I will try to provide as much context, while being as concise as possible.
NOTE: the code in this PR is still is being actively refined/optimized.
Brief look at current design:
The current implementation creates SQEs directly inside the pread | pwrite| sync functions,
those are passed to submit_entry which attempts to push the entry into the submission queue,
and the pending_ops is incremented.
WrappedIOUring also holds an array of preallocated [libc::iovec(nullptr); MAX_IOVECS], where an iovec is handed out in a round robin fashion, one each Entry. The 'next_iovec' field is incremented and used to grab next_iovec % MAX_IOVECS that will overwrite old entries.
When cqe is returned, user_data is used to look up callback fn in map, and it's called with the resulting bytes of the operation.
Considerations:
(either problems that exist currently, or just problems to consider when designing or comparing solutions)

Blob/img writes even smaller than the example above, can easily cause sq overflow by >= MAX_IOVECS.

Creating the Entries inline in the read|write methods then pushing directly to the sq, is overwriting an iovec slot
and creating an Entry object that cannot be rebuilt or reused for resubmission. So if the caller re-submits again
it will advance the iovec slot N times until there is room in the sq.

operations that will overflow the SQ (from my findings) very often will be sequential writes to the same fd.
(overflow page allocs, etc)

many/most kernels (> 5.11) support sqpoll, a flag to set when creating a new uring, which definitely prevents this problem, with no other changes, up to at least writes of 5MB (my tests, possibly more).
(*obviously sqpoll comes with its own set of issues and isn't garaunteed to be supported so doesn't fully solve anything, just a consideration)

we are already setup to do vectored i/o (you probably see where I'm going here with #3)

Hopefully my original statement makes more sense now, trying to solve only that issue quickly turns into a much larger design discussion.

At the moment, without considerable re-design of the outer event loop, returning an error on overflow and returning control to the outer loop, would force re-submission by the caller as mentioned above. That's not to say that the setup won't be different in the future, and I did draft an implementation that tried to go this route... imo it felt like I was pushing io_uring specific logic outside of that module, but that could just be my fault due to a poor implementation.
=============
From the other implementations I have looked at, tigerbeetle being the primary one: maintaining a queue or list of pending io operations, each of which contains all the relevant data to construct an sqe only when the sq has the free space to submit, appears to be common practice. It also bleeds some complexity into the entire async event loop.
e.g.
Because now, a read|write call itself no longer guarantees 1 operation of forward progress (submission), it relies on run_once,
and operations in the queue can sit there idle if run_once isn't not called frequently enough in the outer loop.
NOTE I did notice a TODO in the Btree cursor implementation, that is called during Async* vdbe ops:
    fn wait_for_completion(&mut self) -> Result<()> {
        // TODO: Wait for pager I/O to complete

I was honestly unsure about this, especially because it involved touching the IO trait. Adding wait_for_completion to the io trait did seem like the logical thing, but if I'm wrong there, please let me know.
EDIT: after looking at it, I don't think there is any need for that, removing tomorrow.
This iteration of the design (there have been 4) uses batch submissions, where in the event that there is an overflow,
it will locate and group all sequential write operations to the same file descriptor into a batch, and do all the bookkeeping needed to submit 1 large write operation when the sq frees up, handling the callbacks for each individual operation when it completes.
Since you can submit up to IOV_MAX (1024 on most kernels, I believe) iovecs batched together per single writev syscall, and because the overflow was very likely to be caused by large amounts of sequential writes in the first place: imo, this is a prime opportunity to gather all those operations while waiting for sq space, batch them all together and submit them in 1 syscall.
Although that sounds pretty awesome for performance, (as im sure you will notice) all the bookkeeping and heap allocations of the iovecs (as mentioned, cannot use the static array) and queued pending op/batch info is not cheap and definitely is
not not complex. Maybe at some point there is logic to determine what other circumstances this might be useful, but this design only attempts to batch writes during overflow, with the intended goal of alleviating pressure on the sq.
Thread?
I definitely understand what issues are being avoided by remaining single threaded, but spinning up a short-lived os thread to handle the batching, sq polling, completions, etc in the event of/duration of an overflow: is definitely an option as well.
Although I am pretty sure that it would just be the userspace equivalent to enabling sqpoll, so that's something to consider.
============
Not trying to nerd snipe anyone or cause a big de-rail with this topic, so if this just reaches too far and is out of scope for the time being, let me know and I'll find something else to work on. I just started out to fix this and it clearly ended up being a much bigger and far reaching problem than originally anticipated.
Happy to discuss this more, or not at all, if it's not the right time.
Leaving this PR as a draft/RFC for comments, thoughts, additions (feel free to push to the branch, etc)
Tested this locally and there is no longer an issue writing massive blobs. and I also tested opening and querying the resulting .db file with sqlite3 and it works. (which seemed to be a problem very often during iteration on this)

I think the Idea is good, but from what i could understand, would we not need a limit on the queue Size? How should that be set?

I like the batching stuff it makes sense. But I still think this is more of a layer logic in the btree. If we insert huge blobs then why not suspend until we have enough space to continue? Is this fix going to fix all the starvation problems in io_uring or is batching just a fix until we hit the same error again in other cases?

if we insert huge blobs then why not suspend until we have enough space to continue?

the thought process is if we are going to be waiting either way why not collect the entries and figure out which ones can be batched so they can be submitted all at once, also if we overflowed, there's a really good chance that it's because we are actively getting writes that are batch-able.
I assume batching was in the long term plan anyway since we were already using vectored calls?

Is this fix going to fix all the starvation problems in io_uring or is batching just a fix until we hit the same error again in other cases?

ur definitely right about it also needing some logic in the btree/outer loop. I think is a matter of getting the outer loop to poll run_once or wait_for_completion at often enough intervals and planning for the cases where operations are still pending. I'm sure this is why tigetbeetle uses a tick method that's called on every step.
still has some bugs to work out tho, because right now I can spam inserting 4 1MB blobs 10x repeatedly with no problem, but 4 4mb blobs has an issue with the callbacks, so I'll continue to improve this.


Added Test Cases for 'Vulnerability Testing' #24

Added Test cases for #24. Please merge the PR if you find the changes appropriate.

Thanks @shubhayu-64   for your contribution. Your PR has been accepted for the Hacktoberfest. Please fill this form to get the awesome LoginRadius swags. :)


Division not allowed in graph for MacOS with M1 Max

Click to expand!
Issue Type
Bug
Source
binary
Tensorflow Version
2.8.0
Custom Code
No
OS Platform and Distribution
MaxOS 12.3.1
Mobile device

Python version
3.9
Bazel version

GCC/Compiler version

CUDA/cuDNN version

GPU model and memory
M1 Max 26 cores GPU 32 GB ram
Current Behaviour?
The same code runs on Colab but not on MacOS if using GPU.

Please note that the same exact code uploaded on Colab works perfectly, and same for this code on MacOS using CPU as device

Standalone code to reproduce the issue
# take the VAE example on Keras doc and instead of the KL div use the following
kl_loss = tf.reduce_mean(tf.reduce_sum(tf.square(z_mean)/2, axis=1) + tf.reduce_sum(tf.square(tf.exp(z_var/2) - 1)/4, axis=1))

Relevant log output
NotFoundError: Graph execution error:

Detected at node 'sub' defined at (most recent call last):
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/runpy.py", line 197, in _run_module_as_main
      return _run_code(code, main_globals, None,
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/runpy.py", line 87, in _run_code
      exec(code, run_globals)
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/site-packages/ipykernel_launcher.py", line 17, in <module>
      app.launch_new_instance()
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/site-packages/traitlets/config/application.py", line 972, in launch_instance
      app.start()
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/site-packages/ipykernel/kernelapp.py", line 712, in start
      self.io_loop.start()
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/site-packages/tornado/platform/asyncio.py", line 199, in start
      self.asyncio_loop.run_forever()
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/asyncio/base_events.py", line 601, in run_forever
      self._run_once()
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/asyncio/base_events.py", line 1905, in _run_once
      handle._run()
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/asyncio/events.py", line 80, in _run
      self._context.run(self._callback, *self._args)
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/site-packages/ipykernel/kernelbase.py", line 504, in dispatch_queue
      await self.process_one()
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/site-packages/ipykernel/kernelbase.py", line 493, in process_one
      await dispatch(*args)
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/site-packages/ipykernel/kernelbase.py", line 400, in dispatch_shell
      await result
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/site-packages/ipykernel/kernelbase.py", line 724, in execute_request
      reply_content = await reply_content
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/site-packages/ipykernel/ipkernel.py", line 383, in do_execute
      res = shell.run_cell(
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/site-packages/ipykernel/zmqshell.py", line 528, in run_cell
      return super().run_cell(*args, **kwargs)
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/site-packages/IPython/core/interactiveshell.py", line 2881, in run_cell
      result = self._run_cell(
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/site-packages/IPython/core/interactiveshell.py", line 2936, in _run_cell
      return runner(coro)
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/site-packages/IPython/core/async_helpers.py", line 129, in _pseudo_sync_runner
      coro.send(None)
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/site-packages/IPython/core/interactiveshell.py", line 3135, in run_cell_async
      has_raised = await self.run_ast_nodes(code_ast.body, cell_name,
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/site-packages/IPython/core/interactiveshell.py", line 3338, in run_ast_nodes
      if await self.run_code(code, result, async_=asy):
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/site-packages/IPython/core/interactiveshell.py", line 3398, in run_code
      exec(code_obj, self.user_global_ns, self.user_ns)
    File "/var/folders/6f/83fhb735331g631bmd84c_xc0000gn/T/ipykernel_52845/3646646759.py", line 4, in <cell line: 4>
      history = vae.fit(x_train, epochs=30, batch_size=128)
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/site-packages/keras/utils/traceback_utils.py", line 64, in error_handler
      return fn(*args, **kwargs)
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/site-packages/keras/engine/training.py", line 1409, in fit
      tmp_logs = self.train_function(iterator)
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/site-packages/keras/engine/training.py", line 1051, in train_function
      return step_function(self, iterator)
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/site-packages/keras/engine/training.py", line 1040, in step_function
      outputs = model.distribute_strategy.run(run_step, args=(data,))
    File "/opt/homebrew/Caskroom/miniforge/base/envs/ml-apple-metal/lib/python3.9/site-packages/keras/engine/training.py", line 1030, in run_step
      outputs = model.train_step(data)
    File "/var/folders/6f/83fhb735331g631bmd84c_xc0000gn/T/ipykernel_52845/2081516424.py", line 21, in train_step
      kl_loss = tf.reduce_mean(tf.reduce_sum(tf.square(z_mean)/2, axis=1) + tf.reduce_sum(tf.square(tf.exp(z_var/2) - 1)/4, axis=1))
Node: 'sub'
No registered 'Expm1' OpKernel for 'GPU' devices compatible with node {{node sub}}
	.  Registered:  device='XLA_CPU_JIT'; T in [DT_FLOAT, DT_DOUBLE, DT_COMPLEX64, DT_BFLOAT16, DT_COMPLEX128, DT_HALF]
  device='CPU'; T in [DT_FLOAT]
  device='CPU'; T in [DT_HALF]
  device='CPU'; T in [DT_BFLOAT16]
  device='CPU'; T in [DT_DOUBLE]
  device='CPU'; T in [DT_COMPLEX64]
  device='CPU'; T in [DT_COMPLEX128]

	 [[sub]] [Op:__inference_train_function_136152]

Could you please provide the reproducible code with the value of z_mean and z_var to reproduce the reported behavior in MacOS M1 max.

Could you please provide the reproducible code with the value of z_mean and z_var to reproduce the reported behavior in MacOS M1 max.

It's the code provided by the doc of Keras here: https://keras.io/examples/generative/vae/

Thank you for reporting this. M1 support is currently experimental.

@kulinseth There seems to be an issue finding the Expm1 op on M1 GPU. Could you please help take a look?
No registered 'Expm1' OpKernel for 'GPU' devices compatible with node {{node sub}}
	.   It is required for GPU usage.

issue finding the Expm1 op on M1 GPU

Hi @penpornk, have you managed to figure it out?


Wrong fix for W605 in f-strings

Python: 3.12.3
Version: ruff 0.4.4
Command: ruff check --select W605 .\test.py --fix
And let's given the following code snippet in test.py:
total = 10
ok = 7
incomplete = 3
s = f"TOTAL: {total}\nOK: {ok}\INCOMPLETE: {incomplete}\n"

It is wrongly fixed as:
total = 10
ok = 7
incomplete = 3
s = rf"TOTAL: {total}\nOK: {ok}\INCOMPLETE: {incomplete}\n"

Now the string is a raw string. The newlines will then interpreted as literals and will not remain newlines.

If I do the same for a normal string:
s = "TOTAL: 10\nOK: 3\INCOMPLETE: 7\n"

It is correctly fixed as:
s = "TOTAL: 10\nOK: 3\\INCOMPLETE: 7\n"

The invalid escape sequence '\I' is correctly fixed as \\, and the newlines are still there.

Even if it is a f-string, but without placeholders, it is done correctly:
s = f"TOTAL: 10\nOK: 3\INCOMPLETE: 7\n"

Result is correct:
s = f"TOTAL: 10\nOK: 3\\INCOMPLETE: 7\n"

Thank you for the report!
This is happening because each f-string element is analyzed individually. So, first TOTAL:  is analyzed, then \nOK:  and then \INCOMPLETE: , but the checker isn't aware that one of the element contains a newline character.

Update: The issue in the original post has been resolved, but I'm leaving this open because invalid escapes in format specifiers are not handled yet (see https://github.com/astral-sh/ruff/pull/14748#issuecomment-2517279287 and the surrounding discussion).


Support for new Spaceboard Sensors and better detection of hardware

Added support for the 3 new spaceboard sensors: ISL29125 (RGB), TCS34725 (RGB), and SI1132 (UV).
At this point, this ongoing PR also includes a rewrite of the SDK interface to be more object oriented.

Ok, done with the first round of PR review comments. Biggest things are to refactor to the abstract base class - most of those comments are in the definition for Acceleration, but they apply to all of the Sensor classes.

One last WS issue, then LGTM!


greek translation: remove GenEvent references

Thank you @gemantzu!! @Kosmas do you want to take a peek at this?


bug fix

This PR...
Changes

Fixes

add token to cookie on authentication
get single user fix
jwt middleware fix
fetch user id from req.param or req.users in users endpoint depending on availability

Checklist

[ ] tested locally
[ ] added new dependencies
[ ] updated the docs
[ ] added a test
[X] linter passes
[ ] format check passes

Fixes #

Thanks for the swift response 🙌🏾
We will merge this PR after all our CI scripts pass. This may involve continuous updates to this open PR. Together, we will do this

Please update this PR by pulling the latest changes from the dev branch 🙏🏾


bug: offchain delegations and treasuries are always save with ETH network

Saving treasuries and delegations info on offchain space settings is ignoring network, and always save as ethereum

Same issue is happning on other "network" params in offchain space settings, like the Delegation network.
All those fields are using short name like eth, matic, whereas v1 sequencer is expecting chain ID

Will wait for #810 which is introducing a new component for network selection

@wa0x6e I don't think we want to add just new extended network selector as unknown networks don't play nicely with treasuries.
We will need more groundwork on this front (https://github.com/snapshot-labs/sx-monorepo/issues/830), I was playing with solving this issue and it's quite a bit of refactor to get it right and see if we can get it prioritised this or next cycle.
Let's just fix this issue as-is for now.


File ingestion implementation

Please also add a description to this PR. A well formed description is required for all pull requests.


fix: goreleaser for mfa token caching

mfa token caching does not work for windows / darwin OS when CGO is disabled. Its recommended to turn of CGO for performance reasons, so adding an env variable to gorelaser to only turn on CGO for windows and darwin
fixes:
#908
#2194

@sfc-gh-swinkler One comment: can you link the GH issue that this PR fixes?

@sfc-gh-jcieslak is this merge really tested? Because for me the MFA Caching is not working and I get a prompt on every terraform action I do

@JESCHO99 Let's just keep the discussion in one place, please. You have already left a comment in https://github.com/Snowflake-Labs/terraform-provider-snowflake/issues/2194.

Hey @sfc-gh-asawicki, thanks for the fast reply. Sure. I just searched a little in the last messages and found both places which made me a little insecure where is the right place to put it. Let´s contine with my other comment.


Wrong Url in the showAsLink () method.

Hi. Faced a problem when getting the URL of the resource. Nova generates references to resources in the style of the kebab-case in the plural.
For myself, I took the following action in Select2.php:
        if ($resource) {             $resource = new ReflectionClass($resource);             $resource = (string)Str::kebab($resource->getShortName()) . 's';         }
And on the detailed page the link refers to the page resource, and not to the one specified in the settings.
I think, that problem in vue file DetailField.

Fixed on (Release v1.6)


PRP-66: Add ADR for project architecture

Ticket
https://wicmtdp.atlassian.net/browse/PRP-65
Changes

What was added, updated, or removed in this PR.

Add ADR for project architecture

Context for reviewers

Testing instructions, background context, more in-depth details of the implementation, and anything else you'd like to call out or ask reviewers. Explain how the changes were verified.

Testing

Screenshots, GIF demos, code examples or output to help show the changes working as expected. ProTip: you can drag and drop or paste images into this textbox.

I'm a bit confused about the problem statement line

"Where do we want to integrate the data persistence layer"
I don't think that's answered in the ADR

I think I was trying to get at: does the application have a database or is all the data stored locally in the client's browser until it's submitted to the Mock API? I'll add some content to clarify that


newtask endpoint for pbclient API

Missing enpoint in pbclient API is getting a new task, selected by pybossa. Very similar to get tasks list

Forgot to write test. Update will come soon

We never add it because we use that only from the front end :-) Good add! Once the tests pass, we will merge it and publish it.  Please, increase the version number as well on the setup.py file.

We want to launch pybossa as a service, so we need to upgrade the client to cover most of the API. Do you think I should increase the version for each PR? And which part of the version?

This one is a minor improvement, so go for x.x.Y :-) Then, if you change anything big, we can move it into x.X.x and if we do something muuuch bigger into X.x.x.

Please, don't rush with merging it I made some enhancements. BTW the coveralls isn't in rush anyway :rofl:

Yes, Coveralls usually goes really slow or returns something not good. In any case, we keep it as a way of reviewing if we miss something important.

Heya!
Any chance we might get a status on this PR? We are depending on it for our own project interfacing with PyBossa.
If any changes are still required, please let me know.


pyAFQ error

Hi @mattcieslak and @36000,
I was testing pyAFQ on some single shell data and got an error. I'm using a single-subject test case before running it on everyone. This particular individual was successfully reconstructed using both dsi_studio_gqi and mrtrix_singleshel_ss3t_ACT-fast using the exact same call (only changing the output folder and recon spec).
Call:
singularity run --cleanenv -B /xdisk/adamraikes/cornell/nifti:/nifti:ro -B /xdisk/adamraikes/cornell/derivatives/qsiprep-0.16.0RC3_SyNFS:/xdisk/adamraikes/cornell/derivatives/qsiprep-0.16.0RC3_SyNFS -B ~/misc/license.txt:/opt/freesurfer/license.txt /groups/adamraikes/singularity_images/qsiprep-0.16.0RC3.sif /nifti /xdisk/adamraikes/cornell/derivatives/qsiprep-0.16.0RC3_SyNFS/pyAFQ participant --participant-label 087 -w /xdisk/adamraikes/cornell/derivatives/qsiprep-0.16.0RC3_SyNFS/scratch --nthreads 12 --omp-nthreads 6 --recon-only --recon-input /xdisk/adamraikes/cornell/derivatives/qsiprep-0.16.0RC3_SyNFS/qsiprep --recon-spec pyAFQ
Error:
Node: qsirecon_wf.sub-087_pyAFQ_full.sub_087_space_T1w_desc_preproc_recon_wf.pyAFQ_full.run_afq
Working directory: /xdisk/adamraikes/cornell/derivatives/qsiprep-0.16.0RC3_SyNFS/scratch/qsirecon_wf/sub-087_pyAFQ_full/sub_087_space_T1w_desc_preproc_recon_wf/pyAFQ_full/run_afq

Node inputs:

bval_file = /xdisk/adamraikes/cornell/derivatives/qsiprep-0.16.0RC3_SyNFS/qsiprep/sub-087/dwi/sub-087_space-T1w_desc-preproc_dwi.bval
bvec_file = /xdisk/adamraikes/cornell/derivatives/qsiprep-0.16.0RC3_SyNFS/qsiprep/sub-087/dwi/sub-087_space-T1w_desc-preproc_dwi.bvec
dwi_file = /xdisk/adamraikes/cornell/derivatives/qsiprep-0.16.0RC3_SyNFS/qsiprep/sub-087/dwi/sub-087_space-T1w_desc-preproc_dwi.nii.gz
itk_file = /xdisk/adamraikes/cornell/derivatives/qsiprep-0.16.0RC3_SyNFS/qsiprep/sub-087/anat/sub-087_from-MNI152NLin2009cAsym_to-T1w_mode-image_xfm.h5
kwargs = {'directions': 'prob', 'max_angle': 30.0, 'sphere': None, 'seed_mask': None, 'seed_threshold': 0, 'n_seeds': 1, 'random_seeds': False, 'rng_seed': None, 'stop_mask': None, 'stop_threshold': 0, 'step_size': 0.5, 'min_length': 10, 'max_length': 1000, 'odf_model': 'CSD', 'tracker': 'local', 'nb_points': False, 'nb_streamlines': False, 'seg_algo': 'AFQ', 'reg_algo': None, 'clip_edges': False, 'parallel_segmentation': {'n_jobs': -1, 'engine': 'joblib', 'backend': 'loky'}, 'progressive': True, 'greater_than': 50, 'rm_small_clusters': 50, 'model_clust_thr': 1.25, 'reduction_thr': 25, 'refine': False, 'pruning_thr': 12, 'b0_threshold': 50, 'prob_threshold': 0, 'roi_dist_tie_break': False, 'dist_to_waypoint': None, 'rng': None, 'return_idx': False, 'presegment_bundle_dict': None, 'presegment_kwargs': {}, 'filter_by_endpoints': True, 'dist_to_atlas': 4, 'save_intermediates': None, 'n_points': 100, 'clean_rounds': 5, 'distance_threshold': 5, 'length_threshold': 4, 'min_sl': 20, 'stat': 'mean', 'min_bval': None, 'max_bval': None, 'filter_b': True, 'robust_tensor_fitting': False, 'csd_response': None, 'csd_sh_order': None, 'csd_lambda_': 1, 'csd_tau': 0.1, 'gtol': 0.01, 'brain_mask_definition': None, 'bundle_info': None, 'reg_template_spec': 'mni_T1', 'mapping_definition': None, 'reg_subject_spec': 'power_map', 'profile_weights': 'gauss', 'scalars': ['dti_fa', 'dti_md'], 'import_tract': None, 'sbv_lims_bundles': [None, None], 'volume_opacity_bundles': 0.3, 'n_points_bundles': 40, 'sbv_lims_indiv': [None, None], 'volume_opacity_indiv': 0.3, 'n_points_indiv': 40, 'viz_backend_spec': 'plotly_no_gif', 'virtual_frame_buffer': False, 'omp_nthreads': 6}
mask_file = /xdisk/adamraikes/cornell/derivatives/qsiprep-0.16.0RC3_SyNFS/scratch/qsirecon_wf/sub-087_pyAFQ_full/sub_087_space_T1w_desc_preproc_recon_wf/qsirecon_anat_wf/resample_mask/sub-087_desc-brain_mask_resample.nii.gz
tck_file = <undefined>

Traceback (most recent call last):
  File "/usr/local/miniconda/lib/python3.8/site-packages/nipype/pipeline/plugins/multiproc.py", line 67, in run_node
    result["result"] = node.run(updatehash=updatehash)
  File "/usr/local/miniconda/lib/python3.8/site-packages/nipype/pipeline/engine/nodes.py", line 524, in run
    result = self._run_interface(execute=True)
  File "/usr/local/miniconda/lib/python3.8/site-packages/nipype/pipeline/engine/nodes.py", line 642, in _run_interface
    return self._run_command(execute)
  File "/usr/local/miniconda/lib/python3.8/site-packages/nipype/pipeline/engine/nodes.py", line 750, in _run_command
    raise NodeExecutionError(
nipype.pipeline.engine.nodes.NodeExecutionError: Exception raised while executing Node run_afq.

Traceback (most recent call last):
  File "/usr/local/miniconda/lib/python3.8/site-packages/nipype/interfaces/base/core.py", line 398, in run
    runtime = self._run_interface(runtime)
  File "/usr/local/miniconda/lib/python3.8/site-packages/qsiprep/interfaces/pyafq.py", line 59, in _run_interface
    tck_file = fname_presuffix(self.inputs.tck_file,
  File "/usr/local/miniconda/lib/python3.8/site-packages/nipype/utils/filemanip.py", line 140, in fname_presuffix
    pth, fname, ext = split_filename(fname)
  File "/usr/local/miniconda/lib/python3.8/site-packages/nipype/utils/filemanip.py", line 92, in split_filename
    pth = op.dirname(fname)
  File "/usr/local/miniconda/lib/python3.8/posixpath.py", line 152, in dirname
    p = os.fspath(p)
TypeError: expected str, bytes or os.PathLike object, not _Undefined

Hi, I believe I am getting the exact same error as you. Is there any chance that you solved this problem and would be willing to share?
Thank you so much in advance!!

@araikes sorry for the delay in responding to this.  @araikes or @margokersey , I think this issue may be fixed in #533 . Could you try building a qsiprep docker image using that branch and re-running, to see if that fixes the issue?
You can also try running the pyAFQ_import_trk pipeline instead. This will import the tractography from MRtrix instead of using the in-house DIPY tractography. You can customize which tractography to use inside of the pyAFQ_import_trk.json file.

Thank you!! Would love to test that out. Sorry this may be a very dumb question but how can I build the Singularity image from a Docker image for that branch? I can't figure it out after trying many times. I am using an HPC cluster that supports Singularity.

Thank you so much @pcamach2 ! Apologies for the delay, as I am new to Docker/Singularity. I was able to push the local docker image to my Dockerhub username and then build the Singularity image from inside my HPC. When testing out the 36000 branch, unfortunately I am now getting a new error: @36000
Traceback:
        Traceback (most recent call last):
          File "/usr/local/miniconda/lib/python3.8/site-packages/nipype/interfaces/base/core.py", line 398, in run
            runtime = self._run_interface(runtime)
          File "/usr/local/miniconda/lib/python3.8/site-packages/qsiprep/interfaces/pyafq.py", line 101, in _run_interface
            myafq = ParticipantAFQ(
        TypeError: type object got multiple values for keyword argument 'import_tract'

pyafq fails
Overview
This looks to me like qsiprep does not consider pyafq_tractometry a legitimate option.
I think I spelled it right.  Thoughts?
Background
I've been trying the various reconstruction algorithms just by swapping them into place in the singularity command.  This was working perfectly until I hit pyafq_tractometry!
Environment
Apptainer on HPC
Singularity command
singularity run --cleanenv --bind ${MRIS}/data:/data:ro
--bind ${MRIS}/derivatives2/qsiprep:/qsiprep-output:ro
--bind ${MRIS}/derivatives2:/out
--bind ${MRIS}/qsi_work3:/work
${SIF}/qsiprep.sif /data /out participant
--participant_label ${Subject}
--recon-input /qsiprep-output
--recon-spec pyafq_tractometry
--stop-on-first-crash
--output-resolution 1.3 -w /work -v -v
--n-cpus 16 --omp-nthreads 15
--skip-odf-reports --recon-only
--fs-license-file ${HOME}/license.txt
--freesurfer_input /out/fmriprep2301/sourcedata/freesurfer
SLURM Log
INFO:    underlay of /etc/localtime required more than 50 (103) bind mounts
230427-16:59:18,481 nipype.workflow IMPORTANT:
Running qsirecon version 0.16.1:
  * BIDS dataset path: /data.
  * Participant list: ['CAM003'].
  * Run identifier: 20230427-165916_fe4ef9e8-cc79-498c-a33a-df89768c4080.

IMPORTANT:nipype.workflow:
Running qsirecon version 0.16.1:
* BIDS dataset path: /data.
* Participant list: ['CAM003'].
* Run identifier: 20230427-165916_fe4ef9e8-cc79-498c-a33a-df89768c4080.
Process Process-2:
Traceback (most recent call last):
File "/usr/local/miniconda/lib/python3.8/multiprocessing/process.py", line 315, in _bootstrap
self.run()
File "/usr/local/miniconda/lib/python3.8/multiprocessing/process.py", line 108, in run
self._target(*self._args, **self._kwargs)
File "/usr/local/miniconda/lib/python3.8/site-packages/qsiprep/cli/run.py", line 1175, in build_recon_workflow
retval['workflow'] = init_qsirecon_wf(
File "/usr/local/miniconda/lib/python3.8/site-packages/qsiprep/workflows/recon/base.py", line 89, in init_qsirecon_wf
single_subject_wf = init_single_subject_wf(
File "/usr/local/miniconda/lib/python3.8/site-packages/qsiprep/workflows/recon/base.py", line 168, in init_single_subject_wf
spec = _load_recon_spec(recon_spec, sloppy=sloppy)
File "/usr/local/miniconda/lib/python3.8/site-packages/qsiprep/workflows/recon/base.py", line 257, in _load_recon_spec
raise Exception("{} is not a file that exists or in {}".format(spec_name, prepackaged))
Exception: pyafq_tractometry is not a file that exists or in ['amico_noddi', 'csdsi_3dshore', 'dipy_3dshore', 'dipy_dki', 'dipy_mapmri
', 'dsi_studio_gqi', 'mrtrix_multishell_msmt_ACT-fast', 'mrtrix_multishell_msmt_ACT-hsvs', 'mrtrix_multishell_msmt_noACT', 'mrtrix_sin
gleshell_ss3t_ACT-fast', 'mrtrix_singleshell_ss3t_ACT-hsvs', 'mrtrix_singleshell_ss3t_noACT', 'mrtrix_tckglobal', 'multishell_scalarfe
st', 'pyafq', 'pyafq_input_trk', 'reorient_fslstd']
Detailed performance metrics for this job will be available at https://metrics.hpc.arizona.edu/#job_viewer?action=show&realm=SUPREMM&r
esource_id=73&local_job_id=7058510 by 8am on 2023/04/28.
$

@dkp: I believe that this is a new name for that pyafq pipeline as of 0.17 (see https://github.com/PennLINC/qsiprep/pull/533). Could you please try running the newer version rather than 0.16.1?

Thanks, I did not realize there was a new version available.  I will try it.

That works perfectly with qsiprep 0.17.0!  Thanks so much.


Prompt hangs on windows 10 v2.1.6

Running ember-cli-windows v2.1.6 on my machine will not let me clear the prompt & won't accepted -h value to proceed without it.
in powershell I am not able to type in any input and in cmd I can type a value but once I hit enter the prompt hangs.
I noticed in my local node_modules folder it does not have prompt listed as a dependency in package.json but it is on this repo.
Possibly related issues

Prompt
Node

calling it directly via node node_modules\ember-cli-windows\bin\ember-cli-windows
so the issue is spawning it via ember-cli-windows-addon

I get the same issue


Translate Readme.md to JP

Hi, Ochrons, its my initial pull request, concerning a translation to Japanese :)

Thanks, looks like a great start! If you have questions regarding the original text, feel free to reach out on the Gitter channel!


B 18312 multi move dashboard Integration

Agility ticket
Integration PR
Summary
This PR introduces the Multi Move dashboard. When the service member logs in they are routed to the multi move dashboard. This PR also introduces changes to the playwright tests where they are now using the multi-move feature flag.
Verification Steps for Reviewers
These are to be checked by a reviewer.

[ ] Has the branch been pulled in and checked out?
[ ] Have the BL acceptance criteria been met for this change?
[ ] Was the CircleCI build successful?
[ ] Has the code been reviewed from a standards and best practices point of view?

Setup to Run the Code

Instructions for starting storybook
Instructions for starting the MilMove application
Instructions for running tests

How to test

Ensure that the multiMove feature flag is switched on for your respective environment
Log into MilMove as a service member
You should have been routed to the Multi Move Dashboard

Frontend

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, Edge).
[ ] There are no new console errors in the browser devtools.
[ ] There are no new console errors in the test output.
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.
[ ] This change meets the standards for Section 508 compliance.

Screenshots

Warnings

:warning:

Files located in legacy directories (src/shared or src/scenes) have
been edited. Are you sure you don’t want to also relocate them to the new file structure?
View the frontend file org ADR for more information

:warning:
This PR does not include changes to unit tests, even though it affects app code.

:warning:

Changes were made to package.json, but not to yarn.lock - Perhaps you need to run yarn install?

  Generated by :no_entry_sign: dangerJS against 3497b078fbd479b77dbc7d774a1e2f48f97db6b8


adds the SimpleCov gem to the project

Somewhat related to #1024
Description
To help keep track of the test coverage we are currently working to improve, I think it's a good idea to have the SimpleCov gem on the project, so we can have a better idea of the areas we need to focus on.
List any dependencies that are required for this change. (gems, js libraries, etc.)

gem install 'simplecov'

Type of change

New feature (non-breaking change which adds functionality)

Screenshots

Here's how the simplecov navbar looks right now:

As you can see, this gem can be of great assistance for developers in the future looking to help increase the test coverage of the project.

Nice!


Add Forwarded header

The router should include the Forwarded header as defined in RFC 7239.

Most of the params look straightforward.
Do we want to populate the by field? If so need to work out the best identifier to use. I think the host IP is the obvious one.

The by field appears to be used to distinguish which network interface the request that is being forwarded came in on. So the destination IP of the client request seems like a reasonable thing to put in there.

Flynn is unmaintained and our infrastructure will shut down on June 1, 2021. See the README for details.