Fix:mysql-cdc can't run in sql mode

com.ververica.cdc.connectors.mysql.table.MySqlTableSourceFactory not merge in META-INF/services/org.apache.flink.table.factories.Factory when maven package

Purpose of this pull request

Which issue you fix
Fixes # (issue).
Checklist:

[x] I have executed the 'mvn spotless:apply' command to format my code.

[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] New and existing unit tests pass locally with my changes.
[ ] I have checked my code and corrected any misspellings.
[x] My commit is only one. (If there are multiple commits, you can use 'git squash' to compress multiple commits into one.)

Thanks for your contribution. We will merge it.


Tests version_test() fail on the latest release

Tests version_test() in src/distribution/version.rs, src/image/version.rs, and src/runtime/version.rs fail on v0.6.4 due to VERSION_DEV being set to empty but the tests expecting it to still be "-dev".
I suggest creating a v0.6.5 release that fixes this issue so that pipelines packaging this crate for e.g. Fedora continue to work.

Fixed by #161.


wasi-sdk-20+threads sysroot does not include libc++/libc++abi

Hi! I downloaded the wasi-sdk-20+threads pre-release sysroot file and it does not contain libc++.a or libc++abi.a in its subfolder:
$ tar ztf wasi-sysroot-20.0.threads.tar.gz | grep libc++
wasi-sysroot/lib/wasm32-wasi/libc++experimental.a
wasi-sysroot/lib/wasm32-wasi/libc++abi.a
wasi-sysroot/lib/wasm32-wasi/libc++.a

By comparison, this is the result for libc:
$ tar ztf wasi-sysroot-20.0.threads.tar.gz | grep libc.a
wasi-sysroot/lib/wasm32-wasi-threads/libc.a
wasi-sysroot/lib/wasm32-wasi/libc.a

Is this intentional? If so, should we expect C++ support eventually?

I was interested in how #301 might change things and indeed, when I retrieve the artifacts from #301 using ci/download-workflow-artifacts.sh and unpack them I see the C++ libraries that were missing.
$ tar ztf wasi-sysroot-19.5g1e4440799c2d.tar.gz | grep libc++
wasi-sysroot/lib/wasm32-wasi/libc++abi.a
wasi-sysroot/lib/wasm32-wasi/libc++experimental.a
wasi-sysroot/lib/wasm32-wasi/libc++.a
wasi-sysroot/lib/wasm32-wasi-threads/libc++abi.a
wasi-sysroot/lib/wasm32-wasi-threads/libc++experimental.a
wasi-sysroot/lib/wasm32-wasi-threads/libc++.a

I think this is good evidence that we should merge #301, but let me try to answer some comments first.

But it would be nice to have some level of C++ support until we have https://github.com/WebAssembly/wasi-sdk/pull/301 done (such as a non-threaded version). At least, it would enable WASM with C++ and shared-memory.

Unless I'm missing something, the C++ support you're looking for is already present in the wasm32-wasi target. You should be able to compile C++ programs with that target unless they have anything to do with threads. To compile any C++ program that has anything to do with threads, you will need #301 and to use the wasm32-wasi-threads target — it's a bit of an "all or nothing" deal right now. The reason for this is that WebAssembly atomic instructions, shared memory, and wait/notify are all feature-gated behind the atomics flag (some tools call this threads) and due to how WebAssembly compilation in LLVM works, all of the object files must have the same feature flags. Hence the need for a new target, wasm32-wasi-threads, in which both the library files and your program code are compiled with the atomics flag.

I was able to copy C++'s libs from wasm32-wasi to wasm32-wasi-threads and they kinda worked

I think we should try to test out the atomics-enabled libc++ libraries from #301 with whatever you are trying to do. Obviously the C++ support for wasm32-wasi-threads is very experimental so any feedback here would be good. E.g., you mention that things "kinda worked" — did something actually break? If you would like I can upload the sysroot you need here and you can test things out? Feel free to contact me on Zulip if you want to coordinate in real-time about this.

Any plan to publish a new pre-release that includes libc++?

I think the release itself which needs #313 and #314 to merge is probably closer to happening--they need a review before I do anything (cc: @sunfishcode, @sbc100).

Sorry for the delay on my feedback and thanks for the prompt responses!

But it would be nice to have some level of C++ support until we have #301 done (such as a non-threaded version). At least, it would enable WASM with C++ and shared-memory.

Unless I'm missing something, the C++ support you're looking for is already present in the wasm32-wasi target. You should be able to compile C++ programs with that target unless they have anything to do with threads. To compile any C++ program that has anything to do with threads, you will need #301 and to use the wasm32-wasi-threads target — it's a bit of an "all or nothing" deal right now. The reason for this is that WebAssembly atomic instructions, shared memory, and wait/notify are all feature-gated behind the atomics flag (some tools call this threads) and due to how WebAssembly compilation in LLVM works, all of the object files must have the same feature flags. Hence the need for a new target, wasm32-wasi-threads, in which both the library files and your program code are compiled with the atomics flag.

The C++ support I need is in wasm32-wasi, yes. But I need atomics in order to share memory between WebWorkers. It is a classic producer-consumer: one worker producing audio samples, another worker consuming it (honestly, only because WebAudio forces me to use workers).
In other words, I was trying to link a WASM file that can accept a SharedArrayBuffer from its Memory when instantiated. But the link flag complains about the atomics feature in one of libc's .o file.
In theory, I would be able to share a Memory between workers using postMessage, then instantiate one container per worker using a shared memory - effectively using JS's environment to manage the threads while C/C++ takes care of locks/thread-safety.

I was able to copy C++'s libs from wasm32-wasi to wasm32-wasi-threads and they kinda worked

I think we should try to test out the atomics-enabled libc++ libraries from #301 with whatever you are trying to do. Obviously the C++ support for wasm32-wasi-threads is very experimental so any feedback here would be good. E.g., you mention that things "kinda worked" — did something actually break? If you would like I can upload the sysroot you need here and you can test things out? Feel free to contact me on Zulip if you want to coordinate in real-time about this.

The "kinda-work" part is the short of: my code compiles and runs on a specific browser, using shared memory, without any visible weirdness but I can't properly run my spike tests because I'm facing different browser-specific issues.
I had all sort of issues from different browsers when trying to actually use shared memory. Safari can't postMessage a WebAssembly.Memory. Chrome fails on different audio issues. Firefox I can't recall... 😅
But my code is C++ without STL or threads inside the WASM container, so I don't need libc++ from #301 - I only need really basic stuff (like cxx_personality, etc).

If you are targeting the web, and audio workers in particular, you might want to consider using emscripten.  Its has builtin support for threads and workers and audio worklets: https://emscripten.org/docs/api_reference/wasm_audio_worklets.html

Ok, I think this issue can be closed now that the wasi-sdk-20 release is published. We can reopen if needed--feel free to try it out and see!


Duplicate short codes for two ministries

The following organizations share the same key in the short code list:

Short Code
Organization Name

FLNR
Forests, Lands, Natural Resource Operations & Rural Development

FLNR
Indigenous Relations & Reconciliation

Fixed elsewhere. Closing.


Conflicts since Magit updated to v 2

I noticed there are several conflicts since magit was auto updated to the latest version. Ie: git-commit-mode.

The config has been updated to work with the latest magit, so please first make sure you have the latest version of this config installed. Re. the conflicts, you should get a startup message which tells you to remove the obsolete git-commit-mode and git-rebase-mode packages. Once you've done that, and restarted Emacs, please let me know whether you still experience any problems.

Magit seems to work fine now, there are some other issues after the upgrade, like mousescroll no longer works the same way. I noticed the build is not currently passing, should I just wait until it passes and then pull again?

Don't necessarily trust the build -- that's mainly a smoke test to make sure the config starts up cleanly on all the supported Emacs versions.
What's changed with the mouse scrolling?

Before the mouse scroll was a smooth scroll now its scrolls down a screen and half, There also seems to be an issue with flycheck now and when opening a new file it does not bring the buffer to the foreground. Here is what shows up in messages on startup:
byte-code: Symbol's function definition is void: flycheck-checker-option-vars
Could not find config file "~/.tidyrc".  Winging it. [2 times]

It's probably aborting start-up before it reaches the part of the config which deals with scrolling. The "Symbol's function definition is void" message looks like the problem. Are you getting that even after removing and reinstalling the flycheck-ledger package? You shouldn't be. If in doubt, delete ~/.emacs.d/elpa/flycheck-* and restart Emacs.

That seems to have fixed it. Thanks

Great!


[FeaturDevelopment] Stop/End date for Scheduled Jobs

Hi,
Currently, for an azkaban-job, the user can specify a start-date and the
period-of-interval
for recurrent executions.
In addition to these, it would be a very helpful to specify an stop-date
too, past which the job would
no more be triggered/executed. The current job definition does not support
an stop-date and I am
trying to implement it (picture attached)
However, I am badly stuck with the implementation due to lack of
documentation in the code and below
are my specific questions. Any ideas to move further from here or
connecting to me somebody who
has knowledge of these areas of code, would be much appreciated.

The job details entered on the web-UI (attached page) are persisted as a
blob
in the column 'data' in the database table named 'triggers'. Which java
class
reads this data further and translates them to equivalent
threads/processes. Is this
blob further used by the ExecutorServer to create some other entries in a
different table?

Which existing class/method is the best choice to modify with a tiny
check to ensure that
current execution time is not past the stop-date.
I am thinking it should be
*azkaban.trigger.builtin.BasicTimeChecker.eval() *
Any better ideas?

I did notice that
azkaban.scheduler.TriggerBasedScheduleLoader.scheduleToTrigger(Schedule)
is adding translating the schedule into a Trigger, which consists of a
TriggerCondition
and an ExpiryCondition, both of them composed of
azkaban.trigger.builtin.BasicTimeChecker.
However, they are almost identical, differing only with the id
BasicTimeChecker_1 vs BasicTimeChecker_2.
Where are these two Condition actually called to perform the check?

Thanks in advance.
Venkatesh
+46707188865

@inoviavenkat Thanks a lot for your feature request. Below  are the answers to your questions.
1). The blob is used by webserver only. Jobs in executor will only be triggered by webserver calling.
No other tables.
2). 3). They are called in triggermanager class. There is a thread to check these conditions periodically.
BTW, you might contribute to adding this feature after #719 is merged. Flexible Scheduling will be released soon.

@inoviavenkat still working on this feature? we will be happy to help

