Several IDE Warnings from: nextcord.utils.get(client.voice_clients, guild=ctx.guild)

Summary
nextcord.utils.get is of type VoiceClient, but IDEs see it as VoiceProtocol
Reproduction Steps
vc_connection = await ctx.author.voice.channel.connect()
print(vc_connection.is_playing())
If we get the VoiceClient like this, the type printed is VoiceClient and IDE shows no warnings
vc_connection = nextcord.utils.get(client.voice_clients, guild=ctx.guild)
print(vc_connection.is_playing())
If we get the VoiceClient like this, the type printed is VoiceClient and the code still works, but the IDE shows warnings because it sees it as VoiceProtocol which doesn't have the same methods as VoiceClient.
Minimal Reproducible Code
async def check_voice(ctx):
    if ctx.author.voice:
        vc = ctx.author.voice.channel
    else:
        return await ctx.reply("You must be in a voice channel to use music commands")

    # Connect to requester's VC, unless already connected
    try:
        vc_connection = await vc.connect()
        print(vc_connection.is_playing())
    except nextcord.errors.ClientException:
        vc_connection = nextcord.utils.get(client.voice_clients, guild=ctx.guild)
        print(vc_connection.is_playing())

Expected Results
IDE should nextcord.utils.get(client.voice_clients, guild=ctx.guild) as type VoiceClient
Actual Results
IDE sees nextcord.utils.get(client.voice_clients, guild=ctx.guild) as type VoiceProtocol rather than VoiceClient
Intents
nextcord.Intents.all()
System Information

Python v3.10.5-final
nextcord v2.0.0-final
aiohttp v3.8.1
system info: Windows 10 10.0.22000

All libraries should be up to date (checked with pip list --outdated)
Checklist

[X] I have searched the open issues for duplicates.
[X] I have shown the entire traceback, if possible.
[X] I have removed my token from display, if visible.

Additional Context
I actually just migrated to nextcord from the original discord repo. The original discord repo did not have these IDE warnings. I would also like to note (in case relevent) that the python version was recently upgraded from 3.9 to 3.10, and a whole bunch of libraries were upgraded to their latest versions.

Not a bug since client.voice_clients is a list of VoiceProtocol. These are usually VoiceClient instances, but they don't have to be.
This is expected when dealing with type checkers that sometimes methods will return a type that is a superclass of the type they really are.
If you know you are only dealing with VoiceClients, possible solutions include:
assert isinstance(vc_connection, VoiceClient)

Or
print(vc_connection.is_playing())  # type: ignore

That's pretty cool!  I didn't know you could comment out warnings like that.
Is there a particular reason why IDEs only complains about it in nextcord, but not the original discord.py?

I doubt that's the case since the docs show it's the same.
It's possible you changed your type checking settings between migrating.

I just checked my code again, replacing nextcord with discord and the warnings did go away. If I replace discord with nextcord in my code, then the warnings do appear. Are you getting different results when switching between the two libraries? What IDE are you using?
I also thought it was weird because like you said, the docs show pretty much the same stuff.

I have a virtual environment with discord.py 2.0 and testing out the code you sent gives the same results for me.

Hmm. That's really interesting. I think I'll just blame it on PyCharm or other factors being wack and call it a day then, since we cannot seem to replicate the issue between the two libraries. Thanks for confirming the results with me!


JMS adapter Configuration

while calling the adapter file in worklight i'm getting below error.
[ERROR   ] FWLSE0005W: JMS connection exception received: jms/libertyQCF. Closing the connection. [project Adapters]
[ERROR   ] FWLSE0035E: Invocation of procedure 'writeMessage' failed. [project Adapters]
Failed to create connection from factory: jms/libertyQCF
this error i'm getting in console.but i configured in server.xml

<featureManager>
    <feature>servlet-3.0</feature>
    <feature>jndi-1.0</feature>
    <feature>jdbc-4.0</feature>
    <feature>localConnector-1.0</feature>
    <feature>restConnector-1.0</feature>
    <feature>jsp-2.2</feature>
    <feature>appSecurity-1.0</feature>
    <feature>ssl-1.0</feature>
</featureManager>
<webContainer invokeFlushAfterService="false"/>
<webContainer com.ibm.ws.webcontainer.suppressLoggingServiceRuntimeExcep="true"/>
<webContainer deferServletLoad="false"/>
<httpEndpoint host="*" httpPort="10080" httpsPort="10443" id="defaultHttpEndpoint">
    <tcpOptions soReuseAddr="true"/>
</httpEndpoint>

GitHub is not meant for code support. Open a question on StackOverflow.


Add MethodGenerator::copyMethodSignature without body and phpdoc

Adds a Zend\Code\Generator\MethodGenerator::copyMethodSignature method that is similar to fromReflection() but without the copying the method body and phpdoc as this is quite heavy to parse (it requires file IO, regexes, tokenization of code).
It's for example useful when creating proxies where you normally change the method body afterwards anyway. See https://github.com/Ocramius/ProxyManager/pull/393 where it increases performance.

@Ocramius can you merge this and make a release so I can use it in Ocramius/ProxyManager#393  ?

@Tobion I'll try to do releases over the weekend.


Simulator background color shows black when choosing color '#dedede'

Describe the bug:
Simulator background color shows black when choosing color '#dedede'
Steps to reproduce the behavior:
1.Navigate to https://arcade.makecode.com/beta.
2.Create a project and put 'set background color to' blocks under 'on start'.
3.Choose the first color named '#dedede' and observe simulator.
Actual behavior:
Simulator background color shows black not as selected.
Expected behavior:
Simulator background color shows grey when choose '#dedede'.
Screen Recording:

Additional context:
1.OS: Windows 11,  iMac OS: Ventura 13.6.1
2.arcade version: 1.13.50
3.Microsoft MakeCode version: 9.3.6

the grey you are choosing (#dedede) symbolises transparent, not a color


Failed compiles generate messy output.

Performing some basic sanity checks before we get started...
Looking good, I think I'm set to go!

To begin, let's check your tarball's file name and path...
Excellent!
Done! Let me just double check your design doc...

---------- snip snip ----------
Shoot, I might have found a mistake in your project: Your design doc is missing or named incorrectly
Maybe this tip will help: It should be named "abfoxive_design_p3.pdf"
Every shiny dream that fades and dies, generates the steam for two more tries!
---------- snip snip ----------
Since this is Project 3, I'll now perform Project 3-specific checks...
Now I'll test your Makefile...

---------- snip snip ----------
Shoot, there's potentially a mistake in your project we can't ignore: Your Makefile didn't produce a.out
Maybe this tip will help: Please ensure there are no errors above, and that you haven't used GCC's -o option
Happiness can be found even in the darkest of times, if one only remembers to turn on the the light.
trie.cpp:95:18: error: no member named 'get_children' in 'Node'; did you mean 'has_children'?
    temp = temp->get_children()[idx];
                 ^~~~~~~~~~~~
                 has_children
./node.h:22:8: note: 'has_children' declared here
  bool has_children();
       ^
fatal error: too many errors emitted, stopping now [-ferror-limit=]
20 errors generated.
make: *** [Makefile:4: all] Error 1

The autograder output interrupts the existing compile process.

Odd, this shouldn't be happening. This is the relevant snippet of code:
    make_subprocess = subprocess.Popen(["make", "-j"], cwd=testing_path)
    make_subprocess.wait()
    if not os.path.exists(testing_path + "/a.out"):
        unrecoverable_project_mistake("Your Makefile didn't produce a.out", "Please ensure there are no errors above, and that you haven't used GCC's -o option")

We should be waiting before checking if a.out exists. I wonder what is going wrong here.

Hmm, it's possible that it's terminal related. I'll try running it in Alacritty and Terminal.app to see whether that's the issue.

Weirdly I'm unable to reproduce the bug. I'll close the issue for now and reopen it if I can find it again.

Sounds good


"Uncaught TypeError: Cannot read property '0' of undefined" when switching to package-provided grammar

I'm getting this error every time I switch to the grammar "JavaScript (Semantic Highlighting)" provided by https://github.com/p-e-w/language-javascript-semantic in Atom 0.206.0 on Linux with --include-deprecated-apis (also, the "Copy error report to clipboard" button does nothing, but that's likely an unrelated issue). In a previous version of Atom (not sure which one exactly), the grammar worked without problems.
This problem is currently preventing me from fixing the deprecations in the language-javascript-semantic package.
/usr/share/atom/resources/app.asar/src/tokenized-line.js:89

TypeError: Cannot read property '0' of undefined
    at TokenizedLine.module.exports.TokenizedLine.transformContent (/usr/share/atom/resources/app.asar/src/tokenized-line.js:89:38)
    at new TokenizedLine (/usr/share/atom/resources/app.asar/src/tokenized-line.js:71:12)
    at TokenizedBuffer.module.exports.TokenizedBuffer.buildTokenizedLineForRowWithText (/usr/share/atom/resources/app.asar/src/tokenized-buffer.js:521:14)
    at TokenizedBuffer.module.exports.TokenizedBuffer.buildTokenizedLineForRow (/usr/share/atom/resources/app.asar/src/tokenized-buffer.js:506:19)
    at TokenizedBuffer.module.exports.TokenizedBuffer.tokenizeNextChunk (/usr/share/atom/resources/app.asar/src/tokenized-buffer.js:286:43)
    at /usr/share/atom/resources/app.asar/src/tokenized-buffer.js:263:26
    at /usr/share/atom/resources/app.asar/node_modules/underscore-plus/node_modules/underscore/underscore.js:666:47

/cc @nathansobo

This package uses private APIs that have changed to reduce memory consumption, similar to the package discussed in #6982. The fix will have to occur in the package. Sorry about that.

I see... looks like https://github.com/atom/atom/pull/6757 slammed the door pretty hard on code-defined grammars. I notice that the other two affected packages have not been able to adapt to the new convention – one of them switched to a cson-based grammar and the other remains broken :frowning:
A public API for writing scripted grammars would be a big plus. There are too many things that just cannot be done with TextMate-style grammars.

