Emebed page for documents

We need to introduce a django template for embedded document resources to center various media type in the page.
Currently is not possible to style element inside the iframe so the preview that use direct media source show the content in the original size.
now:

expected:

All the type of documents should be centered in page

can be closed, for this issue:
https://github.com/GeoNode/geonode/issues/8053


Availability check based on providers-common gem

Issue https://github.com/RedHatInsights/sources-api/issues/206
This PR moves Source#availability_check from AnsibleTower to providers-common.

Depends on:

[x] https://github.com/RedHatInsights/topological_inventory-amazon/pull/63
[x] https://github.com/RedHatInsights/topological_inventory-providers-common/pull/25
[x] https://github.com/RedHatInsights/topological_inventory-providers-common/pull/28

TPINVTRY-944

I see, there should be version 3.0


TimelineTextEditor: Better character suggestions

Would be nice also have most recent characters at the top of the suggestions in the text editor.

To add on to this, perhaps make it default to already having the last used character option selected when adding another node. Often multiple nodes are added before switching to another character, so that would also help smooth workflow.
Combined with sorting by last usage in the timeline would make it a great enhancement.

I was also planning on porting the shortcuts from the visual editor, which might help with this:
CTRL+SHIFT+T (new text event with the last speaker), CTRL+ALT+T (new text event with the second to last speaker).
But yeah I think having the last speaker at the top was kinda the plan.


Connect over modbus rtu

Hi,
first of all thank you very much for excellent python library.
I have just been wondering if it is possible to use your library with modbus over rs485/rtu and not over TCP/UDP?
Thank you for answer.
Tonas

In theory yes, if the UDP communication part is replaced with RS485.
But not out of the box and not at this moment.


Fix the build

This project uses semver, please check the scope of this pr:

[ ] #none# - documentation fixes and/or test additions
[ ] #patch# - backwards-compatible bug fix
[X] #minor# - adding functionality in a backwards-compatible manner
[ ] #major# - incompatible API change

Closes: https://github.com/ciena-frost/ember-frost-list/issues/103
CHANGELOG

Added .pr-bumper.json file to correct build problems during merging.
Publish PR #102, #104, #105, #106, and #107, which included:
Added a default comparator which can be overridden with a custom comparator.
Added tests for three scenarios of clicking
Modified frost-list template to have a hook at the top of the container for easier testing of items below it.
Updated to use latest pr-bumper which supports being able to set a PR to none when publishing a new version is not desired.
Added optional basicClickDisabled attribute. While basic click is nice, it is not always desirable. If you have a list of 100+ selected items, a mis-click might reset your selections.
Fixed issue with Ember 2.11 and run loop
Added a default comparator which can be overridden with a custom comparator
Added tests for three scenarios of clicking
Added an ember hook selector to frost-list-item-container

Coverage remained the same at 81.356% when pulling ceffd43d23a3da4af1b252bbbd40b68f1194cb4f on juwara0:fixBuild into c64de8be741fb555b49b3f7c47cafad4fa588d3b on ciena-frost:master.

👍


trying to log the response from api after file is uploaded

`<template>
  <vue-dropzone ref="myVueDropzone" id="dropzone"
  :options="dropzoneOptions" @vdropzone-succes="afterComplete">
  </vue-dropzone>
</template>

<script>
//https://rowanwins.github.io/vue-dropzone/docs/dist/index.html#/events
//dropzone
import vue2Dropzone from 'vue2-dropzone'
import 'vue2-dropzone/dist/vue2Dropzone.min.css'
export default {
  name: 'FileUpload',
  components: {
    vueDropzone: vue2Dropzone,
  },
  data: function () {
    return {
      dropzoneOptions: {
          url: 'http://headcount.test/wp-json/recruiter/v1/upload',
          thumbnailWidth: 150,
          maxFilesize: 0.5,
          headers: { "Content-Type": "multipart/form-data;" },
      },
    }
  },
    methods:{
    afterComplete(file) {
      console.log(file.xhr.response);

    },
  },
}
</script>
`

this is my code, as far as I know I followed the docs
it doesn't log anything i also tried console.log("succes") didn't do anything

I'm trying to do this too. I think you misspelled " @vdropzone-succes" should be  @vdropzone-success

Yep thats my suggestion too - thanks @Fanatic17


bug(HASH): Hash Transformer applies hashing to the entire message rat…

…her than just the specified field. ISSUE-15.

Sweet I will make those shortly

@ivanyu applied the requested changes! thanks so much for the opportunity to work on this.


Make test failures reporting better

@jhadvig when Jenkins fails, it is impossible to parse the Jenkins console output by my poor brain. Can you please print out the test case name and mark the failure using bold color or at least three exclamation marks :-)

@mfojtik, I don't think we can hope for @jhadvig to do this anymore :)

@bparees :+1:
@mfojtik enjoy ;)

@bparees I do believe this is now better, but will see when mongodb explode in Jenkins :-)

mongodb container is not maintained any more in this org. closing.


[CHALLENGE SUBMISSION] Day 9

Please provide the following information for your submission:
What is your name? (First, Last)
Daniel Paulus
Where is your GitHub Repo ?(With your challenge solution)
https://github.com/dpnl87/25daysofserverless2019/tree/master/src/day-9
What Challenge is this for? (Challenge number between 1 and 25)
9
(Optional) Anything else we should know? e.g., language used, location, blog post?
Using nodeJS and Octokit. Function is live and is used on my 25daysofserverless2019 repo.

From all of us on the Cloud Advocacy team at Microsoft, thank you for creating this issue!
Have a Happy Holiday season!
Feel free to share your solution on Twitter with the tag #25DaysOfServerless

We checked out your repository. We're thrilled to say your solution is now listed in our Hall of Fame
Thank you for your contributions!
This issue is now resolved! 🎉


Do simple prechecks to avoid heap allocations

In one of our programs we do a ToPunycode() on all URLs to make sure they are properly encoded. 99% of the URLs don't need any encoding but I noticed we are still seeing a lot of heap allocations being done in punycode.go. This pull request adds some checks to ToPunycode() and FromPunycode() to prevent unneeded heap allocations.
needpuny() doesn't make any heap allocations as demonstrated by: http://play.golang.org/p/F3u7F4ZILD

Some nits, looks good.

I did some tests and Contains is faster than a similar function with NextLabel. Only the solutions with NextLabel doesn't require a strings.ToLower to be correct which makes it overall faster than the Contains solution.
I have updated the pull request with this solution.

The travis failure is my fault. For the rest this looks good.

I added a docstring and cleaned up the code a bit. It's basically the same as dns.Split().


exp/services/recoverysigner: add "encryption-tink-keyset create" cmd

PR Checklist
PR Structure

[x] This PR has reasonably narrow scope (if not, break it down into smaller PRs).
[x] This PR avoids mixing refactoring changes with feature changes (split into two PRs
otherwise).
[x] This PR's title starts with name of package that is most changed in the PR, ex.
services/friendbot, or all or doc if the changes are broad or impact many
packages.

Thoroughness

[ ] This PR adds tests for the most critical parts of the new functionality or fixes.
[ ] I've updated any docs (developer docs, .md
files, etc... affected by this change). Take a look in the docs folder for a given service,
like this one.

Release planning

[ ] I've updated the relevant CHANGELOG (here for Horizon) if
needed with deprecations, added features, breaking changes, and DB schema changes.
[ ] I've decided if this PR requires a new major/minor version according to
semver, or if it's mainly a patch change. The PR is targeted at the next
release branch if it's not a patch change.

What
This PR adds the encryption-tink-keyset create command to output three keysets in JSON:

the cleartext keyset containing the private key
the encrypted keyset containing the private key
the cleartext keyset containing the public key

Why
This command generates the necessary keyset as the value to the  encryption-tink-keyset env to the recovery signer as a part of the feature of having unique signer keys per registered account.
Though we only need the encrypted keyset at this moment, the cleartext keyset containing the private key is used as a backup for devops and the cleartext keyset containing the public key will be used as the value of another env introduced later.
Known limitations
I'm wondering whether this command should be merged into the master so that devops can generate a key and configure the encryption-tink-keyset as well as the KMS key uri before the rest of code gets in given that encryption-tink-keyset is a required config.

@leighmcculloch when I built this PR, I ended up getting the same result in https://github.com/stellar/go/pull/2728
Howards-MacBook-Pro:recoverysigner howard$ go build ./...
go: finding module for package cloud.google.com/go/storage
go: finding module for package cloud.google.com/go/internal/trace
go: finding module for package cloud.google.com/go/internal/fields
go: finding module for package cloud.google.com/go/internal/btree
go: finding module for package google.golang.org/genproto/googleapis/rpc/status
go: finding module for package cloud.google.com/go/internal/version
go: finding module for package google.golang.org/genproto/googleapis/firestore/v1
go: found cloud.google.com/go/internal/btree in cloud.google.com/go v0.58.0
go: found google.golang.org/genproto/googleapis/firestore/v1 in google.golang.org/genproto v0.0.0-20200623002339-fbb79eadd5eb
go: found cloud.google.com/go/storage in cloud.google.com/go/storage v1.10.0
go: finding module for package cloud.google.com/go/firestore
go: found cloud.google.com/go/firestore in cloud.google.com/go/firestore v1.2.0

I wonder what causes the difference between yours environment and mine.


core-animated-pages should be imported by app-router.html

When using core-animated-pages, things break unless you import the core-animated-pages. I kind of expected app-router.html to be importing it, as it's really it's dependency, not the clients.

The router can't load it since there's no standard location for dependencies. There's no guarantee that people are using Bower as the dependency manager. Even Polymer's examples use a folder called components rather than bower_components.


[question] Is there a log or any traceability?

I'm trying to setup the plugin but the rubocop linter doesn't seem to be working. Running rubocop manually I have some diagnositics, but not with guard.
Is there any log that I can tail to see the commands/events being executed?

Hmm I don't do ruby, but maybe repro by using a min config? Also make sure you can execute the following:
  cmd = 'bundle',    args = { 'exec', 'rubocop', '--format', 'json', '--force-exclusion', '--stdin' }

min config:
-- clone guard and guard-collection to /tmp, nvim -u /tmp/min.lua
vim.o.rtp:append("/tmp/guard.nvim")
vim.o.rtp:append("/tmp/guard-collection")
require("guard.filetype")("ruby"):lint("rubocop")
require("guard").setup()

