Figure: haplotype structure

Create a figure showing haplotype structure.

@cclarkson here's some food for discussion tomorrow:

...this is a clustering of haplotypes carrying L995S. Only distinct haplotypes are used to construct the tree. Haplotype frequencies are shown in the adjacent bar plot. Shaded regions show the 5 largest clusters by a method I'll explain tomorrow - I'm thinking we show one network for each of these 5 clusters.

Hey @cclarkson, maybe Python is an option after all, I made this with graphviz:

cc @hardingnj.

That's great, will be cool if we could keep most of it in python. Looking forward to seeing the code.

Making some progress, here's a network with some real data, the Kenya/Uganda L995S cluster:

Will turn this into a usable function and share asap.

Still not sure how to figure out which edge has which mutation, that may be tricky...

Hi @alimanfoo ,
Just to document the TCS comparison, almost all clusters produced identical networks to your function, except for 995F8, which TCS did not circularise and 995F16, which added a number of loops compared to the one you generated - see below (I tried to lay it out the same to compare but TCS kept crashing, this is best I managed, the bottom right hand corner is a bit messy!).

Thanks Chris. Can you compare with the networks I made using median-joining? They should be in the (recently updated) hapclust_demo notebook, look out for the calls to graph_haplotype_network(..., network_method='mjn'). What you have above in TCS looks very similar to what I made with MJN...

@alimanfoo
Yep, I'm pretty sure that is the same as the TCS one, interesting.
ION - having a problem getting graphviz to work due to neato - how did you fix that problem?

No worries, think I've cracked it.

Hi @alimanfoo ,
Now I've got this all up and running again, I was just trying to plot multiple networks on one figure but I'm struggling. I can't see how to get it to use an 'ax', is this because it is just a wrapper for external code and can't talk to matplotlib?

Hi Chris, sorry yes, I haven't figured out the best way to get the networks
into a mpl figure. I'll have a play around...
On Thu, Mar 9, 2017 at 4:19 PM, Chris Clarkson<EMAIL_ADDRESS>wrote:

Hi @alimanfoo https://github.com/alimanfoo ,
Now I've got this all up and running again, I was just trying to plot
multiple networks on one figure but I'm struggling. I can't see how to get
it to use an 'ax', is this because it is just a wrapper for external code
and can't talk to matplotlib?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/malariagen/agam-vgsc-report/issues/25#issuecomment-285399532,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAq8QjyTXWPUdlCPk95wE9ytk49Yuzurks5rkCZ_gaJpZM4MQYub
.

--
Alistair Miles
Head of Epidemiological Informatics
Centre for Genomics and Global Health http://cggh.org
The Wellcome Trust Centre for Human Genetics
Roosevelt Drive
Oxford
OX3 7BN
United Kingdom
Email<EMAIL_ADDRESS>Web: http://purl.org/net/aliman
Twitter: https://twitter.com/alimanfoo
Tel: +44 (0)1865 287721

Hi @alimanfoo ,
I'm trying to find a nice way to bring the dendrogram together with the clusters but I'm struggling to work out how to give the fig_haplotypes_clustered function (from haplclust_utils) an ax to plot on in my composite figure because the dendrogram function is already a composite figure. Any pointers would be gratefully received!
Cheers, C.

Hi Chris, I would suggest building each panel of the figure as a separate
matplotlib figure, saving out to jpeg or png, then composing the whole
figure in libreoffice draw, at least for now.
On Tuesday, April 11, 2017, Chris Clarkson<EMAIL_ADDRESS>wrote:

Hi @alimanfoo https://github.com/alimanfoo ,
I'm trying to find a nice way to bring the dendrogram together with the
clusters but I'm struggling to work out how to give the
fig_haplotypes_clustered function (from haplclust_utils) an ax to plot on
in my composite figure because the dendrogram function is already a
composite figure. Any pointers would be gratefully received!
Cheers, C.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/malariagen/agam-vgsc-report/issues/25#issuecomment-293221001,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAq8QgYwfaX7Z2JqA_q4dTDmXNZEVZTQks5ru1qIgaJpZM4MQYub
.

--
Alistair Miles
Head of Epidemiological Informatics
Centre for Genomics and Global Health http://cggh.org
The Wellcome Trust Centre for Human Genetics
Roosevelt Drive
Oxford
OX3 7BN
United Kingdom
Email<EMAIL_ADDRESS>Web: http://purl.org/net/aliman
Twitter: https://twitter.com/alimanfoo
Tel: +44 (0)1865 287721

Been flailing at this for a while and I think I am finally getting somewhere! I'm now trying to work out how to colour the network's backgrounds like the dendrogram (and align/scale things in a less awful way).

Cool. If you think it isn't going to work to fit the tree and networks in
the same figure, please feel free to rethink.
On Tue, Apr 11, 2017 at 6:13 PM, Chris Clarkson<EMAIL_ADDRESS>wrote:

[image: test_fig]
https://cloud.githubusercontent.com/assets/13287471/24921332/57756eda-1ee2-11e7-8c2b-fa4a4bc6662b.jpg
Been flailing at this for a while and I think I am finally getting
somewhere! I'm now trying to work out how to colour the network's
backgrounds like the dendrogram (and align/scale things in a less awful
way).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/malariagen/agam-vgsc-report/issues/25#issuecomment-293333094,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAq8QrAij8TQYoxCQAkYeHYATY28HGqYks5ru7TRgaJpZM4MQYub
.

--
Alistair Miles
Head of Epidemiological Informatics
Centre for Genomics and Global Health http://cggh.org
The Wellcome Trust Centre for Human Genetics
Roosevelt Drive
Oxford
OX3 7BN
United Kingdom
Email<EMAIL_ADDRESS>Web: http://purl.org/net/aliman
Twitter: https://twitter.com/alimanfoo
Tel: +44 (0)1865 287721

As far as I can see, the importing of the graphviz image as an image array means that you can't overlay or 'draw' on top of those arrays. I'm going to try an kick out the graphviz stuff in a vector format so that I can build something a bit like this:

I think this figure could then be quite small, losing the fine network details, but we could have individual figures for the clusters of interest. What do you think?

Hi Chris,
You should be able to plot over the top of an image if you want to. I just
tried it, did imshow() then plot() to plot a line over the top, seemed to
work. What did you want to overlay on the image?
Re the networks being small, that may be OK for some, I guess the key thing
is to be able to see the topology, the colours of each node (including
singletons), and labels for any non-synonymous edges.
I think F1 will need it's own figure though. But do some playing around and
see what works.
On Wednesday, April 12, 2017, Chris Clarkson<EMAIL_ADDRESS>wrote:

As far as I can see, the importing of the graphviz image as an image array
means that you can't overlay or 'draw' on top of those arrays. I'm going to
try an kick out the graphviz stuff in a vector format so that I can build
something a bit like this:
[image: inkscape_example_vgsc_netclustfig]
https://cloud.githubusercontent.com/assets/13287471/24960804/d779e7d0-1f8e-11e7-86c9-6f0593736589.png
I think this figure could then be quite small, losing the fine network
details, but we could have individual figures for the clusters of interest.
What do you think?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/malariagen/agam-vgsc-report/issues/25#issuecomment-293582947,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAq8Qu9qpuiIwQOMZmaJZCCEvMcfFxrVks5rvNargaJpZM4MQYub
.

--
Alistair Miles
Head of Epidemiological Informatics
Centre for Genomics and Global Health http://cggh.org
The Wellcome Trust Centre for Human Genetics
Roosevelt Drive
Oxford
OX3 7BN
United Kingdom
Email<EMAIL_ADDRESS>Web: http://purl.org/net/aliman
Twitter: https://twitter.com/alimanfoo
Tel: +44 (0)1865 287721

Hi @alimanfoo,
This was what I was trying to do with just code, but I resorted to a bit of Inkscape in the end. Looks okay, but we would need to make it clear that the network nodes are not to scale across networks, they are just relative to that cluster. Just going to make the stand alone for cluster 16.

Should I push on with this format for 995S?

Hi @alimanfoo
Not sure how I feel about the overlapping nodes, but they can all be re-arranged nicely in Inkscape if we decide to use this type of figure...

Figure 2 (draft) - haplogroup distance and frequency

Looking good.
On Thursday, May 4, 2017, Chris Clarkson<EMAIL_ADDRESS>wrote:

Figure 2 (draft) - haplogroup distance and frequency
[image: figure2_dendfreq_draft]
https://cloud.githubusercontent.com/assets/13287471/25711347/9ba8e1c0-30e6-11e7-8f74-e645bdb33155.jpg
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/malariagen/agam-vgsc-report/issues/25#issuecomment-299220645,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAq8QgxRsZeUea786MIXWs8WqzcMU1d0ks5r2e6WgaJpZM4MQYub
.

--
Alistair Miles
Head of Epidemiological Informatics
Centre for Genomics and Global Health http://cggh.org
Big Data Institute Building
Old Road Campus
Roosevelt Drive
Oxford
OX3 7LF
United Kingdom
Email<EMAIL_ADDRESS>Web: http://a http://purl.org/net/alimanlimanfoo.github.io/
Twitter: https://twitter.com/alimanfoo

Superseded by #69 and #73.


mergesort is correct on duplicate values

This is @edre's fix in https://github.com/nikomatsakis/rayon/issues/105

@edre nice! do you think you could also change the mergesort to use a randomly generated initial vector (with a deterministic key), more like quicksort?

like this?

looks good =)


[SVLS-4912] Generate CPU utilization metrics

What does this PR do?

This PR introduces several new enhanced CPU utilization metrics for lambda. Each of the metrics are emitted once per invocation and represent CPU utilization by the lambda function.
The new metrics are:

aws.lambda.enhanced.cpu_total_utilization_pct - total CPU utilization of the function, as a normalized percent (e.g. 35%)
aws.lambda.enhanced.cpu_total_utilization - total CPU utilization of the function, as normalized cores (e.g. 1.2 cores)
aws.lambda.enhanced.num_cores - total number of cores available in the environment (e.g. 4 cores)
aws.lambda.enhanced.cpu_min_utilization - CPU utilization on the least utilized core, as a normalized percent (e.g. 10%)
aws.lambda.enhanced.cpu_max_utilization - CPU utilization on the most utilized core, as a normalized percent (e.g. 80%)

Motivation

Users want insight into their functions' CPU utilization
Product spec
Regarding the metric format: CPU Utilization Metric Representation

Additional Notes

Will likely have to resolve some conflicts with https://github.com/DataDog/datadog-agent/pull/26850 after code freeze / before merging
TODO: double check exact metric naming with product

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Build the lambda extension
Create a lambda function with the extension built in previous step
Turn on lambda insights / add the lambda insights extension to your function
Invoke the function
View the new enhanced metrics in metrics explorer or this dashboard
Compare with lambda insight metrics

