Segmentation fault at random intervals when extracting pictures

I'm attempting to use taglib-ruby to extract metadata from my large collection of flac files and it's segfaulting after reading "some" of them (where "some" varies randomly across runs between 100 and 6000+).
I've reproduced the issue with a single file - I expect this will fail for any flac with embedded art (but I can send you the offending flac if you want).
require 'taglib'

def trigger(file)
    TagLib::FLAC::File.open(file) do |f|
        f.picture_list.each do |p|
            x = p.data
        end
    end
end

10000.times do |i|
    trigger("/home/james/test.orig.flac")
    if (i % 100 == 0) then puts "iteration #{i}" end
end

it will also fail if you replace "x = p.data" with just "p.data" but consistently run to completion if you comment that line out entirely (in other words don't attempt to access the picture data at all).
I'm getting this on ruby 2.5.1p57 and 2.5.5p257 on Ubuntu 18.04/19.04.  The gem is up to date.
I've not got much experience debugging at the segfault level but if you think I can provide any further useful info please ask.  Thanks.

I've just tested with a flac file from Archive.org and had the same problem - hopefully that makes it easier to reproduce.  The first time I ran with this file all 10,000 iterations completed, but subsequent test runs failed partway through.
https://archive.org/download/78_you-are-my-sunshine_paul-ric...sticker-rice-brothers-gang_gbia0000125a/You Are My Sunshine - Paul Ric...sticker.flac
My 2.5.5 Ruby was running on a Ubuntu 19.04 VM inside Virtualbox (the 2.5.1 runs on bare metal).
I also changed my test code a little:
require 'taglib'

def trigger(file)
    TagLib::FLAC::File.open(file) do |f|
        #f.picture_list.each do |p|
            #p.data
        #end
        f.xiph_comment.field_list_map.each do |tag,value|
            tag
            value
        end
    end
end

100000.times do |i|
    trigger("/home/james/sunshine.flac")
    if (i % 100 == 0) then puts "iteration #{i}" end
end

(note 10x more iterations this time)
The code above has run OK (twice) but if I uncomment the f.picture_list... block it will fail.  It appears to fail faster if I also uncomment the p.data line inside the block.
I've had a look at the code and - bearing in mind I really don't understand SWIG - it looks as if the code that's present to free / unlink variables / objects for Vorbis comments etc when the FLAC file is closed might not be there for the picture objects..
Hope this helps.

it looks as if the code that's present to free / unlink variables / objects for Vorbis comments etc when the FLAC file is closed might not be there for the picture objects.

That does sound suspicious. The picture list appears to be part of the "xiph comment" so you'd expect objects to get unlinked here: https://github.com/robinst/taglib-ruby/blob/86567b7b689cdf276264552a5dd10476ef5c33cc/ext/taglib_flac/taglib_flac.i#L78-L81

I've forked and made some changes here https://github.com/jameswyper/taglib-ruby/tree/issue-91-fix-memory-leak-flac-picture
It would be great if someone could review them as it's my first time doing this sort of thing.  In summary I have:

Copied the flac.flac file in test/data to flac_nopic.flac and run metaflac --remove --block-type=PICTURE flac_nopic.flac to get rid of the picture
Added new test flac_picture_memory_test - this runs tests on both flac files (I wanted to be sure that my changes didn't break anything if a picture wasn't embedded).  This new test fails on release 1.0.0
Changed taglib_flac.i to unlink / remove tracking for each entry in the picture list (at least, that's what I hope I've done)
(I had also messed around with flac_create.cpp but was getting nowhere so reverted my changes)
Built and run a successful rake test

If everything looks OK I'll raise a pull request.  Thanks.

It's usually easier to just open a PR and make clear it's not for merging yet. Now we have to click around just to see what changed on your branch.
I think it's this: https://github.com/robinst/taglib-ruby/compare/master...jameswyper:issue-91-fix-memory-leak-flac-picture

It looks reasonable. I don't see why this can't be submitted as a PR.

Fix released in 1.0.1!: https://github.com/robinst/taglib-ruby/blob/master/CHANGELOG.md#101---2020-03-25


Gcc error

For GCC compiler, there should be a pair of parenthesis around the type
For example:
(size_t)(-1) instead of size_t(-1)

Hi, I've noticed your feedback.
So, is compile error occurred when using INFLUX_* macros?

I've fixed it.
Thank you for your feedback again.
Look forward to your further suggestions.


Fix tapret PSBT commitments. Closes #23

Alternative to #22

Hi @dr-orlovsky,
I found two issues in my tests. See below:

convolve_commitalways returns PsbtCommitError::TapTreeError: This occurs because the method adds a HiddenNode when reconstructing the TapTree. The Taptree::from_builder causes a IncompleteTapTree::HiddenParts error.

The embed commit does not use the PSBT_OUT_TAPRET_HOST: the method TaprootScriptTree::embed_commit define the DfsPath as Last instead of uses the value of the PSBT_OUT_TAPRET_HOST.

I made tests, I can transfer and spend UTXO, without "provably unspendable"

@dr-orlovsky,
I found strange behavior in my tests.
I tried to execute rgb-cli finalize  with a PSBT file with key-path output (and without taptree defined in the PSBT). The method returns success, but I think is a wrong behavior, because the new output key generated in the UntweakedPublicKey::convolve_commit does not use the taptree with the commitment script.
I did try to use Output::verify to prevent it but does not work too, I think this scenario causes a non-handle case.
Could you confirm if tapret need always a taptree?
If always need taptree, I will add the TaprootBuilder::is_finalized inside the XOnlyPublicKey::convolve_commit.
Thanks

@crisdut Yes, tapret always need a taptree. If there is no taptree (i.e. no script spending paths) it is created with just a tapret commitment (actually two copies, on the  left and right leaf of the merkle root, since the commitment should always be on the first level below the merkle root).

Like I did in this early prototype concept: https://github.com/BP-WG/stingerjet-docs
See the end of the README, the tapret final section

I think I have found a reason of the bug. Please see my two last commits - https://github.com/BP-WG/bp-core/pull/24/commits/559b04b4564559e0be2e85784e74994308388817 and https://github.com/BP-WG/bp-core/pull/24/commits/ee0792f40e8dbd1847dcb6a44751899761409fa3

I think I have found a reason of the bug. Please see my two last commits - 559b04b and ee0792f

Good!
I will start some regression tests here.

@crisdut this is ready for the review. The code needs much more tests, but I think we can start with merging this bugfix, do a release and than add more tests.


Add support for drone.io

Description
Drone.io support built-in
Add a .drone.yml file that allows projects to be tested/built using drone.io
Rationale
There are options for several areas of the project. Why not add some more flavor to the CI/CD options?
Drone  is very nice, from what I have tested. Also, it is open source, unlike most solutions out there. This makes it a candidate for baseline support.
Use case(s) / visualization(s)
Same as travis, actually. Adding some more options to people who uses other tools. This could be selected at install time.

Sure, feel free to send a pull request. I've heard of it, but I have no experience with this CI, I hope others may help to get the change reviewed... It sounds like it would be all in one config file so I don't mind having this.

Yes, it just requires adding a single file to the repo. The installer should ask for the option (travis, drone)? Or just add them both?

Currently it asks for Travis Y/N so this could be altered to choose a CI: Travis/Drone/None

or travis/drone/gitlab...

Since the project is using GitHub Actions now is this still relevant?

Since the project is using GitHub Actions now is this still relevant?

Probably, since ppl might prefer to use existing ci services instead of GA.

This is still relavant, the template uses GitHub action, but the generated project might be using a totally different CI. We support a few at the moment:
https://github.com/cookiecutter/cookiecutter-django/blob/3bca38c78d77ecdd9b1ae9af52b373c4277a9c1a/cookiecutter.json#L57-L62
I'm open to add this to the existing option, assuming it's still a single file. Caveat: I've never used drone, so I can't do it... @cdvv7788 @shywn-mrk @step21 are you interested to contribute?

@browniebroke Yes I'm willing to contribute. Where can I start?

I would recommend reading our contributing guide which explains the specifics of this project.
You can also check the GitHub docs themselves you need some help to use GitHublike for creating a fork or sending a pull request.
Here are the high level changes:

Add the drone config files in the {{cookiecutter.project_slug}} directory
Add a new option for drone CI in cookiecutter.json:
https://github.com/cookiecutter/cookiecutter-django/blob/8d64f1749461310b04a2b512184bb0623fb22ba0/cookiecutter.json#L42
Add the new option to the SUPPORTED_COMBINATIONS in tests:
https://github.com/cookiecutter/cookiecutter-django/blob/8d64f1749461310b04a2b512184bb0623fb22ba0/tests/test_cookiecutter_generation.py#L45
Make sure the drone config file(s) is/are deleted if another option is selected. We usually do that in the post-generation hooks
Update documentation

Hope that helps


Problem with 255+ LEDs

I'm trying to use your code for my 600 LED strip and when I set the # of pixels to over 255, it hangs someplace, and not sure where.  Any clue? I'm thinking you're somehow setting the # of pixels >255 and that's causing the hang.

hey! sorry for the delay in answering. glad you found my code; i hope it helps you. my first guess at the problem is that i am using uint8_t everywhere as an iterator, which would wrap around to 0 after 255 and so would never be able to satisfy the condition to exit the loops at 600. you might try changing those to uint16_t instead.
a 600 pixel strip is impressive! i hope you have a beefy Arduino and power supply :)


Review redis & celery caching additions to codebase

Lots of additions have been made to the code since we last worked on this - this ticket just notes that these need reviewing and the linter errors we are seeing need addressing

Duplicate of #292


real robot data collection

Do you have the scripts to collect the demos on the real robot?

The data collection on real robots are based on the real robot infra: https://github.com/UT-Austin-RPL/deoxys_control, along with a tailored vision package: https://github.com/UT-Austin-RPL/deoxys_vision .


Do not use password for sentinel

After update to 2.0 it's not possible to connect to sentinel server that worked fine with 1.1.10. I went over
#783 and #782 fixed unit tests there and I'm opened up if something more needs to be created (maybe adding some new option rather to use sentinel auth or no)

We need a test that shows Sentinel works with and without password authentication, no?

So I would need t sentinel on github actions with different auth and run tests against config with password set on sentinel and without password? Do U have already something that can save me some time at least? Did I understand this well?

Or just Mock it. Right now you're changing an existing test case, which is too risky.

My problem is that test that I've changed doesn't really proove anything, just that password is still there and that doesn't actually work on real sentinel setup, so test itself is probably wrong, maybe it works on different version/configuration of sentinel....

I'm closing this since with this options it works... maybe it would be cool to add that to docs because is not really clear...
If there's no password on sentinels then you need to set password in url as empty string, otherwise it will use default redis password for sentinels and fail. This is breaking change from 1.1.10 to 2.0.0
$sentinels = array(
    'tcp://sentinel1:26379?password=',
    'tcp://sentinel2:26379?password=',
    'tcp://sentinel3:26379?password=',
);

