not execute the command

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
[void]Publish([string]$ways,[string]$runtime,[switch]$self_contained){
$waysprojpath="$($this.appsdir)$ways$ways.Gateway$ways.Gateway.csproj"
echo-info $waysprojpath.gettype().fullname
echo-info "$waysprojpath"
# echo-info dotnet --version
# echo-info tsc -v
# echo-info $(dotnet --version)
if(!(test-path $waysprojpath)){
echo-error "the ways $ways is not exist."
return
}
$r=''
if($runtime){
$r="--runtime $runtime"
}
if($self_contained){
echo-info "dotnet publish $waysprojpath $r"
Invoke-Expression "dotnet publish $waysprojpath $r --self-contained"
}
else{
echo-info "dotnet publish $waysprojpath $r"
Invoke-Expression "dotnet publish $waysprojpath $r"
# invoke-command -scriptblock {dotnet publish $waysprojpath $r}
}
# echo-info $(dotnet publish -h)
echo-success "the ways $ways was published."
$this.log("the ways $ways was published.","$($this.appsdir)$ways")
}
//a class method name is Publish in the bizlib.ps1
//i invoke the method the control is in the method is right,
// but if i do not use invoke-expression then the "dotnet publish ..." is not execute
// Why is this happening？
// at the other method I can use dotnet new ... command do not need use invoke-expression cmdlet
Expected behavior
dotnet publish ... is execute

Actual behavior
dotnet publish ... is not execute

Error details
not report error message

Environment data
PS E:\netenv\bofu\dbmanager> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.4.1
PSEdition                      Core
GitCommitId                    7.4.1
OS                             Microsoft Windows 10.0.19045
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals

You can't see the problem from the output in the image, it can be called in its entirety, but if you don't use invoke-expression, it won't actually trigger the dotnet publish command

