[SPARK-25908][CORE][SQL] Remove old deprecated items in Spark 3

What changes were proposed in this pull request?

Existing tests

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/testing-k8s-prb-make-spark-distribution-unified/4693/
Test PASSed.

Test build #98354 has started for PR 22921 at commit 7a0ecd2.

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/testing-k8s-prb-make-spark-distribution-unified/4694/
Test PASSed.

seems good to me; might want to leave this open for a few days so more people can take a look

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/testing-k8s-prb-make-spark-distribution-unified/4702/
Test PASSed.

Test build #98363 has started for PR 22921 at commit d50b5b5.

Test build #98354 has finished for PR 22921 at commit 7a0ecd2.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98354/
Test FAILed.

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/testing-k8s-prb-make-spark-distribution-unified/4709/
Test PASSed.

Test build #98373 has started for PR 22921 at commit bd4f5ab.

Test build #98373 has finished for PR 22921 at commit bd4f5ab.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98373/
Test FAILed.

LGTM except a minor comment about the test case. Also we need to fix the PySpark test failure

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/4728/
Test FAILed.

Test build #98414 has started for PR 22921 at commit 57ef4e8.

Looks okay to me too but I'd also leave this open for few more days.

Yeah it's a good point that these weren't deprecated, but I assume they should have been. Same change, same time, same logic. given that it's a reasonably niche method, I thought it would be best to go ahead and be consistent here?

Test build #98433 has started for PR 22921 at commit df92f0f.

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/testing-k8s-prb-make-spark-distribution-unified/4742/
Test PASSed.

Test build #98433 has finished for PR 22921 at commit df92f0f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/98433/
Test PASSed.

Test build #98480 has started for PR 22921 at commit a6891f7.

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/testing-k8s-prb-make-spark-distribution-unified/4776/
Test PASSed.

Test build #98480 has finished for PR 22921 at commit a6891f7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/98480/
Test PASSed.

Test build #98536 has started for PR 22921 at commit 6bcbf79.

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/testing-k8s-prb-make-spark-distribution-unified/4804/
Test PASSed.

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/testing-k8s-prb-make-spark-distribution-unified/4805/
Test PASSed.

Test build #98537 has started for PR 22921 at commit af748d5.

Test build #98537 has finished for PR 22921 at commit af748d5.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98537/
Test FAILed.

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/testing-k8s-prb-make-spark-distribution-unified/4815/
Test PASSed.

Test build #98551 has started for PR 22921 at commit 3070975.

Test build #98561 has started for PR 22921 at commit 9f1ced3.

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/testing-k8s-prb-make-spark-distribution-unified/4823/
Test PASSed.

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/SparkPullRequestBuilder/98551/
Test PASSed.

Test build #98561 has finished for PR 22921 at commit 9f1ced3.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98561/
Test FAILed.

Test build #4419 has started for PR 22921 at commit 9f1ced3.

Test build #4419 has finished for PR 22921 at commit 9f1ced3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged to master


async-data example broken

There are no cross domain headers being send from the heroku api and the data is inserted in data.undefined instead data.contacts

+1, also saw this


bump version to 2.2.0

Codecov Report

Merging #33 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #33   +/-   ##
=======================================
  Coverage   90.34%   90.34%
=======================================
  Files           1        1
  Lines         290      290
=======================================
  Hits          262      262
  Misses         28       28

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 79dc066...363baf0. Read the comment docs.


Feature/ci

travis.yml
script that checks that there are no unused imports
removed unused dependencies

It would be lovely to publish to the 0.19 ecosystem when we can, but maybe you aren't ready to upgrade your app to 0.19 yet?

Definitely. I was about to create an issue for that.


migration-engine-rhel-openssl-1.0.x is not downloaded when running Prisma generate, trying to deploy to AWS with AWS CDK

Bug description
migration-engine-rhel-openssl-1.0.x will not be downloaded
query-engine-rhel-openssl-1.0.x can be find at node_modules/.prisma/client, but not at node_modules/@prisma/engines
actually, no node_modules/@prisma/engines folder at all
How to reproduce

mkdir a
cd a
npx prisma init
put your local db link in .env
yarn add @prisma/client
yarn generate
No node_modules/@prisma/engines created
No migration-engine-rhel-openssl-1.0.x anywhere

Expected behavior
all rhel-openssl-1.0.x drivers should be downloaded in node_modules/@prisma/engines
TBH, I do not know if this is the expected way to work with AWS Lambda, because the documentation does not really show us the steps and necessary steps. Just trying xD
Prisma information
datasource db {
  provider = "postgresql"
  url      = env("DATABASE_URL")
}

generator client {
  provider = "prisma-client-js"
  binaryTargets = ["native","rhel-openssl-1.0.x"]
}

model User {
  id String @id
}

Environment & setup
OS X: 11.3.1
PostgreSQL: 13
DATABASE_URL=postgres://AlbertGao@localhost:5432/test
Prisma Version
"@prisma/client": "^2.25.0"

This behavior is actually expected since the binaryTargets parameter only applies to the generator, so the Query Engine.
You can try adding this env var which will be read during npm install and the corresponding binaries will be downloaded for the CLI:
PRISMA_CLI_BINARY_TARGETS=native,rhel-openssl-1.0.x

https://www.prisma.io/docs/reference/api-reference/environment-variables-reference#prisma_cli_binary_targets
I think the documentation could be better for this.

What are you trying to do on Lambda that you need the Migrations Engine binary @Albert-Gao?
What error message are you getting when you run it anyway?

Ping @Albert-Gao

Ping @Albert-Gao a last time, or we will need to consider this solved with the link that Joel shared.


#2797: Curves: Custom fit function doesn't fit correctly

"fitFunction": {
"name": "LinearLogFunction",
"function": "([a, k], x) => a * Math.log(x+1) + k",
"getInitialParameters": "(xs, ys) => [0.1, -1, 4, 4]",
"parameterNames": ["Slope", "Intercept"]
}

This issue has been mirrored in Jira: https://reddata.atlassian.net/browse/GROK-15523

[ ] include this function into standard Curves fit functions

To make the custom fit function work correctly on the Curves v1.4.0, you will need to change it to the following:
"fitFunction": {   "name": "LinearLogFunction",   "function": "([a, k], x) => a * Math.log(x+1) + k",   "getInitialParameters": "(xs, ys) => [-5, 100]",   "parameterNames": ["Slope", "Intercept"] }

To use the log-linear fit function on the Curves v1.4.2, you will need to change it to the following:
"fitFunction": "log-linear"


fix: changed the link and text

What type of PR is this? (check all applicable)

[] Refactor
[ ] Feature
[X] Bug Fix
[ ] Optimization
[ ] Documentation Update

Description
Changing the sbtc link and text
Issue ticket number and link
Slack context
Checklist:

[X] I have performed a self-review of my code.
[X] I have tested the change on desktop and mobile.
[X] I have added thorough tests where applicable.
[X] I've added analytics and error logging where applicable.

Screenshots (if appropriate):

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


Partial updates of location info.

Currently we do only complete updates of the location info. If a geometry is stored in a location field the complete location is reloaded.
We need to think about a method where only updated fields are requested once a location geometry is modified.

Does this refer to behaviour in v2 only as in M&B v1 when you draw drivetimes the primary catchments don't seem to appear.  Or is that as the primary and secondary catchments don't appear by default?

This is unrelated. It is easy to add a flag display: true to control whether a geometry should be shown. This would however mean that existing workspaces without the flag will not be shown by default.
The issue in this post is what should happen if a new geometry is created. At the moment the complete infoj updates since some values may be related to the geometry. For example a count of points inside a newly created isochrone polygon.
What could happen is that the infoj is parsed upon creation of the polygon and only fields which are affected (regex check on the geometry field name) will be updated accordingly.

This works now as follows.
After the update the entries with newValues are filtered from the infoj array.
If the entry has a dependents array then we iterate through the dependents.
If a dependent matches a field (unique) of another entry that entry value will be updated.
If a dependent matches a query of another entry and that entry has an update method associated with a dataview then the update method will be called.
    location.infoj
      .filter(entry => typeof entry.newValue !== 'undefined')
      .forEach(entry => {

        entry.dependents && entry.dependents.forEach(dependent => {
          if (location.infoj.some(entry => entry.field === dependent)) return dependents.push(dependent);
          location.infoj
            .filter(entry => entry.query === dependent)
            .forEach(entry => entry.update && entry.update());
        });

        entry.value = entry.newValue;
        delete entry.newValue;
      });

There after the view will be updated. New values will be shown the updated dataviews will be added to where they belong.


fix: compatibility on safari

brilliant job! does this mean we can't use position: absolute inside the foreignObject.


change PICO-8 plugin for VSCode to pico8-ls

This PR changes the PICO-8 plugin recommended for VSCode to pico8-ls which has much better language support, a ton more features, and is in active development.
(The currently listed one, vscode-pico8, has 1 commit from 2016 and only provides syntax highlighting.)

Thanks for merging!


docs: add docs for java smart contracts

Description:
Commit Message
docs: add docs for java smart contracts

see the guidelines for commit messages.
Changelog Entry
version: <log entry>

Checklist:

[X] I have performed a self-review of my own code
[X] I have documented my code in accordance with the documentation guidelines
[X] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have run the unit tests
[X] I only have one commit (if not, squash them into one commit).
[X] I have a descriptive commit message that adheres to the commit message guidelines

Please review the CONTRIBUTING.md file for detailed contributing guidelines.

Codecov Report

Merging #622 (53f2c3b) into main (d75ae87) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 53f2c3b differs from pull request most recent head d82b71d. Consider uploading reports for the commit d82b71d to get more accurate results

@@            Coverage Diff            @@
##               main     #622   +/-   ##
=========================================
  Coverage     68.32%   68.32%
  Complexity      408      408