$client = new Predis\Client($sentinels, array(
    'replication' => 'sentinel',
    'service' => 'mymaster',
    'parameters' => ['database'=> 0, 'password' => 'redis-password']
));

We are running around 4 sentinel replication cluster in production for over three years now.
And we had never ever bevor to auth with the sentinel. Version 2.0.0 of Predis is breaking everything and this workaround doesn't feel good to mee. Upvote for a merge.

@rmertins I'll gladly accept any PR with tests.


Fix assertions for Bonfire: Symmetric Difference

fix #2797

This is so much more pleasurable than my #2770. Thank you!
Tested in latest Firefox and Chromium; merging.


Bug: Stream starts before resolution change fully occurs, leading to incorrect aspect ratio

I'm using gamestream launchpad to stream for a retropie set up, leveraging the moonlight package that is built in to it's experimental packages. One issue I've found is that when selecting the playnite bat through moonlight to start streaming, the stream seems to occur before the resolution change fully happens? It leads to a 1080p image, but a 21:9 aspect ratio (which is my hosts aspect ratio). I can solve this by quitting the stream quickly, and restarting, but often times that leads to me ended up with a correct resolution+ratio of just my desktop, and no longer Playnite.
Any ideas?

I think this same problem is affecting me. Mi host is at 4K, and my client at 1366x768.
After stream starts, you can see the resolution change, and then the mouse pointer looks too big.

I think this same problem is affecting me. Mi host is at 4K, and my client at 1366x768.
After stream starts, you can see the resolution change, and then the mouse pointer looks too big.


kube-image-keeper-registry-0 goes into a crash back-off when using AWS EBS persistent storage.

Sometimes when the kube-image-keeper-registry-0 pod is restarted it goes into a crash back-off with something like this at the end of its logs
garbage-collector docker.elastic.co/beats/filebeat: marking blob sha256:89732bc7504122601f40269fc9ddfb70982e633ea9caf641ae45736f2846b004                                          │
garbage-collector docker.io/jgraph/drawio                                                                                │
garbage-collector manifest eligible for deletion: sha256:fb2a84c7a2e04d4ea2e5aa0c57385e0e61dd3c7c5ea559a09d5a3a2cca6de28f

I haven't found any errors in the logs, but it always ends with "garbage-collector manifest eligible for deletion"
My workaround is to delete the PVC and then restart the pod again. So there must be something on the volume that breaks it.
We are deploying it using the kube-image-keeper helm chart from https://charts.enix.io/ v1.4.0
It is hosted in EKS with k8s version 1.24. And uses the following stroage class:
allowVolumeExpansion: true
apiVersion: storage.k8s.io/v1
kind: StorageClass
metadata:
  annotations:
    storageclass.kubernetes.io/is-default-class: "false"
  name: encrypted-ebs
parameters:
  csi.storage.k8s.io/fstype: ext4
  encrypted: "true"
  type: gp3
provisioner: ebs.csi.aws.com
reclaimPolicy: Delete
volumeBindingMode: WaitForFirstConsumer

Any help would be greatly appreciated

Hello @mccullough-ea , can you please test again with our last beta release (1.5.0-beta.1, published yesterday) OR update the deployment and set registry.persistence.deleteUntagged helm value to false ?

Thanks for the suggestion, 1.50-beta.1 seems have fixed it! I'll keep testing just in case I've got lucky..

@Nicolasgouze Thanks again for the suggestion, I can't seem to break it any more no matter how hard I try! Closing issue..


Exchange/Transfer Button for trials of import

The exchange button should allow to transfer an event record from one platform to another.
For the SMW compatible platforms WikiSon and the MediaWiki API is used (with py-3rdpartywiki) and for other platforms it depends on avaialability (e.g. Wikidata using pywikibot).
confref might be accessible via API ...

see https://wiki.tib.eu/jira/browse/CONFIDENT-240


Food channel (hear me out)

Is your feature request related to a problem? Please describe.
Not really. There are a few food pictures each day in general, and I think it would be nice to make another channel in keyboard warriors to contain them. IF THIS IS TOO MANY CHANNELS, I propose merging of linus-1 and linus-2 into just linus. That way, the number of channels would stay the same.
Describe the solution you'd like
Make #food, optionally consolidate #linus-1 and #linus-2
Describe alternatives you've considered
I mean, you could just not, I'm not forcing you to do anything
Additional context
https://cdn.discordapp.com/attachments/411573884597436416/811596116407484416/IMG_20210217_135244.jpg

You could also delete america which is stale

I would like this.

Alternatively, turn america into food

food where? 👀

Food where?

https://cdn.discordapp.com/attachments/411573884597436416/811596116407484416/IMG_20210217_135244.jpg

Can you even classify this as food though?

@CyberDiscovery/server-moderator-team @CyberDiscovery/server-admin-team Any progress on this?

IMO there's already too many channels, maybe delete the server and remake it with just the food channel?

Open this.


Request for Beta Bittrex support.

Hello, i hope you can add support to Beta Bittrex please.
https://beta.bittrex.com

Thanks mate, Bittrex Beta is up as live now. I hope you can add support soon


Add "if"

Change "Even users close" to "Even if users close".

@rbrayb : Thanks for your contribution! The author, @davidmu1, has been notified to review your proposed change.

Confirmed editorial correction. Thanks!


Add Change Password script for custom databases

Changes
WIP
Testing

[x] This change adds unit test coverage
[x] This change has been tested on the latest version of WordPress

Checklist

[x] All existing and new tests complete without errors
[x] All code quality tools/guidelines in the Contribution guide have been run/followed
[x] All active GitHub CI checks have passed

Codecov Report

Merging #605 into master will increase coverage by 0.6%.
The diff coverage is 92.85%.

@@             Coverage Diff             @@
##             master     #605     +/-   ##
===========================================
+ Coverage     34.02%   34.62%   +0.6%
- Complexity     1317     1321      +4
===========================================
  Files            54       54
  Lines          4174     4182      +8
===========================================
+ Hits           1420     1448     +28
+ Misses         2754     2734     -20

Impacted Files
Coverage Δ
Complexity Δ

lib/WP_Auth0_Api_Operations.php
51.92% <50%> (-0.08%)
15 <0> (ø)

lib/WP_Auth0_Routes.php
91.39% <96.87%> (+14.83%)
60 <16> (+4)
:arrow_up:

Continue to review full report at Codecov.

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

Looked into this and with the way that we're doing custom databases (importing users), this script is not required. Will trim all of the password endpoint work out and re-submit.


I can't enter mods menu

Describe your bug here.
I was modding and then this error pop up

Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No

broken pack.json file

oh thanks


[BAZEL-6] Add coverage support for JS files

The snapshot updates make sense but any idea why the binding test returns a different value?

I'm guessing it has to do with the update to node@18 -- changes to how the Error.message is serialized


iOS Clients will not be able to receive >1 call unless they close and reopen the app

Currently, when an iOS client ends a call or declines a call, it will disconnect from the server and be removed from the list of clients

see: https://github.com/vcuse/Neurology-iOS-Client/issues/19


KAFKA-1215: Merge master and fix conflicts

@allenxwang, I merged master and fixed a minor conflict.


MyST plugin

Build mdformat-myst plugin.
Come up with a set of parser and renderer extensions to be able to format MyST. Many of the syntax extensions make sense outside the MyST context, so they should be self-contained plugins that mdformat-myst requires and activates, similar to how mdformat-gfm does with mdformat-tables.
Syntax extensions

[x] Front matter (mdformat-frontmatter)
[x] Tables (mdformat-tables)
[x] Directives (This works even with no changes implemented. The blocks will be parsed as if they were code blocks, which means that the block content will be left unchanged. This is fine. Special parsing/formatting logic can be added though if desired.)
[ ] Math (both block level and inline) (parser in mdit-py-plugins/dollarmath)
[ ] LineComment (parser in mdit-py-plugins/myst-blocks)
[ ] BlockBreak (parser in mdit-py-plugins/myst-blocks)
[x] Footnote (mdformat-footnote)
[ ] Admonitions EDIT: this is disabled by default in myst-parser
[ ] Role (parser in mdit-py-plugins/myst-role)
[ ] Target (parser in mdit-py-plugins/myst-blocks)

Links

This seems to be the place where myst-parser itself prepares a MarkdownIt instance for myst parsing. We should set the same extensions in myst_plugin.update_mdit(). Also see this place where myst-parser sets MarkdownIt options.

Started implementing in https://github.com/hukkinj1/mdformat-myst/

@chrisjsewell do you know the story behind these .disable("footnote_inline") and .disable("footnote_tail")? Is special handling something an mdformat plugin would need to replicate?

Well footnote_inline does not have a built-in equivalent in the sphinx AST and footnote_tail is just a "post-processing" step that moves all the footnote tokens to the end of the stream, which is not necessary for myst-parser


Enso LoginController extending Local LoginController - unable to customize attepmtLogin()

This is  a possible feature request.

Prerequisites

[N] Are you running the latest version?
[Y] Are you reporting to the correct repository?
[N] Did you check the documentation?
[Y] Did you perform a cursory search?

Description
I'm trying to extend the Auth middle-ware with an additional check similar to "is_active" user flag.
In theory I'd like to extend the implementation of attemptLogin method from lbdash/vendor/laravel-enso/core/src/app/Http/Controllers/Auth/LoginController.php with an additional check similar to
if ($user->isDisabled()) {
            throw new AuthenticationException(__(
                'Your account has been disabled. Please contact the administrator'
            ));
        }

additionally I need a new local middle-ware similar to LaravelEnso\Core\app\Http\Middleware\VerifyActiveState...

Well, because the lbdash/vendor/laravel-enso/core/src/app/Http/Controllers/Auth/LoginController.php extends "Controller" local App\Http\Controllers\Auth\LoginController as Controller, I cannot override the method attemptLogin() since it will be further overridden by Enso's implementation.
I don't see an elegant solution at the moment.
I can only think of using bindings and bind the EnsoLoginController to another local LoginController (App\Http\Controllers\Auth\OverrideEnsoLoginController) that will not override attemptLogin(remove implementation) and used it from my local App\Http\Controllers\Auth\LoginController
Any ideas?

I can only think of using bindings and bind the EnsoLoginController to another local LoginController (App\Http\Controllers\Auth\OverrideEnsoLoginController) that will not override attemptLogin(remove implementation) and used it from my local App\Http\Controllers\Auth\LoginController
I guess you can bind it directly to the existing local LoginController
Or you can override locally the specific route for the action you need to customize.

Thanks


add Wordpress.com to the app list

So it looks like Wordpress.com.app is made using Electron
https://developer.wordpress.com/calypso/

Nice! We should wait until we hear from them though, don't want to post things unless people OK it themselves :+1:

This has been added in #129.


Hey there

