Pytest Setup Bug (seems duplicated test module filepath)

Hello Michael,
I met some problems when I executed docker-compose exec web python -m pytest to test the pytest app. (Session: Pytest Setup in part 1)
Error message:
$ winpty docker-compose exec web python -m pytest
======================================================== test session starts =========================================================
platform linux -- Python 3.10.1, pytest-6.2.5, py-1.11.0, pluggy-1.0.0
rootdir: /usr/src/app
plugins: anyio-3.5.0
collected 79 items / 1 error / 78 selected

=============================================================== ERRORS ===============================================================
_______________________________ ERROR collecting env/Lib/site-packages/sniffio/_tests/test_sniffio.py ________________________________
import file mismatch:
imported module 'sniffio._tests.test_sniffio' has this __file__ attribute:
  /usr/local/lib/python3.10/site-packages/sniffio/_tests/test_sniffio.py
which is not the same as the test file we want to collect:
  /usr/src/app/env/Lib/site-packages/sniffio/_tests/test_sniffio.py
HINT: remove __pycache__ / .pyc files and/or use a unique basename for your test file modules
====================================================== short test summary info =======================================================
ERROR env/Lib/site-packages/sniffio/_tests/test_sniffio.py
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! Interrupted: 1 error during collection !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
========================================================== 1 error in 4.54s ==========================================================

Self-check
This error seems to have happened by the duplicate pytest module name; I checked that adding __init__.py in my tests dir should fix the bug (as the instruction did), but it still does not work. I can't figure out what other reasons might cause this. (I did also try to rebuild the container and then do a test command, still did not work out.)
So my current directory structure is: (as same as the course demo, except .pytest_cache which is automated popped out when the test error happens.
fastapi-tdd-docker
├─ .gitignore
├─ docker-compose.yml
├─ project
│  ├─ .dockerignore
│  ├─ .pytest_cache
│  │  ├─ .gitignore
│  │  ├─ CACHEDIR.TAG
│  │  ├─ README.md
│  │  └─ v
│  │     └─ cache
│  │        ├─ lastfailed
│  │        ├─ nodeids
│  │        └─ stepwise
│  ├─ app
│  │  ├─ config.py
│  │  ├─ db.py
│  │  ├─ main.py
│  │  ├─ models
│  │  │  ├─ tortoise.py
│  │  │  └─ __init__.py
│  │  └─ __init__.py
│  ├─ db
│  │  ├─ create.sql
│  │  └─ Dockerfile
│  ├─ Dockerfile
│  ├─ entrypoint.sh
│  ├─ migrations
│  │  └─ models
│  │     └─ 0_20220216230908_init.sql
│  ├─ pyproject.toml
│  ├─ requirements.txt
│  └─ tests
│     ├─ conftest.py
│     ├─ test_ping.py
│     └─ __init__.py
└─ README.md

Thanks in advance.

@phwangktw I was facing something similar, when I noticed that the number of items collected by pytest was way too high.
What worked for me was specifying the test directory in the command using docker-compose exec web python -m pytest tests/.
Hope that helps!


Question not an issue: How to skip"provisioning and prepare phase" but run "running phase" directly and only?

Hi guys:
Everytime I run the benchmark, ./pkg.py always do the provisioning, prepare, running and clean up phases.
For example, if I run iperf on gcloud:
./pkb.py --project=xxxx --benchmarks=iperf --machine_type=f1-micro, the benchmark failed for some reasons. But the virtual machines and networks are still available.
How do I restart the benchmark without setting up the environments, download dependencies on VMs,
but run the ./iperf directly?
thanks!

There are two ways. You can use the --run_stage=... option to run the provision and prepare phase once and then run multiple times. You'll have to be sure to pass the same --run_uri=... to each stage of the benchmark so PKB will know where to find the machines. (You can let the provision stage pick its own run_uri and then pass that to the later stages.)
If you already have your own VMs sitting around, you can also create static VM files for them (see the manual) and tell PKB to use those.
I'll close this bug for now, but please reopen if the options above don't answer your question.

Hi Noahl:
I run my ./pkg.py like this:
./pkb.py --project=xxxxxx --machine_type=f1-micro --benchmarks=aerospike --run_stage=run,cleanup --run_uri=19673e88-6fc3fa80-290a-4caf-ac66-daf2419757d1

Hi Noahl:
Thanks a lot for your answers. I finished one benchmark and I search the run_uri in the log, its:
19673e88-6fc3fa80-290a-4caf-ac66-daf2419757d1
Then I RE-run my ./pkg.py like this:
./pkb.py --project=xxxxxx --machine_type=f1-micro --benchmarks=aerospike --run_stage=run,cleanup --run_uri=19673e88-6fc3fa80-290a-4caf-ac66-daf2419757d1

Hi Noahl:
Thanks a lot for your answers. I finished one benchmark and I search the run_uri in the log, its:
19673e88-6fc3fa80-290a-4caf-ac66-daf2419757d1
Then I RE-run my ./pkg.py like this:
./pkb.py --project=xxxxxx --machine_type=f1-micro --benchmarks=aerospike --run_stage=run,cleanup --run_uri=19673e88-6fc3fa80-290a-4caf-ac66-daf2419757d1
I want to only run "run and cleanup" phases, but there is an error:
"ERROR    run_uri must be alphanumeric and less than or equal to 8 characters in length.
"
What's wrong with it?
And I have VMs and Networks:
pkb-a827d375-0  us-central1-a  f1-micro                   <IP_ADDRESS>   <IP_ADDRESS>   RUNNING
pkb-a827d375-1  us-central1-a  f1-micro                   <IP_ADDRESS>   <IP_ADDRESS>  RUNNING
pkb-ada18ec9-0  us-central1-a  f1-micro                   <IP_ADDRESS>   <IP_ADDRESS>   RUNNING
pkb-ada18ec9-1  us-central1-a  f1-micro                   <IP_ADDRESS>   <IP_ADDRESS>  RUNNING
Will you please tell me how to port them into the static VM file and use it in the ./pkg.py command???
Thanks a lot!

Hi Martin,
Yes, the run_uri thing is a known issue in PKB. We use the name "run_uri" for two different things. Sorry!
In your example, you have two different run_uris of the kind we need: a827d375 and ada18ec9. Any time you run PKB with --run_stage set not to do teardown, it should print at the end of the benchmark exactly the --run_uri flag you need to use to continue with those VMs. It will look something like "to run other benchmarks with this setup, please use --run_uri=...", but it will give you the correct string.
To port them to static VM files, you'll need the IP addresses and information on how to login via SSH. PKB's README.md file will tell you how to do it. But since it looks like you are creating your VMs with PKB anyway, getting --run_uri working should be much easier for you, and I would recommend doing that.

I use "--run_uri=ada18ec9", pkg works fine.
But I don't understand your statements: "--run_stage set not to do teardown".
I run pkg with "  --run_stage=provision --run_uri=ada18ec9", PKG continoulsy runs
"Running: ssh -A -p 22<EMAIL_ADDRESS>-2 -o UserKnownHostsFile=/dev/null -o StrictHostKeyChecking=no -o IdentitiesOnly=yes -o PreferredAuthentications=publickey -o PasswordAuthentication=no -o ConnectTimeout=5 -o GSSAPIAuthentication=no -o ServerAliveInterval=30 -o ServerAliveCountMax=10 -i /tmp/perfkitbenchmarker/runs/ada18ec9/perfkitbenchmarker_keyfile hostname
", it repeated like forever.
I tried " ./pkb.py --project=local-catalyst-141222 --machine_type=f1-micro --benchmarks=aerospike  --run_uri=ada18ec9" without --run_stage options, it happens the same.

Hi Martin,
The hostname loop you're seeing is how PKB checks whether it can ssh into a VM. It should repeat until either PKB gets a response from the VM or PKB gives up and stops benchmarking. Did PKB eventually do something different?
For the run_stage flag, PKB will only teardown (delete) your VMs in three cases: if you don't pass --run_stage, if you set --run_stage=all, or if --run_stage is a list that includes teardown. Otherwise, PKB will not delete the VMs it creates. That's what I meant. If it doesn't delete the VMs, then it should tell you what --run_uri you need to use to reuse those same VMs.


Consider explaining the brackets like [R1] in Typographical Conventions

Looking at the bracketed numbers, e.g. [R1] or [D1], they look like they might be numbered requirements, maybe tied to the conformance clauses. I think it would help to explain what they mean in the Typo Conventions section.

@OASIS-OP-Admin  - Done in PR#63 : https://github.com/eea-oasis/baseline-standard/pull/63/commits/e0bb3da000ced73082ef55a853f790b78cb29068


Add unscape for macos

In response to comment on #82

Sorry for the slow response.
Check out how to test it: https://github.com/craftzdog/react-native-sqlite-2/pull/82#issuecomment-662211188

Ran the test inside my app, it doesn't crash, but the output is not quite the same as the screenshot you sent, is this correct?

if I remove the escaping/unescaping completely for macos, then it renders correctly, but then I'm not sure if I will face #51 again.

I guess for now I can remove the escaping/unescaping and see what happens, and if necessary open another PR

Seems like macOS Text component behaves differently.
In the second screenshot, I suspect that it dropped "\u0000Null" in the database, so I guess escaping nulls is necessary.
I updated the test to stringify data as JSON so that we can see the exact data: https://github.com/craftzdog/react-native-sqlite-2/commit/7b138336b037d0d40fa002767d7fe03a7969f1dc
If it stored/read data properly, you should be able to see "Zero\u0000Null" like so:

This is what I see with escaping/unescaping enabled:

this is without escaping/unescaping, sometimes it looks like it hangs:

and sometimes it finishes:

Actually they both hang sometimes and do not load all the output of the list, but with escaping I can see Zero\u0000Null string

Right. Can you respect formatting?
Now escapeForIOSAndAndroid is not a proper name, btw.

how about now?

Thanks, LGTM!


Reference formatting

@wrynearson @naomatheus
Description:
Citations in the reference section are not currently formatted to AGU (APA style).
Tasks:

[x] The journal issue is not populating in the reference section.
[ ] The journal title and volume should be italicized. Issue should be in parentheses.
[x] When more than one reference is added to a section, only the first reference entered shows up on the PDF.

Acceptance Criteria:

[ ] References are formatted correctly in the PDF downloads.
[ ] All references that a user enters should display on the PDF.

Current Reference Format (Note that the issue number does not display):
Hanson, B. (2014). AGU to Launch a New Open-Access Journal Spanning the Earth and Space Sciences. Remote Sensing and Environment, 95, 56-57.
Desired Format:
Hanson, B. (2014). AGU to Launch a New Open-Access Journal Spanning the Earth and Space Sciences. Remote Sensing and Environment, 95(6), 56-57.
Adding two references:

PDF result:

Opening a PR to address this issue.
@bwbaker1 cc @wrynearson

Hey @bwbaker1
Once the PR for this ticket is in staging, would you be comfortable checking the following criteria, to separate the italic formatting pointed out?

All references that a user enters should display on the PDF, including multiple reference in a row.

I may end up proposing that we create a ticket for bold/italic formatting in general because bold/italic formatting is affected across the document and the fix will across the whole document, rather than a single part of the document.

@bwbaker1 this is fixed and is on staging for your team's review. cc @deborahUAH

@wrynearson There are some testing comments here. This is ready for production.
cc @deborahUAH

Thanks @bwbaker1. I'm not sure specifically which comments you're referring to. Could you please add them to this ticket so that we can give our developers better visibility?

This ticket can be closed for Document PDF. When Journal PDF work is prioritized, we will implement the relevant changes and will list them in another ticket.


Add transform icon

Resolves #585

Please don't merge quite yet, I will push a commit to add the icon to the toolbar 😄


tabler_icon change $default param

Hey @kevinpapst,
wondering what is the purpose of $default in the tabler_icon TWIG function.
https://github.com/kevinpapst/TablerBundle/blob/8a70eaa6d8d41b116803d0fe43a7f184210324c1/src/Twig/RuntimeExtension.php#L131-L134
Due to that,
all of my calls of this function (when not using buttons), are made with null for that parameter.
{{ tabler_icon('email', null, false) }}

Two questions arise from this:

Should we use false to remove .icon class.
Should we override .icon css when using it outside of .btn.

My current problem is that the render with icon class and font-awesome is kinda ugly:

Too big
Not vertically centered
Overlap with label when large icon (e.g : fas fa-cogs)

Today .icon class: 🥉

.icon class overridden: 🥈

Without .icon class: 🥇

WDYT ?
Originally posted by @cavasinf in https://github.com/kevinpapst/TablerBundle/discussions/67

If we change the argument order, we should rename $isIcon to $isButton to make the purpose clearer.
Not sure about your app, but mine has probably 50 places where the icon function is used and less than 10 are outside of a button.
So having the icon class by default made sense to me.


Fix broken link for redis cache in home readme

Codecov Report

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

@@            Coverage Diff            @@
##             master     #526   +/-   ##
=========================================
  Coverage     54.43%   54.43%
  Complexity      651      651
=========================================
  Files           158      158
  Lines          2897     2897
  Branches        169      169
=========================================
  Hits           1577     1577
  Misses         1202     1202
  Partials        118      118

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 cd48ae5...51b5da4. Read the comment docs.


Return to the current place in the DOM after browserHistory back

I have a list of posts. After I go back from a single post page, the browser always brings me back up. How can I get him to take me back to my previous place in dom?

Have a look at https://github.com/taion/react-router-scroll for managing scroll behaviors with react-router


DataBinding dependent library

2017-5-8：

[Add] databinding dependent library.
[Add] Through databinding plugin refacting NormalActivity to DataBindingNormalActivity.

2017-5-9：

[Update]  Delete the extra code in DataBindingBinder, and remove some unchecked warnings.
[Update]  Remove the unnecessary genericity in DataBindingViewModel, and keep the saved items of MultiTypeAdapter.
[Update]  Delete some unnecessary variables in the demo, and add some comments.
[Modify]  Rename the VHandler to Collaborator, and remove the unchecked warning on onCreateViewHolder.
[Modify]  Rename DataBindingViewModel to DataBindingCollaborator.
[Modify]  readjust demo and xml.
[Modify]  DataBinding >> Binding.
[Update]  In the item xml show the role of the collaborator, there are hidden variable position.
[Update] configure the properties of demo xml to values package.
[Update] com.camnter.multitype.databinding >> me.drakeet.multitype.extensions.databinding.
[Remove] the androidTest package and espresso in databinding module.

2017-5-10：

[Remove]  the BindingCollaborator class and the Collaborator.
[Modify]  itemValue >> item.
[Fix]  BindingNormalActivity.
[Remove] Faker in databinding module.
[Remove] collaborator from databinding and sample module.
[Modify] replace LinearLayout to Space in cover layout of databinding module.

Thank you very much, but MultiType does not currently intend to provide special support for Data Binding, and the implementation of PR is unsatisfactory, I'm sorry,  at present I will close the PR. If you could make it better, or if you have any question, you can reopen it.


#1857 Use @ApiModelProperty required value if present instead of @NotNull

As stated in #1857, in this situation:
@ApiModelProperty(required = false, value = "Required only for Ipsum group")
@NotNull(groups = Ipsum.class)
private int lorem;

field lorem is marked as required, as it has @javax.validation.constraints.NotNull annotation. This change modifies behavior so if @ApiModelProperty is present, it will always be used it's required value. Swagger annotations should have precedence over other annotations.
Only drawback is that in existing code this:
@ApiModelProperty(value = "Smth")
@NotNull
private int lorem;

currently is marked as required (because of @NotNull), and after change it will be marked as not-required (as required=false is default value in @ApiModelProperty).
Every other code will act as before, so this:
@NotNull
private int lorem;

will be marked as required, since there is no Swagger annotation that would override it.
This change can be done for 2.0 version as well, so if it will be accepted I can make another PR for 2.0. Or maybe this change should go only to 2.0, as it somehow changes previous behavior. Please let me know what do you think.

Thanks with your effort with this one! I don't think we can go this way actually, because of the issue you mention in your comment about required defaulting to false which wouldn't allow to use the annotation "freely". A solution similar to what done for readOnly in #2675 for 1.5 and #2695 for 2.0 should be applicable here as well, adding an annotation field named e.g. necessity (or better naming..) with related enum with values like AUTO, REQUIRED, NOT_REQUIRED, with related support in reader/resolver code, and deprecating required.
Please let me know if you can help, and/or any comment.

I face the same issue when working around the lack of swagger supporting regex with (?i)
`import javax.validation.constraints.Pattern;
@Pattern(regexp = "^(?i)(new|closed)$")
@Schema(description ="In fact the field allows the pattern ^(?i)(new|closed)$ but this causes an invalid swagger document", pattern = "^(new|closed|NEW|CLOSED)$")`
Would be great to favor the own annotations over external ones.


Go: do not write chunk indexes for chunks without messages

Prior to this commit, if a chunk contained only messages or only
schemas, an index record would still be written. The record would
incorrectly reflect a start time of 0, which resulted in tooling
interpreting the file as starting at zero instead of the time of the
first message contained within.

fixes #255


react-color does not work in iframes

We're hosting the email editor in an iframe, so the users cannot change color of the text because of this bug: https://github.com/casesandberg/react-color/issues/798. There's a fix: https://github.com/casesandberg/react-color/pull/801 but the react-color package doesn't seem to be maintained anymore.

Please provide a full working example, I will check later.

Please nevermind — we've updated the email editor to the latest version and the problem appears to be fixed. Thank you!


Latest plugins added to CasC bundle instead of correct CAP versions

After upgrading to 2.303.3.3 the following plugins were added to the CasC bundle:
pipeline-stage-tags-metadata -> 1.9.3
pipeline-model-definition -> 1.9.3
plugin-util-api -> 2.5.1
pipeline-model-extensions -> 1.9.3
pipeline-model-api -> 1.9.3
However these were the latest versions instead of the CAP versions for 2.303.3.3
I think I found the bug in the script as its not considering the CAP plugin. It requires update in "jenkins-update-center" to use like "https://jenkins-updates.cloudbees.com/update-center/envelope-core-mm/update-center.json?id=cap-cje-mm&version=$CI_VERSION" based on this link https://support.cloudbees.com/hc/en-us/articles/230655088-A-plugin-release-is-not-available-from-Update-Center
And disabled usage of environment variable JENKINS_UC_HASH_FUNCTION & CACHE_DIR.
With those change, I was able to generate the plugins.yaml and plugin-catalog.yaml for 2.303.3.3 and managed controller was able to start successfully.
For further details, see support ticket number: 210888

I do think the CAP version of the UC is needed, but that seems to be the only thing I can find that needs to change to make this work. A little hard to reproduce, too.

I have tried to reproduce this without success.


Canvas CachedMode with DesignSize has a different scale than non Cached mode

http://www.babylonjs-playground.com/#22PLQU#0
Report to @wyklion

@deltakosh I can't reproduce this bug.  The only bug I see on that PG now is fixed by my latest PR.

Closing then:)


