Link from Daily Desk to a note in a folder doesn't work

Hello 👋
I would like to link a note stored in a folder, and the system seems to fail and instead of creating a link, it creates a new note at the root level. See in video
https://github.com/user-attachments/assets/42fc6458-aa76-4521-a214-53d2657e58dd
If you need more information, let me know

Is there a space after the folder names? I'm guessing that's causing the issue?
Can you try removing the space after sociaux folder and patron?

Is there a space after the folder names? I'm guessing that's causing the issue?
Can you try removing the space after sociaux folder and patron?

I confirmed, we had a space in each folder name, at the end. I removed them directly (no refresh), and it works now.
If you want me to do more test, let me know. If you want to close the issue, feel free to do so.

Nope that should be. Glad it works.
Thanks for reporting the bug! Hope you are enjoying the app!


Details abaout packagist mirror alternatives

More information about packagist mirror creation and alternatives

Thank you for contributing! 👍


flask-ask_Zappa (Alexa skill) internal server error

Zappa returns "Internal server error" after setting up an Alexa skill (works with ngrok)
Context
I tried to load up an Alexa skill to a server by using Zappa and flask-ask. The Alexa skill works with ngrok. After installing Zappa by using this guide from John Wheeler, everything works but if I enter the deploy-link to the browser it says "{"message": "Internal server error"}".
Therefore, I thought there should be some issues in the python code which can't be handeled by Zappa, or in selection the region at aws configure (I'm from Germany). The Alexa skill returns headlines from a website.
Code:
`
from flask_ask import Ask, statement, question
import requests
from bs4 import BeautifulSoup
import logging
from flask import Flask
import unidecode
import time
import json
import sys
reload(sys)
sys.setdefaultencoding('utf-8')
app = Flask(name)
ask = Ask(app, "/")
logging.getLogger("flask_ask").setLevel(logging.DEBUG)
webpage = requests.get("http://www.wochenblatt.de/nachrichten/straubing/")
soup = BeautifulSoup(requests.get("http://www.wochenblatt.de/nachrichten/straubing/").text, "html.parser")
articles = []
for story_heading in soup.find_all(class_="h3 cStandard"):
articles.append(story_heading.text.replace('\n', '').replace('  ', ''))
@app.route('/')
def homepage():
return "Hallo?"
@ask.launch
def start_skill():
welcome_message = 'Hallo bei Wochenblatt Straubing, moechtest du die Nachrichten hoeren?'
return question(welcome_message)
@ask.intent("YesIntent")
def share_headlines():
headlines = articles
headline_msg = 'Die aktuellen Nachrichten sind {}'.format(str.join(".....   ",articles))
return statement(headline_msg)
@ask.intent("NoIntent")
def no_intent():
bye_text = 'Tschuess'
return statement(bye_text)
if name == 'main':
app.run(debug=True)
`
Expected Behavior
Upload Alexa Skill (Python + flask-ask) to server by using zappa.
Actual Behavior
Returning "{"message": "Internal server error"}"
Possible Fix
Error in code or selection of region in aws configure
Your Environment

Zappa version used: 42.0
Operating System and Python version: 2.7
Zappa settings: Default
aws configuration:
AWS Access Key ID [None]: ---
AWS Secret Access Key [None]: ---
Default region name [None]: eu-central-1
Default output format [None]:

Can you capture and provide a traceback from the AWS Lambda logs?

@voutilad do you have a pypi login? I'd like to give you release access, which you can use as much or little as u desire (no expectations whatsoever on this end).

@johnwheeler  Sure, it's voutilad

Thanks Dave - I added you as a maintainer so you can push releases as you see fit.

@voutilad thank you for your quick response. Sorry I am quite a beginner in this area and this is my first project. I thought it is only possible to see the AWS Lambda logs by using an AWS Lambda blueprint but not by using Zappa with Lambda. Do you know where I can find it?
When testing the App with Alexa, I get this Error: "There was an error calling the remote endpoint, which returned HTTP 502 : Bad Gateway".
I noticed that I used Python 3.6 instead of 2.7, could this be the issue?
Thank you for your help!, also to @johnwheeler for providing flask-ask. Hope I will get it run soon!

Finally I got the AWS Lambda logs but is there maybe another way to release the Alexa skill in Python without using Zappa? Does it work with a ngrok solution somehow?


Feature/fine grained sec groups

Requirement is to enable the creation of security groups for specific ports at the time of service instance creation. Currently, application security groups are created for entire port range of 1024-65k, exposing service instance components to attack initiated from inside the bound apps.

Modified the flow of building rules for security group creation to consume relevant exposed port information from services catalog.

Tested on dev landscape for blueprint service and will run through the dev pipeline once catalogs of other services are also updated.

This change does not take care of update of application security groups of existing instances.

Pull Request Test Coverage Report for Build 1291

6 of 6 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 89.018%

Totals

Change from base Build 1220:
0.01%

Covered Lines:
6939

Relevant Lines:
7554

💛  - Coveralls

@kamath-prasad @abh1kg  I think modifying update instance flow inside broker to incorporate this will involve extra calls to CC and overall logic will also get messy. So maybe, if needed, we can do this update as a one time thing?

@abh1kg / @jayeshjk : Ok , then let us avoid update of existing rules.

@jayeshjk  : please check if you need to rebase your branch as the code coverage looks reduced now.


List select expands selection

This PR:

Changes the list.select command such that it expands the selection, and not to set the selection to 1 element like before
Changes that the list fires the open event for all the newly selected elements (not just one)
Adopts the explorer to react to this, thus fixing #42573

fixes #42573

Closing as this work is not that good.


Question: How to handle other environments?

Hi~. You said that "Put a .env file, that must never be committed" in your article.
Then how do you handle other environments like production, test, etc??
If each branch has different .env and not committed, then how do other members know .env info? and how do you deploy them?

logically, a .env-sample file could be committed showing the variables that should be set in real environments.

There should be a list of required .env keys (without values) as a courtesy. When we're making changes to the .env we should be aware to document which keys we've added or subtracted whether it be on the readme or a .env-sample as @devskope suggests. For deployment, it depends upon the hosting, usually it's a matter of supplying our production .env for our application to use. Keep your dev/prod .env in a secure place at all times and far away from github.

@jackson-elfers @devskope I see. Thanks for your reply~:D


Improve docs to be more explanatory

Why:
Fixes #974

What's being changed:

Added {:copy} snippet to all code snippets
Improved Dockerfile snippet to make scripts executable here

# Container image that runs your code
FROM alpine:3.10
# Copies your code file from your action repository to the filesystem path `/` of the container
COPY entrypoint.sh /entrypoint.sh
# Makes your code file executable
RUN chmod +x entrypoint.sh
# Code file to execute when the docker container starts up (`entrypoint.sh`)
ENTRYPOINT ["/entrypoint.sh"]

Removed a step here because it was unclear, this step is now covered by the previous change,

Check off the following:

[x] All of the tests are passing.
[x] I have reviewed my changes in staging.
[x] For content changes, I have reviewed the localization checklist
[x] For content changes, I have reviewed the Content style guide for GitHub Docs.

@avinal Thanks so much for opening a PR! I'll get this triaged for review 🌟

Is there more to add to this pr?

Is there more to add to this pr?

Azure

Organizer

There's nothing you need to add @avinal! Thanks for your patience as our small team works through all of the amazing contributions 🌟

Thanks, @janiceilene for confirmation.

<EMAIL_ADDRESS>

Thanks for this PR! 🎉  I'm just checking this recommendation with our support folks.

Hi @avinal 👋
So sorry about this, but we're having some unrelated backend issues with this PR. Would you mind re-raising these awesome changes as a new PR (without deleting the forked repository)? 🙇
For the new PR, I've also included some feedback about chmod +x entrypoint.sh in a separate comment for you to consider.
Thank you!

Hello @martin389,
No worries, I will submit a PR again :) Since this PR is almost a month old, unknowingly I deleted the fork.  May I fork again and then redo the changes? If this doesn't create any problem.

For the new PR, I've also included some feedback about chmod +x entrypoint.sh in a separate comment for you to consider.

I have added my feedback on this, please resolve that issue and let me know what is decided.
Thanks !


Benchmark obs parsing + faster parser

This PR adresses the current bottleneck of prx's speed, which is observation file parsing, by adding

a benchmark for parsing speed by splitting a 24h, 30s obs interval file into a sequence of smaller files and recording the parsing time
a faster rinex 3.05 obs parser making heavy use of pandas
a unit test comparing georinx parser outputs to those of the new prx parser

Note that the PR adds the new parser, but does not enable it, so prx still uses georinex as its parsing backend. We'll probably want to add more testing before enabling it.
What about the loss-of-lock indicator
The loss-of-lock indicators are not extracted yet, will deal with that in a follow-up PR.
What about nav files?
We can continue to rely on georinex's nav file parser, since parsing e.g. a 1-day file takes only about 45 seconds (M2 Mac), and most web-prx users will benefit from caching, since it is likely that a previous users has caused prx to download and parse the same nav files.
Testing
Running the benchmark yields

Note that the pandas-based parser's speed is pretty much independent of file size, at least in this benchmark.

@plutonheaven OK if I merge this?


PHP: don't destruct persistent channel at the end of request under php-fpm mode

Testing the memory usage in App Engine.

Seems works fine regarding to the memory usage.
It can solve the severe issues that persistent channel not working under php-fpm mode by not deleting it at the end of each request.
It works fine because within the same php script, the key([target] + [channel arguments] + [credentials]) are always the same. Thus the new channel is not created very often.


Fixed a check in validations.py

Closes: #1
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[x] Appropriate changes to README are included in PR

@googlebot I signed it!


how to access json data received in url call

Below is the code to generate barchart using C3 js library
var chart = c3.generate({
    bindto: '#barchart',
    data: {
      url: './API/get_bar_chart_data.php',
      mimeType: 'json',
      type: 'bar',
      keys: {
             x: 'category', // it's possible to specify 'x' when category axis
             value: [ 'number']
            },
      onclick: function (d, i) {

                              console.log(chart.categories()[d.index]);

                              onclick_barchart();
                              }
      },
  bar: {
        width: {
            ratio: 0.5 // this makes bar width 50% of length between ticks
               }
       },
    axis: {
        x: {
            type: 'category' // this needed to load string x value
        }
    }

});

In the above code I am getting data from URL in json format.
Now on click I want to pass this complete json received to onclick_barchart() function.

Hi,
The issue queue is to be used for reporting defects and problems with C3.js, in addition to feature requests and ideas.
It is not a catch-all support forum.
For general support enquiries, please use the Google Group at https://groups.google.com/forum/#!forum/c3js.
All questions involving the interplay between C3.js and any other library (such as AngularJS) should be posted there first!
Thank you!


Documentation : example

In the documentation for example application, under Set Up instructions, shouldn't it be pushd example/bin/ instead of pushd bin/

When I wrote it, I was assuming you were already in the example directory since the README is there - did that not track for you?

