bad pattern: CLIFrameworkCommandBase::offsetExists

➜  phpbrew phpbrew use php-7.4.28
results on:
PHP Deprecated:  Return type of CLIFramework\CommandBase::offsetExists($key) should either be compatible with ArrayAccess::offsetExists(mixed $offset): bool, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in phar:///usr/local/bin/phpbrew/vendor/corneltek/cliframework/src/CommandBase.php on line 898
PHP Deprecated:  Return type of CLIFramework\CommandBase::offsetGet($key) should either be compatible with ArrayAccess::offsetGet(mixed $offset): mixed, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in phar:///usr/local/bin/phpbrew/vendor/corneltek/cliframework/src/CommandBase.php on line 908
PHP Deprecated:  Return type of CLIFramework\CommandBase::offsetSet($key, $value) should either be compatible with ArrayAccess::offsetSet(mixed $offset, mixed $value): void, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in phar:///usr/local/bin/phpbrew/vendor/corneltek/cliframework/src/CommandBase.php on line 903
....
PHP Deprecated:  strpos(): Passing null to parameter #1 ($haystack) of type string is deprecated in phar:///usr/local/bin/phpbrew/vendor/corneltek/getoptionkit/src/Option.php on line 133
....
bad pattern: CLIFrameworkCommandBase::offsetExists

Running: MacOS 12.2.1 with PHP installed first via homebrow:
➜  php -v
PHP 8.1.4 (cli) (built: Mar 18 2022 09:45:20) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.1.4, Copyright (c) Zend Technologies
    with Zend OPcache v8.1.4, Copyright (c), by Zend Technologies

