Module instances and state issue

I have a simple toy specification (a traffic light) and I am trying to compose / reuse a specification by instancing 2 times the same module.
I am using the syntax / example found in the cheat-sheet.
I would expect the two instances to have their own private variable light, but I get the following error
Multiple updates of variable light - quint(QNT202) which suggest both instances share the same variable.
As you can see I have added a fake constant, to make sure the modules are parameterized.
Is there a mistake (or misunderstanding on how modules are supposed to work) on my side ?
Or do you think it is possible issue of quint ?
Also in my case, it would be possible to work with an array of lights in the same module, it would be nice to have some examples on composition of stateful modules in the doc.
module TrafficLight {
    const N: int
    type Light = str

    pure val all_lights = Set("green", "orange", "red")

    pure def transition(l: Light): Light =
        if (l == "green")
            "orange"
        else if (l == "orange")
            "red"
        else
            "green"

    var light: Light

    action init = {
        nondet l = oneOf(all_lights)
        light' = l
    }

    action step = {
        light' = transition(light)
    }

    def type_invariant = all_lights.contains(light)
}

module main {
    import TrafficLight(N=1) as TL1
    import TrafficLight(N=2) as TL2

    action init = all {
        TL1::init,
        TL2::init
    }

    action step = all {
        TL1::step,
        TL2::step
    }
}

$ quint --version
0.17.0

Hello, thanks for reporting!
This is an issue with quint. It is a known issue for us, but we didn't have any real occurrences of this so far. Although we currently support this at the execution level, our effect checker, which statically checks your updates, is not updated to deal with variables from different instances. I will take a look at that and see if I can fix it in a short spike!

Also in my case, it would be possible to work with an array of lights in the same module, it would be nice to have some examples on composition of stateful modules in the doc.

I've written a traffic lights spec in the past, (in TLA+ actually, before quint was a thing), and I had a set of traffic lights, as you suggest. I do like the idea of doing that with instances tho, but beware that this path on composition of stateful modules is mostly unexplored - and that's why we don't really have documentation on it. We have some ideas on this matter, but didn't have a lot of time to explore them yet.
So, if you want a smoother experience, I suggest you try and write this spec in a single module with an array of traffic lights - feel free to come to me for help with that if you have any questions! And if you are interested in exploring the constitutionality problem, we can try to see how far we can get with this version of the spec! On our side, I'll start by looking into the effect error problem :smile:

Ok, I have a local hacky fix and it works:
An example execution:

[State 0] { main::TL1::light: "orange", main::TL2::light: "green" }

[State 1] { main::TL1::light: "red", main::TL2::light: "orange" }

[State 2] { main::TL1::light: "green", main::TL2::light: "red" }

[State 3] { main::TL1::light: "orange", main::TL2::light: "green" }

[State 4] { main::TL1::light: "red", main::TL2::light: "orange" }

[State 5] { main::TL1::light: "green", main::TL2::light: "red" }

[State 6] { main::TL1::light: "orange", main::TL2::light: "green" }

[State 7] { main::TL1::light: "red", main::TL2::light: "orange" }

[State 8] { main::TL1::light: "green", main::TL2::light: "red" }

[State 9] { main::TL1::light: "orange", main::TL2::light: "green" }

[State 10] { main::TL1::light: "red", main::TL2::light: "orange" }

[State 11] { main::TL1::light: "green", main::TL2::light: "red" }

[State 12] { main::TL1::light: "orange", main::TL2::light: "green" }

[State 13] { main::TL1::light: "red", main::TL2::light: "orange" }

[State 14] { main::TL1::light: "green", main::TL2::light: "red" }

[State 15] { main::TL1::light: "orange", main::TL2::light: "green" }

[State 16] { main::TL1::light: "red", main::TL2::light: "orange" }

[State 17] { main::TL1::light: "green", main::TL2::light: "red" }

[State 18] { main::TL1::light: "orange", main::TL2::light: "green" }

[State 19] { main::TL1::light: "red", main::TL2::light: "orange" }

[State 20] { main::TL1::light: "green", main::TL2::light: "red" }

[ok] No violation found (1326ms).
Use --seed=0x14428d6697625b to reproduce.
You may increase --max-samples and --max-steps.
Use --verbosity to produce more (or less) output.

I'll have a PR by tomorrow, probably!

I have a branch with the fix if you are interested in building quint from source (in the meantime, while we don't have a release with the fix): gabriela/namespaces-in-effects.

Thanks a lot for your amazing feedback @bugarela
I will give a try to the branch if I have time before next release.

We should have a release in a few hours!

The release is out! 0.17.1 should fix your issue. Also, you should update your VSCode extension (or @informalsystems/quint-language-server for other editors).
Please, let us know if you face any other issues!


Control Character Escapes

Checking interest in escaping the whole A-Za-z range at the start of escaped strings in order to support ControlCharacter escapes:
> new RegExp('\\cJ').test('\n') // true
> new RegExp("\\c" + RegExp.escape('J')); // matches "\n" but not the string "\cJ"

Are we interested in these escaped? Personally I never even knew these were a thing before, let alone in scenarios where .escape would be used. I definitely see the appeal for safety though.
Summoning @mathiasbynens @anba who are knowledgable on the topic, @bergus and @nikic who led hex escapes and @allenwb @cscott and @domenic for the spec's PoV on the subject.

I agree with @anba: if you decide to prevent RegExp('\\u004' + RegExp.escape('A')) from expanding to RegExp('\\u004A'), then we should be consistent and apply the same logic to control character escapes.

I agree - if we do one, then we should do the other.
However I'm leaning towards the position that we should do neither. Can we get some data whether this is used anywere? A \c followed not by [A-Z], a \u not followed by {…} or 4 hexadecimal digits, a \x not followed by 2 hexadecimal digits? In any expression?
I suspect this "feature" is not used anywhere. We might even go so far to propose to TC39 that this usage should be forbidden in all expressions, not just those with the /u flag.

@bergus any idea on how to collect data? I have not found \\c inside the RegExp constructor or \c to be common by a shallow search but good data would go a long way here.

The spec has been updated with the escaping semantics that advanced to stage 2 today.


Add unit tests for LoremImage component

Use the LoremText component tests as an example.

Hey @CodeDraken I will work on that!

Great @d-ivashchuk be sure to check out the storybook here to help understand how it works. I'll update the readme with setup instructions soon.


Stray Clone Left in CephFS Filesystem When Deleting PVC During Cloning Process

Background
We are still struggling with Kasten.io exports and the fact that they use the old RW snapshot clone method which on volumes with any significant size will result in a timeout.
The trouble is not just that the backups aren't working so that we needed to put a workaround in place (which is something for Kasten to solve), but also that whenever the issue happens we get a stray folder on CephFS which eventually eats up our disk space and is difficult and a bit scary to clean up when you don't have a clear reference where it comes from.
We blame it on the issue below and I'd love to hear if anyone else has experienced this. If there's interest to work on this and you need to me to reproduce it manually, I'm happy to share details.
Issue
When cloning a CephFS snapshot, if the PVC is deleted while the cloning process is still ongoing, a stray clone remains in the CephFS filesystem.
Affected Versions
Tested on ceph-csi 3.8
Steps to Reproduce
Initiate a clone from a snapshot of significant size in traditional RW mode.
Before the cloning process completes and the volume becomes available, delete the PVC.
Expected Behavior
The cloning process should either be interrupted and the clone should be removed from CephFS, or there should be a reference retained in Kubernetes.
Actual Behavior
The cloning process continues uninterrupted, resulting in a new folder appearing on CephFS. However, there is no reference to this folder in Kubernetes, neither as a PV nor a PVC.

Yes, this is a known issue. Especially when backup softwares try to take backups of cephfs PVC too frequently.
By default cephfs can only handle 4 simultaneous clone creation requests(cloner thread limit), all others are queued.
refer: https://bugzilla.redhat.com/show_bug.cgi?id=1894038
This limit is configurable but is limited by the number of cores that mds has.
There is a tracker in external provisioner for PVC being deleted when provisioner pod restarts https://github.com/kubernetes-csi/external-provisioner/issues/486 .
We've also requested ceph to reject cephfs clones creation request preemptively to avoid this scenario.
refer: https://tracker.ceph.com/issues/59714

Thank you very much for summarizing this here. It definitely helps us to understand it's a known issue and get that full perspective on it.
We'll try to find a solution that uses the new RO snapshot clones then to avoid this from happening in the first place.

There seems to be progress on the ceph tracker https://tracker.ceph.com/issues/59714.
Converting this issue into disucssion since issue https://github.com/ceph/ceph-csi/issues/3996 already exists to track and fix this.


Use extract geometry filter to enforce PolyData

VTKjs currently only supports vtkPolyData. These changes use an extract geometry filter to get a useable representation of any VTK data object.
Note that these changes make creating compelling scenes with vtki a breeze!!
Examples
import vtki
vtki.set_plot_theme('document')
from vtki import examples
import panel
panel.extension('vtk')
from panel.pane import VTK as panel_display

def display(p):
    return panel_display(p.ren_win)

data = examples.load_uniform()

p = vtki.Plotter(notebook=False, off_screen=True)

p.add_mesh(data.threshold_percent([0.25, 0.75]))
p.add_mesh(data.wireframe())
p.add_mesh(data.slice_orthogonal())

p.show(auto_close=False)
display(p)

data = examples.download_st_helens().warp_by_scalar()

p = vtki.Plotter(notebook=False, off_screen=True)
p.add_mesh(data, opacity='linear')
p.show(auto_close=False)
display(p)

data = examples.download_lidar()

p = vtki.Plotter(notebook=False, off_screen=True)
p.add_mesh(data)
p.show(auto_close=False)
display(p)

Note these changes will make https://github.com/vtkiorg/vtki/pull/144 possible:

Looks good to me, I've restarted the Windows build which timed out the first time. Will merge shortly.


DAWN-266 ⁃ implement remaining "to do" sync-reassignment function

this has a performance impact.
The function is called when a sync-block provider times out or disconnects while providing a chunk of blocks. now that syncing no longer retrieves multiple chunks in parallel, this is a simpler problem to solve, but it has to be done.
now all the sync_manager object has to do is nominate another peer to continue the fetch from.

Overtaken By Events


Operator Overlaoding support

Currently, it seems that rustgpu does not support operator overloading (there is no ops module under spirv_std, and none of the examples seem to do this).
This is useful because although linear algebra is the most popular mathematical representation for transformations in computer graphics, it is not the only option.
For example, I currently use projective geometric algebra for skin animations. To do this I need to define a multivector type on the shader that is capable of addition, multiplication, scalar multiplication...
Of course, one can work around the issue by defining methods, but this is less ergonomic than having operators.

I thought there was a form of operator overloading via glam? Though, I might be mistaken.


Testing on LFW after an epoch fails

Hello there,
Context of the issue.
Openface is having trouble when calling the LFW testing code after every epoch.
Expected behavior.
I would expect it to test on LFW. My aligned folder is in ../data/lfw/aligned.
This folder is generated as per the instructions of http://cmusatyalab.github.io/openface/models-and-accuracies/, but I renamed it to aligned so that test.lua can find it.
Actual behavior.
Training works OK, but testing uses all RAM and swaps, irregardless of the -testBatchSize parameter.
Steps to reproduce.
I made a 224 sized dataset [1]. I did the same for LFW as per the instructions and put it to ~/openface/data/lfw/aligned
My training command is th ~/openface/training/main.lua -data <path_to_large_ds> -imgDim 224 -modelDef ~/openface/models/openface/nn2.def.lua
cudnn is set to false, imgDim to 224, peoplePerBatch to 8, imagesPerPerson 11 to fit on the GPU.
OS and hardware information.

Operating system: Linux
Torch version: Lua 5.1, master branch from today
CPU architecture: x86_64
GPU type (if using): Tesla K40c

I would be grateful for any assistance or information on how I could solve this.
[1] On a new dataset that I really hope to share after my thesis is complete :-)

