The Planner does not handle / recognize that an output is a list and should pass the values using an index

I am working with Copilot with a custom plugin for RAG. The plugin has two functions:

ListDocuments, that returns a set of metadata objects about documents  that have embeddings.
DoSearch, perform a search on a document by passing the doc id and doc type.

The prompt is something like this
iterate thru the documents available on <custom plugin name>, and ask the question "what is foo?" for each document returned

The plan gets the two steps right - list documents and the search doc, but always presents the plan as setting the input value to $RESULT.document_id not $RESUL[0].document_id. It also doesnt seem to know how to iterate thru each document. If the list documents is run prior to the asking for the iterative search, then sometimes it will try to create multiple steps for the number of documents it knows was returned in the list (but inconsistently).
I also noticed that if you ask that the plan be fixed by using the indexed values, it occasionally will get it right but then forget something else, like the original question, or that the output field is id instead of document_id
How can the handling of this sort of multi-step process be made better? Would it make sense to override the planning prompt that SK uses? How do you represent iterations in that case?

Look for MiscSkill.ElementAt for an example of a function that can help with what you describe. I'd also suggest updating the description of your functions to describe the shape of the output in lieu of those constructs in the kernel (coming soon* I think).

Closing this issue. @ramarnat, let us know if @lemillermicrosoft, solution does not work for you.


add support for kind v0.8, and upgrade the docker image to kind v0.8

Hello @nicks,
Please review the following commits I made in branch nicks/ch6753/kind:
6979eefcf1fb46a38e1781e1a26c1ec121ae29f4 (2020-05-01 12:57:36 -0400)
add support for kind v0.8, and upgrade the docker image to kind v0.8
Code review reminders, by giving a LGTM you attest that:

Commits are adequately tested
Code is easy to understand and conforms to style guides
Incomplete code is marked with TODOs
Code is suitably instrumented with logging and metrics

ok, this is ready for review! Based it on https://github.com/kubernetes-sigs/kind/pull/1542, but added some guards to handle incremental upgrades a bit better


Added development.md

closes #853
Added development.md

Ah I see what you mean, I was curious how that worked with the PR and if it would move them over here, but I guess that's not how that works, I'll see about moving them over here

Seems they may have to be added manually by someone with write access to that wiki file @shanedell

Another option is to replace images with text where possible. For example, the image that shows the fork button could be replaced with the a link to https://github.com/apache/daffodil-vscode/fork. Making things a user can copy paste or click is probably a bit easier than telling people to find a button on another page. Avoiding images where possible is also friendlier to people with visual impairments.


[5.5] Offer autowire with PSR-11 compliant container

This is a simpler alternative for #21327. After reading through https://github.com/laravel/internals/issues/803 I see that my interpretation of https://github.com/laravel/framework/pull/19822 may have been a little bit flawed. PSR-11 leaves room of the implementation details to the concrete class. has doesn't necessarily needs to return false for auto-wire objects because Laravel Container does offer auto-wire. It is better consistency to also offer auto-wiring with has and get.

If this gets merged, we'll be 1 step away from 3k

I have no plans to change this interface.

@taylorotwell any plans on looking into this again? Having the ability of autowiring + PSR-11 compatibility would be really great.


Print more information on log line interop test failures

resolves #5658
Description
Adds more information to std out when these log tests fail. Also refactors the code to be higher quality.
Reviewers
This has a few whitespace changes. My non-whitespace changes start at 167 of the original. You can start at test_deserialize_serialize_is_unchanged which is the test. This calls the helper functions that have been modified.
The biggest risk of this change is that I could have weakened the test so it would pass in more situations. I, of course, don't suspect this is the case but it should be verified by the reviewer.
Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[ ] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[x] I have run changie new to create a changelog entry

@McKnight-42 the risk with changing tests like this is that they could be accidentally modified to fail less often. I've frequently found tests in old code bases that could never fail which are obviously not good tests. I don't think this modification falls into that category, but I wanted to call it out as something for reviewers to look for since we're modifying a test.


Generate Javadoc and auto-push to gh-pages.

As extensively discussed in #176 this PR features two enhancements:

Create Javadoc without package internal
Auto-push Javadocs to gh-pages to host them online

Justifications:

I created a dedicated task for the Javadoc creation. In a previous PR I just configured Javadoc to omit the package, but since Javadoc is also used in another task (packageJavadoc), configuring an own tasks does not influence the other task.
Since cloud CI (travis) was not desired, I wrote a shell script which runs on a local machine to fulfill auto-push

Usage:

Generate Javadoc first ./gradlew clean generateJavadocForGhPages (use clean to ensure internal package has not been generated from other tasks)
Set & export your GH_TOKEN

> GH_TOKEN=<your token>
> export GH_TOKEN

Execute the script

> .buildscript/push-javadoc-to-gh-pages.sh
Deploying javadoc...
Creating temporary directory...
mkdir: created directory '/var/folders/hx/4lyxw6bs0417ztrz3w92ndn80000gp/T/javadoc.C6nzwxLo/javadoc-latest'
Copying files to temporary directory...
Cloning current gh-pages branch...
Cleaning...
Copying javadocs to gh-pages...
javadoc deployed!
Removing temporary files...

Remark: I wrote and tested the script on OS X, where mktemp might have a slightly other usage than on a linux machine. It may be necessary to replace mktemp -d -t javadocwith just mktemp -d

nobody asked for published javadoc so far. I don't think our users would need that. please raise a hand if you disagree. but for now I will close this pull request without merging. adding anything to the build means more stuff to support/maintain. if there is no clear value, then we should not do it.
how about we add more examples to README.md file instead?


[stdpar] Add struct reflection support; only offload if no invalid device pointers are found

Adds initial infrastructure for reflection builtins;
Adds first reflection builtin: introspect_flattened_struct, which can provide information about all members contained in aggregate data types (like structs or classes), such as member offsets, sizes and whether it is an int, a floating type, pointer or something else. This also supports nested structs; in this case the struct is flattened and the information is provided as per the data layout of the struct in memory. This operates on the LLVM IR level, so there is no distinction between e.g. pointers or references. This builtin can be used to e.g. find, validate or otherwise introspect all pointers that are stored in a kernel lambda!
The builtin is used to add support for detecting PSTL calls that might reference host memory, and not offload in that case. As a side effect, this adds functional support for capture-by-reference in PSTL: Previously, something like

int y = 2;
std::for_each(... [&](auto& x){ x += y; });

might have been offloaded, and then crashed on device due to the reference to host-memory for y. With this PR, we detect that an argument to for_each contains a pointer that is not valid on device, and execute the call on the host to be on the safe side.
Note that, since the pointer validation is ultimately done at runtime, we can distinguish these cases:
// We rewire heap allocations to device-accessible USM allocations,
// so this call would get offloaded
int* y = some_function_that_returns_a_heap_ptr();
std::for_each(... [=](auto& x){ x += *y; });

// We would detect that z now does not contain a valid device pointer,
// and not offload.
int* z = some_function_that_returns_a_stack_ptr();
std::for_each(... [=](auto& x){ x += *z; });

@tom91136 I believe you might have some use case too for the new reflection functionality ;)

At least modifications for such use cases would be limited to playing with the headers, instead of fiddling around with compiler/runtime internals :) And it works seamlessly across all our clang-based compilation flows: SSCP, cuda, hip, omp.accelerated.
I'd like to look into adding some heuristics for automatic prefetch/memadvise next, but we first need an efficient data structure to map pointers to some allocation parameters - particularly allocation size.


dep: bump k8s to 1.12.6 and regenerate codegen

This PR bumps k8s to 1.12.6, golang to 1.11.5 and regenerated codegen.
Signed-off-by: Sam Batschelet<EMAIL_ADDRESS>

@hasbro17 PTAL

SGTM. We should also update the CHANGELOG to point out the update to k8s 1.12.6.


[YUNIKORN-99] Add state based app sorting

To allow FIFO scheduling to work when lots of applications are submitted
an extra state is added to the applications. This state shows a newly
starting application with just one allocation. If more allocations are
added the application goes into a running state. The starting state
times out after 5 minutes to prevent starvation of other apps.
A new sorting policy for applications in a queue is added which only
allows one starting application to be scheduled at a time. If there are
no starting applications in the list a new application withut
allocations, accepted state, is added to the list of apps to schedule.
The sorting policy can be set on a leaf queue by specifying "status" as
the sort policy.

closed by wrong commit message tag, re-opening

Travis tests have failed
Hey @wilfred-s,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.
1st Build
View build log

     curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(go env GOPATH)/bin v1.22.2

golangci/golangci-lint info checking GitHub for tag 'v1.22.2'
golangci/golangci-lint info found version: 1.22.2 for v1.22.2/linux/amd64
golangci/golangci-lint info installed /home/travis/gopath/bin/golangci-lint

     make lint

running golangci-lint
pkg/scheduler/scheduler.go:302: unnecessary trailing newline (whitespace)

	}
