Strange C macro generated by tensorflow.lite.python.util.convert_bytes_to_c_source()

1. System information
Although I believe the system setup does not matter in this issue, here goes:
Windows 10 and Ubuntu 22.10
Python 3.7, Python 3.8, Python 3.10
Tensorflow Lite 2.10 and 2.11 (pip package)
2. Code
tensorflow.lite.python.util.convert_bytes_to_c_source() generates the following strange C macro:
// We need to keep the data array aligned on some architectures.
#ifdef __has_attribute
#define HAVE_ATTRIBUTE(x) __has_attribute(x)
#else
#define HAVE_ATTRIBUTE(x) 0
#endif
#if HAVE_ATTRIBUTE(aligned) || (defined(__GNUC__) && !defined(__clang__))
#define DATA_ALIGN_ATTRIBUTE __attribute__((aligned(4)))
#else
#define DATA_ALIGN_ATTRIBUTE
#endif

The strange part is: defined(__GNUC__) && !defined(__clang__))
If the code is compiled with the GNU C compiler, then of course it is not compiled with the Clang compiler. The && !defined(__clang__) seems redundant. And if it's not redundant, can someone explain it to me?

Found the answer at https://stackoverflow.com/a/55926503/7268445


fixed_categories will use the same text as used to paint views. This …

…will avoid any issues related to encoding.

This PR resolves #648.


update opengov scores

Adjusts open gov and opengov delegation score to 0 so it's less confusing (they're already not included in the total, although still show via the api)

should we also align the api response, hard coding a 0 for retro compatibility sake, since opengov has no relevance anymore in this context ?

should we also align the api response, hard coding a 0 for retro compatibility sake, since opengov has no relevance anymore in this context ?

Would probably do a few things related to that in a follow up pr that also cleans things up and removes commented lines


Overwrite pyproject when workspaces have same normalized name

I have the following folder structure:
/private/tmp/super-project git:[main]
tree
.
├── README.md
├── pyproject.toml
├── super_project
│   ├── README.md
│   └── pyproject.toml
└── uv.lock

This is a common use case in my company. The repo name is super-project and the name of the python package we publish is super_project.
This is the content of the root pyproject.toml:
[project]
name = "super-project"
version = "0.1.0"
description = "Add your description here"
readme = "README.md"
requires-python = ">=3.11"
dependencies = [
    "super_project"
]

[tool.uv.workspace]
members = ["super_project"]

[tool.uv.sources]
super_project = { workspace = true }

[group-dependencies]
dev = [
    "pytest >=7.2.2",
]

This is the pyproject of the package:
[project]
name = "super_project"
version = "0.1.0"
description = "Add your description here"
readme = "README.md"
requires-python = ">=3.11"
dependencies = [
    "matplotlib"
]

