Automatically prefill Custom Shortcuts

From path like C:/Games which most gamers often have.
For example, I ain't seeing Starcraft II and other games from blizzard. GOG games etc.
I know it might sound  uneasy, for example in SC2 folder we have some more executables:

Plus, sometimes we have files like unins000.exe, UnityCrashHandler64.exe, etc. But I think it's possible to find a solution to filter out unnecessary executables.
For example, like with Stacraft II, and many other games, – it's good practice to name your executable the same way the folder named:
C:/Games/StarCraft II/StarCraft II.exe

So, in most cases it should be fairly easy to determine from what exact executables we need to create links.
Sometimes they might be slightly different though, for example:
C;/Games/Project Zomboid/ProjectZomboid32.exe
C;/Games/Project Zomboid/ProjectZomboid64.exe

So, there's two other things to take into consideration:

Obviously, we wanna prefer x64 version ((?:64) in regex)
Plus, the executables don't always have a space in it (gonna put ? for each space for regex)

I guess there should be a couple regular expressions to test. Something like (pseudo-code):
executableNameRegex = gameFolderName..replaceAll(' ', ' ?') + ' ?(?:\\(?x?64\\)?)?' + '\\.exe'"

This takes folder name, optionally checks for 64 x64 (64) (x64), plus ends with .exe – a string ready to be used as regex input to iterate through files to find actual game executables.
This way, it should be possible to find 90% of game executables, I think. Automatically.

Hello @jerrygreen .
Thanks for your contribution!
Regarding your idea, I have developed plugin with strict no guessing policy. That's why I introduced custom shortcuts to allow users to explicitly add games, this allows us not to add game with rule "maybe it's a game, but maybe not, we don't really know". My second concern is that users might not want some plugin to scan all their files and directories 😉. Security Concerns
Regarding GOG games, support is coming soon, and Battle.net is planned for the future.
That said, I am not against the idea of adding a new sync engine designed to search for games from the user's local drive. But it needs to be separated from custom shortcuts, marked as experimental, and disabled by default. And of course this is not the highest priority to work on right now.
I hope this explanation is understandable for you, if you had any questions then go ahead and write.


SAPT0_ROHF test failing

When I run pytest locally, it's failing the test_SAPT0_ROHF test on the second to last line of Symmetry-Adapted-Perturbation-Theory/SAPT0_ROHF.py on the psi4.compare_values(Eind, Ind20r, 5, 'Ind200,r') call with the following error: Ind200,r: computed value (-0.000251) does not match (0.000000) to 5 digits. @dgasmith has suggested that perhaps Psi4 has changed the name of its variables, causing this error?

I just tested master p4n with a near-master psi4, and it was clean. Is your p4n rebased? And what's your psi4 version?


it no work

poc no work, please do poc correctly

poc fucked. me need help. bring help. me want poc work also.


Continuous integration test bin/unit-test.sh is broken

I think this is also the case for exercism/java, but the CI test fails when it tries to download the configlet:
>>> Running configlet...
gzip: stdin: not in gzip format
tar: Child returned status 1
tar: Error is not recoverable: exiting now
The command "$SCRIPT" exited with 2.

This is a problem with the bin/unit-tests.sh execution. More specifically, I think it is a problem in bin/fetch-configlet at this line:
curl -s --location $URL | tar xz -C bin/;;

For whatever reason, the value returned does not apparently have a proper gzip format and it is breaking that integration test.

FYI: I think I have found the bug for this. https://github.com/exercism/java/issues/1799#issuecomment-613521822


Add support for TopLevelEncoder in XMLEncoder

After #160 is merged, rootKey property is not required, thus making it possible to add an implementation of TopLevelEncoder.

Resolved in #175


Typing in the documentation search bar scrolls the page

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
When typing something in the search bar the page scrolls up.
https://user-images.githubusercontent.com/24257556/155130144-4d48b9e5-1725-4593-9356-d45afc7e7afd.mp4
Reduced test cases
Visit the docs, scroll down some page and type stuff in the search bar.
What operating system(s) are you seeing the problem on?
Linux
What browser(s) are you seeing the problem on?
Chrome, Firefox
What version of Bootstrap are you using?
Whichever the docs for v5.1 use

I think we already tracked this down to scroll-margin-top or something. I can't find the related issue or PR though.

PR: #35393

Closing as duplicate of #33993, thanks @nkdas91.


CCLayerPanZoom: Mac support

Subject. Add methods for mouse and trackpad.
Add target to Mac project.

Moved to "0.2.x" milestone.
We will release only iOS version of CCLayerPanZoom in 0.2


rolling up main to beta

Check all that apply
Type of Change

[ ] WWST Certification Request

If this is your first time contributing code:

[ ] I have reviewed the README.md file
[ ] I have reviewed the CODE_OF_CONDUCT.md file
[ ] I have signed the CLA

[ ] I plan on entering a WWST Certification Request or have entered a request through the WWST Certification console at developer.smartthings.com

[ ] Bug fix
[ ] New feature
[ ] Refactor
[x] Release

Checklist

[ ] I have performed a self-review of my code
[ ] I have commented my code in hard-to-understand areas
[ ] I have verified my changes by testing with a device or have communicated a plan for testing
[ ] I am adding new behavior, such as adding a sub-driver, and have added and run new unit tests to cover the new behavior

Description of Change
Summary of Completed Tests

Duplicate profile check: Warning - duplicate profiles detected. fan-wind.yml == fan.yml

This is intentional based on new naming conventions used in the matter-thermostat driver when assigning a profile when a device is onboarding.


feat: add emitLegacyCommonJSImports flag

Description
See changeset
Related #8067

Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Screenshots/Sandbox (if appropriate/relevant):
Adding links to sandbox or providing screenshots can help us understand more about this PR and take action on it as appropriate
How Has This Been Tested?
Updated dev-tests
Checklist:

[X] I have followed the CONTRIBUTING doc and the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[X] Any dependent changes have been merged and published in downstream modules

Further comments
Couldn't figure out how to make it work for oclif plugin

Close https://github.com/dotansimha/graphql-code-generator/pull/8094/files is this is merged first


PowerShell Core servicing release for v6.0.4

From:  https://github.com/PowerShell/PowerShell/blob/master/docs/maintainers/releasing.md

[X] Create a private branch named release-v6.0.4 in VSTS repository.
All release related changes should happen in this branch.
[ ] Run tests on all supported Linux distros and publish results
[ ] Prepare packages

[x]  Build release packages
[ ]  Sign the MSI packages and RPM package.
[ ]  Install and verify the packages.

[ ]  Update documentation, scripts and Dockerfiles

[ ]  Update CHANGELOG.md with the finalized change log draft.
[ ]  Update other documents and scripts to use the new package names and links.

[ ]  Create NuGet packages and publish them to powershell-core feed.
[ ]  Publish Linux packages to Microsoft YUM/APT repositories.
[ ]  Create the release tag and push the tag to PowerShell/PowerShell repository.
[ ]  Create the draft and publish the release in Github.
[ ]  ~Merge the release branch to Github master with a merge commit. Then delete the release branch.~
[ ]  Trigger the release docker builds for Linux and Windows container images.

hi ,Please https://packages.microsoft.com administrators pay attention to packagename .  see: #7385

1 Congratulations, you deleted the package prew.2     ，i tested，it work fine.
2 However, due to historical reasons, it cannot be used from prew.2
Yum install -y powershell, or yum update -y powershell upgrade. Because prew.2 is newer than the 6.04 version.
3 need yum remove -y powershell ; yum install -y powershell . Can it be added to the document（ release note ）?
4 If not included in the documentation, prew2 will be the most used version.

ubuntu has same problem.

Will there be a build for Ubuntu 18.04?  17.04 went EOL Jan 2018.

@tmknight Ubuntu 18.04 is supported on PowerShell Core version 6.1.*.


🚀 Remove Annotations and Tag All text elements (optionally)

Tarsier now tags using spans with a unique ID (allows easy removal)
Removes annotations after tarsier is done with them
Adds option to tag all text elements

Bonus: Since we are now annotating using spans we can now add CSS to the annotations so that vision model can see them easier
fixes #5 #7

Run black ⬛

LGTM!


Vanessa vancleef hero

Could someone add vanessa vancleef to the script please?

the main branch already added vanessa vancleef

the data of this hero have already been added to the main branch

Thank you


Using commas as table separators is annoying

It requires escaping comma in sentences. There should be an alternative. We could use the | like other Markdown flavors, but we already use that for images and videos.

Need to fix this soon, before writing a million books with annoying escapes everywhere.


Add a websocket server feature

For data streaming and visualization web frontends cannot use the STOMP format, and the most suitable streaming protocol is WebSockets.
See the implementation in the ViSR - plotting server https://github.com/DiamondLightSource/ViSR/tree/main/src/plotting_server
as well as davidia (though there the example is more complicated and not for live data): https://github.com/DiamondLightSource/davidia
The server state for now would encompass only a collection of websocket connections, and the RMQ url. Possibly a session holding logic could be introduced later if a race condition appears.
the race condition case
given a set of distributed processes - job scheduler, message bus, websockets relay service (this) and the client:
if a client sends a job to the job scheduler, receiving a job_id in response and messages start to be emitted in the message bus and consequently in the relay service, the client either needs to be already subscribed OR only once getting a job_id would ask the relay server  to listen for data for this particular job.
Caching should be server side as the client can be easily disconnected, a pythonic solution might be needed, for instance redislite . JWT authentication might be necessary - I am not sure how would the auth gateway work for a WS protocol, which is not http.
Acceptance Criteria

I can spin up a pod on a k8s cluster to listen to a given RMQ instance and listen for incoming WS connections
web clients can connect to the server and

@keithralphs has worked on something similar
We need to decide if this repo is where this functionality belongs
I agree this is needed short term, long term should it be part of the supergraph? I believe that will support websocket subscriptions

3 we don't need to consider longterm to deliver this
2 the user story delivered (plotting ) will work if this functionality is here. that is one concerns addressed. what criteria do you have in mind @callumforrester  and how much of delay do you expect this decision to cause. modular functionalities can be portable. I already thought that this is a closed aspect when you said it's not in the blueapi.

we don't need to consider longterm to deliver this

This library is designed to be stable and well-maintained with a focus on long-term goals. After considering your proposed solution, it seems better suited for a different context, so I'm going to close this ticket.
I suggest creating a monorepo for short-term prototypes aimed at delivering SWIFT functionality rapidly. You could even coopt i18-bluesky for that purpose.


Update react-router dependency.

There is a new (major) release of react-router.
It would be great if someone could update the peerDependency so we get rid of the warning:
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

Is it me or this isn't published yet to npm?

@mattecapu No, you're right. This isn't published yet.