=========================================
  Files           129      129
  Lines         12626    12626
  Branches        294      294
=========================================
  Hits           8627     8627
  Misses         3843     3843
  Partials        156      156

Flag
Coverage Δ

java
80.08% <ø> (ø)

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


name resolution failures inside openbazaard in docker

Using openbazaar/server:v0.12.0 (and my own image which substitutes alpine for scratch for the final image created), apparently any service that requires openbazaard to resolve DNS names fails.

new nodes cannot use DNS to find coin-daemon peers to sync block headers

adding a trusted peer works around this issue

smtp notifications cannot look up the smtp gateway hostname
exchange rates are unavailable

this causes "Vendor rejected order" messages when customers try to purchase fiat-priced items
also causes the node to be unable to purchase items form other vendors

unverified, but I'm betting fee estimation isn't working right

  18:59:01.991 [fetch] [ERROR] Failed to fetch from
  https://api.bitcoincharts.com/v1/weighted_prices.json Get
  https://api.bitcoincharts.com/v1/weighted_prices.json: dial tcp: lookup api.bitcoincharts.com: no
  such host
  18:59:01.991 [fetchCurrentRates] [ERROR] Failed to fetch bitcoin exchange rates

tcpdump shows that no lookups are being performed at all
with my alpine based ob container, I've tested name resolution inside the container, and that's working fine from a shell.
nodes are configured as per this config: ob://QmY8HpD6iLHqZUErP6PZhW9QGiqFehFRBbAGVLcA34mxdk/store/docs-ob-server-config-remote-server-config-example
Nodes are also running with the -l debug switch, and there's really nothing more in the logs.

Strange, I've not seen this. Our search infrastructure and other systems use the docker image and haven't had this issue.
Am I correct in understanding that it's not working with an Alpine base either? That's weird because my only guess would be something missing in SCRATCH.
@christroutner Have you seen this?

OK. Appears I've tracked this down a long standing issue in docker - it's not clear why you guys aren't seeing this, but that's kinda docker for you.
Per this docker forum thread they're setting an obscure option for no reason that I can imagine (ndots:0)  in /etc/resolv.conf which I confirmed to be in my containers.
Changing this while openbazaard is running did not appear to sole the issue, but changing it prior to running the daemon did solve the problem. No hostname lookup failures when it checks exchange rates, smtp connection works, buy buttons that rely on exchange rates work.
I've added an entrypoint.sh to my docker image to modify the contents of /etc/resolv.conf prior to running openbazaard. This, of course requires a minimal shell, and alpine + ash seems to work fine, with a few compile errors that should be investigated.
I can file a pr if you like

@guruvan Nice work tracking this down. Glad you've got something working for now. It's definitely a strange issue that I haven't seen before.

I can file a pr if you like

I'd definitely like to see the change you made to /etc/resolve.conf but I want to research the issue some more before modifying our Dockerfile. Could you push your changes to a branch somewhere so I can check them out?

https://github.com/mazaclub/openbazaar-go/tree/fix/docker-ndots

automated builds on dockerhub are setup for this now - https://hub.docker.com/r/mazaclub/openbazaar-go/
mazaclub/openbazaar-go:latest is just these changes based on  v0.12.0 as is the above branch


feat: add support for importing functionality from other libs

This would address this https://github.com/pkg-size/action/pull/45#discussion_r877451432

Thanks for this PR.
Closing as I withdrew my suggestion to use npx ci in pkgsize-action.


You can make compilation and testing easier

Hello! I wanted to run the code with the command yarn watch and it throws the error:
"npm-run-all" is not internal or external
command, executable program, or batch file. error Command failed with exit code 1.

Can you do testing and compilation not through yarn, but through npm?
If you can help me, please write me on my Discord: IThackpe#4756

/needsMoreInfo
Are you trying to compile and test VS Code itself? If so, have you seen https://github.com/microsoft/vscode/wiki/How-to-Contribute?

/ needsMoreInfo
Вы пытаетесь скомпилировать и протестировать сам VS Code? Если да, то видели ли вы https://github.com/microsoft/vscode/wiki/How-to-Contribute ?

Yes

/needsMoreInfo
Are you trying to compile and test VS Code itself? If so, have you seen https://github.com/microsoft/vscode/wiki/How-to-Contribute?

Yes, I see this page. But I can't find a command for testing and contibure 😢 (I find a command yarn watch but it doesn't moving. Do you know how to test or contibure or solve the problem?)

@connor4312 can you please help with the testing.

We do not have any plans to make VS Code run using npm instead of yarn.


Chisel is not inferring the correct width of register

Hi,
I have experienced some issues in trying to have Chisel inferring the correct number of widths for a register.
In particular, the register should be 6-bits, but instead, a 1-bit register is defined in the Verilog.
I tried to create an example code to reproduce the issue:
class width_error extends Module{
  val io = IO(new Bundle{
    val p1 = new HMC_port()
  })

  val regTag = Reg(init = 0.U)
  val readCounter = Counter(100)

  when(readCounter.value < 50.U){
    io.p1.cmd.bits.tag := regTag
    regTag := regTag + 1.U
    readCounter.inc()
  }

}

where HMC_port is defined as follows
class HMC_port extends Bundle{

  val cmd = Decoupled(new HMCCommand)
  val wr = DecoupledIO(UInt(128.W))
  val rd = Decoupled(new HMCResponseChannel).flip

  override def cloneType = {new HMC_port().asInstanceOf[this.type ]}
}

class HMCCommand extends Bundle{
  val addr = UInt(34.W)
  val size = UInt(4.W)
  val tag = UInt(6.W)
  val cmd = UInt(4.W)

  override def cloneType = { new HMCCommand().asInstanceOf[this.type] }
}

class HMCResponseChannel extends Bundle{
  val data = UInt(128.W)
  val errstat = UInt(7.W)
  val dinv = UInt(1.W)
  val tag = UInt(6.W)

  override def cloneType = { new HMCResponseChannel().asInstanceOf[this.type ]}
}

I attach also a version of the Verilog that is produced from this code.
width_error.v.zip

I assume you are talking about the width of regTag.  regTag is taking the width from the init statement.  The width of 0.U is one.  When creating registers with an initial value you should specify the desired width as part of the initializing constant, in your case, the proper line should be something like
val regTag = RegInit(0.U(6.W)

Chisel does not back-propogate width inference, only forward-propogation up to the specified IO outputs.
  val regTag = Reg(init = 0.U) // creates a register of unknown width
  ...
  regTag := regTag + 1.U // adds a one-bit wire to a register of unknown width
  ...
  // chisel is NOT going to back-propogate width-inference, so a 1-bit register is written into a 6-bit output.
  io.p1.cmd.bits.tag := regTag

Thanks @lorenzoditucci, I have opened an issue #650  to help us track adding the proper documentation for this behavior.

As an addendum:
val regTag = Reg(init = 0.U)
regTag := regTag + 1.U
The init adds the constraint that width >= 1
The + is a non-extending add so the connection adds the constraint that width >= max(width, 1)
The width inference attempts to minimize widths so the result is width=1. Note that using +& would result in an error (assuming there are no remaining FIRRTL bugs surrounding this issue).

ok I see, thanks for the help,  I did not know that.
I was also wondering if would not be worthy to at least generate a Warning for such a behavior.

Following up on the point made by @lorenzoditucci, how might a warning have made this less unintuitive?
Could this particular case (register declared without width and width of input to it is unknown) be detected?
Additionally, the IO assignment io.p1.cmd.bits.tag := regTag leaving 5 bits unassigned is also troubling. I don't want to reignite the debate about warnings for unconnected IOs, but perhaps in this special case where the unconnected IOs are due to width inference, this could be also be detected.
Either way, +1 for clearing this up in the docs.


Get General Help

What's your nodejs Version.
Show the output of: node -v
v18.18.0
What's the version you downloaded the bot?
From the releases e.g.: latest
Desktop (please complete the following information):

OS: windows 11
Version latest

Additional context, text and screenshots!

Add any other context about the problem here.

can you guide me though installing?

DM me on discord,  username bestgamershk
On Fri, 29 Sept, 2023, 6:46 pm hotsu0p, @.***> wrote:

What's your nodejs Version.
Show the output of: node -v
v18.18.0
What's the version you downloaded the bot?
From the releases e.g.: latest
Desktop (please complete the following information):

OS: windows 11
Version latest

Additional context, text and screenshots!
Add any other context about the problem here.
can you guide me though installing?
—
Reply to this email directly, view it on GitHub
https://github.com/BestGamersH/Multipurpose-Discord-Bot/issues/13, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AY4XKL6GCN5YX3TWRQKVQGLX43C4HANCNFSM6AAAAAA5MOY2OE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Colour with transparency

With the new text display entities comes an obvious good use case to stick components everywhere to make stuff pretty. However, I can't use adventure in the entire creation of a display entity because we do not have a colour object/interface/etc. that has transparency.

We specifically filter out alpha on TextColor -- otherwise you can get components with differing colours that are visually identical. I think RGBLike would mostly fulfill the role of "interface allowing colour with alpha". Would it be sufficient to do a pass over the API to make sure RGBLike is accepted in more places?

Shadow colour adds the ARGBLike interface, does that meet your needs here?


Update sbt-wartremover, wartremover to 3.0.12

About this PR
📦 Updates

org.wartremover:sbt-wartremover
org.wartremover:wartremover

from 3.0.7 to 3.0.12
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.wartremover" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.wartremover" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #312.


Vim Plug install fails

Followed the instructions for vim-plug:
Plug 'Exafunction/codeium.vim' in my .vimrc.
running :PlugInstall:
x codeium:
    fatal: invalid reference: master

To fix it, you can either use master branch as default or change readme to point to your custom default branch name for Plug:
 Plug 'Exafunction/codeium', { 'branch': 'main' }

Thanks for the report. I think we can accept a PR to change the docs to point to main if you wanted to submit something to close this issue.

Created a PR: https://github.com/Exafunction/codeium.vim/pull/178


Facilitate code scan and report retrieval directly within plugin

The plugin could allow to scan code and load reports directly without the intermediate step of using the sechub client first.

Albert Weißert<EMAIL_ADDRESS>Daimler TSS GmbH, imprint

At https://github.com/Daimler/sechub/wiki/IDE-integration we provide an overview of features and when they are available inside dedicated IDEs.


Remove redundant prop from DropdownProps

Props defined in Dropdown.tsx contain both 'defaultValue' and 'initialValue'. This should be one prop. Se #1031 or the following link for more.
https://github.com/Samfundet/Samfundet4/pull/1031/files#diff-7af625ab247008903c3dce2faa3d6cc6117c12701a8c103994cd8f9a0bb0dbc9

Additionally, the options prop and the parameter of the onchange prop being optional could be reconsidered


RAS and IAS servers is under Builtin NOT Users

RAS and IAS servers are located under Builtin NOT Users.   Also, you do not double-click RAS and IAS Servers.  You right click RAS and IAS Servers, select Properties then select Members and select the member you want to remove.

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

ID: 222a55e3-1b84-8f0e-2f26-8039c86a6925
Version Independent ID: fa9843be-d359-23f9-2a77-32455769066e
Content: Unregister an NPS from an Active Directory Domain
Content Source: WindowsServerDocs/networking/technologies/nps/nps-manage-unregister.md
Product: windows-server
Technology: networking
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_296d675b-bdab-4474-8b3c-b4a1fd7d57d3/pull/1 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-archive-test-target_296d675b-bdab-4474-8b3c-b4a1fd7d57d3.git

Docs Build status updates of commit 7e23e78:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-ServiceAccountCannotAccessRepository] Your repository https://github.com/VSC-Service-Account/docs-archive-test-source has enabled Docs Build service account. It seems that at least one of the accounts, e2ebd4, does not have Read permission to the repo. Please ask your repo admin to grant correct permission to service accounts (refer to this document: https://review.docs.microsoft.com/en-us/engineering/projects/ops/engdocs/how-to-grant-service-account-permission-in-your-repository?branch=master#ops-e2e-ppe) and then retry the build. Don't know who to contact? Refer https://repos.opensource.microsoft.com/VSC-Service-Account/repos/docs-archive-test-source/permissions/ to get admin list of the repo if the repo is in one of the organizations Microsoft owns.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Flake8