But before that you said
$ pushd example/config/
$ cp ./monorail_rack.cfg.example ./monorail_rack.cfg
$ popd

so if you follow these step by step, it doesn't work.. not a big deal.. it's minor issue..thought of letting you know.

yep, I see what you mean - will cover it in a PR that I'm doing right now


Clear old scripts

I don't think we need:

json2xml-localFiles.xsl
pipe2-formats.xsl (I should check once more to make sure this isn't called from the new ...-all.xsl or ...-core.xsl, but I don't think it is...)
xml2json.xsl

The following is only needed to output JSON-in-XML for RDA profile only for further transformation into HTML:

json2xml.xsl

Top 3 in your list can be deleted, yes. If we're deleting, why keep anything but formatSpecificProfiles-all.xsl, jsonXmlProfile2HtmlProfile.xsl and json2xml.xsl? Aren't those the only ones we use? If we want to resuscitate any scripts, pull them out of an old commit?

I think the scripts directory is in pretty good shape for the time being.


Create schema

Potential values to include:
id
plant_name
zones (zones it can grow in)
light (sunlight required)
space (space needed to grow)


refactor: Decouple backend from Repository

It is a first step to have multiple repository backends.

Codecov Report

Merging #13 into master will decrease coverage by 0.1%.
The diff coverage is 88.67%.

@@            Coverage Diff             @@
##           master      #13      +/-   ##
==========================================
- Coverage   96.62%   96.52%   -0.11%
==========================================
  Files          13       14       +1
  Lines         326      345      +19
==========================================
+ Hits          315      333      +18
- Misses         11       12       +1

Impacted Files
Coverage Δ

tests/integration/test_services.py
100% <100%> (ø)
:arrow_up:

backend/services.py
95.23% <100%> (-0.22%)
:arrow_down:

backend/api.py
93.22% <100%> (-0.12%)
:arrow_down:

backend/storage/vendors/postgres.py
80% <80%> (ø)

backend/storage/repository.py
92.3% <85.71%> (+12.3%)
: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 87cc943...6505c02. Read the comment docs.


Thickness of axis on Safari 9 is thicker than other browsers

Thickness of axis on Safari 9 is thicker than other browsers. And renders differently on different displays.
(See Wade for details.)

Browser bug. Cannot fix. Might want to try to open a bug on Safari if that is possible.

With release of Safari 11 last month, we only need to support Safari 10 & 11.


Remove Target type from the code

Uses of type Target and friends are:

one unnecessary internal use in Rewrite/Axiom/EvaluationStrategy.hs (pending removal by Serokell at the time of writing)
one unused export in Rewrite/Step.hs
Rewrite/Step.hs:315:toConfigurationVariablesCondition ::
unused exported function in Internal/OrPattern.hs
Internal/OrPattern.hs248:targetBinder ::

Remove Target and see what breaks.


Error "A task was canceled" in Dashboard.

I connected my bot to Skype, Slack and Facebook Messenger. I was able to go through my dialog once in each of those applications, and then after session.endDialog() the bot didn't respond to anything and the error status appeared in the Dashboard.
What could be the cause of this?