Yeah, it took me a while, but I solved it with:
      ft('ruby'):lint({
        cmd = 'bundle',
        args = { 'exec', 'rubocop', '--format', 'json', '--force-exclusion', '--stdin' },
        stdin = true,
        parse = function(result, bufnr)
          local decoded = vim.json.decode(result)
          local diagnostics = {}
          local severities = {
            convention = lint.severities.info,
            error = lint.severities.error,
            fatal = lint.severities.fatal,
            info = lint.severities.info,
            refactor = lint.severities.style,
            warning = lint.severities.warning,
          }

          for _, diagnostic in ipairs(decoded.files[1].offenses) do
            table.insert(diagnostics, {
              bufnr = bufnr,
              source = diagnostic.cop_name,
              lnum = diagnostic.location.start_line,
              end_lnum = diagnostic.location.last_line,
              col = diagnostic.location.start_column,
              end_col = diagnostic.location.last_column,
              code = diagnostic.cop_name,
              severity = severities[diagnostic.severity],
              message = diagnostic.message,
            })
          end

          return diagnostics
        end,
      })

But, for that I hacked a linter on the code. I think it'd be nice to have a guard.log showing the command that's being run and potentially the results as well.

I used this to debug my problem, maybe we could add it as a debug: true option on the setup, or something like that?
https://github.com/nvimdev/guard.nvim/commit/3cf869265758710a8074cc17cefb6261d8d164d6

Maybe sth like LspLog and LspInfo


Flags empty offshore wind shapes

Closes #140
Changes proposed in this Pull Request

