SGDP_continental prediction fails

Hi, I'm getting this error every time the SGDP_continental step is reached.
SGDP_continental
Check ordered length of SNPs. "None of [Index(['chr1_69270_A_G', 'chr1_69511_A_G', 'chr1_69761_A_T', 'chr1_69897_T_C',\n       'chr1_930314_C_T', 'chr1_939285_G_A', 'chr1_939354_C_T',\n       'chr1_939398_GCCTCCCCAGCCACGGTGAGGACCCACCCTGGCATGATCCCCCTCATCA_G',\n       'chr1_941282_C_A', 'chr1_942930_T_C',\n       ...\n       'chr22_50627172_G_A', 'chr22_50627388_G_A', 'chr22_50679152_T_C',\n       'chr22_50714943_G_A', 'chr22_50720830_C_T', 'chr22_50744827_A_G',\n       'chr22_50744993_C_T', 'chr22_50745030_A_C', 'chr22_50745036_A_C',\n       'chr22_50776695_C_T'],\n      dtype='object', length=97995)] are in the [columns]"
Cannot coerce data. Check sparse matrix: None
Traceback (most recent call last):
  File "/opt/conda/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/opt/conda/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/opt/Ancestry/igm_churchill_ancestry/__main__.py", line 4, in <module>
    run_ancestry()
  File "/opt/Ancestry/igm_churchill_ancestry/cli.py", line 182, in run_ancestry
    ofn=ofn)
  File "/opt/Ancestry/igm_churchill_ancestry/pipelines/ancestry_prediction.py", line 111, in run_ancestry_pipeline
    yprob, ylabel = predict_ancestry(s_matrix, ml_dir=ml_dir, n_classes=n_classes, model_type='svm')
TypeError: cannot unpack non-iterable NoneType object

I've tried different vcfs, including the one posted in the "best practice VCF input" issue. I've also tried building with docker compose and using the heritagehelix/snvstory:3.0.0 docker image, same error. All the preceding predictions appear to run just fine. Any ideas as to what might be causing this?
Not sure if it helps, but the log files always end with these three lines:
2024-03-05 17:06:10,180:DEBUG:==== SSA block rewrite pass on 511
2024-03-05 17:06:10,180:DEBUG:Running <numba.core.ssa._FixSSAVars object at 0x7fdaf4acee90>
2024-03-05 17:06:10,180:DEBUG:on stmt: jump 154

ran into this same issue. seems like the code is trying to grab the sgdp var_id.json but is incorrectly reading in some other jsons. renaming the resource files:
sgdp/continental/matrix_attributes/sgdp_country.json
sgdp/continental/matrix_attributes/sgdp_sample.json

to something that does not end in .json got past that error for me.
that said, i still haven't been able to run the code to completion

@SigveLan could you provide the docker command you used to run snvstory?

Also could you verify that your sgdp/continental/matrix_attributes looks like this:

Yes, so this is the command I've used:
docker run -it --rm -v $PWD:/data heritagehelix/snvstory:3.0.0 \
	--resource /data/resource_dir/ \
	--genome-ver 38 \
	--mode WES \
	--output-dir /data/output/ \
	--path /data/sample_1.vcf.gz

As for the files they are exactly the same as you posted.

Your command looks good to me, when I run it locally it works w/o any issues. I'm going to try and add some tests to the code base this week and hopefully you'll be able to run them and we'll be able to extract more diagnostics from that. By the way what operating system are you using?

Hello! Just adding that I'm seeing the same error on my side when running a test with the provided HG00096_extracted.vcf.gz file.
I'm using singularity with the provided docker image with this command:
singularity exec -B $PWD:$PWD,/projects/btl/lcoombe/bin/snvstory/3.0.0/dev/data/resource_dir/:$PWD/dev/data/resource_dir \
   --env PYTHONPATH=/opt/Ancestry/:$PYTHONPATH --env TMP_DIR=$PWD/tmp \
    /projects/btl/lcoombe/bin/snvstory/3.0.0/snvstory-3.0.0.sif python3 -m igm_churchill_ancestry  \
   --path HG00096_extracted.vcf.gz --resource $PWD/dev/data/resource_dir --genome-ver 38  \
   --mode WES --output-dir /snv_out

The matrix_attributes resource dir also looks as expected:
(btl) [lcoombe@hpce706 HG00096_test]$ ls /projects/btl/lcoombe/bin/snvstory/3.0.0/dev/data/resource_dir/sgdp/continental/matrix_attributes/
ordered_sgdp.intersect_exome.sparse_matrix.var_ids.txt  sgdp.intersect_exome.sparse_matrix.var_ids.json  SGDP_umap_attributes.csv
sgdp_country.json                                       sgdp_sample.json

I think I found the issue; @timchu90  was barking up the right tree. The problem has to do with the matrix_attributes folder having multiple JSONs. On my local machine (and I guess the machine where we developed this on) sgdp.intersect_exome.sparse_matrix.var_ids.json loaded first, whereas when I run the singularity command sgdp_country.json is what loads first, which is not what we want.
 I'll try and put a fix in this weekend.

@SigveLan @lcoombe and @timchu90 I've put out a PR for this fix and pushed a new version of the docker heritagehelix/snvstory:3.0.1 Feel free to use the new docker version and I'll merge the PR once I have confirmation from one of you that it works, or if I get a around to running it on the server with singularity.

Works for me!

Thanks for the quick fix! I was able to run the new image using singularity successfully on the HG00096_extracted.vcf.gz test data 👍


Delete nanotipbot/cron

delete

Not really necessary, the 3.0 branch was already merged into master


[style]: Add button for add your profile link.

What's the style idea?
We need to improve the style of link that is used to direct the users to the github repo to guide them how to add their profile. If we make a button for this link, it would look more nicer.
Add screenshots
Add screenshots
Code of Conduct

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

Hi @shyamtawli,
I want to work on this issue. Could you please assign it to me?

Go ahead @Sakshi29G
Try to make the button similar to the theme and match the colors.

Sure.

@shyamtawli I have added the button but it is showing conflict in src/ProfilesList.json. Could you please help me with it?

Yeah I have checked it. The PR you opened for profile addition contains conflicts but as you closed it, you can now open a new PR for that with a separate branch.

Hey @RahulSoni2532 This is Issue is closed. You can look on other issues.


InvalidArgumentError when using tensorboard in keras

I am training a sequence-to-sequence encoder-decoder model using Keras with a tensorflow backend. everything is up to date. I tried adding in the TensorBoard callback like this:
earlystopping = EarlyStopping(monitor='val_loss', patience=3) tensorboard = TensorBoard(log_dir="log/{}".format(time()),                             histogram_freq=1, embeddings_freq=1) his = model.fit([training_encoder_input, training_decoder_input],                 [training_decoder_output],                 batch_size=BATCH_SIZE,                 epochs=EPOCHS,                 validation_split=0.2,                 callbacks=[earlystopping, tensorboard])
However, when I try to run the code, it gives me the error:
InvalidArgumentError: indices[11,17] = 1265 is not in [0, 256)
[[Node: embedding_1/embedding_lookup = GatherV2[Taxis=DT_INT32, Tindices=DT_INT32, Tparams=DT_FLOAT, _class=["loc:@training/Adam/Assign_2"], _device="/job:localhost/replica:0/task:0/device:CPU:0"](embedding_1/embeddings/read, embedding_1/Cast, training/Adam/gradients/embedding_1/embedding_lookup_grad/concat/axis)]]
I am not sure what is wrong. It works perfectly without it.

the same issue,wait for help


[website_docs] Use console code-block style, and spellcheck

Contributes to https://github.com/open-telemetry/opentelemetry.io/issues/948 and https://github.com/open-telemetry/opentelemetry.io/issues/949. /cc @austinlparker
Corresponding website PR: https://github.com/open-telemetry/opentelemetry.io/pull/950

Thanks for the review and approval @arielvalentin. Could we get both website docs PRs merged soon? Thanks.

@chalin I don't have merge powers just yet. That honor is only restricted to @open-telemetry/ruby-maintainers


Update Selenium Tests

What is the context of this PR?
Current Selenium test for MCI survey is out-of-date. This PR brings it in line with the current application. It also updates some dependencies.
How to review

run the application.
run npm run test_functional to run test against local Chrome instance.

Who worked on the PR
@hamishtaplin

What does this do?

@dhilton Updates the selenium test to pass on the current version of the app.

LGTM 👍 - helps if I read the PR instructions...


ARROW-9062: [Rust] json reader dictionary support

Changes in this PR allow the JSON Reader to read into Arrow arrays of type dictionary.

Hey @nevi-me as @vertexclique approved as well, can you merge this? Or how shall we proceed? Thanks, Sven


Bag register should handle the case where it receives the same message twice

Closing as "yes, I'm pretty sure it does".


Search attachment Selector

#1306
Checklist

[x] the [contributor license agreement][] is signed
[x] commit message follows [commit guidelines][]
[x] tests are included
[x] screenshots are included showing significant UI changes
[ ] documentation is changed or added

Description of change
So here is another Refine Search control - Search attachment Selector. By default, the option of 'yes' is selected. And because there are no Search settings controlling this component's visibility, it's always visible.
The structure of this component is almost same as StatusSelector, and so are its tests and stories. So we can create an abstract component called TwoOptionsSelector , but I am not sure how much value we will gain at this stage so I did not implement this.

The structure of this component is almost same as StatusSelector, and so are its tests and stories. So we can create an abstract component called TwoOptionsSelector , but I am not sure how much value we will gain at this stage so I did not implement this.

Yup I was ponder this too when you started on it. However it'd be a very thin layer on top of ButtonGroup. But maybe in the future we may want a component that is called something like ButtonToggle which could take a collection of button definitions so you could have two or more in the group. The definition would probably specify a label and what the output value would be when it's clicked/selected. And the component would have the logic to ensure only one can be selected at a time.
But for now, let's just wait and see if we end up with at least one more.


[verilator] Remove clock gating waiver and add extra warning

The -Wwarn-IMPERFECTSCH flag is suggested in the Verilator docs (to
help spot if we get a clock gating hint wrong). The waiver is no
longer needed because we now add the relevant hint in the
RTL (vendored in prim_generic_clock_gating.sv from OpenTitan).
Closes #212.

Thanks @rswarbrick , LGTM!

Thanks @rswarbrick , LGTM!


[master] Upgrade to latest dependencies

Produced via:
./hack/update-deps.sh --upgrade
./hack/update-codegen.sh

Justification, Sync deps after eventing cut -slinkydeveloper
/cc knative-sandbox/channel-wg-leads
/assign knative-sandbox/channel-wg-leads