Anyone? Please? :(

Anyone? Please? :(

Can you share you code at all?

Alright, so I have one dialog with a long waterfall of Prompts.
bot.add('/', function (session) {
    session.beginDialog('/doDialog');
});

bot.add('/doDialog', [
    function (session) {
            builder.Prompts.choice(session, 'Choose stuff!', ['This', 'That']);
    },
....

It goes like this for a while, just Prompts after Prompts. When I reach the end of the dialog, the bot stops responding to anything and the error appears in the dashboard. I tried both session.endDialog(), session.reset() at the end of the dialog, didn't help at all.
Am I doing something wrong? I though when the dialog is over we're just back to where we started.

Is that really your root dialog code? If so it has a problem. When you call endDialog() from the second dialog your root dialogs handler is going to just get called again which would send you right back into /doDialog again. This is a trap that people seem to keep falling into so I'm thinking about making a breaking change that prevents that in a future release. In the short term you can avoid the trap by turning your root dialog into a single step waterfall. So just add array brackets around it.
If you're not comfortable sharing your actual code I understand but I really need to see the whole thing to tell you whats wrong. You can email it to me too at<EMAIL_ADDRESS>Another thing you might try is to make your second /doDialog the root dialog. If things start working that will help narrow down the issue.

Any progress on this?

Closing this issue. Please open a new issue for further discussion if necessary. Thanks!


Setting isEncrypted:false in notify: verb doesn't actually set it to false.

In the above image, I sent notification with isEncrypted:false
Then in the receiving notification has isEncrypted:true

Intended behaviour:
Running notify:isEncrypted:false:<...>
Should have a receiving notification of:
notification:{..., "isEncrypted":false, ...}

P1 to look at it, then reevaluate the priority of the underlying issue.

Could not get to this in the previous sprint, will be taking this up first in PR88.

Hi @srieteja I've been looking at this as it came up in another context. I'll pick it up this sprint.
Summary of the current behaviour for 'update' notifications

at_secondary_server NotifyVerbHandler._getIsEncrypted ignores whatever it has been sent and always sets isEncrypted to 'true'
at_client NotificationServiceImpl.notify always encrypts the value but does not set isEncrypted in the metadata. Thus, usually (unless app code explicitly sets metadata.isEncrypted to true), metadata.isEncrypted is apparently false (default value) but the value has in fact been encrypted
at_commons NotifyVerbBuilder.buildCommand only includes isEncrypted in the generated command if metadata.isEncrypted is true (which it ~never is, right now)
at_secondary_server ResourceManager.prepareNotifyCommandBody only includes isEncrypted in the generated command if isEncrypted is true

Summary of the changes I have in mind:

at_secondary_server: these are the most important changes. Once implemented, the problem Jeremy encountered will be resolved

NotifyVerbHandler._getIsEncrypted

if a value for isEncrypted was set in the received command, respect it, otherwise default it to 'true' (current behaviour)

ResourceManager.prepareNotifyCommandBody

always include isEncrypted in the generated command

at_client

NotificationServiceImpl.notify

add an optional bool parameter encryptValue default it to true
set the metadata.isEncrypted to the value of encryptValue

NotificationRequestTransformer.transform

if metadata.isEncrypted is true then encrypt the value, otherwise do not

at_commons

NotifyVerbBuilder.buildCommand

always include isEncrypted in the command (currently it is only sent if isEncrypted is true, which it typically never is, right now)

In the distant future, once all of the client-side sdks are doing the right thing with their generated notify commands, we can default isEncrypted to 'false' if it is not set in the command but for now we have to continue to default it to 'true' so we don't break the world.
I need to think a little more about the sequencing of the at_commons and at_client changes; what we want to ensure is that we only ever send a notify command with isEncrypted:false when that is genuinely the case. We will likely need to do an at_commons major version change, although we may be able to do something fiddly in at_client to ensure the right behaviour

See also https://github.com/atsign-foundation/at_client_sdk/issues/1332

at_server part is complete, pending review and merge
at_client part will be completed this sprint

at_server part is complete, have requested review
at_client part will be merged once the at_server part has been rolled out

@JeremyTubongbanua please verify this issue with latest published version of at_client

Sure, will pick this up this sprint @murali-shris

Dart
Summary
I sent a notification with this key (with isEncrypted = true
AtKey sharedRecordID = AtKey()
    ..key = keyName
    ..sharedBy = myAtsign
    ..sharedWith = otherAtsign
    ..namespace = nameSpace
    ..metadata = (Metadata()
      ..isEncrypted = true
      ..ttl = 60000
      ..ttr = -1);

And successfully received it
INFO|2024-10-07 11:31:23.957379| at_notify |message received from @jeremy_0 notification id : d3a8502a-1823-4449-b7f2-541a67276e0d
@jeremy_0: hi
``

**Full logs**

Sender
```dart
// dart packages
import 'dart:io';

// atPlatform packages
import 'package:at_client/at_client.dart';
import 'package:at_cli_commons/at_cli_commons.dart';

// External Packages
import 'package:args/args.dart';

Future<void> main(List<String> args) async {
  ArgParser argsParser = CLIBase.argsParser
    ..addOption('other-atsign',
        abbr: 'o',
        mandatory: true,
        help: 'The atSign we want to communicate with')
    ..addOption('message',
        abbr: 'm', mandatory: true, help: 'The message we want to send');

  late final AtClient atClient;
  late final String nameSpace, myAtsign, otherAtsign, message;
  try {
    var parsed = argsParser.parse(args);
    otherAtsign = parsed['other-atsign'];
    message = parsed['message'];

    CLIBase cliBase =
        await CLIBase.fromCommandLineArgs(args, parser: argsParser);
    atClient = cliBase.atClient;

    nameSpace = atClient.getPreferences()!.namespace!;
    myAtsign = atClient.getCurrentAtSign()!;
  } catch (e) {
    print(argsParser.usage);
    print(e);
    exit(1);
  }

  String keyName = 'message';

  AtKey sharedRecordID = AtKey()
    ..key = keyName
    ..sharedBy = myAtsign
    ..sharedWith = otherAtsign
    ..namespace = nameSpace
    ..metadata = (Metadata()
      ..isEncrypted = true
      ..ttl = 60000
      ..ttr = -1);

  await atClient.notificationService.notify(
      NotificationParams.forUpdate(sharedRecordID, value: message),
      waitForFinalDeliveryStatus: false,
      checkForFinalDeliveryStatus: false);

  // Using notifications we get all the updates
  // for (int a = 1; a < 101; a++) {
  //   await atClient.notificationService.notify(
  //       NotificationParams.forUpdate(sharedRecordID, value: a.toString()),
  //       waitForFinalDeliveryStatus: false,
  //       checkForFinalDeliveryStatus: false);
  // }

  await Future.delayed(Duration(seconds: 1));
  exit(0);
}

Receiver
// dart packages
import 'dart:io';

// atPlatform packages
import 'package:at_client/at_client.dart';
import 'package:at_cli_commons/at_cli_commons.dart';
import 'package:at_utils/at_logger.dart';

// External Packages
import 'package:chalkdart/chalk.dart';

Future<void> main(List<String> args) async {
  late final AtClient atClient;
  late final String nameSpace, myAtsign;
  try {
    CLIBase cliBase = await CLIBase.fromCommandLineArgs(args);
    atClient = cliBase.atClient;

    nameSpace = atClient.getPreferences()!.namespace!;
    myAtsign = atClient.getCurrentAtSign()!;
  } catch (e) {
    print(CLIBase.argsParser.usage);
    print(e);
    exit(1);
  }

  final AtSignLogger logger = AtSignLogger(' at_notify ');

  atClient.notificationService
      .subscribe(regex: 'message.$nameSpace@', shouldDecrypt: true)
      .listen(((notification) async {
    stdout.writeln(chalk.blue(notification.key));
    String keyName = notification.key
        .replaceAll('${notification.to}:', '')
        .replaceAll('.$nameSpace${notification.from}', '');
    if (keyName == 'message') {
      logger.info(
          'message received from ${notification.from} notification id : ${notification.id}');
      var talk = notification.value;
      print(chalk.brightGreen.bold('\r\x1b[K${notification.from}: ') +
          chalk.brightGreen(talk));

      stdout.write(chalk.brightYellow('$myAtsign: '));
    }
  }),
          onError: (e) => logger.severe('Notification Failed:$e'),
          onDone: () => logger.info('Notification listener stopped'));
}

Receiver logs
dart_playground dart run bin/receiver.dart -a @smoothalligator -n dart_playground -v
Connecting ... INFO|2024-10-07 11:31:18.625480|AtLookup|Creating new connection
INFO|2024-10-07 11:31:18.839638|AtLookup|New connection created OK
INFO|2024-10-07 11:31:18.946834|AtLookup|auth success
INFO|2024-10-07 11:31:18.948024|AtClientManager|setCurrentAtSign called with atSign @smoothalligator
INFO|2024-10-07 11:31:18.948202|AtClientManager|Switching atSigns from null to @smoothalligator
INFO|2024-10-07 11:31:18.969193|HiveBase|commit_log_0fe9fb2745a1a4a3f9c43a7dd8ac860e8291d5bafda9f25df26b45a1ad92726e initialized successfully
INFO|2024-10-07 11:31:18.978238|HiveBase|0fe9fb2745a1a4a3f9c43a7dd8ac860e8291d5bafda9f25df26b45a1ad92726e initialized successfully
INFO|2024-10-07 11:31:19.008244|AtClientCommitLogCompaction (@smoothalligator)|Starting commit log compaction job running for every 11 minute(s)
INFO|2024-10-07 11:31:19.024572|AtClientManager|setCurrentAtSign complete
Connected
INFO|2024-10-07 11:31:19.050899|Monitor (@smoothalligator)|starting monitor for @smoothalligator with lastNotificationTime:<PHONE_NUMBER>048
INFO|2024-10-07 11:31:19.296285|Monitor (@smoothalligator)|monitor started for @smoothalligator with last notification time:<PHONE_NUMBER>048
@smoothalligator:message.dart_playground@jeremy_0
INFO|2024-10-07 11:31:23.957379| at_notify |message received from @jeremy_0 notification id : d3a8502a-1823-4449-b7f2-541a67276e0d
@jeremy_0: hi
@smoothalligator:

Sender logs
dart_playground dart run bin/dart_playground.dart -a @jeremy_0 -o @smoothalligator -m hi -n dart_playground -v
Connecting ... INFO|2024-10-07 11:31:23.034396|AtLookup|Creating new connection
INFO|2024-10-07 11:31:23.294716|AtLookup|New connection created OK
INFO|2024-10-07 11:31:23.400184|AtLookup|auth success
INFO|2024-10-07 11:31:23.401164|AtClientManager|setCurrentAtSign called with atSign @jeremy_0
INFO|2024-10-07 11:31:23.401307|AtClientManager|Switching atSigns from null to @jeremy_0
INFO|2024-10-07 11:31:23.415247|HiveBase|commit_log_3c074c7d3712aa56c1d3e81561c463442f282dfda4ddcc62a724ea0844c2f0e0 initialized successfully
INFO|2024-10-07 11:31:23.420138|HiveBase|3c074c7d3712aa56c1d3e81561c463442f282dfda4ddcc62a724ea0844c2f0e0 initialized successfully
INFO|2024-10-07 11:31:23.432690|AtClientCommitLogCompaction (@jeremy_0)|Starting commit log compaction job running for every 11 minute(s)
INFO|2024-10-07 11:31:23.448292|AtClientManager|setCurrentAtSign complete
Connected
INFO|2024-10-07 11:31:23.595595|AtLookup|Creating new connection
INFO|2024-10-07 11:31:23.765248|AtLookup|New connection created OK
INFO|2024-10-07 11:31:23.848275|AtLookup|auth success
➜  dart_playground

I plan to do a check with the C repl now this spritn

Monitor receiving end
[DEBG] 2024-10-15 00:49:57.508558 | connection |        SENT: "monitor"
[DEBG] 2024-10-15 00:49:58.449249 | connection | pos: 0, ret: 389
[DEBG] 2024-10-15 00:49:58.449304 | connection |        RECV: "notification: {"id":"65c71a9c-1c35-477a-a9b0-30dd88d025b0","from":"@jeremy_0","to":"@smoothalligator","key":"@smoothalligator:test.c_playground@jeremy_0","value":"asd","operation":"update","epochMillis":1728953398431,"messageType":"MessageType.key","isEncrypted":true,"metadata":{"encKeyName":null,"encAlgo":null,"ivNonce":null,"skeEncKeyName":null,"skeEncAlgo":null,"sharedKeyEnc":null}}"
recv (388): notification: {"id":"65c71a9c-1c35-477a-a9b0-30dd88d025b0","from":"@jeremy_0","to":"@smoothalligator","key":"@smoothalligator:test.c_playground@jeremy_0","value":"asd","operation":"update","epochMillis":1728953398431,"messageType":"MessageType.key","isEncrypted":true,"metadata":{"encKeyName":null,"encAlgo":null,"ivNonce":null,"skeEncKeyName":null,"skeEncAlgo":null,"sharedKeyEnc":null}}

Notification sending end
[DEBG] 2024-10-15 00:49:57.807150 | connection |        SENT: "notify:update:isEncrypted:true:@smoothalligator:test.c_playground@jeremy_0:asd"
[DEBG] 2024-10-15 00:49:57.847440 | connection | pos: 0, ret: 52
[DEBG] 2024-10-15 00:49:57.847498 | connection |        RECV: "data:65c71a9c-1c35-477a-a9b0-30dd88d025b0"
recv (41): data:65c71a9c-1c35-477a-a9b0-30dd88d025b0

notiec that in the Monitor receiving end, we get isEncrypted":true
which means that the metadata was properly set by the atServer
I will go ahead and close the ticket as I have found the feature to be working from my own 2 manual tests


Index scans are broken

@Tishj mentioned in #97 that we don't have tests for index scans. So I tried adding some basic tests for this. And as it turns out index scans are very much (and possibly completely) broken. This trivial example crashes the server for me:
CREATE TABLE test_table (id int primary key, name text);
SELECT * FROM test_table where id = 1;

Yes, imo all index functionality and code should be revisited again probably when i return back to finalize #71 feature.

It was working in the past iirc, also at minimum it shouldn't crash the server, but throw a clear error.

As discussed over chat. Our current Postgres index scan logic is incorrect. For 0.1.0 we've decided to revert the feature completely and only support sequential scans on postgres data from duckdb. It's definitely a feature we want, but we're not going to let it block version 0.1.0. We'll probably redesign it for version 0.2.0.


Provide CLI snippet to Associate a WAF policy with an existing Application Gateway

Could you please provide a CLI snippet to Associate a WAF policy with an existing Application Gateway ?

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

ID: 5ee1c7b9-9a37-6bff-c82f-86266767d738
Version Independent ID: fde4aea7-e78a-129d-8f6b-d6de65285aa5
Content: Associate a Web Application Firewall policy with an existing Azure Application Gateway
Content Source: articles/web-application-firewall/ag/associate-waf-policy-existing-gateway.md
Service: web-application-firewall
GitHub Login: @vhorne
Microsoft Alias: victorh

Thanks for the feedback! We are currently investigating and will update you shortly.

@ezYakaEagle442 Thanks for the feedback! It looks like the existing version of the CL does not have a way to add a WAF policy. I have assigned the issue to the content author to evaluate and update as appropriate.

I run latest CLI :
{
"azure-cli": "2.3.1",
"azure-cli-command-modules-nspkg": "2.0.3",
"azure-cli-core": "2.3.1",
"azure-cli-nspkg": "3.0.4",
"azure-cli-telemetry": "1.0.4",
"extensions": {
"aks-preview": "0.4.39",
"azure-firewall": "0.3.0"
}
}
What about this ? :
az network application-gateway waf-policy create
--name "${appName}-waf-policy"
--resource-group $rg_name
waf_policy_id=$(az network application-gateway waf-policy show --name "${appName}-waf-policy" -g $rg_name --query id -o tsv)
echo "WAF Policy id " $waf_policy_id
az network application-gateway waf-policy policy-setting update --policy-name "${appName}-waf-policy" --mode Detection --state Enabled -g  $rg_name
az network application-gateway create --name $appgw_name
--resource-group $rg_name
--capacity 1 --min-capacity 1 --max-capacity 3
--connection-draining-timeout 1
--http-settings-cookie-based-affinity Enabled
--http-settings-port 443 --frontend-port 80
--http-settings-protocol Https
--http2 Enabled
--location $location
--zones 1 2 3
--private-ip-address <IP_ADDRESS>
--public-ip-address $appgw_IP
--routing-rule-type Basic
--sku WAF_v2
--subnet $appgw_subnet_id
--subscription $subId
--waf-policy "${appName}-waf-policy"

@ezYakaEagle442 That should work just fine. The only CLI limit is with adding a WAF policy to an existing Application Gateway.

az network application-gateway update does not currently support waf policy parameters.
#please-close


Add Repository Variables and Secrets

Hey, I just made a Pull Request!
Adding Repository Variables and Secrets to github scaffolder actions
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Closes https://github.com/backstage/backstage/pull/17722

Ok I need some help w/ these errors.  Not sure the right node/typescript way to solve this.

Ugh. Still being affected by https://github.com/octokit/rest.js/issues/290 it seems. Is this rebased on the very latest master?

Ugh. Still being affected by octokit/rest.js#290 it seems. Is this rebased on the very latest master?

Nope.  Just rebased.  Lets see what transpires

Did as you suggested.  Still having type compilation issues.  I need at least 2.0.3 of octokit to get createRepoVariable() but every time I touch that it seems to spew all sorts of tsc issues.  I can't seem to find the magic combination

Nice, I was just looking for this.
Does this change the required scope on the GH tokens and apps?

I'd say for one in the changelog, so that it's part of the release notes.
Also, https://backstage.io/docs/integrations/github/locations#token-scopes and https://backstage.io/docs/integrations/github/github-apps#app-permissions.
Maybe @ahhhndre has some more tips, I know he struggled with the last change that flew under the radar.


Install

Hi I am working on learning linux and postgres and I don't get the instructions for installing the library.  I have libpostal working and installed and i have it working for python.  Install postgressql and the contrib packages and i found Pg_config file.  After that i am not sure what to do.  I looked at the makefile for libpostal and i don't see the options in the instructions. Do you happen to have any more detailed instructions.
Thanks,
Matt

Look at the Makefile for pgsql-postal, edit the POSTAL_INCLUDE and POSTAL_LIBS variables to point to your libpostal install location (often /usr/local as in the defaults). Check that pg_config is in your path by just typing it at the command line. At that point you should be able to just type make in the pgsql-postal directory and the build should go. It's all in the README and I'm not sure how to make it more detailed. I could try automating it, but that would cause as many problems as it solves (using cmake means now... you need cmake).


uninstall verb fails

broken since https://github.com/ament/ament_tools/pull/114
dhood@osrf-esteve:~/ros2_test_ws [ros2_test_ws]$ ament uninstall
# Topological order
Traceback (most recent call last):
  File "/home/dhood/ros2_ws/install_isolated/ament_tools/bin/ament", line 9, in <module>
    load_entry_point('ament-tools', 'console_scripts', 'ament')()
  File "/home/dhood/ros2_ws/build_isolated/ament_tools/ament_tools/commands/ament.py", line 88, in main
    rc = args.main(args)
  File "/home/dhood/ros2_ws/build_isolated/ament_tools/ament_tools/verbs/uninstall/cli.py", line 91, in main
    print_topological_order(opts, packages)
  File "/home/dhood/ros2_ws/build_isolated/ament_tools/ament_tools/verbs/build/cli.py", line 240, in print_topological_order
    if pkg_name in opts.skip_packages:
AttributeError: 'Namespace' object has no attribute 'skip_packages'

See #123.


Added ForceRemoveCmd

This pull request...

[ ] Fixes a bug
[x] Introduces a new feature
[ ] Improves an existing feature
[ ] Boosts code quality or performance

Description
This PR adds a forceremove command, which bulk deletes all entries made by a user. The command accepts mentions and id's.
Purpose
As described in #237, DJ's may use it in case a user maliciously or accidentally adds many songs to the queue.
Relevant Issue(s)
This PR closes #237

need short commands
like
play=p
nowplay=np
forceskip=fs
make all short command please

@ShisirAhm3d that should be up to jagrosh to implement. I could make another pull request implementing custom aliases you can configure in the config.txt file.


Determine production OSCARS 1.0 service owner

As with many upcoming software projects, the engineers who developed the software are not necessarily the most appropriate people to be responsible for the service that's implemented by it.
This issue is to make that determination for OSCARS 1.0 at ESnet.

According to Patrick, it'll be David and Indira.  Closing.


GEODE-7157: SSLConfig&SSLConf..Factory thread-safe

SSLConfig and SSLConfigFactory have been changed to be
tread-safe by means of the following:

SSLConfig has been converted into an immutable class

The registeredSSLConfig HashMap member of SSLConfigurationFactory
has been changed into a ConcurrentHashMap and accesses to this
map other than put and get has been synchronized.

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

I think it is still worth adding a test that demonstrates that this class is now thread safe.
You have made it so by making it immutable but the test would prove that it actually is immutable.
And the test could catch future regressions if someone breaks the immutable contract.

I think it is still worth adding a test that demonstrates that this class is now thread safe.
You have made it so by making it immutable but the test would prove that it actually is immutable.
And the test could catch future regressions if someone breaks the immutable contract.

I have been thinking about what test(s) I could add but have not come up with any idea.
Regarding SSLConfig, after the changes it does not offer any public method that would allow to modify its state. It only provides the constructor and getter methods so I cannot think of a test that would assure that in the future the class will remain immutable.
Regarding SSLConfigurationFactory, having removed the deprecated method that returned shared SSLConfig objects (stored in the static HashMap member), now it always returns a new SSLConfig object each time the getSSLConfigForComponent method is invoked.
In this case, the only test case I can think of is one that makes sure that invocations of these methods with the same parameters return different objects. Would this be useful?
I'd appreciate any insight on what tests to add.
Thanks in advance.

I you went back to the initial condition (before this pr's work) wouldn't you be able to write a concurrent test that demonstrates that these classes are not thread safe? If so, then that test should start passing with this pr's changes.

I you went back to the initial condition (before this pr's work) wouldn't you be able to write a concurrent test that demonstrates that these classes are not thread safe? If so, then that test should start passing with this pr's changes.

The fact is that the method that had thread-safety issues (public static SSLConfig getSSLConfigForComponent(SecurableCommunicationChannel sslEnabledComponent)) has been eliminated.
Therefore, I could use in a test prior to the PR to see failures but I could not in the new version as it would not be present.

I know that I am VERY late into this discussion... I think the essence of WHY has been lost be this refactor. Whilst approving code on correctness of code is good, but context is lacking and thus we have now added a new potential performance issue. Given that there is no documentation as to what the constraints are/were initially this fix is good.
But to fill in the gaps in context:

SSLConfigurationFactory was supposed to cache SSLConfig objects, for performance reasons. This has now been lost. In addition we are now adding more time in processing the DistributionConfig EVERY time we need SSLConfig for a SecurableChannel
The DistributionConfig stored on the SSLConfigurationFactory was really to tie the "cached" SSLConfig objects to a DistributionConfig
SSLConfigurationFactory was never supposed to be static. At the time we had to do it, because we could not find a good "home" for it. It should really be an instance that takes a DistributionConfig and provides the SSLConfig for that config. Not an ad-hoc.
Thread safety is really less of a concern, given that the DistributionConfig should never change throughout the lifetime of the DistributedSystem. If you happen to have 10 threads asking for SSLConfig for the SERVER at the same time, YES, you could have 10 threads recreating the same 10 SSLConfigurations, BUT, they would contain the SAME information. In the end you will 1 cached copy of SSLConfig.

I think DO like the new SSLConfig.Builder pattern applied now. Immutability of the SSLConfig object is great! Maybe, if we can couple, it would "caching" we might be able to offset the performance hit we would now have.
It is also interesting to me that we annotate final properties with Immutable.

I know that I am VERY late into this discussion... I think the essence of WHY has been lost be this refactor. Whilst approving code on correctness of code is good, but context is lacking and thus we have now added a new potential performance issue. Given that there is no documentation as to what the constraints are/were initially this fix is good.
But to fill in the gaps in context:
1. `SSLConfigurationFactory` was supposed to cache `SSLConfig` objects, for performance reasons. This has now been lost. In addition we are now adding more time in processing the `DistributionConfig` EVERY time we need `SSLConfig` for a `SecurableChannel`

2. The `DistributionConfig` stored on the `SSLConfigurationFactory` was really to tie the "cached" `SSLConfig` objects to a `DistributionConfig`

3. `SSLConfigurationFactory` was never supposed to be static. At the time we had to do it, because we could not find a good "home" for it. It should really be an instance that takes a `DistributionConfig` and provides the `SSLConfig` for that config. Not an ad-hoc.

4. Thread safety is really less of a concern, given that the `DistributionConfig` should never change throughout the lifetime of the `DistributedSystem`. If you happen to have 10 threads asking for `SSLConfig` for the `SERVER` at the same time, YES, you could have 10 threads recreating the same 10 `SSLConfig`urations, BUT, they would contain the SAME information. In the end you will 1 cached copy of `SSLConfig`.

I think DO like the new SSLConfig.Builder pattern applied now. Immutability of the SSLConfig object is great! Maybe, if we can couple, it would "caching" we might be able to offset the performance hit we would now have.
It is also interesting to me that we annotate final properties with Immutable.

Hi Udo,
I noticed that this class was providing a cache for SSLConfig objects. Nevertheless, the method that was making use of the cache appeared as deprecated since Geode 1.3. In the comment, about the deprecation, it was recommended to use another method of the class that created an SSLConfig each time.
Given that the cache had some thread-safety issues to be solved and given that the method managing the was deprecated I figured the best option would be to just remove it.
Anyway, if it was a mistake to remove it, I offer myself to put it back.

@albertogpz Thank you for this.
I think there are some good fixes in the PR. I think we need the SSLConfig cache back.. but it needs to be better managed. Possibly there is some stale state and possibly some better approaches to some of the deficiencies.
I had not noticed the deprecation on the method. Possibly another commit that did not take into account the "caching".
Tbh, let's see how this evolves, maybe "cache vs not cached" is a non-event given that we don't really cycle connections that often.
Thank you for the PR though. Great work!


please surpport adb shell input --ext-display  tap x,x   on android 9

adb shell input --ext-display tap/swipe...   This   commend can sending event to extented display,
so it is very useful  for    scrcpy  --display  2   .
it also  can solve  the  issue  "Input events are not supported for secondary displays before Android 10".
Because   "adb shell input --ext-display keyevent  "  can  send input  to secondary displays  on android 9.
Thank  you  very much.

Thank you very much.

C:\Users\平时>adb shell input
Usage: input [--ext-display] []  [...]
The sources are:
dpad
keyboard
mouse
touchpad
gamepad
touchnavigation
joystick
touchscreen
stylus
trackball
The commands and default sources are:
text  (Default: touchscreen)
keyevent [--longpress]  ... (Default: keyboard)
tap   (Default: touchscreen)
swipe     [duration(ms)] (Default: touchscreen)
draganddrop     [duration(ms)] (Default: touchscreen)
press (Default: trackball)
roll   (Default: trackball)
keydown   (Default: keyboard)
keyup   (Default: keyboard)
**_The ext-screen support:
  --ext-display, sending event to extented display_**

Please open an issue instead of a pull request.

adb shell input --ext-display tap/swipe..

What is this parameter --ext-display?
It seems it does not exist in the source code of the input command: https://android.googlesource.com/platform/frameworks/base.git/+/master/cmds/input/src/com/android/commands/input/Input.java

Please open an issue instead of a pull request.

adb shell input --ext-display tap/swipe..

What is this parameter --ext-display?
It seems it does not exist in the source code of the input command: https://android.googlesource.com/platform/frameworks/base.git/+/master/cmds/input/src/com/android/commands/input/Input.java

--ext-display  means  send event to extented display  ,
when secondary displays started  ,  that  commend  appears.
That  commend  is on  some  phone  that  surpport   displayport

Please open an issue instead of a pull request.

adb shell input --ext-display tap/swipe..

What is this parameter --ext-display? (oh it's maybe a vendor specific feature, what is the device?)
It seems it does not exist in the source code of the input command: https://android.googlesource.com/platform/frameworks/base.git/+/master/cmds/input/src/com/android/commands/input/Input.java
The device  is  android phone  smartisan  nut  pro3,based on  android 9,
the commend  can control  samrtisan  tnt  mode.


updated the class path for Eclipse

Related to issue #367 I've updated the eclipse .classpath file.
The steps to set up lwjgl 3 in eclipse are:

Check out git project (do not add as existing project)
Run ant

Open terminal
CD to lwjgl3 directory
ant init
ant compile-templates compile-tests

Copy .classpath and .project from lwjgl3/config/ide/eclipse to the root lwjgl3/ directory
In Eclipse (File > Import > Existing Project Into Workspace)

select the root lwjgl3/ directory

Related to issue #349

Open Build Path > Libraries tab and add the Tools.jar file as an external jar from your /usr/lib/jvm/<java implementation>/libs directory

If you have TestNG installed right click the /modules/lwjgl/core/src/test/java folder and Run As > TestNG Test

If you're running Linux expect 25 of 30 passes. For some reason the 5 windows specific tests don't work ¯\_(ツ)_/¯

There was a class path missing from this request that I discovered this morning when I re-re-re-followed my steps.


It doesn't work at all with Palo Alto Networks firewalls

it is able to authenticate, but when I try to invoke a command nothing happens.
what I suspect is that it doesn't get to a point where it is able to see that the terminal is ready for input.
I was able to get my result from an expect script:
spawn ssh [lindex $argv 1]@[lindex $argv 0]
expect "yes/no" {
send "yes\r"
expect "?assword" { send "[lindex $argv 2]\r" }
} "?assword" { send "[lindex $argv 2]\r" }
expect "> " { send "ping source [lindex $argv 0] count 10 host [lindex $argv 3]\r" }
expect "ping statistics ---"
expect "> " { send "exit\r" }
expect eof
is there any additional logs I can get to help figure out why?
Thank you for your work on this

Does the same happen with a streamshell? most network equipment have their own shell where invoke-sshcommand does not work Sent from my iPhoneOn Dec 22, 2023, at 9:13 PM, Steve Borba @.> wrote:
it is able to authenticate, but when I try to invoke a command nothing happens.
what I suspect is that it doesn't get to a point where it is able to see that the terminal is ready for input.
I was able to get my result from an expect script:
spawn ssh [lindex $argv @. $argv 0]
expect "yes/no" {
send "yes\r"
expect "?assword" { send "[lindex $argv 2]\r" }
} "?assword" { send "[lindex $argv 2]\r" }
expect "> " { send "ping source [lindex $argv 0] count 10 host [lindex $argv 3]\r" }
expect "ping statistics ---"
expect "> " { send "exit\r" }
expect eof
is there any additional logs I can get to help figure out why?
Thank you for your work on this
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

Thank you - I think it does.  Now I just need to wrap my head around this.
I also tried without streamshell on  a cisco and it handled it better, but not as well as a linux machine.
thank you for the quick response


🚀 [Feature Request] - Improve error handling ( UX work)

Is your feature request related to a problem ? Please describe. ✍
We had not consider all case and error response...
we have Taostify to display error/success.
you just have to work on all cases frontend can display error which can be shown by frontend..
Describe briefly how you would implement this enhancement.
for eg:
If any field is blank , it should be show error.
The thing is to consider all possible case of error so user can get what causing error.
Major area: form filling

Additional context or screenshots
Additional context
Are you contributing for a program?
Other
Code of Conduct

[X] I have read and agree to the Contributing Guidelines of this project.

@anmode  alright will take care.

@anmode is this repo open?

@inbuilt-aura  yes i am working on it.


Logo

We need the correct ones .. :)
You find them in dropbox

[ ] Looks weird in safari
[ ] Check that footer is correct in page examples


Adding a new server.Router interface and config.Server.RouterType option with 2 implementations

Gizmo users who need the performance of julienschmidt/httprouter more than the handy URL matching of Gorilla web toolkit's mux, can now configure the server.SimplerServer and server.RPCServer to use an alternate Router implementation.
A new attribute in config.Server, RouterType, accepts the value "gorilla" (the default) to use Gorilla mux.Router or "fast" to use httprouter.Router.
To keep the use of idiomatic http.Handlers throughout Gizmo, instead of using httprouter.Handle, a new middleware function HTTPToFastRoute injects the httprouter.Params into a Gorilla request context. To retrieve the parameters, a new FastRouteVars(r *http.Request) map[string]string function is available and can be used like Gorilla's mux.Vars.
New benchmarks with the "Fast" prefix have been added to compare the two router types:
BenchmarkFastSimpleServer_NoParam-4         	 5000000	      2891 ns/op
BenchmarkFastSimpleServer_WithParam-4       	 5000000	      2606 ns/op
BenchmarkSimpleServer_NoParam-4             	 3000000	      4885 ns/op
BenchmarkSimpleServer_WithParam-4           	 5000000	      3143 ns/op
BenchmarkFastJSONServer_JSONPayload-4       	 3000000	      5688 ns/op
BenchmarkFastJSONServer_NoParam-4           	 5000000	      3306 ns/op
BenchmarkFastJSONServer_WithParam-4         	 5000000	      4434 ns/op
BenchmarkJSONServer_JSONPayload-4           	 1000000	     10510 ns/op
BenchmarkJSONServer_NoParam-4               	 3000000	      7444 ns/op
BenchmarkJSONServer_WithParam-4             	 1000000	     10735 ns/op
BenchmarkFastContextSimpleServer_NoParam-4  	 2000000	      9392 ns/op
BenchmarkFastContextSimpleServer_WithParam-4	 2000000	      8427 ns/op
BenchmarkContextSimpleServer_NoParam-4      	 1000000	     15932 ns/op
BenchmarkContextSimpleServer_WithParam-4    	 1000000	     10570 ns/op

Coverage decreased (-1.4%) to 39.684% when pulling eadc24fb4fe17f0e2c1b40a56e062e4a5c76aae5 on router into c13e48c14b9df8f9f542ffc0dc83c653db034ac9 on master.

Coverage decreased (-1.4%) to 39.684% when pulling 55dd606d3600dd2c49ea8fbeb1516a3ac9117110 on router into c13e48c14b9df8f9f542ffc0dc83c653db034ac9 on master.

Coverage decreased (-1.0%) to 40.137% when pulling 9d4bff63f8779806c3ec5af96c1b5adfdd2bce48 on router into c13e48c14b9df8f9f542ffc0dc83c653db034ac9 on master.

Coverage increased (+4.9%) to 46.055% when pulling 88d34d5f38246acb35954c8879a8ed5fdbe63b5b on router into c13e48c14b9df8f9f542ffc0dc83c653db034ac9 on master.

Coverage increased (+4.9%) to 46.055% when pulling 44c43d80894ff74dcbd65d960d49352c6087cddb on router into c13e48c14b9df8f9f542ffc0dc83c653db034ac9 on master.

:boom:


Add CDN link

I added a jsDelivr CDN link to your readme to make it easier to use. jsDelivr is the fastest opensource CDN available and built specifically for production usage.
I also fixed the headlines.

Better late than never ...
-- Drake

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


using cog_translate on netCDF4 with improper georeferencing

I am trying to use cog_translate to convert some netCDF4 datasets to COG, and it appears these files do not have proper georeferencing.
I have modified cogeo.py to accept vrt_params, so that I can manually pass the src_crs and src_transform via the WarpedVRTReaderBase mixin of WarpedVRT, as you can see here:
https://github.com/ryanjdillon/rio-cogeo/blob/cog_translate_vrt_params/rio_cogeo/cogeo.py#L87-L99
Then I'm passing the following vrt_params:
vrt_params = dict(src_transform=Affine(gdal_geotransform), src_crs='EPSG:4326')

This get's me past the CRS is None error, but it doesn't seem to acknowledge the Affine transform I pass to src_transform, resulting in the following error:

CPLE_AppDefinedError: The transformation is already "north up" or a transformation between pixel/line and georeferenced coordinates cannot be computed for /home/ryan/data/webstep/kvt/NVE_Sorlandet/NVE_Sorlandet_wslev0_xyt_synth.nc4. There is no affine transformation and no GCPs. Specify transformation option SRC_METHOD=NO_GEOTRANSFORM to bypass this check.

Any suggestions on how I might get this to work?

@ryanjdillon can you try transform instead of src_transform ? https://github.com/cogeotiff/rio-cogeo/blob/webOpt/rio_cogeo/cogeo.py#L141

Excellent. I'll report back as soon as I've tried that out. Thanks!
Den fre. 5. apr. 2019, 16:18 skrev Vincent Sarago<EMAIL_ADDRESS>
:

@ryanjdillon https://github.com/ryanjdillon can you try transform
instead of src_transform ?
https://github.com/cogeotiff/rio-cogeo/blob/webOpt/rio_cogeo/cogeo.py#L141
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cogeotiff/rio-cogeo/issues/74#issuecomment-480292397,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA39Gv8T72PKy_nfaR93XtF17TUwujv0ks5vd1slgaJpZM4ce6K0
.

@ryanjdillon did it fix the problem ?

I was just about to report back. That didn't seem to fix things.
It looks like dst_crs and  dst_transform are depreciated in favor of crs and transform, so I'm passing things with my modification like this

    vrt_params = dict(
        src_transform=nc_affine, src_crs=crs,
        transform=nc_affine, crs=crs,
        )

        cogeo.cog_translate(
            src_path=band_fp,
            dst_path=dst_fp,
            dst_kwargs=config.GDAL_TRANSLATE_PROFILE,
            vrt_params=vrt_params,
            indexes=[i,],
            overview_resampling='nearest',#resampling_method,
            config=config.GDAL_CONFIG
            )

Which yields the same error. If I pass width and height, as used in the Affine instance, I get another error:

ObjectNullError: Pointer 'hDS' is NULL in 'GDALSetProjection'.

And that seems to be related to the output file not existing. It sounded as though this could have been due to the slightly outdated Debian packages I'm using, so I compiled everything from source, which didn't do anything for me.
gdalinfo --version

GDAL 2.4.1, released 2019/03/15

nc-config --all

This netCDF 4.6.3 has been built with the following features:
--cc            -> gcc
--cflags        -> -I/usr/local/include -I/home/ryan/opt/hdf5-1.10.5/hdf5/include -I/home/ryan/opt/hdf-4.2.14/hdf4/include
--libs          -> -L/usr/local/lib -lnetcdf
--has-c++       -> no
--cxx           ->
--has-c++4      -> no
--cxx4          ->
--has-fortran   -> no
--has-dap       -> yes
--has-dap2      -> yes
--has-dap4      -> yes
--has-nc2       -> yes
--has-nc4       -> yes
--has-hdf5      -> yes
--has-hdf4      -> yes
--has-logging   -> no
--has-pnetcdf   -> no
--has-szlib     -> no
--has-cdf5      -> yes
--has-parallel4 -> no
--has-parallel  -> no
--prefix        -> /usr/local
--includedir    -> /usr/local/include
--libdir        -> /usr/local/lib
--version       -> netCDF 4.6.3

I've tried to use cog_translate() as I described above on a file that is correctly georeferenced (Copernicus global NDVI from their Land service), and that gives me the same error.
However, gdalwarp seems to work by doing the following:
gdalwarp -s_srs EPSG:4326 -t_srs EPSG:4326 -of GTIFF 'NETCDF:"c_gls_NDVI_201808110000_GLOBE_PROBAV_V2.2.1.nc":NDVI' ./test.tiff

Whereas my NetCDF file lacking georeferencing throws the same error with gdalwarp as with cog_translate.

It seems like it is a bit of a combo. rio-cogeo passes the dataset as a band, which I was opening with gdal. I got it to work when I opened the subdataset with a rasterio formatted band path, rather than gdal, and also by passing the vrt_params (including src_crs, src_transform, crs, transform, width, and height).
I also tried using cog_validate on the generated tiff, and it bugged out when src.get_tag_item("BLOCK_OFFSET_0_0", "TIFF", bidx=1) was None.
I could submit a PR if you want to look over / incorporate these changes.

🤔 thanks @ryanjdillon,

I also tried using cog_validate on the generated tiff, and it bugged out when src.get_tag_item("BLOCK_OFFSET_0_0", "TIFF", bidx=1) was None.

Can you checkout what https://svn.osgeo.org/gdal/trunk/gdal/swig/python/samples/validate_cloud_optimized_geotiff.py gives you ?

I could submit a PR if you want to look over / incorporate these changes

Allowing vrt_params as input might complicated a lot the code (especially with the changes from https://github.com/cogeotiff/rio-cogeo/pull/62). That said, I think we could document it and tell how to generate a proper temporary file from the netCDF4 before passing it to rio-cogeo.
I'm no expert with netCDF so I'm may be missing some knowledge here

I'll take a look at as soon as possible.
Once I have a clear handle on things I can contribute an example to the docs or something.

I managed to get things to work by first converting the NetCDF to GeoTIFF using GDAL, and then doing the COG conversion with rio_cogeo.
I first created a VRT file with gdal_translate, passing the corner coords as GCPs. Then I used gdalwarp to convert from NetCDF to GeoTIFF. Passing the same GCPs to rasterio did not seem to correctly preserve the georeference in the resulting GeoTIFF. I am guessing this has to do with the files not conforming to a NetCDF "CF" standard.

@ryanjdillon Long time no see 😄
I finally had to work with netcdf recently and thanks to some recent change in rio-cogeo we can now pass Memory or VRT files to cog_translate.

Latest version

def translate(
    src_path,
    cog_path,
    cogeo_profile="deflate",
    cogeo_options={"blockxsize": 128, "blockysize": 128, "predictor": 2, "zlevel": 7},
):
    """Convert a netcdf to COG."""
    config = dict(GDAL_NUM_THREADS="ALL_CPUS", GDAL_TIFF_OVR_BLOCKSIZE="128")

    with rasterio.Env(**config):
        with rasterio.open(src_path) as src_dst:
            with WarpedVRT(
                src_dst,
                crs="epsg:4326",
                warp_mem_limit=0.1,   # Reduce chunk size, see: https://github.com/OSGeo/gdal/issues/1989
                num_threads=8
            ) as vrt:
                profile = cog_profiles.get(cogeo_profile)
                profile.update(cogeo_options)
                cog_translate(
                    vrt,
                    cog_path,
                    profile,
                    config=config,
                    in_memory=True,
                    quiet=True,
                )

First version

def translate(
    src_path,
    cog_path,
    cogeo_profile="deflate",
    cogeo_options={"blockxsize": 128, "blockysize": 128, "predictor": 2, "zlevel": 7},
):
    """Convert a netcdf to COG."""
    config = dict(GDAL_NUM_THREADS="ALL_CPUS", GDAL_TIFF_OVR_BLOCKSIZE="128")

    with rasterio.Env(**config):
        with rasterio.open(src_path) as src_dst:
            dst_transform, dst_width, dst_height = calculate_default_transform(
                src_dst.crs, "epsg:4326", src_dst.width, src_dst.height, *src_dst.bounds
            )
            meta = dict(
                driver="GTiff",
                dtype=src_dst.dtypes[0],
                count=src_dst.count,
                height=dst_height,
                width=dst_width,
                crs="epsg:4326",
                transform=dst_transform,
                nodata=-999,
                tiled=True,
                compress="deflate",
                blockxsize=256,
                blockysize=256,
            )
            with MemoryFile() as memfile:
                with memfile.open(**meta) as mem:
                    reproject(
                        rasterio.band(src_dst, 1),
                        rasterio.band(mem, 1),
                        resampling=rasterio.enums.Resampling.nearest,
                        warp_mem_limit=1,  # Reduce chunk size, see: https://github.com/OSGeo/gdal/issues/1989
                        num_threads=8,
                    )
                    mem.update_tags(**src_dst.tags(1))
                    profile = cog_profiles.get(cogeo_profile)
                    profile.update(cogeo_options)
                    cog_translate(
                        mem,
                        cog_path,
                        profile,
                        config=config,
                        in_memory=True,
                        quiet=True,
                    )

since we can use MemoryFile or VRT, we can now fix the data upfront and pass it to cog_translate so I think we can close this issue.

Excellent. Thanks for the ping.


[Bug] [Enhancement]: livestream logic bug in 2.0.0-beta.5 and beta.6

What happened?
Hi!
I think I have found a somewhat severe logic bug in the way local livestreams (Dufy uses the term p2p I think) are handled in the current beta. Please correct me if I'm wrong - but at least for me this happened multiple times.
My Hardware:
Eufy Battery Doorbell (T8210)
The way streamingDelegate.ts works now, consider the following situation (as said I'm only referring to local streams only and no RTSP streams):

You open HomeKit on your device and see the camera device as snapshot in the HomeKit-UI. Chances are high that a new snapshot will be requested, so the stream is started. This (for me at least) takes in total somewhere from 3-6 seconds.
In this timeframe the users touches the camera to open the livestream. Again the process of starting the local stream is executed.
The Snapshot is finally returned and the fetchSnapshot function calls stopStationLivestream() on the EufyClient. This then results in alle the streams stopping. Including the one the user wanted to watch.

This happened several times to me. So I can confirm that this is an issue. Also I think many open issues for the beta could be explained with this.
I have implemented an alternative approach with some major advantages. I'll open a pull request with my changes and would like you to comment your view of the approach.
Here's a summary of what I've done to mitigate the issue (again all this applies only for p2p/local streams - rtsp remains untouched):
For every camera streaming delegate there is a new 'cached' livestream.
Once a livestream is requested it is opened via the startStationLivestream function but is not immediately stopped after one request is finished. Instead the cache keeps track of how many instances use the livestream and only stops it, when the livestream wasn't used  for a given time.
So the aforementioned scenario would go something like this:

The user opens HomeKit. The snapshot is requested and the cache starts the livestream.
The user clicks on the camera to watch the livestream. Now the livestream mustn't be loaded again, since the cache already had 1-2 seconds to do this.
The snapshot-request is fulfilled and releases one instance of livestream use. One instance remains since the user keeps watching the stream.
The user closes the stream. The cache now has no more instances that require the stream and starts a timeout to stop the stream.
After a few seconds a new snapshot is requested. Since the stream is still in the cache it can now be used instantaneously. (For me the snapshot requests came down to under a second from 3-6 seconds before)
The user closes HomeKit. Again no instance is requiring the stream so the cache sets a timeout to stop it. When this timeout fires, the livestream finally is released.

I have tested this with multiple devices using the same stream simultaneously and it not only works like a charm. It also reduces the start times of the streams enormously.
Advantages:

Removes the aforementioned bug / behaviour.
Reduces starting time for livestreams, after the stream was successfully cached.
Could be implemented in such a way that camera events (motion, doorbell, ...) started the cached stream in advance (before the user opens HomeKit) - so starting times would (in theory) even be better than the native Eufy app

Drawbacks (of course there are some):

slightly higher resource cost (cpu for HomeKit, maybe battery for battery powered devices - since the livestream would generally be running a bit more) - have not tested this and could imagine that this is negligible
This approach somewhat conflicts with the maxCameraLivestreamDuration of the eufy client. If this setting is set to a low value, the cache will hardly be able to keep the stream running for long.
Also if this setting is set to a high value it could result in a somewhat strange behavior for the user. For example:

The user opens HomeKit. The cache is started. For this scenario I assume a max livestream duration with Dufy of 10 minutes.
the users stays in the snapshot view for 5 minutes. The snapshots are frequently refreshed with the cached livestream.
Then the users clicks on the camera and watches the livestream.
After another 5 minutes the Dufy client would terminate the stream, since the total 10 minutes would be over. This would be counterintuitive for the users, since he had set 10 minutes in the configuration.

Nonetheless I'd favor this approach more. I'd imagine that this would not be the usual use case of most users.
Also I have tried to mitigate this new issue by letting the cache calculate how much time of the max livestream duration remains. If there is not sufficient time left when the last instance releases it's stream then the cache will immediately
stop the stream and the whole process will start over.
Again. I'm open to suggestions for improvement and try to open a pull request for my approach (will be my first pr, so be gentle with me :-)
edit: Pull Request added:   #31
Device Type
Doorbell (Supported)
Plugin Version
beta (Best Effort)
HomeBridge Version
v1.4.x (Supported)
NodeJS Version
v16 (Supported)
Operating System
Debian 10+ (Supported)
Relevant log output
No response

Hey nice ! Will push into the beta and allow people to test.

Glad you like it.
Is there a demand to get the following working?

Could be implemented in such a way that camera events (motion, doorbell, ...) start the cached stream in advance (before the user opens HomeKit) - so starting times would (in theory) even be better than the native Eufy app

And if so, should this, in your view, be controlled via HomeKit configuration? Since every event would start a stream in advance this could have a significant impact on battery powered devices. What would be the preference for default behaviour?

Yeah I've put a message below that setting saying that could drown the battery

Should be resolved in #37


Hi, i cant use this plugin to change the temperature

Hello,
I can not use this plugin to change the temperature.
Only can set the opeation mode to heating, cooling, off.
Cuz i can not see the temperature adjust setting from homekit with this plugin.
Please advice.
Thank you


Extract the chart colours ?

Does the library allow us to extract the colours used by the chart for each category (category:name in the data) ?

try
var option = myChart.getOption();


warning : The Location of LibreOffice Python has changed!

I just find that the LibreOffice python has moved  to  /Applications/LibreOffice.app/Contents/Resources in the lastest Version (<IP_ADDRESS>) , FYI

OK, thanks


Can i use portia to scrap facebook post (image+text)

i purpose of crapping facebook content for certain user. portia can do this?

It's not possible because Facebook is using different names for ids & classes for each request. Try this https://developers.facebook.com/tools/explorer/.

oh, thanks
http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
Virus-free.
www.avg.com
http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
On Thu, Sep 7, 2017 at 5:02 PM, Igor Ermentraut<EMAIL_ADDRESS>wrote:

It's not possible because Facebook is using different names for ids &
classes for each request. Try this https://developers.facebook.
com/tools/explorer/.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/scrapinghub/portia/issues/818#issuecomment-327754422,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIFceBuOdwXtp2uggirFIK_ylZRl6xHUks5sf78wgaJpZM4PPKRQ
.


Docker Extension doesn't work

Since today,
I'm experiencing following issue when Docker Extension is used:
[ERROR] New-AzureRmResourceGroupDeployment : 3:13:27 PM - VM has reported a failure
[ERROR] when processing extension 'DockerExtension'. Error message: "Enable Docker
[ERROR] failed: ERROR: executing /bin/sh [-c curl -sSL https://get.docker.com/ | sh]
[ERROR] failed: exit status 1".
May be there are some issue while trying to download Docker bits ?
Thanks

It seems this issue is related to CenOs 7.2.
No issues with Ubuntu 16.04

After some diggings, it seems CentOs has updated a package and OpenLogic repository is still point to the old one.

Issue still available

Can you check again, this should be resolved now.

Confirmed. Docker extension on CentOs now work again.
Packages have been updated


help

Describe your problem here.
so i just changed the psych name to db. but i cant compile the game. pls help
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
Yes, i edited mainmenustate.hx to change the engine name and project.xml to change the app name.

Did it leave a error when building/compiling the game? If yes, show it.

no

so i just changed the psych name to db. but i cant compile the game. pls help

Give more information. Did you change it in TitleState.hx?
MainMenu.hx?
Project.xml?
And where did the error happen?
Opening the game?
In the Main Menu?
Going into a song?
etc.

Wait…. It cant compile the game?

Is it you first time compiling?

yapp

You must wait. Compiling might take 5 mins to 4 hours.

You must wait. Compiling might take 5 mins to 4 hours.

Sometimes an entire day…

😳


Update packages

close #209

Codecov Report

Merging #213 (7a27fda) into develop (e28f89d) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           develop      #213   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           44        44
  Lines         1186      1186
=========================================
  Hits          1186      1186

Flag
Coverage Δ

unittests
100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us.


Get-Counter corrupts/disables PerfLib or returns invalid data

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
When querying more counters at once and one of them is [User Input Delay Per Session] counterset, it fails with breaking the perfmon functionality (corrupted/disabled PerfLib, etc...).
It happens in some specific combinations like [User Input Delay Per Session] and [Memory] or [System] countersets. In such combinations invoking Get-Counter will terminate powershell process or returns non-zero Status property for the CounterSamples.
[User Input Delay Per Session] counterset works with .e.g. [SMB Server] counterset so not all are affected.
When two different get-counter cmdlets are invoked ([User Input Delay Per Session] separately), all counters are returned without issues.
Tested on Windows 2019, Powershell 5.1 and 7.2.2, same behavior.
Expected behavior
When querying array of counters in one "dataset" and [User Input Delay per Session] is used, valid data should be returned without needing to read [User Input Delay per Session] separately.
So this should work:
(Get-Counter -Counter @('\smb server\receive bytes/sec','\memory\page faults/sec','\user input delay per session(max)\max input delay') -SampleInterval 1 -MaxSamples 1).CounterSamples|Select InstanceName,Path,CookedValue,Status

Actual behavior
When querying array of counters in one dataset and [User Input Delay per Session] is used, it starts to return invalid data or terminates powershell completely.
Issuing command:
(Get-Counter -Counter @('\smb server\receive bytes/sec','\memory\page faults/sec','\user input delay per session(max)\max input delay') -SampleInterval 1 -MaxSamples 1).CounterSamples|Select InstanceName,Path,CookedValue,Status

Returns error and non-zero STATUS for MEMORY Counters:
Get-Counter: The data in one of the performance counter samples is not valid. View the Status property for each PerformanceCounterSample object to make sure it contains valid data.
InstanceName Path                                                         CookedValue     Status
------------ ----                                                         -----------     ------
             \\ov2p2194\smb server\receive bytes/sec                                0          0
             \\ov2p2194\memory\page faults/sec                                      0<PHONE_NUMBER>
max          \\ov2p2194\user input delay per session(max)\max input delay           0          0

Error details
Exception             :
    Type    : System.Exception
    Message : The data in one of the performance counter samples is not valid. View the Status property for each
PerformanceCounterSample object to make sure it contains valid data.
    HResult : -2146233088
CategoryInfo          : InvalidResult: (:) [Get-Counter], Exception
FullyQualifiedErrorId : CounterApiError,Microsoft.PowerShell.Commands.GetCounterCommand
InvocationInfo        :
    MyCommand        : Get-Counter
    ScriptLineNumber : 1
    OffsetInLine     : 2
    HistoryId        : 36
    Line             : (Get-Counter -Counter @('\smb server\receive bytes/sec','\memory\page faults/sec','\user input
delay per session(max)\max input delay') -SampleInterval 1 -MaxSamples 1).CounterSamples|Select
InstanceName,Path,CookedValue,Status
    PositionMessage  : At line:1 char:2
                       + (Get-Counter -Counter @('\smb server\receive bytes/sec','\memory\page …
                       +  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    InvocationName   : Get-Counter
    CommandOrigin    : Internal
ScriptStackTrace      : at <ScriptBlock>, <No file>: line 1
PipelineIterationInfo :

Environment data
Name                           Value
----                           -----
PSVersion                      7.2.2
PSEdition                      Core
GitCommitId                    7.2.2
OS                             Microsoft Windows 10.0.17763
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals

When [user input delay per session] removed, returned values are OK:

Seems no interest to this issue from MS side? Thanks for some update.

Another source of the same issue when called via C#:
https://stackoverflow.com/questions/66333570/performancecounter-user-input-delay-per-session-disappears

I'm not able to repro this with 7.3, there was a change to how the cmdlet interops with the perf API, can you see if this repros for you with 7.3?

@SteveL-MSFT Hello Steve
Unfortunately not fixed (OS Win 2019, Posh 7.3.0-preview8):

@ExSport Please share Get-Error output.

Get-Error output:
Exception             :
    Type    : System.Exception
    Message : The data in one of the performance counter samples is not valid. View the Status property for each
PerformanceCounterSample object to make sure it contains valid data.
    HResult : -2146233088
CategoryInfo          : InvalidResult: (:) [Get-Counter], Exception
FullyQualifiedErrorId : CounterApiError,Microsoft.PowerShell.Commands.GetCounterCommand
InvocationInfo        :
    MyCommand        : Get-Counter
    ScriptLineNumber : 1
    OffsetInLine     : 2
    HistoryId        : 4
    Line             : (Get-Counter -Counter @('\smb server\receive bytes/sec','\memory\page faults/sec','\user input
delay per session(max)\max input delay') -SampleInterval 1 -MaxSamples 1).CounterSamples|Select
InstanceName,Path,CookedValue,Status
    PositionMessage  : At line:1 char:2
                       + (Get-Counter -Counter @('\smb server\receive bytes/sec','\memory\page …
                       +  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    InvocationName   : Get-Counter
    CommandOrigin    : Internal
ScriptStackTrace      : at <ScriptBlock>, <No file>: line 1
PipelineIterationInfo :

I spent some time debugging this.  The actual error being returned as part of the PDH sample is PDH_CSTATUS_NO_OBJECT.  I can easily repro this with:
(Get-Counter -Counter @('\memory\page faults/sec', '\user input delay per session(max)\max input delay') -SampleInterval 1 -MaxSamples 1).CounterSamples

Same with PS7 and 5.1.  So this is not a regression.  The interesting thing is if I use perfmon with the same two counters, I see the same problem.  No data is returned for page faults/sec.  However, if I ONLY have that counter (either the cmdlet or in perfmon), it shows data.  But in perfmon if I add back the second counter, it then doesn't show any data for the first one.  Based on the results of perfmon (and the cmdlet is consistent here), it seems that the problem is with the counter being used with another counter.

@SteveL-MSFT You can combine more counters without issues but when multiple ones are used, you can't use "new" [user input delay] one which will affect other ones in most cases (as described in the first post):
https://learn.microsoft.com/en-us/windows-server/remote/remote-desktop-services/rds-rdsh-performance-counters
Without this one, you are able to read more counters at once or your are able to read this one alone or with e.g. SMB counters but not with other ones like memory ones, etc.
It is possible to read them "together" in separate threads via RunSpace/PSJobs so you can read them almost at once, without any crashes but they can't be queried in one request.
It is not a flaw in PowerShell cmdlet only (all versions) but in perfmon API or the way how PowerShell/.net/C#/... queries these counters.


Navigate to Dashboard if AUTH token is not expired

Describe the bug 🐛
Direct Navigate to Dashboard if user AUTH token is not expired

I would like to solve this issue! Could you assign it to me?

Sure!

@goliakshay357 I'd like to work on this.

Hello @IshanManchanda,
Thanks for your interest. Looking forward to your contribution :smile: .

@goliakshay357 I'm assuming routes 'home', 'login', and 'sign-up' should redirect to dashboard, is that right?

Yes


Mark the Business Coronavirus Support Finder as a draft

Trello: https://trello.com/c/kjtIlrE8/212-bs-smart-answer-set-up-a-prefix-redirect
This flow is currently offline as it contains out of date policy. It is
expected to return in the coming days.
⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠

Is this the same as doing this: https://github.com/alphagov/smart-answers/blob/b78e022315a6ce28108301638b09da6d4f1[…]3/docs/smart-answer-flow-development/retiring-a-smart-answer.md
Sorry, I’ve forgotten how smart-answers work.

Is this the same as doing this: https://github.com/alphagov/smart-answers/blob/b78e022315a6ce28108301638b09da6d4f1[…]3/docs/smart-answer-flow-development/retiring-a-smart-answer.md
Or has that already been done?
Sorry, I’ve forgotten how smart-answers work.

I've already done the unpublishing step from that document. However I didn't follow the process of deleting the files for the Smart Answer as it seemed so likely that we'd need to revert it.
There doesn't seem to a particularly great process to temporarily take down one. I thought using a draft was a good approach so then it couldn't accidentally be published by the sync task.


need to dump some ideas, niggles, code snippets etc

@chrisns and not sure of the protocol, process. should I create a page to dump stuff in and you can cherry pick anything you think has value and formalise it into the readme or indeed some working code? Am playing the part of someone new to your product and thinking of all the questions id be asking, want to capture them

join the slack and chat there is probably the easiest immediate direction
We need to figure out the formats and where things fit and how to organise it all for (for a first stab) so we can talk through what is in your head and find the best place to capture it :)

ta


dart pub --disable-dartdev-analytics works but dart --disable-dartdev-analytics pub doesn't

This is because the vm doesn't recognize the flag, but this flag belongs to the main dart command, not the subcommand.
This might not be urgent, as this flag is not exposed by default.
But it makes embedding the dart tool into eg. flutter awkward.
See https://github.com/dart-lang/pub/issues/2736

Filed: https://dart-review.googlesource.com/c/sdk/+/171284

@sigurdm is this still relevant?

No! It was fixed with: https://dart-review.googlesource.com/c/sdk/+/171284 !


CNN custom implementation

Hello,
What are the differences between CNN custom implementation (Conv1d, BatchNorm1d...) and PyTorch implementation (torch.nn.Conv1d, torch.nn.BatchNorm1d...).  I've noticed that custom cnn are used in the ECAPA-TDNN model.  Is the output of custom layers the same as PyTorch?
Thanks!

Hi, it is mostly linked to: 1. input shape might differ; 2. Padding is handled in SpeechBrain. But internally, we call the same torch.conv layers. It's just a matter of compliance with the input Tensor format of SpeechBrain that is the same across ALL the layers (not the case for Torch ..)

Thanks, @TParcollet!


Add option for custom elements

Add option for custom elements.
Also commits previous pull request.
Inteded for SVG, but can be used for anything really...
SVG's color + width/height is automatically set.
Basic implementation of #185 and #48
Usage:

This can be closed. HTML was add in #217


Update documentation with link to benchmarks and architecture

Fixes #537

@sarahwooders Is there anything else we should cover in the README? I'm adding a link to the overview video as well.


unicodeEscapeDict function assumes all keys are strings

Came across this issue today.
When attempting to update a dictionary object in the database, the DB class calls the unicodeEscapeDict function:
https://github.com/z1pti3/jimi/blob/e79aed10890b4af0c94adf61f4325d8982ffef39/core/db.py#L117-L122
However, this assumes that all keys in the dictionary are strings, as it attempts to do a replace on the key on line 345.
https://github.com/z1pti3/jimi/blob/e79aed10890b4af0c94adf61f4325d8982ffef39/core/helpers.py#L342-L358
This function should probably check that the key is a string, or skip this check. I think a try-except would be the most efficient, but @z1pti3 will know best!

 File "C:\Users\patrick\source\repos\jimi-opensource\core\db.py", line 121, in update   File "C:\Users\patrick\source\repos\jimi-opensource\core\helpers.py", line 327, in unicodeEscapeDict     newKey = key.replace(".","\\u002E").replace("$","\\u0024") AttributeError: 'int' object has no attribute 'replace'

This set of code is a bit inefficient as well as this replace is run on every key pulled back, do we have a option around making this code only execute when needed?

As suggested a try except would be the most efficient for this given the use of int keys. I do agree with unicorn that we should be looking at how we can reduce the calls made to replace on every document we load.


Can't recover from state loss after close/update and before settle

If a pair of nodes call closeChannel/updateNonClosingBalanceProof, but then for any reason lose the state (namely, transferred_amount, locked_amount & locksroot of each side which produced the balance_hash of the close/update calls), the channel can never be settled, the funds stay forever locked in the contracts, and that pair of nodes can never have a new channel between them on the same TokeNetwork.
Of course we don't expect both nodes to lose state often, but nonetheless IMO this lockout scenario isn't desired in any situation, and maybe we can consider this a bug to be fixed.

@andrevmatos is your focus more on not being able to recover the funds or not being able to open a new channel within that pair?

[ ] Discussion: Is the assumption still valid in the current raiden
[ ] Is this a problem which we want to cover somehow?
[ ] If yes, make a proposal how to solve this issue

I think both, but on our testnet testings, what was more annoying was not being able to open a new channel within that pair.
The main point of this issue is that closing/updating with hashes, but requiring the actual values to settle, works as an unintended secrecy mechanism between closing and settling. Although settling is expected to be able to be performed by anyone, in practice, current method can only be called by someone who knows the amounts which composed both side's balanceHash.

In a discussion with andre he stated that this problem appears to be more relevant in the LC since it only stores the latest balance hashes. This issue is addressed in https://github.com/raiden-network/light-client/issues/1607

That LightClient issue isn't a replacement this one. It was just a new issue we discovered about our LC implementation upon thinking more about it in the light of this one, but this is a different issue, which can affect not only the light-client.
It happens if both nodes lose state (e.g delete ~/.raiden) after close/update, but before settling a channel. Of course, it isn't expected for this to happen and surely not advisable, but IMO this doesn't excuse the contracts from holding the funds forever with no way to settle this, and also preventing the peers from connecting again on this token network.
This issue happen because on close/update, the nodes set a balanceHash, which is the hash of 3 fields: transferredAmount, lockedAmount and locksroot, on each other's side of the channel, and this works as a kind of secret. After settleTimeout, anyone can call settleChannel, but need to know the 6 values which generated the hash stored for both sides. If the state is lost and the peers can't know the values which composed the hashes, the SC don't allow anyone in the world, not even the peers accounts, to settle the channel, the funds stays locked forever on it and the peers can never reopen a channel on this network with each other again.

We could still allow a cooperative settle in that case. That would nicely solve the issue without introducing much additional code to write and test and even allows the nodes to decide on the fair split in this case in a flexible way.
I don't think we need to provide a way for totally uncooperative nodes to resolve this rare case. If the alternative is to not get any funds out, the nodes should be willing to cooperate somehow.

other options

allow to open concurrent channels (n closed, not settled, channels can coexist with one open channel)
allow for a default settlement (back to initial deposits) after waiting, e.g. 10 * settlement_timeout
allow for full balance proofs at closing time
don't fix it: addresses are cheap

For this current sprint, we want to come to a decision on the way forward.

Looking through the TokenNetwork code, it seems like withdraws are possible for closed (but not settled) channels. So the tokens are not lost if both participants can agree on the balances. If both participants lost their dbs, I would expect them to revert to the on-chain balances and would thus be in agreement about the balances. This is similar to what I wrote about coop settle above, but more interesting, since we already have working withdraws.
This does not help with the "we can't get an open channel between those participants again"-problem, though.

Ok, after a call with @karlb , we understand that allowing for a default settle after no matter how long timeout isn't advisable, since the nodes are responsible for settling (there's no MS for settle) and a node which knows their partner will be offline for long can take advantage this to do a unilateral settle.
Full BPs at closing time has the drawback of being less private, since the MSs would need to not only be provided with hashed/shadowed balanceHashes but actually with full values which would reveal too much about the transfer history.
So the best way which don't put user's funds at risk (at least not anymore than they would already be on state loss) would be cooperative settle. Possibly we could have normal coopSettle (like one done without close, to speed up settling, as in commented code in current contracts) but taking care of allowing it also after close/update, but regardless of its values, so a coopSettle can be performed with zeroed/agreed values even on this unexpecter state, at any point in time (before or after close, before or after settleTimeout).
Related #399


Add support for EIP-1191

Non-breaking change.
This will help differentiate between networks and should be encouraged.

push CI has passed with 100% coverage and previous tests all still passing as-is, the correctness of the new feature is tested by an official test vector from the EIP.
I'm going to merge this, publish as minor update. (Wish I had learned about this EIP before merging the previous update)


As a user I would like to use the graphs on the prev page for presentation (high resolution, big fonts)

I would like to discuss the possibility to download graphs with higher resolution and with bigger fonts.

@mahtabiltarabian The text in the graphs is still a bit small. Can we make it even bigger?

yes of curese, now please check this

The layout of tge graph has changed when I download it. It does look abit squised now. We will need even better resolution of the plot. I would like to reopen the ticket therefore.


Calling dict() on some awkward.highlevel.Array object leads to ValueError.

So sometimes calling dict(ak_array) gives an error.

In [10]: import awkward as ak
    ...: print(ak.__version__)
    ...: dog = ak.from_iter([[1., 2.], [5.]])
    ...: pets = ak.zip({"dog": dog[np.newaxis]}, depth_limit=1)
1.1.2

In [11]: dict(pets)
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-11-f7ecb9a598b4> in <module>
----> 1 dict(pets)

~/Programs/anaconda3/envs/tree/lib/python3.7/site-packages/awkward/highlevel.py in __getitem__(self, where)
   1765             2
   1766         """
-> 1767         return ak._util.wrap(self._layout[where], self._behavior)
   1768
   1769     def __setitem__(self, where, what):

ValueError: scalar Record can only be sliced by field name (string); try "0"

(https://github.com/scikit-hep/awkward-1.0/blob/1.1.2/src/libawkward/array/Record.cpp#L198)

It is more than possible I have done something odd with my packages, so let me know if you don't reproduce it. Also apologies if it's already been reported.
Side note for future readers; if you got here with the same goal I had then you wanted the list of column names, it's ak.fields(pets).

I don't know what you want dict(ak_array) to do. Technically, what happens is that the dict constructor iterates over what it's given and tries to cast each item as a 2-tuple to use as key-value pairs. Some Awkward Arrays will have that structure and some won't.
Are you wanting to turn the Awkward Array into Python lists and dicts? That's ak.to_list or ak_array.tolist() as a method (following NumPy's naming convention).
>>> pets
<Array [{dog: [[1, 2], [5]]}] type='1 * {"dog": var * var * float64}'>
>>> pets.tolist()
[{'dog': [[1.0, 2.0], [5.0]]}]


SP parameters

ARFlow is a great job. I learn a lot from it.
I want to know that the parameter in
Sintel_ft_ar.json : line 61
"rotate": [-0.2, 0.2, -0.015, 0.015]
and sp_transfroms.py: line 229
phi.uniform_(-min_rotate, max_rotate)
So the rotation is always 0,2 or 0.015. Why not set it as a random number like "trans" or "zoom"?

It seems like a bug, and the correct one should be phi.uniform_(min_rotate, max_rotate)
Thanks for you pointing it out, I will test random rotation and update the results.

Sorry for the late reply. I have tested the correct one and the results are almost unchanged. You can re-open this issue if there are any other problems :)

Many thanks for your update!
Btw, I am working on adapting ARFlow to my work. I found that the parameters of the second transformation are quite small, e.g. 0.015 vs 0.2. I want to know why not set it larger as it can introduce much more noise. Because I only have a few GPUs, I could not test it. Would you mind giving me some hints about it?
Thanks.


First basic data types

This PR adds basic data types for the documentation. This types provides the needed metadata to create a table of contents for the manual.

@vanross @Plexar Maybe you can do a review? :)


Consider using Svelte's compiler.mjs

Encountered your project while experimenting with similar use cases.
Svelte production build features the compiler itself as a one-file module (compiler.mjs) including pre-bundled dependencies, so I think it is possible to ditch the existing compiler/core.js, unless they are some pros I'm not aware of.
This will help maintaining Snel without rebuilding the core.js every time Svelte releases a new update.

Hello, sorry for the slow response, I was testing and it works great, that would solve very easily update to more recent versions of svelte

you can update your branch again, delete the provider folder (it was hotfix) and use<EMAIL_ADDRESS>

I already replaced the compiled file for the cdn, btw thank you very much for the contribution, any other help or improvement will be well received


Fix joining fabric print in ChipDeviceController

Problem
The print Joining the fabric at index 1. Compressed fabric ID is: 0x00000000 would always emit a value of 0 for the compressed fabric ID. This is because GetCompressedFabricId eventually calls GetFabricInfo, which checks if mState is State::Initialized to return a valid value.
The print is being called from within InitControllerNOCChain, which is well before the state gets set to initialized.
Fix
Just move the print.

By the new criteria for inclusion to 1.0, this PR will not be justifiable. Closing it out and will re-open once we branch for 1.0.


Feature/is first time object

ending the tutorial is saved (context)
the player doesn't have to cancel the new tutorial modal


allow manual recoloring of taxa

would be ideal to use something similar to the color palette in emperor, where clicking on a sample's color box brings up a palette. Here's a screenshot:

Could possibly also support patterning of bars? I hate how patterned barplots look, but this is useful, e.g., if a figure needs to be greyscale, or if there are just a few too many taxa for colors to distinguish. I'd rate this as very low priority, far below manual color selection, but could be useful for some users eventually.

forum xref
Though in that user's case, it sounds like a more useful feature might be using some sort of preferences/feature metadata file to specify color settings for each taxon. @thermokarst should I raise a separate issue for this? (the idea would be to save/load color settings)

Good point @nbokulich - this actually sounds more like a framework level issue, for supporting a new metadata directive for colors (like the new #q2:types directive released in 2018.2). Then, we can start making visualizations aware of that directive.