Cross-build to Scala 2.13.0-RC1

Builds on #230
Needs a release of Slick for 2.13.0-RC1.

As long as the new slick version has not yet been released yet, this will not build, so perhaps we should keep this PR open until we are sure we can get it to work.

Yeah, it's passing because it's not being tested.  I'll push the CI update and then we can wait for Slick.

That's more like it
unresolved dependency: com.typesafe.slick#slick-hikaricp_2.13.0-RC1;3.3.0: not found

Let's pick this one up when Slick moves forward.


ServiceWorker error after redirect

I am receiving a serviceWorker error after redirecting the user to updated state. I believe the serviceWorker does not auto refresh when state changes. Maybe I'm missing something here..
serviceWorker.js:81 Uncaught TypeError: Cannot read property 'onSuccess' of undefined
    at ServiceWorker.a.onstatechange (serviceWorker.js:81)
a.onstatechange @ serviceWorker.js:81

Error is here..
function registerValidSW(swUrl, config) {
  navigator.serviceWorker
    .register(swUrl)
    .then(registration => {
      registration.onupdatefound = () => {
        const installingWorker = registration.installing;
        installingWorker.onstatechange = () => {
          if (installingWorker.state === 'installed') {
            if (navigator.serviceWorker.controller) {
              // At this point, the old content will have been purged and
              // the fresh content will have been added to the cache.
              // It's the perfect time to display a "New content is
              // available; please refresh." message in your web app.
              console.log('New content is available; please refresh.');

              // Execute callback
              if (config.onUpdate) {
                config.onUpdate(registration);
              }
            } else {
              // At this point, everything has been precached.
              // It's the perfect time to display a
              // "Content is cached for offline use." message.
              console.log('Content is cached for offline use.');

              // Execute callback
              if (config.onSuccess) {
                config.onSuccess(registration);
              }
            }
          }
        };
      };
    })
    .catch(error => {
      console.error('Error during service worker registration:', error);
    });
}

Environment

macOS Mojave Version 10.14 (18A391)
Chrome Version 69.0.3497.100 (Official Build) (64-bit)
react-scripts v2.0.5