@ameyamk
I have implemented this feature on a forked repository at : https://github.com/inoviaazkaban/azkabanfork/commits/master
There are 5 or 6 commits containing several changes for this feature, the latest of these being the Sep 13, 2016 commit of
https://github.com/inoviaazkaban/azkabanfork/commit/ca9d4497cef243f02118b0334b47246660d92893
(None of the subsequent commits occurring after this are related to this feature though, and can be ignored. The code will have to be manually merged to your master, if required. Good luck :) )

From: Ameya<EMAIL_ADDRESS>Sent: Saturday, April 15, 2017 1:05:01 AM
To: azkaban/azkaban
Cc: Venkatesh Gowda; Mention
Subject: Re: [azkaban/azkaban] [FeaturDevelopment] Stop/End date for Scheduled Jobs (#721)
@inoviavenkathttps://github.com/inoviavenkat still working on this feature? we will be happy to help

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/azkaban/azkaban/issues/721#issuecomment-294252894, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AQVHRdmU-47lbMgynjfn5SXsywX3Vp9yks5rv_udgaJpZM4J3G-H.

@ameyamk
Are you Ameya K Shetty, who studied at NIE, Mysore? I just ask out of curiosity since it is a rare Indian name and I had a classmate long back with this name and he was a good programmer. Just wondering about the probablity of you being him..!

No I am not :-) - i am this guy - https://www.linkedin.com/in/ameyakanitkar/
On Fri, Apr 21, 2017 at 5:13 AM, Venkatesh Changegowda <
<EMAIL_ADDRESS>wrote:

@ameyamk https://github.com/ameyamk
Are you Ameya K Shetty, who studied at NIE, Mysore? I just ask out of
curiosity since it is a rare Indian name and I had a classmate long back
with this name and he was a good programmer. Just wondering about the
probablity of you being him..!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/azkaban/azkaban/issues/721#issuecomment-296174683,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGN0AWxy0Ofi8LnDawBRIqvukGXDkZU7ks5ryJ1pgaJpZM4J3G-H
.

Something new about the issue?!
Is there some way to set an EndDate to some flow??


typo in database schema

Hey, probably not a good place to ask, but i wanted to thank you for this awesome starter repo, also is it okay for me to use it as a base for mine?

Thank you! And yes go ahead, it's MIT so no need for my permission :)


Merging Develop to Master 4.17.2018 - small whitelabel updates.2

Hello @ksigler7,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


fix: add missing netlify graph tokens in env

🎉 Thanks for submitting a pull request! 🎉
Summary
In https://github.com/netlify/cli/pull/4235 we added this polling in functions server and stopped injecting variables in the main dev command, but frameworks like e.g. Next.js don't serve through the functions server.
Fix is to inject the netlify graph variables before any server starts so that they inherit the environment.

For us to review and ship your PR efficiently, please perform the following steps:

[ ] Open a bug/issue before writing your code 🧑💻. This ensures we can discuss the changes and get feedback from everyone that should be involved.
[ ] Read the contribution guidelines 📖. This ensures your code follows our style guide and
passes our tests.
[ ] Update or add tests (if any source code was changed or added) 🧪
[ ] Update or add documentation (if features were changed or added) 📝
[ ] Make sure the status checks below are successful ✅

A picture of a cute animal (not mandatory, but encouraged)

Released in https://github.com/netlify/cli/releases/tag/v9.6.1


https support enabled

Users can now choose to have livereload served across https. I'm currently developing a site on https and this functionality was necessary.

Thank you for this. I will review and merge it in.


The amount of connections isn't equal to the amount of success + connecting + error

Description

An image sums it up. It is weird that the amount of the left isn't equal to the sum of the amounts on the right.
Steps to Reproduce

go to settings>networks
see the issue

Expected Behavior
Have the sum of the numbers on the right equal to the number on the left
Actual Behavior
They are not equal
Screenshots
See above
Possible Solution
I looked into it and realized that we do not take into account the connections with the status isDisconnectedStatus. Also in the list we mark them as "Connecting..." which is misleading.

Connected to #1455 and may be fixed with it.

It's somewhat different, as in it's a very specific omission, but it can be addressed in the same PR yes, I discovered it while looking at #1455

Status verification: ✅


USB audio samples fails if ASSERT=y

Describe the bug
Running the USB audio samples for nrf5340dk_nrf5340_cpuapp fails on ASSERT in nrf_usbd.h:1273 if asserts are turned on
To Reproduce
Steps to reproduce the behavior:

cd samples\subsys\usb\audio\headphones_microphone
west build -b nrf5340dk_nrf5340_cpuapp
west flash
See error on UART

The issue is visible when trying to set dtoggle for ISO endpoint. It is not applicable as those type of endpoints does not support dtoggle, will post a patch that will filter it out.


Calculate all lockup balances from contract state

@vgrichina I've tried to recover my account (that has lockup) but keep getting error about total balance

Went to account with 50 N lockup on mainnet
Sent 2 N (had liquid outside lockup) worked
Sent 40 N (needed lockup) worked, but now I can't see my account

Assuming glitch with lockup being destroyed, funds sent and remainder going to account? Now no lockup, but there's lingering lockup calc and lookup?

@vgrichina Looks like accounts without lockup also triggering an error: https://near-mainnet-staging-wallet-pr-1225.onrender.com/profile/5b4378098c67c414022d65d1d0feb1943e76017acee696a511b5aa960d64e35a
Uncaught (in promise) Error: [-32000] Server error: account 7e4c8cf56dda30a6bb00bd3059d429e5cd760876.lockup.near does not exist while viewing

@vgrichina Oh, by the way, will it work correctly when viewing the lockup account itself (instead of the owner of the lockup account)?

@vgrichina Looks like accounts without lockup also triggering an error: https://near-mainnet-staging-wallet-pr-1225.onrender.com/profile/5b4378098c67c414022d65d1d0feb1943e76017acee696a511b5aa960d64e35a
Uncaught (in promise) Error: [-32000] Server error: account 7e4c8cf56dda30a6bb00bd3059d429e5cd760876.lockup.near does not exist while viewing

yeah, somehow it worked in my local deploy (usual story from eng lol :D)
I'm debugging

@vgrichina Oh, by the way, will it work correctly when viewing the lockup account itself (instead of the owner of the lockup account)?

it just shows it as regular account:
https://near-mainnet-staging-wallet-pr-1225.onrender.com/profile/5082a3aeb4203818da5a217d53b6365e5cd31b95.lockup.near

@vgrichina I would find it confusing, especially when the native tokens are delegated from a lockup contract. What is your opinion on it?


Vector shouldn't expose Directory when trying to scroll past the bottom of the room list

When scrolling past the top of the room list, Vector exposes the "Browse directory" option at the top of the room list. This is normal and expected.
However, the same thing happens when trying to scroll past the bottom of the room list, causing a jump in view position.

https://github.com/vector-im/vector-android/commit/783343bb56e8ea9549728ce01b2a737a0f6695df


Fix data not controlled by var.enabled

Closes #9

Done @aknysh
There was an error when running make readme. I have to install gomplate manually to workaround it.
❯ make readme
curl --retry 3 --retry-delay 5 --fail -sSL -o /Users/yujunz/Code/github.com/cloudposse/terraform-aws-kops-metadata/build-harness/vendor/terraform-docs https://github.com/segmentio/terraform-docs/releases/download/v0.4.5/terraform-docs-v0.4.5-darwin-amd64 && chmod +x /Users/yujunz/Code/github.com/cloudposse/terraform-aws-kops-metadata/build-harness/vendor/terraform-docs
make: gomplate: No such file or directory
make: *** [readme/build] Error 1

Is it possible to have GitHub Actions to take the chore of formatting and updating, e.g. suggest changes automatically after running terraform fmt and make ....


File flake issues part 1

Next step is to manage github issues. Right now this produces a page with the following output:
[
  {
    "Job": "kubernetes-e2e-gce",
    "Number": 17234,
    "Test": "[k8s.io] Kubectl client [k8s.io] Kubectl logs should be able to retrieve and filter logs [Conformance] {Kubernetes e2e suite}",
    "Reason": "/go/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/kubectl.go:940\nMay 19 16:16:10.333: Verified 0 of 1 pods , error : timed out waiting for the condition"
  },
  {
    "Job": "kubernetes-e2e-gce-scalability",
    "Number": 7615,
    "Test": "[k8s.io] Load capacity [Feature:Performance] should be able to handle 30 pods per node {Kubernetes e2e suite}",
    "Reason": "/go/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/load.go:71\nExpected\n    \u003cint\u003e: 1\nnot to be \u003e\n    \u003cint\u003e: 0"
  }
]

I think the tests caught all my bugs.

lint errors fixed.

Sigh, need to vendor gofuzz...

OK. tests all pass now.

Closing this since everything is in #1031


Goenrichment

A tutorial for GO Enrichment analysis.