It is published now. Sorry for delay. Was busy moving to another city...

Thanks @denvned , but the latest version on npm (0.8.5) still defines ^2.3.0 as peer dependency for react-router.
https://registry.npmjs.org/isomorphic-relay-router/-/isomorphic-relay-router-0.8.5.tgz

@aduth Thank you for noticing. Fixed.


pull request fails when running on a repo which has no pull requests.

It seems like the issue has to do with grabbing the pr['number'] which is then appended to the string. Since this is a repo with no pull requests this fails because the generated string ends incomplete and github api returns 404 for the api endpoint not being found.
Here is the error I am getting:
Identity added: /tmp/git-resource-private-key (/tmp/git-resource-private-key)
/usr/lib/ruby/gems/2.4.0/gems/octokit-4.8.0/lib/octokit/response/raise_error.rb:16:in `on_complete': GET https://api.github.com/repos/syncaide/agent/pulls/: 404 - Not Found // See: https://developer.github.com/v3 (Octokit::NotFound)
	from /usr/lib/ruby/gems/2.4.0/gems/faraday-0.14.0/lib/faraday/response.rb:9:in `block in call'
	from /usr/lib/ruby/gems/2.4.0/gems/faraday-0.14.0/lib/faraday/response.rb:61:in `on_complete'
	from /usr/lib/ruby/gems/2.4.0/gems/faraday-0.14.0/lib/faraday/response.rb:8:in `call'
	from /usr/lib/ruby/gems/2.4.0/gems/faraday-http-cache-2.0.0/lib/faraday/http_cache.rb:290:in `fetch'
	from /usr/lib/ruby/gems/2.4.0/gems/faraday-http-cache-2.0.0/lib/faraday/http_cache.rb:195:in `process'
	from /usr/lib/ruby/gems/2.4.0/gems/faraday-http-cache-2.0.0/lib/faraday/http_cache.rb:142:in `call!'
	from /usr/lib/ruby/gems/2.4.0/gems/faraday-http-cache-2.0.0/lib/faraday/http_cache.rb:115:in `call'
	from /usr/lib/ruby/gems/2.4.0/gems/faraday-0.14.0/lib/faraday/rack_builder.rb:143:in `build_response'
	from /usr/lib/ruby/gems/2.4.0/gems/faraday-0.14.0/lib/faraday/connection.rb:387:in `run_request'
	from /usr/lib/ruby/gems/2.4.0/gems/faraday-0.14.0/lib/faraday/connection.rb:138:in `get'
	from /usr/lib/ruby/gems/2.4.0/gems/sawyer-0.8.1/lib/sawyer/agent.rb:94:in `call'
	from /usr/lib/ruby/gems/2.4.0/gems/octokit-4.8.0/lib/octokit/connection.rb:156:in `request'
	from /usr/lib/ruby/gems/2.4.0/gems/octokit-4.8.0/lib/octokit/connection.rb:19:in `get'
	from /usr/lib/ruby/gems/2.4.0/gems/octokit-4.8.0/lib/octokit/client/pull_requests.rb:31:in `pull_request'
	from /usr/lib/ruby/gems/2.4.0/gems/octokit-4.8.0/lib/octokit.rb:46:in `method_missing'
	from /opt/resource/lib/commands/in.rb:78:in `pr'
	from /opt/resource/lib/commands/in.rb:20:in `output'
	from /opt/resource/lib/commands/in.rb:110:in `<main>'

@crstamps2 Getting back to you a little late but thanks for the quick response. The issue was on my end. I have built another resource that notifies other resources. Sorta a meta resource that uses the fly check-resource via the api. The reason I was getting the issue was because my resource was returning success while the other (prs) were not. This is more of a bug for a feature that doesn't exist yet. :smiley:
Going to go ahead and close this issue.


Implemented Foundation for Sites 6.x style

Hello,
I implemented the Foundation for Sites 6.x styling (though I think it'd work for Foundation 5.x as well). I updated an affected test and added another test for Foundation styling.
Since this now makes three visual styles, I could update the README to note the different styles it now supports.
Let me know if you need anything else.
Thanks

It would be awesome if you could update the readme for me.
Looks great though!

Updated README. Let me know if anything looks odd or if you'd like the wording changed.
Thanks again!

Awesome! Thanks!


Ensure wlSqlOptions are always used when processing raw query

Hey @tjwebb – sorry, I think I made an incomplete fix yesterday on #20 .  I was trying to only affect toKnexRawQuery() for the one code-path that I understood, and maintain its original behavior in other cases.  Turns out those other cases (i.e. when it's used while building join queries) needed to be adjusted as well.  I've adjusted it so that toKnexRawQuery() always takes the adapter's wl-sql options into account.

@devinivy ok, makes sense. I'll take a look this afternoon and try to publish before the end of the day. I know you guys are working hard right now :)


[LayoutLM]Code and pre-trained model with image features.

Thanks a lot for the great work and releasing pre-trained LayoutLM models!
After playing with the two pre-trained models, they don't seem to have any image features. Is there a plan to release code and pre-trained models with image features? Thanks.

@w-cheng Thanks for your interests in LayoutLM! As mentioned in our paper, currently we do not pre-train the LayoutLM model with the image information, but sideload the image features after the pre-training step. So, you may integrate any fast(er) RCNN models and generate the image features that can be fine-tuned together with the LayoutLM.

@wolfshow Thanks for the answer, that makes a lot of sense.


Admin UI won't load after deploy to server

Hi,
After deploy to server with Strapi v4.1.9  Admin UI won't load.
Are you aware of this issue? Should I provide details?
Thank you!

Hello,
I am not aware of this issue, please do provide all possible details (plugin version, error in console etc).

Closing due to inactivity, will reopen if additional information/reproduction is provided.


can't override dialog container padding.

how to override paddingTop which is atleast '16px' inDialog? I want my modal to be fullscreen.
In material-ui's dialog.js minPaddingTop=16 is written.

Was this resolved for you? I am closing this issue for now. If you still have the issue do let me know in comments below I will reopen it.

@tintin1343 I'm currently having this issue. The dialog code enforces a padding top of 16. This is a blocker for having a model dialog be full screen. You can see the line here:
https://github.com/callemall/material-ui/search?utf8=✓&q=minPaddingTop
Please advice if I can propose a fix in a PR.
thanks!


Can I save previous State of search when we use ajax ?

I have use Ajax call to get list data in steps .. in single selection mode suppose i am searching something and minimum Search length is 0 so when it open it start new search and previously loaded list gone ... for me it should only rest when user do search ..is there any hack for it or any control ?
here is my [stack-overflow] (http://stackoverflow.com/questions/31585865/select2-save-current-list-and-values) question with fiddle
here is same [Fiddle] ( http://jsfiddle.net/amitwork/pbrr5dfg/ )
Also after searching text get clean on next click or open drop-down

Let's keep this to Stack Overflow.

This appears to be more of a usage question than a bug report or common feature request.  As we continue to keep the Select2 issue tracker geared towards these two areas, bug reports and common feature requests, we are redirecting usage questions to other communities such as the mailing list, IRC, or Stack Overflow.


Adjust JVM options to fix Jolokia agent problems

This is a workaround for https://github.com/rhuss/jolokia/issues/198

If merged, this will remove the need for #292.

Works for me locally. +1.

Thanks, merged.


samtools command failed (align-trim)

Hello,
Following the nCoV-2019 novel coronavirus bioinformatics protocol, when I try to run the "Run the MinION pipeline" part
artic minion --normalise 200 --threads 4 --scheme-directory ~/artic-ncov2019/primer_schemes --read-file run_name_barcode03.fastq --fast5-directory path_to_fast5 --sequencing-summary path_to_sequencing_summary.txt nCoV-2019/V3 samplename
...it stops with the following error message: "samtools sort: failed to read header from "-""
What could be the trouble here? A different samtools version maybe?
Best regards,
Bernardo Werner

Hi Bernardo
Sorry to reopen a closed thread, but I was wondering if you managed to find a solution to this (or even if this was the issue)? I am getting the exact same error (running artic through nextflow) and this is literally the only thing to be found online regarding it.


Enabling prometheus on traefic causes container to crash on kube 1.8.1

Is this a request for help?:

Is this a BUG REPORT or FEATURE REQUEST? (choose one):
Bug

Version of Helm and Kubernetes:
Client: &version.Version{SemVer:"v2.7.0", GitCommit:"08c1144f5eb3e3b636d9775617287cc26e53dba4", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.7.0", GitCommit:"08c1144f5eb3e3b636d9775617287cc26e53dba4", GitTreeState:"clean"}

1.8.1
Which chart:
NAME                    REVISION        UPDATED                         STATUS          CHART           NAMESPACE
sanguine-butterfly      3               Sat Nov  4 21:11:19 2017        DEPLOYED        traefik-1.13.0  kube-system

What happened:
Pod wont start
2017/11/04 21:06:42 Error reading TOML config file /config/traefik.toml : Near line 22 (last key parsed 'web.metrics.prometheus'): expected a top-level item to end with a newline, comment, or EOF, but got ',' instead

What you expected to happen:
Container to come up and ingress controller to work.
How to reproduce it (as minimally and precisely as possible):
My values.yaml
metrics:
  prometheus:
    enabled: true
    buckets: 0.1,0.3,1.2,5
  datadog:
    enabled: false
    # address: "localhost:8125"
    # pushInterval: 30s
  statsd:
    enabled: false
    # address: "localhost:8125"
    # pushInterval: 30s

Anything else we need to know:

Same issue on 1.7.4
Without Prometheus everything works fine.

Similar problems with the datadog and statsd additions, it's because the config values are not quoted in the config map, introduced in https://github.com/kubernetes/charts/pull/1622.
Workaround is to specify a value including quotes:
metrics:
  prometheus:
    enabled: true
    buckets: "\"0.1,0.3,1.2,5\""

Thanks @c-knowles. I'll give that a go.
Spins up fine if I edit the configmap traefik.toml file and changed the metrics section to:
[web.metrics.prometheus]
  Buckets = [0.1,0.3,1.2,5.0]

@c-knowles the quotes cause it to be treated as a string while Traffic requires a slice.
I ended up creating a value.yaml like this:
loadBalancerIP: <myLBIP>
ssl:
  enabled: true
acme:
  enabled: true
  email: <myEmail>
  staging: false
metrics:
  prometheus:
    enabled: true
    buckets: "[0.1,0.3,1.2,5.0]"

Make sure you change the 5 to a float by adding the .0 or the pod throws an error.

Ah sorry, didn't realise it was expecting that - I'd only tried the datadog entry which needs a string host.

Nice, Thanks for the work around. @troywilson

Oh hey, anyone else notice in the traefik dashboard under /health it's blank? No Data? Mine worked back when I had 1 ingress (the dashboard itself) but no longer works now that I added a couple of services.


Caching side effects

The caching from @nicooga https://github.com/twg/devour/commit/44701c4d674ee944539afb366192ec3bdcba0ac5 has some unexpected side effects ...
Example
You first request is Entity A
Your second request is Entity A again with some included relationships, e.g. SubEntityA
The cache is used for Entity A and SubEntityA is null, but it was send to the Client ...

Im sorry to hear that. Can you expand?

The cache is used for Entity A and SubEntityA is null, but it was send to the Client ...

What do you mean by entity is null?
I suggest flushing the cache just before returning deserialized model to se wether that solves the issue

Flushing is a good solution...
The returned model of the second request don't contain sub entity a, because it was missing on the first request and in the cache

👍
I also got heavy side-effects and just had to downgrade to 1.4.1 to avoid them.
In my case I got a call to an endpoint that serves and included object like this:
{
  "data": {
    "id": "428",
    "type": "patients",
    "attributes": {
       ...
    },
    "relationships": {
      "leads": {
        "data": [
          {
            "id": "9",
            "type": "leads"
          }
        ]
      }
    }
  },
  "included": [
    {
      "id": "9",
      "type": "leads",
      "attributes": {
         ...
        },
      },
  ],
  "jsonapi": {
    "version": "1.0"
  }
}

In my case the lead of ID 9 has been loaded with devour before.
devour 1.4.2 will serve the lead with it's OLD attributes, not with the ones where lead would be included in a subsequent call.

@toadle does the pr solves your problem?https://github.com/twg/devour/pull/83

@IDragonfire Is there any easy way to test your branch in my project?
I tried just replacing the files in node_modules/devour-client but that doesn't seem to work.

@toadle you first need to run npm run compile  and then copy the folder, lib should be enough ;)

@IDragonfire OK, thanks for the advice. Your PR actually solves the cache problem, but introduces another that JSONAPI originally aimed to solve: It causes JSON-object-recursions.
Like this, which is a practical example from my app:
  "data": {
    "id": "8",
    "type": "authentications",
    "attributes": {
       ...
    },
    "relationships": {
      "user": {
        "data": {
          "id": "10",
          "type": "users"
        }
      }
    }
  },
  "included": [
    {
      "id": "10",
      "type": "users",
      "attributes": {
          ...
      },
      "relationships": {
        "current_authentication": {
          "data": {
            "id": "8",
            "type": "authentications"
          }
        }
      }
    }
  ],
  "jsonapi": {
    "version": "1.0"
  }
}

Will result in an endless: authentication.user.current_authentication.user.current_authentication.... object.
I suppose it should stop at the point, where in my example the current_authentication of the user is not included anymore. Therefore should be empty.

@toadle Oh, you are correct, I created an issue for this #84 but it depend on the include parameter ...
If you pull only https://github.com/twg/devour/pull/83/commits/614e62f266cb989a5040fc947b780d8f20fbaa23 does this solve the endless object?

@IDragonfire Do you still need me to test the above commit or was #85 enough?

The commit above is #85 ;)
I decided after my comment above to close the PR and create an new one ^^

Fixed by https://github.com/twg/devour/pull/85


Don't mention author of release in credits

Hi @leo,
I think this would be good, but an alternative could be some sort of excludeFromCredits array that excludes certain usernames from the credits. It could be configured using the config file (#38) or somewhere else. This array could include the author of the release, bots etc by default, but the user could add to it for their repo's needs. Regexes could be included as well.
let excludeFromCredits = [getReleaseAuthor(), getRepoAuthor(), /[bot]/]

// Then you could check the commits that match the list...

It might be best to wait to see if the config file gets implemented, but you could use a CLI flag or package.json. What do you think?

@Facel3ss1 LGTM! But it should work without a flag.

Closing in favor of https://github.com/zeit/release/issues/77!


argparse is missing on Python 2.6

It is required by core.configuration. It cannot be auto-installed because installbundle imports core.configuraion.
Should we give an error, make installbundle work without configuration, or drop support for Python 2.6?

Python 2.7 is now 7 years old.
Reasons for dropping 2.6

Unsupported since 2013-10-29
https://snarky.ca/stop-using-python-2-6/
http://www.curiousefficiency.org/posts/2015/04/stop-supporting-python26.html
http://www.python3statement.org
Current pip 9 deprecates Python 2.6 support, pip 10 won't support it (https://github.com/pypa/pip/issues/3955)
Not much PyPI traffic (June 2016) https://github.com/pypa/pip/issues/3796


Revert "Add sentry"

Reverts priznanie-digital/priznanie-digital#553

Need to add env variable to dokku, I don't know how to do that now.


Testing to see if CI changes can't be made in forks

Ah ok, that settles it then - CI changes can't really be submitted via fork. That makes sense for security purposes. Have to be done via branches in the central repository instead.


[5.5] Don't fail if no file to email

Avoid failing because we can't open the file to email.
This prevents the following error:
[2017-03-07 20:44:56] production.ERROR: ErrorException: file_get_contents(/web/LaravelLogs/check-approved-2017-03-07T13:44:55-07:00.
log): failed to open stream: No such file or directory in /web/vendor/laravel/framework/src/Illuminate/Console/Scheduling/Event.php:408

To replicate the error, delete the zero length file in the $schedule method chain's ->after(function()).

this is to wrong branch. 2) we should just use a simple file_exists. I will fix it on the right branch.

Thanks @taylorotwell


Click at center of given Bitmap

Hello
Currently I don't see a method to find the center of a Bitmap and clicking on it.
Any suggestions on how to do this.
I would suggest this as an feature.

I found a method to do this using the default packages. Don't know if you want to do it that why.
func getImageDimension(imagePath string) (int, int) {
	file, err := os.Open(imagePath)
	defer file.Close()
	if err != nil {
		fmt.Println(err)
	}

	image, _, err := image.DecodeConfig(file)
	if err != nil {
		fmt.Println(imagePath, err)
	}
	return image.Width, image.Height
}

.png image: unknown format, use the following method is better :
func getImageDimension(imagePath string) (int, int) {
	bitmap := robotgo.OpenBitmap(imgPath)
	gbit := robotgo.ToBitmap(bitmap)

	fmt.Println("gbit...", gbit.Width, gbit.Height)

	x := gbit.Width / 2
	y := gbit.Height / 2

	return x, y
}

Oh, Yes I found that issues and fixed my imports...BUT your answer is AWESOME @vcaesar
Thank You very much !!!

Ok, I updated your sample code, so that someone encountered the same problem reference.


Adjusting HTTP response for various hyperdrive statuses

Hi Mauve,
@josephmturner  and I were talking about how to distinguish various cases when requesting information about a hyperdrive, and we came up with this matrix of what would be ideal for the application side of the protocol.  Would it be possible for hyper-gateway and/or hypercore-fetch to provide responses like these?
Response matrix
Currently observed behavior.

Request for
Returns
#

URL to valid hyperdrive without content
Etag == 1

URL to unknown (i.e. network-inaccessible) hyperdrive
Etag == 1
!

URL to valid hyperdrive that has ever had content
Etag > 1

URL to valid hyperdrive directory but invalid file
HTTP 404

URL with too-short public key
HTTP 500

Ideals?

Request for
HTTP
Etag

Obviously malformed URLs
400 Bad Request
N/A

Unknown hyperdrive
404 Not Found
N/A

Known hyperdrive (never had content)
204 No Content
N/A

Known hyperdrive (has or had content)
200 OK
>= 1

To distinguish whether peers are available, ideally we would use another header, something like X-Hyperdrive-Peers.

Glossary

Unknown hyperdrive
An unknown hyperdrive may or may not exist.  We don't know whether it does.  We have never received any information about it.

Known hyperdrive (never had content)
A hyperdrive that we know exists, and we know it is empty, and it has never yet had any content.  Possibly created by us.

What should the expected etag be for unknown hyperdrives with no peers?

Looks like right now invalid drives or unknown drives end up creating a new key which is a bug.

SHould double check if this is still the case

See https://github.com/RangerMauve/hypercore-fetch/issues/60#issuecomment-1448863526
for handling POST requests to non-writable drives with 405 "You can't POST here"

SHould double check if this is still the case

It still is. See #68.

Still TODO:

[ ] Valid z32-encoded public key for which no drive is found should
give 404 (current: 200)
[ ] Known existent drive with no content should return 204 (current: 200)
[ ] Show available peers in X-Hyperdrive-Peers header (move this to another issue?)

What does known existent with no content mean in this context? Is that  when the hyperdrive header is there but there are no files (.e.g length = 0)

Also yes please move the x-hyperdrive-peers to another issue. I think it might be good to place that into another $/peers endpoint or similar.

What does known existent with no content mean in this context? Is that when the hyperdrive header is there but there are no files (.e.g length = 0)

Yes.

Have a new issue for the peers thing, #74
Not gonna bother with a special case for empty valid drives for now.
404 on drives without initial metadata block is in place in latest version


Still Flagging

I saw in the notes that changes were made to make the standalone version more palatable, unfortunately, it's not. That's not the case for me anyway. Still eagerly waiting for the day I can replace faststone with this. As for now, my PC literally won't allow it.

Thank you for the feedback. We're continuously working on improvements.


Per-theme colors

Thank you for your work on this extremely useful plugin. It aids me every day, allowing me not to get lost in the brackets.
It would be nice to have different colors per theme in the vscode settings.json, using the same structure as "workbench.colorCustomizations":
{
  "bracket-pair-colorizer-2.colors": {
    "[Min Dark]": [
      "#BBBBBB",
      "#B392F0",
      "#FF7A84",
      "#1976D2",
      "#FF9800",
    ],
    "[Nord]": [
      "#ECEFF4",
      "#A3BE8C",
      "#8FBCBB",
      "#B48EAD",
    ]
  }
}

Would this be hard to add?

You can already do this, you just have to reference an existing color in your theme. Here's what I have in mine, it tends to work well:
"bracket-pair-colorizer-2.colors": [
	"terminal.ansiCyan",
	"terminal.ansiMagenta",
	"terminal.ansiBlue"
]

The README.md doesn't really mention it, but I was wanting the same thing and then I saw this in src/settings.ts:
for (const color of this.colors) {
    const decoration = vscode.window.createTextEditorDecorationType({
        color: color.includes(".") ? new ThemeColor(color) : color,
        rangeBehavior: vscode.DecorationRangeBehavior.ClosedClosed,
    });
    decorations.set(color, decoration);
}

So the expression color.includes(".") ? new ThemeColor(color) : color looks like it will interpret any string containing a . character as a property name to pull from the active theme.

I also use this extension and find it very useful. Per-theme customisation would be great, though.
My problem with the above solution is that it's only possible to use a colour that already exists in a theme, and that it's not guaranteed that the colour with the same name in a different theme would also be good to use for bracket colouring. Redefining the colours also affects whatever they are used for originally.
A different, and perhaps not very difficult solution would be to add specific themable colour names via the contributes.colors API. I admit that my knowledge in this regard is superficial, so there might be reasons why this isn't viable.

Created a pull request that adds theme colours (#341) that can be configured like other VSCode theme colours, individually for dark/light or specific themes.

@zEdS15B3GCwq this is a fantastic idea for a PR... Hope this gets merged soon. 👍👍👍
Really look forward this level of control over the bracket colors used in my preferred dark and light schemes 🤩

@zEdS15B3GCwq this is a fantastic idea for a PR... Hope this gets merged soon. 👍👍👍
Really look forward this level of control over the bracket colors used in my preferred dark and light schemes 🤩

Ofc, I also hope it'll be merged soon, however, looking at the response rate of the creator, I'd say you're better off compiling your own build of this extension with improvements. It's not that difficult - I was able to do it with very little prior knowledge in less than half an hour.

Funny thing is, that VSCode theme colours have default values for general dark and light (and high contrast) theme kinds, but users can only set colours for specific themes. So, obviously, the PR also has that limitation.
If by any chance you also like the idea of being able to specify VSCode theme colours for dark/light theme kinds in general (as opposed to each theme separately), then head over to https://github.com/microsoft/vscode/issues/101418 and submit a vote.


Hiding columns raises - Maximum update depth exceeded

material-react-table version
2.11.3
react & react-dom versions
18.2.0
Describe the bug and the steps to reproduce it
I'd like to render a big table containing lots of data, virtualised. An error is being returned upon hiding a column:
Error: Maximum update depth exceeded. This can happen when a component repeatedly calls setState inside componentWillUpdate or componentDidUpdate. React limits the number of nested updates to prevent infinite loops.

Minimal, Reproducible Example - (Optional, but Recommended)
Run the below component and hide the only column, you should hit the error.
import React, { useMemo } from "react";
import {
  MaterialReactTable,
  MRT_ColumnDef,
  useMaterialReactTable,
} from "material-react-table";

type Row = {
  a: string;
};

const ipsum =
  "Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum";

const data: Row[] = Array.from(Array(1000)).map(
  (): Row => ({
    a: ipsum,
  }),
);

const Example = () => {
  const columns = useMemo<MRT_ColumnDef<Row>[]>(
    () => [{ accessorKey: "a", header: "A" }],
    [],
  );
  const table = useMaterialReactTable({
    columns,
    data,
    enableRowVirtualization: true,
    enablePagination: false,
  });
  return <MaterialReactTable table={table} />;
};

export default Example;

Screenshots or Videos (Optional)
No response
Do you intend to try to help solve this bug with your own PR?
No, because I do not know how
Terms

[X] I understand that if my bug cannot be reliably reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

Can you turn your code into a sandbox? None of the docs examples have this problem.

@novucs Looks like you are not memoizing the rows but creating it on each render which docs mention to avoid. Can you fix that and see if its an issue?

No, the rows/data are defined outside of the component with a stable reference properly. This is throwing a real error. I just haven't been able to reproduce it in my own codebase outside of this sandbox yet.

@KevinVandy I’ve found its dependant on the number of rows in the table, maybe your machine needs more rows to see this problem. I’m also using Firefox to reproduce this if that helps


pretty state: Dumping state in the insertion order

Since Python 3.6, the dict will preserve the insertion order of keys1,
in stead of using OrderDict in prettystate.py for dedicate options,
use yaml.add_representer(dict, represent_dict) to dump all dict in
the order of insertion.
This could make nmstatectl showing the information in the order of
property insertion without hacks.

There are no test cases for the requested order of OVS bond name and team port name. It would be great to add test cases for states using ideally all defined settings to show the intended order and keep it stable.

There are no test cases for the requested order of OVS bond name and team port name. It would be great to add test cases for states using ideally all defined settings to show the intended order and keep it stable.

The output of nmstatectl is not a API. And even so, the order of hashmap/dict is not a API.
This is decorate issue and I don't have quick way to test whether bond name come first than other properties.

There are no test cases for the requested order of OVS bond name and team port name. It would be great to add test cases for states using ideally all defined settings to show the intended order and keep it stable.

The output of nmstatectl is not a API. And even so, the order of hashmap/dict is not a API.
This is a decorate issue and I don't have quick way to test whether bond name come first than other properties.

There are no test cases for the requested order of OVS bond name and team port name. It would be great to add test cases for states using ideally all defined settings to show the intended order and keep it stable.

The output of nmstatectl is not a API. And even so, the order of hashmap/dict is not a API.

Yes, it is not an API, therefore the order can be changed but this should happen deliberately. For example to show the name for team interfaces first. If there is a good reason to show the mac address before the lldp information, it makes sense to change this. Otherwise, it does not. The original issue is about making nmstatectl output more user-friendly, therefore this should also be considered here in my opinion.

This is a decorate issue and I don't have quick way to test whether bond name come first than other properties.

Here is an example how to write an test based on #840 for team interfaces:
from libnmstate.prettystate import PrettyState

TEAM_TEST_STATE = {
    "interfaces": [
        {
            "name": "team0",
            "state": "up",
            "team": {
                "ports": [
                    {"link_watch": {"name": "ethtool"}, "name": "vethteam0"},
                    {
                        "link_watch": {"name": "ethtool"},
                        "name": "vethteam0_ep",
                    },
                ],
                "runner": {"name": "roundrobin"},
            },
            "type": "team",
        }
    ]
}

EXPECTED_YAML = """interfaces:
- name: team0
  type: team
  state: up
  team:
    ports:
    - name: vethteam0
      link_watch:
        name: ethtool
    - name: vethteam0_ep
      link_watch:
        name: ethtool
    runner:
      name: roundrobin
"""

def test_prettystate_team():
    pretty_state = PrettyState(TEAM_TEST_STATE)
    assert pretty_state.yaml == EXPECTED_YAML

def test_prettystate_team():
pretty_state = PrettyState(TEAM_TEST_STATE)
assert pretty_state.yaml == EXPECTED_YAML

Ah, given that PrettyState will not ensure the correct order anymore, I guess the tests needs to be moved to the actual integration test for team and and then assert the output from PrettyState(show_only("team0")) or similar.

The link_watch is not supported any more in team and will never show.
Uploaded new patch with new unit test cases for ovs bond name.

The link_watch is not supported any more in team and will never show.
Uploaded new patch with new unit test cases for ovs bond name.

The proper order of bond name also needs to be tested in the integration tests to ensure that they are inserted in the correct order, it needs something like this, I guess:
diff --git a/tests/integration/ovs_test.py b/tests/integration/ovs_test.py
index a00c364..9525af8 100644
--- a/tests/integration/ovs_test.py
+++ b/tests/integration/ovs_test.py
@@ -22,6 +22,7 @@ from subprocess import CalledProcessError
 import pytest

 import libnmstate
+from libnmstate.prettystate import PrettyState
 from libnmstate.schema import Interface
 from libnmstate.schema import InterfaceIP
 from libnmstate.schema import InterfaceIPv4
@@ -73,6 +74,9 @@ def bridge_with_ports(port0_up):
 def test_create_and_remove_ovs_bridge_with_min_desired_state():
     with Bridge(BRIDGE1).create() as state:
         assertlib.assert_state_match(state)
+        current_state = statelib.show_only((port0_name,))
+        pretty_state = PrettyState(current_state)
+        assert pretty_state.yml == MINIMAL_OVS_BOND_YAML_STATE

     assertlib.assert_absent(BRIDGE1)

Integration test case added.

@tyll @ffmancera I have completely rewrite this PR. Please review and check the first comment for detail designs.


Add a test that measures memory consumption of malformed proposal decoding

Motivation

Closes #3014

The added test shows that https://github.com/nullstyle/go-xdr/xdr3 consumes 3 GB when trying to decode a malformed proposal with 3 KB length:
=== RUN   Test_decodeMalformedProposal
    proposal_test.go:101: Total cumulative alloc: 267K
    proposal_test.go:101: Total cumulative alloc: 3G
--- PASS: Test_decodeMalformedProposal (0.05s)
PASS

Changes

Add a test that measures memory consumption of malformed proposal decoding

DevOps Notes

[x] This PR does not require configuration changes (e.g., environment variables, GitHub secrets, VM resources)
[x] This PR does not affect public APIs
[x] This PR does not rely on a new version of external services (PoET, elasticsearch, etc.)
[x] This PR does not make changes to log messages (which monitoring infrastructure may rely on)

@countvonzero
Not any malformed data causes such consumption. There're malformed proposals that consume just a few KBs of memory:
=== RUN   Test_decodeMalformedProposal1
    proposal_test.go:128: Total cumulative alloc: 265.5K
    proposal_test.go:128: Total cumulative alloc: 272.9K
--- PASS: Test_decodeMalformedProposal1 (0.00s)
PASS

But also there're ones that consume even more than in the description:
=== RUN   Test_decodeMalformedProposal2
    proposal_test.go:128: Total cumulative alloc: 268.5K
    proposal_test.go:128: Total cumulative alloc: 28.1G
--- PASS: Test_decodeMalformedProposal2 (0.64s)
PASS

I think it's specific to a data pattern and may be reproduced with other data structures.

@dshulyak I agree, we don't need this in the repo. It's more like a PoC that I wanted to show


Favicon in every page

Describe the solution you'd like
Adding favicon in every page

Can I work on this issue? I'm a participant of LGM Soc21.

Use this logo -> https://github.com/QAZIMAAZARSHAD/Voters-Registration-Portal/blob/master/Images/flag.png.

can u plz assigm me this issue?i can add the logo in title section

can u plz assign me this issue?i can add the logo in title section

@Jagannath8 if you didn't solve this issue till tomorrow then I will assign @tanzzz-tech.

Can you assign me this issue?

Can you assign me this issue?

Many people are ahead in line.
Look for some other issue.

This issue is already solved by someone.

This issue is already solved by someone.

https://qazimaazarshad.github.io/Voters-Registration-Portal/
Check again, not a single page has a favicon.

Use this logo -> https://github.com/QAZIMAAZARSHAD/Voters-Registration-Portal/blob/master/Images/flag.png


Updated dependencies

Switched the react-addons-css-transition-group package (which is being deprecated) to react-transition-group/CSSTransitionGroup, and updated minimum React version to 15.5.0.

Thanks!


Figure G3 and G4 are the same

Looks like a simple duplication, should remove one

My favourite kind of issue. [I even bolded the difference in the caption ;) ]


Fixed #45284

o JenkinsTriggerHelper can wait for too long if job allows
concurrent builds.

Looks like something is wrong with travis.

Please make a rebase against master and re push the changes...The travis issue should be fixed for now.

@khmarbaise Rebased and travis passes!


Mind2Web webshop data is not found in this repo?

如题

I'm also curious about this. How can I replicate the Mind2Web results?

We recommend pulling docker images for these tasks (except for OS, DB, KG for which we do not provide docker images)

You can refer to https://github.com/THUDM/AgentBench/blob/main/docs/tutorial.md#step-2-prepare-docker-environment


vkGetPhysicalDeviceSurfaceCapabilitiesKHR gives VkErrorInitializationFailed for xcb surface

Issue:
I obtained the following error in my code for an xcb surface during vulkan command 'vkGetPhysicalDeviceSurfaceCapabilitiesKHR'.
File "~/.local/lib/python3.5/site-packages/vulkan/_vulkan.py", line 5207, in vkGetPhysicalDeviceSurfaceCapabilitiesKHR
    raise exception_codes[result]
vulkan._vulkan.VkErrorInitializationFailed

To investigate the issue, I used your example_sdl2.py script and substituted all definitions pertaining to xlib with xcb. And I got the same error msg.
Changes made to example_sdl2.py:

replaced line 60 with extensions.append('VK_KHR_xcb_surface')
replaced lines 101 to 109 with

def surface_xcb():
    print("Create Xcb surface")
    vkCreateXcbSurfaceKHR = vkGetInstanceProcAddr(instance, "vkCreateXcbSurfaceKHR")
    surface_create = VkXcbSurfaceCreateInfoKHR(
            sType = VK_STRUCTURE_TYPE_XCB_SURFACE_CREATE_INFO_KHR,
            connection = wm_info.info.x11.display,
            window = wm_info.info.x11.window,
            flags = 0)
    return vkCreateXcbSurfaceKHR(instance, surface_create, None)

replaced line 140 with sdl2.SDL_SYSWM_X11: surface_xcb

The xlib version of example_sdl2.py works well on my system.
Question:
Is VkErrorInitializationFailed due to an issue in vulkan or Vulkan?

Hello @sunbearc22,
Sorry for my late reply.
I think the problem comes from your Vulkan driver.
To be sure, you can enable the debug layer, maybe you will find informations.

Hello @sunbearc22,
I close this issue, you can reopen if you have more elements.


Simplified contribution guidelines

Simply moved CONTRIBUTING.md into .github for the sake of simplification and organization

A link to CONTRIBUTING.md is presented at the top of every pull request being created. Like README.md, it's a magical part of GitHub.
See: https://github.com/blog/1184-contributing-guidelines

@xwu I'm more familiar with the Issue and Pull Request template feature that was more recently announced; in spirit of that feature and for the sake of organization, I updated this PR with alternative approach.

It's easier to find CONTRIBUTING.md when it's at the root of the repo, don't you think?

The purpose of its existence is to make its content accessible to users; if we've already established that a GitHub feature makes its content easily accessible whenever someone opens a PR or Issue and we've also established that this file can be placed in either the root of the repo or in the .github directory then why not place it in the .github directory with the other file(s) (i.e. PULL_REQUEST_TEMPLATE.md) which take advantage of the aforementioned GitHub features? PULL_REQUEST_TEMPLATE.md could also be placed in the root of the repository and provide the same functionality it currently provides but neither that file or CONTRIBUTING.md are specifically mentioned or formally referenced in any documentation therefore making their purpose for existence a means to display their contents to users through the aforementioned GitHub features.

The custom of placing CONTRIBUTING.md at the root of the repo pre-dates the GitHub feature, and there are those who would look for it in that location. For instance, a person who has checked out a local working copy may wish to consult that document while browsing through the folder on their file system, independent of the GitHub user interface, just as it may be the case for README.md. By contrast, the PR template is not meant to be useful independent of GitHub, which is why it is in a hidden folder.

Your logic is based on functionality that is independent of GitHub but the contents of CONTRIBUTING.md specifically reference and instruct users on how to make contributions using GitHub; furthermore, the contents of the file are specific to how one should go about opening a pull request which is a feature of GitHub.
@practicalswift care to weigh in on this?

I'm with @xwu on this. It's also a bit of a process and presentation thing, so @tkremenek and @DougGregor might be the best people to make the final call.

Not all open-source projects on GitHub accept changes via GitHub. (Some, for instance, are mirrors.) The contents of CONTRIBUTING.md answer certain questions for a prospective contributor (in the case of Swift, via a link to a more extensive page of guidelines):

What sorts of contributions are welcomed?
How can it be contributed?
What are the legal conditions surrounding contributed code?

This is important information to have before someone begins tackling a bug and does not necessarily involve GitHub--it just so happens that Swift accepts contributions via GitHub. By contrast, the PR template addresses other questions. Namely, now that you've already made some changes you would like to contribute and are about to open a PR:

What is the content of those changes?
What bugs, if any, have you resolved?
How do you go about testing and merging those changes upstream?

But of course, it's up to core team members to make the call. Just putting in my two cents.

I respect the motivation behind this change, but I agree with @xwu's arguments about keeping this at the root.  I think it accomplishes two things (which @xwu outlined):

It makes the contribution guidelines easily discoverable on the local file system, and elevates their importance.
There is a separation of concerns (IMO) between contribution guidelines to the project and the hosting infrastructure, in this case GitHub.  While the guidelines may naturally reference that infrastructure, the guidelines can encompass other things like the points @xwu outlined, such as legal considerations, etc.

The second point is a bit subtle—we would have contribution guidelines anyway even if the project was not hosted on GitHub, and we would naturally put them at the top as a peer of README.


Replace ResizableTriggerIcon with LucideIcon

Description
Replace ResizableTriggerIcon with LucideIcon
Problem Statement/Justification
Replace the icons from @yamada-ui/icon with those from @yamada-ui/lucide.
Proposed Solution or API
No response
Alternatives
No response
Are you willing to participate in implementing this feature and create a pull request include the implementation?
None
Additional Information
No response

can i work on this?


fix: retrieve organization information with the correct id

Issue
After introducing multi-tenancy, the publication of trial's API v4 is fetching the user with the organization id known by Cockpit.
Description
The organization's information needs to be retrieved from the orgService before fetching the user information

🏗 Your changes can be tested here and will be available soon:
Console: https://pr.team-apim.gravitee.dev/6348/console
Portal: https://pr.team-apim.gravitee.dev/6348/portal
Management-api: https://pr.team-apim.gravitee.dev/6348/api/management
Gateway v4: https://pr.team-apim.gravitee.dev/6348
Gateway v3: https://pr.gateway-v3.team-apim.gravitee.dev/6348

Fixed on Cockpit side, without impact on APIM
https://github.com/gravitee-io/gravitee-cockpit/pull/4124


ListView.builder causing spacing in a SliverList

I am having issue's with a ListView.builder inside a SliverList.
I have a few pages with ListView.builder inside and the add spacing above the Listview's
Code:
CupertinoPageScaffold(
            child: CustomScrollView(
              slivers: <Widget>[
                buildAppBar(context),
                SliverList(
                  delegate: SliverChildListDelegate(
                    [
                      new Column(
                        mainAxisAlignment: MainAxisAlignment.center,
                        children: <Widget>[
                          ListView.builder(
                              itemCount: selectedDayTimes == null
                                  ? 0
                                  : selectedDayTimes.length,
                              physics: NeverScrollableScrollPhysics(),
                              shrinkWrap: true,
                              itemBuilder: (BuildContext context, int index) {
                                return Padding(
                                    padding: new EdgeInsets.only(
                                      top: 10,
                                      bottom: 5,
                                    ),
                                    child: _buildListTitle(
                                        context,
                                        selectedDayTimes[index].period.toInt(),
                                        selectedDayTimes[index].key.toString(),
                                        selectedDayTimes[index]
                                            .start
                                            .toString(),
                                        selectedDayTimes[index]
                                            .end
                                            .toString()));
                              }),
                        ],
                      ),
                    ],
                  ),
                ),
              ],
            ),
          ),

There should only be 15px of spacing between these but there is more like 40 or something.

This is the Flutter Doctor
[✓] Flutter (Channel beta, v1.12.13+hotfix.6, on Mac OS X 10.15.2 19C57, locale en-US)
    • Flutter version 1.12.13+hotfix.6 at /Users/elias/flutter
    • Framework revision 18cd7a3601 (9 days ago), 2019-12-11 06:35:39 -0800
    • Engine revision 2994f7e1e6
    • Dart version 2.7.0

[!] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/elias/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses

[✓] Xcode - develop for iOS and macOS (Xcode 11.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.3, Build version 11C29
    • CocoaPods version 1.7.5

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 41.1.2
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[✓] VS Code (version 1.41.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.7.1

[✓] Connected device (3 available)
    • iPhone 11 Pro Max • C5EDAE24-E60A-4373-A52E-84ED09B845F9 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-13-3 (simulator)
    • Chrome            • chrome                               • web-javascript • Google Chrome 79.0.3945.88
    • Web Server        • web-server                           • web-javascript • Flutter Tools

! Doctor found issues in 1 category.

Hi @EliasDeuss
This platform is not meant for assistance on personal code.
Please see https://flutter.dev/community for resources for asking questions like this,
you may also get some help if you post it on Stack Overflow.
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you

@iapicca I disagree there is no reason that a ListView.builder should create a large space in a SilverList.

had to do this to fix the issue. @iapicca
MediaQuery.removePadding(
        removeTop: true,
        context: context,
        child: ListView.builder(
            itemCount: selectedDayTimes == null ? 0 : selectedDayTimes.length,
            physics: NeverScrollableScrollPhysics(),
            shrinkWrap: true,
            itemBuilder: (BuildContext context, int index) {
              return _buildListTitle(
                  context,
                  selectedDayTimes[index].period.toInt(),
                  selectedDayTimes[index].key.toString(),
                  selectedDayTimes[index].start.toString(),
                  selectedDayTimes[index].end.toString());
            }))

In order to debug this please post a fully runnable code sample that we can copy, paste, and run to see the issue. Your current code samples are incomplete, unfortunately.


[SKY30-234] Add toggle to Location selector to choose between regions & countries/highseas

Overview
In this PR:

Added Location Type Toggle component for use in the Location Selector
Used LocationTypeToggle to filter locations in the LocationSelector
Moved LocationDropdown into its own component

Screenshot

Feature relevant tickets
SKY30-234

Converting to draft because there were design changes, which are currently pending review.


Ability to control and link between multiple viewing windows on same timeline

Description
I’ve produced an exhaustive restoration of a old film and want to showcase the restoration, both the main film and sources/segments/commentary as a single continuous viewing experience. A very controlled experience can be done on a single canvas (which has one timeline) but additional facilities would be required to facilitate this with multiple viewing windows.
Variation(s)
Also  interested in exhibiting all the extra / surplus / repeat / unclean / cleaned / previously-digitised / re-coloured footage in a more interactive or curated viewing experience, alongside the restoration. In this case: content will overlap, repeat, over-run the timeline of the main film.
I have multiple recordings of multiple scores which I want to provide to the user.
Additional Background
BFI use case explored at BL 2017-02 workshop

From discussion at BL workshop:

Synchronous playback that can extend before and after the "main" artefact. Might have top level canvas 8 hours, restoration canvas 6 hours, annotated starting 1 hour in to top.
Other content is annotated into first and last hour of top, with some overlap.
Potential for independent clocks in different manifests/canvases, with links to start them either automatically or manually

General requirement to link from part of timeline in A to part of timeline in B:

related ranges, with description/label/metadata etc.  alternate perhaps?
viewer can then present UI to switch between current alternates, or 2-up them
possible transitivity confusions, may need more thinking

Might replace related with a more complex list of related items to be shown to the user when the user views the described resource, e.g.:
{
  "type": "Range",
  "links": [ {
		"type": "Display?" // as:Link,
		"label": "Alternate Camera Angle (View 6)",
		"href":  {"id": "http://example.org/.../range/7", "type": "Range"}
		// why Range and not Canvas with time fragment?
    }, // more links here ...
  ]
}

Consensus at meeting is that the idea of having multiple viewing windows tied to a single timeline with links and controls between them is a significant step up in complexity from our current use cases. Will defer this for now.


Numeral with zero decimal places not working

On a numeral, setting numeralDecimalScale: 0 is not working and defaults to 2 decimal places.

Oh, oops, we will have a look, thanks.

Basically on the param setting it has:
owner.numeralDecimalScale = numeralDecimalScale || 2;
Problem with this is that 0 || 2 is 2 :cry:
Not sure how to solve, though.

Published. Thanks


[Spark-27584][MLlib] Add 'Mean reciprocal rank' to RankingMetrics

What changes were proposed in this pull request?
Add 'Mean reciprocal rank' to RankingMetrics, which is usually used for ranking evaluation.
How was this patch tested?
Please review http://spark.apache.org/contributing.html before opening a pull request.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

I don't think this is important enough to add... .mllib is a legacy API anyway.

+1 mllib is no longer under active development (bug fixes only).

What about adding mean recipricoal rank to the new ML apis?

@holdenk yeah fair point. Right now the .ml implementation doesn't already support meanAveragePrecision or anything. I think that could be a fine separate PR.


[chore] Create/generate types for parsed binary JSON

This is different from a proto (which is a nicely grouped json format). It relates to the binary format where many nmbers are not grouped
e.g. team colors are out of order:
const team = {
  // ...
  color1red: 1,
  color1blue: 2
  color2: {
    red: 1,
    green: 2,
    blue: 3
  }
  color1green: 3,
}

Ideally these can be generated from a jBinary typeset

Done


cloud.redhat.com hostname mentioned inrepository insights-results-smart-proxy in file docs/rest_api.md

cloud.redhat.com hostname mentioned inrepository insights-results-smart-proxy in file docs/rest_api.md

REST API is not moved -> closing this on.


All cmd does not work in Diffview

Hi,
When working in Diffview I would like to be able to resize / enable auto resizing of the differents windows (diff files) except the Diffview panel.
Actually when I open Diffview, all the cmd (like FocusEnable FocusAutoresize or FocusMaximise) are not working.
How can I proceed so the Diffview panel is not resized but the diff files are ?
Actual config :
{
	"nvim-focus/focus.nvim",
	event = "VeryLazy",
	opts = {
		enable = true,
		commands = true,
		split = {
			bufnew = false,
			tmux = false,
		},
		autoresize = {
			enable = true,
			width = 0,
			height = 0,
			minwidth = 0,
			minheight = 0,
			height_quickfix = 10,
		},
		ui = {
			number = false,
			relativenumber = false,
			hybridnumber = false,
			absolutenumber_unfocussed = false,

			cursorline = false,
			cursorcolumn = false,
			colorcolumn = {
				enable = false,
			},
			signcolumn = false,
			winhighlight = false,
		},
	},
}

Autocmds related :
				{
					"WinEnter",
					function()
						local ignore_buftypes = { "nofile", "prompt", "popup" }
						if vim.tbl_contains(ignore_buftypes, vim.bo.buftype) then
							vim.b.focus_disable = true
						end
					end,
					description = "Disable focus autoresize for BufType",
				},
				{
					"FileType",
					function()
						local ignore_filetypes = {
							["neo-tree"] = true,
							["dap-repl"] = true,
							SidebarNvim = true,
							Trouble = true,
							terminal = true,
							dapui_console = true,
							dapui_watches = true,
							dapui_stacks = true,
							dapui_breakpoints = true,
							dapui_scopes = true,
							OverseerList = true,
							noice = true,
							DiffviewFiles = true,
						}
						vim.b.focus_disable = ignore_filetypes[vim.bo.filetype]
					end,
					description = "Disable focus autoresize for FileType",
				},

You recognize that you disabled focus for DiffviewFiles?

You recognize that you disabled focus for DiffviewFiles?

@cryptomilk
Of course, in Diffview, the left panel has a filetype of Diffviewfiles but other splits are not of that filetype.
So, the commands should work for the other splits right ?
For example, with Overseer (which I disabled too), the pane of Overseer is not changed by focus, but if I have splits opened the splits are being changed by focus

@cryptomilk
Please see the behavior with Overseer and then with Diffview
https://github.com/nvim-focus/focus.nvim/assets/58638832/cd025723-ef32-44bf-9ad0-6277d8549e76
So, can I achieve the same behavior focus has with Overseer with Diffview
And can the probleme be related to Diffview opening a new tab while Overseer don't ?

a) You can disable it for windows, buffers or globally. See :help windows.
b) I don't know what you mean, there a config for reproducing things here: https://github.com/nvim-focus/focus.nvim/tree/master/extras Use it and explain how to t reproduce the behavior with it.


Migrate to dep

Ran dep init and then dep ensure. I had to remove elastic/beats from Gopkg.toml and re-run dep ensure to fix the strage error target/filebeat-master-go1.10- flag redefined: strict.perms in travis builds.
Uses the latest stable release of dep:
$ dep version
dep:
 version     : v0.4.1
 build date  : 2018-01-27
 git hash    : 37d9ea0
 go version  : go1.9.3
 go compiler : gc
 platform    : darwin/amd64

Resolves #9

Codecov Report

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

@@           Coverage Diff           @@
##           master      #16   +/-   ##
=======================================
  Coverage   24.74%   24.74%
=======================================
  Files           3        3
  Lines          97       97
=======================================
  Hits           24       24
  Misses         68       68
  Partials        5        5

Continue to review full report at Codecov.

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

Finally the travis build is passing. I have already verified it to work on my dockerized filebeat+awsbeat. Will submit the Dockerfile later.


vagrant: Get vagrant up and running with lxc 3.0.0

Vagrant doesn't start when lxc>=3.0.0.  Release 2.1.0 of lxc had deprecated some configuration keys (which also caused #7838) and they are now unsupported in lxc 3.0.0.
Testing Plan:
Got vagrant up working again with the newer version of lxc after it was crashing mysteriously.
GIFs or Screenshots:

Cool, merged, thanks for debugging this @punchagan!


apache2.service not found

The line to set the timezone in the Vagrantfile will fail upon with "apache2.service not found" when running "vagrant up" the first time, since it requires apache2, which gets installed afterwards via the provision.sh script.
so the timezone set should be run in the provision script or vagrantfile after installing apache2

Thanks for the report mate.

Yeah, that didn't fix it.. moving to provision script like you suggested, turns out Vagrant has script argument support, so I'll try that.


Update overview.md

This PR is created to improve the documentation for the "Azure Compute Fleet overview" by making grammatical corrections and fixing typos for easier reading. Documentation Improvements:

The grammatical errors and typos were corrected in the articles/azure-compute-fleet/overview.md file to improve readability.

@rrajeesh
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#please-close

Thanks for explanations, and update @ju-shim !


Bug: MP Qualification LEE

Issue:
When a user tries and edits MP Qualification LEE they get an "Internal server error"

User can edit MP Qualifications LEE


forever under windows not supported?

Hello,
I have just tried to run forever on a windows baased server, however when i try to start my app i get:
D:\node\swig>forever app.js
warn:    --minUptime not set. Defaulting to: 1000ms
warn:    --spinSleepTime not set. Your script will exit if it does not stay up f
or at least 1000ms

events.js:72
        throw er; // Unhandled 'error' event
              ^
Error: spawn ENOENT
    at errnoException (child_process.js:975:11)
    at Process.ChildProcess._handle.onexit (child_process.js:766:34)```