Actually, it would suffice if a stable createToken was available. I posted this pull request a year ago in preparation for code-defined grammars, but without a supported method for creating tokens it does not really make sense anymore.

Yeah, the TextMate grammars are really limited and I'd like a better–and supported–interface. The unfortunate reality is that we're dealing with trees here, not tokens, and repeating the scopes on every token is like describing a tree over and over again for every path from root to leaf. So I'd like any interface we come up with to acknowledge that reality.
This change shouldn't be the death of these packages by any means. It should be pretty easy to use atom.grammars.startIdForScope and endIdForScope to register start/end scopes. If you season string run lengths in between as non-negative integers you'll match the format.
I would be interested in your thoughts on a supported interface. There's sort of 2 levels we could do. One would be just a drop-in replacement for the TextMate grammar that is expected to parse a single line and where we thread a state object through. Another is much more general, where we inform the parser of change events and allow it to manage all its state itself. Either way, I think the interface would involve this nested word style open/close scope interface.

IMO it would be best if the tokenizeLine API returned to what it was and the translation from the classical "value/scopes" representation to the cryptic but efficient integer array was performed outside of tokenizeLine. That way, grammars can tag line tokens in an intuitive way and yet the memory usage remains low (whatever memory is allocated inside individual tokenizeLine calls should be minimal and will be garbage collected soon enough. The size of source files comes from their number of lines, not their line length.)

I managed to update the format and my package is working again. Thank you for your help!

You're very welcome. I'm open to your idea of a more convenient interface, but now isn't a great time as I'm focused on things for 1.0. Expanding the API is always a careful process, and we haven't even contemplated custom grammar extensions at all. If you could extract a library from your package that others could use to make the conversion, that would be pretty nice. No pressure though: :smile:


LCFS - Calculate and display available compliance unit balance for the compliance period (Line 17 of the compliance report summary)

Description:
Implement functionality to calculate and display the available compliance unit balance for the compliance period. This involves pulling data from previous compliance reports to account for positive compliance units carried forward, subtracting any future debits recorded in the system, and ensuring the final balance is not negative.
Wireframe: Link
Purpose and benefit to user:
To provide users with an accurate and clear view of their available compliance unit balance, ensuring they can monitor and manage their compliance obligations effectively. This helps avoid penalties by ensuring users are aware of their unit balance status.
Acceptance Criteria:

[ ] Given I am a BCeID user, When I view the compliance report, then I see the available compliance unit balance displayed in Line 17.
[ ]  Given I am a BCeID user, When the system pulls data from previous compliance reports, then it accurately adds positive compliance units carried forward year by year and subtracts any future debits recorded in the system.
[ ]  Given I am a BCeID user, When there is no previous compliance report available, then the system calculates the balance by taking the current positive compliance units reported minus the current negative compliance units reported and displays the remaining balance.
[ ]  Given I am a BCeID user, When I view Line 17, then the balance displayed is never negative. If the calculation results in a negative balance, the system will instead display zero and trigger an automatic penalty for the current period.

Development Checklist:

[ ]  Implement logic to pull data from previous compliance reports to account for positive compliance units carried forward and subtract any future debits recorded in the system.
[ ]  Develop a fallback mechanism to calculate the balance using current compliance unit data if no previous report is available.
[ ]  Ensure the balance displayed in Line 17 is never negative. If the balance is negative, display zero and trigger an automatic penalty.
[ ]  Test data retrieval, calculation logic, and display to ensure accuracy and proper functionality.

Notes:

Confirm the data retrieval process from previous compliance reports for accurate calculation.
Ensure the system can handle cases with no previous compliance reports by correctly calculating the balance using current data.

Tested and this is currently still reflecting the balance of the total balance, rather than the available balance for this specific year (I was testing 2024). The available balance should only reflect what is available for this year (2024 for this example).
This includes:

validations or compliance unit balance changes from assessments listed with a 2024 or prior compliance period
-reductions listed with a 2024 or prior compliance period
+compliance units purchased through credit transfers with and effective date on or before March 31st, 2025
-compliance units sold through credit transfer with an effective date on or before March 31st, 2025
+compliance unites issued under IA/P3A with an effective date on or before March 31st, 2025
+/- admin adjustments with an effective date on or before March 31st, 2025
-all future debits (such as transfers or reductions)

Hi @prv-proton. Thanks for reaching out about this. I can give you an example of this that we recently worked on to better illustrate this calculation. You know the query in Metabase that you helped me with that shows the year by year of units remaining? This available balance calculation adds an additional step to that calculation. I'm going to share the example with you directly through Teams, but wanted to document the comment here on the card for future use.

Tested and this is currently still reflecting the balance of the total balance, rather than the available balance for this specific year (I was testing 2024). The available balance should only reflect what is available for this year (2024 for this example).
This includes:
* validations or compliance unit balance changes from assessments listed with a 2024 or prior compliance period
  -reductions listed with a 2024 or prior compliance period
  +compliance units purchased through credit transfers with and effective date on or before March 31st, 2025
  -compliance units sold through credit transfer with an effective date on or before March 31st, 2025
  +compliance unites issued under IA/P3A with an effective date on or before March 31st, 2025
  +/- admin adjustments with an effective date on or before March 31st, 2025
  -all future debits (such as transfers or reductions)

Hi @Grulin , for admin adjustments and initiative agreements, should we only consider the approved transactions?

Difficult to test as a front end user because we are limited by how far ahead or behind we can report. Will collaborate with the dev team to figure out how to test if this is working the way it is intended to.


Redirect the catboost_info subfolder created by CatboostEstimator

Feedback from sebhrusen (from the automlbenchmark)
CatboostEstimator is creating and filling a catboost_info subfolder in the running directory. We should be able to pass a 'train_dir' param to Catboost to avoid that.
For example at AutoML level, accept a tmpdir and pass it to each algo supporting an equivalent property (or pass a dedicated subfolder, for example tmpdir/catboost for Catboost and so on).
Reference:
https://github.com/openml/automlbenchmark/pull/270

or find the directory of log_file_name and pass that to catboost

@sonichi I would like to contribute to this awesome framework. As this is my first open-source contribution. Could you provide me some guidelines?

@bnriiitb Thanks for considering to contribute. Please read https://github.com/microsoft/FLAML#contributing to get started. We can chat on gitter if you have questions.


Add use_v1 to world_locations_news search links

The v2 search api isn't finding any documents for queries like
https://www.gov.uk/search/all?order=updated-newest&world_locations[]=uk-delegation-to-council-of-europe
This is presumably because it doesn't have the same logic for indexing world locations as v1 did. We should probably fix that and remove the use_v1 flag, but for the short term this is a quick fix for the immediate issue.
This was raised by FCDO on Zendesk[1]

[1] https://govuk.zendesk.com/agent/tickets/5664229

Better to put this logic in Finder Frontend, rather than exposing a debugging parameter in a link. We'll pick it up there.


pip3 not found as internal or external command

Describe the bug
pip3 gives an error that it cannot be found
To Reproduce
Steps to reproduce the behavior:
Run pip3 install -r requirements.txt
Expected behavior
Successful installation.
Actual behaviour
error as stated.
Anyone experiencing this, please post as much info as possible (as guided by make bug template)

This is actually a common issue with python 3.x.
I'm working on a fix that will work for all (if encountered) but need info from those who have encountered this.

I found this while trying to run python from the Git Bash window.
To make sure you aren't doing this, have the Git Bash window and Cmd Window side by side to make sure you are doing it in the correct window. In the Cmd Window run pip3 install -r requirements.txt.
Please post results.

old post. closing.


[Net] Add WebSocketServer handshake_timeout property.

Allows customization of the maximum time a client is allowed to stay in the the "pending" state (i.e. awaiting HTTP handshake).
This used to be 1 second by before, the new default is 3 seconds.
See #45681

Thanks!

Cherry-picked for 3.4.


Keep backup of deleted or updated objects

Also, optionally notify cluster admin if critical components are deleted.
https://github.com/appscode/go-notify

-- backup-dir={PV-mounted-dir}
/{backup-dir}/apis/extensions/v1beta1/namespaces/default/deployments/aphlictserver/{uid}.yaml

We can also take full backups periodically


Cargo check has false negatives in a workspace with internal dependencies

Problem
When using a workspace that has multiple crates that depend on each other, enabling of feature flags prevents running of cargo check --no-default-features on all crates.
Steps
Example workspace:

crates

A [features: Foo]
B [depends on: A with feature Foo enabled]

Running
cargo c --no-default-features

inside {workspace-dir}/crates/A will cause errors when anything that should be hidden behind the Foo feature flag is referenced
cargo c --no-default-features --all

inside {workspace-dir} does not cause any errors.
Possible Solution(s)
I'd potentially expect two version of A to be checked:

One that complies with the cargo check --no-default-features command
One that suffices the dependency for B (Foo enabled)

Notes
No response
Version
cargo 1.70.0 (ec8a8a0ca 2023-04-25)
release: 1.70.0
commit-hash: ec8a8a0cabb0e0cadef58902470f6c7ee7868bdc
commit-date: 2023-04-25
host: x86_64-pc-windows-msvc
libgit2: 1.6.3 (sys:0.17.0 vendored)
libcurl: 8.0.1-DEV (sys:0.4.61+curl-8.0.1 vendored ssl:Schannel)
os: Windows 10.0.22621 (Windows 10 Home) [64-bit]

We generally refer to this as "feature unification" and this is expected.  When only do one build combination within a run.  I believe #4463  is the most relevant existing issue for this.
One workaround is to use tools like cargo-hack to do multiple cargo check runs.
Going to go ahead and close this but if there is something I missed, let us know and we can re-open it!

For more on feature unification, see https://doc.rust-lang.org/cargo/reference/features.html#feature-unification


class based routing doesn't inherit templated parameters

