Redirecting an anchored URL to a different language breaks the anchor

Accessing https://manpages.debian.org/man-db/man.1#Controlling_formatted_output automatically redirects me to https://manpages.debian.org/buster/man-db/man.1.de.html#Controlling_formatted_output without jumping to the correct part of the man page (https://manpages.debian.org/buster/man-db/man.1.de.html#Steuerung_der_Ausgabeformatierung).
Ideally, the redirect would somehow land me in the correct place of the man page - for the short one of man, I can figure that out on my own, but for longer ones (bash/zshall) I can't easily do that.

That would be nice, but the hash of the URL (everything after #) is not transferred to the server. Therefore, this feature would need to be implemented client-side using JavaScript, which is generally unpopular in parts of Debian, but perhaps justifiable.
We’d also need to generate a mapping between sections in the different languages, which would need to be done heuristically.
So, this is not a quicky or easy fix, but it could be done, if someone wanted to volunteer the work.

That would be nice, but the hash of the URL (everything after #) is not transferred to the server. Therefore, this feature would need to be implemented client-side using JavaScript, which is generally unpopular in parts of Debian, but perhaps justifiable.
We’d also need to generate a mapping between sections in the different languages, which would need to be done heuristically.
So, this is not a quicky or easy fix, but it could be done, if someone wanted to volunteer the work.


Move Alist.Alist 3.32.0 to AList.AList 3.32.0

Microsoft Reviewers: Open in CodeFlow

@wingetbot run


Error while proxying request: connection reset by peer

What happened:
Mentioned in this issue on kubernetes https://github.com/kubernetes/kubernetes/issues/97125 but will give a quick rundown here:
When using kubectl to port-forward, proxy, or tail logs we will eventually get a connection reset by peer issue. This does not seem to impact inter-pod or inter-node communication, only communications via kubectl.
This issue started when I expanded and upgraded my bare-metal kubernetes cluster to 1.19.4 (was previously running 1.18.3)
I've done some deep dives into networking but this issue only seems to appear on our development machines, when doing similar tests on the master nodes themselves they never timeout or get reset. Example outputs from the api-server pod:
E1211 08:55:25.690288       1 watch.go:251] unable to encode watch object *v1.WatchEvent: write tcp <IP_ADDRESS>:6443-><IP_ADDRESS>:47194: write: connection reset by peer (&streaming.encoder{writer:(*framer.lengthDelimitedFrameWriter)(0xc0094e05a0), encoder:(*versioning.codec)(0xc020a82460), buf:(*bytes.Buffer)(0xc008d42990)})
E1211 08:55:32.886828       1 status.go:71] apiserver received an error that is not an metav1.Status: &errors.errorString{s:"context canceled"}
I1211 08:55:34.822409       1 client.go:360] parsed scheme: "passthrough"
I1211 08:55:34.822824       1 passthrough.go:48] ccResolverWrapper: sending update to cc: {[{https://<IP_ADDRESS>:2379  <nil> 0 <nil>}] <nil> <nil>}
....
I1211 08:59:40.072171       1 clientconn.go:948] ClientConn switching balancer to "pick_first"
E1211 08:59:54.321951       1 watch.go:251] unable to encode watch object *v1.WatchEvent: client disconnected (&streaming.encoder{writer:(*framer.lengthDelimitedFrameWriter)(0xc01a713fe0), encoder:(*versioning.codec)(0xc0150ccdc0), buf:(*bytes.Buffer)(0xc00a4da720)})
I1211 09:00:18.270353       1 trace.go:205] Trace[757390492]: "cacher list" type:*node.RuntimeClass (11-Dec-2020 09:00:15.270) (total time: 3000ms):
Trace[757390492]: [3.000186753s] [3.000186753s] END
error: read tcp <IP_ADDRESS>:60814-><IP_ADDRESS>:8443: read: connection reset by peer

When proxying:
$ kubectl proxy
Starting to serve on <IP_ADDRESS>:8001
E1211 09:20:54.196928    4143 proxy_server.go:144] Error while proxying request: read tcp <IP_ADDRESS>:53710-><IP_ADDRESS>:8443: read: connection reset by peer
E1211 09:21:02.176601    4143 proxy_server.go:144] Error while proxying request: read tcp <IP_ADDRESS>:53780-><IP_ADDRESS>:8443: read: connection reset by peer

When port-forwarding:
$ kubectl port-forward
Forwarding from [::1]:8001 -> 8080
E1211 09:21:04.551775    4157 portforward.go:233] lost connection to pod

What you expected to happen:
Kubectl should be able to handle invalid packets or minor disruptions (if this turns out to be a network issue)
How to reproduce it (as minimally and precisely as possible):
Run a HA cluster with 3 master nodes with 2 cpu cores and 6 gig of RAM, use keepalived for HA, deploy 10 slaves with 4 cpu and 15 gig off RAM each. All running on Proxmox on different servers in the same datacenter. Use Flannel as the pod-network.
Anything else we need to know?:
Environment:

Kubernetes client and server versions (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.4", GitCommit:"d360454c9bcd1634cf4cc52d1867af5491dc9c5f", GitTreeState:"archive", BuildDate:"2020-11-25T13:19:56Z", GoVersion:"go1.15.5", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.4", GitCommit:"d360454c9bcd1634cf4cc52d1867af5491dc9c5f", GitTreeState:"clean", BuildDate:"2020-11-11T13:09:17Z", GoVersion:"go1.15.2", Compiler:"gc", Platform:"linux/amd64"}

Cloud provider or hardware configuration: bare-metal kubernetes deployed with kubeadm on Proxmox VM's, Keepalived v2.0.19 for HA.
OS (e.g: cat /etc/os-release): Ubuntu 20.04.1 LTS, 5.4.0-56-generic kernel

/triage needs-information

hi @eddiezane, I'm helping @MicDeDuiwel with this problem. And it seems to be that the cluster works fine with kubectl commands, but anything needing a connection, like port-forward and proxy (for kubernetes dashboard) gets a sort-of time-out with a "peer reset" error. My suspicion is packets are being dropped or invalidated along the network.
It's hard to make this a feasible way to reproduce the err. Still not 100 sure what the error is and why the CLI is no longer keeping connections.
Regards

Update: We managed to find the issue, it was a result of two of our keepalived instances sharing the same id and being able to see each other on the subnet causing connection timeouts.


Fixed hyperlink for vectors in uxntal reference

On the uxntal reference page, the description for the BRK opcode contains a hyperlink to the vector description with id vector
I think this should be vectors, to reference the section on the page, of the same name, because a link above this redirects to the correct hyperlink vectors

Thanks :)

Thanks :)


[🐛] BUG with temporary filename

Description
On my iPhone, I have DIFFERENT 2 photos with the same filename. If I selected BOTH via ImagePicker I loaded only ONE. I think it happened because all selected photos ImagePicker save to TMP directory with real photo name and files reload each other. To fix this BUG I think ImagePicker must save photos with a name format like this:
UUID.extension

ping

Any solution for this issue?


Fix columns sometimes dropped in two layer operations

closes #66

Codecov Report

Merging #67 (949042a) into master (b081892) will decrease coverage by 0.84%.
The diff coverage is 77.77%.

@@            Coverage Diff             @@
##           master      #67      +/-   ##
==========================================
- Coverage   83.63%   82.78%   -0.85%
==========================================
  Files          16       16
  Lines        2688     2695       +7
==========================================
- Hits         2248     2231      -17
- Misses        440      464      +24

Impacted Files
Coverage Δ

geofileops/util/sqlite_util.py
64.63% <77.77%> (+1.87%)
:arrow_up:

geofileops/util/general_util.py
63.00% <0.00%> (-21.00%)
:arrow_down:

geofileops/util/io_util.py
75.00% <0.00%> (-6.67%)
:arrow_down:

geofileops/util/ogr_util.py
78.04% <0.00%> (-0.61%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b081892...949042a. Read the comment docs.


Docs

Modify this template as needed to best reflect your specific pull request.
Type
Please state what type of addition your pull request is:
[ ] Add App |
[ ] Add Network |
[ X ] Repository Enhancement |
[ ] Bugfix (App) |
[ ] Bugfix (Network) |
[ ] Bugfix (ncappzoo) |
[ ] Content Update (Documentation) |
Description
Please include a summary of the addition or change that you are making. If making a change to an existing application or network, make sure to state which network or app you are changing. If you're making a change to an existing network or app, don't forget to update the relevant code author documentation:
This enhancement creates the first version of the ncappzoo GitHub pages. It includes a copy of most of the documentation available in the ncappzoo in a more readable and searchable format.
Checklist

[X] I have self-reviewed my code
[X ] I have commented my code, especially in hard-to-understand areas
[X ] I have made corresponding changes to the documentation
[X ] Any dependent changes have been merged and published in downstream modules.

This also includes some small updates to existing documentation and templates:

updated documentation to include markdown in places where html tags were used.
updated issue template to change wording
updated licensing information to clarify that rights to content is held by their original owners, not by Intel or other future maintainers by including them in the app zoo.


text() returns empty string/ whitespace for multiline content

Hi,
as soon as I have elements containing line breaks, the "text()" or "ownText()" methods return only whitespace.
E.g:
<h2 class="myClass">

Some text
Another text

</h2>

String content = docTherapeut.getElementsByClass("myClass").first().text();
the content is empty.
Any idea?

Works for me: https://try.jsoup.org/~vfU9apxXRmJfPJG96F04HbwfqYY
(With pretty print on or off so it's not that.)
Can you post some an executable code snippet?


Make Samples comparable by value again

What happened?
Equality comparison between two Sample objects is passing even when they have different values.
What did you expect instead?
Two Sample objects should be considered equal only if they have the same timestamp and value.
Affected version(s)
No response
Affected part(s)
Data pipeline (part:data-pipeline)
Extra information
No response

Blocked by #148


correction of broken pathways

correction of broken pathways

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

Check please


Delete presentation directory

The directory was only needed at the final presentation of the project

Superseded by #124


Add default responses

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of “no breaking changes
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Please follow the link to find more details on PR review process.

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

Hi @erich-wang ,
I received BreakingChange error, however I didn't add any breaking changes, only added default response where it was missing.
Can you please help?

@yuvlhv , sorry for late response, I'm just back from long holiday. Adding response code is considered as breaking change, please check https://github.com/Azure/openapi-diff/blob/master/docs/rules/1011.md.
Could you please let me know whether this change is to fix bug or there's implementation change in service side?

Hi @erich-wang, this addition in to fix a bug, it's not because of a change in the service side.


Ask for toml++ v3.0.1

This will simplify https://github.com/conda-forge/bipedal-locomotion-framework-feedstock/issues/14

macOS failing because of https://github.com/actions/setup-python/issues/577


[Table] Clear all filters button shouldn't show in empty state when table empty

Describe the bug
If the table has column filters applied that clears the table, don't show the Clear All Filter buttons in the Empty State, request from @CynZhang

Remove the second Clear All Filters button

:tada: This issue has been resolved in version 2.78.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 2.79.9 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Opening settings menu with keyboard doesn't show settings

Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
Nightly (v3.0.5-master-44-gebb1e34b)
Can you reproduce the issue with our latest release version?
Yes
Can you reproduce the issue with the latest code from master?
Yes
Are you using the demo app or your own custom app?
Hosted demo
If custom app, can you reproduce the issue using our demo app?
N/A
What browser and OS are you using?
Chrome on Mac
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
N/A
What are the manifest and license server URIs?
N/A
What did you do?

Use Tab and Shift-tab to navigate to settings menu (gear icon).
Hit Enter

What did you expect to happen?
Settings menu open and shows settings.
What actually happened?
Menu opened, but is empty.  If I click the menu, it shows settings correctly.

There is probably some event other than click which needs to be used to act when the menu is opened.


Release a new version of dmidecode with a LICENSE file

In order to use this crate, I need it to include a LICENSE file. I submitted a pull request and got that merged in. Unfortunately, in order to properly download it a new version of the crate would have to be published to crates.io.
I'm happy to help with this, but AFAIK the owner of the crate must be the one to publish it.

Released v0.7.7

thank you so much!


Error executing .bat

I keep getting this error when executing the bat file
PS C:\Users\user\source\repos\GettingStartedTypeScriptDynamics
365\WebResources\spkl> .\deploy-webresources.bat
Using ''
'""' is not recognized as an internal or external command,
operable program or batch file.
Press any key to continue . . .

In case anyone else hits this issue, it happens when the project you are using spkl tasks for is not in the root folder of the solution (ie at the same level as the packages folder).
So running it for this 'Plugins' project will work fine because it is in the same folder as 'packages':

but not for the 'MySolution' project because it is one folder lower down.

To fix this, change the path in the *.bat file you are trying to run.
By default the path is set to this:
set package_root=..\..\
In the example above, changing it to this fixes the problem:
set package_root=..\..\..\
so that the task goes up one folder further to find the spkl package folder


Proto removes comments when it modifies TOML files

What version?
0.31.1
Which command?
proto pin
What happened?
If you use comments in your .prototools, then after pinning a version using proto pin or adding a plugin with proto plugin add the comments are gone. This defeats one of the niceties of using TOML. :(
Trace logs?
No response
Operating system?
MacOS
Architecture?
arm64

Yeah it's on the roadmap. I need to switch over to toml_edit on the rust side.


WIP: Introduce SwiftSetting API for controlling whether testability is enabled for a target

This introduces a new enableTesting API in SwiftSetting which allows targets to explicitly control whether testability is enabled, as some projects do not want to use the @testable feature. This can also improve debug build performance as significantly fewer symbols will be exported from the binary in the case where @testable is disabled.
The default is equivalent to enableTesting(true, .when(configuration: .debug)), so there is no behavior change from today without explicitly adopting this new API.
The --enable-testable-imports/--disable-testable-imports command line flags now acts as overrides -- if specified, they will override any build settings configured at the target level, and if unspecified, the target-level settings will be respected.

Untested draft. Will write up a proposal soon.

Thanks! Yeah, really want to get some wide community feedback on this. Pretty much everyone fights with enable testing at some point or other.

pitch: https://forums.swift.org/t/pitch-swiftpm-testable-build-setting/75084
proposal: https://github.com/swiftlang/swift-evolution/pull/2580


Adding via bib or doi

Codecov Report
Base: 90.61% // Head: 91.48% // Increases project coverage by +0.87% :tada:

Coverage data is based on head (2300baf) compared to base (13700e8).
Patch coverage: 85.50% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #35      +/-   ##
==========================================
+ Coverage   90.61%   91.48%   +0.87%
==========================================
  Files           8        8
  Lines         341      376      +35
  Branches       44       47       +3
==========================================
+ Hits          309      344      +35
+ Misses         25       23       -2
- Partials        7        9       +2

Impacted Files
Coverage Δ

src/routes/reference.py
89.55% <78.78%> (+8.15%)
:arrow_up:

src/services/reference.py
95.65% <91.66%> (-0.90%)
:arrow_down:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Local Development with Glue using Lando

Closes #78

sorry, I just saw you had requested review.  my github alerts are a mess


produce NaN when running UMAP with min_dist=0.1

Hi,
I encountered a wired problem when I run UMAP with min_dist=0.1
sc.tl.pca(adata_f,n_comps=250)
adata_f.obsm['X_pca'] *= -1
sc.pp.neighbors(adata_f, n_neighbors=10)
scv.pp.moments(adata_f,renormalize=True,mode='connectivities')
sc.tl.umap(adata_f,min_dist=0.1)
error is about produce NaN, and then I checked adata_f.obs['X_umap'], all NaN in array.
However, when I use min_dist=0.2, everything seems well.
Could you help me to figure it out? Thank you~

Sorry, this seems to be a UMAP issue. To really dig down to this, you'd probably run this using the UMAP package and submit a bug report there.

 I imagine that you'll get a weird output from that...

the umap result of 250 PCs and min_dist=0.2 seems ok, and I will check the source of umap.
Thank you~


Omit ruff config from wheel distributions

https://github.com/pypa/setuptools/blob/main/setuptools/_vendor/ruff.toml is currently being included in wheels, which is probably unintended since it's only relevant to maintainers.

Thank you very much @carsonyl.
I believe it to be important to distribute ruff.toml in the sdist as it contains configurations important for tests (pytest-ruff).

The reason why I am waiting to see the review of other maintainers in this PR, is because previously we had a decision to simply install all the files anyway (since we have extra complexity for configuration and it is not like the presence of these files are hindering the functioning of setuptools): https://github.com/pypa/setuptools/pull/4479.

Thanks. This issue only came to my attention because I was working with an application that assumed *.toml was for its exclusive use, so it tripped up on the presence of this file.

I was working with an application that assumed *.toml was for its exclusive use

That sounds like a bug in that application.
That said, I don't feel strongly about this setting.


Fix menu action selection not being shared between center and RHS

Summary
Follow-up to something I broke with the fix for MM-12982. After that fix, choosing an option in the center channel would not reflect the option if that post was open in the RHS or vice versa. This fixes it.

cherry picker 71eefcf75c32dee93a57eb83a43d37c93b08de87 into release 5.5


kernel: sleep: kernel.common.timing.minimallibc fails

Describe the bug
The test kernel.common.timing.minimallibc fails.
Observed for nrf5340dk_nrf5340_cpunet.
To Reproduce
Steps to reproduce the behavior:

have nrf5340dk_nrf5340 connected
go to your zephyr dir
call ./scripts/twister -T tests/kernel/sleep -p nrf5340dk_nrf5340_cpunet --device-testing --device-serial /dev/ttyACM1 -v --inline-logs
See console output with error

Expected behavior
Valid console output
Impact
Not clear
Logs and console output
*** Booting Zephyr OS build zephyr-v3.5.0-4014-g0d7d39d44172 ***
Running TESTSUITE sleep
===================================================================
START - test_sleep
PASS - test_sleep in 2.006 seconds
===================================================================
START - test_sleep_forever
PASS - test_sleep_forever in 0.001 seconds
===================================================================
START - test_usleep
Assertion failed at WEST_TOPDIR/zephyr/tests/kernel/sleep/src/usleep.c:93: sleep_test_usleep: (elapsed_ms <= UPPER_BOUND_MS is false)
overslept
FAIL - test_usleep in 30.411 seconds
===================================================================
TESTSUITE sleep failed.
------ TESTSUITE SUMMARY START ------
SUITE FAIL -  66.67% [sleep]: pass = 2, fail = 1, skip = 0, total = 3 duration = 32.418 seconds
- PASS - [sleep.test_sleep] duration = 2.006 seconds
- PASS - [sleep.test_sleep_forever] duration = 0.001 seconds
- FAIL - [sleep.test_usleep] duration = 30.411 seconds
------ TESTSUITE SUMMARY END ------
===================================================================
RunID: bdc079553fb65c3b828ec69c58c3e5d4
PROJECT EXECUTION FAILED

Environment (please complete the following information):

OS: Ubuntu 20.04.1 LTS
Toolchain Zephyr SDK 0.16.1
Commit SHA or Version used: zephyr-v3.5.0-4014-g0d7d39d44172

no longer present since zephyr-v3.5.0-4395-ga1d33f

The issue occurred once again on the toolchain Toolchain Zephyr SDK 0.16.5, on the version zephyr-v3.5.0-5140-g055ac615427f.


global/session/workflow level parameter and artifacts

It can be irritating to pass parameters/artifacts around templates as arguments. One way to make this easier in our YAMLs is to provide the notion of global/session/workflow level parameters and artifacts. For example, one idea is that all top level parameters/artifacts at the workflow.spec.arguments level could be referred to via a special variable (e.g. {{workflow.parameters.revision}}). This would ease the burden of passing these around to templates who all want the same information.

This is merged with https://github.com/argoproj/argo/pull/571


get grayscaled image

Could anybody give me some fast hint about how to extract a grayscale image from any depth camera callback?
Thanks in advance!

Check out this demo code which extract both amplitude (grayscale) image and depth image.
https://github.com/3dtof/DemoApplications/tree/master/TinTin/people_tracking


versioning.mdx, releases-oss.mdx, delta-resources.mdx template review complete

adding versioning-oss.mdx instead of versioning, so closed out pr


Fix linkcall registry bugs

Types of changes

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Proposed changes
While developing the a new (similar) node i noticed 2 issues in the code...

attempt to re-assign variable declared as const
attempt to access property of undeclared variable

Other changes are mearly linting suggestions applied to the registry closure.
Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[x] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage decreased (-0.01%) to 68.298% when pulling 5c69599e78f68b09a7376656ee96731a2ee3804b on Steve-Mcl:fix-linkcall-registry into e14dd06a94040e26c1bf9b91dac932a153655dbb on node-red:master.


dependency issue using localAPI in a sveltekit application

Link to reproduction
https://sveltekit-payload-frontend.vercel.app/
Describe the Bug
I get some dependency issues when using the localAPI in a Sveltekit application. I tried using it like so:
// payload-client.ts
import { type BasePayload, getPayload, type GeneratedTypes } from 'payload';
import { importConfig } from 'payload/node';
import 'dotenv/config';
import path from 'node:path';
import { fileURLToPath } from 'node:url';

let payloadInstance: BasePayload<GeneratedTypes>;

// payload-client.ts
export async function initializePayload() {
	const __dirname = path.dirname(fileURLToPath(import.meta.url));
	const payloadConfigPath = path.join(__dirname, './payload/payload.config.ts');
	const awaitedConfig = await importConfig(payloadConfigPath);

	payloadInstance = await getPayload({ config: awaitedConfig });
}

export async function getPayloadInstance() {
	if (!payloadInstance) {
		throw new Error('Payload has not been initialized. Call initializePayload first.');
	}

	return payloadInstance;
}

// +page.server.ts
import { getPayloadInstance, initializePayload } from '$lib/payload-client';

export async function load() {
	await initializePayload();
	const payloadInstance = await getPayloadInstance();
	const user = await payloadInstance.findByID({
		collection: 'users',
		id: '1'
	});

	return { msg: 'welcome', user: user };
}

Running it locally works just fine, both using pnpm dev and making a production build with pnpm build pnpm serve... however deploying it to vercel gives the following error:
Error [ERR_MODULE_NOT_FOUND]: Cannot find module '/var/task/node_modules/.pnpm/payload@3.0.0-beta.35_@swc+core@1.5.7_@swc+helpers@0.5.11__@swc+types@0.1.7_graphql<EMAIL_ADDRESS>imported from /var/task/.svelte-kit/output/server/entries/pages/payload/payload.config.ts
    at finalizeResolution (node:internal/modules/esm/resolve:269:11)
    at moduleResolve (node:internal/modules/esm/resolve:937:10)
    at moduleResolveWithNodePath (node:internal/modules/esm/resolve:1161:14)
    at defaultResolve (node:internal/modules/esm/resolve:1204:79)
    at nextResolve (node:internal/modules/esm/hooks:866:28)
    at Hooks.resolve (node:internal/modules/esm/hooks:304:30)
    at ModuleLoader.resolve (node:internal/modules/esm/loader:352:35)
    at ModuleLoader.getModuleJob (node:internal/modules/esm/loader:234:38)
    at ModuleLoader.import (node:internal/modules/esm/loader:322:34)
    at Hooks.register (node:internal/modules/esm/hooks:165:51) {
  code: 'ERR_MODULE_NOT_FOUND',
  url: 'file:///var/task/node_modules/.pnpm/payload@3.0.0-beta.35_@swc+core@1.5.7_@swc+helpers@0.5.11__@swc+types@0.1.7_graphql<EMAIL_ADDRESS>}

I tried both inside a turborepo and in a separate project without turborepo. Both failed with similar errors. The error above is from the non-turborepo version, and this error is from the turborepo:
Error [ERR_MODULE_NOT_FOUND]: Cannot find module '/var/task/node_modules/.pnpm/payload@3.0.0-beta.32_@swc+core@1.5.7_@swc+types@0.1.7_graphql<EMAIL_ADDRESS>imported from /var/task/apps/web/.svelte-kit/output/cms/src/payload.config.ts
    at finalizeResolution (node:internal/modules/esm/resolve:269:11)
    at moduleResolve (node:internal/modules/esm/resolve:937:10)
    at moduleResolveWithNodePath (node:internal/modules/esm/resolve:1161:14)
    at defaultResolve (node:internal/modules/esm/resolve:1204:79)
    at nextResolve (node:internal/modules/esm/hooks:866:28)
    at Hooks.resolve (node:internal/modules/esm/hooks:304:30)
    at ModuleLoader.resolve (node:internal/modules/esm/loader:352:35)
    at ModuleLoader.getModuleJob (node:internal/modules/esm/loader:234:38)
    at ModuleLoader.import (node:internal/modules/esm/loader:322:34)
    at Hooks.register (node:internal/modules/esm/hooks:165:51) {
  code: 'ERR_MODULE_NOT_FOUND',
  url: 'file:///var/task/node_modules/.pnpm/payload@3.0.0-beta.32_@swc+core@1.5.7_@swc+types@0.1.7_graphql<EMAIL_ADDRESS>}

Before I opened this issue I discussed it on Discord. The linked discord thread can be found here: https://discord.com/channels/967097582721572934/1238608849099948212
To Reproduce
In order to test the turborepo version:

Fork my repo: https://github.com/Snailedlt/svelte-payload-example
Pull down the fork locally
Copy the .env.example file cp .env.example .env
Run pnpm i and pnpm dev to start the cms and web projects. Create a new user in Payload (http://localhost:3000)
Go to the frontend (http://localhost:5173) and notice how your email is displayed at the bottom of the page similar to this:

If you wanna test the production build you can run pnpm build and pnpm start. This should work the same if done locally.
Now that we know it works locally we can test in vercel, but first we need a database. Create a hosted postgres database. I use neon for this: https://neon.tech/ (login link: https://console.neon.tech/app/projects)
Deploy both the cms app and the web app as two separate vercel deployments.

Here's how the cms app should be deployed:

Replace the DATABASE_URI env with your connection string. If you're using neon that can be found in your dashboard. Make sure it's a pooled connection:

Here's how the web app should be deployed:

Remember to add the env vars here too.

Once it's all deployed it should all be green, but if you visit the url for your frontend (web), it should show a 500 internal error. And if you go to the logs in vercel you should be able to see the same error message as described in this issue.

If you want to test the simplified non-turborepo version:

Follow all the steps from the turborepo version
fork my repo: https://github.com/Snailedlt/sveltekit-payload-frontend
If you wanna test it locally it's just cp .env.example .env,  pnpm i and pnpm dev
deploy it to vercel using the default SvelteKit (v1) settings. Use the same DATABASE_URI as in the turborepo example
Go to the deployed website and notice that you get a 500 error here too. If you go to logs in vercel should see the same errors as shown in this issue.

Payload Version
beta (3.0.0-beta.34 as of the time this issue was posted)
Adapters and Plugins
db-postgres, lexical

@Snailedlt my gut tells me that this is a limitation from Vercel. If you check out their docs related to root directory, they say that you cannot access files outside the root of your app.
It looks like you are trying to go up and out of your web app root dir and into another folder within your mono repo in this file.
Could you try copying the payload config into your web app directory and adjust your imports to go to that config file to test this theory?

@JarrodMFlesch Thanks for the response :)

they say that you cannot access files outside the root of your app.

Looks like it should have access to the files outside of the root:

I actually got an error about the file path to the config being wrong. After correcting it, it worked even though  the file path is outside of the root of the app.

Could you try copying the payload config into your web app directory and adjust your imports to go to that config file to test this theory?

This is what I've done in the other repo I linked to at the bottom of the "To reproduce" section:

If you want to test the simplified non-turborepo version:
Follow all the steps from the turborepo version
fork my repo: https://github.com/Snailedlt/sveltekit-payload-frontend
If you wanna test it locally it's just cp .env.example .env, pnpm i and pnpm dev
deploy it to vercel using the default SvelteKit (v1) settings. Use the same DATABASE_URI as in the turborepo example
Go to the deployed website and notice that you get a 500 error here too. If you go to logs in vercel should see the same errors as shown in this issue.

@AlessioGr let me know if you need anything, whether it be clarification or help testing.

Fixed by https://github.com/payloadcms/payload/pull/7620

@AlessioGr I tried redeploying now, but I get the same 500 error that I did before.

This is my package.json files for the web and cms in the svelte-payload-example repo
Do I need to update something in my dependencies, or somewhere else?


Improve context awareness 🪬

User story

As a User.
I want the bot to be aware of everything said in the current conversation.
So complex communication or changes of request will be supported.

Acceptance criteria

The bot is context-aware.
There is no degradation of the bot's performance compared to when the feature is off
Example of degradation: Code not being outputted / the outputted code is lacking.

Definition of done (DoD)
Feature DoD:

Code review has been completed and code has been merged.
User interaction tests pass on all major browsers.

Sprint Release DoD:

Project builds, deploys, and tests successfully.

Project Release Definition of Done

User interaction tests pass on all major browsers.
Design documentation has been updated

DoD general criteria

Feature has been fully implemented
Feature has been merged into the mainline
All acceptance criteria were met
Product owner approved features
All tests are passing
Developers agreed to release

Est Rem = 5


Vite 4.3.9 compatibility

Using<EMAIL_ADDRESS>with following config, I get TypeRrror.

import preact from '@preact/preset-vite'
import { defineConfig } from 'vite'
import inlineSource from 'vite-plugin-inline-source'

// https://vitejs.dev/config/
export default defineConfig({
  plugins: [
    preact(),
    inlineSource(),
  ],
  server: {
    host: true,
  },
})

Providing an option:
inlineSource({
  replaceTags: ['style'],
}),

Throws another error, when running pnpm dev:
failed to load config from D:\___projects\tests\comps\vite.config.ts
error when starting dev server:
TypeError: inlineSource is not a function
    at file:///D:/___projects/tests/comps/vite.config.ts.timestamp-1687956498822-c3b6e3b189067.mjs:22:5
    at ModuleJob.run (node:internal/modules/esm/module_job:194:25)
 ELIFECYCLE  Command failed with exit code 1.

Sorry I am just now seeing this :/ the typings on options need adjusted slightly for that first error, and the second one probably just needed inlineSource.default() due to the way tsc builds things for cjs/esm support.
#274 / v2.3.0 should fix both - added a different build process which produces separate cjs/esm builds and fixed that options type.


Support PyQt 5.6

Fixes #3073
Fixes #3145

TODO:

[x] Review all Qt slots to see if they have the right signatures (PyQt 5.6 fails with the wrong signatures).
[x] Fix loading js and css files from local computer (WebEngine refuses to do it).

Pinging @stonebig and @Nodd about this one.

I just launched spyder successfully. I'll let you know if there are any problems when using it.

a picture is worth a thousand words:

Ok, this is ready! I'll merge it after I fix our Travis and AppVeyor integrations :-)

Testing further, the help onobject is not working well, the "top_right' help aera doesn't work:

I have no welcome message over it explaining to do "Ctrl+I"
trying to type "numpy" on the object, I loss focus after each character is typed.
crtl+I on the editor feed that obhect field but nothiong happen.

@stonebig, I think the problem you're seeing is caused because the PyQt5 wheel is packaged incorrectly (at least that's what I discovered on Linux). WebEngine (the new and shiny Qt web backend) fails because it needs some files that are placed in another directory of the wheel.
However, I need to merge this one to keep testing things, so please open a new issue about your problem and I'll try to look at it when I have some time :-)

Hi @ccordoba12 ,
Could you be specific about "it needs files that are placed in another directory of the wheel" ?
The PyQt maintainer may fix it, if enough details are provided
https://www.riverbankcomputing.com/pipermail/pyqt/2016-June/037636.html

I'll check it out in a couple of hours on Windows.
On Linux I needed to move all files in
site-packages/PyQt5/Qt/resources

to
site-packages/PyQt5/Qt/libexec

so that they are placed next to the QWebEngineProcess executable.
Maybe you need to do the same on Windows :-)

can you give me the list of files in this site-packages/PyQt5/Qt/resources ? I don't have the same directory but I should have the same detailed files.

I copy:

what is in python-3.5.1\Lib\site-packages\PyQt5\Qt\resources\
in python-3.5.1\Lib\site-packages\PyQt5\

icudtl.dat
qtwebengine_resources.pak
qtwebengine_resources_100p.pak
qtwebengine_resources_200p.pak

then:

First spyder start (loooong) then nothing, have to kill "python*." and "spyder.exe" process
second "spyder reset"
third "spyder" ...... and then I get locked out spyder, but the first "web" window did display.

Those are the exact files I copied next to QtWebEngineProcess. Could you post a screenshot of what you have now?

spyder hangs in this state

with these processes:

I have this problem too (should I open a new issue?):
I'm running:
➜  ~ conda create -n testenv python=3 spyder
➜  ~ . activate testenv
(testenv) ➜  ~ spyder
Traceback (most recent call last):
  File "/home/fernando/anaconda2/envs/testenv/lib/python3.6/site-packages/qtpy/QtWebEngineWidgets.py", line 22, in <module>
    from PyQt5.QtWebEngineWidgets import QWebEnginePage
ModuleNotFoundError: No module named 'PyQt5.QtWebEngineWidgets'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/fernando/anaconda2/envs/testenv/bin/spyder", line 6, in <module>
    sys.exit(spyder.app.start.main())
  File "/home/fernando/anaconda2/envs/testenv/lib/python3.6/site-packages/spyder/app/start.py", line 103, in main
    from spyder.app import mainwindow
  File "/home/fernando/anaconda2/envs/testenv/lib/python3.6/site-packages/spyder/app/mainwindow.py", line 92, in <module>
    from qtpy import QtWebEngineWidgets  # analysis:ignore
  File "/home/fernando/anaconda2/envs/testenv/lib/python3.6/site-packages/qtpy/QtWebEngineWidgets.py", line 26, in <module>
    from PyQt5.QtWebKitWidgets import QWebPage as QWebEnginePage
ModuleNotFoundError: No module named 'PyQt5.QtWebKitWidgets'

My system:
➜  ~ uname -a
Distributor ID:	Ubuntu
Description:	Ubuntu 16.04.2 LTS
Release:	16.04
Codename:	xenial

I can't find the directory you mentioned:
➜  ~ ls anaconda2/envs/testenv/lib/python3.6/site-packages/PyQt5/
__init__.py                     QtBluetooth.pyi  QtGui.pyi                QtNetwork.pyi       Qt.pyi              QtSql.pyi         QtWebSockets.pyi   QtXml.pyi
__pycache__                     QtBluetooth.so   QtGui.so                 QtNetwork.so        QtQml.pyi           QtSql.so          QtWebSockets.so    QtXml.so
_QOpenGLFunctions_2_0.pyi       QtCore.pyi       QtHelp.pyi               QtNfc.pyi           QtQml.so            QtSvg.pyi         QtWidgets.pyi      uic
_QOpenGLFunctions_2_0.so        QtCore.so        QtHelp.so                QtNfc.so            QtQuick.pyi         QtSvg.so          QtWidgets.so
_QOpenGLFunctions_2_1.pyi       QtDBus.pyi       QtMultimedia.pyi         QtOpenGL.pyi        QtQuick.so          QtTest.pyi        QtX11Extras.pyi
_QOpenGLFunctions_2_1.so        QtDBus.so        QtMultimedia.so          QtOpenGL.so         QtQuickWidgets.pyi  QtTest.so         QtX11Extras.so
_QOpenGLFunctions_4_1_Core.pyi  QtDesigner.pyi   QtMultimediaWidgets.pyi  QtPrintSupport.pyi  QtQuickWidgets.so   QtWebChannel.pyi  QtXmlPatterns.pyi
_QOpenGLFunctions_4_1_Core.so   QtDesigner.so    QtMultimediaWidgets.so   QtPrintSupport.so   Qt.so               QtWebChannel.so   QtXmlPatterns.so


joint inference on 4 nodes

Use Garbo.jl to run joint inference on 2 or 4 supercomputer nodes. It's unclear whether we want the mulitnode distribution of work to look more like sky division parallelism (implemented in ParallelRun.jl) or source division parallelism (implemented in source_division_inference.jl on the src_div_par branch). Each had some issues, and neither seems exactly compatible with joint inference. But in either case, each node has ownership of some light sources. Optimized parameters are stored in a global array now. We probably need another global array to synchronize optimization across nodes.
@kpamnany , @agnusmaximus -- Let's discuss this during the conference call tomorrow, or right afterwards.

Joint inference will apply to how many light sources concurrently? I assume there'll be a maximum cap of some sort?

Ideally there's no maximum. Any sources that overlap with each other would be optimized jointly. Jointly doesn't have to mean "concurrently" though. Just that the optimal value for one light source's parameters depend on the other light source's parameters.
We should probably do a walkthrough of Max's code sometime soon.

It's no longer necessary for Cyclades manage work for multiple nodes with the red-black approach to node-level parallelism.


Hotfix/Operator Test  JS bug

Describe Changes
Fixed bug on JS side that causes 2 operator tests to fail randomly.
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] Code styles have been enforced
[x] All Hardhat tests are passing

Sorry prettier:fix applied some "improvements" to the *.md files.


Callback of setState

Only one callback will be executed with multiple setState before batch update. Add a callback array for dirtyInstance to save all callbacks.

Fix at commit d397b98


libfido2: 1.10.0 -> 1.11.0 and refactor to enable NFC support via pcsc

Description of changes
libfido2 supports nfc via pcsc, which could be used to let ssh access a yubikey through an NFC reader. This PR refactors dbus, polkit, python-dbusmock, bluez and pcsclite to remove a dependency loop on systemd, and updates libfido2 to the latest version.
Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

I actually can't reproduce this ofborg failure, nix-build -A aws-sdk-cpp builds successfully for me.

also requesting review by @prusnak and @dtzWill, maintainers of libfido2

@GrahamcOfBorg eval

@jtojnar can you please review freedesktop stuff? (changes to dbus+polkit)

I can't reproduce this failure, so I'm going to request a re-evaluation.
@GrahamcOfBorg eval

Excellent, looks like the only thing left is failure to build on darwin. I'm not sure why bison is being built or timing out.

Weird, ofborg is hanging but the logs look fine?

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1008

Please drop all the merge commits from this PR and rebase instead.

3b9048a3cefb6e86af60348513624ea691cd7f41 broke cross-compilation because recursivePthLoader (which is only used for tests) doesn't cross-compile. Maybe that's the reason why it was conditional on doCheck? Now it's introduced in the closure for bluez and thus breaks a lot of downstream deps.
I can check if recursivePthLoader can be made to cross-compile properly, otherwise maybe we should revert that change that makes the test output unconditional?
cc @Artturin, maybe you have an idea?
patching sources
updateAutotoolsGnuConfigScriptsPhase
configuring
no configure script, doing nothing
building
/nix/store/4f9yfmxslh79x3v3zdnxy4hb2k1hw6zb-stdenv-linux/setup: line 1397: /nix/store/p3v2zvs7mgrf22c0wqv3vjjqvw1va9n6-python3-aarch64-unknown-linux-gnu-3.10.6/bin/python3.10: cannot execute binary file: Exec format error

Seems possibly an easy fix but out of time for today :)

3b9048a3cefb6e86af60348513624ea691cd7f41 broke cross-compilation because recursivePthLoader (which is only used for tests) doesn't cross-compile. Maybe that's the reason why it was conditional on doCheck? Now it's introduced in the closure for bluez and thus breaks a lot of downstream deps.
I can check if recursivePthLoader can be made to cross-compile properly, otherwise maybe we should revert that change that makes the test output unconditional?
cc @Artturin, maybe you have an idea?
patching sources
updateAutotoolsGnuConfigScriptsPhase
configuring
no configure script, doing nothing
building
/nix/store/4f9yfmxslh79x3v3zdnxy4hb2k1hw6zb-stdenv-linux/setup: line 1397: /nix/store/p3v2zvs7mgrf22c0wqv3vjjqvw1va9n6-python3-aarch64-unknown-linux-gnu-3.10.6/bin/python3.10: cannot execute binary file: Exec format error

Seems possibly an easy fix but out of time for today :)

https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/python-modules/recursive-pth-loader/default.nix
It's using ${python} which is for host
It should be using ${python.pythonForBuild}

See https://github.com/NixOS/nixpkgs/pull/185635 for someone working on a solution


Doublejumping while bhopping

Fedoraware Version Date: 6.07.2022
Config used:

https://files.catbox.moe/t7xckc.fw
Description:

I've noticed that bhop can often make you doublejump as scout
and its very annoying
Steps to reproduce:

[X] I am able to consistently reproduce this error

Steps to reproduce the behavior:

Enable AutoJump and directional AutoStrafe
Try to bhop with autostrafe
Notice that you cant doublejump midair sometimes because autojump makes you doublejump on the ground

Additional context

This happens very often while you strafe sideway

[X] I have included the steps to reproduce to the best I am able to share.

Well I can't say I ever experience this unless I'm lagging hard

it happens even if my fps is about 89

And it happens "anywhere"?

yes

Do you have any other cheats you could try

i haven't experienced this with other cheats even when i lagged, only with fedoraware

I'm trying to rule out the possibility that your keyboard is wacky

Since I genuinely haven't experienced this other than when I'm lagging really really badly

Do you have a strange config/bind to jump in base tf2?

Hmm idk but even when i had 15 fps with cathook or lmaobox i havent seen this problem
and i dont have jump binds other than default

i dont have auto fan jump enabled and i cant record because ill have bad fps then

89 fps is already bad.
i cant figure out what is causing your issue, there is nothing i know about that would cause anything even close to your issue, and clearly, neither can mfed.
show the issue happening, or figure it out yourself.

ok ill try to find what causes it

89 fps is already bad.
i cant figure out what is causing your issue, there is nothing i know about that would cause anything even close to your issue, and clearly, neither can mfed.
show the issue happening, or figure it out yourself.

I'm confused about this but 89 FPS is more than enough to not mess with bhop still

Closing cos still havent experienced this


How to evaluate the Road Lane Markings?

Hi,
I'd like to know how to evaluate the performance of lane line extraction.
Since the Lane marking contains the following categories:

lane/crosswalk
lane/double other
lane/double white
lane/double yellow
lane/road curb
lane/single other
lane/single white
lane/single yellow

And I can't find the categories in the semantic segmentation evaluation script.
Best regards,
Jin

@haopo2005  did you use the lane line marking extraction for semantic segmentation ? in the dataset i am seeing that their are only single line marking on the image


Watcher broken for empty cache

I’m subscribing for cache changes:
apolloClient.query(query).responseFetcher(ApolloResponseFetchers.CACHE_ONLY).watcher()
and then update cache:
apolloClient.query(query).responseFetcher(ApolloResponseFetchers.NETWORK_ONLY)
unfortunately watcher do not actually watch for cache changes and load data from updated cache.  Is it proper behaviour or bug?
But if I'm recreating watcher it grabs cached data. But only after recreation =(
Important note: I’m fetching full collection of objects not several keys
I think it is happening because in CacheOnlyFetcher there is no dependant keys are provided  in cacheMissResponse() method

When cache is empty this apolloClient.query(query).responseFetcher(ApolloResponseFetchers.CACHE_ONLY).watcher() doesn't make any sense, as internally it won't be able to resolve dependant keys to watch.
I'm guessing you want to register local cache watcher much earlier then actual fetching from network?

Is it possible to check that cache is empty to trigger network request manually in that case?


Do we have NextJs @12.X.X version support for this Module Federation Plugin?

Bug report

What is the current behavior?
Plugin: Webpack Module Federation
Next JS Version: 12.1.6
Host App : Build and stand alone App is working as expected
Micro App: Here's the issue, Remote entry file that is generated during the build process does not contain the exposed modules. Below is the sample of Remote Entry File.
"use strict"; var moduleMap = { 	"./blogPage": function() { 		var e = new Error("Cannot find module './templates/BlogPage'"); e.code = 'MODULE_NOT_FOUND'; throw e; 	}, 	"./articlePage": function() { 		var e = new Error("Cannot find module './templates/ArticlePage/ArticlePage'"); e.code = 'MODULE_NOT_FOUND'; throw e; 	}, 	"./tagPage": function() { 		var e = new Error("Cannot find module './templates/TagPage'"); e.code = 'MODULE_NOT_FOUND'; throw e; 	}, }
Error Observed: Cannot Find Module
Where is with NextJS 11.X.X Version the remote entry file is loading all the components correctly which are exposed by the microapp.
What is the expected behavior?
Remote Entry File generated after the build should contain successfully loading of the modules which are been exposed by the micro app in Microfrontend Architecture.
Other relevant information:
webpack version:  5.66.0
Node.js version: 16.13.1
Operating System:  Windows 10
Additional tools: NA

MODULE_NOT_FOUND means there is no such module. you probably get more informative error in stdout..

Maybe you have change directory structure and now there is no such modules?


Numbered lists don't align well with paragraph text

Numbered (ordered) lists in body copy on Drought site pages are not aligning with paragraphs, but start a little to the left of them.
We had an example of this on the Drought assistance page, but that list got deleted. You'll have to create an example on a sample page to see it.
This is not urgent, but should be fixed before we need another numbered list.

Possible fix in cagov/design-system#146.


Add Yet Another Palindrome Problem Code

https://www.codechef.com/submit/ARRPAL

@Sahiljawale can you assign me here
I would like to provide neat and clean solution to this problem under hacktoberfest '22

Can i do it under hacktoberfest-2022?


Fixed importing SPC music pack

Fixed the issue where importing an SPC wouldn't generate a new music pack for the selected track.
Previously the user would have to make a change on the tracker tab for a pack to be generated, but now it gets generated on import.
See issue #42

This is ready to merge in, right? My friend BinaryCounter wants to use EBMusEd for another upcoming demoscene project and I'd like for this change to be in place.


Inline Webviewer Integration

It would be great if this could work in conjunction with Inline Webviewer (https://foundryvtt.com/packages/inlinewebviewer) to allow webpages to render inside of the GM screen widgets!

It totally does, this is 100% on me for not understanding that feature of the other module.


Fix outdated client cacheTime docs

The default client cache for the client appears to be 60 seconds - not 10 seconds as documented in the client options.
Here is the definition in the client:
https://github.com/medplum/medplum/blob/30050b411dbd58b52733e61e14e03abedc2d5873/packages/core/src/client.ts#L86
It appears that the underlying value was changed in this PR without the associated docs being updated. (cc @codyebberson)

Thanks @josh- !


calico in kube-proxy ipvs mode don't access kubernetes svc.

Current Behavior

[root@k8s-1 ~]# kubectl version --short
Client Version: v1.16.6-beta.0
Server Version: v1.16.6-beta.0
[root@k8s-1 ~]# kubectl get svc kubernetes
NAME         TYPE        CLUSTER-IP   EXTERNAL-IP   PORT(S)   AGE
kubernetes   ClusterIP   <IP_ADDRESS>   <none>        443/TCP   274d
[root@k8s-1 ~]# kubectl get endpoints kubernetes
NAME         ENDPOINTS                                                  AGE
kubernetes   <IP_ADDRESS>:6443,<IP_ADDRESS>:6443,<IP_ADDRESS>:6443   274d

[root@k8s-6 kubernetes]# ipvsadm -S -n |grep <IP_ADDRESS>
-A -t <IP_ADDRESS>:443 -s rr
-a -t <IP_ADDRESS>:443 -r <IP_ADDRESS>:6443 -m -w 1
-a -t <IP_ADDRESS>:443 -r <IP_ADDRESS>:6443 -m -w 1
-a -t <IP_ADDRESS>:443 -r <IP_ADDRESS>:6443 -m -w 1

[root@k8s-4 ~]# ipvsadm -Lnc
IPVS connection entries
pro expire state       source             virtual            destination
TCP 00:51  SYN_RECV    <IP_ADDRESS>:37954   <IP_ADDRESS>:443     <IP_ADDRESS>:6443

when i set kube-proxy use ipvs mode, i cannot access kubernetes svc on every node。
It stays in SYN_RECV  state.
Possible Solution

when I delete the following route entry, I can access it normally。
[root@k8s-6 ~]# ip rou del local <IP_ADDRESS> dev kube-ipvs0 proto kernel scope host src <IP_ADDRESS>
# <IP_ADDRESS> is ip of tunl0
[root@k8s-6 ~]# ip rou add local <IP_ADDRESS> dev kube-ipvs0 proto kernel scope host src <IP_ADDRESS>

Your Environment

Calico version : 3.12.0 (3.11.1)
Orchestrator version (e.g. kubernetes, mesos, rkt):  kubernetes-v1.16.5(v1.15.8, v1.14.10)
Operating System and version: CentOs-7.7
kube-proxy mode: ipvs

ip rou del local <IP_ADDRESS> dev kube-ipvs0 proto kernel scope host src <IP_ADDRESS>

Interesting. I'm not sure where that route comes from - I don't think it's something that Calico manages. @song-jiang do you know?

Calico won't program routes to kube-ipvs0 interface. This route got proto kernel so it could be auto generated by kernel.
AFAIU, for proto kernel, it is a route added automatically when you assign an address to an interface which is not /32. However, kube-proxy should not program any non /32 routes.
@gitbeyond Could you show us the log of ip a?  Do you know if there is any other process on your system which would monitor and create routes?

Thank you for your reply. I recently found that this problem was caused by some other software. After stopping those software, it returned to normal. Thanks twice. It's my own fault.


Added ability to work with 'legacy' System.Configuration in Mono

Modified corefx System.Configuration.ConfigurationManager to be Mono machine.config aware and work with System.Configuration.

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

/cc @marek-safar

@steveisok what's motivation behind this change? We won't be using this code for a while

I'd be very surprised to see the whole assembly switching to CoreFX version with just these changes but let's see. I made few comments anyway

@marek-safar changes updated.


GH-35698: [C#] Update FlatBuffers

Rationale for this change
The FlatBuffers definitions in the C# code don't seem to have been updated for a while. Adding the latest FlatBuffers code ensures that we're up to date with both the Arrow definitions and the FlatBuffers compiler.
What changes are included in this PR?
The latest version of the C# FlatBuffers runtime files were copied into the project (with visibility changed from public to internal).
The latest version of the FlatBuffers compiler was used against the latest version of the Arrow .fbs source. The output of the compiler was edited to more closely match the existing code by moving it into the directory structure and namespaces of the existing classes and by changing class visibility from public to internal.
A few small changes were made to the existing source to accommodate changes in the FlatBuffers runtime, most specifically that the files have moved from the namespace "FlatBuffers" to the namespace "Google.FlatBuffers".
Are these changes tested?
No substantive product changes were made. All tests still pass.
Are there any user-facing changes?
No.
Resolves #35698

Closes: #35698

@CurtHagenlocher is there still value in this PR (I'm not aware of any reason there isn't but it's been a few minutes and I just wanted to double-check before spending any significant time reviewing).

@westonpace Yes, this is still worthwhile. At some point we'll want to implement whatever the final "alternate string view" design is, and being up-to-date on fbs-derived stuff will make that an easier to follow change.


Simple Frontend Responsive Coffee Shop Website

Coffe Shop Website
This is a coffee shop website using HTML,CSS,Javascript.
Issue Type:

[x] New Project

Features Added
User gets to choose the type of coffee he can order.
User can book a table.
Menu availability.
Contact details.
Working hours.
Only Frontend with beautiful CSS animations and Javascript will be implemented.

Please assign this issue to me

Now we don't accept ecommerce website


Unable to archive with local deployment

Following this guide on debian testing and Docker version 18.06.0-ce, build 0ffa825 doesn't let me archive anything.
Here are some logs from the containers:
webrecorder_app_1-log.txt
webrecorder_nginx_1-log.txt
webrecorder_frontend_1-log.txt

Sorry about that, I think the base pywb Docker image is out of date. Webrecorder works with the pywb develop branch as the pywb release are generally a bit slower, so this is likely the issue
Can you try changing the FROM line in the Dockerfile to be FROM webrecorder/pywb:develop and confirm if that works? If so, we'll just switch it to use the develop image.

So I've renamed it to FROM webrecorder/pywb:develop and ran docker-compose build on the root folder of the repo, then docker-compose up -d and got this

These are the logs now
webrecorder_app_1-logs2.txt
webrecorder_redis_1-logs2.txt
webrecorder_warcserver_1-logs2.txt
webrecorder_recorder_1-logs2.txt
webrecorder_nginx_1-logs2.txt
webrecorder_frontend_1-logs2.txt
webrecorder_dat_1-logs2.txt

Hm, the logs seem to be ok, did you give it a minute before loading? The frontend can take around a minute or so to build.

It works thanks!


Upgrade Test stack trace cleaning function

The stack format change slightly and we were no longer matching on the correct stuff to clean up: "(internal/..." became "(node:internal/..."
Took the occasion to swap them out with regex and inline the one-off cleanup logic too (IIRC I kept it separate because not all repos wanted that cleanup. But nobody cares now)
Before:

After:

Random note we should put somewhere more prominent:
So someone mentioned wanting to make this Test.run(__POS_OF__("describe things"), ...) a bit prettier, which is understandable. Though I wanna mention a nuance about the __POS_OF__ part that folks are probably not leveraging. You can optionally carry that first-class, into a helper function, deep into its loop or whatever, and the stack trace will still show your original test's location. For example, compare this:

To this, where testHelper receives __POS_OF__ from its caller:

The second one's directly informative of where the call went wrong, which makes the code frame more useful. It also shows the proper test description


Release SDK v1.0, guarantee backwards compatibility

What problem does your feature solve?
This SDK version is pre-1.0, which implies that we may break the interface between versions without warning to developers. This means developers can't have total confidence in using it, and may need to make significant changes on their wide when just pulling in minor version bumps. It also disincentivizes our team from caring about maintaining backwards compatibility, which proliferates the problem. This SDK shadows the Horizon API, which is stable and defined a major version, so there isn't really a good reason why we would need to make significant breaking changes repeatedly to this SDK. Breaking changes can happen more frequently than horizon's API but it shouldn't be a free-for-all.
What would you like to see?
An official release on Github of this SDK that versions itself at >= 1.0. Going forward, we properly respect semantic versioning and will not introduce backwards incompatible changes without bumping to a new major version.
It would also be worth it (before officially cutting it) to do a bit of a holistic audit and cleanup here, to make sure we are confident and proud of what we release as 1.0.
What alternatives are there?

I plan to release the stable version 1.0.0 with the launch of Protocol 22.


Testnet configuarion

what parameters need to be specified to run the library to work with testnet?

I have the same question.
I believe the connection endpoint for spot is testnet.binance.vision
Do you know if anyone has created a fork capable of connecting to the spot testnet?

A solution would be to include "testnet.binance.vision" in BinanceApiConfig, so that BinanceApiConfig#getApiBaseUrl() will return "testnet.binance.vision", if user creates a connection like this:
Boolean useTestNet = true;
BinanceApiClientFactory factory = BinanceApiClientFactory.newInstance("API-KEY", "SECRET", useTestnet);
BinanceApiRestClient client = factory.newRestClient();

with
/**
   * Instantiates a new binance api client factory.
   *
   * @param apiKey the API key
   * @param secret the Secret
   * @param useTestnet the connection to testnet.binance.vision
   */
  private BinanceApiClientFactory(String apiKey, String secret, boolean useTestnet) {
    this.apiKey = apiKey;
    this.secret = secret;
    this.useTestnet = useTestnet;
  }

Currently, the API can only be used on real accounts, however, especially during development it would be nice if user can bind this API to testnet.

I think having the ability to connect to the spot testnet endpoint is more than just nice. Ever since Binance blocked US access to binance.com accounts there is no simple way to run test code for binance.com from a US IP addr without using a (slower) VPN and additional complications.
Thank you ts-00 for your suggestion. It is overall the right general idea. However, it is a bit more complicated.
The Spot Test Network does not appear to need the SECRET param. It seems it authenticates requests only using RSA generated API public/private Keys (see https://testnet.binance.vision). But I haven't tried it yet. Eliminating the secret param has to be done cleanly. It seems we would need to bypass call to AuthenticationInterceptor in class BinanceApiServiceGenerator and pass the useTestnet flag in a way that does not affect the real spot network endpoint behavior (and call to retrofitBuilder).
This has to be implemented carefully. At this point I just want to find out if someone has already done a pull request for this feature- to add a testnet endpoint to BinanceApiClientFactory. Don't want to duplicate any work. I am new to Github. How can we find out if someone is already working on this?

How can we find out if someone is already working on this?

Searching like https://github.com/binance-exchange/binance-java-api/pulls?q=is%3Apr+is%3Aopen+testnet, it seems that no one is working on it. However, I don't know if the binance team is working on it.
I also need to test code by testnet with the API, because without we can only begin to use this API with real money which is dangerous while developing..

I have a solution ready and tested.
I created branch 'georgesoler/binance-java-api' by forking from repository 'binance-exchange/binance-java-api' which in turn is a fork from the original project 'joaopsilva/binance-java-api'. I forked from 'binance-exchange/binance-java-api' because it's the repository forked by the most people and likely most up-to-date, but being new to GitHub it is possible I screwed up and forked the wrong branch...
You can see my changes to three source files by checking out url = https://github.com/georgesoler/binance-java-api.git
I have also written this test driver that handles most use cases. Its a java source disguised as .txt and you can download it here-
BinanceSpotTestnetDriverTest.java.txt
Please feel free to modify the driver to test in your environment and let me know how it goes. It would be great if someone besides me could test this before I create a pull request.
-George

@georgesoler I've just made some tests with your branch and so far it's working perfectly! I especially tested the websocket with testnet and I didn't find any remarkable issue.
Thank you so much!!

UPDATE:
I am not able anymore to get my test account info. I was working on the original master branch.
I am using "getAccount()" of BinanceApiRestClient after connecting my client with my key and secret generated for testing.
I get a com.binance.api.client.exception.BinanceApiException: Invalid API-key, IP, or permissions for action.
And the error is raised here:
at com.binance.api.client.impl.BinanceApiServiceGenerator.executeSync(BinanceApiServiceGenerator.java:93)

You should not work on the original master branch because the code has not
been merged yet. I placed a pull request and I don't know how long before
the code is reviewed and merged. Please work on the georgesoler branch for
testnet purposes.
On Wed, 14 Apr 2021 at 08:38, Antonio @.***> wrote:

UPDATE:
I am not able anymore to get my test account info. I was working on the
original master branch.
I am using "getAccount()" of BinanceApiRestClient after connecting my
client with my key and secret generated for testing.
I get a com.binance.api.client.exception.BinanceApiException: Invalid
API-key, IP, or permissions for action.
And the error is raised here:
at
com.binance.api.client.impl.BinanceApiServiceGenerator.executeSync(BinanceApiServiceGenerator.java:93)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/binance-exchange/binance-java-api/issues/326#issuecomment-819615239,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AM5GCDCDFRVL7DYRWX3JHRTTIWZGFANCNFSM4YUQO77A
.

Hi Mina,
I cannot reproduce the error you report.
I uploaded the driver I use to test the testnet code changes (see below).
The driver includes a call to
BinanceApiClientFactory.newInstance(apiKey, secret, true, true)
If you haven't resolved the issue, please test by modifying the driver to suit your needs and let me know if you still get the error.
Thank you.
BinanceSpotTestnetDriverTest.java.txt

A solution would be to include "testnet.binance.vision" in BinanceApiConfig, so that BinanceApiConfig#getApiBaseUrl() will return "testnet.binance.vision", if user creates a connection like this:
Boolean useTestNet = true;
BinanceApiClientFactory factory = BinanceApiClientFactory.newInstance("API-KEY", "SECRET", useTestnet);
BinanceApiRestClient client = factory.newRestClient();

with
/**
   * Instantiates a new binance api client factory.
   *
   * @param apiKey the API key
   * @param secret the Secret
   * @param useTestnet the connection to testnet.binance.vision
   */
  private BinanceApiClientFactory(String apiKey, String secret, boolean useTestnet) {
    this.apiKey = apiKey;
    this.secret = secret;
    this.useTestnet = useTestnet;
  }

/**
   * Instantiates a new binance api client factory.
   *
   * @param apiKey the API key
   * @param secret the Secret
   */
  private BinanceApiClientFactory(String apiKey, String secret) {
    this.apiKey = apiKey;
    this.secret = secret;
  }

Currently, the API can only be used on real accounts, however, especially during development it would be nice if user can bind this API to testnet.

its not possible to edit the file, how do I proceed with this ?


mypy not compatible with any named axes?

When I specify a type like TensorType["batch_size", "num_channels", "x", "y"], I get a mypy error like error: Name "batch_size" is not defined for each of the named axes. Is this expected? Am I doing something wrong? This is with the most recent mypy, 0.950.

This is expected - mypy thinks that the string is being used as part of a forward reference, rather than as a literal string. Python's typing system can be a bit of a mess in edge cases like this.
One solution is to actually define some objects with the name of these strings. Another is to use the appropriate annotations to have mypy ignore the error.

Got it - no worries, I understand the constraints here. It might be helpful to update the section of the documentation discussing mypy to explain this more clearly, though. When I read that mypy was "mostly" supported I would have expected that this core feature would work without hacks.

Latest version of Pyright (1.1.262) starts to throw the similar errors. ("batch_size" is not defined)

Another is to use the appropriate annotations to have mypy ignore the error.

How exactly is this supposed to work? Even with the following
from torchtyping import TensorType  # type: ignore

def batch_outer_product(
    x: TensorType[
        "batch",  # type: ignore
        "x_channels",  # type: ignore
    ],
    y: TensorType[
        "batch",  # type: ignore
        "y_channels",  # type: ignore
    ],
) -> TensorType[
    "batch",  # type: ignore
    "x_channels",  # type: ignore
    "y_channels",  # type: ignore
]:

    return x.unsqueeze(-1) * y.unsqueeze(-2)

I'm getting:
test.py:6: error: Name "batch" is not defined  [name-defined]
test.py:6: error: Name "x_channels" is not defined  [name-defined]
test.py:7: error: Unused "type: ignore" comment
test.py:8: error: Unused "type: ignore" comment
test.py:10: error: Name "batch" is not defined  [name-defined]
test.py:10: error: Name "y_channels" is not defined  [name-defined]
test.py:11: error: Unused "type: ignore" comment
test.py:12: error: Unused "type: ignore" comment
test.py:14: error: Name "batch" is not defined  [name-defined]
test.py:14: error: Name "x_channels" is not defined  [name-defined]
test.py:14: error: Name "y_channels" is not defined  [name-defined]
test.py:15: error: Unused "type: ignore" comment
test.py:16: error: Unused "type: ignore" comment
test.py:17: error: Unused "type: ignore" comment

I assume this line in the documentation is no longer valid, right?

Additionally mypy has a bug which causes it crash on any file using the str: int or str: ... notation, as in TensorType["batch": 10].

The underlying issue (https://github.com/python/mypy/issues/10266) has been closed.


Alter JSON string parsing from shell

Closes #2680.
Updates references where parameters required JSON strings to allow Python lists/dictionaries. Valid JSON is a valid Python list/dict, but the reverse is not true. On CMD, you would need to escape characters with something like the following:
--parameters "{\"foo\": \"doo\"}"
but certain escaped strings are simply not parsed correctly by the shell. For example, you cannot use:
--parameters "{\"foo\": \"doo & boo\"}"
This results in CMD terminating the command line at the &, resulting in errors. Previously, the only workaround would be to use the @file syntax, but there should be a way to specify purely on the command line.
This update will now allow the following:
--parameters "{'foo': 'doo & boo'}"
This simplifies the input on CMD greatly.

@troydai added unit test for shell_safe_json_parse

Codecov Report

Merging #2705 into master will increase coverage by 0.02%.
The diff coverage is 94.21%.

@@            Coverage Diff             @@
##           master    #2705      +/-   ##
==========================================
+ Coverage   61.13%   61.16%   +0.02%
==========================================
  Files         480      480
  Lines       25700    25710      +10
  Branches     3888     3888
==========================================
+ Hits        15712    15725      +13
+ Misses       9058     9056       -2
+ Partials      930      929       -1

Impacted Files
Coverage Δ

...or/azure/cli/command_modules/monitor/validators.py
13.51% <100%> (ø)
:arrow_up:

...cli-vm/azure/cli/command_modules/vm/_validators.py
68.5% <100%> (ø)
:arrow_up:

src/azure-cli-core/azure/cli/core/commands/arm.py
89.21% <100%> (+0.64%)
:arrow_up:

...azure-cli-core/azure/cli/core/commands/__init__.py
79.75% <100%> (ø)
:arrow_up:

...re-cli-vm/azure/cli/command_modules/vm/commands.py
90.47% <100%> (ø)
:arrow_up:

...zure/cli/command_modules/keyvault/_command_type.py
77.77% <100%> (ø)
:arrow_up:

...re-cli-dls/azure/cli/command_modules/dls/custom.py
80.43% <100%> (ø)
:arrow_up:

...ure-cli-vm/azure/cli/command_modules/vm/_params.py
96.13% <100%> (+0.03%)
:arrow_up:

src/azure-cli-core/azure/cli/core/_pkg_util.py
23.52% <100%> (ø)
:arrow_up:

...re-cli-sql/azure/cli/command_modules/sql/custom.py
86.09% <100%> (ø)
:arrow_up:

... and 64 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ce75078...5c0361a. Read the comment docs.


[SPARK-16816] Add api to get JavaSparkContext from SparkSession

What changes were proposed in this pull request?
Add api to get JavaSparkContext from SparkSession
How was this patch tested?
Run all test cases of spark

Can one of the admins verify this patch?

Isn't this just
new JavaSparkContext(session.sparkContext)

?
Perhaps we should just update the documentation to say that.

Yea seems like we should just update the documentation for this one.

@petermaxlee @rxin ok so i go for make changes in documentation...!! What you guyss suggest what is the correct place to add this.

Can we update the doc for sparkContext?

@rxin But according to user they read it from starting and while they read about sparkContext they dont know about spark session so i think we just add a single line below the example where we extract sparkContext from spark session we just show that with sparkContext you can also create java spark context like this new JavaSparkContext(session.sparkContext) or JavaConverter.fromSparkContext(spark.sparkContext). What do you think its looks better?

I meant update the sparkContext field in SparkSession.

(hence the lower case)

@rxin ok i look into it soon and make changes in that.

@srowen @rxin @petermaxlee I make some changes according to me you can suggest something else if you have some good idea else just merge it if its looks good.

@phalodi these changes don't seem to be what Reynold suggested. You have your old change and added some example change (?) but didn't update documentation.

@srowen @rxin @petermaxlee i will close this pull request and create new one with documentation changes and also modify jira issue.


Replace notifications library with using platform APIs directly, on Android

After some time studying how we're doing notifications, I think the wix/react-native-notifications library isn't doing us a great deal of good, and on Android in particular it's getting in our way more than it's being useful.
Specifically, it's all a bunch of pretty thin wrappers around the underlying Android and GCM APIs. Which could have value, if it meant we got to write JS code that was the same for iOS and Android... but for handling a GCM message that just came in, the RN startup cost would be unacceptable, so we're going to have to write Java code anyway. Plus, the library doesn't even do a good job of making the APIs that we do call from JS actually be similar.
And conversely, it sticks its own set of APIs between us and the platform APIs; and those APIs aren't well documented, sometimes don't make a lot of sense, and don't stay up to date with useful features of the last few years.
So, let's replace the use of this library on Android with just invoking the underlying Android and GCM APIs directly. In particular, let's do this as a prerequisite step before #2691; I think it will make that task quite a bit easier.
PS - We'll want to do the same thing eventually on iOS, but there's no rush. Currently we don't take advantage of any of the features Apple provides for doing something fancier than just displaying each notification one after the other, like we do on Android; so there's very little code of ours on the iOS side. When we do decide it's time to add those features, we'll want to do this as a prerequisite step then, for the same reasons as on Android.

Since I've just spent some more time studying this code in the course of reviewing #2874, here's a more detailed plan:
1 Replace the use of the library's ProxyService for our "Clear" action on the notification. Instead, just have our own little service.

First, just add the new service, declare it in the manifest, and have the dismissIntent point to it instead -- but change as little as possible about the interface, so send it the same bundle with ACTION_NOTIFICATIONS_DISMISS.
Then, now that we control that code, we get to clean up how it works! The sad block-comment just above that dismissIntent code of ours gets to go away, because we make the intent look how we want it to, with e.g. dismissIntent.setAction(ACTION_CLEAR), and have our service interpret it appropriately.
This means the nasty conditional in getPushNotification gets to disappear, too; the logic in the body moves somewhere reasonable and gets invoked by our proxy's handler in the ACTION_CLEAR case.

Take control of the manifest: Pull the library's manifest into our own source manifest, and stop including the one from the library.

Something like tools:node="replace" tools:selector="com.wix.reactnativenotifications" on our <manifest> element should do it. See https://developer.android.com/studio/build/manifest-merge .

Replace the use of the library's ProxyService for the notification's main action, too.

I think we only have to say builder.setContentIntent, after constructing our own Intent, again using Intent#setAction to distinguish.
Now we can cut that from the manifest.

Replace the library's GcmMessageHandlerService. Just follow upstream GCM docs; this for the manifest blob, and this for the service implementation.

Now our getPushNotification can disappear entirely.

Replace the library's services for handling GCM Instance IDs.

Here, the combination of this library and how we're currently using it is actually quite broken already. We might be better off writing this from scratch.
We'd follow the upstream GCM docs: this one and some it links to, notably the example app.
For the bits where the GCM docs say to store information in your app, we'll of course want to send it to our JS code. So we'll follow the upstream RN docs for sending an event to JS.
Some key conceptual observations from the GCM docs:

The app has one Instance ID token. There's no need to keep asking for it.
Occasionally the system may decide the token needs to change. This is what InstanceIDListenerService is for, and its onTokenRefresh method; the system will call it in that rare situation. There's no reason for us to call it. (This is one thing the library is pretty confused about.)
In our case, after we get the token once and send it to one Zulip server, there may be other Zulip servers later that the user logs into and we want to send the token to. So we should remember the token locally, and do that on login.

At this point, there's nothing from the library left that we're still using! We can remove it entirely.

Hello! My community just started using Zulip and though we love it, we have noticed a host of issues around notifications for mobile. Android seems to be worst (no notifications show up for several users), but iOS also only shows notifications when someone is specifically mentioned (and there's never a red badge shown). I am happy to investigate and find a solution, but the notes above seem to potentially fix a lot. What is the progress on this?

@anoukruhaak these problems sound serious but are likely not addressed directly by this issue. We have started working actively on the list above but implementing all of the listed tasks will probably take a few months.
We consider the notifications to be a core functionality and any issue we find and can resolve is of highest priority. So, if let's continue in a separate GitHub issue. We can also continue in our #Mobile stream

Another strategy might be to fork a wrapper library and use a hack like this for adding the group notifications feature.
Also want to think about updating to fcm since gcm is being deprecated.

We don't even need to fork it. As long as we have confidence in the library and it is actively developed we can contribute code to it.
This options is worth evaluating 👍

Also want to think about updating to fcm since gcm is being deprecated.

Yeah definitely. I think they've made it a pretty easy upgrade, so we just need to do it sometime.

Another strategy might be to fork a wrapper library and use a hack like this for adding the group notifications feature.

Hmm -- taking someone else's low-quality wrapper library is how we got notifications to the state they're currently in. :wink: If a given library doesn't support notification grouping except by hacking it into a fork, that's a clear sign that it is, at best, not really maintained and hasn't been for several years. (That feature was new in Android 7, IIRC, over two years ago.)
I'm open to considering a third-party library, but it has to be good enough that it's less work for us to get to a high-quality result with it than without it. For this task, the job these libraries are doing isn't hard -- it's really just a matter of reading the upstream docs (for the several different upstreams involved) and writing what they say to write. So the bar is pretty high for how well made one of them would have to be; otherwise we'd end up spending more time studying the upstream APIs while debugging how the library is broken than we would just writing against the same upstream APIs ourselves.


Broken netmagazine link

http://www.netmagazine.com/features/create-perfect-favicon
Couldn't find it anymore. So we should consider removing it.

I've found the original content at the Internet Archive.
The earliest version at the archive is the most pristine. Most of the newer snapshots are missing images. http://web.archive.org/web/20120208174804/http://www.netmagazine.com/features/create-perfect-favicon.
Additionally, the archive got a 302 redirect on Feb 26, 2014. http://web.archive.org/web/20140226050146/http://www.netmagazine.com/features/create-perfect-favicon -> http://www.creativebloq.com/design/create-perfect-favicon-12112760
Furthermore, this page still exists on the regular Internet. http://www.creativebloq.com/illustrator/create-perfect-favicon-12112760. This version however has many technical issues that may prevent it from being useful. Most of the images fail to load and there are several third-party sharing plugins that lock up the page (Probably from non-async script tags).

I just reviewed [http://www.creativebloq.com/illustrator/create-perfect-favicon-12112760], But I'm not able to reproduce issues you mentioned ( @infomaniac50 ). I think its better to commit new link (under creativebloq) to update cheat-sheet.


Dilato: Core: There is no warning message appears on main brackets window after click "Restart" button on the one display the update options

Steps:

Open brackets 1.13
Create a new file and do some edit and don't save it
Go to Debug > New Brackets Window
Go to Help Menu in newly opened window
Click on Check for Updates --- Update Notification Dialogue should open in the same window where user clicked on Check for update.
Click on Get it Now button --- Download will start and you can see the download status in status bar
After download the update file, click restart --- the current Brackets windows will be colsed, Save Changes dialogue will appear on the main Brackets window
Click Cancel button --- there is no warning message appears
Save the file and close the Brackets --- It will update the brackets

Result: After step 8, there is no warning message appears
Expect: The warning message "The update will be applied on relaunch." should appears after step 8.

@RuchiAgrawalAdobe This is duplicate of #14375


UIMA 6245 Fix CVD makeNodes to use array length

A general fix for this, explicitly pass in array length to makeNodes

https://github.com/augustoribeiro - Can you please test / review this alternative fix, which should fix this for all types?  Thanks!

Works with my examples. Thanks!

augustoribeiro tested, works.


Simplify ternary expressions

Not sure if this is a style thing 🙂

Nice! :)
I don't have any excuses ;)


Helm charts not being deployed, when accessing cluster remotely

Describe the bug
When trying to deploy a helm chart manifest using kubectl apply while accessing the cluster remotely kubectl returns successful, but the chart is never actually deployed.
To Reproduce
Steps to reproduce the behavior:

Configure a local kubectl to remotely access your cluster
Copy the default traefik manifest and make a change

apiVersion: k3s.cattle.io/v1
kind: HelmChart
metadata:
  name: traefik
  namespace: kube-system
spec:
  chart: stable/traefik
  set:
    rbac.enabled: "true"
    ssl.enabled: "true"
    debug.enabled: "true"

Run kubectl apply -f traefik.yaml

Expected behavior
The traefik configuration is changed and a new pod is launched to replace the outdated one.

I don't even think this works locally. I tried it on the master and it didn't deploy.

I can confirm this doesn't work locally.

Are you running k3s with --no-deploy traefik?

No, I'm running with default options

If you don't want k3s to manage traefik then just launch the server with --no-deploy traefik.
erik@lubuntu:~/go/src/github.com/rancher/k3s$ ./install.sh --no-deploy traefik
...
erik@lubuntu:~/go/src/github.com/rancher/k3s$ cat >custom-traefik.yaml
apiVersion: k3s.cattle.io/v1
kind: HelmChart
metadata:
  name: traefik
  namespace: kube-system
spec:
  chart: stable/traefik
  set:
    rbac.enabled: "true"
    ssl.enabled: "true"
    debug.enabled: "true"
erik@lubuntu:~/go/src/github.com/rancher/k3s$ kubectl get pods --all-namespaces
NAMESPACE     NAME                       READY   STATUS    RESTARTS   AGE
kube-system   coredns-7748f7f6df-k9bw6   1/1     Running   0          8s
erik@lubuntu:~/go/src/github.com/rancher/k3s$ kubectl apply -f custom-traefik.yaml
helmchart.k3s.cattle.io/traefik created
erik@lubuntu:~/go/src/github.com/rancher/k3s$ kubectl get pods --all-namespaces
NAMESPACE     NAME                             READY   STATUS      RESTARTS   AGE
kube-system   coredns-7748f7f6df-k9bw6         1/1     Running     0          2m21s
kube-system   helm-install-traefik-8nvs8       0/1     Completed   0          2m2s
kube-system   svclb-traefik-6d4d77b959-fm5lt   2/2     Running     0          115s
kube-system   traefik-6487fddfff-6nzx7         1/1     Running     0          115s
erik@lubuntu:~/go/src/github.com/rancher/k3s$ kubectl logs -n kube-system traefik-6487fddfff-6nzx7
{"level":"info","msg":"Using TOML configuration file /config/traefik.toml","time":"2019-03-29T15:20:05Z"}
{"level":"info","msg":"No tls.defaultCertificate given for https: using the first item in tls.certificates as a fallback.","time":"2019-03-29T15:20:05Z"}
{"level":"info","msg":"Traefik version v1.7.9 built on 2019-02-11_11:36:32AM","time":"2019-03-29T15:20:05Z"}
{"level":"debug","msg":"Global configuration loaded {\"LifeCycle\":{\"RequestAcceptGraceTimeout\":0,\"GraceTimeOut\":10000000000},\"GraceT

I'm not asking about not managing traefik, that was just an simple example.
But I guess updating charts that are deployed from /var/lib/rancher/k3s/server/manifests through kubectl is not intended usage?

I can confirm this happens to me (using the stable/unifi helm chart) no matter if applied locally, remotely, or by putting the manifest directly in /var/lib/rancher/k3s/server/manifests.
Curiously, it worked once on a fresh install, but no updates are applied. Even if I delete the namespace I created for it, it never gets redeployed.

Same here :(
How do I force update any edits made on /var/lib/rancher/k3s/server/manifests ?

Same here

Closing due to age.


Windows 2000 demo: enable TLS 1.0

Hi All,
It might be nice to enable TLS 1.0 in the default Windows 2000 demo, which would allow visiting more websites.
It's under Internet Options -> Advanced -> Security -> Use TLS 1.0
Most websites no longer enable 1.0, but some websites still allow it.
Thanks,
Collin

on windows 2000 (ie5,6), there is only sha-1 support (so i guess only https://badssl.com works). everything uses sha-2 encryption since https://shattered.io/ happened. check out their website to see more

Hah, I guess badssl.com was the only website I tested. Ok make sense. Thanks

If you'd want sha-2 tls 1.0 (and newer https protocols), download opera 12.02 http://www.oldversion.com/windows/opera-12-02. Don't save as .php, save as any extention then replace .php with .exe


[shell] Question: is it possible to use the shell plugin without needing to define command in capability?

It seems like the current shell plugin can only execute shell commands pre-defined in capability file.
Is it possible to remove all restrictions and make it possible to execute any command?

Is it possible to remove all restrictions and make it possible to execute any command?

No, and it won't ever be. One thing we want to add at some point (we had a PR open for v1) was what we have in the fs plugin, a method in rust to extend the scope (for the js side) at runtime.
Some people use cmd.exe / bash as the command and run their actual commands as shell commands (confusing wording lol), but of course we do not recommend this, only if it's really really necessary and Rust is totally out of the question.


DLPX-73623 SSHJ remote port forwarding buffers can grow without limits

Problem
When storage is slow, information coming through a tunnel (remote forwarding) at a fast rate causes the forwarding buffers to grow without control and OOM the program.
Diagnosis
SSHJ's tunneling functionality has the ability to pause the data coming from the server, but it does not use it effectively. Under specific conditions such as the Linux ZFS write pauses we observed, that functionality grows the buffer when it does not really need to and fails to stop asking the server for more data.
This is how the buffer looks like from start (left) to end (right):
  start     rpos   wpos   end
    | wasted | data | tail |

Here, rpos is the current read position where buffered data is read and sent to the destination, whereas wpos is where the next data from the server is written.
Both rpos and wpos move to the right as SSHJ reads from and writes to the buffer. When rpos reaches wpos, both are reset to the start of the buffer. However, if rpos does not catch up with wpos, the "tail" (remainder of the buffer to the right of wpos) shrinks until no more space is left there, at which point the code grows the buffer to the right. This, however, is not necessary because there is typically a large "wasted" portion of the buffer to the left of rpos that is not being used.
Solution
First, make the buffer used by remote forwarding circular and ensure that the whole buffer is used before it needs to grow.
Second, add a configurable limit to the size of the buffer. By default, use no limit, which is the previous behavior. But if the limit is set, make sure that the SSH_MSG_CHANNEL_WINDOW_ADJUST messages sent back to the server are only sent when there is space left in the buffer, and limit the size requested from the server so that the buffer cannot grow beyond the limit.
I decided not to fully convert the existing Buffer class, which would have been a larger and riskier change, but to create a separate CircularBuffer which contains only the methods used by remote forwarding (ChannelInputStream). I have no evidence that the other areas of SSHJ that use Buffer suffer from this bug, but if that's the case, in the future this solution can be extended to add to CircularBuffer all methods from Buffer used by the rest of SSHJ, and get rid of Buffer altogether.
Testing
Created a performance test RemotePFPerformanceTest (which I'll have to move it out of the unit test suite because of its performance impact) that simulates a fast producer and slower reader of data in an remote-forwarding ssh tunnel. This test reproduced the bug consistently: it runs out of memory due to unlimited buffer growth. After the fix, the test completes. I verified that the buffer grows only up to the specified limit.
To do: test this fix in the context of the Delphix product in the scenario that originally hit this bug.
Note: this review is about the SSHJ bug fix only. Separate reviews will be created for updating the use of this library in the Delphix product.

An update on the latest version: there was a race condition in ChannelInputStream which caused sometimes to request the server more bytes than the buffer had the capacity to hold. This happened because reading and writing to the buffer was done in separate synchronized blocks, so that after reading from the buffer the reading thread would base its message on a window size that was larger than it actually is,

This is merged to parent repo https://github.com/hierynomus/sshj/pull/913


new: usr: #162: Added Hadoop2ClusterManagerV2 implememtation of hadoop cluster manager for correct consistent hashing logic

Pull Request Test Coverage Report for Build 140

68 of 122 (55.74%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.8%) to 24.172%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

rubix-core/src/main/java/com/qubole/rubix/core/utils/DummyClusterManager.java
0
2
0.0%

rubix-hadoop2/src/main/java/com/qubole/rubix/hadoop2/Hadoop2ClusterManager.java
4
6
66.67%

rubix-presto/src/main/java/com/qubole/rubix/presto/PrestoClusterManager.java
0
2
0.0%

rubix-spi/src/main/java/com/qubole/rubix/spi/ClusterManager.java
0
6
0.0%

rubix-hadoop2/src/main/java/com/qubole/rubix/hadoop2/Hadoop2ClusterManagerUtil.java
30
49
61.22%

rubix-hadoop2/src/main/java/com/qubole/rubix/hadoop2/Hadoop2ClusterManagerV2.java
34
57
59.65%

Totals

Change from base Build 139:
0.8%

Covered Lines:
569

Relevant Lines:
2354

💛  - Coveralls

One big question is composition vs inheritance between the two implementations of Hadoop Cluster Managers. I probably prefer inheritance in this specific case because tests that are valid for V1 is also valid for V2 except for a few scenarios where you want different scenarios. If you choose composition, then you have to repeat all the tests.

@vrajat Yes I thought about it. First the tests are same in terms of checking the condition. The output of the tests are going to be different. For example, for testUnhealthyNodeCluster_decommissioned, the old HadoopClusterManager will return 1 node whereas the new implementation is going to return 2 nodes.
Both the clustermanagers are extending the abstract class and quite a few implementations are different for these two cases such as initialize, getNextRunningNodeIndex and getPreviousRunningNodeIndex. The common methods which can be used as utility methods have been extracted out to a different Utility class. Same logic has been applied to TestClasses as well.