/merge


Fix hyperlinks for Native Modules / UI Component

Motivation
Multiple links on the More Resources documents page point to an invalid path for both platforms (iOS and Android).
Release Notes
[DOCS][BUGFIX][more-resources.html] - Fix broken hyperlinks for Native UI Components and Native Modules.

The docs were moved a while ago to https://github.com/facebook/react-native-website. Can you open the PR there instead?


Adding more cards on reverse swipe jumps to last index

When working with dynamic list of cards (eg. fetching more cards from a datasource), adding cards while swiping in the forward direction works as expected. However, if loop is enabled, adding more cards while reverse swiping will cause the index to jump to the last index.
Suppose we have the following:
  final cards = ['card0', 'card1', 'card2'];

  // ...

  onIndexChanged: (index) async {
            if (index == cards.length - 1) {
              setState(() {
                cards.add('card[${cards.length}]');
              });
            }
          },

In the above code:

if you swipe in the forward direction card0 > card1 > card2
a new card3 is added
the view shows card2
when you swipe to card3
a new card4 is added
the view shows card3
...and so on.

However:

if you swipe card0 > card2 (in the reverse)
a new card3 will be added
the view shows card3

The expectation is that the view should show card2, because the swipe action takes the user to card2 first, then it adds card3 at the end of the list.
Once this bug is triggered, the forward direction swipe will also bug out. Meaning:

if you reverse swipe card0 > card2 to trigger the bug
the view will be on card3
now swipe in the forward direction card3 > card0 > card1 > card2 > card3
a new card4 will be added
the view will also show card4 immediately
meaning that card3 was skipped entirely

Imagine you're fetching 10 items at a time, that means that if this bug is triggered, 10 items will be skipped, and swiping back will see 10 new items before the user reaches the item they were on previously.
I've forked the repo to try and figure out the problem with an example, but I can't figure this out. Looking for someone with more experience with the package to help fix this bug. In the fork, you can run the example app and choose the last "Dynamic list" to option see the bug in action:
https://github.com/lhengl/card_swiper

Hi. Did you get a fix for this?

Hi. Did you get a fix for this?

Hi,
I just disable loop... it didn't make sense to have a loop while fetching new data anyway... I would only use loop if the of items are known in advance.


Update to jQuery 3.4.1

Codecov Report

Merging #22 into master will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #22   +/-   ##
=======================================
  Coverage   99.93%   99.93%
=======================================
  Files           2        2
  Lines        1462     1462
=======================================
  Hits         1461     1461
  Misses          1        1

Impacted Files
Coverage Δ

test/unit/EventCalendarTest.js
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e1194c6...7bc00a2. Read the comment docs.


[Usage] change GPU

Describe the issue
Issue: I run llava/eval/run_llava.py. By default, cuda:0 is used. How can I change it to cuda:1
The following is my modification, adding options：to(args.device), but an error was reported, how can I solve it?

