We can continue to use WEBSITE_VNET_ROUTE_ALL if Route All is disabled

[Enter feedback here]
https://docs.microsoft.com/en-us/azure/app-service/web-sites-integrate-with-vnet#application-routing

The Route All configuration setting replaces and takes precedence over the legacy WEBSITE_VNET_ROUTE_ALL app setting.

As far as I can see, the Route All configuration setting doesn't take over the WEBSITE_VNET_ROUTE_ALL app setting when WEBSITE_VNET_ROUTE_ALL is "1" and Route All is disabled. Under this condition, I made sure that we can use WEBSITE_VNET_ROUTE_ALL is "1" as we have ever experienced.  If my understanding is correct, in my opinion, this document should include this context to prevent from misreading.

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

ID: a7a98803-1438-b1b5-f543-7dd88bc4294e
Version Independent ID: 37ff1d0f-ed8e-5e4d-1f4c-1b9f6cffb938
Content: Integrate app with Azure Virtual Network - Azure App Service
Content Source: articles/app-service/web-sites-integrate-with-vnet.md
Service: app-service
GitHub Login: @ccompy
Microsoft Alias: ccompy

@mshyatsu Thank you for your query . We will investigate and update the thread.

Yes, you can continue to use the App Setting. We have made updates to the documentation and the portal will be updated with the correct status very soon. Hope this helps

Thank you so much for the update!


Create a internal training video on how to use CRM

Describe the issue
We want to save time for future users to onboard CRM and understand how to navigate through.
Additional context

Fair amount of learning debt in this ticket

This is internal facing, so nothing for the demo (the CRM is private)

The video is intended to be "clicking around and doing stuff", no voice or face work.

There could be a PPT slide or two mixed in if Text is needed

Definition of done

Meet with Julian and get trained on the below items

Figure out with the team's help how to make a video in Windows (what format, how to edit and stitch together etc)

Silent screen recording training video demonstrating filters, how to find a particular entry, the different data views, how to edit

Have a section explaining gotchas (known bugs, little problems that could surprise a new user)

Dup, no problem, pls see 2043


[Task] Graphing lib for displaying data

This is reliant on the completion of https://github.com/BuidlGuidl/impact-calculator/issues/4
We need to do discovery and then implement a library for displaying the graph as defined in the design.
https://www.figma.com/file/OS8DSbKyuiYPqcRTmmhTIM/Impact-Calculator-🌱?type=design&node-id=0-1&mode=design

@0xChijoke is working on adding recharts.


Update Ohai to 19

Description

bundle install was conducted for a manual version bump of ohai
Related Issue

Types of changes

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

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] I have run the pre-merge tests locally and they pass.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] If Gemfile.lock has changed, I have used --conservative to do it and included the full output in the Description above.
[ ] All new and existing tests passed.
[ ] All commits have been signed-off for the Developer Certificate of Origin.

Pending failures: kitchen test, In verify pipeline windows unit tests and windows plan
All of the above errors are irrelevant to the change, and we will work on fixing them separately hence merging this PR.


Cache Game.map.findRoute calls

Saves in global by default, persists to Memory if length is >= 4.

This Pull Request has been merged by GitConsensus.
Vote Totals

Yes
No
Abstain
Voters

3
0
0
3

Vote Breakdown

User
Yes
No
Abstain

tedivm
✔

TheRealMaxion
✔

ViezeVingertjes
✔

Vote Results

Criteria
Result

Has Quorum
True

Has Votes
True


Fix cache clearing strategy for collection reset

Corrected location of clear function off of app object

@marcos-abreu commited the build files


⚠ Bofrost.at has degraded performance

In 4e9d187, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1450 ms

Resolved: Bofrost.at performance has improved in 8f61fc5.


List row actions

Here's idea behind the change: abstract away per-row actions and allow adding them right from ModelView without overriding the template. Reason - I was doing some pet-projects and found myself adding way too much 4-line templates for sake of adding custom button to the list view.
Here's how it looks like: https://github.com/flask-admin/flask-admin/pull/1242/files#diff-50bea377c39931e487a83ccb0a5a164eR776
@pawl - any objections?

:+1: Tried it locally and everything looks good. I made some comments on a few minor things.
Cool idea, it's going to be handy to be able to add new row actions like this:
class CarAdmin(sqla.ModelView):
    column_list_row_actions=[
        template.LinkRowAction('glyphicon glyphicon-off', 'http://localhost:5000/?id={row_id}'),
    ]

Pushed some fixes, made property name shorter - column_extra_row_actions.

Just looked over the changes again. I think this one is ready to be merged.


No logs about c-icap in GUI but there are in system

Describe the bug
There is no log visible in GUI for plugin c-icap in its log section.
But if I ssh on my opnsense:
% sudo ls -l /var/log/c-icap/
total 4672
-rw-r--r--  1 c_icap  c_icap  2225286 Sep  3 21:27 access.log
-rw-r--r--  1 c_icap  c_icap     8124 Aug 25 00:00 access.log.0.gz
-rw-r--r--  1 c_icap  c_icap    55567 Aug 23 00:00 access.log.1.gz
-rw-r--r--  1 c_icap  c_icap    15260 Aug  5 00:00 access.log.2.gz
-rw-r--r--  1 c_icap  c_icap    32844 Sep  3 21:22 server.log
-rw-r--r--  1 c_icap  c_icap     1210 Aug 25 00:00 server.log.0.gz
-rw-r--r--  1 c_icap  c_icap     2807 Aug 23 00:00 server.log.1.gz
-rw-r--r--  1 c_icap  c_icap     1167 Aug  5 00:00 server.log.2.gz

There is something!
To Reproduce
Steps to reproduce the behavior:

Go to Services/C-ICAP/logs on WebGUI (or something like that, mine is in French)
Nothing in logs

Expected behavior
Something in logs, since they exists in the system!
Screenshots

Environment
Software version used and hardware type if relevant.
e.g.:
OPNsense 21.7.1-amd64
FreeBSD 12.1-RELEASE-p19-HBSD
OpenSSL 1.1.1k 25 Mar 2021
Intel Core Processor (Skylake, IBRS) (4 cores)

@JulienDlq @mimugmail I assume same issue as dnscrypt-proxy


backport label regex

It would be useful to be able to configure the label name with a regex that extracts the branch name:
backport_label_pattern: '^backport-to-(.*)$'

Hi @domenkozar, glad to see you're using the bot 👍
I sounds like a useful feature. Let me make sure I understand what you mean: the label name pattern is currently hard coded to match the regex ^backport ([^ ]+)?$, and you'd like to make this pattern configurable.
Note that in the pattern ^backport ([^ ]+)?$ a space is not an allowed as branchname character, while it is allowed in github labels. see https://regex101.com/r/pS1wzb/1
I guess we should restrict somehow what these patterns are supposed to look like to make configuration easy. Tthere must be only 1 capture group that captures branchname compliant strings. For this, we could consider a special identifier or a keyword to match a branchname and allow the rest of the label pattern to be customized. For example, an identifier like ${branchname}. Let's say you then configure backport_label_pattern: 'foo-${branchname}-bar' which would result in a regex pattern ^foo-([^ ]+)?-bar$, that can match labels like foo-1.0.0-bar and then capture 1.0.0 as branchname.
It's a bit complex, but this at least makes sure no wrong configurations can be provided. How do you feel about this solution and would it solve your case?

