pip install script doesn't work on windows

Here's the error message:
(fpipy) C:\MyTemp\fpipy>pip install .
Processing c:\mytemp\fpipy
Collecting numpy (from fpipy==0.1.0)
  Downloading https://files.pythonhosted.org/packages/2e/11/f006363050b24fb19a235e5efd219e7ac549398d531110d80b8f2ba3a909/numpy-1.16.3-cp36-cp36m-win_amd64.whl (11.9MB)
    100% |████████████████████████████████| 11.9MB 3.1MB/s
Requirement already satisfied: xarray in c:\mytemp\miniconda3\lib\site-packages (from fpipy==0.1.0) (0.10.6)
Collecting rasterio (from fpipy==0.1.0)
  Downloading https://files.pythonhosted.org/packages/ce/90/8af39cdfed9a40bd59801f101f74eab5f1562d3557806ddfa2ed86d4be0b/rasterio-1.0.22.tar.gz (1.9MB)
    100% |████████████████████████████████| 1.9MB 6.0MB/s
    Complete output from command python setup.py egg_info:
    ERROR: Numpy and its headers are required to run setup().

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in C:\MyTemp\Temp\pip-install-eyubteat\rasterio\

My guess is that it has something to do with rasterio. It has been a trouble for me earlier while installing other stuff. I usually install it separately.

Rasterio has been made an optional dependency if you need it for ENVI, so now installing through pip should work fine for most cases. The optional dependency will probably still fail, but that is down to rasterio being a nasty install without using conda.


In the select menu it shows only the first 10 items

Hi, is this a bug ?
I need more then 10 elements ... so in the code I listed 17 elements ...
but without the maxrows parameter only 10 elements are displayed ...
menu.select('s', ['1', '2', '3', '4', '5', '6', '7', '8', '9', '10', '11', '12', '13', '14', '15', '16', '17'], {
  setFunc: async (ctx, key) => {
    selectedKey = key
    await ctx.answerCbQuery(`you selected ${key}`)
  },
  isSetFunc: (_ctx, key) => key === selectedKey
  , columns: 1

})

But when running ...

Massimo

The README ... stated this ... so I close the issue ...

maxRows (Integer, optional) can be provided to limit the maximal rows of buttons. (default: 10)

There is columns, maxRows and getCurrentPage/setPage in order to manage bigger numbers of buttons.
I think 6 columns / 10 rows are already more than a normal user should see per page of buttons 🤔

You are absolutely rightght, but the text I need to show will be truncated if I use 2 columns ... and page usage is not clear for me ... have you a bit of code yet ?
Grazie
Massimo

With the pages you are in charge to store them wherever you want to. In this example I simply store it in the session. Doing so the menu will get the current page from the session and when the user hits one of the pagination buttons to change the current page the setPage function is called to update the value.
menu.select('foo', […], {
  columns: 1,
  maxRows: 5,
  getCurrentPage: ctx => ctx.session.page,
  setPage: (ctx, page) => {
    ctx.session.page = page
  },
  setFunc: ctx => ctx.answerCbQuery('TODO')
}

It's Ok
Thank You !


Cannot load providers on Platforms main page

The message says: "Fetching items from server..." on https://metriq.info/Platforms under the Providers section

There was a problem with API route resolution order; it's been fixed.


[Bug]: Checkbox inside a DataTable component moves once checked

Describe the bug
The Checkbox inside a DataTable component moves once checked and unchecked.
Unchecked

Checked

Affected component/components
DataTable, Checkbodx
How to reproduce

Go to https://ui.shadcn.com/docs/components/data-table
Check one of the checkboxes of the columns in the data table example

You will see that the checkbox floats up once its checked.
Link to reproduce
https://ui.shadcn.com/docs/components/data-table
Logs
No response
System Info
https://ui.shadcn.com/docs/components/data-table

Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

Hello @lohrm-stabl I tried recreating this issue,
Unchecked:

Checked:

Could it be a browser/OS specific issue as I am currently using brave and ubuntu to run this locally?

Hi @anantraghuvanshi,
I encountered this locally, but then verified that it's not an issue with my implementation by visiting https://ui.shadcn.com/docs/components/data-table.
It was verified on latest Firefox on Windows 11 and latest Firefox on Arch, With Edge Browser on Windows 11 the effect is really subtle, but still there.

That could be the case, @lohrm-stabl. I tested it on chrome in Windows 11 no issues over there as well. So help from my side in replicating this issue, sadly!

This seems to only appear for me on firefox but not Edge. Seems to be a browser issue?

Same bug, Firefox 124.0.2 (64-bit), Sonoma 14.4.1 (23E224)

I think maybe it's a combination of the TableCell as well as the Checkbox component:
const TableCell = React.forwardRef(({ className, ...props }, ref) => (
  <td
    ref={ref}
    className={cn("p-2 align-middle [&:has([role=checkbox])]:pr-0 [&>[role=checkbox]]:translate-y-[2px]", className)}
    {...props}
  />
));

specifically this bit: [&>[role=checkbox]]:translate-y-[2px]
I also had to add flex to CheckboxPrimitive.Root and move the h-4 w-4 (or size-4) from the CheckIcon to the Indicator
<CheckboxPrimitive.Indicator className={cn("size-4  items-center justify-center text-current")}>
Haven't looked at any knock on effect this might have as I've only used it in a Table

Removing the tailwind classes of "w-4 h-4" from the CheckIcon fixed it for me. The button that renders from CheckboxPrimitive.Root doesn't change in dimensions, but the containing table cell increases by 0.8px in computer height when the CheckIcon gets rendered. Removing the w/h classes and letting the SVG figure out its height by itself fixes that.
To not cause issues in other places I decided to change the CheckIcon w/h to max-w/max-h instead of entirely removing them. That also fixed the table issue for me and should prevent any major breakages in other locations, where there might not be any implicit constraints on the CheckIcon size. Although the CheckboxPrimitive.Root also has explicit w/h set, so not sure if this concern is founded. In either case, this change seems safe while fixing the table behavior.

This for checkbox.tsx seems to do the trick, easy to copy for those arriving to this thread ;)
"use client"

import * as React from "react"
import * as CheckboxPrimitive from "@radix-ui/react-checkbox"
import { Check } from "lucide-react"

import { cn } from "@/lib/utils"

const Checkbox = React.forwardRef<
  React.ElementRef<typeof CheckboxPrimitive.Root>,
  React.ComponentPropsWithoutRef<typeof CheckboxPrimitive.Root>
>(({ className, ...props }, ref) => (
  <CheckboxPrimitive.Root
    ref={ref}
    className={cn(
      "flex peer size-4 justify-center items-center shrink-0 rounded-sm border border-primary ring-offset-background focus-visible:outline-none focus-visible:ring-2   focus-visible:ring-ring focus-visible:ring-offset-2 disabled:cursor-not-allowed disabled:opacity-50 data-[state=checked]:bg-primary data-[state=checked]:text-primary-foreground ",
      className
    )}
    {...props}
  >
    <CheckboxPrimitive.Indicator
      className={cn("flex items-center justify-center text-current")}
    >
      <Check className="size-3" />
    </CheckboxPrimitive.Indicator>
  </CheckboxPrimitive.Root>
))
Checkbox.displayName = CheckboxPrimitive.Root.displayName

export { Checkbox }

For those who use the non-variables strategy. @ZakEDS's fix basically just adds the following classes to the checkbox flex  items-center justify-center and size-3 for the icon. I recommend adding inline-flex instead of flex for the root instead and size-full for the icon.


B-18109

Integration branch PR

Warnings

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

  Generated by :no_entry_sign: dangerJS against 8691f0d620b95262552539f5c92e021ca029e193


Fabric Bot Configuration

The focus of these changes is to add the Fabric Bot rules into the repository.

@lmazuel -- Can you help us merge this? It's stuck in validation but there don't appear to be any checks running.


Action: adjust disksleep setting

I know this was kind of mentioned in #50 (energy settings in general), and it was suggested that disk sleep was not very interesting or useful, but I have a use case for it, and think it's worth at least a re-consideration.
At work I plug into a RAID array, multiple mechanical disks which are quite slow to spin up. I use them intermittently through the day, but that's not so much a problem just when I want to use them directly, but at least once an hour, when I go to an open/save dialog or sometimes even just while browsing, various apps will completely lock up (pending disk access) while the RAID array spins up. I think it's a problem with OS X, there's no real reason why Chrome should need to wait for the disks to spin up in order to let me keep browsing, or even why a file Open/Save dialog should hang if one drive is not yet available... but that's the way it seems to be.
I'd like to just run sudo pmset -c disksleep 60 whenever that array is plugged in to avoid this, but it needs root privileges. Even leaving that setting always enabled (which is a bit of a waste of battery and maybe not great for my other disks elsewhere) would probably be OK, but the system seems to occasionally reset this parameter back to the default 10, so a regular forced setting would be good.

I recommend writing a script to handle this for you. First you'll need to setup sudo to allow your user to run pmset without a password with sudo visudo to edit the sudo file. Add a line like this, use your username
username  ALL=(ALL) NOPASSWD:/usr/bin/pmset
Now you can create a pair of wrapper scripts you can call with the shell script action.


Bug: Dark mode - ETH Token Color is wrong

Use this dark mode purple ETH icon for dark mode.

@lingzizheng I need both icons to have the same dimensions. The dark mode eth icon is smaller after setting its width.

Done!


chore(cloudflare): update name servers

Follow-up of #138
Ref: https://github.com/jenkins-infra/helpdesk/issues/2649

Terraform will perform the following actions:

  # azurerm_dns_ns_record.cloudflare_jenkins_io will be updated in-place
  ~ resource "azurerm_dns_ns_record" "cloudflare_jenkins_io" {
        id                  = "/subscriptions/dff2ec18-6a8e-405c-8e45-b7df7465acf0/resourceGroups/proddns_jenkinsio/providers/Microsoft.Network/dnsZones/jenkins.io/NS/cloudflare"
        name                = "cloudflare"
      ~ records             = [
          - "cody.ns.cloudflare.com",
          - "kallie.ns.cloudflare.com",
          + "anton.ns.cloudflare.com",
          + "bailey.ns.cloudflare.com",
        ]
        tags                = {
            "repository" = "jenkins-infra/azure-net"
            "scope"      = "terraform-managed"
        }
        # (4 unchanged attributes hidden)
    }

Plan: 0 to add, 1 to change, 0 to destroy.

LGTM, self-merging.


Serialize all Docker Compose operations

... to avoid concurrency issues when running tests in parallel (Docker Compose exhibits race conditions under concurrent usage - HT to @pcornish)

@pcornish would you mind trying this out some time and let me know if it addresses the concurrency issues you were seeing?

Nice, thanks @rnorth :-)


Requirement stockstats >=0.4.0

