How do I contribute?

Hi,
I really like this repo and want to contribute. I am able to add many more subdomains and sites.
Do I just add them to each hosts file and then run remove_duplicate_lines.py? Because I see that everything is in alphabetic order.
Would be nice if you added instructions to the readme.md on how to contribute.

Hi @pornblocking thanks for your interest in contributing.
So how to contribute. This will be put into 3 answers

The current way
The best way
The future way, If @Clefspeare13 approves my suggestions of improvement in https://github.com/Import-External-Sources/pornhosts (still have a few bugs that needs to be solved.)

The current way
Yes you simply add all your domains one by one into each of the hosts file.
I use this bash script for that. (Beta version)
ad_hosts.sh
#!/usr/bin/env bash

# Copyright: https://www.mypdns.org/
# Content: https://www.mypdns.org/p/Spirillen/
# Source: https://github.com/Clefspeare13/pornhosts
# License: https://www.mypdns.org/w/License
# License Comment: GNU AGPLv3, MODIFIED FOR NON COMMERCIAL USE
#
# License in short:
# You are free to copy and distribute this file for non-commercial uses,
# as long the original URL and attribution is included.
#
# Please forward any additions, corrections or comments by logging an
# issue at https://www.mypdns.org/maniphest/

set -e
#set -x

# This Script is only to commit new domains only one ad the time

printf "\nNext porno domain issues\n\n"

read -p "Enter domain to handle as 'domain.tld': " domain

read -p "Enter Clefspeare13 Pornhost Issue ID: " issue

printf "\nAdding domain: $domain\n"
printf "<IP_ADDRESS> $domain\n" >> "<IP_ADDRESS>/hosts"
printf "<IP_ADDRESS> $domain\n" >> "<IP_ADDRESS>/hosts"
printf "<IP_ADDRESS> $domain\n" >> "Mobile_0_0_0_0/hosts"
printf "<IP_ADDRESS> $domain\n" >> "SafeSearch/hosts"

printf "\nGit commit $domain\nwith Pornhost issue ID: $issue\n"

git commit -am "Adding new porno domain \`${domain}\`

Closes https://github.com/Clefspeare13/pornhosts/issues/${issue}
"

For you question about the order, that doesn't matter as is that is sorted with PyFunceble by @funilrys
Next part is the actual contributing part, there is a somewhat guide here https://github.com/Yodatoo/hostfiles/pull/1 , (read it to the bottom)
The best way
Since a git is the perfect history book, the optimal way is for each domain you'll add enough information to the issue/PR to have a proper history of why a contribution is:

Which to be added and confirmed without a secondary lookup for confirmation.
Reason for adding the submission.
Enough information for other to see why this domain was added 3 years ago, and nobody can remember it.
Submit one domain per issue/PR. for example use this template https://github.com/Import-External-Sources/pornhosts/edit/master/.github/ISSUE_TEMPLATE/default.md
Add it secondary to https://github.com/Import-External-Sources/pornhosts/ in a wild-carded format. ex: porndomain.com + www.porndomain.com + cdn.porndomain.com becomes porndomain.com + *.porndomain.com

The future
IF @Clefspeare13 approves my suggestions (in previous mentioned repo) then you still follows the "The Best Way" but you would only need to submit you the domain once in a file as it should be able to add your contribution for 3 projects. 1. This repo, 2. My repo and 3 the rpz zone adult.mypdns.cloud + creating a proper link to origin of contribution and to whom. 3. Automatically take a screenshot for prove :thinking:  But this is in future.....
My idea to this approach is inspired by EasyList repo.
End note
All the above is currently a bit time consuming vs. what else is being seen on this kind of lists, however, @nobody is able to remember why they added anything 2 or 5 years ago. This approach also adding a better option to challenging a suggestion for submission and verify same.
About README
I'll leave this to @Clefspeare13 to decide whether this should be added or not or put on hold.
Please feel free to ask more questions, together we are strongest. :muscle:

@spirillen Thanks for the detailed comment.
Can I just append this to each hosts file?
<IP_ADDRESS> porndomain.com
<IP_ADDRESS> www.porndomain.com

Of course <IP_ADDRESS> for windows

Yep, but please do everyone the favor of also provide the why and vary preferably a screenshot. (It would relieve us from a manual lookup in our own browser)
A history (commit) book is not to much use if every chapters only contains the title :smiling_imp:
And a little note, only add the one that have rel interest. See this thread about www. vs non www. that https://github.com/Clefspeare13/pornhosts/issues/12

@spirillen maybe a contribution guideline may help!
You could also provide a commit.template. Read more about it (scroll down to commit.message)

Hi @funilrys That would be up to @Clefspeare13 to decide, but a very good point which as already have in mind for a later PR... working on some issue templates right now.
Maybe you can remember how to set default reviewers in a PR template... I can't find the line for that
Is it moved to .github/workflows/?
https://help.github.com/en/github/building-a-strong-community/creating-a-default-community-health-file
https://help.github.com/en/github/building-a-strong-community/manually-creating-a-single-issue-template-for-your-repository


Update uProxy Github logo to new colors

Right now it is on the pink&purple version. Here is the current logo you can use below:

Done!


[cudaaligner] added missing virtual and noexcept to destructors

@ahehn-nv looks good to me. Can you explain the value of noexcept in this change? If it is to declare explicitly that the destructors don't throw any exception, where can that information be used?

This change was mostly about clear semantics and a bit about optimization. However, in the meantime, I learned that destructors are (in almost all cases) noexcept by default since C++11. So there is no need to be more verbose about it. Closing.


Error showing while training

While training it's giving error in train.py line number 182 as well as in data/custom_dataset_data_loader.py in line 50 for i, data in enumerate(dataset):. If you can help me!

Sorry Distranta, I forgot to reply.
Did you solve the issue? If not, could you show the error that it's raising?

Yes, I solved it. Thanks for the reply. I'm facing different issue now,
while training it's showing cuda error , no kernel image is available for
execution on the device. Is there any specific reason for such error? If
you can help me with that issue!
On Wed, 9 Jun, 2021, 4:12 pm Pau, @.***> wrote:

Sorry Distranta, I forgot to reply.
Did you solve the issue? If not, could you show the error that it's
raising?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/paumarquez/mono2binaural-conv-tasnet/issues/2#issuecomment-857588609,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQY3H5WBRXZERHJFQCPVDNTTR5ASVANCNFSM46A4PBPQ
.


chore: release main

:robot: I have created a release beep boop
errors: 1.1.1
1.1.1 (2022-08-05)
Miscellaneous

additional relatesToSystems documentation and tests (63f094d)

@dotcom-reliability-kit/log-error: 1.3.2
Dependencies

The following workspace dependencies were updated

dependencies

@dotcom-reliability-kit/serialize-error bumped from ^1.1.0 to ^1.1.1

@dotcom-reliability-kit/middleware-log-errors: 1.2.2
Dependencies

The following workspace dependencies were updated

dependencies

@dotcom-reliability-kit/log-error bumped from ^1.3.1 to ^1.3.2

serialize-error: 1.1.1
1.1.1 (2022-08-05)
Miscellaneous

additional relatesToSystems documentation and tests (63f094d)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/Financial-Times/dotcom-reliability-kit/releases/tag/errors-v1.1.1 :sunflower:

:robot: Release is at https://github.com/Financial-Times/dotcom-reliability-kit/releases/tag/log-error-v1.3.2 :sunflower:

:robot: Release is at https://github.com/Financial-Times/dotcom-reliability-kit/releases/tag/middleware-log-errors-v1.2.2 :sunflower:

:robot: Release is at https://github.com/Financial-Times/dotcom-reliability-kit/releases/tag/serialize-error-v1.1.1 :sunflower:


Add starter to add healthchecks to a webapp

This would be especially useful for Spring Boot apps to be deployed in GAE Flex. Currently, if an app doesn't have a healthcheck endpoint set up, Flex deployments will run until they time out, which can take a really long time.
It appears that Spring Actuator might help setting up a healthcheck endpoint on a custom URL (i.e., not the default one).

I think by default, Spring Boot simply returns HTTP 404 on requests to /_ah/health, and that should be OK for health checks, but we might want more meaningful health check responses.

What do you mean 404 should be OK for health checks?
The end goal here would be to come up with an "App Engine-friendly" config that would allow you to deploy a Spring Boot app without having to disable deployment health checks.

From: https://cloud.google.com/appengine/docs/flexible/python/how-instances-are-managed

You do not have to do anything special to implement health checking. If your app does not handle health checks, a HTTP 404 response is interpretated as a successful reply.
You can write your own custom health-checking code. It should reply to /_ah/health requests with a HTTP status code 200. The response must include a message body, however, the value of the body is ignored (it can be empty).

Hmm I was just able to deploy my Spring Boot app to Flex without any special health check treatment, so my initial diagnosis must have been wrong.
At this point, I'll need to look and see if there is any structure to health check responses, or if's just a trivial response. If the latter, this issue can probably be obsoleted.


Describe pros/cons of 🗝  keycloak

Not designed to be cloud native from the outset 👎🏻

Pros:

