Android - Size of the symbols are not same

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action performed

Open the chat
Go to chatting
Type symbol from the device keyboared

Expected results:
Symbols size should be same
Actual results :
Symbols size aren't same
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native

Version Number: 1.2.47-0
Reproducible in staging?: y
Reproducible in production?: y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/210275709-693f8655-53e2-48aa-94ee-bdc0f7bc2950.mp4
https://user-images.githubusercontent.com/43996225/210275715-916a2aa2-0d98-4c26-9f15-fca26ad7d7cf.mp4
Expensify/Expensify Issue URL:
Issue reported by: @ekramkhan66
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1672216294616209
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01e028289600d719d6
Upwork Job ID:<PHONE_NUMBER>625341440
Last Price Increase: 2023-01-12

I can take a look!

FYI this happens on web too:

I don't think this is a bug, technically those symbols are part of the emoji unicode set: https://www.unicode.org/Public/15.0.0/ucd/emoji/emoji-data.txt

Made a comment on Slack thread here: https://expensify.slack.com/archives/C049HHMV9SM/p1673904568468629?thread_ts=1672216294.616209&cid=C049HHMV9SM
But to put it another way the Extended_Pictographic matching set we're using here ensures we match emojis with a picture component. The spade, copyright symbols, etc all fall within that category according to Unicode.

Search for # All omitted code points have Extended_Pictographic=No in here https://www.unicode.org/Public/15.0.0/ucd/emoji/emoji-data.txt


Upgrade Prisma to version 3

Updating Prisma to v3 will resolve the following issues our engineers had encountered:

https://github.com/prisma/prisma/issues/11356
Database doesn't get populated (we assume it's related to Prisma)

How to upgrade to version 3

Prisma is good.

https://github.com/garageScript/c0d3-app/pull/2026 Version 4 is out as well
https://www.prisma.io/docs/guides/upgrade-guides/upgrading-versions/upgrading-to-prisma-4


Documenting C++ Pinocchio

We need to start to a general framework to produce the documentation of Pinocchio. Two documentations would be useful: one for the C++ API; a second one for the Python interface.
I open the discussion here about the C++ documentation. The best candidate for documentation is Doxygen. Then main issue is to hide the CRTP complexity in the documentation. Right now, I consider that the Doxygen HTML is not usable by novice.
I take a look at Eigen documentation.
http://eigen.tuxfamily.org/dox/index.html
The documentation is organized by "modules". See for example the modules "QR", "LLT", etc for the dense solvers of Eigen:
http://eigen.tuxfamily.org/dox/group__DenseLinearSolvers__Reference.html
In addition, the modules are wrapped in a tutorial-style chapter organization. Finally, the class are listed (while hiding internal classes).
I like the organization by modules. We could have the following modules:

spatial
multibody model
joints
algorithms
python interface  ===> maybe not to document

I think that it would also be important to accurately select which C++ objects, types, classes and namespaces we want to expose to the documentation. If we reason module by module, it should be pretty evident to chose which to hide.
The bad point is that I do not understand (yet) how the documentation of Eigen is generated. They used advance doxygen features that I do not fully catch.
From the exposition in the doc of the main classes organized by modules, we can complete the doc with presentation pages, tutorials, etc.

Stating the obvious, I do not think the documentation is inside the header files.

:+1: for a documentation by modules.
Is this an ongoing process ?

Yes somehow. A template of the new doxygen is in, which wait for completion. We did not share the documentation work yet.

Doxygen uses the C-preprocessor of CPP files, so the section with:
#ifdef WITH_HPP_FCL
...
#endif

are hidden in the doc.
To enable them, either deactivate the preprocessing via ENABLE_PREPROCESSING   = NO or use PREDEFINED = WITH_HPP_FCL to define the right preprocessor variable.
For further reading, see https://www.stack.nl/~dimitri/doxygen/manual/preprocessing.html


[Enhancement] Add minReadableVersion to replica info to help debug

Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.1
[x] 3.0
[x] 2.5
[ ] 2.4

[FE PR Coverage Check]
:heart_eyes: pass : 2 / 2 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/LocalTablet.java
2
2
100.00%
[]

@Mergifyio backport branch-3.1

@Mergifyio backport branch-3.0

@Mergifyio backport branch-2.5


Game Config

Each task should be encoded in a module, all of them placed on a single folder.
There must be a module that manages all task logic, mostly providing a single function tasks/0 that returns the list of available tasks in the system.
There must be a task behaviour defining callbacks for

retrieving the task spec (name, description, examples?)
testing user input

xref failed with exit status: 1


'IdentityRole.cs' does not set 'NormalizedName' when using constructor 'Role(string roleName)'

In IdentityRole.cs the following code:
        /// <summary>
        /// Initializes a new instance of <see cref="IdentityRole"/>.
        /// </summary>
        /// <param name="roleName">The role name.</param>
        /// <remarks>
        /// The Id property is initialized to form a new GUID string value.
        /// </remarks>
        public IdentityRole(string roleName) : this()
        {
            Name = roleName;
        }
```

should be:

```
        /// <summary>
        /// Initializes a new instance of <see cref="IdentityRole"/>.
        /// </summary>
        /// <param name="roleName">The role name.</param>
        /// <remarks>
        /// The Id property is initialized to form a new GUID string value.
        /// </remarks>
        public IdentityRole(string roleName) : this()
        {
            Name = roleName;
            NormalizedName = roleName.ToUpperInvariant();
        }