Hey,
please update the requirements of stockstats to
stockstats>=0.4.0
to avoid issues from using inconsistencies with the int_date package (in version 0.2 it misses the function get_date_from_diff --> became from_diff) causing breaks of the colab notebooks (see #409).
Kind regards and thanks!

Thanks for pointing it out. We have updated it.


what the meaning of receiver and code of an apply function

is the receiver the message receiver? is the code the message sender? If so, I am confused that the implement of EOSIO_ABI requires receiver == code to process the message. then only the contract account itself can send message to itself.

when an action is sent to the network, it specifically addresses a single contract account (the "code" account) and an action on that account.  However, the contract can use require_recipient to notify another account of the action so that a contract on that account may respond.  When it does this, it does not change the "code" account.  This resolves any ambiguity between receiving the action "myaction" as a relay from another contract "notme" vs directly to your account "me".
In these cases
initial action is delivered to "notme" with code and receiver == "notme" then redelivered to "me" with code == "notme" and receiver == "me".
receiver should always be the name of the account the contract is running on however, as it is possible to upload the same contract to multiple accounts it is convenient to have this passed in at runtime.


Handle interruptions on various heart beats

Fixes https://github.com/Alluxio/alluxio/issues/9658
pr-link: Alluxio/alluxio#9695
change-id: cid-93eb0adc32a9a6b0d8052f260c5c613f3e3700c0

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/5075/
Test PASSed.

alluxio-bot, merge this please


[FR] Make Unbound Default to Recursive Queries First and Forward to Public Resolver on Failure

Current behavior
Is there a current behavior that the feature relates to?
The forward-first option is the opposite of what I'm looking for, as it forwards queries before attempting recursion.
Describe the desired feature
I'm using Unbound to set up a recursive resolver, which works well most of the time. However, due to network issues, I sometimes have trouble connecting to some NS servers, resulting in DNS query failures. I would like to configure Unbound to default to using recursive queries for my DNS requests. When a recursive query fails, I want it to fall back and forward the query to a specified public DNS resolver for failover, while also caching the result.
I've reviewed the Unbound documentation, and it seems there isn't a specific option to achieve this functionality.
If there's a way to configure Unbound to meet this requirement, please let me know. I would greatly appreciate any guidance or assistance. Thank you!
Potential use-case
Using Unbound as a recursive server can effectively safeguard privacy and ensure accurate DNS resolution results. However, due to network issues, DNS queries may occasionally fail. Adding this feature can strike a good balance between privacy protection, caching, and failover.

Should this just be closed as a duplicate of #51?


update controller.md

Codecov Report

:exclamation: No coverage uploaded for pull request base (pipeline@0d7a961). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head 95e5366 differs from pull request most recent head 7921431. Consider uploading reports for the commit 7921431 to get more accurate results

@@             Coverage Diff             @@
##             pipeline     #665   +/-   ##
===========================================
  Coverage            ?   78.67%
===========================================
  Files               ?       93
  Lines               ?    10098
  Branches            ?        0
===========================================
  Hits                ?     7945
  Misses              ?     1683
  Partials            ?      470

Continue to review full report at Codecov.

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


feat: add meta-event service

Signed-off-by: Ruihang<EMAIL_ADDRESS>Content

Add a new service MetaEventService and related messages

Many of the new messages are current placeholders. The prototype is https://github.com/CeresDB/CeresDB/blob/f5fe58254c3fb7048a9fea60b26c8d344607cc83/meta_client/src/types.rs#L146-L155
pub enum ActionCmd {
    MetaNoneCmd(NoneCmd),
    MetaOpenCmd(OpenCmd),
    MetaSplitCmd(SplitCmd),
    MetaCloseCmd(CloseCmd),
    MetaChangeRoleCmd(ChangeRoleCmd),

    CreateTableCmd(CreateTableCmd),
    DropTableCmd(DropTableCmd),
}

 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.

Shall we remove some duplicate events defined in meta_sevice.proto?

CeresmetaRpcService is served by CeresMeta and MetaEventService is served by CeresDB. Maybe the name MetaEventService is ambiguous but they are different services in different instances.
As for duplicated messages, all those *Request in meta_service.proto has a RequestHeader which is unnecessary to MetaEventService. And another duplication is DropTableResponse -- still an empty placeholder on both sides, I think it is not appropriate to reuse at this time.


cluster-operator e2e: Install python-devel tools

This patches up the e2e job for cluster-operator by installing the python-devel package. The real long-term fix is to use the ansible image to run any playbooks and not rely on software installed on the ami. That will be coming in a follow-up fix.

@dobbymoodge @stevekuznetsov ptal

@csrwng you push this out

done


token parameter not in video issue

Hi,
No matter which youtube video I try (including the example one) I always get the following response from https://youtube-dl.appspot.com/api/info:
{
  "error": "ERROR: xxx: \"token\" parameter not in video info for unknown reason; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output."
}

Could it be that the version of youtube-dl in that server needs updating?

Duplicate of https://github.com/jaimeMF/youtube-dl-api-server/issues/55


Add reduce algorithm listing

This adds a listing that demonstrates std::reduce, hpx::reduce(seq) and hpx::reduce(par).
@diehlpk  please update me on the naming convention you want to go with here and how do you want for the files to be organized.

Can you please add a folder with the example, so we have all examples in a separate folder?

will do


Removed hard coded value for spinner size. Adjusted large spinner label…

… left position.

Nice, looks like this will fix #328.

Looks good to me.

Looks great!


Show multiple months at once

Something like this (taken from kayak.com):

At a minimum I'd want the option to show 2 months at a time instead of 1. Although I'd imagine the implementation wouldn't be much different to just allow that number to be configurable (e.g. showing 5 at a time).

Thanks for your suggestion, I've added a multiMonth setting which allows multiple months to be displayed at the same time.

Hi, how can i use this multiMonth attribut?
Here's my code but it only shows 1 month
$('#example2').calendar({
	ampm: false,
	type: "date",
	multiMonth: 2,
})

same question, could not get this to work ?


Save Trading Capital into Redis (db=0)

Redis db=0

Save initial capital and current cash into Redis as string separately.


Should be able to playback a recorded script successfully

Expected behavior
Should be able to playback a recorded script successfully
Actual behavior
When I playback the script I get the exception
 STATUS: 403, STATUS_TEXT:
    at Object.handleNavigation (.../taiko/lib/pageHandler.js:104:15)

Steps to reproduce

open the REPL
Record the flow

> openBrowser()
 ✔ Browser opened
> goto("https://www.zomato.com/bangalore/taco-bell-koramangala-6th-block/order")
 ✔ Navigated to URL https://www.zomato.com/bangalore/taco-bell-koramangala-6th-block/order
> .code
const { openBrowser, goto, closeBrowser } = require('taiko');
(async () => {
    try {
        await openBrowser();
        await goto("https://www.zomato.com/bangalore/taco-bell-koramangala-6th-block/order");
    } catch (error) {
        console.error(error);
    } finally {
        await closeBrowser();
    }
})();

Now, execute the recorded flow

Get the exception

Error: Navigation to url https://www.zomato.com/bangalore/taco-bell-koramangala-6th-block/order failed.
 STATUS: 403, STATUS_TEXT:
    at Object.handleNavigation (.../taiko/lib/pageHandler.js:104:15)

Versions
commit - 7a49c17886d51b619dcfa53717fd88518d6499dc

This is happening because zomato blocks accessing the site when launched in headless mode. Hence the 403 status code which denotes Forbidden.

Observations
The script is recorded for the headful mode in the REPL. But when it is played back it is in the headless mode. There can be a difference in the behaviour of headful and headless modes. This is not evident while replaying the recorded script. Maybe it is worth considering that the REPL mode and generated code both are consistent in the modes they run? Currently headful is the only state that the REPL supports.

This we should update in our documentation.


Upgrade ORC to 1.6.9

Description
This PR aims to upgrade Apache ORC from 1.5.10 to 1.6.9 and licenses.yaml correspondingly to bring the latest bug fixes and updates.

https://orc.apache.org/docs/releases.html#current-release---169

Apache ORC 1.5.10 is too old and the following versions are available. Apache ORC community highly recommends to use 1.6.9.

1.6.9: https://orc.apache.org/news/2021/07/02/ORC-1.6.9/
1.5.12: https://orc.apache.org/news/2020/09/30/ORC-1.5.12/

This PR has:

[v] been self-reviewed.
[v] added or updated version, license, or notice information in licenses.yaml

@dongjoon-hyun - Thanks for the change - I wanted to ask that in the orc-extensions pom, do we have to include orc-core are a dependency separately (it is so in the current code)? I saw that orc-mapreduce has marked orc-core as its dependency so maybe we could just use that. I also checked and orc-core is only used in orc extension. wdyt?

Hi, @rohangarg . Thank you for review.
Yes. AFAIK, the orc-core dependency is there because we want to cut the transitive dependencies of it. If we use only orc-mapreduce dependency and rely on the Maven's transitive dependency resolutions, the global dependency tree might be affected.

do we have to include orc-core are a dependency separately (it is so in the current code)?

        <dependency>
            <groupId>org.apache.orc</groupId>
            <artifactId>orc-core</artifactId>
            <version>${orc.version}</version>
            <exclusions>
                <exclusion>
                    <groupId>com.google.protobuf</groupId>
                    <artifactId>protobuf-java</artifactId>
                </exclusion>
                <exclusion>
                    <groupId>commons-lang</groupId>
                    <artifactId>commons-lang</artifactId>
                </exclusion>
                <exclusion>
                    <groupId>javax.xml.bind</groupId>
                    <artifactId>jaxb-api</artifactId>
                </exclusion>
                <exclusion>
                    <groupId>org.apache.hadoop</groupId>
                    <artifactId>hadoop-hdfs</artifactId>
                </exclusion>
                <exclusion>
                    <groupId>org.slf4j</groupId>
                    <artifactId>slf4j-api</artifactId>
                </exclusion>
            </exclusions>
        </dependency>

Oh, I double-checked it. It seems that the exclusion rules of orc-mapreduce includes all exclusion rules of orc-core. I'll revise this PR according to your advice, @rohangarg . Let's see what CIs say.

@rohangarg . Are you sure about your comment? Druid analyze check seems to enforce the declaration.

I saw that orc-mapreduce has marked orc-core as its dependency so maybe we could just use that.

https://app.travis-ci.com/github/apache/druid/jobs/528073697

[WARNING] Used undeclared dependencies found:
[WARNING]    org.apache.orc:orc-core:jar:1.6.9:compile
[INFO] Add the following to your pom to correct the missing dependencies:
[INFO]
<dependency>
  <groupId>org.apache.orc</groupId>
  <artifactId>orc-core</artifactId>
  <version>1.6.9</version>
</dependency>

I'll revert the last commit because this PR already passed before the last commit.

@dongjoon-hyun : Yes, you are right. Apologies, I missed that dependency analysis in druid fails compilation if we're not declaring any directly used dependency (the default maven behavior gives a warning). Maybe that was the original reason to keep it explicit :) I had missed the -DfailOnWarning=true flag during local check so the compile had passed for me.
Also, I did a quick lookup and the observation you had wrt to the transitive dependencies is correct - the maven page states that Exclusions work on the entire dependency graph below the point where they are declared. (https://maven.apache.org/guides/introduction/introduction-to-optional-and-excludes-dependencies.html)
You updated change LGTM - thanks for the tryout and changes!

Could you review this PR when you have a time, @jihoonson ?
This PR already passed at 9698b7d19227d2fe6388d9749a9a576cdeaa5b91 and now I reverted this to that commit.
Among, the latest two failures,

The dependency change is reverted
The IOException issue will be fixed https://github.com/apache/druid/pull/11522 .

:) I also learned it at this time. Thank you for you reviews and helps, @rohangarg .

@dongjoon-hyun sure, I will have a look soon.

The one failure looks irrelevant to this one because the last commit (https://github.com/apache/druid/pull/11518/commits/2feb9a3c7246bffbc4c9e47b9f1ee926f7d3805d) is identical to the this commit (https://github.com/apache/druid/pull/11518/commits/9698b7d19227d2fe6388d9749a9a576cdeaa5b91) which passed all tested.
[ERROR] Failures:
[ERROR]   ITKafkaIndexingServiceNonTransactionalParallelizedTest.testKafkaIndexDataWithWithAutoscaler:62->AbstractStreamIndexingTest.doTestIndexDataWithAutoscaler:342 » ISE
[ERROR] Tests run: 4, Failures: 1, Errors: 0, Skipped: 0

Thank you so much, @jihoonson !


scaff_bwa segmentation fault

Hi there,
I am encountering a segmentation fault when running Scaff10X.
[main] Version: 0.7.17-r1188
[main] CMD: /home/sejoslin/miniconda3/envs/scaffold_10x/bin/Scaff10X/scaff-bin/bwa mem -t 16 tarseq.fastq /group/millermrgrp2/shannon/projects/assembly_genome_Hypomesus-transpacificus/00-raw_data/data-10X_M/Male2_S63_L004_R1_001.fastq.gz /group/millermrgrp2/shannon/projects/assembly_genome_Hypomesus-transpacificus/00-raw_data/data-10X_M/Male2_S63_L004_R2_001.fastq.gz
[main] Real time: 140745.891 sec; CPU: 2245179.580 sec
Segmentation fault
Error running command: /home/sejoslin/miniconda3/envs/scaffold_10x/bin/Scaff10X/scaff-bin/scaff_bwa -edge 50000 tarseq.tag align.dat align2.dat > try.out

Here is the stderr file associated with the error:
test_assembly.j23957420.run_scaff10x.hi.err.txt
Here is the stdout file associated with the job:
test_assembly.j23957420.run_scaff10x.hi.out.txt
I have the following lines in the tmp_rununik_10443
-rw-rw-r-- 1 sejoslin millermrgrp    0 Jul  3 06:46 align2.dat
-rw-rw-r-- 1 sejoslin millermrgrp  23G Jul  3 06:46 align.dat
-rw-rw-r-- 1 sejoslin millermrgrp 930M Jul  1 15:31 tarseq.fastq
-rw-rw-r-- 1 sejoslin millermrgrp  257 Jul  1 15:38 tarseq.fastq.amb
-rw-rw-r-- 1 sejoslin millermrgrp 201K Jul  1 15:38 tarseq.fastq.ann
-rw-rw-r-- 1 sejoslin millermrgrp 465M Jul  1 15:38 tarseq.fastq.bwt
-rw-rw-r-- 1 sejoslin millermrgrp 117M Jul  1 15:38 tarseq.fastq.pac
-rw-rw-r-- 1 sejoslin millermrgrp 233M Jul  1 15:40 tarseq.fastq.sa
-rw-rw-r-- 1 sejoslin millermrgrp 187K Jul  1 15:31 tarseq.tag
-rw-rw-r-- 1 sejoslin millermrgrp    0 Jul  1 15:31 try.out

and my align.dat file looks like this:
(base) sejoslin@farm:tmp_rununik_10443$ head align.dat
A00351:291:HVMC5DSXX:4:1101:1307:1000 69 tarseq_648 132029 0
A00351:291:HVMC5DSXX:4:1101:1398:1000 99 tarseq_462 123666 60
A00351:291:HVMC5DSXX:4:1101:1524:1000 81 tarseq_236 424566 0
A00351:291:HVMC5DSXX:4:1101:1597:1000 97 tarseq_874 104495 0
A00351:291:HVMC5DSXX:4:1101:1687:1000 99 tarseq_16 418743 60
A00351:291:HVMC5DSXX:4:1101:1940:1000 97 tarseq_2149 50794 0
A00351:291:HVMC5DSXX:4:1101:2284:1000 99 tarseq_338 173397 60
A00351:291:HVMC5DSXX:4:1101:2302:1000 83 tarseq_302 99246 60
A00351:291:HVMC5DSXX:4:1101:2483:1000 83 tarseq_5085 17812 60
A00351:291:HVMC5DSXX:4:1101:2591:1000 99 tarseq_102 116777 41
(base) sejoslin@farm:tmp_rununik_10443$ tail align.dat
A00351:291:HVMC5DSXX:4:2678:13132:36949 99 tarseq_1383 26746 60
A00351:291:HVMC5DSXX:4:2678:13277:36949 65 tarseq_1446 66935 60
A00351:291:HVMC5DSXX:4:2678:13313:36949 99 tarseq_569 12813 60
A00351:291:HVMC5DSXX:4:2678:13639:36949 83 tarseq_1401 40168 27
A00351:291:HVMC5DSXX:4:2678:13747:36949 99 tarseq_84 92647 60
A00351:291:HVMC5DSXX:4:2678:14091:36949 99 tarseq_445 203003 60
A00351:291:HVMC5DSXX:4:2678:14561:36949 99 tarseq_512 71920 11
A00351:291:HVMC5DSXX:4:2678:14597:36949 83 tarseq_411 18587 60
A00351:291:HVMC5DSXX:4:2678:14868:36949 97 tarseq_2112 1850 10
A00351:291:HVMC5DSXX:4:2678:15157:36949 83 tarseq_613 158418 60

I ran Scaff10X with the following parameters:
#SBATCH -J hi_scf10
#SBATCH -e slurm/test_assembly.j%j.run_scaff10x.hi.err
#SBATCH -o slurm/test_assembly.j%j.run_scaff10x.hi.out
#SBATCH --nodes=1
#SBATCH --ntasks=16
#SBATCH --mem=480G
#SBATCH --time=06-10:08:07
#SBATCH -p bigmemh

and used the following command to run scaff10x:
${scaf_bin}/scaff10x \
    -nodes ${threads} \
    -align bwa \
    -matrix 2000 \
    -reads 12 \
    -link 10 \
    -plot barcode_length.png \
    ${asm} ${R1} ${R2} ${output}"

I didn't think this step much memory and indeed it fails at the same place if I use a partition with less memory (62000M/node).
Please advise! Thank you for your time :)

Hi, I get exactly the same error with exactly the same files in the tmp (with file size 0 for both try.out and align2.dat).
Was there a solution?
Best and thanks

Hi, I get exactly the same error with exactly the same files in the tmp (with file size 0 for both try.out and align2.dat).
Was there a solution?
Best and thanks

Many thanks for all your bug reporting efforts. I spent some time on this and sorry to say I haven't found the bug or the problem. VGP used a lot on scaff10x and it seems that they didn't have this issue and also I never had this problem. If any of the users had this problem and also put the data (reads and assembly) in a location for download, I would be really grateful! My problem is that I need to repeat this issue and then fix the bug. For the time being, please use scaffolding_reads to decode the barcodes and then use the paired reads for scaffolding.
Best regards,
Zemin

Hi all,
Apologies for not posting when I got things up and running. I believe I "solved" this (or at least I no longer run into the) problem by supplying the input 10X fastq's through -data input.dat (see format below) and sym-linking the reference genome to be in the working directory .
input.dat file format:
q1=/group/millermrgrp2/shannon/projects/assembly_genome_Hypomesus-transpacificus/00-raw_data/data-10X_M/Male2_S63_L004_R1_001.fastq.gz
q2=/group/millermrgrp2/shannon/projects/assembly_genome_Hypomesus-transpacificus/00-raw_data/data-10X_M/Male2_S63_L004_R2_001.fastq.gz

Run command:
scaff10x -nodes $((SLURM_CPUS_PER_TASK-2)) -longread 1 -gap 100 -matrix 2000 -reads 10 -link 8 -score 20 -edge 50000 -block 50000 -data input.dat sym-linked.reference.fasta output.scaff10x.fasta

which was run in the following working directory : /group/millermrgrp2/shannon/projects/assembly_genome_Hypomesus-transpacificus/03-assemblies/sandbox_hicanu/
As a note I installed scaff10x with conda.
@zning-sanger I'd be happy to upload the original files I used to a server if you'd like/it would be useful. I haven't tried to recreate the error since it was happening but maybe knowing how I fixed it helps??


Fix stack overflow in circular assignment declaration

The declaration also needs to have multiple assignments so that it has a ValueModule flag.
The fix is to remove the possibly-circular call to getTypeofFuncClassEnumModule in the circularity-detection code path.
Fixes #33006

This was introduced in #29335, but that test still passes with the fallback calls to getTypeOfFuncClassEnumModule removed.

Oops, forgot that some tests were skipped. The tests do still fail.

After some discussion, I don't think that #29335 is the correct fix; umdNamespaceMergedWithGlobalAugmentationIsNotCircular should still have an error, but a duplicate declaration error instead of a circular reference error.

Since 3.7's beta ends today, I decided on a compromise fix, which is to avoid the fallback for symbols with flag Assignment. Those symbols will have already been checked using the JS code path, so aren't candidates for the special kind of merges handled in #29335.
Later I'll see whether I can catch the merge from #29335 in mergeSymbol during checker startup and issue an error there.

@weswigham the React merge works because we allow aliases to merge with anything. If I resolveSymbol first and forbid merges between the source and the resolved alias, it breaks four tests and the breaks all look right to me. Opinions?


[Unity][Dlight] Handle Epilogue Broadcasting

This PR improves the Decode-GEMV scheduling by further analyzing its epilogue pattern.
The existing behavior assumes that the outcome of cross-thread reduction stays in register files local to each thread, which is further used to calculate the epilogue in the same thread.
This strategy means the cross-thread reduction outcome is stored only on thread 0, while the other threads cannot participate in subsequent computation (i.e. epilogue). Related: https://github.com/apache/tvm/pull/15192.
When the epilogue is relatively lightweight, i.e. elementwise add, casting on scalars, this strategy is optimal. However, once the outcome needs to be broadcasted to compute over a non-trivial region, for example, act as a normalizer of np.mean, it would become much slower because only one thread in a thread block is effectively used.
In this case, we will need to broadcast the cross-thread reduction outcome in shared memory, making it visible to other threads, and then bind the compute region to all threads in the threadblock.

CC @MasterJH5574 on cross-thread reduction, @Hzfengsy and @spectrometerHBH for e2e performance


Single step invite and approve partner banks

Resolves #3432
Description

Organizations may opt-in an "invite and approve" feature for their partner banks to skip the approval process. A considered alternative solution was to create a new service, rather than using two services in one function in the partner controller. But that is probably duplicate and unnecessary code. I tried squashing the commits, but I couldn't figure out how to. I ended up making a lot of unnecessary extra commits - sorry about that!
The unit test uses primary_user.delete, which is what the test allows a user to invite a partner uses. If it is not included, then the PartnerInviteService will automatically return. I'm unsure why that is, so I just used this piece of existing test code.
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Organization admin users can see and edit the single-step-invite-and-approve feature in the organization pages. For an organization that opted in the single-step-invite-and-approve feature, users can 'Invite and Approve' partner agencies.
spec/models/organization_spec.rb: added a test that checks updating the new organization model.
spec/system/organization_system_spec.rb: added a test that checks updating the new organization feature via the UI, as well as a test that checks that the feature is displayed on the show page.
spec/system/partner_system_spec.rb: added a test that checks if the single step invite and approve is true, then a new "Invite and Approve" button will lead to an approved partner.
Screenshots

As user:

As organization admin (can edit the field):

The original issue has been solved by another PR.  Closing this one. Thanks for the try!


Fixed vulnerabilities and moved to spring doc

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

Superseded. Closing.


Build sd2emmc installer image

Hi, this is the script to build the sd2emmc image. I add pine64_sd2emmc.sh to install pv application, autologin, install pinebook.img from sd card to emmc and it will execute pv and dd as sudo without password. So is it the correct way?

I wonder if we would not be able to do that in very simple initrd.img. Maybe that would be simpler :)

