[SPARK-8784] [SQL] Add Python API for hex and unhex

Also improve the performance of hex/unhex

cc @rxin  @zhichao-li

Merged build triggered.

Merged build started.

Test build #36358 has started for   PR 7181 at commit 1a24082.

Merged build triggered.

Merged build started.

Test build #36362 has started for   PR 7181 at commit c3af78c.

Test build #36358 has finished for   PR 7181 at commit 1a24082.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
trait ExpectsInputTypes
trait AutoCastInputTypes
abstract class BinaryExpression extends Expression with trees.BinaryNode[Expression]
abstract class BinaryOperator extends BinaryExpression
abstract class BinaryArithmetic extends BinaryOperator
case class Unhex(child: Expression)
abstract class BinaryComparison extends BinaryOperator with Predicate

Merged build finished. Test FAILed.

The test failed because of ExpectsInputTypes, blocked by #7175

Test build #36362 has finished for   PR 7181 at commit c3af78c.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Unhex(child: Expression)

Merged build finished. Test FAILed.

Build triggered.

Build started.

Test build #36416 has started for   PR 7181 at commit 25156b7.

Test build #36416 has finished for   PR 7181 at commit 25156b7.

This patch fails to build.
This patch does not merge cleanly.
This patch adds the following public classes (experimental):
case class Unhex(child: Expression)

Build finished. Test FAILed.

Build triggered.

Build started.

Test build #36423 has started for   PR 7181 at commit b31fc9a.

Merged build triggered.

Merged build started.

Test build #36429 has started for   PR 7181 at commit f032fbb.

Test build #36423 has finished for   PR 7181 at commit b31fc9a.

This patch passes all tests.
This patch does not merge cleanly.
This patch adds the following public classes (experimental):
case class Unhex(child: Expression)

Build finished. Test PASSed.

Test build #36429 has finished for   PR 7181 at commit f032fbb.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class CreateNamedStruct(children: Seq[Expression]) extends Expression
case class Unhex(child: Expression)
case class ShiftLeft(left: Expression, right: Expression) extends BinaryExpression
case class ShiftRight(left: Expression, right: Expression) extends BinaryExpression

Merged build finished. Test PASSed.

Thanks - merging this in master.

I reverted the patch since it broke the build. Can you submit a new PR?

Was the build break caused by racing merges?

Not 100% sure.


Polardb 11 dev

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.3 out of 6 committers have signed the CLA.:white_check_mark: weiji0:white_check_mark: mrdrivingduck:white_check_mark: higuoxing:x: fanshen.kfs:x: fengzunbao:x: biwentangfanshen.kfs seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Fix out-of-bounds access in append_string function.

Adjust the math in calculating 'num' when total size exceeds 'max_len'.

[approve ci clang-analyzer]

[approve ci clang-analyzer]

[approve ci clang-analyzer]

[approve ci clang-analyzer]


Update master to V2

Build scripts have changed so we need to merge to master first :)

Travis PR fails because it's done to master: Error: Pull request cannot be done on master
link

Travis PR fails because it's done to master: Error: Pull request cannot be done on master

We'll have to merge manually then :/

We can still merge here, why would it need to be manual @bobylito

We can still merge here, why would it need to be manual @bobylito

Because I thought it was illegal :)

I guess this is good, but it's hard to review 😄

TBH this is impossible to review completely. Everything was already reviewed. Just check that we are on script side of things. Thanks for the time :)

Possibly wrong place to report, but this broke the links on https://community.algolia.com/instantsearch.js/examples/
https://github.com/algolia/instantsearch.js/tree/master/docs/examples/tourism
needs changing to https://github.com/algolia/instantsearch.js/tree/master/docgen/src/examples/tourism etc

Congrats guys! Looking forward to using it 😃

@Mark-Howe I don't see any errors. The new website lives on /v2 and we made the mistake to use the name of the former doc source folder for the new destination folder.
Let us know if you find any broken links :)


Request for Documentation: Add step to run webhook test server to e2e docs.

Mattermost user christopher from https://community-daily.mattermost.com has requested the following be documented:
`npm run start:webhook`

See the original post here.
This issue was generated from Mattermost using the Doc Up plugin.

@crspeller - Would you be open to creating a PR for this requested documentation update?

Request is old and out of date.


live server not showed

Type: Feature Request
output is not showed and live option is not showing on the screen
VS Code version: Code 1.92.2 (fee1edb8d6d72a0ddff41e5f71a671c23ed924b9, 2024-08-14T17:29:30.058Z)
OS version: Windows_NT x64 10.0.22631
Modes:

Is this related to an extension? Live Server or Live Preview? Please take this to the respective issue trackers:
Live Preview: https://github.com/microsoft/vscode-livepreview/issues
Live Server: https://github.com/ritwickdey/vscode-live-server/issues


You can't click the exit button on the connecting screen

Describe the Bug:
When on the connecting screen, it is not possible to click the exit button.
Steps to Reproduce:
Try to connect to an online game, and try to click the exit button.
Expected Behavior:
You should be able to click the exit button to exit the connecting screen.

This was fixed by 6ea693e.

This was fixed by 6ea693e.


Swagger Resolver is not able to resolve a ref pointer inside the following path: Paths/path/responses/responseCode/schema/$ref

Here is the swagger section:
    "/accesstoken/check" : {
      "get" : {
        "tags" : [ "access-token-controller" ],
        "summary" : "checkAccessToken",
        "operationId" : "checkAccessTokenUsingGET",
        "consumes" : [ "application/json" ],
        "produces" : [ "*/*" ],
        "parameters" : [ ],
        "responses" : {
          "200" : {
            "description" : "OK",
            "schema" : {
              "$ref" : "#/definitions/Principal"
            }
          }
        }
      },
      "parameters" : [ ]
    }
}
...
"definitions": {
Principal" : {
      "type" : "object",
      "properties" : {
        "name" : {
          "type" : "string"
        }
      }
    }
}

The error is inside Property Resolver class.

What is the error exactly?  What you've pasted above is definitely supported.

Please reopen with more info.


Add grouping settings for query field name and type

Description
Describe what this change achieves.
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

https://github.com/opensearch-project/query-insights/issues/136 cc @ansjcy


Mobiles

Great idea for the hot loading.
Have you tried this on a mobile emulator as a way to develop faster.
If I understand this you run the golang code and the qt process separately ?

Hi,
(I will comment on mobile in a moment)
So my build process tends to go:

Build a console app and build out all the functionality of the app (see _cuiinterface folder). It doesn't need to be the greatest user experience, just enough so that you can test all the apps features, get everything working, and have 95% of the functional code in place. I do this, because console app compilation is very fast.
Once I have built out what i think the functionality needs to be, I start work on the qml front end using the hotloader. While designing the front end, I can work out what interaction I need between the front end and the back end, and can start putting the needed signals and slots in place (but not recompile yet). Once the front end is able to do everything the console app can do (visually/functionally), I wire the signals/slots up to the backend functions (this should be very easy as it should be identical to how the console app works).
Recompile everything and test the front end is doing what the console app did.

Rinse and repeat until the app is complete. Its quite a nice discipline as you build out the functionality in a way totally decoupled to the front ends (and minimal compiles).
So... to your question.
I did try on an Android emulator, but there was something about the emulated chip that caused it to crash. I'm sure if I had spent a bit of time I could have got it working, however, having my android phone wired to computer, and using adb install -r deploy/android/build-debug.apk I could test the app very quickly (and frankly the compile/build is the slowest part by a long way). So I didn't find it necessary to configure the emulator.
Iphone haven't tried - would love someone to work out this process.
The golang/qt process is the same process always (I don't bother with Qt creator or anything). However they run on different Go routines.

thats makes sense.
I do exactly the same methodology with Flutter and golang.

Console app first in golang with golang backend
Then flutter app
then wire up flutter to golang.

I used therecipe stuff for a while but the licensing thing scared me off :)
If you could get your code a bit more filled out with examples and
makefile, i reckon its very possible that we can make a library with work
for Flutter and QML. That would be a pretty cool thing.
On Wed, 11 Apr 2018 at 12:13 Alex Walker<EMAIL_ADDRESS>wrote:

Hi,
(I will comment on mobile in a moment)
So my build process tends to go:

Build a console app and build out all the functionality of the app
(see _cuiinterface folder). It doesn't need to be the greatest user
experience, just enough so that you can test all the apps features, get
everything working, and have 95% of the functional code in place. I do
this, because console app compilation is very fast.
Once I have built out what i think the functionality needs to be, I
start work on the qml front end using the hotloader. While designing the
front end, I can work out what interaction I need between the front end and
the back end, and can start putting the needed signals and slots in place
(but not recompile yet). Once the front end is able to do everything the
console app can do (visually/functionally), I wire the signals/slots up to
the backend functions (this should be very easy as it should be identical
to how the console app works).
Recompile everything and test the front end is doing what the
console app did.

Rinse and repeat until the app is complete. Its quite a nice discipline as
you build out the functionality in a way totally decoupled to the front
ends (and minimal compiles).
So... to your question.
I did try on an Android emulator, but there was something about the
emulated chip that caused it to crash. I'm sure if I had spent a bit of
time I could have got it working, however, having my android phone wired to
computer, and using adb install -r deploy/android/build-debug.apk I could
test the app very quickly (and frankly the compile/build is the slowest
part by a long way). So I didn't find it necessary to configure the
emulator.
Iphone haven't tried - would love someone to work out this process.
The golang/qt process is the same process always (I don't bother with Qt
creator or anything). However they run on different Go routines.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/amlwwalker/got-qt/issues/4#issuecomment-380401185,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATuCwsQk1S6Q_A2Aen6aDerpMgY-BmvZks5tndddgaJpZM4TPLQl
.

Can you suggest some examples that would give you more/enough? I'll try and produce some.
Currently working on a project using this repo as a basis, so am slowly moving things I need into this repo.

Ok, lets see how this goes.
Ive created a branch (makefile) where I have added a make file and instructions on using it: https://github.com/amlwwalker/got-qt/blob/makefile/MAKEFILE.md
So the only thing you will need to do differently is to make sure you checkout the makefile branch before following instructions.
It hopefully will run you through all the setup to have a project running. Let me know if you get stuck. Heading out but back on this tomorrow
A

screw it, I've merged this in as I have run it multiple times on my machine. It's tricky to know if the setup/install works as I already have all that configured, but I figure if its in master more people will come across it and more likely to discover a bug....
So its now in master - the makefile I mean

i know what you mean. Wil give it a whirl :)
On Sat, 14 Apr 2018 at 12:58 Alex Walker<EMAIL_ADDRESS>wrote:

screw it, I've merged this in as I have run it multiple times on my
machine. It's tricky to know if the setup/install works as I already have
all that configured, but I figure if its in master more people will come
across it and more likely to discover a bug....
So its now in master - the makefile I mean
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/amlwwalker/got-qt/issues/4#issuecomment-381321079,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATuCwpxOZv4L70af9K1bl1iD3nPgVMcGks5todZFgaJpZM4TPLQl
.


feature flag for upcoming CDC updates

Implement a feature flag based on environment variables. Create React App (CRA) uses a similar system than Next.js, but the environment variable names need to be prefixed with REACT_APP_ (see the CRA Docs).
In development and preview environments, we can set the value of the environment variable in the .env.development file. For the production environment, I added the environment variable in Vercel settings and set it to false.
Note that I only disabled the vaccination scoreboard in this PR, I will leave other changes for a separate PR (moving the vaccination map, etc).
Preview & Local Development