On an API that looks something like this -
@hug.object.urls('/objects/{model}/', requires=auth_verified)
class CRUD:

    @hug.object.patch('/{guid}')
    def set_by_guid(self, model, guid, **kwargs):
        # Update an existing object

    @hug.object.get()
    def get_all_of_type(self, model: hug.types.text,...)
        # Get existing objects

Examples doc generation are definitely broken using this approach - they don't build from inherited path but instead, each sub-route is based directly off root, without versioning, or the class's base route.

The routing itself is 'broken', in that this kind of templating does not provide the parameters as expected.

A solution is to route like this -
@hug.object.urls('/objects/{model}', requires=auth_verified)
@hug.object.urls('/objects/{model}/{guid}', requires=auth_verified)
class CRUD:

    @hug.object.patch()
    def set_by_guid(self, model, guid, **kwargs):
        # Update an existing object

    @hug.object.get()
    def get_all_of_type(self, model: hug.types.text,...)
        # Get existing objects

I find this a bit strange, because it means exposing the same parameters to every child method. Additonally, overloading the class with urls means you can't overload contained methods e.g. you can't define both of these methods:
@hug.object.get(examples=['/fruit/apple'])
    def get_by_guid(self, model, guid):
        # Get an existing object

@hug.object.get(examples=['/fruit'])
    def get_all_of_type(self, model: hug.types.text,...)
        # Get existing objects

Instead, I would expect this to behave like a continued extension of the rest of the routing system, just that the class defines a common parent route for the methods. Ideally I'd like the framework to be a flexible and helpful as possible - this feels a bit constrained/opinionated? If I'm wide of the mark or it's intentionally opinionated I would appreciate an explanation - I could then write some docs for it.
Overall I get the impression that it's actually templated parameterised routes that aren't fully developed. For sure they aren't fully documented :)
Very happy to help out. I would need to dive much deeper into the Router code to figure out what needed doing for this though.

Thanks for alerting me to this issue! I agree with you that the existing implementation leaves something to be desired. In order to allow full flexibility, which you are correct is indeed a goal, I've implemented support for relative sub routes, as demonstrated here:
https://github.com/timothycrosley/hug/pull/494/files#diff-52d976e719062f2d5d9af708e53ba65eR45
These allow you to add sub routes based off the main route, simply by not prefixing the url with a / for the child routes, while still being backward compatible with the old behavior. This is merged into develop and will make its way into the next release.
Thanks!
~Timothy


libgd-dev can not be installed

Description
libgd-dev can not be installed due to version mismatch
Area for Triage:
Question, Bug, or Feature?:
Bug
Virtual environments affected

[ ] Ubuntu 16.04
[ ] Ubuntu 18.04
[x] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11.0
[ ] Windows Server 2016 R2
[ ] Windows Server 2019

Expected behavior
I expected libgd-dev to be installed.
Actual behavior
Error message: The following packages have unmet dependencies:
libgd-dev : Depends: libgd3 (= 2.2.5-5.2ubuntu2) but 2.3.0-2+ubuntu20.04.1+deb.sury.org+1 is to be installed
Repro steps
Run these in workflow:
sudo apt-get update -q
sudo apt-get install -yq libgd-dev

Example run: https://github.com/taginfo/taginfo-tools/runs/1459251139?check_suite_focus=true

Hi, @joto !
These packages are installed by php 7.4, you can use this workaround:
sudo apt-get remove libgd3
sudo apt-get install -yq libgd-dev

Thank you!

Thanks for the quick reply. The workaround works for me.

Hi, @joto!
I'll close the issue for now, but if the problem persist, please feel free to reopen the ticket.
Thank you!

The suggested workaround stopped working within the last 24 hours:
apt-get remove libgd3
Reading package lists...
Building dependency tree...
Reading state information...
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:

The following packages have unmet dependencies:
 nginx : Depends: nginx-core (< 1.18.0-0ubuntu1.1~) but it is not going to be installed or
                  nginx-full (< 1.18.0-0ubuntu1.1~) but it is not going to be installed or
                  nginx-light (< 1.18.0-0ubuntu1.1~) but it is not going to be installed or
                  nginx-extras (< 1.18.0-0ubuntu1.1~) but it is not going to be installed
         Depends: nginx-core (>= 1.18.0-0ubuntu1) but it is not going to be installed or
                  nginx-full (>= 1.18.0-0ubuntu1) but it is not going to be installed or
                  nginx-light (>= 1.18.0-0ubuntu1) but it is not going to be installed or
                  nginx-extras (>= 1.18.0-0ubuntu1) but it is not going to be installed
E: Error, pkgProblemResolver::Resolve generated breaks, this may be caused by held packages.

Example run: https://github.com/vergoh/vnstat/runs/1834018377?check_suite_focus=true

hi @vergoh! Since nginx package depends on libgd3 it needs to be removed as well.
sudo apt-get remove nginx
sudo apt-get remove libgd3
sudo apt-get install -yq libgd-dev

