Feature request: inverse operation, start with a GitHub URL which contains hash line number(s), and open the file in VSCode at the designated line / code selection

Hello, great extension, thank you!
Very useful for code peer reviews over Slack / instant messaging, to exchange URLs whilst working locally from VSCode.
However the inverse functionality would be very useful too.
For example I would like to be able to copy such URL: https://github.com/differentmatt/vscode-copy-github-url/blob/ad85a468eb8f1b05979591335ba391c58abeb5a7/src/main.js#L6-L8
https://github.com/differentmatt/vscode-copy-github-url/blob/ad85a468eb8f1b05979591335ba391c58abeb5a7/src/main.js#L6-L8
...and to show src/main.js in VSCode with the code selection line 6-8 (extracted from the URL #L6-L8).
Any thoughts?

Cool idea.  What do you think the user experience would be to enter this URL into VSCode and open the correct file with selection?

Could it be a command that would prompt the user to paste?

Hello again!
Disclosure: some months ago I settled on https://github.com/d4rkr00t/vscode-open-in-github for the vscode-to-browser operation, but I am glad other alternatives such as differentmatt/vscode-copy-github-url exist :)
For the reverse browser-to-vscode operation I have been using https://github.com/alexander-yu/vscode-open but the experience is not totally friction-free due to necessary configuration, e.g.:
    "open.uriMappings": [
        {
            "pattern": "https://github.com/owner1/repo1/(?<blob>[^/]+)/(?<ref>[^/]+)/${regex:file}${regex:lines}",
            "output": "/Users/danielweck/GitHub/owner1/repo1/${match:file}${match:lines}",
            "lineSeparator": "#"
        },
        {
            "pattern": "https://github.com/(?<owner>[^/]+)/(?<repo>[^/]+)/(?<blob>[^/]+)/(?<ref>[^/]+)/${regex:file}${regex:lines}",
            "output": "${match:file}${match:lines}",
            "lineSeparator": "#"
        }
    ]


Add elevation to routing evaluations

Objective
This PR aims to tackle issue #18 with the omission of allowing end users to toggle edge colouring by steepness.
Changes
Config will now accept a dem attribute which is the location of the tif file users want to extract elevations from, but this is all optional.
The Edge struct now has the new attributes:

forward_cost the cost of travelling on an edge from node1 to node2
backward_cost the cost of travelling on an edge from node2 to node1
slope the slope returned for a given edge as a result of the input tif file
slope_factor an optional tuple that holds the slope_factor for the forward_cost and backward_cost

The steepness is used when doing batch calculations in the calculate_batch function which returns an option tuple of costs now.
For now the forward and backward costs are calculated by multiplying a base cost by the slope_factor. For more details see here.
Breaking changes
Users with external commands need to edit them to make sure they output a tuple of costs not a float now as a result of the changes.

Thanks so much for this! It's a very impressive first PR in Rust. How has learning the language been so far?
A bunch of comments, mostly Rust style stuff. Let me know if anything's unclear (or if you don't have time to address them). And could you please run cargo fmt to auto-format everything?

Cheers man it is very fun to learn, definitely think reading the book first helped me out ALOT, but I still have a lot of trouble with the different pointers it uses and the traits system - I probably suck at design if it is in Rust. I am also slower when using it, but also much more thoughtful with what I am writing so I consider that a win. I will carry on using it, just because it is a very nice language.
And sweet I will look through the changes and makes some updates now!

Thanks for the comments btw they were really helpful! Also for the examples, how will you add the geotiff file?

I'll change the setup.py scripts to download the file, probably.
I'll open a PR and CC you on any changes I make tomorrow

I'll change the setup.py scripts to download the file, probably.
I'll open a PR and CC you on any changes I make tomorrow

Please do man!


fix: omit double parse

See https://github.com/reasonml-community/graphql_ppx/pull/120#issuecomment-627241933

Can you update the snapshot so the tests pass? Then I will merge ASAP.

Oh wait no others test are broken and they should. It isn't the right fix. Sorry for the noise.

@jsiebern somehow there is a bug that a color field is parsed twice (once with the incorrect datetime decoder). I looked through the source but couldn't find an obvious explanation why this is happening (the commit is not the correct fix). Any ideas?

Probably forgot to add an
if (!raw) {};

somewehre 🤔 I'll check later tonight

Ah I found it! It double encodes because it is a custom field AND it has an override with @ppxDecoder, this case isn't covered atm.


Provide a way to parse JWT claims without validating

In some scenarios getting access to an in memory structure representing the claims can be really useful, for example to integrate with https://github.com/xeipuuv/gojsonschema.
The feature is mostly already there, just would need to be exposed separately.
Thoughts?

@dgrijalva how to do the parsing in recently released V3?

@dgrijalva late the part but I'm curious, is there a way to parse the token and then validate it once you retrieve the key because retrieving the key means parsing the token to get the "user_id" inside the token?

This is why the api accepts a callback function instead of just a key, so you can look it up. Just be sure you double check your signing method.
You can also use ParseUnverified and then Call the validation stuff yourself but I do not recommend this method.
-dave

On Oct 13, 2020, at 5:18 PM, Kareem H<EMAIL_ADDRESS>wrote:

@dgrijalva late the part but I'm curious, is there a way to parse the token and then validate it once you retrieve the key because retrieving the key means parsing the token to get the "user_id" inside the token?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


Allow nonce to be set for debugbar_body

Allow nonce to be set for content security policies

@julienbourdeau any thoughts on this?

Hey @julienbourdeau just wanted to ask again if you had any questions or feedback on this change.

Hi @trevorrjohn
I'm really sorry for the delay. I don't see any reason not to merge it. I'll merge and release it this weekend 🙏

Hey @julienbourdeau sorry to bother again. I know you have a lot going on, but I didn't want this to get dropped.

Hey @trevorrjohn Sorry again for the delay!
I just published 0.3.3 with this fix https://github.com/julienbourdeau/debugbar/commit/6df7e2ebfda55b529f59e22e36bafdfc9adbc945