Nice tutorial! I didn't know you could do this type of GO analysis in Galaxy - simplifying GO terms and using GO slims.
I'm just going to suggest some minor edits and have one comment, I noticed you've got screenshots of the parameters but I think the preference (requirement?) here is to list the parameters instead of screenshots (e.g. https://github.com/galaxyproject/training-material/pull/1116) but would be good if someone could confirm that before you edit.

Looking good to me @igcbioinformatics. I just added a few small edits, see what you think.
One other comment, there's quite a bit of file renaming e.g. the last step has 12 files to rename one-by-one, so wondering if another option could be to use the hashtags e.g if you add tags like #overexpressed and #underexpressed to the files with the over and underexpressed genes, the output files would then be tagged accordingly, if you wanted to save time on renaming.

Yes, avoiding renaming would be good. I have to see how these tags work. Thanks!

@igcbioinformatics @mblue9 that awesome! Thanks a lot!


Make sure AutoCloseable resources are closed

This is the result of auditing the codebase for resource leaks.  Includes
many fixes to unit tests that are likely unnecessary, but included for the sake
of cleanliness
Possibly related to #126

@birkland I'll try and test this in isolation before https://github.com/Islandora-CLAW/Alpaca/pull/42 to see if I can pin down precisely what's causing our issue.

@dannylamb Great. Jenkins should finish and publish its snapshot build of API-X in a few minutes

So this hasn't resolved the issue, but it has turned it from intermittent to easily reproducable.  :+1: to that.

That seems strange, but I'll take it.  Also taking another look at the initial CLAW issue..


not always snipe vip pokemon if using snipe_high_prio_threshold

Short Description:

I think we don't need to always snipe vip pokemon, if that pokemon is needed for sniping, it should have higher priority than threshold. By that we can sure vip pokemon is till catch with Berry & Best Ball but no need to snipe (avoid ban)

Snipe and vip should not merged in configuration.
The snipe score is very clear.

@solderzzc how about the case when I want Dratini is Vip to secure catch rate but don't want to snipe on Dratini?

Hmm, I don't get it. But I realized one reason for this commit, catch with Berry & Best Ball

Yes, it is. I want that pokemon is catch with Berry & Best Ball but not snipe on it

Oh, I was thinking the config is in the reverted way.

Why you just don't enable the Snipe task?

I want snipe on other pokemons.
Example:
my vip pokemon is Dratini, Dragonite because I want to catch them with berry & best ball. But I only want snipe on Dragonite

I see, that's reasonable.


Amolexcompany patch 1

Coverage remained the same at ?% when pulling e943281f9429130b1842982fd5dea9607a50926f on amolexcompany:amolexcompany-patch-1 into baf532e1b143525bc4ef845072b9e073f9d58be0 on xtaci:master.


feat: tune default start/stop sequence

load balancer health check will be performed every 5 seconds (previously every 30 seconds), this will speed up the default minimum time for a container to be considered ready from 90 seconds to 15 seconds and the time to detect unhealthy tasks.
load balancer deregistration delay (how long the load balancer keeps a task in draining state) is lowered to 60 seconds (previously 5 minutes) as common applications don't need to maintain connection for such a long time. This value should be aligned with the idle timeout (how long a backend connection is kept alive waiting for a backend response) which already has a default value at 60 seconds.
default grace period for ECS task is set to 30 seconds so tasks are not prematurely killed during start up (was previously unset and the default is 0 second, this parameter would have protected demeter task start up during incident #127). With this settings a new task has up to 45 seconds to answer the health check before getting killed (30 seconds of grace period + 3 consecutive health check failure)
stopTermination parameter can now be overridden so we can control how long application have to exit after receiving SIGTERM (default is set to 5 seconds). It's important to note that this timeout only applies after the deregistration of the task from the loadbalancer (see the task lifecyle diagram below).

ECS task lifecycle

Configured values for critical apps need to be reviewed before merging this PR:

data-capture
demeter
bali

I did a quick check on Bali and Demeter, Bali uses only the default values for those variables so it seems ✅ .
Demeter has a custom value of 360s for the private_lb_idle_timeout which may conflict with the deregistration_delay if it stays at 60s.
The 360s seem too much but at the same time I see requests to /forecasts/models lasting up to 7 minutes which means the data probably never comes back to the application...?

idle timeout have been aligned to 60 seconds on demeter: https://github.com/sencrop/sencrop-demeter-api/pull/790


Sylvanie85/issue74

Api endpoints and tests for batch import
Related to issue #74

Make sure to name your issue with the relevant title!


Add 30 day availability to status page

Currently the status page shows only the 24 hour availability. This is neither what usually is expected, nor it is configurable currently.
rn, this is just a hack, as it is a env var based setting and not something that can be configured in the frontend. But people waiting for https://github.com/louislam/uptime-kuma/issues/1888 might find this helpful.
We can see to complete a feature for this, however I haven't looked too deep at the frontend yet.

As I wrote, this is ment rather a hack than a contribution yet. And yes I know the PR description is there for a reason. But I have a good reason to delete it when I write a custom explanation.
All you wrote I already mentioned. This is a fast way to get the functionality, that should be there (b/c on a status page a 24 hour availability is really weird...). So PR'ing this already is for the community, who can take it until this is solved for good. Working on it for real may be on the agenda soon, just not now.

this is ment rather a hack than a contribution

I am going to close this PR as I don't see this PR getting merged in this state.

i also think 24 hours are too short，would implement this features by official

@joebnb
Please refer to https://github.com/louislam/uptime-kuma/issues/1888#issuecomment-2068191172 for the status on the issue and how to get involved in this effort ^^


Create create.js

Dependent on #1 , #2 , #13, #14

This file has four jobs.

Check user input
Convert user input into a routine object.
Incorporate new routine object into data
Send updated data back to python

[ ] Load routine data into an object
[ ] Create function to check input is appropriate (has title, has steps)
[ ] Create function to collate information into routine object
[ ] Add new routine object to routine array
[ ] Add updated routine array to the data object
[ ] Send POST request to server (/data) with the entire data object in JSON format
[ ] Handle positive response - if success- put a pop up that data was updated and go back to homepage (/app)
[ ] Handle negative response - if unsuccessful - put a pop up with the issue or that something has gone wrong and it didn't save. Remain on the same page.

Would be nice to offer ability to reorder steps while creating routine, but save for later iteration


[SPARK-35697][SQL][TESTS] Test TimestampWithoutTZType as ordered and atomic type

What changes were proposed in this pull request?
Add TimestampWithoutTZType to DataTypeTestUtils.ordered/atomicTypes, and implement values generation of those types in LiteralGenerator/RandomDataGenerator. In this way, the types will be tested automatically in:

ArithmeticExpressionSuite:

"function least"
"function greatest"

PredicateSuite

"BinaryComparison consistency check"
"AND, OR, EqualTo, EqualNullSafe consistency check"

ConditionalExpressionSuite

"if"

RandomDataGeneratorSuite

"Basic types"

CastSuite

"null cast"
"up-cast"
"SPARK-27671: cast from nested null type in struct"

OrderingSuite

"GenerateOrdering with TimestampWithoutTZType"

PredicateSuite

"IN with different types"

UnsafeRowSuite

"calling get(ordinal, datatype) on null columns"

SortSuite

"sorting on TimestampWithoutTZType ..."

Why are the changes needed?
To improve test coverage.
Does this PR introduce any user-facing change?
No
How was this patch tested?
By running the affected test suites.

This one is to follow https://github.com/apache/spark/pull/31782

Test build #139574 has started for PR 32843 at commit 6a4fc48.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44101/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44101/

thanks, merging to master!

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/44101/


Current date/time icon

Icon name
calendar-current, clock-current
Use cases
For a time/date management component that user will be able to click button ant scroll/select to the now/today/current.
Design ideas
It's a combination of a calendar, clock with flag-triangle that means "here/now" in for the date/time.
It's other than "selection/check"
Checklist

[X] I have searched if someone has submitted a similar issue before and there weren't any. (Please make sure to also search closed issues, as this issue might already have been resolved.)
[X] I have searched existing icons to make sure it does not already exist and I didn't find any.
[X] I am not requesting a brand logo and the art is not protected by copyright.
[X] I am not requesting an icon that includes religious, political imagery or hate symbols.
[X] I have provided appropriate use cases for the icon(s) requested.

I'm not sure we can fit that flag with optimal legibility within either calendar or clock:

calendar-dot may be a valid alternative design:

However, as far as the use case itself is concerned, from an accessibility point of view I'd suggest just using "today" as the label of this element without any further iconography (as is done in pretty much all major calendar apps).

Dot is fine i gues. Also the use case is that i have a list of dates/times and i want to mark them Past/Future/Now. Where for now none of existing icons fit for me.
For past - calendar-arrow-down, future - calendar-arrow-up.... Today/now none of other icons seems to tell me "we are here".


:robot: Triggering CI on branch 'release-next' after syncing to upstream/main

Seems infra.
/test 415-test-e2e-aws-415

OCF Webhook is merging this PR


fix: project 25 + 5

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

@moT01 @scissorsneedfoodtoo @raisedadead @ojeytonwilliams Read about this PR in the team chat.

I did some digging and found that the titles are turned into "start case" here - which seems to be removing the +. Removing that method fixes the issue. I looked around and didn't find and negative side effects of removing the method.

@moT01 Per https://github.com/lodash/lodash/issues/3383, it appears that the method does remove symbol characters.
@ojeytonwilliams @moT01 The lodash startCase method was implemented to make sure each first letter of a word was capitalized.  We could use the workaround suggested in the GitHub issue (above) for that repo and use something like the following for this line of our code:
{name.replace(/\w+/g, _.capitalize)}

Let me know and I can push another commit to this PR.

Nice catch, Tom.

Let me know and I can push another commit to this PR.

Sure, that looks like a good fix.

We had to end up reverting back to using startCase due to the issue described here once PR #39544 was merged.  In order to get these changed merged faster, we will temporarily live with the issue pointed out by Tom above for this one particular challenge.


Up And Down Arrow Blocked On Paged Output

[x] I am on the latest Hyper.app version
[x] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name: Windows 10 Enterprise 1903
Hyper.app version: hyper --version: The option "version" is unknown but I installed 3.0.2
Link of a Gist with the contents of your .hyper.js: https://gist.github.com/dagrooms52/9a6d216d0d5ebe9de0940407ef93704d
Relevant information from devtools (CMD+ALT+I on macOS, CTRL+SHIFT+I elsewhere): N/A
The issue is reproducible in vanilla Hyper.app: Using shell Git Bash

Issue

Set shell to Git Bash
Make a few edits in a git repo
git diff in Hyper terminal at repo
up and down arrows will not scroll the diff. PgUp and PgDn work as expected.
up and down arrows do work in Git Bash to scroll git diff output

The functionality works in a Vanilla Hyper shell on technicality; Windows 10 cmd also will not scroll a git diff, so not communicating up and down input to the shell produces the same functionality.

Sure, any paging command works for repro. This was the only one I used on a daily basis that makes Hyper frustrating to use.
Side note, the man page for less lists multiple ways to walk one line at a time; I'm using enter in place of down arrow and k in place of up arrow. My beard just turned one shade more grey; the light of the unix gods shines upon my terminal.


TARGET parameter no longer valid in CAS 5.1.5 for service login or ticket validation (affects CAS integration in Ellucian Banner 9)

Just wanted to point out that under CAS 5.1.5, applications that used to pass their service URLs via the TARGET parameter are now broken while this still works in CAS 5.1.1.
Login URL (example): https://cas.example.com/login?TARGET=...
Under CAS 5.1.1:
AUTHENTICATION_EVENT_TRIGGERED
SERVICE_TICKET_CREATED
SERVICE_TICKET_VALIDATED

AUTHENTICATION_EVENT_TRIGGERED
SERVICE_TICKET_CREATED
SERVICE_TICKET_VALIDATED

Under CAS 5.1.5:
AUTHENTICATION_EVENT_TRIGGERED
SERVICE_TICKET_CREATED

AUTHENTICATION_EVENT_TRIGGERED
SERVICE_TICKET_CREATED

Users are merely presented with the "Login successful" page and CAS does not perform any service redirect nor receives service ticket validation of the same.
For this reason, we will not be deploying CAS 5.1.5 as it breaks CAS integration in Ellucian Banner 9.

Thanks for the logging the issue report. This seems like something that might need fixing. As the next step, please review this guide to get started.
You can also learn how to build and run CAS from here.
Thanks again!


[layer] Support get weights

Support getWeights functionality equivalent to getGradients
Resolves #232
Signed-off-by: Parichay Kapoor<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #238. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.


Potential data race

Hi,
https://github.com/lotusdblabs/lotusearch/blob/main/searcher/engine.go#L540C21-L540C21
incrementing a counter concurrently without using lock/atomic inc.
you can assign this to me and ill be happy to open a PR

good job -:)

@amityahav  done. can you help to code review? i try to fix it in short time,if you have better advise ,tell me, please. THX :-)