This is happening again as of yesterday, with the GHA config not having changed at all (https://github.com/PDLPorters/pdl/runs/7473608493?check_suite_focus=true):
  The following packages have unmet dependencies:
   libgd-dev : Depends: libgd3 (= 2.2.5-5.2ubuntu2.1) but 2.3.3-5+ubuntu20.04.1+deb.sury.org+1 is to be installed
  E: Unable to correct problems, you have held broken packages.


Rate Limiter Service

Fixes https://github.com/dotnet/AspNetCore.Docs.Samples/issues/14/
@Rick-Anderson please review. I found that naming HomeController as Home2Controller is a bit weird. I renamed it back to HomeController. Let me know if the name Home2 was intentional or not. The project was giving me some errors when I started working on it. I tried to fix the issues. Please run the app with different rate limiter configurations. If I miss any configuration. let me know. Thanks 😸

HomeController as Home2Controller is a bit weird.

I did that because when using Razor Pages and Home controller you have two defaults for the home page.
Thanks for doing this, I'll check out the code.

@maartenba @wtgodbe all the rate limiters stopped working. I'm testing with localhost and hitting refresh quickly. The timestamp changes but I never hit the limit.
using Microsoft.AspNetCore.RateLimiting;
using System.Threading.RateLimiting;

var builder = WebApplication.CreateBuilder(args);

builder.Services.AddRateLimiter(_ => _
    .AddFixedWindowLimiter(policyName: "fixed", options =>
    {
        options.PermitLimit = 4;
        options.Window = TimeSpan.FromSeconds(120);
        options.QueueProcessingOrder = QueueProcessingOrder.OldestFirst;
        options.QueueLimit = 2;
    }));

var app = builder.Build();

static string GetTicks() => (DateTime.Now.Ticks & 0x11111).ToString("00000");

app.MapGet("/", () => Results.Ok($"FixedWindowLimiter {GetTicks()}"))
                           .RequireRateLimiting("fixed");

app.Run();

Haven't tested yet, but looks from that snippet a app.UseRateLimiter(); is missing.

@maartenba @Rick-Anderson, I guess it should have both AddRateLimiter and UseRateLimiter? Although, I'm a bit confused about the configurations. In AddRateLimiter, an Action<RateLimiterOptions> can be configured. Then again in UseRateLimiter(), an instance of RateLimiterOptions can also be set. However, it throws an exception when running the app,

InvalidOperationException: This endpoint requires a rate limiting policy with name fixed, but no such policy exists.

Please take a look at the following configuration for the scenario I'm talking about
using Microsoft.AspNetCore.RateLimiting;
using System.Threading.RateLimiting;

var builder = WebApplication.CreateBuilder(args);

builder.Services.AddRateLimiter(_ =>
{
    _.AddFixedWindowLimiter(policyName: "fixed", options =>
        {
            options.PermitLimit = 4;
            options.Window = TimeSpan.FromSeconds(12);
            options.QueueProcessingOrder = QueueProcessingOrder.OldestFirst;
            options.QueueLimit = 2;
        });

    _.RejectionStatusCode = StatusCodes.Status429TooManyRequests;
});

var app = builder.Build();

app.UseRateLimiter(new RateLimiterOptions
{
    RejectionStatusCode = StatusCodes.Status429TooManyRequests
});

static string GetTicks() => (DateTime.Now.Ticks & 0x11111).ToString("00000");

app.MapGet("/", () => Results.Ok($"Hello {GetTicks()}"))
                           .RequireRateLimiting("fixed");

app.Run();

Any reason behind the overload of UseRateLimiter that also takes the RateLimiterOptions?

First one is registered as options in the service container and lets you add multiple policies etc.
Latter one is "fixed" and initialized immediately.

@maartenba got it. thanks 😺
@Rick-Anderson it would be great if you can come up with a scenario where a rate limiter needs to be initialized immediately using app.UseRateLimiter(new RateLimiterOptions()). As of right now, every sample has a simple call to `app.UseRateLimiter()``

So it comes down to this: you always need the app.UseRateLimiter() middleware in the pipeline without any parameters along with the ServiceCollection extension. And if you want to initialize the rate limiter immediately without any specific policy or other configurations, no need to call the  app.AddRateLimiter(Action<RateLimiterOptions>); simply go with just the app.UseRateLimiter(new RateLimiterOptions())! Correct me if I'm wrong @wtgodbe

So it comes down to this: you always need the app.UseRateLimiter() middleware in the pipeline without any parameters along with the ServiceCollection extension. And if you want to initialize the rate limiter immediately without any specific policy or other configurations, no need to call the app.AddRateLimiter(Action); simply go with just the app.UseRateLimiter(new RateLimiterOptions())! Correct me if I'm wrong @wtgodbe

Yep, you got it!


can’t establish a connection to the server at wss://meet.jit.si/xmpp-websocket?room=my-secret-conference.

i just cloned and tried your example app.
but on click of the Connect button i get  error in the dev console:
GET wss://meet.jit.si/xmpp-websocket?room=my-secret-conference  [HTTP/1.1 403 Forbidden 304ms]
developer.mozilla.org on 403 error code:

The HTTP 403 Forbidden client error status response code indicates that the server understood the request but refuses to authorize it.
This status is similar to 401, but in this case, re-authenticating will make no difference.
The access is permanently forbidden and tied to the application logic, such as insufficient rights to a resource.

i found this topic in the jitsi community forum - but as fare as i understand it only state that the tokenised version is only available in the jitsi as a service version...
does this code still work for you with the public service?

ok seems that this low-level usage of meet.jit.si is not allowed.
maybe that change some time in the past.
i got an answer in the jitsi forum:

damencho
On meet.jit.si you should be using the iframe API there as per the terms and conditions of that service.
You may want to try https://jaas.8x8.vc though. We can probably help you achieve the same thing there.

i will leave this open for some days and then close it.

Good to know. Didn't know that. Thanks for the info!


K-allagbe/issue3-integrate-azure-cognitive-search

Closes #3 : Integrate azure cognitive search

Very neat! I'll come and pick your brain for the implementation of the Blueprint on Nachet-Backend.

Sure. I copied myself from https://github.com/ai-cfia/louis-backend

Looks great, thanks!!


Caddyfile parsing fails when import is used as a "short" CLI Caddyfile

1. What version of Caddy are you using (caddy -version)?
Caddy 0.10.10 (non-commercial use only)
2. What are you trying to do?
Run Caddy without a main Caddyfile by importing all matching *.caddy files from a specific folder (a la Apache sites-enabled).
3. What is your entire Caddyfile?
test.caddy file in $PWD:
localhost:2015
browse

4.  Please paste any relevant HTTP request(s) here.
N/A

6. What did you expect to see?
Activating privacy features... done.
http://localhost:2015

7. What did you see instead (give full error messages and/or log)?
whitestrake at apollo in ~/Projects
→ caddy -log stdout "import $PWD/*.caddy"
2017/10/11 15:44:32 /Users/whitestrake/Projects/test.caddy:1 - Error during parsing: Unknown directive 'localhost:2015'

8. How can someone who is starting from scratch reproduce the bug as minimally as possible?
mkdir test && cd test
echo "localhost:2015\nbrowse" > test.caddy
caddy -log stdout "import $PWD/*.caddy"

According to the documentation at https://caddyserver.com/docs/cli, the "short" Caddyfile uses default host and port. Therefore, it can't contain other hosts inside of it.
Looking at the code https://github.com/mholt/caddy/blob/master/caddyhttp/httpserver/plugin.go#L249,
this is what you actually have (by default):
localhost:2015
arguments, joined by newline

Ah, good point -- I hadn't noticed before, for some reason, @francislavoie that you were trying to define a host and port in the Caddyfile as well. You can change the host and port with the -host and -port flags though, and then just put directives in the Caddyfile. Otherwise import itself should work, I think?

Are we open to changing this behaviour?
Currently I'm running Caddy with -conf /path/to/Caddyfile, the entirety of which is import /path/to/*.caddy. I'd be nice to remove the necessity for that Caddyfile altogether, if possible.

Well the point of the "short" Caddyfile is to be... short... and simple. Are you really looking for -conf /path/to/*.caddy to work instead?

Y'know, in school I was taught not to listen to what a client thinks they want, rather, figure out what they actually want. Ironic...
Yep, that's pretty much it exactly.
We can definitely remove bug from this issue at this point, I think, given that it's working as expected right now. At this stage it's a feature request (for globbing on the -conf CLI flag).


clear all extended function

could you provide a clear all function

Good idea, can do :)

Thanks very much @shentao

I really need this function

I need it too. Thank you.

Adding a special slot clear, that makes it possible to create a clear-all button. The slot receives the search query as props so it can be used also for creating search button (coupled with :internal-search="false.
Will be available with the next release.

Is there a good simple example of how this works?  I'm having trouble understanding how it's working in the docs' Asynchronous Select html code.

how i can use this  ?

i cant use this ! its hidden


add find more episodes feature for playlists

[ ] find the top channels for a playlists and get the latest unwatched/unadded episodes
[ ]

dup #303


3.8.0 - Crash on loading woods raid

General information:
Server crashed while starting a Woods raid, game was stuck on Starting Local Game.
Appears to be related to an event quest, The TerraGroup Trail - Part 15, which was removed after the event concluded. Looks to be the second flash drive attempting to spawn, but it doesn't exist.
https://escapefromtarkov.fandom.com/wiki/Sliderkey_Secure_Flash_drive_2

Attachments

server-2024-01-06.log (1102 KiB)
2024.01.06_19-06-24_<IP_ADDRESS>.28375 traces.log (68 KiB)
LogOutput.log (11 KiB)

Originally written by Jester_Stacks

This is due to me generating woods using dumps from when the event was active. I'll look into removing dumps from that time period.

Originally written by chomp

Should be fixed with: https://dev.sp-tarkov.com/SPT-AKI/Server/commit/9f99c8d2eb571d08a4a7488edd2f189649233546

Originally written by chomp


start signup tweaks

closes #1625

[x] Better titles
[x] No pre-scrolled forms
[x] Feedback form fixes
[ ] Community guidelines

Backend checklist

[ ] Formatted my code by running autoflake -r -i --remove-all-unused-imports src && isort . && black . in app/backend
[ ] Added tests for any new code or added a regression test if fixing a bug
[ ] All tests pass
[ ] Run the backend locally and it works
[ ] Added migrations if there are any database changes, rebased onto develop if necessary for linear migration history

Frontend checklist

[ ] Formatted my code with yarn format && yarn lint --fix
[ ] There are no warnings from yarn lint
[ ] There are no console warnings when running the app
[ ] Added any new components to storybook
[ ] Added tests where relevant
[ ] All tests pass
[ ] Clicked around my changes running locally and it works
[ ] Checked Desktop, Mobile and Tablet screen sizes

This can be reviewed except for Jail-related files

Build failed - there's some import error. Can this be reviewed through the preview site (assuming the new sign up backend changes has been merged)?

Added --runInBand to prevent multiple tests running at once. This is bad but I don't even know which test is causing failures

Still failing :/


Use CMAKE_CURRENT_SOURCE_DIR instead of CMAKE_SOURCE_DIR.

Hi Chaoren,
Any chance you could send this patch to the "develop" branch?  "master" is provided as a convenience for end users as it packages all dependencies as submodules.  We only merge from "develop" to "master", never from "master" to "develop".
Cheers,
Stewart

I'll submit it through gerrit instead.


JS debugger showing wrong file when there are files with the same name

VSCode Version: 1.11.1
OS Version: macOS El Capitan (10.11)
JS environment: React, Mocha, Enzyme

Steps to Reproduce:

Create a module (e.g. src/myfile.js).
Create a test file with the same name in a different directory (e.g. test/myfile.js).
Set a breakpoint in src/myfile.js.
Run the test.
When the breakpoint hits in src/myfile.js, try and go back in the callstack to test/myfile.js. It will instead show src/myfile.js with the debugger stopped on some random line.

And as a sanity check, rename one of the files and see everything work correctly.
PS I love VSCode, thank you guys for the best editor ever <3

Never mind for now. Those repro steps don't work. I think it could be an issue in more complex environments when debugging and you have to click Load More Stack Frames, but I'm trying to create a simple set of repro steps to prove that. Sorry!


throw hadoop comptible exception

Change our logic to throw exceptions by following below guide,
https://hadoop.apache.org/docs/current/hadoop-project-dist/hadoop-common/filesystem/index.html

@minming, you can take this task after you completed your test.


Window resizing/zooming if not native

📝 Provide a description of the new feature
What is the expected behavior of the proposed feature?  What is the scenario this would be used?
I would like to be able to resize windows even when the 'maximize' isn't available and there are no resizing options to drag. I feel like this is a perfect thing to include in powertoys because it's not something needed natively in windows ten but has that niche use case that'll increase productivity. We have an issue where some older applications behave this way. Additionally in my personal world there are some games that behave this way, and it would be nice to resize those as well.

If you'd like to see this feature implemented, add a 👍 reaction to this post.

Can you give more detail here.  This is still unclear on the use cases and current blocking scenarios.
Most of the time the application is intentional for not wanting to be resized past certain restrictions.


Version of Mamba and Causal-conv1d

Hello! I'm looking forward to replicating your work and I've cost a lot of time in preparing the environment.
There are some errors like TypeError: Mamba.__init__() got an unexpected keyword argument 'scan_type' and 'causal_conv1d_fwd(): incompatible function arguments'. I tried different solutions but it doesn't work, seeming cycling in these two errors.
What I really want to know is that what the version of Mamba and Causal-conv1d you use in this work?
Thank you for your attention!

Mamba: 1.0.1
Causal-conv1d: 1.0.0


Add docker-autotest for PR testing

Our primary supported platforms are all RH-like distros.  Maintaining work-a-like with Travis/Ubuntu is becoming a major headache for otherwise trivial reasons.  Having docker-autotest PRs tested here will allow us to focus more on the code and less on the CI testing environment.

 (2) [CI/CN] Fix selinux: permissive in ADEPT and Docker Autotest


golangci-lint misspell isn't warning on all typos

Problem
https://goreportcard.com/report/github.com/celestiaorg/celestia-app#misspell reports that there are in this repo even though we have misspell enabled
https://github.com/celestiaorg/celestia-app/blob/9451ef45c4dbba6f1917651a4e6b9d72137e98a1/.golangci.yml#L9
It looks like golangci-lint's misspell isn't flagging us on typos that misspell has caught. To confirm, I installed misspell and ran it locally:
$ misspell .
docs/architecture/adr-004-qgb-relayer-security.md:88:218: "custumization" is a misspelling of "customization"
docs/architecture/adr-004-qgb-relayer-security.md:394:33: "occured" is a misspelling of "occurred"
proto/qgb/genesis.proto:17:39: "persistant" is a misspelling of "persistent"
third_party/proto/tendermint/crypto/proof.proto:30:49: "nessecary" is a misspelling of "necessary"
third_party/proto/confio/proofs.proto:48:57: "collission" is a misspelling of "collisions"
x/payment/spec/docs.md:146:57: "contians" is a misspelling of "contains"
x/qgb/types/genesis.pb.go:71:39: "persistant" is a misspelling of "persistent"

Proposal

[x] https://github.com/celestiaorg/celestia-app/pull/932
[ ] Determine why golangci-lint misspell isn't catching these

hmm no more issues identified at https://goreportcard.com/report/github.com/celestiaorg/celestia-app#misspell so maybe it's working now? Closing until we find other issues.


 That should take care of the fields issue.

Yes, that fixes the fields print outs.
The invalid parameters error is still present though.

I am unable to reproduce this, both with OpenCL and HIP backends

The version of MIOpen in the logs is 2.3.0

I ran it again after a fresh build/install with: 9216cc08e20c56c257862446904ee0c53fa12172
removed the .config/miopen direction
made sure miopen.db in rocm directory was latest
winograd_miopenFreshInstall_noCfgDir.txt

I'm not sure what the issue is yet, but It seems there is a way to build MIOpen such that this problem doesn't arise. I'll report again when I find out what the source is.

The incorrect parameter error seems to be a result of clang not installing correctly, or being in the wrong directory.
Following the errors to the external interface, we get to src/ocl/gcn_asm_utils.cpp. The failure is at ValidateGcnAssemblerImpl, where the path returned by GetGcnAssemblerPath is tested invalid. It should point to a clang version supporting amdgcn, but in my case it was pointing to an empty directory. This string is normally populated by cmake, but can also be controlled by an environment variable MIOPEN_EXPERIMENTAL_GCN_ASM_PATH (perhaps a sign that this has been a problem). Having re-installed the rocm opencl package and run cmake with a clean build directory multiple times, I suspect that this process is volatile on a system that isn’t sparkly clean.
short:
Winograd wanted to use assembly kernels, but MIOpen didn't find the compiler.


New moral cutscenes

Description
Created new moral screens as cutscenes and deleted old moral scenes.
Fixes / Closes #426
Type of change

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

When the game loads, press the Cutscene Temp button to see a moral cutscene.
To change which cutscene is shown, in GdxGame line 129, change to MORAL_x

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

Commits already covered by PR #598


thanks for support bidirectional layout

hi guys,i am very happy for this framework ,because this framework support bidirectional layout,
many people need to for right-to-left layout,i finally found this framework ,then it helped me too,thanks

Were happy to help!


ARC72 Support

Hi,
Love the work that has gone into Algoverify. A major boon for NFT collections.
It would be great to get support added for even more asset types!
There is currently a bounty live on Voi for adding ARC72 support into existing bots here.
With Algoverify being open sourced anyone could tackle this so long as it gets merged.

Hi,
I don't have any plan to add support for non-asset NFTs for now. But if you want to contribute to the bot, we look forward to your contributions. Or you can share some docs about ARC72 and would be great for devs.
Thank you :)

Thanks! I may take a poke at it, if anyone else is interested the details can be found here:
Spec:
https://arc.algorand.foundation/ARCs/arc-0072
Implementation Tools:
https://github.com/temptemp3/ulujs/blob/main/impl/arc72/README.md
Example:
https://gist.github.com/temptemp3/f0c576170262f524022ee5c3bf8f3118


[Question] Django Framework

Hi, I tried the dashboard but it didnt work nicely. Seems like theres missing static folder where it should've been there. I wonder if it could be fixed or readded? Since without it, it looks like literal text in html, lol.

@RafaelHGOliveira

We might need to remove the source from the repo since both of us aren't knowledgeable to maintain it, and contributors aren't technically responsible to do so.

Yep


dummy issue

What would you like to be added:
Why is this needed:

asdfasd

hello

hello

hello

hi

jjjj

jjjj


some small syntax thing

Signed-off-by: Mathias Fechner<EMAIL_ADDRESS>Syntax foo sorry

@itrich sorry there was a little syntax foo


[Spark]Throw deletionVectorSizeMismatch for non-incremental DV updates.

Which Delta project/connector is this regarding?

[X] Spark
[ ] Standalone
[ ] Flink
[ ] Kernel
[ ] Other (fill in here)

Description
Update the sanity check for non-incremental updates to throw a deletionVectorSizeMismatch instead of using require().
How was this patch tested?
Update existing test.
Does this PR introduce any user-facing changes?
Changes an exception from IllegalArgumentException to DeltaChecksumException.

jenkins merge


Augment LD_RUN_PATH at runtime on *nix.

In https://github.com/twadleigh/jlcall/issues/2#issuecomment-146190337 @bermanmaxim says:

Correction: apparently it is the setenv('LD_RUN_PATH', '/home/maxim/julia/lib/julia'); command that really helped, not LD_LIBRARY_PATH... And I have to set it at each new instantiation of Matlab for it to work.
I added this LD_RUN_PATH definition to the initialization of the Jl class, and it conveniently works.

On windows, the analogous thing is achieved by adding julia_bin_dir to the system PATH. (Windows uses the same path for executables and shared libraries.) We need to provide the right analogous handling for POSIX systems.

Another solution is to link the shared library with -rpath during compilation
-Wl,-rpath=$JULIA_DIR/lib/julia
Tested to work on Ubuntu.

Yes, that's a better solution. Thanks.

I've been having issues the dynamic libraries not being loaded on OS X. So far I haven't found a great solution as -rpath isn't working with clang but I found a work-around by explicitly setting the dynamic library paths. Note that ... should be replaced with an absolute path:
otool -L jlcall.mexmaci64
install_name_tool -change @rpath/libjulia.dylib .../lib/julia/libjulia.dylib jlcall.mexmaci64
install_name_tool -change @rpath/libmx.dylib .../lib/julia/libmx.dylib jlcall.mexmaci64
install_name_tool -change @rpath/libmex.dylib .../lib/julia/libmex.dylib jlcall.mexmaci64
install_name_tool -change @rpath/libmat.dylib .../libmat.dylib jlcall.mexmaci64
otool -L jlcall.mexmaci64

I'll keep looking for a good solution in OS X.

Hi @bermanmaxim,

You said:

> Another solution is to link the shared library with -rpath during compilation
> -Wl,-rpath=$JULIA_DIR/lib/julia
> Tested to work on Ubuntu.

By adding -rpath to LDFLAGS, it builds on my PC. However, calling double_it.m encountered the following errors:

double_it
Error using jlcall
Unhandled Julia exception: ErrorException

Error in Jl.eval_string (line 29)
Jl.mex(0, expr);
Error in Jl.include (line 33)
Jl.eval_string(['include("' Jl.forward_slashify(fn) '")']);
Error in double_it (line 4)
Jl.include('double_it.jl');

I use Matlab 2015b under Ubuntu 15.10 64bit. A log for the `mex` command:

mex  -largeArrayDims -O -output jlcall -outdir '/home/plzh/Dropbox/work/jlcall/m' -I'/usr/include/julia' -L'/usr/lib/x86_64-linux-gnu/julia' '/home/plzh/Dropbox/work/jlcall/m/../src/jlcall.cpp' -ljulia -v LDFLAGS='\$LDFLAGS -Wl,-rpath=/usr/lib/x86_64-linux-gnu/julia'

-> mexopts.sh sourced from directory (DIR = $PREF_DIR)
   FILE = /home/plzh/.matlab/R2015b/mexopts.sh
----------------------------------------------------------------
->    MATLAB                = /usr/local/MATLAB/R2015b
->    CC                    = gcc-4.7
->    CC flags:
         CFLAGS             = -ansi -D_GNU_SOURCE  -fexceptions -fPIC -fno-omit-frame-pointer -pthread
         CDEBUGFLAGS        = -g
         COPTIMFLAGS        = -O -DNDEBUG
         CLIBS              = -Wl,-rpath-link,/usr/local/MATLAB/R2015b/bin/glnxa64 -L/usr/local/MATLAB/R2015b/bin/glnxa64 -lmx -lmex -lmat -lm -lstdc++
         arguments          =
->    CXX                   = g++-4.7
->    CXX flags:
         CXXFLAGS           = -ansi -D_GNU_SOURCE -fPIC -fno-omit-frame-pointer -pthread
         CXXDEBUGFLAGS      = -g
         CXXOPTIMFLAGS      = -O -DNDEBUG
         CXXLIBS            = -Wl,-rpath-link,/usr/local/MATLAB/R2015b/bin/glnxa64 -L/usr/local/MATLAB/R2015b/bin/glnxa64 -lmx -lmex -lmat -lm
         arguments          =
->    FC                    = gfortran
->    FC flags:
         FFLAGS             = -fexceptions -fbackslash -fPIC -fno-omit-frame-pointer
         FDEBUGFLAGS        = -g
         FOPTIMFLAGS        = -O
         FLIBS              = -Wl,-rpath-link,/usr/local/MATLAB/R2015b/bin/glnxa64 -L/usr/local/MATLAB/R2015b/bin/glnxa64 -lmx -lmex -lmat -lm
         arguments          =
->    LD                    = g++-4.7
->    Link flags:
         LDFLAGS            = -pthread -shared -Wl,--version-script,/usr/local/MATLAB/R2015b/extern/lib/glnxa64/mexFunction.map -Wl,--no-undefined -Wl,-rpath=/usr/lib/x86_64-linux-gnu/julia
         LDDEBUGFLAGS       = -g
         LDOPTIMFLAGS       = -O
         LDEXTENSION        = .mexa64
         arguments          =  -L/usr/lib/x86_64-linux-gnu/julia -ljulia
->    LDCXX                 =
->    Link flags:
         LDCXXFLAGS         =
         LDCXXDEBUGFLAGS    =
         LDCXXOPTIMFLAGS    =
         LDCXXEXTENSION     =
         arguments          =  -L/usr/lib/x86_64-linux-gnu/julia -ljulia
----------------------------------------------------------------

-> g++-4.7 -c  -I/usr/include/julia -I/usr/local/MATLAB/R2015b/extern/include -I/usr/local/MATLAB/R2015b/simulink/include -DMATLAB_MEX_FILE -ansi -D_GNU_SOURCE -fPIC -fno-omit-frame-pointer -pthread  -O -DNDEBUG  "/home/plzh/Dropbox/work/jlcall/m/../src/jlcall.cpp" -o /home/plzh/Dropbox/work/jlcall/m/jlcall.o

In file included from /home/plzh/Dropbox/work/jlcall/m/../src/jlcall.cpp:2:0:
/usr/include/julia/julia.h:1521:5: warning: ‘mexPrintf’ is an unrecognized format function type [-Wformat]
/usr/include/julia/julia.h:1523:5: warning: ‘mexPrintf’ is an unrecognized format function type [-Wformat]
/usr/include/julia/julia.h:1525:5: warning: ‘mexPrintf’ is an unrecognized format function type [-Wformat]
-> g++-4.7 -O -pthread -shared -Wl,--version-script,/usr/local/MATLAB/R2015b/extern/lib/glnxa64/mexFunction.map -Wl,--no-undefined -Wl,-rpath=/usr/lib/x86_64-linux-gnu/julia -o  "/home/plzh/Dropbox/work/jlcall/m/jlcall.mexa64"  /home/plzh/Dropbox/work/jlcall/m/jlcall.o  -L/usr/lib/x86_64-linux-gnu/julia -ljulia -Wl,-rpath-link,/usr/local/MATLAB/R2015b/bin/glnxa64 -L/usr/local/MATLAB/R2015b/bin/glnxa64 -lmx -lmex -lmat -lm

LD_LIBRARY_PATH is ignored by MATLAB. Other attempts to circumvent this issue have been implemented. Please open a new issue if you continue to experience issues after updating.


Visual delete error

Visual delete error
E5108: Error executing lua: ...l/share/nvim/lazy/lspmark.nvim/lua/lspmark/bookmarks.lua:499: bad argument #1 to 'sub' (string expected, got nil)
stack traceback:
	[C]: in function 'sub'
	...l/share/nvim/lazy/lspmark.nvim/lua/lspmark/bookmarks.lua:499: in function 'get_visual_selection'
	...l/share/nvim/lazy/lspmark.nvim/lua/lspmark/bookmarks.lua:549: in function 'delete_visual_selection'

Additionally, upon restarting Neovim, the bookmarks do not load.

Thanks for reporting the bug, could you provide more hints on how to reproduce the error log? it works fine on my side.

Additionally, upon restarting Neovim, the bookmarks do not load.

The bookmarks are organized project-wise so they will only be loaded when changing to the cwd that has bookmarks associated with it.

Select visual
Press d
:messages : error text

Select visual
Press d
:messages : error text

Is this 100% reproducible?

-- bookmarks.lua file
	-- get all lines of text
	local lines = vim.api.nvim_buf_get_lines(0, start_line - 1, end_line, false)
	vim.notify(vim.inspect(lines))

result = {}

Maybe you bind the key in normal mode to this function such as ["n|d"] rather than in visual mode ["v|d"], could you have a check?

vim.keymap.set("v", "d", function()
    require("lspmark.bookmarks").delete_visual_selection()
end)

Could you show me your Lua config? I still cannot reproduce it on my side.


JS error after closing editor instance

Informations
Browser + version: Chrome last version
OS: Ubuntu
Resolution: Maybe, add exception from undefined data in function.
Text of error:
Uncaught TypeError: Cannot read property 'toLowerCase' of undefined
at trumbowyg.js:1514
at Function.each (jquery.min.js:2)
at Trumbowyg.updateButtonPaneStatus (trumbowyg.js:1513)
at trumbowyg.js:595
(anonymous) @ trumbowyg.js:1514
each @ jquery.min.js:2
updateButtonPaneStatus @ trumbowyg.js:1513
(anonymous) @ trumbowyg.js:595
How to reproduce the bug?
Click on the close button of editor

This bug should be fixed in 2.8.0 which will be released soone.


gateway: get SSL cert signed by StartSSL

It's already running on the gateways, and graded A by SSL Labs: https://www.ssllabs.com/ssltest/analyze.html?d=gateway.ipfs.io&latest
Closes #6

enable OCSP stapling
bring in the StartSSL cert
reload nginx for cert/key/dhparam changes

License: MIT
Signed-off-by: Lars Gierth<EMAIL_ADDRESS>

@lgierth this is fantastic, solid work!! :+1:


stream.delete_all_rules() is broken.

Found the issue to be in this line of delete_all_rules():
    def delete_all_rules(self) -> json:
        """Deletes all your rules automatically"""
        try:
--->            rules = self.get_rules().json()
            ids = list(map(lambda rule: rule["id"], rules["data"]))
            return self.delete_rule({"delete": {"ids": ids}})
        except Exception as e:
            pass

No need for .json() as self.get_rules() already returns it as such.

Thanks for pointing this out! If you create a pull, create I'll happily test and merge it.
The module needs updating -- some of these have been written when v2 was in its early stages. We are closer to releasing this CLI Tool. I hopefully will announce soon.

Sure. I'll make PR with this change as well as the one I discussed/highlighted earlier about not being able to add filters to FilteredStream as well (FilteredStream was missing _has_params = True.


Improved About Us Section, Removed Unnecessary Comments

Is your feature request related to a problem? Please describe.
The about section didn't follow a visual hierarchy, which I have improved with some added CSS effects.
Describe the solution you'd like.
In the about us section, the heading is in the centre while all other headings in the whole web page aren't in the centre, hence the about us heading was out of the format being followed by the whole web page.
Moreover, the italic paragraph makes it hard for any user to read properly, hence making that italic to normal. To add more to it, the line height is very less, which makes the paragraph look clumsy, hence increasing the line height
The services section is having a low contrast on its heading like doubt clearing while the description is having a high contrast, hence fixing that issue.
Please assign me this issue under GSSOC.
Describe alternatives you've considered.
No other alternatives were considered.
Add any other context or screenshots about the feature request here.
Before:

After:

@agamjotsingh18 @arora-jasjot, would anyone of you like to assign me this issue?

Sure @PriyanshuSaxena2612

Hi @agamjotsingh18.
If this task is being delayed, then I'd be glad to work on this issue.
I'd be grateful if you could assign this task to me under GSSoC'22.


Under row selection column, the select all checkbox in the header shows incorrect status while using react table v8

Describe the bug
Row selection works well with data that resides locally.
But, let's say the paginated data is being fetched from the server, the limit is set to 10 per page and you have selected 7 rows from the 1st page. Now, go to page 2 and select 3 rows from the second page once the data is fetched. The Select All checkbox is unticked on page two whereas it should show indeterminate.
I believe this happens because row selection is only aware of 10 items per page. Although it has a memory of selected rows, it's not aware of out of how many items those rows were selected.
I see it used to work as expected in react-table v7 - https://codesandbox.io/s/react-table-server-side-pagination-row-select-5gfrp
Your minimal, reproducible example
https://codesandbox.io/s/pedantic-austin-uqt9ry?file=/src/table.tsx
Steps to reproduce

Select 7 rows out of 10 available on Page 1
Go to page 2
Data would be fetched from the server for Page 2
Select 3 more rows
The column checkbox in the header is unticked

Expected behavior
The column checkbox in the header is unticked whereas it should be indeterminate as out of 20, 10 rows are selected
How often does this bug happen?
Every time
Screenshots or Videos
No response
Platform
Any browser
react-table version
v8
TypeScript version
No response
Additional context
No response
Terms & Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I understand that if my bug cannot be reliable reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

Hi there!
I've noticed a related bug, well I think is the same bug really but put in other way:

click on select all header checkbox, in page one
move to any other page, the intermediate state stops working

If this needs another issue please let know, I believe is the same case.
Thanks!


Fix PostgreSQL data type incompatibility issue in Notification query

The issue was caused by a data type incompatibility between postgres and the given query. Laravel defined the 'data' column as 'text' type.  It should be noted that there may be reasons why the 'data' column is defined as 'text' type, but regardless, this problem can be circumvented by using the 'like' operator to search for a specific string within the 'data' column. This allows for the retrieval of records that correspond to the 'format' key with the value 'filament', regardless of the data type defined in the 'data' column."
To resolve the issue, this PR updates the query to use appropriate syntax for each database driver. For PostgreSQL, the query now uses '%"format":"filament"%' using the 'like' operator. . For all other drivers, the query continues to searches for the key 'format' using the '->' operator
This fix ensures that notifications are queried correctly regardless of the underlying database driver, allowing for consistent behavior across different database setups.

Thank You!  Hoping this gets approved soon.

To be honest, I don't like the use of like to get a JSON property. We have this note in the docs about just using a json() column, does this not work?

What is the best way to extend this functionality without modifying the package code ?

Extend Notifications, override getDatabaseNotificationsQuery() and use that as the LW component instead.
But I would urge you to not store plain text in that column. JSON is the only thing that will work.


add hami to Kubernetes device plugin examples

Description
add hami to Kubernetes device plugin examples

Issue

Closes: #

Hello @wawa0210,
Thanks for your contribution. It appears that this PR includes changes to files across multiple languages English (en) and Chinese (zh-cn). However, docs follows different processes for each localization, and we typically don't accept pull requests that impact multiple languages simultaneously. I would recommend modify only in primary language(English) initially with this PR. After the merge of the English content, you can create separate pull request for the remaining language(s).

Explicit hold until this PR only covers one language.
(Ok to unhold once that's done)
/hold

Explicit hold until this PR only covers one language. (Ok to unhold once that's done) /hold

Thanks for the reminder, I have changed to a one language

/remove-language zh
/unhold

/remove-area localization

Thanks your review, already update.


FALCON-1776 Process instance update in titan DB based on JMS notifications on coordinator actions

In JMS notifications from the coordinator action, it is not guaranteed to provide information on timestamp and user workflow version. Need to allow these properties to be optional when adding process instance to titan DB. Also need to address the different format when there is no input/output feed. See the picture below for an example result (i.e. WAITING instance).

+1 , jenkins test failures seem unrelated.

👍

+1. Looks good.


developer guide updated

updating user cases, glossary, user story and NFR
#20 #21 #32 #31

@cxo05 @Bombbird2001 @Sasmik23 @PetrichorPrecipice Guys, can check if there is anything yall wanna add

LGTM


How to change quality for an image using the base64 encode (or sharp) ?

What is your question?
How to change quality for an image using the base64 encode ?
Hello,
Could you please help me with a JSON example on how to add the quality parameter for an image using the new base 64 encoding?
I've tried
{   edits: {     quality: 50,   }   bucket: 'test',   key: 'test.jpg', }
and
{   edits: {     resize: {       quality: 50,     }   }   bucket: 'test',   key: 'test.jpg', }

@rapgodnpm You did not include an error message that you might be getting in the backend Lambda CloudWatch logs but you may have a json syntax error in what you provided.
The following is an example that might work for you.
const imgReq = JSON.stringify({
    bucket: 'test',
    key: 'test.jpg',
    edits: {
      quality: 50
    }
  });

  const encStr = Buffer.from(imgReq).toString('base64');
  const imageUrl = `https://api.cloudfront.net/${encStr}`;
  console.log(imageUrl);

Including the resize would look like the following
    bucket: 'bucket',
    key: 'test.jpg',
    edits: {
      quality: 50,
      resize: {
        width: 200,
        hight: 100
      }
    }
  });

  const encStr = Buffer.from(imgReq).toString('base64');
  const imageUrl = `https://api.cloudfront.net/${encStr}`;
  console.log(imageUrl);