/assign

/assign

/lgtm
/approve

/lgtm
/approve

/retest

/retest


Update doviz.com api endpoints

@muratbsts can you add more details to this so that i can pick it up?


Issue with "docroot/static/style" directory with "theme: foobar"

My mission was to take the default_theme/post.html and override it so that I could move the Author and Date below the content, not above. As part of this, I started setting up my own styles/foobar config (copied from 'classic'). Latest pykwiki from pip (2.1.18) being used.

The post.html template contains {{conf.style_post}} which turns out to just load the raw content from styles/foobar/post.html but doesn't send it through Jinja2, so it gets the raw template inside the parsed template - OK, so the design is this is HTML styling only, no Jinja2 in there.
The above led me to reading the code, the render_theme_template() routine sends conf.theme_path not conf.style_path through Jinja2, and because the new default in config.yaml is theme: None it actually sends the same path (data/default_theme) through it twice, so
I set up a themes/foobar/post.html and configured theme: foobar to align to the code, which worked (pykwiki cache -f) but....
...if docroot is missing (my default, it's a Gitlab CI runner so always a clean docker) and themes/foobar/static/ directory is missing, it causes a python crash right about here when it tries to copy content to static/ and the directory hasn't been made: https://github.com/nullism/pykwiki/blob/master/pykwiki/core.py#L422

So the tl;dr to get where I wanted:
make themes/foobar/static/.gitkeep
create themes/foobar/post.html
set theme: foobar

The docs for v2 are kind of light, the doc says "theme is now null" but that's about it, so it's hard to tell if this was intentional or not, as it's not really described anywhere (but it works). I think though I found a bug there in the last item, what's actually crashing with a "no such file or directory" is the code right after the copying:
        with open(os.path.join(style_dest, 'style.css'), 'w') as fh:
            fh.write(style_data)

Because the code above it is only triggered when theme != None, it kind of goes wonky a bit when I set theme = foobar. The logic looks like if it were written to better check stat_dest and style_dest a little better it might work, but I can see maybe 3 ways to "fix" it depending on design intent.

Hi troyengel,
You did the correct thing by creating your own theme. Styles do not (and should not) be parsed by Jinja, as they're just custom HTML to be loaded in the page.
Just to confirm, when you manually added the static/ directory to your foobar theme, it didn't throw an exception, correct?
Maybe the correct thing to do is to make a static directory inside the target's docroot when it's not present. I'll look into it a bit more before making the change.
Also I agree that the docs are very light. Shortly after creating Pykwiki2, Gitbook added their search feature, so I assumed that'd become much more popular (Pykwiki's claim to fame was the only static content generator that had built-in search) and didn't spent a lot of time on the docs.
Glad folks are using this; might be time to update the docs.
Thanks for the feedback!

I've testing this change and published a new version (2.1.19) to pip. If I understand the issue correctly, this should now be resolved. I fixed a few other issues (trailing whitespace, redeclared variables), but the main change is here: https://github.com/nullism/pykwiki/blob/6ad6f1ec716d9888c49b8851564417154371adaa/pykwiki/core.py#L430
Feel free to re-open if the issue persists and we can work through it.

Working great, thanks!


Não Tem Visualização

I've always found getting Pascal projects up and running a bit intimidating because difficult and complex (for me).  It would be nice to have a preview, perhaps even animated.  But I cannot demand anything 😄

I've always found getting Pascal projects up and running a bit intimidating because difficult and complex (for me).  It would be nice to have a preview, perhaps even animated.  But I cannot demand anything 😄

I understand what you mean. I have a preview (ux/ui) preview on the following link:
https://www.sketch.com/s/f97cac13-7538-47a1-83b1-7e8e9286ff87

Muito obrigado 🙇 I am impressed with the interface.

Muito obrigado 🙇 I am impressed with the interface.

Obrigado, você! I really apreciate your comment on the interface.

Dear @GirkovArpa ,
I've added some images as you suggested some time ago. Hope you enjoy this updade and if you want, you can download the 1st Windows and/or Android release.


Codes X, C and B

Census crew David, @jess-shue, @Iriskennedy @CarolineTroy @rmh24 @krystalbagnaschi,
I thought we decided to use codes C (dead above 1.3) and X (broken below 1.3) and B (Broken above 1.3) but I am noticing you have not been using them.
The main reason I am asking is because now the DBH=0 gets flagged because they are not justified. My current check is looking in the codes for a justification of why dbh = 0. It is looking for X and 'C' (although now I am noticing that I should not look at C since we have decided to measure dead trees anyways).
My question is: did you guys decide to drop those codes, or did you forget to record them?
If you decided to drop them, I'll just look at the status of the tree and say that DC (or DN) justifies a DBH=0.
But what about DS trees? those DBH do need to be measured, to be used in the mortality census.

This is what was discussed last Wed:

@ValentineHerr Yes, I believe it would be best to go off of the status column first, then use the codes as justification.
Stems < 100 mm (10 cm) are not assessed for mortality. So, we discussed in the field last week to not measure DS stems < 100 mm as it would save time and will not be used for mortality.

Here are my new thoughts:

Suzanne said:

I check for dbhs that are 0 (not measured), they should have a code explaining why it was not measured (dead, stem broke off, stem does not reach 1.3 m in case of multiple-stemmed trees, etc.).

My understanding is that the status column does not get sent to ForestGEO. I know BCI does not even have that column.

We are requiring DBH (so we have it for mortality census), so a rightfully non measured DBH will be 0 (instead of NA, like Suzanne may expect).

Conclusion: We do need to add a code for those dead trees that have dbh=0 even if they are <100mm. Don't measure them but add the X (if status is DC) or C (is status is DS). OR I need to change them to NA.
Does that sound right @jess-shue ? We can discuss on teams in a few minute.

After chatting with @jess-shue, the status column can be sent to Suzanne, if concatenated into the codes column. So she will see that DS, DN or DC justify a dbh = 0. So I'll reflect that in my checks and we will be able to close this issue.
No need to change anything in the field or the data.


search_after has no method to clear value but only can append

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.v7 (for Elasticsearch 7.x)
[x] elastic.v6 (for Elasticsearch 6.x)
[ ] elastic.v5 (for Elasticsearch 5.x)
[ ] elastic.v3 (for Elasticsearch 2.x)
[ ] elastic.v2 (for Elasticsearch 1.x)
Please describe the expected behavior
When I have to use search_after for several times to fetch all the data, I found the client can only append new search_after values but not replace the old one.
Code Example
baseQuery := e.client.Search().Index(query.TableName).Size(e.cfg.MaxFetchDocs).
		Query(hashTermQuery).Sort("_uid", true)