Production

Doesn't this create a risk that someone makes a change that looks great in dev and in staging but completely breaks prod?


irb 1.13.1 only logging IRB::Statement::Expression rather than actual lines

Bug coincides with upgrade to IRB. I've confirmed that rolling back to 1.12.0 causes the issue to disappear.
-    irb (1.12.0)
-      rdoc
+    irb (1.13.0)
+      rdoc (>= 4.0.0)

This causes all statements/commands to be logged as: <IRB::Statement::Expression> rather than the actual command:

Thank you @rosa !


Converted tutorials to use JsonCardDataList

@dennyglee this is mostly formatting, we can pull it in if needed


Custom zsh theme

Need to move change ZSH_CUSTOM to symlink directory.
~{SHORT}~[{git|svn}~{BRANCH}~        |||      ~[{USERNAME}~]{FULL}
>

Done pariall with bullet train and customization.


OntopPRO generates duplicates resolving sparql query

Hi, I am using the prepared bundle which contains Protege 5.0.0 and Ontop OBDA Protege plugin 1.18.0.
I'm experimenting unexpected results when I perform a sparql query. For example, in my ontology there is a class which represents a category and there is another class which represents an object that is related to a category. Basically I have the following relation:
:Object :hasCategory :Category
When I make the following sparql query
SELECT * WHERE {
    ?category rdf:type :Category .
}

I expect that the results came from the database table where Category entities are stored. However, the results include this table and the table where Object entities are stored, creating a new Category for each Object. This fact generates duplicated results. I've checked the SQL translated query and it consists of the following:
SELECT
   1 AS "sQuestType", NULL AS "sLang", ('http://www.example.com/ontology#category' || REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(CAST(QVIEW1."IDCATEGORY" AS VARCHAR(4000)), ' ', '%20'), '!', '%21'), '''', '%22'), '#', '%23'), '$', '%24'), '&', '%26'), '(', '%28'), ')', '%29'), '*', '%2A'), '+', '%2B'), ',', '%2C'), '/', '%2F'), ':', '%3A'), ';', '%3B'), '=', '%3D'), '?', '%3F'), '@', '%40'), '[', '%5B'), ']', '%5D')) AS "s"
 FROM
"SCHEMA"."OBJECT" QVIEW1
WHERE
QVIEW1."IDCATEGORY" IS NOT NULL
UNION ALL
SELECT
   1 AS "sQuestType", NULL AS "sLang", ('http://www.example.com/ontology#category' || REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(REPLACE(CAST(QVIEW1."IDCATEGORY" AS VARCHAR(4000)), ' ', '%20'), '!', '%21'), '''', '%22'), '#', '%23'), '$', '%24'), '&', '%26'), '(', '%28'), ')', '%29'), '*', '%2A'), '+', '%2B'), ',', '%2C'), '/', '%2F'), ':', '%3A'), ';', '%3B'), '=', '%3D'), '?', '%3F'), '@', '%40'), '[', '%5B'), ']', '%5D')) AS "s"
 FROM
"SCHEMA"."CATEGORY" QVIEW1
WHERE
QVIEW1."IDCATEGORY" IS NOT NULL

It looks like categories are being created from object data and then from their own table. The UNION ALL clause causes duplicated results. Is correct this behaviour?
Note: This is a simplified case of the problem I am experimenting, my real  ontology is bigger.

Hi,
can you tell us also how your mappings for :Category and :Object looks like?

Hi skomlaebri.
The mappings are similar to the following:
mappingId	Category
target		:category{IDCATEGORY} a :Category .
source		SELECT IDCATEGORY
			FROM CATEGORY

mappingId	Object
target		:object{IDOBJECT} a :Object ; :hasCategory :category{IDCATEGORY} .
source		SELECT IDOBJECT, IDCATEGORY
			FROM OBJECT

I am going to develop this little test case in order to check if the behaviour I'm experimenting with the whole ontology occurs here too.
Thanks.

Hi all, I've develop the previous test case but the issue is not happening. I'm checked out that the problem appears when I use subclasses. The issue appear in the following test case:
The ontology is the following:
:Category
:Object
    :Object1
    :Object2

Where sibling classes are disjoined.
In the database are CATEGORY, OBJECT1 and OBJECT2 tables (OBJECT1 and OBJECT2 have the same structure with IDOBJECT and IDCATEGORY).
The mappings are the following:
mappingId	Category
target		:category{IDCATEGORY} a :Category .
source		SELECT IDCATEGORY FROM Category

mappingId	Object1
target		:object1{IDOBJECT} a :Object1 ; :hasCategory :category{IDCATEGORY} .
source		SELECT IDOBJECT, IDCATEGORY FROM OBJECT1

mappingId	Object2
target		:object2{IDOBJECT} a :Object2 ; :hasCategory :category{IDCATEGORY} .
source		SELECT IDOBJECT, IDCATEGORY FROM OBJECT2

In the database there are the following data:
CATEGORY TABLE:

OBJECT1 TABLE:

OBJECT2 TABLE:

Then, when run the query
PREFIX : <http://www.semanticweb.org/fabad/ontologies/2016/11/untitled-ontology-52#>

select * where {
	?category a :Category .
} order by ?category

I get the following results, which are duplicated:

Am I making any mistake?
Thanks in advance.

Hi,
the redundant information is given by the fact that in your ontology you defined :hasCategory with range :Category. So I would expect that the information is extracted three times (from IDCATEGORY in Category, in OBJECT1 and in OBJECT2). If you want to avoid the duplicates you can use the select distinct in your sparql query.
You can also try  adding foreign keys to your tables for example
ALTER TABLE OBJECT2
ADD FOREIGN KEY (IDCATEGORY)
REFERENCES CATEGORY(IDCATEGORY) OBJECT1
to allow ontop to do further optimizations

The issue has been solved by adding foreign keys to the database. In this case the problem was caused by the bad design of the database. Ontop works perfectly.
Thank you for your quick answer.


Add Sam Ritchie as a COMMITTER

I think it was an oversight to not include Sam as one of the original COMMITTERs in summingbird, so this PR adds him.
I don't think a vote is necessary, as we did not vote on the original set of COMMITTERS, but after this, I think we can consider the original 'grandfathered' committers to be closed and all other committers should go through a vote (which is not a difficult process, so if we missed somebody obvious, we can vote, it'll pass, and that'll be that).

👍

Are com.twitter.summingbird.storm.StormLaws known to be flakey? They seemed to have failed.

Also given how long these tests run, I wonder if we can disable whatever causes PRs to run tests twice (seems to run them because they are PRs and also because they are branches?)

@isnotinvain yes, stormlaws are flakey. See #605

👍

thanks all!


OCPBUGS-44396: OCPBUGS-44680: upgrade packages for cve

upgrade css-loader to have min 8.4.31 in postcss. With this upgrade we'll use 8.4.49
upgrade cypress, eslint and resolve-url-loader to have cross-spawn min 7.0.5. With the upgrade, we'll use 7.0.6

/cherry-pick release-4.17


Reverse Engineer

Hi guys hugs from Brazil here, I'd love to contribute with money at least 10 dollar, my suggestion is to analyze this site https://skiplagged.com/pokemon/, although it says that it's not working, it's ;)

skiplagged is crowdsourced.... that is completely different from how the pokemon map works. The Pokemon map uses the API and stuff. That website simply allows people to report where they saw a certain Pokemon, and places it on the map.

I don't know anything about programming, I would have thought about that but it got timers it is possible to get time even from reports?

Pokemons on skiplagged are not reported by people. They use mitm to scrape the data.

that's explained, sorry guys, bets of luck I still have the gyms with me, but can't scan anything in Brazil expect for big concentration of people, my city doesn't work, closing this here


BVH fix stale changed items, causing dangling pairs

In the octree collisions are flushed as objects are moved, whereas in the BVH they are usually flushed once per frame.
This was causing problems in the render tree in some rare situations where objects were being created (perhaps deleted and recreated using the same handle in the same frame). This PR flushes the collisions before creating objects, and set_pairable.
set_pairable may not be necessary but it is done for safety until proven not necessary.
Also a small potential for a bug is closed in remove_unordered use.
Fixes #45461
May fix #45292
Notes

Part of the reason for the difficulty, is that I'm not absolutely sure this is a bug per se in the BVH.
Even after investigating the fixed version I can't see exactly why the client code is depending on this. There don't appear to be dangling references. It could be for instance that the order of callbacks is for some reason crucial in the renderer.

Thanks!

Thanks!


perf_event_open: Invalid argument

Hello,
I'm trying to run Mitos on an Intel machine, I manage to install both Dyninst
and Mitos but when I run a command with Mitos I got the following output:
mitosrun ./examples/matmul
perf_event_open: Invalid argument
Not ready to begin sampling!
6.47073e+07
Segmentation fault

Obviously matmul does not segfault without mitos.
I've run mitos under strace to identify the faulty arguments:
strace mitosrun ./examples/matmul 2>&1 | grep perf_event
read(6, "8:perf_event:/\n7:blkio:/\n6:net_c"..., 1024) = 138
perf_event_open(0x7f51d322d148, 3613, -1, -1, 0) = -1 EINVAL (Invalid argument)
write(2, "perf_event_open: Invalid argumen"..., 34perf_event_open: Invalid argument

But according to the documentation this seems to be right ...
I've also tryed to put the pid argument at -1 and to add the
PERF_FLAG_FD_NO_GROUP without any results ...
Any idea on what can be wrong ?
I'm running debian jessie with a linux kernel 3.16.0-4-amd64 on an Intel
xeon E5520 8 cores processor.
Thanks,
David

Right now Mitos sets up the event encoding for SandyBridge and IvyBridge processors, and looks like you're on a Nehalem processor. From https://software.intel.com/sites/default/files/m/5/2/c/f/1/30320-Nehalem-PMU-Programming-Guide-Core.pdf it looks like the raw event encoding should be 0x100B instead of 0x5101CD . Also see https://lkml.org/lkml/2010/12/22/24 .
The raw event encoding is set here: https://github.com/LLNL/Mitos/blob/master/src/perfsmpl.cpp#L82 .
Could you try changing the encoding? There might be some additional tweaks needed.

Thanks, the fix seems to work.Still when I launch Mitos several times (on a loop), it does not manage to create its traces directorys
Command completed! Processing samples...

Done!

Mitos: Failed to create output directories!
Mitos: Failed to create output directories!
Mitos: Failed to create output directories!
Mitos: Failed to create output directories!

While there is still plenty of space on the drive:
root@edel-24:/tmp# df -h
Filesystem                                 Size  Used Avail Use% Mounted on
/dev/sda5                                   33G   64M   32G   1% /tmp

May it be an issue with the way mitos choose its directories name ?

When you say you launch Mitos several times, what exactly are you doing? Unfortunately you cannot run two samplers simultaneously, there are limits on the performance monitoring units. It also may be a bug in the creation of output directories, they use timestamps and are not necessarily guaranteed to be unique.

I mean on a for loop like:
for b in ./bin/*
do
    mitosrun $b
done

I'll try to add a few second sleep or do file renaming to avoid conflicts.

Ah I see. I'll keep the issue open so I remember to detect Nehalem and issue the correct encoding. Thanks!

After a small test I confirm that renaming mitos directories fix the Failed to create output directories! issue.
Thanks for your help


STYLE: Add missing source file extension to Common module tests

Add missing source file extension to Common module tests.
Inadvertently missed in commits 244567b and 1e3df39.
Fixes:
CMake Warning (dev) at CMake/ITKModuleTest.cmake:62 (add_executable):
  Policy CMP0115 is not set: Source file extensions must be explicit.  Run
  "cmake --help-policy CMP0115" for policy details.  Use the
  cmake_policy
  command to set the policy and suppress this warning.

  File:

  /Users/builder/external/ITK/Modules/Core/Common/test/itkRealTimeIntervalTest.cxx
Call Stack (most recent call first):
  Modules/Core/Common/test/CMakeLists.txt:192 (CreateTestDriver)
This warning is for project developers.  Use -Wno-dev to suppress it.

for all three test files:
itkRealTimeIntervalTest.cxx
itkRealTimeStampTest.cxx
itkMultiThreaderTypeFromEnvironmentTest.cxx

raised in:
https://open.cdash.org/build/7079554/configure
PR Checklist

[X] No API changes were made (or the changes have been approved)
[X] No major design changes were made (or the changes have been approved)

/azp run ITK.macOS

Oh, that's what it was saying!?  I probably could have made such a patch after all :)
Thanks @jhlegarreta


WIP: Add "wipe=true" support for pod's instances endpoint

…ented for app's tasks
Summary: added api flag and tests.
JIRA issues: MARATHON-8326

rebased

Nit: The title states that this is still a work in progress. Is this the cases? What do you thinkg about using the wip label and removing it once the diff is ready for reviewing?


Optimize MMR.append()

Changes include:

Remove arithmetic checks using unchecked keyword/block, none of the calculations can realistically over/underflow.
Reduce iterative memory mutations.

Future Improvements:

Rewrite using inline assembly.

testAppendEmpty - 13%+ improvement
testAppendNonempty - 30%+ improvement
function append(MMR memory self, bytes32[] memory leaves)
    internal
    pure
    returns (uint32 peaksChanged)
{
    unchecked {
        uint32 shift;
        uint32 prevLen = self.len;
        uint32 toAdd = uint32(leaves.length);
        self.len += toAdd;
        while (toAdd != 0) {
            shift = (prevLen >> peaksChanged) & 1;
            uint32 nextAdd = (toAdd + shift) >> 1;
            for (uint32 j = 0; j < nextAdd; j++) {
                uint256 doubleJ = j << 1;
                leaves[j] = shift == 1
                    ? keccak256(
                        abi.encodePacked(
                            j == 0
                                ? self.peaks[peaksChanged]
                                : leaves[doubleJ - 1],
                            leaves[doubleJ]
                        )
                    )
                    : keccak256(
                        abi.encodePacked(leaves[doubleJ], leaves[doubleJ + 1])
                    );
            }
            if (toAdd & 1 != shift) {
                self.peaks[peaksChanged] = leaves[toAdd - 1];
            } else if (shift == 1) {
                self.peaks[peaksChanged] = 0;
            }
            toAdd = nextAdd;
            ++peaksChanged;
        }
        if (peaksChanged > self.numPeaks) {
            self.numPeaks = peaksChanged;
        }
    }
}

Also, wondering why the entire MMR needs to be stored, as opposed to a root which could prove any given peak/leaf.


Host documentation on readthedocs.org

It's quite nice to be able to generate a PDF from the documentation for offline reading.

I should also note that MkDocs doesn't currently allow PDF export

The Keras documentation is just a set of Markdown files (Github-flavored). You can very easily convert them to PDF. A few solutions are available online, including: https://gitprint.com/
readthedocs.org is compatible with the Keras documentation, but the reason readthedocs exists is to offer easy hosting to projects that don't have a dedicated website. Keras does, so it's not clear to me what readthedocs would bring that Keras.io doesn't do... what do you think?

There are a number of features that make Read the Docs preferable to self-hosted Markdown. They are listed on the page that I linked above. I think the most important are PDF export, versioning, and search. Note that you can have Read the Docs serve your documentation under the keras.io domain, if that's what's giving you pause.
Also, your solution for exporting your documentation to PDF does not work well as it can only handle one Markdown document at a time. I'd like to export all of the documentation as a single PDF.

Search would be nice to have. I'll look into it.

Thanks for your consideration! You've done such a great job on API design and documentation content that it would be a shame to fall short on documentation presentation.

Here you go: https://keras.readthedocs.org/en/latest/
Search doesn't seem to work though (maybe it requires more time to build an index?).
The inability to choose a source folder (the un-editable default is docs/, whereas Keras uses docs/sources) means the documentation structure is messed up.
I'm also pretty sure Google will get pissed at the duplicate content. That is a potentially significant issue.

Hi,
It works with this docker image : https://hub.docker.com/r/fiware/md2pdf/.
I generated the documentation, with Keras==2.0.8.
PDF : https://drive.google.com/open?id=0BxaH5r4iW8cWeEdEMG1lQ0lIMFk !

Now you can find a PDF for 2.1.6 : https://github.com/JGuillaumin/keras-doc-pdf/releases


New Version: Odin-lang.Odin version dev-2024-11

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

This package can be installs successfully, but when I run odin.exe from the command prompt a message box pops up about missing LLVM-C.dll.
Manually add the Odin installation directory to PATH solves this problem.
I fear this might be a more common problem for all packages, since winget just create a symlink under %USERPROFILE%\AppData\Local\Microsoft\WinGet\Links for the main executable in the package, but what if the executable depends on other DLLs?


postdeploy scripts

In some cases it is useful to run a script when the deployment is finished, for example for use Airbrake Deploy Tracking or notify by Slack.

Yes, intend to implement callbacks 👍

Can I go with a PR for this?

Yes, intend to implement callbacks 👍

Can I go with a PR for this?

Solved via https://github.com/mrsked/mrsk/pull/51.


failed to read entry x-relative

On reading an ICS file with a key X-RELATIVE in the field RRULE:, the error occurs:
Failed to read entry /home/konfekt/.local/share/vdirsyncer/todo/all/task.ics.
Traceback (most recent call last):
  File "/home/konfekt/.local/lib/python3.6/site-packages/todoman/model.py", line 1007, in update_cache
    self.cache.add_vtodo(component, entry_path)
  File "/home/konfekt/.local/lib/python3.6/site-packages/todoman/model.py", line 651, in add_vtodo
    self._serialize_rrule(todo, 'rrule'),
  File "/home/konfekt/.local/lib/python3.6/site-packages/todoman/model.py", line 584, in _serialize_rrule
    return rrule.to_ical().decode()
  File "/home/konfekt/.local/lib/python3.6/site-packages/icalendar/prop.py", line 676, in to_ical
    typ = self.types[key]
  File "/home/konfekt/.local/lib/python3.6/site-packages/icalendar/caselessdict.py", line 42, in __getitem__
    return super(CaselessDict, self).__getitem__(key.upper())
KeyError: 'X-RELATIVE'

This key is used for a relative due date in repetitive tasks.

I don't think I quite follow. Can you provide a sample of an ICS file that fails? (feel free to change the description and anything sensitive in it).

For example,  the following file would produce this error:
BEGIN:VCALENDAR
VERSION:2.0
BEGIN:VTODO
DUE;TZID=Europe/Berlin:20200516T100000
SUMMARY:Test
DESCRIPTION:Test
CLASS:PRIVATE
LAST-MODIFIED:20200419T131234Z
RRULE:FREQ=MONTHLY;INTERVAL=3;X-RELATIVE=1
STATUS:NEEDS-ACTION
UID:4283265
END:VTODO
END:VCALENDAR

Yeah, X-RELATIVE is non standard (that's what the X- indicates). Strictly speaking, that's not a valid RRULE.
That said, we use python-dateutil to parse RRULEs (eg: todoman doesn't do the actual parsing, so you should request support for this kind of rule there.

Thank you. Could this error be handled more gracefully then? Something like a warning that a non-standard keyvalue is used? At least if the field, such as RRULE, is ignored anyway?


Allow to accept sync function forNullable/orElseAsync

We close this once by https://github.com/option-t/option-t/issues/1683#issuecomment-1456004819.
We'll reopen this when we conclude about #1683.
If you have some opinions, please comment to #1683


fix(readme): Use Correct Test Link

Also added a filter for the master branch, so only relevant tests are shown

whoops, copy/paste fail..


Bug inside the RFC6265 S5.1.1 date parser parseDate() on days 29, 30, 31

Every date on a day 29, 30 or 31 are wrongly parsed if the current date month has less days than the parsed one.
Small exemple of a cookie expiring on Mars 29 parsed Mars 01 :
$> git clone https://github.com/goinstant/tough-cookie.git && cd tough-cookie && npm install # latest

# current date is february
$>node -e 'console.log(new Date())'
Fri Feb 27 2015 15:35:53 GMT+0100 (CET)

# wrongly parsed cookie example
$> node -e "console.log(require('./lib/cookie').Cookie.parse('foo=bar; path=/; expires=Sun, 29 Mar 2015 15:13:10 GMT;'));"
Cookie="foo=bar; Expires=Sun, 01 Mar 2015 15:13:10 GMT; Path=/; hostOnly=?; aAge=?; cAge=0ms"

This is a bug inside parseDate() ( https://github.com/goinstant/tough-cookie/blob/master/lib/cookie.js#L91 )
The parser is looping on date's tokens : "Sun", "29", "Mar", "2015", "15:13:10", "GMT"
The parser is using the tokens to modify a date object : new Date();
but this date object is initialized with the current date, (currently "February")
so, when processing "29", it tries to set day 29 of month february which cause the bug :
date before setUTCDate() : Fri Feb 27 2015 16:31:16 GMT+0100 (CET)
date.setUTCDate(result[1]);
date after setUTCDate() : Sun Mar 01 2015 16:31:16 GMT+0100 (CET)
``

Sorry for the delayed response. This seems pretty serious so I'll try to get a fix going right away.

Fixed in #30. Released with<EMAIL_ADDRESS>


ValidationError: " '..' is not of type .. "

I bumped on following error using the tap-closeio on target-postgres.
target_postgres.singer_stream.SingerStreamError: ('Invalid records detected above threshold: 0. See .args` for details.', [(<ValidationError: "'2019-04-15T14:38:04.000000Z' is not of type 'object'">
It says that date: "2019-04-15T14: 38: 04.000000Z" behaves as an object type, while it should be a string with date-time format right?
I have added a file that contains the complete error, as well a file that shows my schema.
postgres_error.txt
catalog_activities.zip

@DylanDriessen ahha! This one is actually pretty straightforward:
    "date": {
      "inclusion": "automatic",
      "anyOf": [
        {
          "type": [
            "string"
          ],
          "format": "date-time"
        },
        {
          "type": [
            "string"
          ]
        }
      ]
    },

Your problem types are of anyOf type. Currently target-postgres doesn't support this...

JSON Schema combinations such as anyOf and allOf are not supported.

We've talked before about trying to support it in one mechanism or another, but right now it's not gonna go well 😢. We could make the error message better though if you think that'd be helpful.

@AlexanderMann I've tried fixing this already before this post, seems like my current code didn't save to the one I've uploaded. I changed this with:
 "date": {                   "inclusion": "automatic",                   "format": "date-time",                   "type": [                     "string"                   ]                 }
But this resolves in the same error..

@DylanDriessen can you upload the new schema and error logs for the updated?

Here is the new specific schema with his error log which is the exactly the same as before
postgres_error.txt
catalog_activities.zip

@DylanDriessen I threw together a simple set of tests in here: https://github.com/datamill-co/target-postgres/pull/115
If you look at the commit history, I have the first commit as a fail with your original anyOf type. After I change just that everything seems to work correctly.
I'm wondering if maybe some of your configs are pointing at the wrong place?

I see, after changing it to your schema it still gives me the same error.. :cry:
Do you mean my target-postgres config file? Because despite the leads and activities schemas, everything else works (tasks, user,..).
Could the problem be with my metadata configuration?

Yeah, I mean it might be worth simplifying your schema down to just the working stuff. ie, get the trouble schema working even by deleting the troublesome field from the schema altogether, maybe remove the metadata, then start adding things back one by one till you get the issue again.
I am curious as to whether the problem you're having might be that somehow what you're editing/making changes to isn't being picked up by target-postgres, like somehow the file path to your config/schema is off by a little bit, there's something cached, etc.?

Seems like it caches indeed, even when I only set 1 property it returns the whole schema

dylan@Dylan-linux:~/Documents/Stage/Singer$

  ~/.virtualenvs/tap-closeio/bin/tap-closeio
    --config tap-closeio/config.json
    --catalog tap-closeio/catalog_activities.json |

  ~/.virtualenvs/target-postgres/bin/target-postgres
    --config target-postgres/config.json

Based on your command, I'd do the following:

check your config.json for target-postgres. I presume that's a standard config which just has a few connection things in it
pipe the output from tap-closeio into a json file so that you can look at that. I am willing to bet that it's always outputting the same thing no matter what changes you make

test.zip
It does :man_facepalming:, what do I owe?

Is there a way I can solve this @AlexanderMann ? I've been trying some things, but nothing seems to work out.

@DylanDriessen welllllllll tap-closeio (and most api taps) hardcode their schemas. This makes development of this suppppper simple and allows for validating breaking schema changes from the source.
For targets, there are a couple options:

fully support JSON Schema (for target-postgres, currently we don't have support for anyOf, allOf, and global/non-local $refs)
ignore/gracefully handle the bits of JSON Schema we don't handle

The better option is to support as much of JSON Schema as possible. We have a couple issues kicking around in here which speak to that effect.
The simpler option is to gracefully handle things (potentially a config/feature flag).
If you wanna discuss trying to support the missing features that'd make your life simpler here I'm happy to help however I can!

This issue has been open for a while without any activity. @DylanDriessen did you ever get a conclusion to this?
For the record, I think this issue is most closely related to these other open issues:
#14

I've changed the hardcoded schema from closeio. The problem was indeed the anyOf in the schema. Thanks for all the help @AlexanderMann!


Small changes to dockerfiles

Update flower/monitor with proper changes for the new image
Change the frontend and backend container images to the appropriate s2i scripts

https://github.com/sclorg/s2i-python-container/tree/master/3.8 for the backend
https://github.com/sclorg/s2i-nodejs-container/tree/master/14 for the frontend

Remove root Dockerfile that is no longer used
Fix/update docker-compose (I am not very familiar with docker-compose but I wasn't able to get it working without network_mode: host), anyways, I think https://github.com/ibutsu/ibutsu-server/pull/191 using podman pods is a much better strategy

The changes to the dockerfiles for the frontend/backend are meant to mimic the s2i strategy we use in OCP, for more information, see the README's at the sclorg links above.

This got everything running in Docker Compose without host networking.
With these changes I get in the backend/worker/scheduler:

sqlalchemy.exc.OperationalError: (psycopg2.OperationalError) could not connect to server: Connection refused
	Is the server running on host "localhost" (<IP_ADDRESS>) and accepting
	TCP/IP connections on port 5432?
could not connect to server: Cannot assign requested address
	Is the server running on host "localhost" (::1) and accepting
	TCP/IP connections on port 5432?

There must be something messed up with my local docker though so I'll take your word for it :D I don't plan on using docker anymore anyways. Went ahead and made those changes.


I Would like to be able to customize the InternalLogin timeout period

The default timeout for internal login is 3s, which is not appropriate in some places and often results in a timeout error. But I didn't find any place to set the timeout for internal login.

Hi, @ByYogi.
B1SLayer does not specify a timeout value for the login request. Therefore, it should use the default 100 seconds for the HttpClient.
Where did you get this 3s value from?

Hi, @ByYogi.
B1SLayer does not specify a timeout value for the login request. Therefore, it should use the default 100 seconds for the HttpClient.
Where did you get this 3s value from?

I got 3s value from my error log
Finally, I modified the source code to add WithTimout solution

Hi @bgmulinari
Thank you for responding so positively.
I think it might be possible to solve this problem in other ways. Instead of WithTimeout?
This is B1SL responsed timout error:
<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">

      <title>502 Proxy Error</title>

Proxy Error

The proxy server received an invalid  response from an upstream server.
  The proxy server could not handle the request

Reason: Error reading from remote server

@ByYogi, the HTTP error 502 Bad Gateway is not a timeout error.
This indicates something wrong on the server side. I recommend you check your server and Service Layer installation, or even contacting SAP support.
I don't believe this issue is related to B1SLayer.

ok


Refactor PreviewViewModel

Refactor preview view model

More readable code that uses new syntaxes
Avoids constantly re-instantiating the same objects.
Improves visibility (private/protected/public)
Remove unused code.

Also, I added new xml extensions (xsl, xslt, xlf, xsd)
Validation
How did you test these changes?

[x] Built and ran the app
[ ] Tested the changes for accessibility

ok

@winston-de Thank you a million times for your review.


Fix #7950: Prevent NPD dasync cipher helpers when cipher is null.

Prevent NPD when using dasync without AES_ASM available.
Checklist

[ ] tests are added or updated - needed??

The crash scenario in https://github.com/openssl/openssl/issues/7950#issuecomment-454475739 will also not apply if dasync_aes128_cbc_hmac_sha1 is not available.

The transcripts below show that in the default setup the dasync_aes_128_cbc_hmac_sha1 cipher gets selected, and after disabling AES-NI (export OPENSSL_ia32cap=~0xFFFFFFFF) the dasync_aes_128_cbc is selected instead. (Watch out for "USING CIPHER: ...")
If it is true that the dasync_aes_128_cbc_hmac_sha1 gets deselected automatically when EVP_aes_128_cbc_hmac_sha1() returns NULL, then this means that the NPD checks of this pull request are not  necessary. However, I can't prove it because I don't understand where and when this deselection happens in the code.
Two more things puzzle me: The first one is the internal error, which occurs in the case where dasync_aes_128_cbc_hmac_sha1 is used.
<PHONE_NUMBER>93760:error:14068044:SSL routines:do_ssl3_write:internal error:ssl/record/rec_layer_s3.c:1031:

The second one is the fact, that the dasync_aes_128_cbc_hmac_sha1 was only chosen because the peer was (unintentially) an openssl s_server using the preinstalled OpenSSL 1.0.2q app. If I use openssl 3.0 for both the client and the server, then dasync_aes_128_cbc gets selected always.
I have to admit that I am a bit lost and don't know what to do with this pull request. @mattcaswell, you worked on the dasync engine previously, maybe you can provide some helpful insight?
openssl s_client (default setup)
~/src/openssl$ util/shlib_wrap.sh apps/openssl s_client -no_tls1_3 -CAfile test/certs/rootcert.pem -cipher AES128-SHA -engine engines/dasync.so
engine "dasync" set.
CONNECTED(00000003)
depth=1 CN = Root CA
verify return:1
depth=0 CN = server.example
verify return:1
USING CIPHER: dasync_aes_128_cbc_hmac_sha1
139820422893760:error:14068044:SSL routines:do_ssl3_write:internal error:ssl/record/rec_layer_s3.c:1031:
---
Certificate chain
 0 s:CN = server.example
   i:CN = Root CA
---
Server certificate
-----BEGIN CERTIFICATE-----
MIIDJTCCAg2gAwIBAgIBAjANBgkqhkiG9w0BAQsFADASMRAwDgYDVQQDDAdSb290
IENBMCAXDTE2MDExNDIyMjk0NloYDzIxMTYwMTE1MjIyOTQ2WjAZMRcwFQYDVQQD
...

openssl s_client (no AES-NI)
~/src/openssl$ export OPENSSL_ia32cap=~0xFFFFFFFF
~/src/openssl$ util/shlib_wrap.sh apps/openssl s_client -no_tls1_3 -CAfile test/certs/rootcert.pem -cipher AES128-SHA -engine engines/dasync.so
engine "dasync" set.
CONNECTED(00000003)
depth=1 CN = Root CA
verify return:1
depth=0 CN = server.example
verify return:1
USING CIPHER: dasync_aes_128_cbc
USING CIPHER: dasync_aes_128_cbc
---
Certificate chain
 0 s:CN = server.example
   i:CN = Root CA
---
Server certificate
-----BEGIN CERTIFICATE-----
MIIDJTCCAg2gAwIBAgIBAjANBgkqhkiG9w0BAQsFADASMRAwDgYDVQQDDAdSb290
IENBMCAXDTE2MDExNDIyMjk0NloYDzIxMTYwMTE1MjIyOTQ2WjAZMRcwFQYDVQQD
...

The transcript was obtained on the pr-8255 branch with the following patch. (Same results on master).
commit c1493bd88d84dbfaa282f9b48a08ca2f2b8e9eca (HEAD -> pr-8255)
Author: Dr. Matthias St. Pierre<EMAIL_ADDRESS>Date:   Tue Feb 19 22:51:57 2019 +0100

    Add some printfs

diff --git a/engines/e_dasync.c b/engines/e_dasync.c
index 46985a2149..fe8f1e3a50 100644
--- a/engines/e_dasync.c
+++ b/engines/e_dasync.c
@@ -150,6 +150,7 @@ struct dasync_pipeline_ctx {
 static EVP_CIPHER *_hidden_aes_128_cbc = NULL;
 static const EVP_CIPHER *dasync_aes_128_cbc(void)
 {
+    fprintf(stderr, "USING CIPHER: dasync_aes_128_cbc\n");
     return _hidden_aes_128_cbc;
 }

@@ -160,6 +161,7 @@ static const EVP_CIPHER *dasync_aes_128_cbc(void)
 static EVP_CIPHER *_hidden_aes_128_cbc_hmac_sha1 = NULL;
 static const EVP_CIPHER *dasync_aes_128_cbc_hmac_sha1(void)
 {
+    fprintf(stderr, "USING CIPHER: dasync_aes_128_cbc_hmac_sha1\n");
     return _hidden_aes_128_cbc_hmac_sha1;
 }

Two more things puzzle me: The first thing is the internal error, which occurs in the case where dasync_aes_128_cbc_hmac_sha1 is used.

Looks like that cipher is broken. Fix in #8291.

The second thing is the fact that the dasync_aes_128_cbc_hmac_sha1 was only chosen because the peer was (unintentially) an openssl s_server using the preinstalled OpenSSL 1.0.2q app. If I use openssl 3.0 for both the client and the server, then dasync_aes_128_cbc gets selected always.

That cipher is closely tied to TLS - and in particular the "old" way of doing things, i.e. mac-then-encrypt. It does not get used if we negotiate the encrypt-then-mac (ETM) extension - which will happen if two 3.0 instances are talking to each other. If you use an older version of OpenSSL as one of the endpoints then ETM does not get negotiated and we end up using this cipher.
It is possible to force s_client (or s_server) to disable its ETM support. Unfortunately I don't think its possible using a command line arg - you have to use a config file:
openssl_conf = default_conf

[ default_conf ]

ssl_conf = ssl_sect

[ssl_sect]

system_default = ssl_default_sect

[ssl_default_sect]
Options=-EncryptThenMac

And then:
OPENSSL_CONF=/path/to/config/file/noetm.cnf openssl s_client -no_tls1_3 -cipher AES128-SHA

If it is true that the dasync_aes_128_cbc_hmac_sha1 gets deselected automatically when EVP_aes_128_cbc_hmac_sha1() returns NULL, then this means that the NPD checks of this pull request are not necessary. However, I can't prove it because I don't understand where and when this deselection happens in the code.

In ossl_init_ssl_base we add all the ciphers that we might want to use:
https://github.com/openssl/openssl/blob/7dec815ecde29c1eabed6b5832c7025c4eafad19/ssl/ssl_init.c#L58-L61
EVP_add_cipher() does nothing if it is passed NULL and otherwise adds the cipher to the OBJ database.
We find the cipher to use in ssl_cipher_get_evp. This finds the EVP_CIPHER and EVP_MD objects for the constituent parts of the TLS ciphersuite. Having successfully found those (and only if successful) it then goes on to check if we have an optimised "stitched" version:
https://github.com/openssl/openssl/blob/7dec815ecde29c1eabed6b5832c7025c4eafad19/ssl/ssl_ciph.c#L548-L583
The EVP_get_cipherbyname("AES-128-CBC-HMAC-SHA1") call will return NULL if the cipher is not in the OBJ database. If that occurs then we don't use the "stitched" version and just use the EVP_CIPHER and EVP_MD objects that we already found.

Thank you Matt for your detailed answer. This helps me a lot to better understand the internals.
Regarding this pull request: If I read your reply correctly, you confirm my conjecture that the dasync_aes_128_cbc_hmac_sha1 gets deselected automatically when EVP_aes_128_cbc_hmac_sha1() returns NULL. For me the logical consequence is that the null pointer checks in this pull request are not necessary. Instead, adding an explaining comment somewhere would be sufficient. Do you agree with this point of view?

It is possible to force s_client (or s_server) to disable its ETM support. Unfortunately I don't think its possible using a command line arg - you have to use a config file

In my case, since I just want to debug the behaviour of the app, I might as well just patch the source code:
https://github.com/openssl/openssl/blob/7dec815ecde29c1eabed6b5832c7025c4eafad19/ssl/ssl_ciph.c#L553-L554

Regarding this pull request: If I read your reply correctly, you confirm my conjecture that the dasync_aes_128_cbc_hmac_sha1 gets deselected automatically when EVP_aes_128_cbc_hmac_sha1() returns NULL.

Correct.

For me the logical consequence is that the null pointer checks in this pull request are not necessary. Instead, adding an explaining comment somewhere would be sufficient. Do you agree with this point of view?

That sounds reasonable.

In my case, since I just want to debug the behaviour of the app, I might as well just patch the source code:

Yes - but probably not at the point in the code you highlight since ETM touches a number of different parts of the code. You could simply patch s_server or s_client to set the option SSL_OP_NO_ENCRYPT_THEN_MAC.

@iamihalcea since you reported this issue, would you like to revert the null pointer checks yourself and add an explaining comment in this pull request instead?  If not, just let me know, then I will create a separate pull request for it.  The argumentation goes roughly as follows:
Although in principle EVP_aes_128_cbc_hmac_sha1() could return a null pointer if AES-NI is not available, it will not happen here,
https://github.com/openssl/openssl/blob/644ab5c2b59bdd76cef6511d45fa9189a8b5a054/engines/e_dasync.c#L596
because the  code is only used by the dasync_aes_128_cbc_hmac_sha1() cipher, which gets automatically deselected if EVP_aes_128_cbc_hmac_sha1() == NULL.
https://github.com/openssl/openssl/blob/644ab5c2b59bdd76cef6511d45fa9189a8b5a054/engines/e_dasync.c#L161-L164

@iamihalcea since you reported this issue, would you like to revert the null pointer checks yourself and add an explaining comment in this pull request instead?

Sorry @iamihalcea, I mixed it up. It was @wurongxin1987 who reported this issue, not you.
Please take a look at pull request #8305.

Issue #7950 has been fixed now on master and 1.1.1 by #8305, so I'm closing this pull request. Thank you nevertheless for your efforts @iamihalcea.


Request:  street view animation on view update

I am wondering if it is possible to add animation effect (eg. fade in / fade out) to smooth the view updating on each step? Both map view and the street view look jumping in drive mode.

IMO, it's not possible, but I am open to an idea.


It's fail to Installing dependencies

i quick start  storybook from npm i -g getstorybook  & getstorybook .but it's fail  at the second step "Installing dependencies  ". can anyone help?

Please open an issue over at the new repo if you're still experiencing this issue, thanks!


Crash on exit

See https://github.com/TheQwertiest/foo_spider_monkey_panel/issues/30#issuecomment-431593465

Build: 21:35:16, Oct 20 2018

I haven't had any 'out of memory' issues so far
exit crashes are now occurring at a much reduced frequency compared with the previous build
in several hours trying to find a reproducible way of triggering the crashes for you I had 3 exit crashes
there was no particular thing that seemed to trigger the exit crashes
I tried making DUI layouts more complex: adding many tabs each with several splitters; I always kept at least 1-2 Yt Mgr in image mode in each tab in case it was related to image handling; other panels in splitters weren't SMP
1 crash occurred after adding tabs, another after removing one tab and another after clicking through all the tabs. But further tries with these actions didn't trigger the exit crashes
So I haven't been able to pin down a precise trigger

I was hoping that it would be something trivial or at least reproducible (suspected that it was not the case though), but oh well...
Anyways, thanks for testing =)

Thanks for the testing! I was afraid, that this would be the case...
Could you attach crash dumps, that were generated while you were using this specific build (Build: 21:35:16, Oct 20 2018)?
PS: For future reference - to debug crash dumps I need to know the specific build of the component that was used, since I need to download corresponding debug symbols, so that I can see the stack trace.

Build: 21:35:16, Oct 20 2018 (according  "About Spider Monkey Panel" in components)
Here's 6:
Exit crash dumps.zip

Should be fixed in the latest build (once it's done building)

I thought this issue had gone away... Well it had until now! But I just had another crash on shutdown. This was after benchmarking tests and switching between SMP/JSP, altering code in those panels etc... Although the recent events show the last accessed panel was JSP, crash location is stated as being in foo_spider_monkey_panel. The listed components, in failure....txt, confirm version as 1.0.4. DUI multi-tab layout.
Exit crash v1.0.4.zip

Ugh, this shucks...
Thanks for the report!


Support winit-0.26

This adds winit 0.26 support to imgui-winit-support as non-default. This is a non-breaking change and I've tested all versions of winit with the crate.
I have bumped the version number, as it would be greatly appreciated a patch version could be released. This is currently the last blocking piece of updating my project to winit 0.26.

Thanks -- I need to release a patch anyway. I'll try to get that done on Sunday night

Great, thanks!

We should update these places too:

https://github.com/imgui-rs/imgui-rs/blob/main/.github/workflows/ci.yml#L60
https://github.com/imgui-rs/imgui-rs/blob/main/xtask/src/main.rs#L80

Along with the additions Thom mentions - if the version number is left untouched it'd make it easier to merge in, then we can cherry-pick the change into the branch referenced in #581 (which includes my attempts at backporting the text-input bugfixs)

All three things changed!

going to merge this in once it goes green

thanks @cwfitzgerald !

We should remember to move from 0.25-by-default to 0.26-by-default on the next semver-major change of imgui-winit-support


possibility of displaying 2 images side by side

In my case, I want to label 2 modalities simultaneously. For example, if I have -
└───RGB
│   rgb_image1.png
│   rgb_image2.png
└─── DEPTH
│  depth_image1.png
│  depth_image2.png
I wish to see both rgb_image1.png and depth_image1.png on the canvas, label the RGB image and save the same annotations for both images as xml format. How would you advise me to proceed with this?

OS: Ubuntu 16.04
PyQt version: Python 2 QT 4

@priyam1994 You can write a simple script that copies and modifies the xml of rgb_image1.png so that it works for depth_image1.png, and then open two instances of labelImg and look at them.
This is a fairly specific use case, so I don't think this is a feature that we'll be adding in the foreseeable future.


Failing to Compile

https://pastebin.com/pmmcA70K
Trying to compile gives me this error, still new to using spigot plugins so any help would be appreciated

Thanks so much it compiles now :)


#CMock #Ceedling Adding va_list to :treat_as_array: produces a TypeError

Hi,
So im using Ceedling to create tests for some inherited code I didnt write. I was looking forward to using CMock so I could avoid having to create mocks myselfs but im running into some issues. Currently im just testing that ceedling works by running empty autogenerated tests.
CMock wrongli mocks the following function:
void tx_push (struct txbuffer *tx, const char *format, va_list ap);
Outputing the following error:
build/test/mocks/mock_txbuffer.c: In function ‘CMockExpectParameters_tx_push’:
build/test/mocks/mock_txbuffer.c:420:31: warning: ‘sizeof’ on array function parameter ‘ap’ will return size of ‘__va_list_tag *’ [-Wsizeof-array-argument]
          sizeof(va_list[sizeof(ap) == sizeof(va_list) ? 1 : -1])); /* add va_list to :treat_as_array if this causes an error */
                               ^
build/test/mocks/mock_txbuffer.c:414:149: note: declared here
 void CMockExpectParameters_tx_push(CMOCK_tx_push_CALL_INSTANCE* cmock_call_instance, struct txbuffer* tx, int tx_Depth, const char* format, va_list ap)
                                                                                ^~
build/test/mocks/mock_txbuffer.c:420:24: error: size of unnamed array is negative
          sizeof(va_list[sizeof(ap) == sizeof(va_list) ? 1 : -1])); /* add va_list to :treat_as_array if this causes an error */
                        ^
ERROR: Shell command failed.

I tried fixing this issue by doing as recommended and adding va_list to :treat_as_array: , but now i get a different error:
Test 'test_conn.c'
------------------
Generating include list for log.h...
Generating include list for memory.h...
Generating include list for rxbuffer.h...
Generating include list for txbuffer.h...
Creating mock for log...
rake aborted!
TypeError: no implicit conversion of String into Integer
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/vendor/cmock/lib/cmock_header_parser.rb:421:in `block in parse_args'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/vendor/cmock/lib/cmock_header_parser.rb:411:in `each'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/vendor/cmock/lib/cmock_header_parser.rb:411:in `parse_args'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/vendor/cmock/lib/cmock_header_parser.rb:588:in `parse_declaration'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/vendor/cmock/lib/cmock_header_parser.rb:45:in `block in parse'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/vendor/cmock/lib/cmock_header_parser.rb:44:in `map'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/vendor/cmock/lib/cmock_header_parser.rb:44:in `parse'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/vendor/cmock/lib/cmock.rb:48:in `generate_mock'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/vendor/cmock/lib/cmock.rb:32:in `block in setup_mocks'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/vendor/cmock/lib/cmock.rb:31:in `each'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/vendor/cmock/lib/cmock.rb:31:in `setup_mocks'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/lib/ceedling/generator.rb:50:in `generate_mock'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/lib/ceedling/rules_cmock.rake:8:in `block in <top (required)>'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/lib/ceedling/task_invoker.rb:60:in `block in invoke_test_mocks'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/lib/ceedling/task_invoker.rb:58:in `invoke_test_mocks'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/lib/ceedling/preprocessinator.rb:28:in `preprocess_test_and_invoke_test_mocks'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/lib/ceedling/test_invoker.rb:84:in `block in setup_and_invoke'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/lib/ceedling/test_invoker.rb:51:in `each'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/lib/ceedling/test_invoker.rb:51:in `setup_and_invoke'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/lib/ceedling/rules_tests.rake:70:in `block (2 levels) in <top (required)>'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/bin/ceedling:345:in `block in <top (required)>'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/bin/ceedling:332:in `<top (required)>'
/usr/local/bin/ceedling:23:in `load'
/usr/local/bin/ceedling:23:in `<main>'
Tasks: TOP => build/test/mocks/mock_log.c
(See full trace by running task with --trace)

--------------------
OVERALL TEST SUMMARY
--------------------

No tests executed.

TypeError: no implicit conversion of String into Integer
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/vendor/cmock/lib/cmock_header_parser.rb:421:in `block in parse_args'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/vendor/cmock/lib/cmock_header_parser.rb:411:in `each'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/vendor/cmock/lib/cmock_header_parser.rb:411:in `parse_args'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/vendor/cmock/lib/cmock_header_parser.rb:588:in `parse_declaration'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/vendor/cmock/lib/cmock_header_parser.rb:45:in `block in parse'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/vendor/cmock/lib/cmock_header_parser.rb:44:in `map'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/vendor/cmock/lib/cmock_header_parser.rb:44:in `parse'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/vendor/cmock/lib/cmock.rb:48:in `generate_mock'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/vendor/cmock/lib/cmock.rb:32:in `block in setup_mocks'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/vendor/cmock/lib/cmock.rb:31:in `each'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/vendor/cmock/lib/cmock.rb:31:in `setup_mocks'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/lib/ceedling/generator.rb:50:in `generate_mock'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/lib/ceedling/rules_cmock.rake:8:in `block in <top (required)>'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/lib/ceedling/task_invoker.rb:60:in `block in invoke_test_mocks'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/lib/ceedling/task_invoker.rb:58:in `invoke_test_mocks'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/lib/ceedling/preprocessinator.rb:28:in `preprocess_test_and_invoke_test_mocks'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/lib/ceedling/test_invoker.rb:84:in `block in setup_and_invoke'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/lib/ceedling/test_invoker.rb:51:in `each'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/lib/ceedling/test_invoker.rb:51:in `setup_and_invoke'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/lib/ceedling/rules_tests.rake:70:in `block (2 levels) in <top (required)>'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/bin/ceedling:345:in `block in <top (required)>'
/var/lib/gems/2.5.0/gems/ceedling-0.31.1/bin/ceedling:332:in `<top (required)>'
/usr/local/bin/ceedling:23:in `load'
/usr/local/bin/ceedling:23:in `<main>'
Tasks: TOP => build/test/mocks/mock_log.c
(See full trace by running task with --trace)

--------------------
OVERALL TEST SUMMARY
--------------------

No tests executed.

I just want to make sure that this is not something im doing wrong, in the way I am adding va_list to :treat_as_array: , so here is a view of my .yml file :
:cmock:
  :mock_prefix: mock_
  :when_no_prototypes: :warn
  :enforce_strict_ordering: TRUE
  :plugins:
    - :array
    - :ignore
    - :callback
  :treat_as:
    uint8:    HEX8
    uint16:   HEX16
    uint32:   UINT32
    int8:     INT8
    bool:     UINT8
  :treat_as_array:
      - va_list
  :includes:
    - <assert.h>
    - <getopt.h>
    - <libgen.h>
    - <stdbool.h>
    - <stdint.h>
    - <stdio.h>
    - <stdlib.h>
    - <string.h>

EDIT: My ceedling version (Ubuntu 18.04.2):
   Ceedling:: 0.31.1
      Unity:: 2.5.4
      CMock:: 2.5.4
 CException:: 1.3.3

Also If I remove va_list from :treat_as_array: I can run tests of modules with depencies succesfully, as long as there are no va_list functions. So this leads me to think there may be an issue with the way I add va_list to :treat_as_array:.
Any help will be greatly appreciated, as I seem to have reached the end of the tunnel as far as solution I can come up with and test. I guess I just want to find out wether this is a mistake on my end, or an unsupported feature as it might be suggested in some other issues :
#397
#204
#129
Best regards,
GCT.

So it seems I was adding the va_list wrongly.
It needs to be :
  :treat_as_array:
    va_list

Now cmock is no longer complaining about the mocked module.
I couldnt find any examples of how to add something to :treat_as_array:, so hopefully this helps.


Bug Report: can't find a way to compile the game my self

Please check for duplicates, similar issues or closed issues with the same problem, as well performing simple troubleshooting steps (such as deleting the export folder if it's an issue with compiling, reinstalling Kade Engine) before submitting an issue.
If you are playing a downloaded version of the game, what operating system are you using?
Windows (x86), Windows (x86_64), Linux, or macOS? Specify below.
What version of the game are you using? Look in the bottom left corner of the main menu.
Have you identified any steps to reproduce the bug? If so, please describe them below. Use images if possible.
Please describe your issue. Provide extensive detail and images if possible.

literally BUILDING.md
we officially support macos now too

you gave us absolutely no information, also this is a discussion


Difference between stable-ts and whisperx

Hi, I noticed that you use Whisper model from stable-ts for transcribing, instead of using the WhisperX approach. Curious to know the difference and why picking stable-ts. Because WhisperX also claims to use cut/merge/phoneme model to improve the correctness of timestamps. Does stable-ts show better performance? The reason is WhisperX is super fast compared to using vanilla whisper model.
Also I see in readme that the evaluation data usually contains 1~3 speakers. If the audio contains more speakers, sometimes even 10, will the current pipeline work out of box? Thank you.

Hi @bryanyzhu,
When this procedure was developed stable-ts seemed to provide a more consistent splitting. This may have changed as the WhisperX roadmap mentioned that improvements in this area were to be expected in the upcoming versions. So perhaps it has already happened. I do not know. However, what I like about stable-ts is the fact that is easily configurable, which may come handy for multilingual projects, as is ours.
Yes, WhisperX is faster than vanilla Whisper, but perhaps stable-ts can be hacked to use something like Faster Whisper or even WhisperX (but just for the raw transcription).
Regarding the number of speakers, please, try and let me know. The results may still be reasonable but they are unlikely to be optimal. My guess is that you will need to adjust the UMAP and HDBSCAN parameters for your specific dataset in order to obtain optimal results.
If you are familiar with these tools you will probably have some hints of what may work in order to obtain more clusters. Otherwise, it will involve some learning curve and a considerable amount of trial and error.
The idea for the future would be to have some sort of ML or heuristics that will adjust those parameters automatically. But that is a lot of work and I have moved into something else.

Thanks a lot for your reply. Let me play with these tools for a bit, and see if the chosen parameters can generalize. I'm new to this domain, so probably will take some time.


Individual Membership Subgroup next meeting

We have an upcoming board meeting where @hackygolucky needs to present our proposal. We need Tracy and @amiller-gh to attend. Looking for something in the evening Eastern Time.

As an FYI I'm away for 2 weeks starting Dec 2nd.

It's likely we can do this async – Tracy and I are meeting up today to get a more formal draft proposal written. We’ll share out for additional word-smithing after we get a baseline and determine if we need another in-person meeting from there. I think we’re in relative alignment about the details, it just needs codification 🙂
Adam Miller
[in] Design Systems Tech @ LinkedIn
⬡ Node.js Community Committee Chairperson
<EMAIL_ADDRESS>(413) 454-8261
On Nov 26, 2018, 11:19 AM -0800, Michael Dawson<EMAIL_ADDRESS>wrote:

As an FYI I'm away for 2 weeks starting Dec 2nd.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Closing because of discussion from our last meeting 🙂
See: https://github.com/nodejs/community-committee/issues/423#issuecomment-442985247
We'll evaluate the need for a final doc planning meeting depending on feedback from the proposal Tracy and I are polishing!


Missing dependency glog when trying to install libunilog via rpm

want to install the vitis-ai runtime libraries on my ZCU16 custom device. However, upon copying the respective files and trying the following command:
rpm -ivh --force --ignoresize libtarget-factory-1.2.0-r10.aarch64.rpm
I get greeted with this error message:
error: Failed dependencies:
libunilog >= 1.2.0 is needed by libtarget-factory-1.2.0-r10.aarch64
/bin/sh is needed by libtarget-factory-1.2.0-r10.aarch64

@kevinkit
You can follow the step 5 in https://github.com/Xilinx/Vitis-AI/tree/v1.1/Vitis-AI-Library#setting-up-the-host to build the glog-0.4.0,  then install it on the target, as the following show.

Thank you, I will try that.
Why is this mentioned in the v1.1, but not in v1.2 anymore?

I tried it, but sadly the error remains:

Besides glog it also points out that /bin/sh is needed, however when I look at the folder /bin it is clearly there.
Nevetheless, I checked with ldconfig -p | grep libglog to show me the installed libraries:

However, when I check, libglog.so.0.4.0 is there.

@kevinkit
add this option when install the lib: --nodeps
and try it again.

thank you very much - this resulted in promising results:

I will report back when this may lead to other errors, however for now I will close this issue.
Again, thank you very much for your help


Custom name for exported pdf file

Is your feature request related to a problem? Please describe.
I would like to be able to specify a custom name for the exported pdf file and not always have slidev-exported.pdf.
Describe the solution you'd like
I would like to be able to specify the pdf file name on the front matter, where download: true is.

You can do it already with the --output flag
npx slidev export --output my-custom-name


30: replace assert(a == b) for assert.equal(a, b)

Fixes #30 and improves on PR #69.
PR #69 blindly replaces all strict === comparisons for assert.equal(), this PR replaces:

String and id strict === comparisons for assert.equal();
Non-string strict === comparisons for assert.strictEqual();
non-strict == comparisons for assert.equal().

PR #69 original text:

Fixes #30 by replacing instances of assert(a === b) for assert.equal(a, b). For those curious this was done with sed:
find . -name '*.js' -type f -print0 | xargs -0 sed -i '' 's/assert(\([^|&]*\) ===\([^|&]*\))/assert.equal(\1,\2)/g'

I've checked the number of tests ran was the same before and after the change and visually inspected all changes so we should be good.
Original issue:

Specifically, assert.equal should always be used with strings. Some string comparisons will fail triple-equality check. I've run into this in getting the tests to pass in the couch adapter.
From a UX perspective, the methods will produce more helpful and descriptive failure messages.

cc: @tjwebb, @devinivy

We should do the same for waterline core.

Wild!  This looks good :+1:

Thanks @devinivy! I'll merge this now because any other PR will probably cause merge conflicts in this.


Impossible to use in ASM.JS

wasm-bindgen, as written, makes it impossible to convert to ASM.JS, as it leaves unresolved symbols, and the stub file generated assumes WebAssembly.

Well, the tool is dedicated toward Wasm, as it's even in the name.
Nothing here could be polyfilled in Asm.js anyway, all asm.js function imports must be for side-effects only, that is, they must have the signature () -> (), which wasm-bindgen does not limit itself to.
If I may ask, out of personal interest, what did you intend to use asm.js for? What situation did you find yourself in where wasm is unsupported, yet asm.js was?

If I may ask, out of personal interest, what did you intend to use asm.js for? What situation did you find yourself in where wasm is unsupported, yet asm.js is?

Sure. This was going to be for a small personal project of mine.
(Warzone 2100)[https://github.com/Warzone2100/warzone2100] has recently replaced its deprecated Qt4-based JavaScript engine with QuickJS in 4.x, which supports asm.js. Personally, I'm not a fan of JavaScript as a language, and so I was hoping to use Rust to write a new AI for the game. Obviously an embedded interpreted-only JavaScript engine like this does not support WebAssembly, so you see my use-case.

Sorry, I should've been clearer, by "asm.js support," I was referring to asm.js support, as seperate from "normal" JavaScript support, e.g. an AoT compiler dedicated to asm.js code.
As having used QuickJS before, it doesn't support either.
But, I guess that is one of the key parts of asm.js: compatibility with older engines.

Sorry, I should've been clearer, by "asm.js support," I was referring to asm.js support, as seperate from "normal" JavaScript support, e.g. an AoT compiler dedicated to asm.js code.
As having used QuickJS before, it doesn't support either.
But, I guess that is one of the key parts of asm.js: compatibility with older engines.

Ahh, no I should have been more clear. Yeah QuickJS has no dedicated fast paths for asm.js, but it does execute asm.js correctly, which was something the old engine was incapable of doing.

Do you really need **asm.**js for your use-case though? Since there's no dedicated fast path, you can just as well use wasm2js from the Binaryen toolchain to transpile the resulting Wasm: https://github.com/WebAssembly/binaryen

Had you checked out https://rustwasm.github.io/docs/wasm-bindgen/examples/wasm2js.html?
Are you suggesting that the steps there had failed?

Yes, the steps from https://rustwasm.github.io/docs/wasm-bindgen/examples/wasm2js.html do fail :(

Yes, the steps from https://rustwasm.github.io/docs/wasm-bindgen/examples/wasm2js.html do fail :(

Fail how? Is there an error message?


How to get verbose/debugging output from Pylance?

While using pylance with PyTorch, I've found a lot of methods are missing. For example,
import torch
torch.ones((1,))

gets me a
"ones" is not a known member of module

error from pylance. But when I run pyright on the same file, it reports everything is a-okay:
root@5049e7c0c119:/code# pyright tmp.py
stubPath /code/typings is not a valid directory.
Assuming Python platform Linux
Searching for source files
Found 1 source file
0 errors, 0 warnings
Completed in 0.694sec

This leaves me with a couple of questions:

Why does Pylance not pick up on some torch methods?

I suspect that it's because that method is missing from pytorch's stubs, but I'd like to be able to check that's the case.

Why is Pylance giving me different results to Pyright?

I suspect it's some extra parameter that Pylance is passing to its own instance of Pyright, but again I'd like to be able to verify that's the case.

but really I haven't given you detailed enough information to give me a concrete answer to either of those. I'm running this in a fairly vanilla container, but some random configuration value might be messing with things. So my ultimate question is

How can I get Pylance to emit more verbose/debugging output so that I could figure out the answer to (1, 2) on my own?

Make sure you dont have both extensions installed at the same time.
to enable tracing add this to your settings.json
"python.analysis.logLevel": "Trace",

In this case, the difference you're seeing is because Pyright and Pylance use a different default value for one particular setting: "useLibraryCodeForTypes". Pyright defaults this setting to false, and Pylance defaults it to true. We generally like to keep the settings defaults the same between the two, but there are a couple of exceptions. We should probably document these explicitly. The other setting that differs is "typeCheckingMode". Pyright defaults to "basic" and Pylance defaults to "off".
The justification for these differences is that most Pylance users are not interested in type checking. They are instead interested primarily in language server features like completion suggestions. Users of Pyright — especially the CLI version — are primarily interested in type checking.
The "useLibraryCodeForTypes" setting is disabled in Pyright by default because type checking is dependent on high-quality type stubs. Type information inferred from library code is often incomplete and leads to false positives. The error you're seeing is an example of this.
So if you want to use the CLI version of Pyright alongside the Pylance extension, I recommend configuring them in the same manner, whatever is most appropriate for your use case. You should then get consistent results for both.

Thanks very much for the fast response folks!

Pyright and Pylance use a different default value for one particular setting: "useLibraryCodeForTypes".

Great - running pyright tmp.py --lib replicates the error I'm seeing in pylance.

The other setting that differs is "typeCheckingMode". Pyright defaults to "basic" and Pylance defaults to "off".

And yup, turns out I have type-checking set to basic. I don't remember setting that, which is either because pyright set it when I had it installed previously, or more likely because I've forgotten that I messed with it.

to enable tracing add this to your settings.json
"python.analysis.logLevel": "Trace",

And hooray!
[BG] analyzing: /code/tmp.py (329ms)
[BG] analyzing: /code/tmp.py ...
[BG]   parsing: /code/tmp.py (14ms)
[BG]   parsing: /root/.vscode-server/extensions/ms-python.vscode-pylance-2020.8.1/server/typeshed-fallback/stdlib/2and3/builtins.pyi (194ms)
[BG]   binding: /root/.vscode-server/extensions/ms-python.vscode-pylance-2020.8.1/server/typeshed-fallback/stdlib/2and3/builtins.pyi (30ms)
[BG]   binding: /code/tmp.py (0ms)
[BG]   checking: /code/tmp.py ...
[BG]     parsing: /opt/conda/lib/python3.8/site-packages/torch/__init__.py (91ms)
[BG]     binding: /opt/conda/lib/python3.8/site-packages/torch/__init__.py ...
[BG]       parsing: /opt/conda/lib/python3.8/site-packages/torch/_C/__init__.pyi (16ms)
[BG]       binding: /opt/conda/lib/python3.8/site-packages/torch/_C/__init__.pyi (24ms)
[BG]       parsing: /opt/conda/lib/python3.8/site-packages/torch/functional.py (13ms)
[BG]       binding: /opt/conda/lib/python3.8/site-packages/torch/functional.py (3ms)
[BG]     binding: /opt/conda/lib/python3.8/site-packages/torch/__init__.py (60ms)
[BG]   checking: /code/tmp.py (152ms)
[BG] analyzing: /code/tmp.py (391ms)

This is exactly what I was after, thanks. It looks like it's pulling in torch/_C/__init__.pyi, but omitting torch/_C/_VariableFunctions.pyi, which - looking at the file - is where the stub for torch.ones is.
Curiously, mypy does find this file, along with the definition:
root@5049e7c0c119:/code# mypy tmp.py --verbose 2>&1 | grep Variable
LOG:  Metadata fresh for torch._C._VariableFunctions: file /opt/conda/lib/python3.8/site-packages/torch/_C/_VariableFunctions.pyi

but it looks like the pytorch devs hacked this in with a if False: import.
I think this is now more of a pyright issue than a pylance issue, but do either of you know if there's another setting I can flip to get pylance/pyright to correctly find this _VariableFunctions.pyi file?

Pytorch 1.5 worked fine for me. With Pytorch 1.6, they have done made some modifications that broke things.
I don't understand why they chose to place that import under an if False: condition. Type checkers need to honor provably-false conditions, and Pyright does so in this case. If they want to hide the import at runtime but make it available to type checkers, they should change if False: to if TYPE_CHECKING:. If I make that change in my local copy of torch/__init__.py, the ones symbol (and other symbols declared in torch/_C/_VariableFunctions.pyi) become visible to the type checker.


Can't pickle PySCF object?

I want to save the PySCF variables to the file, but I when I run the code below, I got error and don't know how to solve it.
import pyscf
import pickle
geom = 'Li 0 0 0; H 0 0 1'
basis = 'sto-3g'
mol = pyscf.gto.M(atom = geom, basis = basis)
with open ('filename', 'wb') as f:
	pickle.dump(mol, f)
Error:
Traceback (most recent call last):
  File "pyscf_usage.py", line 12, in <module>
    pickle.dump(mol, f)
TypeError: cannot pickle '_io.TextIOWrapper' object

You can use pyscf.lib.chkfile.save_mol to save mol object to a hdf5 file.

But I also want to save mf and other results.
If I separately save these different type objects, the saved results will become very messy.
I want to save them in a consistent way, can I modified the mol/mf object make it suitable to pickle?

But I also want to save mf and other results. If I separately save these different type objects, the saved results will become very messy. I want to save them in a consistent way, can I modified the mol/mf object make it suitable to pickle?

pyscf.scf.chkfile.dump_scf saves the mol and some scf's arrays into one hdf5 file. This hdf5 file is interoperable so you can dump any other arrays into it with lib.chkfile.save('chkfile', {'array_name': array})

But I also want to save mf and other results. If I separately save these different type objects, the saved results will become very messy. I want to save them in a consistent way, can I modified the mol/mf object make it suitable to pickle?

pyscf.scf.chkfile.dump_scf saves the mol and some scf's arrays into one hdf5 file. This hdf5 file is interoperable so you can dump any other arrays (or dict) into it with lib.chkfile.save('chkfile', 'array_name', array)

Why so inconvenient?

I think that's convenient enough. Usually dump_scf is enough for dumping a scf object. Do you have anything else to be dumped?

I think that's convenient enough. Usually dump_scf is enough for dumping a scf object. Do you have anything else to be dumped?

For a LiH system, I want to calculate FCI results in different bond lengths, for benchmarks. I need to save mol, mf and FCI energy and vectors for different bond lengths.

So you need one call to dump_scf and two calls to lib.chkfile.save. Three lines of code altogether.

So you need one call to dump_scf and two calls to lib.chkfile.save. Three lines of code altogether.

OK, I will try it. Thank you!

But I also want to save mf and other results. If I separately save these different type objects, the saved results will become very messy. I want to save them in a consistent way, can I modified the mol/mf object make it suitable to pickle?

pyscf.scf.chkfile.dump_scf saves the mol and some scf's arrays into one hdf5 file. This hdf5 file is interoperable so you can dump any other arrays (or dict) into it with lib.chkfile.save('chkfile', 'array_name', array)

Why so inconvenient?

pickle serialization is platform dependent, not transferable between different systems. Mole object can be serialized to a  json string mol.dumps(), which can be deserialized to an almost identical Mole object using gto.loads(mol.dumps()). dumps and loads function for other classes were not implemented. They can be added in the future.

But I also want to save mf and other results. If I separately save these different type objects, the saved results will become very messy. I want to save them in a consistent way, can I modified the mol/mf object make it suitable to pickle?

pyscf.scf.chkfile.dump_scf saves the mol and some scf's arrays into one hdf5 file. This hdf5 file is interoperable so you can dump any other arrays (or dict) into it with lib.chkfile.save('chkfile', 'array_name', array)

Why so inconvenient?

pickle serialization is platform dependent, not transferable between different systems. Mole object can be serialized to a json string mol.dumps(), which can be deserialized to an almost identical Mole object using gto.loads(mol.dumps()). dumps and loads function for other classes were not implemented. They can be added in the future.

OK, thank you!

This problem can be fixed by defining __getstate__ and __setstate__ for Mole objects, preferably using the existing loads and dumps methods. I will make a PR with this solution for everyone to consider.


Frontend not showing error messages

I am getting this kind of error on console constantly.

There is the problem. The alert is not returning the proper error-message from Routes.py /add_references-method.
Tried fixing this myself but did not succeed. @ismomehdi Hopefully can fix this.

Did some debugging and the response handling in referenceService.js should look like this:
if (!response.ok) {
            const responseData = await response.json();
            throw new Error(JSON.stringify(responseData.message));
        }

Thanks @Wincewind !


Add possibility to set ApplicationIntent=readonly for SqlServer

If you use a Microsoft SQL Server availability group you can route readonly connections to the slave replicas. This can be done if you add ApplicationIntent=readonly to the database connection.

Can you please delete this PR. I accidently did a PR based on master not 5.2


GH-1054 Added support for display extra binding information

Added support for displaying

state
pausable
name/group
destination info
extended binding info
etc

Resolves #1054

Codecov Report

Merging #1266 into master will decrease coverage by 0.38%.
The diff coverage is 4.16%.

@@             Coverage Diff              @@
##             master    #1266      +/-   ##
============================================
- Coverage     62.36%   61.98%   -0.39%
+ Complexity      823      820       -3
============================================
  Files           133      133
  Lines          3877     3898      +21
  Branches        551      554       +3
============================================
- Hits           2418     2416       -2
- Misses         1185     1207      +22
- Partials        274      275       +1

Impacted Files
Coverage Δ
Complexity Δ

...g/springframework/cloud/stream/binder/Binding.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...gframework/cloud/stream/binder/DefaultBinding.java
26.66% <0%> (-16.2%)
4 <0> (-3)

...ud/stream/binder/AbstractMessageChannelBinder.java
45.9% <12.5%> (-0.98%)
22 <0> (ø)

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 368e761...a6fb3c3. Read the comment docs.


for some reason the second create() fails (using model with slug PK)

Opened to expose issue (#111)
Not sure what's going on here; the create() call with a model that uses a slug for a primary key has issues with the dictionary...

It's clearly a bug. Have a proposed resolution?

Not as yet... sorry, I haven't had time to look at it. A workaround is to create the object with an empty data dict and then add the values per key (i.e. x.data['mykey'] = someval), but isn't ideal

@stringfellow should be fixed


Consistently getting 7.5fps using examples

Hey, sorry for the issue.
I seem to be consistently getting about 7.5FPS running any capture examples (stream_forward_mmap, stream_capture_mmap). This happens even when varying the format that I'm using for the device.
I'm wondering if there could be something I'm doing wrong? the format I'm using doesn't even seem to support 7.5 FPS for any of it's formats... which is strange.
Let me know if there's any advice you can give me.

What kind of camera are you using?
Please add the active format and parameters you are using. Also, are you running a release or debug mode build?

Hey, I'm using a pretty standard laptop webcam (goes up to 720p, and with MJPEG it'll do 30FPS, slightly lower on YUYV).
I've tried both release and debug.
I messed around with a few different formats (and shrinking it right down to eliminate a slow processor or something as an issue).
Here is some output from a couple of frames of stream_capture_mmap:
Using device: /dev/video0

Active format:
width          : 320
height         : 180
fourcc         : YUYV
field          : progressive
stride         : 640
size           : 115200
colorspace     : sRGB
quantization   : default
transfer       : default transfer function

Active parameters:
capabilities : TIME_PER_FRAME
modes        : (empty)
interval     : 1/30 [s]

Buffer
  sequence  : 2
  timestamp : 1665.073533 [s]
  flags     : MAPPED | TIMESTAMP_MONOTONIC | TSTAMP_SRC_SOE
  length    : 115200
Buffer
  sequence  : 3
  timestamp : 1665.205535 [s]
  flags     : MAPPED | TIMESTAMP_MONOTONIC | TSTAMP_SRC_SOE
  length    : 115200

FPS: 7.462410290914528
MB/s: 0.8201908832008339

Also, for info here is the formats supported from v4l2-ctl:
v4l2-ctl --list-formats-ext                                                                     INT ↵  default 🐍  2.6.3 
ioctl: VIDIOC_ENUM_FMT
	Type: Video Capture

	[0]: 'YUYV' (YUYV 4:2:2)
		Size: Discrete 640x480
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 960x540
			Interval: Discrete 0.067s (15.000 fps)
		Size: Discrete 848x480
			Interval: Discrete 0.050s (20.000 fps)
		Size: Discrete 640x360
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 424x240
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 352x288
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 320x240
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 320x180
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 1280x720
			Interval: Discrete 0.100s (10.000 fps)
	[1]: 'MJPG' (Motion-JPEG, compressed)
		Size: Discrete 640x480
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 960x540
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 848x480
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 640x360
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 424x240
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 352x288
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 320x240
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 320x180
			Interval: Discrete 0.033s (30.000 fps)
		Size: Discrete 1280x720
			Interval: Discrete 0.033s (30.000 fps)

Also, one thing to note, it seems to be consistently very close to 7.5 FPS. it's not a rough number, and it's never been anything else during my testing. (I can't fully remember what it was when I was testing the other day, but wouldn't be surprised if it was the same).

I'm afraid I can't reproduce this using my Logitech C922 Pro Stream - I'm getting consistent 30 FPS in MJPG or 20-30 in YUYV. What kind of frame times are you getting when using v4l2-ctl for mmap streaming?

Oh wow... this is strange...:
time v4l2-ctl --device /dev/video0 --stream-mmap --stream-to=frame.raw --stream-count=100
<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<< 30.15 fps, dropped buffers: 1
<<<<<<<<<<<<<<<<<<<<<<<<<<<<<< 30.14 fps
<<<<<<<<<<<<<<<<<< 26.12 fps
<<<<<<<<<<< 22.37 fps
<<<<<<<< 19.25 fps

I then Ctrl+C-ed it, and did it again afterwards:
 time v4l2-ctl --device /dev/video0 --stream-mmap --stream-to=frame.raw --stream-count=1000
 <<<<<<<< 7.52 fps, dropped buffers: 1
<<<<<<<< 7.50 fps
<<<<<<< 7.51 fps
<<<<<<< 7.50 fps
<<<<<<<< 7.50 fps
<<<<<<< 7.50 fps

I wonder what could be causing this...
Thanks for your help. I guess I'll close this issue, as it doesn't seem to be to do with this library.
If you have any ideas for places I could look do let me know, but I'll dig into this by myself.
(and thanks for the tip, didn't realise you could stream directly from v4l2-ctl).

oh wow... I figured it out, that was quick https://stackoverflow.com/questions/46228379/v4l2-framerate-drops
moving to a brighter room sorts it out. Sorry for the distraction!
time v4l2-ctl --device /dev/video0 --stream-mmap --stream-to=frame.raw --stream-count=1000                                                                                 INT ↵  5s    default 🐍  2.6.3 
<<<<<<<< 7.52 fps, dropped buffers: 1
<<<<<<<<<<<<<< 10.83 fps
<<<<<<<<<<<<<<<<<<<<<<<<<<<<<< 17.17 fps
<<<<<<<<<<<<<<<<<<<<<<<<<<<<<< 20.36 fps
<<<<<<<<<<<<<<<<<<<<<<<<<<<<<< 22.29 fps
<<<<<<<<<<<<<<<<<<<<<<<<<<<<<< 23.59 fps
<<<<<<<<<<<<<<<<<<<<<<<<<<<<<< 24.50 fps
<<<<<<<<<<<<<<<<<<<<<<<<<<<<<< 25.20 fps
<<<<<<<<<<<<<<<<<<<<<<<<<<<<<< 25.73 fps
<<<<<<<<<<<<<<<<<<<<<<<<<<<<<< 30.08 fps

that explains why the framerate seems to drop when I code at night :smile:

Oh sorry, I could have saved you some time there :D
Most, if not all, laptop cameras are really garbage products and way too sensitive to changing light conditions.


Domain International Extension

Issue Summary
Hi, I have issue sending email from domain with international extension, I don't find one solution for this.
Thanks
Code Snippet
<EMAIL_ADDRESS>, 'Name');
<EMAIL_ADDRESS>, 'Name');
            $mail->addTo('me@gmail.com);

Exception/Log
"$emailAddress" must be a valid email address. Got<EMAIL_ADDRESS>
Technical details:

sendgrid-php version:  7.9.1
php version: 7.3

Hello @jarscr,
Thanks for submitting a GitHub issue! We are very sorry that you are running into this problem. In order to better serve you, as this does not present itself as a library specific issue, we would like to ask that you reach out to our support team at https://support.sendgrid.com.
Thank you!
SendGrid DX Team


Pushed resources from production websites cause false h2spec failures

Regarding ... https://twitter.com/tunetheweb/status/988522917532184580

To be honest I didn’t install it and just ran it against the webtide website (Jetty 9.4.9 according to server header): 81/146 (55%)

The overwhelming majority of those failures are because hitting https://webtide.com/ will result in pushed resources that h2spec doesn't like seeing.

Perhaps allow for path to be hit on server as well?
So you can avoid requesting resources that will result in PUSH_PROMISE frames?
eg:
$  ./h2spec --insecure --strict --host=webtide.com --port=443 --path=/favicon.ico --tls --verbose

Thank you for the report.
It seems good to add the option to specify path. I'll add the option.

Fixed in v2.2.0.


DEVPROD-596 PoC: jq in Parsley

DEVPROD-596
Description
Logs in lines-of-JSON format are commonplace, both at MongoDB and elsewhere. jq is a popular tool for manipulating such data. This changeset offers a proof-of-concept of integrating jq into Parsley.
Screenshots

Testing
I manually tested with a small JSON document. I verified a few simple transforms, and I verified that this behaves sensibly when the given filter is invalid.
I verified that Parsley’s existing (regexp-based) filters run after JQ’s filtering.
Evergreen PR
n/a
Notes
jq provenance
The JQ version that this includes is built from my update to jq-web. jq/index.js has a special export default jq; added at the end so that it can be imported as an ES6 module. I would expect a production deployment to use an npm-provided JQ. (jq-web’s maintainer doesn’t seem very interested in the project; I may offer to take it over. A significant part of whipping this up involved just getting it working with modern Emscripten!)
Internal notes
This is a simple, proof-of-concept implementation that makes no claim to production-readiness.
Right now Parsley stores the jq-processed logs in memory. It’s probably more ideal to run jq in the renderer instead so that large files don’t cause trouble.
UI
It would be good, when/if we put this into production, to have a link handy that either a) goes to jq’s official docs, or b) goes to a “cheat sheet” of handy examples (which itself should probably link to jq’s docs).

Thanks for opening this Felipe this is really cool! We will close this PR for now as we plan on building out more official plugin support in the future which should enable this type of functionality. I attached a link to this pr in the related ticket so that we can reference it in the future. Great work!