Generally, please format your post properly, using fenced code blocks (place a ```  line both before and after your code). Without that, not only is the code hard to read, but its content may be altered by GitHub's rendering.

If you think you've found a problem with PowerShell itself, please try to provide an MCVE (Minimal, Complete, and Verifiable Example)


Italian vocabs added

Hi, I added the Italian translations of the vocabs. Very nice skill btw!

Hey @rbcolom, thanks for the PR! I think you missed the dialog directory. :)

@goldyfruit yep, you're quite right! :sweat_smile:  Fixed it

Thanks for your contribution @rbcolom !


FEATURE: keybinding for selecting project / status bar message

Like the package so far! Best project management package I have tried.
It would be nice to be able to set up keybindings for cycling between projects, something like next-project and previous-project, looping around from last to first and vice versa. I prefer working with keyboard over mouse, and it would be very smooth to switch between the projects that way.
Also, an indicator in the status bar of what project is currently selected would be most convenient (enabled/disabled in settings of course).
Thanks!

Hi @fireflyer95,
I am glad you find this package of use to you!
This could be a great feature for the next release!
Thanks for the feedback.

:+1:

@fireflyer95 and @cedvan i know this is a long FEATURE, but half is implemented, just didn't managed to get in my head the right workflow for completing this.


does not work well when text is pre-set

when the text is pre-set, the floating label stays down.
also consider the case when new text is simply passed as new prop
this might fix the first issue it but I cannot test it very well atm
    _handleChildren(props) {
        let edtChild = React.Children.only(props.children);
        this._oriEdtChild = edtChild;
        this._oriEdtStyle = StyleSheet.flatten([edtChild.props.style])
        this._oriOnFocus = edtChild.props.onFocus;
        this._oriOnBlur = edtChild.props.onBlur;
        this._oriOnChangeText = edtChild.props.onChangeText;
+        const textValue = edtChild.props.value
+        if (textValue) {
+            this._edtText = textValue
+            this.state.showLabel = true
+            this.state.labelAnimationValue = new Animated.Value(1)
+        }
....

@vonovak your fix worked for me.. maybe submit a PR

Having same problem when text is preset placeholder comes over the text.

@vonovak your fix worked for me.. submitting a PR

Sorry for no reply for a long time, a little busy these days. This bug has been fix on version 1.0.1, you can test it. Thanks !


fix(Enterprise Management): update service after recent API changes and add examples

PR summary
This PR updates the Enterprise Management service after recent API changes. Also adds working examples.
PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Current vs new behavior

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

:tada: This PR is included in version 0.18.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


feat: add option maxMatchLength

I've described the option in README.md. Please read it in Files changed for the detail. Thanks.

Sorry.  I don't want to add the description in README.

Merged.  I will fix it later.


About the pretrained weights loading problem in training

While loading UNet2DConditionModel from pretrained SD-2, ValueError appears, as parameter weights related to image_attentions are missed.

Traceback (most recent call last):
File "train_seesr.py", line 680, in
unet = UNet2DConditionModel.from_pretrained(
File "/opt/conda/envs/seesr/lib/python3.8/site-packages/diffusers/models/modeling_utils.py", line 646, in from_pretrained
raise ValueError(
ValueError: Cannot load <class 'models.unet_2d_condition.UNet2DConditionModel'> from preset/models/stable-diffusion-2-base
because the following keys are missing:
down_blocks.1.image_attentions.0.transformer_blocks.0.ff.net.2.bias, up_blocks.3.image_attentions.0.norm.weight, down_blocks.1.image_attentions.1.transformer_blocks.0.attn1.to_out.0.weight, ...

I have the same problem. It seems that the problem arises when use_image_cross_attention=True in UNet2DConditionModel.from_pretrained.  It is not used in some inference scripts.
Can someone clarify is it a mandatory setting for training?

May I ask if you get a mismatch between the output SR image and the input LR image at the beginning of the training?
Meanwhile, I think the parameter of image_attentions is included in the model trained by the author, not the sd2. And you can specify the unet_model_name_or_path and controlnet_model_name_or_path in the train_seesr.py.
@llstela @Zebraside

The error occurs when only --pretrained_model_name_or_path is set. (# resume from pretrained SD branch).
I'm using stable-diffusion-2-base from HuggingFace as base model.


Kwickset Lock not reporting manual operation [bug]

Version
Build/Run method

[X] Docker
[ ] PKG
[ ] Manually built (git clone - npm install - npm run build )

zwavejs2mqtt version: 6.5.0
zwavejs version: 1.0.0
Describe the bug
When using my SmartLock (910 Smart Code with Home Connect Technology from Kwikset) manual state changes are not reflected in the mqtt topic 98/0/currentMode. I see that the lock actually reports in 113/0/alarmType changing vlaues from 21 (locked) to 22 (unlocked), there are also other value's reported when the lock is unlocked via RF. This means that Homeassistant and homebridge-mqttthing don't pick up the status if the lock is changed manually.
To Reproduce
Open or Close lock manually and there is no change in MQTT Exporler to 98/0/currentMode or the zwavejs2mqtt webui.
Expected behavior
3/98/0/currentMode value to change to 255 when locked and 0 when unlocked manually
Additional context
I found a thread on this topic a while back for zwave2mqtt as well. I think it was that the locks aren't 100% compliant so the status needed to be updated whenever 113/0/alarmType changed 98/0/currentMode was updated with corresponding value.

LOGs pleae

zwavejs2mqtt.log
here you go, I redacted the homeid and user codes. There are 2 manual unlocks and locks in the logs

Hi All,
I’ve experience  this as well and tracked it down a little bit. If you look at the kwikset integration from OZW, in the 910.xml, you’ll find the following:
<CommandClass id="113">
    <!-- These Door Locks don't send a DoorLockReport when the
		Lock Status is Changed, but instead send a Alarm Message -
		So we trigger a Refresh of the DoorLock Command Class when
		we recieve a Alarm Message Instead -->
    <TriggerRefreshValue Genre="user" Index="512" Instance="1">
      <RefreshClassValue CommandClass="98" Index="1" Instance="1" RequestFlags="0"/>
    </TriggerRefreshValue>
  </CommandClass>

I checked this via MQTT and sure enough, only the alarm message was coming through. The integration exposes two entities related to alarms, theres alarmlevel, which always seems to be a 1, and alarmtype that seems to be 21 for locked and 22 for unlocked. Both send a message each time the lock changes.
I used this blueprint to fix it https://community.home-assistant.io/t/zwave-js-kwikset-910-manual-status-updater/283691
Another problem I found was the battery didn’t update regularly like it did with OZW every time the status changed. I changed the blueprint to add the following to update battery as well, although there might be a better way I’m a bit of an amateur.
payload_template: '{"args": [{"nodeId":{{ node_id }},"commandClass": 128,"endpoint":0,"property":"level"}]}'

@xImportExportx  this is a known issue and will be fixed  by this initiative https://github.com/zwave-js/node-zwave-js/issues/1713 in the meantime the workaround is to use the blue print or purchase a zwave+ module for your lock.

@rhomat27 thanks for the blueprint, i have added it and tried to set it up but it doesnt seem to be publishing anything to mqtt. The variables aren't that clear i think i got the idea but not working for me yet. Any tips? @scyto thanks for the info, ill wait for that to be fixed. I can roll back to zwave2mqtt pretty easily and move over once this is sorted. Tnings were working but zwavejs is so much faster in comparison I didn't want to loose that :(

@AlCalzone


No Error, No Timeout，Too

`var usb = require('usb')
console.log("22222222")
var device = usb.findByIds(0x1ad4,0x00c8)
if(device){
device.open();
var interface = device.interfaces[0];
interface.claim();
var inpoint = interface.endpoints[0]; // interface.endpoint(id) don't work for id=0 btw
var outpoint = interface.endpoints[1];
}
console.log(device)`
use “electron .”
but ,i got nothing

then i find there is no "usb.getDeviceList()" function in usb.js
where is wrong

The getDeviceList function is defined in C++. Endpoint address 0 is the control endpoint -- use device.controlTransfer to make control transfer requests. Your code is not actually issuing any transfers. Do you know which call is failing? With Electron, make sure you've built the library against the correct Electron ABI version.

in my code
“var device = usb.findByIds(0x1ad4,0x00c8)
console.log("22222222")
”
this row can‘t get device info . and the next row will't be print out

Closing due to inactivity, please reopen this issue if it persists with the latest release.


sflow GATEWAY extention

Hi,
I did not see any reference to extensions and specifically to the gateway extension for BGP attributes.
Are these supported and if not could this be added?

@HBLiB it doesn't support but nice to have gateway data and router data. I'll keep this open as enhancement.

Hi,
It's an old question and was drawn as an enhancement, has anything changed or is it planned to implement in a future release?

not yet, can you send me a pcap including sflow gateway data?

Hi @mehrdadrad,
was it ok for you or did you need more from my side?
with kind regards
Bernd


Make the UN based on a signature instead of a MAC

Currently the construction of a UN is based on a MAC. This works fine in the online setting, but we are getting into situations where we need to be able to do validate offline. In particular in the blockchain.
For this reason it makes sense to change the structure of the unpredictable number to be based on a digital signature (ECDSA secp256k1) instead of a MAC.

In relation to this and the current UN implementation I think we should increase it's length from 64 bits to 128 bits since it now  not only work to give randomness but also authenticity of the randomness.

Closed with PR #284


set response header on backend network errors

Setting some header values like req.id or simple strings should be send to the client, even with api connect errors, where applicable.

Are you proposing a set of response headers that are sent by default (on error responses)? Or do you want that to be user configurable?
What header/info besides the req.id do you think of?
Regarding the id, I would propose a bunch of settings that are related to that id rather than error responses:

send_request_id
send_request_id_downstream
send_request_id_upstream
`request_id_header = "Trace-ID"
…

Are you proposing a set of response headers that are sent by default (on error responses)? Or do you want that to be user configurable?
What header/info besides the req.id do you think of?
Regarding the id, I would propose a bunch of settings that are related to that id rather than error responses:

send_request_id
send_request_id_downstream
send_request_id_upstream
`request_id_header = "Trace-ID"
…

My proposal is to apply user configured headers even on connect errors if applicable.
For example:
set_response_headers = {
  x-request-id = req.id
  Cache-Control: no-store
  ...
}

This information is available while throwing an upstream connect error. Currently this would be applied on succeeded upstream requests only.
What we have to change here is evaluating some steps instead of an (too) early out.

My proposal is to apply user configured headers even on connect errors if applicable.
For example:
set_response_headers = {
  x-request-id = req.id
  Cache-Control: no-store
  ...
}

This information is available while throwing an upstream connect error. Currently this would be applied on succeeded upstream requests only.
What we have to change here is evaluating some steps instead of an (too) early out.


refactor some code

Pull Request Test Coverage Report for Build 1599

28 of 28 (100.0%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.07%) to 80.918%

Totals

Change from base Build 1576:
0.07%

Covered Lines:
1128

Relevant Lines:
1394

💛  - Coveralls


Usefull packet drops when sending packets more than NetConstants.NumSequenceNumbers

I found that NetReliableUnorderedReceiver drops usefull packets but send acks when sending too much packets (more than NetConstants.NumSequenceNumbers).
Therefore packets will lost forever.

or maybe more than window size...

when channel type - ReliableOrdered - this bug happens too.

Why closed?

After long analyze of code and rewriting some parts for better stability (increase Sequence numbers to 32768) i not found anything drops in code.
Only two possible leaks in NetReliableOrderedReceiver and NetReliableUnorderedReceiver.
in this line if (relate > m_windowSize) i think when relate == m_windowSize - this too very early message that can rewrite m_earlyReceived
In my situation i think server can block flooding.


createchannel pod Error reading configuration Unsupported Config Type ""

Please check this issue during deployment
Similar issue found: https://jira.hyperledger.org/browse/FAB-15582 and refers to variable FABRIC_CFG_PATH
How deployment performed:
cp setup_blockchainNetwork_v2.sh setup_blockchainNetwork.sh
./setup_blockchainNetwork.sh
Deployment ends with error
...
...
Waiting for createchannel job to be completed
Waiting for createchannel job to be completed
Waiting for createchannel job to be completed
Waiting for createchannel job to be completed
Create Channel Failed

K8s version
k version
Client Version: version.Info{Major:"1", Minor:"15", GitVersion:"v1.15.3", GitCommit:"2d3c76f9091b6bec110a5e63777c332469e0cba2", GitTreeState:"clean", BuildDate:"2019-08-19T11:13:54Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"14", GitVersion:"v1.14.6+IKS", GitCommit:"44b769243cf9b3fe09c1105a4a8749e8ff5f4ba8", GitTreeState:"clean", BuildDate:"2019-08-21T12:48:49Z", GoVersion:"go1.12.9", Compiler:"gc", Platform:"linux/amd64"}

PODs status
kubectl get pods                                                    ltop: Fri Sep 27 17:02:13 2019

NAME                                    READY   STATUS      RESTARTS   AGE
blockchain-ca-886988b8b-rxlfz           1/1     Running     0          4m50s
blockchain-orderer-6db47d79c5-mjkhc     1/1     Running     0          4m50s
blockchain-org1peer1-66d79b769c-s922r   1/1     Running     0          4m49s
blockchain-org2peer1-64c85fd696-tfdm5   1/1     Running     0          4m49s
blockchain-org3peer1-75f44fcbcb-wdg8c   1/1     Running     0          4m49s
blockchain-org4peer1-5fbc796dfb-pd4qz   1/1     Running     0          4m49s
copyartifacts-n9sw9                     0/1     Completed   0          5m19s
createchannel-x2hh7                     1/2     Error       0          4m30s
docker-dind-6d74f44c75-84xtt            1/1     Running     0          5m31s
utils-56fd4                             0/2     Completed   0          4m57s

Error:
k logs -f createchannel-p7fwx createchanneltx
/shared
bootstrap
keepalived
systemd-private-b1e7ce2665e74f0ba199a5978a3e93de-ntp.service-bZ8Odk
systemd-private-b1e7ce2665e74f0ba199a5978a3e93de-systemd-resolved.service-5oUCv1
2019-09-27 15:29:53.338 UTC [common.tools.configtxgen] main -> INFO 001 Loading configuration
2019-09-27 15:29:53.338 UTC [common.tools.configtxgen.localconfig] Load -> PANI 002 Error reading configuration:  Unsupported Config Type ""
2019-09-27 15:29:53.338 UTC [common.tools.configtxgen] func1 -> PANI 003 Error reading configuration:  Unsupported Config Type ""
panic: Error reading configuration:  Unsupported Config Type "" [recovered]
	panic: Error reading configuration:  Unsupported Config Type ""

goroutine 1 [running]:
github.com/hyperledger/fabric/vendor/go.uber.org/zap/zapcore.(*CheckedEntry).Write(0xc00011bc30, 0x0, 0x0, 0x0)
	/opt/gopath/src/github.com/hyperledger/fabric/vendor/go.uber.org/zap/zapcore/entry.go:229 +0x515
github.com/hyperledger/fabric/vendor/go.uber.org/zap.(*SugaredLogger).log(0xc0000ac240, 0xc0001a3804, 0xc0000f0d00, 0x38, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0)
	/opt/gopath/src/github.com/hyperledger/fabric/vendor/go.uber.org/zap/sugar.go:234 +0xf6
github.com/hyperledger/fabric/vendor/go.uber.org/zap.(*SugaredLogger).Panicf(0xc0000ac240, 0xc0000f0d00, 0x38, 0x0, 0x0, 0x0)
	/opt/gopath/src/github.com/hyperledger/fabric/vendor/go.uber.org/zap/sugar.go:159 +0x79
github.com/hyperledger/fabric/common/flogging.(*FabricLogger).Panic(0xc0000ac248, 0xc0001a3908, 0x1, 0x1)
	/opt/gopath/src/github.com/hyperledger/fabric/common/flogging/zap.go:73 +0x75
main.main.func1()
	/opt/gopath/src/github.com/hyperledger/fabric/common/tools/configtxgen/main.go:260 +0x1a9
panic(0xd62c40, 0xc00009f9c0)
	/opt/go/src/runtime/panic.go:513 +0x1b9
github.com/hyperledger/fabric/vendor/go.uber.org/zap/zapcore.(*CheckedEntry).Write(0xc00011bc30, 0x0, 0x0, 0x0)
	/opt/gopath/src/github.com/hyperledger/fabric/vendor/go.uber.org/zap/zapcore/entry.go:229 +0x515
github.com/hyperledger/fabric/vendor/go.uber.org/zap.(*SugaredLogger).log(0xc0000ac220, 0xc0001a3c04, 0xc0000f0c00, 0x38, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0)
	/opt/gopath/src/github.com/hyperledger/fabric/vendor/go.uber.org/zap/sugar.go:234 +0xf6
github.com/hyperledger/fabric/vendor/go.uber.org/zap.(*SugaredLogger).Panicf(0xc0000ac220, 0xc0000f0c00, 0x38, 0x0, 0x0, 0x0)
	/opt/gopath/src/github.com/hyperledger/fabric/vendor/go.uber.org/zap/sugar.go:159 +0x79
github.com/hyperledger/fabric/common/flogging.(*FabricLogger).Panic(0xc0000ac228, 0xc0001a3d88, 0x2, 0x2)
	/opt/gopath/src/github.com/hyperledger/fabric/common/flogging/zap.go:73 +0x75
github.com/hyperledger/fabric/common/tools/configtxgen/localconfig.Load(0x7ffe691e4f6b, 0xf, 0x0, 0x0, 0x0, 0xc00018b9e0)
	/opt/gopath/src/github.com/hyperledger/fabric/common/tools/configtxgen/localconfig/config.go:276 +0x41f
main.main()
	/opt/gopath/src/github.com/hyperledger/fabric/common/tools/configtxgen/main.go:271 +0xce0

peer chaincode query -C channel1 -n cc -c '{"Args":["query","a"]}'^C
...
...
2019-09-27 14:00:21.689 UTC [msp] Validate -> DEBU 035 MSP Org1MSP validating identity
2019-09-27 14:00:21.690 UTC [grpc] DialContext -> DEBU 036 parsed scheme: ""
2019-09-27 14:00:21.690 UTC [grpc] DialContext -> DEBU 037 scheme "" not registered, fallback to default scheme
2019-09-27 14:00:21.690 UTC [grpc] watcher -> DEBU 038 ccResolverWrapper: sending new addresses to cc: [{blockchain-org1peer1:30110 0  <nil>}]
2019-09-27 14:00:21.690 UTC [grpc] switchBalancer -> DEBU 039 ClientConn switching balancer to "pick_first"
2019-09-27 14:00:21.690 UTC [grpc] HandleSubConnStateChange -> DEBU 03a pickfirstBalancer: HandleSubConnStateChange: 0xc0003f2630, CONNECTING
2019-09-27 14:00:21.695 UTC [grpc] HandleSubConnStateChange -> DEBU 03b pickfirstBalancer: HandleSubConnStateChange: 0xc0003f2630, READY
2019-09-27 14:00:21.695 UTC [grpc] DialContext -> DEBU 03c parsed scheme: ""
2019-09-27 14:00:21.695 UTC [grpc] DialContext -> DEBU 03d scheme "" not registered, fallback to default scheme
2019-09-27 14:00:21.696 UTC [grpc] watcher -> DEBU 03e ccResolverWrapper: sending new addresses to cc: [{blockchain-org1peer1:30110 0  <nil>}]
2019-09-27 14:00:21.696 UTC [grpc] switchBalancer -> DEBU 03f ClientConn switching balancer to "pick_first"
2019-09-27 14:00:21.696 UTC [grpc] HandleSubConnStateChange -> DEBU 040 pickfirstBalancer: HandleSubConnStateChange: 0xc000455eb0, CONNECTING
2019-09-27 14:00:21.700 UTC [grpc] HandleSubConnStateChange -> DEBU 041 pickfirstBalancer: HandleSubConnStateChange: 0xc000455eb0, READY
2019-09-27 14:00:21.704 UTC [msp] GetDefaultSigningIdentity -> DEBU 042 Obtaining default signing identity
2019-09-27 14:00:21.705 UTC [msp.identity] Sign -> DEBU 043 Sign: plaintext: 0AA4070A6408031A0C08F5ABB8EC0510...120263631A0A0A0571756572790A0161
2019-09-27 14:00:21.705 UTC [msp.identity] Sign -> DEBU 044 Sign: digest: CE7327A88C6BEF47C34483617ACF351E9BFAE295DB3D70F744B71EFD8A670D0F
Error: error endorsing query: rpc error: code = Unknown desc = access denied: channel [channel1] creator org [Org1MSP] - proposal response: <nil>

@kant , @maheshwarishikha  ,
Could you please have a look into this?

@y0zg , Please check https://github.com/IBM/blockchain-network-on-kubernetes/issues/54 - if it helps.

Thanks @maheshwarishikha , this helped.
Do you know what might be the root cause if setup hangs with message "Waiting for copyartifacts job to complete"?

Hi @y0zg ,  if setup hangs with message "Waiting for copyartifacts job to complete" - it is mostly intermittent issue...with any reson say network or something else. Retrial solves that.

Closing this issue, as it worked for you. Feel free to reopen or open a new issue if required. Thanks.


Update Readme.md (minor changes)

made minor changes in readme according to github orientation format.  Please review.
#eager_to_learn

Please help on real coding projects instead. Thank you.


drivers: dma/i2c: nios2: config used as non-const

Describe the bug
These drivers seem to treat device configuration as non-const. For example:
https://github.com/zephyrproject-rtos/zephyr/blob/40bb3d808acb3e40b4a96f3bee794eab2f021b52/drivers/dma/dma_nios2_msgdma.c#L107-L110
To Reproduce

Don't cast config to non-const (as it should be):

#define DEV_CFG(dev) \
	((const struct nios2_msgdma_dev_cfg *)(dev)->config)

/scripts/twister -T tests/boards/altera_max10/msgdma -p altera_max10 -vv

Expected behavior
Device driver should use data to store non-ROMable data.
Impact
Impacts https://github.com/zephyrproject-rtos/zephyr/pull/41918
Logs and console output
N/A
Environment (please complete the following information):

OS: Linux
Toolchain Zephyr SDK 0.13.1
Commit SHA or Version used e8df8a579033be238beb774e4909d637da649f3a

Additional context
N/A

I'm not sure this is a real issue on nios2, but intend on fixing it anyways given the const attributes we have going. Will have a finalized PR once I have the hardware to test against (early next week).


Build policy docs additions

TODO:

Add PC IDs
Add to summaries
Add to book

@JBakstPaloAlto finished updating based on your review and added to the summaries and book


Update ocsp.c

adding Host Header to OCSP if no host header is set via -header

Shouldn't this also happen in s_ciient?

hmm. could make sense. i have not check this.
had only the problem on ocsp request. but yes it sould happens on any http / https request

ocsp.c fixed in master; thanks.


Support transform exp tree into RPN

What have you changed? (mandatory)
This PR adds RpnExpressionNodeVec, which can transform an expression tree into RPN form (a vector of RpnExpressionNode).
This PR is extracted from #3898.

[x] Merge #4321 first.
[x] Merge #4346 first.

What are the type of the changes? (mandatory)

New feature (change which adds functionality)

How has this PR been tested? (mandatory)
Unit test

/test

Please merge master and fix the conflicts.

/run-integration-tests

@brson Thanks a lot for the approval XD


[site-support-request] thehill.com

[x] I've verified and I assure that I'm running youtube-dl 2017.02.11
[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Site support request (request for adding support for a new site)

thehill.com (news site for Washington, DC -- i.e. "Capitol Hill") seems to use a lot of different embedded players on different article pages, and youtube-dl is hit-or-miss with them.
The one I got stuck on today is
http://thehill.com/homenews/senate/319143-franken-says-he-would-have-told-trump-calling-warren-pocahontas-was-racist
which seems to use Akamai AMP via something called inform-news, although I'm not really sure.
I wasn't able to find any similar extractors, though obviously there is some support for Akamai AMP, so maybe I am just looking in the wrong place.

$ youtube-dl -v 'http://thehill.com/homenews/senate/319143-franken-says-he-would-have-told-trump-calling-warren-pocahontas-was-racist'
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'http://thehill.com/homenews/senate/319143-franken-says-he-would-have-told-trump-calling-warren-pocahontas-was-racist']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2017.02.11
[debug] Python version 2.7.10 - Darwin-14.5.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 3.1.5, ffprobe 3.1.5, rtmpdump 2.4
[debug] Proxy map: {}
[generic] 319143-franken-says-he-would-have-told-trump-calling-warren-pocahontas-was-racist: Requesting header
WARNING: Falling back on generic information extractor.
[generic] 319143-franken-says-he-would-have-told-trump-calling-warren-pocahontas-was-racist: Downloading webpage
[generic] 319143-franken-says-he-would-have-told-trump-calling-warren-pocahontas-was-racist: Extracting information
ERROR: Unsupported URL: http://thehill.com/homenews/senate/319143-franken-says-he-would-have-told-trump-calling-warren-pocahontas-was-racist
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/generic.py", line 1741, in _real_extract
    doc = compat_etree_fromstring(webpage.encode('utf-8'))
  File "/usr/local/bin/youtube-dl/youtube_dl/compat.py", line 2526, in compat_etree_fromstring
    doc = _XML(text, parser=etree.XMLParser(target=_TreeBuilder(element_factory=_element_factory)))
  File "/usr/local/bin/youtube-dl/youtube_dl/compat.py", line 2515, in _XML
    parser.feed(text)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/xml/etree/ElementTree.py", line 1642, in feed
    self._raiseerror(v)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/xml/etree/ElementTree.py", line 1506, in _raiseerror
    raise err
ParseError: not well-formed (invalid token): line 342, column 21
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 696, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 369, in extract
    return self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/generic.py", line 2589, in _real_extract
    raise UnsupportedError(url)
UnsupportedError: Unsupported URL: http://thehill.com/homenews/senate/319143-franken-says-he-would-have-told-trump-calling-warren-pocahontas-was-racist

The HTML seems to have this form for the video:

although after JS runs on the page the DOM has:

<object type="application/x-shockwave-flash" id="flashObject1486927047763" name="flashObject1486927047763" data="http://launch.newsinc.com/167/js/lib/amp.premier/AkamaiPremierPlayer.swf" style="visibility: visible;" class="akamai-media-element" height="100%" width="100%">
...

Here's another inform-embed video on a different site: http://observer.com/2017/02/donald-trump-administration-mike-flynn-russian-embassy/


Script Fix for RAW Burst Capture

Fix applied in sensor_bin_to_sensor_raw_burst_capture.py to acquire correct RAW images

burst capture raw conversion script verified for 5 cases by @abdulrahman-10xe


Upgrade k8s api to v0.21

https://github.com/open-cluster-management/backlog/issues/14782
Signed-off-by: Philip Wu<EMAIL_ADDRESS>

/lgtm


Update release CI action versions

actions/upload-artifact@v3 will be deprecated soon. Migrate to v4.
Ref: https://github.com/actions/upload-artifact/blob/main/docs/MIGRATION.md

1st run was to check the files are uploaded correctly: https://github.com/quickjs-ng/quickjs/actions/runs/11863849122/job/33066303760?pr=694
After the 2nd commit the CI will run only on release, like before.


Object.defineProperty called on non-object at ReactFlow.esm.js

Describe the Bug
Bug on v9, error on function
Object.defineProperty called on non-object
at defineProperty ()
at Function.defineProperties (:133:7)
at ReactFlow.esm.js:545:1
at Array.forEach ()
at getAugmentedNamespace (ReactFlow.esm.js:543:1)
at Module.../../node_modules/react-flow-renderer/dist/ReactFlow.esm.js
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
In url : https://www.youtube.com/
Expected behavior
In other url, it's ok
Screenshots or Videos

Platform

OS: [Linux]
Browser: [Chrome]
Version: [e.g. Version 97.0.4692.99 (Build officiel) (64 bits)]

Additional context
No response

Hey @is343
could you provide any more context here? Is React Flow running in an extension? How did you build it?

I think you incorrectly tagged me instead of the original poster for this issue.
@lysa21 @moklick

hello,  @moklick,  yes it is in extention and if i remove content of function getAugmentedNamespace in node_module it's work

This doesn't seem to be an issue with React Flow. If you think, that it is an issue with React Flow, please provide a repo.


How to disable "Remote Desktop" feature in Builder?

Problem description
How to disable "Remote Desktop" feature in Builder? Thanks.
Proposal
Disable "Remote Desktop" as default like "keyboard logging" in Builder
Additional context
No response

It's disabled by default you must activate it thru the Server. Or edit the code to remove/edit the code to disable it completely.

It's not possible to disable specific features in the builder and not planned. However you are free to edit the source code as @BurntDog mentioned.


[Relax][Op] Add and Fix Operators

This PR adds these operators:

relax.repeat
relax.tile
relax.conv2d_transpose

This operator is intended to find the gradient of conv2d w.r.t its input. For details see torch document.
Now TOPI support of conv2d_transpose is limited. It does not support dilations; layouts other than default; symbolic output_padding.

Adds these modifications to Relax operators and parser:

Previously strided_slice cannot be legalized when strides = None. The reason is that TOPI does not support None strides. This PR will assign a default value for None strides when constructing relax.strided_slice Op.
Support optional weight parameter for relax.nll_loss.

Also support optional weight parameter for topi.nll_loss.

Remove cross_entropy_without_logits. cross_entropy_with_logits is reserved.
Substitude List[...] with Union[List[..], Tuple[..] in the annotation for some ops.

Now we use, for example, both expand_dims(arr, [1, 2]), and expand_dims(arr, (1, 2)). List[...] annotation will make type checks not pass.

0 parameter for reshape.

If there exist 0-s in the shape, that denotes the dimension of the output shape equals to the corresponding input dimension.
The dimension is counted from left to right.
This a feature implemented in relay.reshape, paddlepaddle, matlab, etc.

Support R.ShapeExpr in IRBuilder.

0 parameter for reshape.

If there exist 0-s in the shape, that denotes the dimension of the output shape equals to the corresponding input dimension.
The dimension is counted from left to right.
This a feature implemented in relay.reshape, paddlepaddle, matlab, etc.

@Ubospica I don’t think this is a standard behavior. None of the mainstream frameworks and specifications (NumPy, PyTorch, TF, DataAPI) has this specification. We should align our API with them. If, at some time point, you need a frontend importer say from paddlepaddle, you can handle the zeros in the importer.

Support R.ShapeExpr in IRBuilder.

Could you do this as a separate PR?

closing for now in favor of mlc branch


Transcode all titles - Batch Encoding?

Is there any chance of this suite receiving a feature to encode multiple DVDs or at least multiple titles on the same DVD?
I have a few music video DVDs that each have between 2 and 9 titles that represent different songs.
Any chance of accepting --titles=2-4 or something like that? It would have to be usable only with dry-run which I'm sure diminishes it's value a bit but I'm just thinking of batch jobs/loops for such use-cases.

@ProxyCell I'm sorry but HandBrake doesn't support transcoding multiple titles at the same time from disc image directories so this would be a significant investment in a feature that I may remove in the future anyway. Transcoding from disc image directories has many limitations and bugs already so I try to discourage use of that feature as it is.
Thanks for opening this issue. My apologies for being the bearer of bad news. I'll close this now but feel free to comment here again or elsewhere.


Protected branches plugin

Fixes #3 .

@bkeepers I didn't actually test this manually. I couldn't figure out how to get everything setup correctly. My creds were failing. But I figured I'd go ahead and get this submitted anyway.

@bkeepers I have successfully tested this in a custom app. Any idea when you can take a look?

Is your custom app still live? I can do a review, but my JS chops are such that I'd feel best just using it for a week on a non-critical project :)

Maybe there's room for a larger discussion on whether we could create two apps (prod and staging), and have a pipeline that deploys to prod on master merge. The idea being that @bkeepers can merge master a tiny bit more liberally, knowing that a small crew (maybe technical, but maybe less technical) is on the bleeding edge and will give feedback and/or exercise new features

@patcon hopefully this works... I just made it public. https://github.com/apps/probotsettingsprotectedbranches

Maybe there's room for a larger discussion on whether we could create two apps (prod and staging), and have a pipeline that deploys to prod on master merge. The idea being that @bkeepers can merge master a tiny bit more liberally, knowing that a small crew (maybe technical, but maybe less technical) is on the bleeding edge and will give feedback and/or exercise new features
Happy to open another issue if that sounds plausible

@jwsloan Playing with it here: https://github.com/patcon/sample-civictech-app
(I'll prob fully exercise it tomorrow -- and it will also help me create a best-practice project starter template for civictech projects for the weekly hacknights I co-organize! so thanks so much!)

Thanks for taking a look, @bkeepers. Currently removing the key means you no longer want to manage Protected Branches via this file. It will no longer make any changes. If you leave the protection key, but it is empty, it will remove protection.

Great point @bkeepers. I like @jwsloan's approach, the removing the key completely should leave protection as-is.
Should we treat null and [] as empty, just like we do with {}? That would seem to be forgiving, even if perhaps not totally consistent. If we do this, would be great to have it reflected in tests :)

@patcon I went ahead and implemented that. What do you think?

@patcon, @bkeepers have either of you had a chance to try it out?

Yep, I've used it, and it's pretty rad. One thought is that the config is pretty complex, and it might make sense for it to be more forgiving, since probot fails silently.
I know you already extended null and {}, but what about treating all these the same? [], {}, null, false.
(Happy to defer this into a different issue, as I think it works fine as-if if the person configuring is careful :)

Thanks for the feedback, @patcon! [] should already work. It is definitely reasonable for false to be treated as falsey... :) I'll create a test for that.

@patcon, I've pushed a commit adding tests for [] and false. They show that it already works as expected. Any other feedback?

@patcon, sorry. I thought we were just discussing the top-level protection key. I'm not sure it makes sense for all of those to be accepted in every situation. I need to look closer at what the API expects.

@patcon According to the API documentation, it looks like it is not valid to send null for dismissal_restrictions. If it is present, it needs to be an object. It seems like this API may not be 100% consistent in how it expects each key to be set to turn it off.
It seems like an answer is to not have probot fail silently. Is there something we can do about that? It would definitely not be good for someone to think they've successfully protected a branch, but not had it work.

@bkeepers, @patcon are you guys willing to go ahead and merge this? We could see if people run into pain with the configuration, and go from there?

@bkeepers the issue this came from just got marked stale. Do you think we could get it merged soon? Thanks!

@travi's comment:

i see a number of the more complex options for protected branches detailed out above, but my most common use case for simple repos is to simply mark it as protected so no one can accidentally force push. what might that simple configuration look like?

replying here since it has to do with code here :)
My quick impression is that this would work. Perhaps worth having a larger comment code block giving some simple examples though.
branches:
  master:
    protection: {}

@bkeepers I'd like to also see this PR merged. Could we get a yes/no either way?

Mr. @bkeepers, any chance we can move forward with this?

thanks @bkeepers, @jwsloan, and everyone for getting this in place. i'm excited to be able to automate this one away

unfortunately, i'm not seeing this setting take effect.
i initially tried just setting what i hoped would be the minimum. when that didn't work, i paid more attention to the "Required" properties. since that didn't enable protection for the branch either, i tried enabling one of the rules i didnt need, just to see if more was necessary, but still had no luck. am i missing something obvious?
also, just a side note, but the new required properties somewhat conflict with the note under Usage that states:

All settings are optional.

I realize that they are only required if the higher-level settings are provided, but some clarification might be valuable there.

@travi Since it is live now, I'll give it a shot myself. It could be that the Protected Branches API has changed since I submitted the PR, and I didn't do a good job of keeping track of that.

@travi would you put in an issue with the settings you're using?

I can add more detail tonight, but just in my phone for now. The links above show the variations I've tried

opened a separate issue for what i mentioned above: https://github.com/probot/settings/issues/87

@patcon does your install of my test app still work? As mentioned in #87, the feature doesn't appear to be working anymore.


Test K3s, Kubernetes, and MicroK8s workflow periodically failing

Describe the bug
A clear and concise description of what the bug is.
Akri's workflow to test Akri on K3s, Kubernetes, and MicroK8s periodically fails due to GitHub Runners running out of compute resources and the kubelet evicting Pods. The failure rate has been reduced by using release builds in PRs (https://github.com/deislabs/akri/pull/301), reducing the size of Debug Echo brokers by using nginx:stable-alpine (https://github.com/deislabs/akri/pull/301), reducing compute resource eviction limits in the MicroK8s kubelet (https://github.com/deislabs/akri/pull/301) and the K3s kubelet (https://github.com/deislabs/akri/pull/313).
Also, tests are being done with unshared debug echo devices to reduce the length of time the tests run by 5 minutes (shared devices get a 5 min offline grace period) and therefore the chance of failure. This should be changed back to using shared once the failures are resolved as it is the more complex scenario.
Potential solution
Akri should host its own workflow runners that have more disk and RAM.
GitHub runners have the following hardware specifications for Linux virtual machines:
2-core CPU
7 GB of RAM memory
14 GB of SSD disk space
However, MicroK8's documentation states that "At least 20G of disk space and 4G of memory are recommended."

Workaround: rerunning job typically resolves the problem


⚠ Bofrost.fr has degraded performance

In 9beda8b, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 948 ms

Resolved: Bofrost.fr performance has improved in 7c20247 after 6 minutes.


Dropped transactions lead to un-resumable scripts

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (251d7d6 2022-11-21T20:12:54.748601Z)
What command(s) is the bug in?
forge script
Operating System
macOS (Apple Silicon)
Describe the bug
High Level
forge script enters an un-resumable state if broadcast transactions are dropped from the mempool
Details
Resuming a deployment performs the following steps:

Load the sequence from the file
Retrieve and store receipts for any pending transactions found in the pending array of the sequence
Broadcast any sequence transactions for which receipts are not known

See code here:
https://github.com/foundry-rs/foundry/blob/master/cli/src/cmd/forge/script/cmd.rs#L237-L263
When waiting for a transaction, if it has been dropped from the mempool, an error will be generated in wait_for_receipts. This error will prevent progress to the broadcast step
If a transaction has been dropped from the mempool, then wait_for_receipts can never succesfully execute, and the dropped transaction can never be re-broadcast.
Solution
Instead of invoking wait_for_receipts via wait_for_pending,a resume should invoke a new function resume_check_pending that does the following:

Attempts to retrieve the pending receipt
If unsuccessful, attempts to retrieve the tx from mempool
If unsuccessful, deletes the tx hash from pending so that it will be rebroadcast

This function should NOT use the PendingTransaction struct, as that will be less reliable

@prestwich can we close this issue now that #3781 has been merged?

yes


@W 13102797 Updating OperatorPower for offline support

@W 13102797 Updating OperatorPower for offline support

Updating OperatorPower for offline support.
Having isNaN(), guardZero and guardBigNo in getJavaScript() to make the result similar to that of SQL.
Adding a new testData file specifically for test testExponentiationOperator.
Changes in Divide and Multiply is not needed unless we explicitly want to do multiplyByZero check and divide by zero check. These anyways would happen at later stage.

@mayank-lohiya @ashwanikmr121
I wanted to check what all testing is done to start supporting OperatorPower in offline mode. In this PR, I just see javascript-specific unit tests only. Is there a plan to test this out with Core in offline mode?

@mayank-lohiya @ashwanikmr121 I wanted to check what all testing is done to start supporting OperatorPower in offline mode. In this PR, I just see javascript-specific unit tests only. Is there a plan to test this out with Core in offline mode? Once this operator is made available offline also, are you going to test if all its functionalities are working as expected in offline mode?

Yes @sahil-here, We are going to do sanity around all the major features we know that use offline Javascript support, and @aditya-jain3 is already testing it on one of the major consumers for offline javascript i.e. Flows from which the request came. Do you see any concerns?

Yes @sahil-here, We are going to do sanity around all the major features we know that use offline Javascript support, and @aditya-jain3 is already testing it on one of the major consumers for offline javascript i.e. Flows from which the request came. Do you see any concerns?

Is it only the javascript part of the operator that gets used in offline mode? I am just concerned about the other methods like getSQL()/getCommand() if they also need to be tested in offline mode and if you are planning on testing those as well.

Is it only the javascript part of the operator that gets used in offline mode? I am just concerned about the other methods like getSQL()/getCommand() if they also need to be tested in offline mode and if you are planning on testing those as well.

@sahil-here Yes. Let's say you have a flow with two inputs(a,b) and 1 output(c). The output is a^b. If offline mode is enabled then this getJavascript() part would be part of the generated javascript in the flow UI output variables. Thus any dynamic change in the inputs would be instantly reflected and no server trip would be needed to execute that formula.
We are trying to ensure the result from getJavascript() and getSQL() are same, hence the changes in getJavascript(). If you also see the test gold files, For each pair of inputs we have 8 outputs.
[1.00596, 240]
4.16265990153128262
4.1626599015312826
4.162659901531303
4.1626599
So, the tests are not only checking JS output, but also output from SQL and others.

Is it only the javascript part of the operator that gets used in offline mode? I am just concerned about the other methods like getSQL()/getCommand() if they also need to be tested in offline mode and if you are planning on testing those as well.

@sahil-here Only getJavascript() and getCommand() is getting called in offline mode. getSQL() is not getting called. By using hardcoded variables in Flow Builder, we have done sanity testing around getCommand() and it was returning expected result. If we also see the test gold files, For each pair of inputs we have 8 outputs. inputvalues:[1.00596, 240] formula:4.16265990153128262 sql:4.1626599015312826 javascript:4.162659901531303 javascriptLp:4.1626599
So, the tests are not only checking JS output, but also output from SQL and others.

Just to be clear, I have a few queries to understand the final outcomes.

You preciously said you are trying to ensure the result from getJavascript() and getSQL() are same. But I see that the results are close but not the same. Is that intended behavior?

In the comments earlier, it was mentioned that offline mode is not supported because of precision loss. Are you trying to fix that shortcoming with this change and hence enable it in offline?
The new calculation does not seem to fix that shortcoming as the precision loss is still there with reduced precision of pow.

@aditya-jain3 Just wanted to check if you were able to take a look at my above queries.

@aditya-jain3 Just wanted to check if you were able to take a look at my above queries.
I have seen your queries, but was occupied with other priority items, and parked this story for a while. Will reply soon, once I am back on it.

@aditya-jain3 Any reason we have parked on this PR?


indicate git status

Right now we're gathering git status but not showing it. Figure out a nice way to show it

added in https://github.com/llimllib/git-ls/commit/b1ced6f7a1bdabb51a5e94906c6cdccbd44298a4
right now I'm just passing through the git status from git status --porcelain


Detect other images

Hello, I have a question: If I want to  detect other Dataset(e,g: my car's DVR images), do I need to retrain the model(I use culane_model to detect, the results are not well)?
thanks!

Sorry for being late, tough months about lots of deadlines.
Two methods:

No training and directly use pre-trained models
1-1 TuSimple
Make sure the aspect ratio of your image is 360 to 640, then our test protocol resizes your input image to 360 x 640.
1-2 CULane
Make sure the aspect ratio of your image is 295 to 820, then our test protocol resizes your input image to 295 x 820.
1-3 Why not good?
Currently, images in TuSimple contain only highway scenes, and the quantity is small. Another CULane contains urban scenes a lot but still suffers the lack of lane shape pattern (curved, closely curved, remotely curved, ...).
1-4 How to improve?
Crop (or any other transformations) your images to make them have similar shape patterns like CULane (or TuSimple) since we treat each lane as a whole object.

Annotating your image and training
2-1. How to annotate? (Some recommendations)
1). Annotate each lane with keypoints (10 ~ 20 points is ok, you can sparsely label the close part and densely label remote curved part for better performance on remote curved structures).
2). Make sure the occluded part is correctly imaged when you annotate a lane
3). Make sure the heavily occluded lane is labeled as ignored
4). 2) and 3) are important to form consistent shape guidance, which avoids the problem of marker-appearance misalignment brought about by arbitrary markers.
5) 2), 3) and 4) are found based on the fact that we treat each lane as a whole object (like a cat or dog in the object detection field) rather than low-level pixels.
6) Remember and think 5), you will make your mind about the advantages and disadvantages of LSTR.
2-2. Big gains
1)  Adding channels in the backbone, at least 4% CULane F1 improvement.
2)  Pre-train your model on a larger dataset, e.g. pre-trained on LLAMAS can bring at least 0.2% TuSimple accuracy improvement.

Sorry for being late, tough months about lots of deadlines.
Two methods:

No training and directly use pre-trained models
1-1 TuSimple
Make sure the aspect ratio of your image is 360 to 640, then our test protocol resizes your input image to 360 x 640.
1-2 CULane
Make sure the aspect ratio of your image is 295 to 820, then our test protocol resizes your input image to 295 x 820.
1-3 Why not good?
Currently, images in TuSimple contain only highway scenes, and the quantity is small. Another CULane contains urban scenes a lot but still suffers the lack of lane shape pattern (curved, closely curved, remotely curved, ...).
1-4 How to improve?
Crop (or any other transformations) your images to make them have similar shape patterns like CULane (or TuSimple) since we treat each lane as a whole object.

Annotating your image and training
2-1. How to annotate? (Some recommendations)
1). Annotate each lane with keypoints (10 ~ 20 points is ok, you can sparsely label the close part and densely label remote curved part for better performance on remote curved structures).
2). Make sure the occluded part is correctly imaged when you annotate a lane
3). Make sure the heavily occluded lane is labeled as ignored
4). 2) and 3) are important to form consistent shape guidance, which avoids the problem of marker-appearance misalignment brought about by arbitrary markers.
5) 2), 3) and 4) are found based on the fact that we treat each lane as a whole object (like a cat or dog in the object detection field) rather than low-level pixels.
6) Remember and think 5), you will make your mind about the advantages and disadvantages of LSTR.
2-2. Big gains
1)  Adding channels in the backbone, at least 4% CULane F1 improvement.
2)  Pre-train your model on a larger dataset, e.g. pre-trained on LLAMAS can bring at least 0.2% TuSimple accuracy improvement.


Query: Translate DateTime.Month/Day/Year etc.

I want to filter database records:
item.PurchDate.Year==2016&&item.TransactionType==PCTransactionType.Purchase (PCTransactionType is type of Enum)
when I check the log, it shows that these two conditions could not be executed in database server and must be evaluated locally.
Does EF7 support these kind of filtering?

@kennywangjin enum is supported so I guess the problem is with date.
As a temporary workaround, you could use:
var from = new DateTime(2016,1,1,0,0,0);
var to = new DateTime(2017,1,1,0,0,0);
...
item.PurchDate >= from && item.PurchDate < to &&
item.TransactionType == PCTransactionType.Purchase

fixed in c4776ac7fc24cdca6a64051f8e32377c1015b1d5


Support libraries stored deep in the repositories

This also covers recently reported problem with importing internal libraries using the full fmf identifier. Would be good to have fixed for the Thursday hacking session. Please, review soon if possible.

Q: What was the original issue.

This original issue is this: https://github.com/beakerlib/beakerlib/pull/72 I've added it to the comment as well. This is kind-of a hotfix but covers also cases when libraries are stored deeper in the structure.


A11y compliant tags

added title / alt / rel tags to links and images in the .jade files.
Note: Tags for store page has not been added

Closing - please see comments in #130


DetailPageHeader detaches the wrong event listener on componentWillUnmount

In https://github.com/carbon-design-system/carbon-components-react/blob/e8faa8c0be6da847c28e61013074bb70efe99aa2/components/DetailPageHeader.js#L32 it attaches creates a debounced wrapper around the scroll handler and attaches this to the window scroll event.
However, in componentWillUnmount(), it attempts to remove the original handleScroll method instead of the debounced wrapper.
I'll send a PR with a fix shortly.

Bluemix toolchain: Delivery Pipeline deployed carbon-components-react to Bluemix Design System, including commit 59cf4d9e8ea0363b9acb1b8e285d185d6dbbad24


added "DIAGRAM" tab for graphql-birdseye

Implements #935.
Changes proposed in this pull request:

Adds a new side tab which displays the graphql-birdseye diagram.

Preview: https://deploy-preview-967--graphql-playground.netlify.com/v2/new

There is an issue with the component after compilation:

It is happening because of the UglifyJS plugin. @Huvik or @timsuchanek, do you guys have any suggestions?

Maybe we could update build process or use a different plugin for minifying output 🤔

Maybe we could update build process or use a different plugin for minifying output 🤔

hmm, good idea. I tried excluding graphql-birdseye from uglify as well, but the issue persisted. Perhaps a different library would be good.

Any progress?

@rohit-ravikoti hey there! this is a fantastic feature. can I help you create a plugin for the new graphiql within the next month?

@acao I'm interested in creating GraphQL Voyager plug-in for GraphiQL.

awesome! do you want to open a plugin proposal issue in the graphiql repo then? i haven’t finished the template, but whatever you propose will be great!

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


ti-apps-launcher: wait for dev-dri-card1 before starting on eglfs

On eglfs based systems, the ti-apps-launcher is starting before the display and gpu cards are enumerated resulting in blank screen and crashes. So add a udev rule for /dev/dri/card1 and wait for it before starting apps on eglfs.

Applied with 27f555f23c284ed3e7f5191d8568de114c71118b


deps: Upgrade to latest version of emoji dependency

To add support for new emojis in Hugo, we need to upgrade our internal
dependency on the emoji package.
Note that we also need to update our tests, as the underlying emoji that
is rendered has changed.
Follow-up to #6391. (170f18d9352d39213170dd9d5e947eb45854c84b and
2df5d202c6fc6eb18ee0b259c3dd1156094947c5).

Let's hold off on merging until Travis has completed - I've tried against my fork and so far it's happy, and is just building Windows now.


feat: snap

Adding snap configuration and badge

Agreed, this looks good. I would just like to understand the process a bit.
Thanks for putting this together @AlexsJones!

Currently this is adding in the settings for publishing, the publishing itself is still manual at this point e.g. you have to be logged in and run snapcraft what I will do is add a bit of documentation on how to manually release until we automate it


[Bug]: Http Keep-alive not enabled

What happened?
Http Keep-alive seems to not be enabled in deployment. Running locally it handles keep-alive just fine, but in deployment it does not.
Version
v0.35.2
Which operating system(s) are you seeing the problem on?
In deployment
Which CPU architectures are you seeing the problem on?
In deployment
Relevant log output
No response
Duplicate declaration

[X] I have searched the issues and there are none like this.

Hey @hamiltop! To clarify, enabling keep-alive on your server does not work in deployment, but it works locally? I suspect this is due to your application being behind our proxies in deployment, and the proxies don't have keep_alive enabled. Whether they should warrant further investigation, though.

That would make sense. I would strongly suggest enabling keep-alive on the proxies, both from client->proxy and proxy->server. Throughput is much better due to reduced tcp connection overhead (and may be even more important for HTTPS efficiency).

Thanks for the suggestion @hamiltop! We'll need to do some investigation to explore potential drawbacks, as well as what to set it to and where to set it, but it makes sense! If you have thoughts on that, it would be great if you could expand on it a bit, if you have the time.
For context, requests to your service first go through an AWS LB, then to a proxy in shuttle-gateway, then to a proxy in shuttle-deployer (your project container) and on to your application. You can find the code for our proxies in the gateway/src/proxy.rs and deployer/src/proxy.rs if that interests you.

I dug in a bit and found that hyper-reverse-proxy removes the connection header which is valid for http 1.1. That does break keep-alive for http 1.0. Maybe http 1.0 isn't that big of a deal. It only showed up on my radar because I tried using apache bench (ab) to benchmark things and it's an http 1.0 client.


KeyError: 'Cannot detect if using keras or tf.keras.'

Following your instruction, I have performed this:
git clone https://www.github.com/keras-team/keras-contrib.git
&& cd keras-contrib
&& python convert_to_tf_keras.py
&& USE_TF_KERAS=1 python setup.py install
But when I tested this:
python cartoonize.py --input_dir input_images --output_dir output_images --styles shinkai hayao --comparison_view horizontal
The following error error happened:
Traceback (most recent call last):
File "cartoonize.py", line 11, in
from style_transfer.cartoongan import cartoongan
File "/home/myself/Documents/sandbox/playground/testtwo/CartoonGan-tensorflow/style_transfer/cartoongan/cartoongan.py", line 4, in
from keras_contrib.layers import InstanceNormalization
File "", line 971, in _find_and_load
File "", line 955, in _find_and_load_unlocked
File "", line 656, in _load_unlocked
File "", line 626, in _load_backward_compatible
File "/home/myself/anaconda3/envs/cartoongan/lib/python3.6/site-packages/keras_contrib-2.0.8-py3.6.egg/keras_contrib/init.py", line 4, in
File "", line 971, in _find_and_load
File "", line 955, in _find_and_load_unlocked
File "", line 656, in _load_unlocked
File "", line 626, in _load_backward_compatible
File "/home/myself/anaconda3/envs/cartoongan/lib/python3.6/site-packages/keras_contrib-2.0.8-py3.6.egg/keras_contrib/layers/init.py", line 3, in
File "", line 971, in _find_and_load
File "", line 955, in _find_and_load_unlocked
File "", line 656, in _load_unlocked
File "", line 626, in _load_backward_compatible
File "/home/myself/anaconda3/envs/cartoongan/lib/python3.6/site-packages/keras_contrib-2.0.8-py3.6.egg/keras_contrib/layers/advanced_activations/pelu.py", line 4, in
File "", line 971, in _find_and_load
File "", line 955, in _find_and_load_unlocked
File "", line 656, in _load_unlocked
File "", line 626, in _load_backward_compatible
File "/home/myself/anaconda3/envs/cartoongan/lib/python3.6/site-packages/keras_contrib-2.0.8-py3.6.egg/keras_contrib/utils/test_utils.py", line 188, in
KeyError: 'Cannot detect if using keras or tf.keras.'
Can you please help? Thanks very much.

Have you solve this problem?

Not yet. Right now I work on something else. Do you have any idea how I can try?
Thanks very much!

@changbaishan Make sure you follow every install commands specified in our demo Colab notebook including the correct TensorFlow version.
I just confirmed that the Colab notebook worked perfectly well, so you should be able to solve the issue by just following it.
If even the notebook is not working, we can reopen the issue.

Following your instruction...

Can you provide the instruction that you used to build your environment, along with your OS?

i meet the same qusetion; when i open the source code i installed, i found that it is not match to the webside:
"""Utilities related to Keras unit tests."""
import sys
import numpy as np
from numpy.testing import assert_allclose
import inspect
import keras
from keras.layers import Input
from keras.models import Model
from keras import backend as K
i do not know what's wrong with the git and install code

You could try using using tensorflow addons layers, instead of
from keras_contrib.layers import InstanceNormalization use
from tfa.normalizations import InstanceNormalization
That did the trick for me

@iszotic I am facing the same problem as in the issue and then, tried doing what worked for you. Could you please let me  know your Python version?
I am working with Python 3.7.5 and am unable to install tensorflow-addons.

I was facing the same issue; found out that I installed tensorflow-gpu==2.0.0 instead of tensorflow-gpu==2.0.0-alpha0. Installing the latter fixed the issue.

Solved. Try running cconvert_to_tf_keras.py from the directory where train.py is located.
cd directory/where/train_py_is_located
python directory/where/convert_to_tf_keras_py_located/convert_to_tf_keras.py
This should convert all keras calls to tf.keras calls and solve the issue.


Question about Page https://firebase.google.com/codelabs/firestore-web#5     Add Mock Data.

The text is in this chapter is as follows:
"The app will automatically generate a random set of restaurants objects, then call your addRestaurant function. However, you won't yet see the data in your actual web app because we still need to implement retrieving the data (the next section of the codelab)."
My question: where is the function "addRestaurant" situated?
I can't find it in the index.html file nor in the \scripts directory
Hoping for an answer

Hi @MarGer1962, it's in scripts/FriendlyEats.Data.js: https://github.com/firebase/friendlyeats-web/blob/e87f6d2c98791e6855438639c3c283f5abd70c44/scripts/FriendlyEats.Data.js#L18-L22


Update sbt-scalajs, scalajs-compiler to 1.0.0

Updates

org.scala-js:sbt-scalajs
org.scala-js:scalajs-compiler

from 0.6.32 to 1.0.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-js" } ]

labels: library-update, semver-major

Blocked by (at least)

[ ] shapeless
[ ] argonaut

Done in https://github.com/alexarchambault/argonaut-shapeless/pull/171.


Add headshots

Lets' get our faces on the landing page. I need a serious avatar by the way

Yeah, totally. Send a photo.
On Wed, Feb 3, 2016 at 8:12 PM, Clément Renaud<EMAIL_ADDRESS>wrote:

Lets' get our faces on the landing page. Need a serious avatar by the way
—
Reply to this email directly or view it on GitHub
https://github.com/duangish/duangish.github.io/issues/7.

<EMAIL_ADDRESS>does the avatars, send me your photo bro, already started talking about gettingggg


Operations on Matrix

Algorithm for various operation on matrix such as addition, subtraction and transpose.
Seperate functions for operation implementing iterating througout the matrix using two loops for indices.

@nikgautamgithub  thanks for contributing.
Can you provide a brief description of the algorithm and add the same as comment in file.

Added the description and comment in the code. Please check it. Thanks.


Rare possible race condition with xvfb-run causing issues launching mriqcp in parallel

Started 290 of parallel QC runs on the same node, and the first 20 of them failed with the below error. It may have something to do with xvfb-run trying to determine a free display number by incremental search and causing a race condition. After some run time, the parallel jobs become unsynchronized and there have been no additional errors.
Not sure this is a mriqcp-specific bug.
Traceback:
Traceback (most recent call last):
File "/home1/03872/wtriplet/software/mriqc/src/nipype/nipype/pipeline/plugins/linear.py", line 39, in run
node.run(updatehash=updatehash)
File "/home1/03872/wtriplet/software/mriqc/src/nipype/nipype/pipeline/engine/nodes.py", line 404, in run
self._run_interface()
File "/home1/03872/wtriplet/software/mriqc/src/nipype/nipype/pipeline/engine/nodes.py", line 514, in _run_interface
self._result = self._run_command(execute)
File "/home1/03872/wtriplet/software/mriqc/src/nipype/nipype/pipeline/engine/nodes.py", line 640, in _run_command
result = self._interface.run()
File "/home1/03872/wtriplet/software/mriqc/src/nipype/nipype/interfaces/base.py", line 1046, in run
runtime = self._run_wrapper(runtime)
File "/home1/03872/wtriplet/software/mriqc/src/nipype/nipype/interfaces/base.py", line 1476, in _run_wrapper
runtime = self._run_interface(runtime)
File "/home1/03872/wtriplet/software/mriqc/src/nipype/nipype/interfaces/base.py", line 1510, in _run_interface
self.raise_exception(runtime)
File "/home1/03872/wtriplet/software/mriqc/src/nipype/nipype/interfaces/base.py", line 1434, in raise_exception
raise RuntimeError(message)
RuntimeError: Command:
3dSkullStrip -input /corral-repl/utexas/poldracklab/openfmri/inprocess/ds030_QAP/ds030-sub-10150.wrk/aMRIQC/SkullStripWorkflow/_data_sub-10150.session_1.anat_1...corral-repl..utexas..poldracklab..openfmri..inprocess..ds030_QAP..ds030-sub-10150..sub-10150..anat..sub-10150_T1w.nii.gz/skullstrip/sub-10150_T1w_resample.nii.gz -prefix sub-10150_T1w_resample_skullstrip.nii.gz
Standard output:
The intensity in the output dataset is a modified version
of the intensity in the input volume.
To obtain a masked version of the input with identical values inside
the brain, you can either use 3dSkullStrip's -orig_vol option
or run the following command:
3dcalc -a /corral-repl/utexas/poldracklab/openfmri/inprocess/ds030_QAP/ds030-sub-10150.wrk/aMRIQC/SkullStripWorkflow/_data_sub-10150.session_1.anat_1...corral-repl..utexas..poldracklab..openfmri..inprocess..ds030_QAP..ds030-sub-10150..sub-10150..anat..sub-10150_T1w.nii.gz/skullstrip/sub-10150_T1w_resample.nii.gz -b ./sub-10150_T1w_resample_skullstrip.nii.gz+orig -expr 'a*step(b)'
-prefix ./sub-10150_T1w_resample_skullstrip.nii.gz_orig_vol
to generate a new masked version of the input.
Standard error:
/home1/03872/wtriplet/bin/xvfb-run: line 186: kill: (10002) - No such process
Return code: 1
Interface SkullStrip failed to run.

@oesteban I think the nipype afni preprocess interface for 3dSkullStrip has _redirect_x = True set explicitly (https://github.com/nipy/nipype/commit/280b582eebcc7cef15e407c4d19cfbd3390a3d3c). It may have been a requirement at some point, but it doesn't seem to be needed with the Jan 1 2016 compiled 3dSkullStrip (AFNI_16.0.00).
If I remove the line in nipype/preprocess/afni that sets _redirect_x = True and re-run the workflow I have no issues running it in parallel on the same node.

Thanks a lot @wtriplett . Could you open two issues in nipype about this?:
The first one would let the community know that our current approach to implement _redirect_x chokes when there are many parallel processes. I'm very aware that this is a pending task for us in nipype.
The second issue would point to the first one, and would basically say what you mention: is there a reason to have _redirect_x = True in that interface? And that as of AFNI 16 it's doesn't seem to need that flag?
For now, I will see if there's a way to modify that attribute and force mriqc to use it.

I think this is fixed in the fix/xvfb-run branch. Since the fix doesn't work for afni versions <16, we keep it in this branch.

Hey @wtriplett, have a look on https://github.com/nipy/nipype/pull/1328

@oesteban that's great and thanks for the helpful nypipe discussions!

@wtriplett ok, the fix worked ok on lonestar5 :)

Excellent my plan is to run the dataset again tomorrow and try to test the SlurmGraph plugin. I'll let you know how it goes!

+1 for using SlurmGraph!
On Jan 23, 2016 10:41 AM, "William Triplett"<EMAIL_ADDRESS>wrote:

Excellent my plan is to run the dataset again tomorrow and try to test the
SlurmGraph plugin. I'll let you know how it goes!
—
Reply to this email directly or view it on GitHub
https://github.com/poldracklab/mriqc/issues/25#issuecomment-174211280.


suggestions

where does the community hang out? gitter?
github doesn't look right. Maybe wiki would be better?
we need a proper discussion platform, slack?

Hey @hyqneuron one thing that was suggested here: #5 is having each repo as a starting point for collecting relevant work, sources and such.
I believe each project do have a google-group for now as well.
Also, It could depend on each group's preference perhaps for different discussion different platforms can help. There might be a slack channel coming (I believe), but from experience (Gitter) has been a more open-friendly approach, while Slack could be slower for onboarding, though it has been used for more focused conversations.

+1 for gitter over slack.

I attempted to create a AION Gitter but ran into issues (bugs in the Gitter web API -- seems to be a frequent occurrence). Please feel free to create one yourself, if you can!

Additionally, I would advise against using Skype. We should stay in touch via platforms that have free unlimited searchable archives of communications.

Sounds good!
Here is the Gitter link.
I will add it to the Main Repo + Site as well.
Hopefully this can be used for topic discussions and as an open communication style 😄

Gitter link seems to be broken

@cicobalico https://gitter.im/ai-open-network/Lobby seems to work for me. Maybe Gitter was down

try this link
https://gitter.im/ai-open-network/Lobby
ᐧ
On Wed, Oct 26, 2016 at 5:12 AM, Michele Moretti<EMAIL_ADDRESS>wrote:

Gitter link seems to be broken
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AI-ON/ai-on.org/issues/7#issuecomment-256177472, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABQoTTbdw-m6RXQWVmptYjTgesSmXSm3ks5q3nDEgaJpZM4Kff2N
.


Formatting mistake

in https://docs.microsoft.com/en-us/typography/script-development/bengali#shape-glyph-sequences-gsub-processing
Apply feature_'akhn'_ to substitute, it has no spaces before and after _ which breaks formating. Should be Apply feature _'akhn'_  to substitute.

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

ID: c254a65e-1b44-8a56-0f53-13badf120a10
Version Independent ID: a61fab42-a0ff-c5a6-50eb-3d0bdd7c8d75
Content: Developing OpenType Fonts for Bengali Script - Typography
Content Source: typographydocs/script-development/bengali.md
Product: typography
GitHub Login: @alib-ms
Microsoft Alias: alib

Fixed.


[Types] Expose Transaction types

Is your feature request related to a problem? Please describe.

I am trying to create a dynamic transaction entry generator. Bear with me please.
A third party tool my company uses exports data in a JSON format.
I am trying to go through this JSON and create an array of transaction operations.
Something like:
function generateTransactions(jsonData) {
  const transactions = jsonData.map((entry) => {
    const table = entry.identityName;
    const record = sanitizeData(entry)

    return {
      insert: {
        table,
        record
      }
    }
  })
}

This example is simplified, I will have to make updates as well in case the ID already exists in the DB.
There is another use case in the project as well where I have a debouncer on a set of form inputs that process a dynamic set of transaction operations after the timer runs out to avoid constantly updating the DB as the user types.
Describe the solution you'd like

I'd like to be able to have proper types for my dynamic function based on the types that Xata already has on the client package.
Is it possible to expose the following types?

TransactionOperation
InsertTransactionOperation
UpdateTransactionOperation
DeleteTransactionOperation

Describe alternatives you've considered

In some parts of the project I am pretty much duplicating those and using specific record types:
type UpdateTransactionOperation<O extends XataRecord> = {
  id: string;
  fields: Partial<EditableData<O>>;
};

type TaskUpdateTransaction = UpdateTransactionOperation<TaskRecord>;

And in another instance I did infer from the parameter, which isn't very nice:
import { getXataClient } from "@/xata";

import type { CreateProjectResponse, ProjectWithEmployees } from "../types";

const xata = getXataClient();

type TransactionParameter = Parameters<typeof xata.transactions.run>[0][0];

Additional context

Something like this would help, right?
https://github.com/xataio/client-ts/commit/9077d966c6136803c012c49b305bc00516d280c0

Going to tag a release now with the change (release might take a few minutes).
Please update and let me know if we can help you with anything else 😉

Yess! Thanks @SferaDev 🌟


Add Macross Vf1 Parts  from Kerbal Stuff

This pull request was automatically generated by Kerbal Stuff on behalf of Ynoz, to add Macross Vf1 Parts  to CKAN.
Please direct questions about this pull request to Ynoz.

The KS page is a 404. I believe the author deleted the page in lieu of #1736


Add support for the Clarion language

Clarion is a Rapid Application Development tool (RAD) with a 4th Generation language. The language was create by Bruce Barrington back in the early 1980 and operated in DOS until Clarion for Windows came out in 1994.
Some examples of Clarion on GitHub:
https://github.com/devroadmaps/DevroadmapsClarionLibrary
https://github.com/fushnisoft/ClarionMetroWizard
https://github.com/CarlosGtrz/ClarionJson
https://github.com/MarkGoldberg/ClarionCommunity
https://github.com/MarkGoldberg/CwUnit
https://github.com/Rick-UpperPark/ClarionLiveDLLTutor
https://github.com/ClarionLive?tab=repositories

@arfon Is there a way on your end to get 1,000 (or more) .inc files randomly?

I'm not sure there is sorry. What are you looking for here? Just more variety? Perhaps I could sort by last updated which would add some randomness to it.

I'm looking for a representative sample of .inc files. Maybe this search could work: https://github.com/search?o=desc&q=if+OR+mov+OR+where+OR+end+OR+procedure+OR+html+extension%3Ainc&ref=searchresults&s=indexed&type=Code&utf8=✓ (it should match most languages).

@pchaigno I've just sent you a bunch of files for that search :zap:

@pchaigno I've been using this trick: http://github.com/search?q=extension%3Ainc+NOT+wordwhichisnotfound&ref=searchresults&type=Code

@larsbrinkhoff Thanks for the suggestion! I noticed this "hack" too. However, I'm not sure I understand how it works (meaning "if it really works"). For example you'll notice that with two different keywords you get two slightly different numbers of results...
I chose to not use this technique here because, whichever sorting option you select, the results are grouped by repositories. Thus, the samples would be less random, or at least less representative of the entire collection of .inc files.
Does this make sense?

@pchaigno Yes, that makes sense.  It's too bad we can't get a truly random sample.

I only just noticed that there was a conflict on this pull request. Hopefully I managed to update everything ok. Thanks for sticking with this guys, if there is anything else I can provide to help let me know.


Enable string quote formatting for black

References and relevant issues
Setting up for #6383.
Description
On request from @czaki, splitting out the bulk of formatting changes from #6383, which come in the vast majority from enabling the string quote formatting (enforcing double quotes).
This makes it easier to spot what ruff-format does differently from black.

I approve of the feeling.
Though I think we should find a way to progressively migrate the string to be formatted.
We could try to use https://pypi.org/project/darker/ that only apply formatting to changed code.
On the other hand, the "churn" on other PRs should be relatively easily fixable by running black on the HEAD of the PR. But indeed we do have to do it to 75 PRs.
(I'd also love to have a few focus weeks with a soft moratorium on New PRs and finishing/closing existing ones)

I've received private feedback that my post above was a tad much, which I recognise. I sincerely apologise for letting emotion seep into my review. 🙏
Here's what I should have written:

Hi folks,

Since it takes black just ~1.2s (on my machine) to run on the full repo, switching to ruff for speed does not seem to provide a huge benefit to our infrastructure.
Additionally, ruff-format is still in beta. There are many changes in the horizon for ruff format (see the beta feedback discussion on the ruff repo), so I don't think we should migrate until the formatter has settled.
Indeed, once it has settled, it may well be that it can drop-in replace black with much more minimal changes and less disruption.
Regarding our black configuration skipping string formatting: this was intentional, because PEP8 allows either quote style, and single quotes have historically been preferred in the community, I think simply because they are easier to type. Indeed, they remain easier to type, so I would like to keep the single quote default, unless we have strong reasons not to.

Given all of the above, I would prefer to close this PR and #6383 for now. We could perhaps revisit them once ruff format is more mature.

Regarding our black configuration skipping string formatting: this was intentional, because PEP8 allows either quote style, and single quotes have historically been preferred in the community, I think simply because they are easier to type. Indeed, they remain easier to type, so I would like to keep the single quote default, unless we have strong reasons not to.

At this moment, we have mixed-string quotations. Not a single quote. Ruff formatter allows to define if use single or double quote.
From my perspective, I prefer a double quote, as my background is C/C++/Java where a single quote is a single char not a string (char *).
In the context of using formatter with a string normalisation argument which is easier to type is no longer valid. You could type one type of quotation and formatter will normalize them to have a uniform codebase.

Though I think we should find a way to progressively migrate the string to be formatted. We could try to use pypi.org/project/darker that only apply formatting to changed code.
On the other hand, the "churn" on other PRs should be relatively easily fixable by running black on the HEAD of the PR. But indeed we do have to do it to 75 PRs.

To fix multiple problems with testing we need to switch to src layout. Doing such a switch will require doing the same thing with 75 PR. So making another change that may need assistance at the same time will not increase the amount of work to do.

I definitely agree that the big quote migration would be an awful pain; I did talk about this in the community meeting yesterday, and I had no intention to no-scope this PR and merge it... maybe communicating that clearly by marking the PRs as draft would have saved you some stress @jni :wink:
My intention here was mostly to see if and what would have to change between black and ruff, and potentially report back (the open beta's purpose is to get feedback, after all, and how better than to run it on a complex repo like napari). So this PR is just a trick to make that diff easier on the other PR.
As for the quotes themselves: I'm all for single quotes, that's what I type all the time, but I agree with @Czaki that it doesn't really matter once you have a formatter. IMO we could wait to see if they add this option back in, and think about it again then, I really don't feel strongly about this.

To fix multiple problems with testing we need to switch to src layout. Doing such a switch will require doing the same thing with 75 PR

I don't think so as git should recognize file moves – as long as moving to src layout only move the files, it will resolve the conflicts by itself.

In the context of using formatter with a string normalisation argument which is easier to type is no longer valid. You could type one type of quotation and formatter will normalize them to have a uniform codebase.

I agree. Especially if you have a formatter set up as a editor hook. I personally have darker set up, so regardless of the project, it autoformat only what I type.

@brisvag It doesn't make sense to open this PR if you just want to show the ruff diff — you can just split the #6383 into multiple commits and show the diff of one commit. Wanna do that? (ie this one can indeed be closed.)
At any rate the comments read like @Czaki and @Carreau were on various parts of the spectrum of merging this "soon".
For future endeavours, maybe making a PR on your own fork and posting on #general on Zulip would be a potentially better way to achieve this?

In the context of using formatter with a string normalisation argument which is easier to type is no longer valid. You could type one type of quotation and formatter will normalize them to have a uniform codebase.

I agree. Especially if you have a formatter set up as a editor hook. I personally have darker set up, so regardless of the project, it autoformat only what I type.

  So, I prefer to rely on my nose rather than the autoformatter for those reasons.
I understand that these are small complaints. But it's death by a thousand cuts with the napari dev chain. And anyway, if y'all don't care one way or another (autoformat ftw!), let's do single quotes yeah? 😃😉

Yeah agree with single quotes if we do go in that direction!


Modified Dockerfile for multi stage build and GHA to add the build arg

Solves the issue
#35

I created a Dockerfile for multi stage build
Added build args to pass Github SHA and openweather API key to the Dockerfile
Changed the line in GHA to use build args for Github SHA and openweather API

Hey @torbencarstens I have made the requested changes.

Thanks for contributing @DipayanP007

Closes #35


_.contains(list, value) not works with objects

var list = [{name: "kiran"},{name: "smith"}];
_.contains(list, {name: "kiran"});

returns false
_.where(list, {name: "kiran"});

works perfectly

_.contains uses strict equality. _.contains(list, list[0]) works perfectly.
Use _.find / _.any for predicate finding.


Remove internal actors

Description

Issue reference

Please reference the issue this PR will close: #[issue number]
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

Conflicts after 1.14 to master merge

Close, in favor of
https://github.com/dapr/dapr/pull/8298


View of some code segments are corrupted

Chrome Version 71.0.3578.98 (Official Build) (64-bit) on WIN10

I'm checking it :) thanks


feat(sso): supports ldap

What type of PR is this?
ODC integrate support LDAP

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes # https://github.com/oceanbase/odc/issues/824
Special notes for your reviewer:

Additional documentation e.g., usage docs, etc.:

Has this PR been tested?

Has this PR been tested?
Yep. another pr will been created after frontend and backend integration testing finished.

If you have already made changes based on the comment, or have a different opinion about the comment, please explain it in text, otherwise I don't know if and how you have changed it or not.

Incidentally, this PR doesn't meet the specification, please add the PR description a little, otherwise it will be very confusing for the reviewer.


Add circle CI

Added a quick circle CI config. Free service, runs fast, looks like this.
You'll have to enable it through their app after merging.
It'd be pretty easy to add a gofmt check, or to have it save the binary after compiling so people can download test builds. The build fails right now because I invoke make without Version or BuildId, what would you like for me to set there?

This change is

@kofalt Thanks for this PR. This will be closed in favour of moving to Github actions (as it is one less service to maintain)


release resource problem

Hi, thanks for your greate project, it is really help for me.
Here is your part of codes:
		V4L2DeviceParameters inParam(videoDev.c_str(), videoformatList, width, height, fps, ioTypeIn, verbose, openflags);
		StreamReplicator* videoReplicator = rtspServer.CreateVideoReplicator( inParam, queueSize, captureMode, repeatConfig,
					output, ioTypeOut, out);
...
			StreamReplicator* audioReplicator = NULL;
#ifdef HAVE_ALSA
			audioReplicator = rtspServer.CreateAudioReplicator(audioDev, audioFmtList, audioFreq, audioNbChannels, verbose,
					queueSize, captureMode);
#endif

If i want to restart this process, and I found sometime the video is busy, I think I should do something when the process before quit. And I found that when did you release videoReplicator and audioReplicator ? I search all of project and not found. because I think this resource should be released when process stop.  Thanks!

Hi @maohai
I made some modifications, and maybe this code may helps you https://github.com/mpromonet/v4l2camera/blob/master/src/v4l2web.cpp#L127, keeping v4l2 connection and recreate replicators (with updated format).
Best Regards,
Michel.


New content

Bad commit - issues with Compute

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

:x:Error

Details

[Error] System.ArgumentException: An item with the same key has already been added.
at System.ThrowHelper.ThrowArgumentException(ExceptionResource resource)
at System.Collections.Generic.Dictionary2.Insert(TKey key, TValue value, Boolean add) at System.Linq.Enumerable.ToDictionary[TSource,TKey,TElement](IEnumerable1 source, Func2 keySelector, Func2 elementSelector, IEqualityComparer`1 comparer)
at ECMA2Yaml.Models.ECMAStore.Build()
at ECMA2Yaml.Program.LoadAndConvert(CommandLineOptions opt)
at ECMA2Yaml.Program.Main(String[] args)
Line 210: [Error] W:\z5ck-s.optemp\packages\opbuild.scripts.1.35.4\tools\opbuild\plugin.executor.ps1 failed to execute customized script in the plugin step docset_postbuild in the docset azuredotnet
Line 80: [Warning] Failed to run script W:\z5ck-s_dependentPackages\ECMA2Yaml\tools\Run.ps1 with exit code -1
Line 80: [Warning] Failed to run script W:\z5ck-s_dependentPackages\splitToc\tools\run.ps1 with exit code -532462766
Line 80: [Warning] Failed to run script W:\z5ck-s_dependentPackages\ECMA2Yaml\tools\PostBuild.ps1 with exit code 16