Flake8 pre-commit checks
setup.cfg
isort pre-commit checks

Going to merge this, if there are no more comments.

Cool, looks fine
On Tue, Jun 9, 2020 at 1:04 AM Sharad Chitlangia<EMAIL_ADDRESS>wrote:

Going to merge this, if there are no more comments.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/SforAiDl/genrl/pull/159#issuecomment-640839279, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AGQWIY42OC73YWA5IGORDKLRVU4MFANCNFSM4NXEEATQ
.


Build/k2

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

:exclamation: No coverage uploaded for pull request base (renovate/patch-kotlin-monorepo@ceeea45). Click here to learn what that means.

Additional details and impacted files
@@                        Coverage Diff                        @@
##             renovate/patch-kotlin-monorepo     #429   +/-   ##
=================================================================
  Coverage                                  ?   69.71%
=================================================================
  Files                                     ?       53
  Lines                                     ?      733
  Branches                                  ?       79
=================================================================
  Hits                                      ?      511
  Misses                                    ?      191
  Partials                                  ?       31

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


prql

Fixes #1075.
Doesn’t work with interpolated expressions, e.g.,
from gaia
filter phot_g_mean_mag < ${min}
sort {phot_g_mean_mag}
take 10

I’m not sure how we’d implement that… we’d need some way to represent placeholder values (such as ?) that are appropriately substituted during the transformation. PRQL does appear to support parameters so we should be able to use that: https://prql-lang.org/book/reference/syntax/parameters.html.
I’m also not sure if we also want to support a prql tagged template literal, since that would mean loading npm:prql-js in the browser (importing it dynamically, presumably?).

Just so you're aware -- with the next release, importing prql-js might change? I've been pursuing prql in observable-framework myself here, and that change -- switching the browser build to ESM module, hasn't been part of a release yet AFAICT.

That’s great to hear @srenatus. An ESM distribution would be welcome.


Replace gen.Return() with bare return statements

According to http://www.tornadoweb.org/en/stable/gen.html#tornado.gen.Return, this is preferred when returning early from a coroutine.
A bare return has the benefit of not being caught by an except Exception block in the same coroutine.

https://ocd.appscale.com:8080/job/Daily Build/4892/


revendor resinit

revendor this after landing the upstream
A possible fix for https://github.com/keybase/keybase-issues/issues/3022. This PR is mainly for CI.

Updated with a proper vendor commit. @maxtaco you've already seen this upstream, could you rubberstamp here?


lcov: fix build on Darwin

Since Memory-Process is not listed as supported on Darwin, and according to https://github.com/linux-test-project/lcov/issues/238#issuecomment-1709020564, it's not required as a dependency. Therefore, it's safe to remove Memory-Process from the perlDeps on Darwin and get it working again.
This should resolve #323085.
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[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

[x] 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/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review run on aarch64-darwin 1

2 packages built:

lcov
makeGCOVReport


Recycle list fixbug

Please make sure these boxes are checked before submitting your PR, thank you!

[ ] Make sure you follow DiDi's contributing guide.
[ ] Make sure you are merging your commits to dev branch.
[ ] Add some descriptions and refer relative issues for you PR.

Codecov Report

Merging #410 into dev will increase coverage by 0.19%.
The diff coverage is 96.55%.

@@            Coverage Diff             @@
##              dev     #410      +/-   ##
==========================================
+ Coverage   92.68%   92.88%   +0.19%
==========================================
  Files         139      139
  Lines        2967     2992      +25
  Branches      489      487       -2
==========================================
+ Hits         2750     2779      +29
  Misses        108      108
+ Partials      109      105       -4

Impacted Files
Coverage Δ

src/components/recycle-list/recycle-list.vue
97% <96.55%> (+6.33%)
:arrow_up:

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 f6d4cf6...a8d3559. Read the comment docs.


TODOS for read_txt_Witec_ASCII

[x] find out better name - waiting for Ute's reply
[x] provide axis labels

[x] x axis units are in $SpectrumHeader$XDataKind

[x] allow users to specify extra data to keep
[x] unit testing
[x] documentation

I think Claudia and I cleared everything off of this list. Closing this now.


Added Chinese translation

哈哈哈感谢，用ai翻译过，正校对中，发现您已经上传过了。感谢，省了我的脑细胞

赞👍

@RobertD502 Can it be merged? thanks

Hi, the new PetKit feeder will be included in a future release and will have additional keys that aren't in the current translation files that will need to be translated. I am keeping this open and will get back to you once the additions are ready to be translated.


Fix Missing key when creating sub-partial tries error

Two blocks from the hard test chain, namely 19240095 and 19240361, are failing with error
Missing key {...} when creating sub-partial tries, hence during witness pre-processing before actual Kernel witness generation.
It seems to hint that the aggressive trie pruning approach may still have some incomplete edge cases?

Both of these blocks now prove fine with William's (currently unmerged) PR #117, so I'm going to close preemptively.


EnttecOpenUSBDMXDriver

hello,
I'm using it with a 8-channel mode with that.
CLNB600_Cameo_DMX_Control_Table_EN.pdf
I'm just trying to move to proper position with a white light
import { DMX, EnttecOpenUSBDMXDriver, Animation } from "dmx-ts";
import { EnttecOpenDMXUSBDevice } from "enttec-open-dmx-usb";

const dmx = new DMX();

enum C {
  MechPan = 1,
  MechTilt = 2,
  Dimmer = 3,
  Strobe = 4,
  Red = 5,
  Green = 6,
  Blue = 7,
  White = 8,
}

(async () => {
  const universe = await dmx.addUniverse(
    "demo",
    new EnttecOpenUSBDMXDriver(await EnttecOpenDMXUSBDevice.getFirstAvailableDevice()),
  );
  universe.updateAll(0);
  universe.update({ [C.MechPan]: 255, [C.Dimmer]: 255, [C.White]: 255 });
})();

But this is glitchy as hell look at this :
https://user-images.githubusercontent.com/6640835/212352663-b77a8078-423d-4cf9-a5be-3a5a5244f123.MOV
What I'm supposed to understand ?

Hi @ScreamZ,
the readme states the following:
enttec-open-usb-dmx: driver for "Enttec Open DMX USB". This device is NOT recommended, there are known hardware limitations and this driver is not very stable. (If possible better obtain a device with the "pro" chip)

I cannot watch the video, I only hear audio. Maybe it only works on Mac? Don't know.
However, you might want to try the following things:

Play around with the options. Maybe lower the baud rate?
You could also try the TypeScript Version of this library, https://github.com/node-dmx/dmx-ts It has slightly more up-to-date drivers.
I don't think that this is going to change much, though, as your code seems to send the values only once. That means, flickering may be introduced by the Interface and not the library?

Are you using a DMX Terminator?

I changes the video to mp4, check it :)
For now I've this one, I'll check if  ifound something but this gets expensive and i want to be sure its related to interface.
I'm using the typescript already, really nice.
Yes I'm using a terminator. Its the only device. The baud rate seems to be a constant of the protocol, ima check,.
Also used : https://github.com/moritzruth/node-enttec-open-dmx-usb/issues/82 but have other issues.
I'll maybe try implement a new driver ?