I am just wondering where in your code one could change how fast the stairs llight up? Hope you still read these messages here!
Thanks so much for the awesome project as well, I had really hoped to not have to dive into the coding from scratch, and yours was by far the best out of all the different ones I tried!

Different effects require different speeds using effectInterval.
There are different random effects so stairs light up in the "stage_grow" section of each effect function.  The speed is set in the "stage_init" for each effect.  Look at lines:
225: effectInterval = 5;
394: effectInterval = 27;
463: effectInterval = 5;

Duplicate question.  I'm not sending gui photos of the code.  Use the code line counter to find what to adjust.    Answered again https://github.com/dmonty2/ArduinoMotionStairLights/issues/9


Description of "native" function is inadequate.

The entire description of the "native" function is "Returns myname."
That is not at all helpful.

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

ID: c75fe219-1699-a208-f101-4cf493950cea
Version Independent ID: 92f865c3-405b-59c7-ce24-14342ee8583b
Content: path Class
Content Source: docs/standard-library/path-class.md
Product: visual-cpp
Technology: cpp-standard-libraries
GitHub Login: @TylerMSFT
Microsoft Alias: twhitney

@RHolton, agreed. I added some color and a code example to show the difference (on Windows at least) between the native and generic path string.


Fixed-layout font-size affected by device settings

Bug Report
One issue that we’re experiencing concerns font-size for EPUB3 fixed-layout titles in Android. For titles opened in Android, the text font may be enlarged if the OS settings for text size are set above ‘Normal’.
Are the Readium team aware of this issue? Is it possible to restrict the OS access to font-size in Readium when the EPUB3 file is rendition:layout pre-paginated?
Attached screenshots:
Normal, when device text size is set to 'Normal'
Extra large, when device text size is set higher than normal.

Readium version:

Development environment

OS: Huawei P30
App: Adilko
Android version: 9

Thanks for reporting this issue, I opened a twin issue on Readium CSS to discuss this in more depth: https://github.com/readium/readium-css/issues/115


MantraJS-atom vs Mantra-CLI

Hi,
It appears that MantraJS-atom and Mantra-CLI are in disagreement about a few things.

Mantra-atom creates index.js, while mantra-cli creates main.js in server and client folders.
Using both Mantra-atom and Mantra-CLI therefore causes duplication errors for methods and publications in meteor server.

Mantra-atom requires ./configs/config in client/index.js (which it doesn't create and causes unable to resolve type errors), whereas Mantra CLI creates and requires ./configs/context

Mantra-atom adds file routes.js to core-module. Mantra CLI does not.

There might be other incongruencies.
Please advise on what we should do. :)
PS. Thanks for two awesome tools. My life has improved <3

Maybe the atom-package could use mantrajs-cli under the hood?
Can't say much about the 3 disagreements though. I think the mantra-sample application is the best guideline here

If you read docs, the mantra atom package is fully configurable via yaml file, whatever you do not like can be changed with no hassle. You can change file names, file contents, directories, even add or reorganise panes. This is not possible with cli package. If you feel that these changes would benefit the whole community, just create a PR, like the recent one which changed stories folder to .stories folder.

@tomitrescak , yeah the atom-package's yaml configs are great! Would be cool if the cli could use the same templates. @sungwoncho 's idea of having a .mantra-config-folder sounds also promising: https://talk.mantrajs.com/t/do-we-have-to-use-the-mantra-generated-eslintrc/178/3


Wroking version confirmation

Can you confirm that this script still works on 7.2.8 ? Also, will api-samples/siem/01_GetOffenses.py be able to give me the ability to get report of offenses per day ?

The current version in master states it is for 7.2.8 in the README if you find it doesn't work please open an issue with details on what isn't working.


Will already authenticated connections continue to work after refreshing credentials via credentialsResolver dynamically?

Will already authenticated connections continue to work after refreshing credentials via credentialsResolver dynamically?
The javadoc for credentialsResolver say, so my question is will already authenticated old connections continue to work when the password is rotated?
* Credentials resolver object which is invoked during connection/reconnection process.
 * It makes possible to specify dynamically changing Redis credentials.

When using retryAttempts > 1, for example the old connection stopped working due to expired credentials, will Redission internally create new connection by calling resolve() method before throwing exception to client?

will already authenticated old connections continue to work when the password is rotated?

On WRONGPASS error a connection will be reconnected with authentication data got from credentialsResolver. It was fixed in 3.26.0 version.


3.0.0-beta7 CLI errors

Not able to create resources

ionic cordova resources -s

 ✔ Collecting resource configuration and source images - done!
 ✔ Uploading source images to prepare for transformations - done!
 ⠴ Generating platform resources Error: encountered bad status code (400) for http://res.ionic.io/api/v1/transform
     at Request.cli_utils_1.createRequest.timeout.type.send.on (/Applications/XAMPP/xamppfiles/htdocs/Excalibur-mobile/node_modules/@ionic/cli-plugin-cordova/dist/lib/resources.js:159:24)
     at emitOne (events.js:96:13)
     at Request.emit (events.js:188:7)
     at Request._emitResponse (/Applications/XAMPP/xamppfiles/htdocs/Excalibur-mobile/node_modules/superagent/lib/node/index.js:745:10)
     at ClientRequest.<anonymous> (/Applications/XAMPP/xamppfiles/htdocs/Excalibur-mobile/node_modules/superagent/lib/node/index.js:360:10)
     at ClientRequest.g (events.js:292:16)
     at emitOne (events.js:101:20)
     at ClientRequest.emit (events.js:188:7)
     at HTTPParser.parserOnIncomingClient (_http_client.js:473:21)
     at HTTPParser.parserOnHeadersComplete (_http_common.js:99:23)

Also when I seeked for help I got following error

ionic cordova resources --help

Automatically create icon and splash screen resources
Usage
  $ ionic cordova resources

Options
  --icon, -i ............... Generate icon resources
  --splash, -s ............. Generate splash screen resources

Examples
  $ ionic cordova resources

 TypeError: Cannot read property 'private' of undefined
     at inputs.filter (/usr/local/lib/node_modules/ionic/node_modules/@ionic/cli-utils/dist/lib/command/command.js:91:31)
     at Array.filter (native)
     at HelpCommand.getCleanInputsForTelemetry (/usr/local/lib/node_modules/ionic/node_modules/@ionic/cli-utils/dist/lib/command/command.js:90:18)
     at HelpCommand.<anonymous> (/usr/local/lib/node_modules/ionic/node_modules/@ionic/cli-utils/dist/lib/command/command.js:63:48)
     at Generator.next (<anonymous>)
     at fulfilled (/usr/local/lib/node_modules/ionic/node_modules/@ionic/cli-utils/dist/lib/command/command.js:4:58)

My Ionic Info
    Cordova CLI: 6.5.0
    Ionic Framework Version: 3.0.1
    Ionic CLI Version: 3.0.0-beta7
    ios-deploy version: 1.9.1
    ios-sim version: 5.0.4
    OS: macOS Sierra
    Node Version: v7.3.0
    Xcode version: Xcode 8.3.1 Build version 8E1000a

Run these 2 commands again.
npm install -g ionic@beta
npm install --save-dev @ionic/cli-plugin-ionic-angular@beta @ionic/cli-plugin-cordova@beta
Seems to have fixed it for me.

if u try ionic cordova resources -i it should work, but the ionic cordova resources -s does not work for me either

This has been fixed and beta8 will be out soon! Thank you!

@dwieeb any idea when? Still facing same issue

@mubasshir This fix has been out a long time now.


[SPARK-13974][SQL] sub-query names do not need to be globally unique while generate SQL

What changes were proposed in this pull request?
We only need to make sub-query names unique every time we generate a SQL string, but not all the time. This PR moves the newSubqueryName method to class SQLBuilder and remove object SQLBuilder.
also addressed 2 minor comments in https://github.com/apache/spark/pull/11696
How was this patch tested?
existing tests.

cc @liancheng @yhuai

LGTM

Test build #53430 has started for PR 11783 at commit e4edc0e.

Test build #53430 has finished for PR 11783 at commit e4edc0e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

test this please

Test build #53436 has started for PR 11783 at commit e4edc0e.

Test build #53436 has finished for PR 11783 at commit e4edc0e.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

test this please

Test build #53448 has started for PR 11783 at commit e4edc0e.

Test build #53448 has finished for PR 11783 at commit e4edc0e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

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

merging to master!


Error: 0x8007005 when Installing Ubuntu

I've check other related threads but can't find solution so I post a new one.

Your Windows build number:  (Type ver at a Windows Command Prompt)
Microsoft Windows [Version 10.0.16299.726]

Windows(R), Enterprise edition:
    Volume activation will expire 5/12/2019 5:48:52 PM

What you're doing and what's happening:  (Copy&paste specific commands and their output, or include screen shots)
I choose to manual install ubuntu1804 but after activated WSL I can't run the ubuntu1804.exe, got Error: 0x80070005 problem despite using an elevated Powershell to run command.

PS C:\users\longnx> Enable-WindowsOptionalFeature -Online -FeatureName Microsoft-Windows-Subsystem-Linux
(OK, Windows ask me for restart)
(Restarted PC)
PS C:\wsl\ubuntu> .\ubuntu1804.exe
Error: 0x80070005

What's wrong / what should be happening instead:
Can't install Ubuntu1804 on my Windows 10
It should be installed sucessfully.