Any idea how to fix?

I installed it using
npm i -g forever
How did you get to work it as a windows native software?
When I do
C:\myapp>forever app.js
'forever' is not recognized as an internal or external command,
operable program or batch file.


@Builder should get a @Default companion annotation.

I want final fields to receive a default value, if they not have been set at the builder. Let's say I'd like to have a nonnull enum field. Example:
@Data
@Builder
@FieldDefaults(makeFinal = true, level = AccessLevel.PRIVATE)
public class Configuration {
    @Default(Automation.NONE)
    Automation automation;

    public enum Automation {
        INTERNAL,
        EXTERNAL,
        NONE
    }
}

assert Configuration.builder().build().getAutomation() == Configuration.Automation.NONE;

Your case requires something like
public @interface Default {
    Configuration.Automation value();
}

There's no way to specify an annotation taking an arbitrary object (and there are primitives, too). Lombok might be able to hack its way around it, but it's surely not easy and not nice. See this workaround for a simple solution.

I see your point. That's a nice workaround. But it does not work for immutable classes.

You're right. And immutable classes are the main target for Builder. This might work as lombok should complete the class with missing stuff:
@Value
@Builder(builderClassName="Builder")
@FieldDefaults(makeFinal = true, level = AccessLevel.PRIVATE)
public class Configuration {
    public static class Builder {
        Automation automation = Automation.NONE;
    }