Hello @dougtoppin ,
Thank you for your quick reply. I tried the above but it doesn't seem to work. Here is an example of what I am sending:
{
    "edits": {
        "quality": 50,
        "resize": {
            "width": 600,
            "height": 460
        }
    },
    "bucket": "bucket",
    "key": "image.jpg"
}

I compare it with the image with 100 quality or no quality set at all and they have the same size and look the same. I need to mention that I have autowebp enabled.
Thank you in advance for your kind help and support.

@rapgodnpm I am moving this to the Discussions/Q&A as it is more appropriate in that section
Quality changes are associated with the image format and the following is the correct way to specify it in a request.
This was tested using quality values of 100 and 50. The results can be compared using Preview on Mac with Tools->Adjust Color.
The color palette that appears shows a difference between the two along with the returned image file size being smaller for lesser quality values.
const imgReq = JSON.stringify({
    bucket: 'test',
    key: 'test.jpg',
    edits: {
        resize: {
            width: 1000,
            height: 500
        },
        jpeg: {
            quality: 100
        }
    }
});

const encStr = Buffer.from(imgReq).toString('base64');
const imageUrl = `https://api.cloudfront.net/${encStr}`;
console.log(imageUrl);


Add the installation and update steps in the README #63

#63

@mark-at-pieces merge

