Removal of e-Tugra root certificate

Certifi 2023.07.22 removes root certificates from "e-Tugra" from the root store. These are in the process of being removed from Mozilla's trust store.

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

Comparison is base (beb44ca) 91.38% compared to head (fc3ea32) 91.38%.
 Consider uploading reports for the commit aa08987 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #2563   +/-   ##
========================================
  Coverage    91.38%   91.38%
========================================
  Files          454      454
  Lines        17189    17189
========================================
  Hits         15708    15708
  Misses        1481     1481

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


Use aggregate pool in Spark query runner

In #10568, an 'aggregate' pool is passed to query runner's constructor. This PR
uses it to create child pools in the Spark query runner.

@kgpai A follow up to https://github.com/facebookincubator/velox/pull/10790. Would you take a review? Thanks!

@mbasmanova This is a tiny fix for the merge conflict of https://github.com/facebookincubator/velox/pull/10357 and https://github.com/facebookincubator/velox/pull/10568. Would you help take a review? Thanks.


SAP Community // Profile Tutorial

Resized the images (they're quite blurry now though).

You didn't have to make a real change. You could have just added an unimportant space anywhere :-)


in Home Component,  can't use <>: Fragment.

I always use jsx-slack for this kind of situation.
It is very helpful because I can write Block Kit very sensibly.
Thank you very much.
On topic, I tried to use Fragment when describing a component in the Home Block this time, but I couldn't describe Fragment using the <> notation that I always use.
After a while, I wondered if Fragment would work, so I modified the code and the jsx-slack transpile went through.
Is this a specification?
I have confirmed that the original <> works for .
If you know anything about this, I would like to know.
Thanx.

The parent container block do never restrict using <>. I guess the cause of not working <> is in your JSX transpile setting or your source code, not in jsx-slack. How do you have set up JSX transpiler?
If you're using TypeScript

When react-jsx/react-jsxdev mode is selected in jsx compiler option of tsconfig.json:

:arrow_right: Check whether jsx-slack has defined in either of jsxImportSource compiler option or /** @jsxImportSource **/ comment pragma.

When react mode is selected in tsconfig.json (classic way):

:arrow_right: Check whether JSXSlack.Fragment has defined in either of jsxFragmentFactory compiler option or /** @jsxFrag **/ comment pragma. It requires TypeScript v4 and newer, and needs to import JSXSlack at the source code like below./** @jsx JSXSlack **/
/** @jsxFrag JSXSlack.Fragment **/
import { JSXSlack } from 'jsx-slack' /* <- Required to use JSXSlack.Fragment */

If you're using jsxslack template literal
You don't have to use fragments. It supports multiple elements in the top level.

Thanx 4 your rapid reply @yhatt !!
This is my source code
/** @jsx JSXSlack **/
/** @jsxFrag JSXSlack.Fragment **/
import { Divider, Fragment, Header, Image } from 'jsx-slack';
import { JSXSlack } from 'jsx-slack'; /* <- Required to use JSXSlack.Fragment */

export const HomeHeader = () => {
  return (
    <>
      <Image src="https://source.unsplash.com/random/960x240?home" alt="home" />
      SlackApp
      <Divider />
    </>
  );
};

And this is tsconfig.json
{
  "compilerOptions": {
    "module": "commonjs",
    "jsx": "react-jsx",
    "jsxImportSource": "jsx-slack",
    "jsxFragmentFactory": "jsxFragmentFactory",
    "declaration": true,
    "removeComments": true,
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,
    "allowSyntheticDefaultImports": true,
    "lib": ["es2017", "dom"],
    "target": "es2017",
    "sourceMap": true,
    "outDir": "./dist",
    "baseUrl": "./",
    "incremental": true,
    "skipLibCheck": true,
    "strictNullChecks": false,
    "noImplicitAny": false,
    "strictBindCallApply": false,
    "forceConsistentCasingInFileNames": false,
    "noFallthroughCasesInSwitch": false
  }
}

I'll take your advice.
I changed jsxFragmentFactory to "jsxFragmentFactory".
However, I am getting the following error
Is tsconfig.json wrong? Thanx.
[0:00:50] File change detected. Starting incremental compilation...

error TS5052: Option 'jsxFragmentFactory' cannot be specified without specifying option 'jsxFactory'.

error TS5089: Option 'jsxFragmentFactory' cannot be specified when option 'jsx' is 'react-jsx'.

[0:00:50] Found 2 errors. Watching for file changes.

It looks like you are using react-jsx mode. In this case, @jsx, @jsxFrag comment pragma and jsxFragmentFactory option cannot use because they are only for react mode. Just forget previous advices 🤷
Following codes are minimum example to work <> syntax in react-jsx mode. Empty import for jsx-slack may resolve your problem.
// tsconfig.json
{
  "compilerOptions": {
    "jsx": "react-jsx",
    "jsxImportSource": "jsx-slack"
  }
}

// May require empty import at least, to make TypeScript compiler recognize jsx-slack.
import {} from 'jsx-slack'

export const Example = () => (
  <>

Test

Hello!

  </>
)

UPDATE: Some of transpilers might have been updated to depend on Fragment that is not exported by jsx-slack JSX runtime. I've opened new issue in #267.

Lacked Fragment export in JSX runtime (jsx-slack/jsx-runtime) has been fixed in the latest update jsx-slack v4.6.1. TypeScript should resolve Fragment from JSX runtime automatically when you tried to use <> in your TSX code.


upgrade to snabbdom 6.1

I fixed a bug in the htmldomapi.ts in snabbdom that caused SVG's to fail in IE11 when removing elements.
This has been fixed, and a new version has been published, please update @cycle/dom to snabbdom 6.1
https://github.com/snabbdom/snabbdom/pull/210

Duplicate of issue #433.
We'll do the update in the context of Cycle Unified, because snabbdom v0.6.1 uses TypeScript v2.1 and we can't just update snabbdom without updating all our packages to TypeScript v2.1 too. We're doing that in Cycle Unified. TypeScript v2.1 is a breaking change and affects a lot of stuff.


feat: add more checks to util.js

Problem
util.js should cover more type checks
Solution

[ ] add checks for sdk types
[ ] add checks for js types
[ ] add unit tests for util.js

Alternatives
No response

if I made a unit test for util.js - #675 wouldn't exist


Trial replaying

Currently, replaying a trial takes ~2-4 seconds per epoch (I'm using a hand-made Tqdm module for Jupyter, so the output below may look a bit different):
0/5(v): 100%| 263/263 [00:00<00:00, 605.24it/s] - val_acc: 0.962 - val_acc_std: 0.190 - val_loss: 0.164 - val_loss_std: 0.186
1/5(t): 100%| 1050/1050 [00:03<00:00, 395.73it/s] - running_acc: 0.961 - running_loss: 0.153 - acc: 0.973 - acc_std: 0.162 - loss: 0.102 - loss_std: 0.132
1/5(v): 100%| 263/263 [00:01<00:00, 396.25it/s] - val_acc: 0.964 - val_acc_std: 0.187 - val_loss: 0.156 - val_loss_std: 0.176
2/5(t): 100%| 1050/1050 [00:03<00:00, 377.02it/s] - running_acc: 0.966 - running_loss: 0.124 - acc: 0.973 - acc_std: 0.163 - loss: 0.095 - loss_std: 0.124
2/5(v): 100%| 263/263 [00:01<00:00, 300.80it/s] - val_acc: 0.967 - val_acc_std: 0.178 - val_loss: 0.143 - val_loss_std: 0.160
3/5(t): 100%| 1050/1050 [00:04<00:00, 294.61it/s] - running_acc: 0.976 - running_loss: 0.083 - acc: 0.973 - acc_std: 0.161 - loss: 0.092 - loss_std: 0.117
3/5(v): 100%| 263/263 [00:01<00:00, 304.99it/s] - val_acc: 0.968 - val_acc_std: 0.175 - val_loss: 0.144 - val_loss_std: 0.178
4/5(t): 100%| 1050/1050 [00:03<00:00, 405.98it/s] - running_acc: 0.978 - running_loss: 0.088 - acc: 0.973 - acc_std: 0.163 - loss: 0.098 - loss_std: 0.131
4/5(v): 100%| 263/263 [00:01<00:00, 421.27it/s] - val_acc: 0.966 - val_acc_std: 0.182 - val_loss: 0.158 - val_loss_std: 0.177
<torchbearer.trial.Trial at 0x7f1dc6e7d4a8>

Do you think it'd be possible to do something like self._loader.update(len(<dataloader>)) in the printing callback on every epoch to avoid the delay?
Thanks!

Currently we track how many steps were performed in the run call and call all the callback methods that would be called in the run, including, for example, on_sample once per batch/step. This is so that any callbacks that might update each batch can still do so in a replay.
We'd like to keep this behaviour but I see no reason we can't add a flag that tells the replay method to only call one batch worth of calls. How does this sound @ethanwharris @kl2792 ?

@MattPainter01 Sorry about the late reply, things have been hectic recently. That sounds great; thank you!


Add Team Description to UI

Codecov Report
Base: 39.50% // Head: 39.47% // Decreases project coverage by -0.03% :warning:

Coverage data is based on head (e1d0012) compared to base (eb6179c).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #119      +/-   ##
============================================
- Coverage     39.50%   39.47%   -0.04%
  Complexity      452      452
============================================
  Files           138      138
  Lines          2567     2569       +2
  Branches        324      324
============================================
  Hits           1014     1014
- Misses         1488     1490       +2
  Partials         65       65

Impacted Files
Coverage Δ

...rc/main/java/seedu/address/ui/TeamDetailsCard.java
0.00% <0.00%> (ø)

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

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


Promise.prototype.finally is not aligned with the most current spec draft

According to draft 4 of the Promise.prototype.finally spec , the callback parameter might not be callable.
Spec: https://tc39.es/proposal-promise-finally/
Similar issues: https://github.com/stefanpenner/es6-promise/issues/336

I would also like to see more info about finally polyfill implementation.
For example FF 2-57 (a very large group of FF versions still used a lot), implement Promises natively, but don't implement Promise.finally. Looks like this polyfill does a nice job of detecting that and polyfills finally separately on a native Promise, but it needs clarification in the README if possible please.

Thanks for bug report. Can you create a PR? I will happily merge it.

Closing because of inactivity. Please feel free to create a PR with a fix


Remove bash scripts

Removes old bash scripts used for installing pre-reqs and
installing / uninstalling P4CP4D

Released 1.0.1


Planning: Tool calls Keptn or Keptn calls Jenkins

Signed-off-by: Meg McRoberts<EMAIL_ADDRESS>This captures a response @agardnerIT gave in https://keptn.slack.com/archives/CNRCGFU3U/p1661478530360699 .
This is useful information that is not otherwise included in the documentation.  Feel free to refine the text

I would flip the sentences around to have the current method (and code snippet) first. The shared library is only applicable to Keptn versions 0.12.x and older so that should go further down the page.

The current "best practice" is for tools (Jenkins) to be triggered from Keptn sequences (so Jenkins would respond to the task event). Wording to that effect should go first. If a user really does want to integrate "the other way around" (for example to trigger only a quality gate from Jenkins) they still need a Keptn sequence with a single evaluation task. In that respect, they don't "gain" anything and, it could be argued, actually lose something because they need to code logic into their pipeline to wait for the evaluation result. All-in-all it's a harder, more legacy approach.

Thanks, Adam.  I rewrote as you suggested and made the prose a bit more opinionated about which approach is preferred.


Import accounts from Metamask

Can't build. Added some comments.
There are artefacts SwiftStore/SwiftStore/Store.mm, please review changes before submitting a pull-request.


Destination role ARN not working for cross-account

I'm trying to setup a mapping that works with cross-account support. When I register the mapping and try to use the --destination-role-arn, the response is the following:
Invalid ARN 'arn:aws:iam::012345678910:role/fanoutCrossAccountRole', must be a fully qualified AWS IAM Role ARN
The format I'm using for the register action is as follows:
./fanout register kinesis --function fanout --source-type kinesis --source source-stream --id target1 --destination arn:aws:kinesis:us-west-2:012345678910:stream/target-stream --destination-region us-west-2 --destination-role-arn arn:aws:iam::012345678910:role/fanoutCrossAccountRole --active true --profile redacted --region us-west-2

However, the following does work including the --destination ARN, but it does not provide cross-account support because it's missing the --destination-role-arn, and is really just to prove that the ARN format is correct:
./fanout register kinesis --function fanout --source-type kinesis --source source-stream --id target2 --destination arn:aws:kinesis:us-west-2:012345678910:stream/target-stream --destination-region us-west-2 --active true --profile redacted --region us-west-2

Is the --destination-role-arn feature for cross-account support functional? If so, what am I missing?

I see now that the conditional on line 248 of targets.sh is the following:
if [ ! -z "${DESTINATION_ROLE_ARN}" ]; then
    DESTINATION_ROLE_NAME=$( echo "$DESTINATION_ROLE_ARN" | sed -E -n 's#^arn:aws:iam::role/([a-zA-Z0-9+=,.@_-]{1,64})$#\1#p' )
    if [ -z "${DESTINATION_ROLE_NAME}" ]; then
      echo "Invalid ARN '${DESTINATION_ROLE_ARN}', must be a fully qualified AWS IAM Role ARN" 1>&2
      exit -1
    fi
fi

when it should be the following:
if [ ! -z "${DESTINATION_ROLE_ARN}" ]; then
    DESTINATION_ROLE_NAME=$( echo "$DESTINATION_ROLE_ARN" | sed -E -n 's#^arn:aws:iam::[0-9]{12}:role/([a-zA-Z0-9+=,.@_-]{1,64})$#\1#p' )
    if [ -z "${DESTINATION_ROLE_NAME}" ]; then
      echo "Invalid ARN '${DESTINATION_ROLE_ARN}', must be a fully qualified AWS IAM Role ARN" 1>&2
      exit -1
    fi
fi

Note the DESTINATION_ROLE_NAME variable line and how it's missing the regex for the account number in the original.
With this update I no longer receive the message about the ARN not being fully qualified when registering the mapping, however, I also don't see any update to the DynamoDB table that suggests the destination role has been configured to be used.

Also discovered that line 83 of configuration.js is missing the regex for the account number, and this prevents any cross-account access.
var roleRegex = /^arn:aws:iam::role\/([a-zA-Z0-9+=,.@_-]{1,64})$/;
should be
var roleRegex = /^arn:aws:iam::[0-9]{12}:role\/([a-zA-Z0-9+=,.@_-]{1,64})$/;
Once this is updated, cross-account access works.

Lastly, I've discovered that the following should be updated for targets.sh so that the role and externalId properties can be properly added to DynamoDB when registering a target using the fanout command:

Line 185 should contain the following line:
EXTERNAL_ID=
Starting at line 234 should be the following logic:

elif [ "$CODE" == "--external-id" ]; then
  if [ $# -ne 0 ]; then
    EXTERNAL_ID=$1
    shift
  else
    echo "readTargetParams: You must specify a value for parameter $CODE" 1>&2
    doHelp
    exit -1
fi

Lines 412 and 413 should be changed to:

if [ ! -z "${DESTINATION_ROLE_ARN}" ]; then
      appendJsonProperty "$1" "role" "{\"S\":\"${DESTINATION_ROLE_ARN}\"}"

I would create a pull request for this issue, but it appears the repo has push restrictions.
fatal: unable to access 'https://github.com/awslabs/aws-lambda-fanout.git/': The requested URL returned error: 403

PR accepted on this


Form model binding

Any plans to add form model binding? Thanks, this looks like a great package, looking forward to playing with it.

Yes, I'm planning to create entire module not only for binding model with forms but also generating forms from model. It will be a big feature so it needs time. Right now, I'm preparing video tutorial. It will include information about using Astronomy with the forms. You can also check out this sample application https://github.com/jagi/meteor-astronomy-examples
I hope you will like this package :). Let me know if there are any other features to implement or problem you've encountered.

Awesome, I look forward to it.

I've gotten a decent start on connecting with forms, though haven't thought about automatically building the forms.
The approach I've taken is two-fold:

Register a form for submit handling (with template name and form name); it attaches a submit event handler that automatically copies the fields from the form to a Class object (assuming matching names for now), then validates and if it passes, saves it.  It calls optional user-defined functions before (ex. for attaching an existing _id for edits or perhaps for setting other fields that aren't on the form) and after (ex. for going to a new route, etc.).

Register a form for error handling.  Haven't started this yet, but I think the approach is to check for each field whether there's a matching (by name/id) error span.  If there is, it will register onBlur events for those fields that will validate the field and update the error field text.

Happy to share the code, if it's helpful, though #1 was pretty straightforward, and I think #2 will be, as well.

Awesome, is this going to be a separate package?

@dovrosenberg I would be happy to see code and example use. I will start creating Form module for Astronomy soon so maybe we could cooperate in this matter. We could discuss details. I have some ideas about the forms implementation that comes from the usage of Doctrine library, but I don't want to discuss it here.

Are you guys part of the Meteor Club Slack channel?

No, what's that? :)

Any updates on this?

@jagi https://www.patreon.com/meteorclub

It's in its early phase. Right now and for next two weeks I'm out of my country so not developing anything. I will be notifying about the progress when I'm back.

Any updates on this? Thanks.

Sorry for the late response. Still working on it. I will also have to implement module for binding model to the view. Together with forms module it will be complex toolset.

@jagi we are soon to release a public version of our internal forms package (most up-to-date branch is https://github.com/usefulio/forms/tree/README.md), and will probably add a simple extension to support astronomy validation, I'll keep you updated.

Good to hear. I will take a look at out in free time. In the new astronomy release there will be a lot of changes so you should take it into account.

When is the new release?

Right now Astronomy version is 1.0.0-rc.5. It should achieve 1.0.0 in the next week.
It won't introduce form model binding and forms package. I will be developing forms module after releasing 1.0.0.
As @ianserlin stated, they made the forms package that looks promising. I like its form - model binding feature. It's definitely something worth including in the forms package for Astronomy. I will have to work closely with developers on making the forms package the right way :)

I'm closing this issue. I've prepared a list of feature requests that will appear in Astronomy 1.0 and Astronomy 2.0. Some of them are already being developed. Other will appear in near future in 1.0 or 2.0 version. I will be updating this list. I will probably have to create some tool for voting on features to implement.


empire error python 2.7

##Empire Version
latest from repository
##OS Information (Linux flavor, Python version)
Ubunut 14 , python 2.7
##Expected behavior and description of the error, including any actions taken immediately prior to the error. The more detail the better.
./empire
Traceback (most recent call last):
File "./empire", line 11, in
from lib.common import empire
File "/opt/Empire-master/lib/common/empire.py", line 25, in
import helpers
File "/opt/Empire-master/lib/common/helpers.py", line 52, in
import netifaces
ImportError: No module named netifaces
root@ExPl0I73R:/opt/Empire-master# pip install helpers
Collecting helpers
Could not find a version that satisfies the requirement helpers (from versions: )
No matching distribution found for helpers
root@ExPl0I73R:/opt/Empire-master# pip install netifaces
Requirement already satisfied: netifaces in /usr/local/lib/python3.4/dist-packages
##python version
2.7
Any help to get this run
Regards

You need to run the install.sh script


Fixing invalid filename on Windows preventing clone

The file Mini OLED Unit 0.42" 72x40 Display.md contains a " and the repo cannot be cloned under Windows.
I've replaced the " with inch in the file name - that's it.
Ste

Ah cool thanks for this, didn't know Windows didn't like cloning with speechmarks. Appreciate it @Tmp2k


no such instruction: `v4fmaddps (%rax),%zmm0,%zmm4'

See https://godbolt.org/g/R5pm1P:
#include` <immintrin.h>
__m512 knm(__m512 Vdata[4]){
        __m128 c;
        return  _mm512_4fmadd_ps(
                                _mm512_setzero_ps( ),
                                Vdata[0],
                                Vdata[1],
                                Vdata[2],
                                Vdata[3],
                                &c);
}

With compiler flags -mavx5124fmaps and compiler gcc 7.2 it outputs:
/tmp/ccMGJEbN.s: Assembler messages:
/tmp/ccMGJEbN.s:64: Error: no such instruction: `v4fmaddps (%rax),%zmm0,%zmm4'
Compiler exited with result code 1

A very strange one, the compiler emitting an instruction the assembler can't handle. I build the assembler alongside the compiler, so they should be in step.

Looks like I need to upgrade binutils. Will try with the nightly tonight and check tomorrow. If the nightly GCC starts working I'll rebuild the 7.x compilers.

Both gcc 7.1 and 7.2 has this error (-mavx5124fmaps is not available for gcc 6 and older).
gcc trunk is compiling successfully [https://godbolt.org/g/3uMvUQ] But without v4fmaddps instruction.
Strangely on my windows machine, gcc 7.2 compiles same code successfully with v4fmaddps instruction.

@Mandar12 I rebuilt the latest GCC with the latest binutils. It now compiles the instruction: but the objdump command doesn't recognise it, hence in your link there's an opcode that says (bad) .
I'll update the other GCCs at some point soon. I'm surprised at objdump - I build that alongside gas etc so it seems like either I'm running the wrong objdump or it doesn't support it.

@mattgodbolt looks like binutils issue. On my ubuntu setup, binutils version is 2.26.1. While on my windows setup, binutils version is 2.29.1.

(I have'nt made progress on this, so unassigning)

Fixed since GCC 7.3 -- let me know if you need it in older versions of GCC.


split out openscad/scad api to separate repository

This PR  goes in tandem with the new repository here https://github.com/jscad/jscad-scad-api

removed local code
modified a few imports
modified package.json with VERSIONED dependencies
it also means all tests for this repo are now passing :)

I like it but have some questions about the versioning. There are really two (or more versions) needed. One for the SCAD API supported or simulated. And one for the release. How to manage these?

The SCAD API  should  follow strict semantic versioning : so that any breaking change increase the major version number ie : go from 1.0.0 to 2.0.0 etc this is essential as it guarantees that any parts coded with that version will work with that version .
For this repo I am not entirely sure : it depends entirely on WHAT we consider it to be:  a front end for the SCAD version ? for the SCAD and csg.js ? What is the api of OpenJSCAD.ORG: version changes should reflect a level of 'safety' for users.
It does not impact this specific PR I think , since I added explicit SCAD api version number , so it locked for now :)
But I believe a discussion about this in the Issues is needed.
(Also I am preparing a small diagram to visualize more clearly the 'layers of the onion' ie what depends on what.


bug(mat-tab): mat-tab content disappearing when order changes, using Ivy

Overview
I am trying to implement a mat-tab-group with multiple tabs where the user can drag and drop the tabs to re-arrange their order. When a tab gets moved (via the drag and drop), the content of the other tabs will disappear (depending on which tab is moved). However I was only able to replicate this issue when Ivy was enabled.
Reproduction
Use StackBlitz to reproduce your issue:
https://stackblitz.com/edit/components-issue-vmj8fs
Steps to reproduce:

Open the StackBlitz and re-arrange the tabs by dragging and dropping. The content of each tab will be visible when you click between then.
Go to the Settings panel on StackBlitz and check Enable Ivy.
Once compiled, re-arrange the tabs again; the content of some of the tabs will disappear. (e.g. if you move tab 1 to the last position, tabs 2, 3, 4, and 5 will have no content).

Expected Behavior
The content of each tab to be preserved.
Actual Behavior
The content of tabs disappearing.
Environment

Angular: 11.0,4
CDK/Material: 11.0.2 both
Browser(s): Chrome
Operating System (e.g. Windows, macOS, Ubuntu): Windows

I believe that this is an issue in the framework. Potentially the same as https://github.com/angular/angular/issues/37895.

I'll close this since it appears to be the same issue as the linked one.


ENH: Issue warning when NIFTI IO coerces matrix to orthogonal.

addresses #3476

@jhlegarreta, updated commit message to be more informative.

While having a look at the coverage today, I realized that it looks like this commit introduced 66 Valgrind warnings :worried: :
https://open.cdash.org/viewDynamicAnalysis.php?buildid=8049748
https://open.cdash.org/viewDynamicAnalysis.php?buildid=8054367

Addressed in #3525


QLever IndexBuilder crashed reading an old 26Go nt triples files

2023-02-28 02:04:58.504	- INFO:  Processing input triples from /dev/stdin ...
terminate called after throwing an instance of 'ad_utility::Exception'
what():  This code should be unreachable. In file "/app/src/parser/RdfEscaping.cpp " at line 142
How can I print the problematic line ?

Can you provide a link to the dataset?

@fbelleau Thanks for the hint.
In addition, I will set up a fix for QLever that prints at least the faulty literal / IRI
I cannot promise that you can also get the corresponding line number and some characters around it,
but I will try.
Thanks for reporting this, an assertion with such a message should obviously not leak towards end users.


test: Move firewalld tests into one file

Moving firewalld related tests into one file should be convenient for the user who doesn't care firewalld.

ping.
Thanks!

@Mingli-Yu the tests are failing so there isn't much point for me to review this. It is up to you to fix that.
Also you have not fixed my earlier review comment with the $fw_driver in the test name which is not correct

@Luap99  Many thanks for your respond! I actually didn't fix any test cases right now, just try to group all firewalld related cases into one file. And latest PR did have the problem with the $fw_driver in the test name. Do you plan to group all firewalld related cases into one file? If yes, I will send a new PR then.


Implementing IntoResponse for tide::Resulttide::Response

Hey!
I have started to port https://github.com/colinbankier/realworld-tide from v0.3.0 to v0.4.0 - so far so good, no major issues.
I noticed that cargo build fails if I try to mount a route with following signature:
pub async fn list_articles(cx: Request<Repo>) -> tide::Result<Response> {
    [...]
}

with the following error
error[E0277]: the trait bound `std::result::Result<tide::response::Response, tide::error::Error>: tide::response::into_response::IntoResponse` is not satisfied
  --> src/main.rs:28:29
   |
28 |         api.at("/articles").get(web::articles::list_articles);
   |                             ^^^ the trait `tide::response::into_response::IntoResponse` is not implemented for `std::result::Result<tide::response::Response, tide::error::Error>`
   |
   = note: required because of the requirements on the impl of `tide::endpoint::Endpoint<db::Repo<diesel::PgConnection>>` for `fn(tide::request::Request<db::Repo<diesel::PgConnection>>) -> impl core::future::future::Future {web::articles::list_articles}`

I'd be happy to submit a PR to implement IntoResponse for tide::Result<tide::Response>, but I'd like to understand first if this implementation is missing on purpose due to some pending discussions around error handling in tide :grin:

@LukeMathWalker o/ thanks for raising this! To be honest I'm unsure what we should do around error handling. Right now we have a model of: "always cast to a Response". But I'm thinking it might make more sense if we'd expect something akin to Result<Response, BoxError> from each response. But then there's an open question on how to return 4xx/5xx status codes on error, which is tricky.
So the short answer is: I'm unsure too? Do you maybe have any thoughts?

I think it would make sense to allow routes to return anything that implements IntoResponse.
In particular Result<A, B> both A and B implement IntoResponse.
What is the issue with 4xx/5xx? Not sure I followed you there.

@yoshuawuyts I would love to see something like this. Returning a result, and letting users store a function in an app if they want custom mapping from errors to responses.


Not generating  accurate Calender

Your library is high customizable
But is giving such Ui

It occurs if you do like this :-
func configureCalendar(calendar: JTAppleCalendarView) -> (startDate: NSDate, endDate: NSDate, numberOfRows: Int, calendar: NSCalendar) {
let firstDate = formatter.dateFromString("2016 01 01")
print(NSDate())
let secondDate = formatter.dateFromString("2099 01 01")
let aCalendar = NSCalendar.currentCalendar() // Properly configure your calendar to your time zone here
return (startDate: firstDate!, endDate: secondDate!, numberOfRows: 6, calendar: aCalendar)
}

can you join me here -> https://gitter.im/patchthecode/JTAppleCalendar

What you are seeing there is boundary dates.
You are seeing the end of one month, and the start of another.


ItemParsedListOutputParser

Feature request
Different from ListOutputParser (see Motivation section)
Applies an output parser to each item in a newline separated list output
Motivation
The ListOutputParser does not actually parse the list items. They are merely returned as string values and must be parsed downstream. However the Retry/WithErrorOutputParser classes can only handle OutputParsing errors raised during the .parse method call. So if downstream parsing fails, we'll have to re-query the LLM's by hand.
In contrast, this class parses each list item inside its ItemParsedListOutputParser.parse call. That way if the item_parser raises an OutputParsingException, that exception will be caught by the RetryOutputParser and the LLM can make approrpiate changes in its next attempt.
Your contribution
Implemented ItemParsedListOutputParser
Details in PR

https://twitter.com/JacobFValdez


Add israel-hdez to AI Edge Maintainers

Signed-off-by: Landon LaSmith<EMAIL_ADDRESS>Description
Add israel-hdez to the list of AI Edge Maintainers
New Open Data Hub Member Requirements

[ ] New members have reviewed and acknowledged the Open Data Hub:

Code of Conduct
Contribution Guide
Community Membership Guidelines

[ ] Enabled 2FA on their GitHub account

Pull Request Requirements

[ ] New members are added in alphabetical order
[x] Only one new member change per commit (if you add two members separate it in two commits
[x] For individual user changes: Commit message format Add <USERNAME> to <opendatahub-io, mlops-sig, ...>.
[ ] For new team requests: Commit message format Create <TEAMNAME>. If the new team consists solely of existing members, you may
[ ] New GitHub team requests are from an existing opendatahub-io member who will function as the maintainer of the team. Each additional member will follow the same requirements for adding new members

@LaVLaS you're the only one who can approve this PR: Should we add a second person to the maintainers for the team?


How to create users in command line?

How do I create an LDAP user, in command line, with docker command?

first off you need to get into the container
docker exec -it  bash
generate new password if you want to add a user
slappasswd -h {} then hit enter
it will ask you to enter new password and confirm it
once you get your password, copy it so that you can use it when you create a user record
then you do
ldapadd -x -D "cn=admin,dc=,dc= -W
then it will ask for admin password, once you enter that, it will wait for you to enter new record like so
dn: ou=groups,dc=,dc=
objectclass: organizationalUnit
objectclass: top
ou: users
then hit enter twice to push your entry
for users it will go exactly the some way except you need to add more infomation
dn: uid=,ou=,dc=,dc=
cn: first name lastname
givenname: first name
sn: last name
uid: username
uidnumber:
userpassword:
gid: this is important to add of you are creating posixAccount
homedirectory: /home/users/
loginshell: /bin/sh
objectclass: inetOrgPerson
objectclass: posixAccount
objectclass: top
PS: you probably need to lay your user DN like so "uid,ou,dc,dc" especially if you are using ldap for user login and authentication

Hi, I did the command below but didn't get prompt for admin password.
ldapadd -x -D "cn=admin,dc=,dc= -W
Could you pls advise? thanks

Hey @lhty24 .  But let me know @lhty24 if your doubt was satisfied. If not, we can reopen the issue later :+1: :handshake:


🔧 Update template

This pull request merges the current state of the template used here into this project so that it is up to date.

:tada: This PR is included in version 1.0.3-develop.1 :tada:
The release is available on:

v1.0.3-develop.1
GitHub release
Docker container (sebbo2002/vestaboard2mqtt:1.0.3-develop.1)
v1.0.3-develop.1

Your semantic-release bot :package::rocket:

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

v2.0.0
GitHub release
Docker container (sebbo2002/vestaboard2mqtt:2.0.0)
v2.0.0

Your semantic-release bot :package::rocket:


[keyring-controller] Update state in single call when persisting or unlocking

Explanation
persistAllKeyrings and #unlockKeyrings have been refactored to update the state in one single call, to avoid side effects to be executed in the middle of the lock or unlock operations.
persistAllKeyrings and #unlockKeyrings can now be executed once per time, since they are both behind the same mutex: this ensures that each vault read/write operation is executed in a mutually exclusive way

References

N.D.
Changelog

@metamask/keyring-controller

FIXED: State updates are executed in a single call when locking or unlocking the KeyringController.

Checklist

[ ] I've updated the test suite for new or updated code as appropriate
[ ] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
[ ] I've highlighted breaking changes using the "BREAKING" category above as appropriate

Locking for any write operation sounds like a good idea to me

@Gudahtt just opened another PR for the general controller lock: https://github.com/MetaMask/core/pull/4156

@Gudahtt we could take tests from this PR that are currently failing because of run conditions. Those should not fail anymore  with this PR


Feature/iconography

@bbengfort I added icons for the Feeds and Status page.  Let me know what you think.  I tried to pick the most applicable icons.

Coverage decreased (-0.07%) to 60.196% when pulling d9967079207f5e9a4478b49931f95d2fdde80b7c on bahadasx:feature/iconography into 7b99f8107c6293b4e5e7716b23538386c48faac1 on bbengfort:develop.

Coverage decreased (-0.07%) to 60.196% when pulling da288dc23187d08018697f79800d2cc942e1801c on bahadasx:feature/iconography into 7b99f8107c6293b4e5e7716b23538386c48faac1 on bbengfort:develop.

Coverage decreased (-0.07%) to 60.196% when pulling 89fe98d926085d1306d2655e5df3efe3794456b5 on bahadasx:feature/iconography into 7b99f8107c6293b4e5e7716b23538386c48faac1 on bbengfort:develop.


fix: use measured height in view mode [DHIS2-8492]

Fixes issue where reloading a dashboard can cause a map item to cut off the bottom of the map. This may happen if you open the dashboard, then click on the same dashboard item chip to reload the dashboard. Note that the bottom of the map is clipped in the screenshot:

The Jira ticket has a video of the bug.

Clicking the same dashboard again seems to work :+1:
Still see this though (no padding between background map and item border):

Expected:

Clicking the same dashboard again seems to work 👍
Still see this though (no padding between background map and item border):

Expected:

Good eye. Fixed. The height didn't take into account padding on the dashboard-item-content.


Beartype starts logging warnings in my application

I have been using the spdx-3-prototype branch to generate SBOMs, and it has worked well. However after pulling in the last ~2 weeks of commits, whenever I run my code to generate a SBOM document, I get flooded with these warnings:
/Users/williamarmiros/software-projects/local-app/.venv/lib/python3.10/site-packages/beartype/_util/hint/pep/utilpeptest.py:347: BeartypeDecorHintPep585DeprecationWarning: PEP 484 type hint typing.List[spdx_tools.spdx3.model.external_map.ExternalMap] deprecated by PEP 585. This hint is scheduled for removal in the first Python version released after October 5th, 2025. To resolve this, import this hint from "beartype.typing" rather than "typing". For further commentary and alternatives, see also:
    https://beartype.readthedocs.io/en/latest/api_roar/#pep-585-deprecations
  warn(
/Users/williamarmiros/software-projects/local-app/.venv/lib/python3.10/site-packages/beartype/_util/hint/pep/utilpeptest.py:347: BeartypeDecorHintPep585DeprecationWarning: PEP 484 type hint typing.List[spdx_tools.spdx3.model.licensing.any_license_info.AnyLicenseInfo] deprecated by PEP 585. This hint is scheduled for removal in the first Python version released after October 5th, 2025. To resolve this, import this hint from "beartype.typing" rather than "typing". For further commentary and alternatives, see also:
    https://beartype.readthedocs.io/en/latest/api_roar/#pep-585-deprecations
  warn(
/Users/williamarmiros/software-projects/local-app/.venv/lib/python3.10/site-packages/beartype/_util/hint/pep/utilpeptest.py:347: BeartypeDecorHintPep585DeprecationWarning: PEP 484 type hint typing.List[spdx_tools.spdx3.model.software.software_purpose.SoftwarePurpose] deprecated by PEP 585. This hint is scheduled for removal in the first Python version released after October 5th, 2025. To resolve this, import this hint from "beartype.typing" rather than "typing". For further commentary and alternatives, see also:
    https://beartype.readthedocs.io/en/latest/api_roar/#pep-585-deprecations
  warn(
...

I tried following these instructions from beartype but it didn't work, just wanted to raise it in case you all weren't aware of this behavior change.

Thanks for raising this issue, @willarmiros! Having you as live tester is really invaluable! :)
I opened a PR that removes the warning.


all: Use sync.Map instead of sync.Mutex after Go 1.9 release

https://github.com/golang/go/commit/959025c0ac97ec3533ef9f3f70d64453352a7b56 introduced sync.Map and it provides pretty good performance speedup (https://github.com/golang/go/commit/eb6adc27d56687970dd8a49794ca85acc4cf9097, https://github.com/golang/go/commit/33b92cd6ce46c61a4d00a86b88971534773dd4a8, https://github.com/golang/go/commit/c120e449fbc618f9510387d718de0cef5f73af3a).
We also have some mutexes for map access, that could be replaced with sync.Map:
$ grep -nr 'sync.Mutex' .
./hugolib/pageCache_test.go:37: var l1 sync.Mutex
./hugolib/pageCache_test.go:38: var l2 sync.Mutex
./tpl/tplimpl/template.go:53:var amberMu sync.Mutex
./tpl/tplimpl/template_resources.go:36: remoteURLLock = &remoteLock{m: make(map[string]*sync.Mutex)}
./tpl/tplimpl/template_resources.go:44: m map[string]*sync.Mutex
./tpl/tplimpl/template_resources.go:50:         lock *sync.Mutex
./tpl/tplimpl/template_resources.go:55:         lock = &sync.Mutex{}

$ grep -nr 'sync.RWMutex' .
./helpers/general.go:211:       sync.RWMutex
./hugolib/pageCache.go:21:      sync.RWMutex
./hugolib/scratch.go:26:        mu     sync.RWMutex
./hugolib/shortcode.go:173:     sync.RWMutex
./output/layout.go:36:  mu    sync.RWMutex
./tpl/tplimpl/template_funcs.go:435:    sync.RWMutex
./tpl/tplimpl/template_funcs.go:1532:   sync.RWMutex
./tpl/tplimpl/template_funcs.go:1580:   mu sync.RWMutex
./tpl/tplimpl/template_resources.go:39: resCacheMu    sync.RWMutex
./tpl/tplimpl/template_resources.go:43: sync.RWMutex

It's just a note to not forget about it.

We should, but maybe not until Go Go 1.9+1 (if the performance gain is big, I'm willing to drop the "two Go version compability rule"). But we can test and see.


custom entry in csproj is not migrated and new Projectname.AssemblyInfo.cs is added

I use this task to get the SVN revision and set the assembly versions:
https://chivinou.wordpress.com/2008/08/06/automatic-assembly-version-number/
<PropertyGroup>
    <MajorNumber>1</MajorNumber>
    <MinorNumber>1</MinorNumber>
    <!-- Default will be to get the SVN revision number as revision number -->
    <RevisionNumber>0</RevisionNumber>
  </PropertyGroup>

and I see a new file Projectname.AssemblyInfo.cs which contains now duplicated AssemblyVersionAttribute entries.

Where should I add this GenerateAssemblyInfo entry to?

ok, I need to add it to the old csproj. I also needed to add <Deterministic>False</Deterministic> because of the SVN version task I use.
I also found the issue https://github.com/hvanbakel/CsprojToVs2017/issues/86. I'll look now if I find new issues or if I can compile the converted projects.

I think this added a regression (or was this also not deleted earlier? 🤔🤷♂).
I see now a propertygroup that contains a lot of old stuff:
<Configuration Condition=" '$(Configuration)' == '' ">Debug</Configuration>
    <Platform Condition=" '$(Platform)' == '' ">AnyCPU</Platform>
    <ProductVersion />
    <SchemaVersion>2.0</SchemaVersion>
    <ProjectGuid>{60CEB19E-8072-40B8-B230-C000B1769CFF}</ProjectGuid>
    <OutputType>Library</OutputType>
    <AppDesignerFolder>Properties</AppDesignerFolder>
    <RootNamespace>foo.CodedUITest</RootNamespace>
    <AssemblyName>foo.CodedUITest</AssemblyName>
    <TargetFrameworkVersion>v4.0</TargetFrameworkVersion>
    <FileAlignment>512</FileAlignment>
    <ProjectTypeGuids>{3AC096D0-A1C2-E12C-1390-A8335801FDAB};{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}</ProjectTypeGuids>
    <VisualStudioVersion Condition="'$(VisualStudioVersion)' == ''">10.0</VisualStudioVersion>
    <VSToolsPath Condition="'$(VSToolsPath)' == ''">$(MSBuildExtensionsPath32)\Microsoft\VisualStudio\v$(VisualStudioVersion)</VSToolsPath>
    <ReferencePath>$(ProgramFiles)\Common Files\microsoft shared\VSTT\$(VisualStudioVersion)\UITestExtensionPackages</ReferencePath>
    <IsCodedUITest>True</IsCodedUITest>
    <TestProjectType>CodedUITest</TestProjectType>

but those entries should be deleted.
Because of this the VS GUI has issues showing the correct .net version, becasue when I change it in the UI, it updates the  <TargetFramework>net47</TargetFramework> on top of the csproj, but now also finds the old entry later and changes the data in the IDE back to 4.0.


sponge command throwing error with no subcommand

I am currently running

SpongeForge version: 7.1.0-BETA-3218
SpongeVanilla version: 1.12.2-7.1.0-BETA-67

Plugins/Mods: /

Issue Description
type /sponge in console, get reply
[02:41:45] [Server thread/INFO]: Invalid subcommand state -- no more than one mapping may be provided for child arg child0

expected result
[02:40:10] [Server thread/INFO]: Not enough arguments!
[02:40:10] [Server thread/INFO]: Usage: /sponge which|audit|plugins|timings|heap|version|[--global|-g] [--world|-w <world>] [--dimension|-d <dimension>] reload|chunks|tps|save|config

The problem here is that Sponge's CommandElement for non-flag children is created will a fallback executor but not a CommandElement to handle the parsing for it (here). This causes the parse method to return without handling that verification as there's no fallback element (here).
The solution to this to create the ChildCommandElementExecutor with the delegate element as both the fallback executor and fallback element.
final ChildCommandElementExecutor flagChildren = new ChildCommandElementExecutor(null);
final ChildCommandElementExecutor nonFlagChildren = new ChildCommandElementExecutor(flagChildren, flagChildren, true);

Considering it was working previously, is @SimonFlash 's suggestion a work around? or is the current behavior unintended given the current command code?

Workaround should work - at least ish, though I don't think it'll give the correct usage (I think, haven't tested it, this is skimming code while in bed), reasoning is not completely correct.
Note that these child commands are not written in the normal way. The children arguments are specified differently, so the expectation is that these arguments should create the usage. As firstParsing is creating the commands, that should bail without parsing any commands - obviously it is not. This is probably because I've got some check for standard child commands that as messed up in this case.
I'll fix it either tonight or this weekend.


Action after submit validation

I have read other issues on here regarding submit validation but I guess I still don't understand it.
First my problem. Trying to do this causes an invariant violation.
const submit = (values) => {
    if (!values.type) {
     return Promise.reject({type: 'Type must be provided.'});
   }else{
     browserHistory.push('/location)
   }
}

<form onSubmit={handleSubmit(submit)}>

So I figured that if I do the browserHistory.push after the form is done submitting then that would fix the issue because doing onClick={() => browserHistory.push('/location)} works just fine.
sudo:
<form onSubmit={handleSubmit(submit).then(() => browserHistory.push('/location))}>

How can I get this to work? I have tried different variations and using returnRejectedSubmitPromise:true.
Thanks!

What is the nature of the invariant violation? That's pretty strange.
It seems like your
<form onSubmit={handleSubmit(submit).then(() => browserHistory.push('/location'))}>

example should work, as long as you return Promise.resolve() from your submit() if there are no errors.

I think the invariant violation comes from a message that is being displayed on the screen. Further up when the route changes I clear all messages and that seems to be causing the issue. I have racked by brain trying to figure out why.
Uncaught Invariant Violation: findComponentRoot(..., .<IP_ADDRESS>.1:$0.1): Unable to find element.
const submit = (values) => {
    if (!values.recipientType) {
     return Promise.reject({recipientType: 'Recipient Type must be provided.'});
   }else{
     return Promise.resolve();
   }
}
<form onSubmit={handleSubmit(submit).then(() => browserHistory.push('/location'))}>

Always says  handleSubmit(...).then is not a function

This looks like the bug that I just published the fix for. Try v4.1.10.

I get the same error. Maybe I am doing something else wrong. This is the form minus unnecessary stuff.
"name": "redux-form",
"version": "4.1.10",
import React, {Component, PropTypes} from 'react';
import {reduxForm} from 'redux-form';
import { browserHistory } from 'react-router';

const submit = (values) => {
    if (!values.recipientType) {
     return Promise.reject({recipientType: 'Recipient Type must be provided.'});
   }else{
     return Promise.resolve();
   }
}

class RecipientType extends Component {

  static propTypes = {
      fields: PropTypes.object.isRequired
  }

  render(){
    const {fields:{recipientType}, handleSubmit} = this.props;
    return(
      <form onSubmit={handleSubmit(submit).then(()=> browserHistory.push('/location))}>

    </form>
    );
  }
}

export default reduxForm({
  form: 'packageWizard',
  fields: ['recipientType']
})(RecipientType)

<form onSubmit={e => handleSubmit(submit)(e).then(() => browserHistory.push('/comm/admin/secure/app/package/wizard/recipients'))}>

works now, it didn't before 4.1.10


Large constant parameter to Printf causes integer overflow

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
1.8 (Discovered and tested using play.golang.org)
What operating system and processor architecture are you using (go env)?
Unknown (See prev.)
What did you do?
I was trying to figure out a bug and I was getting an integer overflow error when using a constant vs. a variable in a Printf or when using json.Marshal.  Below is the minimal code to make this happen.
https://play.golang.org/p/pJmDXboQfH
If you comment out the last Printf, the program works correctly.
What did you expect to see?
Value:<PHONE_NUMBER>000
Value:<PHONE_NUMBER>000
What did you see instead?
tmp/sandbox290752816/main.go:10: constant<PHONE_NUMBER>000 overflows int

Working as intended. See https://blog.golang.org/constants
This comes up periodically. See #20108 for the last explanation.

Apologies for the bug report.  I realized after you replied that I forgot to check closed issues. :(


Himalaya Darshan College Added

Himalaya Darshan College Added

@gitkailash hdc.txt I am sorry, I have to close the request without merging, because you made a mistake in a request and did not specify any filepath. Please ensure that you are familiar with this repository rules located at the bottom of this page and make a new correct request. Thank you for your understanding!

On Thu, 27 Apr 2023 at 4:24 PM Philip Torchinsky @.***>
wrote:

Closed #17369 https://github.com/JetBrains/swot/pull/17369.
—
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/17369#event-9112825392, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AVNMMQ55ZHPVIWQQCDBNTPLXDJEGFANCNFSM6AAAAAAXNJVHUY
.
You are receiving this because you were mentioned.Message ID:
@.***>

what mistake is there can you help me?
I am not able to find what file path it is mistake. i tried as instructed.


Raw 802.15.4 support

I built this upon libtock-c implementation.
I focused on providing safe abstractions.

Tx part is quite bare, i.e. just passing a byte slice. Should it be more structured?
Rx part is sophisticated. There are:

Frame, RxRingBuffer - these build up the ring buffer that is shared with kernel;
RxSingleBufferOperator, RxBufferAlternatingOperator - these encapsulate the buffer and provide safe operations on them. The former uses a single buffer, which makes it possible to lose some frames when the buffer is not allowed to the kernel. The latter alternates between two buffers, which guarantees no frame loss at cost of doubled memory usage.

Feedback much welcome.
Example is complete.
Unit tests are done.
Haven't tested this on hardware as well.

Looks good!

Tx part is quite bare, i.e. just passing a byte slice. Should it be more structured?

I do not think it should be more structured.

Overall looks good to me (from the 15.4 side of the world; I'm less familiar with libtock-rs and can't speak too much on that front)! A few comments:

The ring buffer size should be the 15.4 frame size + some meta data (this is explained in the 15.4 capsule driver). Looking through this, it seems your ring buffer is just 127 bytes (15.4 frame size).
It appears you are not implementing the ring buffer (just swapping two buffers). This is entirely fine (and will work for most applications). You may need to implement handling multiple packets within each ring buffer if you face workloads with heavy/frequent packet reception (e.g. OpenThread). Swapping buffers is better than just using one buffer, but can still experience dropped/overwritten packets in the case of multiple packets being received prior to the swap.

Exciting to see some 15.4 support making its way into libtock-rs!

Overall looks good to me (from the 15.4 side of the world; I'm less familiar with libtock-rs and can't speak too much on that front)! A few comments:

The ring buffer size should be the 15.4 frame size + some meta data (this is explained in the 15.4 capsule driver). Looking through this, it seems your ring buffer is just 127 bytes (15.4 frame size).

According to phy_driver.rs:
//! The ring buffer provided by the process must be of the form:
//!
//! ```text
//! | read index | write index | user_frame 0 | user_frame 1 | ... | user_frame n |
//! ```
//!
//! `user_frame` denotes the 15.4 frame in addition to the relevant 3 bytes of
//! metadata (offset to data payload, length of data payload, and the MIC len).
//! The capsule assumes that this is the form of the buffer. Errors or deviation
//! in the form of the provided buffer will likely result in incomplete or
//! dropped packets.

I created the following structs:
#[repr(C)]
pub struct Frame {
    pub header_len: u8,
    pub payload_len: u8,
    pub mic_len: u8,
    pub body: [u8; MAX_MTU],
}

#[repr(C)]
pub struct RxRingBuffer<const N: usize> {
    /// From where the next frame will be read by process.
    /// Updated by process only.
    read_index: u8,
    /// Where the next frame will be written by kernel.
    /// Updated by kernel only.
    write_index: u8,
    /// Slots for received frames.
    frames: [Frame; N],
}

I can't see any deviations from the kernel docs. Could you point them out?

It appears you are not implementing the ring buffer (just swapping two buffers). This is entirely fine (and will work for most applications). You may need to implement handling multiple packets within each ring buffer if you face workloads with heavy/frequent packet reception (e.g. OpenThread). Swapping buffers is better than just using one buffer, but can still experience dropped/overwritten packets in the case of multiple packets being received prior to the swap.

I do not understand why you believe so. RxRingBuffer has both read and write indices, which are used to establish ring buffer semantics on the buffer. Both Operators are written in a way that makes them return frames one by one, using ring buffer pop operation. Or am I mistaken?

Exciting to see some 15.4 support making its way into libtock-rs!

Hooray!

@wprzytula my mistake. I missed the RxRingBuffer fields when skimming through this earlier and did not see the popping from the ring buffer etc. Thanks for the clarification!

Finished implementation. I did some minor fixes to the logic, wrote a very decent test suite and polished documentation.

Applied clippy fixes.

@wprzytula what's the state of this?
It says still

Haven't tested this on hardware as well.

Is this still accurate?

@wprzytula what's the state of this?
It says still

Haven't tested this on hardware as well.

Is this still accurate?

I still haven't succeeded in implementing HIL on cc2650 (this is WIP), so yes.
I'd be very grateful if someone tried this out on another piece of hardware, e.g. Nordic boards.

I need help with understanding the Miri check failure. I'm not sure what is exactly believed to be UB and why.

@wprzytula, sorry for the extended silence on this PR. A bunch of us are extensively discussing how to resolve the soundness issues correctly (e.g. https://github.com/tock/tock/blob/master/doc/wg/network/notes/network-notes-2024-07-15.md#154-libtock-rs-driver). Sorry this is not getting more attention while that's happening.
It turns out not to be obvious what the right fix would be, so no specific actionable recommendation just yet.

@wprzytula, sorry for the extended silence on this PR. A bunch of us are extensively discussing how to resolve the soundness issues correctly (e.g. https://github.com/tock/tock/blob/master/doc/wg/network/notes/network-notes-2024-07-15.md#154-libtock-rs-driver). Sorry this is not getting more attention while that's happening.
It turns out not to be obvious what the right fix would be, so no specific actionable recommendation just yet.

I have read the discussion you refer to. I understand your doubts and the current point now, thank you.

@wprzytula what's the state of this?
It says still

Haven't tested this on hardware as well.

Is this still accurate?

I still haven't succeeded in implementing HIL on cc2650 (this is WIP), so yes. I'd be very grateful if someone tried this out on another piece of hardware, e.g. Nordic boards.

I finally succeeded in writing HIL for cc2650 802.15.4 radio, and so I ran tests of this PR on it. Everything seems to work correctly, both for RxSingleOperator and RxBufferAlternatingOperator.

When I think of it, it's only the RxRingBufferInKernel, required for RxBufferAlternatingOperator to work, that causes UB according to Miri.
RxSingleBufferOperator is accepted. As it is good enough (RxBufferAlternatingOperator is only better in the way that it does not miss frames if a frame comes when user is examining the buffer, preventing it from being accessed by kernel), perhaps we could merge this PR without RxBufferAlternatingOperator-related code? This would give libtock-rs's users access to basic networking!
WDYT @alevy @jrvanwhy @bradjc @tyler-potyondy ?

As it is good enough (RxBufferAlternatingOperator is only better in the way that it does not miss frames if a frame comes when user is examining the buffer, preventing it from being accessed by kernel), perhaps we could merge this PR without RxBufferAlternatingOperator-related code? This would give libtock-rs's users access to basic networking!

I think that is a great move forward! Nonetheless, we do want to figure out a way to support Tock's buffer swapping semantics for "lossless" kernel -- userspace communication in the long term; it's one of the primary advantages of this kernel API. That being said, I agree that having something is better than nothing, and in particular if the potential loss of packets is sufficiently small to not cause havoc or be recoverable in practice.

@wprzytula yes, do this.

I've removed RxBufferAlternatingOperator and its controversial code. Please run CI, it should pass now.

@wprzytula I kicked off CI, there's still some unresolved imports but feel free to keep pushing to this branch and CI should run from this point onward.

@wprzytula I kicked off CI, there's still some unresolved imports but feel free to keep pushing to this branch and CI should run from this point onward.

Unfortunately, an approval is required again.

Unfortunately, an approval is required again.

It'd seem like GitHub changed something. We used to only need to approve once, and the repository is set to require approval only for first-time contributors. In any case, keep posting here to have someone hit the button as much as you'd like, we can hide these comments later.

Once again, please @lschuermann.

It is very unclear why it keeps requiring approval to run actions. The repo and org are set to require it for first-time contributors, but I really thought that was once-only.
To try to "fix" it, I have added @wprzytula as a read-only contributor. The should result in zero gained permissions they didn't have before since the repo is public, but I'm hoping that listing them as a collaborator maybe lets CI run? Maybe?

To try to "fix" it, I have added @wprzytula as a read-only contributor. The should result in zero gained permissions they didn't have before since the repo is public, but I'm hoping that listing them as a collaborator maybe lets CI run? Maybe?

Hooray! This works.

The should result in zero gained permissions they didn't have before since the repo is public, but I'm hoping that listing them as a collaborator maybe lets CI run?

It's great that this works, but I believe this is a bug on GitHub's end. It used to only require approval once.

The current CI failure seems to be about MSRV. Can someone confirm? What are we doing about that?

The current CI failure seems to be about MSRV. Can someone confirm? What are we doing about that?

Our policy is to bump the MSRV whenever doing so is convenient. You may need to increase the nightly toolchain version as well. See https://github.com/tock/libtock-rs/pull/541 for an example of how to do so.

Oh, and I suggest increasing the toolchain versions in a separate PR, just in case the toolchain bump causes compilation failures for somebody else' PR. If the toolchain version increase passes CI we can merge it quickly.

@jrvanwhy Ready.

@lschuermann @jrvanwhy Why is this still not merged? Are there still some action points to be resolved?


modify clone method for derived classes.

Modify DownstreamRestApiOptions.Clone so that derived classes can implement Clone.

@brentschmaltz: let's discuss this along with https://github.com/AzureAD/microsoft-identity-abstractions-for-dotnet/pull/38


Style current day

Hi indrimuska
First at all I would thank you for this great date-picker, it's awesome!
I have a problem with the styling of the picker: Everything works fine with the style, I have only the question, is there a class or somethin else, which can style the current day. For example: Today is the 02.05.16, but i select the 05.05.2016. So I would style the selected and the current day in the same time. So I can see the todays date and the selected. At the moment I only style the selecet. Is there a specific class for styling the current day. I often saw such classes for todays date in other date-picker plugins.
It would be great if you can help me.
Greetings
MrBuggy :-)

Hi @MrBuggy, thank you for the kind words!
I was thinking about adding a new feature to let users have a ready-to-use property to enable/disable highlighting of the current date. What do you think about? Are you interested to create a PR?

FYI latest release v0.5.2 has a new boolean today property to highlight the current date.

Very cool thanks! :D
Von meinem iPhone gesendet

Am 09.06.2016 um 10:56 schrieb indrimuska<EMAIL_ADDRESS>FYI latest release v0.5.2 has a new boolean today property to highlight the current date.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Why is the mode automatically 'w' for dask.dataframe.read_hdf()?

Very sorry if it doesn't belong in Bug Report, it is also my first issue, let me know if this belongs somewhere else. I'm wondering why the read_hdf function has a default 'w' mode? When I read data, I would assume the default mode is 'r' (read, not append or overwrite), when I want to alter the data then I should explicitly mention it is open for writing or appending. pandas.read_hdf() takes 'r' as the default mode, so I had assumed dask would do the same. Was there a strong community preference for default writing mode? Thanks!

First - welcome to dask, and sorry it took a while to get to this. Probably the right place for a question like this would be in the Discussions area. I did a little digging back in the codebase and this piece does not seem to have changed in a long time, which leads me to suspect that it was just an oversight and not intentional. I imagine the project would welcome a PR to change the mode from "a" to "r". Are you interested in opening that Pull Request? It should be changing the mode here: https://github.com/dask/dask/blob/76a4f536237a5e65e03ba471f004226056646b65/dask/dataframe/io/hdf.py#L422 and updating the docstring accordingly.

First - welcome to dask, and sorry it took a while to get to this. Probably the right place for a question like this would be in the Discussions area. I did a little digging back in the codebase and this piece does not seem to have changed in a long time, which leads me to suspect that it was just an oversight and not intentional. I imagine the project would welcome a PR to change the mode from "a" to "r". Are you interested in opening that Pull Request? It should be changing the mode here: https://github.com/dask/dask/blob/76a4f536237a5e65e03ba471f004226056646b65/dask/dataframe/io/hdf.py#L422 and updating the docstring accordingly.


Back button from result screen should go to setup screen

Especially if the board is shown on the result screen (issue #5), there is no reason to go back to the disabled board screen.

Improved back button support in #18

Fixed by the master himself. See 4c9f0423c0f138937d961daf5d216fc5aad8f8f1


The PyQt6 dependency

Hi, we're currently depending on PyQt6, as there's a bug in PySide6 that has remained unaddressed for many years and breaks interactive Matplotlib use:
https://github.com/conda-forge/pyside2-feedstock/issues/226
Now, this wouldn't be much of a problem when using an all pip-installed environment. But on conda-forge, there is no PyQt6 package available (and I doubt there ever will be) – we only have PySide6 and PyQt5.
So when setting up my dev environment by first installing MNE from conda-forge, followed by a pip install -e ".[dev,full]" of mne-python:main, pip will start pulling in PyQt6 since it doesn't know that on my conda environment, I would rather use PyQt5 (or PySide6) instead.
So there's a discrepancy and this leads to a cluttered environment and segfaults.
(In case you're wondering why I'm not simply installing everything via pip from the get-go: I'm working on aarch64 (ARM Linux in a Docker container running on Apple Silicon), and many PyPI packages are not available as wheels for this platform).
I'm wondering what we could do about this. Several solutions (?) come to mind:

Depend on PySide6 instead of PyQt and ask users to use IPython in case they depend on interactive plotting
Add another installation variant that doesn't contain the PyQt dependencies, so I'm free to choose whichever Qt framework I want
Add additional installation variants that explicitly depend either on PyQt5 or PySide6

I'd love to hear your feedback. This is truly a very frustrating situation for me.
Thanks!
cc @cbrnr

How "important" is the interactive plotting problem really, anyway? I never encounter it because a) I never use plot.ion(); b) during interactive use, I run my stuff in Jupyter / IPython anyway. But I'm not sure how others do it.

I actually use plt.ion() in non jupyter/IPython kernel, for instance to display topographic map which updates based on events. What is the issue with using qtpy and not providing any Qt binding in our installation, relying entirely on the user to install one?

What is the issue with using qtpy and not providing any Qt binding in our installation, relying entirely on the user to install one?

Well exactly that: that it would require users to install something additionally that is in fact a hard dependency and should be installed automatically for them. It's both inconvenient and a packaging bug, as a package manager wouldn't be able to determine that MNE actually depends on that particular Qt binding.

I actually use plt.ion() in non jupyter/IPython kernel, for instance to display topographic map which updates based on events.

Interesting, what does that look like? Do you have a minimal script you could share? (out of general interest!)
Would there be any chance you could run the same script in IPython?

Interesting, what does that look like? Do you have a minimal script you could share? (out of general interest!)

It's a small demo we use on our site with dry-electrode EEG systems. I have to fire it up tomorrow, I'll record it and share the associated script.

Would there be any chance you could run the same script in IPython?

Sure it would, but I prefer to start it through an terminal entry-point from the package. In my case however, it's a non blocking issue as I use pip to install everything.

Now back to the subject:

Well exactly that: it would require users to install something additional that is in fact a hard dependency and should, therefore, be installed automatically for them.

I agree, but Qt is a bit special within our dependencies exactly because it has 4 different bindings which offers similar functionalities and that in advance/expert scenarios you might want to install one binding instead of another one. IMO, we should differentiate 2 user populations:

python beginners, who will use the MNE installers -> includes a python binding and they don't even know about what's actually installed, where, and so on.
intermediate and advance python users, who do not use the MNE installers -> in this case, I consider that they can read the error message yielded by qtpy and figure out that they have to install one of the 4 bindings.

This can be further back with documentation in the install section of our website, which already separates those 2 populations.

It's a small demo we use on our site with dry-electrode EEG systems. I have to fire it up tomorrow, I'll record it and share the associated script.

That would be awesome, thanks! :)

I agree, but Qt is a bit special within our dependencies exactly because it has 4 different bindings which offers similar functionalities and that in advance/expert scenarios you might want to install one binding instead of another one.

Yeah but why not provide something that works out-of-the-box (as we do now) for most users, and add in an additional variant (e.g., mne[full-no-qt] or something) for those who need to pick a different binding?
We currently do make the choice already, as we depend on PyQt6. So this is what everyone gets when installing mne[full]. I need an additional way to special-case my scenario. Unless we switch the dependency to PySide6 altogether.

For sure, I would not provide PySide2 or PySide6 by default if it yields broken interactive plots. As for adding a mne[full-no-qt] option, sure, why not.

Another argument against PyQt6 (and in favor of PySide6): there's no PyQt6 wheels for aarch64 on PyPI. I contacted the developers about if and when this might happen, and the response was something like, "maybe someday" (I cannot find the original posting anymore)

I've opened #12628
Any feedback appreciated

Another argument against PyQt6 (and in favor of PySide6): there's no PyQt6 wheels for aarch64 on PyPI. I contacted the developers about if and when this might happen, and the response was something like, "maybe someday" (I cannot find the original posting anymore)

I found it again:
https://gitlab.kitware.com/vtk/vtk/-/issues/18648#note_1526361

But this is about VTK and not PyQt/PySide?

Oh. Right. I'm getting all confused here… sorry.

Yes, only PySide has aarch64 wheels. Plus it is always more up to date, currently it's at 6.7.1, which fixes segfaults on macOS that occur with 6.7.0. PyQt is still at 6.7.0.


Column not found - v0.11.8 => v0.11.12

Laravel Version: #.#.#
Voyager Version: #.#.#
PHP Version:
Database Driver & Version:

Description:
I recently ran composer update which ran  tcg/voyager (v0.11.8 => v0.11.12) and now my admin panel is broken when I click on pages. I get his error. Looking at the db that did work there is no Order in the data_rows table. Do I need to run some migrations to update?
SQLSTATE[42S22]: Column not found: 1054 Unknown column 'order' in 'order clause' (SQL: select * from `data_rows` where `data_rows`.`data_type_id` = 2 and `data_rows`.`data_type_id` is not null and `browse` = 1 order by `order` asc)

Steps To Reproduce:

Hi @packytagliaferro, check Release v0.11.10.

@akazorg perfect. Forgot to check the release notes

Happy coding :)


Mutect2 and HC always use NON_ZERO_REFERENCE_LENGTH_ALIGNMENT read filter

This filter takes place pre-reassembly and removes reads that do not take up any reference bases. Without the filter we get an error like java.lang.IllegalArgumentException: Invalid interval. Contig:3 start:100170602 end:100170601

Although, if @droazen approves, you can keep the HC change as well.

Can you add a unit or integration test that fails before the change and passes after (for both tools)?
Merge after adding test :+1:

@davidbenjamin I added integration test sto HC and M2. Without the change (i.e. if you check out just the first commit), the tests should fail with java.lang.IllegalArgumentException: Invalid interval. Contig:20 start:5624960 end:5624959
I made the test an integration test, not a unit test as @droazen suggested. This was because I don't understand the read filter code enough to identify the appropriate unit.

@takutosato You may squash and merge.  I think that an integration test is fine for our purposes.


Multiline editing doesn't fix cursor when user hits enter with cursor halfway up

So I'm working on a little terminal-based REPL for my programming language Phoo and I found this for the input part. So far works great (though I did have to apply the patch described in #24 to be able to use color prompts and a similar patch as the one in #31 for custom is-complete handlers).
One big problem is when a multiline edit is triggered, the user can move their cursor up, but it isn't moved back down again when they press enter and the command is already complete.

In the screenshot I just had the terminal set up to prompt in blue (-> and ..), and then echo the output back again in red.
I typed the first few lines, hit enter with the cursor at the bottom, and it worked fine.
I hit 'up', moved the cursor to the middle of the b's, and hit enter from there, and you can see it clobbered over the input prompt and text!
How do I fix this?

I'm pretty sure it's somewhere in here:
https://github.com/wavesoft/local-echo/blob/8d0b7f55c5cf4b0b5f7c5132825dc5bd984bf017/lib/LocalEchoController.js#L424-L434

I have a hunch it is in here:
https://github.com/wavesoft/local-echo/blob/8d0b7f55c5cf4b0b5f7c5132825dc5bd984bf017/lib/LocalEchoController.js#L424-L434
Somewhere between 427 and 428 there should be some code that gets the cursor position in the input text, and if it is not at the bottom, moves it down.


Using Yarn to install Library fails

Expected behavior
Successfully install react-intersection-observer to my react application
Current behavior
gets the error message below
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version ">=10.18.1". Got "10.15.3"
Steps to reproduce

yarn add @researchgate/react-intersection-observer

Upgraded node version and it installed.
Docs should indicate the required Node version.

@Tooluloope Thank you for your recommendation. We've updated the readme in #167 to include this notice.

@stefanhoth Great👍

It was also in package.json -> https://github.com/researchgate/react-intersection-observer/blob/master/package.json#L7


oh-my-zsh issues

Hi.
When I am installing oh-my-zsh it's working for the current window, but when I am closing Hyper and opening Hyper again it's not initialized. Do I have to init. it in some way? Or tell Hyper to always use oh-my-zsh?

did you change your system shell to zsh – chsh -s /bin/zsh?
there's also a shell option in the Hyper config (see https://hyper.is/#cfg)

Thank you very much! setting shell = '/bin/zsh' in ~/.hyper.js did the trick for me.

Did you resolve your issue @sebastianks? Are we okay to close this now?

Hi @CodeTheory
Yes - Thank for your help

This appears to have no effect for me. I'm running on a network/managed account, so changing my shell system-wide isn't obvious. (chsh doesn't work) Was the shell key in config deprecated?

How to get rid of the % sign on startup when doing this?
See

@sinedied Same problem here

This is the Z Shell configuration function for new users,
zsh-newuser-install.
You are seeing this message because you have no zsh startup files
(the files .zshenv, .zprofile, .zshrc, .zlogin in the directory
~).  This function can help you with a few settings that should
make your use of the shell easier.
You can:
(q)  Quit and do nothing.  The function will be run again next time.
(0)  Exit, creating the file ~/.zshrc containing just a comment.
That will prevent this function being run again.
(1)  Continue to the main menu.
(2)  Populate your ~/.zshrc with the configuration recommended
by the system administrator and exit (you will need to edit
the file by hand, if so desired).
Hy guys, someone can help me? my zsh keep send me this msg when i try to open a new tab.

How to get rid of the % sign on startup when doing this?
See

https://github.com/zeit/hyper/issues/2144


Add support for creating pipelines without an estimator as the final component and add transform() to component graphs and pipelines

Closes #2103
Design doc here: https://alteryx.atlassian.net/wiki/spaces/PS/pages/541622308/Component+Graph+Updated+Design+and+Notes

Adds a transform method for ComponentGraph and pipelines

Similar to transform for components, transform for pipelines / component graphs will return a tuple if transforming y, otherwise just one df (transformed X)
Will raise a ValueError if the final component is an Estimator

predict and predict_proba will raise a ValueError if the final component is a Transformer

@bchen1116

LGTM! Have the changes already been made for samplers to have fit_transform() be the same as .fit().transform()?

Yup, changes were made in https://github.com/alteryx/evalml/pull/2583 :)


Made it so pageTitle will only look for title in head tag

I filed an issue https://github.com/cssstats/cssstats/issues/159 and realized the issue was coming from the module in this repository.
function parseHtml(html) {
    var $ = cheerio.load(html);
    result.pageTitle = $('title').text();

Inevitably, this would grab all <title> tags on the page. I made it so that it would only grab the <title> tag under <head>
Hope that works! :)

This is great, thanks!

Published to npm as<EMAIL_ADDRESS>


set-config

Describe the bug

When running the make setup-config and after entering the LLM Model name, LLM API KEY, LLM Embedding Model, Workspace, iam getting at an error
process_begin: CreateProcess(NULL, mv config.toml.tmp config.toml, ...) failed.
make (e=2): The system cannot find the file specified.
make: *** [Makefile:64: setup-config] Error 2
Screenshot:
()

Guess more like you local env problem. https://stackoverflow.com/questions/33674973/makefile-error-make-e-2-the-system-cannot-find-the-file-specified

BTW, if you use windows, it seems like opendevin only can work on WSL on windows. I am not a windows user, so I am not familiar with this parts. But you may check some other issues. Many user discuss the error they have meet on windows.

@sravan792 if you're using WSL and still hitting this problem, feel free to open a new issue with the template filled out


do* works differently in top-level functions vs in methods

At the top level:
->
  do*
   yield foo
   yield bar
// vvv
function () {
  return (function* () {
    {
      yield foo;
      yield bar;
    }
  })();
}

In a method:
{
  f()
    do*
     yield foo
     yield bar
}
// vvv
{
  *f() {
    (function* () {
      {
        yield foo;
        yield bar;
      }
    })();
  }
}

I think the latter is a bug, since it never returns or yields anything.

The latter example is also missing a return.
This bug also causes getters to fail to transpile with error "Getters and setters cannot be generators":
{
  get f()
    do*
      yield foo
      yield bar
}

So we might add this to tests once fixed.


Update dependencies

Changed

Updated dependencies.
Updated prettier config.

Codecov Report

Merging #129 (79e0348) into master (e78a7cf) will decrease coverage by 0.26%.
The diff coverage is 94.33%.

@@            Coverage Diff             @@
##           master     #129      +/-   ##
==========================================
- Coverage   96.39%   96.12%   -0.27%
==========================================
  Files          96       87       -9
  Lines        1691     1601      -90
  Branches      228      231       +3
==========================================
- Hits         1630     1539      -91
- Misses         58       59       +1
  Partials        3        3

Flag
Coverage Δ

cuaktask
99.25% <100.00%> (ø)

iocuak
94.60% <94.23%> (-0.03%)
:arrow_down:

porygon
?

porygon-typeorm
98.90% <ø> (ø)

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

Impacted Files
Coverage Δ

...ask/modules/cuaktask/CreateInstanceTaskLazyNode.ts
87.50% <87.50%> (ø)

...cuaktask/src/task/modules/RootedTaskGraphRunner.ts
100.00% <100.00%> (ø)

...ask/models/cuaktask/GetInstanceDependenciesTask.ts
100.00% <100.00%> (ø)

...ask/CreateInstanceTaskGraphExpandCommandHandler.ts
100.00% <100.00%> (ø)

...s/porygon/src/crud/models/domain/CrudModuleType.ts

.../src/crud/modules/domain/UpdateEntityInteractor.ts

.../src/crud/modules/domain/CreateEntityInteractor.ts

...crud/modules/iocuak/DomainCreateContainerModule.ts

.../src/crud/modules/domain/DeleteEntityInteractor.ts

...crud/modules/iocuak/DomainUpdateContainerModule.ts

... and 5 more

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 aeb6f3f...79e0348. Read the comment docs.


Compilation fails when space after/before parens

This works
(a, b)

But this doesn't
( a, b )

Which can be a problem, given that the latter is the default output of elm-format

Fixed in #19


Can't build the binary

Thanks for a wonderful project!
When I do a go get -u github.com/gophergala/dnsp I can't find the dnsp binary in $GOPATH/bin.
I am using Mac Yosemite 10.10.1 (14B25), go version 1.4
Any hints will be appreciated.

Apparently the README is wrong. It should be go get -u github.com/gophergala/dnsp/... (note the final /...).


Allow suppressing the changelog header

Motivations
By default, the changelog has this header:
All notable changes to this project will be documented in this file.

The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

The descriptions may not be accurate for some projects, and has information that is somewhat self-explanatory.
Solution
Add an easy way to disable the header, like with a default_header = false setting in [changelog]
Alternatives
This is possible with a custom template, but having an easy way to disable it would be nice.

Can't you set https://release-plz.ieni.dev/docs/config#the-header-field to ""?

Oh 😆 yeah that works. Thanks!


Better code formatting

Hello @PifyZ,
Thanks for your contribution but we can't accept your PR. Indeed, we are following PSR-2 standards and your code is not compliant. Also, when you make a PR, make sure to do it on dev branch.
Regards

No problem.
This is normal in the "dev" branch there is the CSS ID "#reassurance_block", but in the HTML "block-reassurance" is used?


adonis: not found (exit status 1) when using podman

Describe the bug
I created adonis app using the instructions here;
https://kool.dev/docs/presets/adonisjs
Kool version and environment
kool version 2.1.1
To Reproduce
Steps to reproduce the behavior:

create application
edit .env
kool run setup

not found (exit status 1)
Expected behavior
Once kool run setup finishes, you should be able to access your new site at http://localhost:3333/ and see the AdonisJs "It works!" welcome page. Hooray!
Screenshots

Additional context
I am using podman with docker and compose support

@yasirmturk sorry for the delay on this.
I have made a few tweaks to the Adonis preset and confirmed it is working now; should merge and release a minor version within the upcoming week.
Will make more tests on your specific Podman setup, but it should not be a problem.

Latest release got fixes for the Adonis preset, this should be fixed now.


offline installation

Hello,
Is there any way to install nmslib offline for python, i.e., pack everything in a package and we can do 'import nmslib' by adding the package to PYTHONPATH without installing it via pip?
Thanks in advance!

Hi @Zheweiqiu you can

build the library locally from the similarity_search folder
create a Python egg in the python_bindings directory by then running python setup.py build.
You can then add the *full path of the egg to PYTHONPATH.

Hi @Zheweiqiu you can

build the library locally from the similarity_search folder
create a Python egg in the python_bindings directory by then running python setup.py build.
You can then add the full path of the egg to PYTHONPATH, e.g.
PYTHONPATH=/Users/leo/tmp/nmslib-1.8-py3.6-macosx-10.7-x86_64.egg

Are these two steps two ways of building nmslib library locally or twos steps of one approach? And for the second part, how to 'create a Python egg in the python_bindings directory'?
Thank for any help!

Hi, these are basically the setups. It's actually easier to go directly to the python_bindings folder and type:
python setup.py build

If I understand correctly, the egg in the the build/lib... subdirectory. You can google for additional info. NMSLIB is not different in this respect compared to other python libraries. Once you build the egg, you can use it without installing. I did try this yesterday and it worked.


Add maling list link on the front page

I think it makes sense to have a link to discuss-webrtc under the get-involved section even though we already have it hidden in the contacts section of the footer.

Error is not related to my PR:
Gem::InstallError: public_suffix requires Ruby version >= 2.0.
Installing jekyll-paginate 1.1.0
An error occurred while installing public_suffix (1.5.3), and Bundler cannot
continue.
Make sure that gem install public_suffix -v '1.5.3' succeeds before bundling.
The command "eval bundle install --jobs=3 --retry=3 --deployment" failed. Retrying, 3 of 3.

erm... someone enabled Travis?

@samdutton we don't need Travis, we already have a CI. It's doing exact same things as Travis would.
I've just deactivated Travis.

drone is what you want to look at, which is passing.

Oops — sorry! Thanks for deactivating...
On 12 January 2016 at 17:26, alex<EMAIL_ADDRESS>wrote:

drone https://ci.cloudware.io/webrtc/webrtc-org/24 is what you want to
look at, which is passing.
—
Reply to this email directly or view it on GitHub
https://github.com/webrtc/webrtc-org/pull/24#issuecomment-170982488.

Adding mailing list link SGTM


D3D12nBodyGravity threading issues and DreamBuildPlay2017

I have been working on threading in my app for a while now.  nBodyGravity was my goto for learning.  After the windows 10 anniversary update, my app started crashing.  I narrowed it down to threading of the code below.  Are these commands not thread safe?  Do they have to be run on the main thread?  If they do have to be run on the main thread, then why did it work on separate threads without crashing before windows 10 update, did you change something?  I have searched Microsoft documentation after documentation and it was vague at best on how to use "your own" command queues but never mentioned anything about how thread safe it was.  I posted on directxtech, gamedev, and here and had zero responses.  Is there anywhere to get solid support for directx 12, or is it like a use at your own risk product?  I was planning on entering my game in the Dream Build Play 2017, but its going to be hard to do so if I cant find support for random issues with the D3D12.dll with no explanations.
		D3D12_COMMAND_QUEUE_DESC queueDesc = { D3D12_COMMAND_LIST_TYPE_COMPUTE, 0, D3D12_COMMAND_QUEUE_FLAG_NONE }; 		ThrowIfFailed(m_device->CreateCommandQueue(&queueDesc, IID_PPV_ARGS(&m_computeCommandQueue[threadIndex]))); 		ThrowIfFailed(m_device->CreateCommandAllocator(D3D12_COMMAND_LIST_TYPE_COMPUTE, IID_PPV_ARGS(&m_computeAllocator[threadIndex]))); 		ThrowIfFailed(m_device->CreateCommandList(0, D3D12_COMMAND_LIST_TYPE_COMPUTE, m_computeAllocator[threadIndex].Get(), nullptr, IID_PPV_ARGS(&m_computeCommandList[threadIndex]))); 		ThrowIfFailed(m_device->CreateFence(0, D3D12_FENCE_FLAG_SHARED, IID_PPV_ARGS(&m_threadFences[threadIndex])));

This is the error I am getting on windows 10 update.
Faulting application name: App1.exe, version: <IP_ADDRESS>, time stamp: 0x597662c3
Faulting module name: d3d12.dll, version: 10.0.15063.0, time stamp: 0xf28276bc
Exception code: 0xc0000005
Fault offset: 0x00085afb
Faulting process id: 0x2c2c
Faulting application start time: 0x01d304c1f396c77e
Faulting application path: D:\ZeroTutorials\Tutorial Code\Dynamic Vertex\App1\Debug\App1\AppX\App1.exe
Faulting module path: C:\WINDOWS\SYSTEM32\d3d12.dll
Report Id: dd87f202-b3bb-4f4f-995b-d7542514d902
Faulting package full name: c86a24d9-8f28-42ba-a584-0f7c5150f20b_<IP_ADDRESS>_x86__e8mhtqen2ce40
Faulting package-relative application ID: App

APIs on the ID3D12Device interface are thread-safe, including CreateCommandQueue, CreateCommandAllocator, CreateCommandList and CreateFence functions you posted above.
Could you please send us a dxdiag.txt and a crash dump of the problem you hit?  Does the sample also crash on your machine too?
To generate a crash dump in Visual Studio, select "DEBUG" menu and then click "Save Dump As" while the app hits an exception that causes the crash. To get a dxdiag.txt, run dxdiag.exe and then click "save all information".
Thanks!

app1.dmp zipped
https://www.dropbox.com/s/byk6dfnzkopf4gk/App1.zip?dl=0
DxDiag
https://www.dropbox.com/s/xwzdomykvdc3eo0/DxDiag.txt?dl=0

This is the line it is crashing on when threading

Assuming your device pointer isn't null, there might be issues with creation flags.  Try setting queueDesc.NodeMask = 1 instead of 0.

Didn't work.  And even if it did, it wouldn't explain why it magically started throwing the error after windows 10 update.  Is there a limit the number of commandAllocators that can be used at one time?  Maybe that is the cause?

The dump indicates an command queue access violation (due to null pointer) during present.
Try checking if the app deletes the command queue currently associated with the swapchain.
The app will get an E_OUTOFMEMORY error if it hits create limitation for command allocators. This is probably not the case here.

I never delete the command queue associated with the swapchain, I only reset it, reset the command list with it, and execute the command list with it.  But it is possible the command queue in question goes out of scope before execution is complete.  I am going to investigate.

@cpyburn got anything? 😄

The issue was resolved when I created the command queue on the main game thread instead of in the thread.


Auto-format Postgresql JSON/JSONB cells when editing and viewing

Is your feature request related to a problem? Please describe.
I hate it when JSON contents appear on one line when I go to edit/view them.
Describe the solution you'd like
I'd like to see a standard auto-format applied to JSON cells, which splits contents onto new lines as applicable.
Describe alternatives you've considered
Currently, you're required to copy-paste JSON contents to a text editor with JSON formatting capabilities, or to an auto-format website.

Hello, @DeflateAwning, please update the app. Probably we already fixed it.
Do you mean this JSON editor?

How do you get to that?

Right mouse click on cell -> edit -> edit cell

Is there a way to auto-format the JSON? It appears as a massive single-line string still (albeit with syntax highlighting).
I want it to display auto-formatted across multiple lines, with proper indenting.

Very slick, I am satisfied. Thank you!


Add expected_forecast_length and ingested_forecast_length coords along init_time dimension

expected_forecast_length is filled in based on the time of day-- 35 day for 0z forecasts and 10 day for 6z/12z/18z forecasts
ingested_forecast_length is used to indicate when an operational forecast is partially ingested (For 0z, the first 10 days of lead_time are released before the other 25 days)

Approved!


On a high dpi screen the layout of the Build properties dialog is broken

Description:
I think this is because my laptop is high dpi.  But it could apply to standard screens too.Image below
Note:

the prefer 32-bit label is cropped
the output section is cropped

Expected:
Layout to be good for a range of screen types including my laptop
Repro:

Open an existing F# console app or create a new one
Select Project properties
Select the Build Tab

Twitter ID: @codecutterman

High dpi and windows form is tricky, got bugfix til 4.5.2.
maybe is better to have a cleanup strategy instead of single bugfix (these fix the problem, but not make easier to improve the code)
A good medium term option is an up-for-grab to convert the project view in xaml.
Using WindowsFormHost and ElementHost to easier migration, we can convert a view at time.

easier: convert single page to xaml and embedd with ElementHost
best: container control converted to xaml, page (like build/debug/etc) are embedded with WindowsFormHost

when are all converted, remove host and we are good to go :D.
After the first page, the others are easy to do
Also a good idea is to convert the whole project from vb to c# with a project converter. vb->c# converter are pretty solid

Same bucket as #284

I'm unable to repro this on exactly the same machine.  Although it is a different build of VS and Visual F#.  I will leave this active for a bit, in case I can figure out the steps that caused it to get out of whack.  But it is probably best not to burn too many cycles on this.

Closing as it is old and not-reprod


Export DPC file error

Export file error please help me : (example : https://www.mediafire.com/file/supisz9sisi9j85/example.rar/file )
Error : 0/616 thread 'main' panicked at 'Parsing requires 4 bytes/chars on block 1', src\fuel_dpc.rs:452:31

Hello, thank you for checking out the tool.  Unfortunately that looks like a "A Plague Tale: Innocence" DPC file judging from the version "v<IP_ADDRESS> - Asobo Studio - Internal Cross Technology". The README lists the supported games and "A Plague Tale: Innocence"  is not one of them. You are welcome to submit a pull request adding support for this version of the format but I am unlikely to add it myself soon.


bug_fixes

Description

1.) Optional API key
2.) Show activity logs of in-review stories

Related Issues

Solution and Design

Test Plan

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Docs update

Checklist

[ ] My pull request is atomic and focuses on a single change.
[ ] I have documented my changes clearly and comprehensively.
[ ] I have added the required tests.

Frontend Looks Fine


UiKitView should participate in the Flutter focus tree

Right now selecting a text field inside a WebView doesn't change the framework's focus state so e.g a previously focused Flutter text field is still focused(and will show the caret animation).
On Android we've added added a focus node for AndroidViews which stays in sync with the platform view's focus state.
This is what it currently looks like on iOS when focus is moved from a Flutter text field to a WebView text field:

cc @cyanglaz

Reproducible on the latest master 1.21.0-6.0.pre.223 using webview_flutter: ^0.3.22+1. On Android this does not repro but #34169 does instead.

Minimal reproducible code sample
import 'package:flutter/material.dart';
import 'package:webview_flutter/webview_flutter.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Sample',
      home: Scaffold(
        appBar: AppBar(
          title: Text('Sample'),
        ),
        body: SingleChildScrollView(
          child: Column(
            children: [
              TextField(),
              TextField(),
              Container(height: 400, child: WebView(initialUrl: "https://www.google.com/")),
            ],
          ),
        ),
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel master, 1.21.0-6.0.pre.223, on Mac OS X 10.15.6 19G73, locale en-GB)
    • Flutter version 1.21.0-6.0.pre.223 at /Users/markus/development/flutter_master
    • Framework revision 6d487355a3 (6 hours ago), 2020-08-05 20:21:02 -0700
    • Engine revision d728f691f4
    • Dart version 2.10.0 (build 2.10.0-3.0.dev d5ff386abe)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.1)
    • Android SDK at /Users/markus/Library/Android/sdk
    • Platform android-30, build-tools 30.0.1
    • Java binary at: /Users/markus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/193.6626763/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.6)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.6, Build version 11E708
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Users/markus/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/193.6626763/Android Studio.app/Contents
    • Flutter plugin version 48.0.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] Connected device (4 available)
    • SM G950F (mobile) • ce12171c51cc001c03 • android-arm64  • Android 9 (API 28)
    • macOS (desktop)   • macos              • darwin-x64     • Mac OS X 10.15.6 19G73
    • Web Server (web)  • web-server         • web-javascript • Flutter Tools
    • Chrome (web)      • chrome             • web-javascript • Google Chrome 84.0.4147.105

• No issues found!

@hellohuanlin also consider adding integration test for this, perhaps added to:
https://github.com/flutter/flutter/blob/master/dev/integration_tests/ios_platform_view_tests/test_driver/main_test.dart

@jmagman thanks for the info!

@hellohuanlin I believe this was fixed in https://github.com/flutter/flutter/pull/105050, correct?

@jmagman yep


"feat : Replace 'checkout master' with 'git checkout master',which is a common mistake beginners makes"

help for beginners

LGTM!
PS1:
This PR includes 'feat: replace 'commit' with 'git commit'...' PR, which is rejected before because the committed file is empty.
I will merge this PR though, because the empty does not affect any harmful effect, but make sure not to make such a mistake after!
PS2:
same with #12, this PR is based on nvbn/master branch, not on jaehyeonpy/master!
I will merge this PR though, because it does not affect any harmful effect, but make sure not to make such a mistake after!