Update nats streaming  image version to 0.22.0

Problem
Update nats streaming image before we deprecate it. Update from 0.11.0 to 0.22.0
And the nats streaming image version list is here
Persona:
Exit Criteria
Time Estimate (optional):
How many developer-days do you think this may take to resolve?
0.5
Additional context (optional)
Add any other context about the feature request here.

/assign


Update Expat to 2.6.3

Updated given the few security issues in Expat 2.6.2. I did not test it because I currently don't have my Windows PC nearby, but I'll probably use it in the next few days.

Thanks, @Semphriss! In w64dk, libexpat only parses trusted inputs, so none
of its security issues are ever relevant. Though the old version no longer
being hosted is relevant, so the upgrade is still required.


Bundle size tracking

Following https://github.com/onfido/castor/issues/118 also add same approach as taken on https://github.com/onfido/castor/pull/699 for this repo.

Bundlewatch currently has issues with watching many files which would be required for this package.
We could watch e.g. a sprite of icons, then that sprite conversion in TS/JS, however it does not represent the best effort, and also adding new icons we know it is going to grow.
That said, we have decided that for Castor Icons we will not require to track bundle size.


Support bigger seed.sql files using docker copy

What kind of change does this PR introduce?
Bug fix
What is the current behavior?
For seed.sql files that exceed the maximum argument length of the Linux distribution used to run the Postgres container the postgres -c command will fail. Supabase CLI will then silently fail to apply the seed file.
See related StackOverflow Max size of string CMD that can be passed to Docker answer about command argument sizes.
https://github.com/supabase/cli/issues/277 (See related comment)
https://github.com/supabase/cli/issues/274 (I think this might be related, but not sure)
What is the new behavior?
A new Docker utility function DockerAddFile can be used to write a byte buffer to a container, in this case the seed.sql file. Then migrating the seed file is just a matter of using the file option of the pgsql command rather than the command option.
Additional context
Let me know if the code needs some adjustments and if the selected approach is good or not.

LGTM, thanks for this!

LGTM, thanks for this!

My pleasure. Thank you for reviewing and merging this so fast! This will improve my development flow so much!


[qt5-base] fix x64-osx -> arm64-osx cross compile

Make it possible to cross-compile qt5-base from x64 mac to arm64 by detecting cross compilation and passing the correct flags

What does your PR fix?

Which triplets are supported/not supported? Have you updated the CI baseline?
osx, Yes

Does your PR follow the maintainer guide?

Yes

If you are still working on the PR, open it as a Draft: https://github.blog/2019-02-14-introducing-draft-pull-requests/

Somewhat depends on #18630 , #18625 and  #18608

Any progress?

Right now the problem is that fontconfig fails to cross compile from x86 to arm. I can rebase this patch, but if merged without fontconfig fixes, it is kinda useless, since the dependencies can't cross compile.

cc @Neumann-A

Does this PR no longer depend on other PRs? I see the checks are okay now.

It can be merged now.

@Neumann-A Do you against this changes?

@Neumann-A Do you against this changes?

only against the changes which removes the removal of the third party deps. These folders are removed to not accidentally build them.

Qt's build system really wants to use vendored zlib to compile the bootstrap binary (the actual arm64 binary uses system zlib). And i haven't been able to convince it to use system zlib for that.

Any progress?
Can you please merge to master?

Convert this PR to draft since there is no progress.

Closing this PR since it seems that no progress is being made. Please reopen if work is still being done.


v9.0.5 still appears to have memory usage problems

Describe the bug
#6196 describes the original problem, but when running a gradlew dependencyCheckUpdate w/ the new version of the tool using cached definitions from vulnz the memory usage is growing beyond Gradle's default 512MB limit and causing an OOM error.
Version of dependency-check used
The problem occurs using version 9.0.5 of the Gradle plugin, however I would assume the other tools are impacted too.
Log file
Here are the logs, but I also have a heap dump so I'll attach some screenshots from it.

All 4 of my threads have sizeable collections of io.github.jeremylong.openvulnerability.client.nvd.DefCveItem in memory. Each DefCveItem itself then contains a large collection of io.github.jeremylong.openvulnerability.client.nvd.Config where the bulk of the data appears to be. All that then adds up to ~400MB and Gradle's default heap limit gets filled with some of the other stuff laying around inside it.
To Reproduce
Run a database update against a fresh project w/ no cache.
Expected behavior
The database is updated without a hitch and the analysis works.
Additional context
The heap size can be increased to get around the issue, but I'm hoping to avoid that as it'll mean updating the gradle.properties for all my projects.

Hi Team,
It's happen to me also where I'm using 9.0.5 and sets nvd.datafeedUrl to the cache.
❯ ./gradlew dependencyCheckUpdate

> Configure project :
The 'sonarqube' task depends on compile tasks. This behavior is now deprecated and will be removed in version 5.x. To avoid implicit compilation, set property 'sonar.gradle.skipCompile' to 'true' and make sure your project is compiled, before analysis has started.
The 'sonar' task depends on compile tasks. This behavior is now deprecated and will be removed in version 5.x. To avoid implicit compilation, set property 'sonar.gradle.skipCompile' to 'true' and make sure your project is compiled, before analysis has started.
Error downloading NVD CVE - https://assets.dev.tomoloyalty.io/nist-data/mirror/nvdcve-2010.json.gz Reason: Java heap space
Failed to process CVE-2011-0142
org.owasp.dependencycheck.data.nvdcve.DatabaseException: Unable to retrieve id for new vulnerability for 'CVE-2011-0142'
        at org.owasp.dependencycheck.data.nvdcve.CveDB.updateOrInsertVulnerability(CveDB.java:1063)
        at org.owasp.dependencycheck.data.nvdcve.CveDB.updateVulnerability(CveDB.java:866)
        at org.owasp.dependencycheck.data.update.nvd.api.NvdApiProcessor.call(NvdApiProcessor.java:121)
        at org.owasp.dependencycheck.data.update.nvd.api.NvdApiProcessor.call(NvdApiProcessor.java:42)
        at java.base@<IP_ADDRESS>/java.util.concurrent.FutureTask.run(FutureTask.java:264)
        at java.base@<IP_ADDRESS>/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
        at java.base@<IP_ADDRESS>/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
        at java.base@<IP_ADDRESS>/java.lang.Thread.run(Thread.java:833)
Caused by: org.h2.jdbc.JdbcSQLNonTransientException: General error: "General error: ""org.h2.mvstore.MVStoreException: java.lang.OutOfMemoryError: Java heap space [2.1.214/3]"" [50000-214]" [50000-214]
        at org.h2.message.DbException.getJdbcSQLException(DbException.java:554)
        at org.h2.message.DbException.getJdbcSQLException(DbException.java:477)
        at org.h2.message.DbException.get(DbException.java:212)
        at org.h2.engine.Database.throwLastBackgroundException(Database.java:1782)
        at org.h2.engine.SessionLocal.beforeCommitOrRollback(SessionLocal.java:754)
        at org.h2.engine.SessionLocal.commit(SessionLocal.java:668)
        at org.h2.command.Command.stop(Command.java:157)
        at org.h2.command.CommandContainer.stop(CommandContainer.java:259)
        at org.h2.command.Command.executeQuery(Command.java:229)
        at org.h2.jdbc.JdbcPreparedStatement.executeQuery(JdbcPreparedStatement.java:128)
        at org.apache.commons.dbcp2.DelegatingPreparedStatement.executeQuery(DelegatingPreparedStatement.java:123)
        at org.apache.commons.dbcp2.DelegatingPreparedStatement.executeQuery(DelegatingPreparedStatement.java:123)
        at org.owasp.dependencycheck.data.nvdcve.CveDB.updateOrInsertVulnerability(CveDB.java:1058)
        ... 7 more
Caused by: org.h2.message.DbException: General error: "org.h2.mvstore.MVStoreException: java.lang.OutOfMemoryError: Java heap space [2.1.214/3]" [50000-214]
        at org.h2.message.DbException.get(DbException.java:212)
        at org.h2.message.DbException.convert(DbException.java:395)
        at org.h2.mvstore.db.Store.lambda$new$0(Store.java:125)
        at org.h2.mvstore.MVStore.handleException(MVStore.java:3318)
        at org.h2.mvstore.MVStore.panic(MVStore.java:593)
        at org.h2.mvstore.MVStore.serializeAndStore(MVStore.java:1605)
        at org.h2.mvstore.MVStore.lambda$storeNow$4(MVStore.java:1518)
        at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:539)
        ... 4 more
Caused by: org.h2.jdbc.JdbcSQLNonTransientException: General error: "org.h2.mvstore.MVStoreException: java.lang.OutOfMemoryError: Java heap space [2.1.214/3]" [50000-214]
        at org.h2.message.DbException.getJdbcSQLException(DbException.java:554)
        at org.h2.message.DbException.getJdbcSQLException(DbException.java:477)
        ... 12 more
Caused by: org.h2.mvstore.MVStoreException: java.lang.OutOfMemoryError: Java heap space [2.1.214/3]
        at org.h2.mvstore.DataUtils.newMVStoreException(DataUtils.java:1004)
        ... 7 more
Caused by: java.lang.OutOfMemoryError: Java heap space
        at org.h2.mvstore.cache.CacheLongKeyLIRS$Segment.evictBlock(CacheLongKeyLIRS.java:900)
        at org.h2.mvstore.cache.CacheLongKeyLIRS$Segment.evict(CacheLongKeyLIRS.java:884)
        at org.h2.mvstore.cache.CacheLongKeyLIRS$Segment.put(CacheLongKeyLIRS.java:809)
        at org.h2.mvstore.cache.CacheLongKeyLIRS.put(CacheLongKeyLIRS.java:166)
        at org.h2.mvstore.MVStore.cachePage(MVStore.java:3469)
        at org.h2.mvstore.Page.write(Page.java:776)
        at org.h2.mvstore.Page$NonLeaf.writeUnsavedRecursive(Page.java:1329)
        at org.h2.mvstore.Page$NonLeaf.writeChildrenRecursive(Page.java:1344)
        at org.h2.mvstore.Page$NonLeaf.writeUnsavedRecursive(Page.java:1330)
        at org.h2.mvstore.MVStore.serializeToBuffer(MVStore.java:1669)
        at org.h2.mvstore.MVStore.serializeAndStore(MVStore.java:1598)
        at org.h2.mvstore.MVStore.lambda$storeNow$4(MVStore.java:1518)
        at org.h2.mvstore.MVStore$$Lambda$1554/0x0000000800b26798.run(Unknown Source)
        ... 5 more
The Daemon will expire immediately since the JVM garbage collector is thrashing.
The project memory settings are likely not configured or are configured to an insufficient value.
The memory settings for this project must be adjusted to avoid this failure.
These settings can be adjusted by setting 'org.gradle.jvmargs' in 'gradle.properties'.
The currently configured max heap space is '512 MiB' and the configured max metaspace is '384 MiB'.
For more information on how to set these values, please refer to https://docs.gradle.org/8.5/userguide/build_environment.html#sec:configuring_jvm_memory in the Gradle documentation.
To disable this warning, set 'org.gradle.daemon.performance.disable-logging=true'.
Daemon is stopping immediately since the JVM garbage collector is thrashing
Error downloading NVD CVE - https://assets.dev.tomoloyalty.io/nist-data/mirror/nvdcve-2021.json.gz Reason: Java heap space