It looks like this project is not maintained and is a fork of GoFound . Anyways I would use an atomic counter instead of mutex

yeah，i create PR but no one CR。 so，it is here.
"Anyways I would use an atomic counter instead of mutex" ,good idea,I thoght about it. but finanly abort it. i think it is a thankless thing,.of course just do it.You can try,hope you can get success.


Julia implementations

Two implementation in Julia based on the Python code.
I tried to be as close as possible as the python code.
Om my CPU, naive implementation takes 1.2s and the one based on numpy takes 450ms

Hey, thanks for the PR, could you modify the code to just execute the script and not run the benchmark? I know nothing about julia so it'd be great if you edited the run.sh file to include a function for it.

Sure, I will update the run.sh

Hey, also remember to install the deps in the script. Similar to how the run_python_np function is done.

is there a way to install the needed packages from the command line? I had to open the julia shell in stall them

I updated the script and the run.sh to install the packages


Relocation manager should relocate segments from any servers (irrespective of the tag) to completed servers

The relocation manager was only moving segments which were on consuming servers to the completed servers. This would not work if the segments were for some reason on servers which are not the "consuming" servers according to the RealtimeTagConfig (if consuming tag was changed, and then relocation performed)
This PR ensures that no matter where the segments are, they will be moved if they are not on "completed" tagged servers

Codecov Report

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

@@           Coverage Diff            @@
##             master   #3466   +/-   ##
========================================
  Coverage          ?   70.5%
  Complexity        ?       4
========================================
  Files             ?     999
  Lines             ?   45671
  Branches          ?    6109
========================================
  Hits              ?   32200
  Misses            ?   11358
  Partials          ?    2113

Impacted Files
Coverage Δ
Complexity Δ

...elix/core/relocation/RealtimeSegmentRelocator.java
80.2% <100%> (ø)
0 <0> (?)

Continue to review full report at Codecov.

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


error description should be in the 2nd parameter

found this issue when there is no description on the error thrown. turns out that the description is there, but in the incorrect parameter.

This is actually deliberate - the idea is that it's sent to your error handler/logs but not sent to the user :)


wd_hepler.au3 - tidy code

Pull request
Proposed changes
Tidy code before next PR
Checklist
Put an x in the boxes that apply. If you're unsure about any of them, don't hesitate to ask. We are here to help!
This is simply a reminder of what we are going to look for before merging your code.

[x] I have read and noticed the CODE OF CONDUCT document
[x] I have read and noticed the CONTRIBUTING document
[ ] I have added necessary documentation or screenshots (if appropriate)

Types of changes
Please check x the type of change your PR introduces:

[ ] Bugfix (change which fixes an issue)
[ ] Feature (change which adds functionality)
[x] Code style update (formatting, renaming)
[ ] Refactoring (functional, structural)
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?
not tidied code
What is the new behavior?
tidied code
Additional context
none
System under test
not related

@mlipok You are living up to your nitpicky ways. 😝

Yeah ;)

@mlipok You are living up to your nitpicky ways. 😝

Hope now is better PR ;)

I will take a look again here after you finish... release version with OPERA support.

btw.
I recomend to firstly merge https://github.com/Danp2/au3WebDriver/pull/244
Then I suplement with $iExt in this PR here:
	Return SetError(__WD_Error($sFuncName, $iErr, $sMessage, $iExt), $_WD_HTTPRESULT, $vResult)
EndFunc   ;==>_WD_ElementSelectAction


Support parameterized dynamics

[x] implement basic functionality
[x] update docstrings
[x] add tests

Lgtm


lints: implement json.Unmarshaler for LintStatus.

The lints.LintStatus type implements json.Marshaler to marshal to a human readable string (e.g. "error" instead of 6). Without providing a compatible json.Unmarshaler implementation downstream users that marshal a lints.LintStatus to JSON will encounter an error when they try to unmarshal it due to a mismatch of types (int vs string).
Notably this is causing me problems because when I added pre-issuance linting to CFSSL I made the configuration value on the cfssl.config.SigningProfile type a zlint.lints.LintStatus after a suggestion by the maintainers. That seemed fine at the time but I later discovered in Boulder we expect to be able to round-trip our CFSSL config to/from JSON without error.
Without the changes in this branch the de-serialization of the CFSSL config fails with an error like:

Failed to create CA: {"code":5200,"message":"failed to unmarshal configuration: json: cannot unmarshal string into Go struct field SigningProfile.lint_error_level of type lints.LintStatus"}

@zakird Thanks for the quick :mag:


add option for "real" dictionaries to improve performance.

By default, TicDatFactory tables are rendered as super-powered dictionaries. You can either freeze the dictionary, and thus create read-only tables, or render tables as something equivalent to a defaultdict, in which case added rows will read from the default values to populate otherwise unset data values.
Now, all this nice stuff comes at a performance price. For example, reading a (row, field) pair from a TicDat dict object is around 10X slower than reading from real dictionary-of-dictionaries (but of course, still 100X faster than reading (row, field) from a pandas.DataFrame using .loc).
This issue tracks the feature of changing the default TicDat rendering of tables for a specific TicDatFactory. It should be a pretty easy feature to implement, as all it requires is by creating true dict objects for the row of each table.

This issue was moved to ticdat/ticdat#2


Update to JSpecify 0.3-alpha-2

Change all occurrences of org.jspecify.nullness to org.jspecify.annotations to match the new package structure.  Note that this change only impacts test code; the main NullAway code does not (yet) have any JSpecify dependence.

Pull Request Test Coverage Report for Build #1004

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 92.875%

Files with Coverage Reduction
New Missed Lines
%

../nullaway/src/main/java/com/uber/nullaway/dataflow/AccessPath.java
1
95.24%

Totals

Change from base Build #1003:
-0.02%

Covered Lines:
5123

Relevant Lines:
5516

💛  - Coveralls


Look into changing sql-native lib as external library

Description:
This is used in all the database connectors, however, we need to define this library in the toml file as a dependency for the specific module again.
We have to change such that defining the dependency again is removed in all db modules.

This was suggested since there can be version clashes however after project api change this is invalid


Generate type hints for Python code

I.e. https://www.python.org/dev/peps/pep-0484/#suggested-syntax-for-python-2-7-and-straddling-code

Make this an epic with the following task:

Step 0: introduce mypy in testing and CI https://github.com/Azure/autorest.python/issues/82
Step 1: full annotation on Python 3 only files https://github.com/Azure/autorest.python/issues/83
Step 2: full annotations on everything with Py 2 compat. Let's wait until step 1 to discuss if this is needed of if we move to Python 3 only. :)


Fix CFInfo on 64-bit big endian platforms.

Prior to this change 64-bit platforms treated _cfinfoa as both a
64-bit and 32-bit variable. When accessing _cfinfoa as a 32-bit
variable the code makes the assumption that it is accessing the
lowest (or rightmost) bits of _cfinfoa. Unfortunately on big-endian
platforms it is actually accessing the high (or leftmost) bits.
Fix this issue by unconditionally treating _cfinfoa as a 64-bit
value when targeting a 64-bit platform.

Seems reasonable. Want to make sure this lines up with the work @millenomi is doing to merge right now.

In addition to the specific comment, I'll point out that #1689 already fixes _CFInfo in a cleaner way.

As soon as I'm finished with the merge, I want to sit down and figure out what is the best solution here. #1694 we want for sure, and let's chat about the definition in CFRuntimeBase.

Yeah, sorry; it's just been a bit of bad timing at the moment because of the behind-the-scenes work for the merge.
You can contact me via private message on the Swift forums if necessary!

And I can also see to set up something for multiple people if needed.

Sounds good.
I have had trouble setting up appropriate Slack stuff, but if you’re all comfortable with iMessage, please send me your contact (mention your GitHub username) via iMessage at<EMAIL_ADDRESS>and I’ll figure out the rest. (If that’s not okay either, I’ll figure out another way.)

Worked for me.

@swift-ci Please test.

Hi, I've brought this PR up to date with the changes upstream. Can you take another look? Thanks!

Friendly ping.

friendly ping

ping

The Swift project moved the default branch to main and deleted master branch, so GitHub automatically closed the PR. Please re-create pull request with main branch.
More detail about the branch update - https://forums.swift.org/t/updating-branch-names/40412


Improve process to bring up separate, independent stratos db

bringing up a separate, independent stratos db is a bit of a struggle
currently

build and run mariadb image /deploy/db/Dockerfile.mariadb
create empty stratos db (via cannibalising deploy/ci/automation/cfpushtest.sh)
add development section to deploy/db/dbconf.yml with correct connection string
run migrate using /deploy $ ../src/jetstream/jetstream up
update config.properties with the required env vars

we should be able to improve some of these steps.

script to bring up mariadb image with required admin password, run create/setup database sql commands
dbconf.yml could be removed (?)

This is much easier now the db config has been simplified/refactored.


KYLIN-5166 Use SQL hint to specify the cube that the user wants to query

Proposed changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue.
Github Branch
As most of the development works are on Kylin 4, we need to switch it as main branch. Apache Kylin community changes the branch settings on Github since 2021-08-04 :

The original branch kylin-on-parquet-v2 for Kylin 4.X (Parquet Storage) has been renamed to branch main, and configured as the default branch;
The original branch master for Kylin 3.X (HBase Storage) has been renamed to branch kylin3 ;

Please check Intro to Kylin 4 architecture and INFRA-22166 if you are interested.
Types of changes
What types of changes does your code introduce to Kylin?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have create an issue on Kylin's jira, and have described the bug/feature there in detail
[ ] Commit messages in my PR start with the related jira ID, like "KYLIN-0000 Make Kylin project open-source"
[ ] Compiling and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged

Further comments
If this is a relatively large or complex change, kick off the discussion at user@kylin or dev@kylin by explaining why you chose the solution you did and what alternatives you considered, etc...

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@121a8a1). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1825   +/-   ##
=======================================
  Coverage        ?   24.61%
  Complexity      ?     4446