Arguably the biggest and most well-known open source IAM / SSO project, with 990 Contributors and >20.000 commits.
Supports OAuth 2.0 and SAML from the outset.
Identity brokering: "Keycloak can also authenticate users with existing OpenID Connect or SAML 2.0 Identity Providers. Again, this is just a matter of configuring the Identity Provider through the admin console. "
Seems KeyCloak has great support for multi tenancy (will only become relevant in the Future, beyond the POC, but still good to know)

Cons:

Great open source community but seems cloud native aspects are tied to the Red Hat enterprise version.
Written in Java. Great for performance, but we are not Java experts.


add db support for "local" storage

add db support for "local" storage (mysql/mongo) instead of current file based repository and make it configurable

Closed because no real need at the moment


cannot get section node

maybe this is the cheerio problem

$('section').html() // this is null

No idea what happened? any update?

I think it's cheerio's problem

@guotie do you have an example page with this issue?


db4: fix pthread configure detection

Description of changes
add header stdlib.h so pthread detection doesn't fail do to a missing function prototype for exit.
configure:21863: clang -o conftest -O3   -D_GNU_SOURCE -D_REENTRANT
conftest.c  >&5
conftest.c:52:2: error: call to undeclared library function 'exit' with
type 'void (int) __attribute__((noreturn))'; ISO C99 and later do not
support implicit function declarations [-Wimplicit-function-declaration]
   52 |         exit (
      |         ^
conftest.c:52:2: note: include the header <stdlib.h> or explicitly
provide a declaration for 'exit'

additionally, add a leading space to line 218 to uncorrupt the patch -- git apply was complaining and failed to apply without fix.
this reverts the previous workaround https://github.com/NixOS/nixpkgs/pull/320187
testing: nix-build -E 'with import ./.{}; db4.override { stdenv = clangStdenv; } builds

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

cc: @@drupol @RossComputerGuy

Don't you think this patch should be submitted upstream instead ?

Don't you think this patch should be submitted upstream instead ?

it is fixed for db5, tho there are still a bunch of patches in nix for that version as well, one being that main needs to return an int. the patches for the various db versions are:
https://github.com/NixOS/nixpkgs/tree/master/pkgs/development/libraries/db
and if i untar the source for db4 the dates are all Apr 12 2010 and the homepage url points to https://www.oracle.com/database/technologies/related/berkeleydb.html so not sure where to send a patch to.
so i get the feeling that this version is pretty old so am not sure where to upstream the patch.

Fair enough.

FWIW this is also needed to fix compiling with gcc14.
nix-build -E 'with import ./.{}; db4.override { stdenv = gcc14Stdenv; }'

fails the same way as if using clangStdenv

change seems fine to me as is and i don't want to put any more time into it so closing.

I’m happy to merge this as‐is if you wanted to reopen. Sorry for nitpicking about the filename when I lacked the full context of the patches.


Add environment info to prepared requests docs

When using prepared requests, the environment is not taken into account. This should be reflected in the documentation.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #4264   +/-   ##
=======================================
  Coverage   88.66%   88.66%
=======================================
  Files          18       18
  Lines        2021     2021
=======================================
  Hits         1792     1792
  Misses        229      229

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 369be8a...785c733. Read the comment docs.


Mid-Quarter OKR Scoring

Hello friends,
It's that time of the quarter... the middle!  And that means big fun:  mid-point scoring of OKRs.
Background on what we do at mid-quarter:  https://github.com/ipfs/pm/issues/668
For libp2p, I want to follow the same process as David's issue above.  I'd like each KR owner to assign a mid-quarter score to your KRs this week.  On next week's call, we'll do a check-in as @diasdavid describes in https://github.com/ipfs/pm/issues/668.
Updating the OKR sheet is a great opportunity to communicate how things are going, where you might need help, thoughts on whether we should change plans, concerns or frustrations, etc.
Remember that we're mostly focused on the P0s for this quarter.  But let's score everything.
@bigs:  You have P0 OKRs on the Rust tab, Go tab and PL tab.
@diasdavid:  You have P0s on the JS tab, overall project tab.
@marten-seemann:  You have P0's on Go tab, overall project tab
@Stebalien:  You have P0s on Go tab, PL tab
@jacobheun & @vasco-santos:  You both have P0's on the JS tab
@whyrusleeping:  You have P0s on the Go tab, PL tab
@mgoelzer:  I've scored all my P0's.  I'll make another pass to get the others scored.  Oh, and I'll fix those broken formulae at the top.
/ cc @lgierth (you have some P1 KRs)
/ cc @raulk (for general interest)

✅ @diasdavid: You have P0s on the JS tab, overall project tab. Note that the Overall Project Tab ones are just the reflection of OKRs on the JS tab

Closing - thanks everyone.


Gotcha:  Max 100 items per batch!?!

I tried to create 101 documents in a batch and received a 400 (Bad Request) result from the server with the error Batch request has more operations than what is supported.
Creating 100 documents works fine.  Clearly, there is a limit of 100 operations per batch.
This is consistent with the table storage API.  But ...really... who's ever heard of that, even :D
There's no documentation I could find anywhere which says the SQL API has the same limitations.  The only hint I could find was this issue over here: https://github.com/Azure/azure-documentdb-datamigrationtool/pull/39
I suggest:

Add documentation that 100 docs is the max
Throw an exception when adding operation 101 to TransactionalBatch.  This makes it clear what the problem is exactly when it happens

...I also suggest removing the 100 doc and 2mb payload limit altogether, but... ...yeah, good luck with that :D

FYI: While testing, I was able to create ~8400 documents (empty-ish) with a single call to a stored procedure on a collection with only 400 RU/s allocated.  Results are variable, of course.

@abhijitpai Do we have the differences between Stored Procedures and Batch documented somewhere?

We have the transactional batch limit now documented at https://docs.microsoft.com/en-us/azure/cosmos-db/concepts-limits#per-request-limits. We will look at increasing this over time but as discussed in https://github.com/Azure/azure-cosmos-dotnet-v3/issues/1059 the limit allows for predictability.


Cloudflare CDNJS is outdated.

This package is available via the cloudflare CDN: http://cdnjs.com/libraries/bootstrap-rtl
But the latest package there is only 3.2.0-rc2
Can this be updated?

Sure, it's just cumbersome process. Read cdnjs readme if you are interested submitting the latest version to cdnjs.
Otherwise you can use rawgit cdn (#50). In this case, the cdn url to the latest version is http://cdn.rawgit.com/morteza/bootstrap-rtl/master/dist/cdnjs/3.3.1-rc2/css/bootstrap-rtl.min.css


Add support for RANDOMKEY Command

We need to implement the RANDOMKEY command similar to the RANDOMKEY command in Redis. Please refer to the following commit in Redis to understand the implementation specifics - source.

Write unit and integration tests for the command referring to the tests written in the Redis codebase 7.2.5.
For integration tests, you can refer to the tests folder. Note: they have used TCL for the test suite, and we need to port that to our way of writing integration tests using the relevant helper methods. Please refer to our tests directory.
For the command, benchmark the code and measure the time taken and memory allocs using benchmem and try to keep them to the bare minimum.

Please assign this to me, i will understand code base first. Good for initial understanding of code flow.

Please assign this to me, i will understand code base first. Good for initial understanding of code flow, I have un-assign myself from other issue.

Assigned

@codeimmortal if you are not working on this, can I pick this one?

@JyotinderSingh / @soumya-codes - Can you please assign this issue? Would like to try the SCAN approach to get the next valid key for this issue.

This is yours @ArjunMenonPe . Thanks for picking this up.

Hello @ArjunMenonPe,
There has been no activity on this issue for the past 5 days.
It would be awesome if you keep posting updates to this issue so that we know you are actively working on it.
We are really eager to close this issue at the earliest, hence if we continue to see the inactivity, we will have to reassign the issue to someone else. We are doing this to ensure that the project maintains its momentum and others are not blocked on this work.
Just drop a comment with the current status of the work or share any issues you are facing. We can always chip in to help you out.
Thanks again.

Hello @ArjunMenonPe,
There has been no activity on this issue for the past 5 days. It would be awesome if you keep posting updates to this issue so that we know you are actively working on it.
We are really eager to close this issue at the earliest, hence if we continue to see the inactivity, we will have to reassign the issue to someone else. We are doing this to ensure that the project maintains its momentum and others are not blocked on this work.
Just drop a comment with the current status of the work or share any issues you are facing. We can always chip in to help you out.
Thanks again.

I would like to pick this, if there has been no activity. Thanks

@JyotinderSingh @soumya-codes Could you please assign this to me?

Assigned, thanks for picking this up. However, please keep in mind that implementing RANDOMKEY in our setup has proven to be a little challenging up until now, particularly due to the data structures we are using internally. Feel free to discuss any blockers on the Discord channel.

@JyotinderSingh @soumya-codes Could you please assign this to me?

Assigned, thanks for picking this up. However, please keep in mind that implementing RANDOMKEY in our setup has proven to be a little challenging up until now, particularly due to the data structures we are using internally. Feel free to discuss any blockers on the Discord channel.

Thank you. Sure, could you please point me to where I can find the link to join the Discord channel?

Just wanted to give an update, I am still working on it and hope to raise a PR tomorrow

Hey @Vidyap23, just stumbled upon this, and read the RANDOMKEY implementation of Redis.
Are you working on this? If yes, how are you making sure that this command runs in O(1) with current DB methods (store/store.store() methods) we currently have in DiceDB?

Hey, if this is open I would like to work on it. I was tinkering around and kindof have a working prototype. But will need to discuss some trade-offs before initiating the PR

Hello @Vidyap23,
There has been no activity on this issue for the past 5 days.
It would be awesome if you keep posting updates to this issue so that we know you are actively working on it.
We are really eager to close this issue at the earliest, hence if we continue to see the inactivity, we will have to reassign the issue to someone else. We are doing this to ensure that the project maintains its momentum and others are not blocked on this work.
Just drop a comment with the current status of the work or share any issues you are facing. We can always chip in to help you out.
Thanks again.

Hey @JyotinderSingh , would like to figure out how we can do it in O(1) or similar. Please assign to me.

Hello @xanish,
There has been no activity on this issue for the past 5 days.
It would be awesome if you keep posting updates to this issue so that we know you are actively working on it.
We are really eager to close this issue at the earliest, hence if we continue to see the inactivity, we will have to reassign the issue to someone else. We are doing this to ensure that the project maintains its momentum and others are not blocked on this work.
Just drop a comment with the current status of the work or share any issues you are facing. We can always chip in to help you out.
Thanks again.

Hey @arpitbbhayani I have initiated the MR, just waiting for confirmation if it can be merged or needs any further changes.

Hey @JyotinderSingh , may I work on this issue since it has been inactive for quite a while

Hey @JyotinderSingh , may I work on this issue since it has been inactive for quite a while?

This issue does not require further work at this time.


Bug: NormalizationRule::PushPredicateThroughJoin is executed incorrectly during correlated subqueries

          > It seems that there is something wrong with `HashJoin`, I'm debugging now, I'll merge this first

ops, I found the problem. In correlated subqueries, there is no left data in right.
so this case will return empty data (wrong)
let fnck_sql = DataBaseBuilder::path(temp_dir.path())
    .build()
    .await?;
let _ = fnck_sql
    .run("create table t2(id int primary key, a int not null, b int not null);")
    .await?;
let _ = fnck_sql
    .run("create table t3(id int primary key, a int not null, c int not null);")
    .await?;
let _ = fnck_sql
    .run("insert into t2 values (0, 1, 2), (3, 4, 5), (8, 0, 0);")
    .await?;
let _ = fnck_sql
    .run("insert into t3 values (0, 2, 2), (3, 8, 5);")
    .await?;
let (schema, tuples) = fnck_sql.run("select id,a,b from t2 where id in (select a from t3 where a > t2.a);").await?;
println!("{}", create_table(&schema, &tuples));

there is only t3.a on the right but no t2.a, so filter always returns false
+--------------------------------------------------------------------+
| PLAN                                                               |
+====================================================================+
| Projection [t2.id, t2.a, t2.b] [Project]                           |
|   LeftSemi Join On t2.id = (t3.a) as (_temp_table_0_.a) [HashJoin] |
|     Scan t2 -> [id, a, b] [SeqScan]                                |
|     Projection [(t3.a) as (_temp_table_0_.a)] [Project]            |
|       Projection [t3.a] [Project]                                  |
|         Filter (t3.a > t2.a), Is Having: false [Filter]            |
|           Scan t3 -> [a] [SeqScan]                                 |
+--------------------------------------------------------------------+

I think this problem is caused by NormalizationRule: PushPredicateThroughJoin
Originally posted by @KKould in https://github.com/KipData/FnckSQL/issues/192#issuecomment-2028094727

mistake


Adding code example about Physics-Informed Neural Networks for solving Eikonal equation

Hello,
I would like to contribute to the Keras examples by adding a code example implementing Physics-Informed Neural Networks (PINNs).
This example would be focused on solving the Eikonal equation via implementation of the Neural Eikonal Solver which is based on recent research proposing fast and robust neural-network solution to the equation. The Eikonal equation is a nonlinear Partial Differential Equation (PDE) and it might be of interest to a wide audience because it is used in 3D rendering and computer vision via Signed Distance Functions, as well as commonly used in seismology. The proposed code example will also demonstrate the usage of AutoDiff operations (tf.gradients) inside the trainable models to solve the PDE.
The example would contain the implementation of the NES architecture and its training on one example with a complex model. The training of ~1-2 min will be enough to provide a very accurate solution.
To date, the Keras repository does not have examples about Physics-Informed Neural Networks.
Please, let me know if this example could be a valuable contribution to the Keras repository.
Thank you for your time.

Yeah, I would like to contribute in Physics Informed Neural Network and I am also currently working on project on PIINs in my college i would like to know the possibilities of PIINs in different domains

That sounds great! Please feel free to open a PR. I think it would make a valuable addition.


What is the purpose of the Recipes(especially typescript) in the documents?

While reviewing the documentation, I noticed that the articles on Typescript, Markdown, and Asset Handling in the "Recipes" section are still in progress. What is the purpose of these items? Given the content of other Recipes documents, such as MDX Support and Code Highlighting, it seems that the intention might be to explain code handling the respective resources (type, markdown, assets).
However, types are well described at https://velite.js.org/reference/types, and if it's about schema types, they are detailed at https://velite.js.org/guide/velite-schemas. Additionally, the content in the Markdown document could be easily adapted from the MDX Support article with minor modifications. What is the intended purpose of the Typescript and Markdown documents in Recipes?
If it's okay, I would like to help with writing the documentation. Perhaps starting with the seemingly lacking Asset Handling could be a good place to start. Could you tell me what content you expect in those sections(Typescript, Markdown, and Asset Handling in "Recipes")?

plus, in the Javascript API section( https://velite.js.org/reference/api ) type signature of the Entry, Result, outputFile(maybe same with outputEntry in code), outputImage(maybe same with outputAsset in code) are different from the current code in github.
If these differences are mistake or the intention for the future improvement, whatever, I would like to help fixing/improvement.

First of all, thank you very much for your willingness to help Velite's community.
My original idea about recipes in the document is that it can help users to use Velite better and provide some personal best practices in the past
Markdown:

How to organize the structure of content and assets
Description of internal integrated plug-ins
Autolink heading
Toc
Image processing. It is better to use a separate recipe, like code highlighting, because it may be more popular
Custom plug-in implementation is similar to GitHub Alert

Asset Handling:

Briefly describe how Velite's resource processing works
Some possible use scenarios, such as uploading assets to OSS, examples maybe

TypeScript:

Using Velite in TypeScript
Better support for Velite in TypeScript priority projects

Of course, this is just my idea. If you want, you can adjust the document structure according to your idea.
Thanks again

https://codehike.org/docs/introduction#limitations
Velite built-in s.mdx() use the MDX 3.x. it's a bit awkward.
If you must use this plug-in, you can consider implementing an mdx v2 schema yourself

https://codehike.org/docs/configuration#auto-import

Thank you for the kind explanation regarding the original purpose. I plan to familiarize myself with the code of Velite first, then work on the documentation for Markdown, Asset Handling, and Typescript, and contribute to the code if needed.
If it's alright, I will link the documentation-related PR to this issue later.


link to blog

Hi,
Suggest adding a link to https://www.microsoft.com/itshowcase/Article/Content/1040/Streamlining-business-processes-with-SAP-connectors-and-Azure-services
As it gives the background for the code in the repo

Thanks for the suggestion. We are working on a refined documentation. We will upload in next couple of days. Thanks.

We have uploaded the enhanced documents. Closing this issue.


Fix display of wrapped whitespace in show-whitespace

Fixes: #2991
Uses clipped SVG background images for spaces and tabs, this makes it more flexible for text wrapping, and any other future characters (#2150).
Test

text wrapping issue from #2991

standalone file

diff

gist

This PR is still in the works, as SVG content used are static paths, which should ideally be text nodes that inherit font from DOM (Figma still flattens all text nodes while exporting).
I tried to match the existing styles (and size) as much as possible, but we might have to reconsider that anyway (i.e. using actual text).

Edit: Updated GIFs

Great idea if it works reliably. However I’d preserve the previous opacity, this one isn’t very visible.

That's just the very thin tab character from Inconsolata flattened to <path> in SVG.
I'm still working on using <text> nodes here to get proper font rendering (if possible). Wanted to put this PR out for sharing the idea.
(As mentioned Figma flattens all text nodes to <path>s in SVG, and I'm still afraid to use Inkscape 😅, will have to find some other vector edit tool)

You can make the SVG from scratch, it’s just 2 tags (text tag inside svg tag)

Looks like there is no way for SVG backgrounds (either using data: or http:) would inherit properties like font and line-height from parent DOM. Even if that were to happen, it would be very complex to handle, especially baseline was very tricky with background-size.
Instead I'd propose the use of static path elements, this way the backgrounds says consistent on any platform regardless of the font.
Open to other ideas as well.

That’s right 👍 let’s match the current color though

That’s right +1 let’s match the current color though

Done!

This is a clever solution and it appears to work. Let's see if it causes any issues in the real world, I'll publish this today.


How to write $canActivate component hook in typescript

I am using Angular 1.5.5 with typescript.How Can I write canActivate hook for my component?
I can write other hooks but don't know about canActivate hook.
Below is my code.
 class workspaceOptions implements ng.IComponentOptions {
    public controller: any;
    public templateUrl: string;
    public bindings: any;
    public $routeConfig: any;

    $canActivate() {
        console.log('check can active');
    }
    constructor() {
        this.bindings = {
            $router: '<'
        };
        this.templateUrl ='app/workspace/workspace.html' ;
        this.controller = workspaceCtrl;
        this.$routeConfig = [
            { path: '/calmenu/...', name: 'Calmenu', component: 'calmenu', useAsDefault: true }
        ];
    }
}

Current Behavior
$canActivate() is not getting called.
Expected Behavior
Call $canActivate(), inject service and set routing dynamically based on service data.

It depends on how you are registering your directive/component.
$canActivate should be a property of the DDO/CDO.
Here is an example using plain JS.
--
That said, this sounds more like a general support question, so please use one of the appropriate support channels.
GitHub issues are reserved for bug reports and feature requests.


Sub-Model used as layer does not support masking

Documentation for functional API says that you can "treat any model as if it were a layer" this works unless you try to use masking. Simple example that does not compile with mask_zero=True:
import numpy as np
from keras.layers import Input, Embedding, Dense, GRU
from keras.models import Model

X_words = np.random.randint(2, size=(5, 10))
Y_first = X_words[:, 0].astype(np.float32)

words_in = Input(shape=(10,), dtype='int32', name='x')
#words_emb = Embedding(2, 10)(words_in)
words_emb = Embedding(2, 10, mask_zero=True)(words_in)

### a) masking works with GRU layer:
#x = GRU(10, return_sequences=False)(words_emb)

### b) but not with an equivalent sub-model used as layer:
def build_submodel(batch_shape):
    inputs = Input(batch_shape=batch_shape)
    outputs = GRU(10, return_sequences=False)(inputs)
    return Model(input=inputs, output=outputs)
x = build_submodel(batch_shape=words_emb._keras_shape)(words_emb)

y = Dense(1, activation='softmax', name="y")(x)

model = Model(input=[words_in], output=[y])
model.compile(optimizer='adam', loss='mse')
model.fit([X_words], [Y_first], nb_epoch=100)

Output (with latest Keras master):
$ ./test_model_mask.py
Using Theano backend.
Traceback (most recent call last):
  File "./test_model_mask.py", line 32, in <module>
    x = build_submodel(batch_shape=words_emb._keras_shape)(words_emb)
  File "/home/app/.local/lib/python2.7/site-packages/keras/engine/topology.py", line 515, in __call__
    self.add_inbound_node(inbound_layers, node_indices, tensor_indices)
  File "/home/app/.local/lib/python2.7/site-packages/keras/engine/topology.py", line 573, in add_inbound_node
    Node.create_node(self, inbound_layers, node_indices, tensor_indices)
  File "/home/app/.local/lib/python2.7/site-packages/keras/engine/topology.py", line 150, in create_node
    output_tensors = to_list(outbound_layer.call(input_tensors[0], mask=input_masks[0]))
  File "/home/app/.local/lib/python2.7/site-packages/keras/engine/topology.py", line 2050, in call
    output_tensors, output_masks, output_shapes = self.run_internal_graph(inputs, masks)
  File "/home/app/.local/lib/python2.7/site-packages/keras/engine/topology.py", line 2193, in run_internal_graph
    output_masks = to_list(layer.compute_mask(computed_tensor, computed_mask))
  File "/home/app/.local/lib/python2.7/site-packages/keras/engine/topology.py", line 608, in compute_mask
    'but was passed an input_mask: ' + str(input_mask))
Exception: Layer input_1 does not support masking, but was passed an input_mask: Elemwise{neq,no_inplace}.0
> /home/app/.local/lib/python2.7/site-packages/keras/engine/topology.py(608)compute_mask()
-> 'but was passed an input_mask: ' + str(input_mask))

I guess you can do some modification, like setting self.supports_masking=True in class InputLayer

Is it possible to monkey-patch this at runtime? Since this recent refactoring I am confused how to access individual layers?

any updates on this?

Any updates? I ran into the same problem

I stumbled upon the same issue. Any updates on this?

Hi, I got the same problem.
I loaded a model to use as a block in a residual model. I set the model.support_masking=True but got a similar error. My first layer is a masking layer the error  is as follows:

TypeError: Layer masking_7_input does not support masking, but was passed an input_mask: Tensor("masking_14/Any_1:0", shape=(?, 622), dtype=bool)

Hi,Is this problem solved? I got the same problem.

same problem here. why the issue is closed???


The getting started documentation needs instructions on how to install the Kafka admin utils

This document should have the instructions:
https://github.com/linkedin/kafka-tools/blob/master/docs/getting_started.rst


Remove Unbox dependency

Context
After the project has been migrated to Swift 4 we can remove the Unbox dependency and use the language encoding/decoding feature.
What
Use the language encoding/decoding feature instead of Unbox.
Proposal

Update all the models that support decoding.

Maybe we should update the description now that #84 was merged.
Context mentions #82 which ended up not being the «correct» one for this issue

Updated @esttorhe 😛


vr command not found after installation

Command I ran from https://velociraptor.run/docs/installation/ :
deno install -qAn vr<EMAIL_ADDRESS>
New terminal window in zsh
$ vr
zsh: command not found: vr

Tried to reinstall it by running command again (with -f flag) but get a module not found error
error: Cannot resolve module "file:///Users/<username>/pathtomyapp/vr".

MacOS Catalina 10.15.4
deno 1.13.1 (release, x86_64-apple-darwin)
v8 9.3.345.11
typescript 4.3.5
Let me know if there is any other information I can provide. Thanks!

@eklemen Have you added Deno's executable installation root to your PATH?

This fixed it. My apologies!


DirectLine Conversation Ids get expired?

We store conversation ids (direct line) against our application users which will be later used to resume conversations. However, when trying to resume a conversation after keeping them inactive for several days, directline service returns HTTP 404 error. Please refer the issue  where it's mentioned that conversation ids do not expire. Is this the expected behavior?

I am not aware of an expiration for Conversation Ids.  As Dan Driscoll stated here: https://github.com/Microsoft/BotFramework-DirectLineJS/issues/9#issuecomment-283555436

Are conversation IDs valid forever? The ID doesn't have a lifetime, but the contents of the conversation slowly expire, and I've heard requests for us to support the user/bot sending EndOfConversation, which would terminate communication.

What code are you using to re-connect to the conversation?  What call is resulting in a 404?

Update: apparently conversation ids do now expire.
https://github.com/Microsoft/BotFramework-DirectLineJS/issues/119#issuecomment-445066667

Conversation messages are only stored for 24 hours in DirectLine. Conversation metadata (such as conversation id and couple other housekeeping data) are removed after a period of inactivity (between 15 to 20 days).

Thanks for the update @EricDahlvang. Well, if that's case, the only option we have is to re-create conversations. Need to figure out a way to expire the user/conversation/private conversation state automatically as well. I'm thinking of using Time to live feature as we are using CosmosDB for state persistence. Is there a recommended approach?

I'm not aware of a recommended approach to expiring a conversation in Direct Line.  Using time-to-live sounds like a great idea for the bot and storage side.  Something else might be required for the client.
I'm closing this, as the question has been answered.  Thank you for opening this issue.  I was not aware of the protocol change with expiring conversation ids until today. (I've updated my previous comment)

We will update the client accordingly. Many thanks @EricDahlvang.

Then how can we handle the old conversation data of the user. If we need to show the transcript history of the user. Is there any recommended way of that ?

@neetibirla If you use TranscriptLoggerMiddleware and a Transcript Store implementation, you can retrieve conversation history at any time and send it to the Direct Line service using the SendConversationHistory  API.  A Node.js example can be found in this closed pull request: https://github.com/Microsoft/BotBuilder-Samples/pull/1266
Mapping multiple conversations to a single user, and managing conversations with conversation history for a specific user is left to the developer.


How can I end the speech to text recognition and switch to recognize command.

                                  var addPageDesc = function(){
				$('#pageDescription').focus();

				var r = document.getElementById('pageDescription');
				var speechRecognizer = annyang.getSpeechRecognizer();
				speechRecognizer.continuous = true;
				speechRecognizer.interimResults = true;

				var finalTranscripts = '';

				speechRecognizer.onresult = function(event){
					var interimTranscripts = '';
					for(var i = event.resultIndex; i < event.results.length; i++){
						var transcript = event.results[i][0].transcript;
						transcript.replace("\n", "<br>");
						if(event.results[i].isFinal){
							finalTranscripts += transcript;
						}else{
							interimTranscripts += transcript;
						}
					}
					r.innerHTML = finalTranscripts + interimTranscripts;
				};
				speechRecognizer.onerror = function (event) {
				};
			};

I can switch from command recognition to speech to text recognition. I want to know how can I end the
speech to text recognition and  switch back to command recognition.
Thanks

Your code effectively overwrites annyang's onresult event handler so that it cannot listen to any further results.
This is way outside of what annyang was designed to do, as your code effectively "breaks" annyang... but here are two ideas of what you might try.

Instead of speechRecognizer.onresults = func, try speechRecognizer.addEventListener('result', func). This will not delete annyang's speech recognition. Be sure to pause annyang when you want to use your "text recognition" mode.
I have never tried this... but it may be possible to create more than one speech recognition object in one window. Let annyang do it's thing, and when you want to switch to "text recognition", pause annyang and use your own speech recognition (replacing var speechRecognizer = annyang.getSpeechRecognizer(); with var speechRecognizer = new webkitSpeechRecognition();

Thanks for your reply with possible solution.
I will pause annyang when I want to use your "text recognition" mode.  How can I decide that I have done with 'text recognition' want to switch back to 'command recognition'.
Thanks,
Anil Kumar

When you want to turn annyang back on just run annyang.resume().
I encourage you to read the API docs to see all of the available options.


Kmonad debugger says it is emitting the key press, but it is not showing up

Below you will see that the debugger says that the keys are being emitted, but no text is showing up on the right. Any advice?
Using dext on MacOS Monterey

kbd file is:
;; See keymap/tutorial.kbd for a more complete tutorial

(defcfg
  input (iokit-name "Apple Internal Keyboard / Trackpad")
  output(kext)
  fallthrough false

)

(defsrc
  esc  f1   f2   f3   f4   f5   f6   f7   f8   f9   f10  f11  f12
  grv  1    2    3    4    5    6    7    8    9    0    -    =    bspc
  tab  q    w    e    r    t    y    u    i    o    p    [    ]    \
  caps a    s    d    f    g    h    j    k    l    ;    '    ret
  lsft z    x    c    v    b    n    m    ,    .    /    rsft up
  fn   lctl lalt lmet           spc            rmet ralt left down rght
)

;; Below, I use `around' to compose `layer-toggle' with `fn' (instead
;; of using `layer-toggle' directly) so that the `fn' keypress
;; actually gets emitted.


I hope when alert in silence status, I can receive the notice that alert has been resolved

What did you expect to see?
I hope when an alert is in silence status, I can receive the notice that the alert has been resolved .
when I use alertmanager, I am sure the message be sent to alertmanager must has endsAt time，
Let me describe how we used,

when an alert has been sent to the user
user ack this alert, make the alert be silenced 15 minutes
but just 5 minutes this alert has been resolved, and now this alert during be silenced time, I want to receive resolved message

Alertmanager version:

  alertmanager, version 0.22.2 (branch: HEAD, revision: 44f8adc06af5101ad64bd8b9c8b18273f2922051)
  build date:       20210602-07:50:37
  go version:       go1.16.4
  platform:         linux/amd64

Alertmanager configuration file:

global:
  resolve_timeout: 30d

route:
  group_by: ['app', 'id', 'rule', 'severity']
  group_wait: 30s
  group_interval: 5m
  repeat_interval: 5m
  receiver: 'webhook'

inhibit_rules:
  - source_match:
      severity: 'critical'
    target_match:
      severity: 'warning'
    equal: ['id']

receivers:
- name: 'webhook'
  webhook_configs:
    - url: 'http://<IP_ADDRESS>:8888/xxxxx'

It provided a api to notify a server when silence expired in some old version but removed now. So that seems we can only search the result by self.

https://github.com/prometheus/alertmanager/pull/2811
I have a pull request, it can make message can be sent when alert in silence

this is a duplicate of #226

It looks like yes, I've used it in production and can achieve the purpose of receiving resolved messages during silent periods. In my implementation, a configuration file parameter "receive_silence_resolved" has been added to ensure that the resolved message is obtained during the execution phase of 'MuteStage' and sent out through the webhook. Now should I update the document?

I'm going to close this in favor of #226 since @sthaha should start collecting requirements soon.


Added canvas to pipeline template cards

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


i can't use renderContent

i can't use renderContent

Okay, with what code are you failing?

i'm getting an error: "_this.props.renderContent is not a function, is an instance of object"
<Lightbox renderContent={this.renderCarousel(event)} />

@vcebotari did you manage to solve this problem? having the same issue here


Report an error when item is empty

When I tested, I used an rss with an empty item, and his error was as follows
TypeError: Cannot read properties of undefined (reading 'map')
    at mapRss2ToFeed (https://deno.land/x/rss@0.5.8/src/mappers/mapper.ts:245:26)
    at toFeed (https://deno.land/x/rss@0.5.8/src/mappers/mapper.ts:33:14)
    at parseFeed (https://deno.land/x/rss@0.5.8/src/deserializer.ts:36:10)
    at eventLoopTick (ext:core/01_core.js:181:11)
    at async Server.handler (file:///src/main.ts:16:23)
    at async Server.#respond (https://deno.land/std@0.177.0/http/server.ts:299:18)

Hi, thank you for reporting on this issue and your PR 🤝, your changes are included in the 0.6.0 release.
I'll extend our tests so we cover this issue for all the feed types.
BR
Micke

No, I'm happy to contribute to this package


--tags option missing for "iot hub" and "iot dps"

The --tags option appears to be missing from the az iot hub create and az iot hub update commands.
It's not documented, and trying to use it results in an error az: error: unrecognized arguments: --key=.
I'd consider this a bug rather than a feature request, as every other create/update command seems to have a --tags option; --tags is like part of the create/update contract.

iot

@Azure/azure-iot-cli-triage

@cocowalla @yonzhan @yiyiguo --tags should be supported in update/create for both Hub and DPS

Closing the issue due to no response. If you still need help with this issue, please re-open.


Concierge message has extra breaks - Reported by @thesahindia

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

Action Performed:

Open app
Navigate to Concierge chat
Type any random chat

Expected Result:
Message in Concierge should have proper spacing
Actual Result:
Some message received in Concierge has double spacing some has none
Workaround:
unknown.
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: 1.1.33-0
Reproducible in staging?: Y
Reproducible in production?: Y
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
Issue reported by: @thesahindia
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1642794817283200
View all open jobs on GitHub

Triggered auto assignment to @laurenreidexpensify (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

https://www.upwork.com/jobs/~01783677d9cfaf2e77

Triggered auto assignment to Contributor-plus team member for initial proposal review - @parasharrajat (Exported)

Triggered auto assignment to @marcochavezf (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

Actually @marcochavezf I may have jumped ahead here - can you confirm this is okay for External?

Actually @marcochavezf I may have jumped ahead here - can you confirm this is okay for External?

Yeap, this one looks good for External

Proposal
Root cause
the Concierge  send some message as HTML and we render the HTML
congrats send the message with paragraph tag

 and this tag have some padding and margin by default.
Fix
sense we use react-native-render-html we can add tagsStyles  to p tag and this solve the issue .
will add same padding we use between message 8px top and bottom .
in https://github.com/Expensify/App/blob/2c14f01af550f554bc1f488a35654c36b437ba65/src/pages/home/report/ReportActionItemFragment.js#L66
-<RenderHTML
-        html={`<comment>${props.fragment.html + (props.fragment.isEdited ? '<edited></edited>' : '')}</comment>`}

+ <RenderHTML
+      html={`<div>${props.fragment.html + (props.fragment.isEdited ? '<edited></edited>' : '')}</div>`}
+      tagsStyles={{
+          p: {
+              margin: 0,
+              paddingTop: 8,
+              paddingBottom: 8,
+          },
+      }}
+    />

we will move the style to style file
also I notice we use RenderHTMLSource and it's not support the tagStyles  so in
https://github.com/Expensify/App/blob/2c14f01af550f554bc1f488a35654c36b437ba65/src/components/RenderHTML.js#L4
I will import the full Feature component
import RenderHtmlSource from 'react-native-render-html';

and pass the props to component  as {...props}
preview
before : -

after : -

Proposal
Reason
The two text are rendered inside two sepatate paragraph element. The default styling for the paragraph has marginTop and marginBottom set to 14. Therefore if two paragraphs are adjacent. It will double the spacing to 28.

Solution
https://github.com/Expensify/App/blob/2c14f01af550f554bc1f488a35654c36b437ba65/src/styles/styles.js#L118-L124
Set marginTop to 0
        img: {
            borderColor: themeColors.border,
            borderRadius: variables.componentBorderRadiusNormal,
            borderWidth: 1,
        },

+        p: {
+           marginTop: 0,
+        }
    },

Result

Both proposals are good but I like @mdneyazahmad 's proposal more as he pointed out the correct location where the change is needed.
@ahmdshrif :-1:  -1 for not noticing why we are using import {RenderHTMLSource} from 'react-native-render-html'; . If we go with your suggestion it will hurt chat list performance badly.

:green_circle: signal for @mdneyazahmad 's proposal
Although, we need to remove the margin-bottom as well, this is just part of design tweaks.
cc: @marcochavezf
:ribbon: :eyes: :ribbon:  C+ reviewed

yes, Mohamed does the same but from the correct place.  👌

Although, we need to remove the margin-bottom as well, this is just part of design tweaks.

I think here you will need this margin because we need space between the 2 paragraphs (assume them as 2 messages ).
if you delete it and  I suggest adding padding with the same value we use between messages 8 top and bottom as I do in my proposal .

I see thanks for pointing out.

📣 @mdneyazahmad You have been assigned to this job by @marcochavezf!
Please apply to this job in Upwork and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑💻
Keep in mind: Code of Conduct | Contributing 📖

@mdneyazahmad please apply to this job on upwork so we can hire you thanks

@laurenreidexpensify applied

The solution for this issue has been :rocket: deployed to production :rocket: in version 1.1.39-3 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

https://github.com/Expensify/App/pull/7592

If no regressions arise, payment will be issued on 2022-03-02. :confetti_ball:

@thesahindia @mdneyazahmad I've issued payment
@parasharrajat just waiting for you to accept contract in UW and then I'll issue for you 👍🏽

and everyone is paid now. nice work all.


chore(website): trying to fix playground

Description
This PR tries to fix the playground.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.516%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5600

Relevant Lines:
5901

💛  - Coveralls


File check should be clickable

Firstly, great job with this project!!
The only thing that I find kind of awkward is that only the filename is clickable in the git-control view. I would have expected even the check icon to it's side to be clickable as it's something that (at least) I would naturally want to do: uncheck a checked item, or vice versa, by clicking the check[box].
So, it would be pretty nice to do be able to do this as I find it annoying at times...

Should this be closed?


no installation proccess

Hi,
i dunno how to install this, there's no option from azure data studio, only a link to github
thanks

Hi,
When you press the install button in Azure Data Studio
it takes you to the Releases page
and there you have to download the vsix file

Hi,
When you press the install button in Azure Data Studio
it takes you to the Releases page
and there you have to download the vsix file (https://github.com/pacoweb/extraSqlScriptAs/releases/download/0.3.0/extra-sql-script-as-0.3.0.vsix)

yeah but then i dunno what to do with that vsix,,

Hi,
You must use the Install from VSIX option on extension´s section in Azure Data Studio
I paste a gif with how to do it

thanks! thats great :D


Fix AWS SDK missing symbols on Linux PPC

TF on linux ppc64le builds but when loading the environment, some AWS related symbols are missing and thus resulting in a failure to load TF. This patch adds the same file inclusions/macro definitions to linux_ppc as one does to linux_x86 and this seems to fix the problem.
Error message reads:
Traceback (most recent call last):
  File "gru_ops_test.py", line 23, in <module>
    from tensorflow.contrib.rnn.python.kernel_tests import benchmarking
  File "/localhd/tjin/tensorflow_latest/local/lib/python2.7/site-packages/tensorflow/__init__.py", line 24, in <module>
    from tensorflow.python import *
  File "/localhd/tjin/tensorflow_latest/local/lib/python2.7/site-packages/tensorflow/python/__init__.py", line 49, in <module>
    from tensorflow.python import pywrap_tensorflow
  File "/localhd/tjin/tensorflow_latest/local/lib/python2.7/site-packages/tensorflow/python/pywrap_tensorflow.py", line 72, in <module>
    raise ImportError(msg)
ImportError: Traceback (most recent call last):
  File "/localhd/tjin/tensorflow_latest/local/lib/python2.7/site-packages/tensorflow/python/pywrap_tensorflow.py", line 58, in <module>
    from tensorflow.python.pywrap_tensorflow_internal import *
  File "/localhd/tjin/tensorflow_latest/local/lib/python2.7/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 28, in <module>
    _pywrap_tensorflow_internal = swig_import_helper()
  File "/localhd/tjin/tensorflow_latest/local/lib/python2.7/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 24, in swig_import_helper
    _mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
ImportError: /localhd/tjin/tensorflow_latest/local/lib/python2.7/site-packages/tensorflow/python/_pywrap_tensorflow_internal.so: undefined symbol: _ZN3Aws4Time9LocalTimeEP2tml

Failed to load the native TensorFlow runtime.

See https://www.tensorflow.org/install/install_sources#common_installation_problems

for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.

Can one of the admins verify this patch?

Any update?

@tensorflow-jenkins test this please


Upgrade dependencies to core-js@3

Version
4.2.3
Environment info
  System:
    OS: Windows Server 2016 10.0.14393
    CPU: (4) x64 Intel(R) Xeon(R) CPU E5-2680 v4 @ 2.40GHz
  Binaries:
    Node: 13.9.0 - C:\Program Files\nodejs\node.EXE
    Yarn: Not Found
    npm: 6.13.7 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Not Found
  npmGlobalPackages:
    @vue/cli: Not Found

Steps to reproduce
C:> npm i -g @vue/cli
npm WARN deprecated<EMAIL_ADDRESS>request has been deprecated, see https://github.com/request/request/issues/3142
npm WARN deprecated<EMAIL_ADDRESS>core-js@<3 is no longer maintained and not recommended for usage due to the number of issues. Please, upgrade your dependencies to the actual version of core-js@3.
What is expected?
Installation of the vue/cli without error messages or deprecation warnings.
What is actually happening?
Deprecation warnings pop up about packages that are no longer safe to use.

I think these are just internal dependencies and have no negative effect on your application code. Vue cli will upgrade it's own dependencies as it's own version is updated.

The core-js one should have been fixed with the latest release of apollo.
The request one is tracked at https://github.com/vuejs/vue-cli/issues/5220


[FLOC-1968] Automate copyright changes in license

Fixes https://clusterhq.atlassian.net/browse/FLOC-1968
This duplicates the LICENSE file into a template and makes two changes to it:

A set of braces "{}" are replaced with "{{}}", but this does not change the rendered file
Instead of 2014, 2015 the LICENSE file now has 2014-{current_year}.

Thanks. Address comments and merge when green.


[BUG] Nvidia screwing up the menu

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[ ] My answer given in the checkbox above is a lie.

What problem did you encounter?
Whenever I open the Bloxstrap menu, the

nvidia overlay thing thinks it is Roblox that has opened, and starts to screw up the menu.
Sometimes, it even just starts to become a yellow and orange mess. Tried reinstalling, didnt do anything. And ive noticed more frequent fps drops in-game. Any help?
CPU: Intel 14th generation 14650HX Core i7
GPU: RTX 4060 Laptop GPU

it also starts to ramp up my gpu lol

Find settings relating to performance, power, etc. if Bloxstrap is in it then remove it or set it to a lower level.

Find settings relating to performance, power, etc. if Bloxstrap is in it then remove it or set it to a lower level.

Ive set the settings in NVidia control panel to Roblox game client for it to use more DGPU, but I doubt that has to do something with the menu itself?

its starting to show the fps in getting in the bloxstrap menu 💀

theree it is

I'm not sure what you want me to do about this. Since you're the only person in existence having this issue as far as I'm concerned, you configured something in the Nvidia control panel that ended up causing this. You'll have to find out what it is.

possibly its because i did this?? I did it just to improve the use of dgpu in Roblox. I dont really know how this would affect roblox if it clearly states robloxbetaplayer.exe

I mean i dont think this would bring any issues.


Expose StoreContext in public API

Coverage remained the same at 100.0% when pulling d0b3ed4af5f07ff404f9da0972236be1e796137b on tonycuadra:master into 53843d4cab8abae67a9126dd2491996b0c87aea4 on Roblox:master.

Please update the version number in the rotrievet.toml file as well!

Done! Thanks


Getting chunk from Region generates a lot of heap

self describing, there is issue in current implementation of getting chunk.
It may be either from BinaryReaderEndian itself, or it could be from these possible constructors: Region, Chunk, Section:
Folder to those files

Use internal array pooler to reuse instance of arrays for sections, chunk highest block buffers etc


[FLINK-11617] [kinesis connector] Kinesis Connector getRecords() failure logging is misleading

What is the purpose of the change
Fix misleading logging in Flink Kinesis Connector.
Brief change log
Changed a couple of log messages.
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

@flinkbot approve all


sensu-chef loading some vendored gems in a different cookbook

Related:

https://github.com/sensu/sensu/issues/1139
https://github.com/chef-cookbooks/docker/issues/591

If I restart it by hand, it works fine, but not through the Chef run.
Never ran into this issue so I'm not sure what kind of ruby-fu Sensu/Chef are doing here.

Issue related to docker-cookbook, closing.


Pytest test discovery confused by packages

Environment data

VS Code version: 1.30.2
Extension version (available under the Extensions sidebar): 2018.12.1
Relevant/affected Python packages and their versions: pytest 3.9.1

Expected behaviour
Tests are discovered correctly
Actual behaviour
Any package causes subsequently discovered tests to be considered IN the package
Steps to reproduce:

Create a tests folder
Create test/testa/test_first.py test/testc/test_last.py with regular python tests (any function will suffice)
Create test/testb/init.py and test/testb/test_second.py with a python test function
Run discover tests

Logs
Example pytest output:
============================= test session starts ==============================
platform linux -- Python 3.6.0, pytest-3.9.1, py-1.7.0, pluggy-0.8.0
Using --randomly-seed=1547049040
rootdir: /home/alex/git/HP-DLF, inifile:
plugins: repeat-0.7.0, randomly-1.2.3, profiling-1.3.0, cov-2.6.0, console-scripts-0.1.7, nbval-0.9.1
collected 3 items
<Module 'test/testa/test_first.py'>
  <Function 'test_foo'>
<Package '/home/.../test/testb'>
  <Module 'test_second.py'>
    <Function 'test_foo'>
<Module 'test/testc/test_last.py'>
  <Function 'test_foo'>

========================= no tests ran in 0.11 seconds =========================

The test output parser will report the last one in file /home/.../test/testb/test/testc/test_last.py  which is clearly wrong.
Reason is that the packagePrefix is not reset

@ericsnowcurrently 	Please ensure your python code works in this scenario, if it does, then we can close this.
Just need to ensure this gets tracked and tested, hence assigning to you & myself.

Feel free to give me a ping once #4605 is used so I can check against my test tree if it works too.

When will this be out. I currently makes the pytest integration unusable for me.

I compiled the latest master locally and for me it is working. If it isn't for you you may watch https://github.com/Microsoft/vscode-python/pull/4795 which seems to contain the actual implementation of the Python based discovery.

Thanks @Flamefire. I just don't want to spend time compiling. I hope they supply an update soon.

Seems to have not made it into release 1.33.1 :/

Can we get an update on when this will be released?

This should be resolved in the most recent versions of the Python extension.  If that is not the case then please create a new issue.  (This one has a bit of outdated info, so better to start fresh.)


Prevent deprecation warnings in React 15.5

Problem: React 15.5 deprecated React.createClass and will now throw errors.
Solution: The simplest solution is to use create-react-class and check the React version.

@necolas this can't be merged without a signed CLA

@kborchers it was signed... the app seems confused because I first committed with a different email address than the one I signed (and ultimately committed) with. Notice the github check went away.

Sorry about that. I just happened to look on mobile and only saw the comment image and the details link to see the check isn't there unless I switch to desktop view. Y'all can ignore me :)

Hi @paularmstrong thanks for your PR.
About CLA... I see you have used Paul Armstrong<EMAIL_ADDRESS>as author in your commit message and the bot says it's not a github users. First of all, It's unfortunate that the bot has such a restriction. Having said that, did you find any issues? Is there anything you need help with? cc @kborchers


Occasional critical errors

Hi,
we experience occasional errors from the library from time to time when fetching large datasets in iterations:

or:

Did you get anything like this? The dataset we're using has about 500k records, about 10 columns of regular types - some integers, some characters and a timestamp, nothing special.
Any thoughts? This is probably not something we would like to put in production in this stage, so any help would be appreciated.

I didn't get anything like this before but I will try same the condition many time to produce the error and see the cause

Nah, it's random. We didn't find any patterns in it, so I would say it's not related to data itself.
Here's another one that keeps popping up:

what value of prefetch_rows you use?

We tried it all, from 1000 to 10000

this is what I expect
I will use same prameter and see where is the problem and solve

ok, so it happens with high value of prefetch_memory. Anything higher than 100 causes serious instability to the driver(!). Value 50 proved to be optimal in our case:
prefetch_rows=5000&prefetch_memory=50
With this set, the performance is so far stable. Will keep you posted after a few days.

do you have any column with CLOB or BLOB?

See my original comment:
"some integers, some characters and a timestamp, nothing special."

I use code like this
func getResults(conn *sql.DB) error {
	rows, err := conn.Query(
		`SELECT result_id, visit_id, group_id, item_id, operation_id, result_data1,
result_data2, result_data3, result_data4, result_text, row_index, page_id, print_number,
last_modification, method_id, reffered_to_id, add_date, status_id, panel_id,
test_id, bill_id, pk_no, edit_by, signed_by, report_info_id
from results`)
	if err != nil {
		return err
	}
	var (
		result_id sql.NullInt64
		visit_id sql.NullInt64
		group_id sql.NullInt64
		item_id sql.NullInt64
		operation_id sql.NullInt64
		result_data1 sql.NullFloat64
		result_data2 sql.NullFloat64
		result_data3 sql.NullFloat64
		result_data4 sql.NullFloat64
		result_text sql.NullString
		row_index sql.NullInt64
		page_id sql.NullInt64
		print_number sql.NullInt64
		last_modification sql.NullTime
		method_id sql.NullInt64
		refferal_id sql.NullInt64
		add_date sql.NullTime
		status_id sql.NullInt64
		panel_id sql.NullInt64
		test_id sql.NullInt64
		bill_id sql.NullInt64
		pk_no sql.NullInt64
		edit_by sql.NullInt64
		signed_by sql.NullInt64
		//printed_result []byte
		report_info_id sql.NullInt64
	)
	index := 0
	for rows.Next() {
		err = rows.Scan(&result_id, &visit_id, &group_id, &item_id, &operation_id, &result_data1, &result_data2,
			&result_data3, &result_data4, &result_text, &row_index, &page_id, &print_number, &last_modification,
			&method_id, &refferal_id, &add_date, &status_id, &panel_id, &test_id, &bill_id, &pk_no, &edit_by,
			&signed_by, &report_info_id)
		if err != nil {
			return err
		}
		index ++
	}
	if rows.Err() == nil {
		fmt.Println("Number of results: ", index)
	}
	return rows.Err()
}

setting prefetch_rows from 1000 - 60000 no errors occur
result is like this
Number of results:  2995388
when I use 70000 and more I get this error from the server
ORA-03146: invalid buffer length for TTC field
my server is 12.2 c

what is database charset

I sent you an email.

actually it support Prefetch row only no prefetch memory
according to official C# driver prefetch rows is calculated automatically from the following equation:
noOfRowsToFetch = (int) (m_fetchSize / (long) this.m_maxRowSize) + 1;
where m_fetchSize is fixed value = 0x20000
and maxRowSize is calculated from the following equation:
m_maxRowSize = m_maxRowSize + m_numOfLOBColumns * Math.Max(86, 86 + (int) lobSize) + m_numOfLONGColumns * Math.Max(2, longSize) + m_numOfBFileColumns * 86;

According to my testing:
setting of PrefetchRows to 1000 or less is better for stability
you can use higher values when working on local network and with small and medium size query
if you have LOB, CLOB, LONG or BFILE in you query PreFetchRows better to be 25

Agreed.  Tested on several occasions, PrefetchRows of 1000 is stable, however, the performance is obviously degraded. So yeah, for local databases it should be high, for remote ones it should be 1000 or less.

The TTC error happened to me at once.
Any solution to fix it ?

can you write your SQL string and go code

SQL string is like this :
  SELECT /*+rule+*/ *
  FROM SV_WuJi_Instances
  WHERE StudyUid IN (
          SELECT max(StudyUid)
          FROM SV_WuJi_Instances
                  WHERE ACCESSIONNUMBER IN (Translate(:1 USING CHAR_CS))
          GROUP BY ACCESSIONNUMBER)

My go code is like this:
	stmt, err := db.Conn.Prepare(sqlString)
	if err != nil {
		log.Error("Prepare Sql Error -> ", err)
		return rdata, err
	}
	defer stmt.Close()

	rows, err := stmt.Query(args...)
	if err != nil {
		log.Error("Query Sql Error -> ", err)
		return rdata, err

	}
	defer rows.Close()

	for rows.Next() {
           ......
        }

the error that occur with High PrefetchRows value in 2 condition:
1- remote database or slow connection
2- query that contain lob like issue #123
you write above that the error occur once only is this correct?

Actually, I mean it occurs when I connected to ONE database for ANY SQL query.
It's my productive environment database provided by third party.
So I can't get more information about the table structure or other details.

@LawyZheng
Please answer these questions, this will help narrow down the possible cause:

is your production database remote, meaning not in the same machine / network as the client application querying it (I would assume - yes)? What is the ping time to the server?

do you use prefetch_rows parameter in connection string?
If yes: what is the value of it,
if not: try to set it to 50, 100, 1000 and see whether it fixes your problem.

when you execute a query, is one of the columns BLOB or CLOB ?
I would assume you know what you are querying or you can get this info from an administrator.

@gorandumic
Thanks for the reply.
Unfortunately, I'm only able to answer the second question right now.
I didn't use  prefetch_rows in my connection string, and I will try to use it in my next step.
BTW, is there any introduction about how to use it?
Concerning other questions, I can't get the answers either, since I don't have access right to my productive database.  But I will notify you as soon as I get this info from the administrator.

@LawyZheng and please include some screenshots to begin with to help us understand what exactly are you doing when you get TTC errors, what client are you using, is it:
your custom Go code executed locally
or
you're using an application (for which you don't know what queries it sends to the database but gets TTC errors)


Fix FhirIO javadoc format broken

Current: https://beam.apache.org/releases/javadoc/current/org/apache/beam/sdk/io/gcp/healthcare/FhirIO.html
See also links not working and has wrong html; the paragraph after see also does not render links; code snippets missing a right brace}
Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Fixed preview


removed the use of the scss lib as its broken (no styling for now)

Motivation and Context
it looks the the pyscss lib is broken (https://travis-ci.org/github/Kronuz/pyScss) and we can't build a new version of the ops tool because of this, for now i've stripped it out so we can carry on until we find a proper fix, all this means is there wont be much styling
What has changed

removed pyscss from Pipfile
regenerated Pipfile.lock
removed the use of the scss lib from app/init.py

How to test?

build locally by running make build then make docker
run docker dev enviroment and test you can still load a sample and set up an action plan

 Investigate ops tool package issues

Might be worth updating qid-batcher runner to it's latest version as well https://github.com/ONSdigital/census-rm-qid-batch-runner/releases

yes good point will do that now 👍

Travis failing due to a unit test failure

just committed a fix for that and have bumped the qid-batch-runner version

Ryan was saying that we've removed a method which was needed, but if this is good enough to get the testers able to load samples then happy to approve.

what method was this?


useFind with sort return error 'no_usable_index'

Using useFind hook for getting sorted data.
Description
Im trying to get some data from my pouch (local, synced with remote couch), sorted by index number.
My data is like the following:
{
  "_id": "extendedField#Financiadora#96dd868704c9efb218b0c98fa701d106:612C04170D87B5C76B4C480A107B621C",
  "_rev": "2-f66bc8ab831e80f842f3b2fc746a8c38",
  "entidad": "Financiadora",
  "label": "Nombre del contacto",
  "permiso": "edit_basic",
  "step": 1,
  "index": 2,
  "org": "96dd868704c9efb218b0c98fa701d106",
  "app": "alfred",
  "model": "extendedFields"
}

My code to get the data:
const { docs: entityFields, error: efError } = useFind({
        // Create and query an index for all extended fields
        index: {
            fields: ['model', 'app', 'org', 'entidad', 'index'],
        },
        selector: {
            model: 'extendedFields',
            app: 'alfred',
            org: `${editData?.org}`,
            entidad: `${editData?.nombreEntidad}`,
            index: { $gt: 0 },
        },
       sort: [{ index: 'asc' }],
        db: 'alfred',
    })

I got the error:
{error: 'no_usable_index', message: 'There is no index available for this selector.'}
I see the index was there in the pouchdb-debug log
Steps to Reproduce
Just use the useFind hook in a react component, of course having some sample data like the provided one.
Expected Behavior
I expected to get the data sorted by index number.
Your Environment
This are the related installed packages:
"pouchdb": "^7.2.2",
"pouchdb-debug": "^7.2.1",
"pouchdb-find": "^7.2.2",
"use-pouchdb": "^1.3.0",
 "react": "^17.0.1",

Browser: Firefox 91.0.2 (64-bit)
Operating System: macos BigSur [ 11.2.1 (20D75) ]
Additional Context
Using just puchdb-find with create index just works as expected.
localDb
        ?.createIndex({
            index: { fields: ['model', 'app', 'org', 'entidad', 'index'] },
        })
        .then(() => {
            return localDb.find({
                selector: {
                    model: 'extendedFields',
                    app: 'alfred',
                    org: `${editData?.org}`,
                    entidad: `${editData?.nombreEntidad}`,
                    index: { $gt: -1 },
                },
                sort: [{ index: 'asc' }],
            })
        })
        .then((res) => {
            // console.log('FIELDS :', res.docs)
            setEntityFields(res.docs)
        })
        .catch((e) => {
            console.log('<<<<ERROR CREATE QUERY INDEX>>>>>:', e)
        })

I have been looking examples and i found on the source code of use-pouchdb that the sort parameter is a list of strings not a list of objects like in pouch-find. All examples were with only one field, in the index fields, in the selector and in the sort.
As i was doing a query with many fields in the selector (an the index fields), I discovered by trial and error, that the sort parameter must be the same that the index fields, exactly the same, and also in the selector.
The final code that is working is:
const { docs: entityFields, error: efError } = useFind({
        // Create and query an index for all extended fields
        index: {
            fields: ['app', 'org', 'model', 'entidad', 'index'],
        },
        selector: {
            app: 'alfred',
            org: `${editData?.org}`,
            model: 'extendedFields',
            entidad: `${editData?.nombreEntidad}`,
            index: { $gte: 0 },
        },
        sort: ['app', 'org', 'model', 'entidad', 'index'], // * NOTE 2
        db: 'alfred',
    })

It would be nice to document this.


serving/xrefs: return error when both file and diagnostics are missing [phab:D1634]

Created by schroederc at 2017-05-10 18:54:30:
Allow the serving data to fully elide the FileDecorations.file field if
the file is truly missing, but FileDecorations were written to include
related diagnostics.  Treat this case specially in the server.

fromberger accepted this revision at 2017-05-10 19:04:47

schroederc closed via commit 6c9c1958e563b26b2378e6a774d7c50994388ee3 at 2017-05-10 19:31:57


.Include(blog => blog.Owner.AuthoredPosts) vs .Include(blog => blog.Owner).ThenInclude(x => x.AuthoredPosts)

"You can also load multiple navigations using a single Include method. This is possible for navigation "chains" that are all references, or when they end with a single collection."
While the "Tip" suggests that it's possible to use .Include(entity => entity.A.B), I couldn't find any explanation what is the difference compared to when .Include(entity => entity.A).ThenInclude(x =>x.B) is used.

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

ID: 25a3ce6e-43ab-e0af-01b3-a82fdaf3cb31
Version Independent ID: 320e7305-e0a3-a1c6-77f2-1d3029b660b1
Content: Eager Loading of Related Data - EF Core
Content Source: entity-framework/core/querying/related-data/eager.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @roji
Microsoft Alias: avickers

There is no difference in behavior.

Even when ".A" was defined as "virtual" ?

@TomasB It makes no difference whether the navigation is virtual or not. All any of the syntax choices do is provide a "path" of navigations. You could even use a string: "A.B".
The reason that ThenInclude is preferred is that it is a consistent and type-safe API that works for both reference and collection navigations. The "dotting" shortcut only works for reference navigations.

This kind of information what I was looking in docs:
"The reason that ThenInclude is preferred is that it is a consistent and type-safe API that works for both reference and collection navigations. The "dotting" shortcut only works for reference navigations."
With links into more information related to reference vs collection navigations.

@TomasB There is always a balance between stating everything and keeping the docs concise enough to be easily consumable. We don't think making any change here will be overall advantageous.


Tracker

Hello! I ran julia> ] add https://github.com/FluxML/Gym.jl from terminal and then tried using Gym and got the following error
ERROR: LoadError: UndefVarError: Tracker not defined
Stacktrace:
[1] include at ./boot.jl:328 [inlined]
[2] include_relative(::Module, ::String) at ./loading.jl:1105
[3] include(::Module, ::String) at ./Base.jl:31
[4] top-level scope at none:2
[5] eval at ./boot.jl:330 [inlined]
[6] eval(::Expr) at ./client.jl:425
[7] top-level scope at ./none:3
in expression starting at /Users/christywarden/.julia/packages/Gym/ODARK/src/Gym.jl:4
ERROR: Failed to precompile Gym [56b9baea-2481-11e9-37ae-75904354ad8c] to /Users/christywarden/.julia/compiled/v1.3/Gym/Z6zPi_B9rLB.ji.
Stacktrace:
[1] error(::String) at ./error.jl:33
[2] compilecache(::Base.PkgId, ::String) at ./loading.jl:1283
[3] _require(::Base.PkgId) at ./loading.jl:1024
[4] require(::Base.PkgId) at ./loading.jl:922
[5] require(::Module, ::Symbol) at ./loading.jl:917
Is the use of Tracker out of date? Any help would be greatly appreciated

I had faced the same issue. Go to ~/.julia/packages/Gym using terminal and then navigate to src/Gym.jl and comment out the line using Flux.Tracker.

I had faced the same issue. Go to ~/.julia/packages/Gym using terminal and then navigate to src/Gym.jl and comment out the line using Flux.Tracker.

Thanks so much! This fixed the issue :)

I also see this.


"route: ['foo', 'bar', '{integers}']" doesn't match but "route: 'foo.bar[{integers}]'" does?!?

@jhusain, @trxcllnt, @michaelbpaulson
Ran in to some confusing behavior in defining routes today.
Apparently the special path segments '{integers}' '{ranges}' and '{keys}' are only available in routes given as Strings.
Is this intended, or did I miss something with how to define routes as arrays?
This does not work: ['foo', 'bar', '{integers}'] but this does work 'foo.bar[{integers}]'

Thanks,
;-j

You're correct. We may provide a way to specify these with the Array syntax in the future. Before we do that though, we have to reserve some of the key space explicitly so that we do not conflict with keys included in the domain model by users. We're considering how to do this now.

Yeah. Figured that out. Once I switched to using strings everything worked.
The main reason I wanted to use arrays was to have access to the path as an array in the execution of the various get/set/call methods of the route object and slice/dice/julian it for returning path-values/jsonGraphs.
However, I came to realize (with judicious uses of JSON.stringify) that falcor-router is mutating my router's route property. The router replaces my string value with the equivalent array of ['foo', 'bar', { type: 'integers', named: false }, { type: 'keys', named: false }] so I can still slice off the base part and use it my handlers.
Very un-intuitive (and shame on you for mutating my object), but the result is the same.

;-j