I also initially tried this library but because it didn’t work I wrote https://github.com/moritzruth/node-enttec-open-dmx-usb.
I assume it has to do something with the timing of BRK and slight inaccuracies of setTimeout, but I really don’t feel like debugging that stuff. Especially as it is working for me.


Update dependencies

Pull Request Test Coverage Report for Build 3534

16 of 19 (84.21%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.09%) to 66.003%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/util/import_device.ts
8
11
72.73%

Totals

Change from base Build 3527:
0.09%

Covered Lines:
7546

Relevant Lines:
10772

💛  - Coveralls


close() is not closing struct (0.3.0-alpha1)

What version of CUE are you using (cue version)?

$ cue version
cue version devel darwin/amd64

Does this issue reproduce with the latest release?
it works as expected with 0.2.2
What did you do?

-- test.cue --
A: close({
	a: 1
	b: 2
})

B: A & {
	c: 3
}

Now, run

cue export test.cue

What did you expect to see?
An error such as:

B: field "c" not allowed in closed struct:
    test.cue:8:5

What did you see instead?
A happy output, as if A had not been closed:

{
    "A": {
        "a": 1,
        "b": 2
    },
    "B": {
        "a": 1,
        "c": 3,
        "b": 2
    }
}

This issue has been migrated to https://github.com/cue-lang/cue/issues/490.
For more details about CUE's migration to a new home, please see https://github.com/cue-lang/cue/issues/1078.


Make leap_day=True default for PSM3 (deprecation)

[x] Closes #1481
[x] I am familiar with the contributing guidelines
[x] Tests added
[ ] Updates entries in docs/sphinx/source/reference for API changes.
[ ] Adds description and name entries in the appropriate "what's new" file in docs/sphinx/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).
[x] New code is fully documented. Includes numpydoc compliant docstrings, examples, and comments where necessary.
[x] Pull request is nearly complete and ready for detailed review.
[x] Maintainer: Appropriate GitHub Labels (including remote-data) and Milestone are assigned to the Pull Request and linked Issue.