@trevorrjohn What did you have to do to get the nonce set? I might be doing something wrong, but what I see is:
<script type="text/javascript" data-turbo-permanent nonce="">
  window._debugbarConfigOptions = {"cable":{"url":"...
</script>


[Bug] - forEach with concurrency much slower than without

forEach with concurrency much slower than without

CPU seems very high when using this feature

There is an overhead that forEach will always encounter as it spawns the threads so usage should only be when the flow it executes is slow performance i.e. external script or process.


[multistage] group-by throws NPE when dealing with null values

several issues with handling null in group-by, (typical scenario: group-by is applied after left-join)
the problems are:

when the nullable column is in the group-key.  the group-set aggregation causes the results (or error) to not populated through.
when the nullable column is in the agg function. it simply throws NPE

the problem might also happen to other operators that doesn't handle null arguments

Fixed with #10799


Add CI Security, SAST and OSSF checks

You don't need to be blocked by reported issues from our CI tools for now. But it's a good thermometer to see if we are stumbling in vulns and license issues to remediate them when needed.

Looks like CodeQL is using an outdated Swift tools version

error: 'web5-swift': package 'web5-swift' is using Swift tools version 5.9.0 but the installed version is 5.7.1

Curious if there's a path to upgrade this. I could lower the required Swift version, but it would require some code changes. Happy to do it though if that's the only path forward.


Improvement in your stack

Hi,
First of all I would like to say "Good job". I was creating a Stack for Prestashop and when I saw your repo, I picked up. It is a really good stack.
I have made some changes because I need multiAZ RDS database and the default timeout you stablished are 15 minutes, and to create a multiAZ database, it takes more than 15 minutes. I use it in eu-central-1 region, and it took 19 minutes to create a MultiAZ Database.
On ther other hand, I think you should add two extra features:

A Bastion Host. Sometimes we need to get access to the database. The securest way is by a Bastion Host creating a SSH Tunnel between the RDS server and internet.
HTTPS in Load Balancer. In production mode, we need this features and now we have to create it manually, it would be a good idea to create this from Cloudformation Stack.

Regards.

Hi @antoniomperez, thanks for the comments.
How did you extend the timeout for creating an RDS Database? Feel free to submit a PR to fix this.


Unicode path

Hi,
thanks for the great piece of software! I encountered a small inconvenience:
Using magic-wormhole 0.9.2, I get an UnicodeDecodeError originating in os.path.join used in cmd_send.py.
I'm not exactly sure how to globally fix it, but treating all paths as unicodes should reduce further issues.
Thanks and keep up the good work!
Matthias
Minimal example for reproduction:
:~$ mkdir 'testö'
:~/testö$ touch test
:~/testö$ wormhole send test

Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/twisted/internet/defer.py", line 1447, in unwindGenerator
    return _inlineCallbacks(None, gen, Deferred())
  File "/usr/local/lib/python2.7/dist-packages/twisted/internet/defer.py", line 1301, in _inlineCallbacks
    result = g.send(result)
  File "/usr/local/lib/python2.7/dist-packages/wormhole/cli/cmd_send.py", line 58, in go
    d = self._go(w)
  File "/usr/local/lib/python2.7/dist-packages/twisted/internet/defer.py", line 1447, in unwindGenerator
    return _inlineCallbacks(None, gen, Deferred())
--- <exception caught here> ---
  File "/usr/local/lib/python2.7/dist-packages/twisted/internet/defer.py", line 1301, in _inlineCallbacks
    result = g.send(result)
  File "/usr/local/lib/python2.7/dist-packages/wormhole/cli/cmd_send.py", line 70, in _go
    offer, self._fd_to_send = self._build_offer()
  File "/usr/local/lib/python2.7/dist-packages/wormhole/cli/cmd_send.py", line 196, in _build_offer
    what = os.path.join(args.cwd, args.what)
  File "/usr/lib/python2.7/posixpath.py", line 73, in join
    path += '/' + b
exceptions.UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 19: ordinal not in range(128)
ERROR: 'ascii' codec can't decode byte 0xc3 in position 19: ordinal not in range(128)

Doh! Thanks, I'm always messing up unicode stuff. I'll see if I can add a test case to trigger this, and then figure out a clean way to handle it properly.

Hrm, I'm having problems reproducing this locally (both macOS and Ubuntu-17.04 work correctly, on both py2 and py3).
Could you download a copy of https://github.com/tahoe-lafs/tahoe-lafs/blob/master/misc/build_helpers/show-tool-versions.py , run it, and attach the output? Or at least the first three parts, which show encoding settings. I know it's a real problem (we've seen it in Tahoe before, and added that version tool to help trace things down), but I want to make sure I understand it fully. I think I should just set my locale settings to match yours.

note to self: the fix will probably be to copy over argv_to_unicode() and the io_encoding support from tahoe (https://github.com/tahoe-lafs/tahoe-lafs/blob/master/src/allmydata/util/encodingutil.py#L87)

Sure, I'll give it a go. Unfortunately I won't have time before next week, though.
FYI: I run a Debian testing (stretch) with cinnamon desktop and the gnome-terminal emulator.

would it be easier for you to paste the output of printenv? That might get me halfway there. I'll see if I can spin up a stretch box in the meantime.

right, sorry, I was just skimming your message and thought it was a bigger deal ... I just ran show-tool-versions.py  std out is:
show-tool-versions_shell.txt

cool, thanks.. I see locale.getpreferredencoding() is UTF-8, so it looks like the technique we use in Tahoe ought to work here too. I'll make a branch to test out.
I got a stretch box running (in a docker container), but for some reason it's preferred encoding is ANSI_X3.4-1968, which I think is the technical name for ASCII, which means it just can't handle non-ascii filenames at all (I think). The travis-ci boxes seem to have sensible encodings, though. So I think this branch is likely to work on everything (my box, travis, your box) except the stretch image I launched. Good enough :)

@mbuschmann when you get a chance, could you try the 157-unicode branch? I discovered that Click (our argument parser library) already has a mode that processes filepaths, and that sounds easier than copying over a bunch of code from Tahoe.
I added a test that should exercise the unicode stuff, and it passes on this new branch, but it didn't fail on trunk, so I think we need a manual test to see if it helps. At least doesn't seem to break anything :).

I see Config is using os.getcwd and building its own paths, etc. -- doesn't Click deal with this stuff properly?
I'm not sure if there are "bare helper" functions or not. This is somewhat complicated by the fact that the "what" arg is "maybe a file, maybe some text" so send is not using the file-handling Click options :/

The os.getcwd thing: yeah, I think it might.. maybe we should pull that out.
args.what should only be a filesystem path (either a file or a directory). If you use --text, you'll get args.text instead.

I looked at Click's click.Path, and it defaults to using resolve_path=False, which means we need to add CWD ourselves (which is also used by the current tests, to exercise resolution from various directories). We could switch that to True, but that would also resolve symlinks ahead of time, which means the "basename" we send to the far side would be that of the target of the local link, not the source. I think users would prefer to use the source name, so that e.g. if I symlink "today.csv -> all_reports/2017-06-20.csv", and then do wormhole send today.csv, the recipient will store a file named today.csv and not 2017-06-20.csv.
I'll add a note to the branch on why we're still doing an os.path.join in cmd_send.py

I landed this and made the 0.10.0 release. It might fix the unicode problem, but it might not (now I'm worried about what will happen if the directory you're sitting in has a non-ascii character in the path, independent of whether the filename you're sending is ascii or unicode).
@mbuschmann , when you get a chance, could you install the new 0.10.0 release and see if it helps? Thanks!

I just virtualenv'd 0.10.0 and the unicode problem still remains if non-ascii character is somewhere in the root path, see attached log.
log_wormhole_0.10.0_unicode.txt

I had another look at this and it seems to me that in v0.10.0 changing line 245 cli/cmd_send.py to
what = os.path.join(args.cwd.decode('utf-8'), args.what)
fixes the problem on my end. An ugly fix for sure, but it confirms, that the issue is with the encoding of args.cwd. My guess is, that decoding all str args to the same encoding will fix this issue globally. os.path.join should do the rest, I think.

It seems very unlikely to me that just pretending everything is utf-8 is the right solution; @mbushmann is your environment "odd" in some way? Can you dump the environment you're running this in?

I agree.  I can't see anything odd about the environment. Debian 9 with Cinnamon Desktop. locale is LANG=en_GB.UTF-8

Ugh, so I was right about cwd still being a problem. We could either use os.path.realpath() like Click does (and remove args.cwd, which to be honest is really only there so we can test the os.path.join), or do the other thing Click does and use sys.getfilesystemencoding() or sys.getdefaultencoding() to decode os.getcwd() too.
I'm leaning towards the former (realpath), but I want to pull off the basename before doing that, so the receiver gets the filename that the sender typed, rather than the target of the symlink it references.

I wonder if it's worth breaking the CLI a little so that there's --send (for file paths only) and --send-text or similar for "definitely not a file path". Then --send can just use Click's Path and it handles all the py2 vs. py3 and encoding stuff.
To me it feels a little odd anyway to overload a single argument as "might be a file, but if it's not then we pretend it's text".
A hypothetical --send-text will still have the encoding problems inherited from Python2 though, right? I.e. you're getting a smattering of bytes from the args but don't know what encoding they're actually in (right?)

you mean wormhole send FILE and wormhole send-text TEXT, right? We don't currently have any polymorphic "keyword" arguments (--something=SOMETHINGELSE). There is no argument that might sometimes be a filepath and at other times might be a text string to be sent. And the only polymorphism we've got in the "positional" arguments is whether it's present or not. The current CLI is just:

wormhole send FILE
wormhole send DIRECTORY
wormhole send --text TEXT

(note that in the text case, there are no positional arguments)
That said, I'm open to considering separate subcommands for sending filesystem things and sending text strings, as long as it doesn't make the commands too verbose.

Ahhh, right sorry :( I was misremembering the command-line. So is there a reason we can't use a click.Path argument directly then?

That's what we're doing, as of 758dd5b9, except the lingering problem is when the parent directory is not ascii-safe. I think (but need to test) that click.Path(resolve_path=True) would fix that, but it would also change the basename in a way that I don't want. So I think the next step is to use a call to os.path.realpath() inside cmd_send.py (which I'm assuming does the right thing w.r.t. decoding os.getcwd(), but maybe it doesn't), except we should copy out the basename before doing realpath() so we get the pre-symlink-chasing value. And then stick with our current click.Path(resolve_path=False).

Hm, args.cwd makes the file-copying unit tests easier: to make the sender and the receiver use different directories. I've started on the branch to make this change, but fixing up those tests is proving annoying.


Esc discards changes after compilation error

Hello,
First, thank you for this new version supporting macOS; it's incredibly useful.
I did notice a behavior that, while perhaps not strictly a bug, feels quite counterintuitive. When I write a piece of LaTeX and the compilation fails, a modal opens up asking if I want to re-edit the code, open the logs, etc. Often when this happens, my instinct is to press 'Esc' to close the modal and return to editing the code to fix the mistake. However, pressing 'Esc' not only closes the modal but also the text editor, which results in the loss of any modifications I have made.
I would expect pressing 'Esc' to close just the modal, allowing me to continue editing the code, or at the very least to prompt a confirmation before discarding changes.
Thanks again for your great work!
Best,
Lorenzo

Thanks for using LaTeX2AI and posting your preferences on how to use the PlugIn! I remember that the current behaviour you describe was implemented by choice, since it seems natural that exiting on ESC and pressing Cancel yields the same behaviour. I will talk to the LaTeX2AI users I know and see what they prefer. (Others are welcome to post their preferences here as well).


Clarify activeParameter in SignatureHelp

activeParameter is currently speced to be only valid on the active signature. However we use it as well if the users scrolls through signature options to highlight the active parameter. We should adjust the doc accordingly and we might want to consider whether signartures that have less number of parameters than the active parameter should be filtered.

I believe we have all of that now, esp the user selected value is passed on to extensions now


Fix Integration tests

Describe the bug
Some Integration tests are in fail or error
Expected behavior
Integration tests must pass
Additional context
Maybe some fixes on SaagieAPI

Trello card: [#224] Fix Integration tests


SECURITY issues during deployment

I performed a simple check with:
python manage.py check --deploy

and I got the following warnings:
WARNINGS:
?: (security.W004) You have not set a value for the SECURE_HSTS_SECONDS setting. If your entire site is served only over SSL, you may want to consider setting a value and enabling HTTP Strict Transport Security. Be sure to read the documentation first; enabling HSTS carelessly can cause serious, irreversible problems.
?: (security.W006) Your SECURE_CONTENT_TYPE_NOSNIFF setting is not set to True, so your pages will not be served with an 'X-Content-Type-Options: nosniff' header. You should consider enabling this header to prevent the browser from identifying content types incorrectly.
?: (security.W007) Your SECURE_BROWSER_XSS_FILTER setting is not set to True, so your pages will not be served with an 'X-XSS-Protection: 1; mode=block' header. You should consider enabling this header to activate the browser's XSS filtering and help prevent XSS attacks.
?: (security.W008) Your SECURE_SSL_REDIRECT setting is not set to True. Unless your site should be available over both SSL and non-SSL connections, you may want to either set this setting True or configure a load balancer or reverse-proxy server to redirect all connections to HTTPS.
?: (security.W012) SESSION_COOKIE_SECURE is not set to True. Using a secure-only session cookie makes it more difficult for network traffic sniffers to hijack user sessions.
?: (security.W016) You have 'django.middleware.csrf.CsrfViewMiddleware' in your MIDDLEWARE, but you have not set CSRF_COOKIE_SECURE to True. Using a secure-only CSRF cookie makes it more difficult for network traffic sniffers to steal the CSRF token.
?: (security.W018) You should not have DEBUG set to True in deployment.
?: (security.W019) You have 'django.middleware.clickjacking.XFrameOptionsMiddleware' in your MIDDLEWARE, but X_FRAME_OPTIONS is not set to 'DENY'. The default is 'SAMEORIGIN', but unless there is a good reason for your site to serve other parts of itself in a frame, you should change it to 'DENY'.

We should review and address them!

Relevant: https://reversepython.net/lab/django-web-security-checklist-deployment/


[vCenter] Boot order functionality

Description
Capability of use BOOT attribute of OpenNebula in vCenter.
Allow you to choose the boot order for the vm.
Use case
Import a vCenter template, and change the boot order of the spawned vms.
Interface Changes
We need to add the same section of OS | BOOT that is available in KVM sunstone but not in vCenter view.
Additional Context
The mandatory thing is to choose between disk order.
This feature aims to provide that functionality, we will consider to keep this feature until the whole process boot is done (nics and other kind of devices) or make a new one.

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

For the moment is only going to work with unmanaged disks (safe feature), however is going to be possible to use any kind of disk soon.


Can't open files larger than 1MB

I really like your editor. However sometimes I need to open files larger than 1MB and the app doesn't allow it.

Duplicate issue.


css properties that generate a class starting with "ad" might get hidden by ad blocking extensions

Noticed this while working with animationDuration and animationDirection
they will map to atomic classes that start with ad so ad_1, ad_2, a_x
the funny thing is that some ad-blocking extensions will mistake these as ads and will apply display:none to them 😄
Overall, I'm leaning towards disabling these short incremental classes that get generated when the library is running client-side and completely rely on hashed versions even if it's a tiny bit more costly as this would be more consistent and might even solve #72 since currently react complains during hydration that the dom isn't matching when SSR isn't enabled.

haha brilliant.
I do think there's a certain benefit of showing the abbreviated property though. Do you think we'd have the same issue if that was at the end of the hash?

I just have to say I am so impressed by how thorough you are @hadihallak 😂

@peduarte
I think it should be fine in this case. should we open an issue to disable them in production as I don't think this behavior is limited to development mode currently
@christianalfoni that's only because I'm enjoying my time playing around with it 😄
kudos to you for making it an enjoyable experience

@hadihallak I think this only happens in development. Here's a vercel preview build, inspect the buttons, they're built with Stitches. The classnames don't seem to contain the abbreviated property name

@peduarte I think that's because you're using SSR which forces hashed classes. I believe in a CRA app things will be different

fixed in #120


Documenting the json_extract SQL function

Added documentation for the json_extract function as recently introduced by https://github.com/questdb/questdb/pull/4633.

LGTM! Will merge after release ships. 👍


squid:S1488 - Local Variables should not be declared and then immediately returned or thrown

This pull request is focused on resolving occurrence of Sonar rule
squid:S1488 - Local Variables should not be declared and then immediately returned or thrown.
You can find more information about the issue here:
https://dev.eclipse.org/sonar/rules/show/squid:S1488
Please let me know if you have any questions.
George Kankava

Looks good.


UpStream Dev Merge

?

wrong direction?

Yea lmao, was trying to pull main into mine to stay upto date


Cauchy|Added Info component as well as mmr instruction on MMR page.

UI as shown, new to react xD

@blukai @howardchung Fix for #1121 , please review, thanks!

Change pushed, please review @blukai @howardchung

Is it possible to make it show if last mmr update was more than ~1 day ago for example?

@blukai I should be able to implement that, will give it a try :) 👍

@blukai Just started up the front end and found that if you don't have a game a day ago, the MMR graph won't show MMR for that day, seems it is only catching the MMR changes. For example I haven't played ranked game for 5 days and it only shows my MMR until 5 days ago. Don't know how we could detect the MMR update though.

{
  account_id: 210377205,
  match_id:<PHONE_NUMBER>,
  solo_competitive_rank: 5481,
  competitive_rank: null,
  time: "2017-04-26T15:09:11.489Z"
}

I guess u can compare the last time when it was updated to current time

@blukai But I mean it is still possible that MMR was updated 5 (more than 1 day) days ago even though it is up to date. Just like in my case I didn't play for five days so the updated time should have 5 days difference to current date.

I think it's fine to just always show the message.


Add C4 Engine

SteamDB app page links to a few games using this
https://steamdb.info/app/283600/
Brief explanation of the change
Added C4 Engine detection

Looks good to me now, thanks for the PR!


Confusing order

By keeping the list sorted by alphabetical order using the username and not the original names has made the ordering confusing. It will be neater to use full name as the sorting key.

Sure. Feel free to submit a pull request for the same.


Performance degredation with v2.5.2 and JSON serialization after upgrade from 1.6

NEST/Elasticsearch.Net version: 2.5.2
Elasticsearch version: 2.4.4
Description of the problem including expected versus actual behavior:
After upgrading from 1.6 we are seeing significant performance degradation with the JSON serialization logic. It appears as though the client serializer may have been static in version 1 however it is not now perhaps. Timings around serialization are up from about 3ms on average with v1.x to about 300ms with v2.x.
I have seen a few issues around this but none have called out a solution that I could discern. Note that establishing a lazy singleton for connection settings appears to correct this with only the initializer incurring the larger performance hit but wanted to see if (1) this is the correct solution and (2) if a shared connection setting object (example below) may be considered thread safe.
Example of shared connection setting:
private static Lazy connectionSettings = new Lazy(() =>
{
var uri = new SingleNodeConnectionPool(new Uri("http://localhost:9200"));
var esSettings = new ConnectionSettings(uri).DefaultIndex("myIndexName");
return esSettings;
});
Steps to reproduce:

Nest query containing aggs, filters, etc.
Call to search or client.Serializer.SerializeToBytes (settings below)
Review of timings around calls to NEST in comparison with response time from ES

Provide ConnectionSettings (if relevant):
var settings = new Nest.ConnectionSettings();
if (forceLowercasePropertyNames)
{
settings.DefaultFieldNameInferrer(x => x.ToLower());
}
var client = new Nest.ElasticClient(settings);
return client.Serializer.SerializeToBytes(o);

Hi!
All of our AND json.net caches are per ConnectionSettings and your absolutely encouraged to share a single instance of it over multiple clients and that should be totally thread safe to do so.
The first call will typically incur overhead while these caches are built.
Our Lifetimes documentation goes into this as well.
Let me know if this answers your question or if I missed anything and closed this too soon!


Add module for potions/infusions and food

Pre-cast pots only show up as the Medicated status, which we can't check ID for - will need to be looked into.
Likewise, Food may not have any events for an entire fight, so will need to look into how that can be checked - potentially a separate query?

Partially implemented by 7eecda2.

Staaaaaaaaale

Food is actually useless, but at least status medicated can be displayed, it makes very strong understanding how it was lined up with pty and self buffs and also could be good point for optimizations

Food is not useless. Food stat buffs are sufficiently substantial to be worthwhile, and BiS sets are often built with stat tiers assuming that food is being used.

I mean you cant make any suggestion regarding to food as no info about gear or what food was applied. But medicated status(does not matter what exact pot) has value to be displayed in timeline. some where near raid buffs to see their intersection


Bug/HomeSignIn-page-issues

Please review/merge #56 first.

[x] Login Email & Password input fields should both have name attributes
[x] Login Form should support Enter to submit
[x] Buttons on page need icons

 Bug: HomeSignIn page issues


Add container 'owned' transient lifetime manager ContainerControlledTransientManager

Normally Unity container does not keep track of created transient objects. It creates them and forgets them. In certain situations it is useful to make container remember all transient objects it have created.
So, ContainerControlledTransientManager is required. This lifetime manager is the same as TransientLifetimeManager except if the object implements IDisposable it will keep strong reference to object and dispose it when container is disposed.
If created object is not disposable, container does not maintain any object references so when that object is released GC will collect it immediately.

Hello,
Great work with the container so far. Is this something like this https://thorarin.net/blog/post/2013/02/12/Unity-IoC-lifetime-management-IDisposable-part1.aspx extension?
If not do you have any guidance how to convert the extension to the new version?

ContainerControlledTransientManager is what the blog calls DisposingTransientLifetimeManager. It can be used without any extensions.
DisposedShared could be done as ContainerControlledLifetimeManager.
There are no teardown in Unity anymore, but it all should work with properly scoped child containers


Unnecessary overflows added to menu styles create scrollbars

Short description
A menu item added to a toolbar adds overflow-y:auto which causes a scrollbar to appear.
What is current behavior

What is desired behavior
No scroll bar.
Steps to reproduce or sample
Create a layout and add the toolbar below.
    toolbar: {
        name: 'toolbar',
        items: [
            { id: 'options', type: 'menu', text: 'Options', icon: 'fa fa-cog',
                items: [
                    { id: 'logout', text:   'Logout     ', icon: 'fas fa-sign-out-alt', route: 'logout' },
                ]
            }
        ]
    }

Cannot reproduce: http://jsfiddle.net/ja189qk0/1/
Tested with Firefox and Chrome.
Please provide a working fiddle next time.
You can use any demo as a starting point (e.g. http://w2ui.com/web/demos/#/toolbar/1) and then click "Show Source Code" button and then the "fiddle with code online" link.
Maybe you have a browser zoom level set?
Or maybe you afer more CSS that changes how the toolbar item is displayed?

Cannot reproduce: http://jsfiddle.net/ja189qk0/1/

I'll take that back. I cannot reproduce this in Chrome 88, but in Chrome 91 it looks like this:

Possible work-around until this is properly adressed and fixed:
div.w2ui-menu {
    width: unset !important;
}

See: http://jsfiddle.net/vb5u20nj/


reference links are spellchecked

Describe the bug
The https links are spellchecked, which makes them have red lines under a lot of the text.
Code example
[2](https://ourworldindata.org/children-not-in-school)
            ^^^^^^^^^^^^^^ <- this is underlined in red

Expected behavior
links shouldn't be spellchecked
Actual behavior
links are spellchecked

This is not a bug with this parser, but rather the integration into the editor. But I can point you in the direction of the right people to ask. What editor are you using? If it's neovim then this issue belongs to nvim-treesitter.

Ohh, ok thanks so much! I literally started using treesitter yesterday with nvim so I don’t know how most things work.


Transfer possible without cables

MC 1.10.2
Forge 2297
XNet 1.3.1
So ever you announced XNet with cables being no TileEntities I was excited, but was also wondering what side effects this could have and could come up with one. I wasn't able to check XNet out for my pack and not until now, but today I could finally test it and was able to break it. Basically you can use pistons to move every single cable and then break the cable afterwards. The Controller indeed removes the connected block from the list, but you can still use an already established connection even though the cables are removed. Pushing the cable back (for example the piston being sticky) in place also doesn't re-add the connected block.
I mean not sure if ever anyone would use it to create wireless transfer, but maybe there are other scenarios possible using a similar mechanic.

Right when I posted I came up with a more pracitical solution and indeed it works. The Destruction Wand from ExtraUtilities2 can break a lot of connected cables in one go. The connection is still intact though.

I will check out what is going on. Normally those breaking things should notify the block that it is getting broken. Somehow this doesn't seem to work

Fixed in 1.3.2


Apply scrollify.js to only certain sections on the page, and let other sections scroll normally

Hi Lukehaas,
Nice plugin. but what if I have to apply it to certain sections on a page.
Actually I am having a small footer at the end of all sections.
when I scroll to the last section, and try to scroll to the footer. the plugin does not allows me to do so.
all the sections are having a height of 100%. but the footer is set to height auto which how should i exclude the footer from the plugin and allow it to be scrolled normally.

Put the footer at the bottom of the last section.

Hi Luke,
Yup got that
Thank you
Thanks & Regards
Yogendra U. Karekar
Mobile:-9769856284
<EMAIL_ADDRESS>On Sat, Mar 5, 2016 at 4:39 AM, Luke Haas<EMAIL_ADDRESS>wrote:

Closed #44 https://github.com/lukehaas/Scrollify/issues/44.
—
Reply to this email directly or view it on GitHub
https://github.com/lukehaas/Scrollify/issues/44#event-578508641.

Sir,
I am creating a parallax website where I want to implement multiple scrollify panels and multiple paginations. please help.


Updating CC list

Updating the CC list in our docs. @RackHD/corecommitters

Thanks for updating, Paul!


Issue while running command - external server

I have deployed your solution in a different server and when I am trying to run the command, I am getting below error, Could you help me to debug it.
const { copyToPath, playbackFile } = require('./env');
      ^

SyntaxError: Unexpected token {
    at exports.runInThisContext (vm.js:53:16)
    at Module._compile (module.js:374:25)
    at Object.Module._extensions..js (module.js:417:10)
    at Module.load (module.js:344:32)
    at Function.Module._load (module.js:301:12)
    at Function.Module.runMain (module.js:442:10)
    at startup (node.js:136:18)
    at node.js:966:3

Here is my .env file
rtmpUrl=rtmp://a.rtmp.youtube.com/live1/MyKey
ffmpegServer=ws://localhost
ffmpegServerPort=4000
auth=mZFZN4yc
copyToPath=/var/www/
playbackFile=playback.html

Node Version Problem, Solved by Upgrading the Node Version.


add GitHub issue templates

Issue #, if available:
Description of changes:

adds bug-report issue template
adds feature-request issue template
adds config.yml to add a link for our Discord community

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

updated issue templates to include npx envinfo command. templates will look something like this (but with required fields)

and render something like this


[FEATURE] Include number of atomic orbitals

From Umberto:

a field that was required from the standardized output is the number of orbitals, you can return this info with the data that you already have in your dictionary by returning this
len(data["atomic_result"]["extras"]['qcvars']["orba_energies"])

Can add this in the job_output_to_atomic_result helper function.


Add an assortment of new "always-on" metrics.

This is a followup to #8477 with a bunch of additional metrics in the "always on" UnifiedStatsReporter class. These are intended either for CI-regression-prevention or helping users characterize performance problems in the field.
Note: by "always on" I mean always available in the build; they're not actually counted unless a user runs with -stats-output-dir.

@swift-ci please test

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - b5292f04f8cb464ef08693cef7c3288513e0c5dc
Test requested by - @graydon

@swift-ci Please test platform Linux

@swift-ci Please test Linux Platform

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - b5292f04f8cb464ef08693cef7c3288513e0c5dc
Test requested by - @graydon

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - b5292f04f8cb464ef08693cef7c3288513e0c5dc
Test requested by - @graydon

@swift-ci Please clean smoke test

Seems reasonable to me!


alias

Hi Copych, I'm very curious about how synthvoice deals with alias. In a previous uploaded version there was two variables referring to saw and square bleps, but none code was using them. I've disabled the lowpass filter in order to analyze the spectrum of the 303 oscillator and alias was present.
Modifying some of Marcel License's synth code I've come up with a very begginer solution, which is to generate multiple wavetables that are already band limited, and to point to them accordingly to the desired pitch. That system works but its very inefficient, so I was wondering what about blips, bleps and so on.
Any help is well received, cheers!
Nicolas,.

Hi, Nicolas, for the time being I have cut out blep code, and only left linear interpolation, because of two reasons: first is that I couldn't make it work as supposed, and the second is that within those two low octaves aliasing is almost not heard. Maybe I return to this matter later, and there're ready-made solutions, but I want to keep it reasonably ascetic when talking about cpu comsuming. Actually I want to make a real-world "toy" with real knobs and buttons, and for now it seems possible to push in one more sampler for pads, and this is where blep will be of high demand.

I understand. As you have mentioned it, in the original project was hardly audible. Your code helps me to understand a lot of things, especially the dual core usage, which is very rare in what I've seen in audio code in esp32.
Do you have a version with the blep code?

I don't have the code ported, but I can address you to this code : https://github.com/electro-smith/DaisySP/tree/master/Source/Synthesis , they have a lot of things done.

BTW, Open303 uses mipmapped wavetables (i.e. the same method Marcel Licence used) it requires some extra kB's, but not that cpu-comsuming. We also can use some kind of LPF to cut the frequences higher than SampleRate/2. When LPF coefficients are precalculated, it's quite fast to make some float multiplications and additions, using FPU of an ESP32. I am quite sure that it's faster than some "if" instructions.

Thank you very much for the data...now I have some days to read and learn. Code examples like Marcel's and yours help me a lot.
From his basic synth example I wrote a Prophet 5 like synth (with almost every mod included), which is very fun to play, but it has a ton of "if" in the sample generating function, in order to point the correct band limited lookup table. That makes the esp slow as hell. That lead me to search different ways to deal with the problem. Also your code uses both cores to generate sound, its amazing.
I have one question, if you start from naive waveforms or non band limited tables, how do you cut alias with filters without making the sound dull?

I have one question, if you start from naive waveforms or non band limited tables, how do you cut alias with filters without making the sound dull?

You'll have to filter naive waveforms, then resample

You can try this very cheap variant of antialiasing =)
line 109 of synthvoice.ino
    // Increment and wrap phase
    _phaze += _currentStep;
    if ( _phaze >= WAVE_SIZE) {
     // _phaze -= WAVE_SIZE ;
      _phaze=0.0f;
    }

The price for simplicity is rounding wave periods to integer count of samples. It's inaudible at lower frequencies, but miss the note freq's at high octaves ) So not solving the problem.

I understand this is a topic where you have to make compromises between pros and cons, and not always the better solution fits for all situations.
I'm very happy with the sound of band limited lookup tables calculated at the start of the program, because memory is very little used in my project, but I can't find a solution to get rid of the annoying chain of IFs. With seven tables, from 220 to 8 harmonics, I've got a clean saw, which then it's recalled two times, inverted 180º and with some phase offset, and then summed to get a clean pwm square wave. Triangle wave made this way had too much ringing to be usable but the discontinuity in this waveform is barely hearable as alias.
This awful block of code is two times computed every sample...works ok? yes, but it cuts down polyphony a lot.
The values of transition from one wave to another was carefully chosen to always have at least harmonics up to 22khz and down to 15khz. It is visible on the spectrum analyzer but not to my ears :)
          if(addVal_aux <= 280601216)  // point saw table with 8 harmonics
            osc->waveForm = &saw7;
          if(addVal_aux <= 176767744)  // point saw table with 13 harmonics
            osc->waveForm = &saw6;
          if(addVal_aux <= 93639432)   // point saw table with 22 harmonics
            osc->waveForm = &saw5;
          if(addVal_aux <= 62496828)   // point saw table with 35 harmonics
            osc->waveForm = &saw4;
          if(addVal_aux <= 39370532)   // point saw table with 49 harmonics
            osc->waveForm = &saw3;
          if(addVal_aux <= 26276678)   // point saw table with 80 harmonics
            osc->waveForm = &saw2;
          if(addVal_aux <= 15624207)   // point saw table with 130 harmonics
            osc->waveForm = &saw1;
          if(addVal_aux <= 9290209)    // point saw table with 220 harmonics
            osc->waveForm = &saw0;

My wild guess on optimizing performance of this portion of code is to try to find some simple f(addVal_aux) giving you index of an array containing addresses of saw0..saw7. Surely there'll be some loss in accuracy and some excess in array length, but maybe acceptable for your needs.

Since aliasing is effective a couple of octaves higher than 303 working range, any further discussions on this matter are welcome, as well as it's not an issue )


Properly use ProxyBroker with Selenium

What is the correct way to scrape N amount of proxies and then use them in a loop? for instance, here is my code:
import asyncio
from proxybroker import Broker
from selenium import webdriver

async def show(proxies):
    proxy_list = []
    while True:
        proxy = await proxies.get()
        if proxy is None:
            break
        # check accurately if the proxy is working
        if proxy.is_working:
            protocol = 'https' if 'HTTPS' in proxy.types else 'http'
            line = '{0}://{1}:{2}'.format(protocol, proxy.host, proxy.port)
            proxy_list.append(line)

    return proxy_list

def proxy_get():
    proxies = asyncio.Queue()
    broker = Broker(proxies)
    tasks = asyncio.gather(broker.find(types=['HTTP', 'HTTPS'], countries=['US', 'GB', 'CA'], limit=10), show(proxies))
    loop = asyncio.get_event_loop()
    results = loop.run_until_complete(tasks)
    return results

if __name__ == '__main__':
    for n in range(0, 10):
        print('run #{0}'.format(str(n)))
        proxies = proxy_get()[1]
        count = 1
        for proxy in proxies:
            print("Loop #{0}".format(str(count)))
            count += 1
            print("Using proxy: {0}".format(str(proxy)))
            chrome_options = webdriver.ChromeOptions()
            chrome_options.add_argument('--proxy-server=%s' % proxy)
            chrome_options.add_argument('headless')
            chrome = webdriver.Chrome(r'C:\chromedriver.exe', options=chrome_options)
            chrome.get('http://httpbin.org/ip')
            output = chrome.find_element_by_css_selector("body > pre").text
            print("Site output: {0}".format(output))
            print()
            chrome.quit()

on the 4th loop consistently, it seems to fail and give me a RuntimeError: Could not get external IP
Apologies if this is a moronic/obvious question, but it has had me stumped for quite awhile.

Instead of gathering proxies from ProxyBroker perhaps it would be more beneficial to use ProxyBroker's serve functionality, which will just serve you a single IP that you can use for anything. That single IP, which is ProxyBroker, will automatically switch between proxies when there is some sort of error on the response request. 👍

Instead of gathering proxies from ProxyBroker perhaps it would be more beneficial to use ProxyBroker's serve functionality, which will just serve you a single IP that you can use for anything. That single IP, which is ProxyBrokers, will automatically switch between proxies when there is some sort of error on the response request. 👍

hello:)
Thank you for your answser!
I would be very interested to know how to do that robustly.. open a selenium firefox browser with 1 IP, get to a page, close browser, open immediately a new browser with another ip...

Instead of gathering proxies from ProxyBroker perhaps it would be more beneficial to use ProxyBroker's serve functionality, which will just serve you a single IP that you can use for anything. That single IP, which is ProxyBrokers, will automatically switch between proxies when there is some sort of error on the response request. 👍

hello:)
Thank you for your answser!
I would be very interested to know how to do that robustly.. open a selenium firefox browser with 1 IP, get to a page, close browser, open immediately a new browser with another ip...

So... that's not quite what the serve functionality will do. Instead, it will utilize a single IP until that IP is unable to reach the website you are shooting requests at. At that point it is when ProxyBroker switches proxy.

oh ok!!! thanks I didn't get it.
so I'm gonna look at the common use of proxybroker..
thank you have a good day
Le lun. 23 déc. 2019 à 03:17, Felipe Faria<EMAIL_ADDRESS>a
écrit :

Instead of gathering proxies from ProxyBroker perhaps it would be more
beneficial to use ProxyBroker's serve functionality, which will just
serve you a single IP that you can use for anything. That single IP, which
is ProxyBrokers, will automatically switch between proxies when there is
some sort of error on the response request. 👍
hello:)
Thank you for your answser!
I would be very interested to know how to do that robustly.. open a
selenium firefox browser with 1 IP, get to a page, close browser, open
immediately a new browser with another ip...
So... that's not quite what the serve functionality will do. Instead, it
will utilize a single IP until that IP is unable to reach the website you
are shooting requests at. At that point it is when ProxyBroker switches
proxy.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/constverum/ProxyBroker/issues/94?email_source=notifications&email_token=AKST3BF4U6UDBZ5BCIH5FODQ2ANUJA5CNFSM4GAJZKQ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHQAR2Y#issuecomment-568330475,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKST3BH3ON4ZIPHX2ZUUF2LQ2ANUJANCNFSM4GAJZKQQ
.


