Crash in SavedMatrix#finished: IllegalStateException: step.testExecutionStep.testTiming must not be null

We encountered this crash using Flank version v4.2.0, while running 75 shards.
It looks like there's a null check for step.testExecutionStep, but in this case step.testExecutionStep.testTiming was null.
10:42:50   4m 48s matrix-285i5yv89ulkl RUNNING
10:43:47   5m 43s matrix-285i5yv89ulkl Non-retryable error occurred during results processing.
10:52:53 Request failed, retrying 1x java.net.SocketTimeoutException: Read timed out
11:02:45   24m 38s matrix-285i5yv89ulkl FINISHED
11:02:45 Exception in thread "main" picocli.CommandLine$ExecutionException: Error while running command (ftl.cli.firebase.test.android.AndroidRunCommand@6d367020): java.lang.IllegalStateException: step.testExecutionStep.testTiming must not be null
11:02:45     at picocli.CommandLine.execute(CommandLine.java:1168)
11:02:45     at picocli.CommandLine.access$800(CommandLine.java:141)
11:02:45     at picocli.CommandLine$RunLast.handle(CommandLine.java:1367)
11:02:45     at picocli.CommandLine$RunLast.handle(CommandLine.java:1335)
11:02:45     at picocli.CommandLine$AbstractParseResultHandler.handleParseResult(CommandLine.java:1243)
11:02:45     at picocli.CommandLine.parseWithHandlers(CommandLine.java:1526)
11:02:45     at picocli.CommandLine.run(CommandLine.java:1974)
11:02:45     at picocli.CommandLine.run(CommandLine.java:1920)
11:02:45     at ftl.Main$Companion.main(Main.kt:48)
11:02:45     at ftl.Main.main(Main.kt)
11:02:45 Caused by: java.lang.IllegalStateException: step.testExecutionStep.testTiming must not be null
11:02:45     at ftl.json.SavedMatrix.finished(SavedMatrix.kt:81)
11:02:45     at ftl.json.SavedMatrix.update(SavedMatrix.kt:57)
11:02:45     at ftl.run.TestRunner.pollMatrices(TestRunner.kt:249)
11:02:45     at ftl.run.TestRunner.newRun(TestRunner.kt:353)
11:02:45     at ftl.run.TestRunner$newRun$1.invokeSuspend(TestRunner.kt)

Thanks for reporting this bug. I'll push a fix when I'm back next week.

One day Google will publish a client library with nullability annotations and we won't have to discover the edge cases by trial and error. 🙂
I merged the fix to master. The snapshot release will be automatically published in a few minutes.

🤞 someday! Thanks for fixing.


Updated to ReactiveCocoa v4.0-alpha.4

Fab, looks great! Want to add an entry to the changelog?

Sure, will do after I look into this build issue.

@ashfurrow Can you trunk push this updated podspec for the latest ReactiveCocoa alpha? It's needed in order to set the version in the Podfile for the Demo project in Moya.
See https://github.com/mpurland/Specs/blob/rac-update/Specs/ReactiveCocoa/4.0.4-alpha-4/ReactiveCocoa.podspec.json

For reference: https://github.com/ReactiveCocoa/ReactiveCocoa/issues/2590

Sure, can do. Where did you get the original to modify? We're using https://github.com/mhuusko5/Podspecs as a source for Ruby-based pod specs. /cc @mhuusko5
Also, I'm afraid to say that the ReactiveCocoa developers are unlikely to do anything but close your issue. They've declined repeated offers from myself and others to help maintain the podspec in their repo, and choose to be poor sports instead.

I used the podspec for 4.0.3-alpha-3 as a base since that's the last one you added. 4.0.4-alpha-1 is a much older version actually...

Is it because they're pushing for carthage instead?

Makes sense – the Ruby ones are a bit easier to reason about.
I'm not sure why they insist on declining, but it's been that way for as long as I can remember – far before Carthage.

Not sure what the next step here is. The carthage build works, but the Demo project will not build because it depends on CocoaPods and the Moya podspec which can't reference a branch or commit.

The next step is we need to push a podspec to trunk. Could you update the podspec on https://github.com/mhuusko5/Podspecs and send him a PR? Then we can push to trunk, then we can merge this :tada:

Done https://github.com/mhuusko5/Podspecs/pull/1

Pushed to trunk

Re-pushed commit, now let's see if travis builds it...

Current coverage is 84.84%

Merging #329 into master will not affect coverage as of 0a4461e

@@            master    #329   diff @@
======================================
  Files           13      13
  Stmts          640     640
  Branches         0       0
  Methods          0
======================================
  Hit            543     543
  Partial          0       0
  Missed          97      97

Review entire Coverage Diff as of 0a4461e

Uncovered Suggestions

+2.50% via Source/Result.swift#53...68
+1.72% via Source/Result.swift#38...48
+1.25% via ...ource/Response.swift#75...82
See 7 more...

Powered by Codecov. Updated on successful CI builds.

@ashfurrow This should be good now?

Looks good to me, if someone else wants to review + merge after the changelog is updated, that'd be :100:
Thanks again @mpurland – great work!

Ready to :boat:

Thanks!

I've added you as a contributor to Moya, so you've got full push access :tada: You can read more about that here: https://github.com/Moya/contributors
I've also released 5.1.0 of Moya so you can use your new commits from Trunk! Thanks again!

@ashfurrow Thanks. Glad to help out now and in the future.

@ashfurrow Any reason why 4.0.2 is the Latest release and every release since is just a pre-release?
Also, any thoughts on submitting .framework.zip to releases for carthage compatibility so that Moya doesn't need to build (there are a lot of dependencies)?

@mpurland of ReactiveCocoa? Not sure.
I have no feelings towards Carthage support. If that's a thing we can do (easily) to help them, sure sounds good :+1: My only requirement is that it gets automated: https://github.com/Moya/Moya/blob/e069bc22e28b2d69cfcfbbae3cb15304d112fbd6/Rakefile#L27-L57

the problem that I've encountered with the prebuilt binaries from Carthage is that when a new version of swift/Xcode comes out it has to be recompiled right now. I saw myself using the "-no-use-binaries" option a lot to circumvent that since even though the source didnt have errors it just doesn't work and has to be rebuilt.
Right now I would advise against distributing prebuilt binaries until that problem with swift is solved.

I've run into that issue as well. I hope it will be addressed soon.

@ashfurrow The above issue about tags and releases was encountered in #332.
It looks like the Rakefile will need to be adjusted to publish releases vs. just tags if you would still only want to cut releases that way.

Oh I see, so specifying GitHub instead of Git in Carthage means very different things, one uses GitHub-specific thingies and the other just uses tags? I did not know that – seems silly. Anyway to create a "release" from the command line?

There is a dependency for rakefiles that I've seen referenced that can be used.

I think this is what I saw earlier: https://github.com/mpalmer/github-release


Property name should not be color32x32

Type of issue
Other (describe below)
Feedback
This will create an issue if we decide to change the resolution of icon.
Page URL
https://review.learn.microsoft.com/en-us/microsoftteams/platform/resources/schema/manifest-schema-dev-preview?branch=pr-en-us-10437
Content source URL
https://github.com/MicrosoftDocs/msteams-docs/blob/main/msteams-platform/resources/schema/manifest-schema-dev-preview.md
Author
@surbhigupta
Document Id
b2536bbc-eb27-547d-d7f5-bab3ded3bedf

Hello @surbhigupta12
Could you please help here as per document suggested?

This is not a doc bug. Need to raise it to the manifest team.


Flesh out framework for RAII-able ICU objects

I can make this a code comment if you want but I will just put it here for now.
In general there is only one new class, ScopedICUObject<T, Function>, which is only meant to be used in one of its typedef'd forms, which all take the naming form of Scoped%ICUObjectName%, like ScopedUCollator. Its fairly straightforward, as its constructor takes a single argument of type T and has an operator T such that the only update it requires compared to not using any wrapper is changing ICUObject a = open_icuobject() to ScopedICUObject a(open_icuobject()). There are some template/macro/inline shenanigans because C++ is awful, but its API should be mostly invisible.
FinalizableICUObject used to be used directly and required the relevant close function to be passed. I updated it to use the same template/macro/inline shenanigans that ScopedICUObject uses, such that it is now designed to be used in one of the typedef'd versions like FinalizableUNumberFormat

And I went with the template/macro/inline + typedef shenanigans because I assumed that there was more runtime overhead of duplicating the classes, but I suppose I didn't test anything ahead of time to confirm that assumption.


Possible to have a button enable drawing a polygon on the map?

I'm working on adding a toolbar that has controls for enabling drawing a polygon on the map. I can't see an example of this in the RLayers examples though. I tried using a state variable (drawActive) that gets set to true when the user clicks the button, and then passing this as the condition for drawing being enabled. This results in a TS error.
Before I dive further into this, I want to make sure that what I'm trying to accomplish is possible with RLayers? Thanks!

Figured it out! Passing in the boolean as the draw condition works, but the TS error
"Type 'boolean' is not assignable to type '(e: MapBrowserEvent) => boolean'.ts(2322)" still persists.

You can do it yourself or you can use an interaction, here is the example: https://mmomtchev.github.io/rlayers/#/draw

Passing the condition like so:
condition={() => {               return drawActive;             }}
resolves the TS error. However, passing the boolean directly as the condition also works.
Having to return the boolean from this function is not ideal.

The condition argument is the event, you have to use a closure if you want to pass an external variable.

Thanks @mmomtchev! It is in working order now, so will mark this as closed. Cheers


Specify shortcode convention

First of all, great addon!
I was wondering which convention you follow for the codes. I know at least three of them: github, slack, emojipedia. This would be very useful information to be added to the readme.
I am also asking because I wanted to add some missing emojis, such as the shrugging ones #18 and there the codes are all distinct.

You can see the whole List here: https://github.com/phibr0/obsidian-emoji-shortcodes/blob/master/src/emojiList.ts
I really don't know which convention it is, I grabbed the List of of here: https://gist.githubusercontent.com/housamz/67087a81eaf78837a420fdef4accf263/raw/8c6a8bd28af5431f8ad394d433bd2ae201ec4bf9/emojis.json

If you can find a better, more complete List that would be great.

Hmm, we could web scrape emojipedia in the worst case, right?
I might try out https://github.com/bcongdon/python-emojipedia today, this seems to already provide an interface to emojipedia.


token-cli: Add PermanentDelegate support

Problem
https://github.com/solana-labs/solana-program-library/pull/3725 added the PermanentDelegate extension and support in token-client, but we need it in the CLI.
Solution
Add it!

I would like to work on it , just to be sure these are the changes i am thinking for adding it's support in cli , please add if there are some other changes needed  :
1). adding a enable-permanent-delegate flag to make mint-authority also permanent-delegate in create-token
2). return permanent-delegate in Authorize for previous authority
3). tests

Yep, that's all perfect!


System.ArgumentNullException: Value cannot be null. (Parameter 'clientId') in bot V 3.30.0