MemoryMapRef/MemoryMapRefMut constructors

I think right now there's no way from the public API to construct MemoryMapRef or MemoryMapRefMut. I figure we want to be able to construct them from a MemoryMapOwned reference and from byte slices?

I'll do it after #1240 is merged.


examples/client.py fails on second interaction

Error when entering a second name:
websockets.exceptions.ConnectionClosed: WebSocket connection is closed: code = 1000, no reason.

Guessing the while and the context manager need to be switched?

It's an unintended change that was committed accidentally in a41c8310807af4de9b2275c685988b3cbcf01ae2.
Thanks for the heads up.


No default export defined for file "X" or export does not satisfy filename. with ts-jest

Versions

sequelize: 5.22.3
sequelize-typescript: 1.1.0
typescript: 3.9.7

I'm submitting a ...

[x] bug report
[ ] feature request
Actual behavior:

Using sequelize-typescript with ts-node is perfectly working, however, when running the E2E tests with ts-jest, I am getting the following error:
 No default export defined for file "X" or export does not satisfy filename.

The class is named X like its own file
// X.ts

import {
  Table,
  Column,
  Model,
  AllowNull,
} from "sequelize-typescript"
@Table({ underscored: true })
class X extends Model<X> {
  @AllowNull(false)
  @Column
  name!: string;

}

export default X

I did add some logging here to see what's happening and
Running with ts-node:
{ default: X } '/private/var/www/project/models/X'

Running with ts-jest:
{ } '/private/var/www/project/models/X'

Expected behavior:

Make the tests working with ts-jest
Steps to reproduce:

Related code:

insert short code snippets here

any ideas?

Hi @oktapodia , I am facing the same error, where I have models = [__dirname + "/../path/to/my/models"] add in my Sequelize. I'm wondering did you solve this issue?

@oktapodia
How were you able to solve this issue  , im having the same issue

@oktapodia ,
How were you able to solve this issue , im currently having the same issue


Adding training data for table reservation

Added training data for following intents:
locate_restaurant
restaurant_type
table_reservation
restaurant_review

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

Thanks for submitting a pull request 🚀 @melindaloubser1 will take a look at it as soon as possible ✨


Update index.md