It has been decided to change the default value for the leap_day argument in the get_psm3 function from False to True (see #1481). This PR adds a deprecation warning stating that the default will be changed to leap_day=True in pvlib 0.11.

@pvlib/pvlib-maintainer This PR is ready for a second review if anyone wants to take a look


change colored text from default Teal on input[type=text]:focus

I'm able to change the underline color on the animation but not the text color:
input[type=text]:focus{
border-bottom: 1px solid black;
box-shadow: 0 1px 0 0 black;
}

This is trivial to do in Sass. But we have added more explanation on how to do this in pure CSS.
Added more documentation in ebdce3bee395f1255c57f8a0fde5ac71a59cca3f


DPOTrainer Problem: trl/trainer/utils.py:456

The problem happened in trl/trl/trainer /utils.py in line 456
else:
    # adapted from https://stackoverflow.com/questions/73256206
    if "prompt" in k:
        to_pad = [torch.LongTensor(ex[k][::-1]) for ex in batch]
    else:
456        to_pad = [torch.LongTensor(ex[k]) for ex in batch]
    if k.endswith("_input_ids"):
        padding_value = self.tokenizer.pad_token_id

I am using Qwen/Qwen-1_8B-Chat model and official finetune.py to do the DPOTrain.
My training datasets are like this:

{"question": "1+2=", "response_chosen": "4", "response_rejected": "3"}
{"question": "1+2=", "response_chosen": "4", "response_rejected": "3"}
{"question": "1+2=", "response_chosen": "4", "response_rejected": "3"}
{"question": "1+2=", "response_chosen": "4", "response_rejected": "3"}
{"question": "1+2=", "response_chosen": "4", "response_rejected": "3"}
{"question": "1+2=", "response_chosen": "4", "response_rejected": "3"}
{"question": "1+2=", "response_chosen": "4", "response_rejected": "3"}
{"question": "1+2=", "response_chosen": "4", "response_rejected": "3"}
{"question": "1+2=", "response_chosen": "4", "response_rejected": "3"}
{"question": "1+2=", "response_chosen": "4", "response_rejected": "3"}
{"question": "1+2=", "response_chosen": "4", "response_rejected": "3"}
{"question": "1+2=", "response_chosen": "4", "response_rejected": "3"}
{"question": "1+2=", "response_chosen": "4", "response_rejected": "3"}
{"question": "1+2=", "response_chosen": "4", "response_rejected": "3"}
{"question": "1+2=", "response_chosen": "4", "response_rejected": "3"}
{"question": "1+2=", "response_chosen": "4", "response_rejected": "3"}

If I direclty run the DPO code will meet the problem:

File "/data/ketadb/condaenv/envs/qwen/lib/python3.9/site-packages/torch/utils/data/dataloader.py", line 678, in _next_data
data = self._dataset_fetcher.fetch(index)  # may raise StopIteration
File "/data/ketadb/condaenv/envs/qwen/lib/python3.9/site-packages/torch/utils/data/_utils/fetch.py", line 54, in fetch
return self.collate_fn(data)
File "/data/ketadb/condaenv/envs/qwen/lib/python3.9/site-packages/trl/trainer/utils.py", line 490, in call
return self.collate(tokenized_batch)
File "/data/ketadb/condaenv/envs/qwen/lib/python3.9/site-packages/trl/trainer/utils.py", line 449, in collate
to_pad = [torch.LongTensor(ex[k]) for ex in batch]
File "/data/ketadb/condaenv/envs/qwen/lib/python3.9/site-packages/trl/trainer/utils.py", line 449, in
to_pad = [torch.LongTensor(ex[k]) for ex in batch]
TypeError: an integer is required (got type NoneType)

If I debug the code in line 483:
for feature in features:
    prompt = feature["prompt"]
    chosen = feature["chosen"]
    rejected = feature["rejected"]

483    batch_element = self.tokenize_batch_element(prompt, chosen, rejected)
    print(batch_element)
    tokenized_batch.append(batch_element)

If I print the batch_element out, there will be another extra None at the end of the array:

batch_element:{'chosen_input_ids': [16, 10, 17, 28, 19, None], 'chosen_attention_mask': [1, 1, 1, 1, 1, 1], 'chosen_labels': [-100, -100, -100, -100, 19, None], 'rejected_input_ids': [16, 10, 17, 28, 18, None], 'rejected_attention_mask': [1, 1, 1, 1, 1, 1], 'rejected_labels': [-100, -100, -100, -100, 18, None], 'prompt_input_ids': [16, 10, 17, 28], 'prompt_attention_mask': [1, 1, 1, 1], 'prompt': '1+2=', 'chosen': '1+2=4', 'rejected': '1+2=3', 'chosen_response_only': '4', 'rejected_response_only': '3'}

My chosen_input_ids 1+2=4 length should be 5, but after self.tokenize_batch_element the 'chosen_input_ids': [16, 10, 17, 28, 19, None] length is 6, and there is another extra None lead the TypeError: an integer is required (got type NoneType) problem.
So, I changed the line 456  to_pad = [torch.LongTensor(ex[k]) for ex in batch] to  456 to_pad = [torch.LongTensor(ex[k][:-1]) for ex in batch] and It worked

{'loss': 0.2599, 'learning_rate': 0.0003, 'rewards/chosen': 0.0, 'rewards/rejected': 0.0, 'rewards/accuracies': 0.0, 'rewards/margins': 0.0, 'logps/rejected': -0.21053116023540497, 'logps/chosen': -4.585531234741211, 'logits/rejected': -2.686852216720581, 'logits/chosen': -2.6731910705566406, 'epoch': 1.0}
{'loss': 0.2599, 'learning_rate': 0.00015, 'rewards/chosen': 0.0, 'rewards/rejected': 0.0, 'rewards/accuracies': 0.0, 'rewards/margins': 0.0, 'logps/rejected': -0.21053116023540497, 'logps/chosen': -4.585531234741211, 'logits/rejected': -2.686852216720581, 'logits/chosen': -2.6731910705566406, 'epoch': 2.0}
{'loss': 0.1227, 'learning_rate': 0.0, 'rewards/chosen': 0.27905863523483276, 'rewards/rejected': -0.17719139158725739, 'rewards/accuracies': 1.0, 'rewards/margins': 0.45625001192092896, 'logps/rejected': -1.9824450016021729, 'logps/chosen': -1.7949450016021729, 'logits/rejected': -2.546565055847168, 'logits/chosen': -2.5510566234588623, 'epoch': 2.67}
{'train_runtime': 2.826, 'train_samples_per_second': 3.185, 'train_steps_per_second': 1.062, 'train_loss': 0.2141884664694468, 'epoch': 2.67}
100%|███████████████████████████████████████████████████████████████████████████████████████████████| 3/3 [00:02<00:00,  1.06it/s]
***** train metrics *****
epoch                    =       2.67
train_loss               =     0.2142
train_runtime            = 0:00:02.82
train_samples            =          3
train_samples_per_second =      3.185
train_steps_per_second   =      1.062
Training metrics: {'train_runtime': 2.826, 'train_samples_per_second': 3.185, 'train_steps_per_second': 1.062, 'train_loss': 0.2141884664694468, 'epoch': 2.67, 'train_samples': 3}

I do not know whether am I right, or I did not use it the right way.
I think the problem may happened because Qwen has it own tokenizer.
My prompt dict :
return {
            "prompt": ["Question: " + question + "\n\nAnswer: " + for question in examples["question"]],
            "chosen": examples["response_chosen"],
            "rejected": examples["response_rejected"],
        }

DPOTrainer :
trainer = DPOTrainer(
        model,
        ref_model=deepcopy(model),
        args=training_args,
        beta=0.1,
        tokenizer=tokenizer,
        peft_config=lora_config,
        max_prompt_length=training_args.model_max_length,
        max_length=training_args.model_max_length,
        train_dataset=data_module['train_dataset'],
        eval_dataset=data_module['eval_dataset'],
    )

tokenizer :
    tokenizer = transformers.AutoTokenizer.from_pretrained(
        model_args.model_name_or_path,
        cache_dir=training_args.cache_dir,
        model_max_length=training_args.model_max_length,
        padding_side="right",
        use_fast=False,
        trust_remote_code=True,
    )
    tokenizer.pad_token_id = tokenizer.eod_id
    if tokenizer.pad_token_id is None:
        tokenizer.pad_token_id = 0  # set as the <unk> token

@xzqxnet0990  I believe we have fixed the tokenization in the PR #885 if you want to give that branch a try?

@kashif Thanks，I will try.

The main reason for the error is that “tokenize_row”  in dpo_trainer will add bos and eos to prompt and answer. If these two toekn are not configured for the qwen model in advance, the default is None, and errors will occur later.

The main reason for the error is that “tokenize_row” in dpo_trainer will add bos and eos to prompt and answer. If these two toekn are not configured for the Qwen-chat model in advance, the default is None, and errors will occur later.

Hi, @wen2cheng
Your suggestion is correct. I fix this issue by adding:
tokenizer.add_special_tokens({"bos_token": tokenizer.eos_token})
tokenizer.bos_token_id = tokenizer.eos_token_id

When the model belongs to Qwen series.
Not sure about the final results since I am still training. However, the issue do fixed.
Junru

Thanks, just had the same issue.
Since the chosen completions (and rejected) are tokenized on their own, is there a risk that a bos_token is being added there? (which wouldn't happen if tokenizing a complete prompt+completion?


Use annotation package name for static check

The static check of marketplace bundles previously relied on a annotation package name. When the check was converted to a stati test suite a package name from csv was used instead.
This commit fixes a check and use a package name from annotations again.
JIRA: ISV-4524

/lgtm


fix env setup

This PR fixes broken conda environment setup. This bug caused all dependabot builds to fail (e.g. https://readthedocs.org/projects/symbiflow/builds/14358207/). The problem was previously discussed and solved in: https://github.com/SymbiFlow/symbiflow-examples/issues/169 but it seems that it is a common issue for multiple repositories.
Signed-off-by: Paweł Czarnecki<EMAIL_ADDRESS>

LGTM, merging.


members: update user's infromation

When a user is updated, the community members that reference that user needs to be re-index so it shows the latest information.

Status update:
Local tests work at invenio-communities level see PR #535.  To do list:

Fix invenio-users-resources tests
Fix change notification handler registration in invenio-communities
Test from UI (e.g. change a user profile name, and see it changes in the members)
Fix indexer task to receive/use a queue per record_cls

Having an issue with the datastore commit signals.
The userprofile change happens within a begin_nested (see here). Then, outside the context the datastore.commit(). The problem is that the user will not be in the session.dirty objects since the begin_nested context manager performs a flush on exit. Therefore, the signals will not send the user to reindex nor notify change.
I've tried alternatives:

Commit the datastore within the context manager, then the session outside. This is not feasible, it will leave the session in a commited/dirty state, then any following request (e.g. reindexing when checking permissions) will fail to run SQL.
Avoid using the context manager, it is not possible since we need to open a new transaction or we would be messing with any subsequent operation.

I need to take a step back and learn more about DB session handling.

The issue above has been solved, using a custom method mark_changed to keep track of inside ctx manager changes.
The local E2E test of creating a community, changing the user name and checking it changed in the community members is working. However, the process_bulk_queue is triggered manually by calling it on the indexer through a pyshell. I've noticed that after processing the queue any new update to the user is blocked (on DB) until the pyshell executes db.session.commit() which leads me to think that we are keeping a reference somewhere in the reindex method. In a real life scenario it should not be an issue since the task would run on a celery worker and close the db connection on exit... Nonetheless, there could be concurrency issues?


fix lint errors, add dummy unit test structure

Signed-off-by: Evan Flynn<EMAIL_ADDRESS>

This change is

@landerU you might find this interesting too since you added the CI to the main repo...for now there is just a dummy test in there until we add more.
what im thinking is we run the exe thats passed into the CMake function for some time limit and we run some tests that check number of nodes, publishers, subscribers, message size, etc. to make sure it matches whats defined in the requirements for the reference system

@flynneva, why don't we use ros_lauch for that?


#M02. Admin Triggers Distribution

Once an admin triggers the event to distribute teams/judges:

Number of teams and participants per team should depend on total number of participants.
This should assign a random Judge to each team as well.
The aim is to have between 4-7 participants per team ideally.

SEE WIKI FOR MORE INFO
https://github.com/Code-Institute-Community/ci-hackathon-app/wiki/User-Stories#miscellaneous-user-stories

To ensure a balanced range of skill levels in each team, the distribution algorithm could assign participants to teams based on their experience (LMS Module) ie. divvy out all Alumni first (if any), then MS4, MS3, MS2, MS1.

Thanks @tdunn891! Yes exactly! 💯  Smart call, that's precisely what we do for regular Hackathons, somehow I failed to add that on this user story! Will get that added now! 👍

@tdunn891
Also, with this one... were you working on this Issue (still "Open"), or were you simply adding a comment as such?


feat: Use the BlueprintJS table component (spreadsheet like experience) for query results

Loom video
https://www.loom.com/share/5812043826f2454f89d6d0ce8b6ec8b7
Closes https://github.com/mergestat/mergestat/issues/753

Loom video
https://www.loom.com/share/09fa07237e4e4c2997b6f237dc9fb7d3


Sid. modes in pyswisseph

Sidereal modes 'True Mula' and 'True Pushya' are included in pyswisseph now, which could maybe simplify code in file jyotisha/panchaanga/temporal/names/__init__.py

Fixed, thanks!


MMFF94 energies of the validation suite

Some MMFF94 energies produced by the RDKIT don't match those from the MMFF94 validation suite energies.
Ubuntu 16.04 Xenial
RDKit version: 2017.09.3 (via anaconda)
conda version : 4.4.8
conda-build version : 3.4.0
python version : 2.7.14.final.0

So I was looking at the energies produced by RDKit for the molecules contained in the MMFF94 validation suite as a sanity check. I used the code below where I feed a SDF file containing a single molecule at a time. Now for many molecules the energy produced matches almost perfectly the ones supplied in the validation suite, but some of them are quite far off like:

AMHTAR01:  RDKIT_ENERGY is 43.1645421554 kcal/mol (instead of 66.18011)
BAOXLM01:  RDKIT_ENERGY is 49.0690926988 kcal/mol (instead of 278.92570)
BBSPRT10: RDKIT_ENERGY is 47.6343721111 kcal/mol (instead of -13.55387)
CAGREH10: RDKIT_ENERGY is -24.7421671305 kcal/mol (instead of -259.51340)
CAMALD03: RDKIT_ENERGY is -25.5893313194 kcal/mol (instead of 36.83909)

I included the full list in annex.
mmff94_calibration.txt
MMFF94.energies.txt
MMFF94_dative.txt
So I was wondering, is that a bug in my distribution or a mistake from my part or just reflects that not all data point can be well fitted within the flexibility of the model?
Many thanks for developing the RDKit, this is a very useful tool and I use it a lot!
Best Regards

N_STEPS = 10000
tol = 1e-12
def get_rdkit_energy(sdfile):
	def converge(yo):
		ff = AllChem.MMFFGetMoleculeForceField(yo, pyMMFFMolProperties=AllChem.MMFFGetMoleculeProperties(yo), ignoreInterfragInteractions=False, nonBondedThresh=100.0)
		converged = ff.Minimize(N_STEPS, tol, tol)
		return converged, ff.CalcEnergy()

	mol = Chem.SDMolSupplier(sdfile, removeHs=False)[0]
	flags = converge(mol)
	return flags

Actually I was using the structures where the formal charges weren't set properly. When using the SDF file provided in the article:

Tosco, Paolo, Nikolaus Stiefl, and Gregory Landrum. "Bringing the MMFF force field to the RDKit: implementation and validation." Journal of cheminformatics 6.1 (2014): 37.

the energies match those from the reference file. My bad!


Fix/remove old context

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


Prevent the file watching loop from still being alive after we kill it

Under certain conditions, execution of blocks of code inside the file watching loop continues although the loop's condition is false, causing unwanted actions to be performed (e.g. using the closed directory handle)
This PR aims to:

Prevent a crash from occurring upon changing the working directory quickly
Remove unneeded checks before adding/removing files during the watcher operation
Convert any direct uses of the FilesAndFolders property to use a "snapshot" list instead

Please test this PR by doing the following:

Removing/Adding/Renaming a file after refreshing the directory multiple times
RELEASE MODE: Changing the WorkingDirectory by rapidly invoking various sidebar locations

@duke7553 Is this supposed to fix the issue where the app sometimes crashes when opening a folder?

Bug with duplicated items still exists.
Sometimes getting an exception after item rename
2020-06-11 16:34:23.6512|ERROR|Files.App|Sequence contains no matching element|System.InvalidOperationException: Sequence contains no matching element
   at System.Linq.Enumerable.First[TSource](IEnumerable`1 source, Func`2 predicate)
   at Files.Filesystem.ItemViewModel.<>c__DisplayClass72_0.<WatchForDirectoryChanges>b__0(IAsyncAction x)
--- End of stack trace from previous location where exception was thrown ---
   at Files.Filesystem.ItemViewModel.WatchForDirectoryChanges(String path)
   at System.Threading.WinRTSynchronizationContextBase.Invoker.InvokeCore()
   at Windows.ApplicationModel.Core.UnhandledError.Propagate()
   at Microsoft.AppCenter.Utils.ApplicationLifecycleHelper.<.ctor>b__17_1(Object sender, UnhandledErrorDetectedEventArgs eventArgs)
--- End of stack trace from previous location where exception was thrown ---
   at Microsoft.AppCenter.Utils.ApplicationLifecycleHelper.<.ctor>b__17_1(Object sender, UnhandledErrorDetectedEventArgs eventArgs)

I'm not experiencing the duplicated elements bug anymore, but:

creating a new file
deleting it
hitting refresh
results in the same exception mentioned by @tsvietOK

@gave92 Do you think a cancellation token would be appropriate to fix this one?

@duke7553 The bug I mentioned above appears to be solved 👍

:shipit:


[Target][Lowering] Update Op Intrinsic Lowering Mechanism And Intrinsic Lowering Pass

This PR updated the intrinsic lowering pass to support the new op registry and avoid overloading the global tvm registry. Meanwhile, it kept the fallback mechanism to find the most suitable lower intrinsic function, e.g., llvm.FLowerIntrinsic vs. default.FLowerIntrinsic. All previous op registration are ported to new functions, and some missing ops would be added in separate PR.

Python side op lowering function has been changed from register_func to register_op.

Thanks for the contribution! A few high-level comments:

Do not add any C API - we can use packed func to expose the interface. Op is a compile-time concept so please move the registration logic out of the runtime :-)
Please change register_op to some name like register_intrin_lowering, and consider accept a target kind like default/llvm/cuda

Thanks for the advice! I have changed the name of function in python-side to register_op_intrin_lowering and removed change to C API. Instead, I utilized the global function registration and packedfunc to register op intrinsic lowering function.

@junrushao1994 Please review, thanks!

note, please do not checkin vta-hw submodule change(do a git submodule update)

note, please do not checkin vta-hw submodule change(do a git submodule update)

Noted with thanks!

CC: @comaniac @yzhliu @icemelon9

@tqchen would you like to take a second look? If there is other comments, I will get it merged the other day

Thanks @zxybazh @tqchen!


Fixed most hairlines for iPhone6 plus and other retina devices

Before and after screenshots below.
Hairlines are more consistent throughout and also now visible in the browser (chrome when you set iPhone6 plus emulation).
TODO: Figure out how to improve hairlines for input boxes (messagebar and prompt) without impacting style or performance.

@nolimits4web Just wondering if you had looked at this yet?
I think its good work, but maybe you can improve it.
Also, side note, I added (again in my testing branch) an infinite scroll for the top of the page. This is mostly for the messaging so you can load more (older) messages.
The only issue is you cannot set the event listener until after the messages are loaded, if not it loads more on top and scroll from the bottom of the messages.

Confirming this is an issue on the iPhone 6+. 0.5px gets scaled down to 0 as 1/3 is less than ½.

Yes, i had looked. And as @johnjamesjacoby mentioned, i am trying to look for a better approach. The biggest problem of such approach is the later custom customisation of such borders. I am sure that there will be a lot of troubles and questions from dev.
I haven't real iPhone 6 Plus device, so i ask you to check it on real device. From what i have tried in simulator:

setting CSS border-width to 0.7px gives "stable" (not buggy) 2 physical-pixels border width (instead of 3px like with 1px border). This is already not bad
setting CSS border-width to 0.67px gives 1 physical-pixels border width. But for some reason, it becomes buggy and such borders disappear during scrolling. And i don't know, maybe it is just a simulator bug. So if it is fine on real iPhone 6 Plus then it could be a "holy grail" solution :)

@MarcGodard i saw your infinite scroll on top. I think this is could be useful, so i will add that code to F7

Here's how I've circumvented it for now: https://github.com/nolimits4web/Framework7/commit/8ed43b470f79fddb02eeafbaf52adb071fafd2a1
Then in all the LESS files, instead of targeting .retina target .pixel-ratio-2. In my experience, a 1px border on the iPhone 6+ isn't horrible, thanks to the immense amount of pixels and the overall scaling that takes place to fit everything.

I agree that my approach adds complications. As far as left and right borders, I agree, this is still a problem.
The benefits of this approach is that it will be visible on all devices and browsers as scaling down cannot go to 0 and will go to the smallest visible line.
@johnjamesjacoby I prefer the lines to be one visible pixel (3px in css). The thicker lines don't look as nice on the iphone 6 plus.

Yes. It doesn't appear to work for me. Anything up to 0.9px still had the same rounding-down effect of being invisible.

@nolimits4web When I look at the changed files for this pull request, all the correct changes are there. There shouldn't be anything about the lazy load.
These changes were done on my master branch as I didn't quite know what i was doing when i started contributing.
I played around with 0.33 scaling and it also works well, tested on iPhone 6plus, and retina iPad mini.

Just merged #350 based on @MarcGodard pseudo selectors solution


State

This is a new version which contains:

Cleaner state management.
All static and using Now v2.
Responsive design.
More date formats.
Copy button for each date.
Support for timezones.

Closes https://github.com/dinkbit/epoch/issues/3 and https://github.com/dinkbit/epoch/issues/1


[feature] CLI completion

Support for CLI completion on the command line for environments and applications, e.g.
myks render  yields all configured environments
myks render my-env  yields all applications of that particular environment
bash and zsh should be supported

Closed by #240


add initial content to website sections

I wrote some sections to fill the current website. Please add, delete, change...

Addresses #5 and #6


Track which network zone a server is in

Blocked by #62

The simplest way I've found to find a vm:
vim = RbVmomi::VIM.connect(opts)
dc = vim.serviceInstance.find_datacenter() # gets SUL datacenter
dc.find_vm('path/to/vm')

but note that you need to know the path -- hence multiple examples online of recursing through a folder

This might be another approach, based on some looking around Erin and I did this morning:
vim = RbVmomi::VIM.connect opts
rootFolder = vim.serviceInstance.content.rootFolder
dc = rootFolder.childEntity.grep(RbVmomi::VIM::Datacenter).find { |x| x.name == opts[:datacenter] } or fail "datacenter not found"
vm = dc.vmFolder.childEntity.grep(RbVmomi::VIM::VirtualMachine).find { |x| x.name == vm_name } or fail "VM not found"

Current idea is to add hiera lookups to common representing our nets, since we have some there anyway, then build a custom fact that determines the net from the eth0 (or whatever) ipaddress. The vmware api seems less of a 'read' api, and more of a tool to do things to vms with.

Sorry this wasn't working well.  All the vmware things I've done before myself for reporting were just 'get every VM', so I'm not good at finding specific VMs except through getting them all and iterating.


performance issues?

I just built https://github.com/loveencounterflow/basic-stream-benchmarks to get a handle on how heavy stream transforms are. The code is quite simple:
input         = FS.createReadStream   input_path
output        = FS.createWriteStream  output_path
#.........................................................................................................
p = input
p = p.pipe $split()
#.........................................................................................................
p = p.pipe through2.obj ( data, encoding, callback ) ->
  unless S.t0?
    start_profile S
    S.t0 ?= Date.now()
  S.byte_count += data.length
  S.item_count += +1
  @push data
  callback()
#.........................................................................................................
for _ in [ 1 .. n ] by +1
  if mode is 'sync'
    p = p.pipe through2.obj ( data, encoding, callback ) -> @push data; callback()
  else
    p = p.pipe through2.obj ( data, encoding, callback ) -> setImmediate => @push data; callback()
#.........................................................................................................
p = p.pipe through2 ( data, encoding, callback ) -> @push data + '\n'; callback()
p = p.pipe output

Now I may be totally wrong here and maybe there's some blunder I committed (in a way I hope so), but if
the numbers are correct then adding single stream transform with through2 may cost you like almost a second (depending on hardware, amount of data processed and so on).
Now I'm aware that you normally won't build pipelines with hundreds of stream transforms as I do for the test, but shouldn't a no-op transform weigh in more like a no-op function call, i.e. orders of magnitude below a whole second?
The flamegraphs I produced (and that you can produce after cloning and e.g. running in Chrome DevTools)
would indicate that indeed most of the time is spent in through2 and readable-stream code (have a look at them over at https://github.com/loveencounterflow/basic-stream-benchmarks#flamegraphs).
Now I'm aware that the tests I ran process a file with ~50k lines, so each transform is called that often. Still it means that just in order to work through a run-of-the-mill 1.4MB text file I have to be prepared that adding just 10 steps to the processing will make the time required go from 4 seconds to 10 seconds just for the overhead of it. Compare that to how fast I can, say, grep the same file with a regex (time says 0m0.006s).
Any ideas how to improve the performance story?

transpiled your coffeescript to vanilla javascript, because that's what most speak here, and what this project is written in:
var input = FS.createReadStream(input_path);
var output = FS.createWriteStream(output_path);

var p = input;
p = p.pipe($split());

p = p.pipe(through2.obj(function(data, encoding, callback) {
  if (S.t0 == null) {
    start_profile(S);
    if (S.t0 == null) {
      S.t0 = Date.now();
    }
  }
  S.byte_count += data.length;
  S.item_count += +1;
  this.push(data);
  return callback();
}));

var _, i, ref;
for (_ = i = 1, ref = n; i <= ref; _ = i += +1) {
  if (mode === 'sync') {
    p = p.pipe(through2.obj(function(data, encoding, callback) {
      this.push(data);
      return callback();
    }));
  } else {
    p = p.pipe(through2.obj(function(data, encoding, callback) {
      return setImmediate((function(_this) {
        return function() {
          _this.push(data);
          return callback();
        };
      })(this));
    }));
  }
}

p = p.pipe(through2(function(data, encoding, callback) {
  this.push(data + '\n');
  return callback();
}));

p = p.pipe(output);

Can you provide us with a code example that doesn't use any external variables, like S, so others can run it too?
The setImmediate looks suspicious, why not push the data in the same tick?
And exactly which invocation of through2 in your example are you referring to?

thanks for going through the trouble to convert my sample code to JS. Yes, it could've been more minimal, so here you go:
var FS            = require 'fs';
var through2      = require 'through2';
var $split        = require 'binary-split';
var input_path    = 'test-data/Unicode-NamesList.txt';
var output_path   = '/dev/null';
var input         = FS.createReadStream(input_path);
var output        = FS.createWriteStream(output_path);
var n             = 50;
var $split
var p = input;
p = p.pipe($split());

var _, i, ref;
for (_ = i = 1, ref = n; i <= ref; _ = i += +1) {
    /* 1 */
    p = p.pipe(through2.obj(function(data, encoding, callback) {
      this.push(data);
      return callback();
    }));
  };
p = p.pipe(through2(function(data, encoding, callback) {
  this.push(data + '\n');
  return callback();
}));

p = p.pipe(output);

The setImmediate clause (here removed, would've been below line marked 1) is just there to enable an asynchronous mode of operation. As can readily be gleaned from the flamegraphs I posted, using asynchronous in place of synchronous stream transforms has an absolutely huge effect on call stacks (unsurprisingly), but, as the figures show, but a small effect on execution time (indeed, setImmediate would appear to introduce a little overhead itself). Full code at https://github.com/loveencounterflow/basic-stream-benchmarks/blob/master/lib/main.js#L165
"And exactly which invocation of through2 in your example are you referring to?"—not sure I understand your question. Thing is, the above processing pipeline consists of:

opening a read stream (for a file of 1,413,484 bytes of ASCII-only text);
splitting the byte chunks into (48,770) lines of text;
piping each line through n bare-bones, do-nothing transforms built with through2;
adding a newline character to each line; finally,
writing lines to /dev/null.

Now, step 3 is the critical one here. Again, as the figures show, I can go through (no pun intended) the entire 1.4MB file in like under 2 seconds on my notebook (well over 0.5MB per second), but with a mere 20 or 40 no-op transforms (not a lot I'd say), that time quickly goes up to 10 or 20 seconds.
To rephrase this finding in a more user-oriented way: When processing text files of non-trivial, but still moderate size with a stream pipeline, be aware that as the length of your pipeline grows beyond a certain small limit (of less than 10 transforms), your overall execution time will be dominated by the number of stream transforms.
I'm doing little performance testing, so there may be some methodological blunders I'm not aware of; also, I'm lacking comparable data, so this is one reason to bring my findings here. Frankly, to me these results look a tad daunting, given that the streaming philosophy tells me to process my data in lots of little, focused steps. I have fully assimilated that approach, I find it great and software quality has greatly benefitted from it, but I wasn't prepared to this kind of overhead / loss of performance per stream transform added. Well, go ahead and, e.g., demonstrate that another approach that would likewise involve making (40 times 50,000 =) 2,000,000 no-op calls will suffer from a similar impact.
Coming to think of it, I'm going to try and write a non-streaming 'counter-test' to put those figures into perspective. But I believe I can say right away that making two million ordinary function calls is not going to take anywhere near 20 seconds on 'modern average' hardware (anyone have a single number to characterize the performance class of a given machine?).

Maybe the overhead of streams themselves is just too much (as you found), especially as each stream step maintains its own buffers.
Transform streams is not the only way of splitting up a complex byte processing operation into small pieces. You can use something like transducers to combine a bunch of small tiny synchronous steps into one stream pipeline element. What streams and pipelines are good at are handling buffering/back-pressure, especially when one or more step in the pipeline is asynchronous and takes an unpredictable amount of time to complete.


Add support for testnet2

Describe your changes
Add support for testnet2
Linked issues
Closes https://github.com/software-mansion/starknet-jvm/issues/204
Breaking changes

[ ] This issue contains breaking changes

Codecov Report
Base: 70.53% // Head: 69.56% // Decreases project coverage by -0.97% :warning:

Coverage data is based on head (03abb38) compared to base (6e65c4c).
Patch coverage: 4.34% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #205      +/-   ##
==========================================
- Coverage   70.53%   69.56%   -0.98%
==========================================
  Files          53       53
  Lines        1541     1564      +23
  Branches      109      111       +2
==========================================
+ Hits         1087     1088       +1
- Misses        424      446      +22
  Partials       30       30

Impacted Files
Coverage Δ

...nsion/starknet/provider/gateway/GatewayProvider.kt
81.92% <0.00%> (-7.95%)
:arrow_down:

...m/swmansion/starknet/data/types/StarknetChainId.kt
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Minimizing memory and CPU consumption with Face API

I am using Face API to detect faces on a webcam. I call the detectSingleFace() function every 500ms. Everything works fine, but I can see that RAM and CPU usage is increasing. I realize that this is normal, finally face detection needs some resources, but I am asking if there is a way to optimize this. I am using the tiny models version. Maybe the ram consumption is related to storing the model in the browser memory, because it weighs a bit (if so, will cdn help something)?

assuming you have no memory leaks in your code (check for tf.engine().memory.numTensors if it's growing after each frame), it's really not up to face-api, its up to browser to perform garbage collection
and modern browsers are lazy doing so as they value responsiveness more than anything
if youre using webgl backend, you can force gpu memory deallocation (which does slow things down) with tf.ENV.set('WEBGL_DELETE_TEXTURE_THRESHOLD', 0)
another thing is that this face-api is using very old tfjs v1.x while latest is v3.9.0 - but if memory is your biggest concern, newer tfjs definitely doesn't use less memory

How do you think that  I solve this ??? did you can to solve the problem about memory?

If you have a specific problem, describe it in a new issue. This is not a "problem" on its own, it depends how you use face-api and what is TFJS backend and then maybe it can be tuned for your case.
Posting on a closed issue without any details doesn't help.

also, I face the same problem when I use face-api to track user emotions it take all my CPU on my server so how can custom CPU usage because it leads to slow site and sometimes stops other services that runs on same server


How to get RemoteID working?

So I tried a few things, but no luck.
By RemoteID, I mean, Universal Dismiss. I didn't find any DOC also, just the MSDN article talking about it.
Any help would be appreciated.
Where I want to implement it:
foreach(Tuple<string, string, string> newNote in newNotifications)
{
// Make the visual
ToastVisual visual = new ToastVisual();
visual.TitleText = new ToastText() { Text = newNote.Item1 };
if(!String.IsNullOrWhiteSpace(newNote.Item2))
{
visual.BodyTextLine1 = new ToastText() { Text = newNote.Item2 };
}
                // Make the toast content
                ToastContent toastContent = new ToastContent();
                toastContent.Visual = visual;
                toastContent.Launch = c_messageInboxOpenArgument;
                toastContent.ActivationType = ToastActivationType.Foreground;
                toastContent.Duration = ToastDuration.Short;

                var toast = new ToastNotification(toastContent.GetXml());
                toast.Tag = newNote.Item3;

                // Only show if we should and this is the first message to show.
                toast.SuppressPopup = hasShownNote || updateSliently || AddToNotificationCenterSilently;
                ToastNotificationManager.CreateToastNotifier().Show(toast);

                // Mark that we have shown one.
                hasShownNote = true;
            }

full code to better understanding: https://github.com/vitorgrs/Reddunt/blob/rd_develop_rs1/BaconBackend/Managers/Background/BackgroundMessageUpdater.cs
PS: Not getting markdown insert code working. Sorry.

Hey! You're correct that the latest documentation is the MSDN blog article: https://blogs.msdn.microsoft.com/tiles_and_toasts/2016/06/10/action-center-in-the-cloud-universal-dismiss/
So you added the RemoteId, installed the app on both your phone and PC, and popped the notification with the same RemoteId? And you're logged into Cortana with your MSA on both of your devices? And when you dismissed one, the other didn't dismiss?
What build are you on?

I'm on 14393 with 14388 SDK. And the problem here is that it doesn't recognize account and homework. "The name 'account' does not exist in the current context".
Also tried following this methodbut not look too. :/

Oh, "account" and "homework" are local variables to my own app... you'll have to generate your own RemoteId string using variables that make sense to your app.
What are you sending a notification for? If you tell me what you're sending a notification about, I could help suggest what you should use to generate your RemoteId!

I'm sending a pull toast (timer,  check every 15 min). It's Reddunt app, Baconit fork :)
https://github.com/vitorgrs/Reddunt/blob/rd_develop_rs1/BaconBackend/Managers/Background/BackgroundMessageUpdater.cs

Ah I saw Reddunt on the WindowsPhone subreddit :P
Sorry I forgot to reply, if you haven't figured out what to do for the RemoteId, I just took a look at that code and here's my suggestion...
Is "message.Id" something that is consistent across devices? For example, is the "Id" property something that the Reddit server provides? And therefore the message ID for a message will be the same on two different devices?
If so, you should use that for the RemoteId (you already use it for Tag, you can use it for RemoteId too).
So at line #213, I would do
toast.RemoteId = "Message_" + newNote.Item3;

And that should work, assuming that the message ID is something that's shared across devices and therefore can identify the same message across devices.


Levana eth airdrop contract [WIP]

Rebranching bc other branch diverged from main

Codecov Report
Base: 69.44% // Head: 70.21% // Increases project coverage by +0.77% :tada:

Coverage data is based on head (24eb378) compared to base (3f06b70).
Patch coverage: 82.41% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #447      +/-   ##
==========================================
+ Coverage   69.44%   70.21%   +0.77%
==========================================
  Files          27       30       +3
  Lines        1430     1521      +91
==========================================
+ Hits          993     1068      +75
- Misses        437      453      +16

Impacted Files
Coverage Δ

contracts/sg-eth-airdrop/src/signature_verify.rs
66.66% <66.66%> (ø)

contracts/sg-eth-airdrop/src/ethereum.rs
68.18% <68.18%> (ø)

contracts/sg-eth-airdrop/src/contract.rs
97.77% <97.77%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


get_my_trades

Describe the bug
Calling get_my_trades with default limits not always produces the same results.
In my example, sometime it returns 114 items while others 116.
Additionally, putting get_my_trades in a loop starting with "fromId=0" and limit to 50 (or any other number required to have at least two loops) I am having an additional issue.
Indeed, it happens that sometime I get 50, 50, 14 and 2 items, other times 50, 50, 14 and 0 items.
This means that I can't rely on quitting from the loop when the number of items retrieved are less than the limits, moreover, even in this situation I am getting sometime 114 items, others 116.
This is leading to a severe issue for me as I need to rebuild the orders/trades history per each ticker I am managing.
To Reproduce
def get_my_trades(self, symbol):
my_trades = []
items_to_collect = 500
    try:
        my_trades = self.connect.get_my_trades(symbol=symbol, limit=items_to_collect, fromId=0)
        trades = my_trades
    except (BinanceAPIException, BinanceRequestException) as e:
        trades = ""
        self._logger.warning(f'Warning raised by Binance Exceptions: {e}')

    # Sometime it happens that the items retrieved are less than "items_to_collect" although there are still others to pop
    # Sometime it happens that not all the items are retrieved ... bug?
    while len(my_trades) > 0:
        try:
            my_trades = self.connect.get_my_trades(symbol=symbol, limit=items_to_collect, fromId=my_trades[-1]['id']+1)
            trades += my_trades
        except (BinanceAPIException, BinanceRequestException) as e:
            trades = ""
            self._logger.warning(f'Warning raised by Binance Exceptions: {e}')

    return trades

Expected behavior
In the first scenario I shoud expect always 116 items
In the second scenario I should expect always 50, 50 and 16 items
Environment (please complete the following information):

Python version: [e.g. 3.5] 3.8
Virtual Env: [e.g. virtualenv, conda] conda
OS: [e.g. Mac, Ubuntu] Wiindows 10 Enterprise
python-binance version 0.7.9

Logs or Additional context
Add any other context about the problem here.

Looks like a Binance API issue, I get the same flaky behaviour even without using this library.
Similar issue here: https://github.com/sammchardy/python-binance/issues/973

FYI,
I have built a user data wrapper on top of this library. check it out on github.com/binance-reporting. There you can download all your user data from binance. Just need to provide a config file where you specify what info you want.
Maybe that helps.
Cheers,
Jan


Random crashes to black screen with mouse cursor

I realize similar issues have been opened recently, but I was not able to find a solution:
My Magic Mirror randomly stops showing all modules and defaults to a black screen showing just the mouse cursor.
I have installed MMM-Logging, however the logs only show problems related to what I suppose is an internet outage (which does happen occasionally).
Errors look such as the following
0|mm       | [2020-04-27 21:50:01.138] [ERROR]  2020-04-27T21:50:01 <error> Calendar Error. Could not fetch calendar:  http://XXX/Calendar/personal.ics Error: getaddrinfo EAI_AGAIN XXX
0|mm       |     at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:60:26) {
0|mm       |   errno: 'EAI_AGAIN',
0|mm       |   code: 'EAI_AGAIN',
0|mm       |   syscall: 'getaddrinfo',
0|mm       |   hostname: 'XXX'
0|mm       | } (/home/pi/MagicMirror/modules/default/calendar/node_helper.js:63 )

My guess is that those cannot or at least should not be the cause for the problem.
I have not found any patterns in the crash, it happens at random times with no interaction via SSH or any other input modes.
Platform: Raspberry Pi 3
Node Version: Make sure it's version 8 or later.
MagicMirror Version: 2.11.0
Config file:
/* Magic Mirror Config Sample
 *
 * By Michael Teeuw http://michaelteeuw.nl
 * MIT Licensed.
 *
 * For more information on how you can configure this file
 * See https://github.com/MichMich/MagicMirror#configuration
 *
 */

var config = {
        address: "<IP_ADDRESS>", // Address to listen on, can be:
                              // - "localhost", "<IP_ADDRESS>", "::1" to listen on loopback interface
                              // - another specific IPv4/6 to listen on a specific interface
                              // - "<IP_ADDRESS>", "::" to listen on any interface
                              // Default, when address config is left out or empty, is "localhost"
        port: 8080,
        ipWhitelist: ["<IP_ADDRESS>", "::ffff:<IP_ADDRESS>", "::1", "<IP_ADDRESS>/24"], // Set [] to allow all IP addresses
                                                               // or add a specific IPv4 of <IP_ADDRESS> :
                                                               // ["<IP_ADDRESS>", "::ffff:<IP_ADDRESS>", "::1", "::ffff:<IP_ADDRESS>"],
                                                               // or IPv4 range of <IP_ADDRESS> --> <IP_ADDRESS> use CIDR format :
                                                               // ["<IP_ADDRESS>", "::ffff:<IP_ADDRESS>", "::1", "::ffff:<IP_ADDRESS>/28"],

        useHttps: false,                // Support HTTPS or not, default "false" will use HTTP
        httpsPrivateKey: "",    // HTTPS private key path, only require when useHttps is true
        httpsCertificate: "",   // HTTPS Certificate path, only require when useHttps is true

        language: "en",
        timeFormat: 24,
        units: "metric",
        // serverOnly:  true/false/"local" ,
                             // local for armv6l processors, default
                             //   starts serveronly and then starts chrome browser
                             // false, default for all  NON-armv6l devices
                             // true, force serveronly mode, because you want to.. no UI on this device

        modules: [
                {
                module: 'MMM-Logging',
                config: {
                // Module functions out of the box with nothing set here.
                // See below for configurable options.
                        }
                },
                {
                        module: 'MMM-ModuleScheduler',
                        config: {
                                global_schedule: {from: '0 7 * * *', to: '0 22 * * *' },
                                global_schedule: {from: '0 7 * * *', to: '0 10 * * *', groupClass: 'morning_schedule'},
                        }
                },
                {
                        module: "alert",
                },
                {
                        module: "updatenotification",
                        position: "top_bar"
                },
                {
                        module: "clock",
                        position: "top_left"
                },
                {
                        module: "calendar",
                        header: "Calendar",
                        position: "top_left",
                        config: {
                                timeFormat: "dateheaders",
                                dateFormat: "D.M. HH:mm",
                                maximumNumberOfDays:"2",
                                calendars: [
                                        {
                                                symbol: "calendar-check",
                                                url: "",
                                                auth: {
                                                        user: '',
                                                        pass: '',
                                                        method: 'basic',
                                                },
                                        },
                                        {
                                                symbol: "calendar-check",
                                                url: "",
                                                auth: {
                                                        user: '',
                                                        pass: '',
                                                        method: 'basic',
                                                },
                                        },

                                ],
                        },
                },

                {
                    module: "MMM-WienerLinien",
                    position: "top_left",
                    config: {
                        api_key: "",
                        stations: ["1266", "1259"],
                        updateInterval:"60000",
                        rotateInterval:"5000",
                        incidentLines:["35A","39A","U4","U6"],
                        shortenDestination:"12",
                        incidentShort:"true",
                    }
                },
                {
                        module: "compliments",
                        position: "lower_third",
                        classes: "morning_schedule",

                },
                {
                        module: 'MMM-GoogleMapsTraffic',
                        position: 'lower_third',
                        classes: "morning_schedule",
                        config: {
                                key: '',
                                lat: 48.2107566,
                                lng: 16.362128,
                                zoom:11,
                                height: '300px',
                                width: '100%',
                                styledMapType: "black",
                                disableDefaultUI: true,
                                backgroundColor: 'hsla(0, 0%, 0%, 0)',
                                markers: [
                                {
                                        lat: ,
                                        lng: ,
                                        fillColor: '#ff0000',
                                },
                                ],
                        },
                },
                {
                        module: "currentweather",
                        position: "top_right",
                        config: {
                                location: "",
                                locationID: "", //ID from http://bulk.openweathermap.org/sample/city.list.json.gz; unzip the gz file and find your city
                                appid: ""
                        }
                },
                {
                        module: "weatherforecast",
                        position: "top_right",
                        header: "Weather Forecast",
                        classes: "morning_schedule",
                        config: {
                                location: "",
                                locationID: "", //ID from http://bulk.openweathermap.org/sample/city.list.json.gz; unzip the gz file and find your city
                                appid: ""
                        }
                },
                {
                         module: 'MMM-bringList',
                         position: 'top_right',
                         header: 'Groceries',
                         config: {
                                listname: 'Home',
                                email: '',
                                password: '',
                                columns: 3,
                                maxrows: 5,
                                updateInterval: 120000,
                                verboseLogging: false
                        }
                },
                 {
                        module: 'MMM-MinecraftStatus',
                        position: "top_right",
                        config: {
                                banner: " ",           // Banner to display
                                hostname: "", // or IP address
                                port: "25565",
                                intervalSeconds: 60               // how often to ping the Minecraft server
                        }
                },
                {
                        module: "newsfeed",
                        position: "bottom_bar",
                        classes: "morning_schedule",
                        config: {
                                feeds: [
                                        {
                                                title: "derStandard.at",
                                                url: "https://www.derstandard.at/rss"
                                        },
                                ],
                                showSourceTitle: true,
                                showPublishDate: true,
                                broadcastNewsFeeds: true,
                                broadcastNewsUpdates: true,
                                },
                },
                {
                        module: 'MMM-Remote-Control',
        // uncomment the following line to show the URL of the remote control on the mirror
        // , position: 'bottom_left'
        // you can hide this module afterwards from the remote control itself
                        config: {
                                apiKey: ''
                        }
                },

        ]

};

/*************** DO NOT EDIT THE LINE BELOW ***************/
if (typeof module !== "undefined") {module.exports = config;}
(END)

when it goes to black screen again, open the developers console, f12 or ctrl-shift-i on the keyboard, and select the console tab.  there should be a fatal error  somewhere..

Closing due to inactivity. Feel free to reopen.