    Automation automation;

    public enum Automation {
        INTERNAL,
        EXTERNAL,
        NONE
    }
}

In case it doesn't, @FieldDefaults might be the culprit (do they influence the nested class?).

Ah, nice, but it's easy to misspell the field name in the builder, or forget to rename it when you rename the parent field. Nothing will warn you.
This is my current minimal, working, compiler checked workaround (not using @Builder):
@RequiredArgsConstructor
@Data
@FieldDefaults(makeFinal = true, level = AccessLevel.PRIVATE)
class Configuration {
    Automation automation;

     public enum Automation {
        INTERNAL,
        EXTERNAL,
        NONE
    }

    static Builder builder() {
        return new Builder();
    }

    @Accessors(fluent = true)
    @Setter
    @FieldDefaults(level = AccessLevel.PRIVATE)
    public static class Builder {
        Configuration.Automation automation = Configuration.Automation.NONE;

        GlobalConfiguration build() {
            return new Configuration(automation);
        }
    }
}

Lombok does byte code rewriting, doesn't it? So how about you remove final field initialization, if the field is annotated @DefaultVale?
Let's pick up the example from previously:
@Builder
class Configuration {
    private final int CONSTANT = 3;
    @DefaultValue
    private final Automation automation = Automation.NONE;
}

