Avoid declaration invariant on HStatic

Try to avoid the need for calling [Element.declaration] before creating a [HStatic].

Will be obsolete with the new backend.


crop cannot work with tint in canvas mode

This Issue is about (delete as applicable)

A bug in the API

For example:
    pic = game.add.sprite(0, 0, 'trsi');
    pic.tint = 0x8CA7E9;
    cropRect = new Phaser.Rectangle(100, 100, 128, 128);
    pic.crop(cropRect);

The code has different behaviors under CANVAS mode and WEBGL mode. In CANVAS mode, the rectangle's (x, y) doesn't work, and pic is always cropped from its top-left.
As for the Phaser source code,
renderSession.context.drawImage(this.tintedTexture, 0, 0, cw, ch, dx, dy, cw / resolution, ch / resolution);
(https://github.com/photonstorm/phaser/blob/dev/src/pixi/display/Sprite.js#L495)
it seems that the crop's cx, cy are always 0.
So should I always not use a tint when I need to crop in canvas mode, or it can be fixed so that it has the same behavior as the WEBGL mode?
Thank you.

Using your example I'm not getting the results you're seeing.
the following code:
var sprite = game.add.sprite(20, 20, 'phaser-dude');
sprite.tint = 0x33cc00;
var rect = new Phaser.Rectangle(8, 10, 11, 30);
sprite.crop(rect);

Gives me this for WebGL:

And this for Canvas:

Using the Phaser Examples: http://phaser.io/examples/v2/sprites/dynamic-crop#gv, and adding a tint to pic:
function create() {

    pic = game.add.sprite(0, 0, 'trsi');

    w = pic.width;
    h = pic.height;

    pic.tint = 0x33cc00;

    cropRect = new Phaser.Rectangle(0, 0, 128, 128);

    pic.crop(cropRect);

}

Under CANVAS mode:

And under WEBGL mode:

BTW, I'm using Google Chrome 49.0.2623.110 m

Thanks for submitting this issue. We have fixed this, and the fix has been pushed to the dev branch.


High Inference time on warmup state in android

System information

OS Platform and Distribution (Windows 10 Pro/Andriod 10,11):
Mobile device (Redmi note 8)
TensorFlow installed from (source or binary): tflite .so built from source
TensorFlow version: 2.4
Python version: 3.8

Problem: I am working on Real time Audio processing application in android studio. I made a model using LSTMs and converted the model to the TFLITE format. After that when I tested model by giving some data it gives inconsistent inference time every time. Model inference time is 3-4ms. If I run inference in a loop, 3-4ms is the average inference time. If I shift towards the real-time scenario, I have to perform inference every 10ms. My inference time increases in this scenario. This is probably because the inference is not running back to back but actually waits till we get audio data again (10ms callback -3ms inference time = 7ms wait in thread for more audio data). I read on tensorflow site (https://www.tensorflow.org/lite/performance/measurement) that this is due to the warmup state and steady state. For warmup state it gives 9ms inference time and on steady state it gives almost 3ms inference time. But in my case, since audio is coming continuously and I want the model to run in steady state. Is there any way to run the TFLITE model in steady state once its loaded in the application ?
Describe the current behavior
Android tflite model warmup everytime when inference function is called
Describe the expected behavior
Android tflite model should warmup once at start and stay in steady state after that.
I run the model on tensorflow benchmark tool with and without delay. When I run model without any delay it gives avg 2ms time for inference. When I put 5ms delay between each inference then model inference time inreased to 6ms. Please refer to screenshots below:
Without any delay

With 5ms delay between each inference

Any help is greatly appreciated, thank you!

@srjoglekar246 could you take a look?

@impjdi @ymodak Please reveiw

@Humza1996,
In order to expedite the trouble-shooting process, could you please provide a minimal code snippet so that we can reproduce the issue on our end. Thanks!

@amahendrakar Please find below
loadModelFile is loading function
If I use tflite.runForMultipleInputsOutputs(inputs, outputs); it gives high inference time i.e 13-14 ms
But when I run this line in for loop it gives me around 3-4ms inference time. Althou I have declared tflite outside the while loop.
`private MappedByteBuffer loadModelFile() throws IOException {
AssetFileDescriptor fileDescriptor = this.getAssets().openFd("lstm.tflite");
FileInputStream fileInputStream = new FileInputStream(fileDescriptor.getFileDescriptor());
FileChannel fileChannel = fileInputStream.getChannel();
long startOffSets = fileDescriptor.getStartOffset();
long declaredLength = fileDescriptor.getDeclaredLength();
return fileChannel.map(FileChannel.MapMode.READ_ONLY, startOffSets, declaredLength);
}
Interpreter tflite;
float[][][] model_output;
float[][][][] model_st;
private void foo() {
Thread bar = new Thread(new Runnable() {
@Override
public void run() {
try {
tflite = new Interpreter(loadModelFile());
} catch (Exception e) {
e.printStackTrace();
}
while (true) {
float[] data = network.getData(); // blocking call // Audiorecord is being used in the audioHandler Object // Data is returned every 10ms
                Object[] inputs = {data, states};
                outputs.put(0, model_output);
                outputs.put(1, model_st);

                /////////// Inference ////////////////
                tfLiteLock.lock();
                try {
                    tflite.runForMultipleInputsOutputs(inputs, outputs);
                } finally {
                    tfLiteLock.unlock();
                }
                System.out.println("inference_time",""+tflite.getLastNativeInferenceDurationNanoseconds())

                //Computing
                //for(int i=0;i<500;i++){

                //try {
                //  tflite.runForMultipleInputsOutputs(inputs, outputs);
                //} finally {
                //  tfLiteLock.unlock();
                //}

                //System.out.println("inference_time",""+tflite.getLastNativeInferenceDurationNanoseconds())
                //}

                foo.bar(model_output);
            }
        }
    });
    bar.setPriority(Thread.MAX_PRIORITY);
    bar.start();`

@amahendrakar any update ?

I replied at the same question posted at https://stackoverflow.com/questions/66882466/high-inference-time-on-warmup-state-in-android-studio

@abattery Thankyou for your feedback but I am not able to solve the issue. Model is working fine when its run in a continuous loop (without any delay). Pipeline is, app gets voice data  and take its FFT. Then pass it to the model. Model outputs some data and I take IFFT of that data. Issue is , when there is a delay of getting microphone data, model takes long time for prediction as well. I guess model terminate itself or go on sleep. I want , model should take same time for everytime irrespective of input delay.

Model does not terminates and goes on sleep unless the inference object is destroyed. Instead, the related memory space can be evicted from the cache menory, which can impact the latency.
TFLite model is just another program so it also is under controlled by CPU scheduling and its performance is related to the memory usage patterns. So, in my view point, it is kind of complicated problems and hard to analyze the overall behaviors with the limited information. To get the consistent result, I suggest doing some more experiments on the runtime environments, model's characteristics, and threading strategy.
Also you can relay on the other performance techniques, model optimizations and hardware acceleration paths in order to reduce overall latency.

@abattery I have tested all optimizations technique to the model. Now my model is fully optimized and in Quantized form.

@multiverse-tf any thoughts on https://github.com/tensorflow/tensorflow/issues/48190#issuecomment-812831324 ?

@rjpower @gbaned Please check this comment

I'm not sure, this looks TF-Lite specific, so perhaps someone from that team can chime in.
My guess for why the performance is different: by sleeping between inferences, you are letting the CPU state drop back to idle. When the inference request starts, the task then has to wake up before it can run the inference. So this is giving you an idea of how much time the model takes if it is starting "cold".

CC'ing @multiverse-tf who might know more.

@multiverse-tf Please see the comment

@multiverse-tf Please see the comment

Acked. I agree w/ @rjpower's comment above that "sleeping between inferences, you are letting the CPU state drop back to idle" and then the "CPU" starts "cold" in terms of CPU frequency or execution being gradually migrated from little core to big core etc. Note CPUs on mobile phones are prone to being throttled to save power consumption.
To validate this idea, if possible, could you lock the CPU frequency and set the cpu affinity (see here for reference) and check the perf. again?
Btw, I noticed "tfLiteLock.lock()" and "tfLiteLock.unlock();" in the code you pasted above. Just wondering whether the locking here will introduce additional overhead here or not.


Add support for org and enterprise level alerts

Currently, the action only supports the octokit.dependabot.listAlertsForRepo(...) function.
Would be nice to support the organization and enterprise editions as well.
Instead of having to define a workflow for all repos in all organizations in the whole enterprise, one could simply create one single workflow in an org-level repo (.github) to schedule a weekly report for one or more organizations (or for the whole enterprise).

@mauritz-lovgren - this sounds like a great feature. I'll see how to plan this, thanks for the suggestion!

Great, thank you! 👍


feat: Added new proposed links to the sidebar

#1857
Fixes #[replace brackets with the issue number that your pull request addresses].
Changes proposed in this pull request:

Added two sublinks to Patients link
One to display Patients List, the other one to display Patient Creator Module
Added two sublinks to Appointment link
One to display Schedule and the other one to display the Appointment Generator module
New added links are highlighted when these modules are displaying
Links at the top navigation bar disappear when the Sidebar is available, otherwise, they'll show up.
Added expansion UI dynamics to the Patients and Appointments link

hi!, @jackcmeyer just uploaded the solution, would appreciate a lot that you guys check it out.

Thanks for your contribution!


Add docs folder

Resolves #51 Add GitHub Pages Documentation
Details

Adds docs folder with Jazzy generated documentation
Updates dependencies
Updates Fastfile to generate documentation

Notes
The GitHub Pages won't work until the /docs folder is merged into master and the repo settings are updated.

Codecov Report

Merging #52 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #52   +/-   ##
=======================================
  Coverage   59.67%   59.67%
=======================================
  Files          44       44
  Lines         548      548
=======================================
  Hits          327      327
  Misses        221      221

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 a096e26...5dccbc7. Read the comment docs.


Feat(extras): Add Database Support

Added Dadbod UI for Database Support, As far as i know it works with either .env file present or With and Environment Variable present for that

Similar to #1740.

so you mean this wont get approved?

No, I'm not saying anything.  The maintainer makes the decision if something gets approved or not. I'm just noting that there's already a quite similar PR. No idea which one the maintainer will prefer.

You also synced your forked repo with main LazyVim repo and your PR now contains changes irrelevant to this. You should try to use different branches instead of the main for future PRs and only update your main branch in your forked repo, so your other branches with your PRs don't get polluted by irrelevant changes.

No problem. I just mentioned it for future reference. It'll be easier if you use separate branches for your PRs in your forked repo and only sync your main branch with LazyVim's repo.
Again, I'm just a regular user like you. So, no need to apologize to me. It just doesn't feel right. I'm not saying these things to scold you. I'm just mentioning these, so you're aware in the future.

edgy.nvim should also have
optional = true,

in there since it's an extra

No problem. I just mentioned it for future reference. It'll be easier if you use separate branches for your PRs in your forked repo and only sync your main branch with LazyVim's repo.
Again, I'm just a regular user like you. So, no need to apologize to me. It just doesn't feel right. I'm not saying these things to scold you. I'm just mentioning these, so you're aware in the future.

thanks for the tips , i will keep this in mind when doing something next time

im gonna close this pr and and reopen it with only relevant changes since all of the features for Dadbod ui are integrated into it

#3425 Check these changes now the history is much cleaner


[9.x] Present exception before logging

Currently, the logged exceptions have a comprehensive stack trace including a lot of called methods in the Laravel core which makes it hard to scroll through and filter out the application code by eye.
What we need in our application level (App\Exception\Handler@presentException) is to remove the lines containing:
vendor\laravel\framework\src\Illuminate\Pipeline\Pipeline.php
and
vendor\laravel\framework\src\Illuminate\Routing\Router.php
and etc...
Exploding the string by "\n" then loop through all lines, modify them, and imploding by "\n" at the end.

Sample fluffy stack trace:
laravel-2022-07-13.log

Sample affiliated stack trace:
laravel-2022-07-13-after.log

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


Command line argument for customizing the location of the UI web assets

In the new Rust version the location of the UI web assets is expected to be a public folder inside the current working directory. We need a command line argument to customize this location. Some possible names: --ui-path. --ui-dir, --assets-dir, --frontend-dir.

@shaunmulligan

Implemented in #114


feat(autocomplete): Custom template support for autocomplete

Add support to render the suggestion for autocomplete using custom template. Add a new option for directive auto-complete, template-html-url, this is the url to load template html. If the option is not defined, the suggestion is using the default html. Otherwise, it is using the custom html. Custom html might look like  “{{text}} - {{author}}”. Template could be defined in html by using “”
Issue #99

Changes Unknown when pulling 0cb50a9c5c094198472fe7a07c27329d1a3125fa on cheonhyangzhang:frank-custom-template into ** on mbenford:master**.

The template is a good idea, I think It should be extended to tempting form an angular template so that html is allowed with angularJS data binding.   I'll start working on this if no body has worked in progress

Are you suggesting providing an option like template-html, so that you can literally pass in the custom template html code into this option?

https://github.com/mbenford/ngTagsInput/issues/99
Something like what angular uses for directives and include on html angular include
https://thinkster.io/egghead/templateurl

@cheonhyangzhang Thanks for sending your contribution! :+1:
I'm working on bringing custom template support to the autoComplete directive, and I'm following a different approach than yours. The main difference is that I'm making Angular do the heavy lifting for me. :) Check out this prototype to understand it better.

Yep. Got it.   I am not a master at Angular and might do things not in the best practice way.
I am looking forward to see your changes.

I've pushed my changes to a remote branch and I wonder if you could take a look at the feature and share your feedback? Please read my comments here for more information.

I'm closing this PR since this feature has been implemented in b550b1190509e399742f32abb6299c179fe7bae1.


Add the survey analysis conducted by stack overflow in 2023

Is your feature request related to a problem? Please describe.
The current analysis in the repository only includes data up to 2020 and the analysis of 2021 and 2022 is currently being updated, while Stack Overflow surveyed in 2023. This creates a gap in the analysis timeline and hinders the ability to provide insights based on the latest data.
Describe the solution you'd like
I would like to address this issue by integrating the dataset for the Stack Overflow 2023 survey into the existing analysis. This involves obtaining the dataset, cleaning it to ensure consistency with previous years' data, merging it with the existing dataset, and updating visualizations to incorporate insights from the 2023 data.
Describe alternatives you've considered
An alternative solution could be to continue the analysis without the 2023 data, but this would result in incomplete insights and potentially misrepresent trends in the Stack Overflow community.
Additional context
The integration of the 2023 dataset will provide a comprehensive analysis spanning from 2018 to 2023, allowing for a more accurate understanding of trends and patterns over time.
What problem is this feature trying to solve?
This feature aims to address the gap in the analysis timeline by incorporating the dataset for the Stack Overflow 2023 survey, ensuring that the analysis reflects the most up-to-date information available.
How do we know when the feature is complete?
The feature will be considered complete when the dataset for the Stack Overflow 2023 survey has been obtained, cleaned, merged with the existing dataset, and visualizations have been updated to incorporate insights from the 2023 data.
Data source :- https://survey.stackoverflow.co/
Please assign this task to me.

Hi @deepti-50 i have assigned this issue to you. I wanted to integrate 2023 dataset.
I know this involves bit of coding and time so assigned the label3 which is 45 points towards leaderboard.

Sure @deepti-50


In code rule and feature mismatch when defined

alpha_users = Togls::Rules::Group.new([1,23,], target_type: :user_id)
Togls.release do
  # Should raise an exception that target types do not match
  feature(:foo, 'desc', target_type: :user).on(alpha_users)
end

We need to raise an exception when the target types of a rule and a feature do not match correctly. This will help ensure the in code state of the toggle to be correct.

This feature seems to have already been implemented see, https://github.com/codebreakdown/togls/blob/master/spec/togls_spec.rb#L210-L242.
Therefore, I am closing this issue.


infrastructure limits and security implications

security:
please do not use any:

ssh connections
ssh keys
sudo ALL
single system and php user
undefined acl
writeable folders
executable awscli
IMDSv1

infrastructure:
please do not limit to nor use:

only single region
only 2 AZ in region
bastion hosts
parameters/ip sync

add more randomness to parameters and variables.
many devs will use it for production and deployment own shops, adopting and replicating these issues.
also you create dual ALB with IGW to internal private network and with CloudFront, doesn't looks like quick start reference infrastructure.

Hi @magenx
Many thanks for the feedback, We updated these items in develop that has since been merged.
Please let us know how you find the setup.

you only removed some sudo config,
but other elements not fixed.
and it looks like you have magento setup running from user_data.


Print active version after "use" command

I think it's good to have a feedback when running a command line, and in our case specifically when using keywords like latest or lts.

Perhaps behind a --verbose flag or fin_verbose config?
I find too much feedback rather annoying. If fin fails, it should tell you so, of course, but as usual in Unix, showing you nothing is how you know everything ran fine.

Ah, one more thing. Please note that, fin is way more opinionated than others like nvm. While nvm is a great tool that supports a large variety of use cases, I want fin to be more pragmatic, get out of my way and always (try its best to) do what the user really wants.
The CLI is similar to tj/n, which I found much more convenient and straight-forward than nvm, but without all the dangerous things tj/n does like writing to /usr/local/bin (which requires sudo), etc.

showing you nothing is how you know everything ran fine

I can agree when running "fin 5.5.0", but when running "fin latest" I feel the contract is not fulfilled without specifying which version fin consider the latest (same for other keywords). Thoughts?

Perhaps behind a --verbose flag or fin_verbose config?

I'd prefer the config way, because needing to add --verbose on each command would discourage IMHO ;)

Added a fin_verbose check :)

Thanks! @yohanboniface, introducing a verbose mode would require revisiting most of the source code and update it accordingly.
Here is what I came up with:

I still need to improve the CLI in the special use case of switching to latest or lts when that version is already installed (in the cache) so we don't have a chance to display the humanized URL.


Emoji with underscores aren't always displayed properly

If you create markdown slides that look like this (minimal case to reproduce):
:money_with_wings:

Then I'd expect to see a single slide with the :money_with_wings: emoji.
Instead, it's translated into:

:moneywithwings:

I suspect this is happening because the markdown processor looks "into" the emoji when it should respect the boundaries of the emoji.
The same thing happens with emoji that have a single underscore but appear on the same line:
:blue_heart: :+1: :blue_heart:

Should be :blue_heart: :+1: :blue_heart:, but instead the opening _ and closing _ become s:

:blueheart: ... :blueheart:

Yeah. reveal-ck's markdown processor was finding the _ and presuming it was the beginning of emphasis.

This was fixed in release 0.6.1.


Nonsticky-session support: touching unmodified/not-updated sessions must be solved differently

From martin.grotzke on January 03, 2011 01:50:56
For sticky sessions there's an optimization that sessions are only updated in memcached if the session content has changed.
Such sessions are "touched" in memcached regularly (some time before they would expire), this makes use of tomcats background processing and the locally stored sessions.
For non-sticky sessions this does not work as there are simply no sessions stored locally, therefore this must be handled differently.
Original issue: http://code.google.com/p/memcached-session-manager/issues/detail?id=74

From martin.grotzke on January 04, 2011 13:14:01
Related:

https://code.google.com/p/memcached/issues/detail?id=110
http://groups.google.com/group/memcached/msg/6065589855271fcf

From martin.grotzke on January 13, 2011 23:51:21
For each request it must also be provided that the lastModified timestamp is updated, e.g. it's saved under a separate key (which would have to be used for validation checking).

From martin.grotzke on January 22, 2011 12:42:35
This is implemented now (in non-sticky branch).
One part of the solution is that session's maxInactiveInterval, lastAccessedTime and thisAccessedTime are stored separately in memcached. This information is updated in memcached for each request to know exact session validity information.
Additionally non-sticky sessions are stored in memcached using the double session timeout for expiration in memcached. It must be provided, that a session in memcached has no shorter expiration than session timeout, so a session is stored in memcached (even if it was not modified) if this requirement were not be fulfilled.
Status: Fixed

From martin.grotzke on January 27, 2011 14:36:22
(No comment was entered for this change.)
Labels: Milestone-1.4.0


CHEF-4538 - add an option for gateway_identity_file that will allow key-based authentication on the gateway.

Add an option for gateway_identity_file that will allow key-based authentication on the gateway.
Description
Rebase of https://github.com/chef/chef/pull/1651
Issues Resolved
N/A
Check List

[x] New functionality includes tests
[ ] All tests pass
[ ] RELEASE_NOTES.md, has been updated if required (not required for bugfixes, required for API changes)
[ ] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco

Not sure how the DCO works since i am not the origional author of the commit in this PR

Darn, you beat me to it! :smile: I just got done setting up to resolve those conflicts when I saw your branch open. Don't hesitate to poke me if you think I can do something to help. I'm very much looking forward to seeing this merged!
Thanks a lot by the way!

WRT DCO, the original PR was submitted under a CLA, so this is fine as is.

Fixed 2 nits
Sounds good WRT the CLA


python setup.py install

When I run this command, it shows:
python: can't open file 'setup.py': [Errno 2] No such file or directory.
Why occurs this problem?

hello，I met the same question. Have you resloved it? How can I get the setup.py?


[zh] sync /kubeadm/ha-topology.md

Performed actions below:

sync two images
minor changes

in https://kubernetes.io/zh-cn/docs/setup/production-environment/tools/kubeadm/ha-topology/
See preview please.

/lgtm
/approve


Adding eureka aware Loadbalancers that make use of new notification mechanism from eurekClient

NetflixOSS » ribbon » ribbon-pull-requests #63 FAILURE
Looks like there's a problem with this pull request

NetflixOSS » ribbon » ribbon-pull-requests #64 FAILURE
Looks like there's a problem with this pull request

NetflixOSS » ribbon » ribbon-pull-requests #65 SUCCESS
This pull request looks good

NetflixOSS » ribbon » ribbon-pull-requests #67 SUCCESS
This pull request looks good

NetflixOSS » ribbon » ribbon-pull-requests #68 FAILURE
Looks like there's a problem with this pull request

NetflixOSS » ribbon » ribbon-pull-requests #69 SUCCESS
This pull request looks good

NetflixOSS » ribbon » ribbon-pull-requests #71 SUCCESS
This pull request looks good

#276 contains an alternate impl that's probably cleaner.

Closed in favour of #276


Fixed Issue #5

Used a different method to determine if iPhone 6+
Reference:
http://www.appcelerator.com/blog/2014/07/new-in-alloy-1-4-custom-query-styles/
It seemed inside of the device it was not consistently return true for iPhone 6+

Thanks @WillDent So the problem wasn't specific to the device as I thought. You're simply detecting the iOS version and keying off that. I tested on a co-worker's iPhone 6s Plus and my iPhone 5s as well as iPhone 6 sims and it seems to work on all of them. Thanks for figuring that out!


pep8speaks integration

I noticed pydicom is using https://pep8speaks.com/, which is quite nice -- see example in https://github.com/pydicom/pydicom/pull/627#issuecomment-383238169.
@JoostJM should we add it to pyradiomics as well?

:+1:
For reference:

https://github.com/NeurodataWithoutBorders/pynwb/blob/dev/.pep8speaks.yml
https://blog.kitware.com/how-to-improve-software-quality-in-open-source-projects-part-iii-code-coverage-documentation-and-style/

That said, setting up flake8 as a step within CircleCI is also useful. For example, see https://github.com/NeurodataWithoutBorders/pynwb/blob/f749097718cf344f4c95de7771a1ef523f26762f/.circleci/config.yml#L84-L92

The advantage of pep8speaks is that it is basically instantaneous, while CircleCI can take some time to schedule.

The advantage of pep8speaks is that it is basically instantaneous

Setting up a flake8 job as a part of a CircleCI 2.0 workflow will also report quasi-instantaneous results. For example, in this project results come back in 23s.
FYI, we are now getting rid of pep8speaks, it is using the less complete tool pep8 and doesn't accept the same parameter as flake8.

CircleCI 2.0 workflow will also report quasi-instantaneous results

Are you saying it will run this analysis without waiting for the job to start?

@fedorov, @jcfr,
For the release of 2.0, I updated the CircleCI to use workflows. I did not split-out the flake8 tests, but I did transfer the testing of the notebooks to a separate job. This moves the slowest part of the testing to a moment after all the other tests have run, including flake8.
The testing which includes flake8 now returns in less than a minute, which is fast enough for me.


switching a branch from the list of branches fails

Branch: multi-remotes (loaded by Metacello)
How to reproduce:

clone repository: https://github.com/tesonep/pharo-talents.git
select this repository
go to tab "Branches"
select branch origin/talentsWithFullHierarchy
right click for the context menu
select "Switch to this branch"
The RedBoxOfDeatch is displayed instead of branches table with the error:
"LGit_GIT_ERROR: Reference 'HEAD' is not a local branch."

I can switch branches from the context menu on the repositories list.
But if I switch to origin/talentsWithFullHierarchy and then try to "Pull incoming commits" from the context menu of the repositories list, I get the same libGit error

git status
HEAD detached at 6e04f56
nothing to commit, working directory clean

fixed on issue #292


Automatically join new public channels

By default, new public channels are joined by all members. At channel creation, channel creator can choose not to.

hello there! thanks for taking the time to suggest this feature, but that would be related to the webapp repo. Can you please reopen it here (webapp shares issues with the server repo)
another option would be to add it to our uservoice list of suggested features
I'm closing the issue, but feel free to reopen if you think it is actually relevant for the desktop app.


USHIFT-2260: Add support to run Stress Testing conditions from Jenkins

Which issue(s) this PR addresses:

Closes Jira: https://issues.redhat.com/browse/USHIFT-2260

/lgtm


Deprecation warnings after updating to TypeScript 4.8

Hi!
After updating my project to TS4.8, I started getting some deprecation warnings logged into my console:
70% sealing React Docgen Typescript PluginDeprecationWarning: 'createIdentifier' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: '' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createLiteral' has been deprecated since v4.0.0. Use `factory.createStringLiteral`, `factory.createStringLiteralFromNode`, `factory.createNumericLiteral`, `factory.createBigIntLiteral`, `factory.createTrue`, `factory.createFalse`, or the factory supplied by your transformation context instead.
DeprecationWarning: 'createBinaryExpression' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createExpressionStatement' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createPropertyAssignment' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createNull' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createTrue' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createObjectLiteralExpression' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createExpressionStatement' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createTypeOfExpression' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createElementAccessExpression' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createIfStatement' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createBlock' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createVariableDeclaration' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createCatchClause' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createTryStatement' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createArrayLiteralExpression' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.
DeprecationWarning: 'createFalse' has been deprecated since v4.0.0. Use the appropriate method on 'ts.factory' or the 'factory' supplied by your transformation context instead.

Is there any plan to update the plugin to remove the use of deprecated methods?

Hello, just checking in on this as it's still an issue. @hipstersmoothie

Hello,
I've tried ts-version: 4.7.4 before but works fine. And now, when I created a new repo with ts-latest-version 4.8.4, I encountered this issue. And when I tried to get back to the previous version, which is 4.7.4, the issue still there.


Function request: get_player_ids

Hello,
Thanks for this terrific package. One small feature request - a function that can make a list of all player ids (or names - but these aren't always unique - e.g. Ben Davies) for the current season.
get_player_details works great if you have the id or name, as does get_player_id if you know the player name - so what would be great is something like get_player_ids that returns all the player ids (or at least the maximum one, given they're assigned sequentially per season). Then we can do something like:
player_ids <- get_player_ids
*apply / purrr:map_*(player_ids, get_player_details)

to automatically get the details for all players at once. Currently, if I want to do that, I have to do a binary search on the player id until I get the maximum id get_player_details will accept without throwing an error.

Hi there, thank you for this comment, and I agree with the conceptual idea. I will include this in the next package update, likely ahead of the new season in August. Or, if you are willing, you can submit a pull request with the functionality, then I will review and merge. Thanks again.

Hello, thanks for the reply. Have had a quick look at the code and this seems pretty easy depending how much functionality you want to make inherent to the package. For example, in player_functions.R the function get_player_id has the line(s):
elements <- jsonlite::fromJSON("https://fantasy.premierleague.com/api/bootstrap-static/")$elements

I'm completely new to JSON and the FPL API so I didn't realise that that returns the info for all players for the current GW. That then means that nrow(elements) will return the total number of players - and this can then be used in my example above. e.g. something like:
# I haven't actually run this so may not work as is, but roughly this:
max_id <- nrow(jsonlite::fromJSON("https://fantasy.premierleague.com/api/bootstrap-static/")$elements)
purrr::map_dfr(1:max_id, get_player_details)

As an aside - I really don't understand what FPL are doing as their API is confusing to me. For example, elements above contains completely different columns as get_player_details (which uses $history not $elements from the FPL API) - I mean, why don't they just include all columns in both so elements and history match? Aaaaanyway.
But this is where the questions come of how you want to do this.

It could be as simple as creating a function like get_player_ids which literally returns the ids (of max id) for the players (maybe a data.frame of name and id?). Obviously this can change during the transfer window(s).
It could be a function which incorporates that with actually getting the player details - e.g. another function like get_all_player_details.
Or it could be incorporated into the get_player_details function itself. Something like an all_players = TRUE argument.
What do you think?

We are now live for the 21-22 season and the function update includes a new function - get_player_ids() - which returns a data frame of all player ids and names. Should make things easier to combine datasets and work with the API data. Thanks for the suggestion!


Can we use spconv 2.x with pytorch 1.1 ?

Can we use spconv 2.x with pytorch 1.1 ? or should we should we shift to another pytroch version

The main issue is that spconv2.x can only work on CUDA >= 10.2. You can refer spconv for more details

The main issue is that spconv2.x can only work on CUDA >= 10.2. You can refer spconv for more details

I'm currently using this version, so it should be fine right ?

pip3 install torch==1.10.0+cu113 torchvision==0.11.1+cu113 torchaudio==0.10.0+cu113 -f https://download.pytorch.org/whl/cu113/torch_stable.html


Prevent "Mutating style is deprecated" warning

In witch scenario you are seeing this warnings, @alexkuz ?

Sorry, I forgot to mention I'm using<EMAIL_ADDRESS>now - I guess there is no such warning in 0.13, had no time to check it. The warning appears when I'm calling addNotification().

Ok. I'll put this PR on hold, since the idea is keep it stable with the latest stable version of React. Never nows when React API will change.
But thanks for the PR!


Added ledc devicetree node for ESP32 board.

Signed-off-by: Andrei-Edward Popa<EMAIL_ADDRESS>

@anangl - please review.


Add the date time in the video streaming to server

How to Add the date time in the video streaming to server, Is there any option by which I can do it.
Please let me know. The need is to show the date time in the streaming video.

Hello,
I deleted the diccussion because it is duplicated.
About the issue, you can do it with filters. Check OpenGl example where you can find how to send android views or text to stream using filters:
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/blob/master/app/src/main/java/com/pedro/rtpstreamer/openglexample/OpenGlRtmpActivity.java#L164
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/blob/master/app/src/main/java/com/pedro/rtpstreamer/openglexample/OpenGlRtmpActivity.java#L315


I'm sorry, my problem is still not resolved. My function start() opens session in Postgres database and I need to make this session still open. I checked the function prepareStatement, where is written "Any previous Cursor for this Connection is closed.". Is there a solution to keep the session open?

I'm sorry, my problem is still not resolved. My function start() opens session in Postgres database and I need to make this session still open. I checked the function prepareStatement, where is written "Any previous Cursor for this Connection is closed.". Is there a solution to keep the session open?
Originally posted by @AMBIENTE1 in https://github.com/codewinsdotcom/PostgresClientKit/issue_comments/711085413

Duplicate of #28.


fix(groups): Check translationString is a string

Fixes a bug we're experiencing where:

We're translating a string within a translation group - eg i18n('About')Company
There is another translation group with the same name as the string being translated - eg i18n('Company')Location
The first translation group has not been loaded into i18nConfig yet (we do this dynamically on demand) - so translations does not yet contain an About key.

In this case Line 298 appears to try and find a translation without a group (personally I would not desire this behaviour but I understand the use case), but because translations[translationKey] is coincidentally another translation group _getTranslation returns that group's object instead of a string, causing a crash later on.
This PR checks that translations[translationKey] is a string before returning it.

Coverage remained the same at 100.0% when pulling 62b662948f176fa32ad1e591ac4b6fb3f51d3f28 on simlrh:patch-2 into 6a072df36bdd4cfb839d7e6ad0a1e3f948b17990 on skolmer:master.

@all-contributors please add @simlrh for infrastructure, tests and code


Unifying Distillation Broken [Bug] [0.10]

When doing unifying distillation of [0, 1, 2] with [0, 1, 2], rather than the output being [0, 1, 2] it is
[]
2
1
0

This is probably because in the line
list1 + list2.filter { x -> list1.none { Iota.tolerates(x, it) } }.asActionResult

there needs to be brackets around the list addition so that asActionResult applies to the sum of the lists, rather than taking the first list and adding it to the second list as an action result
e.g.
(list1 + list2.filter { x -> list1.none { Iota.tolerates(x, it) } }).asActionResult

i wish all [Languages where open braces are lambdas] a very [Die]


Automate updates

This preset is very simple and does not implement anything unique so we can probably track preset-mini automatically.
Looking for help since I am not good at automation.
Tasklist:

[ ] UnoCSS releases are detected.
[ ] Dependencies are bumped and match latest UnoCSS.
[ ] Version is changed to match UnoCSS.
[ ] A test is executed to confirm preset can be used without errors (should probably ask UnoCSS team how to do this best).
[ ] New version is published to NPM automatically.

@eshimischi If you are interested in trying, i think you can go to the UnoCSS repo for reference, they have an extensive integration suite might help. The tests and examples dirs should be especially useful.
To safely deligate the update task to CI we should be sure that the plugin still still works with at least Vite every time and produces expected CSS.


Ignore MQTT Issues

For various reasons my tablet loses the connection to the MQTT server on occasion. It always reconnects, however sometimes a stack of dialog notifications get thrown warning the user that we have lost connection to the MQTT server. These events are external to the App and I can't mitigate them.
It would be awesome to have a "ignore errors" mode where you don't get a dialog, or instead these get thrown in a more passive manor.
Thanks for the consideration!

Please provide the application version you are using, its under the settings then the about option.

This happens with the latest/all versions. To be clear, this is not a bug with the app. With dialog messages (or similar) it requires intervention from me to clear 1 or many of them in a stack - even if the connection is restored.
Something like a passive status indicator off to the side might solve this.

I need to be able to reproduce the issue, so I need to know which version of the application you are using.   There was an issue like this reported and fixed many versions ago, so if you can tell me which version you see this, I might be able to verify it.

The original issue was with 0.8.1, but I just got home to a message saying "Alarm Disconnected, the network connection is unavailable".

You're original issues was that you received many stacked dialogs.   Does this still occur on v0.8.6 Beta 1 or are you seeing a single alarm disconnected message?

The message that you see will disappear if the network is connected again, most likely culprit is your phone is disconnected from the network or for some other network reason it can't reconnect with the server.   The message is valid and it's there most likely because it can't reconnect.  However, if you are seeing multiple stacked dialogs, then sure there is an issue.   I don't receive these messages, not even during a HA restart because the app reconnects back to the MQTT broker.

I received a single message, so that seems to be working as intended now.
However this still requires intervention to press OK when in the background the connection is usually back online. My original intent was to make something more along the lines of a "quiet alert" option that wouldn't require the user to actually press anything.

Can you confirm the MQTT broker is indeed back online?  The message means that it can not reconnect on its own or its given up on trying.    The user is not required to dismiss this message unless the application cannot reconnect with the MQTT broker.  That is the reason you are seeing this message.  I will look at how this message is generated and dismissed to be sure the logic is correct, but on my HA setup, the application handles the MQTT broker disconnect and connect through HA restarts without issue.

Ah, makes sense. I upgraded from 0.8.1 to latest at the same time I filed this issue. More than likely this is no longer an issue.
This weekend I'll try spinning up a second MQTT instance and see if I can replicate previous issues.

Thanks, would be great if you could test this.

I've not found any issues, we've also survived several unexpected restarts of my primary stack.
I think it's safe to say this was a false alarm. Thanks!

Awesome!!


Content in multiple sections in docs.html is corrupted

After starting the website locally, the content of multiple sections in the "Plasma 101" (docs.html) section presents itself as unintelligible gibberish.
Example:
Challenge Period
For who thoroughly her boy estimating conviction. Removed demands expense account in outward tedious do. Particular way thoroughly unaffected projection favourable mrs can projecting own. Thirty it matter enable become admire in giving. See resolved goodness felicity shy civility domestic had but. Drawings offended yet answered jennings perceive laughing six did far.

Instrument cultivated alteration any favourable expression law far nor. Both new like tore but year. An from mean on with when sing pain. Oh to as principles devonshire companions unsatiable an delightful. The ourselves suffering the sincerity. Inhabit her manners adapted age certain. Debating offended at branched striking be subjects.

Exit strategies
Whole wound wrote at whose to style in. Figure ye innate former do so we. Shutters but sir yourself provided you required his. So neither related he am do believe. Nothing but you hundred had use regular. Fat sportsmen arranging preferred can. Busy paid like is oh. Dinner our ask talent her age hardly. Neglected collected an attention listening do abilities.

The him father parish looked has sooner. Attachment frequently terminated son. You greater nay use prudent placing. Passage to so distant behaved natural between do talking. Friends off her windows painful. Still event you being think nay for. In three if aware he point it. Effects warrant me by no on feeling settled resolve.

Other sections in Plasma by example are also affected: Overview, Architecture, CLI Documentation
There might be more ... I stopped reading further.

I'm just now realizing that the entire website still contains "placeholder" texts in most of the sections. If this is still all "under construction", please consider putting a disclaimer at the beginning of the site and in the README! This repo was mentioned and promoted on social media and left the impression that it's finished (hint: not having a master branch would've also signaled that the site is not ready, yet).
You can close this ticket, if the above is correct and the site is still being built.

Sorry! Yes, this is under construction. Content will start to be published today as we try to wrap things up for the IC3 boot camp. I'll change the readme!

README updated, closing.


Update to Scylla 1.4

Remove the explicit install of collectd_exporter
Open ports 9180, 9100

close with #106


Action fails but is marked as success

If the credentials are wrong, the GitHub Action shows an error on the logs but the job status is still displayed as successful. The job should instead be marked as a failure so it would be easier to fix the credentials and re-run just the failed jobs instead of the whole flow.

Workflow usage:

jobs:
  # ...
  deployment-marker:
    name: Create deployment marker
    needs: deploy
    runs-on: ubuntu-latest
    steps:
      - name: Retrieve release version
        run:
          echo "RELEASE_VERSION=$(echo ${{ github.event.ref }} | grep -Eo '[0-9]+\.[0-9]+\.[0-9]+$')" >> $GITHUB_ENV

      - name: Create deployment marker on New Relic
        uses<EMAIL_ADDRESS>        with:
          guid: ${{ secrets.PRODUCTION_NEW_RELIC_GUID }}
          apiKey: ${{ secrets.PRODUCTION_NEW_RELIC_API_KEY }}
          region: "EU"
          version: "${{ env.RELEASE_VERSION }}"
          user: "${{ github.actor }}"

Example when wrong credentials are provided:

Log shown

level=fatal msg="Argument \"deployment\" has invalid value $deployment.\nIn field \"entityGuid\": Expected type \"EntityGuid!\", found \"***\"."

Job status returned

SUCCESS ✅

Job status expected

FAILURE ❌

I am having the same issue.

cc @sanderblue

@edisonspencer, you might be doing the same thing I did. So if you are using jq. add a -r to remove the double quote for GUI in your previous graphql call.
ref: | jq -r ".data.actor.entitySearch.results.entities[].guid"

Hi @pragmaticivan , I am not using graphql nor jq. I was just using the NewRelic GitHub Actions on my workflow file as per their example. I then found out I was using the wrong key. I created this issue because the action should mark the step/job (depends where you are using it), as a failure and not allow it to be marked as success.

Edited the original issue with code example of the workflow file in order make the issue clearer.


[google_maps_flutter_web] Initial support for custom overlays.

Saves tile bytes to blobs and uses img elements to decode and render. Does not implement opacity, perform caching, or serve placeholder images.
Issue: flutter/flutter#98596
Known issues:

flutter/flutter#116132
AsturaPhoenix/trip_planner_aquamarine#22

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[x] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

For some reason this seems to be non-transiently timing out legacy_version_analyze CHANNEL:2.10.5 DART_VERSION:2.16.2.
https://cirrus-ci.com/task/4717210967474176
https://cirrus-ci.com/task/6608561120215040

Locally when I try to run against flutter 2.10.5, I get
dart run ./script/tool/bin/flutter_plugin_tools.dart analyze --lib-only --skip-if-not-supporting-flutter-version="2.10.5" --skip-if-not-supporting-dart-version="2.16.2" --custom-analysis=script/configs/custom_analysis.yaml --packages google_maps_flutter_web
Resolving dependencies in C:\Users\imagi\Documents\projects\flutter\plugins\packages\google_maps_flutter\google_maps_flutter_web...
Because every version of flutter_test from sdk depends on collection 1.15.0 and google_maps_flutter_web depends on collection ^1.16.0, flutter_test from sdk is forbidden.
So, because google_maps_flutter_web depends on flutter_test from sdk, version solving failed.

Which isn't the kind of hang we're seeing on CI.
Not sure about next steps.

Oh, I think I get it. This is its way of telling me to bump something in pubspec.yaml huh?
environment:
  sdk: ">=2.12.0 <3.0.0"
  flutter: ">=2.10.0"

Per discussion offline with @ditman, google_maps_flutter reviews have been on hold for now due to high-priority work on another plugin, but will be next on the queue. Thanks for being patient on the review here!


UX for loading data/spreads

Datasets in the "data" folder should appear on a dropdown. Loading one should determine which dataset is active.
Similarly, there should be a dropdown for different card spreads. Loading one should swap between the card layouts in main.js

I'm going to go ahead and claim this one! I'll try to pull together some sort of draft in the next day or so

@mcorrell do you have any visual references for the look of the tool you are interested in using? Or are we aiming for close alignment with the vibe of general va tools?

I was envisioning something sort of minimal and monochrome for the cards themselves, with mostly white and 333333 dark grey. And since we don't need shelves or anything beyond the two drop downs, just having minimal floating UI elements with the card spread front and center would be nice.

One issue: the "cover" card is rotated 90 degrees, so the tooltip ends up being rotated also...

Closing, as I believe this all now works as desired


Added CodeCrafters

Added CodeCrafters. Lets you recreate complex software like Redis, Git, Docker, and SQLite from scratch, providing a deep understanding of how these tools work.

@tbacalla Thanks


ensure all mpi builds of metapackage are uploaded

closes #170
this doesn't solve the problem in general because we'll need every dependency for which we have multiple variants included here, but it should be enough for now.
I took the opportunity to remove the run_constrained requirement from the components, which make a circular restriction. This is based on discussions around a possible new release of legacy fenics, and these prevent making a release of a partial package, where dolfin 2023.1 might depend on the existing dijitso 2019.1, without needing a coordinated release of everything.

To confirm the fix, check that the build ids for two builds for the same Python+platform are different, e.g.

https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=648351&view=logs&j=9a864fd9-6c8f-52ca-79ce-2aa6dca1a1de
https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=648351&view=logs&j=696704cc-6fef-57a3-ea36-f27779b8cd5e

which are both linux-64/py311

fix confirmed, should be all set when CI finishes


an inconsistent indentation in stackhourglass.py

Thanks for your great work.
When I use the Test_img.py to get the disparity img, I notice an error in stackhourglass.py at line137
        if self.training:
	cost1 = F.upsample(cost1, [self.maxdisp,left.size()[2],left.size()[3]], mode='trilinear')
	cost2 = F.upsample(cost2, [self.maxdisp,left.size()[2],left.size()[3]], mode='trilinear')

	cost1 = torch.squeeze(cost1,1)
	pred1 = F.softmax(cost1,dim=1)
	pred1 = disparityregression(self.maxdisp)(pred1)

	cost2 = torch.squeeze(cost2,1)
	pred2 = F.softmax(cost2,dim=1)
	pred2 = disparityregression(self.maxdisp)(pred2)

        cost3 = F.upsample(cost3, [self.maxdisp,left.size()[2],left.size()[3]], mode='trilinear')
        cost3 = torch.squeeze(cost3,1)
        pred3 = F.softmax(cost3,dim=1)
	#For your information: This formulation 'softmax(c)' learned "similarity"
	#while 'softmax(-c)' learned 'matching cost' as mentioned in the paper.
	#However, 'c' or '-c' do not affect the performance because feature-based cost volume provided flexibility.
        pred3 = disparityregression(self.maxdisp)(pred3)

it seem that previous line using space,this line uses tabs.
I got an error :"local variable 'pred3' referenced before assignment". I think that inconsistent indentation cause the problem.
Am I wrong or how could I improve it?
Thanks again!

@BenedictGreen
Hi, you can use spaces to replace tabs for these indentation.
If it is not working, you could past your error messages.

@BenedictGreen
Hi, you can use spaces to replace tabs for these indentation.
If it is not working, you could past your error messages.

Looks like you mix 2 tabs with 1 tab + 4 space, where your tab, surprisingly, turns out to be 8 spaces.
Do you want to fix this? It's not something hard. Or I can PR u. I have fixed it now.


Not able to attach debugger to already running nodejs web app on port 3001

Hi,
I am not able to attach debugger to the already running NodeJS web app.
Following are my VSCode launching configurations:
"configurations": [
   {
       "type": "node",
       "request": "attach",
       "name": "Attach",
       "port": 3001
   },

I am getting following error message:

Cannot connect to runtime process (reason: This socket has been ended by the other party).

Now, when I add protocol : inspector in the above configuration, it shows me the following error message:

Ensure Node was launched with --inspect. Cannot connect to runtime process, timeout after 10000 ms

I am currently using v8.6.0 version of NodeJS

Well the message appears correct to me. When you launch your node app from the command line you should pass in the --inspect flag. Also in your launch.json you are using port 3001 and in the title port 3000, make sure you are using the correct port.
Not sure if we support attaching to node that is not running in inspect mode. @roblourens can comment more

Hi,
Thanks for your response.
I have corrected the port number in the title (Issue is not related to the port number)
I am trying to run the NodeJS+Typescript+Express starter template provided by Microsoft:

https://github.com/Microsoft/TypeScript-Node-Starter

And want to attach it to the debugger after running it from the command prompt.
I am not running the app with node command, instead I am using npm start to run it.
I hope above link will help you to better understand my issue, if you can replicate it.
I am currently using v8+ version of nodejs.
Thanks,
Harsh

npm start doesn't start with the inspect flag. I see a serve-debug task which does, it looks like npm run debug might do what you want.

Thanks, I am able to attach to the debugger after changing the port from 3001 to 9229 and by using the command npm run debug
I have one more doubt, why there are so many request logs with following detail:

[Node] GET /json/version 404 1.145 ms - 151
[Node] GET /json 404 0.837 ms - 143
[Node] GET /json 404 1.050 ms - 143
[Node] GET /json/version 404 1.479 ms - 151
[Node] GET /json 404 1.640 ms - 143
[Node] GET /json/version 404 1.630 ms - 151
[Node] GET /json/version 404 1.112 ms - 151
[Node] GET /json 404 1.726 ms - 143

Awesome, glad that worked.
Are all of those just from a single start of the debugger? Hm, not sure.

Yes... but happy it worked
On 10/12/17, Rob Lourens<EMAIL_ADDRESS>wrote:

Closed #36016.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/Microsoft/vscode/issues/36016#event-1289573079


Upgrading?

HI
I cant find info on upgrading. I am using 1.6.0 on Debian Wheezy and I am not sure about how to go ahead.
thanks

Update packages?
How did you install?

Hi
To be honest I do not remember but I think I used the install script, whatever that was provided back then. It seems like it got some new updates so I am not sure about how to go about it.
Am I supposed to uninstall? I do not even know how to uninstall, did the script provide uninstall functionalty?

I am guessing that this is how I installed it then
-rwxr-xr-x  1 8.6K Jan  2  2016 dnscrypt-autoinstall.sh

https://wiki.installgentoo.com/index.php/DNSCrypt#Building

Download it again! Looks like there is an uninstall option.
You can then use the up-to-date binaries from https://github.com/jedisct1/dnscrypt-proxy-installer if you like.

Thanks guys trying it now.


Inconsistent payment types with HIDE_PAYMENT_INFORMATION

Version: 16.0.5
Date: 2016/06/08
[Test category]: creative Test
[Steps to reproduce]

Set HIDE_PAYMENT_INFORMATION to "true" when upgrading to 16.0.5
As supplier remove payment types from the customer
Subscriptions of the customer become "suspended"
Customer has to update the payment type

[Expected result]
Customer can update the payment type
[Observed result]
Customer cannot update the payment type
The purpose of HIDE_PAYMENT_INFORMATION set to "true" is to not ask customers for information, which is not required. Therefore this issue is not solved by letting the customer update the payment type. Rather hide the "Manage payment types" page for suppliers, in order to avoid the situation described above.

Please make the "manage payment types" page hidden in admin portal. Take care that it will also not be accessible with deep links, like #119.

Fixed in 272c858

Please adapt the online documentations to this change of behaviour.

Red, I just completed the documentation in English and German for this.
Please adapt the Japanese.

Adapted the Japanese documentation

verified 30.06.2016


How to obtain testfiles for CI without running into licensing issues?

Is it okay to copy the files test-data/GeoLite2-City-Test.mmdb and test-data/GeoLite2-Country-Test.mmdb into other (MIT licensed) projects ? The CC-BY-SA 3.0 license on this repository seems to forbid it (as I understand the license, it would be viral)
If not, are these or similar testdata files available somewhere  ? (so we can avoid our CI builders from hitting the download limits)

Thanks for raising this. The CC license was originally intended for the specification and we didn't reconsider it once we started adding the test databases and associated code. We have change the license to the Apache License, Version 2.0, or the MIT license, at your choice. This matches what we do for most of our other projects. Hopefully this is suitable to your use case.


Plugin not working

Uncaught TypeError: f.getClientRects is not a function
    at r.fn.init.offset (jquery-3.1.1.slim.min.js:4)
    at jquery.paroller.js:55
    at Function.each (jquery-3.1.1.slim.min.js:2)
    at r.fn.init.each (jquery-3.1.1.slim.min.js:2)
    at r.fn.init.$.fn.paroller (jquery.paroller.js:52)
    at HTMLDocument.<anonymous> ((index):411)
    at j (jquery-3.1.1.slim.min.js:2)
    at k (jquery-3.1.1.slim.min.js:2)

Same error to me.

Same here... @frank1006 older versions work as expected though...

Got the same issues and was with the last version of plugin 1.3.1... so I've downgrade to 1.2.9 and it works fine. DON'T update to latest version then.


Using fastlane with multiple schemes

Hi,
Is it possible to use fastlane with multiple schemes? I've searched the documentation and tried some different things, but it couldn't find the answer. Maybe i have to create the deliver file each time i want to upload by myself?
I'm using the same xcodeproject for multiple apps (and they have different app ID's).
Thank you for your help!
Regards,
Bas

Hi Bas,
Does this guide help you? https://github.com/KrauseFx/fastlane/blob/master/docs/FAQs.md#multiple-targets-of-the-same-underlying-app
In general @joshdholtz is an expert in that field :+1:
Also, check out this example fastlane setup: https://github.com/fastlane/examples/tree/master/SunApps

I am an expert in that field :grin: That FAQ should be enough but let me know if you have any other questions on it :blush:

@joshdholtz Is there anything different between schemes and targets when using this setup?

@KrauseFx Depends on how you have your Xcode project setup :innocent: I, personally, just keep Debug and Release as targets for all my different schemes so I never have to set that in my .env files

Ah thats a great example thank you! I'll try to make it work ;)

@bkrijgsman Great to hear :+1:

I've searched the documentation and tried some different things, but it couldn't find the answer.

So where would you expect this part of the documentation? Anything I can improve to make it easier for new users?

A link to the SunApps example :)

I successfully used the tools of fastlane and all the functions with multiple schemes.
Only one strange thing is happening: i can't upload anymore.
I got the following error message:
Return status of iTunes Transporter was 1: ERROR ITMS-4064: "File extension of file '7831df527cc49504cd3b301167effc05' is invalid for this software; use 'zip,ipa' instead."
I opened the .itmsp file in the /tmp/ folder. There are two files inhere:
-7831df527cc49504cd3b301167effc05
-metadata.xml
Does somebody know if this is normal?

7831df527cc49504cd3b301167effc05, that's odd, it should be 7831df527cc49504cd3b301167effc05.ipa.
Did this work before? Are you using gym with deliver?

Yes i do. If i change the name of the output ipa (instead of ./.ipa --> test.ipa) this bug is somehow fixed.

So does it work now in an automated manner? Or do you need to manually update it?

It's the automated. I had some problems with the code signing.
I used --xcargs PROVISIONING_PROFILE = "{UUID}" to automatically select the right profile.
I used this script to find the UUID:
def profile_uuid(provisioning_profile_path)
profile_contents = File.open(provisioning_profile_path).read
profile_contents = profile_contents.slice(profile_contents.index('<?'), profile_contents.length)
doc = Nokogiri.XML(profile_contents)
return doc.xpath('//key[text()="UUID"]')[0].next_element.text
end


shaderc is not thread safe.

So trying to compile shaders in a multithreaded manner doesn't work since i believe there is a problem in the glsl-optimizer library it maybe has some kind of shared state. My repo is that i embedded the shaderc tool into my editor and tried to compile multiple shaders on different threads. Removing the optimization pass seems to solve the issue. Im just throwing it out there. I guess you can't do much here but any ideas? It's either lock during the optimization pass or remove it all. I guess locking isn't that bad since its just the editor that will suffer.

shaderc is process safe... Since it's command line utility, you can spawn as many processes as your OS allows you. :)

ya i know :) but my case is that I dont spawn different processes but want to compile from my own process :). Never mind me :). If anyone is interested more in this here is an issue from the glsl-optimizer https://github.com/aras-p/glsl-optimizer/issues/76