pkg/scheduler/tests/scheduler_recovery_test.go:619:27: string `app-1` has 11 occurrences, make it a constant (goconst)
		if app.ApplicationID == "app-1" || app.ApplicationID == "app-2" {
		                        ^
Makefile:45: recipe for target 'lint' failed
make: *** [lint] Error 1

TravisBuddy Request Identifier: ef9d4cd0-9aab-11ea-ae6f-3d1c6927e18c

The lint remarks are not simple to fix: if one file is changed the other files in the package are showing up. Fixing the issues would bloat the patch unnecessarily. Leaving it as is for now.
Unit tests pass.

hi @wilfred-s this LGTM. once the UT is fixed, feel free to merge this PR.

The failing unit test TestSchedulerRecoveryWhenPlacementRulesApplied is covered in YUNIKORN-155 it is a known flaky test.
Based on the approval ready for merging.


Unbreak build straight from Git

Currently, any attempt to build the sources from Git will fail due to missing fields in APISecrets.

Hmm, looks like I forgot to add those, thanks ! merging now !


Login and self-management

Allow login on the Pub site
See list of packages logged in user is an uploader of
Package self management

Hide email
Remove uploader
Add uploader
Discontinue email
Notification on upload
Require 2FA auth

Couple of notes and my current plans for implementing this:
It is very likely that we want to have a single-page app for the account- and package-administration, while at the same time we'd like to support other on-page, smaller interactions (e.g. star-ing or up/down-voting a package page).
At the moment it is not entirely clear if the SPA vs. a traditional form-based app is faster to implement (esp. wrt. testing), but I think in the long term a SPA provides more benefit for us. The default stack choice would be angular + material components.
The auth mechanism should work for both, and it is likely that it will be a combination of:

auth cookie (JWT or just a random id that is stored in Datastore with the detailed information)
CSRF-token (cookie value that gets to be added in the request headers)

There is one concern with the CSRF-token approach: any third-party script that is hosted on the domain could be able to read the token value, and pose a CSRF-risk. It is a very low risk, and at the moment the source of such external script could be either our app (e.g. code highlighter) or dartdoc-generated assets (e.g. typeahead.js).
I'm planning to use the /api/admin/* and the /admin/* URLs for implementing these functions.
The 2FA is a bit unknown to me: not sure how to approach it, e.g. if we are using Google's Oauth2, what does it mean for us to add 2FA on top of it?

Jonas to write a design doc


series_num_seen_unseen() should take 'now' into account

Meaning, it should not count unreleased episodes as unseen.
This implies baking a bit more data into the meta. I hope it's enough to add a release date per unseen episode.
This will likely fix many issues, e.g. #47.

I guess we only need to store the dates for unreleased episodes? (as of the time of update)


What’s the correct mark-up for custom radios and server-side validation?

I’m having trouble rendering stacked custom radios with server-side validation.
My raw mark-up looks like this (a Laravel Blade template):

    Category

        @foreach($categories as $id => $name)

                <input type="radio" id="category-{{ $id }}" name="category_id" value="{{ $id }}" class="custom-control-input" {{ old('category_Id') == $id ? 'checked ' : '' }}/>

                {{ $name }}

        @endforeach

    @foreach($errors->get('category_id') as $error)

{{ $error }}

    @endforeach

Where is the appropriate place to put the .is-invalid class that both makes the radio inputs red, and displays the
 elements?

https://github.com/twbs/bootstrap/issues/23426
Sadly can't be done at the moment, you'll need custom css until it is fixed. Hopefully in Beta 3.

@istratos Urgh. Thanks for confirming my fears 😄
What I‘ve done in the meantime is this:

    Category

        @foreach($categories as $id => $name)

                <input type="radio" id="category-{{ $id }}" name="category_id" value="{{ $id }}" class="custom-control-input{{ $errors->has('category_id') ? ' is-invalid' : '' }}" {{ old('category_id') == $id ? 'checked ' : '' }}/>

                {{ $name }}

        @endforeach

    @if($errors->has('category_id'))

    @endif
    @include('partials.invalid-feedback', ['field' => 'category_id'])

So if there’s an error, I add an empty
 with .form-control and .is-invalid classes, which because of the sibling selector styles shows the .invalid-feedback element.

The problem with any other practical options is that we shouldn't place form validation feedback within the form label. This is largely driven by my decision to wrap checkbox and radio inputs with the label to avoid an explicit for attribute.
Here's an example of what our options are: https://codepen.io/anon/pen/WXKgzv. First is feedback in the label, second is rewriting the components.


(PDB-2478) Add admin/v1/clean endpoint

Might or might not be useful to review the individual commits.

Realized there's still a race in the tests - I'll fix it later today or in the morning.

OK, this is probably ready for another round.

Lgtm codewise modulo my comment about :as i18n. I'll test this out now.


utils/ruby: silence which errors when finding ruby

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?
[ ] Have you successfully run brew man locally and committed any changes?

The which command is used to find ruby on systems where portable-ruby doesn't work. In my case, my WSL instance of Fedora 33 was missing one or more libraries that portable-ruby requires.
This pull request redirects stderr to /dev/null so that nothing is printed if ruby is not found in the usual places (/usr/bin, /bin, /usr/sbin, /sbin).
Before:
$ brew help
which: no ruby in (/usr/bin:/bin:/usr/sbin:/sbin)
Example usage:
  brew search [TEXT|/REGEX/]
...

After:
$ brew help
Example usage:
  brew search [TEXT|/REGEX/]
...

Review period will end on 2020-12-17 at 00:42:36 UTC.

Review period skipped due to critical label.

Thanks @SeekingMeaning!


[Bug report]: I cannot recover/verify my account

Version
1.7.2
Description
App Version: 1.7.2
platform: macOS
platformVersion: 13.2.1
platformArchitecture: x64
cpuCount: 8
totalMem: 16384.0 MB
freeMem: 78.2 MB
Error Type: Callback Error createAccount
TypeError: Cannot read property 'usd' of null
at He (file:///Applications/Firefly.app/Contents/Resources/app.asar/public/build/index.js:2:1448616)
at onSuccess (file:///Applications/Firefly.app/Contents/Resources/app.asar/public/build/index.js:2:1440124)
at e.. (file:///Applications/Firefly.app/Contents/Resources/app.asar/public/build/index.js:2:1438028)
at Function. (file:///Applications/Firefly.app/Contents/Resources/app.asar/public/build/index.js:2:1435210)
Expected behaviour
I should be able to recover my account, define new password and pin
Actual behaviour
I tried to recover my account using the recovery phrase, the initial steps went well, then the bug happened after I tried to verify my recovery phrase
Can the issue reliably be reproduced?
No
Steps to reproduce the issue
No response
Operating System
Mac
Diagnostics
Mac M2

Developer Profile
No
Profile Type
Software (Stronghold)
Staking
Previously staked
Crash Status
None of the above
Error Messages
App Version: 1.7.2
platform: macOS
platformVersion: 13.2.1
platformArchitecture: x64
cpuCount: 8
totalMem: 16384.0 MB
freeMem: 78.2 MB
Error Type: Callback Error createAccount
TypeError: Cannot read property 'usd' of null
at He (file:///Applications/Firefly.app/Contents/Resources/app.asar/public/build/index.js:2:1448616)
at onSuccess (file:///Applications/Firefly.app/Contents/Resources/app.asar/public/build/index.js:2:1440124)
at e.. (file:///Applications/Firefly.app/Contents/Resources/app.asar/public/build/index.js:2:1438028)
at Function. (file:///Applications/Firefly.app/Contents/Resources/app.asar/public/build/index.js:2:1435210)
Error Logs
App Version: 1.7.2

platform: macOS
platformVersion: 13.2.1
platformArchitecture: x64
cpuCount: 8
totalMem: 16384.0 MB
freeMem: 78.2 MB

Error Type: Callback Error createAccount

TypeError: Cannot read property 'usd' of null
    at He (file:///Applications/Firefly.app/Contents/Resources/app.asar/public/build/index.js:2:1448616)
    at onSuccess (file:///Applications/Firefly.app/Contents/Resources/app.asar/public/build/index.js:2:1440124)
    at e.<computed>.<computed> (file:///Applications/Firefly.app/Contents/Resources/app.asar/public/build/index.js:2:1438028)
    at Function.<anonymous> (file:///Applications/Firefly.app/Contents/Resources/app.asar/public/build/index.js:2:1435210)

Duplicate declaration

[X] I have searched the issues tracker this issue and there is none

Code of Conduct

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

You are running a very old version of Firefly. You should upgrade to latest version since the one you are using is not compatible with the latest protocol version, as of today the latest version is V2.0.11


exposed logger as node module

This makes it easier for 3rd party modules to use the logger in the node helper with the according logLevels instead of console logs.
You only need to do const Log = require('logger'); in a module.

Codecov Report

Merging #2464 (b18d98f) into develop (09ddd3d) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop    #2464   +/-   ##
========================================
  Coverage    39.33%   39.33%
========================================
  Files           20       20
  Lines         1612     1612
========================================
  Hits           634      634
  Misses         978      978

Impacted Files
Coverage Δ

js/app.js
70.33% <100.00%> (ø)

js/electron.js
69.38% <100.00%> (ø)

js/node_helper.js
93.33% <100.00%> (ø)

js/server.js
91.11% <100.00%> (ø)

modules/default/calendar/calendarfetcher.js
62.79% <100.00%> (ø)

modules/default/calendar/node_helper.js
84.00% <100.00%> (ø)

modules/default/newsfeed/newsfeedfetcher.js
74.62% <100.00%> (ø)

modules/default/newsfeed/node_helper.js
78.12% <100.00%> (ø)

modules/default/updatenotification/node_helper.js
54.71% <100.00%> (ø)

Continue to review full report at Codecov.

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

Thanks! Might be a good idea to add info to the docs about this.


Support for RC5?

Does this library support Angular 2 RC5?

No at the moment. But feel free to pull request

Elsewhere I am going to wait angular2 release because looks like angular2 devs don't want to freeze API changes even during RC.

Hi @gevgeny.  It is not intended API changes in the RC6. So would be good to give support for RC6.

There won't be any breaking changes in API from RC6 onward. Please update this library as many people are currently unable to generate production builds because this library is not up to date

vote+1, I face the same issue

Support RC6!? :/

+1

@gevgeny thank you for your awesome wrapper! it's been very helpful til rc.5 breaking changes.
so now angular 2 is finally here! can we expect an update here to?

+1

Yes, sorry for the delay

See https://github.com/gevgeny/angular2-highcharts/issues/64#issuecomment-249513601


something broke in 0.6.2

getting this error now:
/Users/michaelreinstein/wwwroot/level-editor/node_modules/jscrambler/bin/jscrambler:52
  commander.accessKey = commander.accessKey || config.keys.accessKey;
                                                          ^
TypeError: Cannot read property 'accessKey' of undefined
    at Object.<anonymous> (/Users/michaelreinstein/wwwroot/level-editor/node_modules/jscrambler/bin/jscrambler:52:59)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Function.Module.runMain (module.js:497:10)
    at startup (node.js:119:16)
    at node.js:929:3

I'm using the config file feature. It has something to do with this.

I'm not getting the error any more, but now I'm experiencing other problems. I'll open a separate ticket.


Create a Galaxy Tool to manually submit specimens in bulk

Should support:

IIIF manifests
Simple metadata?
DarwinCore?

CSV upload working on welcome page
Next step is to do a per-row submission to Galaxy API

Relevant Galaxy SPI docs:
https://docs.galaxyproject.org/en/latest/lib/galaxy.webapps.galaxy.api.html#module-galaxy.webapps.galaxy.api.workflows

Feature now working - demo tomorrow

This tool is functioning and performing as expected, there is some technical debt and refactoring would be a good idea later.


Congrats!

Very nice library!
Have you thought about including Benchmarks with BenchmarkDotNet? I'm interested how the performance compares with python numpy and NumSharp on Github.

Thanks for the response. We're working now on adding LAPACK for speed ups.

I'm also looking forward to check the performance comparison. I think MKL will be the winner because it optimized by Intel, perfect running on multiple core. @fdncred We're also considering support multiple provider solution, LAPACK and MKL.

@Oceania2018, Ya, that's why I liked Armadillo. It seemed to support multiple libraries. Last I looked it seemed like OpenBlas was the winner across platforms and architecture.
@thp-proxem, Did you try other linear algebra/maths libraries before choosing MKL? If so, what were they and what pushed to to choose MKL over the others?


Fix issue #172 and #227

Fix #227  gnt & gnt in upper case
Fix #172 certificate & UTF-8 encoding and removal of BOM  on JSON files
Fix RAT fields, Removal of ma from NAAT

Looks good now, thanks for the update!


Adding instructions for running RRS from the command line to support the Cicero paper.

Added instructions for executing Rapid RNA-Seq workflows from the command line. The Cicero paper authors want to include instructions to facilitate batch processing of samples.

Content replicates https://www.stjude.cloud/docs/faq/#how-can-i-run-an-analysis-workflow-on-multiple-sample-files-at-the-same-time. That FAQ is sufficient for our needs.


Connecting to GigE camera from Docker container

Hello, is there an alternative way to connect to a GigE camera from a Docker container other than using --network host?
The camera is Basler a2A1920-51gcPRO (40232887) and the Docker containers are configured via IoT Edge deployment manifests. Setting network to be host removes the original network (azure-iot-edge) which is added automatically (i.e. not by us).
For context, this is how we do it (based on this thread - https://github.com/Azure/iotedge/issues/6128):
"createOptions": {
                "NetworkingConfig": {
                    "EndpointsConfig": {
                         "host": {}
                    }
                },
                "HostConfig": {
                    "NetworkMode": "host"
                }
            }

Thank you very much!

one can use a ipvlan or macvlan Docker network and connect the container to that network in order to see and connect to a GigE camera (https://docs.docker.com/network/drivers/ipvlan/ and https://docs.docker.com/network/drivers/macvlan/).


Physics Player

cursor rotation + set to center
movements, gravity, flying

All fixed except set cursor to the center of the screen (not possible?).
Rotation can be smoother with better arguments values (sensitivityMouse, speedRotation).
Press F Key enable flying state.


about the code

thanks for publishing the code, but i faced with an error. I run the code and add these lines in the main.py:
tf.app.flags.DEFINE_string('word2id', '', 'word to id')
tf.app.flags.DEFINE_string('max_sentence_len', '', 'max length of sentences')
tf.app.flags.DEFINE_string('max_aspect_len', '', 'max length of aspects')
after passing the first error, the new one appear. the error is :
There are 5198 words in the dataset, the max length of sentence is 79, and the max length of aspect is 23
Loading training data and testing data ...
Read 3602 sentences from data/restaurant/train.xml
Read 1120 sentences from data/restaurant/test.xml
Loading pre-trained word vectors ...
Traceback (most recent call last):
File "main.py", line 47, in
tf.app.run()
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/platform/app.py", line 125, in run
_sys.exit(main(argv))
File "main.py", line 38, in main
FLAGS.word2vec = load_word_embeddings(FLAGS.embedding_fname, FLAGS.embedding_dim, FLAGS.word2id)
File "/content/RAM/utils.py", line 158, in load_word_embeddings
word2vec[word2id[content[0]]] = np.array(list(map(float, content[1:])))
ValueError: could not convert string to float: '.'
can you help me in this ? best regards

It happens when it processes the line which contains "...", you can simply add the conditional statement if len(content) != 301: continue or try modifying word2vec[word2id[content[0]]] = np.array(list(map(float, content[1:]))) to word2vec[word2id[content[:-300]]] = np.array(list(map(float, content[-300:])))

Woww. Thanks a lot for your respond. I fixed it and it runs.
I think it is necessary to mention some other points in this project which may help other in future:
first of all, I replaced word2vec[word2id[content[0]]] = np.array(list(map(float, content[1:]))) with word2vec[word2id[content[:-300]]] = np.array(list(map(float, content[-300:]))), but another error appeared, it was about hashing error:
Loading pre-trained word vectors ... Traceback (most recent call last):   File "main.py", line 47, in <module>     tf.app.run()   File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/platform/app.py", line 125, in run     _sys.exit(main(argv))   File "main.py", line 38, in main     FLAGS.word2vec = load_word_embeddings(FLAGS.embedding_fname, FLAGS.embedding_dim, FLAGS.word2id)   File "/content/RAM/utils.py", line 160, in load_word_embeddings     word2vec[word2id[content[:-300]]] = np.array(list(map(float, content[-300:]))) TypeError: unhashable type: 'list'
then I use the other solution that you said. I mean using if len(content) != 301: continue in load word embeddings function in utils.py . finally it works but it pointed to another issue. the error was this:
Traceback (most recent call last):   File "main.py", line 47, in <module>     tf.app.run()   File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/platform/app.py", line 125, in run     _sys.exit(main(argv))   File "main.py", line 38, in main     FLAGS.word2vec = load_word_embeddings(FLAGS.embedding_fname, FLAGS.embedding_dim, FLAGS.word2id)   File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/platform/flags.py", line 88, in __setattr__     return self.__dict__['__wrapped'].__setattr__(name, value)   File "/usr/local/lib/python3.6/dist-packages/absl/flags/_flagvalues.py", line 499, in __setattr__     return self._set_unknown_flag(name, value)   File "/usr/local/lib/python3.6/dist-packages/absl/flags/_flagvalues.py", line 375, in _set_unknown_flag     raise _exceptions.UnrecognizedFlagError(name, value) absl.flags._exceptions.UnrecognizedFlagError: Unknown command line flag 'word2vec'
but with your previous tip I fixed it by defining word2vec before main in main.py
tf.app.flags.DEFINE_string('word2vec', '', 'word to vec')
So i appreciate the time that you gave me and helped me in this. the paper and its performance is great. I want to delve deeper in this and find out is it possible to improve it or not?! if I get good result certainly i share it with you here. Again, Thanks a lot

I'm sorry that I haven't verified my modification. And I find the best way to correct the problem: just modify content = line.strip().split() to content = line.strip().split(' ')


[CERT-TEST-FAILURE] TC-IDM-1.2.py

Feature Area
Other
Test Case
TC-IDM-1.2
Reproduction steps
Command to execute the TC_IDM_1_2.py :  python3 TC_IDM_1_2.py --commissioning-method on-network --discriminator 3840 --passcode 20202021 --storage-path admin_storage.json --PICS ci-pics-values --paa-trust-store-path /credentials/development/paa-root-certs/
Bug prevalence
Whenever I do
GitHub hash of the SDK that was being used
19771ed7101321d68b87d05201d42d00adf5368f
Platform
python, raspi
Anything else?
TC_IDM_1_2.py is failed to pair the TH2.

[MatterTest] 10-05 12:37:18.524 ERROR Exception occurred in test_TC_IDM_1_2.
Traceback (most recent call last):
File "/root/python_testing/TC_IDM_1_2.py", line 210, in test_TC_IDM_1_2
TH2.GetConnectedDeviceSync(nodeid=self.dut_node_id+1, allowPASE=True, timeoutMs=1000)
File "/usr/local/lib/python3.10/dist-packages/chip/ChipDeviceCtrl.py", line 776, in GetConnectedDeviceSync
raise TimeoutError("Timed out waiting for DNS-SD resolution")
TimeoutError: Timed out waiting for DNS-SD resolution
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.10/dist-packages/mobly/base_test.py", line 783, in exec_one_test
test_method()
File "/root/python_testing/matter_testing_support.py", line 1008, in async_runner
return asyncio.run(body(*args, **kwargs))
File "/usr/lib/python3.10/asyncio/runners.py", line 44, in run
return loop.run_until_complete(main)
File "/usr/lib/python3.10/asyncio/base_events.py", line 649, in run_until_complete
return future.result()
File "/root/python_testing/TC_IDM_1_2.py", line 212, in test_TC_IDM_1_2
asserts.fail("Unable to establish a PASE session to the device")
File "/usr/local/lib/python3.10/dist-packages/mobly/asserts.py", line 475, in fail
raise signals.TestFailure(msg, extras)
mobly.signals.TestFailure: Details=Unable to establish a PASE session to the device, Extras=None
[MatterTest] 10-05 12:37:18.532 INFO [Test] test_TC_IDM_1_2 FAIL

Log:
tc-idm-1-2.txt

This is setup issue, tried with other setup and passed.
tc-idm-1-2.txt


[Bug Report] BUG in the tree menu component

Element UI version
2.0.8
OS/Browsers version
Google Chrome	58.0.3029.110
Vue version
2.5.10
Reproduction Link
https://jsfiddle.net/5eh0hxkn/
Steps to reproduce
After entering, we first close the first level 2 menu, then click on the sub menu of the first level 3, the first level 2 menu will expand automatically, not the design, so I think it's a BUG.
This is a code link：https://jsfiddle.net/5eh0hxkn/
What is Expected?
It is expected that the start menu is unfolded, then the first level 2 menu is closed, and then the 3 menus submenu is clicked, and the level 2 menu should not be automatically expanded.
What is actually happening?
The first level 2 menu is open

This problem affects my actual use

感觉好像和这个default-expanded-keys属性有关

JSFiddle能重现，本地一直没重现，昨天修了个类似的问题，修复方法在这几行：https://github.com/furybean/element/blob/9b35b4bd5072f5d3c9e6f2a11b353159fd5811d2/packages/tree/src/model/tree-store.js#L66
需要等下个版本发布了再看这个问题还存不存在。

@LongYue9608 在最新的版本中，经过我的测试，已经不会再出现这个问题了。


test signed commit adding .DS_Store to .gitignore

just to test if signed commits would work properly on this repo.
In the meantime I'll be working on a sandbox branch on my fork and once I have some or all of the bindings working I'll rebase squash and submit PRs.
I'm thinking of having at first two dart packages for this: lurk_bridge and crypto_bridge to wrap the work you've done in rust for ilxd and then have a bridge_demo flutter app that helps test/showcase how to use the bridge packages.
perhaps as I learn more both the lurk_bridge and crypto_bridge could be merged into a single rust_bridge that has all those symbols in a single .a/.so/.dll or .lib the wallet will be able to use.

a lot more coming, perhaps some minor updates in ilxd/zk/rust/Cargo.toml and ilxd/crypto/rust/Cargo.toml


Additional schema metadata from a source

Need the table name from the JdbcSource and other relevant information like schema/database name if applicable
HiveSource schema should also return table name, database, partition info, Hdfs location, Storage Format (parquet, Orc)
CsvSource schema should give directory location or full path name and any other encoding information - headers, escaping characters etc...

For CSV Source, the path is what you provide, the headers you can get out via schema() which will return a schema, where each header is a field in the schema.

Eel 0.97.0 and use HiveSource(...).spec()


Add ddd() helper function

ddd() is identical to dd() except that before dumping the given variables it dumps the code location of the call to ddd(). This way, if one uses ddd() instead of dd() it's easier to find dd calls which are left in the codebase without having to grep "dd(".

Erm... or a regex search for /\bdd(/ over your codebase.
I could maybe see adding some context to dd() itself, but even then... meh. A whole new method seems excessive.

Yes, that's fair. I debated implementing the change to dd() which is how I do this in my own projects but in the end wagered that people would get upset about the change in output to `dd()’

No plans on this.

Understood, thanks for taking a look.


[#2762] Improved remote call testcase.

Remove http://google.com

Codecov Report

Merging #2763 into master will increase coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2763      +/-   ##
==========================================
+ Coverage   40.68%   40.73%   +0.04%
==========================================
  Files        1773     1773
  Lines       61116    61116
  Branches     8586     8586
==========================================
+ Hits        24868    24896      +28
+ Misses      36248    36220      -28

Impacted Files
Coverage Δ

...m/navercorp/pinpoint/rpc/packet/PayloadPacket.java
77.77% <0%> (-11.12%)
:arrow_down:

...m/navercorp/pinpoint/rpc/packet/RequestPacket.java
87.17% <0%> (-2.57%)
:arrow_down:

.../profiler/sender/StandbySpanStreamDataStorage.java
79.66% <0%> (-1.7%)
:arrow_down:

...ofiler/sender/StandbySpanStreamDataSendWorker.java
88.05% <0%> (-1.5%)
:arrow_down:

...orp/pinpoint/rpc/server/DefaultPinpointServer.java
81.81% <0%> (+0.75%)
:arrow_up:

...m/navercorp/pinpoint/test/OrderedSpanRecorder.java
79.06% <0%> (+1.16%)
:arrow_up:

.../pinpoint/rpc/client/PinpointClientHandshaker.java
74.3% <0%> (+6.94%)
:arrow_up:

...vercorp/pinpoint/rpc/packet/ClientClosePacket.java
83.33% <0%> (+11.11%)
:arrow_up:

...rp/pinpoint/rpc/control/ControlMessageDecoder.java
80.35% <0%> (+14.28%)
:arrow_up:

...p/pinpoint/rpc/client/WriteFailFutureListener.java
93.75% <0%> (+18.75%)
:arrow_up:

... and 2 more

Continue to review full report at Codecov.

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


Add coveralls

Coveralls is a great tool to find out where praw needs more test coverage. If you enable the repo on coveralls this will work.

Changes Unknown when pulling f729edcc1121dcc355fa39921546dad0d873bc70 on JamieMagee:coveralls into ** on praw-dev:master**.

Thanks! This looks like an awesome tool.


Add git submodule update command

Signed-off-by: David Kwon<EMAIL_ADDRESS>Forgot to provide this feedback for this PR: https://github.com/che-incubator/devspaces-demo/pull/4. When the workspace starts, the submodule is not cloned. The git submodule update command clones the missing submodule
Related issue: https://github.com/devfile/devworkspace-operator/issues/949

Sorry @dkwon17 I didn't saw this PR earlier and I pushed a different change that should include your change. See https://github.com/che-incubator/devspaces-demo/commit/f0f38ec7c3737d6e44771ea33e266a10ba662890.

@l0rd no worries, closing


Handling large Hypercores persisted to disk (Windows)

Something I came across while working on the p2p-indexing-and-search workshop is that because of how the data is written to disk any hypercore (sparse or otherwise) takes the full space on disk (and then some). Even though solution 14 downloads only ~100kb of data from the imdb database it generates 8.92GB of files on disk.

you can see the code I use here
When reading about how hypercore handles storage it seems like it should handle these sparse cases better.

Because Hypercore and Dat support partially downloading data, a useful feature is to implement sparse persistence. This means that we can write data into memory or to disk with spaces in between, but without paying any cost.

Is this something that needs to be implemented, a regression, or am I just not understanding it properly?

Upon looking into it more, I presume this is because RAF would need to implement creating these files with the OS's sparse flag, which is complicated. I know that in Windows you require Admin rights to create sparse files with fsutil, and nodejs's fs library doesn't implement the fallocate function.

Surprising no one, actually setting the sparse flag on these files results in huge savings. Still seems huge but I presume that's just Windows trying the best it can.

Set the compress flag, and we hit the size I was expecting.

I wrote this a while back so it may be out of date, but here's a random-access storage that's designed for filesystems that can't do sparse indexing: https://github.com/pfrazee/random-access-indexed-file. (This is basically a storage plugin which hypercore can use)

@xori thanks! Which flags do you set on windows for this? Then I’ll update it. Sorry we aren’t Windows experts

I only know how to do it via the fsutil command in Windows.
fsutil sparse setflag <file>;  # enables the "<file>" to be marked as "sparse"
fsutil behavior set disablecompression 0 <file>; # by default this is enabled?

The downside is that the fsutil sparse requires Admin privileges, and I'm unaware of any pure javascript way of marking the file for the OS.
For linux, I only know of fallocate --dig-holes <file>, with once again, no pure javascript way of marking this.
It sucks that we have to break out to the OS for this, but thanks @pfrazee for linking me to your RAIF package. I'll monkey patch that into my local hypercore fork for my project for now because I'd rather take the performance hit than hit incompatibility with certain OSs.

No linux/mac all use sparse files per default

🤯

He he he... feel free to beat me to it with the PR. currently focused on other work and might be a few months off.
I'd be happy to help set it up and integrate with the hyper stack in your apps.

Fixed in 9.9.0


Make LAPACK from E3SM optional in Spack builds

This is needed because MPAS standalone expects Netlib-LAPACK, which conflicts with other LAPACK installations in a given spack environment.

Testing
I have tested this in compass where I was not able to add Netlib-LAPACK previously but I am with these changes when I set include_e3sm_lapack=False.

I have also verified that I can build compass test cases requiring Netlib-LAPACK after making this change.


Provide example of DSN in docs

In the documentation: https://godoc.org/github.com/go-kivik/couchdb#hdr-Authentication
For most uses, you don't need to worry about authentication at all--just include authentication credentials in your connection DSN. This will use Cookie authentication by default.
This is nice, but an example of DSN or where to look up this information is not provided.

It uses standard URL notation for authentication credentials, but I can add that to the docs.


Update Binary Ninja API submodule to 3.0 stable

https://github.com/jonpalmisc/ObjectiveNinja/blob/master/README.md?plain=1#L48 might wanna change this note too :)

Good call @brinlyau — I'll make sure that gets included as I work on #31 https://github.com/jonpalmisc/ObjectiveNinja/tree/better-readme


[SPARK-44332][CORE][WEBUI] The Executor ID should start with 1 when running on Spark cluster of [N, cores, memory] locally mode

What changes were proposed in this pull request?
The Executor ID should start with 1 when running on Spark cluster of [N, cores, memory] locally mode.
Why are the changes needed?

Alignment logic:  the executorId of Yarn and k8s both start with 1.
1.On yarn

https://github.com/apache/spark/blob/d4277b8e78347fd4e3163c6218edc4675ebb6db2/core/src/main/scala/org/apache/spark/scheduler/cluster/CoarseGrainedSchedulerBackend.scala#L124
https://github.com/apache/spark/blob/d4277b8e78347fd4e3163c6218edc4675ebb6db2/resource-managers/yarn/src/main/scala/org/apache/spark/scheduler/cluster/YarnSchedulerBackend.scala#L386-L387
https://github.com/apache/spark/blob/d4277b8e78347fd4e3163c6218edc4675ebb6db2/resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala#L162-L163
https://github.com/apache/spark/blob/d4277b8e78347fd4e3163c6218edc4675ebb6db2/resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala#L735-L738
https://github.com/apache/spark/blob/d4277b8e78347fd4e3163c6218edc4675ebb6db2/resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala#L760-L773
2.On k8s
https://github.com/apache/spark/blob/d4277b8e78347fd4e3163c6218edc4675ebb6db2/resource-managers/kubernetes/core/src/main/scala/org/apache/spark/scheduler/cluster/k8s/ExecutorPodsAllocator.scala#L51
https://github.com/apache/spark/blob/d4277b8e78347fd4e3163c6218edc4675ebb6db2/resource-managers/kubernetes/core/src/main/scala/org/apache/spark/scheduler/cluster/k8s/ExecutorPodsAllocator.scala#L452-L458

Fix UI Sort bug.
Detailed presentation in the following comment.
PS: Can be reproduced using: sh bin/spark-shell --master "local-cluster[2,1,1024]"

Executor ID starts with 0, making it more readable and friendly for ordinary people in the UI web.

Does this PR introduce any user-facing change?
Yes, The Executor ID will start with 1 when running on Spark cluster of [N, cores, memory] locally mode.
How was this patch tested?

Pass GA.
Manually test.

Before:
After:

Thank you very much for the scenario provided by @LuciferYang ❤

So the executorId of Yarn and k8s both start with 1? I think it's fine to unify to start with 1

cc @srowen FYI

So the executorId of Yarn and k8s both start with 1? I think it's fine to unify to start with 1

Yes, that's right.

Does mesos also start with 1?

Does mesos also start with 1?

Let me to check it.

I have fixed this issue by adjusting the JavaScript logic.

cc @yaooqinn

Merged to master


Feature: OCM plugin: decouple backend auth from kuberentes plugin

Is your feature request related to a problem? Please describe.
Kuberentes plugin dependency may not be desired for multiple reasons:

Adding a cluster to kubernetes plugin expects workload discovery on the cluster. If this is not permitted by a role, Backstage raises errors. Discovering workloads on the Hub cluster may not be desired.
Accessing the Hub requires ServiceAccount access - it's accessed by the backend server at all times, therefore no identity provider can be used. That results in no RBAC applied, all users see the same/all data from the Hub cluster via Kuberentes plugin.

Describe the solution you'd like
Provide a different connection method. Consider changing the config schema to a partial of kubernetes-backend cluster type:
# app-config.yaml
ocm:
  # either use kubernetes
  name: # required if url not defined, string, current setup - match a cluster from kuberenetes plugin config
  # if name is not defined, use direct setup
  url: # required if name not defined, string, hub cluster api endpoint
  skipTLSVerify: # optional, bool, defaults to false
  skipMetricsLookup: # optional, bool, defaults to false
  caData: # optional, str, defaults to null
  caFile: # optional, str, defaults to null

Describe alternatives you've considered
N/A
Additional context
N/A

I can take a look at this.


Score breakdown by section in results

It would be helpful for the AIA results to contain the breakdown of the score by section. This would better convey the areas that contribute to the overall score of the initiative. This should be provided for both the risk profile and the mitigation measures.

Risk Area
No. of Questions
Project Score
Maximum Score

1. Project
15
9
15

2. System
1
0
0

...
...
...
...

5. Data
13
29
44

RAW IMPACT SCORE
48 questions
67
107 maximum

(This suggestion came from Norma at ESDC, thank you!)

The pull request #896 works when running locally (e.g. npm run serve), but it fails when running in production (e.g. in the version deployed on github pages, or built via npm run build). In production, the tables always come out empty.

Digging into the code a bit, in Results.vue#riskAreaItems(), the attribute myResult.questionHeader is always undefined when running in production. It's as if $store.getters.resultDataSections didn't do its job.

If I prevent webpack minification, then the bug goes away... But obviously not a workable solution cry
in vue.config.js:
  configureWebpack: {
    devtool: "source-map",
    optimization: {
      minimize: false
    }
  },

Final product looks like this:


fix nil-pointer dereference in assignSingleChildProperty

Prior to this commit, the ofbx parser would crash with the
following panic as the filename and relativeFilename fields
of Texture were not allocated before call from parseTexture
to assignSingleChildProperty.
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x52a23e]

goroutine 1 [running]:
github.com/oakmound/ofbx.assignSingleChildProperty(0xc000302880, 0x566ced, 0x8, 0x0, 0x0)
	/home/u/Desktop/ofbx/property.go:140 +0x5e
github.com/oakmound/ofbx.parseTexture(0xc00010c000, 0xc000302880, 0x0)
	/home/u/Desktop/ofbx/parse.go:381 +0x85
github.com/oakmound/ofbx.parseObjects(0xc000078040, 0xc00010c000, 0x0, 0x0, 0x0)
	/home/u/Desktop/ofbx/parse.go:730 +0xb54
github.com/oakmound/ofbx.Load(0x578628, 0xc00000e028, 0x0, 0x0, 0x0)
	/home/u/Desktop/ofbx/scene.go:101 +0x2ba
main.parse(0x7ffc4886dabb, 0x31, 0x0, 0x0)
	/home/u/Desktop/ofbx/foo/main.go:28 +0x1b3
main.main()
	/home/u/Desktop/ofbx/foo/main.go:16 +0xd4

Not sure what the contribution guidelines are for oakmound projects. In either case, I hereby release this code change into the public domain.
Cheers,
Robin


REST: resource_exists/2 together with content_types_provided/2

Let's consider the following cowboy handler module:
-module(my_rest_handler).

-export([
         init/2,
         to_json/2,
         resource_exists/2,
         content_types_provided/2
        ]).

init(Req, State) ->
    {cowboy_rest, Req, State}.

resource_exists(Req, State) ->
    {false, Req, State}.

content_types_provided(Req, State) ->
    {[{{<<"application">>, <<"json">>, '*'}, to_json}], Req, State}.

to_json(Req, State) ->
    {<<"{}">>, Req, State}.

This module always replies with
< HTTP/1.1 404 Not Found
< content-length: 0
< content-type: application/json
< date: Wed, 20 Sep 2017 00:33:06 GMT
< server: Cowboy

and an empty body, which is incorrect (notice the content-type: application/json header; that reply is incorrect, since an empty body is not a valid JSON).
It seems to me cowboy shouldn't set the content type header from content_types_provided/2 if it never calls the provided callback (eg as in the example above because resource_exists/2 returns false, but possibly in other cases).
@essen : do you agree this behaviour is not the intended? If yes, happy to propose a PR to fix it, please let me know :)

It's a fairly old issue: https://github.com/ninenines/cowboy/issues/413

Thanks for the quick answer @essen , will follow that other thread


Add our recent work

Hi, I add our recent work to README😊

Hello @Matrix53 ,
it is now fixed. Thank you for your awesome work!


Add vcpkg installation instructions

Nanopb is available as a port in vcpkg, a C++ library manager that simplifies installation for nanopb and other project dependencies. Documenting the install process here will help users get started by providing a single set of commands to build nanopb, ready to be included in their projects.
We also test whether our library ports build in various configurations (dynamic, static) on various platforms (OSX, Linux, Windows: x86, x64, UWP, ARM) to keep a wide coverage for users.
I'm a maintainer for vcpkg, and here is what the port script looks like. We try to keep the library maintained as close as possible to the original library.

Thanks!


Japanese Screenshot on TypeScript Polish Pipeline Page

Describe the bug
on the https://cdkworkshop.com/20-typescript/70-advanced-topics/200-pipelines/5000-test-actions.html page is a screenshot where the language is in japanese(?)
Expected Behavior
Language in the Screenshot should be same as the selected language.
Current Behavior
Japanese Text in Screenshot
Reproduction Steps
goto https://cdkworkshop.com/20-typescript/70-advanced-topics/200-pipelines/5000-test-actions.html  page scroll down a little bit to the first screenshot with the source "./stack-outputs.png"
Possible Solution
No response
Additional Information/Context
No response
CDK CLI Version
No response
Section
No response
Browser
Chrome
Language
Typescript

Thanks for reporting this!


Added the missing installation step

Thanks for the request, a new documentation page is now available that contains a lot more including the suggested improment. I am going to remove most of the documentation from this page.


ACTG = Percentage of ACTG bases in the 06.OTU_representative_script.R

Hi vmikk
I am interested in running the '06.OTU_representative_script.R' to choose my representative sequences. However, I do not understand the 'ACTG = Percentage of ACTG bases' requirement on line nine of the script. Could you please clarify what this is asking for (e.g., percentage of ACTG individually separated by some delimiters) or alternatively could you provide the head of an example data sheet for me to copy the formatting from?
Thank you for your time
Luke

Thanks, Vladimir! That is clear as day now.
Cheers
Luke


Syntax error for two different function definitions in compiler conditionals

That might be a tricky one to fix and it's not a big deal at all (it doesn't stop code from parsing/highlighting), but for consistency sake it's should have a ticket.
#if flash
   @:setter(height)
   public function set_height(value:Null<Float>):Void // An error is highlighted at the end of this line.
#else
   override public function set_height(value:Null<Float>):Float
#end

TIR: STB-7297

This is not important at all.


Initial Commit

This is initial set of minimum requirements defined by the sub-group.  We might want to consider shortening the title of file.  It seems a bit unwieldy.
Signed-off-by: Tim Michels<EMAIL_ADDRESS>

Trying to kill this pull request to start over.


WIP: Pre-compiled config schema json

WIP experimentation related to https://github.com/netlify/netlify-cms/issues/2138
Summary

Test plan

Checklist
Please add a x inside each checkbox:

[ ] I have read the contribution guidelines.
[ ] Code is formatted via running yarn format.
[ ] Tests are passing via running yarn test.
[ ] The status checks are successful (continuous integration). Those can be seen below.

A picture of a cute animal (not mandatory but encouraged)

Thanks @taras, FYI custom widgets can declare their schema:
https://www.netlifycms.org/docs/custom-widgets/#registerwidget
Custom widgets are registered at runtime, so I'm not sure if we can compile their schema at build time (or maybe we need to require plugin authors to do so).

@erezrokah thank you for bringing this to my attention.
Are these custom widgets 3rd party widgets distributed via npm packages or widgets created by the developer making the site? (or both?) It seems that to make NetlifyCMS work without unsafe CSP both of these groups need to use AVJ CLI or Webpack plugin that converts schemas into compiled functions.

@erezrokah one thing I found with this experiment is that AJV standalone mode doesn't support 2 validations that the config uses: uniqueItemProperties and instanceof. So they need to be added to AJV before the existing schema will compile.

Are these custom widgets 3rd party widgets distributed via npm packages or widgets created by the developer making the site? (or both?)

Both

@taras, another question on this. Can you confirm this fixes the CSP issue? I think we have other dependencies using eval.

@taras, another question on this. Can you confirm this fixes the CSP issue? I think we have other dependencies using eval.

it doesn’t fix the issue completely, we’re looking at the other dependencies now. eval is one issue and ‘new Function’ is another version of it. there are 8 instances of ‘new Function’ coming from netlify-cms bundles. it looks like it might be caused by webpack bundling.

@erezrokah we're starting to push updates to fix these CSP issues. Can you please take a look at https://github.com/netlify/netlify-cms/pull/6009

@taras are you still interested in moving this forward?


[PE-1236] fixed transform python

https://adatao.atlassian.net/browse/PE-1236
Due to a bug in Jython (http://sourceforge.net/p/jython/mailman/message/34658459/), deserializing python bytecode in DDF is not working well.
So clients are gonna send the full source code, DDF will exec() and call the function in TransformPython.
This also support Boolean data types.

Can one of the admins verify this patch?

@Huandao0812 @PangZhi , can you take a look?

retest this please

retest this please

Test passed, merge now.


Fix #909 Add option to provide pre-defined classes for img elements.

Fixes #909 by adding a new option to the image section named availableClasses which can be a list of classes or a list of tuples with class name and label.

Hello. Sorry for delay. We had to finish a big refactoring( Thanks for your PR it's in the new version

Thank you. Looking forward to contribute more stuff :)


Added throw timeout exception when waiting time elapsed before the ad…

…b execution command has exited

Changes is on the master branch


[HPC] add note about pruned logs validity

Added a bullet to hpc training rules throughput measurement section clarifying that pruned logs should be rules-compliant.
Fixes https://github.com/mlcommons/training_policies/issues/493

We need to augment this PR with an issue against mlperf logging. The compliance, RCP checkers and result summarizers need to be made aware of this. For example, we need to define how a submitter specifies pruned logs and need to adjust these tools accordingly.

Copying my discord comment here.
the pruned logs go into a pruned subfolder as before. That's explained in the submission rules.
https://github.com/mlcommons/policies/blob/master/submission_rules.adoc#562-hpc
We may not even need to actually run the checker on them. It's only really needed in the case that someone does HP borrowing for throughput measurement and has pruned logs. An easy thing to do would be to not change the logging package at all in this respect. If needed, we can inspect pruned logs manually or run the checker on them manually.

ok, that makes sense, thank you


Nick/Bailey/Andrew FRQ 1 Review Ticket

[x] Implement FRQ1 in Spring project, use Spring Portfolio mvc/calendar for starter code
[x] Enhance APCalendar.java to provide interesting facts on years, be creative and enhance/finish provided methods. For instance, use a random number or date to complete APCalendar day-of-week method.

[x]  Enhance APIs and @RestController to work with the all POJOs and APCalendar.
Json Formatting within the APIController

Using year object

[x]  Test APIs with Browser, Curl, or Postman. Illustrate or show tangibles.

[x]  Test and use VSCode Debugger in development. Illustrate or show tangibles.

2.9 facts.


[add] russian language

Thanks!

@ymatuhin Will you be updating this? We are interested in having Russian language support and would hate to duplicate your effort.

@adamschwartz @Thunderforge Hey, guys. I'm sorry for waiting, just have a lot of work and lost notification from github about my pull request.
I have some improvements:

Remove magic padding right for .offline-ui (for button)
Remove absolute position for reconnect button
Vertically align blocks in .offline-ui

It allows to use any languages without fear and customise special padding for long words in foreign language. I've tested dark, slide, chrome and default themes. Check it please. Good luck.

Please resolve conflicts


Window MinHeight property doesn't word when set values

Window MinHeight property doesn't word when set values

@Luoyingliang this issue exist in nuget package but This problem seems to be fixed in the Github version so You'll have to wait for the next version of Nagt

thx for feedback
------------------ 原始邮件 ------------------
发件人: "Mahdi<EMAIL_ADDRESS>发送时间: 2020年7月4日(星期六) 晚上10:24
收件人<EMAIL_ADDRESS>抄送: "Acoris"<731984870<EMAIL_ADDRESS>主题: Re: [HandyOrg/HandyControl] Window MinHeight property doesn't word when set values (#425)
@Luoyingliang this issue exist in nuget package but This problem seems to be fixed in the Github version so You'll have to wait for the next version of Nagt
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


TS ignores IF statement. Causes error: "Property X does not exist on type of import(“…”)"

One module exports function "A". Second module exports functions "A" and "B".
At first I need to create a map array, like this:
const map = [
  {
    name: 'A',
    data: moduleA
  },
  {
    name: 'B',
    data: moduleB
  }
];

I'm trying to run functions "A" and "B" in a cycle like this:
item.data.A('some data');
if(item.data && item.data.B)    // here's error
    item.data.B('other data');

But getting this weird error:
Property 'B' does not exist on type 'typeof import(".../moduleA")'
Codesandbox live example: https://codesandbox.io/s/youthful-joliot-qyihx

Simplified repro (in .ts mode, but the same exact thing applies in .js mode): Playground

The short version of this is
let m = { x: 1 }, n = { x: 1, y: 2 };
let a = [m, n];
if (a[1].y) {
  a[1].y
}

Absent other hints, arrays will infer the element type based on the most general type amongst its elements.

@RyanCavanaugh
Working as Intended
No it's not. It should consider if statement and not show this error. As in C# for example, which TS inspired by.

@nmain I found a workaround for this:
Replaced this part of code:
for(const item of map){
  item.data.doThis();

  if(item.data && item.data.doThat)
    item.data.doThat();
}

With this:
for(const item of map){
  item.data.doThis();

  const {
    data = {}
  } = item;

  if(data && data.doThat)
    data.doThat();
}

Seems kind of silly to jump through hoops just to try to fool the compiler, no?  Why not just annotate the type you want?
/** @type Array<{ name: string, data: { doThis(): void, doThat?(): void } }> */
export const map = [ /* ... */

@nmain No. It's not a fooling of the compiler, It's telling to compiler the details he wants and it's more short than providing this looooong typing annotation.
Strange that the compiler is not smart enough to understand such easy type by itself.

@akopchinskiy language design involves trade-offs and this is an intentional trade-off. The behavior is intentional.


nitcorn: skip the listening loop when running the tests

Until now, each nitcorn example checked if it was being tested, and if so it would quit to not block jenkins. This PR moves this check in the nitcorn module so it is the default behavior.
This should help for #1915 and future nitcorn programs.

+1


switching some factors to chars

@rwdudley-usgs I switched the factors to characters in the huc08Poly data. If you pull this down and rebuild you'll have that change.


Website Logo has been resolved KIndly check it

As the issue was to design the logo.
I have designed the logo , so kindly go through it.
Here by im attaching more logo picture if any of you like it we can replace it.

Logos are not good... please create logos on figma and share figma file.

Here can share the figma file so i am previewing the logo in png file. KIndly go through it and suggest your changes.

I didn't get what you said. please share the figma file link.

https://www.figma.com/design/dj2ujMVEbEnisQXNgKOdWg/LOGO?node-id=3%3A3&t=1murT6eOygMTwd6P-1
Here is the link of logo figma file

Logo is really basic, please don't just try getting your PRs merged, try to learn stuff that is more important. Please give some time to logo (you can't do it in 10mins) watch a course on YT. please come up with  a good logo or else I will have to close the PR.

https://www.canva.com/design/DAF9KKQHw60/8pZeuTz1URhFGc5WORQvqw/edit?utm_content=DAF9KKQHw60&utm_campaign=designshare&utm_medium=link2&utm_source=sharebutton
I am highly Proficient in CANVA instead of FIGMA, so i designed in canva.
Here i design some logo kindly go through

Please attempt other Isssues. I think we should close this... really sorry.


Is this faster than panda(qemu)?

Support you, I think this project is great.I hope it will succeed.I am always looking for a debugger that supports system-wide taint analysis.but I only found free tools like panda.Unfortunately, its performance is too bad.When I run some large software in the vm, the whole system is very slow, I can't debug it at all.Finally I found a commercial tool. It is called tetrane.But I asked some people, they said that this tool is not as high-performance as it is advertised.I don't know if there is a debugger that supports full-system taint analysis on the market, and if you run Firefox, the system won't slow down.In addition, I am looking for an open source project, which is implemented by c or c++, and it support forward and backward taint analysis.I found an open source ida plugin. It's called bincat, but it's not c language, I studied it for a few days, headache, and couldn't understand.If you know any information, please let me know, thank you.

You're trying to compare apples to pears I'm afraid.
This is a kernel not a tool per se.
You'd run this on Metal or in a virtualized environment and you'll need to write an Operating System to run with it.


[Doc]: not found vector connector docs

not found vector connector docs

https://db-gpt.readthedocs.io/en/latest/modules/vector.html


Database Transactions

As pointed out by @daussho  in #177 a database transaction feature with automatic rollback would be awesome.
Interesting articles regarding database transactions:
https://en.wikipedia.org/wiki/Database_transaction
https://en.wikipedia.org/wiki/ACID
https://en.wikipedia.org/wiki/Software_transactional_memory

@Timu57 @rakibtg
Did you did further reading about this idea ?!
how about, making temporary files of the query, then verifying if everything went good, just renaming the files with the next id
for example, when trying to make 2 writing in 2 different tables, that are related, just make a temporary file table1.x(id).temp
and make the second writing, table2.x(id).temp, if both operations were successful, then rename the files
table1/256
table2/317
what do you think of this practice


Bugfix/selector no union class name nesting

Small improvement over #346 to allow nesting of @-rules

Thanks for merging 👍

Thanks for the fix, I'll release a new patch version.

https://github.com/kristerkari/stylelint-scss/releases/tag/3.9.2


[1.x] Fix file path in SlowQueries.php AND Exceptions.php

I have windows operating system, it was showing wrong path in both the files.
I replaced the '/' with the DIRECTORY_SEPARATOR constant to ensure consistent directory separation across different operating systems.
before

after

in Exception

Tested locally and working as expected.


Auto-generate columns

Prerequisites

[X] I have searched for duplicate or closed feature requests
[X] I have read the contributing guidelines

Proposal
I have an idea like this: If we set the value for the div tag = 400px:

If the parent element has a width of 900px, the row will have 2 div tags with a width of 450px
If the parent element has a width of 1200px, the row will have 3 div tags with a width of 400px
If the parent element has a width of 700px, the row will have 1 div tag with a width of 700px
Is there a way to write css or use bootstrap to solve the above problem?

Motivation and context
Responsive flexibly

You can accomplish things like this with the current grid system with our auto columns.


Upgade resolutionator

Update homepage and add depends_on

Thank you for the contribution. It was merged directly as fbbe7a153833617b079f53b3ec6664cc91257958 to keep commit history cleaner. Your contribution is still credited to you.


Adjust to move Semantic Conventions out of OTel spec

As part of OTEP #227 we're moving the semantic conventions out of the Specification and into a new repository.
As part of this:

We're leaving the Markdown files for semconv in the specification repository with links to the new location warnings.
Semantic Conventions will be released with their own version number from this repository: https://github.com/open-telemetry/semantic-conventions

We may be re-architecting the structure of this repository, but we can negotiate how best to do so for the website to easily pull it in.

Tasks

[x] #2785
[x] #2918
[x] #2907
[x] #2982
[x] #2987
[x] https://github.com/open-telemetry/semantic-conventions/pull/170
[x] Ensure every section has a README

[x] https://github.com/open-telemetry/semantic-conventions/pull/171

[x] https://github.com/open-telemetry/semantic-conventions/pull/175
[x] https://github.com/open-telemetry/semantic-conventions/pull/177
[x] #2984
[x] #2998
[x] #3008

[x] #3010

[x] https://github.com/open-telemetry/semantic-conventions/pull/187
[x] Ensure that links to OTel spec from semconv pages refer to website pages (/docs/specs/otel)

[x] Waiting on https://github.com/open-telemetry/semantic-conventions/pull/185
[x] #3012

[x] #3033
[x] Ensure that links into semconv from other specs refer to website pages:

[x] #3037

[x] Add semconv version to landing page title - #3033
[x] Add version update to autoupdate script - #3033
[x] Fix links in semconv pages, and drop workaround in this repo: |(/resource/faas\.md)#function-as-a-service|$1|; - https://github.com/open-telemetry/semantic-conventions/pull/180
[x] Ensure "View page source" and other links are working, esp. for index files (originally README.md files)
[x] Address:

[x] https://github.com/open-telemetry/opamp-spec/issues/150 -- until then, this causes a build warning; the warning is skipped by an entry in .warnings-skip-list.txt
[x] Drop skip pattern from .warnings-skip-list.txt (after #3026 is merged)

Other to-dos from #2908:

[ ] Add semantic conventions to ToC / index
[ ] Ensure semantic-conventions -> specification links are accurately internalized to the website
[ ] Ensure specification -> semantic-conventions links are accurately internalized to the website.
[ ] Remove specification/semantic_conventions/*.md from being pulled in OpenTelemetry specification.
[ ] Figure out if supplementary guidelines belong on the website

Note that I've closed the following PR since the semconv pages have moved

https://github.com/open-telemetry/opentelemetry-specification/pull/3498

@chalin Let me know how best we can start moving this issue forward.  Happy to have you at the semconv working group to explain what you need to us, or we can attend whatever SiG/WGs meetings you attend.

@jsuereth - I was OOO. Catching up now. I didn't have the time to work on this before I left. I see that you've submitted a PR. I'll take a look as soon as I can.

Oh, I remember that I had issues regarding the structure of the semconv repo docs. I'll write more about that soon.

Hi @chalin, I volunteered to take up this work on the semconv repo side. Let me know how we can best work together. I'm available on Slack as well.

@jsuereth - I see from the following that you'll be restructuring the spec:

https://github.com/open-telemetry/semantic-conventions/issues/137

Do you want to wait for that restructuring to land before starting to publish the semconv spec on the website?

Hi @joaopgrassi @jsuereth: I see that part of the work has already been done in

#2908

My suggestion would be to bring in the changes incrementally and delay publishing anything to the production server until work on https://github.com/open-telemetry/semantic-conventions/issues/137 is complete. That way we can avoid having to introduce a bunch of redirect rules. WDYT?

In terms of bringing things in incrementally, here's a first PR:

#2918

PTAL

Do you want to wait for that restructuring to land before starting to publish the semconv spec on the website?

Yeah I think that would be best, to avoid double work. I will be helping out in the restructuring so should be "quick"

Questions from https://github.com/open-telemetry/opentelemetry.io/pull/2920#pullrequestreview-1503181544:

How do we get left-hand outline like the spec has for sections?

That will happen automatically once the semconv reorg lands.

Can we change all links to specification in semconv to point at the otel.io copy of the specification?

Yes. I didn't address this yet because the semconv pages currently link directly to v1.21.0 of the OTel spec, whereas the website is now hosting v1.22.0.

Can we change all links to semconv from the specification to point at the otel.io copy of the semconv?

Yes, once we're closer to publishing the new semconv spec, I can work on addressing this.

Hi @chalin the restructure is now done. How should we tackle the website work now?

@chalin We need to decide if we do that change in this release or not.

@chalin Do you think enough work has been done on the Semantic Convention repository that we could cut a release tag to continue the work on the website?  (Showing the version and bumping to the release version).
I'm assuming that's the case from your last PR, but wanted to confirm before we cut a release.

Hi @jsuereth - oh, I missed your request to sync last Friday because I ended early. I'm catching up on notifications now. I'd like to bring in the latest semconv, clean up the scripts and make sure that all is good. Cutting a release is seems reasonable.

The v1.12.1 tag has been cut.

FYI - With this tag, we need to start serving the 1.21.0 file under the schemas/ directory.

Does this mean that all schemas in that directory are to be served? If so, I'll submit a PR doing that now.

Does this also mean that we can publish the semconv pages to the production server?

@jsuereth - while there will be ongoing work to support semconv and other specs, can we close this issue considering that the bulk of the work has been done?

We're well beyond the original migration effort. Closing. Feel free to comment here if you'd like this to stay open.


chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-k8s-25-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2135

Questions ?
Please refer to the Backport tool documentation


Do not let users add multiple paypal.me logins

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

Action Performed:
Context in this PR comment. A user can add multiple paypal.me usernames to their account.

Add paypal.me login
Navigate to settings/payments
Click on "Add new payment method"
Click on PayPal.me
Add a Paypal.me account and save
Click on add new Payment method again
See paypal.me and be able to add another paypal.me login

Expected Result:
A user should not be able to add multiple paypal.me logins. More specifically, the flow should work as follows:

User adds Paypal.me login.
Once paypal.me is added, it is shown in the payment method list
A user can tap the existing paypal.me user name to edit it
Add Payment Method no longer shows paypal.me as an option so long as one exists for the user

Actual Result:
Describe what actually happened
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platform:
Where is this issue occurring?

[x] Web
[x] Mobile Web
[x] Desktop
[x] iOS
[x] Android

Version Number: has not deployed yet
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/22508290/125133125-5d1f8480-e0d3-11eb-92e4-215132589466.mp4
View all open jobs on Upwork

@roryabraham holding off on the auto assigner until we know whether the original PR is adding multiple paypal.me usernames or just overriding the existing user name

From my testing, it seems like the "second" Paypal.me account that can be "added" will replace the old one.

Triggered auto assignment to @JmillsExpensify (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

Hi, I see this issue has External tag, I would like to work on this issue. Here is my proposal:

I will add payPalMeUsername prop to PaymentsPage module and load it from Onyx like this:

https://github.com/Expensify/Expensify.cash/blob/039b671cc6683372317327fde6dc4f9ecb78818b/src/pages/settings/Payments/PaymentMethodList.js#L169-L171

I will use this prop to show/hide Paypal.me menu item here:

https://github.com/Expensify/Expensify.cash/blob/039b671cc6683372317327fde6dc4f9ecb78818b/src/pages/settings/Payments/PaymentsPage.js#L107-L111
Do you want to hide Add Payment Method button if there isn't any available menu item? It doesn't look good right now when we hide Paypal.me menu item now.

To edit existing Paypal.me account addPaymentMethodTypePressed  function will be called here to open Paypal.me account settings page:

https://github.com/Expensify/Expensify.cash/blob/039b671cc6683372317327fde6dc4f9ecb78818b/src/pages/settings/Payments/PaymentsPage.js#L50-L52
https://github.com/Expensify/Expensify.cash/blob/039b671cc6683372317327fde6dc4f9ecb78818b/src/pages/settings/Payments/PaymentsPage.js#L69-L75

By the way, there's a bug in getUserDetails function and it puts an empty string to ONYXKEYS.NVP_PAYPAL_ME_ADDRESS since that API call doesn't return a Paypal.me account name.
https://github.com/Expensify/Expensify.cash/blob/039b671cc6683372317327fde6dc4f9ecb78818b/src/libs/actions/User.js#L64-L82

Do you want to hide Add Payment Method button if there isn't any available menu item?

Yes 👍

By the way, there's a bug in getUserDetails function and it puts an empty string to ONYXKEYS.NVP_PAYPAL_ME_ADDRESS since that API call doesn't return a Paypal.me account name.

Thanks for reporting! It seems like we might need to fix this internally to fix the API. I will investigate and let you know.
But overall @tugbadogan your proposal looks good! Feel free to submit a PR once an Upwork job has been created and you've been hired on Upwork.

Picked this one up for Mills. Job is posted here!

@tugbadogan The Upwork job is ready for you to apply 👍

Thanks @kevinksullivan @roryabraham I applied via Upwork. I will send the PR soon.

Thanks for merging the PR and closing the issue 👍 I am still waiting for the offer on Upwork by the way. cc @kevinksullivan

Hired! Waiting till all relevant PRs have been merged for 7 days, then will make the payment.


My IOS app crashed in release mode.

Environment
React Native Environment Info:
System:
OS: macOS 10.14
CPU: x64 Intel(R) Core(TM) i5-5257U CPU @ 2.70GHz
Memory: 390.63 MB / 8.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 10.11.0 - /usr/local/bin/node
Yarn: 1.10.1 - /usr/local/bin/yarn
npm: 6.4.1 - /usr/local/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
SDKs:
iOS SDK:
Platforms: iOS 12.0, macOS 10.14, tvOS 12.0, watchOS 5.0
Android SDK:
Build Tools: 23.0.1, 25.0.3, 26.0.1, 26.0.2, 26.0.3, 27.0.0, 27.0.3, 28.0.2, 28.0.3
API Levels: 22, 23, 25, 26, 27, 28
IDEs:
Android Studio: 3.1 AI-173.4907809
Xcode: 10.0/10A255 - /usr/bin/xcodebuild
npmPackages:
react: 16.4.1 => 16.4.1
react-native: 0.56.0 => 0.56.0
npmGlobalPackages:
react-native-cli: 2.0.1
react-native-git-upgrade: 0.2.7
react-native-rename: 2.2.2
Description
My app is working perfect in emulator and my phone without release, but when I run react-native run-ios --configuration Release this command, my app crashed. This is my log.

It looks like you are using an older version of React Native. Please update to the latest release, v0.57 and verify if the issue still exists.
The "⏪Old Version" label will be removed automatically once you edit your original post with the results of running react-native info on a project using the latest release.

Not working....

You're using an old RN version, please upgrade first. Also without any repro we won;t help you.

I think problem is due to Mobx.Because I tried new clear project and I could build as release , but when I install mobx, I encountered crash again

Have you tried running with/without Chrome debugger?

Have you tried running with/without Chrome debugger?
I am talking about release mode. In debug mode there is no problem.

Hey I found these. Please check and help me :( @kelset @radko93

I solved with downgrade to 0.55.4.


Fix issues running on wxPython 4.0.7.post2 | remove modifying system ui colours when not using forced dark theme

Context: I'm writing packaging descriptions for yt-dlg for Arch User Repository, and I had issues using the package with the system libraries plus theming issues with dark system themes.
Changes:

On wxPython 4.0.7.post2 (at least on Arch Linux) when buttons are clicked it will throw errors that the Colours are wrong. You have to click them twice to work. Alpha must be specified for all Colours when setting them.
System theme is used correctly in wxPython, so overriding colours with no dark_theme set is unnecessary and causing issues when the system theme is dark theme, so I removed lines setting colours when dark_theme is not set. Regressions:

Different colour for every second row is not in place anymore
Text input colour and some other elements are white with light system theme and forced dark theme (probably not introduced here)
When changing system theme from light to dark the Download list colour doesn't change to dark. It's correct after restart (probably not introduced with this PR either, but I didn't test it).

I'm happy to make changes and when I have some more time I'll have a look at the regressions as well.

Thank very much!


Use a patched version of Cassandra

This uses a version of Cassandra that was built from source. The patch from https://issues.apache.org/jira/browse/CASSANDRA-12942 is the only modification.
This fixes an issue with nodetool status and nodetool ring occasionally not working. The rebalance script, Cassandra monitoring, and service start all rely on those tools.

https://ocd.appscale.com:8080/job/Daily Build/5474/


Google-Sign feature

Can start with this.
You can assign me so that I could start off.
Also either activate google sign in firebase or you can add me<EMAIL_ADDRESS>Just need to ask I need to implement in sign in the page only?

@KejariwalAyush  I have added your email as a member on Firebase Console.
Google Sign Button is already there, just implement the functionality. You can create a separate file for it in the Services folder.

Can I add the functions in firebase_user.dart file as all signing part is done there only.


Develop 221 mozart data

Adds data files and new functionality for all photolysis rate calculations in the MOZART and MOZART-TS1 mechanisms. The cross section configuration format has been modified to allow users to specify cutoff ranges for input data that are applied after interpolation.
The updated documentation is here

Codecov Report
Base: 76.65% // Head: 77.70% // Increases project coverage by +1.04% :tada:

Coverage data is based on head (4759307) compared to base (402bbcf).
Patch coverage: 86.66% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #229      +/-   ##
==========================================
+ Coverage   76.65%   77.70%   +1.04%
==========================================
  Files         105      104       -1
  Lines        7939     7939
==========================================
+ Hits         6086     6169      +83
+ Misses       1853     1770      -83

Impacted Files
Coverage Δ

src/cross_section_factory.F90
42.39% <ø> (+0.28%)
:arrow_up:

src/grid_warehouse.F90
85.83% <ø> (ø)

src/profile_warehouse.F90
89.16% <ø> (ø)

src/radiative_transfer/radiator_warehouse.F90
62.50% <ø> (ø)

src/cross_sections/no2_tint.F90
78.74% <67.79%> (+14.50%)
:arrow_up:

src/cross_sections/tint.F90
87.30% <84.74%> (+15.24%)
:arrow_up:

src/cross_sections/hno3-oh_no2.F90
88.88% <85.10%> (+6.17%)
:arrow_up:

src/cross_sections/rono2.F90
88.42% <88.00%> (+6.60%)
:arrow_up:

src/interpolate.F90
88.88% <91.66%> (+5.44%)
:arrow_up:

src/cross_section.F90
88.05% <91.95%> (+4.15%)
:arrow_up:

... and 13 more

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.


Missing file extension

Hi
While trying to upload a .deb file to minio server, file is getting uploaded without the extension.
Best
Ashutosh

I have a fix and works while testing locally. Would it be possible for you to try this out and let us know if it works for you as well. We can then make a plugin release. You'll need to unzip the file first and then install the .hpi plugin file
minio-storage.zip

Closing as fixed by https://github.com/jenkinsci/minio-storage-plugin/commit/af5ee3a3b3f33f7bdd2e54f3ab4befe1d0672cde . I see the extension is retained after upload in local testing.
@neosis feel free to reopen if you see otherwise


Slight changes in README.md

middleName for profile
Docs improved
Several typo fixed
TravisCI badge on the top

Finally someone who's not just complaining about the docs, but actually fixes them. :)
Great job there, Max! :+1:

@stevebest, Thank you, for the lib) I have saved a little time on VK.com integration;)


Update the installation guide for Databricks users.

The installation guide for Databricks users is out-dated. Let's just refer the official guide.

Thanks! merging :)

Nice


MAJOR BUG VSCODE automatic instance restart on reboot corrupt because VSCODE auto update prompts "Docker update" when in fact it is Docker Desktop NOT Docker!

Type: Bug
MAJOR BUG: At reboot VSCODE once worked to restore 2 or 3 VSCODE instances:  VSCODE WSL2 + Terminal + edit sessions, VSCODE docker Container + Terminal + edit sessions inside the container  and VSCODE Normal Windows Developer project.  VSCODE auto update attempted to install/update "Docker" that turned out to be NOT Docker but Docker Desktop!  Oops!
My Environment:
Install VSCODE and WSL2 with Ubuntu 22 without Docker Desktop (good blogs exist for this because WSL2 runs 3x faster without Docker Desktop and  Docker Desktop google shows 6.8 million reports of malware, crashes and data loss down to Docker Desktop and angry users losing all data).  WSL2 and Docker and VSCODE integrate beautifully without Docker Desktop runs 3x faster and is stable.
Use native Docker with WSL2 not Docker Desktop switching to newer Ubuntu 22.?? Create and run VSCODE and also start VSCODE with views into a container.  Install WSL2 plugins as needed.
Sadly some time later VSCODE update attempts to do an unexpected update stating it is "updating Docker" (which is a lie so you say yes new "Docker" but the next screen pops up installing Docker Desktop ... hiding the fact that in fact it is Docker Desktop update NOT Docker which is strongly known to be very bad and unstable crashing out to lose ALL your data).  The update fails as it is running native Docker, but corrupts the system so containers no longer restart from VSCODE at reboot.  Disrespect is extreme as VSCODE auto update pretends Docker Desktop is Docker calling it Docker!  Naturally we need up-to-date Docker but do not wish instability from something called Docker that is NOT Docker but bloatware that can't integrate or sync with a wet paper bag.
Containers will start manually using docker start and are visible in VSCODE and are usable.
Restart now spews these errors on the VSCODE Container instance every 1 min.
[2235762 ms] Extensions cache, remote removals: None
[2239171 ms] Start: Run in Host: docker exec -i -u dude:dude -w /home/dude/.vscode-server/data/Machine 62152b50150b6e0774741ca633d94af49c686f4f2d9fa25171ec4be00679b7e7 /bin/sh -c # Watch machine settings
[2239177 ms] Start: Run in Host: docker exec -i -u dude:dude -w /home/dude/.vscode-server/extensions 62152b50150b6e0774741ca633d94af49c686f4f2d9fa25171ec4be00679b7e
VS Code version: Code 1.77.3 (704ed70d4fd1c6bd6342c436f1ede30d1cff4710, 2023-04-12T09:16:02.548Z)
OS version: Windows_NT x64 10.0.19045
Modes:
Sandboxed: No
Remote OS version: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2
Remote OS version: Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i7-1165G7 @ 2.80GHz (8 x 1690)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
15.41GB (4.37GB free)

Process Argv

Screen Reader
yes

VM
0%

Item
Value

Remote
WSL: Ubuntu-22.04

OS
Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

CPUs
11th Gen Intel(R) Core(TM) i7-1165G7 @ 2.80GHz (2 x 1689)

Memory (System)
5.80GB (4.82GB free)

VM
0%

Item
Value

Remote
Container dev42:Tim (scan3_alpha)

OS
Linux x64 <IP_ADDRESS>-microsoft-standard-WSL2

CPUs
11th Gen Intel(R) Core(TM) i7-1165G7 @ 2.80GHz (2 x 1689)

Memory (System)
5.80GB (4.82GB free)

VM
0%

Extensions (5)

Extension
Author (truncated)
Version

jupyter-keymap
ms-
1.1.0

remote-containers
ms-
0.288.1

remote-wsl
ms-
0.77.0

makefile-tools
ms-
0.7.0

powershell
ms-
2023.3.3

VSCODE update attempts to do an unexpected update stating it is "updating Docker"

Can you show me a screenshot of this specifically?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


A couple changes to improve the Spark benchmark

Cody actually opened an issue, #4, for these suggestions already, but I thought I'd still go ahead and submit a PR.
Changing groupByKey to a reduceByKey helps reduce shuffle output as well as task memory usage for stage 1. Also, for the final output I simply use foreachRDD/foreachPartition rather than mapPartitions/count which ends up reducing from 3 to 2 stages.
Some of the comments mention using repartition to increase or decrease parallelization. As Cody mentioned, stage 1 parallelization is going to be limited by the number of Kafka partitions. Setting the PARTITIONS environment variable to a higher value definitely improved execution time for me. Using a Spark repartition would come at the cost of increased latency vs just increasing Kafka partitions.

@granturing sorry this has taken so long.  We are trying to pay more attention to this project.
That you so much for the contribution.  I am +1 for merging this in, the only thing that seems to be missing is a single CLA.  If you are still interested in contributing if you could follow the instructions you received earlier on submitting the CLA then I would be happy to merge this in.

CLA is valid!

oops yahoocla just said that it is valid so +1.  Thanks for the contribution.


Add tests

Add colorize tests.
Blocked on determining how to run colorize tests.
http://stackoverflow.com/questions/38667476/adding-colorizer-tests-to-an-extension

I would like to see support for pester tests within vscode output if thats what you mean?

hey @jsloyer, I stumbled on your issue while trying to figure this out for myself on my own extension. I know this was from a while ago, but I figured I'd pass on the info anyways.
You need to add a test file like this to your test directory. This test looks for files in a "/test/colorizer-fixtures" directory to find a file, and then outputs the info in JSON into a new directory colorizer-results. If there's already a file in colorizer-results, it will test the current results against the previously generated results.


Failing onehop test.

In https://github.com/NCATSTranslator/testing/tree/main/onehop we have collected triples from each KP for testing.  The Genetics KP contains this testing triple:
      {
            "subject_category": "biolink:Gene",
            "object_category": "biolink:PhenotypicFeature",
            "predicate": "biolink:gene_associated_with_condition",
            "subject": "NCBIGene:388364",
            "object": "EFO:0004339"
        },

We run this query:
{
    "message": {
        "query_graph": {
            "nodes": {
                "a": {
                    "categories": [
                        "biolink:Gene"
                    ],
                    "ids": [
                        "NCBIGene:388364"
                    ]
                },
                "b": {
                    "categories": [
                        "biolink:PhenotypicFeature"
                    ]
                }
            },
            "edges": {
                "ab": {
                    "subject": "a",
                    "object": "b",
                    "predicates": [
                        "biolink:gene_associated_with_condition"
                    ]
                }
            }
        },
        "knowledge_graph": {
            "nodes": {},
            "edges": {}
        },
        "results": []
    }
}

This returns a response, including several results, but none of the results contain the expected id for b: EFO:0004339

Attn @marcduby

Fixed the json; I'll also write an integration test that draws from this data which will alert me when this becomes an issue again since our data is somewhat fluid due to its computational nature.

Great, thanks!

Issue fixed; closing.


Update sbt-github-actions to 0.13.0

Updates com.codecommit:sbt-github-actions from 0.12.0 to 0.13.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.codecommit", artifactId = "sbt-github-actions" } ]

labels: sbt-plugin-update, semver-minor

Codecov Report

Merging #212 (30fccb9) into master (9a973f1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #212   +/-   ##
=======================================
  Coverage   37.80%   37.80%
=======================================
  Files           3        3
  Lines          82       82
=======================================
  Hits           31       31
  Misses         51       51

Continue to review full report at Codecov.

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


assign out_data to null ptr (DO NOT MERGE)

This is a proof of concept that "fixes" an issue seen in PRs #173 and #166 where the autograd test fails.
This is providing an example for PR #175
DO NOT MERGE ME

See discussion in #175


Channel-specific responses for REST channel with using input_channel in post request

Description of Problem:
Our problem is simular to the topic
Overview of the Solution:
We would like to use one channel like Rest and have possibilities set other input_channel in body of user-message for getting channel-specific utterances.
Examples (if relevant):
A company prefers rest protocol for interactions between systems. But sources of messages is different, with different options for displaying. eg IVR, sms, social media etc
We wouldn't like to implement a lot of channels with simular code. It will be very comfortable, if we could set different input_channels (sources) in message of post request body and get different channel-specific utterances for them

Thanks for submitting this feature request 🚀 @dakshvar22 will get back to you about it soon! ✨

@degiz Can you please take a look at this?