I'd keep it simple and flexible by using regex and then check there's only one capture group.
Why do you forbid the space (I don't need that, just wondering), it could be just ^backport (.+)$?

I would also keep it simple and iterate from there.

I'd keep it simple and flexible by using regex

Sounds good, we can always extend it later if the need is there. For now, I agree that we can start by making it configurable.

Why do you forbid the space

A branchname cannot contain a space, so it would just fail creating the branch. Failure could be fine too, but at the time we chose to just not execute the backporting logic for such labels.

@domenkozar I've just released a new version (v0.0.4) that adds support for a custom label pattern. See #42 for details. Please let me know if you run into any problems.


VMService._connect NoSuchMethodError: The getter 'path' was called on null.

Flutter crash report; please file at https://github.com/flutter/flutter/issues.
command
flutter --no-color attach --machine --device-id=1b6eae756ec0f26f3134b3d0c03f7932a8be1a61 lib/main.dart
exception
NoSuchMethodError: NoSuchMethodError: The getter 'path' was called on null.
Receiver: null
Tried calling: path
#0      Object.noSuchMethod (dart:core-patch/object_patch.dart:53:5)
#1      VMService._connect (package:flutter_tools/src/vmservice.dart:325:80)
#2      VMService.connect (package:flutter_tools/src/vmservice.dart:315:21)
#3      FlutterDevice.connect.<anonymous closure> (package:flutter_tools/src/resident_runner.dart:175:35)
#4      _rootRunUnary (dart:async/zone.dart:1134:38)
#5      _CustomZone.runUnary (dart:async/zone.dart:1031:19)
#6      _CustomZone.runUnaryGuarded (dart:async/zone.dart:933:7)
#7      _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:338:11)
#8      _BufferingStreamSubscription._add (dart:async/stream_impl.dart:265:7)
#9      _SyncBroadcastStreamController._sendData (dart:async/broadcast_stream_controller.dart:377:20)
#10     _BroadcastStreamController.add (dart:async/broadcast_stream_controller.dart:252:5)
#11     _AsBroadcastStreamController.add (dart:async/broadcast_stream_controller.dart:476:11)
#12     _rootRunUnary (dart:async/zone.dart:1138:13)
#13     _CustomZone.runUnary (dart:async/zone.dart:1031:19)
#14     _CustomZone.runUnaryGuarded (dart:async/zone.dart:933:7)
#15     _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:338:11)
#16     _BufferingStreamSubscription._add (dart:async/stream_impl.dart:265:7)
#17     _SyncStreamControllerDispatch._sendData (dart:async/stream_controller.dart:766:19)
#18     _StreamController._add (dart:async/stream_controller.dart:642:7)
#19     new Stream.fromFuture.<anonymous closure> (dart:async/stream.dart:174:18)
#20     _rootRunUnary (dart:async/zone.dart:1134:38)
#21     _CustomZone.runUnary (dart:async/zone.dart:1031:19)
#22     _FutureListener.handleValue (dart:async/future_impl.dart:139:18)
#23     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:680:45)
#24     Future._propagateToListeners (dart:async/future_impl.dart:709:32)
#25     Future._completeWithValue (dart:async/future_impl.dart:524:5)
#26     _AsyncAwaitCompleter.complete (dart:async-patch/async_patch.dart:32:15)
#27     _completeOnAsyncReturn (dart:async-patch/async_patch.dart:290:13)
#28     MDnsObservatoryDiscovery.getObservatoryUri (package:flutter_tools/src/mdns_discovery.dart)
<asynchronous suspension>
#29     AttachCommand._attachToDevice (package:flutter_tools/src/commands/attach.dart:238:47)
<asynchronous suspension>
#30     AttachCommand.runCommand.<anonymous closure> (package:flutter_tools/src/commands/attach.dart:177:19)
#31     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:146:29)
#32     _rootRun (dart:async/zone.dart:1126:13)
#33     _CustomZone.run (dart:async/zone.dart:1023:19)
#34     _runZoned (dart:async/zone.dart:1518:10)
#35     runZoned (dart:async/zone.dart:1465:12)
#36     AppContext.run (package:flutter_tools/src/base/context.dart:145:18)
#37     AttachCommand.runCommand (package:flutter_tools/src/commands/attach.dart:176:19)
<asynchronous suspension>
#38     FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:615:18)
#39     _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:73:64)
#40     _rootRunUnary (dart:async/zone.dart:1134:38)
#41     _CustomZone.runUnary (dart:async/zone.dart:1031:19)
#42     _FutureListener.handleValue (dart:async/future_impl.dart:139:18)
#43     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:680:45)
#44     Future._propagateToListeners (dart:async/future_impl.dart:709:32)
#45     Future._completeWithValue (dart:async/future_impl.dart:524:5)
#46     Future._asyncComplete.<anonymous closure> (dart:async/future_impl.dart:554:7)
#47     _rootRun (dart:async/zone.dart:1126:13)
#48     _CustomZone.run (dart:async/zone.dart:1023:19)
#49     _CustomZone.runGuarded (dart:async/zone.dart:925:7)
#50     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:965:23)
#51     _microtaskLoop (dart:async/schedule_microtask.dart:43:21)
#52     _startMicrotaskLoop (dart:async/schedule_microtask.dart:52:5)
#53     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:118:13)
#54     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:175:5)

flutter doctor
[✓] Flutter (Channel master, v1.12.11-pre.8, on Mac OS X 10.14.5 18F132, locale zh-Hans-CN)
    • Flutter version 1.12.11-pre.8 at /Users/apple/Library/flutter
    • Framework revision f40dbf8ca0 (2 days ago), 2019-11-23 01:02:54 -0500
    • Engine revision d1cac77d5a
    • Dart version 2.7.0

[!] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/apple/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • ANDROID_HOME = /Users/apple/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses

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

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 41.0.2
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[✓] IntelliJ IDEA Ultimate Edition (version 2019.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 40.2.3
    • Dart plugin version 192.7359

[✓] Connected device (2 available)
    • “apple”的 iPhone • 1b6eae756ec0f26f3134b3d0c03f7932a8be1a61 • ios • iOS 12.4.2
    • iPhone 8        • D2483163-94BA-4A1C-9EB8-26964C81824C     • ios • com.apple.CoreSimulator.SimRuntime.iOS-13-0 (simulator)

! Doctor found issues in 1 category.

Hi @lixianjun0903
give the error you are reporting, there might be something wrong with the code;
could you please provide a minimal code sample to reproduce the problem?
Thank you

observatoryUri getting passed into VMService.connect() is null?  Interesting, that's coming right from inside a .listen((Uri observatoryUri)... block

Can we get the output of flutter --verbose --no-color attach --machine --device-id=1b6eae756ec0f26f3134b3d0c03f7932a8be1a61 lib/main.dart?  There should be a Connecting to service protocol: log in there.

Fixed by https://github.com/flutter/flutter/pull/45439


Editor: Preview renders shadows with artifacts

Description
Preview renders shadows with artifacts
Prerequisites
Create a Direct Light source and enable shadows
Steps to Reproduce

Check shadows in Preview

Expected behavior:
Shadows render like in Viewport
Actual behavior:

Reproducibility:
100%
Versions
2023.2
Additional Information
n/a


Scripts to convert raw data into input data for the model

Hello,
I really like the paper and the summarizer-based approach! I am sorry if I missed it, but there do not seem to be scripts that go from the data of just the body and headline of articles to summarizer results, and especially the values of features (such as 'cosineSimilarity' 'max_score_in_position' 'overlap') used to create the data files that can be used to train/predict using the provided models. Are there plans to release those data processing scripts? Are there any ready pointers to get the feature values for the body and the headline in order to use the pretrained models on a different news article dataset?
Thanks!

Hello
I commit a script to preprocess a new dataset. This script uses TextRank summarizer instead of PLM summarizer because PLM is not available yet.


Negation re-applies preceding operation

Describe the bug

Steps To Reproduce

Open a scientific calculator.
Enter a simple calculation, e.g. 5 x 2.
Press = to obtain the result, e.g. 10.
Press the +/- button to negate the result.
Observe that the intermediate result (negate(10)) shown is -10.
Press = to clear the calculator's screen and obtain the result; it is now -20.
Press +/- again to negate; intermediate result (negate(-20)) is shown as 20.
Press = again; the screen now shows 40.

Expected behavior
I expect the calculator to only apply calculations shown in the secondary line, i.e. negate(10), not to implicitly re-apply the preceding operation (e.g. x2).

Screenshots

The first column shows entered calculations the second results after pressing '='. The last image on the right shows the history of calculations

Device and Application Information

OS Build: 10.0.17763
Architecture: X64
Application Version: 10.1903.21.0
Region: en-US
Dev Version Installed: False

Additional context

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.

Thanks for the detailed bug report! This appears to be a duplicate of #264. The behavior you are seeing is a result of differences in how we handle interactions between the enter key and binary/unary operations.


How to pass variadic function instead of value?

How can we pass n variables to sum() function instead of separate values
package main

import (
	"fmt"
	wasm "github.com/wasmerio/go-ext-wasm/wasmer"

)
func main() {

	fmt.Println("result is", test(3, 4))
	fmt.Println("result is", test(3, 4, 5))

}

func test(input1, input2, input3 int) int{
	bytes, _ := wasm.ReadBytes("test.wasm")

	// Instantiates the WebAssembly module.
	instance, _ := wasm.NewInstance(bytes)
	defer instance.Close()

	// Gets the `sum` exported function from the WebAssembly instance.
	//sum := instance.Exports["sum"]
	sum2 := instance.Exports["sum2"]

	// Calls that exported function with Go standard values. The WebAssembly
	// types are inferred and values are casted automatically.

	// result, _ := sum(3, 4) // pass variadic function value
       // result, _ := sum2(input1, input2, input3)) // works fine

	fmt.Println(result)
	return result

}

how to pass instead of this
func test(input ...int)int {
      // body goes here

      // pass values in generic way
	result, _ := sum(input)
       fmt.Println(result)
	return result

}

Thanks

@Hywan can we pass array of variables?
package main
import (
    "fmt"
    wasm "github.com/wasmerio/go-ext-wasm/wasmer"
)
func main() {
   	val := []int{3, 2}
	test(val)
}
func test(input []int) int{
    bytes, _ := wasm.ReadBytes("test.wasm")
    instance, _ := wasm.NewInstance(bytes)
    defer instance.Close()
    // Gets the `sum` exported function from the WebAssembly instance.
    sum := instance.Exports["sum"]
    // result, _ := sum(input) // pass array of values
    fmt.Println(result)
    return result
}

It returning as 0. Is this feature also not supported?

check your error. You are getting zero because an error is occurring that you aren't printing in your example.
The main issue on the Go side of things is that you are passing the slice directly into the sum closure. You want to do this instead: sum(input...) which will pass the slice of ints in variadically. In other words it is akin to doing sum(input[0], input[1], ..., input[len(input)-1]).
However, I highly doubt this will work for any slice of ints because the underlying Wasm function that implements sum cannot be variadic, as there is no variadic specification in Wasm currently.

When I pass variadic function like this func test(input ...interface{}) and pass it to sum(input...) it works.
But i was figuring out the way to pass complete array at a time if i don't know the array length or it is automated by some other function. ( passing as slices will work test(val[0], test[1])

@AchalaSB have you considered encoding the array into protobuf or json and then just passing it normally as offset and length? that makes life easier :)

yes. In my use case have considered encoded json array. Actually those inputs i'm reading from json file. That is why I said i may not know the length.
Anyway variadic function works better. will follow that for now

Then, I'm closing this issue :-). Feel free to re-open if you have any more questions!


ncurses: configure: error: Cannot link with GPM library

[x] Ran brew update and retried your prior step?
[x] Ran brew doctor, fixed as many issues as possible and retried your prior step?

Bug reports:
ncurses is required for many packages, but brew install ncurses fails with the following:
Last 15 lines from /home/bjsm/.cache/Homebrew/Logs/ncurses/01.configure:
checking if you want to build shared libraries... yes
checking if you want to build static libraries... yes
checking if you want to build debug libraries... yes
checking if you want to build profiling libraries... no
checking if you want to build C++ shared libraries... no
checking for specified models...  shared normal debug
checking for default model... shared
checking if you want to have a library-prefix... auto
checking for PATH separator... :
checking if you want to build a separate terminfo library... no
checking if you want to build a separate tic library... no
checking if you want to link with the GPM mouse library... maybe
checking for gpm.h... yes
checking for Gpm_Open in -lgpm... no
configure: error: Cannot link with GPM library

READ THIS: https://github.com/Linuxbrew/linuxbrew/blob/master/share/doc/homebrew/Troubleshooting.md#troubleshooting
If reporting this issue please do so at (not Homebrew/homebrew):
  https://github.com/Homebrew/homebrew-dupes/issues

These open issues may also help:
Fish: depends on ncurses https://github.com/Linuxbrew/linuxbrew/pull/1080
Fish depends on ncurses https://github.com/Linuxbrew/linuxbrew/issues/1079

libgpm itself requires ncurses, so brew install libgpm doesn't help.
gist-logs link:
https://gist.github.com/86e900649aeb67b561041639bda39c39.

Wow, a circular dependency between ncurses and libgpm? That's not fun at all. Really not. For weird situations like this Linux From Scratch usually has helpful information. Off the top of my head, I'd suggest trying adding an option like --without-gpm (you'll have to check its exact name with ./configure --help) to the configure command of ncurses.

--without-gpm appears to be the correct flag, but brew install ncurses --without-gpm fails with the same error.
It would appear that that flag is not being passed through to the configure script.

I am able to build and install ncurses myself with the --without-gpm flag.

I was able to solve my issue by adding that flag directly to the formula.  I am curious why the flag wasn't being passed through, though.

You do have to add the flag to the formula. There's no automatic passing of brew command line arguments to the configure command. Please submit a pull request.


expression query returns empty

if I use any expression query like:
http://localhost:3011/api/cached?criteria={name:{'$gte':amazonitem}}
it returns empty.
can you illustrate why? and give an example if I am wrong.

Hi, have you tried to use "correct" json notation:
http://localhost:3011/api/cached?criteria={'name':{'$gte':'amazonitem'}}?
And how can amazonitem be greater than name? Have you tried that query on the database directly? What does it return?


Use p2p to fetch blocks and headers

Uses the p2p interface to fetch block hashes and headers much more efficiently. Fetches headers 2k at a time, and fetches blocks in chunks of 10.
Syncs the first 200k or so blocks much faster, but haven't benchmarked the whole chain. I don't suspect it will be much more gain for the whole chain since most of the time will be spent indexing to the db, and blocks are already being fetched in the background. However, this is a much more efficient way of fetching this data.
Still have to fix tests because they don't support connecting via p2p yet.
Closes https://github.com/casey/ord/issues/1502.

Might fix issues related to https://github.com/casey/ord/issues/1377, since those look like the RPC threads are falling over from too many requests.
I haven't fully groked the indexing logic for fetching raw transactions, but if we are doing a lot of getrawtransaction RPC calls it might be worth using REST interface for that. Benchmarking a getrawtransaction RPC vs a REST binary request for a large tx shows REST request is about 2x faster.

This is awesome! Keep in mind that https://github.com/casey/ord/issues/1364 will wind up making sync faster.
Some thoughts!

I don't want to make things much more complicated, so keeping an initial PR as simple and focused as possible would be great.
We need to make sure that they must use a trusted BTC node, since ord doesn't enforce consensus.
All our tests run against our own mocked bitcoind, which is in test-bitcoincore-rpc. This is required to keep tests fast.  As long as they stay fast and comprehensive, the method we use for testing doesn't matter, but one way to keep them fast would be to add P2P protocol support to test-bitcoincore-rpc. It only needs the minimum for the tests to work, so doesn't have to have comprehensive support for the full protocol.
I didn't even know that bitcoind had a rest interface. Is it documented anywhere? How is it different from the JSON RPC interface? I love rest and prefer it over JSON RPC now that I've had experience with both.

Thanks @casey.

Sorry for the REST tangent, that can be a follow-up.
Right. I suppose we can just use the same address as rpc and just specify the p2p port as an option.
Ok, will give that a shot.
REST is documented here. I have an implementation that we can pull in as a cargo crate here. But, that uses async/await so not sure if it will be useful in this project. REST is unauthenticated and doesn't have to go through the json parsing, so is much faster. It can also reply with raw binary data which is faster to serialize and smaller to send over the wire. But it also only shares public data, so we can't use it for things like the wallet. If https://github.com/bitcoin/bitcoin/pull/26415 was merged then I think that would be an even better candidate to fetch blocks than p2p, but the requirement to deserialize the block before serializing back into binary makes it slower than using the p2p endpoint right now. We can query it for utxo set data as well, but not sure what your ideas are for #1364.

@casey I think this is ready for a closer look.

@andrewtoth Awesome, thank you! Putting it on the project backlog so it doesn't fall off our radar.

WOW thank you! I was ticking <1 block per second. Found this and pulled your changes, now at ~50-100 per second. This should definitely be a top priority as most questions on the discord are about this right now (3 days - 2 weeks to index is rough).
Quick note: commit seems to happen every 5k blocks & takes about 3 minutes on my machine (I believe that's what was causing the json-rpc errors before this patch). This may confuse users when they see the block count hang for some time - I killed & restarted indexing myself a few times thinking that it had frozen.

After 16 hours the index is at block 420001 and moving very slowly, no errors/crashes yet.
I am on p2p-indexing branch git cloned from andrewtoth/ord. @andrewtoth

This has got a lot of successful testing from users in #1648.

This has got a lot of successful testing from users in #1648.

With rest=1 and on a speedup-improvements branch I got 99% of all blocks in a few minutes.
 :)

One user had set maxuploadtarget to something too low in their bitcoin.conf, and so this disallowed the p2p interface from serving everything it needs to. We should advise users to add whitebind=<IP_ADDRESS>:8333 to their config files.

@casey We should definitely consider this, did an initial review and test.

I'm sorry for taking so long to collect my thoughts on this! Bitcoin's RPC and P2P interfaces are very different. The P2P interfaces is an untrusted interface used between full nodes on the bitcoin network. The RPC interfaces is an authenticated interface between a client and a trusted node. Using the P2P interface for applications where the node must be trusted is dicey. You can use it to directly connect to a trusted node, but you can also use it to connect to a totally untrusted node. I think that expressing this subtle difference to our users would be hard, and that lots of people, because running a full node is challenging, would try to connect to some random node, and have their security degraded as a result.
Also #1364 fixes the same issue as this would, namely, speeding up initial sync, and is a very clean and low-impact chain that doesn't introduce another connection to manage. So, I would like to see that implemented first, before deciding if also using P2P or REST is a good idea.
Also, keeping ord simple to configure is important (The degens are dying out there 😆) and every new outgoing connection introduces code, configuration, and  maintainence complexity.
Sorry for not figuring this out sooner, it's only been in the last few days that I've been able to sit down and review PRs in detail, and previously I didn't give it enough attention.
Let's make sure we nail down the following in an issue before anyone writes a big PR:

What is the problem we're trying to solve or the benefit we're trying to eliminate?
What is the best design?
What is the testing strategy?

Sorry to close! We're still figuring out this whole massive open source project thing ourselves!

@casey disappointing conclusion. I disagree with your reasoning.
This requires no other configuration options. It works out of the box. It's been tested by many users already without problems. The only configuration issue was someone who had maxuploadtarget already set, which is obviously an advanced configuration option.
This patch would require using the same node as the RPC connection. It cannot be used to connect to a different node because only the port is configurable, not the host.
It is much more efficient for all the reasons stated in the initial issue #1502 . Blocks are fetched directly from disk without serialization steps, and can be requested in batches. It puts less stress on bitcoind and is faster.
Without this change, #1364 would be making RPC requests on multiple threads. This can overload the RPC workqueue and cause disconnections.
You can see the issues with fetching blocks via RPC in the updater code. There is a lot of code that has retries, and users are complaining that it aborts after retrying forever.

Does anyone know how much space ord index files take and where they're located?


Matrix sigunp functions added as modular file

UwU pwease use my code for adding Mwatwix accwounts

Good enough, will fix after merge


YT songs fetch

Added a React app for fetch song based on search text along with links

Migration of search api from javascript to c++


Fix 1549

WA for issue #1549
Changed IsApplicable in BnBwdTrainingSpatialSingle and BnBwdTrainingSpatialMultiple.
This kernel contains dead code if MIOPEN_USE_FP16=true

@shurale-nkn let me review this as well.

@shurale-nkn let me review this as well.

I can't. Only @junliume  or JD can add you as reviewer to any PR

This feature is only for the owner of the repository, not for the author of the pull request.
The pull request author still can't request anyone to review his request.

Organization members can also request a pull request review from a team with read access to the repository.

@shurale-nkn Please also replace [WA] to [WORKAROUND] in the PR's title. Thanks.


[Super Errors] - refmt preprocessing is eating syntax error messages

Code:
open Bs_webapi.Dom;

let html = Document.querySelector "html" document;

switch html {
  | Some element => Element.setAttribute "style" "background: red" element
  | None "" /* instead of `| Node => ()` */
};

Shell:
Error: 614: <SYNTAX ERROR>

  We've found a bug for you!
  /.../reason/basic/src/demo.re

  Is there an error before this one? If so, it's likely a syntax error.
  The more relevant message should be just above!
  If it's not, please file an issue here:
  github.com/facebook/reason/issues

  Error while running external preprocessor
Command line: /Users/raphael.rafatpanah/Code/reason/basic/node_modules/bs-platform/bin/refmt.exe --print binary '/Users/raphael.rafatpanah/Code/reason/basic/src/demo.re' > /var/folders/nx/t1yvhq8s3vb2lryj9b6pxjm80000gp/T/ocamlpp0f1d97

ninja: error: rebuilding 'build.ninja': subcommand failed
>>>> Finish compiling

I'm using VSCode on Mac OS X and am using:
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>

with the following (no refmt):
let f ?x =
  match x with
    | Some _ -> print_endline "Hello"
    | None ""

i'm getting:
/Users/yawar/code/wsh-bs-test/node_modules/bs-platform/bin/bsc.exe    -nostdlib -I '/Users/yawar/code/wsh-bs-test/node_modules/bs-platform/lib/ocaml' -bs-super-errors -no-alias-deps -color always -w -40+6+7+27+32..39+44+45 -c -o src/test.mlast -bs-syntax-only -bs-binary-ast /Users/yawar/code/wsh-bs-test/src/test.ml

  We've found a bug for you!
  /Users/yawar/code/wsh-bs-test/src/test.ml

  4 ┆ | None ""
  5 ┆

  Syntax error

It's a bad display. It's not eating the syntax error. <SYNTAX ERROR> is actually the error. Pretty bad; I'm making a big issue list for these kind of things.
Sorry about that, closing for now. The new super-errors message for this one is nicer I think


Added support for UniqueValue renderer in ArcGIS feature format

Implemented ArcGIS's UniqueValue renderer to allow features to be symbolized based on attribute values.
So instead having a simple renderer where all features are symbolized the same way, we can have a dynamic symbolization:

This apply to line, polygon and symbol marker symbolization.

should add @eringram as a codeowner to the map-layers-* pkgs as well

Done.


Install fails on Ubuntu LTS perhaps due to Fish?

Install details as follow with the final error message.
❯ curl -Ls https://github.com/r-lib/rig/releases/download/latest/rig-linux-latest.tar.gz |
        sudo tar xz -C /usr/local

ubuntu in fdsx in ~ via ஃ v1.6.7 via  v12.22.9 via 📐 v4.2.1
❯ rig add release
[INFO] Running `sudo` for adding new R versions. This might need your password.
[INFO] Downloading https://cdn.posit.co/r/ubuntu-2204/pkgs/r-4.3.0_1_amd64.deb -> /tmp/rig/r-4.3.0_1_amd64.deb
[INFO] Running apt-get update
[INFO] > Hit:1 https://download.docker.com/linux/ubuntu jammy InRelease
[INFO] > Hit:2 http://security.ubuntu.com/ubuntu jammy-security InRelease
[INFO] > Hit:3 http://nova.clouds.archive.ubuntu.com/ubuntu jammy InRelease
[INFO] > Hit:4 http://nova.clouds.archive.ubuntu.com/ubuntu jammy-updates InRelease
[INFO] > Hit:5 https://ppa.launchpadcontent.net/fish-shell/release-3/ubuntu jammy InRelease
[INFO] > Hit:6 http://nova.clouds.archive.ubuntu.com/ubuntu jammy-backports InRelease
[INFO] > Hit:7 https://ppa.launchpadcontent.net/neovim-ppa/stable/ubuntu jammy InRelease
[INFO] > Reading package lists...
[INFO] Running apt install
[INFO] >
[INFO] > WARNING: apt does not have a stable CLI interface. Use with caution in scripts.
[INFO] >
[INFO] > Reading package lists...
[INFO] > Building dependency tree...
[INFO] > Reading state information...
[INFO] > The following additional packages will be installed:
[INFO] >   libopenblas-dev libopenblas-pthread-dev libopenblas0 libopenblas0-pthread
[INFO] > The following NEW packages will be installed:
[INFO] >   libopenblas-dev libopenblas-pthread-dev libopenblas0 libopenblas0-pthread
[INFO] >   r-4.3.0
[INFO] > 0 upgraded, 5 newly installed, 0 to remove and 2 not upgraded.
[INFO] > Need to get 11.5 MB/73.0 MB of archives.
[INFO] > After this operation, 200 MB of additional disk space will be used.
[INFO] > Get:1 /tmp/rig/r-4.3.0_1_amd64.deb r-4.3.0 amd64 1 [61.5 MB]
[INFO] > Get:2 http://nova.clouds.archive.ubuntu.com/ubuntu jammy/universe amd64 libopenblas0-pthread amd64 0.3.20+ds-1 [6803 kB]
[INFO] > Get:3 http://nova.clouds.archive.ubuntu.com/ubuntu jammy/universe amd64 libopenblas0 amd64 0.3.20+ds-1 [6098 B]
[INFO] > Get:4 http://nova.clouds.archive.ubuntu.com/ubuntu jammy/universe amd64 libopenblas-pthread-dev amd64 0.3.20+ds-1 [4634 kB]
[INFO] > Get:5 http://nova.clouds.archive.ubuntu.com/ubuntu jammy/universe amd64 libopenblas-dev amd64 0.3.20+ds-1 [18.6 kB]
[INFO] > Fetched 11.5 MB in 12s (980 kB/s)
[INFO] > Selecting previously unselected package libopenblas0-pthread:amd64.
[INFO] > (Reading database ... 275134 files and directories currently installed.)
[INFO] > Preparing to unpack .../libopenblas0-pthread_0.3.20+ds-1_amd64.deb ...
[INFO] > Unpacking libopenblas0-pthread:amd64 (0.3.20+ds-1) ...
[INFO] > Selecting previously unselected package libopenblas0:amd64.
[INFO] > Preparing to unpack .../libopenblas0_0.3.20+ds-1_amd64.deb ...
[INFO] > Unpacking libopenblas0:amd64 (0.3.20+ds-1) ...
[INFO] > Selecting previously unselected package libopenblas-pthread-dev:amd64.
[INFO] > Preparing to unpack .../libopenblas-pthread-dev_0.3.20+ds-1_amd64.deb ...
[INFO] > Unpacking libopenblas-pthread-dev:amd64 (0.3.20+ds-1) ...
[INFO] > Selecting previously unselected package libopenblas-dev:amd64.
[INFO] > Preparing to unpack .../libopenblas-dev_0.3.20+ds-1_amd64.deb ...
[INFO] > Unpacking libopenblas-dev:amd64 (0.3.20+ds-1) ...
[INFO] > Selecting previously unselected package r-4.3.0.
[INFO] > Preparing to unpack /tmp/rig/r-4.3.0_1_amd64.deb ...
[INFO] > Unpacking r-4.3.0 (1) ...
[INFO] > Setting up libopenblas0-pthread:amd64 (0.3.20+ds-1) ...
[INFO] > update-alternatives: using /usr/lib/x86_64-linux-gnu/openblas-pthread/libopenblas.so.0 to provide /usr/lib/x86_64-linux-gnu/libopenblas.so.0 (libopenblas.so.0-x86_64-linux-gnu) in auto mode
[INFO] > Setting up libopenblas0:amd64 (0.3.20+ds-1) ...
[INFO] > Setting up libopenblas-pthread-dev:amd64 (0.3.20+ds-1) ...
[INFO] > update-alternatives: using /usr/lib/x86_64-linux-gnu/openblas-pthread/libopenblas.so to provide /usr/lib/x86_64-linux-gnu/libopenblas.so (libopenblas.so-x86_64-linux-gnu) in auto mode
[INFO] > Setting up libopenblas-dev:amd64 (0.3.20+ds-1) ...
[INFO] > Setting up r-4.3.0 (1) ...
[INFO] > Processing triggers for libc-bin (2.35-0ubuntu3.1) ...
[INFO] > NEEDRESTART-VER: 3.5
[INFO] > NEEDRESTART-KCUR: 5.15.0-69-generic
[INFO] > NEEDRESTART-KEXP: 5.15.0-71-generic
[INFO] > NEEDRESTART-KSTA: 3
[INFO] > NEEDRESTART-SVC: dbus.service
[INFO] > NEEDRESTART-SVC: networkd-dispatcher.service
[INFO] > NEEDRESTART-SVC: systemd-logind.service
[INFO] > NEEDRESTART-SVC: unattended-upgrades.service
[INFO] > NEEDRESTART-SVC<EMAIL_ADDRESS>[INFO] Adding /usr/local/bin/R-4.3.0 -> /opt/R/4.3.0/bin/R
[INFO] Adding R-release alias to R 4.3.0
[INFO] Setting default CRAN mirror
[INFO] Setting up RSPM
[INFO] Setting up automatic system requirements installation.
[INFO] Installing pak for R 4.3.0 (if not installed yet)
[INFO] > /opt/R/4.3.0/bin/R (line 4): Unsupported use of '='. In fish, please use 'set R_HOME_DIR /opt/R/4.3.0/lib/R'.
[INFO] > R_HOME_DIR=/opt/R/4.3.0/lib/R
[INFO] > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~^
[INFO] > warning: Error while reading file /opt/R/4.3.0/bin/R
[INFO] >
[ERROR] Failed to to install pak for R 4.3.0: command ["su", "ubuntu", "--", "/opt/R/4.3.0/bin/R", "-s", "--vanilla", "-e", "invisible()"] exited with code 127

Output of lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 22.04.2 LTS
Release:	22.04
Codename:	jammy

This is actually coming from R:
[INFO] > /opt/R/4.3.0/bin/R (line 4): Unsupported use of '='. In fish, please use 'set R_HOME_DIR /opt/R/4.3.0/lib/R'.

Does it work from bash? If it does, then once R is installed, can you use it from fish?

Hmm, sorry, I didn't catch that detail. I'll give it a go with Bash and see what happens, thanks @gaborcsardi.

It appears that you were right. I got it to work. Thanks for the guidance.


Invalid ViewComponent definition breaks tag helper discovery

This code breaks tag helper discovery in 15.3
namespace WebApplication1.ViewComponents
{
    public class PriorityListViewComponent : ViewComponent
    {
        private readonly ToDoContext db;

        public PriorityListViewComponent(ToDoContext context)
        {
            db = context;
        }

        public async Task<IViewComponentResult> InvokeAsync(
        int maxPriority, bool isDone)
        {
            var items = await GetItemsAsync(maxPriority, isDone);
            return View(items);
        }

        private Task<List<TodoItem>> GetItemsAsync(int maxPriority, bool isDone)
        {
            return db.ToDo.Where(x => x.IsDone == isDone &&
                                 x.Priority <= maxPriority).ToListAsync();
        }
    }
}

It's throwing an exception:
Microsoft.CodeAnalysis.Razor.ViewComponentTagHelperDescriptorFactory.GetInvokeMethodParameters(INamedTypeSymbol componentType)
   at Microsoft.CodeAnalysis.Razor.ViewComponentTagHelperDescriptorFactory.CreateDescriptor(INamedTypeSymbol type)
   at Microsoft.CodeAnalysis.Razor.DefaultTagHelperResolver.VisitViewComponents(Compilation compilation, IEnumerable`1 assemblyNameFilters, List`1 results)
   at Microsoft.CodeAnalysis.Razor.DefaultTagHelperResolver.GetTagHelpers(Compilation compilation, IEnumerable`1 assemblyNameFilters)
   at Microsoft.CodeAnalysis.Razor.TagHelperResolver.<GetTagHelpersAsync>d__1.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.CodeAnalysis.Remote.Razor.RazorLanguageService.<GetTagHelpersAsync>d__1.MoveNext()

We need to fix this for the VS 15.3 release.
Note that this is already fixed in dev and in 2.0.0/preview-2 https://github.com/aspnet/Razor/blob/rel/2.0.0-preview2/src/Microsoft.CodeAnalysis.Razor/ViewComponentTagHelperDescriptorFactory.cs
This is an error now, but doesn't throw an exception.

@rynowak is this the right milestone? Where does this work need to happen?

This will happen in the rel/15.3 branch along with the two other tooling bugs we fixed today 👍

OK, so which package version will it ship in, if any?

None. This is a VS-only fix.
This code was rewritten during preview 2 and this bug was (unknowingly) fixed at that time.
We branched for VS stabilization over a month ago so lots of things have been cleaned up and refactored since thing.

OK, I'll create a separate milestone for it then and put it in there.

We branched for VS stabilization over a month ago so lots of things have been cleaned up and refactored since thing.

Also, we all know this isn't an ideal situation... It's the reality to having two simultaneous ship cycles.

f5ecb9ee60db0d042931a83539a289fd3f11d23a


trajectory_dataset.py issues when training on MOT17

I have noticed a couple of issues:

The Dataloader  ( _load_data, _load_image_anns) is internally loading images ( img = cv2.imread(img_path)) which are  never used in the __getitem__ function therefore dramatically increasing the overhead of the Dataloader.

No bounding boxes are being returned by __getitem__  and the Dataloader is eventually terminated with a recursion depth error because _get_bboxes    cls_id > self.opt.num_classes is True due  to num_classes not being set in opts.py Adding  --"num_classes 1"  to  mot17_tracking.sh solves this.  i.e python train_prediction.py tracking --exp_id mot17_motion_model --dataset mot --dataset_version 17trainval --gpus 0 --num_classes 1

thanks for your comments. we will check if num_classes 1 is required in MOT17 training.


[kotlin] update outdated files

Hey,
I created this PR to help you with the outdated files to make CI pass in https://github.com/OpenAPITools/openapi-generator/pull/4518.
🙂

Thanks!!! :)


Python error: no attribute 'keccak_256'

I run this:
pip install -r requirements.txt
And I get this:
Requirement already satisfied: ecdsa in /usr/local/lib/python2.7/dist-packages (from -r requirements.txt (line 1))
Requirement already satisfied: pysha3 in /usr/local/lib/python2.7/dist-packages (from -r requirements.txt (line 2))
But I get an error when I run it:
./ethereum-wallet-generator.py
Traceback (most recent call last):
File "./ethereum-wallet-generator.py", line 9, in
keccak = sha3.keccak_256()
AttributeError: 'module' object has no attribute 'keccak_256'

Commit 9360c060780a0aed264bec09673641c205495517 should fix this issue.
I guess you don't have pysha3 latest version.
Does it works now?


Analyzer error when binding to RadioButton.IsChecked property in Avalonia 11.1.0-rc2 compared to 11.0.11

Describe the bug
While binding to the RadioButton.IsChecked property in XAML, Rider's analyzer mistakenly assumes that I intend to bind to the private IRadioButton.IsChecked property. This assumption results in an analyzer error in XAML, although the binding functions correctly at runtime. Additionally, there is no IntelliSense hint for IsChecked when typing it out in XAML.
To Reproduce
        <StackPanel>
            <RadioButton Name="radioButton" />
            <RadioButton />
            <NumericUpDown IsEnabled="{Binding #radioButton.IsChecked}" />
        </StackPanel>

Expected behavior
The analyzer should recognize that the binding is intended for the public ToggleButton.IsChecked property. Maybe there is a way to cast #radioButton to ToggleButton within the binding expression as a workaround?
Avalonia version
11.1.0-rc2
OS
Windows
Additional context

This is a Rider bug.
RadioButton.IsChecked property is inherited from ToggleButton.IsChecked which is still public.
But Rider mistakenly reads internal explicit interface implementation of another property with the same name: https://github.com/AvaloniaUI/Avalonia/blob/master/src/Avalonia.Controls/RadioButton.cs#L36
Can you open an issue in the Rider Youtrack? I will add it to the list of other issue https://github.com/AvaloniaUI/Avalonia/issues/10356 to track

Sure, as soon as YouTrack is out of read-only mode.

https://youtrack.jetbrains.com/issue/RIDER-115130/Rider-analyzer-error-when-binding-to-RadioButton.IsChecked-property-in-Avalonia-11.1.0


Build fails with the following error

Build failed.
Expected Behavior

Build should succeed.
Current Behavior

Build now fails with the following error.
Successfully built uwsgi pygraphviz watchdog django-allauth django-rest-auth coreschema inflection PyYAML validate-email strict-rfc3339 billiard dockerpty docopt texttable pathtools itypes pyrsistent
ERROR: docker-compose 1.23.2 has requirement jsonschema<3,>=2.5.1, but you'll have jsonschema 3.1.1 which is incompatible.
Installing collected packages: pytz, Django, uwsgi, djangorestframework, django-filter, itypes, uritemplate, idna, urllib3, chardet, certifi, requests, MarkupSafe, jinja2, coreschema, coreapi, ruamel.yaml.clib, ruamel.yaml, six, inflection, drf-yasg, PyYAML, jsonpointer, rfc3987, click, validate-email, strict-rfc3339, flex, pyrsistent, more-itertools, zipp, importlib-metadata, attrs, jsonschema, swagger-spec-validator, psycopg2-binary, holdup, django-extensions, pygraphviz, billiard, vine, amqp, kombu, celery, redis, dockerpty, cached-property, websocket-client, docopt, docker-pycreds, docker, texttable, docker-compose, argh, pathtools, watchdog, defusedxml, python3-openid, oauthlib, requests-oauthlib, django-allauth, django-rest-auth, PyJWT, djangorestframework-jwt, pyasn1, rsa, pyasn1-modules, cachetools, google-auth, python-dateutil, kubernetes, django-cors-headers
ERROR: docker-compose 1.23.2 has requirement requests!=2.11.0,!=2.12.2,!=2.18.0,<2.21,>=2.6.1, but you'll have requests 2.22.0 which is incompatible.
Possible Solution

Steps to Reproduce

Context (Environment)

OS:
Architecture:
Docker version:
Docker compose version:
Kubernetes version:

Detailed Description and log

Possible Implementation

@hightall I have tried to update docker-compose required components such as jsonschena and request to the allowable version, but the build still fails. The build produces so much warning and this python dependency thing becomes basically unsustainable, this needs to be addressed right now.
These are the changes I tried on my local env. still not working though
jsonschema<3,>=2.5.1 requests!=2.11.0,!=2.12.2,!=2.18.0,<2.21,>=2.6.1
I had to disable the hyperledger.cello to merge my patches. I will leave it off for now before we make the build process better.


Allow access to ResourceObject.id

Currently the ResourceObject.id property is private and can therefore not be accessed once the object is instantiated. Is this on purpose?
Rationale
I would like to be to access it, so the id can be used to (dynamically) generate a DELETE request like:
https://domain.ext/api/v1/resourceName/id/$id

Something like:
//application.php
$articleResource = ArticleResource::getInstance(123);
if( $apiClient->delete($articleResource)){
    echo 'deleted Article!';
}

//apiClient.php
public function delete(Resource $resource) : bool{
    $request = $this->someRequestBuilder->getDeleteRequest($articleResource)
    $response = $this->sendRequest($request);
    return $this->analyzeRespons($response);
}

Maybe the ResourceObject.id should not be accessed, that is possible, but in that case I would like to know "why" 😄

Hi @holtkamp,
The only purpose of the id field being private without any getter/setter is that I didn't have any meaningful use-case for being able to access these fields in mind.
Would it be OK for you if I added the id(), type(), attributes() and relationships() accessors as well as the setId() and setType() mutators?

Done in https://github.com/woohoolabs/yang/commit/fb539603f759a0fb410c66e65b0e186ee9eec75b :)


Prints twice all layers

Hi,
The summary module seems to behave weirdly on my model. The function outputs all my layers twice in duplicate fashion.
The code is the following:
import torch
import torch.nn as nn
from torchsummary import summary

class Model(nn.Module):

    def __init__(self):

        super(Model,self).__init__()

        K = 69
        L = 1000

        self.group1 = nn.Sequential(
            nn.Conv3d(3, K, kernel_size=3,padding=(1,1,1)),
            nn.LeakyReLU()
        )

        self.group2 = nn.Sequential(
            nn.Conv3d(K, L, kernel_size=4, padding=(1,1,1)),
            nn.MaxPool3d(2, stride=2,ceil_mode=True),
            nn.ReLU()
        )

        self._features = nn.Sequential(
            self.group1,
            self.group2
        )

    def forward(self,x):
        out = self._features(x)
        return out

if '__main__' == __name__:

    model = Model()
    inp = (3, 30, 30, 30)

    summary(model, inp, device='cpu')

The output is the following:
----------------------------------------------------------------
        Layer (type)               Output Shape         Param #
================================================================
            Conv3d-1       [-1, 69, 30, 30, 30]           5,658
            Conv3d-2       [-1, 69, 30, 30, 30]           5,658
         LeakyReLU-3       [-1, 69, 30, 30, 30]               0
         LeakyReLU-4       [-1, 69, 30, 30, 30]               0
            Conv3d-5     [-1, 1000, 29, 29, 29]       4,417,000
            Conv3d-6     [-1, 1000, 29, 29, 29]       4,417,000
         MaxPool3d-7     [-1, 1000, 15, 15, 15]               0
         MaxPool3d-8     [-1, 1000, 15, 15, 15]               0
              ReLU-9     [-1, 1000, 15, 15, 15]               0
             ReLU-10     [-1, 1000, 15, 15, 15]               0
================================================================
Total params: 8,845,316
Trainable params: 8,845,316
Non-trainable params: 0
----------------------------------------------------------------
Input size (MB): 0.31
Forward/backward pass size (MB): 532.00
Params size (MB): 33.74
Estimated Total Size (MB): 566.05
----------------------------------------------------------------

As you can see in the self.group1() I have one nn.Conv3d(). However, in the output I get Conv3d-1 and Conv3d-2 for that layer.
When model.apply(register_hook) runs it seems to already append twice the number of module.register_forward_hook(hook) expected in the hooks list.
Any clues about what is happening?
Thanks,
Matthias

@sksq96 Any clues? Thanks

I'm not entirely sure what that is happening, but you can get around the issue with this model structure instead. Notice that you no longer use the attributes group1 and group2. It does mean its not as neatly structured, but I wonder if its something wrong with what is being returned by register_forward_hook(hook)) . Not entirely sure though.
         self._features = nn.Sequential(
            nn.Sequential(
                nn.Conv3d(3, K, kernel_size=3,padding=(1,1,1)),
                nn.LeakyReLU()
            ),
            nn.Sequential(
                nn.Conv3d(K, L, kernel_size=4, padding=(1,1,1)),
                nn.MaxPool3d(2, stride=2,ceil_mode=True),
                nn.ReLU()
            )
        )

@mangotree3
This is not a pytorch-sumamry's bug.
This is due to the implementation of PyTorch, and your unintended results are that self.group1 and self.group2 are declared as instance variables of Model.
Actually, when I change self.group1 and self.group2 to group1 and group2 and execute, I get the intended results:
----------------------------------------------------------------
        Layer (type)               Output Shape         Param #
================================================================
            Conv3d-1       [-1, 69, 30, 30, 30]           5,658
         LeakyReLU-2       [-1, 69, 30, 30, 30]               0
            Conv3d-3     [-1, 1000, 29, 29, 29]       4,417,000
         MaxPool3d-4     [-1, 1000, 15, 15, 15]               0
              ReLU-5     [-1, 1000, 15, 15, 15]               0
================================================================
Total params: 4,422,658
Trainable params: 4,422,658
Non-trainable params: 0
----------------------------------------------------------------
Input size (MB): 0.31
Forward/backward pass size (MB): 266.00
Params size (MB): 16.87
Estimated Total Size (MB): 283.18
----------------------------------------------------------------

I got the same issue, not a big problem, but definitely a bug. I have a model from Colab which I am not running on Google Cloud, changed noting on the notebook, but also got it to print twice.

@luisdiaz1997
Hmmm, can you share the code?

https://github.com/luisdiaz1997/DataScienceCourse/blob/master/Week_10_solutions.ipynb
In Colab it only shows once, but in my cloud it shows twice, is worth to note that in Colab I did not install it from here, in my cloud I had to.
It doesn't show up twice if I do
some_var = summary(model, (some_shape) )
if I save it to a variable it only prints once.
I assume there is a print() inside summary()

I assume there is a print() inside summary()

It is true that there is a print() in summary(), but since summary() has no return value, I cannot understand its behavior:/

Thanks everyone for your comments and answers. I agree with @skmatz there not much one can do other than adapt one model's code. The package is just built on top of pytorch and one has to work with the tools they give you... I confirm that the problem effectively lies in having a  double self assignment of the models parts in the __init__().
As mentioned by @Naireen, this works:
class Model(nn.Module):
    def __init__(self):
        super(Model,self).__init__()

        K = 69
        L = 1000
        self._features = nn.Sequential(
            nn.Sequential(
                nn.Conv3d(3, K, kernel_size=3,padding=(1,1,1)),
                nn.LeakyReLU()
            )
            nn.Sequential(
                nn.Conv3d(K, L, kernel_size=4, padding=(1,1,1)),
                nn.MaxPool3d(2, stride=2,ceil_mode=True),
                nn.ReLU()
            )
        )

    def forward(self,x):
        out = self._features(x)
        return out

Alternatively this also works:
class Model(nn.Module):

    def __init__(self):

        super(Model,self).__init__()

        K = 69
        L = 1000

        self.group1 = nn.Sequential(
            nn.Conv3d(3, K, kernel_size=3,padding=(1,1,1)),
            nn.LeakyReLU()
        )

        self.group2 = nn.Sequential(
            nn.Conv3d(K, L, kernel_size=4, padding=(1,1,1)),
            nn.MaxPool3d(2, stride=2,ceil_mode=True),
            nn.ReLU()
        )

    def forward(self,x):
        x = self.group1(x)
        x = self.group2(x)
        out = self._features(x)
        return out

However, this doesn't work (prints double) because you have two elements in the __init__:
class Model(nn.Module):

    def __init__(self):

        super(Model,self).__init__()

        K = 69
        L = 1000

        self.group1 = nn.Sequential(
            nn.Conv3d(3, K, kernel_size=3,padding=(1,1,1)),
            nn.LeakyReLU()
        )

        self.group2 = nn.Sequential(
            nn.Conv3d(K, L, kernel_size=4, padding=(1,1,1)),
            nn.MaxPool3d(2, stride=2,ceil_mode=True),
            nn.ReLU()
        )

        self._features = nn.Sequential(
            self.group1,
            self.group2
        )

    def forward(self,x):
        out = self._features(x)
        return out


Integration with scalite

Scalatags provides great type safety to html template. But it also begets loads of parentheses/curly braces .
I wonder whether Scalite is still under maintenance so that we can try out its integration with Scalatags ?

go ahead and try it out

This is impossible for now. Because function call cannot be sugared into block


Bug: The go test command cannot parse custom args with flag.Parse()

What version of Go are you using (go version)?

go version go1.14 darwin/amd64

Does this issue reproduce with the latest release?
yes 1.14
What operating system and processor architecture are you using (go env)?
go env Output
GO111MODULE="on"
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/alex/Library/Caches/go-build"
GOENV="/Users/alex/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/alex/go"
GOPRIVATE=""
GOPROXY="direct"
GOROOT="/usr/local/Cellar/go/1.14/libexec"
GOSUMDB="off"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.14/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/alex/go/src/github.com/channelmeter/cp-go-api/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/qh/l8tr6tfd3gv7fmdg0b5z0p340000gn/T/go-build971392475=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
I ran:
go test -v github.com/cm/some-pkg -- --env=local

What did you expect to see?
I expected flag.Parse() to capture the flag
What did you see instead?
the --env flag is not captured, even though it appears in the args list when I use:
  fmt.Println(len(os.Args), os.Args)

I see:
[/var/folders/qh/l8tr6tfd3gv7fmdg0b5z0p340000gn/T/go-build589736069/b001/roles.test -test.timeout=10m0s -test.v=true  -- --env=local]

I put the Q on SO first:
https://stackoverflow.com/questions/61944527/how-to-pass-custom-flags-to-go-test?noredirect=1#comment109560255_61944527

This is not a bug. From the documentation for package flag:

Flag parsing stops just before the first non-flag argument ("-" is a non-flag argument) or after the terminator "--".

and your --env flag appears after the terminator "--", so it's correctly not being parsed.

This is a common idiom you will see in many command line tools, not just Go.
-- terminates the argument parser explicitly, because sometimes you have arguments, such as file names, that start with - which you want to capture as a value instead of a flag.

well then you know what i am trying to do. so what is the solution?

Flags in tests are really no different than in any other program, except that you don't get to call flag.Parse() yourself:
package main

import (
	"flag"
	"log"
	"os"
	"testing"
)

var x = flag.Bool("env", false, "")

// run with `go test --env`
func TestFlags(t *testing.T) {
	log.Println(os.Args)     // /tmp/go-build197321626/b001/tmp.lW3vriEIbp.test -test.timeout=10m0s --env
	log.Println(flag.Args()) // []
	log.Println(*x)          // true
}

To treat arguments as flags create a dedicated FlagSet:
// run with `go test -- --env`
func TestArgs(t *testing.T) {
	fs := flag.NewFlagSet("my-test", flag.ContinueOnError)
	y := fs.Bool("env", false, "")

	log.Println(os.Args)     // [/tmp/go-build483180072/b001/tmp.lW3vriEIbp.test -test.timeout=10m0s -- --env]
	log.Println(flag.Args()) // [--env]

	if err := fs.Parse(flag.Args()); err != nil {
		t.Fatal(err)
	}

	log.Println(*y) // true
}

The docs for the interaction between testing and flag.Parse were a bit ambiguous, so I made them clearer a couple of weeks ago in https://github.com/golang/go/issues/38952.
If anything is still confusing, please suggest doc improvements. Aside from that, I agree that the current behavior seems to be correct.


feat: Page templates.

Began implementing https://github.com/vapid/vapid/issues/158 here so we can play with it and get a feel for the new data model features 🙂
Data Model Updates

[x] Rename sections table to templates
[x]  Add a type column in the templates table of type ENUM(setting, form, page, collection).
[x] Set all multiple=1 templates to type collection
[x] Set all form=1 templates to type form
[x] Set all other template types to setting
[x] Drop form and multiple columns from templates
[x] Rename the records.section_id column to records.template_id
[x] Generate type=page template entries for each discovered /[^_]*.html file.
[x] Generate a default page record instance for each template entry

Template Parser Updates

[x] Use the Handlebars parser ASTs for improved template parsing, hash validation, template rewrites, partials support, and page rendering
[x] Discover all data paths that start with this. inside of page templates and add them to the page's template entry as fields.
[x] Discover all data paths with no context prefix and log a deprecation warning. Automatically insert general. as the path context to avoid this. conflicts.
[x] Discover all data paths with the _ pseudo-private prefix and log a deprecation warning. Automatically rewrite _ paths to @ for proper Handlebars data attribute rendering support.
[x] Add the {{#collection name as |value|}} block helper to the runtime and analyzer.
[x] Update the {{#section name multiple=true}} helper to analyze a type=collection
[ ] Deprecate and the {{#section name multiple=true}}
[ ] Rewrite the {{#section name multiple=true}} to {{#collection name as |value|}} (this is hard and requires children path rewrites... may not be reasonably possible.)

Dashboard Updates

[x] Display Pages in the admin dashboard as single records.
[ ] Allow users to create new Pages, selecting from the project Page templates as data structure options.
[ ] Allow users to delete Pages (including the default created ones)
[ ] Allow users to create new Collection instances, selecting from the available templates.
[ ] Allow users to delete Collection instances – including the default Collection created.
[ ] Add collection directive for the dashboard – users may select from a list of collections that use the same template.

Came up with an idea as I was working here: What if we allow {{#collection this.name as |value|}} (note: this.name instead of name). This would make a page-specific collection. It won't be shared between other pages and will show up as a direct child of the page in the dashboard. Essentially, each page instance will get a private collection to reference.

Closing in favor of https://github.com/amiller-gh/vapid/pull/2


Scope package.json to digitalbazaar org, publish to npm

At the moment, ezcap is not published. We should rename package to be scoped (@digitalbazaar/ezcap) and publish it to NPM.

Published, closing.


Infographics

@kand

👍 thanks

Coverage decreased (-0.1%) to 86.147% when pulling 040683414f25fc63b03ba5b845257cedadceaa6a on infographics into b06f4d33d4a6e64eed0c82228227e1b559becca2 on master.


feat(construct): Add new construct - textToSql

Fixes #
aws-text-to-sql construct
This CDK construct leverages state-of-the-art generative AI capabilities to facilitate natural language-based SQL query generation. It seamlessly integrates with both Amazon Aurora and Amazon RDS database services, currently supporting MySQL databases. If you have an existing MySQL database hosted on an Amazon Aurora or Amazon RDS instance, you can seamlessly integrate it with the construct. Alternatively, the construct can provision the necessary database infrastructure for you.
To address the inherent ambiguity present in natural language, the system employs a sophisticated semantic linking mechanism. This mechanism utilizes a knowledge layer loaded from configuration files or an Amazon Bedrock knowledge base to reformulate the user's query, disambiguating and clarifying the intent. The system then prompts the user for feedback on the reformulated query, ensuring accurate interpretation of the user's intent. Upon user approval of the reformulated query, the construct generates the corresponding SQL query and presents it to the user for further validation.
Once the user approves the generated SQL query, they have the option to execute it against the target database. In the event of any issues with the generated query, the construct is equipped with an autocorrection mechanism that allows for a configurable number of retries.
The system's architecture is built upon a serverless workflow orchestrated by AWS Step Functions and AWS Lambda, triggered by an Amazon EventBridge event. The AWS Step Functions workflow publishes a task token on an Amazon Simple Queue Service (SQS) queue and awaits user feedback, ensuring a seamless and efficient user experience. The feedback strategy is configurable in config files and can be skipped by updating the config files.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.

Seems build is failing: https://github.com/awslabs/generative-ai-cdk-constructs/actions/runs/10392747887/job/28778696219?pr=631#step:5:13

Seems build is failing: https://github.com/awslabs/generative-ai-cdk-constructs/actions/runs/10392747887/job/28778696219?pr=631#step:5:13

Now .NET and Go packages:
https://github.com/awslabs/generative-ai-cdk-constructs/actions/runs/10423426486/job/28870254294?pr=631

Update of status: since some deps are not available in go/.net, the construct will be modified to remove those

Update of status: since some deps are not available in go/.net, the construct will be modified to remove those

Done 0fcad21c45d028d3380d2fedcd8d89be1de0c306

Seems build is failing: https://github.com/awslabs/generative-ai-cdk-constructs/actions/runs/10392747887/job/28778696219?pr=631#step:5:13

Now .NET and Go packages: https://github.com/awslabs/generative-ai-cdk-constructs/actions/runs/10423426486/job/28870254294?pr=631

resolved 0fcad21c45d028d3380d2fedcd8d89be1de0c306


[Fix #2898] Allow method definitions inside Class.new(SuperClass)

Before submitting a PR make sure the following are checked:

[x] Wrote [good commit messages][3].
[x] Used the same coding conventions as the rest of the project.
[x] Feature branch is up-to-date with master (if not rebase)
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog accordingly. See changelog entry format.
[x] All tests are passing.
[x] The new code doesn't generate RuboCop offenses.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.

:+1:


When I encounter elements that change frequently, how should I click?

generally：
chromedp.Click(#react-root > section > main > div > div > div > section > div > button, chromedp.ByID),
Sometimes it's like this,
chromedp.Click(#react-root > div > div > section > main > div > div > div > section > div > button, chromedp.ByID),
How should I achieve accurate clicks, ask the author to help me

<button class="sqdOP  L3NKy   y3zKF     " type="button">保存信息</button>
This is the button I finally need to click

Use the CSS selector list to select all the matching nodes:
 chromedp.Click("#react-root>section>main>div>div>div>section>div>button, #react-root>div>div>section>main>div>div>div>section>div>button", chromedp.ByQuery),`


Handle bundle custome extensions

The cnab spec provides a mechanism for bundle authors to define custom extensions. It looks like cnab-to-oci currently does not handle these when a bundle is pushed, so this data would be lost on a pull.
Porter plans on storing some things in the extensions and would like to see this persisted.

Closed with #46


feat: swap webview content size listener

✅ Checklist

[x] npx changeset was attached.
[ ] Covered by automatic tests.
[ ] Impact of the changes:

...

📝 Description
Adds web view listener for content size, size that will allow the app to set the height and let the web view display entire contents into the swap page.
❓ Context

JIRA or GitHub link:

🧐 Checklist for the PR Reviewers

The code aligns with the requirements described in the linked JIRA or GitHub issue.
The PR description clearly documents the changes made and explains any technical trade-offs or design decisions.
There are no undocumented trade-offs, technical debt, or maintainability issues.
The PR has been tested thoroughly, and any potential edge cases have been considered and handled.
Any new dependencies have been justified and documented.
Performance considerations have been taken into account. (changes have been profiled or benchmarked if necessary)

/generate-screenshots

I'm not sure about this change, is it really necessary ? Couldn't we update the webview to adapt to the content instead ?
It feels very weird to have an app being able to set the webview size, it's similar to allowing a website to resize my browser, doesn't feel right
Also if you really need this you could do everything in SwapWebView.tsx IMO and avoid touching the base Web3AppWebview

Hey, thanks for review 🙏
I'll link the conversation internally, basically, we need to allow the web view to resize based on the content, to allow swap page scroll on content, rather than only inside the web view.
The quickest way, was to have an observer trigger inside the web app and a listener inside the desktop app, that way every time the content is resized we update the web view with the new size. That's strictly related to Demo1 and will be removed starting with Demo3.
Happy to discuss a better way 🙏


Higurashi When They Cry

Can this be fixed? At the beginning of a sentence, two characters always appear.
Comparison:

This is a particularly nasty issue because not only is the game using the same component to display character name and text, but they input encoded text into that text component, which is why '\n' is not interpreted as a newline, like it should.
In order to provide a real fix for this game I would need to look into how the ADV engine used in that game handles updating text components and displays text.
For now you may be able to get away with adding a text file alongside the other translation files that contains
sr:"^([\S\s]+)\\\\n([\S\s]+)$"=$1\n$2
sr:"^([\S\s]+)\\\\n([\S\s]+)\\\\n([\S\s]+)$"=$1\n$2\n$3
sr:"^([\S\s]+)\\\\n([\S\s]+)\\\\n([\S\s]+)\\\\n([\S\s]+)$"=$1\n$2\n$3\n$4
sr:"^([\S\s]+)\\\\n([\S\s]+)\\\\n([\S\s]+)\\\\n([\S\s]+)$\\\\n([\S\s]+)$"=$1\n$2\n$3\n$4\n$5

This may or may not reduce the quality of generated translations.
You should also remove all already generated translations.

This is now solvable through the _Preprocessor.txt file.
Check the following https://github.com/bbepis/XUnity.AutoTranslator/issues/35#issuecomment-557924677


[Bug] Query data date format output is incorrect

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
1.0
What's Wrong?
create table statement：
CREATE TABLE IF NOT EXISTS expamle
(
    `user_id` LARGEINT NOT NULL COMMENT "用户id",
    `username` VARCHAR(50) NOT NULL COMMENT "用户昵称",
    `city` VARCHAR(20) COMMENT "用户所在城市",
    `age` SMALLINT COMMENT "用户年龄",
    `sex` TINYINT COMMENT "用户性别",
    `phone` LARGEINT COMMENT "用户电话",
    `address` VARCHAR(500) COMMENT "用户地址",
    `effect_date` DATE COMMENT "用户注册时间",
    `create_time` DATETIME COMMENT "用户注册时间"
)
UNIQUE KEY(`user_id`, `username`)
DISTRIBUTED BY HASH(`user_id`) BUCKETS 1
PROPERTIES (
"replication_allocation" = "tag.location.default: 1"
);

insert :
INSERT INTO `expamle` VALUES ('10000','zhangfeng1','北京',20,0,'13347420869','test_123_234_446',NULL,'2017-10-01 07:00:00'),('10001','zhangfeng2','北京',30,1,'13347420869','test_123_234_446','2022-05-18','2017-10-01 17:05:45'),('10002','zhangfeng3','上海',20,1,'13347420869','test_123_234_446','2022-05-18','2017-10-02 12:59:12'),('10003','zhangfeng4','广州',32,0,'13347420869','test_123_234_446','2022-05-18','2017-10-02 11:20:00'),('10004','zhangfeng5','深圳',35,0,'13347420869','test_123_234_446','2022-05-18','2017-10-01 10:00:15'),('10004','zhangfeng6','深圳',35,0,'13347420869','test_123_234_446','2022-05-18','2017-10-03 10:20:22');

mysql> select if(effect_date is null,create_time,effect_date) from expamle limit 11;
+---------------------------------------------------------+
| if(`effect_date` IS NULL, `create_time`, `effect_date`) |
+---------------------------------------------------------+
|<PHONE_NUMBER>0000 |
|                                                20220518 |
|                                                20220518 |
|                                                20220518 |
|                                                20220518 |
|                                                20220518 |
+---------------------------------------------------------+
6 rows in set (0.01 sec)

mysql> select create_time,effect_date from expamle;
+---------------------+-------------+
| create_time         | effect_date |
+---------------------+-------------+
| 2017-10-01 07:00:00 | NULL        |
| 2017-10-01 17:05:45 | 2022-05-18  |
| 2017-10-02 12:59:12 | 2022-05-18  |
| 2017-10-02 11:20:00 | 2022-05-18  |
| 2017-10-01 10:00:15 | 2022-05-18  |
| 2017-10-03 10:20:22 | 2022-05-18  |
+---------------------+-------------+
6 rows in set (0.03 sec)

What You Expected?
Expected output date should be yyyy-mm-dd or yyyy-mm-dd HH:mm:ss
How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

If has a problem with two different types of inference, and I'll fix it soon.


Update validation.rst

Since a long time this filed was empty so i try to fill with some of my knowledge...TIA in Advance

Hi @PawanKJ, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

@PawanKJ This article is actually already in PR: see https://github.com/aspnet/Docs/pull/1053. We'd love to get your feedback on the article, so feel free to comment on it.


Query Rewriting

Query Rewriting in Postgres enables substring replacement of query strings. You can maintain a set of rewrite rules in a database table, and modify the rules on the fly without having to reindex.
An example from the docs linked above:
CREATE TABLE aliases (t tsquery PRIMARY KEY, s tsquery);
INSERT INTO aliases VALUES('a', 'c');

SELECT ts_rewrite('a & b'::tsquery, 'SELECT t,s FROM aliases');
 ts_rewrite
------------
 'b' & 'c'

Here's an idea of how this could work:
class Person < ActiveRecord::Base
  include PgSearch::Model
  pg_search_scope search_by_full_title:,
                  against: [:title, :first_name, :middle_name, :last_name, :suffix],
                  using: {
                    tsearch: {
                      dictionary: "simple",
                      rewrite: "SELECT t, s FROM aliases"
                    }
                  }
end

class Alias < ActiveRecord::Base
end

Alias.create t: "Rev", s: "Rev|Reverend"
Alias.create t: "Dr", s: "Dr|Doctor"
Alias.create t: "Jr", s: "Jr|Junior"

mlk = Person.create title: "The Rev. Dr.", first_name: "Martin",
  middle_name: "Luther", last_name: "King", suffix: "Jr."

Person.by_full_title("Dr Martin Luther King Jr") # => [mkl]
Person.by_full_title("Reverend Doctor Martin Luther King Junior") # => [mkl]

I am interested in adding support for this to the tsearch feature. Would a pull request for this feature be welcome? Any feedback on this interface?

This seems like a reasonable enough interface. I'm not a huge fan of strings of hard-coded SQL, but I am not sure how this could be avoided. Also, it would be interesting to figure out how to interpolate in the query like in the documentation you linked to help with very large alias tables.


Fix events id mapping

Description & motivation
The snowplow_events model was fully incremental, meaning it was not backfilling user_ids for old events.
This has been fixed by breaking it into snowplow_events_tmp which is incremental and snowplow_events which implements the user stitching
Checklist

[X] I have verified that these changes work locally
[X] I have updated the README.md (if applicable)
[X] I have added tests & descriptions to my models (and macros if applicable)

do we lose the performance benefit of incremental if one of the tables is a full load each time? it is possible to full load once then be incremental after that?
thks


Bundle updates for qwiklabs scripts

Mark the script type as required, which in the code, it has been all along and document the new option for the qwiklabs script type.

cc: @supersam654

I believe the current desired procedure is that humans/authors take their cues on what to commit to github from main/github-lab-bundle-spec.md, while the the file api/lab-bundle-spec.md is meant to define the output of GW (aka input of Website), and we never want an author to lay eyes upon it. :)  Of course, the two are almost identical. :) TLDR, I think you also need to update https://github.com/CloudVLab/qwiklabs-content-bundle-spec/blob/main/github-lab-bundle-spec.md#gcp-project-gcp_project
correct, @eniskonuk ?

I believe the current desired procedure is that humans/authors take their cues on what to commit to github from main/github-lab-bundle-spec.md, while the the file api/lab-bundle-spec.md is meant to define the output of GW (aka input of Website), and we never want an author to lay eyes upon it. :) Of course, the two are almost identical. :) TLDR, I think you also need to update https://github.com/CloudVLab/qwiklabs-content-bundle-spec/blob/main/github-lab-bundle-spec.md#gcp-project-gcp_project
correct, @eniskonuk ?