Summary
Addresses fixes from https://github.com/dotnet/docs/projects/35#card-19059588
Fixes #11388

Thank you, this is my first time doing this, so i'll know better next time :)


Wrangler publish outputs to stderr instead of stdout

🐛 Bug Report

Environment

operating system: Linux mint

output of rustc -V: None

output of node -v: 10.16.3

output of wrangler -V: 1.13.6

contents of wrangler.toml

name = "[redacted]"
type = "webpack"
account_id = [redacted]
workers_dev = true
route = [redacted]
zone_id = [redacted]
[site]
bucket = "./www"
entry-point = "workers-site"
Steps to reproduce
wrangler publish outputs to stderr instead of stdout.
command --> wrangler publish 2>stderr.txt
What did you expect to see?
if publish outputs to stdout, stderr.txt would be empty.
What did you see instead?
instead, output is in stderr.txt

This is intended behavior, we reserve stdout for structured output (ex. for CI), while stderr is for diagnostic output, progress reports/logging, and similar


Add annapurna workflows and dependencies (from fork)

This PR merges all ubuntu windows and multiplatform actions.
This is all joint in a PR because multiplatform actions requires ubuntu and windows ones, and most ubuntu and windows does not have sense anymore once multiplatform os implemented.
It would be nice to check in this PR:

[ ] All actions has their name correctly set in action.yaml and it is the same as the directory
[ ] All actions and arguments have descriptions
[ ] All shell actions use ::group:: at the beginning and ::end group:: at the end
[ ] There are no mix actions (using shell and other actions) or they are well documented
[ ] No relative paths are used along the arguments

This PR is tested in the following external PR: https://github.com/eProsima/Fast-DDS-spy/pull/30

Close in favor of #17


Azure connected service updates

There was an error while validating job d93ba42b-64d3-4555-ac1a-7a58ba556049 Message:Error clonning repositoryDetails:System.ArgumentException: Error clonning repository ---> System.Exception: Error when trying to initialize shared repository ---> LibGit2Sharp.NotFoundException: Branch 506 not found. Available branches:
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.GetBranch(Repository repo, String name)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.<>c__DisplayClass3e.b__3d()
at Microsoft.Practices.EnterpriseLibrary.TransientFaultHandling.RetryPolicy.<>c__DisplayClass1.b__0()
at Microsoft.Practices.EnterpriseLibrary.TransientFaultHandling.RetryPolicy.ExecuteAction[TResult](Func1 func)    at Microsoft.Practices.EnterpriseLibrary.TransientFaultHandling.RetryPolicy.ExecuteAction(Action action)    at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.InitializeSharedRepository(String url, IDictionary2 branches, String sharedRepoPath)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.VerifySharedRepository(String url, IDictionary2 branches, String tempPath)    --- End of inner exception stack trace ---    at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.VerifySharedRepository(String url, IDictionary2 branches, String tempPath)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.CloneRepository(String url, IDictionary2 branches)    --- End of inner exception stack trace ---    at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.CloneRepository(String url, IDictionary2 branches)
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.CloneRepository()
at Microsoft.DPS.Services.LibGit2SharpSourceControlProvider.GetTreeAsync(String path, Boolean includeSubDirectories, String excludePath, IList1 extensionWhiteList)    at Microsoft.DPS.Commands.IdentifyDocumentsToPublish.ListSourceControlFilesCommand.Do(IDictionary2 context)
at Microsoft.LightweightJobEngine.Core.Language.JobWorkflow1.RunCommands(Guid jobId, IDictionary2 context, CancellationTokenSource cancelSignal)


fix(deps): update dependency protobufjs to v6.10.1

This PR contains the following updates:

Package
Type
Update
Change

protobufjs (source)
dependencies
minor
6.9.0 -> 6.10.1

Release Notes

protobufjs/protobuf.js
v6.10.1
Compare Source
v6.10.0
Compare Source
Features