I've tried to collect log but got error:
PS C:\wsl\ubuntu> logman.exe create trace lxcore_kernel -p {0CD1C309-0878-4515-83DB-749843B3F5C9} -mode 0x00000008 -ft 1
0:00 -o .\lxcore_kernel.etl -ets
logman.exe : A value that is not valid (.\lxcore_kernel.etl) was specified for the outputFormat parameter. Valid
values are Text and Xml.
At line:1 char:1
+ logman.exe create trace lxcore_kernel -p {0CD1C309-0878-4515-83DB-749 ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidArgument: (:) [], ParameterBindingException
    + FullyQualifiedErrorId : IncorrectValueForFormatParameter

Do you have McAfee Endpoint Security installed? If so, then this is likely a duplicate of #2940.

@chrisjbreisch sorry for late reply, yes, I have McAffee Endpoint Security install on my machine.

try below WA:

download lxrun from below url
https://github.com/DDoSolitary/LxRunOffline/releases
install subsystem linux from microsoft store.
execute below command to install your linux in you dir with administrator.
PS C:\Users*> C:\Users*\Downloads\LxRunOffline-v3.4.0\LxRunOffline.exe install -n  -d 'dir which you want to installed' -f 'C:\Program Files\WindowsApps<your linux distribution>\install.tar.gz'
example:
PS C:\Users\test> C:\Users\test\Downloads\LxRunOffline-v3.4.0\LxRunOffline.exe install -n ubuntu1804 -d 'C:\subsystem\ubuntu1804' -f 'C:\Program Files\WindowsApps\CanonicalGroupLimited.Ubuntu18.04onWindows_1804.2019.522.0_x64__79rhkp1fndgsc\install.tar.gz'
then you can use bash/wsl to login your subsystem with root user. then uninstall subsystem linux from microsoft store, you don't need it anymore.


Optimal Buffer Level control ?

Do I have any control over pensive in telling the network what an optimal buffer level would be?
Comparing with algorithms festive [1] and panda [2]. There is included functionality for maintaining a certain buffer level, ~30 seconds in both papers. Festive [1] section 3.1, Panda [2] section III paragraph 4.
--> pretense: Allowing the application to delay video download request to maintain a buffer level promotes better bandwidth sharing as well prevents possible waste of resources should the client decide to exit before finishing.
The pensive network just returns a bitrate selection, not a timing delay, so I understand that receiving a download delay estimation is out of scope, however if i manually add a request delay based on my desired buffer level outside of pensive, then pensive appears to observe the buffer level not growing despite there being a high throughput estimation and decreases video quality.
Are there any variables or control options to telling pensive what an optimal buffer level would be, so that it may request high bitrate chunks until the buffer decreases significantly below a certain threshold or notices a significant throughput drop ?
[1] Junchen Jiang, Vyas Sekar, and Hui Zhang. 2012. Improving fairness, efficiency, and stability in HTTP-based adaptive video streaming with FESTIVE.
[2] Z. Li et al. 2014. Probe and Adapt: Rate Adaptation for HTTP Video Streaming
At Scale. IEEE Journal on Selected Areas in Communications (2014).

Thanks for your question. What you propose is along the line of buffer-based algorithms. If you like, you can change the implementation in bb.py with the logic you described above: https://github.com/hongzimao/pensieve/blob/master/test/bb.py#L77-L84. Hope this helps.


LoopyBeliefPropagation may not work with varying number of variables

Hi,
When I try to define a factor template that have varying number of neighbors, I got this error
Exception in thread "main" java.lang.ClassCastException: cc.factorie.variable.SeqVariable cannot be cast to cc.factorie.variable.VectorVar
	at cc.factorie.infer.BPFactor1Factor2.<init>(BP.scala:222)
	at cc.factorie.infer.BPFactorFactory$.newBPFactor(BP.scala:46)
	at cc.factorie.infer.LoopyBPSummary$$anonfun$apply$2.apply(BP.scala:511)
	at cc.factorie.infer.LoopyBPSummary$$anonfun$apply$2.apply(BP.scala:511)
	at scala.collection.mutable.LinkedHashSet.foreach(LinkedHashSet.scala:93)
	at cc.factorie.infer.LoopyBPSummary$.apply(BP.scala:511)
	at cc.factorie.infer.InferByBPLoopy$.infer(BP.scala:814)
	at cc.factorie.infer.InferByBPLoopy$.infer(BP.scala:810)
	at cc.factorie.optimize.LikelihoodExample.accumulateValueAndGradient(Example.scala:113)
	at cc.factorie.optimize.BatchTrainer$$anonfun$processExamples$1.apply(Trainer.scala:55)
	at cc.factorie.optimize.BatchTrainer$$anonfun$processExamples$1.apply(Trainer.scala:55)
	at scala.collection.mutable.ArraySeq.foreach(ArraySeq.scala:74)
	at cc.factorie.optimize.BatchTrainer.processExamples(Trainer.scala:55)
	at cc.factorie.optimize.Trainer$.train(Trainer.scala:406)
	at cc.factorie.optimize.Trainer$.batchTrain(Trainer.scala:441)

This is my template
new DotTemplate2[TripleLabel, SeqVariable[TripleLabel]]{
  factorName = "SubStructure"
  val nFeature = 1
  val weights: Weights1 = Weights(new DenseTensor1(nFeature))

  def unroll1(label: TripleLabel): Iterable[Factor] = {
    if (label.triple.childrenLabel.isEmpty) Nil
    else Factor(label, new SeqVariable(label.triple.childrenLabel))
  }
  def unroll2(children: SeqVariable[TripleLabel]) = throw new Exception("Should not call this")

  override def statistics(v1: BooleanValue, v2: IndexedSeq[TripleLabel]): Tensor = {
    // return feature here
    val allChildrenIsWrong = if (v2.forall(_.booleanValue == false)) 1.0 else 0.0
    new DenseTensor1(Array(allChildrenIsWrong))
  }
}

I have tried VarArg instead of SeqVariable but still have this error. I take a look the code at cc.factorie.infer.BP.scala line 44-49
      case factor:Factor2[DiscreteVar @unchecked,DiscreteVar @unchecked] =>
        if (varying.contains(factor._1) && varying.contains(factor._2)) new BPFactor2Factor2(factor, new BPEdge(summary.bpVariable(factor._1)), new BPEdge(summary.bpVariable(factor._2)), summary, summary.ring)
        else if (varying.contains(factor._1)) new BPFactor1Factor2(factor.asInstanceOf[Factor2[DiscreteVar,VectorVar]], new BPEdge(summary.bpVariable(factor._1)), summary)
        else if (factor._1.isInstanceOf[DiscreteVar]) new BPFactor1Factor2(factor.asInstanceOf[Factor2[DiscreteVar,VectorVar]], new BPEdge(summary.bpVariable(factor._2)), summary)
        else new BPFactor1Factor2Left(factor.asInstanceOf[Factor2[VectorVar,DiscreteVar]], new BPEdge(summary.bpVariable(factor._2)), summary)

It seems that even if I make a sub-class contains all varying neighbors and inherit VectorVar, the code here will treat the second variable of a factor as constant, and won't update the value properly.
Can you please show me the correct way to work though this issue? Any help would be greatly appreciated!

I don't believe belief propagation inference supports variable-neighbored (or frankly even fixed 3- or 4-neighbored) factors right now.
So this is a (very reasonable) feature request. In the meantime you could try one of the other inference methods like MCMC (including Gibbs), iterated conditional modes, MPLP?
Discrete mean field doesn't support >2 neighbors but it would be quite easy to extend to your case, much easier than BP.
Sorry for the unsatisfying answer, patches are definitely welcome!


Fix: Mark class as abstract

This PR

[x] marks what appears to be an abstract test case as abstract
[x] marks protected method declared in this abstract test case and never overridden by extending classes as final

Thanks1

Thank you, @svpernova09!


Connection TimeOut api.razorpay.com

Laravel logs the following error.
Requests_Exception: cURL error 28: Failed to connect to api.razorpay.com port 443: Connection timed
But it only happens in 1 out of 10 transactions. Can someone suggest what can be the issue?

Hi @bhagchandani could you please let us know if this issue still persist ?
Closing this issue for now but if you are still experiencing this issue , please repen it


[HUDI-1350] Support Partition level delete API in HUDI

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
Support Partition level delete API in HUDI  base on replace action. the spark datasource and hudi cli can use this api
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@xushiyan @vinothchandar  hello , the Travis CI may have problem. Just from yesterday  all the ci is queued for a long time

Codecov Report

Merging #2254 (7cdd2db) into master (c8d5ea2) will decrease coverage by 6.97%.
The diff coverage is 50.00%.

@@             Coverage Diff              @@
##             master    #2254      +/-   ##
============================================
- Coverage     53.51%   46.53%   -6.98%
+ Complexity     2769     2435     -334
============================================
  Files           348      342       -6
  Lines         16107    15743     -364
  Branches       1643     1627      -16
============================================
- Hits           8619     7326    -1293
- Misses         6789     7846    +1057
+ Partials        699      571     -128

Flag
Coverage Δ
Complexity Δ

hudicli
38.37% <ø> (ø)
0.00 <ø> (ø)

hudiclient
100.00% <ø> (ø)
0.00 <ø> (ø)

hudicommon
55.32% <50.00%> (+0.05%)
0.00 <0.00> (ø)

hudihadoopmr
32.94% <ø> (ø)
0.00 <ø> (ø)

hudispark
65.58% <ø> (ø)
0.00 <ø> (ø)

huditimelineservice
?
?

hudiutilities
10.41% <ø> (-59.66%)
0.00 <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ
Complexity Δ

...g/apache/hudi/common/model/WriteOperationType.java
55.17% <50.00%> (-0.39%)
2.00 <0.00> (ø)

...va/org/apache/hudi/utilities/IdentitySplitter.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-2.00%)

...va/org/apache/hudi/utilities/schema/SchemaSet.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-3.00%)

...a/org/apache/hudi/utilities/sources/RowSource.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-4.00%)

.../org/apache/hudi/utilities/sources/AvroSource.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

.../org/apache/hudi/utilities/sources/JsonSource.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

...rg/apache/hudi/utilities/sources/CsvDFSSource.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-10.00%)

...g/apache/hudi/utilities/sources/JsonDFSSource.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-4.00%)

...apache/hudi/utilities/sources/JsonKafkaSource.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-6.00%)

...pache/hudi/utilities/sources/ParquetDFSSource.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-5.00%)

... and 36 more

@satishkotha  hello , can you help to review ?

@satishkotha @n3nash @bvaradar  can you help to review , thanks

@satishkotha Have resolved the comments.

@n3nash @bvaradar Can one of you  review as well?

@vinothchandar  hello this is about replace commit , have users need this feature , just as @bvaradar  comment  (blocker 0.7.0)

@n3nash @satishkotha @bvaradar anyone have context to review this today?

@n3nash @satishkotha @bvaradar anyone have context to review this today?

@vinothchandar I already reviewed this earlier. Functionality looks good to me. You may want to review high level API. Another thing to note, like 'insert overwrite', this doesn't update index. So that is a limitation.

@vinothchandar @satishkotha have resolved the comments, can you help to review again ? thanks so much


Custom Lua Sections

What feature do you want to get added on the base engine?
My idea: have 2 dropdowns for each section, one for Boyfriend and one for the opponent, any Lua scripts that we put into the custom_sections folder will show up on both dropdowns unless otherwise stated with a Lua callback maybe named setSectionDropdown('custom_section', dropdown{0 or 1}).  This way we can do different things for a section based on the code in the Lua script.
To test your sight, and reliability, please select the option of what should NOT be requested.
Stage Editor, 6K+ support, and winning icons.

unnecessary as you can just make the script happen on the first beat of that section

But it makes charting things like this less tedious

That's not how it works


Upgrade to react 17?

I just upgraded to<EMAIL_ADDRESS>and now get a dependency error (see below).
Is it possible to upgrade the peer dependency to v17?
Full error log:
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer react@"~16" from<EMAIL_ADDRESS>npm ERR! node_modules/react-qr-reader
npm ERR<EMAIL_ADDRESS>from the root project

Sorry wrong repo

Sorry wrong repo


Update FAQ.rst

Added a call to jobqueue.task_done(). This removes the respective job from the queue after it was obtained by .get(). As the associated action is jobqueue.put the queue will fill up with every scheduled run if finished jobs are not removed. #134

