Http subscriptions stopped working in IE 11 (Angular 10)

Bug Report
Current Behavior
IE 11 throws "TypeError: Invalid calling object" exception when I subscribe to very simple, without params, http get requests. Chrome works fine. I'm working with Angular v10.1.1, rxjs v6.6.3.
Reproduction

REPL or Repo link:
(you can use https://stackblitz.com/ to create one to attach here)

    testService() {
        const apiHeader = new HttpHeaders()
            .set('Content-Type', 'application/json')
            .set('cache-control', 'no-cache');
        const url = decodeURIComponent(`https://.../api/test/test`);
        return this.http.get<any>(url, { headers: apiHeader });
    }

    this.myService.testService()
      .subscribe(data => {
        console.log('ok', data)
      },
        err => {
          console.log('err', err)
        });
  }

**Expected behavior**
A clear and concise description of what you expected to happen (or code).

**Environment**
- Runtime: IE 11
- RxJS version: 6.6.3
- (If bug is related) Loader, build configuration: [e.g webpack, angular-cli version, config]

**Possible Solution**
<!--- Only if you have suggestions on a fix for the bug -->

**Additional context/Screenshots**
Add any other context about the problem here. If applicable, add screenshots to help explain.

It seems that you've asked this question on Stack Overflow - which is a better forum for problems like this, IMO - and in that question you mention:

After upgrading from Angular 10.0.1 to 10.1.1

You've not mentioned that here. For this to be reported as an RxJS problem, a more minimal reproduction would be required. Additionally, Angular does not use RxJS's HTTP implementation; it uses its own. For those reasons, I'm closing this.


Fix prediction-pmm urls and a tiny comment.

great stuff @adrinjalali !
btw, the format for the urls will be changing here soon to include a version for the api's (ie. /v1/)


Duplicate Items vanish when removed from a container

Foundry version: 0.8.6
SFRPG version: 0.11.3
When creating a container with duplicate items, if the items were dragged from the compendium, the container will show multiple copies, but they will all be removed when any one of them is dragged out of the container.
Once the item has been placed in a character's inventory, it stops exhibiting this behavior.
replication steps:

create a new container on the map
drag 2+ of the same item from the compendium
open the container and remove one of the duplicate items (delete, move to player, move to map)

https://user-images.githubusercontent.com/24704061/121520401-dc307880-c9c0-11eb-8015-0d36ef981c21.mp4

Fixed, the underlying issue was that everything had the same id which made some systems a little too happy to delete duplicates. :-)


Add missing semicolon in mode/css/css.js