Anvil API test - can_remove_pool_transactions fails on master

Component
Anvil
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
No response
What command(s) is the bug in?
cargo test anvil
Operating System
macOS (Apple Silicon)
Describe the bug
The anvil API test can_remove_pool_transactions is failing on the master branch. This is testing whether there is an initial pending transaction present.

cc @yash-atreya

@mw2000 I'm unable to reproduce this, it's working for me on the latest commit bdc04c278f8. Anyways, we're migrating this code to Alloy currently.

@yash-atreya just double checked commits and it seems to still be failing on the bdc04c2 commit for me. No worries if not able to reproduce and have future plans to migrate to alloy.

Marking as resolved as these tests have been passing since the migration to Alloy
Feel free to re-open if you are still experiencing issues with this @mw2000


Updated imports for 0.25.0.

React & ReactNative have been split in 0.25.0. This change is to add support for the split - don't merge until 0.25.0 is released (assuming this is a breaking change).

Bump. This is producing quite some warnings in RN 0.25 React has been split out for quite some time already, I think 0.18 (https://github.com/facebook/react-native/releases/tag/v0.18.0), so this should be backwards compatible. With 0.24 it did def work.

Thanks for your PR! Wanted a slightly different implementation tho. Released in 0.6.0.


Zuul won't stop retry after hystrix is timed out

I'm on spring cloud dependencies Camden.SR6, and a setup of zuul with the following config
ribbon:
    MaxAutoRetriesNextServer: 0
    MaxAutoRetries: 2
    ReadTimeout: 3000
    OkToRetryOnAllOperations: true #false to only allow get to retry

hystrix:
  command:
    sc-test-node-0:
      execution:
        isolation:
          strategy: THREAD
          thread:
            timeoutInMilliseconds: 5000

and I put a simple resource behind zuul which sleeps 4000ms when invoked.
    @GET
    @Path("getthensleep/{sleepms}")
    @Produces(MediaType.APPLICATION_JSON)
    public Response getThenSleep(@PathParam("sleepms") int sleepms){
        LOGGER.info("get then sleep {} ms", sleepms);
        try {
            Thread.sleep(sleepms);
        } catch (InterruptedException e) {
            LOGGER.error("Sleep error", e);
        }
        return Response.ok().build();
    }

and I use postman to call the getthensleep resource to let it sleep 4000ms, I expect zuul's RibbonRoutingFilter times out after 3000ms and retry until hystrix times out, 2 separate call should be made from zuul. However, after hystrix times out, zuul returns with code 500, another call is made to getthensleep resource again. In total 3 calls are made by zuul. I think the zuul should stop trying when hystrix times out, and It shouldn't make other attempt after hystrix times out.

I found this issue in the Hystrix project that is particularly interesting.
It seems like the application thread should try to respond to a thread interruption (when using an isolation strategy of thread).
Might need to dig into this a little bit more to understand it better.

Because of the way the RibbonRoutingFilter is coded, the default way of specifying THREAD isolation doesn't work. Try zuul.ribbonIsolationStrategy=THREAD. You are probably still using semaphore.

@spencergibb thank you for pointing out it's hystrix isolation issue, I tried to set
'''
zuul:
ignored-patterns: //health/,//internal/
routes:
test-node-0:
path: /node0/**
serviceId: sc-test-node-0
sensitiveHeaders: Cookie,Set-Cookie
retryable: true
ribbon-isolation-strategy: THREAD
'''
but it seems not working.

What does "but it seems not working." mean?

Hystrix uses RxJava, and when command should be failed by timeout, hystrix unsubscribes from command observable (see com.netflix.hystrix.AbstractCommand.HystrixObservableTimeoutOperator)
If THREAD isolation level is used, command thread can be interrupted. But @ryanjbaxter found issue.
For SEMAPHORE isolation level, which default for Zuul, command thread is not interrupted.
Now we have Spring's AbstractRibbonCommand. It extends HystrixCommand and implements run method. We can rewrite AbstractRibbonCommand to extend HystrixObservableCommand and implement construct method that returns observable. This will give unsubscription propagation to command implementation.
I tried to do this, see my branch. I am using ribbon's LoadBalancerCommand that also returns observable.
But I faced another problem: retryable clients. When retryable client is used, ribbons retry logic is ignored and retry is performed in client. I don't know what is the best way to make it work with retryable clients. Add smth like toObservable to client, and implement execution in react way?
Anyway, see my branch. It has tests for retry abort. If you revert all changes and leave only tests - they will fail.

My gut tells me that we can probably do something in RibbonLoadBalancedRetryPolicyFactory so that when we check if we can retry we return false because of the timeout.

I think, what I did in my branch is wrong. Retry logic should be only in client.

My gut tells me that we can probably do something in RibbonLoadBalancedRetryPolicyFactory so that when we check if we can retry we return false because of the timeout.
Yeah, but you will need to ask hystrix somehow, is command timed out.

I have a another question. What was the point to add those retryable clients? To be able to write custom LoadBalancedRetryPolicy?

What was the point to add those retryable clients? To be able to write custom LoadBalancedRetryPolicy?

Beginning in Camden we no longer used the Netflix RestClientsince it was deprecated.  The new HTTP clients we used did not have the retry logic that the Netflix client did so the retry clients were made to fill that gap.
Can you perhaps submit a PR with your changes so we can take an easier look at the code changes?

Can you perhaps submit a PR with your changes so we can take an easier look at the code changes?

https://github.com/spring-cloud/spring-cloud-netflix/pull/1793

Beginning in Camden we no longer used the Netflix RestClient since it was deprecated. The new HTTP clients we used did not have the retry logic that the Netflix client did so the retry clients were made to fill that gap.

Deprecated RestClient extends AbstractLoadBalancerAwareClient that has executeWithLoadBalancer method with retry logic. And execute method does not retry requests.
New retryable clients also extend AbstractLoadBalancerAwareClient, but overrides getRequestSpecificRetryHandler to disable logic in executeWithLoadBalancer method. And retry logic is implemented in execute method.
Comparing those implementations

method\client
RestClient
New retryable client

execute
does not retry
does retry

executeWithLoadBalancer
does retry
does retry

What is the contract for client? execute method should retry failed requests?

Yes execute should retry the failed requests, executeWithLoadBalancer calls execute.

So I did some investigation into how Hystrix communicates that a timeout has occurred to the wrapped operation and it looks like it is just interrupting the thread, so all we should have to do is to check Thread.interruped().  Here is a quick sample I put together that seems to work.
@Service
public class HelloService {
	@HystrixCommand(fallbackMethod = "reliable")
	public String getHelloString() throws Exception {
		MySimplePolicy policy = new MySimplePolicy();
		RetryTemplate template = new RetryTemplate();
		template.setRetryPolicy(policy);
		return template.execute(new RetryCallback<String, Exception>() {
			@Override
			public String doWithRetry(RetryContext context) throws Exception {
				RestTemplate rest = new RestTemplate();
				((SimpleClientHttpRequestFactory)rest.getRequestFactory()).setReadTimeout(100);
				return rest.getForObject("http://localhost:8080/hello", String.class);
			}
		});

	}

	public String reliable() {
		return "fallback";
	}
}

class MySimplePolicy extends SimpleRetryPolicy {
	@Override
	public boolean canRetry(RetryContext context) {
		if(Thread.interrupted()) {
			context.setExhaustedOnly();
			return false;
		}
		return super.canRetry(context);
	}
}

I think you should be able to apply this to the retryable clients as well.

Thank you for clarifications!
Unfortunately Thread.interruped will not fix case when hystrix isolation level is SEMAPHORE. This is critical when you use Zuul - it has SEMAPHORE isolation level. Switching Zuul to use THREAD isolation level might affect performance - you will have 2 times more number of busy threads - one servlet container thread plus one hystrix thread for each request.
I am trying to refactor clients right now, to make some general fix. Long short story is to add AbstractLoadBalancingClient.getExecutionObservable method and use it everywhere. Here is example:
	@Override
	public T execute(S request, IClientConfig clientConfig) throws Exception {
		return executeWithLoadBalancer(request, clientConfig);
	}

	@Override
	public T executeWithLoadBalancer(S request, IClientConfig clientConfig) throws ClientException {
		try {
			RequestSpecificRetryHandler handler = getRequestSpecificRetryHandler(request, clientConfig);
			return getExecutionObservable(request, clientConfig, handler)
					.toBlocking()
					.single();
		} catch (Exception e) {
			Throwable t = e.getCause();
			if (t instanceof ClientException) {
				throw (ClientException) t;
			} else {
				throw new ClientException(e);
			}
		}
	}

	public Observable<T> getExecutionObservable(final S request, final IClientConfig requestConfig, RetryHandler handler) {
		...
	}

	protected abstract T executeInternal(S request, IClientConfig requestConfig) throws Exception;

I meant to add a note to my previous comment stating that it would only work for thread isolation, sorry about that.
I think the Observable approach will work for both thread and semaphore isolation.  I went back and read this comment, which seems to confirm that approach.  Thanks for diving on this!

I updated my pull request with latest changes. First of all - OkHttpLoadBalancingClient and RibbonLoadBalancingHttpClient start retrying failed requests on execute.
Is it ok to remove RetryableRibbonLoadBalancingHttpClient and RetryableOkHttpLoadBalancingClient? I had to ask about this before, but decided to remove them, because OkHttpLoadBalancingClient and RibbonLoadBalancingHttpClient are now do same.

No it is not OK we want those clients so we can use Spring Retry to retry the failed requests and not rely on the deprecated Netflix client to do so.

Well, I am confused. Old and new retryable clients extends Netflix's AbstractLoadBalancerAwareClient. Netflix's client does retry using Ribbon. Also, new retryable clients configured via Ribbon's configuration. So it looked like those clients just replace Ribbon by custom implementation using spring retry, and make execute method retryable.
Probably, I just didn't get something or don't know about some future refactoring you keep in mind.
Anyway, see my updated pull request. All clients are doing same as before now. Basically, I just fix the issue.

I'd be very uncomfortable merging the changes you have made.

No problem. At least I showed one of possible ways how to fix the issue. Thanks.

I am going to close this issue for now.  If we find there is others who want retries to stop after a hystrix timeout we can come back and address this.

I was following your discussion as this issue is actually happening for our service in Production. As soon as services behind Zuul begin to respond slower, Zuul retries requests until some of our nodes die. :(
Since this is actually hard to fix, what should be done next? Or Spring Cloud Zuul will live with this bug?
Thanks

Are you saying the retries that dont stop from Zuul are causing your apps to crash?

It caused Zuul to crash. We had issues with network and downstream service was unreachable.
Even if call was cancelled by hystrix, ribbon was still retrying (ribbon has default timeout set to 5 secs).  This led to 10 secs response time and huge spike of busy threads on tomcat. This also led to JVM freeze and made node non-operational.

@spencergibb even ribbon-isolation-strategy is set to THREAD, retry can't be interrupted by hystrix timeout.

@ts-huyao yes because we are not checking if the thread has been interrupted

@Aloren and @rterentiev we talked about this issue on our team call today and we can to the consensus that the proper way to configure the Hystrix timeout is to make sure you account for the total timeout of all retries.  This means, for example, if you have the Ribbon timeout configured to be 1 second and you are going to retry the request 3 times that the Hystrix timeout should be at least 3 seconds.  Is there a reason why you have not configured your Hystrix timeout this way?

If I recall correctly, the hystrix timeout should be slightly greater than the max expected duration of the underlying command. So if the http client is configured with a 1s connect+read timeout and 2 potential retries, the hystrix timeout should be slightly more than 3s (to account for jvm overhead). I'm trying to find where I read that.

@ ryanjbaxter, then defaults in spring cloud config needs to be adjusted as well, 5 secs for zuul hystrix was set to override 1 sec default. WDYT?

If that is the case then yes we should make them match, however as soon as a second instance of a service is added than that timeout needs to be adjusted.
I wonder if we could automatically calculate the hystrix timeout based on the number of instances and the configured ribbon timeout....

We don't currently set the defaults for hystrix in zuul except for the max number of semaphores.

here is the list of settings which needs to match:
hystrix.command.default.execution.isolation.thread.timeoutInMilliseconds
ribbon.ReadTimeout
ribbon.ConnectTimeout
ribbon.MaxHttpConnectionsPerHost
ribbon.MaxTotalConnections
ribbon.MaxAutoRetriesNextServer
ribbon.MaxAutoRetries

By default read timeout is 5000ms and auto retry is 1...

I'm not sure we should be setting those values automatically.

There is no sense in hystrix timeout, if command can't be interrupted. I would just disable hystrix timeout by default. Maybe log some warning message, if it was enabled.

@rterentiev why do you want to want Hystrix to interrupt the command if it hasn't finished retrying the requests?

Because Hystrix will return timeout error, even if command was executed successfully after retrying. In high load applications, this will add additional load and problems described by @Aloren.
I am not pushing you to fix this issue. I was able to do workaround implementing custom command and command factory.
But if you going to not fixing it for now, I would disable hystrix timeout by default. It just make things more consistent - hystrix timeout is not working and disabled by default.

@rterentiev you wouldn't have those problems if the hystrix timeout were > the combined potential timeouts of all retries, correct?

Because Hystrix will return timeout error, even if command was executed successfully after retrying.

Right so why not configure Hystrix so it at least waits until all retrys complete?

In high load applications, this also will add additional load and problems described by @Aloren.

This could potentially be dealt with by implementing some kind of backoff policy, it would be an enhancement.

@spencergibb, correct, but default setup is broken. There is 1 sec of hystrix timeout and 5 secs of ribbon timeout.
So, by default hystrix timeout can be disabled to avoid broken behaviour. (or default ribbon needs to match 1 sec max to accommodate hystrix)

I've opened #1827 to see about adjusting the ribbon timeouts for the default apache httpclient and the okhttp client.
I've opened #1828 to document hystrix/retry/timeout relationships.
BTW, I've chatted with a netflix engineer and I am correct in those relationships. As a side note, even they struggle with this some.

Right so why not configure Hystrix so it at least waits until all retrys complete?

@rterentiev you wouldn't have those problems if the hystrix timeout were > the combined potential timeouts of all retries, correct?

You will need to keep in mind this problem. Each time you update ribbon configuration, you need to recalculate new timeout value for hystrix. Its much more simplier to  just disable hystrix timeout. Behavior is same for disabled hystrix timeout, and when hystrix timeout > the combined potential timeouts of all retries. What is the point to complicate things?

Disabling the hystrix timeout seems like a bad idea. May as well remove hystrix at that point.


Do not install ruby-debug on travis-ci.org

see "Exclude non-essential gems like ruby-debug" for rationale

Thanks. Was unaware of this.


Captcha error

Friends, I am getting this error when checking the google captcha when confirming the sending.
When I click submit, I click to confirm the captcha and, a little later, says that the "verification has expired".
Inspecting the page, in the network tab, I get a 404 error. Does anyone have a light?

@yanvancelis thank you for share this issue with us.
It was fixed. Now all works fine.

Now, I will close this issue. Please, feel free to open a new one if you find something wrong.
@yanvancelis thank you again.


Update Readme to add back the License section

Summary
The Links to Sections previously had a failing link. Added the License section with a link to the license and the link to the contribution guidelines.
Details and comments
that is all.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Temporarily labeled as "on hold" while some finer points are discussed.


[1.18.2, Forge] Chat formatting not synced

Chat formatting seems to not be synced between clients. Meanwhile your own messages are colored, you can't see other's messages colored. Also name format is set without brackets <>, they show up anyway. Permissions were set.
Discord is not provided by this mod.
Forge 40.1.51
BFC 1.2.1

This is very strange, the brackets shouldn't be showing, and I haven't experienced this issue on my private server (granted I am on a compiled version straight from the dev branch instead of the 1.3 beta but not much has changed).
Are there any other chat modifications you're using? I noticed you're using V1.2.1 of the mod with some discord mod, What is the discord mod?

I can try update the mod on the server and test it again once I meet there with my mate. However we are also little outdated with forge right now but I'm waiting for two mods to update so I can release modpack update. May it be because of older forge version?
Discord mod is https://www.curseforge.com/minecraft/mc-mods/lightmans-discord-integration

I would seriously doubt the forge version really messes with anything, I think it is more likely the discord mod you're using is interfering with the ServerChatEvent the mod uses.
Would have to setup some tests on my private testing server to test it with the discord mod you're using. I currently have little access to my homelab since it's Christmas, I will work on this later and get back to you.
For now though I'd suggest trying without the discord mod to see if the problem still exists, if not then we know that's the issue. If the discord mod is the issue you can use the beta of my mod with discord integration, until (if possible) I am able to fix the incompatibility with your mod, however my version unfortunately lacks the account linking functionality of the mod you're using (the functionality is however being developed).

I do use Lightman's because he also has very decent economy mods. With his economy mod, this integration can even list traders with best prices for given item through discord. However I definitely test it without that mod because I had no idea there could be some connection to this problem.
Just no hurry!
Thank you for your help and definitely let me know what's new on this issue. I do some tests as well and let you know.
Have a nice holidays!

Lightman's Discord Integration deleted, server restarted, issue remains, chat still exactly the same.

I cannot replicate your issue, please send your exact mod list:

Advanced Chimneys
Pollution of the Realms
Alex's Delight
Alex's Mobs
Ambient Environment
Ambient Sounds
AntiXray
AppleSkin
Applied Energistics 2
Architectury
Balm
BendyLib
BetterF3
BiomesOPlenty
BrewinAndChewin
Catalogue
ComputerCraft Tweaked
Crafish Furniture Mod
Citadel
Cloth Config
Configured
ControlEngineering
Controlling
Corpse
Crafting Dead Core
CraftTweaker
CreativeCore
DynamicTrees
DynamicTreesBOP
DynamicTreesPlus
Emotecraft
EnergyControl
Energymeter
EngineersDecor
EntityCulling
EveryComp
FancyMenu
FarmersDelight
FarmersResprite
Farsighted Mobs
Flan (Forge LANd claiming mod)
Forge Endertech
FTB Backups2
FTB Essentials
FTB Library
FTB Ranks
GuardVillagers
HiddenNames
Chipped
ImmersiveEngineering
Immersive Posts
InControl
InventoryProfilesNext
IronChest
ItemPhysics
Jade
JEI
JeiTweaker
JER Integration
JourneyMap
JustEnoughProffesions
JustEnoughResources
Kiwi
Konkrete
Kotlin
LCTech
Lightman's Currency
LoadMyResources
Macaw's Doors
Macaw's Fences
Macaw's Paths
Macaw's Roofs
Macaw's Trapdoors
Macaw's Windows
MouseTweaks
Observable
Optifine H7
Patchouli
Placebo
PlayerAnimationLib
RSGauges
ScalableCatsForce
Selene (Moonlight lib)
SereneSeasons
ServerTabInfo
SimpleLogin
SimplePlanes
SimpleRPC
SmarterFarmers
SoundPhysics
Starlight
TerraBlender
ThisRocks
ToastControl
ToughAsNails
Towns and Towers
Voicechat
WorldEdit
YungsApi
Yungs Better Dungeons
Yungs Better Mineshafts
Yungs Better Strongholds
Yungs Better WichHuts
Yungs Extras
Server Mods:
Lightmans Discord
Connectivity
LuckPerms
BetterForgeChat
WelcomeMessage
FTB Library
FTB Essentials
Login Protection
Spark
CrashUtilities
Block History
VanishMod
Shutdowner

Wow! That's alot, I'm seeing a few that could cause problems, specifically the VanishMod, Login Protection, Connectivity, and WelcomeMessage. Some other clientside stuff could be doing it, but I'd bet this is a mod interference issue. If you can find the mod causing the issue send me it and I might be able to fix it, otherwise I simply don't have the time to test each of these mods individually.
I would go through each serverside mod first, then the clientside ones, especially clientside mods that do stuff with chat messages.

Umm... I downloaded mods from my server FTP and used them on my local server. I boot up the local server and there is no problem... But with exactly the same mods, there is that issue on hosted dedicated server.  I even use the same forge version, config folder, world config, just everything is copy pasted and it's different. This one is pretty difficult one, sorry.

not a issue with https://www.curseforge.com/minecraft/mc-mods/mc2discord/

Can't reproduce anymore.


Display 2-page view for published books

Who it's for
Users viewing the materials online
Problem to be solved
During the viewer pop-up research, we learned the majority of users preferred 2-page views when viewing books or manuscripts.

provide context e.g. caption, annotation, table
a clear understanding of the material type
less click and visually less overwhelming

Acceptance Criteria

[ ] Display 2-page view in both list and detail views by default for relevant material types. e.g. books, manuscripts, etc.

Open Issues

There is some back-end logic we need to sort out to determine what gets displayed as a 2-page view
Can we ensure the correct two pages are displayed together?
Some manuscripts have additional pages/inserts which throw of the sequence of recto/verso. Need to follow up with Ashley to investigate these examples.
Do we know what people are trying to achieve? Can we solve that another way?

Current state (updated April 2021):
We can only display materials tagged as monographs in Goobi. This means that archival material (like bound manuscripts) won't fall under the same category. In order to offer 2-page view for bound manuscript material, these will need to be identified in Goobi as having a monograph structure.

Added to Productboard


Ability To send Custom Attribute to Internal Backends

Hi Guys,
Is there an out of the box way to pass along or send info from API cast to backend APIs?
This is so that the latter can recognize the origin (user).
Specifically, In registration we have custom info attribute (numeric) added through Porta, and would like to pass/proxy it to backends in headers?
Any suggested solution?
Thanks.

You should be able to achieve this easily with the Header Modification Policy, adding the relevant fields in as liquid tags
https://access.redhat.com/documentation/en-us/red_hat_3scale_api_management/2.10/html/administering_the_api_gateway/apicast_policies#header_modification

Hi @es02,
Thanks so much for the quick feedback!
Looking into it now.
Do I have access for the registration custom attributes in policies?

I'm not 100% sure what you mean by tha, but you can check exactly what fields you have access to with the Liquid Debug Policy
https://access.redhat.com/documentation/en-us/red_hat_3scale_api_management/2.10/html/administering_the_api_gateway/apicast_policies#liquid_context_debug

I meant additional signup fields here.

I believe so, but the Liquid Debug Policy will tell you for sure.

Many thanks.
I guess this close the issue overall.


Matterbridge does not forward all messages to Mattermost (Auth related?)

Describe the bug
Messages are sometimes not forwarded to Mattermost, couldn't find something related to that in the debug logs, the messages was received and it look like it was pushed to mattermost - but it didn't appear there.
To Reproduce
Unclear how to reproduce, bridge stopped forwarding them and continued later.
Expected behavior
Bridge forwards messages.
Screenshots/debug logs
Log on Mattermost side could be:
{"level":"error","ts":1534925964.771138,"caller":"app/web_conn.go:263","msg":"Invalid session err=GetSession: Invalid session token=[[TOKENHERE]], err=<no value>, "}
The token is, of course, valid. Maybe there could be some check, if a message was correctly posted to mattermost? My golang and Mattermost API knowledge are unfortunaltely not good enough to provide a PR - sorry.
Environment (please complete the following information):

OS: linux
Matterbridge version: 1.11.2 release

I guess this only happens when on rather a long-running matterbridge ?

Yes, think that is an expired session or something like that.

Could you try using personal access tokens: https://docs.mattermost.com/developer/personal-access-tokens.html they don't expire.

Currently, personal access tokens are being used. Try to get some info the next time it happens.

I've reworked some code regarding to private tokens and reconnects in mattermost.
The latest stable release should fix your issues.
If it doesn't, feel free to reopen this ticket.


add infinite devices to the adaptors

[ ] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes

Verification

Hi @2pk03, can you fix the DCO and combine https://github.com/thanos-io/thanos/pull/4511 together with this pr?

Hi @yeya24 - fixed it in a new PR, this one was a bit .. off :)
thank you