In this PR, I implement logic to handle empty offshore wind shapes until they are flagged by an existing assert in the build_renewable_profiles rule. Example below:
Traceback (most recent call last):
  File "/home/trevor/repos/pypsa-usa/workflow/.snakemake/scripts/tmp10hpligv.build_renewable_profiles.py", line 228, in <module>
    assert not regions.empty, (
AssertionError: List of regions in resources/texas/regions_offshore.geojson is empty, please disable the corresponding renewable technology

@ktehranchi, do you think what I have implemented here is fine. I guess there are two alternatives I can see:

Raise the assertion error earlier in the build_shapes.py file. Basically, replace the if offshore.empty with an assert statement.
Add logic to handle empty regions, then maybe add a result plot for offshore wind regions (and also onshore wind and solar) for troubleshooting purposes

Checklist

[x] I tested my contribution locally and it seems to work fine.
[x] Code and workflow changes are sufficiently documented.
[x] Changed dependencies are added to envs/environment.yaml.
[x] Changes in configuration options are added in all of config.default.yaml.
[x] Changes in configuration options are also documented in doc/configtables/*.csv.

Okay cool. I will revert this change and replace it with option 1 outlined in the PR (even though the eez should always give offshore wind shapes)


Halo regions v2: switching to offset arrays

This PR is part 1 of n implementing halo regions (originally PR #167). Here I just replace Arrays and CuArrays with OffsetArrays containing an Array or CuArray.
An important change is the introduction of the data(::Field) function which returns the data stored by the field, i.e. without halo data.
Lots of refactoring had to happen but it was mostly in updating the tests to use the data function.
The Poisson solver had to be refactored because FFTW and cuFFT won't work properly on an offset array containing halos. The Poisson solver now keeps track of the right-hand-side, Fourier coefficients, and solution using solver.storage which is a regular Array or CuArray. This cuts down memory usage as well.
This is not ready to merge yet. For some reason the velocity field is not divergence-free at the bottom layer (k = Nz) causing the regression tests to fail, and the incompressibility test as well. Will look at this with fresh eyes tomorrow. I also need to finish updating some of the dynamics tests.
Works on #104
Resolves #45
Resolves #122
Resolves #177

All tests pass on the CPU now, so it's almost ready to merge.
Going to make sure all GPU tests pass and I'll clean up the code a bit, especially with more concrete typing and changing data(::Field to return a view.

When data returns a view, all of the redundant @views should be removed.

When data returns a view, all of the redundant @views should be removed.

Yup, that'll be nice!

data returning a view is important; it means that you can use data when you need to in the inner algorithm.

Alright, I think this is ready to be merged.
Performance looks alright. Model has slowed down a tiny bit but that's expected because we're using halo regions but still computing differences the "slow" way with incmod1 and decmod1. The next PR will address this.

Why would the model have slowed down?

6% for a 3D kernel and a 2D loop over a vertical integral isn't bad.

Why would the halo regions themselves cause a slowdown?

Probably just extra opportunities for type instabilities, I noticed allocations went up by a tiny bit as well.
I know that  it's supposed to be a zero-cost abstraction but I may have used them in a way that produced some overhead.

True. I guess a performance loss due to type instability isn't super important --- we need to solve type instability anyways.


Pooling / Pooled term

Is there a good term for pooling or pooled? I'm not seeing something that stands out and makes sense to me exactly. Or could not translate it since it defines it afterwards:

Event Pooling
The SyntheticEvent is pooled. This means that the SyntheticEvent object will be reused and all properties will be nullified after the event callback has been invoked.
This is for performance reasons.

Thoughts?

I guess "grouping" should get the point across. Sound ok?
Raggruppamento Eventi
Il SyntheticEvent è raggruppato

Maybe "Il SyntheticEvent è gestito in un pool" (and then the explanation)

Addressing this in code-review #31


Add parallel execution code for container operations

This code will run container operations in parallel, up to a given maximum number of threads. Currently, it has only been enabled for local podman rm as a proof of concept.

Hm. This might actually break the build on windows... Let me fix that.

@jwhonce PTAL

@mehon, clean up comments and i will merge

Comments addressed, should be ready to merge

/hold

/lgtm

@mheon great work! as time allows, can you sprinkle this into other commands?

/hold cancel


When loading with dotlottie-web, the animation will be stretched

Overview

When I wrote it this way, the animated image was stretched when I opened the private window of the Chrome browser.
How to deal with this problem?

@guide90 Which dotlottie-web version you're using ?

@guide90 this should be fixed in v0.29.1


New Desmos Geometry support

With the expression list and normal functions available, would be nice to have all the features of Desmodder inside the new geometry calculator.

Thanks for bringing this up! Here's a checklist of fixes before releasing DesModder for /geometry-beta. Mostly CSS-related issues.

[ ] Enable DesModder for /geometry-beta
[ ] Move the pillbox buttons: they are misplaced because of the toolbar. Best option would be to put them to the left of the wrench menu, like idea'd in https://github.com/DesModder/DesModder/issues/557, but they could also just be shifted down a little
[ ] Disable Text Mode on /geometry-beta since it is going to have a bunch of rough edges
[ ] Pinned expressions should go above the token explorer, not below.
[ ] Set-primary-color makes all tokens have (primary-color) border, regardless of the color of the token. This should be fixed. Need some way to merge the generated CSS with primary-color replacements from both products, or maybe choose conditionally.
Certain objects like circle() do not work with video creator. Seems to be a bug with asyncScreenshot. I think we keep Video Creator enabled despite that.

Added in #563


Create a docker image for insights-cli

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Optional requirements

Make the visualization dependencies optional since they pull a bunch of indirect dependencies (especially tiledb-plot-widget), which increases the environment footprint and increases the chance of version conflicts.
To preserve the current behavior when installing use pip install tiledb-cloud[all]==0.7.22 (adjust for latest version).

@gsakkis can you update this PR and provide example update to docs for how this is to be used?

@Shelnutt2 the README points to https://docs.tiledb.com/cloud/client-api/installation for installation instructions; should I update the docs there?

@Shelnutt2 the README points to https://docs.tiledb.com/cloud/client-api/installation for installation instructions; should I update the docs there?

Yes those, and also a snippet in this PR for us to reference in the next release.

Updated the description with the equivalent pip install command; I'll update the docs separately.

Should I update or close this?

I'm in favor of updating and getting it merged. Then also updating the downstream packages.

Updated; I'll push PRs for the docker images when 0.7.28 is released.


[alembic] fix cmake error when using find_package in down stream project

@jasjuang,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Thanks for the PR!
Since this is making a change to the package output, could you bump the package version? (Adding a "-1" or incrementing the dash number if it's already there)

Just added. Please check, thanks!
On Mon, Jul 17, 2017 at 4:06 PM, Robert Schumacher<EMAIL_ADDRESS>
wrote:

Thanks for the PR!
Since this is making a change to the package output, could you bump the
package version? (Adding a "-1" or incrementing the dash number if it's
already there)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/vcpkg/pull/1471#issuecomment-315910508, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ALNuYDRjfZbzsm5Mb1U9Tj5Xw2uEehsQks5sO-kTgaJpZM4OaoGO
.


evp/e_chacha20_poly1305.c: further improve small-fragment TLS performance.

Improvement coefficients vary with TLS fragment length and platform, on
most Intel processors maximum improvement is ~50%, while on Ryzen - 80%.
The "secret" is new dedicated ChaCha20_128 code path and vectorized xor
helpers.
[extended tests]

Just in case for reference. Original estimate was ~40% (for Haswell), while measured is ~50%. This is uncommon, because estimates tend to be optimistic. Profiling new code revealed that dedicated benchmark was triggering redundant call to ChaCha20_ssse3. Removal of the redundant call affects the estimate, positively, so 50%, which less than new optimistic, is ok...


Typescript import/named with types in d.ts

I think I have found a bug with import/named. I have created a repository that reproduces the issue.
The line that gives problems is
import { AreaProps } from 'recharts';

It imports an interface defined in a d.ts file. The following warning is given:
1:10  error  AreaProps not found in 'recharts'  import/named

PR #1304 should solve importing types to typescript, but maybe this doesn't work for declaration files? Or is there something wrong with my configuration?

Struggling with the same issue and express-serve-static-core from node_modules/@types/express-serve-static-core.
import { RequestHandler, Request } from 'express-serve-static-core';

results in an error:
8:41  error  Unable to resolve path to module 'express-serve-static-core'  import/no-unresolved

The problem seems to come for types installed through http://definitelytyped.org/.

can you try using https://www.npmjs.com/package/eslint-import-resolver-typescript as your resolver and see if it fixes?

@manuelbieh I think your case is a little different than OP. I see you're using the typescript resolver but it works fine for me... and no-unresolved should report based on the existence of the true code file and not the types.
it would be possible for the typescript resolver to try a @types/... resolution but that would be in scope for that resolver, and not this plugin.

@benmosher , thanks the typescript resolver worked for me. Note for reference that you can have multiple import resolvers (I didn't want to turn off the webpack resolver so I hadn't tried the typescript one earlier).

@ThomasdenH can you try using https://www.npmjs.com/package/eslint-import-resolver-typescript as your resolver and see if it fixes?
if it does, I think it may solve many TS-related issues. I just used it to solve some weird ones in my day job codebase
edit: I think it won't help after all, based on alexgorbatchev/eslint-import-resolver-typescript#17 + associated PR but it's an easy experiment

Sorry for the late reply!, Unfortunately it doesn't fix the issue, but thanks for the suggestion.

@ThomasdenH: yep, makes sense. I didn't fully understand the scenario when I originally suggested.
@pcorpet: great! I'm doing the same thing -- I have the webpack resolver first for most things but the typescript resolver does resolution in some of my untranspiled first-party TS dependencies

I am having the same issue for an intl-type that is generated by DefinitelyTyped. Is there any solution or further ideas on how to fix it?

For anything maybe interested, you can try eslint-import-resolver-typescript@JounQin/node_modules#eslint-import-resolver-typescript temporarily (PR made at https://github.com/alexgorbatchev/eslint-import-resolver-typescript/pull/20).

☝ @JounQin's solution is my recommendation as the permafix. I haven't tried it myself but I scanned through the changes and they look like they should do the trick!
@JounQin: I'm not sure how to read that link. Did you publish your fork to npm?

@benmosher It's just using npm package with github branch.

I'm running into this issue as well. Imports from declarations are very common, so this is a real pain. It would be great to get a fix

again, resolution is out of scope for this plugin. will close when @JounQin's PR is accepted or other similar fix is implemented by eslint-import-resolver-typescript (see https://github.com/benmosher/eslint-plugin-import/issues/1341#issuecomment-507016603 above)

The way I understand it, its not possible for a resolver to solve this problem. Resolvers can only return a single file, but in this case of separate source and declaration files, the exports are distributed across 2 files.
e.g.
// file.d.ts
export type Fruit = {
  color: string,
};

// file.js
export const tomato = {
  color: 'red',
};

// index.ts
import { Fruit, tomato } from './file'; // <-- these exports do not come out of any one file

@hedgepigdaniel You should add export const tomato: Fruit into .d.ts also.

@benmosher Should this issue be closed in favor of eslint-import-resolver-ts?

Make sure you have inside the package.json
{   ....   "dependencies": {     "module-name": "version"   },   "devDependencies": {     ...     "@types/module-name": "version" // version should be same with dependencies   }   .... }

@benfletcher @ThomasdenH @ljharb @stekycz
This issue can be closed definitely in favor of<EMAIL_ADDRESS>

@JounQin which change in that new version addresses this issue?

@hedgepigdaniel You can always read the CHANGELOG by yourself.

@JounQin i've done so, and i have no idea which change addresses this issue. Could you clarify?

@ljharb I was thinking we were in the same context here: https://github.com/alexgorbatchev/eslint-import-resolver-typescript/pull/20#issuecomment-530032941, didn't we?

Yes but those don't tell me which specific commits in the changelog address this issue :-)

@ljharb Did you read https://github.com/alexgorbatchev/eslint-import-resolver-typescript/blob/master/CHANGELOG.md#features?

resolve .ts/.tsx/.d.ts first, and then fallback to @types/* (b11ede3)

Does that mean there is now a problem where if there is a declaration file, there is a false error when importing a value from the module that has the declarations?

I guess that isn't a problem if you have types for all exported symbols

@hedgepigdaniel Sorry my English is not so good enough for understanding what's your point? Do you have an example?

// apple.js

export const color = 'red';
export const shape = 'round';

// apple.d.ts

export const color: string;

// index.ts

// vv Error: shape is not defined in `apple.d.ts`!
import { shape } from './apple';

@hedgepigdaniel Yeah, but that is a bad practice and should never happen.

@JounQin it's not about what's your responsibility; it's about being friendly and helpful :-) Thanks for pointing me to https://github.com/alexgorbatchev/eslint-import-resolver-typescript/commit/b11ede3c9fafbc548db011729fd64e958cde6e51.

@JounQin it's not about what's your responsibility; it's about being friendly and helpful :-) Thanks for pointing me to alexgorbatchev/eslint-import-resolver-typescript@b11ede3.

Emm... I'm always friendly, but my pointed CHANGELOG and PR both target the same commit which fixed this issue. Maybe we both need to be more patient and find out the key point  by ourselves? Cheers.


IPFS Asset should be stored according to a price

Right now our create_ipfs_asset has these parameters:

origin: OriginFor
addr: Vec
cid: Vec
size: u64

We need to add another one called price and it will be responsible for measuring how long the asset is going to stay in the node.
For starters let's make it so that for every 1000 units, the deleting_at attribute get's an additional 100 blocks.
The DefaultAssetLifetime is always the initial value of deleting_at.
Update
Let's cover the extend of deleting_at with refactoring code that is already commented:
https://github.com/CherryNetwork/Cherry-Node/blob/531bb9f7f3a301c3ff0064266faf099e3aee0eea/bin/node/pallets/pallet-ipfs/src/lib.rs#L311-L333
It should work in the same manner, find the IPFS Asset, mutate it's deleting_at attribute according to a price that the user paid.

@elsuizo Description has been updated


Add note on advanced configuration for mongodb

Proposed changes:

Documentation update based on #1502

Status (please check what you already did):

[x] made PR ready for code review
[x] added some tests for the functionality
[x] updated the documentation
[ ] updated the changelog

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Could please review it @MetcalfeTom ?

OK, thanks for the note.  In the future we may want to consider support for MongoDB config files.


Still Throwing Pokeballs After Caught

Dev branch, Mac OS x

Which commit on dev though?

How do I check the exact commit? I downloaded the latest one from maybe 20 hours ago?

git rev-parse HEAD

423e4dea72c0b04d8882ef90f03949fe3bf35ac2

Please use issue template


Cpalmer/mayrelnotes

Docs Build status updates of commit 77b41f2:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/relnotes/index.md
:warning:Warning

Details

articles/libraries/chemistry/schema/toc.yml
:white_check_mark:Succeeded

articles/relnotes/index.md

[Warning] 3 invalid cross reference(s) "microsoft.quantum.numerics.usage", "microsoft.quantum.chemistry.concepts.multireference", "microsoft.quantum.libraries.chemistry.schema.spec_v_0_2".

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 1a03499:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/relnotes/index.md
:warning:Warning

Details

articles/libraries/chemistry/schema/toc.yml
:white_check_mark:Succeeded

articles/relnotes/index.md

[Warning] 3 invalid cross reference(s) "microsoft.quantum.numerics.usage", "microsoft.quantum.chemistry.concepts.multireference", "microsoft.quantum.libraries.chemistry.schema.spec_v_0_2".

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 3329e45:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/relnotes/index.md
:warning:Warning

Details

articles/libraries/chemistry/schema/toc.yml
:white_check_mark:Succeeded

articles/relnotes/index.md

[Warning] 3 invalid cross reference(s) "microsoft.quantum.numerics.usage", "microsoft.quantum.chemistry.concepts.multireference", "microsoft.quantum.libraries.chemistry.schema.spec_v_0_2".

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 80a33a7:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/relnotes/index.md
:warning:Warning

Details

articles/libraries/chemistry/schema/toc.yml
:white_check_mark:Succeeded

articles/relnotes/index.md

[Warning] 3 invalid cross reference(s) "microsoft.quantum.numerics.usage", "microsoft.quantum.chemistry.concepts.multireference", "microsoft.quantum.libraries.chemistry.schema.spec_v_0_2".

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 80a33a7:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/relnotes/index.md
:warning:Warning

Details

articles/libraries/chemistry/schema/toc.yml
:white_check_mark:Succeeded

articles/relnotes/index.md

[Warning] 3 invalid cross reference(s) "microsoft.quantum.numerics.usage", "microsoft.quantum.chemistry.concepts.multireference", "microsoft.quantum.libraries.chemistry.schema.spec_v_0_2".

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 531e4ba:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/relnotes/index.md
:warning:Warning

Details

articles/install-guide/csharp.md
:white_check_mark:Succeeded

articles/libraries/chemistry/schema/toc.yml
:white_check_mark:Succeeded

articles/relnotes/index.md

[Warning] 3 invalid cross reference(s) "microsoft.quantum.numerics.usage", "microsoft.quantum.chemistry.concepts.multireference", "microsoft.quantum.libraries.chemistry.schema.spec_v_0_2".
Line 17: [Warning] Invalid file link:(~/relnotes/microsoft.quantum.install.csharp#update).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit df6409b:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/relnotes/index.md
:warning:Warning

Details

articles/install-guide/csharp.md
:white_check_mark:Succeeded

articles/libraries/chemistry/schema/toc.yml
:white_check_mark:Succeeded

articles/relnotes/index.md

[Warning] 3 invalid cross reference(s) "microsoft.quantum.numerics.usage", "microsoft.quantum.chemistry.concepts.multireference", "microsoft.quantum.libraries.chemistry.schema.spec_v_0_2".
Line 17: [Warning] Invalid file link:(~/relnotes/microsoft.quantum.install.csharp#update).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 6ef0147:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/relnotes/index.md
:warning:Warning

Details

articles/install-guide/csharp.md
:white_check_mark:Succeeded

articles/libraries/chemistry/schema/toc.yml
:white_check_mark:Succeeded

articles/relnotes/index.md

[Warning] 3 invalid cross reference(s) "microsoft.quantum.numerics.usage", "microsoft.quantum.chemistry.concepts.multireference", "microsoft.quantum.libraries.chemistry.schema.spec_v_0_2".
Line 17: [Warning] Invalid file link:(~/relnotes/microsoft.quantum.install.csharp#update).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit c8c74ea:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/relnotes/index.md
:warning:Warning

Details

articles/install-guide/csharp.md
:white_check_mark:Succeeded

articles/libraries/chemistry/schema/toc.yml
:white_check_mark:Succeeded

articles/relnotes/index.md

[Warning] 3 invalid cross reference(s) "microsoft.quantum.numerics.usage", "microsoft.quantum.chemistry.concepts.multireference", "microsoft.quantum.libraries.chemistry.schema.spec_v_0_2".
Line 17: [Warning] Invalid file link:(~/relnotes/microsoft.quantum.install.csharp#update).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 8adce34:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/relnotes/index.md
:warning:Warning

Details

articles/install-guide/csharp.md
:white_check_mark:Succeeded

articles/libraries/chemistry/schema/toc.yml
:white_check_mark:Succeeded

articles/relnotes/index.md

[Warning] 3 invalid cross reference(s) "microsoft.quantum.numerics.usage", "microsoft.quantum.chemistry.concepts.multireference", "microsoft.quantum.libraries.chemistry.schema.spec_v_0_2".

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit aa4476a:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/relnotes/index.md
:warning:Warning

Details

articles/install-guide/csharp.md
:white_check_mark:Succeeded

articles/libraries/chemistry/schema/toc.yml
:white_check_mark:Succeeded

articles/relnotes/index.md

[Warning] 3 invalid cross reference(s) "microsoft.quantum.numerics.usage", "microsoft.quantum.chemistry.concepts.multireference", "microsoft.quantum.libraries.chemistry.schema.spec_v_0_2".

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 85b68c9:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/relnotes/index.md
:warning:Warning

Details

articles/libraries/numerics/installation.md
:warning:Warning

Details

articles/install-guide/csharp.md
:white_check_mark:Succeeded

articles/libraries/chemistry/schema/toc.yml
:white_check_mark:Succeeded

articles/relnotes/index.md

[Warning] 2 invalid cross reference(s) "microsoft.quantum.chemistry.concepts.multireference", "microsoft.quantum.libraries.chemistry.schema.spec_v_0_2".

articles/libraries/numerics/installation.md

Line 31: [Warning] Invalid file link:(~/media/vs2017-nuget-console-menu.png).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit fa49378:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/relnotes/index.md
:warning:Warning

Details

articles/install-guide/csharp.md
:white_check_mark:Succeeded

articles/libraries/chemistry/schema/toc.yml
:white_check_mark:Succeeded

articles/relnotes/index.md

[Warning] 2 invalid cross reference(s) "microsoft.quantum.chemistry.concepts.multireference", "microsoft.quantum.libraries.chemistry.schema.spec_v_0_2".

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 086dde8:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/libraries/chemistry/concepts/second-quantization.md
:warning:Warning

Details

articles/install-guide/csharp.md
:white_check_mark:Succeeded

articles/libraries/chemistry/schema/toc.yml
:white_check_mark:Succeeded

articles/relnotes/index.md
:white_check_mark:Succeeded

articles/libraries/chemistry/concepts/second-quantization.md

[Warning] 2 invalid cross reference(s) "<xref:Microsoft.Quantum.Chemistry.LadderOperators.LadderOperator>", "<xref:Microsoft.Quantum.Chemistry.LadderOperators.LadderSequence>".

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit cb2f338:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/install-guide/csharp.md
:white_check_mark:Succeeded

articles/libraries/chemistry/schema/toc.yml
:white_check_mark:Succeeded

articles/relnotes/index.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


FOAM models of BrowserAPI and WebAPI using one table implementation.

Add ORDER_BY(INTERFACE_KEY) to ensure the returned CSV are in alphabet order.
Removed .split(...).sort() in api_matrix test.


Fix travis error build

All travis builds are failing (in both branches: master and 0.5.6)
https://travis-ci.org/firebase/FirebaseUI-iOS/builds
First error appeared after commit: https://github.com/firebase/FirebaseUI-iOS/commit/5a6bd83fb0024d4da971648d444551ade2519a98
I can see that later new/other errors were added on top of that one.

duplicate of #145

I made a commit that didn't touch our xcodeproj or source code at all and travis fixed itself
l m a o
https://travis-ci.org/firebase/FirebaseUI-iOS/builds/162863736

Latest master build fails too https://travis-ci.org/firebase/FirebaseUI-iOS/builds/161772490

we haven't merged v0.5.6 into master

master is now passing https://travis-ci.org/firebase/FirebaseUI-iOS/builds/162880841


 Fix jcli rest commands.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)

LGTM


Frontend7: Allows user to download processed images as JPEG, PNG, and TIFF

reads json data from server and converts it to usable image format
should allow download of image(s) in multiple formats

Need to check that this is still functional with the new .zip server code


[BUG] color contrast issue fix (WCAG color contrast issue)

Description
Requirements:
Our app should pass all the WCAG color contrast checks. I've added a sample screenshot for reference.
Please install WCAG Color contrast checker extension and fix all the issues in every single page.
It should pass all the checks in all pages.
Screenshots

@rohitdasu Can I work on it?


Workflow cloud_tenant fix

fixes https://bugzilla.redhat.com/show_bug.cgi?id=1746931
Introduced https://github.com/ManageIQ/manageiq/pull/18353
This did not bring back the type column so all classes were VmOrType.
Since the cloud_tenant method is introduced in a subclass, it was missing.
Now that we are instantiating the correct class (with help from type) all is well.
[----] F, [2019-08-29T09:40:01.670066 #3373:2adf69a8c12c] FATAL -- : Error caught:
[NoMethodError] undefined method `cloud_tenant' for #<VmOrTemplate:0x000055bee25ff310>
Did you mean?  cloud_tenant_id
gems/activemodel-5.1.7/lib/active_model/attribute_methods.rb:432:in `method_missing'
app/models/miq_provision_virt_workflow.rb:1049:in `create_hash_struct_from_vm_or_template'

@kbrock I believe the fix is actually a little different and as you stated in the description it does not make sense that the cloud_tenant_id would be set if the model does not respond to cloud_tenant.
I was thinking that there must be some additional column that is needed for the cloud_tenant relationship to be defined.  (And I had previously determined that removing the recently added select call allowed the code to work.
Through a little trail and error I found that adding the type column to the select list fixes the issue.  And this all makes sense, as you can see below, that the initial where call returns an OpenStack Template model, but after the select it now contains a base model VmOrTemplate instance for the same VM.  (The cloud_tenant association is defined down at the OpenStack sub-classed model.)  Finally, adding type back we are able to load the proper sub-class.
irb(main):001:0> vms = VmOrTemplate.where(:id => 3773)
=> #<ActiveRecord::Relation [#<ManageIQ::Providers::Openstack::CloudManager::Template id: 3773, vendor: "openstack"...

irb(main):002:0> vms = vms.select(:id, :name, :guid, :uid_ems, :ems_id, :cloud_tenant_id)
=> #<ActiveRecord::Relation [#<VmOrTemplate id: 3773, name: "HUIS-EvmSnapshot", guid: "6e45d27a-3d57-11e6-ab22-6003089fa294"...

irb(main):003:0> vms.first.cloud_tenant
NoMethodError (undefined method `cloud_tenant' for #<VmOrTemplate:0x00007fe989c36778>)

irb(main):004:0> vms = vms.select(:id, :name, :guid, :uid_ems, :ems_id, :cloud_tenant_id, :type)
=> #<ActiveRecord::Relation [#<ManageIQ::Providers::Openstack::CloudManager::Template id: 3773, name: "HUIS-EvmSnapshot"...

irb(main):005:0> vms.first.cloud_tenant
=> #<ManageIQ::Providers::Openstack::CloudManager::CloudTenant id: 10, name: "admin", description: "admin tenant"...

In the end I think the proper code change is to add :type to the select statement so the sub-class is loaded.
     # Only select the colums we need
-    vms = vms.select(:id, :name, :guid, :uid_ems, :ems_id, :cloud_tenant_id)
+    vms = vms.select(:id, :name, :guid, :uid_ems, :ems_id, :cloud_tenant_id, :type)

@gmcculloug oh yeah, that makes so much sense now.  Since I was the original author to this mess, I am happy to here that the cloud_tenant_id idea wasn't too far off, and the only real issue was me missing something in the select.  Thanks for the great sleuthing!
@kbrock we probably should have a test case for this if possible, but unsure how you would do that.

Ivanchuk backport details:
$ git log -1
commit b8dd448f4e4c08cc0798fc20c7ea78ce608686e7
Author: Joe Rafaniello<EMAIL_ADDRESS>Date:   Tue Sep 3 11:06:20 2019 -0400

    Merge pull request #19237 from kbrock/miq_provision_virt_workflow_cloud_tenant

    Workflow cloud_tenant fix

    (cherry picked from commit b598db0dec6fbbe8fdaa4554fac193f7d6e0d3be)

    https://bugzilla.redhat.com/show_bug.cgi?id=1746931


⚠ BeyondHD has degraded performance

In 7eb34b2, BeyondHD ($BHD) experienced degraded performance:

HTTP code: 200
Response time: 14080 ms

Resolved: BeyondHD performance has improved in e96bd64.


Add a method to query data using multiple paged queries

Description
This PR adds new methods: query_all and execute_all.
They use multiple paged queries to fetch results, and then merge all those results into one.
page_size must be set on the query that is passed to those methods.
Currently we use a single unpaged query to fetch data, but this might cause stalls (#298).
It's better to use many smaller queries and merge results locally.
Session::query_all(
        &self,
        query: impl Into<Query>,
        values: impl ValueList,
    ) -> Result<QueryResult, QueryError>;

Session::execute_all(
        &self,
        prepared: impl Into<PreparedStatement>,
        values: impl ValueList,
    ) -> Result<QueryResult, QueryError>;

Connection::query_all(
        &self,
        query: &Query,
        values: impl ValueList,
    ) -> Result<QueryResult, QueryError>;

Connection::execute_all(
        &self,
        prepared_statement: &PreparedStatement,
        values: impl ValueList,
    ) -> Result<QueryResult, QueryError>;

TODO

I have to add a few tests to make sure that everything is working as expected, will add soon

Fixed Issues
Fixes: #298
Pre-review checklist

[x] I have split my patch into logically separate commits.
[x] All commit messages clearly explain what they change and why.
[ ] I added relevant tests for new features and bug fixes.
[x] All commits compile, pass static checks and pass test.
[x] PR description sums up the changes and reasons why they should be introduced.
[x] I added appropriate Fixes: annotations to PR description.

I thought the execute_iter gave back an iterator which lazy loads new page row results when the current page is exhausted, doesn't it do the same thing? It looks more memory efficient as well (if you have millions of rows, execute_iter won't care since it lazy loads the results and deallocates old rows). If you have millions of rows and you call this method, changes are you get out of memory exceptions, or am I misunderstanding this method?

@Jasperav using iterators is absolutely preferred, especially for large datasets. This set of helper functions is mainly for internal use, where we know that the set is small, but we still want to use paging in order to avoid unnecessarily stressing the server - e.g. when fetching topology information.

Ok @psarna @piodul I think it's ready for review

@Jasperav ongoing. Note however, that this interface is intended to be used mostly for internal purposes, e.g. topology fetching. User data is often too large to gather everything in memory at one go (like these functions do), so it's strongly preferred to use our iterator interface. query_all and friends should only be used if you know that the result will be small and you don't want to worry about fetching multiple pages manually or advancing an iterator by hand.

@piodul I am using this crate in my toy project and I actually needed query_iter on Session. It is a query on a small (<20 rows) set. If I don't have this method, I will end up writing my own implementation (probably initializing an empty vec, use the iterator to add rows).
That is something this method already will do. I think other people sometimes just need all the rows in memory to send back to a client. So IMO this methods should remain on Session.

The try_collect method bounds are Default and Extend, which are implemented for simple primitives in your example (i32, String) but not for every struct. Forcing clients to implement default and extend for types that will collect an async stream isn't desired IMO.

v2:

Removed Session::query_all, it's backed up on the queryall-backup branch in case we wanted it in the future

v2:

Removed Session::query_all, it's backed up on the queryall-backup branch in case we wanted it in the future
Removed new BadQuery variant, Connection::query_all now uses ProtocolError to report bad usage.


Fixed some minor issues.

Hi @PaoloPia, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;


[DOC] Data Source Connectivity Outside OpenSearch

What do you want to do?

[x] Request a change to existing documentation
[x] Add new documentation
[ ] Report a technical problem with the documentation
[ ] Other

Tell us about your request. Provide a summary of the request and all versions that are affected.
Introduce the ability to reach data at rest in an object store (e.g., S3 or data lake) via Spark (Spark to be supplied by the user). Our first release will focus on data stored in Parquet format. Users will be able to query this data as though it is regular OpenSearch data and visualize it alongside their OpenSearch data in OpenSearch Dashboards.
What other resources are available? Provide links to related issues, POCs, steps for testing, etc.
https://github.com/opensearch-project/sql/issues/1116

@brijos: do we have a timeline on when this should be complete.


_pickle.UnpicklingError: invalid load key, '\x00'.

Dear Author,
I encountered an error when running bash scripts/train_and_eval_w_geo.sh ManiGaussian_BC 0,1,2,3,4,5 5678 ${try_without_tmux}    .
I run this code with 6 RTX3090s in Ubuntu20.04, torch==2.0.0+cu117.
However, it shows an error like this during training:
Error executing job with overrides: ['method=ManiGaussian_BC', 'rlbench.task_name=ManiGaussian_BC_20240627', 'rlbench.demo_path=/home/gjf/codes/ManiGaussian/data/train_data', 'replay.path=/home/gjf/codes/ManiGaussian/replay/ManiGaussian_BC_20240627', 'framework.start_seed=0', 'framework.use_wandb=False', 'method.use_wandb=False', 'framework.wandb_group=ManiGaussian_BC_20240627', 'framework.wandb_name=ManiGaussian_BC_20240627', 'ddp.num_devices=6', 'replay.batch_size=1', 'ddp.master_port=5678', 'rlbench.tasks=[close_jar,open_drawer,sweep_to_dustpan_of_size,meat_off_grill,turn_tap,slide_block_to_color_target,put_item_in_drawer,reach_and_drag,push_buttons,stack_blocks]', 'rlbench.demos=20', 'method.neural_renderer.render_freq=2000']
Traceback (most recent call last):
  File "/home/gjf/codes/ManiGaussian/train.py", line 96, in main
    run_seed_fn.run_seed(
  File "/home/gjf/codes/ManiGaussian/run_seed_fn.py", line 147, in run_seed
    train_runner.start()
  File "/home/gjf/codes/ManiGaussian/third_party/YARR/yarr/runners/offline_train_runner.py", line 200, in start
    batch = self.preprocess_data(data_iter)
  File "/home/gjf/codes/ManiGaussian/third_party/YARR/yarr/runners/offline_train_runner.py", line 121, in preprocess_data
    sampled_batch = next(data_iter) # may raise StopIteration
  File "/home/gjf/miniconda3/envs/manigaussian/lib/python3.9/site-packages/lightning/fabric/wrappers.py", line 178, in __iter__
    for item in self._dataloader:
  File "/home/gjf/miniconda3/envs/manigaussian/lib/python3.9/site-packages/torch/utils/data/dataloader.py", line 634, in __next__
    data = self._next_data()
  File "/home/gjf/miniconda3/envs/manigaussian/lib/python3.9/site-packages/torch/utils/data/dataloader.py", line 678, in _next_data
    data = self._dataset_fetcher.fetch(index)  # may raise StopIteration
  File "/home/gjf/miniconda3/envs/manigaussian/lib/python3.9/site-packages/torch/utils/data/_utils/fetch.py", line 41, in fetch
    data = next(self.dataset_iter)
  File "/home/gjf/codes/ManiGaussian/third_party/YARR/yarr/replay_buffer/wrappers/pytorch_replay_buffer.py", line 17, in _generator
    yield self._replay_buffer.sample_transition_batch(pack_in_dict=True)
  File "/home/gjf/codes/ManiGaussian/third_party/YARR/yarr/replay_buffer/uniform_replay_buffer.py", line 722, in sample_transition_batch
    store = self._get_from_disk(
  File "/home/gjf/codes/ManiGaussian/third_party/YARR/yarr/replay_buffer/uniform_replay_buffer.py", line 391, in _get_from_disk
    d = pickle.load(f)
_pickle.UnpicklingError: invalid load key, '\x00'.

Set the environment variable HYDRA_FULL_ERROR=1 for a complete stack trace.
[W CudaIPCTypes.cpp:15] Producer process has been terminated before all shared CUDA tensors released. See Note [Sharing CUDA tensors]
  0%|▎                                                                                | 134/100010 [03:10<35:31:44,  1.28s/it]/home/gjf/miniconda3/envs/manigaussian/lib/python3.9/multiprocessing/resource_tracker.py:216: UserWarning: resource_tracker: There appear to be 11 leaked semaphore objects to clean up at shutdown
  warnings.warn('resource_tracker: There appear to be %d '`

After that, one of the GPU stopped work, the whole program stuck at this place even I pressed Ctrl + C. This happened every time soon after training.
By the way, I did not use tmux or wandb, would this matter?
Could you please help me with this issue?

Yes, I've also encountered this issue multiple times, but I haven't found an essential solution yet because it seems to occur randomly. I suspect it could be due to loading a broken file that was removed by other processes, since the data is cached in a shared directory ('/tmp/arm/replay' by default). I recommend trying the training with two GPUs one more time.

Thanks for replying, I have not faced the same error when using 2 cards.


data preprocess

Hello, as far as I know, the data dimensions of each subject in the heart dataset are inconsistent, and the physical spacing is also different. I would like to know how the authors preprocessed these datasets to [160,192,224], resampling to a physical spacing, post-center clipping or zeroing @junyuchen245

Hi,
The input dimensions were actually set to 128x128x32, different from what was previously stated, as indicated in the training script for the TransMorph model, which you can see here:
https://github.com/junyuchen245/Registration_Uncertainty/blob/main/TransMorph_Uncert/train_TransMorphTVF.py#L49
The images were specifically cropped around the center of the heart and resampled to have an isotropic voxel size in the x and y dimensions. I might share my preprocessing scripts at a later time.
Junyu

Thank you very much for your answer. I will try according to what you said. I hope everything goes well with you


[BUG]Removal of subscription Folder from Artifacts Deletes All Subscriptions in APIM Instance

Release version
v<IP_ADDRESS>
Describe the bug
It has been observed that when the subscription folder is removed from the artifacts, it results in the deletion of all subscriptions present in the Azure API Management (APIM) instance. This is a critical issue because it leads to unintended loss of subscriptions and disrupts the management of API access.
Impact:
Data Loss: Loss of all subscriptions in the APIM instance can lead to significant disruptions in service and access control.
Configuration Management: Failure to override values from the .yaml file can result in inconsistent and unexpected behavior.
kindly provide the solution.
Expected behavior
Removing the subscription folder should not affect existing subscriptions in the APIM instance. Instead, the process should handle this folder appropriately without deleting existing data or should provide a warning about potential impacts.
Actual behavior
The removal of the subscription folder leads to the deletion of all existing subscriptions in the APIM instance.
Additionally, the configuration values from the configuration..yaml file are not being overridden as expected.
Reproduction Steps
Remove the subscription folder from the artifacts directory.
Deploy the artifacts to the APIM instance.
Observe that all existing subscriptions in the APIM instance are deleted.

Not sure what's the problem. If you delete the subscription folder and publish, all subscriptions in that folder should get deleted from your APIM instance. The publisher works by utilizing git commits and if it detects that the changes in the last commit include the deleting the subscriptions folder then it will go ahead and delete the subscriptions.

Thank you @waelkdouh for your input. You’re right that deleting the subscription folder in APIM will indeed result in the removal of those subscriptions, as the publisher depends on git commits to synchronize changes. However, this approach might not be suitable when dealing with different environments (e.g., dev, preprod, prod) where subscription values vary across environments.
Please guide me how to manage environment-specific values and prevent validation errors during deployment in publisher pipeline.

I still don't understand your question. If you are referring to overriding values across environments you need to use namedvalues.
Not sure what you mean by preventing validation errors. Apiops utilizes the apim restful api. If there is a validation error that happens on the server side as a result of the apiops attempted put request then we surface this error through logging within your devops pipeline and fail the publisher. Just remember that the publisher doesn't execute in an atomic fashion. Meaning, if the publisher fails everything that has been applies already on the apim server side can't be rolled back. You would have to reexecute a full publish operation after you resolve the issue that cause the publisher to fail. Please make sue you read past closed issues, our wiki, as well as the repo assosicated github page.

I would also like to add that this is an issue for our environment as well. Our stage apim instance and prod apim instance have completely different subscriptions and keys.
I think it is due to the nature of subscriptions and how they are intended to be treated (at least from our perspective). It seems they are meant to be created by the potential audiences of your apis/products in a specific environment (mostly prod).
In our environment, we may have some subscriptions in stage, but for the majority of our apis/products, users would just subscribe to prod as they have no need to deal with the lower environments. This creates a gap between stage and prod that we can no not feasibly manage with overrides.
Is there a recommended way to manage subscriptions that would be API Ops friendly?
Maybe some kind of exclusion could be in the configuration where we can tell it that we don't care to see certain types be managed by api ops or at least published to prod?

@dagraham7 - I would suggest excluding subscriptions completely from ApiOps. In your configuration.extractor.yaml, set subscriptionNames: [ignore]. This will prevent subscriptions from being extracted when the extractor runs.
Note that if you already have subscriptions in your artifacts directory, you should probably delete them so they're no longer managed by ApiOps.

@guythetechie - thank you for the suggestion! Is this information in the wiki? I didn't see it when I was looking, but I could have easily overlooked as well.
And just for my understanding. When the extractor ignores it and it is no longer in the git repo,  api ops will simply ignore subscriptions altogether? I don't have to worry about api ops removing them from my Dev (and of course prod) instance when the publisher runs a "full sync"?

@dagraham7 - that's correct. Another scenario where subscriptions get deleted is when ApiOps creates a brand-new product. Here is the flow:

dev has productA, prod does not have productA.
Operator runs extractor against dev. /artifacts/products/productA gets created in Git.
Operator runs publisher against prod.

Publisher creates productA in prod.
APIM automatically creates a subscription and associates it with productA.
Publisher checks whether productA existed in prod before it ran.

If it existed, nothing happens
If it did not exist, publisher deletes the automatically created subscription. That subscription would not exist in source control, so ApiOps removes it to ensure Git and APIM are as much in sync as possible.

This scenario is a bit of an edge case, but mentioning it in case you see unexplained subscription deletions when publishing.

@guythetechie Thanks for the information! That is helpful and will give us something to lookout for when testing.


No license

I'd quite like to reuse some of this in one of my projects (specifically: your xvfb-chromium script), but you don't include any licensing details in the repo, so I technically can't.
Assuming that you intended this to be open-source, any chance you could drop in an appropriate license?

Absolutely! https://github.com/mark-adams/docker-chromium-xvfb/blob/master/LICENSE

Awesome, thanks!


Async op samples usage

This adds a "samples" directory with sample usage of async_op. It shows how async_op may be used by various compositions of layered modules, including when a module calls another module with asynchronous retries or when a module must cancel all pending async_ops that are in-flight when it is closed.
The samples include some basic test coverage (though they do not have any "chaos" testing) to validate.
For now, this serves as a sample set of "design patterns", but it may be possible to extract some of this into macros or other modules for re-use.

/azp run

/azp run

/azp run

/azp run

/azp run

/azp run


UX / Save the Digital Ocean key under "API & Keys"

Shouldn't the Digital Ocean key be stored under "API & Keys"?
Currently, every time I launch a new host I need to look this up, while it could have been stored.

Our current feeling is we don't really want to be in the business of storing 3rd party credentials in our database.  This might be an option once we add pluggable encryption for sensitive fields (in the next few weeks).
But my immediate plan is to add storing the credentials into the browser's temporary window.sessionStorage or persistent window.localStorage (#494) so that they are at least saved locally instead of having to fill it out repeatedly.
There is also a 'Quantity' option when adding a docker machine host as of v0.19 (which is not released/tagged beta yet, but will be available this week).  So you can create several machines at once without going through the whole form again now.

So on a small tangent, you say we'll be able to start several machines. What happens if one machine goes down permanently (over x minutes), could you spawn a new one it its place?
IMO there should be a way to set the minimum number of containers to be running, AND the minimum number of hosts running. Is this in he roadmap?

A minimum number of containers will be possible before our beta launch by creating a service and setting a scale.
Doing the same with hosts is not currently on the roadmap.  If you wanted you could implement this outside of Rancher by calling the machine API, or completely outside of Rancher by just creating machines in $provider through their API and having cloud-init or similar run a script that installs Docker and registers the host into Rancher on startup.
There will be some nasty failure cases to think about... e.g. if the hosts are still actually up but the server running the auto-scaling becomes partitioned.  Or if more than one host disappears it's likely that provider is having a problem, so spinning up a new one might fail too.  And of course any actions you take cost actual money, so there should be limits on how many times it tries.

Thanks for the explanation. Is there a way I create a service and minimum scale in the web UI at the moment?
About you failure scenarios.

If the server running autoscaling fails, the recovery is something Rancher will have to address regardless of this specific case.
True, if the VMs are failing on digital ocean it may not be helpful to run more. But as a High Availability feature, it would be great to spin up more machines on AWS (or alternate vendor), in cases of catastrophic failure. Rancher is already 95% there with networking features.


[SPARK-44831][PYTHON][DOCS] Refine DocString of DataFrame.{union, unionAll, unionByName}

What changes were proposed in this pull request?
Refine DocString of Union*:

fix minor grammar mistakes
add more examples

Why are the changes needed?
to improve the docs
Does this PR introduce any user-facing change?
yes
How was this patch tested?
CI

thanks @srowen @allisonwang-db for review, merged to master and branch-3.5


Fix for Uri macro with org variable

without the _root_ in the uri macro, the added test fails to compile with error: value http4s is not a member of String
This issue was originally fixed in #741, but in the intervening years + refactors the _root_ qualifier was dropped. I'm just re-adding it, and I'm adding it to all the literal macros just to be safe.
fixes #704

Also realized that uri macro wasn't working if an implicit variable org was in scope
scala> import org.http4s.implicits._
import org.http4s.implicits._

scala> implicit val org: String = "the-problem"
val org: String = the-problem

scala> uri"test"
       ^
       error: value http4s is not a member of String

Added a test for this behaviour as well and confirmed that it fails without 532c401 and passes with it.


Add Fwastudents.org txt

Website: www.faithwest.org
AP Computer Sciences: https://www.faithwest.org/apps/pages/index.jsp?uREC_ID=2547359&type=d&pREC_ID=2254027
Proof of Email Domain: https://www.faithwest.org/apps/pages/index.jsp?uREC_ID=2548382&type=d&pREC_ID=2254194 (Under Accessing Online Textbooks)

@Shakenbeet Pull request merged. Thank you!


health_check_type variable for worker groups

I'm submitting a query

[ ] bug report
[ ] feature request
[x] support request - read the FAQ first!
[ ] kudos, thank you, warm fuzzy

If this is a bug, how to reproduce? Please include a code sample if relevant.
Not a bug
What's the expected behavior?
Is there a variable to set the health_check_type for the worker group? It defaults to EC2 while I would like to be able to set it to ELB. Or if there is any other way to do this while still using the module?
Environment details

Affected module version: 12.2.0
OS: Ubuntu/Linux
Terraform version: 0.13

Thanks in advance!

Nope, it doesn’t exist. But I'll happy to review your PR.

@barryib I have a PR.

I'm still waiting for someone to review my PR. Anyone?


Upgrade to ASM 9.3 for Java 19 compatability

See:
https://asm.ow2.io/versions.html
https://gitlab.ow2.org/asm/asm/-/compare/ASM_9_2...ASM_9_3

Thank you for your contribution @mnoteberg-non!


Add flag to output JSON

Several people on reddit asked for flag that would make output in JSON. This might be useful within scripts/pipelines.

done


No 'Access-Control-Allow-Origin' header is present on the requested resource

Before opening an issue:

Search for duplicate or closed issues
Prepare a reduced test case for any bugs
Read the contributing guidelines

When asking general "how to" questions:

Please do not open an issue here
Instead, ask for help on StackOverflow

When reporting a bug, include:

Operating system and version (Windows, Mac OS X, Android, iOS, Win10 Mobile)
Browser and version (Chrome, Firefox, Safari, IE, MS Edge, Opera 15+, Android Browser)
Reduced test cases and potential fixes using JS Bin, JSFiddle or CodePen

When suggesting a feature, include:

As much detail as possible for what we should add and why it's important to Cropper
Relevant links to prior art, screenshots, or live demos whenever possible

See the checkCrossOrigin option for more info.


[Question] Any plan to improve the Page-Holding?

I like your implementation of Virtualizing Data, but, for a my project, it's not possible to use the current implementation because does not discharge the pages when are not used!
There are any plan to improve the current Page-Holding mechanism with a new 'strategy' that reuse memory?

@gigios Sure, I had some ideas, but for me the hoarding strategy was sufficient sofar, so I continued to work on the project which uses this data-virtualizing library.
However, knowing that others are using this library and that other strategies are demanded is a great motivation to get back here.
I am currently at work, but I'll get back to this topic this weekend. Thank you for opening this issue :)

@gigios Hi,
I looked into that in the weekend and have concept in mind. I have also started to implement, but unfortunately I didn't finish it in last weekends time.
Sofar my plan is to implement the "Least Recently Used"-strategy (i.e. if necessary the least recently used page is dismissed) where the calling side has to provide the page limit. Additionally, I am going to implement the possibility to provide their own page replacement strategies for the users. Hence, if everything worksout as expected, then there'll be an interface to implement your own strategy against so you can infect it into the data-virtualizing collection.
I'm eager to include other strategies which make sense in general later on. The function for injection custom page-replacement strategies will provide a way to also use highly specialized strategies.
Will the "Least Recently Used"-strategy be sufficient for your requirements? Or would you need another strategy?
Another topic are IDisposables. In scope of this issue my plans at the moment are to automatically dispose the disposables of page which are dismissed. This is consistent to the behavior sofar, because if you dispose the data-virtualizing collection itself then it is going to dispose all currently hold disposables.
My intention is to implement a setting in future which switches of the disposal behavior in general. Would that be one of your requirements?

@Yeah69 Hi,
First of all thanks for the answer and for the investigation of a new solution.
I think your first solution is, for now, perfect (also about the IDisposable mechanism).
My first requirement is to reduce the memory usage, and probably your solution it's perfect!
I like also your idea to allow the user to 'extend' the possibility of this behavior!
Additional question (I don't have tested for the moment and not related directly this improvement):
I need to manage also new data to add to the collection. Is it possible in the current implementation without big problems in the UI? (sorry for this additional request)

@gigios Thank you for the feedback!
The data-virtualizing collection (from now on I'll abbreviate DVC) itself doesn't support functions like insert, add or remove and so on, because it doesn't make any assumptions about the backend you use. You just need to provide a function which fetches the total count of the elements and a function which gets a page of elements on demand (I call that functions fetchers). The DVC doesn't get any other information. It has no way to add or remove any element. Furthermore, the DVC has no way to know when the backend changed. All these are not the responsibilities which the DVC assigns to itself.
However, in my hobby project I do additions and removal of elements outside of the DVC. I use a SqLite backend. Whenever I do an operation which alters the data-virtualized SqLite table, I just dispose the old DVC and create a new one as a replacement. The new one will then fetch the pages with the updates included. The benefit of data-virtualizing is that it doesn't matter how big your virtualized data set is, you can dispose the old one anytime and replace it with a new one. The new one will only load the data it needs at the moment of replacement. You can see my approach to this in my other hobby project BFF.
Be aware if you use WPF as a frontend. It doesn't support data virtualization well and has some pitfalls and restrictions which require special care. In BFF I implemented a set of workarounds, but managed to integrate the DVC as ItemsSource for a DataGrid. If you get problems with that you can ask me.

@Yeah69 Thanks again for the answer.
I'm using WPF and my idea is to use your DVC library to manage the virtualization in a ListView, the problem (but probably it's not a real problem) it that the I must manage additional data (insert at top of my ListView, in a similar way of an email application when coming new emails).
Probably, as you have suggested, the best solution is to reload the data.

@gigios I have pushed the implementation of the new least recently used (LRU) page removal strategy. As well as the custom way to create own page removal strategies.
The LRU requires the parameter pageLimit and has an optional parameter removalCount (if not set explicitly its default is 1). As soon as a page is requested which leads to overflowing the pageLimit the LRU request page removals of the least recently used page. The count of the pages requested for the removal will force the page store to keep only "pageLimit - removalCount". Hence you could use the removalCount as a buffering function. For example, if you set the pageLimit to ten and the removalCount to five then the eleventh page will lead to five page removal so there will be a buffer of four pages until the next overflow.
But you can keep the removalCount at one, then after the eleventh page load all further page loads will lead to a page  removal.
For the custom was you just need to implement a lambda function. The only Parameter is an IObservable<(int PageKey, int PageIndex)> which is an event stream of element requests. PageKey is the index of the page in the page store and PageIndex is the index of the element inside of the page. That way you can track all element requests. Your lambda should return an IObservable<IReadOnlyList> which is an event stream of page keys which should be removed. I implemented the LRU that way, so I am sure that this will work. However, the custom way is the expert mode, I recommend to look into the code of this project before you use it.

@Yeah69 Thanks for this new 'very welcome' implementation!


Convert BooleanClause class to record class

Description
Convert BooleanClause class to record class which addresses #13207 partially.

Please add a note to lucene/MIGRATE.txt

By the way, I wonder how we should handle all the classes that we are converting to records, should we have a single section in lucene/MIGRATE.txt that lists all classes for simplicity, instead of having one entry per class?

That's a good idea @jpountz. I can add this change under this section: https://github.com/apache/lucene/blob/main/lucene/MIGRATE.md#some-classes-converted-to-records-classes-github13207

That's a good idea @jpountz. I can add this change under this section: https://github.com/apache/lucene/blob/main/lucene/MIGRATE.md#some-classes-converted-to-records-classes-github13207

That was also my plan. This is why I created the section in the migration guide. No need for separate sections, but a new entry in the enumeration is required.

The failing test is a known issue (see mailing list). Just ignore that.

Hi all. I understand the change to a record class but this does create a breaking change that would be avoided through the addition of deprecated accessors matching the old class ie getQuery and getOccur, which could be later removed in Lucene 11. Obviously other changes have been easier to manage in moving from Lucene 9->10 because methods have been flagged as deprecated in 9 and it has been straightforward to prepare for these changes. It is tricky because for users of Lucene with codebases that might run on either Java 11, or 21, one previously could have a codebase that would run using different Lucene versions, but this change means one has to choose the correct Lucene version AND keep two branches of their code.

Hi,
This was planned to be backwards incompatible. There is a migration guide entry.
Lucene generally revises APIs between major versions in incompatible ways. With Lucene 9 this was even more dramatic as a large part of the class and package structure was renamed to be Java module system compatible.
Lucene 10 was released last week and there won't be any changes anymore.
Uwe


ObsidianExtractNote leaves text in the original buffer, new note is empty

🐛 Describe the bug
Hi! ObsidianExtractNote is not working for me. With a visual selection, the text remains in the original note and the new note is empty.
Here is a video demonstrating the issue:
Screencast from 2024-03-09 15-57-43.webm
Config
return {
  {
    "epwalsh/obsidian.nvim",
    lazy = true,
    ft = "markdown",
    dependencies = { "nvim-lua/plenary.nvim" },
    opts = {
      workspaces = {
        {
          name = "personal",
          path = "~/Downloads/test-vault",
        },
      },
      note_id_func = function(title)
        local suffix = "untitled"
        if title ~= nil then
          suffix = title:gsub(" ", "-"):gsub("[^A-Za-z0-9-]", ""):lower()
        end
        return tostring(os.date("%Y-%m-%d")) .. "-" .. suffix
      end,
      daily_notes = {
        folder = "daily",
      },
    },
    init = function()
      vim.opt.conceallevel = 2
    end,
  },
}

Environment
NB: I have tried this with version = "*" as well as from the default branch.
 ~/.home.nix => nvim --version
nvim --headless -c 'lua require("obsidian").info()' -c q
NVIM v0.10.0-dev-6525832
Build type: Release
LuaJIT 2.1.1693350652
Run "nvim -V1 -v" for more info
Obsidian.nvim v3.7.0 (6888c7620109684371ed9304df2ba68abc78d558)
Status:
   buffer directory: nil
   working directory: /home/fausto/.home.nix
Workspaces:
  ✓ active workspace: Workspace(name='personal', path='/home/fausto/Downloads/test-vault', root='/home/fausto/Downloads/test-vault')
Dependencies:
   plenary.nvim: f7adfc4b3f4f91aab6caebf42b3682945fbc35be
   nvim-cmp: 04e0ca376d6abdbfc8b52180f8ea236cbfddf782
   telescope.nvim: d90956833d7c27e73c621a61f20b29fdb7122709
Integrations:
   picker: TelescopePicker()
   completion: enabled (nvim-cmp)
Tools:
   rg: ripgrep 13.0.0
Environment:
   operating system: Linux
Config:
   notes_subdir: nil%

I can reproduce

Ah, silly mistake. Fixed in dc60381.


.post data request problem in Alamofire 4

using alamofire4 cant request data in post method
 let parameters = [
            "name": "xx",
            "encoding": "utf8",
            "data": "1"
            ]
Alamofire.request(urlString, method: .post, parameters: parameters, encoding: JSONEncoding.default).responseJSON { response in
        if(response.result.isFailure){
               print("no data!");
        }else{
              print(response.result.value);
       }
}

my console always print no data

Please check Url type : http or https

you can use to configure App Transport Security:
NSAppTransportSecurity

NSExceptionDomains

yourserver.com

NSIncludesSubdomains

NSTemporaryExceptionAllowsInsecureHTTPLoads

NSTemporaryExceptionMinimumTLSVersion
TLSv1.1

i already set NSAllowsArbitraryLoads to true
my server is using http , previous alamofire3 no problem , but alamofire version 4 got problem

i fixed already , just change the encoding
encoding: URLEncoding.httpBody

Thanks for reporting back @rudian. In the future, please take questions like these to Stack Overflow.
Cheers. 🍻

I have this identical problem...  its not fixed by changing the encoding


support specifying the provider port at runtime

Because multiple project share the same build node, so we need use a random port for the service provider. So it would be nice if pact jvm gradle can support specifying the provider port at runtime just like it support specifying the provider host at runtime.

I guess this could be done. But I'd like to suggest another approach which we do in my org. We run our provider service in a docker container, and use --expose to expose the static port in which its running. This way you can run as many containers as you want on the same build node, without collisions.

This has been released with version 3.3.2


Adding the Ballerina style guide

Purpose
Adding the Ballerina style guide to the Ballerina website.
Related PRs

List any other related PRs.

Special notes

Any special testing needed to verify integrity.

Any checklist items to be verified before production deployment?
Special styling concerns?
Related media files?

BTW commit message should not end with a period ...
Follow this guideline when writing commit messages ....
https://chris.beams.io/posts/git-commit/


Expose integrated terminal extension API

The initial extension API will allow:

Creating and optionally naming of a terminal instance
Sending commands (text sequences + '\n') to these "owned" terminal instances

Additional APIs will be considered via feedback from consumers.
Related: https://github.com/Microsoft/vscode/issues/547

please keep me in the loop

The current plan is to be able to create terminals and send text to them:
CreateTerminal(name?: string): Terminal
Terminal.runCommand(text: string): void

@weinand what sort of API would you expect for the debugging support which we eventually want (#10514)?

I would also suggest dispose, show, hide, name, like so
interface Terminal {
  name:string;
  runCommand(cmd:string)
  show(preserveFocus: boolean): void
  hide(): void;
  dispose(): void;
}

Also, I wonder if runCommand can return a promise that signals when execution is done, resolving to the exit code?

@Tyriar the proposed API seems to cover most of our needs.
Ideally it would be great if Terminal.runCommand() would return the process id of the command, but that's probably too tricky.

Thanks @weinand @jrieken, I'm not sure its possible to get the process ID as we're tracking the shell process, I'll look into whether we can get any info about the running process from the shell process.

There doesn't seem to be an easy way using pty.js to get the current process, however platform-specific options may exist. Given the process ID of the shell, we can get the child process(es) for it:
~
❯ ps aux | grep ./terminalProcess | head -n 1
daimms    6698  0.0  0.3<PHONE_NUMBER>2 ?        Sl   05:48   0:00 /usr/share/code-insiders/code-insiders

~
❯ ps aux | grep $(pgrep -P 6698) | head -n 1
daimms    6702  0.0  0.0  24628  6116 pts/19   Ss   05:48   0:00 /bin/bash

~
❯ ps aux | grep $(pgrep -P 6702) | head -n 1
daimms    8661 23.1  8.5 2147228 1041596 pts/19 Sl+ 06:00   0:37 gulp

On OS X I'm using an script that filters the output of psbased on the 'pty' (pseudo tty) of the terminal window and the command name:
ps -c -t pty | awk '$4== "name" { print $1 }'

I'm think sendText(text: string, addNewLine: boolean = true) may be better than runCommand as that more accurately describes what is actually happening.

@weinand that allows fetching of child processes on OS X?

Man page info about -t option of ps: "Display information about processes attached to the specified terminal devices".
So this is not exactly fetching the child processes... but it works fine for me.

I can only get process info about the currently running tty using -t, we would need to do on a separate process otherwise output would show to the user:
This gives what we want if run in the shell currently being run in:
❯ ps -t
  PID TTY      STAT   TIME COMMAND
 8939 pts/20   Ss     0:00 /bin/bash
12135 pts/20   R+     0:00 ps -t

We would want something like ps -t <where shell PID=...>, not sure if it's possible using -t currently.

On OS X the 't' option takes as an argument the name of a tty and I can retrieve that name from the  new terminal.

@weinand where name is something other than the PID? The proposed name param in the Terminal constructor is only a label for the dropdown currently.

@Tyriar assume I want to launch 'mono' in an external terminal as a debug target.
I need mono's process id because I need to kill it when debug sessions ends.

I create a terminal on OS.
I launch 'mono' in that terminal (but I cannot get the process id from this).

here is the pseudo code of what I do to find the pid of the mono process:
pty = terminal.tty    // get the tty device name from the terminal
name = "mono"    // the command that was launched in the terminal

// find all processes that were launched in the given terminal by the given command name
pid = ps -c -t $pty | awk '$4== $name { print $1 }'    // ideally this results in exactly one process id

Reopening as Terminal.hide() closes the panel regardless of whether the terminal is active or not.

Is there any terminal API provided for extension developers to create a terminal with the specified path as the  initially start ,thank you @weinand

@Tyriar One more thing - we have an extension vscode-api-tests in which you can write tests against the (latest) API. They will be run as real extension with scripts/test-integration.sh.

@guogaigai please direct questions about the integrated terminal to @Tyriar. I'm only a client of that API.

Closing this off as it believe it's done, forking the test task to https://github.com/Microsoft/vscode/issues/11132


WIP: Fix boulder in tests

Fixing boulder in tests

Btw, may want to pin the tests to a specific version of boulder so the tests aren't tied to changes to that package.

Yeah I'll work on this more this week. Should I just make a submodule in a test directory?

 Downloading this package should be a light operation.


upgrade expected but low level API in use when using gotham w/ websocket

Gotham's example of websocket is broken on my computer. Its hyper keeps complaining "upgrade expected but low level API in use".
https://github.com/gotham-rs/gotham/tree/master/examples/websocket

In order to use hyper::Body::on_upgrade, the server needs to enable upgrades on the server Connection. Looks like Gotham is using Http::server_connection(socket, service), it needs to add .with_upgrades() to that.


Run dependency actions on entire hierarchy

Make sure we run the dependency actions on the entire hierarchy before traversing that hierarchy for dependencies and excludes. This is because the dependency actions may themselves modify the hierarchy, so before traversing we need to make sure the hierarchy isn't going to change out from under us.
This should be fine since configurations' dependency actions are idempotent. The actions automatically de-register themselves, so once they're called, they're not triggered for subsequent calls.
Fixes: #24234

@bot-gradle test this

@bot-gradle test and merge

I've triggered a build for you.


Updating fill on G element will not fire re-render of paths

So, I tried to update fill of Group of Paths, but Paths color was not updated.

Same issue here. My hacky workaround was to set the key of the component to the color I wanted ti to be, so it forced a re-render whenever the color changed.


[zh] Sync content/zh/docs/concepts/configuration/configmap.md

[zh] Sync content/zh/docs/concepts/configuration/configmap.md

/lgtm
This PR looks good. Preferably, you can sign on a few items outlined in issue #27672, so that the synchronization work can be coordinated.

OK, I'll do it


Update .openpublishing.publish.config.json

OPS Build status updates of commit a41b605:
:clock10: Pending: waiting for processors (8 builds ahead of you)
:warning: OPS Build is busy, currently there are 8 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

OPS Build status updates of commit a41b605:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details
Dependent Files

.openpublishing.publish.config.json
:x:Error

Details

.openpublishing.publish.config.json

[Error] Publish service's config file: .openpublishing.publish.config.json is not valid. Details: System.ArgumentException: Invalid json string ---> Newtonsoft.Json.JsonReaderException: Unexpected character encountered while parsing value: e. Path 'docsets_to_publish[0].moniker_ranges', line 9, position 27.
at Newtonsoft.Json.JsonTextReader.HandleNull()
at Newtonsoft.Json.JsonTextReader.ReadStringValue(ReadType readType)
at Newtonsoft.Json.JsonTextReader.ReadAsString()
at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.ReadForType(JsonReader reader, JsonContract contract, Boolean hasConverter)
at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.PopulateList(IList list, JsonReader reader, JsonArrayContract contract, JsonProperty containerProperty, String id)
at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateList(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, Object existingValue, String id)
at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.SetPropertyValue(JsonProperty property, JsonConverter propertyConverter, JsonContainerContract containerContract, JsonProperty containerProperty, JsonReader reader, Object target)
at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.PopulateObject(Object newObject, JsonReader reader, JsonObjectContract contract, JsonProperty member, String id)
at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateObject(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.PopulateList(IList list, JsonReader reader, JsonArrayContract contract, JsonProperty containerProperty, String id)
at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateList(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, Object existingValue, String id)
at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.SetPropertyValue(JsonProperty property, JsonConverter propertyConverter, JsonContainerContract containerContract, JsonProperty containerProperty, JsonReader reader, Object target)
at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.PopulateObject(Object newObject, JsonReader reader, JsonObjectContract contract, JsonProperty member, String id)
at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateObject(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.Deserialize(JsonReader reader, Type objectType, Boolean checkAdditionalContent)
at Newtonsoft.Json.JsonSerializer.DeserializeInternal(JsonReader reader, Type objectType)
at Newtonsoft.Json.JsonConvert.DeserializeObject(String value, Type type, JsonSerializerSettings settings)
at Newtonsoft.Json.JsonConvert.DeserializeObject[T](String value, JsonSerializerSettings settings)
at Microsoft.OpenPublishing.Build.Common.JsonUtility.FromJsonString[T](String value) in D:\a\1\s\Libraries\Common\JsonUtility.cs:line 93
--- End of inner exception stack trace ---
at Microsoft.OpenPublishing.Build.Common.JsonUtility.FromJsonString[T](String value) in D:\a\1\s\Libraries\Common\JsonUtility.cs:line 97
at Microsoft.OpenPublishing.Build.WorkflowTasks.SyncGitSourceWithDownloadingPackage.CheckPublishConfig() in D:\a\1\s\Libraries\WorkflowTasks\SyncGitSourceWithDownloadingPackage.cs:line 68

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

OPS Build status updates of commit f4ccfcb:
:clock10: Pending: waiting for processors (8 builds ahead of you)
:warning: OPS Build is busy, currently there are 8 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

OPS Build status updates of commit f4ccfcb:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

.openpublishing.publish.config.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

OPS Build status updates of commit 01abb9b:
:clock10: Pending: waiting for processors (8 builds ahead of you)
:warning: OPS Build is busy, currently there are 8 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

OPS Build status updates of commit 01abb9b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

testver/index.md
:bulb:Suggestion
View
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

testver/index.md

[Suggestion] Missing attribute: author. Add a valid GitHub ID.
[Suggestion] Missing attribute: ms.author. Add a valid Microsoft alias.
[Suggestion] Missing attribute: either ms.prod or ms.service is required. Use ms.prod for on-premise products, or ms.service for cloud services.
[Suggestion] Missing attribute: ms.topic. Add a valid value to indicate the article type.
[Suggestion] Missing attribute: ms.date. A freshness date is required for this content set. Add a date in format MM/DD/YYYY.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

OPS Build status updates of commit 323393a:
:clock10: Pending: waiting for processors (8 builds ahead of you)
:warning: OPS Build is busy, currently there are 8 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

OPS Build status updates of commit 323393a:
:warning: Validation status: warnings

File
Status
Preview URL
Details
Dependent Files

testver/TOC.yml
:warning:Warning

Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

testver/docfx.json
:white_check_mark:Succeeded

testver/index.md
:white_check_mark:Succeeded

testver/TOC.yml

Line 0: [Warning] Invalid file link:(~/index.md).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

OPS Build status updates of commit 323393a:
:clock10: Pending: waiting for processors (30 builds ahead of you)
:warning: OPS Build is busy, currently there are 30 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

OPS Build status updates of commit 323393a:
:warning: Validation status: warnings

File
Status
Preview URL
Details
Dependent Files

testver/TOC.yml
:warning:Warning

Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

testver/docfx.json
:white_check_mark:Succeeded

testver/index.md
:white_check_mark:Succeeded

testver/TOC.yml

Line 0: [Warning] Invalid file link:(~/index.md).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fakeban

Hi, how fakeban works and how to use it?

Fake ban is broken, but you enable and shot at your friends (only works on team kill server).

@rosesware not true. The correct way to use it is after u enable it u type disconnect in the console and everyone else will see u disconnect from the game b/c u got banned

color is working, however only in competitive it seems.

@xAkiraMiura for me, its white color, its doesnt change color

@gosu821 u can't see urself only others can see after u disconnect

@thomas0aotian thx a lot

@gosu821 what gamemode are you trying this in? in my testing gamemodes in which you dont actually do damage it shows up as white but if you can do damage (wingman, comp, dangerzone) it shows up in color.

Fake ban is broken (color's), but you enable and shot at your friends (only works with team kill enable).
I use the clan tag changer to make it work, I understand very well but it works perfectly when the clan tag is on.

@Evilhelder Forgot to mention that fixes it, you can use unicode ZERO_WIDTH_SPACE to fix it without having a tag show up in chat.

can confirm you don't need to have clan tag on to make it work https://i.imgur.com/PiPlsyA.png just did this in a wingman:)