[Feature Request] Reduce test runtime by parallelizing tests

Feature request
Overview
SBT tests currently run sequentially. It would be good to reduce the total test runtime by parallelizing the SBT tests.
Motivation
SBT tests are taking longer and longer. This is not scalable. While we have already split various version of Scala tests into two CI builds in the repo, each build takes a long time. This is a burden for local testing as well. Any parallelization without causing test flakiness is going to be help here.
Further details
Running multiple concurrent SBT tests in the same JVM is likely going to cause major test flakiness. So we have to configure the parallel tests to run in separately JVMs. A quick google search led me to this resource which seems to explain the possibilities pretty well - https://www.yannmoisan.com/sbt-parallel-test.html
A while ago, Apache Spark did something like this as well - https://github.com/apache/spark/commit/9c238b8a462c483644237bd038edcf58f42a7db9
Willingness to contribute
The Delta Lake Community encourages new feature contributions. Would you or another member of your organization be willing to contribute an implementation of this feature?

[ ] Yes. I can contribute this feature independently.
[ ] Yes. I would be willing to contribute this feature with guidance from the Delta Lake community.
[x] No. I cannot contribute this feature at this time.

@tdas one possible solution is to split the test in N partitions and execute them sequentially on separate JVM, would it be ok? An higher speed can certainly be achieved by parallelizing within a VM, but that would probably require tagging tests that are safe to be executed in parallel vs tests that are flaky.
If it sounds reasonable for you I will implement a simple hash partitioning of the test by name into a configurable number of VM, which we might default to 2 or 4.  Does it make sense?


Link fixes to CSI blog post

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
For 1.11 Features: set Milestone to 1.11 and Base Branch to release-1.11
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
NOTE: After opening the PR, please un-check and re-check the "Allow edits from maintainers" box so that maintainers can work on your patch and speed up the review process. This is a temporary workaround to address a known issue with GitHub.>
Please delete this note before submitting the pull request.

@kbarnard10 I'm afraid there's a merge conflict that you'll need to resolve before you'll be able to merge this.

/lgtm

Afraid I can't approve this right on through since it's a blog update - assigning to the approver pool there
/assign @zacharysarah @bobsky @natekartchner @SarahKConway

/LGTM

/lgtm


Adding user facing documentation to README

Fixes # .
Did you remember to?

[ ] Add test case(s)
[ ] Update CHANGES.txt

ping @cbeust / @juherr

@cbeust - I fixed the typos.


Would like support for AndroidStudio gradle plugin

Which doesn't seem to play nice with the java plugin
FAILURE: Build failed with an exception.

What went wrong:
A problem occurred configuring project ':app'.

Failed to notify project evaluation listener.
The 'java' plugin has been applied, but it is not compatible with the Android plugins.
Could not get unknown property 'javaPreCompileDebug' for project ':app' of type org.gradle.api.Project.

As I don't do Android development myself, I'm afraid the help I'm able to provide may be limited.
If you use the com.commercehub.gradle.plugin.avro-base plugin instead of the com.commercehub.gradle.plugin.avro plugin, it shouldn't require the "java" plugin, and will let you define Avro Java class generation tasks yourself.  Documentation of that feature is available here: https://github.com/commercehub-oss/gradle-avro-plugin#alternate-usage
I'm potentially open to including Android support in this plugin, but would need assistance to do so.  An example project showing what type of integration is desired would help.


[Heap out of Memory] - Session Db not cleaned up and locked by process

Hi,
Love the project so far, just having this issue which is a bit of a problem when running on limited ressources. I am still investigating the issue and going trough the ulixee code base but for the sake of speed i am going to post the issue here also.
The .db file is not removed at the end of a session even with sessionPersistence set to false. The file is also locked up by the process which makes it impossible to remove. Eventually it leads to an heap out of memory which i confirmed using V8 profiler.
Tested it on Window 11 and Ubuntu 20.0.4 running inside Docker, same result.
Here is a minimal setup to reproduce.
1 - Copy both files listed below to a directory
2 - Cd into directory
3 - open a terminal and run "node client.js"
4 - open a terminal and run "node main.js"
As you can see, the heap usage keeps increasing and the .db files in /tmpDirectory/hero-sessions/ are not deleted at the of a session. Attempting to manually delete those files results in a EBUSY error
// client.js
import { CloudNode } from '@ulixee/cloud';
import { Session } from '@ulixee/hero-core';
import { memoryUsage } from 'process';

async function log(type, message) {
  const now = new Date().toUTCString();
  console.log(
    `\x1b[35m[${type}] \x1b[0m\x1b[33m[${now}]\x1b[0m -> ${message}`
  );
}

function getMemoryUsage() {
  const memory = memoryUsage();
  return `rss: ${memory.rss / 1000000}mb, arrayBuffers: ${memory.arrayBuffers / 1000000}mb, external: ${memory.external / 1000000}mb, heapTotal: ${memory.heapTotal / 1000000}mb, heapUsed: ${memory.heapUsed / 1000000}mb`;
}

(async () => {
  const cloudNode = new CloudNode({
    port: process.env.SCRAPER_CORE_PORT,
    rejectUnauthorized: true,
  });

  Session.events.on("new", function (data) {
    log('Perf', getMemoryUsage());
    log("Event", `Opening new connection: ${data.session.id}`);
    log("Info", `Currrent listener count: ${cloudNode.heroCore.connections.size}`);
  });

  await cloudNode.listen();
  log('Event', `CloudNode started on port: ${await cloudNode.port}`);
})().catch(error => {
  log('Event', 'Error starting Ulixee CloudNode', error);
  process.exit(1);
});

// main.js
import Hero, { ConnectionToHeroCore } from "@ulixee/hero";

function wait() {
  return new Promise((resolve) => {
    setTimeout(() => resolve(), 5000);
  })
}

async function main() {
  for (; ;) {
    console.log("fetching")
    const connectionToCore = ConnectionToHeroCore.remote("ws://localhost:1818");
    const browser = new Hero({
      connectionToCore,
      showChromeInteractions: false,
      showChrome: false,
      sessionPersistence: false,
      sessionKeepAlive: false,
    });

    const tab = await browser.newTab();
    await tab.goto("https://google.com");

    await wait();

    await browser.close();
    await connectionToCore.disconnect();
  }
}

main().then(() => {
  console.log("started");
})

I have also had this issue - the sqllite DB just continues to grow and eventually uses up all disk space. Not ideal for a production process.
I've create a simple script using recommended code from the docs
https://ulixee.org/docs/hero/advanced-concepts/sessions#sessions
https://ulixee.org/docs/hero/advanced-concepts/deployment
import { CloudNode } from '@ulixee/cloud';
import { Session } from '@ulixee/hero-core';
import * as Fs from 'fs';

(async () => {
  Session.events.on('closed', async ({ id, databasePath }) => {

    Fs.unlink(databasePath, (err) => {
      if (err) throw err; //FAILS due to file being busy
      console.log(databasePath);
    });
  });

  const cloudNode = new CloudNode();
  await cloudNode.listen({ port: 1818 });
})();

The session closed event fires, but when it attempts to FS.unlink I get 'EBUSY: resource busy or locked'

@StuartFuller
I tried to dwelve into the code base, see if it was a quickish fix perhaps. It kind of became a rats nest issue and I was short on time and not very knowledgable about the current project architecture. Here's what I found in regards to .db files not cleaned up.
Two files important
DefaultSessionRegistry
HeroSessionsSearch
In HeroSessionSearch, the sessions are accessed with retain() which increments the connection count. When we close the session, it checks if the connection count is < 1 (make sense) but in practice the count is always 2. So the session dbs are never closed since there are active connections. I tried using the get() instead of retain() in HeroSessionSearch which fixed the issue, the .db files are removed.
Buttt the Heap Out of Memory is still there. I profiled the memory and noticed that sessions data (in my case, around 5-10mb per session) are never removed from the memory because there are active references in a couple of places. This is where i left off, my knowledge being very limited about this project, I could not remove those references easily without breaking many things. I am in a crunch right now so i can't afford to spend more time on the issue so i went back to puppeteer with stealth plugins. When time is right, ill try to fix it, since i like my experience so far and the bots were reliably undetected.
Cheers!

@blakebyrnes Is this anything you could shed some light on? Seems like there is a memory leak of some sort which could also be preventing the db clean-ups.

This looks like it's happening because of Ulixee desktop apis. There's an env var to turn off the apis that might help, but I haven't tested: ULX_DISABLE_DESKTOP_APIS=true


cannot refresh msys.db from sourceforge