Done.


Update Go tutorials

Using Channel.Publish is deprecated. Tutorials should use updated API
Channel.PublishWithContext.

I guess this should also be backported to main or stable?


Upgrade to jQuery 3.3.1 and jQuery UI 1.12.1

I run into a problem with old version of jQuery that tends to fail lots jQuery widgets. So i have to manually upgrade the version of jQuery in the Orchard CMS release build. It would be better to have updated version  of these libraries.

Hi @sebastienros,
I would like to contribute about this upgrade of jQuery libraries. How can i start contribute?? What is the process to get start. I would be better to have discussion to let me know about the process if possible...
Thanks,
Niranjan

Create a PR that updates the version.

Merged PR-8183 related to this issue.


Add time_value

https://opensource.apple.com/source/xnu/xnu-4570.71.2/osfmk/mach/time_value.h.auto.html

Sorry for the delay, could you check the failure on Xcode 13 and 14? It seems these versions cannot find the items.

Fixed. Reason of the CI failure is that the generated C code in mach-test uses time_value rather than struct time_value, which can to be fixed by adding time_value in mach-test's build script.


12/11 Grading

Grading from Miguel, Brandon, Marcus, Nihar, Jayden:
Aiden Kim: showed lots of effort and thorough understanding of code on his parts
1.8/1.8, 0.9 team + 0.9 individual
Rayane Souissi: showed lots of effort and good understanding of binary concepts in parts
1.8/1.8, 0.9 team + 0.9 individual
Ethan Rahtmat: showed good effort and teamwork capability
1.8/1.8, 0.9 team + 0.9 individual
Chris Park: showed good understanding of hex and binary
1.8/1.8, 0.9 team + 0.9 individual
Anthony Yoo: showed good effort and understanding of code and concepts
1.8/1.8, 0.9 team + 0.9 individual