The decompiled bytecode would look similar to:
class Configuration {
    private final int CONSTANT = 3;
    private final Automation automation;

    static class ConfigurationBuilder {
        private final Automation automation = Configuration.Automation.NONE;

        // setters, and build method
    }

    // constructor
}

Will this work?

Ah, nice, but it's easy to misspell the field name in the builder, or forget to rename it when you rename the parent field. Nothing will warn you.

It should become an unused private field. But I agree that it's neither fool-proof nor nice.

Lombok does byte code rewriting, doesn't it?

No, it does AST manipulations.

So how about you remove final field initialization, if the field is annotated @DefaultVale?

Not me (I'm not a project owner and I don't have enough spare time to do this). But yes, moving the initialization to the builder is the right thing.

Will this work?

Apart from that the field shouldn't be final in the builder, it sounds good.
I'm not sure, if the annotation is needed. You obviously want to keep CONSTANT constant and move the annotated initialization to the builder. But a constant should actually be static final which excludes it from the builder already. There are uses for a non-static final field with an initializer expression, but there are too rare to care about. IMHO what your annotation does should be the default behavior. For the current behavior, I'd suggest @Builder.Exclude.

I agree. @Builder.Exclude will be the better approach.

Moving the initialization to the builder has all sorts of hairy issues because you're messing with scope. What if the initialization refers to another field, for example?
I think the best way forward is to either silently take the initializer, or possibly only if there's some sort of @Default annotation, and move it to a sibling constant (a static final field), and then have the builder refer to that.