Version
Bot Builder Version : 3.30.0
Describe the bug
I have .net core 3.1 bot application , I am getting Value cannot be null. (Parameter 'clientId') in bot v3.30.0 version even I have configured Microsoft App Id and Password in appsettings.json
To Reproduce
Steps to reproduce the behavior:
using (var scope = DialogModule.BeginLifetimeScope(Conversation.Container, activity))
{
var dialog = scope.Resolve<IDialog>();
await Conversation.SendAsync(activity, () => dialog);
this.logger.LogInformation("Sent conversation reply");
}
Expected behavior
It should load the desired dialog.
Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Add any other context about the problem here.
[bug]
[dotnetcore]

@dhanutpk - BotBuilder V3 is not supported, please try with BotBuilder V4.


v3.5.9 UI: Clicking pod logs button crashes UI

Pre-requisites

[X] I have double-checked my configuration
[X] I have tested with the :latest image tag (i.e. quay.io/argoproj/workflow-controller:latest) and can confirm the issue still exists on :latest. If not, I have explained why, in detail, in my description below.
[X] I have searched existing issues and could not find a match for this bug
[ ] I'd like to contribute the fix myself (see contributing guide)

What happened? What did you expect to happen?

Install 3.5.9
Run a workflow
Click a node on the running workflow
Click 'logs' in the 'summary' tab
crash:

There are no relevant server logs.
Workaround:
Roll back to 3.5.8
Edit:  I would encourage others to use the thumb emoji rather than an unnecessary "we have this issue too"-style comment.
Version(s)
v3.5.9
Paste a minimal workflow that reproduces the issue. We must be able to run the workflow; don't enter a workflows that uses private images.
Any workflow.

Logs from the workflow controller
Nothing useful

Logs from in your workflow's wait container
not relevant

Verified, we have that, too.

Just upgraded from v3.5.8 to v3.5.9 and we have the same issue.

Huh, I'm pretty surprised this didn't get caught by static analysis by the type-checker during build or the linter. The release-3.5 branch never brought in linter migrations (#12163), but this would still be surprising that an older linter wouldn't detect it 🤔 My type-checker in my editor detects this instantly.
This appears to be due to out-of-order cherry-picking; #12973 was cherry-picked before #12964. In fact, it was cherry-picked into an older patch (v3.5.7) because it was a fix, while #12973 is a refactor that was cherry-picked only for 3.5.9 in order to resolve merge conflicts for #13039 etc.

I'm still not sure how the build's ts-loader doesn't catch this on type-check though. That's odd 🤔

I suspected that the transpileOnly flag doesn't actually accept false and is just "supplied or not". Removing the flag indeed causes type errors to pop up: this one, but also a few other internal ones (that are purely type-related and do not affect the resulting JS), and the node_modules ones due to https://github.com/ramda/types/issues/64. As a result, I can't quite enable that in the release-3.5 branch to prevent this in the future. I think it might be worthwhile in this case to backport UI build/toolchain changes from main, but will confer with other contributors as it outside our normal process.
In the interim, I'll push up a small fix for this and should have 3.5.10 out tonight as this is affecting lots of people (I guess that it's good that the UI is popular at least!)

This has been fixed in 72d0d22e6254c2871f7f4f3798a362094409064f and included in v3.5.10
(commit ref is bc it only exists on the release-3.5 branch since this bug does not exist on main as it was due to out of order cherry-picking per above)

Thanks @agilgur5
that was a quick fix 💯

Indded. Thank you!


More room types (e.g corridor)

Hello, currently with procthor, only 4 room types seem to be supported (bedroom, bathroom, kitchen and living room). Is there any way to build a scene with other room types (like a corridor to connect some rooms) to have a layout closer to reality?
Thanks
Matthieu

I came here with a similar question; the website and paper emphasize that ProcTHOR is customizable. The paper states:

PROCTHOR supports many room, asset, material, and lighting specifications. With a few simple lines of specification, one can easily generate customized environments of interest. Fig. 9 shows examples of such varied scenes (classroom, library, and office).

I would like to see the lines of specification that generated these examples, which would really help me to build my own scenes.

Same question.


adding support for python 2.6

python 2.6 is still heavily used thanks to redhat/centos distributions so supporting it is useful.
this PR makes two minor changes to add the support:

replacing dict comprehensions
replacing relative imports

will resubmit


Not issue - Dropping Columns and Table View

Hi. This isn't an issue. I love how convenient this widget is and how much it had helped me.
This is just a suggestion not an issue :

Can we add a direct drop option in it. That way the flow would be smoother.
In the option of bulk dropping the data, could we add in a full view of the data frame in table form atleast, we can have a quick glance of the data column before dropping ?

Thank you very much.
Best regards,
Jee

hi Jee,
Thanks for commenting. I think by drop u mean deleting a column. let me know if my understanding is right or if you mean row deletion it's already available in the very first view itself. will include yours and other's recommendations in the next release.


Screenshot in Heroes example page needs to be updated

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
The screenshot used in Examples > Heroes shows an old version of the documentation.
The corresponding image is https://github.com/twbs/bootstrap/blob/main/site/content/docs/5.2/examples/heroes/bootstrap-docs.png
I don't do it myself because I can't reproduce the exact same "scene" on my computer.
@mdo this one is maybe for you OR could you give me the details of your "configuration" to make those screenshots and I can do it myself :)
Note: The image will need to be compressed as well.
Reduced test cases
Examples > Heroes
What operating system(s) are you seeing the problem on?
N/A
What browser(s) are you seeing the problem on?
N/A
What version of Bootstrap are you using?
v5.2.2

I can attach current screenshots.

https://github.com/twbs/bootstrap/blob/2a2c4abaf4791e00274ba78cbb9fc65674ab339f/site/content/docs/5.3/examples/heroes/bootstrap-docs.png is the file to replace. Create a PR instead of attached

I'll handle updating the screenshots once we're done with modifications to them before we go stable on the next release :).


make data doesn't work

