test(intersect): lines that cross at a shared starting point

Adds unit tests that expose #10
Short simple lines that only intersect at their shared starting point return that starting point as an intersection.
Long complex lines with a shared starting point don't have that starting point in the resulting list of intersections.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Closing in favor of https://github.com/bpmn-io/path-intersection/commit/8d92730da8d91e22c40825f12dd9caa10fb42405. Cf. https://github.com/bpmn-io/path-intersection/issues/10#issuecomment-565795765, too.


The scope is improper after heredoc

As described in https://github.com/jfcherng/Sublime-ShellScriptImproved/issues/1#issuecomment-206750685.

Related post: https://forum.sublimetext.com/t/how-to-deal-with-partially-overlapping-context/19122

Should be resolved in 1.0.12.

Hello once again. Nice that you could do it, however your fix brought two new issues regarding normal "boring" syntax.
First: An indented (with tabs) EOF is no longer detected
	cat <<-EOF
			This is some text
		EOF

Second: if the heredoc starts on such a line, it actually demands the closing } or fi. See for example:
fun () { cat <<EOF
	Some text with a curly bracket }
	Some text with a semicolon and a curly bracket; }
	Even more text
EOF
}

So you should return to the normal heredoc context when hitting the newline or something like that

@dasisdormax Nice catch.
The first one is actually an accident. An space indentation before EOF raised a warning but I thought it is an error at that time. I will revert the commit in the next release.
The second one will be fixed in the next release.


Golang programs that import flag or httptest will panic due to uninitialized os.Args

When I try to run fastly compute serve with webassembly code that I build with GOARCH=wasm GOOS=wasip1 go build  ./... if I have code that even indirectly includes flag then the flag package imports httptest  ( _ "net/http/httptest") which will panic when it gets to:
https://github.com/golang/go/blob/3ed007d754b685cd8f6011a8e96a3c9303c785db/src/flag/flag.go#L1199
An example piece of code
package main

import (
	"fmt"
	"os"

	_ "bytes"
	_ "context"
	_ "encoding/json"
	_ "net"
	_ "net/http"

	// This causes issues!
	_ "net/http/httptest"

	_ "net/url"
	_ "strconv"
	_ "strings"
)

func main() {
	fmt.Println("FASTLY_SERVICE_VERSION:", os.Getenv("FASTLY_SERVICE_VERSION"))
}

and run fastly compute serve
with this fastly.toml file:
# This file describes a Fastly Compute package. To learn more visit:
# https://www.fastly.com/documentation/reference/compute/fastly-toml

authors = ["XXXXacademy.org"]
cloned_from = "https://github.com/fastly/compute-starter-kit-go-default"
description = "fastly compute go service"
language = "go"
manifest_version = 3
name = "fastly-khanacademy-compute"
service_id = "XXXX"

[scripts]
  build = "go build -o bin/main.wasm ./cmd/serve"
  env_vars = ["GOARCH=wasm", "GOOS=wasip1"]
  post_init = "go get github.com/fastly/compute-sdk-go@latest"

# Despite most of the local_server configuration being in the fastly.{env}.toml
# files, this actually needs to be defined in our basic fastly.toml file.
# It MUST be defined as the full semantic version, but without the 'v'.
# The 'fastly' cli must be at version 10.2.0 or later to use this.
[local_server]
  viceroy_version = "0.9.6"

I can reproduce this problem with tinygo, but wasmtime doesn't seem to have the problem directly, so I assume it is a viceroy problem.
I mentioned this in the gophers slack and @dgryski said he passed it on to a colleague, but I didn't want others to duplicate bug reports, so I thought I'd record it here for others to track.

Could you try with 0.9.7? It has a newer wasmtime, it's possible there may be some interaction there.

This still happens with viceroy 0.9.7.
A much smaller reproducer is:
package main

import (
	"fmt"
	"os"
)

func main() {
	fmt.Println(os.Args[0])
}

Would it be reasonable to supply at least one WASI arg to the guest? Either with some default value like wasm, or the name of the service?

@ydnar Yes.  The baseame of the wasm file I believe is accessible; worst case we just lie.

@ydnar Do you have a pointer for where this could be plumbed through? I'm not much of a rustacean.


WinGet installation instruction is broken

Docs say that I should install App Installer from Microsoft Store if I'm using Windows older than 11. I'm using Windows 10 21H2.
However, Microsoft Store page for App Installer is missing install button:

So, I don't understand how I am supposed to install App Installer.

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

ID: 98654069-5528-4f98-7218-a803bee0a532
Version Independent ID: b49c32ea-2efa-b090-7123-87c31d10279b
Content: Use the winget tool to install and manage applications
Content Source: hub/package-manager/winget/index.md
Product: dev-environment
Technology: package-manager
GitHub Login: @KevinLaMS
Microsoft Alias: kevinla

Hey @slonopotamus - Are you sure that you don't already have the App Installer? Usually this is why the 'Get' button wouldn't show up. Could you try opening PowerShell and entering: winget --version to see whether it may already be there?

You can also check to see if you have App Installer by opening PowerShell and entering:
Get-AppxPackage microsoft.desktopappinstaller
You will see "Status: OK" at the bottom if you have it as well as a version # for App Installer... the version # should be greater than 1.11.11451. If it's not, following the instructions here: https://github.com/microsoft/winget-cli/tree/master/doc/troubleshooting#executing-winget-doesnt-display-help

Hey @slonopotamus - Are you sure that you don't already have the App Installer? Usually this is why the 'Get' button wouldn't show up. Could you try opening PowerShell and entering: winget --version to see whether it may already be there?

Yes, it turned out that

App Installer is already installed (it looks like that it comes pre-installed in modern Windows 10 builds?)
I cannot even uninstall it through Microsoft Store

Anyway, this isn't documented and Microsoft Store UI is extremely unhelpful. It could clearly state on app page that app is already installed.

I'll make an update to that winget install page to try to make this a bit more clear. Yes, from what I understand App Installer is included with all modern Windows 10 builds (but may not have been in some early builds). I don't have the ability to update the Store description, but will try to find the right person and shoot them a mail to see if they could add that App Installer is typically included in Windows by default to the description.

Update was included in: https://github.com/MicrosoftDocs/windows-uwp/commit/5688dbd2ab20808c6e181932f9874b8c51606b69
Closing this out. Please reopen if you feel that the docs need any further clarification. Thanks for the contribution and bringing this to our attention!


How to run strider as a daemon service in EC2?

I find the document of strider shows we can run strider by typing strider in the terminal. However, it does not show how to run it as a daemon service. I write a service script to put it into /etc/init.d/. But the strider process is killed directly after started. I need some help. Thank you.
Here is my service script:
#!/bin/bash
#
# chkconfig: 35 90 12
# description: strider server
#
# Get function from functions library
. /etc/init.d/functions
# Start the service strider
start() {
        echo -n $"Starting strider CI server: "
        PORT=3888 SERVER_NAME="http://ec2-ip:3888" /home/ec2-user/.nvm/v0.10.36/bin/strider &
        ### Create the lock file ###
        touch /var/lock/subsys/strider
        success $"strider server startup"
        echo
}
# Restart the service strider
stop() {
        echo -n $"Stopping strider server: "
        killproc strider
        ### Now, delete the lock file ###
        rm -f /var/lock/subsys/strider
        echo
}
### main logic ###
case "$1" in
  start)
        start
        ;;
  stop)
        stop
        ;;
  status)
        status strider
        ;;
  restart|reload|condrestart)
        stop
        start
        ;;
  *)
        echo $"Usage: $0 {start|stop|restart|reload|status}"
        exit 1
esac
exit 0

instead put the script in /etc/init.d creat a script in /etc/init/ and use something like this
#!upstart
description "Strider upstart job"

start on (local-filesystems and net-device-up IFACE!=lo)
stop on shutdown

script
    export PORT=
    export DB_URI=""
    export SMTP_HOST=""
    export SMTP_PORT=""
    export SMTP_USER=""
    export SMTP_PASS=""
    export SERVER_NAME=""
    export STRIDER_CLONE_DEST=""
    export PLUGIN_GITHUB_APP_ID=""
    export PLUGIN_GITHUB_APP_SECRET=""
    echo $$ > /var/run/strider.pid
    exec

/path/to/strider/bin/strider >> /var/log/strider.log 2>&1
end script

pre-start script
    echo "[`date -u +%Y-%m-%dT%T.%3NZ`] (sys) Starting" >> /var/log/strider.log
end script

pre-stop script
    rm /var/run/strider.pid
    echo "[`date -u +%Y-%m-%dT%T.%3NZ`] (sys) Stopping" >> /var/log/strider.log
end script


add kcp cli completion command

Implement KCP CLI: completion command which  support for Powershell, bash and zsh completion (see task 932)

/test all

/test all

/test all

/test all

/test all

/test all


[WIP] Use multidot in NCA and MLKR

Uses np.linalg.multi_dot in gradient computations in NCA and MLKR. This should provide optimal performance, no matter the values of n_components, n_features, and n_samples.

I just realized that in fact in NCA one of the reason why we replaced A.dot(X.T) by X_embedded.T was that we already had it computed when computing the cost, so I'll update the PR to reuse it in NCA as it was before, and update MLKR in the same way

Turns out in this case where we reuse X_embedded there is no need anymore to use np.linalg.mutli_dot since the optimal order of the products is to do first X_embedded.T.dot(W_sym) and then doing this dot X. (it makes a cost of n_components* n_samples**2 + n_components * n_samples * n_features instead of n_features* n_samples **2 + n_components * n_samples * n_features)
I think the PR is ready to merge if travis passes


refactor: merge game info into home

Action items before merging