Okay, here's the plan:
we turn this:
@Builder @Value
public class Whatever {
    long created = System.currentTimeMillis();
    String name = "anonymous";
}

into:
public class Whatever {
    static final long $default$created() {
        return /*moved expr */System.currentTimeMillis();
    }

    static final String $default$name() {
        return /* moved expr */ "anonymous";
    }

    private final long created = $default$created();
    private final String name = $default$name();

    public static class WhateverBuilder {
        private long created;
        private boolean created$set;
        private String name;
        private boolean name$set;

        public created(long c) {
            created = c;
            created$set = true;
        }
        // all the other builder stuff

        public Whatever build() {
            return new Whatever(created$set ? created : $default$created(), name$set ? name : $default$name());
    }
    //builder() method here.

}
By using a static method we do NOT mess with invocation order/time based stuff such as initializing a 'created' value.
Open issues:

What about intentional constant stuff? For example, in the above snippet, what if there shouldn't even BE a created(long) method in the builder, it should remain a final field whose expression is initialized by its initializer expression during construction and that's that, it's not otherwise settable?

We could make an annotation for that. Something like @Builder.Constant.


Add 1.16.6, 1.15.14 and 1.17beta1

Go 1.16.6, 1.15.14 and 1.17beta1 has been released. Would you please support them? (Ref. https://github.com/syndbg/goenv/issues/182)

Announcement for 1.16.6 and 1.15.14

Announcement for 1.17beta1

Thanks!


Please implement scraping via HTTPS (SSL/TLS)

For a small distributed application that I intend to monitor, we’re using HTTPS (with real certificates, so certificate verification is not an issue) and basic authentication instead of a separate private network, mostly because the setup should stay simple and the application runs on multiple different servers, distributed over the internet.
Therefore, could you please add a configuration option to the JobConfig proto message? I’m using SRV record resolving, so I’d like to see e.g. a schema key which defaults to http, but can be set to https.

Makes sense. Thanks for the input.

@stapelberg I guess then you'll also need to be able to configure the username/password for the basic auth, so that the final URL will be something like <scheme>://<username>:<password>@<SRV host:port>/<metrics path>?

Indeed, yes. For now, I’ve just set up a couple of static targets, but being able to specify userinfo and scheme would be great :).

Cool, already started coding it. There are just some remaining code hygiene issues I need to figure out before doing a PR.

+1
(A document about best security practices should be also amazing)

From my POV, this issue is done. Prometheus 0.14.0 allows me to use the following config:
- job_name: robustirc
  scheme: https
  basic_auth:
    username: 'robustirc'
    password: 'secret'
  dns_sd_configs:
  - names:
    - '_robustirc._tcp.robustirc.net'
  relabel_configs:
  - source_labels: ['__address__']
    regex:         '(.+)\.robustirc\.net:[0-9]+'
    target_label:  'instance'
    replacement:   '$1'

Indeed, thanks for pointing that out! Closing this issue.


Angular JS is not found on his own site

HTTP 404 for angular js files and libs.

See 12877

They're looking into it :) #12877

Tracked in #12877


exception when operating PyWren

all tests fail on current master branch, for example:
python -m pywren_ibm_cloud.tests -t test_call_async

produces:
Uploading test files...
Testing call_async()...
PyWren v1.2.1-SNAPSHOT init for IBM Cloud Functions - Namespace: pywren_dev_us_east - Region: us_east
ExecutorID 2360-4ee0 | JobID A000 - Selected Runtime: ibmfunctions/action-python-v3.6 - 256MB (Installing...)
ExecutorID 2360-4ee0 | JobID A000 - Uploading function and data - Total: 656.0B
ExecutorID 2360-4ee0 | JobID A000 - Starting function invocation: hello_world()  - Total: 1 activations
E
======================================================================
ERROR: test_call_async (__main__.TestPywren)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/omerbelhasin/IdeaProjects/pywren-ibm-cloud/pywren_ibm_cloud/tests.py", line 209, in test_call_async
    pw.call_async(hello_world, "")
  File "/Users/omerbelhasin/IdeaProjects/pywren-ibm-cloud/pywren_ibm_cloud/executor.py", line 143, in call_async
    future = self.invoker.run(job)
  File "/Users/omerbelhasin/IdeaProjects/pywren-ibm-cloud/pywren_ibm_cloud/invoker.py", line 196, in run
    if self.failed_calls_queue.qsize() > 0:
  File "/Users/omerbelhasin/.pyenv/versions/3.6.7/lib/python3.6/multiprocessing/queues.py", line 117, in qsize
    return self._maxsize - self._sem._semlock._get_value()
NotImplementedError

----------------------------------------------------------------------
Ran 1 test in 5.701s

FAILED (errors=1)
Deleting test files...

@JosepSampe this is something in your latest pull request. I have the same issue

Yep, fixing it..

Fixed in #229
I temporally deactivated test_internal_executions

I already fixed test_internal_executions test, i.e. pywren inside pywren. Will include it in the next PR

@JosepSampe I submited a PR with more tests that have not passed yet and I also reverted test_internal_executions, lets keep all tests active even if they currently aren't passing

@JosepSampe all tests work except test_multiple_executions
when I do:
python -m pywren_ibm_cloud.tests -t test_multiple_executions

I get:
Uploading test files...
Testing multiple executions...
PyWren v1.2.1-SNAPSHOT init for IBM Cloud Functions - Namespace: pywren_dev_us_east - Region: us_east
ExecutorID c2a0ec/0 | JobID M000 - Selected Runtime: ibmfunctions/action-python-v3.6 - 256MB
ExecutorID c2a0ec/0 | JobID M000 - Uploading function and data - Total: 715.0B
ExecutorID c2a0ec/0 | JobID M000 - Starting function invocation: simple_map_function()  - Total: 2 activations
ExecutorID c2a0ec/0 | JobID M001 - Selected Runtime: ibmfunctions/action-python-v3.6 - 256MB
ExecutorID c2a0ec/0 | JobID M001 - Uploading function and data - Total: 715.0B
ExecutorID c2a0ec/0 | JobID M001 - Starting function invocation: simple_map_function()  - Total: 2 activations
ExecutorID c2a0ec/0 - Getting results...