```

This is expected, the POCO isn't responsible for any of that manipulation, the managers are responsible for setting the normalized columns.


Check for UnauthorizedAccessException when doing the Setup or Uninstall

And display a human friendly message.
Also, handle the Uninstall exceptions. (They weren't handled)

Thx


Multi monitor support

Hey, first of all, fantastic work! I love this software. It's absolutely amazing.
I have encountered a couple of issues with multi-monitor support.
My setup is as follows:

2x traditional monitors
1x LG C2

I have enabled and run through the setup wizard to enable multi-monitor topology. Through using the software I have noticed some issues with the power state of the TV. For better display management I'm using Display Fusion to enable/disable different sub-sets of monitors+TV. I have a config for only the two monitors being enabled and one for just the TV.
I have these two options enabled:

Some scenarios:

TV is turned off. I have the two monitor config loaded. I switch my config to just TV. This successfully and correctly manages the TV's power state. For example, the TV powers on when I switch my profile just to use the TV, and vice versa when I switch back the TV turns off. Brilliant.
TV is turned on but I have a different input on or an app running. I have the two monitor config loaded. I switch my config to just TV. This doesn't change the TV's power state. Would it be possible to change the input on the TV only if it is enabled in monitor settings under Windows? I can see in the logs that the topology is changed, i.e. it is correctly detected.
TV is turned on but I have a different input on or an app running. I turn on my PC with the two monitors config. This turns my TV's input to PC. This is fine as I have the second checkbox under Miscellaneous device options. It would be great if this only happened when the TV is enabled under Windows.
TV is turned off. I turn on my PC with two monitors config already set. I can see the TV being disabled in the Nvidia Control panel and Windows 11 monitor settings. Unfortunately, this also turns my TV on despite being disabled. It seems like the monitor topology condition in this scenario is not checked.

Is this an issue with my configuration? Thanks! I guess what I would like to achieve is better synchronisation of the TV input and power state based on it being enabled/disabled in Windows.

Hi @ka5p3rr apologies for the delayed response - totally missed your message. Thanks for the detailed scenarios

Great!
Do I understand correctly that you would like the TV to switch to the PC input when you enable the "just TV" profile? If so that might make sense to implement
Gotcha. I will add it to the feature requests
Topolgy changes are currently triggered in one of two ways.
a) when windows signals a change (after you changed display fusion profile for example)
b) eight seconds after the user has logged on

Regarding 4) I'm not sure if you mean a cold boot or system resume. If you mean a cold boot then the behaviour should be that the TV power on as soon as the service loads, before the user logon screen and to allow for the user to properly login. Then, after user logon, and if the topology function is enabled in the app and the TV is disabled in the windows monitor settings you will notice the TV turning off eight seconds after logon (after the daemon has loaded actually). The monitor topology is a user space feature so there is not so much else to tweak regarding this behaviour.
If you refer to a system resume (after sleep) then the topology state (disabled/enabled) should be preserved and adhered to.
If you can clarify on no 4 above please and also please paste a log I will have a look.

Hi @ka5p3rr any input wrt the above? Check out the next version as well which allows for restoring the topology on boot (before logon)

Hello @JPersson77, apologies for not getting back to you earlier.
I have just updated the app, thank you!
In terms of your comments on the provided scenarios:

Yes, exactly. I would like the inputs on the TV to switch to my PC as the display output is enabled under Windows.
Awesome, thank you!
Ahh, that makes sense. Thanks for the explanation.

In terms of the changes in the update, is that addressing point number 3.?

point no 4 :)

Exciting, I will check that out. Thank you :)


fix add event on broken ami and empty response

Hello!
Please see https://github.com/marcelog/PAMI/pull/61/ for a similar issue.
Could you verify if this is  a broken AMI response and/or a broken AMI command? If that's the case, the solution would be to use another similar command (or set of commands) or upgrade to an asterisk version that fixes this.
Thanks!

Hi, problem reproduced constantly and randomly when i call method $asterisk->process() when try to parse events by registerEventListener(), version of Asterisk is 11.15.1.

Hello,
Please provide more information and a proper justification for the patch proposed. Logs for the events that cause the issue and any actions you might be sending too.
Thanks.

Closed due to lack of activity


roadmap: Cortex works on ARM-based SoCs (i.e. Raspberry Pi)

Goal

Cortex should work on Raspberry Pi and ARM-based SoCs
This will likely be the future direction for Cortex
Likely Sprint 27

Tasklist

[ ] What hardware is needed for this?

Raspberry pi 5 for testing on real device: https://mlab.com.vn/raspberry-pi-5-mlab-com-vn
More mac mini silicon to creat VM linux arm64 base for CICD agent

[ ] CI/CD pipeline
[ ] Installation docs update

Build cortex.cpp and cortex.llamacpp successfully but not able to run it

cortex is able to run on orange pi 5

Macos silican can be create more than 2 VMs following this instruction https://khronokernel.com/macos/2023/08/08/AS-VM.html

Orange pi 5

Tinyllama: 4.69146374767 tokens per second


Abnormal result when register onCleanup conditionally

Vue version
3.2.47
Link to minimal reproduction
https://sfc.vuejs.org/#__DEV__eNp9UVtqwzAQvMpWP3HAtugDCqkTUgo9haA49tpxoxeSnHwY372SnHdL/ePdnZ0ZdjSQd63zfY9kQQpbmU47sOh6vWKyE1oZBwMYbFI4lK7awgiNUQJmnjFjkslKSeug4WULy7CXNCW3OH8LWGQkAUshkXj4jNVXCkp+cCxlr+ewXMHAJAClEKQUx5yrNplFLtYPsxSOzKAJ0DVnqfnEhItcklwEw3ejWIWdPxQBxqkYmYxVQaccfAK+cSg0Lx36DqDYPq6GYTp3HAvq2zje9M4pCeuKd9Vuycgxj4fwZyTuADjVthzjPp0IHijo2YCkZEo8E6XOv62S/k3iLewIWEYWp+sY8U8Qeka2zmm7oLSXetfmlRJ07TFqeuk6gVmtxPo5f8pfXmndWXc9z9GKbGPUwaLxjoykV+LUD/doMoOyRoPmX7O73RvDO+yX6Sl8Mv4ABlLaxw==
Steps to reproduce

repeat clicking toggle.
see console log.

What is expected?
cleanup function should not be called in not registering onCleanup condition.
What is actually happening?
cleanup function called in not registering onCleanup condition.
System Info
No response
Any additional comments?
No response

Your if condition controls when registers onCleanup function. it does not control when vue call onCleanup function.
If you register an onCleanup function it would always exist. so you will see the watch method is triggered, and the onCleanup method is also triggered

I see. But there is no way to unregister onCleanup right? Then condition should be inside cleanup function.
I think official guide about onCleanup is unclear for understanding its logic. 😢

Same issue as https://github.com/vuejs/core/issues/5151


unnecessary params in url

I create a client:
vk_client = OAuth2::Client.new(vk_id, vk_secret,  :site => 'https://vk.com',
:authorize_path => 'https://oauth.vk.com/authorize',
:access_token_path => 'https://oauth.vk.com/access_token')
And try to get access_token:
vk_client.web_server.access_token(code, :redirect_uri => vk_redirect)
And i have error, due to unnecessary parameter in the query
?act=s&client_id=%my_client_id%&client_secret=%my_client_secret%&code=%my_code%&grant_type=authorization_code&pid=28264373&redirect_uri=%my_redirect_uri%&subdir=oauth&subsubdir=token
Parametr "act" is unnecessary and i don't know  where does it come

How did you get the code value?

exports.getHome = function(req, res) {
code = req.query.code;
}

@jhmoore you've had your fingers in the relevant places recently.  Any idea where a random parameter like act might come from?
It certainly could be any other middleware injecting it, and wholly unrelated to OAuth2.

Actually the term act never appears in this codebase, so it is unlikely to be a bug in this codebase.  Closing.


SYSLOG_FACILITY custom option patch

Select LOCAL0 to LOCAL7 on configure.
--with-syslog-facility=LOCAL0 - LOCAL7

Thanks for the patch, but we need a configure.ac patch for changes to configure.  I moved the changes to configure.ac, and committed the changes for --with-syslog-facility.

Thank you for the changes.
I'll be careful next time.


Access Multi Repository in Azure DevOps Classic Pipeline

[Enter feedback here]
We are using Classic Azure DevOps Pipeline(Due to organization restriction). Normally we can use multiple repose in yaml during setup but same doesn't apply to Classic. I wanted to other repo in same project in my classic pipeline.
I got this Document from Microsoft for accessing additional repo Microsoft Document. I followed the same steps. Added script task with git clone -c http.extraheader="AUTHORIZATION: bearer $(System.AccessToken)" Task and enabled oauth Oauth
Starting: Bash Script
==============================================================================
Task         : Bash
Description  : Run a Bash script on macOS, Linux, or Windows
Version      : 3.214.0
Author       : Microsoft Corporation
Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/bash
==============================================================================
Generating script.
Script contents:
git clone -c http.extraheader="AUTHORIZATION: bearer ***"<EMAIL_ADDRESS>"C:\Program Files\Git\bin\bash.exe" -c pwd
/d/a/_temp
========================== Starting Command Output ===========================
"C:\Program Files\Git\bin\bash.exe" /d/a/_temp/d0ad30a0-9750-4561-9854-3bb4d4da9743.sh
Cloning into 'DevOps%20Integration%20Demo'...
remote: TF401019: The Git repository with name or identifier DevOps Integration Demo does not exist or you do not have permissions for the operation you are attempting.
fatal: repository 'https://dev.azure.com/Project/DevOps%20Integration%20Demo/_git/DevOps%20Integration%20Demo/' not found
##[error]Bash exited with code '128'.
Finishing: Bash Script

I get the error while running the pipeline. This Project is accessible and I am Project Administrator of the project.

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

ID: 8b846aea-82f7-a988-67d8-1b5e74b61ed2
Version Independent ID: 2d632729-bb33-c0a0-c996-e1d8e86c2e23
Content: Build Azure Repos Git repositories - Azure Pipelines
Content Source: docs/pipelines/repos/azure-repos-git.md
Service: azure-devops-pipelines
Sub-service: azure-devops-pipelines-repos
GitHub Login: @steved0x
Microsoft Alias: sdanie

@h-karthik12 -- thank you for your feedback.  This may be implementation-related; please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

@steved0x -- Steve, please look into this potential documentation issue.

Please see this section here:
https://learn.microsoft.com/en-us/azure/devops/pipelines/repos/azure-repos-git?view=azure-devops&tabs=yaml#limit-job-authorization-scope
And here:
https://stackoverflow.com/questions/70781149/tf401019-the-git-repository-with-name-or-identifier-public-does-not-exist-or-yo

AB#60499


Ensure node data for refs on pre-rendered nodes with no attributes

I've come across an exception in diff.js where the node does not have the [ATTR_KEY] property, leading it to throw on this block of code near the end of the idiff function
if (originalAttributes && originalAttributes.ref) {
    (out[ATTR_KEY].ref = originalAttributes.ref)(out);
}

The node may not have had ensureNodeData called on it because:
a) it has no attributes
b) the DOM node it's diffing against has been pre-rendered (and thus doesn't have an existing [ATTR_KEY] property)
c) there is an attempt to get a ref on the component for whom the node represents the root node
Here is the failing test case (a part of this PR):
it('should add refs to components representing DOM nodes with no attributes if they have been pre-rendered', () => {
	// Simulate pre-render
	const parent = document.createElement('div');
	const child = document.createElement('div');
	parent.appendChild(child);
	scratch.appendChild(parent); // scratch contains:

	let ref = spy('ref');

	function Wrapper() {
		return

;
	}

	render(
<Wrapper ref={ref} />
, scratch, scratch.firstChild);
	expect(ref).to.have.been.calledOnce.and.calledWith(scratch.firstChild.firstChild);
});

But it passes if I make a small change by adding at least one attribute to the element:
function Wrapper() {
	return

;
}

The test case is slightly convoluted because it does not work if the node with the ref is the root node in the render function (hence the artificial component structure)
I'm not 100% sure the solution proposed in this PR is the best one, or if there is a more elegant way of describing the test case, but this PR fixes the issue.

Will take another look at this tomorrow!

Wondering if we can further simplify this by removing the return value of ensureNodeData() and placing a call to it at the top of idiff().  I'll probably merge this and play with it post-merge if that's alright.

Hi @ivantm - I ended up taking some of your suggestions here as part of the 6.2 release. However, I'd still like to merge your tests. I'll close this PR out once I've done so. Thanks for the help!

Merged! I changed the author in my merge commit so the tests will get properly attributed to you. Thanks!!


version support

Describe your idea
i dont know if its simple or not but can we get this mod on version 1.19 and not 1.19.x

Already there:
https://www.curseforge.com/minecraft/mc-mods/libx/files/all?filter-status=1&filter-game-version=2020709689%3A9186
https://modrinth.com/mod/libx/versions (Filter for 1.19)


[1.12.2 - Suggestion] Possible to change the Hurt Threshold to be scaled by percentage instead of a flat value?

I don't know of any mods out there that increases the max Hunger/Saturation value from the default 20 but for Health... some mods out there allow you to increase it's max value from the default 20.
Would it be possible to have a config option to have these thresholds play out based on a percentage of the remaining Health instead of a flat value? A few examples...
Config option is set at 25% for Hurt Threshold. It's sound effect will only play if the player's health is at 5 or less if their max health is 20. If the player increased their health to... say 40 and their health is 10 or less... the sound effect plays.
10% would be 2/20 | 4/40 | 10/100, etc. for the sound to play.
50% would be 10/20 | 20/40 | 50/100, etc. for the sound to play.

I can probably do this for the <IP_ADDRESS> version (which I am currently working on).

Pushed a BETA of v3.5 to CurseForge.  Note that the new version has a library dependency on OreLib (another library mod I made).  Make sure you read the release notes prior to upgrade.


Adding binary search in python

Hey, I want to add binary search algorithm in python language in this repository. Please assign me this issue.

@Durgeshsejekar assigning you

Merged 👍


Fix various issues

:tada: This PR is included in version 1.2.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[LINUX] Only show black screen and button when set traffic

I am using the latest version of TMPE. When I set traffic, all that appears is the black area with the button buttons.
Even so, I can still use it to manage traffic and run it. It's just inconvenient to use because there isn't a single text that I can read.
Is there any solution for this?

Make sure the Arial font is installed, the mod is using default Unity font. You may try changing fonts with a mod, I'm pretty sure there is one working on the workshop.


DLS: add info button to show probabilities

Info button, on mouse hover it displays the following text: “Chances: 1% Legendary, 5% Epic, 10% Swanky, 14% Uncommon, 70% Empty.”
1pg: https://docs.google.com/document/d/16gBeK1v5wrNBIs2Imagxh1SwjQzpRqolgQ5VQQEhVS4

Closing stale issue. I figure we will create new issues related to this when this feature is re-introduced.


feat: modify deprecation warning for DO analysis in builder

From 2021-03-15 to 2021-03-31

:robot: please make sure all the checks pass before merging @VictorVelarde @Josmorsot

Thx Rafa, I'm sorry for this one.
I have reviewed the PR and honestly I'm not sure the reason of the failure

I will appreciate if this check is becoming mandatory now. I would also appreciate a theory on what happened here with that process and test


Change of request in CC

Hi,
We have got below changes requested for Company Communicator app, kindly let know how we can handle the changes including its deployment.

Ability to choose image from local system while configuring the message by authors.
Remove the option "Send message to everyone in the organization" from config tab

Kindly help in implementing these changes as we are quite to new this app and technology.

Hi @raghavn ,
The feature "Ability to choose image from local system" is going to be added in the upcoming releases. So we consider it to our backlog and take it in future release. And for the second option, we won't be able to assist on the customization part.
If there is any issue with deployment or functionality of Company Communicator, please let us know.
Thank you.

Hi @gsv022,
Thanks for the update on the ability choose image from local system. Is it going to be included in the next version??,  we are not looking for exact timeline but atleast the ball park like in next 3 months or 6 months if it will be available.
What we would like to know in terms of customization is that, if we make the changes in the code then how do we deploy it back to already existing deployed app. For example, if we make changes in the clientapp which is a react component, then how do we build that and deploy only that part to azure?
It would be really a great help if you could guide us on this.

Hi @raghavn ,
We won't be able to commit any timeline but this feature will be taking up in the upcoming release i.e., 5.2 version.

Hi @raghavn, Ability to choose image from local machine will be available in v5.2 which is planned to be released by end of July 2022. If you want to customize any changes, you can fork the main branch and make changes on top of it and sync to your repo in the deployment center of app service and azure functions. This way your existing app will point to your repo and changes will be reflected. Please note that we'll not be able to support any customized version of the app. We'll proceed with closing this issue since the needed response provided, if you've any further questions request to open new issue.


[BUG] Cancel API doesn't restrict cancel operation on streaming job

What is the bug?
Cancel API doesn't restrict the job type to to cancelled. If user runs cancel API with streaming job, the index state will be REFRESHING. Later, CREATE index statement with same index name will fail.
How can one reproduce the bug?
Steps to reproduce the behavior:

Create a skipping index
Cancel with the job ID
Create the skipping index again

What is the expected behavior?
For streaming job, user should always use DROP index statement which cancels the EMR-S job and move index state to DELETED. Cancel API should reject cancel request on streaming job. In this way, CREATE index statement can pass without issue.
What is your host/environment?

OS: 2.11

If we want to support cancel, PPL should trigger a new Flint Spark API, such as CANCEL INDEX JOB (as opposed to current RECOVER INDEX JOB). This command will stop the streaming job and transit index state back to ACTIVE.

does create skipping index failed in step expected?
Create a skipping index
Cancel with the job ID
Create the skipping index again

Cancel API definition
The cancellation of the "create skipping index" statement would typically mean instructing the system to stop the index creation process that has been initiated by the above statement. Depending on where in the process the cancellation request is made, this can entail different things:

Before Index Creation Completes: If the cancellation request is made while the index is still being created, it would ideally halt the creation process. This means the index would not be created, or if partially created, the system should ideally roll back any changes made up to that point.
After Index Creation but During Auto-Refresh: If the index has already been created and the cancellation request pertains to the auto_refresh process, it would involve stopping the automatic refresh process associated with that index. The index itself would remain unless explicitly dropped, but it would no longer automatically update.

Solutions
alt-1 [preferred]

if query result state is success, the query can not be cancelled.
if query is not executed, the query can be cancelled.

CREATE SI/CI/MV (auto_refresh=true) is submitted as EMR-S job, it is considered as executed when job is submitted,

if query is executing,

if it is select query, it can be cancelled. https://github.com/opensearch-project/opensearch-spark/issues/243
if it is refresh query, it can be cancelled.

limitation:CREATE SI/CI/MV (auto_refresh=true) can not be cancelled, error message is "Can not cancel create flint index query"

alt-2 Support cancel CREATE SI/CI/MV (auto_refresh=true) if EMR-S job is not in executing state

EMR-S client does not provide transitional get and cancel api.


Load trained models on torch versions below 1.7

Loading the trained models in pytorch 1.6 leads to errors like the following:
aten::_convolution(Tensor input, Tensor weight, Tensor? bias, int[] stride, int[] padding, int[] dilation, bool transposed, int[] output_padding, int groups, bool benchmark, bool de
terministic, bool cudnn_enabled) -> (Tensor):
Expected at most 12 arguments but found 13 positional arguments.

Fortunately, it seems like these are largely due to signature changes in pytorch 1.7, and the model does not use pytorch 1.7 specific features. I have written a script to patch these models, in case it is useful for others: https://gist.github.com/achalddave/12e82c3c879589ee287e9c2769c489f0
It would be nice if there were a way to save the models in a way that they work with earlier versions of torch, but I'm not sure if this is possible. Just filing (and closing) this issue in case it is helpful for others who, like me, are working in an environment where upgrading to torch 1.7 is not possible.

Can't agree more!
I find a difficult way which may help you to load the model in the lower version pytorch.(for me is 1.5.0)

install pytorch 1.7.1 (just cpu version is ok, we only use this pytorch version to get model.state_dict() )
2.Then download the model you want from here:
"RN50": "https://openaipublic.azureedge.net/clip/models/afeb0e10f9e5a86da6080e35cf09123aca3b358a0c3e3b6c78a7b63bc04b6762/RN50.pt",  "RN101": "https://openaipublic.azureedge.net/clip/models/8fa8567bab74a42d41c5915025a8e4538c3bdbe8804a470a72f30b0d94fab599/RN101.pt",  "RN50x4": "https://openaipublic.azureedge.net/clip/models/7e526bd135e493cef0776de27d5f42653e6b4c8bf9e0f653bb11773263205fdd/RN50x4.pt",  "ViT-B/32": "https://openaipublic.azureedge.net/clip/models/40d365715913c9da98579312b702a82c18be219cc2a73407c4526f58eba950af/ViT-B-32.pt",

Use this code in python or some place to get model
model = torch.jit.load(model_path, 'cpu')

Use the code model = model.state_dict() to get the state dict of the weight

Use the code torch.save(model, 'your_path.pth.tar') to save the model

Then Back to the pytorch version you want now

Use the build_model method code in model.py to build a model, and edit the code to load the weight which we save at your_path.pth.tar


glfwDestroyWindow() erase data?

Hi I have a question.
after call glfwDestroyWindow() all the VertexArrays BufferData and so on will be destroy or I should call  glDeleteBuffers() and
glDeleteVertexArrays() before call  glfwDestroyWindow()?

OpenGL assets should all be freed automatically when the context is destroyed, so there is no need to delete everything individually.

OpenGL assets should all be freed automatically when the context is destroyed, so there is no need to delete everything individually.

ok thanks
I take the "should" for good :)

ok thanks
I take the "should" for good :)


Added Temitope Agbaje to the list

Thank you! We will merge this as soon as we treat the prior PRs.
In a bid to avoid further merge conflicts, I recommend that you make another commit and remove your name from the last line and then add it anywhere else in between. Thanks again!

Thank you! We will merge this as soon as we treat the prior PRs.
In a bid to avoid further merge conflicts, I recommend that you make another commit and remove your name from the last line and then add it anywhere else in between. Thanks again!

Thank you! We will merge this as soon as we treat the prior PRs.
In a bid to avoid further merge conflicts, I recommend that you make another commit and remove your name from the last line and then add it anywhere else in between. Thanks again!

Thank you! We will merge this as soon as we treat the prior PRs.
In a bid to avoid further merge conflicts, I recommend that you make another commit and remove your name from the last line and then add it anywhere else in between. Thanks again!

Please I am sorry for all the trouble we have caused you but count it all joy. Your PR is about to be merged and you will be able to see your name on this url.
To enable me to do this, READ the README again carefully, update your code with the latest and push the update. I am here for you for the next 6 hours before I close the PR so you can initiate a fresh PR which will be easier to merge.

Updated , Kindly review

Please update again.

Kindly update again. I ran some optimizations.

Or create a new PR. Sorry for the inconvenience

Merged in #569 🥳


Salesforce CDC data/ChangeEvents schema ID support

I can subscribe to the generic CDC event data/ChangeEvents, when it receives the notification it's not able to handle it.
The problem is in the schema validation logic from the gRPC library and Schema ID is mandatory according the proto file of the API.
But Schema ID is not sent from the server for the messages for data/ChangeEvents subscrition.

See https://github.com/pozil/pub-sub-api-node-client/issues/34


Update examples/meteor to use API v1beta3

@chees @nikhiljindal

I have updated the json files. I have verified that these json files can create service/pod/replicationcontroller correctly. But I don't know how to check if meteor is running correctly. Can some one verify?

I change the way to get the external IP of the meteor service. Now it uses kubectl instead of the gcloud specific command.

LGTM modulo one comment.

LGTM.
As per https://github.com/Q42/meteor-gke-example/blob/master/README.md, you can verify if it is working fine by verifying that it "prints the public IP address of the machine the app is running on"?

I verified that following these steps with the v1beta3 api works.
However, at first it took a long time before Mongo was not "pending" anymore and running gcloud alpha container kubectl describe pod mongo showed:
Thu, 07 May 2015 11:24:50 +0200 Thu, 07 May 2015 11:24:50 +0200 1 {kubelet k8s-cluster-2-node-1}            failedMount Unable to mount volumes for pod "mongo_default": Could not attach disk: Timeout after 10s
Thu, 07 May 2015 11:24:50 +0200 Thu, 07 May 2015 11:24:50 +0200 1 {kubelet k8s-cluster-2-node-1}            failedSync  Error syncing pod, skipping: Could not attach disk: Timeout after 10s

Not sure if that's related to this change. After a while the problem fixed itself and mongo came up anyway, so I guess it's not a serious problem.

Great. Will merge then


Assertion failure reporting precedence

If a particular request fails both the status code assertion and Swagger spec validation, the report shows spec validation failure first.  I think by and large showing the status code assertion failure first would be more useful when debugging.

Makes sense to me. Do you want to take a stab at it?

Will do.


feature request: email to reset password

Hi,
Another feature request that might be very interesting: a button that the user could use to send an email to   his/her email address (the one in the "mail" attribute on his/her AD account).
The email would have a link which would allow to reset the password without knowing it.
I know it would make a lot of changes (including storing data in a file or database) and would change the original philosophy of passcore but that would be a great feature.
Of course that would only work when using a service account and not when UseAutomaticContext=true.
Thank you for considering this suggestion.
Regards,

It's possible but we have no plans to include it.


Introduction To Pandas

Introduction To Pandas

Info about the Introduction To Pandas

[x] DevIncept Participant

[x] Contributor

Closes: #34
Describe the changes you've made
This issue will help readers in giving all the guidance that one needs to learn about pandas. Tutorial to pandas and how it's applied using sample code.
The main intention of this repo is to educate beginners with all the concepts related to data science so that they could feel confident to apply these things learned in practical exposure.
Type of change
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented on my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added test cases that prove my fix is effective or that my feature works.

Changes to be done:
The project folder should be as "Introduction to Pandas"
The project file should be as "introduction_to_pandas.ipynb"
Instead of 2 separate project files, as you had used pandas libarary in merging dataframes, you can add this up in "introduction_to_pandas.ipynb" project file as a continuation.
Kindly do these changes. @suy1968

Maam Please once again  i do all things one things donot do that is Introduction to Pandas (i write this Introduction_To_Pandas)
Please merge this time Maam , I donot make mistake another time @prathimacode-hub

Changes to be done:
The project folder should be as "Introduction to Pandas"
The project file should be as "introduction_to_pandas.ipynb"
Instead of 2 separate project files, as you had used pandas libarary in merging dataframes, you can add this up in "introduction_to_pandas.ipynb" project file as a continuation.
Kindly do these changes. @suy1968

Maam Please once again i do all things one things donot do that is Introduction to Pandas (i write this Introduction_To_Pandas)
Please merge this time Maam , I donot make mistake another time @prathimacode-hub

I didn't get you, what you meant to say? @suy1968

Changes to be done:
The project folder should be as "Introduction to Pandas"
The project file should be as "introduction_to_pandas.ipynb"
Instead of 2 separate project files, as you had used pandas libarary in merging dataframes, you can add this up in "introduction_to_pandas.ipynb" project file as a continuation.
Kindly do these changes. @suy1968

Maam Please once again i do all things one things donot do that is Introduction to Pandas (i write this Introduction_To_Pandas)
Please merge this time Maam , I donot make mistake another time @prathimacode-hub

I didn't get you, what you meant to say? @suy1968

Maam please check once @prathimacode-hub

Kindly fill the form for your PR's to get counted. You had filled only once and that too wrong PR number.
You need to fill it up for your 2pr's: Introduction to Numpy, Introduction to Pandas. That is PR no's : 117 and 118. Fill them up before tomorrow as I had to send scores to the team. 1 PR = 1 Filled Form. It means in total 2 forms with PR links I had to get them. @suy1968

Kindly fill the form for your PR's to get counted. You had filled only once and that too wrong PR number.
You need to fill it up for your 2pr's: Introduction to Numpy, Introduction to Pandas. That is PR no's : 117 and 118. Fill them up before tomorrow as I had to send scores to the team. 1 PR = 1 Filled Form. It means in total 2 forms with PR links I had to get them. @suy1968

Done Maam @prathimacode-hub

Have you done it for both PR's. @suy1968

Have you done it for both PR's. @suy1968

Yes Ma'am, Introduction to numpy and pandas

Have you done it for both PR's. @suy1968

Yes Ma'am, Introduction to numpy and pandas

Cool.


Y-axis does not scale to very small values

What did you do?
With default install and default scraping config, browse the scrape_duration_seconds metric
What did you expect to see?
Nicely labelled graph
What did you see instead? Under which circumstances?
The Y axis labels are "0.00", "0.00", "0.00", "0.00"

Checking some data points, the labels here should be "0.001", "0.002", "0.003" and "0.004" (the largest data point is 0.00396 and is just below the 0.004 line)
Or perhaps these would be better rendered as 1.0m, 2.0m, 3.0m, 4.0m
Environment

System information:
  Linux 4.4.0-87-generic x86_64

Prometheus version:
prometheus, version 2.0.0-beta.1 (branch: HEAD, revision: 4dcb465029f39a2a3030333a81753f8a40ec90ce)
  build user:       root@d3f9974fac5a
  build date:       20170811-12:12:36
  go version:       go1.8.3

Alertmanager version:
n/a

Prometheus configuration file:
unmodified default

Alertmanager configuration file:
n/a

Logs:
n/a

@yamatoya @grobie @beorn7 Can this be closed ref: #3184 ?

Yes, this should be addressed in 1.7.2. @candlerb please re-open if that's not the case.


Any plan to release annotated RoboNet dataset?

Thanks for the excellent work.
Do you have any plan to publicly release annotated bounding boxes for the RoboNet images?
Thanks!

Any news on this?
Thanks!


Fix pluralization when there are no issues

“0 warning”

thanks for your contribution. sorry for taking a long time to review this I was on vacation.


Feature/rbac (beta)

Beta release - UI features for adminstrating users and groups
Implemented Features:

User and Group Edit UI
New Admin Dropdown Menu
Several additional user and group fields to prepare for future functionality
User creation UI (and #128)

WIP Features:

User group UI for modifying primary group
Secondary group UI for users
Conducts and models UI

I think we need to write a standard on how API calls will work, reviewing all of the code I think we have about 3 or 4 different standards on the API format.


Newsletter Color Scheme Change for Light and Dark mode. [Accessibility]

Describe the bug
This isn't a bug specifically, but I am trying to figure out how to change the Newsletter button color scheme for dark and light mode. I updated to version 2.0 and now I am not sure how to only change the Newsletter button for light and dark mode separately like I was before. Using the Tailwind.css colors without customization usually results in color contrast issues. I just would like an explanation on how to change that now with the implementation of plini for maintaining the Newsletter. I am trying my best to make this website and template as accessible as possible.
I am still learning about JAMStack and Next.js as I go along so, be please be patient.
For instance, I am trying to make this button customized differently for light and dark mode. Currently in neither mode is it accessible and I am unsure how to access this component directly after the update:
Light Mode:

Dark Mode:

Hey there, bear with me, as I'm learning too, but - I got around this by taking a look in the repository for Pliny and when I saw this note on the Pliny page.

Note: The previous cli and starter template have been deprecated. Please use the new components directly in your favourite Next 13 websites.

I then (believe the term is "scaffolded"?), took the source "NewsletterForm" that was being imported, (from here)and then added use client and I was able to call and use the form and verify that it still works, as well as style the button and divs inline with tailwind. Can't speak to the "correct"ness or anything as I'm just getting started but it works and it's pretty!

Thank you for responding I will look into this solution. I appreciate you sharing it. I was discussing this in a Discord and I think this feels more like a solution that I am comfy exploring.

This fix did work and it seems to be the best solution currently. For more context I followed @brohvis suggestion and copied the NewletterForm source code from (https://github.com/timlrx/pliny/blob/main/packages/pliny/src/ui/NewsletterForm.tsx) and added it to my app directory with use client at the top. Then in Main.tsx I simply changed it to import to that file instead of from pliny.


Expand Regexp.quote spec to cover dollar sign characters

https://github.com/natalie-lang/natalie/pull/2222

Looks good to me

Thank you!


update the error test

infact want to test the waittask

@huangyiminghappy can you send the PR against dev branch?

@huangyiminghappy can you remove the conductor.iml and send the PR against dev branch?

thanks


Feature: mingw  bundle install option

Official GUI installer offers Mingw installation but Qt repository don't have it.
I think Mingw itself should be installed by its own installer.

tools_mingw holds mingw binaries. now It can be intalled.


Problems with running turtle programs in Spyder

Description
when i try to launch a script using turtle the turtle graphics window is not working.
What steps will reproduce the problem?

write a script like:

import turtle

wn=turtle.Screen()
wn.bgcolor("lightgreen")
tess = turtle.Turtle() # Create tess and set some attributes
tess.color("hotpink")
tess.pensize(5)
alex = turtle.Turtle() # Create alex

tess.forward(80) # Make tess draw equilateral triangle
tess.left(120)
tess.forward(80)
tess.left(120)
tess.forward(80)
tess.left(120) # Complete the triangle

turtle.done()

launch it

What is the expected output? What do you see instead?
The graphic window opens up but it doesn't produce the output because it stops saying it is not working
Version and main components

Spyder Version: 3.2.4
Python Version: 3.6.3
Qt Versions: 5.6.2, PyQt5 5.6 on Windows

Dependencies
pyflakes >=0.6.0 :  1.6.0 (OK)
pycodestyle >=2.3:  2.3.1 (OK)
pygments >=2.0   :  2.2.0 (OK)
pandas >=0.13.1  :  0.20.3 (OK)
numpy >=1.7      :  1.13.3 (OK)
sphinx >=0.6.6   :  1.6.3 (OK)
rope >=0.9.4     :  0.10.5 (OK)
jedi >=0.9.0     :  0.10.2 (OK)
nbconvert >=4.0  :  5.3.1 (OK)
sympy >=0.7.3    :  1.1.1 (OK)
cython >=0.21    :  0.26.1 (OK)
qtconsole >=4.2.0:  4.3.1 (OK)
IPython >=4.0    :  6.1.0 (OK)
pylint >=0.25    :  1.7.4 (OK)

@ccordoba12 Should we close and resolution:Wontfix or tag and community:Help Wanted this one?

If you think maybe someone can explain the problem leave it open. Otherwise
close it. It was a minor problem since i don t have to use turtle for
serious problem obviusly.
Il 01 feb 2018 3:05 AM, "C.A.M. Gerlach"<EMAIL_ADDRESS>ha
scritto:

@ccordoba12 https://github.com/ccordoba12 Should we close and
resolution:Wontfix or tag and community:Help Wanted this one?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/6278#issuecomment-362134587,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AiFUkAFfGAd5d3l1H0wdOqU5d1cFz7prks5tQRvegaJpZM4Rp-Y8
.

I just ran the code above and had no issues using a windows computer.

Ok there is some problem with mine evidently. I use windowa as well. You
can close the problem . Thank you anyway
Il 01 feb 2018 7:14 PM, "louieabx"<EMAIL_ADDRESS>ha scritto:

I just ran the code above and had no issues using a windows computer.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/6278#issuecomment-362354055,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AiFUkMUp1yRK-GNMzx0wtCLnn6LiuifRks5tQf76gaJpZM4Rp-Y8
.

Thanks @louieabx and @dododoooo , will do.

I know this is closed because of lack of resources to address.  I'm adding the following
for informational purposes.
Turtle graphics are a useful tool for visualizing L-Systems and fractals, as described in "The Algorithmic Beauty of Plants".  I find it a very useful educational and visualization tool for some things.  Everything works great in Linux.  Unfortunately, for teaching purposes, I'm force to use a Windows computer.  The module appears to work fine, as long as I don't click on the turtle window itself.  When I click on the turtle window, it crashes and takes the python kernel down with it.
This is a python 3 anaconda installation of spyder

@ccordoba12  testing a little bit with Turtle I came with some tips to run a script multiple times in Spyder or prevent the crash of the IPython kernel when trying to run a script that uses Turtle (at least in Windows):

Call turtle.done() with turtle.bye() and use the Inline or Automatic backend for the IPython Console (Preferences > IPython Console > Graphics > Graphics Backend). As an example:

import turtle

wn=turtle.Screen()
wn.bgcolor("lightgreen")
tess = turtle.Turtle() # Create tess and set some attributes
tess.color("hotpink")
tess.pensize(5)
alex = turtle.Turtle() # Create alex

tess.forward(80) # Make tess draw equilateral triangle
tess.left(120)
tess.forward(80)
tess.left(120)
tess.forward(80)
tess.left(120) # Complete the triangle

alex = turtle.forward(100)

turtle.done()
turtle.bye()

The call of turtle.bye() will raise a Terminator error but the IPython kernel will not die. To prevent the output of the exception you can use a try-except:
...
turtle.done()
try:
    turtle.bye()
except turtle.Terminator:
    pass

The reason for this is to clean some elements used by Turtle after running.

Instead of using the IPython Console you can use an external system terminal. Go to Run > Configuration per file... > Select Execute in an external system terminal over the Console section.

These are great news @dalthviz! Thanks a lot for digging into this one.
So we need to do two things to fix this issue:

Please open a page in our wiki (similar to the PyQt one), explaining people how to run turtle programs in Spyder.
Please patch turtle.bye in spyder-kernels to avoid it raising a Terminator exception. See here for an example of patching a built method and do this against the 0.x branch.

@juanis2112, could you confirm if @dalthviz findings work on macOS too? Thanks!

@ccordoba12 the new wiki page to run turtle scripts

Great job @dalthviz ! Your writing looks very good to me.

Great, thanks @dalthviz! It looks really good, I only made a couple of minor modifications to it.

Pinging @juanis2112 here again. @juanis2112, does @dalthviz solution works for you on macOS?

@ccordoba12 Yes it works on macOS.

Nice, thanks for the confirmation @juanis2112!

These are great news @dalthviz! Thanks a lot for digging into this one.
So we need to do two things to fix this issue:
1. Please open a page in our wiki (similar to the PyQt one), explaining people how to run `turtle` programs in Spyder.

2. Please patch `turtle.bye` in `spyder-kernels` to avoid it raising a `Terminator` exception. See [here](https://github.com/spyder-ide/spyder-kernels/blob/0.x/spyder_kernels/customize/spydercustomize.py#L203) for an example of patching a builtin method and do this against the `0.x` branch.

How and where can i edit the spyder-kernels so that this piece of code is runned automaticaly at start of spyder? Is it to add in the Pythonstartupfile?

Hi @trixxx3 what do you mean by this piece of code? Do you refer to the patch done for turtle at PR spyder-ide/spyder-kernels#90?

From the beginning,
i started to write a skript with turtle and wanted to use in the ipyhon console. After i added
turtle.done()
at the end of my skript I was able to close the window and end the script. But with a second run i got the TERMINATOR error. Like You posted on 17. Feb I added
turtle.done()
try:
    turtle.bye()
except turtle.Terminator:
    pass

prevents the error but i have to close th e window manually. Then i found what  ccordoba12 posted on 19. Feb

2.) Please patch turtle.bye in spyder-kernels to avoid it raising a Terminator exception. See here for an example of patching a builtin method and do this against the 0.x branch.

where you find
#==============================================================================
# Turtle adjustments
#==============================================================================
# This is needed to prevent turtle scripts crashes after multiple runs in the
# same IPython Console instance.
# See Spyder issue #6278
try:
    import turtle
    from turtle import Screen, Terminator

    def spyder_bye():
        try:
            Screen().bye()
            turtle.TurtleScreen._RUNNING = True
        except Terminator:
            pass
    turtle.bye = spyder_bye
except:
    pass

Which should fix it automatically on running the ipython console but i dont know how to include it.
Is the solution to add your code at every end of turtle skripts?
Thank You for helping ; -)

@trixxx3 which Spyder version are you running? try updating Spyder. Also, here we have some options to run turtle scripts within Spyder

I was running spyder 3.3.3 and its working now in 3.3.6. But turtle.bye( ) does not close the window

I was running spyder 3.3.3

This was fixed in Spyder 3.3.4, as you can in the "Milestone" tag you can find to the right of this issue. So you need to update to our latest version (3.3.6) and you'll get the fix for this.


Multi-line Strings

For String data that spans multiple lines, specify /T REG_MULTI_SZ and then the data after /D all on the same line with each desired line of text seperated by \0.  You can change the seperator character to any single character of your choice by specifying /S followed by the desired character without double quotes; in this case, only symbols should be used.
Samples:
REG ADD "HKCU\Test Key" /V "Multi-line String Value" /T REG_MULTI_SZ /F /D "This is a test string\0that spans 2 lines"
Value: Multi-line String Value
Data:
This is a test string
that spans 2 lines
REG ADD "HKCU\Test Key" /V "Multi-line String Value" /T REG_MULTI_SZ /S * /F /D "This is a test string*that spans 3 lines*using an asterisk as the seperator"
Value: Multi-line String Value
Data:
This is a test string
that spans 3 lines
using an asterisk as the seperator
REG ADD "HKCU\Test Key" /V "Multi-line String Value" /T REG_MULTI_SZ /S \0 /F /D "This is a test string\0that spans multiple lines"
ERROR: Do not explicitly specify \0 when using the /S argument; it must simply be omitted when using \0.

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

ID: 3deae269-de28-fb85-f52e-334e5b222de8
Version Independent ID: 792402db-afda-e1fc-18e5-551a4b625b83
Content: reg add
Content Source: WindowsServerDocs/administration/windows-commands/reg-add.md
Product: windows-server
Technology: manage-windows-commands
GitHub Login: @coreyp-at-msft
Microsoft Alias: coreyp

Thanks for your dedication to our documentation. If we have been unable to review your issue in a timely manner, we sincerely apologize for the delayed response. We have captured the informtion you have provided for review. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close


Add more information on how to locate design issues in a project, for designers with no prior knowledge

Based on the assumption that many designers are not familiar at all with Github's organization of issues (and tags) and might feel overwhelmed when landing on a project's GitHub home page (Code), we can assist them by adding more information on what to do after finding a project in the "/projects" page. This could be achieved by adding information and learning material on the "/for-designers" page and/or on each project's page about how issues are organized based on tags.

Should we create a UI Kit or something? If multiple people are going to work on the designs, we need a way to keep everything consistent.

Should we create a UI Kit or something? If multiple people will work on the designs, we need a way to keep everything consistent.

@pavlosts Sure, this can be another separate issue: building a design system to maintain consistency as the design expands and more designers are involved.
However, for this issue, I think that the current design files, their structure, and their assets (which can be found at  .design) cover our needs. Let's try those and find out the process we should follow.

This could be achieved by adding information and learning material on the "/for-designers" page and/or on each project's page about how issues are organized based on tags.

I like this idea.... I could def. see the For designers section to expand over time and include not only best practises around GitHub but also communication hints, what to do, what not, setting "basic expectations" for designers and guide them how to participate / communicate :)


Add Hive DelegationTokenIdentifier ServiceLoader file

Why are the changes needed?
Add Hive DelegationTokenIdentifier ServiceLoader file so that DelegationTokenIdentifier can be decoded.
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

cc @pan3793

Thanks, merged to master/0.3


Selection is not properly set after removal of last character in a text node

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
Imagine: A|Entity|B|Entity|C
When removing character 'A' or 'B' the cursor jumps to the end of the line. In this case character 'C'
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem
Try to delete the whitespace before 'Italic' or between 'Italic' and 'Bold' in my Demo and you will see the cursor jumping to the end of the line.
Demo: https://jsfiddle.net/less/m6z0xn4r/224/
What is the expected behavior?
The cursor should remain at the current location after removing character 'A' or 'B'.
Which versions of Draft.js, and which browser / OS are affected by this issue? Did this work in previous versions of Draft.js?
My demo is using 0.10.1 but its also occuring in 0.10.0.
The problem seems to be that after removal of the character the selection is no longer inside a text node.
Basically it lands here because anchor and focus is no longer a text node
https://github.com/facebook/draft-js/blob/213cd764f61cc64552bddd672ae1748529d55333/src/component/selection/getDraftEditorSelectionWithNodes.js#L96
Then some internal draftjs code is looking for the nearest children to place it properly.

Similar issue: #1749. I think it is necessary to write a modifier that works on the editorState object, not just contentState.


[BUG] LLM tool reports error code: 422 with Mistral-large

Describe the bug
LLM tool reports error code: 422 when using Mistral-large based serverless connection.
How To Reproduce the bug

deploy Mistral-large from model catalog as serverless endpoint in azureml studio
create serverless connection using the endpoint
create a flow by cloning 'web classification' example
use the newly created connection from step 2 in 'summarize_text_content' node
run 'fetch_text_content_from_url' node and then 'summarize_text_content' node

Expected behavior
'summarize_text_content' node runs successfully.
Additional context
same error can be reproduced by posting to the serverless endpoint using curl:
curl -X POST -L https://your-Mistral-large-serverless.eastus2.inference.ai.azure.com/v1/chat/completions -H 'Content-Type: application/json' -H 'Authorization: apikey' -d '{"messages":[{"content":"You are a helpful assistant.","role":"system"},{"content":"What is good about Paris?","role":"user"}], "temperature": 1, "top_p": 1, "stream":false, "user":"", "max_tokens": 50}'
if "user":"" (which is right in front of "max_tokens": 50 in the above command) is removed , the curl command would get proper response instead of erroring out:
curl -X POST -L https://your-Mistral-large-serverless.eastus2.inference.ai.azure.com/v1/chat/completions -H 'Content-Type: application/json' -H 'Authorization: aaaa' -d '{"messages":[{"content":"You are a helpful assistant.","role":"system"},{"content":"What is good about Paris?","role":"user"}], "temperature": 1, "top_p": 1, "stream":false, "max_tokens": 50}'
It looks like when the LLM tool generates request to a model, it always include "user":"". I don't think this can be modified or turned off when authoring the flow. I tested other open models, so far llama3 and cohere's command-r-plus worked fine, just not Mistral-large.

We have a change to fix this and will be rolled out in next week's release.  You can specify this image (mcr.microsoft.com/azureml/promptflow/promptflow-runtime:20240520.v6) in the flow.day.yaml to unblock first.

the image worked, thanks.
one issue remains:
if I add a "response_format": {"type":"json_object"} or "response_format": {"type":"text"}, the node works with llama3 and mistral models, but gives this error with command-r-plus model:
Run failed: OpenAI API hits BadRequestError: Error code: 400 - {'message': 'invalid type: parameter response_format is of type object but should be of type string.'} [Error reference: https://platform.openai.com/docs/guides/error-codes/api-errors]
is this fixable too?


Multidatabase?

Hi ,My database name is dynamic. Do you have any suggestions?

Vote for my pull  #1206

👍 for this issue.
It is actually duplicated, there is a closed one about the same, mentioning it is not planned to be supported, but it would be a great win for scenarios when the App is used by different users.

I database name not in application get,
need user login  just get.

@carmas123
My The database name is not available in the application
need a user login to get it

Yes I know

@myhcqgithub
I understand...so now I make an updated fork from lastest version and share on it my changes for you

@carmas123  Thanks,  I Expect Your Reply

I will add it as a configuration parameter, still have to decide best way to do it.

@agrosner   FlowConfig Which parameter can clearly point out?

For me this is also a big issue. I have system which has one main db and separate dbs per logged users. All databases are encrypted especially users based on user password. To make it work in dbFlow I had to prepare custom DatabaseHelper and under App class define specific
public SQLiteDatabase openOrCreateDatabase(String name, int mode, SQLiteDatabase.CursorFactory factory)
public SQLiteDatabase openOrCreateDatabase(String name, int mode, SQLiteDatabase.CursorFactory factory, DatabaseErrorHandler errorHandler)
public File getDatabasePath(String name)
public boolean deleteDatabase(String name)

With that I can have two separate database files based on separate databases classes. First initialisation works ok and problems started when I want to move to another instance of User Database (the same class but different file - file name the same but separate folder). I forced call just closeDb() but FlowManager remember first instance and after changing database and init again databse I hade errors about that my old db is already closed.  That generally works ok but causes integrity issues as it close all databases which are linked to FlowManager. After testing I got few times db corruptions and t is really hard to find safety place to close all db. Generally I think any db shouldn't be closed to prevent multithreading issues. So the best solution for me will be not real close db but ability to remove db configuration from FlowManager and any configuration keep per database file path not based on database class.

@baranowskinet  How do you use dbflow to open different database files?

I am creating encrypted databases on sd card based on:

🙂Hello there! My database name is also dynamic in the project

I really need to ability close individual database (release specific loadedModules which is now prvate in FlowManager). On the end I could make a fork but I prefer no do that and use original source.

in 4.1.0 will replace static @Database(name = "name") with DatabaseConfig.builder().databaseName("name") dynamically. This will ensure that you can pass in a dynamic db name. You can also call DatabaseDefinition.reset(DatabaseConfig) to pass in a new configuration upon opening if you want to change DB name dynamically at runtime.

in develop. use 10-digit commit hash 63158448c2 as version to check it out.


docsregen blocked

Using tag b53f7c7041f5a7e1dd25ac459c4ea9740012482f latest commit, I cannot call the docsregen endpoint.
It seems the database is locked.

after a restart of the docker container, I got:
The operation failed in the backend. (error returned from database: (code: 8) attempt to write a readonly database)

Yes I am aware, I am fixing this.

This should be fixed on main with 69e30f9


GPUDrive compiling to 310 file on python 311

This build file:
gpudrive.cpython-311-x86_64-linux-gnu.so
Compiles as
gpudrive.cpython-310-x86_64-linux-gnu.so
with default python 3.11. Tested on puffertank (docker pull pufferai/puffertank:1.0 for same test env). No idea why, all I had to do was rename it to 310.

Thanks for catching this! @aaravpandya is looking into it.

Thanks @jsuarez5341 for noticing this. I can confirm the issue as I am able to reproduce it with pulling the puffertank image. We will fix the python minimum dependency to be 3.11 as it is set to 3.9 currently. As such CMake is actually picking up the default 3.10 (in /usr/bin/python3.10) installation in puffertank instead of 3.11.
In the meantime, to resolve this issue, the best course is to either use an environment or set the python environment variable in the shell before running CMake.

@aaravpandya If you still have that puffertank image around -- any idea how to get madrona to compile with GPU support? Tried upgrading CMake to 3.29, didn't help. I have the sim working fine, but only on CPU

@jsuarez5341 I'll look into it. One quick thing that catches my eye is that puffertank is based on CUDA 12.1 . The minimum version of CUDA that GPUDrive has been tested is with 12.2.

oh - madrona says 12.1 on the page. I can update the container, but do you know if there's anything in GPUDrive specific to 12.2?

We have updated the poetry config to use py311 only. In @jsuarez5341's case, the environment had two pythons available and CMake was finding the 3.10 version. This was resolved once the env variables were set to python 3.11.
Note this wont be a problem if using a conda env or if the env correctly resolves to python3.11


Add playbooks and roles to deploy log aggregation container

This change is for issue/942 with the overall goal of filtering
and pushing customer logs from the local elk stack to a shared
global elk cluster.  This should give support easier access to
search customer logs for a quicker resolution with customer issues.
This is done in two roles with a playbook for each.  One takes
the elixir based aggie code, builds it in a local lxc continer,
packages it up and copies it to the local repo servers. The second
playbook will set up an lxc container created through openstack-ansible's
setup-hosts playbook and deploy the aggie code to it.  Where
the roles are in their own repo's, future changes should only
require a version update in the ansible-role-requirements.yml file
for anxible-galaxy.
Connects https://github.com/rcbops/u-suk-dev/issues/942

Looks like I did this wrong.  Killing and trying again after creating a local branch.


Initialize workdir thread-safe

Parallel tmt processes should not race any more.
Fixes: #659
Todo:

[ ] Unit test

Using reproducer from 659 I have no exception.


Make Deployment Configurations Persistent

On the frontend, we have a config directory that allows system admins to specify configurations for a deployment. We want make it so the admin can pull our new HydroServer releases in to their repo and not overwrite their configurations.

I created a config/ directory which I think will be good enough for users that fork the frontend repo. As long as we don't modify the config files once they're created, the users will be able to merge our changes without overwriting their configurations. If we do modify an existing config file, it's not the end of the world since they could easily rebase their configurations on top of the updates.


”Bătarci” is listed in the drop-down menu

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

Step 1
Type "Bătarci" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Bătarci, SATU MARE".


Dynamic cognito_options inner block to avoid permission problems in AWS China

what
Amazon Cognito authentication for Kibana is not supported on AWS China.
Therefore we need to have a way to avoid setting the cognito options inner block on the aws_elasticsearch_domain terraform resource.
why
When we try to run this module on AWS China we always get the following error:
Error: Error creating ElasticSearch domain: DisabledOperationException: You don’t have permissions to integrate with Cognito. Contact your admin if you need help. on .terraform/modules/elasticsearch/terraform-aws-elasticsearch-0.18.0/main.tf line 122, in resource  "aws_elasticsearch_domain" "default": 122: resource "aws_elasticsearch_domain" "default" {
Even though we tried to set the correct IAM roles (see references below) and permissions the error still occurs.
references

https://docs.amazonaws.cn/en_us/aws/latest/userguide/elasticsearch.html
https://docs.aws.amazon.com/elasticsearch-service/latest/developerguide/es-cognito-auth.html#es-cognito-auth-config

/test all


Light source on the models

Hello.
Is your feature request related to a problem? Please describe.
It's only a visual thing. Neither of the options to set the colours of models are particularly nice. Currently, neither of them give the appearance of a "real" model spinning on screen. The colours are either flat all along a plane, no matter how far away a point on it is, or a bit psychedelic.
Describe the solution you'd like
I think the main thing that might be missing is a light source shining against the model, to help make the shading more realistic and introduce shadows. This would make them more pleasant to view.
Describe alternatives you've considered
NA
Additional context
NA
Thanks,
Mike

I'd definitely like to improve this 👍🏻


'Ngnix was unavailable' message loop during quickstart installation

I am new to ADOP trying to get started with quickstart script. After setting up AWS credentials and prerequisites for ADOP evaluation mode, I tried executing quickstart.sh using default region eu-west-1 ( Ireland) where the script executes well up until Ngnix installation !
The script fails at "Waiting for Ngnix to become available " message followed by "Ngnix was unavailable" message loop. Is there any work around for this ? Am I missing something here. I looked up for some solution but couldn't find any.

@parvez99 does the docker-machine security group, which is assigned to your ADOP EC2 instance have TCP port 80 open? If not you'll need to add the rule.

Hi RobertNorthard,
Thanks for the reply. Yes it was the port 80 issue. I now have the ADOP platform up and running .
Thanks again


Crunchyroll stream without subtitles

I'm trying to create an app that will let you watch Crunchyroll videos with Japanese subs instead of English ones. Since Crunchyroll doesn't have Japanese subs I have to attach them on my end. To do this I need (want) to grab the just the video and ignore the subtitles. Is this possible?
I can take a look myself if someone can give me a nudge in the right direction.
Relates to: https://github.com/chrippa/livestreamer/issues/364

Don't think you can actually do this, at least not with livestreamer which uses the iOS API (which is anyway limited to max. 480p, regardless of premium or not). You could try it with --crunchyroll-locale=ja_JP.

I've been able to download them at 1080p and attach them locally. Are you saying you just can't live stream it at 1080?
I know you can turn off subs in their player, I'm just not sure if how to get the stream pre-subs has been discovered.

Also, had to check but 1080p streaming works, so I'm not sure what you're referring to.

Just an old issue(?) when CR only supported up to 480p on most devices except browser, regardless of premium access or not. Seems like CR enabled HD content on all other devices, so its no longer restricted.

@BenKettlewell I had to do a bit of research because I haven't been on this topic for a while, but after a quick read on some info I comfirmed some stuff that may be useful to you: currently Crunchyroll has two separate streaming APIs, one used by their webpage (which serves the videos as an RPC stream, with subtitles and video separated) and another for other devices (the one we use here, which serves the videos using HLS with embedded subs).
So, the bad news is that you probably can't remove the subs from the stream we use here (at least from what I have seen); you can try playing around with the API's parameters, but I can't really help you with that since it's a private API and most of what I got was from a Plex plugin whose developer probably just sat there looking at the request the iOS app made when trying to watch an episode. I can give you some pointers about how the plugin works if you decide to go down that route and you need some guidance d:
The good news is that there's people that already got around using the other API (look at Horriblesubs for example), so I'm pretty sure you can find some other scripts/tool that can help you with that. It's obviously not as pretty as using the livestreamer python API (since it has to deal with Flash and stuff), but it's something.
Hope that gives you some insight on the little I know about it, let me know if I can help in any other way!

That's super useful, thanks so much! I'll take a look a look and will probably have some questions for you later.

@BenKettlewell
You can use YOUTUBE-DL to get access to Crunchyroll's RPC API. This means: RTMP streams without subtitles. I don't have an account, so I only get the 480p stream, but it should be possible to enter username and password (see authentication options) to download a higher resolution (720p / 1080p).
youtube-dl -F "http://www.crunchyroll.com/naruto-shippuden/episode-433-the-search-mission-684443"
[Crunchyroll] 684443: Downloading webpage
[Crunchyroll] 684443: Downloading media info
[Crunchyroll] 684443: Downloading media info for 360p
[Crunchyroll] 684443: Downloading media info for 480p
[Crunchyroll] 684443: Downloading media info for 720p
[Crunchyroll] 684443: Downloading media info for 720p
[Crunchyroll] 684443: Downloading media info for 1080p
[Crunchyroll] 684443: Downloading media info for 1080p
[info] Available formats for 684443:
format code  extension  resolution note
360p         flv        656x368
480p         flv        848x480
720p-0       flv        848x480
720p-1       flv        848x480
1080p-0      flv        848x480
1080p-1      flv        848x480    (best)

Subtitles are still available but as separate files:
youtube-dl --list-subs "http://www.crunchyroll.com/naruto-shippuden/episode-433-the-search-mission-684443"
[Crunchyroll] 684443: Downloading webpage
[Crunchyroll] 684443: Downloading media info
[Crunchyroll] 684443: Downloading media info for 360p
[Crunchyroll] 684443: Downloading media info for 480p
[Crunchyroll] 684443: Downloading media info for 720p
[Crunchyroll] 684443: Downloading media info for 720p
[Crunchyroll] 684443: Downloading media info for 1080p
[Crunchyroll] 684443: Downloading media info for 1080p
[Crunchyroll] 684443: Downloading subtitles for English (US)
[Crunchyroll] 684443: Downloading subtitles for العربية
[Crunchyroll] 684443: Downloading subtitles for Italiano
[Crunchyroll] 684443: Downloading subtitles for Deutsch
Available subtitles for 684443:
Language formats
deDE     ass, srt
arME     ass, srt
itIT     ass, srt
enUS     ass, srt

^This works perfectly!
Thank you very much!


[1.11] Your item handler is incorrect

com.buuz135.industrial.tile.misc.BlackHoleControllerTile.BlackHoleControllerHandler.getSlots should be returning how many slots the inventory has, not the counts of the items within!
public interface IItemHandler
{
    /**
     * Returns the number of slots available
     *
     * @return The number of slots available
     **/
    int getSlots();

Black Hole controller provides that number to make available all the items provided by the black hole units.

I haven't looked is something is wrong but that's why it is coded like that. https://github.com/Buuz135/Industrial-Foregoing/blob/1.11.2/src/main/java/com/buuz135/industrial/tile/misc/BlackHoleControllerTile.java#L185

See the javadoc comment I quoted; the purpose of the function is to determine the number of slots, not items

Im not counting the items, The Black Hole Controller holds Black Hole Units (BHU) that have stored in the NBT and Integer.MAX_INT of items. My getSlots() gets the number of items of each BHU and divides it with it stackSize to simulate how many slot would have.

Then perhaps your extractItem is wrong as iteration of the slots fails. See https://gist.githubusercontent.com/leagris/77c32db6e2519d9edf10ca10e5f837fe/raw/671effe58bb04a0e9f6e9aa5471cde0205c1a354/fml-server-3.log

That's another thing that could happen.

Yes it is . As I add extra slots as a buffer the last slots will throw an error.


worker API: introduce job types and architecture

This is an effort to future-proof the worker API. It removes the assumption that all jobs are osbuild jobs and requires a worker to pass both the jobs it can handle and the architecture it runs on.

Update to resolve the conflict and set a correct list of accepted job types.

Thanks! (CI failures are, again, unrelated and caused by the ostree tests).

There was a failure in aws.sh as well, which might have been a flake. I restarted the tests to make sure.

There was a failure in aws.sh as well, which might have been a flake. I restarted the tests to make sure.

This was an actual bug (yay testing and thanks for tracking it down @ondrejbudai). I've copied the logic to assemble job types from osbuild-composer to osbuild-composer-cloud. I didn't bother sharing the code because we're about to merge the two.

That last fix did it indeed. The only remaining failure is due to https://github.com/osbuild/osbuild-composer/issues/997.


Add option for the developer to disable all non-critical functions during FW Update

As a developer, I should have an easy way to shut down all but critical functions while downloading a firmware update.
In many cases, running anything during the FW download could potentially cause the download to fail which would leave the device with a firmware image that it cannot verify the integrity of.

This is already possible, add some way to turn off what you're doing (put em in a thread or something), then override this (https://github.com/ARMmbed/simple-mbed-cloud-client/blob/master/update-helper/update-helper.cpp#L75 )  function in your main.cpp and have it terminate the thread. problem solved.

In fact, the update test is making use of the update helper routines.


Don't use value class for RuleSet.Id

This PR is to follow this idea: https://github.com/detekt/detekt/pull/6851#discussion_r1451588053
Right now the value class give us little value and make our ABI messy. So for now we can just use a normal class. Once we have our API "settled" we can rethinkg about this and go back to value class if needed.

Warnings

:warning:
It looks like this PR contains functional changes without a corresponding test.

  Generated by :no_entry_sign: dangerJS against ba0d5e7344c621ce4c3b94b1303e38af35082b75


pkg/tool/exec: add .Default() to struct env value

Encountered an error that was quite confusing: invalid environment variable value "perfectly-reasonable-string", looking into what caused that I ended up with this little diff.

Thank you for contributing, and apologies that we hadn't replied yet :)
Per the contributing docs, if this is meant to fix a bug, please raise an issue following the bug template first. That way we can be sure that we're on the same page and can reproduce the issue before we consider a fix.

I've left the code that required this behind so I would be more likely to close this PR than create a bug report. The test change should demonstrate the motivation for this, though.

Thanks - now imported to Gerrit as https://review.gerrithub.io/c/cue-lang/cue/+/1174128.

It turns out that an issue was filed for this half a year ago (https://github.com/cue-lang/cue/issues/2572) and https://github.com/cue-lang/cue/pull/2764 was just sent as an alternative PR, which also fixes this for lists and has more tests, so I'll go with that one assuming it's okay with you. Thank you for contributing nonetheless!


Generator bug when encoding with a to-many include path that is empty

When including a to-many relationship that is empty, we're encountering a bug in the underlying encoder:
Exception: Cannot traverse an already closed generator
src/Parser/RelationshipData/RelationshipDataIsCollection.php:132

Have opened the issue and submitted a PR here:
https://github.com/neomerx/json-api/issues/252

No reply to the issue raised in the Neomerx library. For the moment, I've updated our laravel-json-api/encoder-neomerx package to require the branch that has the fix on it.
If we don't hear back from Neomerx about the bug, we're going to have to publish our own fork. I've already forked the library here as it's critical to the Laravel JSON:API implementation, so makes sense to have a fork copy:
https://github.com/laravel-json-api/neomerx-json-api

The fix for this is included in v1.0.0-alpha.4


upgrade rabbitmq

ingest used rabbit-mamagement-alpine image which always took the latest image.
This created an inconsistency in versions whenever rabbit was redeployed. It is a rare event, but it happened when I upgraded the aws cluster (see #774). K8s chose a newer image than the one installed which resulted in startup errors. I changed the image tag to specify 3.8.5, which is the version used on the other cluster (checked on dev, as prod was inaccessible). This version, however, is https://endoflife.date/rabbitmq, so it should be upgraded to at least 3.11.

upgrade procedure
kubectl edit statefulset rabbit
# change version to 3.8.5
kubectl scale --replicas=0 statefulset rabbit
kubectl scale --replicas=1 statefulset rabbit
# check pod in Started status
kubectl describe rabbit-0
# check no errors in log
kubectl logs rabbit-0


monir: 添加触发事件

Codecov Report

Merging #779 (5e8da78) into feature_ticket_no_permission (ac4603e) will not change coverage.
The diff coverage is n/a.

@@                      Coverage Diff                      @@
##           feature_ticket_no_permission     #779   +/-   ##
=============================================================
  Coverage                         61.67%   61.67%
=============================================================
  Files                               445      445
  Lines                             28454    28454
=============================================================
  Hits                              17548    17548
  Misses                            10906    10906

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 ac4603e...5e8da78. Read the comment docs.


Create Software Bill of Goods (SBOM)

The Wave service has reported clients are looking for SBOMs to be provided alongside the generated images.
Since we now use trivy to scan the repository, we should consider doing the same to better support installations into environments with more rigorous security protocols.

A few ideas:

Software Package Data Exchange (SPDX and SPDX Lite), an ISO standard hosted by the Linux Foundation, which outlines the components, licenses, and copyrights associated with a software package.
CycloneDX, an open source, lightweight SBOM standard, which is used in application security and supply chain analysis and originated from the Open Web Application Security Project (OWASP).


Version update

new chart version -> 1.3.5
cytomine/postgis:1.3.0
cytomine/core:4.4.0
cytomine/pims-ce-package:0.2.0
cytomine/web_ui:2.6.2
nginx update creates problems, needs to be understood clearly. I suggest to create a seperate card for that.

it will be fixed in another PR.


How to save nested data type

Hi, I have a question, how to save record to nested data type with this package ?
example field map like this :
"authors": {
    "type": "nested",
    "include_in_parent": true,
    "properties": {
        "id": {
            "type": "text"
        },
        "name": {
            "type": "text"
        }
    }
},

Hey @gavnyx , you can just save the PHP array in the field:
$authors = [
   [id => '', name => ''],
   [id => '', name => ''],
   [id => '', name => ''],
    ...
];

$model->authors = $authors;
$model->save();

See https://opensearch.pdphilip.com/nested-queries for querying nested data


feat: finish mobile scren for track details page

Changes

Mobile version of track details page finished
Header changed and renamed it as navbar
Footer is adjusted to mobile

API Updates

Collapsible component from shadcn was added through manual copy-paste

#39
#40

@recepcankaya can you update this branch with main latests version?

With the latest commit, I updated the code as recommended by @Markkos89

Made changes for navbar, track-details page and footer

For feat: finish the desktop version of track-details page commit:
Changes

Desktop version of track details page is finished. Later, it will be updated for the every screen will be made full responsive
Toggle image is added onto header
Even though it does not show very well, the images which are below are a clue of what it looks like in the desktop


Course list should always be sorted alphabetically

Describe the bug
Our ordering of courses is arbitrary, and should be sorted alphabetically (ascending, A-Z).
To Reproduce

Go to https://wayfair.github.io/awesome-learning/courses/
Observe that the ordering of courses does not follow a meaningful pattern

Expected behavior
Courses should be sorted alphabetically to make it easier for visitors to find a course they are looking for

Let me try it. Can I pick this up?

@BalSharma It's all yours!

Sorry, for the confusion. I decided to work on this and I didn't see that @BalSharma decided to work on this as well. I pushed my changes anyway, feel free to pick the solution you like better (no hard feelings 😄  ).

I did not know that someone already working on the issue. I am sorry @jaysiee. Please go ahead @jaysiee. I have just forked it, done nothing. I am stepping out.

@BalSharma please feel free to jump on another issue, or to make your own issue if you see anything we should improve. We are so appreciative of your enthusiasm!


fix(guide): add better explanation of monad laws

Description

Pre-Submission Checklist
Common mandatory requirements

[x] This pull request targets the master branch of freeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/
(e.g. fix/signin-issue)
[x] Reviewed necessary guidelines in CONTRIBUTING.md.

By submitting this pull request, I have verified the following:

[x] Added descriptive name to pull request. (mandatory)
Your PR should NOT be called Update index.md, since it does not help the maintainer understand what has been changed.
Good Guide PR title examples:
fix(guide): Git - edit Git Commit article
feat(guide): PHP - create PHP section and add Data Structures article

Good Coding challenge PR title examples:
fix(coding): add tests to Basic HTML challenge
fix(coding): update instructions to CSS Grid challenge

Good Learning platform PR title examples:
fix(platform): fixed sign in api
fix(platform): removed css from editor

Issues Closures

[ ] Addressed currently open issue. (replace XXXXX with an issue no in next line)

Closes #XXXXX
Since this change is a copy edit or translation for Guide articles or Coding Challenges:

[x] I confirm no plagiarized, duplicate, or repetitive content that has been directly copied from another source.
[x] I am adding only https links to external sources.
[x] I am not adding short links instead of full URLs to webpages.
[x] I have followed the recomended style guides for the copy edit.

Requirements, for changes which are NOT simple copy editing or translations:
You may skip the below ONLY if your changes do not touch tests, ui logic, api-server, etc. Your PR may not be accepted, if you did not meet these mandatory requirements. Simply put, we can't assume it will work if we never saw it working.
Since this change adds, updates or removes coding challenge tests:

[ ] I completed the coding challenge locally, and the changes look good.
[ ] Other tests or user stories in instructions are not affected.

Since this change is for the Learning platform (UI, logic, etc.):

[ ] I confirm my changes look good by running freeCodeCamp locally.
[ ] All new and existing tests pass the command npm test.

(Optional) Additional Context and Screenshots:

@healym Awesome! Thanks! Congrats on your first open source contribution to this repository! We would welcome more Haskell-related contributions. This is an exciting language and we want to get as many people as we can into coding in it.


ng test produces errors in 1.0.0-rc.4

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting

Versions.

@angular/cli: 1.0.0-rc.4
node: 7.5.0
os: win32 x64
@angular/animations: 4.0.0-rc.5
@angular/common: 4.0.0-rc.5
@angular/compiler: 4.0.0-rc.5
@angular/core: 4.0.0-rc.5
@angular/http: 4.0.0-rc.5
@angular/platform-browser: 4.0.0-rc.5
@angular/platform-browser-dynamic: 4.0.0-rc.5
@angular/forms: 4.0.0-rc.5
@angular/router: 4.0.0-rc.5
@angular/compiler-cli: 4.0.0-rc.5
@angular/platform-server: 4.0.0-rc.5
@angular/language-service: 4.0.0-rc.5
@angular/cli: 1.0.0-rc.4
Repro steps.

Just upgraded from 1.0.0-rc.2 to 1.0.0-rc.4 and ran ng test
The log given by the failure.

On previously working tests there're now errors like:
Cannot find name 'Event'.
Cannot find name 'alert'
Cannot find name 'confirm'
Property 'distinctUntilChanged' does not exist on type 'Observable<any>'

Desired functionality.

No errors ;)
Mention any other details that might be useful.

Confirmed by reverting back to 1.0.0-rc.2 - no errors. Back to 1.0.0-rc.4 - errors.
(I did delete the whole node_modules folder when switching)

I also get the following when I do ng -v

Same issue here:
`ERROR in C:/VSTS/Promethee/Frontend/src/app/notification/notif.ts (23,23): Cannot find name 'Element'.)
C:/VSTS/Promethee/Frontend/src/app/notification/notif.ts (24,28): Cannot find name 'Element'.)
C:/VSTS/Promethee/Frontend/src/app/notification/notif.ts (38,26): Cannot find name 'document'.)
C:/VSTS/Promethee/Frontend/src/app/notification/notif.ts (46,25): Cannot find name 'Element'.)
ERROR in C:/VSTS/Promethee/Frontend/src/app/services/notification.service.ts (9,35): Cannot find name 'window'.)
ERROR in C:/VSTS/Promethee/Frontend/src/tdcmp/treeview/treeview.ts (47,9): Supplied parameters do not match any signature of call target.)
ERROR in C:/VSTS/Promethee/Frontend/src/tdcmp/smartSearch/smartSearch.ts (72,21): Cannot find name 'Element'.)
C:/VSTS/Promethee/Frontend/src/tdcmp/smartSearch/smartSearch.ts (73,19): Cannot find name 'Element'.)
C:/VSTS/Promethee/Frontend/src/tdcmp/smartSearch/smartSearch.ts (74,19): Cannot find name 'HTMLElement'.)
C:/VSTS/Promethee/Frontend/src/tdcmp/smartSearch/smartSearch.ts (75,22): Cannot find name 'HTMLElement'.)
C:/VSTS/Promethee/Frontend/src/tdcmp/smartSearch/smartSearch.ts (80,21): Cannot find name 'document'.)
C:/VSTS/Promethee/Frontend/src/tdcmp/smartSearch/smartSearch.ts (166,11): Supplied parameters do not match any signature of call target.)
C:/VSTS/Promethee/Frontend/src/tdcmp/smartSearch/smartSearch.ts (184,9): Supplied parameters do not match any signature of call target.)
C:/VSTS/Promethee/Frontend/src/tdcmp/smartSearch/smartSearch.ts (312,5): Supplied parameters do not match any signature of call target.)`

@ctrl-brk @viztastic @mikezag Is there a repo we can look at.

@sumitarora, creating a repo with real tests is quite a challenge ;)
But I've got it down to less errors by adding imports of all the rxjs operators app uses to the test module. Was compiling without it before. Now it only complains about 'alert' and 'Event' used by the app.
Should I import anything else?
C:\Projects\Hlr\Zenith\Zenith.Web.Client>ng test -sr
22 03 2017 21:40:54.012:ERROR [karma]: Error: C:/Projects/Hlr/Zenith/Zenith.Web.Client/src/test/shared/shared.ts (14,30): Cannot find name 'Event'.),C:/Projects/Hlr/Zenith/Zenith.Web.Client/src/app/shared/log.service.ts (22,9): Cannot find name 'alert'. Did you mean the instance member 'this.alert'?),C:/Projects/Hlr/Zenith/Zenith.Web.Client/src/app/shared/json-http.helper.ts (20,13): Cannot find name 'alert'.)
C:/Projects/Hlr/Zenith/Zenith.Web.Client/src/app/shared/json-http.helper.ts (22,13): Cannot find name 'alert'.),C:/Projects/Hlr/Zenith/Zenith.Web.Client/src/app/shared/shared.ts (126,16): Cannot find name 'confirm'.),C:/Projects/Hlr/Zenith/Zenith.Web.Client/src/app/shared/guards/pending-changes.guard.ts (17,45): Cannot find name 'confirm'.),C:/Projects/Hlr/Zenith/Zenith.Web.Client/src/app/shared/guards/role.guard.ts (19,13): Cannot find name 'alert'.)
C:/Projects/Hlr/Zenith/Zenith.Web.Client/src/app/shared/guards/role.guard.ts (42,9): Cannot find name 'alert'.)
    at Compiler.compiler.plugin (C:\Projects\Hlr\Zenith\Zenith.Web.Client\node_modules\@angular\cli\plugins\karma-webpack-throw-error.js:10:23)
    at Compiler.applyPlugins (C:\Projects\Hlr\Zenith\Zenith.Web.Client\node_modules\tapable\lib\Tapable.js:61:14)
    at Watching._done (C:\Projects\Hlr\Zenith\Zenith.Web.Client\node_modules\webpack\lib\Compiler.js:91:17)
    at onCompiled (C:\Projects\Hlr\Zenith\Zenith.Web.Client\node_modules\webpack\lib\Compiler.js:50:17)
    at C:\Projects\Hlr\Zenith\Zenith.Web.Client\node_modules\webpack\lib\Compiler.js:480:13
    at next (C:\Projects\Hlr\Zenith\Zenith.Web.Client\node_modules\tapable\lib\Tapable.js:138:11)
    at Compiler.<anonymous> (C:\Projects\Hlr\Zenith\Zenith.Web.Client\node_modules\webpack\lib\CachePlugin.js:62:5)
    at Compiler.applyPluginsAsyncSeries (C:\Projects\Hlr\Zenith\Zenith.Web.Client\node_modules\tapable\lib\Tapable.js:142:13)
    at C:\Projects\Hlr\Zenith\Zenith.Web.Client\node_modules\webpack\lib\Compiler.js:477:10
    at Compilation.applyPluginsAsyncSeries (C:\Projects\Hlr\Zenith\Zenith.Web.Client\node_modules\tapable\lib\Tapable.js:131:46)
    at self.applyPluginsAsync.err (C:\Projects\Hlr\Zenith\Zenith.Web.Client\node_modules\webpack\lib\Compilation.js:640:19)
    at next (C:\Projects\Hlr\Zenith\Zenith.Web.Client\node_modules\tapable\lib\Tapable.js:138:11)
    at Compilation.<anonymous> (C:\Projects\Hlr\Zenith\Zenith.Web.Client\node_modules\webpack\lib\ProgressPlugin.js:122:5)
    at Compilation.applyPluginsAsyncSeries (C:\Projects\Hlr\Zenith\Zenith.Web.Client\node_modules\tapable\lib\Tapable.js:142:13)
    at self.applyPluginsAsync.err (C:\Projects\Hlr\Zenith\Zenith.Web.Client\node_modules\webpack\lib\Compilation.js:631:11)
    at next (C:\Projects\Hlr\Zenith\Zenith.Web.Client\node_modules\tapable\lib\Tapable.js:138:11)

I don't know if it's the samme issue, but upgrading from 1.0.0-rc.2 to 1.0.0-rc.4 gives me errors about tests that have never been modified since the moment they were created by Angular CLI.
The error is: "Supplied parameters do not match any signature of call target.)" <- Also, there is no opening for this closing bracket in the error message. Should we all just inject the dependencies to the tests? If so, why didn't we need to do that before?

We are getting the same error. This is probably caused by Jasmine Typings. Is there any fix around this?
@angular/cli: 1.0.0-rc.0
node: 6.9.4
os: win32 x64
@angular/common: 2.4.8
@angular/compiler: 2.4.8
@angular/core: 2.4.8
@angular/forms: 2.4.8
@angular/http: 2.4.8
@angular/platform-browser: 2.4.8
@angular/platform-browser-dynamic: 2.4.8
@angular/router: 3.4.8
@angular/cli: 1.0.0-rc.0
@angular/compiler-cli: 2.4.8

Generated new project using latest cli and then compared and updated all of my tsconfg files to the current format.
Issue seems to be solved now.

Just updated to @angular/cli 1.0.1 and everything seems to be fine except for pipes. I have a custom pipe with several dependency injections in the constructor and pipe's test throws an error when newing-up the pipe because "Supplied parameters do not match any signature of call target". I tried providing all the parameters but gave up when one of the parameters had to be angular/http service...
Is this still a bug and pipe was just forgotten to fix, or should I NOT use dependency injections in pipe?

My bad - I updated to 1.0.1 without deleting the node_modules directory first...
So the bug still exists on 1.0.1.
I'm not sure how to supply repro steps though since the only steps I could think of are:

In the middle of developing a big project, update @angular/cli to 1.0.0-rc.4.

Closing as above.


Generate reports on qulice execution

Qulice execution must generate reports on checkstyle, PMD, findbugs, etc checks so we could use these reports on our CI tools.

Answered question on SO and reported bug on qulice:

https://stackoverflow.com/questions/53047247/use-maven-transitive-dependency-in-build
https://github.com/teamed/qulice/issues/937