Peer Individual Grading For Aiden Kim: /.9
0.9/0.9: binary quiz creation shows lots of effort and thinking, color wheel interface was very interactive and user friendly, lightbulb function works and shows decimals, i liked how you could customize the amount of lightbulbs

Peer Individual Grading for Anthony Yoo
Team Grade: 0.9/0.9
Individual Grade: 0.88/0.9
Anthony's created Binary Number guessing game, and Anthony mainly worked on the loading screen sequence after each guess, aswell as the different outputs for each guess. If you get all 3 guesses wrong it shows you the answer, if you get one right then it will display a "win" message and will tell you that you are correct. He also worked on a random binary number generator, from 1-16.


Transparent background color for password input view

Hi,
I love this UI component. I am using it in a app i am developing and i have a question regarding customising the look and feel.
Basically i want the password input views to have a solid border color (white) with a transparent background color.
I tried setting sit up with the following settings:
//customize password UI
passwordContainerView?.tintColor = UIColor(hexString: "#959595")       passwordContainerView?.highlightedColor =  UIColor(hexString:"#50C5D7")
passwordContainerView?.passwordInputViews.forEach({ (view:PasswordInputView) in
view.circleBackgroundColor = UIColor.ClearColor()
view.borderColor = UIColor(hexString: "#959595") ?? UIColor.whiteColor()
view.highlightBackgroundColor = UIColor(hexString:"#50C5D7") ?? UIColor.blackColor()
view.textColor = UIColor.whiteColor()
view.highlightTextColor = UIColor.whiteColor()
})
But the password input field do not have a clear background color but instead have the border color is this correct? There is no way to set them up separately from each other?