  100%|███████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 4/4

ExecutorID c2a0ec/0 - Cleaning temporary data
ExecutorID c2a0ec/0 | JobID M002 - Selected Runtime: ibmfunctions/action-python-v3.6 - 256MB
ExecutorID c2a0ec/0 | JobID M002 - Uploading function and data - Total: 715.0B
ExecutorID c2a0ec/0 | JobID M002 - Starting function invocation: simple_map_function()  - Total: 2 activations
ExecutorID c2a0ec/0 - Getting results...

    0%|                                                                                | 0/2

ExecutorID c2a0ec/0 | JobID M002 - There was an exception - Activation ID: fe3fb5a3cf8e449cbfb5a3cf8ef49cd5

Traceback (most recent call last):
  File "/action/pywren_ibm_cloud/runtime/function_handler/jobrunner.py", line 237, in run
  File "/action/pywren_ibm_cloud/runtime/function_handler/jobrunner.py", line 89, in _get_function_and_modules
  File "/action/pywren_ibm_cloud/storage/storage.py", line 101, in get_func
  File "/action/pywren_ibm_cloud/storage/backends/ibm_cos/ibm_cos.py", line 154, in get_object
Exception: No such key /omeruseast/pywren.jobs/c2a0ec/0/M002/func.pickle found in storage.

ExecutorID c2a0ec/0 - Cleaning temporary data
E
======================================================================
ERROR: test_multiple_executions (__main__.TestPywren)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/omerbelhasin/IdeaProjects/pywren-ibm-cloud/pywren_ibm_cloud/executor.py", line 367, in wait
    THREADPOOL_SIZE=THREADPOOL_SIZE, WAIT_DUR_SEC=WAIT_DUR_SEC)
  File "/Users/omerbelhasin/IdeaProjects/pywren-ibm-cloud/pywren_ibm_cloud/wait/wait_storage.py", line 72, in wait_storage
    THREADPOOL_SIZE=THREADPOOL_SIZE)
  File "/Users/omerbelhasin/IdeaProjects/pywren-ibm-cloud/pywren_ibm_cloud/wait/wait_storage.py", line 229, in _wait_storage
    pool.map(get_result, f_to_wait_on)
  File "/Users/omerbelhasin/.pyenv/versions/3.6.7/lib/python3.6/multiprocessing/pool.py", line 288, in map
    return self._map_async(func, iterable, mapstar, chunksize).get()
  File "/Users/omerbelhasin/.pyenv/versions/3.6.7/lib/python3.6/multiprocessing/pool.py", line 670, in get
    raise self._value
  File "/Users/omerbelhasin/.pyenv/versions/3.6.7/lib/python3.6/multiprocessing/pool.py", line 119, in worker
    result = (True, func(*args, **kwds))
  File "/Users/omerbelhasin/.pyenv/versions/3.6.7/lib/python3.6/multiprocessing/pool.py", line 44, in mapstar
    return list(map(*args))
  File "/Users/omerbelhasin/IdeaProjects/pywren-ibm-cloud/pywren_ibm_cloud/wait/wait_storage.py", line 213, in get_result
    f.result(throw_except=throw_except, internal_storage=internal_storage)
  File "/Users/omerbelhasin/IdeaProjects/pywren-ibm-cloud/pywren_ibm_cloud/future.py", line 220, in result
    self.status(throw_except=throw_except, internal_storage=internal_storage)
  File "/Users/omerbelhasin/IdeaProjects/pywren-ibm-cloud/pywren_ibm_cloud/future.py", line 171, in status
    raise FunctionException(self.executor_id, self.job_id, self.activation_id, self._exception, msg)