Dear all,
I reinstalled msys2 yesterday both under W10 andd W7 and though I can use pacman to install packages such as gcc make, etc..., I have a problem with pacman -Syu: I cannot download msys.db from sourceforge (mingw32 and mingw64 OK).
Please find below the debug output.
pacman -Syuv --debug
debug: pacman v4.2.1-257-g3356d-dirty - libalpm v9.0.1
debug: config: attempting to read file /etc/pacman.conf
debug: config: new section 'options'
debug: config: HoldPkg: pacman
debug: config: arch: x86_64
debug: config: SigLevel: Required
debug: config: SigLevel: DatabaseOptional
debug: config: LocalFileSigLevel: Optional
debug: config: new section 'mingw32'
debug: config file /etc/pacman.conf, line 73: including /etc/pacman.d/mirrorlist.mingw32
debug: config: new section 'mingw64'
debug: config file /etc/pacman.conf, line 76: including /etc/pacman.d/mirrorlist.mingw64
debug: config: new section 'msys'
debug: config file /etc/pacman.conf, line 79: including /etc/pacman.d/mirrorlist.msys
debug: config: finished parsing /etc/pacman.conf
debug: setup_libalpm called
debug: option 'logfile' = /var/log/pacman.log
debug: option 'gpgdir' = /etc/pacman.d/gnupg/
debug: option 'cachedir' = /var/cache/pacman/pkg/
debug: registering sync database 'mingw32'
debug: database path for tree mingw32 set to /var/lib/pacman/sync/mingw32.db
debug: GPGME version: 1.5.4
debug: GPGME engine info: file=/usr/bin/gpg, home=/etc/pacman.d/gnupg/
debug: checking signature for /var/lib/pacman/sync/mingw32.db
debug: 1 signatures returned
debug: fingerprint: AD351C50AE085775EB59333B5F92EFC1A47D45A1
debug: summary: valid
debug: summary: green
debug: status: Success
debug: timestamp:<PHONE_NUMBER>
debug: exp_timestamp: 0
debug: validity: full; reason: Success
debug: key: AD351C50AE085775EB59333B5F92EFC1A47D45A1, Alexey Pavlov (Alexpux<EMAIL_ADDRESS>owner_trust unknown, disabled 0
debug: signature is valid
debug: signature is fully trusted
debug: setting usage of 15 for mingw32 repository
debug: adding new server URL to database 'mingw32': http://downloads.sourceforge.net/project/msys2/REPOS/MINGW/i686
debug: registering sync database 'mingw64'
debug: database path for tree mingw64 set to /var/lib/pacman/sync/mingw64.db
debug: checking signature for /var/lib/pacman/sync/mingw64.db
debug: 1 signatures returned
debug: fingerprint: AD351C50AE085775EB59333B5F92EFC1A47D45A1
debug: summary: valid
debug: summary: green
debug: status: Success
debug: timestamp:<PHONE_NUMBER>
debug: exp_timestamp: 0
debug: validity: full; reason: Success
debug: key: AD351C50AE085775EB59333B5F92EFC1A47D45A1, Alexey Pavlov (Alexpux<EMAIL_ADDRESS>owner_trust unknown, disabled 0
debug: signature is valid
debug: signature is fully trusted
debug: setting usage of 15 for mingw64 repository
debug: adding new server URL to database 'mingw64': http://downloads.sourceforge.net/project/msys2/REPOS/MINGW/x86_64
debug: registering sync database 'msys'
debug: database path for tree msys set to /var/lib/pacman/sync/msys.db
debug: checking signature for /var/lib/pacman/sync/msys.db
debug: 1 signatures returned
debug: fingerprint: AD351C50AE085775EB59333B5F92EFC1A47D45A1
debug: summary: valid
debug: summary: green
debug: status: Success
debug: timestamp:<PHONE_NUMBER>
debug: exp_timestamp: 0
debug: validity: full; reason: Success
debug: key: AD351C50AE085775EB59333B5F92EFC1A47D45A1, Alexey Pavlov (Alexpux<EMAIL_ADDRESS>owner_trust unknown, disabled 0
debug: signature is valid
debug: signature is fully trusted
debug: setting usage of 15 for msys repository
debug: adding new server URL to database 'msys': http://downloads.sourceforge.net/project/msys2/REPOS/MSYS2/x86_64
Root      : /
Conf File : /etc/pacman.conf
DB Path   : /var/lib/pacman/
Cache Dirs: /var/cache/pacman/pkg/
Lock File : /var/lib/pacman/db.lck
Log File  : /var/log/pacman.log
GPG Dir   : /etc/pacman.d/gnupg/
Targets   : --
:: Synchronisation des bases de données de paquets...
debug: url: http://downloads.sourceforge.net/project/msys2/REPOS/MINGW/i686/mingw32.db
debug: maxsize: 26214400
debug: using time condition:<PHONE_NUMBER>
debug: opened tempfile for download: /var/lib/pacman/sync/mingw32.db.part (wb)
debug: curl returned error 0 from transfer
debug: response code: 304
debug: file met time condition
mingw32 est à jour
debug: url: http://downloads.sourceforge.net/project/msys2/REPOS/MINGW/x86_64/mingw64.db
debug: maxsize: 26214400
debug: using time condition:<PHONE_NUMBER>
debug: opened tempfile for download: /var/lib/pacman/sync/mingw64.db.part (wb)
debug: curl returned error 0 from transfer
debug: response code: 304
debug: file met time condition
mingw64 est à jour
debug: url: http://downloads.sourceforge.net/project/msys2/REPOS/MSYS2/x86_64/msys.db
debug: maxsize: 26214400
debug: using time condition:<PHONE_NUMBER>
debug: opened tempfile for download: /var/lib/pacman/sync/msys.db.part (wb)
debug: curl returned error 28 from transfer
erreur : échec de récupération du fichier « msys.db » depuis downloads.sourceforge.net : Connection timed out after 10700 milliseconds
debug: failed to sync db: erreur de la bibliothèque de téléchargement
erreur : la mise à jour de msys a échoué (erreur de la bibliothèque de téléchargement)

Guys, how many time you will ask the same questions? Looking into current issues will help you a lot.
SourceForge IS DOWN.
See issue https://github.com/Alexpux/MSYS2-packages/issues/296

Dear Alexpux,
I did have a look at the titles of the messages on the current page, but nothing seemed like my problem for me, although now I remember going through #302... => I need vacations! ;)
Anyway, thanks a lot for the link, for your patience and, not the least, for your work on msys2.
(Since I have 2 systems, I tried both methods given in Mingw-packages #702: edit of /etc/pacman.d/mirrorlist.xxxx and wget http://repo.msys2.org/msys/x86_64/pacman-mirrors-20150722-1-any.pkg.tar.xz
pacman -U pacman-mirrors-20150722-1-any.pkg.tar.xz and both worked)
Many Thanks,
Eviv


Add: make full-name visibility configurable

Business Description
User should be able to determine whether their full name, first name only or no name displays in wallet transactions. This should be configured in the Settings.
More Detail

[ ] Easier to implement for all transactions moving forward or all transactions (past and future)?

Acceptance Criteria
User can change preferences to display full name...
User can change perferences to display first name only...
User can change preferences to not display name at all, and only their wallet address will show in transactions

@L03TJ3 - which is easier to implement:

User makes a change and that change takes effect for all future transactions
User makes a change and that change takes effect for all transactions, past, present and future?

@decentralauren I think what we get by default is:
User makes a change and that change takes effect for all transactions, past, present and future?
Based on changes done to contract names for example which also iterated back over past transactions
@johnsmith-gooddollar @sirpy for confirmation on this

@johnsmith-gooddollar @sirpy
based on acceptance criteria.
is it easy to change the db to handle the 'first name only' scenario?
the other two is what we have now (when set to private > full name is not saved to database display value

@L03TJ3 no, for all I recall the name is fetched only once for new txs. so the name will not update for older txs. it probably is also better this way, less confusing for the parties invovled. you did a TX with Y and suddently you cant find it because he renamed to X.

@sirpy
Yes okay you are right, once it set it does not check and update again.
@decentralauren so we go for it will be affecting any future transactions. It will be immediate for third parties who request to use full-name, but we cannot control if 3rd parties cache it themselves ofcourse.

@L03TJ3 @sirpy thank you both! This is great.
Are we able to set the default here for new wallets or should I create a separate ticket for that?

@decentralauren what do you mean? the default is public, if we enable to change, then the user should do it via the wallet privacy settings.
but since the plan is to move to goodwalletv2, which doesn't currently have profiles, i'm not sure the effort is justified.

@sunela-tr - this is worth considering as a Profile sections materializes in the NGW.  LMK if you want me to move it to the NGW backlog.


Move releases to Fastlane

We have a Rakefile now, which is good, but Fastlane is better. Specifically because it has a GitHub Release Action that would make my life easier :sweat_smile:

Never got around to this, the Rakefile we have works okay.


MIC-322 update so that swagger doc matches actual behaviour

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

Project coverage is 82.90%. Comparing base (aed868a) to head (10a0ca6).

Additional details and impacted files
@@             Coverage Diff              @@
##               main      #68      +/-   ##
============================================
- Coverage     82.91%   82.90%   -0.02%
  Complexity      434      434
============================================
  Files           107      107
  Lines          1627     1626       -1
  Branches         48       48
============================================
- Hits           1349     1348       -1
  Misses          258      258
  Partials         20       20

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


Week 2 Review Ticket

code.org
java using objects
shared repo

2.7/3 - Graded by Kian Pasokhi

Completed code.org and provided necessary notes that showed understanding.
Java using objects: Coded a significant figure that determines the significant number between two inputs.


Add lint-staged and husky to auto lint code before committing changes

Please provide a summary of the feature
Currently, running the linter is a part of the steps to take before submitting a PR. The tools I'm suggesting will automate this so people won't have to think about doing it...
lint-staged: This will run eslint on files that have been staged for commit. If it runs into an error the contributor will not be able to commit the changes until they fix the errors.
husky: This will allow us to use git hooks. We will use the git hook, pre-commit, to run lint-staged whenever someone tries to commit changes.
Are there any potential issues you foresee with this feature
I discovered eslint was not picking up the prettier config so prettier has pretty much never been run on the code. If you run prettier you will see it flags a lot of errors. This can be fixed along with this issue, but since it will change pretty much every file, there are bound to be some conflicts that arise.
Additional context
Running eslint showed some things to fix

After fixing the issue with eslint using prettier config this is the result

The purpose here is to keep code syntax/formatting consistent throughout the codebase since we have a variety of people working on it.
Which Branch should this fix be Pulled into?
PR into development
If you're interested in working on this feature, please comment so you can be assigned
This can be assigned to me.

Love this idea, and the team is on board with some form of implementation.
What if we set this to on push instead of on commit?

Once you commit changes they are no longer staged so they won't be picked up by lint-staged. Unless you mean to lint the whole codebase before pushing?

Once you commit changes they are no longer staged so they won't be picked up by lint-staged. Unless you mean to lint the whole codebase before pushing?

I guess? Assuming the code base is properly linted prior to the commits wouldn't the push effectively only lint all the staged commits?

No because lint-staged only targets staged(git add) files. Once you commit the files the stage is empty per se so it won't check anything. When you run eslint .   it will check all of the files  even if you only make a change to one.

No because lint-staged only targets staged(git add) files. Once you commit the files the stage is empty per se so it won't check anything. When you run eslint . it will check all of the files even if you only make a change to one.

Gotcha. Others have mentioned that some form of checking is possible on push instead of on commit. Are you familiar with how that would be possible?

Yea we can do that, but that would result in extra commits just for fixing lint errors. If thats no problemo then it's all good.

@luciousmc, are you working on this?

Yea, Caleb just assigned it to me the other day. I'll do it tonight.👍

Cool! I'm curious how you do this. I tried to add the lint-stage and husky to one of my projects and ran into some issues


The semantics of run_stats is changed in the latest lwAFTR release

Something I noticed after the release is that run_stats for intel_mp now means something slightly different than what it's intended to mean. In the docs, the run_stats option has this description:

Boolean indicating if this app instance should collect device statistics. One per physical NIC (conflicts with master_stats). There is a small but detectable run time performance hit incurred. The default is false.

In particular, each physical NIC should only have one app with the flag on. In the latest lwAFTR, we use the flag on multiple apps at once on a single NIC.
Instead, I think we should have NIC apps always sync their own RX/TX counters (regardless of what run_stats is) or maybe return to having one app always sync the stats (and find another way to show the queue counters in snabb top).

Fixed by #1148


rkt image gc doesn't work

Environment
rkt Version: 1.28.1
appc Version: 0.8.10
Go Version: go1.8.3
Go OS/Arch: linux/amd64
Features: -TPM +SDJOURNAL
--
Linux 4.4.0-93-generic x86_64
--
NAME="Ubuntu"
VERSION="16.04.3 LTS (Xenial Xerus)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 16.04.3 LTS"
VERSION_ID="16.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"
VERSION_CODENAME=xenial
UBUNTU_CODENAME=xenial
--
systemd 229
+PAM +AUDIT +SELINUX +IMA +APPARMOR +SMACK +SYSVINIT +UTMP +LIBCRYPTSETUP +GCRYPT +GNUTLS +ACL +XZ -LZ4 +SECCOMP +BLKID +ELFUTILS +KMOD -IDN

What did you do?
rkt image gc --debug=true --grace-period=0s

What did you expect to see?
Unused images be removed
What did you see instead?
No output

What do rkt list and rkt image list show? (If there is nothing that can be removed, image gc won't print anything even in debug mode).

# rkt list
UUID		APP				IMAGE NAME										STATE	CREATED		STARTED		NETWORKS
16fb5e00	aleph-record-caretaker-ubuntu	foo.bar/melinda/aleph-record-caretaker-ubuntu:1.0.0	running	1 day ago	1 day ago	default:ip4=<IP_ADDRESS>
c108a331	eresource-tool-legacy-ubuntu	foo.bar/melinda/eresource-tool-legacy-ubuntu:1.0.0	running	1 day ago	1 day ago	default:ip4=<IP_ADDRESS>
# rkt image list
ID			NAME											SIZE	IMPORT TIME	LAST USED
sha512-317f64e44225	foo.bar/melinda/eresource-tool-legacy-ubuntu:1.0.0	833MiB	3 days ago	3 days ago
sha512-78fa3914d96e	foo.bar/melinda/aleph-record-caretaker-ubuntu:1.0.0	636MiB	2 days ago	2 days ago
sha512-887890e697d9	coreos.com/rkt/stage1-coreos:1.25.0							112MiB	2 days ago	2 days ago
sha512-93d54be07b14	foo.bar/melinda/aleph-record-caretaker-ubuntu:1.0.0	636MiB	2 days ago	2 days ago
sha512-f76ec4963ef2	foo.bar/melinda/aleph-record-caretaker-ubuntu:1.0.0	1.2GiB	2 days ago	2 days ago
sha512-1308db08fa77	coreos.com/rkt/stage1-coreos:1.28.1							206MiB	1 day ago	1 day ago

I'm not completely sure, but I think this is the same issue as https://github.com/rkt/rkt/issues/3772.


Revert "Table Query Parser fixes (PLFM-4892)"

using this  to create a revert branch so i can cherry pick changes into stack-235. DO NOT MERGE

just needed this pull request so github would create the automatic revert branch


Hard to understand error message when the identity file does not end with a \n

Environment

OS: Debian 12
rage version: v0.10.0 (cargo install)

What were you trying to do
decrypt a test content using the following command line:
 rage --decrypt --identity /tmp/tmpnq_roxuq --output - /tmp/encrypted.bin

with an identity file like:
$ hexdump /tmp/tmpnq_roxuq
0000000 4741 2d45 4553 5243 5445 4b2d 5945 312d
0000010 4d4b 3636 3953 3959 3859 4747 4646 3354
0000020 5446 434a 5853 4b37 4855 4847 5a46 5343
0000030 5a50 4448 5335 3439 5a43 4639 4846 3734
0000040 3443 5341 5148 5737 4455
000004a

(so the actual key without any trailing character)
What happened
Following error message:
$ rage --decrypt --identity /tmp/tmpnq_roxuq --output - /tmp/encrypted.bin
Error: Tried to reset after the underlying buffer was exceeded.

[ Did rage not do what you expected? Could an error be more useful? ]
[ Tell us: https://str4d.xyz/rage/report                            ]

This used to work with rage < 0.10.
Thank you

The error does go away when there is a newline present, but it also goes away when the identity file contains its usual comment header (and no trailing newline). So I think the issue is that the identity file parser, which needs to distinguish between three different possible file types (encrypted identity, SSH identity and native age identity) tries to read an entire line, and in the case of your file that results in the entire file being read, which messes with the assumptions being made about the underlying buffer being larger than the read.


use logger.debug  in getAllToggles

Describe the feature request
Hey there,
Inside the getAllToggles api there is a logger usage:
this.logger.info(
            'Get all feature toggles for provided context',
            inContext,
        );

This log can be very spammy with high traffic, and it seems the purpose of this log is for debugging/troubleshooting, so I would like to suggest changing it to:
this.logger.debug(
            'Get all feature toggles for provided context',
            inContext,
        );

If it is accepted as a request for change I can submit a PR for it.
Background
No response
Solution suggestions
No response

yes, this was an oversight.

released as part of 0.15.1.

@ivarconr thank you so much for responding so quickly!


RFC: RMLTC0007h-CSV & RMLTC0007h-JSON & RMLTC0007h-XML

RFC for:

RMLTC0007h-CSV
RMLTC0007h-JSON
RMLTC0007h-XML

The description of these test cases reads:

Tests the generation of triples to a non-IRI named graph, which is an error

Yet, the tests do not expect an error to be the result. This is curious, since according to the specification an error should be thrown.
R2RML tests also seem to indicate an expected error: https://www.w3.org/2001/sw/rdb2rdf/test-cases/#R2RMLTC0007h
Proposed:

Change 'Error expected' to 'Yes' for these test cases.
Change 'Expected knowledge graph' to 'None' for these test cases.

Thanks for your RFC proposals! Indeed, RMLTC007h-* test case should raise an error to comply with the R2RML test cases. As the test case's description mentions, assigning triples to a non-IRI named graph is an error. We will update the test cases to correct this.
Once the test cases are updated, we will close this issue.


feat(rate-limit) Rate limit by any http header

NOTE: Please read the CONTRIBUTING.md guidelines before submitting your patch,
and ensure you followed them all:
https://github.com/Mashape/kong/blob/master/CONTRIBUTING.md#contributing
Summary

Support new value in limit_by config "http_header".
New config parameter http_header, value can be any http header for e.g. User-Agent
When http_header is configured, the header value is extracted from request and used in ratelimit keys thus supporting ratelimit by any http request header

curl -i -X POST --url http://localhost:8001/apis/test_api/plugins
--data "name=rate-limiting" --data "config.minute=5"
--data "config.policy=redis" --data "config.limit_by=http_header"
--data "config.http_header=X-My-Client-Id"
--data "config.redis_host=localhost"
Full changelog
Support config.limit_by http_header in schema and use it in idetntifier
Issues resolved
Fix #XXX

Thanks  @p0pr0ck5 for the review, added an integ test as well. Wanted to add a unit for get_identifier in handler, but it's not a public method.

@sachin-shetty looks like there are some linting failures: https://travis-ci.org/Mashape/kong/jobs/239042805
Can we address these?

Thanks @p0pr0ck5, fixed the lint warnings.

Thanks @p0pr0ck5 - Switched to using ngx.var[header_name]. Nginx behavior is to only provide first header in http_ vars when multiple headers are specified, so I guess we can leave it at that.

Hi @p0pr0ck5 - any update?

@sachin-shetty I will have a look in the next day or two. Thanks again for the submission!

So, while I think the idea presented here, I have to wonder about future proofing and use cases. Would it not be equally valid to rate limiting based on a query string or request body parameter? If so, how do we define this? Configuring such a plugin in the given environment could quickly turn into a nightmare (this would likely be covered in a planned refactor of our data model and plugins development ecosystem). Again, I think this idea if perfectly valid, I just have to wonder about the "and the kitchen sink" that will follow this shortly, and how we handle it. Not sure what the best approach is here.
@thibaultcha any thoughts on this?

Yes, query (maybe url path parts), header, and the body are basic options. And in headers there are special headers like cookies that might warrant different processing. Also body can be in different forms (raw json (or other formats), multipart/form-data or application/x-www-form-urlencoded). Also it might be that you want to find parameter in each of these (AND), or search a few of them (OR). Also same parameter can maybe be presented with slightly different key depending where it exists, e.g. in headers the parameters may be prefixed with X-. I think this same thing happens in many plugins currently, and that warrants somekind of blueprint or recommendation.

I kind of agree, for our own needs I am writing a custom plugins that combines various headers and applies some custom filters and rules.
The primary reason for this pull request was a request from a user on the user group: https://groups.google.com/forum/#!topic/konglayer/VsyjSsDVB3A
Sure a more sophisticated rule engine would be good, but in the short term a header support would be nice to have I think.

Marking this is 'to discuss', as it doesn't really need any more review. IMO the code is reasonable, just a matter of what direction we want to take this in.

@sachin-shetty also, in the mailing list post discussed, the user in question realized that a header value wouldn't be appropriate (as it would be the same thing as ratelimiting by consumer).

Yes, the user found a way. IMO the rate limit by header is important for anybody not using auth plugin and only using kong for ratelimiting just like we do. I am already using header as well as request uris in the keys with changes similar to what I have posted in this pull request.

Also this would be a good fix for us, where we use RS256 which has multiple clients using one Kong 'consumer'

Hi, any luck? fsargent expressed his need for this feature as well

Hi, did you get a chance to discuss this?

I need this feature as well. Another option to solve this would be to let Kong create Consumers on-the-fly for a header value (something like "OIDC_claim_sub" in my case) if such a Consumer doesn't exists yet.
In my case, a user has already been authenticated by an upstream server, so the username inside the header "OIDC_claim_sub" can be trusted.

Sorry for the delay on our side. @sachin-shetty can you address the merge conflict here? It is a small adjustment. We will circle back to this shortly when time becomes available.
@salazar any thoughts here?

@p0pr0ck5 - I have resolved the conflicts.

Alright, closing this then. Haven't heard much desire for rate-limiting by HTTP header, but if is a feature that is desired by anobody else out there, feel free to send another PR :)


prep build for release (and release)

i guess this should include removing the links from the sidebar too?

Done: https://github.com/cosmos/cosmos-ui/releases/tag/0.2.0


Quotes

WIP

I know I know. I'll finish it! :smile:

@katiedelfin Should be ready now. However, I'll probably need your help adding uphearts to quotes. It looks to be tied to compliments right now, so I didn't want to muck around with anything :smile:

The only other issue is that the quote is tied to the person that's being quoted, but I'm not storing who is the one quoting, so that might be worth changing, too.

Yay! So... what if, instead of uphearts, we just use emoji reactions for :allthethings:? See #4.
Also, yeah, would be nice to store who was adding the quote.

That's awesome!

Okay, @katiedelfin - this stores both the person being quoted and the person quoting, and the setup reflects what you did for compliments. That should be it now.

I know it might mess stuff up, but we could change the URL to chatter.preschool.io to fit with both compliments and quotes. Not a big deal, though :wink:

Oooh, yes. On it.


Add other preconditioner package recommendations

Adds AMG.jl and Preconditioners.jl to the recommended packages to check out.

Needs rebasing now!

Done.


Why not use memoryStore in production?

var store = new ExpressBrute.MemoryStore(); // stores state locally, don't use this in production

The only two downsides I see are:

Rebooting clears any limits.
Node.js clusters and load balanced servers don't share state

Both of which seem like false negatives.  If I can save a request to Redis and the complexity, that's a win for me.
Other drawbacks?  Does it leak memory over time?  Thanks!

You hit the nail on the head for the downsides. The second one is particularly an issue because it tends to be easier to get node servers to restart than other servers in my experience (if you can cause an un-caught exception then you can bypass the brute force protections).
The only "memory leak" is that it stores the request history in ram, and uses setTimeout for each ip to clean it up after the the lifetime is over. I haven't tested large numbers of setTimeout calls with node (other people have it looks like http://stackoverflow.com/questions/12168708/is-there-any-limit-to-settimeout) so I wasn't sure if that would cause issues. If you didn't have a lifetime set then it would leak memory over time because it would never clean up the list. Also setTimeout would have issues if you had extremely large lifetimes (e.g. over 24.8 days). There probably should be a range check in there to prevent that... I'll look at a patch. Realistically this probably isn't a huge issue because we're not store much data and it shouldn't be possible to spoof IP addresses for TCP packets.
Long-story-short, If you are in an single-threaded environment, and are confident in your error handling, and set a reasonable lifetime, then it's probably fine. I just wanted to make sure people thought about it before simply using MemoryStore just because that's what I used in the examples. If you think there is a better way to talk about this in the README feel free to change it and submit a PR.

Thanks for the clarification.  Exactly what I was looking for :-)


enhancement: keep element type when merging

As outlined in #927 in the unstructured library, currently when we merge the extracted elements with the inferred elements, if it's determined that an element from the extracted set is the same element as an element in the inferred set, we use the extracted element because we trust that the text is correct. What would be better is to update the classification of the element using the inferred type, because this tends to be more accurate. That's what this PR is for.

Closing for now, after investigation it looks like the issue might be something else.


feat(consensus): signer field in Signed tx.

Motivation
Ref #481

Solution
Added signer: OnceLock<Address> in Signed

PR Checklist

[ ] Added Tests
[ ] Added Documentation
[ ] Breaking changes

I did a little bit of work here, and because we're attempting to be generic over Sig, this is harder than it seems without a Recoverable trait for the signature type. which i'm not sure we want to introduce yet. Going to close this as needing more design work
another problem specialization would solve 🤔


Edit allocation IP Address & add nginx config location to Pterodactyl…

…NginxProxy.md
Edited the allocation to match "Pterodactyl pass-through IP" and added the default nginx config location.

I am not sure if this is correct.
The <IP_ADDRESS> is only for accessing the docker-container from the host, but the container bind-ip should still be <IP_ADDRESS>, shouldn't it?
At least that's how it works for my containers ^^

Pterodactyl actually automatically converts the <IP_ADDRESS> to <IP_ADDRESS> (or whatever is the ip of pterodactyl0 interface)
Also just a side note but pterodactyl doesn't have to run on that ip, maybe there should be instructions to check wings' config.yml for the ip.

Also just a side note but pterodactyl doesn't have to run on that ip, maybe there should be instructions to check wings' config.yml for the ip.

Have you ever triedusing another docker interface (IP)? Yes, it does technically work, but I advise strongly against it, I didn't get it to work and it broke my pterodactyl interfaces. This is the config.yml part where you could submit changes:
docker:
  network:
    interface: <IP_ADDRESS>
    dns:
    - <IP_ADDRESS>
    - <IP_ADDRESS>
    name: pterodactyl_nw
    ispn: false
    driver: bridge
    network_mode: pterodactyl_nw
    is_internal: false
    enable_icc: true
    network_mtu: 1500
    interfaces:
      v4:
        subnet: <IP_ADDRESS>/16
        gateway: <IP_ADDRESS>
      v6:
        subnet: fdba:17c8:6c94::/64
        gateway: fdba:17c8:6c94::1011

Apart from my potential incompetence: The point why I created this pull request is that I simply find this conversion unnecessary. You can bind your services directly to this IP, so I see the conversion as just an extra step. It's like if you have a system with multiple public IP addresses, a service only binds the port to one and not to all IP addresses.


[BUG]: Failed to build installable wheels for some pyproject.toml based projects (evdev)

In raising this issue I confirm that

[X] I have fully completed the issue template
[X] I have searched open and closed issues for duplicates
[X] I have read the Contribution Guidelines
[X] I have read the Code of Conduct
[X] I have read the Documentation

Describe the bug
I was following the Ubuntu guide to install the streamdeck gui on Debian 12
Steps to reproduce the behavior
Follow the tutorial up to python3 -m pipx install streamdeck-linux-gui, this command will fail
Expected behavior
The package gets installed
Screenshots
No response
System Information
Debian 12 Gnome Wayland
Stream Deck Version
Stream Deck XL

The output:
Building wheels for collected packages: evdev
  Building wheel for evdev (pyproject.toml): started
  Building wheel for evdev (pyproject.toml): finished with status 'error'
Failed to build evdev

PIP STDERR
----------
  error: subprocess-exited-with-error

  × Building wheel for evdev (pyproject.toml) did not run successfully.
  │ exit code: 1
  ╰─> [40 lines of output]
      running bdist_wheel
      running build
      running build_py
      creating build
      creating build/lib.linux-x86_64-cpython-311
      creating build/lib.linux-x86_64-cpython-311/evdev
      copying evdev/events.py -> build/lib.linux-x86_64-cpython-311/evdev
      copying evdev/eventio.py -> build/lib.linux-x86_64-cpython-311/evdev
      copying evdev/device.py -> build/lib.linux-x86_64-cpython-311/evdev
      copying evdev/__init__.py -> build/lib.linux-x86_64-cpython-311/evdev
      copying evdev/ff.py -> build/lib.linux-x86_64-cpython-311/evdev
      copying evdev/ecodes.py -> build/lib.linux-x86_64-cpython-311/evdev
      copying evdev/eventio_async.py -> build/lib.linux-x86_64-cpython-311/evdev
      copying evdev/evtest.py -> build/lib.linux-x86_64-cpython-311/evdev
      copying evdev/util.py -> build/lib.linux-x86_64-cpython-311/evdev
      copying evdev/uinput.py -> build/lib.linux-x86_64-cpython-311/evdev
      copying evdev/genecodes.py -> build/lib.linux-x86_64-cpython-311/evdev
      running egg_info
      writing evdev.egg-info/PKG-INFO
      writing dependency_links to evdev.egg-info/dependency_links.txt
      writing top-level names to evdev.egg-info/top_level.txt
      reading manifest file 'evdev.egg-info/SOURCES.txt'
      reading manifest template 'MANIFEST.in'
      warning: no previously-included files found matching 'evdev/ecodes.c'
      adding license file 'LICENSE'
      writing manifest file 'evdev.egg-info/SOURCES.txt'
      copying evdev/input.c -> build/lib.linux-x86_64-cpython-311/evdev
      copying evdev/uinput.c -> build/lib.linux-x86_64-cpython-311/evdev
      running build_ext
      running build_ecodes
      writing /tmp/pip-install-32b1ll06/evdev_4fa6b67351da4ff19647dd26ebca73fc/evdev/ecodes.c (using /usr/include/linux/input.h /usr/include/linux/input-event-codes.h /usr/include/linux/uinput.h)
      building 'evdev._input' extension
      creating build/temp.linux-x86_64-cpython-311
      creating build/temp.linux-x86_64-cpython-311/evdev
      x86_64-linux-gnu-gcc -Wsign-compare -DNDEBUG -g -fwrapv -O2 -Wall -g -fstack-protector-strong -Wformat -Werror=format-security -g -fwrapv -O2 -fPIC -I/home/michael/.local/pipx/venvs/streamdeck-linux-gui/include -I/usr/include/python3.11 -c evdev/input.c -o build/temp.linux-x86_64-cpython-311/evdev/input.o -std=c99 -Wno-error=declaration-after-statement
      evdev/input.c:10:10: fatal error: Python.h: No such file or directory
         10 | #include <Python.h>
            |          ^~~~~~~~~~
      compilation terminated.
      error: command '/usr/bin/x86_64-linux-gnu-gcc' failed with exit code 1
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for evdev
ERROR: ERROR: Failed to build installable wheels for some pyproject.toml based projects (evdev)

The solution is to also install the python3-dev package on Debian 12

I had this same issue with evdev
I installed the gcc package and build completed
pipx install  streamdeck-linux-gui                                             1 ✘  5s 
Fatal error from pip prevented installation. Full pip output in file:
    /home/jrclark/.local/state/pipx/log/cmd_2024-09-27_21.10.58_pip_errors.log

pip failed to build package:
    evdev

Some possibly relevant errors from pip install:
    error: subprocess-exited-with-error
    error: command 'gcc' failed: No such file or directory
    ERROR: ERROR: Failed to build installable wheels for some pyproject.toml based projects (evdev)

Error installing streamdeck-linux-gui.

pacman gcc -Syu

pipx install  streamdeck-linux-gui                                               ✔  9s 
  installed package streamdeck-linux-gui 4.1.3, installed using Python 3.12.5
  These apps are now globally available
    - streamdeck
    - streamdeckc
done! ✨ 🌟 ✨


[error]The $pubspecFilePath does not contain a "flutter" key. ```

build log:
2:  Ubuntu
...

105:  branch 'develop' set up to track 'origin/develop'.
106:  ##[endgroup]
107:  [command]/usr/bin/git log -1 --format='%H'
108:  'xx'
109:  ##[group]Run zgosalvez/github-actions-get-flutter-version-env@v2
110:  with:
111:  pubspec-file-path: pubspec.yaml
112:  ##[endgroup]
113:  ##[error]The $pubspecFilePath does not contain a "flutter" key.

CI File fragment:
  buildAndroid:
    name: 🤖 Build Android
    if: github.event.pull_request.draft == false
    runs-on: ubuntu-latest
    timeout-minutes: 30
    steps:
      - name: ⬇ Checkout repository
        uses: actions/checkout@v4
        with:
          ref: develop
      - name: Get Flutter version
        id: get-flutter-version
        uses: zgosalvez/github-actions-get-flutter-version-env@v2
      - name: ⚙ Setup Flutter
        uses: subosito/flutter-action@v2
        with:
          flutter-version: ${{ steps.get-flutter-version.outputs.version }}

pubspec.yaml:
name: test
description: Test

publish_to: 'none' # Remove this line if you wish to publish to pub.dev

version: 2023.0.1+1

environment:
  # Dart
  sdk: 3.3.0
  # Flutter SDK
  flutter: 3.19.2

# Flavor config
flavorizr:
  ide: 'idea'

Please try switching to v3


Merge the dev into the main for 1st prototype

Summary
Merge all work carried out so far on the MOOCs to the main during the selection period of Outreachy.
Fixes #2
DEVELOPMENT PROGRESS
Design

[x] Landing page, Sign up, and login pages
Issue https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/issues/11
Figma link

Frontend Development

[x] Set up Project structure
issue https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/issues/20
PR https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/pull/21
[x] Landing-page
issue : https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/issues/36
PR https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/pull/43
[x] Header
issue https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/issues/19
PR https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/pull/37
[x] Footer
issue https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/issues/19
PR https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/pull/37
[x] Login
issue https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/issues/24
PR https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/pull/31
[x] Sign up page
issue https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/issues/24
PR https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/pull/31

Backend Development

[x]  Setup project structure
issue https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/issues/13
PR https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/pull/14
[x]  Database model for users and admin and passwords
issue https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/issues/30
PR https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/pull/32
[x]  Asynchronous function wrappers -(abstracts try catch block)
issue https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/issues/33
PR https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/pull/34
[x]  Error handler middleware
issue https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/issues/33
PR https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/pull/34
[x] Authentication middleware
PR https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/pull/44
[x] Permissions router middleware
issue https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/issues/45
PR https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/pull/46
[x]  Signup
issue https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/issues/7
PR https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/pull/38
[x]  Login
issue https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/issues/7
PR https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/pull/38
[x]  Social signin/signup with Google
issue https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/issues/48
PR https://github.com/Open-Science-Community-Saudi-Arabia/MOOCs/pull/49

What should a reviewer concentrate their feedback on?

[ ] Everything looks ok?

This is a super huge PR, which we will try to avoid but we would like to add this work to the main branch so we can start polishing it.

Alright @BatoolMM, I'll go through the backend src code if everything is intact

I will merge this now and will go though each part of the front-end and back-end together after the break!
Even if part of it is broken, we will fix it in a separate PR!


update 1.4.5

Mgrs 1.4.5 Update
Jira Ticket
Upstream
Dependencies
Changes

Updated version number and sha256
Updated pinnings on dependencies
Added doc_url and description
Added pytest requirements

Linter check found the following problems:
The following problems have been found:
WARNING: /tmp/abs_465z9wr_7_/clone/recipe/meta.yaml:29: python_build_tool_in_run: The python build tool packaging is in run depends
WARNING: /tmp/abs_465z9wr_7_/clone/recipe/meta.yaml:46: missing_description: The recipe is missing a description
ERROR: /tmp/abs_465z9wr_7_/clone/recipe/meta.yaml:46: missing_documentation: The recipe is missing a doc_url or doc_source_url
Errors were found

Linter check found the following problems:
The following problems have been found:
WARNING: /tmp/abs_cbqhhwpvi3/clone/recipe/meta.yaml:29: python_build_tool_in_run: The python build tool packaging is in run depends
WARNING: /tmp/abs_cbqhhwpvi3/clone/recipe/meta.yaml:47: missing_description: The recipe is missing a description
Warnings were found

Linter check found the following problems:
The following problems have been found:
WARNING: /tmp/abs_2bm0ikr8c3/clone/recipe/meta.yaml:29: python_build_tool_in_run: The python build tool packaging is in run depends
Warnings were found

Linter check found the following problems:
The following problems have been found:
WARNING: /tmp/abs_6bemhwvd5o/clone/recipe/meta.yaml:29: python_build_tool_in_run: The python build tool packaging is in run depends
Warnings were found


Add a helper for UtxoM to lookup by txId, to simplify examples

https://github.com/Plutonomicon/cardano-transaction-lib/blob/683d167fca72fad950f806ca2277b234aa9ca242/examples/AlwaysSucceeds.purs#L78
Can be replaced with
lookupTxId :: TransactionHash -> UtxoM -> Maybe (TransactionInput /\ TransactionOutput)

Maybe it should return Array instead of Maybe


current version and higher than 4.0.0 doesn't work in older browser "this.state.model.renderer is null"

Hello, I have a problem, after update from 4.0.0 to 4.1.2, I can't run it in older browsers like IE11 and firefox 48. In the console: "this.state.model.renderer is undefined or null" in the file 'webpack:///~/deck.gl/dist/layers/core/solid-polygon-layer/solid-polygon-layer.js'

@SlavaDev We focus on supporting so called "evergreen" browsers, and we expect users to stay reasonably up to date. With this in mind:

Firefox: With Firefox currently being on v55 we can't justify debugging an issue on Firefox v48. That said, it is probably a very minor breakage, so if you or someone else was able to find the issue and the fix turns out to be simple, we would of course accept a PR (or even considering making a PR if the problem description was clear enough).
IE11: This is the one exception to our "evergreen" policy. We would really like to support IE11 as the one non-evergreen browser. The effort is likely small. Unfortunately a lack of access to Windows machines on our side keeps this from happening. Again we'd happily take PRs here.

Hello, seems like problem on luma.gl. After investigate I found:  ```
'deck.gl error while initializing ScatterplotLayer({id: 'scatterplotLayer'})
Error: Could not parse GLSL compiler error: Shader compilation errors
(27, 1): Unknown extension: GL_OES_standard_derivatives
(27, 48): Internal compiler error
(36, 1): Required extension GL_EXT_draw_buffers is not supported
(36, 41): Internal compiler error'

@SlavaDev Thanks for investigating. Yes this is a luma.gl issue. It seems that the shader compiler is seeing the request to enable the extension even though that line is under #ifdef https://github.com/uber/luma.gl/blob/master/src/shadertools/lib/platform-defines.js#L86
This is pretty easy to fix and could be addressed in the next luma.gl patch release (maybe within a week or so).
Would you consider opening the issue in luma?

@1chandu

@ibgreen Sure


west: Update hal_stm32 module to head following cube updates

This update contains the following changes

Introduce STM32G0 Cube V1.2.0
Update API for STM32MP1 IPCC
Update Cube version for following series:
STM32F0XX: from version: V1.9.0 to version: V1.10.1
STM32F1XX: from version: V1.6.1 to version: V1.7.0
STM32F4XX: from version: V1.21.0 to version: V1.24.1
STM32F7XX: from version: V1.12.0 to version: V1.15.1
STM32L0XX: from version: V1.10.0 to version: V1.11.2
STM32L1XX: from version: V1.8.0 to version: V1.9.0
STM32L4XX: from version: V1.13.0 to version: V1.14.0
STM32WBXX: from version: V1.0.0 to version: V1.1.1

Signed-off-by: Erwan Gouriou<EMAIL_ADDRESS>

This PR contains update to head of hal_stm32 modules.
It replaces following commits:
https://github.com/zephyrproject-rtos/zephyr/pull/16983
https://github.com/zephyrproject-rtos/zephyr/pull/16976
https://github.com/zephyrproject-rtos/zephyr/pull/16975
https://github.com/zephyrproject-rtos/zephyr/pull/16974
https://github.com/zephyrproject-rtos/zephyr/pull/16960
https://github.com/zephyrproject-rtos/zephyr/pull/16959
https://github.com/zephyrproject-rtos/zephyr/pull/16956
https://github.com/zephyrproject-rtos/zephyr/pull/16955
Wait https://github.com/zephyrproject-rtos/zephyr/pull/16983 CI to complete before Merge
It also includes dependency for https://github.com/zephyrproject-rtos/zephyr/pull/16962 and https://github.com/zephyrproject-rtos/zephyr/pull/16873.

As this change imapct all STM32 based targets, CI will likely fail.
All series have beenpushed independantly and got CI passed (except #16962 which is a different case).
Remaining one is STM32L4 (#16983), so when checks are passed on it, it is safe to merge current one.

As this change imapct all STM32 based targets, CI will likely fail.

it should not fail.

it should not fail.

I meant time out, sorry.
Shouldn't it trigger full non regression on all boards impacted by the change, so in this case all STM32 based boards? This is why I'd expect it to time out.
Unless it does not build against all impacted boards for some reason I don't get.

Unless it does not build against all impacted boards for some reason I don't get.

this happens only if you actually change the board files directly.

this happens only if you actually change the board files directly.

ouch!

Waits https://github.com/zephyrproject-rtos/hal_stm32/pull/13 to be merged.


Can you provide a Visio stencil set for download?

The SVG icons are great, but have to be inserted one at a time into Visio to use them.  Could you provide a Visio stencil file with these icons for use in the desktop client?
Ideally Visio would have the ability to bulk import SVGs to create stencil files.

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

ID: 6624d44c-f9f6-02f8-2c67-0f0637fed62e
Version Independent ID: 9346a8ff-bab5-be2c-9dc1-d546fda1efa9
Content: Azure Icons - Azure Architecture Center
Content Source: docs/icons/index.md
Service: architecture-center
GitHub Login: @doodlemania2
Microsoft Alias: pnp

Thanks for reporting - this issue is under review.  This is a Microsoft Internal DevOps Bug ID AB#313893

Visio stencils are on the roadmap.  Currently available for Visio Web as announced here.  https://techcommunity.microsoft.com/t5/visio-blog/visually-represent-your-azure-architecture-using-the-latest/ba-p/1634509.
A new version of the Zip file with SVG also published today.  https://docs.microsoft.com/en-us/azure/architecture/icons/


[ENH] Update bundled .NET runtime to version 6.0

breaking-change

what is breaking?
you mean GmodNET.API breaking changes?

@SupinePandora43 Despite .NET 6 is compatible with .NET 5, I believe users must retarget their projects. Thus change is breaking.


make gt(.json) consistent with feature extracted by action model(eg, I3D)

when doing sample with images from video, how to generate gt json consistent with feature extracted by action model(eg, I3D)?
look forward to your reply? Do you have same code for us inference? Thanks very much!

The json file stores the video information and action annotations of a dataset, and is mostly irrelevant of how frames are sampled from videos. This is because action annotations are stored in absolute timestamps (seconds) in the json file, and thus are independent of how the video features are extracted.
When using different video features, you will need to specify how the features are extracted in the yaml config file (e.g., thumos_i3d.yaml). The parameters are under dataset and include

feat_stride: Temporal stride of the local sliding window for feature extraction.
num_frames: Number of frames within each local window
default_fps: Default FPS for all videos. If not specified, FPS will be derived from video information stored in the json file (not recommended).

You might want to refer to #4 for an illustration of these parameters.

Thanks very much!

Mark as resolved.


Fix Windows environment

#73 should work after this PR. I might merge the branch simulation into the branch master later.

test_all is failing because of the following ECC:
"26a980e4db4e": [
[[3,2,5,5,["26a980e4db4f"],[7.60674934824959598e-02,3.79730310662971804e-02]],[["add", ["P2"],["P0", "P0"]],["add", ["P3"],["P0", "P1"]],["add", ["P4"],["P1", "P1"]],["rx", ["Q1"],["Q1", "P3"]],["rx", ["Q1"],["Q1", "P4"]]]]
,[[3,2,5,5,["26a980e4db4e"],[7.60674934824958765e-02,3.79730310662972012e-02]],[["add", ["P2"],["P0", "P0"]],["add", ["P3"],["P0", "P1"]],["add", ["P4"],["P1", "P1"]],["rx", ["Q1"],["Q1", "P4"]],["rx", ["Q1"],["Q1", "P3"]]]]
]

but it's unrelated to this PR. Merging for now.


Make use of partial prerendering to render /blog and /blog/:slug

Current
/blog and /blog/:slug are ISRed (revalidate for every hour). This is mostly for the sake of updating the view count periodically since the rest of the content is static.
To be explore
Partially render the two pages while wrapping the view count in Suspense, leaving it SSRed. This can potentially re-introduce like count, which is previously removed due to de-synchronized count between /blog and /blog/:slug.
unstable_noStore() may be helpful in this case since the counts are retrived using fetch internally.
Reference

https://github.com/vercel/next.js/pull/57287
https://github.com/vercel/next.js/pull/56930

Potential blockers

Emotion is unable to pre-render styles when a page contains dynamic content


rename term "protein body"

GO:0042735 protein body
Definition
A membrane-bounded plant organelle found in the developing endosperm, contains storage proteins.
-> rename to endosperm protein body

@tberardini OK?

ping @tberardini

ok

Just for fun, I asked ChatGPT :). For this case, since the def specifically refers to the 'endosperm', renaming is sound.


Re-add broken fade arg to BsAlert component

Looks like this PR removed the fade property from the bs-alert component. In the current version, whatever you set fade to, it will always be false and the 'fade' class is never added.
The alt approach to fix this is to just change this.fade to this.args.fade in the template, but I went this way because of the documented default of true that is still in the js file.

@gfmartinez I merged this as-is now, as I want to cut a release soon, and wanted to have this fix in, even without a test. Should you still find the time to add a test in a follow-up PR, this would be highly appreciated! Thanks again!

@simonihmig Thanks! Work just got crazy this week. Will aim to followup asap.


Invalid username persisted in Auth example

In the Authentication example:

place an invalid username/password into the login form
Let the sign in operation fail
Restart the app,
The invalid username is now persisted as it's shown in the "Sign in with [username]" button

@dfeinzimer Please re-test this with latest changes.

This looks fixed, thank you @njarecha!


Remove unused code

@mikekatz please review

Current coverage is 42.73%

Merging #655 into master will increase coverage by +19.90% as of 596a268

@@            master    #655   diff @@
======================================
  Files          752     418     -334
  Stmts        74338   32602   -41736
  Branches         0       0
  Methods          0
======================================
- Hit          16975   13934    -3041
  Partial          0       0
+ Missed       57363   18668   -38695

Review entire Coverage Diff as of 596a268

Uncovered Suggestions

+1.04% via ...ableViewController.m#637...973
+0.50% via ...deosViewController.m#1265...1424
+0.47% via ...ideoPlayerControls.m#1416...1568
See 7 more...

Powered by Codecov. Updated on successful CI builds.

:+1: nice increase to coverage.


I can't verify the zkSync Era contract.

I am using: yarn hardhat verify --network zkSyncMainnet [contract address]
However, I am getting an error:
Error: Axios error (code: ERR_BAD_REQUEST) during the contract verification request. Reason: There is no deployed contract on this address.

bump. getting the same issue here with version 0.1.7

Hi @zkLaunch-dev @michecode , we released a new version of the plugin - 0.1.8. Please try to use that one and see if the error persists.
If you still can not verify your contract, please provide as much context as possible about your project (contract address, hardhat config, contract code, ...) so we can try to help you solve the issue.

Hey everyone! I'm still having this issue with hardhat-zksync 1.2.0
Unfortunately, the repo is private but I can provide an access if required.

Hello, thanks for the reporting can you give us some access so we can try.

Ah, @kiriyaga-txfusion, never mind, I forgot to add verifyURL to the network config 😅


Remove unneeded array-flatten dependency

Back in 2015, express had its own ~9 line flatten() function, which got replaced by an external library in #2682 and #2624
That was just before the left-pad fiasco in 2016, when dependencies were a magical source of free code that someone else maintains who never gets hacked, bought-off, or becomes malicious ;)
https://owasp.org/Top10/A06_2021-Vulnerable_and_Outdated_Components/
https://en.wikipedia.org/wiki/Npm_(software)#Notable_breakages
This PR restores the old flatten function (we still have the test!) and removes the need for the external array-flatten dependency.

Hello, and thank you for your pull request. The ecosystem has changed a lot, as well as the controls npm has. Our philosophy is yo move to more usage of smaller packages, and to remove these util code blocks from source.

The array-flatten module is authored and maintained by an Express.js TC member.


Block Layout: Text Alignment (Left, Right, Center)

Context of use: Change Block Layout
Primary Actor: User
Triggers: User selects text
Main Success Scenario:

[ ] 1. The user selects some text
[ ] 2. The system displays a popup above the selected text (see screenshot below)
[ ] 3. The user selects the Text Alignment dropdown to open the options
[ ] 4. The option of the current text alignment of this paragraph is highlighted (left in the example below)
[ ] 5. and chooses right
[ ] 6. The whole paragraph is rendered right-aligned, the popup and the selected text is removed
[ ] 6a. If the user selects center the paragraph is centered

Screenshots:

This looks like a product ticket rather than a package ticket. super_editor might be missing some features that are needed for this goal, but I don't think we want to bake in a floating menu of any kind.
@salihgueler let's figure out what Superlist needs super_editor to do so that Superlist can achieve this product goal.

Ignore my previous comment. This is a ticket for the example app.


chore: change pathPrefix from /ModuleSite to /preview

Let's close this as I don't think we want to bother with changing the prefix now just for a couple of weeks before we make the switch to the new site completely.
The question where to place configuration and metadata remains, though. We can pick that up whenever it bothers us again.


[Shared/Android/iOS/UWP/.NET][Markdown] Whitespace

What platform is your feature request related to? (Delete other platforms).

[ ] .NET WPF
[ ] Android
[ ] iOS
[ ] UWP
[ ] Shared

Is your feature request related to a problem? Please describe.

Update the shared model markdown parser to handle whitespace characters. TBA for detailed spec.
Investigate if all listed platforms will honor parsed HTML by the markdown parser.

Closing per triage


Things are still being hidden with a non-clip method

There are references to an older method for visually hiding things off the screen. We switched to using the clip method a while back, so these should be investigated and brought into line.
.ir { text-indent: -999em; }
.visually-hidden, .visuallyhidden { left: -9999em; }
.skiplink { left: -9999em; }
All of these things are happening in this file:
govuk_template/source/assets/stylesheets/_accessibility.scss

Following the launch of the GOV.UK Design System, GOV.UK Template will now only get major bug fixes and security patches, so I'm going to close this.


Support etl-to-synbiohub-pipeline

etl-to-synbiohub-pipeline requires SBOL extensions, and may require other features that are not yet implemented in this sbol module.

etl-to-synbiohub-pipeline support is complete as of v1.0b5.
There is still a failing unit test but the issue is with etl-to-synbiohub-pipeline and the same test fails with pySBOL as well. Here is the failure:
======================================================================
ERROR: test_revisions (test_sbh_submissions.TestRevisionManagement)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/tommy/sd2/etl-to-synbiohub-pipeline/tests/test_sbh_submissions.py", line 134, in test_revisions
    local_file = etl_to_sbh.download_from_agave(agave_uri)
TypeError: download_from_agave() missing 1 required positional argument: 'agave_client'

----------------------------------------------------------------------


Add support for prometheus

This is blocked on https://github.com/prometheus/procfs/pull/553 being merged and released.

Build is expected to fail as I've committed this with a rewrite to a local checkout of procfs that has the required changes. I'll remove that rewrite when a release is available. After all that is done, I'll unmark this as a draft.


Elasticsearch helm install failing in k8s 1.24.3 due to need for privileged access

Chart version: 7.16.2
Kubernetes version: 1.24.3
Kubernetes provider: GKE
Helm Version: v3.8.0
Describe the bug: Elasticsearch 7.16.2 helm install from https://helm.elastic.co fails due to lack privileged access. Initcontainer that increases the max_map_count to 262144 is failing to run as root user as k8s 1.24.3 has restricted podSecurityPolicy
Steps to reproduce:

Make sure that you are using kubernetes 1.24.3 or higher
Follow the instructions in ReadMe to install a released version of Elasticsearch
Follow the installation of StatefulSet for elasticsearch-master
container creation fails for "configure-sysctl" with need for escalated access needed

Expected behavior: Elasticsearch installation fails
Provide logs and/or server output (if relevant): Should throw the following error
Event  : elasticsearch-master   StatefulSet   elasticsearch-master   datahub   create Pod elasticsearch-master-2 in StatefulSet elasticsearch-master failed error: admission webhook "pod-security-webhook.kubernetes.io" denied the request: pods "elasticsearch-master-2" is forbidden: violates PodSecurity "restricted:latest": privileged (container "configure-sysctl" must not set securityContext.privileged=true), allowPrivilegeEscalation != false (container "configure-sysctl" must set securityContext.allowPrivilegeEscalation=false), unrestricted capabilities (container "configure-sysctl" must set securityContext.capabilities.drop=["ALL"]), runAsNonRoot != true (pod or container "configure-sysctl" must set securityContext.runAsNonRoot=true), runAsUser=0 (container "configure-sysctl" must not set runAsUser=0), seccompProfile (pod or container "configure-sysctl" must set securityContext.seccompProfile.type to "RuntimeDefault" or "Localhost")   FailedCreate

Hey @upendrao,
The sysctl init container can be disabled in K8S environments that don't allow escalated access. You can find an example here: https://github.com/elastic/helm-charts/blob/819b23bafe837c17f16e39f1f1495a729db4ba4c/elasticsearch/examples/openshift/values.yaml#L10-L11

@jmlrt we have security requirement which will not allow to run container as root user. does filebeat container needs to run as root ? or is it possible to make it to run as non root user ?


Add cryptographic primitives to WASM API

Title: Add cryptographic primitives like SHA256 and RSA to the WASM API
Description:
This is a feature request.
It would be nice to have cryptographic functions like SHA256 and RSA (signature creation and validation) in the API.
Currently I can't find these in proxy_wasm_api.h.
Even though it would be perfectly possible to compile a C or C++ implementation of these algorithms to WebAssembly, many implementations like BoringSSL or nettle do this in assembly to utilize operations that are directly offered by the CPU.
(See for example https://github.com/google/boringssl/blob/master/crypto/fipsmodule/sha/asm/sha256-586.pl)
Therefore it would be nice if such an implementation could be used directly from WASM.
For me in particular interesting would be functions to

import an RSA key so that it can be reused between multiple calls
Compute a SHA256 hash
Validate an RSA signature (with or without previous hashing)
Generate an RSA signature (with or without previous hashing)

We are going to use the WASI crypto module going forward.   Assigned to @PiotrSikora for the spec part.  Assign back for the implemenation.


Cherry-Pick -- Remove Contents of StateLevelDB Directories and Not Directories Themselves

[FAB-18191] Remove contents of leveldb dir instead of the dir itself when dropping dbs (#1828)
When dropping a leveldb, remove all contents of the directory instead of the directory itself. This is common code for upgrade-dbs, rebuild, reset, and rollback.
Signed-off-by: Wenjian Qiao<EMAIL_ADDRESS>

/azp run


Adding new destinations for GLMR

This PR adds missing GLMR destinations:

Moonbeam GLMR > Acala
Acala GLMR > Moonbeam, Parallel
Parallel GLMR > Acala

Was temporary removed destinations from Acala

Task linked: CU-2qfa3n8 Adding more XCM for GLMR


Cannot build on rustc 1.15.1

The API of Command::args that watchexec relies upon was introduced in Rust 1.16:

1.15.1 Command::args;
1.16.0 Command::args.

So either update README to reflect this change or revert/modify https://github.com/mattgreen/watchexec/commit/f88f1b36c31a62e4dcf92e8601a823a43846d6ce.

Fixed, thanks.


Plop skips validation if the argument is given in the CLI

Hi. I've noticed that plop skips inquirer validation if the prompt is given in the CLI.
Take the following example. This should always fail
plop.setGenerator('component', {
		description: 'this generates a new common component',
		prompts: [
			{
				type: 'input',
				name: 'name',
				message: 'the name of the component. (PascalCased)',
				validate: () => 'nope',
			},
		],
		actions: [
			() => {
				return 'yay';
			}
		],
	});

As it does.
However when using the shorthand it skips the validation:

It would be nice if it did the validation no matter which way the arguments were entered.

agreed, this is something that we should definitely add. Thanks for calling it out!

fixed in v2.0


Update Changelog

What this PR does / why we need it:
It updates the current change log up to Feast 0.4.3
Does this PR introduce a user-facing change?:

NONE

Ran into the following error during changelog generation
Warning: PR 412 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 407 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 388 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 382 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 380 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 374 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 372 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 370 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 360 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 244 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 241 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 240 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 238 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 236 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 235 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 234 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 232 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 227 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found

This means these PRs were NOT considered. Could be due to the PRs only being found on the non-master branch. Please go through them manually to ensure that they are captured.

Ran into the following error during changelog generation
Warning: PR 412 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 407 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 388 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 382 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 380 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 374 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 372 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 370 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 360 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 244 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 241 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 240 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 238 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 236 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 235 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 234 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 232 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found
Warning: PR 227 merge commit was not found in the release branch or tagged git history and no rebased SHA comment was found

This means these PRs were NOT considered. Could be due to the PRs only being found on the non-master branch. Please go through them manually to ensure that they are captured.

PR 244 and older are merged on 0.3-dev (or older), and was already on 0.3.0. Those can be ignored.
PR 412, 407, 360 are merged in branch 0.3 but not released.
PR 370, 372, 374, 388 are merged into project namespacing branch.

/lgtm


improvements to monorepo developer lifecycle tooling

the biggest change here is to standardize on using invoke to handle dev tasks and repo management tasks, instead of a homegrown combination of poethepoet (originally chosen for its simplicity and deep integration with poetry, which we no longer use), some custom typer scripts in the tools directory meant to be called by poethepoet, and a single bash script.

@Kyle-Fozo plz review :]

there we go, fixed my git client autosign config, GPG-signed and force-pushed all existing commits in this branch


MainActivityTest failing

com.google.android.gms.location.sample.locationupdates.MainActivityTest > testUsingMockLocation[Nexus 4 - 5.1.1] FAILED
    junit.framework.AssertionFailedError: expected:<51.106829> but was:<90.0>
    at com.google.android.gms.location.sample.locationupdates.MainActivityTest.testUsingMockLocation(MainActivityTest.java:92)
:app:connectedAndroidTest FAILED

Actually it seems like I can't receive mock locations while using FusedLocationProviderApi at all. I pulled this repo to check what I'm doing wrong.
Btw it seems like assertEquals parameters in said test are in wrong order (first actual, then expected, should be the other way around).

The assertEquals params were in the wrong order. Thanks for pointing out. Now fixed. Also made a few changes to the test. Can you see if the test runs (and passes) for you? Also, can you tell me what version of Google Play Services you are using? And what build of Android? I'd like to be able to replicate what you found.

I just cloned the repo again, exactly the same error. The only steps I did were git clone, gradle build and gradle connectedCheck. Allow mock locations in developer options are selected.
I have HTC One M8 phone (Android 5.0.1, HTC Sense 6.0), Google Play Services 8.1.15 (2250156-238), although on older Play Services version the test was failing the same.

Hey,
I see that you have removed the test from the repo (https://github.com/googlesamples/android-play-location/commit/f01b254b87dc61a8dedca6e8a46b881d8ad7dac4) because it was flaky.
Yet still when I have to set mock location using new fused location provider client, after setting mock mode, adding my application as mock location app in developer setting, and setting mock location, nothing is received by the listener. Google Play Services 11.4.2. Do you have any idea whether the functionality is even supported, and how to use it? Otherwise I suppose you can close the issue, as it's no longer relevant, since the test is removed

As for "nothing is received by the listener", that could be because you have a static location after using the mock location, and there are no new updates to report.

that could be because you have a static location after using the mock location, and there are no new updates to report

Shouldn't the new mock location be received, though? I understand that setting mock mode disables all other providers, but I don't get any updates, including those I set as mock locations


Added console-menu to requirements.txt

Added console-menu to requirements.txt to fix the module missing error.

this isn't used in any documented code as yet.

Moved code that used this to another branch


How to unit test a component that uses the markdown-to-html helper?

The default tests throws:
Error: Assertion Failed: A helper named 'markdown-to-html' could not be found

Adding to the test:
needs: ['helper:markdown-to-html']

Results in:
beforeEach failed on it renders: Attempting to register an unknown factory: `helper:markdown-to-html`

This is not an ember-cli-showdown problem per se, but anyone using it migh already now how to do it.

needs: ['component:markdown-to-html']


Bindings on CommandBar not working

After I changed to the MyToolkit MtPage my binding stop working, when checking the DataContext of the buttons they are all null, and even setting the DataContext directly on the CommandBar doesn't seam to fix.

Do you set the DataContext directly on the MtPage DataContext property?

Yes

Ok, automatically copying the DataContext from the MtPage to the DataContext of the BottomAppBar and TopAppBar should work then?

Something strange is happening
Even using a relative binding on the command bar ( DataContext="{Binding DataContext,ElementName=RootGrid}") it also doesn't work and should.
Just setting the DataContext on the CommandBar also doesn't work. The icons seam to never receive the update

The problem is that internally the class uses the original Page class and the app bars are "copied" over... Currently the resources from the MtPage are also copied to the app bars... Currently the datacontext is not copied and thus the bindings do not work... I think relative bindings will never work. I have to reproduce the problem so that i can fix this. Will inform you as soon as i have more information...

Sure until then the workaround the what I posted earlier. Not pretty but works for now.

What is your working workaround?

Manually setting the datacontext on each button
foreach (AppBarButton source in AppBar.PrimaryCommands.Union(AppBar.SecondaryCommands))
            {
                source.DataContext = pageRoot.DataContext;
            }

Maybe this is not working at all?
http://stackoverflow.com/questions/25753649/how-to-set-appbar-datacontext
Is it working when setting the DataContext of the default Page class?

Hum probably.
But before I switched to the MtPages I was using the regular binding and everything worked fine.

Should now be fixed in NuGet version 2.5.4, please update and report if it works...

I confirm, you can close the issue


Issue with webhook callback

Hi
We are having an issue with the webhook call back from Reepay after upgrading til vendr 1.4.1, it seems like it doesn't recognize the "Invoice" parameter of the body as orderNumber, which it did earlier. Is there a workaround, do we need to downgrade or is this something that will be fixed soon?
The stack trace:
{     "message": "An error has occurred.",     "exceptionMessage": "Value cannot be null.\r\nParameter name: orderNumber",     "exceptionType": "System.ArgumentNullException",     "stackTrace": "   at Vendr.Core.AssertionExtensions.MustNotBeNull[T](T param, String paramName)\r\n   at Vendr.Core.AssertionExtensions.MustNotBeNullOrEmpty(String param, String paramName)\r\n   at Vendr.Core.Web.PaymentProviders.OrderReference..ctor(Guid orderId, String orderNumber)\r\n   at Vendr.Core.Web.OrderReferenceExtensions.GenerateOrderReference(OrderReadOnly order)\r\n   at Vendr.Core.Web.Controllers.VendrPaymentController.Callback(IPaymentProvider paymentProvider, OrderReference orderReference)\r\n   at Vendr.Core.Web.Controllers.VendrPaymentController.HandleRequestWithoutOrder(String paymentProviderAlias, Guid paymentMethodId, Func3 handler)\r\n   at Vendr.Core.Web.Controllers.VendrPaymentController.Callback(String vendrPaymentProviderAlias, Guid vendrPaymentMethodId)\r\n   at lambda_method(Closure , Object , Object[] )\r\n   at System.Web.Http.Controllers.ReflectedHttpActionDescriptor.ActionExecutor.<>c__DisplayClass6_2.b__2(Object instance, Object[] methodParameters)\r\n   at System.Web.Http.Controllers.ReflectedHttpActionDescriptor.ActionExecutor.Execute(Object instance, Object[] arguments)\r\n   at System.Web.Http.Controllers.ReflectedHttpActionDescriptor.ExecuteAsync(HttpControllerContext controllerContext, IDictionary2 arguments, CancellationToken cancellationToken)\r\n--- End of stack trace from previous location where exception was thrown ---\r\n   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at System.Web.Http.Controllers.ApiControllerActionInvoker.<InvokeActionAsyncCore>d__1.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at System.Web.Http.Controllers.ActionFilterResult.<ExecuteAsync>d__5.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at System.Web.Http.Filters.AuthorizationFilterAttribute.<ExecuteAuthorizationFilterAsyncCore>d__3.MoveNext()\r\n--- End of stack trace from previous location where exception was thrown ---\r\n   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)\r\n   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)\r\n   at System.Web.Http.Dispatcher.HttpControllerDispatcher.<SendAsync>d__15.MoveNext()" }

@larsnikander thanks for reporting - sounds strange it should be affected by an upgrade to Vendr v1.4.1 though.
Which version of Vendr was it running on previous?
Could you check if the payment has these meta data?
I recall these are stored an an JSON object in Reepay:
{
   "orderReference":  "",
   "orderId": "",
   "orderNumber": ""
}

https://github.com/vendrcontrib/vendr-payment-provider-reepay/blob/dev/src/Vendr.Contrib.PaymentProviders.Reepay/ReepayCheckoutPaymentProvider.cs#L71-L76
Do you see any error regarding payments in checkout or only in callback from webhook event?

Hi Bjarne
I can't find the metadata in the json object on reepay no, so this might be the issue.
On reviewing our git I also see that we actually added Reepay after upgrading to 1.4.1 where the webhooks were correctly triggered, so my guess now is that we have a wrong configuration somewhere. The data sent from Reepay is the same back when the webhooks were working, but I did not check the reepay json model back then.

You should be able to find the metadata under an invoice (payment) via the Actions > Metadata

Okay, it does seem like it has the metadata then on the ones that are failing and the orderNumber does seem to match the ones that we are trying to finalize. But I found an interesting behaviour which I think is due to how we have designed the checkout, which might have caused the issue.
We have a detached headless cms instance which is responsible for 3 sites, of which this shop is one of them. This means that we have to interface with vendr and umbraco through api calls which are mostly timed by the frontend. In the final call where we generate the payment form we also save a few things to the order, to pick up any last minute changes. I think this design is the culprit, because saving the order does seem to run asynchronous and I think the order might get a new ordernumber during or after the form has been generated.
I think I have managed to change it now so it's reliable by splitting the two processes up so orders aren't saved during the generation of the payment form.

Oh, that sounds like an interesting use-case and probably tricky one as well :)
Great you managed to track down the culprit and hopefully it will work going forward.
Let me know if there are something else not working as expected or something which need to be enhanced or feel free to fork the project or submit a PR 👍
Furthermore I am sure @mattbrailsford would be interested in knowing about this specific site using Vendr as commerce. I know he plans to add a "Cases" section to https://vendr.net
Also it would be interesting to know more about which payment providers are the most popular ones used in stores based on Vendr 😁 🚀


Feature: Use spread operator to pass down props.

Hello,
I propose adding a feature that allows the use of ES6 spread operator to destructure an object into props passed into a component.  This feature idea comes from the use of it in react, which is documented here.  https://facebook.github.io/react/docs/jsx-in-depth.html#spread-attributes
Imagine an ActorCard that displays a users firstName, lastName, email and imageUrl picture.
Currently
<ActorCard
  :firstName="currentUser.firstName"
  :lastName="currentUser.lastName"
  :email="currentUser.email"
  :imageUrl="currentUser.imageUrl"
>
</ActorCard>

//or by passing object
<ActorCard :user="currentUser"></ActorCard>

Using spread would allow to pass flat props to the component without all the boiler plate of manually specifying each prop.
<ActorCard {{...currentUser}}></ActorCard>

would compile into the first example above

Doing this way encourages keeping props flat, which ultimately makes components more reusable (i.e. i don't have to construct an object that has property of firstName and lastName just to reuse this actor card if i go the easy route and just pass in a user.)
If this is something that is agreeable, i'm happy and willing to put in the work to get a PR for it.
Thanks!
Austin

You can already do this with v-bind="currentUser".

Awesome, thanks!

Can you use .sync this way?

Ends up the answer is yes!

the v-bind="currentUser" works for props / data binding.
And is there a way to pass down event binding / directive props
like
<custom-comp
  :subProps="{
      prop1: 1, prop2: 2,
      'v-on:click': function($e){ console.log($e) },
      'v-custom-directive': true
   }"
/>

custom-comp definition
<template>
<div class='custom-comp'>
  <input
    type='text'
    name='name'
    v-bind='subProps'
  />
</div>
</template>

but the v-on:click & v-custom-directive will not work, only results
<input data-v-2a851830="" type="text" name="name" prop1="1" prop2="2" v-on:click="function($e) {
          _vm.console.log($e)
        }" v-custom-directive="true">

the directive is rendered as attributes but not directives

@esbenam not yet with templates
if you are tired of writing render function, you can try vue jsx
https://github.com/vuejs/babel-plugin-transform-vue-jsx#difference-from-react-jsx
it supports object spread pattern and express vue terminologies (e.g directives / events) in a prop way

@magicdawn yeah, that is what I have ended up doing for now.

Just got the same problem, props are rended to attributes, not directives.
Seems jsx will work.

how to use sync
    currentUser: { val: 0, min: 0, max: 2048 ,dynamicMax: 500, dynamicMin: null}    <ActorCard  v-bind="currentUser"  :val,sync= currentUser.val  />
@yyx990803

@caokunyu you can ask questions on the forum, the Discord server or StackOverflow
don't do it on issues please 😛


Allow admins to update user project preferences

Current behavior: Only project owners are allowed to edit the settings key for a user's project preferences. Example usage: see Python Client documentation here for workflow leveling up (i.e., for Gravity Spy and others).
Desired behavior: Also allow Zooniverse admins to edit the settings key.
Potential Solution: Add a check for admin as an or condition to this statement:
https://github.com/zooniverse/panoptes/blob/24a67c6f4a56492a56e65170fad89ab3bda4c1e0/app/controllers/api/v1/project_preferences_controller.rb#LL25C73-L25C73

Closed by #4218


More complete cudf support.

Handles missing value.
Accept all floating point and integer types.
Move to cudf 9.0 API.
Remove requirement on null_count.

@kkraus14

@kkraus14 Just out of curious, the mask from __cuda_array_interface__ is a SimpleNamespace object.  Is that a possible bug?

Added a has_missing parameter to c interface, since there's no way our internal code can know whether the NaN is from our default value or is supplied by user.

Removed check_data, it was my mistake.  Since pandas array are converted into numpy ndarray, so missing value is supported.

@RAMitchell Arbitrary column types supported.

Nice work. I would like to see a test with cudf with multiple data types.

@RAMitchell I have a Python test for that.  But a c++ test seems can be more specific

@trivialfis Sorry for the delay. I'm trying to find a way to provision P3

@hcho3 Take your time.  I'm doing a lots of c++ tests so things should be relatively safe.

@RAMitchell More tests added.

Codecov Report

Merging #4850 into master will increase coverage by 0.01%.
The diff coverage is 18.18%.

@@            Coverage Diff             @@
##           master    #4850      +/-   ##
==========================================
+ Coverage   77.51%   77.53%   +0.01%
==========================================
  Files          11       11
  Lines        2055     2039      -16
==========================================
- Hits         1593     1581      -12
+ Misses        462      458       -4

Impacted Files
Coverage Δ

python-package/xgboost/core.py
76.06% <18.18%> (+0.02%)
: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 c89bcc4...63215f4. Read the comment docs.


Add Support for Compact mode (js-Controller 2.0 Feature)

From @ldittmar81 on January 20, 2019 17:57
The upcoming version 2.0.0 of js-controller will support a new mode for low memory devices. For this, a few minor adjustments to the adapter must be made. Please follow these instructions: https://forum.iobroker.net/viewtopic.php?f=24&t=20387#p213466
Copied from original issue: soef/ioBroker.yamaha#20

should be fixed in 0.4.0


[azservicebus] Possibility for setting retryOptions when creating a new sender

I encounter a problem if the sender has been idle for > 10 minutes, then the link recovery and message sending will take ~5 seconds, which is a long time. My suspect is that the default retryDelay is set to 4 seconds. NewSenderOptions is currently empty, it should be possible to set retryOptions when creating a new sender.

I see that this issue is solved in #16831


De-hardcode ConwayEra, parametrise & use IsBabbageBasedEra / IsConwayBasedEra where applicable

As in the title.
I'd like to use Convex in my protocol, I think it's a cool library, but I don't feel like hardcoding ConwayEra everywhere where I use Convex tools.

I'll have a look - but in my experience, as soon as the new era lands we don't care about the old eras anymore. Yes there's a transition period but after that it's okay to have a single hard-coded era.

I have a fork of Convex and it took me about 2 days to era generalise it. It comes handy when testing hard forks in a transitional period. In Axo we had the same codebase running in parallel for 2 months on preview / preprod Conway and prod Babbage. But I understand if that’s too much of a hassle.

I agree that era transitions are a concern - I actually had a branch of sc-tools that we used for the transition of Djed from babbage to conway era (here) I just wasn't sure if the added complexity is worth it in the majority of time where there is no era transition to be expected.
Is there some code you could share from your work on generalising it?

It's part of a private multirepo so I don't think I can share only a folder in GitHub. I've made a zip.
sc-tools.zip

Thanks for the file. I finished the changes, just need to get the tests to pass.  PR here: #211

Closed in #211