Do you mean the installing script should be written in initrd.img?

Closed due to this feature is no longer needed.


[FEATURE] Complete User account functionality

Summary
NA
Description
Create complete User account functionality end to end including normal login/signup (no need of google oauth n all for now). - all the events a user attends should be visible in their profile(All those who attended the first django india meetup, create their accounts automatically and attached that meetup to their account)

If someone is signing up for the first time(from the existing user), ask them to create a new password.
functionality to reset password
functionality to forgot password

Use Case
No response
Alternatives
No response
Affected Area
Backend

@DevilsAutumn, in this issue, will we be using the built-in Django user model, or do we need to create a custom user model for now?

@DevilsAutumn, in this issue, will we be using the built-in Django user model, or do we need to create a custom user model for now?

Would be better to extend the built-in django user .

@PankajVishw50 Thankyou so much for showing interest in this issue. But this issue is on our top priority so I will be picking it up.


Issue/oak 10022

https://issues.apache.org/jira/browse/OAK-10022

@smiroslav I think there's a potential problem here: tar archives are append-only, so if we write the references after each flush, we will produce a lot of waste. That isn't a problem in the Azure implementation where tar archives are actually just files and directories, so we might want to have different TarWriter implementations for local file system and Azure.
But I haven't spent much time on this so perhaps what I'm saying is totally wrong...