=======================================
  Files           ?     1097
  Lines           ?    61985
  Branches        ?     8890
=======================================
  Hits            ?    15259
  Misses          ?    45115
  Partials        ?     1611

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 121a8a1...8bea146. Read the comment docs.


Cannot read compressed files using multithreading

A sample CSV file:
1,23.9
2,5.6

compressed using hadoop-xz yields:
$ hexdump -C out/000000_0.xz
00000000  fd 37 7a 58 5a 00 00 04  e6 d6 b4 46 02 00 21 01  |.7zXZ......F..!.|
00000010  16 00 00 00 74 2f e5 a3  01 00 0c 31 2c 32 33 2e  |....t/.....1,23.|
00000020  39 0a 32 2c 35 2e 36 0a  00 00 00 00 dc 43 5b 17  |9.2,5.6......C[.|
00000030  b5 3f 3f e0 00 01 25 0d  71 19 c4 b6 1f b6 f3 7d  |.??...%.q......}|
00000040  01 00 00 00 00 04 59 5a                           |......YZ|
00000048

Same file, compressed manually using a single core yields the same result
$ /software/astro/sl6/xz/5.2.2/bin/xz csv/1.csv -c | hexdump -C
00000000  fd 37 7a 58 5a 00 00 04  e6 d6 b4 46 02 00 21 01  |.7zXZ......F..!.|
00000010  16 00 00 00 74 2f e5 a3  01 00 0c 31 2c 32 33 2e  |....t/.....1,23.|
00000020  39 0a 32 2c 35 2e 36 0a  00 00 00 00 dc 43 5b 17  |9.2,5.6......C[.|
00000030  b5 3f 3f e0 00 01 25 0d  71 19 c4 b6 1f b6 f3 7d  |.??...%.q......}|
00000040  01 00 00 00 00 04 59 5a                           |......YZ|
00000048

But compressing it using multiple threads yields somewhat different file that haddop-xz is unable to read and fails with a:

java.io.IOException: XZ Stream Footer is corrupt

$ /software/astro/sl6/xz/5.2.2/bin/xz csv/1.csv -T0 -c | hexdump -C
00000000  fd 37 7a 58 5a 00 00 04  e6 d6 b4 46 04 c0 11 0d  |.7zXZ......F....|
00000010  21 01 16 00 00 00 00 00  00 00 00 00 88 88 cd 68  |!..............h|
00000020  01 00 0c 31 2c 32 33 2e  39 0a 32 2c 35 2e 36 0a  |...1,23.9.2,5.6.|
00000030  00 00 00 00 dc 43 5b 17  b5 3f 3f e0 00 01 2d 0d  |.....C[..??...-.|
00000040  79 93 1d 7e 1f b6 f3 7d  01 00 00 00 00 04 59 5a  |y..~...}......YZ|
00000050

:+1:
I've seen this issue as well.
Worth noting that multi threading xz has been implemented in xz > 5.0

Forcing a newer version of https://search.maven.org/search?q=g:org.tukaani AND a:xz&core=gav does not fix this.

Seems like this stems from the XZ library under the hood. From it's official page:

Single-threaded streamed compression and decompression and random access decompression have been fully implemented. Threading is planned but it is unknown when it will be implemented.

Glancing at the code and at https://tukaani.org/xz/xz-file-format-1.0.4.txt I suspect this is not related to multithreaded decompression, but bad handling in io.sensesecure.hadoop.xz.XZSplitCompressionInputStream.nextStreamOffset() when the optional "Compressed Size" field is present in the initial block header. At first glance, xz only includes it when enabling multithreads ('80' in the first block), and there is a different code path when the size is not found.
$ xz -1e -T2 -k /tmp/allmsg
$ head -c16 /tmp/allmsg.xz | hexdump -C
00000000  fd 37 7a 58 5a 00 00 04  e6 d6 b4 46 04 c0 80 9c  |.7zXZ......F....|
$ xz -1e -T1 -k /tmp/allmsg
$ head -c16 /tmp/allmsg.xz | hexdump -C
00000000  fd 37 7a 58 5a 00 00 04  e6 d6 b4 46 02 00 21 01  |.7zXZ......F..!.|

This chunk looks pretty suspicious:
    // Stream Footer
    inData.readFully(streamFooterBuf);
    if (streamFooterBuf[10] == XZ.FOOTER_MAGIC[0] && streamFooterBuf[11] == XZ.FOOTER_MAGIC[1] && DecoderUtil.isCRC32Valid(streamFooterBuf, 4, 6, 0)) {
        throw new IOException("XZ Stream Footer is corrupt");
    }

    long streamOffset = ((Seekable) seekableIn).getPos();
    ((Seekable) seekableIn).seek(offset);
    return streamOffset;


新支持两个控制函数

Codecov Report
Base: 83.92% // Head: 84.46% // Increases project coverage by +0.54% :tada:

Coverage data is based on head (55651a3) compared to base (50407a9).
Patch coverage: 100.00% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master       #3      +/-   ##
==========================================
+ Coverage   83.92%   84.46%   +0.54%
==========================================
  Files           6        6
  Lines         255      264       +9
==========================================
+ Hits          214      223       +9
  Misses         35       35
  Partials        6        6

Impacted Files
Coverage Δ

json/json.go
83.24% <100.00%> (+0.82%)
:arrow_up:

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.


convert data export field names to use underscores instead of spaces

testing/review notes
Check that field names in CSV and JSON for all exports now use underscores instead of spaces; sanity check that data still looks correct.

Sorry @rlskoeser but can you give me an example of a field name so I know where to look?

Ha, sorry @clmahoney ! By field name, I mean the labels of the fields in the data: that is, the header row in the CSV files, and the text before the : in the JSON files.

I feel like I'm still not looking in the right place but I don't see any spaces OR underscores!

@clmahoney So sorry I gave you the wrong version of the exports, you're right that there weren't any underscores! Here is a new set of exports that actually reflect the change I wanted you to check.

YES, I checked that the field names in all data exports in both CSV and JSON formats have underscores instead of spaces!


Added support to exclude nested keys

Hello,
I like your plugin, but I'm missing a way to define nested keys that I would like to be excluded from deep.equal() comparison.
What do you think about the change? Could you approve it?

Hello @buksy90, thank you for putting an effort to create the PR. I haven't gone through your code yet but did look at the tests and I was thinking if this is even necessary?
Why would you not assert the nested key separately and still use excludingEvery?
For example,
      const { a: actual }  = { a: { a1: 'a1', a2: 'a2' }, b: 'b', c: 'c' }
      const expected = { a2: 'a2' }

      assert.deepEqualExcludingEvery(actual, expected, 'a1')

      // Or you can assert the same object with a1 in it but will be excluded from comparison
      assert.deepEqualExcludingEvery(actual, actual, 'a1')

Hi, I have some communication service that sends and receives messages, I need to assert that message has correct format. There are multiple message types supported and so I have multiple tests. I'm using before() hook to setup the instance and then use that instance in every test. Every message contains id and that id depends on the order of message.
I would like to have my tests independent from the order of execution plus I dont really care about the message id in these tests, so I would like to use nested key to ignore this property.
But I cannot use *excludingEvery because message data can contain some other structures with id properties.
here is an example of my message structure
interface Message<T> {
  type: MessageType,
  messageKey: { id: string, initiatior: string },
  data: T
  ...
}

Did I answer your question?

Closing because issue is stale.


Undefined class 'AccessToken'

In your example you have a class AccessToken but there is no such class while doing the real implementation

Hi @Novinyo, you are right, I just updated the README.
The actual class name is AccessTokenResponse!


Day 454

Good morning! Here's your coding interview problem for today.
This problem was asked by Facebook.
Describe an algorithm to compute the longest increasing subsequence of an array of numbers in O(n log n) time.

Identical to Day 364: https://github.com/vaskoz/dailycodingproblem-go/issues/718


Set voice search feature to disabled by default

Task/Issue URL: https://app.asana.com/0/0/1205628562025903/f
Description
See title
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes
No UI changes

Current dependencies on/for this PR:

develop

PR #3477

PR #3583

PR #3585

PR #3587

PR #3607

PR #3608   👈

PR #3574
PR #3489

PR #3516
PR #3491
PR #3490

PR #3528

PR #3541

PR #3564

PR #3567

PR #3550

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

develop

PR #3477

PR #3583

PR #3585

PR #3587

PR #3607

PR #3608   👈

PR #3574
PR #3489

PR #3516
PR #3491
PR #3490

PR #3528

PR #3541

PR #3564

PR #3567

PR #3550

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

develop

PR #3477

PR #3583

PR #3585

PR #3587

PR #3607

PR #3608   👈

PR #3574
PR #3489

PR #3516
PR #3491
PR #3490

PR #3528

PR #3541

PR #3564

PR #3567

PR #3550

This comment was auto-generated by Graphite.


"User is typing" does Flyer chat support this ?

Does Flyer chat support "typing" or some animation while the other user is typing or some flag to enable it?

Yes, you need to provide typingIndicatorOptions see the implementation with comments https://github.com/flyerhq/flutter_chat_ui/blob/main/lib/src/widgets/typing_indicator.dart#L430


"weed server" peer check thinks it's a master when master has been disabled.