I am redirecting from / to `/home on (see below). ! --- After this occurs is when I am seeing the serviceWorker error.
class Intro extends Component {
  componentDidMount() {
    const { history } = this.props;
    setTimeout(() => {
      history.push('/home')
    }, 5000);
  }
  render() {
    return (
      <div>...</div>
    );
  }
}

export default Intro;

Steps to Reproduce

Visit https://lepage.cc and wait 5 seconds for the redirect.
You'll be redirected to https://lepage.cc/home
Inspect console and you'll see the error.

Error goes away if the user "refreshes" the https://lepage.cc/home page.
Expected Behavior
What I am trying to accomplish is the redirect and have the serviceWorker not error.

 function registerValidSW(swUrl, config = {}) { ...
Or when you init in your code with SW, do register({})

I think there's a bigger issue here actually. I first registered the serviceWorker and then tried unregistering the serviceWorker and noticed that it was not actually unregistered. Then I went back to registering the serviceWorker. So the actual issue is that you cannot unregister the serviceWorker.

@march08, can you provide more information? Please be aware that I did follow the documentation in changing serviceWorker.unregister() to serviceWorker.register().

@march08, can you provide more information? Please be aware that I did follow the documentation in changing serviceWorker.unregister() to serviceWorker.register().

As you can see in the serviceWorker.js
export function register(config) {
Accepts config object, which it tries to read in function registerValidSW(swUrl, config) {, so when you register it somewhere in your app, just do register({}) instead of register()

About 4 weeks ago this was resolved as I changed the service worker back to serviceWorker.register() and it "unregistered" the service worker correctly. Please keep this closed. thank you.


greenplum informatica connector docs link to vmdocs

merge after vmware tanzu greenplum connector for informatica docs are available on docs.vmware.com.

@lisakowen, let me know when this ready to merge, thanks. I tried the new link myself and saw that it's not live yet.

thanks @RichardJJG .  i just pushed the docs live, this is ready to merge.  @dyozie


feat: Update to non-eager FutureListener

Work has been done in https://github.com/googleapis/gax-java/pull/1878.
This is the result of the patch file and migrating the PRs to the GAPIC monorepo.

ci / build (8) job failure:
Run echo "Building Storage lib from generated source..."
Building Storage lib from generated source...
(21:15:16) INFO: Current date is 2022-12-19
(21:[15](https://github.com/googleapis/gapic-generator-java/actions/runs/3735156598/jobs/6338068902#step:10:16):[16](https://github.com/googleapis/gapic-generator-java/actions/runs/3735156598/jobs/6338068902#step:10:17)) Loading:
(21:15:16) Loading: 0 packages loaded
(21:15:[17](https://github.com/googleapis/gapic-generator-java/actions/runs/3735156598/jobs/6338068902#step:10:18)) Analyzing: target @com_google_googleapis//google/storage/v2:google-cloud-storage-v2-java (1 packages loaded, 0 targets configured)
(21:15:17) ERROR: /home/runner/work/gapic-generator-java/gapic-generator-java/rules_java_gapic/BUILD.bazel:7:8: //rules_java_gapic:gradlew: no such attribute 'extension' in '_real_pkg_tar' rule
(21:15:17) ERROR: /home/runner/work/gapic-generator-java/gapic-generator-java/rules_java_gapic/BUILD.bazel:7:8: //rules_java_gapic:gradlew: no such attribute 'files' in '_real_pkg_tar' rule
(21:15:17) ERROR: /home/runner/work/gapic-generator-java/gapic-generator-java/rules_java_gapic/BUILD.bazel:7:8: //rules_java_gapic:gradlew: no such attribute 'mode' in '_real_pkg_tar' rule
([21](https://github.com/googleapis/gapic-generator-java/actions/runs/3735156598/jobs/6338068902#step:10:22):15:17) ERROR: /home/runner/work/gapic-generator-java/gapic-generator-java/rules_java_gapic/BUILD.bazel:7:8: //rules_java_gapic:gradlew: no such attribute 'modes' in '_real_pkg_tar' rule
(21:15:17) ERROR: /home/runner/.cache/bazel/_bazel_runner/1f815583ebc8da6875b9074f31ba9a5d/external/com_google_googleapis/google/storage/v2/BUILD.bazel:93:31: Target '//rules_java_gapic:gradlew' contains an error and its package is in error and referenced by '@com_google_googleapis//google/storage/v2:google-cloud-storage-v2-java'
(21:15:17) ERROR: Analysis of target '@com_google_googleapis//google/storage/v2:google-cloud-storage-v2-java' failed; build aborted: Analysis failed
(21:15:17) INFO: Elapsed time: 1.981s
(21:15:17) INFO: 0 processes.
(21:15:17) FAILED: Build did NOT complete successfully (5 packages loaded, 5 targets configured)
Error: Process completed with exit code 1.

Investigating...

All checks are green except SonarCloud.

Thanks for resolving the issue @suztomo! I've re-ran the Sonar check and it passed -- Seems like it was a transient issue.
CC: @blakeli0 Can you take a look at this when you get a chance?


Using new /macro/folder_documents API endpoint (WL-395)

Getting a list of documents with a folder id will now use the new /macro/folder_documents API endpoint instead of /folders/{id}/documents, which responds with only ids. This avoids the need to make individual requests for each of the documents in the folder.
This is backwards breaking because api.documents.list now expects a folder_id instead of a folderId property.
The response is not backwards breaking. The promise still resolves with an array of objects containing an id property, but the objects now include various other document metadata.

From the documentation:

These comments are function scoped meaning that if you put them inside a function they will affect only this function's code

Ah true! :+1:


Add docs about tested Gateway API/Ingress versions and unavailable features

This patch add docs about tested Gateway API/Ingress versions and unavailable features.
It will be added into release note for each release as well.
Also, the information are centralized into hack/test-env.sh.

/cc @markusthoemmes @dprotaso @ZhiminXiang @carlisia
Here is the preview https://github.com/nak3/net-gateway-api/blob/add-support-matrix/docs/test-version.md (it will be added to the each release note as well.)


function "url_filename" doesn't work as expected

Case
I want to install an app using manifest like this
{
	"homepage": "https://www.bypass.cn",
	"description": "12306 ticket buyer",
	"license": "Freeware",
	"version": "1.13.54",
	"url": "https://www.bypass.cn/api/download?name=Bypass.zip",
	"extract_dir": "Bypass",
	"bin": [ [ "Bypass.exe", "bypass" ] ],
	"checkver": {
		"url": "https://www.bypass.cn/",
		"regex": "Version: ([\\d.]+)"
	}
}

Problem:
It suceessfully download the file in cache, but unable to unzip it properly. It just rename the zip file into "download" and put it in "current" folder.
My discovery
So I run Set-PSDebug -Trace 2 in powershell and reinstall the app to see what happenend.
I found it called a function url_filename in core.ps1 to give the return value to a variable fname, using which scoop will determine whether it's an archieved file.
function url_filename looks like this:
function url_filename($url) {
    (split-path $url -leaf).split('?') | Select-Object -First 1
}

In my case, the input parameter is $url = "https://www.bypass.cn/api/download?name=Bypass.zip", thus causing $fname = download, not Bypass.zip, which makes scoop think it's not a zip file.
I'm not familiar with the download link format, but I can see there is another function called usl_remote_filename just below this one and succeed finding the true name of the file, why not just use this one instead?

{
	"homepage": "https://www.bypass.cn",
	"description": "12306 ticket buyer",
	"license": "Freeware",
	"version": "1.13.54",
	"url": "https://www.bypass.cn/api/download?name=Bypass.zip#/whatever.zip",
	"extract_dir": "Bypass",
	"bin": [ [ "Bypass.exe", "bypass" ] ],
	"checkver": {
		"url": "https://www.bypass.cn/",
		"regex": "Version: ([\\d.]+)"
	}
}

@Ash258 Thanks for the quick response, and it works like a charm!


Add support for IoT Hub quota

Hi, it seems at the moment it's not possible to get the quota of a provisioned IoT Hub resource from within Azure. The plugin already supports various kinds of metrics for IoT Hub but with this addition I think it would allow for a more reliable way of setting up thresholds and alerts from within Grafana (at the moment you could but it would only be by manually setting a threshold on i.e. the max number of messages sent/received).
I am specifically referring to this API: https://docs.microsoft.com/en-us/rest/api/iothub/iothubresource/getquotametrics
If this is something you guys would think it's worth adding it to the plugin I would be happy to contribute to it when I have some bandwidth.
Thank you

@walterdolce please excuse the delay - took me a while to get around to setting up an iot hub in Azure.
The link you posted seems to say that that route returns two metrics - total message and total devices.
Once I added them via diagnostics in the Azure portal then they seem to show up for me:

Or are there extra quota metrics?

Hi @daniellee, thanks for coming back to me on this and no worries!
Yes, I'm referring to an extra quota metrics currently not supported by the plugin. It would be great to have this supported!

@walterdolce can you explain more? I only see two metrics. What are the extra quota metrics that are not supported?

@daniellee apologies for the delay.
I'm referring to these APIs: https://docs.microsoft.com/en-us/rest/api/iothub/iothubresource/getquotametrics
It would be great if those were supported.

I'd just like to add my endorsement for this request.  We do a lot of work with IoT hub an knowing when it is going to hit it's quota would be very useful.


How can I fire the animation from a outside button?

I'm building a tinder like app and would be nice if when the user press the "I don't like it" button for example, the card goes to the left by itself.

Just make a ref to the swipe card component and call _forceLeftSwipe() and _forceRightSwipe() accordingly.

Nice, Worked like a charm! I added this ref to redux store and it's working fine too.
The problem now is that if I update the cards prop right after the click, it won't let the animation finish and already start the "entrance animation" of the next card. Would be nice if we have something like: swipeCardRef._forceLeftSwipe().then(()=> ...); so on then() we would put the code to update the cards array. Or we already have something like that?

Sorry, but you are using the lib in the wrong way!
You shouldn't update the cards list after a swipe. You should let the lib do its job and replace the shown card from the list when the user swipe right/left.

@eyalyoli I'm building an app in tinder style, so, I can't download all possible profiles at once. At the moment I'm downloading 2 profiles per time, so when the user likes or dislike someone, the app will download another profile and store in n array in redux store, that's directly connected to cards prop.
What do you suggest in this use case?

You have a design issue that is not related to the lib. I suggest you load more profiles in advance as 2 is too low.
When you wish to load more profiles, networking on mobile phone can take time (not as fast as your testing env), so you'll have to make a "loading" state for the card deck. After this loading is done it is okay to fire the entrance animation and it is right. This behavior will not change.
I can make an option to cancel the entrance animation, but this is not the right solution for you design issue and it will take time developing (or you can suggest a PR).


Developing on Metadata Ingestion docs missing Cargo requirement

Describe the bug
The Developing on Metadata Ingestion docs at https://datahubproject.io/docs/metadata-ingestion/developing are missing Cargo as a requirement.
To Reproduce
Steps to reproduce the behavior:

Uninstall Cargo (if installed)
Run gradlew :metadata-ingestion:installDev (from https://datahubproject.io/docs/metadata-ingestion/developing#set-up-your-python-environment)
See error

  × Preparing metadata (pyproject.toml) did not run successfully.
  │ exit code: 1
  ╰─> [6 lines of output]

      Cargo, the Rust package manager, is not installed or is not on PATH.
      This package requires Rust and Cargo to compile extensions. Install it through
      the system's package manager or via https://rustup.rs/

      Checking for Rust toolchain....
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

> Task :metadata-ingestion:installDev FAILED

FAILURE: Build failed with an exception.

Expected behavior
Cargo should be listed as a requirement in https://datahubproject.io/docs/metadata-ingestion/developing#requirements

I believe the root cause of this was https://github.com/delta-io/delta-rs/issues/972, which has now been fixed (although it hasn't been released yet)
I'm closing this for now, but @remisalmon let me know if this issue is still present in a week and we can re-open it.


cleanup: fix merge conflict errors

What this PR does / why we need it:
cleanup merge conflict errors in docs
Release note:
NONE

cc: @franknstyle

@akhilerm Thank you.


tvOS support in CocoaPods

Please add support for tvOS in CocoaPods.
Tried your framework and its works perfectly :)

@cherpake thank you! Was fixed in 2.2.


Different version of ruby would give us faster build?

https://travis-ci.org/WGBH/openvault3/builds/114572259#L122:
$ rvm use $(< .ruby-version) --install --binary --fuzzy
ruby-2.2.3 is not installed - installing.
Searching for binary rubies, this might take some time.
Found remote file https://s3.amazonaws.com/travis-rubies/binaries/ubuntu/12.04/x86_64/ruby-2.2.3.tar.bz2
Checking requirements for ubuntu.
Requirements installation successful.
ruby-2.2.3 - #configure
ruby-2.2.3 - #download
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 21.1M  100 21.1M    0     0  77.0M      0 --:--:-- --:--:-- --:--:-- 77.8M
No checksum for downloaded archive, recording checksum in user configuration.
ruby-2.2.3 - #validate archive
ruby-2.2.3 - #extract
ruby-2.2.3 - #validate binary
ruby-2.2.3 - #setup
ruby-2.2.3 - #gemset created /home/travis/.rvm/gems/ruby-2.2.3@global
ruby-2.2.3 - #importing gemset /home/travis/.rvm/gemsets/global.gems................................................
ruby-2.2.3 - #generating global wrappers........
ruby-2.2.3 - #uninstalling gem rubygems-bundler-1.4.4.
ruby-2.2.3 - #gemset created /home/travis/.rvm/gems/ruby-2.2.3
ruby-2.2.3 - #importing gemset /home/travis/.rvm/gemsets/default.gems....................
ruby-2.2.3 - #generating default wrappers........

http://rubies.travis-ci.org/ : If it has to do this, then I don't think we have one of the "preinstalled Ruby versions"
I can't tell exactly what we should aim for: https://docs.travis-ci.com/user/languages/ruby/#Supported-Ruby-Versions

The Ruby VM provides recent versions of:
Ruby: 2.2.x, 2.1.x, 2.0.0, 1.9.3, 1.9.2 and 1.8.7

@mccalluc I would think they're "preinstalled" rubies would be a moving target over the course of months. Are build times a major blocker?

No. So, yeah, if there's not a stable version they'd be likely to have, then it's not worth trying to do anything here. Good call.


WIP rfc8288 obsoletes rfc5988

This PR
updates reference to the latest Web Linking rfc 8288

@whiskeysierra FYI I just received this https://github.com/sdatspun2/deprecation-header/blob/master/draft.md
While these guidelines forbid Link headers, you may be interested in the Deprecation and Sunset header proposal.

@ioggstream Thanks for notifying us. I will take over.

@tkrop Ok, sorry :)

Replaced by #511.

@ioggstream No prob. Thanks for the deprecation link, too. Do you know, whether this has or will be specified in an RFC?

@tkrop they will propose that to the IETF.  Comments are welcome.
if you are interested, I'm working on Digest :P https://ioggstream.github.io/draft-polli-resource-digests-http/draft-polli-resource-digests-http.html


GenIdea: Improved Scala 3 Support

Implement language-level property for Scala library in IntelliJ IDEA.

@informarte I hopefully improved the situation this time.


BoseFS2D

Pull Request Test Coverage Report for Build 695940224

75 of 82 (91.46%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.5%) to 73.707%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Hamiltonians/abstract.jl
1
2
50.0%

src/Hamiltonians/offdiagonals.jl
17
19
89.47%

src/BitStringAddresses/bosefs.jl
23
27
85.19%

Totals

Change from base Build 692501531:
0.5%

Covered Lines:
2024

Relevant Lines:
2746

💛  - Coveralls

This will be done differently, so I'm closing it.


Test cases for decode Signed and Unsigned integers

Any follow ups?
Please review #73 before this one.
Description
Creating unit tests for decoding strings to signed and unsigned integers.
Why?
Continuation of issue #54
How?

Update signed int tests with decode test group
Update unsigned int tests with decode test group

Testing Plan
Run dart test on packages/polkadart_scale_codec folder.

I saw your comments on the last PR @pastre.
I'll update this branch with those comments

Closing this PR because requested changes will be on futures PRs (as #95)


Migrate Trail functionality to an API

Description
The core TIKI publish SDK is written in Dart and is embedded within our SDKs. This leads to complexity on the implementation side. To simplify things for the end user, we'd like to rewrite all this core logic in Rust and deploy it as a regular REST API.
Issues

[ ] https://github.com/tiki/publish-license-upload/issues/134
[ ] https://github.com/tiki/publish-license-upload/issues/130
[ ] https://github.com/tiki/publish-license-upload/issues/136
[x] https://github.com/tiki/publish-license-upload/issues/131
[x] https://github.com/tiki/core-account-service/issues/329
[x] https://github.com/tiki/publish-license-upload/issues/137
[x] https://github.com/tiki/publish-license-upload/issues/135
[x] https://github.com/tiki/publish-license-upload/issues/133
[x] https://github.com/tiki/publish-license-upload/issues/132
[ ] https://github.com/tiki/publish-license-upload/issues/129
[ ] https://github.com/tiki/publish-license-upload/issues/62
[x] https://github.com/tiki/publish-license-upload/issues/61
[x] https://github.com/tiki/publish-license-upload/issues/138
[x] https://github.com/tiki/publish-license-upload/issues/139
[x] https://github.com/tiki/publish-license-upload/issues/140
[ ] https://github.com/tiki/publish-license-upload/issues/141

The following issues need to be resolved to close this out.

Move the transaction signing code to the endpoint - required to create the asset ref between titles and licenses
Update the SQS payload for transactions being compiled in the endpoint
Rust pattern for transactions having multiple schemas - use serde with custom serialization for generics
Move the transactions outside of the block slice


Sample Programs written with olivere/elastic v3

Please use the following questions as a guideline to help me answer
your issue/question without further inquiry. Thank you.
Which version of Elastic are you using?
[ ] elastic.v2 (for Elasticsearch 1.x)
[ ] elastic.v3 (for Elasticsearch 2.x)
Please describe the expected behavior
Hi,
Is any sample programs/projects which is using this package to interact with ES? Am a newbie to both Go & ES . Any lead would be helpful :-)
Please describe the actual behavior
Any steps to reproduce the behavior?

Best advice is to read the Wiki and the tests in the project. There's tons of tests.
Next, you might look up packages that import elastic via godoc.org.

Thanks olivere for ur quick response... I would like know, is it good practice to pool the elastic connections and is it possible through this package ?

You typically create one *elastic.Client per application and share it across your Goroutines. It's thread-safe. Pooling of HTTP connections is done by net/http.

Thanks Again Olivere for both tips 👍 👍


Each window refreshes token

When using the JWT authentication, all open windows issue refresh requests. Only one refresh is needed. Not sure how best to improve this. I was thinking something along the lines of this:

Add a small amount of randomness to when refreshes are made otherwise all requests will be scheduled for the exact same time.
Before making a refresh, check that the token still needs to be refreshed. This check would be in the refresh function that is scheduled.
If the token has already been refreshed (wait > 0), don’t refresh, just schedule the next refresh.

@jpadilla What do you think about attempting to tackle this "issue"? I don't want to add any more complexity than needed and it isn't really an issue, but it would be nice to optimize this behavior.

@fenichelar We can wrap refresh-token call inside an ember-concurrency task as set max-concurrency to 1. Then again, we need to consider we want to depend upon it here.

just commenting to keep issue open

This can be an issue, I have opaque refresh tokens that exist within redis and once used are removed. If one tab wins the request race another tab can receive an expired token response and that will logout all of the tabs.

@evoactivity

I wonder if a service worker could be used to intercept all refresh token requests and return a cached response from the first successful refresh.

Yes. You can extend the JWT authenticator and use a custom refreshAccessToken function: https://github.com/fenichelar/ember-simple-auth-token/blob/master/addon/authenticators/jwt.js#L183-L194.

I've solved this quite easily by adding ember-master-tab and using my own authenticator like so.
import { service } from '@ember/service';
import JWTAuthenticator from 'ember-simple-auth-token/authenticators/jwt';

export default class authenticator extends JWTAuthenticator {
  @service masterTab;

  refreshAccessToken(refreshToken, attempts) {
    const data = this.makeRefreshData(refreshToken);

    return this.masterTab.run(() => {
      return this.makeRequest(this.serverTokenRefreshEndpoint, data, this.headers)
        .then((response) => {
          const sessionData = this.handleAuthResponse(response.json);
          this.trigger('sessionDataUpdated', sessionData);
          return sessionData;
        })
        .catch((error) => {
          this.handleTokenRefreshFail(error.status, refreshToken, attempts);
          return Promise.reject(error);
        });
    });
  }
}

This is the same strategy used by https://github.com/caiorasc/ember-implicit-grant-authenticator to ensure only 1 tab or window controls refreshing.
I think this is something that could be handled by the library if you're willing to take a PR @fenichelar

@evoactivity Thanks for the information. I really don't want to add any dependencies to this library. Because this library is designed to be extended, it should be easy for people to add this if they need it. I will take a look at how ember-master-tab works under the hood and see if I can built something similar internally. If not, then an example and link to ember-master-tab in the readme will probably be the way forward.


Create template for grafana dashboard

@MichaelLi202112 Yikes!  Build has Failed!

@MichaelLi202112 update the readme as well

@MichaelLi202112 Yikes!  Build has Failed!

@rasheedamir Yikes!  Build has Failed!

@faizanahmad055 Validation Successful

Add URL approach to specify json data for GrafanaDashboard

@MichaelLi202112 Yikes!  Build has Failed!

Build failed because someone is playing around with cert manager

@MichaelLi202112 Yikes!  Build has Failed!

Helm doesn't support to access external files until the following PR merged. We can update grafana dashboard template to read json data from external files then.
https://github.com/helm/helm/pull/10077


Cyclosporiasis/ Cyclosporosis

Mondo term (ID Label)
MONDO:0005725 cyclosporiasis
MONDO:0016212 cyclosporosis
Bug/Typo/Error description
Both terms are defined as being caused by infection with  "Cyclospora cayetanensis."
MONDO:0005725 definition is sourced from [ NCIT:C128409 ]
MONDO:0016212 definition is sourced from [ Orphanet:210 ]
UMLS C0343398 "Cyclosporiasis" has two definitions: MSH D021866 says it is "Infection with parasitic protozoa of the genus CYCLOSPORA. It is distributed globally and causes a diarrheal illness. Transmission if waterborne." and NCI C128409 again cites specific infection by Cyclospora cayetanensis.
SCTID: 716860005 "Cyclosporosis" defines that condition as caused by infection with Cyclospora cayetanensis.
I am going to open separate tickets with UMLS and NCI. But the Mondo definitions don't make sense as currently displayed unless these terms are equivalent. Will update here as I get additional info from NCI thesaurus team.
Your nano-attribution (ORCID)
If you don't have an ORCID, you can sign up for one here

My NCI inquiry:

To whom it may concern, This NCI term (C128409 - "Cyclosporiasis" ) seems to be either misspelled or incorrectly defined. UMLS Metathesaurus has mapped this concept to CUI C0343398, which MSH D021866 defines as being caused by any member of the genus CYCLOSPORA. Meanwhile, UMLS C4274225 "cyclosporosis" and SNOMED CT 716860005 define that concept as being attributed to infection by cyclospora cayetanensis.
I hope you can provide some clarity regarding this concept.
Thank you for your consideration and time.  -Megan

NCI's response:

Hi, The definition in question belongs to MeSH. NCI Thesaurus is unable to change it. The other definitions are correct. Are you needing to use the MeSH definition? (NCI Enterprise Vocabulary Services, Nicole Thomas, PhD)

Thanks for reporting this @kanems.
I think these terms are equivalent, I think we should merge MONDO_0016212 cyclosporosis into MONDO_0005725 cyclosporiasis (and use the logical def  below).
'infectious disease'
and ('disease has infectious agent' some 'Cyclospora cayetanensis')

Thanks @pnrobinson!


Open notebook in JupyterLab

Can now open a notebook in JupyterLab by selecting the option to do so in the 3dot menu on the notebook card.
Note: Waiting on more info/understanding of JupyterLab's autosaving semantics from callisto before proceeding

NOTE: this PR is being put on hold until we are able to properly change juypterlab's autosave to match how we do notebooks. Currently lab only autosaves every 2min which is a) slow and b) different from regular notebooks where our autosaving is more complete

@katevoss yes, probably because you don't have lab on your cluster and this doesn't currently include a cluster version bump.


[feature] Getting TTL value of entry

proposing a new method that gets ttl value of an entry.
let val_ref = cache.get(&key).unwrap();
let ttl: std::time::Duration = val_ref.ttl();
let value = val_ref.value();

Hi @ClSlaid, I am busy in recent weeks, would you like to help implement this feature?

Hi @ClSlaid, I am busy in recent weeks, I may have a look this weekend if I have time, would you like to help implement this feature?

It's pretty hard, so many generics go bruhhhh...
I'll see what I can do...


Add support for JSON columns and nullability for string column types

While I'm here, for DATE/DATETIME I guess string is also the only reasonable output.. unless PHPStan has a concept of date-string (which would be kinda neat;)

I guess regarding nullability there is one more complicating aspect.. if you do a LEFT JOIN then any column in the joined table becomes nullable. Is this already taken care of? :s

if you do a LEFT JOIN then any column in the joined table becomes nullable

just verified that the left-join nullability is already supported because of the result-set type inference.
added a test for that in https://github.com/staabm/phpstan-dba/pull/126


[driver] sqla results are not dict-like anymore

See https://pydoc.dev/sqlalchemy/latest/sqlalchemy.engine.row.Row.html
Fixes #295

Bugs prevented by mypy: 0
Hours wasted debugging mypy issues: too damn many

Yikes, @dionhaefner, those issues with SQLite files not getting released has haunted me before - on Linux, though. https://github.com/DHI-GRAS/terracotta/actions/runs/4055501441/jobs/6978731806#step:11:588
Never got to dig deeper because we migrated to use MySQL everywhere.
Wonder how you can force the underlying SQLite library to release the file, like https://stackoverflow.com/a/12679840/512111...

Hallelujah.


Post editor and parser strange behaviour

I am using BB Editor, but the parser translates markdown's "> text" to block quotes.
"?v=videocode" is added after embeds and [youtube]videocode[/youtube] is ignored. And the editor itself looks broken, though can be easily fixed by ".sceditor-toolbar * { box-sizing: content-box; }".
There's also no option to add or edit BB tags without forking the code.

Sorry the BB Editor is no longer supported


Support cert-manager by having 1 cert per secret

/kind feature
Instead of having multiple certs per secret we want to be able to support cert-manager by having 1 cert per secret
/assign @moshloop
/milestone v0.1
/priority important-soon

closed as duplicate


WIP Fix 2427

qiita_script in order to get automated the installation in Travis.

Codecov Report

Merging #2511 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #2511   +/-   ##
=======================================
  Coverage   94.63%   94.63%
=======================================
  Files         160      160
  Lines       19457    19457
=======================================
  Hits        18414    18414
  Misses       1043     1043

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 38ade71...0b659b9. Read the comment docs.


Tool to compare group cats.

We have several metrics we can use so what we should do now is write a module that makes it easy for us to do a full comparison to other group catalogs.
This module would also be very useful for comparing settings that a user may use on their own, single, survey. I.e. use it test how effective changing the linking lengths are or the shelter function.
I imagine that the idea would be to have two group catalogs and then a single galaxy catalog.
We might also need a helper module to get literature group catalogs into the right format. data_handling could probably do that.

Currently working on doing this, will be deploying in the latest release as an alpha version subject to further testing and writing of docs.

Releasing a rough draft in release v1.0.8/v1.0.9.


Added the scan embed that was requested by a guy in discord

That's all.

Thank you for your contribution, the only thing I changed in your code is that it is now a slash command instead of a traditional one.


includeCallerData Field

Given a scenerio , where I have two spring boot micro services . Service A calls Service B , both services are equipped with logstash-logback . Now if includeCallerData is enabled in logback-spring.xml
how it behaves while logging incomplete request from Service A in Service B

The includeCallerData option only refers to the class that calls the logger method.  It does not have any concept of multiple applications.
For example, if you have:
public class MyClass {
    private static final Logger logger = LoggerFactory.getLogger(MyClass .class);

    public static void main(String[] args) {
        logger.info("foo");
    }
}

Then the caller data would be caller_class_name=MyClass, caller_method_name=main, etc.


Error reports after project has closed down.

A project of mine with three drivers regularly spews out pages of errors when I shut it down with Control-C:
DEBUG - animation_threading - Animation complete
INFO - driver - Closing connection to: /dev/cu.usbmodem39
INFO - driver - Closing connection to: /dev/cu.usbmodem23451
INFO - driver - Closing connection to: /dev/cu.usbmodem38
DEBUG - project - Project <bibliopixel.project.project.Project object at 0x1073370b8> finishes run()
INFO - run - Received signal SIGINT
ERROR - driver - Serial exception Attempting to use a port that is not open in write
ERROR - io - Serial exception Attempting to use a port that is not open in read
ERROR - devices - There was an unknown error communicating with the device.
ERROR - driver - Serial exception Attempting to use a port that is not open in write
ERROR - io - Serial exception Attempting to use a port that is not open in read
[... 30 more lines omitted ...]

Never seen that before. Mac thing maybe?
On Wed, Aug 8, 2018 at 7:55 AM Tom Swirly<EMAIL_ADDRESS>wrote:

A project of mine with three drivers regularly spews out pages of errors
when I shut it down with Control-C:
DEBUG - animation_threading - Animation complete
INFO - driver - Closing connection to: /dev/cu.usbmodem39
INFO - driver - Closing connection to: /dev/cu.usbmodem23451
INFO - driver - Closing connection to: /dev/cu.usbmodem38
DEBUG - project - Project <bibliopixel.project.project.Project object at 0x1073370b8> finishes run()
INFO - run - Received signal SIGINT
ERROR - driver - Serial exception Attempting to use a port that is not open in write
ERROR - io - Serial exception Attempting to use a port that is not open in read
ERROR - devices - There was an unknown error communicating with the device.
ERROR - driver - Serial exception Attempting to use a port that is not open in write
ERROR - io - Serial exception Attempting to use a port that is not open in read
[30 more lines omitted]
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/issues/910, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AA6a6ssLIoDQFwgguHAzA9Uw7p2VnF7hks5uOtG4gaJpZM4VzyqV
.


UserSecrets gets odd path for secrets file under 4.7.1 in Windows

Functional impact
This results in the app never finding the secrets file
Minimal repro steps

Create a new "ASP.NET Web Application (.NET Framework)
Choose "MVC" as the template, default settings
Add nuget ref to "Microsoft.Configuration.ConfigurationBuilders.UserSecrets"
update web.config so appsettings appears like this:

 <appSettings configBuilders="Secrets">

Expected result
The app to load and merge the settings in the secrets.xml file with my appsettings
Actual result
Running the app results in it trying to read the secrets file from this location: e:\repro\repro\repro\bin\Microsoft.Configuration.ConfigurationBuilders.UserSecrets.DLL.UserSecretsId.txt

Further technical details
It also does this in my main solution, I checked the code for https://github.com/aspnet/Configuration/blob/dev/src/Config.UserSecrets/PathHelper.cs and have no clue how it is resulting this path.
The EnvironmentConfigBuilder works fine, if that's any help.
I'm using this https://docs.microsoft.com/en-us/azure/key-vault/vs-secure-secret-appsettings#aspnet-and-net-applications and the secretsFile attribute also seems to have no effect. It's also not clear if the file should be json or xml.
Resolution
It seems that specifying UserSecretsId as a folder name to nest my secrets.xml and adding dummy values into the appsettings fixed it.

@simonmurdock I'm running into the same issue. I'm trying to understand how you fixed it exactly. Are you saying you added a folder named UserSecretsId to the solution and moved the secret.xml file in that folder and that's it?

sorry @RoLYroLLs I've since switched to a newer version of .net core, and not run into any issues again

I recall now - <user_secrets_id> is the folder name which needs to exist inside the  normal user secrets ID, once that was specified, it was ok


don't hardcode camera aspect ratio

camera uses hardcoded 800/600 aspect ratio. When viewport is defined, compute the actual aspect ratio. Note that when viewport is not defined, it should still use screen size from engine instead of 800/600. Not sure how to get them here but this PR at least fixes the case when there's a viewport defined.
The problem is that you have to call look_at AFTER defining the viewport. This dependency is not explicit which is not great...

We could delay the perspective matrix computation in Engine::setup_camera to fix the dependency problem.

By the way, i would rewrite this part of camera code after gameObject hierarchy is landed. We do not need to store the camera matrix itself in camera, we should add the camera component to a game object and use its GameObject::transform instead.


fatal: Not a git repository (or any of the parent directories): .git

occur when build from source which direct download.

Try building using mkdir -p bin; go build -o bin/confd .. I'll think about the way to backport a fix into the latest release.

Fix in #635


Missing instructions on using plugins in mbed-host-tests

Description
** Customer request**
It's not clear how to use the plugins included in the mbed-host-tests here:
https://github.com/ARMmbed/mbed-os-tools/tree/master/packages/mbed-host-tests/mbed_host_tests/host_tests_plugins
Docs are very thin and aren't really useful.
https://github.com/ARMmbed/mbed-os-tools/tree/master/packages/mbed-host-tests#plugins
We need to provide basic guidance on how to set up the system to use a specific plugin (STLink preferred) and be able to run tests.
Issue request type
[ ] Question
[ ] Enhancement
[X] Bug

Internal Jira reference: https://jira.arm.com/browse/MBOCUSTRIA-1067

Hi, how is the progress on this?

Currently have had any time to work on this, but I agree this documentation is missing and should be written!
I'll be honest, in general I'm not a huge fan of "plugin" architectures. In my experience they have a tendency to accumulate bugs over time (especially since they are usually optional or have optional dependencies). Do you have any suggestions to improve what we currently have?

Well my goal is to make greentea work with our curstom device.
We went through this previously here.
More recently however we were informed that greentea should work using the plugins, so here we are :)
Regarding your questions.
How that is implemented in mbed tooling doesn't really matter for us customers, I'd guess.
In the end most of us want a out-of-the-box experience, or alternatively a good manual.
What I don't understand now: should we script something ourselves to make the plugins work? Or would the correct command line syntax suffice?
Given the fact that the plugins come within the same repo I'd guess maintenance shouldn't be that problematic. It's just that it includes extra testing, you need to verify each plugin before shipping. The good thing about a plugin architecture is that people can extend it themselves, given that some guidelines are provided on how to do that. Also, code doesn't just rot because it uses a plugin architecture. One example that comes to my mind would be gstreamer. Code rots because it's left unmaintained. If you aren't planning to expose the plugin architecture to the outside world and encourage others to plugin on the tooling then I'd say chose any architecture that suits you, without giving in on functionality that is :)
As far as documentation goes, its always helpful to have a good command  line help menu which exposes all the options, also for accessing plugins. After upgrading mbedhtrun I can now see in the command line help menu the --plugins option + some info on what it does and if it is supported on my system. So that's ok. If I'm not mistaken this is what I'm looking for:
mbedhtrun -c stlink -f /path/to/file/binary.bin
I'd need to verify this on my Windows machine as stlink with linux is not supported currently.

Hi, bridadan.
That seems to be working well!
C:\work-mbed\myfirmware\BUILD\myhw\GCC_ARM-DEBUG>mbedhtrun -c stlink -f myfirmware.bin -p COM14
[1557407884.40][HTST][INF] host test executor ver. 0.0.8
[1557407884.40][HTST][INF] copy image onto target...
STM32 ST-LINK CLI v<IP_ADDRESS>
STM32 ST-LINK Command Line Interface

ST-LINK SN : 37FF6D064D4E383510261143
ST-LINK Firmware version : V2J29S7
Connected via SWD.
SWD Frequency = 4000K.
Target voltage = 3.2 V.
Connection mode : Normal.
Device ID:0x419
Device flash Size : 2048 Kbytes
Device family :STM32F42xxx/F43xxx
Loading file...
Flash Programming:
  File : myfirmware.bin
  Address : 0x08000000
Memory programming...
██████████████████████████████████████████████████ 100%
Memory programmed in 31s and 438ms.
Verification...OK
Programming Complete.
Programmed memory Checksum: 0x06FA02ED

[1557407920.42][MBED][WRN] Target ID not found: Skipping flash check and retry
[1557407920.44][HTST][INF] starting host test process...
[1557407921.35][CONN][INF] starting connection process...
[1557407921.35][CONN][INF] notify event queue about extra 60 sec timeout for serial port pooling
[1557407921.35][CONN][INF] initializing serial port listener...
[1557407921.36][SERI][INF] serial(port=COM14, baudrate=9600, read_timeout=0.01, write_timeout=5)
[1557407921.37][HTST][INF] setting timeout to: 60 sec
[1557407921.39][SERI][INF] reset device using 'default' plugin...
[1557407921.64][SERI][INF] waiting 1.00 sec after reset
[1557407922.64][SERI][INF] wait for it...
[1557407922.64][SERI][TXD] mbedmbedmbedmbedmbedmbedmbedmbedmbedmbed
...

It does have some problems though when the device is in standby mode but I believe that's something we can deal with.


Add a keyboard shortcut for "Add line comment"

Since introducing reviews the CMD/Ctrl+Enter shortcut now adds a Review comment, rather than a single line. This would be fine, but we've been used to this shortcut adding a single line comment for years.

I see your point, but I'm using the new reviews extensively and the new default is more convenient for me. I use Cmd + Enter almost always to start a review and then on subsequent comments I continue using Cmd + Enter to add more comments to that review (the default green submit button then changes to "Add a comment to review").
When I do need to add a single comment I either use the mouse or tab navigation to select the "Add a single comment" button. It's a bit frustrating indeed given the old habit we old have, but the new default is quite productive for me with the new reviews.

👎 I think reviews as default is good. Very often you start with one comment and then you suddenly find more things to comment on and realize you should have done a review. Better to just do reviews all the time.

Okay, well how about adding a new shortcut to create single-line comments?

Sure, that would be useful.

@jbrooksuk Could you suggest a concrete keyboard shortcut? Thanks!

Maybe CMD+Shift+Enter?

@jbrooksuk Do you think you could work on a PR about this?

I have a PR open to add a keyboard shortcut for a single comment, but due to the way the event handling is done in public GitHub (it's different in Enterprise GitHub), I don't think a shortcut containing both ctrl and enter is going to work.  The PR currently uses shiftenter, which is a little wonky, but is at least somewhat related to ctrlenter.  But I'm open to suggestions.

Why is there no option to change the behavior in https://github.com/settings/accessibility ?

@woidda you're commenting on the repo of a browser extension. This feature is now native to GitHub: https://blog.github.com/changelog/2018-07-24-keyboard-shortcut-to-leave-a-single-comment-in-pull-requests-reviews/
For questions about native features, contact GitHub directly.


question about convalution layer

Hi DAA233, I found some differences between your code and Jiahuiyu's code, my question is, why at end of each gen_conv layer in Jiahuiyu's code tensor will be divided into two parts and two different activation functions will be used, do you have any idea about it? because it seems like that you just remove this part
  x, y = tf.split(x, 2, 3);     x = activation(x)     y = tf.nn.sigmoid(y)     x = x * y     return x

Hi @shouldnotfail, sorry for the late response.
This repo is a PyTorch implementation of just DeepFillv1, not including DeepFillv2. Refers to Jiayu's DeepFillv1 code here.


Fix e2e test

Issue #, if available:
Description of changes:
The e2e test builds MIE from MIE's development branch. That branch now uses an MIE layer for python 3.9 instead of python 3.8. So, we need to update the e2e test to validate that the stack outputs include an MIE layer for python 3.9 instead of python 3.8.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

This PR has a trivial change, so I'm skipping the code review.


Could not register mbeans access denied

I am getting the following error when trying to run Elasticsearch with the --user option
main ERROR Could not register mbeans java.security.AccessControlException: access denied
Docker run command:
docker run -d --name ES --user ES:ES  --log-opt max-size=1g -m 3G --cpus="1.0" -v "$PWD/esdata":/usr/share/elasticsearch/data -p 9200:9200 elasticsearch
if i just remove the --user option the error don't show up.
it's running on a Debian host and everything is updated to the latest version.

@jackhold When you specify just elasticsearch as the docker image you are not using the official image of Elastic, described in https://www.elastic.co/guide/en/elasticsearch/reference/current/docker.html. Instead you are using an image provided by Docker Hub.
To use the image maintained by Elastic, you need to use docker.elastic.co/elasticsearch/elasticsearch:5.2.1 or which ever release you wish to target.
Please try going through the documentation here to see examples of how to use the Elastic maintained Elasticsearch Docker image.
As an aside topic, I am not sure what you are trying to achieve by setting the --user option. AFAICS the image shipped by Docker Inc. has an elasticsearch user and group (uid:gid 105/108) but starts as root and then switches to the elasticsearch user in order to mutate your local host dir to match the uid:gid needed by the elasticsearch user in the image.

now that you are saying it... i have worked with ES in docker for a little over 3 weeks and haven't realised there where 2 "official" docker images for Elasticsearch.. I will take a look into the docker image you are releasing just to compare them.
The reason i want to change the user is because the uid and gid is already in use on the host machine so all the files the Elasticsearch is dumping can be accessed by a user that don't need to access them, and when you are debugging on the host machine it's just nice to have a user name that reflects the process.
so yes it might be a more nice to have then need 2 have but i cant see a reason why it shouldn't be possible.

Just to be clear the official image that Elastic supports is the one related to this repo and the documentation mentioned earlier. Eventually the one from Docker Inc. will be deprecated, some details here.
Regarding the user issue:
In general most Docker images run the main process as a non root user (which is a good security practice). For example docker.elastic.co/elasticsearch/elasticsearch:<release> runs elasticsearch as 1000:1000.
Altering the user this runs as is going to create problems as other dirs under /usr/share/elasticsearch are owned by 1000:1000.
Instead you should consider using a named volume to store your data, with something like:
$ docker run -v esdata:/usr/share/elasticsearch/data docker.elastic.co/elasticsearch/elasticsearch:5.1.2
This will create a docker volume, shown with:
$ docker volume ls
DRIVER              VOLUME NAME
local               esdata

that will persist even if you terminate the instance.
If you really want to see where it resides on your localfilesystem you can find the path with docker volume inspect esdata.

Thanks the link to the discussion, looking at the date's it's more understandable that i didn't see that the image docker is distributing isn't in collaboration  with you.. and from what i can read i am goner switch to you're image.
The problem with not having the option to switch the user id is not just the files but if you wanner debug on the host machine and have a user named bob with the UID 1000 all the elasticsearch processes will show up as bob.
but like i said it's a minor problem that i can work around by making my own docker file that changes the userid and the permissions of the folders.
thanks again for you're time and the extra information about the docker image.

I am glad the comments were of use to you! Closing this issue.


Dependency issue every 6 months

To avoid spending too much time on it (1 every two sprints)
If you disagree @Kerollmops, for security or any reason, please close the PR

bors merge


Add md_line Twig filter to expose Markdown::parseLine()

If accepted, also docs should be updated accordingly.

Could you provide an example of this being used and why it would be used?


Date Picker is too picky

OpenUI5 version:
Browser/version (+device/version):
Chrome, Edge Chromium
Any other tested browsers/devices(OK/FAIL):
URL (minimal example if possible):
https://openui5.hana.ondemand.com/entity/sap.m.DatePicker/sample/sap.m.sample.DatePicker
User/password (if required and possible - do not post any confidential information here):
Steps to reproduce the problem:

enter "01/01/2022" gives red box, but this is a completely standard format?!
enter "JAN 1, 2022" gives red box, but "Jan 1, 2022" works - why be case sensitive?!
enter "    01/01/22" gives red box, surely you can call javascript "trim" to remove leading and trailing whitespace!
enter "01.02.2022" gives red box, you should support European formats
enter "Jan 1 2022" without the comma, red box?!

What is the expected result?
All of these test strings should "just work".  Please support all standard formats, It's easy to do using Javascript Date Formats
What happens instead?
great user frustration, resulting in huge number of support requests
Any other information? (attach screenshot if possible)

Hello @lovettchris
Thank you for sharing your feedback. I've created an internal incident __________. The status of the issue will be updated here in GitHub.
Regards,


Tiny tooltip tweaks

:eyeglasses: @keybase/react-hackers

👍

:mag_right: The commits 33d5ac00a663...f0423b49a9db introduced some visual changes on linux:

Changed: Search-Group_non_user_No_Avatar (before) (after)

Changed: Search-Group_non_user_Has_Invite (before) (after)

Changed: Search-Group_non_user (before) (after)

Didn't expect the visual diff here. Working on a fix.

@chrisnojima Could I please have a re-review? Needed to change my approach here to fix the visdiff above.

:+1:


Carbon Black Cloud App: v1.1 release - bugfixes and new actions

Please ensure your pull request (PR) adheres to the following guidelines:

Please refer to our contributing documentation for any questions on submitting a pull request, link: Contribution Guide

Pull Request Checklist
Please check if your PR fulfills the following requirements:

[x] Testing of all the changes has been performed (for bug fixes / features)
[ ] The readme.html has been reviewed and added / updated if needed (for bug fixes / features)
[ ] Use the following format for the PR description: <App Name>: <PR Type> - <PR Description>
[ ] Provide release notes as part of the PR submission which describe high level points about the changes for the upcoming GA release.
[ ] Verify all checks are passing.
[ ] Do NOT use the next branch of the forked repo. Create separate feature branch for raising the PR.
[ ] Do NOT submit updates to dependencies unless it fixes an issue.

Pull Request Type
Please check the type of change your PR introduces:

[ ] New App
[x] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation
[ ] Other (please describe):

Security Considerations (REQUIRED)

[ ] If you are exposing any endpoints using a REST handler,
please document them in the readme.html.
[ ] If this is a new connector or you are adding new actions

[ ] Please document in the readme.html all methods (eg, OAuth) used to authenticate
with the service that the connector is integrating with.
[ ] If any actions are unable to run on SOAR Cloud, please document this in the readme.html.

[ ] Are you introducing any new cryptography modules? If yes, please elaborate their purpose:
[ ] Are you are accessing the file system? If yes, please verify that you are only accessing paths returned through
the Vault API.
[ ] Are you are marking code to be ignored by Semgrep with nosemgrep?
If yes, please provide justification in an additional comment next to the ignored code.

Release Notes (REQUIRED)

Provide release notes as part of the PR submission which describe high level points about the changes for the upcoming GA release.

What is the current behavior? (OPTIONAL)

Describe the current behavior that you are modifying.

What is the new behavior? (OPTIONAL)

Describe the behavior or changes that are being added by this PR.

Other information (OPTIONAL)

Any other information that is important to this PR such as screenshots of how the component looks before and after the change.

Pay close attention to (OPTIONAL)

Any specific code change or test case points which must be addressed/reviewed at the time of GA release.

Screenshots (if relevant)

Thanks for contributing!

@mrangelov-cb apologies, I noticed that the start release job failed once this PR was merged (https://github.com/splunk-soar-connectors/carbonblackcloud/actions/runs/5242387439/jobs/9465726903) because **Unreleased** was not the first line and later because of use of - instead of *.
I pushed both those changes to the next branch (https://github.com/splunk-soar-connectors/carbonblackcloud/pull/7/commits/dbb8181fe59d85b192838a7e7b27f33dc43fd75d and https://github.com/splunk-soar-connectors/carbonblackcloud/pull/7/commits/875d98e95951fcd9c8108133d037cd3c75b2d8b4) and now the start release job successfully created the next -> main PR: https://github.com/splunk-soar-connectors/carbonblackcloud/pull/7
Once you merge that PR then the app will automatically get published to Splunkbase as long as the checks still pass.


Change how to get Dispatcher

This PR:

may fix the issue of drives not appearing on the sidebar #930
fixes the issue of removable drives not always appearing/disappearing from the sidebar when connected/disconnected

Basically it changes how to get the UI Dispatcher

CoreApplication.MainView.Dispatcher -> this often results in System.Exception even when app has started
CoreApplication.MainView.CoreWindow.Dispatcher -> seems ok

Can someone confirm this? (and explain me what the difference is? 😅)
This PR also makes sure that async methods inside a try-catch are always awaited as I don't think that exceptions are caught if the method is not awaited.
try
{
    StorageFolder.GetFolderFromPathAsync(App.AppSettings.OneDrivePath); ----> async without await
}
catch (Exception) ----> the exception can't be caught
{
    App.AppSettings.PinOneDriveToSideBar = false;
}

To properly test this changes i think we are need the way to reproduce this bug.

I usually get it like this:

I open 3-4 instances of the files app
I connect my phone
The phone will not appear in all instances
I'll share a screen of this

CoreApplication.MainView.Dispatcher = The event dispatcher associated with the current view itself.
CoreApplication.MainView.CoreWindow.Dispatcher = Event dispatcher for the Window
My best guess is that the UI dispatcher from MainView is sometimes not ready because the CoreWindow itself is initialized before the ApplicationView is.


Feature/assign sync p rs

Get creator of Tools PR
Add the Tools PR creator alias as a reviewer and assignee.
Move logic for adding members (labels, reviewers and assignees) into the eng\common\scripts\Submit-PullRequest.ps1 script.
Replaces https://github.com/Azure/azure-sdk-tools/pull/1043

The following pipelines have been queued for testing:java - templatejs - templatenet - templatepython - template You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejs - templatenet - templatepython - template You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejs - templatenet - templatepython - template You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

LGTM

/azp run azure-sdk-tools - sync - eng-common

The following pipelines have been queued for testing:java - templatejs - templatenet - templatepython - template You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejs - templatenet - templatepython - template You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejs - templatenet - templatepython - template You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejs - templatenet - templatepython - template You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejs - templatenet - templatepython - template You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

Test Comment

The following pipelines have been queued for testing:java - templatejs - templatenet - templatepython - template You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejs - templatenet - templatepython - template You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejs - templatenet - templatepython - template You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejs - templatenet - templatepython - template You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

/check-enforcer override


php_sapnwrfc-2.0.0-beta2-8.1.0-vs16-ts-x64.dll Unable to load dynamic library

At present we are using the lib in php 7.4 (widows 2012) and we are planing for php 8 upgrade. As part of POC, I tried using the prebuilt windows package on both PHP 8.1.13 and PHP 8.1.0 , but the extension is not loading.
I copied the SAP library that is working in php 7.4 and have setup the path in the new server.

the built is working in windows 2012 server, but not in 2019 server. Any input??

Correct Visual C++ libs?

Try to find out with procmon or similar what library PHP or the extension does not find.

Warning: PHP Startup: Unable to load dynamic library 'php_sapnwrfc.dll' (tried: c:\php\ext\php_sapnwrfc.dll , rest of the extensions are loading ok.  I think I did something wrong while installing the SAP NW RFC library. May I check with you how to extract the lib from the nwrfc750 zip file using sapcar utility. I tried few commands but I got some error like header not found.

As I said, you have to find out which library cannot be found using procmon or similar.


ros: 1.14.7-1 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository ros to 1.14.7-1:

upstream repository: https://github.com/ros/ros.git
release repository: https://github.com/ros-gbp/ros-release.git
distro file: melodic/distribution.yaml
bloom version: 0.6.9
previous version for package: 1.14.6-0

mk

No changes

rosbash
* zsh launch args completion (#217 <https://github.com/ros/ros/issues/217>)
* use _rosrun_ prefix for temporary variable (#228 <https://github.com/ros/ros/issues/228>)
* restore IFS after changing it in rosbash and rosrun (#227 <https://github.com/ros/ros/issues/227>)
* [revent filename glob expansion in _msg_opts in rosbash (#218 <https://github.com/ros/ros/issues/218>)
* correct name of the findpath script (#27 <https://github.com/ros/ros/issues/27>) (#213 <https://github.com/ros/ros/issues/213>)

rosboost_cfg

No changes

rosbuild

No changes

rosclean
* use condition attributes to specify Python 2 and 3 dependencies (#226 <https://github.com/ros/ros/issues/226>)
* use rd on Windows (#224 <https://github.com/ros/ros/issues/224>)

roscreate
* use condition attributes to specify Python 2 and 3 dependencies (#226 <https://github.com/ros/ros/issues/226>)

roslang

No changes

roslib
* use condition attributes to specify Python 2 and 3 dependencies (#226 <https://github.com/ros/ros/issues/226>)
* symlink search for roslib (#225 <https://github.com/ros/ros/issues/225>)

rosmake
* use condition attributes to specify Python 2 and 3 dependencies (#226 <https://github.com/ros/ros/issues/226>)

rosunit
* use condition attributes to specify Python 2 and 3 dependencies (#226 <https://github.com/ros/ros/issues/226>)
* Python 3 support (#212 <https://github.com/ros/ros/issues/212>)

Waiting for catkin release...


react/destructuring-assignment oneliner

Simple and clean oneliner:
  handleClick = () => this.props.firebase.logout();

Right now, I have to write
  handleClick = () => {
    const {firebase} = this.props;
    firebase.logout();
  };

This destructuration is useless in this case. Please remove warnings for oneliner methods where no params are given to the method.

It's definitely not useless; in this case, it doesn't cover the most dangerous problem:
handleClick = () => this.props.foo();
// passes `this.props` as the `this` value, which exposes it to code that could mutate it.

Separately, you never want to put arrow functions in class fields, so I wouldn't recommend that being a one-liner anyways :-)

The props object should not be mutated, but is in no way immutable.
As for arrow functions in class fields, you always want instance methods with a constructor-binding (doing a constructor-bind along with an arrow function is redundant) for performance, clarity, and testability.

  handleClick () {
    const {firebase} = this.props;
    firebase.logout();
  }

it is, then. Looking forward to the rule, please share it in this issue if you don't forget about it by then, thanks :)

@ljharb ...until decorators are also stage 4, then an @autobind on the handler is much less jumping around than binding in constructors.

@jeffvandyke sure. With class fields, you can also do:
foo = this.foo.bind(this);
foo() { }


Relative wrapper

This is a multi functional PR.
First it changes all tests to use the urllib syntax file:wrapper for relative paths. This requires snakemake >=3.10.0.post1. At the same time I also went through and pinned the requirements.

awesome, thanks!


fix: a bug, when csr is true, actually there is no input

#6777
What is it?

[ ] Feature / enhancement
[ ] Bug
[ ] Docs / tests / types / typos
[ ] Infra

Description

Checklist:

[ ] My code follows the developer guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have ran pnpm change and documented my changes
[ ] I have made corresponding changes to the Qwik docs
[ ] Added new tests to cover the fix / functionality

this error because it can't be null
Object.values(null)

VM790:1 Uncaught TypeError: Cannot convert undefined or null to object
at Function.values ()
at :1:8

I checked it after modified

@wmertens May I ask you to take another look? The qwik template in create-vite is broken for a while.

@wmertens May I ask you to take another look? The qwik template in create-vite is broken for a while.

This Pr will cause another problem?

@wmertens May I ask you to take another look? The qwik template in create-vite is broken for a while.

Has This caused another problem?

@JerryWu1234 No. The qwik template has been broken for a while because of https://github.com/QwikDev/qwik/issues/6777 and I found that this PR.


Immutable client

Extension of #198. @kelunik thoughts on the last commit?

:+1: Yeah, immutable client would be really nice. I could then have it as a service in my dependency injection container and add additional interceptors as needed without touching the service itself. Ofc I can still do it with cloning but this feels better.

@trowski Could you modify this PR to only contain the immutability change, please? I'd like to get this merged, as #200 will probably depend on this.

@kelunik So remove the follow redirects portion? I think that should be done as well, but I can put that in another PR after this is merged. I can rebase after #198 is merged, since that has diverged a bit now too.

Rebasing on master was simple, so I did that quickly. Will need to rebase #198 on this after merging.


[Feature]: Scrolling window alternative to pagination for large SQL data-sets

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
The current pagination method is awkward when working with large datasets,  but where page boundaries obscure the application's logic.  For example my app groups articles with printed pages.   I need to scroll so that the articles on a printed page are visible together.   If I used pagination,  a page boundary might occur in the middle, and make it hard to see them together.
I envisage the system maintaining a scroll window and prefetching data immediately before and after.  Data would be presented with scroll bars within a  table. The traditional "run" would be replaced by an "invalidate" that told the framework it needed to re-fetch data.
Why should this be worked on?
Pagination doesn't fit well when the data viewed might have local coherence (i.e. all the articles on a printed page) and a pagination page boundary just happens to occur in the middle, thereby destroying the visual coherence of the data.    The alternative currently supported is just to scroll the entire data-set,  but this introduces unnecessary delays when there is a lot of data,  because time is spent transferring data that might never appear on screen.

We're tracking this feature here #2896


Added my version of joke generator by following the given guidelines-…

…Komal Reddy
Developer Checklist

[x] Followed guidelines mentioned in the readme file.
[x] Followed directory structure. (e.g. ProjectName/{USERNAME}/...yourfiles)
[x] Starred ⭐ the Repo (Optional)

Summary
"Joke Generator" app is a simple web application that fetches a random programming-related joke from an API when the user clicks the "Generate Joke" button. The fetched joke is then displayed on the webpage, and a laughter sound effect is played shortly after the joke is displayed. The app provides a lighthearted and entertaining experience for users, combining programming humor with an audio element for added fun.
Screenshot

Responsiveness in mobile

Responsiveness in laptop

Live Project Link
https://komalreddy3.github.io/JokeGenerator/

Updates are done as requested
Laptop responsiveness

Mobile responsiveness

Live Project Link
https://komalreddy3.github.io/JokeGenerator/

I have done the changes as requested , please let me know if further
changes are to be made
On Sun, 31 Dec 2023 at 05:31, Nitkarsh Chourasia @.***>
wrote:

@.**** requested changes on this pull request.
The fonts and the background should be worked upon.
Visibility of the words is not good.
Please, look to it and I will merge it.
Thank you.
—
Reply to this email directly, view it on GitHub
https://github.com/thinkswell/javascript-mini-projects/pull/1098#pullrequestreview-1799596985,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATUMHYLKATU7TJCLRH5O723YMCTL5AVCNFSM6AAAAABA6BJ3TGVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTOOJZGU4TMOJYGU
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

Thank you so much
On Mon, 1 Jan, 2024, 6:16 pm Nitkarsh Chourasia, @.***>
wrote:

@.**** approved this pull request.
—
Reply to this email directly, view it on GitHub
https://github.com/thinkswell/javascript-mini-projects/pull/1098#pullrequestreview-1799796692,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATUMHYOE37APV6VQICW22LDYMKV3TAVCNFSM6AAAAABA6BJ3TGVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTOOJZG44TMNRZGI
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

@komalreddy3 Also do consider adding this.

@komalreddy3
It's included in the Repositories's README.md file.
Thank you
@NitkarshChourasia


the icon for changing your accounts

Describe the bug
The icon and text doesn't look right in the status menu
Screenshots

Putting this on hold for a future update

Moving this to #165


Update quickstart documentation with new 'Commonly Reported Solutions…

Add "Commonly Reported Solutions" section to quickstart guide
Motivation and Context
Added a new section to the troubleshooting guide that addresses frequently reported issues and their solutions, making it easier for new users to resolve common setup problems.
How Has This Been Tested?

[x] Verified documentation renders correctly
[x] Confirmed all links are valid
[x] Checked formatting and markdown syntax

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Documentation update

Checklist

[x] I have read the MCP Documentation
[x] My code follows the repository's style guidelines
[x] I have added or updated documentation as needed

Additional context
This addition helps reduce common support requests by providing solutions to frequently encountered setup issues upfront.

Added a section for commonly reported solutions to issues that arise when installing MCP.
This will increase the value of the documentation by enabling easy access to common solutions. This additional context will provide increased efficiency in troubleshooting. Humans and AI models will benefit from this additional section by giving them pointers for where to look first when troubleshooting common issues.
Instead of adding "common issues" which promotes rabbit holes, we call it "common solutions" to promote taking action.

Hey @ctavolazzi, thanks so much for doing this!
Overall, this looks great - just added a quick comment about a potential typo.


Cost associated with Message Passing

Possibility of HP decay with message passing to disincentivize screaming 24/7

Alternative - limiting the message sending to < X messages a day(some time period)?

Alternative - limiting the message sending to < X messages a day(some time period)?

Definitely works, but should probably differentiate between message types (assuming we have the shout vs talk), otherwise agents will always shout, compared to having to weigh up the method with the HP drain


Number types do not set 'min' and 'max' atributes

When my field is generated as a type number with a minimum and maximum, the min and max correctly validate input, but the min and max attributes aren't set on the input, so the up/down arrows still let a user go above/below the boundaries.

Hi @otherkade, good idea!
All templates with an input as base use the same template. I think the number types maybe  should be moved to their own template.

Moved this to the bootstrap repo:
https://github.com/json-schema-form/angular-schema-form-bootstrap/issues/18

This has been done and added to webpack (v1.0) branch


cmake c++ build for Windows Visual Studio

Thank-you for this repo. It is a great addition to TensorRT.
I use Windows 11, Visual Studio 2022 and cmake for c++ development. I would like to be able to build the c++ folder using just these tools.
Right now, with v0.6.1, this is a bit painful. The main issues to clear up are:

Finding the TensorRT root directory:  This is a trivial task in cmake. Right now the code is Linux only.
The coarse workaround I use to get the build to work is:

# TRT dependencies
set_ifndef(TRT_LIB_DIR "C:/Program Files/NVIDIA GPU Computing Toolkit/TensorRT/v<IP_ADDRESS>/lib")
set_ifndef(TRT_INCLUDE_DIR "C:/Program Files/NVIDIA GPU Computing Toolkit/TensorRT/v<IP_ADDRESS>/include")

NCCL Dependencies/Setting NCCL_LIB: This is not available for Windows from reading through the readme files.
The coarse workaround I use:

set_ifndef(ENABLE_MULTI_DEVICE 0)
I set the NCCL_LIB to c:/Program Files as a placeholder in the cmake project configuration file.
My ask is to be able to build the cpp folder using cmake without depending on the python build wheel script from the parent directory.

Additionally...
With the above changes, I can compile the sources, but with a debug build the link stage fails due to the batch_manager static library is non-debug:
E:\GitHub\TensorRT-LLM\cpp\out\build\x64-Debug\tensorrt_llm_batch_manager_static.lib(kvCacheManager.cpp.obj) : error LNK2038: mismatch detected for '_ITERATOR_DEBUG_LEVEL': value '0' doesn't match value '2' in assert.cpp.obj
E:\GitHub\TensorRT-LLM\cpp\out\build\x64-Debug\tensorrt_llm_batch_manager_static.lib(kvCacheManager.cpp.obj) : error LNK2038: mismatch detected for 'RuntimeLibrary': value 'MD_DynamicRelease' doesn't match value 'MDd_DynamicDebug' in assert.cpp.obj
     Creating library tensorrt_llm\tensorrt_llm.lib and object tensorrt_llm\tensorrt_llm.exp

Related to https://github.com/NVIDIA/TensorRT-LLM/issues/618

@ToddThomson Do you still have the problem? If not, we will close it soon.


added number of bins parameter

Description
This is to support an alternative way of obtaining a histogram from a Quantiles Sketch using the DoublesSketchToHistogramPostAggregator. The parameter "splitPoints" was made optional, and another parameter "numBins" was added. The new parameter is optional as well, but it has a default of 10. Therefore if both parameters are omitted, the post agg must produce a histogram with 10 equally-spaced bins between min and max values in the sketch.
I am not familiar with SQL adaptors, so I did not add this feature there.

[x ] added documentation for new or modified features or behaviors.
[x ] added unit tests or modified existing tests to cover new code paths.

Hmm, I'm not sure how you'd want to adapt this into SQL anyway. Return a string that represents the histogram, maybe?

There is some way currently, but using the split points. The result must be an array of values that represent bins.

I am not sure why this was forgotten.


copr: fix wrong function cast double to double

Signed-off-by: lzmhhh123<EMAIL_ADDRESS>
What problem does this PR solve?
Issue Number: close https://github.com/pingcap/tidb/issues/25200
What is changed and how it works?
What's Changed: cast_real_to_signed_real and cast_real_to_unsigned_real is wrong.
Related changes

Need to cherry-pick to the release branch

Check List
Tests

Unit test
Integration test

Release note
fix wrong function cast double to double

/merge

/merge

/merge

/needs-cherry-pick-5.1

/label needs-cherry-pick-5.1

/label needs-cherry-pick-5.0

/run-cherry-picker

/run-cherry-picker


unknown network: mainnet

With 4.2.0, bcd fails to load with:
GET http://localhost:14000/v1/config 400 (Bad Request)
The response for this request is: {"message":"unknown network: mainnet"}.
Using a flextesa ithaca sandbox.

Awesome, thanks!

woops, actually, now I get this (on pr-913):
In js console: GET http://localhost:14000/v1/head 500 (Internal Server Error)
api log:
api_1        | [GIN] 2022/05/20 - 15:25:21 | 200 |       32.79µs |      <IP_ADDRESS> | GET      "/v1/config"
api_1        |
api_1        |
api_1        | 2022/05/20 15:25:21 [Recovery] 2022/05/20 - 15:25:21 panic recovered:
api_1        | GET /v1/head HTTP/1.1
api_1        | Host: localhost:14000
api_1        | Accept: application/json, text/plain, */*
api_1        | Accept-Encoding: gzip, deflate, br
api_1        | Accept-Language: en-GB,en-US;q=0.9,en;q=0.8
api_1        | Connection: keep-alive
api_1        | Dnt: 1
api_1        | Origin: http://localhost:8000
api_1        | Referer: http://localhost:8000/
...
api_1        | Sec-Fetch-Dest: empty
api_1        | Sec-Fetch-Mode: cors
api_1        | Sec-Fetch-Site: same-site
api_1        | User-Agent: ...
api_1        |
api_1        |
api_1        | runtime error: invalid memory address or nil pointer dereference
api_1        | /usr/local/go/src/runtime/panic.go:220 (0x449915)
api_1        | /usr/local/go/src/runtime/signal_unix.go:818 (0x4498e5)
api_1        | /go/src/github.com/baking-bad/bcdhub/cmd/api/handlers/head.go:31 (0x108425a)
api_1        | /go/pkg/mod/github.com/gin-contrib/cache@v1.1.0/cache.go:164 (0x10a7235)
api_1        | /go/pkg/mod/github.com/gin-gonic/gin@v1.7.4/context.go:165 (0x1086e92)
api_1        | /go/src/github.com/baking-bad/bcdhub/cmd/api/handlers/middleware.go:52 (0x1086e79)
api_1        | /go/pkg/mod/github.com/gin-gonic/gin@v1.7.4/context.go:165 (0xae3486)
api_1        | /go/pkg/mod/github.com/gin-gonic/gin@v1.7.4/logger.go:241 (0xae3469)
api_1        | /go/pkg/mod/github.com/gin-gonic/gin@v1.7.4/context.go:165 (0xae4361)
api_1        | /go/pkg/mod/github.com/gin-gonic/gin@v1.7.4/recovery.go:99 (0xae434c)
api_1        | /go/pkg/mod/github.com/gin-gonic/gin@v1.7.4/context.go:165 (0xae253d)
api_1        | /go/pkg/mod/github.com/gin-gonic/gin@v1.7.4/gin.go:489 (0xae21c5)
api_1        | /go/pkg/mod/github.com/gin-gonic/gin@v1.7.4/gin.go:445 (0xae1d24)
api_1        | /usr/local/go/src/net/http/server.go:2916 (0x6fddba)
api_1        | /usr/local/go/src/net/http/server.go:1966 (0x6f8db6)
api_1        | /usr/local/go/src/runtime/asm_amd64.s:1571 (0x463c00)

Not sure if this is the same issue, maybe not.

It's very similar bug. I fixed it in this PR

Hope you don't mind me re-using this same issue.
On pr-914 I now get:
GET http://localhost:14000/v1/account/sandboxnet/tz1VSUr8wwNhLAzempoch5d6hLRiTh8Cjcjb 404 (Not Found)
GET http://localhost:14000/v1/account/sandboxnet/tz1VSUr8wwNhLAzempoch5d6hLRiTh8Cjcjb/token_balances?offset=0&size=1 404 (Not Found)
GET http://localhost:14000/v1/contract/sandboxnet/tz1VSUr8wwNhLAzempoch5d6hLRiTh8Cjcjb/operations?with_storage_diff=false 400 (Bad Request)
Didn't see anything specifically wrong in the logs, on these request it just reports:
api_1        | [GIN] 2022/05/20 - 18:59:30 | 200 |       27.13µs |      <IP_ADDRESS> | GET      "/v1/config"
api_1        | [GIN] 2022/05/20 - 18:59:30 | 404 |    7.803315ms |      <IP_ADDRESS> | GET      "/v1/account/sandboxnet/tz1VSUr8wwNhLAzempoch5d6hLRiTh8Cjcjb/token_balances?offset=0&size=1"
api_1        | [GIN] 2022/05/20 - 18:59:30 | 404 |     9.25483ms |      <IP_ADDRESS> | GET      "/v1/account/sandboxnet/tz1VSUr8wwNhLAzempoch5d6hLRiTh8Cjcjb"
api_1        | [GIN] 2022/05/20 - 18:59:30 | 204 |    8.122554ms |      <IP_ADDRESS> | GET      "/v1/account/sandboxnet/tz1VSUr8wwNhLAzempoch5d6hLRiTh8Cjcjb/metadata"
api_1        | [GIN-debug] [WARNING] Headers were already written. Wanted to override status code 400 with 404
api_1        | [GIN] 2022/05/20 - 18:59:30 | 404 |       57.78µs |      <IP_ADDRESS> | GET      "/v1/contract/sandboxnet/tz1VSUr8wwNhLAzempoch5d6hLRiTh8Cjcjb/operations?with_storage_diff=false"

(yes, that account should exist).

You have to deploy or call at least one contract using  tz1VSUr8wwNhLAzempoch5d6hLRiTh8Cjcjb. BCD indexer finds addresses when they communicate with a contract. Flextesa addresses are not hardcoded at indexer. If you deploy any contract BCD will be work.

Makes sense, thanks!


Reduce size/dependencies?

You could remove is-typedarray?
and use offset+length from the start instead of slicing afterwards
module.exports = function typedarrayToBuffer (arr) {
  return ArrayBuffer.isView(arr)
    ? Buffer.from(arr.buffer, arr.byteOffset, arr.byteOffset + arr.byteLength)
    : Buffer.from(arr)
}

I think it should actually be:
module.exports = function typedarrayToBuffer (arr) {
  return ArrayBuffer.isView(arr)
    ? Buffer.from(arr.buffer, arr.byteOffset, arr.byteLength)
    : Buffer.from(arr)
}

But you're right. The browser support has improved enough that we can probably simplify this package a lot!

Released as 4.0.0


What and how to use SingleSourceoftruth in case of network call only and no Database.

@Eli-Fox , I am having a project in which only network calls are to be made and no use of local DB. So how to follow that approach in this project? How will SingleSourceofTruth file will work?

you can revamp into like this
fun <T> resultLiveData(networkCall: suspend () -> Result<T>): LiveData<Result<T>> =
    liveData(Dispatchers.IO) {
        emit(Result.loading<T>())
        val responseStatus = networkCall.invoke()
        if (responseStatus.status == SUCCESS) {
            emit(Result.success(responseStatus.data!!))
        } else if (responseStatus.status == ERROR) {
            emit(Result.error<T>(responseStatus.message!!))
        }
    }

in my project i didnt have db too, after revamp to that, it will work fine

This is my version similar to @fjr619 @shahzadhemani17 you can follow anyone.
fun <A> resultLiveData(
                          networkCall: suspend () -> Result<A>
): LiveData<Result<A>> =
        liveData(Dispatchers.IO) {
            emit(Result.Loading())
            val responseStatus = networkCall.invoke()
            if (responseStatus is Result.Success) {
                emit(Result.Success(responseStatus.data))
            } else if (responseStatus is Result.Error) {
                emit(Result.Error(responseStatus.message))
            }
        }

I expect that the data could be null, so I am not asserting it with not null.


[CWE-690] in beval.c dict.c ex_docmd.c ex_eval.c list.c ops.c option.c quickfix.c regexp.c and scriptfile.c

Hello again :smile:. Continuing my research from #7200 I found a lot of null-dereference issues. Here what I found (I left only [CWE-690]) :
vimlog2.txt
To Reproduce
Detailed steps to reproduce the behavior:

Clone the vim repo: git clone https://github.com/vim/vim.git
Compile with -fanalyzer flag: env CFLAGS=-fanalyzer make 2> /tmp/log.txt
Read /tmp/log.txt

Environment (please complete the following information):

Vim version : Latest
Arch Linux

I don't see any commands you are giving to Vim.
If this is just a static analyzer then the warnings are probably false positives.

I had a quick look at beval.c.

Note: It is taking the false branch here (VIsual_active not being set).
    |......
    |   74 |       col = vcol2col(wp, lnum, col);
    |      |             ~~~~~~~~~~~~~~~~~~~~~~~
    |      |             |
    |      |             (17) ...to here
    |......
    |   77 |       if (VIsual_active
    |      |          ~~~~~~~~~~~~~~
    |      |          |
    |      |          (18) following ‘true’ branch...

Note: VIsual_active here is now true.
    |   78 |        && wp->w_buffer == curwin->w_buffer
    |      |        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    |      |        |    |
    |      |        |    (19) ...to here
    |      |        (20) following ‘true’ branch...
    |   79 |        && (lnum == spos->lnum
    |      |                    ~~~~~~~~~~
    |      |                        |
    |      |                        (21) ...to here
    |      |                        (22) dereference of NULL ‘spos’

This cannot happen. If VIsual_active is true, than spos must have been set.
I had a quick look at some of the others, but there were too many and some are not really easy to follow. I am not sure this all makes sense.


Dart analyzer should give an error for a final field without initializer

The following code fails with compilation error on execution:
final int v;

main() {
  print(v);
}

Output:

dart --checked final_or_static_initialization_t01.dart
'file:///D:/dev/dart/co19/Language/Variables/final_or_static_initialization_t01.dart': error: line 16 pos 11: missing initializer for final or const variable
final int v;

This is expected behavior for Dart VM, however Dart Analyzer produce only warning instead of error:

dartanalyzer.bat --strong --enable_type_checks final_or_static_initialization_t01.
dart
Analyzing [final_or_static_initialization_t01.dart]...
[warning] The final variable 'v' must be initialized (D:\dev\dart\co19\Language\Variables\final_or_static_initialization_t01.dart, line 16, col 11)
1 warning found.
dartanalyzer.bat  --enable_type_checks final_or_static_initialization_t01.dart
Analyzing [final_or_static_initialization_t01.dart]...
[warning] The final variable 'v' must be initialized (D:\dev\dart\co19\Language\Variables\final_or_static_initialization_t01.dart, line 16, col 11)
1 warning found.

I believe, Dart Analyzer should behave the same way (i.e. produce error) as Dart VM in this case.
Dart VM version: 1.19.1 (Wed Sep 07 09:07:46 2016) on "windows_x64"

The analyzer now reports an error:

error • The final variable 'v' must be initialized. • 27510.dart:1:11 • final_not_initialized


Call trim-low-abund from kevlar partition

In combination with the recent kevlar reaugment bugfix, I'm hoping this improves some assemblies. In any case, it should make it easier to compose the entire workflow.

Closing as unnecessary now with #156.


Images height is wrong

Different sized images don't scale properly

According to the example the width should adjust to the height, not the other way around

Closing this as this is basically a duplicate of #271


Register customer with existing administrator user data

Summary
Much same as #358 but for the register customer case.
For registering a new customer the initial organization admin is created with an existing user in the OP by overtaking his user data.
Details
It's only UI implementation (rest is part of #358)

When creating a new customer  in SSO mode, the administrator only provides a User ID for the initial customer administrator user. Other fields are removed from Administrator data section in the customer registration page.
User details are retrieved from the oscm-core IdentityService (mocked):
VOUserDetails IdentityService#loadUserDetailsFromOIDCProvider(String userId, String tenantId)

Acceptance criteria

[ ] No user data input on create oranization page
[ ] Initial organization admin is created successfully with JSON data from TenantService
[ ] email is sent out the configured administrator email address
[ ] User details are validated and user-friendly messages are returned

Additional context
Later

https://github.com/servicecatalog/oscm/pull/364

https://github.com/servicecatalog/oscm/pull/367


So sad this project was discontinued

It saved my life many times, so easy to integrate native code into cordova, does anyone know about a similar project?

+1

@carnivash Yes. There is a similar project called Tabris.js and what is important an active and matured one. They've just released beta of 2.0 version. As you can learn from this article below, it offers quite similar capabilities to Ace. It uses Cordova's plugin API to provide an access to native UI elements.
I can't say much more than I read about it, but please share your opinion when you compare both Tabris and Ace.

@baldram looks nice, but unfortunately its native components are closed source.

@cordovapolymer I'm not that sure. Let's take a look on this plugin: https://github.com/eclipsesource/tabris-plugin-maps . You have full Android native source, not only JS examples.
So even if there are no source for the core part, you have at least a control on this and other 3rd party add-ons. Plus possibility to create own ones. So you are able to build your own solution on the top of Tabris.
When considering this technology, we can see that EclipseSource is a foundation with a long history.
There is a growing community, the Github project and Slack support is very active. They have built a business around this solution. Finally it looks like they simply develop it for own projects. I think they are trustworthy if you like this solution.
I must admin I have no interest to write this everything. Those are just my observations I want to share.
The decision is yours ;)

Why go for an alternative, when you could still use ACE? I think with ACE being open source as it is, anyone could fork it and pick up where Microsoft left off, and continue development with it. Right?

@uniphonic , nobody did it so far https://github.com/Microsoft/ace/network

Can someone please pick this up? It's a genius initiative. Tabris.js is a good effort, but I refuse to pay their rates or be limited to one private GIT repo.  These companies need to realise that we want the dev tools for free and are prepared to pay for back-end services, like hosting, analytics etc... and even the deployment platform. The IDE and build tools should be free, with a cloud build option, the way Nativescript Sidekick and React Native's Expo does...

+1

+1 for @delanick

@delanick, thoughts like yours make that projects like this are never going to be rescued. Develop this framework is a huge effort, so, following your thoughts, if someone take care of they are not making money if they don't work much more creating a building cloud... of course they will have to have a free plan with unlimited projects... all that for you to make money using their work... wow you make me cry and laugh at the same time

@DavidGOrtega that's a cute response, spoken like someone only in it for the money (or perhaps a representative of Tabrisjs??). I never said the tools all had to be free, but if it's free with React Native, which is far more complex than Tabrisjs, why can't others do it. I don't mind paying for cloud services, like back-ends, but the IDE should be free. Most, if not all, Cordova plugins are free, which is what this post is about - a Cordova plugin created by Microsoft for native UI, which was FREE, might I add... My entire point was that why pay for Tabris' crazy-expensive plans when others like Microsoft were giving the plugin away for free? It's just a plugin, not an entire environment. I'm not asking for an entire environment like Tabris, I just want someone to fork this specific plugin so I can use it - so the rest of us can use it. If I had the skill I would do it myself and release it for free, as I believe that level of knowledge should be free to all as it further raises the general standard of cross-platform apps that get developed.
There are many services that offer cloud builds, even Intel XDK offered it for free. They just stopped it as they felt there are many other more mature services which could do it better. But they never had any interest in charging for their technology. I say it again, because you obviously missed it... I don't mind paying for back-end services, never have. The dev tools should be free as that's what the world wants. And again, Tabrisjs is just insanely expensive!! I really doubt their business is making any real money as I don't believe their commercial model is at all sustainable.

@delanick , tabris.js had a discussion for opening their core, but it seems stalled.

@cordovapolymer I would love to have that happen. Pity it's stalled.

@cordovapolymer sad part is I've played with Tabris and really enjoy using it, far more than Nativescript even, but small one-man business, especially from South Africa (like me) won't spend that money on those solutions, it's just not feasible. If Tabris was open source and free to use it would be the perfect tool!

@delanick I'm not associate to Tabris in any way. I'm interested in buying their product. I truly understand that coding wastes many human hours that have to be PAID. If you don't want to pay for someone else work don't blame them just because they don't give it to you for free. I wonder what do you have in your private repos that you don't want others to have a peek and pick YOUR code.
Tools don't have to be nor free nor open source, people is free of doing whatever they want with their work. If you had the knowledge to work in this framework as you state I bet you that you would not maintain it, specially for free, maintaining an open source repo is a huge amount of work. Microsoft, React (facebook), etc... are HUGE teams with HUGE resources, don't spect Tabris or anyone else to be that huge... If you are wondering who is paying them... I will and many others like me I presume.
I wonder why people here is asking Tabris to be open source... This project is OPEN SOURCE and no one is actually specting to work on top of this! You have to use the word FREE as FREE BEER and the real thing is that if Tabris would be FREE as FREE BEER you wont mind if it is closed or open sourced...


Guzzle v6, client error 403 on valid request

Hello,
I'm using the psr7 feature branch using Guzzle6.
I'm having issues running the following request:
$this->linkedIn->get('v1/companies', ['format' => 'json', 'query' => ['is-company-admin' => 'true']]);
// Same with:
$this->linkedIn->get('v1/companies', ['query' => ['is-company-admin' => 'true', 'format' => 'json',]]);

Which is returning a Client error: 403. Although when checking the built URL I have:
https://api.linkedin.com/v1/companies?is-company-admin=true&format=json

Which is working on apigee, with the exact same URL.
Am I missing something ?
Thank you,

Hi, nWidart.
I download your source code and install it by composer.
as a result, vendor folder is created.
I changed the name 'happyr' to Happyr and 'linkedin-api-client' to 'LinkedIn'
and create a "index.php" to call it.
it's some code is
require_once 'autoload.php';
$linkedIn=new Happyr\LinkedIn\LinkedIn('756k6wdtntngffs','AtXwifuEIoff4xWvbb');
and copy index.php to vendor folder.
But When i test it, it occurs "Server 500 error".
I made something wrong when install it.
Please tell me the way.
thx.

Hello,
Any updates on this by any chance ?
Thank you

Hey. The Httplug is about to be stable at any day now. I'm waiting on their changes.
Thank you for taking the time testing this.

I've debugged this. I cant reproduce the error. Are you sure that you have the correct scope?
Try to use this when you authenticate the user:
$linkedIn->getLoginUrl(['scope'=>'w_share']);

Hm I have that scope configured in my linkedin app. I'll have to try with specifying it in loginurl

Currently, on version 0.6.3, when running the above command, I'm getting:
$this->linkedIn->get('v1/companies', ['format' => 'json', 'query' => ['is-company-admin' => 'true']]);

[RuntimeException]
Puli Factory is not available

I saw that you opened an issue on php-http/discovery and that they included puli/cli as a dependency.
I just updated the master (https://github.com/Happyr/LinkedIn-API-client/commit/c291cbe302c2dc0bfcc74d7c22cbd6920941a8b7) so now you may require the lastest from php-http/discovery

Thank you.
Quick update, I started working on this feature again, and now making the same api call (v1/companies), returns the following:

I am admin of one company on linkedin so not sure why I'm getting that.

Do you have the rw_company_admin permission?
Ref: https://developer.linkedin.com/docs/company-pages

Via the screenshot posted here , https://github.com/Happyr/LinkedIn-API-client/issues/80#issuecomment-168627370 , the linkedin app has all the boxes ticked, rw_company_admin too.  So I didn't think of adding it again to the method to get the login url.
However by explicitly adding it to the getLoginUrl method as well, it seems to work.
$loginUrl = $this->linkedIn->getLoginUrl(['redirect_uri' => route('connect.linkedin.callback'), 'scope' => 'w_share,rw_company_admin']);

Their documentation is pretty confusing regarding this. :astonished:
Thank you for your time and great support!

Sorry, I saw that screenshot but I missed the rw_company_admin.

However by explicitly adding it to the getLoginUrl method as well, it seems to work.

That is weird. I think I've read somewhere that what you add on the api admin page is the defaults...

Their documentation is pretty confusing regarding this.

Their documentation is way better now than 2 years ago when I started developing this client.
Thank you for trying out the dev-master!

Man, I'm scared to see their documentation from 2 years ago than :astonished:


FWXV-193 CAN debug module

@JarvisWeng I moved the library files to the can directory because there were some implicit dependency errors when I had it in ms-common