@smiroslav I think there's a potential problem here: tar archives are append-only, so if we write the references after each flush, we will produce a lot of waste. That isn't a problem in the Azure implementation where tar archives are actually just files and directories, so we might want to have different TarWriter implementations for local file system and Azure.

@ahanikel If I understand this code well, only the AbstractRemoteSegmentArchiveWriter class calls the lambda, while the SegmentTarWriter is ignoring it (as per the default implementation in SegmentArchiveWriter), preventing the waste you mention for local files.
Although I think this implementation works, it's true that the design is somewhat fragile: someone could unknowingly decide in the future to override flush(Runnable onSuccess) in SegmentTarWriter and call the lambda. One option to prevent this, could be to implement it now explicitly in SegmentTarWriter by ignoring the lambda and adding a comment explaining why. Not pretty, but at least it should avoid future mistakes.
The other option would be, as you suggest, to create 2 TarWriter implementations. However, currently TarWriter is not a public class and seems to be considered an implementation detail. We would need to create and expose a new public interface and change users of the current class to give the ability to accept different implementations of the interface. The design would probably be more robust and explicit, but I don't know how much effort it would need. It would probably make a lot of sense to work on this option if we anticipate other benefits of having specialised implementations of TarWriter for Azure or AWS, for exemple better optimisations.

@jelmini  @ahanikel  how about declaring method in  SegmentArchiveWriter
public boolean isRemoteStoreWriter()

And use it in org.apache.jackrabbit.oak.segment.file.tar.TarWriter#flush
if (archiveWritter.isRemoteStoreWriter) {
    writeBinaryReferences();
    writeGraph();
}
archiveWritter.flush();

@jelmini @ahanikel how about declaring method in SegmentArchiveWriter
public boolean isRemoteStoreWriter(); // or some other better name ;)

Yep, could be a good solution: simple and explicit where it needs to be. The name could just be isRemote().

@smiroslav @jelmini Yes, I think that's fine.

This new SonarCloud check is failing the build because of not enough coverage (<80%) for the modified classes. For example Sonar considers that AbstractRemoteSegmentArchiveWriter has 0% coverage, which is true in the context of the oak-segment-remote module where it's declared. However, tests in oak-segment-azure are exercising a sub-class of AbstractRemoteSegmentArchiveWriter, which indirectly cover it with a 94% line coverage...
Because Sonar only computes coverage at the module level and not at the project level, it seems that we are forced to add tests to AbstractRemoteSegmentArchiveWriter in oak-segment-remote to make it happy... Or, is there a way to bypass this check?

SonarCloud checks have been added recently, I believe, by @nit0906 .
It will gradually improve the codebase, but I do not think every check should be a blocker for a PR merge, especially for classes like AbstractRemoteSegmentArchiveWriter.
It seems that checks rely on the jacoco-maven-plugin plugin. Maybe the class can be excluded from the plugin configuration.

By all means: please update JIRA issues when merging fixes.


Build failed

I don't know how two fix it and i'm a newer for gradle, please help me. thx!
The stackstrace is:


Write Deployment/Ops Guide

It looks like Tatiana/Tramel set a lot of this up and broke the sections out so I filled in the one section that we didn't have a ticket for and updated a bit of the intro as part of this ticket

Added missing section as ticket for next sprint that will depend on what new APIs we add and if we want to include instructions for the previous ones. I recommend ripping them from the previous cohort (at least the screenshots) if we do


installation error.

Hi,
when i am going to install your application it gives error:
file_get_contents(E:\wamp\www\starter-laravel-angular\public/build/rev-manifest.json): failed to open stream: No such file or directory (View: E:\wamp\www\starter-laravel-angular\resources\views\layout.blade.php)
node is installed
npm is installed but gulp is not found.
please guide me to make it.
Thanks

If Gulp is not found, then I guess you haven't got it installed (globally). To install Gulp globally execute npm install -g gulp. Then try again.
file_get_contents(E:\wamp\www\starter-laravel-angular\public/build/rev-manifest.json): failed to open stream: No such file or directory (View: E:\wamp\www\starter-laravel-angular\resources\views\layout.blade.php) is an error that occurs when composer install --prefer-dist wasn't run.

Hello!
I made a fresh install and got the same error:
file_get_contents(C:\xampp\htdocs\starter-laravel-angular\public/build/rev-manifest.json): failed to open stream: No such file or directory (View: C:\xampp\htdocs\starter-laravel-angular\resources\views\layout.blade.php)
gulp installed and run succesfully
composer install --prefer-dist  run more than once without errors
What else?

I found the solution myself:
In gulpfile.js I added public/ path to files listed in version, resulting in:
...
.version([
'public/css/all.css',
'public/js/all.js'
])
...
running gulp again it generate rev-manifest.json properly.


multiple controllers

when using multiple controller for example 2.
In Steam Big Picture Mode there is always an input what it makes nearly impossible to navigate in there,
also in games I notice this problem that there is unwanted input. I think always games where you can control the main menu with all four controllers not only with the first.
When I make the config file only with one controller everything works as expected.
My config file:
[ds4drv]
daemon=false
hidraw=false
[controller:1]
led=0000ff
emulate-xpad=true
battery-flash=true
trackpad-mouse=true
[controller:2]
led=ff0000
emulate-xpad=true
battery-flash=true
trackpad-mouse=true
[controller:3]
led=00ff00
emulate-xpad=true
battery-flash=true
trackpad-mouse=true
[controller:4]
led=ffff00
emulate-xpad=true
battery-flash=true
trackpad-mouse=true
[bindings]
PS+Triangle = exec-background killall -KILL pcsxr PCSX2 mupen64plus dolphin-emu snes9x-gtk reicast PPSSPPSDL
Edit:
found out that the default axis values for the non used controllers, that were created by the config file are wrong. This explains the strange behavior in steam and some games.

I'm having this issue as well. Axes 0-6 all have a default minimum value of -32767 which is causing spurious input. This will happen with even a single controller if it has not yet been connected. I've tried to modify the code myself but unfortunately python is not a language that I am comfortable in, and all attempts were for naught.
One thing I have noticed, though, is that these values all get set to 0 when a controller disconnects.
Current workaround is to connect the amount of devices that I'm going to use, but that means manually inputting commands rather than have a script running automatically at boot. This tool would be infinitely better if this problem was solved.

Would like to add an update.
Adding the following line to the constructor of UInputDevice in uinput.py fixes the issue:
self.emit_reset()
Of note, it does not change the output of the joystick or jscal tool, but it does get rid of the spurious input that occurs in Steam and other applications.
I suggest anyone that has this problem try this and reinstall ds4drv.

@vesayth
awesome!!!, thank you that is working for me!!!
Great work. :+1:

It doesn't seem like the original developer is active anymore.
There are available lot of useful patches now,  but nobody merging them in.  Maybe it's best if someone can maintain a fork, and accept the patches there.  At least until/if the original developer comes back.

I merged the fix.

fix doesn't work on new version 0.5.1.
Problem is the same like in 0.5.0.
cant find out why, when I apply the patch on 0.5.0 uinput.py its working.

@Mernst1 Could you test if #75 with latest master fixes the issue for you?

@Ape  Yes!!! the new fix makes it running again. Great work! thank you @Ryochan7.


Enable RSpec/StubbedMock

I know this cop can be controversial, and maybe we don't want to always enforce its recommended style.
But in all three cases found in our spec suite, the mock (defined by expect) is entirely unnecessary - using a stub (with allow) is enough. If any of the stubs are changed, other expectations in each example will fail.

Before submitting the PR make sure the following are checked:

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

Some more context: https://github.com/rubocop-hq/rubocop-rspec/pull/226#issuecomment-723494927

I agree they are not necessary.
I'd like to hear an argument for them not being harmful (or even for not being useful)
Here's my argument to their usefullness.
These expect act as sanity check that the implementation being tested does indeed call said method as expected. I see no value in removing these sanity checks.
For example, if the caching mechanism is changed to call something else than external_dependency_checksum and if this PR is merged in, only one of the two modified specs would fail. With the current expect they would both fail.

only one of the two modified specs would fail. With the current expect they would both fail.

Do you really need both of them to fail to see that something's not right?
I understand that if both would fail, it would probably take less time to figure out what is the root cause since the expect would fail.
This is what I meant in:

properly balanced examples

To be fair, there was another valid argument just recently, that unused stubs would hang in specs while not actually called from the code.
I can think of something like this to address all of those concerns at once:
      it 'uses external_dependency_checksum under the hood' do
        cache.save(offenses)
        expect(team).to receive(:external_dependency_checksum)
        cache2 = described_class.new(
          file, team, options, config_store, cache_root
        )
        cache2.valid? # not sure if this line is necessary, and what calls `external_dependency_checksum`, `initialize` or `valid?`
      end

      context 'when team external_dependency_checksum changes' do
        it 'is invalid' do
          cache.save(offenses)
          allow(team).to(
            receive(:external_dependency_checksum).and_return('bar')
          )
          cache2 = described_class.new(
            file, team, options, config_store, cache_root
          )
          expect(cache2.valid?).to eq(false)
        end
      end

      context 'when team external_dependency_checksum is the same' do
        it 'is valid' do
          cache.save(offenses)