pywren_ibm_cloud.future.FunctionException: ExecutorID c2a0ec/0 | JobID M002 - There was an exception - Activation ID: fe3fb5a3cf8e449cbfb5a3cf8ef49cd5

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/omerbelhasin/IdeaProjects/pywren-ibm-cloud/pywren_ibm_cloud/tests.py", line 279, in test_multiple_executions
    result = pw.get_result()
  File "/Users/omerbelhasin/IdeaProjects/pywren-ibm-cloud/pywren_ibm_cloud/executor.py", line 457, in get_result
    WAIT_DUR_SEC=WAIT_DUR_SEC)
  File "/Users/omerbelhasin/IdeaProjects/pywren-ibm-cloud/pywren_ibm_cloud/executor.py", line 387, in wait
    sys.exit()
SystemExit

----------------------------------------------------------------------
Ran 1 test in 6.759s

FAILED (errors=1)
Deleting test files...

@omerb01 Do you have data_cleaner : True in config file?

@JosepSampe yep

@omerb01 Yep, I already found the issue, will submit in next PR ;)

@omerb01 Running some tests I can see that the results file from tests (__pywren.test/result) is always kept in storage. Is this the correct behavior? or it should be deleted at the end of the tests?

@omerb01 @gilv Submitted fix in #233

@omerb01 Running some tests I can see that the results file from tests (__pywren.test/result) is always kept in storage. Is this the correct behavior? or it should be deleted at the end of the tests?

@JosepSampe changed behavior on #234


[cudamapper] optimize overlapper

Optimize the overlapper in cudamapper to keep anchors generated from matcher in GPU memory when finding overlaps. Also port the overlap detection algorithm to GPU completely.

@tijyojwad @akkamesh
Overlapper is far for optimized, but the things this issue describes have been done, right?

@tijyojwad @akkamesh
Overlapper is far for optimized, but the things this issue describes have been done, right?

Yes, the issue described here is complete. Closing it.


Issues on windows

Hi!
Nice to see how you've configured your plugins into a ready to go Emacs! I realize it has not been configured for Windows though, but still thought of registering this issue. Do with it what you like!
So, this is my log in emacs after initially managing to install this radian package in windows:
$ cd "c:/Users/gusta/"
$ "find" "/dev/null" "-newermt" "2018-01-01 12:00:00"

FIND: Invalid switch

[Return code: 2]

$ cd "c:/Users/gusta/"
$ "touch" "-d" "2020-06-15 14:24:25" "c:/Users/gusta/.emacs.d/straight/mtimes/2020-06-15 14:24:25"

[File error while searching for program]

the FIND: Invalid switch ofc. is because it looks for the windows-version of find. I have configured this in init.local.el already though:
(setq straight-find-executable (expand-file-name "C:/msys64/usr/bin/find.exe"))
So a bit surprising with that error. Maybe find is used to early in the init-process?
Second error is due to touch not being available on Windows. Haven't found a workaround for that yet, except adding msys64/usr/bin to $path. Something I'd rather avoid.
Best
Gustav

Oh, should mention that the code above is from straight-process.

Maybe you can change the windows OS env  to make sure the msys2 prior to OS.

In fact, you've found a bug. I was not respecting the value of straight-find-executable for that particular find command. Try with the latest commit :)

Regarding your second concern, would it be good enough if there were a straight-touch-executable user option which you could customize?

To be clear, by the way, you'll need to go into ~/.emacs.d/straight/repos/straight.el or its equivalent on Windows and then pull, since I have not pushed an update to Radian.

In fact, you've found a bug. I was not respecting the value of straight-find-executable for that particular find command. Try with the latest commit :)

Great, will try it out. Side-question; Any reason for not using find-program and creating a separate variable for straight?

Regarding your second concern, would it be good enough if there were a straight-touch-executable user option which you could customize?

Yes, that sounds resonable

To be clear, by the way, you'll need to go into ~/.emacs.d/straight/repos/straight.el or its equivalent on Windows and then pull, since I have not pushed an update to Radian.

Got it.

Since the name for this issue is kind of broad I'll take the opportunity to raise another concern. And it relates to the compilation process that is set up. This also breaks on windows. Relying on make and bash-scripts for compilation as is done in radian-byte-compile will break in 99% of windows users configurations. Either it should only be active on non-windows machines or some kind of conditional logic is needed to determine how to do the external compilation.
Regards.

Your point is fair. You should open a new issue just for that, so that others may easily find it and perhaps contribute a solution.


[Jumping Off Code] Adds simple, untested implementation of rust deploys

This is a jumping off branch for someone who wants to finish adding support for deploying Rust projects. This code has been untested but essentially hands off a project directory to the cross compilation server and returns a tarred executable to the regularly scheduled deploy process.
There are two functions that need to be completed and these functions are called automatically by the logic that gets run when t2 run/pass is provided to the CLI. First, exportables.tarBundle provides an opts object with info about the directory being deployed (including path) and needs to resolve a Promise with a Buffer of the tarred executable. exportables.preRun needs to make sure the permissions of that executable are set properly on Tessel 2 for it to be run. There is also an exportables.preBundle function but we don't need to worry about that for Rust (it's used for special cases in JavaScript projects, primarily).
So here is a checklist of what needs to get done:

[ ] Make exportables.tarBundle production-ready. This means making sure it's working, checking error conditions, considering corner cases, etc. I also recommend changing the cross compilation server to tar up the executable server side before sending it down as that would speed up transfers and reduce logic in the CLI.
[ ] Set up the cross compilation server at a publicly available address
[ ] Ensure that exportables.preRun works
[ ] Reduce redundant mentions of rust_executable. I believe that name should be a variable instead of hardcoding it several times in this file and in /tessel/commands.js.
[ ] Add some unit tests
[ ] Make sure grunt is happy.

I'll grab the lock on this and see what I can do :)

Closing in favor of #774.


[zh-cn]add 6 feature-gates

allow-dns-only-node-csr
allow-insecure-kubelet-certificate-signing-requests
app-armor-fields
concurrent-watch-object-decode
disable-allocator-dual-write

/lgtm

/approve


Possibly add Changelog?

Hey! Nvim-org is awesome! And as more features get added it would be really easy for us to get updated on features that are now added and usable if there was a changelog saying what's been added. If you'd be up for that or making that a prerequisite for PRs I think it'd be a large help in staying up to date with the project!

Hey,
I agree that proper changelog would be helpful, but I didn't investigate some automatic ways how to generate it.
Currently creating a release allows generating changelog to some extent by Github (example: https://github.com/nvim-orgmode/orgmode/releases/tag/0.2.1), but this registers only PRs. If i push something directly to master it is not shown.
Do you know some tools that could help me on this to automate it?

No i don't know of any automated tools for this. But if it's a requirement for everyone to update the changeling when they add a feature it shouldn't be too much of a burden, considering we have to update the readme regardless.
I'm assuming the difficulty would be in making it match release versions in an automated way wouldn't it? I'm not really sure regarding that 😅.

If i push something directly to master it is not shown

release-drafter solved this by marking them as uncategorized changes. I did not test this action, but it looks like a good start. Perhaps they will add conventional commits in the future, making it even easier to categorize some loose pushes to master.

@kristijanhusak are you still interested in automating changelogs? Just found release-please and they also address the issue you brought up. Looks like it may require is to use conventional commits, but I don't consider that a bad thing tbh.


[terra-time-input] Fix to allow only 24 hour format for es locale

Summary
Time Input to allow only 24 hour notation for es locale.

Closes #1227
Deployment Link
Deployment Link
Testing

Additional Details

Thank you for contributing to Terra.
@cerner/terra

The variant description should also be updated to include es in TimeInput.jsx - L91 and DateTimePicker - L119

The variant description should also be updated to include es in TimeInput.jsx - L91 and DateTimePicker - L119
Also, i guess the following jest test should also be updated:
https://github.com/cerner/terra-framework/blob/main/packages/terra-time-input/tests/jest/TimeInput.test.jsx#L322-L332

Updated 7bf2148


undocking and redocking scene view breaks rendered scene

Release Type: GitHub
Version: master and <IP_ADDRESS>3
Platform(s): Windows
Describe the bug
Currently in Master when you undock and the redock the scene view it causes the scene render to go blank. You have to reload the scene to get the view back to normal.
To Reproduce
Steps to reproduce the behavior:

drag the scene tab into its own window.
place the scene back into the dock

Expected behavior
Sceneview should not be blank after redocking to GameStudio.
Screenshots

Additional context
This works fine in the release <IP_ADDRESS>8

Closing this one in favor of #2161


[Live] Adds Live screenshots.

Fixes #1706.
Note that the 3.5" screenshot has changed in height; I believe the existing one (920px / 460pt) is incorrect, and the new one (960px / 480pt) has the correct dimensions.

640 x 920 pixels for hi-res portrait (without status bar) minimum
640 x 960 pixels for hi-res portrait (full screen) maximum

The previous and the new size should both be ok, but we’ll know for sure when I try to submit them to the store.

Thanks 👍


Fix a couple of protocol issues found when documenting it

The startup sequence has been shortened, the connection parameters are no
longer sent as a separate message, and are instead sent as part of the
initial ClientHandshake message.
The ServerKeyData message now contains a sequence of 32 bytes instead of
a 32-bit integer.
This is a breaking change, but no bump to the protocol version, since we're
in alpha.

PR updated with the revert of Postgres OID elision in array and tuple data.