Same problem. I compiled and installed 7.4. Everything went through, but when we run which php it returns /usr/local/bin/php, the preinstalled PHP. Attempting to run phpbrew switch php-7.4.29 returns
/Users/mbx/.phpbrew/init:3: bad pattern: CLIFrameworkCommandBase::offsetExists(^[OB
/Users/mbx/.phpbrew/init:5: bad pattern: CLIFrameworkCommandBase::offsetGet(^[OB
/Users/mbx/.phpbrew/init:7: bad pattern: CLIFrameworkCommandBase::offsetSet(^[OB
/Users/mbx/.phpbrew/init:9: bad pattern: CLIFrameworkCommandBase::offsetUnset(^[OB
/Users/mbx/.phpbrew/init:17: unknown file attribute:

Also I got a lot of warnings when using phpbrew list and a bunch of other warnings. Isn't really playing nice with the latest MacOS 12.3.1 (21E258)

@peckrob Thanks for clarification. In an attempt to get PHP7 installed I decided to go that route as well. For<EMAIL_ADDRESS>you need to brew tap shivammathur/php and then you can quickly switch between versions that you don't have to compile in the first place. It's faster and more robust, since you can avoid all the necessities of building PHP by yourself.

Duplicated : #1245


Packagist not updating to 1.0.6

Still don't see the 1.0.6 release on Packagist.

Sorry about that, should be good to go now!


logging one plot per step

Here's an example of logging neural network spectra through matplotlib/ SummaryWriter.add_figure
TensorBoard gives a nice interface to scroll between them
https://app.wandb.ai/yaroslavvb/curv_train_tiny/runs/gjekrtio/tensorboard

Wandb version shows a single image, and the sizes are too large
https://app.wandb.ai/yaroslavvb/curv_train_tiny/runs/gjekrtio?workspace=user-yaroslavvb

Hey @yaroslavvb
In the past year we've majorly reworked the CLI and UI for Weights & Biases. We're closing issues older than 6 months. Please comment to reopen.


Adaptive Expressions: Add crypto functions

I'm trying to build a generic set of dialogs for working with cosmos DB over REST using adaptives HttpAction.  I can't currently generate the required HMAC Signature needed for the access token due to a lack of crypto functions in the Adaptive Expressions library.  I would imagine that any sort of sophisticated REST call is going to require a similar signature so it would be great if we could get these added...

Before we cut any new code here, I want to have a discussion first.

Hey, before we go down this route, I'd like to have a conversation first.  Tom, Steve plus SDK leadership

@Stevenic can't we create a package, and use Expression.Functions.Add() to extend Adaptive Expressions to support your scenarios?  I'm really reluctant to go down this route.

You can but they would need to be namespaced with "Crypto." and then the developer would need to tell composer to ignore anything under this namespace within their bots settings file.


More jtc tests

These changes originate from the partial-joint branch in https://github.com/ros-controls/ros2_controllers/pull/68
Let's wait until that is merged before accepting this one.

I've squashed & rebased this to the latest master, with the recent merges to master the merge algorithm was confused, there were no conflicts really.

@v-lopez I've created a backup of the old branch at pal/more-jtc-tests-orig

Codecov Report

Merging #75 into master will increase coverage by 1.95%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #75      +/-   ##
==========================================
+ Coverage   33.12%   35.07%   +1.95%
==========================================
  Files          12       23      +11
  Lines        1123     2141    +1018
  Branches      740     1340     +600
==========================================
+ Hits          372      751     +379
- Misses        103      195      +92
- Partials      648     1195     +547

Flag
Coverage Δ

#unittests
35.07% <ø> (+1.95%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...include/joint_trajectory_controller/trajectory.hpp
46.66% <0.00%> (-20.00%)
:arrow_down:

...int_trajectory_controller/test/test_trajectory.cpp
20.95% <0.00%> (-5.43%)
:arrow_down:

.../joint_state_controller/joint_state_controller.hpp
100.00% <0.00%> (ø)

...jectory_controller/joint_trajectory_controller.hpp
100.00% <0.00%> (ø)

...ontroller/test/test_load_diff_drive_controller.cpp
14.28% <0.00%> (ø)

...troller/include/diff_drive_controller/odometry.hpp
27.27% <0.00%> (ø)

...rs/diff_drive_controller/test/test_accumulator.cpp
57.14% <0.00%> (ø)

...controllers/diff_drive_controller/src/odometry.cpp
68.42% <0.00%> (ø)

...diff_drive_controller/rolling_mean_accumulator.hpp
85.71% <0.00%> (ø)

...ive_controller/test/test_diff_drive_controller.cpp
20.91% <0.00%> (ø)

... and 10 more

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 3c6c176...938d49d. Read the comment docs.

Thanks for the review.
I cannot address thisuuntil the week after the next one.
In https://github.com/ros-controls/ros2_controllers/issues/49 I am writing down the ROS1 tests in each file, I believe these cover just 3 files, there's a bunch more. I tried to keep below the 500 line limit suggestion

Should I keep adding tests to this PR or continue on a separate branch?

@v-lopez I think this is a good amount of new stuff to be merged, please open a new PR with additional tests.

Will do!

@bmagyar I've rebased this and linters and tests are passing on my machine. Should be good to go when the actions finish.


Unhandled promise rejection: Error: getaddrinfo EAI_AGAIN gateway.discord.gg

Hi!
My bot have been woking as it should for abt a week, but yesterday night, my bot keeps turning off by itself. It is coded in replit, and hosted on uptimerobot.
But now i get this error when i start up my bot
Unhandled promise rejection: Error: getaddrinfo EAI_AGAIN gateway.discord.gg
at GetAddrInfoReqWrap.onlookup [as oncomplete] (node:dns:82:26) {
errno: -3001,
code: 'EAI_AGAIN',
syscall: 'getaddrinfo',
hostname: 'gateway.discord.gg'
}
What is the problem, and how do i fix it?

My bot have been woking as it should for abt a week, but yesterday night,
my bot keeps turning off by itself. It is coded in replit, and hosted on
uptimerobot.
But now i get this error when i start up my bot
Unhandled promise rejection: Error: getaddrinfo EAI_AGAIN
gateway.discord.gg
at GetAddrInfoReqWrap.onlookup [as oncomplete] (node:dns:82:26) {
errno: -3001,
code: 'EAI_AGAIN',
syscall: 'getaddrinfo',
hostname: 'gateway.discord.gg'
}
What is the problem, and how do i fix it?
—
Reply to this email directly, view it on GitHub
https://github.com/CorwinDev/Discord-Bot/issues/221, or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6DKCGAH5SI6PGUKRW6DO33YM6HRDAVCNFSM6AAAAABBN6E64OVHI2DSMVQWIX3LMV43ASLTON2WKOZSGA3DMNZWGUYTKMA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I tried to port it to an other hosting, but it just messed it up. Replit is really bad :/


Update postgres.md - Add NServiceBus.PostgreSQL.Transport  to libraries

NServiceBus.PostgreSQL.Transport allows Postgresql to be used as a message broker for use with NServiceBus

Hi,
Thanks for your contribution.
This link is related to NService bus and it is not directly related to postgres.
I think it is better you add https://github.com/Particular/NServiceBus in the Libraries section in the microservices/tools/nservicebus page, because NServiceBus has alot of nugets and having main repo is enough.


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 16:12:29 on 2023.05.03
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 4, errors out: 0
Original problem ID: 20261
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=20261

Problem has been resolved in 6m 0s at 16:18:29 on 2023.05.03
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 20261


Null provider message isn't specific

I'm testing my module using mocks. I'm not supplying a particular property (somewhere). I get the following error message:
com.google.inject.CreationException: Unable to create injector, see the following errors:

1) Binding to null instances is not allowed. Use toProvider(Providers.of(null)) if this is your intended behaviour.
  at com.authzee.kotlinguice4.internal.KotlinBindingBuilder.toInstance(KotlinBindingBuilder.kt)

1 error

Please ensure you are naming the null property in this message, otherwise it's extremely hard to track down.

There are no implementations of toInstance in kotlin-guice. All toInstance methods that are part of the kotlin-guice API directly delegate to the Guice implementation of toInstance, which does not add the null property name to the errors.
Unless I'm misunderstanding the issue (a Minimal Complete Verifiable Example would help), this seems like something that would need to be changed in Guice

Why does it mention a kotlin-guice class (and only that class) in the stack?
You can reproduce this by passing in mocked resources into a module unit test.

It looks like Guice uses the method #getElementSource() to get the calling line for toInstance bindings when there are errors.
It might be possible to get a better error message in this case by adding some more exclusions to the error source inclusions like this: https://github.com/authzee/kotlin-guice/blob/master/kotlin-guice/src/main/kotlin/com/authzee/kotlinguice4/KotlinModule.kt#L58

Fixed by #13


Assertion error for pmap out_axes with nested None value

Dear JAX team,
is the following no longer considered a reasonable pattern? I could swear this used to work and found it quite concise. Would be nice to have it back.
Is it more or less equivalent to have an dimension reduction outside of the pmap?
import jax
import jax.numpy as jnp
import functools as ft

@ft.partial(jax.grad, has_aux=True)
@ft.partial(jax.pmap, in_axes=(None, 0, 0), out_axes=(None, 0), axis_name="pmap")
def loss_fn(params, X, y):
    yhat = params @ X
    mse = jnp.mean((y - yhat)**2)
    return jax.lax.pmean(mse, "pmap"), yhat

Thanks for raising this.
pmap's out_axes=None was only ever partially implemented, and I don't think reverse-mode autodiff over it was ever implemented. It could be, though! (It works with vmap, and in `pmap with things other than reverse-mode, so maybe you were remembering one of those?)

Is it more or less equivalent to have an dimension reduction outside of the pmap?

When the output of a pmap function is equal across all mapped instances of the function, then out_axes=None means "just choose one of these (equal) values". That is, when all values are equal pmap(f, ..., out_axes=None)(xs) == pmap(f, ..., out_axes=0)(xs)[0]. I don't expect using out_axes=None to be any more efficient either, though I could be wrong.
So you might consider just doing the slicing yourself, i.e. replace any None entries of out_axes with 0, and then pick any index along those axes of the output.
Just a few days ago we've started landing an upgrade to pmap called shmap (or shard_map); you can read the design doc here. It has an analogue of out_axes=None, basically having an out_spec entry which doesn't mention one of the mesh axis names. Not only does that work (with reverse-mode AD), it's also typically more efficient than pmap. In particular, with shmap not mentioning a mesh axis name in an out_spec basically switches the output to have a replicated layout along that mesh axis, without having to move or delete any buffers.
I expect we'll ultimately implement pmap in terms of shard_map, and at that time out_axes=None will probably start working. But until then I don't think we plan to work more on pmap.

Ha, you are probably right. I just tried it in an old conda env with jax=0.2.26. The assertion is the same there.
Just as a side-note, the assertion is a little unhelpful; maybe it would be better to simply print out that currently pmap does not support None in out_axes but either way its a minor point.
That's really interesting, thanks for the link. Especially the pjit sounds like a nice addition -- i really don't have to right the parallel logic myself ;)
Anyways, thank you for working on this amazing piece of software, and feel free to close this any time.

Thanks for the kind words :)

Just as a side-note, the assertion is a little unhelpful; maybe it would be better to simply print out that currently pmap does not support None in out_axes but either way its a minor point.

 Let's do that.

After working on this for more than a year, #22585 will finally fix it.


Connector update in-place on failed connector does not fix it. Manually applying the same creds does.

Hey team, Ive recently updated our connectors in-place with the new TF version(1.1.0) but the AWS AK/SK were longer valid. I added them to our TF process with Vault Key Value v2 and though it registers the change and applies them they do not switch back to running. I then took the same creds and applied directly to the connector manually in UI and it will then self-heal. Im curious if anyone else has encountered this issue. We're working through a process to automate the credential rotation - on a side note, happy to hear any thoughts on best case for this - but currently just waiting til they break while testing unless I remember t do it sooner.
Currently Im getting this:
module.sqs_dpe_streams["https://sqs.us-west-2.amazonaws.com/{acct-id}/feedback-sqs-queue-dev"].confluent_connector.sqs_source will be updated in-place
~ resource "confluent_connector" "sqs_source" {
~ config_sensitive    = (sensitive value)
id                  = "lcc-o3622j"
# (2 unchanged attributes hidden)
    # (2 unchanged blocks hidden)
}

The bolded section say to me that it should change the values and my assumption is that it should self-heal after that. Is there something else that needs to happen for it to do that?
The error after the change shows this still from UI: Failed
Unable to validate configuration. If an update was made to the configuration, this means that the configuration was invalid, and the connector continues to operate on a previous configuration that passed validation. Errors:
aws.access.key.id: The provided AWS Access Key is invalid
The one that was added manually has no errors and is in running status.

Thanks for creating the issue!
Could you describe the process of updating credentials manually via the UI? More specifically, let's say a connector was in RUNNING state and then you updated aws.access.key, was there an update of status or something? Looks like you mentioned

applied directly to the connector manually in UI and it will then self-heal.

cc @nick-osborn

Thanks for the reply! Let me ask a couple of other questions:

the AK/SK change

what "AK / SK" stand for here?

When I manually change the connector, the status begins in failed

That's interesting, I think the expected behavior of a connector is to continue to be in RUNNING state unless the credentials are invalid.
But yeah overall I don't think we specifically tested the credential rotation workflow for connectors so we might want to test it on our side too.

Thanks for the reply! Let me ask a couple of other questions:

the AK/SK change

what "AK / SK" stand for here?
AWS access key/secret key

When I manually change the connector, the status begins in failed

That's interesting, I think the expected behavior of a connector is to continue to be in RUNNING state unless the credentials are invalid.
Sorry, yes they start in running on the initial resource creation, but after the TTL for the AK/SK creds ends and they are invalid then they stay in failed state until manually changed.
But yeah overall I don't think we specifically tested the credential rotation workflow for connectors so we might want to test it on our side too.
That would be great. We've been discussing the best way to manage this somehow as we're using vault to manage creds in the terraform pipeline build but we'll need to make sure that if we use the same method to update the creds that the cred update process is solid from TF-perspective.
Ideally we would have another mechanism(thinking possibly lambda/k8s cronjob as a work-around) that isnt tied to connector/topic/etc resource creation for rotation but I know your team is already really busy with ksql/sr mgmt for terraform(saw a few ksql updates on api-side... exciting :) ).

Ill also let you know once ive had a chance to test on the 1.1.0 version to see if there was something there that I missed.

^Sorry... trying to figure out the best way to format this

Based on this code snippet it doesn't look like 1.1.0 support updates for sensitive attributes in config_sensitive block so we'll definitely need to update a source code.
Luckily, the change seems to be pretty simple so I might be able to test out a change and ship it in our new release shortly.

Thanks so much for digging in and finding that Kostya! I hadnt yet had a chance to test it out but ill hold off until the new change comes up since you were able to verify.

@nick-osborn we've just released 1.2.0 that hopefully should fix the issue, could you test whether it helps?

Will do, thanks!

@linouk23 This is fixed! I did a manual break and then pushed new creds with Terraform 1.2 to fix and it worked as expected. Thanks Kostya


license is invalid!!!

Hi Im planing to use this player only for play local videos. Do I need to pay for it?
When I go to the page -> https://help.aliyun.com/document_detail/434250.html.
They suggested that <-> connect Log in to the Smart Media Services Console.
But when I click activate button I get this error

Any links how to create license for this player? I could find anything. I searched about ApsaraVideo Player SDK but it seem for remote playing, and there you need to configure domains and so on to obtain a license, but I don't need it, I only need to play local videos. Please help me.

如果你只是播放本地视频，是不需要申请license的。播放功能是免费使用的。


add usage guide and mac os x compilation guide to README

I think it's better to add some Mac OS X compilation and usage guide to README file, and a new source release for the ease of add it to homebrew formula.

Technically, those are requirements for Linux too and not just OS X. Can you update it to reflect that?

It does not seem to work on linux. I try to use make but I get a relocation error.

My bad. It works once I uninstall java jdk.


Admin created languages do not work with the built in document generator

Description:
When creating a new language for example be-FR that inherits from fr-FR.json and when you create a snippet set for this language you're unable to use this language for document generation on orders, when not matching the original language, which is french, because of the base json file.
For example, when creating an invoice in belgium the snippet will not get translated, since the snippet set is based on french. To actually fix this you have to change the iso column for this snippet set in the snippet_set table in the database and change it to the iso code/locale you wan, which matches the sales channel domain configured language.
Also, if the field is not filled after doing the database fix(even though it is inheriting from fr-FR) it will not fallback to fr-FR and you need to change the snippet, since the “file to snippet set” relation is broken.
Possible Solution:
This probably happens, because the documents check for the language AND the snippet set. This check is unnecessary, since it should only use the selected snippet set for document generation.
In this regard, it the use of the iso code in the snippet_set table should be checked, to either remove/deprecate this field or make it configurable.
Environment:
[https://www.tomandco.com/|https://www.tomandco.com/]
Steps to reproduce:

Create a language as target language (Belgium)
Create a snippet set with a json as a base, meant for a diffent language (Belgium, using fr-FR.json)
Configure your domain with both selections mentioned above
Create an order in that language/domain via the storefront
Try and generate an invoice or another document

Expected result:
The document is translated in the language it was created in, using the correct snippet et
Actual result:
The Document uses a snippet set associated to the language, not the snippet set itself

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/NEXT-33082.
[created from NEXT-33082, comment 486121]

I can reproduce the issue in Shopware <IP_ADDRESS>. The workaround, (in my case: chagning de-de to de-at for the AT snippet_set), works as described.

[public] Hello,
Within our company roadmap and work capacity, we try to address each bug or improvement request but admit that not each one will be resolved. To continue our culture of honesty and openness, we are closing this issue to focus on our roadmap on behalf of all Shopware users.
If you feel like this issue is still impacting you, please create a new issue and let us know. Thank you again for your valuable feedback.
[created from NEXT-33082, comment 501292]


Upgrade python crytpography module

Desired Outcome
Use secure versions of cryptography module.
Implemented Changes
Updates the cryptography version in requirements.txt / setup.py to 39.0.1. I also updated pyopenssl to 23.0.0 while I was making changes.

Huh it does. Looks like main's been failing in that stage for a few weeks. I'll take it out. Thanks, @gl-johnson!


[Feature Request] ACE archive support

ACE archive support would be nice, especially for version 1 archives as that seems to be better documented. The ArchiveTeam wiki has good resources for ACE archives, including a link to example files and software that is able to extract it. http://fileformats.archiveteam.org/wiki/ACE

I just discovered that there's a newer version of unace that's seemingly available that claims to be able to extract newer ACE versions, though I haven't personally confirmed that it works: https://github.com/wdlkmpx/unace


RELEASE: bvecvx unlock handling, digg aura vault auto keep

Codecov Report

Merging #288 (04f3e11) into main (7badb32) will decrease coverage by 0.55%.
The diff coverage is 10.00%.

@@            Coverage Diff             @@
##             main     #288      +/-   ##
==========================================
- Coverage   39.78%   39.23%   -0.56%
==========================================
  Files          19       19
  Lines        1518     1547      +29
==========================================
+ Hits          604      607       +3
- Misses        914      940      +26

Impacted Files
Coverage Δ

src/earner.py
0.00% <0.00%> (ø)

config/constants.py
100.00% <100.00%> (ø)

src/web3_utils.py
60.00% <100.00%> (+0.44%)
:arrow_up:


Add a "thumbs-down and skip this" thing...

It would be nice to add a 'thumbs down & skip' options. Maybe 'hold a button down when skipping' feature?

Added capability in c5d05011fa1fc04480c66eb994b69a87934c5ca2 :)


pd.NaT.date() returns datetime.date(1, 255, 255)

Hello,
pd.NaT.date() returns datetime.date(1, 255, 255)
that's an odd results
I was expecting None (or something else) because that's really strange to have
date of year "1" in an Excel file
Kind regards

Could we make NaT.date() just raise a ValueError? I think I would prefer that over None, which would usually just mean differing the error until later.

Yes raising an a ValueError is more understandable.

If you'd like to put together a PR to fix this, that would be great!

Yes but where is defined Not A Time.
What kind of behaviour should we have ? Does getting any attribute / applying any method should raise an error ?

NaT is defined by the NaTType class here: https://github.com/pydata/pandas/blob/bceb342d76ca4061fc8d3a1089bc4e83a32da656/pandas/tslib.pyx#L568
NaTType is an indirect subclass of datetime, so it inherits the default date method. You simply need to supply the method on NaTType, e.g.,
def date(self):
    raise ValueError('some helpful message')

(and yes, this is Cython, but you write pure Python code here.)
This will also need a test to verify that the error is raised, probably somewhere around here:
https://github.com/pydata/pandas/blob/bceb342d76ca4061fc8d3a1089bc4e83a32da656/pandas/tseries/tests/test_tslib.py#L589

If there are other methods that are not returning appropriate values, feel free to override them to raise an error as well. But most of them should be supplying appropriate returns values already (e.g., nan for NaT.year).

I directly code on GitHub and send PR this way (because I don't have a Python install here)... I just hope that Travis CI will be ok.

Sorry but it seems that my PR fails with CI.

PR https://github.com/pydata/pandas/pull/9523 seems ok now

Probably related:
When writing the NaT to postgres using SQLAlchemy:
sqlalchemy.exc.DataError: (psycopg2.DataError) invalid input syntax for type timestamp: "0001-255-255T00:00:00"
Although I thought this was fixed a while back
https://github.com/pydata/pandas/issues/6701
https://github.com/pydata/pandas/issues/2754

@kelvin22 Can you open a separate issue for this (because as you say, this should be fixed). If possible, please provide a reproducible example.

should be closed too?


Success check in M2MNsdlInterface::create_nsdl_list_structure() checks only success of last call to create_nsdl_structure()

success is assigned multiple times at
https://github.com/ARMmbed/mbed-cloud-client/blob/9b0bc6a2f3f1f5dbb5be1827db83004d531b99c9/mbed-client/source/m2mnsdlinterface.cpp#L375
There is no early return on fail. Thus only the last assignment will be returned. This assumes that all following calls after an initial fail will always fail. Was not the case in our codebase.

FYI: @MarceloSalazar @karsev

ARM Internal Ref: IOTCLT-2906

Thanks for your report. This has now been fixed and the change will be available in next release.


Provide more details about where the "Endpoints" page is

The sentence "Retrieve these values from the Endpoints page in your Azure AD tenant." is really confusing and does not tell you where the "Endpoints" page is.  It took me about 40 minutes of browsing around all parts of AAD to find it.

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

ID: 823eb96f-e5f1-0832-b175-0b31009be92f
Version Independent ID: 330085ae-dcf6-e930-86a8-0e4a60ab1314
Content: Protect API backend in API Management using OAuth 2.0 and Azure AD - Azure API Management
Content Source: articles/api-management/api-management-howto-protect-backend-with-aad.md
Service: api-management
GitHub Login: @miaojiang
Microsoft Alias: apimpm

Never mind - misunderstood the text.

rumenavramov,  No worries, thanks for leveraging Azure docs feedback channel. Glad the concern was figured out. Much appreciate the follow-up.


tlsglobals on Linux only works with GCC and Clang

Original author: Tarun Prabhu
Original issue: https://charm.cs.illinois.edu/redmine/issues/1205

When compiling AMPI with clang and passing the -tlsglobals flag, charmc attempts to pass the -mno-tls-direct-seg-refs flag which is currently unsupported in clang. This results in a build failure. If the option is omitted, tlsglobals may work correctly under clang, but it is guaranteed to fail under either compiler if built with -O3 optimizations.

Original date: 2017-01-11 01:57:15

This missing support is known to Clang/LLVM since 2013. I added a comment about AMPI's desire for this support: https://llvm.org/bugs/show_bug.cgi?id=16145

Original date: 2017-02-24 19:37:41

On a related note, Clang's -femulated-tls option may be useful for us.

Original date: 2017-02-24 23:07:05

So, first note is that on LLVM (as of today) the flag in question is only relevant for the "initial exec" and "local exec" TLS models. So, if we're compiling for PIC, then the flag is simply unnecessary.

Original date: 2017-02-24 23:15:40

Nope, never mind. No matter what TLS model is in use, it generates instructions that access an offset from the segment register, rather than loading fs:0 to a register and adding to that.

Original date: 2017-12-29 17:51:53

Mass re-assign AMPI-related issues on my plate to Sam, for subsequent redistribution.

Original date: 2017-12-29 17:52:15

Mass re-assign AMPI-related issues on my plate to Sam, for subsequent redistribution.
For real this time.

Original date: 2018-10-18 14:39:37

LLVM now has support for the "-mno-tls-direct-seg-refs" flag in the development version: https://bugs.llvm.org/show_bug.cgi?id=16145
This should ship in Clang v8.0.0. Currently, charmc warns users that tlsglobals might not work if using anything besides GCC.

As Phil mentioned, this is the effect -mno-tls-direct-seg-refs has on the following example code:
static volatile thread_local int ThreadLocal = 1;
int GetThreadLocal()
{
    return ThreadLocal;
}

With -mno-tls-direct-seg-refs:
GetThreadLocal():
        movq    %fs:0, %rax
        movl    ThreadLocal@tpoff(%rax), %eax
        ret

Without -mno-tls-direct-seg-refs:
GetThreadLocal():
        movl    %fs:ThreadLocal@tpoff, %eax
        ret

This allows us to swap out the entire thread-local context by poking a single pointer. Technically the functionality does not require this, but swapping would become linear in the number of thread-local variables.
It is possible that the FSGSBASE instruction will eliminate the need for this compiler option once it is commonly supported by kernel versions in production. https://www.phoronix.com/scan.php?page=news_item&px=Intel-FSGSBASE-Linux-5.7
On macOS, TLS-Globals and thread-local storage in general works slightly differently. Instead of indexing into the TLS segment for each variable, indexing happens per shared object. For example, the privatization test swaps two pointers during each context switch: one for the test program, and one for a dependent shared object that it loads. The -mno-tls-direct-seg-refs option is present in GCC and Clang on macOS but appears to be a no-op.
The -mno-tls-direct-seg-refs option is detected at configure time and any compiler that adds support for it will automatically benefit.


Moves to GitHub actions build

reuses scripts used in other projects like zipkin-storage-kafka
test images are reoganized under [docker/test-images]
[build-bin/test] runs Maven tests, and builds and tests zipkin and zipkin-slim Docker images.

[docker/test-images] are tested in parallel in GitHub actions
[build-bin/docker_push] still pushes test-images, but only to ghcr.io

[build-bin/djavadoc_to_gh_pages] pushes Javadoc to the gh-pages branch on N.M.L branch
[build-bin/dmaybe_install_npm] is used to build [../zipkin-lens] on an unsupported node.js architecture.
[build-bin/dmaven_go_offline] additionally seeds the NPM cache

Main thing here is that the build-bin/*/** files will be common across projects. It was very hard to make them work for everything, but it is now the case. As mentioned in the README, files directly in build-bin/ are project specific, while directories like build-bin/docker are not.
hopefully the tests pass.. this is the first time we are using GH actions to actually run the docker storage tests. Before, we only built the server in GH actions, not ran heavy tests inside it.

ok all works. I'll refactor the tests slightly back as they were for merge.


feat: group dirs and files in file_browser

As per reddit request. This PR introduces the grouped option which initially groups entries in the file_browser by directory and files and only then sorts alphabetically. I would probably mark this optional as this is behavior that cannot be easily achieved with an fd finder, but happy to be convinced otherwise.
fd is required for depth=false and fast folder browsing. For classic file_browser, plenary.scandir is already plenty fast.
Before:

After

I seem to be getting the sorting in the "preview" pane, but not in the "fuzzy finder" section on the left of the screenshot.

Am I doing something wrong? I'm adding this as my setup:
telescope.load_extension("file_browser", { grouped = true })

And calling the extension with the following:
vim.api.nvim_set_keymap("n", "<leader>ff", ":Telescope file_browser<cr>", { noremap = true })

Hi!
You can set it by defaul:
require"telescope".setup {
  ...
  extensions = {
    file_browser = {
      ...
      files = false,
    }
  }
}

Via the vim command:
:Telescope file_browser grouped=true
Or with the lua API:
:lua require"telescope".extensions.file_browser.file_browser( { grouped = true } )
Read the readmesection  for more info :)

Thank you @ikerurda, the key maps worked great.


请问用的什么操作系统呢？

ubuntu、centos还是windows？

应该都可以

Unknown platform: darwin

@buttercannfly 请问有更详细的报错信息吗

Explicitly passing a revision is encouraged when loading a model with custom code to ensure no malicious code has been contributed in a newer revision.
Explicitly passing a revision is encouraged when loading a configuration with custom code to ensure no malicious code has been contributed in a newer revision.
Explicitly passing a revision is encouraged when loading a model with custom code to ensure no malicious code has been contributed in a newer revision.
Unknown platform: darwin
Running on local URL:  http://<IP_ADDRESS>:7860
To create a public link, set share=True in launch().

To create a public link, set share=True in launch().
Explicitly passing a revision is encouraged when loading a model with custom code to ensure no malicious code has been contributed in a newer revision.
Explicitly passing a revision is encouraged when loading a configuration with custom code to ensure no malicious code has been contributed in a newer revision.
Explicitly passing a revision is encouraged when loading a model with custom code to ensure no malicious code has been contributed in a newer revision.
Unknown platform: darwin
@imClumsyPanda

check https://github.com/THUDM/ChatGLM-6B/issues/6


[ccat] refactor cat context

Reduce the usages of g_cat_context, which is a global variable.

@wangyapu0714


Do not install global-scope tests package

Modify setup.py to prevent it from installing the tests directory as a global-scope Python package.

thanks 😃

Thank you!


CURLOPT_FOLLOWLOCATION  trouble with youtube

When open a youtube video page, it shows this
Warning: curl_setopt(): CURLOPT_FOLLOWLOCATION cannot be activated when an open_basedir is set in /wwwroot/vendor/athlon1600/youtube-downloader/src/YouTubeDownloader.php on line 110

your server is not configured properly. Not something I can fix from my end.
https://stackoverflow.com/questions/6918623/curlopt-followlocation-cannot-be-activated


snet: fix UDP port selection on Windows

SNET is checking the wrong error code when probing for available UDP ports.
Windows socket error codes are completely different from Linux, thus different OSes must be distinguished.

This change is


Removing some deprecation warnings

Most of these changes were based on "inline" refactoring of deprecated methods.

A lot of the files touched are autogenerated, is that ok?

@igorbernstein2, thanks for the catch.  I did a blind refactor.  We'll have to change the generator.


Intermittent failure in activity API

See https://travis-ci.org/oaeproject/Hilary/builds/61470610
1) Activity Activity Stream Aggregation verify aggregated data is automatically deleted after the idle expiry time:
     Uncaught AssertionError: "undefined" == true
      at /home/travis/build/oaeproject/Hilary/node_modules/oae-activity/tests/test-activity.js:1862:40
      at /home/travis/build/oaeproject/Hilary/node_modules/oae-activity/lib/internal/dao.js:9:12760
      at b (domain.js:183:18)
      at try_callback (/home/travis/build/oaeproject/Hilary/node_modules/redis/index.js:592:9)
      at RedisClient.return_reply (/home/travis/build/oaeproject/Hilary/node_modules/redis/index.js:685:13)
      at HiredisReplyParser.<anonymous> (/home/travis/build/oaeproject/Hilary/node_modules/redis/index.js:321:14)
      at HiredisReplyParser.emit (events.js:95:17)
      at HiredisReplyParser.execute (/home/travis/build/oaeproject/Hilary/node_modules/redis/lib/parser/hiredis.js:43:18)
      at RedisClient.on_data (/home/travis/build/oaeproject/Hilary/node_modules/redis/index.js:547:27)
      at Socket.<anonymous> (/home/travis/build/oaeproject/Hilary/node_modules/redis/index.js:102:14)

Assigning to @mrvisser for investigation

Uploaded "tests.log.gz" to Amazon S3: "oae-testing/travisci/logs/hilary/build-2408-f978f610da91fe10f1b23ee8c736e28e8bcd0e51-Wed May 06 2015 14:49:13 GMT+0000 (UTC).log.gz"

I feel like I could rip out aggregation entirely in the amount of time it takes me to debug something like this :(

Added better messaging if this issue happens again. To @simong for review.


UTF32Encoder.GetBytes does not set index in EncoderFallbackException

When encoding the sequence "\u00C4 \uD802\u0033 \u00AE" the index of the high-surrogate character D802 should be set in the EncoderFallbackException. In this case it should be index 2.

Closing this as it is replaced by issue #70.


Update .travis.yml

This change is

Coverage remained the same at 82.305% when pulling 13f6618256626d104ac446865059979d2f44522f on justin808-skip-docker into 13f2cf0e83ee17040363872365cf702afbc1bbc2 on master.

@dylangrafmyre Please review this PR. Should we remove the docker-compose.yml file? I removed docker from the rake ci task a long time ago because we need to lock linting versions in the package.json file.

@justin808 I would ask @robwise. If no one is using Docker for this project, you can remove the compose file and Dockerfile_tests. Also if you are not using Selenium but Poltergeist, you can remove the other dependencies like the xvfb server.
I really don't know where this project stands.

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

Comments from Reviewable

I think we were only using the container for the linting, so since we lock that down via gemfile/package.json we should be able to delete this without a problem

Coverage remained the same at 82.305% when pulling 8cec9cf345a7a940a1306a02d353704e12955cad on justin808-skip-docker into cdb246bb7934283fbe5a99c122d5f801a10e67c0 on master.

Coverage remained the same at 82.305% when pulling 8cec9cf345a7a940a1306a02d353704e12955cad on justin808-skip-docker into cdb246bb7934283fbe5a99c122d5f801a10e67c0 on master.


Use correct dtype for RGB image alpha channel

[x] Closes #1880
[x] Tests added (for all bug fixes or enhancements)
[ ] Tests passed (for all non-documentation changes)
[x] Fully documented (bugfix for earlier change, no additional note)

The cause of the bug in #1880 was that I had forgotten to specify the dtype when creating an alpha channel, and therefore concatenating it cast the all the data to float64.  I've fixed that, corrected the alpha value for integer arrays, and avoided a pointless copy to save memory.

Ping @fmaussion

Thanks @Zac-HD ! Again (and sorry for the hassle ;), this needs a regression test. Also, could you change the rasterio example in the docs and see if your change indeed makes the normalisation obsolete? (https://github.com/pydata/xarray/blob/master/doc/gallery/plot_rasterio_rgb.py#L27)

Appart from the comment in the gallery example and the flake8 error, this looks good to me! Thanks @Zac-HD !

The failing tests in appveyor should be fixed by https://github.com/pydata/xarray/pull/1904

Thanks @Zac-HD !


[NEW] Translation to pt-BR

cc: @sadamo @diegoparadeda

Please split
https://github.com/shopinvader/shopinvader-template/pull/53/commits/0521f1fdb7d143b672027bb60f2cd9235ec0f7a9
https://github.com/shopinvader/shopinvader-template/pull/53/commits/b40f73ecfab011784b2908b8a988a39d9848b504
in another PRs


Swoole TS image?

For Swoole 6 threads there mention that its required TS PHP, but in docker hub there no TS image, only NTS. Will TS image be added to docker hub https://hub.docker.com/r/phpswoole/swoole/tags?
alias php='docker run -it --rm --network=local-environment_local-network -v $(pwd):/app -w /app -p 8000:8000 phpswoole/swoole:6.0.0-php8.4-alpine php'
~ % php -v
PHP 8.4.1 (cli) (built: Nov 21 2024 18:24:03) (NTS)
Copyright (c) The PHP Group
Built by https://github.com/docker-library/php
Zend Engine v4.4.1, Copyright (c) Zend Technologies

邮件已收到 谢谢！


Client Throws "DBException(message:User matching query does not exist. Lookup parameters were {'username': None})"

The following piece of code throws the above error:
this.transport = new THttpClient("http://datahub.csail.mit.edu/service");
this.protocol = new  TBinaryProtocol(transport);
this.client = new DataHub.Client(protocol);

this.con_params = new ConnectionParams();
this.con_params.setUser("anantb");
this.con_params.setPassword("anant");
this.conn = this.client.open_connection(con_params);

ResultSet updatelogExists =  this.client.execute_sql(this.conn, "select * from anantb.test.demo", null);

Gives the following error:
DBException(message:User matching query does not exist. Lookup parameters were {'username': None})...

Would appreciate a fix for this! Thanks!

Ah, the API has changed.


Grammar improvements and created end-links in web-sdk-best-practices.md

Per prior arrangement, if this PR needs approval, or if the commits need to be pulled into the private repo -- and while @eriklindeman is OOF April 10-17 -- only @FarazGIS should be notified. Thanks.

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

#sign-off


sweep-visualizer: Remove

meta.homepage links to a nonexistent subdomain and
https://scanse.io/downloads still only has instructions for ancient
Ubuntu 14.04.
This package has not seen updates in years and likely never will.
It depends on GNOME2/GTK2 which will go eventually.
It is also marked as unfree, so there's even less incentive to keep
such dead software around.
It crashes on start:
$ NIXPKGS_ALLOW_UNFREE=1 nix-shell -p sweep-visualizer
...
$ sweep_visualizer
Segmentation fault (core dumped)

See #39976.
@mt-caret

No value in keeping crashing things around.


[fix] Prevent default onError from breaking wallet provider memoization

Problem
The WalletProvider presently supplies a default error handler as part of the component's method signature.
function WalletProvider({
  onError: _onError = (error: WalletError) => console.error(error),
  /* ... */
}) { /* ... */ }

The way that default properties work in ES6 is such that every time you call the function, it will construct a brand new default function, distinct from the one constructed in the previous function call (demo).
Because we use _onError as a memoization parameter to useMemo, we're busting wallet memoization way more than is necessary – if the developer didn't supply their own, stable, onError function.
I've seen this manifest in dApps as a tendency of the app to fetch and refetch the same API calls over and over. One of the root causes, is that we're passing apps a brand new wallet object through context, which causes their React effects to run again, and some of those effects might think ‘oh hey, the wallet changed; I should refetch some data.’
This PR
In this PR, we inline the default error handler implementation so that:

It doesn't get recreated over and over every time the provider re-renders, and
Only a genuine change in a user-supplied onError function will bust memoization.

Makes sense, thanks for this!

Published: @solana/wallet-adapter-react => 0.14.1


ITT-2240 - Multitenancy: Check that apps are enabled before trying to access their properties

This PR introduces a check for enabled apps before trying to reset the lastSubUrl when switching tenants. Without the check, chrome.getNavLinkById(appId) may throw an error.

Only merged into master so far.


estimate of the number of clusters is always 1?

No matter how my sample points are spread out (often very obviously clustered), Estimate() always returns 1.

Same things with my data.


How to filter by relation type?

I think this is supposed to work like this:
https://pmb.acdh.oeaw.ac.at/apis/api/relations/personinstitution/?format=json%2Bnet&relation_type__id=1183
But that returns 500 Error.

this is a data problem; related to #126

ok, the netjson renderer threw an error in case of an empty queryset, fixed this now


Define Knowledge Graph Schema

I think we might want to use AllenNLP on ASKE to parse the documentation and find the Verb-Argument trees in software documentation. For the Initial approaches we need a plan of how NLP on the text embedded in code/model descriptions/documentation can be used to pull out a knowledge graph of concepts and how they relate to each other.
I'm thinking that the knowledge graph will have
Vertex Types:

Code symbols
Functions
Variables
Symbols

Math/Science Concepts:

Math Expressions
Concept
Unit

Edge Types: SourceType -> DestinationType

IsCalledBy: function -> function
Co-occurs with: Any -> Any
IsComponentOf: {Symbol,Function,Variable} -> {Function, Expression}
IsMeasuredIn: Any -> Unit
Implements: Function -> {Math Expression, Concept}
VERB: Any -> Any
IsSubClassOf: SpecificConcept -> More General Concept

AllenNLP verb-argument trees can be used to find the verbs from the text and some of these relations.
If you can think of any other vertex types or relation types, add them to this list.

The AutoMATES team has a format they use for knowledge extracted from fortran codes. Called a grounded function network. We should consider integrating with their representation. https://delphi.readthedocs.io/en/master/grfn_spec.html#top-level-grfn-specification

Some thoughts on edge weights w/respect to path-finding algorithms:
The end-user may define cost in different ways; some that come to mind are:

Computational efficiency (e.g., willing to accept a parameterized level of errror in exchange for improved runtime and/or storage requirements)
Accuracy of results relative to empirical/observational data (e.g. minimizing accumulated error and/or final error)
Explainability (this amounts to pruning the DAG; algorithm nodes can be assigned Boolean literals to indicate whether they are explainable or not; edges that connect unexplainable algorithms can be ignored)

We can pull out asymptotic runtime and accuracy guarantees. We need to extract arbitrary tags on nodes.

Just added IsSubClassOf: SpecificConcept -> More General Concept to the list above. Am trying to capture the OO idea of inheritance, which I think is related but distinct from the IsComponentOf: edge type. I think this can be used to capture information such as "independent variable -> Variable", which can be helpful down the road when we need information about which variables are more likely to be inputs vs.outputs of a given set of function(s).

That paper is a good find, it looks like they have Syntax edges that map onto the programming language syntax and Dataflow edges that represent the flow of data between variables.
The Julia syntax is pretty simple, everything is in an Expr object which has a head which tells you what type of syntax it is and a list of args that are the subcomponents. Here is a list of what various Julia syntax forms turn into as an Expr https://docs.julialang.org/en/v1/devdocs/ast/#Surface-syntax-AST-1 that whole page is worth reading for sure.
The dataflow for julia is Single Static Assignement which can be accessed in the lowered code form in Cassette or Base.Meta.lower(quote <some code here> end)
What kind of tasks do you think we could apply to the code once we have extracted out a graph? Variable Misuse seems like a good tool for a debugger or static analysis tool, but what is the higher level goal that scientists need?

Great question. I think to answer that will partially require outlining our initial meta-modeling uses cases. I just created a new issue #23 to start tracking ones we want to support, those that can't be supported, and the undecided. :) I think we could develop uses cases and see if there's learning tasks that would be helpful to accomplish to make them happen or build use cases based on kinds of learning tasks we think would be useful to employ, sorta top-down or bottom-up, or maybe a bit of both. I put up one potential case about merging model instances from same modeling family together where its just variables (and potential parameter settings) that are being integrated. Of course there's many more use cases of varying complexity. I think it would be useful for us to get a sense of what scientist in a particular field, in our case, Epidemiology, are wanting and/or doing when they read about new models published in the literature. Here's some questions we might want to answer to determine that (not sure where we might get data except from informal survey or reading about sociology of science in a particular discipline):

Does the scientist download a new paper and script and run it?
Do they then attempt to change the model code or integrate?
What kinds of modifications do they make?
3.1 Variables,
3.2 switching to different model families all together, keeping same data or other sim parameters?
3.3 Other things?

So, this is to say, I'm going to think about it a little more and reflect on more cases we come up with in #23 .

@scottagt interesting paper; thanks for posting! Am reading through now.
@jpfairbanks @scottagt re the question:

What kind of tasks do you think we could apply to the code once we have extracted out a graph?

To me it is interesting to think about graphs (derived from code) that may not be isomorphic at the lowest level (e.g., when we extract directly from code), but at higher levels of abstraction may be isomorphic. This can help us to identify families of functions, and/or when different subgraphs may be serving similar purposes (and can thus be used as component blocks for meta-modeling).

Will take a look at #23 as well.

@crherlihy I think those are good tasks on the code graphs.
When doing matching of "isomorphic" subgraphs, we will definitely have to either change level of abstraction or have a fuzzy definition of "isomorphic". I think there will be very few exact isomorphisms larger than 3-5 vertices.


lymph node atrophy

Add new term:
lymph node atrophy ( as child of small lymph nodes MP:0002217)
DEF: acquired diminution of the size of lymph node tissue associated with wasting as from death and reabsorption of cells, diminished cellular proliferation, decreased cellular volume, pressure, ischemia, malnutrition, reduced function or malfunction, or hormonal changes; occurs with age, immobilization, weightlessness, malnutrition or denervation
SYN: atrophic lymph node

Added new term:
lymph node atrophy MP:0031071


AttributeError: 'module' object has no attribute 'rnn_cell'

Hi!  Thanks for sharing the code
When I try to run the code python train_GPU.py
I met

Traceback (most recent call last):
File "train_GRU.py", line 243, in
tf.app.run()
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/platform/app.py", line 44, in run
_sys.exit(main(_sys.argv[:1] + flags_passthrough))
File "train_GRU.py", line 159, in main
m = GRU(is_training=True, word_embeddings = wordembedding, settings = settings)
File "train_GRU.py", line 49, in init
gru_cell_forward = tf.nn.rnn_cell.GRUCell(gru_size)
AttributeError: 'module' object has no attribute 'rnn_cell'

my tensorflow verison is
>>> print(tf.__version__)
1.0.0-rc1

I was wondering which tensor flow version is used in your code?
Thanks!

I guess You may try tf r0.11.
If you use r1.0, tensorflow API, you have to change the code to be tf.contrib.rnn.GRUCell
You can refer to tensorflow website at https://www.tensorflow.org/api_docs/python/tf/contrib/rnn/GRUCell

The tf version I'm using is r0.11.0.

Thanks！ after change some part of the code to r1.0, this error was solved.

I am a new student .I do not know where i need to change about code?? please help me . Thank you

As per the latest version use this " tf.contrib.rnn import core_rnn_cell"


change default of use_residue_map to be False

PR Summary:
Address https://github.com/mosdef-hub/mbuild/issues/1111. Change the default of use_residue_map in Forcefield.apply to False, since this may be produced unexpected behavior if users try to atomtype of systems with similar residue name.
PR Checklist

[ ] Includes appropriate unit test(s)
[ ] Appropriate docstring(s) are added/updated
[ ] Code is (approximately) PEP8 compliant
[ ] Issue(s) raised/addressed?

Tests failed due to weird issue when calling pmd.version (stating that parmed doesn't have version module). I still couldn't replicate this locally (update parmed to v4.1.0, and parmed.version is still returning a version object.

Update, the issue has drifted a bit, now the tests are failing because of the apply_NBFIX test, there is some behavior change due to new version of parmed (which affect the uniqueness of atom_type). Working on the fix atm


PSA: create examples to demonstrate the use of each error type

I don't know if we have ever had such example for v1model, but we should have them for PSA.

@vgurevich I have been hacking on an example program that sends packets that experience parser errors to the control plane, with a small 'to-CPU' header giving the parser error code and a few other fields that a control plane might want to know about the bad packet.
When you mentioned the idea of a P4 implementation that when experiencing an error during parsing, left the headers in an unknown/undefined state, do you mean a state where:
(a) emit'ing the headers in a reasonable order would still be guaranteed to reproduce the original input packet?
(b) Or do you mean that the resulting state might be one where emit'ing the headers would not be guaranteed to reproduce the original packet?
If you mean (b), then we are OK to send the original packet as received by the ingress parser to the control plane, because ingress-to-egress clone is specified by PSA to send the original packet, before parsing began.  However, we would have no way defined in PSA to send the pre-parsed packet contents if a parser error occurred in the egress parser, because egress-to-ingress clones are currently defined in PSA such that the cloned packet contents come out of the egress deparser.
If you don't mind such an implementation having no way to recover the original pre-egress-parser packet when the egress parser encounters a parse error, then no worries.

Created this PR with a proposed example program that demonstrates how to send a copy of packets that experience parsing errors to the control plane with a header containing info about where the packet came from, and what went wrong.  I can't think of much reason to do different things for different errors, except perhaps some should be copied to the control plane, where perhaps for other types of errors it might be preferable not to send a copy to the control plane, but simply count them.  https://github.com/p4lang/p4-spec/pull/541


static-adapter doesn't work with dynamic routes

Describe the bug
SvelteKit built using static-adapter with dynamic routes simply does not work on S3 (static hosting) and github pages.
When you start your site's journey from / and then navigate using links to a dynamic route (i.e. /s/x/y), it works.
However if you directly try and go to /s/x/y (or alternatively refresh the page), you get a 404 page.
Is there any way to make the site work on s3 or github pages?
Or is dynamic routing with static-adapter not feasible?
Perhaps an option to make sveltekit use #-based routing.
Reproduction
Create any site with dynamic routes and run it on s3.
Logs
No response
System Info
s3

Severity
blocking all usage of SvelteKit
Additional Information
No response

I found this https://dev.to/alexeyromanchenko/how-to-deploy-spa-as-a-static-website-using-aws-s3-1adg and this https://medium.com/@P_Lessing/single-page-apps-on-aws-part-1-hosting-a-website-on-s3-3c9871f126 and this just by googling S3 SPA.
You can probably use something from those resources.

As for the Github Pages, natively they don't support SPAs, but there is hacky workaround https://github.com/rafgraph/spa-github-pages
essentially you create 404.html file inside static folder with this content:
<!DOCTYPE html>
<html>
  <head>
    <meta charset="utf-8">
    <title>Single Page Apps for GitHub Pages</title>
    <script type="text/javascript">
      // Single Page Apps for GitHub Pages
      // MIT License
      // https://github.com/rafgraph/spa-github-pages
      // This script takes the current url and converts the path and query
      // string into just a query string, and then redirects the browser
      // to the new url with only a query string and hash fragment,
      // e.g. https://www.foo.tld/one/two?a=b&c=d#qwe, becomes
      // https://www.foo.tld/?/one/two&a=b~and~c=d#qwe
      // Note: this 404.html file must be at least 512 bytes for it to work
      // with Internet Explorer (it is currently > 512 bytes)

      // If you're creating a Project Pages site and NOT using a custom domain,
      // then set pathSegmentsToKeep to 1 (enterprise users may need to set it to > 1).
      // This way the code will only replace the route part of the path, and not
      // the real directory in which the app resides, for example:
      // https://username.github.io/repo-name/one/two?a=b&c=d#qwe becomes
      // https://username.github.io/repo-name/?/one/two&a=b~and~c=d#qwe
      // Otherwise, leave pathSegmentsToKeep as 0.
      var pathSegmentsToKeep = 0;

      var l = window.location;
      l.replace(
        l.protocol + '//' + l.hostname + (l.port ? ':' + l.port : '') +
        l.pathname.split('/').slice(0, 1 + pathSegmentsToKeep).join('/') + '/?/' +
        l.pathname.slice(1).split('/').slice(pathSegmentsToKeep).join('/').replace(/&/g, '~and~') +
        (l.search ? '&' + l.search.slice(1).replace(/&/g, '~and~') : '') +
        l.hash
      );

    </script>
  </head>
  <body>
  </body>
</html>

and in the app.html you add this before %svelte.body% (that's really important):
<script>
      (function(l) {
        if (l.search[1] === '/' ) {
          var decoded = l.search.slice(1).split('&').map(function(s) {
            return s.replace(/~and~/g, '&')
          }).join('?');
          window.history.replaceState(null, '',
              l.pathname.slice(0, -1) + decoded + l.hash
          );
        }
      }(window.location))
</script>

Github Pages solution is a really hacky one, so if you can help it, don't use it.
And you're probably better of with Cludflare Pages, Vercel, Netlify.
And if you want to have something like S3 (and cheaper) then you can use BunnyCDN (read this article https://bunny.net/blog/host-your-frontend-applications-on-the-edge/ on their blog)
(I don't have any affiliation with them, I just use them)
And for all of these you DO need to use fallback option for adapter config.
Hope that helps

Maybe sveltekit or static-adapter should have an option that will make it use # based routing. That will solve the problem for most people..

Maybe sveltekit or static-adapter should have an option that will make it use #-based routing. That will solve the problem for most people..

That's really really bad for SEO, because the normal SPA Google can actually render, but if it's using hash routing it can't figure out there are any other page except index/homepage.

@PH4NTOMiki How does the fallback option in static-adapter help since if you go to a dynamic route, the /index.html page is not available to display the fallback option.

Maybe by now, Google understands #-based routing since it's so common, that they can successfully interpret it like they do for / based routing.

yes the server can be configured to fallback, but how does static-adapter do anything about it's fallback option?

adapter-static (not static-adapter) creates file with a file name that was set in fallback option so that you can configure a server to fall back to that file if the specific route is unavailable.

Aha. Thanks for explaining.

You need to set trailingSlash: 'never' if you're using S3. Closing as duplicate of https://github.com/sveltejs/kit/issues/3852

@PH4NTOMiki can you explain the significance of trailingslash = never?

https://kit.svelte.dev/docs/configuration#trailingslash
https://kit.svelte.dev/docs/adapters#supported-environments-static-sites

@PH4NTOMiki I set a fallback to 404.html. I can see that a 404.html file is generated, but the index.html page is no longer generated, so the site no longer works. It just displays the directory listing.

@PH4NTOMiki Can you help me diagnose this issue: https://github.com/sveltejs/kit/issues/4378


Updated Doc Strings

Took a first pass at updating doc strings. README still needs to be updated and set of demos and examples should be added to explain the use cases for offered widgets.

@sassanh I had to do a rebase which I don't know if It was a good idea. I updated the docs and added backticks. We can further improve the docs but perhaps we can get this PR merged...

@mehrdadfeller I checked some files, it looks good. A full review would be very time consuming and considering our priorities, I can't manage to do it. I say lets merge it and then improve it over time if we find any issues.

@sassanh no merge conflict :) all changes were on docstrings and no heavy review was needed. just wanted to get your feedback before merge


More reliable recording

You won't want 9e103f5, but I can't easily exclude it. Sorry bout that.
The count_ps stats match how StatsD aggregates, and helps when your FlushInterval isn't 1s.  And the error logging makes dependency problems easier to track down.

See #7 instead.


gulp-stylelint failing out of gulp process on error

When I have a stylelint error, it is failing out of gulp, which means I have to completely restart Gulp every time I make a mistake.
(At first I thought this was a gulp-notify issue, but it persists when gulp-notify is disabled.)
My gulp task:
// Build CSS.
const compileCSS = () => {
  const sassOptions = {
    includePaths: ['node_modules']
  }

  const stylelintOptions = {
    reporters: [
      {
        formatter: 'string',
        console: true
      }
    ]
  }

  return gulp.src(paths.css.src)
    .pipe(stylelint(stylelintOptions))
    .pipe(gulpif(isDev, sourcemaps.init()))
    .pipe(sass(sassOptions))
    .on('error', notify.onError((error) => {
      return error.message;
    }))
    .pipe(postcss([ autoprefixer() ]))
    .pipe(gulpif(isDev, sourcemaps.write()))
    .pipe(gulpif(!isDev, rename({ suffix: '.min' })))
    .pipe(gulp.dest(paths.css.dest))
    .pipe(browserSyncServer.stream())
}

And the terminal output:
[16:02:18] Starting 'compileCSS'...
[16:02:19]

_src/css/global/_images.scss
 90:1   ✖  Expected empty line before comment   scss/double-slash-comment-empty-line-before
 90:13  ✖  Expected a space after //            scss/double-slash-comment-whitespace-inside

[16:02:19] 'compileCSS' errored after 884 ms
[16:02:19] Error in plugin "gulp-stylelint"
Message:
    Failed with 2 errors
Details:
    domainEmitter: [object Object]
    domainThrown: false

This should show a notification on my machine, but it doesn't; it's just failing in the terminal.
I am running the latest of all relevant packages as of this writing:

gulp 4.0.2
gulp-stylelint 13.0.0
stylelint-scss 3.17.2

I'm also having the same issue.
gulp.task('sass:stylelint', function () {
  var src = config.sass.watchSrc;
  if (config.sass.lint.extraSrc) {
    src = src.concat(config.sass.lint.extraSrc);
  }
  return gulp.src(src)
    // .pipe(gulp.$.cached('validate:css'))
    .pipe(gulp.$.stylelint({
      reporters: [
        {
          formatter: 'string',
          console: true,
          failAfterError: false,
        }

      ]
    }))
});

gulp 4.0.2
gulp-stylelint 13.0.0
stylelint-scss 3.18

Getting the same error as well. Running stylelint with the CL runs just fine. Using the default gulp-stylelint implementation from the readme. Stylelint config just the default as well.
gulp 4.0.2
gulp-stylelint 13.0.0
stylelint 13.10.0
stylelint-config-standard 20.0.0

Just noticed the same. What's up?

By setting the failAfterError option to false of gulpStylelint got it working fine. Appears that Stylelint considers any violation as an error. Which then causes the task to fail. I don't agree with that as it prevents the linting of files. My setup generates a JSON file which is then rendered as a report in the browser. Even if Stylelint wants to say that a violation is an error, gulp-stylelint should disregard that. Please let me know what I'm missing here. Is there some config option in Stylelint to prevent a file with violations to be considered "errored?"

Nothing to add other than to say I'm also running into the same issue. My setup is the same as @danparm . Setting failAfterError to false also worked for me.


openmoji link broken

was checking out the footer and tried to click through the link. looks like they may have changed their website since this link was originally written. happy to open a PR but it wasn't clear from the URL what the link should go to - let me know!

Fixed in the latest version: https://axodotdev.github.io/oranda/


fix: traps formatting and documentation update

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


Added more customization

I added color customization  to drawer. Also, now user drawer background may be a color instead necessarily a bitmap.
Finally fixed a bug where the action bar where the action bar color misses the correct one for the default fragment.

Color customization is added to the 1.3 alpha, committed just now.
What do you mean with:
Finally fixed a bug where the action bar where the action bar color misses the correct one for the default fragment. ?

Commit 51aa1ae fixes a bug where if you provide a color to the first section, action bar will be default color until you change the current section.

Oh, thank you for finding this bug, I will update the library. Sorry But with the last commit I can't simply merge your code to the library, but your help is appreciated.
With the 1.3 alpha you can set the drawer color in your custom Theme using this item:
        <item name="drawerColor">#fafafa</item>

Nice. I'll close the pull request then.
Glad to help.


Nothing Showing in the table

Love todo-show, thank you.
v. 1.11.0, Atom 1.15.0
I was dragging the pill-buttons around "On Table" and "Off Table" and think I dragged them all off, I'm not entirely sure, but once I did that, nothing shows in the table. I've tried dragging them around, back to what I'm guessing is the "On Table" section, but that doesn't seem to work.
When I save the todos to a file, I get a line for each result (you can see from the screenshot that there are 24 results) with the text "- No details."
I've tried deleting it, but of course Atom seems to save the settings and I'm not sure how to get around that. I've also changed the table items in "Show in Table" in the config, but that doesn't help either (and I'm not sure the difference between the draggables and doing it with text in the config.).
Any way to reset, or otherwise fix this? Thanks again!

Solved this by clearing the "show in table" config item to allow defaults.


fix: Solve the problem of losing connection with the server and not e…

…xiting
After detecting a loss of connection with the server,
the client should exit

you have to determine whether the err is Network err.

Reinitiated submission.


Point cloudflare-worker to specific preview deployment

Sometimes, we want to test the integration of a preview deployment with de.serlo.org (or serlo-staging.dev). For this case, it would be useful to set the frontend url manually, e.g. by using a cookie.

Yep, I have that on my list but didn't get to it yet.

This won't work in this direction: we decided that Vercel deployments are fixed to a specific API URL so that we don't need a server request. So if we need this, we need to implement this on frontend side (i.e. tell via config or something to deploy something in a specific environment). Problem might be: AFAIK Vercel accepts only one production deployment, so we probably need to handle that in preview environment.

preview environment would be fine, too

let's say, we want to preview https://frontend-git-client-side-routing.serlo.now.sh in "real production", which means using the cloudflare-worker under de.serlo.org:
I would change set a cookie with the new url and use it here instead:
https://github.com/serlo/serlo.org-cloudflare-worker/blob/99f7730197bca3e46091830df8b26fadcc8d19da/src/frontend-proxy.ts#L75
Optionally we can also point the endpoint to staging if this cookie is set:
https://github.com/serlo/serlo.org-cloudflare-worker/blob/99f7730197bca3e46091830df8b26fadcc8d19da/src/frontend-proxy.ts#L90
This way, when I access de.serlo.org, I can preview the changing in this PR pretty much under live conditions (which would be awesome).
@inyono, @kulla: Does this sounds feasable?

That covers only one direction. Basically:

When you want to test the integration of a preview deployment trough the Cloudflare Worker (e.g. to test if the proxy / other stuff handled in the Worker works as expected), we need a cookie at Cloudflare Worker level (similarly to what you described). But: from frontend side, the deployment is still tied to a specific environment (production or staging) at build-time (because of caching & optimization reasons). Hence:
To test a preview deployment with the production API, we need a way for the frontend to tell Vercel that they want to get build with the production API stuff (e.g. by setting a constant value useProduction that is false by default). This is more common case, I assume.

And to use both, e.g. to test a preview deployment with production API & Cloudflare Worker, we'd need both.

At https://github.com/serlo/serlo.org-cloudflare-worker/pull/27 I have started to implement the first direction...

First direction is live, second is on my list.

Second direction also works (by setting NEXT_PUBLIC_ENV to production in .env.production), see https://github.com/serlo/frontend/pull/331


I found an incomprehensible key error case.

I'm not good at English. Please understand.
I made Web wallet by eosjs(16.0.9), eosio(1.6.3) on Ubuntu 18.04.
It is a structure to trade tokens using the Web.
but, I found an incomprehensible key error case.
Despite the difference in the ending digit of the private key,  A Token transfer to the other user.
For example, If my private key number is "....................bZV5K9SqseNwxMSa4",  "....................bZV5K9SqseNwxMSa1" is also transmitted.
This error only occurs in ending digit of private key.
It only happens to a few users.
Fortunately, Cleos is not found in that case
Thanks your advise.

Could you please provide a step-by-step process to reproduce this error?
I'm unable to determine exactly what you are trying to explain without any context.

The last 4 bytes of an EOS private key expressed in binary are the first 4 bytes of the ripemd160 hash of the key material.  Expressed in base58, this is the last 6 characters.  If the code you are using to sign transactions does not validate the key checksum, errors in the last few bytes of the key will be ignored.  Errors in the key checksum do not affect the integrity of the resulting signatures.  It does mean that if the key itself is corrupted that will go undetected until a signature validation is attempted, which will fail.


GVFS requires a OS downgrade to use

I don't see why I should have to downgrade from 7 to 10 to use this.
It should be easy enough to port to 7
This team had a choice, accept windows 10 as the required version, or try to get windows 7 supported.
They choose wrong.
Windows 7 STILL has more users then windows 10. http://www.w3schools.com/browsers/browsers_os.asp
So they said that the majority of users being unable to use their product was ok,
This is me pushing against them on that, and rightfully so.
They should have tried harder
In order to improve windows 10, people have to do the following:

Not accept windows 10 in its current state as an primary os

Meaning not accepting the idea of software only being supported on windows 10

Not use windows 10 where they have the choice
Give microsoft feedback on why they are doing the above

Pushing back against only supporting windows 10, (regardless of why) is a necessary evil to improving windows 10.

I don't think GitHub is really the place for this kind of trolling.
I'm sure you'll reel off a bunch of problems (real or perceived) with Windows 10 that in your eyes make it inferior to Windows 7, but you know perfectly well that Windows 10 is newer than Windows 7, Microsoft regards Windows 10 as the current version of Windows, and that as such Windows 10 is an upgrade to Windows 7.

newer != upgrade
update != upgrade
It has to be better to be an upgrade, other wise it's a downgrade.
Pre Installed Adware, Spyware, and borderline Malware doesn't make a good upgrade against an operating system that doesn't have that.

It has to be better to be an upgrade, other wise it's a downgrade.

And in Microsoft's view it is unambiguously better. And you know this. You're trolling.

> Microsoft's view
But microsoft is not the one making the issue report.
I am.
And in my view, it's not.
Windows 10 is worst in every way compared to windows 7.
UWP locks the user out of control over their apps. With win32 I can make the choice to install an application that modifies how another application operates or works, with UWP i can not, that choice is taken from my hands.
UWP alone wouldn't be a downgrade, but windows 10 /pushes/ it over win32, with the plan clearly being to limit and remove win32. It's already been stated that a later update may restrict running as admin to UWP apps installed from the windows store, meaning that with windows 10 I may wake up one day to find half my programs don't even fucking work because of a forced update I'm not suppose to be able to disable.
UWP goes against the very foundation of Open Source Software by locking the user out of control over the app.
Windows 10 removes user control over updates and antivirus, two things I heavy scale back as the first thing I do in a new install. So now disk performance is slower because of antivirus scanning, and its slower half the time because of some update installing in the background (let alone what forcing a restart does to any overnight operations I leave running)
Because of the above: Windows update being forced is a downgrade, I can basically trust that my computer will break because of an update. (oh, you'll say, updates work fine for 99% of computers, but that just means I only have to wait for 100 updates before something breaks, only it only took ~10 before the first major work stoppage at work from an windows 10 update.)
Windows 10 forces Desktop Window Manager to be running, a service I disable to remove a noticeable overhead in creating of gdi objects.
Windows 10 forces ads on you, in the form of notifications, start menu suggestions, etc. some can be disabled, some seem to not be easily disableable. This alone makes it a downgrade over 7 I could stop here, hell I could have just included this line, but that's not as fun.
Windows 10 keeps enabling features user disable like telemetry or cortona
Windows 10 doesn't even allow you to officially disable these features requiring registry hacks.
Windows 10 keeps sending details back to microsoft about how you use your computer even after you disable telemetry.
Connected accounts.
Windows 10 is objectively a downgrade from Windows 7. Windows 8 is as well. Windows 10 is only an upgrade when compared with windows 8, and barely one at that, but both still fall short of Windows 7.

@MrStonedOne If you have all these issues I ask why are you still using Windows?  But why stay. Yes you can still run Windows 7 but you can still run Windows 1. Just because something runs does not mean you should. macOS does plenty of the same stuff so pick a Linus distro and be happy!

But microsoft is not the one making the issue report.

No, but they're the one who closed the previous issue report, and who will inevitably close this one too.

Windows 10 is objectively a downgrade from Windows 7

"Objectively" a downgrade unless you value all the improvements that Windows 10 makes and Windows 7 lacks, you mean? Indeed, you were told in the previous issue report that changes in the way Windows 10 AU handled reparse points were critical to the virtualized file system's performance. This sure sounds to me like a way in which the new operating system is objectively superior to the old one.
In any case: Microsoft isn't going to go backwards. The company isn't going to scrap all development that took place after Windows 7 and pretend that Windows 8 and 10 never happened, and complaints that Windows 10 is a "downgrade" are going to fall on deaf ears.

I think you are confusing hating something, and being disappointed it's not as good as it could have been.
I was excited for windows 8, until it came out....
I was excited for windows 8.1, until it came out....
I was excited for windows 10, until it came out....
Now I have a choice, stand by while the amount of new technologies I can use dwindles, use an OS that is built from the ground up to tell me I don't know what i'm doing so it will protect me, or switch to linux for my daily driver and deal with compatibility issues.
Or the better options, push back.
Fight against obsoleting windows 7, fight against software devs not supporting it, fight for windows being as good as it could be.
If I hated microsoft and windows, I would have switched to linux a long time ago.
I have a rule, If I think the world would be a better place if everybody did something, I'll do that something, even if I stand alone. Because apathy is the enemy of progress.
 I'd rather improve windows then let it fall to the wayside.

Windows 10 is objectively superior to Windows 7.

@SamuelEnglard think of how good windows 10 could be if it didn't have all of the downsides?

@MrStonedOne strive for? Yes. I'd rather strive for it from an OS that is getting updates.  Hopefully a future version of Windows would be more to your liking and you can come back

This team is part of microsoft, they have a better ability to express feedback then microsoft has given it's users.
And if this team (as well as any software dev company/team) didn't get push back for only supporting windows 10, then they wouldn't express frustration over the situation to their support contact at microsoft.
In order to improve windows 10, people have to do the following:

Not accept windows 10 in its current state as an primary os
Meaning not accepting the idea of software only being supported on windows 10
Not use windows 10 where they have the choice
Give microsoft feedback on why they are doing the above

Pushing back against only supporting windows 10, (regardless of why) is a necessary evil to improving windows 10.

This team had a choice, accept windows 10 as the required version, or try to get windows 7 supported.
They choose wrong.
Windows 7 STILL has more users then windows 10. http://www.w3schools.com/browsers/browsers_os.asp
So they said that the majority of users being unable to use their product was ok,
This is me pushing against them on that, and rightfully so.
They should have tried harder

Not accept windows 10 in its current state as an primary os

But why would I want to go back to Windows 7? It's inferior.

This is me pushing against them on that, and rightfully so.

Fair, fair. Doing so once was pushing back. Once they said they heard you and you opened an new issue for the same exact thing... That's not giving feedback, that's harassing.

This is me pushing against them on that, and rightfully so.

Nah. Your first issue might be taken as that. But it was closed. Creating another identical issue--with the same disingenuous title--is just plain trolling.

The issue still exists.

Not in the opinion of the developers. You're welcome to fork it and write your own filter driver if you want.

@MrStonedOne I am open to discussing ideas for improving GVFS. And I assure you that if I could make it work on Windows 7, I would. As I said in the other issue that I closed, it is a matter of prioritizing where we put our time. My initial wave of customers are all on Windows 10. My next highest demand is for Mac. That is the order in which we will tackle them.  If you want to submit code to make it work on other operating systems, we will evaluate that code.
Also, I am not part of the Windows org. Let's keep the conversations here focused on how to improve GVFS.


[Mobile Services] "Returns" description not matching actual return type

Issue Transfer
This issue has been transferred from the Azure SDK for .NET repository, #31891.
Please be aware that @qbqaf is the author of the original issue and include them for any questions or replies.
Transfer note:
Though this text is likely sourced from build output on the library, this package has not been released since 2015 and is unlikely to be undergoing any further code changes.   As such, I've transferred to the MS Docs repository for a decision on whether or not the doc comments should be updated directly via the XML.
Details
For the GetSyncTable<T>() method, the description says the method "Returns a IMobileServiceTable instance", but the "Returns" section shows it returns the type "IMobileServiceSyncTable", which is the Sync version of the type mentioned.
Should be an easy fix, simply add Sync to the return type in the description.

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

ID: 7dcece93-fa05-e81e-c681-83f8c78352c6
Version Independent ID: e7c85e82-5019-fc0c-2613-8ae833927f03
Content: IMobileServiceClient.GetSyncTable Method (Microsoft.WindowsAzure.MobileServices) - Azure for .NET Developers
Content Source: xml/Microsoft.WindowsAzure.MobileServices/IMobileServiceClient.xml
Service: app-service-mobile
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

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

@qbqaf I have assigned to content author @rloutlaw to review and share their valuable insights on this.


fix: set http output contenttype to text/plain when json output is disabled

What type of PR is this?
/kind bug
Any specific area of the project related to this PR?
/area engine
What this PR does / why we need it:
The PR sets the http output Content-type header to a more meaningful value (text/plain) when json_output is disabled.
Which issue(s) this PR fixes:
Fixes #1824
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
fix: set http output contenttype to text/plain when json output is disabled

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: FedeDP
To complete the pull request process, please assign leogr after the PR has been reviewed.
You can assign the PR to them by writing /assign @leogr in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: FedeDP
To complete the pull request process, please assign mstemm after the PR has been reviewed.
You can assign the PR to them by writing /assign @mstemm in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: efa6e71a68f1ba9ee93f1fae8d2e682838f0737a

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone 0.31.0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, jasondellaluce, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Bump readme for 0.7

Thank you for helping out with embedded-graphics development! Please:

[x] Check that you've added passing tests and documentation
[x] Add a CHANGELOG.md entry in the Unreleased section under the appropriate heading (Added, Fixed, etc) if your changes affect the public API
[x] Run rustfmt on the project
[x] Run just build (Linux/macOS only) and make sure it passes. If you use Windows, check that CI passes once you've opened the PR.

PR description
Update the readme as per #539. I'm not going to shuffle the sections around as the last comment(s) mention as it's not critical for release, and they can be moved later in a patch release.
Closes #539.

I should've tested the build locally :facepalm: I knew there was a reason I bought a 5950X...
I've applied the suggestions too - thanks for the review.

Did you upgrade because the 5900X wasn't available?

Precisely this. My credit card isn't too happy but I'm glad I could get something...
I changed the simulator section to point to the examples repo, and removed the attribution section entirely.
I also updated the screenshot, but now that I put the commit hash in the URL, I think this will break immediately after merge, so I'll open another PR to update the hash when we're done here.
I gotta go do a BBQ so I'll be AFK for a while after this review pass.

Precisely this. My credit card isn't too happy but I'm glad I could get something...

I had considered to do the same, but I'm still stuck with my ancient CPU. Without being able to buy a matching graphics card for a reasonable price in the foreseeable future I couldn't bring myself to spend the extra money.

Danke!


exception.h: Fixed minor issues in API DOC

Signed-off-by: “Rico<EMAIL_ADDRESS>

Coverage remained the same at 100.0% when pulling a18a380189dd9be7253c9c3c4af7d27f9e887749 on RicoAntonioFelix:master into 2f8d31762ef6d88a9261ab1b3f931e8991e067a3 on Bareflank:master.


Objects highlighted with pointer are intermittently un-highlighting / flickering

Environment

Github
Latest Github release but have been seeing this issue from version 3.0.1 up to latest Github.
Unity 5.6b3 - b6

Steps to reproduce

Open example scene 003_Controller_SimplePointer
Expand the world WorldObjects object in the hierarchy and select the Person object
On the Person root object, add a Box Collider and change the bounds to generously surround the Person (Size 1.35, 2.8, 1.25 will do)
On the Person root also add a VRTK_InteractbleObject component. Give it a visible Touch Highlight colour. Also add a VRTK_MaterialColorSwapHighlighter component.
Expand the [VRTK] heirarchy object and on one of the controllers add a VRTK_InteractTouch component. On the Simple Pointer component, turn on the 'Interact With Objects' option and set Pointer Visibility to Always_On.
Press play and point the pointer at the Person object. Move the pointer across the object and notice how the highlight regularly flickers on and off as the object is rapidly selected and deselected even though the pointer remains in continuous contact with it.

This also affects the objects 'use' state meaning if you want to 'use' the object it becomes very hard as it is only intermittently selected.
Expected behavior
Item remains highlighted and useable while the pointer is in contact with it's box collider.
Current behavior
Item is only intermittently selected and highlightable.

I've experienced this too in my own scenes.

PR to fix the issue https://github.com/thestonefox/VRTK/pull/913

This definitely did not fix the issue for me. If anything the flickering is worse.

@aantn if you can provide steps to reproduce the issue you're having in an example scene then it can be looked into

Interestingly this completely resolves the issue in the example scene for me but it's still just as broken in my actual scene. I'm going to investigate the differences between my scene and the example scene for differences.

@thestonefox @aantn I've found out why this fix works in the example scene but not in my scene. My camera rig has a Y Rotation of 180 on the transform component. If I remove this, the fix works perfectly. Additionally, if I put a Y Rotation of 180 on the transform of the rig in the example scene, the issue appears there also! I suppose this allows us to 'fix' around the issue in our scenes but it'd be useful to be able to set the initial camera rotation in the scene.

Thanks I'll check if that fixes it for me. (Sorry about the delay... I have
finals.)
On Feb 6, 2017 4:57 PM, "JonDadley"<EMAIL_ADDRESS>wrote:

@thestonefox https://github.com/thestonefox @aantn
https://github.com/aantn I've found out why this fix works in the
example scene but not in my scene. My camera rig has a Y Rotation of 180 on
the transform component. If I remove this, the fix works perfectly.
Additionally, if I put a Y Rotation of 180 on the transform of the rig in
the example scene, the issue appears there also! I suppose this allows us
to 'fix' around the issue in our scenes but it'd be useful to be able to
set the initial camera rotation in the scene.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/thestonefox/VRTK/issues/911#issuecomment-277706919,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAeKBzc8fCFt7Cfe_skRAgJUmL_4UtLVks5rZzTxgaJpZM4L1B92
.

@JonDadley I'll take a look at why there is an issue on rig rotation.
I'm guessing it's to do with the forward direction affecting the shape of the collider.

@JonDadley  @aantn try this PR instead
https://github.com/thestonefox/VRTK/pull/921

It works perfect now. Thanks.

Yep that seems to work better for me too


Add list_pictures function path

When I used from keras.preprocessing.image import list_pictures I got this error:
from keras.preprocessing.image import list_pictures
Using TensorFlow backend.
Traceback (most recent call last):
  File "<input>", line 1, in <module>
ImportError: cannot import name 'list_pictures'

Because list_pictures function was forgotten, I added it.

list_pictures is not explicitly marked as private, but I don't think it should be officially listed as part of the public API. It's not listed in the docs, it's not included in tf.keras.

How can I use this function?
import keras from keras_preprocessing.image import list_pictures
like this?


Divided headings and added new text with regards to issue #29670

Made two separate headings as asked for in the issue #29670  and wrote some additional text under each heading explaining

signed the CLA

/check-cla

Hi @Guneetconvent2002 and welcome 👋
It seems your CLA still isn't detected. Can you check that the email address used for these commits (git log) and the CLA membership match?

also I'm going to /assign and suggest some revisions

Hi @Guneetconvent2002 and welcome 👋
It seems your CLA still isn't detected. Can you check that the email address used for these commits (git log) and the CLA membership match?

That's strange cause I have made a CLA profile and it uses the same email ID as on my github

also I'm going to /assign and suggest some revisions

do you mean that you're going to take over the issue or give me some suggestions to make changes??

Could you maybe help me cause I have the same email ID for both git commit and EasyCLA

Yes, ill review and suggest changes. I'll also take a look at the CLA.

/assign

@jlbutler could maybe have  a look on this and tell me what changes to make?

@jlbutler could maybe have a look on this and tell me what changes to make?

Hi there! :wave:  For your commits, avoid using @'s (as well as # references) in the commit message. As you're working through your edits, you may consider rebasing and collapsing into a single commit (though not necessary right now).
For the content, @gcilne can you spin back through and review edits so far?
Thanks everyone!

@jlbutler could maybe have a look on this and tell me what changes to make?

Hi there! 👋 For your commits, avoid using @'s (as well as # references) in the commit message. As you're working through your edits, you may consider rebasing and collapsing into a single commit (though not necessary right now).
For the content, @gcilne can you spin back through and review edits so far?
Thanks everyone!

Okay, I'll keep in mind in the future. I don't think that I need to undo that commit now. do I?
Also, It's been a week and I have no response regrading any changes to be made so, if you or @geoffcline could look at this and let me know what needs to be done or taken care of?

Just two small changes suggested 👍

I have implemented the changes that you asked for @geoffcline
Hope it's fine now. Also, do I need to delete the commit mentioning you? cause the bot has reacted on it multiple times. If yes, then  could you maybe tell me how I could do it?

/retitle Divided headings: labels, annotations, taints

is this PR ready to merge or are there any other changes that need to made? if there are any please let me know; if not, maybe this PR could be approved

Hi @Guneetconvent2002
Could you squash this down to 1 commit? (If you're not sure how, you can instead let us know that you agree and a maintainer can do the squash for you).

/label tide/merge-method-squash
/lgtm

ok, now it will be automatically squashed when approved :)

@Guneetconvent2002 I'd pinged on this earlier, still unsure if this PR needs to come into dev-1.23 or main. If there isn't anything specific to the 1.23 release (or later), this PR can point at main.

When I started working on this, the query mentioned that changes were required in dev-1.23 specifically. Since the topics where I've made changes; I was not able to find them in the main branch when I was trying to do so

@Guneetconvent2002 can you squash your commits.  One of your commit messages uses an invalid character #

I'm sorry but I don't really know how to do it. Perhaps if the admins could do it. @jlbutler could you maybe help me with that?

@Guneetconvent2002 i'll ping on slack again and see if we can work through it there.

I'll squash these down for you @Guneetconvent2002 using my maintainer access to write to your fork

/approve

/remove-label tide/merge-method-squash


Whether to support the latest version of django2.0 xadmin

Whether to support the latest version of django2.0 xadmin

It is now supported: https://github.com/alexsilva/xadmin-guardian-plugin/tree/bs4_dj2


[docs-infra] Move ApiPage to TS

Extracted from #43128
This PR can be read commit by commit
I tried to move useTranslate from (key, options) => any to (key, options) => string. But some part of the codebase rely on it returning null is the key translation is not defined (the API pages titles) and (key, options) => string | null leads to a lot of place where t('...') needs to be marked as non-null t('...')!. SO I dropped this idea

But some part of the codebase rely on it returning null is the key translation is not defined (the API pages titles) and (key, options) => string | null leads to a lot of place where t('...') needs to be marked as non-null t('...')!.

Perhaps it could make sense to have two complementary functions? One that throws and one that returns null when a  translation doesn't exist. The throwing one can then have return type string. This is a bit like the react testing library approach of using getBy vs. queryBy.


Testing markdown preview editor : Links don't seem to work?

Testing #138040
Opening this file here:
vscode-docs\docs\getstarted\tips-and-tricks.md
And clicking on any of the links doesn't seem to work? Maybe the path is invalid though. Not sure.
Example:
[Running VS Code on Linux](/docs/setup/linux.md)

Fails.
If I change it to this, it works:
[Running VS Code on Linux](..\\setup\\linux.md)

I see the same behavior in my case.Clicking on link that points to md file within the project don't open the  file.Links to website opens up.