add configurable Root.prototype.fetch (ad3cffd)
better comment parse (#1419) (7fd2e18)
parsed options (#1256) (7a25398)

Bug Fixes

allow Windows unc paths to be resolved and normalized (#1351) (cd4aeda)
do not fail if no process (#1440) (f2faa8c)
fix util.global (#1441) (742b8dc)
google.protobuf.Any type_url fixes (#1068) (192f5f1)
handling of map entries with omitted key or value (#1348) (b950877)
properly parse empty messages in options (#1429) (7fbc79f)
updated isNode check (#1221) (#1363) (5564e7b)

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

Available now for Enterprise: Renovate Pro with real-time webhook handling and priority job queue.

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

v5.1.9
GitHub release

Your semantic-release bot :package::rocket:


Issue 35 : Upgrade the Issue Template to use new GitHub Issue forms

Linked Issue
It fixes https://github.com/stepzen-samples/stepzen-spacex-graphql/issues/35
Description
This PR updates the Issue templates for Feature, Bug, and Documentation requests to use the latest Github Issue forms.
Methodology
Respective yaml files have been added that would be used to provide the newest template for any issue requests.

@Cerchie Please review and let me know if it looks good to you.

thanks @Rutam21  ! We'd love to send you stickers to thank you, you can submit your address here https://stepzen.com/stickers


Ensures integration tests won't hang

This adds a global timeout of 5s/method for HTTP integration tests.
It also carries over the 3s/poll timeout from taking a span for other
queue based operations. This is an effort to reduce the impact of
testing async libraries with bugs in their callback management.

specifically, I'm trying to test the reactor netty client in sleuth, and it is hanging and this would allow me to know which tests don't

ack but that would only apply to this project. the downstream would have to
copy that in maven or gradle etc. maybe I can add a comment clarifying why
here
On Thu, Feb 6, 2020, 11:38 AM Anuraag Agrawal<EMAIL_ADDRESS>wrote:

@anuraaga approved this pull request.
This is fine but it can be hard to make sure any piece of code times out.
How about also setting a global timeout in surefire?
http://maven.apache.org/surefire/maven-surefire-plugin/test-mojo.html#forkedProcessTimeoutInSeconds
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/openzipkin/brave/pull/1076?email_source=notifications&email_token=AAAPVVZLEUXDMMUXSYSPBLDRBOA3FA5CNFSM4KQVYDR2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCUOGXJQ#pullrequestreview-354184102,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAPVV3F77UVHZODV4VBEVLRBOA3FANCNFSM4KQVYDRQ
.

Ah yeah that's right - a comment could be a good reminder.
Also just a random thought for future reference that is probably too much work, but junit5 does support a global timeout we could propagate downstream using a properties file. Maybe someday ;)

I will help in fallout if you feel up to converting the tests to junit5. we
could change it for the next minor (not next patch)

thx for the advice @anuraaga

While a timeout to pull a request or span off the queue makes sense. GlobalTimeout should be reverted as it interferes with debugging. You can imagine trying to step through a debugger in less than 5 seconds :)

Ah, can wrap with DisableOnDebug to solve that.
https://junit.org/junit4/javadoc/4.12/org/junit/rules/DisableOnDebug.html

wow cool!
On Sat, Feb 8, 2020, 7:06 AM Anuraag Agrawal<EMAIL_ADDRESS>wrote:

Ah, can wrap with DisableOnDebug to solve that.
https://junit.org/junit4/javadoc/4.12/org/junit/rules/DisableOnDebug.html
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/openzipkin/brave/pull/1076?email_source=notifications&email_token=AAAPVVY2SHW5WCNBC3ABFEDRB3C5TA5CNFSM4KQVYDR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELFTX3I#issuecomment-583744493,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAPVV5KKWTNW5GWSD7QJOLRB3C5TANCNFSM4KQVYDRQ
.


Allow same line comments

Awesome project, simple and useful!
What doesn't work are comments at the end of a line, example:
db.port=8080 # this is some port
Of course, this would also mean that values may not contain hashtag. Maybe an option for the reader can be set whether to enable or disable this same-line comments feature?

@salim7
You are proposing a feature that I have never thought about. I am not against it, but from my point of view, whole-line comments are more common and more clear.
Anyway, feel free to send a PR for supporting that style of comments with a flag and I'll evaluate it for sure.


A way to link custom framework for iOS custom module

I'm not sure there is already way to do this or not documented yet.
I like to have a way to link custom framework like admob or chartboost.
https://developers.google.com/admob/ios/quick-start
https://answers.chartboost.com/hc/en-us/articles/201220095-iOS-Integration

just add a regular module on ios and compile .mm file


Disable an ability to create view for CSV connection

System information:

Operating system (distribution) and version win10 x64
DBeaver version 7.1.3

Connection specification:

Database name and version CSV

Describe the problem you're observing:
We need to disable views and its creation for CSV connection

Fixed.
Also indexes, unique keys and foreign keys were disabled for CSV and DBF drivers.

verified


[API] The API is out of date !

The API is out of date !!
There's no method called '.userSelf' right?

bot.userSelf() is still usable in latest Wechaty, although it's deprecated. You should use bot.currentUser now.
The API doc is out of date indeed. We are working on that.


Get a production gsa.gov url

What:
Get a production gsa.gov url
How:

[x] Ask in #infrastructure about the process
[ ] Start the process
[ ] Finish the process

Notes:
We will most likely want this service to live on a GSA domain. We have grand ambitions of the being a TTS run government wide service though. So whatever we do to get a GSA domain, don't lock out the possibility of getting new domains later.

We make the request here https://gsa.service-now.com/
Service Catalog > Enterprise Services > DNS
I think we want to request internal + external DNS
I e-mailed Richard and Marcela to ask them what domain they wanted.


Test fixing Snyk warning

Overview
Brief description of what this PR does, and why it is needed.
Connects #XXX
Demo
Optional. Screenshots, curl examples, etc.
Notes
Optional. Ancillary topics, caveats, alternative strategies that didn't work out, anything else.
Testing Instructions

How to test this PR
Prefer bulleted description
Start after checking out this branch
Include any setup required, such as bundling scripts, restarting services, etc.
Include test case, and expected output

Checklist

[ ] fixup! commits have been squashed
[ ] CI passes after rebase
[ ] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines
[ ] This PR is targeted at the correct branch (develop vs. ogr/develop)
[ ] If this PR applies to both OAR and OGR a companion PR has been created

Oops, meant to add this to gh-poc fork for testing


PWA-594 function test login

Pull Request Test Coverage Report for Build 1488

0 of 16 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 37.994%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

e2e/integration/functional/LoginPage.js
0
16
0.0%

Files with Coverage Reduction
New Missed Lines
%

e2e/integration/functional/LoginPage.js
1
0.0%

Totals

Change from base Build 1486:
-0.1%

Covered Lines:
1375

Relevant Lines:
3593

💛  - Coveralls


Update for 2021

[x] Add Recursion assignment
[x] Update Portfolio culminating
[x] Update Close of Project culminating
[x] Update Contemplation culminating
[x] Update ET
[x] Update TCS
[x] Update Algo
[x] Update DS
[x] Update Documentation
[x] Update OLG visual

Only portions left at this point are the Culminating Activities and the OLG graphic.

All major updates completed. Want to update Recursion work for spring 2022.

All major updates completed for semester 1.


Fancy Zones is broken

Microsoft PowerToys version
0.60.0
Running as admin

[X] Yes

Area(s) with issue?
FancyZones
Steps to reproduce
Running Fancy Zones on 3 monitors
✔ Expected Behavior
Fancy Zones should remember its settings
❌ Actual Behavior
Fancy Zones uses the wrong screen settings
Other Software
No response

/dup of https://github.com/microsoft/PowerToys/issues/19203
Working on a fix. Thank you.


Need some free BTC Testnet RPC server information

Hello:
I want to use the repo to test some BTC RPC calls, but I want to
find some public BTC testnet RPC server, I need some IP address and port number, I don't want to setup a localhost RPC server.
Please advise where I can find such BTC RPC servers on testnet.
Thanks,

Hi.
For public rpc servers, you can visit getblock.io. But if you need an rpc server to test this repo you can try bitcoin regtest.
You can check this gist to learn how to use the regtest mode.


fix: add state_dict to avoid lacking of the method argument

During fine-tune embedding or reranker models under the latest transformers(may > 4.28.1), it will cause an exception due to the lacking ofstate_dict in current Trainer _save method when save the checkpoint.
Close: https://github.com/FlagOpen/FlagEmbedding/issues/233

Thanks！


Add Important Aditional Detail Pages to SAP CDC

I would recommend that you add important additional information pages / samples / Next steps like
https://learn.microsoft.com/en-us/azure/data-factory/solution-template-replicate-multiple-objects-sap-cdc
https://learn.microsoft.com/en-us/azure/data-factory/sap-change-data-capture-introduction-architecture
https://learn.microsoft.com/en-us/azure/data-factory/sap-change-data-capture-advanced-topics

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 24e254c6-6f8e-d271-8122-4c86d6278592
Version Independent ID: 6481d2fa-3edd-28ea-bee6-c423898a1a65
Content: Transform data from an SAP ODP source with the SAP CDC connector in Azure Data Factory or Azure Synapse Analytics - Azure Data Factory & Azure Synapse
Content Source: articles/data-factory/connector-sap-change-data-capture.md
Service: data-factory
Sub-service: data-movement
GitHub Login: @ukchrist
Microsoft Alias: ulrichchrist

@hmayer1980
Thanks for your feedback! We will investigate and update as appropriate.

@hmayer1980
Thank you for bringing this to our attention.
I've delegated this to content author @ukchrist, who will review it and offer their insightful opinions.

@ukchrist
Could you please review add comments on this, update as appropriate.

Thanks @hmayer1980 for your help to improve the docs! I noted these links are now there on the page, so I will close this issue out, but we can continue the discussion if necessary on this thread, even after it is closed. #please-close


Release AnnotationPlugin 0.1.0

Changelog
Initial release! 🎉
Release checklist

[x] Cut the release branch
[x] Run through testapp/demo app and test all activities
[x] Update README.md
[x] Update CHANGELOG.md
[x] Change version name to 0.1.0 in gradle.properties
[x] Publish artifact to Maven Central
[x] Change version name back to 0.2.0-SNAPSHOT
[x] Merge the release branch back to master
[x] Tag v0.1.0-annotation in GitHub

Post-release  @langsmith

[x] Update constants.js file in android-docs repo
[x] Generate updated javadocs and add to android-docs repo

/cc: @mapbox/android

🏁 🎉


⚠ Jenkins Plugins Artifactory has degraded performance

In 6869715, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1184 ms

Resolved: Jenkins Plugins Artifactory performance has improved in dcced02 after 12 minutes.


Add documentation about caching options

Existing documentation URL(s)

https://developers.cloudflare.com/pages/framework-guides/nextjs/deploy-a-nextjs-site/

What changes are you suggesting?
It's currently not clear what options you have for Nextjs caching with Cloudflare. Specifically how you can hook up a reliable distributed cache setup by adding Workers KV as your cache binding. This to me is critical coming from Vercel where caching performs as well as Workers KV out-of-the-box.
The suggestion is to add a clear section on this page outlining the standard cache and the Workers KV cache. Much like the information on this page. Or at a minimum, having a section called "Caching" on the page with a link to that caching doc.
Until finding this documentation, I'd spent a week debugging why my Nextjs app (using SSR / app router) was showing old data on browsers in various locations. This was happening via initial load, page refresh and the router cache. It was very unstable. Once we added Workers KV as the cache store it solved all these issues 💥
Additional information
No response

Thanks a bunch for the issue @scottmackenzzz :pray:
I'm sorry you had a hard time with the Next.js caching due to the difficult to find/unclear documentation on our side :sweat:
Hopefully
https://github.com/cloudflare/next-on-pages/pull/796
and
https://github.com/cloudflare/cloudflare-docs/pull/14859
should improve the situation :slightly_smiling_face:
Please let me know what you think :pray:

Howdy @onepixelsolid, this is actually a new page at https://developers.cloudflare.com/pages/framework-guides/nextjs/ssr/caching/


Standalone daemon installation package without vagrant / kubernetes?

Hey team,
After reading the announcement of the project, I've been eager and excited to play around and compare how the tool compares to others like Falco.
I was able to follow the instructions and get an environment running within a Vagrant image and monitoring the sample kuburnetes container images.
At a glance (might be wrong here), it seems that the Makefile makes some assumptions about being installed in a container and not the host itself.
Are there any plans to offer an installation package (deb, rpm) that allows for a stand-alone installation and configuration of the daemon?

Hey team,
After reading the announcement of the project, I've been eager and excited to play around and compare how the tool compares to others like Falco.
I was able to follow the instructions and get an environment running within a Vagrant image and monitoring the sample kuburnetes container images.
At a glance (might be wrong here), it seems that the Makefile makes some assumptions about being installed in a container and not the host itself.

Nope, you should be able to build a standalone binary as follows: make tools-install && LD_LIBRARY_PATH=$(realpath ./lib) make. Then you could just copy the tetragon binary to your desired location. You can pass in a CRD to a standalone tetragon using the --config-file command line flag.

Are there any plans to offer an installation package (deb, rpm) that allows for a stand-alone installation and configuration of the daemon?

I don't know of any plans per se, but I'm sure a PR would be welcome if you want to make a package for your favourite distro.

Hey @Happy-Dude, I had a similar requiremt for testing purposes - Wanted to see how Tetragon compares to other eBPF security tools on my test machines that don't have Kubernetues clusters running on them.
Helpfully @tixxdz on the Slack shared this gist to run things inside a docker container:
# 1.  Get events:
docker exec -it tetragon \
bash -c "/usr/bin/tetra getevents -o compact"

@willfindlay , while I understand this isn't a 'proper' way to deploy Tetragon, having this example in the repo (under "local testing" or something) would be extremely useful for people wanting to try and compare Tetragon without needing a full k8s environment

having this example in the repo (under "local testing" or something) would be extremely useful

Agreed, would be happy to take a PR. :)

@pathtofile @Happy-Dude https://github.com/cilium/tetragon/pull/384 this should do it for now right? for pure deb or rpm packages it is just matter of priorities we will try to do it, or maybe someone will contribute it. Thank you!

We do have a systemd installation now, so closing this.

Very, very, late, but thanks @tixxdz !


Check if its possible to parse a chunked response

This patch is a simple check for either read the chunks
of the response of use the standard read method.
Currently if it is not possible to use the 'read_chunked'
method the client fails.
This commit fixes #139

Codecov Report

Merging #140 into master will decrease coverage by 0.05%.
The diff coverage is 80%.

@@            Coverage Diff            @@
##           master    #140      +/-   ##
=========================================
- Coverage   92.86%   92.8%   -0.06%
=========================================
  Files          13      13
  Lines        1345    1349       +4
=========================================
+ Hits         1249    1252       +3
- Misses         96      97       +1

Impacted Files
Coverage Δ

watch/watch.py
98.85% <80%> (-1.15%)
:arrow_down:

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 474e9fb...484e776. Read the comment docs.

Hey folks @mbohlool @roycaihw, I'm hitting all over #139 and I wanted to know if this can be done. I already have patched my clusters and seemed to work fine.
Thanks!!

@mmazur @tripledes reviews are welcomed :)

Right, so let me start with the fact that I'm not familiar with any of this code and I'm not in a position to merge anything.
However, from doing some digging around it looks to me that the proper way to handle chunky responses is not with exceptions, but by emulating what the authoritative source (urllib3 itself) does. Which is this. Now, I'm not sure what the is_fp_closed() thing does and whether it's applicable in our case, but it looks to me like if self.chunked and self.supports_chunked_reads() is what we want to be doing here in lieu of catching exceptions.

@mmazur yeahp, you are actually right as we can handle the chunked case in that way. The main problem is that the urllib3 version we deliver does not have those methods.
rpm -qf /usr/lib/python2.7/site-packages/urllib3/response.py
python-urllib3-1.10.2-5.el7.noarch
Presently from 1.25.3
That's the main reason I decided to use an exception instead, just to avoid the dependency.

may I ask what responses from Kubernetes APIs do not support read_chunked?

/remove-lifecycle stale

Hi @roycaihw thanks a lot for your reply! And sorry for not replying faster.
The issue in question is described in #139, basically when calling the watch() method the client breaks.
rpm -qf /usr/lib/python2.7/site-packages/urllib3/response.py
python-urllib3-1.10.2-5.el7.noarch
For example:
stream = watch.Watch().stream(crds.list_cluster_custom_object, DOMAIN, "v1", "guitars", resource_version=resource_version)
The problem is that the urllib version we ship does not have this method read_chunked, and handling it with an exception allows having backwards compatibility with older versions of urllib.
The fix is actually really simple and we are already using this fix but it would be awesome if we can include it by default as it affects any OpenShift and OKD distribution based in the 4.0 version.
Thanks!

/remove-lifecycle stale


This description is incorrect, the verification test does not support: the aggregate column storage index now supports LOB columns (nvarchar(max), varchar(max), varbinary(max))

TRANSLATED
@sqlora2019 commented on Wed Jul 03 2019
This description is incorrect and the validation test does not support: The aggregate column storage index now supports LOB columns (nvarchar(max), varchar(max), varbinary(max)).
Please check the function.

文档详细信息
⚠ 请勿编辑此部分。 docs.microsoft.com ➟ GitHub 问题链接必须具有此部分。

ID: 9547ecaa-d979-ee91-5657-9c302a829d38
Version Independent ID: baf83ba3-4555-e3ba-123c-adab825200ba
Content: 数据库引擎中的新增功能 - SQL Server 2017 - SQL Server
Content Source: docs/database-engine/whats-new-in-sql-server-2017.md
Product: sql
Technology: release-landing
GitHub Login: @rothja
Microsoft Alias: jroth

@CeciAc commented on Wed Jul 03 2019
你好，@sqlora2019

感谢你的反馈。

我们正在积极查看你的评论，并会很快地再次和你取得联系。

深深的祝福，
Microsoft DOCS International Team

This issue was Machine Translated

@CeciAc -- thank you for raising this feedback.
@rothja -- please process this issue 2717.

@CeciAc Thanks for passing this to us. I think the feedback is that the stated new feature about LOB columns does not actually work? I can try to check on this. Thanks!

@CeciAc Hello. I have not been able to test and validate this yet. I have opened a work item to track this work internally (1610832) and I will try to update this git issue when it is verified/resolved. Thank you for your patience.


docs(api): Add missing update users limits endpoint

Add PUT: /v2/users/me/limits to docs (ep already exists in api, but was not documented yet.)
Resolves: https://github.com/apify/apify-client-js/issues/329

👏 Sadly this endpoint is missing on several places:

missing in JS client https://docs.apify.com/api/client/js/reference/class/UserClient
missing in Python client https://docs.apify.com/api/client/python/reference/class/UserClient
Missing in console API docs

Can you look and fix or create issue to fix?

The JS client is ready here: https://github.com/apify/apify-client-js/pull/595
I will look at python client and console docs as well, thanks


npm build does not use data from JSON files

When using the following file structure (based on the provided example in VItuum repo)

and running build  npm script, the resulting HTML files are not build using any of the data present in the JSON files.
When running dev script, everything works fine

Fixed in 1.0


enum metaclass methods

This PR implements the following metaclass methods for enum types: __iter__, __contains__, __getitem__, __len__ and __reversed__, similar to those of standard Python enums.
Unfortunately, this also reintroduces entanglement between nb_type.cpp and nb_enum.cpp, so I'm marking this as a draft for now, but posting to start the discussion of possible implementations. To me it seems like keeping them detangled would require adding some kind of metaclass API, but maybe there is a more simple way?
I've also noticed that stubgen.py crashes when it encounters an empty enum due to them not having an @entries attribute and therefore not being recognized as an enum. Should we perhaps always create this attribute? Or handle __doc__ more carefully in stubgen.py?
[27/29] Generating test_enum_ext.pyi
FAILED: tests/test_enum_ext.pyi /home/roman/projects/nanobind/build/tests/test_enum_ext.pyi

cd /home/roman/projects/nanobind/build/tests && /home/roman/venv/master-3.12/bin/python3 /home/roman/projects/nanobind/src/stubgen.py -q -i /home/roman/projects/nanobind/build/tests -m test_enum_ext -o test_enum_ext.pyi
Traceback (most recent call last):
  File "/home/roman/projects/nanobind/src/stubgen.py", line 1463, in <module>
    main()
  File "/home/roman/projects/nanobind/src/stubgen.py", line 1434, in main
    sg.put(mod_imported)
  File "/home/roman/projects/nanobind/src/stubgen.py", line 838, in put
    self.put(child, name=name, parent=value)
  File "/home/roman/projects/nanobind/src/stubgen.py", line 845, in put
    self.put_type(value, name)
  File "/home/roman/projects/nanobind/src/stubgen.py", line 544, in put_type
    self.put_docstr(docstr)
  File "/home/roman/projects/nanobind/src/stubgen.py", line 283, in put_docstr
    docstr = textwrap.dedent(docstr).strip()
             ^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/roman/.pyenv/versions/3.12.2/lib/python3.12/textwrap.py", line 435, in dedent
    text = _whitespace_only_re.sub('', text)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: expected string or bytes-like object, got 'getset_descriptor'

cc @oremanj

Having a dependence from nb_type -> nb_enum is something I feared and would like to avoid.
How about giving nb_type_tp the ability to add custom slots? Rather than exposing this externally to the user, the enum constructor could internally use this to create an enums-specific metaclass that adds the functions you listed above.
 The smaller this all can be, the better.

Any thoughts on this @rzhikharevich? Note that if API-breaking changes are needed, then it would be good to still include them in 2.0.0. (And that window of opportunity will be closing in the next weeks.)

How about giving nb_type_tp the ability to add custom slots? Rather than exposing this externally to the user, the enum constructor could internally use this to create an enums-specific metaclass that adds the functions you listed above.

Sounds good to me, I'll try to implement this in the next few days.

Isn't any object with len and getitem implicitly reversible and iterable?

I think the problem is that we define mp_subscript (accepts an object) instead of sq_item (accepts an index), so it is not automatically reversible / iterable. I'll double-check this later.

@wjakob I've updated the PR to set up the metaclass through fields in type_init_data: metaclass_type_slots for extra slots and metaclass_name for setting a custom metaclass name (as well as using it as a key in the metaclass dictionary).

closing in favor of #533


Increase committee size in sim tests

Motivation
Increase committee size in sim tests to avoid any regressions we may have

Implemented due to #3919 not being detected from sim tests potentially from small committee sizes. Taking note that we should also seek to improve our sim tests with more depth when able.


Hard Coded GOPATH In bashrc File

There is a hard coded GOPATH in the bashrc file of the common role. Should this be removed since it is also set in the env10.sh file?

Fixed here: https://github.com/jasonodonnell/CoreDNS-Ansible/commit/c913804af2e8192f53ae8e4db961bc28ecb4dcac
Thanks @amayer5125 !


Fix Automatic-Module-Names to use dots instead of dashes (not allowed…

… by JPMS).
Issue #, if available:
Description of changes:
I made a mistake in #295 - dashes are not allowed in module names. This PR uses dots instead. I also opened #297 which uses underscores - I'll leave the choice up to the maintainers :).
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #296 (bb8822c) into master (297a0bf) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #296      +/-   ##
============================================
- Coverage     58.86%   58.84%   -0.02%
+ Complexity     1202     1201       -1
============================================
  Files           131      131
  Lines          5061     5061
  Branches        590      590
============================================
- Hits           2979     2978       -1
  Misses         1807     1807
- Partials        275      276       +1

Impacted Files
Coverage Δ

...ava/com/amazonaws/xray/AWSXRayRecorderBuilder.java
68.42% <0.00%> (-0.88%)
:arrow_down:

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 297a0bf...bb8822c. Read the comment docs.

Closing in favor of #297


configure deployment on playground with circleCI

Still WIP since the creation of new apps from the CI does not work yet, but apart from this the deployment works.

@almarklein I think this is now ready for review (I'll squash commits before merging :-).

Out of curiosity, I assume the playground is used by more dash-related projects. If two projects that use it happen to have a pr open with a branch name that is the same, we'll have a name clash, right? (not a big deal, but good of be aware of)

This LGTM, by the way!

Out of curiosity, I assume the playground is used by more dash-related projects. If two projects that use it happen to have a pr open with a branch name that is the same, we'll have a name clash, right? (not a big deal, but good of be aware of)

You're right and I considered that this is not a big deal since the playground is just a sandbox (used for things like continuous deployment and work in progress). But it's easy to prefix the branch name with 'slicer' so that the risk of fortuitous clashes is very much reduced, I'll do that.

I'll avoid branch names called "dev" from now on ;)


WIP: AST Traverse

WIP: AST Traverse
fix: add checks to make sure structs aren't deriving Copy trait.

#2987  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rzvxa and the rest of your teammates on  Graphite

@rzvxa So how is this working exactly? Does #[ast_node] mutate the current AST node? Assumed we would create a duplicate node, like TraversableProgram.

@rzvxa So how is this working exactly? Does #[ast_node] mutate the current AST node? Assumed we would create a duplicate node, like TraversableProgram.

It would modify and validate our ast_node and generate their appropriate traversable versions, mutation wrappers, etc.
It will also ensure that the 2 types have the same representation and size; It adds some constant assertions to test it out.

I've started going through this.
So far just been tidying up a little. Have pushed a few commits to remove changes which aren't relevant to the thrust of this PR, and submitted a couple as separate PRs (#3026, #3027). Aim is to reduce the diff a little, and make it a bit easier to follow the substantive changes.
Will start going through the meat of it soon as I can.

@rzvxa It's starting to come into focus for me now.
I attempted to make it work without changing the AST further, but unfortunately can't find a way to avoid soundness issues without all enum variants being boxed, and other changes (my failed attempt to get MemberExpression working here).
My conclusion is: If there's a visit_* function for a type, then that type needs either:

to be an enum (which is Copy) or
to be enclosed in Box<_> or Vec<_> everywhere it's used (so that Box<T>/Vec<T> can be safely transmuted to &GCell<T>, which is what's passed to the visit_* function).

Unfortunately, that will involve quite a lot of changes to the AST.
NB: A variant on my failed attempt above should work in some cases, as long as the type is always in a fixed position in all other types (i.e. not within Option<_>), so a &GCell<T> will always point to a valid T forever, no matter how the AST is altered. But it's tricky, so I think better just box everything for now to get it working, and then try to scale the boxing back again later if it hurts performance.
I've added a commit to use layout_inspect on the AST, which allows running queries on the AST types:
node tasks/inspect_ast

Do have a look if you're interested.
Clocking off for tonight now, but will continue tomorrow.

PS: I also made some bug fixes and improvements to demo impl (the last 6 commits here: https://github.com/overlookmotel/ast-cell-test/commits/main/), which we should apply to this full implementation at some point.

All nested enums in the AST have now been squashed. So e.g. ArrayExpressionElement contains all the variants of Expression, rather than having its own variant ArrayExpressionElement::Expression(Expression<'a>). This was a hard slog (4 x 14-hour days) but, aside from cleaning up the macro madness, it's done.
As a side-effect, all the type sizes are back to what they were originally, and in fact a few got smaller.

I am pretty rinsed-out after going at it on #3115 all week. Can I ask for some help?
The last step of getting the AST in shape to be able to traverse over it is that a bunch of types need to be enclosed in Box<_> wherever they're used.
The types in question are:

AssignmentTargetRest
BindingIdentifier
BindingPattern
CatchParameter
Elision
Hashbang
IdentifierReference
JSXEmptyExpression
JSXIdentifier
LabelIdentifier
PrivateIdentifier
StringLiteral
TSImportAttributes
TSThisParameter
TSThisType
TemplateLiteral
WithClause

Anywhere they appear in the AST and they're not already wrapped in Box<_> or Vec<_>, they need to be. e.g.:
pub struct ObjectAssignmentTarget<'a> {
    pub span: Span,
    pub properties: Vec<'a, AssignmentTargetProperty<'a>>,
-   pub rest: Option<AssignmentTargetRest<'a>>,
+   pub rest: Option<Box<'a, AssignmentTargetRest<'a>>>,
}

If anyone is willing to lend a hand, please add commits to this PR directly (if you have write access) or submit as PRs onto this branch (not main).
To get an up-to-date list of types which are not yet sorted, run on this branch:
node tasks/inspect_ast

Rebased on main after #3115 merged.

Rebased on latest main.

A lot of work went into this, and I would like to save it somewhere. It's possible we want to come back to this later.
I'll save it in a branch and then will close again.

A lot of work went into this, and I would like to save it somewhere. It's possible we want to come back to this later.
I'll save it in a branch and then will close again.

Do you think we could close this for housekeeping reasons? It seems like this PR has drifted too far from our current trunk, So I guess what I'm trying to ask here is do you still believe that the code here can be useful at some point?

Yes sorry, I requested it be kept, and then forgot about it.
I've merged #3149 into this, and saved it on traversable-ast branch.
I don't know for sure if we'll need it, but there was a lot of interesting stuff in this PR, so might want to go back to it/pull some stuff out if it later on.
Closing this PR.


Consider exposing test results to Prometheus via /metrics endpoint

cc: @chentex @kurtismullins
see https://prometheus.io/docs/guides/go-application/#adding-your-own-metrics

We have a dashboard being created and finished at the moment.


Highfive crashes where a PR has no description

When a PR is opened but has no description (like https://github.com/rust-lang/rust/pull/56989), highfive crashes before assigning a reviewer or commenting.
Traceback (most recent call last):
  File "/home/ec2-user/highfive/highfive/newpr.py", line 433, in <module>
    handler.run()
  File "/home/ec2-user/highfive/highfive/newpr.py", line 71, in run
    self.new_pr()
  File "/home/ec2-user/highfive/highfive/newpr.py", line 359, in new_pr
    reviewer = self.find_reviewer(msg)
  File "/home/ec2-user/highfive/highfive/newpr.py", line 238, in find_reviewer
    match = reviewer_re.search(msg)
TypeError: expected string or buffer

This is caused by GitHub setting the body to null instead of an empty string where there is no PR description:
{
  "action": "opened",
  "number": 56989,
  "pull_request": {
    "number": 56989,
    "state": "open",
    "locked": false,
    "title": "Fix compiletest `trim` deprecation warnings",
    "user": {
      "login": "phansch",
      "id": 2042399,
      ...
    },
    "body": null,
    ...
  }
}

JFYI we often get highfive just ignoring PRs in the cargo repo.  But that might be for a different reason.

Yeah, there are probably other bugs. Ping me on #infra as soon as it happens, so I can fetch the logs and investigate the crash!


Update to promtail 2.6.1

It appears that grafana is building the promtail release binaries with journal support now so we attempt to use them again.
Still doesn't. But the one in the published image on dockerhub does so we're going to switch to that.

Ok I admit defeat. Back to building my own binary in a base image.


NCCL integration

Description
This PR provides new KVStore type with integration for NCCL communication library.
 I finished coding on this PR)
[ ] All changes have test coverage
[x] For user-facing API changes, API doc string has been updated.
[x] To my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[x] New nccl type of kvstore, using ncclReduce and ncclBcast
[x] test_nccl.py added to tests/python/gpu, but not enabled,  since NCCL is not present and enabled in CI

Comments

Interesting edge cases to note here:

NCCL KVStore requires the same set of devices to be used for all communications (as is the case in typical data parallel training)
in NCCL KVStore push and pull are implemented using 2 steps - launching NCCL kernels in 1 step and synchronizing in the second step. This was made to enable seamless aggregation support - several reductions are scheduled before a synchronization.

@mli @eric-haibin-lin

Thanks for addressing all these review comments. Is anyone helping you to setup the CI test with NCCL?

Not really, no.

@mbaijal We'll need your help setting up a new CI test with NCCL build after this is merged.

@eric-haibin-lin Please create an issue and mention me so I can keep track of this case. I've been thinking about adding a p2/p3-specific job to the Unit Tests - this would cover features which are unsupported by our usual CI-machiens.

Integrated last comments from @piiswrong and merged with 2-bit compression PR (NCCL does not support gradient compression and prints error message when trying to use it).

Is this ready to go in?

@cjolivier01 As far as I am concerned - yes, it is done. @piiswrong?

Yes, we decided to deal with the aggregation policy later


Reference the proper type in the Javadoc section.

This PR addresses the Javadoc issue that prevented artifact publishing. Wrong type has been referenced in the description of the type registry factory.
The library version is not changed along with the fix — as nothing has been modified except for a Javadoc portion.

@dmdashenkov PTAL.


URL validation inconsistency between SAML2 and OAuth/OpenID Connect configs

Description
When adding inbound authentication configurations for a service provider, the callback URL in the OAuth/OpenID is validated as a URL but the assertion consumer URL in the SAML2 does not.
Steps to Reproduce
Try to configure the following URL as both OAuth/OpenID callback URL and SAML2 assertion consumer URL, in the service provider inbound authentication configurations.
https://www.goo  gle.com/

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Can't resolve module with dot in its name

I'm having trouble using a module with a dot in its name. I've installed it, and it requires just fine in node on the command line, but not in a require 'array.from' line in a Coffee file which webpack is processing. I'm told:
Module not found: Error: Cannot resolve module 'array.from'
I can't see anything obvious wrong with the module, the way it installed, or its package.json file.


500 errors with no message

When trying to do several operations on the server, I get 500 errors with no message in the body:

This is preventing me from doing most things on the server. I tried adding APP_DEBUG=true to my .env but the only log in the storage folder is the laravel.log. No error log gets reported.
Edit: Just for kicks, I tried rebuilding with sqlite. No errors. Not sure if this helps figure out the issue. This does not mean it's fixed, as using a pg db is more reliable to me than sqlite.

Same issue here. Try to also enable debug in config/app.php (second line, 'debug' => true,) see if it helps, it didn't change anything for me, still can't figure out these random Error 500's.

Hello,
I've made severa search on this subject and I've possibly found the reason. If you look in the payload generated when you click on the "Add new offer" button, you can see the request is failing while parsing and creating the options for the select menu.

I've decided to see in the php files where this menu is generated, it's in microweber/userfiles/modules/shop/offers/edit_offer.php.
If you print out the values contained in the $a_price variable, you should noticed that the field values_plain don't exist but there is one with name as value.
By replacing this in the php file, you would be able to create offers.


Support for the comma as a decimal mark

Many countries uses the comma as the decimal mark (Spain, France, South Africa, French Canada, etc). GradebookNG should eventually support the comma as a decimal mark.

This would need to be a display only feature as I don't think the , is supported as a decimal mark in the database (at least in MySQL, Oracle may support it via the NLS settings). So it could be translated going in and then back out again.
ref: http://stackoverflow.com/questions/8669212/change-decimal-separator-in-mysql

It looks like in gradebook classic this is done in the app for both entry and display and it's stored always as a decimal in the database.
https://jira.sakaiproject.org/browse/SAK-17985

Yep and that's the simplest way to do it too - we can just pass the grade through a formatter before displaying it.

Closing this in favour of #3930.


add import-schema.sh/bat and export-schema.sh/bat in cli/tools

link https://github.com/apache/iotdb/pull/12564/


bintray latest version do not download latest version - download url for any automation script

I did this
bintray url for latest version do not work for this url ->https://bintray.com/vszakats/generic/curl/_latestVersion
I expected the following
when i click this url i always expect zip file to be download. Default 64 bit but need to have a way to to form the url for both 32(https://dl.bintray.com/vszakats/generic/:curl-7.71.1-win32-mingw.zip) and 64 (https://dl.bintray.com/vszakats/generic/:curl-7.71.1-win64-mingw.zip)
Ask his because we have an automation script that needs to download latest version of curl everytime. So that's why I'm placing this requst. Can you help understand the right way to download latest zip via an automation script
curl/libcurl version
[curl -V output]
operating system

The latest official curl for windows builds are provided at https://curl.haxx.se/windows/.

I forgot to update the Bintray page, sorry. Sadly it's a manual process (at least I haven't found a way to automate it.)
Anyhow as said already, the official download URL is: https://curl.haxx.se/windows/
The value of Bintray is to retrieve historical versions. (Plus it's an advertised download site for OpenSSL binaries.)


[SMALLFIX] Alluxio/new-contributor-tasks#434 Replace multi if with switch to make it more readable in alluxio/tests/src/test/java/alluxio/client/rest/TestCase.java

[SMALLFIX] Replace multi if with switch to make it more readable in alluxio/tests/src/test/java/alluxio/client/rest/TestCase.java

@bf8086 Does the check re-run after I changed the title?

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4024/
Test PASSed.

@jiacheliu3 Please do not include the Github issue id in the title. Your title should start with a verb from this list - https://raw.githubusercontent.com/Alluxio/alluxio/master/docs/resources/pr/pr_title_words.md
The title check will rerun every time you change your PR title.

alluxio-bot, merge this please.


Update glide and client code to use Kubernetes 1.16

Rebuild Helm using Kubernetes 1.16 go module.

Changed glide.yaml
Change couple of import path in cmd/client.go and cmd/client_test.go

What this PR does / why we need it:
Special notes for your reviewer:
If applicable:

[ ] this PR contains documentation
[ ] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

This is something we cannot merge at this time unfortunately. By upgrading to 1.16 client libraries, we lose the ability to interact with APIs that work in 1.15. Helm tries to maintain backwards compatibility of n-1 Kubernetes releases, so we need to wait for 1.17 before we can consider bumping client-go.


libuv: 1.34.1 -> 1.34.2

Motivation for this change
This should fix some flaky tests on darwin: https://github.com/NixOS/nixpkgs/pull/79372
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build libuv


[TUBEMQ-445]Adjust the status check default sleep interval of pullConsumeReadyChkSliceMs

Since the available status of the partition is also controlled by the server, if the detection sleep interval is too long, the partition consumption delay response will not be sensitive enough, so adjust the value to a smaller value.

Codecov Report

Merging #345 (014d9a0) into master (0d3f9c9) will decrease coverage by 0.09%.
The diff coverage is 0.00%.

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


Small fixes

fix: objective.value should return None if not in model
fix: get correct primal values from glpk MIPs
test: add test for obj value without model
fix: allow older versions of cplex
Numerous gurobi fixes

Fixes #77

Codecov Report

Merging #78 into master will decrease coverage by 0.01%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##           master      #78      +/-   ##
==========================================
- Coverage   88.67%   88.66%   -0.02%
==========================================
  Files           8        8
  Lines        2376     2381       +5
  Branches      540      542       +2
==========================================
+ Hits         2107     2111       +4
  Misses        167      167
- Partials      102      103       +1

Impacted Files
Coverage Δ

optlang/scipy_interface.py
85.15% <100%> (ø)
:white_check_mark:

optlang/interface.py
90.11% <75%> (ø)
:white_check_mark:

optlang/glpk_interface.py
85.55% <80.85%> (ø)
:white_check_mark:

optlang/cplex_interface.py
89.23% <94.73%> (-0.1%)
:x:

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 da0eca7...7872400. Read the comment docs.

fixes #74


[Outlook] (on send) Add support section

Docs Build status updates of commit 7700a03:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/outlook/outlook-on-send-addins.md
:white_check_mark:Succeeded

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

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

File
Status
Preview URL
Details

docs/outlook/outlook-on-send-addins.md
:white_check_mark:Succeeded

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


Added PR template for Celo Sage

@joenyzio  Added a PR template in the .github directory that all the Sage participants will use to create the PR for blogs. Feel free to make any changes you think are needed.

@joenyzio  Merge this request first to test the PR template flow and then merge #465

Fix typo "Upto" to "Up to" @viral-sangani

Included in https://github.com/celo-org/docs/pull/465


Fixed torch type checking

Enforce exact type matching for non-pattern strings. This fixes
false positives like a 'torch.LongTensor' object matching 'torch.Long'.
Also, a getmetatable() call was missing for the non-torch version
of env.type().

thanks!


Add discord webhook job

Should send a notification in the discord server whenever a PR is taken out of draft

bypass merging, this pytest job will fail until #12 is merged


连接oceanbase数据库表字段描述乱码

Description
连接oceanbase数据库表字段描述乱码

通过idea的database 插件可以正常显示表字段描述

DBeaver Version
Community <IP_ADDRESS>307091549
Operating System
windows
Database and driver
驱动包：oceanbase-client-2.4.3.jar
Steps to reproduce
No response
Additional context
No response

Hello @pengyong255237
Please translate your ticket text into English, including the title.

Hello @pengyong255237
Please translate your ticket text into English, including the title.

done

And what server version do you use?

Hello @pengyong255237,
Does the issue occur in the "DDL" tab? Please make sure to enable the "show comments" option.

Are you asking about the window version or the dbeaver version?

您使用什么服务器版本？

Are you asking about the window version or the dbeaver version?

你好@pengyong255237,
问题是否出现在“DDL”选项卡中？请确保启用“显示评论”选项。
The ER chart tab displays garbled code, and the DDL tab does not display the DDL SQL for field descriptions, as shown in the following figure

您使用什么服务器版本？

Are you asking about the window version or the dbeaver version?

After executing the update field description sql statement, only English alphabet and Arabic numerals will be displayed. Chinese is still garbled. The ddl tab does not display the field description at all

您使用什么服务器版本？

Are you asking about the window version or the dbeaver version?

After executing the update field description sql statement, only English alphabet and Arabic numerals will be displayed. Chinese is still garbled. The ddl tab does not display the field description at all

Hello @pengyong255237
On your screens, I see that you have a custom connection to OceanBase.
Did you try to connect with the supported DBeaver connector?

The custom connection through OceanBase cannot connect to my OceanBase database, and I must change the driver package for OceanBase https://github.com/oceanbase/obconnector-j The downloaded driver package can be connected, but it will not display comments. Please note that using this driver package will not display comments in dbeaver at all!
And then I started https://github.com/oceanbase/obconnector-j I downloaded the source code and modified the SQL for extracting data table field annotations in the source code, and created a new driver jar package. After connecting to the database through the new driver package, annotations appeared in the dbeaver, but they were all garbled,
Dbeaver comes with oceanbase driver

Driver package source code modification

What if you perform a select from user_col_comments, will it contain garbled characters in the result set?

如果执行 select from user_col_comments，结果集中会出现乱码吗？
Executing this SQL through the dbeaver tool can display it normally. Do you need to modify the parameters of the dbeaver tool?

Thank you. It looks like a UI encoding issue. We'll investigate this issue soon.

Probably will be fixed in scope of dbeaver/pro#1905

@pengyong255237
IF it is possible, please check our EA version https://dbeaver.io/files/ea/ to verify if this issue was fixed

@pengyong255237 如果可能，请检查我们的 EA 版本https://dbeaver.io/files/ea/以验证此问题是否已修
Okay, I'll try it now

@pengyong255237 如果可能，请检查我们的 EA 版本https://dbeaver.io/files/ea/以验证此问题是否已修复
Through the Early Access version, the Chinese description can be displayed normally. Thank you!

Thank you for checking.


ISPN-12925 doc updates and edits

https://issues.redhat.com/browse/ISPN-12925

integrated! thanks @oraNod


Fallout games support

Hello stumbled upon this and was wondering if there's any interest to get this working for Fallout series like 3/new vegas/ and 4? I have a steam deck and would love to get this working for these games. Happy to test or figure out how to get this working. I'm a systems engineer by trade so comfortable with making custom builds/ getting my hands dirty in linux.

DLSS2FSR is a DLSS to FSR emulator. It is not a mod to implement FSR in games.
To my knowledge, Fallout 4 (let alone 3) doesn't have any DLSS support.

Ahh sorry for the misunderstanding , thanks!


MoveSenseBindingiOS <IP_ADDRESS> Missing

Plugin.Movesense <IP_ADDRESS> expects on iOS MoveSenseBindingiOS <IP_ADDRESS>, but only <IP_ADDRESS> is available. Thus, currently, you can't use the latest version on iOS.

Apologies, my mistake. Just pushed the package to nuget.org, so will be fine now.


Saving my settings

Description of the new feature / enhancement
I would like to have the same setting after reinstalling powertoys or windows.
Scenario when this would be used?
Import my setting after reinstallation of the program.
Supporting information
No response

This is already possible. What version are you using?

Hello, I have already found this function on the very beginning home page!-)
))
thank you. b.
---------- Původní e-mail ----------
Od: Jay @.>
Komu: microsoft/PowerToys @.>
Kopie: verzich @.>, Author @.>
Datum: 11. 11. 2022 15:35:39
Předmět: Re: [microsoft/PowerToys] Saving my settings (Issue #22007)
"
This is already possible. What version are you using?
—
Reply to this email directly, view it on GitHub
(https://github.com/microsoft/PowerToys/issues/22007#issuecomment-1311769541)
, or unsubscribe
(https://github.com/notifications/unsubscribe-auth/APRMMRKLUKA2PZ7G4QO724TWHZKTNANCNFSM6AAAAAAR5UEVMM)
.
You are receiving this because you authored the thread. Message ID: <
@.***>
"


How I can login without Database?

Im using Laravel 5.3 and im trying to connect via LDAP using Adldap2-Laravel using the email without using database and model someone can help me?

This isn't possible at the moment unfortunately. I'm planning on implementing this though.


C# sample for AppWindow in WinUI 3 won't compile

IntPtr hWnd = WinRT.Interop.WindowNative.GetWindowHandle(window);
should be:
IntPtr hWnd = WinRT.Interop.WindowNative.GetWindowHandle(this);
Also, I find the choice of the variable name m_appWindow quite surprising because does not follow any Microsoft naming convention. _appWindow or appWindow would be a name that anyone expects.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: b2621305-73dc-26eb-cbd7-ed48c7bc0534
Version Independent ID: e18ea4cd-0750-0207-e2f8-c97ff97de697
Content: Manage app windows (Windows App SDK) - Windows apps
Content Source: hub/apps/windows-app-sdk/windowing/windowing-overview.md
Product: reunion
Technology: windowing
GitHub Login: @rkarman
Microsoft Alias: rokarman

Thank you for bringing this to my attention. The snippet was lifted from the WinUI sample, in which the window is defined. This is being addressed in PR #3332


Is there ignoring a log operation in SCL?

Hi HobbitLong:
Thank you for releasing code. Here, i have a question. Is there ignoring a log operation in 89 line of loss.py.
# compute log_prob
        exp_logits = torch.exp(logits) * logits_mask
        log_prob = logits - torch.log(exp_logits.sum(1, keepdim=True))

Hi, it's because the log and exp for the numerator get cancelled out.


An issue about dataset.

Thanks for your great work!
Are there any segmentation masks and depth in your dataset.

Thanks for your interest in our work, currently we do not have these in our datasets. But those are provided in Robosuite, so I believe if you re-rollout the dataset you can get the mask/depth from the simulator.

Thanks for your interest in our work, currently we do not have these in our datasets. But those are provided in Robosuite, so I believe if you re-rollout the dataset you can get the mask/depth from the simulator.

Thanks for your reply.
Can you tell me how to re-rollout the dataset？
Are there any tutorials ?

You can take a look at our README page. Basically, you choose a task, set the initial state, and then simulate the action, and then get the observation with depth/mask.

You can take a look at our README page. Basically, you choose a task, set the initial state, and then simulate the action, and then get the observation with depth/mask.

Thanks for your quick reply!
Best wishes!


Use GITHUB_ACTOR for CI_ORGANIZATION for Github action

@wmdietl Could you review this PR? Looks like this is related to https://github.com/eisop/checker-framework/pull/1013 and https://github.com/eisop/checker-framework/pull/862/.
The reason is CI_ORGANIZATION is set to eisop and echo About to run git-find-fork eisop eisop jdk. So the branch is not found.
In github action, we should use the following:
GITHUB_ACTOR=Ao-senXiong

@wmdietl Ping for review. I can not assign you in this org.


Reworked Readme and introduced Building doc page

Description

All in the title

Close #132
What type of PR is this? (check all applicable)

[ ] Refactor
[ ] Feature
[ ] Bug Fix
[ ] Optimization
[x] Documentation Update

[optional] Minor fixes

Renamed ENABLE_TEST_COVERAGE to ENABLE_COVERAGE and renamed default branch master to main in contributing.
Also reformated the build workflow.
Added/updated tests?

[ ] New tests written
[ ] Existing tests updated
[x] Tests are not required because this is a documentation update
[ ] I need help with writing tests

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 83.31%. Comparing base (8074dde) to head (f39afe7).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #138   +/-   ##
=======================================
  Coverage   83.31%   83.31%
=======================================
  Files          60       60
  Lines        1073     1073
  Branches      331      331
=======================================
  Hits          894      894
  Misses        121      121
  Partials       58       58

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


feat: define signal type for notifications

Hey, I just made a Pull Request!
Defines signal type for notifications. Also refactoring the web notification handling a bit + preparations for done removal.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@Rugvip good idea, have to think about that later!


Repot frame_support::traits; introduce some new currency stuff

Repots everything from frame_support::traits. Remains backwards compatible though.
Introduces some new traits which should close the gap between the Assets pallet style Fungibles and the Balances pallet style Currency.
TODO:

[x] Full docs
[x] Proper errors

Separate PR:

[ ] Implement fungible::* for Balances pallet.
[ ] Introduce fungibles::Lockable/fungible::Lockable and implement for Balances pallet.
[ ] Implement fungibles::Unbalanced for Assets pallet.
[ ] Introduce ReservableAssets derivative pallet and implement fungibles::Reserve.
[ ] Introduce LockableAssets derivative pallet and implement fungibles::Lock.
[ ] Swtich over trait requirements from Currency to fungible::* in pallets.
[ ] Deprecate Balances pallet, migrate to Assets pallet.
[ ] XCM for can_withdraw, can_deposit, balance, reserved_balance

Why do we need to deprecate Balances pallet?

We don't really. It's a possibility since Assets should be a functionally complete replacement, however I removed the action item since I think Balances will always be a bit faster owing to being able to amalgamate the storage accesses of the system nonce with the balance.

🚀


add auto-formatter (black, yapf, etc) to GH actions

as a collaborator on the garden project and a natural pedant, I dislike reading or writing code which is poorly-formatted, and the worst kind of formatting is the inconsistent kind. To ease the cognitive (and emotional) burden of reading or writing poorly-formatted code, we should add a GH action to format the code automatically before the repository is tainted
I currently use black locally, but would be happy to switch to yapf or something else more configurable to accommodate any strongly-held opinions.
Acceptance Criteria
Given proposed changes in a PR, when running GH actions (sometime before the flake8 check), the changes should be formatted by an auto-formatter such as black to be stylistically consistent with the rest of the codebase

@OwenPriceSkelly have you all decided on which format to use (looked like it might have been black)? Also would having a pre-commit hook for formatting be an option? I have seen the pre-commit formatting work well, as this led to fewer issues when formatting conflicts occurred if someone pushed and forgot to pull back in the linted/formatted code.

agreed! especially like the idea of having it as a pre-commit hook. Super open to alternatives, but at this point I think it's probably the path of least resistance to just use black if that's tolerable to everyone

Sounds good! If there is any way to share any current configs for the formatting that would be amazing; I know that when using flake8 and black there can be conflicting information on things like line length etc. Would love to know if there are any current examples of parameters you plan on using such as line length set to 79 or black's 88 or if we are ignoring any other formatting rules; I'm sure this is not a massive priority but would likely allow for us to get something set up in the actions workflow as well as document the standards for any future contributors.


stable: all matrix function bound with test case

This change is

Reviewed 7 of 7 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


helm: Remove chart fields planned for removal in 1.12

Remove chart fields planned for removal in 1.12
Please ensure your pull request adheres to the following guidelines:

[x] For first time contributors, read Submitting a pull request
[x] All code is covered by unit and/or runtime tests where feasible.
[x] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[x] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[x] All commits are signed off. See the section Developer’s Certificate of Origin
[x] Provide a title or release-note blurb suitable for the release notes.
[x] Thanks for contributing!

Fixes: #issue-number
<!-- Enter the release note text here if needed or remove this section! -->

The error report of ci seems to have nothing to do with the modification of this pr.

Could you give some help for the failed test, thanks? @sayboras

Can you rebase with master ?

Can you rebase with master ?

OK

Can you rebase with master ?

Thanks, all test has been pass


Request reserved namespace prefix "ibm"

On behalf of IBM, I kindly request to reserve the namespace 'ibm' for use in IBM produced public SBOMs.
Namespace to be reserved

Namespace
Description
URN equivalent

ibm
Namespace for use in IBM SBOM's  within property name field values
urn:ibm:names

Planned usage
Currently, we plan to use this as an alias (short form) for our current URN (i.e., urn:ibm:names) to add/associate domain specific information such as:

IBM product and deliverable IDs to component properties
additional scan/internal tooling/classification information to component properties
risk and confidence scoring information to component properties
legal (usage, coverage) disclaimers to metadata properties for customers

Goal: Simplification
Reduction in length of property name value:
{
"name": "urn:ibm:names:identifier:product",
"value": "5737-I23"
}

would be simplified to:
{
"name": "ibm:identifier:product",
"value": "5737-I23"
}

Note It is my hope we support URN syntax as well (and update the ABNF for package names correspondingly). See issue: https://github.com/CycloneDX/cyclonedx-property-taxonomy/issues/9

@coderpatros How do you want to handle urns?
IMO, the urn should be optional as to provide backward compatibility, but should be recommended. Thoughts?

I think we should reserve the usage of the "urn" namespace for URNs. I can update the ABNF to support both.

@coderpatros no need to update the ABNF, as long as the namespace is reserved, i guess. will open a PR soon.
regarding urn support in ABNF - this would be an own topic, right? see #9

@mrutkows ,
the Top-Level namespace ibm was reserved a while ago.
Could you add the respective taxonomy documentation?

@jkowalleck I really have no idea what a "published doc-space" mention in #109 is nor what taxonomy docs. are needed. Can you provide me references for what you are asking for?

@mrutkows ,
the namespace ibm was registered and RESERVED.
Reserving a namespace is okay for a while, but what purpose has a registered namespace, if it is unclear how it is used? To solve this, it is intended to publish the taxonomy behind that reserved namespace - usually in a doc-space - like the one for siemens https://github.com/siemens/cyclonedx-property-taxonomy#readme, or amazon's https://docs.aws.amazon.com/inspector/latest/user/cyclonedx-namespace.html.
Even an empty doc-space would be sufficient -- like dependency-track's https://github.com/DependencyTrack/cyclonedx-property-taxonomy.


Open database object not searching properly for names starting with upper case

Hi, Im trying to search for tables
like this:

I made two copies of table:
D0F15148F
and
d0F15148F
The first one not found again but for the second works properly.
So I think the problem is when the name of table starts with upper case

Yes, PG extension used case-sensitive search. Thanks for report.


log warn instead of error when user denied request

Log a warning instead of an error if the user denied the oauth request.

Hey @trungpham,

Is this still an issue for you?
Keeping in mind that this is technically a breaking change, what's the full reasoning for this?
Please rebase on Master if you'd still like to consider this.

Thanks


Unable to find the fontsDirectory

Hi I try to render my formulas but I get errors on console that says:

The files are present on my node_module

I tried to resolve this by adding on HTML tag :options={fontsDirectory='/src/node_modules/mathlive/dist/fonts' and testing multiples paths but no change.
The funny part is when I compare the original mathfield and the vue mathfield, the original works perfectly but I need to extract the data with v-model. The vueField is present and I can enter content but it will not render.

Here is my code:
<template>

            Mathfield (Original)
            <math-field
              virtual-keyboard-mode="manual"
              v-bind:value="question"
              v-on:input="question = $event"
            ></math-field>

Résulat : {{question}}

            Mathlive-Mathfield (Vue version) not render
            <mathlive-mathfield
              :options="{smartFence:false, fontsDirectory:'./fonts'}"
              v-model="question"
            ></mathlive-mathfield>

Résulat : {{question}}

</template>

You told me on #695 to check the fonts binds with firefox but I have the same errors on different browsers.
I think #23 refer to this errors too.

I would really like to help you but there just isn't enough information in the bug report for me to do that. I would need to have access to your code, either as a GitHub project or a CodePen in order to debug it.
The "rejected by sanitizer"error is produced by FireFox. If you see the same error in other browsers, I have no idea how that happens.
Have you checked that the font file was actually reachable? Does the URL "http://localhost:8080/js/fonts/KaTeX_Fraktur-Regular.woff" appear in your Network tab? If you paste this URL in the browser, does the fontfile get downloaded?
Is your server configured to accept WOFF (and WOFF2) files. (i.e. return the proper MIME types)

I tried to download by the URL but nothing. I will check on my server config.

Hello,
I did a sandbox with a basic example with Vue-mathlive . Just click here
And unfortunately the Vue Mathfield not render.

Thanks for this sandbox, that was very useful.
(Unrelated to this problem, but you should use v-on:input="formula = $event.target.value" to read the value of the formula)
Also not directly related, but the property to specify a custom directory for the fonts is fontsDirectory, not fonts. It's not directly related because the default value actually works (the fonts you use are stored in the node_modules/mathlive/fonts directory, and this directory is already adjacent to the mathlive.min.js library. So, you can safely remove the fonts option.
The problem is an interaction between codesandbox and FireFox. Firefox does some "font sanitizing"/validation and it appears that codesanbox is not properly setup to send 'woff'/'woff2' fonts in a way that Firefox accepts. This triggers an error (the downloadable font: rejected by sanitizer messages displayed in the console), and as a result the fonts are marked as "not loaded". This, in turns, result in the content of the mathfield appearing to be blank (although the content is not displayed, the mathfield is live and you can interact with it). In fact, despite Firefox complaints, the fonts are loaded. I have changed Mathlive to now assume that the font might be loaded, even if an error is received.
Why did the <math-field> display correctly? It displays correctly because to prevent the fonts from displaying when they are not loaded relies on a CSS selector set on the body of the document. This selector is not applicable when inside the dark shadow root of a custom element.
Your workarounds include:

using a browser other than Firefox
requesting that codesandbox serves woff and woff2 fonts to Firefox liking (they probably just need to add a MIMEType to their server configuration)
using <math-field> instead of <mathlive-mathfield> (is there any reason you would need `?)
updating to a future version of MathLive that will include the workaround to display fonts even if they appear to have failed to load

Thanks for your comments, it really helps me to understand and unlock my project!
I used the <mathlive-mathfield> because I thought it was the only way to use the v-model property on Vue.
I followed the ReadMe and did research on cortexjs.io but it wasn't enough.
And like I mentioned before, I tested the field also on chromium-browser. And I've found a similar issue:


Need to clarify distinction between multiple ALExhibitDocuments and multiple ALEXhibitDocument.exhibits[i]

We only have one lead document now, but we let people upload multiple documents within the lead_doc object.exhibits DAList.
This isn't quite right because all of the various attributes of the document are attached at the document level, not the individual exhibit level.

I think probably we can just limit folks to 1 exhibit. @BryceStevenWilley let me know your thoughts--not seeing anything in the certification tests that indicate we need to allow multiple documents.