Describe the bug
I'm getting Only odd number of masters are supported: [node1:9333 node3:9333 node4:9333 node2:9333] on weed server -options=/etc/xdg/seaweedfs/server.conf -ip=node2 when there is a master=false flag in the config file. Node2 shouldn't be counted as a master.
I suspect it's the if *isStartingMasterServer { here removed that's causing the bug https://github.com/chrislusf/seaweedfs/commit/e5fc35ed0c970fea060a5b3b7a3f5efb5af425d6#diff-25bae991aa8050efe57406452575e53bb89eb91244dcfa772587872b05f862aeL170
Relevant logs
weed server -options=/etc/xdg/seaweedfs/server.conf -ip=node2 log
Oct 03 20:17:22 node2 systemd[1]: Started Seaweed Master.
Oct 03 20:17:22 node2 weed[1015247]: I1003 20:17:22 15247 master.go:170] current: node2:9333 peers:node1:9333,node3:9333,node4:9333
Oct 03 20:17:22 node2 weed[1015247]: F1003 20:17:22 15247 master.go:186] Only odd number of masters are supported: [node1:9333 node3:9333 node4:9333 node2:9333]
Oct 03 20:17:22 node2 weed[1015247]: goroutine 1 [running]:
Oct 03 20:17:22 node2 weed[1015247]: github.com/chrislusf/seaweedfs/weed/glog.stacks(0x0)
Oct 03 20:17:22 node2 weed[1015247]:         github.com/chrislusf/seaweedfs/weed/glog/glog.go:767 +0xc5
Oct 03 20:17:22 node2 weed[1015247]: github.com/chrislusf/seaweedfs/weed/glog.(*loggingT).output(0x5581706f8620, 0x3, 0xc000262310, {0x55816f9531f3, 0x9}, 0xba, 0x0)
Oct 03 20:17:22 node2 weed[1015247]:         github.com/chrislusf/seaweedfs/weed/glog/glog.go:718 +0x465
Oct 03 20:17:22 node2 weed[1015247]: github.com/chrislusf/seaweedfs/weed/glog.(*loggingT).printf(0x5581706f8620, 0x3, {0x55816eaf1622, 0x2d}, {0xc000be38a8, 0x1, 0x1})
Oct 03 20:17:22 node2 weed[1015247]:         github.com/chrislusf/seaweedfs/weed/glog/glog.go:656 +0x1bb
Oct 03 20:17:22 node2 weed[1015247]: github.com/chrislusf/seaweedfs/weed/glog.Fatalf({0x55816eaf1622, 0x2d}, {0xc000be38a8, 0x1, 0x1})
Oct 03 20:17:22 node2 weed[1015247]:         github.com/chrislusf/seaweedfs/weed/glog/glog.go:1149 +0x57
Oct 03 20:17:22 node2 weed[1015247]: github.com/chrislusf/seaweedfs/weed/command.checkPeers({0x7ffd910c4f15, 0x5}, 0x2475, 0x0, {0xc00013436d, 0x20})
Oct 03 20:17:22 node2 weed[1015247]:         github.com/chrislusf/seaweedfs/weed/command/master.go:186 +0x42c
Oct 03 20:17:22 node2 weed[1015247]: github.com/chrislusf/seaweedfs/weed/command.runServer(0x5581706165a0, {0xc00004e0b0, 0x0, 0x0})
Oct 03 20:17:22 node2 weed[1015247]:         github.com/chrislusf/seaweedfs/weed/command/server.go:174 +0x388
Oct 03 20:17:22 node2 weed[1015247]: main.main()
Oct 03 20:17:22 node2 weed[1015247]:         github.com/chrislusf/seaweedfs/weed/weed.go:78 +0x5b3
Oct 03 20:17:22 node2 systemd[1]: seaweedfs.service: Main process exited, code=exited, status=255/EXCEPTION
Oct 03 20:17:22 node2 systemd[1]: seaweedfs.service: Failed with result 'exit-code'.
Oct 03 20:17:22 node2 systemd[1]: seaweedfs.service: Scheduled restart job, restart counter is at 5.
Oct 03 20:17:22 node2 systemd[1]: Stopped Seaweed Master.

System Setup

Arch linux
version 30GB 2.70  linux amd64
server.conf:

master=false
master.defaultReplication=001
volume.dir.idx=index
dir=/brick/0
filer=true
filer.peers=node1:8888,node2:8888,node3:8888,node4:8888
master.volumePreallocate=true
master.peers=node1:9333,node3:9333,node4:9333
rack=rack1
dataCenter=dc1
volume.max=100
volume.concurrentUploadLimitMB=999999
filer.concurrentUploadLimitMB=999999
metrics.address=node3:9091

Expected behavior
The server should not be counted as a master on master=false during peer checking.

Thanks a lot for the bug report and debugging!

@chrislusf Thanks for the fix, however I might be unclear on what I meant, the bug comes from the checkPeers() call right after your fix https://github.com/chrislusf/seaweedfs/commit/e5fc35ed0c970fea060a5b3b7a3f5efb5af425d6#diff-25bae991aa8050efe57406452575e53bb89eb91244dcfa772587872b05f862aeR174
so the bug is actually still here.

ok. Added some more fix.

Great, it's working on my end, thanks a lot.


About training speed

I found when train Deeplab from Imagenet pretrained weight, loss drop fast.
But when train FCN8s, loss drop extremely slow.
Did you get the same pattern?

I also observed that pattern. I think it is reasonable to some degree since the residual network such as ResNet101 is easier to train than Vgg in my opinion. As a result I would spend more epochs to train FCN8s.


docs: life of a request.

This documentation goes deeper than the existing architecture overview,
by diving inside Envoy and discussing how internal subsystems work. The
target audience are primarily new Envoy developers, but also anyone
interested in the low-level detail provided, e.g. Envoy operators doing
performance work.
The patch also updates the style guide to request that this be kept
up-to-date, since reliance on low-level internals means that this
description is a moving target.
Signed-off-by: Harvey Tuch<EMAIL_ADDRESS>Co-authored-by: Alyssa Wilk<EMAIL_ADDRESS>Co-authored-by: Matt Klein<EMAIL_ADDRESS>Co-authored-by: Snow Pettersen<EMAIL_ADDRESS>Co-authored-by: Jose Nino<EMAIL_ADDRESS>Co-authored-by: Greg Greenway<EMAIL_ADDRESS>

@envoyproxy/maintainers @mattklein123 @alyssawilk this is a very early draft of, the long procrastinated on, Life of a Request. I'm putting this together for our summer interns, but the goal is to make this a standard part of onboarding at Google and OSS.
It's currently more a sketch than a full doc, but I would find any early feedback to set the direction here very useful. I'm specifically after ideas on how to structure the narrative, places I might have the details wrong and any feature requests to make this more useful to the above purpose.

Thoughts on tossing in a Google doc where the maintainers can all toss in suggestions on sections we're familiar with, or suggest sections? I don't know how collaborative work works with DCO but I think it might be easier to iterate in something other than github if you'd appreciate a hand

+1 for gdoc for initial editing if possible.

Sure, I've created https://docs.google.com/document/d/1lVzS2Rz6-iIa023A7oXkVvbO6JxEtHN-NwS3--cm7eI/edit with review feedback so far. Further input welcome there.

This is now ready for review!

Thanks @alyssawilk. I'll merge this one at EOD if there isn't any other feedback.


Generate standalone libs so that projects can link without the Windows SDK

Fixes #142

Going to simplify this and avoid generating libs as doing so for x86 is doable but quite problematic.


Add instructions for using STARTTLS with s_client

Thanks for contributing!

No problem. I have a few more things on the Java side I would like to
contribute when I get some time. I like having all this info in a
conveniently accessible place. As it happens, a few hours after you
accepted my previous pull request someone came to me asking how to disable
certificate validation. I went through the Java instructions with them
instead, so I am already getting benefits from this site.
I am also going to be giving a 5 minute lightning talk at the local Java
user group, so I will mention the site there too.
Dan
On 22 February 2016 at 06:26, Andrew Sorensen<EMAIL_ADDRESS>wrote:

Thanks for contributing!
—
Reply to this email directly or view it on GitHub
https://github.com/AndrewX192/checkthatcert/pull/8#issuecomment-187032099
.


ADBDEV-5541: Backup statistics by one row

Backup statistics by one row
gpbackup backuped statistics, loading them entirely into RAM. This can create
difficulties in the case of a huge number of tables and may come down to OOM.
This patch changes the backup logic: now statistics are not loaded entirely at
once, but are requested in portions by one row. This significantly reduces RAM
consumption when backing up statistics.
The test was not added because the result does not change. And to watch the
difference in memory consumption, we need a lot of tables with large statistics,
which is not very good for running tests.

When reading the description:
...
gpbackup backups statistics, first loading them entirely into RAM.
...
This patch allows to backup statistics not entirely at once, but by requesting them in portions by one row.
...

I can interpret in a way that this change adds an option: either use old way (load everything into RAM) or new way (get the data in portions).
I think it is better to tune the description, for ex:
...
gpbackup backuped statistics, loading them entirely into RAM.
...
This patch changes the backup logic: now statistics are not loaded entirely at once, but are requested in portions by one row.
...

to avoid possible wrong interpetation.

When reading the description:
...
gpbackup backups statistics, first loading them entirely into RAM.
...
This patch allows to backup statistics not entirely at once, but by requesting them in portions by one row.
...

I can interpret in a way that this change adds an option: either use old way (load everything into RAM) or new way (get the data in portions).
I think it is better to tune the description, for ex:
...
gpbackup backuped statistics, loading them entirely into RAM.
...
This patch changes the backup logic: now statistics are not loaded entirely at once, but are requested in portions by one row.
...

to avoid possible wrong interpetation.

applied

In the description:
gpbackup backups statistics, first them entirely into RAM.

I guess the 'loading' word is missed. Also, it is better to use past tense for old problematic behavior.

In the description:
gpbackup backups statistics, first them entirely into RAM.

I guess the 'loading' word is missed. Also, it is better to use past tense for old problematic behavior.

fixed


Update API for extension planning to include logical plan

Which issue does this PR close?
Closes https://github.com/apache/arrow-datafusion/issues/642
Rationale for this change
With the introduction of qualified identifiers in https://github.com/apache/arrow-datafusion/pull/55 I am having trouble creating plans with user defined nodes because the extension traits don't have sufficient information about the logical plan to create the correct PhysicalExprs.
https://github.com/apache/arrow-datafusion/issues/642 describes in more detail what is going on
What changes are included in this PR?

Add a PhysicalPlanner::create_physical_expr to the planer trait
Change the signature of ExtensionPlanner::plan_extension to include a reference to the planner as well as the logical inputs

Are there any user-facing changes?
Yes: Extension API and PhysicalPlanner interfaces have changed

Thank you for the review @houqp.
Any thoughts @andygrove  / @Dandandan  / @jorgecarleitao ?  Otherwise I would like to merge this in the next day or so.


initialState columnFilters ignored when onFilter is used

Description
When the property onFilter / state={} is used to preserve the state a filter initialized with initialState is ignored
Expected Behaviour
Initial state should be derivated from initialState
Additional Information
might be related to #157
Example: https://codesandbox.io/s/exciting-lederberg-y296ww?file=/src/App.js
comment the state to see a working version of the example
state={{
        // deactivate columnFilters to fix problem with initalFilterState
        // columnFilters,
        rowSelection: selected
      }}

When using onFilter / state={} the initial values need to be set in useState function:
const initialFilters = [
    {
      id: "id",
      value: {
        op: "eq",
        value: "1",
      },
    },
  ];

  const [columnFilters, setColumnFilters] =
    useState<ColumnFiltersState>(initialFilters);