Speed up compilation

A collection of self-contained, orthogonal commits. I could have filed 20 PRs, one per commit, or just one.
On my workstation, the compilation times w.r.t. master become

tti so12 : ~27s -> ~12s
smoothers : ~5.5s -> ~1s
elastic so12: ~18s  -> ~8.5s
viscoelastic so12 : ~41s -> ~14s
iso-acoustic so12 :  ~4s -> ~2.5s

@tjb900 I'm not sure if you care about this ... tagging you, just in case
@ggorman I've been working on this intermittently for a few days, but there is still work to do
@mloubout @rhodrin if you take a profile with Python's cProfile, you shall see there's a margin of improvement especially in the DSL/SymPy layers... wonder whether/when you could this give a go. Not a top priority of course!

Forgot to add: depends on the removal of OPS and YASK.

On my Azure workstation, the compilation times w.r.t. master become with Intel Compiler:
(DEVITO_ARCH=icc, DEVITO_OPENMP=1, 10 threads-ht off)
tti so12 : ~5s -> ~5s
elastic so12: ~12s -> ~12s
viscoelastic so12 : ~16s -> ~14s
iso-acoustic so12 : ~2s -> ~2s
Am I doing sth wrong?


Working with multiple losses and their weights in keras

Training a GAN model using train_on_batch with multiple losses, can I use random loss_weights while compiling a model or is there some specific strategy to use these loss weights as mentioned Here. In my problem, mean_sqaured_error is a loss function for generated_image and original_image and binary_crossentropy  is a classification loss function for 0 and 1 class.
model.compile(optimizer=optimizer, loss=['mean_squared_error', 'binary_crossentropy'], loss_weights=[100,1])