[Error] System.ArgumentException: An item with the same key has already been added.
at System.ThrowHelper.ThrowArgumentException(ExceptionResource resource)
at System.Collections.Generic.Dictionary2.Insert(TKey key, TValue value, Boolean add) at System.Linq.Enumerable.ToDictionary[TSource,TKey,TElement](IEnumerable1 source, Func2 keySelector, Func2 elementSelector, IEqualityComparer`1 comparer)
at ECMA2Yaml.Models.ECMAStore.Build()
at ECMA2Yaml.Program.LoadAndConvert(CommandLineOptions opt)
at ECMA2Yaml.Program.Main(String[] args)
Line 210: [Error] W:\z5ck-s.optemp\packages\opbuild.scripts.1.35.4\tools\opbuild\plugin.executor.ps1 failed to execute customized script in the plugin step docset_postbuild in the docset azuredotnet
Line 80: [Warning] Failed to run script W:\z5ck-s_dependentPackages\ECMA2Yaml\tools\Run.ps1 with exit code -1
Line 80: [Warning] Failed to run script W:\z5ck-s_dependentPackages\splitToc\tools\run.ps1 with exit code -532462766
Line 80: [Warning] Failed to run script W:\z5ck-s_dependentPackages\ECMA2Yaml\tools\PostBuild.ps1 with exit code 16

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


might be an error when generating list for tinygo

I have a very simple interface in wit like so
export foo: interface {
    record pos {
        x: s64,
        y: s64,
    }
    type bar = list<pos>;
    record line {
        pos: list<pos>,
    }
}

after compiling it with wit-bindgen-cli 0.24.0, I got this for the types in my .go file
type FooPos struct {
  X int64
  Y int64
}

type FooBar = FooPos  // <- I think there's something wrong here, where it should be `[]FooPos`
type FooLine struct {
  Pos []FooPos
}

While the .h file got generated along seems to be correct
typedef struct foo_pos_t {
  int64_t   x;
  int64_t   y;
} foo_pos_t;

typedef struct foo_bar_t {
  foo_pos_t   *ptr;
  size_t len;
} foo_bar_t;

typedef struct foo_list_pos_t {
  foo_pos_t *ptr;
  size_t len;
} foo_list_pos_t;

typedef struct foo_line_t {
  foo_list_pos_t   pos;
} foo_line_t;

additional information

I have tried making it type bar = list<list<pos>>, and the result is type FooBar = []FooPos. So seems like it is always missing a [].

Hey! Thanks for reporting this bug.
ugh I am actually not sure what the right fix it and this is becuase in Go generator, we don't actually use alias types and whenever the generator sees an alias type, it just resolves that type to it's raw form. For example, test: func() -> bar will just resolve bar to []pos in the function type signature, so this effectively renders the alias type to be useless in the generated code.
Perhaps the fix is to just remove the confusing line type bar = list<pos>; all altogether. What do you think?

The type alias is somewhat wierd, it is trying to work.
When I have type bar = pos;, it will generate type FooBar = FooPos. (which is correct)
When I have type bar = list<pos>;, it will still generate type FooBar = FooPos. (suppose to be type FooBar = []FooPos)
But if I have type bar = list<list<pos>>;, it will generate type FooBar = []FooPos, which suppose to be the result for type bar = list<pos>;. (type FooBar = [][]FooPos would be correct)
But I think your opinion is right. Type aliasing isn't that important and we can do it ourselves after importing the generated code.


Single loading of css module (e.x. for bundles)

@artemkaint ur changes looks very close to the same to #164 PR.
I will review these both in common, and will approve/decline PR soon.

Even if this will be merged, need to be sure there is test for single loading...


[CELEBORN] Rename CelebornHashBasedColumnarShuffleWriter to CelebornColumnarShuffleWriter

What changes were proposed in this pull request?
Rename CelebornHashBasedColumnarShuffleWriter to CelebornColumnarShuffleWriter
How was this patch tested?
CI

@kerwin-zk, why does CelebornHashBasedColumnarShuffleWriter to CelebornColumnarShuffleWriter? Any concern?

@SteNicholas Because Gluten currently supports both hash and sort shuffle.


Terminal suggest: Add all pwsh keywords as completions

Include real keywords, not var, etc. - reserved for future use
https://learn.microsoft.com/en-us/powershell/module/microsoft.powershell.core/about/about_language_keywords?view=powershell-7.4

To verify:

Enable suggest setting
Run Terminal: Clear Suggest Cache command
Open pwsh
Run ls, this should trigger the global completions to get loaded
Type to make sure that the pwsh keywords will get completions when in the first/command position in the prompt:


Update theme at runtime

Is there a way to update which theme should be used at application runtime?

Hi @fedoranimus. Currently there is no support for dynamically switching themes.
Should probably add it to the readme...
But will leave the issue open for now as a feature request.


Add Access managed cluster solution.

This task aims to integrate https://github.com/open-cluster-management-io/OCM/issues/18 with https://github.com/open-cluster-management-io/OCM/issues/17 to provide a more pragmatic solution for addon usage.

/assign @xuezhaojun

/work-in-process

this is done
/close


TemplateEngine fails if you have teams starter but don't include teams in profile

Added to the troubleshooting:

The bean 'templateEngine',... could not be registered. : This happens when you have included the Teams Spring Boot Starter but have disabled it with spring.profiles.active.  This is because the Teams starter brings in Thymeleaf, as does the Symphony starter, and they collide.  When Teams Spring Boot Starter is enabled, it renames the Teams Thymeleaf templateEngine to avoid this happening.  Don't include the Teams Spring Boot Starter unless you're going to enable it.

assaign


Fails to find Package 'delib_ws_p1' while running setup instructions for ROSCon 2024 workshop on Deliberation Technologies

Hello,
I'm trying out the setup instructions in advance of the ROSCon 2024 workshop on Deliberation technologies, because I wanted to see if could run it in a Docker container on my Macbook.
However, before I could fully test it out, I found that the 4th step in the Setup instructions (https://github.com/ros-wg-delib/roscon24-workshop/blob/main/README.md) fails with the following error:
root@docker-desktop:/delib_ws# ros2 run delib_ws_p1 run
Package 'delib_ws_p1' not found
I suspect that the package has been renamed.
Could you take a look and update the instructions?
Thanks.

Hi @srikant-v -- thanks for testing things early! We are still actively working on this repo and those instructions are not up to date at the moment.
 Feel free to open another issue if something else comes up.

Thank you!


Enable monitoring for Windows services on a normal OS monitoring run

Works for SharePoint, but not for standard OS monitor

Note: also changed the configuration setting from "SpecialWindowsServices" to "WindowsServices"