I get the following error when running make data (I'm running remotely so had to use a screencap to get the error message, sorry for the poor quality):

The error looks like you're calling tar with the -z option on a file that is not zipped. The -z option with tar should only be used when the file has been compressed with gzip (these files usually end with .tgz or .gz)
Please fix and push the changes (see the piazza announcement) or affix the "wontfix" label if you want us to try to proceed manually.

Hi Ross,
We edited our Makefile  to correct the -z flag, however now Travis isn't passing for both versions of Python. Here's what it shows:

And here are the only changes we made:

Do you know what's going on? If it's some dependency issue with libraries, how can we get Travis to install them? Does this mean we have to tamper with the travis.yml file?

Hmm - I'm getting errors on apt-get update myself : https://travis-ci.org/matthew-brett/travis-wheel-builder/jobs/97180487
I think this might be a temporary travis problem.

For now, try adding sudo apt-get update before your line starting sudo apt-get install in the .travis.yml file.

Try testing locally: re-clone your project in a fresh directory and run the commands in your README in the order you specify (preferrably on a linux machine so mac compatibility issues will be caught). If everything runs without errors, notify us and I will pull and re-run the project.

@rossbar @matthew-brett @jarrodmillman
Hi all,
We ran everything locally and it was reproducible. However, will you be running it remotely on a Linux box? I tried simulating that by running everything on the SCF's Linux cluster and here's what I got:

One possible solution was proposed here at https://github.com/ioam/holoviews/issues/68 by adding this to the code:
from matplotlib import pyplot as plt
plt.switch_backend('agg')

Would that interfere with anyone running it locally, or with you running it remotely?
Thanks,
Zeta

I am running remotely, but that shouldn't be a problem. The error you got above is because your code is trying to generate graphics, but there is no hardware to display graphics on so the renderer is getting confused.
You can fix this with the -X command to ssh, as long as the remote server is running an X server (this is true in my case, I'm not sure about the SCF though).
Thanks for taking a look, I will close the issue once the analysis completes.

So I ssh'ed with the -X flag, but it still got the same error. So I guess SCF isn't running an X server?
I also tried the trick mentioned in that link, and it worked.
Please let me know if you get the error, and I'll edit all the scripts to follow the link's trick.
Lastly, we just pushed our final changes to make everything reproducible. Please pull now and try out the roadmap.
Thanks!

Your analysis ran to completion without any errors. I was on commit 00d224c (I was already running before your last comment) so I don't have those final changes you mention in the previous comment. If you'd like me to re-run everything with the latest commit, let me know: if it's not going to affect your results or the paper then I don't think it's necessary. It's up to you though.

Hi Ross,
We would like to rerun it with the latest commit since it fixes something in the reproducibility of the report.
Thanks!

Okay, will do. Thanks for taking a look at this.


Autorun

How to add the component to autorun? In /etc/init.d, for example

I use the following init script for OpenWrt:
https://github.com/openlumi/openwrt-lumi-packages/blob/main/apps/ble2mqtt/files/ble2mqtt.init


Do not fail upgrade path test if there is no version available to upgrade from

Fixes #5

Made a test run here, seems to be working nicely.


Release noetic / foxy

Noetic add thereturn true at end of updateFootprint

Hi, could we get a Noetic release that includes the PCL removal?

Hi, could we get a Noetic release that includes the PCL removal?

Done - both have been released

Done - both have been released

So fast! Thank Steve! Appreciate it!

So fast! Thank Steve! Appreciate it!


Error when trying to build micro-ROS/micro_ros_zephyr_module on top of Zephyr RTOS docker container (clean environment)

Environment:

Hardware description: the problem is in the micro_ros_zephyr_module installation step. No hardware used at this step yet.
RTOS: Zephyr
Installation type: trying to install micro_ros_zephyr_module on top of the docker container: https://github.com/zephyrproject-rtos/docker-image:main
Version or commit hash: cloned latest repo for the module: git clone https://github.com/micro-ROS/micro_ros_zephyr_module

Steps to reproduce the issue

docker run -ti --name zephyr_main -v .:/workdir -p 5900:5900 docker.io/zephyrprojectrtos/zephyr-build:main
then in the container:
west init
git clone https://github.com/micro-ROS/micro_ros_zephyr_module
pip3 install catkin_pkg lark-parser empy colcon-common-extensions
west build -b arduino_due micro_ros_zephyr_module

Expected behavior
module build without errors
Actual behavior
-- west build: generating a build system
Loading Zephyr default modules (Zephyr base).
-- Application: /workdir/micro_ros_zephyr_module
-- CMake version: 3.22.1
-- Found Python3: /usr/bin/python3 (found suitable version "3.10.12", minimum required is "3.8") found components: Interpreter
-- Cache files will be written to: /home/user/.cache/zephyr
-- Zephyr version: 3.7.0-rc1 (/workdir/zephyr)
-- Found west (found suitable version "1.2.0", minimum required is "0.14.0")
-- Board: arduino_due, qualifiers: sam3x8e
-- Found host-tools: zephyr 0.16.8 (/opt/toolchains/zephyr-sdk-0.16.8)
-- Found toolchain: zephyr 0.16.8 (/opt/toolchains/zephyr-sdk-0.16.8)
-- Found Dtc: /opt/toolchains/zephyr-sdk-0.16.8/sysroots/x86_64-pokysdk-linux/usr/bin/dtc (found suitable version "1.6.0", minimum required is "1.4.6")
-- Found BOARD.dts: /workdir/zephyr/boards/arduino/due/arduino_due.dts
-- Found devicetree overlay: /workdir/micro_ros_zephyr_module/app.overlay
devicetree error: /workdir/micro_ros_zephyr_module/app.overlay:2 (column 14): parse error: undefined node label 'usbotg_fs'
-- In: /workdir/build/zephyr, command: /usr/bin/python3;/workdir/zephyr/scripts/dts/gen_defines.py;--dts;/workdir/build/zephyr/zephyr.dts.pre;--dtc-flags;'';--bindings-dirs;/workdir/zephyr/dts/bindings;--header-out;/workdir/build/zephyr/include/generated/zephyr/devicetree_generated.h.new;--dts-out;/workdir/build/zephyr/zephyr.dts.new;--edt-pickle-out;/workdir/build/zephyr/edt.pickle;--vendor-prefixes;/workdir/zephyr/dts/bindings/vendor-prefixes.txt
CMake Error at /workdir/zephyr/cmake/modules/dts.cmake:296 (message):
gen_defines.py failed with return code: 1
Call Stack (most recent call first):
/workdir/zephyr/cmake/modules/zephyr_default.cmake:132 (include)
/workdir/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:66 (include)
/workdir/zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:92 (include_boilerplate)
CMakeLists.txt:7 (find_package)
-- Configuring incomplete, errors occurred!
FATAL ERROR: command exited with status 1: /usr/bin/cmake -DWEST_PYTHON=/usr/bin/python3 -B/workdir/build -GNinja -DBOARD=arduino_due -S/workdir/micro_ros_zephyr_module

Can I somehow disable USB OTG if it is not present in the board implementation for Zephyr?


Universal Library fails to execute spec target

Undefined symbols for architecture i386:
  "std::terminate()", referenced from:
      -[CDRSpy asOriginalObject:] in Cedar-iOS(CDRSpy.o)
  "___gxx_personality_v0", referenced from:
      Dwarf Exception Unwind Info (__eh_frame) in Cedar-iOS(CDRSpy.o)
ld: symbol(s) not found for architecture i386
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Commit b15ef8c5f28e985d6271bdd7b958aa1585394389 works when building and integrating the universal library.
Commit a1ae0d83da6a8f05cca34c3b2e92b1ccbd6170c7 produces a library that generates the above error message.

what's your os, xcode, and compiler version?

OS 10.7.3
Xcode 4.2.1
Xcodebuild 4.2.1

There might be some other version information that you want that I don't know to include. I did notice that the the CDRSpy is a (.mm) file which may be the problem for me.

Are you linking against c++ standard library? (-lstdc++ in other linker options)

Thanks, that got the project to launch.
https://github.com/pivotal/cedar/pull/29


Token frontend error on Proposal Details for a new freeform proposal

Instead of rendering the details, it renders 'Something went wrong' and in the console it says 'Unknown proposal type'

This has been fixed


does this support perpetual contracts?

Hi, does this library support perpetual contracts live trading?

Hello, can you provide live data support for continuous futures transactions?


bump QGIS to 3_34_x

important fix https://github.com/qgis/QGIS/pull/55914

fixed https://github.com/MerginMaps/mobile-sdk/commit/9fdbb2a49f61ec734cfd7b4f34ceeee1877981fc


Replace synchronized with Java Locks on the allocator (Fixes #12621)

Motivation:
JDK 19 Virtual Thread preview is not compatible with synchronized blocks and Netty pooled allocator code is making uses of it to protect arenas/chunk/subpages accesses
Modification:
Replacing existing synchronized usages to make the existing core more Virtual threads friendly: no evident effects should exist on performance, but an increased allocation rate under contention.
Result:
Virtual Threads compliant pooled allocator

Please @anavarro takes a look as well  (@joeyleeeeeee97 too)

@chrisvest I see that we have an
private final class PoolThreadLocalCache extends FastThreadLocal<PoolThreadCache> {
        // ...

        @Override
        protected synchronized PoolThreadCache initialValue() {
        ```
I didn't bother myself to replace it, because pool thread cache shouldn't be used for virtual threads, but...
why `synhronized` tere?? isn't supposed to be accessed by a single thread already?

The additional benefit of this is that we can decide to use other lock types too (read/write and/or optimistic ones - to help with metrics gethering, maybe).
Downsides are:

garbage under contention
more verbose (!!!!) - but I could have use an AutoCloseable abstraction here
won't benefit by some JIT optimizations (not a biggie)

synchronized

Honestly I think there is no need.

@franz1981 can you also do a PR for main ?

@chrisvest I see that we have an
private final class PoolThreadLocalCache extends FastThreadLocal<PoolThreadCache> {
        // ...

        @Override
        protected synchronized PoolThreadCache initialValue() {

I didn't bother myself to replace it, because pool thread cache shouldn't be used for virtual threads, but... why synhronized there?? isn't supposed to be accessed by a single thread already?

I don't know. The initialValue() method is only called by the thread owning the initial value being created there, so it should not be necessary to synchronise that method. The method was made synchronised in b88a980482d2aef1163a9abe759f68e685550052.

@chrisvest @franz1981 @normanmaurer I'm seeing BlockHound complaining now for blocking calls
reactor.blockhound.BlockingOperationError: Blocking call! sun.misc.Unsafe#park
           at sun.misc.Unsafe.park(Unsafe.java)
            at java.util.concurrent.locks.LockSupport.park(LockSupport.java:175)
            at java.util.concurrent.locks.AbstractQueuedSynchronizer.parkAndCheckInterrupt(AbstractQueuedSynchronizer.java:836)
            at java.util.concurrent.locks.AbstractQueuedSynchronizer.acquireQueued(AbstractQueuedSynchronizer.java:870)
            at java.util.concurrent.locks.AbstractQueuedSynchronizer.acquire(AbstractQueuedSynchronizer.java:1199)
            at java.util.concurrent.locks.ReentrantLock$NonfairSync.lock(ReentrantLock.java:209)
            at java.util.concurrent.locks.ReentrantLock.lock(ReentrantLock.java:285)
            at io.netty.buffer.PoolArena.lock(PoolArena.java:744)
            at io.netty.buffer.PoolArena.freeChunk(PoolArena.java:256)
            at io.netty.buffer.PoolArena.free(PoolArena.java:245)
            at io.netty.buffer.PooledByteBuf.deallocate(PooledByteBuf.java:176)
            at io.netty.buffer.AbstractReferenceCountedByteBuf.handleRelease(AbstractReferenceCountedByteBuf.java:111)
            at io.netty.buffer.AbstractReferenceCountedByteBuf.release(AbstractReferenceCountedByteBuf.java:101)
            at io.netty.buffer.WrappedByteBuf.release(WrappedByteBuf.java:1037)
            at io.netty.buffer.SimpleLeakAwareByteBuf.release(SimpleLeakAwareByteBuf.java:102)
            at io.netty.buffer.AdvancedLeakAwareByteBuf.release(AdvancedLeakAwareByteBuf.java:942)
            at io.netty.handler.ssl.SslHandler.attemptCopyToCumulation(SslHandler.java:2345)
            at io.netty.handler.ssl.SslHandler.access$2800(SslHandler.java:169)
            at io.netty.handler.ssl.SslHandler$SslHandlerCoalescingBufferQueue.compose(SslHandler.java:2304)
            at io.netty.channel.AbstractCoalescingBufferQueue.remove(AbstractCoalescingBufferQueue.java:176)
            at io.netty.handler.ssl.SslHandler.wrap(SslHandler.java:818)
            at io.netty.handler.ssl.SslHandler.wrapAndFlush(SslHandler.java:799)
            at io.netty.handler.ssl.SslHandler.flush(SslHandler.java:780)
            at io.netty.channel.AbstractChannelHandlerContext.invokeFlush0(AbstractChannelHandlerContext.java:750)
            at io.netty.channel.AbstractChannelHandlerContext.invokeFlush(AbstractChannelHandlerContext.java:742)
            at io.netty.channel.AbstractChannelHandlerContext.flush(AbstractChannelHandlerContext.java:728)
            at io.netty.channel.CombinedChannelDuplexHandler$DelegatingChannelHandlerContext.flush(CombinedChannelDuplexHandler.java:531)
            at io.netty.channel.ChannelOutboundHandlerAdapter.flush(ChannelOutboundHandlerAdapter.java:125)
            at io.netty.channel.CombinedChannelDuplexHandler.flush(CombinedChannelDuplexHandler.java:356)
            at io.netty.channel.AbstractChannelHandlerContext.invokeFlush0(AbstractChannelHandlerContext.java:750)
            at io.netty.channel.AbstractChannelHandlerContext.invokeFlush(AbstractChannelHandlerContext.java:742)

The NettyBlockHoundIntegration has no mention of the pooled allocator, so I guess blockhound doesn't notice synchronised methods?

The NettyBlockHoundIntegration has no mention of the pooled allocator, so I guess blockhound doesn't notice synchronised methods?

Yep that's my assumption also ...
Should we add to the exclusions PoolArena.lock and PoolSubpage.lock


Custom tracer doesn't seem to work with Kaminari pagination

I have the following setup in the initialiser:
  tracer = ActiveSupport::Logger.new("log/elasticsearch.log")
  tracer.level = Logger::DEBUG
  Elasticsearch::Model.client = Elasticsearch::Client.new(
    hosts: ELASTICSEARCH_HOSTS,
    tracer: tracer
  )

# Paginate Elasticsearch searches directly on Elasticsearch with Kaminari
# see : https://github.com/elastic/elasticsearch-rails/tree/main/elasticsearch-model#pagination
Kaminari::Hooks.init if defined?(Kaminari::Hooks)
Elasticsearch::Model::Response::Response.include Elasticsearch::Model::Response::Pagination::Kaminari

When I run Model.__elasticsearch__.search(query) it prints the CURL and other logs to log/elasticsearch.log
However, when I run Model.__elasticsearch__.search(query).page(1).per(10), nothing is printed to log/production.log.
Environment

Rails - <IP_ADDRESS>
Elasticsearch-rails - 7.2.0
Elasticsearch-model - 7.2.0
Elasticsearch (Ruby gem) - 7.16.3
Elasticsearch (instance) - 7.16.3

Apologies. Seems to be working correctly. There seem to be delays with the file sync from the docker container.


cohre: don't merge, testing a feature in pylibjuju: new wait for idle

Thanks, I'll cook up something new next week.


Is there a way to jump to the line in preview window?

I am using this to search for text in long pdfs, so the searched term might not be visible in the preview window without scrolling a lot.
From what I can tell the preview window always starts on the first line, is there any way to have it start on the line of the first match?
Thanks

You could tweak the grep call of the preview window to show the context around the matches instead of the whole text. For instance, in the .bashrc code, change grep -E "^|$v" -i --color=always to grep -E "$v" -i -B 3 -A 10 --color=always for 3 lines before the match and 10 lines after, see https://stackoverflow.com/a/9083/13430450.
The fact that grep -E "^|$v" -i outputs the whole text is based on https://superuser.com/a/1514824: ^| ensure that every line will be a match.

You might prefer to use https://github.com/phiresky/ripgrep-all with the fzf integration given there.


Twitter post images not generating inside image tag due to verify_image_url function

Describe the bug
verify_image_url Function is assuming twitter post images with ":orig", ":large" ":medium" or ":small" as invalid images and thus is sending empty image value on the fetch return array.
Generated example:
Array
(
    [0] => Array
        (
            [robots] => NOODP
            [msapplication-TileImage] => //abs.twimg.com/favicons/win8-tile-144.png
            [msapplication-TileColor] => #00aced
            [facebook-domain-verification] => moho2ug7zs57jijiywrewd8wb5a08h
            [swift-page-name] => permalink
            [swift-page-section] => permalink
            [al:ios:url] => twitter://status?id=1524828581536440320
            [al:ios:app_store_id] => 333903271
            [al:ios:app_name] => Twitter
            [al:android:url] => twitter://status?status_id=1524828581536440320
            [al:android:package] => com.twitter.android
            [al:android:app_name] => Twitter
            [type] => article
            [url] => https://twitter.com/MexicanSkynet/status/1524828581536440320
            [title] => SKYNΞT | 🔴 twitch.tv/mexicanskynet on Twitter
            [image] =>
            [image:user_generated] => true
            [description] => “Hoy tenemos el mas pintoresco y tradicional torneo M7GP a las 9:00 pm CUU / 10:00 pm CDMX

Recuerda unirte al chat de voz: https://t.co/GGyxQJteMB
y recuerda sintonizarnos por: https://t.co/L0KmK5iRHt

#MarioKart8Deluxe #mariokart #nintendoswitch #twitchaffiliate”
            [site_name] => Twitter
            [fb:app_id] =><PHONE_NUMBER>
        )

)

After commenting line 56 trough line 51 in src/OpenGraph.php it successfully generated the image url on this example:
Array
(
    [0] => Array
        (
            [robots] => NOODP
            [msapplication-TileImage] => //abs.twimg.com/favicons/win8-tile-144.png
            [msapplication-TileColor] => #00aced
            [facebook-domain-verification] => moho2ug7zs57jijiywrewd8wb5a08h
            [swift-page-name] => permalink
            [swift-page-section] => permalink
            [al:ios:url] => twitter://status?id=1524828581536440320
            [al:ios:app_store_id] => 333903271
            [al:ios:app_name] => Twitter
            [al:android:url] => twitter://status?status_id=1524828581536440320
            [al:android:package] => com.twitter.android
            [al:android:app_name] => Twitter
            [type] => article
            [url] => https://twitter.com/MexicanSkynet/status/1524828581536440320
            [title] => SKYNΞT | 🔴 twitch.tv/mexicanskynet on Twitter
            [image] => https://pbs.twimg.com/media/FSlHM2bWIAAQn-8.jpg:large
            [image:user_generated] => true
            [description] => “Hoy tenemos el mas pintoresco y tradicional torneo M7GP a las 9:00 pm CUU / 10:00 pm CDMX

Recuerda unirte al chat de voz: https://t.co/GGyxQJteMB
y recuerda sintonizarnos por: https://t.co/L0KmK5iRHt

#MarioKart8Deluxe #mariokart #nintendoswitch #twitchaffiliate”
            [site_name] => Twitter
            [fb:app_id] =><PHONE_NUMBER>
        )

)

Expected behavior
Url array key should be populated if a twitter post has an image.
** System **
Laravel 8
PHP 7.4
Side note
anyone having problems getting metadata from socialmedia please use the following user agent set on this example:
$opg_array = OpenGraph::fetch('URL', true, null, null, 'facebookexternalhit/1.1 (+http://www.facebook.com/externalhit_uatext.php)');

@ItsSkynet Thanks for pointing this out. Let me know if you would like to open a PR to fix this issue?

i did a little bit of research and it seems that the php constant "FILTER_VALIDATE_URL" is too strict  when it comes to validate url with this kind of characteristics.
On php 5.5 it was deemed buggy and in today php versions it's deemed too strict by the community.
Removal of this constant would do no good, but instead it would be best advised to find a way to better sanitize the url before reaching this point. i will try to come up with a way to solve this. in the meantime if anyone has an idea, by all means PR it.

Closed by PR #84


How to config nginx default file.

I've created ssl connection succesfully, but currently websocket connection is not working.
just googled and knew that it has problem in nginx config.
is there standard config method for nginx?
I'm using AWS EC2 Ubuntu 18.04 instance.

just found problem myself.


How do i pronounce Endlessh

is it "Endless ess-ess-aitch" ?
is it "Endless-aitch" ?
is it "Endless-shh" ?

I pronounce it "end-lesh" but whatever is convenient or most effective for
you when communicating is just fine. I've also heard "endless ssh" as two
words often enough.


[Performance] ruby-prof-flamegraph support

Hi,
I found this gem "gem 'ruby-prof-flamegraph'" at https://github.com/oozou/ruby-prof-flamegraph which may help us to detect performance issues.
Thanks.

@kiranfractal Are you seeing perfromance issues? This is definitely a worth while task, but it may be useful to leave this until there are complaints and/or a new UI.
Closing for now.

I have not seen any issues so far.
I agree with you and we can take it up later.
Thanks.
On Mon, Mar 23, 2015 at 7:47 AM, joffems<EMAIL_ADDRESS>wrote:

@kiranfractal https://github.com/kiranfractal Are you seeing
perfromance issues? This is definitely a worth while task, but it may be
useful to leave this until there are complaints and/or a new UI.
Closing for now.
—
Reply to this email directly or view it on GitHub
https://github.com/msjit/testcasedb/issues/63#issuecomment-84754571.


feature-wip iceberg v2

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 38.05 seconds
load time: 571 seconds
storage size:<PHONE_NUMBER>0 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221102124710_clickbench_pr_38569.html

这个功能什么时候支持 EQUALITY_DELETES 有计划么？


Add information to wiki on maintaining overview of distributions on developer/distributions.rst

In #220 an image is used in developer/distributions.rst. First, this needs to be updated every time new distributions are added. As such, once it has been merged this should be added to the wiki to inform contributors.

This image was removed, as it would be too time consuming to keep it updated.


Feature: npm missing script, did you mean this

Feature request
npm ERR! Missing script: "dev-discord-"
npm ERR!
npm ERR! Did you mean this?
npm ERR!     npm run dev-discord-bot # run the "dev-discord-bot" package script
npm ERR!
npm ERR! To see a list of scripts, run:
npm ERR!   npm run

Detect "npm ERR! Did you mean this?" and run the suggested command

oh wow nobody made this
Time to make it a thing soon (don't close this issue please)


Deprecate Text proptypes

Summary
This pull request moves Text's prop types to the DeprecatedPropTypes folder.
This was already partly in progress - there were redundant TextPropTypes and DeprecatedTextPropTypes files so I removed one, and made sure the version with the doc strings was the one used.
Changelog
[General] [Deprecated] - Move Text component's proptypes to DeprecatedPropTypes
Test Plan
Flow checks pass for iOS and Android

Awesome, thanks!


Last Docker image seems broken

I started the last Docker image and I got the error:
Poco::Exception. Code: 1000, e.code() = 0, e.displayText() = File not found: ${CLICKHOUSE_CONFIG}, e.what() = File not found
Clickhouse is started by:
CMD ["/usr/bin/clickhouse-server", "--config=${CLICKHOUSE_CONFIG}"]
But CMD doesn't support variables.

It seems it broken starts from 18.14.* I checked today several published versions and only 18.12.17 works fine for me. All shows the same error reported above.

If you use ClickHouse with docker compose you can fix this by adding the correct upstart command. Here's my docker-compose.yml:
clickhouse:
    ulimits:
      nproc: 65535
      nofile:
        soft: 20000
        hard: 362144
    image: yandex/clickhouse-server
    logging:
      driver: json-file
    volumes:
      - /dev/shm:/dev/shm
      - ./config.xml:/etc/clickhouse-server/config.xml
    ports:
      - "8123:8123"
      - "9000:9000"
    command: /usr/bin/clickhouse-server --config=/etc/clickhouse-server/config.xml

Fixed.
$ docker run -d --name some-clickhouse-server --ulimit nofile=262144:262144 -p9000:9000 yandex/clickhouse-server
7db6d5c8a12df92bb71ff65c0281bfae20c9401180b18ea0aaa8eb7dd43afcbc
$ clickhouse-client
ClickHouse client version 18.11.0.
Connecting to localhost:9000.
Connected to ClickHouse server version 18.14.9 revision 54409.

7db6d5c8a12d :)


dashboard docs

Pull Request Template
Type of Change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Description
Please include a summary of the change and which issue it addresses. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Checklist

[ ] I have read the CONTRIBUTING document.
[ ] My code follows the code style (BLACK) of this project.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] I have updated the documentation accordingly.
[ ] I have added an appropriate CHANGELOG entry.

Author's Note
Please provide any additional information, questions, or concerns you have about this pull request.

Can we put Dashboard above Evaluations?

Can we put Dashboard above Evaluations?

Idts
Evaluations should come before dashboard, a user would want to know what the evaluations are first


Merge release/0.9.9 back to develop

This is a merge with fixed conflicts.

Minor typo, otherwise LGTM.


Game of blocks collection submission

We will need to update this every couple of days as our game progresses.
We are also running a server with the updated off chain images.
I have included the links in the metadata if it's possible to use off chain image links, we will update these each time so that the images on the marketplace are correct as quickly as possible after each stage of the game progresses
Each phase of the game goes for about 48 hours so we need to update each time until the halving

Updated inscriptions.json with the most recent metadata
Would be great to get this merged as soon as possible so that the collection supply and images update
Our collection is an onchain game and so the supply decreases every 48 hours (288 blocks), depending on how people play the game
Would also be great to have the filters added if that is possible


Multiple AWS accounts?

This project looks really cool. I'm guessing it would take a lot of work to support multiple accounts? For example, let the user select via drop-down which account to view.
Thanks.

It supports multiple accounts today, but it only supports using a single set of AWS credentials.  So that means you need to configure each account's CloudTrail settings to send notifications to a single SQS queue and S3 bucket.  CloudTrail stores the logs in S3 beneath an account ID, so it works well with many accounts.
In Kibana, you can easily add a filter based on account ID.


WIP - GeoPySpark VectorPipe: Tobler

This is built off feature/vectorpipe on GeoPySpark repo.
This branch and PR can serve to track the deployment changes for that work.

s3://vectortiles/orc/europe/luxembourg.orc as test dataset


Documentation: Update curl for the helm template init container example

On this page, the second example for version 24.0.5 the curl command should use the mounted volume
currently:
curl -L -f -S -o /extensions/keycloakify-starter.jar  ...

updated:
curl -L -f -S -o /emptydir/app-providers-dir/keycloakify-starter.jar ...

Hey @jdockeray,
Thank you for reaching out,
Are you sure though?
It depends where you have you empty dir mounted. In the instruction we have:
keycloak:
  initContainers: |
    - name: realm-ext-provider
      image: curlimages/curl
      imagePullPolicy: IfNotPresent
      command:
        - sh
      args:
        - -c
        - |
          # Replace USER and PROJECT.
          curl -L -f -S -o /extensions/keycloakify-starter.jar https://github.com/USER/PROJECT/releases/latest/download/keycloak-theme-for-kc-24.jar

      volumeMounts:
        - name: extensions
          mountPath: /extensions

  extraVolumeMounts: |
    - name: extensions
      mountPath: /opt/bitnami/keycloak/providers

  extraVolumes: |
    - name: extensions
      emptyDir: {}

If you want your suggestion to work you would have to write:
keycloak:
  initContainers: |
    - name: realm-ext-provider
      image: curlimages/curl
      imagePullPolicy: IfNotPresent
      command:
        - sh
      args:
        - -c
        - |
          # Replace USER and PROJECT.
          curl -L -f -S -o /emptydir/app-providers-dir/keycloakify-starter.jar https://github.com/USER/PROJECT/releases/latest/download/keycloak-theme-for-kc-24.jar

      volumeMounts:
        - name: extensions
          mountPath: /emptydir/app-providers-dir

  extraVolumeMounts: |
    - name: extensions
      mountPath: /opt/bitnami/keycloak/providers

  extraVolumes: |
    - name: extensions
      emptyDir: {}

But it's apple to oranges isn't it?

Hey @jdockeray,
Thanks for replying and maintaining this great project!
I am pretty sure. There are two code samples on that page, the first one (which you shared) is fine because the mount path is extensions. But in the second sample, which looks like this:
keycloak:
  initContainers: |
    - name: keycloak-plugin
      image: curlimages/curl
      imagePullPolicy: IfNotPresent
      command:
        - sh
      args:
        - -c
        - |
          # Replace USER and PROJECT.
          curl -L -f -S -o /extensions/keycloakify-starter.jar https://github.com/USER/PROJECT/releases/latest/download/keycloak-theme-for-kc-24.jar

      volumeMounts:
        - name: empty-dir
          mountPath: /emptydir

  extraVolumeMounts:
    - mountPath: /opt/bitnami/keycloak/providers
      name: empty-dir
      subPath: app-providers-dir

  extraVolumes:
    - emptyDir: {}
      name: empty-dir

The mountPath for volume mounts is /emptydir, but the curl command still references /extensions/, so the downloaded jar would not get mounted in the volume.

Ohhhh right!
I didn't scrolled down to see the second example.  I tend to merge the pull request on the documentation without being carefull enoug.
I have updated it. I don't think it's a requirement to have it in the sub path app-providers-dir, I would glodaly have your confirmation on that.
https://docs.keycloakify.dev/importing-your-theme-in-keycloak

Thank you for testing it!


docs: Add AWS-LearningResource to Extra Content table

Add https://github.com/tungbq/AWS-LearningResource to this table:
https://github.com/tungbq/devops-basic#extra-content-from-other-repos

AWS Learning Resource has been added to content table, please check and review my PR

AWS Learning Resource has been added to content table, please check and review my PR

Thank you, please check my comment


History list sorts out of ID order

https://recruit.karmafleet.org/recruits/application_queue/history/
The first 3 applications are ID's 33, 35, then 34. They are out of order all throughout the page as well.

Added viewing of app ids in the queue (including history)
it appears to sort correctly

seems to be fine now.


Consider pex instead of pip for locking

We are currently using pip install --dry-install --report in fetchPipMetadata to create our python lock files. While this works better than earlier FOD-based approaches for many use-cases, it has (at least) to major issues:

cross-platform locking: We can't currently use that approach to e.g. create aarch64-darwin lock files on x86_64-linux, because we can't easily fake platform environment markers, see https://github.com/pypa/pip/issues/11664#issue-1500189934
evaluation of extras & environment markers currently happens at lock-time, not at build-time. While this approach was easier to implement because we could to it in python, it means that

the lock file becomes much larger as we need to lock each platform-extra combination ahead of time
it can result in incorrect cross-platform locks for legacy sdists which use setup.py. This is because we would need to prepare metadata by executing the setup.py on the build-platform. If the code in setup.py introspects the interpreter to determine the platform that could result in wrong metadata being output.

https://github.com/nix-community/dream2nix/issues/583#issuecomment-1650443589 recommended https://pex.readthedocs.io as an alternative to pip to tackle the first problem (and support cyclic dependencies, as that's what the linked issue was about).
pex lock files include unevaluated enviroment markers,same as pips installation report, i.e. "brotli>=1.0.9; platform_python_implementation == \"CPython\" and extra == \"brotli\"",. So we would need to evaluate those in nix. Prior work for that includes poetry2nix pep425.nix.
What do you think, is that something we should try? Would obsolete my work in https://github.com/phaer/untangled_snakes/ but I am happy about that as long as it improves our python support.
@DavHau @chaoflow

pex lock files include unevaluated enviroment markers,same as pips installation report, i.e. "brotli>=1.0.9; platform_python_implementation == "CPython" and extra == "brotli"",. So we would need to evaluate those in nix.

What is the transform you do at a high level? From lock file to ... venv? sys.path entry (flat directory of installed wheels) or something else? Pex has alot of tools to work with a lock file.

Would it still work with the max-date setting that's currently supported by the pip locker? That's very handy!

What is the transform you do at a high level? From lock file to ... venv? sys.path entry (flat directory of installed wheels) or something else?

I'd say "something else", but a sys.path entry sounds close; We eventually put each python package in separate derivations.
Those derivations can be build "purely", meaning they get all their info from inputs declared in the repo (lock file, nixpkgs, etc) but don't have network access during build-time.
With the current approach, we just evaluated the markers during lock-time and wrote the evaluated/"effective" dependencies into the lock file, see e.g. https://github.com/nix-community/dream2nix/blob/c93ace7b6b4ba54296442efc9ddd8734826d661a/modules/drvs/ansible/lock-x86_64-linux.json#L67 for what that looks like atm.
This has the significant drawback that we'd need to lock each extra/platform combination separately, so moving marking evaluation to build-time seems like the right way forward to me.
Poetry2nix re-implements pep425 and pep508 in nix.
Another alternative would be to call pex again during build-time (pex shouldn't need network connectivity or so for those commands).
The latter has the advantage of probably having a better tested implementation, but the disadvantage of adding pex to our build-closure.
I'll try to write a proof-of-concept for the latter approach next week or so :)

Would it still work with the max-date setting that's currently supported by the pip locker? That's very handy!

@DavHau and me recently wondered if there are any real use cases for that, so curios what you are using for? In any case, that feature works by mitm-proxying pypi, so that should be doable with pex as well if needed.

what you are using for?

I'm packaging Odoo. It's a mastodon composed of many pieces and tons of dependencies (js, binary, ruby, python...). This beast moves fast in some points and slow in some others. Their approach towards dependencies is "we officially support those supported by current debian stable at the time of launching Odoo". Odoo releases have a lifespan of 3 years.
As you can imagine, the deps diff accumulated between nixpkgs and debian stable in 3 years is abysmal. Besides, it's quite easy that there's a new dependency release that breaks Odoo in some way. One recent example: https://github.com/odoo/odoo/pull/124351 (all CI is ❌ there).
Our derivations include many more modules. Those modules evolve at a different pace and may introduce new dependencies.
As you can imagine, keeping the balance between having a stable and an updated-enough deployment is no easy task.
So, summarizing, all I use this feature for is to be able to tell my devs: "if you add a new addon that has new dependencies, just run nix run .#refresh and commit those locks". And I like the peace of mind that the date capping gives me. That command will only add new stuff for that new module, but will not update any other dependencies that may break some other portion of the system. Then I can set up a renovate action that upgrades that date and refreshes the lock files, to let CI detect incompatibilities if any.

I think the max-date setting is very valuable as of now as it allows us to update individual dependencies. If we instead use a third party tool to manage the lock file, then this tool might already come with the ability to only update a single dependency, so we might not need the max-date feature anymore.

Hello,
I am giving a python module for dream2nix based on pex lock files and pyproject.nix a try - see #611 for details.

pex and pex3 lock create don't seem to work when installed via nix. I've opened an issue at https://github.com/NixOS/nixpkgs/issues/246448
https://github.com/pantsbuild/pex/issues/2100 would be very useful for us. Would also be open to try and help implement that.

If we instead use a third party tool to manage the lock file, then this tool might already come with the ability to only update a single dependency, so we might not need the max-date feature anymore.

This is in fact true for Pex. You can use pex3 lock update -p "just-this<3" lock.json to update the existing lock.json if possible by keeping everything the same but trying to bump the "just-this" project to the maximum compatible version less than 3.

I've tested it pex3. Indeed the lock file is quite cool!
However, regarding UX, until now, dream2nix provides the interface to refresh the lock file:
nix run -L .#package.config.lock.refresh

This, together with the mitmproxy, gives a reproducible output based on the date. It means, in practice, that the target package can be packaged no matter with wich framework: poetry, pdm, flit, setuptools, or just a raw requirements.txt file. As long as the dependency is properly fed into dream2nix, it will produce the correct lock file.
I'm not talking only about times when you have to install dependencies, but also times where you use dream2nix to maintain a python project itself. In your example, it's using setuptools. But if I were using poetry, it'd work too.
If we rely on pex3 to maintain the lock file by following the strategy you're saying, then dream2nix users would need to manually keep it up to date, instead of the current process of:

Update your dependency.
Re-lock.

Given the use case I have (https://github.com/nix-community/dream2nix/issues/601#issuecomment-1653338668), I really appreciate the current simplicity of that process.
FWIW this is the requirements.txt file I feed into dream2nix. Pretty please don't make me maintain manually a lock file based on those requirements. 🙏🏼 😓

Hello & thanks for the detailed description, but could you clarify the following part:

If we rely on pex3 to maintain the lock file by following the strategy you're saying, then dream2nix users would need to manually keep it up to date, instead of the current process of:

Because i am not sure if I follow here: You'd like to have a mode where you update the lock-file to the newest packages that satisfy the constraints in your pyproject.tom/requirements.txt/setup.py?
If so, wouldn't it work like that if you just set the snaphot date to null?
Or are you talking about the the incremental aspects of https://github.com/nix-community/dream2nix/issues/601#issuecomment-1664271430, where one could re-use the existing lock-file?
Agree that this would be a nice feature to have and plan to do so, but it's not implemented yet.

You'd like to have a mode where you update the lock-file to the newest packages that satisfy the constraints in your pyproject.tom/requirements.txt/setup.py?

Yes.

If so, wouldn't it work like that if you just set the snaphot date to null?

Well, the difference is that each time you run it, you'd get different results. Right now, OTOH, each time you run it, you get the same results because of the mitmproxy + pypi snapshot date.
Imagine the problem when one dependency fails; then you patch it; then you re-lock but another dependency slips in and breaks differently. I've been there when using poetry and poetry2nix: poetry has the nasty habit of updating dependencies every time it can. It's really frustrating.
So, the current approach on dream2nix is awesome. That's what I meant! ❤  The only thing it's lacking is multi-arch locking. So, if possible, please fix just that (maybe with pex) and leave the rest of the UX as it is, because it's great!
It's so great that I've stopped using poetry and went back to setuptools on a project because poetry now adds no value. It's so great because I could use hatch, setuptools, or flit, or poetry, or whatever... and the workflow would be exactly the same.

TIL about pip-tools... would this help? https://pip-tools.readthedocs.io/en/stable/#using-hashes
It seems to produce a hashed lock file from any requirements.txt

I think we should just focus on stabilizing the pdm module. It fixes most of the major issues we currently have with pip.

@yajo the pdm module does allow you to update individual dependencies via the pdm cli and it also has multi platform lock files. Maybe you want to give it a shot? I could use someone thoroughly testing it.

I'll give it a shot then and report. However I can't promise it'll be soon 😅

I still didn't get a chance to test the pdm module. Is that the final answer to this issue?

No, don't worry. I think the issue title is a bit misleading because it focuses on the solution rather than the problem.
The problem is that dream2nix can't build multiarch flakes. If that's solved it's OK closing it AFAIK.


Need to have Registration Tickboxes

Need to have Tickboxes/flags in the Registration system. All (most?) would be flagged by internal staff and only visible to internal staff. We would want to uses these flags to pull lists/reports.
Some would be "forever" flags and some would need to be reset each year.
Tickboxes needed for the following:
Guests of Honor (yr/yr)
guests of Guests of Honor (yr/yr)
Invited Participants (yr/yr)
Volunteers (those interested in volunteering) (yr/yr)
Dealers (yr/yr)
ADA (forever/temp)
Founding Board Members (forever)
VSP-Brown (Forever/temp)
VSP-Black (Forever/temp)

We've captured this slightly different (at least some of it) in other requirements gathering. @dleverentz


System reports incorrect audible "Smart lock not responding" alert

This is ridiculous but since I got a pair of smart locks I have been getting an incredibly annoying "smart lock not responding" error. It seems other people do as well so I assumed it was an issue with SS. Guess what, today I discovered at least in my case it happens when the plugin refreshes the sensors just after an alarm arm / disarm.
See https://www.reddit.com/r/simplisafe/comments/f80abl/simplilock_not_responding_message/
Already have a fix in progress, this issue is for documentation.

I’m still having this issue despite the update.

Hi @asabatino88, do you have your sensorRefresh set to the default 15 seconds? Do you get the error every time or just sometimes? This fix completely solved it for me and others.
Would you be able to post your log output and config? You have to enable debug on this plugin and the homebridge debug option (-D). Then arm/disarm and send the logs starting there for the next minute or two. Thanks

Let me give it a few more days - I have not heard the error message today.
The refresh is set to the default. Perhaps it took a minute to cycle
through the base station. Thx for following up!
On Sun, Jul 19, 2020 at 9:42 PM Michael Shamoon<EMAIL_ADDRESS>wrote:

Hi @asabatino88 https://github.com/asabatino88, do you have your
sensorRefresh set to the default 15 seconds? Do you get the error every
time or just sometimes? This fix completely solved it for me and others.
Would you be able to post your log output and config? You have to enable
debug on this plugin and the homebridge debug option (-D). Then
arm/disarm and send the logs starting there for the next minute or two.
Thanks
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nzapponi/homebridge-simplisafe3/issues/134#issuecomment-660769126,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANB6JP44FW6JGC6GSZGYVKTR4OVI3ANCNFSM4OQD7GSQ
.

--
Anthony Sabatino
Email<EMAIL_ADDRESS><EMAIL_ADDRESS>

Be advised, this issue, "Warning: XYZ Smartlock not responding", exists with the standard Simplisafe architecture using only Simplisafe provided (not Homebridge) API, hardware, etc.  Just replaced both the lock and the base station to try to resolve without success.  There is something wrong in the code in the base station, I think.

Be advised, this issue, "Warning: XYZ Smartlock not responding", exists with the standard Simplisafe architecture using only Simplisafe provided (not Homebridge) API, hardware, etc.  Just replaced both the lock and the base station to try to resolve without success.  There is something wrong in the code in the base station, I think.

Thanks for the heads up @ericjackson1969 . Yea their API programming leaves something to be desired... 😒

Thanks for the heads up @ericjackson1969 . Yea their API programming leaves something to be desired... 😒


[WIP] Test with DMD 2.068.0

This is more of a tracking issue, so that you are aware it is being worked on :)
This will go red because the following snippet doesn't compile anymore:
void main()
{
  import std.string;
  enum Test = "Hello\0World";
  char[64] foo;
  foo[0 .. Test.length] = Test[];
  auto greetings = foo[0 .. foo[].indexOf('\0')];
}

I'll report once issues.dlang.org comes back online (currently returning 504 :( ).

So, libasync segfault ? Ping @etcimon :)


[JENKINS-60985] - Fix checkout serialization error

JENKINS-60985 Fix checkout serialization error
The previous call to checkout was entirely inside the CliGitAPIImpl and JGitAPIImpl classes.  The replacement used a CheckoutCommand that is not serializable.  This change reverts the replacement.  Issue has been in the git client plugin since git client plugin 3.1.0.
The change development process visited each of the changes in git-client-plugin#399 and retained the changes from that PR which are safe from serialization issues.
Would appreciate review by @dwnusbaum and @fcojfernandez so that we can deliver a new version to fix the issue.
Would love to have suggestions or pointers to ways that this could be tested in the git client plugin test automation.
Checklist

[x] I have read the CONTRIBUTING doc
[x] I have referenced the Jira issue related to my changes in one or more commit messages
[ ] I have added tests that verify my changes
[x] Unit tests pass locally with my changes
[x] I have added documentation as necessary
[x] No Javadoc warnings were introduced with my changes
[x] No spotbugs warnings were introduced with my changes
[ ] I have interactively tested my changes

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)

@dwnusbaum I've added comments to clarify the retention of the deprecated calls.  Also annotated a few deprecated methods so that the annotation matches the Javadoc.


Chat - Mention whisper returns, incorrect mention buttons

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: <IP_ADDRESS>
Reproducible in staging?: Y
Reproducible in production?: N
Found when validating PR : https://github.com/Expensify/App/pull/41406
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:

Log in with a nexpensifail user
Create a room
Write any message
Thread the message and write any text
Tag a non-existing user in the thread
Copy the code from the PR
Paste it into Chrome console and replace "id" in the last row with the thread ID from the URL
Click on "Yes"

Expected Result:
The buttons should be "Create it" and "Do nothing".
The whisper message shouldn't return.
Actual Result:
The whisper buttons turned to "Yes" and "No".
The whisper message returns after 10-80 seconds.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/fbfc06e1-0eef-4851-b0ca-a4feb3e0f7cd

View all open jobs on GitHub

Production
https://github.com/Expensify/App/assets/115492554/1bdf9a37-f5d5-455d-8398-b580308ed842

@izarutskaya is this only occurring with Expensifail accounts?

Confirmed reproduction, though not sure if this is a deploy blocker if only impacting expensifail accounts. cc @pecanoro

Copy the code from the PR

Which PR? Is this coming from QAing something in particular?

We changed this so the buttons should be Yes and No, but also I think we can mark this PR as NO QA and close this issue, there is still a pending Auth PR https://github.com/Expensify/Auth/pull/10734 to make everything work and the end user won't be able to see the report mention whisper until we merge the Auth PR.


Bellman form equations for the "cyclical" example

(Related to #562)
There is a "cyclical" example at the end of this example notebook:
https://github.com/econ-ark/HARK/blob/master/examples/ConsIndShockModel/IndShockConsumerType.ipynb
It demonstrates the functionality of HARK that it can model a situation with cyclically changing parameter values.
I cannot find a Bellman form representation of this model.

Am I correct in thinking that what HARK is doing for cyclical models is essentially:

Introducing a new state variable c and new parameter C
update rule: c' = c +1 mod C
allowing parameters to be vectors of length C, in which case other update rules will refer to the scalar value of that vector indexed at c

Sort of?  But time-varying parameters are "lists of length C" anyway,
regardless of whether the model is cyclical.
On Tue, Mar 24, 2020 at 5:38 PM Sebastian Benthall<EMAIL_ADDRESS>wrote:

Am I correct in thinking that what HARK is doing for cyclical models is
essentially:

Introducing a new state variable c and new parameter C
update rule: c' = c +1 mod C
allowing parameters to be vectors of length C, in which case other
update rules will refer to the scalar value of that vector indexed at
c

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/issues/563#issuecomment-603518446, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADKRAFORBLKL4HJQVBV26HLRJER5RANCNFSM4LFVDADQ
.

I'm trying to pin down the math.
My understanding is that there are four cases:

A) infinite horizon, non-cyclical ("simple")
B) finite horizon, non-cyclical ("lifecycle", maybe confusingly.)
C) infinite horizon, cyclical ("seasonal")
D) finite horizon, cyclical (???)

I am under the impression that in (A), all parameters are time-invariant and scalars.
I am under the impression that in (C), there may be time-varying parameters that are vectors of length C, indexed by c.
I think what you are saying is that in (B), there can be time-varying parameters. These would be vectors of length...I guess it's of length L, the lifespan of the agent.
I would expect that in case (D), you might have time-varying parameters of either length C or length L, depending on whether they were seasonal or age-dependent variables.
Am I getting warmer?

Whether a particular parameter is in time_vary or time_inv does not depend
on whether it's finite or infinite horizon, or on whether the number of
periods in the cycle is 1 or greater than 1.  It's not particularly
meaningful to speak of "time-varying" parameters when the same period is
repeated over and over again, but it's there.
When you make an instance of an AgentType subclass, you would put the same
data type into an attribute no matter what.
In case B, each attribute named in time_vary should be a list of length
T_cycle.  But that's true in A, C, and D as well.
On Tue, Mar 24, 2020 at 5:58 PM Sebastian Benthall<EMAIL_ADDRESS>wrote:

I'm trying to pin down the math.
My understanding is that there are four cases:

A) infinite horizon, non-cyclical ("simple")
B) finite horizon, non-cyclical ("lifecycle", maybe confusingly.)
C) infinite horizon, cyclical ("seasonal")
D) finite horizon, cyclical (???)

I am under the impression that in (A), all parameters are time-invariant
and scalars.
I am under the impression that in (C), there may be time-varying
parameters that are vectors of length C, indexed by c.
I think what you are saying is that in (B), there can be time-varying
parameters. These would be vectors of length...I guess it's of length L,
the lifespan of the agent.
I would expect that in case (D), you might have time-varying parameters of
either length C or length L, depending on whether they were seasonal or
age-dependent variables.
Am I getting warmer?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/issues/563#issuecomment-603526426, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADKRAFLVU5M2ITLJ4AKKDE3RJEUIXANCNFSM4LFVDADQ
.

Ok. I think I understand.
All model variables named in time_vary are "seasonal" variables. They vary over cyclic time.
I think what i just realized is that HARK does not really support the representation of what I've called case B. I.e, there's no native notion of an age-varying variable.
Rather, if I understand correctly, the way HARK currently handles a lifespan is like case (D): a finite horizon model, with L = C. L being the length of the horizon, C being the length of the cycle.
Ergo, variables like DiscFac_timevary here:
https://github.com/econ-ark/HARK/blob/master/examples/LifecycleModel/EstimationParameters.py#L41-L64

No, case B is completely normal.  You just set cycles=1, and then have the
attributes named in time_vary be lists of length T_cycle.  That's it.
That's a direct representation of the lifecycle.  The "cycle" of periods
happens only once in a standard lifecycle model.
You don't even need to go into the SolvingMicroDSOPs example, which treats
DiscFac as age-varying (which is non-standard).  LivPrb is in time_vary in
the basic IndShockConsumerType and PerfForesightConsumerType.
On Wed, Mar 25, 2020 at 11:04 AM Sebastian Benthall <
<EMAIL_ADDRESS>wrote:

Ok. I think I understand.
All model variables named in time_vary are "seasonal" variables. They
vary over cyclic time.
I think what i just realized is that HARK does not really support the
representation of what I've called case B. I.e, there's no native notion of
an age-varying variable.
Rather, if I understand correctly, the way HARK currently handles a
lifespan is like case (D): a finite horizon model, with L = C. L being
the length of the horizon, C being the length of the cycle.
Ergo, variables like DiscFac_timevary here:
https://github.com/econ-ark/HARK/blob/master/examples/LifecycleModel/EstimationParameters.py#L41-L64
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/issues/563#issuecomment-603892326, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADKRAFMNGVJOSDJT5FHH44DRJIMQ5ANCNFSM4LFVDADQ
.

I'm afraid I'm having difficulty mapping between Bellman equations and the way HARK represents models.
I think T_cycle maps to the variable C, in the schematic math I've been using in this issue.
Where would you put cycles into a Bellman equation?


Stop rotation

Please stop cube... stop... stop...

no

There's an easier solution for this: rotate your screen the opposite direction and the rotations cancel each out.

@xTibor OMG. Can you enter the screen and follow the rotation of the cube? I need the glitch!

I added "faster", "slower", "start", "stop" and "hammer_time" commands in #38

I have moral and ethical objections to this PR

Please stop cube... stop... stop...

Evil pull request

NOTHING CAN STOP THE CUBE.  BOW BEFORE IT!  BOW DOWN!!


BackSlash that inserted in wysiwyg disappear in markdown (2MD)

Version
v1.4.0
Test Environment
Chrome
Current Behavior
/\/
'\'
"\"
(\_)

rendering in markdown like this

Expected Behavior
If backslash is inserted in Wysiwyg, it should not disappear.

On current latest (2.3.1). This issue still occurs on the latest version (2.3.1) and on the online demo. If I do:
foo\
bar

in WYSIWYG, and then switch to markdown and back to WYSIWYG it drops the \.


Alonzo landing page update

What? (required)
What has changed/been added?
Why? (optional)
Why has this change occurred if applicable?
How can this be tested? (optional)
Describe how this change can be tested
Screenshots (optional)
Upload some screenshots of the changes

Approved.


Troubles with using env variables or secrets

      - uses: joshuaavalon/flyway-action@v1
        with:
          url: jdbc:mysql://$MYSQL_HOST:3306/database
          user: $MYSQL_USER
          password: $MYSQL_PW
          locations: filesystem:./conf/db_migrations/sql

I set envs like this:
env:
  MYSQL_HOST: ${{ secrets.MYSQL_STAGING_HOST }}
  MYSQL_USER: ${{ secrets.MYSQL_STAGING_USER }}
  MYSQL_PW: ${{ secrets.MYSQL_STAGING_PW}}

I also tried to set em like this:
          url: jdbc:mysql://${{ secrets.MYSQL_STAGING_HOST }}:3306/database
          user: ${{ secrets.MYSQL_STAGING_USER }}
          password: ${{ secrets.MYSQL_STAGING_PW }}

Both ways resulted with the same error, am I missing something?
ERROR: Unable to obtain connection from database (jdbc:mysql://$MYSQL_HOST:3306/database) for user '$MYSQL_USER': Communications link failure

Communications link failure

It means the host is not reachable. Please check your secrets.
I also added some examples.


use loop in build.sh

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?

Improve build.sh by giving it a CLI interface
Simplify build.sh with loop

Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Documentation
If your pull request contains user-facing changes, please specify the types of the changes, and create a release note. Otherwise, please feel free to remove this section.
Types of user-facing changes
Please keep the types that apply to your changes, and remove those that do not apply.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note
Please create a release note for your changes. In the release note, focus on the impact on users, and mention the environment or conditions where the impact may occur.
Refer to a related PR or issue link (optional)

@skyzh Does this lgty now?

Thanks for the approval @skyzh


feat(catalog-backend-module-okta): add profileNameGroupNamingStrategy

Closes #1077

:heavy_check_mark: Checklist

[x] Added tests for new functionality and regression tests for bug fixes
[x] Added changeset (run yarn changeset in the root)
[ ] Screenshots of before and after attached (for UI changes)
[x] Added or updated documentation (if applicable)

@punkle I can't add reviewers, so hopefully this approach is ok for surfacing the PR


feature/animate-menus

Please check if the PR fulfills these requirements:

[x] The commit message follows our guidelines.
For bug fixes and features:

[x] You tested the changes.
[x] You updated the docs or changelog.

Related Issue #30
New feature
What is the current behavior?
Upgrade buttons just pop in and out of existence
What is the new behavior?
Upgrade buttons animate in and out before and after selecting an upgrade
Other information
First timer.

Good job, I'll merge and make some tweaks.

Thanks much for your contribution!

Thanks much for your contribution!

#30 @NathanLovato
Is the issue closed? Is anything more required?


Change menu to use scale-independent pixels

Just realized about the kivy.metrics module. Use this concept of scale-independent pixels to fix the size of menu, and messages.

Done. Changed the menu height, and buttons font to be scale-independent. Need to check how this works on the phone, will create a new version.
Before that, I will change the final message too.

Done on the end message as well.


change wait to task.wait

Thanks for pointing this out! I'm planning to remove this module entirely in an upcoming refactor. Because of this, we won't be making any further changes to this module.


Tomhaile/ch17466/categorical market weird issue with volume

https://app.clubhouse.io/augur/story/17466/categorical-market-weird-issue-with-volume-gte-big-number-conditional

Coverage remained the same at 57.113% when pulling d4973d8d0e783d90891fdd397245925a250a66f9 on tomhaile/ch17466/categorical-market-weird-issue-with-volume into 8942ce14c9fdd9c7c21a746bf373ca6af13393dd on master.

Tom is going to create a better fix for this by modifying createBigNumber to always convert the first param to a string if it's a number. Closing this PR.


Clarify currentFilterType behavior

Alright, after chatting offline, I think this isn't accurate, but I think there is a very edge case bug here. Closing for now and I'll take a crack at fixing this.


Chrome freezing with floating players running? v2.0.28

Thanks for the great work, I use Floating Player almost everyday!
Unfortunately, I've noticed some strange behavior. Would it be possible to investigate if Floating Player is conflicting with the newer Chrome versions? Ever since Chrome updated a few weeks ago to version 62 (I'm now using v62.0.3202), I've noticed consistent freezing of Chrome with Floating Player running.
I've tried using Chrome 63 (Beta) and 64 (Dev v64.0.3269.3) and the same freezing appears to be happening. I've disabled all extensions to eliminate the culprit, with only Floating Player enabled.
Basically, with Floating Player running, if a Chrome alert message pops up (like to request permission to use my location, to offer to save a password/form fill, or to edit a bookmark), Chrome freezes completely. Floating Player operates without issue until the Chrome alert box pops up. Then everything is frozen and I cannot click on any tabs, or even the upper corner "X" to close the Chrome window.
Please see a screenshot below (or at https://i.imgur.com/Cgkocbq.jpg)

Here, I have a new Chrome Dev window with no installed extensions except Floating Player (with DeskPins). I randomly created a bookmark and right clicked on the bookmark to edit it. As soon as the Chrome pop-up "Edit bookmark" box appears, Chrome is completely frozen and I have to force close it in Task Manager.
I'm using Windows 10, Floating Player v2.0.28
Any suggestions/ideas appreciated. Thanks!

Thanks for letting me know, @prech
Well, I've tested it with Windows 7 and it's working fine. I can't test it with Windows 10 right now but I'll do it in a few days.
For now you can try the following alternatives:

Download 4t Tray Minimizer Free. After installing it, right click the popup title then click "Stay on top", like this:

I think it's a good idea to disable the "additional titlebar buttons":

Use the Floating Player add-on, following these instructions carefully.

Let me know if one of these alternatives worked for you. Have a nice day!

Thanks @gabrielbarros, really appreciate the suggestions and prompt response.
I tried out both alternatives. Interestingly, 4t Tray Minimizer is working well, and when using it, instead of DeskPins, I've encountered no freezing and Floating Player has worked well after at least 5 attempts with a Chrome alert pop-up :)
When I tried DeskPins again, unfortunately, sure enough, Chrome froze immediately when I pinned a Floating Player on top with the Chrome alert pop-up -- this seems to suggest it's the combination of DeskPins-Floating Player that's responsible. I also used DeskPins to pin some other items like an image editor and text editor, and those didn't lock up Chrome, so it seems it's interacting in some fashion with Floating Player.
Thanks also for the Addon for Floating Player alternative. I did give that a try, and enabled the "Open popup using addon on Chrome OS" option, but unfortunately, Floating Player stopped loading altogether after enabling the Addon. I had to go back to the Floating Player options menu and disable the "Open popup..." parameter to get Floating Player to load, at all. Every time I clicked on the Floating Player icon in the Chrome menu, nothing would happen -- no pop up video window. So unfortunately, I wasn't able try out the pinning/stay on top feature as there was no player window to attempt it on.
For reference, I'm using Windows 10 Pro, 64-bit, version 1709.
Thanks

You said Floating Player stopped loading after enabling the Addon? Probably you copied the wrong ID!
Check this out:

Thanks, Gabriel -- I was indeed using the Floating Player extension ID for the Javascript. Using the Addon for FP ID worked to re-enable FP for my videos, but unfortunately, the "Open popup using addon on Chrome OS" option didn't force the player window to stay on top. Any clicks or open applications would appear over the Floating Player.
I've still been having success with 4t Tray Minimizer. No crashes yet after several attempts, and it seems more reliable than DeskPins, which exited by itself every day or two for me.

The popup must stay above the taskbar, so you have to increase the vertical margin to 40 or 50. Give it a try!

Thanks, Gabriel. I had set the vertical margin to 40, then tried 50 and 60, but the same. Any other application or window still seems to take priority

Okay. Well at least 4t Tray Minimizer worked well for you 😊

Thanks for all the troubleshooting tips, Gabriel.
May I ask if you or others have reproduced the Chrome freezing issue with Floating Player and DeskPins?

I just tested Floating Player on Windows 10 and it's working fine. No freezing.
Tested with Windows 10 Version 1703 Build 15063.726 on Chrome 62.0.3202.94 64-bit


Changing Jambonz API Server Base URL not possible

How to reproduce:

Go to "Settings Advanced"
Set a URL like https://api.foo.bar/api/v1
Set a valid Account SID + API Key
Save
Change the API Server Base URL to https://api.foo.bar/v2
Save

Expected:
URL is being changed to https://api.foo.bar/v2
Actual:
The URL remains https://api.foo.bar/api/v1
Screencast

This issue is because Dialer will autocomplete the base URL base on current Jambonz Support API server. Currently Jambonz just support /v1
If we want to support any string can be base URL then we have 2 options:
1/ @davehorton agree on this and I will open new PR for that
2/ YOu can do your own build by changing this: https://github.com/jambonz/chrome-extension-dialer/blob/main/src/window/settings/advanced.tsx#L57C18-L57C30

@Catharsis68 what is this "v2" api you want to reach?


atomic-cli has subcommand for mysterious config file

It is unclear what atomic-cli config really does - what config file?!?

I discovered that subcommand by running atomic-cli --help (which in my Debian package is passed to help2man to generate a minimal man page).
Instead running atomic-cli config --help provides more detail, including even more mysteriously talking about "the resource to be destroyed" - destroyed?!?


Download Images when scrapping

App should download the images in a folder images when scrapping data.

Fixed with #14


Hide awkward DecoupledIO .bits attribute from user

Type of change:  other enhancement
Impact: API addition (no impact on existing code)
Development Phase:  implementation
Release Notes
Accessing the elements of a DecoupledIO(Bundle) is awkward from an hardware point of view because the ready/valid and the elements of the Bundle does not stand on the same level.
class MyModule(WIDTH: Int) extends Module {
    val flow = new Bundle {
      val data       = UInt(WIDTH.W)
      // ...
    }
    val io = IO(new Bundle {
      val in  = Flipped(Decoupled(flow))
      val out = Decoupled(flow)
      // ...
    })

    when(io.in.ready){
        io.out.valid     := io.in.valid
        io.out.bits.data := io.in.bits.data
    }
    io.in.ready := io.out.ready
}

Hardware engineers do not care about our syntactic sugars and implementation details, they want things to be straight accessible and not being hidden behind an irrelevant .bits attribute.
This PR enables the following access:
    when(io.in.ready){
        io.out.valid := io.in.valid
        io.out.data  := io.in.data
    }

Note0: renaming flow as bits in an attempt to bring some (actually misleading) consistency is not a solution since it will break as soon as you have 2 distinct DecoupledIOs.
Note1: The match is incomplete, I think it should raise a proper (not the java mismatch) error when  this.bits is not a Bundle.
Please share your thoughts =)

Can one of the admins verify this patch?

FYI The test failed for the following reason:
[info] .target on DecoupledIO
[info] - should not work *** FAILED ***
[info]   Expected a type error, but got none for code: Decoupled(UInt(8.W)).target (ImplicitConversionsSpec.scala:40)

Which goes with my question about handling the not Bundle case.

Flattening things in this way is going to create confusion when there are fields in the payload with the name 'valid' or 'ready'.  You might never do that, but it does happen, e.g., when making a Decoupled[Valid[_]].
I also have concerns about reducing the amount of static type safety we provide.  This change will move errors to elaboration time that are currently caught at compile time.  When writing generators, the errors might not even manifest in all elaborations.
I would recommend that, if you want to do this sort of thing, you introduce custom types in your designs, rather than changing the ones in chisel3.util.

I think there's also something to be said about consistency - since it's special case syntax that only works on Decoupled, a beginner (or anyone who isn't aware that this is a special case) might wonder why the same thing doesn't work across some different Bundle. Especially if one is looking at the methods overview generated by the ScalaDoc, there would be a bits, ready, and valid field and nothing else. And to understand why this works given a list of member methods, one now needs to understand the Scala Dynamic system on top of all the other Scala stuff needed to use Chisel effectively.
I do think you raise a valid point, but I these are not the trade-offs I would make. It may be worth thinking about how else boilerplate can be minimized within the constraints of the features Scala provides, and whether trade-offs (if any) can be acceptable.

Hi @edwardcwang @ducky64 @aswaterman and thank you for your feedbacks :)
I was not actually expecting this proposal to be merged but I rather wanted trigger a debate about the idea of flattening things at some point.
I fully understand your points and I agree that this would probably add even more confusion.

I would recommend that, if you want to do this sort of thing, you introduce custom types in your designs, rather than changing the ones in chisel3.util.

Yes, definitely.
I just want to bring 2 other points to the discussion.
First from the chisel3 cheatsheet (https://chisel.eecs.berkeley.edu/doc/chisel-cheatsheet3.pdf)

DecoupledIO is a Bundle with a ready-valid interface

I would tend to say this formulation is misleading: if it is a Bundle its elements should be accessible without having to explicitly refer to .bits.xxx.
It is indeed actually a bundle of 3 elements: ready, valid and bits but from the conception point of view it is a (Bundle of) Data with a ready/valid interface. Then one is free to put a bundle as Data.
Last but not least (I know it is not possible to change it by now but I wanted to raise the topic as well) why bits?
In our hardware vocabulary (which may not be shared far outside of the place I worked) bits would tend to refer to a number of  bit, pretty much like a width.
Indeed the verilog function $bits(XXX) returns the width of XXX.
I am glad that a quick search on ready/valid interface pointed me straight on this document, a first-year lecture from berkeley: https://inst.eecs.berkeley.edu/~cs150/Documents/Interfaces.pdf
As in many (all?) other places a decoupled interface is a bundle of 3 elements

ready
valid
data

So why bits and not value or val or data ?

So why bits and not value or val or data?

Fun fact, this artifact of Chisel DecoupledIOs actually harkens back to the primordial times when The Creators used to write Verilog, and the coding style involved writing decoupled interfaces with val, rdy, and bits.
For example: https://github.com/aswaterman/trainwreck/blob/7b47bb55b53faba3e1ab44209517c5ea8accd861/vector/vuVMU-Ctrl-vec.v
  input  [`VMCMD_SZ-1:0] vmu_vcmdq_bits,
  input                  vmu_vcmdq_val,
  output                 vmu_vcmdq_rdy,

  input  [`VMIMM_SZ-1:0] vmu_vbaseq_bits,
  input                  vmu_vbaseq_val,
  output                 vmu_vbaseq_rdy,
  etc.

As Andrew said, flattening is a complete no-go, since there are times where the bits payload is holding a token that may itself have a valid field.

Fun fact, this artifact of Chisel DecoupledIOs actually harkens back to the primordial times when The Creators used to write Verilog, and the coding style involved writing decoupled interfaces with val, rdy, and bits.
  input  [`VMCMD_SZ-1:0] vmu_vcmdq_bits,
  input                  vmu_vcmdq_val,
  output                 vmu_vcmdq_rdy,

IMHO dat would have made much more sense in this context ;)

One could create an alias method:
def dat: T = this.bits

but I'd argue this is just as confusing to a new user, who might expect it to be a different hardware field than bits (whose name, however confusing, is so widely used now it could never be changed)

There seems to agreement that this implementation is not what we want, so I'm closing this PR.
Feel free to PR other ideas, or open an issue to continue the discussion of the broader underlying problem.


Add version: WinFsp.WinFsp version 2.0

Pull request has been created with WinGet Automation :rocket:
Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/116817&drop=dogfoodAlpha

Reset PR


Prisma migrate

This PR is for managing DB schema with Prisma Migrate, as planned in #33.
Things to note:

npm run cli migrate is no more (I don't see much benefit in wrapping prisma migrate * in it)
I removed the read-only database access entirely; there's just one pool now instead of readWritePool. I think RO access was broken anyway since we changed the databases, and I think there were zero active users for it (right?); it can be restored if necessary, but there must be better ways to achieve this

one is our future GraphQL API
another possibility is to implement a custom job which dumps the entire database state in an archive and exposes it through HTTP or stores on S3

DB schema is kinda different from before

Prisma complained about some stuff, e.g. missing primary keys, so I had to change this; one example is that history table got a pk int autoincrement field
I also took the opportunity to change all fields to non-nulls (nulls in sql are confusing) and all json fields to jsonb

new DB structure will require one final manual migration (implemented in src/backend/migrate/prepareForPrisma.ts)
and also one prisma-based manual migration according to prisma's baselining docs

(I'll do both of these by hand when I'll merge this PR)

Looks good!


Update List docs

This PR updates List documentation for all packages in response to @dazuma's commentary on #1324.
Please comment on the proposed change for Log::List. However, DO NOT MERGE until commits for all packages have been added.

:+1: Looks good so far.

👍  LGTM

Waiting on an approval from @dazuma before accepting.


[Support,Feature] How to revert from pyramid tiles to complete dataset?

Dear @ndpyramid Team,
I am really happy to see such a nice code about pyramids in xarray. I have implement a very similar way to chunk netcdf on my own.
Actually I want to move from a netcdf4 store to a zarr store. For this purpose I need to revert the batching into the tiles.
I have opened a stackoverflow thread here.
I hope you can help me to find a proper way to revert the pyramid representation into one array.

Have you tried using Xarray's combine_by_coords function? This could be a natural way to stitch tiles back together.

@jhamman , combine_by_coords needs a fixed dimension to concatenate the arrays along.
I have found another way using the origin grid dimension and the same where statement to define the indices to write at. This works for me.
Thanks
Daniel


Add a raspberry pi article to the 'articles' section

Added an article relating to the creation of a car hacking tool using a raspberry pi.

Unfortunately, this article doesn't mention anything about vehicle hacking (though I know you can use Kali to do stuff with vehicles).


Quantity kind incomplete type

Hello,
I tried to get quantity kinds working from the Quantity Kinds documentation, however the compiler always complains about an incomplete type
Compiler-explorer: https://compiler-explorer.com/z/9PorMr417
#include <units/isq/si/length.h>
#include <units/kind.h>

using namespace units;
namespace si = units::isq::si;

struct radius : kind<radius, si::dim_length> {};

int main()
{
    using namespace si::references;

    quantity_kind<radius, si::kilometre, double> d(123 * km);
}

Error:
<source>: In function 'int main()':
<source>:13:51: error: variable 'units::quantity_kind<radius, units::isq::si::kilometre, double> d' has initializer but incomplete type
   13 |     quantity_kind<radius, si::kilometre, double> d(123 * km);
      |                                                   ^

Am I maybe missing some include?

Hi,
Thanks for trying out the library. Yes, you are missing a units/quantity_kind.h header file. See here: https://compiler-explorer.com/z/3Mz6eMdsE.

I guess I could have figured that one out from the glide_computer example. There units/quantity_point_kind.h is used which includes units/quantity_kind.h
Thank you very much for your fast answer

@mpusz, consider moving this issue to Discussions, just in case someone else has a similar problem.


internal/v1: Skip crypt() tests on macOS

this commit add checks in several tests to skip them when running on macOS (darwin)
using runtime.GOOS == "darwin"
this prevents the tests from failing on macOS since the crypt() function is not supported on this platform.

/retest