return (
<DataGrid
        onFilter={setColumnFilters}
        state={{
          columnFilters,
        }}


Non-zero minimum isolation connection pool size causes infinite function recursion (instant crash)

Hi,
When calling createClient, specifying a positive number for the isolationPoolOptions.min value causes an infinite recursion through the RedisClient constructor. It seems the initial create client calls the constructor, which checks the isolation pool options, which sees there are (say) 3 connections required, so it calls duplicate to create them. duplicate passes the same isolation options to the new RedisClient, which repeats the same check. Thus, infinite recursion.
I suggest that nested RedisClients created via duplicate should be given empty/undefined isolationPoolOptions. I think it is also surprising/unintuitive that calling duplicate on the top-level redis client would result in an entire new pool being created (i.e. doubling the number of connections) rather than just creating a single new (separate) connection.
Environment:

Node.js Version: 16.4.2
Redis Server Version: N/A
Node Redis Version: 4.0.0-rc.2
Platform: Windows 10

I think that .duplicate() should be changed to .duplicate(overrides: Partial<RedisClientOptions>), and the isolationPool.create function should use that to reset the isolationPoolOptions on the "duplicated" connections. What do you think?


Fetching opportunity stages

Maybe I'm looking over it or it's named differently in the API, but I can't seem to find how to fetch opportunity stages. I am able to fetch opportunities themselves but not the stages they have a relation with. Do you know where I can find the stages? The Guid in the opportunity has a value, so it's definitly linked to an entity somewhere.
Thank you in advance!

I also don't see any api to retrieve opportunity stages at first sight...
See here for the whole list of api's: https://start.exactonline.nl/docs/HlpRestAPIResources.aspx?SourceAction=10
If you see it in there, it's possible to add support for it in this sdk... otherwise that's difficult ;-)

That's what I was afraid of, I also couldn't find any endpoint in their docs.
I'll contact Exact support.


Expand Basic Usage Guide

Additional explanation of usage of some of the included features would be very helpful.  For example, a simple guide that shows how to use the the Nui emulate function.

Hey, sorry the late response. I am no longer involved in FiveM, and apart from the occasional dependency bump this project is  unmaintained.
I did make an example for my FiveM React boilerplate. The concept is fairly similar, however it uses Redux.
Check out https://github.com/calumari/fivem-react-boilerplate/tree/master/example if you're interested.

Thank you! I will look into that


System.TimeoutException

Hello,
I am using vb.net. I get the following error
{"Timed out trying to read data from the socket stream!"}
comftpclient.Host = "sftp.comerica.com"
comftpclient.Credentials = New NetworkCredential("clientid", "password")
comftpclient.Port = 990
comftpclient.Connect() ----this is where I get the error.
Thanks,
Reed

Nevermind. Switched product. 23 hours for a response is too long.


unit testing ?

I would like to test my login form as followed but loginForm is the decorated element so loginForm.refs is undefined. Should I export the form 2 times (decorated and undecorated), test the undecorated one (suggested here in the Connected Components part) with the required mocked up properties or is there another way? Thank you
describe('login form ', function () {
    it('should validate', function () {
        const createFakeStore = fakeData => ({
            getState() {
                return fakeData;
            },
            subscribe(fn) {
            },
            dispatch() {
            }
        });

        let data = {auth: {tokenExpiry: parseInt(moment().add(1, 'days').toDate().getTime()/1000), isFetching: false}};
        var handleSubmit = jasmine.createSpy();
        var loginForm = TestUtils.renderIntoDocument(
            <LoginForm onSubmit={handleSubmit} store={createFakeStore(data)} />);
        var form = TestUtils.findRenderedDOMComponentWithTag(loginForm, 'form');
        loginForm.refs.username.getDOMNode().value = 'username';
        loginForm.refs.password.getDOMNode().value = 'password';
        TestUtils.Simulate.submit(form);
        expect(handleSubmit).toHaveBeenCalledWith({username: 'username', password: 'password'});
    });
});

I'm sorry, but I am woefully inept at React component testing. But yes, I would definitely test the undecorated form (one of the benefits of redux-form, and react-redux, in general, is that your decorated component can be relatively dumb, and therefore more easily testable), and maybe test the decorated version as well, although it's more unclear to me how that would work.

@samidarko @erikras I feel that it would make sense to declare form component as pure functions (as of react 0.14) and then wrap them inside of container component which gets decorated.
This has two benefits:

We can be sure that the form component is really 'dumb'
We do not have any issues getting to the form component in our unit tests (we do not have to add another export etc.)

The test for the actual component would then only have to pass in a mocked set of fields (which can be easily done with a helper function) and other used props and only has to test that functions are being called properly. This leaves redux-form completely out of the mix.
Technically you wouldn't have to test the decorated component very much, because we should assume that redux-form works properly. So the container could be tested in terms of whether it passes the correct props to the dumb component and whether it called the proper actions when the form is submitted in a valid state.
Thoughts?

Thanks @erikras for your answer. Finally I will do as recommended in the redux doc, export the class and the module and test the class
But I suppose I should mock up the properties added by connectReduxForm ?

Yes, mock up those properties. Re-open if you have any more questions.

Hi @samidarko ,
Can you provide the way you mocked up the connectReduxForm properties for unit testing?
Thanks.

Hi @yugal2 ,
Finally I never did it but I could explain you how I would have done.
Lets say you have a component Form.js, and usually redux-form is wrapping it like following:
Form = reduxForm({
    form: 'myForm',
    fields: ['username'],
    validate: validate
})(Form);

export default Form;

