Refactor Solr schema scripts for ACL and CORE collections

Currently, JSON data for our Solr schemas are put directly into the setup scripts. There are two ways we can refactor this:

Put JSON data in a separate file and have different schema files for adding or replacing fields, and allow our bash setup script to take the JSON data as a command line argument. In other words, we have a single setup script but multiple schema data files

Keep JSON data in the bash setup scripts as-is, but support different flags such as "--replace" to replace existing fields or "--add" to add new fields. Here, we have multiple setup scripts for each specific collection (ACL, CORE).

We will also need to update documentation after this is done.
Option 1 seems cleaner to me but would like to hear other opinions @nikhilro @lintool

sounds good

Issue resolved in #1069 and and #1068.


Bloxstrap Not Launching Issue.

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
Hello pizzabloxer! I am a player who really enjoyes Bloxstrap and its fps boost. You have made a really good product - Bloxstrap.Exceptions.HttpResponseException: Could not connect to https://clientsettingscdn.roblox.com/v2/client-version/WindowsPlayer/channel/zlive because it returned HTTP 401 (Unauthorized)good product and I love it. But recently, it has been getting an issue when I'm trying to play I really hope you can solve this problem:

just read wiki, please
https://github.com/pizzaboxer/bloxstrap/wiki/'HTTP-401-(Unauthorized)'-when-trying-to-launch-Roblox

Thank you for your assistance. Now I am able to use bloxstrap as usual
again! Thank you so much
On Thu, 2 Nov 2023 at 13:48, pizzaboxer @.***> wrote:

Closed #885 https://github.com/pizzaboxer/bloxstrap/issues/885 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/pizzaboxer/bloxstrap/issues/885#event-10840694916,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BDVZMU757C3GXG5JFMGBRUTYCNJNHAVCNFSM6AAAAAA62MWCWGVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJQHA2DANRZGQ4TCNQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Redis is changed to access with password, and trylock takes more time

Redis used to access without password, and then upgraded to access with password. It was found that trylock took more than one second。
redisson version is 3.16.0
The configuration is as follows：
config.useClusterServers()                     .setPassword(password)                     .setScanInterval(2000)                     .addNodeAddress(xxxxx);             return Redisson.create(config);
so, I tried to reduce the reission version to 3.11.6, and the time consumption decreased significantly
I don't know what caused this. Is it the wrong way to use the new version?

It was caused by introduction of WAIT command to sync with slaves. It seems that sync takes much time in your case.

It was caused by introduction of WAIT command to sync with slaves. It seems that sync takes much time in your case.
Is there any way to solve it? I found this problem after version 3.14.1. Or can I only use the old version, and the new version all has this problem?

And My waittime input parameter is 0

Could you set TRACE logging level for org.redisson package and share output?