#          This in no way makes it clear that "the checksum is the same"
#          allow(team).to(
#            receive(:external_dependency_checksum).and_return('foo')
#          )
          cache2 = described_class.new(
            file, team, options, config_store, cache_root
          )
          expect(cache2.valid?).to eq(true)
        end
      end

But pretty sure someone will be dissatisfied with it anyway, not meaning anyone particular :D
I believe this all boils down to "what is more effective: be able to track down the root cause quickly when the implementation of SUT changes and some spec break, or to be able to understand how SUT works by looking at the spec".
I'm personally more biased towards the latter, but don't insist that it's the only acceptable option.

This in no way makes it clear that "the checksum is the same"

Agreed, would be better with simply expect(team).to receive(:external_dependency_checksum) # sanity check (assuming that works, idk).

But pretty sure someone will be dissatisfied with it anyway

IUC, this is adding a whole test just to isolate the "sanity check" part from the rest, just so it satisfies this "rule". You were right to be pretty sure 😅

I believe this all boils down to...

To me it boils down to an idealized abstract theory that doesn't jive with pragmatic reality.
This is the same as with MultipleMemoizedHelpers and I must admit being still frustrated about that being a default, despite being clearly more problematic and even less fixable.
 I officially have no objection to whichever way to turn this cop back on.

I'm also ambivalent towards the proposed change. Feel free to merge it or not, but let's not keep the PR open for an extended period of time.

How about this compromise – I fix the offenses, but don’t enable the cop?

Or reverse compromise - we disable the cop for this spec?
From that, we'll see if it helps improving or stands in the way of specs yet to be created and adjust accordingly.

I'm also ambivalent

It's not clear who the "also" refers to, but since this comment follows mine, I'll underline that I am not ambivalent at all, only that I'm not going to make any further opposition to whatever changes are deemed positive.

How about this compromise – I fix the offenses, but don’t enable the cop?

I like this compromise and I don't want this PR to stay open forever. Merged. :-)


Refactor scheduler extender-related API fields

For trace ability and historical reasons, it's vague that which scheduler APIs should be used internally, while which ones should be externalized, and which should be supported in a Kubernetes API's fashion, and which to serve as a plain protocol contract for HTTP communications only.
API cleanup is a line item , starting with this PR, we will address most of the API concerns in the scheduler.
Specifically for this PR, we're going to move all API fields related to extender implementation (not including Extender or Policy configuration APIs) to pkg/scheduler/apis/extender/v1

Linting is required. Hence, marking this as Invalid, and closing.


Adding dynamic changes

How can we update the index dynamically when changes are made to the storage account, such as adding documents?

These docs describe how to update the index:
https://github.com/Azure-Samples/azure-search-openai-demo/blob/main/docs/data_ingestion.md
For dynamic updating, I recommend using the Integrated Vectorizer approach, which will soon be enabled in this repo. See https://github.com/Azure-Samples/azure-search-openai-demo/pull/1159
That sets up indexers and skillsets, and indexers can grab new data automatically.


Rename discussion modal translations

@wiwatsrt Thanks for your contribution!
@dcsjapan This was correct, or at least consistent with the keys in the code. I now changed this as you suggested - which also seems to be consistent with other modals. However, is this still correct? Right now, the key rename_discussion is nested under discussion_controls. Is that intended? I'd guess not, since - in theory - the modal is reusable in other contexts as well.

@franzliedke Right, each of those keys now has one more namespace than it should.
Just remove the discussion_controls. from each key and it will match the resources in the YML.

@dcsjapan Note that the nesting in the YML file was not correct, either. I fixed both now.

Thanks, didn't notice that it was out of order.


Add fontColor as a configuration option for node's  fill property

This enables users to set a fontColor under the node configuration option that will set the Node Components  svg elements fill color.

I'll get and update in tonight to fix the tests

Nice one @dmmulroy :ok_hand:, I'll complete the code review asap.

Fixed up the tests, everything should be good to go!

Changes should be in


Some tests with varargs does not seem to work

Hi all,
i've commented several tests in SplitterOptionTests because it does not seem to work.
Did have i've done something wrong ?
see https://github.com/sormuras/command-line-interface/blob/main/test/test/unit/SplitterOptionTests.java#L140

The tests were wrong see 5ca898faea6f8524e469d40a8bd8c16035f7f065


fix logging error

fix error with logging configuration
some formatting changes by formatter-maven-plugin

Addresses part of #132


(MODULES-8553) Further cleanup for package tag issues

Update all this modules tagged packages to use 'puppetlabs-postgresql' instead of just 'postgresql' which is ambiguous and causing dependancy issues.

Thanks for the review Charlie, that's it up to date with those two manifests as well :) Let me know if this is ready to go and I'll get it merged and a release of the back of it.


Determining Input and output channel for "test"

Hi @ZhaoqxCN
Can you explain me how did you calculate the input and output channels in  this file main.cpp  ?
	AXI_VAL status = 0;
	AXI_VAL batch_size = 2;
	AXI_VAL Ker_DIM = 0;
	AXI_VAL In_CH = 1;
	AXI_VAL In_DIM = 28;
	AXI_VAL Out_CH = 10;
	AXI_VAL Out_DIM = 1;

you mentioned batch_size is 2, it represents the two convolution layers?

Hi @ZhaoqxCN
How did you calculate the #pragma HLS loop_tripcount in every layer?


Test

@thisismattmiller This PR will merge several major improvements and at least one breaking change, which is that the /trans and /feedback endpoints will accept a JSON payload. This needs to be reflected by all clients currently using those endpoints.
Let me know if we need some special arrangements to deploy this. I am still working on some details and documentation, so I'm not ready for a review yet.
I'm planning to tag this as 1.0.0 because of the breaking API change.

NB This PR also introduces the splitting of Docker image into a scriptshifter-base and scriptshifter to alleviate the very long build times that installing ML tools requires. the base image installs all the major dependencies, while the ap image only copies the scriptshifter code and runs the entrypoint script.


Kata Containers 1.10.7

actions: Add action to perform checks for pull requests

b59d813 actions: Add action to perform checks for pull requests

/test

/test


Run Cell and Below button does not work with F# interactive

Describe the bug
Create a new file .net interactive notebook called test.ipynb. Try to use the "Run cell and below" or "Run cell and above" buttons. You get errors about ipython. However, if I create a powershell cell, the buttons work.

Please complete the following:
Which version of .NET Interactive are you using? There are a few ways to find this out:

OS

[ X] Windows 10
[ ] macOS
[ ] Linux (Please specify distro)
[ ] iOS
[ ] Android

Browser

[ ] Chrome
[ ] Edge
[ ] Firefox
[ ] Safari

Frontend

[ ] Jupyter Notebook
[ ] Jupyter Lab
[ ] nteract
[ ] Visual Studio Code
[ X] Visual Studio Code Insiders
[ ] Other (please specify)

Screenshots
If applicable, add screenshots to help explain your problem.

cc @brettfo