Hi @Conrad-m-wise,  thanks for you love this component.  :-)
Sorry for that currently, there is no elegant way to use a clear background color.
As shown on figure below, you can see that below the circleView of PasswordInputView(the white color view below the number) has the gray color view that is the root view of PasswordInputView.

So it does not work, because the root view is not clear color.
I recommend you use the blur effect, like BlurPasswordLoginViewController, how about it?

I have one more idea as shown below. 💡
        //customize password UI
        self.passwordContainerView.tintColor = UIColor.color(.TextColor)
        self.passwordContainerView.highlightedColor = UIColor.color(.Blue)

        self.passwordContainerView.passwordInputViews.forEach {
            $0.borderColor = UIColor.clearColor()
            $0.circleBackgroundColor = UIColor.clearColor()
        }

But it will lose the circle border when the button on normal state.

Hi,
Yeah i tried the second option but indeed you loose the border. I will check-out the BlurPasswordLoginViewController set up and see if it fits my need. Thanks. You will hear back from me.


Requester portfolio backend

Some improvements to PR #262 .

Some of the serialisers are made dynamic, i.e. they now also accepts 'fields' argument. So that only relevant fields are included in response.
Now, portfolio will be requested from requester viewset, instead of module viewset .

#66 #65 #64

what's the module icon for?