// init first query
			searchResult, err := baseQuery.Do(ctx) // execute
			if err != nil {
				level.Error(util.Logger).Log("msg", fmt.Sprintf("Query in index %s met error!", query.TableName))
				return nil
			}
			if searchResult.Hits.TotalHits > int64(e.cfg.MaxFetchDocs) { // need to run "search_after"
				var sortValue interface{}
				for {
					//  Get Sort value from last SearchResult
					sortValue = searchResult.Hits.Hits[e.cfg.MaxFetchDocs-1].Sort[0]
					fmt.Println(sortValue)
					// Search
					searchResult, err = baseQuery.SearchAfter(sortValue).Do(ctx) // execute

Running Result
lokiindex#HDU5aW8BUA0rscJrN6Fs
map[query:map[term:map[hash:fake:d18161:logs:job1]] search_after:[lokiindex#HDU5aW8BUA0rscJrN6Fs] size:5 sort:[map[_uid:map[order:asc]]]]
lokiindex#ITU5aW8BUA0rscJrN6Fs
map[query:map[term:map[hash:fake:d18161:logs:job1]] search_after:[lokiindex#HDU5aW8BUA0rscJrN6Fs lokiindex#ITU5aW8BUA0rscJrN6Fs] size:5 sort:[map[_uid:map[order:asc]]]]

Please describe the actual behavior
Can we have a method to clear the search_after in SearchService, like "baseQuery.clearxxx()" in above? Otherwise I have to deep clone the baseQuery above which is so heavy.
Something like this
func (s *SearchService) SearchAfterClear() *SearchService {
	s.searchSource.searchAfterSortValues = nil
	return s
}

Any steps to reproduce the behavior?

You can work around this by creating a new query instead of reusing the old one.

building new query will make the code really ugly especially in the cases I have to change search_afater for multiple times to iterate all the data set. Is that fair enough?

Agreed with @mizeng If I have to make an entirely new query in the middle of the logic it's going to make quite a mess of an otherwise clean codebase. 😱


DC-3504 :: Provide implementation for the new /confirm endpoint

@pangiole-hmrc Do you want to have a call with Satya Chuck and Micheal to answers the TODO questions?

superseded by https://github.com/hmrc/channel-preferences/pull/28


Add app_version to GetPlayerMessage

This seems to be a new field that the app is sending now. It contains a string such as "0.31.1".

Does this fix https://github.com/AeonLucid/POGOProtos/issues/131 ?

No, but it's something else that changed.

Cool, useful. Do you mind adding a comment right before the field explaining the string format? Just add something like you said here

It contains a string such as "0.31.1".

Sure, like this? :)

Thanks!


Addition of Diva Styles

Credits to Diva - https://civitai.com/models/135307/comfyui-diva-styles-workflow

Need to check with Diva about adding it here.


Left hand Panel Connect with community in new area.

Bug description
When editing in a new area we get social media and connection tools for Open Street Map.  This would be good to change to Open Historical Map.
What should be happening?
Appropriate links on brand
Repro Steps
Please provide detailed instructions to reproduce the behavior:
I don't know how this is triggered exactly
Used a new account in a new area "first edit" to get this to pop up after saving.
Screenshots & URL
https://photos.smugmug.com/photos/i-T8Lt7g9/0/020c6011/XL/i-T8Lt7g9-XL.png

One of ways to achieve this is to setup OHM specific https://github.com/osmlab/osm-community-index and change what is being loaded there.
Making OHM specific https://github.com/osmlab/osm-community-index or at least gathering materials (listing OHM mailing lists/Discords/FB pages/forums and local communities if any) is relatively simple compared to a typical coding

My local OSM community welcomes OHM contributors, so there would be no problem with presenting it inside of OHM iD. I don’t know how common it is for the two projects to share a local community, since OHM is still relatively obscure, but at this stage, an OHM contributor would be better off connecting with OSM contributors than being stranded without a similar-sized community.

OpenHistoricalMap/iD#183 replaces global OSM community resources with the OHM Forum and IRC channel, respectively. We can keep this issue open to track adding other OHM-specific resources like Mastodon and Facebook.


Request: Shortcut Arrow Removal

Summary of the new feature/enhancement
Toggle to remove/replace the shortcut arrows on desktop icons
Proposed technical implementation details (optional)
It's a simple registry edit, detailed at https://www.windowscentral.com/how-get-rid-shortcut-arrows-windows-10
That said, many people dislike mucking about in the registry due to the potential to bork the computer.

I would like a switch that toggles shortcut arrows on/off, even if the user needs to log out & back in again to see the change (prefer it not be necessary, but code is code)

Unless it is documented on a microsoft documentation site, i don't know if we'll be allowed to do it.  Stuff that isn't documented can change.

#137

Closed as duplicate of #137


[NEW ALGORITHM] Equilibrium Index

Overview
This pull request introduces a C program that determines the equilibrium index of a user-provided array of integers. The algorithm efficiently checks for the equilibrium index by calculating the sum of elements on both sides of each index, providing a linear-time solution.
Changes Made

Program.c: Contains the C implementation of the equilibrium index algorithm.
README.md: Provides comprehensive documentation including:

Description of the problem and solution approach.
Detailed structure of input data and function parameters.
Time and space complexity analysis of the algorithm.
Example inputs and outputs demonstrating functionality.

How to Test

Compile the Program.c using a C compiler (e.g., GCC).
Run the executable and provide the array elements when prompted.
Verify the output to confirm the equilibrium index or absence thereof.

Issue Resolved

Issue: Equilibrium Index #1438

Additional Notes

The implementation is designed to handle a variety of input lengths and values, ensuring robustness and efficiency.
This program operates in O(n) time complexity and O(n) space complexity due to the auxiliary arrays used for left and right sums.

Please review my PR and add labels as per my work @pankaj-bind.

not assigned


implement realtime extension support: AAPXS SysEx8

As it has been written in several issues and documents (e.g. issue #73, issue #145, issue #158), current Binder-based extension() support does not cover realtime usage scenario and they should be covered as in MIDI2 system exclusive messages at process().
Now that we have refresh implementation for plugin manager with aap::MidiDestinationNode which is ready to be used in aap::PluginPlayer, it is time to actually implement it.
There are handful of blocks that need new code:

[x] `PluginPlayer needs to provide users ways to handle its MIDI2 outputs, either via event handler or provide direct buffer to MIDI outputs.
[x] cmidi2 needs sysex8 data extraction feature
[x] implement MIDI2 AAPXS SysEx8 parser and generator
[x] MIDI AAPXS service needs to parse MIDI2 sysex8 inputs into extension invocation, serialized as extension bytes.
[x] aap::LocalPluginInstance needs to integrate this MIDI input parsing support.
[x] MIDI AAPXS client needs to generate MIDI inputs from extension invocation, serialized as extension bytes. -> post-0.7.8 task
[x] aap::RemotePluginInstance needs to integrate this MIDI output generation support.
[x] the new plugin manager needs to implement realtime extension invocation. MIDI Program Change is a good example for this (involves preset setter).
[x] There should be asynchronous callback support in every async-ified (non-RT) extension function, and they need to be invoked at each AAPXS implementation.
[x] C++ wrappers in standard-extensions.h and its implementation also needs async-ification.

Unless it is going to be too complicated, it should be done before 0.7.8 release.

Unless it is going to be too complicated, it should be done before 0.7.8 release.

As issues #171 is post-0.7.8 work, the entire work is inherently moved to post-0.7.8 as well. Until then, we will have proof-of-concept setPresetIndex() call at aap::AudioPluginNode::setPresetIndex(int index) to conditionally generate AAPXS SysEx8 and send it.

Most of the work is now done in aapxs-v080 branch, but I noticed that binder-client-as-plugin's get_extension() requires public extension API implementation for each AAPXS. It also needs to be provided by each AAPXS developer.
It is possible to achieve that by TypedAAPXS implementation classes, but that requires AAPXSDispatcher and its instantiated shared memory buffers, while AAPXSDefinition itself is basically all statically resolved. We likely need to provide another way to instantiate the registry, kind of.

^ is resolved as get_plugin_extension_proxy() member function in AAPXSDefinition.

We can close this issue once aapxs-v080 branch is merged to main.

It is now merged into main.


@decorator function to compose classes

Hi @choonkending, when i use a decorator function on a react class i get the following console.warn:
Warning: React attempted to reuse markup in a container but the checksum was invalid. This generally means that you are using server rendering and the markup generated on the server was not what the client was expecting. React injected new markup to compensate which works but you have lost many of the benefits of server rendering. Instead, figure out why the markup being generated is different on the client or server:
 (client) 3">WebRTC
WebRTC<h1 data-reactid=".vt

here is the example decorator
function withWebRTC(ComposedComponent) {
  return class WebRTC extends React.Component {
    constructor() {
      super();
      this.state = {
        whatever: true
      };
    }
    render() {
      return (
        <ComposedComponent {...this.props} whatever ={this.state. whatever} />
      );
    }
  };
}

The code is working properly but i guess it wouldn't be always the case.

@tsm91 Yea, this occurs when the server rendered template returned is not the same as what the client expected.
If you are using the UniversalRenderer function, you might need to bootstrap the store in the store in routes.js, instead of initialising the state like this this.state = {         whatever: true       };
Also I noticed that the client has
 after your '', whereas your server has
 after `'.
If you have a branch somewhere, I could have a look at it when I have the time.
Btw, WebRTC :)

Im not sure this is related to decorators. I've gotten that warning a couple of times, I think if you update your jsx too much the server and client get out of sync. Anytime i see that a clean and a build fix me right up.

I got this message on every change at any jsx file
this weird trick helped me to get rid of that annoying message
http://stackoverflow.com/a/33521172/1221082

Closing this issue, as it's been quite for a long time, and the code had changed a lot since. If this problem resurfaces, please create a new issue.


position, colour, and background colour of text labels in draw_bounding_boxes

🚀 The feature
Text labels from torchvision.utils.draw_bounding_boxes are currently always inside the box with origin at the top left corner of the box, without a background colour, and the same colour as the bounding box itself. These are three things that would be nice to control.
Motivation, pitch
The problem with the current implementation is that it makes it hard to read the label, particularly when the bounding box is filled (because the text has the same colour as the filling colour and is placed inside the box.
For example, this is the results from the current implementation:

Moving the label to outside the box already makes things better:

But by controlling those three things (placement of label, background colour behind the label, and text colour) one could fit to whatever they have. For what is worth, in the original issue for this feature, the only example image had labels outside the box, text coloured different from the box (black), and background of the same colour as the box. See https://github.com/pytorch/vision/issues/2556#issuecomment-671344086
I'm happy to contribute this but want to know if this will be accepted and with what interface.

Thanks for opening this issue @carandraug. I think the proposal is reasonable, the current position of the label does make them difficult to read.
In terms of API / functionality, what exactly would you have in mind?

Hi, I'd have the same request!

Option 1
How about adding a 'text_position' argument to torchvision.utils.draw_bounding_boxes, with options 'inside', 'above', 'below', 'left', 'right', and 'auto' (with auto taking one of the options as default but falling back to different ones if text would end up outside of the image or on top of other text). I guess, auto would be a next iteration :)

Option 2
The same  'text_position' argument is a function returning x,y coordinates where the text should start w.r.t. bbox

In terms of API / functionality, what exactly would you have in mind?

For the colours, I think something like draw_label_kwargs could then be passed to draw.text. This exposes all the flexibility that PIL provides us.
For label positioning, there is a lot of things going on. There is the position to the box, and then whether the label goes outside or inside the box.  I think we could just copy matplotlib syntax for placement of the legend box. Effectively, a string defines location ("upper right" or "center left") and a tuple defines a sort of offset to that (see bbox_to_anchor). This stackoverflow answer explains it it detail https://stackoverflow.com/a/43439132 )

That SGTM, thanks for the details.
Provided that the licensing terms of matplotlib allow it, I hope there exist a piece of code that we can just copy/paste from matplotlib to get the label position based on the user-defined parameter and on the current bbox position. It would be a lot more complex if we had to implement all of that logic ourselves.

That SGTM, thanks for the details.

Can you confirm that both (text colour/font/etc and label positioning) look good?  Just want to make sure that you're not only referring to the label positioning?  I If so, I will prepare a PR for text colour/etc first and then another for the label positioning.

Provided that the licensing terms of matplotlib allow it, I hope there exist a piece of code that we can just copy/paste from matplotlib to get the label position based on the user-defined parameter and on the current bbox position. It would be a lot more complex if we had to implement all of that logic ourselves.

I'll take a look at matplotlib license and logic but we probably can't use it as is. At the very least, we need to handle the case where placing the label outside the resion places the label outside the image. With matplotlib, the plot has marging but we do not. I'll experiment and see what I think is the most reasonable behaviour when I start coding but I think it will be try to place the closest to the desired position while ensuring that the text stays inside the image.

I If so, I will prepare a PR for text colour/etc first and then another for the label positioning

Yes, that sounds good. Happy to consider better default for these.


add the first version of the rfc8794 XML source

As provided by RFC editor during the AUTH48 phase.
It doesn't exactly match with the mmark output.

It is worth because ultimately that's the XML that will create the real RFC.
But it also helps to fix the remaining changes (shown in the diff but not in their emails) and know how/if we can generate it properly. Particularly because we're going to generate more documents for Matroska and it's better if they are in the right format from the start.
I'll see about adding some text in the README


Add a new option to save the boostrap token as a claim

See issue #17

Any update?

Sorry, I've been away for the last couple of days. I'll look at it today.
On Wed, Aug 16, 2017 at 12:12 AM epot<EMAIL_ADDRESS>wrote:

Any update?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/chrisdrobison/aspnetcore-wsfed/pull/18#issuecomment-322675860,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABu24Wp_MO5Y_Syl_Nl_ZSOC6AINoa-Yks5sYohSgaJpZM4O0SDE
.

Wow that was fast :). Could you let me know when this is published? Thanks!

This is published.


hyphens are getting removed

all of the hyphens get's removed while converting
2015-10-04

becomes
20151004

What build are you using? I cannot reproduce in develop or master.

Yes that was issue with something else . Closing


Testnet send fails

So running nosetests will run tests/test_wallet.py which will try to send on testnet.
It makes this transaction:<PHONE_NUMBER>fccf6f80010410c7a11b1447801e61ce2c11dae0c60f6952ecfa9e4a4f6c2ff0010000006a47304402201030c0e1e59b799d2cd49bfec5ea86925537a1b28a7add6b29ba315dd6ec057802205068b5962ee57c77512984f67725967bb34e4adc25cab26e7e8325171d4fdc5f4121033d5c2875c9bd116875a71a5db64cffcb13396b163d039b1d9327824891804334ffffffff0210270000000000001976a914e7c1345fc8f87c68170b3aa798a956c2fe6a9eff88acfaa9b704000000001976a914990ef60d63b5b5964a1c2282061af45123e93fcb88ac00000000
Which looks valid when deserialized but Blockdozer says: Missing inputs. Code:-25
I tend to think it's a Blockdozer issue but will need to look into it more.
Broadcast TX on https://test-bch-insight.bitpay.com/ seems like it does absolutely nothing.

Hrmph.
https://bch-insight.bitpay.com/tx/send can broadast it fine but https://bccblock.info/tx/send cannot. Though it is a testnet transaction, I guess it's valid on both. Wonder if the cashaddress library is not translating correctly for testnet addresses.

I dont think bccblock does testnet, only mainnet.

So pushing it through a local bitcoind works fine:
$ bitcoin-cli -testnet sendrawtransaction "0100000001fccf6f80010410c7a11b1447801e61ce2c11dae0c60f6952ecfa9e4a4f6c2ff0010000006a47304402201030c0e1e59b799d2cd49bfec5ea86925537a1b28a7add6b29ba315dd6ec057802205068b5962ee57c77512984f67725967bb34e4adc25cab26e7e8325171d4fdc5f4121033d5c2875c9bd116875a71a5db64cffcb13396b163d039b1d9327824891804334ffffffff0210270000000000001976a914e7c1345fc8f87c68170b3aa798a956c2fe6a9eff88acfaa9b704000000001976a914990ef60d63b5b5964a1c2282061af45123e93fcb88ac00000000"
17b50bc1bb49e71a336664c95e61ef00defba28df4b787dfe6dc660c5c9139f5

Looking up that txid shows it got broadcast

Wow, no kidding. Seems like a bug in Insight.
Thanks for digging into that.

Ok, this does work:
curl https://test-bch-insight.bitpay.com/api/tx/send -d rawtx=0100000001fccf6f80010410c7a11b1447801e61ce2c11dae0c60f6952ecfa9e4a4f6c2ff0010000006a47304402201030c0e1e59b799d2cd49bfec5ea86925537a1b28a7add6b29ba315dd6ec057802205068b5962ee57c77512984f67725967bb34e4adc25cab26e7e8325171d4fdc5f4121033d5c2875c9bd116875a71a5db64cffcb13396b163d039b1d9327824891804334ffffffff0210270000000000001976a914e7c1345fc8f87c68170b3aa798a956c2fe6a9eff88acfaa9b704000000001976a914990ef60d63b5b5964a1c2282061af45123e93fcb88ac00000000

Problem seems unique to Blockdozer:
Missing inputs. Code:-25```

Meanwhile, check this out:

```curl -k https://tbcc.blockdozer.com/insight-api/tx/send -d rawtx=0100000001fccf6f80010410c7a11b1447801e61ce2c11dae0c60f6952ecfa9e4a4f6c2ff0010000006a47304402201030c0e1e59b799d2cd49bfec5ea86925537a1b28a7add6b29ba315dd6ec057802205068b5962ee57c77512984f67725967bb34e4adc25cab26e7e8325171d4fdc5f4121033d5c2875c9bd116875a71a5db64cffcb13396b163d039b1d9327824891804334ffffffff0210270000000000001976a914e7c1345fc8f87c68170b3aa798a956c2fe6a9eff88acfaa9b704000000001976a914990ef60d63b5b5964a1c2282061af45123e93fcb88ac00000000
transaction already in block chain. Code:-27```

If I force tbcc it works.

I sent a message to Blockdozer via the form on the website.
I should still add the Bitpay Insight API as well.

@teran-mckinney We fixed the issue on blockdozer.com (there was a bad configuration on our load balancer), our apologies for the inconvenients.. Let me know if you need something else.!
And btw, thanks for bringing this to our attention.

Thank you! No problem at all.
I appreciate you fixing this. I confirmed on my end now that the bitcash tests are passing.


Handle leader cluster disconnection scenarios during bootstrap recovery

Background
Cross Cluster Replication uses repository recovery mechanism of OpenSearch to copy the primary shard from leader cluster to follower cluster. It’s like a snapshot restore flow and we can check current status in cat/recovery output. We depend directly on the primary shard node for all file transfers, if that particular node goes down replication fails.
Failures :

Bootstrap fails if node having primary shard on leader cluster leaves the cluster.
Bootstrap fails if all nodes of primary cluster are out of service transiently.

Expected Behaviour

Bootstrap should be restarted if primary shard is available on any other node on leader cluster.
Bootstrap should be failed permanently after all of our retries are exhausted.

Pushed as part of 1.1 release.


Infer typeof self if unspecified on cast

Suggestion
🔍 Search Terms

type casting
implicit typing
syntactic sugar

✅ Viability Checklist
My suggestion meets these guidelines:

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

⭐ Suggestion
It would be a lot cleaner if when you cast a type to some other type (for example NonNullable) you don't have to specify typeof self.
📃 Motivating Example
if you have an Array that could be undefined you can filter out undefined like so
const array = ["one", undefined, "two"].filter(Boolean);
console.log(array) // ["one", "two"]

but in this case the type would still be (string | undefined)[] so it would cause linting errors later in the code. While this may be specific to the .filter method, it may be unreasonable to expect all methods accurately account for type checks like this. So one easy way to fix this would just be to do
const arrayUndefined = ["one", undefined, "two"]

const array = arrayUndefined.filter(Boolean) as NonNullable<UnArrayify<typeof arrayUndefined>>

what I'm suggesting is that instead of requiring you do typeof arrayUndefined you can instead do an implicit cast to make the code cleaner
const arrayUndefined = ["one", undefined, "two"]

// it's implied that UnArrayify would be called with <typeof array.filter(Boolean)>
const array = arrayUndefined.filter(Boolean) as NonNullable<UnArrayify>

💻 Use Cases
I would like this because it would result in cleaner typing for unhandled edgecases in the typescript linter

would result in cleaner typing

The clean approach would be to use a type guard and avoid the type assertion (aka "cast") entirely.
Perhaps you can provide a better example?
Also you checked that this wouldn't be a breaking change, but introducing a new global type could very well be breaking for anyone having such a type defined.

@MartinJohns what do you mean by global type? I guess I don't know how this would be implemented from the Typescript side. But I don't know how it would involve a global type.
What do you mean by type guard? Do you mind providing an example

here's a couple examples that may help explain what I mean
type CastTo<T> = T

const array = ["string"]

// currently..
const str = array[0] as CastTo<number>;
str -> number
const str = array[0] as CastTo; // expected required type T

// I propose
const str = array[0] as CastTo<number>;
str -> number
const str = array[0] as CastTo; // i.e. CastTo is seen as CastTo<typeof array[0]> to the linter
str -> string

also it's not always possible to define type as non-nullable without running a potentially null array through a filter to assure the contents are defined.
I think the main reason this may cause issues right now though is because people might not notice that the type their casting to is being called with the implicit type which would be confusing. So maybe this isn't a great idea 🤷♀

What do you mean by type guard? Do you mind providing an example

function isNotUndefined<T>(value: T | undefined): value is T {
  return value !== undefined;
}

const array1 = ["one", undefined, "two"].filter(Boolean); // type is (string | undefined)[]
const array2 = ["one", undefined, "two"].filter(isNotUndefined); // type is string[]

As proposed, this would be a breaking change for any generic type with a type parameter default.  For example, given type Foo<T = string> = ..., writing 123 as Foo is currently the same as writing 123 as Foo<string>, but you are suggesting it should be interpreted as 123 as Foo<123> instead?

@jcalz it would only apply to required types, Foo<T = string> is an optional type. So it wouldn't affect those as proposed.

Also I encountered another use case where I want to make a type mutable (using type-fest) but I have to first define the const and then redfine it to get that type.
My exact example is
const _context = (await import("github")).context
const context = _context as Mutable<context>

So my proposed change would instead be this
const context = (await import("github")).context as Mutable

You can achieve something like that trivially using a helper function without weird and confusing type shenanigans:
function asMutable<T>(value: T): Mutable<T> { return value; }

You can achieve something like that trivially using a helper function without weird and confusing type shenanigans:
function asMutable<T>(value: T): Mutable<T> { return value; }

const context = asMutable(await import("github")).context);

Yeah but it's still adding unnecessary complexity to my code. Which defeats the purpose

Checking in on this, I still think this would be a helpful addition.


Latest rerun and analysis

Description
Please include a summary of the changes.
Fixes # (issue)
Instructions for Reviewer
In order to test the code in this PR you need to ...
Please pay special attention to ...
Checklist:

[ ] I have refactored my code out from notebooks/
[ ] I have checked the code runs
[ ] I have tested the code
[ ] I have run pre-commit and addressed any issues not automatically fixed
[ ] I have merged any new changes from dev
[ ] I have documented the code

[ ] Major functions have docstrings
[ ] Appropriate information has been added to READMEs

[ ] I have explained this PR above
[ ] I have requested a code review

messed up the rebase, starting again in a new PR


Syntax error, unrecognized expression on checkboxes

Having an issue similar to this one: https://github.com/guillaumepotier/Parsley.js/issues/881
<input type="checkbox" name="QID30142||CHECKBOX||95494" id="QID30142||CHECKBOX||95494" />

This input gets a data-parsley-multiple="QID30142||CHECKBOX||95494" id="QID30142||CHECKBOX||95494" attribute added, resulting in this error:
Error: Syntax error, unrecognized expression: [data-parsley-multiple=QID30142||CHECKBOX||95492]
I was able to get around this by telling parsley to only look at the email and text inputs. I still got the error when trying to exclude the checkbox inputs though.

Which version are you using?

We are using 2.0.5 on this project.

I'm a bit surprised I have to point out the fact that this was resolved in 2.1.0+, as is written in the issue you already found.

Okay, thanks for the reply. Wasn't sure if that update took care of other characters or just spaces.


issue#111 Extract plant click listener logic from Adapter to Fragment

As explained on issue 111, the logic of the plant item click listener was implemented on PlantAdapter. The new changes are about extracting this logic from the PlantAdapter to PlantListFragment. Instead of creating an interface and passing it to adapter for using it as callback, the approach implemented make use of a lambda function.

I signed it!! 👍


Fix Undefined Symbol _ftelli64

Fix undefined symbol _ftelli64 when trying to compile with Raylib's Notepad++ & TCC on Windows.

@audinue Thanks for the fix! Please note that this fix should be submitted to original cglft repo or next time I update the library it will break again.


Is it possible to pass multiple model paths to sequelize.addModels() ?

I have several sub-apps and want to have my models organized per sub-app instead of in one models directory. Like, app1/models, app2/models, etc. The models are in the same database, but I just want to organize them with their related app. I tried,
sequelize.addModels(__dirname + 'path/to/app1/models/');
sequelize.addModels(__dirname + 'path/to/app2/models/');
and I get,
Error: Model not initialized: "Example" needs to be added to a Sequelize instance before association can be resolved.
There is an association between two models, but they happen to be in separate directories.

Hey @byebyebears, yes this is possible. Either with separate addModels calls like you did in your example or
sequelize.addModels([__dirname + 'path/to/app1/models/', __dirname + 'path/to/app2/models/']);

You can also use globs like sequelize.addModels([__dirname + 'path/to/*/models/*']);
Please note that models, which are related to each other, have to be added at once! For example if User is related to Group you cannot add them separately like sequelize.addModels([User]);
sequelize.addModels([Group]);butsequelize.addModels([User, Group]`) would do the job
Hope this helps

That was a key answer, since my dirname could not add the related models
i finally got it done, by adding each of the models once, because they were related
You can also use globs like sequelize.addModels([__dirname + 'path/to//models/']);
sequelize.addModels([Model1, Model2, Model3]) and it did the job!


Enable 'GetServiceEndpoint' to search through all given ingress names.

Today, GetServiceEndpoint will exit if the first given ingressName does not contain a valid endpoint, even though the second one does.
I noticed this when testing using Docker for Mac, where I have a defunct knative-ingressgateway (which is only there for compatibility reasons anyway).
This enables GetServiceEndpoint to search through the whole list to find something fitting.

/assign @adrcunha

/lgtm
/approve


Support for non-blocking JSON requests.

Where possible.  Some commands we don't respond any differently for success vs failure -- or simply don't care.  This speeds up the 'apparent' response time from Alexa quite a lot.  There is a small 'real' gain as well in that we're not waiting for the response from Kodi, though the responses for these are small anyway.
This includes:

All "direct plays" -- telling Kodi to play an item ID
Navigation commands -- go up, down, left, etc
Volume commands
Clean/update library

Problems connecting or transmitting the request will still raise an error.  This only applies to the JSON response from Kodi, since for the above things we don't report any differently to the user.  Might as well report earlier to avoid potential timeouts on the Alexa side of things.
A test package is up here for those that need/want it.

All merged :)


Resync watch resource if receive stop event

Summary
Fixes #

https://github.com/rancher/dashboard/issues/5997
Occurred changes and/or fixed issues

Call the dispatch('resyncWatch', obj)  instead of the dispatch('watch', obj);
Technical notes summary

The UI should not compare the resourceVersion of a collection with the resourceVersion of a collection element to get the maximum value as the watch's resourceVersion parameter, because the resourceVersion of a collection is not the same concept as the resourceVersion of a collection element, refer to the following Link:
https://kubernetes.io/docs/reference/using-api/api-concepts/#resourceversion-in-metadata
ui code:
https://github.com/rancher/dashboard/blob/master/shell/plugins/steve/subscribe.js#L604-L612

@rak-phillip
If the cluster is being created, the backend will push a stop message for the relevant resource(ui.cattle.io.navlink,namespace ,schema  ...), which will trigger resyncWatch and cause the js to report an error. I still need to check why this is happening.

@ly5156 thanks again for raising this PR and contributing to Rancher.
I'm closing this PR because we have a working resolution via rancher/steve#49. We still need to follow-up after reviewing our web socket implementation and caching strategy to properly resolve other issues identified in #5997.


Build issue when install Gulp-cordova-build

https://github.com/MichaelTaylor3D/gulp-cordova-bump
So I got this error when I was trying to install a package that requires libxmljs.
Figured I open an issue here and on the packages repo.
2015-04-09 18:04:03.678 xcodebuild[40174:696370] [MT] PluginLoading: Required plug-in compatibility UUID 9F75337B-21B4-4ADC-B558-F9CADF7073A7 for plug-in at path '~/Library/Application Support/Developer/Shared/Xcode/Plug-ins/Alcatraz.xcplugin' not present in DVTPlugInCompatibilityUUIDs
  CC(target) Release/obj.target/libxml/vendor/libxml/buf.o
  CC(target) Release/obj.target/libxml/vendor/libxml/catalog.o
  CC(target) Release/obj.target/libxml/vendor/libxml/chvalid.o
  CC(target) Release/obj.target/libxml/vendor/libxml/dict.o
  CC(target) Release/obj.target/libxml/vendor/libxml/encoding.o
  CC(target) Release/obj.target/libxml/vendor/libxml/entities.o
  CC(target) Release/obj.target/libxml/vendor/libxml/error.o
  CC(target) Release/obj.target/libxml/vendor/libxml/globals.o
  CC(target) Release/obj.target/libxml/vendor/libxml/hash.o
  CC(target) Release/obj.target/libxml/vendor/libxml/HTMLparser.o
  CC(target) Release/obj.target/libxml/vendor/libxml/HTMLtree.o
  CC(target) Release/obj.target/libxml/vendor/libxml/legacy.o
  CC(target) Release/obj.target/libxml/vendor/libxml/list.o
  CC(target) Release/obj.target/libxml/vendor/libxml/parser.o
../vendor/libxml/parser.c:13492:43: warning: passing 'const xmlChar *' (aka 'const unsigned char *') to parameter
      of type 'const char *' converts between pointers to integer types with different sign [-Wpointer-sign]
        hdlr = xmlFindCharEncodingHandler(doc->encoding);
                                          ^~~~~~~~~~~~~
../vendor/libxml/include/libxml/encoding.h:175:42: note: passing argument to parameter 'name' here
        xmlFindCharEncodingHandler      (const char *name);
                                                     ^
1 warning generated.
  CC(target) Release/obj.target/libxml/vendor/libxml/parserInternals.o
  CC(target) Release/obj.target/libxml/vendor/libxml/pattern.o
  CC(target) Release/obj.target/libxml/vendor/libxml/relaxng.o
  CC(target) Release/obj.target/libxml/vendor/libxml/SAX2.o
  CC(target) Release/obj.target/libxml/vendor/libxml/SAX.o
  CC(target) Release/obj.target/libxml/vendor/libxml/tree.o
  CC(target) Release/obj.target/libxml/vendor/libxml/threads.o
  CC(target) Release/obj.target/libxml/vendor/libxml/uri.o
  CC(target) Release/obj.target/libxml/vendor/libxml/valid.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xinclude.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xlink.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xmlIO.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xmlmemory.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xmlmodule.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xmlreader.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xmlregexp.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xmlsave.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xmlschemas.o
../vendor/libxml/xmlschemas.c:24203:10: warning: explicitly assigning value of variable of type 'xmlSchemaValType'
      to itself [-Wself-assign]
        valType = valType;
        ~~~~~~~ ^ ~~~~~~~
1 warning generated.
  CC(target) Release/obj.target/libxml/vendor/libxml/xmlschemastypes.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xmlstring.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xmlunicode.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xmlwriter.o
  CC(target) Release/obj.target/libxml/vendor/libxml/xpath.o
../vendor/libxml/xpath.c:12402:13: warning: comparison of constant 'XML_NAMESPACE_DECL' (18) with expression of
      type 'xmlXPathTypeVal' is always false [-Wtautological-constant-out-of-range-compare]
                        if (type == XML_NAMESPACE_DECL)
                            ~~~~ ^  ~~~~~~~~~~~~~~~~~~
In file included from ../vendor/libxml/xpath.c:185:
../vendor/libxml/trionan.c:194:1: warning: unused function 'trio_is_special_quantity' [-Wunused-function]
trio_is_special_quantity
^
../vendor/libxml/trionan.c:218:1: warning: unused function 'trio_is_negative' [-Wunused-function]
trio_is_negative
^
3 warnings generated.
  CC(target) Release/obj.target/libxml/vendor/libxml/xpointer.o
  LIBTOOL-STATIC Release/xml.a
  CXX(target) Release/obj.target/xmljs/src/libxmljs.o
In file included from ../src/libxmljs.cc:7:
In file included from ../src/libxmljs.h:7:
In file included from ../node_modules/nan/nan.h:63:
../node_modules/nan/nan_new.h:207:43: error: inline declaration of 'NanNew' follows non-inline definition
template <typename T> inline v8::Local<T> NanNew(v8::Handle<T> h);
                                          ^
../node_modules/nan/nan_implementation_pre_12_inl.h:210:1: note: previous definition is here
NanNew(v8::Handle<T> h) {
^
In file included from ../src/libxmljs.cc:7:
In file included from ../src/libxmljs.h:7:
In file included from ../node_modules/nan/nan.h:63:
../node_modules/nan/nan_new.h:208:43: error: inline declaration of 'NanNew' follows non-inline definition
template <typename T> inline v8::Local<T> NanNew(v8::Persistent<T> const& p);
                                          ^
../node_modules/nan/nan_implementation_pre_12_inl.h:216:1: note: previous definition is here
NanNew(v8::Persistent<T> const& p) {
^
2 errors generated.
make: *** [Release/obj.target/xmljs/src/libxmljs.o] Error 1
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:269:23)
gyp ERR! stack     at ChildProcess.emit (events.js:98:17)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (child_process.js:820:12)
gyp ERR! System Darwin 14.3.0
gyp ERR! command "node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /Users/mhartington/gulp-cordova-bump/node_modules/gulp-xml-editor/node_modules/libxmljs
gyp ERR! node -v v0.10.38
gyp ERR! node-gyp -v v1.0.3
gyp ERR! not ok
npm ERR! Darwin 14.3.0
npm ERR! argv "node" "/usr/local/bin/npm" "install"
npm ERR! node v0.10.38
npm ERR! npm  v2.7.5
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the libxmljs package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls libxmljs
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/mhartington/gulp-cordova-bump/npm-debug.log

The full npm output is available here
https://gist.github.com/mhartington/e1fa2c107c53d499f0bd
OSX 10.10.3
Node 0.10.38
What other information do you need?

Those look like NAN errors to me. ping @rvagg

Seconded...
In my case libxmljs is a dependency on another library. Installation of 0.13.0 fails with the above, but 0.12.0  is ok.
OSX Yosemite
NPM 2.7.5
Node 0.10.38
Python 2.7.9

Which GCC/clang?

$ gcc -v
Configured with: --prefix=/Applications/Xcode.app/Contents/Developer/usr --with-gxx-include-dir=/usr/include/c++/4.2.1
Apple LLVM version 6.1.0 (clang-602.0.49) (based on LLVM 3.6.0svn)
Target: x86_64-apple-darwin14.1.0
Thread model: posix

Can confirm:
npm install<EMAIL_ADDRESS>npm install libxmljs

Fails at
  CC(target) Release/obj.target/libxml/vendor/libxml/xpath.o
../vendor/libxml/xpath.c:12402:13: warning: comparison of constant 'XML_NAMESPACE_DECL' (18) with
      expression of type 'xmlXPathTypeVal' is always false
      [-Wtautological-constant-out-of-range-compare]
                        if (type == XML_NAMESPACE_DECL)
                            ~~~~ ^  ~~~~~~~~~~~~~~~~~~
In file included from ../vendor/libxml/xpath.c:185:
../vendor/libxml/trionan.c:194:1: warning: unused function 'trio_is_special_quantity'
      [-Wunused-function]
trio_is_special_quantity
^
../vendor/libxml/trionan.c:218:1: warning: unused function 'trio_is_negative' [-Wunused-function]
trio_is_negative
^
3 warnings generated.
  CC(target) Release/obj.target/libxml/vendor/libxml/xpointer.o
  LIBTOOL-STATIC Release/xml.a
  CXX(target) Release/obj.target/xmljs/src/libxmljs.o
In file included from ../src/libxmljs.cc:7:
In file included from ../src/libxmljs.h:7:
In file included from ../node_modules/nan/nan.h:63:
../node_modules/nan/nan_new.h:207:43: error: inline declaration of 'NanNew' follows non-inline
      definition
template <typename T> inline v8::Local<T> NanNew(v8::Handle<T> h);
                                          ^
../node_modules/nan/nan_implementation_pre_12_inl.h:210:1: note: previous definition is here
NanNew(v8::Handle<T> h) {
^
In file included from ../src/libxmljs.cc:7:
In file included from ../src/libxmljs.h:7:
In file included from ../node_modules/nan/nan.h:63:
../node_modules/nan/nan_new.h:208:43: error: inline declaration of 'NanNew' follows non-inline
      definition
template <typename T> inline v8::Local<T> NanNew(v8::Persistent<T> const& p);
                                          ^
../node_modules/nan/nan_implementation_pre_12_inl.h:216:1: note: previous definition is here
NanNew(v8::Persistent<T> const& p) {
^
2 errors generated.
make: *** [Release/obj.target/xmljs/src/libxmljs.o] Error 1

And master works, with nan 1.7.0.

@polotek @defunctzombie Looks like you just need to publish out a new version to npm.
The code here in master works fine.

Getting this here as well. Temporary work around is locking libxmljs as mentioned to 0.12.0.

^ can confirm, locking it is the way to handle this error temporarily.

+1

new version was published as 0.14.0

I had same problem. Just try command "npm install libxmljs" inside your project with package.json. And then it will install gulp and another packages. after this command try to "gulp" in place where is your gulpfile.js is.


Add nested flag for read_dict_from_hdf()

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

5 of 19 (26.32%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-2.3%) to 91.829%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

h5io_browser/base.py
5
19
26.32%

Totals

Change from base Build<PHONE_NUMBER>:
-2.3%

Covered Lines:
517

Relevant Lines:
563

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

5 of 19 (26.32%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-2.3%) to 91.829%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

h5io_browser/base.py
5
19
26.32%

Totals

Change from base Build<PHONE_NUMBER>:
-2.3%

Covered Lines:
517

Relevant Lines:
563

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

21 of 21 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 94.326%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
532

Relevant Lines:
564

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

20 of 20 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 94.326%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
532

Relevant Lines:
564

💛  - Coveralls


The links added to the utter template action is not displaying as link in slack channel

Rasa version: rasa==1.0.1
Python version: Python 3.6.7
Operating system (windows, osx, ...):
Distributor ID: Ubuntu
Description: Ubuntu 18.04.2 LTS
Release: 18.04
Codename: bionic
Issue:
If you add a template utter action with a link in the text, the link is displaying as plain text in the slack channel. This was showing correctly as hyperlink in the rasa_core version.
For Ex:
templates:
  utter_rasa_url:
    - text: "Hey, this is the url of rasa \n  https://github.com/RasaHQ/rasa/issues"

Kindly note that I have to display it as two lines. The first line will only display the text "Hey, this is the url of rasa"  and the second line will display the url "https://github.com/RasaHQ/rasa/issues". In this case, the url should actually be displayed as hyperlink.
I am using slack integration and this was working correctly, if you check the rasa_core 0.14 versions. Please let me know, if this is a configuration issue only.

Thanks for raising this issue, @erohmensing will get back to you about it soon.

Not a configuration issue, I can see where this is going wrong, it's another block thing 😅Thanks!

yes found that the block is using "plain_text" type. Thanks for fixing it  :)

In btw, it will be really helpful, if you could answer a question which I have asked in the forum. Expecting your help on this case. Thanks in adv :)


Move all HC conditions into its own file

What this PR does / why we need it:
This is a follow up for
https://github.com/openshift/hypershift/pull/1853
https://github.com/openshift/hypershift/pull/1852
https://github.com/openshift/hypershift/pull/1851
To continue to be more consistent on the conditions between HC and HCP, reasons and messages. This puts all HC conditions together in a single file for easier access and visibility.
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/lgtm

/retest

/lgtm

/retest-required

/test e2e-aws

/test e2e-aws

/test e2e-aws

/hold cancel


[Refactor] Standalone library

Do you think it would be possible to separate the Kodi-specific parts so that this implementation would be usable as a standalone library, for example to use with mpv or youtube-dl ?
I just had a quick look in the resources/lib/ directory, but I am not familiar enough with the internal of Kodi plugins to have an idea if it is easily possible or not.

With the https://github.com/asciidisco/plugin.video.netflix/tree/chore/code-trim/resources/lib branch (that I haven't really found the time working on lately) I planned to factor out 2 libraries out of the plugin.

Some kind of generic framework that deals with all Kodi (18) related things & could be reused for other plugins
A library that contains all Netflix API related things (most probably the thing you want)

But (I don't know a thing about mpv or youtube-dl, so take this with a grain of salt), without something like https://github.com/peak3d/inputstream.adaptive build for those tools, you probably wouldn't have much luck playing a video. That library actually does the hard work of interfacing with the DRM decoder & decrypter to basically give you a proper video stream to watch.
Also, youtube-dl looks like a tool that is mainly used to download videos, not saying that this is impossible for DRMed content like you get from Netflix, but it probably is not legal to do so. I´m not saying you´re after that specifically, but I wouldn't support something like this. With this plugin we´re trying to stay within legal bounds as much as possible, which also means respecting the DRM protection of the content. Not trying to be harsh, or put words in your mouth, so please don't get me wrong, I just wanted to clarify that.

Thanks for your detailed answer (the notification got lost somewhere and I am reading it only today).
I was not aware of the existence of inputstream.adaptive and of the work done by this component, I will have a look.
Concerning your legal concerns, youtube-dl is not anymore only a download tool but also a streaming client used as a backend for other media player (such as mpv). I don't know the position of netflix about third party clients but the problem may be also the existence of the --dump-video flag of mpv (or equivalent features of any player). Anyway, as soon as an libre part is involded somewhere, the drm are fucked-up...
I personally do not want to download anything, it is the exact contrary: I would be happy to not download anything and to stream the netflix contents to my client of choice.


[2.x] {TASKID} returns task uuid instead of ID

Hello,
I noticed that {TASKID} was being replaced by the uuid of the task, and not the ID. Is this by design? Maybe we could TASKID be the ID and TASKUUID be the uuid just to prevent confusion?

This is leftover syntax from 1.x, and I agree this is an issue we need to address. I have a different proposal though:
We eliminate {TASKID} and replace it with {TASK_UUID}. Reasons why:

Task IDs change, while UUIDs are consistent
Any command that takes a task ID also takes a task UUID
Supporting both {TASKID} and {TASK_UUID} could be potentially confusing, and result in more support requests.

Thoughts?

Yes, that sounds good!

Sounds good to me too. I can't think of a situation where relying on the task ID would be preferred to relying on the task UUID.


Store data on how many users we have and how many times they logged in

How many times they logged in = how many times they accessed the login page and 1. either logged in for real, or 2. got redirected to home page since they have already logged in previously.
Active user = logged in 2 times per week? Think about how to measure later.

Implemented.


STORM-3105 Update hive.version to stable hive release.

STORM-3105

cc: @roshannaik

This actually require full tests on storm-hive as well as storm-autocreds (delegation token). Could you share how do you test the patch manually?

In our case, we have copied the hive dependencies into our pom with the right version and can confirm that it works. I see that storm-hive already has some tests.
The point here is that hive.version should match the hive version being used by the user. Setting it here is just a default and the default may not work if there's a version mismatch with the actual hive version. To address that I have opened STORM-3106

The current build is failing because of an Enforcer rule.
...
[INFO] --- maven-enforcer-plugin:1.4.1:enforce (enforce-maven-version) @ storm-hive ---
[WARNING] Rule 1: org.apache.maven.plugins.enforcer.BannedDependencies failed with message:
Found Banned Dependency: jdk.tools:jdk.tools:jar:1.8
Use 'mvn dependency:tree' to locate the source of the banned dependencies.
...
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-enforcer-plugin:1.4.1:enforce (enforce-maven-version) on project storm-hive: Some Enforcer rules have failed. Look above for specific messages explaining why the rule failed. -> [Help 1]
...

Closing, see reasoning at<EMAIL_ADDRESS>


Exception in instrumenting Spring Cloud Stream with Solace Binder

Hi all,
In my spring boot app, I used Spring Cloud Stream (with Solace binder), and tried to integrate with Open Tracing API implementation for Spring Cloud (which supports Spring Cloud Stream) but failed with the stack trace as below, probably due to incompatible data type (not supported by Solace SDTMap) of Open API info in message header.
My question is if I can DISABLE the instrumentation related to Spring Cloud Stream, such that I can have the transaction complete normally, and able to keep using OpenTracing API for most of instrumentation.
Any advice is welcome!
Thanks
[account-web-v1-7ff588d759-hbqqr account] 2020-05-13 09:21:06.610 ERROR 1 --- [ask-scheduler-6] o.s.integration.handler.LoggingHandler   : org.springframework.messaging.MessageHandlingException: error occurred in message handler [org.springframework.cloud.stream.binder.AbstractMessageChannelBinder$SendingHandler@31c62948]; nested exception is com.solace.spring.cloud.stream.binder.util.SolaceMessageConversionException: java.lang.IllegalArgumentException: Invalid type as value - ThreadLocalScope, failedMessage=GenericMessage [payload=byte[14], headers={messageSent=true, id=0ecc2e6c-6585-a781-00db-d613a97bb696, contentType=application/json, io.opentracing.contrib.spring.integration.messaging.OpenTracingChannelInterceptor.SCOPE=io.opentracing.util.ThreadLocalScope@72c13964, uber-trace-id=b69af8c09af7a43f:b69af8c09af7a43f:0:1, timestamp=1589361666604}]

The instrumentation is loaded by this auto configuration https://github.com/opentracing-contrib/java-spring-messaging/blob/e57ad163b8640dd8362a200cd835f83746d9608c/opentracing-spring-messaging-starter/src/main/java/io/opentracing/contrib/spring/integration/messaging/OpenTracingChannelInterceptorAutoConfiguration.java#L29.
Normally our auto instrumentation have a property that can be used to disable it, however, this does not have it. Then the auto-configuration can be disabled by excluding jar or via https://www.baeldung.com/spring-data-disable-auto-config#disable-using-annotation.
I am moving this to java-spring-messaging repository. Would you like to submit a PR to fix it?


[feature request] handle contents failing schema validation, or autofixing

Hi I'm new to velite, and in my use case, I plan to assign a random ID to each markdown file in frontmatter, and prefer the generation is automatic: The file is created without ID and then the ID is written to file after saving.
I wanted to make use of prepare velite config to autofix this but looks like the data available in this hook are all contents that pass the schema validation and those who fail are dropped.
So I'm asking if we could add a new hook or some mechanism to allow fixing validation errors on the fly in velite (watch mode maybe). Here's some thoughts

this hook should have access to the files that did not pass the validation and respetively the reason(s)
this hook should have access to the files that passed the validation because they might be required in the fix (my use case is an example, ID should be unique)
we might need a physical path schema to ease the file writing

If this request is acceptable, I can also help if I find some spare time

First of all, thank you for your participation,
I want to confirm your needs first.
Do you want Velite to automatically add some fields such as id to each entity when working, and this process is programmable?
Maybe the transform and superRefine of zod schema can realize your idea.
const plans = {
  name: 'Plan',
  pattern: 'plans/*.md',
  schema: s.object({
    name: s.string(),
    slug: s.slug('plan'),
    icon: s.string(),
    prices: s.object({ yearly: s.number(), monthly: s.number() }),
    description: paragraph,
    featured: s.boolean().default(false),
    benefits: s.array(s.string())
  }).transform(data => ({ ...data, id: uuid() }))
}

If it is not what I understand, can you provide a pseudo code to help me understand your needs

Thanks @zce
I'd like to have a persistent/stable ID for each markdown document, which means the ID should be written to the file the first time it is generated. I feel the transform would produce a random ID each time velite runs, and the ID will change next time. please let me know if I'm wrong.

Looking at the transform and superRefine that you mentioned, I realize I am able to generate the ID then write it to the original file and finally return the document with the key, I'll give it a try, and I'm not sure if this is the best practice for zod

Result:
I was using s.unique() for ID, and turns out the document is dropped at unique() validation, so superRefine, transform or prepare would not have chance to fix the file.
Changing to s.optional(s.unique()) could allow the validation pass and the missing data issue could be fixed by myself in the above places

https://github.com/zce/velite/issues/108#issuecomment-2041738210

Thanks for the information. Let me explain my thoughts, ID id not only used to distinct documents.
There do exist a need behind this constant/persistent ID, for example, creating a short URL so that it could be shared more easily, general hashing algorithm usually outputs a long ID as they are designed to reduce collision possibilities
Also if we use hash of the title as ID, then if we mofidy the title, the ID would change, which would confuse people who saved the URL or added it to browser favorites.

I totally agree that ensuring a unique ID does not have to happen inside velite, we might use git hooks, validation tests in CI, etc to handle that. I just feel it is more suitable to be triggered by content generation because we have all content data together as well as validaton failure issues. Maybe this ID issue is not a proper case

As my initial requests could be nearly fulfilled (the imperfect part is the id becomes optional in typing), I'm closing this issue. Thanks for you help.

As you mentioned, this is the reason why Velite doesn't assertively add an ID for each item.
Honestly, I believe that hardcoding is the most appropriate approach for IDs. If an ID is required, each data should be assigned a constant and unchanging ID at the time of creation.


First argument should be of type Context

A transaction function must have at least one argument, which is of type Context.
Originally, if a transaction function has 0 arguments, the runtime exists. Now with this PR, the runtime gives some cues.

/azp-run


Revert  sorting order

Pull Request Description

Important Notes

Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[ ] The documentation has been updated, if necessary.
[ ] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[ ] All code follows the
Scala,
Java,
TypeScript,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
[ ] Unit tests have been written where possible.
[ ] If meaningful changes were made to logic or tests affecting Enso Cloud integration in the libraries,
or the Snowflake database integration, a run of the Extra Tests has been scheduled.

If applicable, it is suggested to paste a link to a successful run of the Extra Tests.

@MrFlashAccount I know its revert so lets save the comments for feature PR


Text associated with MGRS is not parallel to the latitude lines.

I have attached a screen capture along with camera settings.

To test use branch #97-Text-mgrs-is-not-parallel

Run test basic and select setting / map grid lines / MGRS from the menu.
The map grid lines will appear.
Change the heading, roll and tilt of the camera.
The Text of the gird lines should remain aligned to parallel lines.

reintegrate


Jinja 2.9 causes lamvery to fail

When I use Jinja 2.9 in my own project (i.e. with the requirements.txt shown below) it appears to cause problems when running lamvery deploy - see ouput below.
Removing the requirements.txt line or setting it to 2.8.1 makes the error disappear.
requirements.txt snippet:
Jinja2==2.9

output of lamvery deploy:
(.venv) EricMBP:lambda-prototype eric$ lamvery deploy -a test
  File "/Users/eric/sources/lambda-prototype/.venv/lib/python2.7/site-packages/jinja2/asyncfilters.py", line 7
    async def auto_to_seq(value):
            ^
SyntaxError: invalid syntax

ERROR:root:[Errno 2] No such file or directory: '/Users/eric/sources/lambda-prototype/.venv/lib/python2.7/site-packages/jinja2/asyncfilters.pyc'
Traceback (most recent call last):
  File "/Users/eric/sources/lambda-prototype/.venv/lib/python2.7/site-packages/lamvery/cli.py", line 336, in main
    args.func(args)
  File "/Users/eric/sources/lambda-prototype/.venv/lib/python2.7/site-packages/lamvery/cli.py", line 39, in deploy
    DeployAction(args).action()
  File "/Users/eric/sources/lambda-prototype/.venv/lib/python2.7/site-packages/lamvery/actions/deploy.py", line 43, in action
    zipfile = builder.build()
  File "/Users/eric/sources/lambda-prototype/.venv/lib/python2.7/site-packages/lamvery/build.py", line 68, in build
    self._archive_dir(zipfile, p)
  File "/Users/eric/sources/lambda-prototype/.venv/lib/python2.7/site-packages/lamvery/build.py", line 117, in _archive_dir
    zipfile.writepy(path)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/zipfile.py", line 1418, in writepy
    self.write(fname, arcname)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/zipfile.py", line 1123, in write
    st = os.stat(filename)
OSError: [Errno 2] No such file or directory: '/Users/eric/sources/lambda-prototype/.venv/lib/python2.7/site-packages/jinja2/asyncfilters.pyc'
[Errno 2] No such file or directory: '/Users/eric/sources/lambda-prototype/.venv/lib/python2.7/site-packages/jinja2/asyncfilters.pyc'

It's probably a pip/jinja compatability problem.
https://github.com/marcy-terui/lamvery/pull/69


support GRU op

and some bug fix

@xysmlx you can use this branch and gru_bidirectional.onnx to debug blockfusion.


Vulnerability found in Nswag library using Mend Bolt

Hi,
we can see Vulnerability in Nswag library using Mend Bolt.
We have tried the latest version  -13.8 of Nwag but same issue.
Our application is targeting .net6 Can you guys please check which version of nswag code is targeting itself.

Hi @RicoSuter can you please suggest


Increase windows container build timeout to 120m

Reason for Change:

Issue Fixed:

Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests

Notes:

Let's link the incident or work item which invoked this change in the description.


Design and implement type system in logical layer

As part of #70, we planned to have a separate type system on the physical layer and the logical layer. While the former deals with low-level types like u32 or floats, the latter deals with more abstract concepts like "String", "Integer" and so on. The task is to implement this functionality.
Some more considerations:

This would require to have a translation from the logical type into a TrieSchemaElement which currently represents the type in the physical layer
Some type analysis on the program level is necessary. E.g. Joins can only be performed on columns of the same logical type. This would require an implementation of type propagation through a predicate position graph
One should think about having a uniform system of representing values. Currently the CSV-loading code for example is independent from the rule parsing code.
Type declarations should be part of the rule language: For example in the @source statements
Users should be allowed to not specify a type. This would then be automatically deduced while loading the files. This could be tricky because of the type analysis that needs to be performed (E.g. one might find that constraints of the type analysis are violated while loading the file).
Users should be allowed to overspecify a type. E.g. a user might say that a column should be of type Integer(32) even though we might not differentiate between integers of different sizes in the logical layer

Related discussion: #185

A basic type system with Any, String, Integer and Float64 has been implemented.
Since there is no type casting when reasoning yet, only Any and String are compatible because they happen to have the same physical representation.
Still, since this marks a basic milestone of the type system development, I hereby close this issue.
We should open new issues for open points and use the existing discussion for clarifying additional questions.


OpTestSystem: python3 encode sock.send

Need to encode the string for socket.
Signed-off-by: Deb McLemore<EMAIL_ADDRESS>

Can you include what you did to hit the bug in the commit message? InstallUbuntu test?


Regex /Not-Regex rule is not working

I used the regex rule as mentioned in the documentation but it throws an error as shown in the attached screenshot

I tried to find out the reason and I might be wrong but I think the reason is .... ok I will explain with the help of screenshot attached below

It was expected to be working like the one, underlined with green colour, but in the regex/not_regex function, the parameters[0] (underlined with red colour) gives string value( e.g "/^.+@.+$/i"). And of course, string.test is not the function. So it gives the error.
If I have misunderstood things, please let me know. Also, Is there any other solutions to work with Regex ?

Hey @sudip571, this is the wrong repo for this issue.
The validation rules have their own repo to keep the tests and codebase much more concise. Please move the contents of this issue over here ~ https://github.com/zhorton34/vuejs-validators/issues
Closing this issue out on this repository as the code that you're referencing is not fixable within this vuejs-form code base.


[PROMOCODE] As an admin, I want promo code requests to be automatically provisioned from the server

As an admin, I want promo code requests to be automatically provisioned from the server so that I don't have to use a ledger and the private key for gas can be placed on a server.
Acceptance Criteria

Create a Promocode admin key to hold gas for promocode provisioning
Update nodesale contracts so the promocode key only used for promo gas
Update promocode provisioning so that the key is on the server when customer requests promo code and no manual intervention will be needed.

This Promocode admin key should be able control the create, edit, and delete of promocodes and nothing else.

Closing as not needed.


Add *.bak to .gitignore

This just makes my life easier. It's not related to the issues in my other current PRs.

Closing this to re-create it in a way that results in CircleCI running on it.


Running docker in Windows

I try to run pymarl in a docker container on a windows pc. Besides from adjusting the run.sh to:
#!/bin/bash
HASH=$(cat /dev/urandom | tr -dc 'a-zA-Z0-9' | fold -w 4 | head -n 1)
GPU=$1
name=$(whoami)_pymarl_GPU_${GPU}_${HASH}

echo "Launching container named '${name}' on GPU '${GPU}'"
# Launches a docker container using our image, and runs the provided command

if hash nvidia-docker 2>/dev/null; then
  cmd=nvidia-docker
else
  cmd=docker
fi

NV_GPU="$GPU" ${cmd} run \
    --name $name \
    --user $(id -u):$(id -g) \
    -v /$(pwd):/pymarl \
    -t pymarl:1.0 \
    ${@:2}

I receive an error from OpenGL

Attempting to initialize EGL from file libEGL.so ...
Failed to find EGL functions in library file!

on execution of command:

[INFO 18:15:22] absl Launching SC2: /pymarl/3rdparty/StarCraftII/Versions/Base69232/SC2_x64 -listen <IP_ADDRESS> -port 15923 -dataDir /pymarl/3rdparty/StarCraftII/ -tempDir /tmp/sc-and62dp2/ -eglpath libEGL.so

I am not very familiar with docker but it looks like the docker image is missing support for headless opengl rendering which does not cause problems on Ubuntu16.04 systems.
Am i doing somethin wrong is there indeed no support for windows in the current image?
PS: could using this dockerfile as a base help: https://hub.docker.com/r/nvidia/cudagl ?

I got it working. I executed the file from inside the Docker without any change.

We haven't tested the dockerfile/code on Windows.
This looks to be a StarCraft 2 issue, but I don't have a nice way to debug/fix it sorry!


meta: make workflows only run on Python file change

Turns out it is not possible without some hack so I give up


dotLess does not seem to work at all in ASP.NET 4.5

Hi there!
I'm just now getting my feet wet with dotLess in another project and am loving it thus far.  This past weekend though, I cracked open the new Visual Web Developer 11 Beta to play around with a few things and discovered that dotless doesn't seem to work at at all there.  I was first trying to use it with a custom Bundle and the new native bundling/minification system in MVC 4, and that didn't work.  So I then tried to just use it normally and sadly got the same result.  All of the normal css is read correctly, but the less pieces (such as mixins and variables) are totally ignored.
Am I missing something, or is this a known problem in that dotLess just isn't compatible with ASP.NET 4.5 and MVC 4?  Thanks again!

You might be the first or second to try it. If you do come up with any tips or a handler for asp.net 4.5 / MVC it would be nice to include it.
I imagine that writing a new handler that works like the old one but is good for asp.net 4.5 should be simple.


Create a basic Admin Reporting page

What are we trying to do?
Create the MVP of the admin reporting page outlined below!
For the MVP:

Create the page available to admin users
have it show the three numbers for the current line/week

Out of scope!

changing the date window
changing the line
personal summary

What feature or behavior is this required for?
Eventual goal!

How could we solve this issue? (Not knowing is okay!)
Creating a new view, and figuring out where to best store the logic of our numbers.
Anything else?
Design & thinking happened over in #634

we're on postgres now and I frankly bet that we can do this thru mode analytics or a similar platform. Let's see.


strange behavior

hi
I have used datepicker , when I open and select a date to insert, the values are in jalaali date,
then i open edit form and save . the dates are in jalaali
but when I open edit form, and change the date, date value is in Gregorian.
why its behavior is like that ?

Please make a demo
I didn't understand the problem but I think you might better look at jalali-moment


Added a dockerfile for ubuntu20.04 with dependencies of webserver agent

More changes to build in ubuntu would be added in other PRs

Would it make more sense to put the docker folder inside the instrumentation/otel-webserver-module folder? The current path name gives no indication to the user that it is for otel-webserver-module.

Thanks for pointing this out. It should be insided instrumentation/otel-webserver-module'


Diagonal elements of the distance matrix are not 0 when w != 1

When the parameter w (lambda in the original paper) is not equal to 1, the distances of the elements to themselves are not equal to 0. They are quite small but still cause errors when trying to use the distance matrix in other functions (e.g. scipy.cluster.hierarchy.linkage).
Here is the simple code to reproduce this using the toy data provided in the package:
import WassersteinTSNE as WT
import numpy as np
import matplotlib.pyplot as plt

dataset, HGM = WT.ToyDataset()
Gaussians    = WT.Dataset2Gaussians(dataset)
GWD          = WT.GaussianWassersteinDistance(Gaussians)

ws = np.linspace(0, 1)
traces = [np.trace(GWD.matrix(w=w)) for w in ws]

plt.plot(ws, traces)
plt.xlabel("w")
plt.ylabel("Sum of diagonal elements")

Result:

I believe, this issue is numeric, and propose setting diagonal elements to 0 before returning the distances matrix. This will help potential users to avoid problems

Hi Vladimir,
thanks for opening this issue. You are right, this was due to line 98 in Distances.py
return matrix - matrix.min()
which I replaced now (after renaming 'matrix' with 'K') by
np.fill_diagonal(K, 0)
return K
This solves the problem, so I'm closing this issue.


Fix for Read The Docs Documentation Page

This PR has fixes for the ReadTheDocs Partitura website:

Moved docs files to /docs/source;
Added requirement for docs;
Added .readthedocs with some configuration;
Corrected API path;
Added to example krn and mei to package data.

Codecov Report
Base: 80.59% // Head: 80.59% // No change to project coverage :thumbsup:

Coverage data is based on head (668a51f) compared to base (d1bee81).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #156   +/-   ##
========================================
  Coverage    80.59%   80.59%
========================================
  Files           63       63
  Lines        10278    10278
========================================
  Hits          8284     8284
  Misses        1994     1994

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

If the PR is merged and the Branch deleted consider redirecting latest to valid documentation at ReadTheDocs.

This PR fixes the documentation generation.


Update dependencies and refactor

I have this tested and working on my own cloud
Resolves #26

Hi @innovate-invent, we are overhauling the driver in https://github.com/cernops/cvmfs-csi/pull/39. The PR also updates the dependencies, so feel free to close this one, or let us know you notice something missing. Sorry for the inconvenience!

Hi @innovate-invent, we have released v2.0.0. I'm closing this PR as it is out-of-date.


Indent Formatting Option

Add ability to customize the formating. Would be nice to choose between a tab indent or double spaces.

It takes indent settings from npp settings. Try it once and let me know if it works.

It honors notepad++ settings.


IOS CDP Neighbors collector fails when neighbor does not have capabilities

Resolved in commit https://github.com/InsightSSG/Net-Manage/commit/9038e835f9053b3fd0d445deb7689e36174d0cde


Provide a middleware for Data Tracker Chain

It is helpful to have a middleware that a RESTful service can implement to ensure a Data Tracker Chain is sent in the HTTP header and is a valid chain.

close #21


Add views counter

On a different database

added in json api

used only for analytics...not shown right now....


FileNotFoundError

when i run restore.py in my windows, i found bug "FileNotFoundError: [Errno 2] No such file or directory: './checkpoints\MPM_stripe_1\latest_net_G_A.pth'"
i tried to use docker in ubuntu 20.04 and found the same error "FileNotFoundError: [Errno 2] No such file or directory: './checkpoints/MPM_stripe_1/latest_net_G_A.pth'"
i cannot generate 'latest_net_G_A.pth' ,how should i do?
thank you!

The "latest_net_G_A.pth" file can only be generated after training is completed. You need to first run the sampling script to sample patches and thus create the training set. Then you can train the model based on the sampled patches.


feat(fastlane-env): Add secrets archive and deploy script. Improve CodePush handling.

Hi, thanks for submitting a Pull Request! :balloon:
Before submitting, please be sure to check a few things:
Commit names formatting
This repository is automatically published to npm with
semantic-release.

Thanks!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


new control buttons: stop, back, forward

Writing sorting algorithms is easier and more suitable now. All you have to do is to call the sorting function within the record method. You can focus on sorting a simple array rather than an array of towers, the animation object will automatically record and translate the sorting operation to an animation and allow you to play it on command.. You can then play it on demand.

Nice work!


Publish a version for 2.12.7

I believe it's also necessary to update the ci build in order to be able for a new version to be published for
https://github.com/lihaoyi/Ammonite/issues/870

Thanks. I'm not sure what this script is for, the latest version already depends on 2.12.7 both on Central (1.2.1-12-bc459e3) and from the GitHub release page (2.12-1.2.1-12-bc459e3). The version was bumped in build.sc at the root.

Oh I did not realize that the unstable build had already been published! The README mentions that the scripts in the ci/ folder controlled publishing so I thought this change was also necessary.


Feature request - dataKey and categories inference

Right now, the data has an any[] type, dataKey has a string type, and categories string[] type. And if the values passed to the accessor props aren't keys of data then the chart simply won't render without throwing an error. So, it would be nice to have those accessors be actual keys of the type of data type and see errors if they aren't.

hey @dmytro-tinybird, thanks a lot for the PR! Will have a look as soon as I have a bit more time :)


[MouseJump]Also simulate mouse input event

Summary of the Pull Request
Using Mouse Jump with Mouse Crosshair turned on, the Crosshair will lag behind when jumping.
This is because we're only moving the cursor but not doing any mouse input.
This PR simulates mouse input so that software that is listening to it responds accurately.

PR Checklist

[x] Closes: #24523
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Validation Steps Performed
With Mouse Pointer Crosshairs activated, configured 3 screens with different DPI values and verified jumps are still accurate while the crosshairs still follows the mouse around accurately.

@mikeclayton , you might want to include this in your app as well :)


WV-342, add an option to hard delete Voter Guide Possibility entries

Added a new option to the "Mark selected row as" drop-down box and the code to process it.

👍 Thank you @stilwellmg!