FAILURE: Build failed with an exception.

* What went wrong:
Gradle build daemon has been stopped: since the JVM garbage collector is thrashing

* Try:
> Run with --stacktrace option to get the stack trace.
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.
> Get more help at https://help.gradle.org.

Thanks

I've ran with 9.0.6 and aren't seeing the crashes anymore, so closing this off.

I see the same issues on 9.0.7

Same issue indeed on 9.0.7

are people seeing the memory issues using the API or the datafeed or both?

I've re-ran again with both 9.0.7 and the newer 9.0.8 and everything is still working fine for me:

If people are still seeing problems some more detail is probably required as this issue no longer appears relevant.

Issue is still here with 9.0.8 when using NVD Cache, with Java 11 and gradle 7.6.1

We have been having OOM since about 2 hours ago
When execute maven -X i see a lot of errors like this

I tried to update from 9.1.0 to 9.2.0 and errors are still present
Heap memory grown with each error logged

We have been having OOM since about 2 hours ago
When execute maven -X i see a lot of errors like this

I tried to update from 9.1.0 to 9.2.0 and errors are still present
Heap memory grown with each error logged

We are running into this as well and it seems to begin at the same time you mentioned as well.

We have been having OOM since about 2 hours ago
When execute maven -X i see a lot of errors like this

I tried to update from 9.1.0 to 9.2.0 and errors are still present
Heap memory grown with each error logged

We are running into this as well and it seems to begin at the same time you mentioned as well.

After update to 10.0.0 problem looks solved


Any one use this with Typescript?

Also interested in any d.ts file for this lib.

Me too! It would be really great to add definition files into the package itself on npm, for example.

I've started working on it.  Not have the foggiest how long would it take.

Feel free to use it with typescript.
TS definitions were added to definitely typed.


Issue: I have a problem with my Flutter application.

Steps to Reproduce

Run flutter create bug.
Update the files as follows: ...
...

Expected results:
Actual results:

Logs

في اثنين، 21 يونيو، 2021 في 2:40 م، كتب Nsimba @.***>:

Steps to Reproduce

Run flutter create bug.
Update the files as follows: ...
...

Expected results:
Actual results:
Logs
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/84972, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AURFF3RNX4JS3FVLN3EBOS3TT4JLRANCNFSM47BJ7K6Q
.

@Super-Dealer
When opening an issue, please try to provide all or some of the information indicated in the template.

Issue Template
If you are looking for support, please check out our documentation
or consider asking a question on Stack Overflow:
- https://flutter.dev/
- https://api.flutter.dev/
- https://stackoverflow.com/questions/tagged/flutter?sort=frequent

If you have found a bug or if our documentation doesn't have an answer
to what you're looking for, then fill our the template below. Please read
our guide to filing a bug first: https://flutter.dev/docs/resources/bug-reports

Steps to Reproduce

Please tell us exactly how to reproduce the problem you are running into.

Please attach a small application (ideally just one main.dart file) that
reproduces the problem. You could use https://gist.github.com/ for this.

If the problem is with your application's rendering, then please attach
a screenshot and explain what the problem is.

1. ...
2. ...
3. ...

Please tell us which target platform(s) the problem occurs (Android / iOS / Web / macOS / Linux / Windows)
Which target OS version, for Web, browser, is the test system running?
Does the problem occur on emulator/simulator as well as on physical devices?

Target Platform:
Target OS version/browser:
Devices:

Run your application with `flutter run --verbose` and attach all the
log output below between the lines with the backticks. If there is an
exception, please see if the error message includes enough information
to explain how to solve the issue.

Run `flutter analyze` and attach any output of that command below.
If there are any analysis errors, try resolving them before filing this issue.

Finally, paste the output of running `flutter doctor -v` here.

Closing this issue, since no information has been provided.
Thank you.


Can't off feature "Auto recognize URLs"

Windows Terminal version
1.22.2281.0
Windows build number
10.0.19045.3448
Other Software
No response
Steps to reproduce
broken features:
"experimental.detectURLs": false,
work fine:
Microsoft.WindowsTerminal_1.17.11461.0_x64.zip
Microsoft.WindowsTerminal_1.18.2681.0_x64.zip
broken:
Microsoft.WindowsTerminal_1.19.10302.0_x64.zip
...
Microsoft.WindowsTerminal_1.20.11781.0_x64.zip
latest Canary terminal-1.22.2281.0 also broken
test URL:
https://aka.ms/terminal-documentation
Expected Behavior
recognized URLs are not highlighted
Actual Behavior
recognized URLs is highlighted
anyway
when
"experimental.detectURLs": false,
or
"experimental.detectURLs": true,

This regressed in cd80f3c76496c7664ed710907d7d473f634191bc because this line got removed from the branch:
https://github.com/microsoft/terminal/blob/5651f087702d97f8c5e48db6cd4199aed8a04479/src/cascadia/TerminalCore/Terminal.cpp#L1340-L1342
The fix is to add a _detectURLs check here:
https://github.com/microsoft/terminal/blob/1ef497970f8409def5ac03e26ba54439d0aad4e0/src/cascadia/TerminalCore/Terminal.cpp#L1422-L1452

Eh, I figured I'd drive by and fix it


Cannot read elevatino data from USF FVCOM

The problem is describe in this notebook:
http://nbviewer.ipython.org/github/ioos/secoora/blob/master/sandbox/USF_FVCOM.ipynb
@rsignell-usgs Is it possible that the problem is in the aggregation?  I am lost here...

OK, I just found out that iris is failing to create the Auxiliary coordinates for  longitude and latitude.  Therefor node is loaded as anonymous dimensions and iris does not merge that into one cube.
Now I have to find out why that is happening!

I remember that in the past some piece of code (pyugrid perhaps?) was having a problem when nv (the connectivity array) was real instead of int.  That wouldn't be it, would it?

Thanks I will look into it.  This is what I found so far:
http://nbviewer.ipython.org/github/ioos/secoora/blob/19a9a9d2dce1e60aa4cd9f3615b46954cc7d58a3/sandbox/USF_FVCOM.ipynb

That's it.  Spend the whole morning on this...  Is this a problem that should be fixed at the data level?  Or did you guys solved it at a software level with pyugrid?

For what it's worth, this USF FVCOM URL seems to work with your test:
http://comt.sura.org/thredds/dodsC/data/comt_1_archive/inundation_tropical/USF_FVCOM/Hurricane_Ike_2D_final_run_with_waves.html
So perhaps the secret is in the NcML we used?

I see a type=int in the NcML:
https://github.com/ioos/comt_1_archive/blob/master/inundation_tropical/USF_FVCOM/Hurricane_Ike_2D_final_run_with_waves/00_dir.ncml#L37-40

That's just adding the start_index attribute.  The connectivity array is still a real array.

OK.  Got it.

pyugrid cannot read that dataset as well.  It searches for the cf_role: 'mesh_topology'  and raises: There are no standard-conforming meshes
https://github.com/pyugrid/pyugrid/blob/master/pyugrid/read_netcdf.py#L45-46
Both individual files and the aggregation misses that attribute.

@jcothran sent me the catalog.xml they were using and I made some modifications to split into multiple separate catalogs linked via catalogRef, as well as updating the FVCOM NcML.   It looks like the FVCOM aggregation is now working?
http://nbviewer.ipython.org/gist/rsignell-usgs/d9c1e4d26390ad4eab20

Thanks a lot @rsignell-usgs.
Is this right?
http://nbviewer.ipython.org/gist/ocefpaf/80f66778394f854141ef
There several duplicated times.

Is what right?

The duplicate times.  I have 4 2014-11-01 00:00:00 for example:
import numpy as np
from datetime import datetime

time = cube.coord(axis='T')

np.where([date == datetime(2014, 11, 1, 0, 0) for date in time.units.num2date(time.points)])
(array([34115, 34199, 34283, 34367]),)


GH108: Fixed analyzers running before build

With this change both the DupFinder task and the InspecCode task will be ran after
the project have been built.
Fixes #108

@AdmiringWorm thanks for making this change!

@gep13 no problem, I'll probably open a few PR's later (tomorrow maybe) for a few other open issues.

@AdmiringWorm epic! Thank you very much! 😄


Hide top-level dialog from the taskbar

Swing has a special constructor for that (without arguments). It says:
A shared, hidden frame will be set as the owner of the dialog.

We need exactly this as a default behaviour for top-level Dialog's. This will hide the icon in the taskbar. If the user need to show icon in the taskbar, Window is a better option.
If SwingUtilities.getSharedOwnerFrame() was public, I would just use it as a default value in the constructor, but instead we have to create a separate constructor and keep it consistent with JDialog, even if it is unusual API.
The reproducer:
import androidx.compose.ui.awt.ComposeDialog
import javax.swing.SwingUtilities

fun main() = SwingUtilities.invokeLater {
    val dialog = ComposeDialog()
    dialog.setSize(800, 600)
    dialog.isVisible = true
}

The icon shouldn't be in the taskbar now.
Old behavior:

New behavior:

See also https://github.com/JetBrains/compose-jb/issues/1767#issuecomment-1027260055

I need to also show the app tray icon. How do I do that while still hiding the application icon in the dock / taskbar.
I basically need the same window / dialog management as the Jetbrains Toolbox app.


hover effect on the navigation Tabs

when user hover on the navbar Tabs Content or Text then display some animated hover effect like this :

it match foodies web Themes and color So,look is matched.
display BoxShadow and animated view

I added Navbar hover Effect checck the PR and merged It.

Hey @VanshKing30  ,i solved the conflict ,now you can check the #PR and merged it


Extract character name for all slots.

I'm in a project and I'm based on things I find on the internet, but unfortunately not everything is rosy, I can't extract the names by offset from the files, because whenever I make a save with another character the offset changes the position with the nick of the character, I'm doing it in Rust and not in C# so it could be more complicated.

There are some observations, I don't write the slots to files, I simply open file the sl2 file and give decrypt and I read the decrypted file.


Output files tracked by git?

Do we want the output files *.dat to be tracked by git?

not really, but since the files are pretty small, we dont really care if git tracks them. You can add those files to git ignore if you want.


Need prefab to build zos flist

Need to access zos (guest) created on zos (host).
Want to make sure if I mount something from host to guest that it is mounted.

Currently we don't have "automated" builder for zos flist.
There are 2 methods to build a zos flist
IPXE flist
This one contains ipxe framework with a ipxe script that boots the latest efi build from the hub plus the boot.yaml file. This currently has some issues:

Download the efi image every time doesn't make use of fs cache because the image is downloaded from the hub.
No control on which version to boot
No way (currently) to pass extra boot arguments to image (may be we need to change the embedded script)

EFI flist
This one contains the full initramfs efi file, plus of course the boot.yaml file. This one is better than the previous approach in few points:

U control which version of the efi u boot (0-core version, etc...)
It make uses of the cache because the efi is served over the fs, which means booting other zos machines on the same node will go much faster
Supporting passing extra arguments, which allow us to do shared cache with the host.

Downside

We need to build a new flist for each efi build.

Conclusion
I am not sure which flist approach is better but we need to unify. Whatever approach we take it MUST support passing extra arguments to the boot cmdline, otherwise we won't be able to have shared cache, or fine tune the zos gues (farmerid, zerotier, etc...)

auto builder flist


[Bug] output diff when temperature set zero

Checklist

[X] 1. I have searched related issues but cannot get the expected help.
[X] 2. The bug has not been fixed in the latest version.

Describe the bug
I used the latest code from LMDeploy to run the vicuna 13b model. The client used temperature 0 and made two requests, resulting in some differences in the output.
Theoretically, when the temperature is 0, multiple requests should yield consistent results without any differences.
Reproduction

modified in benchmark/profile_restful_api.py

import hashlib

def text_to_sha256(text):
    text_bytes = text.encode('utf-8')
    sha256_hash = hashlib.sha256()
    sha256_hash.update(text_bytes)
    hash_digest = sha256_hash.hexdigest()
    return hash_digest

    def _inference(self, req_queue: Queue, res_queue: Queue, session_id: int,
                   stream_output: bool):

        stats = []
        client = APIClient(self.server_addr, api_key=self.api_key)

        for prompt, input_seqlen, output_seqlen in iter(
                req_queue.get, [None, None, None]):
            timestamps = []
            timestamps.append(time.perf_counter())
            for output in client.chat_completions_v1(
                    model=self.model_name,
                    messages=prompt,
                    temperature=self.temperature,
                    top_p=self.top_p,
                    n=1,
                    max_tokens=output_seqlen,
                    stream=stream_output,
                    session_id=session_id,
                    ignore_eos=True):
                timestamps.append(time.perf_counter())
            print(text_to_sha256(output['choices'][0]['message']['content']))

results reproduction

# server
python3 -m lmdeploy serve api_server /workdir/vicuna-13b-v1.3

# client first try
python3 benchmark/profile_restful_api.py <IP_ADDRESS>:23333 /workdir/vicuna-13b-v1.3 /workdir/ShareGPT_V3_unfiltered_cleaned_split.json --concurrency 128 --num_prompts 128 --temperature 0 | tee first

# client second try
python3 benchmark/profile_restful_api.py <IP_ADDRESS>:23333 /workdir/vicuna-13b-v1.3 /workdir/ShareGPT_V3_unfiltered_cleaned_split.json --concurrency 128 --num_prompts 128 --temperature 0 | tee second

# only keep the hash results and sort
sort first -o first_output
sort second -o second_output

# compare and find the diff
icdiff first_output second_output

Environment
sys.platform: linux
Python: 3.9.16 (main, Aug 15 2023, 19:38:56) [GCC 8.3.1 20190311 (Red Hat 8.3.1-3)]
CUDA available: True
GPU 0,1: NVIDIA A100-SXM4-80GB
GCC: gcc (GCC) 10.2.1 20210130 (Red Hat 10.2.1-11)
PyTorch: 2.3.0+cu118
LMDeploy: 0.4.2+e6468e7
triton: 2.3.0

Error traceback
No response

You may need to set top_k = 1 and a fixed random seed to stop sampling because we don't have an alternative argmax kernel to bypass sampling.
Also, as split-kv is taking effect automatically, variable batch size and sequence length at runtime may result in different split-kv factor. This will lead to differnt accumulation order and thus differnt outcome.

You may need to set top_k = 1 and a fixed random seed to stop sampling because we don't have an alternative argmax kernel to bypass sampling.

In the benchmark/profile_restful_api.py, the seed is default 0 and top_p is default 1.
https://github.com/InternLM/lmdeploy/blob/21be189628d064931edc5f166ee00f61c8fb1412/benchmark/profile_restful_api.py#L226
https://github.com/InternLM/lmdeploy/blob/21be189628d064931edc5f166ee00f61c8fb1412/benchmark/profile_restful_api.py#L67
Do I still need to set top_k = 1?

Also, as split-kv is taking effect automatically, variable batch size and sequence length at runtime may result in different split-kv factor. This will lead to differnt accumulation order and thus differnt outcome.

In terms of design and implementation, do we have to ensure that when temperature is 0, under batch inference, the results of the two requests are completely consistent?

TurboMind will support do_sample sampling parameter ref https://github.com/InternLM/lmdeploy/pull/1966


Adjust axis maximum value

Is it possible to set the maximum value of a axis? For example how can I set the maximum value of the X-Axis in the simple bar chart example presented in the notebook to 1000?
  withData(Seq(
    Map("a" -> "A", "b" -> 28), Map("a" -> "B", "b" -> 55), Map("a" -> "C", "b" -> 43),
    Map("a" -> "D", "b" -> 91), Map("a" -> "E", "b" -> 81), Map("a" -> "F", "b" -> 53),
    Map("a" -> "G", "b" -> 19), Map("a" -> "H", "b" -> 87), Map("a" -> "I", "b" -> 52)
  )).
  encodeX("a", Ordinal).
  encodeY("b", Quantitative).
  mark(Bar).
  show

@Tiatzi How about this?
Vegas().
  withData(Seq(
    Map("a" -> "A", "b" -> 28), Map("a" -> "B", "b" -> 55), Map("a" -> "C", "b" -> 43),
    Map("a" -> "D", "b" -> 91), Map("a" -> "E", "b" -> 81), Map("a" -> "F", "b" -> 53),
    Map("a" -> "G", "b" -> 19), Map("a" -> "H", "b" -> 87), Map("a" -> "I", "b" -> 52)
  )).
  encodeX("a", Ordinal).
  encodeY("b", Quantitative, scale = Scale(domainValues = List(0.0, 1000.0))).
  mark(Bar).
  show

(Is my understanding of this correct?)

Thank you very much! This is exactly what I was looking for.


Create codeql-analysis.yml

Added codeql scanning on pull / push requests

Merged, reviewed security alerts.


updates to the SendGrid docs

Hi!
I'm on the SendGrid docs team, and we've received a lot of requests for us to update this page:
https://docs.microsoft.com/en-us/azure/sendgrid-dotnet-how-to-send-email#how-to-create-an-email
(particularly from the how to create an email section down to the bottom of the page, but we can verify the rest)

How can we get started with this? We're picturing a similar format, just updated content and particularly updated code examples.
Can I sign the CLA before I make the PR? This is something our legal team has to review and I prefer to start that process as early as possible.

Thanks,
Kaylyn

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

ID: 73d91616-a93f-84af-7900-5aa34b361ffd
Version Independent ID: f202a678-4853-d024-55c5-19f89558415c
Content: How to use the SendGrid email service (.NET)
Content Source: articles/sendgrid-dotnet-how-to-send-email.md
Service: multiple
GitHub Login: @thinkingserious
Microsoft Alias<EMAIL_ADDRESS>

@Erikre I was not able to assign this issue to @thinkingserious, could you let me know if you need this reassigned to someone else?
We have a request directly from the SendGrid docs team to collaborate on this document.

I've actually taken over docs from @thinkingserious - he also works at SendGrid

@ksigler7 @jason-j-MSFT @Erikre I have added the code sample for .NET Core with sendgrid C# SDK few months back. I have worked on the Sendgrid SDK as a contributor . If anything else needs to be added to the document then I can assist.

@Niladri24dutta - thanks so much for your offer! Would for sure love to work with you on this.
@jason-j-MSFT - I guess I am just confused here - why can't I make the updates I am suggesting? I was planning on submitting a PR. I filed this issue because that what your contributing guidelines suggest - file an issue on bigger requests so that you can get assigned to work with someone in house on latest guidelines and style stuff.

@ksigler7 @Niladri24dutta  you can definitely start the processes on your end right now, I'm looking for a content author who I assume will sign off on your PR.  I'll keep digging on this, but you guys please procceed, and thanks!

@ksigler7 Do you have any list of changes that needs to be incorporated?

@Niladri24dutta - so I am hoping to submit a PR with my changes in the next week or so. Would you be willing to let me tag you to code review the code examples? I am not a C# expert so I would really appreciate you double checking them.

@SyntaxC4 - Cory, could you follow up?

@ksigler7 ok I will do it.

@ksigler7 When are you going to create the PR for this? let me know when you are done.

@Niladri24dutta - sounds good! I am aiming for Monday or Tuesday - I'll let you know when I have it

Hi team, we haven't had progress on this item. Unless someone wants to keep it open, I'd like to close it.

#please-close

I'm confused as to why I'm doing the step below if I'm simply running a mail service as a .Net API on IIS running on my Azure VM:
"You may store these credentials via your Azure Portal by clicking Application settings and adding the key/value pairs under App settings."
Is this step for an Azure Virtual Machine IIS App or is it for an Azure API running in some other Azure cloud space?  Maybe I'm missing something.


Question Regarding Internet Disconnect

Hi AVS team,
I have a question regarding how the sdk should handle internet disconnects. I'm using SDK 1.8.1 in embedded linux and raspberry pi 3(for testing comparison). How can I use the sdk to handle commands when internet is disconnected? Here's a log of the device's behavior when disconnected, line 242 is when it's disconnected and asked "Alexa what's the weather" log.txt
In the echo dot, if I say "Alexa what's the weather?" while the internet is disconnected(wifi router disconnected), it listens for a while then says something like "I can't connect to the wifi". However on the embedded unit(and even in rpi3 running same sdk 1.8.1) the command triggers a listening state and just stays there for a couple of minutes then goes straight to idle. It takes 5 minutes before it checks for connection. So in those 5 minutes, calling Alexa will result in multiple listening->idle state. After 5 minutes the connection gets checked then we get to the connecting state.
I've tried decreasing DEFAULT_TEST_PERIOD from InternetConnectionMonitor from 5 minutes to 1 minute to run the testConnection() more frequently, this helps us in putting visual cues(red LEDs) to notify of no connection but that is still not fast enough. I've tried putting less than a minute and I think it still took a minute to get something in onConnectionStatusChanged(). I've also tried changing the timeout time in HTTP2Transport, and this sure did made it much faster in detecting internet disconnection. Meaning I can get from idle state to connecting state as fast as a few seconds(depending on timeout time I set) and elminating the listening->idle state. However I'm wondering if there's a better way of checking disconnects and changing the states accordingly.
I guess my question here is: Is there a way for us to determine(or run a ping/ or check connection) when in Listening state so that disconnects can be detected without the need of frequent polling/ pinging(and without that case where it stays in listening state for a few minutes)? What I'm trying to do here is implement the same reaction that the echo dot does(which I think is: go to listening state, realize that there's no connection to the server, then play a clip saying there's no internet connection).
Thanks,
jpatu001

Hi @jpatu001,
The best way to improve the dis-connectivity awareness of the device is by integrating it with the OS/HW. So, whenever a disconnection occurs, the SDK will get notified and update the state to disconnected.
Decreasing  DEFAULT_TEST_PERIOD to 1 minutes is still not fast enough and also consumes resources from the device.
Please let me know if that answers your questions.
Thanks,
Benny