icons for front end of portfolio. For example, fa-pencil for writing task.

those will be attached by task categories

but a module has multiple categories !

yeah, that's how we get the icons

but front-end shows only one icon ! which one to show ?

front end shows icons for task categories, not for modules..

I am completely confused.
Acc. to my current understanding, portfolio shows cards (or divisions) for each module of the requester, which may have multiple tasks and multiple categories. So we will need one icon for each module.
Please let me know, where I am getting it wrong.


Optionally show metadata for dependencies

I'm not sure if this goes outside the scope of this plugin but I thought I'd raise an issue here to discuss the possibility of add ManifestMetadata information to the output of cargo tree.
For example, I have a small example patch which displays license and repository information for the dependencies:
[~/cargo-tree]$ ./target/debug/cargo-tree tree -p petgraph
petgraph v0.2.5 MIT/Apache-2.0 https://github.com/bluss/petulant-avenger-graphlibrary
└── fixedbitset v0.1.0 MIT/Apache-2.0 https://github.com/bluss/fixedbitset

Would this be something that would be reasonable to add to the --verbose output (maybe with another flag)?

This seems like a thing that could be configured by a formatting flag. I've seen a couple of other command line utilities that support this kind of thing, but unfortunately can't remember their names right now. Something like:
cargo tree -p petgraph -f "{n} {v} {l} {r}"