Should I remove the update part?


Updated doc structure and added contribution guide

This pull requests largely consolidates the documentation available on Read the Docs based on Issue https://github.com/transit-analytics-lab/spur/issues/46.
Before this is changed to a full pull request, the following must be done:

[x] Creation of a "contribution guide" page showing how to set up a development environment for Spur
[x] Update of the structure of the "User Guide" and the developer reference
[x] Create a "quick start" example vignette using Line 4

@peterlai1 take a look.


superset memory is continuing increasing

Running for a long time，the memory usage of superset is continuing increasing
Expected results
the decrease of memory usage
Actual results
memory usage of superset is continuing increasing
Environment
(please complete the following information):

python version: 3.6

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[ ] I have reproduced the issue with at least the latest released version of superset.
[ ] I have checked the issue tracker for the same issue and I haven't found one similar.

Hi @zcfrank1st , If you using Gunicorn as a webserver,  I offer a trick to avoid this problem.
Gunicore provides some options for the restart web workers. you can add these option in your startup script
 --max-requests INT    The maximum number of requests a worker will process before restarting. [0]
--max-requests-jitter INT   The maximum jitter to add to the *max_requests* setting. [0]


Fragment chains

Cc: @cairomassimo @tomhoule

This handles the test case as reported. I suspect there's probably some other way to "hide" items from the "requires" sweep logic. I'll try to make more test cases.