[ ] fix "Learn more" link (gatsby "Link" and "navigate" don't support in-page routing)

fix "Learn more" link (gatsby "Link" and "navigate" don't support in-page routing)

this might be the point at which we add the react/router dep...

Looks like we don't need the gatsby-plugin-anchor-links after all, I just needed to put the id on the row instead of the section as the section didn't do anything with it - we could also consider adding logic to add the id to the section div such that basically any section can be linked to.
Also, Link in comparison to AnchorLink doesn't scroll smoothly, but that would also be something which - if we want it - can be achieved using a react plugin like smooth-scroll instead of with a gatsby plugin.


beforeNavigate does not fire after clicking link with download attribute

Describe the bug
This application has two pages - a home page at / and an about page at /about.
You can click between the two pages using the links in the header, or the links that appear on each page.
Each time you click, BEFORE NAVIGATE IS RUNNING logs to the console.
If you click the PDF Url link at the bottom, beforeNavigate runs as normal, logging BEFORE NAVIGATE IS RUNNING to the console.
The next link you click, beforeNavigate does not run.
Reproduction
https://stackblitz.com/edit/sveltejs-kit-template-default-mgk5se?file=src/routes/+page.svelte
Logs
No response
System Info
System:
    OS: Linux 5.0 undefined
    CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 16.14.2 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 7.17.0 - /usr/local/bin/npm
  npmPackages:
    @sveltejs/adapter-auto: ^2.0.0 => 2.0.0
    @sveltejs/kit: ^1.5.0 => 1.15.1
    svelte: ^3.54.0 => 3.58.0
    vite: ^4.2.0 => 4.2.1

Severity
serious, but I can work around it
Additional Information
No response

@s3812497 This was the issue I was referring to here: https://github.com/sveltejs/kit/issues/9170#issuecomment-1446761061 that I wasn't able to reproduce at the time. We have a working reproduction for it now, forked with the latest SvelteKit.

Sorry for the late response! I have started looking into a fix for this last night. Thank you for reporting it again.
The issue is caused by what I explained previously https://github.com/sveltejs/kit/issues/9170#issuecomment-1446725050 so I just need to try out a fix and test it :)


The health Facility of the user that is in transit must be registered

Add 2 fields
The system must have registered that the patient is a patient in transit during the registration

I don't think this is an issue yet. Lets make it a new requirement first

It's required to add a Field to collect the information of the patient in transit.

I know but lets not make it an issue yet, but a new requirement because this is just coming to us this way

 The health Facility of the user that is in transit must be registered

Can we create issues only for things already implemented?


Compacting drawer items not inserted into crafting grid properly

Issue description: Items in compacting drawers from Storage Drawers don't insert into the crafting grid when you press the "+" button
What happens: Pressing JEI crafting "+" button does not fill in items from compacting drawers (i.e. nuggets). This only started happening after updating Refined storage.
What you expected to happen: Pressing the "+" button should fill in the grid with appropriate items from my network.
Steps to reproduce:

Setup compacting drawer with gold ingots attached to network via external storage cable. Set cable to insert and extract mode
attempt to craft golden carrot in crafting grid by searching in JEI and hitting plus button
no/only a few nuggets will be inserted, despite having enough in the network.
...

Version (Make sure you are on the latest version before reporting):

Minecraft: 1.10.2
Forge: 1.10.2-forge1.10.2-<IP_ADDRESS>6
Refined Storage: 1.2.8

Does this issue occur on a server? [yes/no] no
If a (crash)log is relevant for this issue, link it here:
[pastebin/gist/etc link here]

UPDATE: Removing and replacing all of the external storage cables fixed the problem. Not sure if this is a bug or just a symptom of updating.

duplicate of #370

I ran across this also, two days ago, it has also randomly effected crafting as well. I have a recipe for ender io to create vibrant crystals and with 8 nuggets in a compacting drawer it would not + click via JEI and the crafting monitor would just sit. I had to craft more vibrant ingots, once there was about 18 nuggets worth in the drawer the craft completed.
I had updated for both SD and RS on the 2nd (last Friday) and that is when I started to notice this issue. Did not occur prior to that.

Interesting that when you make more nuggets it works in a compacting drawer. I disconnected mine from the system for the time being but my cobblestone is still in a regular storage drawer and it won't recognize any of it when I hit the "+" button (I have 6k+ cobble).

Are you able to retrieve the items from the storage WITHOUT using JEI "+"?

Yes, I can manually click in and out, only some items, mostly it seems like oredict ones, will not fill on a jei plus. I also have a crafting pattern that uses 8 vibrant nuggets and an emerald, I think it was created as oredict but can't verify right now, that with 8 nuggets in a compacting drawer, would sit in the crafting queue until I added more ingots to the drawer, via another crafting pattern.
-----Original Message-----
From: "Raoul"<EMAIL_ADDRESS>Sent: 12/5/2016 12:54 PM
To: "raoulvdberge/refinedstorage"<EMAIL_ADDRESS>Cc: "xannor"<EMAIL_ADDRESS>"Comment"<EMAIL_ADDRESS>Subject: Re: [raoulvdberge/refinedstorage] Compacting drawer items notinserted into crafting grid properly (#700)
Are you able to retrieve the items from the storage WITHOUT using JEI "+"?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

I'm having just about the same problem. Things connected a storage drawer Drawer Controller via an external storage just don't seem to want to work properly.  Seems to work once or twice when first logging in (might just SEEM this way, no actual proof) then it won't "+ click" into the crafting grid. I can manually grab them out however.  Also, it hangs on my auto crafting when things from the drawers are needed. No matter how long it sits, they never resolve.
refinedstorage-1.2.8.jar
StorageDrawers-1.10.2-3.5.6

I have noticed a pattern that maybe someone else can verify. It seems, to me at least, to only be ore dictionaried items from a drawer, and when 4 or more are needed. I could plus click an immersive engineering hv connector but not the light engineering block. Never had a problem with Redstone dust, which is also in my drawer system.
-----Original Message-----
From: "bubblesayoyo"<EMAIL_ADDRESS>Sent: 12/6/2016 10:32 AM
To: "raoulvdberge/refinedstorage"<EMAIL_ADDRESS>Cc: "xannor"<EMAIL_ADDRESS>"Comment"<EMAIL_ADDRESS>Subject: Re: [raoulvdberge/refinedstorage] Compacting drawer items notinserted into crafting grid properly (#700)
I'm having just about the same problem. Things connected a storage drawer Drawer Controller via an external storage just don't seem to want to work properly. Seems to work once or twice when first logging in (might just SEEM this way, no actual proof) then it won't "+ click" into the crafting grid. I can manually grab them out however. Also, it hangs on my auto crafting when things from the drawers are needed. No matter how long it sits, they never resolve.
refinedstorage-1.2.8.jar
StorageDrawers-1.10.2-3.5.6
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Mine is just completely hit or miss.  I was just playing and I needed 4 pieces of iron for a recipe and just go 3 of the 4 shift clicked into the grid. I cleared it and tried again, and got 0 of 4, and now it won't get any again. As for 4 or more, I've tried several times to get an EnderIO capacitor (2 gold nuggets, 1 copper, 4 red stone) and it only seems to grab the gold nuggets, although every now and then it will work completely. Again just seems so random...

Idk about the ore-dictionary thing unless stone is in that too because even my cobblestone (which is in just a regular storage drawer with an external storage) won't fill in with the jei +, only when I put it in manually. I have also noticed the pattern where sometimes it will fill 3 out of 4 when I hit the +, but with cobble it won't insert any of the required items.

Well, I dont know if it was SD 3.5.8+ or RS 1.2.10 but I no longer have this issue it seems.

I couldn't reproduce this either (tried the golden carrot thing).

I will update both today and see if it is also fixed on my end!
On Sat, Dec 10, 2016 at 10:00 AM, Raoul<EMAIL_ADDRESS>wrote:

I couldn't reproduce this either (tried the golden carrot thing).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/raoulvdberge/refinedstorage/issues/700#issuecomment-266214421,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AV4ycCiR11C27xAqcR10ksKF0pM4sPdDks5rGr6FgaJpZM4LCzKS
.

I'm assuming this thing is fixed


Snapshots tests not passing in CI for all database versions

Describe the bug
Snapshot integration tests aren't passing in continuous integration (CI). It may need custom updates to a *.cnf file within the Docker containers.
Affected versions
MySQL 8.0 is working fine. But these release versions are not:

MySQL 5.7
MariaDB 10.5

Additional context
dbt snapshots might not work properly due to automatic initialization and updating for TIMESTAMP if:

the output of SHOW VARIABLES LIKE 'sql_mode' includes NO_ZERO_DATE

A solution is to include the following in a *.cnf file:
[mysqld]
explicit_defaults_for_timestamp = true
sql_mode = "ALLOW_INVALID_DATES,{other_sql_modes}"

where {other_sql_modes} is the rest of the modes from the SHOW VARIABLES LIKE 'sql_mode' output.

--explicit-defaults-for-timestamp=ON was sufficient to enable snapshots for both MySQL 5.7 and MariaDB 10.5.
Problem
The implementation of Type 2 Slowly Changing Dimensions within dbt snapshots uses NULL values for the dbt_valid_to column to indicate rows that are currently valid. The default treatment of TIMESTAMP columns in MySQL 5.7 and MariaDB 10 is non-standard compared to other databases:

TIMESTAMP columns are NOT NULL and have implicit DEFAULT clauses

This non-standard behavior breaks the dbt implementation of snapshots by preventing the dbt_valid_to from being NULL
Solution
The --explicit-defaults-for-timestamp option overrides the default behavior and causes CREATE TABLE to create all TIMESTAMP columns as NULL with DEFAULT NULL attribute.
Documentation

https://dev.mysql.com/doc/refman/5.7/en/server-system-variables.html#sysvar_explicit_defaults_for_timestamp
https://mariadb.com/docs/reference/mdb/cli/mariadbd/explicit-defaults-for-timestamp/


Need a way to delete confirmed contacts.

-We need a way to delete even confirmed contacts.

@antuz123 Need to think further on this. Confirmed TC(non-Keeper & non-ward) removal has been implemented already.

Expired Keeper and 'I am the Keeper of', should also be something you can delete
Contacts should be deletable irrespective of their status - Expired, Pending or Confirmed
When Contacts are deleted, the reflection should be on the other side also. A notification is sent to the counterparty and then the contact is deleted from the other end too
On any deletion, a message is shown to the user and only on confirming the same is the action taken

-Verified this issue on Dev app vrersion 1.3.0(150)


Execute arping after ip was assigned, especially after it was moved to another node

Use https://github.com/google/gopacket

Corresponding ARP announcements should be made for each added IP (same as arping -U).
ARP support: https://github.com/google/gopacket/blob/master/layers/arp.go
example: https://github.com/google/gopacket/tree/master/examples/arpscan

Additional criteria: ARP announcement should be optional and corresponding flag should be introduced in daemonset configuration yaml(ARP_ANNOUNCEMENT). It should be enabled by default.

A simple CLI argument will be enough, so I could run daemonset with the following:
        command:
          - ipmanager
          - --iface=lo
          - --logtostderr
          - --disable-arping


encoding/csv: Incorrectly parse records with \r as the record separator

Before filing a bug, please check whether it has been fixed since the
latest release. Search the issue tracker and check that you're running the
latest version of Go:

Run "go version" and compare against
http://golang.org/doc/devel/release.html  If a newer version of Go exists,
install it and retry what you did to reproduce the problem.

Thanks.

What does 'go version' print?
go version go1.2.1 darwin/amd64

(the bug is probably in other versions, given that src/pkg/encoding/csv/reader.go
explicity does not
handle and single \r (lines 180-193)

What steps reproduce the problem?
If possible, include a link to a program on play.golang.org.

http://play.golang.org/p/dr_cz65h_u

(changing the delimiter to \n fixes the problem

1. Read a CSV file with \r as the record delimiter

What happened?
http://play.golang.org/p/dr_cz65h_u

Record 0, []string{"One", "Two", "Three\rFour",
"Five", "Six\rSeven", "Eight", "Nine"}

What should have happened instead?

Record 0, []string{"One", "Two", "Three"}
Record 1, []string{"Four", "Five", "Six"}
Record 2, []string{"Seven", "Eight", "Nine"}

Please provide any additional information below.

aarrggrggrgrgrg! I'm sure I'm just screaming into the wind here, but the obvious solution would be to add a field to the reader to denote the type of line ending expected, or even add an option to deal with either. .csvs of both types have existed for 10s of years. There are many, many ways to deal with this without breaking backward compatibility


Automatic resizing for smaller devices

Hi again guys.
I am having a problem making my app layout look good on some screens I made by hand on smaller devices. The app is designed for iPhone 6 and I am testing on iPod Touch 5g. Is there a recommended way of doing this? I'm not sure how to set up RMQ so that the views auto resize but it looked like this was supposed to happen by default. I would provide screen shots but they kind of give our app away.  The basic problem is that stuff is too big and it is leaking off the edge. Do I have to detect each screen size and provide sizes for each? That seems... tedious.
Anyway thanks again for a great library and I hope I'm not taking up too much of your guys time. Lol

It's hard to guess without some code. But here are some quick ideas that may help.  How are you setting the width of the view or the height?
I often use 'screen_width' which will be wider on the larger devices and smaller on the smaller. If you don't want the full width you can take some padding off etc

I am setting the t and l keys in the frame and the height and width. I will
try screen width. But what you're saying makes sense I think. Use the
screen width to calculate the values? Right now our designer is designing
for the other screens which might be over kill but will probably help me do
this properly.
On Monday, July 6, 2015, David Larrabee<EMAIL_ADDRESS>wrote:

It's hard to guess without some code. But here are some quick ideas that
may help. How are you setting the width of the view or the height?
I often use 'screen_width' which will be wider on the larger devices and
smaller on the smaller. If you don't want the full width you can take some
padding off etc
—
Reply to this email directly or view it on GitHub
https://github.com/infinitered/redpotion/issues/126#issuecomment-118806542
.

so that means that no matter what device you are on you are going to go 47 points left and make a view 281 points wide
the iphone 4 and 5 are 320 points wide, 281 + 47 ends up being 328 points wide (8 points larger than the view you are working with)
Are you just trying to make a 47 margin on both sides? 281 + (47 * 2) = 375, the size of the 6
its worth mentioning that on the 6Plus this wouldnt be centered either, because we have 414 points to work with there.
assuming you are just trying to add 47 points padding to both sides, but then use the rest of the area to center the view....you could do something like this.
st.frame = { t: 107, w: screen_width, padding: {l:47, r:47} }
we have to send the full hash to padding because you dont want the same padding on top and bottom (I assume...)
you could do the math yourself too, if you prefer....
st.frame = {t: 107, l: 47, w: screen_width - (47 * 2) }
this should look the same on all devices

Thanks David. I will try this soon and let you know how it worked out for
me.
On Mon, Jul 6, 2015 at 5:59 AM, David Larrabee<EMAIL_ADDRESS>wrote:

so that means that no matter what device you are on you are going to go 47
points left and make a view 281 points wide
the iphone 4 and 5 are 320 points wide, 281 + 47 ends up being 328 points
wide (8 points larger than the view you are working with)
Are you just trying to make a 47 margin on both sides? 281 + (47 * 2) =
375, the size of the 6
its worth mentioning that on the 6Plus this wouldnt be centered either,
because we have 414 points to work with there.
assuming you are just trying to add 47 points padding to both sides, but
then use the rest of the area to center the view....you could do something
like this.
st.frame = { t: 107, w: screen_width, padding: {l:47, r:47} }
we have to send the full hash to padding because you dont want the same
padding on top and bottom (I assume...)
you could do the math yourself too, if you prefer....
st.frame = {t: 107, l: 47, w: screen_width - (47 * 2) }
this should look the same on all devices
—
Reply to this email directly or view it on GitHub
https://github.com/infinitered/redpotion/issues/126#issuecomment-118832966
.

Same things as @squidpunch said, but "fr (from right)" works too:  `st.frame = {l: 47, fr: 47}'
The only place I've had issues with this is on iPhone 6 & 6+ while inside a UITableViewCell.  And the work around for that is to style the cell & contentView explicitly with {w: screen_width}.
Seems like an extra step (and it is), but will ensure the child controls you add to your UITableViewCell will have the proper super_width.   I've been meaning to dig into why, but I've been too swamped.  :non-potable_water:

I know why. :)
UitableViewCells are created internally with a width of 320. We have no control over this. When they're laid out on the screen they're auto resized to the screen width. It's not a problem when the cell is reused. If we were using auto layout it would all just work. But since we're using explicit layouts when the cell is styled it think me its got a width of 320.

and yeah, steve's suggestion is yet another way, I always forget that you can just l and fr together like that!
@markrickert @skellock if its a custom cell you can get around that bug too by styling the content view

Yeah, :l and :fr is a good way to go. And yes you should always set the cell width to the screenwidth or the parent width. My 2-cents.

hey guys. thanks. this is working well.
i try similar stuff and it doesnt work with height. is there a recommendation on height?

the full screen height of the device in question is available as screen_height in the stylesheet.  Are you saying that you want a view that is different heights depending on the device?
you can also set the height to half which would be half the screen size.  I dont think I've ever changed the height of views based on phone size, I usually go with more like a flow (add view, then below that view, and on an on - then the view is as big as it may need to be)
maybe an example of the problem would be easier to help you, if this doesnt match up?

Hi David,
Thanks for the response. I was not setting the actual height of any views
with screen height. What I did end up needing it for was some minor
differences between iPhone 5 and 6 in the "top" setting of some frames.
There may be a better way to do this like the one you just mentioned. But
for now this has worked.
If that sounds completely terrible, please stop me and I will consider
switching to the "this-below-that" style you mentioned :D
Thanks,
Jonathan
On Wed, Jul 15, 2015 at 7:22 AM, David Larrabee<EMAIL_ADDRESS>wrote:

the full screen height of the device in question is available as
screen_height in the stylesheet. Are you saying that you want a view that
is different heights depending on the device?
you can also set the height to half which would be half the screen size.
I dont think I've ever changed the height of views based on phone size, I
usually go with more like a flow (add view, then below that view, and on an
on - then the view is as big as it may need to be)
maybe an example of the problem would be easier to help you, if this
doesnt match up?
—
Reply to this email directly or view it on GitHub
https://github.com/infinitered/redpotion/issues/126#issuecomment-121614835
.

@jsilverMDX its not terrible, there is always more than one way to do something :)  If that works for you and you like the implementation then there is no harm.  You can definitely check device types and set tops based on different devices, it really depends on the layout you are rendering.  For me most of my screens start at the top, are X points tall, and then the next view goes below with a bit of a padding so that works for any screen size (though it could end up being empty on the bottom on something really long like an ipad or the iphone 6 - if it only had 2 views.  So I could totally see how you may want to space things differently or make them larger per device type - its all about the final effect you want to implement.  We here at RedPotion (and the included libraries we are choosing) try to make it easy and flexible to do what you want.  Glad you got it working!

We here at RedPotion...

You make it sound like we're all organized and responsible or something. lol

corporate life taking over lately, it bleeds over sometimes :hamster:

No worries guys. We had good success on this so far. Thanks everyone :) :+1:

@jsilvermdx glad to help!

@jsilvermdx glad to help!

Thanks
On Thursday, July 30, 2015, David Larrabee<EMAIL_ADDRESS>wrote:

@jsilvermdx https://github.com/jsilvermdx glad to help!
—
Reply to this email directly or view it on GitHub
https://github.com/infinitered/redpotion/issues/126#issuecomment-126316015
.


fix issue 13325 - Supplemental change for the pull 774

https://issues.dlang.org/show_bug.cgi?id=13325

Rename Vector rule to TypeVector
Fix BasicType rule in declaration.dd to be consistent with grammar.dd

@Hackerpilot How about this?

This looks reasonable.

Thanks.


Vec<Vec>

Sorry for the deluge of questions! Is it possible to return Vec<Vec<SomeStruct>> from a bridge function? It says the inner Vec is not a supported type. We've tried various things (currently resorting to a single-field struct which kind of sucks).

You can wrap either the outer or inner vec in a struct. Inner vec (probably what you found):
#[cxx::bridge]
mod ffi {
    struct VecT {
        vec: Vec<T>,
    }
    extern "Rust" {
        type T;
        fn f() -> Vec<VecT>;
    }
}

Outer vec, which may be more convenient depending on what this is interacting with on the Rust side:
struct VecVecT {
    vec: Vec<Vec<T>>,
}

#[cxx::bridge]
mod ffi {
    extern "Rust" {
        type T;
        type VecVecT;
        fn get(&self, i: usize) -> &Vec<T>;

        fn f() -> Box<VecVecT>;
    }
}

You can wrap either the outer or inner vec in a struct. Inner vec (probably what you found):
#[cxx::bridge]
mod ffi {
    struct VecT {
        vec: Vec<T>,
    }
    extern "Rust" {
        type T;
        fn f() -> Vec<VecT>;
    }
}

Outer vec, which may be more convenient depending on what this is interacting with on the Rust side:
struct VecVecT {
    vec: Vec<Vec<T>>,
}

#[cxx::bridge]
mod ffi {
    extern "Rust" {
        type T;
        type VecVecT;
        fn get(&self, i: usize) -> &Vec<T>;

        fn f() -> Box<VecVecT>;
    }
}

Ah wrapping the outer Vec is nicer but we need to be able to access the contents from C++ unfortunately. I presume supporting Vec<Vec<>> and other nested types isn't really hard, it just hasn't been done yet?
Thanks for all the help!

Ah wrapping the outer Vec is nicer but we need to be able to access the contents from C++ unfortunately. I presume supporting Vec<Vec<>> and other nested types isn't really hard, it just hasn't been done yet?
Thanks for all the help!


type-level width

Type of issue: feature request
Impact: API modification
Phase: proposal
Currently, Chisel manages the width of Data at value-level. So we write modules like this:
class SomeModule extends Module {
  val io = IO(new Bundle {
    val rx = Input(UInt(8.W))
    val tx = Output(UInt(1.W))
  })

This is fine while we write modules as class, but when I try to write the same thing as a function, a problem arises, because I can't specify the width of UInt.
def someModule(rx: UInt(8.W)): UInt(1.W) = {
  // syntax error!
}

So we are forced to write modules as class, but there's too much boilerplate IMO...
With Scala 3 supporting match types, we can implement type-level natural numbers and its operations(+, -, *, max, min, etc...) quite easily, so I think we can provide API like this:
def someModule(rx: UInt[W8]): UInt[W1] = {
  // valid syntax
}

class SomeModule extends Module {
  val io = IO(new Bundle {
    val rx = Input(UInt[W8])
    val tx = Output(UInt[W1])
  })
}

With this API we can get a better errors from IDE because a mismatch of width emerges at compile-time.
What do you guys think? Will this change cause other problems? One thing I could imagine is that width-inference of literals won't work? I mean there's no way we could determine the width of a value "b11111" at type-level...
I know this is a big change so I don't think we can implement it soon, but when we move to Scala 3, backward-compatibility breaks anyway, so that time is the good timing to do a big change I think...

Not having widths in the type system has been one of my biggest bugbears. Moving to dependent or refinement types for widths has always seemed logical to me.
One thing to note is that even when you do the class, technically the same problem is there. The Scala types of rx and tx are "just" UInt. There's no width information exposed to the Scala compiler.
class SomeModule extends Module {
  val io = IO(new Bundle {
    val rx: UInt = Input(UInt(8.W))
    val tx: UInt = Output(UInt(1.W))
  })

There would also need to be some sentinel value for unknown width (as having modules with purely unknown widths, e.g., an adder) are useful for library writers.
The space of this problem is related to the "Chisel Type" vs. "Hardware Type" split where this information is tracked as global, mutable state in the Builder. It may be similarly beneficial to know if something is hardware that you can connect to as opposed to relying on run-time Builder information to figure it out:
val r: HW[UInt[1]] = Reg(UInt[1])

Conversely, a problem here is that this is pushing a lot of stuff into the Scala type system. (Completely contradicting what I said before!) If you start pushing more stuff into the Scala type system, then there's less control over it by the actual compiler infrastructure. Ideally, you'd like to be able to express the API you want here (widths in the type system) where the resolution is handled by a well-structured hardware IR where the act of constructing this results in errors back to the user immediately.
This is a weird design space and I haven't fully grokked the trade-offs.
As a cool comparison, DFiant does it just like you suggest with widths, but I think it pulls in the Shapeless library to do it as opposed to leaning on Scala 3.

Something that might help with your immediate problem of wanting to write a function:
def someModule(rx: UInt): UInt = {
  require(rx.getWidth == 8)
  // your code here
}

Not that the width of rx may not actually be defined in the Chisel frontend, since most of the width inference (even for trivia things) happens at the firrtl level. So encoding a struct requirement like an 8bit width could be too restrictive. Chisel embraces more of a pythonic approach when it comes to widths, i.e., just work with a UInt and if the width is insufficient, one of your operations is going to cause an error from firrtl.

More long term your idea is pretty awesome and I think we would appreciate it if you wanted to prototype that API with Scala 3. However, just as a heads up, it will probably take years and a new major version (Chisel 4) to make such a big API change that will break all currently existing Chisel code.

make such a big API change that will break all currently existing Chisel code.
That’s what I’m worry about :(
We already have a compatibility layer to Chisel2, and users are reluctant to port to newer version of tool, I think that might be the greatest barrier :(
I think the step might be
droping support to 2.11;
supporting 3.00;
droping support to 2.12;
adding another type system layer;
porting original type system to new one as a compatibiliy layer;
deprecate original one.

But the very first thing, “deprecating Chisel2 grammar” seems impossible to happen, since the dependence of RocketChip :(


Support for adding extra properties not included in the message template

This issue was raised in the serilog issue tracker and a solution provided there. Is there a way to add additional properties not included in the log message using LibLog?

See the Wiki for how to use OpenMappedContext: https://github.com/damianh/LibLog/wiki/Using
It hits the Serilog LogContext: https://github.com/serilog/serilog/wiki/Enrichment#the-logcontext


Unclear item details in the request

Thanks for providing this api, associated docs, and the opportunity to provide feedback.

Top of page section HTTP request has some odd endpoints. Should these not end with items/{itemId}:/{item-path}:/createUploadSession?

Request section is unclear if the {item-path} should be equal to the item.name and include an extension. Through testing seems to accept an extension. Also unclear what happens if the name provided in the body differs from that in the url.

The use of the body in the request is quite unclear and could use more examples (such as how to include fileSystemInfo, also there seem to be a few issues with it:

Including<EMAIL_ADDRESS>"microsoft.graph.driveItemUploadableProperties" results in a 400 error (tested using OneDrive for Business)
Including size results in a 400 error (tested using OneDrive for Business)

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

ID: c42d74d1-8c6f-5282-9fba-30fdd6449362
Version Independent ID: 88bd8f18-730d-6970-8a02-641346661404
Content: Resumable file upload - Microsoft Graph v1.0
Content Source: api-reference/v1.0/api/driveitem-createuploadsession.md
Product: sharepoint
Technology: microsoft-graph
GitHub Login: @JeremyKelley
Microsoft Alias: MSGraphDocsVteam

As part of a repository clean up effort we are closing older issues. If this issue remains, please: open a new issue, reference this issue, and provide any additional details that may help in resolution. Thank you for your understanding as we work to improve our responsiveness.


Release v225

To pick up:

#1404
#1405
#1407

$ make publish
Deploy as version: v225 [y/n]? y

Auto packing the repository in background for optimum performance.
See "git help gc" for manual housekeeping.
Tagging commit 01d50e12230b89d450c6a65491bf119e66674c9f with v225...
Total 0 (delta 0), reused 0 (delta 0), pack-reused 0
To https://github.com/heroku/heroku-buildpack-python
 * [new tag]         v225 -> v225

Publishing to the buildpack registry...
Two-factor code: ********************************************
Publishing buildpack heroku/python... done
Publishing buildpack was successful

Version  Released At               Status
───────  ────────────────────────  ─────────
225      2023-02-08T14:49:00.375Z  published


feat: increment version

BREAKING CHANGE: for new version

:tada: This PR is included in version 2.0.0-next.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

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


fix progress-computation in FacebookTabBar

The progress should be computed by using the absolute distance between each tab and the current animation-position. This enables smooth transitions from both sides rather than only during back-transitions.

@flxbe one line fix is much better. I'll merge if you reopen


Meeting Updates from 1/29/2024

This PR implements the cosmetic updates—as well as a few related bugs—from yesterday's meeting, including:

Make table borders darker (to see when both are red)
Remove n fields counter
Make the accordions less obtrusive (remove padding)
Ensure all top-level metadata is required (so no exclusion is possible)
Remove control + control_description + comments everywhere
Replace checkbox with “Skip” button
Fix validation using Species.sex on the Subject Table as a test case
Ensure you can submit global metadata with loose requirements (i.e. not validated when empty)

Remaining issues that can be tackled in a follow-up include:

Provide a better description for what resolution means
Ensure that DfOverF—as well as the other nested Ophys properties—cannot be skipped (required, if present, on the NeuroConv side)
Provide a test case for multiple imaging planes

Make the accordions less obtrusive (remove padding)

Still looks like it could go down by another increment on this

Also tests failing; does that represent a back-compatibility break?

represent

Since the proper validation has been restored, the tests were failing because ElectrodeColumns was not specified on the test data but now we're correctly trying to check this to see if all our columns are valid.
I've just ignored the check if ElectrodeColumns is blank.

Good here? Turns out I'd been holding it at a min-height value which kept some "padding" despite removing any explicit declaration.

The height is still able to grow to fit auxiliary elements like the Skip button.

Good here? Turns out I'd been holding it at a min-height value which kept some "padding" despite removing any explicit declaration.

Better, maybe one more increment?
Also is it possible to make that box a bit darker gray than the background?
Also note that for DfOverF/Fluorescence/SegmentationImages they weren't supposed to be 'skippable'; that change was only for the source data selector

Better, maybe one more increment?
I can remove the padding entirely, but this will make things difficult to read and click. The previous image was at 5px at the top and bottom.

Here is 0px.

This is 3px.

Also note that for DfOverF/Fluorescence/SegmentationImages they weren't supposed to be 'skippable'; that change was only for the source data selector
I thought we'd agreed that this is best dealt with through updates to the way that NeuroConv handles the schema. I can patch over this on the GUIDE end, but I didn't want to forget that we have some issues with the schema at a lower level.

I thought we'd agreed that this is best dealt with through updates to the way that NeuroConv handles the schema. I can patch over this on the GUIDE end, but I didn't want to forget that we have some issues with the schema at a lower level.

What specific issue on the NeuroConv side? If the NeuroConv schema request returns it we can proceed to assume it's required for the GUIDE

Is this dark enough? Happy to take an rgb value from you, this is rgb(235, 235, 235)

What specific issue on the NeuroConv side? If the NeuroConv schema request returns it we can proceed to assume it's required for the GUIDE

As Ben mentioned yesterday, if there's no way to not pass DfOverF or any of the other properties (e.g. Subject, Ophys, etc.) to NeuroConv, the schema should include it in the required list.
This would require dynamically changing the returned schema, which may present a more substantial change that we can readily implement now.

Also once the accordion is opened, it doesn't matter how small the padding was because the content pushes all the others beneath the fold.
IMO the size of these dropdown headers now aren't well-distinguished from the content, making it all more difficult to process once open.

Is this dark enough? Happy to take an rgb value from you, this is rgb(235, 235, 235)

This is pretty bizarre... Maybe it's happening with your screenshots too, but when I take a screenshot of what it looks like to me the gray gets whitened

(the one that looks gray there is from me hovering over it, which was even a shade darker than the default grayness)
On my screen, the current gray looks OK
3px on the padding looks good

Alright just pushed an update to require all sub-levels of the Ophys metadata, if included in the schema.

Thanks; it will take quite some thought about how to do something similar on the NeuroConv side since the way you interact with metadata is starting to become very very different. For example, if the key reference to a plane changes in the metadata instance (which is perfectly allowed in the programmatic side) you'd also have to update the requirements to exactly match that update to the pattern property in the schema I think. Which would be really weird
In the end I think they are becoming two separate things: the GUIDE is a very limited scoped way of altering metadata fields on a fixed instance of the metadata dict, constrained by the default schema for each interface. So I think saying the required = properties here specific to the GUIDE is perfectly OK

IMO the size of these dropdown headers now aren't well-distinguished from the content, making it all more difficult to process once open.

Fixable by making the font size of the headers slightly larger, no?

This is pretty bizarre... Maybe it's happening with your screenshots too, but when I take a screenshot of what it looks like to me the gray gets whitened

Hmmm weird. I used a color picker on your image and it's still the original rgb(240,240,240). Must be a perceptual illusion as a result of all the other grays in the application.

Fixable by making the font size of the headers slightly larger, no?

Actually, it looks best to me if I reset the location of the colored status bar to the bottom (rather than the left). Changing the font size will start messing with the visual balance, requiring more padding to look intentionally designed.

Actually, it looks best to me if I reset the location of the colored status bar to the bottom (rather than the left). Changing the font size will start messing with the visual balance, requiring more padding to look intentionally designed.

Oky doky
Just keep in mind I'm still of the opinion that top level tabs (like the way the pydata docs theme looks) instead of accordions would remedy all of these issues

Right. I'll add that back to my to-dos and see if I can get that implemented shortly.

Right. I'll add that back to my to-dos and see if I can get that implemented shortly.

Sounds good, no rush if it's tricky. Was just a dream
The rest of these cosmetic updates look/feel good


Download a simple soundtrack - fails

youtube-dl fresh installed with current debian version jessie -> up to date!
in a specific folder -> have all rights with my user /same issue with root-privileges
please contact me if you need further information
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['-v', '--extract-audio', '--audio-format', 'mp3', 'https://www.youtube.com/watch?v=H_t5pWaQKLE']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2014.08.05
[debug] Python version 2.7.9 - Linux-3.16.0-4-amd64-x86_64-with-debian-8.7
[debug] Proxy map: {}
[youtube] Setting language
[youtube] H_t5pWaQKLE: Downloading webpage
[youtube] H_t5pWaQKLE: Downloading video info webpage
[youtube] H_t5pWaQKLE: Extracting video information
[youtube] H_t5pWaQKLE: Encrypted signatures detected.
WARNING: unable to extract html5 player; please report this issue on http://yt-dl.org/bug
[youtube] {43} signature length 41.42, html5 player None
ERROR: Signature extraction failed: Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 479, in _decrypt_signature
video_id, player_url, s
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 356, in _extract_signature_function
raise ExtractorError('Cannot identify player %r' % player_url)
ExtractorError: Cannot identify player u'https://s.ytimg.com/yts/jsbin/player-en_US-vflQBHHdn/base.js'; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.
; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.
Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 479, in _decrypt_signature
video_id, player_url, s
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 356, in _extract_signature_function
raise ExtractorError('Cannot identify player %r' % player_url)
ExtractorError: Cannot identify player u'https://s.ytimg.com/yts/jsbin/player-en_US-vflQBHHdn/base.js'; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.
Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 516, in extract_info
ie_result = ie.extract(url)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 170, in extract
return self._real_extract(url)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 860, in _real_extract
encrypted_sig, video_id, player_url, age_gate)
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 489, in _decrypt_signature
u'Signature extraction failed: ' + tb, cause=e)
ExtractorError: Signature extraction failed: Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 479, in _decrypt_signature
video_id, player_url, s
File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 356, in _extract_signature_function
raise ExtractorError('Cannot identify player %r' % player_url)
ExtractorError: Cannot identify player u'https://s.ytimg.com/yts/jsbin/player-en_US-vflQBHHdn/base.js'; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.
; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.

You are using an outdated version of youtube-dl. Read this FAQ entry if you have troubles updating.


Add explicit constructor for the ExcelReaderConfiguration

This change is needed to be able use library inside Blazor (mono/wasm).
Due to way how it was implemented, the application attempt to get
encoding 1251, which is not available inside Blazor runtime.
By providing spearate constructor, it is possible to workaround issue
in the environments which does not have codepage 1252

Hi,
First, have you tried to take on a dependency on the System.Text.Encoding.CodePages package and register the codepage providers in your Blazor project? From what I understand, it should work the same as for .NET Core
The code pages are vital for parsing xls correctly, so I want to ensure we have exhausted all options before considering a workaround like in this PR.

@andersnm Yes. I do try. See https://github.com/aspnet/Blazor/issues/1338 for more details

@andersnm I indeed create isolated repro and issue does not exhibit itself. So it is something different on my side and not generic error which should be fixed in you library. Sorry for taking you time.


Add bower.json file

Added bower.json file which defines the main file used. This will allow wire dependency tasks (grunt, gulp) to wire just the correct files needed for tota11y to run.

[clabot:check]

Hey @tricinel. Thanks for taking the time to put this together. Did this come out of your own necessity to have a bower.json for a project? I ask because we made a conscious decision not to include a bower.json in tota11y because we didn't think it would be very useful.

Hi @jdan, I like the ease of use when putting together a workflow with bower and gulp. With gulp, you can look through your bower_components (or where you're installing bower frontend components) and wire all the project's dependencies and inject them into your html file where needed.
For a single page app, when you want to do stuff fast, setting this up is painless and you don't have to do manual work when adding a new component. Just do bower install and gulp will inject the file(s) that new component uses. The catch is that the new component needs to have a bower.json file with the "main" property so that the gulp plugin will know which file to inject.
For me, it is useful, hopefully I'm not the only one :)

Awesome - thank you for the explanation :) I left a few comments inline, but I'm happy to merge this to make it easier to consume.

Awesome. Let me know if you want me to make those changes to bower.json (version number and homepage change).
Cheers!

@tricinel Yes if you don't mind! I can merge right after. Thanks again.

Done. No problem, thank you for the tool :)

Woohoo! :star2:

Awesome! Back to building stuff :+1:
Thanks.


[Feat] Add the plugins as submodules

@nraboy did a great pull request for ng-cordova-oauth on this project ( #841 ) adding it as a submodule, it's a great idea. Don't you think it's a better idea to do that with all the others plugins? Doing that it's better to make ng-cordova updated with all of them? What do you think

Thanks for the compliment!  I'm not so certain this would work in all cases though.  I specifically designed ng-cordova-oauth for use with AngularJS, but others are wrapped inside ngCordova rather than the plugin itself.
Worth looking into though :-)

That's bad, :stuck_out_tongue: If we could do that it will be great, it will be easier to maintain updated.

Agreed!  Puts less need for intervention on the plugin itself as changes will be automatically piped in.

Yes... If some of the other projects are like plugins, like yours, they could starting making pull requests doing that, putting their project on submodules here.

I like the idea, one issue is stability.

I believe there is at least one easy project to do that just as a kickstart to incentive the others

This might be totally unrelated, but just a couple of hours ago our CI started failing:
Module 'ngCordovaOauth' is not available! You either misspelled the module name or forgot to load it. If registering a module ensure that you specify the dependencies as the second argument.

First time I saw this.
Were there any changes made as part of this submodule request that might have caused it?

we are also having the same issue as @nielskrijger

We are also having the same issue as @nielskrijger

Also having the same problem as @nielskrijger

I opened a issue about it (#912) @nraboy I think you should take a look too.

I saw here that @nraboy already fix it, on the next release will works again.

I think we're going to collect more data on this; think of the oAuth module as the test subject.  We need to hold off on making this a wider thing until we've got a rock solid deployment and regression check process around submodules.

@gortok do you have something in mind how to do that on deployment?

@matheusrocha89 http://38.media.tumblr.com/8eb1eb849b8b5fb2452bb74a697900a0/tumblr_mzvc3qNsII1t7xe6go1_400.gif

hahahahahaha Ok @gortok . Well I am not an expert on it, but if you need any help I will do my best. @nraboy do you have any idea how to do that? :arrow_up:

I personally don't use Gulp beyond ng-cordova, so I don't know how to do any regression checks with it.  Because we're using it for building and testing only, I think the best thing we could do in the immediate is switch to 4.x and make all tasks synchronous.  I'm actually surprised we haven't hit other issues in the build process.

@nraboy  I see 3.9 on npm; I don't see a 4.0 yet.https://github.com/gulpjs/gulp

Lame, so all the forums I've read must be referring to what is coming soon on the Gulp timeline.  I had figured it was already out.

@nraboy 4.0 i think is WIP yet, there is a branch on gulp called 4.0.

Thank you all for the fast response and the quick investigation!
I don't fully understand the cause yet, but I am truly grateful.
As submodules appear to be a recent introduction to this project, I have always been told to stay away from them.
I never asked my colleagues why, I never investigated them because I never felt the need to. I did a quick g00gle search;

http://blogs.atlassian.com/2013/05/alternatives-to-git-submodule-git-subtree/
http://blogs.atlassian.com/2013/03/git-submodules-workflows-tips/#tip5
https://codingkilledthecat.wordpress.com/2012/04/28/why-your-company-shouldnt-use-git-submodules/

I didn't read them thoroughly but it feels like submodules do have some pitfalls.
Given ng-cordova's unique project nature git submodules might very well be the right way to go.
I've not looked into ngCordova's integration strategy, but a switch to submodules appears to be more of a minor or major version upgrade rather than a patch upgrade?
If you feel the need for more information of submodules, I'll gladly assist in doing some  research. But as I've said, I'm no expert.

@nielskrijger I personally think it is dependent on contributor buy in.  For example, not all plugins included in ngCordova were built for AngularJS.  That is why people have created wrappers for them.  Because of this you cannot just go ahead and include someones Apache Cordova plugin as a submodule because if they introduce breaking changes, you won't have a way to stop it.  So one of two things would need to happen:

All plugins get split into separate ngCordova repositories, ex: ng-cordova-pluginnamehere and then that gets included in ng-cordova via a git submodule.  Then people can contribute fixes to the particular plugin rather than the entire ngCordova repository
Only plugins designed for AngularJS get pulled out and added to ngCordova.  For example my ng-cordova-oauth plugin was designed for AngularJS.  That is why it works as a submodule.

I'm very strict in my library.  I deny pull requests all the time in order to keep it working flawlessly.  Other repository owners may not be this strict.
Just my two cents.

No submodules.
Too many moving parts for little to no gain.
If people have their own plugin wrapper they want to contribute, they can make a PR to add them to ngCordova, or maintain their own repos.

Traumatic experience with submodules @mhartington?
The gain would be that ngCordova might get more buy in from developers who don't want to (a) manage and support their libraries in two repositories (b) drop their repository and only contribute to ngCordova.
Unless Ionic gets a fleet of internal developers working on this project, ngCordova will only be as good as the contributions it receives.  The last few months have proven not so good.  For example, isn't SQLite still majorly broken in ngCordova?
Just trying to help fix the shaky past that this library has had.

Unless Ionic gets a fleet of internal developers working on this project, ngCordova will only be as good as the contributions it receives.

That's true; and that's why I want people to give us PRs instead of just raising issues.  It's easy to say, "you need to do this", it's harder to say, "This is an issue; and I don't know how to help, but I want to."
ngCordova only succeeds if we do the latter.  If this were a project with one or two plugins, that'd be one thing -- but ngCordova supports lots of plugin wrappers for 64+ plugins; not all of which have community support behind them.
Submodules mean not knowing when there's an unstable build; and with the free time we have, it can't be spent tracking down unstable builds because a submodule was updated and we have no Issue tracking/commit tracking in this repository to see what changed upstream.  It'd be great if the tooling were there, but it's not.

The last few months have proven not so good. For example, isn't SQLite still majorly broken in ngCordova?

The last few months haven't been so awesome; you're right. That's partly why I stepped up.  I rely on ngCordova, and I'm not the only one.  I bet there are dozens of companies building Apps that need what ngCordova provides.  We've got to ensure ngCordova is stable. If you rely on ngCordova, then your help is desired.  Submit PRs, even if it's just a PR to make documentation better.
As far as last night's submodule fun, I won't just be merging PRs to mainline; and there will be development branches; more on that to come.   I'm still on week one here; so expect a little shakiness.
As far as SQLite goes; I'm using that plugin next, so unless someone else gets to it first, I'll be able to see pretty quickly what's broken with it.
Part of this being an open source project means that anyone who can read these pages can help make it better.   Be the change you wish to see in the world.


docs(cache) update the cache documentation

The in lua-memory cache documentation is added as well as a problem
regarding the locking mechanism.

@thibaultcha @theetrain :+1: updated


Implement WOD range check

Code up the range test described at http://data.nodc.noaa.gov/woa/WOD/DOC/wodreadme.pdf on page 47. The program file should be called WOD_range_check.py and be put into the qctests directory. It should follow the structure shown in EN_range_check.py.
This requires reading in a data file that is used each time the quality control check is run. It would therefore be most efficient to read this in once in the main program and pass the data to the quality control check.

Resources for composing this test:

lat / long associated with region code
region code associated with region name
region name associated with max / min temperatures by depth, appendix 9.1

Does appendix 9.1 listed above exist as a CSV somewhere?

Hi @BoyerWOD - is appendix 9.1 from the WOD readme doc available as a CSV file?

@BoyerWOD yes please! If an official csv or json of that table is available, that would be ideal; #78 extracts it well enough for development to proceed, but the validation there is not up to production code atm.

Please find the following min/max range files:
ftp://ftp.nodc.noaa.gov/pub/WOD/SELECT/WOD_ranges_Temperature.csv
ftp://ftp.nodc.noaa.gov/pub/WOD/SELECT/WOD_ranges_Salinity.csv
ftp://ftp.nodc.noaa.gov/pub/WOD/SELECT/WOD_ranges_Oxygen.csv
ftp://ftp.nodc.noaa.gov/pub/WOD/SELECT/WOD_ranges_Nitrate.csv
ftp://ftp.nodc.noaa.gov/pub/WOD/SELECT/WOD_ranges_Phosphate.csv
ftp://ftp.nodc.noaa.gov/pub/WOD/SELECT/WOD_ranges_Silicate.csv
Note that ranges have been updated in some cases from the User Manual
and new geographic areas have been added in the Northwest Pacific in conjunction with our regional atlas of the East Asian Seas.

I should add that we will update the one-degree geographic range distribution file and list of range areas on http://www.nodc.noaa.gov/OC5/WOD/wod_masks.html once the East Asian Seas atlas is released, probably in the next month.

Brilliant! Thanks very much @BoyerWOD; will use these to replace #78.
With that, the links in the top post above should be enough to settle this issue - will have a look myself this weekend. Thanks again!

hmm - on closer look, there seems to be a little glitch with the last 5 lines of WOD_ranges_Temperature.csv; the 5th to last doesn't have the right number of columns, and all 5 seem to repeat information for depths already reported; @BoyerWOD, I assume these lines should just be dropped?

Yes, last five lines are an artifact of a previous, imperfect run of the software which created the csv files.  I should have checked closer or erased files before rerunning.  I have posted corrected files to our ftp site:
ftp://ftp.nodc.noaa.gov/pub/WOD/SELECT/WOD_ranges_Temperature.csv
ftp://ftp.nodc.noaa.gov/pub/WOD/SELECT/WOD_ranges_Salinity.csv
ftp://ftp.nodc.noaa.gov/pub/WOD/SELECT/WOD_ranges_Oxygen.csv
ftp://ftp.nodc.noaa.gov/pub/WOD/SELECT/WOD_ranges_Nitrate.csv
ftp://ftp.nodc.noaa.gov/pub/WOD/SELECT/WOD_ranges_Phosphate.csv
ftp://ftp.nodc.noaa.gov/pub/WOD/SELECT/WOD_ranges_Silicate.csv
or you can simply ignore/remove the last five lines of each file from the previous post.

OK, that's got it, looks good - thanks again @BoyerWOD.


chore(native_dio_adapter): support cupertino_http 2.0.0

New Pull Request Checklist

[x] I have read the Documentation
[x] I have searched for a similar pull request in the project and found none
[x] I have updated this branch with the latest main branch to avoid conflicts (via merge from master or rebase)
[ ] I have added the required tests to prove the fix/feature I'm adding
[x] I have updated the documentation (if necessary)
[x] I have run the tests without failures
[x] I have updated the CHANGELOG.md in the corresponding package

Additional context and info (if any)
supports cupertino_http ^2.0.0, which has to break changes that don't affect this package.
No code changes.
Also fix typo (<= -> < )  in pubspec.yaml which would support cronet_http 2.0.0, which is not released yet and might contain breaking changes which are not compatible with this package

Is there an ETA on this release? Not pressuring, just curious. This blocks upgrades to 2.0.1 of cupertino_http


"cargo install cargo-semver-checks" fails to build since version 0.24.0 due to compile issue with cmake

Steps to reproduce the bug with the above code
Run the following command: cargo install cargo-semver-checks --version 0.24.0.
Trying to install older versions work fine.
Actual Behaviour
error: failed to run custom build command for `libz-ng-sys v1.1.15`

Caused by:
  process didn't exit successfully: `/var/folders/sd/hgkqcwv90sz_q1pqjww78nfm0000gn/T/cargo-installHyjJoV/release/build/libz-ng-sys-953997c03653e4dc/build-script-build_zng` (exit status: 101)
  --- stdout
  CMAKE_TOOLCHAIN_FILE_aarch64-apple-darwin = None
  CMAKE_TOOLCHAIN_FILE_aarch64_apple_darwin = None
  HOST_CMAKE_TOOLCHAIN_FILE = None
  CMAKE_TOOLCHAIN_FILE = None
  CMAKE_GENERATOR_aarch64-apple-darwin = None
  CMAKE_GENERATOR_aarch64_apple_darwin = None
  HOST_CMAKE_GENERATOR = None
  CMAKE_GENERATOR = None
  CMAKE_PREFIX_PATH_aarch64-apple-darwin = None
  CMAKE_PREFIX_PATH_aarch64_apple_darwin = None
  HOST_CMAKE_PREFIX_PATH = None
  CMAKE_PREFIX_PATH = None
  CMAKE_aarch64-apple-darwin = None
  CMAKE_aarch64_apple_darwin = None
  HOST_CMAKE = None
  CMAKE = None
  running: cd "/var/folders/sd/hgkqcwv90sz_q1pqjww78nfm0000gn/T/cargo-installHyjJoV/release/build/libz-ng-sys-988330083ec46cbc/out/build" && CMAKE_PREFIX_PATH="" "cmake" "/Users/henrik/.cargo/registry/src/index.crates.io-6f17d22bba15001f/libz-ng-sys-1.1.15/src/zlib-ng" "-DCMAKE_OSX_ARCHITECTURES=arm64" "-DBUILD_SHARED_LIBS=OFF" "-DZLIB_COMPAT=OFF" "-DZLIB_ENABLE_TESTS=OFF" "-DWITH_GZFILEOP=ON" "-DCMAKE_INSTALL_PREFIX=/var/folders/sd/hgkqcwv90sz_q1pqjww78nfm0000gn/T/cargo-installHyjJoV/release/build/libz-ng-sys-988330083ec46cbc/out" "-DCMAKE_C_FLAGS= -ffunction-sections -fdata-sections -fPIC --target=arm64-apple-darwin -mmacosx-version-min=13.3 -I/opt/homebrew/opt/openssl@3/include" "-DCMAKE_C_COMPILER=/usr/bin/cc" "-DCMAKE_CXX_FLAGS= -ffunction-sections -fdata-sections -fPIC --target=arm64-apple-darwin -mmacosx-version-min=13.3" "-DCMAKE_CXX_COMPILER=/usr/bin/c++" "-DCMAKE_ASM_FLAGS= -ffunction-sections -fdata-sections -fPIC --target=arm64-apple-darwin -mmacosx-version-min=13.3 -I/opt/homebrew/opt/openssl@3/include" "-DCMAKE_ASM_COMPILER=/usr/bin/cc" "-DCMAKE_BUILD_TYPE=Release"

  --- stderr
  thread 'main' panicked at /Users/henrik/.cargo/registry/src/index.crates.io-6f17d22bba15001f/cmake-0.1.50/src/lib.rs:1098:5:

  failed to execute command: No such file or directory (os error 2)
  is `cmake` not installed?

  build script failed, must exit now
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Expected Behaviour
The build should succeed.
Generated System Information
Software version
cargo-semver-checks 0.22.0
Operating system
macOS 14.4 (Darwin 23.4.0)
Command-line
/Users/henrik/.cargo/bin/cargo-semver-checks semver-checks --bugreport

cargo version
> cargo -V
cargo 1.80.0 (376290515 2024-07-16)

Compile time information

Profile: release
Target triple: aarch64-apple-darwin
Family: unix
OS: macos
Architecture: aarch64
Pointer width: 64
Endian: little
CPU features: aes,crc,dit,dotprod,dpb,dpb2,fcma,fhm,flagm,fp16,frintts,jsconv,lor,lse,neon,paca,pacg,pan,pmuv3,ras,rcpc,rcpc2,rdm,sb,sha2,sha3,ssbs,vh
Host: aarch64-apple-darwin

Build Configuration
No response
Additional Context
No response

Thanks for the report, and sorry for the frustrating situation.
It appears that one of our dependencies has started requiring that cmake be available, per the error message: is cmake not installed?
Unfortunately, it doesn't seem to me like we have any great options here:

We don't directly depend on that library — it's a transitive dependency somewhere deep in the tree — so we can't easily drop it.
There's no way to specify to cargo that cmake is required that will make cargo install work.
We could document that cmake is needed in our README, but since this is a transitive dependency, that requirement might drift over time. And the README is probably not something people will check before running cargo install anyway.
We already provide a prebuilt binary for aarch64-apple-darwin which users are encouraged to use. But obviously some users will prefer to compile the tool themselves, so there's still a problem.

What do you think we should do?

So I had a look at the repository for libz-sys if there is a known issue for that and I found https://github.com/rust-lang/libz-sys/issues/93. It's not directly related but installing cmake via brew install cmake as mentioned over there fixes the error for me.
Most people might have cmake already installed so feel free to close, or maybe have an entry in the FAQ that mentions that?

Sorry to necro this, but I stumbled upon this issue and wanted to mention that you can drop the dependency on libz-ng-sys by changing the gix features from features = ["max-performance", "revision"] to features = ["max-performance-safe", "revision"] which causes gix to use the pure rust implementation of flate2 (though I don't know how big of an impact this would be performance wise).


docusaurus and redocusaurus version compatibility

Hi Rohi, thanks for this great plugin.
I am having difficulties getting it working due to some version incompatibilities.
Could you confirm what version of redocusaurus and   docusaurus/core are compatibles?
Thanks

It's tested with version 2.3.1 in the docs website. I don't think there should be any problems with later versions, we had a lot of issues during the beta versions but the docusaurus API is pretty stable now.

Thanks so much. I got it working with 2.3.1


nixos/pam: fix deprecated gnome3 reference

Description of changes
gnome3 alias was changed from aliasing gnome to throwing which causes a nixos-rebuild issue.
Related: #192681
Diff: https://github.com/NixOS/nixpkgs/pull/192681/files#diff-29ed0f5fda84c91253503c1664d60a194a324e29518d472adf846f30b8db52e3L518
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Thanks!


Add projects questions

Add redirection to na-2018 by default
Fix some quick display issue based on the event info (landing page)

Needs splitting into master/usa

Replaced by https://github.com/CoderDojo/coolest-platform/pull/41 and https://github.com/CoderDojo/coolest-platform/pull/42


Baseline blog template - instagram card is off

Looks fixed now - this was an instagram issue anyways looks like we fixed it in the component


Using HTTP.jl for serving

using HTTP
HTTP.listen() do req::HTTP.Request
     req.target == "/" && return HTTP.Response(200, read("index.html"))
     file = HTTP.unescapeuri(req.target[2:end])
     return isfile(file) ? HTTP.Response(200, read(file)) : HTTP.Response(404)
end

does the job (needs a REPL).

add a catch for InterruptException
check if it does live updating of things?
how to interweave with the judoc(single_pass=false) ?

need to think about the workflow as well. if everything is called within a REPL, then it should be doable to trigger a refresh whenever judoc sees something new.

Bukdu can be used as well but, for the moment, it looks like browser-sync is just the fastest out there with live resync.

Put this up as open for contribution, now with browser sync delivers what you want.


Compatible scrollWidth, scrollHeight for Firefox and IE

Problem
List does not updates when scrolling with Firefox (tested version > 44.0) and IE (tested version > 11)
Analysis
Firefox always returns same document.body.scroll(Width | Height) even if scrolling size changes.
document.documentElement.scroll(Width | Height) returns correct size.
Fix
Use Math.max of two variables in body or documentElement

Can you provide an example of the current code failing? I've tried to reproduce the problem but I'm not able to.

@caseywebdev Please open https://libre.market in Firefox and use developer tool to test.

I tried scrolling very far down on that site in Firefox 47.0 and didn't notice any issues. Is there something specific I need to do to trigger the error?

@caseywebdev We are already using patched version of react-list in our project. You can check scrollHeight in developer tool > console.
For example
>> document.body.scrollHeight
943
>> document.documentElement.scrollHeight
9387

Ahh I see now. Thanks for the fix! I'll push out a new version shortly.


Bugfix/heros

Fixes an issue where heros were not displaying on new Wagtail pages
Removals

Removed heros from old WordPress pages

Changes

Updated hero molecule to just import (values are passed by wagtail)
Snuck in a simplification of the flush top modifier to the landing and sub-landing templates

Testing

you'll need to import Wagtail data to see it locally or create your own landing or sub-landing pages w/ heros
/the-bureau/ and /careers/application-process/ should not have heros (to be replaced by FCM) soonly

Review

@kave
@KimberlyMunoz
@anselmbradford
@sebworks

Screenshots
Hero (image missing due to not having assets locally)

No longer a Hero

Notes

Thanks @kave for the boost on debugging this.

Checklist

[x] Changes are limited to a single goal (no scope creep)
[x] Code can be automatically merged (no conflicts)
[x] Code follows the standards laid out in the front end playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[x] Placeholder code is flagged
[x] Visually tested in supported browsers and devices
[x] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

:+1:

:+1:


Handle errors from plugins in tablet management iterator

In #3460 the TabletManagementIterator stated calling compaction related plugins to determine it a tablet needed compaction.  If these plugins throw an exception it could interfere with other manager functions like tablet assignment.  Need to handle this situation in such a way that other manager functions can continue and the minimum relevant information needed for debugging is logged.  Since the plugins are callled for each tablet, need to be careful about spamming the logs with lots or redundant information.

Chatted w/ @dlmarion about this.  We discussed that if the TabletManagementIterator reports back an enums indicating a problem that the manager can aggregate this and emit a metric.  So if the TabletManagementIterator runs on 10 tablets servers and each sees 100 errors trying to plan compactions, then the manager could emit a metric that 1000 tablets are having problems planning compactions.


HTTPCORE-586 Improve max pool size management

@coxlinton Could you please add the JIRA ticket to the commit message?

Done. Apologies for leaving it out in the first place.

@coxlinton Great work. Thank you for all your contributions.


⚠ API - Lens has degraded performance

In cb0faae, API - Lens ($LENS_API_URL) experienced degraded performance:

HTTP code: 503
Response time: 582 ms

Resolved: API - Lens performance has improved in 35a8233 after 8 minutes.


 Max rows: 1001, max columns: 27">

The link results in this:
https://i.gyazo.com/8345e5a5d26d72dad3ca10a0bf3a79d6.png
I have tried editing the scopes, and even with the so called "admin" scopes it still gives me this error.
This spreadsheet has about 20 sheets, but the sheets are 80x8 at max. The Max rows: 1001, max columns: 27 changes whenever I edit rows, so it's somehow a counter of the rows.
I even tried copying the first sheet, the "('gas007'!GAS7)" one into a seperate spreadsheet, and it parsed just fine.
I'm curious if it has anything to do with the deprecation of OAuth.

To replicate:
Try and do .get(https://docs.google.com/spreadsheets/d/1ptJ7vHTFzsrHoL5ZsVB6D7FbZccL8K8Ob28xQ0XvYZc/edit#gid=1686701513).
This fails for some reason.
Then do .get(https://docs.google.com/spreadsheets/d/1ey4cHVM0HvYZtC4UOoV1DwzZjR-YH6EkUpiHfVQoLvw/edit#gid=0) and it goes through.
The second spreadsheet is actually seriously a copy of the first one, I even used Google's built in tools. But the last one doesn't work.

I want to add however that this is by far the best wrapper for the Google Sheets API.
After failing with this one, I ended up going through this Google's raw API, gspread and pygsheets, and they all had worse way of doing stuff that this project has gotten down.
Notably, I ended up having to download and export the sheet as a .csv using pygsheets, but because it had no options for dialect nor encoding, I had to make a seperate function that re-opened the file and replaced all commas with excel tabs, and encoded with utf-16.
So I can't wait for this problem to be resolved haha

Thanks for the detailed report (and sorry for the late response).

This spreadsheet has about 20 sheets, but the sheets are 80x8 at max.
The Max rows: 1001, max columns: 27 changes whenever I edit rows

IIUC loading the two sheets you link to goes over the current API limit for values.batchGet (assuming it is 1000 rows and 26 columns, i.e. column A to Z).
Currently we are loading all worksheets in one batch request (requesting one range per worksheet). This fails for the haa037 worksheet in 1ptJ7vHTFzsrHoL5ZsVB6D7FbZccL8K8Ob28xQ0XvYZc (which has 1070 rows and 28 columns) and for the TEMPLATE worksheet in 1ey4cHVM0HvYZtC4UOoV1DwzZjR-YH6EkUpiHfVQoLvw (which has 1027 rows and 28 columns).
Can you try if deleting empty trailing rows/columns from the worksheets or copying the wanted 80x8 area into a new worksheet replacing the old one helps to work around this?
Will investigate more into the API limits, probably need to break loading down into smaller requests where needed.

Closing stale issue (inable to reproduce).


Add vrajdesai78 profile

Add vrajdesai78.json file

Looks good, Welcome to the community @vrajdesai78

Thank you @aayush89890

Why my profile is not there on linkfree homepage ?

Why my profile is not there on linkfree homepage ?

Why my profile is not there on linkfree homepage ?

Why my profile is not there on linkfree homepage ?

Please do not spam the same comment. Your profile won't appear immediately; it may take some minutes or even more time for your profile to appear.

Please do not spam the same comment. Your profile won't appear immediately; it may take some minutes or even more time for your profile to appear.

Sorry I didn't know about that. I won't comment again.


Add modal for application deployment

Signed-off-by: MUzairS15<EMAIL_ADDRESS>Description
This PR adds Confirmation Modal in Applications Page.
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@MUzairS15 Will, you also ensure to add this update to the Designers and Filters too in both views.


⚠ Bofrost.de has degraded performance

In d6a3f75, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1081 ms

Resolved: Bofrost.de performance has improved in 91f4f15.


Threading

Hello
I wanted to ask for one is the jit compiler thread safe?
My other question is can I start with my jit compiled code new threads
and run some functions there?

Hi,
it's not really stated in documentation, but yes. The Runtime is thread-safe and Assembler/Compiler  are reentrant - you cannot use the same instance of these from multiple threads, but you can create as many instances you need in two or more threads.
Hope it answers the question.

So do I understand it correctly that the Assembler/Compiler create the instructions and the Runtime executes them in the current thread?
Which would mean that I handle all threading in my application without jit but give jit the chance to start threads via some function from me?

AsmJit never runs the generated code. Assembler/Compiler are used to generate the code stream, and Runtime is used to provide a memory where the code will be stored and relocated (you can use your own runtime if you just inherit from BaseRuntime).
I still don't know if you want to create threads from JIT code or use JIT code from threads you created before. You can do both, but I would just create threads outside of JIT and use them to run JIT code.

Well, with my questions I wanted to determine which was possible and which one you would  recommend.
So I probably do as you said and try to run it from the outside.

Yeah, I think it's just simpler to create threads from outside instead of calling the APIs to create them from a generated code, but it really depends on what you are trying to do :)

Well I try to create a jit compiler / vm for a language I am starting to create.
In which I want to make it easy to do threading


change how the frontend is built

#TICKET_NUMBER

Reviewers
Functional:
Readability:

Changes

add
remove
modify

QA steps (optional)
Notes for Reviewer

no longer needed


Readme contains wrong version tag

In your readme you use
 uses: benoitchantre/setup-ssh-authentication-action@v1
However, there is no tag v1 but only 1.0.0. (also note the missing v)
So the correct line is
 uses<EMAIL_ADDRESS>

@yubiuser Thank you for raising the issue.


Refusing to install perfmon as a dependency of itself

i get this message when i try to install.
Any idea ?
Thanks

I just had this issue, it was due to my project also being called perfmon - so i renamed that in my package.json and ran the npm install perfmon again


convolution translator consistently hangs run in mixed mode

The convolution code seems to consistently hang in the same spot, in some sort of infinite loop (100% GPU usage) when run with horovod on 4 GPUs. The system is using cuda 10.1, cudnn7.5, nccl 2.4.2 on ubuntu16.04.5
cuda-repo-ubuntu1604-10-1-local-10.1.105-418.39_1.0-1_amd64.deb
libcudnn7_<IP_ADDRESS>-1+cuda10.1_amd64.deb
libcudnn7-dev_<IP_ADDRESS>-1+cuda10.1_amd64.deb
libcudnn7-doc_<IP_ADDRESS>-1+cuda10.1_amd64.deb
nccl-repo-ubuntu1604-2.4.2-ga-cuda10.1_1-1_amd64.deb
from the logs
levinth@csig6ztmoxl003:~/OpenSeq2Seq$ tail conv*
==> conv_cuda101_75_mpiexec_2.log <==
*** Global step 157500: ***     Train loss: 0.8891
time per step = 0:00:0.316
*** Global step 157600: ***     Train loss: 1.0271
time per step = 0:00:0.313
*** Global step 157700: ***     Train loss: 0.9281
time per step = 0:00:0.310
*** Global step 157800: ***     Train loss: 0.9351
time per step = 0:00:0.315
*** Global step 157900: ***     Train loss: 1.2120
time per step = 0:00:0.320
==> conv_cuda101_75_mpiexec.log <==
*** Global step 157500: ***     Train loss: 0.8886
time per step = 0:00:0.303
*** Global step 157600: ***     Train loss: 0.9680
time per step = 0:00:0.307
*** Global step 157700: ***     Train loss: 0.9262
time per step = 0:00:0.310
*** Global step 157800: ***     Train loss: 1.0593
time per step = 0:00:0.304
*** Global step 157900: ***     Train loss: 1.2313
time per step = 0:00:0.300

left this out...this was run in mixed mode with loss scaling
(ie uncommenting the 2 lines and commenting out the fp32 declaration for dtype)

David, can you attach  a complete log please?

do you want me to attach then to the bug as well?
On Mon, Mar 18, 2019 at 10:34 AM Boris Ginsburg<EMAIL_ADDRESS>wrote:

David, can you attach a complete log please?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NVIDIA/OpenSeq2Seq/issues/385#issuecomment-474022413,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIUuTx1Fx5BgEQxmvpK7LgqyEBPSdKgUks5vX84dgaJpZM4b6RRA
.

yes, please.

conv_cuda101_75_mpiexec.log
conv_cuda101_75_mpiexec_2.log

same thing happens (ie hangs in exactly the same place) when run in FP32 mode on 4 V100s
conv_cuda101_75_mpiexec_fp322.log

Do use “train” only mode, or ״train_eval”?
Does the code hang in the case when there is only 1 GPU without Horovod?
Can you re-run with checkpoint_steps=100000, please?

I am running with train only..I find train_eval produces too much output
I will run on 1 gpu without horovod and checkpoint 100K
d
On Sat, Mar 23, 2019 at 10:06 AM Boris Ginsburg<EMAIL_ADDRESS>wrote:

Do use “train” only mode, or ״train_eval”?
Does the code hang in the case when there is only 1 GPU without Horovod?
Can you re-run with checkpoint_steps=100000, please?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/NVIDIA/OpenSeq2Seq/issues/385#issuecomment-475886969,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIUuT2eIW0_ZX3giagb9DJD1oSnsarnkks5vZl74gaJpZM4b6RRA
.

well.that was not a rousing success
TypeError: Failed to convert object of type <class 'dict'> to Tensor.
Contents: {'source_tensors': [<tf.Tensor 'IteratorGetNext:0' shape=(?, ?)
dtype=int32>, <tf.Tensor 'IteratorGetNext:1' shape=(?,) dtype=int32>],
'target_tensors': [<tf.Tensor 'IteratorGetNext:2' shape=(?, ?)
dtype=int32>, <tf.Tensor 'IteratorGetNext:3' shape=(?,) dtype=int32>]}.
Consider casting elements to a supported type.
On Sat, Mar 23, 2019 at 10:19 AM David Levinthal<EMAIL_ADDRESS>wrote:

I am running with train only..I find train_eval produces too much output
I will run on 1 gpu without horovod and checkpoint 100K
d
On Sat, Mar 23, 2019 at 10:06 AM Boris Ginsburg<EMAIL_ADDRESS>wrote:

Do use “train” only mode, or ״train_eval”?
Does the code hang in the case when there is only 1 GPU without Horovod?
Can you re-run with checkpoint_steps=100000, please?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/NVIDIA/OpenSeq2Seq/issues/385#issuecomment-475886969,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIUuT2eIW0_ZX3giagb9DJD1oSnsarnkks5vZl74gaJpZM4b6RRA
.

Looks like we have a bug in convs2s :(
On Mon, Mar 25, 2019 at 8:16 AM David Levinthal Ph.D. <
<EMAIL_ADDRESS>wrote:

well.that was not a rousing success
TypeError: Failed to convert object of type <class 'dict'> to Tensor.
Contents: {'source_tensors': [<tf.Tensor 'IteratorGetNext:0' shape=(?, ?)
dtype=int32>, <tf.Tensor 'IteratorGetNext:1' shape=(?,) dtype=int32>],
'target_tensors': [<tf.Tensor 'IteratorGetNext:2' shape=(?, ?)
dtype=int32>, <tf.Tensor 'IteratorGetNext:3' shape=(?,) dtype=int32>]}.
Consider casting elements to a supported type.
On Sat, Mar 23, 2019 at 10:19 AM David Levinthal <
<EMAIL_ADDRESS>wrote:

I am running with train only..I find train_eval produces too much output
I will run on 1 gpu without horovod and checkpoint 100K
d
On Sat, Mar 23, 2019 at 10:06 AM Boris Ginsburg <
<EMAIL_ADDRESS>wrote:

Do use “train” only mode, or ״train_eval”?
Does the code hang in the case when there is only 1 GPU without Horovod?
Can you re-run with checkpoint_steps=100000, please?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<
https://github.com/NVIDIA/OpenSeq2Seq/issues/385#issuecomment-475886969>,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AIUuT2eIW0_ZX3giagb9DJD1oSnsarnkks5vZl74gaJpZM4b6RRA

.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/NVIDIA/OpenSeq2Seq/issues/385#issuecomment-476242942,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHMWqbMbq5D8Kl6MV3sYaUTD86fT_Iqcks5vaOhWgaJpZM4b6RRA
.

it fails in the same way with both mixed and fp32 modes on 1 gpu with
horovod disabled (100K checkpointing)
On Mon, Mar 25, 2019 at 9:41 AM Boris Ginsburg<EMAIL_ADDRESS>wrote:

Looks like we have a bug in convs2s :(
On Mon, Mar 25, 2019 at 8:16 AM David Levinthal Ph.D. <
<EMAIL_ADDRESS>wrote:

well.that was not a rousing success
TypeError: Failed to convert object of type <class 'dict'> to Tensor.
Contents: {'source_tensors': [<tf.Tensor 'IteratorGetNext:0' shape=(?, ?)
dtype=int32>, <tf.Tensor 'IteratorGetNext:1' shape=(?,) dtype=int32>],
'target_tensors': [<tf.Tensor 'IteratorGetNext:2' shape=(?, ?)
dtype=int32>, <tf.Tensor 'IteratorGetNext:3' shape=(?,) dtype=int32>]}.
Consider casting elements to a supported type.
On Sat, Mar 23, 2019 at 10:19 AM David Levinthal <
<EMAIL_ADDRESS>wrote:

I am running with train only..I find train_eval produces too much
output
I will run on 1 gpu without horovod and checkpoint 100K
d
On Sat, Mar 23, 2019 at 10:06 AM Boris Ginsburg <
<EMAIL_ADDRESS>wrote:

Do use “train” only mode, or ״train_eval”?
Does the code hang in the case when there is only 1 GPU without
Horovod?
Can you re-run with checkpoint_steps=100000, please?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<
https://github.com/NVIDIA/OpenSeq2Seq/issues/385#issuecomment-475886969
,
or mute the thread
<

https://github.com/notifications/unsubscribe-auth/AIUuT2eIW0_ZX3giagb9DJD1oSnsarnkks5vZl74gaJpZM4b6RRA

.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/NVIDIA/OpenSeq2Seq/issues/385#issuecomment-476242942
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AHMWqbMbq5D8Kl6MV3sYaUTD86fT_Iqcks5vaOhWgaJpZM4b6RRA
.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/NVIDIA/OpenSeq2Seq/issues/385#issuecomment-476281254,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIUuT7RtskssQjwkGeZlTqKsP0CwXUPyks5vaPwlgaJpZM4b6RRA
.


GH-39326: [C++] Flaky DatasetWriterTestFixture.MaxRowsOneWriteBackpresure test

Rationale for this change

This patch reduce the number of open files in testing first. I've verify the test in 14.0.2, it hangs forever.
What changes are included in this PR?

Change the test file number from 100 to 20
Are these changes tested?

Already
Are there any user-facing changes?
no

Closes: #39326

@kou I've read the code, I think the previous code I found could merely hang, because even a IO thread call WriteRecordBatch in callback, it might do Write without blocking, because actually Write is happened in ScheduleBatch. I'd like to first decrease the count. I've try it in 14.0.2 and it hangs forever

I think this could reducing the error rate, and even though there're other reason, we have to read the code to see why it hang. Let me first checkin this


errors runningn AlexaClientSDKConfig.json

IMPORTANT: Before you create an issue, please take a look at our Issue Reporting Guide.
Briefly summarize your issue:
When I run the json file listed in the title, I get a ,load of errors after the last NOTIFICATION INDICATOR STATE:OFF message.
I was able to see an auth code in the text, but got a "failed to create to sample application"
What is the expected behavior?
no errors
What behavior are you observing?
Getting these errors when I run AlexaClientSDKConfig.json :
2018-06-26 02:24:14.971 [  1] I sdkVersion: 1.7.1
configFile Integration/AlexaClientSDKConfig.json
##################################
NOT YET AUTHORIZED
##################################
################################################################################################
To authorize, browse to: 'https://amazon.com/us/code' and enter the code: EZHT3G
################################################################################################
#################################################
Checking for authorization (1)...
#################################################
#################################################
NOTIFICATION INDICATOR STATE: OFF
#################################################
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.rear
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.center_lfe
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.side
ALSA lib pcm_route.c:867:(find_matching_chmap) Found no matching channel map
ALSA lib pulse.c:243:(pulse_connect) PulseAudio: Unable to connect: Access denied
ALSA lib pulse.c:243:(pulse_connect) PulseAudio: Unable to connect: Access denied
ALSA lib pulse.c:243:(pulse_connect) PulseAudio: Unable to connect: Access denied
ALSA lib pulse.c:243:(pulse_connect) PulseAudio: Unable to connect: Access denied
Expression 'paInvalidSampleRate' failed in 'src/hostapi/alsa/pa_linux_alsa.c', line: 2043
Expression 'PaAlsaStreamComponent_InitialConfigure( &self->capture, inParams, self->primeBuffers, hwParamsCapture, &realSr )' failed in 'src/hostapi/alsa/pa_linux_alsa.c', line: 2714
Expression 'PaAlsaStream_Configure( stream, inputParameters, outputParameters, sampleRate, framesPerBuffer, &inputLatency, &outputLatency, &hostBufferSizeMode )' failed in 'src/hostapi/alsa/pa_linux_alsa.c', line: 2838
2018-06-26 02:24:16.824 [  1] C PortAudioMicrophoneWrapper:Failed to open PortAudio default stream
2018-06-26 02:24:16.824 [  1] C PortAudioMicrophoneWrapper:Failed to initialize PortAudioMicrophoneWrapper
2018-06-26 02:24:16.824 [  1] C SampleApplication:Failed to create PortAudioMicrophoneWrapper!
2018-06-26 02:24:16.828 [  1] C SampleApplication:Failed to initialize SampleApplication
Failed to create to SampleApplication!
Provide the steps to reproduce the issue, if applicable:

Tell us about your environment:

What version of the AVS Device SDK are you using?
1.7.1
Tell us what hardware you're using:
AMD Athlon(tm) 64 X2 Dual Core Processor 4400+ × 2
Ubuntu 17.10

Hi bmcintosh123 ,
This looks like a microphone issue on your platform.
Follow the instructions https://github.com/alexa/avs-device-sdk/wiki/Raspberry-Pi-Quick-Start-Guide to check your portaudio installation and test you microphone and make sure it works before you launcg the SampleApp.
Thanks
Priya

I am closing this issue due to inactivity. Please feel free to re-open it if it has been closed in error.