Where {n} prints the dependency name, {v} prints the version, {l} prints the license and {r} prints the
repository.

A template parameter to print the list of active features (and maybe inactive with a minus in front or something) could be useful. Also maybe one to show the dependency version spec as well as the actual version.

Fixed by #3


Use broadcasting rules from ChainRules

This removes the reverse-mode broadcasting definitions here, in favour of the ones from ChainRules.
To make tests work, it also adjusts a few other rules, to add unthunk, or handle trivial cases, or to ensure zero(x) is only called on arrays of numbers. Not all were required by the tests here, some were for testing some Flux models (and some are in https://github.com/JuliaDiff/ChainRules.jl/pull/673). Test from #82 passes.
All the Most of the second order tests fail. They fail before this PR, too.
Built on top of #88, have not tested whether it would work without that. (rebased)
Closes #53

Codecov Report

Merging #89 (9d4352e) into main (fb1d4ec) will increase coverage by 2.55%.
The diff coverage is 78.57%.

@@            Coverage Diff             @@
##             main      #89      +/-   ##
==========================================
+ Coverage   51.69%   54.25%   +2.55%
==========================================
  Files          21       21
  Lines        2124     2105      -19
==========================================
+ Hits         1098     1142      +44
+ Misses       1026      963      -63

Impacted Files
Coverage Δ

src/stage1/broadcast.jl
0.00% <ø> (-20.00%)
:arrow_down:

src/extra_rules.jl
45.04% <70.00%> (+9.59%)
:arrow_up:

src/runtime.jl
80.00% <100.00%> (+1.05%)
:arrow_up:

src/stage1/generated.jl
78.18% <100.00%> (+5.45%)
:arrow_up:

src/stage1/hacks.jl
6.38% <0.00%> (+4.25%)
:arrow_up:

src/stage1/recurse.jl
95.84% <0.00%> (+4.35%)
:arrow_up:

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

This mostly looks really good.


 Those licenses are now nominative, so let me know if you want one. The current team is:

@lookfirst
@hgomez
@cyrille-leclerc
@bijugs
@kerlandsson
@zonArt

I will probably get back to JetBrains by tomorrow, so let me know quickly if possible. In any case, we can request license again at a later time if needed.

Thanks @gehel !

I just requested 5 licenses. We should be able to upgrade to more if we need at some point. I'll transfer the licenses as soon as I get them.

Just got the licenses from JetBrains (communicated to the team via private channel).
JetBrains asks (very nicely, this is not a condition to get the licences or to renew them) if we would like to add a JetBrains logo to our project page to show their support (that phrasing seems backward, late here and my English is not what it should be).
Exactly:

If you feel that IntelliJ IDEA has helped you a great deal, we'd be grateful if you could mention JetBrains' support for your project by placing an IntelliJ IDEA logo at the project's website. Please, follow the link if you wish to choose the logo: http://www.jetbrains.com/company/press/materials.html

I am already privately recommending IntelliJ to anyone I know, I'm not sure how I feel about putting a commercial company logo on the project page. Your call (all of you).

Why not Apache?

@utkarshcmu not sure I understand what you mean...

@gehel Oops, I misunderstood this thread/issue. Please ignore my question. :(

Since IntelliJ is helping the project by providing the license +1 to have their company logo as Thank You.

My issue is that we could end up with a lot of logos. SonarSource, CloudBees, Nespresso (to some extent), ... for providing resources both in people and in infrastructure. At the same time, JetBrains are the only one who asked (to my knowledge).

I +1 a "thank you" page listing all the companies who help the project giving free access to their paying products / services and for the companies who give engineering time.

Ok, thank you page created. Feel free to add other "thank you" for other companies...
Closing this issue as resolved...


Fix EQ for decreased urine phosphate level MP:0002902

Fix EQ for decreased urine phosphate level MP:0002902
currently uses:
has part some (
increased amount and
inheres in some (
phosphate and
part of some urine) and
has modifier some abnormal)
change to decreased amount

Changed EQ to use decreased


broken select

Issue Type: Bug

expected behavior: as soon as the cursor turns to text selection, you should be able to select text.
actual behavior: move cursor slowly from left until you see the text selection cursor, then click and try to select text. If you try on the first pixel on which the cursor changes, you cannot select text. Hoever as you can see on the second try, if you move over 1 extra pixel it works correctly.
VS Code version: Code 1.38.1 (b37e54c98e1a74ba89e03073e5a3761284e3ffb0, 2019-09-11T13:31:32.854Z)
OS version: Darwin x64 18.7.0

Interestingly, I cannot reproduce, I can always click when seeing the I cursor. Are you using some kind of zooming? (i.e. window.zoomLevel ?)

This appears to be an inconsistent bug.
 As you can imagine it is unnerving when selection doesn't work quite right.
I never manually adjusted the window zoom level, but it is currently set to -1.
I will try setting to 0 and let you know if/when the bug shows up again.. not sure if you wanna keep the issue open or not.. up to you.

Zooming via window.zoomLevel is super convenient for presenting/projecting, but it is implemented by Chromium ... in a way where pixel perfection gets lost... All of our integer coordinates / sizes get then projected via the zoom factor into ugly floats which sometime get rounded in funny way which end up causing 1px gaps here and there, depending on the window position, etc.
My recommendation is to avoid window.zoomLevel if you care about pixel perfection and to customize editor.fontSize, editor.lineHeight and ... to upvote #519 to get this implemented throughout the workbench...


add XmlnsPrefix which is in WPF

Added similar because it is in WPF. Its for Xaml files. Same use as in WPF

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: bmello4688 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

It’s so you can pre-define namespaces and associated namespace key in your Xaml file.
Brian
Brian

From: Dariusz Komosiński<EMAIL_ADDRESS>Sent: Tuesday, October 20, 2020 5:06:13 PM
To: AvaloniaUI/Avalonia<EMAIL_ADDRESS>Cc: Brian Mello<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [AvaloniaUI/Avalonia] add XmlnsPrefix which is in WPF (#4891)
@MarchingCube commented on this pull request.
Thanks for your contribution. I wonder - what is the use case here - tooling, or something else?

In src/Avalonia.Base/Metadata/XmlnsPrefixAttribute.cshttps://github.com/AvaloniaUI/Avalonia/pull/4891#discussion_r508836391:

@@ -0,0 +1,30 @@

+using System;

+namespace Avalonia.Metadata
+{

[AttributeUsage(AttributeTargets.Assembly, AllowMultiple = true)]

public sealed class XmlnsPrefixAttribute : Attribute

I guess we should also include class level comment to make it clear what this attribute does for the users.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/AvaloniaUI/Avalonia/pull/4891#pullrequestreview-513129912, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AB3WTQRFN5BQMF6Y2MLFMQDSLX3ULANCNFSM4SYKBQLQ.

Not key prefix
Brian

From: Brian Mello<EMAIL_ADDRESS>Sent: Wednesday, October 21, 2020 6:36:55 AM
To: AvaloniaUI/Avalonia<EMAIL_ADDRESS>AvaloniaUI/Avalonia<EMAIL_ADDRESS>Cc: Author<EMAIL_ADDRESS>Subject: Re: [AvaloniaUI/Avalonia] add XmlnsPrefix which is in WPF (#4891)
It’s so you can pre-define namespaces and associated namespace key in your Xaml file.
Brian
Brian

From: Dariusz Komosiński<EMAIL_ADDRESS>Sent: Tuesday, October 20, 2020 5:06:13 PM
To: AvaloniaUI/Avalonia<EMAIL_ADDRESS>Cc: Brian Mello<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [AvaloniaUI/Avalonia] add XmlnsPrefix which is in WPF (#4891)
@MarchingCube commented on this pull request.
Thanks for your contribution. I wonder - what is the use case here - tooling, or something else?

In src/Avalonia.Base/Metadata/XmlnsPrefixAttribute.cshttps://github.com/AvaloniaUI/Avalonia/pull/4891#discussion_r508836391:

@@ -0,0 +1,30 @@

+using System;

+namespace Avalonia.Metadata
+{

[AttributeUsage(AttributeTargets.Assembly, AllowMultiple = true)]

public sealed class XmlnsPrefixAttribute : Attribute

I guess we should also include class level comment to make it clear what this attribute does for the users.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/AvaloniaUI/Avalonia/pull/4891#pullrequestreview-513129912, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AB3WTQRFN5BQMF6Y2MLFMQDSLX3ULANCNFSM4SYKBQLQ.

I like idea

Sorry closed this by accident after approval


fffff

active branch=None
active analyses=
Traceback (most recent call last):
  File "/anaconda/envs/pychron/lib/python2.7/site-packages/pyface/ui/qt4/action/action_item.py", line 163, in _qt4_on_triggered
    self.controller.perform(action, action_event)
  File "/anaconda/envs/pychron/lib/python2.7/site-packages/pyface/tasks/action/task_action_controller.py", line 31, in perform
    return action.perform(event)
  File "/Users/Sidney/Programming/pychron/pychron/pipeline/tasks/actions.py", line 127, in perform
    task = app.get_task('pychron.pipeline.task')
  File "/Users/Sidney/Programming/pychron/pychron/envisage/tasks/base_tasks_application.py", line 109, in get_task
    win = self.create_window(w)
  File "/anaconda/envs/pychron/lib/python2.7/site-packages/envisage/ui/tasks/tasks_application.py", line 212, in create_window
    window.set_window_layout(layout)
  File "/anaconda/envs/pychron/lib/python2.7/site-packages/pyface/tasks/task_window.py", line 333, in set_window_layout
    self.activate_task(state.task)
  File "/anaconda/envs/pychron/lib/python2.7/site-packages/pyface/tasks/task_window.py", line 156, in activate_task
    task.activated()
  File "/Users/Sidney/Programming/pychron/pychron/pipeline/tasks/task.py", line 108, in activated
    foo
NameError: global name 'foo' is not defined

Test bug report.


Toxicity example

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Using mitmproxy as a library

Sorry if this question is a bit blurry, I'm trying to adapt an old wrapper for mitmproxy to the current version. I haven't touched the code in a long time and I pretty much have no idea what I'm doing.
In previous versions of mitmproxy I was able to use it as a library by creating a class InjectingMaster(flow.FlowMaster) and calling flow.FlowMaster.__init__(self, server, state), defining handle_request and such...
How can I do it in the current version?

Hi @dutzi,
Long time no see! Subclassing FlowMaster is deprecated, you can however now write your own addons as shown here: https://docs.mitmproxy.org/stable/addons-overview/

Indeed!
Thank you
On Wed, 17 Jul 2019 at 23:58 Maximilian Hils<EMAIL_ADDRESS>wrote:

Hi @dutzi https://github.com/dutzi,
Long time no see! Subclassing FlowMaster is deprecated, you can however
now write your own addons as shown here:
https://docs.mitmproxy.org/stable/addons-overview/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mitmproxy/mitmproxy/issues/3573?email_source=notifications&email_token=AAHCMTW7YFL5ZHFZQS6ABU3P76BW5A5CNFSM4H4HQTCKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2GSFOI#issuecomment-512565945,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAHCMTX3DZFCNHJTWZH2OCLP76BW5ANCNFSM4H4HQTCA
.

--

Eldad


Handle existing outputs, for stage re-use

We previously made changes to allow the inputs of stages to defer from a rigid structure #535
We failed to handle existing outputs in the same way. This is causing problems.
https://batch.hail.populationgenomics.org.au/batches/430489/jobs/1
Further, .crai files don't exist, and these are an expected output of the align stage.
Also, we need to create CramPaths and GVCFPaths instead of just GSPath objects so we can access things like .index_path.
DO NOT MERGE until we have a successful hail batch run in test for this one.

Looks good, https://batch.hail.populationgenomics.org.au/batches/430506