[basic-porter-tadata-x.x8.31.50-0000] 2022-05-10 21:14:28.554 DEBUG 25384 [http-nio-8080-exec-6] [ClusterConnectionManager.java:751] slot 2875 for sync_tadata_lock
[basic-porter-tadata-x.x8.31.50-0000] 2022-05-10 21:14:28.555 DEBUG 25384 [http-nio-8080-exec-6] [ClusterConnectionManager.java:751] slot 2875 for sync_tadata_lock
[basic-porter-tadata-x.x8.31.50-0000] 2022-05-10 21:14:28.949 DEBUG 25384 [redisson-netty-2-2] [ClusterConnectionManager.java:466] cluster nodes state got from x.x.x.x/x.x.x.x:7001:
a45ecb9201e0bf48665ed7e5a9846cbe6d753x5 x.x.x.x:7006@17006 slave f5d943dbd5806937e204ee35ea5f6aa65e46a6ca 0<PHONE_NUMBER>214 6 connected
40e866c9d83596f7d50ff09ee92370f580c9add9 x.x.x.x:7001@17001 myself,master - 0<PHONE_NUMBER>000 1 connected 0-5460
f226329e04bda39971a532552f18a32bb2c1b7a8 x.x.x.x:7005@17005 slave ced813a2fd0db10432dbc999befd3709ac2234a2 0<PHONE_NUMBER>217 5 connected
ced813a2fd0db10432dbc999befd3709ac2234a2 x.x.x.x:7002@17002 master - 0<PHONE_NUMBER>208 2 connected 5461-10922
f5d943dbd5806937e204ee35ea5f6aa65e46a6ca x.x.x.x:7003@17003 master - 0<PHONE_NUMBER>212 3 connected 10923-16383
eccdd6be867c3fc36a27f2c1cc105c98c2d08879 x.x.x.x:7004@17004 slave 40e866c9d83596f7d50ff09ee92370f580c9add9 0<PHONE_NUMBER>000 4 connected
[basic-porter-tadata-x.x8.31.50-0000] 2022-05-10 21:14:30.415 DEBUG 25384 [redisson-netty-2-5] [RedisExecutor.java:522] connection released for command null and params null from slot NodeSource [slot=null, addr=null, redisClient=null, redirect=null, entry=MasterSlaveEntry [masterEntry=[freeSubscribeConnectionsAmount=1, freeSubscribeConnectionsCounter=value:50:queue:0, freeConnectionsAmount=24, freeConnectionsCounter=value:64:queue:0, freezeReason=null, client=[addr=redis://xxx.xx.xxx.xxx:7001], nodeType=MASTER, firstFail=0]]] using connection RedisConnection@1857575476 [redisClient=[addr=redis://xxx.xx.xxx.xxx:7001], channel=[id: 0xd41a442c, L:/xx.xxx.xxx.xx:59598 - R:xxx.x.x.x/x.x.x.x:7001], currentCommand=null]

@mrniko can you help me on this queston?

I reproduced your case only if masterauth is not set for slave nodes.

I reproduce your case only if masterauth is not set for slave nodes.

thank you very much !  That's why.


Update readme with Serverless info

Update the readme with some (non-private) info on the Serverless deployments.

@fkondej This next, please.

@ashleyvega do you want something more than vegaprotocol/devops-infra#502 ?


Bug: FzfLua lsp_definitions throwing an error

RTFM Checklist

[X] I have searched exisiting issues / discussions
[ ] I have read the Wiki including the Advanced section
[ ] I have read man fzf / I am well versed in shell fzf

Operating system
Linux
Shell
zsh
Neovim version (nvim --version)
NVIM v0.11.0-dev-1035+g50e63c817
Fzf version (fzf --version)
0.56.3 (add1aec6)
Output of :lua print(os.getenv('FZF_DEFAULT_OPTS'))
--height 50% --layout=default --border --color=hl:#933737
Is the problem reproducible with mini.sh?

[ ] My issue is reproducible with mini.sh
[ ] My issue IS NOT reproducible with mini.sh
[X] I have not tested with mini.sh (not relevant, requires LSP, Windows, etc)

Fzf-lua configuration
Folke's LazyVim default FzfLua settings.

Describe the bug / steps to reproduce
Attempting to run the command FzfLua lsp_definitions jump_to_single_result=true ignore_current_line=true via LazyVim's default keymap gd results in the following error:
   Error  11:11:52 PM msg_show.lua_error Error executing Lua callback: /usr/share/nvim/runtime/lua/vim/lsp/rpc.lua:289: Cannot serialise function: type not supported
stack traceback:
	[C]: in function 'encode'
	/usr/share/nvim/runtime/lua/vim/lsp/rpc.lua:289: in function 'encode_and_send'
	/usr/share/nvim/runtime/lua/vim/lsp/rpc.lua:334: in function 'request'
	/usr/share/nvim/runtime/lua/vim/lsp/client.lua:673: in function 'request'
	/usr/share/nvim/runtime/lua/vim/lsp.lua:882: in function 'buf_request'
	/usr/share/nvim/runtime/lua/vim/lsp.lua:945: in function 'buf_request_all'
	/usr/share/nvim/runtime/lua/vim/lsp.lua:966: in function 'buf_request_sync'
	...al/share/nvim/lazy/fzf-lua/lua/fzf-lua/providers/lsp.lua:496: in function 'fn_contents'
	...al/share/nvim/lazy/fzf-lua/lua/fzf-lua/providers/lsp.lua:662: in function <...al/share/nvim/lazy/fzf-lua/lua/fzf-lua/providers/lsp.lua:658>
	...e/fic/.local/share/nvim/lazy/fzf-lua/lua/fzf-lua/cmd.lua:35: in function 'run_command'
	/home/fic/.local/share/nvim/lazy/fzf-lua/plugin/fzf-lua.lua:12: in function </home/fic/.local/share/nvim/lazy/fzf-lua/plugin/fzf-lua.lua:11>

It happened using basedpyright and lua_ls, I have not tested other language servers yet.

Seems you're using a rather old neovim nightly build, the error seems to come from the runtime, can you please update to the latest nightly (works for me with lua_ls) and also try on 0.10.2 and let me know?

It works both on the latest nightly and on 0.10.2, thank you very much!
I didn't realize my install was so outdated. I was using the neovim-nightly-bin AUR package, and replacing it with neovim-git fixed the issue. Thanks!

Yap, your build is 1035 (in the OP), we're currently on 1383 :-)


Add barcode logic to CytoSnake's CLI

About this PR
This PR adds CytoSnake to have logic when handling CLI user based inputs. Specifically, this update introduces barcode logic,
By default, a barcode is not required as an input to run CytoSnake; however, there are some exceptions when barcodes are needed.
If a user provides a dataset that has been generated from multiple experiments, then multiple plate maps are associated with the generated data. This will require a barcode file in order for CytoSnake to know which plate dataset is associated with which experiment.
What's new?

A new module known as input_guard.py was created. This will handle all the CLI logic
New function: check_init_parameter_inputs() where it takes user based parameters and checks for discrepancies.

Implementation

Barcode Logic Design. This image above present a diagram on how the barcode logic is handled within CytoSnake. In this example, we have 6 plate datasets that are seperated into two groups. Each group represents an experiment that has been conducted in order to generate the data. In addition, there is a metadata folder. where each platemap file is associated with the group of plate data. A) Demonstrates a user providing both plate data groups and a metadata file but fails to complete the init mode causing the raise barcode error to light up. B) Demonstrates a succesful init run where the user provides all the necessary inputs and makes the conduct init to light up.

What dictates the barcode logic is the number of plate maps found within the metadata data folder. If CytoSnake see's that there is more than 1 plate map, then it requires a barcode. Therefore, users must provide barcodes if multiple plate maps are present.
additional Notes
Changes in workflow
Update on workflows:

This should have been taken care of in #37, where workflow configs were created for cp_process_singlecells; however, workflow config was not created for cp_process hence this PR contains a new workflow config for cp_process

@d33bs Hopefully I have attended all your comments. Also, thanks for the great questions!

why are barcodes only required for multiple experiments (and not for single experiments)? For example, if two experiments need to be compared but they are processed individually, would we run into issues when attempting to compare things later on?

I will be used this repo to explain my current understanding.
From what I understand, the barcode file provides an assay-to-platename pairing. Where the assay plate are sqlite files (in this case) from cytominer-database. In the barcode file we see that there is an association between a specific plate map (Plate_Map_Name column) per assay., which contains metadata information that includes: well position, perturbations, etc. Looking at the barcode structure, there are some plate names that repeat 3 times (first 3, middle 3 and last 3) indicating that 3 experiments were conducted in triplicates. (Different plate map name = separate experiment)
Technically, there is no need to have a barcode for a single experiment because it will contain the same external factors among all plates (assuming that more than one plate was used in the experiment). The only time when barcodes are required is if 3 separate experiments were conducted on multiple plates. Therefore, the barcode will help find which plates have been involved with which experiment, thus mapping the correct metadata to those plates when conducting downstream analysis.

would we run into issues when attempting to compare things later on?
Since the metadata (platemaps) can be incorporated within the single-cell / aggregate morphological profiles, you can stratify them based on experiments. The merging of the metadata to the morphology profiles is conducted by using the pcytominer's annotate where it requires both the profile and platemap as inputs.

However, one needs to map the correct assay with the associated platemap, which CytoSnake does when annotating multiple plate datasets (assays)

Similarly: are there ever scenarios where we don't have the barcode file but need to run analyses on the experiments? Here especially I'm thinking about previously gathered data where one may no longer have access to all data, or perhaps the data is stored in an unrecognizable format. In these scenarios could you simulate the barcode file's data (providing notation that it's simulated) to help facilitate the work involved with this PR?

The barcodes only provides information that distinguishes which plates came from which experiment. Assuming that the data you are talking about came from 3 separate experiments and no barcodes were provided. A potential solution is that we contact the person who generated this dataset and asks which plates came from which experiment.
However, with the scenario, if no plate maps were provided, then we will not know what types of external treatments were added to the cell and which experiments contained the types of treatments/cell lines used. Therefore, it will be difficult to simulate due to the lack of important metadata data like treatments, well positions, and cell lines used.

I have applied all the changes. Merging now. If there is more work need to be done, please feel free to re-open this PR.


feat: report endpoint sunset

What/Why/How?
Warn user about planned endpoint sunset.
Currently there is no way to let user know about breaking changes in the endpoints or endpoint deprecation used for specific command.
Lookup for Sunset header in responses received from Reunite to notify users if there are resources that is going to deprecate or is already deprecated (but is still functioning).
Reference
Testing
Screenshots (optional)
Endpoint is about to be deprecated

Endpoint is already deprecated

Check yourself

[ ] Code changed? - Tested with redoc/reference-docs/workflows (internal)
[ ] All new/updated code is covered with tests
[ ] New package installed? - Tested in different environments (browser/node)

Security

[x] Security impact of change has been considered
[x] Code follows company security practices and guidelines

Can we go back a step, I don't think I understand what's happening here - are we adding a user-friendly outcome in the CLI for what happens when the CLI tries to do something that we've stopped supporting in our API? Is it generic, or specifically for a planned change to the push command? What's the change?
In terms of communicating to users, the user thinks of the CLI has having "commands", they don't care about the implementation details such as endpoints. Can we say "this command is unavailable, updating the CLI tool might help " ?


Discussion: Creation/handling of additional resources required

There can be cases where the infra/workloads would require to create additional resources which could be something like RBAC and since it's not required for the operator itself, It wouldn't make sense to mandate user to create them. So thus there are 2 ways to approach this problem:

Request user to create the necessary resources after deploying operator and before applying the cr. This would put the burden of managing the resources lifecycle on the user, and thus would not require the operator to do anything more. The issue with this is relying on user to make the appropriate changes and also that it'll require more user involvement.
Have operator create the resources on the go i.e. if user applies a cr that requires a workload/infra to have additional resources such as RBAC it'll create on the fly. The issue with this is that it requires the operator to delete these RBAC based on conditions( requiring significantly more logic). But if the user somehow manages to delete the operator, then that'd be futile.

What do you think is the best way to move forward?

We have decided to not have Ripsaw manage this.

Closing this as it's been decided to not have Ripsaw manage the resources required by other operators.


⚠ Mitsubishi has degraded performance

In db283d1, Mitsubishi (https://mitsubishimotorsid-mobileapp.com/app/login) experienced degraded performance:

HTTP code: 200
Response time: 15134 ms

Resolved: Mitsubishi performance has improved in 7cd7dd7 after 1 minute.


Global Settings: Settings: 'ui-brand' value automatically reverted to none when navigating away from global settings

Setup

Rancher version: v2.7.5
Rancher UI Extensions: N/A
Browser type & version: Any

Describe the bug
'ui-brand' value is automatically reverted to  when navigating away from global settings.
To Reproduce

login to Rancher UI as an admin user
go to Global Settings -> Settings (/dashboard/c/local/settings/management.cattle.io.setting)
scroll down to 'ui-brand' option
click elipsis
click 'Edit Setting'
update ui-brand value (e.g. 'suse')
click 'Save'
notice rancher logos are used
navigate to the home page
notice rancher logos are no longer used

Result
'ui-brand' settings value is automatically reverted to  when navigating away from global settings.
Expected Result
'ui-brand' settings value should be preserved.
Screenshots

https://github.com/rancher/dashboard/assets/127343932/75743452-2e98-4865-92b1-42827e738445

Based on the video, the ui-brand parameter has been changed. @codyrancher tried a repro and noted that upon leaving the page, 2 PUTs are made and the value is blank.

Caused by this section of code in branding https://github.com/rancher/dashboard/blob/b99b65003ea4a8ff6f41298db371edf90cf662fd/shell/mixins/brand.js#L147-L151

In terms of branding the cspAdapter is king, so if that thinks there's no branding and a branding option is set... it removes the branding option. This catches errors if the user goes from prime / suse world to not prime / suse world.
However that introduces the bug here where there is no cspAdapter branding and the user tries to set a branding value.
Need to check with @nwmac to see which is best here, cater for the case an instance goes non-prime or allow the user to manually set a branding.

@richard-cox I'm confused - we'll need to discuss this

Discussed with Neil, we can chop out the code that resets the branding setting if there's no cspAdapter app. i.e.
shell/mixins/brand.js
      else if (!neu) {
        const brandSetting = findBy(this.globalSettings, 'id', SETTING.BRAND);

        if (brandSetting) {
          brandSetting.value = '';
          brandSetting.save();
        }

Fixed: ui-brand no longer reset automatically when navigating away from page
https://github.com/rancher/dashboard/assets/127343932/2c5b30fa-4069-411c-a40a-35f31e374def


SHRINK does not alter bucketize status

SHRINK on bucketized GTS does not modify bucketcount nor lastbucket.
Shrinked bucketized GTS end up with lastbuckets that are not aligned with their last data point.
Fix options:

reset bucketize status in shrink (unbucketize)
or update bucketize parameters in shrink (but must account for non bucket data points)
or precise in shrink documentation the current behavior

SHRINK is focused on restricting the number of values, the bucketization parameters should not be altered as a result, so your last option is the one to adopt.

Documentation has been modified.


85 create detail pages

This PR introduces the ability to jump from search results to corresponding pages, and see the details for Chairs, Rooms, and People.
closes #85
closes #87

We will not test the auto-generated controllers because they work by definition.


Application in Production

Greetings!
I'm having an issue setting up the server. Whenever I run
php artisan migrate
It gives me "Application In Production! Do you really wish to run this command? (Y/N)"
When I respond with yes I get the following error:
[PDOException]
SQLSTATE[HY000] [2002] No connection could be made because the target machine actively refused it.
I'm not sure what I need to do or if I have done anything wrong.
Thanks

Never mind. I was able to figure out the problem. Mysql server kept closing and failing which messed up the connection. Thanks

Hi @Jazz527 !
It looks like you have some problem with MySQL or with connection configuration.
Connection configurations you can manage in .env file.
Also you can check our requirements for web server and mysql here http://connfa.com/integration-server/server-requirements/

Thank you so much for the help. I will be posting a separate issue on a new thread.
Thanks again for the help. I'm very new to this!


extendable options using window.LiveFormOptions

Example:
var LiveFormOptions = {
    showValid: true,
    messageErrorPrefix: ''
};

Solved bit differently, see b0a277e4b16e0c953399bb34c139c779a2d4795f.

Yes, this is more elegant, thanks! Just tested and works fine.


Leaves without instance

Leaves that have no instances have also appeared on the tree. I suggest eliminating that ones as they might mislead the ones investigating the tree.

The problem is fixed. The nodes decisions without instance are decided by observing sibling's dominant decision


It's not the cleanest way of supporting multiple monitors/windows, however you can do the following:

          It's not the cleanest way of supporting multiple monitors/windows, however you can do the following:

File > New Window

Now drag a tab in your already existing Visual Studio Code window into the new window you just opened.

I agree that it would be really nice to just be able to drag an existing tab to a second monitor but this is at least a pretty painless workaround until they support dragging tabs to another monitor.
Originally posted by @Blackbaud-DustinLunsford in https://github.com/microsoft/vscode/issues/10121#issuecomment-315778189

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Add Ubuntu

Thank you for your contribution! Some pointers to get it merged quickly:
For contributions in software/:

[X] PR Title: Please use "Add <vendor/product name>" (instead of "Update README.md")
[X] Status: please select a value from the status table at the top
[X] Version: Status Vulnerable / Workaround? -> List vulnerable versions.
Status Fix?                     -> List fixed versions.
[X] Links: make sure you link to a public statement by the developer.
Alternatively, include and link a file in the software/vendor-statements/ directory
[X] Please mind the sorting

Hi, thanks a lot for the contributions. However, Ubuntu is partially listed in the entries already, under the vendor Canonical. However, not everything you mentioned is added currently. Could you make you commits in line with the current entries and then I'll merge.

Hi @rkokkelk, sorry for the mistake, I forgot to check at Canonical in the list. I've updated the PR as requested.

No problem, your help is highly appreciated. Thank you!


Isort import

Just boy-scoutin'

K looks good. Funny I forgot about pulling code that never made it from the weekend.


fixed jemalloc heap profiler build

This fixed an issue in the CMakefile in case jemalloc is not locally installed AND the heap profiler is enabled.

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/13411/

Tests blue except known unrelated failures.


fix: refreshing tokens at the wrong time

StoredSession.isValid was being calculated incorrectly, because by comparing to "1 minute ago", it means an access token which expires "30 seconds ago" is still valid for 30 seconds. Changing to plus 60 seconds eliminates this problem, and bakes in plenty of time for latency issues.
In a future evolution I believe this value should be configurable.
What kind of change does this PR introduce?
This fixes is a critical bug in the token refresh logic, by ensuring refresh tokens are refreshed before they become expired.
What is the current behavior?
The logic to calculate if a stored access token is valid is flawed and results in consistently getting expired access tokens from await auth.session if it's within 60 seconds of expiring.
The current logic is:
isValid = expirationDate > Date().addingTimeInterval(-60)
In other words, a token is valid if its expiration date is later than a minute ago.
So let's say you have an access token which expires at 11:59:50.
And let's say you call await auth.session at 12:00:00. (hh:mm:ss)
isValid is then calculated as 11:59:50 > (12:00:00 - 60), or 11:59:50 > 11:59:00.
So, isValid is true even though actually the token expired 10 seconds ago.
What is the new behavior?
The new behavior changes the isValid calculation from subtracting 60 seconds from "now" to adding 60 seconds. This means the access token returned from auth.session will be guaranteed to be valid for at least 60 seconds. In a future evolution this could be a user-configurable value, but it's important it's greater than 0 or you will consistently get expired access tokens if you ask for one in <60s from when it expires. Being greater than 0 (rather than just 0) has the benefit of being proactive about the token, meaning it handles any server latency etc that could arise as you attempt to use the token.
Additional context
For context, my app which uses Supabase would 401 requests seemingly randomly. Then I realized it was happening every hour, which I realized is what I set my auth token expiry time to. This change fixes it!

@devi-prsd I noticed you recently committed, do you mind taking a look at this?

@grsouza mentioning you too if you could take a look!

@MilesV64 thanks for this!

@MilesV64 I don't have permissions to approve it, but looks good to me!

Hey @MilesV64 thanks for fixing this.

Released on 1.1.1


TASK-1161 Store sub events in database

Now stores sub events in database and generates an ID for the sub event.
Does not yet check to avoid storing duplicate events or reprocessing
successful sub events.
Removed the id field from StatusEvent (now in StatusEventWithID).
Remove a not particularly helpful level of indirection by marking event
processing state in the storage system with a direct storage call,
rather than via an iterator then a queue then the storage system

Merging per @sychan


Add support for passthrough arguments to NumpyArrayIterator

There have been some requests for more flexible behaviour for the ImageGenerator (see for example issue #9877). This pull request takes a step towards that direction (although it doesn't yet solve the problem in the issue I linked).
I add a support for multiple input arguments in NumpyArrayIterator (and thus, the flow() function). It now accepts following syntax: flow((image_data, [other_data1, other_data2]), ...). The image_data part gets processed as before and the optional other_data arguments get passed through along with the image data without manipulations. This makes it possible to use flow() to feed a model multiple miscellaneous data arguments while still using the Keras image augmentations for the image data.
No old behaviour gets modified. All the modified functions can still be used as before by just leaving the additional arguments out.

I added a new commit to fix a pep8 issue. The Travis CI check failed because of pep8 errors that I did not catch on my machine. Apparently E226 error is ignored by default in pep8 while in Keras Travis CI they are not ignored. Live and learn...

Thanks for the PR!
@Dref360 could you please take a look and tell us what you think?

@Dref360 Thank you for the comment. I did that and it shortened the code by about two lines.
I also found a small error that I fixed in the new commit. The previous version was slightly incompatible with the old Keras behaviour: instead of returning just batch_x, the function returned (batch_x,) whenever self.y is None. Now this is fixed.

I also found a small error that I fixed in the new commit. The previous version was slightly incompatible with the old Keras behaviour: instead of returning just batch_x, the function returned (batch_x,) whenever self.y is None. Now this is fixed.
We should have a test for that. Could you please add it in this PR?
LGTM after that :)


API: define _constructor_expanddim for subclassing Series and DataFrame

Closes #9762. Does this needs release note?

as an aside can u out a small doc that shows how to subclass DataFrame now that it is reasonably supportered - maybe put it in the FAQ

Sure, will do. By the way, is there any difference between FAQ and Gotcha? Otherwise, it may better to merge them to be more searchable?

@jorisvandenbossche what about merging faq and gotchas? I would say yes

Added subclassing documentation. Pls review.

@sinhrks This documentation on subclassing is great
I don't have much feedback on the particulars because I have not done much subclassing of dataframe myself.
However, we should clarify two things:

Is this subclassing API officially supported? My impression was that up until now, this has not been the case. We should think carefully about what we can guarantee this and in any case include a warning against using other internal methods. Perhaps we should explicitly label this subclassing API as "experimental" (we reserve the right to break it in the future if necessary).
We should encourage using composition instead of inheritance when possible. This is a good opportunity to point out that the Index class, in particular, has a well defined API. For example of using composition with the pandas.Index to enable label based indexing, you could point users to xray.

Nice docs! I agree with the questions raised by @shoyer, although I cannot say much about it myself as I never did some subclassing of dataframes.
But one remark: I don't think this belongs in a FAQ, but that we should start with some 'advanced' section in the docs (together with internals or so)

@sinhrks pls add a mention to try using monkey-patching first if you just want to add a function http://pandas.pydata.org/pandas-docs/stable/faq.html#adding-features-to-your-pandas-installation
maybe put this as the first warning box! (2nd would be the composition/inheritance)

@sinhrks ok as @jorisvandenbossche just mentioned, let's move this to the internals.rst section.

Fixed @jreback 's points, and moved it to internals.rst. And preparing a composition example in separate PR.

@sinhrks lgtm, some minor comments.
I think you can add a release note in enhancements. Trying not to 'advertise' this too much :)

@jreback Thanks, fixed your points, and added a release note.

@sinhrks awesome.
I sent you an invite to change the pandas push permissions. As soon as you accept, pls make this your first PR to merged!

Thanks, let me merge this once after rebased.

@sinhrks awesome!

Did we ever resolve the question of whether the subclassing API is officially guaranteed or not? I suppose the reality is that people are going to subclass whether we tell them it's OK or not.

@shoyer well enough warnings to not encourage it
but u r right people will do it anyways
it's pretty doable nowadays (though imho composition is much more useful)


Simplify configuration of test coverage

@mejackreed I pushed this commit to #11 just after you merged it. Mind taking a quick look? This gets the coverage to :100:.

Pull Request Test Coverage Report for Build 9

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+100.0%) to 100.0%

Totals

Change from base Build 7:
100.0%

Covered Lines:
12

Relevant Lines:
12

💛  - Coveralls


Use full compute metadata DNS name

Use the metadata.google.internal instead of just metadata. This enables Compute Engine auth to work on Kubernetes, Container Engine,  and Container-Optimized VMs and setups with custom DNS servers. Otherwise, the DNS lookup for metadata will fail.

I just noticed that #61 was created in Sept, however, it seems to be out of date now and the tests are failing.

Thanks. I'll keep an eye on the next release.


LUIS Date recognition without separator

Bot Info

SDK Platform: Node
Active Channels: FB Messenger
Deployment Environment: Azure App Service

Issue Description
I'm using LUIS for recognising date ranges for a travel agent chatbot. All works for me, but for some reason, I can't get utterances without a date separator to work. For example "12 13 Sep" is not recognized as a date range, even though "12-13 Sep" is. I read somewhere that LUIS uses chrono for date parsing, but "12 13 Sep" works perfectly with chrono.
I'm in a fix because many of my users are being affected by this. What can I do?

Are you using datetimeV2?
Some suggestions you may want to try:

In your prompt specify the date format needed to properly recognize the date range.
You may be able to create a composite entity like {{startDate}{enDate}{month}} or {{startDate}{enDate}{startMonth}{endMonth}}. This is kind of a hacky solution, but it might work for you.

Yup, I'm using datetimev2.
By specifying the date format in the prompt, do you mean specifying it to the user?

I'm assuming you prompt the user i.e. "what days would you like to travel" or "what are your travel dates" adding something like "please format you date as start day - end day month" or something similar may do the trick to help guide your users.

going to post this answer on SO and close this Issue here, that way it is indexed for others that may be having the same issue


[react-search-refiners] Added local configuration for pnp js library instead of global

Q
A

Bug fix?
yes

What's in this Pull Request?
Added local configuration for pnp js library instead of global to avoid conflicts with other web parts.

thx @FranckyC, appreciated.


Get Unauthorized while trying the UMA flow

Followed the steps here<EMAIL_ADDRESS>Get Unauthorized error in step 6. Use admin/admin to get the PAT.

Issue is not reproducible.

Closing since not reproducible.


Graph Traversal

I've implemented graph traversal in this PR. Although the functionality has been implemented in the GraphApiClient, it does not fit our traditional pattern of one client method per REST endpoint. One method takes an object, convert it to an AQL query for graph traversal and executes it through the second method which takes an AQL query and executes it in the database through the CursorApiClient.

ApiClientBase now exposes a new member protected ArangoDBClient _arangoDBClient; and accepts a new parameter in its constructor ArangoDBClient arangoDBClient = null. The parameter is null by default to ensure backward compatibility. The above features mean that the client is now aware of the ArangoDBClient it is associated with and has the ability to use its other features. This was necessary so that we could consume the CursorApiClient functionality from within the GraphApiClient to do the traversal.

@DiscoPYF , thanks for your recommendations. I have implemented as you suggested in 1, 2 and 4.

This PR (and several others) have gathered quite a bit of dust...
If anyone watching this still thinks having a graph traversal API would be useful, I'd like to suggest that it could be implemented in a separate library, packaged as a separate lib in nuget.  It can perhaps be implemented as a (set of) extension method(s), call the separate package ArangoDBNetStandard.GraphTraversalExtensions or something like that?

Closing PR, I believe such an abstraction belongs in a separate library/package.


Align tabs to center in JFXTabPane

I was wondering if there was a way to align the tab tops to the center(preferably using css).At the moment they always align to the left.

Hi,
Will close this issue as it's a duplicate for #623
Regards,


Country selection list is incomplete 4.1.0 + Postgresql 9.6

This is  a bug.

Prerequisites

[x] Are you running the latest version?
[x] Are you reporting to the correct repository?
(enso is made of many specialized packages: https://github.com/laravel-enso)
[ ] Did you check the documentation?
[ ] Did you perform a cursory search?
PHP 7.4, Postgresql 9.6, fresh install from git

Description

Country selection list is incomplete, even countries table is filled.

Steps to Reproduce

Create new Address
You will get this, where Réunion already selected:

Try to find Hungary -> no result, the list ends like this:

Expected behavior
Full list of all countries.
Actual behavior
As shown above.

Inspection inside the browser shows this:

So, the list ends with Åland Islands.

I checked the database query in debug mode and I found that, there is a pagination settings to 100. "paginate":"100"

/api/core/countries/options?customParams=%7B%7D&paginate=100&params=%7B%7D&pivotParams=%7B%7D&query=&searchMode=full&trackBy=id&value=184"

I just count the numbers of existing countries, there are 100 elements, so, bingo.

My question, is it a user settings of global one to paginate this list? In other words, could the user change this behaviour?

Same effect on the laravel-enso.com site, too.

Having the full option list in the front-end would pretty much cancel the need for server side load.
The select displays only the first 100 entries initially, but you can type in the searchbox anything and the server will return the records the match (first 100).
Or disable the server side for this input and send all the options when the form loads from the form builder.

Thanks, maybe it is only me, but for me this function seems a little bit inconsistent.
Here I started typing on my installation: united, expecting some results, but nothing appears.

Meanwhile it now works on your demo site.
My infrastructure is:
Internet ---- apache2 reverse proxy --- dedicated enso server with nginx  --- dedicated Postgresql database server

Ok, I got it.
The query is case sensitive and that is the reason I got no results when I started to type like this: united.
But when I started to type with Unite... I got the expected result.
As I checked the database server log (debug), I found this (only a snippet):
where ((\"name\"::text LIKE ''%fran%'') or (\"iso_3166_3\"::text LIKE ''%fran%''))

and the LIKE is definitely case sensitive.
Ok, it seems the "problem" solved like this: there is no problem, but the input is case sensitive in case of Postgresql backend.
I guess it is related to Eloquent which translate the query for specific databases.

Update this to suit your needs

Thanks, I missed that config option.


refactor(nodebuilder): moving service/ services to respective node sub-packages

Moves services into nodebuilder and makes the internal structs private, offering a public interface instead.
Closes #960
Packages:

[ ] Service
[ ] Share
[X] Header
[X] State

Not uploading all packages at once in case I'm not going in the right direction yet - so draft PR
Also, I took the branch from @renaynay's remote and pushed it to our remote here so that the diffs will be clear. #997 can then rebase off of that branch when things get merged into it.

Codecov Report

:exclamation: No coverage uploaded for pull request base (node-refactorings@2441857). Click here to learn what that means.
The diff coverage is n/a.

@@                 Coverage Diff                  @@
##             node-refactorings    #1056   +/-   ##
====================================================
  Coverage                     ?   61.54%
====================================================
  Files                        ?      149
  Lines                        ?     8522
  Branches                     ?        0
====================================================
  Hits                         ?     5245
  Misses                       ?     2817
  Partials                     ?      460

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

Hmm too many CI issues @distractedm1nd

All tests pass for me locally, let's see if something flakes


gerror add IgnoreStack and CodeIgnoreStack #2559

#2559
When not needed error stack. Can using the IgnoreStack feature lead to improved performance.

Codecov Report
Patch coverage: 100.00% and project coverage change: -0.02% :warning:

Comparison is base (e0e0043) 79.22% compared to head (ea2fafa) 79.21%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2811      +/-   ##
==========================================
- Coverage   79.22%   79.21%   -0.02%
==========================================
  Files         633      630       -3
  Lines       52242    51923     -319
==========================================
- Hits        41391    41131     -260
+ Misses       8808     8764      -44
+ Partials     2043     2028      -15

Flag
Coverage Δ

go-1.15-386
?

go-1.15-amd64
79.21% <100.00%> (+0.02%)
:arrow_up:

go-1.16-386
?

go-1.16-amd64
?

go-1.17-386
?

go-1.17-amd64
?

go-1.18-386
?

go-1.18-amd64
?

go-1.19-386
?

go-1.19-amd64
?

go-1.20-386
?

go-1.20-amd64
?

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

Files Changed
Coverage Δ

errors/gerror/gerror_api.go
100.00% <100.00%> (ø)

errors/gerror/gerror_api_code.go
96.87% <100.00%> (+0.36%)
:arrow_up:

... and 9 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@oldme-git 你好，感谢参与贡献，不过你的需求可以通过gerror.NewOption方法来实现：https://github.com/gogf/gf/blob/53e5a040734ea6c79085c233ce4ac25880f66c4c/errors/gerror/gerror_api_option.go#L21


Disabled stdlib dependent function invocation tests

Description:
As a part of stdlib migration(https://github.com/ballerina-platform/ballerina-lang/issues/21313), we had to disable the following function invocation tests which depend on stdlibs.
https://github.com/ballerina-platform/ballerina-lang/pull/25567/commits/3691d0c0e31750230d781ae874d7956d25ac9145

Enabled in https://github.com/ballerina-platform/ballerina-lang/commit/08638fc66a1fde0cd9a3bde39bd48033423981ec and https://github.com/ballerina-platform/ballerina-lang/commit/cfce4ee3734bb42eb7c92bb5ebe6d448f86ed4fd.


Installation failure due to Matrix.utils dependency

Hi there - I am looking forward to trying out this tool, but wasn't able to get it fully installed due to the Matrix.utils dependency, which now appears to officially be off CRAN (https://cran.r-project.org/web/packages/Matrix.utils/index.html). Running R 4.1.2, not sure if anyone else has come across this. It was just taken off 9 days ago it appears, so I am unable to install this dependency, and don't know to what extent Pando utilizes it

I've ran into the same issue. Installing Matrix.utils from Github (remotes::install_github('cvarrichio/Matrix.utils')) worked as a temporary fix, but it'd be great if the dependency could be replaced. From what I saw, its only use in Pando is Matrix.utils::aggregate.Matrix() in utils.R/aggregate_matrix(), so hopefully this should be feasible.

Gotcha, thanks, missed the github option when searching earlier. I guess I was also concerned to see that package removed from CRAN on account of persistent bugs, but hopefully this should work OK for now given the specific usage you pointed out.

Heyhey, I removed the Matrix.utils dependency in the newest version since the package is also not maintained anymore. Building on 4.0-4.2 should work again.


Statistics - separate thousands

Hi,
could you consider separating the numbers by thousands in the tables and elsewhere? tnx

DOne

It's not finished in vaccination center detail, but bigger change is needed there -> it will be included in another task


Fix/36878 next step when reimbursing

Details

The PR fixes a next step message prediction, when users use a payment variant with Expensify.
Fixed Issues

$ https://github.com/Expensify/App/issues/36878
PROPOSAL: N/A
Tests

Pre requirements:

Create/have 2 users at least. One of them is a member of workspaces. The 2nd one is an admin.
Create/have 2 paid workspaces via OD: Collect and Control. The feature does not work for a free workspace.
Use OD to manipulate with the workspaces.
The member: opens a workspace chat.
The member: requests money.
The admin/the member: clicks on a “Submit” button.
The admin: clicks on an “Approve” button.
The admin: clicks on a “Pay with Expensify” button.
Verify: a next step with a comment: “You have marked these expenses as paid.” and label is “Finished!“.

[x] Verify that no errors appear in the JS console

Offline tests

Same as "Tests".
QA Steps

Same as "Tests".

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/Expensify/App/assets/57314631/8d58969f-ab98-4e88-89a3-cf4697b7dda6

Android: mWeb Chrome

https://github.com/Expensify/App/assets/57314631/1acc395a-601c-4494-b63a-b4facb3438e6

iOS: Native

https://github.com/Expensify/App/assets/57314631/f5b33d9d-71e8-4bc7-a52f-794430b5f6ec

iOS: mWeb Safari

https://github.com/Expensify/App/assets/57314631/4cab0c86-2346-444e-b646-e9a112c9aa1d

MacOS: Chrome / Safari

https://github.com/Expensify/App/assets/57314631/95ee8f25-ff62-4748-895d-5c490647b224

MacOS: Desktop

https://github.com/Expensify/App/assets/57314631/79225455-6012-4ab0-846e-a11bdddbe8b3

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If the PR modifies the form input styles:

[ ] I verified that all the inputs inside a form are aligned with each other.
[ ] I added Design label so the design team can review the changes.

[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/mountiny in version: 1.4.44-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/puneetlath in version: 1.4.44-13 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


[5.1] vendor:publish is mangling files

php artisan vendor:publish --provider="MyVendor\MyPackage\Providers\MyProvider" --tag="public" --force
The provider referenced above contains this line in the boot() method:
       $this->publishes([             __DIR__.'/../../public' => public_path('packages/myvendor/mypackage'),         ], 'public');
I have verified that all files in the vendor's public folder contain what is expected. When I run the publish command at the top of this issue, the resulting files in my app's /public/packages/myvendor/mypackage folder are randomly truncated.
For example, one of my assets is a gulped javascript file that contains moment.js. This javascript file is perfectly fine in my package in the vendor folder. After vendor:publish, the published version of this javascript file starts in the middle of the text for the warning that points to this issue -- https://github.com/moment/moment/issues/1407. In this case, vendor:publish appears to have trashed the first 80% of this file.
I can't imagine what could possibly cause this.
Is it possible that a recent framework update could be causing this? I frequently update this package and its assets, and I have never come across this behavior before.

A composer update pulled in a new commit from Flysystem. The issue appears to be resolved now. Maybe it was a composer bug that mangled a Flysystem file?


Compatibility with jupyterhub 2.0-2.2

Fixes call to shutdown_cancel_tasks for jupyterhub < 2.3, and remove suppression of error during cleanup.
Adds test matrix to cover supported JupyterHubs
I think this might fix some of the failures in https://github.com/jupyterhub/dockerspawner/pull/476 which for jupyterhub 2.2 are caused by a proxy still being running in tests after the first.

LGTM! 🚀
Thank you for this PR @minrk


fix: TestMaxBlockSize flake

Closes https://github.com/celestiaorg/celestia-app/issues/3086 by reverting https://github.com/celestiaorg/celestia-app/pull/3075
Re-opens https://github.com/celestiaorg/celestia-app/issues/3066

None of these CodeRabbit comments are helpful b/c go mod tidy results in no changes.

So was app version returning 0 or 2?

Closing this for now. We're hoping that the flake resolves after we bump to a celestia-core release with https://github.com/celestiaorg/celestia-core/pull/1202 and another fix for versions (@cmwaters will create an issue for).


Output support for 'fxcored query block'

The fxcored query block command lacks of output format. E.g. default it prints JSON, not YAML, and there is no option to change that.
Related: GH-23.
$ fxcored version
master-cecc3f6a56867c8d3d4bc2dffca2c4dd3258e07d

fixed on 9ba99411061a4926c0892d07aabb4bbb2ae62b23


[Joint State Broadcaster] Add option to publish joint states to local topics

This PR is related/inspired by #217 and #218.
Changes:

Use local topics for publishing joint states
Parameterize local namespace topics.
Unify test structure.
Use get_node() instead of node_ everywhere.

What do you think?

Can't this be handled with a simple remap in the launchfile? Is just as much configuration for a user as modifying a parameter.

Not really. What is you have multiple JointStateBroadcasters running? Then, which one would be remapped?

@destogl this looks good to me, feel free to merge after resolving the conflicts


[Hydrogen docs]: Update guidance for ShopifyProvider component

This PR:

Updates the guidance associated with the ShopifyProvider component
Relates to https://github.com/Shopify/hydrogen/pull/597, https://github.com/Shopify/hydrogen/pull/573, https://github.com/Shopify/shopify-dev/pull/15936 and https://github.com/Shopify/shopify-dev/issues/10171

@jplhomer @blittle I've made some updates based on Bret's feedback. Would you mind taking another look at this PR? Thanks in advance!


cannot set in targetSdk 33

me update targetSdk 33
error :
java.lang.IllegalStateException: Not allowed to start service Intent { act=STOP_ALARM ...
E/Zygote: isWhitelistProcess - Process is Whitelisted
E/Zygote: accessInfo : 1
3689-3689/? I/.best.deskcloc: Late-enabling -Xcheck:jni
3689-3689/com.best.deskclock W/.best.deskcloc: Accessing hidden field Landroid/os/Trace;->TRACE_TAG_APP:J (light greylist, reflection)
3689-3689/com.best.deskclock W/.best.deskcloc: Accessing hidden method Landroid/os/Trace;->isTagEnabled(J)Z (light greylist, reflection)
3689-3689/com.best.deskclock I/DigitalWidgetProvider: onReceive: Intent { act=android.app.action.NEXT_ALARM_CLOCK_CHANGED flg=0x21000010 cmp=com.best.deskclock/com.best.alarmclock.DigitalAppWidgetProvider }

if you touch any value between target sdk, min sdk, the alarm does not sound it is a problem with the aosp clock source.

Done


Lines connecting label data with actual calendar are not matching

Hi! Firstly I want to thank you for your work.
I installed today your pap, but lines constraining resource on the left seem a little bit off (not matching calendar grid):

I have plain app with only your lib configured:
import React from 'react';
import './App.css';
import {Scheduler, SchedulerData} from<EMAIL_ADDRESS>import<EMAIL_ADDRESS>
function App() {
  return (
      <section>
        <Scheduler
            data={mockedSchedulerData}
            isLoading={false}
        />
      </section>
  );
}

const mockedSchedulerData: SchedulerData = [
  {
    id: "070ac5b5-8369-4cd2-8ba2-0a209130cc60",
    label: {
      icon: "https://picsum.photos/24",
      title: "Joe Doe",
      subtitle: "Frontend Developer"
    },
    data: [
      {
        id: "8b71a8a5-33dd-4fc8-9caa-b4a584ba3762",
        startDate: new Date("2023-04-13T15:31:24.272Z"),
        endDate: new Date("2023-08-28T10:28:22.649Z"),
        occupancy: 3600,
        title: "Project A",
        subtitle: "Subtitle A",
        description: "array indexing Salad West Account",
        bgColor: "rgb(254,165,177)"
      },
      {
        id: "22fbe237-6344-4c8e-affb-64a1750f33bd",
        startDate: new Date("2023-10-07T08:16:31.123Z"),
        endDate: new Date("2023-11-15T21:55:23.582Z"),
        occupancy: 2852,
        title: "Project B",
        subtitle: "Subtitle B",
        description: "Tuna Home pascal IP drive",
        bgColor: "rgb(254,165,177)"
      },
      {
        id: "3601c1cd-f4b5-46bc-8564-8c983919e3f5",
        startDate: new Date("2023-03-30T22:25:14.377Z"),
        endDate: new Date("2023-09-01T07:20:50.526Z"),
        occupancy: 1800,
        title: "Project C",
        subtitle: "Subtitle C",
        bgColor: "rgb(254,165,177)"
      },
      {
        id: "b088e4ac-9911-426f-aef3-843d75e714c2",
        startDate: new Date("2023-10-28T10:08:22.986Z"),
        endDate: new Date("2023-10-30T12:30:30.150Z"),
        occupancy: 11111,
        title: "Project D",
        subtitle: "Subtitle D",
        description: "Garden heavy an software Metal",
        bgColor: "rgb(254,165,177)"
      }
    ]
  },
];

export default App;

"SchedulerData" is copied from your example.
Also buttons on the upper right to manipulate the view are not aligned to screen by default.
How to fix it?
Sincerely,
Ignacy

Hi @Lepruz,
thanks for raising the issue.
We've created a fix for scheduler styling problem: https://github.com/Bitnoise/react-scheduler/pull/54

Hi @Lepruz
FYI: new version of scheduler was released with fix for this issue.
Please update the version and see if it was fixed :)


can I use ajax for get the sources?

I want to dynamic get the source with ajax, can I do that?
thanks!

You mean suggestions from ajax?

Yes!

Currently, it does not have such option.
I will add this option and update the code as soon as possible.

Thanks a lot, that's will be great!

Added suggestions through ajax option, check readme and example files for reference.


refactor(font): revamp FontFamily and Text APIs

The objective here is primarily to remove ambiguity, and secondarily to reduce verbosity.

At first I wasnt sure if we should change the names of italicized and monospaced but I think it's better just because it's shorter.

"italic" is also the more common form (by an order of magnitude according to google) and more consistent with properties like "bold" (vs "bolded"). "monospaced" is trickier though, because it doesn't have an adjective form as far as I'm aware, and is soooo long, but can also be ambiguous outside the context of a font, so I've left it in the full form in some places.


Nested SoftDelete not detected since 0.7.5

Larastan Version: ^0.7.5
--level used: 8
Pull request with failing test: #838

Description
Larastan does not detect a nested SoftDeletes trait after updating from 0.7.4 to 0.7.5 or higher.
Laravel code where the issue was found
class User extends Authenticatable
{
    public function group(): BelongsTo
    {
        return $this->belongsTo(Group::class)->withTrashed(); // Call to an undefined method Illuminate\\Database\\Eloquent\\Relations\\BelongsTo<App\\Group, App\\User>::withoutTrashed().
    }
}

class Group extends Model
{
    use NestedSoftDeletes;
}

trait NestedSoftDeletes
{
    use SoftDeletes;
}

Hi,
We are using the PHPStan's MethodReflection method hasTraitUse here to determine if a class uses the given trait. Looks like this needs to recursive. But I don't know it's possible.
Please feel free to create a PR that fixes it in the code I pointed out. Also you can ask in PHPStan if you need help with how to solve this.

@ondrejmirtes Could you help us whether hasTraitUse is recursive?

Try using ClassReflection::getTraits() instead, or send a fix to PHPStan :)

@canvural I've added a function in #838 that collects nested traits as the PHPStan's collectTraits function is private. Let me know if this is acceptable.

@IAmRGroot Did you try ClassReflection::getTraits() first as @ondrejmirtes suggested?

@canvural Yes!
ClassReflection::getTraits gives  App\Group: {"App\\Traits\\NestedSoftDeletes":{}}
(new) collectTraitNames gives App\Group: ["App\\Traits\\NestedSoftDeletes","Illuminate\\Database\\Eloquent\\SoftDeletes"]
See the ClassReflection::collectTraits private function. One could send a PR to make it public, but until then you would need to use your own function.

I'm not sure if it's the best thing to duplicate collectTraits method in Larastan. @ondrejmirtes ClassReflection::getTraits() does not look like it's recursive, as @IAmRGroot pointed out. Would you be fine with a PR that is making collectTraits public, or making hasTraitUse use collectTraits, or do you have any other idea for a solution?

I'm fine with changing getTraits() to be recursive, as getInterfaces already are...

@IAmRGroot You can send a PR to PHPStan repo first that changes getTraits to be recursive. Then we can use it here, to solve the issue.

Is it Done now?

PHPStan 0.12.90 has just been released - you can now call ClassReflection::getTraits(true) and get traits recursively.

Closed with https://github.com/nunomaduro/larastan/pull/838


Add Support for Non-Letter/Number Characters ($,@,*) in Passwords

This would greatly benefit the security of the mod, since passwords would be more secure.

The current keyboard in beat saber quest doesn't support those characters.

To be more secure using diceware is probably a better approach anyway: https://diceware.dmuth.org


Unable to dynamically change ol.source.ImageStatic URL and imageExtent

How do I dynamically change the url and imageExtent for ol.source.ImageStatic ?

You don't. If you have multiple static images, create a source for each. You can dynamically change the source of the layer:
imageLayer.setSource(staticSource);

That is what I am doing at the moment. I have a grid and when user selects
a record from grid it loads the static image from the selected record into
the map. Problem is that memory builds up as you move through the grid.
Don't you need to destroy the existing source before creating a new source?
On Wed, 21 Oct 2015, 14:03 Andreas Hocevar<EMAIL_ADDRESS>wrote:

You don't. If you have multiple static images, create a source for each.
You can dynamically change the source of the layer:
imageLayer.setSource(staticSource);
—
Reply to this email directly or view it on GitHub
https://github.com/openlayers/ol3/issues/4300#issuecomment-149869540.

Problem is that memory builds up as you move through the grid. Don't you need to destroy the existing source before creating a new source?

As long as you don't reference the source anywhere when it's not used any more, the garbage collector should remove it from memory eventually.


Cannot except PanicException in python

Bug Description
Hi thanks for the helpful crate! I need to catch rust panics in python. Example code:
def test_catch_rust_panic():
    caught = False
    try:
        deliberate_panic()
    except Exception as e:
        assert 'deliberate panic' in str(e)
        caught = True
    assert caught

pub fn deliberate_panic() -> () { panic!("this is deliberate panic") }

However it does not work, error log:
    def deliberate_panic(self) -> None:
>       self.inner.deliberate_panic()
E       pyo3_runtime.PanicException: this is deliberate panic

../../rust_wrapper/ineq.py:75: PanicException

Yes I can change it to BaseException but that will capture a lot of other exceptions which I should not catch.
I have tried to import pyo3_runtime but seems that package does not exist at all.
Steps to Reproduce
see above
Backtrace
No response
Your operating system and version
macos
Your Python version (python --version)
3.9
Your Rust version (rustc --version)
1.66.1
Your PyO3 version
0.17.3
How did you install python? Did you use a virtualenv?
pip
Additional Info
No response

Due to ABI issues/static linkage, every PyO3-based extension has its own PanicException. Meaning that to explicitly catch it, you need to explicitly make it part of your Python API, e.g. by calling
module.add("PanicException", <pyo3::panic::PanicException as pyo3::PyTypeInfo>::type_object(py))?;

on a suitable module: &pyo3::types::PyModule and import from there on the Python side.

Yes I can change it to BaseException but that will capture a lot of other exceptions which I should not catch.

This is also true for PanicException - you shouldn't catch it, you should fix the bugs in your code.

latest

As an aside, please try to be specific here. At the moment, we all know what we are talking about. But the issue might be read by other people in the future where "latest" does not apply any more.

@adamreichold Thank you! So it is possible to catch all such PanicExceptions in different modules?

@mejrs This is also true for PanicException - you shouldn't catch it, you should fix the bugs in your code.

I see. Indeed I am using it for some machine learning data generation, and bugs are acceptable as long as it is not frequent - I just throw away that part of data and restart randomly.

latest

changed :)

@adamreichold Thank you! So it is possible to catch all such PanicExceptions in different modules?

I don't think so. As each Python extension built using PyO3 has a different PanicException (meaning different type objects), you need to import and catch all of them. (And I think you are out of luck for extensions which do not export their exception type at all.)
One workaround that might work is to catch BaseException and doing something questionable like
case BaseException as e:
  if not "PanicException" in str(type(e)):
    raise

Thank you!

As this is now resolved and there's no intention to change PyO3's design here, I'm going to close this issue. Thanks both!


Provide IPC mechanism between a web embedded and an extension

Ideally we would enable a direct communication channel between the embedded and a specific extension without significantly expanding the breadth of the VS Code API.
One idea would be to use MessagePort instances: the embedder could provide, at workbench ctor time, a collection of MessagePort instances which would be given to specific extensions at their activation time.
Proposal implementation: https://github.com/microsoft/vscode/pull/138054

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


Timeline section

delete extra images and put social media links on footer

Thanks for including me on the review process @hanlinc27
There are some suggestions I would like to be integrated if possible in this merge
Will reply more on tmrow on this (mostly related to best practices around keeping hard-coded values away from production files, and trying to find a balance between hardcode and dynamic variables)

Thanks for including me on the review process @hanlinc27
There are some suggestions I would like to be integrated if possible in this merge
Will reply more on tmrow on this (mostly related to best practices around keeping hard-coded values away from production files, and trying to find a balance between hardcode and dynamic variables)

What about config files?

I think we can tackle the refactoring (dynamic values etc) separately and can merge this pull request when others have already approved


Failed to ingest events

Failed to ingest events .catch() Timeout
Machine ID: f21ad435-30ac-41b0-8058-dff6801e528b

[!NOTE]
Book a call

key
value

id
YIZuvpeCTcQLtw5Zdh2bp

url
http://localhost:2702/

wifi
4g

cpu
8

mem
8

gpu
ANGLE (Apple, ANGLE Metal Renderer: Apple M1 Pro, Unspecified Version)

time
1726321906661

items
8942

bytes
15138

77dad7df-041b-4a6e-a171-18df209852f71726256479714 (localhost:52921 - true)
77dad7df-041b-4a6e-a171-18df209852f71726256479714 (<IP_ADDRESS>:52921 - true)

Hi @envindavsorg , we've just deployed a fix for you. Can you try running the setup script npx million@latest. Once it's done, please restart your VSCode/Cursor. Let me know if that works for you!

Everything works !

Awesome!


fix: ime

I dont think its a good idea to use a personal fork.
if you wanna fix it only in tailchat, use pnpm patch is a more better idea.
https://pnpm.io/cli/patch
BTW, can you provider more context about this PR?

I don't know that. Thanks

fine


No longer works with D9

The robot constantly shows as cleaning even when it's docked. The Robot also doesn't respond to requests nor is the data from the robot visible.

This was never intended to work with the D9.


SQLAlchemy dialect uses deprecated dbapi() method

Minor issue, but it appears that this library is using a deprecated method from SQLAlchemy:
SADeprecationWarning: The dbapi() classmethod on dialect classes has been renamed to import_dbapi().
Implement an import_dbapi() classmethod directly on class <class 'databricks.sqlalchemy.base.DatabricksDialect'>
to remove this warning; the old .dbapi() classmethod may be maintained for backwards compatibility.
  engine = create_engine(

Installed:

sqlalchemy 2.0.23
databricks-sql-connector 3.0.0

Good catch. We'll push a change fixing this in the next point release.

I have 2.0.25 which was cut after this was merged but the problem persists.
SADeprecationWarning: The dbapi() classmethod on dialect classes has been renamed to import_dbapi().  Implement an import_dbapi() classmethod directly on class <class 'pyathena.sqlalchemy.rest.AthenaRestDialect'> to remove this warning; the old .dbapi() classmethod may be maintained for backwards compatibility.


Reference different certificate for Cloudfront Production

The assets Cloudfront distribution was referencing the (expired)
cert in us-east-1, for *.publishing.service.gov.uk - which had
been manually created, thus not auto-renewed.
https://us-east-1.console.aws.amazon.com/acm/home?region=us-east-1#/certificates/108d0f34-cf9f-4e04-81a2-231e42824b08
Instead we should be using the certificate that's created and
managed in Amazon Certificate Manager (ACM), in the eu-west-1
region:
https://eu-west-1.console.aws.amazon.com/acm/home?region=eu-west-1#/certificates/063a18cc-d4bf-4eaf-b1c1-41e86bbdd69d
That cert covers a number of domains, including the necessary
*.publishing.service.gov.uk.
However, we can't simply change the aws_cloudfront_certificate
from us-east-1 to eu-west-2, as we still need the www Cloudfront
distribution to work, and that certificate is in us-east-1!
Therefore we've had to create multiple variables, to support
the multiple AWS regions where our certificates live. NB: the
default values have been applied so that the plan continues to
work on Integration & Staging.
Fixing the certificate configs will allow us to apply the changes
from #1691 to Production.
Successful plan on production: https://deploy.integration.publishing.service.gov.uk/job/Deploy_Terraform_GOVUK_AWS/4756/console
Successful plan on integration:
https://deploy.integration.publishing.service.gov.uk/job/Deploy_Terraform_GOVUK_AWS/4758/console
Trello: https://trello.com/c/PMsAXSoT/3029-review-govuks-usage-of-node-2

Whoops sorry didn't spot the region there. Has to be us-east-1 for CloudFront because that's where its control plane runs :/


Adding the license information to the package.json

Tools like VersionEye are using this information for license compliance analysis.

Duplicate of #34.
+1 though.


image histogram UI

Ideas for creating a histogram UI to adjust vmin-vmax of image graphic similar to pyqtgraph HistogramLUTWidget

[ ] When used, the viewport in the Subplot needs to be split so that a thin rectangular "sub-viewport" can be used for the histogram LUT widget.
[ ] Colorbar next to histogram

more ideas on the HistogramLUTWidget itself:

Disable panning in x-direction, only allow y-axis panning

Superseded by a more recent issue


Membership expiration date incoherence

As you could see, expiration of my identity membership will expire 13 july 2015:

But, in the WoT graphic, expiration of certifications is between 20 july and 6 august 2015.

I'm not sure this is a bug. The membership expiration is distinct from the certifications expiration.

It is not a bug. It is your membership expiration, as ucoin understand it, that is, your decision to be part of the community. This status need to be renewed.
It is separate from the certifications.


Uint8List

The package no longer supports Uint8List data

No, it still supports Uint8List.
There was a discussion on how we deal with const on PdfViewer and PdfDocumentRef on #42.
And, now I just keep asset, uri, and file on PdfViewer constructor.
Other sources can be opened using PdfDocumentRef variations. See #49 for Uint8List case.


WIP: Remove black; Add Mypy and flake8 plugins

Black is great tool for python code formatting, however it doesn't follow all the PEP rules, so I suggest we don't use it.
Instead we use some of flake8 plugins.
And to solve problem with us forgetting to set up types correctly I suggest we use Mypy.

Added
 flake8-commas = {version = "^2.0.0", optional = true}
 flake8-comprehensions = {version = "^3.6.1", optional = true}
 flake8-bugbear = {version = "^21.4.3", optional = true}
 flake8-eradicate = {version = "^1.1.0", optional = true}
 flake8-broken-line = {version = "^0.3.0", optional = true}
 flake8-return = {version = "^1.1.3", optional = true}
 flake8-simplify = {version = "^0.14.1", optional = true}
 flake8-class-attributes-order = {version = "^0.1.2", optional = true}
 mypy = {version = "^0.910", optional = true}

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@e0ddddc). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #27   +/-   ##
=========================================
  Coverage          ?   56.49%
=========================================
  Files             ?       41
  Lines             ?     1754
  Branches          ?        0
=========================================
  Hits              ?      991
  Misses            ?      763
  Partials          ?        0

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 e0ddddc...f3cd25a. Read the comment docs.


Update num-rational dependency

The ecosystem seems to be moving to v2, and I'd like to avoid duplication of this crate.
(Would be nice if this could be accompanied by a version bump / publish)
r? @bvssvni

Thanks! Merging.


Old Font Functions should switch to Text Mode

When trying: \sf{for $a < b$ and $c < d$.} an error is thrown as $ is used within math mode. When using the \sf and other old font functions, the mode should be switched to text.

As you probably already figured out, \sf (and old font commands) don't switch to text mode in LaTeX.

Indeed I did haha.


Blacklisted sites to consider adding

Here's my list of blacklisted sites to consider blocking by default.
Zoro.to

I clicked around on a couple pages, but I didn't see where the comments are. Do you have an example page?

Closing this for now, but let me know if you have more details and I'll check into it.


Setting up ElastAlert on Windows 7

I am trying to set elastalert on my windows 7 laptop where my development ELK stack is working. I have done the below but I am not able to get a simple test rule to fire.
Steps Followed

git clone https://github.com/Yelp/elastalert.git -- downloaded to local folder

pip install -r requirements.txt
(the above step completed fine except for error for supervisor package complaining about version.My python version is 3.4.2).

renamed config.yam.example to config.yaml and changed the below settings
rules_folder:myrules
run_every(minutes):1
es_host: <IP_ADDRESS>
es_port: 9200 (my http port)
use_ssl: False

created a sample frequency type rule and tried running the below command
" elastalert-test-rule myrules/myrule.yaml " . When executed the command gave the below error
ImportError: No module named 'alerts'

Please help , also if there Is any guide to set this up on windows that would be helpful.
Thanks

I don't see why it shouldn't work on Windows, but I have not tested it. I don't think there's anything specific to Unix-like environments.
However, it does require Python 2 to work.

I try the same thin on win 7.  i installed python-2.7.11 and add python as Environment variable to start with the python command. testrule didn't work on my setup, too. But i tried to start my rule with:
python -m elastalert.elastalert --verbose --rule myrule.yaml
it seemed to work but then i got this error... i think it's a problem of missing packages, what do you think @Qmando?
INFO:elastalert:Starting up
ERROR:root:Traceback (most recent call last):
File "C:\Users\pat\elastalert\elastalert\elastalert.py", line 746, in run_all_
rules
num_matches = self.run_rule(rule, endtime, self.starttime)
File "C:\Users\pat\elastalert\elastalert\elastalert.py", line 539, in run_rule
if not self.run_query(rule, rule['starttime'], endtime):

File "C:\Users\pat\elastalert\elastalert\elastalert.py", line 419, in run_quer
y
data = self.get_hits(rule, start, end, index)
File "C:\Users\pat\elastalert\elastalert\elastalert.py", line 309, in get_hits
elastalert_logger.info("Queried rule %s from %s to %s: %s hits" % (rule['nam

e'], pretty_ts(starttime, lt), pretty_ts(endtime, lt), len(hits)))
File "elastalert\util.py", line 106, in pretty_ts
return dt.strftime('%Y-%m-%d %H:%M %Z')
File "C:\Python27\lib\site-packages\dateutil\tz.py", line 41, in inner_func
return myfunc(*args, **kwargs).encode()
UnicodeDecodeError: 'ascii' codec can't decode byte 0xe4 in position 11: ordinal
not in range(128)
ERROR:root:Uncaught exception running rule Example rule: 'ascii' codec can't dec
ode byte 0xe4 in position 11: ordinal not in range(128)
INFO:elastalert:Rule Example rule disabled
INFO:elastalert:Sleeping for 59 seconds

ok fixed it by changing from CET to UTC on my computer, elastalert runs. on my german windows it's called Mitteleuropäische Zeit and i think phython has a problem with the 'ä'. is there a way to keep my timezone and running elastalert?

The problem is in the dateutil library. You can add this patch to fix the problem.
https://github.com/Yelp/elastalert/issues/471#issuecomment-209657248


UISegmentedControl stops responding after switching segment

When I switch segment with swipe, it's ok, but when I switch segment on segmented control, then it stops responding and not changing at all.
EDIT:
Ok, I found a solution, but I don't know why it was the reason.
Problem disappeared when I removed these method:

(void)moveToViewControllerAtIndex:(NSUInteger)index

Issue is now solved!


Support TLS

Add support for TLS without need for a proxy.
(tiny-ssl-reverse-proxy works well, but complicates things like handling origins other than sameorigin)

Looks good but please run go fmt.

Done. Apologies -- this is my very first toe dipping in go.


Httpserver demo benchmark then stack overflow

Listening on http://<IP_ADDRESS>:8080
Unexpected mark stack overflow
Aborted (core dumped)
The source code is just the documentation:
require "http/server"

server = HTTP::Server.new(8080) do |request|
    HTTP::Response.ok "text/plain", "Hello world! The time is #{Time.now}"
end

puts "Listening on http://<IP_ADDRESS>:8080"
server.listen

Linux tuned
ulimit -SHn 65535
vagrant@vagrant-ubuntu-vivid-64:~/workspace/oss/wrk$ wrk -c 10000 -d 10 http://<IP_ADDRESS>:8080
Running 10s test @ http://<IP_ADDRESS>:8080
  2 threads and 10000 connections
  Thread Stats   Avg      Stdev     Max   +/- Stdev
    Latency     0.00us    0.00us   0.00us    -nan%
    Req/Sec     0.00      0.00     0.00      -nan%
  0 requests in 10.14s, 0.00B read
  Socket errors: connect 0, read 4943, write 529811, timeout 0
Requests/sec:      0.00
Transfer/sec:       0.00B

It is to many connections you specify for work -c 10000. Try -c 100

I know it . but the Unexpected mark stack overflow should not happened.

I cannot reproduce this consistently. It only happened once to me in a VirtualBox VM.
Can you share some of your hardware or VM specs? Which Crystal version are you using? Are you compiling in release mode?
Any other hint you can provide to make it fail more often will be really helpful to solve the issue.
I found this topic that seems to be related: http://www.hpl.hp.com/hosted/linux/mail-archives/gc/2005-September/000989.html

Yes, i am using VirtualBox VM with Ubuntu vivid. the crystal version 0.7.3 , installed via deb repository.

I still cannot make it fail. Do you always get the error every time you run the test?
How much memory and how many cores do you have assigned to your VM?

3GB RAM, 2CPU core.
vagrant@vagrant-ubuntu-vivid-64:~/workspace/ops/books$ cat /proc/cpuinfo
processor	: 0
vendor_id	: GenuineIntel
cpu family	: 6
model		: 58
model name	: Intel(R) Core(TM) i5-3470 CPU @ 3.20GHz
stepping	: 9
microcode	: 0x19
cpu MHz		: 3210.067
cache size	: 6144 KB
physical id	: 0
siblings	: 2
core id		: 0
cpu cores	: 2
apicid		: 0
initial apicid	: 0
fpu		: yes
fpu_exception	: yes
cpuid level	: 5
wp		: yes
flags		: fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx rdtscp lm constant_tsc rep_good nopl pni ssse3 lahf_lm
bugs		:
bogomips	: 6420.13
clflush size	: 64
cache_alignment	: 64
address sizes	: 36 bits physical, 48 bits virtual
power management:

processor	: 1
vendor_id	: GenuineIntel
cpu family	: 6
model		: 58
model name	: Intel(R) Core(TM) i5-3470 CPU @ 3.20GHz
stepping	: 9
microcode	: 0x19
cpu MHz		: 3210.067
cache size	: 6144 KB
physical id	: 0
siblings	: 2
core id		: 1
cpu cores	: 2
apicid		: 1
initial apicid	: 1
fpu		: yes
fpu_exception	: yes
cpuid level	: 5
wp		: yes
flags		: fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx rdtscp lm constant_tsc rep_good nopl pni ssse3 lahf_lm
bugs		:
bogomips	: 6420.13
clflush size	: 64
cache_alignment	: 64
address sizes	: 36 bits physical, 48 bits virtual
power management:

Can we using GDB to debug coredump file?

Core was generated by `./httpserver'.
Program terminated with signal SIGABRT, Aborted.
#0  0x00007f360059c267 in __GI_raise (sig=sig@entry=6) at ../sysdeps/unix/sysv/linux/raise.c:55
55	../sysdeps/unix/sysv/linux/raise.c: No such file or directory.
(gdb) core
No core file now.
(gdb) core core
[New LWP 8605]
[New LWP 8606]
Core was generated by `./httpserver'.
Program terminated with signal SIGABRT, Aborted.
#0  0x00007f360059c267 in __GI_raise (sig=sig@entry=6) at ../sysdeps/unix/sysv/linux/raise.c:55
55	../sysdeps/unix/sysv/linux/raise.c: No such file or directory.

@netroby can you check if f54463d fixes the issue for you?