[BUG] Crash when selecting biome changer remote linked to a satellite in orbit

[Please fill out the form below and delete the sections in square brackets after reading them]
Version of Advanced Rocketry

1.12.2 2.0.0-17
1.12.2 2.0.0-13

Have you verified this is an issue in the latest unstable build

[x] N/A

Version of LibVulpes

1.12.2 <IP_ADDRESS>-universal

Version of Minecraft

1.12.2

Does this occur without other mods installed

[x] Y

Crash report or log or visualVM (if applicable)

https://pastebin.com/HP4XX47T

Description of the problem

Build a satellite with biome changer (my specific configuration was biome changer and 3 large solar panels)
You can still hover over the biome changer remote safely
Launch the satellite
Once satellite is in orbit, hover over the remote in your inventory
Game crashes
opening the temp.dat file in NBT Editor reveals the satellite is in fact in orbit
Switching to the hotbar slot the remote is in yielded mixed results, sometimes crashing, sometimes not
trying to use the remote to change the biome also crashed the game, but that should probably be its own report

yes this is so bad always happens

fixed in "Advanced Rocketry - Reworked"
(https://github.com/dercodeKoenig/AdvancedRocketry)


custom ui payment crashes the app

I got this error some times when I try to pay using custom ui
Thread 1: signal SIGABRT
An abort signal terminated the process. Such crashes often happen because of an uncaught exception or unrecoverable error or calling the abort() function.
Cast from '(any Error)?' to unrelated type 'String' always fails

ok I found something that If I send the price equals to 0


Debugging no longer possible after updating WSL to version <IP_ADDRESS>

Windows Version
Microsoft Windows [Version 10.0.22631.3155]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>-microsoft-standard-WSL2
Distro Version
Ubuntu 22.04
Other Software
Docker Desktop Windows  (v4.28.0)
WSL-Version: <IP_ADDRESS>
Kernelversion: <IP_ADDRESS>-1
WSLg-Version: 1.0.59
MSRDC-Version: 1.2.4677
Direct3D-Version: 1.611.1-81528511
DXCore-Version: 10.0.25131.1002-220531-1700.rs-onecore-base2-hyp
Windows-Version: 10.0.22631.3155
Visual Studio 2022 Professional (17.9.1)
Repro Steps

Update WSL-Version from 1.2.5 to 2.0.9 with wsl.exe --install
Create a new vs project (with docker compatibility)
select windows docker container
Add this to the docker arguments: --network="WSL"
start debugging
notice that the network is not found
Check which Hyper V switches exist and find the following: WSL (Hyper-V firewall)
customize docker arguments with the existing network
start debugging again

Expected Behavior
VS compiles the project and will start the debugging process.
Among other things, Docker uses the Hyper v virtual switch with the name "WSL".
Actual Behavior
The project is not compiled.
VS reports that the remote debugging service is not available and cancels the process.
The following switch appears in the Hyper v virtual switches list:
WSL (Hyper-V firewall)
VS gives the following error:

Microsoft Visual Studio
Debugging cannot be started.
Visual Studio Remote Debugger (MSVSMON.EXE) does not appear to be running on the remote computer. This may be because communication with the remote computer is prevented by a firewall. Information on configuring remote debugging can be found in the help.
OK
I have already switched off all firewalls. The behavior persists.
Even reinstalling my computer did not solve the problem.
Diagnostic Logs
No response

I have exactly the same issue and can't find a solution for it.

/logs

I think I have an idea what's going on here... When we introduced support for Hyper-V Firewall, the name of the network we create is now "WSL (Hyper-V firewall)".

Try passing that instead.


Improve calendar agent date-handling reliability

Changes:

Add fuzzy date parsing as a fallback if the translator LLM passes an absolute date as a parameter (i.e. passes a date that isn't relative to today)
Add "next week" as a recognized relative date range
Handle dayRange and timeRange parameters in the findEvent action

Reasoning:
Currently the calendar agent only succeeds if the translator LLM passes relative natural language dates like "tomorrow" or "next Tuesday" to the calendarActionHandler.ts.
When testing, sometimes the translator LLM translates "tomorrow" to a month/day format, like "December 2", and passes that to the calendarActionHandler.
The following user queries would result in invalid date parsing, before this PR:

User input
LLM-specified parameter
Previous result

"next Monday"
"December 2"
invalid parse

"tomorrow"
"11/30/24"
invalid parse

"next week"
"next week"
invalid parse

Changes look good to me. Thanks @ValerianClerc for the PR.


Parallel ops on the CPU?

Can someone point out the implementation to generate code to execute independent ops in parallel on the CPU?

tvm do not yet explicitly exploit task parallelism where two code path are parallelized, such parallelism is too high level and can be done at higher level of the engine(graph executor). We do support data parallelism, which can be done via parallel construct


How can we enable memory in the model ?

It doesn't seem to have a memory of the conversation.  Or does it ?

The bot can only see messages in the current reply chain. To continue a conversation you have to reply to the bot's message. This functionality is pretty powerful once you get used to it.

ah ok. so you reply to the message so it can reference it. nice.

Yes :) and you'll find it's quite reliable, e.g. you can:

restart the bot entirely and still continue any previous conversations
continue a super old conversation from months ago
etc.

It should always "just work".


Build should restart when typescript crashes and not die

*** Starting...
/Users/bpasero/Development/monaco/node_modules/gulp-tsb/lib/typescript/typescriptServices.js:35645
                throw e;
                      ^
TypeError: Cannot read property 'kind' of undefined
    at Object.nodeIsDecorated (/Users/bpasero/Development/monaco/node_modules/gulp-tsb/lib/typescript/typescriptServices.js:5708:21)
    at emitExpressionForPropertyName (/Users/bpasero/Development/monaco/node_modules/gulp-tsb/lib/typescript/typescriptServices.js:29648:28)
    at emitComputedPropertyName (/Users/bpasero/Development/monaco/node_modules/gulp-tsb/lib/typescript/typescriptServices.js:30183:17)
    at emitJavaScriptWorker (/Users/bpasero/Development/monaco/node_modules/gulp-tsb/lib/typescript/typescriptServices.js:34585:32)
    at emitNodeWithoutSourceMap (/Users/bpasero/Development/monaco/node_modules/gulp-tsb/lib/typescript/typescriptServices.js:34480:21)
    at emitNodeWithSourceMap (/Users/bpasero/Development/monaco/node_modules/gulp-tsb/lib/typescript/typescriptServices.js:29022:36)
    at emitNodeConsideringCommentsOption (/Users/bpasero/Development/monaco/node_modules/gulp-tsb/lib/typescript/typescriptServices.js:34472:21)
    at emitNodeWithCommentsAndWithSourcemap (/Users/bpasero/Development/monaco/node_modules/gulp-tsb/lib/typescript/typescriptServices.js:29036:21)
    at emitIndexedAccess (/Users/bpasero/Development/monaco/node_modules/gulp-tsb/lib/typescript/typescriptServices.js:30349:17)
    at emitJavaScriptWorker (/Users/bpasero/Development/monaco/node_modules/gulp-tsb/lib/typescript/typescriptServices.js:34589:32)

Actually, now that I think about it... won't fix. These are very rare and it's good they jump in our faces since it's an opportunity for us to communicate the issue back to TypeScript.


emacsPackages.emacs-conflict: init at 9f236b

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@jian-lin Thank you for the review. FYI I copy/pasted most of it from the sibling youtube-dl/default.nix file.


Techfab circuitboards should get lockbox recipes

Description

So it's easier to get more lathes if some are destroyed.

You could just access restrict the lathes.


load_model() function can not load training config

When I trying to save a model by model.save(), we found implementation is
def save(self, filepath, overwrite=True, include_optimizer=True):
so this function suggest that training configuration(optimizer, etc.) was saved in h5 file
But when I'm trying to load a model by load_model function（which default param of compile is True）:
def load_model(filepath, custom_objects=None, compile=True):
it warns:

/usr/local/lib/python3.5/dist-packages/keras/models.py:252: UserWarning: No training configuration found in save file: the model was not compiled. Compile it manually.
warnings.warn('No training configuration found in save file: '

so I cannot load my training configurations,I think this is a bug or I'm doing something wrong..
keras version 2.1.2 with tensorflow version 1.3.0

Did you try model.compile() before save it ?


README PEP8 Update

Updates the README file to match the recent changes made to the naming convention of methods.
In the examples, methods and variables now use the lower_case_with_underscores naming convention as opposed to UpperCamelCase, as was used previously.
Also removed one space and added an import for the library whenever it was used so the instructions are more consistent.
Update: Fixed some grammar in the README, there may still be more minor errors left. It would be safest to read through once and see if you can catch any I might have missed before this is merged.

Good work, this fixes #5 .


added simple intro to things that needed to be subdocs and improved categorisation to make gitbook easier to navigate

Thanks, merging <3


Gradle Sync Issue

Hi guys,
Following guide steps on Codelab, I cloned project and got issue when open in Android Studio. Please advise
Sync Error:

ERROR: The specified Gradle distribution 'https://services.gradle.org/distributions/gradle-5.1.1-all.zip' does not appear to contain a Gradle distribution.

Execute gradle.bat build command

Exception in thread "main" java.lang.NullPointerException
at org.gradle.wrapper.BootstrapMainStarter.findLauncherJar(BootstrapMainStarter.java:38)
at org.gradle.wrapper.BootstrapMainStarter.start(BootstrapMainStarter.java:26)
at org.gradle.wrapper.WrapperExecutor.execute(WrapperExecutor.java:108)
at org.gradle.wrapper.GradleWrapperMain.main(GradleWrapperMain.java:61)

Thanks,
Huey

Thanks for reporting this.
I'm closing this issue because I cannot reproduce it in this codelab.
You may have a corrupted download of the gradle-5.1.1-all.zip file. You may try re-downloading it or start from a fresh copy of this repository.
Please open a new issue if you still encounter this issue.


svg use on external sprite file

How would I use the generated SVG sprite by referencing the file directly as an external resource?
<svg>
	<use xlink:href="/static/sprite.svg#icon-html5-usage"/>
</svg>

This doesn't seem to render anything. I suppose the inlined CSS in the SVG sprite prevents it being used as an external resource.
Is there a solution to use it this way?

Basically I'm trying to use this method.

It's just not working in browsers at all with external sprite, you can try it yourself without sprite-loader. I think it's because <use> tag should refer to existing node in DOM (even it's a part of a shadow DOM).

Nowadays it works in most (modern) browsers. Even in Edge. (But not in IE.) Temporary demo.

There are also solutions to kinda polyfill it for non-supported browsers like svg4everybody and they also show an example of how to use external files.

Hmm, interesting. Looks like browsers supports only references to symbols itself. Use URL to symbol then - /static/sprite.svg#icon-html5 (without -usage).

Oh, true - it didn't work for me in the first place because I had a protocol-mismatch. What is the use-case for actually wrapping the symbol elements in a defs element and adding separate use elements within the generated SVG file? Wouldn't it just work right away by using the symbol elements directly?

For referencing from CSS - no, see related discussion.


Support RTL layouts

Hi there. Is it possible for you to support RTL layouts? I used Butter Slider and it was broken in a Right-to-Left direction.

Hi @Mehdi-Hp ! There is no option yet to support RTL, but it's on the roadmap for the V2 🔥 Not sure when it will be publish


Docs on set/get buffer size contradicting

The documention at https://pubsubclient.knolleary.net/api#setBufferSize refers to MQTT_MAX_MESSAGE_SIZE and that the default size is 256 bytes. Later on the same page https://pubsubclient.knolleary.net/api#setBufferSize it refers to MQTT_MAX_PACKET_SIZE with default size 128 bytes.
Looking at the source code the currently used define is calle MQTT_MAX_PACKET_SIZE and has a default size of 256 bytes.

I guess **MQTT_MAX_MESSAGE_SIZE ** is old used, and the doc not changed it.


net/http: Various tests flaky on Dragonfly builder

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
1.10
Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
DragonflyBSD amd64
What did you do?
The Dragonfly builder very sporadically fails on a few different tests in net/http. This is a recent example:
https://build.golang.org/log/25f7cc4010ecf00d0bdb9afb5aa6471ef12e7c9b

Some more examples:
2018-01-23T15:59:15-cafb36b/dragonfly-amd64
2018-02-14T15:51:46-eab06e6/dragonfly-amd64
2018-04-01T06:55:54-fdf3373/dragonfly-amd64
2018-04-05T23:14:14-3f483e6/dragonfly-amd64
2018-04-19T20:33:49-1c439e6/dragonfly-amd64
2018-05-31T18:21:53-462c182/dragonfly-amd64
2018-06-07T14:58:07-cf901a7/dragonfly-amd64

example of a failure on TestRetryRequestsOnError:
https://build.golang.org/log/acf2959a136589575226bb5b6345d7fd904db592

I don't know if this is related or not, but I saw this error today:
builder: dragonfly-amd64
rev: 72ce047a6ceae5490bb5e7d7cfb635463cdc6ea2
buildlet: http://dragonflybsd-amd64 reverse peer dragonflybsd-amd64/<IP_ADDRESS>:51023 for host type host-dragonfly-amd64-tdfbsd
started: 2018-06-13 16:23:26.554450402 +0000 UTC m=+495931.005849026
ended: 2018-06-13 16:35:46.033695842 +0000 UTC m=+496670.485094427
success: false
Events:
2018-06-13T16:23:26Z checking_for_snapshot
2018-06-13T16:23:26Z finish_checking_for_snapshot after 0s
2018-06-13T16:23:26Z get_buildlet
2018-06-13T16:23:26Z wait_static_builder host-dragonfly-amd64-tdfbsd
2018-06-13T16:23:26Z waiting_machine_in_use
2018-06-13T16:34:46Z finish_wait_static_builder after 11m19.2s; host-dragonfly-amd64-tdfbsd
2018-06-13T16:34:46Z clean_buildlet http://dragonflybsd-amd64 reverse peer dragonflybsd-amd64/<IP_ADDRESS>:51023 for host type host-dragonfly-amd64-tdfbsd
2018-06-13T16:34:46Z finish_clean_buildlet after 62.7ms; http://dragonflybsd-amd64 reverse peer dragonflybsd-amd64/<IP_ADDRESS>:51023 for host type host-dragonfly-amd64-tdfbsd
2018-06-13T16:34:46Z finish_get_buildlet after 11m19.5s
2018-06-13T16:34:46Z using_buildlet dragonflybsd-amd64
2018-06-13T16:34:46Z write_version_tar
2018-06-13T16:34:46Z get_source
2018-06-13T16:34:46Z finish_get_source after 0s
2018-06-13T16:34:46Z write_go_src_tar
2018-06-13T16:35:46Z finish_write_go_src_tar after 59.4s; err=writing tarball from Gerrit: Put http://dragonflybsd-amd64/writetgz?dir=go: net/http: HTTP/1.x transport connection broken: write tcp <IP_ADDRESS>:443-><IP_ADDRESS>:51023: use of closed network connection
Build log:
Error: writing tarball from Gerrit: Put http://dragonflybsd-amd64/writetgz?dir=go: net/http: HTTP/1.x transport connection broken: write tcp <IP_ADDRESS>:443-><IP_ADDRESS>:51023: use of closed network connection

The TestIdleConnChannelLeak failure mode appears to have been fixed in 2020.
My experience with the net/http tests is that they tend to have test-specific root causes, so I'm going to close out this “various tests” issue in favor of tracking failures in specific tests as we identify them.
greplogs --dashboard -md -l -e 'FAIL: TestIdleConnChannelLeak'
2020-01-09T19:50:42-10bfe5b/dragonfly-amd64
2019-12-06T01:11:02-a037582/dragonfly-amd64
2019-10-23T18:41:38-7833302/dragonfly-amd64
2019-07-03T14:12:22-1c1e517/dragonfly-amd64


Remove quotes in -var-list-children

lldb does not accept quotes by default and gdb does not required them.

:+1:

:+1:


Add L.Icon as a property of L.CircleMarker

Checklist

[X] I've searched through the plugins to make sure this feature isn't already available, or think it shouldn't require a plugin.
[X] I've searched through the current issues to make sure this feature hasn't been requested already.
[X] I agree to follow the Code of Conduct that this project adheres to.

Motivation
I'm always frustrated, when I'd like to display an object on a map, and:

I'd like to add an image to it (L.Icon supports this)
I'd like to dynamically change its border-color from javascript (L.CircleMarker supports this)

However, both cannot be done right now:

With L.Icon, you can only specify its border-color via css, so not dynamically from javascript.
L.CircleMarker does not have an icon property.

Suggested solution
L.CircleMarker should have an icon property.
Alternatives considered
Either stick to dynamic coloration of the border, or use image within.

Cannot do - a L.CircleMarker is a L.Path, drawn into either a L.Canvas or a L.SVG, whereas a L.Icon implies a HTMLImageElement. It's not possible to mix those.

With L.Icon, you can only specify its border-color via css, so not dynamically from javascript.

marker.getElement().style.borderColor = "red";


Exercise 60: mention new float type f80

As Zig 0.11 has added an f80 type, I thought it would be a good idea to mention it in the exercise.

@kimshrier Thank you!


Visual Studio >2012 builds

This issue was originally created at: 2016-04-20 23:59:25.
This issue was reported by: flow86.
flow86 said at 2016-04-20 23:59:25

Hi,
I ran into a problem with Visual Studio builds.
It seems that newer vcvars32.bat's need 2 Paths ( ProgramFiles and
ProgramFiles(x86) ) inside its environment, otherwise i.e "msbuild" does not
find the newer platform definitions
Index: SCons/Tool/MSCommon/common.py
===================================================================
--- SCons/Tool/MSCommon/common.py	(revision 10149)
+++ SCons/Tool/MSCommon/common.py	(revision 10150)
@@ -146,9 +146,13 @@
    # settings in vs.py.
    vars = [
        'COMSPEC',
+        'ProgramFiles',
+        'ProgramFiles(x86)',
# VS100 and VS110: Still set, but modern MSVC setup scripts will
# discard these if registry has values.  However Intel compiler setup
# script still requires these as of 2013/2014.
+        'VS140COMNTOOLS',
+        'VS120COMNTOOLS',
        'VS110COMNTOOLS',
        'VS100COMNTOOLS',
        'VS90COMNTOOLS',

bdbaddog said at 2016-04-21 08:52:00

Thanks for your bug report.
However we request that users bring their issues to the users mailing list to
confirm that it is a new bug and also that it's not user error.
http://scons.org/bugs.html
In addition we prefer pull requests on bitbucket.org
(see https://bitbucket.org/scons/scons/wiki/DeveloperGuide )
Also if you can create a simple testcase for each issue you've filed to put in
your pull request along with your patch that would be ideal.

I think this is issue can be closed. SCons has similar but different code in this file currently, and seems to work out of the box with Visual Studio 2015.

VS up to 2017 builds work with current 3.0.1 scons. Closing.


Client Allocation Request for: CoinSummer

Client Allocation Questions
Core Information

Name: CoinSummer
Website / Social Media: https://coinsummer.io
DataCap Requested: 5TiB
Addresses to be Notarized: f3s47wuxlqofvu6cnhucyiwa4zmw4cmg43g263mdbam32fvmrdq6aq6bt27jrmownkkckyz4p4ljiy2lrpgcbq
Notary Requested: philippbanhardt

Hey @yuwenhui - thanks for your request. Can you kindly add more information / context here on:

yourself , e.g. include links to social media / news mentions etc
your history working on IPFS / Filecoin, e.g. notable contributions to the community, type of engagement shown over time etc
type, size and source of the data you (and by extension other Discover participants) are looking to store on Filecoin, e.g. public data vs private data etc
how you intend to store the data on Filecoin, e.g. redundancy, location / miner preferences, etc

Thanks

Hey @philippbanhardt - here are our information:
Question1:
Wenhui Yu, co-funder of CoinSummer Lab. I have been a Web Developer and Product Manager for about 6 years, and worked on image recognition use deep leaning tow years ago. I am focus on Filecoin and Polkadot develop and research currently.
Social Media: Linkedin
Question2:
I and our team created and maintaining Filecoin Chinese technology WeChat group with more than 1k miners and developers. There are tens of thoursands of messages every day including hardware configuration, mining tutorial, troubleshooting, development sharing, bug trace and so on. We believe that these are the most active and useful technical community for Filecoin ecosystem.
I also provide proof parameters(v25, v26, v27) and Filecoin blockchain synchronization datastore backup mirrors on Mainland China from the Filecoin TestNet. More than 200 miners use our download service for free.

CoinSummer Filecoin proofs parameters mirrors
Blockchain synchronization datastore backup

Additionally, we organized and collected awesome Filecoin resources repo for Chinese commuity. Including common resource links, lotus node operations, common storage miner operations, seal worker operations, deal operations, environment variables, debugging, benchmarks, frequently asked questions, etc.
Question3:
Public data such as:
Openimages: a dataset of ~9 million URLs to images that have been annotated with image-level labels and bounding boxes spanning thousands of classes.
ImageNet: an image database organized according to the WordNet hierarchy.
There will be more than 10TiB of all these dataset with .tar format.
Question4:
Firstly, we store these dataset on Filecoin for redundancy, additionally, we are planning to build a decentralized image dataset which everyone can use to train their own Machine-learning and AI models. We are developing more feature to using Filecoin and IPFS as the storage solution.

Hey @yuwenhui thanks for providing the additional information. Are you OK to post here transaction links to the storage deals, once awarded data cap? Also on a related note - wanted to highlight that we are keen to foster decentralization of the network, so would encourage clients to store (redundantly) across a number of miners, as well as refrain from self-dealing as much as possible.

@yuwenhui quick nudge on the above

@philippbanhardt  Yes, i will store my data file to a number of miners for decentralization and post all the transaction links of those deals to here after i granted the data cap.


Added a fallback boringssl rule in build.yaml

Internal build environment doesn't have boringssl at the expected location so it fails to generate build files from templates. It's mostly benign not to have build files internally but it shows error when importing it, which is bad. This helps removing errors when generating build files while importing.

I'm afraid I don't have enough context to review this - can @nicolasnoble  review instead?


feat: make heap stronger

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[x] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] It's submitted to the dev branch, not the main branch
[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] All tests are passing
[x] New/updated tests are included

If adding a new feature, the PR's description includes:

[x] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:
See https://github.com/js-sdsl/js-sdsl/issues/51.

Coverage decreased (-0.1%) to 99.853% when pulling 8fff45fd20e36092512891908b3d7220ef850792 on feat/make_heap_stronger into 7bed7bd134882c990fee578b3f4a91e970e0c11a on dev.


Issues when upgrading to Bazel 5

Seeing some strange behavior when upgrading to Bazel 5 for a mid-sized Scala project I'm looking at.
The first issue we worked through was missing classes, and appeared after moving from bazel 4.2 and bazel 5; we  resolved this by adding the missing dependency directly to the failing scala_library in question, so it's a trivially easy fix, but I'm not sure exactly why the behavior changed just in bumping the bazel version :thinking: but I'm a little rusty on how the dependency mode options in the scala toolchain interact with Bazel, could be there were changes there I guess?
Second issue that I find even more perplexing (redacted with foo, this is a real project):
ERROR: file 'foo/it.runfiles/MANIFEST' is generated by these conflicting actions:
Label: //foo:it
RuleClass: scala_test rule
Configuration: fb3f630c9664150b7a5df27d2d6586a50d90442e5149d0267894e093602b1d77, a5e5a51a25dac85941e5a079e772850bacd3ab2905204bf78bdc62c76d905fec
Mnemonic: SymlinkTree
Action key: 3f5eff7b31775ca3d96a53c213cba4dcb22aea3b09155835e560871a66c79c31, adf6238ad9f28306822b15250551f05b4ccf4d21e476410e8f59a5627aca3ca4
Progress message: Creating runfiles tree bazel-out/k8-fastbuild/bin/foo/it.runfiles
PrimaryInput: File:[[<execution_root>]bazel-out/k8-fastbuild/bin]foo/it.runfiles_manifest
PrimaryOutput: File:[[<execution_root>]bazel-out/k8-fastbuild/bin]foo/it.runfiles/MANIFEST
Owner information: ConfiguredTargetKey{label=//foo:it, config=BuildConfigurationValue.Key[fb3f630c9664150b7a5df27d2d6586a50d90442e5149d0267894e093602b1d77]}, ConfiguredTargetKey{label=//foo:it, config=BuildConfigurationValue.Key[a5e5a51a25dac85941e5a079e772850bacd3ab2905204bf78bdc62c76d905fec]}
MandatoryInputs: are equal
Outputs: are equal
....

:point_up: dozens of these kinds of things. Diffing the respective configs, they're identical other than the following which is present in only one of the configs, and maybe the culprit:
diff <(bazel config 96f337ca1992d337c541dd7d730d4727cf7fe642e811ddb77c51a8069b4f9e34) <(bazel config af0d32aeca223632866b193bd65f1c19bae8377f6d9f0ca2a508907a49227b6d)
INFO: Displaying config with id af0d32aeca223632866b193bd65f1c19bae8377f6d9f0ca2a508907a49227b6d
INFO: Displaying config with id 96f337ca1992d337c541dd7d730d4727cf7fe642e811ddb77c51a8069b4f9e34
1,2c1,2
< BuildConfiguration 96f337ca1992d337c541dd7d730d4727cf7fe642e811ddb77c51a8069b4f9e34:
< Skyframe Key: BuildConfigurationValue.Key[96f337ca1992d337c541dd7d730d4727cf7fe642e811ddb77c51a8069b4f9e34]
---
> BuildConfiguration af0d32aeca223632866b193bd65f1c19bae8377f6d9f0ca2a508907a49227b6d:
> Skyframe Key: BuildConfigurationValue.Key[af0d32aeca223632866b193bd65f1c19bae8377f6d9f0ca2a508907a49227b6d]
77a78,97
> }
> FragmentOptions com.google.devtools.build.lib.analysis.test.TestConfiguration$TestOptions {
>   cache_test_results: AUTO
>   coverage_report_generator: @bazel_tools//tools/test:coverage_report_generator
>   coverage_support: @bazel_tools//tools/test:coverage_support
>   experimental_cancel_concurrent_tests: false
>   experimental_fetch_all_coverage_outputs: false
>   experimental_persistent_test_runner: false
>   experimental_retain_test_configuration_across_testonly: false
>   experimental_split_coverage_postprocessing: false
>   incompatible_exclusive_test_sandboxed: false
>   runs_per_test: [(?:(?>.*)) Options: [1]]
>   runs_per_test_detects_flakes: false
>   test_arg: []
>   test_filter: null
>   test_result_expiration: -1
>   test_runner_fail_fast: false
>   test_sharding_strategy: EXPLICIT
>   test_timeout: {short=PT1M, moderate=PT5M, long=PT15M, eternal=PT1H}
>   trim_test_configuration: true

I'm not sure exactly what to make of it thus far :thinking:

A minor update:
The badly behaving targets here turned out to be integration tests which depended on other integration test targets, in an attempt to make use of shared utility code which should have been factored out to common libraries. Interestingly this seemingly worked in Bazel 4 and earlier, but fails in 5 with the colliding actions.
The simple answer here is that the offending project code should be fixed, so maybe this is a non-issue, but that said, I wonder if it makes sense to look into and/or document the change in behavior at all?

@zachgrayio in Bazel 5, --trim_test_configuration was changed to true, see if setting it to false helps with your issue

@zachgrayio in Bazel 5, --trim_test_configuration was changed to true, see if setting it to false helps with your issue

thats gotta be it, yeah. was lookin for something like this in the patch notes but failed at grep i guess :laughing:

If it helped, it probably means that there's bug Rules Scala or Bazel 5

My team actually hit this exact issue as well.  Adding that flag allowed us to proceed with further testing on Bazel 5.

So following up here, yes, that flag alone is enough to avoid the issue here in this build.
That said I'm not convinced this is evidence a bug per se; as I said, the root of the issue is that I discovered tests depending on other tests in this project, and this is what's driving the colliding actions. I assume that after bazel 5 flipped this flag, one of the redundant configs is getting trimmed enough that it ends up conflicting rather than being identical, like it was under bazel 4.
The takeaway here for me is that maybe this shouldn't have ever worked in the first place and the behavior now is probably more sensible.
FWIW, fixing this correctly really wasn't that hard, and if you're hitting this issue it's likely you need to take another look at your build graph and fix the root cause.

Is Bazel 5 officially supported? It's not yet mentioned in the compatibility table.

That said I'm not convinced this is evidence a bug per se; as I said, the root of the issue is that I discovered tests depending on other tests in this project

This isn't actually the root cause, after I dug into this a bit.  I think the root cause is non-tests depending on test libraries.  I.e., if you had some targets like this, it would fail:
scala_library(
    name = "mytarget",
    deps = [ ":tests" ],
)

scala_test(
    name = "tests",
)

https://bazel.build/reference/command-line-reference#flag--trim_test_configuration

When this flag is active, tests cannot be built as dependencies of non-test rules


[Question] Why does scales have 2 dimensions?

From the paper, I'd expect scales to be a set of scalar values.
But, in practice, the following assertion seems to be true:
assert scales.shape[1] * group_size == linear.in_features

around here:
https://github.com/mit-han-lab/llm-awq/blob/f0b4b68004f76d562658143cddea5aad8c1b8266/awq/quantize/qmodule.py#L68

I"m silly -- from the paper, scales should not be a set of scalar values. Since, we're scaling weights.


Document Grafana dashboard usage

This should include how to install and configure the dashboard.

First draft: https://gist.github.com/Alan-Cha/d82bb0124d50ae3d7a142dec992a7924

The documentation will be incorporated into the tutorials. See https://github.com/iter8-tools/iter8/issues/1513


Fixing the FCM categories

The new FCM categories now align with what's in the design manual.
Fixes #1337
Testing
Create a sublanding page and check out the new categories in the Featured Content Module
Preview

Review

@kave
@kurtw
@richaagarwal

This all looks great!


File_Mpeg4v::video_object_layer_start()

//Coherancy
if (shape!=2 && shape==0 && (object_layer_width==0 || object_layer_height==0 || ((float32)object_layer_width)/object_layer_height<((float32)0.1) || object_layer_width/object_layer_height>10))
Trusted_IsNot("Problem with width and height!");
Remove shape!=2 ?

Should be removed


Update MD file on new branch 2017-11-15_20-24-23

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[test] Remove external URLs from tests.

Some test were using external URLs. External URLs are not ideal for
testing for serveral reasons: they can change without warning, making
the test start failing, they take longer to resolve, and finally they
will not work if the test machine do not have internal access for
security reasons.
The commit modifies those URL to hit the in-memory HTTP server, setting
each request with a 5 second pause to allow to check the needed
behaviours of pausing and resuming.
/cc @pushkarnk @karthikkeyan

@swift-ci please test Linux platform

+1!
@drodriguez I wonder is there a way where we could ensure that if we aren't going to <IP_ADDRESS> in a test process, it crashes. (Maybe some form of sandboxing, not sure). Then we can be sure these never sneak in /ever/ again. = ).

Thanks for make this changes @drodriguez. I wasn't aware of the constraints around external URL, I will be conscious of this henceforth.

Thanks @drodriguez I agree. While I was aware of this unwritten rule, I must admit I ignored it while reviewing those tests!

@swift-ci please test

@gottesmm: In Linux one can use unshare to do something like you propose (ref: http://manpages.ubuntu.com/manpages/bionic/man1/unshare.1.html). It might be nice to run the test under that, but one needs to also open loopback again, and I don’t know if a normal user can do all that. If someone has some time, it might be worth looking into it. There's another test in TestHost.swift that resolves a real URL, and I haven’t been able to find out if it should work without connection or not.
I will review the error in CI (seems that the socket is used after being closed).

@swift-ci please test

@swift-ci please test

Rebased on top of current master. Can I have a thumbs up/approval in this PR if there's no more feedback?
Might fail because of the fix in #2428.
@swift-ci please test Linux platform

The failure was because of the missing PR. Re-testing since that PR has been merged.
@swift-ci please test Linux platform


OS-X: Building from source

Hi:
I was wondering if there are any special switches necessary to generate configure etc. I do have automake installed from ports and brew, but I get errors when trying to run automake.
Thanks,
-Clint

First of all you are aware that you can compile and install from the latest master branch using brew install sile --HEAD right? Just checking.
That being said if you want to compile from your own source location there shouldn't be much you need to specify unless you have non-standard paths for things like the Lua interpreter. You might have a look at the homebrew formula for ideas. Specifically the install block on L28–37 show the sequence of commands run to install from source.  Also make sure you have the prerequisites listed there.
If that doesn't help, let us know more specifically what errors you are seeing.

No, I didn't know about that. Is this written somewhere in the docs?
 I don't think this is going to be easily fixable. It suggested I do:
checking for required Lua library zlib... not found
configure: error: cannot find Lua library zlib - install from luarocks package lua-zlib

READ THIS: http://docs.brew.sh/Troubleshooting.html

Error: You have MacPorts or Fink installed:
  /opt/local/bin/port

This can cause trouble. You don't have to uninstall them, but you may want to
temporarily move them out of the way, e.g.

  sudo mv /opt/local ~/macports

I'm relatively new to brew, so I'm not about to wipe my MacPorts to enable this.
I'll try going through the recipe to see if it's easy to replicate.

On that case dispense with brew and just do it from source. You'll just need the right prerequisites and path to Lua and friends. There should be enough information in that formula to clue you in on how to run ./configure.
The --HEAD thing is a standard way of using brew to install from source master branches rather than the last release version. It's documented as part of Homebrew and I believe it's mentioned in the SILE docs or somewhere in previous issues on this. However I haven't been pushing the use of Homebrew much in the documents because they still haven't really fixed the Lua dependency issues which it doesn't handle for you as could be expected.
We could probably setup a Macports package as well. I've never used Macports and the old iMac I have hanging around  for testing is currently dead so I can't really do this right now but I can't image it would be that difficult to setup a package for that system. Who knows, it might even handle the Lua issues better than Homebrew.

I would love to dispense with brew. How can I find out where it's building things and how to see the error messages? These opaque systems are really annoying. I tried running install --debug and I can't even get a directory name where I can find this stuff being built.

OK, I see that if I exit to shell, I can examine the build contents. Digging...

I was able to get brew to work. For whatever reason I didn't have the lua-zlib package installed under luarocks.
$ which sile
/usr/local/bin/sile
$ sile -v
This is SILE 0.9.5-unreleased

! Could not find file at -v

That looks good. We should probably support a -v flag. :-)


Type Iterator#next() argument (yield expression)

Search Terms
Iterator.next() type, yield expression type
Suggestion
Type Iterator.next()/yield. I suggest Iterator<A, B>, where A is the input type, and B is the output type.
Mentioned in #10148 and #14883.
Required by #32283.
Alternative proposal in #32523, those this requires more drastic changes to TypeScript's type model.
Use Cases
Pretty self-explanatory; I want typed expressions. It's why I use TypeScript.
And currently, all yield ... expressions have no typing. (They are typed as any).
Examples
function* totalLength() {
  let result = 0;
  while (true) {
    result += (yield result).length;
}

const t = totalLength();
t.next('abc'); // 0
t.next('def'); // 3
t.return(''); // 6

Checklist
My suggestion meets these guidelines:

[X] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[X] This wouldn't change the runtime behavior of existing JavaScript code
[X] This could be implemented without emitting different JS based on the types of the expressions
[X] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[X] This feature would agree with the rest of TypeScript's Design Goals.

And actually, Iterator has additional problems, like return?(value?: any): IteratorResult<T> which is just simply wrong.
To be more concrete, the types should be
interface Iterator<T> {
    next(): IteratorResult<T>;
}

interface Generator<A, B> extends Iterator<B> {
    next(value?: A): IteratorResult<B>;
    return<T>(value?: T): IteratorResult<T>;
    throw(e?: any): IteratorResult<B>;
}

The optional argument is problematic for having non-undefined values inside the generator, but I don't have a better suggestion.

And currently, all yield ... expressions have no typing. (They are typed as any).

I think you should take a look at the TS 3.6 release notes
https://devblogs.microsoft.com/typescript/announcing-typescript-3-6-rc/

See #30790 and its linked issues

Excellent, thanks @AlCalzone for linking to the 3.6 RC notes.
@typescript-bot I think you might not be working correclty.


Patch over buggy fixRange behavior

This closes #26. Unlike the other three PRs I just filed, this one is kinda hacky. :stuck_out_tongue_winking_eye:
I don't fully understand fixRange, other than to see that it's trying to patch over problems in coffeescript redux. But I can see that it is failing due to off-by-one-or-two errors, so this patches over the problem by doing a little search for the correct piece of original sourcecode.
This was a necessary bandaid so I could make progress for now. But others may also find it helpful.

Thanks! Released in v1.16.0.


nested table not beautified correctly

Description
Nested tables are not correctly beautified.
This is the minimum code to reproduce this issue, you have to make at least one column in the inner-table, otherwise the program will run well.
I don't know if it's ok to use table in a table. But I was writing a spring mvc thymeleaf template, I wanted to render table modals in each outer-table line. I was using vim-autoformat plugin, and I have tested this issue in command line as well, both reproduced.
Input
The code looked like this before beautification:
<!DOCTYPE html>

Expected Output
The code should have looked like this after beautification:
<!DOCTYPE html>

Actual Output
The  code actually looked like this after beautification:
<!DOCTYPE html>

Steps to Reproduce
Environment
OS: Ubuntu 18.04 LTS
with default settings.

Technically, it looks like the thead must have a tr inside it:
https://www.w3.org/TR/html5/tabular-data.html#elementdef-thead
That said, it shouldn't unwind that far if one is missing.


Replace MOVE-CORRESPONDING

First of all - Great Job for what you have so far! I really appreciate it. Here is one idea from me:
according to the new features in ABAP syntax MOVE-CORRESPONDING should be replaced with the following:
For table:
lt_adrt = CORRESPONDING #( lt_adrt ).
For structures:
ls_result = CORRESPONDING #( lt_t005k_result[ 1 ] ).
Thank you!

Hi Pavel,
thanks for opening this issue! This was indeed already proposed in #36, but it is nevertheless good to have a dedicated issue for possible discussions.
As you can see in the ABAP documentation chapter on CORRESPONDING, Basic Form, Addition ... BASE ( base ) ..., MOVE-CORRESPONDING would more precisely have to be replaced in the following way:
" possible replacement for structures
MOVE-CORRESPONDING struct1 TO struct2.
struct2 = CORRESPONDING #( BASE ( struct2 ) struct1 ).

" possible replacement for internal tables
MOVE-CORRESPONDING itab1 TO itab2 KEEPING TARGET LINES.
itab2 = CORRESPONDING #( BASE ( itab2 ) itab1 ).

Without BASE, the result would be significantly different, because for structures, MOVE-CORRESPONDING only overwrites components that appear in both structures, but keeps all others unchanged, while the CORRESPONDING # constructor expression would return initial values for all components that are not found in struct1.
This makes me hesitate a bit, because with BASE, the statement actually looks more complex and less readable. What's more important, though, is that ABAP cleaner might not be able to determine in all cases whether the variables represent structures or tables (obviously we can't rely on naming conventions, and the definition of the variables might be out of sight for ABAP cleaner). Therefore, if you find
MOVE-CORRESPONDING a1 TO a2.

you could only change it into
a2 = CORRESPONDING #( BASE ( a2 ) a1 ).

if you are sure that a1 and a2 are structures. If they were table, this would change the behavior of the statement, because the original statement didn't have KEEPING TARGET LINES.
Therefore, not entirely sure yet whether (or under what exact conditions) this would be possible.
Kind regards,
Jörg-Michael

How about having an option to only execute the rule if the target variable wasn't set beforehand? For all other cases, I would prefer to keep it as MOVE-CORRESPONDING.

Hi ConjuringCoffee,
yes, that might work – or if the target variable was just CLEARed in the previous line.
Kind regards,
Jörg-Michael


[PHP8] Declare sfDoctrineFormGenerator $params property

The sfDoctrineFormGenerator class stores a $params property that is not declared in the class, leading to a deprecation notice:

PHP Deprecated:  Creation of dynamic property sfDoctrineFormGenerator::$params is deprecated in /path/to/friendsofsymfony1/symfony1/lib/plugins/sfDoctrinePlugin/lib/generator/sfDoctrineFormGenerator.class.php on line 61

This PR simply declares the class property to satisfy the deprecation notice.

Thanks @connorhu for merging
Please let check to not have commits like "Merge branch 'master' into fix/sfDoctrineFormGenerator", in the history: this should be replaced by rebasing the branch on "master"

I see, sorry. I will do that from now.


Emulator doesn't enable the user to select/enter authentication information when using oAuthPrompts

When using the Emulator to test a Bot that leverages oAuthPrompts it would appear the Emulator (and likely Chromium) is caching prior authentication across instances of the emulator and even reboots of the PC.
As the attached video shows, this means that when you click a sign-in link you get a popup window which would normally show authentication information (e.g. type email address) which quickly disappears having taken cached authentication from before and returns a token. In my case for my MSA not my domain credentials which are needed for my scenario.
You get similar behaviour with browsers/edge and InPrivate mode works well to work around this. We should perhaps look at removing cached state from Chromium after Emulator has closed or have a button somewhere to do this?
Bot Framework Emulator (V4 PREVIEW) - CalendarSkill 2018-08-15 14-42-34.zip

@darrenj @sgellock  - With the completion of the AAD -> LUIS flows, we have a requirement to cache previous sessions when the user chooses "Keep me signed in". This means allowing electron to cache their session.  If the user does not wish to remain signed in, this code signs the user out when the emulator is closed and causes the user to be prompted for credentials on the next auth attempt.
Also, once the user is signed in, they can sign out using the file menu.
I believe this should address the issue you're seeing. Since this code was pushed yesterday, it might be a good idea to get you a fresh binary to begin banging on.

@darrenj assumed resolved.  if not, please reopen.

hi there @sgellock , it appears to me that v4.0.41504 is not allowing OAuthPrompt to trigger.
Perhaps we can reopen this item?

Yes.  Let’s do that.  Can you re-open the ticket please?
From: Zac<EMAIL_ADDRESS>Reply-To: Microsoft/BotFramework-Emulator<EMAIL_ADDRESS>Date: Tuesday, October 16, 2018 at 5:10 PM
To: Microsoft/BotFramework-Emulator<EMAIL_ADDRESS>Cc: Scott Gellock<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/BotFramework-Emulator] Emulator doesn't enable the user to select/enter authentication information when using oAuthPrompts (#768)
hi there @sgellockhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fsgellock&data=02|01|scott.gellock%40microsoft.com|3cda8051dac04943b7ee08d633c50216|72f988bf86f141af91ab2d7cd011db47|1|0|636753318568764155&sdata=M2eNrVU2xKkQLemjDKRTD8AejNIiw6YpBF0oXK%2F2Ogs%3D&reserved=0 , it appears to me that v4.0.41504 is not allowing OAuthPrompt to trigger.
Perhaps we can reopen this item?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2FBotFramework-Emulator%2Fissues%2F768%23issuecomment-430444444&data=02|01|scott.gellock%40microsoft.com|3cda8051dac04943b7ee08d633c50216|72f988bf86f141af91ab2d7cd011db47|1|0|636753318568774168&sdata=okeCDXe2E7DSG%2Biltix3dCz77yd8ifpvczFmqqP52zc%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAsxd7ckvpIsuhkUlykETU8zZN2KuHlhks5ulnWPgaJpZM4V-ItF&data=02|01|scott.gellock%40microsoft.com|3cda8051dac04943b7ee08d633c50216|72f988bf86f141af91ab2d7cd011db47|1|0|636753318568774168&sdata=OmsTUFg7iooOsjle%2B7QTukry1H3m9POb3s%2F5X5d0iMk%3D&reserved=0.


Adding upcoming honeypot DApp

Honeypot is a DApp designed to challenge the security of Cartesi Rollups. Bug hunters are incentivized to hack the application to obtain the funds locked in the rollup contract. Honeypot holds real assets with a dual objective: setting a financial benchmark for secure asset management and providing a gamified battlefield for the community to help audit and test Cartesi Rollups.
More info here

Hey @claudioantonio,
what is the state of this PR? I see some TODOs, can I help you somehow?

Hey @claudioantonio, what is the state of this PR? I see some TODOs, can I help you somehow?

Hi @antooni ! These TODOs are related to info that will only have when the DApp is deployed on mainnet. For example, the smart contract address, the github repo link, etc.
The idea would be to update the honeypot.ts file right after the Mainnet deployment.
Can I keep it? Or do you prefer that I remove these comments?

@antooni

TBH @claudioantonio I am not sure about this one, you filled purpose with "bug bounty", can you provide some reasoning behind this?

Sure @antooni ! You can think of the Honeypot DApp as a kind of game/challenge for developers. 😄 There will be some amount of DAI (that will increase over time) locked in the DApp and the developers have to find a way to transfer these DAIs from the DApp (break the pot). It is a challenge for developers and a security test for Cartesi's Rollups solution.
Do you have a suggestion for a better purpose that we should use?

why should we list it on L2BEAT?

The Honeypot is a DApp developed using a Layer-2 solution (more specifically an application-specific rollups) which has its security derived from L1 and has some value locked in it.

what technologies are there,

As mentioned above the honeypot was built using the Cartesi Rollups SDK which is an application-specific rollups with a Linux runtime and Ethereum's security guarantees. You can check the Cartesi Rollups thesis here.
At the heart of the Cartesi Rollups solution there is the Cartesi Machine which is a real Linux runtime environment. It allows developers to use the programming languages and tools of their choice to write the code for the logic of their DApp's. You can read more about the Cartesi Machine here.
Cartesi Rollups official examples written in Python, JavaScript/Typescript, Rust and C++ can be found here.

what are the smart contracts, what is the desired architecture?

I will create a diagram to show the solution design. Just for my planning, is it a requirement for the DApp to appear as an upcoming project?
Please let me know any other question that you have, ok?

Hi @antooni !
As promised here comes a diagram that shows the overall solution design for the Honeypot DApp and its underlying technology (Cartesi Application-specific Rollups).
Some observations:

Inside the Ethereum network, the orange boxes are the Cartesi Rollups contracts that are common to all DApps built using Cartesi Rollups and the black box is the specific instance for the Honeypot DApp.
You can see the DApp owner/deployer creating the Honeypot DApp instance and depositing the DAIs to the pot.
Cartesi will initially provide a execution infrastructure where the validator node for the Honeypot DApp will run. This validator node is a specific instance for the Honeypot DApp with it own Cartesi Machine (Linux runtime where the Honeypot logic will run). The validator is responsible to save the rollups state for the DApp at the end of each epoch and also to make claim in case of disputes.

Please take a look and don't hesitate to make any question you need, ok?

Hey Claudio, pls have a look at my discord dm, I would like to have a chat
about HoneyPot
El lun., 24 abr. 2023 19:21, Cláudio Silva @.***>
escribió:

Hi @antooni https://github.com/antooni @luke7211
https://github.com/luke7211 @michalsidzej
https://github.com/michalsidzej @torztomasz
https://github.com/torztomasz , is there anything else from my side
that I can do/provide for you?
Should I create an listing request entry on L2Beat governance's site?
—
Reply to this email directly, view it on GitHub
https://github.com/l2beat/l2beat/pull/1391#issuecomment-1520555298, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHIAAFRIIGSYYOLPA3FBTADXC2ZDHANCNFSM6AAAAAAW3UJRDM
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey Claudio, pls have a look at my discord dm, I would like to have a chat about HoneyPot

Thanks @luke7211 ! Already answered there.

Commit ab5f02 adds Cartesi provider icon in the technology column of the project's table. See screenshot below.


proposing corrections for week01 seminar

Just finished the week01 of the blog course. This is awesome! I'm learning a lot - although I have to say I'm skipping most of the complex formulas, not my string point -.
In any case, found a few modifications that I feel could be of use. Hope they are of help, I'll continue with the week02 these next days.

size is now vector_size in Word2Vec
added print for model save path on gensim's api.load
corrected PCA's bold initials, since asteriks seem to be best practice and some viewers don't work with underscores

Hi! Sorry for the late response. The week1 changes are perfect, but some of the changes you propose to week3  include solutions, e.g.

If you modify the code to exclude your solution, i'd be happy to merge

Oh that's bad, it seems the commit for my solutions got mixed up in the
pull request.. Is there any way you can use the corrections for week01? If
not I'll try to make them on a separate branch.
Btw, now that I am about a half into the course I have to tell you it is
amazing what you've done. I'm thinking of applying to the online courses at
Yandex at some point.
Thank you for your response, hope we can be in touch.
Juanma
El jue, 21 sep. 2023 12:20, justheuristic @.***>
escribió:

Hi! Sorry for the late response. The week1 changes are perfect, but some
of the changes you propose to week3 include solutions, e.g.
[image: image]
https://user-images.githubusercontent.com/3491902/269667056-a4da891d-4f3e-4ae3-8f0f-d6f361986a3e.png
If you modify the code to exclude your solution, i'd be happy to merge
—
Reply to this email directly, view it on GitHub
https://github.com/yandexdataschool/nlp_course/pull/117#issuecomment-1729800779,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALUJ5APW7HUIDO2LQCO2HTTX3RLMNANCNFSM6AAAAAA32YK6F4
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi, @lopezjuanma96
to save your work locally you may want to create a new branch from the latest commit here.

For additional contributions please use branch 2023
note to ourselves: pick this PR result into 2023 branch.

I had a separate branch for solutions but it would seem i messed up and push some commits with solutions on main branch.
Okey, I'll try my best to do so, I can't promise I'll be successful in my first try, but I'll tell you when I'm done and we can recheck.

replaced with #151


Create VS project to implement the P2P communication engine.

Scope:

Use Standard .NET framework.
Add https://github.com/lhartikk/naivechain java script to be a starting point for further development.

Unt test project must be added at this initial point.


Update MaxPool2D to Support Width and Block Sharding

Ticket
#13077
#13078
Problem description
This PR updates MaxPool2D to support both width and block sharding.
What's changed
A default parameter applied_shard_scheme has been added to ttnn.max_pool2d.  When not specified, MaxPool2D will use height sharding.  But, if either width or block sharding is specified, MaxPool2D will use that shard scheme instead.
This was primarily achieved by:

scaling the references to the channel dimensions of the activation matrix in the program factory by the number of shards in the channel dimension
adjusting the sliding window config calculation to pad the top left index shards to ensure all cores' shards have uniform length

Checklist

[ ] Post commit CI passes
[N/A] Blackhole Post commit (if applicable)
[N/A] Model regression CI testing passes (if applicable)
[N/A] Device performance regression CI testing passes (if applicable)
[x] New/Existing tests provide coverage for changes

Now that the PR is no longer a draft, please update the description of the PR to describe the changes.

Never mind. I saw that this is done already.


No video formats found

Checklist

[x] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2021.06.06
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://cdnlb.tvplayhome.lt/live/eds/TVPlaySportPlusHD/GO3_LIVE_DASH/TVPlaySportPlusHD.mpd?begin=1638186375']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.06.06
[debug] Python version 3.8.10 (CPython) - Linux-5.4.0-90-generic-x86_64-with-glibc2.29
[debug] exe versions: ffmpeg N-102732-g5509235818, ffprobe N-102732-g5509235818
[debug] Proxy map: {}
[generic] TVPlaySportPlusHD: Requesting header
[redirect] Following redirect to https://cdn9.tvplayhome.lt:443/live/eds/TVPlaySportPlusHD/GO3_LIVE_DASH/TVPlaySportPlusHD.mpd?begin=1638186375
[generic] TVPlaySportPlusHD: Requesting header
WARNING: Falling back on generic information extractor.
[generic] TVPlaySportPlusHD: Downloading webpage
[generic] TVPlaySportPlusHD: Extracting information
ERROR: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/dist-packages/youtube_dl/YoutubeDL.py", line 815, in wrapper
    return func(self, *args, **kwargs)
  File "/usr/local/lib/python3.8/dist-packages/youtube_dl/YoutubeDL.py", line 836, in __extract_info
    ie_result = ie.extract(url)
  File "/usr/local/lib/python3.8/dist-packages/youtube_dl/extractor/common.py", line 534, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/lib/python3.8/dist-packages/youtube_dl/extractor/generic.py", line 2525, in _real_extract
    self._sort_formats(info_dict['formats'])
  File "/usr/local/lib/python3.8/dist-packages/youtube_dl/extractor/common.py", line 1374, in _sort_formats
    raise ExtractorError('No video formats found')
youtube_dl.utils.ExtractorError: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to callyoutube-dl with the --verbose flag and include its complete output.

Description

Just tried to download DASH stream, and get error that No video formats found

No video formats found
How to workaround

Please provide an uncensored example URL

edited main post with uncensored example URL

Two things:

This is a dynamic MPD, IOW it's a LIVE MPEG-DASH stream, and yt-dl, to the best of my knowledge, doesn't support LIVE MPEG-DASH streams (only static MPDs are supported).

Both Audio and Video elementary streams use Common Encryption (cenc), specifically Microsoft PlayReady (MSPR) DRM:

<MPD
  xmlns="urn:mpeg:dash:schema:mpd:2011"
  xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
  xsi:schemaLocation="urn:mpeg:dash:schema:mpd:2011 http://standards.iso.org/ittf/PubliclyAvailableStandards/MPEG-DASH_schema_files/DASH-MPD.xsd"
  xmlns:cenc="urn:mpeg:cenc:2013"
  xmlns:mspr="urn:microsoft:playready"
  profiles="urn:mpeg:dash:profile:isoff-live:2011"
  type="dynamic" availabilityStartTime="2021-11-29T11:46:15Z"
  publishTime="2021-11-29T22:29:57.841001Z"
  minimumUpdatePeriod="PT2H"
  minBufferTime="PT10S"
  timeShiftBufferDepth="PT10H43M38S"
  maxSegmentDuration="PT8S">

Sorry, doesn't seem to be recordable... 😉


Search Slot by start is not working.

Describe the bug
Such as the title.
To Reproduce
GET https://spark.incendi.no/fhir/Slot?start=ge2021-08-05T00:00:00
Response
{
    "resourceType": "OperationOutcome",
    "issue": [
        {
            "severity": "error",
            "diagnostics": "FormatException: The string 'ge2021-08-05T00:00:00' is not a valid AllXsd value."
        }
    ]
}

@whyfate PR #416 will fix this issue


failure to narrow from maybe prop to defined in specific case

In the following code, if (filterPattern) should narrow the type of filterPattern from ?string to string, but flow still thinks it's ?string.
/* @flow */

import React, {Component} from 'react';
import _ from 'lodash';

export function replaceNonMatching(input: string, pattern: string): string {
  return input.replace(new RegExp(`(${pattern})|.`, 'gi'), (match, p1) => p1 || '');
}

type Props = {
  restrictValue?: (value: string) => string,
  filterPattern?: string,
};

export default class RestrictedInput extends Component<void,Props,void> {
  onChange: (e: {target: {value: ?string}}) => void = e => {
    let {value} = e.target;
    value = value || '';
    let {restrictValue, filterPattern} = this.props;
    if (!restrictValue) {
      if (filterPattern) {
        restrictValue = value => replaceNonMatching(value, filterPattern);
      }
      // irrelevant code removed
    }
    // irrelevant code removed
  };
  render(): ReactElement {
    return <input {...this.props} onChange={this.onChange}/>;
  }
}

Flow error:
$ flow status
common/RestrictedInput.jsx:22
 22:         restrictValue = value => replaceNonMatching(value, filterPattern);
                                                                ^^^^^^^^^^^^^ undefined. This type is incompatible with
  6: export function replaceNonMatching(input: string, pattern: string): string {
                                                                ^^^^^^ string

The error goes away when I merely copy the value to a variable beforehand:
      if (filterPattern) {
        let flowWorkaround = filterPattern;
        restrictValue = value => replaceNonMatching(value, flowWorkaround);
      }

This is a KP, and is documented on the Web site. Another workaround is to denote the refined variable (filterPattern in this case) as const.

ohhhh, right, it's so easy to forget that filterPattern could technically be mutated in this case.  If the error message had said something like "it could have been mutated before being used here" I might have understood off the bat.  Thanks!


[stable/kong] initcontainer error after helm upgrade

Is this a request for help?:
Yes
Is this a BUG REPORT or FEATURE REQUEST? (choose one):
BUG REPORT

Version of Helm and Kubernetes:
helm version
Client: &version.Version{SemVer:"v2.12.1", GitCommit:"02a47c7249b1fc6d8fd3b94e6b4babf9d818144e", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.12.1", GitCommit:"02a47c7249b1fc6d8fd3b94e6b4babf9d818144e", GitTreeState:"clean"}

kubectl version
Client Version: version.Info{Major:"1", Minor:"13", GitVersion:"v1.13.3", GitCommit:"721bfa751924da8d1680787490c54b9179b1fed0", GitTreeState:"clean", BuildDate:"2019-02-01T20:08:12Z", GoVersion:"go1.11.5", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"12", GitVersion:"v1.12.6", GitCommit:"ab91afd7062d4240e95e51ac00a18bd58fddd365", GitTreeState:"clean", BuildDate:"2019-02-26T12:49:28Z", GoVersion:"go1.10.8", Compiler:"gc", Platform:"linux/amd64"}

Which chart:
stable/kong
What happened:
After helm upgrade init container has errors
Error: [PostgreSQL error] failed to retrieve server_version_num: FATAL: password authentication failed for user "kong"
   Run with --v (verbose) or --vv (debug) for more details

What you expected to happen:
Init container should have finished and kong started
How to reproduce it (as minimally and precisely as possible):
Changed the kong image tag to 1.0.3 from 1.0.2 in values.yaml. After this ran helm upgrade command
Anything else we need to know:
After purge delete and deploy again it works seamlessly. Same problem if any other update to values.yaml. Helm upgrade never seems to work.

I have same problem. Upgrade does not work. No matter if I use exactly same values.yaml - STATUS: FAILED with --dry-run --debug

Update: Also with Helm Chart Kong version 0.10.3 we experience the same issue still.

The root of this problem is here: https://github.com/helm/charts/issues/5167
You can solve this problem only if you specify a postgres password yourself as part of your installation or upgrade, otherwise every time you perform an upgrade, a new password will be generated and Kong will not be able to talk to psql anymore.
This is really a limitation in Helm and not Kong's Helm chart.

Are there any news on this?
How do you work with this limitation?
Can I set my password somewhere in plain text to get upgrades working somehow?
Is there another way to work around this issue?
We cannot use the helm charts like this.

@rolandg You can set the password (and other values for the postgresql dependency) in your values file:
postgresql:
  postgresqlPassword: K6W9qBFzTQPj8xQiSouA

Found out that if you setup your own postgres and disable the one from the kong charts then upgrades are possible.

Hi.
I'm having this issue even when running for example:
helm install stable/kong --name kong --namespace api-gateway --set env.pg_database=kong --set env.pg_user=kong --set env.pg_password=0000000 --set postgresql.postgresqlPassword=0000000
Still, in the end, I keep getting the following:
2019-06-18 09:22:41.205 GMT [1635] FATAL:  password authentication failed for user "kong"
2019-06-18 09:22:41.205 GMT [1635] DETAIL:  Password does not match for user "kong".
	Connection matched pg_hba.conf line 95: "host     all             all             <IP_ADDRESS>/0               md5"

  Run with --v (verbose) or --vv (debug) for more details
waiting for db
Error: [PostgreSQL error] failed to retrieve server_version_num: FATAL: password authentication failed for user "kong

Any ideas?

@ammggm I needed to disable postgres with --set postgresql.enabled=false
here are my answers:
---
  image:
    tag: "latest"
  proxy:
    http:
      hostPort: "80"
    tls:
      hostPort: "443"
    type: "ClusterIP"
  admin:
    type: "ClusterIP"
    useTLS: true
  postgresql:
    enabled: false
  env:
    pg_password: "00000000"
    pg_host: "kong-psql.kong.svc.cluster.local"
    pg_database: "kong"
    pg_user: "kong"
    pg_port: 5432
  nodeSelector:
    deploy/kong-proxy: "true"
  replicaCount: "2"

But in that case it will not create the pod for the psql, what I don't seem to see working is that the user and pass I'm setting are just ignored.

So, in the end, managed to have it running if I don't give a --name during the helm install command.
Kinda strange for me that if you don't keep a random-name the all helm install will fail.

So what is the definitive fix when "--name" is specified during the helm install?

The error shouldn't really happen if --name is specified.
A potential solution that can work here is creating a secret in k8s for postgres password and then passing it to Kong and postgresql helm chart (via postgresql.postgresqlPassword param)

^^Didn't see that env/param - did see this one:
env.pg_password | Postgres database password (required if you are using your own database) | kong

Attempted to set both the env.pg_password and postgresql.postgresqlPassword in the values.yml / or via --set and still getting the same failure when attempting to specific --name for the helm release

Issue still exist, Kong does not work if the name parameter is specified

Upgrading Helm release causes reroll of the postgres-password in the <release_name>-postgresql secret, without rerolling the actuall postgresql password.
Workaround is setting the postgresql password.

I am facing the same issue.

Same issue after I installed with kong-operator.
/cc @gyliu513 @zxdiscovery

Workaround the issue by the following paramaters:
  # workaround for https://github.com/helm/charts/issues/12575
  env:
    pg_password: password
    database: postgres
  postgresql:
    enabled: true
    postgresqlUsername: kong
    postgresqlPassword: password
    postgresqlDatabase: kong
    service:
      port: 5432


sets initial signal value when none supplied during encoding

This is a 2 fold PR because one change is dependent on the other. When encoding a message if a signal was not supplied in the data to be encoded but the signal has an initial value an EncodeError would be raised. This PR changes that behavior to check for an initial value and if one is available it will use that to encode the message with
The second part is to allow encoding a message from the signal directly. The use case for this is if all other signals that are apart of the message have a default value the signal can directly be used to encode a message with the supplied value. It also takes into account if the signal has a multiplexer signal and if it does it sets the value of the multiplexer signal to the name of the signal where encode() was called.
I am sure that there might be a better way to go about handling this and I do not have an in depth knowledge of can databases.
In my specific use case I have updated the OBD2.dbc file that is available from
https://www.csselectronics.com/screen/product/obd2-dbc-file/language/en
which in it's downloaded form only decodes messages and does not have the ability to encode messages. I have updated it so it now does have the ability to decode messages and this is done by using initial values and multiplexers. So no actual data needs to be supplied in order to encode an OBD2 message.
If a copy of the updated version of the OBD database file is wanted let me know and I will upload it.
use code example
import cantools

db = cantools.db.load_file(r'OBD2.dbc')
message = db.get_message_by_name('OBD2_TX')

signal = message.get_signal_by_name('Odometer')
print(signal.encode(message))

data = {
    'ParameterID_Service01': 'Odometer',
    'Odometer': 0,
    'service': 1,
    'length': 2
}

print(message.encode(data))

and the output is exactly the same
b'\x02\x01\xa6\x00\x00\x00\x00\x00'
b'\x02\x01\xa6\x00\x00\x00\x00\x00'

so in order to transmit the obd2 request the only information that is needed is the name of the message and the name of the signal. Where as the current code makes it so that quite a bit of information needs to be known in order to encode the exact same message.
This is more of a proof of concept then anything else and I am sure that it can be improved upon. My database alterations are within the specification and I am using Vector CANdb++ to make the changes so it is within the ability of the current specification to function in this manner.

Coverage increased (+0.3%) to 96.768% when pulling c8785875f6fb554c2cf75060ef8f8feeb1c44134 on kdschlosser:initial_signal_values into 51b886b810f9ea848ef94622bcdae4df0a1696c3 on eerimoq:master.

Hi!
I like the idea of simplify encoding by using initial values if available. However, I think something like this fits cantools better:
import cantools

db = cantools.database.load_file('OBD2.dbc')
message = db.get_message_by_name('OBD2_TX')

data = message.initial_data()
print(message.encode(data))

That is, creating a new method initial_data on the Message class which returns a dictionary with all signals set to their initial values.
Would something like that work for you?

That would work as well. The only thing is how to handle multiplexer signals without having to specifically know the name of the dict key (the multiplexer signal name) that is needed in order to make it all work. When dealing with a multiplexer signal it would be hard to query the message it's self to build the data needed because it is not going to know exactly what you are trying to do because it's using signals inside of signals technically.

here is a copy of the modified OBD2 database file.
OBD2.zip
take a look at it and it should make it easier to understand what i am rambling on about.

Do you need multiplexing for your project? If not, we could simply raise an NotImplementedError exception if the message is multiplexed in a first implementation, and add multiplexing later. Or can a multiplexer not have an initial value? Hmm...

Feel free to implement initial_data().
Looks like you need multiplexing after looking in your DBC-file.

I would think that having an OBD database that is made up for sending and receiving (decoding and encoding) would be an extremely popular thing. It sure would make the DIY projects for communicating with vehicles exceedingly easy so long as there is a tool to read the database file and makes it easy to access the information in the database in the proper way.
That being said the name of this library is "cantools" and not "dbc reader" or something of that sort. An option would be to make a dedicated set of classes for dealing with OBD specifically. these classes would subclass Database, Message and Signal and override any methods that needs to have their functionality changed so it would be able to work. OBD is a really odd protocol and it doesn't really lend well to running on top of CAN, because of this a special case needs to be made for it.

There is no way around the multiplexing in order to handle the OBD protocol

Sounds interesting. I don't have any OBD experience myself, but if someone implements the required functionality I'll gladly merge (given it is high quality code).

The issue with using the initial value coupled with multiplexing is that there then becomes the need to know the name of the multiplexor when encoding the data for a signal that is encompassed in that signal.
With the current design of cantools I believe that it is impossible to keep the multiplexor transparent to an application unless the signal has an encode method. I would have preferred to not pass a message instance to Signal.encode but a Signal instance has no idea what Message instance it belongs to so there is no way to handle that aspect internally. A large code change would need to be made to assign a Message instance as a "parent" of a Signal instance or in the case of multiplexed signals setting a Signal instance as the parent of a Signal instance

We could add a method on the Message class that, say get_signal_multiplexers(signal_name) which returns a list of all multiplexer signals and their values for given signal to be sent.


torchrun --nnodes=1 --nproc_per_node=2 train_with_img.py --config ./configs/sky/sky_img_train.yaml error

A very impressive job. There are several issues when using SkyTimelapse data for image and video pre training

From utils import (clip_gradnorm, create'logger, update_ema,

Requires_grad, cleanup, create_tensorboard,
Write_tensorboard, setup_distributed, fetch files by numbers,
Fetch files by numbers and separation content motion were not found in utils in get_experiment_dir, separation content motion,)
"ImportError: Unable to import the name 'fetch files by num bers' from' utils' (Last/utils. py). After commenting out the corresponding file, it is sufficient. Can you ask what this mainly does? Does it directly use the original video?" Commenting out directly is not a problem, right? "”

If args. dataset=='webvideo2mlaion ':

Traceback (most recent call last):
File "/data/zqzx/latte/latte_main/latte/train_with_img. py", line 361, in
Main (OmegaConf. load (args. config))
File "/data/zqzx/latte/latte_main/latte/train_with_img. py", line 221, in main
Logger. info (f "Dataset contains {len (dataset):,} videos ({args. webvideo_data_path})")
File "/data/miniconde3/envs/yxl/lib/python3.9/site packages/omegaconf/docconfiguration. py", line 355, in getattr_
Self_ Formad_and_raise()
This can be directly solved by adding the corresponding solution to the sky_img_train.yaml corresponding to the actual video, which is. mp4? Or can we think of our own video dataset through this path?

If args. test_run: After commenting it out directly, it can be run now

Thank you.

fetch

These functions are auxiliary functions for my previous experiments, you do not need them. I've removed the redundant functions.
The training code for T2V is not currently supported in the Latte repo. However, we provide a complete training code for four datasets such as UCF101, SkyTimelapse, etc. (including video-image joint training). You can refer to them to modify it for your data set. Please use the UCF101 dataset as the base if your dataset has classes.
args. test_run is also an auxiliary function. I will clean it.

A very impressive job. There are several issues when using SkyTimelapse data for image and video pre training

From utils import (clip_gradnorm, create'logger, update_ema,

Requires_grad, cleanup, create_tensorboard,
Write_tensorboard, setup_distributed, fetch files by numbers,
Fetch files by numbers and separation content motion were not found in utils in get_experiment_dir, separation content motion,)
"ImportError: Unable to import the name 'fetch files by num bers' from' utils' (Last/utils. py). After commenting out the corresponding file, it is sufficient. Can you ask what this mainly does? Does it directly use the original video?" Commenting out directly is not a problem, right? "”

If args. dataset=='webvideo2mlaion ':

Traceback (most recent call last):
File "/data/zqzx/latte/latte_main/latte/train_with_img. py", line 361, in
Main (OmegaConf. load (args. config))
File "/data/zqzx/latte/latte_main/latte/train_with_img. py", line 221, in main
Logger. info (f "Dataset contains {len (dataset):,} videos ({args. webvideo_data_path})")
File "/data/miniconde3/envs/yxl/lib/python3.9/site packages/omegaconf/docconfiguration. py", line 355, in getattr_
Self_ Formad_and_raise()
This can be directly solved by adding the corresponding solution to the sky_img_train.yaml corresponding to the actual video, which is. mp4? Or can we think of our own video dataset through this path?

If args. test_run: After commenting it out directly, it can be run now

Thank you.

These functions are auxiliary functions for my previous experiments, you do not need them. I've removed the redundant functions.
The training code for T2V is not currently supported in the Latte repo. However, we provide a complete training code for four datasets such as UCF101, SkyTimelapse, etc. (including video-image joint training). You can refer to them to modify it for your data set. Please use the UCF101 dataset as the base if your dataset has classes.
args. test_run is also an auxiliary function. I will clean it.

Thank you very much


Update replaceAll function with index in regex on 3.x

Purpose

Update replaceAll function with index in regex

Fixes #39655
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

The master PR is https://github.com/ballerina-platform/ballerina-lang/pull/39662


Windows likes its files closed before removing them.

Cacheddownloader has a number of issues on Windows, mostly due to irritating differences between Windows and Linux file open/closing and removal. This fixes one of them.
There's no additional test code necessary, because this change actually fixes three tests that currently fail when run on Windows.
After this change, there will still be 29 failing tests (previously 32), and we'll be submitting more pull requests as we stomp those.

Thanks, @lvarvel! I've created this Diego story to evaluate the PR.

Could you please clarify which tests were failing on Windows? Was it the cacheddownloader unit tests, or other tests from diego-release?
Thanks,
@krishicks && @ematpl

The tests were the cacheddownloader unit tests.  I don't think we had any other failing diego tests.

There is also one final test that fails intermittently on Windows. We've dropped a chore into our backlog, and we'll get back to you on that one later.  This PR should be good to go.

@lvarvel @mavenraven: The first commit was fine, but the second one (with the windowsSafeRename function) is failing on the Travis linux build. Could you please remove that commit and submit a separate PR for that one?  It may be possible to put the architecture-dependent details into separate files with different go build tags for the different platforms to get this to compile correctly, but it would be far preferable for us to find a solution that doesn't rely on these low-level details.
Thanks,
@ematpl && @matt-royal

Sorry about that.  We've made some changes to use build tags.

Related: https://github.com/golang/go/issues/8914
TLDR: os.Rename doesn't work as expected on Windows, and the Golang core team won't fix it.  They might add an os.Replace function in go 1.5 that does the right thing (essentially what we've done here).

Excellent, thanks for the reference to the outstanding issue. If we can remove this with Go 1.5 or later, that would be great.
We tried the entire units/inigo/DATs run on OS X/bosh-lite and it seems fine, so we can merge and push this once ketchup is green. In the meantime, I have a few minor comments to make inline.
Thanks,
Eric

We've completed the requested changes, waiting for Varvel to give us push access.

Access granted, mavenraven.
We've completed the requested changes, waiting for Varvel to give us push
access.
—
Reply to this email directly or view it on GitHub
https://github.com/pivotal-golang/cacheddownloader/pull/6#issuecomment-75969499
.


Unexpected crash

Rust analyzer crashed in a unexpected way: https://gist.github.com/caemor/2d0b3a427574f257062fcd092137b7d2
I am not sure if that output is even helpful in any way.
After restarting vs code the crash didn't happen anymore.
Version was rust-analyzer-0.2.297

thread '<unnamed>' panicked at 'Failed to make ast node syntax::ast::generated::nodes::Name from text mod ::from_reader;', crates/syntax/src/ast/make.rs:325:13

Can you share a project where this happens? The crash is in the generate_function assist.

It only happened once and sadly the project where it happened is private.

Closing because I don't think the crashlog is helping and the other above called reasons


Update examples to use tokio 0.2

romio is no longer actively maintained.

bors r+

bors r-

bors r+


QueryRewrite: re-enable non-query tests related to tracking

Disabled for now since tracking is not working correctly. Affected tests:
GraphUpdates,
ProxyGraphUpdates
Load

Remaining work here is in-memory tests:

GraphUpdatesInMemoryTest
ProxyGraphUpdatesInMemoryTest
LoadInMemoryTest
InMemoryComplianceTest
FixupTest.Collection_nav_props_remain_fixed_up_after_manual_fixup_and_DetectChanges

Closed in favor of #16963


Clarify status of addObject and addObjects methods on SearchIndex

We are trying to upgrade our usage of this library to the latest version in the 4 series from version 3.35.1, and ran into the following error when compiling the TypeScript:
Property 'addObject' does not exist on type 'SearchIndex'.

I did not see anything in the migration guide regarding addObject and addObjects being removed, but I have seen some PRs and issues in this repository mentioning it.
It's also noteworthy that the "Add object" documentation no longer seems to mention addObject - it uses saveObject instead.
What's going on here? Is it just missing from the types, or is it actually removed? If it's removed, what happened to the migration guide, and how should we be migrating?
Thanks!

The replacement is saveObjects(..., { autoGenerateObjectIDIfNotExist: true }), but this indeed doesn't seem to be mentioned in the migration guide, which is a mistake, cc @nunomaduro
We'll update the migration guide, thanks

This migration guide was being updated: https://www.algolia.com/doc/api-client/getting-started/upgrade-guides/javascript/#the-addobject-and-addobjects-methods.


NIFI-1909 Adding ability to process schemaless Avro records to

NIFI-1909 Adding ability to process schemaless Avro records to ConvertAvroToJson.  A new property named schema is added to the processor.  If a value is provided for this property, then it is assumed that incoming Avro records are schemaless and the value of the schema property is used when parsing incoming records.

I made the changes that @markap14 suggested and removed unused imports that were identified by checkstyle.  I've attempted to run mvn -Pcontrib-check on 3 different machines, but I get failures in tests for the nifi-framework-core module that seem unrelated to my modifications.  For example, running the build on a Linux machine, I get the following error:
Running TestSuite
Tests run: 172, Failures: 2, Errors: 0, Skipped: 1, Time elapsed: 272.879 sec <<< FAILURE! - in TestSuite
testStateTooLargeExceptionThrownOnReplace on testStateTooLargeExceptionThrownOnReplace(org.apache.nifi.controller.state.providers.zookeeper.TestZooKeeperStateProvider)(org.apache.nifi.controller.state.providers.zookeeper.TestZooKeeperStateProvider)  Time elapsed: 24.341 sec  <<< FAILURE!
org.junit.runners.model.TestTimedOutException: test timed out after 20000 milliseconds
at java.lang.Object.wait(Native Method)
at java.lang.Object.wait(Object.java:502)
at org.apache.zookeeper.ClientCnxn.submitRequest(ClientCnxn.java:1342)
at org.apache.zookeeper.ZooKeeper.setData(ZooKeeper.java:1268)
at org.apache.nifi.controller.state.providers.zookeeper.ZooKeeperStateProvider.setState(ZooKeeperStateProvider.java:318)
at org.apache.nifi.controller.state.providers.zookeeper.ZooKeeperStateProvider.setState(ZooKeeperStateProvider.java:283)
at org.apache.nifi.controller.state.providers.zookeeper.ZooKeeperStateProvider.setState(ZooKeeperStateProvider.java:228)
at org.apache.nifi.controller.state.providers.zookeeper.TestZooKeeperStateProvider.testStateTooLargeExceptionThrownOnReplace(TestZooKeeperStateProvider.java:176)

@ryanpersaud thanks for getting this in! I have verified the checkstyle and pushed to both the 0.x and 1.0.0 baselines. Thanks again for knocking this out!


Recent change for callables in model factories attributes breaks existing factories

Laravel Version: 5.4.34
PHP Version: 7.0.8
Database Driver & Version: SQL server

Description:
The recent change to add support for callables in model factories attributes broke our testsuite. https://github.com/laravel/framework/pull/20692
I filtered the problem down to the change made in this commit https://github.com/laravel/framework/commit/91765f5a35a6410d6315042638721242ed0e9c5f where it checks if the attribute you give the factory is a callable.
In one of our factories, a possible value of an attribute is DIR, and it PHP will tell you this is callable (the dir() global function). So it will try to call dir() and pass it the attributes array, which will result in an exception: dir() expects parameter 1 to be a valid path, array given
Steps To Reproduce:

Create a model factory
Set one of the attributes to some global function name
Try to use the factory and see it fail

fixed

Thanks a lot!


[Feature Request]: Filter / Customize nodes generated by CodeSplitter

Feature Description
I propose adding a new feature to codesplitter to split based on capture groups:
For instance consider below example where I am only interested in just the function and class definations at the root level of the python source code:-
def foo():
    """Helllo, world!"""
    def bar(**x):
        print("Hello, world!")
        print(x)
    if bar:
        baz(x=3, y=6)

class MyC:
  def hello():
    print(22)

if __name__ == "__main__":
    foo()
    f=3

I can use the following tree sitter query to named capture the function definations and class definations of the root:-
((module
  (function_definition) @root.function
  (class_definition) @root.class
))

You may test tree sitter queries here: https://tree-sitter.github.io/tree-sitter/playground
Reason
Current implementation is to split recursively based on size, without any metadata being generated which feel limiting for some scenarios.
Value of Feature
In the sample abve I captured function and class definations at root level ignoring the exectution statements, comments etc. from the root level.
It could also help us to tackle scenarios such as capturing the top level imports as metadata (hello java?) which is normally lost after chunking.

I highly encourage a PR to improve code splitting :)


Scanning with HydraNFC seems to do nothing

Hi,
I'm working on HydraBUS with HydraNFC and I can't get NFC scan to work.
I don't know if i'm doing something wrong. I just uploaded HydraFW v9 on the board via DFU and followed the steps desribed in the folowing guide: https://github.com/hydrabus/hydrafw/wiki/HydraFW-HYDRANFC-guide#read-uid-of-an-isoiec_14443-tag-only-type-a-limited-to-4bytes-uid
Nothing happend when I try to scan my NFC Tag.
Type: ISO 14443-3A, NXP MIFARE Ultralight
So in the console, I did:

nfc typea
NFC> scan

And also tryied:
NFC> scan continuous period 500 (or just continuous without specifying a period)
But in both cases nothing happend. I just got a message "Scanning MIFARE with 500ms period. Press user button to stop." when I specify. Otherwise, I just got nothing..
Do you have any idea of what am i doing wrong and how to fix it?
Thank you for your help!
BR,
Pierre

I found the reason of my issue. It seems that the antenna frequency where it emit is affected by the tag that I used. After analysation, I fixed the issue by placing my tag at 3 centimeters from the antenna and not directly sticked. Now, the antenna is capable to scan my tag correctly and displays the results that I expected.

Thanks for the feedback this limitation is because TRF7970A chipset is not clever enough (even if it was the best when it was released) to manage correctly some tags when they are too close.
This issue will be solved with HydraNFC v2 which use a totally new chipset with some very advanced hardware/software features to do automatic tuning, a very good AGC (Automatic Gain Control) integrated and tons of other unique features ...


feat: cowswap buildTrade

This implements buildTrade method for CowSwapper (https://github.com/shapeshift/lib/issues/725)
It also fixes a couple of issues related to previous PRs after successful local integration with Web, notably in getCowSwapTradeQuote.ts.
Some screenshots of the local integration with web

@0xdef1cafe fixed comments + added one small fix into helpers.ts/getUsdRate() method relative to USDC rate


Fix typo at SpecificHostPolicy

What changes are proposed in this pull request?
Please outline the changes and how this PR fixes the issue.
Why are the changes needed?
Please clarify why the changes are needed. For instance,

If you propose a new API, clarify the use case for a new API.
If you fix a bug, describe the bug.

Does this PR introduce any user facing changes?
Please list the user-facing changes introduced by your change, including

change in user-facing APIs
addition or removal of property keys
webui

The error has nothing to do with my own modification.

alluxio-bot, merge this please


Replace haste imports with path-based imports

Summary
Haste imports have been removed from react-native in 0.60.0. This PR replaces those imports with path-based ones so that the CheckBox component will work with react-native 0.60.0+.
Fixes https://github.com/react-native-community/react-native-checkbox/issues/3
Fixes https://github.com/facebook/react-native/issues/25558
Test Plan
A react-native 0.60.0 app that imports @react-native-community/checkbox will compile and not crash the metro bundler.
What's required for testing (prerequisites)?
A react-native 0.60.0 app, this module
What are the steps to reproduce (after prerequisites)?
Build the app
Compatibility

OS
Implemented

iOS
✅
CheckBox is not supported on iOS but I've updated the UnimplementedView import

Android
✅

Checklist

[ ] I have tested this on a device and a simulator
[ ] I added the documentation in README.md
[ ] I mentioned this change in CHANGELOG.md
[ ] I updated the typed files (TS and Flow)
[ ] I added a sample use of the API in the example project (example/App.js)

Thanks for your work!


⚠ MineVN Website has degraded performance

In 56cba07, MineVN Website (http://minevn.net) experienced degraded performance:

HTTP code: 200
Response time: 9692 ms

Resolved: MineVN Website performance has improved in b166ce3.


tokenField always has keyboard focus

in my project I have ZFTokenField and UITextView.
when I tap on ZFTokenField and after that when I tap on UITextView then focus could not be get into UITextView, its always focus on  ZFTokenField

+1

I have not find any solution so that I decided to change library, now I am using VENTokenField


ContextProvider setting usage in api?

While we have the environment variable ContextProvider in api examples, I don't see any code that use it.
Does it means that currently Tweek can't be loaded with two context providers (couchbase and redis)?

You're correct. It's not used anywhere in the code. It was introduced during development, and was left there accidentally.
It does mean that we currently do not support more then one context provider.

Let's fix it by adding a reflection attribute ContextProviderName and choose the relevant driver based on that setting

Is this still relevant after the change to that way we load addons in #658 ?

I think we can close it for now.


rerun vndr

run latest vndr so as to collect more LICENSE files
remove unused packages
vendor github.com/philhofer/fwd with LICENSE.md (MIT)
vendor github.com/bsphere/le_go with LICENSE (MIT)

Before this PR:
$ make validate
WARNING: could not find copyright information for github.com/kr/pty
WARNING: could not find copyright information for github.com/mattn/go-shellwords
WARNING: could not find copyright information for github.com/docker/libkv
WARNING: could not find copyright information for github.com/opencontainers/go-digest
WARNING: could not find copyright information for github.com/philhofer/fwd
WARNING: could not find copyright information for github.com/aws/aws-sdk-go
WARNING: could not find copyright information for github.com/bsphere/le_go
WARNING: could not find copyright information for github.com/docker/containerd
WARNING: github.com/mreiferson/go-httpclient is vendored but unused
WARNING: bitbucket.org/ww/goautoneg is vendored but unused
WARNING: could not find copyright information for github.com/spf13/cobra
WARNING: could not find copyright information for github.com/docker/go-metrics
WARNING: could not find copyright information for github.com/xeipuuv/gojsonpointer
WARNING: could not find copyright information for github.com/xeipuuv/gojsonreference
WARNING: could not find copyright information for github.com/xeipuuv/gojsonschema

After:
$ make validate
WARNING: could not find copyright information for github.com/mattn/go-shellwords

(this is not a legal issue, see vendor/github.com/mattn/go-shellwords/README.md (MIT) )
addendum to https://github.com/docker/docker/pull/31076
Signed-off-by: Akihiro Suda<EMAIL_ADDRESS>

mm, need to install the latest vndr on Jenkins..
https://jenkins.dockerproject.org/job/Docker-PRs-vendor/2936/console

@AkihiroSuda  ah, it's in binaries-commits 😉

thank you, updated PR

Looks like the matching needs a small tweak in vndr (https://github.com/docker/docker/pull/31566#pullrequestreview-25225672)

updated PR with LK4D4/vndr@19220953c4a45310a4c404b7905154e29120249e

For reference; for the version bumps;
for https://github.com/bsphere/le_go/ (includes https://github.com/philhofer/fwd/commit/8fd9a4b75098b2125fe442e48a3ffbf738254e13)
https://github.com/philhofer/fwd/compare/899e4efba8eaa1fea74175308f3fae18ff3319fa...98c11a7a6ec829d672b03833c3d69a7fae1ca972
for https://github.com/bsphere/le_go/ (relates to https://github.com/docker/docker/issues/31020)
https://github.com/bsphere/le_go/compare/d3308aafe090956bc89a65f0769f58251a1b4f03...7a984a84b5492ae539b79b62fb4a10afc63c7bcf

ignoring power and z CI for now, looks like that's still being worked on


[BUG]: Package.json exports for billing config incorrect

Describe the bug
The export for billing_configuration is pointing to a non existent file.
Additional context
I believe you meant to point to lib/model/customer_billing_configuration instead of lib/model/billing_configuration

Hello @Ces-D, thanks we'll fix this!


Update for 0.6.0 of Kanri

Wondering if this is planned soon? I tried installing via brew install --cask trobonox/cask/kanri and noticed that it was installing version 0.5.0 instead of 0.6.0. I also got a failure with a sha256 mismatch, so wondering if the update to 0.6.0 would solve that. Work requires me to have homebrew in a user-local directory, so that could also be causing problems.
==> Tapping trobonox/cask
Cloning into '/Users/ME/homebrew/Library/Taps/trobonox/homebrew-cask'...
remote: Enumerating objects: 19, done.
remote: Counting objects: 100% (19/19), done.
remote: Compressing objects: 100% (14/14), done.
remote: Total 19 (delta 3), reused 2 (delta 0), pack-reused 0
Receiving objects: 100% (19/19), 4.13 KiB | 2.07 MiB/s, done.
Resolving deltas: 100% (3/3), done.
Tapped 1 cask (14 files, 10.7KB).
==> Downloading https://github.com/trobonox/kanri/releases/download/app-v0.5.0/kanri_0.5.0_aarch64.dmg
==> Downloading from https://objects.githubusercontent.com/github-production-release-asset-2e65be/494602314/8b
####################################################################################################### 100.0%
Error: SHA256 mismatch
Expected: e8ef1fc645fb1487b5891ff056713b7e07a121c585365a7eefb21fc6867b49fd
  Actual: 94cffd8dfbb268085c5d45445b3d5e589bfbc79723fa31532f72b022f56d93b4
    File: /Users/ME/Library/Caches/Homebrew/downloads/bd80d392e3995e8bb95ca6527d022a4636c4a23d255883b7dd46ea539c88b66e--kanri_0.5.0_aarch64.dmg
To retry an incomplete download, remove the file above.

My apologies, I forgot to update the cask. Will do it soon.

Should be fixed. Please note that you will have to run following command to run the app (since it is not signed):
xattr -cr /Applications/kanri.app

Good to go! Thanks 😄