Needs #289 to be green. Will rebase once it is merged.


Facilitate writing new components/managers

Currently when creating a new component manager, there is a lot of boilerplate code to handle the SoA nature of the data (allocating, deleting). It would be very helpful to have some helper code somewhere to automate this process more.

Yes, a small set of macros/functions would help a bit.


⚠ Bofrost.es has degraded performance

In bc3a341, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1763 ms

Resolved: Bofrost.es performance has improved in e81c245 after 7 minutes.


Configuration#4: Step#3: Splitting Single 600K events from final state hadron file into 40 smaller event array files: MATTER

Using jetscape-ml-tensorflow-nn-dataset-builder-single-file-event-splitter.sh

no output sciprt


update to macOS

macOS is the current writing

Can one of the admins verify this patch?

Can one of the admins verify this patch?


403 Forbiden no UI

I cannot reach the ui with my localip:5800 and container is runnig ok?
handrbake_inspect.txt

You seem to have created a different network for the container.  Try to use the default "bridge" network mode.

This time I did not use docker compose but same result
handrbake_inspect#4.txt

On a new OMV installation with Portainer data outside of docker volume everything is working normally no more perms issue

I think this should not longer be an issue with the latest image.
Closing this issue.  Please re-open if needed.


fix(build): update jest version to avoid vulnerability

PR summary
Updates jest version to avoid alert for node-notifier dependency
PR Checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[x] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

:tada: This PR is included in version 0.17.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 0.17.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


The page is not working and the renders are not loading on the profile

Describe the bug
The server is down perhaps as nothing is being returned from the link and the page is also not working.
To Reproduce
Open the official website for the application or try to see your own GitHub quotes card
Screenshots
If applicable, add screenshots to help explain your problem.

Issue has been resolved..

Thanks for raising the concern!

It happened again, please fix

@version0chiro : We have migrated this successfully to custom cloud, please refer to: <IP_ADDRESS>:3004


Fix chat scroll and word-wrap when typing a message

Chat scrolling broke because it wasn't scrolling all the way to the bottom because scrollHeight was calculated before the new message was added to the chat window. After long enough messages, this made the code think that the user had intentionally scrolled higher and therefore wouldn't try to scroll to the bottom. Rendering on every message does have some cost since we lose the asynchronicity polymer dom-repeat normally gives us. https://www.polymer-project.org/2.0/docs/devguide/templates#synchronous-renders

adding a comment because cla...


FileSystem.ReplaceFile fails if there are FileSystemWatchers applied to the directory

Describe the bug
Under certain circumstances, FileSystem.ReplaceFile can fail:
System.IO.IOException: Unable to remove the file to be replaced.
         at System.IO.FileSystem.ReplaceFile(String sourceFullPath, String destFullPath, String destBackupFullPath, Boolean ignoreMetadataErrors)
         at Tenray.ZoneTree.AbstractFileStream.LocalFileStreamProvider.Replace(String sourceFileName, String destinationFileName, String destinationBackupFileName)
         at Tenray.ZoneTree.AbstractFileStream.DurableFileWriter.WriteAllBytes(String path, Byte[] bytes)
         at Tenray.ZoneTree.Core.ZoneTreeMetaWAL`2.SaveMetaData(ZoneTreeMeta zoneTreeMeta, Int64 mutableSegment, Int64 diskSegment, Int64[] readOnlySegments, Int64[] bottomSegments, Boolean createNew)
         at Tenray.ZoneTree.Core.ZoneTreeLoader`2.LoadZoneTreeMetaWAL()
         at Tenray.ZoneTree.Core.ZoneTreeLoader`2.LoadZoneTree()
         at Tenray.ZoneTree.ZoneTreeFactory`2.OpenOrCreate()

The Internet seems to suggest that this call can fail if there are any FileSystemWatchers applied to the directory, which can happen incidentally due to various programs (VSCode, etc.)
To Reproduce
The Internet suggests this can be replicated by setting up a FileSystemWatcher in another program: https://stackoverflow.com/questions/46824487/unable-to-remove-the-file-to-be-replaced-error-during-file-replace-if-filesys
Expected behavior
ZoneTree should fall back to Delete/MoveFile if ReplaceFile fails.
please complete the following information:

Operating System: Windows
ZoneTree Version: 1.6.0

Hi, @hach-que,
Thank you for reporting this issue.
I understand that you want to use a fallback method using MoveFile / DeleteFile, but please note that this is not the recommended way of solving this issue.
ReplaceFile operation is an atomic operation that is executed at the operation system level.
Changing it with custom MoveFile / DeleteFile operation breaks the atomicity, which can cause unexpected data loss in case of power cut or crashes.
The best solution in your case is to move your data folder out of the file system watchlist region.
Please also make sure that the metafile is not opened by any other process.
In an ideal world, your data folder should not be accessed by other processes.
In production, you can protect your data folder by giving read/write access to a specific user/group and by running the process with privileged user/group.
If you understand the circumstances of not using ReplaceFile method and still want to use a fallback method using MoveFile / DeleteFile, you can create your custom class that implements the IFileStreamProvider interface and pass your custom file stream provider to the ZoneTreeFactory constructor.
I hope this helps. Let me know if you have any other questions.

It should be possible to do a recoverable MoveFile operation with something like. As long as you're consistently calling File.Replace based on the presence of current, it should work.
if (File.Exists(destination))
{
  File.Delete(destination);
}
if (File.Exists(current))
{
  File.Move(current, destination, true);
}