Closing due to lack of recent activity. Please update the issue when new information becomes available, and we will open a new issue. Thanks!


Query for block reports INTERNAL_SERVER_ERROR to browser

After querying various blocks (they happen to have plutus scripts in them - but that may not be the cause) the browser returns:
SORRY, WE COULD NOT FIND ANY RESULTS MATCHING:
9635fd19af660a4f7db984499cd3097304d43e79f7589cb7ca81a777454f104d
for blocks, ones that - a few moments earlier - it was able to show.
The browser JS console has the following contents that may help isolate the issue:

Uncaught (in promise) Error: Cannot return null for non-nullable field Token.asset.: {"response":{"errors":[{"message":"Cannot return null for non-nullable field Token.asset.","locations":[{"line":57,"column":5}],"path":["blocks",0,"transactions",0,"mint",0,"asset"],"extensions":{"code":"INTERNAL_SERVER_ERROR","exception":{"stacktrace":["Error: Cannot return null for non-nullable field Token.asset.","    at completeValue (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:560:13)","    at completeValueCatchingError (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:495:19)","    at resolveField (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:435:10)","    at executeFields (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:275:18)","    at collectAndExecuteSubfields (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:713:10)","    at completeObjectValue (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:703:10)","    at completeValue (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:591:12)","    at completeValue (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:557:21)","    at completeValueCatchingError (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:495:19)","    at /nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:618:25"]}}},{"message":"Cannot return null for non-nullable field Token.asset.","locations":[{"line":57,"column":5}],"path":["blocks",0,"transactions",1,"mint",0,"asset"],"extensions":{"code":"INTERNAL_SERVER_ERROR","exception":{"stacktrace":["Error: Cannot return null for non-nullable field Token.asset.","    at completeValue (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:560:13)","    at completeValueCatchingError (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:495:19)","    at resolveField (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:435:10)","    at executeFields (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:275:18)","    at collectAndExecuteSubfields (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:713:10)","    at completeObjectValue (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:703:10)","    at completeValue (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:591:12)","    at completeValue (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:557:21)","    at completeValueCatchingError (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:495:19)","    at /nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:618:25"]}}},{"message":"Cannot return null for non-nullable field Token.asset.","locations":[{"line":57,"column":5}],"path":["blocks",0,"transactions",2,"mint",0,"asset"],"extensions":{"code":"INTERNAL_SERVER_ERROR","exception":{"stacktrace":["Error: Cannot return null for non-nullable field Token.asset.","    at completeValue (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:560:13)","    at completeValueCatchingError (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:495:19)","    at resolveField (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:435:10)","    at executeFields (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:275:18)","    at collectAndExecuteSubfields (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:713:10)","    at completeObjectValue (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:703:10)","    at completeValue (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:591:12)","    at completeValue (/nix/store/iyhi9v4xg9xzijx8lvrkp0cn9kch0l7y-cardano-graphql-6.1.0/node_modules/graphql/execution/execute.js:557:21)","    at comple    at e.

It may be that the URL https://explorer.cardano.org/en/block?id=7e147a6ed3461d7f787eff98b495ab16313a989e95ba4fa4de103791e4531616 is sufficient to invoke this issue

Are you saying that you were able to query them with no issues and then it later fails?
Can you please post the entire query?
My guess is that if the block was a few seconds old, it could have been removed by the chain due to the network choosing the other fork which doesn't include this block.


Improved test coverage

Changes:

Added tests
Improved test coverage to 99%

Screenshot (from Travis build)

I'm gonna merge this since it shouldn't break the application. Feel free to review


ci: enable CodeQL code scanning

:tada: This PR is included in version 0.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Import instructions for 'lite' version refer to non-existant file.

Issue description

I believe that import { Easing, Interpolation, Tween, autoPlay } from 'es6-tween/src/index.lite' should end with .js., not .lite.

Related
This issue is related to: README
What you excepted?

I expected to be able to paste that line into my code and have access to the lite version of Tween. Instead, I had to change the file extension.

Also, what is the difference between importing { Easing, Tween, autoPlay } from index.js and from es6-tween? I found I had to change .onUpdate() to .on('update', ...) when importing the library.

Hi. Thanks for using our library.
But Lite was deprecated in v4.x.x

Also, what is the difference between importing { Easing, Tween, autoPlay } from index.js and from es6-tween? I found I had to change .onUpdate() to .on('update', ...) when importing the library.

Lite version has been optimized, memory effecient and faster until 4.x.x introducted

Oh, I see. I didn't realize I was using a different version than what was available over CDN.
Maybe the real bug here is "README isn't updated for the 4.x release"?

README for 4.x is updated in only GitHub, so please sorry for this. I will fix issue in few days

No worries! Let me know if you'd like any help.

I will making something really nice for my users like you (hope you'd like) and should optimize the performance under the hood by some memory cost

Look at v4.0.2, that should fix docs issue and much some new i few time ago said


Ensure header in front of other elements

Resolves #26
I spent a while trying to figure out what was wrong with select elements in particular. Couldn't figure out a more robust solution, but this seems to work.

oooops this should be a PR


Issue to trigger new releases

@JuliaRegistrator register branch=master
Release notes:

Add search options to measures(...) method

Registration pull request created: JuliaRegistries/General/84185
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.1 -m "<description of version>" be112a3b6f5a54fa079f366728f3d3b023e2fea5
git push origin v0.1.1

@JuliaRegistrator register branch=mastet

Registration pull request created: JuliaRegistries/General/89310
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.2 -m "<description of version>" 7feb8883e21d088098a9c89ec1d8e4794b740168
git push origin v0.1.2

@JuliaRegistrator register branch=master

Registration pull request created: JuliaRegistries/General/89807
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.3 -m "<description of version>" 3e486fd842811a911a9f6fde7c5017dbc390488b
git push origin v0.1.3

@JuliaRegistrator register branch=master

Registration pull request created: JuliaRegistries/General/99283
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.4 -m "<description of version>" 40bad359601edae1d2926395f4296d171d9398d1
git push origin v0.1.4

@JuliaRegistrator register branch=master

Registration pull request created: JuliaRegistries/General/101673
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.5 -m "<description of version>" c9ffa632d0b3f4819e185ca0f493ad4c1063c762
git push origin v0.1.5

@JuliaRegistrator register branch=master

Registration pull request created: JuliaRegistries/General/101751
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.6 -m "<description of version>" c484e07788e2f5752863434c292ba356288fc365
git push origin v0.1.6

@JuliaRegistrator register branch=master
Release notes:

balanced_accuracy can now be called on a confusion matrix (#28)

Registration pull request created: JuliaRegistries/General/115455
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.7 -m "<description of version>" c9a254f741a0bce37cd9a06c282a7cb39fd4d707
git push origin v0.1.7


Travis issue "package writes outside sandbox"

Travis can fail when I add ck as a dependency to my project with a message "package writes outside sandbox". Any ideas why?

Thanks for reporting @jmrtin72!
I think I know what is the problem: I copy the default CK repo (core CK modules) to $HOME/.ck directory during installation to help CK find it. But it seems to be "outside sandbox" for Travis.
I tried to simplify our installer in the latest PR and do not copy default CK repo to $HOME/.ck. I just uploaded the pre-release version <IP_ADDRESS> to PYPI - can you please check if it solves your problem?
Thanks!

@gfursin, the problem seems to disappear in <IP_ADDRESS>. thank.


[Development] Setup Server for Wdb

Implementation of

#19

Tasks

[ ] Explore server libraries that can be used - go-gin or fiber
[ ] Setup Basic server
[ ] Setup Initial routes for

databases
collections

https://mockoon.com/tutorials/create-api-golang-mocking/

Graceful Shutdown

https://medium.com/tokopedia-engineering/gracefully-shutdown-your-go-application-9e7d5c73b5ac
https://gist.github.com/aladhims/baea548df03be8f1a5f78a61636225f6


Websocket connection argument

Hi @bennyn, I have been running the library and so far so good. I would like to point out an issue I got today, though this is not a library issue, I think.
So, today websocket was trying to reconnect for more than 15 minutes and it couldn't. I restarted the program and it connected. Even though the Coinbase Pro documentation says that it is 4 seconds interval I think it prevents to reconnect at some point (if you keep reconnecting).
Don't you think it would be a good idea to provide an option to set a reconnection interval when websocket connects (in connect() method)?
Please let me know what do you think.

Hi @baloian, accepting reconnecting options in the connect() method is an excellent idea! I will add this in a few hours. This will also help @Slurpgoose's use case in bennyn/coinbase-pro-node#102. 😃

Great! please let me know when you will add it.

The changes are there!
You can now pass reconnection options to the connect method.
There are a few things to mention:

You can avoid automatic reconnects with options set to: {maxRetries: 0}
Calling connect twice (without calling disconnect inbetween) will cause an error
The connect method does not return a Promise anymore and instead returns you the instance of ReconnectingWebSocket directly, so you have to listen to the WebSocketEvent.ON_OPEN event to know if it is connected

All changes mentioned above are in coinbase-pro-node v1.6.0.
Please report if it solves your issue. ✏

@bennyn this is great thank you! I will close my issue as well.

@bennyn thank you very much, I will test it. For now I am closing the issue.


Module resolution error

With<EMAIL_ADDRESS>I run into the following error
./node_modules/orchid-orm/dist/index.mjs
The requested module 'orchid-core' contains conflicting star exports for the name 'templateLiteralToSQL' with the previous requested module 'pqb'

I already made sure I have the latest orchid-core (0.21.2) and pqb (0.42.7).
The error is also present in 1.37.9; I had to downgrade to 1.37.8 to fix this.

The latest is 1.37.10, did you try it?
templateLiteralToSQL failed in CI and I fixed and published it immediately a couple days ago.

Yes, I tried both 1.37.9 and 1.37.10

I can confirm, after upgrading to 1.37.10 I have the exact same issue.

Published a fix - simply republishing helped.
Thanks for reproduction.
Apparently, CI process isn't ideal and may leak failed builds (in rare cases, because usually it's acting normal).


ballerina help openapi xxx does not work

Description:
Since ballerina help dist xxx works, IMO this should also work.
Affected Versions:
1.0.5
1.1.0-rc1

This is not applicable now. So closing this issue.


Update all the package dependencies

Run a pub upgrade to update all of the dependencies to the latest version.

bors merge

LGTM! 🌮 Maybe we should add a dependabot script to create this kind of PR?

Oh yeah @alallema last time I looked about this, I saw an open issue saying dependabot does not support Dart yet, but checking it again, they released a beta a couple of weeks ago! I'll create an issue about this, thanks! :D


Documentation / implementation mismatch on "set_dbh" method

https://github.com/ledgersmb/PGObject-Simple-Role/blob/master/lib/PGObject/Simple/Role.pm#L74 says that the _dbh attribute should be set through the set_dbh() method, yet the object/role doesn't implement it.

1a191c2 has been reverted.  This issue is a combination of a misunderstanding of the code (obfuscated-ness, possibly) in combination with an old version being installed in local testing: the problem is that the set_dbh() method is being imported from PGObject::Simple and that I locally had 1.13.2 which doesn't have that method at all.


Added to populate eppn, sn and givenName from response while testing…

… with Shibboleth testshib.org IdP

While I think it's a nice feature that passport-saml parses email from the urn:oid, the verify function provides an option to do any additional parsing from identifiers to names of your choosing, thus I would not be in favor of supporting additional mappings here, it can be left up to the implementer.

Thanks Paul, I'm closing this PR.


Updated README.md

Added section for installation of the package.

Coverage remained the same at 94.444% when pulling fdc9c1765b2f0cb367441d8721a5e8773b6c9e9d on alstat:master into 429ddf8264da159d5cabdf412c9ddb96135cc19f on WizardMac:master.

Thanks!


state transfer fails with panic: Our blockchain is already higher than a sync target, this is unlikely, but unimplemented message

Scenario is

4 validating peers
security enabled
VP0-2 start at beginning of test,  VP3 manually started after VP0-2 have processed chaincode deploy and 5 TXs
after VP3 starts, invoke 5 more TXs. This will cause VP3 to initiate state transfer

Result:

VP3 initiates state transfer, tries to sync with VP1,VP2 but stops on panic message.

Attaching VP3 debug log:
vp3_err.txt

Have a fix pending, waiting for PR #677 to fix failing behave tests before submitting PR.


thrown error when accessing global

https://github.com/feross/buffer/blob/master/index.js#L43

This package is meant to be used with browserify which does define a global variable. As a workaround, you can set window.global = window before using this package in your code.
However, this line is going to be removed in the coming v5 release anyway. You can also just wait until then.


Custom filter in headerCellTemplate

I created a grid with custom external filter. It works perfectly fine, as long as it is not part of the headerCellTemplate. When it is inside:

Data is filtered correctly but the display is not adjusted
I didn't manage to apply the same formatting as for Grid Mene, i.e. the drop down overlapping the header and grid cells and not changing header height.

Please take a look at the Plunker http://plnkr.co/edit/z2Ua5CAypvjSNPb9Psjv?p=preview

this might be a duplicate of #2796

Issue #2796 is a feature request, my case (point 1.) is probably a bug (or hopefully my wrong understanding).
The bottom line is: I was able to create filter using AngularjJs mulit select and it works well as long as it is not part of  headerCellTemplate.

Changes were done on the header height recently that may have fixed point 1.

This is a duplicate of #2731.
@pszawlis see my comment there to understand how you can achieve the behaviour you want.

I think the new filterHeaderTemplate option resolves this: http://ui-grid.info/docs/#/tutorial/306_custom_filters


Add Polish translations

Description
This Feature will add Polish translations to the checkout.
closes #383

:tada: This issue has been resolved in version 4.5.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Stubbed out api calls

moved containers to models
updated functionality for roaster accounts
added stub methods for customer accounts and billing subscriptions.

@jonnykry i'll let you review and merge this :)


Map freezes after touch zoom if no marker is present on the map

Following example:

<!DOCTYPE html>

    <title>Leaflet mobile example</title>

    <meta name="viewport" content="width=device-width, initial-scale=1.0, maximum-scale=1.0, user-scalable=no">

    <link rel="stylesheet" href="./vendor/leaflet/leaflet.css" />

Tested with actual master version. Open this example in ios simulator and start zooming in by pressing alt. Everything is working fine. But when removing L.marker(e.latlng).addTo(map); from the code and start zooming in again as before the map completely freezes. I don't know excatly if this is related to #2693.

If the issue is still in latest master, a minimal jsfiddle would help: http://playground-leaflet.rhcloud.com/

Does this problem exist on 1.X at all?

Hi! I'm closing this since it doesn't seem to have been reproduced in a couple of years. If you still experience this issue, add an example and I'll re-open!


Filter the RPM > 20.

We will start with a "loose" filtering method.

Substitute the RPM with z-score?


Fix typo in manifest docs

Change 'BodyAsHtml' to 'BodyAsHTML' in the Rule element article.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference/manifest/rule.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Expand buffer size to avoid buffer overflow

I have increased the buffer size in the GetCompletePath function in Source/Graphics/textures.cpp to avoid a buffer overflow detected by SonarCloud (S5782)

Thanks for the patch, @DrCrinkle!


Add subcommand for linenumbers/code that only looks for codeblocks

Instead of linenumbers copying the full message, and format breaking when the message has a codeblock / multiple.
Allow them using a possible optional argument, to only look for codeblocks, a.e:

Hm, does that work?, or 'maybe this'?

// format command

   work?
   maybe this

Possible regex to find "big" code blocks: \{3}(.+)?\n?((?:.|\n)+)?`{3}`


tzdb::local_tz() function get None in linux, expect get Some value .

~ uname -a
~ Linux 3.10.x86_64 #1 SMP Wed Feb 23 16:47:03 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

~ timedatectl
      Local time: Tue 2022-08-09 17:46:20 CST
  Universal time: Tue 2022-08-09 09:46:20 UTC
        RTC time: Tue 2022-08-09 09:46:20
       Time zone: Asia/Shanghai (CST, +0800)
     NTP enabled: yes
NTP synchronized: yes
 RTC in local TZ: no
      DST active: n/a

"Asia/Shanghai" is contained in the shipped database, so the problem probably lies with getting the timezone name: iana_time_zone::get_timezone()
Linux 3.10 is ancient, but it should still be supported. Which distribution do you use. Can you show be the output of lsb_release -a?

~  lsb_release -a
LSB Version:	:core-4.1-amd64:core-4.1-noarch
Distributor ID:	CentOS
Description:	CentOS Linux release 7.9.2009 (Core)
Release:	7.9.2009
Codename:	Core
~

I could verify the problem on CentOS 7, and filed an issue at https://github.com/strawlab/iana-time-zone/issues/47.  Will work on it tomorrow if no one else does in the meantime. :)


add some API on readme.md

add some API on readme.md

merged

thank u

Considered reverting due to typo infromation -> information.
Should have double-checked...


Add Badge to list of apps

Hey there!
Made this bot in response to a Hacktoberfest issue: http://github.com/reactiveui/ReactiveUI/issues/1501
Didn't really know how to add a screenshot of what the bot does, though. :/

Hi @bkeepers, glad to see that you like the app! I do see the potential confusion that the name could cause, though. Maybe something more straightforward like org-inviter or invite?
By the way, did you give the talk on Probot during GitHub Universe? I actually stumbled upon the live stream midway through making the bot and it was pretty helpful. 😃

Suggestions also appreciated 😉.

invite-contributors? 🤷♂

By the way, did you give the talk on Probot during GitHub Universe? I actually stumbled upon the live stream midway through making the bot and it was pretty helpful. 😃

No, @jbjonesjr did! It was great.

Updated the repository name! You can find it at https://github.com/erickzhao/invite-contributors
:)
Cheers!


Cache profiling (WIP)

Still working on this, but here are some benchmarks for the cache classes.
Results so far are that the LRUCache is about 1.5-2x as fast as the LRUMap through all initial tries, and that the differences between LRUCache and LRUCacheWithDelete (or Map/MapWD) is small.
flat writes, 1x gentle read spread LRUCache x 27.93 ops/sec ±1.28% (50 runs sampled)
flat writes, 1x gentle read spread LRUCacheWithDelete x 24.34 ops/sec ±2.58% (44 runs sampled)
flat writes, 1x gentle read spread LRUMap x 12.61 ops/sec ±1.83% (36 runs sampled)
flat writes, 1x gentle read spread LRUMapWithDelete x 11.42 ops/sec ±4.76% (33 runs sampled)
flat writes, 1x gentle read spread LRUCache x 21.32 ops/sec ±1.21% (39 runs sampled)

Random-order reads (no writes) of 60,000 distinct values.
The top 30k of values occur ~70% of the time and the top 10k values 33% of the time.

individual get then set, sharp spread LRUCache x 2,192,917 ops/sec ±1.47% (92 runs sampled)
individual get then set, sharp spread LRUCacheWithDelete x 2,348,333 ops/sec ±1.19% (89 runs sampled)
individual get then set, sharp spread LRUMap x 1,560,643 ops/sec ±2.33% (80 runs sampled)
individual get then set, sharp spread LRUMapWithDelete x 1,408,341 ops/sec ±6.98% (70 runs sampled)
individual get then set, sharp spread LRUCache x 2,331,573 ops/sec ±1.57% (86 runs sampled)

individual get then set, flat spread LRUCache x 3,931,479 ops/sec ±1.92% (85 runs sampled)
individual get then set, flat spread LRUCacheWithDelete x 4,175,905 ops/sec ±1.28% (86 runs sampled)
individual get then set, flat spread LRUMap x 2,242,931 ops/sec ±3.71% (76 runs sampled)
individual get then set, flat spread LRUMapWithDelete x 2,541,944 ops/sec ±3.40% (79 runs sampled)
individual get then set, flat spread LRUCache x 3,748,182 ops/sec ±2.24% (84 runs sampled)

Pre-loaded 30k capacity caches, random-order reads (no writes) of 42,000 distinct values.
The top 30k of values occur ~97% of the time and the top 10k values 75% of the time.

read-only sharp spread LRUCache x 112 ops/sec ±1.28% (72 runs sampled)
read-only sharp spread LRUCacheWithDelete x 100 ops/sec ±1.62% (73 runs sampled)
read-only sharp spread LRUMap x 59.71 ops/sec ±2.01% (62 runs sampled)
read-only sharp spread LRUMapWithDelete x 59.43 ops/sec ±4.16% (61 runs sampled)
read-only sharp spread LRUCache x 102 ops/sec ±3.10% (75 runs sampled)

Pre-loaded 30k capacity caches, random-order reads (no writes) of 60,000 distinct values.
The top 30k of values occur ~70% of the time and the top 10k values 33% of the time.

read-only gentle spread LRUCache x 90.15 ops/sec ±2.23% (73 runs sampled)
read-only gentle spread LRUCacheWithDelete x 86.24 ops/sec ±1.25% (74 runs sampled)
read-only gentle spread LRUMap x 57.07 ops/sec ±1.77% (60 runs sampled)
read-only gentle spread LRUMapWithDelete x 62.46 ops/sec ±1.45% (65 runs sampled)
read-only gentle spread LRUCache x 94.03 ops/sec ±1.14% (76 runs sampled)

I have added the ubiquitous benchmark.js library to help power this if that's OK.
@Yomguithereal I fumbled together a method to give back a pareto-distributed random integer, or at least fake it well enough to serve this benchmark, but I suspect that you will have good advice on how to do it right. If this isn't built in to pandemonium it would be nice to have a few such distributions in the toolbag.

Hello @mrflip,
Have you checked this: https://github.com/dominictarr/bench-lru? Are you only trying to assess whether the cache is faster than the map and what is the impact of deletion here?
Note that the Map is sometimes, on some node versions, faster if you handle specific keys such as long strings etc.

If this isn't built in to pandemonium it would be nice to have a few such distributions in the toolbag.

Why not. What would the API looks like? Something taking a rng and some alpha params and returning a distributed rng? At some point it might start being too statistically complicated for pandemonium which is mostly about algorithms to fit another lib such as simple-statistics.

This is primarily in service of the later PR for TTK time-expiring the cache and knowing what the tradeoffs were on Cache vs Map, not so much on comparisons to others (I found out about this lib via that suite). Mixed in with that PR (I'll pull it back to here) is a shell script to run any of the benchmarks with the node profiler turned on.
I'll make the test exercise long strings, shortish strings and number keys.
Do you have any concerns on adding the dev dependency of benchmark.js? I'd also recommend adding the chai-js library -- it makes tests more explanatory and beautiful to read. I was surprised by how many more and deeper tests we wrote after adopting it. For just one example it becomes very pleasant to add type guards like expect(result).to.include.keys([...]), expect(arr).to.be.an('array').length(5), which give much clearer errors than the runtime exceptions when you access that wrong-typed return.


Add blurred rectangles

This is ported from the Floem fork.

Thanks!

Was there are reason to go with this gaussian blur approach over a box gradient? @Zoxc @dzhou121
See for example: https://github.com/memononen/nanovg/blob/f93799c078fa11ed61c078c65a53914c8782c00b/src/nanovg.h#L408

@wtholliday I just used the existing code. I suspect either would be fine though.


feat: workflow queries/actions rename code split

Description
This PR is code split PR for allowing rename of queries, actions and apis for workflows editor. This PR introduces update in the type of the query rename payload API.
PR fixes following issue(s)
Fixes #30963
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[ ] Manual
[ ] JUnit
[ ] Jest
[ ] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

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

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

Summary by CodeRabbit

New Features

Enhanced action and JavaScript object renaming capabilities with optional workflow association.

/ok-to-test<EMAIL_ADDRESS>@tag.IDE @tag.Datasource"


Directory/File is not found!

Hello there!:)
Just installed ClamAV server on my dedicated Windows 7 server and that is 100% properly installed.
However, when I test the application in my C# project using this:
`private void button1_Click(object sender, EventArgs e) {
OpenFileDialog open = new OpenFileDialog();
if (open.ShowDialog() == DialogResult.OK) {
            MessageBox.Show(open.FileName);

            ClamClient client = new ClamClient("<IP_ADDRESS>", 3310);
            //ClamScanResult res = client.ScanFileOnServer(open.FileName);
            ClamScanResult res = client.ScanFileOnServer("C:\\test.txt");

            switch (res.Result) {
                case ClamScanResults.Clean:
                    MessageBox.Show("Clean");
                    break;
                case ClamScanResults.VirusDetected:
                    MessageBox.Show("Virus found: " + res.InfectedFiles.First().VirusName);
                    break;
                case ClamScanResults.Error:
                    MessageBox.Show("Error: " + res.RawResult);
                    break;
            }

        }

    }`

I have a text file and the file exists on  C:\test.txt
However it keeps giving ClamScanResults.Error and saying:
"Error: \?\C:\test.txt lstat() failed: No such file or directory. ERROR
How can I fix this? The directory is correct and the file exists, however it keeps saying it does not. :(
Please help me out :'c

Is the app running on the same machine as the server?  Does the ClamAV service have permissions to the file?  Try to switch to SendAndScanFile and see if that works.

Changed to SendAndScanFile, works perfectly!
Sorry, did not see that method :(
Thanks for the help!


SSL Error api.godaddy.com doesn't match

raise SSLError(e, request=request)
requests.exceptions.SSLError: hostname 'api.godaddy.com' doesn't match either of 'developer.godaddy.com', 'www.developer.godaddy.com'

fixed it by upgrading a few packages on Ubuntu. http://stackoverflow.com/questions/29099404/ssl-insecureplatform-error-when-using-requests-package

Closing this as invalid, as it related to local package issues.


mitmproxy 0.15 breaks python 2 scripts

I have the following python2 script to run with mitmproxy:
print "This is an " + "error"

I use the following bash command:
mitmdump -q --anticache -s ../script.py

mitmproxy version 0.14 correctly prints out the string.
mitmproxy version 0.15 gives me:
mitmdump: Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/libmproxy/script/script.py", line 65, in load
    execfile(self.args[0], self.ns, self.ns)
  File "../script.py", line 1
    print "This is an " + "error"
                      ^
SyntaxError: invalid syntax

Thanks for the report. This happend because we had a
from __future__ import print_function

statement in libmproxy/script/script.py, which was transitively applied to your inline script. Adding this to your script as well (and then using print() should work fine as a workaround.


Adding required files for the repo in Hyperledger

Hey @amanji - some cleanup of the root folder to add things that Hyperledger needs to see.  Will be adding a task for a person from our team that can help with the transfer.
Please review and merge (and let me know if you see any problems). Note that I did add some .gitignore and .gitattribute commands so we should try a post merge test :-)
Thanks!

Thank you! Let me know if there's anything I need to do on my end to facilitate the transfer.


add MatchData#pre_match and #post_match and document MatchData

I don't have a lot of experience with unicode so hopefully those parts are OK.
While working on this I realized for the first time that Crystal has Regex whereas Ruby has Regexp. I appreciate this, because when spoken aloud in English, usually the P is either not pronounced, or spat out like a :watermelon: seed: "Reg Ex Puh".
It seems like #begin, etc, could accept 0 as a default argument, to reflect that 0 references a "capture group" that is created implicitly for the entire regex. Ruby doesn't do this, and I have wanted it once or twice over the years. Would this be a welcome change?

@bjmllr Thanks!! :heart::yellow_heart::blue_heart::purple_heart::green_heart:
Pull requests with new functionality and bonus documentation are the best :-)
I'll just change the implementation to use byte_begin and byte_end which should be faster (no need to compute the character index from the byte index).
I don't remember why we decided to go with Regex instead of Regexp: I think it was because it's easier to type and pronounce, and it isn't a big deal because most of the time you create them with literals.
The 0 as a default argument also makes sense to me, MatchData#begin would return where the entire match begins, right? Sounds reasonable.

Thanks for merging! I didn't even think of byte_slice, that's much better.


updateSearchParameters doesn't update the search query

We have our own search input because we want our own placeholder. When we try to update the search parameters it doesn't pick up what we pass. It's always a '*'.
automagic.store.updateSearchParameters({ search: searchValue });
How can we update the search query ourselves?

Try:
automagic.store.setInput(input_text)
automagic.store.search();
input_text: text from your inputsearchbox


create openid connect "sub" column for migrating Google Accounts

You'll need to add this column to the table in warehouse/legacy/tables.py as well.

The index too.

@dstufft is that correct?

Yup.


All fromXXX should take a zone

Current offenders are fromJSDate and fromMilliseconds

This is done but needs tests.


Highlight identifiers within template literals please

See the identifiers in ${lang} and ${i18nDir} highlighted apart from the rest of the literal content. I'd love to see that in this syntax as used by GitHub please!
Expected
Screenshot from Sublime Text 3

with inbuilt JavaScript syntax, no packages installed that are known to affect this (I might be wrong)

Actual
Code
const locales = ['en-gb', 'de-de'];
locales.forEach((lang) => {
    const loaderId = `angular-i18n-loader_${lang}`;
    i18nFiles[lang] = `<%= config.paths.bowerDir %>/${i18nDir}/angular-locale_${lang}.js`;
    concatLocales[`${lang}-locale`] = {
        src: [
            `<%= config.paths.i18nFiles["${lang}"] %>`,
            '<%= config.paths.srcScriptsDir %>/start-app.js'
        ],
        dest: `<%= config.paths.compiledScriptsDir %>/<%= config.paths.i18nDir %>/${loaderId}.js`
    };
    i18nLoaders.push(`

    `);
});

Screenshot from above code block at time of submission (taken from Preview)

I've just tested in Lightshow and I can see a very slight blue colour on the strings, quoted ' ' and templated ` `, whilst the ${} bits are black, but that's not showing here on GitHub.
Test link here

You seem to be interested in Sublime Text syntax highlighting for Javascript however this repository provides support for the J language.
Are you sure you're commenting on the correct repository?

Ack! Sorry! Thanks for letting me know =)


Queue problem

I've requested forced photometry [Task 1227354], but it's taking too long (for every task I request). This problem also happened yesterday with a task I've canceled. Is there any problem with the server?

Hello, thanks for the notice. The task runner was crashing when trying to send emails (SSL certificate problem on the new server), which prevented the jobs from completing. I've fixed the crashing and task 1227354 has now completed. Please re-open the issue if you continue to see problems.


Multli-line annotations for text files need page refresh to display correctly

Steps to repro:

Start marking an assignment with a text file
Select some text and create a new annotation with the following content (select an annotation category other than 1 time)

here is some content

I've added some new lines

it seems that new lines break things

not quite sure what yet. I'm also ensuring that this gets truncated in the annotation list.

Create the annotation
Notice that the selected lines do not get highlighted
Refresh the page
The lines are now highlighted and the annotation is working correctly

Additionally, this also occurs when the multiline annotation is selected from the predefined list (the reason the steps above suggest selecting an annotation category).

I think I've figured it out. Should be fixed upon mathjax integration.

Seems fixed now.


feat: Marquee effect in testimonial section

Title and Issue number

Title : Testimonial section into infinite Marquee
Issue No. : #743
Code Stack :
Close #743

Description

Video/Screenshots (mandatory)

https://github.com/apu52/Travel_Website/assets/103347688/116b50c8-da7e-43d9-9b44-49ef8c4799c9
Type of PR

[ ] Bug fix
[X] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Checklist:

[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 tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have gone through the  contributing.md file before contributing

Additional context:

Please add the level label as well in the issue and PR. Thank you.
##Are you contributing under any Open-source programme?

GSSOC'24

@apu52 please check the PR and also add the level label on the PR and its related issue.


Consider switching to MaybeUninit<[T; N]> rather than [MaybeUninit<T>; N] before you get too far along with this.

Just based on my experience writing staticvec, [MaybeUninit<T>; N] has no real tangible advantages over MaybeUninit<[T; N]> for a crate that is already (necessarily) making heavy use of unsafe code internally and is specifically based around const generics.
I started out using [MaybeUninit<T>; N] as the internal array for StaticVec, but switched to MaybeUninit<[T; N]> after about a month or so as it's just significantly more ergonomic to work with in every way.
For example, if using MaybeUninit<[T; N]>, your current from_maybe_uninit_array could be rewritten to simply take a normal array with the only code being MaybeUninit::new(array).

Hey, thanks for the suggestion!
Yeah, using MaybeUninit<[T; N]> would definitely simplify from_maybe_uninit_array, although I'm hopeful that transmute will be capable of doing this in the future. I think it might at the same time make get and get_mut more complicated, though — they're currently implemented by calling SliceIndex::get_unchecked[_mut] on the underlying [MaybeUninit<T>; N]. If I use MaybeUninit<[T; N]>, I think I'd first have to convert it to a [MaybeUninit<T>; N] before indexing it. Wouldn't that just move the problem?
Honestly, the whole reason this crate exists is to get rid of the boilerplate that comes with either of [MaybeUninit<T>; N] and MaybeUninit<[T; N]>. It started when I tried to see if rewriting staticvec to use [MaybeUninit<T>; N] would simplify things, and that attempt made clear that neither is very elegant.
One of the main reasons for choosing this representation is to be able to use exisitng slice APIs. For instance, std::mem::MaybeUninit now has some APIs that work with arrays/slices, like uninit_array and slice_get_[ref, mut]. And if MaybeUninit ever becomes unconditionally Copy, ConstBuffer::copy_within will be able to just call copy_within on the inner array.
"Before I get too far along with this" isn't really something I worry about, because I don't think the API surface of this crate will grow significantly from this point. And further abstractions like vectors and ring buffers would just use ConstBuffer without relying on the exact representation.

Reasonable enough! Just thought I'd give my two cents, haha.

Reasonable enough! Just thought I'd give my two cents, haha.