leading zeros not rendered in numbers in helm charts

I don't know where the issue lies exactly but when you have charts with values like this:
  defaultMode: 0555

they end up being rendered as 555.

Seems that YAML supports both octal and decimal notation for integers and nix only does decimal. So unfortunately I think the only way to fix this is to override the defaultMode as 365 (0555 in decimal). It would be useful to provide a helper function to do this conversion in nixidy lib but I can't think of how to implement this quickly in nix.
Docs about defaultMode:
Optional: mode bits used to set permissions on created files by default. Must be an octal value between 0000 and 0777 or a decimal value between 0 and 511. YAML accepts both octal and decimal values, JSON requires decimal values for mode bits. Defaults to 0644. ...

See: https://github.com/NixOS/nix/issues/7578

Hmm, we should be converting octal to decimal then automatically. The parsing is not handled by nxiidy, is it?

Yeah you're correct. What's the resource kind that's causing issues?

Oh wait! I just need to update nix-kube-generators in my flake lock file! https://github.com/farcaller/nix-kube-generators/commit/2a1fe17dd9eaf0990239a44576ffb46db12d6b57

nice. in this case it was Kustomization  I believe but it's probably applicable to all yamls 😄.
Great find thank you.

I made a new release v0.2.0 which should address this issue. Please let me know if it doesn't.

thanks, I'll let you know!


support customized kubelet configuration

Besides customizing node configuration to change the default kubelet config, we introduce in this PR the ability to configure both the args of kubelet and download URL, as part of our custom configuration feature.

@alexeldeib @ganeshkumar5699 @feiskyer  please take a look.

Thanks @alexeldeib. The change now looks more simple and straightfoward.


Install checkcommits binary

Hi @chavafg - it looks like Semaphore isn't configured correctly - it's trying to call .ci/setup.sh which doesn't exist.

Oh right, that setting should exists after #45 is merged. Checking with @devimc

@jodh-intel could you please rebase your PR?

@devimc - done.

lgtm


Public api to bypass hasSession cache

We have use cases where we want to bypass the hasSession cache.
We know the cache is cleared when using the Account.login() method.

We sometimes render login links from other origins than the destination where users end up after login. E g our sales poster pages have login and purchase links rendered in an iframe (this having a different origin) than the composing page. So even if we embed and use Account.login() from the iframe, the cache will be cleared on iframe.aftonbladet.se rather than www.aftonbladet.se. We could start posting messages from the iframe, but we´d prefer not to.
We sometimes wish to use static  links to login.

If we do one of the above and user returns to www.aftonbladet.se a stale state is still cached.
We can detect this, but are then forced to either do
localStorage.removeItem('hasSession-cache') or  temporarily set identity. _enableSessionCaching = false.
None of those are good options, operating with SDK internals.

Hi. I see your point. Now that all sites started using v3 of the sdk at the same time, and also iOS 12 (with ITP2) was released, we’re drowning in support at the moment, so I think I have to say “it’s gonna be a few days until I can get to this”.. Hope you understand and can work around it with internals in the mean time


change to use - for empty attribute value causes bad results on morphology equivalency check

Reported by Polina Yordanova
Caused by #694, specifically the change in 4f53518a8604dfd51b02046cd0b30733d634f909
I'm not sure how many other places in the code are effected by this, but it was found in the sg tab.
The sg.morphHasChanged function is called by sg.checkAndUpdateGrammar to see if the user has changed the morphology since it was last stored in the state.  The problem is that this is always returning true for elements with empty attributes (i.e. '-' in the postag) because when the original sg object is populated, the morph object doesn't get the empty attributes, but the morph object on the token does have them.
I haven't fully isolated exactly where this difference occurs - I think it has some interaction with the browser cache of morphological forms.
This goes back to my original worry mentioned in #694 about the overloading use of specialEmptyAttributes to present the short and long values for newly added forms was not the right thing to do. I couldn't pinpoint my concern then, as we hadn't isolated any occurrences depending upon it other than the concatonated forms display, but apparently this is another bit of code that was relying on the - value producing an empty object (as evidenced by this comment in morph.js: // attrObj can be undefined when the postag is -
The question really is what to do now. I think it's not a good thing that there are implicit dependencies on the fact that '-' doesn't produce a short/long value so I am tempted to just address them as we find them like this, but @Masoumeh I'd like to know what you think.

I should add that this is also exposing a bug in the SG code because the actual problem that is preventing the proper display of the SG attribute is that the ancestors property is not being carried along when the object is updated after a morph change

Actually, I cannot say which decision is the best, since there might be some points in the requirements and also the logic of the source code that I don't know exactly. I would also go for the solution that was chosen before (i.e. using underscore). But if it is decided to omit underscore in the end, I can go through the code to find dependencies and we can make an effort to test the changes for a while.

Per discussion with @gcelano and @Masoumeh we will back the fix for #673 out of the production environment for now as a hotfix for this issue. But I'm going to leave it in master so that we can work through the bugs there.


Add Glue UpdateDatabase

Leaning heavily on CreateDatabase, this PR adds support for Glue's UpdateDatabase operation.

Codecov Report

Merging #5407 (8e70117) into master (87683a7) will decrease coverage by 0.00%.
The diff coverage is 90.90%.

:exclamation: Current head 8e70117 differs from pull request most recent head 7565591. Consider uploading reports for the commit 7565591 to get more accurate results

@@            Coverage Diff             @@
##           master    #5407      +/-   ##
==========================================
- Coverage   96.06%   96.06%   -0.01%
==========================================
  Files         708      708
  Lines       70968    70979      +11
==========================================
+ Hits        68178    68188      +10
- Misses       2790     2791       +1

Flag
Coverage Δ

servertests
35.82% <9.09%> (-0.01%)
:arrow_down:

unittests
96.00% <90.90%> (-0.01%)
:arrow_down:

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

Impacted Files
Coverage Δ

moto/glue/responses.py
98.76% <85.71%> (-0.29%)
:arrow_down:

moto/glue/models.py
97.00% <100.00%> (+0.02%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Make modules tree-shakeable

I haven't done extensive testing on this, so I may be wrong, but the way this package is currently set up isn't tree-shakeable by compilers that have that capability. This could be done by splitting files up and only importing what each export needs. The way I imagine this working is having an encode.js and a decode.js and having them import only the utility functions they need. The main file would simply be an export of those two (along with the other utility functions currently exported.)
This would also allow people who don't have tree shaking setup to import specific exports, like so:
import decode from 'bolt11/decode';

More info about Webpack's approach to tree shaking here: https://webpack.js.org/guides/tree-shaking/

Pull Requests are welcome!


how I can do with flexx or PScript to load plotly.js from my local project document? Thank you very much.

Hi @almarklein ,
I was developing an app with using the amazing software “flexx”. And there is good feature that there is plotlyWidget which is really useful for me to plot my scientific data. I read the PlotlyWidget code, there is only one function Plotly.newplot(),,  that is not enough for me. So I read a lot of mannual about “flexx” in order to create a new class which can satisfy my needs.
But there is a proplem which plague me a lot. Besides I know nothing about javascript programming, and I want write my code in python way.

How to load plotly.js assset from local computer?The PlotlyWidget gets its plotly.js assset from remote url with the code app.assets.associate_asset(__name__,’https://cdn.plot.ly/plotlylatest.min.js’) . I want my app running independent with the internet, so I don’t want my code run that way. I download the file “plotlylatest.min.js” from the plotly website and place it under my project. I copy the PlotlyWidget code and change only one line app.assets.associate_asset(__name__,'https://cdn.plot.ly/plotlylatest.min.js’) to app.assets.associate_asset(__name__,’plotlylatest.min.js’). But it does’t work. I do itin a lot of ways such as app.assets.associate_asset(__name__,’my.js’, ’plotlylatest.min.js’) and so on. All of it failed.

After all, how I can do with flexx or PScript to load plotly.js from my local project document? Thank you very  much.

find solution in example

Can you please explain what you did, for future reference? :)

I think he meant this example:
https://flexx.readthedocs.io/en/stable/examples/local_assets_src.html#local-assets-py


Bugfix/issue#21

fixes #21

I hope changing the fixtures in this way is alright, otherwise I will revert it to the old structure

Codecov Report

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

@@          Coverage Diff           @@
##           master     #22   +/-   ##
======================================
  Coverage    97.7%   97.7%
======================================
  Files           2       2
  Lines         131     131
  Branches       28      28
======================================
  Hits          128     128
  Misses          3       3

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 a82d496...915e90a. Read the comment docs.

@ts-de That's fine. I already wanted to make that structure change 😆
I just added a note there.


Fix SAF Solution template code style

Prevent a newly created solution from failing the code style check.

Created from the wrong branch.


WIP: Implementation of saucectl-run-action

Base implementation of saucectl run action.

@FriggaHel I don't think we need to check in the node_modules, no?

@FriggaHel I don't think we need to check in the node_modules, no?

@FriggaHel I don't think we need to check in the node_modules, no?

@christian-bromann  Action is just fetch and executed, there is no step where dependencies are installed. This node_modules contains only the productions one.
https://docs.github.com/en/actions/creating-actions/creating-a-javascript-action#commit-tag-and-push-your-action-to-github

@FriggaHel I don't think we need to check in the node_modules, no?

@christian-bromann  Action is just fetch and executed, there is no step where dependencies are installed. This node_modules contains only the productions one.
https://docs.github.com/en/actions/creating-actions/creating-a-javascript-action#commit-tag-and-push-your-action-to-github

What about the suggestions of using vercal/ncc or similar what we do in saucelabs/sauce-connect-docker use the TypeScript compiler to create a single distribution file?

What about the suggestions of using vercal/ncc or similar what we do in saucelabs/sauce-connect-docker use the TypeScript compiler to create a single distribution file?

What about the suggestions of using vercal/ncc or similar what we do in saucelabs/sauce-connect-docker use the TypeScript compiler to create a single distribution file?

I will take a look to that option @christian-bromann ! Thanks for the tip !

What about the suggestions of using vercal/ncc or similar what we do in saucelabs/sauce-connect-docker use the TypeScript compiler to create a single distribution file?

I will take a look to that option @christian-bromann ! Thanks for the tip !


Centos 6 build

just an overview of the CentOS 6 related changes

This pull request didn't trigger Jenkins as its author isn't in the whitelist.
An organization member must perform one of the following:

To have this branch tested by Jenkins, use the "ok to test" command.
To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"

Closing this for now


unable to send authorization token on server when playing file from url

Version of react-native-audio-recorder-player: 2.5.1
Platforms you faced the error (IOS or Android or both?) Both Android And IOS
Expected behavior
Send authorization token for the url when playing file from the server
Actual behavior
Authorization is not supported when playing file from url
Steps to reprodue the behabior
I have the following snippet:
const playPausePlayer = async () => {
const audioUrl = "https://server-url/attachments/VoiceFile_0.mp4";
if (isPlaying) {
console.log("WAS PLAYING");
stopPlayer();
isPlaying = false;
return;
}
try {
const msg = await audioRecorderPlayer.startPlayer(audioUrl);
console.log(msg);
isPlaying = !isPlaying;
} catch(e) {
console.log(e);
}
}
const stopPlayer = async () => {
audioRecorderPlayer.stopPlayer();
audioRecorderPlayer.removePlayBackListener();
}
The code works fine when
https://server-url/attachments/VoiceFile_0.mp4 is not authorized. The problem is that we have a bearer token a bearer token authorization so we should be able to send the token to the startPlayer method.
something like:
const msg = await audioRecorderPlayer.startPlayer(audioUrl, authroizationToken);
Please note that the url is a dummy one since the server where we do the call is not public on the internet

I will pay for this weekend! Sorry for the delay!

@AndreiToader1 You can now pass httpHeaders when startPlayer. I've done this in android. Also, I am not sure if this is also needed in iOS. Please tell me. Since, I've not tested this strictly, I hope you can test this out which is released in 2.6.0-rc1. Will remove rc when this feature is working well.

To make this work on iOS, I found it that we need to migrate AVAudioPlayer which we currently used to AVPlayer followed by the article.
The problem is now is that I only have my Chromebook and I don't have a mac. Therefore, I can't refactor this and test previous features that are working well.


Gasoline Introduction

Problem Link - https://www.codechef.com/LTIME90B/problems/BEGGASOL

Please verify the problem and assign it to me. I will do it in c++.

I can do this in C

@Nivedita967  i would to contribute in python

Please assign this task to me I have not yet added the code of this problem.

Open for Java.

@Abhichal18 Are you working on this ?


Disabled spelling in documentation

Removed sphinxcontrib-spelling from docs/requirements.txt, as it
was causing build errors on Read the Docs.
This will need to be reinstated in due course.

Coverage remained the same at 77.778% when pulling 8d15aa709ae523987dd5ceaaf40415d4b1ecb676 on test-docs into fe553bcc45af101cae9f31b4d05a9a12fddf6325 on develop.


Support fluentd as an alternative logger

Can you describe steps needed to add fluentd as an alternative to collectd?
Usecase: fluentd to ship json messages to a kafka 0.9+ (with tls) cluster.

Since we've never used fluentd as part of mantl, I'm not sure this is the best place to ask this question. Try taking a look at fluentd's documentation.


docs: Make DIRECT_URL naming consistent with the previous pattern

Describe this PR
Current naming makes a mess in any env, because DIRECT_URL doesn't tell what that is, at the very least it has to stay consistent with the previous pattern, in ideal case both will be renamed because they are semantically wrong.
Changes
More consistent env variable naming
What issue does this fix?
Now people are copying bad naming and getting "DIRECT_URL" as a global env variable which says nothing about its usage and isn't consistent with Prisma's other url

Hi there! Thank you very much for the contribution 🙏
Unfortunately, I don't think we'll be able to include this. While you're correct, DATABASE_DIRECT_URL is a more clear variable, we have several years of content pointing to the current setup of DATABASE_URL and DIRECT_URL.
Again, thank you very much, and I'll also bring notes back to our team to consider this naming change in the future.


Make mvim smarter about remote logins

Every now and then I need to ssh to an osx machine. Given that I have following aliases defined:
$ alias | grep '^vi'
vi=mvim
vim=mvim
vimdiff='mvim -d'

I can't easily launch MacVim in terminal. Instead, a window opens on the remote machine, I smack my forehead, kill it, unalias vi and use that. One workaround is to have vi/vim/vimdiff symlinks in place, that will affect argv0 and make mvim launch in terminal. This solution is suboptimal though - when I'm logged in directly, I actually like having MacVim launch the gui version for vim (muscle memory is a horrible thing to change). The shell I'm using (zsh) allows for argv0 manipulation, but mvim is a shell script, and $0 it is using is unaffected by those kind of manipulations.
This small patch to mvim makes it skip the -g flag if SSH_CONNECTION environment variable is present. Doing it this way has the extra benefit of working seamlessly with tmux, as by default it'll add SSH_CONNECTION flag to newly created windows if you attach a session after sshing in.

@yacoob could you squash the git commits?

Done. I think you can also ask github to squash automatically during merge
nowadays.

thanks!


Message sharing

There are two kinds of message sharing,

Sharing a message with a fellow user
Sharing a message externally

If we allow users to share messages within the app, perhaps we shouldn't let them share private messages?
I am not sure what sharing messages externally might look like. Thoughts?

Sharing, like the share feature you see in most apps where it opens up options to share to someone on email, hangouts, or a social media app?

I think @bookercodes means deep linking to a given message in a given room.


fix: web control plane endpoint and auth

fixes build so that build gets run on each pr
fixes control plane client to use correct endpoint

Integration tests now pass in my dev cell using the updated envoy config from https://github.com/momentohq/control-plane-service/pull/306

integration tests should pass once this pr https://github.com/momentohq/control-plane-service/pull/306 is merged and deployed to alpha


ast-grep: install shell completions

Description of changes
Install completions for bash, fish and zsh
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 294455 run on x86_64-linux 1

1 package built:

ast-grep


Add application environment details view

Fixes
https://issues.redhat.com/browse/HAC-1943
https://issues.redhat.com/browse/HAC-1944
Description
Updates the application details view to use the application environment cards as drill downs into an application's environment details view.
Add an application environment details view showing components in the application. This view is switchable from a list view to a topology graph view with a side panel showing more details for the selected component.
Type of change

[x] Feature

Screen shots / Gifs for design review

How to test or reproduce?
Navigate to the application details view.
Select the environment card you wish to view details for.
Select a component in the environment to view the side panel details.
Use the graph/list icons to switch between list and graph views.
Browser conformance:

[x] Chrome
[x] Firefox
[x] Safari
[x] Edge

/assign @rohitkrai03
/cc @sahil143 @rottencandy

There are a couple of issues causing the topology positioning error. Fixes are known but not yet available:

https://github.com/RedHatInsights/insights-chrome/pull/2001
https://github.com/patternfly/patternfly-react/pull/7809

cc @beaumorley @Ranelim @serenamarie125

@karthikjeeyar @rottencandy  Could you please take another look?
@beaumorley @Ranelim @serenamarie125 Any comments from UX/PM?

Hard coded values such as commit info and status & date in environment card will be addressed in separate ticket ?

Hard coded values such as commit info and status & date in environment card will be addressed in separate ticket ?

Yes, unless those are now available. The hard coded values were there already.

Collapsed state form environments on Application page:

@rohitkrai03 in my opinion, the more we can show the better.  It helps us to get feedback.

I created a doc of notes about the implementation of the application view, and have included the environment view specific notes in this comment, since those seem to be the focus of this PR.

“Application healthy” “Deployment strategy” should all have the second word lowercase
“Automated” should be “Automatic” deployment strategy
“Last deploy” label should be “Last deployment”
Route, Image, and Code repository links in the side panel should all seemingly include external link icons at the end
A kebab menu was shown in the topology side panel in the design to match the list view, whereas an Actions menu appears in the implementation

Note: This list deosn't capture any sort of adjustments based on the absence of the ‘Components’ card and the selection behavior that drives changing the detailed content below. My understanding is that this was an iteration for milestone 7 of this view's development and those interactions could still come in later milestones.

“Application healthy” “Deployment strategy” should all have the second word lowercase

👍

“Automated” should be “Automatic” deployment strategy

This is in the application detail page, I have not updated that and I know there are changes being made. Not part of this PR.

“Last deploy” label should be “Last deployment”

👍

Route, Image, and Code repository links in the side panel should all seemingly include external link icons at the end

It looks bad to me to have 4 links in the side panel each with the external link icon. To me it is fairly obvious that they are links and therefore also expect them to open in a new tab. Seems extraneous and cluttering. If UX feels strongly I will add them, but against my better judgement.

A kebab menu was shown in the topology side panel in the design to match the list view, whereas an Actions menu appears in the implementation

The kebab menu looks bad next to the close (x) button. Openshift uses the Actions dropdown so I went with that. If UX feels strongly I will change to the kebab, but against my better judgement.

Note: This list doesn't capture any sort of adjustments based on the absence of the ‘Components’ card and the selection behavior that drives changing the detailed content below. My understanding is that this was an iteration for milestone 7 of this view's development and those interactions could still come in later milestones.

Regarding this interaction, @jeff-phillips-18 mentioned that he might be able to implement the switching interaction in M7 as a separate PR, just so we could try iterating on the view to see how it looks and where any issues lie. If that PR can't merge in time or we decide not to go that route for now, at least this existing PR will have already merged with the environment view functionality as a drill-in for M7 and we can revisit in M8.
I suspect when we implement the switching behavior, we will confirm that the space for the topology graph view and side panel is fairly constrained when the cards are expanded. My hope is that by iterating on that PR we can find an acceptable layout that offers the user sufficient space to interact with the environment’s components, and also still retains the card flow view of the components into the Development environment into subsequent environments, since I do think that is beneficial to orienting the user, and also very explicitly shows the user what environments’ details they are looking at (which seems to be important.)
cc @beaumorley @rohitkrai03 @serenamarie125

@rohitkrai03 Every other view filters out the non-matches. Do we want this view to be different? Only the graph view or both the graph and list view?

@rohitkrai03 Every other view filters out the non-matches. Do we want this view to be different? Only the graph view or both the graph and list view?

@jeff-phillips-18 I think I didn't notice the non-matches nodes because I only had one node. We can continue to use the same convention as before. For selection of nodes the issue is in graph mode. This is how you can reproduce the issue -

Select a node in list view.
Switch to graph mode and close the side panel.
Try selecting the same node again.

I've shown this behaviour in my previous screencast as well.

I've shown this behaviour in my previous screencast as  well.

@rohitkrai03 This is fixed in Jeff's last commit, I have tested it.

/lgtm

@rohitkrai03 Any further thoughts?

/retest

/lgtm

There is a test failure due to new route in ImportForm submit handler.

/lgtm


MYSQL error return, seems MYSQL is crashing

** SANDSTORM SUPERVISOR: Starting up grain. Sandbox type: userns
Installing MySQL system tables...
170614  2:21:10 [Warning] Using unique option prefix key_buffer instead of key_buffer_size is deprecated and will be removed in a future release. Please use the full name instead.
170614  2:21:10 [Note] Ignoring --secure-file-priv value as server is running with --bootstrap.
170614  2:21:10 [Note] /usr/sbin/mysqld (mysqld 5.5.55-0+deb8u1) starting as process 31 ...
OK
Filling help tables...
170614  2:21:10 [Warning] Using unique option prefix key_buffer instead of key_buffer_size is deprecated and will be removed in a future release. Please use the full name instead.
170614  2:21:10 [Note] Ignoring --secure-file-priv value as server is running with --bootstrap.
170614  2:21:10 [Note] /usr/sbin/mysqld (mysqld 5.5.55-0+deb8u1) starting as process 37 ...
OK

To start mysqld at boot time you have to copy
support-files/mysql.server to the right place for your system

PLEASE REMEMBER TO SET A PASSWORD FOR THE MySQL root USER !
To do so, start the server, then issue the following commands:

/usr/bin/mysqladmin -u root password 'new-password'
/usr/bin/mysqladmin -u root -h sandbox password 'new-password'

Alternatively you can run:
/usr/bin/mysql_secure_installation

which will also give you the option of removing the test
databases and anonymous user created by default.  This is
strongly recommended for production servers.

See the manual for more instructions.

You can start the MySQL daemon with:
cd /usr ; /usr/bin/mysqld_safe &

You can test the MySQL daemon with mysql-test-run.pl
cd /usr/mysql-test ; perl mysql-test-run.pl

Please report any problems at http://bugs.mysql.com/

waiting for mysql to be available at /var/run/mysqld/mysqld.sock
170614  2:21:10 [Warning] Using unique option prefix key_buffer instead of key_buffer_size is deprecated and will be removed in a future release. Please use the full name instead.
170614  2:21:10 [Warning] Insecure configuration for --secure-file-priv: Current value does not restrict location of generated files. Consider setting it to a valid, non-empty path.
170614  2:21:10 [Note] /usr/sbin/mysqld (mysqld 5.5.55-0+deb8u1) starting as process 41 ...
waiting for mysql to be available at /var/run/mysqld/mysqld.sock
waiting for mysql to be available at /var/run/mysqld/mysqld.sock
waiting for mysql to be available at /var/run/mysqld/mysqld.sock

build.sh
#!/bin/bash
# Checks if there's a composer.json, and if so, installs/runs composer.

set -euo pipefail

cd /opt/app

if [ -f /opt/app/composer.json ] ; then
    if [ ! -f composer.phar ] ; then
        curl -sS https://getcomposer.org/installer | php
    fi
    php composer.phar install --no-dev -o
fi

sudo /etc/init.d/mysql stop
sudo /etc/init.d/mysql start
echo "CREATE DATABASE IF NOT EXISTS cachet; GRANT ALL on cachet.* TO 'cachet'@'localhost' IDENTIFIED BY 'cachet'; FLUSH PRIVILEGES;" | mysql -h localhost -u root
php artisan key:generate
php artisan app:install

laucher.sh
#!/bin/bash

# Create a bunch of folders under the clean /var that php, nginx, and mysql expect to exist
mkdir -p /var/lib/mysql
mkdir -p /var/lib/nginx
mkdir -p /var/lib/php5/sessions
mkdir -p /var/log
mkdir -p /var/log/mysql
mkdir -p /var/log/nginx
# Wipe /var/run, since pidfiles and socket files from previous launches should go away
# TODO someday: I'd prefer a tmpfs for these.
rm -rf /var/run
mkdir -p /var/run
rm -rf /var/tmp
mkdir -p /var/tmp
mkdir -p /var/run/mysqld

# Ensure mysql tables created
HOME=/etc/mysql /usr/bin/mysql_install_db --force

# Spawn mysqld, php
HOME=/etc/mysql /usr/sbin/mysqld &
/usr/sbin/php5-fpm --nodaemonize --fpm-config /etc/php5/fpm/php-fpm.conf &
# Wait until mysql and php have bound their sockets, indicating readiness
while [ ! -e /var/run/mysqld/mysqld.sock ] ; do
    echo "waiting for mysql to be available at /var/run/mysqld/mysqld.sock"
    sleep .2
done
while [ ! -e /var/run/php5-fpm.sock ] ; do
    echo "waiting for php5-fpm to be available at /var/run/php5-fpm.sock"
    sleep .2
done

# Start nginx.
/usr/sbin/nginx -c /opt/app/.sandstorm/service-config/nginx.conf -g "daemon off;"

setup.sh
#!/bin/bash

# When you change this file, you must take manual action. Read this doc:
# - https://docs.sandstorm.io/en/latest/vagrant-spk/customizing/#setupsh

set -euo pipefail

export DEBIAN_FRONTEND=noninteractive
apt-get update
apt-get install -y nginx php5-fpm php5-mysql php5-cli php5-gd php5-curl git php5-dev mysql-server
service nginx stop
service php5-fpm stop
service mysql stop
systemctl disable nginx
systemctl disable php5-fpm
systemctl disable mysql
# patch /etc/php5/fpm/pool.d/www.conf to not change uid/gid to www-data
sed --in-place='' \
        --expression='s/^listen.owner = www-data/;listen.owner = www-data/' \
        --expression='s/^listen.group = www-data/;listen.group = www-data/' \
        --expression='s/^user = www-data/;user = www-data/' \
        --expression='s/^group = www-data/;group = www-data/' \
        /etc/php5/fpm/pool.d/www.conf
# patch /etc/php5/fpm/php-fpm.conf to not have a pidfile
sed --in-place='' \
        --expression='s/^pid =/;pid =/' \
        /etc/php5/fpm/php-fpm.conf
# patch /etc/php5/fpm/pool.d/www.conf to no clear environment variables
# so we can pass in SANDSTORM=1 to apps
sed --in-place='' \
        --expression='s/^;clear_env = no/clear_env=no/' \
        /etc/php5/fpm/pool.d/www.conf
# patch mysql conf to not change uid, and to use /var/tmp over /tmp
# for secure-file-priv see https://github.com/sandstorm-io/vagrant-spk/issues/195
sed --in-place='' \
        --expression='s/^user\t\t= mysql/#user\t\t= mysql/' \
        --expression='s,^tmpdir\t\t= /tmp,tmpdir\t\t= /var/tmp,' \
        --expression='/\[mysqld]/ a\ secure-file-priv = ""\' \
        /etc/mysql/my.cnf
# patch mysql conf to use smaller transaction logs to save disk space
cat <<EOF > /etc/mysql/conf.d/sandstorm.cnf
[mysqld]
# Set the transaction log file to the minimum allowed size to save disk space.
innodb_log_file_size = 1048576
# Set the main data file to grow by 1MB at a time, rather than 8MB at a time.
innodb_autoextend_increment = 1
EOF

/var/log/mysql/error.log
170614  2:51:07 [Warning] Using unique option prefix myisam-recover instead of myisam-recover-options is deprecated and will be removed in a future release.$170614  2:51:07 [Note] Plugin 'FEDERATED' is disabled.
170614  2:51:07 InnoDB: The InnoDB memory heap is disabled
170614  2:51:07 InnoDB: Mutexes and rw_locks use GCC atomic builtins
170614  2:51:07 InnoDB: Compressed tables use zlib 1.2.8
170614  2:51:07 InnoDB: Using Linux native AIO
170614  2:51:07 InnoDB: Initializing buffer pool, size = 128.0M
170614  2:51:07 InnoDB: Completed initialization of buffer pool
InnoDB: The first specified data file ./ibdata1 did not exist:
InnoDB: a new database to be created!
170614  2:51:07  InnoDB: Setting file ./ibdata1 size to 10 MB
InnoDB: Database physically writes the file full: wait...
170614  2:51:08  InnoDB: Log file ./ib_logfile0 did not exist: new to be created
InnoDB: Setting log file ./ib_logfile0 size to 5 MB
InnoDB: Database physically writes the file full: wait...
170614  2:51:08  InnoDB: Log file ./ib_logfile1 did not exist: new to be created
InnoDB: Setting log file ./ib_logfile1 size to 5 MB
InnoDB: Database physically writes the file full: wait...
InnoDB: Doublewrite buffer not found: creating new
InnoDB: Doublewrite buffer created
InnoDB: 127 rollback segment(s) active.
InnoDB: Creating foreign key constraint system tables
InnoDB: Foreign key constraint system tables created
170614  2:51:08  InnoDB: Waiting for the background threads to start
170614  2:51:09 InnoDB: 5.5.55 started; log sequence number 0
170614  2:51:09  InnoDB: Starting shutdown...
170614  2:51:09  InnoDB: Shutdown completed; log sequence number 1595675

Like https://github.com/sandstorm-io/vagrant-spk/issues/195

Change for SQLite and works well but MYSQL isn't working.


Heap Sort in C++

This issue is about [Algo/DS Name/Question Name](link to resource for the Algo/DS/Question)

[ ] I searched or browsed the repo’s other issues to ensure this is not a duplicate

[ ] Issue Title is in form [Algo/DS Name] in [Language] (ex:- BubbleSort in Java)

[ ] I will send my PR only after I'm assigned to this issue by any maintainer

[ ] Assign this issue to me (I want to work on this)

[ ] I ensure that I am not already assigned to 2 or more issues

if you don't mind would you like to assign this task to me?

Assign this issue to me (I want to work on this)

Hello iamkumar-gaurav,
Please request on a different issue you will be provided with that.
Thanks


Database backups

This is a placeholder. I'm not sure if we have them automatically. It's probably worth investigating before we start accepting user sign ups.

We do have it automatically.
production:

staging:

Mentioned in the djangonaut dot com documentation: https://docs.google.com/document/d/1T47kDAkVUQ_yC89y9RZ049KRskvNiObA_YGiIsnBWeY/edit?pli=1#heading=h.1afi41uioo3j

I wonder if we could export one of those every month for long-term storage.

I wonder if we could export one of those every month for long-term storage.
I think it's a good idea as long as it's secure. Especially as we start scaling up.


Wrong detection of root dir with subdomain

Prerequisites

[x] Are you running the latest version of Bootstrap Listr (stable or alpha/beta)?
[x] Did you check your browser's console for errors? See how in Chrome, Firefox, Edge)
[ ] Did you build a distribution? Use npm run build
[ ] Are you running the Node v6 (or higher)?
[x] Did you check the FAQ?
[x] Did you perform a cursory search to see if your issue has already been reported?

Description
I host my stuff at http://one.com. They have a feature where each folder in your root directory becomes a subdomain, so my file listing is available at both http://download.kragenet.dk/billeder and http://kragenet.dk/download/billeder, but I would like to only use the first one. When you visit the root (http://download.kragenet.dk/billeder/) everything is ok, but entering the subdir http://download.kragenet.dk/billeder/Gulvvarme/, the home url is wrongly detected to http://download.kragenet.dk/download/billeder, which makes CSS and JS for modals fail loading.
This could probably be fixed by just adding a config for overriding normal root url detection, but it would be cool if it just worked automatically.
PHP info: http://info.kragenet.dk/

Could this be a case of server configuration as mentioned in the FAQ?

It might. I think I might have overlooked something ehre. Thanks! I will test immediately.

It seems the issue was that the root dir change whether I was at the root page or at a sub dir.
You may have an error in your FAQ. you write that:

If you run Bootstrap Listr in /html/files, this should be RewriteBase files/.

i my case I had to write RewriteBase /billeder, but I am far from being an expert in .htaccess.

This however also makes http://kragenet.dk/download/billeder useless, but I can live with that.


add Text as an underlining option too

add Text as an underlining option too its pretty common everywhere so please it would help me a lot to move to my text

you can use "wrapper" feature to support it, which can be found in readme.

sorry that I thought I was responding issues of Text Format. You can install my another plugin Text Format to add custom wrapper.


Pan/zoom controls

Middle click-drag on the background pans
Right click-drag on the background zooms

Drag up to zoom in
Drag down zoom out

This is pretty nice, but does not work well on a trackpad. I think the mousewheel with no clicking should zoom instead of right-click. There is no way to middle-click on the trackpad. Perhaps right-click and drag for pan. Something like this - except replace left with right click for drag:
http://bl.ocks.org/mbostock/3680999

That's odd - I thought two-finger tap == middle click and three-finger tap
== right click on a trackpad...
I will update this to use the mousewheel instead of right-click-drag though.
On Tue, Jul 14, 2015 at 4:51 PM Jeffrey Baumes<EMAIL_ADDRESS>wrote:

This is pretty nice, but does not work well on a trackpad. I think the
mousewheel with no clicking should zoom instead of right-click. There is no
way to middle-click on the trackpad. Perhaps right-click and drag for pan.
Something like this - except replace left with right click for drag:
http://bl.ocks.org/mbostock/3680999
—
Reply to this email directly or view it on GitHub
https://github.com/Kitware/clique/pull/56#issuecomment-121381521.

Nope two finger is right click, three finger does other crazy stuff like swiping between desktops.


vmware-workstation: 17.5.1 -> 17.5.2

Description of changes
This PR should close #310121
This derivation now uses https://softwareupdate.vmware.com as source, as of changes to distribution stated in this blog
Those changes by Broadcom causes derivation to become a little more complex by now needing to download all the components separately.
Many thanks to @kulczwoj for providing overlay that this PR was based on.
Change in software itself are minor for now.
Also fixes SQLite INSERT INTO syntax.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

For some reason tools iso only work if i disable enableInstaller on my 23.11 system, otherwise it will do an SSL error when trying to pull them off the internet. More information can probably be found in /tmp/vmware-root or /tmp/vmware-user dirs with logs.

When will this get merged and then show up in the 24.05 repo? I really need to get vmware-workstation operational to finish up a photo project I am working on.

Things take a while in big repos, meanwhile you can apply this overlay in this NixOS option for now to get vmware-workstation working.

Things take a while in big repos, meanwhile you can apply this overlay in this NixOS option for now to get vmware-workstation working.

Thanks but until the documentation becomes usable by normal people (ie: people who do not eat, sleep, breathe, sweat, bleed, urinate and deficate Nix), I am not going to mess with overlays, flakes or home-manager. I will wait for the repos to update as this has merged.

Would it be a good idea to backport this to 24.05 as well considering VMSA-2024-0010 with fixes in 17.5.2?


1.20.+

When to support 1.20.+

I believe the plugin does run in 1.20.+.It just hasn't been officially tested by us yet so we haven't officially added support for it.

think


emojis removed from nav bar dropdowns

Emojis removed from items in dropdowns

Fixes #331

✅ By submitting this PR, I have verified the following

[x] Checked to see if a similar PR has already been opened 🤔
[x] Reviewed the contributing guidelines 🔍
[x] Added my name to the bottom of the list under the Contributors section in the README.md with a link to my personal website or GitHub profile 👥

@teisenhower could we also get the emojis removed from the spanish navbar version?

@fvcproductions my apologies! I'll remove them as well. Sorry for that oversight.

no worries 😸

Emojis have been removed from Spanish nav bar as well. Thank you again!

LGTM, thanks for the contributions!

Of course! Happy to be apart of it. Looking forward to another!


Add inflatable door

About the PR
This PR:
Change DisassembleOnActivate(system/component) to DisassembleOnAltVerb(system/component) (its only use is in inflatable)
Add a basic wooden door for deriving inflatable door off of (if this is alright, I'm going to grab all material doors at once and make their build recipes in a separate pr)
Add inflatable wall box for use in cargo crate or other locations
Add inflatable walls and door to engineer tool vend
Screenshots

Changelog

:cl:

add: Added inflatable doors, which can now be found in the engineer Tool Vend
change: Inflatable walls and doors are now taken down with alt-click

reduced stack sizes
doors - 4
walls - 10

conflicts apparently

Rebased and pushed changes, it was conflicting with the auto naming.


Adding Dialog Prompts to Remove Smartcard (at times that make sense)

Summary
Why the CBA dialog flow is being changed again
Feedback from testers of the NFC dialog flow showed that it would be beneficial to have dialog prompts that let the user know when they should remove their smartcard. For example, after the first NFC tap, once the client determines that there are/are not any errors and have read off the certificates, we should let the user know that they do not need to hold their smartcard to the device anymore.
There are two main places that a user should see such a dialog for NFC:

After initially tapping in order to retrieve certs from the smartcard (before the cert picker)
After tapping in order to confirm PIN and sign with a private key

There is a potential use for the removal prompt dialog in the USB flow:
-After signing with the private key, and other processes involving the smartcard have completed.
- UPDATE: Upon positive feedback, this is included in this PR.
Including this prompt in the USB flow could potentially decrease the amount of configuration change related errors, depending upon when most users will plug in their smartcard. If they plug in their smartcard only during the times our activity is running (when the authentication WebView is showing), then implementing such a change will keep the configuration changes occurring within our activities, which have the appropriate configChanges flags. The flip side of this is that a user who plugged in their smartcard within the host app activity will experience a configuration change within the host app activity upon our activities destroying. Logic has been added to detect if this is the case, and no removal prompt will be displayed if a smartcard is still detected by the end of the flow.
What main changes were made

Added a new ShowSmartcardRemovalPromptDialog method to DialogHolder to be called when we want a user to remove their smartcard (after NFC taps, before sending auth results, for NFC and USB).

corresponding Dialog is SmartcardRemovalPromptDialog

Made changes to WebViewAuthorizationFragment and AzureActiveDirectoryWebViewClient to set a callback that sends the auth result and finishes (in order to delay sending results until the user removes their smartcard). This callback will be invoked immediately if the cba challenge handler is null, and otherwise will be invoked based on the status of the challenge handler. A callback is needed here in order to make sure the user removes their smartcard before our library's activity destroys.
onPausedSmartcardDiscovery renamed to prepForNextUserInteraction. Though we are stopping discovery in NFC's case, the emphasis is put on giving some time to prepare for an expected user interaction, which would be pausing for the user to remove their smartcard (in NFC case; USB implementation of this method just invokes callback).
Got rid of the onCancelCba() method in smartcard dialogs. This was originally used to run logic when a smartcard was unexpectedly unplugged. To keep more consistent with the connectionCallback variable of the CBA managers, I created a DisconnectedCallback local variable that should be set for unexpected/expected unplugging on the USB side, or expected removal on the NFC side.  Actually ended up keeping the onCancelCba() logic, as I was overthinking things. The method was renamed to be onUnexpectedUnplug() to be more specific and to avoid confusion with the cancelCallbacks.

Was able to get rid of callback generics for AbstractSmartcardCertBasedAuthManager.

Consolidated the local cancelCallback interfaces attached to each dialog into a universal ICancelCbaCallback. Done in https://github.com/AzureAD/microsoft-authentication-library-common-for-android/pull/1972#pullrequestreview-1316779182
Consolidated the local dismissCallback interfaces attached to each dialog into a universal IDismissCallback Done in https://github.com/AzureAD/microsoft-authentication-library-common-for-android/pull/1975
-Added a disconnect method to the NFC manager that should disconnect a smartcard and wait for the user to remove the smartcard.

Testing
The Units tests associated with CBA have been updated to check for SmartcardRemovalPromptDialog being shown at correct times.
I've manual tested different scenarios to make sure that the changes made to AzureActiveDirectoryWebVIewClient does not interfere with regular auth. I'll continue to manual test to make sure all bases that I can think of are covered.

I want to note that the removal prompt dialog is no longer being dismissed automatically in the NFC flow because of an issue I am investigating, where certain Android devices are getting blocked when the removal method is being called.
A dismiss button was added to the dialog so that the user does not get blocked here. I'm going to continue to investigate this issue with YubiCo so that the remove() YubiKit method can be brought back in upon next release.


Glucose measurement is in mmol/l but it is really in KG/L?

I used a glucometer with bluetooth  to send the reading to the app. I compared my reading from the glucometer to the data that is shown by the app and found that the app converts the glucose measurement to KG/L and not to mmol/l? The reading on the glucometer is in mg/dl so when I multiplied the app reading by 100000 , that is when they are the same. Also the parsed time is late by an hour. Right now I don't have an idea why but I'm checking out the downloaded code. Just a heads up.

Released in version 2.5. Thank you for bug report!


Math expression evaluation

Feature ✅
Description
Write a program which can evaluate mathematical expressions in infix format.
Example
Input: 2 + 3 * ( 4 - 2 )
Output: 8
Note: Use stack
Mention programming language while requesting assignment

I would like to solve this using C++. Could you please assign this to me ?

i want to write that in c++ using stacks can you please assign to me

I would like to contribute in C
Please assign it to me

@sid-1207 It has been already assigned in C

@pratik-choudhari What about C++?
Do you need it

@sid-1207 C and c++ are not distinguished.


[SPARK-10540] [SQL] Ignore HadoopFsRelationTest's "test all data types" if it is too flaky

If hadoopFsRelationSuites's "test all data types" is too flaky we can disable it for now.
https://issues.apache.org/jira/browse/SPARK-10540

Merged build triggered.

Merged build started.

Test build #42284 has started for   PR 8705 at commit 7c4791f.

Test build #42284 has finished for   PR 8705 at commit 7c4791f.

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

Merged build finished. Test PASSed.

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

Found two possible sources of flakiness:

For double and float, NaN == NaN is always false

checkAnswer doesn't handle Map[K, V] properly. For example:
scala> Map(1 -> 2, 2 -> 1).toString
res0: String = Map(1 -> 2, 2 -> 1)

scala> Map(2 -> 1, 1 -> 2).toString
res1: String = Map(2 -> 1, 1 -> 2)

We can't rely on toString to compare Map instances.

Need to update checkAnswer to special case NaN and Map[K, V].

Filed https://issues.apache.org/jira/browse/SPARK-10591 to track this issue.

For the issue of NaN, looks like there is something wrong because we want to return true when compare two NaN.


Feature request: Calendar Legend

I would like to be able to define a legend for when a calendar has multiple different colors that fit in the same category but mean different things
const TASK_MAP = {
	'10 push ups': 'orange',
	'100 (x kg) lifts': 'orangeToRed',
	'100 (y kg) lifts': 'pacific',
	'20 push ups: 'red',
}

const GROUP_MAP = {
	'100 (x kg) lifts': 'Lifts',
	'100 (y kg) lifts': 'Lifts',
	'10 push ups': 'Push ups',
        '20 push ups: 'Push ups'
}
const data = {
				date: new Date(page.file.day).toISOString().split('T')[0],
				intensity: INTENSITY_MAP[taskText],
				color: TASK_MAP[task.text] || TASK_MAP['default'],
			}

and maybe below the calendar I would be able to see what each column means.

thank you.


Cherries

Cherry-picks
https://github.com/bitcoin/bitcoin/pull/7164
https://github.com/bitcoin/bitcoin/pull/7922
https://github.com/bitcoin/bitcoin/pull/8317
https://github.com/bitcoin/bitcoin/pull/7514

Merged.  I kept having to diagnose the same test failures.


Callback in Electron Render thread not called until reload

Trying to build an app in electron using NEDB.
I have this odd situation where a callback isn't called until you trigger a 'force reload' in the electron window.
Steps
In the render thread before I launch Vue I run the following code.
var Datastore = require('nedb')
var db = new Datastore({ filename: 'theDatastore/callings.db' });
db.loadDatabase(function (err) {    // Callback is optional
  db.count({}, (err,docs) => {
    console.log(err)
    console.log(docs)
  })
});

The code that executes right after the electron app comes up does not execute the callback.
If in the electron window I use the force reload menu the code exectues.
Any thoughts? I am new to nodejs and now sure how to see if their is some deep exception I'm missing.

@huntsfromshadow Which Electron version do you use? I experienced the same issue with 6.0.0-beta.15


[3.2] Misleading option to select key type in try out page in devportal

Describe your problem(s)
Currently in the tryout page as shown below we have an option to select the key type which is used to generate a test key. However, if a person generates a key from the subscription page (Prod key) and pastes key here and selects a different key type than the generated (sandbox), the API call still goes to prod. Which can be misleading since the option is only used when generating a test key

Describe your solution
We can change the UI so that when the user clicks on generate test key option a pop will appear and he can select which key type to generate. Therefore no option will be shown to a user who copies the key from the subscription page

Tracked with https://github.com/wso2/product-apim/issues/8879


Modification of cli parameters

Remove project concept
Remove deployment concept
Add/Change --servers/-s cli input and --variables/-v
$ dplz -s server1.json server2.json server3.json ...
$ dplz -s *server.json (PRERFIX)
$ dplz -s server_*.json (WILDCARD)
$ dplz -s x/*_server_*.json

I'd like to start working on this issue.
I'll come back with updates.

Issue should be closed:
Link #10


append changelog to fulfill the license obligation

Reminding about missing "notices" for modifications on original work (violation of APACHE-2.0)
Hello,
[Modification terms and conditions in license] The original work should be honored while the modifications on it should be attached with prominent notices as the license APACHE-2.0 dictates. Your project is a forking project from stascorp/rdpwrap which adopts APACHE-2.0 as its project license.
The modification terms and conditions of APACHE-2.0 dictate that:

APACHE-2.0:
... You must cause any modified files to carry prominent notices stating that You changed the files ...

Therefore, we think the project is obligated to abide by the modification terms and conditions in APACHE-2.0.
[Violation of modification terms and conditions] We run through your history, and it turns out that you have conducted modifications to the source files of original work while didn't provide prominent notices.
You have 14 commits which conduct modifications to the files of original work, all of which are not declared in README.md.
According to the license terms and conditions above, it is regarded as license violation.
Thus, to avoid potential legal risks, I would suggest you carry out prominent notices for the 14 commits.
[Recommended Fix] There are 2 recommended solutions to fulfill the modification related obligation of APACHE-2.0:

Git Mechanism.

Just put a note in the top directory, something like MODIFICATION.md, or any file but the license text and NOTICES, in order to give people who follow you the most latitude to comply with the license without making a mess,
Write "In compliance with the APACHE-2.0 license: I declare that this version of the program contains my modifications, which can be seen through the usual "git" mechanism."

Changelog Text.
Or add changelog for modification to comply with the license.

 It adds a few lines in changelog without touching the codebase. Please be comfortable to check it out.
Full details of the report you could refer to the html file attached.
Hope it helps!
If there is anything wrong in this reminding, I'm sorry for my bothering, and please feel free to close this PR.
Cheers!

Overwrited by https://github.com/sebaxakerhtc/rdpwrap/commit/a48214d1a0e39021749839afad1719b657cfcff7
Hi!
First of all, thank you for your attention to this fork. I think ihformation from releases page will be more helpful, understandable and detailed.
If you think i'm wrong - feel free to share your mind about this.

Thanks for your review. Yes, the information in release page of this fork is enough.

Thank you for your help!


herbs update - Should create migrations for new entities

A new migration should be created for new entities when herbs update runs.

Hi,
We can work on this issue.
@danilokatayama

I'm working on this issue as well

We tested ourselves and the functionality is ok.

Thank you for checking it! Closing this issue.


⚠ Cioos Pacific CKAN has degraded performance

In 2210f3f, Cioos Pacific CKAN (https://catalogue.cioospacific.ca) experienced degraded performance:

HTTP code: 200
Response time: 9071 ms

Resolved: Cioos Pacific CKAN performance has improved in fdbad2b after 5 minutes.


max_stops query parameter addition

Added optional ability to specify max_stops for flight data search along with example.py reference usage script.
Acceptable values are 0, 1, 2 or undefined for "all flights".

Hey,
My apologies, but I'm on mobile. Could you describe how you made max_stops possible?
Thanks a lot for your awesome work!
AWeirdDev

My apologies, but I'm on mobile. Could you describe how you made max_stops possible?

Field 5 of the FllightData protocol buffer object accepts an integer for number of stops on each defined flight. The values according to the selectors on the UI (determined after decoding protocol buffers string) are as follows:

Undefined = Flights with any number of stops
0 = non-stop flights only
1 = one or fewer stops
2 = two or fewer stops

Higher numbers are allowed (15 for example) but don't seem to have any impact.
Implementation was achieved by adding the optional integer for field 5 to the protobuf definition and recompiling:
message FlightData {
  string date = 2;
  Airport from_flight = 13;
  Airport to_flight = 14;
  optional int32 max_stops = 5;
}

After that, was just adding support for it in your core:
def request_flights(
    tfs: TFSData,
    *,
    max_stops: Optional[int] = None,  # Optionally pass max_stops if needed
    currency: Optional[str] = None,
    language: Optional[str],
    **kwargs: Any,
) -> requests.Response:
    params = {
        "tfs": tfs.as_b64(),
        "hl": language,
        "tfu": "EgQIABABIgA",  # show all flights and prices condition
        "curr": currency,
    }
    if max_stops is not None:
        params["max_stops"] = max_stops  # Handle max_stops in request

As well the filter to fill it for all FlightData objects, if it's supplied:
def create_filter(
    *,
    flight_data: List[FlightData],
    trip: Literal["round-trip", "one-way", "multi-city"],
    passengers: Passengers,
    seat: Literal["economy", "premium-economy", "business", "first"],
    max_stops: Optional[int] = None,  # New argument
) -> TFSData:
    """Create a filter. (``?tfs=``)

    Args:
        flight_data (list[FlightData]): Flight data as a list.
        trip ("one-way" | "round-trip" | "multi-city"): Trip type.
        passengers (Passengers): Passengers.
        seat ("economy" | "premium-economy" | "business" | "first"): Seat.
        max_stops (int, optional): Maximum number of stops. Defaults to None.
    """
    for fd in flight_data:
        fd.max_stops = max_stops


New version: ApproxFunSingularities v0.3.12

Registering package: ApproxFunSingularities
Repository: https://github.com/JuliaApproximation/ApproxFunSingularities.jl
Created by: @jishnub
Version: v0.3.12
Commit: 5e540b0a569acee703a146f52a1e477d14b2a51e
Reviewed by: @jishnub
Reference: https://github.com/JuliaApproximation/ApproxFunSingularities.jl/commit/5e540b0a569acee703a146f52a1e477d14b2a51e#commitcomment-117016639
Description: Support for spaces with singularities in ApproxFun

hold on


Explorer scrolls on file close

VSCode Version: 1.21.0
OS Version: Sierra 10.12.6

Steps to Reproduce:
Basically, this "bug"  is still a thing. Can't comment there, thus creating a new issue.

"explorer.autoReveal": false

No, this does not help.

@LAITONEN what do you see?

@bpasero This when I have the leftmost tab open, scroll to the right and close rightmost tab.

@LAITONEN when you say "Explorer" I assumed you mean the file explorer to the left. So you say when you close a tab, the tab-well scrolls? How do you close a tab?

@bpasero My apologies, this was a silly mistake, rectified by now.
I close it by clicking the cross icon.

@LAITONEN yes I see it, looks like a regression.

Unfortunately our refactorings and repeated layout calls make this not as straight forward as I had hoped. I will have to leave this open until I found a better solution...

@bpasero Thank you! That will work temporally.


LatLng2Place library using maps API

I will mark comments resolved after approval.

Oh, there's approval - marking things as resolved.


WIN32: fix boost build b2.exe for non-msvc compilers

propagate the selected compiler from cmake down to the boost bootstrap process
without these correction the boost bootstrap build of b2 will fail on windows with e.g. clang
for clang 'if(MSVC)' is false and the build process assumes a unix style shell and the build process fails
I'm using Clion (Jetbrains) IDE for Multiplatform development on macOS, Linux and Windows, which makes it very easy to switch between different compilers and test Code compatibility

This seems like a reasonable change, but it breaks the build.

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:

mmh, strange.
I noticed, that the respective "Windows MSVC / build (23, windows-2022) (pull_request)" already failed on your last check-in (April 1) with exactly the same error (https://github.com/tzlaine/parser/actions/runs/8513670468/job/23317833028)
and that fedora changed the GCC compiler from 13.0.2 (https://github.com/tzlaine/parser/actions/runs/8513670482/job/23317832249 , see configure section) to 14.0.1 (https://github.com/tzlaine/parser/actions/runs/8901839715/job/24588909706 , see configure section)
I'll try to check if the baseline (i.e.  the develop branch with only whitespace changes in CmakeList.txt) has no errors.

I think issue #169 breaks the build for the respective os-compiler-c++ variant (see #174)

Yep!  I just hadn't noticed those previous failures.  Thanks for doing this!


No module named copy_reg - Installation Issue

Currently this is my output after installing Tensorflow. I thought it installed six as well, but was wondering why copyreg doesn't exist in six. Anyone know what to do?
tensorflow % python
Python 2.7.6 (default, Sep  9 2014, 15:04:36)
[GCC 4.2.1 Compatible Apple LLVM 6.0 (clang-600.0.39)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
Input:
import tensorflow as tf
Result:
Traceback (most recent call last):
File "", line 1, in
File "/Library/Python/2.7/site-packages/tensorflow/init.py", line 4, in
from tensorflow.python import *
File "/Library/Python/2.7/site-packages/tensorflow/python/init.py", line 13, in
from tensorflow.core.framework.graph_pb2 import *
File "/Library/Python/2.7/site-packages/tensorflow/core/framework/graph_pb2.py", line 8, in
from google.protobuf import reflection as _reflection
File "/Library/Python/2.7/site-packages/google/protobuf/reflection.py", line 58, in
from google.protobuf.internal import python_message as message_impl
File "/Library/Python/2.7/site-packages/google/protobuf/internal/python_message.py", line 59, in
import six.moves.copyreg as copyreg
ImportError: No module named copyreg

Please see if our detailed installation instructions page helps:
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/g3doc/get_started/os_setup.md#common_install_problems
On Mon, Nov 9, 2015 at 9:09 PM, Uday Singh<EMAIL_ADDRESS>wrote:

Currently this is my output after installing Tensorflow. I thought it
installed six as well, but was wondering why copyreg doesn't exist in six.
Anyone know what to do?
tensorflow % python
Python 2.7.6 (default, Sep 9 2014, 15:04:36)
[GCC 4.2.1 Compatible Apple LLVM 6.0 (clang-600.0.39)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
import tensorflow as tf
Traceback (most recent call last):
File "", line 1, in
File "/Library/Python/2.7/site-packages/tensorflow/init.py", line 4, in
from tensorflow.python import *
File "/Library/Python/2.7/site-packages/tensorflow/python/init.py",
line 13, in
from tensorflow.core.framework.graph_pb2 import *
File
"/Library/Python/2.7/site-packages/tensorflow/core/framework/graph_pb2.py",
line 8, in
from google.protobuf import reflection as _reflection
File "/Library/Python/2.7/site-packages/google/protobuf/reflection.py",
line 58, in
from google.protobuf.internal import python_message as message_impl
File
"/Library/Python/2.7/site-packages/google/protobuf/internal/python_message.py",
line 59, in
import six.moves.copyreg as copyreg
ImportError: No module named copyreg
—
Reply to this email directly or view it on GitHub
https://github.com/tensorflow/tensorflow/issues/75.

@vrv Just saw this minutes after I posted. Thanks, but I got it all figured out, I forgot to use Python 2.7.10 and was using the 2.7.6 that comes with my computer.

Below solution in detailed installation page resolved my issue:
Upgrade the system-wide copy of six:
sudo easy_install -U six


API grouper for specs

Hello there.
I would like to know if it's possible to use an API grouper for specs, e.g.:

This is possible using a '' with the spec combobox.
Please take the time to search the repository, if your question has already been asked or answered.

[x] What version of the library are you using? Is it the latest version? The latest released version is

springfox-swagger2 - 2.8.0
springfox-swagger-ui - 2.8.0
What kind of issue is this?

[x] Question. Is this a question about how to do a certain thing?

[ ] Feature Request. Start by telling us what problem you’re trying to solve. Often a solution
already exists! Don’t send pull requests to implement new features without first getting our
support. Sometimes we leave features out on purpose to keep the project small.
Please do consider starring this repository if you do like this project and find it useful.

[ ] Bug report. If you’ve found a bug, spend the time to write a failing test. Bugs with tests or
steps to reproduce get fixed faster. Here’s an example: https://gist.github.com/swankjesse/6608b4713ad80988cdc9

[ ] spring xml/java config that is relevant
[ ] springfox specific configuration if it makes sense
[ ] include any output you've received; logs, json snippets etc.; and what the expected output should be
[ ] if you have a repo that demonstrates the issue for bonus points! See this example

@guilhermefarto Not out of the box. sorry

Alright, thank you! 👍


add missing rspec test tag

without the 'couchdb' tag no test is run.

This is indeed a problem for testing.  But the DEVELOPER.md file may be overwritten in the future.  It's a blanket-level form we pushed to ALL of the plugins.  If it were to happen again, it would overwrite this change.
I think we were planning on having a file with local instructions for this sort of thing.  Let me verify before we merge this.

@emig can you please perform step 2 of https://github.com/elasticsearch/logstash/blob/master/CONTRIBUTING.md#contribution-steps

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then  say 'jenkins, test it'.


Update: Telegram.TelegramDesktop version 2.8.10

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

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.0 schema?

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

Microsoft Reviewers: Open in CodeFlow

@ItzLevvie you gotta approve my PRs, jedieaston seems to be too busy lately since hes not even looking at his own automated PRs


NTR: Endoscopic Retrograde Cholangiopancreatography

Wiki def: Endoscopic retrograde cholangiopancreatography is a technique that combines the use of endoscopy and fluoroscopy to diagnose and treat certain problems of the biliary or pancreatic ductal systems.
syn: ERCP

close


where can i find published (pretrained) models for tiny imagenet?

Checklist

I have searched related issues but cannot get the expected help.
I have read related documents and don't know what to do.

Describe the question you meet
Thank you for the comprehensive library. Its capabilities are a little overwhelming to absorb at first. I was led to this repo from the paperswithcode tiny imagenet benchmark. I am just looking for a pretrained model of resnet18 on tiny imagenet. Usually such links are part of the model zoo, but i think the structure is a little different here.
Is there any link to such a model? I see on getting started there is facility to publish a model, so is there any listing of publicly available such models?
thanks again for your work.

Hi, @tumble-weed, please refer to README in MogaNet configs. You can download all files from openmixup-moganet-in1k-weights, timm-moganet-in1k-weights, or Baidu Cloud: MogaNet (z8mf). I will close this issue, and you can open a new issue if there are more questions.


Lock a meeting

Add a button to lock a meeting

A conference locked indicator is displayed.

A message : Waiting for the Host is displayed for a new guest connexion
In meeting :
we hear the sound of a door knocking
the participants icon changes to warn that there is a person in the waiting room
all host participant can accepte or reject the guest with participant menu

a keyboard shortcut can lock or unlock meeting

It's on our roadmap


Multiple HTTP requestors

Now that Push support is in, need at least 2 simultaneous HTTP worker threads. That way the push can operate and PCAP web download also. currently they will block each other.
Its also a general feature improvement across all the systems

closing


Blurry flipped cursor on line number

VSCode Version: 1.40.1
OS Version: macOS 10.15.1

Steps to Reproduce:

Change the color theme to a dark theme
Move the mouse cursor to the line numbers
The mouse cursor blurs and it looks quite different from the system's default mouse cursor

Does this issue occur when all extensions are disabled?: Yes
Or maybe you can offer an option to disable the flipped cursor feature to solve this issue?

@misolori Here is the CSS we have for the inverse cursor: https://github.com/microsoft/vscode/blob/ff7e302ad6c3ce70e2546ef136b1bf3163bcdeef/src/vs/editor/browser/viewParts/lineNumbers/lineNumbers.css#L29-L34

@alexandrudima what was the rationale for flipping the cursor in this instance? I always thought it was a bit odd.

I think we took it from Visual Studio or from Word, not 100% sure...

@alexandrudima @rebornix what do you think about making this a regular cursor: pointer on hover instead of using a custom one? This would match all of our other hover cursors in the gutter:

@misolori We can try removing the flipped cursor, but I wouldn't make it a hand/pointer, I would just leave it as is... Having it be a pointer over the line numbers feels weird to me.

I'm ok with the default cursor as well, the pointer was mostly to indicate that you can select an entire line.

This now uses the default cursor (non-flipped) when hovering over line numbers:


Python 2.7 compatibility

Hi
is it really necessary to drop Python 2 support? So far the only thing incompatible that I've seen are the type hints for example in openvr/__init__.py:2719.
I'm using pyOpenVR in a ROS environment which so far still uses Python 2. I know, Python 2 is going to be EOL by the end of this year. But working around these issues is a pain in the butt when you have a million packages that still require it while others simply don't work with it. I would suggest keeping Python 2 compatibility up as long as possible or at least make a backwards compatible branch.
So, a little more concrete suggestion from my side: make an intermediate release with the latest still Python 2.7 compatible code. That would be awesome!
Cheers

@HWiese1980 Can you use pyopenvr version 1.3.2201? That version might still work with python 2.

Yes, that version works. Ah, and there's the corresponding tag. Just out of curiosity... without having taken a deeper look into it myself, what else makes it Python 2 incompatible other than the mentioned type hints?

I don't know of anything specifically other than the type hints that breaks python 2 compatibility. The problem is that I lost all interest in python 2 two years ago, and I can only test and maintain things here that interest me. I like type hints. I no longer maintain a sister installation of python 2 for testing. I don't want to support something that I cannot personally test.
On the other hand, if you, or someone else interested in python 2, were to express a genuine commitment to testing and debugging python 2 support on an ongoing basis, with each new release of pyopenvr, for, say, at least the next year, I might be willing to work with you. But I'm not going to maintain such support on my own.

I can comprehend. I've personally also lost interest in Python 2 because of its imminent EOL. It's just that my job requires using stuff that is not yet supported by Python 3, like ROS and its many packages for instance (unfortunately). It's an inconvenient situation. On one hand we all know that it's going to end, on the other we have to stick with it because others don't seem to really care. I'm really curious to see what happens 2020 when Python 2 is killed off.
Anyway, there are other ways to do the testing. I suppose you've heard of continuous integration? TravisCI? CircleCI? That way you could have your code tested for Python 2 compatibility at every push without the need of a local installation of Python 2.

@HWiese1980 are you able to get by for now using version 1.3.2201?

Yes. I do for the time being. I just hope that triad_openvr will not break anything and stay compatible. I mean, after all I could stick with some older versions of everything but if there are bugfixes and new, fancy features that I need, it's be a pity if compatibility breaks in the process.


about Q smoothing loss

https://github.com/YangRui2015/RORL/blob/8924ee56a6df8b5710bbe022494b056243d7a6af/lifelong_rl/trainers/q_learning/sac.py#L250
I think that maybe the Q smoothing loss is implemented differently from the paper.
It seems that the 'diff' is comparing different observations. (instead of comparing original observations and perturbed observations).
diff = noised_qs_pred - qs_pred.repeat(1, 1, size).reshape(self.num_qs,M*size,1)
maybe correct?...  or my misunderstanding?

Hi, you are right. I have fixed it and reported it in the README.

Thank you for your reply.
However, after fixing, I wonder that whether the performance reported in the paper is still reproducible.
If you check this out later, please let me know too.
-----Original Message-----
From: @.>
To: @.>;
Cc: @.>; @.>;
Sent: 2022-11-26 (토) 14:04:54 (GMT+09:00)
Subject: Re: [YangRui2015/RORL] about Q smoothing loss (Issue #1)
Hi, you are right. I have fixed it and reported it in the README.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Sure. In our ablation study, the order of performance contribution is: the OOD loss > the policy smooth loss > the Q smooth loss. And we use a small coefficient 0.0001 for the Q smooth loss. I think the fix can even improve the performance a bit and I will have a check about the difference.
Thanks a lot！

Thank you for your reply. However, after fixing, I wonder that whether the performance reported in the paper is still reproducible. If you check this out later, please let me know too.
…

Hi, I have checked several experiments and the results are still within the std range as reported. The benchmark results is shown below. I also noticed that walker2d-medium-v2 task requires slightly larger OOD regularization than before (by tuning the OOD regularization coefficient $\lambda$ and decay pace $d$ ) to stabilize the instability raised from the updated Q smooth loss. I have updated the code with its new hyper parameters.
The benchmark results for hopper and walker2d:
Hopper-random 31.31 +- 0.11
Hopper-medium 104.60 +- 0.24
Hopper-medium-expert 113.39 +- 1.50
Hopper-medium-reply 102.61 +- 0.44
Hopper-expert 114.43 +- 1.57
Walker2d-random 21.56 +-  0.05
Walker2d-medium-v2 101.06 +- 2.02
Walker2d-medium-exp 121.24 +- 1.72
Walker2d-medium-replay 93.06 +- 2.99
Walker2d-expert 116.75+-1.43

Thank you for your reply. However, after fixing, I wonder that whether the performance reported in the paper is still reproducible. If you check this out later, please let me know too.
…

Thanks!


inconsistent "mode" capabilities

Does it make sense to have both rcMode and rcSendKeysStrategy?  I don't think it does.  It would seem that rcMode: <proxy|remote> would be sufficient, and control both regular key presses and text input behavior.
I'd also like to rename rcMode js to proxy
@jlipps

hmm. the reason i chose js is because we're sending javascript to do it. the fact that we're using chromedriver is kinda beside the point (we could use direct remote debugger calls as well). it's not as if chromedriver has any kind of "remote" functions.
i'm provisionally open to combining the send keys strategy and remote mode into a single capability, i just know someone will want to pick and choose at some point though :-p

at the very least i think we should be consistent with the names.
picking and choosing is only sort of possible. you can use the proxy to send keystrokes in either mode, but you can't use the remote to send keystrokes if you're not already in remote rcMpde.
another way to do it would be to just use one mode flag and provide a second command, e.g. "tizen: setValue" which does it via rc. and setValue itself is removed and instead just proxies. I'm not sure if that jives with peoples expectations about these APIs though

yeah... that's probably not a bad idea. it means people can't do element.send_keys('foo') with the remote, but it was always a bit of a lie, because in rc send keys mode it didn't really matter what element you called it on.


Add requiresMainQueueSetup to fix warning in RN 0.49+

After upgrading to RN 0.49 I get the following warning:
Module ReactNativeConfig requires main queue setup since it overrides constantsToExport but doesn't implement requiresMainQueueSetup. In a future release React Native will default to initializing all native modules on a background thread unless explicitly opted-out of.

This PR should fix that

Rocking! Thank you!


Make FakeJWTAuthenticatorService working without started app

When the encryptSubject configuration is activated, the encryption rely on Crypto.encryptAES which require a started application (https://github.com/playframework/playframework/blob/2.3.x/framework/src/play/src/main/scala/play/api/libs/Crypto.scala#L59).
By just setting encryptSubject to false in test, it should allows to initiate the JWSAuthenticator before the WithApplication block.

Coverage increased (+1.08%) to 100.0% when pulling db3b22576588d752e03bf5b9605441076f712e8f on studiodev:patch-2 into 76bb34cc1f0b4f8b8162762d00033c702bed1f59 on mohiva:master.

Nice catch. Thanks


[Doc]Add in the home page full AOT support feature

here https://thomhurst.github.io/TUnit/
I would say "loudly" that the adapter support or is designed to work full native AOT/Single file in home page.
It's pretty unique feature for a testing framework, at the moment at least.

Good shout! I haven't actually managed to add the Single File testing into my pipeline yet though due to https://github.com/microsoft/testfx/issues/3758

All done now 😄


Test support for arm64

What does this PR do?
This PR adds ARM64 job to SemaphoreCI.
Modified files mesh/.semaphore/semaphore.yml for running the test for arm64 platform.
Signed-off-by: odidev<EMAIL_ADDRESS>

 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.

Hello,
We don't need to run all the tests on arm64 because we use nothing related to a specific arch.
Spending energy for that, through our CI, is not environment friendly and it's useless.
Also, the workflow already creates an image for the different arch.


Package readme display creates a  for every markdown newline, against markdown specification

See the readme-provided badges on this package: https://www.nuget.org/packages/Devlooped.TableStorage/2.0.1.

The source readme is https://github.com/devlooped/TableStorage/blob/v2.0.1/readme.md.
Notice how github.com does not add newlines even if the badges are defined in the readme source in separate lines (but without an empty line break between them), and instead just considers the line break as a single whitespace instead:

snip of source readme:
![Icon](https://raw.githubusercontent.com/devlooped/TableStorage/main/assets/img/icon-32.png) TableStorage
============

[![Version](https://img.shields.io/nuget/v/Devlooped.TableStorage.svg?color=royalblue)](https://www.nuget.org/packages/Devlooped.TableStorage)
[![Downloads](https://img.shields.io/nuget/dt/Devlooped.TableStorage.svg?color=green)](https://www.nuget.org/packages/Devlooped.TableStorage)
[![License](https://img.shields.io/github/license/devlooped/TableStorage.svg?color=blue)](https://github.com/devlooped/TableStorage/blob/main/LICENSE)
[![Build](https://github.com/devlooped/TableStorage/workflows/build/badge.svg?branch=main)](https://github.com/devlooped/TableStorage/actions)

Repository pattern with POCO object support for storing to Azure/CosmosDB Table Storage

I'd expect the nuget.org render to follow more closely with github.com in this particular case, since it's not uncommon for projects to provide several such badges and forcing to put them all without line breaks (as I had to do for this package) hinders source readability and maintainability. I think it's also the more common markdown behavior for single line breaks.

@kzu , thanks for reporting! We will take a look in July. If you are interested in sending a PR we will gladly work with you on merging and shipping this change.

Sure thing! Which repo has the code for the gallery?

@kzu, here is repo: https://github.com/NuGet/NuGetGallery
I will work with you to ship this change once you have PR ready.
Let me know need if you anything

Another similar report, simply based on newline behavior in text. From our assessment, this seems to be the same root cause as https://github.com/NuGet/NuGetGallery/issues/8647.

Yes, indeed this is just a case of creating a <br> for every newline in the markdown, which is something explicitly called out in the markdown specification as something that should not be done. See:

The implication of the “one or more consecutive lines of text” rule is that Markdown supports “hard-wrapped” text paragraphs. This differs significantly from most other text-to-HTML formatters (including Movable Type’s “Convert Line Breaks” option) which translate every line break character in a paragraph into a  tag.

See https://www.nuget.org/packages/dotnet-gcm/ for another example.

@lyndaidaii This still seems to happen (again) when uploading a package to nuget.org. Could you please check that again?

This issue should be reopened as Nuget creates new lines against the Markdown spec.


Merge changes from feature/edge: introduce CI/CD

Closing this pull request due to error in configuration.


feat: manage tables and fields as objects

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

278 of 304 (91.45%)  changed or added relevant lines in 17 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.9%) to 93.692%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Field/TypeDrop.php
3
4
75.0%

src/Field/TypeIncrement.php
4
5
80.0%

src/Field/TypeRename.php
8
9
88.89%

src/Field/TypeString.php
12
13
92.31%

src/Table.php
27
28
96.43%

src/TableBuilder.php
37
40
92.5%

src/Field/TypeDate.php
6
15
40.0%

src/Field/TypeDatetime.php
6
15
40.0%

Totals

Change from base Build<PHONE_NUMBER>:
-1.9%

Covered Lines:
906

Relevant Lines:
967

💛  - Coveralls


fix(docs-table): Fix inaccurate property name usage for selectionMode.

Thanks! @zhulien-ivanov 🙏🏻


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_95422574-281a-4bd6-a5af-8948829342d2/pull/1 is merged into live branch.
Auto archive content to https://github.com/OPS-E2E-PPE/docs-archive-test-target_95422574-281a-4bd6-a5af-8948829342d2.git

Docs Build status updates of commit 32cb004:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Archive_master_2019-09-30-03-03-14 has been deleted or has been force pushed: fatal: Couldn't find remote ref Archive_master_2019-09-30-03-03-14

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


Bind services to localhost in tests

To avoid approval requests from macOS' firewall.

Description
Changes proposed in this pull request:

Bind services to localhost in tests

Related issue(s)

/retest


Remove duplicate reports_count parameter

Duplicate parameters now return an error starting with Puppet 3.8.5 and 4.3.2. https://tickets.puppetlabs.com/browse/PUP-5590

Agreed

Same for me. 2.7.3 became useless without that patch

+1


Fix "reference before assignment" error when logging ignored entities

This only causes an error when the earlier for entity in ordered_entities: does not create a variable named entity (i.e. when there are not yet any replaceable entities in the project). However even in the case where it doesn't throw an error, it will log erroneous information (because it is referencing entity rather than db_entity which is the object being considered).

Codecov Report

Merging #42 (c6baff0) into master (a1060ee) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #42   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           17        17
  Lines          787       787
=========================================
  Hits           787       787

Flag
Coverage Δ

unittests
100.00% <ø> (ø)

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

Impacted Files
Coverage Δ

src/alembic_utils/replaceable_entity.py
100.00% <ø> (ø)

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 a1060ee...c6baff0. Read the comment docs.

thank you
the fix is available in alembic_utils==0.5.5


MatSidenav with position="end" is in the wrong tab order

Bug, feature request, or proposal:
Sidenav doesn't seem to meet https://www.w3.org/WAI/WCAG21/Understanding/meaningful-sequence.html and https://www.w3.org/WAI/WCAG21/Understanding/focus-order.html. Visual order should generally match DOM order and tab sequence.
What is the expected behavior?
Being able to position sidenav either before or after the main content.
What is the current behavior?
Sidenav always appears first in DOM and tab order even if positioned after the content.
Looking at this example you will notice that as you tab from the url to the page it goes from the right panel (Link2) to the main content (Link1), rather than the other way around.

What are the steps to reproduce?
Providing a StackBlitz reproduction is the best way to share your issue.
StackBlitz starter: https://goo.gl/wwnhMV
https://stackblitz.com/edit/angular-xww88d
What is the use-case or motivation for changing an existing behavior?
To meet WCAG 2.1 requirements
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Material 7.3.1
Is there anything else we should know?
Would be nice if this allowed users the option to control order in the DOM. It may need to change depending on the sidenav mode (over, push, side).

Thanks @crisbeto !
I will watch this issue and the PR.
I did not know that this had already been opened.
This will be a create add on for
https://github.com/angular/components/issues/18100

This also seems to be relevant issue https://github.com/angular/components/issues/11992

I am still seeing this issue. has it been addressed?
stackblitz example.  Open both sidenavs, and tab.  the div in the content comes after the sidnavs
https://stackblitz.com/edit/angular-hh7pzd?file=src/app/sidenav-responsive-example.ts


update network management

What current issue(s) from Github does this address?
What problem does this PR solve?
How did you solve this problem?
How did you make sure your solution works?
Are there any special changes in the code that we should be aware of?
Is there anything else we should know?

[ ] Unit tests written?

@prophit987 Please, rebase your branch with CoZ/master and resolve the conflicts.


Expose the node's min/max/step

After creating the DSP, the ability to inspect the State to retrieve the parameter's metadata would be very useful, both for creating UI, or to avoid ranges duplication between the .dsp and the rust sources.

Closing this in favor of https://github.com/Frando/rust-faust/pull/12/


HTML entities contained in an XML string are not decoded by the SvgXml component

This is the XML string I have which consist of two HTML entities in the tspan tag
Everything functions perfectly when I use this string in react-svgr.com; the entities are immediately decoded.

However, the text is presented in the encoded version when I use this string as an xml prop in the SvgXml component.

Hello @hellonobug9,
Could you provide an example to reproduce that issue, thank you.


Add outline for a User's Guide through RTD

Create initial documentation framework so that the docs live in the repository and are served up via RTD.
Start with a User's Guide that is a mostly empty outline. Recommend including chapters for:

System Architecture
Administration

Instructions to set up and run the WRF cloud framework on AWS.
Adding new users
Setting user permissions

User Interface
Graphical Outputs

@pkucera22 I have 2 questions for you?

Is COMET an acronym for anything? If it is, I certainly can't find it on it's website. I did find a competing AGU/COMET defined as "Cooperative Program for Operational Meteorology, Education, and Training" but I assume that's different.
Should the documentation for this work reference only UCAR/COMET or should it also reference NCAR/RAL? I'm working on this initial version of the user's guide and need to know what org(s) should be listed.

Thanks!

Hi John,
Thanks for reaching out with these two questions.  The story that I have
been told is that COMET used to mean the acronym that you mention, but now
it is just COMET ... like it is its own brand.  We definitely want to
include RAL in the reference as this is a joint, collaborative project.
You could standardize the references to something like: UCAR/UCP/COMET and
UCAR/NCAR/RAL or something like that.
Cheers,
Paul
On Tue, May 10, 2022 at 3:26 PM johnhg @.***> wrote:

@pkucera22 https://github.com/pkucera22 I have 2 questions for you.

Is COMET an acronym for anything? If it is, I certainly can't find
it on it's website. I did find a competing AGU/COMET defined as
"Cooperative Program for Operational Meteorology, Education, and Training"
but I assume that's different.
Should the documentation for this work reference only UCAR/COMET or
should it also reference NCAR/RAL? I'm working on this initial version of
the user's guide and need to know what org(s) should be listed.

Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/NCAR/wrfcloud/issues/1#issuecomment-1122884881, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHGC645OVRSILRAEVZM4LSLVJLH7ZANCNFSM5VORHJQQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
//
Paul A. Kucera, Ph.D.
COMET Assistant Director
International Capacity Development Program
University Corporation for Atmospheric Research
P.O. Box 3000
Boulder, CO 80307-3000 USA
Ph:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
Email: @.
/**/


CLIMATE-729 - Remove config file from NARCCAP examples

Remove Fig7_w from NARCCAP Examples configuration file

Can one of the admins verify this patch?


Add required action for phone validation

[x] Add required action for phone validation
[x] Implement SMS code validation for this action

After phone number submission, the phone is stored in AuthenticationSessionModel and a PhoneValidationRequiredAction is triggered. Phone validation sends SMS code and finally stores users phone number after valid SMS code submission.
TODOs
When the SMS code validation is aborted without valid code submission, the PhoneNumberRequiredAction is considered as "successful" despite of SMS Authentication could not be setup due to the missing SMS code.
In this case PhoneNumberRequiredAction should be considered as failed.
Fixes #16

Potential solution: at the beginning of step 2 create another required action for step 1. If step 2 is successful, delete the required action for step 1.

@melegiul I'd suggest to merge as is and create a new issue for the static Config Alias. I already updated the README in the main branch accordingly.


chore(panoviewer): add projects built with panoviewer to page

Issue
#30
Details

Add projects built with panoviewer to page
Fix file size on page
Add PanoViewer thumbnail image to assets folder

Coverage remained the same at 16.397% when pulling 06a09b79caecb3b459e93aacb0650493daa9b1f4 on happyhj:panoviewer#30 into 60512767b10015aec6fc775640bff46f9160c422 on naver:master.


[Question] Reading the framebuffer

Is there a way, on a remarkable 2, to read the framebuffer when xochitl is running (in order to stream it to another device)?

There is a pull request on reStream that does just that: https://github.com/rien/reStream/pull/31

there are two ways (in this repo):

you can look at src/xofb for a standalone LD_PRELOAD shim (librm2fb_xofb) that will expose the framebuffer in /dev/shm
if you use the rm2fb server, when you hook xochitl with LD_PRELOAD of librm2fb_client, it will also expose the framebuffer in /dev/shm/

i believe people have had success with xofb and restream (by updating a few variables in reStream)