In our case though, I'll just override LocalFileStreamProvider because I know it's safe (we know there isn't another instance of the process running, so the only thing that could cause File.Replace to fail is a filesystem watcher).

If your process terminates in the middle, your destination file will be missing.
Although the current file stays there with a temporary name, the application won't find the destination file on the next run.
It can be recovered automatically by changing the loading logic to lookup a secondary file name.
Let's see if your custom class solves the problem. Then we can reconsider to include it in ZoneTree by addressing the issue above.


Exception caused by data format different between preview & production LUIS API

The json object that Preview API return, does not always contain the key intents.
So this line will throw exception when we use Preview API:
https://github.com/Microsoft/BotBuilder/blob/master/Node/src/dialogs/LuisDialog.ts#L66
Detail:
If I use Production API, the json object return is like
{
  "query": "hi",
  "intents": [
    {
      "intent": "Greeting",
      "score": 0.6110722
    },
    {
      "intent": "None",
      "score": 0.451821864
    },
    {
      "intent": "BizPlan",
      "score": 0.000137667419
    }
  ],
  "entities": []
}

If I use Preview API, it became like this:
{
  "query": "hi",
  "topScoringIntent": {
    "intent": "Greeting",
    "score": 0.6110722
  },
  "entities": []
}

We haven't updated LuisDialog to support the preview API's yet. There are actually a number of issues. It's on our TODO list. But their schema is changing as well so it would be a little premature for us at this point.

i suggest you support two type result.
in my case , i define the class like below
public class EmilyBot
    {
        public string query { get; set; }
        public intents[] intents { get; set; }
        public topScoringIntent topScoringIntent { get; set; }
        public Entity[] entities { get; set; }
    }

We kind of have that. the on('intent') handers only match on the top intent but if you want to do all of the inten processing yourself you can simply use onDefault() which gets passed the raw list of intents. I already let you mix the two approaches as well. If the top intent doesn't have a handler or if things are below a threshold we call onDefault()

Closing this issue/suggestion due to age. Please reopen if needed. Thanks!


duplicate text removed

+1

Coverage remained the same at 99.919% when pulling 7d6fcbdb22ecd7c1a2d6d25f852f7339bc5409ca on fix-dup-text into 573d186d231559c01f07f7efc7f2ed9c47b31725 on master.


JWM crashes Xorg server at first attempt using Move menu item

Here's an interesting one for you...
Using example.jwmrc and the latest git version (d664eba), JWM crashes Xorg to the virtual terminal with the following steps:

Start X and JWM
Click on JWM's root window / desktop to bring up JWM's menu
Choose 'Terminal' (Xterm must be installed)
When terminal window comes up, do NOT move or resize it with the mouse
Right-click the terminal's title bar and choose 'Move'
The small move coordinates pop-up shows briefly then...
Xorg will crash to the terminal

This happens right after Xorg and JWM starts.  I've tried it after using JWM for a while and it works properly, so it's something to do with maybe garbage initial values or something.
I will build with Debug enabled and see if I get any other output beyond Xorg's typical stuff.
Thanks! 😄
ℹ  Gentoo Linux 64-bit * Xorg 1.19.3 * AMD Radeon HD 6670 * 16 GB RAM

Here is the debug output when the crash happens:
JWM: warning: could not load tray icon: "jwm-blue"

Program received signale SIGSEGV, Segmentation fault.
UpdateDesktop (now=0x7fffffffd700) at move.c:847
847        currentClient->state.status |= STAT_HIDDEN;

This should be fixed as of snapshot 1645 (thanks @rdnvndr!)

@rdnvndr @joewing Thank you so much! 😄


1.11.0-rc2: Missing containerd flag  in contrib/completion

Containerd integration add a containerd flag for docker daemon
--containerd                             Path to containerD socket
it missing in  contrib/completion.
Maybe there are some other flag missing for this release, it needs a check.
ping @albers

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Thanks for the notice, I'll look into this.
#dibs


Use contrasting color on hovered links in footer

fixes #57

It looks like this was subsequently fixed by 1fbe788bfbd84398dc9f72256c9a69ab513695f4.


Fix for build failures on docker-hub

Resolves a number of issues to get the build going again on docker-hub.

@petehayes tested this image locally to confirm an install (specifically with UDF enabled) works as expected. Furthermore, this should get a more reproducible build as we are building pljava off of release branch instead of master.

Looks good to me.  Thanks Chris.


Return the user's Discord ID in the Supabase Auth user_metadata

Feature request
Is your feature request related to a problem? Please describe.
Many interactions with the Discord API require the authenticated user's Discord ID. The user's Discord ID is available in the Oauth flow here https://github.com/supabase/gotrue/blob/master/api/provider/discord.go#L26 but it is omitted from the Supabase Auth user_metadata here https://github.com/supabase/gotrue/blob/master/api/provider/discord.go#L90-L93
This is a show stopping issue for Discord specifically, but I imagine it may be useful to have the user's ID on other provider's platforms as well.
Describe the solution you'd like
Include the user's Discord ID from the Oauth flow in the Supabase Auth user_metadata. Currently, when fetching either the Supabase Auth Session or the Supabase Auth User, the user_metadata looks like this:
user_metadata: {
  avatar_url: "some url",
  full_name: "some name"
}

This feature would change it to this:
user_metadata: {
  discord_id: "some id",
  avatar_url: "some url",
  full_name: "some name"
}

Or if it makes more sense to have a generic solution for all providers:
user_metadata: {
  provider_id: "some id",
  avatar_url: "some url",
  full_name: "some name"
}

Describe alternatives you've considered
It's easy enough to Oauth with Discord directly through their /users/@me endpoint, without using the supabase-js client, and therefore have access to the user's Discord ID from the Discord Oauth response. However, this solution does more harm than good because you lose the automatic user management on the Supabase side if you bypass the supabase-js client.

It does seem odd that this issue has never been raised with another provider, but I think the reason may be that most of the familiar auth providers have made their platform APIs more restricted and less useful over time, such that most developers use them exclusively for authentication at this point. Discord is the exact opposite. It has a very broad API that has only expanded in functionality over time, such that most developers primarily use it to build integrations, and therefore use the auth as well in a supporting role. In any case, the user's Discord ID is essential.

Hi @fredguest, thanks for spotting this! I've included it in the PR here and it will be in the upcoming release, so stay tuned! Keep hacking 😎

@kangmingtay thanks for getting this merged. Has this change flowed through to supabase-js yet? I'm still not seeing the 'provider_id' in the user_metadata.
Also, not directly related but another Discord issue, I don't see how we can specify Oauth scopes. Discord has a bunch of different scopes that can be requested during the Oath flow https://discord.com/developers/docs/topics/oauth2#shared-resources-oauth2-scopes and the scopes determine which parts of the Discord API your app is allowed to use, so they are super important. It looks like there is a placeholder for them here https://github.com/supabase/gotrue/blob/master/api/provider/discord.go#L50 but I don't see how I can actually pass any additional scopes through. Am I missing something in the docs or is it just not implemented yet?
The scopes our app specifically needs are guilds and guilds.join and there's no way to get them outside of the Oauth flow. I guess maybe I would expect to be able to specify additional  scopes on the https://app.supabase.io/project/{projectId}/auth/settings page in the same place where you toggle to enable the provider. I realize this might not the best place to discuss the issue, but I wasn't sure which of the various supabase repos to open the issue in, I figured you might have a better idea about that.


chore(apps/whale-api): database module

What this PR does / why we need it:
Adds database module for apps/whale-api as part of larger consolidation effort of whale into jellyfish.
Which issue(s) does this PR address?:

https://github.com/DeFiCh/jellyfish/issues/977

Additional comments?:
Depends on PR:

https://github.com/DeFiCh/jellyfish/pull/1201

This PR will stay in draft mode until parent branch is merged to master.

An ongoing effort to split up the larger apps/whale-api PR into smaller digestible pieces while providing incremental testing:

https://github.com/DeFiCh/jellyfish/pull/1168

No longer applicable, moved to https://github.com/DeFiCh/jellyfish/pull/1248


[FLINK-10907] Fix Flink JobManager metrics from getting stuck after a job recovery.

JobManager loses and regains leadership if it loses connection to ZooKeeper.
When it regains the leadership, it tries to recover the job graph.
During the recovery, it will try to reuse the existing JobManagerMetricGroup
to register new counters and gauges under the same metric name, which causes
the new counters and gauges to be registered incorrectly.
The old counters and gauges will continue to report the stale values and
the new counters and gauges will not report the latest metric.

What is the purpose of the change
Fix Flink JobManager metrics from getting stuck after a job recovery.
Brief change log

Close the existing JobManagerJobMetricGroup associated with the job.
Create a new JobManagerJobMetricGroup in JobManagerMetricGroup.

Verifying this change
This change is already covered by existing tests in JobManagerGroupTest.java
A test is modified to ensure that new JobManagerJobMetricGroup is used.
This change was also tested against a running Flink job by causing the scenario described in the JIRA ticket. Before the fix, we can reliably reproduce the issue. After the fix, JobManager metrics are reported correctly.
Does this pull request potentially affect one of the following parts:

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

Documentation

Does this pull request introduce a new feature? no

Not a problem in 1.5 and above, see the JIRA for more details.


tab indexes for create/edit campaign fields (issue #227)

Set explicit tab indexes for campaign fields. Order is as follows:

Campaign title
Federal: Senate (if enabled)
Federal: House (if enabled)
State: Senate (if enabled)
State: Assembly (if enabled)
State select dropdown (if visible)
Script
Thank you message
Learn more URL
Save as draft
Submit
Cancel

Tab-selecting the state dropdown will bring it into focus (but not expand it). To expand it, you can either click it or hit the spacebar. While the dropdown is focused, you can also begin typing your state name to select it.
By default, Safari will ignore the legislative checkbox tab indexes unless some settings are changed in Preferences -> Advanced -> Press tab to highlight each item on a page

Amazing, thanks Patrick!
I'm in process of setting up local branch so I can't test the change but maybe @kbertash can.


[Modal] - Neutral action button does not close modal

Steps to Reproduce

Open MODAL with neutral action button (button without approve or deny class).
Click the "neutral" button.

Expected
The modal should close.
Result
The modal does nothing and stays on screen.
Testcase
http://jsfiddle.net/mcuru0b8/

You forgot to save the jsfiddle.

Closing the issue until there’s a test case.

@Banandrew You should make a bot to ask people to post a test case if the “Needs Test Case” label is added, then close the issue with a comment 2 weeks later if the label is still present.