@bennyAv10  yes that answers my question. By implementing a OS/HW specific program to detect disconnection from the internet, how will I be able to relay that info to the sdk? Relay in a sense that how can I inform the SDK so that it will perform the appropriate steps of such disconnection such as changing state to connecting, and notifying observers of connection status changed.
Thanks

Follow up question, I was looking at the ACL HTTP2Transport.cpp and I saw one of the default timeout define in that file is only 30 seconds(STREAM_PROGRESS_TIMEOUT which is used for streamProgressTimedout). Is this function responsible for timing out of listening state when there's no connection to the avs server? I just saw that in the logs that I attached above, it took more than 30 seconds before I got the streamProgressTimedout.

Hi @jpatu001, the STREAM_PROGRESS_TIMEOUT is used to set the time allowed for a particular message request to not see network activity before the message is considered to have timed out.
You may implement the InternetConnectionMonitorInterface interface using a OS/HW internet monitor to more quickly detect disconnection from the internet.  And with this, since your application knows that it's not connected to the internet, inside KeywordObserver::onKeyWordDetected(), you can put in the necessary visual/audio cues indicating device is no longer connected to the internet.
Hope this helps!


Close item to public comment

As City Staff,
I want to be able to close an item to public comment,
so that I don't collect public comment that doesn't have a process for being considered.

Can you distinguish between this issue and #17, or else consolidate them?

Dupe of #45


78MB hugo binary is full of paths and urls

Hi, many thanks for Hugo (hugo v0.125.2-4e483f5d4abae136c4312d397a55e9e1d39148df linux/amd64 BuildDate=2024-04-20T15:29:44Z VendorInfo=gohugoio)
I just checked the binary version I downloaded from github, and it is huge. Furthermore, it contains 42181 github.com urls and thousands of long paths.
I wonder if "strip" is finding everything it needs to in the hugo binary release. Running strip myself does not decrease the binary size.
$ du -h hugo
79M	hugo
$ strings -a -n 10 hugo  | grep -c github
42181
$ strings -a -n 10 hugo  | grep -c /root/
2412
$ strings -a -n 10 hugo  | grep -c /usr/local/
794

Does this issue reproduce with the latest release?
Yes

I just checked the binary version I downloaded from github, and it is huge. Furthermore, it contains 42181 github.com urls and thousands of long paths.

It stores the dependency information which I guess can explain some of the bove.

Sounds more like an issue to be filed in https://github.com/golang/go (if at all)

I'm closing this, this needs to be improve upstream. If there are some build flags that I dont't know of that could improve this, please open a PR with changes in:
https://github.com/gohugoio/hugo/blob/master/hugoreleaser.toml

Seems like the dependency/module information isn't going away (see https://github.com/golang/go/issues/50501#issuecomment-1010225207) but potentially the baked-in build paths can be cleaned up with the -trimpath compiler argument.

It appears there are 180 dependencies in the Hugo binary:
$ strings -n 10 ./hugo | grep -i github | grep ^dep -c
180

Many of the other github.com instances in the binary might in fact be paths, so -trimpath could well be very useful to try.

You’d get the list of built-in dependencies using go version -m like this:
# go version -m hugo | grep -e "^\s*dep" | wc -l
143


GitHub private repository sync error