Hi @daydreamt - I just committed some minor changes to the training code that adds imgDim to batchRepresent, before it was defaulting to 96 instead of 224. I wouldn't expect this to cause the swapping issue though. Can you try manually running batch-represent on your LFW aligned images to see if you still have swapping issues?
-Brandon.

Hello, thank you for your quick response!
I took a closer look (with print statements),
and the actual line that causes my program to freeze is
local nnModel = model:float():clone():clearState()
in train.lua
I will try running batch-represent now!

\cc @melgor

Hi @daydreamt. Are you sure about that line? This line convert model to float, clone it and then clear all buffers not needed for prediction. I am now testing this code and  everything works fine.
Could you verify which of the commend (float, clone, clearState) cause freeze?

The code is also working well and not hanging on my system, but we just added these changes over the past few days in #106. @daydreamt - can you try reverting to b8de89375a14ec26f90c817bfd917b23b23b491d and running the training code again? This way we'll know if #106 is causing the issue or not.

Hello @melgor,
yes, that is the one.
   print("Converting model to float")    local nnModel = model:float()    print("Converted to float")    nnModel = nnModel:clone()    print("Cloned.")    nnModel = nnModel:clearState()    print("Cleared state.")
I attach my program output.
output.txt
@bamos regarding batchRepresent, the trained nn4.small2.v1.t7 model works fine on 96 dimensional images. Will retrain with the older revision now!

Retraining on the older revision unfortunately did not fix it. This is a fresh installation of everything, but I think I messed up installing cunn.  The descriptions do not specify it - should luarocks install cunn be sufficient for installing it?
Terribly sorry for the misleading issue name.

This is strange, it's hanging converting the model to a float. #106 could have introduced more memory demands on the host, but I think you'll have the same issue on the previous commit I sent you. How much RAM does your system have? I run on a machine with 64GB of RAM and haven't optimized or profiled memory usage and requirements for training.

Can you try adding a collectgarbage() call immediately before the float conversion?

This machine has 16GB, and I successfully trained the same model with openface in the past. Then something  changed (I think the inception layer -
In 24 module of nn.Sequential:
~/torch/install/share/lua/5.1/nn/THNN.lua:109: bad argument #2 to 'v' (input image smaller than kernel size at ~/torch/nn/lib/THNN/generic/SpatialAveragePooling.c:68)
and I had to reinstall everything.

If you can find a commit with training code that works, can you try using git bisect to find where the issue was introduced?

Sure, I will try a fresh installation and older commits. Do you usually install cunn through luarocks?

Thanks @daydreamt - I'm hope you find a version that works. And yes, I install cunn and cutorch with luarocks.

Thank you @bamos, it seems more memory is needed than before. On a 32GB machine the testing progresses farther.
Actually, I never got the testing on LFW to work, and I had always disabled it. This might be a good opportunity to report it.
It seems that  part of the LFW path (the "..") gets lost, and I am not sure how to fix it. I would thankful for any ideas.
/home/username/openface/training/work/010/model_1.t7
/home/username/openface/training/work/010/lfw-1
{
data : "../data/lfw/aligned"
imgDim : 224
cache : false
model : "/home/username/openface/training/work/010/model_1.t7"
outDir : "/home/username/openface/training/work/010/lfw-1"
cuda : true
batchSize : 800
}
../data/lfw/aligned
cache lotation:         /home/username/openface/data/lfw/aligned/cache.t7
Loading metadata from cache.
If your dataset has changed, delete the cache file.
nImgs:  13233
Index start, end: 1 800
The imgpath is data/lfw/dlib-affine-sz:224/Dalia_Rabin-Pelosoff/Dalia_Rabin-Pelosoff_0001.jpg
/home/username/torch/install/bin/luajit: ...e/username/torch/install/share/lua/5.1/image/init.lua:346: data/lfw/dlib-affine-sz:224/Dalia_Rabin-Pelosoff/Dalia_Rabin-Pelosoff_0001.j$g: No such file or directory
stack traceback:
[C]: in function 'error'
...e/username/torch/install/share/lua/5.1/image/init.lua:346: in function 'load'
/home/username/openface/batch-represent/dataset.lua:329: in function 'sampleHookTest'
/home/username/openface/batch-represent/dataset.lua:388: in function 'get'
...username/openface/batch-represent/batch-represent.lua:41: in function 'batchRepresent'
../batch-represent/main.lua:42: in main chunk
[C]: in function 'dofile'
...akis/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:145: in main chunk
[C]: at 0x00406670
And this happens when I specify the lfwDir absolutely:
{
data : "/home/username/openface/data/lfw/aligned"
imgDim : 224
cache : false
model : "/home/username/openface/training/work/011/model_1.t7"
outDir : "/home/username/openface/training/work/011/lfw-1"
cuda : true
batchSize : 800
}
/home/username/openface/data/lfw/aligned
cache lotation:         /home/username/openface/data/lfw/aligned/cache.t7
Loading metadata from cache.
If your dataset has changed, delete the cache file.
nImgs:  13233
Index start, end: 1 800
The imgpath is data/lfw/dlib-affine-sz:224/Dalia_Rabin-Pelosoff/Dalia_Rabin-Pelosoff_0001.jpg
/home/username/torch/install/bin/luajit: ...e/username/torch/install/share/lua/5.1/image/init.lua:346: data/lfw/dlib-affine-sz:224/Dalia_Rabin-Pelosoff/Dalia_Rabin-Pelosoff_0001.jpg: No such file or directory
stack traceback:
[C]: in function 'error'
...e/username/torch/install/share/lua/5.1/image/init.lua:346: in function 'load'
/home/username/openface/batch-represent/dataset.lua:329: in function 'sampleHookTest'
/home/username/openface/batch-represent/dataset.lua:388: in function 'get'
...username/openface/batch-represent/batch-represent.lua:41: in function 'batchRepresent'
../batch-represent/main.lua:42: in main chunk
[C]: in function 'dofile'
...akis/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:145: in main chunk
[C]: at 0x00406670
Thank you for your trouble.

Hi @daydreamt - I'm not sure what's causing that. Let me know if you find a fix on your system.

Could those incorrect data paths be stored in the cache.t7 file?

@daydreamt are you still active on this project?


why assert image.shape[0] == 1, "Batchsize of only 1 is allowed for now" ??

Thanks for nice implementation!
I found you commented 'assert image.shape[0] == 1, "Batchsize of only 1 is allowed for now"' in the code,
and wonder why you limited the batch size to 1.
Is there any operation that force you to limit batchsize = 1 ?
Thanks!

Sorry for the late reply.
I had asserted because PascalVOC dataset had images of variable sizes. I rather thought of limiting the batch size to one in order to circumvent that. You could very well resize the images and the target labels and try with multiple batch


CI: Style Checks, Automate testing

Github Actions are an easy way to build and test the project with each new Pull Request.

[x] Check Whitespaces
[x] python Linter
[ ] java Linter
[ ] Spellcheck e.g. Codespell

Have merged the initial compliance check PR #23. Also opening a separate pull request to work on java linter and Codespell.


Update CHANGELOG.md

/azp run prepare-pipelines

/azp run js - voiceservices - mgmt - ci

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-arm-voiceservices


Question about protobuf version (ibis-substrait requires protobuf 3.20.1)

I was able to write substrait using the following versions in one repo:
duckdb = "^0.6.1"
ibis-framework = "^4.0.0"
pyarrow = "^10.0.1"
duckdb-engine = "^0.6.8"
ibis-substrait = "^2.19.0"

I have a separate repo where I started from substrait and have the following dependencies:
protobuf = "^4.21.12"
ibis = "^3.2.0"

Understandably, I came across this error (my repo is called encoder-substrait for now):
Because no versions of ibis-substrait match >2.19.0,<3.0.0
 and ibis-substrait (2.19.0) depends on protobuf (3.20.1), ibis-substrait (>=2.19.0,<3.0.0) requires protobuf (3.20.1).
So, because encoder-substrait depends on both protobuf (^4.21.12) and ibis-substrait (^2.19.0), version solving failed.

I can downgrade just to play around with things, but just wondering if updating protobuf is on the roadmap or if there are version-related problems I should be aware of? I saw the following comment in the pyproject.toml, but it doesn't seem relevant for a newer version of protobuf:
# protobuf==3.20 C extensions aren't compatible with 3.19.4

(also, if there is somewhere better to these questions, please let me know! I wasn't sure so I've just been putting them here)

oof, I just realized that the protobuf version is loosely based on the version specified by whatever compiles the protobuf definitions, yeah?
https://github.com/substrait-io/substrait/blob/main/buf.gen.yaml#L9-L10:
  - remote: buf.build/protocolbuffers/plugins/python:v3.20.0-1
    out: gen/proto/python

Hey @drin -- this is a fine place to ask questions like these.  And yeah, I think you found the main reason.  protobuf is technically "backwards compatible", at least within major versions.  This isn't always true, but that's the nominal guarantee.  So we track whichever version of protobuf the upstream substrait project is using.  We can probably relax the hard pin to allow higher versions, although I'm not sure if allowing versions >=4 will work

thanks!


Weird event names

It is weird that some events are in past tense, some aren't:

RequestReceived
RequestBefore (could be RequestCreated)
RequestAfter (could be RequestSent)

@alexbilbie ping :)

Oh, just realised the event names have a suffix Event. Is it really necessary? (They are already under the Events namespace)

I like suffixes. :)

I'm thinking it's best to remove the library specific event classes and just fire normal League\Event\Events.
i.e. replace this:
$this->emit(
   (new Events\ResponseBeforeEvent($request, $response))
);

with:
$this->emit('response.before', $response);

There is not too much domain logic here, so that could work as well. However the past tense thing is still valid IMO:

request.recieved
response.created
response.sent

All three of those work for me.
Want to create a PR for me? :smile:

Sure, I'm on it.

Resolved


Helm create say creating but doesnt create