Log:
  File "/datasets/xcm/LLaVA-main/llava/model/multimodal_encoder/clip_encoder.py", line 56, in forward
    image_forward_outs = self.vision_tower(images.to(device=self.device, dtype=self.dtype), output_hidden_states=True)
  File "/home/cg/anaconda3/envs/lmdrive/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/cg/anaconda3/envs/lmdrive/lib/python3.8/site-packages/accelerate/hooks.py", line 166, in new_forward
    output = module._old_forward(*args, **kwargs)
  File "/home/cg/anaconda3/envs/lmdrive/lib/python3.8/site-packages/transformers/models/clip/modeling_clip.py", line 917, in forward
    return self.vision_model(
  File "/home/cg/anaconda3/envs/lmdrive/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/cg/anaconda3/envs/lmdrive/lib/python3.8/site-packages/transformers/models/clip/modeling_clip.py", line 841, in forward
    hidden_states = self.embeddings(pixel_values)
  File "/home/cg/anaconda3/envs/lmdrive/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/cg/anaconda3/envs/lmdrive/lib/python3.8/site-packages/accelerate/hooks.py", line 166, in new_forward
    output = module._old_forward(*args, **kwargs)
  File "/home/cg/anaconda3/envs/lmdrive/lib/python3.8/site-packages/transformers/models/clip/modeling_clip.py", line 182, in forward
    patch_embeds = self.patch_embedding(pixel_values.to(dtype=target_dtype))  # shape = [*, width, grid, grid]
  File "/home/cg/anaconda3/envs/lmdrive/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/cg/anaconda3/envs/lmdrive/lib/python3.8/site-packages/accelerate/hooks.py", line 166, in new_forward
    output = module._old_forward(*args, **kwargs)
  File "/home/cg/anaconda3/envs/lmdrive/lib/python3.8/site-packages/torch/nn/modules/conv.py", line 463, in forward
    return self._conv_forward(input, self.weight, self.bias)
  File "/home/cg/anaconda3/envs/lmdrive/lib/python3.8/site-packages/torch/nn/modules/conv.py", line 459, in _conv_forward
    return F.conv2d(input, weight, bias, self.stride,
RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cuda:1! (when checking argument for argument weight in method wrapper_CUDA__cudnn_convolution)

Same. The args doesn't take effect.


Add Zig example

Do you think it would be worthwhile to maintain an example of Zig in the wazero repository? I extended the allocation example for Zig. I can open a PR if you have an interest. Please feel free to close the issue if it is not the case.
https://github.com/knqyf263/zig-wasm-example

I think so. I've also at times ad-hoc defined a zig project.
Please raise a PR and also don't forget to update the GitHub examples workflow and the Makefile accordingly!

Thanks for confirming. I opened #741.


Generated Loads - From Area Loads via Plane

Hi Diego,
is there an equivalent feature in your tool to the RFEM feature "Generated Loads - From Area Loads via Plane" or any other possibility to receive the same?
Kind regards

Hello,
I'm sorry but the automatic generation of loads is not included in the RF-COM API and therefore I can't included in the Toolbox...
Kind regards,
Diego


HTTPS true not working

Describe the bug
When running in https, chrome prevents access to the site
To Reproduce
Steps to reproduce the behavior:
serve svrx with https as true
Expected behavior
A way to configure svrx to serve with a cert that I can manually trust on my local machine
Desktop (please complete the following information):

OS: macosx
Node: v12.13.0
Browser: chrome

Additional context
Not sure if this is part of the tool, but was just playing around with it and noticed that if I choose https to true, I wont be able to load the site as expected.  Also, now since most browsers require a https for access to webcam/audio/etc, it would be great to point my keychain to a cert file that I can always trust (for local development)

We want to confirm that have you got the following information

And the page should still be successfully opened after trusting the certificate?


typescript error on npm start

TypeScript error in .../my-app/node_modules/jest-diff/build/diffLines.d.ts(8,13):
'=' expected.  TS1005

     6 |  */
     7 | import { Diff } from './cleanupSemantic';
  >  8 | import type { DiffOptions } from './types';

tracked at https://github.com/DefinitelyTyped/DefinitelyTyped/issues/44714

I think this is fixed in PR #19 as mentioned on Stack Overflow.
The TypeScript version needs to be 3.8.0 or higher in package.json. (Don't forget to npm install again, as I did!)

Fixed by https://github.com/microsoft/create-react-app-uifabric/pull/24

Fixed by https://github.com/microsoft/create-react-app-uifabric/pull/24


Access app.connector from a worker

I am not sure this is an appropriate place for questions like mine, as I haven't found a good place for procrastinate related questions - I apologize if it is not & please let me know, so I will move it to an appropriate place.
I want my workers to write results from completed jobs to a psql database - the same database initiated for the App connector. Is there a way for workers to access to app.connector from within a task function?
I have tried my best to search for sample code in the repository and the document and haven't found any. I also tried to pass app or an aiopg.Pool object to my task functions, but I am getting a "TypeError: Object of type Pool is not JSON serializable".
Related to this, what is the best way to pass an object like an aiopg.Pool that is to be shared across jobs and close it before all workers exit? Besides the connection pool, I hope to share aihttp.ClientSession across jobs and then close it before all workers exit. Is it possible?
At this point, I resort to pass the dsn to each job and have each worker create its own psql connection and aihttp.ClientSession. It works but is less than optimal. I like how the arq package (an async job queues with redis) accepts a startup & shutdown function to initiate & dispose resources that are to be shared across jobs (https://arq-docs.helpmanual.io/#startup-shutdown-coroutines). Is this something possible with procrastinate?
Thanks for creating this powerful package!

@lmwang9527 please take a look at the pass_context argument in the doc. When pass_context is True is the task declaration the task function receives a JobContext instance as its first argument.

Yep, I think JobContext can be a way to go, indeed.

At this point, I resort to pass the dsn to each job and have each worker create its own psql connection and aihttp.ClientSession. It works but is less than optimal. I like how the arq package (an async job queues with redis) accepts a startup & shutdown function to initiate & dispose resources that are to be shared across jobs (arq-docs.helpmanual.io/#startup-shutdown-coroutines). Is this something possible with procrastinate?

Yes we could imagine a way of adding arbitrary info to the JobContext.
At this point, it's also worth considering storing common information a module-level variable. I agree that it's better to receive parameters than relying on a global variable, but it's a possibility :)

Thank you both for your responses! it is exactly what I'm looking for. I did try using a module-level variable, but I cannot pass objects that cannot be serialized to my job function though. But the job_context would avoid having to do that.
Any recommendation what is the best way to close an object before all workers exit? For example, closing an aiohttp.ClientSession object I insert as an attribute to JobContext. I am getting an "Unclosed client session" warning if I don't call await client_session.close in my close before workers quit.

As a temporary method, you could subclass Worker, overload run where you'd call super() and add your own logic here. You'd definitely be tapping on the private API, but it should work.
So to summarize, an API could be setting on the App an async context manager, that would __enter__ when the worker is started, and return some context info that would be attached to the context for each task, and then the __exit__ would be called when all the workers have stopped.
Right ?

@ewjoachim how about the following?
    async with app.open_async() as app:
        await app.run_worker_async(context={"http_session": http_session})
        http_session.close()

yep, but that only works when launching the worker by hand, not by CLI. That being said, I'm ok with this too.
(we can already override the context in run_worker ? I had forgotten that !)
We may write the same as:
async with app.open_async() as app:
	async with AsyncSession as http_session:
    	await app.run_worker_async(context={"http_session": http_session})

or
async with app.open_async() as app, AsyncSession as http_session:
    await app.run_worker_async(context={"http_session": http_session})

(we can already override the context in run_worker ? I had forgotten that !)

No, that was more a proposal for a new argument to run_worker_async.

then I'm good with that.

Thanks again for your responses! That gives a direction to aim for. It'd be helpful if run_worker_async accepts a context argument - looking forward to its implementation.

@lmwang9527 for your information, @ewjoachim and I have worked on a PR that makes it possible to define additional context for the tasks. See #392.

Thank you so much for implementing the feature so quickly! I'm taking advantage of it and it works flawlessly.

@lmwang9527 thanks, this is great feedback!


Remove "socket" key from message data

Noticed that all messages being sent form Laravel broadcast events included an extra socket key in the message data. Depending on the structure of data I was sending from Laravel (and in turn receiving in the client) this extra attribute could cause the data structure to be different from what was expected and cause errors.

It appears that this socket value comes from the InteractsWithSockets trait which is included by default in all new events created by artisan so this should be handed as the default. Using the Pusher implementation as inspiration it appears they are handling this by pulling the socket key from the payload array before building up the message.
https://github.com/illuminate/broadcasting/blob/68939fc64fb3d553e066dbfcee7c0d835d7d24bc/Broadcasters/PusherBroadcaster.php#L151
This pull request implements the same logic that is used by the Pusher implementation. The Arr::pull function returns a default value of null which is the same as data_get function currently used.

Hi @kwhohasamullet, thanks for raising the PR. we will take a look 👍

Hi @sacOO7.
I do not have a huge amount of experience writing unit tests. Had a quick go at it below at am able to get the serialised string when it hits the post message but the de-serialisation does not seem straight forward (and I imagine is not really required in the php SDK). Strictly speaking it would be easier to test the buildAblyMessage method directly but it is a protected method and a quick google suggests the general view is not to test protected methods (and if you do requires using reflection to make it public for testing)
If there is an easier way to do this that I am not seeing and you can nudge me in the right direction I am happy to keep looking at this - else I imagine it would be a pretty simple test for someone with experience to write.

I do not have a huge amount of experience writing unit tests. Had a quick go at it below at am able to get the serialised string when it hits the post message but the de-serialisation does not seem straight forward (and I imagine is not really required in the php SDK). Strictly speaking it would be easier to test the buildAblyMessage method directly but it is a protected method and a quick google suggests the general view is not to test protected methods (and if you do requires using reflection to make it public for testing)

@qsdstefan would you be able to take a look at this? This does look simple ...

@sacOO7 @kwhohasamullet I've decided to expose the protected method by creating a new class that inherits AblyBroadcaster and provides a single public method which we use in the test. This seems a bit cleaner than using reflection.


cri-o: 1.20.1 -> 1.20.2

Motivation for this change
Update to the latest release.
cc @NixOS/podman
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 117468 at b6d5b09a run on aarch64-linux 1

2 packages built successfully:

cri-o
cri-o-unwrapped

2 suggestions:

warning: missing-phase-hooks
buildPhase should probably contain runHook preBuild.
Near pkgs/development/go-modules/generic/default.nix:145:5:
    |
145 |     buildPhase = args.buildPhase or ''
    |     ^

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/development/go-modules/generic/default.nix:220:5:
    |
220 |     installPhase = args.installPhase or ''
    |     ^

somehow the tests failed
@ofborg test cri-o
@ofborg build cri-o

@ofborg test cri-o

https://logs.nix.ci/?key=nixos/nixpkgs.117468&attempt_id=999bf7a8-4688-4c63-a830-f69aa118cb5b
trying https://github.com/cri-o/cri-o/archive/v1.20.2.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   120  100   120    0     0    394      0 --:--:-- --:--:-- --:--:--   393
100 12.4M    0 12.4M    0     0  4535k      0 --:--:--  0:00:02 --:--:-- 5760k
unpacking source archive /build/v1.20.2.tar.gz
hash mismatch in fixed-output derivation '/nix/store/2cli1aawsarn7x2la69nh46i026ijjiv-source':
  wanted: sha256:1hv3p0fqcvcd8wagszi646623vqmj9k05w86rzlfjc9asjp4bvja
  got:    sha256:1zyq0p7swbd53cg4k8za9p6apm60ywfvsyrjhxd34qa0pzg2ffs1

The hash is correct when I try to build this locally.

Update: hash is now failing locally.

Changed the hash, but for me this is failing locally.

Hash is still wrong.

Hash is wrong for current 1.20.1 version as well.
Are the upstream guys love to change tags?
Or is there a new non-determinism issue in fetchFromGitHub?

sha256-QTsn3r9AYTJahzJ7vR33wNSrzE3qo0keG6Utrs8F2P8= for x86_64-linux
sha256-Su5FrtQqMenozwbxAmaSFe8hjCEmfv0UR41thh24Y8M= for aarch64-linux
Weird.

I don't think we changed the tag upstream. It seems indeed an architecture related issue.

Do we need to fetch different sources for different arches?

Do we need to fetch different sources for different arches?

No. How are those hashes being calculated?

Do we need to fetch different sources for different arches?

No, usually not. How are those hashes being calculated?

after cloning the repo and doing some cleanup the directory is converted into a fixed output derivation IIRC.

Updated to v1.21.0


Custom Shaders: Revise how attributes are exposed to the shader

Right now, the Attributes struct includes only the raw values of the glTF attributes. Some attributes would be more useful if some preprocessing was done automatically, e.g.:
From https://github.com/CesiumGS/cesium/pull/9725#discussion_r697667455

glTF stores tangents as vec4 but they should appear to the user as vec3
vec3 and vec4 colors should both be seen as vec4 (with alpha = 1.0 when the vec3 is promoted to a vec4)
Derived attributes could be part of this list, like positionShapeSpace for voxels

Now, this could be done as a separate struct for derived attributes, or the existing attributes could be replaced with the nicer forms.
This impacts some of the code generation in CustomShaderStage. Also, #9761 may modify how the Attributes struct is generated
See also this comment which is somewhat related: https://github.com/CesiumGS/cesium/pull/9725#discussion_r697757302

This was handled in #9806


There is no accessible history.md or changelog.md file

It is hard to know if the newest versions have any breaking changes or what the changes are at all.

Going forward I will create a GitHub release and note any break changes. I would say that this library has fairly strictly followed semver, so can can rely on that.


Add Python GI bindings

Would be great to add Python GI bindings for those gnome apps that use Python

Hi @sergiusens we do have it and they are also used. You can give a look at wike or gfeeds.


Tweak no configuration cache is available message

Change it from:

Calculating task graph as ~no configuration cache~ is available for tasks: …

to:

Calculating task graph as no cached configuration is available for tasks: …

Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test ACC

I've triggered the following builds for you. Click here to see all build failures.

AllConfigCacheTestsPullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test and merge

I've triggered a build for you. Click here to see all failures if there's any.

Pre-tested commit build failed.


TaskId() requires a nodeId in the request

Please use the following questions as a guideline to help me answer
your issue/question without further inquiry. Thank you.
Which version of Elastic are you using?
[ ] elastic.v2 (for Elasticsearch 1.x)
[ ] elastic.v3 (for Elasticsearch 2.x)
[ ] elastic.v5 (for Elasticsearch 5.x)
[x ] elastic.v6 (for Elasticsearch 6.x)
Please describe the expected behavior
As of 6.3, the POST request looks like this: POST _tasks/node_id:task_id/_cancel. The request requires the nodeId followed by taskId. Can be seen in this documentation: https://www.elastic.co/guide/en/elasticsearch/reference/current/tasks.html#task-cancellation
Please describe the actual behavior
In your code the request looks like POST _tasks/task_id/_cancel, so it is not cancelling tasks properly.
Any steps to reproduce the behavior?
Here is the change I made in our local copy to get cancellation to work properly.
tasks_cancel.zip

As outlined in #864: The Elastic Java source was a bit convoluted around taskId when I implemented this. Sometimes they used a long, sometimes they used a string. Maybe they have cleaned up a bit. I have to check, but it might take a while. Sorry.

The REST API specifies a single parameter called task_id, defined as a node_id:task_number for the parent_task_id parameter in the Task Cancel API. The task_id is also sometimes returned individually and sometimes returned as a string.
However, in the Java source there's a special TaskId.java class that holds a nodeId and taskNumber internally.
So we need to decide whether we want users to specify both nodeID and taskNumber individually or just ask them to pass a task_id as a string (combining both as mentioned above)--as specified in the REST API. Overloading might be an option (so that we could use both), but I'm not sure if there a design problem in the API and we would make things even worse by allowing to program around that.


Update the connected user roles section of store on roles change

includes renaming of role change actions from 'permissions' to 'roles'
Fixes #2181

Merging. Tests failing in master


Ignore pydev metadata for eclipse users.

Changes Unknown when pulling 195a85362ae5b23943ceabe3ebb455c9d056a66f on ignore into ** on master**.

Changes Unknown when pulling 052a34a455c5de1319ade266dcfb0226ce8bc517 on ignore into ** on master**.

Coverage remained the same at 70.942% when pulling 052a34a455c5de1319ade266dcfb0226ce8bc517 on ignore into 1f8dd4539ce6914a96f488f644318c54bd4c96be on master.

Adding cla: yes label, as the PR author is a googler at the time of this PR was created.


Moved the warning on the grading type to below the form label

Part of GCI
Describe the changes you have made in this pr - I have moved the grading warning to be below the form label
Screenshots of the changes (If any) -

Coverage remained the same at 82.507% when pulling b840113c79501766af4bd5221d0db7314e9bdb52 on BBroadbent:patch-4 into d64e1a0c3db31871160b4af3f98946a7ae934242 on CircuitVerse:master.


Trigger password-reset email automatically

 After creating a scratch org, I want a pwd-reset email triggered automatically

@davisagli @jgerigmeyer for some reason all Plan installs are failing on me (even without new changes to the code):
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/rq/worker.py", line 936, in perform_job
    rv = job.perform()
  File "/app/metadeploy/rq_worker.py", line 12, in wrapper
    return orig(self)
  File "/usr/local/lib/python3.8/site-packages/rq/job.py", line 684, in perform
    self._result = self._execute()
  File "/usr/local/lib/python3.8/site-packages/rq/job.py", line 690, in _execute
    return self.func(*self.args, **self.kwargs)
  File "/app/metadeploy/api/jobs.py", line 147, in run_flows
    repo_root = stack.enter_context(
  File "/usr/local/lib/python3.8/contextlib.py", line 425, in enter_context
    result = _cm_type.__enter__(cm)
  File "/usr/local/lib/python3.8/contextlib.py", line 113, in __enter__
    return next(self.gen)
  File "/app/metadeploy/api/github.py", line 23, in local_github_checkout
    zip_file = download_extract_github(repo, repo_owner, repo_name, ref=commit_ish)
  File "/usr/local/lib/python3.8/site-packages/cumulusci/utils/__init__.py", line 179, in download_extract_github
    return download_extract_github_from_repo(
  File "/usr/local/lib/python3.8/site-packages/cumulusci/utils/__init__.py", line 189, in download_extract_github_from_repo
    zip_file = zipfile.ZipFile(zip_content)
  File "/usr/local/lib/python3.8/zipfile.py", line 1269, in __init__
    self._RealGetContents()
  File "/usr/local/lib/python3.8/zipfile.py", line 1336, in _RealGetContents
    raise BadZipFile("File is not a zip file")
zipfile.BadZipFile: File is not a zip file

Besides that our ScratchOrg ids being 18-char actually requires them to be the same length in other places. For example Jobs (our Job model, not RQ jobs) have their own org_id field that it uses to get ScratchOrgs as well. I can feel I'm a couple hours away of solving it but I'm also 6 hours in and don't know if that's too much (based on David's comment of not spending too long on this).

@jerivas Hm, strange. I get that error on other plans, but EDA plans work okay for me... it seems like we're so close, I'd love to wrap this up. I'll take a look over the weekend and see if I can quickly get a sense of the scope.

for some reason EDA Plan installs are failing on me (even without new changes to the code):

@jerivas Shoot, you're right -- they're now failing for me too, with the same error. I wonder if there might have been changes to the repo itself (https://github.com/SalesforceFoundation/EDA/) that caused it? I'm guessing @davisagli can weigh in on Monday.

Besides that our ScratchOrg ids being 18-char actually requires them to be the same length in other places. For example Jobs (our Job model, not RQ jobs) have their own org_id field that it uses to get ScratchOrgs as well.

Yeah, we might want to just be sure we always normalize to 18 char every time we store or access the org_id? Seems like that might be the best future-proof option, and even if pwd reset itself isn't top priority I think we want to get this resolved (pwd reset is actually working great -- the blocker is having a consistent way to compare orgs).

@jerivas @jgerigmeyer I think the error is happening because:

Running a job starts by retrieving the git archive for the Version's "commit_ish" (commit sha or branch)
The populate_db script created a Version of EDA without specifying a commit_ish, so it defaulted to master.
The default branch of EDA was recently renamed from master to main.

Suggested fix:

Change populate_db to set commit_ish to e785195d07a3ac9e395f27829866005dbdcd5bd0 (the same commit referenced in particular installation steps) when creating the EDA Version.
Manually edit the EDA product record in your db to change master to e785195d07a3ac9e395f27829866005dbdcd5bd0

Back to the org_id issue, I think the proper solution would be to create a new SalesforceID field that enforces 18 chars. Off the top of my head ScratchOrg and Job would be the ones using it. What do you think? @jgerigmeyer @davisagli

@jerivas Can we just convert to 18 characters before storing in the ScratchOrg's org_id field, rather than adding a new field type? I am a little worried that using a field which always normalizes to 18 characters may create as many problems as it solves, since there may be other existing code that is comparing to the 15 character version. So I'd like to make sure we're fixing specific cases where we know what form it's in, rather than making a change which would have broad effects.

@davisagli I made sure we're storing a 18-char id on ScratchOrg.org_id and that Jobs and PreflightResults are reading from that instead of the config for consistency. Also added a migration to homogenize past orgs, jobs, and preflights, though that might not be strictly necessary as the install jobs are never run again, right?


Unresponsive after couple of days

Hi,
first of all, great project and very well put together!
I made the hardware according to the documentation, flashed the software and added the blinds without any issues.
Used everything last summer with no errors at all.
A week or two ago I noticed that the entity became unavailable in home assistant.
so I went to the web interface and saw that the following was happening?

After rebooting the hardware it immediately works again for a couple of days. Then this behavior starts again.
do you have any clue what is going on here?
No chantes to the hardware
Latest version of your software
Latest version of the home assistant integration
Latest version of home assistant
thanks in advance!

The message you are receiving is not actually coming from the ESP32.  The client is assuming that since it cannot open a socket to the device that it is busy.
First make sure you do not have more than 5 clients connected to our little ESP32.  That message is displayed when a socket connection cannot be established from the client.  If the device is working properly the most common reason is that it is too busy.  In this case I am more inclined to think that there may be something else going on.
If you have a failure, attempt to access the device using its IP address and not the host name.  Sadly, mDNS name resolution is dependent on your local router keeping the information in sync.  If you have not set up a DHCP reservation in your router for the host then you should do this.  This will make sure the device always receives the same IP address.
If you are running the most recent release of ESPSomfy RTS v2.3.0, then in Home Assistant you will see a Wifi strength entity.  If the WiFi is degrading in signal strength we should see it there.  The closer this value is to 0dBm the better.  An acceptable signal is -80dBm but it will become unreliable as it approaches -90dBm.   For instance, here is a device that has been running since the release of 2.3.0.  It is pretty solid but if it were dipping into the -90 area it would likely go offline.

The funny thing is that the WiFi connection is solid as a rock. The esp has a -47dBm connection. When this happens, the esp is still connected and I do not see any dropouts. Only home assistant is connected to the esp via your integration.
It's strange it happens after a day or three and it comes back to life a day or so when I don't powercycle the esp.

Most often when a device drops off the network the first thing to check is whether you are accessing it over mDNS or using the IP address.  mDNS can be fickle.  Do you have an IP reservation in your router?

There is no dropoff from the network. It's online all the time. I do have a ip reservation set for the device. It looks like the esp freezes and a powercycle is the only option to bring it back.
Is there a way to see some (more) logging to understand what's going on?

Yes you can connect the device to the usb with a data cable.   In web.esphome.io connect the device click on the logs and you will see the information currently being processed from the device.

Well I have exactly the same problem.... Is there any way to add a possibility to have a reboot command on this integration ? So I can automate a daily reboot?

A reboot function and entity exists in the current integration v2.3.1.

You are right...  I had version 2.3.0 !
Well also I passed using ethernet, maybe it will have a better responsiveness ?

Interesting I have 1 WT32-ETH01 that occasionally drops off but I have another that never does.  When I get a chance I am going to swap out the adapters and take a look at what it is doing.  I can ping it but I cannot load anything over the webserver, udp, or sockets.

Yesterday I tried to do the update to 2.3.1 and it hung on the rotating circle for about 20 minutes. I thought it was ready because the other updates took a minute or so. So I rebooted the esp and now it constantly hangs on the screens as shown earlier also after a fresh reboot. I am not able to update the application.
Today or tomorrow I will do the onboarding from scratch to get it back alive.
It looks like it created two backup files when updating the first firmware. Which of the two I use to restore all settings?

In a browser type http://<ip address>/downloadFirmware?ver=2.3.1 in the address bar and press enter.  It will take a couple of minutes but it will pull the firmware from github.

The reboot button is nice, but does not work when the device becomes unavailable in HA. The reboot button is also unavailable.
Is there a browser request i can send to the device just like the "downloadfirmware" link you have sent earlier?

And now a reboot doesn't help either. It hangs on the same screens as in first post... It looks like it's doing it's own thing :(

Well I though the problem was because of Wifi, but since I move to Ethernet, the problem is still there... :(
I will schedule a reboot twice a day on HA... but there is still hidding the issue :(
My board in Lyligo Internet POE...

So I got a bit of time to troubleshoot as it still was unavailable in home assistant, constant spinning icon in the web interface but rock solid on the WiFi. I took the board out, wiped it completely and flashed the onboarding.bin file again. After browsing via web interface and connecting to my wifi, it happens again. This WiFi network is a hidden IoT WiFi network to which all of my esp devices are connected. I reflashed the board again but now I connected the board to my normal WiFi which is not hidden and it starts to work again without issues. This all starten to happen without making changes to my WiFi setup. All my other ESP's are rock solid. I also tried this with a different board but this gave the same behaviour. Is there some sort of issue with the WiFi driver you use?

The libraries used are dependent on the SSID to resolve mesh networks.

I assume you have figured this out but the issue is that hidden networks do not have an overriding identifier like an SSID and will not work with mesh networks.   I am going to close this for now.   Also, if you were having stability issues please install the release fw v2.4.2.   This has some fixed to improve stability.


Enable few disabled tests under "brokenOnJBallerina"

Purpose
$subject.
Related to #16177
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

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

[ ] Increased Test Coverage
[ ] Added necessary documentation

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

Codecov Report

Merging #18689 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #18689   +/-   ##
=======================================
  Coverage   15.04%   15.04%
=======================================
  Files          48       48
  Lines        1263     1263
  Branches      199      199
=======================================
  Hits          190      190
  Misses       1059     1059
  Partials       14       14

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 099bce1...b10d1d4. Read the comment docs.


Publish Admin GUI App to Github Pages

Pages published and CI pipeline created to automatically publish on PR / Push to main


[UI] Calculate the total cost of launched interceptors

Every time the SimManager creates a new interceptor, we tally up the unit costs of each launched interceptor.

@daniellovell I tried (and failed) to create the textbox in the bottom overlay panel, feel free to do it and add to this PR

I tried (and failed) to create the textbox in the bottom overlay panel, feel free to do it and add to this PR

Done, let me know if it looks ok


Add myself to gem authors

@thomasfl, are you OK with it?
Contributions: https://github.com/filewatcher/filewatcher/graphs/contributors

It was long overdue that you were added to the list of authors. :-)

OK, thank you. ❤


use github action to run ormolu

And also remove the no longer used check-cabal-project pipeline from buildkite. This will let us get rid of buildkite altogether.

@JaredCorduan I was thinking about this and maybe we should define it as a separate job in haskell.yaml instead of a whole separate workflow. This has a benefit of the same github settings as our normal CI. So placing it in haskell.yaml like so:
jobs:
  build:
  .... all the stuff from Haskell CI
  ormolu:
    runs-on: ubuntu-latest

    defaults:
      run:
        shell: bash

    strategy:
      fail-fast: false
    steps:
    - uses: actions/checkout@v1

    - name: Install ormolu
      run: |
        mkdir -p "$HOME/.local/bin"
        curl -sL https://github.com/tweag/ormolu/releases/download/<IP_ADDRESS>/ormolu-Linux -o "$HOME/.local/bin/ormolu"
        chmod a+x "$HOME/.local/bin/ormolu"
        echo "$HOME/.local/bin" >> $GITHUB_PATH
    - name: Run ormolu
      run: ./scripts/ormolise.sh

That seems pretty reasonable @lehins . I guess the question is, do we want to make that haskell CI action required? I think we do want the ormolu check required (though I'm having trouble figuring out how atm).

do we want to make that haskell CI action required?

Not sure I understand. Isn't it already required? Eg. https://github.com/input-output-hk/cardano-ledger-specs/pull/2499/checks?check_run_id=3695098564

oops, I did not mean to edit your comment @lehins , I didn't even know that was possible :scream:

I did not mean to edit your comment

lol. It's one to communicate :) I see what you mean now. Looks like it should be possible to protect it in settings, see SO answer And yeah, we want to protect on all of the CI checks

lol. It's one way to communicate :) I see what you mean now. Looks like it should be possible to protect it in settings, see SO answer And yeah, we want to protect on all of the CI checks

yea, I saw that, but the ormolu check isn't showing up there, only hydra.


FingerDecompser cannot finish the last one

If the trace has multiple trails, the last one will be chop off by FingerDecomposer. something is wrong with the ending msg.

9cda696df56ceb6af4447276191155744b32b967


Apply change applied on the original gem

Update project SHA lib for a new one used in the original project
Basically change digest-sha3 for keccak.
Nothing more was changed in the original project just the SHA engine

Looks good, we can keep maintaining this fork until the time to move completely to the eth gem arrives.


Attached debugging using Delve is not showing code at breakpoints

I am attempting to debug a go application that is compiled and bundled prior to running the debugger.   This is done using the command line:
guark build -rm && dlv --listen=:2345 --headless=true --api-version=2 --accept-multiclient exec dist\windows\Demo.exe
Delve starts up just fine, VSCode connects to Delve and issues the proper API commands, but it doesn't display the code when the breakpoint is hit.   I can see from the Delve logs, that it is transmitting the breakpoint to VSCode, and VSCode is displaying the stacktrace, but it does not display the source code.
Attached is a Delve log and a screenshot of VSCode during a breakpoint
From the log you can see:

At [14:01:24.729 UTC] the brekpoints were verified
At [14:01:50.444 UTC] the breakpoint was triggered
At [14:01:50.471 UTC] Delve replied with location of breakpoint
At [14:01:50.479 UTC] VSCode requested a stack trace

From the screenshot, you can see that the callstack is populated and showing where it is paused, however, the editor doesn't reflect the breakpoint in the code.   It remains where it was initiated at (in this case launch.json).

Please file against the GO extension


build error on pc

Describe the bug
I am on the master branch, without any modifications.
follow https://github.com/commaai/openpilot/blob/master/tools/README.md
run scons -u -j$(nproc),the error is as follows:
In file included from panda/board/stm32fx/inc/core_cm4.h:34:0,
from panda/board/stm32fx/inc/stm32f413xx.h:197,
from panda/board/stm32fx/inc/stm32f4xx.h:189,
from panda/board/stm32fx/stm32fx_config.h:2,
from panda/board/config.h:34,
from panda/board/bootstub.c:7:
/usr/lib/gcc/arm-none-eabi/6.3.1/include/stdint.h:9:26: fatal error: stdint.h: No such file or directory
include_next <stdint.h>
                      ^

compilation terminated.
scons: *** [panda/board/bootstub-panda.o] Error 1
Compiling /home/ada/newdisk/ctt/openpilot/opendbc/can/parser_pyx.pyx because it changed.
[1/1] Cythonizing /home/ada/newdisk/ctt/openpilot/opendbc/can/parser_pyx.pyx
Compiling /home/ada/newdisk/ctt/openpilot/opendbc/can/packer_pyx.pyx because it changed.
[1/1] Cythonizing /home/ada/newdisk/ctt/openpilot/opendbc/can/packer_pyx.pyx
scons: building terminated because of errors.
OS Version
ubuntu18.04
openpilot version or commit
master
Additional info
No response

Ubuntu 18.04 is not supported. Try ubuntu 20.04 or 22.04

hello，can I compile and run on ubuntu18.04 with ubuntu20.04 environment in docker?Or do I have to install an ubuntu20.04 environment on the computer?

@chenti2x
you don't need correctly set up the env on pc if you only want to run the openpiot in docker


Should GlobalTracer be a no-op tracer by default?

As we've decided to go with having a Global tracer implementation for ease-of-use (see prior discussion at opentracing/opentracing-go#10), a thought has crossed my mind:

I'd really like to be able to submit PRs to open-source projects that may not know what tracing is, and in such cases they might be more inclined to like using globals rather than modifying their existing API.
They wouldn't want existing users of their API to have to do OpenTracing setup with a Noop tracer implementation, just to get the same behavior as previously.
This leaves me with an idea: should GlobalTracer default to being the no-op tracer implementation? Then libraries could make calls to opentracing safely and it'd be no-op by default unless the user of such a library opted-in to installing their tracing implementation as the global.

Thoughts?

Having just written the JS lib to work as a no-op by default, I agree! :)
I think I can say with some confidence of confidence that at least in Node -- which is the only case I've already thought this through on -- @slimsag's 2nd point is very important. If I want to use a random package off NPM in my own code, I should not need to care about where that random package uses OpenTracing or not and require me to set up a no-op implementation.  Otherwise, it adds an additional setup step to use of the package (the NPM world, as I understand it, is very much a "install it and expect it to just work" kind of world) and thus package writers will be disincentivized from including OpenTracing in their packages.
However, I will say there's a chance this might be a decision left to be decided on a per-platform basis.  Linkage, configuration, time-of-evaluation concerns can vary a lot from between platforms -- and I would not be surprised at all if there's a case I'm not considering where not being no-op default is more correct.

Default/global tracer already defaults to no-op impl in all langs except Java (no impl there yet). So yes, I think we should continue with that pattern.

looking at existing tracers that have a global, ex finagle. it is no-op by
default
https://github.com/twitter/finagle/blob/develop/finagle-core/src/main/scala/com/twitter/finagle/tracing/Tracer.scala#L135
ex. if no implementation could be loaded, it does nothing.

I'd really like to be able to submit PRs to open-source projects that may not know what tracing is, and in such cases they might be more inclined to like using globals rather than modifying their existing API.

I like no-op tracers as well.  At AppNeta, we use them despite the fact that our instrumentation is injected, rather than built into projects.  There's a few advantages, some of which are mentioned above.
There's also one consideration we might take into account in designing the tracing API, which comes out of the comment I referenced above: a goal of a no-op tracer would be to incur near-zero overhead.  This is fairly easy at the API method level, e.g. have a method like setTag simply return with no action.  However, if the work required to retrieve/format data passed to setTag is expensive, then you have a case where work is being done outside the API that you'd like to avoid in a non-sampled or no-tracer case.  Examples of potentially expensive operations are getting a backtrace or executing an EXPLAIN on a query, etc.
For this reason, I suggest that we might consider adding an API to guard expensive work in the non-tracing or no-tracer case.  Options that come to mind would be either a boolean method that determines whether tracing is occurring, or a method that invokes a callback only if valid context is present.  I've opened a separate discussion for this in #31

@slimsag ok with you if we close this with a resounding "yes" as the answer?

Tags are set-only, but we expose an api for getAttribute() no-op returns
null?

@adriancole that sounds right if we aren't supporting null attributes.

well null is usually "I don't have this attribute". The problem we have in
exposing getAttribute is that we might end up needing to document the no-op
case.

I propose that the default tracer for each platform be a true noop tracer whose primary goal is to avoid performance overhead. This also means that Trace Attribute support will not work.
An alternative would be to have the default tracer handle Trace Attributes and injection/extraction (nee "propagation"), but IMO that's too much work for a default anything to do.
If there are no objections in the next 48 hours, I'll assume we have consensus and close this issue.

+1. If people want just the attributes, there can be a half-op tracer.

time to close this. @bensigelman should we add it to the semantic doc?  To me the most important part is that the default tracer is safe to use in prod, its methods will not throw exceptions or anything, and generally will behave as expected, only without any side-effects.

Yeah, for now I'd like to say that the noop tracer will not crash but has no side effects (including trace attributes). I was previously thinking that the noop tracer should support just the trace attributes, but there's really no point in that without injectors/extractors, and if we deal with those as well it's hardly a noop tracer :)
It would be cool to support a trace-attributes-only Tracer someday, though.

(and yes, good to put this in the spec: let's close this issue once that's done if there are no objections)


reset-array-pointer of item-Array, to make sure that key() is actuall…

Description
reset-array-pointer of item-Array, to make sure that key() is actually accessing the first element, because it can be that with PHP7 the pointer is not always on the beginning of an array

Questions
Answers

Why?
to make sure that key() is actually accessing the first element, because it can be that with PHP7 the pointer is not always on the beginning of an array

BC breaks?
no

Tests pass?
yes

Related tickets?
--

How to test?
backend/ImportExport/exportOrders -> chose XML -> the generated XML should not contain the numeric-index of the order-nodes

Requirements met?
yes

Thanks for your work @viosys. Since the xml converter is deprecated and will be removed in the future, we won't merge any non security relevant changes to it anymore.


CustomerSearchResult Data field is incorrect

public struct CustomerSearchResult: Codable {
    /// A string describing the object type returned.
    public var object: String
    /// A list of charges, paginated by any request parameters.
    public var data: [Charge]?
    /// Whether or not there are more elements available after this set.
    public var hasMore: Bool?
    /// The URL for accessing this list.
    public var url: String?
    /// The URL for accessing the next page in search results.
    public var nextPage: String?
    /// The total count of entries in the search result, not just the current page.
    public var totalCount: Int?

**See CustomerSearchResult object above. Data should be of type [Customer], not [Charge]**

Not all heros wear capes.
Thanks Andrew


Dropdown | valueContainer props aren't applied to the Button correctly

While we could pass the valueContainerProps to the containerProps of Button - this would overwrite the isOpen data. We need to merge the props.

:tada: This issue has been resolved in version 1.0.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


script Mac support

Hi it would be cool if someone made it also compatible with mac. Currently i have windows machine where i show propresenter and mac which livestreams. But script doesnt work on mac :(

There is this too: https://github.com/sdboer78/propresenter-stagedisplay


Update touchablehighlight.md

Thank you for the PR! Contributors like you keep React Native awesome!
Please see the Contribution Guide for guidelines:
https://github.com/facebook/react-native-website/blob/master/CONTRIBUTING.md
If your PR references an existing issue, please add the issue number below:
#

Closing since the feedback was not actioend on, and I think it's clear that this component requires onPress to work as expected.


Different look table APP Platform Setting on GL3 and GL4

Build: PRIVATE-BUILD-ON-ESTESSBESCIJS1 2017/05/16
Windows Server 2016
jdk1.8u121
SAML mode
Glassfish 4
[How to reproduce]
Go to http://<server>:<port>/oscm-app/
[Expected result]
The APP Platform Setting looks as on Glassfish 3

[Observed result]
The APP Platform on Glassfish 4 looks different:

[Details]
The structure of the table on GL3 is:
<table id="appSettings:j_idt65" class="rf-dt table_row_linetitle_class"
<colgroup span="2">

but on GL 4 is unnecessary tag

<colgroup span="2">

#677

Hi @kwodzynski,
you are right, this default branded admin UI style isn't that fancy. However, I don't see much difference in your both screenshots, except for that the GF4' one has this footer space and wider Setting key column. Since we got an JSF upgrade with shift to GF4 and some UI tweaking here and there, it's not necessarily unexpected or defect to result in a different appearance.
Both styles are improvable. E.g. in both cases the input field widths are same short. Since the white label styling is not responsive to resizing, the solution here could be to create and deploy a nice overlay branding, which on-demand dynamically resizes the controls in a user-friendly fashion. Could be a cool feature.
For here, let's check if it's quick and save to drop the mentioned table footer. If yes we go for it, if not we'll leave it.

Hi @GoebelL,
The different width of Setting key column is caused by different size of browser in the screenshot moment. When I open both (gl3 and gl4) browsers on the same monitor, the width is the same as well.

Actually, it seems that that footer should be populated with {platformConfigurationCtrl.model.items[key]} (see settingsEditor.xhtml)
I propose a fix that only renders this footer when platformConfigurationCtrl.model.items is not empty by adding in settingsEditor.xhtml in newControllerId the attribute:
rendered="#{not empty platformConfigurationCtrl.model.items}"
I would also add the entire footer to the GF3 master so that it renders there too when the list is not empty.
Thoughts? Any objections?

We will merge gf3 master to gf4 master soon. So if the gf4 is OK, then you can close this.

Tested on Windows Server 2016, jdk1.8u121, build BES_MASTER_UNIT-1778 2017/06/29 (Official 17.2.0) in Saml mode. The table looks good.


intel: videoDrivers: intel -> modesetting

This seems to be the current best practice.
https://www.phoronix.com/scan.php?page=article&item=intel-modesetting-2017&num=1
https://jlk.fjfi.cvut.cz/arch/manpages/man/modesetting.4

Anything blocking this?


Infinitely increasing amount of data in session during a loop in conversation

Node.js v20.11.0
grammyjs v1.21.1
@grammyjs/conversations v1.2.0
typescript v5.4.2
I used the example code with loops from the plugin documentation, and found that with each loop the data in the session increases in volume.  It is not a good idea to let it run indefinitely.

It is documented that all received update objects are stored in the session. This implies that the more update objects you handle, the more data needs to be stored.


LWM2M: fix handle_reqest() errors

Refactor the error handling section (split from #1049)
Respond 4.5 (Method Not Allowed) when option PATH is not given.
'/' is only allowed when performing bootstrap-delete
Respond 4.4 (Not found) when object doesn't exist
Respond 4.4 (Not found) when URI contains character other than digits
Respond 5.0 (Internal server error) when OP is not supported
Respond 4.5 (Method Not Allowed) when request method is not GET for URI .well-know/core

Update according to @tbursztyka 's review.
Fold 3, 5 into 1 and add more description to the commits.

@jukkar , @mike-scott please help to take a look at the patches. thanks

Will do later.  Sorry for delay.  I had too many patches in too many branches today.

@rbtchc Won't this patch conflict with changes in your other PR: https://github.com/zephyrproject-rtos/zephyr/pull/1049 ?

@mike-scott I think so. I suggest that we can merge #1255 first and then this one and then #1049.

@tbursztyka could you please help to take a look at commits and share your opinions? thanks

@tbursztyka could you please help to take a look at commits and share your opinions? thanks

Tomasz is on vacation still next week. I try to get @nashif to merge this one as @tbursztyka comments were addressed in later versions of the PR.


How does the mustRegexMatch returns an error?

I am trying to using the helm function  mustRegexMatch to validate a timeout values from value.yaml.
it seems I can't get any errors.
but from the https://helm.sh/docs/chart_template_guide/function_list/#regexmatch-mustregexmatch  documentation  "regexMatch panics if there is a problem and mustRegexMatch returns an error to the template engine if there is a problem."
my yaml file as below
{{- $connectionIdleTimeout := .Values.timeouts }}
{{- $asdas := $connectionIdleTimeout | mustRegexMatch ^(([1-9])|(1[0-9])|(2[0-4]))h$}}
timeouts:
connection-idle-timeout: {{ $connectionIdleTimeout | quote }}
asdcs: {{ $asdas | quote  }}
command 1 : $  helm template . --set timeouts="24h" --show-only  templates/mustRegexMatch.yaml
Output:
timeouts:
connection-idle-timeout: "24h"
asdcs: "true"
command 2: $  helm template . --set timeouts="26h" --show-only  templates/mustRegexMatch.yaml
Output:
timeouts:
connection-idle-timeout: "26h"
asdcs: "false"
I not get any errors from mustRegexMatch.
is there anyone known this?
is there anyone have a good solutions for validate parameters values on value.yaml
Output of helm version:
version.BuildInfo{Version:"v3.4.1", GitTreeState:"clean", GoVersion:"go1.14.11"}
Output of kubectl version:
N/A
Cloud Provider/Platform (AKS, GKE, Minikube etc.):
N/A

1

The wording on that is pretty confusing. This comes from the sprig library: https://github.com/Masterminds/sprig/blob/3ac42c7bc5e4be6aa534e036fb19dde4a996da2e/regex.go#L12
This is just a wrapper for https://pkg.go.dev/regexp#MatchString the only difference being that regexMatch throws away the error and mustRegexMatch doesn't.
If you write and invalid regex mustRegexMatch with throw an error:
Error: template: testchart/templates/deployment.yaml:2:23: executing "testchart/templates/deployment.yaml" at <mustRegexMatch "^(qwert$">: error calling mustRegexMatch: error parsing regexp: missing closing ): `^(qwert$`

where regexMatch will not error and will just fail to match anything, always returning false.


listener: split listenerfilterfactorycontext and networkfilterfactorycontext

Description:
Extract from #9773
The ultimate goal is to update network filter chains without destructing all the network filter chains in previous listener config.
Splitting lifetime of listener filter and network filter, so that we can reuse the references to the existing listener filters in old listener (will do in follow up PR or see #9773)
The next PR is to arm the connection handler with in place filter chain update.
Also eliminate the risk of triggering warmed up listener when destroying ListenerImpl.
Risk Level: LOW since the life time is equivalent with and without this PR
Testing: Fixed existing tests
Docs Changes:
Release Notes:
[Optional Fixes #Issue]
[Optional Deprecated:]

/assign @mattklein123 since you have the contexts on #9773
I admit this is not the smallest change I can extract but the filter chain manager and the drain manager change is necessary in the nest steps

/azp run envoy-macos

bazel tsan failing because the integration test is accessing listener manager from different thread but listener manager is not thread safe. If the failure rate is low it should be fine

test should be fixed in https://github.com/envoyproxy/envoy/pull/10523

I removed most of the TODOs.
Two major TODOs are left:

Race condition due to deferred destruction. As we agreed, it's not a regression and I will fix afterwards.
Shim the interface FactoryContext for network filter and listener filter. It's an arguable what scope a network filter should refer a listener, comparing with a listener filter. Prior to this PR, there is no need to discuss because both network filter and listener filters refers the same factory context instance. For compatibility I grants the same ability to network filter as it was prior to this PR. I will take several iterations to gradually tailor the dependencies of network filter factory context.

early introduce another fix https://github.com/envoyproxy/envoy/pull/10579 to fix the surfacing flaky h1 fuzz persistent tests

merging master and resolving conflict with my other PR
Seems I hit another static initialized string issue :(

This is the closest situation to all check passing during these days. @mattklein123 I addressed all your comments. Could you take another (hopefully last) look? Thanks!


How to import our own csi300.txt?

❓ Questions and Help
If download the demo 1d data using provided script, it would import a csi300.txt in instruments folder.

python scripts/get_data.py qlib_data --target_dir ~/.qlib/qlib_data/cn_data --region cn

When importing our own data say in csv format, how do we import this csi300.txt file as well? This file is not in usual csv format, so cannot be provided through CSV import I assume.

Import csi100 or csi300, reference: https://github.com/microsoft/qlib/blob/main/scripts/data_collector/cn_index/README.md
python collector.py --index_name CSI300 --qlib_dir <user qlib data dir> --method parse_instruments

Thanks, this works. What is the difference between parse_instruments and save_new_companies?

parse_instruments： generate component stock changes with "index" history
save_new_companies: download only the "index" list of latest included stocks

Thanks for the explanation!

@arisliang
Would you mind sending a PR to improve the docs about this Question?
Other users who have similar confusion will be grateful to you :)
Thanks


Labels missing for several training images

The ribseg labels appear to be missing for many of the ribfrac images.
This is causing data_prepare.py to fail with errors such as:
$ python data_prepare.py
Traceback (most recent call last):
  File "/home/david/miniconda3/envs/RibSeg/lib/python3.7/site-packages/nibabel/loadsave.py", line 42, in load
    stat_result = os.stat(filename)
FileNotFoundError: [Errno 2] No such file or directory: './data/RibSeg/nii/RibFrac43-rib-seg.nii.gz'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "data_prepare.py", line 102, in <module>
    main()
  File "data_prepare.py", line 20, in main
    label = nib.load('./data/RibSeg/nii/'+data[:-12]+'rib-seg.nii.gz')
  File "/home/david/miniconda3/envs/RibSeg/lib/python3.7/site-packages/nibabel/loadsave.py", line 44, in load
    raise FileNotFoundError("No such file or no access: '%s'" % filename)
FileNotFoundError: No such file or no access: './data/RibSeg/nii/RibFrac43-rib-seg.nii.gz'

I updated data_prepare.py locally to handle the missing ribseg labels for certain ribfrac images. Happy to open a PR if you grant me permission.

Thanks! Yes, I noticed data_prepare.py used the complete RibFrac dataset, which contains 660 CT scans, while RibSeg contains 480 cases. I just updated the code, let me know if there's still an issue. Appreciate your help!


Fix unquoted shell variables according to ShellCheck

This is just applying all the fixes suggested by https://www.shellcheck.net/. If you paste the old commands there (and optionally add a #!/usr/bin/env bash line at the beginning to get rid of the Tips depend on target shell and yours is unknown. message) and click one of the "apply all SC2086" links, you'll end up with the same changes.
See https://github.com/koalaman/shellcheck/wiki/SC2086 for more details.

I think the most important fix is echo ${JOBINFO} -> echo "${JOBINFO}" one. The JOBINFO variable is the full JSON response from the GitHub REST API endpoint. The existing unquoted version echo ${JOBINFO} is subject to the same behavior as described at https://github.com/koalaman/shellcheck/wiki/SC2086#rationale:

The first code looks like "print the first argument". It's actually "Split the first argument by IFS (spaces, tabs and line feeds). Expand each of them as if it was a glob. Join all the resulting strings and filenames with spaces. Print the result."

For starters, this means that every group of consecutive whitespace (because by default IFS=$' \t\n', as you can check with declare -p IFS) will be replaced by a single space in the entire JSON contents before it is processed by jq. Since JSON mostly ignores whitespace, this replacement will go unnoticed most of the time, but note that string literals in the JSON are also affected by this. So for example, it would be a problem if someone had a job name with a double space in it, for example:
jobs:
  build:
    name: Double  space
    steps:
      - uses: Tiryoh/gha-jobid-action@v1
        id: jobs
        with:
          github_token: ${{ secrets.GITHUB_TOKEN }}
          job_name: Double  space

Although the job_name is correct, the old commands would not able to find it. Here is why:
JOBINFO='{
  "total_count": 1,
  "jobs": [
    {
      "name": "Double  space"
    }
  ]
}'
( set -x; echo ${JOBINFO} )

Output:
+ echo '{' '"total_count":' 1, '"jobs":' '[' '{' '"name":' '"Double' 'space"' '}' ']' '}'
{ "total_count": 1, "jobs": [ { "name": "Double space" } ] }

When JOBINFO is wrapped into double quotes (as it should be), no such incorrect text replacement occurs - here is the output of ( set -x; echo "${JOBINFO}" ):
+ echo '{
  "total_count": 1,
  "jobs": [
    {
      "name": "Double  space"
    }
  ]
}'
{
  "total_count": 1,
  "jobs": [
    {
      "name": "Double  space"
    }
  ]
}

Fixing this bug might in theory solve https://github.com/Tiryoh/gha-jobid-action/issues/35#issuecomment-2042077814 (although we can't know for sure, because @p0fi didn't provide much information), because it would match the information provided by @p0fi:

job_name is supposedly correct,

With this context the current action (1.2.0) does not work while 0.1.2 still works.

This matches as well, because the old entrypoint.sh used in 0.1.2 doesn't have the problem with unquoted JSON contents. This problem only exists since https://github.com/Tiryoh/gha-jobid-action/commit/ffbe0045fa580bc0821a817d2f5d28a203cb0d34, which first appeared in 1.0.0.

Hi @generalmimon,
Sorry to kept you waiting and thanks for your PR.
I'll take a look at your PR.


Double star path matching not working as expected

When trying to match something.java to the path **/something.java, the package returns false.
This seems to be incorrect. Every online glob match tester I've tried makes a successful match, for example:
https://www.digitalocean.com/community/tools/glob?comments=true&glob=**/something.java&matches=false&tests=something.java&tests=/something.java
But I could also be misunderstanding something here. Any clarification would be appreciated!

FYI, https://github.com/bmatcuk/doublestar works as expected.

{**/something.java,something.java} works...


Box or Seal

Should we use crypto_box, or crypto_box_seal - if we use seal, then the "sender" can't go back an decrypt it later, which is likely what they expect. But this also breaks any authentication, as you can't know who sent it.
Options:

Use box, and document that either party can decrypt.
Use seal, and document that it isn't authenticated.
Use box with an ephemeral key pair, signing the ephemeral public key with the sender's private key. This would be the best of both, but more complex.

Thoughts?

This might sound clunky or counterintuitive, but what about a two-step process:

crypto_sign() the plaintext message with a long-term identity key
crypto_box_seal() the signed message (and, maybe, their long-term identity public key?)

(Note: Signing with EdDSA then encrypting the signed message is different than MAC-then-Encrypt. IIRC crypto_box_seal() still uses authenticated encryption to prevent tampering.)

Perhaps add an Identity.GenerateIdentity() method that returns an Identity object (basically a wrapper for Sodium.KeyPair, with an Ed25519 key pair) - that would give a clear way to generate the long term idenity key. That key could be used to either perform the sign of the plaintext, or the ephemeral public key.
I'm thinking something like this (ephemeral public key, signature on the public key):
version[1] || identity_public_key[32] || ephemeral_public_key[32] ||
  signature[64] || nonce[24] || data[length - 153]

This should confirm that the identity of the sender, without allowing them to decrypt it, though does make the header rather large. The recipient can then chose whether or not to verify the identity of the sender.

You might want to take a look at noise boxes.


Contact compact layout not getting assigned on package install

When installing the package, the Contact compact layout is assigned for resource contact, but not for Primary or for Family Member (which inherits from Primary)

In test packaging org, manually setting to see if this fixes issue.

HUB Post: https://powerofus.force.com/s/question/0D51E00005LWXlxSAH/compact-layouts-not-getting-assigned-in-package

With latest version, this has been resolved.


Option to copy current page from corrector output to verifier output

Add an icon button on the menu bar to copy the current page from corrector output to verifier output.{This feature is already there in the tool but when verifier clicks on the corrector output pages then only it works}
This button should be visible for verifier only , corrector should not be able to use this feature.

Completed on commit number :-> 666fcb6

This feature is working for linux but not for windows....

Optimized copy to verifier feature at commit number :- 39953ed5940cc10e7020099c59c5e4d2027b19fd
Working for windows also.


http and file resolver should be added via optional features

Currently, jsonschema lib can't be compiled down to a wasm module since it relies on HTTP and File IO which aren't supported by the WASM standard.
It'd be better to allow users to add the ref resolver module as an optional feature.
jsonschema = { version = "0.16.0", features = ["ref-resolver"]}

Are the resolve-http and resolve-file features not sufficient for this?

@Stranger6667 Oops missed it! Yeah disabling those would totally work.

Great to hear! :) I'd like to have jsonschema compiled to WASM eventually
Related: #222


add filter and sort options to state to save selected values

https://app.clubhouse.io/augur/story/15171/search-filter-settings-should-be-saved-when-navigating-around-the-app-going-back-etc

Coverage increased (+0.05%) to 66.553% when pulling bf60bd62724be841485ad76a453e10090f26967d on phoebemirman/ch15171/search-filter-settings-should-be-saved-when into ec7f4cb61a0a0530cb3d74a14c8835b1d542dee8 on master.

Coverage increased (+0.05%) to 66.553% when pulling bf60bd62724be841485ad76a453e10090f26967d on phoebemirman/ch15171/search-filter-settings-should-be-saved-when into ec7f4cb61a0a0530cb3d74a14c8835b1d542dee8 on master.

I see the filter and sorting go back to default even though the drop downs keep their state.

to repro:

from fresh start, select in reporting and creation time
click on reporting
click back to markets

notice that the drop downs are correct but the markets shown are in open state


Reconsider descending numbering

I'm thinking that descending numbering is possibly a mistake. It's intended to fix the problem that satoshis with short, desirable names were mined in early blocks, which presumably won't ever move. We could fix this by defining the naming scheme to go from long to short, instead of reversing the numbering of all satoshis. I think this would be better, since ascending numbers feel more natural and are easier to explain. Using ascending numbers would mean that desirable numbers, like single-digit satoshis, were mined in the genesis block, which is still unfortunate. However, it might be worth the additional simplicity.

Comments welcome!


Fixed incorrect cybox patterning in indicators and formatted JSON with JSTool

Thanks, @ATweedMITRE.
I'm honestly not a huge fan of the way JSTool formats JSON, (for instance }, { on the same line), but as long as it's consistent, I'm not too worried.
I would like to leave spaces around the " = " in patterns, though. I'm not 100% sure it's required, but it will help with consistency, I think. Can you fix that?


Add Mapping for General Insert and Update that Renders null Values More Simply

Rather than rendering null values as a null parameter, the new mapping will simply render null values as "null" in the generated SQL.
This mapping will be important in the updated Kotlin DSL where we more clearly express nullability exppectations.

Coverage remained the same at 100.0% when pulling d84743ebb1094547ae0b7a3fb48547ea1050d941 on jeffgbutler:value-or-null-mapping into b1fa3a6562c3ccf6798a64651def0e5019d5ac8d on mybatis:master.


Enhance NPSP Data Imports to pass dml.DuplicateRuleHeader.AllowSave = true;

Even if your rule says to just warn, but allow the save to go thru, any code doing inserts (apex, triggers) must explicitly pass in a flag to the Salesforce API to allow the save. (dml.DuplicateRuleHeader.AllowSave = true;)
Unfortunately, NPSP does not pass in this flag in most scenarios. we should see if we can add in this flag when inserting contacts and accounts for data import.
hub thread: https://powerofus.force.com/0D580000035esUw

**lurch: add

Tracking W-017008

Included in beta release 3.115 (Beta 8)

Included in production release rel/3.115


playbooks/3_setup_kvm_host.yaml failes, when 'ansible_user: root' is used

playbooks/3_setup_kvm_host.yaml failes, when 'ansible_user: root' is used.
TASK [Set libvirt management to libvirt group instead of root.] *****************************************************************************
ok: [a3elp60]

TASK [Create file for user's custom libvirt configurations.] ********************************************************************************
fatal: [a3elp60]: FAILED! => {"changed": false, "msg": "Error, could not touch target: [Errno 2] No such file or directory: b'/home/root/.config/libvirt/libvirt.conf'", "path": "/home/root/.config/libvirt/libvirt.conf"}

Fix with PR #197

Code is merged.


Fix papercuts for working with Salesforce

cc @criswit

https://github.com/sigmavirus24/gobayeux/pull/8/checks?check_run_id=2050684964#step:3:71 is marked as failing but has no issues found ...


feature: Options to reject floating-point NaN and Inf values on encode and decode

Is your feature request related to a problem? Please describe.
I am implementing CBOR support as a binary alternative to an existing (encoding/json-based) JSON serialization layer. It's essential that all objects entering the system can continue to be JSON-encoded for consumption by existing clients. The JSON serializer refuses to encode or decode floating-point NaN or +/- Inf values, so I want to be able to configure the CBOR serializer to reject them as well.
Describe the solution you'd like
Based on https://www.rfc-editor.org/rfc/rfc8949.html#section-5-3:

CBOR-based protocols MAY specify that they treat arbitrary valid data as unexpected.

I would like to be able to configure both EncMode and DecMode to return errors rather than encode or decode NaN and Inf floating-point values. Specifically:

Extend the existing NaNConvertMode and InfConvertMode with new modes NaNConvertReject and InfConvertReject, respectively, because configuring an accepted conversion is mutually exclusive with rejection. If so configured, marshaling will return UnsupportedValueError when asked to encode a NaN or infinity value. The default mode for both options will remain unchanged.
Add new decode options NaNDec and InfDec with "Allow" (default) and "Reject" modes. In the reject modes, unmarshaling will return UnacceptableDataItemError if a NaN or infinity floating-point value is encountered in the input.

Describe alternatives you've considered
Additional context

I have a complete implementation in https://github.com/fxamacker/cbor/pull/513 if this proposal seems reasonable and useful.

Thanks Ben! :tada: Closed by #513.


Unexpected packet before version

Expected Behavior
syncing to local or uploading
Actual Behavior
get error Unexpected packet before version
Steps to Reproduce the Problem
SFTP: upload  or download
Requisites (important to diagnose the problem!)
Extension Logs
[error] Error: Unexpected packet before version
	at SFTPStream._transform (C:\Users\user\.vscode\extensions\liximomo.sftp-1.1.11\node_modules\ssh2-streams\lib\sftp.js:306:27)
	at SFTPStream.Transform._read (_stream_transform.js:167:10)
	at SFTPStream._read (C:\Users\user\.vscode\extensions\liximomo.sftp-1.1.11\node_modules\ssh2-streams\lib\sftp.js:183:15)
	at SFTPStream.Transform._write (_stream_transform.js:155:12)
	at doWrite (_stream_writable.js:329:12)
	at writeOrBuffer (_stream_writable.js:315:5)
	at SFTPStream.Writable.write (_stream_writable.js:241:11)
	at Channel.ondata (_stream_readable.js:557:20)
	at emitOne (events.js:96:13)
	at Channel.emit (events.js:191:7)
	at readableAddChunk (_stream_readable.js:178:18)
	at Channel.Readable.push (_stream_readable.js:136:10)
	at SSH2Stream.<anonymous> (C:\Users\user\.vscode\extensions\liximomo.sftp-1.1.11\node_modules\ssh2\lib\Channel.js:166:15)
	at emitOne (events.js:96:13)
	at SSH2Stream.emit (events.js:191:7)
	at parsePacket (C:\Users\user\.vscode\extensions\liximomo.sftp-1.1.11\node_modules\ssh2-streams\lib\ssh.js:3445:10)
	at SSH2Stream._transform (C:\Users\user\.vscode\extensions\liximomo.sftp-1.1.11\node_modules\ssh2-streams\lib\ssh.js:671:13)
	at SSH2Stream.Transform._read (_stream_transform.js:167:10)
	at SSH2Stream._read (C:\Users\user\.vscode\extensions\liximomo.sftp-1.1.11\node_modules\ssh2-streams\lib\ssh.js:253:15)
	at SSH2Stream.Transform._write (_stream_transform.js:155:12)
	at doWrite (_stream_writable.js:329:12)
	at writeOrBuffer (_stream_writable.js:315:5)
	at SSH2Stream.Writable.write (_stream_writable.js:241:11)
	at Socket.ondata (_stream_readable.js:557:20)
	at emitOne (events.js:96:13)
	at Socket.emit (events.js:191:7)
	at readableAddChunk (_stream_readable.js:178:18)
	at Socket.Readable.push (_stream_readable.js:136:10)
	at TCP.onread (net.js:560:20)

Specifications

Version: 1.1.11 - 2018-05-21
Platform: Windows 10 x64
VS Code: 1.23.1

Log in to your box via regular ssh first.
If you're seeing random text coming into your terminal, that is most likely your problem.
Figure out what's causing it and stop it (checking for rogue echo statements in the .bash* files is a good start)

thanks, it was a rogue echo statement.

CentOS7 配置SFTP用于支持root用户登录

修改配置文件/etc/ssh/sshd_config

##释掉以下语句
#Subsystem    sftp /usr/libexec/openssh/sftp-server
##后面加入
Subsystem sftp internal-sftp

重启sshd服务

systemctl restart sshd


happy new year wishes using symbols in cpp #19

Hey @kartikthakur7107 ...Happy New Year to you. May this year be more prosperous and joy filling for you.

Hey @kartikthakur7107 ...Happy New Year to you. May this year be more prosperous and joy filling for you.

About the pull request, please fetch and merge the changes first and then send the pull request again.

About the pull request, please fetch and merge the changes first and then send the pull request again.


Added Instagram Post Carousel

Fixes Issue🛠

Closes #254
Description👨💻
Added instagram post which hovers above a insta post like ui and after clicking anywhere it changes its body background color.!
Type of change📄

[] Bug fix (non-breaking change which fixes an issue)
[] New feature (non-breaking change which adds functionality)

How this has been tested✅

Checklist✅

[] My code follows the style guidelines of this project
[] I have performed a self-review of my own code
[] I have commented my code, particularly in hard-to-understand areas
[] I have added demonstration in the form of GIF/video file
[] I am an Open Source Contributor

Screenshots/GIF📷
https://github.com/Rakesh9100/Beautiify/assets/142514166/255e41ae-6aee-4bef-88a3-ca00b31ed905

Why background is changing from purple to yellow and vice versa on any clicks @AbhineshJha

its feature @Rakesh9100

Then attach the demo showing this feature as well @AbhineshJha

Done @Rakesh9100


'get_news' search function returns NoneType error

When I run the 'get_news()' function on a GoogleNews object it returns the error - error ''NoneType' object has no attribute 'find'". However, when I run the 'search' function on the same GoogleNews object it works perfectly fine. Something wrong with the get_news() function.
@bp.route('/news', methods=['POST'])
def fetchTopNews():
    gNews = GoogleNews(period='2d')
    news_articles = []
    gNews.clear()
    gNews.get_news('top news') # this code errors out
    gNews.search('top news') # this code works

This issue is fixed by latest version (v1.6.11)

I upgraded project to latest version of GoogleNews and am still having the same issue. search() works fine but get_news() returns nothing and gives error 'NoneType' has no attribute 'find'

I upgraded project to latest version of GoogleNews and am still having the same issue. search() works fine but get_news() returns nothing and gives error 'NoneType' has no attribute 'find'

Use print(googlenews.getVersion()) to make sure you get version 1.6.11


Add Decoding Token Economics event

From https://github.com/ethereum-magicians/scrolls/issues/17
Committed https://github.com/bmann/ethereum-events/commit/8b4a3f669895273fcf0d8a4f5e49df52c7e388ee
Thanks @aogunwole

Hey Boris!
Thanks for setting up so quickly! The profile looks good and I like the
authorship aspect so that we all have a sense of responsibility for the
site. Absolutely comfortable with the authorship here and Twitter/Github
references (also helps, if anyone has questions on the event to ping me/the
author directly).
On Wed, Aug 8, 2018 at 6:21 PM Boris Mann<EMAIL_ADDRESS>wrote:

Hi @aogunwole https://github.com/aogunwole -- thanks for playing along
and doing your first issue.
I experimented and credited you as submitter by creating a little author
profile for you with your name and twitter, github handles. See
https://ethereumevents.global/events/decoding-token-economics-workshop/
Let me know if you want / are OK with being credited as a submitter and
any thoughts you have on giving people "credit" for helping maintain this
site
It also appears on the home page as an upcoming event.
Clearly I need to also display the event info directly on the "full page"
https://ethereumevents.global/events/decoding-token-economics-workshop/
-- I've created #2 https://github.com/bmann/ethereum-events/issues/2
where I'll work on that.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bmann/ethereum-events/issues/1#issuecomment-411466226,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AnM-C6Slox44Q7BfPiVucTg1VqZlOWVWks5uOxAVgaJpZM4V0ImE
.

--
linkedin/in/adeolaogunwole https://www.linkedin.com/in/adeolaogunwole/ |+44
7494 837 210<EMAIL_ADDRESS>

Great! Closing this issue.


Cannot build homebridge on FreeBSD 11

root@fileserver:/usr/local/share/homebridge # node --version
v9.2.0
root@fileserver:/usr/local/share/homebridge # npm install -g homebridge
WARNING: You are likely using a version of node-tar or npm that is incompatible with this version of Node.js.r /usr/local/lib/node_modules/.staging
Please use either the version of npm that is bundled with Node.js, or a version of npm (> 5.5.1 or < 5.4.0) or node-tar (> 4.0.1) that is compatible with Node.js 9 and above.
/usr/local/bin/node[15495]: ../src/node_zlib.cc:437:static void node::(anonymous namespace)::ZCtx::Init(const FunctionCallbackInfov8::Value &): Assertion `args.Length() == 7 && "init(windowBits, level, memLevel, strategy, writeResult, writeCallback," " dictionary)"' failed.
1: node::Abort(void) [/usr/local/bin/node]
2: node::Assert(char const* const[4]) [/usr/local/bin/node]
3: ZNSt3__16vectorIPN4node14SigintWatchdogENS_9allocatorIS3_EEE21__push_back_slow_pathIRKS3_EEvOT [/usr/local/bin/node]
4: v8::internal::FunctionCallbackArguments::Call(void()(v8::FunctionCallbackInfov8::Value const&)) [/usr/local/bin/node]
5: v8::internal::Isolate*v8::internal::Object v8::internal::Builtins::InvokeApiFunction(v8::internal::Isolate*, bool, v8::internal::Handlev8::internal::HeapObject(int, v8::internal::Object*, v8::internal::HeapObject) [/usr/local/bin/node]
6: v8::internal::Builtin_HandleApiCall(int, v8::internal::Object**, v8::internal::Isolate*) [/usr/local/bin/node]
Abort
root@fileserver:/usr/local/share/homebridge #

Downgraded node to 8.9.1 and now building.

How do i do that?

cd /usr/ports/www/node
make deinstall clean
cd /usr/ports/www/node8
make install

When i try installing node8 i get this error:
log.txt

Try using pkg then:
pkg remove node
pkg install node8

That with npm install npm-node8 fixed it
Thanks


TypeScript client generation: toJSON methods break JSON.stringify

The generated objects all have a toJSON method defined, which returns a json string.
JSON.stringify, however, uses that method and expects something to be stringified instead.
From mozilla.org:

Instead of the object being serialized, the value returned by the toJSON() method when called will be serialized.

In other words, those objects all serialize to strings.

Weird, I find the serialization code that's generated for me...
toJSON() {
    return JSON.stringify(this.toJS());
}

...only in a directory call NSwag.Integration.TypeScriptWeb\scripts in NSwag's sources, ie. tests.

The client transformation code is generated in NJsonSchema which is a dependency of NSwag.
I'm not sure, but I believe this stringify behavior is intentional. What would you expect the output to be if not a string?

Yes, DTOs are generated in NJsonSchema.CodeGeneration.TypeScript...

@grovesNL The output should be a string, yes, but it's a json-encoded string string, ie. something like this:
""{ ... }""
The generated client itself calls JSON.stringify on the toJS-ed plain object, which is why this is usually not an issue. It would be nice though if one could still call JSON.stringify on the generated types' objects themselves.
I have a number of other issues with the generated code though (I'd really prefer plain objects defined with interfaces - that's going to play nicer with a state management library I use, mobx) and so will probably need to go with a customized generation anyway.

You can generate interfaces instead of classes, theres an option for that

... and you should not serialize the generated classes externally because it would not serialze correctly (the properties may be renamed)

I see. You could make it serialize correctly as JSON.stringify's behavior can be configured by defining toJSON on the to be serialized objects (which nswag does, just not in a way that fits). I realize now that it had hardly any benefit though, especially since I don't think JSON.parse can be similarly configured.
The best suggestion I could make in this topic is that perhaps a JSON.stringify on the generated types should throw an exception as a heads-up to the programmer.
This issue should probably be closed then.

Maye we should rename toJS to toJSON and toJSON to JSONstringify so that the behavior is as expected...