[BUG] Wrong total number of items when migrating a VM

Describe the bug
When migrating a VM, the total number of items shows all of them, but you can only choose from the suitable targets which is much less - this makes the dialog confusing.
To Reproduce
Steps to reproduce the behavior:

Log in as Admin
Prepare several clusters and hosts with different storage types.
Go to 'Compute > Instances' and choose an instance.
Click on 'Migrate' button

Expected behavior
The total number of items must be calculated only for the suitable targets, not all.
Screenshots

Desktop (please complete the following information):

OS: Ubuntu 18.04 LTS
Browser: Chrome
Version: 85.0.4183.121 (Official Build) (64-bit)

Additional context
None.

@rhtyd @davidjumani cc @svenvogel I think this comes from the result count of API was wrong?

That's right @utchoang Will look into creating a backend fix for this

@vladimirpetrov can you test latest master and close this? thnx

Fix confirmed.


Add GitHub action to run tests

To run tests:
mkdir build-tests
cd build-tests
cmake -DBUILD_TESTS=On ..
make -j4
ctest

Resolved in #2304 #2310


[User Story] As a data analyst I want the API to summarize database data and send summaries to the web application to be displayed

Is your user story related to a problem? Please describe.
It frustrates me when I cannot view the data in a clear, concise manner.
Describe the solution you'd like
I want the API to be able to create a summary of the data to be displayed on the web application for easy access.
Describe alternatives you've considered
A summary is the only applicable solution for displaying the data in a concise manner, and command line or shell output is not preferable to the web application.
Additional context
The summaries should be organizable by machine.

This was accomplished via the front end


Page iterator

This is definitely a first cut but the desire on Slack was to create a PR for it now.  I have not had a chance to clean this up, some specific things that need cleaning are:

error handling is sloppy and inconsistent, it many cases we just panic when encountering some parquet feature or variant that is not yet handled
various bits of duplicated code
little attention has been given to comments, names of functions/variables, ordering of functions in the source filed, etc.
the wide reader interfaces that have methods for every parquet primitive type make for very verbose code.  @mccanne 's suggestion for this is to push the zng encoding all the way into the iterators which would also improve performance.

I think the last item would be best done in a separate follow-up.  As for the rest, I'm aware of all the cleanup that needs to happen, a review with detailed comments about those issues is probably not a good use of anybody's time at this stage...

As discussed in person, we'd like to merge this as-is (pending review) and then clean up other things (consistent error handling in particular) in a follow-up PR.

I let @aswan know that I hit a panic with an internal test file:
https://gist.github.com/alfred-landrum/cab2c5c67f906f94aa2430a7c9e96366
He's taking a look.


Auth::attempt() and Redirect::intended() has issues

Starting on /
Go to /login to enter the form
Press submit

            if (Auth::attempt($credentials, $remember)) {
                Log::info('Is logged in.');
                return Redirect::intended('/');
            } else {
                return Redirect::to('/login');
            }

After successfully entering the credentials (log tells me so), Redirect::intended() thinks the user is not logged in and redirects them to /logout and logs them out.
If I start on any other page, then the log in works and the user gets redirected to the intended page.
I know this because when I insert a Log::info($redirect) here it logs this.
I am trying to understand where it go the logout route from. I grepped my folder and only 2 instances of the text came inside the logout method in the routes, which when the route was disabled, I get a 404 during the redirection.

Routes file (relevant routes):
// Auth
Route::group([
], function() {
    Route::get('/login', 'AuthController@show');
    Route::any('/logout', 'AuthController@logout'); // comment this out and get 404 in above redirection
    Route::get('/password/forgot', 'AuthController@forgot');
    Route::get('/password/reset/{token?}', 'AuthController@reset');

    Route::group([
        'before' => ['csrf'],
    ], function() {
        Route::post('/login', 'AuthController@login');
        Route::post('/password/forgot', 'AuthController@send');
        Route::post('/password/reset', 'AuthController@update');
    });
});

What laravel version?

Laravel Framework version 4.2.16

I figured out what is happening and I think it might be a bug with Laravel.
Relevant controller method:
// Log out route
public function logout()
{
    Auth::logout();

    return Redirect::to('/');
}

I think because going to http://site.com/logout redirects the user somewhere after logging them out, Laravel sets the session url key to: array:1 [ "intended" => "https://site.com/logout" ] because of the redirection. Now when the user tries to log in again, because the (falsely) "intended" route is saved as the logout url, it kicks them back out thanks to return Redirect::intended('/');
Current workaround is to manually forget the session 'url' key (which is not always desired): Session::forget('url');

Actually the issue stems from the Redirect::guest() which preserves the intended url.
The solution is to use Redirect::to().


(WIP) Add hyperdrive api tests & fix indexing

Adds tests for hyperdrive API
TODO: fix indexing bug...

I though it'd be as easy as setting the default as {indexing: false} and setting it later on dat.importFiles() if we wanted indexing: true. But we can't set it later in hypercore and there is no indexing option on write. Need to fix it there first.

This is superseded by the dat-sdk


Building fennel plugin with hotpot

I saw your paperplanes.nvim plugin and thought that it would be nice to try build my own fennel plugin. I tried to find a way to compile fnl directory into lua. I found #18 issue, but seems like it's closed without providing info whether it is implemented or not. After trying to copy build.fnl and hotpotfile.fnl i get an error about trying to index "f_out" variable which is not even being called inside those .fnl files. I hope you could explain any way to compile fennel plugins using hotpot.nvim

It's probably easiest to just use one file, call it build.fnl and copy most of the code out of build.fnl from pact which has some code to ignore compiling macro files.
;; Copied
(local fennel ((. (require :hotpot.api.fennel) :latest)))
(local uv vim.loop)

(fn compile-file [in-path out-path]
  (if (not (string.match in-path "macros?%.fnl$"))
    (with-open [f-in (io.open in-path :r)
                f-out (io.open (string.gsub out-path ".fnl$" ".lua") :w)]
               (print :compile-file in-path :-> out-path)
               (local lines (fennel.compile-string (f-in:read "*a")))
               (f-out:write lines))))

(fn clone-file [in-path out-path]
  (with-open [f-in (io.open in-path :r)
              f-out (io.open out-path :w)]
             (print :clone-file in-path :-> out-path)
             (let [lines (f-in:read "*a")]
               (f-out:write lines))))

(fn handler-for [name]
  ; if lua file, just clone over
  ; if fnl file, compile out
  (match (string.match name "%.[%w]+$")
    nil (error (: "Could not find extension in %q" :format name))
    ".lua" clone-file
    ".fnl" compile-file
    any (error (: "Could not handle file type %q" :format any))))