The error messages in the cells show that the Python kernel is being used to execute the cells and not .NET Interactive.  The Python kernel is overly-aggressive in setting its own kernel (see #1189), but if you select .NET Interactive in the bottom right then you'll get the proper kernel execution.

Hi @brettfo , which "bottom right" for selecting the kernel? In my screenshot I show that had the f# interactive kernel selected for the cell.
I'm trying to reproduce it for you, but .NET interactive updated to a new version and now I cannot get any of the buttons in test.ipynb execute any code. However, if I save the same file as a test.dib file and open the .dib file it works. I tried changing folders, uninstalling jupyter and .net interactive extensions and reinstalling, etc.

I should also clarify that when I had this issue Saturday, the big arrow on the left of the cell to run that specific cell with the F# kernel worked fine. The precise issue was the "Run cell above" and "Run cell below" buttons tried to use the python kernel.

@nhirschey I suspect your VS Code looks similar to this:

In this example, the cell's language is set correctly to F#, but the notebook kernel in the bottom right of the window is what's still set to Python.  Click on the red circle then select ".NET Interactive" from the dropdown:

Thanks for the tip (My earlier problem today was because I disabled the Python extension. I enabled that and am now able to reproduce my issue from the weekend).
See this screenshot, I seem to have every kernel selected correctly at the bottom left. The button on the left of the cell works (see 2nd cell). But the first cell tries the python kernel when I click the "run cell and below" button on the right of the cell.

That cell is still trying to execute with the Python kernel.  Did you re-run that cell?  If not that may be stale output from the previous attempt, but with the kernel selected on the bottom right of the window and the language set at the bottom right of the cell, the execute command should be sent to .NET Interactive and not Python anymore.

It's not stale. I can clear the cell and run the cell with the button on the left of the cell. But if then clear it again and run it with the button on the right-side of the cell, it will try to use the python kernel.

Perhaps this helps? When I do >view reopen editor with I only see

Text editor
Jupyter Notebook (custom)
Jupyter Notebook (preview)

However, I do not see the .NET interactive for Jupyter versions seen in these screen shots at this issue comment https://github.com/dotnet/interactive/issues/1188#issuecomment-806945421
This is despite being able to switch between .NET and Python kernels at the bottom right of the VS Code window.
Version: 1.56.0-insider (user setup)
Commit: 01466cf692c3e54f0eeb82ef0539a42391f185de
Date: 2021-04-12T05:17:32.143Z
Electron: 11.4.2
Chrome: 87.0.4280.141
Node.js: 12.18.3
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042
I have the .NET interactive, jupyter, and python extensions installed.

Can you "open with..." "Jupyter Notebook (preview)"?  We recently updated our Insiders extension to use the Jupyter extension, but it depends on the preview editor; I'm wondering if that notebook was opened with the older Jupyter editor that always assumes Python.

Thanks for the help. This is a quite frustrating debug.

I just now created a new notebook using the command pallette. The kernel defaulted to c#. I changed it to F#. The kernel at the kernel at the bottom right of the window says .net interactive.
I entered

#r "nuget: FSharp.Data"
open System

I ran the cell using the left button. It worked. But when I try the buttons on the right of the cell, now nothing happens. It doesn't run the python kernel, but it also doesn't run the F# kernel.

Thanks for helping us figure this out.  It appears that the issue is the Jupyter extension (which we're using to handle notebook reading/writing and is using us for execution) isn't properly calling into us when you click the run above/below buttons (but it is working when each cell is directly executed.)
I've filed issue microsoft/vscode-jupyter#5497 against the Jupyter extension to track this.
Re-opening for visibility with external flags set.


invalid utf-8 sequence of 1 bytes from index 2

Had mpv-handler from AUR & tamper-monkey on chromium. What's could be the cause & solution ?
invalid utf-8 sequence of 1 bytes from index 2

Please show me these informations:

The video URL when this error occurred.
The base64 encoded URL (right click on the mpv icon to copy)
config.toml & custom.toml (if you created it)

Please show me these informations:

The video URL when this error occurred.
The base64 encoded URL (right click on the mpv icon to copy)
config.toml & custom.toml (if you created it)

Hey apologize for late reply...
Here's the output from your debug build, as per the one you suggested on other opened issue
$ ./mpv-handler 'mpv://aHR0cHM6Ly93d3cueW91dHViZS5jb20vd2F0Y2g/dj1NdmJsZGNrQUY5NA==/?downloader=mpv&quality=best' [src/protocol.rs:88] &data_base64 = "aHR0cHM6Ly93d3cueW91dHViZS5jb20vd2F0Y2g/dj1NdmJsZGNrQUY5NA==" [src/protocol.rs:90] &data_bytes = [     104,     116,     116,     112,     115,     58,     47,     47,     119,     119,     119,     46,     121,     111,     117,     116,     117,     98,     101,     46,     99,     111,     109,     47,     119,     97,     116,     99,     104,     63,     118,     61,     77,     118,     98,     108,     100,     99,     107,     65,     70,     57,     52, ] [src/protocol.rs:92] &data_string = "https://www.youtube.com/watch?v=MvbldckAF94" [cooper@archlinux Downloads]$
Here's what cargo run output looks like:
$ cargo run -- 'mpv://aHR0cHM6Ly93d3cueW91dHViZS5jb20vd2F0Y2g/dj1NdmJsZGNrQUY5NA==/?downloader=mpv&quality=best'    Compiling proc-macro2 v1.0.28    Compiling unicode-xid v0.2.2    Compiling syn v1.0.75    Compiling serde_derive v1.0.129    Compiling libc v0.2.100    Compiling serde v1.0.129    Compiling base64 v0.13.0    Compiling quote v1.0.9    Compiling dirs-sys v0.3.6    Compiling dirs v3.0.2    Compiling thiserror-impl v1.0.26    Compiling thiserror v1.0.26    Compiling toml v0.5.8    Compiling mpv-handler v0.2.13 (/home/cooper/mpv-handler) warning: unreachable statement    --> src/handler.rs:163:9     | 161 |           return Ok(());     |           ------------- any code following this expression is unreachable 162 |  163 | /         let downloader = std::process::Command::new(bin) 164 | |             .args(downloader_options) 165 | |             .arg(&self.protocol.url) 166 | |             .status();     | |______________________^ unreachable statement     |     = note: #[warn(unreachable_code)]` on by default
warning: unused variable: bin
--> src/handler.rs:158:9
|
158 |         bin: &String,
|         ^^^ help: if this is intentional, prefix it with an underscore: _bin
|
= note: #[warn(unused_variables)] on by default
warning: unused variable: downloader_options
--> src/handler.rs:159:9
|
159 |         downloader_options: Vec<&String>,
|         ^^^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: _downloader_options
warning: mpv-handler (bin "mpv-handler") generated 3 warnings
Finished dev [unoptimized + debuginfo] target(s) in 11.48s
Running target/debug/mpv-handler 'mpv://aHR0cHM6Ly93d3cueW91dHViZS5jb20vd2F0Y2g/dj1NdmJsZGNrQUY5NA==/?downloader=mpv&quality=best'
[src/protocol.rs:88] &data_base64 = "aHR0cHM6Ly93d3cueW91dHViZS5jb20vd2F0Y2g/dj1NdmJsZGNrQUY5NA=="
[src/protocol.rs:90] &data_bytes = [
104,    116,    116,    112,    115,    58,    47,    47,    119,    119,    119,    46,    121,    111,    117,    116,    117,    98,    101,    46,    99,    111,    109,    47,    119,    97,    116,    99,    104,    63,    118,    61,    77,    118,    98,    108,    100,    99,    107,    65,    70,    57,    52,]
[src/protocol.rs:92] &data_string = "https://www.youtube.com/watch?v=MvbldckAF94"
[cooper@archlinux mpv-handler]$`

Oh, thanks! That output shows no problem found.
base64::decode to bytes succeed:
[src/protocol.rs:90] &data_bytes = [ 104, 116, 116, 112, 115, 58, 47, 47, 119, 119, 119, 46, 121, 111, 117, 116, 117, 98, 101, 46, 99, 111, 109, 47, 119, 97, 116, 99, 104, 63, 118, 61, 77, 118, 98, 108, 100, 99, 107, 65, 70, 57, 52, ]

String::from_utf8 to string from bytes succeed:
[src/protocol.rs:92] &data_string = "https://www.youtube.com/watch?v=MvbldckAF94"

Can you try branch main? I think it works fine.

Oh, thanks! That output shows no problem found.
base64::decode to bytes succeed and no invalid UTF-8 sequence found:
[src/protocol.rs:90] &data_bytes = [ 104, 116, 116, 112, 115, 58, 47, 47, 119, 119, 119, 46, 121, 111, 117, 116, 117, 98, 101, 46, 99, 111, 109, 47, 119, 97, 116, 99, 104, 63, 118, 61, 77, 118, 98, 108, 100, 99, 107, 65, 70, 57, 52, ]

String::from_utf8 to string from bytes succeed:
[src/protocol.rs:92] &data_string = "https://www.youtube.com/watch?v=MvbldckAF94"

Can you try branch main? I think it works fine.

Yes. No problems & the stream starts playback without any problems.
But the package from aur repo doesn't seem to work somehow. How to compile the binary from your source btw ? 😼

But the package from aur repo doesn't seem to work somehow.

I think this isn't a problem with aur repo, because it's same as the main branch.
We need one more test.

But the package from aur repo doesn't seem to work somehow.

I think this isn't a problem with aur repo, because it's same as the main branch.
We need one more test.

Also can it be issue from userscript side?

Edit YOUR_DEBUG_BINARY_PATH to the path of debug build binary
git clone https://github.com/akiirui/mpv-handler.git -b debug
cd mpv-handler
git pull
cargo build
echo "$PWD/target/debug/mpv-handler"
mkdir -p ~/.local/share/applications
cp /usr/share/applications/mpv-handler.desktop ~/.local/share/applications/
sed -i 's|Exec=mpv-handler|Exec=YOUR_DEBUG_BINARY_PATH|' ~/.local/share/applications/mpv-handler.desktop
update-desktop-database ~/.local/share/applications/

And click mpv icon on the video page in the browser.

How to compile the binary from your source btw?

Ah, that AUR package is compiling from source.

Edit YOUR_DEBUG_BINARY_PATH to the path of debug build binary
git clone https://github.com/akiirui/mpv-handler.git -b debug
cd mpv-handler
git pull
cargo build
echo "$PWD/target/debug/mpv-handler"
mkdir -p ~/.local/share/applications
cp /usr/share/applications/mpv-handler.desktop ~/.local/share/applications/
sed -i 's|Exec=mpv-handler|Exec=YOUR_DEBUG_BINARY_PATH|' ~/.local/share/applications/mpv-handler.desktop
update-desktop-database ~/.local/share/applications/

And click mpv icon on the video page in the browser.

Replacing exec just throws KIO error from kde dialog client Could not find the program 'YOUR_DEBUG_BINARY_PATH'
And issue still persist with the default 'mpv-handler %u' Exec
Also I don't think AUR package is a problem too.
$ pikaur -Si mpv-handler
Repository      : AUR
Name            : mpv-handler
Version         : 0.2.13-1
Maintainer      : akiirui
URL             : https://github.com/akiirui/mpv-handler/
AUR URL         : https://aur.archlinux.org/packages.php?ID=951581
License         : MIT
Votes           : 1
Popularity      : 0.13%
Installed       : Yes
Out Of Date     : No
Depends On      : mpv
youtube-dl
Make Deps       : cargo
git
Check Deps      : None
Optional Deps   : None
Provides        : None
Conflicts With  : None
Replaces        : None
Package Base    : mpv-handler
Last Update     : Tue Aug 24 06:34:37 2021
Description     : Play website videos and songs with mpv & youtube-dl.

Ah, remember edit YOUR_DEBUG_BINARY_PATH in
sed -i 's|Exec=mpv-handler|Exec=YOUR_DEBUG_BINARY_PATH|' ~/.local/share/applications/mpv-handler.desktop

Ah, remember edit YOUR_DEBUG_BINARY_PATH in
sed -i 's|Exec=mpv-handler|Exec=YOUR_DEBUG_BINARY_PATH|' ~/.local/share/applications/mpv-handler.desktop

Fun fact is that the copied url from mpv-handler userscript button works perfectly fine when ran through cli.
It's just the browser redirect throws this specific issue.

That path is previous step echo "$PWD/target/debug/mpv-handler" output.

It's just the browser redirect throws this specific issue.

Yep, so we are looking at what the browser sends now.

That path is previous step echo "$PWD/target/debug/mpv-handler" output.

It's just the browser redirect throws this specific issue.

Yep, so we are looking at what the browser sends now.

Hm, the path is where I cloned debug branch. I assume there's no problem with the installed one from /usr/bin too.

The debug build will print what browser sends.
I want to know what the browser sent. please try it and paste the output to me, thanks.

Emm, this does not look like debug build output.
Ensure the Exec path in ~/.local/share/applications/mpv-handler.desktop is the debug binary?

Emm, this does not look like debug build output.
Ensure the Exec path in ~/.local/share/applications/mpv-handler.desktop is the debug binary?

I infact tried to have exec path as compiled debug binary too Exec=~/Downloads/mpv-handler/target/debug/mpv-handler %u
This is it's output though. Hope it might be helpful...
[src/protocol.rs:88] &data_base64 = "ahr0chm6ly93d3cuew91dhvizs5jb20vd2f0y2g/dj0yZExhdDhqOFdWMA==" [src/protocol.rs:90] &data_bytes = [     106,     26,     244,     114,     25,     186,     151,     47,     119,     119,     119,     46,     123,     15,     117,     118,     27,     226,     206,     206,     99,     111,     109,     47,     119,     103,     244,     203,     104,     63,     118,     61,     50,     100,     76,     97,     116,     56,     106,     56,     87,     86,     48, ] Error: Failed to convert video URL string: invalid utf-8 sequence of 1 bytes from index 2

Thanks, can you give me URL from video page -> mpv icon -> right click -> copy link address?

Thanks, can you give me URL from video page -> mpv icon -> right click -> copy link address?

mpv://aHR0cHM6Ly93d3cueW91dHViZS5jb20vd2F0Y2g/dj0yZExhdDhqOFdWMA==/?downloader=mpv&quality=best

Oh, thanks! Maybe I found the cause of this issue!
The origin URL:
mpv://aHR0cHM6Ly93d3cueW91dHViZS5jb20vd2F0Y2g/dj0yZExhdDhqOFdWMA==/?downloader=mpv&quality=best

The mpv-handler gets:
ahr0chm6ly93d3cuew91dhvizs5jb20vd2f0y2g/dj0yZExhdDhqOFdWMA==

The characters before the first / are all lowercase, Then base64 decoded data is different with origin data.
I cannot reproduce on GNOME xdg-open, and it's maybe caused by KIO xdg-open , it convert all "domain" part to lowercase.
I will report it to upstream, and I will push a magic to fix this problem for mpv-handler.
Finally, thank you for your help debugging!

Oh, thanks! Maybe I found the cause of this issue!
The origin URL:
mpv://aHR0cHM6Ly93d3cueW91dHViZS5jb20vd2F0Y2g/dj0yZExhdDhqOFdWMA==/?downloader=mpv&quality=best

The mpv-handler gets:
ahr0chm6ly93d3cuew91dhvizs5jb20vd2f0y2g/dj0yZExhdDhqOFdWMA==

The characters before the first / are all lowercase, Then base64 decoded data is different with origin data.
I cannot reproduce on GNOME xdg-open, and it's maybe caused by KIO xdg-open , it convert all "domain" part to lowercase.
I will report it to upstream, and I will push a magic to fix this problem for mpv-handler.
Finally, thank you for your help debugging!

Hehe no prob. I'm always available if you need more help. I'd luv to offload all online streams on mpv, caz why not...😀

Please update mpv-handler and userscript~

Please update mpv-handler and userscript~

Error: Wrong protocol URL^C
Warning: Program '/home/cooper/Downloads/mpv-handler/target/debug/mpv-handler' crashed.

Oh

Please update mpv-handler and userscript~

Error: Wrong protocol URL
Warning: Program '/home/cooper/Downloads/mpv-handler/target/debug/mpv-handler' crashed.

Oh, please remove .local/share/applications/mpv-handler.desktop, that is debug file, doesn't need anymore

Oh

Please update mpv-handler and userscript~

Error: Wrong protocol URL
Warning: Program '/home/cooper/Downloads/mpv-handler/target/debug/mpv-handler' crashed.

Oh, please remove .local/share/applications/mpv-handler.desktop, that is debug file, doesn't need anymore

Previous url issue again.
Above error occurs from the recompiled binary after merging latest commits.

Don't forget upgrade usersciprt... It is updated too

Don't forget upgrade usersciprt... It is updated too
latest mpv-handler: 0.2.14 userscript: 2021.09.21

Yep. Updated both actually.

Yep. Updated both actually.

Still Error: Wrong protocol URL`?
C

Yep. Updated both actually.

Still Error: Wrong protocol URL`?
Please copy mpv icon address to me....

cli result with url in it:
mpv-handler mpv://aHR0cHM6Ly93d3cueW91dHViZS5jb20vd2F0Y2g/dj1kMFMyR3A3a2Q1OA==/?downloader=mpv&quality=best
[1] 583433
[cooper@archlinux debug]$ Error: Wrong protocol URL

Yep. Updated both actually.

Still Error: Wrong protocol URL`?
Please copy mpv icon address to me....

cli result with url in it:
mpv-handler mpv://aHR0cHM6Ly93d3cueW91dHViZS5jb20vd2F0Y2g/dj1kMFMyR3A3a2Q1OA==/?downloader=mpv&quality=best
[1] 583433
[cooper@archlinux debug]$ Error: Wrong protocol URL

Ah, I'm sure you're not upgrade userscript. because new URL format is mpv://play/BASE64_DATA. a simple magic.

I had...
Play with mpv v2021.09.21
AUTHOR	Akatsuki Rui
DESCRIPTION	Play videos and songs on the website via mpv-handler
SOURCE	https://greasyfork.org/scripts/416271-play-with-mpv/code/Play with mpv.user.js

Emm. reload the video page and move cursor over the mpv icon, see the URL?
If it starts with mpv://play/ it will work.

Yep. Updated both actually.

Still Error: Wrong protocol URL`?
Please copy mpv icon address to me....

cli result with url in it:
mpv-handler mpv://aHR0cHM6Ly93d3cueW91dHViZS5jb20vd2F0Y2g/dj1kMFMyR3A3a2Q1OA==/?downloader=mpv&quality=best
[1] 583433
[cooper@archlinux debug]$ Error: Wrong protocol URL

Ah, I'm sure you're not upgrade userscript. because new URL format is mpv://play/BASE64_DATA. a simple magic.

Ah, I've upgraded. Just didn't restart browser. Solved now...👍

Ah, I've upgraded. Just didn't restart browser. Solved now...+1

Okay, enjoy~ thanks for your help!


balena-cli: 15.1.1 -> 15.2.2

Description of changes
closes #224403
Updates package and removes support for windows since theres no easy way to test at the moment.
Most of the package users will likely be using WSL.

Things done

Built on platform(s)

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

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 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

[x] Fits CONTRIBUTING.md.

Yeah, the Windows support seems strange... Was it tested with pkgsCross.mingwW64.balena-cli when it was first introduced?

Yeah, the Windows support seems strange... Was it tested with pkgsCross.mingwW64.balena-cli when it was first introduced?

It was not. This is one of my first contributed packages and I made a few errant assumptions when including it. So I figured it best to remove it now.


lime test neko fails on SimpleImage with lime 3.7.2, haxe 3.4.0rc2

On Linux Mint 18.1 MATE x64.
With neko:
$ lime create SimpleImage
$ cd SimpleImage/
$ lime test neko
Called from ? line 1
Called from ApplicationMain.hx line 57
Called from ApplicationMain.hx line 81
Called from ApplicationMain.hx line 183
Called from lime/app/Application.hx line 218
Called from lime/_backend/native/NativeApplication.hx line 144
Called from a C function
Called from lime/_backend/native/NativeApplication.hx line 417
Called from lime/_macros/EventMacro.hx line 101
Called from lime/app/Module.hx line 46
Called from Main.hx line 39
Called from lime/utils/Assets.hx line 225
Called from lime/utils/Assets.hx line 110
Called from lime/utils/Assets.hx line 328
Uncaught exception - Invalid field access : buffer
$ haxelib list
hxcpp: [3.4.43]
lime-samples: [3.4.0]
lime: [3.7.2]
openfl-samples: [4.7.0]
openfl: 3.6.1 [4.7.3]

And with cpp:
$ lime test linux
...(no errors)...
Link: ApplicationMain
Segmentation fault

Same thing with BunnyMark.

Sorry, I guess this belongs over on lime-samples.


Add Material 3 interactive dartpad samples

This is a tracking issue for adding interactive code samples (with {@tool dartpad}) to components and widgets which support Material 3.
We should consider leveraging existing existing code samples by either

wrapping existing samples with a common widget which shows a toggle to switch the sample to Material 3, at some point, have that be the default (preferred, IMO)
copy the simplest existing sample and adding useMaterial3: true

For new Material 3-only widgets, such as NavigationBar, the API doc will only show the Material 3 look.
cc @darrenaustin WDYT?
Components

[x] Buttons
[x] https://github.com/flutter/flutter/pull/101105
[x] #100532
[x] https://github.com/flutter/flutter/pull/101508
[x] NavigationBar
[x] https://github.com/flutter/flutter/pull/101345
[x] https://github.com/flutter/flutter/pull/102823 and https://github.com/flutter/flutter/pull/103962

Other widgets

@guidezpl
i can help with this, let me know :)

This sounds reasonable to me. I have started doing this as I am migrating each component (starting with Card and the still being worked on buttons. But the earlier FAB, NavRail etc, don't yet have M3 examples.

@guidezpl
Working on PRs for FAB and NavRail

Thanks! Are you able to leverage the existing samples without copying them? @gspencergoog, curious if you have any thoughts on this? Added samples here would be identical, except the bool ThemeData.useMaterial3.

Thanks! Are you able to leverage the existing samples without copying them? @gspencergoog, curious if you have any thoughts on this? Added samples here would be identical, except the bool ThemeData.useMaterial3.

Code would be pretty similar but visual it will look quite different and the example description is also pointing to M3 design specs. When useMaterial3 is deprecated, regular M2 can be deleted without refactoring the code.
I'm happy to update PR/Example if the code cannot be identical and I also added fab variant examples and tests for each existing example

That's good point, code duplication for the transition period is acceptable

Working on samples for https://github.com/flutter/flutter/pull/107166

@guidezpl
I'm closing this as done as almost M3 widgets have examples now.

Great thank you! Which are missing?

@guidezpl
https://github.com/flutter/flutter/pull/120855 is missing an example (or an example update)


analytic measurement of floating point ops

I got the output below from SDE for a call to elbo_likelihood with just one pixel active, and without computing the gradient or the hessian. These numbers are quite a bit higher than I had expected based on analytically assessing the objective function.
Search stanza is "EMIT_GLOBAL_DYNAMIC_STATS"
elements_fp_single_1 = 5110262
elements_fp_single_4 = 0
elements_fp_single_8 = 0
elements_fp_single_16 = 0
elements_fp_double_1 = 663372
elements_fp_double_2 = 50
elements_fp_double_4 = 12262
elements_fp_double_8 = 0
--->Total single-precision FLOPs = 5110262
--->Total double-precision FLOPs = 712520
--->Total FLOPs = 5822782
mem-read-1 =<PHONE_NUMBER>
mem-read-2 = 663934316
mem-read-4 =<PHONE_NUMBER>0
mem-read-8 =<PHONE_NUMBER>43
mem-read-16 = 111496360
mem-read-32 = 258667
mem-read-64 = 472888
mem-write-1 =<PHONE_NUMBER>
mem-write-2 = 99580412
mem-write-4 =<PHONE_NUMBER>
mem-write-8 =<PHONE_NUMBER>9
mem-write-16 = 795799808
mem-write-32 = 955132723
mem-write-64 = 0
--->Total Bytes read =<PHONE_NUMBER>208
--->Total Bytes written =<PHONE_NUMBER>25
--->Total Bytes =<PHONE_NUMBER>133


Support mixed16/float16 GroupNormalization

Fix #7964.

I personally came up with this
https://github.com/chainer/chainer/blob/e0d92fe68a5285287d0b4e150abdca1c5b2a9198/chainer/links/normalization/batch_normalization.py#L214
when I read your patch.
On Sun, Aug 18, 2019 at 22:20 Gentaro Watanabe<EMAIL_ADDRESS>wrote:

@gwtnb commented on this pull request.
In chainer/functions/normalization/group_normalization.py
https://github.com/chainer/chainer/pull/7965#discussion_r314985314:

@@ -83,14 +90,19 @@ def forward_cudnn(self, inputs):
x, gamma, beta = inputs
xp = cuda.cupy

   interm_dtype = numpy.promote_types(x.dtype, gamma.dtype)

It seems that interm_dtype was introduced in #6484
https://github.com/chainer/chainer/pull/6484, so we should ask @hvy
https://github.com/hvy.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/chainer/chainer/pull/7965?email_source=notifications&email_token=AD3Q7U5HSNVPT7YTMKK5SW3QFFEBLA5CNFSM4IMO7SIKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCB32YKQ#discussion_r314985314,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD3Q7U3F6GU4KSFT2FSW7DTQFFEBLANCNFSM4IMO7SIA
.

Thanks for the fix. Could you add tests also?

Jenkins, test this please.

Jenkins CI test (for commit fed9bd9600219d7e38e6d9df018438c9e6deace9, target branch master) succeeded!

Maybe this got slipped away because I triggered the CI could you write tests? For BN, it's tested through the link's test.

Jenkins, test this please.

Jenkins CI test (for commit 578b0727fe245c67f9d5e77a949fdc6b1145fe46, target branch master) failed with status FAILURE.

It seems like the test is flaky.

Jenkins, test this please.

Jenkins, test this please.

Jenkins CI test (for commit 25a4eace0e94f3eecebe0849c37bcc80fd095c8e, target branch master) succeeded!

Jenkins, test this please.

Jenkins CI test (for commit 9b034d810ac13fde47a57b28ee16f8361cf401d9, target branch master) failed with status FAILURE.

Could you fix the CI failures? They could be related to this recent change in parameter initialization? https://github.com/chainer/chainer/pull/8022

It seems that the failure is fixed by the following patch, but it should not be included in this PR.
diff --git a/chainer/initializers/constant.py b/chainer/initializers/constant.py
index e98c2e8a3..3ff10163b 100644
--- a/chainer/initializers/constant.py
+++ b/chainer/initializers/constant.py
@@ -54,7 +54,7 @@ class _Constant(initializer.Initializer):
         # Calling copy to ensures that the fill_value array
         # is moved to the device where array resides
         if isinstance(self.fill_value, chainer.get_array_types()):
-            backend.copyto(array, self.fill_value)
+            backend.copyto(array, self.fill_value.astype(self.dtype, copy=False))
         else:
             device = backend.get_device_from_array(array)
             array[...] = device.xp.asarray(self.fill_value)

A question: Is it intended to fail when backend.copyto is called with different dtypes?

Another fix
diff --git a/chainer/backend.py b/chainer/backend.py
index 0de927d4f..0e35ad224 100644
--- a/chainer/backend.py
+++ b/chainer/backend.py
@@ -65,7 +65,8 @@ def copyto(dst, src):
     elif isinstance(dst, cuda.ndarray):
         if isinstance(src, chainer.get_cpu_array_types()):
             src = numpy.asarray(src)
-            if dst.flags.c_contiguous or dst.flags.f_contiguous:
+            if (dst.flags.c_contiguous or dst.flags.f_contiguous) and \
+               src.dtype == dst.dtype:
                 dst.set(src)
             else:
                 cuda.cupy.copyto(dst, cuda.to_gpu(src, device=dst.device))

Should be fixed in https://github.com/chainer/chainer/pull/8043. I went with your second proposal as this seems more like a bug that occurs only when the destination is a CuPy ndarray (it may otherwise take ndarrays of different dtypes).

It ended up being fixed in https://github.com/chainer/chainer/pull/8089.

Jenkins, test this please.

Jenkins CI test (for commit 9b034d810ac13fde47a57b28ee16f8361cf401d9, target branch master) succeeded!


2.0.8 Pagination fix

I found a bug with ng-grid that was not displaying the correct amount of serverItems for a list.
This was because $scope.totalServerItems in the footer.js file was never defined.
To fix this bug, I changed it from $scope.totalServerItems to $scope.pagingOptions.totalServerItems

@c0bra may look at this on the next 2.x merge he does (if any).  I note that:

it has 5 commits, normally we'd only merge things where the commits are squashed.
there seems to be a weird change in ng-grid.debug.js that I don't quite understand, I'd have assumed that was a built file
the travis build failed, I'm not sure if that's normal on 2.x or is a problem.


chore: change company name

#24

Can one of the admins verify this patch?

ok to test

ok to test

ok to test

ok to test

ok to test

ok to test


updated error message

resolves #1
I took the liberty of making some whitespace additions since I always find maven errors terribly hard to find in the console output. Lemme know if you find that problematic.

+1 to the extra padding
Only question is whether there should be a callout on receiving a 401/403 response code. While useful, if a user receives a 404 or 503, those messages would not be applicable. Possibly bound the 401/403 by another condition?

updated per your request @sabre1041

lgtm. Thanks!


Refresh token

added refresh token mechanism
calls the spotify token url and returns a token
resolved conficts

This code change looks good, but for some reason the token doesn't appear to be saving with the current method of saving the token. That has nothing to do with this code change however, so I'll approve and merge this now. Thank you!

Hey @dvdmuckle can u please add hactoberfest or hactoberfest approved label to this pr please


Add smag3 d diffusivity

What changes does this PR introduce?
Allow to use isotropic 3D Smagorinsky diffusivity:
Smag diffusivity is assumed to be linearly dependent on 3D Smag viscosity
(just scaled by inverse of a constant Prandtl number); setting involves:
a) CPP Option: ALLOW_SMAG_3D_DIFFUSIVITY in CPP_OPTIONS.h (def. #undef)
b) setting smag coeff. "smag3D_diffCoeff" for diffusivity (similar to smag coeff.
"smag3D_coeff" for viscosity) in first namelist of main parameter file "data".
What is the current behaviour?
MITgcm allows to compute and use isotropic 3-D Smagorinsky viscosity but diffusivity
remains constant.
What is the new behaviour
This provides a simple way to make diffusivity to vary as function of strain tensor,
like smagorinsky viscosity, assuming a constant and uniform Prandtl number.
Does this PR introduce a breaking change?
No. Additional array stored in common block only when new CPP option is defined.
Other information:
Implementation details:

computed from same place as "viscAh3d_00" (in mom_calc_smag_3d.F), stored
in common block and located at grid-cell center.
simple averaging from grid-cell center to grid-cell faces is applied when
computing diffusion fluxes.
diffusivity is lagging by one time-step:
a) if staggered time-step, smag3D_diffK is not updated after velocity update
and is applied directly to tracers.
b) if synchronous time-step, smag3D_diffK is not updated at the beginning of
time-step and an older diffusivity field is used to step forward tracers.
As a consequence, smag3D_diffK needs to be stored in pickup files for a proper restart.

Suggested addition to tag-index
o model/src & pkg/generic_advdiff:

Implement simple isotropic 3D Smagorinsky diffusivity:
Smag diffusivity is assumed to be linearly dependent on 3D Smag viscosity
(just scaled by inverse of a constant Prandtl number); setting involves:
a) CPP Option: ALLOW_SMAG_3D_DIFFUSIVITY in CPP_OPTIONS.h (def. #undef)
b) switching on 3D Smagorinsky by setting "useSmag3D = .TRUE." in first
namelist of main parameter file "data" ; and, in the same namelist,
c) setting smag coeff. "smag3D_diffCoeff" for diffusivity (similar to
smag coeff. "smag3D_coeff" for viscosity), which results in constant
Prandtl number = smag3D_coeff/smag3D_diffCoeff.
Note: Since Smag3D_diffK is lagging by 1 time-step, needs to be stored
in pickup files (for restart) when using synchronous time-stepping.
code is compiled in experiment tutorial_deep_convection and ready to use
except smag3D_diffCoeff setting in input/smag3d/data is commented out.
also add few missing initialisations in "ini_dynvars.F" (just for
consistency with coding practice but no practical side effect).

@mjlosch I think I am done with code changes: I documented a little bit how to turn this on in tutorial_deep_convection/input.smag3d/data ; the new run-time parameter is documented in PARAMS.h (the same way we do for most parameters), not sure if I need to do things differently for this one that we don't do for others.
And will need to modify the tag-index addition with:
a) how to turn on this Smag3D diffusivity (+ mention  tutorial_deep_convection/input.smag3/)
b) changes in ini_dynvars.F (still looking at a good way to document a fix without leaving the false impression that it might fix
something while in current Fortran  code it does not really fix anything; will come up with something).

OK, I would still think that it would help to understand which ratio is the (numerical) Prandtl number, but if you don't think that's clear, then I am fine with what we have.

@mjlosch I updated my "suggested addition to tag-index" and now mentions the resulting Prandtl number (+ ini_dynvars.F).

LGTM.
We promise to ourselves that there will be documentation someday.

OK, will merge this one soon.


Publish to NPM / Rename Project

Addresses #85
Unfortunately, the device.js name (and all variations) is already parked in NPM. I've reached out to a few folks over the years but haven't been able to secure the original project name.
Publishing this to NPM is long overdue. The most sensible solution seems to be to simply change the project name to something that's available in the NPM registry.
This PR is a WIP step towards renaming and publishing. At the moment, I think the new name will be uasniffer. Maybe not the best name, but gets us over this hurdle.
Open to suggestions though...

Include ua in the name might mislead people because this lib does not tell if it is firefox or chrome... It gives intel about the device.
You should keep device in the name or a ref to it and avoid ref to ua

Good point. Other potential options:

device-intel
device-check

device-intel is the simplest and match what this lib does. Which I had creative imagination to come up with a fun but descriptive name. Naming things is hard

current-device

@rcotter @JSteunou Beautiful - thank you both. I'm going to let this sit ~36hrs for others to chime in w/ thoughts.

https://www.npmjs.com/search?q=dvice


add forceFullscreen

Issue #90
add an option forceFullscreen which is default to be true, which control if the fullscreen is called when toggle side-by-side mode

by the way. Travis Cl said I have some errors in code but I cannot find it

Did you look at the job's details? Did you read my comments on #90? There's a long way to go.


Add found words to output

Closes #25
Output found words to the terminal message so it reads:
"Expected File.php to not have profanity, but found: 'x, y, z'"

Hey @spoyntersmith 👋
Thanks for this PR! This is exactly how I had in mind how it should work, so awesome! Makes it even easier to now spot Profanity!
I'll get this PR merged and a new release tagged. Thanks again for the contribution!


Discuss Default Policy Behaviour for not existing BPNL

As PO,
I want to understand the different possibilities on how the default behaviour has impact on how a business application is using this ,
so that it has a high level of security and understandability .
Hints / Details

Solution 1: keep as is

Solution 2: remove this

Solution 3: change the default value

rough sketch on implications of a business applicatoin

Topics to take into account: functionality, security, understandability

Describe the pro and cons of each Solution

Recomendation for what.

Acceptance Criteria

[ ] Concept on implications of the options has beed documented in confluence

Out of Scope

...

related to #533


Stack Damage Boss Idea

So this boss is based on the idea that there are certain types of terrain around it, and based upon which terrain you are standing on you gain stacks of a certain thing.. (Ice,fire, etc.) Each stack has a different effect ice slowing, fire is DOT damage, earth/acid armor reduction, and wind miss chance for example. Each stack increases the effect of the applicable stack. Other than this the boss sends out attacks on the different terrain types randomly and aoe of just that terrain and you take more/less damage depending upon the number of stacks you have of that terrain. The boss does not auto attack. This would be a low level stage 1 boss most likely though if others think of good second or third stages might be good later too.

Thank you for the submission, and sorry for the wait. Things have been hectic lately!
Could you format this according to #37 ? It would help a ton!


API Calls Unauthorized (Images Not Loading)

Expected Behavior
View full-size images with frontend ui.
Actual Behavior
Thumbnails are created and present, but clicking on the thumbnail does not display a full-size image as expected.  Instead it acts as though it's perpetually loading with loading circles on some of the top-right icons.
Additional Details
I've set this up on two different installs (CentOS 7 & Ubuntu 18.04).  I've restarted with clean containers on each with no success.  The only indication that something is wrong is in the file gunicorn_django.log below:
Unauthorized: /api/persons/
Unauthorized: /api/albums/place/list/
Unauthorized: /api/albums/date/photohash/list/
Unauthorized: /api/albums/thing/list/
Unauthorized: /api/albums/user/list/
Unauthorized: /api/searchtermexamples/
Unauthorized: /api/stats/
Unauthorized: /api/rqavailable/
Unauthorized: /api/rqavailable/
Unauthorized: /api/rqavailable/
Unauthorized: /api/rqavailable/

Related to #26
Did you scan finish? What does your web console say?

Related to #26
Did you scan finish? What does your web console say?

@derneuere Yes.  Scan completed.

@derneuere Yes.  Scan completed.

I made a duplicate issue #107 just found this one

I made a duplicate issue #107 just found this one

Fixed the issue. Please do a docker-compose down, docker-compose pull and docker-compose up -d

Fixed the issue. Please do a docker-compose down, docker-compose pull and docker-compose up -d

Amazing! Just had the same issue on a fresh install … 12 hours later it's fixed! Thanks!

Amazing! Just had the same issue on a fresh install … 12 hours later it's fixed! Thanks!

@derneuere Thank you!!  Issue is resolved now.

@derneuere Thank you!!  Issue is resolved now.