When I run uv run pytest y get the next error:
error: Failed to spawn: `pytest`
  Caused by: No such file or directory (os error 2

And worse. If I try to do uv add pytest, my root pyproject gets overwriten with the pyproject of the package.
I guess the issue is related to https://github.com/astral-sh/uv/issues/8591

I got the same issue too.

I don't think you can have an outer and inner pyproject.toml with the same name — those names must be unique when normalized and we must normalize them for comparison per the Python standards.

Can you explain more about how it's overwritten? And why you're using this structure in the first place?

Yeah that's not a valid structure. It's like saying you want two packages on PyPI called foo_bar and foo-bar, and you want to be able to use them distinctly.

Not two packages. So for this example:
/private/tmp/super-project git:[main]
tree
.
├── super_project
│   ├── README.md
│   └── pyproject.toml
├── super_project_2
│   ├── README.md
│   └── pyproject.toml
├── README.md
├── pyproject.toml
└── uv.lock

I want only two packages published to Pypi: super-project and super-project-2.
I'd like the two folders to be uv workspaces. When I'm developing locally I'd like to have dependencies of both packages installed. Is there any way I can do this with uv?


HTML5 Link Prefetching:  Link prefetching is a browser mechanism, which utilizes browser idle time t...

HTML5 Link Prefetching:
Link prefetching is a browser mechanism, which utilizes browser idle time to download or prefetch documents that the user might visit in the near future.
https://developer.mozilla.org/en/link_prefetching_faq

This issue was imported from Google Moderator
Moderator votes: +5
Issue added by roberkules on 2012-06-12
To vote this issue up or down, simply include +1 or -1 in your comment.

Duplicate of #981 so moving half the votes there

+1


🍒 Cherry-pick "support for over-aligned allocations without C++17" stragglers

rdar://problem/92458692

@swift-ci Please test


Let the user specify a starting vector for eigsh.

Sometimes a good initial guess for an eigenvector is known and it is advantageous to use it within the eigensolver. This patch adds the keyword argument v0 to eigsh so that it behaves in the same way as in scipy.sparse.linalg.

Thanks, LGTM except minor comment. Could you add test code for v0 != None case?

I have added the requested test in the last commit.

/test mini

Hi, is there a pending step on my end to get this pull request merged? Please let me know if that's the case so I can help.

Hi @jmbr, sorry for the delay! Test failures are irrelevant, so let's proceed to merge.

Hey,
Thanks for the helpful discussion
In the eigsh function i can determine the number of eigenvectors to calculate.
how can i use v0 if i have a different initial guess for each of the k eigenvectors?


Managed Tokens-Related Changes

This issue/document follows a large discussion with @DrDaveD, who maintains htgettoken and condor_vault_storer, both of which we use to get credentials.
Managed Tokens Idea
We want to have a script/service running on a crontab that generates vault tokens for production users (e.g. dunepro, mu2epro, etc.).  It will then transfer these tokens to the experiments' submit nodes, from which they can run jobsub_submit and other grid tools.
Issue with extra authentication
While testing small-scale versions of this service for DUNE, Ken Herner and I have found that we, acting as simulated end-users, often have to reauthenticate via CILogon website and our SERVICES credentials to obtain a bearer token.  The way that htgettoken works, since we already would have a valid vault token, we should not have to authenticate to get a bearer token from the vault.
Root cause
Currently, our Managed Tokens script uses a special Kerberos group principal and invokes condor_vault_storer, which generates two copies of the vault token:

/tmp/vt_u$(id -u)
/tmp/vt_u$(id -u)-SERVICE where SERVICE is simply a concatenation of issuer_role.  So the full path would be something like /tmp/vt_u12345-dune or tmp/vt_u12345-dune_production

Crucially, condor_vault_storer looks in the second location to figure out whether or not it needs to re-generate a vault token and store it in the condor_credmon.  This will become important shortly.
One of the first actions jobsub_submit does is to run htgettoken (via get_creds()):
https://github.com/marcmengel/jobsub_lite/blob/250859b2c8066d375d398bf0850bbbcbc22a86c9/bin/jobsub_submit#L174
By default, htgettoken will only create the first copy of the vault token above.  Thus, when we go to submit the job, condor_submit, invoked by jobsub_submit, runscondor_vault_storer, which will not see a vault token in the second location and will try to regenerate another vault token, even though one exists in the first location.
Further complication
In the troubleshooting of this issue, Ken and I would often regenerate the vault token manually by rerunning the Managed Proxy protoype script.  This script currently clears the kerberos cache of our user kerberos ticket, and puts the special group kerberos ticket into the cache.  If we then try to run jobsub_submit, this causes the obtaining of an X509 proxy to fail.
Solution, and the work item for this issue
We think we can solve both issues by simply having jobsub_lite run condor_vault_storer with the appropriate argument instead of htgettoken on  https://github.com/marcmengel/jobsub_lite/blob/master/lib/fake_ifdh.py#L76.  This would result in condor_vault_storer being invoked twice on each condor_submit (first manually, second automatically by condor_submit), which is OK (we currently similarly invoke htgettoken twice).
The Managed Tokens service would run as is, generating both vault tokens, and transferring them both to the end user's submit machine.  When the user then runs jobsub_submit:

For an Analysis user, condor_vault_storer will get the correct vault and bearer tokens, and when condor_submit is invoked, this second invocation of condor_vault_storer will see the correct vault token and proceed.
For a production user, the first invocation of condor_vault_storer will see that the vault tokens are already in place (due to the Managed Tokens service), and simply get the bearer token.  The second invocation of condor_vault_storer will simply do the same as in the Analysis case.

Additional Notes

The kerberos ticket the end user has (their own) when logging into the production account should not matter for the purpose of getting a token.  The way I describe the use case here, the vault token along with the correct credkey passed to $HTGETTOKENOPTS should be sufficient for authentication to get the proper bearer token.
The Managed Tokens service should possibly NOT create the bearer token when it runs condor_vault_storer.  This can also be done by passing --nobearertoken in $HTGETTOKENOPTS.
I tested the end-effect of doing this using fifeutilgpvm01 and my dev node, fermicloud525.  On fifeutilgpvm01, I ran the Managed Tokens prototype script, and transferred the files to fermicloud525 with the same paths.  I then logged into fermicloud525 as dunepro, and after setting up the environment variables ($HTGETTOKENOPTS and $_condor_COLLECTOR_HOST, was able to submit jobs using those vault tokens.
We should explore reverting to the Managed Tokens service running as a service account (rexbatch) and operating from there, rather than needing to su as the various users.
For the case of managed tokens only (We'd have to figure out how to discern that), @DrDaveD also suggested having --noidc added to the HTGETTOKENOPTS for jobsub_submit.

Managed tokens prototype is complete.  See https://github.com/shreyb/managed-tokens

Note that this is no longer a requirement for managed tokens, since the workaround of having managed tokens simply deposit two copies of the vault token works fine.  We can push this off after some discussion with Dave D about how necessary this actually is.

@DrDaveD - is there any reason we should keep thinking about moving jobsub_lite to using condor_vault_storer as opposed to what we currently do (htgettoken)?  Currently, we set BEARER_TOKEN_FILE and call htgettoken, and the subsequent condor_submit calls condor_vault_storer if necessary.  This works for our current setup, since BEARER_TOKEN_FILE is respected by both utilities.
The other good thing about using htgettoken in our standard token code is that we don't actually have to store the vault token unless we're doing a submit (i.e. jobsub_q, jobsub_fetchlog, etc. don't have to actually store a token in the condor credd).

Closing this.  We can reopen if it comes out that we need to change htgettoken calls to condor_vault_storer

It still might make a difference for people who do not have kerberos, I think.  For them I think they will need to do OIDC authentication twice at the beginning of each week, once for the original htgettoken and again for the condor_vault_storer that condor_submit calls.


[UIExplorer] Fix missing keys in animation example

There were two warnings about missing keys, fixed them.

@facebook-github-bot shipit

Thanks!


Clarity in Flexbox Documentation

I hope this is the right place to report an issue in the documentation. I am almost driving myself crazy trying to search for what "p-2" does.
I'm not a CSS developer. but I'm somewhat familiar with Bootstrap 3, and I'm going through the flexbox page in Bootstrap docs, and the class "p-2" has been applied all over. https://v4-alpha.getbootstrap.com/utilities/flexbox/
A Google search unfortunately won't help me find this.
Can somebody enlighten me, and can somebody add what it means to the documentation page? It would be ideal if every class in the documentation code was a clickable link that took the user to the relevant page.
I deeply apologize for annoying the contributors of this repo with this seemingly trivial problem, and advertising my ignorance and lack of expertise in Bootstrap.
Thanks a lot,
Samir.

p-2 is a custom Bootstrap class that is part of the spacing utilities. See docs. It sets a padding of .5rem.
For help/support of this kind, you should use StackOverflow or similar - see https://github.com/twbs/bootstrap/blob/master/CONTRIBUTING.md#using-the-issue-tracker

A Google search unfortunately won't help me find this.

As mentioned, it's one of our padding utilities. Next time, search the project's CSS instead of Googling for it :). No snark intended, but if it's a class that appears in our code snippets, there's a near 100% chance it's in our code.


Added Number of Divisors in Java | GSSOC21

Have you read the Contributing Guidelines on Pull Requests?
YES
Description
Given a Number. Find it's Number of Divisors.
The Number can be big such as 10^6 to 10^9
So, If we do this in O(N) we will get Time Limit Exceeded [TLE]
Better Approach is run a loop till sqrt(Number)
Example square root of 10^9 is 31622 (approx), so iteration optimized a lot.
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
Fixes #6076

Please review , it's been 4 days
@ankitaggarwal23 @iamrajiv @HarshCasper


WFST add/edit feature Field Validation!!

I am using WFS-T component to add new features to my layer in geoserver (zip code , address, ...)
by fault user inserted a character in zip code which is an integer field . when user click add button no validation on field type ,  the WFST Request return exception error parsing string and no message shown to user about the error
note : I am using the latest version

Right we currently have no client-side validation

Closing 1.x related tickets for now, since they are unlikely to be worked on


Add Kotlin CheatSheet

I want to do this, please assign me this!!

Sure, You can start working :)


Allow different implementations of the HUBViewController

This is the first step to us allowing the HUBViewControllerFactory to return different implementations of HUBViewController.
HUBViewController has essentially become an "abstract" class with no implementation. The "real" implementation has moved to HUBViewControllerImplementation which is a renamed version of the "old" HUBViewController.
We considered making HUBViewController a @protocol again, but felt that it would be too much of a change. It would also be a breaking API change which we're trying to avoid. There are also some categories in our internal Spotify use which would require rework.

@mhallendal

Codecov Report

Merging #307 into master will decrease coverage by 0.3%.
The diff coverage is 90.48%.

@@            Coverage Diff             @@
##           master     #307      +/-   ##
==========================================
- Coverage    93.6%   93.29%   -0.31%
==========================================
  Files          74       75       +1
  Lines        5096     5118      +22
==========================================
+ Hits         4770     4775       +5
- Misses        326      343      +17

Impacted Files
Coverage Δ

sources/HUBViewController.m
0% <0%> (-90.94%)
:arrow_down:

sources/HUBViewControllerFactoryImplementation.m
100% <100%> (ø)
:arrow_up:

sources/HUBViewControllerImplementation.m
90.99% <90.99%> (ø)

sources/HUBUtilities.h
72.91% <0%> (-1.05%)
: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 4538cc5...1f4d6c3. Read the comment docs.

@spotify/objc-dev We're planning on eventually moving away from the existing HUBViewController (now renamed to HUBViewControllerImplementation) and towards a new implementation written from the ground up.
We'll soon support dynamic instantiation of different subclasses of HUBViewController based on some HUBConfig (which is coming in a later PR).

+1

I like the sound of HUBViewController as a @protocol again. Something to consider for 3.0.0?

@marmelroy : There are arguments for and against it as a protocol, but we're definitely thinking about it.


add cells in lungmap ontology not in CL

see https://github.com/mapping-commons/lmha-obo-mappings/issues/1

id
name
category

LMHA:00243
myofibroblast
Cell

LMHA:00253
neuroepithelial body
Cell

LMHA:00644
general visceral afferent nerve
Cell

LMHA:00630
membranous bronchiolar epithelial cell
Cell

LMHA:00629
respiratory bronchiolar epithelial cell
Cell

LMHA:00582
terminal bronchiolar epithelial cell
Cell

LMHA:00066
alveolar interstitial cell
Cell

LMHA:00524
intermediate airway progenitor cell
Cell

LMHA:00816
peripheral blood lymphocytes
Cell

LMHA:00356
telocyte
Cell

LMHA:00575
immunoblast
Cell

LMHA:00386
unclassified fibroblast
Cell

LMHA:00598
proliferative interstitial cell
Cell

LMHA:00614
lymphoid cell
Cell

LMHA:00615
tingible body macrophage
Cell

LMHA:00238
mucus cell
Cell

LMHA:00209
lipofibroblast
Cell

LMHA:00213
interstitial macrophage
Cell

some of these are obviously missing synonyms - e.g. muc(o)us (secreting) cell. I suggest we add syns rather than tune NER

hmm, my list is incomplete, as some LMHA classes that are cells are actually in the structures branch

id
name
category

LMHA:00378
subendothelial connective tissue cell
AnatomicalSystem

LMHA:00146
collecting duct cell
AnatomicalStructure

LMHA:00694
submucosal gland epithelial cell
AnatomicalStructure

LMHA:00693
submucosal gland duct epithelial cell
AnatomicalStructure

Hi @cmungall ,
Wouldn't LMHA:00386 'unclassified fibroblast' be better as mapped to CL:0000057 'fibroblast', rather than as a new CL term?
Thanks,
Paola

A note on LMHA:00356 "telocyte", I've added a NTR for it a while ago, but never got to add it:  https://github.com/obophenotype/cell-ontology/issues/1048
There is also one spreadsheet for it as one example of simple MIANCT sheet:
https://docs.google.com/spreadsheets/d/1pr0CRyNIMtTqtg3WaF82SnttlYmMMWBsSmhSQlICqbs/edit#gid=0

@JoshuaFortriede  - could you have a go at making some more specific term requests based on Chris' analysis?  Please let me know if you need to chat about it - or add it to the CL editor's meeting agenda.


python311Packages.pygithub: 1.59.0 -> 1.59.1

Diff: https://github.com/PyGithub/PyGithub/compare/refs/tags/v1.59.0...v1.59.1
Changelog: https://github.com/PyGithub/PyGithub/raw/v1.59.1/doc/changes.rst
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 249052 run on x86_64-linux 1

28 packages built:

azure-cli
azure-cli.dist
cligh
cligh.dist
ghrepo-stats (python310Packages.ghrepo-stats)
ghrepo-stats.dist (python310Packages.ghrepo-stats.dist)
mathlibtools (python310Packages.mathlibtools)
mathlibtools.dist (python310Packages.mathlibtools.dist)
python310Packages.criticality-score
python310Packages.criticality-score.dist
python310Packages.howdoi
python310Packages.howdoi.dist
python310Packages.keep
python310Packages.keep.dist
python310Packages.pygithub
python310Packages.pygithub.dist
python311Packages.criticality-score
python311Packages.criticality-score.dist
python311Packages.ghrepo-stats
python311Packages.ghrepo-stats.dist
python311Packages.howdoi
python311Packages.howdoi.dist
python311Packages.keep
python311Packages.keep.dist
python311Packages.mathlibtools
python311Packages.mathlibtools.dist
python311Packages.pygithub
python311Packages.pygithub.dist


keep-alive is buggy in 8.0.1 with https

looking if https://github.com/apache/trafficserver/issues/3502 got better it turned out things are much worser in 8.0.1 - with more than 300 requests "ab" don't finish at all while keep-alive is no problem without https
Concurrency Level:      150
Time taken for tests:   0.182 seconds
Complete requests:      300
Failed requests:        150
(Connect: 0, Receive: 0, Length: 0, Exceptions: 150)
Keep-Alive requests:    150
Concurrency Level:      150
Time taken for tests:   0.120 seconds
Complete requests:      150
Failed requests:        4
(Connect: 0, Receive: 0, Length: 0, Exceptions: 4)
Keep-Alive requests:    86
[builduser@buildserver:/rpmbuild/SPECS]$ ab -c 150 -n 400 -k https://proxy.test.rh:8443/rte/upload/logo.gif
This is ApacheBench, Version 2.3 <$Revision: 1843412 $>
Copyright 1996 Adam Twiss, Zeus Technology Ltd, http://www.zeustech.net/
Licensed to The Apache Software Foundation, http://www.apache.org/
Benchmarking proxy.test.rh (be patient)
Completed 100 requests
Completed 200 requests
Completed 300 requests
apr_pollset_poll: The timeout specified has expired (70007)
Total of 300 requests completed

Yes, we ran into this as well, and I've already reverted the offending commit. So, latest 8.0.x branch should not experience this problem any more. It'll go into 8.0.2 as well.

The commit to apply to 8.0.1 is f6fe5f0079005813d33d1cddd9d127c75a1f5e0b.


where is the linux support Docker 0.7.0 Alex Larsson

Problem description
Look for your favorite distro page is missing

Problem location

I saw a problem on the following URL: https://docs.docker.com/en/latest/installation/

I couldn't find the information I wanted. I expected to find it near the following URL: https://docs.docker.com/en/latest/installation/

Other: be more linux friendly

Project version(s) affected

Suggestions for a fix

That's a very very old location for the installation page. We should add a redirect for that
Curious though; where did you find a link to that URL?

ah; looks to be in this blog post https://blog.docker.com/2013/11/docker-0-7-docker-now-runs-on-any-linux-distribution/

yeah thats the one. When i pacman -S docker am I using moby or docker ce? and why does the website only link windows and mac, this whole thing is based off linux containers

The current version of the installation page can be found here; https://docs.docker.com/install/#supported-platforms
The community edition of Docker Engine is supported on Ubuntu, CentOS, Debian, and Fedora.
Docker Engine Enterprise is supported on Ubuntu, RHEL, CentOS, SLES, Oracle Linux, and Windows
Other platforms (e.g.  Debian and Ubuntu forks such as Linux Mint) may work, but are not tested.
In addition, there are statically compiled binaries (https://docs.docker.com/install/linux/docker-ce/binaries/) which may work for unsupported platforms (but have some limitations, as some features require dynamic linking)

Opened a pull-request to add a redirect for the old location; https://github.com/docker/docker.github.io/pull/8269

ok im on arch linux and it works like butter pacman -S docker but I must say you make it look like docker is only for windows and mac

ok im on arch linux and it works like butter pacman -S docker

I guess that works; with a note that those likely are the distro-maintained packages (not the official docker packages).
Arch linux is fast moving distro (they keep up with latest packages), which unfortunately resulted in docker being broken frequently. Because of that, it was removed from the list of distros that Docker provides packages for.

but I must say you make it look like docker is only for windows and mac

Can you explain this a bit more? The installation page https://docs.docker.com/install/#supported-platforms shows the platforms that Docker can be installed on which includes Linux, but also macOS and Windows.
Happy to hear what changes you'd like to see, but trying to understand what led you to that conclusion.

go to docker.com and click "get started"


Node version mismatch

I'm trying to install this module, but am failing because of node version mismatches.
I am using v10.6.0, but wsfetch requires 7.6.0. So I install v7.6.0 - but dexterous requires 10.0.0.
Can wsfetch be updated?

@thetre97 I am not sure what you would like us to do since we don't control wsfetch. Do you mean you wish us to update the version of wsfetch specified in the dexterous package.json file? We have wsfetch working in a v10.. environment. The dexterous package is working on our test machines, which are now at v10.6.0 also.

Ah okay, sorry.
I just assumed you did since it is under the same namespace. (https://github.com/anywhichway/wsfetch)
I will raise a PR in that repo.

@thetre97 engine ref updated.

No worries, thanks @anywhichway  :+1:


GPU metics agent is broken on 2.0+ images because of google-cloud-monitoring library update

Drafted PR with the fix: https://github.com/GoogleCloudPlatform/ml-on-gcp/pull/117
Probably we will need to use different versions of for 2.0+ and pre-2.0 images.
Repro:
bazel test --jobs=25 --local_test_jobs=25 --test_output=errors --nocache_test_results \
  --test_arg=--image_version=preview --test_arg=--skip_cleanup=false gpu:test_gpu

This is fixed, we can close this I believe.

This is fixed, we can close this I believe.

Thanks for fixing this!

Thanks for fixing this!


Improve output documentation regarding retries and brokers

We use http_server as input and http_client as one of outputs (for a part of message batch). In case when there is some error coming from http_client, benthos starts to retry this error message indefinitely (#415). However most significant, is that it stops accepting other, normal messages.
Here is the log when I first try to send message which causes http_client to get 500 error, and then try with the message that should not cause 500 error from the endpoint that http_client connects to:
>benthos.exe -c config-addons.yaml
{"@timestamp":"2020-03-25T14:56:29+03:00","@service":"benthos","level":"INFO","component":"benthos.input","message":"Receiving HTTP messages at: http://:8000/api/v1/image-upload/"}
{"@timestamp":"2020-03-25T14:56:29+03:00","@service":"benthos","level":"INFO","component":"benthos","message":"Launching a benthos instance, use CTRL+C to close."}
{"@timestamp":"2020-03-25T14:56:29+03:00","@service":"benthos","level":"INFO","component":"benthos.output.broker.outputs.0","message":"Sending messages via HTTP requests to: http://<IP_ADDRESS>:9000/internal/v1/save-data/"}
{"@timestamp":"2020-03-25T14:56:29+03:00","@service":"benthos","level":"INFO","component":"benthos.output.broker.outputs.1","message":"Writing message parts as files."}
{"@timestamp":"2020-03-25T14:56:29+03:00","@service":"benthos","level":"INFO","component":"benthos","message":"Listening for HTTP requests at: http://<IP_ADDRESS>:4195"}
{"@timestamp":"2020-03-25T15:06:50+03:00","@service":"benthos","level":"TRACE","component":"benthos.input","message":"Consumed 2 messages from POST to '/api/v1/image-upload/'."}
{"@timestamp":"2020-03-25T15:06:50+03:00","@service":"benthos","level":"TRACE","component":"benthos.output.broker.outputs.1","message":"Attempting to write 1 messages to 'files'."}
{"@timestamp":"2020-03-25T15:06:50+03:00","@service":"benthos","level":"TRACE","component":"benthos.output.broker.outputs.0","message":"Attempting to write 1 messages to 'http_client'."}
{"@timestamp":"2020-03-25T15:06:50+03:00","@service":"benthos","level":"TRACE","component":"benthos.output.broker.outputs.1","message":"Successfully wrote 1 messages to 'files'."}
{"@timestamp":"2020-03-25T15:06:57+03:00","@service":"benthos","level":"ERROR","component":"benthos.output.broker.outputs.0","message":"Failed to send message to http_client: HTTP request returned unexpected response code (500): 500 Internal Server Error"}
{"@timestamp":"2020-03-25T15:06:57+03:00","@service":"benthos","level":"ERROR","component":"benthos.output","message":"Failed to dispatch fan out message: HTTP request returned unexpected response code (500): 500 Internal Server Error"}
{"@timestamp":"2020-03-25T15:06:57+03:00","@service":"benthos","level":"TRACE","component":"benthos.output.broker.outputs.0","message":"Attempting to write 1 messages to 'http_client'."}
{"@timestamp":"2020-03-25T15:07:31+03:00","@service":"benthos","level":"TRACE","component":"benthos.input","message":"Consumed 2 messages from POST to '/api/v1/image-upload/'."}
{"@timestamp":"2020-03-25T15:07:51+03:00","@service":"benthos","level":"ERROR","component":"benthos.output.broker.outputs.0","message":"Failed to send message to http_client: Post \"http://<IP_ADDRESS>:9000/internal/v1/save-data/\": EOF"}
{"@timestamp":"2020-03-25T15:07:51+03:00","@service":"benthos","level":"ERROR","component":"benthos.output","message":"Failed to dispatch fan out message: Post \"http://<IP_ADDRESS>:9000/internal/v1/save-data/\": EOF"}
{"@timestamp":"2020-03-25T15:07:51+03:00","@service":"benthos","level":"TRACE","component":"benthos.output.broker.outputs.0","message":"Attempting to write 1 messages to 'http_client'."}
{"@timestamp":"2020-03-25T15:08:23+03:00","@service":"benthos","level":"INFO","component":"benthos","message":"Received SIGTERM, the service is closing."}

According to logs, second portion of message batch is received, but it's not processed and client (which sends to input) sees timeout.
http_server:
  http_server:
    address: :8000
    path: /api/v1/image-upload/
    timeout: 10s
    rate_limit: ""

http_client:
      - http_client:
          # retry_period: 10s
          max_retry_backoff: 30s
          # retries: 3
          backoff_on:
            - 429
            - 500
          url: http://${MONGOD_ADDR:<IP_ADDRESS>:9000}/internal/v1/save-data/
          verb: POST
          headers:
            Content-Type: application/json
            X-Image-ID: ${!metadata:image_id}
          rate_limit: ""
          timeout: 5s
          max_in_flight: 1
          batching:
            count: 1
            byte_size: 0
            period: ""
        processors:
          - select_parts:
              parts: [0]

Hey @inliquid, Benthos never drops messages unless explicitly told to. The http_client output has a discrete number of retries, but once those retries are exhausted the attempt is considered failed but Benthos doesn't have any other options for the message, and so it will try again.
Eventually new messages will stop being consumed whilst the failed message is blocking, this is intended as the back pressure prevents Benthos from consuming unlimited memory and claiming messages it cannot process.
In your particular case it sounds like you want to simply drop messages if they failed, which you can do with a try broker where you follow it with a drop:
output:
  try:
    - http_client:
        max_retry_backoff: 30s
        retries: 3
        backoff_on:
          - 429
          - 500
        url: http://${MONGOD_ADDR:<IP_ADDRESS>:9000}/internal/v1/save-data/
        ... etc ...
    - type: drop

Hi @Jeffail thanks, will this work with processor section attached to the http_client output or should I add this to try somehow?
I think it's not very well documented as for me there is no way to understand that in order for these parameters to work you have to use some additional complicated try setup: https://www.benthos.dev/docs/components/outputs/http_client/#retries

Yeah I think the output about page needs a bit of an update, ideally it should cover all of these patterns with config examples.
Add your processors to the http_client output like before:
output:
  try:
    - http_client:
        max_retry_backoff: 30s
        retries: 3
        backoff_on:
          - 429
          - 500
        url: http://${MONGOD_ADDR:<IP_ADDRESS>:9000}/internal/v1/save-data/
        ... etc ...
      processors:
        - select_parts:
            parts: [0]
    - type: drop

Thank you, what about max_in_flight parameter? Does it affect in any way fact that until retries either succeed or fail, while pipeline is blocked?

If you're consistently attempting payloads that fail without dropping them then eventually the pipeline will be throttled by the constant retry attempts. If you expect certain payloads to always cause 500 and want to quickly drop them rather than block the pipeline then add the status code 500 to the field drop_on.


Error: Cannot find module '../build/Release/bson']

Might be an oldie, but on sails lift, I get the following error:
{ [Error: Cannot find module '../build/Release/bson'] code: 'MODULE_NOT_FOUND' }
js-bson: Failed to load c++ bson extension, using pure JS version

I saw this: https://github.com/rschmukler/agenda/issues/200 but could not figure out a solution for me.
Any workaround or fix?
Thank you,
Manuel

sails.js 0.12.3, 0.12.4
sails-mongo 0.12.1
npm 2.15.5

This a problem with the Mongodb module.  it's work anyway.                                                                                Envoyé depuis BlackBerry Hub pour Android                                                                                De:notifications@github.comEnvoyé:14 août 2016 3:14<EMAIL_ADDRESS>à:reply@reply.github.comObjet:[vbuzzano/sails-hook-jobs] Error: Cannot find module '../build/Release/bson'] (#9)  Might be an oldie, but on sails lift, I get the following error:
{ [Error: Cannot find module '../build/Release/bson'] code: 'MODULE_NOT_FOUND' }
js-bson: Failed to load c++ bson extension, using pure JS version
I saw this: rschmukler/agenda#200 but could not figure out a solution for me.
Any workaround or fix?
Thank you,
Manuel
sails.js 0.12.3, 0.12.4
sails-mongo 0.12.1
npm 2.15.5
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.


Confusing

I was under the impression that the verification code was in lieu of having a Microsoft account, not in addition to it. According to paragraph two, the recipient of a sharing link must log into their MS account, then enter the verification code, then log into their account again? Also, I'm not clear on the difference between how guests were added previously (a partial guest?)  vs. the new way--as a "full" guest. Can you please clarify these concepts?

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

ID: f75d2b5a-bc14-e0f3-1269-716383939212
Version Independent ID: 879c8636-ab9a-0440-039a-f3eac8d41a1f
Content: Secure external sharing in SharePoint - SharePoint Online
Content Source: SharePoint/SharePointOnline/what-s-new-in-sharing-in-targeted-release.md
Service: sharepoint-online
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@jmuehlenbein if the user does have an Office 365 account, they'll be asked to use the passcode the first time the log in, and thereafter can use their regular user name and password. If they don't have an Office 365 account or other Microsoft account, then they can just use the passcode and don't need to get a Microsoft account.
The word "full" is perhaps misleading. It's just a guest account in Azure AD for users of Microsoft or Office 365 accounts. If you set up the SharePoint and OneDrive integration with Azure AD B2B preview, then passcode users who do not have a Microsoft or Office 365 account will also get a guest account in Azure AD.

@jmuehlenbein Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Move the data that's in the guess function to a database, to better demonstrate more Cloudflare features.

Currently there's a load of standing static data about stations in the guess function code.  Move this to a database on Cloudflare to explore how that part of the product works.  This is a nice to have as it doesn't particularly improve the operation of the game or add new functionality / scaling capabilities.

Could add some value by using the database to count how many times each station's been guessed and showing that to the users as a sort of league table of how commonly known the station is...


router: return 503 when cluster isn't found.

Fixes #1820.

Quick ping on the status of this PR.

@htuch per @mattklein123's request, I'm going to make this configurable (even though I disagree that 404 is a valid response code here)... either tonight, or most likely tomorrow (I want to finish #1984 first).

@PiotrSikora I'm going to go ahead and close this for now. Please reopen when we are ready w/ the config changes.

@mattklein123 could you reopen this, or should I create new a PR?

Reopening

@mattklein123 @rshriram this is now configureable and ready for review.


docs(plugins) new aws-lambda plugin docs

Closing this in favor of #330 that has been merged. Thanks again.


Frame color in mainwindow example

https://github.com/wangwenx190/framelesshelper/blob/0bfefb25d0c28ec792d48e0b9e6c969361d8683b/examples/mainwindow/mainwindow.cpp#L139
Why don't you use the frame color here?
painter.setPen({Utilities::getNativeWindowFrameColor(isActiveWindow()), 1});

Yes you are right. There are some other small issues that need to be fixed, I'll push new code tomorrow to fix them all.

@wangwenx190 despite the latest commits, the border is always grey

OK. Let me check. First glance of the code doesn't find anything wrong.

I agree, the code looks fine. Please note this is tested on win11, I can't test win10 right now.

I can test Win10 and Win7 tomorrow. Currently I'm using a Win11 PC as well.

Tested on both Win11 and Win10. The border color is normal on Win10, but on Win11 it is always gray indeed. However, I tested multiple Qt-based applications on Win11 and found that it seems it's a common issue of Qt-based applications, maybe it's a bug of Qt itself, currently still not sure, need further verify and investigate.

It seems the gray border we see on Win10 is not the one drawn by framelesshelper.

Yes, looks like it's the standard window frame provided by the OS. And I found that some things have changed on Windows 11. Previously I was able to get rid of the whole window frame, but now on Win11 it's no longer possible. No matter what I do, Windows will always add a thin frame border to the window, that's what we observed from the MainWindow example, the gray border. But there's one thing I still don't understand, despite of the window frame, we also draw our own frame border in the paint event, but on win11 it can't be seen anywhere. Maybe it's covered by something? Not sure yet.

Could have something to do with the way they implemented round corners?
Despite the grey color, it works quite fine on win11, I've not noticed any bug yet.

Could have something to do with the way they implemented round corners?

Not sure, I tested this just now, I use DWM api to change the window frame corner to square, the issue still exist.

Use the following code if you want to try:
const DWM_WINDOW_CORNER_PREFERENCE dwcp = DWMWCP_DONOTROUND;
DwmSetWindowAttribute(hwnd, DWMWA_WINDOW_CORNER_PREFERENCE, &dwcp, sizeof(dwcp));

Note that this is only supported on Win11, use this enum on Win10 will fail.


Register Campus Partner - Limiting Entries

When filling out this campus partner form, there should be a control to ensure that nobody is registering a campus partner that is already listed.
At this point, anyone can fill out this form and is able to submit the form for a campus partner that is already in the system.
When a user starts typing a campus partner listed, it should prompt the user to confirm this campus partner rather than allowing the user to submit the form with an existing campus partner.

Moving this to sprint 5. Along with projects, we need to make find a way to better use validation when submitting forms

@shashankpatibandla @ QA tester

Test document - https://docs.google.com/spreadsheets/d/1LsngqCJK_cp2WXT11AeLwwwCm4zKmR5XU2c25BS7AmY/edit#gid=1719049306


Remove inotify scripts since inotify is depricated

Remove inotify scripts. We shouldn't be using these.

And some more very excellent cleanup! 👍 👍


docs: new website

To run locally:
cd website
pnpm i
pnpm start

Hi @dangreen thanks for the PR. I was thinking more of Gatsby SSG. Maybe we can still discuss this? Also, do you think we should have this in the same repo? If yes, we should probably use workspaces or even lerna so we have a bit more monorepo tooling at hand. I personally use Nx in my projects but then this would replace the whole build stack (Nx is based on Webpack and only uses Rollup for specific outputs I think). Or we go poly repo and create chartist/website

@gionkunz Hi. Docusaurus is also SSG. On the build step, it reads sources to generate API docs using typedoc, so it should be in the same repo. If we made monorepo, then pnpm install in the root will also install deps in website/, what not always required. CI steps will be faster without it. CI steps that work with the website will install deps in website/ separately.

@gionkunz I would probably stick with what @dangreen suggests if you'd don't have a really strong opinion about this.

What about search? Can you easily generate a search index and is there an easy way to integrate a local search like with Gatsby?

@gionkunz Yes, with Algolia.

@dangreen just tried to test it and I get a lot of errors. Somehow the spec files get read and the processes freaks out from the global expect:
Error: ../src/core/data/bound.spec.ts:98:7 - error TS2582: Cannot find name 'it'. Do you need to install type definitions for a test runner? Try `npm i --save-dev @types/jest` or `npm i --save-dev @types/mocha`.

98       it('should return single step if range is less than smallest increment', () => {
         ~~

Error: ../src/core/data/bound.spec.ts:105:9 - error TS2304: Cannot find name 'expect'.

105         expect(bounds.min).toBe(999.9999999999999);
            ~~~~~~

Error: ../src/core/data/bound.spec.ts:106:9 - error TS2304: Cannot find name 'expect'.

106         expect(bounds.max).toBe(1000);
            ~~~~~~

Error: ../src/core/data/bound.spec.ts:107:9 - error TS2304: Cannot find name 'expect'.

107         expect(bounds.low).toBe(999.9999999999997);
            ~~~~~~

Error: ../src/core/data/bound.spec.ts:108:9 - error TS2304: Cannot find name 'expect'.

108         expect(bounds.high).toBe(1000.0000000000001);
            ~~~~~~

Error: ../src/core/data/bound.spec.ts:109:9 - error TS2304: Cannot find name 'expect'.

109         expect(bounds.values).toEqual([roundWithPrecision(999.9999999999999)]);
            ~~~~~~

Error: ../src/core/data/normalize.spec.ts:3:1 - error TS2582: Cannot find name 'describe'. Do you need to install type definitions for a test runner? Try `npm i --save-dev @types/jest` or `npm i --save-dev @types/mocha`.

@gionkunz Hi. Did you install dependencies in the root and in the website dirs?

Also, I saw that you're referencing CUBE Slack as a chat for Chartist. I think we should use Github Discussions and the Chartist Gitter Chat instead: https://gitter.im/gionkunz/chartist-js

@gionkunz fixed

Codecov Report

Merging #1352 (0c92340) into main (b431f76) will not change coverage.
The diff coverage is n/a.

 Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@gionkunz Hi. Did you install dependencies in the root and in the website dirs?

I did what you wrote just pnpm i in the website folder. Do I need to do it on the root too?

@gionkunz yes

Hi @dangreen I was able to install the website now. Still, it feels a bit weird that the website is just a subfolder of the project. I think it would be much better if we create a pnpm workspace and have the library separated from the website. It should be possible to reference the files from a different workspace package to parse the TS files to create the API docs right? In nx that's also possible, and there's even a docusaurus community plugin. There we have apps/ui, apps/docs etc.
I also think that we will need some theming to bring that chartist 1 spirit over to chartist 2. It will lose a bit of heart and soul if we just use the standard docusaurus theme. Of course this can be done in a second step.
Other issues I have found:

The code references (view source) still refer to the old repo
The codesandboxes don't work (maybe also an issue with codesandbox, as I cannot access the website at the moment)
On the quickstart we're still referring to the CUBE Slack chat
For the domain of the new website we can use chartist.dev

@gionkunz

create a pnpm workspace

Done

need some theming

let's make it as separate task

The code references (view source) still refer to the old repo

I'm not sure what you mean

The codesandboxes don't work

I think it is a problem on your side, cause all works fine for me.

On the quickstart we're still referring to the CUBE Slack chat

Fixed

For the domain of the new website we can use chartist.dev

Are you own this domain?
js.org domains are free for js open source projects.

Thanks for the changes!
Yes, I already own the domain chartist.dev, so we can use it.
About the "view source" links, this is what I mean:

@gionkunz I think it depends on your local git repo config. In CI env all will be ok.

@gionkunz I hope this can be merged soon. Can't wait to see the new docs website live! 👀

I think technically it looks good. However, I dont think we should go live with it until we make it look a bit more like the charist "brand".

We can merge it but i would only publish it once we have it look like a new version of the old documentation website. Also the getting started guide needs to be migrated and other thibgs which are currently there on the old website. We can add this later but I think its very important.


How to install it with jitsi in docker and based on nginx?

Hey there!
Thanks for the great idea of amazing hacks.
How can I add it to an existing jitsi server in docker based on nginx?
Is there any option?
Thanks!

I don't use the Docker variant. You'll have to adapt the instructions from https://github.com/cketti/jitsi-hacks/issues/3#issuecomment-739332293

I don't use the Docker variant. You'll have to adapt the instructions from https://github.com/cketti/jitsi-hacks/issues/3#issuecomment-739332293

But it is based on apache and not nginx?

But it is based on apache and not nginx?

There's no turnkey solution. You'll have to do some of the work yourself.

There's no turnkey solution. You'll have to do some of the work yourself.


Define new project build structure

Story
Parent Issue
scalajs projects have sub-projects typically under js and jvm directories

merge into parent issue #952


rkt, rktflag: Move optionList, bitFlags and secFlags to rktflag

Splitting out this code, so it can be used in rkt subpackages, which
are imported by rkt package itself.

lgtm

Thanks, lgtm after those nits

I needed to make some more changes to make bitFlags a package private type.

LGTM


Wrong behaviour with decorators (Property 'xxx' is used before its initialization (2729))

TypeScript Version:  4.02

ts 2729 vue
typescript property is used before declaration decorators

Code
@Component
export default class StepViews extends Vue {
  @Prop() steps!: Step[];
  @Prop() initialStep?: number;

  public step = this.initialStep ?? 0;
//               ^^^^^^^^^^^^^^ Error: Property 'initialStep' is used before its initialization
}

Expected behavior:
Typescript doesn't throws an error because this is valid Vue component
Actual behavior:
Take for exemple this valid Vue component using vue-property-decorator.
This code is valid in Vue thanks to the decorators, and the props will be initialized, but typescript think I want to use it before initialization. How can I tell Typescript that I know better and disable this error?
I know if you know Vue you will tell me just put this in a created or mounted hook. It's fine for a simple type, but it's a hassle for more complex types like big form generators where I want Typescript to infer the type for me.

For now the workaround I found is
@Component
export default class StepViews extends Vue {
  @Prop() steps!: Step[];
  @Prop() initialStep?: number = undefined;

  public step = this.initialStep ?? 0;
}

Default values are normaly put inside the decorator @Prop({default: 0}) initialStep?

I don't know how Vue treats decorators and whether the framework has behavior that means @Prop will set initialStep at construction time, but if that's the case then TypeScript doesn't know about it or take it into account (and that's not a bug).
As far as TS is concerned, at the point public step = this.initialStep ?? 0; runs, initialStep cannot be initialized, making that line indistinguishable from public initialStep = 0 and your code indistinguishable from a bug/mistake.

@jgbpercy Yep Vue's @Prop decorator will init initialStep at construction time. Typescript can't know it so I wondered if there is a way to tell Typescript that it will be set before?

If this is always initialized by the @Prop, then it shouldn't be marked as optional. Probably what you want is
@Component
export default class StepViews extends Vue {
  @Prop() steps!: Step[];
  @Prop() initialStep!: number | undefined;

  public step = this.initialStep ?? 0;
}


ARM unwinder goes into infinite loop and runs out of memory

While reviewing the rtld-audit patch, I ran into the following
problems on ARM.
This is on arm1.cs.  I tried a few experiments and this goes back to
at least 2020.08.03.  It seems to have the same behavior in 2020.08.03
as master.
A typical run is REALTIME at 1200, 4 threads, about 21 seconds, and
65,000 total samples, hpcstruct analyzing itself.
hpcrun -e REALTIME@1200 ./hpcstruct-bin -j 4 --time hpcstruct-bin
(1) Running hprun on hpcstruct analyzing itself produces a few
internal segv's that we catch and continue.
Normally, I get about 1-2 segv's.
[2977, N]: error: segv: context-pc=0xffffa9dbe7cc
[2977, N]: -- begin new backtrace (innermost first) [SEGV] ----------
[2977, N]: NULL (0): ip = 0xffffaa8a9668 (0x79668), load module = /home/krentel/audit/hpctoolkit-string/install/lib/hpctoolkit/ext-libs/libsymtabAPI.so.10.2.0 | lip
[2977, N]: NULL (0): ip = 0xffffaa872a38 (0x42a38), load module = /home/krentel/audit/hpctoolkit-string/install/lib/hpctoolkit/ext-libs/libsymtabAPI.so.10.2.0 | lip
[2977, N]: NULL (0): ip = 0xffffaaf7fc64 (0xec64), load module = /usr/lib64/ld-2.25.so | lip
[2977, N]: NULL (0): ip = 0xffffaaf71f84 (0xf84), load module = /usr/lib64/ld-2.25.so | lip
[2977, N]: NULL (0): ip = 0xffffaaf71f84 (0xf84), load module = /usr/lib64/ld-2.25.so | lip
[2977, N]: -- end backtrace ------------------------------------

So, 1-2 segv's maybe we just dismiss as whatever.  It is libunwind,
maybe they segv somewhere.
But sometimes, approx 5% of the time, I get about 20 segv's.  Still,
it's only 20 out of 65,000, so maybe we let this go too.  It's
bimodal, 95% 1-2 errors, and 5% 20 errors.
(2) More seriously, about 5% of the time, something goes wrong and
triggers an infinite memory usage.  This is on arm1 with only 8 Gig
RAM plus 8 swap, so mmap() fails within a few minutes.
[19224, 2]: hpcrun_mmap_anon: mmap failed: Cannot allocate memory
[19224, 2]: hpcrun_malloc: out of memory, shutting down sampling
[19224, 2]: error: segv: context-pc=0xffffac39e648
[19224, 2]: -- begin new backtrace (innermost first) [SEGV] ----------
[19224, 2]: NULL (0): ip = 0xffffac833420 (0x11420), load module = /usr/lib64/libpthread-2.25.so | lip
[19224, 2]: NULL (0): ip = 0xffffac82bd50 (0x9d50), load module = /usr/lib64/libpthread-2.25.so | lip
[19224, 2]: NULL (0): ip = 0xffffad47979c (0x1679c), load module = /usr/lib64/ld-2.25.so | lip
[19224, 2]: NULL (0): ip = 0xffffad479bf8 (0x16bf8), load module = /usr/lib64/ld-2.25.so | lip
[19224, 2]: NULL (0): ip = 0xffffad479bf8 (0x16bf8), load module = /usr/lib64/ld-2.25.so | lip
[19224, 2]: NULL (0): ip = 0xffffad479bf8 (0x16bf8), load module = /usr/lib64/ld-2.25.so | lip
[19224, 2]: NULL (0): ip = 0xffffad479bf8 (0x16bf8), load module = /usr/lib64/ld-2.25.so | lip
[19224, 2]: NULL (0): ip = 0xffffad479bf8 (0x16bf8), load module = /usr/lib64/ld-2.25.so | lip
[19224, 2]: NULL (0): ip = 0xffffad479bf8 (0x16bf8), load module = /usr/lib64/ld-2.25.so | lip
...

I remember sometime about a year ago we had a problem where an unwind
went wrong where the unwind didn't advance, so you had an infinite
loop that quickly ran out of memory.  In the above trace, it gets
stuck at on IP and the IP doesn't advance.
Maybe the previous fix wasn't completely fixed, or maybe the "fix"
didn't include ARM.
But even if the problem is that libunwind doesn't unwind correctly, we
still need to detect that and not go into an infinite loop.
ping @jmellorcrummey

thanks for the diagnostic work. i am meeting with jonathon at 8:30am tomorrow. you are welcome to join us :-).

This repository has moved to GitLab, please continue this issue there.


Link is a 404

This link at the start of the page is not correct and is a 404:
"More information can be found here: Authentication and the Azure SDK"

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

ID: 17824597-7e9b-4ca5-12f7-310b0ce4bdf4
Version Independent ID: 49e1c9ee-fabf-df55-5bd9-1019abda22f1
Content: Service-to-service authentication to Azure Key Vault using .NET
Content Source: articles/key-vault/general/service-to-service-authentication.md
Service: key-vault
Sub-service: general
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

@jamesfm Thank you for bringing this to our attention. We shall evaluate and update the document as appropriate.

@jamesfm Thank you for the feedback. I have submitted a PR to fix the issue, the change is merged internally and will go-live in the public docs shortly.


[FEATURE REQ] Implement a menu of signal type options for killing a process

Is your feature request related to a problem? Please describe.
Currently, the functionality for killing a process is based on sending a SIGKILL to the process. Instead of always sending a SIGKILL, it might be better to provide the user with a list of signal types that can be used to terminate the process, eg: SIGTERM, SIGSTOP, etc.
Describe the solution you'd like
gopsutil provides support for 4 different types of signals that can be sent to a process. Support for signals other than the available ones can also optionally be added. A new menu can be displayed when the keys for "killing" a process are pressed, this menu can display the signal options as a termui list. Based on the selected option, that particular signal should be sent.
Additional context
Run htop and press F9  to kill a process, a menu of signal options is provided to choose from.

To be worked on after #80


static assets 404

When I use standard laravel serve command php artisan serve --port=8050 and go to http://<IP_ADDRESS>:8050/docs, I get 404 error for static assets like http://<IP_ADDRESS>:8050/css/style.css

Can you double check that the URL being linked to in the HTML is actually http://<IP_ADDRESS>:8050/css/style.css?

Do these files exist in the public/docs folder?

but http://<IP_ADDRESS>:8050/docs/css/style.css works as expected
curl -I http://<IP_ADDRESS>:8050/docs/css/style.css
HTTP/1.1 200 OK
Host: <IP_ADDRESS>:8050
Date: Mon, 08 Oct 2018 17:47:37 +0000
Connection: close
Content-Type: text/css; charset=UTF-8
Content-Length: 11894

It works this way
    <link rel="stylesheet" href="/docs/css/style.css" />     <script src="/docs/js/all.js"></script>

Hey...I just remembered you don't need to serve your app to view the docs.
They're static assets, like you pointed out—HTML and CSS. Just open the
index.html in a browser.

Hey...I just remembered you don't need to serve your app to view the docs. They're static assets, like you pointed out—HTML and CSS. Just open the index.html in a browser.

shalvah, thank you! But how to serve this files in order to show to my frontend developer?

I don't understand your question... Just visit the path to the index.html
and open in your browser.

Ok, thanks!

How can I solve this problem? I also encountered it.

Since the normal rewrite rule is like this:
    RewriteCond %{REQUEST_FILENAME} !-d
    RewriteCond %{REQUEST_FILENAME} !-f
    RewriteRule ^ index.php [L]

It will rewrite everything into your laravel application unless it's an actual real file. Since the documentation generator generates the index.html file and by default puts it in the /docs folder, if you go to /docs/index.html on your server it should work, whereas if you tried to go to /docs it would try to route the resource requests through your laravel app and not find them. HTH.

As @goldengecko mentioned: My static assets returned 404 errors at http://<IP_ADDRESS>:5001/docs/ but everything worked fine at http://<IP_ADDRESS>:5001/docs/index.html.


I have made the project on Amazon website clone

🛠 Fixes Issue (#4 )

👨💻 Changes proposed and Brief Description

✅ Check List (Check all the applicable boxes)

[x] My code doesn't break any part of the project
[x] This PR does not contain plagiarized content.
[x] My Addition/Changes works properly and matches the overall repo pattern.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

@swapnilsparsh I have made the project on Amazon website clone...


Add option to wrap inside an iife

I was wondering if it was possible to add an option to wrap a whole file and export it as a default  (I can create the PR if needed).

You're right, this functionality would belong in a different plugin 😀 Will close this issue, thanks


Download picture first to /tmp when checking if duplicate

The way checking for duplicate for picture work is:

download picture
cksum the picture
compare cksum with list of cksum from the others pictures in the folder.
delete if duplicate

This cause a problem if you have another program copying/moving pictures when they are downloaded. They will copy the picture even if it is a duplicate if it is copying fast enough.
The solution would be to:

download picture to /tmp
cksum the picture
compare cksum with list of cksum from the others pictures in the folder.
delete if duplicate, move to download folder if not

Done in 9d95b3c91409dcfa7df8f9681ce6bec68614620e


No enum constant org.flywaydb.core.api.MigrationType.SCRIPT

Which version and edition of Flyway are you using?
The old version was 8.0.0-beta2.  Just update the version to  8.5.4. Both versions have the same issue.
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Command-line
Which database are you using? (Type & version)
PostgreSQL 10.14
Which operating system are you using?
Docker image "adoptopenjdk/openjdk11:alpine-jre"
What did you do? (Please include the content causing the issue, any relevant configuration settings, the SQL statement(s) that failed (if any), and the command you ran)
I'm trying to see the current state of the migrations by issuing flyway info.
Note I already have some applied migration's

What did you expect to see?
flyway info should return the current status of the migrations
What did you see instead?
Flyway is up to date
Flyway Community Edition 8.5.4 by Redgate
See what's new here: https://flywaydb.org/documentation/learnmore/releaseNotes#8.5.4
Database: jdbc:postgresql://localhost:5432/dbname (PostgreSQL 10.14)
ERROR: java.lang.IllegalArgumentException: No enum constant org.flywaydb.core.api.MigrationType.SCRIPT
Note: If I delete the migration type "script" from the flyway_schema_history table, it works as expected.

Script migrations are a Flyway Teams feature so presumably the script migration was applied during the Teams free trial period which was included in some versions (including 8.0.0).
When using Community edition (e.g. after the free trial period ended or the version was upgraded to 8.5.4), Flyway no longer recognizes the script migrations so produces that error.
There should be a fix in the next version that means Flyway no longer produces that error. Instead the script migration will be considered Missing, since Flyway Community cannot resolve script migrations.


Default errror message is displayed even when custom error message is provided.

Hi,
Im validating whether the property name provided matches the pattern that i have provided and., if it doesnt match, i have provided a custom error message, so that default message is overridden by custom message.
But the issue is that along with the custom message even the default message is displayed which is not what i expected.
Please click the below link to check the issue mentioned above.
https://codesandbox.io/s/sweet-taussig-q8ubk6
Steps:
In codepen, click the button that is available on top of angular logo, and check in its console.
Versions:
"ajv": "^8.12.0",
"ajv-errors": "^3.0.0",
"ajv-keywords": "^5.1.0",

Not an ajv issue difficult to debug – please submit to ajv-errors with a code sample as one file in the issue or, better, in runkit.com
Thank you!


Don't read metadata form field as file

Something in the cgi form field logic changed and non-file form fields were given '' as filenames instead of None, which caused failures when metadata is sent when uploading.
🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉
Review Checklist

Tests were added to cover all code changes
Documentation was added / updated
Code and tests follow standards in CONTRIBUTING.md

@ryansanford FYI

CI passed, LGTM


Accessing the siyi a8 mini video stream

Hello !
So, my siyi camera is currently connected to a raspberry pi via ethernet on the eth0 interface with a static ip of <IP_ADDRESS>.. The Raspberry is on the same wifi network as my macbook which i want to use as a sort of ground station.. I Have the code in PyCharm on my macbook and should have all the neccessary dependencies on both my mac and the pi. How exactly do i setup a realtime stream of the siyi cam and access it through the code on my mac ?
Any help would be much appreciated !

I think you need to setup your network correctly.
The example provided in this package assumes that you connect the camera to tour PC (or RPi/Jetson) directly. The camera creates its own network interface. If the camera is not directly connected to your "Ground station", then you may need to do some re-routing (please Google it).


How to authenticate Google Language using API keys

I am looking to authenticate with the Google Language platform using API keys. According to the docs here, Google Language is a service that supports API key authentication.
When going through the docs and source for the google-cloud-ruby client, I am unable to determine how to authenticate with a given project_id and api_key. I only see support for authenticating using a keyfile. How would I be able to instantiate a Google::Cloud::Language client with an api_key?

@geigerj @jbolinger Do you guys have an answer for this? Do the GAPIC-only clients have support, or plan to support, use of API Keys instead of service account credentials?

There is no plan to support this.
On Thu, Mar 8, 2018 at 2:06 PM Mike Moore<EMAIL_ADDRESS>wrote:

@geigerj https://github.com/geigerj @jbolinger
https://github.com/jbolinger Do you guys have an answer for this? Do
the GAPIC-only clients have support, or plan to support, use of API Keys
instead of service account credentials?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-ruby/issues/1999#issuecomment-371640976,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD1jWEE2vziKlUUOB5GAZzU6zr5yNFA-ks5tcauAgaJpZM4SjZYd
.

@Orevarc, the direction given for authentication with the Google Cloud libraries has been to support service accounts as the primary authentication mechanism. The only library that currently supports API Keys is Translate (google-cloud-translate). Looking at the Using API Keys document you linked to, it also says that service accounts should be the primary mechanism as well.
I'm not 100% sure on this, but my understanding is that API Keys would only be able to be used on REST API endpoints. The Google Cloud gems for Natural Language, Speech, and Vision all use gRPC to access the RPC API endpoints. This may explain why there are no plans to support API Keys in those gems.

@blowmage @jbolinger Thank you for both the quick responses! Looks like I'll either have to switch to querying the REST API endpoint or obtain service account credentials and use this gem.

Thanks for the question. Best of luck!


Plan for maintenance mode

What - Need to identify the steps which need to be taken to ensure a smooth transition
Why -
Who needs to work on this
Who needs to review this
Done when

[x] Thing to do

Need to identify the steps which need to be taken to ensure a smooth transition

What dont we know...


Update tutorial-kubernetes-prepare-app.md

If I am not mistaken, it's the RabbitMQ image being downloaded, maybe Redis was used in an older example (?)

@schaffererin

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@microsoft-github-policy-service agree

@microsoft-github-policy-service agree

Thank you for your updates here, @LavredisG. I approved the changes and it's ready for #sign-off :)


WIP on doctype restriction

DAFFODIL-1422 is a ticket about restricting the XML we accept so that we do not allow DOCTYPE declarations in it.
This is a security related provision, as use of DOCTYPEs leaves XML loaders subject to a variety of problems such as documents exploding in size as the DOCTYPE declarations are expanded.
DOCTYPEs are an old obsolete idea, and simply disallowing them entirely is the best option.
There are other jira tickets about disallowing resolvers and loaders from dereferencing URIs treating them as internet URLs.
The upshot of all this is that "loading" XML is tricky, and needs to be done carefully via a centralized library that provides the various options different loading requires, while not exposing/allowing the security vulnerabilities.
This change set does not yet include establishing a single central library that all XML loading goes through.
Right now this is at the point where it is apparent such a library is needed, because there are too many places that are invoking XML loaders for them to just all be "done the right way". Under maintenance this is too likely to drift.
A key starting point is to survey every place Daffodil does XML loading. These include loading of:

DFDL schemas and the include/import schema files they reference
** This can include DFDL schemas, but also XSD for annotation languages (e.g., schematron annotations)
** Note that this validating loader is loading a schema, but loading it not as a schema, but as ordinary XML. This should be validated against the schema for DFDL schemas.
XML Infosets being unparsed
** (Currently not a validating loader)
TDML files for testing - this can in turn lead to loading of DFDL schemas
** Test cases can load XML Infoset files.
** validation here involves validating defineSchema elements which contain DFDL schema.
Config files
daffodil-propgen loads the XSD files for DFDL annotations, tunables, etc.
Xerces validator loads a schema, and the include/import files it references
Xerces validator loads XML being validated

This may not be a comprehensive list.

Note: if we disable DOCTYPE we are disabling general entities, as those can only be supplied in a doctype.


Improve the API unit-tests, and try to expose more API-functionality in the TypeScript definitions

Please refer to the individual commit messages for additional details.

/botio unittest

Given https://github.com/mozilla/pdf.js/pull/13957#issuecomment-918603941, does the last commit make sense to include at this point in time?

Given #13957 (comment), does the last commit make sense to include at this point in time?

I really don't know, since we also have https://github.com/mozilla/pdf.js/issues/12384#issuecomment-695143287 which explicitly recommended this approach.

@tamuratak Would you perhaps be able to shed some light on if the third commit in this PR, https://github.com/Snuffleupagus/pdf.js/commit/95057a4e56cd04ecf4248afcc7e82a303d36c5bb to be precise, is working correctly given your suggestion from https://github.com/mozilla/pdf.js/issues/12384#issuecomment-695143287 one the one hand but https://github.com/mozilla/pdf.js/pull/13957#issuecomment-918603941 on the other hand that claims it doesn't have the desired effect?

The following line does not work as expected.
https://github.com/mozilla/pdf.js/blob/95057a4e56cd04ecf4248afcc7e82a303d36c5bb/src/pdf.js#L21
That is because  RenderTask is not exported in src/display/api.js. PDFDocumentProxy and PDFPageProxy are exported in the file.
https://github.com/mozilla/pdf.js/blob/95435ed66f18cd613af45f138eb1b54110a282f2/src/display/api.js#L3140
For #13957, if we remove the RenderTask type annotation in the sample code, it works with the type inference of TypeScript.
const task: RenderTask = pageProxy.render(renderContext);

const task = pageProxy.render(renderContext);

That is because  RenderTask is not exported in src/display/api.js.

Huh, but it actually is thanks to the previous commit; please see https://github.com/mozilla/pdf.js/pull/14013/files#diff-082d6b37ad01db7ac97cc07c6ddb0dc52040484c5ef91b110b072f50144d9f39
Furthermore, we now have unit-tests that ensure that RenderTask is being exported from that file.
@tamuratak Did you test the entire PR, or only its last commit?

Furthermore, we now have unit-tests that ensure that RenderTask is being exported from that file.
@tamuratak Did you test the entire PR, or only its last commit?

I am sorry I have missed that. I have confirmed that things work as expected.

I am sorry I have missed that. I have confirmed that things work as expected.

No worries; thank you for helping out with testing this!

Let's do this now that we know it's working; thanks!

Thank you for looking into this!


Enkhara's Lament

It should mention on the items list that it requires bolt or higher spells instead of just any 'fire'/'air' spell

Being worked on


database.yml Spelling Error

Leaf Version:
Description of the Bug
a spelling mistake i think
Console Errors
Paste your error here if you get any errors.

database_same: "Database"

should be

database_name: "Database"

Pending fix in #77


GH-34657: [Go] Move Scalar interface to arrow

This is a pre-requirement for https://github.com/apache/arrow/issues/34657 as array interface needs access to the Scalar interface and without it, it will cause cyclic loop.
cc @zeroshade

Closes: #34657

I don't think this resolves the issue unfortunately, which is why the existing GetScalar method is in the Scalar package. Even if you put the Scalar interface into the arrow package, if you try to put GetScalar as a member of the Array objects you'll still run into the issue of trying to actually construct your Scalar objects. You'll still run into the cyclical import if you try to call any of the MakeScalar methods. So I don't think that this is really worthwhile since it doesn't solve the cyclical import problem.


[3.5 rc4] 3D editor viewport doesn't render any contents.

Godot version
v3.5.rc4.official [e047c5006]
System information
Windows 10, Intel Graphics, GLES3
Issue description
3D viewport in the editor doesn't render anything except for background color or the sky.
No origin, no grids, no nodes, gizmos etc.
Everything worked fine on v3.5.rc3.official [af8a02dda].

Minimal reproduction project
No response

Could be related to #61995?

I just checked and the same behavior happens in rc3 when async shader compilation is enabled in project settings.

Fixed by #62226.


Freedom supporter

Hi freedom supporter, I wonder why you are not putting visibility for the mass murdering of Syria government in Syrian civilians..  Or do you prefer to close your eyes in the US? Why bringing this to public GitHub and taking sides in favor to a terror organization?
You are reported, I believe this is against the GitHub community rules.
Enjoy
https://en.wikipedia.org/wiki/Casualties_of_the_Syrian_civil_war

Lets keep this to a single issue

The GraphQL technical steering committee is working on a policy to guide the use of political speech from the voice of a GraphQL project. The original author chose to revert the original change via #1879 until we complete that policy. This should resolve this issue.
As a reminder, all discussion within GraphQL projects must adhere to our Code of Conduct.


remove base image

Remove base image -> its not used by anyone

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fix for build #1

Please pull commits made on June 24, 2015 and June 25, 2015. They broke the build that was passing. Thanks :)

Hmm, can you explain a bit more what you mean?

Sorry, sometimes I have a hard time explaining what I mean. I need a little time to figure out the best way to help you understand. I will get back to you ASAP.


Optional media pre-loading

DO NOT MERGE !
Fixes #435

[ ] Replace all media loading with clickable link that opens media in external browser
[ ] Replace all media loading with "click to download" image/text that will download and show in app
[ ] Introduce pre-loading preferences: never, wifi-only, always
[ ] Optional: feature switch

Current ui:

if you have any ideas how the ui should look please comment here with some example

cc @vitorpamplona
Do you think I should create a settings page or we can continue without one for a while?
Also should i load profile pictures, banners etc or just stop loading the content of the notes?

FYI, the Checkbox is quite ugly. You can use the Switch compose on the right of the text to make it similar to the Android settings page.

Wow... looking great so far. Loving being able to follow the development.
Can we consider adding preview of links to this feature? It would make a truly snappy and lightweight experience on slow mobile connection.... However happy to see it being added later if it speeds up the release of this.
well done!

@vitorpamplona @davotoula
Should I move the zaps and custom emojis configuration to this settings page?

I'm also still loading profile pictures and verifying nip 5. Should i disable that too?

Personally I would prefer that.
It's more natural (from other apps) to single click to do default emoji/zap and hold down to select alternative.
I often make mistake to edit custom zaps/emojis instead of selecting them when holding down.
If custom emoji/zap lives in settings there is less chance for error.
Let's see what @vitorpamplona says, I understand he may be concerned about settings bloat.
Also, maybe better as a future enhancement to ship the original media loading feature quicker.

Small, well defined enhancements with shorter development time are so much easier to code review and to release!

This is getting good!
A few comments:

I wouldn't move the Account Preferences (Relays, filters, backup, Tor) to this page yet. Apologies if my earlier post confused things. They could stay in the sidebar for now to avoid breaking the Settings Screen into two UI concepts: (i) buttons to open dialogs and (ii) combo boxes with save button. We will have to add Zap/NWC setup, Reactions Setup, and Translations setup to these Account Preferences. So, we might leave it for later.
Can we make these combo boxes look better? On the image upload screens (NewPostView.ImageVideoDescription), we use a TextSpinner for servers. That looks better, I think.
The save button should either be at the top, in a similar shape that other dialogs/new posts/etc do, with a way to cancel or the change should take effect immediately (no save action).

This is really nice @greenart7c3 . But isn't it better to have an explicit save button? IMHO it is useful for visual feedback. I guess you could implement it like the Post button, disabled when settings are saved, enabled when Settings are modified.

Yes, I find the button better. I also need to improve the design of the video player and the load image placeholder

@greenart7c3 Yeah. IMHO the 'load image' button could be much smaller. As for video, I am thinking of two options: Either using a button just like the image option, or having a blurred(or not) thumbnail.
cc @vitorpamplona

We won't have a thumbnail without downloading the entire image (only available in NIP-94/95 images). :(
Maybe display the URL with a small inline Load button?

@vitorpamplona that seems to be a good idea. The button could be put at the bottom right, for example, with the URL link in the center.

You mean show the images and videos like this?

This is the LoadingAnimation we have that displays while the image is loading. We could do a similar size loading button

That works. It is hard to press the button? Too small?

For me with wasnt hard, but i think for other people might be depending on the size of the finger and the device size

The logos are more appealing than the simple button.

@greenart7c3 let me know when you are ready for a merge on this


Animate state transition in simulator

Don't just set the next state to be the active state, animate the transition from current active state to next active state.
Highlight transitions and states traversed.  Also allow a configuration of the transition time (in the simulator panel)

http://js.cytoscape.org/#eles.animate


Support private PyPI repositories

This issue is related to the following conversation from fast.ai forum: https://forums.fast.ai/t/nbdev-for-private-pypi-repositories-and-poetry-package-management/100523
Goal of the issue

To gauge the interest in nbdev supporting private PyPi repos (e.g. GCP Artifact Registry).
To discuss the best way to integrate private PyPi repo support into nbdev.

If there's interest, I'd be happy to make the contribution.
Problem
nbdev does not support dependencies from private PyPi repositories.
Workaround
Changing the following line in nbdev CI workflow:
test -f setup.py && pip install -e ".[dev]"

to:
test -f setup.py && pip install --extra-index-url https://$PYPIUSER:$PYPIPASSWORD@$PYPISERVER -e ".[dev]"

fixes the issue.

+1 for interest - this would be a big win for being able to use this for work!

Current nbdev solution
nbdev itself doesn't do anything much, as seen in their "def release_pypi" function in https://github.com/fastai/nbdev/blob/3f0266328c2537a35487767288bc4283308f7048/nbs/api/18_release.ipynb#L748
Therefore, a case could be made that instead of complicating the codebase, users of private pypi servers can just have a couple of additional lines of code in their code base for custom uploading / downloading from pypi.
Proposed solution
How to publish a package to a private pypi server:
test -f setup.py && rm -rf dist build *.egg-info .eggs
python setup.py sdist bdist_wheel
twine upload --repository-url <URL> --username <PYPI_USERNAME> --password <PYPI_PASSWORD> dist/*

How to then install the package from a private pypi server:
pip install --extra-index-url "http://<PYPI_USERNAME>:<PYPI_PASSWORD>@<IP_ADDRESS>:8080" private_package

Don't forget to change the placeholders:

<URL>: The URL of your private PyPI repository.
<PYPI_USERNAME>: The username for accessing your private PyPI repository.
<PYPI_PASSWORD>: The password for accessing your private PyPI repository.
<IP_ADDRESS>: The placeholder IP address of your private PyPI repository.

Note: it's better to use a domain like my-pip-server.com instead of an IP address so that you can set up an SSL connection, otherwise your credentials are passed in plain text over the internet. Instructions how to do that are available here and here.

8080: The placeholder port number for your private PyPI repository.
private_package: The private package you want to install.


Issue/gc issue

I have added the size of the image which depends on the orientation. Please help me to review and merge it
Issue:
https://github.com/esafirm/android-image-picker/issues/368

Hi @chihung93 thanks for the PR, although I'm pretty sure this is already handled by Glide internally, but you can tell me if I'm wrong on this. A before-after video would be great 😃
Also, we have Custom Component API which makes this change less wanted.


Client's response to failed code lens resolve requests

In issue #1738 dbaeumer mentioned that resolve requests that are unable to be resolved should fail with ContentModified. We currently do that in HEAD@haskell-language-server, however, the client behavior to these failures for lenses is not ideal. Specifically, visual studio code shows a "no command" lens for any lens that fails.
Ideally, if the client is not able to resolve the lens, it should not show anything. This should also be clarified in the spec. (In addition, should #1738 also be written into the spec).

Actually the spec tries to avoid specing what a client should do in the UI. For VS Code the resolve request should be canceled now. See https://github.com/microsoft/vscode/pull/180485.
However VS Code might still show no command or missing command in some cases. You should open an issue against VS Code if you still see this even when the cod lens resolve request got canceled.


How can access tokens contain the "roles" claim

I am using the OAuth 2.0 auth code grant flow to acquire access tokens, and I hope the token could have roles claim as described in the doc, which should be populated with the roles the user was assigned to on the target application for user tokens.
I have configured my application in Azure Portal in which the appRole property in the manifest is as below, and I assigned one role to a login user.

However, after receiving tokens by Azure AD, I can only find the roles claim in id token rather than access token. So my question is

Is my understanding of this claim in access tokens incorrect: this claim should be populated with the appRole property of a login user?
How can I get it in access tokens?
Is there a way to make the roles claim contains the group information of a login user?

Thanks.

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

ID: 2471c08b-befe-8e1a-df08-f13e116c231e
Version Independent ID: 0d827d33-2906-6352-9398-13863def9354
Content: Microsoft identity platform access tokens - Microsoft identity platform
Content Source: articles/active-directory/develop/access-tokens.md
Service: active-directory
Sub-service: develop
GitHub Login: @hpsin
Microsoft Alias: hirsin

@yiliuTo Thanks for reaching out. We will investigate and update the thread accordingly.

Hello @yiliuTo thank you for sharing the query. The appRoles that you create in your App Manifest, first needs to be assigned to the user in that app. Let's check the steps below:
I created the following appRoles in the app manifest:

After the appRoles are created in the app manifest, you need to assign them to a user who is already added to access this application under Users and Groups section in the enterprise application.
Then you can request for an access token using the Auth-Code Grant Flow. I tested this and I am sharing the sample request for you to try:
To request the code from /authorize endpoint: https://login.microsoftonline.com/soumimsft.onmicrosoft.com/oauth2/authorize?client_id={App ID}&response_type=code&redirect_uri=http%3A%2F%2Flocalhost%3A1234&response_mode=fragment&resource={App ID}&state=12345
To request the access-token from /token endpoint:

Note: Here you would find that the clientId and the resource have the same value. This is required to fetch the token with the app roles in it.
Access-Token:

How can I get it in access tokens?
Ans: With the above request, I am sure you can fetch the access-token, as Auth-Code grant flow if done, it returns access-token by default.
Is there a way to make the roles claim contains the group information of a login user?
Ans: No, this is not possible.
Hope this helps.

Thanks you so much for such a detailed reply @souravmishra-msft , I can get roles claim now with your help. Now I have a few new questions:

Does roles claim exist only when the resource or scope(I use the V2 endpoints) parameter in access-token request is set as the client-id or client-id/.default? In my scenorios, I hope my application could acquire an access token of a specific resource and permission, e.g., https://graph.microsoft.com/ user.read, and when I set the scope as _https://graph.microsoft.com/user.read_ instead of client-id/.default, then then access token doesn't contain the roles.

Another two questions are about the groups claim in access token: Why does this claim only exists when the aud of an access token is not Graph API like https://graph.microsoft.com? And is there a way to make the content of groups claim to be groups' name instead of groups' object-id?

Thanks~

Hello @yiliuTo, I have updated my previous response, I would request you to please check that once again as I have added few more details and also corrected one of the points.
To get the group claims in the access-token, you need to use the optional group claims. I have updated the previous answer of mine with the new details on Group claims and how to add them in access_token.
Coming to the second part, where you mentioned "Why does this claim only exists when the aud of an access token is not Graph API like https://graph.microsoft.com?", As you remember, while configuring the roles, we are configuring it on the Application's manifest as a property of that application and not for Graph, hence the App Roles property is local to that application where you configured this. So, whenever you want the app-roles to be present in the access-token, you need to mention that app-id where the app-roles are configured. In our test case, we had configured it in that same app, for which we requested the token and hence both the appID/ClientID and the scope/resource parameter had to contain the same clientID.
Hope this helps. Please feel free to let us know if there are more queries around this so that we can help you better.

@souravmishra-msft thank you for your explanation and updates! My new question is about the optional group claims, is there a way to configure the display value of groups claim? I want it to display the name of groups instead of object-id. I have read about these docs: doc1 and doc2, however, all the source attributes to be selected to display I found are as below:

while the value I hope could be displayed in the groups claim is the group name:

Hello @yiliuTo, You cannot get the Group Names (for groups created in AzureAD) in tokens. You only get the group ids as claims in the issued tokens by AAD. Usually, this is good, since the id cannot be changed, unlike the name which can change.
If you want to do authorization based on groups, you can set the ids in a configuration file and then check with the id.
If you want the names for some other purpose, you'll need to query the group's details from Microsoft Graph API. You can find the API documentation here: https://docs.microsoft.com/en-us/graph/api/resources/groups-overview?view=graph-rest-1.0
Hope this helps.

Thanks for your help~~


Issue #257 MinSize of 1 not working when value contains a single character that is removed

Closing pull request and not implementing this fix as it is not a bug. Won't fix.
If after cleanup of a value, it is null or empty string then the MinSize is not looked at because usually null and empty are considered separately from an actual value.
This is because optional values must allow for null and empty values.
To make something mandatory you will need to add @NotBlank or @NotEmpty to your property field validation.


CSL JSON invalid

we have commas at the end of arrays in the CSL JSON:
https://api.catalogueoflife.org/dataset/1081/reference/333976
{
	"created": "2021-03-10T15:10:44.448074",
	"createdBy": 10,
	"modified": "2021-03-10T15:10:44.448074",
	"modifiedBy": 10,
	"datasetKey": 1081,
	"id": "333976",
	"verbatimKey": 21452,
	"csl": {
		"author": [{
				"family": "Grischenko",
				"given": "A. V.",
			},
			{
				"family": "Gordon",
				"given": "D. P.",
			},
			{
				"family": "Morozov",
				"given": "T. B.",
			}
		],
		"issued": {
			"date-parts": [
				[
					2018,
				]
			]
		},
		"DOI": "10.11646/zootaxa.4508.1.4",
		"page": "4508(1): 68",
		"title": "Fatkullina imitata n. sp., second species of a unique cheilostome bryozoanÂ genus with reversed-polarity zooidal budding, and new family Fatkullinidae",
		"URL": "http://www.marinespecies.org/aphia.php?p=sourcedetails&id=333976",
		"container-title": "Zootaxa",
	},
	"citation": "Grischenko, A. V.; Gordon, D. P.; Morozov, T. B. Fatkullina imitata n. sp., second species of a unique cheilostome bryozoanÂ genus with reversed-polarity zooidal budding, and new family Fatkullinidae. Zootaxa. 4508(1): 68 (2018).",
	"year": 2018,
	"page": "4508(1): 68",
	"parsed": true,
}

most JSON parsers are fine with this, but strictly its wrong

This is an artifact of the browser rendering the JSON. The real JSON sent from the API by Jackson does not contain the commas!


Go to definition and code completion does not work in test files

Describe the bug
Given a v project with these two files
// src/file_manager.v
pub struct FileManager {
}

pub fn (f FileManager) read_page() {}

// src/file_manager_test.v
fn test_read_page() {
	file_mgr := FileManager{}
	file_mgr.read_page()
}

Expected Behavior

Completion for methods of struct FileManager should be shown
Ctrl/Cmd + Click on a function name should go to the definition of the function.

Current Behavior

There's no completion for the FileManager struct
Cmd+Click does nothing

Reproduction Steps

Create a v project v new <project_name>
two files file_manager.v and file_manager_test.v
copy this code into file_manager.v

// src/file_manager.v
pub struct FileManager {
}

pub fn (f FileManager) read_page() {}

Copy this code into file_manager_test.v

// src/file_manager_test.v
fn test_read_page() {
	file_mgr := FileManager{}
	file_mgr.read_page()
}

Possible Solution
No response
Additional Information/Context
No response
Environment details (v doctor output)
V full version: V 0.4.6 604eb65.7a36b44
OS: macos, macOS, 14.0, 23A344
Processor: 10 cpus, 64bit, little endian, Apple M1 Pro
getwd: /Users/tomialagbe/dev
vexe: /Users/tomialagbe/v/v
vexe mtime: 2024-05-22 18:07:15
vroot: OK, value: /Users/tomialagbe/v
VMODULES: OK, value: /Users/tomialagbe/.vmodules
VTMP: OK, value: /tmp/v_501
Git version: git version 2.39.3 (Apple Git-146)
Git vroot status: weekly.2023.32-1929-g7a36b44c-dirty
.git/config present: true
CC version: Apple clang version 15.0.0 (clang-15<IP_ADDRESS>)
thirdparty/tcc status: thirdparty-macos-arm64 5c1d002f
Editor name
Vscode
v-analyzer Version
v-analyzer version: 0.0.4-beta.1.7e11a6f
VS Code Extension Version
v0.0.2

Hi. I think the problem is not directly related to v-analyzer, but to how V treats test files as internal and external ones.
Try changing file_manager_test.v to:
module main

fn test_read_page() {
        file_mgr := FileManager{}
        file_mgr.read_page()
}

The module main at the start is important, so that V will consider the test as an internal one (see the note for internal tests in https://github.com/vlang/v/blob/master/doc/docs.md#test-files).
Without it, the test file is considered to be an entirely separate program, and is compiled without having access to any of the symbols from the other files in src/ (i.e. you would have to import everything that you want to test in it, and you can not import the main module).
You can also try v test . in the src/ folder. It will pass for a _test.v file, that has module main at its top, and will fail, for one that lacks it:


Standardize TileMap methods to get and set collision layers and masks (2.1)

Note: this is the 2.1 version of this pull request:
https://github.com/godotengine/godot/pull/8271
Add several methods to the TileMap node to make collision layers and masks work like PhysicBody2Ds and RayCast2Ds:

set_layer_mask()

get_layer_mask()
Those methods only call set_collision_layer() and get_collision_layer(), but they are named like those in PhysicBody2Ds and RayCast2Ds. When you want to change your level organisation from static bodies to a TileMap, you won't have to modify your code anymore (it doesn't work the other way...)
As Godot 3.0 will break compatibility, it may worth to use only one name and remove the other method.

set_collision_layer_bit()

set_collision_mask_bit()

get_collision_layer_bit()

get_collision_mask_bit()

set_layer_mask_bit() ( just calls set_collision_layer_bit() )

get_layer_mask_bit() ( just calls get_collision_layer_bit() )
It is now possible to change bits directly, like with PhysicBody2Ds and RayCast2Ds.

Oh, was possible for 2.1 without breaking anything? good!
This issue complicated the code a bit on a project where I was trying to modify layers of tilemaps and objects with a single call group.

I corrected the code, it should be better now!

I understand the wish to make TileMap have a similar API to PhysicBody2D and Raycast2D, but IMO the addition of {s,g}et_layer_mask is confusing. Users will wonder whether they should use set_layer_mask or set_collision_layer.
Since the {s,g}et_layer_mask ones are no longer in 3.0, I'd propose not to add them here. People should not use it as it's going to be deprecated anyway.

Yes that makes sense. I will modify the commit tomorrow afternoon and remove these changes to only keep the *_bit methods (like in 3.0).


SyntaxError: Missing parentheses in call to 'print'. Did you mean print(...)?

I'm getting the following error "SyntaxError: Missing parentheses in call to 'print'. Did you mean print(...)? " I'm not sure it could be an installation issue. Could you provide a step by step guide to install the repo

File "/home/kali/PassGen/passgen.py", line 14
print password
^^^^^^^^^^^^^^
SyntaxError: Missing parentheses in call to 'print'. Did you mean print(...)?


Add send Email support

Description of changes:

Add SES and SNS permission to serverless config
Send an Email to the administrator upon user request to contact the service provider (check task details screen)

Checklist

[ ] Tests have been added and are passing
[ ] Documentation has been updated

Note:

Make sure to update .env.development file with new configuration entries.

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

:tada: This PR is included in version 1.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


SCNetworkReachibility objects not being freed

SCNetworkReachability objects not freed when startNotify called.
Running Reachability.swift under Instrument's Allocation tool showed
SCNetworkReachibility objects not being freed after startNotify was
called.
Calling SCNetworkReachabilitySetDispatchQueue to NULL out the dispatch
queue in stopNotify resolved this issue.

Thank you for the pull request


swagger-ui won't work because json responses have java object names in them

I am trying to integrate swagger-ui to a spring-boot project. I am using the latest versions and following the demos and keep getting swagger-ui.html with undefined in selection box. After I debugged it, and I realized instead of getting a expected JSON object like this:
[     {         "@class": "springfox.documentation.swagger.web.SwaggerResource",         "name": "batch-rest-api",         "location": "/v2/api-docs?group=batch-rest-api",         "swaggerVersion": "2.0"     } ]
I will get this:
[     "java.util.ArrayList",     [         {             "@class": "springfox.documentation.swagger.web.SwaggerResource",             "name": "batch-rest-api",             "location": "/v2/api-docs?group=batch-rest-api",             "swaggerVersion": "2.0"         }     ] ]
When this URL is being queried: http://localhost:8080/swagger-resources
Because of that, the AJAX code won't work. I think there are some other module's setup requires the Java object to present in Jackson's serialization, and this messed up swagger-ui. Is there a way to get this to be working? Thanks!

No idea.
Please share more details about how you've configured your spring application, is it a boot application etc. See guidelines for issue submission.

Fixed. There is a Jackson MessageConverter brought in by Spring Batch Admin: https://github.com/spring-projects/spring-batch-admin/blob/master/spring-batch-admin-domain/src/main/java/org/springframework/batch/admin/domain/support/VariableTypeJackson2ObjectMapperFactoryBean.java
I removed it and now swagger works.


Nuclide import autocomplete

It does not work anymore. Investigate. Is it my bug or Nuclide?

I don't care.


feat(templates): add tests to show only the deployer can use .set

Closing as we can explain this in docs instead of a test templates!