Python error trying to setup

When following the local setup instrcutions from the README I get this error (using toolbox on Fedora 35):
⬢[tobias@toolbox backend]$ docker-compose up
Traceback (most recent call last):
  File "/usr/lib/python3.10/site-packages/urllib3/connectionpool.py", line 699, in urlopen
    httplib_response = self._make_request(
  File "/usr/lib/python3.10/site-packages/urllib3/connectionpool.py", line 394, in _make_request
    conn.request(method, url, **httplib_request_kw)
  File "/usr/lib64/python3.10/http/client.py", line 1276, in request
    self._send_request(method, url, body, headers, encode_chunked)
  File "/usr/lib64/python3.10/http/client.py", line 1322, in _send_request
    self.endheaders(body, encode_chunked=encode_chunked)
  File "/usr/lib64/python3.10/http/client.py", line 1271, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File "/usr/lib64/python3.10/http/client.py", line 1031, in _send_output
    self.send(msg)
  File "/usr/lib64/python3.10/http/client.py", line 969, in send
    self.connect()
  File "/usr/lib/python3.10/site-packages/docker/transport/unixconn.py", line 30, in connect
    sock.connect(self.unix_socket)
FileNotFoundError: [Errno 2] No such file or directory

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.10/site-packages/requests/adapters.py", line 440, in send
    resp = conn.urlopen(
  File "/usr/lib/python3.10/site-packages/urllib3/connectionpool.py", line 755, in urlopen
    retries = retries.increment(
  File "/usr/lib/python3.10/site-packages/urllib3/util/retry.py", line 532, in increment
    raise six.reraise(type(error), error, _stacktrace)
  File "/usr/lib/python3.10/site-packages/urllib3/packages/six.py", line 718, in reraise
    raise value.with_traceback(tb)
  File "/usr/lib/python3.10/site-packages/urllib3/connectionpool.py", line 699, in urlopen
    httplib_response = self._make_request(
  File "/usr/lib/python3.10/site-packages/urllib3/connectionpool.py", line 394, in _make_request
    conn.request(method, url, **httplib_request_kw)
  File "/usr/lib64/python3.10/http/client.py", line 1276, in request
    self._send_request(method, url, body, headers, encode_chunked)
  File "/usr/lib64/python3.10/http/client.py", line 1322, in _send_request
    self.endheaders(body, encode_chunked=encode_chunked)
  File "/usr/lib64/python3.10/http/client.py", line 1271, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File "/usr/lib64/python3.10/http/client.py", line 1031, in _send_output
    self.send(msg)
  File "/usr/lib64/python3.10/http/client.py", line 969, in send
    self.connect()
  File "/usr/lib/python3.10/site-packages/docker/transport/unixconn.py", line 30, in connect
    sock.connect(self.unix_socket)
urllib3.exceptions.ProtocolError: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.10/site-packages/docker/api/client.py", line 214, in _retrieve_server_version
    return self.version(api_version=False)["ApiVersion"]
  File "/usr/lib/python3.10/site-packages/docker/api/daemon.py", line 181, in version
    return self._result(self._get(url), json=True)
  File "/usr/lib/python3.10/site-packages/docker/utils/decorators.py", line 46, in inner
    return f(self, *args, **kwargs)
  File "/usr/lib/python3.10/site-packages/docker/api/client.py", line 237, in _get
    return self.get(url, **self._set_request_timeout(kwargs))
  File "/usr/lib/python3.10/site-packages/requests/sessions.py", line 542, in get
    return self.request('GET', url, **kwargs)
  File "/usr/lib/python3.10/site-packages/requests/sessions.py", line 529, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/lib/python3.10/site-packages/requests/sessions.py", line 645, in send
    r = adapter.send(request, **kwargs)
  File "/usr/lib/python3.10/site-packages/requests/adapters.py", line 501, in send
    raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/bin/docker-compose", line 33, in <module>
    sys.exit(load_entry_point('docker-compose==1.29.2', 'console_scripts', 'docker-compose')())
  File "/usr/lib/python3.10/site-packages/compose/cli/main.py", line 81, in main
    command_func()
  File "/usr/lib/python3.10/site-packages/compose/cli/main.py", line 200, in perform_command
    project = project_from_options('.', options)
  File "/usr/lib/python3.10/site-packages/compose/cli/command.py", line 60, in project_from_options
    return get_project(
  File "/usr/lib/python3.10/site-packages/compose/cli/command.py", line 152, in get_project
    client = get_client(
  File "/usr/lib/python3.10/site-packages/compose/cli/docker_client.py", line 41, in get_client
    client = docker_client(
  File "/usr/lib/python3.10/site-packages/compose/cli/docker_client.py", line 170, in docker_client
    client = APIClient(use_ssh_client=not use_paramiko_ssh, **kwargs)
  File "/usr/lib/python3.10/site-packages/docker/api/client.py", line 197, in __init__
    self._version = self._retrieve_server_version()
  File "/usr/lib/python3.10/site-packages/docker/api/client.py", line 221, in _retrieve_server_version
    raise DockerException(
docker.errors.DockerException: Error while fetching server API version: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))

Wondering if https://github.com/flathub/website/commit/18ab3e7047b0dbf026a294c5eda8059c87bdf628 has a positive effect on this usage

Did you manage this? Can this be closed?


[POC] Open/Close Live Editor and LiveCodeBlock

depends on https://github.com/vivid-planet/comet-docs/pull/22
I tried two things:
Add an Open/Close button to the Live Editor
https://github.com/vivid-planet/comet-docs/assets/13380047/b321414f-694d-426a-a9fb-f4103a971ba8
I swizzled the @docusaurus/theme-live-codeblock theme and added the button and styling changes.
First, I tried attaching the button without swizzling using the CSS classes and vanilla JS. However, there were two problems:

the theme uses CSS modules, meaning there is a suffix attached to the class. I wasn't sure how stable the suffix is, so I didn't want to rely on that
I didn't know where to place the JS to attach the button so it affects all stories. Also, doing this with vanilla JS felt wrong

Add a LiveCodeBlock component to avoid duplication of the transform code
I tried if it's possible to extract the configured  from mdx to a dedicated file. This could be our default since probably 99% of the time stories will be tsx and the transformCode will be necessary.
@johnnyomair Do you see any downsides to this? (as I'm sure you thought of that too)

:clap: great!
one minor thing: Can you rename "Live Editor" to "Code" and remove the "Result" line
With this in place we now should convert all docs!

@nsams  👍
I made these changes here: https://github.com/vivid-planet/comet/pull/1170

Moved to main repository: https://github.com/vivid-planet/comet/pull/1170


Research Export Command CSV

Currently, the console command exports it as a Lua table string, but the app expects it as a comma-separated list of techs. I've changed the script to write a comma-separated list.

One thing: please update the patch version in package.json. I think it should be 2.4.1. The main branch was just updated to 2.4.0 today.
Hoping to get that automated soon so contributors don't have to worry about it but it's manual for now.


React Native support

Any plans?

Need this also.

problem is that i don't work with RN personally atm, i just don't know the api.

@brunolemos @daryl so this will soon happen, i will make it work with react-three-fiber so it has to be made cross platform, which then includes react-native. do you have any ideas how we can pull this off? i guess it needs to know some platform specifics, and the platform has to support start/move/end.

I don't have experience with gestures but you'll probably need to use this api: https://facebook.github.io/react-native/docs/panresponder

Reach out to the author of react-native-gesture-handler

Using a View, without using a ref, you can use {...bind()} or similar, for now, and it works with https://github.com/necolas/react-native-web

Reach out to the author of react-native-gesture-handler

It looks like they are working on web support for react-native-gesture-handler here: Web Support #555

Looking at the react-native-gesture-handler docs and examples and it's sooo much more complex than this library for simple use cases.
React Native support would be welcome regardless of the work being done on react-native-gesutres even as an abstraction over it. The API for this library is just superior.

@danieloi That's always good to hear! Unfortunately I have zero knowledge about React Native, maybe @thephoenixofthevoid can help?


++

Overview

What this PR does / why we need it

Special notes for your reviewer
Does this PR introduce a user-facing change?

grabage pr


lb

right now I'm purging hyphens around lb, want to test if I can add them via before in html/css so default is no hyphens at all …

this in now handled in xslt


SIANXSVC-1060: Fix queues not cleaned up after an AMQPWaitTimeoutException

Celery has and requires its own TimeoutException instead of the builtin one to properly clean up py-amqp Consumers.

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

Comparison is base (c617b06) 86.66% compared to head (833a431) 87.41%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #7      +/-   ##
==========================================
+ Coverage   86.66%   87.41%   +0.75%
==========================================
  Files           3        3
  Lines         150      151       +1
==========================================
+ Hits          130      132       +2
+ Misses         20       19       -1

Files
Coverage Δ

celery_amqp_backend/exceptions.py
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

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

I guess the linter and test issues are fixed within https://github.com/anexia/celery-amqp-backend/pull/6


Make back navigate to previous scroll position in text

p sure this is because i'm not storing the <Page> scroll position in its props, so the fix is probably refactoring for that. the user can work around it by sliding the page in either direction.

closing this in favor of the everywherebible/app#2 issue


Get two warnings after upgrading to react 16-beta

Warning: GenericChartComponent.childContextTypes is specified but there is no getChildContext() method on the instance. You can either define getChildContext() on GenericChartComponent or remove childContextTypes from it.
and
Warning: GenericComponent.childContextTypes is specified but there is no getChildContext() method on the instance. You can either define getChildContext() on GenericComponent or remove childContextTypes from it.
I'm using react-stockcharts v0.7.0-beta.11

seems fixed in beta.12, close now


Imported world is different than it should be

My friends and I had two servers we played on, one survival and one creative. I decided that I'd host a server using MV to make things easier. Now that I've got MV up and running I come to find out that the creative world is a completely different world from what it should be. I had the two server hosts send me the world files and then replaced the world files generated by CB with the world files from the Survival world (world, world_nether, and world_the_end). I then placed the creative world in the server folder and named it "creative". The only world that transferred successfully was the "world" world. The survival Nether was totally different and the creative was totally different. I'm not sure if this is a CB thing or a MV thing or what exactly the deal is, anyone have any ideas?

So I figured out what happened. When I imported the world, the spawn was changed (by MV I assume). The new spawn was 1000's of blocks away from the original one and that is why I was unable to recognize the world.

That's exactly right. As far as I can tell, and from my experience, important a world doesn't necessarily keep it's spawn point in the same place, which causes confusion in cases like this. However, that may or may not be a MV problem; if what 4am said is true and MV just tells CraftBukkit to import the world rather than doing it itself, that seems like it would be a Bukkit problem. That being said, MV does create a segment in the config file when it imports a world that has all of its information, including the spawn point. If you can find out what the spawn point was before you imported it (press F3 in-game, this has the XYZ coordinate of where you are standing; stand on the spawn point and record these), then you can change it manually in the MV config file without having to fly around to find your stuff.
Good luck!

Yes, thank you for all the help I received, one last comment, setting the world to creative and then flying in a straight line from the new spawn to the old spawn takes about 2 minutes, if that tells you anything about the distance. If I had not seen the message in the console about the spawn change, I never would have found the old spawn (everything in this world was underground, the only landmarks were 6 torches surrounding a 1x2 hole in the ground where were dug down). Thanks again


Several mixins expect Subscriber and don't appear to work with Subscriber if T != Payload

For ReactiveThrift, I want to create a Subscriber<Foo>.  A common pattern for subscribers appears to be to use createManagedInstance.  This uses MemoryMixin, which defines
  void onNext(Payload payload) {

and so does not typecheck if my Subscriber defines
  void onNext(Foo payload) {

This might not be limited to just the MemoryMixin as a grep --include '*.h' Payload shows:
PublisherMixin.h:32:  void onNext(Payload payload) {
SinkIfMixin.h:21:class SinkIfMixin : public Base, public Subscriber<Payload> {
SinkIfMixin.h:29:  void onNext(Payload element) override final {
ExecutorMixin.h:50:  void subscribe(Subscriber<Payload>& subscriber) {
ExecutorMixin.h:78:  void onNext(Payload payload) {
ConsumerMixin.h:33:  void subscribe(Subscriber<Payload>& subscriber) {
ConsumerMixin.h:77:  reactivestreams::SubscriberPtr<Subscriber<Payload>> consumingSubscriber_;
MemoryMixin.h:46:  void subscribe(Subscriber<Payload>& subscriber) {
MemoryMixin.h:71:  void onNext(Payload payload) {
LoggingMixin.h:41:  void subscribe(Subscriber<Payload>& subscriber) {
LoggingMixin.h:67:  void onNext(Payload payload) {

this has been fixed.


Mandd/mc ssolver td

Comments have been addressed and test have been run on local machine.
Board is green on my MAC (see below)

Comments have been addressed and test have been run on local machine.
Board is green on my MAC (see below)

Have you tried to build the manual? @mandd

Have you tried to build the manual? @mandd

@wangcj05 : good catch, I found an error that I have fixed. Now the user_manual compiles correctly

@wangcj05 : good catch, I found an error that I have fixed. Now the user_manual compiles correctly

PR is good to me.

PR is good to me.


Update DonationSite Show to Use New Theme

Theme update
We're upgrading our bootstrap theme to use a bootstrap 4 template and need to upgrade the various pages in our site.
This issue is the update the app/views/donation_sites/show.html.erb view file to use the new bootstrap theme. When submitting your PR please include a screenshot of the completed page!
If you get stuck please refer to the following pages have been done and can be used as a reference in the new theme:

app/views/requests/index.html.erb
app/views/distributions/index.html.erb
app/views/donation_sites/new.html.erb

As this is a work an progress and we can't go live until all the pages have been converted please submit your PR against the new_theme branch.

Hi, I will update this one! :)


Request form fixes

Makes the following changes in the New/Edit Request Form:

Fix the Owners field - now it's a multi-input instead of a multi-select. Much more appropriate for the field type.
Fix many, many linting errors
Use the same field metadata style as the New Deploy form
Validate fields (eg. number fields fail if they can't be parsed as an int)
Feedback to user on which fields are preventing the form from being submitted
Use FormGroup components instead of writing out the form groups

This was branched off of new_new_deploy_form, so don't merge this until that branch is merged (and this branch is rebased onto decaf for easier diff viewing).

@tpetr @wolfd @kwm4385 This is now ready for you to take a look (and the diff is nice).

LGTM


Broccoli Merge Tree failing on Windows OS

This error is comming out when trying to build the EmberCLI app.
*Ember -v *
version: 1.13.13
node: 4.2.4
npm: 2.14.10
os: win32 x64
EINVAL: invalid argument, symlink 'D:\Workspaces\xxx\xxx\Core
xxx\xxxx\xxx\xxx-client-app\tmp\babel-output_path-oshgWKVn.tmp\xxx-
client-app\tests\integration\app-pods\components\xxxx\xxx\xxxx\component-test.jshint.js' -> 'D:\Workspaces
\xxxxx\xxx\xxx\xxx\xxxx\xxxx\tmp\bro
ccoli_merge_trees-output_path-BeD6edxd.tmp\xxx\tests\integration\app-
pods\components\xxx\xxxx\xxxx\component-test.jshint.js'
Then updated to Ember Cli v2.4.3
ember -v
ember-cli: 2.4.3
node: 4.2.4
os: win32 x64
ember build
Build failed.
The Broccoli Plugin: [BroccoliMergeTrees: TreeMerger (appTestTrees)] failed with:
Error: EINVAL: invalid argument, symlink 'file path' <- this length is 281 charachters.
Same error as before.
When the path name is more then 260 the error happens which is the max limit of the windows OS.
Is the broccoli not using the libUV beacuse i think that libUV would fix the problem.

Is the broccoli not using the libUV beacuse i think that libUV would fix the problem.

broccoli uses node, node uses libuv.

@felixrieseberg maybe you have some insight?

i just tried it the fs.writeFile with more than 260 chars and it is not working i guess it will be the same with the other fs functions.

Quick question: Which version of Windows are you using?

Win 7 Enterprise Edition and the Build Server should be on Win Server 2010 i think.

One solution could have been if the tmp folder from the broccoli build could have been moved upper in the directory tree, i dont know if currently it allows you to do it

:warning: SOLUTION :warning:
Long story short: Windows supports file paths with 32k characters, Win32 doesn't. Fix: Instruct Win32 to suspend path parsing by prefixing the path with a \\?\.
Here's a simple test:
let longPath = '\\\\?\\C:\\Users\\feriese\\Desktop\\veeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeerylong';
let i = 0;

function makePath() {
    i++;
    if (i === 10) return;

    require('fs').mkdir(longPath, (e) => {
        longPath += '\\veeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeerylong';
        makePath();
    });
}

makePath();

There's no harm in using the prefix on any version of Windows, even for short file paths. @stefanpenner: We could fix this by simply ensuring that all file operations on Windows get the prefix. Also, Windows Redstone (the next major build - the one with bash and stuff) will allow apps to opt-in into long paths and I assume that Node will do that.

@felixrieseberg ah very interesting thank you very much. (I would not have found this myself).
Does this mean, that every write/rename/unlink/mkdir path related operation in needs the prefix?

Yessir - a bunch of lower system libraries and languages use it by default, but libuv doesn't - meaning that we'd have to add it here. I can just send @joliss and you a PR if that'd help.

@felixrieseberg well if i understand correctly, literally all our FS API need to be abstracted, is that correct?
Seems like maybe we should create a alternative fs module that we start using all over the place?

Having an alternative fs module to handle this seems like a good idea, but also likely a bigger task. The root of the problem seems to be coming from creating symlinks in node-symlink-or-copy, and I haven't seen the problem coming from elsewhere (yet). Would something like the following be acceptable for now? If so I can submit a PR.
https://github.com/jasnross/node-symlink-or-copy/commit/0e4fd177d6fcf0ab9661d2eee622b3abf10e6fa7
We're currently applying this patch manually during our build process and it's working well so far.

Sure, that'd do the trick.

sweet!

@jasnross could i convince you to submit pr \w test?

Sure thing! Unfortunately I can't get to it right away, but I'll put some tests together and send it over in the next couple days.

related: https://github.com/ember-cli/ember-cli/issues/5748

should be fixed by: https://github.com/broccolijs/node-symlink-or-copy/pull/21

I resolved this error as a result of giving super user access to the command thereby it was able to create the tmp folder as is required.


Fix 32bit x86 builds and add workaround for x86_64 miscompilations by gcc 4.6 (including our Travis setup)

gcc 4.6.3 as used in Ubuntu Precise miscompiles e.g. the Sandybridge kernels with the -mavx option, leading to SIGILL in the zgemm tests (probably running out of registers)

@martin-frbg This may add unnecessary technical debt and increase the risk of (present and future) configuration bugs.
Is there any substantial practical reason to support such an ancient release of gcc?
Ubuntu Precise (12.04) is also no longer supported upstream.  It's EOL as of April 2017 (see https://wiki.ubuntu.com/Releases).  Paid "extended security maintenance" finished on April 2019.
gcc 4.6.0 was released on 2011-03-25 (roughly 10 years ago), with the last update (v. 4.6.4) released on 2013-04-12 (roughly 8 years ago).
It might be better to simply place a cut-off on the lowest version of gcc that's still in wide use.  In this case it would be gcc 4.8, which is as part of RHEL 7 / CentOS 7.  This OS is supported until 30 June 2024.

There are a few tests for gcc <= 4.7 in the build system already, so the increase in technical debt should be small (and I have gotten used to seeing ancient systems still running - though obviously shielded from the internet - in industrial settings).
Of course I might as well just update the Travis configuration once I am certain that this is the reason behind the unexpected build failures. (Travis has dropped from the PR checks overview for some reason, but it is still alive behind the badge in the README)

Is anybody likely to install new versions of OpenBLAS on these old systems?  I'd envisage that once something is "stable" on such old systems, stuff wouldn't get upgraded -- otherwise there is a risk of breaking a production system.
It might be worth cleaning out stuff related to gcc <= 4.7, which would decrease the maintenance burden.


Title: Simple Interest Calculator

Is your feature request related to a problem? Please describe.
Adding New Simple Interest Calculator.
Hey, Project admin I'm a GSSOC participant this is my second issue in your project as you tell me that create something that is not in this so I've come up with. It calculates simple interest calculator.
Thanks
Describe the solution you'd like.

Describe alternatives you've considered.

Add any other context or screenshots about the feature request here.

/assign


MAINT: misc: appease the linter in doccer.py

Try silencing the lint CI job which fails with  scipy/misc/doccer.py:28:33: W292 no newline at end of file on recent PRs.

Sorry, I already pushed the same fix in commit 54e1140f03c87. Apologies for the noise.


Incorrect offset and _count in the "previous" link of the last page

https://hapi.fhir.org/baseR4/Patient?family=wang&_count=30
Describe the bug
In the last page of the search results (via pagination), the offset and _count in the "previous" link are both wrong, unless the total number of results is a multiple of the page size (_count).
To Reproduce
Steps to reproduce the behavior:
Try this search:
https://hapi.fhir.org/baseR4/Patient?family=wang&_count=30
Click the "next link", you will see the "previous" link looks something like (note the offset and _count):
http://hapi.fhir.org/baseR4?_getpages=3b68200d-ac42-44d4-aca4-c0434cdd5f07&_getpagesoffset=19&_count=11&_pretty=true&_bundletype=searchset
Environment (please complete the following information):

HAPI FHIR Version: 5.3
OS: Windows
Browser: Chrome

Fixed by #2719


Add more example of how to use "fold"

I would like to share more example of fold in the docs which is  to "publicize" the main function. The similar technique could be used to import the "publicized function" from lib.rs into the main.rs of the library crates which is possible to accelerate the compiling speed by the power of sccache.
Example:
Suppose we have a library crates called pub_fn
Cargo.toml
[package]
name = "pub_fn"
version = "0.1.0"
edition = "2021"

original lib.rs
fn main() {
   println!("print from main fn");
}

transformed lib.rs
pub fn main() {
   println!("print from main fn");
}

main.rs
fn main() {
  pub_fn::main();
}

Thanks for the suggestion! I'll close this per https://github.com/dtolnay/syn/pull/1150#pullrequestreview-921231740.


Add colon_aliases setting

Usage:
emoji.colon_aliases = {
'1f415': 'doge',
'1f346': 'aubergine',
'1f389': 'party_popper'
}

i like the idea, but will probably do some renaming. we already internally (at slack) have a way to add custom emoji to the list (colon->image mapping) and i want to try and name stuff sanely.

I'm getting round to updating our fork, any updates on your alternative solution or shall I just keep maintaining this patch internally?

PR #56 just merged to master with an api for aliases - does that work for you?

Yup looks good, thanks!


Discussion for a new feature - allow union to be optional

I have a use case where I'd like to have a query string parameter (type) to be either a string or an array of strings if defined, otherwise allow it to be optional.
I'm pretty new to AdonisJS so if might just be me learning, but I couldn't figure out how to do it without a more complicated approach.
I tried this approach which seems reasonable:
export const getValidator = vine.compile(
  vine.object({
    type: vine.union([vine.string(), vine.array(vine.string())]).optional(),
  })
)

and I got an error that I couldn't use .optional() on a union.
So, I ended up with something like this (which I'm not sure is the best approach though it worked):
const typeString = vine.string().in(['core', 'addon'])
const typeSchema = vine.group([
  vine.group.if((data) => vine.helpers.isMissing(data.type), {}),
  vine.group.else({
    type: vine.unionOfTypes([vine.array(typeString), typeString]),
  }),
])

export const getValidator = vine.compile(vine.object({}).merge(typeSchema))

But then in the controller when I do:
const payload = await request.validateUsing(getValidator)
return await myService.getAll(params.id, payload.type)

I get the typescript error: Property 'type' does not exist on type '{} & {}'.
I would love if .union() would allow .optional() unless there is a better way that I'm not aware of.
Thanks.

It will be a nice to have feature. But something I will skip for now and come back to it later (once I have more bandwidth or use-cases)

Understandable.
Thank you for the response and thanks for everything you do with AdonisJS!


CB-12693 Configure SQL Stream Builder service for Flink clusters

thanks @ferenc-csaky great work.


[Pulsar-Broker] fetch list of topics under a namespace bundle

Motivation
we need an api to get list of topics under a specific bundle which will be useful when broker is having load issue with specific topics/bundle and we can find out appropriate list of topics belong to the specific bundle.
 Do we still need this? If we don't need this I will close this PR and remove the release/2.7.1 label.

Because this PR is opened for a long time. I will close this PR. Feel free to reopen it if you needed.


build: bump mockito & googleJavaFormat enabling to use JDK 21 for compilation

Waiting for PRs in progress to reformat sources and avoid too much conflicts


Cleanup remaining worldwide priorities

After the changes in https://github.com/alphagov/whitehall/pull/2124
there were some Worldwide Priorities which didn't get archived because
they had a newer draft edition or one which had been submitted. This
migration deletes the latest edition of the reamining Worldwide
Priorities and then archives the last published edition.

Minor point that the data migration assumes that the latest_edition isn't the published edition, but it shouldn't make any material difference when it's run as the only published editions left should theoretically be those that have a newer edition on them now that the previous data migration has been run. Extra care should be taken to check the output of the data migration to make sure none slip through the net. :+1:


Root Management Group or Custom Management Group not part of scope

Is current AzureRM provider for Terraform not allowing us to specify Root Management Group or any custom management Group to include in Scope ?

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

ID: 8e440c59-c78b-f81a-6fd1-ac5b788ab1fc
Version Independent ID: 33f85b87-2e44-d78c-5fe2-8f6d72748bc4
Content: Quickstart: New policy assignment with Terraform - Azure Policy
Content Source: articles/governance/policy/assign-policy-terraform.md
Service: azure-policy
GitHub Login: @DCtheGeek
Microsoft Alias: dacoulte

@techadmin1982 Thanks for your comment. We will review the issue and get back to you shortly.

#reassign:DCtheGeek

@techadmin1982 Thanks for the question.  The Terraform documentation for AzureRM and the Policy Resource azurerm_policy_assignment (here: https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/policy_assignment#argument-reference) indicates that only Subscription and Resource Group are available for the scope argument. If the Terraform provider for Azure Policy does support it or gains that support in the future and once that page is updated to state that, I'll update this Doc to match.  Thanks for the question!
@MicrosoftDocs/azure-cxp-triage #please-close


removing oneself from ownership does not change behavior

i removed myself from DANDI:000004 as owner.
however:

this still shows up under my dandisets
this allows me to click edit metadata

i don't know what the implications are if i try to upload data to this now - since it's really not my dataset, i don't want to.

Are there any console errors when you try to remove yourself as owner?

I don't see you as an owner on that dandiset currently: https://gui.dandiarchive.org/#/dandiset/000004/draft
Are you saying that you can successfully remove yourself, but that the options which should now be restricted, are still available?

Are you saying that you can successfully remove yourself, but that the options which should now be restricted, are still available?

yup.

Are you saying that you can successfully remove yourself, but that the options which should now be restricted, are still available?

yup.

In that case, you shouldn't be able to actually modify any data (all requests should fail), but it's still a visual bug that we should address.

I don't expect there would be console errors, as we had intended removing yourself as the dandiset owner to be a legal operation.
There are a few things going on here:

@satra  is a superuser on Girder, so he has permission to do whatever, perhaps including edit metadata (though he might just be able to go to edit screen and not save). superuser permissions always mask other permission issues and can make debugging tricky.
satra created the dandiset originally, so that is why this dandiset is returned in the my dandisets list
we haven't clearly laid out the dandiset permission model, notions of ownership, how that relates to what we track in the contributors list, and what should be returned in my dandisets list

If you are a Dandiset owner, should you be able to remove yourself as owner? Remove other owners? Remove all owners?
These would be good topics to discuss in the hackathon. I'm going to remove the severity-critical label because I don't see it having a large impact on the site operation for most users.

this is not a superuser issue - i have two accounts - my github account, which i was using is a regular account.
regarding ownership model, i thought we did say that any owner can be added/removed, which is why the manage ui behaves that way.
yes you can remove and add any owners, but the owner count has to be 1+, so you cannot remove the last remaining owner. if that's you, you cannot remove yourself.
it should not matter whether i created it or not. the my dandisets list should reflect girder owners. if i'm in the list it should show up for me, if not it shouldn't. yes, it should show up if i created it, but if i have transferred it by adding another owner and removing myself, it should no longer be my dandiset.

there are longer term things like auditing and provenance tracking of the transition, but i thought the current behavior should not be expected from the gui, since the gui should literally depend on girder ownership information about a dandiset to set permissions for that user w.r.t the dandiset. i think this requires making a call to girder each time a user lands on a dandiset or hits the my dandiset link. since others could have added/removed the user.
so at this point it is really a gui behavior issue about how it maintains state and when it queries it from the server, rather than a server permission model issue.

I believe this issue has been fixed. However, in the course of testing the behavior out, I filed two new issues (both more minor than this one): #747, #748.


User is allowed to use primitive type as package name.

Problem:
While filling the fields to create a new project, the user is allowed to set java primitives as package names. This later makes the project build fail because java doesn't allow it.
Example:

An interesting issue I hadn't considered before. Problematic for sure in the age of floating cities. gdx-setup has the same problem. inb4 TEt comes up with some regex to disallow abstract, assert, boolean, break, byte, case, catch, char, class, const, continue, default, double, do, else, enum, extends, false, final, finally, float, for, goto, if, implements, import, instanceof, int, interface, long, native, new, null, package, private, protected, public, return, short, static, strictfp, super, switch, synchronized, this, throw, throws, transient, true, try, void, volatile and while in package names.

It's annoying but not a priority in my opinion imo.

So, now this does validate to make sure none of the keywords @Frosty-J listed are used, but it also checks for the semi-reserved name _, and any Win32-reserved file names that can and have caused trouble for me before. Here's a check registering as invalid because LPT2 is a Win32-reserved name:

Good shout. As an elitist non-Windows user, I sometimes forget about the reserved filenames. The Java directory structure may be very different if COM (without a number) was restricted.

Good shout. As an elitist non-Windows user, I sometimes forget about the reserved filenames. The Java directory structure may be very different if COM (without a number) was restricted.

I didn't thought about windows reserved files as well lol. gj


'Autorip from Clipboard' option discards the first entry

Tested in 1.4.19<PHONE_NUMBER>02c51c8a47b232504ce808f0d1511a), the first entry in your clipboard is always discarded regardless of whether it's an actual address and regardless of whether it's a supported address. Having a dummy line for the sole purpose of being discarded works around it.
If it makes any difference, it's a Window$ 7 Ultimate SP1 x64 with Oracle Java SE Development Kit 8 Update 131 x64.
For example: from this list the first, http://rautamiekka.deviantart.com/gallery/, is discarded and the rest added:
http://rautamiekka.deviantart.com/gallery/
http://rautamiekka.deviantart.com/gallery/52116956/PHOTOS-cats-Rontti
http://rautamiekka.deviantart.com/gallery/34326766/PHOTOS-cats-Miru
http://rautamiekka.deviantart.com/gallery/34326811/PHOTOS-cats-Monni

, but this list's addresses are perfectly added:

http://rautamiekka.deviantart.com/gallery/
http://rautamiekka.deviantart.com/gallery/52116956/PHOTOS-cats-Rontti
http://rautamiekka.deviantart.com/gallery/34326766/PHOTOS-cats-Miru
http://rautamiekka.deviantart.com/gallery/34326811/PHOTOS-cats-Monni

I'm running 1.4.19 on Ubuntu 16.04 with openjdk and can't repo this bug.
My OpenJDK info is
openjdk version "1.8.0_131"
OpenJDK Runtime Environment (build 1.8.0_131-8u131-b11-0ubuntu<IP_ADDRESS>-b11)
OpenJDK 64-Bit Server VM (build 25.131-b11, mixed mode)

I think I repro'd this on Windows 10, but honestly this autorip feature is pretty confusing. My guess is that it has to do with Windows line endings. This will probably be a simple fix once we figure out where the problem is.

Can't repro either. Possibly Win7 or another program having run for long enough without reboots causing it.


Add new website Eventbrite for hackathons update

Related Issue
Closes #697

Description
Add tool: Eventbrite

Checklist

[x] My code adheres to the established style guidelines of the project.
[x] My changes have not introduced any new warnings.
[x] I have conducted a self-review of my code.

@JasonDsouza212 Kindly merge this


Fix parsing for redirection

Hi.
I have fixed parsing for redirection because of the following reason.

a matches method doesn't return true when a redirecting action of routes contains additional requirements
https://stackoverflow.com/questions/20165485/why-is-string-matches-returning-false-in-java

example: redirect(301, https://google.com) {:subdomain=>"foo"}

a URI can contain # as a URI fragment
https://en.wikipedia.org/wiki/URI_fragment

I'd be glad if you include this into the next release.
Thanks!

Thank you very much for the PR! I've also added tests for your case. Will be released soon.

Thank you very much for the PR! I've also added tests for your case. Will be released soon.


controller detector script

Heya Moss
First, I want to credit you as creators on itch so please could you send
your and Cam's itch names, so I can send you and invitation.
 I particularly wanted to thank you for helping me write the
abstract class I was so pleased to make it work as it was my first real use
of that concept. Afterwards on reflection I realised that you had subtly
been teaching me how to do it  and guiding me to get it right. Anyway I was
really chuffed to get it to work.
Could you also please pass on to Cam that I thought he was an absolute
machine, turning out so many quality assets in such a short time in
Blender. I have worked with 3D artists that would take 6 weeks to do what
he did, not 2 days. It was such a pleasure to work with such talented
people.
I've definitely had dreadful experiences with teams so it was so nice to
have one that ran so happily.
Sean and I were debriefing and thought there was so much functionality we
had done that didn't make it into the gamejam submission. - the random bits
deleted, the various car parts, the truck parts - Sean had a level 2
designed, and so on. We thought at some point it might be fun to complete
it, while we've both got spare time and are learning.
Anyway thanks so much it was really great.experience for me.
Cheers
Jayne
On Sat, 1 Feb 2020 at 10:40, Moss Canwell<EMAIL_ADDRESS>wrote:

You can view, comment on, or merge this pull request online at:
https://github.com/JayneGale/PitStop/pull/1
Commit Summary

controller detector script

File Changes

M Assets/Scenes/SampleScene.unity
https://github.com/JayneGale/PitStop/pull/1/files#diff-0 (18)
A Assets/Scripts/ControllerDetector.cs
https://github.com/JayneGale/PitStop/pull/1/files#diff-1 (40)
A Assets/Scripts/ControllerDetector.cs.meta
https://github.com/JayneGale/PitStop/pull/1/files#diff-2 (11)

Patch Links:

https://github.com/JayneGale/PitStop/pull/1.patch
https://github.com/JayneGale/PitStop/pull/1.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/JayneGale/PitStop/pull/1?email_source=notifications&email_token=AJ7Q5FFA37YY3MZ77UKE5QDRASLGPA5CNFSM4KON6L52YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IKIJ76A,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ7Q5FDSSOPJDBI2HGH5MBDRASLGPANCNFSM4KON6L5Q
.

No worries I'm always happy to help. It's my day job in fact!
Fruitsofdoom is my itch username and has a few of my solo game dev attempts.
GentlemanMog is Cam's profile name.
I struggled with the 48hr format but I enjoyed working with y'all too.
Cheers,
Moss
On Wed, 5 Feb 2020, 9:54 pm Jayne Gale<EMAIL_ADDRESS>wrote:

Heya Moss
First, I want to credit you as creators on itch so please could you send
your and Cam's itch names, so I can send you and invitation.
 I particularly wanted to thank you for helping me write the
abstract class I was so pleased to make it work as it was my first real use
of that concept. Afterwards on reflection I realised that you had subtly
been teaching me how to do it and guiding me to get it right. Anyway I was
really chuffed to get it to work.
Could you also please pass on to Cam that I thought he was an absolute
machine, turning out so many quality assets in such a short time in
Blender. I have worked with 3D artists that would take 6 weeks to do what
he did, not 2 days. It was such a pleasure to work with such talented
people.
I've definitely had dreadful experiences with teams so it was so nice to
have one that ran so happily.
Sean and I were debriefing and thought there was so much functionality we
had done that didn't make it into the gamejam submission. - the random bits
deleted, the various car parts, the truck parts - Sean had a level 2
designed, and so on. We thought at some point it might be fun to complete
it, while we've both got spare time and are learning.
Anyway thanks so much it was really great.experience for me.
Cheers
Jayne
On Sat, 1 Feb 2020 at 10:40, Moss Canwell<EMAIL_ADDRESS>wrote:

You can view, comment on, or merge this pull request online at:
https://github.com/JayneGale/PitStop/pull/1
Commit Summary

controller detector script

File Changes

M Assets/Scenes/SampleScene.unity
https://github.com/JayneGale/PitStop/pull/1/files#diff-0 (18)
A Assets/Scripts/ControllerDetector.cs
https://github.com/JayneGale/PitStop/pull/1/files#diff-1 (40)
A Assets/Scripts/ControllerDetector.cs.meta
https://github.com/JayneGale/PitStop/pull/1/files#diff-2 (11)

Patch Links:

https://github.com/JayneGale/PitStop/pull/1.patch
https://github.com/JayneGale/PitStop/pull/1.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<
https://github.com/JayneGale/PitStop/pull/1?email_source=notifications&email_token=AJ7Q5FFA37YY3MZ77UKE5QDRASLGPA5CNFSM4KON6L52YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IKIJ76A
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AJ7Q5FDSSOPJDBI2HGH5MBDRASLGPANCNFSM4KON6L5Q
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/JayneGale/PitStop/pull/1?email_source=notifications&email_token=AA3QPBQ6J2DCZWYPMOJVQE3RBJ5FDA5CNFSM4KON6L52YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEK2UHHA#issuecomment-582304668,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA3QPBSOKGGRIXFMYKFC6TLRBJ5FDANCNFSM4KON6L5Q
.

Cheers!
Your itch invite https://itch.io/game/accept-admin/560000/43589
Cams invite  https://itch.io/game/accept-admin/560000/2022148
On Thu, 6 Feb 2020, 10:52 Moss Canwell<EMAIL_ADDRESS>wrote:

No worries I'm always happy to help. It's my day job in fact!
Fruitsofdoom is my itch username and has a few of my solo game dev
attempts.
GentlemanMog is Cam's profile name.
I struggled with the 48hr format but I enjoyed working with y'all too.
Cheers,
Moss
On Wed, 5 Feb 2020, 9:54 pm Jayne Gale<EMAIL_ADDRESS>wrote:

Heya Moss
First, I want to credit you as creators on itch so please could you send
your and Cam's itch names, so I can send you and invitation.
 I particularly wanted to thank you for helping me write
the
abstract class I was so pleased to make it work as it was my first real
use
of that concept. Afterwards on reflection I realised that you had subtly
been teaching me how to do it and guiding me to get it right. Anyway I
was
really chuffed to get it to work.
Could you also please pass on to Cam that I thought he was an absolute
machine, turning out so many quality assets in such a short time in
Blender. I have worked with 3D artists that would take 6 weeks to do what
he did, not 2 days. It was such a pleasure to work with such talented
people.
I've definitely had dreadful experiences with teams so it was so nice to
have one that ran so happily.
Sean and I were debriefing and thought there was so much functionality we
had done that didn't make it into the gamejam submission. - the random
bits
deleted, the various car parts, the truck parts - Sean had a level 2
designed, and so on. We thought at some point it might be fun to complete
it, while we've both got spare time and are learning.
Anyway thanks so much it was really great.experience for me.
Cheers
Jayne
On Sat, 1 Feb 2020 at 10:40, Moss Canwell<EMAIL_ADDRESS>wrote:

You can view, comment on, or merge this pull request online at:
https://github.com/JayneGale/PitStop/pull/1
Commit Summary

controller detector script

File Changes

M Assets/Scenes/SampleScene.unity
https://github.com/JayneGale/PitStop/pull/1/files#diff-0 (18)
A Assets/Scripts/ControllerDetector.cs
https://github.com/JayneGale/PitStop/pull/1/files#diff-1 (40)
A Assets/Scripts/ControllerDetector.cs.meta
https://github.com/JayneGale/PitStop/pull/1/files#diff-2 (11)

Patch Links:

https://github.com/JayneGale/PitStop/pull/1.patch
https://github.com/JayneGale/PitStop/pull/1.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<

https://github.com/JayneGale/PitStop/pull/1?email_source=notifications&email_token=AJ7Q5FFA37YY3MZ77UKE5QDRASLGPA5CNFSM4KON6L52YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IKIJ76A

,
or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/AJ7Q5FDSSOPJDBI2HGH5MBDRASLGPANCNFSM4KON6L5Q

.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<
https://github.com/JayneGale/PitStop/pull/1?email_source=notifications&email_token=AA3QPBQ6J2DCZWYPMOJVQE3RBJ5FDA5CNFSM4KON6L52YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEK2UHHA#issuecomment-582304668
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AA3QPBSOKGGRIXFMYKFC6TLRBJ5FDANCNFSM4KON6L5Q
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/JayneGale/PitStop/pull/1?email_source=notifications&email_token=AJ7Q5FG32FJX24L3WU5W3MDRBMYKJA5CNFSM4KON6L52YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEK5DZUQ#issuecomment-582630610,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ7Q5FHSULZHORZQLCYRA2LRBMYKJANCNFSM4KON6L5Q
.


Cannot access wireguard client that is installed on Kubernetes cluster

I tried exposing kube api server in a private network to a public network through ec2 instance.
The wireguard client was connected well but it didn't make a handshake with the Firezone server.
And I cannot send a packet form the ec2 instance Firezone server installed to wireguard clinet.
There's some way to connect my kube api with Firezone server?

So if the client is not handshaking, that means there is no connectivity from it to Firezone. I'd double-check you can reach the expose

@seungbaeji Thinking about this a bit more -- try setting the PersistentKeepalive in the client's configuration to something like 25. That should keep the UDP session open.

@jamilbk I just saw your comments. Thanks, I will try!!


Cloud authentication failed

Description
C210
Setup the camera yesterday and everything was working fine.
I moved the camera somewhere else and when i opened HA it was asking for reauthetication. I inserted the credentials for the camera account and they were accepted. It then asked me for the Cloud password and it won't take it.
I've tried changing it, logging in again to make sure the password was correct, enabling 2FA and then disabling it... Pretty much everything I could think of.
I've also removed the integration all together and added it again. Still no luck.
Any ideas?
Reproduction Steps
Can't really tell you how to simulate it other than trying to login.
Expected behavior
I expected to be to be able to login.
If applicable, add error logs.
No response
Device Firmware
Latest
Integration Version
Latest
Using stream component
Yes
Does camera work via official integrations?
Yes
Camera has all attributes filled out in developer tools
Yes
HASS Environment
HA OS
Search for similar issues
Yes
Additional information
No response

This issue was automatically closed because it is a duplicate.
The issue you are searching for:
Is pinned under issues
Has a section in main readme file with big title right at the top
Has 3 red exclamations marks in the title
Contains a workaround both for accounts still working and the ones affected.

And a warning was sent a month ago as an update via HACS.
You can find it at https://github.com/JurajNyiri/HomeAssistant-Tapo-Control/issues/551.
Please do not post me too comments or any other comment that does not provide any value to the discussion as everyone monitoring it gets notified. If the comments with no value continue maintainer will be forced to lock the issue to contributors only.

Hi JurajNyiri,
I apologize. I scrolled through the issues page but never actually looked at the pinned topic.
Thank you very much for this integration and for your hard work.


[lightbox-media-viewer - with carousel]: QA: The label "previous page"/"next page" does not seem appropriate.

Description
Page Element:
Label for Previous/next buttons
Issue Description:
Since there is no page navigation, The label "previous page"/"next page" does not seem appropriate.
Expected:
The label can be given as "previous slide"/"next slide" respectively
User Impact:
If the label is incorrect or inadequately descriptive, it affects the screen reader user’s ability to understand the purpose of the control.
Component(s) impacted
lightbox-media-viewer - with carousel
Browser
Safari, Firefox
Carbon for IBM.com version
canary
Severity
Severity 4 = The problem is not visible to or noticeable to an average user. Affects minor functionality, no workaround needed.
Application/website
Carbon for IBM.com
Package
@carbon/ibmdotcom-web-components
CodeSandbox example
https://ibmdotcom-webcomponents.s3.us-east.cloud-object-storage.appdomain.cloud/deploy-previews/8757/iframe.html?id=components-lightbox-media-viewer--with-carousel
Steps to reproduce the issue (if applicable)
No response
Release date (if applicable)
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[ ] I checked the current issues for duplicate issues

Now Screen reader announces the label as "previous"/"next". Issue is fixed


chore: update package.json

Since we have some new changes to nitro-contract, the current sdk abi has some incompatibility with the latest nitro contract.

This is a very big yarn.lock diff for just one bump in dependencies. @Jason-Wanxt can you double-check if this is all good?

I think its good there is a new big sol2uml dependency added

@spsjvc I copy-and-pasted main's yarn.lock and ran yarn again with the new package ver and the changes are the same
can you please approve the PR and merge it


Chat – New Message button missing after navigating back from thread in room

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.24-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4886687
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:

Go to https://staging.new.expensify.com/
Log in
Open any room chat with a long history
Mark the latest message as unread
Scroll up the chat list and go into any sub-thread (Create one if needed)
Go back to the main chat

Expected Result:
New Message button appears
Actual Result:
New Message button is missing
Workaround:
Unknown
Platforms:

[x] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/da95912b-3200-428a-a126-5d08b323ad69

View all open jobs on GitHub

@garrettmknight  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

Proposal
Please re-state the problem that we are trying to solve in this issue.
"New message" does not appear when you go back from thread
What is the root cause of that problem?
https://github.com/Expensify/App/blob/685bb4c7bbaa2ef332896de5cace193edb9c6a51/src/pages/home/report/ReportActionsList.tsx#L224
The "New Message" marker appears based on unreadMarkerReportActionID. When we go back to chat from thread, the scroll position is at the bottom of chat for 1 frame and then scrolled to the message with thread on another frame. Due to the useMemo, the unreadMarkerReportActionID is not recalculated for the second frame so the "New Message" marker is not shown.
What changes do you think we should make in order to solve the problem?

Update the second argument of useMemo to include scrollingVerticalOffset.current

Can't reproduce on v9.0.25-13, closing.

Still not reproducing for me:
https://github.com/user-attachments/assets/a969f4e9-b5fe-4078-9a02-8ee182b36b21


OS X 10.10 with homebrew, error in opencv_contrib when Qt is enabled, particularly in cvv

I'm on OSX 10.10 installing via homebrew and compiling with clang. I'm running into compiler errors, which are detailed in the following homebrew issues, but turn out to be a problem with the underlying opencv_contrib:
first homebrew issue 2391
second homebrew issue 2390
Here is the specific compiler error I'm encountering:
In file included from /tmp/opencv320150617-87721-1o80hb4/opencv-3.0.0/opencv_contrib/modules/cvv/src/controller/view_controller.cpp:12:
In file included from /tmp/opencv320150617-87721-1o80hb4/opencv-3.0.0/opencv_contrib/modules/cvv/src/controller/../gui/overview_panel.hpp:12:
/tmp/opencv320150617-87721-1o80hb4/opencv-3.0.0/opencv_contrib/modules/cvv/src/controller/../gui/../stfl/stfl_engine.hpp:1056:30: error: use 'template' keyword to treat 'value' as a dependent template name
                return settings.value(key).value<QStringList>();
                                           ^
                                           template

Also note that homebrew has been updated for opencv3.0, and that is what i'm installing with the opencv_contrib components.
My specific install command in this case was
± brew install opencv3 --c++11 --with-cuda --with-contrib --with-tbb --with-qt5 --with-gstreamer --with-opengl --with-openni

Here is the related homebrew gist containing detailed compilation logs.

I have installed OpenCV3.0 by Homebrew. I would to know if it is possible to install opencv_contrib with Homebrew, and how to do it?

Hmm as far as I know OpenCV does not maintain those homebrew repos. You should rather just build the code from scratch using the provided repos.

@willard-yuan
brew install opencv3 --c++11 --with-cuda --with-contrib --with-tbb --with-qt5 --with-gstreamer --with-opengl --with-openni --with-contrib

The key is that last --with-contrib.
@StevenPuttemans I'm not asking them to fix homebrew, but rather to fix opencv-contrib. Homebrew is how I'm providing a straightforward explanation of how to reproduce the problem. The template keyword as describe in the error is clearly missing.


Add Brazilian Portuguese Translation

Please, Add Brazilian Portuguese Translation

Looks good for me (I've no idea, but google translate told me it seems legit).
Thanks, merging.


run some tests conditionally to reduce overall PR check completion time

We currently run all tests on every pull request. This is a good practice because it gives us confidence that our changes won't break anything. The downside of this setup is that even the smallest of pull requests have to run the whole test suite, including changes that don't affect the codebase, like the README or the contributing docs.
Some of our tests are still pretty slow:

[ci skip] won't work
It's common for some projects to support a pattern like [ci skip] that can be added to commit messages to exempt the commit from triggering a new test run, but there are a few problems with that:

It's kind of a dangerous pattern that can be abused by someone wishing to hastily jump the queue.
We require PR branches to be up to date with the base branch before merging, so we almost always have to "update branch" before merging. This wouldn't skip CI.
Our branch protection requires that certain checks pass. If those checks are skipped, the PR will never turn green.

what about detecting changed files?
There are lots of GitHub Actions out there for collecting a list of changed files in a pull request. This looks like a particularly nice one: https://github.com/futuratrepadeira/changed-files#readme
Using such an Action we could conditionally skip certain steps in our test suite. For example, we could skip the content test group if there have not been any changes to content/ or data/. We could also probably skip the links-and-images test in this case too.
The drawback of this approach is that we wouldn't have 100% confidence that every single PR is passing every single test, but I think that might be worth losing in exchange for a faster release cycle. 🚀
cc @github/docs-engineering

hypothetically, you could write a really simple cli to handle the conditional test suites, have that cli run the tests and aggregate the results somehow then publish them. allowing conditionality & an almost guarantee of the commit passing all the tests if the conditions are well defined

That's a good idea @KieranHolroyd! I'll explore some options on this.

I think this is solved by this! https://github.com/github/docs/commit/c76bf47ea566f47e62e50abe454e078acb8c7a3c. Cool if I close @zeke?


auth: a new auth token provider nop

This commit adds a new auth token provider named nop. The nop provider
refuses every Authenticate() request so CN based authentication can
only be allowed. If the tokenOpts parameter of auth.NewTokenProvider()
is empty, the provider will be used.
Fix https://github.com/coreos/etcd/issues/9349
/cc @stts @gyuho

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@23331f7). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master    #9369   +/-   ##
=========================================
  Coverage          ?   72.63%
=========================================
  Files             ?      362
  Lines             ?    30729
  Branches          ?        0
=========================================
  Hits              ?    22320
  Misses            ?     6794
  Partials          ?     1615

Impacted Files
Coverage Δ

auth/nop.go
0% <0%> (ø)

auth/store.go
81.81% <0%> (ø)

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 23331f7...8eb7cfb. Read the comment docs.


First line check fails with non-visible characters

The first line check against SIGNATURE can fail due to the line ending sequence being included with the response, we need to trim non-visible characters from the line to do a proper equal check.
The other option would be to actually determine proper line-endings based on OS, but I see little value in that approach.

@mcorrigan Thank you for such a well-thought fix. I am happy to merge!


Added-rl-changes

Changes
Added rl-scanner.yml file
References

https://github.com/auth0/auth0-java/pull/655/files

Closing this, as raised another PR - https://github.com/auth0/java-jwt/pull/694


[Bug]: PowerShell Core Incompatibility #379 [Feature Request]: Winget…

Update to Winget v1.5.2201
Proposed Changes
Fixes WSB and Powershell Core compatibility
Related Issues
#379 #381 #384

...or, is this a better option (from Winget-Install-GUI.ps1):
# import Appx module if the powershell version is 7/core
if ( $psversionTable.PSEdition -eq "core" ) {
    import-Module -name Appx -UseWIndowsPowershell -WarningAction:SilentlyContinue
}

Add-AppxPackage "$PSScriptRoot\Microsoft.DesktopAppInstaller_8wekyb3d8bbwe.msixbundle" fails under SYSTEM!
Deployment Add operation rejected on package Microsoft.DesktopAppInstaller_2023.808.2243.0_neutral_~_8wekyb3d8bbwe from
: Microsoft.DesktopAppInstaller_8wekyb3d8bbwe.msixbundle install request because the Local System account is not allowe
d to perform this operation.

Closing...

Should be. If this is the case, readme will be updated.


Question: is there any way to set tags?

We are planning to use elixometer to send metrics to datadog (with exometer_datadog). I know it's possible with some of datadog's libraries to set tags on metrics. For example, the my_app_api_endpoint metric could have a status_code tag to record what code the method returned, a method tag to record what the HTTP method was, and an endpoint tag to record the endpoint.
Is there a way to record tags when updating a metric, maybe with optional parameters in a call like update_spiral("metrics_test.\#{thingie}.qps", 1)? I looked through the source code, but it was not obvious whether this is supported.

Hi @shengwu, unfortunately tags are not supported when updating a metric. However you would be able to do to this by using exometer_core directly and writing a :exometer_probe with a version of :exometer_slide that supports tagging values. While this may seem very complex, I do not think it would be as difficult as it sounds because exometer_core contains a few examples of how do this, and the hardest parts can be copied from the existing :exometer_spiral and :exometer_slide.
Note that this would lose you the automatic creation/subscribing of metrics and thepobox protection that elixometer provides to prevent overloading a :exometer_probe process. However again elixometer also exposes enough to replicate this feature if you require it.

@fishcakez Thank you for responding so quickly! Glad to hear that it's possible with exometer.
By chance, would elixometer be open to a contribution adding this feature?

I'm not immediately sure about this. Likely adding a tagged spiral would
double the size of the code base 😨. It might be preferable for elixometer
to support custom metrics, similar to the update or create feature of
exometer.
On 26 Aug 2017 3:29 pm, "Sheng Wu"<EMAIL_ADDRESS>wrote:

@fishcakez https://github.com/fishcakez Thank you for responding so
quickly! Glad to hear that it's possible with exometer.
By chance, would elixometer be open to a contribution adding this feature?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pinterest/elixometer/issues/92#issuecomment-325165377,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB6JTfaaYw4RGxkm3B9nAxIgFjkky-OBks5scJxigaJpZM4PDky8
.

I have a fork https://github.com/Zorbash/exometer/commit/ac6197b203258efe696b54c33e1ceb002eb09e38 which supports DataDog tags from configuration. I think that maybe exometer_report_statsd.erl can be changes to support dynamic (per metric update) tags.

How would you express the goal to capture response time metrics along with status code with elixometer? Have a unique metric for each status code?
The information I would get is what response time is and how many and what errors are there?
Can I answer the second question with elixometer?
Thanks

Actually, the exometer_influxdb reporter already supports a feature to convert a chunk of metric as a tag: https://github.com/travelping/exometer_influxdb/issues/3
Someone might find this helpful, so I'm still posting the comment


Why can't I assign a variable more than once before using it?

I don't really get the logic behind this error: "One of the times that you gave a a value was incorrect."

I'm afraid this Github is not meant for programming advice. If you have questions about specific programming questions you are working on, then you can post on a service like StackOverflow.
As to the design decision of why we consider that an error, it's because you should never redefine a variable before you use it.


*: some fixes for release 3.0

What problem does this PR solve?
cherry-pick

[x] #1617
[x] #1616
[x] #1613
[x] #1609

Check List
Tests

Unit test
Integration test

Related changes

Need to be included in the release notes

/run-all-tests tidb=release-3.0 tikv=release-3.0 tidb-test=release-3.0

/run-all-tests tidb=release-2.1 tikv=release-2.1 tidb-test=release-2.1

/run-all-tests tidb=release-3.0 tikv=release-3.0 tidb-test=release-3.0

/ok-to-test


Document data directory

Perhaps even allow configuring the data directory by argument:
npm run serve /wow-data

Great idea. Also, I think there are more settings that could be customized. For example, there are further client data currently hardcoded in lib/client/config.coffee
So maybe a more general way to handle custom settings could be introduced? Something like, questions in CLI where the user would input settings on first run?

Ah! Very good point, had forgotten about those. Yeah, something like that would be super neat. :+1:

Documented in 3bda86d1654d434484a76550298b6ecf4067ddb7 until the configuration bit is in place.

My implementation of configurable settings can be seen here https://github.com/petersandor/wowser/commits/feature/configuration (currently it handles only data directory)
Going to rewrite that to ES6. Any feedback appreciated!

Covered by #36.


math: Use more std.math.constants in unittests

No need to recompute it when it's already cached as a constant.

Should we really be making these changes in documentation examples? The goal there should be clarity, not efficiency.

@ibuclaw as @pbackus the purpose of the documented unittests is to be as clear as possible. Using the constants require the users to do an extra look-up whereas the actual operations are self explanatory.  Also, I don't think that a few measly divisions will make a big difference when running phobos' unittests.

I've fixed the main bug anyway, these might also trigger due to logN being computed at 64-bit (instead of 80-bit).


Soul Torches on walls don't activate pact vessels

Pretty much that, soul torches can only make pact vessels give soul strain if placed on the ground, it wont work on walls.

oops
time to fix


Non replit functionality

I assume you're using replit and that's fine. However, it's not very good for hosting & if someone were to host this on their own local computer, they'd be using up useless resources for flask.

Me i only  use replit well in developing discord  bot. May you help me if possible?


[backport] Revert "Fix usage check of TensorCore"

Backport of #2197

[automatic post] Jenkins, test this please.

Closing since it's a duplicate of https://github.com/cupy/cupy/pull/2198.

Jenkins CI test (for commit f400c813a6c963f071438ea17f2977e9daf3b528, target branch v6) succeeded!


removing tabs from markdown block

Description
Removing tabs allow the markdown to be processed.
Issues Resolved
Markdown is currently processed as text because of tabbing in the template.
Contribution Check List

[x ] All tests pass.
[x] New functionality includes testing.
[x ] New functionality has been documented in the README if applicable
[x] New functionality has been documented in CHANGELOG.MD

          1 Error

:no_entry_sign:
Please add labels to this Pull Request

          1 Warning

:warning:
Should this include readme changes

  Generated by :no_entry_sign: Danger


Lift ETag.EntityTag to org.http4s

Most headers are in org.http4s.header, and most HTTP-defined values of those headers are in the org.http4s root package.

Lift the inner EntityTag class out of the org.http4s.headers.ETag and into org.http4s.
Leave a deprecated type alias in ETag pointing to the new location of the class.
Leave a deprecated val in ETag pointing to the new location of the companion object.
Update any references in the code.

This work is a binary-breaking change, and should be done on the main branch.

Hi. I'd like to pick this one.

seems this was already done quite some time ago? thanks again for picking this up!


Add Go version to prompt, only displayed when in a go source tree

Again the failure here appears to be with a previous commit in the readme. I'm not actually sure what has to be done to fix it off hand.

I will take a look on the broken master branch.
On Thu, Sep 24, 2015 at 12:38 AM, Greg Fitzgerald<EMAIL_ADDRESS>wrote:

Again the failure here appears to be with a previous commit in the readme.
I'm not actually sure what has to be done to fix it off hand.
—
Reply to this email directly or view it on GitHub
https://github.com/caiogondim/bullet-train-oh-my-zsh-theme/pull/71#issuecomment-142750100
.

Made some fixes and merged.
Let me know what you think about it.

LGTM!


[Fix] Revise channel_order in low-level vision metrics

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Refers to issue #51 .
Modification
Revise self.channel_order to channel_order in add function, and add warning when self.channel_order is different from passed channel_order.
BC-breaking (Optional)
Does the modification introduce changes that break the backward-compatibility of the downstream repos?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here, and update the documentation.
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects, like MMDet or MMCls.
The documentation has been modified accordingly, like docstring or example tutorials.

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

closed by #89


added nodeset value file for HCI

Added required variable files for NFV-HCI job

recheck

recheck

/approve

/hold till https://github.com/openstack-k8s-operators/ci-framework/pull/2292 merges

/lgtm
indeed, let's wait for the automation file split to land.

recheck


Questions about the rules in the faq

Hi. I have a some questions:

Is it allowed to add sites like the first one in the list (1kb.lejtzen.dev) ? This is a page with one link that leads to the main site with a larger size. Does the rule from the FAQ "Ultra minimal / link pages" not apply here?

If the home page contains varying amounts of content (e.g., previews of the first 5 blog articles, which may contain images), can I specify an approximate upper limit for site size?

Hi @a-samsonov-work,

In short, no. That would be classed as a "super minimal" site. There are a number of SM sites on 512KB Club, and we try to review and cull them from time to time, but, like anything, it's finding the time. :-)

I couldn't possibly specify a limit as there's too many variables. For example, a site could have thousands on paragraphs of text, with links to many posts/pages, and still be a few hundred KB. Conversely, a page could be a single image that's multiple MB. It's impossible to specify a limit, as each site is built differently. That's why we case each site as it comes.

Hope this helps,
Kev

Thank you for your answer!


Project requirements

Rails recipe: Project requirements
In this pull request, we did the following:

Followed the layout of the wireframes provided. We personalized the rest of the design including colors, typographies, spacings, etc.
Login page and registration page:

Was built with Devise.

Food list:

Display a list of food added by the logged-in user as in the wireframe.
Created a form that allows users to add new food.

Recipes list:

Display a list of recipes created by the logged-in user as in the wireframe.
Display recipe details.
If the user is the owner of the recipe, allow the user to delete it.

Public recipe list:

Display a list of all public recipes ordered by the newest as in the wireframe.
Display recipe details.

Recipe details:

Display a toggle button that allows for a recipe to be made public or private.
If the recipe is public or the user is the owner of the recipe, display the recipe details as in the wireframe.
If the user owns the recipe, show the form allowing the user to add new food.

General shopping list view:

Should show the list of food that is missing for all recipes of the logged-in user (compare the list of food for all recipes with the general food list of that user).
Count the total food items and total price of the missing food.

Corrected all the N+1.
Created a navigation menu that allows users to open all of the pages.
Integration tests

Changes made after first review

Updated all designs according to wireframe as suggested by the first reviewer.
Fixed N+1 query problem as suggested.


Fixed minor typos in install_R_packages.R

Added a couple of missing parentheses.

Changes Unknown when pulling 6ceb03193a45e8b6ed844f060db39bf35a8bcdde on sl_fix_Rscript into ** on master**.

is this good to merge, @achevali?


Add namespace support for MBeanModule

Simply create the module by 'new MBeanModule("namespace")', the exported
names in the injector will all contain namespace prefix.

@martint , this is current what's on my mind  how to add MBean namespace support. Will this work or will there be some other things I need to consider?


AggregateTagStorage unit tests

Looks nice, but we need test for remove method as well

Looks nice, but we need test for remove method as well

Done


Convolution

Coverage decreased (-1.03%) to 64.465% when pulling 20be4ed47b92216a7398490a1b72dd9462b71efc on convolution into 39e70de24c5d6e2bb775ce1e31f02b065368bb6b on master.

Coverage decreased (-0.7%) to 64.805% when pulling c777297a5d308364c47c29312c9906ce5eb9b2f4 on convolution into a2ab4ca730392ba6cf44d46a3bddb69e5d531126 on master.

Coverage increased (+0.2%) to 65.783% when pulling f010d985180e01be64caef8ca2dee225e5483631 on convolution into a2ab4ca730392ba6cf44d46a3bddb69e5d531126 on master.

Coverage increased (+0.2%) to 65.783% when pulling f010d985180e01be64caef8ca2dee225e5483631 on convolution into a2ab4ca730392ba6cf44d46a3bddb69e5d531126 on master.

Coverage increased (+0.2%) to 65.783% when pulling f010d985180e01be64caef8ca2dee225e5483631 on convolution into a2ab4ca730392ba6cf44d46a3bddb69e5d531126 on master.

Coverage increased (+0.2%) to 65.748% when pulling 14fc4b1f78eb047b1dfc3c5e02029eac8968ef9a on convolution into a2ab4ca730392ba6cf44d46a3bddb69e5d531126 on master.

Coverage increased (+0.2%) to 65.71% when pulling 932b791dc740c854cc11a8677cb9be98e1b38852 on convolution into a2ab4ca730392ba6cf44d46a3bddb69e5d531126 on master.

Coverage increased (+0.2%) to 65.752% when pulling 8b35b1883fdebab1a16fba4c680788dda799b08e on convolution into a2ab4ca730392ba6cf44d46a3bddb69e5d531126 on master.


SINGA-105 Update GUN make files for compiling cuda related code

To compile with cuda library, try:
./configure --enable-cuda
It searches the system path for cuda header and libs. If cuda is installed in other places, then
./configure --enable-cuda --with-cuda="path to cuda folder"
Do not forget to add those paths to environment variables before run configure script.
If -eanble-cuda or -with-cuda is added, then -DUSE_GPU will be added to the compilation flags.

Merged in #105 .


Use voluptuous for SqueezeBox

Description:
Migration of the configuration check to voluptuous.
Related issue (if applicable): fixes 127528299
Example entry for configuration.yaml (if applicable):
media_player:
  platform: squeezebox
  host: IP_ADDRESS
  port: 9090
  username: USERNAME
  password: PASSWORD

@persandstrom @jaharkes, would be nice if you could take a look at the changes and run a quick test. Thanks.

I have replaced my squeezeplug with chromecast audio so t I can' t help with testing. If I remember correcly username and password can be optional. Besides that I think it looks good.

Thanks. Yes, according to the docs those can be optional.

I approve this - if you want to organize the voluptuous args, please do - otherwise, merge away!

The usual order I used is Required then Optional, both alphabetical.

That's fine then
On Mon, Sep 5, 2016 at 10:21 AM, Fabian Affolter<EMAIL_ADDRESS>wrote:

The usual order I used is Required then Optional, both alphabetical.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/pull/3212#issuecomment-244781038,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC2fZVyKm-cvShpMRquBpHmbOJBAZbatks5qnEGmgaJpZM4J1Kd2
.

I don't remember that there should be any name parameter. Copy paste error?

It has no trouble validating my config, LGTM

Thanks for your feedback. Name will be removed.

@fabaff Merge once built


Integrated into qgis

Hi，It‘s really great work!
Could you please tell me how to integratethis work into qgis software?
Many thanks~

@kamen007
The products produced by samgeo can be directly imported into QGIS (geopackage, geotiff). For example, based on this notebook use
vector = "segment.gpkg"
sam.tiff_to_gpkg(mask, vector, simplify_tolerance=None)

shapefile = "segment.shp"
sam.tiff_to_vector(mask, shapefile)

For direct integration within QGIS, you may want to check out this plugin.

Thank you @BjornNyberg for sharing the QGIS Plugin. I have added it to the README.
https://github.com/opengeos/segment-geospatial#using-sam-with-desktop-gis


1430: Extend card add params

Short description
As a user i want the forms in the self service portal to be prefilled with my personal information if i click "Karte verlängern" with an existing card
Note
I'm not super happy with this solution, but since we can not generate proper functions from the frontend/build-configs/index to the buildConfig.dart to provide a function for each project to generate the applicationUrl with (or without) queryParams i didn't find a better solution.
I mean we could maybe adjust the build config generation to be able to generate functions with parameters but this means lots of effort.
At least this solution avoids multiple linting ignore that would be needed if we provide a query param key object instead of single string values that can be null. Feel free to suggest a better solution but beware that you should run the buildConfig for all the projects to ensure that nothing breaks.
Proposed changes

pass query params keys to the app, ensure that they are the same as in administration
provide a function that return the applicationUrl with queryParams
open the self service portal with queryParams if a card exists
add tests
workaround for #1802

Side effects

may create a wrong applicationUrl for other projects. so open "Beantragen" for other projects

Testing

Create a card for koblenz that is less than 90 days valid from now
Activate this card
Change the end date in "UserEntitlements" to a later date
Click on the "Karte verlängern" Button or go to more actions and extend the card there
All form fields should be prefilled with valid information
Note: I also tested "Umlaute and dots" and everything worked fine. Please also test that properly

Resolved issues

partially fixes: #1430

Tried to test this, the query params are set correctly, but the fields are not prefilled for me.

the url is: localhost:3000/erstellen?Name=Stefanie+Test&Geburtsdatum=10.06.2003&Referenznummer=123K

Interesting I tested this on ios and worked fine even this shouldn't be related to this pr

@ztefanie
i can not reproduce it on android. even dunno why inapp browser is opening maybe this is the issue

Tried to test this, the query params are set correctly, but the fields are not prefilled for me.

it seems I have the same issue on the current branch.
but on main it works if I apply this: https://github.com/digitalfabrik/entitlementcard/pull/1806
I think we need to merge it and try to update the current branch..

ok i will merge recent main when you pr was merged

now I don't understand why it doesn't work 🤔
even if I open the link with params in the desktop browser, the form is not filled.
it works on main, but doesn't work on the current branch...

Just added a couple of optional nit-picks, because overall it looks good 👍
But I'm concerned about the following:

ok i will merge recent main when you pr was merged

The form is still not being filled out for me 🤔
Even if I open the link with params in the desktop browser, the form is not filled out, so it doesn't seem to be related to the app or emulator.
It works on main for me, but doesn't work on the current branch...
Does it work for you? Is it only reproduced for me and Steffi?

Just added a couple of optional nit-picks, because overall it looks good 👍 But I'm concerned about the following:

ok i will merge recent main when you pr was merged

The form is still not being filled out for me 🤔 Even if I open the link with params in the desktop browser, the form is not filled out, so it doesn't seem to be related to the app or emulator. It works on main for me, but doesn't work on the current branch... Especially strange if it's only reproduced for me and Steffi...

Did you run npm i on root folder? Because i set the queryParam keys in frontend buildConfig and only if you run node modules installation these new keys can be found in administration module.
Maybe thats the issue. I tested the queryParam init on all browser (chrome, firefox, safari)

Did you run npm i on root folder?

Ah, no I haven't. I will be available tomorrow only from 2 pm, so feel free to merge if you want to deploy it earlier, I basically tested everything except that. I believe it should be good to go.


Blackberry error: Organize your Meteor application.

One of my users is getting the message of death: "Organize your Meteor application." each time he loads the site on his phone. He's running on a Blackberry so I'm guessing this is the issue. Anyone know about this or why it would happen?

I'm getting this too, but with a brand new app, properly configured. I'm guessing it's the recent update to Meteor 1.3 that's busting up the middleware.

I am also getting reports of one of our users getting this on a Nokia Lumia 520 running Windows 8, but as of yet I can't reproduce in an emulator and don't have access to the user's mobile.


TypeError: fsevents.watch is not a function Error & more

Improvement Idea:
This may be just me, but there is a problem when I attempt to run the site on my local machine. I receive the following error (again, this may be just me, but others may have this issue, so I thought I'd bring it up):

This may require a potential fix to the actual modules or perhaps an improvement to the documentation to avoid this issue. Let me know!

Why:
This way, we allow a more cohesive environment for people to contribute to the TS website's development.

Alternative ideas:
Not that I can think of (besides maybe improving modules and the TypeError or docs to avoid this issue). This may be just in my local machine or perhaps installation I didn't follow properly, but I thought I'd bring it up.

Is it possible you're maybe using an older version of node? We support node 13 and up on the site

I get the same error and am running node 16.
`MREM82EEECAD:nodejs kevin.lewis$ npm start

<EMAIL_ADDRESS>start
nodemon --exec babel-node src/app.js

[nodemon] 2.0.3
[nodemon] to restart at any time, enter rs
[nodemon] watching path(s): .
[nodemon] watching extensions: js,mjs,json
[nodemon] starting babel-node src/app.js
/Users/kevin.lewis/Documents/MSTeams/Microsoft-Teams-Samples/samples/app-hello-world/nodejs/node_modules/nodemon/node_modules/chokidar/lib/fsevents-handler.js:93
const stop = fsevents.watch(path, callback);
^
TypeError: fsevents.watch is not a function
at createFSEventsInstance (/Users/kevin.lewis/Documents/MSTeams/Microsoft-Teams-Samples/samples/app-hello-world/nodejs/node_modules/nodemon/node_modules/chokidar/lib/fsevents-handler.js:93:25)
at setFSEventsListener (/Users/kevin.lewis/Documents/MSTeams/Microsoft-Teams-Samples/samples/app-hello-world/nodejs/node_modules/nodemon/node_modules/chokidar/lib/fsevents-handler.js:148:16)
at FsEventsHandler._watchWithFsEvents (/Users/kevin.lewis/Documents/MSTeams/Microsoft-Teams-Samples/samples/app-hello-world/nodejs/node_modules/nodemon/node_modules/chokidar/lib/fsevents-handler.js:352:18)
at FsEventsHandler.initWatch (/Users/kevin.lewis/Documents/MSTeams/Microsoft-Teams-Samples/samples/app-hello-world/nodejs/node_modules/nodemon/node_modules/chokidar/lib/fsevents-handler.js:429:23)
at FsEventsHandler._addToFsEvents (/Users/kevin.lewis/Documents/MSTeams/Microsoft-Teams-Samples/samples/app-hello-world/nodejs/node_modules/nodemon/node_modules/chokidar/lib/fsevents-handler.js:518:12)
at processTicksAndRejections (node:internal/process/task_queues:96:5)`

Also having the same issue when updating from node 14 to 16. Any updates?

I have the same error -
/node_modules/watchpack/node_modules/chokidar/lib/fsevents-handler.js:93
const stop = fsevents.watch(path, callback);
^
TypeError: fsevents.watch is not a function
Here is my ENV:
Angular CLI: 8.1.3
Node: 16.13.2
OS: darwin x64
Angular: 8.1.3


Not Responding

I added your bot to my server this morning.  I have tried all the commands but it keeps coming back the the application did not respond.

Oh, i should have noted this somewhere long time ago...
The bot is indeed down. Since it is a pet-project and i'm just a student, i really don't want to pay for a hosting. And since Oracle has left Russia and Belarus without service two years ago, i have been searching the internet for a replacement to their "always free" instance, however didn't find anything that suited technical requirements and could store just a small SQLite database :(
So the bot is down for 13 months already and i can't even localhost it because i have my laptop with me almost always so it doesn't have persistent internet connection.
Sorry for inconvenience. I will make a note in the README. And if you know totally free hosting (that also doesn't require adding a payment method, as fly.io does, for example), please give a link.

Bot is up and running!
My old laptop is a hosting now :D

OK.  I added the bot to my server but when we start the game and confirm the game it does not start.  Do I need to open a new ticket for this?

as i see from logs, the bot has not a permission to send messages in the text channel. if you've added the bot to your server long ago (more than a year?), since then required permission could've changed so please try to re-invite the bot using the badge with link in the README.


Hubot can now hear trollicons.

instead of writing the command hubot trollicon foo hubot now listen to your trollicons in emoji format, you can now simply type :fuckyeah: and hubot will know what you meant, a fun example is :fuckyeah: trollicons, :megusta:

I approve :D


crypto/hmac: panics

Hello I have found a weird behaviour of the sha256 Reset() method with the hmac. So I have the following question.
Why does this example produce panics ?
sha256Hash := sha256.New()

salt := []byte{1, 2, 3}

hmac.New(func() hash.Hash {
	sha256Hash.Reset()
	return sha256Hash
}, salt)

panic: crypto/hmac: hash generation function does not produce unique values

But this example does not produce panics
salt := []byte{1, 2, 3}

hmac.New(func() hash.Hash {
	return sha256.New()
}, salt)

What is the difference between these examples.

This on purpose. See: https://github.com/golang/go/commit/2a206c7fcc91854a0ab78fe5799bda38dd330b11
cc: @FiloSottile

An HMAC instance needs two independent hash instances simultaneously. Your first example used to produce incorrect results if you tried to call hmac.Reset.


Reading Exif data is not supported by this PHP installation

Hi,
I have failed running test in laravel projects requiring intervention, after some digging, found out that using some of the intervention functions internally require exif to be installed.
see intervention doc for orientate function.
I was wondering if it is possible to add exif extension to default php extensions?

exact error in build:

Intervention\Image\Exception\NotSupportedException: Reading Exif data is not supported by this PHP installation. in /builds/[..]/[..]/vendor/intervention/image/src/Intervention/Image/Commands/ExifCommand.php:19

Hi @easteregg
Tks for using this project, I'm including the extension asap
What tag are you using?

@edbizarro come on man, thank you for making awesome stuff. I'm just a user.
using 7.0 now.

done!

Impressive work, perfect. I'm closing this now.


Generic conditional doesn't generate correctly for arrow functions

.d.ts generic conditional doesn't generate correctly for arrow functions

interface Y { }

function no_bug<T extends unknown = Y extends { k: infer U } ? U : unknown>() { }

const bug = <T extends unknown = Y extends { k: infer U } ? U : unknown>() => { }

Output
interface Y {
}
declare function no_bug<T extends unknown = Y extends {
    k: infer U;
} ? U : unknown>(): void;
declare const bug: <T extends unknown = unknown>() => void;

Compiler Options
{
  "compilerOptions": {
    "noImplicitAny": true,
    "strictNullChecks": true,
    "strictFunctionTypes": true,
    "strictPropertyInitialization": true,
    "strictBindCallApply": true,
    "noImplicitThis": true,
    "noImplicitReturns": true,
    "alwaysStrict": true,
    "esModuleInterop": true,
    "declaration": true,
    "experimentalDecorators": true,
    "emitDecoratorMetadata": true,
    "target": "ES2017",
    "jsx": "react",
    "module": "ESNext",
    "moduleResolution": "node"
  }
}

Playground Link: Provided

This is working as intended - in the function case, we're reusing the type nodes you wrote verbatim, while in the arrow case, we're writing the witnessed type of the expression - which, the default immediately evaluates to unknown, since there's nothing actually generic in it.


Fix typo

Following https://github.com/bazelbuild/rules_scala/pull/623 fix typo

There a few issues with the previous PR:

scala_import doesn't support srcjar attribute
It doesn't make sense to supply 2 artifacts (one for the actual code the 2nd for the source)

Hence, I'm closing this one and will submit a new one soon


New BLUI demo for UE4.26 - release

I did recently did start back to work on BLUI so in the meantime, I did decide to update the demo to the latest engine 4.26.
https://www.youtube.com/watch?v=LCY-xcj3IjA
I did create two versions one that uses MeshWidget and the other without any other plugin.
Feel free to use it for whatever you like it.
https://github.com/oivio/BLUI/releases/tag/v1
Thank you to @getnamo  for keeping it alive!

HI is it still possible to see the live inspection tools??
I've tried to use the localhost:7777, but it didnt work


Change the semantics of appliesTo fields in the schema

Rendered
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Kesha @khieta, thank you for converting issue to the RFC, it looks great!

@max2me Thanks for filing a clear issue that was easy to convert 😄

⚠ Major revisions in the latest commit! ⚠
Offline discussion on backwards compatibility revealed that this RFC was more backwards compatible then we originally thought: omitting the appliesTo field (or subfields) means that an action applies to the unspecified entity (not any entity) in all released versions of Cedar. This means that the expression principal == User::"alice" will be typed as False (not Bool) in cases where principal is unspecified. Thanks @mwhicks1 for pointing this out.

Update: re-starting discussion on this RFC 🎉 I've just pushed a major update that switches the proposal to what was originally "Alternative A". I feel that this is a more acceptable solution for users that currently have appliesTo.principalTypes: [] and/or appliesTo.resourceTypes: [] in their schemas.

I don't see why this is inconsistent with RFC24. Maybe there's something I'm missing but I don't see why RFC24 needs to map exactly 1:1 to json format. There could be syntactic sugar to it. For instance, both of the following:

action ActionGroup;
action actionGroup appliesTo { principalTypes: [], resourceTypes: [], context: {} };

Could transpile to the same thing in json:
"actionGroup": {
    "appliesTo": {
        "principalTypes": [],
        "resourceTypes": [],
        "context": {}
    }
},

could they not?

It's true that RFC24 doesn't need to map 1:1 to the JSON format: we can easily write a translator between the two syntaxes for the current proposal.
In RFC24, action ActionGroup appliesTo { principalTypes: [], resourceTypes: [], context: {} }; would not be valid syntax, since empty lists are not allowed.

I don't see why this is inconsistent with RFC24. Maybe there's something I'm missing but I don't see why RFC24 needs to map exactly 1:1 to json format.

I think we're all in agreement -- it is "inconsistent" which might be a poor UX / confusing for any folks dealing with both the JSON and RFC24 formats.  But "inconsistent" does not mean "incompatible", and we can easily translate between the two even if they do not use precisely the same syntax for some edge cases.

After much debate, we've decided to abandon this RFC for the following reasons:

The current proposal is a breaking change without a large benefit: it just disallows up a corner case in the schema format. We'd prefer a less invasive change, such as returning a warning rather than failing to parse previously parseable inputs. Cedar doesn't currently allow warnings during schema construction, but may be extended with this in the future.
This RFC only affects the JSON syntax. In general, we will recommend that users use the new schema syntax from RFC24, which will be released as part of Cedar v3.1.0.
The breaking change that we really want is RFC55, which makes the current RFC irrelevant. There is still discussion to be had on whether this new breaking change is worth it, but that can be discussed on new RFC.


Fix installation instructions in README.md

Typically $GOPATH does not contain src directory, so instructions in root README.md might be changed to like below:
mkdir -p $GOPATH/src/github.com/kubeflow
docs/installation/README.md does not contain install section for area itself, add a section and refer to root README.md.

/lgtm


Birthdate Format

The birthday command should indicate that the format of the birthday is DD/MM. Why is it in that format anyway? I forgot why.

Exactly lmao.
Day / Month / Year

Aren't we using Day/Month?


Add another way to get keys

Provide an easier way to get keys, resource ID, and endpoints.

@KuiMing : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

#sign-off

@GitHubber17 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Fix #1870: Add js.constructorOf[JSClass] to retrieve its constructor.

The method js.constructorOf[JSClass] returns the JavaScript constructor function for the class JSClass. The latter must be a class (not trait nor object) extending js.Any. It may be a native JS class or a Scala.js-defined JS class. The restrictions on what types can be used are enforced at compile-time.

Review by @nicolasstucki

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/2147/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/2773/
Test FAILed.

LGTM

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/2148/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/2774/
Test PASSed.


Handle uv_buf_t passed to write/send functions properly

Part of #397
luv_req_t now stores either one or multiple refs to the strings passed to functions that use uv_buf_t.  See https://github.com/luvit/luv/issues/397 for a more complete explanation of the problem.

Introduces a constant called LUV_REQ_MULTIREF which is an arbitrary negative value that luaL_ref will never return that signifies that luv_req_t.data is a LUA_NOREF-terminated array of ints and handles that case accordingly in luv_cleanup_req. This might be more hacky than it needs to be but it makes it possible to leave the structure of luv_req_t unchanged.
Maintains backwards compatibility in that it will still coerce numbers to strings (see https://github.com/luvit/luv/pull/426)
Factored out and moved uv_buf_t-related functions to misc.c (matches where uv_buf_t is in Libuv: http://docs.libuv.org/en/v1.x/misc.html#c.uv_buf_t)

Has the side-benefit of bringing udp_send/udp_try_send in line with the other send/write functions in that it will now accept either a string or a table of strings (before the udp send functions only accepted a single string)

This is only a draft PR because I haven't fully tested it yet: I want to make sure I didn't introduce any leaks and want to get better test coverage.
The implementation is ready for review, though, so comments are welcome.

Thanks your work, I will review it in next 10 hours.


Your MC

According to your contrastive_marching_cubes function,
the returns are vs and fs.
Does it mean your mesh is constructed without normals?

Yes. Although the normal of a face can be inferred automatically from its 3 vertices, the direction may be incorrect, resulting in an inverted normal. We didn't handle this issue in our paper.