Since GitHub rotated its RSA SSH Host Key (https://github.blog/2023-03-23-we-updated-our-rsa-ssh-host-key/), private packages have failed to sync …
Error: Failed to execute git clone --mirror --<EMAIL_ADDRESS>'/home/repman/.composer/cache/vcs/git-github.com-SoakDigital-wordpress-plugin-test.git/'

Cloning into bare repository '/home/repman/.composer/cache/vcs/git-github.com-SoakDigital-wordpress-plugin-test.git'...
remote: Repository not found.
fatal: repository<EMAIL_ADDRESS>not found

Logs:
Failed to clone the git@github.com:SoakDigital/wordpress-plugin-test.git repository, try running in interactive mode so that you can enter your GitHub credentials

I have found that by creating a new organisation even as the same user I am able to add these packages without issue.
Suspect each organisation needs to have authentication kicked to resume service, but as this is a hosted service, I don’t have the tools to do it.

(For reference, the organisation name is soak-digital)

Same error here :(


Running BrowserFetcher().DownloadAsync on Raspberry Pi downloads x64 version instead of ARM

Description
Running BrowserFetcher().DownloadAsync on Raspberry Pi downloads x64 version instead of ARM.
Complete minimal example reproducing the issue
await new BrowserFetcher().DownloadAsync(BrowserFetcher.DefaultRevision);

Expected behavior:
Compatible ARM build downloads.
Actual behavior:
X64 Linux build downloads.
Versions

PuppetSharp 1.17.2
.NET Core 2.2

Additional Information
The workaround is to sudo apt-get install chromium-browser and then
var options = new LaunchOptions() {
            ExecutablePath = "/usr/bin/chromium-browser",
            Headless = true
};
var browser = await Puppeteer.LaunchAsync(options);

There are some recipes here https://github.com/GoogleChrome/puppeteer/issues/550
Puppeteer is not downloading a chrome version for ARM.

Is that an unwanted feature?
I had a look at https://github.com/GoogleChrome/puppeteer/blob/master/lib/BrowserFetcher.js which could easily be expanded to handle ARM. The only question is whether there is an ARM binary on https://storage.googleapis.com/chromium-browser-snapshots/index.html, which is a bit of a mess to navigate.
If there's interest I can look into creating a PR.

@aengl yeah you could create an issue there. The main question is if they are building ARM binaries for every revision.


[Messages] Incorrect timezone when message from user is received

Environment: Windows 10 Pro 22H2 Google Chrome Version 120.0.6099.225 (Official Build) (64-bit)
Reproducible: always
Preconditions

Site https://zero-waste-staging.onrender.com/en is opened.

Steps to reproduce

Go to 'Contact Us' tab
Fill in Title (can't be less than 5 characters)
Fill in Email (in format<EMAIL_ADDRESS>Fill in the message text
Press 'Send' button
Log in as Admin
Go to 'Admin' page
Go to 'Messages' page

Actual result
Time in 'Messages' is different from current time
Postconditions
Timezone should be the same
User story and test case links
E.g.: "User story https://github.com/ita-social-projects/ZeroWaste/issues/657

Fixed


Show the right icon in Dashboard for TXT/ODT/DOC/PDF

In the dashboard, when displaying Journal content TXT/ODT/DOC/PDF (all coming from Write activity), a generic icon is displayed.

In Sugarizer app, due to this, a specific icon is displayed for these contents.

Fixed in 030a3.

Look like it should be change on the home screen too:

Fixed in https://github.com/llaske/sugarizer-server/pull/228


Bottom lines of Etherpads invisible

When viewing a large Etherpad, the bottom ten or so lines are invisible - they fall outside the screen.

Which device (and iOS version) are you using? I can't reproduce this on an iPhone X or an iPad Pro 10.5.
I'm testing with an Etherpad with just over 50 lines, do I need more?

Let me check - I am reporting this for some Chinese people, not for myself.
---- On di, 14 aug 2018 23:33:58 +0200 Aaron Raimist<EMAIL_ADDRESS>wrote ----

Which device (and iOS version) are you using? I can't reproduce this on an iPhone X or an iPad Pro 10.5.
I'm testing with an Etherpad with just over 50 lines, do I need more?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

One person answered: she has an iPhone 8.  See also the attachment.
No truly long pads are required - just enough to not fit on the screen.
---- On di, 14 aug 2018 23:33:58 +0200 Aaron Raimist<EMAIL_ADDRESS>wrote ----

Which device (and iOS version) are you using? I can't reproduce this on an iPhone X or an iPad Pro 10.5.
I'm testing with an Etherpad with just over 50 lines, do I need more?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I don't see any attachment

Here it is.

It was in my mail message, but didn't end up here.

Sorry, didn't mean to close.

The other person reports having a Huawei Maimang running Android.
---- On wo, 15 aug 2018 01:48:52 +0200 Aaron Raimist<EMAIL_ADDRESS>wrote ----

I don't see any attachment
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


[Add]

URL
https://icons8.com/
Description of the above provided site
Icons8 website provides icons, illustrations ,photos and design tools both in free and paid form.
Category
Frontend
Sub Category
"fonts", "icons", "images" ,"music","illustrations"

Thank you Jay!

Can I work on this?

Can I work on this?

yes sure, assigning to you

@jayk-gupta Don't you wanna work on it now?

The issue has been resolved, thanks for closing though!

I have sent a PR for a youtube channel link, kindly review and merge it.
I haven't created an isuue for it, do I need to create it first?


Centralized RPC Providers

edit
fixed! infura now supports this

At time of this writing neither Infura nor any RPC provider's support eth_getProof, but infura should in the very near future. ETC chain supports the call iff using multigeth, but against no service central services are known to be running it.
Implications
getting storage Proofs and Account proofs wont work unless your RPC provider supports eth_getProof. Solution is to run geth/parity-ethereum/multigeth locally for those Additionally it needs to be a full or archive node if you want to get historic proofs.

Try this one liner from the console to see if it's working yet. Or "http://mainnet.infura.io" with whatever URL you want to test:
curl -X POST --data '{"jsonrpc":"2.0","method":"eth_getProof","params":["0x9cc9bf39a84998089050a90087e597c26758685d",["","d471a47ea0f50e55ea9fc248daa217279ed7ea3bb54c9c503788b85e674a93d1"],"latest"],"id":1}' -H "Content-type:application/json" https://mainnet.infura.io

wow! looks like infura finally supports it!

note to self: update readme then close issue

Try this one liner from the console to see if it's working yet. Or "http://mainnet.infura.io" with whatever URL you want to test:
curl -X POST --data '{"jsonrpc":"2.0","method":"eth_getProof","params":["0x9cc9bf39a84998089050a90087e597c26758685d",["","d471a47ea0f50e55ea9fc248daa217279ed7ea3bb54c9c503788b85e674a93d1"],"latest"],"id":1}' -H "Content-type:application/json" https://mainnet.infura.io

How do I know what params to send to get the data of a specific block?
This request works, but I have no clue how to construct the params, and getProof is nowhere to be found in infuras own documentation.

The above curl request is just for troubleshooting that your connection is working. Please read the API for this JS module. It does not require the same arguments.


'certificate verify failed' since today

Yotta does not work since today. It worked fine yesterday. What happened to my environment or the server? The error is as follows:
error: connection error: ("bad handshake: Error([('SSL routines', 'ssl3_get_server_certificate', 'certificate verify failed')],)",)

My environment is yotta-0.17.2 and Python 2.7.12 on Ubuntu 16.04 LTS.

Thanks @sgk,
It seems an SSL certificate has expired, please bear with us while we get it renewed.

Oh, I see. Thank you for keeping the system up and running.

Maybe you should have an unsafe option ie
yotta unsafe target bbc-microbit-classic-gcc
that will allow users NOT to use SSL
What about a simple md5sum check ?
All I need is to install an IDE/compiler to program my BBC microbit
I tried in vain for 8 hours yesterday , needless to say that I was/am pretty upset
This is the last time I use an mbed/yotta project
My next project will use an Arduino
Anyway thanks for your hard work

If you want to quickly hack around this, you could add something like this after the line that imports the request module in yotta/lib/registry_access.py:
import requests

_old_get = requests.get
def newGet(*args, **kwargs):
    kwargs['verify'] = False
    return _old_get(*args, **kwargs)
requests.get = newGet

(untested, but you get the idea)
The SSL is there for good reason though – without it if you were using yotta on an open wifi network then anyone else would be able to intercept your requests to the registry and insert malicious responses (or worse if you were publishing a module: they would be able to insert malicious code into the published module)

Thanks for your patience with this issue. We believe we have fixed it, but I'll keep this issue open for a day or so just in case.
Apologies on the untimely resolution, there was an issue with our hosting provider.


Naming convention (binsize -> bin_size, etc.)

It's hard to do domestic cleaning of the modules and functions that require a rewrite, stripping the C style, which is still present in many elephant functions.
IMPORTANT. The old arguments are deprecated but not removed. Even though it looks like they are removed from the code, the old signature will still work - it just throws a warning, which can become a hard error in 2 or 3 releases since then, if we want.
Below is a compiled list, made by elephant doc team with :heart:
Deprecated arguments mapping

binsize --> bin_size
num_bins --> n_bins in all modules expect GPFA transform_info dict that has the key 'num_bins'; commit 0408896b2e98903b98ca7fd0f63e8c7bb1a33bf2
binned_st[s] --> binned_spiketrain because binned_spiketrains, as spiketrains, would imply a list of BinnedSpikeTrain objects
cubic data -> histogram
victor_purpura_dist

trains --> spiketrains
q -> cost_factor; is there a particular reason, a paper, where q is used?

cv2 and lv v --> time_intervals
cell_assembly_detection

data --> binned_spiketrain
maxlag --> max_lag
min_occ -> min_occurences

cross_correlation_function

ch_pairs --> channel_pairs
nlags --> n_lags
env -> hilbert_envelope

lst_input -> input_spiketrains in BinnedSpikeTrain; closes #311
hilbert N --> padding (N is passed in scipy.signal.hilbert, which has this argument)
wavelet_transform

nco -> n_cycles

welch_psd, welch_cohere

num_seg -> n_segments
len_seg -> len_segment
freq_res -> freq_resolution

spike_extraction extr_interval -> interval; since the function name already has the key "extraction", I think it's clear what the interval means.
peak_detection format -> as_array
single_interaction_process

rate_c -> coincidence_rate
return_coinc -> return_coincidences

JointISI refr_period -> refractory_period
surrogates surr_method -> method
sskernel --> optimal_kernel_bandwidth

w -> bandwidth
tin -> times

spike_train_surrogates module

n --> n_surrogates

estimate_csd coords -> coordinates
kernels t -> time
extract_neo_attrs  -> extract_neo_attributes

obj -> neo_object

generate_lfp

ele_xx -> x_positions or positions_x ?
xlims -> x_limits or limits_x?
res -> resolution;

multiple_filter_test

dt -> time_step or step

Pending/Unsure
General concern:

num_bins or n_bins? Our convention tells n_bins is the way to go, but num_bins is used in so many places in elephant...
how much is fs confusing? For me, an outsider, it was always clear that it means the sampling frequency of a signal; you can also find fs in other great signal processing packages like scipy. I don't know a typical neuroscientist, though. fs is used in 4 elephant functions: butter, wavelet_transform, welch_psd, and welch_cohere.
same for nfft. It became a standard over the years, even though it does not comply with our convention, which renders this variable as n_fft.
_freq -> _frequency? freq is used 4 times.

Specific concerns

cubic ximax -> max_iterations

cell_assembly_detection same_config_cut -> same_configuration_pruning? (Perhaps also revise the parameter description)

welch_cohere x, y -> signal_x, signal_y

spike_train_correlation module

corrcoef() -> corr_coef()? There are corrcoef_ input arguments as well.
cross_corr_coef -> cross_corr_coefficient? Rather not because otherwise we need to broadcast it to cross_correlation_coefficient which may look too large for such a popular method.

compound_poisson_process A -> amplitude_distribution; unsure because A looks suspiciously tailored to a paper or a general concept (like fs stands for the sampling rate) brought from matlab.

surrogates dt -> time_shift or time_delta? On the other hand, dt is already self-descriptive.

multiple_filter_test

t_final -> t_stop

empirical_parameters

dt -> time_step or step
t_final -> t_stop

welch_cohere() -> welch_coherence() ?

single_interaction_process

n --> n_spiketrains or n_generate?

Variable suffix issues
We also have variable indexing inconsistency (pay attention to suffixes). Although readable, I don't know how this would impact the integration with NEST D.
$ grep -P 'def (?!(_[a-z]|test)).*\(.*spiketrains?(?![s,\)])' -R elephant/ --include=*.py
elephant/asset.py:    def __init__(self, spiketrains, spiketrains_y=None, bin_size=3 * pq.ms,
elephant/spike_train_correlation.py:    def __init__(self, binned_spiketrain1, binned_spiketrain2, window):
elephant/spike_train_correlation.py:    def get_valid_lags(binned_spiketrain1, binned_spiketrain2):
elephant/spike_train_correlation.py:def spike_time_tiling_coefficient(spiketrain_1, spiketrain_2, dt=0.005 * pq.s):
elephant/spike_train_correlation.py:    def run_P(spiketrain_1, spiketrain_2):

$ grep -Pi 'def (?!(_[a-z]|test)).*\((.*_)?x, (.*_)?y' -R elephant/ --include=*.py
elephant/spectral.py:def welch_cohere(x, y, n_segments=8, len_segment=None, freq_resolution=None,
elephant/current_source_density_src/utility_functions.py:def distribute_srcs_2D(X, Y, n_src, ext_x, ext_y, R_init):
elephant/current_source_density_src/utility_functions.py:def distribute_srcs_3D(X, Y, Z, n_src, ext_x, ext_y, ext_z, R_init):
elephant/current_source_density_src/utility_functions.py:def large_source_2D(x, y):
elephant/current_source_density_src/utility_functions.py:def small_source_2D(x, y):
elephant/current_source_density_src/utility_functions.py:def gauss_3d_dipole(x, y, z):
elephant/current_source_density.py:    def integrate_2D(x, y, xlin, ylin, csd, h, X, Y):
elephant/current_source_density.py:    def integrate_3D(x, y, z, xlim, ylim, zlim, csd, xlin, ylin, zlin,

Not refactored
Because they require special care (ASSET already follows the naming convention):

SPADE
Unitary Event Analysis

Also, I didn't pay attention to _protected functions not because they can avoid the naming convention but because I didn't want to spend time refactoring them. Keep in mind that any decent renaming may lead to breaking the logic and thus should be justified.
Won't change

homogeneous_gamma_process a is the shape and b is the rate of a distribution will not be renamed to avoid a misinterpretation of the rate of the outcome spiketrain, which is equal to b/a and not b.

Hello @dizcza! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file elephant/spade.py:

Line 1147:80: E501 line too long (80 > 79 characters)

In the file elephant/test/test_conversion.py:

Line 452:80: E501 line too long (80 > 79 characters)
Line 464:80: E501 line too long (80 > 79 characters)
Line 494:80: E501 line too long (80 > 79 characters)

In the file elephant/test/test_spade.py:

Line 152:80: E501 line too long (80 > 79 characters)
Line 224:80: E501 line too long (80 > 79 characters)
Line 272:80: E501 line too long (80 > 79 characters)
Line 376:80: E501 line too long (80 > 79 characters)

In the file elephant/test/test_sta.py:

Line 276:80: E501 line too long (80 > 79 characters)
Line 308:80: E501 line too long (80 > 79 characters)

In the file elephant/unitary_event_analysis.py:

Line 810:80: E501 line too long (81 > 79 characters)
Line 815:80: E501 line too long (81 > 79 characters)

Coverage decreased (-0.006%) to 88.901% when pulling c77654db62309f65ffb51e0abf1047122b8df0ae on INM-6:naming-convention/bin_size into 88682f06e090f157e24cdf83b0cb51689c3fe303 on NeuralEnsemble:master.

/polls Option1 'Option 2' "Option 3"

Regarding cross_coef/cross_corr_coef in spike_train_correlation:
The parameter in cross_correlation_histogram function is passed to the protected class _CrossCorrHist. Therefore, according to the conventions of just using corr as an abbreviation for correlation, cross_corr_coefficient will make the information clearer. The fact that it is passed internally, I don't see as an annoyance of a larger parameter name.
Moreover, in _CrossCorrHist, the class method should be cross_correlation_coefficient to keep the standard as in spike_time_tiling_coefficient, and to align to what we discussed regarding keeping the Elephant function names without abbreviations. The latter also applies to function corrcoef. Although NumPy has this function, in Elephant we are talking about an implementation that works with elephant.BinnedSpikeTrain inputs. I don't see any benefit in shadowing NumPy.
Finally, in the class _CrossCorrHist, the current cross_corr_coef function takes an argument named only cross_corr. It should also define which is the input, as cross_corr* is a prefix for histogram or matrices. Therefore, something in the direction of cross_corr_array or cross_corr_matrix will make the code clear.

Regarding _x and _y or _1 and _2 in welch_cohere and ASSET:
I don't think they can be analyzed together. The idea of _1 and _2 is for the functions where the parameter order does not matter, which is the case of welch_cohere.
In ASSET, the spiketrains_y will actually reflect in the interpretation of the matrices axes. Therefore, _x and _y is meaningful with respect to the method output. ASSET is tough because you don't need both parameters, but I think it is better to name the first (non-optional) as _x and the optional as _y.

@kohlerca Regarding _x and _y or _1 and _2 in welch_cohere and ASSET.
Also in welch_cohere, the order of parameters makes a big differene when interpreting the phase_lag return value, since the order defines the reference and the referenced, so to say. In this way, I think the situation is very similar to ASSET. Now, I think the main reason we dislike 1 and 2 is because it looks terrible in the variable name, and I think we dislike x and y because it is commonly associated to a position. How about a compromise and using i and j? This is commonly used to identify this type of relationship...

@mdenker How about a compromise and using i and j? This is commonly used to identify this type of relationship...
I guess this makes total sense and it is better indeed than either of the previous options.

@mdenker How about a compromise and using i and j? This is commonly used to identify this type of relationship...
I guess this makes total sense and it is better indeed than either of the previous options.

Added _i and _j option in the poll.


When variability is set to 0, initial trait value cannot be changed (random walk)

Fixed with this commit


Update AWS

Closes #3210

What is wrong with Jenkins, it doesn't always auto build

He's just a bit overwhelmed these days. He'll grow into it. :D


Does youtube-dl support corporate authentication on Lynda.com?

I'm struggling to download from Lynda.com using corporate login. I used cookie.txt Chrome extension to bypass login parameters for the command below but it only downloads to the 4 free videos.
youtube-dl --cookies cookies.txt "https://www.lynda.com/vSphere-tutorials/VMware-vSphere-Certified-Professional-vSphere-6-Cert-Prep-2V0-621-Basics/583671-2.html" -o "%(playlist)s [Lynda.com]/%(chapter_number)s - %(chapter)s/%(playlist_index)s - %(title)s.%(ext)s"
[lynda:course] 583671: Downloading course JSON
WARNING: 16 videos are only available for members (or paid members) and will not be downloaded. Use --username and --password options to provide lynda.com account credentials.
[download] Downloading playlist: VMware vSphere Certified Professional on vSphere 6 Cert Prep (2V0-621): The Basics
[lynda:course] playlist VMware vSphere Certified Professional on vSphere 6 CertPrep (2V0-621): The Basics: Collected 4 video ids (downloading 4 of them)
[download] Downloading video 1 of 4
[lynda] 628407: Downloading video JSON
[lynda] 628407: Checking 1-540 video format URL
[lynda] 628407: Checking 1-720 video format URL
[lynda] 628407: Checking 1-360 video format URL
[lynda] 628407: Checking 0-540 video format URL
[lynda] 628407: Checking 0-720 video format URL
[lynda] 628407: Checking 0-360 video format URL
[download] Destination: VMware vSphere Certified Professional on vSphere 6 CertPrep (2V0-621) - The Basics [Lynda.com]\1 - Introduction\1 - Welcome.mp4
[download] 100% of 4.96MiB in 00:03
[download] Downloading video 2 of 4
[lynda] 628408: Downloading video JSON
[lynda] 628408: Checking 1-540 video format URL
[lynda] 628408: Checking 1-720 video format URL
[lynda] 628408: Checking 1-360 video format URL
[lynda] 628408: Checking 0-540 video format URL
[lynda] 628408: Checking 0-720 video format URL
[lynda] 628408: Checking 0-360 video format URL
[download] Destination: VMware vSphere Certified Professional on vSphere 6 CertPrep (2V0-621) - The Basics [Lynda.com]\1 - Introduction\2 - Before we begin.mp4
[download] 100% of 7.05MiB in 00:07
[download] Downloading video 3 of 4
[lynda] 628411: Downloading video JSON
[lynda] 628411: Checking 1-540 video format URL
[lynda] 628411: Checking 1-720 video format URL
[lynda] 628411: Checking 1-360 video format URL
[lynda] 628411: Checking 0-540 video format URL
[lynda] 628411: Checking 0-720 video format URL
[lynda] 628411: Checking 0-360 video format URL
[download] Destination: VMware vSphere Certified Professional on vSphere 6 CertPrep (2V0-621) - The Basics [Lynda.com]\2 - 1. Exam Details\3 - Exam blueprint.mp4
[download] 100% of 8.23MiB in 00:05
[download] Downloading video 4 of 4
[lynda] 628417: Downloading video JSON
[lynda] 628417: Checking 1-540 video format URL
[lynda] 628417: Checking 1-720 video format URL
[lynda] 628417: Checking 1-360 video format URL
[lynda] 628417: Checking 0-540 video format URL
[lynda] 628417: Checking 0-720 video format URL
[lynda] 628417: Checking 0-360 video format URL
[download] Destination: VMware vSphere Certified Professional on vSphere 6 CertPrep (2V0-621) - The Basics [Lynda.com]\3 - 2. The Exam Blueprint\4 - VMware vSphere - Configure and manage networking.mp4
[download] 100% of 6.52MiB in 00:04
[download] Finished downloading playlist: VMware vSphere Certified Professionalon vSphere 6 Cert Prep (2V0-621): The Basics

https://chrome.google.com/webstore/detail/cookiestxt/njabckikapfpffapmjgojcnbfjonfjfg
cookies for this tab works fine

I've created a simple cookies.txt exporter extension which I've verified to work with a lynda.com organizational account. Please try it and see if it solves your problem (it is available for both Firefox and Chrome):

https://github.com/rotemdan/ExportCookies

The issue with some exporters (such as the cookies.txt extension for Firefox) is that they add the #HttpOnly_ prefix to HttpOnly cookies. This doesn't seem to be parsed correctly by youtube-dl. The exporter I published doesn't add this prefix.
Of course the browser need to be properly logged in to lynda.com (or any other website) for this to work.
Please let me know of any issues.


Humble with Python 3.10 SOLVED

For anyone who is using the Doosan Humble Pkg and has a Problem with the DR_common2.py
Just replace "collections.Iterable" with  "collections.abc.Iterable" in that file. Since Python 3.10 there is not "collections.Iterable".

Thank you for your feedback @Dorianproject
I will close this issue.


Bug When deleting a published API

Description:
When a user with only create permissions tries to delete an API both no Error message shown to the end user  and the following logs in the console.
ERROR - ApisApiServiceImpl Cannot remove the API 7d2878da-9950-468c-8e5b-d79837a0c055 as active subscriptions exist

Fixed. A popup message is shown with the error message


RasaNLUHttpInterpreter - Model name required

Rasa Core version: 0.9.3
Python version: 2.7
Operating system (windows, osx, ...): OSX
Issue:
Hello, I use Rasa NLU over http and in Rasa NLU API model_name if not given is essentially taking the latest version of the model.
When I see the implementation at rasa core in Interpreter.py, default value of project is "default" but model_name is a required parameter. Should it not be optional?
My models in rasa NLU are generated with a timestamp and each time I train the NLU, in this case I will have to restart rasa core and provide a new model name.
I am not sure if it is best practice to just allow rasa core to look for the latest model in the NLU  project directly.

is this running the server.py script?

Yeah, I pass the Http interpreter of Rasa

@tmbo do you agree with the approach above? I can make a PR for this, it is simple addition basically.

Yes good point :+1:


Add params model in route options

See https://github.com/ruby-grape/grape-swagger/issues/657#issuecomment-581653945 for the reason behind this PR.
This solution is not the cleanest but it actually give us a way to have a decent model name in the documentation. It can be temporary.
I didn't read the source code all over, but I don't understand why the params key doesn't have the same behavior as the success one. I think it could be a permanent solution.

Coverage increased (+0.002%) to 99.453% when pulling 6b0cbf0c40b665d1f32a3fd538b63f1e70f448fd on PhilippeChab:add-params-model-name-in-route-options into fada2560b6cc329a7dc7792d2f84ae47cfd2a8ed on ruby-grape:master.

Coverage increased (+0.002%) to 99.453% when pulling 6b0cbf0c40b665d1f32a3fd538b63f1e70f448fd on PhilippeChab:add-params-model-name-in-route-options into fada2560b6cc329a7dc7792d2f84ae47cfd2a8ed on ruby-grape:master.

I've experimented more with grape, grape-swagger and grape-entity and I just don't understand the way it's designed at the moment.
Why would you want to add a params key in your desc block like this:
desc: "Some endpoint",
    params: API::V1::Entities::SomeEntity,
    ...
get ":id" do
...

since you can already add documentation to your params with grape this way:
params do
    requires :some_param, type: String, documentation: { desc: "Some description", in: "body" }
    ...
get ":id" do
...

Because encapsulating the params into an entity is a cleaner way to do this? Yes, I agree. But the problem is, the params in the desc: ... block are not validated, so you absolutely need to add the route params in the params block. What is the point to document the params in both block then? I don't see any reason, which would mean the params key in the desc block is useless. If there is something I am missing, please tell.
That being said, I still think entities are a really nice facade to endpoint responses and I don't understand why they can't be use directly in grape params block for endpoint params. Why can't we do something like this:
des "...",
    ....
params API::V1::Entities::SomeEntity
get ":id" do
    ...

This would make an entity really great, since it could provide validation, documentation and encapsulation all at once.
I'm closing this PR since I don't see any reason to integrate more support to the params key.


Add test of phase contrast method

to restore code coverage.
Also use units in the PC example.

Codecov Report
Patch coverage has no change and project coverage change: +5.02 :tada:

Comparison is base (992e9eb) 94.97% compared to head (e651389) 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@             Coverage Diff             @@
##             main       #24      +/-   ##
===========================================
+ Coverage   94.97%   100.00%   +5.02%
===========================================
  Files          10        10
  Lines         338       338
===========================================
+ Hits          321       338      +17
+ Misses         17         0      -17

see 1 file with indirect coverage changes

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


Update available notification should only appear once.

Arguably a feature request.
My Code workflow involves opening a new window quite frequently, e.g., to switch between repos or branches in the same repo (using git worktrees).
The "Restart Visual Studio Code to apply the latest update." notification pops up every time I open a window. I promise I'll restart code someday, but I actually use it for my job and I have a bunch of embedded terminals running that lose state when I restart.
In the meantime, I guess I'll just turn updates off? Seems like a not-great workaround
https://github.com/microsoft/vscode/issues/103907 does request an ability to squelch notifications entirely, but I don't want that.

It's driving me nuts. I think it's because the latest update just doesn't seem to work anyway. It keeps asking for updates even after updating, which exacerbates the problem.


Create POST External User API

Describe the task
Create an API to save external user profile information
Acceptance Criteria

[x] Data is saved to the database
[x] returns appropriate response codes and messages
[x] unit test cases implemented

merged to dev


a simple question about saving npzfile

Hi, I have tried to use np.savez_compressed() to save the npz and read data from the npz I created, it still is NpzFile object, not dict object.
this issue

I can use the key to read values from the object. However, it is a NpzFile object, not a dict object

I think it is by design😂

I can use the key to read values from the object. However, it is a NpzFile object, not a dict object

I think it is by design😂

My question is how to save the npz so that I can get a dict object when I read it.

use dict(np.load(path))
For related question, I suggest you read numpy or python documentation, since it is certainly beyond the scope of this repo.

May I ask how the npz file was generated？

May I ask how the npz file was generated？

use pickle.dump

May I ask how the npz file was generated？

use pickle.dump

sorry，I would like to know how paid_info is generated in the npz file？look forward your reply

May I ask how the npz file was generated？

use pickle.dump

sorry，I would like to know how paid_info is generated in the npz file？look forward your reply

You could refer here:  https://github.com/mihaidusmanu/d2-net/blob/master/megadepth_utils/preprocess_scene.py

May I ask how the npz file was generated？

use pickle.dump

sorry，I would like to know how paid_info is generated in the npz file？look forward your reply

You could refer here: https://github.com/mihaidusmanu/d2-net/blob/master/megadepth_utils/preprocess_scene.py

It seems that there is no calculation related to paid_info

May I ask how the npz file was generated？

use pickle.dump

sorry，I would like to know how paid_info is generated in the npz file？look forward your reply

You could refer here: https://github.com/mihaidusmanu/d2-net/blob/master/megadepth_utils/preprocess_scene.py

It seems that there is no calculation related to paid_info

what is 'paid_info' ?

pair_infos

pair_infos

https://github.com/zju3dv/LoFTR/issues/85

pair_infos

#85

Do you have the corresponding code？

pair_infos

#85

Have you generated MegaDepth's indices yet？I searched for many days but couldn't find a way to solve it


Bitmex API causes getting banned

Hey,
I was trying out the auto sl and tp script and it resulted in having my bitmex account banned. When  I am trying to visit bitmex.com it shows me "forbidden 403" error.
Any ideas what might be causing this and how to fix it?
All I did was download the scripts and copy/paste my api keys.

Hi, thanks for reporting this. I'll review the script soon.

Hi all, everything is working fine. I have tested it with the testnet (changing the url) and it works good ! thanks

I only put a quick-fix out, but I'll work on more robust error-handling procedures soon.


[CELEBORN-1311] Developers Doc introduce Slots allocation

What changes were proposed in this pull request?
Developers Doc introduce Slots allocation.
Why are the changes needed?
Navigation of mkdocs.yml does not include the link of slotsallocation.md, which should be introduced.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
No.

LGTM. Merged into main.


Update Slugify extension - support 'strict' especially useful for apostrophes

Description
Quick little update because I was using the extension today to slugify some blog post titles and the apostrophes were not removed which is not ideal for post URLs. So the extension also shows Strict slugs.
Screencast
https://github.com/user-attachments/assets/42d90620-5764-4b56-a016-60c481c9e33a
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Thanks @xmok for such great finding and work. But let's keep old number of menu items but make them strict (without strict badge)
What do you think?

I'm fine with that but just to give the option, we could do what I did in a previous iteration of the extension i.e. we can move the Strict options into a List.Dropdown defaulting to Strict - this way on the off-chance someone does want the non-strict option, they can select it.

Updated screencast 😄.
https://github.com/user-attachments/assets/fa78f179-7a52-49e3-b7a9-49e97719b5d7

@xmok thanks, let's go this way


Reduce the size of the receipt to meet localStorage limitations

Fixes #100

This change is

Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Add order to pagination params

Codecov Report

Merging #124 (99a4cf9) into main (c337704) will increase coverage by 0.06%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #124      +/-   ##
==========================================
+ Coverage   94.88%   94.95%   +0.06%
==========================================
  Files          19       19
  Lines         528      535       +7
==========================================
+ Hits          501      508       +7
  Misses         27       27

Impacted Files
Coverage Δ

workos/directory_sync.py
100.00% <100.00%> (ø)

workos/organizations.py
100.00% <100.00%> (ø)

workos/sso.py
100.00% <100.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

not sure how optional args work in python but I think we might need to move the order param to be the last param like @sheldonvaughn suggested in the SDK PR for PHP to make sure that this isn't a breaking change.

not sure how optional args work in python but I think we might need to move the order param to be the last param like @sheldonvaughn suggested in the SDK PR for PHP to make sure that this isn't a breaking change.

I don't think param ordering conventions matter much for Python, as they use assignment operators, but I will change the ordering across the board just to match for consistency. Originally I was thinking of setting it after limit because that would make the most sense when thinking about pagination, where you would set limit and order first

Yeah I totally agree. I also think that the new planed methods for pagination would probably render these params obsolete so I totally agree with you.


Flake8 Fails Silently if Path to Binary Wrong

Issue Type: Bug
Behaviour
I noticed when debugging Issue #4074 that flake8 obviously fails to run when
"python.linting.flake8Path"

is set incorrectly (i.e. path is wrong), but vscode does not issue a warning that it cannot find flake8 (i.e. that the path is incorrect).
In my case, I accidentally typed ${workspacefolder} in all lowercase instead of camelCase (i.e. ${workspaceFolder}), so the variable didn't expand.
I have editor.formatOnSave set to true, so after saving my python code, the OUTPUT tab of vscode showed this:
.\.venv\Scripts\python.exe ~\.vscode\extensions\ms-python.python-2022.6.0\pythonFiles\linter.py -p flake8 ${workspacefolder}/.venv/Scripts/flake8.exe the_file.py

This is how I noticed I typed in the variable incorrectly.
Expected vs. Actual
vscode doesn't issue a warning that it cannot find flake8, but fails silently (flake8 doesn't run).
Instead, it would be preferrable that vscode checks the path/binary exists (or that there be a style checker for settings.json that checks if variables like ${workspaceFolder} are typed correctly).
Steps to reproduce:

Create a python virtual envrionment in a project folder

> py -3.8 -m venv .venv

pip install flake8

> py -m pip install flake8

Set editor.formatOnSave to true and add flake8 to linters in workspace settings.json, but type ${workspacefolder} with incorrect case sensitivity

{
    "editor.formatOnSave": true,
    ...
    "python.linting.flake8Enabled": true,
    "python.linting.flake8Path": "${workspaceFolder}/.venv/Scripts/flake8.exe",
    ...
}

Now, flake8 errors will not be caught (linted) by vscode and user will not know why unless thy read the OUTPUT tab log and discover their path to flake8 is incorrect.
Diagnostic data

Python version (& distribution if applicable, e.g. Anaconda): 3.8.10
Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): Poetry
Value of the python.languageServer setting: Default

User Settings

defaultLS: {"defaultLSType":"Pylance"}

envFile: "<placeholder>"

venvPath: "<placeholder>"

venvFolders: "<placeholder>"

condaPath: "<placeholder>"

pipenvPath: "<placeholder>"

poetryPath: "<placeholder>"

languageServer: "Pylance"

linting
• enabled: true
• cwd: "<placeholder>"
• Flake8Args: "<placeholder>"
• flake8Enabled: true
• flake8Path: "<placeholder>"
• lintOnSave: true
• banditArgs: "<placeholder>"
• banditEnabled: true
• banditPath: "<placeholder>"
• mypyArgs: "<placeholder>"
• mypyEnabled: true
• mypyPath: "<placeholder>"
• pycodestyleArgs: "<placeholder>"
• pycodestyleEnabled: false
• pycodestylePath: "<placeholder>"
• prospectorArgs: "<placeholder>"
• prospectorEnabled: false
• prospectorPath: "<placeholder>"
• pydocstyleArgs: "<placeholder>"
• pydocstyleEnabled: false
• pydocstylePath: "<placeholder>"
• pylamaArgs: "<placeholder>"
• pylamaEnabled: false
• pylamaPath: "<placeholder>"
• pylintArgs: "<placeholder>"
• pylintPath: "<placeholder>"

sortImports
• args: "<placeholder>"
• path: "<placeholder>"

formatting
• autopep8Args: "<placeholder>"
• autopep8Path: "<placeholder>"
• provider: "black"
• blackArgs: "<placeholder>"
• blackPath: "<placeholder>"
• yapfArgs: "<placeholder>"
• yapfPath: "<placeholder>"

testing
• cwd: "<placeholder>"
• debugPort: 3000
• nosetestArgs: "<placeholder>"
• nosetestsEnabled: undefined
• nosetestPath: "<placeholder>"
• promptToConfigure: true
• pytestArgs: "<placeholder>"
• pytestEnabled: true
• pytestPath: "<placeholder>"
• unittestArgs: "<placeholder>"
• unittestEnabled: false
• autoTestDiscoverOnSaveEnabled: true

terminal
• activateEnvironment: true
• executeInFileDir: "<placeholder>"
• launchArgs: "<placeholder>"

experiments
• enabled: true
• optInto: []
• optOutFrom: []

tensorBoard
• logDirectory: "<placeholder>"

Extension version: 2022.6.0
VS Code version: Code 1.67.0 (57fd6d0195bb9b9d1b49f6da5db789060795de47, 2022-05-04T12:06:02.889Z)
OS version: Windows_NT x64 10.0.19042
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Xeon(R) Gold 6248R CPU @ 3.00GHz (48 x 2993)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
190.65GB (115.37GB free)

Process Argv
--crash-reporter-id 9e9db912-c31c-4be1-8c40-4080829b5095

Screen Reader
no

VM
0%

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256859
pythonvspyl392:30443607
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscod805cf:30301675
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30449410
pythonvsnew555:30457759
vscscmwlcmt:30465135
cppdebugcf:30475217

Please try the new flake8 extension . It should handle these scenarios better.


$httpBackend coersion of falsy values to null before xhr.send()

Can anyone explain why this line in $httpBackend should coerce falsy values to null?
xhr.send(post || null);

https://github.com/angular/angular.js/blob/master/src/ng/httpBackend.js#L112
This means that, if I need to POST, PUT, etc. an actual boolean false to a web service, I have to work around AngularJS.
I'd like the line to read something like this:
xhr.send(post);

or if it just can't be undefined or something:
xhr.send(isDefined(post) ? post : null);

@johnhoffman I think that your analysis is correct, we are too eager with falsy checks here. Would you be up to sending us a PR. The line to be modified is this one

I've initiated PR https://github.com/angular/angular.js/pull/11567. I had trouble resolving an recurring error while attempting to run the tests on my local machine (Error: Cannot find module 'es6-symbol'), so I'm not sure if it will pass the Travis CI check.

I've closed https://github.com/angular/angular.js/pull/11567 initiated a new PR https://github.com/angular/angular.js/pull/11582 to get the Travis CI tests to run again (sorry if there was a better way to do this).

Last try before trying to dig into what the test results are saying... PR https://github.com/angular/angular.js/pull/11593


Fix emit when type import merges with local value

Fixes #16086

Tests passed on travis, but the github outage preventing travis from updating the PR with its results.


Update sbt-bloop to 1.5.13

About this PR
📦 Updates ch.epfl.scala:sbt-bloop from 1.5.4 to 1.5.13
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "ch.epfl.scala", artifactId = "sbt-bloop" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "ch.epfl.scala", artifactId = "sbt-bloop" }
}]

labels: sbt-plugin-update, early-semver-patch, semver-spec-patch, commit-count:1

@scala-steward rebase


Add summary option.

Hello, Aaron!
I'm not sure if you are accepting PRs, but I would be happy if you were able to include this one. The issue it addresses for my team is that sometimes we just want a quick one-line summary to warn us of circular dependencies and other times we want the full output. So I added a "summary" option to do this, and updated unit tests and readme.md correspondingly.
In any case, I would like to thank you for your open source generosity with circular-dependency-plugin. It's useful and simply written.

I've published version 4 that allows a consumer to provide an onDetected option. This will allow you to accomplish the behavior you want and customize the output. I'm going to go ahead and close this.


Manually adding regened common type

All SDK Contribution checklist:
This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] Please open PR in Draft mode if it is:

Work in progress or not intended to be merged.
Encountering multiple pipeline failures and working on fixes.

[ ] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[ ] I have read the contribution guidelines.
[ ] The pull request does not introduce breaking changes.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[ ] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code. (Track 2 only)
[ ] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK. Please double check nuget.org current release version.

Additional management plane SDK specific contribution checklist:
Note: Only applies to Microsoft.Azure.Management.[RP] or Azure.ResourceManager.[RP]

[ ] Include updated management metadata.
[ ] Update AzureRP.props to add/remove version info to maintain up to date API versions.

Management plane SDK Troubleshooting

If this is very first SDK for a services and you are adding new service folders directly under /SDK, please add new service label and/or contact assigned reviewer.

If the check fails at the Verify Code Generation step, please ensure:

Do not modify any code in generated folders.
Do not selectively include/remove generated files in the PR.
Do use generate.ps1/cmd to generate this PR instead of calling autorest directly.
Please pay attention to the @microsoft.csharp version output after running generate.ps1. If it is lower than current released version (2.3.82), please run it again as it should pull down the latest version.

Note: We have recently updated the PSH module called by generate.ps1 to emit additional data. This would help reduce/eliminate the Code Verification check error. Please run following command:
  `dotnet msbuild eng/mgmt.proj /t:Util /p:UtilityName=InstallPsModules`

Old outstanding PR cleanup
Please note:
If PRs (including draft) has been out for more than 60 days and there are no responses from our query or followups, they will be closed to maintain a concise list for our reviewers.

There are still about ~100s test case failures. Mostly due to the order of the property serialization has been changed to Tags, then Location. This causes many test verification failures. Will fix in follow-up commit.
Here are the manual tweaks I have done on top of the generated.

Added abstract, protected and virtual
Removed inherited properties (TrackedResource -> no id, name, type from Resource base class)
Change the constructor ordering TrackedResource(self declared and inherited)
TrackedResource, Added empty ctor
TrackedResource ctor Added Tag if null new dictionary on line 52
SystemData.CreatedByType enum. x-namespace override unsuccessful.
Removed model from Sku as it is not part of v1/2/3 common type

ErrorResponse is renamed to ErrorDetail (and the new ErrorResponse is actually CloudError), you need to replace ErrorResponse with ErrorDetail in ResourceGroupExportResult.cs and ResourceGroupExportResult.Serialization.cs.


Custom character controller doesn't work when changing scenes

With a custom character controller the player doesn't move when in the map but only moves when in the lobby
`using RiptideNetworking;
using UnityEngine;
[RequireComponent(typeof(Rigidbody))]
public class PlayerMovement : MonoBehaviour
{
[SerializeField] private Player player;
[SerializeField] private Transform camProxy;
private bool didTeleport;
private bool[] inputs;

//Assingables
public Transform orientation;

//Other
private Rigidbody rb;

//Rotation and look
private float xRotation;
private float sensitivity = 50f;
private float sensMultiplier = 1f;

//Movement
public float moveSpeed = 4500;
public float maxSpeed = 20;
public bool grounded;
public LayerMask whatIsGround;

public float counterMovement = 0.175f;
private float threshold = 0.01f;
public float maxSlopeAngle = 35f;

//Crouch & Slide
private Vector3 crouchScale = new Vector3(1, 0.5f, 1);
private Vector3 playerScale;
public float slideForce = 400;
public float slideCounterMovement = 0.2f;

//Jumping
private bool readyToJump = true;
private float jumpCooldown = 0.25f;
public float jumpForce = 550f;

//Input
bool jumping, sprinting, crouching;

//Sliding
private Vector3 normalVector = Vector3.up;
private Vector3 wallNormalVector;

void Awake() {
    rb = GetComponent<Rigidbody>();
    Debug.Log(rb);
}

private void Start()
{
    inputs = new bool[6];
}

private void FixedUpdate()
{
    if (!player.IsAlive)
        return;

    Vector2 inputDirection = Vector2.zero;
    if (inputs[0])
        inputDirection.y += 1;

    if (inputs[1])
        inputDirection.y -= 1;

    if (inputs[2])
        inputDirection.x -= 1;

    if (inputs[3])
        inputDirection.x += 1;

    jumping = inputs[4];
    crouching = inputs[5];
    transform.rotation = Quaternion.Euler(0, transform.rotation.eulerAngles.y, 0);

    Move(inputDirection, inputs[4], inputs[5]);
}

private void StartCrouch() {
    transform.localScale = crouchScale;
    transform.position = new Vector3(transform.position.x, transform.position.y - 0.5f, transform.position.z);
    if (rb.velocity.magnitude > 0.5f) {
        if (grounded) {
            rb.AddForce(orientation.transform.forward * slideForce);
        }
    }
}

private void StopCrouch() {
    transform.localScale = playerScale;
    transform.position = new Vector3(transform.position.x, transform.position.y + 0.5f, transform.position.z);
}

private void Move(Vector2 inputDirection, bool jumping, bool crouching)
{
    //Extra gravity
    rb.AddForce(Vector3.down * Time.deltaTime * 10);

    //Find actual velocity relative to where player is looking
    Vector2 mag = FindVelRelativeToLook();
    float xMag = mag.x, yMag = mag.y;

    //Counteract sliding and sloppy movement
    CounterMovement(inputDirection.x, inputDirection.y, mag);

    //If holding jump && ready to jump, then jump
    if (readyToJump && jumping) Jump();

    //Set max speed
    float maxSpeed = this.maxSpeed;

    //If sliding down a ramp, add force down so player stays grounded and also builds speed
    if (crouching && grounded && readyToJump) {
        rb.AddForce(Vector3.down * Time.deltaTime * 3000);
        return;
    }

    //If speed is larger than maxspeed, cancel out the input so you don't go over max speed
    if (inputDirection.x > 0 && xMag > maxSpeed) inputDirection.x = 0;
    if (inputDirection.x < 0 && xMag < -maxSpeed) inputDirection.x = 0;
    if (inputDirection.y > 0 && yMag > maxSpeed) inputDirection.y = 0;
    if (inputDirection.y < 0 && yMag < -maxSpeed) inputDirection.y = 0;

    //Some multipliers
    float multiplier = 1f, multiplierV = 1f;

    // Movement in air
    if (!grounded) {
        multiplier = 0.5f;
        multiplierV = 0.5f;
    }

    // Movement while sliding
    if (grounded && crouching) multiplierV = 0f;

    //Apply forces to move player
    rb.AddForce(orientation.transform.forward * inputDirection.y * moveSpeed * Time.deltaTime * multiplier * multiplierV);
    rb.AddForce(orientation.transform.right * inputDirection.x * moveSpeed * Time.deltaTime * multiplier);
    SendMovement();
}

private void Jump() {
    if (grounded && readyToJump) {
        readyToJump = false;

        //Add jump forces
        rb.AddForce(Vector2.up * jumpForce * 1.5f);
        rb.AddForce(normalVector * jumpForce * 0.5f);

        //If jumping while falling, reset y velocity.
        Vector3 vel = rb.velocity;
        if (rb.velocity.y < 0.5f)
            rb.velocity = new Vector3(vel.x, 0, vel.z);
        else if (rb.velocity.y > 0)
            rb.velocity = new Vector3(vel.x, vel.y / 2, vel.z);

        Invoke(nameof(ResetJump), jumpCooldown);
    }
}

private void ResetJump() {
    readyToJump = true;
}

private void CounterMovement(float x, float y, Vector2 mag) {
    if (!grounded || jumping) return;

    //Slow down sliding
    if (crouching) {
        rb.AddForce(moveSpeed * Time.deltaTime * -rb.velocity.normalized * slideCounterMovement);
        return;
    }

    //Counter movement
    if (Mathf.Abs(mag.x) > threshold && Mathf.Abs(x) < 0.05f || (mag.x < -threshold && x > 0) || (mag.x > threshold && x < 0)) {
        rb.AddForce(moveSpeed * orientation.transform.right * Time.deltaTime * -mag.x * counterMovement);
    }
    if (Mathf.Abs(mag.y) > threshold && Mathf.Abs(y) < 0.05f || (mag.y < -threshold && y > 0) || (mag.y > threshold && y < 0)) {
        rb.AddForce(moveSpeed * orientation.transform.forward * Time.deltaTime * -mag.y * counterMovement);
    }

    //Limit diagonal running. This will also cause a full stop if sliding fast and un-crouching, so not optimal.
    if (Mathf.Sqrt((Mathf.Pow(rb.velocity.x, 2) + Mathf.Pow(rb.velocity.z, 2))) > maxSpeed) {
        float fallspeed = rb.velocity.y;
        Vector3 n = rb.velocity.normalized * maxSpeed;
        rb.velocity = new Vector3(n.x, fallspeed, n.z);
    }
}

/// <summary>
/// Find the velocity relative to where the player is looking
/// Useful for vectors calculations regarding movement and limiting movement
/// </summary>
/// <returns></returns>
public Vector2 FindVelRelativeToLook() {
    float lookAngle = orientation.transform.eulerAngles.y;
    float moveAngle = Mathf.Atan2(rb.velocity.x, rb.velocity.z) * Mathf.Rad2Deg;

    float u = Mathf.DeltaAngle(lookAngle, moveAngle);
    float v = 90 - u;

    float magnitue = rb.velocity.magnitude;
    float yMag = magnitue * Mathf.Cos(u * Mathf.Deg2Rad);
    float xMag = magnitue * Mathf.Cos(v * Mathf.Deg2Rad);

    return new Vector2(xMag, yMag);
}

private bool IsFloor(Vector3 v) {
    float angle = Vector3.Angle(Vector3.up, v);
    return angle < maxSlopeAngle;
}

private bool cancellingGrounded;

/// <summary>
/// Handle ground detection
/// </summary>
private void OnCollisionStay(Collision other) {
    //Make sure we are only checking for walkable layers
    int layer = other.gameObject.layer;
    if (whatIsGround != (whatIsGround | (1 << layer))) return;

    //Iterate through every collision in a physics update
    for (int i = 0; i < other.contactCount; i++) {
        Vector3 normal = other.contacts[i].normal;
        //FLOOR
        if (IsFloor(normal)) {
            grounded = true;
            cancellingGrounded = false;
            normalVector = normal;
            CancelInvoke(nameof(StopGrounded));
        }
    }

    //Invoke ground/wall cancel, since we can't check normals with CollisionExit
    float delay = 3f;
    if (!cancellingGrounded) {
        cancellingGrounded = true;
        Invoke(nameof(StopGrounded), Time.deltaTime * delay);
    }
}

private void StopGrounded() {
    grounded = false;
}

public void Enabled(bool value)
{
    rb.isKinematic = value;
}

public void SetInput(bool[] inputs, Vector3 forward)
{
    this.inputs = inputs;
    transform.forward = forward;
    // Debug.Log(xy[0]);
    // Debug.Log(xy[1]);
}

private void SendMovement()
{
    if (NetworkManager.Singleton.CurrentTick % 2 != 0)
        return;

    Message message = Message.Create(MessageSendMode.unreliable, ServerToClientId.playerMovement);
    message.AddUShort(player.Id);
    message.AddUShort(NetworkManager.Singleton.CurrentTick);
    message.AddBool(didTeleport);
    message.AddVector3(transform.position);
    message.AddVector3(camProxy.forward);
    NetworkManager.Singleton.Server.SendToAll(message);

    didTeleport = false;
}

public void Teleport(Vector3 toPosition)
{
    bool isEnabled = rb.isKinematic;
    rb.isKinematic = false;
    transform.position = toPosition;
    rb.isKinematic = isEnabled;

    didTeleport = true;
}

}
`

The GitHub issues section of this repository is meant for problems that occur because of the actual code in the repository. Since your issues are due to modifications you've made, you should ask on the Discord server instead.