Perfect, thank you! Mind adding yourself to https://github.com/dbader/schedule/blob/master/AUTHORS.rst and then I'll get this merged :)

Thanks Dan!

On 11/04/2017 08:31, Dan Bader wrote:

  Perfect, thank you! Mind adding yourself to https://github.com/dbader/schedule/blob/master/AUTHORS.rst
    and then I'll get this merged :)
  —
    You are receiving this because you authored the thread.
    Reply to this email directly, view
      it on GitHub, or mute
      the thread.

  {"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/dbader/schedule","title":"dbader/schedule","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/dbader/schedule"}},"updates":{"snippets":[{"icon":"PERSON","message":"@dbader in #135: Perfect, thank you! Mind adding yourself to https://github.com/dbader/schedule/blob/master/AUTHORS.rst and then I'll get this merged :)"}],"action":{"name":"View Pull Request","url":"https://github.com/dbader/schedule/pull/135#issuecomment-293164327"}}}


Bump max_length in models.Site

https://github.com/rackerlabs/scantron/blob/ad542bd0fcc1526f5c30413895d37baaa18ee9a8/master/django_scantron/models.py#L135
Still hitting max limits when including individual IPs.

Changed to 4194304,  # 2^22 = 4194304

Root cause was not specifying a client_max_body_size 4M; setting in nginx.

https://github.com/rackerlabs/scantron/pull/217


more forgotten self usages

Codecov Report

Merging #531 into master will increase coverage by 0.17%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #531      +/-   ##
==========================================
+ Coverage   92.15%   92.32%   +0.17%
==========================================
  Files          37       37
  Lines        2346     2346
  Branches      202      202
==========================================
+ Hits         2162     2166       +4
+ Misses        139      135       -4
  Partials       45       45

Impacted Files
Coverage Δ

api/app/data.py
41.48% <100%> (+4.25%)
:arrow_up:

Continue to review full report at Codecov.

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


Add pt-PT localization and rename previous pt to pt_BR

Added the Portuguese (Portugal) localization and rename the previous one to pt_BR

Hi Miguel, you should update the tests for pt_BR so that they can correctly load their variant, and please add tests for pt_PT.
This is going to be a breaking change for users that previously used the Portuguese localization so it needs to have good coverage and documentation. Also please edit the README section to include pt_PT in the list and list yourself as the author.
When this is done I'll merge,
Thanks!

I forgot to run the tests, sorry. I made the changes requested.

Fantastic, thanks!
Will review, merge and push out a new version soon!

All done, 1.2.5 should be up in maven in an hour or so. Please note that I've changed the localization string to pt for Portuguese, instead of pt-PT.
Thank you for your contribution!


[ReferenceError] "ReferenceError: window is not defined" on<EMAIL_ADDRESS>

Target Platforms
NodeJS
SDK Version
2.11.0
Application Name
Node console app
Problem Description
My project is a node backend app using adaptivecards to parse/render/verify card content on server side.
It was using<EMAIL_ADDRESS>and everything worked well. After switching to the latest<EMAIL_ADDRESS>it throws ReferenceError: window is not defined on require("adaptivecards").
ReferenceError: window is not defined
at Object. (...\node_modules\adaptivecards\lib\utils.js:149:26)
at Module._compile (node:internal/modules/cjs/loader:1105:14)
at Object.Module._extensions..js (node:internal/modules/cjs/loader:1159:10)
at Module.load (node:internal/modules/cjs/loader:981:32)
at Function.Module._load (node:internal/modules/cjs/loader:822:12)
at Module.require (node:internal/modules/cjs/loader:1005:19)
at require (node:internal/modules/cjs/helpers:102:18)
at Object. (...\node_modules\adaptivecards\lib\adaptivecards.js:22:14)
at Module._compile (node:internal/modules/cjs/loader:1105:14)
at Object.Module._extensions..js (node:internal/modules/cjs/loader:1159:10)
Screenshots

Card JSON
{
  "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
  "type": "AdaptiveCard",
  "version": "1.4",
  "body": [
  ]
}

Sample Code Language
javascript
Sample Code
const { AdaptiveCard } = require("adaptivecards");  // error occurs at this line
const card = require("./card.json");
const ac = new AdaptiveCard();
ac.parse(card);

Same for me when I try to run this sample repo code.
https://github.com/OfficeDev/TeamsFx-Samples/tree/v1.0.0/bot-sso

Hey @swatDong - this has been fixed in 2.11.1! Please let us know if you encounter any further difficulties.


#001 Creating Youtube Rendering functionality

[x] Create logger
[x] Create a custom exception.
[x] Youtube file to deal with this module.
[x] Manual check in jupyter notebook.

Its done from development prespective


Add link to docker logos

Currently, the docker logo is just an image , I forgot to add the link in the previous PR. @mdozmorov

Very clean version! Sorry for some delays, meetings.


[Debug] Launch In Terminal (Git Bash) didn't reuse the previous terminal window

VSCode Version:

Version: 1.39.0-insider (system setup)
Commit: 8b6e0efc0a98639085fabeb5226a516785424796
Date: 2019-09-19T03:15:21.720Z
Electron: 6.0.9
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Windows_NT x64 10.0.18362

OS Version:  Windows 10

Steps to Reproduce:

Create HelloWorld.java in VS Code.
Set Git Bash as the default shell.
Click Run codelens on top of main method.

Every click on Run codelens will always open a new terminal window. But if i set the default shell to cmd or powershell, VS Code will try to reuse the previous terminal window.

Expected: The git bash behavior is consistent with cmd and powershell.

/duplicate https://github.com/microsoft/vscode/issues/81345

i don't think this issue is duplicated with #81345.
#81345 is about debug terminal is not switched after changing the default shell.
However, this issue is about cannot reuse current git bash terminal.

@testforstephen I think it's the java extension launching the terminal, so both have the same root cause.


🛑🛑🛑Please contact customer support🛑🛑🛑

Hi All, I am new in AL programming, I am working in ISV product, there is a lots of customization on job card. System working fine. But
I am facing performance issue while opening job card, it will take too much time (with jobs that have 100+ job planning line) while opening the job card.
Please let me suggest how to improve my job card performance.

You should ask on a forum or the DynamicsNAVDev Yammer. This is not on-topic for this repo, which deals only with issues in the core AL language and compiler. Didn't the template you selected, with its title, indicate that? ;-)

Closing this issue as it is out of scope for this repository


What are the downside of disabling JFR

We are running AppD in our pod along with our application. Our application use lettuce library for redis connectivity, for some reason, When we upgraded our lettuce library version to 6.1.3 in the application then we start getting error: Could not initialize JfrConnectionCreatedEvent #1767.
So we found following link and tried to resolved:
https://github.com/lettuce-io/lettuce-core/issues/1767
Based on above link there are two solutions:

Use the following statement System.getProperty("io.lettuce.core.jfr",false); to disable the jfr.
The second is to provide --add-reads=jdk.jfr=ALL-UNNAMED on jvm argument.

Could you guys tell me which one is better approach and what is the downside of disabling the JFR in lettuce?

JFR events make sense if you use these. If you do not evaluate JFR events in your environment, disabling them is better.


3/31: team 1 changes restaurant QA

Pull Request Test Coverage Report for Build #113

61 of 61 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 93.795%

Totals

Change from base Build #108:
0.1%

Covered Lines:
2449

Relevant Lines:
2611

💛  - Coveralls


fix: updated misleading title

Description
Updated misleading title from Datasource Connected to " Datasource Saved "
Fixes #2754
Fixes #7936
Type of change

Please delete options that are not relevant.

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

Checklist:

[ ] 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
[ ] My changes generate no new warnings
[ ] New and existing unit tests pass locally with my changes

@areyabhishek @mohanarpit hey can you please review this for hacktoberfest too? Thankss in advance!

/ok-to-test sha=d201170


Any ideas for fix. Agarinfinity is working well with browsec -.-

Its working but im sick of lag, so anyone got any fix for those beauty bots?
skype: sejo.dedic4

Well the only 2 options i can possibly give is:

Upgrade your internet to a better plan
Get a new PC
I know this may sound harsh but if you were to have those, The lag would be reduced and would make a huge difference


Angryziber.AngryIPScanner376 version 3.7.6

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Installer is Nullsoft. Package installed silently from manifest.
Will test on second workstation later. Stand by.

Requires JDK. Unsure how to combine multiple packages into one manifest.
Close for now. If someone else can figure it out, great. Might come back to this.


Set up KIND test environment for GMC e2e on Xeon and Gaudi cluster

to avoid conflict with AI workload tests

This is done. We will support xeon test for gmc controller itself.

close it


Collapse/Expand square vanish on removal of comment block

As you can see in the image below, I have a collapsed section of code, that happens to start with a comment..

If I remove the start of the comment block, the [+] vanishes from line 428. But the code is still in the file, but just no way to expand it.

Colin.

Javascript file syntax
FD <IP_ADDRESS>