The src/ dir is entirely semicolon-free. For the code outside of that, there's no enforced style, and omitted semicolons are okay, and don't need to be fixed (there's a lot more of them).

Yeah, sorry. I realized that after the fact. 🙏


VLAN_MANU_MODEs  2 and 3 Function

How ONU Stick handle VLAN from PON to ETH interface

Value
Mode
Note
Explaination

0
Transparent Mode
VLAN will mapped as is
Map VLAN without changing original id

1
Tagging Mode
VLAN Stacking / Q-in-Q
Untag first VLAN layer and bridge as is

2
Remote Access Mode

3
Special Case Mode

What do VLAN_MANU_MODEs  2 and 3 do?
When set to 2 or 3 is there a possibility to not be able to access SFP by telnet or WebUI?

these 2 and 3 I no info about it, no one does, it seem might have custom driver? I not really sure

these 2 and 3 I no info about it, no one does, it seem might have custom driver? I not really sure

Does setting to 2 or 3 affect the ability to access SFP by Telnet?

nope, can set as it, it wont bother


spec: one imperfection

The line in spec is not accurate:
var p = 1<<s == 1<<33         // illegal if ints are 32bits in size: 1 has type int, but 1<<33 overflows int

The line is always legal.

Sorry, it looks the comment says the 2nd "1".

There is still an imperfection: a[1.0<<s] will always panic. So it would be better to change the comment from
// 1.0 has type int; x == a[0] if ints are 32bits in size

to
// 1.0 has type int; This line compiles okay but panics at run time.

I'm not sure any change is required here, but CC @griesemer .

It is that a[0] is a non-existing element if ints are 32bits in size.

Understood, I just don't think it is unclear.

I agree with @ianlancetaylor - this seems clear enough.
That said, the change I'd make is declare a separate independent array, rather than using a that's declared on the next line. Then there's no confusion.
I will make the change as it's trivial.


Error: Code=ResourceGroupNotFound in EnterprisePolicyOperations.ps1

https://github.com/microsoft/PowerApps-Samples/blob/ed3b8662ee839650b54735312a1e902d2ab077dc/powershell/enterprisePolicies/Common/EnterprisePolicyOperations.ps1#L32
When running the CreateSubnetInjectionEnterprisePolicy.ps1 script, I got an error on line 32 stating the resource group wasn't found.  After confirming I had input the correct name, I reviewed the code and found that the context appears to be incorrect.  I believe the issue is that my default subscription is different than the one I'm running this script against, and the call to the New-AzResourceGroupDeployment commandlet on line 32 of this included script is running against the wrong subscription.  To confirm this, I added Set-AzContext -Subscription $subscriptionId just before line 32, added $subscriptionId as an additional parameter to this function, and changed line 86 of CreateSubnetInjectionEnterprisePolicy.ps1 to $result = PutEnterprisePolicy $subscriptionId $resourceGroup $body.  This resulted in the policy getting created and the script completing without errors.
Here's an example of what the execution, with errors, looks like:
`PS C:\PowerApps-Samples\powershell\enterprisePolicies\SubnetInjection> .\CreateSubnetInjectionEnterprisePolicy.ps1
cmdlet CreateSubnetInjectionEnterprisePolicy at command pipeline position 1
Supply values for the following parameters:
(Type !? for Help.)
subscriptionId: abcd1234-ab12-ab12-ab12-abcdef123456
resourceGroup: Networking-RG
enterprisePolicyName: PowerPlatformSubnetInjection
enterprisePolicylocation: unitedstates
primaryVnetId: /subscriptions/abcd1234-ab12-ab12-ab12-abcdef123456/resourceGroups/Networking-RG/providers/Microsoft.Network/virtualNetworks/vNet-Primary
primarySubnetName: PowerPlatformIntegration
secondaryVnetId: /subscriptions/abcd1234-ab12-ab12-ab12-abcdef123456/resourceGroups/Networking-RG/providers/Microsoft.Network/virtualNetworks/vNet-Secondary
secondarySubnetName: PowerPlatformIntegration
Logging In...
Logged In...
Creating Enterprise policy...
New-AzResourceGroupDeployment: C:\PowerApps-Samples\powershell\enterprisePolicies\Common\EnterprisePolicyOperations.ps1:32
Line |
32 |  …   $policy = New-AzResourceGroupDeployment -DeploymentName "EPDeployme …
|                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| 11:31:22 AM - Error: Code=ResourceGroupNotFound; Message=Resource group 'Networking-RG' could not be found.
New-AzResourceGroupDeployment: C:\PowerApps-Samples\powershell\enterprisePolicies\Common\EnterprisePolicyOperations.ps1:32
Line |
32 |  …   $policy = New-AzResourceGroupDeployment -DeploymentName "EPDeployme …
|                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| The deployment validation failed
InvalidOperation: C:\PowerApps-Samples\powershell\enterprisePolicies\Common\EnterprisePolicyOperations.ps1:35
Line |
35 |      if ($policy.ProvisioningState.Equals("Succeeded"))
|          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| You cannot call a method on a null-valued expression.
Error creating/updating Enterprise policy
Subnet Injection Enterprise policy not created`

Hi there - I got the same issue but am having trouble implementing your fix. Can you provide the code for your updated EnterprisePolicyOperations.ps1?
thanks

@RobGANA I have the same problem, I've updated the script but I'm still running into the same problem as mentioned in this issue (https://github.com/microsoft/PowerApps-Samples/issues/624). Do you have the updated script so I can try that?

@robjdav and @RyC67,
In EnterprisePolicyOperations.ps1, I altered line 27
function PutEnterprisePolicy($subscriptionId, $resourceGroup, $body)
and I added a new line before line 32
    Set-AzContext -Subscription $subscriptionId
In CreateSubnetInjectionPolicy.ps1, I altered line 86
    $result = PutEnterprisePolicy $subscriptionId $resourceGroup $body
That's all there was to it.  The subscription ID was already a parameter of the CreateSubnetInjectionPolicy script.  It just needed to be passed to and used in the EnterprisePolicyOperations script.

thanks!


🐛 Partial data array outputs in wrong columns

Issue
If the data array only contains the partial amount of data (when compared to the properties in the schema), then the data is output to the incorrect column in the xlsx.
Solution
Ensure that the data is aligned correctly where it is set. I think this is happening in the write_table function.
Screenshots
Data with top three lines commented out:

Outputs to xlsx to incorrect columns:

fixed by #8


openmotif formula

I'm getting this error when installing openmotif:
==> ./autogen.sh --prefix=/usr/local/Cellar/openmotif/2.3.4 --disable-dependency-tracking --disable-silent-rules --disable-jpeg --disable-png
==> make install
Error: No such file or directory - (/usr/local/Cellar/openmotif/2.3.4/share/man/man3/CORE.3, /usr/local/Cellar/openmotif/2.3.4/share/man/man3/openmotif-CORE.3)

Seems like the file:
https://github.com/Homebrew/homebrew-x11/blob/master/openmotif.rb#L62
is missing (/usr/local/Cellar/openmotif dir is missing).
I'm on macOS 10.12. Was able to install on 10.11.
Thanks.

This may have been fixed by #250. Could you brew update and try installing again?

@apjanke yes it seems fixed now thanks.
oliv@oliv:~$ ll /usr/local/Cellar/openmotif/2.3.6/lib/libXm*
-r--r--r--  1 oliv  staff   2.2M Oct  5 07:47 /usr/local/Cellar/openmotif/2.3.6/lib/libXm.4.dylib
-r--r--r--  1 oliv  staff   3.3M Jun 10 19:06 /usr/local/Cellar/openmotif/2.3.6/lib/libXm.a
lrwxr-xr-x  1 oliv  staff    13B Jun 10 19:06 /usr/local/Cellar/openmotif/2.3.6/lib/libXm.dylib@ -> libXm.4.dylib
However, when i try to install nedit (which is the one I was installing in the first place, openmotif coming as a dependency), it seems to not find the library when trying to compile it.
Here's the full log:
==> Installing nedit from homebrew/x11
==> Downloading https://downloads.sourceforge.net/project/nedit/nedit-source/nedit-5.6a-src.tar.gz
Already downloaded: /Users/oliv/Library/Caches/Homebrew/nedit-5.6a.tar.gz
==> Patching
patching file util/motif.c
==> make macosx MOTIFLINK='-lXm'
Last 15 lines from /Users/oliv/Library/Logs/Homebrew/nedit/01.make:
OpenMotif 2.3
OpenMotif is available from:
http://www.opengroup.org/openmotif/
http://www.ist.co.uk/DOWNLOADS/motif_download.html
Also, unless you need a customized NEdit you should STRONGLY
consider downloading a pre-built binary from http://www.nedit.org,
since these are the most stable versions.
If you really want to build an untested version of NEdit you
can override this sanity check by adding -DBUILD_UNTESTED_NEDIT
to the CFLAGS variable in your platform's Makefile (e.g.
makefiles/Makefile.linux)
make[1]: *** [check_tif_rule] Error 1
make: *** [macosx] Error 2
READ THIS: https://git.io/brew-troubleshooting
If reporting this issue please do so at (not Homebrew/brew):
https://github.com/Homebrew/homebrew-x11/issues

Could you post the full build logs as a gist by running brew gist-logs nedit?

@apjanke https://gist.github.com/2e825eff5ca18dca861f88bf36f86657

I can reproduce. Looks like nedit is not happy building against OpenMotif 2.3.6, which we recently upgraded to.
From the build logs:
./check_lin_tif
ERROR:  Untested Motif Version:
	@(#)Motif Version 2.3.6
You are attempting to build NEdit with a version of Motif that
has not been verified to work well with NEdit.  This could be fine,
but it could also lead to crashes and instability.  Historically,
older versions of Motif have quite often been more stable
than newer versions when used with NEdit, so don't assume newer
is better.

NEdit is known to work with Motif versions:
0.92.32
0.93.0
0.93.12
0.93.18
OpenMotif 2.1.30
OpenMotif 2.2.3
OpenMotif 2.3

...
If you really want to build an untested version of NEdit you
can override this sanity check by adding -DBUILD_UNTESTED_NEDIT
to the CFLAGS variable in your platform's Makefile (e.g.
makefiles/Makefile.linux)

We might need to roll back to OpenMotif 2.3.4, or update our patch to nedit to let it work with 2.3.6.
I'll take a look at nedit's documentation to see which is the right choice. I've posted on the upstream bug report asking about 2.3.6.

I've merged #256, which hacks nedit to allow openmotif 2.3.6. This nedit issue should be resolved now.


fix: replace usages of transform proto with update_transform

Porting from node. Also added additional system test port as well.
Conformance tests are passing locally against the custom [branch] of modified conformance tests (https://github.com/googleapis/java-conformance-tests/tree/bump/2020-05-12_141750) in googleapis/java-conformance-test. According to @BenWhitehead, once this PR is LGTM'd, we will:

Merge the conformance test PR into conformance-tests.
Create new release of java-conformance-tests.
Update java-firestore to the new release.
Rebuild this PR and ensure everything passes before merging.

The release of the conformance tests is available. @thebrianchen Can you bump the dependency like is done in this PR: https://github.com/googleapis/java-firestore/pull/218 that should allow your builds to be green.

Pinged someone to update the branch protection settings to unblock this.


Support for "days of week" for KEB maintenance window schedule

Description
Changes proposed in this pull request:

Logic to calculate the next available day for the maintenance window.
Unit tests

/test pre-control-plane-components-kyma-environment-broker


Add client side mutations example

Hi, first of all awesome package. I really like using GraphQL in my projects and Diana.jl is really useful.
When I started reading the documentation, I thought mutations on the client side couldn't be made but, surprisingly, I could!
So, I propose adding explicitly this functionality for better understanding.
Here is a small example using SpaceX API
using Random
using Diana
using Test

GET_EMPTY_USER = """
query(\$name: String!){
  users(where: {name: {_eq: \$name}}) {
    name
    rocket
  }
}
"""

my_random_name = randstring(12)

r = Queryclient("https://api.spacex.land/graphql/", GET_EMPTY_USER, vars=Dict("name" => my_random_name))

@test r.Info.status == 200
@test r.Data == "{\"data\":{\"users\":[]}}\n"

CREATE_USER = """
mutation(\$name: String!, \$rocket: String! ) {
  insert_users(objects: {name: \$name, rocket: \$rocket}) {
    returning {
      name
      rocket
    }
  }
}
"""

my_random_rocket = randstring(12)

r1 = Queryclient("https://api.spacex.land/graphql/", CREATE_USER, vars=Dict("name" => my_random_name, "rocket" => my_random_rocket))

@test r1.Info.status == 200
@test r1.Data == "{\"data\":{\"insert_users\":{\"returning\":[{\"name\":\"$my_random_name\",\"rocket\":\"$my_random_rocket\"}]}}}\n"

new_random_rocket = randstring(12)

UPDATE_USER = """
mutation(\$name: String!, \$rocket: String!) {
  update_users(where: {name: {_eq: \$name}}, _set: {rocket: \$rocket}) {
    returning {
      name
      rocket
    }
  }
}
"""

r2 = Queryclient("https://api.spacex.land/graphql/", UPDATE_USER, vars=Dict("name" => my_random_name, "rocket" => new_random_rocket))

@test r2.Info.status == 200
@test r2.Data == "{\"data\":{\"update_users\":{\"returning\":[{\"name\":\"$my_random_name\",\"rocket\":\"$new_random_rocket\"}]}}}\n"

hi @mattborghi, sorry to say I forgot to add the client side mutations in the docs; your example is very good and I will add it to the documentation. Thank you very much.

hi @mattborghi, sorry to say I forgot to add the client side mutations in the docs; your example is very good and I will add it to the documentation. Thank you very much.


Adding Artifact Repository

Add support for artifact_repository plugins which allow for hosting of package payloads in remote servers. This works in conjunction with a MongoDB instance that contains the package information for querying and lookup.
How To Use
In order to use you will need an existing S3 bucket and a MongoDB instance with a named database and a collection called "packages".
In your rezconfig.py you will need to set the following:
packages_path = [
    "mongo@localhost:27017/db_name",
]

artifacts_path = [
    "s3@s3://bucket-1/packages",
]

default_cachable = True
package_cache_same_device = True
package_cache_async  = False

If your MongoDB instance also requires a username / password, you can set it via:
REZ_MONGO_USERNAME=username
REZ_MONGO_PASSWORD=password

When these are set, it will format the packages path url as:
{REZ_MONGO_USERNAME}:{REZ_MONGO_PASSWORD}@localhost:27017/db_name

rez-build
The rez-build command has been extended to allow for installing to artifact repositories. This is done through the new -a --artifact-path argument from rez-build.
Example:
foo> rez-build --install -p mongo@uri:port/db_name -a s3@s3://bucket-1/packages

This build command will do the following:

Build the foo package normally
Install build results to a temporary location of $TEMP\rez\package_name\package_version, this is what will be pushed to the artifact repository.
Add a new variant entry to the specified mongo@ package repository
Push the installed build results to the s3@ artifact repository.

This is pretty exciting @ttrently! I haven't reviewed any of it yet, but I took some time to link to some resources.
Long ago, someone tried to create a MongoDB repository plugin and even created a PR, see https://github.com/AcademySoftwareFoundation/rez/pull/205. You could potentially re-use part of it or at maybe just take inspirations from it.
Additionally, there is REP-003 that tracks package caching. In the REP (Rez Enhancement Proposal) and its comments, there is some wording that makes me believe the package repository could say "I need all my packages to be locally cached".

Yes, to a degree, and primary comment updated with usage information. I'm still a bit unsure of how to handle complex paths, so for this I kept it close to how normal package repositories are handled, ie. Artifact repository paths are defined and lookups happen based on the order of those paths.

The new description is helpful and thanks for that. Can you update the description to mention that you implement part of that discussion, what's implemented, what's not, etc. I would also suggest that you mention which library it depends, etc.
(Note that I won't have time to review until next week).


Removed middlewares

#70 Removed JsonBodyMiddleware and RawBodyMiddleware from app module imports

Thanks for the proposal
I'll take some time to discuss the topic with the maintainers of webhooks package
The thread is opened at https://github.com/golevelup/nestjs/discussions/385
I'm not certain about the impacts of this change as the application might break if we just remove the middlewares.
@TodorBelchev  Have you tested the stripe webhooks workflow after the changes have been made?


Homepage header updates for b56

The following updates are to be made to the header section of the homepage of beta.clinepidb.org. These are to be done in time for b56.

Shorten the dark blue portion in the header
Remove the “Welcome to ClinEpiDB” text in the header
Remove the release # and date text under ClinEpiDB logo
Replace the tagline with a new one: “Share, explore, and visualize clinical and epidemiological data”
Change placement of tagline and drop down menus according to mock-ups on slide 7 of this doc:
https://docs.google.com/presentation/d/10DBhyRYbd3q6LEYZtiungGSVJ1nncir6llsTzLPyeH0/edit#slide=id.p

This is done, this is all we can do without an overhaul of the header.
the overhaul (dom and cs) with deploy flat elements in a grid, like  we have in genomic sites, which allows for more flexible and maintainable layouts.

Feedback from Steve: please change "ClinepiDB 21" to "ClinepiDB - Release 21" or similar

Updates look good.


chore: bdd-spring-1580994412 to 0.0.1

chore: Promote bdd-spring-1580994412 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
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

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


[QUESTION]

First check

[x] I used the GitHub search to find a similar issue and didn't find it.
[x] I searched the FastAPI documentation, with the integrated search.
[x] I already searched in Google "How to X in FastAPI" and didn't find any information.

Description
How can I chache API responses?
There is no information about that in the docs, other frameworks such as flask cache API responses by adding decorators so, there is a similar way to do that in FastApi? If no, maybe we should work on it.

https://github.com/tiangolo/fastapi/issues/651
On Tue, Feb 25, 2020 at 9:03 AM alvaro garcia fernandez <
<EMAIL_ADDRESS>wrote:

First check

I used the GitHub search to find a similar issue and didn't find it.
I searched the FastAPI documentation, with the integrated search.
I already searched in Google "How to X in FastAPI" and didn't find
any information.

Description
How can I chache API responses?
There is no information about that in the docs, other frameworks such as
flask cache API responses by adding decorators so, there is a similar way
to do that in FastApi? If no, maybe we should work on it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tiangolo/fastapi/issues/1033?email_source=notifications&email_token=AAINSPWCCUWDAUACL2LIPY3RETGGTA5CNFSM4K3ASKG2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IP7LHHA,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAINSPRIMN4MOWO3WESIWLDRETGGTANCNFSM4K3ASKGQ
.

--
benoit barthelet
http://pgp.mit.edu/pks/lookup?op=get&search=0xF150E01A72F6D2EE

Thanks! Have you got an example of it?

yes https://github.com/argaen/aiocache/issues/463#issuecomment-534656962

@tiangolo could you consider to add this post to the FastApi docs?

I mean you can definitely make a PR to the docs

I will prepare it! thanks

Thanks everyone for the discussion here! I think you have a working solution now, right?
May we close this issue?

I got It! Thanks everyone.


migrate to swift 4.0

Fixes #1

Hey @paulz, thank you so much for your work, we really appreciate that!
Unfortunately, we decided to stop the maintenance of this project for this time being. I encourage you to fork the project and update the Swift version there. We will link to your fork from the Readme as well.
Let me know what do you think 😉

@paulz we are archiving this repo so it will become read-only. However, if you want your fork to be linked here please drop a line to opensource(at)droidsonroids.pl.


[Tensor] Add cumsum operation

Implement cumulative sum across an input dimension.
Example:
let x = Tensor::arange(0..6 as i64, &device);
x.cumsum(0)
// [ 0,  1,  3,  6, 10, 15]

cumprod has also been asked for twice on discord:
https://discord.com/channels/1038839012602941528/1038839013735399547/1234609825619644477
https://discord.com/channels/1038839012602941528/1091796857996451942/1202670351322972302
let x = Tensor::arange(1..7 as i64, &device);
x.cumsum(0)
// [ 1,  2,  6,  24, 120, 720]


Play Store parade screenshot

Description of issue
This screenshot needs to replace the current one
Screenshots

I'll do this now! 👍

💪


[BUG] There's no downloads

Describe the bug
None of the games in version 2.0.3 have downloads.
Steps to Reproduce
Enter any game.
Expected behavior
Have downloads.
Screenshots
No response
Operating System
Windows 10
Hydra Version
2.0.3
Additional Information
No response

Add source https://hydralinks.cloud/

@Kumbbar instale todas las fuentes del enlase y todavia la gran mayoria de juegos no tienen descargas

@Kumbbar instale todas las fuentes del enlase y todavia la gran mayoria de juegos no tienen descargas

These links contain a few games as i understand it. There is something similar in this discussion https://www.reddit.com/r/hydralauncher/comments/1e64le2/is_hydralinkscloud_safe/ But i have enough of these games with hydralinks


calibre: 7.13.0 -> 7.14.0

Description of changes

https://github.com/kovidgoyal/calibre/releases/tag/v7.14.0
https://calibre-ebook.com/whats-new

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

[x] Fits CONTRIBUTING.md.

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

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

2 packages built:

calibre
unbook


Update documentation

Update documentation to add precision in geolib.getDistance and add method geolib.isPointInLine.

@manuelbieh hold on this PR. I will check and update all methods, if necessary.

I found the fallow methods undocumented:
alias:
getKeys
getLat
getLon
isInside
withinRadius
getDirection
methods:
validate
getDistanceSimple
getBounds
getBoundsOfDistance
preparePolygonForIsPointInsideOptimized
isPointInsideWithPreparedPolygon
getRhumbLineBearing
getBearing
getCompassDirection
decimal2sexagesimal
isDecimal
isSexagesimal
round
Should I add reference in README.md to create another file? And if yes, should I move all other documented methods to same file? I Think we could create a file: docs/API.md

Oops, now I merged it.
docs/API.md sounds like a good idea since the number of methods is constantly growing. Unfortunately, to be honest, I don't know how every of the methods mentioned above works. getBoundsOfDistance, preparePolygonForIsPointInsideOptimized and isPointInsideWithPreparedPolygon for example were pull requests which I merged without completely understanding how they work or what they do exactly.
I understood just enough to decide that they sound like useful extensions ;)

O will move documentation to that file and checkout how theses two methods works, if I can't I will try contact with original authors


impl(rest): change ServerTimeout type to match type of literals

Options{}.set<ServerTimeoutOption>("3.14") creates a double not a float, which in turn results in a narrowing conversion warning. Change the option type to double as a quality of life improvement.

This change is

google/cloud/rest_options.h line 59 at r1 (raw file):
Previously, scotthart (Scott Hart) wrote…
I was just going off of https://cloud.google.com/apis/docs/system-parameters which says about ServerTimeout:
Timeout (in seconds, float value) for the server to finish processing the request. This system param only applies to REST APIs for which client-side timeout is not applicable.

Yes.  That is the best they can do over HTTP/gRPC.  But the C++ representation does not need to be a floating point?  And representing durations as integers or floats is a "Known Problem" [tm] the units are implicit, implicit conversions can result in bad values, etc.


Need the ability to automate continuous integration checks

Title: Need the ability to automate continuous integration checks
As a Developer
I need automation to build and test every pull request
So that I do not have to rely on manual testing of each request, which is time-consuming
#### Assumptions
* GitHub Actions will be used for the automation workflow
* The workflow must include code linting and testing
* The Docker image should be postgres:alpine for the database
* A GitHub Actions badge should be added to the README.md to reflect the build status
#### Acceptance Criteria
gherkin     Given code is ready to be merged     When a pull request is created     Then GitHub Actions should run linting and unit tests     And the badge should show that the build is passing

Title: Need the ability to automate continuous integration checks
As a Developer
I need automation to build and test every pull request
So that I do not have to rely on manual testing of each request, which is time-consuming
Assumptions

GitHub Actions will be used for the automation workflow
The workflow must include code linting and testing
The Docker image should be postgres:alpine for the database
A GitHub Actions badge should be added to the README.md to reflect the build status

Acceptance Criteria
gherkin Given code is ready to be merged When a pull request is created Then GitHub Actions should run linting and unit tests And the badge should show that the build is passing


when submitting a review (comment/request changes) comment can only be one line

It would be nice if the comment part could be places into a buffer so you can create a multi line comment instead of a single line.
Enter review submit comment: <new lines do not work>

Feel free to open a PR for these suggestions.
I will review.


Update repository after transfer

See https://github.com/senzing-garage/garage-roadmap/issues/3

Pull requests:

https://github.com/senzing-garage/keystore-generator/pull/13


Add ERC721Encoder

Fix #9 and #12
ERC721Encoder is generated from ERC721Encoder.swift.gyb and erc721.json (from https://github.com/0xcert/ethereum-erc721) by gyb.

This make sense, is it possible to keep GYB as separate repo?
Otherwise looks good to me

👍

Let's keep it simple then


Video Library theming and modal functionality

When I use the category filter and the number of of videos displayed is not divisible by 3 the video size expands to fill the region.  Not sure if this is intended behaviour.


question about lora of PuLID-FLUX

Does PuLID-FLUX support lora? If yes, can you provide a code example to use lora in the README?
thanks

@cqflgl Yes it supports.
Check my workflow here:
PuLID FLUX with LoRa support.json

@NeoAnthropocene How can we use it in API?


Adaptivecards support

Now with adaptive cards being available and consoleconnector not supporting adaptivecards is there a way to automate testing for these features?

I haven't tested with it, but the exported TestConnecter should be able to handle it just fine as long as its passed in as an IMessage. Let me know if that works. I'll try it out sometime soon.

Sorry about the delay. I just tested locally and and first glance it looks like the test connector doesn't support it. I'll open this up as a feature request. I'll happily look at PRs. I've got less bandwidth around the holidays, but will address this/work with any collaborators as soon as I can.

#45 feature added


Unable to Add Image to gmail.tools.add_compose_button

Hi! Apologies, as I'm new here and have a feeling my question is pretty easy to resolve. I'm trying to create a button alongside "Send" and am having trouble adding my image icon and styling the button in general. See below my code and my output. Can someone answer:

How do I properly add my png to the button?
How do I style the button in general? Can someone provide a sample code if possible - I'm having a hard time picking up how I'd do this with just the documentation.

Thank you for all of the help!
gmail.observe.on("compose", (compose) => {
      console.log("New compose window is opened!", compose);
      console.log("Compose Ref", gmail.dom.compose());
      gmail.tools.add_compose_button(
        compose,
        '<img src="images/parrot_128.png" width="16" height="16" alt="Parrot Icon" />',
        () => console.log("Custom Compose Button Clicked!"),
        ""
      );
    });

You need to construct a fully qualified URL, based on your extensions runtime root URL/domain.
Your image URL is relative to the page, which is Gmail and not your extension.
https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/runtime/getURL

@josteink Thank you!
For anyone new who may run into the same problem -
I generated my image url in extensionInjector and then stored in a non-script document element. Then in extension, you can access this document element to grab your image.


Dispatch create not making QnA intent

Describe the bug
When running dispatch create against a bot file with 1 qna model and 2 luis models, the resulting dispatch model only has intents for the luis models, even though output shows the qna model being exported:

Dispatch version 1.0.8-14

I can't repro this.  Lauren, could you share your dispatch file in email?  Thanks!

QnA is empty and thus not being added to Dispatch.


Issue deploying to heroku

I've got an app using the activerecord-oracle_enhanced-adapter gem locally and trying to get it running on heroku with this buildpack. I've tried starting with the buildpack and adding it after deploying the app and neither have worked. After I push the buildpack, I'm not able to hit any of the views in the app or run heroku run rake.  Any thoughts on what I am doing wrong?
Here is my output on git push heroku

It looks like your Rails app has no .ruby-version file in its root, so I'd suggest trying to add one, committing it, and then pushing again to Heroku.

Missed that, had it in the Gemfile. I added it in the .ruby-version file and re-deployed. Still not able to hit a view.

Well, your output shows that the buildpack finished doing what it's meant to do without issue. We've only tried OCI8 with the datamapper gem and not the one you're using, so I'm not sure what's going on there.
Perhaps you could try running heroku logs in your application's folder to see why it's failing?


Add support for CF data structures/utils on WASI

A lot of data structures, like strings, characters, data, locales, plists just need a proper check for || TARGET_OS_WASI to be enabled on that platform and to use a suitable implementation. Some code related to filesystem access had to be disabled on WASI though.

@swift-ci please test


Resizing modal sends it outside window view area

Summary
Add/edit plugin action opens plugin configuration in a modal dialog.
When trying to resize it by dragging it using right lower corner, It enlarges in all directions, even going outside of the window view area.
Expected behaviour
Resizing modal should follow mouse dragging movement and resize only to the right and / or down.
Actual behaviour
Modal dialog resizes in all directions, even outside window view area
Environment

Python version: 2.7.6
Django version: 1.8
django CMS version: 3.5.1

It's not perfect, yes, but it works as intended. Would appreciate the pull request to limit the max size of the modal to the window size, but otherwise it's a wontfix.


Plugin Completely Unusable! - Every Sync Deletes Saved Config

Issue Report: Tabby Cloud Sync Settings Plugin; Status: Completely Unusable
Environment:

Tabby Cloud Sync Settings version: 1.6.5
Tabby version: Latest
Operating System: Windows 11 24H2

Description: This Tabby Cloud Sync Settings plugin is currently unusable. Immediately after enabling the plugin, setting it up, and configuring a sync location (in this case, syncing to GitHub secrets using an API key), the plugin cleared my entire configuration and all profiles. Every subsequent sync continues to do this, despite the logs showing "Sync direction: local to cloud"
I was quite upset by this plugin initally wiping out all of my saved profiles, but hoped it was a one-time issue. Unfortunately, this was not the case. Every time the plugin syncs, it removes and wipes out all configuration changes.
Steps to Reproduce:

Enable the Tabby Cloud Sync Settings plugin.
Configure the sync location, I chose GitHub secrets using an API key, but I'm unsure this matters.
Observe that the entire configuration and profiles are cleared.
Attempt to reconfigure any settings.
Notice that every subsequent sync removes all changes.
Logs: The logs indicate that the plugin is syncing from local to cloud and then states, "Your saved settings were synced." This event causes the current configuration to be wiped out.

Example: While selecting a color scheme and customizing colors, my changes were completely removed, and the default color scheme was reselected. At that moment, another "Your saved settings were synced." entry was logged in the plugin. This was observed over and over.
Impact: At this time, the plugin is completely unusable as it consistently wipes out all configuration changes upon syncing.
Debugging: Unfortunately, there aren't any options to enable verbose or debug logging, so I'm unable to provide additional details. The Gist appears to change with each sync, but since it's encrypted, I can't determine its contents. My expectation is that the plugin should detail the sync location and specify whether local settings were synced to the cloud or vice versa. If local settings were synced to the cloud, there should be no modification of local files. However, local files are being modified and replaced, indicating either an incorrect sync direction or erroneous code in the local-to-cloud sync option when using GitHub Gist.
Tested v1.5.2: Conclusion – same behavior. While configuring colors, the plugin replaced local settings again and overwrote everything. This includes the Tabby welcome page and analytic settings, meaning all settings are replaced.
Edit 12-15-2024: It appears that https://github.com/niceit/tabby-cloud-sync-settings/issues/32 may be the same bug. I was originally doubtful this was the case, but upon closer inspection of the comments, it appears that the issue I'm experiencing has been reported for over 2 years.

I would like to add an additional bug: When attempting to submit feedback within the plugin, it errors out with "status code 500" even after multiple attempts.

I'm actually having the same issue. Windows client.

I'm actually having the same issue. Windows client.

Are you using the Gist sync method? I found another sync plugin using the plugins section of Tabby, and that one seems to work, albeit it's a much simpler plugin – but it syncs one-way and allows manual downloading of the configuration data saved in Gist.

Please don't encrypt config file with Tabby while using this plugin, otherwise you could use Tabby sync feature (You should set up yourself), per my point of view that setting has buggy and you will lose your settings and never got it back.

I'm actually having the same issue. Windows client.

Are you using the Gist sync method? I found another sync plugin using the plugins section of Tabby, and that one seems to work, albeit it's a much simpler plugin – but it syncs one-way and allows manual downloading of the configuration data saved in Gist, which one would use if changed were saved on another PC.

No, I sync to an s3 bucket that indeed is the major reason I wanted to transition to Tabby.

Yes per this point

Please don't encrypt config file with Tabby while using this plugin, otherwise you could use Tabby sync feature (You should set up yourself), per my point of view that setting has buggy and you will lose your settings and never got it back.

if you want  to use  Tabby built-in sync plugin, in  case disable my plugin or  uninstall  it, don't mix  them  together

I'm not mixing the sync plugins. I'm using the standard Tabby vault options which encrypts passwords and key files, but otherwise handles config files in the default fashion. With those default options in mind, the behavior I outline in the description of this report is how this plugin is working. On a brand new config blank slate, because this plugin wiped out my settings, it doesn't sync at all. It simply wipes out your current settings, and I have no idea what it uploads to Gist.
@niceit

@MNLierman if you set the master password and encrypted the config file, I cannot guarantee the plugin will work well with the encrypted config. The last hope for you to  recover your previous config, please follow the steps below:

Unload or uninstall this plugin,  then shut down the application.
Locate the Tabby config folder (see this link https://github.com/Eugeny/tabby/wiki/Config-file).
Look if you can see the backup file : config.yaml.backup
If you saw that file, copy the current one config.yaml   to some other place.
Rename  config.yaml.backup to config.yaml

Then open the application again. Good luck.
Again this plugin will not work if the master password is set or built in the plugin, decide which one you will use.

Thank you for your response! While retrieving my previous profile/config isn't a primary concern for me at this moment, I appreciate the information and will look into it. My main focus is getting the plugin to function correctly. It seems that other users are experiencing similar issues. Are you suggesting that the vault functionality is completely unavailable if I am using this plugin?
One of the key reasons for using Tabby is its ability to securely encrypt passwords and key files, and I need this to happen across multiple computers. The synchronization direction (one-way or adjustable per device) isn't a concern for me, as long as I can add passwords on one computer and be able to sync the configuration onto another using this plugin. While alternatives like OneDrive and rsync are options, the simplicity and universality of this plugin are what drew me to using Tabby specifically.

@MNLierman

vault functionality is completely unavailable if I am using this plugin?

Yes, if you use Vault password protected or Tabby built-in sync, please consider remove this plugin to prevent  conflicting.

Should be closed for now


Remove hardwired code in satellite pairing

Initial comparisons include hardwired naming of variables. Generalize this to be more universally usable.

Done prior to develop_satellite merge.


Add TypeScript declarations

This will make it easier/possible to use this library in TypeScript.
I think I got the type definitions right based on the current code.

Awesome! Thanks.


updating tutorial due to sample update

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

@hamiltonha @pkanher617 fyi. I'll make another update once msal v2.0 is up

@navyasric If this PR is approved, the PR team can move the commit to the private repository so the updates can be quality-validated and viewed on staging before merging.
@derisen - Because you make substantial updates in the PR you create, consider working in the private repository (azure-docs-pr). In the private repo, your content gets the benefit of quality validation, and you can verify it on staging before publishing.
Although you can sign off directly in your PR in the private repo, if you want a member of the content team to review your updates in the private repo before merging, you can request a review or at-mention them in a comment before you sign off. See GitHub for content reviews.
For more information, see Should I work in a private or a public repo?
If you need help getting set up to work in the private repo, see Set up Git repository locally for documentation.
Thanks.

@ktoliver thank you very much for the tip -you're right, I should have done in the private repo. I will definitely make sure next time.

@derisen Thanks

@ktoliver I believe this PR can be moved to the private repo for further review. Also, Please assign to @hamiltonha who owns this area now.

@navyasric Thanks. We'll do this today or tomorrow. I'll post the new PR # here and then close this PR.

The commit in the PR was moved to private repo PR https://github.com/MicrosoftDocs/azure-docs-pr/pull/108152 for validation and staging. We'll close this PR.


Extension Request - phpcs

[ ] Extension name: phpcs
[ ] Extension GitHub or homepage: https://github.com/ikappas/vscode-phpcs

Thank you for your request! The extension is now live in the code-server extension marketplace.

Howdy I am trying to use phpcs on code-server and it crashes with:
Error: Cannot find module '/root/.local/share/code-server/extensions/ikappas.phpcs-1.0.5/server/src/server.js'
Require stack:
- /root/.local/share/code-server/extensions/ikappas.phpcs-1.0.5/node_modules/vscode-languageclient/lib/utils/electronForkStart.js
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:965:15)
    at Function.Module._load (internal/modules/cjs/loader.js:841:27)
    at Module.require (internal/modules/cjs/loader.js:1025:19)
    at require (internal/modules/cjs/helpers.js:72:18)
    at Server.<anonymous> (/root/.local/share/code-server/extensions/ikappas.phpcs-1.0.5/node_modules/vscode-languageclient/lib/utils/electronForkStart.js:110:9)
    at Server.emit (events.js:315:20)
    at Pipe.onconnection (net.js:1555:8) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '/root/.local/share/code-server/extensions/ikappas.phpcs-1.0.5/node_modules/vscode-languageclient/lib/utils/electronForkStart.js'
  ]
}
[Info  - 4:45:52 PM] Connection to server got closed. Server will restart.
internal/modules/cjs/loader.js:968
  throw err;
  ^

Looks like the extension isn't building correctly. I opened https://github.com/cdr/code-server/issues/2571

Looks like the extension isn't building correctly. I opened https://github.com/cdr/code-server/issues/2571


Release 2.25.0 for Windows & MacOs

Replacing People talking | Amazing Sounds with Peter Baeten with ASMR Pressing Buttons (clicking, fidgeting, typing sounds).

https://github.com/kkamara/kelvinkamara/releases


config.json: inputs/switches/commands

Thanks for this great plugin!
Now  I can ON/OFF my Onkyo NR636, but would you please tell how to select inputs, or the generic settings with inputs/switches/commands in config.json into e.g. wiki?
the initial config as accessories works, but for what reason it cannot reflect the correct status on/off in HomeKit like Samsung  TV or my TP-link plug(which controls the Onkyo  and TV, etc.) , at moment if I off TV, Onkyo will also off, which is also logical and correct by means of HDMI CEC.
@toshibochan: I tried your config as platform "onkyo", but error shows the platform is not recognized. Have you remove the accessories section for onkyo?  could you elaborate a little more?
BTW: This is what I would like to reach:
https://midnite.im/de8965

Did you install @solowalker27 homebridge-onkyo?

@solowalker27 is platform not accessories

@toshibochan thanks for sharing!
Do you mean Tod's plugin don't have these options of platform? vs. @solowalker27?

Correct. I forked Todd’s plugin to contribute some new features including building our platform support. As that’s a major breaking change, I haven’t merged it to Todd’s yet. Still unsure as to when that would be a good idea. Perhaps we should just rip the bandaid off.
I’m currently trying to solve a couple bugs for iOS 13, though. When I get those fixed I might be more inclined to get a PR going.

@solowalker27 thanks for passing by.
I don't know if I should further discuss here or in your board. But I think  your "platform" is absolutely  a great idea & break change! Merge your with Todd's is likely to give wings to fly.


[NOMERGE] Test bonfire v4

Description
Testing bonfire v4 for pr_check (example: https://github.com/RedHatInsights/insights-rbac/pull/677)
Fixes # (issue)
Type of change
What is it?

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Documentation update
[ ] Tests update
[ ] Refactor

Checklist:

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I run go fmt ./... to check that my code is properly formatted
[ ] I run go vet ./... to check that my code is free of common Go style mistakes

Codecov Report

Merging #907 (1c7476b) into main (86386f9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #907   +/-   ##
=======================================
  Coverage   30.85%   30.85%
=======================================
  Files          45       45
  Lines        5101     5101
=======================================
  Hits         1574     1574
  Misses       3305     3305
  Partials      222      222

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 86386f9...1c7476b. Read the comment docs.

/retest

/retest

/retest

/retest


WebM MSE video stalls after one second in Firefox 42+

I reported this bug on Mozilla's Bugzilla but this appears to be a bug in the simpl.info/mse player, not Firefox:
https://bugzilla.mozilla.org/show_bug.cgi?id=1202580
STEPS TO REPRODUCE:

Set the following about:config prefs:
media.mediasource.webm.enabled = true
media.mediasource.format-reader.webm = true
Load http://simpl.info/mse/
Play the test video if it doesn't start automatically.

RESULT:
In Firefox 42 (Aurora/Dev Edition), the video plays for two seconds then stops. In Firefox 43 (Nightly), the video plays for only one second.
The diagnosis was that this test does not wait for sourceBuffer updateend before calling sourceBuffer.appendBuffer again. This also fails with mp4. Works ok if readChunk_(++i); is called on updateend.

Many thanks for this!
Apologies for the crazy delay in responding — I've been snowed under with work this past year :/.
Looks like this is working now in Firefox :).
I'll close the issue, but reopen if you need to.


[TRIVIAL] Lock Appveyor CI build to scons 2.5.0

Workaround for misbehaving 2.5.0.post1, and can be reverted if the next version works better.

Current coverage is 69.69%

Merging #1649 into develop will decrease coverage by -0.01% as of 0641636

@@            develop   #1649   diff @@
=======================================
  Files           907     907
  Stmts         67382   67382
  Branches          0       0
  Methods           0       0
=======================================
- Hit           46971   46962     -9
  Partial           0       0
- Missed        20411   20420     +9

Review entire Coverage Diff as of 0641636

Powered by Codecov. Updated on successful CI builds.


Fixes MSBuild issue #229

Thanks, i'll have a look but we'll need to either add all VS paths to this (i.e. Enterprise, etc...) or use a trick that we found with Umbraco to auto identify the MSBuild path.

I'll get this merged and will tweak the build script


[Bug] filename must not contain illegal characters

Error saving images (Ctrl+click or drag), if filename contains "%E2%80%8F"
Error text: filename must not contain illegal characters
Image example with filename in question: https://www.a-rubi.co.il/wp-content/uploads/2014/06/ZahalanovK.png
Windows 10. Firefox 69.0.2

https://www.fileformat.info/info/unicode/char/200f/index.htm

I think filename (path excluded) should be sanitized with this:
function gConvertToSpaceRegExp ()  {
  // Note: we remove colons everywhere to avoid issues in subresource URL
  // parsing, as well as filename restrictions on some OSes (see bug 1562176).
  /* eslint-disable no-control-regex */
  switch (runtime.PlatformOs) {
    // On mobile devices, the file system may be very limited in what it
    // considers valid characters. To avoid errors, sanitize conservatively.
    case "android":
      return /[\x00-\x1f\x7f-\x9f:*?|"<>;,+=\[\]]+/g;
    case "win":
      return /[\x00-\x1f\x7f-\x9f:*?|]+/g;
    default:
      return /[\x00-\x1f\x7f-\x9f:]+/g;
  }

}

function sanitize(leafName) {
    if (runtime.PlatformOs == "win") {
      leafName = leafName
        .replace(/</g, "(")
        .replace(/>/g, ")")
        .replace(/"/g, "'");
    }
    return leafName
      .replace(/[\\/]+/g, "_")
      .replace(/[\u200e\u200f\u202a-\u202e]/g, "")
      .replace(gConvertToSpaceRegExp(), " ")
      .replace(/^[\s\u180e.]+|[\s\u180e.]+$/g, "");
  }

We already have an escapeVariable function:
https://github.com/eight04/image-picka/blob/29116aca07f4b92f8eebc21b374b184539276257/extension/background/background.js#L508
It might be better to decide a range (or use the actual list?) and check which character is invalid in downloads API. We have to test Firefox and Chrome, they have different behaviors.

I have posted how Firefox validates for Webextension API.
At least this one is must for Firefox extension:
.replace(/[\u200e\u200f\u202a-\u202e]/g, "")

Can you post the link to the source?


Set up a "Hello world" project as one app with Webpack

Project requirements

Create a new Rails app called 'hello-rails-react'.

Use the command rails new -j webpack my_new_rails_app as described in the previous lesson's tutorial.

Verify that the jsbundling-rails gem is added to your Gemfile.

Initialize your project with Git.

Install needed node dependencies as described in the previous lesson's tutorial.

Create a babel.config.js in the root folder of your project as described in the previous lesson's tutorial.

Update webpack.config.js to include the babel loader as described in the previous lesson's tutorial.

Make sure that your project has a Postgres database set up.

Create a table for storing your messages and create 5 different greetings.

Create an API endpoint that selects a random greeting from your table (you will need a controller with an action and Rails route).

Create a static view that will be the root of your app.

Create the App component with react-router and render it in your static view.

Create the Greeting component that will display a greeting. Set it up as a route in your App component.

Create a store, an action, and a reducer that will connect to your API endpoint to get the random greeting.

Display the random greeting in your Greeting component.

@yusufyusie, Please do not push new changes after requesting a review. It becomes confusing if you push multiple changes within a short interval. Please refrain from doing so.

@yusufyusie, I have been repeatedly rebuilding the app and providing feedback commits. However, each time I try to finish the review, you push new changes. Then I have to rebuild the entire app and check all the changes again. This has happened three times so far and it is quite confusing. So I have provided my feedback based on the review commit number [29fa1bf] (https://github.com/yusufyusie/hello-rails-react/pull/1/commits/29fa1bfbb8346c55f19b88c7ca49f17e4ab8dfe9) which was before the review was received.
It seems that each time I'm on the verge of completing the review, you push new changes, necessitating a rebuild of the entire app. While I understand the iterative nature of development, this has occurred three times already, leading to some confusion on my end.
To streamline the review process and ensure clarity, I suggest before requesting a review check the changes carefully and stop committing until get feedback from the reviewer.  This would allow both of us to work more efficiently and minimize the need for repeated rebuilds and rechecking everything.
For reference, my feedback is based on the review commit number [29fa1bf] (https://github.com/yusufyusie/hello-rails-react/pull/1/commits/29fa1bfbb8346c55f19b88c7ca49f17e4ab8dfe9), which was the state of the app before the review was initially received.


Menu bar rendering not correct with ipython feature.

Issue Type: Bug

open an empty folder, create an 'python jupyter note book' and open it.
(as shown in GIF below)

The menu bar is not shown correctly but functions as normal.
Extension version: 2019.11.50794
VS Code version: Code 1.41.1 (26076a4de974ead31f97692a0d32f90d735645c0, 2019-12-18T14:58:56.166Z)
OS version: Windows_NT x64 10.0.14393

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-4210U CPU @ 1.70GHz (4 x 2394)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
11.90GB (5.63GB free)

Process Argv

Screen Reader
no

VM
50%

@cangyin Thanks for the issue, I was able to reproduce this issue. We will work to get this fixed!

I believe this would be a vscode issue. They're letting the webview UI push up into the toolbar for the main UI.

@rchiodo  Am I supposed to open a new issue for vscode?

You can if you like, or we can do it for you.


Add websocket support to async-http-client fs2 backend

Adds an implementation of AsyncQueue based on what fs2 offers so that users of the fs2 backend can use WebSocket support as well.
Once this PR is merged, I'll try to add a handler which takes a fs2.Pipe to handle incoming and outgoing messages on an even higher level than the current API offers.

Looks good, thanks! :)

@ybasket I've been thinking about the semaphore - as far as I understand, it's there to provide mutual exclusion for clear and offer. However, clear is only ever used before an offer when called from the AHC websocket listener to clear messages in case of an error. As far as I know, the AHC listener is sequential, so maybe it would make sense to change the AsyncQueue contract to specify that offer and clear are thread-unsafe (can't be called concurrently), and then we won't need the semaphore, right?

@adamw It provides mutual exclusion between clear and poll (not offer, that is safe for the reasons you described). As of my understanding, poll is not coupled to the AHC's sequentiality (as the user can always ask to receive from the websocket), so we need to make sure that no poll happens concurrently while clearing the queue (a task fs2 has no direct support for). If that wasn't ensured, we could have the following:

AHC listener calls clear, that gets the current size n of the queue and streams as many elements to Nirvana
While the stream inside clear is still running, the user code calls receive, therefore poll gets called, removing one element
the stream of clear hangs as it sees only n-1 elements and no new element will ever be enqueued

So I don't see a way to way to implement AsyncQueue based on fs2 safely without a semaphore/mutex/similar. One could of course try to do an implementation backed purely by a Ref[List[A] and other cats-effect stuff, but that's probably not very efficient (haven't benchmarked though). Alternatively, the need for clearing the whole queue could be revisited – do you know what was the reason for this design choice? As a user I might be interested in all messages delivered before the error…

Ah yes, you are right of course :) yes, poll and clear can be concurrent
Regarding the clearing: maybe indeed we don't really need clear. The reasoning was to fail-fast when there's an error, and let the user know as fast as possible. However, if there's an error, any sends will fail anyway, and you might be right that there's no reason to block receives. What do you think?

I'm not sure about clear, but I tend to say we should remove it.
Maybe we can achieve configurability, so that the user can choose between both behaviours. The hard part here would be to reflect that in types so that in case the user doesn't want clear, we save the semaphore overhead and allow a broader range of queues being used (those without clear implementation as every fs2 Queue which is not an InspectableQueue). I need to have another look at the code and play around to see whether that's feasible.

Yes, after some thought I think we should remove this logic and thus clear as it won't be needed.

Btw. the above has been released in 2.0.0-M8


Serial routing method needs more clear explanation

Description:
Looking into our documentation, we have contradicting information, we can read:
“Serial routing rings all call agents one by one in the order specified in the Call agents list. "
And we can also read:
“4. It's not possible to set the order the agents are presented with calls.”
Suggestion:
I believe the documentation should include an better explanation on point number 4, maybe adding that sorting with DL/Group is done via alphabetical order automatically and that's the reason why you cannot sort. And/or mention that order agents in this method is possible if you add them one by one in the list, and if you add a DL/Group, it's not possible to sort them manually.
Documentation:
https://learn.microsoft.com/en-us/microsoftteams/create-a-phone-system-call-queue?tabs=agent-selection#step-4-select-your-agent-routing-options

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

ID: b755d9ae-13ac-019f-f5f4-a95b1eee2989
Version Independent ID: f7afd8ad-e6e0-6c79-aed1-4ee87d48ba12
Content: Create a Call queue in Microsoft Teams - Microsoft Teams
Content Source: Teams/create-a-phone-system-call-queue.md
Service: msteams
GitHub Login: @DaniEASmith
Microsoft Alias: danismith

Hello @tiagoroxo
The sentence you propose is very similar to the previous sentence that this article had so I'm trying to understand why it was changed to the current one. I'll get back when I have details.
Thanks

We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


🚀 Lift off

This is an issue reserved for the TU Delft Student Software Project '23 - LaplaceRedux
This part forms the core of the project and is all about scaling things up to lift the package off the ground. The following two issues should probably be tackled in this order:

[ ] #20
[ ] #19
[ ] #17
[x] #12
[ ] #27

Interfacing this package to MLJ (or perhaps directly to MLJFlux) would be really great, but is less of a priority than the other two issues:

[ ] #18

closed by #33


Simplify error messages against intersections of weak types

I spoke a bit with @sandersn about ways we can tackle the type madness issue (#14662). I noted that JSX scenarios are much more broadly applicable right now and are pretty prevalent given the sorts of type arithmetic we see a lot of in the React community. :smiley:
Problem: JSX optional attributes make errors too hard to read
interface FooProps {
    name: string;
    age: number;
}

class Foo extends React.Component<FooProps> {
    constructor(props: FooProps) {
        super(props)
    }
    render() {
        const {name, age} = this.props;
        return
I'm {name} and am {age} years old
;
    }
}

// Type '{ blah: number; }' is not assignable to type 'IntrinsicAttributes & IntrinsicClassAttributes<Foo> & Readonly<{ children?: ReactNode; }> & Reado...'.
//   Type '{ blah: number; }' is not assignable to type 'Readonly<FooProps>'.
//     Property 'name' is missing in type '{ blah: number; }'.
<Foo blah={10} />;

Proposal
When doing relation checks against an intersection target, if a check fails, see if relating against the same type with weak types removed would result in the same outcome. If so, use that comparison when elaborating types.
The great part is that this actually solves a broader set of problems than just JSX.

Keep in mind that this needs to be tested against an intersection that consists of nothing but intersection types. We should never fall back to never.

@DanielRosenwasser I think you mean an intersection that consists of nothing but weak types.


Gitignore parsing does not respect repository boundaries

The current Gitignore parsing does not take into account the boundaries of git repositories, meaning:

.gitignore files that are outside of git repositories are parsed and applied.
.gitignore files from the parent repository are applied to files in git subrepositories.

I see that there have been a couple of PRs associated with this issue. Has the issue been resolved? Or is there still work to be done to address this in its entirety?
There is a section of docs that should be updated if this is complete.

This iteration of this bug hasn't been fixed. There was a previous issue related to this, but that's since been closed.
I don't believe anything has changed since that section of the docs were written, so probably no updates are needed.

@michaelkedar Thanks! I was trying to figure out if it was fully resolved by yesterday's release, but it sounds like it wasn't. I'll wait on the doc update.


[vowpal-wabbit] no absolute paths

For https://github.com/microsoft/vcpkg/issues/20469
https://github.com/microsoft/vcpkg/pull/27323 merged 3 days ago ...

Damn, sorry :)


Segfault restarting T265 Pipe

Required Info

Camera Model
T265

Firmware Version
<IP_ADDRESS>1

Operating System & Version
Raspbian / Raspbery Pi OS

Kernel Version (Linux Only)
4.19.97-v7l+ -> 5.4.44-v7l+

Platform
Raspberry Pi 4

SDK Version
2.33.1 -> 2.35.1

Language
C++

Segment
Robot

Having a segfault when stopping a pipe, making a new pipe (as a shared pointer) and starting the new pipe with the existing config.
As documented in #6362 the T265 has a frequent issue with USB communication on the RPI4, where it seems the SDK stops asking for data, and the device the stops sending with the error:
02/06 01:04:05,676 INFO [2949632528] (tm-device.cpp:1408) T265 FW message:<PHONE_NUMBER>7758289392: [0x�/2:721] Host not reading - stopping
In our efforts to find a solution, we tried to find a workaround whereby we set a timeout, and if no message has been received, catch the error and restart the pipe.
My current code is below:
    rs2::log_to_console(RS2_LOG_SEVERITY_INFO);

    rs2::config cfg;
    cfg.enable_stream(RS2_STREAM_POSE, RS2_FORMAT_6DOF);
    auto pipe = std::make_shared<rs2::pipeline>();
    pipe->start(cfg);
    unsigned int to = 20000;
    while (true)
    {
        try {

            auto frames = pipe->wait_for_frames(to);
            to = 40;
            auto f = frames.first_or_default(RS2_STREAM_POSE);
            // Cast the frame to pose_frame and get its data
            auto pose_data = f.as<rs2::pose_frame>().get_pose_data();

            // Print the x, y, z values of the translation, relative to initial position
            std::cout << "\r" << "Device Position: " << std::setprecision(3) << std::fixed << pose_data.translation.x << " " <<
                pose_data.translation.y << " " << pose_data.translation.z << " (meters)" << std::endl;
        }
        catch (const rs2::error& e) {
            to = 20000;
            std::cout << "T265 crashed. Restarting ..." << std::endl;
            pipe->stop();
            pipe = std::make_shared<rs2::pipeline>();
            pipe->start(cfg);
            std::cout << "... restarted." << std::endl;
        }
    }

This will catch and restart the data pretty well, typically taking under a second to recover. A recovery is performed roughly once every 30 seconds - 3 minutes. However, sometimes instead of recovering we get a segfault.
Currently my debugger points to atomicity.h, but I am working to see if better tools will provide more information.
if (__gthread_active_p())
      return __exchange_and_add(__mem, __val);
else
      return __exchange_and_add_single(__mem, __val);

So, in lieu of an actual solution to the rebooting problem, is there a way to safely (and quickly) dispose of whatever handles, resources, and callbacks belong to the pipe, so that when we make and start a new one maybe we avoid the segfault? Ideally we restart the pipe as fast as possible to avoid the robot's position changing too much during the reboot.

Attaching a PR worth reviewing for this issue:  https://github.com/IntelRealSense/librealsense/pull/8561

Attaching a PR worth reviewing for this issue: #8561

No. This is another issue.
But it's also easy to fix. You just need to call cfg.disable_all_streams() before starting pipeline again. And don't forget enable stream one more time:
catch (const rs2::error& e)
{
    to = 20000;
    std::cout << "T265 crashed. Restarting ..." << std::endl;
    pipe->stop();
    cfg.disable_all_streams();                               // add these two lines
    cfg.enable_stream(RS2_STREAM_POSE, RS2_FORMAT_6DOF);     //
    pipe = std::make_shared<rs2::pipeline>();
    pipe->start(cfg);
    std::cout << "... restarted." << std::endl;
}

Hi @csr-kick  May I ask you a question, where should the code you mentioned be modified in the camera program？
rs2::log_to_console(RS2_LOG_SEVERITY_INFO);

rs2::config cfg;
cfg.enable_stream(RS2_STREAM_POSE, RS2_FORMAT_6DOF);
auto pipe = std::make_shared<rs2::pipeline>();
pipe->start(cfg);
unsigned int to = 20000;
while (true)
{
    try {

        auto frames = pipe->wait_for_frames(to);
        to = 40;
        auto f = frames.first_or_default(RS2_STREAM_POSE);
        // Cast the frame to pose_frame and get its data
        auto pose_data = f.as<rs2::pose_frame>().get_pose_data();

        // Print the x, y, z values of the translation, relative to initial position
        std::cout << "\r" << "Device Position: " << std::setprecision(3) << std::fixed << pose_data.translation.x << " " <<
            pose_data.translation.y << " " << pose_data.translation.z << " (meters)" << std::endl;
    }
    catch (const rs2::error& e) {
        to = 20000;
        std::cout << "T265 crashed. Restarting ..." << std::endl;
        pipe->stop();
        pipe = std::make_shared<rs2::pipeline>();
        pipe->start(cfg);
        std::cout << "... restarted." << std::endl;
    }
}


⚠ Bofrost.at has degraded performance

In 35a5b65, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1629 ms

Resolved: Bofrost.at performance has improved in 0aeb55f after 6 minutes.


Some README copyediting

Fixing some grammar, typos, and adding some punctuation, as a treat.

Thanks !

Released 0.0.5.


Unauthorized Workaround

Override flask_oidc error callback to redirect to /
When any of the Keycloak login querystring parameters are invalid (or stale), flask_oidc returns an error page without a way forward. Override the error callback to redirect to / instead, to start another login attempt.

Hold up @ivan-c  - testing didn't redirect.

Hold up @ivan-c - testing didn't redirect.

Sorry - the "bookmark" i used to confirm it worked, was hitting shared-testing, not my dev system.  I've verified it works, and reverted to see the same breakage.  Nice fix!


Username in airflow.usersTemplates is not used by sync_users.py

Checks

[X] I have checked for existing issues.
[X] This report is about the User-Community Airflow Helm Chart.

Chart Version
8.X.X
Kubernetes Version
Client Version: v1.26.1
Server Version: v1.23.12

Helm Version
version.BuildInfo{Version:"v3.11.1", GitCommit:"293b50c65d4d56187cd4e2f390f0ada46b4c4737", GitTreeState:"clean", GoVersion:"go1.19.5"}

Description
I used the helm chart to deploy Airflow into an Openshift cluster.
I used the docs https://github.com/airflow-helm/charts/blob/main/charts/airflow/docs/faq/security/airflow-users.md to define an admin user.
Password and e-mail address are set as defined in the provided Kubernetes secret. The username was set to ${ADMIN_USERNAME} instead of the corresponding value in the secret.
From my point of view it is either a bug or it should be documented that the username can't be set by usersTemplates.
Relevant Logs
No response
Custom Helm Values
users:
  ## define the user called "admin"
  - username: ${ADMIN_USERNAME}
    role: Admin
    firstName: Admin
    lastName: Admin
    email: ${ADMIN_EMAIL}
    ## use the ADMIN_PASSWORD template defined in `airflow.usersTemplates`
    password: ${ADMIN_PASSWORD}

## bash-like templates to be used in `airflow.users`
usersTemplates:
  ## define the `ADMIN_PASSWORD` template from the `my-secret` Secret
  ADMIN_PASSWORD:
    kind: secret
    name: airflow-admin-secret
    key: password

  ADMIN_EMAIL:
    kind: secret
    name: airflow-admin-secret
    key: email

  ADMIN_USERNAME:
    kind: secret
    name: airflow-admin-secret
    key: username

Currently, as stated in the docstring, only the password, email, firstName, lastName fields can use templates (not the username), do you think we should change this?
The main reason against allowing templates in the username is that the username is the ID field, so "changing" it actually just creates a new user, rather than updating the old user.

@robkuehl Currently, as stated in the docstring, only the password, email, firstName, lastName fields can use templates (not the username), do you think we should change this?
The main reason against allowing templates in the username is that the username is the ID field, so "changing" it actually just creates a new user, rather than updating the old user.


Build fails due to conflicting app and page files and invalid next.config.js options on Cloudflare Pages

Environment

Node.js version: 18.17.1
npm version: 9.6.7
Prisma version: 5.5.2
Next.js version: 14.0.2
@cloudflare/next-on-pages version: 1.7.2
build command: rm app/api/trpc-node/[trpc]/route.ts && npx @cloudflare/next-on-pages@1
output dir: .vercel/output/static
Compatibility flags: nodejs_compat

Description
The build process is failing due to two main issues:

Conflicting app and page files in the Next.js project structure.
Invalid next.config.js environmental variable configurations.

Conflicting Files
The build process is unable to proceed because of the following conflicting files:

pages/api/elevenlabs/speech.ts conflicts with app/api/elevenlabs/speech/route.ts
pages/api/llms/stream.ts conflicts with app/api/llms/stream/route.ts

Invalid next.config.js Options
The environmental variables are expected to be strings, but booleans were provided for several env keys in next.config.js, such as:

HAS_SERVER_DB_PRISMA
HAS_SERVER_KEYS_GOOGLE_CSE
...

Steps to Reproduce

Run the build command npm run build or trigger a build on Vercel deployment.

Expected Behavior
The build should complete successfully without any conflicting files or invalid configurations in next.config.js.
Actual Behavior
The build fails with errors related to conflicting files and invalid next.config.js options.
Additional Information

The deprecated packages warnings for rollup-plugin-inject, sourcemap-codec, debug, and uuid might also need attention, although they are not the direct cause of the build failure.
The next.config.js file should be reviewed to ensure all environmental variables are correctly set as strings.

Logs
2023-11-11T04:27:21.919598Z	Success: Finished cloning repository files
2023-11-11T04:27:22.73495Z	Detected the following tools from environment<EMAIL_ADDRESS><EMAIL_ADDRESS>2023-11-11T04:27:22.735968Z	Installing project dependencies: npm clean-install --progress=false
2023-11-11T04:27:54.80525Z
2023-11-11T04:27:54.805809Z	><EMAIL_ADDRESS>postinstall
2023-11-11T04:27:54.806004Z	> prisma generate
2023-11-11T04:27:54.806155Z
2023-11-11T04:27:55.607727Z	Prisma schema loaded from prisma/schema.prisma
2023-11-11T04:27:56.115231Z
2023-11-11T04:27:56.115686Z	✔ Generated Prisma Client (v5.5.2) to ./node_modules/@prisma/client in 85ms
2023-11-11T04:27:56.115964Z
2023-11-11T04:27:56.116176Z	Start using Prisma Client in Node.js (See: https://pris.ly/d/client)
2023-11-11T04:27:56.11645Z	```
2023-11-11T04:27:56.1167Z	import { PrismaClient } from '@prisma/client'
2023-11-11T04:27:56.116945Z	const prisma = new PrismaClient()
2023-11-11T04:27:56.117166Z	```
2023-11-11T04:27:56.117395Z	or start using Prisma Client at the edge (See: https://pris.ly/d/accelerate)
2023-11-11T04:27:56.117634Z	```
2023-11-11T04:27:56.117831Z	import { PrismaClient } from '@prisma/client/edge'
2023-11-11T04:27:56.118036Z	const prisma = new PrismaClient()
2023-11-11T04:27:56.118251Z	```
2023-11-11T04:27:56.118469Z
2023-11-11T04:27:56.118683Z	See other ways of importing Prisma Client: http://pris.ly/d/importing-client
2023-11-11T04:27:56.118892Z
2023-11-11T04:27:56.191484Z
2023-11-11T04:27:56.191882Z	added 553 packages, and audited 554 packages in 33s
2023-11-11T04:27:56.192833Z
2023-11-11T04:27:56.193139Z	205 packages are looking for funding
2023-11-11T04:27:56.193426Z	  run `npm fund` for details
2023-11-11T04:27:56.195403Z
2023-11-11T04:27:56.195774Z	found 0 vulnerabilities
2023-11-11T04:27:56.224253Z	Executing user command: rm app/api/trpc-node/[trpc]/route.ts && npx @cloudflare/next-on-pages@1
2023-11-11T04:27:58.269214Z	npm WARN exec The following package was not found and will be installed<EMAIL_ADDRESS>2023-11-11T04:28:05.606045Z	npm WARN deprecated<EMAIL_ADDRESS>This package has been deprecated and is no longer maintained. Please use @rollup/plugin-inject.
2023-11-11T04:28:05.811367Z	npm WARN deprecated<EMAIL_ADDRESS>Please use @jridgewell/sourcemap-codec instead
2023-11-11T04:28:06.441368Z	npm WARN deprecated<EMAIL_ADDRESS>Debug versions >=3.2.0 <3.2.7 || >=4 <4.3.1 have a low-severity ReDos regression when used in a Node.js environment. It is recommended you upgrade to 3.2.7 or 4.3.1. (https://github.com/visionmedia/debug/issues/797)
2023-11-11T04:28:06.516222Z	npm WARN deprecated<EMAIL_ADDRESS>Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.
2023-11-11T04:28:09.59149Z	⚡ @cloudflare/next-on-pages CLI v.1.7.2
2023-11-11T04:28:10.181944Z	⚡ Detected Package Manager: npm (9.6.7)
2023-11-11T04:28:10.182286Z	⚡ Preparing project...
2023-11-11T04:28:10.187223Z	⚡ Project is ready
2023-11-11T04:28:10.187503Z	⚡ Building project...
2023-11-11T04:28:11.485837Z	▲  npm
2023-11-11T04:28:11.486428Z	▲  WARN exec The following package was not found and will be installed<EMAIL_ADDRESS>2023-11-11T04:28:14.284344Z	▲  npm WARN deprecated<EMAIL_ADDRESS>Debug versions >=3.2.0 <3.2.7 || >=4 <4.3.1 have a low-severity ReDos regression when used in a Node.js environment. It is recommended you upgrade to 3.2.7 or 4.3.1. (https://github.com/visionmedia/debug/issues/797)
2023-11-11T04:28:14.393232Z	▲  npm
2023-11-11T04:28:14.393836Z	▲  WARN deprecated<EMAIL_ADDRESS>Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.
2023-11-11T04:28:16.021499Z	▲  Vercel CLI 32.5.3
2023-11-11T04:28:16.483437Z	▲  Installing dependencies...
2023-11-11T04:28:18.095504Z	▲  ><EMAIL_ADDRESS>postinstall
2023-11-11T04:28:18.095999Z	▲  > prisma generate
2023-11-11T04:28:18.90853Z	▲  Prisma schema loaded from prisma/schema.prisma
2023-11-11T04:28:19.412233Z	▲  ✔ Generated Prisma Client (v5.5.2) to ./node_modules/@prisma/client in 87ms
2023-11-11T04:28:19.41259Z	▲
2023-11-11T04:28:19.412828Z	▲  Start using Prisma Client in Node.js (See: https://pris.ly/d/client)
2023-11-11T04:28:19.413001Z	▲  ```
2023-11-11T04:28:19.41318Z	▲  import { PrismaClient } from '@prisma/client'
2023-11-11T04:28:19.413368Z	▲  const prisma = new PrismaClient()
2023-11-11T04:28:19.413552Z	▲  ```
2023-11-11T04:28:19.41373Z	▲  or start using Prisma Client at the edge (See: https://pris.ly/d/accelerate)
2023-11-11T04:28:19.413899Z	▲  ```
2023-11-11T04:28:19.414071Z	▲  import { PrismaClient } from '@prisma/client/edge'
2023-11-11T04:28:19.414235Z	▲  const prisma = new PrismaClient()
2023-11-11T04:28:19.414404Z	▲  ```
2023-11-11T04:28:19.414582Z	▲
2023-11-11T04:28:19.414744Z	▲  See other ways of importing Prisma Client: http://pris.ly/d/importing-client
2023-11-11T04:28:19.498824Z	▲  up to date in 2s
2023-11-11T04:28:19.499157Z	▲  205 packages are looking for funding
2023-11-11T04:28:19.49956Z	▲  run `npm fund` for details
2023-11-11T04:28:19.51362Z	▲  Detected Next.js version: 14.0.2
2023-11-11T04:28:19.519358Z	▲  Detected `package-lock.json` generated by npm 7+
2023-11-11T04:28:19.520105Z	▲  Running "npm run build"
2023-11-11T04:28:20.096409Z	▲  ><EMAIL_ADDRESS>build
2023-11-11T04:28:20.096747Z	▲  > next build
2023-11-11T04:28:20.911595Z	▲  ⚠ Invalid next.config.js options detected:
2023-11-11T04:28:20.911984Z	▲  ⚠     Expected string, received boolean at "env.HAS_SERVER_DB_PRISMA"
2023-11-11T04:28:20.912231Z	▲  ⚠     Expected string, received boolean at "env.HAS_SERVER_KEYS_GOOGLE_CSE"
2023-11-11T04:28:20.912428Z	▲  ⚠     Expected string, received boolean at "env.HAS_SERVER_KEY_ANTHROPIC"
2023-11-11T04:28:20.912763Z	▲  ⚠     Expected string, received boolean at "env.HAS_SERVER_KEY_AZURE_OPENAI"
2023-11-11T04:28:20.912965Z	▲  ⚠     Expected string, received boolean at "env.HAS_SERVER_KEY_ELEVENLABS"
2023-11-11T04:28:20.913537Z	▲  ⚠     Expected string, received boolean at "env.HAS_SERVER_HOST_OLLAMA"
2023-11-11T04:28:20.913959Z	▲  ⚠     Expected string, received boolean at "env.HAS_SERVER_KEY_OPENAI"
2023-11-11T04:28:20.914149Z	▲  ⚠     Expected string, received boolean at "env.HAS_SERVER_KEY_OPENROUTER"
2023-11-11T04:28:20.914297Z	▲  ⚠     Expected string, received boolean at "env.HAS_SERVER_KEY_PRODIA"
2023-11-11T04:28:20.914448Z	▲  ⚠ See more info here: https://nextjs.org/docs/messages/invalid-next-config
2023-11-11T04:28:20.926478Z	▲  Attention: Next.js now collects completely anonymous telemetry regarding usage.
2023-11-11T04:28:20.926789Z	▲  This information is used to shape Next.js' roadmap and prioritize features.
2023-11-11T04:28:20.927163Z	▲  You can learn more, including how to opt-out if you'd not like to participate in this anonymous program, by visiting the following URL:
2023-11-11T04:28:20.92749Z	▲  https://nextjs.org/telemetry
2023-11-11T04:28:21.06003Z	▲  ▲ Next.js 14.0.2
2023-11-11T04:28:21.060712Z	▲
2023-11-11T04:28:21.061269Z	▲  Creating an optimized production build ...
2023-11-11T04:28:21.130679Z	▲  ⨯ Conflicting app and page files were found, please remove the conflicting files to continue:
2023-11-11T04:28:21.131087Z	▲  ⨯   "pages/api/elevenlabs/speech.ts" - "app/api/elevenlabs/speech/route.ts"
2023-11-11T04:28:21.131318Z	▲  ⨯   "pages/api/llms/stream.ts" - "app/api/llms/stream/route.ts"
2023-11-11T04:28:31.78122Z	▲  Error: Command "npm run build" exited with 1
2023-11-11T04:28:31.886012Z
2023-11-11T04:28:31.886349Z	⚡ The Vercel build (`npx vercel build`) command failed. For more details see the Vercel logs above.
2023-11-11T04:28:31.886546Z	⚡ If you need help solving the issue, refer to the Vercel or Next.js documentation or their repositories.
2023-11-11T04:28:31.886705Z
2023-11-11T04:28:31.924025Z	Failed: Error while executing user command. Exited with error code: 1
2023-11-11T04:28:31.933731Z	Failed: build command exited with code: 1
2023-11-11T04:28:32.825384Z	Failed: error occurred while running build command

Yep, there were some stray files maybe. Not anymore, both issues are fixed, including the env.HAS_ ...
Now env vars are validated at runtime on the backend, and dynamically checked by the frontend.


Not compatible with the latest pandoc version

The latest pandoc version is 3.1.3, which is not compatible with the latest nbsphinx.
So, it should adapt to the pandoc over v3.0.0.

is not compatible with the latest nbsphinx.

Can you please elaborate?
I remember reading about an upper version bound enforced by nbconvert:
https://github.com/jupyter/nbconvert/issues/1996
https://github.com/jupyter/nbconvert/pull/1997
Did you try the latest nbconvert?

I am sorry that I did not attach any log.
I saw the following warning msg.
/home/koyo/.conda/envs/cherab-lhd-dev/lib/python3.11/site-packages/nbsphinx/__init__.py:1058: RuntimeWarning: You are using an unsupported version of pandoc (3.1.3).
Your version must be at least (1.12.1) but less than (3.0.0).
Refer to https://pandoc.org/installing.html.
Continuing with doubts...
  nbconvert.utils.pandoc.check_pandoc_version()

Anyway, my nbconvert version was 7.2 somehow, so updating it to 7.6 resolved this issue.
Thank you for your advice:)


Adding a composite index query to the data

Hello Bealdung,
I just completed the article on "Composite Index in SQL". I have pushed the SQL script used for the article and would love to create a Pull Request where the new branch from my end called Composite-index-branch will be merged to the main branch.
Thanks,
Temidayo.

Would appreciate it if this gets merged for my article.
Thanks!


introduce mvn wrapper

Including the wrapper scripts and jar means we distribute these artifacts. This means, we have to fulfill the license obligations (redistribute license and notice files of the original artifacts). Thus, please create a 3RD-PARTY-LICENSES  folder in the repository root, add a folder for the Maven wrapper artifacts, and put the license and notice files there. See the client or service modules for examples.
BTW; You can go ahead and make the required changes for castor (see carbynestack/castor#23) as well as a preparation for the review (FYI: @sbckr).

I added the  3RD-PARTY-LICENSES  part to this repo and also updated [carbynestack/castor#23]

Please remove sbom.xml. See carbynestack/castor#23 for a rationale.

Did that.


bugfix link to slakh.com in README

Before: www.slakh.com, redirecting to https://github.com/ethman/slakh-utils/blob/master/www.slakh.com
After: https://www.slakh.com

Thanks for catching that!


Homebrew state centralization distribution strategy

This is a meta issue for homebrew state centralization distribution strategy implementation.

[x] #26 Global state configuration
[x] #27 Global state initialization
[x] #28 Start listeners
[x] #29 Startup coordination

Fixed in #36


Worker issue for V2 large uploads

Possibly the 10gb issue is related to local uvicorn.workers.UvicornWorker timing out.
https://github.com/benoitc/gunicorn/issues/1440

do we want to close this? seems specific to mac osx docker, so not a huge deal since it goes through on linux fine?

Was just about to close the issue actually 👍🏻


Gen 4 Random Battle improvements

Fix Explosion check; the old check always blocked it on many non-Normals.
Allow mono-attacker sets with recovery + status heal, e.g. Refresh + Roost.
Various other minor updates, most notably adding hazards to more movepools.

The Travis build looks stuck. Can you restart it?


Missing "blitz" module while trying to use BNNRegressor as my regressor in Experimental Emulator

Operating System: Linux
Python version: Python 3.9.7
summit version used: Current version

Description
I am trying to use the BNNRegressor in the Experimental Emulator present in the summit/benchmarks folder. The BNNRegressor is present in the function get_bnn() in the python file experimental_emulator.py. Could you please explain to me how I should access this BNNRegressor using your:
exp = ExperimentalEmulator(model_name, domain, dataset=ds, regressor=ANNRegressor)
ANNRegressor is a class in the above mentioned python file.
However the class BNNRegressor is present in the function get_bnn() present in the above mentioned python file.
What I Did
While doing:
exp = ExperimentalEmulator(model_name, domain, dataset=ds, regressor=get_bnn())
I get the following error:

in get_bnn()
   1323 def get_bnn():
-> 1324     from blitz.modules import BayesianLinear
   1325     from blitz.utils import variational_estimator
   1327     @variational_estimator
   1328     class BNNRegressor(torch.nn.Module):

ModuleNotFoundError: No module named ‘blitz'

I was unable to find this blitz folder and wanted to ask you if it is missing in the repo or if there is an alternate way that I can use the BNNRegressor in summit.

Hi! Sorry I misread your question over email. Here, you just need to installed this package:
pip install blitz-bayesian-pytorch

However, I'd personally recommend using the ANNRegressor.


Water breaks loot drops

If water flows over a drop it breaks the head and the drop is permanently on


[7.x] Add storage function in helpers

This storage function will generate absolute path for assets placed in storage directory. It works like asset() function.

We are not accepting new helpers to the framework.

We are not accepting new helpers to the framework.

Please mention this statement in notes for contributors.


KAFKA-6791: Add CoordinatorNodeProvider in KafkaAdminClient

KAFKA-6791: Add CoordinatorNodeProvider in KafkaAdminClient
https://issues.apache.org/jira/browse/KAFKA-6791
Add CoordinatorNodeProvider interface to support batch retrieval for group coordinators.
More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

retest it please

@guozhangwang Please kindly review. Thanks

cc @hachikuji @cmccabe for reviews as well.

retest it please

@cmccabe Sorry for the late response. Okay, will cancel this PR soon.

Closed this PR since it was already fixed by KAFKA-6299.


stable/traefik: Add the acme http challenge entrypoint to the values

In the helm config the acme http challenge endpoint is hardcoded to be http. This may not work if you are behind Cloudflare for example, as Cloudflare handles the http->https redirection itself, so the http endpoint will never receive any traffic.
This change allows overriding that endpoint to be the https endpoint (or any other endpoint for that matter).

@szeber
Would you bump chart version to latest please?

@zanhsieh updated

/ok-to-test

@zanhsieh @emilevauge will this get looked at at any point or should i just close the PR?

@emilevauge
@dtomcej
@timoreimann
@vdemeester
@nmengin
@mmatur
@ldez
@Juliens
@jbdoumenjou
@geraldcroes
@SantoDE
@errm
Would you guys LGTM this PR please?
@szeber
Another version bump please. Thank you and sorry for waiting.

/assign @nmengin

/assign @ldez

/retest

/approve

/lgtm


tour: offline instructions don't work

Context: https://tour.golang.org/welcome/3
After having go installed, I have following errors:
[~] $ go tool tour
go tool: no such tool "tour"
[~] $ go get golang.org/x/tour/gotour

[~] $ gotour
-bash: gotour: command not found

# hmmm ?

[~] $ ~/go/bin/gotour
2018/08/20 12:00:06 Serving content from /Users/myhome/go/src/golang.org/x/tour

For a novice this help doesn't work, and he won't know how to fix them.

How did you install Go? Pre-built binaries from the website, or from source?

I had 3 different installations for go:

osx: brew install go (go version: go version go1.10.3 darwin/amd64)
ubuntu: apt install golang-go (go version: go version go1.6.2 linux/amd64)
freebsd: pkg install go(go version: go version go1.10.3 freebsd/amd64)

I'm afraid the go tool tour line is expected to work only if you installed from the https://golang.org/dl page linked in the sentence above, following the instructions at https://golang.org/doc/install.
Regarding the gotour error: it doesn't work because you likely didn't add GOPATH to your path, as suggested in https://golang.org/doc/code.html#GOPATH. The tour doesn't explain that, because it assumes that you didn't land there first, but you already took the time to set up a working Go environment on your machine.
I agree that this may be a source of confusion if you follow the specific path you followed (install Go from the OS package manager, then try to run the tour without setting up anything else). Keep in mind that the recommended way to install Go is from the pre-compiled binaries.
The reference issue for making the tour instructions more "robust" regardless of the path you choose for setting up Go is #8. I'd close this issue as a dup of that one.


fix #396

added TimeSpan serialization and deserialization using ISO 8601 (https://en.wikipedia.org/wiki/ISO_8601#Durations)

 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.:x: bdparrish sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

There was no real discussion on which way the conversion was intended to be processed, so I was using the best standard that would make sense, hence the ISO 8601 format.  If it makes more sense for a standard .ToString(), then that is easily doable.

@bdparrish TBH IMHO it seem wrong for a json serialization to have a dependency on xml and I would have opted for the ((TimeSpan)input).ToString("c")

((TimeSpan)input).ToString("c") produces a string that cannot be used with a TimeSpan constructor.  Am I missing a constructor overload somewhere that takes a string?
The System.Xml library is a built in .NET library and reduces the code necessary, so it all looked cleaner IMO.  I am happy to change to the .ToString("c") and add the necessary parsing for the deserialization steps.

@bdparrish You can use TimeSpan.Parse(...) for this
I'm obviously not on the team so I'd wait to hear back from someone official before you change things.  However this is the default format used with the JSON serialisers I've come across in the past.

@tdinucci - shoot man, i am just trying to help.  not on the team either.  I'll go with the TryParse().

Thanks @bdparrish - this looks perfect! I'm glad you avoided the System.Xml dependency in the end.
It's now merged.


Compat for Vanilla Races Expanded - Highmate

I think this should patch everything that needs patching so far. However, since the mod is in development - this may change.
No links to the mod as again, it's not ready yet. We don't know what ID it will have on Workshop.

Ready for merge?

Should be yeah


every organismQuantityType should have an organismQuantity

Due to some extra mappings for organismQuantityType, we are now generating records with missing organism quantity:
filter(dwc_data, organismQuantityType != "" & organismQuantity == "")
This chunk should probably be adapted to fix this:
{r map organism_quantity}
input_data %<>%
  mutate(organism_quantity = case_when(
    waarneming_type == "Vastgesteld (in m²)" |
    waarneming_type == "Vastgesteld (aantal)" |
    waarneming_type == "Secundair nest vastgesteld" ~ waarneming_kwantiteit,
    TRUE ~ ""
  ))

This blocks #114
This bug was introduced in #118 but not caught before it went into main. No big disaster so I will not roll back. But I would like it fixed before the next update to GBIF.

Fixed in 1441cc5e50f1e47531e8030757d1f8b1e0445129

Are organismQuantity == 0 always occurrenceStatus  == "absent"


Inline to code block to activate py env

This PR makes minor updates to the WSL documentation with respect to activating the Zulip python environment.
Fixes:
CZO

Screenshots and screen captures:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

@amanagr could you please take a look at this one too?

Looks good, merged, thanks @apoorvapendse!
One small comment: The PR title was a bit confusing; I would have written something longer like "docs: Improve WSL instructions around activating Python environment."


docker run should show output

After running <cli> docker run I can't see the log. The process completes right away.
Instead, I expected to see the log output and to stop the docker container with ctrl + C.
To run it in the background I'd use <cli> docker run &

Can you define the <cli> docker run & more in detail?

The & sign detaches the process from the terminal, running it in the background.
It's just one way to run processes in the background. The default should be that docker run does not run in the background. Similar to flutter run.