(fn compile-dir [in-dir out-dir]
  (print :compile-dir in-dir :=> out-dir)
  ;; TODO: No checks for missing source dirs
  (let [scanner (uv.fs_scandir in-dir)]
    (var ok true)
    (each [name type #(uv.fs_scandir_next scanner) :until (not ok)]
      (match type
        "directory" (do
                      (local out-down (.. out-dir :/ name))
                      (local in-down (.. in-dir :/ name))
                      (vim.fn.mkdir out-down :p)
                      (compile-dir in-down out-down))
        "file" (let [in-file (.. in-dir :/ name)
                     out-file (.. out-dir :/ name)
                     handler (handler-for in-file)]
                (handler in-file out-file))))))
;; Added
(compile-dir :fnl :lua)
;; Open this file and run `:Fnlfile %` with hotpot installed

Then open build.fnl and run :Fnlfile % , you should see a bunch of compile fnl -> lua messages?
I closed that issue because it had been sitting stale for a long while and I wasn't sure if people would really care to use any builder, or if I still wanted to go the same direction with a separate file. I will re-think about it!

It works, thanks! I will close this issue as you have answered my question


[skip ci] Redefine SSL variables

Allow user to pass the certificate in
haproxy_frontend_ssl_certificate_content and to store it in
haproxy_frontend_ssl_certificate_path.
Signed-off-by: Rafał Wądołowski<EMAIL_ADDRESS>

@xelexin I think this is a breaking change.
Unless I'm missing something in the current implementation, I think haproxy_frontend_ssl_certificate excepts a path to a certificate.
With your change, if someone has deployed with the current implementation and doesn't update his haproxy_frontend_ssl_certificate variable, he will get a /etc/haproxy/haproxy.pem containing a path and a haproxy.cfg pointing to this certificate ending up with a broken configuration.

@guits  ready for review

@guits

@guits


Show a note under "Statement" if no statement is available in the browser's locale

When I go to a profile page, I expect to read their statement in my language. When I see that it's missing, I might think that the site hasn't translated something.
It'd be great to have a simple test that shows something like

The user has not provided a statement in a language you've requested, so this statement is in the user's primarily language.

The test is simple: if the profile shown isn't in the browser's Accept-Language request header, show the above.

+1

Made a little progress on this tonight. I realized when doing some testing that we don't currently show a statement at all if one is not available in an accepted language. I might need to revisit this or dumb down that check here.
@Changaco Thoughts?

We should list the languages in which the user has written a statement, and allow the visitor to switch between them. It's only to keep it simple that I didn't implement this in the original PR.


Inline loaders don't work with asset type

Bug report

What is the current behavior?
I'm trying to use an inline loader with !! to have some unique behavior for several modules.
import rawTxt from '!!raw-loader!./raw.txt';

But if rule has one of type asset/resource, asset/inline, asset/source or asset it will be transformed with this type, not the inline loader. But if a rule has use inline loader will be applied and loaders from use will be ignored.
Rule configuration:
{
    test: /\.txt$/i,
    type: 'asset/inline',
}

So in this case rawTxt will be base64, not raw.
If the current behavior is a bug, please provide the steps to reproduce.
https://github.com/7rulnik/webpack-asset-inline-loader-bug-repro

What is the expected behavior?
I expect that behavior should be the same for rules with type and for rules with use. So even for asset/* types inline loader should have priority.
As I understand I can use resourceQuery to workaround this issue.

Other relevant information:
webpack version: 5.24.4
Node.js version: 14.15.4
Operating System: macOS 11.2

As I understand I can use resourceQuery to workaround this issue.

It is not workaround, it is official solution https://webpack.js.org/guides/asset-modules/#replacing-inline-loader-syntax
In long term we need to avoid and remove inline syntax for loaders (we keep them for backward compatibility), because it is out of ECMA modules spec, also it is specific webpack syntax, it is bad for future JS ecosystem.
I think we need update docs on new approach.

Answered


added release workflow

This change is

Codecov Report

Merging #69 into master will decrease coverage by 19.4%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #69       +/-   ##
============================================
- Coverage     90.51%   71.1%   -19.41%
+ Complexity     1565    1288      -277
============================================
  Files           294     292        -2
  Lines          3659    3655        -4
  Branches        130     130
============================================
- Hits           3312    2599      -713
- Misses          274     984      +710
+ Partials         73      72        -1

Impacted Files
Coverage Δ
Complexity Δ

...ter/events/pagefragments/SelectedByIndexEvent.java
0% <0%> (-100%)
0% <0%> (-3%)

...ter/events/pagefragments/SelectedByValueEvent.java
0% <0%> (-100%)
0% <0%> (-3%)

...ec/testit/webtester/pagefragments/NumberField.java
0% <0%> (-100%)
0% <0%> (-3%)

.../testit/webtester/internal/OffersAdHocFinding.java
0% <0%> (-100%)
0% <0%> (-4%)

...ostconstruct/PostConstructInvocationException.java
0% <0%> (-100%)
0% <0%> (-1%)

...vents/pagefragments/AbstractPageFragmentEvent.java
0% <0%> (-100%)
0% <0%> (-2%)

.../novatec/testit/webtester/pagefragments/Table.java
0% <0%> (-100%)
0% <0%> (-12%)

...ec/testit/webtester/pagefragments/GenericList.java
0% <0%> (-100%)
0% <0%> (-4%)

...tester/pagefragments/mapping/MappingException.java
0% <0%> (-100%)
0% <0%> (-1%)

...o/novatec/testit/webtester/pagefragments/Form.java
0% <0%> (-100%)
0% <0%> (-1%)

... and 69 more

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 d0fa21a...7cea2e1. Read the comment docs.

Codecov Report

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

@@            Coverage Diff            @@
##             master      #69   +/-   ##
=========================================
  Coverage     90.51%   90.51%
  Complexity     1565     1565
=========================================
  Files           294      294
  Lines          3659     3659
  Branches        130      130
=========================================
  Hits           3312     3312
  Misses          274      274
  Partials         73       73

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 d0fa21a...e007878. Read the comment docs.


Default formatter in en locale not working

Hello,
I am calling pivotUI using the default set of aggregators and the default locale of en, but no formatting is being applied to my numerical data and instead they are being displayed as the raw values in the data. E.g, values are not being formatted using the thousands comma separator, and values are not being truncated to two numbers after the decimal point.
Could anyone possibly advise as to why this is?
Thanks

Can you provide an example of this happening, say on JSFiddle or something, so I can take a look?
What OS/Browser are you using?
Does this page display correctly-formatted numbers? http://nicolas.kruchten.com/pivottable/examples/mps_prepop.html

I think i've figured out the problem - I didn't realise that the nice formatting for numbers is only applied to the aggregation rows / columns and not the raw data itself. i.e https://jsfiddle.net/w14L7yrn/
Here, the income column (the raw data) is not formatted nicely. I realise that this probably isn't what the formatter is intended for, but is there a way in which I could use the built in number formatters to format the data in this scenario?
Thanks!

I've found a solution; I just passed all numeric data through the default numberFormatter when the rows are rendered in the pivotTableRender


A specific package is always reinstalled since pacaur v4.7.0

Version
pacaur 4.7.1

Description
After pacaur was updated from v4.6.10 to v4.7.0 the package wire-desktop-bin is always reinstalled, even though it hasn't actually changed. I have other AUR packages installed as well, but only this one is affected. Pacaur versions up to 4.6.10 didn't exhibit this.
Output
$ pacaur -Syua
:: Synchronizing package databases...
 core is up to date
 extra is up to date
 community is up to date
 multilib is up to date
:: Starting AUR upgrade...
:: resolving dependencies...
:: looking for inter-conflicts...
:: wire-desktop-bin-2.11.2703-1 is up to date -- reinstalling

AUR Packages  (1)     Old Version  New Version

aur/wire-desktop-bin  2.11.2703-1  2.11.2703-1

:: Proceed with installation? [Y/n]

Debug output
bash -x pacaur -Syua > pacaur_debug.txt 2>&1
https://gist.github.com/pesho/7b3fd9ca52f034894fe02329fdfbd289

Adding --needed seems to be a workaround for now:
$ pacaur -Syua --needed
:: Synchronizing package databases...
 core is up to date
 extra is up to date
 community is up to date
 multilib is up to date
:: Starting AUR upgrade...
:: resolving dependencies...
:: looking for inter-conflicts...
:: wire-desktop-bin-2.11.2703-1 is up to date -- skipping
 there is nothing to do

I cannot reproduce, but it looks like something similar to #563 is happening.
Haven't found any issue yet, but I'll have a closer look tomorrow.

This is due to a sort issue, which is locale dependent.
+ for i in "${!allaurpkgs[@]}"
++ vercmp 2.11.2703-1 20161022-1
+ [[ -1 -gt 0 ]]
+ for i in "${!allaurpkgs[@]}"
++ vercmp 20161022-1 2.11.2703-1
+ [[ 1 -gt 0 ]]

Version comparison between local and AUR version of the w_scan and wire-desktop-bin are swapped.

Should be fixed in 10b882eb4e52b12c66e33a27e99cdebbaf7ed874. Feedback welcome.

Should be fixed in 10b882e. Feedback welcome.

Awesome! Just tested the last commit, the issue is indeed fixed. Thanks a lot! :trophy:


Update files-troubleshoot-linux-nfs.md

NFS cannot be used with windows clients and windows servers. But this document says it cannot be used only for Windows clients which not a relavent information. Hence addind the word server

@khdownie
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Fixed in another PR, changes will be live in a couple hours

#please-close


Cluster support

There is a roadmap or a due date for the cluster support become available? Is there any simple alternative that I could implement to solve this point?

So, it just needs some testing. It should already be possible with the combination of:
https://github.com/mcollina/mqemitter-child-process
https://github.com/mcollina/aedes-persistence-level
https://www.npmjs.com/package/level-party
There might be bugs, and I have never tried it.
Please open bugs if you find them.

Do you recommend any tests? I'm a bit lost in what kind of test I should do.

Write a file where you use all three, and check if it works with cluster.
Put it on a gist, so I can easily check what's going on, and help if needed.

Can https://github.com/mcollina/mqemitter-child-process be used with redis-persistence?

it seems https://github.com/mcollina/mqemitter-child-process only scales-up, what about scaling out (clustering against different machines)?

yes.
currently not. Performance-wise, you are much better in using redis for everything if you are on multiple machines. However, you can send a PR and make this configurable https://github.com/mcollina/mqemitter-child-process/blob/master/index.js#L43.

child-process seems to share an extra process containing actual subscription trie (which may be huge in production) between local processes of broker. This should improve startup time and memory efficiency if so!

you are much better in using redis for everything if you are on multiple machines.
what about extending aedes-persistence-redis to support redis cluster for sharded subscriptions? this seems another approach... Any plans on your side for this?

@behrad yes you are right.

you are much better in using redis for everything if you are on multiple machines.

what about extending aedes-persistence-redis to support redis cluster for sharded subscriptions? this seems another approach... Any plans on your side for this?

what do you mean sharded subscriptions?

instead of keeping a whole trie in a single redis instance and a clone of that inside each broker node memory, we shard them inside a cluster of redis instances. Each broker node will manage it's own shard of trie...  nodes will use redis pub/sub for inter-node publishes across the whole logical trie... broker nodes should be multi master.

that is implemented on https://github.com/mcollina/upring-pubsub. No need for Redis.

Interesting module @mcollina
Can you elaborate more on this? never heart of word upring.
Is this playing the role of a backend only? what about persistence?

the moment I get time to do it, we will have it also for persistence.
It's based on a consistent hashring and the dynamo paper http://s3.amazonaws.com/AllThingsDistributed/sosp/amazon-dynamo-sosp2007.pdf.

I deem it wise to make a separate branch for this, as clustering the aedes could have some complications like race conditions, until the very moment it is deemed stable enough to be pushed to master. This way we can implement some tests for the cluster module as well.
But this is of course from a sideline perspective, I don't know how far @mcollina already is in the development process.

Aedes does not need a separate branch.
All needs to be handled in the packages that I mentioned before.
Feel free to wrap everything in an easy-to-use fashion, and do the testing there.

I've used aedes-persistence-redis, mqemitter-redis and HAProxy, works fine.


dr exp requirements oct 2015

I've trimmed back the experimental metadata requirements substantially

I am happy with these changes.
One thing I wonder if it is worth explciting stating that pointing to PDFs (hosted by FAANG) of protocols is allowed?
That does mean we need a policy for accepting protocols for hosting

Agreed.
We should raise this with the samples and assays group. I guess protocols can be stored in the public confluence, or on the public FAANG ftp site.
On 20 Oct 2015, at 10:18, Laura Clarke wrote:

I am happy with these changes.
One thing I wonder if it is worth explciting stating that pointing to PDFs (hosted by FAANG) of protocols is allowed?
That does mean we need a policy for accepting protocols for hosting
�
Reply to this email directly or view it on GitHub.

Hi
I am forwarding to Huaijun, he is leading the next conf call for ASA and
could raise the issue
all the best
Elisabetta
Le 23/10/2015 17:41, David Richardson a écrit :

Agreed.
We should raise this with the samples and assays group. I guess
protocols can be stored in the public confluence, or on the public
FAANG ftp site.
On 20 Oct 2015, at 10:18, Laura Clarke wrote:

I am happy with these changes.
One thing I wonder if it is worth explciting stating that pointing
to PDFs (hosted by FAANG) of protocols is allowed?
That does mean we need a policy for accepting protocols for hosting
�
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/FAANG/faang-metadata/pull/12#issuecomment-150611574.

--
Elisabetta Giuffra
INRA - laboratory of Animal Genetics and Integrative Biology
Team Genetics, Immunity and Health
Bâtiment 320, Domaine de Vilvert
78350 Jouy-en-Josas France
<EMAIL_ADDRESS>Tel. +33.1.34652648


Update from feinstaub

Dockerfile changes are not part of this PR.

@kilemensi this is to allow tests to run


Splitting discovery and controller containers

Passing ROLE argument to select behaviour for manager executable.

Tested in a current 1.1 downstream environment with the other relevant custom images that support this change and confirmed with a successful deployment. Got the logs fine from the UI, I think we're ready with this.


Feature request: allow relative URLs for SVG images

Motivation: absolute URLs prevent both local offline development and fully offline presentations, i.e. presentations in the absence of an Internet connection.
I found a workaround for local offline development: use HTML img elements, where relative URLs work. The relative URLs are then replaced by absolute ones (using a RawGit pointing to my repository on GitHub) before committing PITCHME.md, using a sed one-liner.

Hi @khinsen thanks for your feedback and this feature request.
In order to render an SVG image using a HTML image tag the file content must be served with a valid Content-Type header. In the case of SVG that header values is image/svg+xml.
However, for security reasons, the leading Git hosting services purposely avoid setting this header when serving raw SVG file content.
As a result of this, relative URLs to Git repository files can not be used for SVG content, as the corresponding absolute URLs generated by GitPitch to the files hosted on the Git service are guaranteed-not-to-have-the-required-content-type-header.
The reason your RawGit workaround works is simply because RawGit does set a valid Content-Type header for SVG files. But RawGit is limited to only files hosted in GitHub repos. And GitPitch aims to support a wide selection of Git services, currently including GitHub, GitLab, Bitbucket, GitBucket, Gitea and Gogs. So directly using a third party service like RawGit is not an option for GitPitch.
I know this is not the answer you were hoping for, but I do not currently see any alternative to your sed one-liner or some equivalent pre-preprocessing when working with SVGs as you've described.

Thanks @gitpitch for this clear explanation of the problem!
A long-term solution (perhaps I'd better say a long-term workaround) would be for GitPitch to provide itself the service of RawGit, but for all supported Git hosting services.
The real solution would be for W3C to define a safe subset of SVG that would probably be sufficient for 99% of real-life use cases.

Thanks again @khinsen for your feedback.
Your suggested long-term workaround makes sense. It is something I've considered in the past but decided, for the present at least, that it falls out-of-scope for planned extensions to GP.
That said, it sill may happen sooner than any change we could hope to see from the W3C :)

@khinsen just a quick update on this thread. The newly released GitPitch Pro (now in beta) supports SVGs served directly from public and private Git repos. Works across all Git providers: GitHub, GitLab, and Bitbucket.

@gitpitch Thanks for the info - that sounds good!


RC of 20 (decimal) in basiccreate after STOW_IFF

RC 20 : https://www.ibm.com/docs/en/zos/3.1.0?topic=bpam-stow-completion-codes
The specified data control block is not open or is opened for input, or a DEB error occurred.
Note that if the line:
#define STOW_IFF_ON 1
is commented out, the STOW works. So this appears to be a problem specific to issuing STOW_IFF and the OPEN/WRITE code works with STOW (I can see a member with a's in it for one block)

fixed. turned out i needed a LR R15,R0 before the SVC 21.


How to show ONLINE/MAX ONLINE

Hiya mate,
Is there variables on the boss bar? If so, can you provide a list? Thanks dan

https://github.com/thebigsmileXD/BossAnnouncement/blob/master/src/xenialdan/BossAnnouncement/Main.php#L88-L96


[ARI] Add svkrieger as capi approver

About Me
I'm a developer at SAP working together with @philippthun, @FloThinksPi, @jochenehret, @stephanme and @johha on operating and improving Capi and related projects.
Contributions
In the past ~18 months I was involved in a number for contributions, discussions etc. which qualifies me to be an approver for Capi.
Issues (newest first)
Prevent service broker deadlock: https://github.com/cloudfoundry/cloud_controller_ng/issues/2937
Service instance state handling in V3: https://github.com/cloudfoundry/cloud_controller_ng/issues/2713
Outstanding requests metric bug: https://github.com/cloudfoundry/cloud_controller_ng/issues/2393
Service binding parameters bug: https://github.com/cloudfoundry/cloud_controller_ng/issues/2161
Hanging cf api v3 delete requests: https://github.com/cloudfoundry/cloud_controller_ng/issues/1950
cloud_controller_ng PRs (newest first)
Prevent service broker deadlock: https://github.com/cloudfoundry/cloud_controller_ng/pull/2939
[MINOR] Add missing parameter in docs: https://github.com/cloudfoundry/cloud_controller_ng/pull/2922
Fix flakey tests:
https://github.com/cloudfoundry/cloud_controller_ng/pull/2902
https://github.com/cloudfoundry/cloud_controller_ng/pull/2747
V3 service instance state handling PRs:
https://github.com/cloudfoundry/cloud_controller_ng/pull/2756
https://github.com/cloudfoundry/cloud_controller_ng/pull/2750
https://github.com/cloudfoundry/cloud_controller_ng/pull/2739
https://github.com/cloudfoundry/cloud_controller_ng/pull/2737
https://github.com/cloudfoundry/cloud_controller_ng/pull/2730
Implement exponential backoff for service fetch operations:
https://github.com/cloudfoundry/cloud_controller_ng/pull/2680
https://github.com/cloudfoundry/cloud_controller_ng/pull/2667
Fix outstanding metric bug: https://github.com/cloudfoundry/cloud_controller_ng/pull/2394
Implement space application supporter: https://github.com/cloudfoundry/cloud_controller_ng/pull/2349
Fix service binding parameters bug: https://github.com/cloudfoundry/cloud_controller_ng/pull/2163
Favor 401 over 429 when token is expired: https://github.com/cloudfoundry/cloud_controller_ng/pull/2079
Fix hanging cf api v3 delete requests: https://github.com/cloudfoundry/cloud_controller_ng/pull/1973
slack conversations
https://cloudfoundry.slack.com/archives/C02FL4A2K/p1658391104997169
https://cloudfoundry.slack.com/archives/C07C04W4Q/p1616585842025800
https://cloudfoundry.slack.com/archives/C07C04W4Q/p1612430252045700

Per https://github.com/cloudfoundry/community/blob/main/toc/ROLES.md#promotion-and-revocation, you need 2 existing approvers to review your application and confirm that you meet the criteria for the role. I see @philippthun already approved the promotion, so you will need one more.
Once that criteria is met, I'll merge this PR.

I would find it valuable if a new committer would not only be approved/promoted by colleagues from the same company (in this case SAP), but also from someone else. So @sethboyles, @moleske, @MerricdeLauney, what do you think about Sven's contributions?

Agreed, Sven would be a welcome addition to CAPI approvers

Welcome!


feat: full support for forward refs

Support | notation for older python versions when using from __future__ import annotations
Fix Annotated not being correctly parsed when using from __future__ import annotations

Hi, thanks for contributing to Strawberry 🍓!
We noticed that this PR is missing a RELEASE.md file. We use that to automatically do releases here on GitHub and, most importantly, to PyPI!
So as soon as this PR is merged, a release will be made 🚀.
Here's an example of RELEASE.md:
Release type: patch

Description of the changes, ideally with some examples, if adding a new feature.

Release type can be one of patch, minor or major. We use semver, so make sure to pick the appropriate type. If in doubt feel free to ask :)
Here's the tweet text:
🆕 Release (next) is out! Thanks to Thiago Bellini Ribeiro for the PR 👏

Get it here 👉 https://github.com/strawberry-graphql/strawberry/releases/tag/(next)

@patrick91 ping :)


Reallocation of data folder

Reallocating the data folders from the CounterfactualExplanations.jl package to the newly created TAIJA data package.

[x] Create TaijaData sub-package
[x] Move the data folder to the newly created package (both source-code and tests)
[x] Remove the data folder from this package
[x] Update the src/deprecated.jl file
[ ] Update the documentation

I'll update the documentation once the package is registered.


using templates button in github does nothing when clicked

when using the using templates button it does nothing in the github meaning there no way to see how to use them

Templates are not implemented yet

understood then

trying to use the exported .cmpproject there anywere that show how

will help to know how to use them or the animations themself as there nothing on the wiki here

I'm working on the wiki. You have to export the model to use it in world: https://github.com/tom5454/CustomPlayerModels/wiki/Exporting


New version: BAT v0.6.0

Registering package: BAT
Repository: https://github.com/bat/BAT.jl
Created by: @oschulz
Version: v0.6.0
Commit: 8c8c8abbcb30c46bc89b70cf3cca77537dc9c682
Reviewed by: @oschulz
Reference: https://github.com/bat/BAT.jl/commit/8c8c8abbcb30c46bc89b70cf3cca77537dc9c682#commitcomment-35680779

Your new version pull request met all of the guidelines for automatic merging.
I will automatically merge this pull request during the next cron job.
If you want to prevent this pull request from being auto-merged, simply leave a comment.
(If you want to post a comment without blocking auto-merging, you must include the text [noblock] in your comment.)

On a separate note, I see that you are registering a release with a version number of the form v0.X.Y.
Does your package have a stable public API? If so, then it's time for you to register version v1.0.0 of your package. (This is not a requirement. It's just a recommendation.)
If your package does not yet have a stable public API, then of course you are not yet ready to release version v1.0.0.

[noblock]


The Breadcrumbs needs to be the label and not the id

@baimingze when a user search by some of the facet the Breadcrumbs use the id (2519 or the one from the facet) .It should be replace in the interface by the label. For example if you filter by Human you should use the human label and not the 9016 id.

done


Nullable and modernization part 2

Enable nullable annotations for the repo, then update a few files with nullable, convert .NET 1 data structures to strongly typed collections, and perform other modernizations. Plus #region removal per a PR comment elsewhere.

@FreeAndNil , @fluffynuts - I started a widespread nullable and modernization wave in the code after #119 / #122 , part of which is presented here. If you're willing to accept a series of these, I'm aiming to churn through all the files in the repo to do the same cleanup and optimization. Removing .NET 1.0 ArrayLists and Hashtables in favor of regular and concurrent generics will cut some CPU overhead out of all related code paths.

@erikmav Thank you very much (especially for adding the tests).
After adding the one missing test (RemoveAppender(IAppender) we are ready to merge.

@erikmav @fluffynuts I've created issue #124 for those improvements.

@erikmav did you see my comment regarding the missing test?

Misunderstood the comment. Working on it.

Thanks a lot.


[Event Request] Table 39 - OnQtyToReceiveOnAfterInitQty

In table 39 Purchase Line, trigger Qty. to Receive - OnValidate, please add the following event:
[…]
  "Qty. to Receive (Base)" := CalcBaseQty("Qty. to Receive");
  InitQtyToInvoice;
END;

// >> NEW
OnQtyToReceiveOnAfterInitQty(VAR Rec,CurrFieldNo);
// << NEW

IF ((("Qty. to Receive" < 0) XOR (Quantity < 0)) AND (Quantity <> 0) AND ("Qty. to Receive" <> 0)) OR
[…]

The already existing event OnValidateQtyToReceiveOnAfterCheck is unfortunately too early.

Note: added as OnValidateQtyToReceiveOnAfterInitQty


Update Builder.php

Get the model instance being queried.

Codecov Report

Merging #2031 into master will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #2031      +/-   ##
============================================
+ Coverage     87.15%   87.16%   +0.01%
- Complexity      672      673       +1
============================================
  Files            31       31
  Lines          1541     1543       +2
============================================
+ Hits           1343     1345       +2
  Misses          198      198

Impacted Files
Coverage Δ
Complexity Δ

src/Jenssegers/Mongodb/Eloquent/Builder.php
88.13% <100.00%> (+0.41%)
23.00 <1.00> (+1.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 736415e...bf8c578. Read the comment docs.

Closing due to inactivity and since repo was removed no sense to keep this.
Thanks!


Change target frameworks to net461 and netstandard2.0

Given a netstandard2.0 TFM we should be able to run scripts using dotnet cake.
~I'll have a crack and submit a PR...~ this is trickier than I thought due to its dependencies.

Cannot be done as netstandard2.0 - needed to be targeted  to concrete versions o framework - done in #83

The last a version that targets netstandard2 is Powershell 6.1.2 but that version doesn't come with the OS, like framework does.
If the draw to framework is that it comes with a base OS install then you would want a netstandard version that targets 'PSVersion 5.1.19041.4170' as that is the current version shipped with Win10.
I would recommend having separate versions for PowerShell5\netstandard and latest\latest. If you're requiring a PowerShell7 be installed then you can also require net8.


Invalid Dotnet version detected and missing versions for the dependencies

Hi,
Recently while running cdxgen on a dotnet framework 4.8 version application, we found that the cdxgen is reporting wrong version of dotnet and all the dependencies are missing version data

I have attached a sample application which can be used for replication purpose
ConsoleApp2.zip
Command used: cdxgen -r -p --deep -o sbom.json
CDXGEN Version : 10.7.1

This is going to be a tough one. For system dlls, the referenced path is in C drive under the install directory as evident from the screenshot. Hence, there is no entry on deps.slices.json from dosai.

To provide an update on the progress on resolving this:

We updated the Dosai plugin to provide assembly version numbers if all we have is the dll file (pull request)
We updated the cdxgen plugin project to pull in the latest Dosai plugin and make it available to cdxgen (pull request)

Next step we are working on now is to update cdxgen to match the assembly version provided by Dosai to the dependencies like these from the Global Assembly Cache to now show the version number in the output. Stay tuned!

@visagansanthanam-unisys This is ready for re-testing from the master branch. A props file must be created and placed in the root directory of the application as explained here:
https://github.com/CycloneDX/cdxgen?tab=readme-ov-file#including-net-global-assembly-cache-dependencies-in-the-results
Let me know how it looks.

Sorry for the delay and thanks @prabhu. The documentation looks good. Is the code already released as part of the latest version?

Tes. Could you run cdxgen with -t dotnet --deep and share the observation here

Hi @prabhu
We have updated to CDXGEN 10.8.2 still same issue observed for DotNet components it's not able to identify version number

Attach SBOM.Json output file for your refence.
sbom.json

Could you try with 10.8.4? Also ensure the file Directory.Build.props is present in the root directory as mentioned in the readme.
cc: @timmyteo

Hi @prabhu
As suggested try 10.8.4 still same issue observed, we have already included Directory.Build.props file in root directory.
Try different command but no luck
cdxgen -t dotnet
cdxgen -r -p -t dotnet -o sbom.json
cdxgen -r -p -t dotnet --deep -o sbom.json
While raising issue we have attached sample application [ConsoleApp2.zip] which you can use to replicate issue.

Hi @OmShrivastava-Unisys please note that since these dependencies are located in the Global Assembly Cache (GAC), it is necessary to build the project first before inspecting with cdxgen. This will ensure the GAC assemblies are copied into the output folder and be made available for cdxgen to inspect. After building, you should get this result.

Hi @timmyteo
Thanks for your support, I am able to view version <IP_ADDRESS> as you shared in screenshot.
I have one doubt about version number, I am using .Net framework 4.8 in my application,
When I check property for each dll in build folder or in C:\Program Files (x86)\Reference Assemblies\Microsoft\Framework.NETFramework\v4.8
it showing File Version: 4.8.3761.0 but in CDXGEN report it showing <IP_ADDRESS>.
Can you clarify this can it possible to get File Version: 4.8.3761.0 in CDXGEN output, I think it more appropriate in this context.

https://github.com/CycloneDX/cdxgen-plugins-bin/releases/tag/v1.6.3 got released yesterday with a fix. Need to update cdxgen to this version and test.

HI @prabhu and @timmyteo
Thanks for your extended support, now I am able to get .Net framework GAC dll version.
Step follow to generate BOM

Updated CDXGEN to latest available version (10.8.7).
Added Directory.Build.props file as suggested for GAC dll

True

Build solution and verify required DLL copied in build folder.
Run CDXGEN command to generate SBOM
cdxgen -r -p --deep -o sbom.json
Below is working output:


review pop-up has title and removed detail link (bug 1203116)

Was:

Now is:

Title seems to appear in the comment when filled in the review form box.

There's one test failure related to your PR (the first three are fixed on master, please rebase and confirm that you don't have the changes to the apps/editors/helpers.py file anymore).
Regarding the design, the title field looks
1/ a bit too short, could it extend to the rest of the line (maybe using flexbox? Remember that the label is localized, and as such can vary in size), or more simply be displayed below the label?
2/ to close to the label, i think it could benefit from some left spacing
3/ not style the same way as the review textarea, the font isn't the same (size)?

@magopian Good point on the localization. What do you think of this? Since the input is on a new line it should allow plenty of space for the text above and conforms to the same styling as the review box below.

Looks better ;) Can you do something about the fontsize?

Sure, I assume make it bigger and more title-y?

I meant "more like the font of the textarea, for coherence" ;)

Ah my mistake. Okay last one, promise.

That looks perfect, thanks ;)
r+


chcpbuild.options override whole config.xml chcp tag

cordova build -- chcp-xxx remove other tags in  other than
in my config.xml, i have auto-install set to false
but when i use cordova build -- chcp-trial
it set the config-file to the url in chcpbuild.options and remove my auto-install tag

Will look into this, thanks.

Fixed in v1.2.5


Comment form should not be cleared on upvote

Expected behavior
The content in the comment form should stay untouched after upvoting the post.
Actual behavior
Comment form is cleared after upvoting the post.
How to reproduce

Start writing a comment for a post. Don't submit it yet.
Upvote the post.
The comment form is cleared.

Browser: Chrome 61.0.3163.100
Operating system: Windows 10

I started looking at this issue. Thanks


github actions badge

Please make sure these boxes are checked, before submitting a new PR.

[x] You ran rustfmt on the code base before submitting (on latest nightly with rustfmt support)
[x] You reference which issue is being closed in the PR text (if applicable)

Removes old CI badge and adds the new one.

Just wait for my PR to get merged. It'll fix the formatting issue.

Just wait for my PR to get merged. It'll fix the formatting issue.


Include instructions on settings up Sites framework

Fixes #258

Coverage remained the same at 100.0% when pulling 37a4e254ef1846c4020595183e73af34af1481a5 on cfc603:site_runtime_error into 09234b595610a0ccfc919386070c3ed187436895 on pinax:master.

Coverage remained the same at 100.0% when pulling a30fd852537447dfba02737ca42d5b492643d6e3 on cfc603:site_runtime_error into 09234b595610a0ccfc919386070c3ed187436895 on pinax:master.

Thanks @cfc603! @paltman Can this be merged?


Allow S3 to be mounted in Studio Lab

Is your feature request related to a problem? Please describe.
I would like to be able to mount external storage such as S3 to Studio lab, as the current 15GB storage is sometimes not enough.
Describe the solution you'd like
Allow s3fs to be installed in the studio lab.
Describe alternatives you've considered

Deleting and redownloading the data needed.

I am sorry if it is rude or inappropriate to you, since I use the online translation site.

Also, it seems that studio lab is running on docker, so you need to enable fuse to mount it

Dear @fkunn1326 , thank you for your great idea. We will discuss at the following issue.
https://github.com/aws/studio-lab-examples/issues/178


Fixes -- block-scoped declarations not available outside strict mode

node server

vue-hackernews-2.0/server.js:14
let indexHTML // generated by html-webpack-plugin
^^^
SyntaxError: Block-scoped declarations (let, const, function, class) not yet supported outside strict mode
at exports.runInThisContext (vm.js:53:16)
at Module._compile (module.js:387:25)
at Object.Module._extensions..js (module.js:422:10)
at Module.load (module.js:357:32)
at Function.Module._load (module.js:314:12)
at Function.Module.runMain (module.js:447:10)
at startup (node.js:146:18)
at node.js:404:3

Thanks, this app requires Node 6+ which doesn't have this problem.


[forc-doc] The Sway logo should be linked to the project root

Currently, if I click the logo from https://fuellabs.github.io/sway/master/std/address/struct.Address.html , It takes me to https://fuellabs.github.io/sway/master/std/address/index.html
I would expect it to take me to https://fuellabs.github.io/sway/master/std/index.html#

Ref #3996 currently the logo will go back a module until you reach the parent module. This is a known issue, will have a PR up soon!


Add more folding rules for vector shuffle.

Adds rule to fold OpVectorShuffle with constant inputs.
Adds rules to fold OpCompositeExtrac being fed by an OpVectorShuffle.

Please resolve the conflict.  I tried a somewhat trivial resolution and got dozens of errors.

I found an error in it that I need to fix anyway.

Made enough changes.  Should have a new review.


Use a single commit to generate all batch ops, before initiating the HTTP calls

Use a single database transaction to write all the pending operations needed to dispatch a private messaging batch
Then initiate all the HTTP calls to DX separately - noting these should be pending until the asynchronous callback comes

Codecov Report

Merging #647 (a9a937d) into main (a70dfbd) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Current head a9a937d differs from pull request most recent head 4e0e4a7. Consider uploading reports for the commit 4e0e4a7 to get more accurate results

@@            Coverage Diff            @@
##              main      #647   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          313       313
  Lines        18975     19012   +37
=========================================
+ Hits         18975     19012   +37

Impacted Files
Coverage Δ

internal/operations/manager.go
100.00% <100.00%> (ø)

internal/privatemessaging/privatemessaging.go
100.00% <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 a70dfbd...4e0e4a7. Read the comment docs.


Error parsing MD Text

Describe the Bug
Hello,
if I enter some special text in the MD Editor, a white screen appears. No log information is available.
I also tried it on your demo-bookstack Instance an can reproduce the error
Steps to Reproduce

Edit a page
Enter this:

Steps:

Download agent

wget https://abc

as an example

save page
white screen appears.

Expected Behaviour
saving with no errors will be done
Screenshots or Additional Context
No response
Browser Details
Edge, Firefox
Exact BookStack Version
BookStack v23.12.1

In addition: I've seen that the problematic MD text will be rendered in GITHUB issue, I've added the MD as an attachment
md.txt

Hi @KarstenHahn,
Thanks for raising.
Strange though, I cannot reproduce this on my dev instance, nor our demo instance; All works fine.
Tested via both creating and updating a page, in both Firefox and Chrome.

Hello,
mysterious. I have tested it again, it works on my instance and on the demo instance.
Maybe a problem with the character set when copying from the clipboard?
Never mind. Now it works.
Thanks for the quick response and best regards, Karsten


OpenAIapi.ChatNoFunctions failed: scrubbedSensitiveData

Hi, I uploaded without errors the app and when testing the Chat it gives me the following error in each "text box" section: OpenAIapi.ChatNoFunctions failed: scrubbedSensitiveData ...
Open AI has correctly connected and also Azure Open AI.
Can you help me in understanding how I can fix it?
Thank you
Bianca

The image generation prompt (auto generated) can trigger sensitive data filters in OpenAI and expecially in Azure OpenAI. In subsequent versions I have extracted only the DallePrompt text and tune the DallePrompt generation to focus on the face/portrait. This reduces the probability of triggering AI safety filters.
For Azure OpenAI, you can request to have greater control on the AI Safety as you are working on clinical scenarios so that you can increase the tolerance.


ARM (not ARM64) image support

Is your feature request related to a problem? Please describe.
I have a Raspberry Pi-based (i.e. arm) cluster I would like to be able to use descheduler on.
Describe the solution you'd like
I would like arm supported to be added to the existing multi-arch build. Since building an arm image for descheduler is trivial, this should be easy to add.
Describe alternatives you've considered
I could just continue building my own multi-arch version of descheduler, but it seems wasteful when I'm almost certainly not the only one wanting to use it on arm.
What version of descheduler are you using?
descheduler version: 0.20.0

@cerebrate so you would like an official descheduler container image for ARMv7?
GOOS=linux GOARCH=arm GOARM=7 go build ...

As you mentioned this should be fairly easy to add, so please confirm that you need ARMv7. We can try to get this setup for the descheduler v0.21.0 release. Thanks!

Yep, that's exactly the one I'd like. Thanks!

@damemi and @ingvagabund any concerns with adding one more container image? We would end up with three(AMD64, ARM64, and ARMv7) if we implement this.
/assign

@seanmalloy I don't have any problem with that, feel free to open the PRs :)

@seanmalloy I don't have any problem with that, feel free to open the PRs :)

No problem here either :)

No problem here either :)

/assign

Works great for me in testing, thanks!


Remove duplicate ID

What?
Remove duplicate - in the code ("announcements").
Why?
Ticket
Fail of WCAG SC 4.1.1
Anything else?
No visual change:

:warning: This application is Continuously Deployed: :warning:

Merged changes are automatically deployed to staging and production.

Make sure you follow the guidance for deployments before you merge.

Check your branch is being deployed in the Release app, after merging.

Hey @injms - I've not got auth to merge, do the honours?


Handling of Enums

Currently encoding and decoding of ordinary enums like System.DayOfTheWeek require own extra coders, e.g.
module Encode =
   let dayofWeek (value: System.DayOfWeek) : JsonValue =
       LanguagePrimitives.EnumToValue (value)
       |>JValue :> JsonValue

module Decode =
    module Helpers =
        let inline isInteger (token: JsonValue) = not(isNull token) && (token.Type = JTokenType.Integer)
        let inline isString (token: JsonValue) = not(isNull token) && token.Type = JTokenType.String
        let inline asString (token: JsonValue): string = token.Value<string>()

    let dayofWeek : Decoder<System.DayOfWeek> =
        fun path token ->
            if Helpers.isInteger token then
                // TODO: Is not enough to convert to int directly? Maybe these checks hurt performance?
                let value = token.Value<decimal> ()
                if value >= (decimal System.Int32.MinValue) && value <= (decimal System.Int32.MaxValue) then
                    Ok (LanguagePrimitives.EnumOfValue ((int32) value))
                else
                    (path, BadPrimitiveExtra("an int", token, "Value was either too large or too small for an int")) |> Error
            elif Helpers.isString token then
                match System.Int32.TryParse (Helpers.asString token, NumberStyles.Any, CultureInfo.InvariantCulture) with
                | true, x -> Ok <| LanguagePrimitives.EnumOfValue(x)
                | _ -> (path, BadPrimitive("an int", token)) |> Error
            else
                (path, BadPrimitive("an int", token)) |> Error

This works pretty good, however I would love to have generic Enum support, either direclty in Thoth.Json or as extra coder.
This naive approach fail:
module Encode =

   let enumInt (value: 'T when 'T:enum<int>) : JsonValue =
       LanguagePrimitives.EnumToValue (value)
       |>JValue :> JsonValue

module Decode =
    module Helpers =
        let inline isInteger (token: JsonValue) = not(isNull token) && (token.Type = JTokenType.Integer)
        let inline isString (token: JsonValue) = not(isNull token) && token.Type = JTokenType.String
        let inline asString (token: JsonValue): string = token.Value<string>()

    let enumInt : Decoder<'T> when 'T:enum<int> =
        fun path token ->
            if Helpers.isInteger token then
                // TODO: Is not enough to convert to int directly? Maybe these checks hurt performance?
                let value = token.Value<decimal> ()
                if value >= (decimal System.Int32.MinValue) && value <= (decimal System.Int32.MaxValue) then
                    Ok (LanguagePrimitives.EnumOfValue ((int32) value))
                else
                    (path, BadPrimitiveExtra("an int", token, "Value was either too large or too small for an int")) |> Error
            elif Helpers.isString token then
                match System.Int32.TryParse (Helpers.asString token, NumberStyles.Any, CultureInfo.InvariantCulture) with
                | true, x -> Ok <| LanguagePrimitives.EnumOfValue(x)
                | _ -> (path, BadPrimitive("an int", token)) |> Error
            else
                (path, BadPrimitive("an int", token)) |> Error

While the encoders and decoders work, they cannot be added withCustom, and I also failt adding them directly in Thoth.Json.Net.
It worked before in 2.x, maybe this just a special case of #9, where all unknown types case were handled by NewtonSoft.Json.

where all unknown types case were handled by NewtonSoft.Json.

Yes, I think before NewtonSoft.Json was doing the job for us on the server-side.
The problem with supporting enum is not the server-side but Fable side because it doesn't have yet all the reflection API info needed (I still need to check if that's still the case).
Related issue: https://github.com/MangelMaxime/Thoth/issues/161
I will try to take a look at the situation this week because you are several asking for it. And will also, make writing low-level decoder easier by making public the helpers module. Because I have it copied myself into several projects so it's worth than exposing it by default as it can go out of sync.

Thank you.
I would appreciate if the server side is capable to support the whole .Net types including Enums. For the Fable side, I would then fake my own matching types.
If you need any help, for example to write some tests, just let me know.

I would appreciate if the server side is capable to support the whole .Net types including Enums. For the Fable side, I would then fake my own matching types.

I understand that wish, but I can't make Thoth.Json and Thoth.Json.Net support different things because they are supposed to be replacement of each others depending on the runtime.
The reason, being otherwise you will have code working differently depending on the runtime and it goes against the security provided by Thoth.Json.
But, when something is missing my goal is to add it to both so in theory at some point we will support everything in both runtimes :)

I understand that wish, but I can't make Thoth.Json and Thoth.Json.Net support different things because they are supposed to be replacement of each others depending on the runtime.

Runtime is not only SAFE stack, e.g. I am supporting a .Net Framework based CMS, written in C#.
I wanna just pass some POCOs to fable, and then I don't want/need to share code between runtime and fable.
IMHO it is totally fine to have a C# Poco in runtime
class PersonPoco{
  public string FirstName {get;set;}
  public string LastName {get;set;}
}

which maps then in Fable with
type Person =
     { FirstName:string;
       LastName:string }

This worked before. It was even possible to mix C# Pocos with F# types, e.g. type Person = Poco of PersonPoco in Runtime.
Now it is required to map everything. I accepted that, too. But sometimes I wish the former comfort back.
(And this should be better discussed in #9)

Ooops, I completely forget about #9 ...
It's sometimes hard to remember that not everyone has the same needs/workflow. :) I will see to fix enum support first and look at the C# / F# mix and see if I can find a solution to make it work out the box or with minimal code on your side.

I always thought to serialize the value, however it is much easier with .ToString() and Enum.Parse

Too early to be happy:
printfn "%s" (System.DayOfWeek.Tuesday.ToString())
// "2" in Fable, "Tuesday" in Runtime

Enums are now treated as ints :-)


ML 8.2 : LSTM(V)

Description
📌  Issues for Week 8
Welcome to 'ML' Team, good to see you here
:arrow_forward: This issue will helps readers in acquiring all the knowledge that one needs to know about ----
LSTM.
:red_circle: To get assigned to this issue, add your Batch Numbers mentioned in the spreadsheet of "Machine Learning", the approach one would follow and choice you prefer (Documentation, Audio, Video). You can go with all three or any number of options you're interested to work on.
✔ Domain : Machine Learning
:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.

"Issue Title" and "PR Title should be the same. Include issue number along with it.

Changes should be made inside the Machine Learning/Machine_Learning/ML_for_Seqential_Data_&_Visual_Data Branch.

Follow Contributing Guidelines & Code of Conduct before start Contributing.

This issue is only for 'GWOC' contributors of 'Machine Learning' domain.

:white_check_mark: To be Mentioned while taking the issue :

Full name
Batch Number
GitHub Profile Link
Which type of Contribution you want to make :

[ ] VIdeo

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎
Domain
Machine Learning
Type of Contribution
Video
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

Please assign this issue to me.
Name: Vidhi Bhatt
Domain: ML Batch 3
Serial no: 490
Github link: https://github.com/VidhiBhatt01
Documentation/Video

Hello, I am Deepthi M with serial number:198, Batch-2. I would like to do video on this issue.my approach: a detailed explanation

@deepthi1107 Make within in 3 days 🚀 .


Add USE_FULL_SCREEN_INTENT for android

Feature request
Add USE_FULL_SCREEN_INTENT
Why it is needed
To create full screen notification (usually over lock screen for voip apps)

The documentation say that its Protection level is normal.
You don't need to request it at runtime.

See https://developer.android.com/about/versions/10/behavior-changes-10

This is a normal permission, so the system automatically grants it to the requesting app.

@zoontek thanks, you right

++++

check android 14 change log, + 1


fixed link to cli Readme

Attempt to fix the broken link to CLI REDME

I don't really understand why it's still shown as pending. I can see it's signed

@blauret, to update the status on this PR, you need to follow the instructions posted by @googlebot:

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


Support Bottom Page Control Indicator

Add support to show content page indicator at the bottom of the screen for each page content.
Specs:

[x] Hide if only contains 1 item of resource.
[x] Static text in container view (non-scrollable)
[x] iPhone X support

Added. #11


Checkbox data-attrs not applied if used as Collapsible trigger

Bug report
Current Behavior
First, thanks for building an library that's a joy to use. Really appreciate all the effort.
When a <Checkbox> primitive is used inside a <Collapsible.Trigger>, it loses the behavior that applies data-state="checked"
Expected behavior
When a <Checkbox> primitive is used inside a <Collapsible.Trigger>, it should still apply data-attrs state changes like data-state="checked" to allow styling changes according to state.
Reproducible example
Example showing behavior
Suggested solution
It's very possible that I'm missing something in my implementation / understanding so let me know if I just need to fix that.
Your environment

Software
Name(s)
Version

Radix Package(s)
"@radix-ui/react-collapsible"
"1.0.1",

Radix Package(s)
"@radix-ui/react-checkbox"
"1.0.1",

React
n/a
17.0.1

Browser
tested on Firefox  109 and Chrome 108

Assistive tech

Node
n/a
16.10

I just realised I can use data-state='open'. I wonder if this still issue applies, since I would imagine wanting to use a re-usable  with styles that apply on data-state='checked'. Happy to hear otherwise though

The child props had to overwrite the parent props but for some reason this is not happening with the data-state. If you reverse the order of Collapsible.Trigger with Checkbox.Root the data-state of the checkbox will be maintained. Seems to be a bug or I missing something :thinking:

This is due a limitation when composing 2 primitives which both use data-state as one will invariably override the other.
This is why @joaom00 is seeing a different result when switching the order between collapsible and checkbox.
A potential solution would be to control the components state.
I will close as a duplicate of #602


The Git Tag of 5.4.1 uses the wrong 5.4.1-SNAPSHOT as the shardingsphere version, causing the build to fail

The Git Tag of 5.4.1 uses the wrong 5.4.1-SNAPSHOT as the shardingsphere version, causing the build to fail. Refer to https://github.com/apache/shardingsphere-plugin/blob/5.4.1/pom.xml#L28 .

A quick CI report from https://jitpack.io/#apache/shardingsphere-plugin/5.4.1 . Refer to https://jitpack.io/com/github/apache/shardingsphere-plugin/5.4.1/build.log .

Should we consider making https://github.com/apache/shardingsphere-plugin independent of the release process for https://github.com/apache/shardingsphere? Or create <IP_ADDRESS> for https://github.com/apache/shardingsphere-plugin to carry the correct shardingsphere version? Refer to https://github.com/apache/shardingsphere/blob/5.4.1/pom.xml#L28 .

[INFO] Reactor Summary for Apache ShardingSphere Plugin 5.4.1-SNAPSHOT:
[INFO]
[INFO] Apache ShardingSphere Plugin ....................... SUCCESS [  7.128 s]
[INFO] shardingsphere-plugin-features ..................... SUCCESS [  0.033 s]
[INFO] shardingsphere-plugin-features-encrypt ............. SUCCESS [  0.019 s]
[INFO] shardingsphere-plugin-features-encrypt-like ........ FAILURE [  1.728 s]
[INFO] shardingsphere-plugin-features-encrypt-rc4 ......... SKIPPED
[INFO] shardingsphere-plugin-features-encrypt-sm .......... SKIPPED
[INFO] shardingsphere-plugin-features-sharding ............ SKIPPED
[INFO] shardingsphere-plugin-features-sharding-cosid ...... SKIPPED
[INFO] shardingsphere-plugin-features-sharding-nanoid ..... SKIPPED
[INFO] shardingsphere-plugin-infra ........................ SKIPPED
[INFO] shardingsphere-plugin-infra-data-source-pool ....... SKIPPED
[INFO] shardingsphere-plugin-infra-data-source-pool-c3p0 .. SKIPPED
[INFO] shardingsphere-plugin-infra-data-source-pool-dbcp .. SKIPPED
[INFO] shardingsphere-plugin-jdbc ......................... SKIPPED
[INFO] shardingsphere-plugin-jdbc-driver .................. SKIPPED
[INFO] shardingsphere-plugin-jdbc-driver-apollo ........... SKIPPED
[INFO] shardingsphere-plugin-mode ......................... SKIPPED
[INFO] shardingsphere-plugin-mode-cluster ................. SKIPPED
[INFO] shardingsphere-plugin-mode-cluster-repository ...... SKIPPED
[INFO] shardingsphere-plugin-mode-cluster-repository-nacos  SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  9.930 s
[INFO] Finished at: 2023-12-15T04:46:31Z
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal on project shardingsphere-plugin-features-encrypt-like: Could not resolve dependencies for project org.apache.shardingsphere:shardingsphere-plugin-features-encrypt-like:jar:5.4.1-SNAPSHOT: The following artifacts could not be resolved: org.apache.shardingsphere:shardingsphere-encrypt-api:jar:5.4.1-SNAPSHOT, org.apache.shardingsphere:shardingsphere-encrypt-core:jar:5.4.1-SNAPSHOT: Could not find artifact org.apache.shardingsphere:shardingsphere-encrypt-api:jar:5.4.1-SNAPSHOT in apache.snapshots (https://repository.apache.org/snapshots) -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException
[ERROR]
[ERROR] After correcting the problems, you can resume the build with the command
[ERROR]   mvn <args> -rf :shardingsphere-plugin-features-encrypt-like

@linghengqian please check the <IP_ADDRESS>
deeply apologize for  that issue.

@linghengqian please check the <IP_ADDRESS> deeply apologize for that issue.

I checked https://jitpack.io/com/github/apache/shardingsphere-plugin/<IP_ADDRESS>/build.log on jitpack.io. The repository is using a series of undeployed modules from the shardingsphere main repository and the build is still failing.

[INFO] Reactor Summary for Apache ShardingSphere Plugin <IP_ADDRESS>:
[INFO]
[INFO] Apache ShardingSphere Plugin ....................... SUCCESS [  6.988 s]
[INFO] shardingsphere-plugin-features ..................... SUCCESS [  0.023 s]
[INFO] shardingsphere-plugin-features-encrypt ............. SUCCESS [  0.018 s]
[INFO] shardingsphere-plugin-features-encrypt-like ........ SUCCESS [ 16.350 s]
[INFO] shardingsphere-plugin-features-encrypt-rc4 ......... SUCCESS [  0.950 s]
[INFO] shardingsphere-plugin-features-encrypt-sm .......... SUCCESS [  1.363 s]
[INFO] shardingsphere-plugin-features-sharding ............ SUCCESS [  0.010 s]
[INFO] shardingsphere-plugin-features-sharding-cosid ...... SUCCESS [  2.734 s]
[INFO] shardingsphere-plugin-features-sharding-nanoid ..... SUCCESS [  0.767 s]
[INFO] shardingsphere-plugin-infra ........................ SUCCESS [  0.008 s]
[INFO] shardingsphere-plugin-infra-data-source-pool ....... SUCCESS [  0.008 s]
[INFO] shardingsphere-plugin-infra-data-source-pool-c3p0 .. FAILURE [  0.449 s]
[INFO] shardingsphere-plugin-infra-data-source-pool-dbcp .. SKIPPED
[INFO] shardingsphere-plugin-jdbc ......................... SKIPPED
[INFO] shardingsphere-plugin-jdbc-driver .................. SKIPPED
[INFO] shardingsphere-plugin-jdbc-driver-apollo ........... SKIPPED
[INFO] shardingsphere-plugin-mode ......................... SKIPPED
[INFO] shardingsphere-plugin-mode-cluster ................. SKIPPED
[INFO] shardingsphere-plugin-mode-cluster-repository ...... SKIPPED
[INFO] shardingsphere-plugin-mode-cluster-repository-nacos  SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  30.264 s
[INFO] Finished at: 2023-12-15T06:25:01Z
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal on project shardingsphere-plugin-infra-data-source-pool-c3p0: Could not resolve dependencies for project org.apache.shardingsphere:shardingsphere-plugin-infra-data-source-pool-c3p0:jar:<IP_ADDRESS>: The following artifacts could not be resolved: org.apache.shardingsphere:shardingsphere-test-fixture-database:jar:5.4.1, org.apache.shardingsphere:shardingsphere-test-util:jar:5.4.1: Could not find artifact org.apache.shardingsphere:shardingsphere-test-fixture-database:jar:5.4.1 in central (https://repo.maven.apache.org/maven2) -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/DependencyResolutionException
[ERROR]
[ERROR] After correcting the problems, you can resume the build with the command
[ERROR]   mvn <args> -rf :shardingsphere-plugin-infra-data-source-pool-c3p0

Snapshot builds located on Jitpack are back to normal. Refer to https://jitpack.io/com/github/apache/shardingsphere-plugin/<IP_ADDRESS>/build.log.

[INFO] Reactor Summary for Apache ShardingSphere Plugin <IP_ADDRESS>:
[INFO]
[INFO] Apache ShardingSphere Plugin ....................... SUCCESS [  4.340 s]
[INFO] shardingsphere-plugin-features ..................... SUCCESS [  0.019 s]
[INFO] shardingsphere-plugin-features-encrypt ............. SUCCESS [  0.016 s]
[INFO] shardingsphere-plugin-features-encrypt-like ........ SUCCESS [  6.233 s]
[INFO] shardingsphere-plugin-features-encrypt-rc4 ......... SUCCESS [  1.086 s]
[INFO] shardingsphere-plugin-features-encrypt-sm .......... SUCCESS [  1.454 s]
[INFO] shardingsphere-plugin-features-sharding ............ SUCCESS [  0.013 s]
[INFO] shardingsphere-plugin-features-sharding-cosid ...... SUCCESS [  1.937 s]
[INFO] shardingsphere-plugin-features-sharding-nanoid ..... SUCCESS [  1.153 s]
[INFO] shardingsphere-plugin-infra ........................ SUCCESS [  0.011 s]
[INFO] shardingsphere-plugin-infra-data-source-pool ....... SUCCESS [  0.010 s]
[INFO] shardingsphere-plugin-infra-data-source-pool-c3p0 .. SUCCESS [  0.488 s]
[INFO] shardingsphere-plugin-infra-data-source-pool-dbcp .. SUCCESS [  0.504 s]
[INFO] shardingsphere-plugin-jdbc ......................... SUCCESS [  0.010 s]
[INFO] shardingsphere-plugin-jdbc-driver .................. SUCCESS [  0.010 s]
[INFO] shardingsphere-plugin-jdbc-driver-apollo ........... SUCCESS [  6.315 s]
[INFO] shardingsphere-plugin-mode ......................... SUCCESS [  0.008 s]
[INFO] shardingsphere-plugin-mode-cluster ................. SUCCESS [  0.007 s]
[INFO] shardingsphere-plugin-mode-cluster-repository ...... SUCCESS [  0.007 s]
[INFO] shardingsphere-plugin-mode-cluster-repository-nacos  SUCCESS [  1.300 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  25.582 s
[INFO] Finished at: 2023-12-15T08:56:35Z
[INFO] ------------------------------------------------------------------------

@taojintianxia Do we need to create a new release at https://github.com/apache/shardingsphere-plugin/releases and<EMAIL_ADDRESS>?

@taojintianxia Do we need to create a new release at https://github.com/apache/shardingsphere-plugin/releases and<EMAIL_ADDRESS>?

sure, I will do what.


Update machine-learning-evaluate-model-performance.md

Just corrected a typo.

Hi @dem108, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

@dem108 : Thanks for your contribution to the Azure documentation! The author, @garyericson, has been notified to review your proposed change.

@dem108 - Good catch, thanks! #sign-off


Add a new range() function to GREL

Fixes #1319
The original specification that @thadguidry suggested for the range function was:

It will work like this...
can take accept a String form:
range("2-4") --> [2,3,4]
or a String form in a value
range(value) --> [2,3,4]
or 2 numbers
range(2, 4) --> [2,3,4]
and doing join(",") will convert the Array to a String
range(2,4).join(","). --> 2,3,4

However, because of issues such as cells being unable to display arrays and the potential confusion it can cause, I have taken the liberty to display the results of range as a string with comma-separated values instead.
Hence, my implementation works like this:
(a) String arguments

range("2-4") --> 2, 3, 4
range("4-2") --> 4, 3, 2
range("2, 4") --> 2, 3, 4
range("4, 2") --> 4, 3, 2
range(value) --> 2, 3, 4 //given that value is written as 2-4 (text) or 2, 4 (text) in the cell

(b) Integer arguments

range(2, 4) --> 2, 3, 4
range(4, 2) --> 4, 3, 2

*range(value) does not apply to integer arguments since 2, 4 is stored as text in a cell.
Additional notes

Given valid arguments, either an increasing sequence or a decreasing sequence is returned depending on whether the rangeStart value is larger than the rangeEnd value.
Integer arguments cannot be negative; otherwise, an EvalError is returned.
Numeric arguments can only be integers, and decimal values will cause an EvalError to be returned.
It is okay to have whitespace(s) within the string (e.g. range("2   -   4  ") will work as well).

Questions to discuss

Should the implementation use integers? What about long or BigInteger? Should we worry about when users try to input very large values (e.g.<PHONE_NUMBER>402385707520357208) to the range function?
Should the result returned be an array as @thadguidry has suggested? In this case, the result will directly be displayed in the cells after the transformation without having to use join, but is this consistent with other functions already provided by OpenRefine?
Should negative integers, or even decimal numbers be supported by this function?

Lastly, I have also added some basic tests for the new function (although more comprehensive testing can be done with real examples). I am willing to help to update the documentation on GitHub pages for this function after it is approved :)
Thanks for reading this lengthy pull request message, and I look forward to any incoming feedback :)

@joanneong good work so far !  My idea was to handle the 80% of the use cases that folks would use the GREL range() function for.  The design should mimic what Jython/Python does and it's range() https://docs.python.org/3/library/stdtypes.html?highlight=range#range

Only Integers supported, I would say, just like Python's range()
Yes, return it as an array.  We have other issues regarding enhancing our displays (via coloring, syntax, etc) that will let folks know that their output is not of a String type that can be written to a cell.  So for range() just handle as we do with other functions that output an Array.
Yes, negative numbers should be supported, just as they are with Python's range()

One more argument that I think would be valuable would be a STEP, just as Python's range() has.
range(start, stop[, step])
Also testing for equality should mimic as Python does and treat the range objects as sequences. See notes on the range() class in Python's docs url above.

I also think that this function should produce an array, not a string. It would be on the one hand more coherent intellectually and on the other more practical in many cases. After all, a range function is most often used as iterable.
The optional step argument is a good idea. In this way, we could make the forRange function obsolete  (who uses it?) and use only forEach to iterate arrays:
forEach (range (number from, number to, number step), variable v, expression e)

Got it, thanks for the prompt responses @thadguidry and @ettorerizza ! I will update this pull request soon :P

Technical detail: must the stop be included in the array? The range function of Python does not include it:
list(range(1,10)) --> [1,2,3,4,5,6,7,8,9]
The current function forRange either:
forRange(1,10,1,e, e) --> [1,2,3,4,5,6,7,8,9]
Personally, I would support the idea of including it, as 90% of humans that would answer the question "give me the numbers from 1 to 10".

@ettorerizza the Step is optional Ettore

@ettorerizza Yes, the Stop needs to be output.

I would be in favour of not including the bound, just to be consistent with forRange, Python and a good deal of other programming languages.
range(10) should return the array of the first 10 integers.
But it is clearly not a hill I want to die on.

It's true that it'll introduce inconsistency, since all OR array functions use a zero-based indexing.
value = [1,2,3]
value.slice(1,2) -> [2] #(and not [1,2] as it would be the case with this range() function)

All this seems very clear to any programmer (except perhaps those who only use R), but I remember that it disturbed me a lot when I started learning OpenRefine...

+1 for returning a array instead of string. It will provide a easier way to loop through it also leave the decision to the end user how to decorate(split, padding etc) the elements

UPDATE SUMMARY

The range function now returns an array and requires users to use .join(",") if they want to see it in the cells.

The range function now has a step parameter (optional).

The range function can accept 1 to 3 string/integer arguments:
(a) 1 argument: 0 becomes the default range start, and the given argument is the range end
e.g. range(5) --> ["0", "1", "2", "3", "4", "5"]
range("5") --> ["0", "1", "2", "3", "4", "5"]
(b) 2 arguments: the first argument is the range start, and the second argument is the range end. If the range start is smaller than the range end, the sequence generated is an increasing sequence (increments by 1 each time). Otherwise, it is a decreasing sequence (decreases by 1 each time).
e.g. range(0, 5) --> ["0", "1", "2", "3", "4", "5"]
range(5, 0) --> ["5", "4", "3", "2", "1", "0"]
(c) 3 arguments: the first argument is the range start, the second argument the range end, and the third argument the step. Steps are all positive numbers, and the sequence generated is a decreasing one only if the range start is larger than the range end. Step 0 will produce an empty array. Steps that are bigger than the range will produce only the range start.
e.g. range(1, 5, 2) --> ["1", "3", "5"]
range(5, 1, 2) --> ["5", "3", "1"]
range(1, 5, 0) --> [""]
range(1, 5, 100) --> ["1"]

Any permutation of strings and integers is possible when using > 1 arguments for the range function (e.g. range("1", 2) --> ["1", "2"]), except when the first argument is an integer and the last two arguments are strings (e.g. range(1, "2, 3")-->EvalError`) (can be fixed if wanted).

Whitespaces are still ignored as long as the values (string/ integer) are valid.

Both given range start and range end are included in the generated sequence.

Let me know your thoughts about this!
P/S Since arrays are now returned instead, the tests became a little unwieldy since I had to use the java String's join method to check against the expected output :/ Not sure if there's a better way to go about doing this...? Particularly since I notice that OpenRefine's testing seems to be quite lacking at the moment :P

@joanneong I revoke my previous comment that STOP should also be output.  The range STOP and the output is inconsistent once I played with your changes.  I strongly agree with @wetneb that we keep our 0-based indexing that we have throughout OpenRefine.  So a 5-element range call looks like
range(5) --> ["0", "1", "2", "3", "4"]
and not
range(5) --> ["0", "1", "2", "3", "4", "5"]

@thadguidry Just to clarify, does that mean that for something like range(2, 4) you should get ["2", "3"] instead of ["2", "3", "4"]?

@joanneong The current behaviour is :
GREL
forRange(2,2,1, e, e) --> []
Python/Jython
return range(2,2) --> []

@joanneong the expectation is that it works like Python 3.3+ range() does

UPDATE SUMMARY
The range function has been updated to follow Python's range function closely.

The range function returns an array and requires users to use .join(",") if they want to see it in the cells.

The range function can accept 1 to 3 string/integer arguments. Sequences generated include any given range start and exclude any given range end.
(a) 1 argument: 0 becomes the default range start, and the given argument is the range end
e.g. range(5) --> ["0", "1", "2", "3", "4"]
range("5") --> ["0", "1", "2", "3", "4"]
range(-1) --> [""]
(b) 2 arguments: the first argument is the range start, and the second argument is the range end. If the range start is smaller than the range end, the sequence generated is an increasing sequence (increments by 1 each time). Otherwise, an empty array is returned.
e.g. range(-1, 5) --> ["-1", "0", "1", "2", "3", "4"]
range(5, -1) --> [""]
(c) 3 arguments: the first argument is the range start, the second argument the range end, and the third argument the step. Step 0 will produce an empty array. Steps that are bigger than the range will produce only the range start.
e.g. range(1, 5, 2) --> ["1", "3"]
range(5, 1, -2) --> ["5", "3"]
range(1, 5, 0) --> [""]
range(1, 5, 100) --> ["1"]

Any permutation of strings and integers is possible when using > 1 arguments for the range function.
e.g. range("1", 5, "2") --> ["1", "3"]
range(1, "5, 2") --> ["1", "3"]
range("1", "5, 2") --> ["1", "3"]
etc.

Whitespaces are ignored as long as the string value(s) is/are valid.

Only the comma separator is supported to reduce code complexity. Dash operators can be replaced with commas by using a simple replace transformation prior to using the range function.

Tests have been updated to check for these functions. Do let me know if there is still something else that I have missed out!

can you remind me why range returns an array of strings and not integers?

I thought first it was to facilitate concatenations, but I forgot that the join() function is a good girl and it automatically cast into strings.

@wetneb @ettorerizza
I am aware that I could have returned either a string array or an integer array, so I just picked string because I felt that most of the inputs are likely to be strings (i.e. most likely from the data/cells rather than directly inputting integers in the transform box) so I might as well keep the output as a string array...
Ah but in that case Assert.assertArrayEquals seems to be more convenient for comparing integer arrays :P
What do you mean by an empty list? The reason why the array is empty but no error is thrown is because all the arguments are valid integers (not decimal etc.) but the step is negative even though the range start is smaller than the range end. The equivalent example in my unit test is range(1, 5, -1) -> [""]. Also, Python's range function exhibits this behaviour too (or at least when I tested here)

@wetneb
I went to think about it again, and I think another possible reason for using string arrays might be to improve performance. If I were to use an integer array, I have to use an Integer array rather than an int array because Integer is an Object but int is a primitive. This will require auto-boxing and auto-unboxing operations which can possibly make things slower, but I'll have to test this to be sure.
In any case, I do agree about the consistency, so I have updated the range function accordingly. So yes, now range(1, 5, -2) will return [ ]. Always ready for more suggestions!

@joanneong @wetneb What if we just added a Range Type as well ?  We could look into using existing libraries to gain benefits for future expansion and also add a Range Type so we can do map-reducing Range overlap/compare/etc once we implement Beam support later this year ?

https://github.com/google/guava/wiki/RangesExplained
https://commons.apache.org/proper/commons-lang/javadocs/api-release/org/apache/commons/lang3/Range.html

I think (in a new issue someone can create) we probably want to additionally have users tell us from a new drop-down menu option
Edit cells -> Common transforms -> To range
that a column is full of Range type values.  Because I'm seeing this kind of data about Ranges in the wild in scientific data sets both old and new... So to make @joanneong life even harder on this issue .... :)

1-9      <-- The OP's original format and why we have this issue
[1-9]
1..9
(1..9)
[1..9]    <-- Not seeing this one in any datasets however

@wetneb Yes of course, but what we have historically tried to do with GREL has always been giving easy power commands to help our users.  I feel strongly that having a Range type will benefit our users and allowing them to work with Range types and specifically in scientific data sets, Interval Notation, a bit easier.  By they way, I did find a generator, http://commons.apache.org/proper/commons-math/apidocs/org/apache/commons/math4/util/IntegerSequence.Range.html


Add custom events and fire them before invalidating and after invalidating the cache

A new inspection was created.

Codecov Report

Merging #6 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##             master     #6   +/-   ##
=======================================
  Coverage       100%   100%
- Complexity       44     50    +6
=======================================
  Files             8      9    +1
  Lines           112    128   +16
=======================================
+ Hits            112    128   +16

Impacted Files
Coverage Δ
Complexity Δ

src/CacheableObserver.php
100% <100%> (ø)
7 <4> (+3)
:arrow_up:

src/Events/CacheableEvent.php
100% <100%> (ø)
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 1d77f82...86f51a3. Read the comment docs.

The inspection completed: 5 new issues, 8 updated code elements

A new inspection was created.

A new inspection was created.

The inspection completed: 8 updated code elements

The inspection completed: 8 updated code elements


Observe files queried

Not sure if this is possible, but it would be good if fsatrace could write some information whenever a file had its modification time queried, something like q|filename. If this worked, then it would be very easy to speed up slow rebuild checkers, using Shake. For example, on my system cabal build takes 0.625s, but in certain circumstances, ghc --make can take > 1 min. If you could run fsatrace - -- cabal build, and then capture everything it reads and queries, and only rerun if any of that changes, you could reduce the rebuild time to 0.01s. The tool ghc-make already does that using custom logic for ghc --make, but with fsatrace it could be totally generic and I could kill ghc-make entirely.

This one is trickier, but https://github.com/jacereda/fsatrace/compare/stat seems to work properly on Darwin.

Linux version is even more ugly. I'll attempt Windows tomorrow.

I guess we need a flag to tell what ops you're interested in to reduce parsing cost.

Agreed, flags would be very useful.

I'm interested in such a tool, in my case stack build takes 2.2 secs.

Writing it on top of the shake forward mode I added is trivial. I was going to have shake --forward -- stack build do it.

Can you try this one? https://github.com/jacereda/fsatrace/compare/file-args
For some reason the travis build gets stuck on Mac, I can't reproduce it here.

Some results:
C:\Neil\neil>cabal --version
cabal-install version <IP_ADDRESS>
using version <IP_ADDRESS> of the Cabal library

C:\Neil\neil>fsatrace rwqmv - -- cabal --version
fsatrace.exe(8092): error: verbose
argv[0]=cabal
argv[1]=--version
cabal --version
cabal-install version <IP_ADDRESS>
using version <IP_ADDRESS> of the Cabal library

C:\Neil\neil>fsatrace rwqmvv - -- cabal configure
fsatrace.exe(7668): error: verbose
argv[0]=cabal
argv[1]=configure
cabal configure
Fatal: win/utf8.c:9: l || !sl, err: 7a
fsatrace.exe(7668): error: command failed with code 777
argv[0]=cabal
argv[1]=configure

C:\Neil\neil>fsatrace rwqmv - -- cmd /c cabal build
fsatrace.exe(1704): error: verbose
argv[0]=cmd
argv[1]=/c
argv[2]=cabal
argv[3]=build
cmd /c cabal build
Building neil-0.10...
Preprocessing executable 'neil' for neil-0.10...
r|C:\Windows\Globalization\Sorting\SortDefault.nls

So:

cabal --version gives different results whether I run it directly or via cmd. Any idea why that might be?
cabal configure fails in your utf8 stuff.
If I do the build, it claims to only read one file, which must be inaccurate, since it does a dependency check.

The neil directory is a checked out copy of https://github.com/ndmitchell/neil, but I have no reason to think that is particularly relevant.

What does this yield?
where cabal
fsatrace q - -- where cabal

As for invoking cabal via cmd, you probably want
fsatrace q - -- cmd /c "cabal build"

(Note the quotes)
I think I fixed the utf8 error.

C:\Neil>where cabal
C:\ghc\minghc-7.10.2-i386\bin\cabal.exe
C:\bin\cabal.exe
C:\Program Files (x86)\MinGHC-7.8.3\bin\cabal.exe

C:\Neil>fsatrace q - -- where cabal
where cabal
C:\ghc\minghc-7.10.2-i386\bin\cabal.exe
C:\bin\cabal.exe
C:\Program Files (x86)\MinGHC-7.8.3\bin\cabal.exe

I'll do some more digging tonight.

Is this one fixed?

Seemingly - both now give the same cabal version. Whether that is due to changes in fsatrace or the things on my PATH I don't really know... Sorry I totally forgot about this bug.

Good, closing.


what is the difference of  instances_train2017_sel_last_40_cats.json and instances_train2017.json in coco

during training last 40 classes data incrementally，we do not use first 40 classes data?

 I have the same problem as you


Unable to install Search Guard Bundle for ES 5.3

Unable to install Search Guard Bundle 5.3.0-12 for ES 5.3 on RHEL 6.6 having JDK as jdk1.8.0_73
As mentioned in Search Bundle Installation Page, unable to run ./elasticsearch--localhost/bin/elasticsearch
 Case 1
Steps performed as root user :

Downloaed  elasticsearch-5.3.0-12-62A9D172-9440-47FF-ABE3-59B73F7C40B5.tar.gz file from  here
extracted tar file.
executed ./elasticsearch-5.3.0-localhost/bin/elasticsearch

Output:
[WARN ][o.e.b.ElasticsearchUncaughtExceptionHandler] [localhost] uncaught exception in thread [main]
org.elasticsearch.bootstrap.StartupException: java.lang.RuntimeException: can not run elasticsearch as root
at org.elasticsearch.bootstrap.Elasticsearch.init(Elasticsearch.java:127) ~[elasticsearch-5.3.0.jar:5.3.0]
at org.elasticsearch.bootstrap.Elasticsearch.execute(Elasticsearch.java:114) ~[elasticsearch-5.3.0.jar:5.3.0]
at org.elasticsearch.cli.EnvironmentAwareCommand.execute(EnvironmentAwareCommand.java:58) ~[elasticsearch-5.3.0.jar:5.3.0]
at org.elasticsearch.cli.Command.mainWithoutErrorHandling(Command.java:122) ~[elasticsearch-5.3.0.jar:5.3.0]
at org.elasticsearch.cli.Command.main(Command.java:88) ~[elasticsearch-5.3.0.jar:5.3.0]
at org.elasticsearch.bootstrap.Elasticsearch.main(Elasticsearch.java:91) ~[elasticsearch-5.3.0.jar:5.3.0]
at org.elasticsearch.bootstrap.Elasticsearch.main(Elasticsearch.java:84) ~[elasticsearch-5.3.0.jar:5.3.0]
Caused by: java.lang.RuntimeException: can not run elasticsearch as root
at org.elasticsearch.bootstrap.Bootstrap.initializeNatives(Bootstrap.java:106) ~[elasticsearch-5.3.0.jar:5.3.0]
at org.elasticsearch.bootstrap.Bootstrap.setup(Bootstrap.java:204) ~[elasticsearch-5.3.0.jar:5.3.0]
at org.elasticsearch.bootstrap.Bootstrap.init(Bootstrap.java:360) ~[elasticsearch-5.3.0.jar:5.3.0]
at org.elasticsearch.bootstrap.Elasticsearch.init(Elasticsearch.java:123) ~[elasticsearch-5.3.0.jar:5.3.0]
... 6 more
 Case 2
Steps performed as 502 user:

Downloaed  elasticsearch-5.3.0-12-62A9D172-9440-47FF-ABE3-59B73F7C40B5.tar.gz file
extracted tar file.
created user (502) and group(games) and provided permissions
su 502
executed ./elasticsearch-5.3.0-localhost/bin/elasticsearch

Output:
Error: Could not find or load main class org.elasticsearch.tools.JavaVersionChecker
Elasticsearch requires at least Java 8 but your Java version from /usr/java/jdk1.8.0_73/jre//bin/java does not meet this requirement

executed sudo ./elasticsearch-5.3.0-localhost/bin/elasticsearch

Output:
[WARN ][o.e.b.ElasticsearchUncaughtExceptionHandler] [localhost] uncaught exception in thread [main]
org.elasticsearch.bootstrap.StartupException: java.lang.RuntimeException: can not run elasticsearch as root
at org.elasticsearch.bootstrap.Elasticsearch.init(Elasticsearch.java:127) ~[elasticsearch-5.3.0.jar:5.3.0]
at org.elasticsearch.bootstrap.Elasticsearch.execute(Elasticsearch.java:114) ~[elasticsearch-5.3.0.jar:5.3.0]
at org.elasticsearch.cli.EnvironmentAwareCommand.execute(EnvironmentAwareCommand.java:58) ~[elasticsearch-5.3.0.jar:5.3.0]
at org.elasticsearch.cli.Command.mainWithoutErrorHandling(Command.java:122) ~[elasticsearch-5.3.0.jar:5.3.0]
at org.elasticsearch.cli.Command.main(Command.java:88) ~[elasticsearch-5.3.0.jar:5.3.0]
at org.elasticsearch.bootstrap.Elasticsearch.main(Elasticsearch.java:91) ~[elasticsearch-5.3.0.jar:5.3.0]
at org.elasticsearch.bootstrap.Elasticsearch.main(Elasticsearch.java:84) ~[elasticsearch-5.3.0.jar:5.3.0]
Caused by: java.lang.RuntimeException: can not run elasticsearch as root
at org.elasticsearch.bootstrap.Bootstrap.initializeNatives(Bootstrap.java:106) ~[elasticsearch-5.3.0.jar:5.3.0]
at org.elasticsearch.bootstrap.Bootstrap.setup(Bootstrap.java:204) ~[elasticsearch-5.3.0.jar:5.3.0]
at org.elasticsearch.bootstrap.Bootstrap.init(Bootstrap.java:360) ~[elasticsearch-5.3.0.jar:5.3.0]
at org.elasticsearch.bootstrap.Elasticsearch.init(Elasticsearch.java:123) ~[elasticsearch-5.3.0.jar:5.3.0]
... 6 more

Pls. try

Download elasticsearch-5.3.0-12-62A9D172-9440-47FF-ABE3-59B73F7C40B5.tar.gz file
extract tar.gz file
./elasticsearch-5.3.0-localhost/bin/elasticsearch

without anything like su/sudo/etc and as a non-root user.
If you want to start it as another user you first need to chown all files

Thanks @floragunncom for your prompt response.
Found the RCA:- I had downloaded and extracted the tar file as root user.
After downloading, extracting and running elasticsearch as non-root user, elasticsearch runs fine.
But after starting elasticsearch, while running logstash I am getting below error in logstash .
[logstash.outputs.elasticsearch] Attempted to resurrect connection to dead ES instance, but got an error. {:url=>#<URI::HTTP:0x75ed84c4 URL:http://localhost:9200/>, :error_type=>LogStash::Outputs::ElasticSearch::HttpClient::Pool::HostUnreachableError, :error=>"Elasticsearch Unreachable: [http://localhost:9200/][Manticore::ClientProtocolException] localhost:9200 failed to respond"}
[2017-04-20T02:48:15,993][INFO ][logstash.outputs.elasticsearch] New Elasticsearch output {:class=>"LogStash::Outputs::ElasticSearch", :hosts=>[#<URI::Generic:0x5bf33964 URL://localhost:9200>]}
[2017-04-20T02:48:16,269][INFO ][logstash.pipeline        ] Starting pipeline {"id"=>"main", "pipeline.workers"=>4, "pipeline.batch.size"=>125, "pipeline.batch.delay"=>5, "pipeline.max_inflight"=>500}
[2017-04-20T02:48:16,986][INFO ][logstash.inputs.beats    ] Beats inputs: Starting input listener {:address=>"<IP_ADDRESS>:5044"}
[2017-04-20T02:48:17,021][INFO ][logstash.pipeline        ] Pipeline main started
[2017-04-20T02:48:17,141][INFO ][logstash.agent           ] Successfully started Logstash API endpoint {:port=>9600}
[2017-04-20T02:48:19,100][WARN ][logstash.outputs.elasticsearch] Marking url as dead. Last error: [LogStash::Outputs::ElasticSearch::HttpClient::Pool::HostUnreachableError] Elasticsearch Unreachable: [http://localhost:9200/][Manticore::ClientProtocolException] localhost:9200 failed to respond {:url=>http://localhost:9200/, :error_message=>"Elasticsearch Unreachable: [http://localhost:9200/][Manticore::ClientProtocolException] localhost:9200 failed to respond", :error_class=>"LogStash::Outputs::ElasticSearch::HttpClient::Pool::HostUnreachableError"}
Please help in understanding where is it trying to connect via HTTP (instead of HTTPS) .

Try curl -k -X GET 'https://localhost:9200' and make sure you have hostname verification disabled in elasticsearch.yml
See https://github.com/floragunncom/search-guard-docs/blob/master/tls_configuration.md
Please ask questions here: For that refer to https://groups.google.com/forum/#!forum/search-guard


[Bug]: Importing App from Git Removes Default Branch Protection

Is there an existing issue for this?

[X] I have searched the existing issues

Description
Upon importing an app from Git, the branch protection feature is removed, allowing changes to be made to the master branch.
Video - https://github.com/appsmithorg/appsmith/assets/43263619/3c7b70bc-b137-4416-bc1f-1c2c708a5ef8
Steps To Reproduce

Import an application using Git.
Open Git settings to see the changes.

Public Sample App
No response
Environment
Production
Severity
Low (Inconsistent UI/UX)
Issue video log
https://github.com/appsmithorg/appsmith/assets/43263619/3c7b70bc-b137-4416-bc1f-1c2c708a5ef8
Version
v1.20.0

Closing this as it does not have an appropriate label associated with a pod


Predefined commands ?

Hello,
I would like to know if it is possible to cut the screen in two.
The idea would be to have a live ssh part at the top and underneath a possibility to have predefined buttons that will interact with the ssh.
Thank you

this is not what ttyd will do, you may want to try tools like tmux.


Bkane/status up

Support status=up for octodns-azure dynamic records.
Uses https://azure.microsoft.com/en-us/updates/alwaysserve/

@viranch will be a more meaningful reviewer for this

I've reviewed and written some of this code and I'm happy with it.

In general though meaty changes to dymamic records need to pay special attention to upgrade/downgrade paths, both in terms of users moving between provider versions and with the details of out existing records (created with old versions) are detected and upgrade when necessary.

Agreed, this needs to be tested /cc @bkane-msft.

In general though meaty changes to dymamic records need to pay special attention to upgrade/downgrade paths, both in terms of users moving between provider versions and with the details of out existing records (created with old versions) are detected and upgrade when necessary.

I tested upgrade & downgrade paths and they seem to work well. status=up is converted to status=obey in previous version with sufficient log entries and the flags are corrected without exception. This is because status=up is a separate flag in Azure that doesn't interfere with previous version. Similarly, status=up is honored in this PR and the appropriate flags are set again without exception.
So good to go on this front 👍

All dependencies upgraded to latest stable releases. This is ready for final 👀 before its merged.