Just a note: adding back /* restores the [+]

Ahh, sorry, thought I had put that in the message. Sorry for not doing so.

This happens when you remove the line above any collapsed piece of code. Really frustrated me when I removed a collapsed function definition like
function x() {
because my code wouldn't compile due to the invisible stray closing brace that was still collapsed, with no way to un-collapse it apart from backspacing on the line after it.


Most recent version 4.2.14 breaks IE11

Likely from this: https://github.com/wwayne/react-tooltip/pull/668
Causes a Internet Explorer throws SCRIPT5022: HierarchyRequestError in IE11
Method of injection is not supported

Have the same issue, it's definitely due to #668.
parentNode.constructor.name returns undefined on IE11, that causes this switch to assume the domRoot is the parentNode instead of parentNode.head.
Think the fix would be to just change that switch to:
    switch (parentNode.constructor.name) {
      case 'ShadowRoot':
        domRoot = parentNode;
        break;
      case 'HTMLDocument':
      default:
        domRoot = parentNode.head;
        break;
    }

But not entirely sure if this is safe or if it'll break something else.

Getting the same issue for safari as well.
hierarchyrequesterror: the operation would yield an incorrect node tree.
The older version works fine

We are also getting the same error. Is reverting to 4.2.13 fixes the issue ?

@KumarSumitD Reverting to 4.2.13 resolved the issue for us.

HierarchyRequestError: The operation would yield an incorrect node tree.
Safari: 14.0.3
react-tooltip: 4.2.17

Same issue here in 4.2.17

Same issue here in 4.2.19

Confirmed that reverting back to 4.2.13 resolves the issue.


Add a benchmark for unicode Strings created via JSON decoding

This hits some string bridging paths we didn't have coverage on yet

@swift-ci please smoke test

@swift-ci please smoke benchmark

@swift-ci please test osx platform

Performance: -O

Regression
OLD
NEW
DELTA
RATIO

StringBuilderWithLongSubstring
1320
1440
+9.1%
0.92x (?)

Improvement
OLD
NEW
DELTA
RATIO

ArrayAppendGenericStructs
2490
1340
-46.2%
1.86x (?)

Added
MIN
MAX
MEAN
MAX_RSS

UnicodeStringFromCodable
554
555
555
—

Code size: -O

Regression
OLD
NEW
DELTA
RATIO

ObjectiveCBridging.o
52065
60838
+16.9%
0.86x

Performance: -Osize

Regression
OLD
NEW
DELTA
RATIO

ArrayAppendGenericStructs
1340
2490
+85.8%
0.54x (?)

ObjectiveCBridgeStubFromNSDateRef
3760
4080
+8.5%
0.92x (?)

Added
MIN
MAX
MEAN
MAX_RSS

UnicodeStringFromCodable
562
562
562
—

Code size: -Osize

Regression
OLD
NEW
DELTA
RATIO

ObjectiveCBridging.o
49605
54698
+10.3%
0.91x

Performance: -Onone

Improvement
OLD
NEW
DELTA
RATIO

ArrayAppendOptionals
2490
2170
-12.9%
1.15x (?)

Added
MIN
MAX
MEAN
MAX_RSS

UnicodeStringFromCodable
572
573
573
—

Code size: -swiftlibs

✅
Benchmark Check Report

How to read the data
The tables contain differences in performance which are larger than 8% and
differences in code size which are larger than 1%.
If you see any unexpected regressions, you should consider fixing the
regressions before you merge the PR.
Noise: Sometimes the performance results (not code size!) contain false
alarms. Unexpected regressions which are marked with '(?)' are probably noise.
If you see regressions which you cannot explain you can try to run the
benchmarks again. If regressions still show up, please consult with the
performance team (@eeckstein).

Hardware Overview
  Model Name: Mac Pro
  Model Identifier: MacPro6,1
  Processor Name: 12-Core Intel Xeon E5
  Processor Speed: 2.7 GHz
  Number of Processors: 1
  Total Number of Cores: 12
  L2 Cache (per Core): 256 KB
  L3 Cache: 30 MB
  Memory: 64 GB

@swift-ci please smoke test


Updating HTTPRoute docs for v1alpha2

What type of PR is this?
/kind documentation
What this PR does / why we need it:
The examples used by the HTTPRoute docs page were out of date, this updates them. It also removes a duplicate example (traffic-split-1 was also in the traffic-splitting directory).
Does this PR introduce a user-facing change?:
NONE

/lgtm
/approve


sparse lookup tables for tf.nn.embedding_lookup_sparse

Hope that “params” of tf.nn.embedding_lookup_sparse support SparseTensor!!!
tf.nn.embedding_lookup_sparse(params, sp_ids, sp_weights, partition_strategy='mod', name=None, combiner='mean')
The efficiency of sparse lookup tables is low,  but it is very useful for large-scale linear regression system, especially when designing a prototype.
For example, the wide part of "wide and deep learning" use sparse features.
Although I don't know when the hashing function is applied, using sparse lookup table is more accurate than hashing.  The hashing function may merge two features into one bin (share the same weight), or discard new features when the lookup table is full.

@concretevitamin might have some thoughts on this.

Any update @ebrevdo? If this is not going to happen soon, I plan on closing this pending a more concrete plan here. It's not clear enough to move to contributions welcome.

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


feat(cli): add --verbose flag to sunodo build command

This PR will add a --verbose flag to sunodo build, making the normal mode of sunodo build more quiet.

I didn't like this very much because in case of docker build error, which is very common, the error is not displayed.
I think we will be safer if we leave the docker build as it is.
On a less important note, we also lose docker color output during --verbose

On a less important note, we also lose docker color output during --verbose

That's because I used --progress=plain, we can just don't change this.

Let the docker pull print


Document formatting is removing characters!

VSCode Version: Code 1.9.0 (27240e71ef390bf2d66307e677c2a333cebf75af, 2017-02-02T08:31:00.827Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions:

Extension
Author
Version

Steps to Reproduce:

code the following code and place it in an html file then format the file.

<script>
      tag = document.createElement('script');
        src = "https://www.youtube.com/iframe_api";
        firstScriptTag = document.getElementsByTagName('script')[0];
        firstScriptTag.parentNode.insertBefore(tag, firstScriptTag);
  </script>

After the formatting it would look like:
<script>
    document.createElement('script');
        "https://www.youtube.com/iframe_api";
        criptTag = document.getElementsByTagName('script')[0];
        criptTag.parentNode.insertBefore(tag, firstScriptTag);
</script>

I uninstalled VS Code, removed the 'code' folder in 'AppData\Roaming' and removed all extensions. The issue still occurs!

duplicate of #19813


Subtype GMRF under AbstractMvNormal

Fixes bijector error when using in a Turing hierarchical model.
@jrfaulkner ran into an error when trying to use Turing to sample a GMRF with observation error. An error is thrown about a missing bijector for GMRF. Subtyping GMRF <: AbstractMvNormal rather than our current GMRF <: ContinuousMultivariateDistribution solves this. Jim sent me the following example that should work now:
using GaussianMarkovRandomFields
using Random
using LinearAlgebra
using Turing
using SparseArrays

k = 100 #5_000   # number of measurements in time series
σ = 0.5     # marginal variance
ρ = 0.4     # autocorrelation coefficient # !! Why doesn't 0.8 work? causes sqrt of negative
μ = 5.0     # mean

Q = ar_precision(ρ, k) ./ σ^2

d = GMRF(μ*ones(k), Q)
x = rand(d)  # generates realization of d

@model function AR1y(y)
    n = length(y)
    σ ~ Gamma(2, 3)
    sigy ~ Gamma(2, 3)
    ρ ~ Uniform(0, 0.5)
    μ ~ Normal(0, 10)
    Q = ar_precision(ρ, n) ./ σ^2
    theta ~ GMRF(μ*ones(n), Q)
    y ~ MvNormal(theta, sigy)  # this kills it for some reason
end

sigy = 1.5
yobs1 = x + randn(k)*sigy

modar1y = AR1y(yobs1)
@time chn_ar1y = sample(modar1y, NUTS(), 1000)

Thanks for this! Looks good, but seeing as I don't have CI set up for this package yet, can you just confirm that the test suite runs locally?

Tests pass locally for me:
     Testing Running tests...
Test Summary: | Pass  Total
Constructors  |    6      6
Test Summary:  | Pass  Total
Linear Algebra |    5      5
Test Summary: | Pass  Total
Likelihood    |    2      2

logdet_ldl (sparse): TrialEstimate(9.090 μs)
logdet_ldl (dense): TrialEstimate(11.830 μs)
logdet (dense): TrialEstimate(38.485 μs)

logpdf (sparse, LDL): TrialEstimate(1.248 μs)
logpdf (sparse, MvNormalCanon): TrialEstimate(867.021 ns)
logpdf (dense, MvNormalCanon): TrialEstimate(1.495 μs)
logdet (dense, MvNormal): TrialEstimate(4.346 μs)
Test Summary: | Pass  Broken  Total
Performance   |    4       1      5
Test Summary: | Pass  Total
AD            |    2      2
┌ Info: Found initial step size
└   ϵ = 1.6500000000000001
Sampling 100%|█████████████████████| Time: 0:00:17
┌ Warning: The current proposal will be rejected due to numerical error(s).
│   isfinite.((θ, r, ℓπ, ℓκ)) = (true, false, false, false)
└ @ AdvancedHMC ~/.julia/packages/AdvancedHMC/MIxdK/src/hamiltonian.jl:47
┌ Info: Found initial step size
└   ϵ = 3.2
┌ Warning: The current proposal will be rejected due to numerical error(s).
│   isfinite.((θ, r, ℓπ, ℓκ)) = (true, false, false, false)
└ @ AdvancedHMC ~/.julia/packages/AdvancedHMC/MIxdK/src/hamiltonian.jl:47
Sampling 100%|█████████████████████| Time: 0:00:09
Test Summary: | Pass  Total
Turing        |    2      2
Test Summary:     | Pass  Total
Marginal variance |    2      2
     Testing GaussianMarkovRandomFields tests passed

Awesome, thanks!


Security Token Issue

Seeing an error when running the example code. I'm pretty confident I'm using the correct key and password so I'm not sure what's going on.
ERROR: Invalid request: The security token included in the request is invalid.
Any guidance on this would be a big help.
Thanks,
Steve

Figured out the problem. I had created new keys.cpp and keys.h files but these files already existed in another library I had. Deleted the old key files and everything is working.
Thanks for the great SDK and example!


Confusion over what to use these days

Hi, really just looking for clarification on what I need to have installed for the best StyleCop "experience".
I'm using VS2013 and Resharper 9.2, plus the R# extension "StyleCop by JetBrains v4.7.1" (whose "more info" link brought me to this repo). Why did the link bring me here - I'd assumed this repo was a port of the "old" StyleCop VS plugin, and unrelated to the R# extension?
Using the R# extension, I get squiggles under code that violates StyleCop rules, but I miss the ability to see a list of all violations in the "Error List" window, which is what the "old" StyleCop VS plugin did. I've resorted to installing that plugin (4.7), which I periodically run on the solution, but it's not ideal. Is there any other way to get a list of all violations?
One of the problems is that our devs often miss the squiggle warnings, and the first they know of any violations is when a load of 'SA...' warnings appear in the build output from our TFS server. We have StyleCop installed on there, and a custom build definition that runs StyleCop on the solution during the build.
Lastly, I've also spotted a R# extension called "StyleCop for Resharper" - what's the difference between that and the "StyleCop by JetBrains" R# extension?

This repository is indeed the "StyleCop by JetBrains" R# extension. The readme file contains some info about its history.
The "StyleCop for Resharper" is pretty much dead and despite claiming to be open source has nothing more than a nearly empty readme file in its GitHub repository. As far as I can see it has disappeared from the extension manager for R# 2016.2.
Additionally there's the StyleCop Analyzers project that hooks directly into the Roslyn compiler. This way it combines the capabilities of the standalone StyleCop (running at build time) and the R# extension (inline notifications and quick fixes) even when you don't have ReSharper. On the negative side it doesn't work with VS2013 because Roslyn was only introduced in VS2015. It's also missing some features such as localized documentation comments (see DotNetAnalyzers/StyleCopAnalyzers#2189)
Disclaimer I'm not an active maintainer of any of the mentioned repositories. I currently use "StyleCop for ReSharper" in VS2013 and will probably switch to "StyleCop Analyzers" as soon as VS "15" is out.


Add Relationships for supporting Find implementations

[x] Classes https://github.com/sourcegraph/scip-clang/pull/393
[ ] Methods. This seems to already work at least in some cases, needs more test cases around templates etc.

Missing relationships for methods in some cases.

https://sourcegraph.com/github.com/sourcegraph/scip-clang@4d53c18/-/blob/indexer/AstConsumer.h?L73

Looks like a backend issue, as a relationship is emitted at the definition with the body.


[shardformer] support pipeline for gpt

support pipeline for gpt models, integrate its policy to shardformer, and add relevant tests.

Resolved by #4245


Add CI/CD to generate releases and deployments

Covered in feature/12-create-infrastructure-in-aws


Add function for loading Kiez object from JSON configuration

Since #5 and #6 enable string-based instantiation of a Kiez object, it makes sense to be able to completely store a Kiez configuration in a JSON object and load from there. This PR implements loading based on a file path that simply opens the file, parses JSON, then splats the arguments into the Kiez constructor. It also adds a corresponding unit test (which weirdly is acting funny on my computer, but is pretty obvious what it does)

Thanks so much for the contribution! (It works fine on my computer btw)


ts: Convert css_variables.js to TypeScript.

Fixes:

Screenshots and screen captures:

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Mistakenly created a PR instead of my fork. Ignore this one :)

Merged, thanks @sbansal1999!


content width of page

Summary
Create a tex length that is \textwidth + \marginparsep + \marginparwidth
Details
In some cases, e.g. the tabularx environment, this length is necessary. Here is an example of how I am using it:
% table across the margin too, see https://github.com/fmarotta/kaobook/issues/123
\begin{table*}[h!]
    \centering
    \newlength{\contentwidth}
    \setlength{\contentwidth}{0pt}
    \addtolength{\contentwidth}{\textwidth}         % width of the text
    \addtolength{\contentwidth}{\marginparsep}      % width between text and margin
    \addtolength{\contentwidth}{\marginparwidth}    % width of the margin

    \begin{tabularx}{\contentwidth}{|X|X|X|X|X|X|}
        \hline
        problem setting & solution & goal & obstacles & collision detection & localization\tabularnewline
        \hline
        \hline
        classical motion planning & path & c-space & known & N/A & perfect\tabularnewline
        \hline
    \end{tabularx}
\end{table*}

Thanks, I accept your suggestion. I've now defined the \contentwidth length.


Simply Not Working (Please View)

Rankup Command Dosent Work
On Load Up Theres An Error Message
This Is It:
[Tesseract] [02:09:19] INFO> Enabling RankUp v0.3.1
Warning: yaml_parse(): reading error encountered during parsing: invalid leading UTF-8 octet (line 1, column 1) in phar:///home/minecraft-pe-2/server/PocketMine-MP.phar/src/pocketmine/utils/Config.php on line 142
Warning: Invalid argument supplied for foreach() in phar:///home/minecraft-pe-2/server/plugins/RankUp_v0.3.1.phar/src/rankup/doesgroups/RankUpDoesGroups.php on line 58

This has just happened when i added some more ranks to it and now i have my server on a whitelist
Please can this be fixed asap :)

In your first added rank, do you have any unusual characters like ß or maybe even with a comma above the letter? Try removing that and restarting.

This has thankfully solved the issue, i was using a sign for colours but have now changed it to & instead
thanks!


fix: not falling back to default slot content correctly

Fixes a bug where rendering a component without child nodes wouldn't render that component's default slot content.
Fixes #37.

LGTM thank you!!


unused token numbering oddities

Reported by Vanessa (commit fda49297242e6423863e6391e9df62fb5d6b86cd ):
The numbering has some oddities.  I am talking about where it tells you how many words are unused either on the left side (how many you have not hung on the tree) or in the right side under the relation tab, where it tells you how many have not been assigned a relation.   I refreshed the file and opened the next untreed sentence.  Two of my three defaults are preset (that is, the keyboard is disabled and the relations are set to Advanced Mode).  Morph does not have the Expanded Selection prechecked, and, when I do it manually, nothing is preselected.  The number on the left says “0 unused’, even though nothing has been done yet and the “highlight unused” button does nothing. I have no clue what the “Selection” and “none” buttons are supposed to do, since I cannot see that they do anything just now.   The number on the right on the relation tab says “30 of 35” which I suspect is correct.  However, if I move forward to other sentences without doing this tree, every one of them says xx of 35 unused, no matter how many words are in it (“75 of 35 unused”).
As I start to build the tree, the number of unused tokens decreases on the left from 0 into negative numbers.  Since I cannot tell how many words are in the sentence, I do not know when all are hung.  “Highlight unused” still does not work.  When I added in two elided words, the number on the right correctly changed to read 32 out of 37.  Now all subsequent sentences also say “xx out of 37.”
When I am done hanging the tree, but have done no labeling, the number on the left says -32 unused and the one on the right says 32 of 37 unused.

Comments from Vanessa:
The numbers on the left appear to be working correctly.  Good fix!  There is still an oddity in the numbers in the Relation box.  If you go to a sentence that is not yet treed, and let's say it says 11 of 13 unused.  If you skip to the next sentence without doing anything to that first sentence, it will say 35 of 13 unused.  I haven't sorted out the exact circumstances, but it keeps the total number from the first sentence that has not been done.
Another thing I would note about the numbers on the left: if you add in elided forms, then the total will be in negative numbers.  So when I added in two elided forms andmy whole tree was hung, it said -2 unused.  That works fine, so long as one knows that it works that way and doesn't stop hanging items at "0 unused."
I think it is keeping the total number of words from whatever sentence you first open up.

Wrong total numbers fixed in b3f0d2c23de29d637555af59e19b95236fcf78bb

Fixed problems with elided forms in 630177be9aa425f7c68119795151344c88b1bc08
@Christof, can you confirm I did this right? If yes (I think I did), close this issue.
Do we want to display the total number of tokens in the tokenSelector? I'd vote yes.

Confirm total numbers fix: see 156882a76c102ce14345aa754bb5cf1903fea13a (you contribution thief :smirk:)
I've also partially fixed the elided forms, although the formatting did not propagate. Your change also looks good, but now we can remove the tokenAdded and tokenRemoved events from the unused highlighter.
Where would you display the total number?

I think we can close this now?


kube-dns entry should be added in all the nodes' /etc/resolv.conf

None of the pods running with hostNetwork: True are able to resolve internal DNS entries. This is because all those pods use the resolv.conf present on the host machine.
I think kismatic should add the kube-dns service IP in the /etc/resolv.conf as part of the installation process. Any downsides there? Or what should be the best practice otherwise.

nvm, this is solvable in k8s using dnsPolicy: ClusterFirstWithHostNet.


"Câmpulung" is listed in the drop-down menu

Description:
"Câmpulung" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Câmpulung" in the search bar.

Expected results
The location is listed in the drop-down menu as "Câmpulung, Argeș".


Fail to import columns as DateTimeOffset

Azure Data Studio Version: 1.44.1 user setup
OS Version: Windows 11

Steps to Reproduce:

Use a CSV file with a column intended to be of type DateTimeOffset with the format '2023-06-01T:13:00:00-4:00'
Attempt to import the file, indicating that the column is of type DateTimeOffset(7)

The following exception is thrown:
---> System.InvalidOperationException: The given value '2023-06-01T:13:00:00-4:00' of type String from the data source cannot be converted to type datetimeoffset for Column <<column index and name>>.

The indicated value is definitely a DateTimeOffset. It should convert.

Does this issue occur when all extensions are disabled?: Yes/No
No. This is regarding the Sql Server Import extension

The support link for the extension links to this repo.

Maybe I'm not familiar with some edge case of the DateTimeOffset serialization format but is "2023-06-01T:13:00:00-4:00" a valid format?  I'm pretty sure the extra colon after the T is not correct.  If I try to use "2023-06-01T13:00:00-4:00" (removing the extra colon) then the import works correctly for me.

In light of @ckaczor comment, this looks like a mistake on my part. I'm not sure how I let that extra colon in there, but ckaczor is correct.
This should probably be closed and ignored.
Apologies for time spent pursuing it.


[Feature Request]: Support streaming (pubsub) in RedisIO connector

What would you like to happen?
Support streaming (pubsub) in RedisIO connector.
Issue Priority
Priority: 2 (default / most feature requests should be filed as P2)
Issue Components

[ ] Component: Python SDK
[ ] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[X] Component: IO connector
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

@lostluck do you know how to create an unbounded source in Go SDK?


Issue574 update saved crystals

Motivations:

Update all crystals saved in inputs_temp/, see issue #574

Main changes:

All crystals updated to tofu 1.4.15
Minor change in _check_optics.py to allow for 3-indices cuts

Issues:

Fixes, in devel, issue #574

Done


Routes for APIs

As a developer
I need to create routes for all APIs created
So that API services can be identified using individual routes

Details and Assumptions
-Different APIs have different routes from root directory

recommend previously bought items
recommend similar items
recommend additional items

Acceptance Criteria
Given I am using a shopping cart service for creating an item
When I want to create an item in shopping cart
Then it should have a unique routes (/shopcarts/create-item) to direct me to the place to create new item in the shopping cart


Rhythm wheels default wheels

From feedback from Dr Eglash's email. A branch dedicated to loading some default wheels with a default rhythmic pattern that students will see once they open up the Rhythm Wheels GUI.

This is what the default wheels are.


DRAFT (WIP) - variables

Depends on https://github.com/softdevteam/pavel.yaiwr/pull/12#discussion_r1129768866

variable store and ; termination symbol
TODO: add description

Can you rebase this once the other PR is in?

Is this ready for review now then?

@vext01 yes its ready for review :)

A few minor comments.
I really think your next PR should define an error type and all expect() should be replaced by error propagation caught at the top-level.
You'd want to make an error type, like:
enum InterpError {
  ParseError(usize), // line number inside
  EmptyStack,
  Numeric(String),
  ...
}

Then most of your functions can return `Result<T, InterpError>` (where `T` varies from function to function).

I really think your next PR should define an error type and all expect() should be replaced by error propagation caught at the top-level.

Agreed. Note that there are now various Rust crates (e.g. anyhow) that can make this less painful. But the longer you leave it, the more the pain accumulates :)

It's up to @vext01 but that's probably best left to the next PR, as this one still has other changes that are requested, and merging two-PRs-in-one is difficult.

Yeah, let's tackle the error separately. I'm easily overwhelmed.

ok merge reverted

Please squash.

squashed :point_right:  48e9b1b9268bfbdbba70ec8c213564fa4c263ac9

bors r+


docs(bun): remove deprecated sharp instructions

What is it?

[ ] Feature / enhancement
[ ] Bug
[x] Docs / tests / types / typos
[ ] Infra

Description
sharp is a native package that requires binaries to run. Before version 0.33.0, sharp relied on a postinstall script to download binaries. This made it incompatible with early versions of bun which did not execute postinstall scripts. In the meantime, sharp relies on optionalDependencies to download the binaries. As of Bun v1.0.16, the top 500 npm packages with lifecycle scripts are allowed by default. There is no more need to add sharp in the trustedDependencies of package.json.
References:

https://github.com/oven-sh/bun/issues/3783
https://github.com/oven-sh/bun/issues/4959
https://github.com/lovell/sharp/issues/3750#issuecomment-1751172728
https://bun.sh/docs/install/lifecycle
https://github.com/oven-sh/bun/blob/main/src/install/default-trusted-dependencies.txt

Checklist:

[ ] My code follows the developer guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have ran pnpm change and documented my changes
[x] I have made corresponding changes to the Qwik docs
[ ] Added new tests to cover the fix / functionality

@antoinepairet could you apply the trustedDependencies change to the second file?


feat: support returnDistinctValues

I get an error trying to convert a featureLayer to an sf object? Any suggestions?
library(arcgislayers)
url <- "https://mapprod3.environment.nsw.gov.au/arcgis/rest/services/Planning/EPI_Primary_Planning_Layers/MapServer/2"
fs <- arc_open(url)
ls <- list_fields(fs)
sf <- arc_select(fs, fields = c("LAY_NAME","LAY_CLASS"))

Error in rlang::set_names():
! x must be a vector
Run rlang::last_trace() to see where the error occurred.

I am able to create a featureLayer object from the featureServer URL, and I can list the field names. I tried using the 'fields' flag to limit what is read in, but there seems to be an issue with my featureLayer object when i run arc_select( )?
For this exercise, my goal is just to wrangle unique values from one field in the attribute table, but in general, I am hoping for a generic solution to wrangle all available features & attributes from a large collection of publicly available featureServer URLs.

Okay, so you didn't copy the whole error 😉

There is an issue in the requests somewhere along the way. This took a minute or so on my side which suggests this is a very big feature service and you want to bring all of it into memory.
I'd recommend you don't do that and only bring down the bits and pieces you need. As you can see there are nearly 71k features that you are reading into memory with this. My guess is that the featureservice sees this as a potential attack and is rate limiting you.
httr2::request(url) |>
  httr2::req_url_path_append("query") |>
  httr2::req_url_query(where = "1=1", outFields = "*", returnCountOnly = "true", f = "json") |>
  httr2::req_perform() |>
  httr2::resp_body_json()

#> $count
#> [1] 70797

thanks for the feedback, josiah. not sure how i managed to crop of the error message, so apologies for any misdirection.
often i can spatially restrict queries to an AOI, but for this task, i need to extract a unique list of available state-wide planning zones (from the field 'LAY_NAME'). i couldn't figure out how to drop the geometries and just work with the table, so i tried pulling everything local.
following your httr example, i'll see if i can pipe the wrangling server-side, and avoid both rate limiting and unnecessary downloads   )

I’d strongly suggest you don’t do that! You’ll be inadvertently reinventing this package. Do you need the geometries immediately?
Do you have more detail to the problem you can provide? I’d be happy to help you through this :)

the arcgislayers package is great, so definitely not looking to re-invent anything!
for this task, i just need a list of unique values from a single field in the attribute table, not any geometries.
in the big picture, our organisation is hoping to rely more on feature services for access to authoritative datasets. i'm in the process of assessing pros and cons. feature services are great for visualisation, and some types queries are well supported. however, for large-scale spatial analysis, if rate limiting is common (and highly variable) amongst host settings, downloading static datasets may still be required? i'm optimistic this will change. for example, with imagery, using STACS, it's easy to select large AOIs, bands, time periods and project CRS all server side, then just pull down only what's required.

Why don't you try this:
library(arcgislayers)

furl <- "https://mapprod3.environment.nsw.gov.au/arcgis/rest/services/Planning/EPI_Primary_Planning_Layers/MapServer/2"

flayer <- arc_open(furl)

res <- arc_select(
  flayer,
  geometry = FALSE,
  fields = c("LAY_NAME", "LAY_CLASS")
)

dplyr::count(res, LAY_NAME, LAY_CLASS, sort = TRUE)
#> LAY_NAME                                     LAY_CLASS     n
#> 1      Zone                             Public Recreation 17047
#> 2      Zone                    Environmental Conservation  8172
#> 3      Zone                                Infrastructure  6600
#> 4      Zone                      Environmental Management  4576
#> 5      Zone                            Primary Production  3192
#> 6      Zone                               Rural Landscape  3093
#> 7      Zone                       Low Density Residential  3037
#> 8      Zone                    Medium Density Residential  2605
#> 9      Zone                                  Local Centre  2504
#> 10     Zone            National Parks and Nature Reserves  2233
#> 11     Zone                           General Residential  1560
#> 12     Zone                         Large Lot Residential  1556
#> 13     Zone                          Environmental Living  1453

thanks, josiah!
the 'geometry=FALSE' flag for the arc_select ( ) function is just what i needed for restricting the query to table attributes   )

I'm glad that helped! There is also the possibility of using returnDistinctValues argument in the rest API but that doesn't work.
So I'm going to reopen this issue and rename it.
library(arcgislayers)

furl <- "https://mapprod3.environment.nsw.gov.au/arcgis/rest/services/Planning/EPI_Primary_Planning_Layers/MapServer/2"

flayer <- arc_open(furl)

res <- arc_select(
  flayer,
  geometry = FALSE,
  fields = "LAY_CLASS",
  returnDistinctValues = "true"
)

this is something that should work. we could support it directly by adding a new argument to the function

even better, josiah! thanks again   )


Force decluttering of markers at a certain zoom level

I'm coming to PruneCluster from Leaflet markerCluster, and the performance is much, much better. What I am building is a simple mobile app with about 1000 markers, and the difference in performance is very noticeable on phones/tablets.
But there is one feature of markerCluster that I can't yet find a solution to in PruneCluster: the disableClusteringAtZoom option. That is, I would like to be able to have the map disable clustering after a certain zoom level.
How would I go about doing this?

Hello,
A disableClusteringAtZoom feature is not available. The easiest solution is removing the PruneCluster layer and adding the Leaflet markers directly.
It's also possible to do that in the ProcessView method, but this method is already huge so I think a separate component is better.
I will add this feature on the TODO list, I think it can be useful in some cases. But if you want something working before a long time, I suggest to remove the PruneCluster layer and add the leaflet markers directly when you are zoomed. It's not the fastest solution, but with 1000 markers it should be ok.

:+1: I'd also need this feature already with two projects.

You can also reduce the size of the grid when zoomed. I will try this approach first.
If you really want to display all the markers on the map, you might need something like this : https://github.com/jawj/OverlappingMarkerSpiderfier-Leaflet

Thanks @yellowiscool for the quick feedback. Reducing the size of the grid can probably do this, that's what I was thinking at first.
My scenario is the following: I have a search tool on the map, once users click on a search result, I map.setView() to center the map to the selected marker and zoom to it. In most cases, this works fine, but in some, the selected marker is inside of a cluster with 2 or 3 markers.
Which makes me think of yet a different approach: maybe disable clusters if there are less than x markers in it (2 or 3 in my case)?

Yes selecting a marker is not satisfactory in the current version. In my application when a marker is selected I remove (or filter) it from PruneCluster and add it again directly as a Leaflet object. But it's not perfect because it can hide the cluster just below.
I need to think about how to select a marker easily, even when it's in a cluster. The selected marker should also be draggable, be able to open popups…
Disabling the cluster when it has 2 or 3 markers is a good idea, but it's not a perfect solution. If you still have a cluster with a small grid and a high zoom level, it's often because the markers are exactly at the same position or are very close. Also the current algorithm doesn't allow this without a big performance cost and a lot of changes (everything is a cluster in the current implementation).

Quick update: for now, I have fixed this issue by changing the maxZoom on the map. It used be set at 19,  which was a little too deep, and now it's at 17. PruneCluster respects it. Some markers that are too close, get the spiderfied treatment.

+1.  I'd really like to see this feature too.

+1 ;)

First of all, you may want to set zoomAnimation to false to avoid evil users clog your app. Then:
map.on('zoomend', function(){
 var oldsize = pruneCluster.Cluster.Size;
 pruneCluster.Cluster.Size = (map.getZoom()>= mymaxclusterzoom)? -1 : oldsize;
 pruneCluster.ProcessView(); // looks like it works OK without this line
})

Does it work for you? You set -1 as grid size, which is a value I haven't think about when I wrote the algorithm. I quickly tried it and it doesn't work with 100 random elements.

You are right, old value is overwritten this way. I use a hardcoded value of 120 (default one), and it works fine this way:
map.on('zoomend', function(){
 pruneCluster.Cluster.Size = (map.getZoom()>= mymaxclusterzoom)? -1 : 120;
 pruneCluster.ProcessView(); // looks like it works OK without this line
})

To use a user selected size, we should store the value outside the event function:
// somewhere else
 mysize = 120;

map.on('zoomend', function(){
 pruneCluster.Cluster.Size = (map.getZoom()>= mymaxclusterzoom)? -1 : mysize;
 pruneCluster.ProcessView(); // looks like it works OK without this line
})

Here's a similar workaround that works as long as there are no clusters with exactly the same latlng.
var decluster = false;
function clusterMarkers() {
if(pruneCluster){
pruneCluster.Cluster.Size = .0000000000001;
pruneCluster.Cluster.Margin = .000000000001;
pruneCluster.ProcessView();
}
}
function declusterMarkers() {
if(pruneCluster){
pruneCluster.Cluster.Size = 35;
pruneCluster.Cluster.Margin = 10;
pruneCluster.ProcessView();
}
}
leafletMainMap.on('zoomend', function () {
if (leafletMainMap.getZoom() > 9) {
if (decluster === false) {
clusterMarkers()
decluster = true;
}
}else{
if (decluster === true) {
declusterMarkers()
decluster = false;
}
}
});

@AbelVM solution isnt suitable for large number of markers. When going from declustered to clustered views, there's a noticable lag since a lot of markers are being rendered while the switch hasnt been made yet.

Looking forward for this nice feature.

any news on this feature?


s2i not work on windows

I've try s2i in windows here is command
PS H:\> c:\app\s2i\s2i build https://github.com/sclorg/s2i-php-container.git --context-dir=7.1/test/test-app centos/php- 71-centos7 php-test-app
Here is the out put

Submodule 'common' (https://github.com/sclorg/container-common-scripts.git) registered for path 'common'
Cloning into 'C:/Users/Pornchai.R/AppData/Local/Temp/s2i669970723/upload/tmp/common'...
Submodule path 'common': checked out '0ed2139bd3ef4758cdc715ba4ebc3c45f648372e'
Build failed
ERROR: An error occurred: fork/exec C:\app\Git\cmd\git.exe: The directory name is invalid.

I suspect this might be an issue w/ the symlinks used in that repo not working well on windows, can you try it w/ a flat repo like https://github.com/sclorg/cakephp-ex ?
s2i build https://github.com/sclorg/cakephp-ex centos/php-71-centos7 php-test-app
alternatively you can try git-cloning the repo manually and using the "--copy" flag w/ s2i to point it to a local dir and not use git to retrieve the content.

Thanks a lot for your solutions.
I've try the flat repo and it just work!
I will try the copy solution and update again!!