I would also export my component class and not test the default export.
export class Form extends React.Component {
...

and I would import the form without the redux-form wrapping as redux-form is already tested and we assume the configuration is done correctly, if not it will be detected soon enough and patched.
In my test, I import:
import { Form } from '../path/to/Form';

and NOT
import Form from '../path/to/Form';

Then from here I would mock up the following props I pass to the component :
const {fields: {username}} = this.props;

(not sure of the exact code, I'm just making up everything)
using Jasmine:
describe('hopefully', function () {
    it('should work', function () {
        let props = {
            fields: {username: {value: 'something', onChange: jasmine.createSpy('onChange')}
        };
        let renderer = TestUtils.createRenderer();
        renderer.render(<Form {...props} />);
        let output = renderer.getRenderOutput();
        ... the test
    });
});

I think this is the idea. We could ask @erikras confirmation. I'm not sure of the props you need to mock up (onBlur, ...) but you can look at  the API documentation.
Hope it helps :)

On #911, @andrewmclagan said he will be writing up some docs on redux-form unit testing. I have not done much testing of usages of the library, so I look forward to seeing what he comes up with.

yes that is forth-coming.

How can we get valid outputs from getValues() by passing an object.  No matter what I pass returns {} as the result.
I have a couple use cases where this is necessary.  For example, I need to make sure a submit button is not disabled with a valid form.  The disabled property relies on my validator which is tested separately.  However, the result from getValues is what is passed to the validator and if it's always {} then I can't really test anything.

I've been waiting on unit test examples and documentation too, and when I figured out a few techniques, I wanted to write about them and share. Hopefully andrewmclagan's docs are better, but in the meantime I created a project to show off both unit testing with forms and integration tests with redux-form.
The project is at https://github.com/tylercollier/redux-form-test.

@bpmckee Here's what I tried in my integration test that worked fine: getValues(store.getState()). If I log that to the console, it outputs the following as expected with my simple test that sets the value of the first name field:
{ form: { contact: { firstName: 'Joe' } } }

If that doesn't get you what you need, please put your question on stackoverflow.


Projects dialog validation is out of order

Steps to repro:

Close any open workspace
open create new project dialog
put invalid location
click "Create"

expected:
Error will show that location is invalid
actual:
popup shows asking if it's ok to restart ADS and the error only shows after the user clicks ok on the notification

@kisantia Is this something you all were planning on looking at or should I investigate? (since it's likely due to my changes)
I assume this is a release blocker?

@Charles-Gagnon yup we'll try to take this for July. Should be a simple fix, but we'll let you know if we need help investigating.

Port Req merged.


Initial commit of local development environments

Description
Adds local development environments based on Docker for running the frontend and backend of Aspen. Initial instructions on how to use are in [DEV_ENV.md].
Left to document: How to update Pipfile, Pipfile.lock, requirements.txt, and package.json in the new world.
It currently is able to bring up all the containers, but we are still working through issues initializing the database contents, and some CORS issues.

these failing tests are strange.. Not sure why it's looking for a module named py.aspen, it should just be aspen.  I'll need to do some more investigating

i think we need to add an __init__.py file somewhere, or maybe we can try removing src/py/__init__.py, i see you added that in but i'm wondering if that might be causing these errors?

yay now the tests are failing for the right reason!


Go to Type Definition on TypeAlias should include location of TypeAlias in results

Code Snippet
from os import PathLike
from typing import TypeAlias

StrPath: TypeAlias = str | PathLike[str]

var: StrPath

Repro Steps

Right-click on var and select "Go to Type Definition"

Expected behavior
List of results includes StrPath alias.
Actual behavior
List of results only includes str and PathLike, the members of the union.

I don't think the type alias should be included in the list here. A type alias is just a convenient name for a particular type (often a union of other types); it's not a type itself.
The pyright type evaluator doesn't currently retain enough information about a type alias associated with a type to determine the original declaration location. It's theoretically possible to add this, but it would increase the amount of information tracked for every internal type object and the amount of work performed when transforming these types.
For comparison, TypeScript also does not include type aliases in in the "Go To Type Definition" list — presumably for the same reasons I mentioned above.

That's fine. For what it's worth my thought process as a user was something like:

Hover over var to see what type it is.
"Oh, it's a StrPath, let's see what that is.
Go to Type Definition
Confused about why I'm not seeing info about StrPath.

In my case StrPath was not defined locally and I didn't have convenient access to a StrPath token in my code that I could Go to Definition on.


sheet-updates

Related Issue
issue no: #344
Proposed Changes
in both modes the colors I have used are are visible clearly as you have mentioned.

Additional Info

Any additional information or context

Checklist

[ ] ✅ My code follows the code style of this project.
[ ] 📝 My change requires a change to the documentation.
[ ] 🎀 I have updated the documentation accordingly.
[ ] 🌟 ed the repo

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Output Screenshots

Screenshot #1
Screenshot #2

Title goes here
Title goes here

Image goes here
Image goes here

if this branch is merged i am having an another idea that we can keep a java  image beside it

and HTML image beside it so that it looks really nice

Its level2 issue, so please make better ui for the cheetsheets page and on opening the website the nav links are coloured green which is an bug


Sanic- Sentry integration: Sending 2 requests consecutively to an endpoint in a test function raised error

Environment
SaaS (https://sentry.io/)
Version
No response
Steps to Reproduce

sanic==22.3.2
sanic-routing==22.3.0
sanic-testing==22.3.0
sentry-sdk==1.5.12
pytest==7.1.2

Hi guys. Recently I upgraded Sanic and Sentry SDK's version of my web app. Yet there is a problem.
Here I have a very rough Sanic web app and I also use Sentry for error tracking to demonstrate the problem:
I got IndexError: pop from empty list When trying to write a test where I fired a GET request to an endpoint and verify the HTTP response code then I did the same in the same test function.](https://github.com/namnguyenbk/sanic-sentry-bug/blob/main/test.py#L10). See full traceback below.
Expected Result
The test could run successfully with Sanic=20.12.1. Still, the latest version of Sanic and Sentry makes it fail. Not sure what's going on and whether it affects other parts of the application. 🙏 Anyone has ideas to work around or thoughts?
Actual Result
ERROR    sanic.error:handlers.py:259 Exception occurred while handling uri: 'http://<IP_ADDRESS>:63443/hello'
Traceback (most recent call last):
  File "handle_request", line 102, in handle_request
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sanic/signals.py", line 193, in dispatch
    return await dispatch
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sanic/signals.py", line 163, in _dispatch
    retval = await maybe_coroutine
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sentry_sdk/integrations/sanic.py", line 184, in _hub_exit
    request.ctx._sentry_hub.__exit__(None, None, None)
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sentry_sdk/hub.py", line 246, in __exit__
    old = self._old_hubs.pop()
IndexError: pop from empty list
ERROR    sanic.error:handlers.py:259 Exception occurred while handling uri: 'http://<IP_ADDRESS>:63443/hello'
Traceback (most recent call last):
  File "handle_request", line 102, in handle_request
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sanic/signals.py", line 193, in dispatch
    return await dispatch
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sanic/signals.py", line 163, in _dispatch
    retval = await maybe_coroutine
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sentry_sdk/integrations/sanic.py", line 184, in _hub_exit
    request.ctx._sentry_hub.__exit__(None, None, None)
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sentry_sdk/hub.py", line 246, in __exit__
    old = self._old_hubs.pop()
IndexError: pop from empty list

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sanic/http.py", line 146, in http1
    await self.protocol.request_handler(self.request)
  File "handle_request", line 133, in handle_request
    "_state",
  File "handle_exception", line 105, in handle_exception
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sanic/signals.py", line 193, in dispatch
    return await dispatch
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sanic/signals.py", line 163, in _dispatch
    retval = await maybe_coroutine
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sentry_sdk/integrations/sanic.py", line 184, in _hub_exit
    request.ctx._sentry_hub.__exit__(None, None, None)
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sentry_sdk/hub.py", line 246, in __exit__
    old = self._old_hubs.pop()
IndexError: pop from empty list
ERROR    sanic.error:connection_task:19 protocol.connection_task uncaught
Traceback (most recent call last):
  File "handle_request", line 102, in handle_request
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sanic/signals.py", line 193, in dispatch
    return await dispatch
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sanic/signals.py", line 163, in _dispatch
    retval = await maybe_coroutine
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sentry_sdk/integrations/sanic.py", line 184, in _hub_exit
    request.ctx._sentry_hub.__exit__(None, None, None)
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sentry_sdk/hub.py", line 246, in __exit__
    old = self._old_hubs.pop()
IndexError: pop from empty list

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sanic/http.py", line 146, in http1
    await self.protocol.request_handler(self.request)
  File "handle_request", line 133, in handle_request
    "_state",
  File "handle_exception", line 105, in handle_exception
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sanic/signals.py", line 193, in dispatch
    return await dispatch
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sanic/signals.py", line 163, in _dispatch
    retval = await maybe_coroutine
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sentry_sdk/integrations/sanic.py", line 184, in _hub_exit
    request.ctx._sentry_hub.__exit__(None, None, None)
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sentry_sdk/hub.py", line 246, in __exit__
    old = self._old_hubs.pop()
IndexError: pop from empty list

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "connection_task", line 15, in connection_task
    from sanic.http import Http, Stage
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sanic/http.py", line 168, in http1
    await self.error_response(e)
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sanic/http.py", line 439, in error_response
    await app.handle_exception(self.request, exception)
  File "handle_exception", line 105, in handle_exception
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sanic/signals.py", line 193, in dispatch
    return await dispatch
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sanic/signals.py", line 163, in _dispatch
    retval = await maybe_coroutine
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sentry_sdk/integrations/sanic.py", line 184, in _hub_exit
    request.ctx._sentry_hub.__exit__(None, None, None)
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sentry_sdk/hub.py", line 246, in __exit__
    old = self._old_hubs.pop()
IndexError: pop from empty list
ERROR    sanic.root:testing.py:116 GET http://<IP_ADDRESS>:63443/hello received no response!
Traceback (most recent call last):
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/httpx/_transports/default.py", line 60, in map_httpcore_exceptions
    yield
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/httpx/_transports/default.py", line 353, in handle_async_request
    resp = await self._pool.handle_async_request(req)
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/httpcore/_async/connection_pool.py", line 253, in handle_async_request
    raise exc
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/httpcore/_async/connection_pool.py", line 237, in handle_async_request
    response = await connection.handle_async_request(request)
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/httpcore/_async/connection.py", line 90, in handle_async_request
    return await self._connection.handle_async_request(request)
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/httpcore/_async/http11.py", line 102, in handle_async_request
    raise exc
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/httpcore/_async/http11.py", line 81, in handle_async_request
    ) = await self._receive_response_headers(**kwargs)
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/httpcore/_async/http11.py", line 143, in _receive_response_headers
    event = await self._receive_event(timeout=timeout)
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/httpcore/_async/http11.py", line 186, in _receive_event
    raise RemoteProtocolError(msg)
httpcore.RemoteProtocolError: Server disconnected without sending a response.

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/sanic_testing/testing.py", line 109, in _local_request
    response = await getattr(session, method.lower())(
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/httpx/_client.py", line 1729, in get
    return await self.request(
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/httpx/_client.py", line 1506, in request
    return await self.send(request, auth=auth, follow_redirects=follow_redirects)
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/httpx/_client.py", line 1593, in send
    response = await self._send_handling_auth(
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/httpx/_client.py", line 1621, in _send_handling_auth
    response = await self._send_handling_redirects(
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/httpx/_client.py", line 1658, in _send_handling_redirects
    response = await self._send_single_request(request)
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/httpx/_client.py", line 1695, in _send_single_request
    response = await transport.handle_async_request(request)
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/httpx/_transports/default.py", line 353, in handle_async_request
    resp = await self._pool.handle_async_request(req)
  File "/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.8/lib/python3.8/contextlib.py", line 131, in __exit__
    self.gen.throw(type, value, traceback)
  File "/sentry-sanic-bug/venv/lib/python3.8/site-packages/httpx/_transports/default.py", line 77, in map_httpcore_exceptions
    raise mapped_exc(message) from exc
httpx.RemoteProtocolError: Server disconnected without sending a response.
INFO     sanic.root:runners.py:154 Starting worker [57202]
INFO     sanic.root:runners.py:157 Stopping worker [57202]
INFO     sanic.root:runner.py:592 Server Stopped
============================================================================================== short test summary info ==============================================================================================
FAILED test.py::test_basic_test_client - ValueError: Request and response object expected, got ([<Request: GET /hello>, None])

@namnguyenbk https://github.com/sanic-org/sanic/pull/2451 may have resolved this; can you try to reproduce with sanic 22.6.0?

@ahopkins Yes, I am still facing this issue.
sanic==22.6.2
sanic-testing==22.6.0
sanic-routing==22.3.0
pytest==7.2.0
pytest-asyncio==0.20.2
pytest-json==0.4.0
asynctest==0.13.0
jsonschema==3.0.2


Webinsights - Sending Diagnostics Data from Microservice to Diagnostics Backend

Logging start simulation, stop simulation and service errors to diagnostics backend.

This change is

Closing this PR since git is unable to receive the status from Travis


load_gltf example not rendering

Bevy version
72b2fc98437f39a8099efcc137697efc32a21800
Operating system & version
NixOS 20.09, wayland
What you did
cargo run --example load_gltf

What you expected to happen
The model rendered
What actually happened
Blank screen, no model rendered
Additional information
@cart noticed this was in master https://github.com/bevyengine/bevy/pull/958#issuecomment-735519477 and just picked it up myself as I rebased from a week or so ago.
I did some jumping around and in-case useful just wanted to point out it looks to be introduced by 72b2fc98437f39a8099efcc137697efc32a21800 the Bevy Reflection commit, if anything there rings a bell... The commit before seems OK.

I got the same problem. 72b2fc9 the Bevy Reflection  commit seem broken the function world.spawn_scene , the component like Draw, Mesh is ignore during the scene spawn with reflection

Took your fix for a spin @cart and looks all good from here, thanks!


Update readme.md to Angular 4 and angular-cli

Solved ng2-ckeditor integration with angular-cli and Angular 4.
There are some bugs in current instructions.
Correct instructions for sample Angular 4.
This issue is solved, documented and demonstrated in stackoverlow here:  https://stackoverflow.com/a/44598775/6266192 .
Just copy and paste to readme.md.

this works, just watch out for the invalid html on the so code!
<script> src="https://cdn.ckeditor.com/4.7.0/standard-all/ckeditor.js"></script>
bad markup!
should be
<script src="https://cdn.ckeditor.com/4.7.0/standard-all/ckeditor.js"></script>

Sorry but I don't see the difference betwteen your stackoverflow and the readme.
And you are free to make a PR on this !
Thanks !

I'm sure what @maroshi meant was that for newbies(like myself) th README.md is pretty confusing. The ckeditorContent element isn't defined in the respective class.

hi how do I keep the  ckeditorContent and [config] value separated from the template html component tag. I have an old code built with angularjs and ckeditor and they do something like this here.
CKEditor.config.customConfig and CKEditor.replace(). How do I leverage this in angular v2 and v4

@chymz  sorry it took me a while to come back to this.
what i was trying to show was that on stack overflow there is an example but that example has a very small error that if one is in a rush to copy will make it not work but also not show any error (at least in chrome).
the error is the html under part 6  "Add latest CKEditor script from CDN to Angular framework"
the script tag is malformed.

it should not have the ">" right at the start.
  works better

Solved in 1.2.0 release.