Helm was installed through kubespray playbook
[root@# helm create mojo
Creating mojo
[root]# ll
total 0
[root]#
helm version
Client: &version.Version{SemVer:"v2.6.1", GitCommit:"bbc1f71dc03afc5f00c6ac84b9308f8ecb4f39ac", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.6.1", GitCommit:"bbc1f71dc03afc5f00c6ac84b9308f8ecb4f39ac", GitTreeState:"clean"}
If I create manually and then run helm install
it says , no Charts.yaml found.

When you say

If I create manually and then run helm install

How are you creating the chart manually, and are you supplying the directory name to helm install? Can you please provide steps on how you installed the Helm client? As far as I understand kubespray it only sets up tiller which is the server-side side of helm, which is not required for helm create.
For example, you should be able to do the following:
$ git clone https://github.com/kubernetes/charts
$ helm install charts/stable/mailhog

And it should install the mailhog chart on your cluster.

I'm going to close this ticket due to inactivity, but please re-open if this still needs to be addressed. Thanks!


openPsbtRequestPopup throws unknown address type

Hiro Web Wallet 6.2.1
Browser Chrome, Firefox
This PSBT uses segwit address for inputs and pays to the taproot sBTC Alpha wallet (tb1qrjgzx3w697f65cz9qdxrsxs0v35ypqk7mqmm9v) triggers the (Unknown address type) error below;
70736274ff0100a00200000001782ba5d65b8551a99228b8d77e4989cf6ef6a197827d13b6d1dc3e8ba3d8e77f0100000000ffffffff0300000000000000001a6a1858323c1a7010183fd1a76976e7b2bb67acdf57cdfe704882dd89000000000000225120b8c8b0652cb2851a52374c7acd47181eb031e8fa5c62883f636e0d4fe695d6caecd70c000000000016001409c7efcd51a08da9b4e38345645866c2a270eb39000000000001011fb7700d00000000001600141c902345da2f93aa6045034c381a0f64684082de00000000
But loads and allows signing with Electrum.

Also tried with this PSBT using wallets cardinal address for inputs with. Throws same error in wallet but loads fine in Electrum
70736274ff0100a002000000017ab0408cd8a95eb821244c67f4924b00c44a4567ca9954cc97d08cddd3e84ef10000000000ffffffff0300000000000000001a6a1858323c1a7010183fd1a76976e7b2bb67acdf57cdfe704882dd89000000000000225120b8c8b0652cb2851a52374c7acd47181eb031e8fa5c62883f636e0d4fe695d6cac42b0f000000000016001409c7efcd51a08da9b4e38345645866c2a270eb39000000000001011f95c40f000000000016001409c7efcd51a08da9b4e38345645866c2a270eb3900000000
Unexpected Application Error!
Unknown address type=unknown
Error: Unknown address type=unknown
    at Object.encode (chrome-extension://ldinpeekobnhjjdofggfgjlcehhmanlj/997.js:194768:19)
    at getAddressFromOutScript (chrome-extension://ldinpeekobnhjjdofggfgjlcehhmanlj/index.js:43693:93)
    at chrome-extension://ldinpeekobnhjjdofggfgjlcehhmanlj/index.js:20333:76
    at Array.map (<anonymous>)
    at chrome-extension://ldinpeekobnhjjdofggfgjlcehhmanlj/index.js:20328:42
    at Object.useMemo (chrome-extension://ldinpeekobnhjjdofggfgjlcehhmanlj/997.js:142995:189)
    at exports.useMemo (chrome-extension://ldinpeekobnhjjdofggfgjlcehhmanlj/997.js:147846:208)
    at useParsedOutputs (chrome-extension://ldinpeekobnhjjdofggfgjlcehhmanlj/index.js:20328:27)
    at useParsedPsbt (chrome-extension://ldinpeekobnhjjdofggfgjlcehhmanlj/index.js:20469:25)
    at PsbtRequestDetails (chrome-extension://ldinpeekobnhjjdofggfgjlcehhmanlj/index.js:21266:7)
    at Xh (chrome-extension://ldinpeekobnhjjdofggfgjlcehhmanlj/997.js:142978:137)
    at Wk (chrome-extension://ldinpeekobnhjjdofggfgjlcehhmanlj/997.js:143101:337)
    at Vk (chrome-extension://ldinpeekobnhjjdofggfgjlcehhmanlj/997.js:143091:389)
    at Uk (chrome-extension://ldinpeekobnhjjdofggfgjlcehhmanlj/997.js:143091:320)
    at Jk (chrome-extension://ldinpeekobnhjjdofggfgjlcehhmanlj/997.js:143091:180)
    at Ok (chrome-extension://ldinpeekobnhjjdofggfgjlcehhmanlj/997.js:143082:88)
    at Fk (chrome-extension://ldinpeekobnhjjdofggfgjlcehhmanlj/997.js:143084:300)
    at jg (chrome-extension://ldinpeekobnhjjdofggfgjlcehhmanlj/997.js:142938:105)
    at Xk (chrome-extension://ldinpeekobnhjjdofggfgjlcehhmanlj/997.js:143094:470)
    at Qk (chrome-extension://ldinpeekobnhjjdofggfgjlcehhmanlj/997.js:143092:398)
    at Hk (chrome-extension://ldinpeekobnhjjdofggfgjlcehhmanlj/997.js:143081:269)
    at J (chrome-extension://ldinpeekobnhjjdofggfgjlcehhmanlj/997.js:155372:203)
    at MessagePort.R (chrome-extension://ldinpeekobnhjjdofggfgjlcehhmanlj/997.js:155373:128)

Still seeing this in v6.3.1 .

Are you seeing the same error with the same PSBT in other wallets?

Xverse works with p2sh address.
Electrum is able to display the transaction given the psbt hex.
I thought I saw it working after a wallet update a few days ago but get the same error now.

Xverse works with p2sh address.

Is it able to sign the exact payload though?

Its signed on first attempt and then failed on second (couldn't work out the right redeem script maybe).
Xverse signed and broadcast this tx:

@markmhx is this an underlying bug in scure-btc-signer ?

Hi @fbwoolf, does this problem originate in scure-btc-signer ?
Can you raise an Issue for Paul if so please ? It's hard for us to do this without knowing the internals of the wallet interaction with Millers code and this is high priority right now as its effects the upcoming release of sBTC.
NB @kenrogers !

Hi @fbwoolf, does this problem originate in scure-btc-signer ?
Can you raise an Issue for Paul if so please ? It's hard for us to do this without knowing the internals of the wallet interaction with Millers code and this is high priority right now as its effects the upcoming release of sBTC.
NB @kenrogers !

Yes, this is hitting an error in the btc-signer lib. I have opened an issue before with this and it was closed by Paul. See the thread here: https://github.com/leather-wallet/extension/issues/3897

It is hitting the error here: https://github.com/paulmillr/scure-btc-signer/blob/47f7ab9fdaba59166bc0ac7746681923e206eef8/index.ts#L1485

We basically just call this method to display the output address in the UI, and it errors on btc.Address().encode().

Xverse is basically doing the same thing, so not sure how would work there but not with our wallet?
https://github.com/secretkeylabs/xverse-core/blob/5cab1f1e24825e4e8114b0e5c41aef6aa23dd4f4/transactions/psbt.ts#L260

@radicleart I loaded your hex into our test-app and hit your error. In my console I can see that one of your outputs has a script type as unknown, so that is why it hits that error trying to get the address form Paul's encode method...

Yes was just discovering this as well. The first ouput is op_return as per the sBTC spec.

Should this prevent signing ?

I'm also pretty sure this was working earlier in the project maybe back in May time?

yes so the error is in here - can't get an address for unknown type ?
	if (outputScript.type === 'unknown') {
	  return btc.Address(net).encode({
		type: outputScript.type,
		script,
	  });
	}

We have to handle 'unknown' here or we get a type error, but it errors in btc-signer bc the type is unknown. It looks like in the code you can try to use an option allowUnknowOutput (which appears to be a misspelling?), but not sure here. You can ask in btc-signer lib repo for clarification.

Ah yes - looks like he deprecated allowUnknownOutput in favour of allowUnknownOutputs in recent update.
Will update everything and let you know - thanks !

Something strange here. The allowUnkno(n)Output(s) doesn't get transferred to the PSBT (see transaction.toPSBT() so these flags are not passed to the wallet. Also Miller's Address encoder doesn't look at that field and assumes the script passed is an address;
  return {
    encode(from: P.UnwrapCoder<typeof OutScript>): string {
      const { type } = from;
      if (type === 'wpkh') return programToWitness(0, from.hash, network);
      else if (type === 'wsh') return programToWitness(0, from.hash, network);
      else if (type === 'tr') return programToWitness(1, from.pubkey, network);
      else if (type === 'pkh') return formatKey(from.hash, [network.pubKeyHash]);
      else if (type === 'sh') return formatKey(from.hash, [network.scriptHash]);
      throw new Error(`Unknown address type=${type}`);
    },

I think bitcoin.utils.ts : getAddressFromOutScript method is only used in [use-parsed-outputs.tsx] for display purposes so maybe it can return 'unknown' for op_return outputs ?

@radicleart Thx for trying, it should run through normalizeOutput here and hit this: https://github.com/paulmillr/scure-btc-signer/blob/47f7ab9fdaba59166bc0ac7746681923e206eef8/index.ts#L2016
But, when I search for that option in his code, that is the only existence of allowUnkownScript so that seems odd. 🤔 I had originally read that as allowUnknownOutput and thought it would be handled.
I think we can prob do what you suggest, I can at least put up a PR so you can test the build and get feedback from our lead engineer on the change. cc @kyranjamie

@radicleart I made a PR you can test with the change. I was able to get past the error with your hex posted above.
https://github.com/leather-wallet/extension/pull/4168
Link to build: https://github.com/leather-wallet/extension/actions/runs/6052120932

Awesome thank you - will test it over the weekend !

Looks good to me @fbwoolf. Thanks again !

@radicleart Mind providing more context on your use case here for @mica000 so she can verify whether the above UI approach is ideal for it, or whether any modifications might make sense for user clarity?
Also @btcopenstamp, I believe the above change will unblock your needs on https://github.com/leather-wallet/extension/issues/3897. Can you confirm if that's the case, and whether the above UI approach makes sense in your mind – or whether you'd expect the user to need a different sort of presentation while signing?

Hi Mark, yes the use case is deposit / withdrawal from sBTC protocol use op_return to store the payloads. Transaction generated by sBTC bridge are likely to have a mixture of outputs of type 'unkown' along with spendable outputs. The issue here was that yhe wallet tried to display an address for usability purposes for each output but unknown output have no address.
I think Fara solution of labelling them unknown works but might be confusing for users. They could be labelled 'data' or 'unspendable' etc. Not my forte!

Would it make sense at all just to hide those outputs completely bc they would still be available to see in the advanced view of the raw psbt?

Thanks for clarifying @radicleart!
Would be possible to know what do you mean when you mention 'unspendable' or 'data'? is it meant to the the same term used for 'Unspent Transaction Output'?

Hey @mica000 - Bitcoin allows unspendable outputs (signified with the OP_RETURN op code). They generally have a value of 0 and allow an 80 byte payload of data that is meaning full to some external protocol - sBTC in my case.
I also agree with @fbwoolf solution to hide these outputs. This makes sense as they don't move BTC around and the details are still available in the other tab.

@markmhx are you ok with ^ solution? Just hiding those outputs in the details view?

Yep I'm okay with simply hiding if we don't expect the user to benefit from seeing it

Updated in the PR and going to get it merged today, will be in next release. @radicleart, thx for the feedback on this!


Get TypeError: Incorrect padding for some files.

Hello,
For some files I get following error :
Traceback (most recent call last):
File "/usr/local/lib/python2.6/dist-packages/tornado/web.py", line 1040, in wrapper
return method(self, *args, **kwargs)
File "/usr/local/lib/python2.6/dist-packages/thumbor/handlers/crypto.py", line 29, in get
opt = cr.decrypt(crypto)
File "/usr/local/lib/python2.6/dist-packages/thumbor/crypto.py", line 63, in decrypt
debased = base64.urlsafe_b64decode(encrypted.encode("utf-8"))
File "/usr/lib/python2.6/base64.py", line 112, in urlsafe_b64decode
return b64decode(s, '-_')
File "/usr/lib/python2.6/base64.py", line 76, in b64decode
raise TypeError(msg)
TypeError: Incorrect padding
so I add '=' padding in the base64 string.

Can you provide a test that shows this error?

I can't get those errors any more.
I suggest to close this PR.


Dev dig-etl-engine cannot load atf doc

The current version of dig-etl-engine that we get from dev will throw an error in ES when loading the attached doc. To reproduce, create an empty dgi-etl project, add the single doc and run, there will be no docs in ES, and an error:
elasticsearch_1    | MapperParsingException[failed to parse]; nested: IllegalArgumentException[mapper [knowledge_graph.website.provenance.confidence.extraction] of different type, current_type [long], merged_type [double]];
elasticsearch_1    | 	at org.elasticsearch.index.mapper.DocumentParser.parseDocument(DocumentParser.java:156)
elasticsearch_1    | 	at org.elasticsearch.index.mapper.DocumentMapper.parse(DocumentMapper.java:309)
elasticsearch_1    | 	at org.elasticsearch.index.shard.IndexShard.prepareIndex(IndexShard.java:584)
elasticsearch_1    | 	at org.elasticsearch.index.shard.IndexShard.prepareIndexOnPrimary(IndexShard.java:563)
elasticsearch_1    | 	at org.elasticsearch.action.index.TransportIndexAction.prepareIndexOperationOnPrimary(TransportIndexAction.java:211)
elasticsearch_1    | 	at org.elasticsearch.action.index.TransportIndexAction.executeIndexRequestOnPrimary(TransportIndexAction.java:223)
elasticsearch_1    | 	at org.elasticsearch.action.bulk.TransportShardBulkAction.shardIndexOperation(TransportShardBulkAction.java:327)
elasticsearch_1    | 	at org.elasticsearch.action.bulk.TransportShardBulkAction.shardOperationOnPrimary(TransportShardBulkAction.java:120)
elasticsearch_1    | 	at org.elasticsearch.action.bulk.TransportShardBulkAction.shardOperationOnPrimary(TransportShardBulkAction.java:68)
elasticsearch_1    | 	at org.elasticsearch.action.support.replication.TransportReplicationAction$PrimaryPhase.doRun(TransportReplicationAction.java:657)
elasticsearch_1    | 	at org.elasticsearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:37)
elasticsearch_1    | 	at org.elasticsearch.action.support.replication.TransportReplicationAction$PrimaryOperationTransportHandler.messageReceived(TransportReplicationAction.java:287)
elasticsearch_1    | 	at org.elasticsearch.action.support.replication.TransportReplicationAction$PrimaryOperationTransportHandler.messageReceived(TransportReplicationAction.java:279)
elasticsearch_1    | 	at org.elasticsearch.transport.RequestHandlerRegistry.processMessageReceived(RequestHandlerRegistry.java:77)
elasticsearch_1    | 	at org.elasticsearch.transport.TransportService$4.doRun(TransportService.java:378)
elasticsearch_1    | 	at org.elasticsearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:37)
elasticsearch_1    | 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
elasticsearch_1    | 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
elasticsearch_1    | 	at java.lang.Thread.run(Thread.java:748)

Input file to reproduce the error
armslist_test.jl.gz

@saggu i think your new release should fix this.  can you update dev on dig-etl-engine and test it?
https://github.com/usc-isi-i2/dig-sandpaper/releases/tag/0.1.4-r058

@jasonslepicka the mapping is overwritten by https://github.com/usc-isi-i2/dig-sandpaper/blob/master/digsandpaper/elasticsearch_mapping/generate.py#L89
It is on my todo list. I am on vacation starting yesterday.


rxjs_1.of is not a function

I get this error when using cspell: 3.0.1:
cspell src/**/*.ts

rxjs_1.of is not a function

npm ERR! Linux 4.4.0-101-generic
npm ERR! argv "/home/travis/.nvm/versions/node/v7.10.1/bin/node" "/home/travis/.nvm/versions/node/v7.10.1/bin/npm" "run" "spell"
npm ERR! node v7.10.1
npm ERR! npm  v4.2.0
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>spell: `cspell src/**/*.ts`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>spell script 'cspell src/**/*.ts'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the mmm-awesome-alexa package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     cspell src/**/*.ts
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs mmm-awesome-alexa
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls mmm-awesome-alexa
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /home/travis/.npm/_logs/2018-04-29T10_50_40_294Z-debug.log

Thank you.  I'll take a look.

Most likely due to the rxjs 6 update:
Full logs of my error is here:
https://travis-ci.org/dolanmiu/MMM-awesome-alexa/builds/372648931

Looks like your still running node 7. Can you run the linter in node 8 or above?

One of the breaking changes in moving to cSpell v3 is a Node 8 requirement.

I will try

Running now:
https://travis-ci.org/dolanmiu/MMM-awesome-alexa/builds/373045712

Thanks! worked

👍


[ASCII-1551][ASCII-1552] Fix flare tests with system probe on Darwin

What does this PR do?

Fix flare unit tests on MacOS.
Motivation

Tests were broken on MacOS by https://github.com/DataDog/datadog-agent/pull/25037 due to the system-probe code returning an error ("system-probe unsupported") on Darwin.
Gating the system probe specific parts of the tests on Darwin fixed the issue.
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

QA covered by https://github.com/DataDog/datadog-agent/pull/25037 and CI.

/trigger-ci --variable RUN_ALL_BUILDS=true --variable RUN_KITCHEN_TESTS=true --variable RUN_E2E_TESTS=on --variable RUN_UNIT_TESTS=on --variable RUN_KMT_TESTS=on

/merge


Submit to Colorsublime

Hey I was wondering what the chances of submitting your theme to https://github.com/Colorsublime/Colorsublime-Themes would be?
Colorsublime just seems to handle themes a lot nicer than installing them individually.

Bump!


data migration and images are not shown

server A's Domin is:     blog.AAAA.com
server B's Domin is:     www.BBBB.com/blog
when migrate data from A to B, the images are not shown.
Only the old path “/content/images/2018/01/getimgdata.jpeg” is replaced to  "/blog/content/images/2018/01/getimgdata.jpeg", the image is shown.
How to fix it？Help！
If the server B's domain is modified to "www.BBBB.com", the images are shown normal.

Hi @lsqswl!
We're sorry to hear you're having problems, but ask that you do not use our GitHub repo for support. Please read our support guide. You can always use our slack community - https://slack.ghost.org, if you're not able to find the answer in our documentation - https://docs.ghost.org.


Errors During User Registration from 3rd Party Providers

I've seen this registration failure maybe a dozen times recently; I don't know how the callback endpoint gets hit with a user object that ends up being all null mind you! It happened three times today, but that may have been the same user. Could it be one of the providers not sending back all the information we're expecting?
[2017-07-17 13:08:19] [ERROR] AuthenticationFacade:261 - Internal Database error during authentication
uk.ac.cam.cl.dtg.segue.dao.SegueDatabaseException: Postgres exception
    at uk.ac.cam.cl.dtg.segue.dao.users.PgUsers.createUser(PgUsers.java:515)
    at uk.ac.cam.cl.dtg.segue.dao.users.PgUsers.createOrUpdateUser(PgUsers.java:392)
    at uk.ac.cam.cl.dtg.segue.dao.users.PgUsers.registerNewUserWithProvider(PgUsers.java:63)
    at uk.ac.cam.cl.dtg.segue.api.managers.UserAccountManager.registerUserWithFederatedProvider(UserAccountManager.java:1311)
    at uk.ac.cam.cl.dtg.segue.api.managers.UserAccountManager.authenticateCallback(UserAccountManager.java:290)
    at uk.ac.cam.cl.dtg.segue.api.AuthenticationFacade.authenticationCallback(AuthenticationFacade.java:236)
    ...
Caused by: org.postgresql.util.PSQLException: ERROR: null value in column "email" violates not-null constraint
  Detail: Failing row contains (63702, null, null, null, null, STUDENT, null, null, 2017-07-17 13:08:19.482, null, null, null, NOT_VERIFIED, null, null, null, null, null, null, null).
    ...
    at uk.ac.cam.cl.dtg.segue.dao.users.PgUsers.createUser(PgUsers.java:500) ~[classes/:?]
    ... 47 more

The only non-null rows are those either generated by the database, or default values. Maybe we're not validating the callback carefully enough?

So this turned out to be Facebook, and it happens because the only thing they now return when you request the user info endpoint is the ID of the user. You need to manually and explicitly request the fields you want. Meurig found this: https://developers.facebook.com/docs/graph-api/using-graph-api/#reading
The way I've fixed it isn't too neat, so it may want looking at by @sacummins at some point!


PlayerEX Healthbar

From what I know, you should be able to change the PlayerEX health bar back to normal (I am using 1.16.5) using the config... which I have no idea how to do.

Hi there,
No problem - just navigate to your config file (in the /config folder wherever your game is installed) and there should be a file with the name playerex-client or similar. Open this file and inside should be an entry similar to enableHUD - set this to false.
Thanks.

Perfect, thanks for the help!


Release notes for 0.20.0

This PR will update CHANGELOG to contain description of the changes since 0.19.0. This PR is required to release 0.20.0.
This is still work in progress!

Hey team! @shamb0t are there breaking changes coming in this release with respect to data structures / formats?

Hey @Schwartz10! :wave:  There will be breaking changes to the API but the new release will still be able to read old data structures and should be backwards compatible w.r.t. old OrbitDB addresses . The shape of new entries will change to include an identity and cid field and will have their version incremented to 1.
Does that answer your question?

What's blocking this?

Hey @fazo96 thanks for stopping by! No blockers but an issue was discovered and fixed in orbit-db-keystore, that meant a bit more work needed to be done to ensure old data-structures could still be read with the new keystore. Once all the keystore PRs https://github.com/orbitdb/orbit-db-keystore/pull/15, https://github.com/orbitdb/orbit-db-identity-provider/pull/23, https://github.com/orbitdb/ipfs-log/pull/238, are merged we should be ready for this and release. Almost there! :)

@shamb0t @aphelionz I believe this still needs to be merged (and updated before?) for 0.20.0 release.

(and rebase before merging)

This looks good to me and ready to merge 👍

Good catch @shamb0t! 👍 LGTM.


MM-22970: Fix highlighting of at mentions of self

Summary
This PR fixes and issue with highlighting at mention myself followed by a Period(.) or underscore (_). This issue has already been fixed in master by this PR -
https://github.com/mattermost/mattermost-webapp/pull/4916
However, that PR, was for another feature and is too large to cherry-pick back to v5.21. This PR only contains the changes necessary to fix the highlighting issue.
This issue was introduced when fixing the highlighting of the header - https://github.com/mattermost/mattermost-webapp/pull/4819
I made sure this fix was retained, see screenshot.
Essentially, when the options.mentionHighlight is not set, it gets passed as false. Therefore, when it is empty, it needs set to true by default.
Ticket Link
https://mattermost.atlassian.net/browse/MM-22970
Related Pull Requests
Screenshots

Creating a new SpinWick test server using Mattermost Cloud.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-4994.test.mattermost.cloud

Account Type
Username
Password

Admin
sysadmin
Sys@dmin123

User
user-1
User-1@123

Test server destroyed

@sbishel This needs to be cherry picked.


docs: Adding the current fuzzers + minitests into documentation.

Using the same approach as in my VTRs pull request
You can see the results here -> http://mithro-prjxray.readthedocs.io/en/add-docs/db_dev_process/overview.html

Lint failed
On Wed, Feb 28, 2018 at 1:14 PM, Tim Ansell<EMAIL_ADDRESS>wrote:

@mithro commented on this pull request.
In docs/db_dev_process/overview.rst
https://github.com/SymbiFlow/prjxray/pull/104#discussion_r171387000:

@@ -25,11 +25,30 @@ By looking at all the resulting specimens, you can correlate which bits in which

Looking at the implemented design in Vivado with "Show Routing Resources" turned on is quite helpful in understanding what all choices exist.
-SymbiFlow/prjxray/tools/
-^^^^^^^^^^^^^^^^^^^^^^^^^^^
+.. toctree::

:maxdepth: 2
:caption: Current Fuzzers
:glob:

fuzzers/*

+Minitests
+^^^^^^^^^
+
+.. toctree::

:maxdepth: 2

Fixed.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/SymbiFlow/prjxray/pull/104#discussion_r171387000, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJBDlxxcZM-K_l-V7KWSxD16SgsPBvc8ks5tZcGpgaJpZM4SWcOq
.


flutter tool not recognizing iPhone device name

Maybe because my iPhone is running iOS 10 ?

Notice the blank name before the dot for the iOS device.

Correct.    I thought we made flutter doctor detect this?  @danrubel

My flutter is 5 days old, I think. I can try upgrading and rerunning flutter devices.

I see the same on my machine:
`

flutter run
More than one device connected; please specify a device with the '-d ' flag.

SM N920S • 05157df5d93b793d                         • android-arm
• ba9c0b7bb6175f61268fb99ed496f5bdf5417566 • ios
`
[✓] Flutter (on Mac OS, channel master)
• Flutter at /Users/raju/dev/flutter
• Framework revision 6777766d2d (12 hours ago), 2016-10-11 15:51:35
• Engine revision e8d046e9e2
• Tools Dart version 1.20.0-dev.10.2

I'm seeing this with my iOS 9.x iPod Touch. Running flutter doctor shows
[✓] iOS toolchain - develop for iOS devices (Xcode 7.3.1)
    • XCode at /Applications/Xcode.app/Contents/Developer
    • Xcode 7.3.1, Build version 7D1014
    x libimobiledevice is incompatible with the installed XCode version. To update, run:
      brew uninstall libimobiledevice
      brew install --HEAD libimobiledevice

after following the flutter doctor instructions above, running it again, I shows
[✓] iOS toolchain - develop for iOS devices (Xcode 7.3.1)
    • XCode at /Applications/Xcode.app/Contents/Developer
    • Xcode 7.3.1, Build version 7D1014

Now, flutter devices correctly shows the name of the connected iOS 9.x device.

Same problem, warning, and fix with my iOS 9.x iPod Touch and XCode 8.x.

Correct, yep I landed the doctor fix in #6111.

@cbracken Thank you!
@sethladd Is this working for you? Issue fixed?

Hm, doesn't look fixed?
~/Code/flutter/examples/flutter_gallery[master*] $ flutter devices
2 connected devices:

Moto G 4 • ZY223GJDRL                               • android-arm
         • ff369dfd5f4a80545a431441c1e3066f8cb8d759 • ios
~/Code/flutter/examples/flutter_gallery[master*] $ flutter --version
Flutter • channel master • git@github.com:flutter/flutter.git
Framework • revision 2805e5f8b8 (26 minutes ago) • 2016-10-12 09:41:35
Engine • revision e8d046e9e2
Tools • Dart 1.20.0-dev.10.2

Solved the problem for me. Two iOS devices connected, shows name of both of them.
flutter run
More than one device connected; please specify a device with the '-d <deviceId>' flag.

Paddington • 7ede958c9d7faa47c7965b41405ff7df107735eb • ios
Shiva      • ba9c0b7bb6175f61268fb99ed496f5bdf5417566 • ios

Sorry, I only ran flutter upgrade, which I was expecting to fully upgrade my system.
After I manually ran the homebrew commands (which, flutter doctor nicely tells me :) I see this output:
~/Code/flutter/examples/flutter_gallery[master*] $ flutter devices
2 connected devices:

Moto G 4 • ZY223GJDRL                               • android-arm
iPhone   • ff369dfd5f4a80545a431441c1e3066f8cb8d759 • ios

Woot!
Thanks for the help :)

I know this issue is closed but it never provided a reliable explanation or fix.... Currently I have the same problem...  Macbook Air 2019 with 2 USB-C ports. While my USB-C will not identify my Moto G6 when plugged in the USB-C but if I use a USB-A/C dongle it connects... and no matter what I connect with the iPhone will not connect to flutter. HOWEVER on my 2012 MBA the iPhone worked ONLY if I launched iTunes first.

This issue was closed 3 years ago. :)  It's very possible that the issue you're encountering is new/different.  Would you be willing to file a new bug @rbucker?

https://github.com/flutter/flutter/issues/41022


[BUG] /extensions not showing anything in the 0.10.6 version

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behaviour:

Go to 'extensions'
See error

Expected behavior
List of extensions
Desktop:

LNbits version: 0.10.6

Additional context
Blank screen on /extensions, extensions are installed, through the super user.

Do you have an extension manifest added?
Try adding this to your .env or the same URL in the Manage Server tab, "Extension Sources"
LNBITS_EXTENSIONS_MANIFESTS="https://raw.githubusercontent.com/lnbits/lnbits-extensions/main/extensions.json

Would you know how to add this, for umbrel specifically?

Is this still an issue? Is it an LNbits issue or an Umbrel one?
Consider closing if fixed.

still a problem in Umbrel

still a problem in Umbrel

@stake2grow Pls check on spelling errors in the manifest and if you change the link dont miss to save afterwards (restart not needed).
You can find some guides on this linked here https://github.com/lnbits/lnbits/wiki/LNbits-Extensions


DKPY2 await_params=True should be the default

@tcr3dr Whatever await mechanism gets decided for awaiting, the default should be to ensure that if no await is specified connect() waits until all parameters are collected. That is consistent with what GCS like Mission Planner does, with DKPY1. Also it simplifies the documentation.

Going to close this in favor of #342.


Question:  When is the next release?

When is the next planned release of dependabot to Github?  We are experimenting with using dependabot with for our Java dependencies and are currently blocked because Github is still using release 0.117.2 which does not include this change

Hi @a1flecke, I'll try to cut a new release of the gem and bump the versions internally soon 👍

Thanks!

The new version is released, i think this issue can be closed :slightly_smiling_face:

@jurre  Any guidance on when the version that Github uses will be bumped?  I just kicked off a new run of dependabot against our repo and it is still using 117.2.

Hey @a1flecke, the new version should be available now, I rolled out the new version earlier today.

@jurre
Thank you.  With this release Dependabot was able to pick up our internal Java dependencies being hosted on Github Package registry

Awesome, happy to hear it, and thanks for your contribution 🎉


Python Project - Time Series Visualizer

Describe your problem and how to reproduce it:
As mentioned in this forum post, the tests appear to be missing a flag to allow the dataframe to be converted to an integer.
Whilst the README.md does allude to starting by converting the data to purely index -> value, I believe it would be more useful to include the flag:

Use Pandas to import the data from "fcc-forum-pageviews.csv". Set the index to the "date" column.

Add a Link to the page with the problem:
https://www.freecodecamp.org/learn/data-analysis-with-python/data-analysis-with-python-projects/page-view-time-series-visualizer
Project to test<EMAIL_ADDRESS>Fix:
Add the numeric-only=True argument to the df.count() method.

Hi, I would like to take a try at this - if it is still available.

Hello @KaniahDunn .
This issue has yet to be triaged. What this means is we are yet to decide whether to go ahead with resolving it.
You are welcome to pick up any issue which has a help wanted label, and does not already have a PR likely to close it.
@scissorsneedfoodtoo Do you think this change would be useful?

Ah, thank you for the clarification @Sky020 . I will look around for other issues that I might be able to pick up.
Thank you 😊

Hi @Sky020, thanks for your patience. This looks good to me. It makes sense to add that flag to the tests since they were probably designed to expect numeric values.
What do you think @beaucarnes?

File with the problem: https://github.com/freeCodeCamp/boilerplate-page-view-time-series-visualizer/blob/master/test_module.py
Inside the boilerplate-page-view-time-series-visualizer repo

@Sky020  I have added a pull request can you review it

Hello @tareq12345 , Thank you, for your PR. However, ricknjacky already opened a PR before yours. So, we are going to prioritise theirs.
We welcome you to work on another open issue.

hi
what is this for?


Show signs in netrw for files

Is your feature request related to a problem? Please describe.
In netrw it is not possible to see files that were modified/created staged/unstaged
Describe the solution you'd like
The solution is to show git signs in netrw
Describe alternatives you've considered
well :Git status but it's not the same as
Additional context
Not sure how complicated it is to integrate with netrw

Sorry this isn't something I'm interested in.


Merge pull request #24008 from allevato/batch-mode-response-files

Cherry-pick response file fix for batch mode into Swift 5.1. Reviewed by @davidungar.
cc @jrose-apple

@swift-ci Please test


[Event-Request] Table 37 "Sales Line" - OnCopyFromGLAccountOnBeforeTestDirectPosting

Hello Microsoft,
This request/question is in relation to #21292, this request is implemented differently in the 21.4 release as seen in code below (notice the added begin..end statement for IsHandled).
First question is why this was implemented differently from our request?
And if this was an intentional change; why include the local procedure  InitDeferralCode() within this new begin..end statement? We cannot call this procedure from an external object if we want to implement our own customization.
Also perhaps a small note, does the name of the event publisher still cover the implementation as below?
Please let us know if we can expect a new change on the code below or that we should refactor?
Thank you in advance for your reply
Kind regards,
Martin Oomen
Cegeka
local procedure CopyFromGLAccount()
    var
        IsHandled: Boolean;
    begin
        GLAcc.Get("No.");
        GLAcc.CheckGLAcc();
        IsHandled := false;
        OnCopyFromGLAccountOnBeforeTestDirectPosting(Rec, GLAcc, SalesHeader, IsHandled);

        if not IsHandled then ++begin
            if not "System-Created Entry" then
                GLAcc.TestField("Direct Posting", true);
            Description := GLAcc.Name;
            "Gen. Prod. Posting Group" := GLAcc."Gen. Prod. Posting Group";
            "VAT Prod. Posting Group" := GLAcc."VAT Prod. Posting Group";
            "Tax Group Code" := GLAcc."Tax Group Code";
            "Allow Invoice Disc." := false;
            "Allow Item Charge Assignment" := false;
            InitDeferralCode();
    ++end;
        OnAfterAssignGLAccountValues(Rec, GLAcc, SalesHeader);
    end;

Please just use existing version, obsoleting it now, to change again seems like adding to the mess, and without obsoleting it would be breaking for someone using it as is.

After more consideration we will obsolete this current event and introduce a new one you can use
OnBeforeTestDirectPosting

Thank you!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 53520.


Combine multi-symbol Altium parts into one component

Part of #105, combines multi-symbol Altium parts into one part, which produces an accurate BOM. With this and the component kind changes, there shouldn't be any major inaccuracies in py-allspice generated BOMs.
Note: The strategy used by this PR relies on Tools' behaviour, which is to append the designator ordinal to the part name when the symbol is one of a multi-symbol part. Ideally we would "just" read the Current Part ID field as it is present in the component of the schematic, but this avoids any changes in Tools.
Stacked on #123 as that should get merged before this.

We don't want to rely on the behavior of Tools if it's going to go away in the future. Is there a change in Tools that would make it more correct that we should consider making?

Unfortunately all of the current Altium BOM gen depends on Tools behaviour because the JSON is just so different. When we move on from Tools, we'll have to rewrite almost all of it anyway.
However, putting this information in the IR is very relevant to crysknife as well, because currently the crysknife IR doesn't include it at all. OrCAD does something similar to Altium here, KiCAD doesn't include it unless you look at the graphics within the component etc. So, right now, we can't make a choice for tools that would make it "future proof" unless we commit to doing the same thing for crysknife.

We don't want to rely on the behavior of Tools if it's going to go away in the future. Is there a change in Tools that would make it more correct that we should consider making?

Unfortunately all of the current Altium BOM gen depends on Tools behaviour because the JSON is just so different. When we move on from Tools, we'll have to rewrite almost all of it anyway.
However, putting this information in the IR is very relevant to crysknife as well, because currently the crysknife IR doesn't include it at all. OrCAD does something similar to Altium here, KiCAD doesn't include it unless you look at the graphics within the component etc. So, right now, we can't make a choice for tools that would make it "future proof" unless we commit to doing the same thing for crysknife.

Can you list the differences? We will need to catalog them anyways. I’m of the mind it will make sense to patch the differences in tools rather than support multiple code-paths here.

Can you list the differences? We will need to catalog them anyways. I’m of the mind it will make sense to patch the differences in tools rather than support multiple code-paths here.

There are many structural and simple differences. Listing them all would require some effort, especially as we need to make changes to crysknife's IR as well. Either way, changing Tools' output to match crysknife would also require significant changes to Hub. This is why we had to switch KiCad from the single document viewer to the multi document viewer. I don't think it makes sense to coordinate large scale changes across py-allspice, tools and hub when compared to moving Altium Schematics to crysknife.
W.r.t multiple code paths, this doesn't involve that right now. Altium has to have a different system than OrCAD anyway. We'd only have multiple paths if we have to support a tools to crysknife (or tools to tools formatted like crysknife) transition. For the latter, that ship has already sailed, and there's no way to avoid multiple code paths. Even if we change tools to match crysknife right now, the current code will have to remain for some time at least, and we'd have to support it until we're sure nobody has tools formatted ouptuts.

Im okay with making the hack fix here (in the meantime), since it’s currently broken, but I would like to prioritize fixing it correctly in tools + crysknife.

I wouldn't call this fix a hack, this is just how Tools marks multi-symbol parts. Even if we were keeping Tools around forever, this would be the way to do it.

Force push with no changes to trigger checks.

Builds fail due to #128.

Force push with no changes to trigger checks.

I think you should be go to the action, e.g. here, and click "Re-run all jobs", if that's easier.

In that case there were no runs, because the base of the branch had changed to main from the stacked PR. AFAIK the force push is the only way to re-triggger in that case.

@kdumontnu Do you want to wait until we've resolved the IR issue before taking on the multi-symbol tasks? If so, I will close this PR (and the OrCAD one at #126) and re-do those when the IR decisions have been made.

In that case there were no runs, because the base of the branch had changed to main from the stacked PR.

Ah, I see. In other repos, I changed the trigger to be on all pull requests, not just to main. Not sure if this repo needs to be different because it's public, but because of stacking PRs, I prefer that the target doesn't matter.

Due to issues with mutli-page components that are hard to fix without changing the Tools IR, and the fact that we're planning on changing Tools anyway, I'm closing this and will re-work it with the Tools changes later.


Properly model 6502's BRK opcode.

Model 6502's BRK opcode by following the sequence defined in the MCS6500 programming manual, as in:

The next instruction is pushed to the stack.
The B flag is set to 1.
The status flag byte is pushed to the stack.
The I flag is set to 1.
PC jumps to the IRQ vector.

I think it is a good fix to implement the BRK instruction. The pcode looks mostly good except that I believe the IRQ vector stores an address. Rather than jump to the IRQ vector, the IRQ vector should be read to get the address to jump to.

This should actually perform the indirect jump through the IRQ vector pointer, sorry for the inconvenience.


Fix nightly integration test script for specfic artifacts

Signed-off-by: Peixin Li<EMAIL_ADDRESS>fix https://github.com/NVIDIA/spark-rapids/issues/3592. related to https://github.com/NVIDIA/spark-rapids/pull/3533
verified locally. This will require a nightly rebuild before re-trigger integration pipelines

build


Persistent effects sometimes persist far too long

In several cases during the game, both player tokens and NPC tokens received persistent effects out of critical hits and misses (62/melee attack, for example) which seemed to be consistently registered with the right token but did not disappear even after another combat action script was executed. In the example, the NPC token received another attack, but after the next attack (which failed) the effect was not gone.

Maybe not actually a bug, if we would use the end-of-round-effects script... to be observed again next time.

Resolved via https://github.com/michael-buschbeck/mychs-macro-magic/pull/234


.Net Core Preview 4 - Background Service?

I'm trying to understand how to configure AppMetrics within a console background worker project using .Net Core preview 4.  I have so far
        public static IHostBuilder CreateHostBuilder(string[] args) =>
           Host.CreateDefaultBuilder(args)
                .ConfigureServices(services =>
                {
                    services.AddMetrics();
                    services.AddHostedService<Worker>();
                });
    }

How do I specify options for the AddMetrics, also outputting metrics to the console?

Do the sample projects help?


ci/download-wasmtime.py: keep empty.go files

Before, the download-wasmtime.py script would remove build/, and
add empty.go files for every architecture it has unpacked library
artifacts for. Since includebuild.go mentions more than that,
notably macos-aarch64 (which there are no unpacked library
files), we ended up in a state where the go module couldn't be
built.
Now, the script will remove all regular files in build/, and re-
populate the directory from the downloaded tarballs. In its final
step, it drops empty.go files in all directories, even those it
that did not get recreated.
Fixes #112.
This is find build after running the script locally:
$ find build
build
build/macos-aarch64
build/macos-aarch64/empty.go
build/linux-x86_64
build/linux-x86_64/empty.go
build/linux-x86_64/libwasmtime.a
build/linux-aarch64
build/linux-aarch64/empty.go
build/linux-aarch64/libwasmtime.a
build/include
build/include/wasmtime
build/include/wasmtime/error.h
build/include/wasmtime/trap.h
build/include/wasmtime/config.h
build/include/wasmtime/global.h
build/include/wasmtime/empty.go
build/include/wasmtime/memory.h
build/include/wasmtime/linker.h
build/include/wasmtime/table.h
build/include/wasmtime/instance.h
build/include/wasmtime/extern.h
build/include/wasmtime/func.h
build/include/wasmtime/module.h
build/include/wasmtime/store.h
build/include/wasmtime/val.h
build/include/wasmtime.h
build/include/wasi.h
build/include/empty.go
build/include/wasm.h
build/include/doc-wasm.h
build/empty.go
build/macos-x86_64
build/macos-x86_64/empty.go
build/macos-x86_64/libwasmtime.a
build/windows-x86_64
build/windows-x86_64/empty.go
build/windows-x86_64/libwasmtime.a

Signed-off-by: Stephan Renatus<EMAIL_ADDRESS>

Thanks! I think windows CI may be failing though?

Oh Windows. I'll have a look later...


[Bug]: widget's name is not getting focused on adding new widget

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When adding a new widget, its name is not getting focused if we are doing a reflow.
https://www.loom.com/share/e82c783cf6b940608bbfd95a3cd9104b
Steps To Reproduce
Add a new widget and while dropping it, make other widgets move. It will make the first property in focus rather than the widget name.
Public Sample App
No response
Version
Cloud

Waiting for the fix for bug - https://github.com/appsmithorg/appsmith/issues/14512 -  to test this issue.


Fix: added jsonref to requirements.txt (Issue 1183)

Fixes issue https://github.com/instructor-ai/instructor/issues/1183

[!IMPORTANT]
Add jsonref to requirements.txt to resolve issue #1183.

Dependencies:

Added jsonref to requirements.txt to address issue #1183.

This description was created by  for 9a1c6763eb9ee1daa1ed6ccb8ab5b7117459bfad. It will automatically update as commits are pushed.

Closing this for now since users are expected to install this dep using pip install "instructor[vertexai]


Fix clickable legends for some elements

Test notebook
https://anaconda.org/ahuang11/test_muted_alpha/notebook

Thanks, rebuilding the test data now. Will merge once it passes.


Customizable device name ~

Add customizable device name for startDeviceScan not always necessary to use device default name.

Can you elaborate more? I'm not sure what is your requirement?

can we customize our device name with js without changing our phone bluetooth display name?

Changing device name must be supported by device itself. You should check if your manufacturer allows that and how to do it. There is common Device Name characteristic which can be used to read and in some cases to override device name (if it's writable).

oh ya, I got another question, can I get device id ( Mac Addr & UUID ) without opening Bluetooth?

Check answer at #92


[th/make-generate-tidy] makefile: run go mod tidy during make generate

We have multiple go.mod files, this does not seem to agree with operator-sdk. operator-sdk will generate files and call make generate. However, that will then fail because we didn't call go mod tidy for api/ directory.
Note that compared to sriov-network-operator, in dpu-operator we have several go.mod files. That does not seem to agree with operator-sdk, or at least, something is not hooked up right.
Hack make generate to call go mod tidy first.
Otherwise:
  $ git checkout -B C origin/main && \
    git reset --hard HEAD && \
    git clean -fdx && \
    make operator-sdk && \
    ./bin/operator-sdk create webhook --group config --version v1 --kind DpuOperatorConfig --defaulting --programmatic-validation
  branch 'C' set up to track 'origin/main'.
  Reset branch 'C'
  Your branch is up to date with 'origin/main'.
  HEAD is now at e58a9c9c830a Merge pull request #194 from thom311/th/hack-scripts-cleanup
  INFO[0000] Writing kustomize manifests for you to edit...
  ERRO[0000] Unable to find the target #- path: manager_webhook_patch.yaml to uncomment in the file config/default/kustomization.yaml.
  INFO[0000] Writing scaffold for you to edit...
  INFO[0000] api/v1/dpuoperatorconfig_webhook.go
  INFO[0000] api/v1/dpuoperatorconfig_webhook_test.go
  INFO[0000] api/v1/webhook_suite_test.go
  INFO[0000] Update dependencies:
  $ go mod tidy
  INFO[0000] Running make:
  $ make generate
  test -s /data/src/dpu-operator/bin/controller-gen && /data/src/dpu-operator/bin/controller-gen --version | grep -q v0.15.0 || \
  GOBIN=/data/src/dpu-operator/bin GOFLAGS='' go install<EMAIL_ADDRESS>  GOFLAGS='' /data/src/dpu-operator/bin/controller-gen object:headerFile="hack/boilerplate.go.txt" paths="./..."
  Error: load packages in root "/data/src/dpu-operator/api": err: exit status 1: stderr: go: updates to go.mod needed; to update it:
          go mod tidy

Doesn't this behave the same way if we didn't have a go.mod?

This patch has only an effect, when make generate is called.
I don't understand what k8s' make generate is supposed to do. So the answer depends on that.

I don't think we want to mix make generate with go mod tidy. I think go mod tidy should be run separately.

I don't think we want to mix make generate with go mod tidy. I think go mod tidy should be run separately.

There is a reproducer in the commit message that you can just run.
How do you suggest to solve this differently? Especially seeing that make generate is not run by me directly, but by operator-sdk indirectly?

It think the right place is make vendor , and we also need to extend make vendor-check

/unhold
/ok-to-test

Note that the build target also includes the generate target (build is also the default target). That means, a plain make or make build now also runs go mod tidy. That seems wrong that a build tries to generate/tidy the source files.
But I think the problem of that is that it's wrong that build does a generate. The problem is not that generate should not do a go mod tidy.

/test make-e2e-test

/test make-e2e-test

See similar comment from @bn222 here:

This prevents separate commits with vendor

That part is now dropped. make generate will no longer also depend on vendor target. Which defeats most of the purpose of why this commit was done. However, that part will temporarily come back in #213.

/lgtm
/approve
/ok-to-test


Wrong version in HystrixMetricsPoller @Deprecated comment

Here is comment:
@Deprecated //since 1.5.4
But latest version is 1.5.3.
Is this is a mistake?
Thank you.

This is in master but unreleased.  Should be out soon.

@mattrjacobs  I understood, that's why I closed it, thank you. What is approximate release date?

Should be in the next week or so

@mattrjacobs  Thank you

@mattrjacobs Any updates about 1.5.4 release?

Should be tomorrow.  Thanks for your patience


Single select with :clear-on-select='false' and :preserve-search="true" throw warn "ClearOnSelect and Multiple props can’t be both set to false."

Why single select can't be with :clear-on-select='false' ?
Reproduction Link
https://jsfiddle.net/karakum/jqofkzxc/3454/
Steps to reproduce

Run as is
Open select
Type search string
Select any item

Expected behaviour 1
First option must stay modified
Actual behaviour 1
First option modified as expected, but:
Console warn: ClearOnSelect and Multiple props can’t be both set to false.
Modify :clear-on-select to true and repeat scenario.
Expected behaviour 2
First option must stay modified
Actual behaviour 2
First option has been reset with suffix (reset) because search query reset on select

Any suggest?

@karakum if you set clear-on-select with single select then there will be no options to deselect it.

@akki-jat but I need to clear-on-select=FALSE with single select to prevent clear of search string after select item

@karakum I also don't think there is any connection betwwen clear-on-select and multiple prop.
But author would know better, so I am adding @shentao in loop.

This is a legacy warning when the single-selection was using the input itself to display the value. After the recent changes (the single value has it’s own span), this no longer seems needed. Feel free to provide a PR that will remove the warning.

@shentao merge PR please


[$500] Task - Pointer shows up for an archived task when hovering over the edge of the checkbox

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

Version Number: <IP_ADDRESS>
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Issue found when executing Pr https://github.com/Expensify/App/pull/31674
Action Performed:

Open ND and log in with any account
Go to announce room
Create a task
Delete the workspace in whose announce room the task is just created
Note that the task report was archived in LHN
Tap task in the archived announce room
Hover over the edge of the checkbox carefully

Expected Result:
No pointer should show up since the task is archived and can not be checked
Actual Result:
A pointer shows up when hovering over the edge of the checkbox
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/5152902f-05bd-4f45-bc06-e0d186e277e0
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0115fb6326c37966b2
Upwork Job ID:<PHONE_NUMBER>944112640
Last Price Increase: 2024-02-26

Proposal
Please re-state the problem that we are trying to solve in this issue.
Task - Pointer shows up for an archived task when hovering over the edge of the checkbox
What is the root cause of that problem?
The canModifyTask function checks if the workspace or parent report is archived.
https://github.com/Expensify/App/blob/297bcec1b2013c22a71c1065ed8d3ddf337862fd/src/libs/actions/Task.ts#L884-L887
What changes do you think we should make in order to solve the problem?

Instead of checking for the parent report is archived, we can remove this condition ReportUtils.isArchivedRoom(parentReport).

Definitely do not need to be blocking deploy on something so minor

Great explanation @brandonhenry let's move forward with your proposal!
Also, once you've submitted your PR, please complete the checklist and be sure to include a regression test to ensure that this bug doesn't crop up again, thanks!

@Krishna2323 my mistake, I didn't realize that the proposal was a duplicate. In fairness, I will assign you, thanks for making me aware.

Deployed to staging 13 hours ago, Melv.

👋 @ntdiary checklist time! :)

Haha, that it does! 🕐 Cool, I've asked Matt for an update on that quickly. If not, I can pay you via Upwork.
In the meantime.. I don't think it's worth having a regression test for this pointer cursor. 👍 Payment summary as follows:
$500 to @ntdiary for the C+ review
$500 to @Krishna2323 for the fix

Yep, we're proceeding with Upwork for now.
@Krishna2323 - paid
@ntdiary - waiting for you to accept the offer. :)

Perfect, paid. Closing! 👍


Make dashboards stateless

At the moment, the dashboard stores the information whether a collection, group, or variable is selected in global variables. This makes it impossible to use the dashboard in a deployed environment, as the state would be the same for users who use it concurrently. Either use states in callbacks or a user store.

Closed by #37


Server link in the technology section is not working

In the technology section the server url is not working.
In the server url serverUsageId is used instead of serverId.

Fixed, thanks for the analysis Muzammil


[CLOSED] Notation for "<" in GHC/Base.v

 Issue by sweirich
Saturday Dec 02, 2017 at 14:13 GMT
Originally opened as https://github.com/antalsz/hs-to-coq/issues/53

The midamble.v file for GHC.Base defines the notation
Infix "<" := (op_zl__) (no associativity, at level 70).

However, this notation conflicts with Coq's default notation for lt, the less-than proposition.
Furthermore, this notation is not defined in a notation scope or in a submodule, so any file that imports the Prelude imports it.
I prefer using the notation "<?" for this operation (in line with the rest of Coq's libraries) instead of Haskell's notation. This makes it more clear that it is a bool returning function, not a prop. Others may disagree --- please discuss.
But, regardless, we should edit Base to make this notation more controllable.

 Comment by nomeata
Saturday Dec 02, 2017 at 15:53 GMT

Doesn’t <? also clash with Coq’s notation for the boolean predicate?
I am fine with any name. Just make sure the method is renamed consistently (i.e. to op_zlzq__), as this correspondence is now an invariant. For a generated method definition, this should just work out of the box:
rename value GHC.Classes.>   = GHC.Base.>?

Since this is a manual definition, you have to manually adjust the method name and notation  in GHC.Base’s midamble. But that should do the trick.

 Comment by sweirich
Saturday Dec 02, 2017 at 17:51 GMT

I know about the clash. That is why I want to have a discussion instead of unilaterally changing it.
We also should be putting these (and other) notations in a scope.

 Comment by nomeata
Saturday Dec 02, 2017 at 18:13 GMT

shrug if the clash with < it gets in the way of your work but <? works nice, then just change it. I don’t think this needs too much discussion. (Especially as it is “just” an edit, and not something hardcoded in the tool.)

We also should be putting these (and other) notations in a scope.

I just read through https://coq.inria.fr/refman/syntax-extensions.html#scopes, but it is not immediately clear to me how to best make use of that. What do you have in mind?
(A guess: Declare all our notations in scope hs and Open Scope hs by default. This allows users to use (a < b)%hs and (a < b)%type (or is it (a < b)%nat) to disambiguate. And after Close Scope hs all notation is disabled, which can be useful for debugging.)

 Comment by antalsz
Monday Dec 04, 2017 at 18:28 GMT

Ssreflect uses < for computational/Boolean equality, which combined with Haskell's use in the same way, pushes me a little to sticking with Haskell's approach.  (There's a bit of "best of a bad bunch" going on with the choices here anyway.)  And yes, we should definitely put everything in our own scope.

 Comment by nomeata
Sunday Apr 08, 2018 at 03:43 GMT

I think this is obsolete, and either not an issue any more, or an issue that can be fixed with a single rename edit.


Setup failing

The docker image is failing setup by this command from the documentation:
docker run -it --rm --link some-postgres:postgres sentry sentry createsuperuser

and this error:
Traceback (most recent call last):
  File "/usr/local/bin/sentry", line 11, in <module>
    sys.exit(main())
  File "/usr/local/lib/python2.7/site-packages/sentry/utils/runner.py", line 428, in main
    initializer=initialize_app,
  File "/usr/local/lib/python2.7/site-packages/logan/runner.py", line 191, in run_app
    configure_app(config_path=config_path, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/logan/runner.py", line 104, in configure_app
    raise ValueError("Configuration file does not exist at %r" % (config_path,))
ValueError: Configuration file does not exist at '/home/user/.sentry/sentry.conf.py'

Any updates?

We are working on a branch to hopefully fix this and make it easier to run sentry.  It will include a conf file to automatically work with linked postgres/mysql, redis, and memcached containers.  Currently it is in https://github.com/infosiftr/sentry/tree/conf.  I welcome any feedback.

Not sure where you want the feedback, so putting it here :)
The thing I was lacking apart from the config file, is the celery workers. They should be started (I believe) to get everything up and running, but a quick look in your proposed branch doesn't seem to start them?
I could be way off the mark here, as I'm not an expert, but I needed to fiddle with the workers to get my manual test-install going, so I suppose a docker-version should also have them?

The workers should be run separately, especially in separate containers.
:smile:

Yeah, I looked at the links file and didn't see the workers, that's why I feedbacked it :)

+1

This is fixed and updated in the documentation now. :+1:

@tianon I am still getting this error somehow.

@lenovouser Are you using the latest version? 8.1.2?

@mattrobenolt I thought I was because I used :latest, but it used a older version somehow. Got some other issues now though, will create a new issue for that. Thanks!


ref 丢失

https://github.com/alibaba/fastjson/issues/3259

 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.

有testcase跑不过

@wenshao 已经修复testCase了，如果不修改src/main/java/com/alibaba/fastjson/parser/DefaultJSONParser.java 则testcase跑不过


[BUG] Planning Worksheet, Routing Description overflow by Text Variable

Environment Version: BC18.2 OnPrem
Localization: IT-it
GetDescriptionAndRcptName (codeunit 99000812 PlanningWkshManagement) triggered in OnAfterGetCurrRecord (page 99000852 "Planning Worksheet")
Good morning, I found a BUG in the “Planning Worksheet (99000852, Worksheet)” page. It seems that the function to get the descriptions of Rountings, is handled by inserting a text variable length 50 char (present in the page), where instead within the procedure's parameter is by var and length 100 characters. This causes a over-flow issue. it would be nice to solve it, in the next CU.
Thank everyone.
Ps: I enclose a screen about the bug.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


7-Add endpoints with tests

#33
Due date
Thursday, 2020 March 12.

Add a service setShipping and submitOrder that contains all the related methods in OrderProductUnitTest #39 .
Follow TODOs.

Order::setShipping() brief funcionality
$this->assertEquals(true, $this->orderProductRepo->markDomesticShipping($draftOrder));
$this->assertEquals(true, $this->orderRepo->markExpressShipping($draftOrder, 'n'));
$this->assertEquals(true, $this->orderProductRepo->markExpressShipping($draftOrder)

$this->assertEquals(true, $this->orderProductRepo->setShippingRates($draftOrder));
$this->assertEquals(true, $this->orderRepo->setOrderCostsFromCartItems($draftOrder));

Pass

Address.
is express.


worker: detect missing runc early

Signed-off-by: Tonis Tiigi<EMAIL_ADDRESS>

@AkihiroSuda Agreed


merge from konsum-frontend-svelte,template-css,template-svelte-app

package.json

chore(deps): add ^10.11.0 remove ^10.10.0 (devDependencies.npm-pkgbuild)

:tada: This PR is included in version 1.32.0 :tada:
The release is available on:

npm package (@latest dist-tag)
v1.32.0
GitHub release

Your semantic-release bot :package::rocket:


Autocomplete on discovery tab not working

Hi,
I'm having issues getting to work the autocomplete for fields when typing into the discovery filter field. Any ideas why this could be happening?

@mporracindie  This feature is part of x-pack and thus it is not provided by open source kibana.

Kibana Query Languageedit
In Kibana 6.3, we introduced a number of exciting experimental query language enhancements. These features are now available by default in 7.0. Out of the box, Kibana’s query language now includes scripted field support and a simplified, easier to use syntax. If you have a Basic license or above, autocomplete functionality will also be enabled.


limelib + mekanism + modular machinery (fluid output/input)= crash

Interesting crash
Minecraft version 1.12.2 forge <IP_ADDRESS>7
3 mods loaded (not counting JEI)
mekanism latest version <IP_ADDRESS>.326
modular machinery 1.6.5 (https://minecraft.curseforge.com/projects/modular-machinery)
limelib 1.12.2-1.7.9
If either one of the three mods are disabled the crash doesn't happen. The crash happens when a fluid output/input from modular machinery is placed in world and limelib tries to read the block's info. I think it has something to do with the fact that it handles Mekanism gasses also when that mod is active.
crashlog: https://gist.github.com/Staegrin/cf967391278c9912f08c5ab466391f55
crashlog client: https://gist.github.com/Staegrin/5f86b54f02a54e4642da31aabb8ee18f
crashlog server: https://gist.github.com/Staegrin/d39af449461189fbd7b219b1c4d037e3

mekanism + modular machinery = no problem

modular machinery + limelib = no problem

I guess it is not my fault.
@HellFirePvP
you should not call from unknown capabilites. not every one does support that. or put it into try-catch.

Sorry, the whole mekanism integration was past-midnight code :P
i'll look into it for the next update.


how to use multi update with sharding-jdbc

I use mybatis sql like this:

update trade_contract_order
set
gmt_modified = now(3)
where user_id=#{item.userId} and id = #{item.id}

also I add "allowMultiQueries=true” in jdbc url;
but it seems like that only first sql works.
How do I solve this?

Not support now


[Package Issue]: Discord.Discord fail to upgrade.

Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue
Installation issue.
Brief description of your issue
Upgrading Discord from 1.0.9007 to 1.0.9008 fails.
Steps to reproduce

Have installed Discord Client v1.0.9007
Call winget upgrade Discord

Actual behavior

You get Error:<PHONE_NUMBER> access denied.
Excerpt from log Discord Installer log:

...
8248> 2022-11-24 13:29:16> ApplyReleasesImpl: Squirrel Enabled Apps: [C:\Users\bartus\AppData\Local\Discord\app-1.0.9007\Discord.exe]
8248> 2022-11-24 13:29:33> Utility: Process Started: C:\Users\bartus\AppData\Local\Discord\app-1.0.9007\Discord.exe --squirrel-install 1.0.9007, pid 17684
8248> 2022-11-24 13:29:39> Utility: Received exitcode 0 from process C:\Users\bartus\AppData\Local\Discord\app-1.0.9007\Discord.exe
8248> 2022-11-24 13:29:40> ApplyReleasesImpl: ran C:\Users\bartus\AppData\Local\Discord\app-1.0.9007\Discord.exe, pid 5036
8248> 2022-11-24 13:29:40> ApplyReleasesImpl: Starting fixPinnedExecutables
8248> 2022-11-24 13:29:40> ApplyReleasesImpl: fixPinnedExecutables: newCurrentFolder: app-1.0.9007
8248> 2022-11-24 13:29:40> ApplyReleasesImpl: fixPinnedExecutables: enumerating path C:\Users\bartus\AppData\Roaming\Microsoft\Windows\Start Menu failed: System.UnauthorizedAccessException: Access to the path 'C:\Users\bartus\AppData\Roaming\Microsoft\Windows\Start Menu\Programy' is denied.
   at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)

Expected behavior
It should upgrade Discord 1.0.9007 to 1.0.9008
Environment
Windows Package Manager v1.3.2691
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.19044.2251
System Architecture: X64
Package: Microsoft.DesktopAppInstaller v1.18.2691.0

Logs: %LOCALAPPDATA%\Packages\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe\LocalState\DiagOutputDir

Links
---------------------------------------------------------------------------
Privacy Statement   https://aka.ms/winget-privacy
License Agreement   https://aka.ms/winget-license
Third Party Notices https://aka.ms/winget-3rdPartyNotice
Homepage            https://aka.ms/winget
Windows Store Terms https://www.microsoft.com/en-us/storedocs/terms-of-sale

Screenshots and Logs
SquirrelSetup.log

Two things:

was Discord running when you tried to upgrade it? That always makes it fail for me.
Start Menu\Programy doesn't seem right, Discord's installer might have a bug?

Close discord and then try to update it.

Close discord and then try to update it.

@stephengillie changing UpgradeBehavior to uninstallPrevious will not help if the application was running in the background?

The behavior for discord has been updated so that a more clear message is shown
Close with reason: UpgradeBehavior changed;


Finally! An Emacs icon as gnarly as Doom

Wanting something other than the Spacemacs icon that came with emacs-plus and the more generic-looking purple one, I wanted something a bit more fun and wild.

doom.icns.zip
The goal would be to submit a PR to emacs-plus as a --with-doom-icon option.
Progress\versions can be tracked at https://github.com/eccentric-j/doom-icon
Anyway, I hope someone out there enjoys it!

It was requested to match the original style of the image on the README. I had reached out to the original artist to ask permission to use my vectorized version but I have yet to hear from them. Instead, I made a few changes to help differentiate it from their original work so we should be able to use it more freely.

The iconsets of both can be downloaded here https://github.com/eccentric-j/doom-icon/releases/download/0.0.2/doom-icons-0.0.2.zip

sorry for the spam, but i just have to say this is awesome!!

@tam5 Thanks. I appreciate the feedback!

@eccentric-j this icon is pretty cool, I even used it to make a banner

Me too! This icon is awesome!

I love this. @eccentric-j are you still planning to open a PR for emacs-plus? I would be happy to initiate this if you don't want to.

@iwahbe I appreciate that offer but I don't feel it should be done without @hlissner's blessing 🤔

Hey there. These are lovely.
I just started learning doom, and am not familiar with github publishing, but I've also made a little icon for the app.

Hope you guys like it, there's .png, .ico and .icns versions in this zip:
doom emacs icons.zip


Simulator base methods

Right now, simulators implement a number of methods. Besides simulate, those include get_ground_truth_parameters, get_ground_truth_observation, normalization_parameters, parameter_plotting_limits.
Ideally, however, a simulator can simply be a function that returns outputs given parameters. Ground truth is not always available, normalisation and plotting parameters should be inferred automatically.

simulator will be reduced to just a __call__ method, such that the current implementation of the simulators as classes will still work, and new simulators can be passed as just a python function.

Is this still WIP or all done?

still WIP. I want to get rid of the name,  and dims properties in the code so that it actually works to just pass a function as simulator.


Noble forwarding addrs

Implement the web version in Prax of https://github.com/penumbra-zone/penumbra/pull/4878. Noble launching feature Early Nov.
Follows this userflow:

Getting sequence numbers
There are three RPC responses to the register call:

unregistered, no deposits (failure 1)
registered already (failure 2)
unregistered, deposit exists (success)

Should use binary search to find the next available sequence number for user. Prax can store it directly and the latest sequence number can be found on demand with at most 16 RPC requests
Let's say someone has already generated 50 addresses. Any sequence <=50 will result in an RPC success (aka already deposited). Anything >50 will return an error. So we keep splitting the address space in half, checking for success/error until we come down to two (50 success, 51 error).
Proposed design

States to consider:

If toggling the noble one, should replace the identicon with Noble icon + Noble address
Likely should ditch the incognito icons
Have a disable UI state so two can't be toggled at once
Requires refactoring this component
Needs loading/loaded/error states

This should be moved back into the backlog – according to henry, "forwarding addresses may have slipped schedule and might be pushed to next year."


Feat: Add data transformations.

Add transformation feature.
Addresses issue documented here: https://github.com/prometheus-community/json_exporter/pull/133
Allows for json object to get filtered by JQ before the json path is applied.

This is working perfectly for our use case. Massive +1! 👍


build: Add GA 4 script once site is live.

This will be added once the site is live since the GA4 setup expects to see a live site. The GA script must only be live on prod, not localhost or stg.

This was fixed in https://github.com/swimos/swimos-docs/pull/68


No way to inline tables during serialization of array of tables

Both these toml docs are equivalent:
[[bins]]
name = 'native-image'

[bins.target]
type = 'Sdk'
name = 'jdk-graal'

[[bins]]
name = 'native-image'
target = { type =  'Sdk', name = 'jdk-graal' }

The second format is more compact and human-friendly. E.g. that's the default style of cargo.toml, promoted by all the docs.
Though seems like there is no way to get the second form by using toml::to_string. Though it can be parsed correctly.
Could you please add a way to use the second output format?

Probably related: https://github.com/alexcrichton/toml-rs/issues/265

i'd love to see an option to serialize into inline tables.
i'm using this crate for serializing a default config file, and it doesn't look particularly good

So this actually isn't supported? Has anyone managed to write a custom deserializer for this or that's not how it works?

So this actually isn't supported? Has anyone managed to write a custom deserializer for this or that's not how it works?

Yeah, that would totally fix the issue.

Maintenance of this crate has moved to the https://github.com/toml-rs/toml repo.  As a heads up, we plan to move toml to be on top of toml_edit, see https://github.com/toml-rs/toml/issues/340.
Closing this out.  If this is still a problem, feel free to recreate this issue in the new repo.


Event details page: Remove the end date when events run past midnight

Description of issue
The Event details page currently shows a date range when events run past midnight.

Expected behaviour
Only show the start date as events aren't intended to run over several days and it should be obvious to users that some events will run into the early hours of the following day.

This still needs a final decision, possibly awaiting some further user testing, so please hold off working on this without UXD approval.

@declanslevin UXD is happy to just have the date when the event starts, and remove the end date


Add Plano ISD to edu

https://www.pisd.edu/
Plano ISD District Page
https://www.pisd.edu/pshs
Webpage for Plano Senior - one of the 3 Senior Highs
https://www.pisd.edu//Page/22082
Computer Science Courses offered by Plano ISD

@jyeeps Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.