add "Include All" toggle to inclusion list header

🏄 frontend stuff looks spot on from code perspective, others should double check

I've created a separate PR (https://github.com/CarnegieLearningWeb/UpGrade/pull/1785) to refine the section card expansion. This new PR merges into the branch associated with this PR.

has anyone reviewed the backend changes for this also?

@bcb37 Please make sure if all filterMode settings default to EXCLUDE_ALL.

@bcb37 I saw your post on Slack. The section card should indeed be collapsed on page refresh when "Include All" is enabled. My apologies for missing that in my PR. I've created another branch that addresses this issue.
Changes include:

Collapsing the section card on page load if the filterMode is set to INCLUDE_ALL.
Ignoring the parent's attempts to expand when the filterMode is set to INCLUDE_ALL.

You can see the diff here. Please review my suggested changes and feel free to make a commit as you see fit. Let me know if you have any questions or concerns.

@bcb37 Sorry, I think this is a simpler fix that still works: https://github.com/CarnegieLearningWeb/UpGrade/compare/feature/feature-flag-include-all-toggle...bugfix/handle-expansion-state2
You can ignore my previous suggestion.


Merge devel branch

Merge release branch
Commits

All commits came from devel.
I applied the idea of commits to the entire files and kept the author.

from update controller

update controller
remove velocity controller
remove default entity
remove unnecessary conditions (trains)
rename and clean demo_drone
add play

from cleanup tasks

move forest, pinball to single
use MultirotorBase make
remove nav, trackV1
merge done_spec (isaac_env)
use stats.clone()
remove info_spec
rename observation_central
move manual_batch
follow other file's format
add 01 drones w/ cams demo, fix cams
clean up import
move ray caster and import
new track from devel branch

unify with drone_state
add run test (tmuxp)

The first PR is Support Isaac Sim 4.0.
The second PR is Fix demos & Clean warnings.
The third PR is Fix tasks.
The fourth PR is Add run_task and wandb trained model
The fifth PR is Merge release branch
Please review it in the order of PRs and commits.

OMG. Thank you so much for your effort. For some unknown reason, my email associated with GitHub notification went silent months ago. I will begin to go over these PRs immediately.


Don't generate releasenotes.md

I've tested the executable in OSX and Windows eg: GitReleaseNotes.exe . /OutputFile ReleaseNotes.md
I runned it under a git repository, (github and bitbucket (without using credentials)).
The ReleaseNotes.md don't event gets generated nor I get any feedback/error.

Very interested in this fix as I too cannot see any releasenotes.md file being generated.

@holytshirt same issue here!

same...

@jzeferino @dazinator @ianceicys @holytshirt @PatrickGrub @CRuppert can you all have a play with the newly released version, and see if this has been resolved?

@gep13 I couldn't find the update on chocolatey.
https://chocolatey.org/packages/GitReleaseNotes.Portable
I am currently running via the chocolatey package. Any chance this could be released there? Or do we now get it from nuget?

Bump

If anyone is interested in testing a trimmed down version from this tool (only support for GitHub git-repositories), take a look at:
https://github.com/StefH/GitHubReleaseNotes
Please create an issue if you experience any troubles.

I use the Octokit.GitHubClient (which uses the free / throtled api from GitHub) to connect to repo.
I never tested on private repo's because I dont have.
Can you please make an issue in my project and provide me with some details? And maybe you can create a dummy private repo and give me access so I can test it?


StreamIterator Throws Confusing Error On End Of Stream

Before null safety, if you iterate to the end of the stream and then call current, you'll get null. After the null safety update, the library attempts to return null even though the return type is non-nullable causing a TypeErrror. This is undesirable because it obfuscates the error from the user. See stream_impl.dart#L988.
Example code:
    var streamIter = StreamIterator(Stream.fromIterable([1]));
    await streamIter.moveNext();
    print(streamIter.current);
    await streamIter.moveNext();
    // Calling `current` after the end of stream has been reached.
    print(streamIter.current);

Before null-safety:
1

null

After null-safety:
1

type 'Null' is not a subtype of type 'int'

Dart Version:
$ dart --version
Dart SDK version: 2.12.0 (stable) (Thu Feb 25 19:50:53 2021 +0100) on "windows_x64"

Possible solutions:

Make the return type type nullable to retain pre null-safety behavior
Throw an informative error that explains to the user that current was called after end of stream

If the type of current is nullable, then it actually returns null. That way you can preserve the pre-null-safety behavior.
We have deliberately tried to avoid adding an unnecessary overhead to reading current. The correct use is to never read current unless moveNext returned true. That means that the error should only be seen while debugging. I admit that it is not easily deciphered, but it's also not a use-case we want to penalize valid code for.

@lrhn Ah that makes sense. What about adding end-of-stream logic to check if T is nullable and throw an error with a helpful message if it's not (idk how straightforward this is, a sloppy solution would be trying to cast null as T in a try catch block)? This won't add any overhead to valid code (when users are properly only calling current after calling and checking moveNext) but will save users considerable time when debugging?
Food for thought, feel free to close this PR if you feel the above is not worth implementing.

A  try/catch block is actually an overhead. What we currently do is the most efficient way we could find to return null if the type is nullable and throw if it's not, while adding minimal overhead on all the valid calls and not adding extra fields to check.
We could have added a boolean hasValue instead, and used that for extra logic when there is no value, but then the internal _current variable would still need to be nullable because it has no value before the first moveNext, and we'd need an as T cast anyway (or let the _current variable be late, which has its own overhead).
I don't remember the exact implementation, but something T get current => _current ?? (null as T); seems hard to beat when _current must be nullable anyway. I'd prefer T get current => _current as T; if the compilers were better at optimizing something-T? as T. That really should be the idiom to use.

The try..catch is overhead, but as long as it's included after the line if (_hasValue) return _stateData as dynamic; it'll only be incurred in the invalid code path (calling current when (_hasVluae is false) right?
So the following implementation should be the best of both worlds in that it is performant in the valid case and informative in the invalid case:
  T get current {
    if (_hasValue) return _stateData as dynamic;
    try {
      return null as dynamic;
    } on TypeError {
      throw <insert applicable exception with helpful message>;
    }
  }

The implicit down-cast is omitted when compiled to JavaScript, but not when running natively.
The optimal approach is return _stateData as T. would probably be:
T get current {
  var value = _stateData;
  if (_hasValue) return value as dynamic;
  if (value is T) return value;
  throw StateError("No value and iterator type is not nullable");
}

It's an overhead, but only on the rare case of reading current when there is no current value (which you shouldn't do).

Cleaning up my old open issues and coming back to this one.
The above looks like a good solution to me. It doesn't add overhead to proper usage, and makes it easier for developers to debug mistakes. I could open a PR for this, but I really don't understand the Dart type system and compilation to JS enough to feel confident I was submitting good code.
Should I open a PR on this, or would you be up to doing so? Or, while a definite improvement to the current behavior, do we consider this change not to be worthwhile and thus we should just close the issue?


docs: Add section for independent agent execution

Introduced a new section titled "Running an Agent Independently" to the agents documentation.
Provided a detailed example of executing tasks with agents outside a crew context.
Included a table explaining key parameters for independent execution.
Documented the internal workflow of the execute_task method for clarity.

Disclaimer: This review was made by a crew of AI Agents.
Code Review Comment for PR #1769: Add Section for Independent Agent Execution
Summary
This pull request adds a new section for independent execution of agents within the docs/concepts/agents.mdx file. While the changes are structured well and provide clarity, there are several opportunities to enhance both the content and usability of the documentation.
Code Quality Findings

Code Example Improvements:

The proposed code example is a solid foundation but could greatly benefit from incorporating comprehensive error handling. For instance, specify edge cases more thoroughly to prepare users for real-world applications:try:
    # Your code...
except (TimeoutError, ValueError) as e:
    print(f"Error: {str(e)}. Please check your inputs.")

Including type hints for function parameters would improve readability and usability:def execute_task(task: Task, context: Dict[str, Any]) -> Any:

Documentation Sufficiency:

Missing Sections: Consider adding a "Best Practices" section regarding independent agent usage, which may cover timeout settings and memory considerations.
Expanded Parameters Table: Enhance the Key Parameters table by detailing more about the optional parameters and default values:| Parameter         | Type                          | Required | Description                                        |
|------------------|-------------------------------|----------|----------------------------------------------------|
| `max_retries`    | `int`                         | No       | Number of retry attempts (default: 3)              |
| `timeout`        | `float`                       | No       | Task execution timeout, in seconds.                |

Historical Context & Recommendations
In reviewing related files such as the tasks.mdx documentation, which outlines task structures and how they interact with agents, ensuring alignment with this current PR is essential. Cross-referencing with agents_creation.mdx can also confirm that the configurations mentioned complement this newly introduced functionality.
Furthermore, it is advisable to include integration examples with external systems for clarity and completeness. Prior PRs that involved agent capabilities or error handling can serve as references; a review of recent PR history may uncover relevant patterns or changes that would enhance user experience.
Security Considerations
Please document best practices regarding sensitive data and API key handling, especially when using language models or third-party integrations. Include caution regarding API rate limits.
Overall Assessment
This documentation addition represents a significant improvement in understanding the capabilities of independent agents within CrewAI. With some enhancements regarding examples, completeness in documentation, and robust testing of edge cases, the utility and robustness of this section will be greatly enhanced.
Action Items

Implement the suggested code improvements, especially focusing on error handling and type hints.
Enhance parameter documentation for clarity.
Add the recommended sections for best practices and troubleshooting.
Review the historical context to align this documentation effectively with existing resources.

The changes are approved with recommendations for improved clarity and completeness to further enhance user understanding and guide effective usage of independent agent capabilities.

Hey @apappascs!
Unfortunately at this time, we want to keep agents inside of Crews.
At a later date, we might make this a more well known and more supported feature. But for now, we're keeping agents in a crew.


AWS: Query by subnet-id, if we have it

If we are matching an existing subnet-id, query using that subnet-id!
(The old code was a little odd)

As well as being "more correct", this is also helpful/required for #10751.

LGTM.


Adds new option to allow configuration of binlog expire

Issue
Solution

@sombrafam as discussed offline, we understand the need for a sane default here. Discussion is open internally and we will come back to this when time allow us. For now, I'm closing it.


Activate JSX support for ESLint

Hello,
If I set your default preset in eslintrc.json which can be seen below:
{
  "extends": [
    "eslint:recommended",
    "plugin:react/recommended"
  ]
}

then I don't need to set manually the JSX support for ESLint in the parserOptions property in .eslintrc.json as it can be seen below?
{
  "parserOptions": {
    "ecmaFeatures": {
      "jsx": true
    }
  }
}

Or I must do both things in order to set the JSX support?
Regards,
Néstor Llamas

That's correct, as you can see here: https://github.com/yannickcr/eslint-plugin-react/blob/master/index.js#L128


@aws_api_key Not applying to generated List queries or mutations

Describe the bug
When API Key is set as an additional auth method to a graphql type, it's auto generated queries and mutations associated with the type fail with an access denied error when called with an access key.
Amplify CLI Version
amplify version is 4.39.0 on macos
To Reproduce
Create a demo amplify project following these instructions:
https://docs.amplify.aws/start/getting-started/setup/q/integration/js
When you get to the amplify add api step, pick the following options:
amplify add api
? Please select from one of the below mentioned services: GraphQL
? Provide API name: amplifyjsapp
? Choose the default authorization type for the API IAM
? Do you want to configure advanced settings for the GraphQL API Yes, I want to make some additional changes.
? Configure additional auth types? Yes
? Choose the additional authorization types you want to configure for the API API key
API key configuration
? Enter a description for the API key: backendaccess
? After how many days from now the API key should expire (1-365): 7
? Configure conflict detection? No
? Do you have an annotated GraphQL schema? No
? Choose a schema template: Single object with fields (e.g., “Todo” with ID, name, description)
The Change the graphql schema file to the following:
type Todo @model @aws_api_key {
  id: ID!
  name: String!
  description: String
}

Now do an amplify push.  Once done go to the console:
amplify console api
select authorisation provider as api key and try to run the following query:
query MyQuery {
  listTodos {
    items {
      id
      name
    }
  }
}

This results in an unexpected error:
Not Authorized to access listTodos on type Query
Expected behavior
I expect to be able to run the listTodos query without error
Desktop (please complete the following information):

OS: Mac
Node Version: v14.15.1

@rewardle-jason You'd need to use the @auth directive - @auth(rules: [{ allow: public }]) in the schema for your model for all the associated  queries, and mutations  tied to your model.


docs: ShareX Setup

{
  "Version": "15.0.0",
  "Name": "Share Me",
  "DestinationType": "ImageUploader",
  "RequestMethod": "POST",
  "RequestURL": "https://Share-Me-Domain.com/api/posts",
  "Headers": {
    "Accept": "*/*",
    "Authorization": "YourTokenHere"
  },
  "Body": "MultipartFormData",
  "FileFormName": "files",
  "URL": "https://Share-Me-Domain.com/posts/{json:id}"
}

How to add

Right click system tray icon
Destinations
Custom Uploader Settings
(1) Import
Paste from Clipboard
(2) Ensure Method is POST
(3) Change Domain in Request URL
(4) Add API Token received from Share-Me
(5) Update Domain URL with Share-Me Domain (can be made a variable if PostURL is an api response field 👀, one less thing to change)
(6) Change Image Uploader to Share-Me
Close window

Thanks so much! I am currently working on the documentation page with Next.js on 32-docs-add-docusaurus-project-with-installation-configuration-and-usage-instructions and will include your instructions for the ShareX integration!


[SPIKE] Private DNS resolver rules

Task
Investigate how the resolver rules for private DNS will be implemented. The customer provided an example script which shows the expected behavior of the operator.
Questions

What AWS resources need to be created by the operator?
What's the AWS policy required by this feature?
How can multiple people work on developing the operator?

Outcome

[ ] list of task or user stories in order to develop the operator incrementally to enable the team to work in parallel;
[x] list of AWS resources which need to be created by the operator;
[x] the AWS policy required by this feature.

No actual code is needed.
Resources

https://github.com/giantswarm/roadmap/issues/1709;
Example Terraform script with AWS resolver rules for private DNS.

# thunder VPC ID
vpc-0d35bc02d0fe39e9a

## thunder subnet IDs
subnet-00608029a8fdc4586
subnet-0687cbee901bdd73a
subnet-0bcea31b9854fc197

# thunder hosted zone ID
Z00105772URQM1433AZX9

# TODO
associate-resolver-rule

## EXECUTE
export CLUSTER=thunder
export NOC_AWS_ACCOUNT_ID=496192190985

# create Security group for the resolver endpoints
SECURITY_GROUP_ID=$(aws ec2 create-security-group --description "Security group for resolver rule endpoints" --group-name ${CLUSTER}-rr-endpoints2 --vpc-id vpc-0d35bc02d0fe39e9a | jq -r .GroupId)

# create ingress rule for the security group
aws ec2 authorize-security-group-ingress --group-id ${SECURITY_GROUP_ID} --protocol udp --port 53
aws ec2 authorize-security-group-ingress --group-id ${SECURITY_GROUP_ID} --protocol tcp --port 53

# create INBOUND resolver endpoint
RR_INBOUND_ID=$(aws route53resolver create-resolver-endpoint --name ${CLUSTER}-inbound --direction INBOUND --security-group-ids $SECURITY_GROUP_ID --ip-addresses SubnetId=subnet-00608029a8fdc4586 SubnetId=subnet-0687cbee901bdd73a SubnetId=subnet-0bcea31b9854fc197 --creator-request-id "$(date)" | jq -r .ResolverEndpoint.Id)

# create OUTBOUND resolver endpoint
RR_OUTBOUND_ID=$(aws route53resolver create-resolver-endpoint --name ${CLUSTER}-outbound --direction OUTBOUND --security-group-ids $SECURITY_GROUP_ID --ip-addresses SubnetId=subnet-00608029a8fdc4586 SubnetId=subnet-0687cbee901bdd73a SubnetId=subnet-0bcea31b9854fc197 --creator-request-id "$(date)" | jq -r .ResolverEndpoint.Id)

# get resolver rule inbound IPs,
RR_INBOUND_IP_ADDRESES=$(aws route53resolver list-resolver-endpoint-ip-addresses --resolver-endpoint-id ${RR_INBOUND_ID} | jq -r .IpAddresses[].Ip |awk '{print "Ip="$1}')

# create Resolver rule
RR_ID=$(aws route53resolver create-resolver-rule  --creator-request-id "$(date)" --name giantswarm-${CLUSTER} --rule-type FORWARD --domain-name ${CLUSTER}.eu-central-1.aws.tkp.hdi.cloud --resolver-endpoint-id ${RR_OUTBOUND_ID} --target-ips $RR_INBOUND_IP_ADDRESES | jq -r .ResolverRule.Id)

RR_ARN=${aws route53resolver get-resolver-rule --resolver-rule-id ${RR_ID} | jq -r .ResolverRule.Arn}

# create resource share item
# in aws go SDK this might be 3 separate operations - create RAM share resource, associate resolver rule ARN with the RAM share resource, associate AWS NOC account principal with the RAM share resource

aws ram create-resource-share --name giantswarm-${CLUSTER}-rr --resource-arns ${RR_ARN} --principals ${NOC_AWS_ACCOUNT_ID} --allow-external-principals

# asociate resolver rule with NOC VPC in the NOC account
# execute with NOC role indicated by other AWS profile - `thunder-noc`
# aws sts assume-role --role-session-name giantswarm-cli-test-vaclav --role-arn arn:aws:iam::496192190985:role/TKP_Assoc_Route53_ResolverRule --external-id SECRET_FILL
aws --profile thunder-noc route53resolver associate-resolver-rule --resolver-rule-id $RR_ID --name giantswarm-${CLUSTER}-rr-asociation --vpc-id vpc-0ffbf3d2258f96ba4

I have an initial implementation here https://github.com/giantswarm/aws-resolver-rules-operator/pull/6
I have not tested it yet, only through automated testing. Also, the logic when a Cluster is deleted is not implemented yet. If no one picks it up in the next two weeks, I can resume my work when I'm back from holidays.

Blocked until @fiunchinho is back.

We continue here https://github.com/giantswarm/roadmap/issues/1709


Sometimes a different image shows when clicking on a thumbnail. Intended image is also not the one selected for download.

What steps will reproduce the problem?
This only happens sometimes, but the last time it did, I unchecked the "Ignore images containing a blacklisted tag", and then the right image loaded when clicking the thumbnail. The blacklisted tag that seemed to trigger it that time was newhalf.
What is the expected behavior? What do you get instead?
The expected behavior when clicking or selecting a thumbnail in the search view is that the image the thumbnail is based on will load, but instead, sometimes a different image loads. This also applies to selecting the images for download. When I "Ctrl + Click" the thumbnail and click "Get selected," the wrong image is what is put in the download list, and the intended image is not.
Re-enactment of the issue behavior:
1.

Update September 4 2018:
Video of this happening, along with the thumbnail stacking bug. The topic of this issue happens in the latter half of the video:
https://youtu.be/a2WAwxODQvM
How often does this problem occur? On which sources?
It only occurs sometimes, and I usually notice it if I'm a few pages in deep through a search query. Last time it happened was on Danbooru and Gelbooru combined search.
What version of the program are you using? On what operating system?
Grabber 5.5.2 on Windows 10.
Please provide any additional information below
I remember this has happened before as well. I'm not sure if it's because of ignored images, or because of problems when loading the thumbnails in the search view, but those are both just guesses.

Here is a link to a video of this bug that I also linked in Issue #1408
https://youtu.be/a2WAwxODQvM

I also got this issue. I click on a different thumbnail but for some reason, the first image I loaded is the one that gets loaded on the next set of clicks I do. It's also automatically downloaded into my grabber folder


Add children to the props for @types/react@18

In relation to React props should not do anything special with children and DefinitelyTyped have updated the react types to v18
https://github.com/DefinitelyTyped/DefinitelyTyped/issues/59802

Also waiting for that! 👍🏼

2 approves and still complaining of unable to merge until an approved approver approves...

Released in v0.11.1.


deploy azure function

Does this occur consistently?
Repro steps:

Action: azureFunctions.deploy
Error type: BadRequest
Error Message: Encountered an error (InternalServerError) from host runtime.
Version: 1.0.1
OS: win32
OS Release: 10.0.19041
Product: Visual Studio Code
Product Version: 1.51.1
Language: en

Call Stack
new RestError extension.bundle.js:46:109835
extension.bundle.js:48:167633extension.bundle.js:48:167633
processTicksAndRejections task_queues.js:94:5

Please provide detailed repro steps.


Improving message for - ArgumentOutOfRangeException

Trying to solve issue - https://github.com/MicrosoftResearch/Naiad/issues/19

Hi team, I am trying to contribute for issue 0 https://github.com/MicrosoftResearch/Naiad/issues/19

Hi team, I am trying to solve issue mentioned in - #19 . So raised this PR.

Hello! Unfortunately, the team members no longer work for Microsoft (circa 2014, actually). Not sure the best way to put you in touch with the maintainers of the repo, but I think the project is inert for now.


[Help Wanted] Cannot sign in to leetcode-cn

Get http error [code=500] when logining leetcode-cn, but I can log in normally on the webpage.
Is it an error caused by a change in the api of leetcode-cn?
The output information:
v8.9.3
 Active  Name               Version         Desc
----------------------------------------------------------------------------------------------------
   ✔     solution.discuss   2019.02.03      Plugin to fetch most voted solution in discussions.
   ✔     company            2017.12.18      Plugin to query by company for free user.
   ✔     cache              default         Plugin to provide local cache.
   ✔     retry              default         Plugin to retry last failed request if autologin.enable is on.
   ✔     leetcode.cn        2018.11.25      Plugin to talk with leetcode-cn APIs.
   ✔     leetcode           default         Plugin to talk with leetcode APIs.
 Active  Name               Version         Desc
----------------------------------------------------------------------------------------------------
   ✔     solution.discuss   2019.02.03      Plugin to fetch most voted solution in discussions.
   ✔     company            2017.12.18      Plugin to query by company for free user.
   ✔     cache              default         Plugin to provide local cache.
   ✔     retry              default         Plugin to retry last failed request if autologin.enable is on.
   ✔     leetcode.cn        2018.11.25      Plugin to talk with leetcode-cn APIs.
   ✔     leetcode           default         Plugin to talk with leetcode APIs.
 Active  Name               Version         Desc
----------------------------------------------------------------------------------------------------
   ✔     solution.discuss   2019.02.03      Plugin to fetch most voted solution in discussions.
   ✔     company            2017.12.18      Plugin to query by company for free user.
   ✔     cache              default         Plugin to provide local cache.
   ✔     retry              default         Plugin to retry last failed request if autologin.enable is on.
   ✔     leetcode.cn        2018.11.25      Plugin to talk with leetcode-cn APIs.
   ✔     leetcode           default         Plugin to talk with leetcode APIs.
 Active  Name               Version         Desc
----------------------------------------------------------------------------------------------------
   ✔     solution.discuss   2019.02.03      Plugin to fetch most voted solution in discussions.
   ✔     company            2017.12.18      Plugin to query by company for free user.
   ✔     cache              default         Plugin to provide local cache.
   ✔     retry              default         Plugin to retry last failed request if autologin.enable is on.
   ✔     leetcode.cn        2018.11.25      Plugin to talk with leetcode-cn APIs.
   ✔     leetcode           default         Plugin to talk with leetcode APIs.
[ERROR] You are not login yet?
login: pass: - Signing in leetcode.com
[ERROR] http error [code=500]

Update by @jdneo:
I tried it locally, seems it has been fixed from the leetcode-cn side. Is there anyone still having this issue?
Please let us know if it has been fixed at your side.

Added by @jdneo:
Thank you all for reporting the issue. If you met this issue as well. You can simply click the 👍 icon to vote up for it. It's not recommended to add comments like +1, me too, since it will cause the thread too long and make some of the key information hard to find.
So far @poppinlp and I will take a look to it. If anyone is willing to give some help, feel free to let us know.
Thanks @poppinlp in advance.

meet this too

when login, get [ERROR] http error [code=500] too
and change the internet to my hot spot，still 500

- Downloading maximum-product-subarray
[ERROR] http error [code=500]

meet this too

I meet it too

me too

me too

v10.15.3
 Active  Name               Version         Desc
----------------------------------------------------------------------------------------------------
   √     solution.discuss   2019.02.03      Plugin to fetch most voted solution in discussions.
   √     company            2017.12.18      Plugin to query by company for free user.
   √     cache              default         Plugin to provide local cache.
   √     retry              default         Plugin to retry last failed request if autologin.enable is on.
   √     leetcode.cn        2018.11.25      Plugin to talk with leetcode-cn APIs.
   √     leetcode           default         Plugin to talk with leetcode APIs.
 Active  Name               Version         Desc
----------------------------------------------------------------------------------------------------
   √     solution.discuss   2019.02.03      Plugin to fetch most voted solution in discussions.
   √     company            2017.12.18      Plugin to query by company for free user.
   √     cache              default         Plugin to provide local cache.
   √     retry              default         Plugin to retry last failed request if autologin.enable is on.
   √     leetcode.cn        2018.11.25      Plugin to talk with leetcode-cn APIs.
   √     leetcode           default         Plugin to talk with leetcode APIs.
 Active  Name               Version         Desc
----------------------------------------------------------------------------------------------------
   √     solution.discuss   2019.02.03      Plugin to fetch most voted solution in discussions.
   √     company            2017.12.18      Plugin to query by company for free user.
   √     cache              default         Plugin to provide local cache.
   √     retry              default         Plugin to retry last failed request if autologin.enable is on.
   √     leetcode.cn        2018.11.25      Plugin to talk with leetcode-cn APIs.
   √     leetcode           default         Plugin to talk with leetcode APIs.
 Active  Name               Version         Desc
----------------------------------------------------------------------------------------------------
   √     solution.discuss   2019.02.03      Plugin to fetch most voted solution in discussions.
   √     company            2017.12.18      Plugin to query by company for free user.
   √     cache              default         Plugin to provide local cache.
   √     retry              default         Plugin to retry last failed request if autologin.enable is on.
   √     leetcode.cn        2018.11.25      Plugin to talk with leetcode-cn APIs.
   √     leetcode           default         Plugin to talk with leetcode APIs.
 Premium   User                 Host
------------------------------------------------------------
    ×      xnouhz               https://leetcode-cn.com
- Downloading problems
[ERROR] http error [code=500]
login:

Me too...
Is the plugin is forbidded?

我也是......

me too

Maybe leetcode upgrade its API? I met the error about 8 hours ago, wish the author can fix the bug soon~~~

Hey guys,
Thank you for reporting the issue. I'll try ti figure out what's going on. But I'm busying with my work recently. It would be great if we can have any volunteer to help.

Maybe this is an error from 'leetcode-cli' ?

me too

+1

me too, can't login in leetcode-cn, wish the author can fix the bug soon~

me too
Active  Name               Version         Desc
√     solution.discuss   2019.02.03      Plugin to fetch most voted solution in discussions.
√     company            2017.12.18      Plugin to query by company for free user.
√     cache              default         Plugin to provide local cache.
√     retry              default         Plugin to retry last failed request if autologin.enable is on.
×     leetcode.cn        2018.11.25      Plugin to talk with leetcode-cn APIs.
√     leetcode           default         Plugin to talk with leetcode APIs.
[ERROR] You are not login yet?
login:  Active  Name               Version         Desc
√     solution.discuss   2019.02.03      Plugin to fetch most voted solution in discussions.
√     company            2017.12.18      Plugin to query by company for free user.
√     cache              default         Plugin to provide local cache.
√     retry              default         Plugin to retry last failed request if autologin.enable is on.
√     leetcode.cn        2018.11.25      Plugin to talk with leetcode-cn APIs.
√     leetcode           default         Plugin to talk with leetcode APIs.
Successfully logout as null-228
login: pass: - Signing in leetcode.com
[ERROR] http error [code=500]

Sry for all guys.
@jdneo I'll check it tonight.

Thank you @poppinlp. I'll also take a look if I have time tonight. Thank you for offering the help.

it works now

httpcode:500, Seems leetcode-cn.com's  website internal error?

我也是
mee too

I tried it locally, seems it has been fixed from the leetcode-cn side. Is there anyone still having this issue?
Please let us know if it has been fixed at your side.

Already fixed! Thanks~

Hey guys,
Thank you for reporting the issue. I'll try to figure out what's going on. But I'm busying with my work recently. It would be great if we can have any volunteer to help.

thanks, fixed

Thank you all. I'm closing this issue since the issue should already fixed.
Happy coding!

i am facing this issue
anyone help me sort it out?


Bug: Failed to download native client for Raspbian

Bug report

Installation fails with a 404 - not found message when downloading native binary. I've added a verbose flag to curl, see the output below:
Looking for a previous installation of SDKMAN...
Looking for unzip...
Looking for zip...
Looking for curl...
Looking for sed...
Installing SDKMAN scripts...
Create distribution directories...
Getting available candidates...
Prime the config file...
Installing script cli archive...
* Downloading...
############################################################################################################################################################################################################## 100.0%
* Checking archive integrity...
* Extracting archive...
* Copying archive contents...
* Cleaning up...

* Downloading...
*   Trying <IP_ADDRESS>:443...                                                           #                #                #               #
* Connected to api.sdkman.io (<IP_ADDRESS>) port 443 (#0)
* ALPN, offering h2                                                                            #                #               #               #
* ALPN, offering http/1.1
* successfully set certificate verify locations:
*  CAfile: /etc/ssl/certs/ca-certificates.crt
*  CApath: /etc/ssl/certs
} [5 bytes data]
* TLSv1.3 (OUT), TLS handshake, Client hello (1):
} [512 bytes data]
* TLSv1.3 (IN), TLS handshake, Server hello (2):                                                      #               #               #               #
{ [122 bytes data]
* TLSv1.3 (IN), TLS handshake, Encrypted Extensions (8):
{ [25 bytes data]
* TLSv1.3 (IN), TLS handshake, Certificate (11):
{ [4578 bytes data]
* TLSv1.3 (IN), TLS handshake, CERT verify (15):
{ [264 bytes data]
* TLSv1.3 (IN), TLS handshake, Finished (20):
{ [52 bytes data]
* TLSv1.3 (OUT), TLS change cipher, Change cipher spec (1):
} [1 bytes data]
* TLSv1.3 (OUT), TLS handshake, Finished (20):
} [52 bytes data]
* SSL connection using TLSv1.3 / TLS_AES_256_GCM_SHA384
* ALPN, server accepted to use http/1.1
* Server certificate:
*  subject: CN=*.sdkman.io
*  start date: Apr 10 00:00:00 2022 GMT
*  expire date: Apr 28 23:59:59 2023 GMT
*  subjectAltName: host "api.sdkman.io" matched cert's "*.sdkman.io"
*  issuer: C=GB; ST=Greater Manchester; L=Salford; O=Sectigo Limited; CN=Sectigo RSA Domain Validation Secure Server CA
*  SSL certificate verify ok.
} [5 bytes data]                                                                                #                #               #              #
> GET /2/broker/download/native/install/0.0.15/linuxarm32hf HTTP/1.1
> Host: api.sdkman.io
> User-Agent: curl/7.74.0
> Accept: */*
>
{ [5 bytes data]
* TLSv1.3 (IN), TLS handshake, Newsession Ticket (4):
{ [265 bytes data]
* TLSv1.3 (IN), TLS handshake, Newsession Ticket (4):
{ [265 bytes data]
* old SSL session ID is stale, removing
{ [5 bytes data]                                                                                #               #               #              #
* Mark bundle as not supporting multiuse
* The requested URL returned error: 404 Not Found
* Closing connection 0
} [5 bytes data]
* TLSv1.3 (OUT), TLS alert, close notify (256):
} [2 bytes data]
curl: (22) The requested URL returned error: 404 Not Found

'curl -v --fail --location --progress-bar "$download_url" > "$sdkman_zip_file"': command failed with exit code 22.

To reproduce

On a Raspberry Pi 2 Model B, running raspbian newest raspbian (see OS version below), try running install script $ curl -s https://get.sdkman.io | bash, notice the 404 error.
System info

System:
GNU/Linux
uname -a
Linux raspberrypi 5.15.61-v7+ #1579 SMP Fri Aug 26 11:10:59 BST 2022 armv7l GNU/Linux

Shell
Bash:
bash --version
GNU bash, version 5.1.4(1)-release (arm-unknown-linux-gnueabihf)
Copyright (C) 2020 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>

Hi @tarczynskitomek!
Because we are currently rewriting parts of SDKMAN! in Rust, and GitHub Actions doesn't provide ARM runners yet, parts of SDKMAN! may not work on ARM anymore.
I see two (three?) options:

provide a "legacy" version which doesn't rely on Rust binaries (yet)
use the Oracle Cloud free tier to setup an ARM GH Runner and provide ARM native binaries (@aalmiray did this for JReleaser if I remember correctly).

Wdyt @marc0der?

I have not done it yet for JReleaser for Michael Simons did for neo4j-migrations https://info.michael-simons.eu/2022/10/15/use-oracle-cloud-infrastructure-compute-instances-as-custom-github-runners/

Had the same issue, it could not find https://api.sdkman.io/2/broker/download/native/install/0.0.15/linuxarm32hf. I went on and modified the script locally to remove the hf at the end and then it worked ...

pi@raspberrypi:~ $ curl -s "https://beta.sdkman.io" | bash
-+syyyyyyys:
/yho: -yd.
/yh/ +m.
.oho. hy . .sh/ :N -/o +dyyo:.
.yh: M- -/osysoym :hs -+sys: hhyssssssssy+ .sh: N: ms/-`` yy.yh- -hy. .N-````````+N.
od/ N- -/oM- ddd+ sd: hNNm -N: :do .M. dMMM- ms. /d+ NMMs do
.yy- :N```mMMM. - -hy. /MMM: yh+d+ :/oo/ -/osyh/ossssssdNMM .sh: yMMN /m.
-dh- :ymNMMMMy -/shmNm-:N/-. `.sN /N- `NMMy .m/ `oNs` -hysosmMMMMydmNmds+-.:ohm : sd` :MMM/ yy .hN+ /d: -MMMmhs/-.` .MMMh .ss+- `yy` sMMN` :N. :mN/ `N/ `o/-` :MMMo +MMMN- .` `ds mMMh do /NN/ `N+....--:/+oooosooo+:sMMM: hMMMM: `my .m+ -MMM+ :N. /NMo -+ooooo+/:-....`...:+hNMN. `NMMMd` .MM/ -m: oMMN. hs -NMd` :mm -MMMm- .s/ -MMm. /m- mMMd -N. `mMM/ .- /MMh. -dMo -MMMy od. .MMMs..---yh +MMM. sNo`.sNMM+ :MMMM/ sh`+MMMNmNm+++- mMMM- /--ohmMMM+ :MMMMm. `hyymmmdddo MMMMh. ```` `-+yy/`yMMM/ :MMMMMy -sm:...-:-.`
dMMMMmo-.``````..-:/osyhddddho. `+shdh+. hMMM: :MmMMMM/ ./yy/` `:sys+/+sh/
.dMMMMMMmdddddmmNMMMNNNNNMMMMMs sNdo- dMMM- `-/yd/MMMMm-:sy+. :hs- /N`
`/ymNNNNNNNmmdys+/::----/dMMm: +m- mMMM+ohmo/.` sMMMMdo- .om: `sh
`.-----+/.` `.-+hh/` `od. NMMNmds/ `mmy:` +mMy `:yy.
/moyso+//+ossso:. .yy` `dy+:` .. :MMMN+---/oys:
/+m: `.-:::-` /d+ +MMMMMMMNh:`
+MN/ -yh. `+hddhy+.
/MM+ .sh:
:NMo -sh/
-NMs `/yy:
.NMy `:sh+.
`mMm` ./yds-
`dMMMmyo:-.````.-:oymNy:`
+NMMMMMMMMMMMMMMMMms:`
-+shmNMMMNmdy+:`
Now attempting installation...
Looking for a previous installation of SDKMAN...
Looking for unzip...
Looking for zip...
Looking for curl...
Looking for sed...
Installing SDKMAN scripts...
Create distribution directories...
Getting available candidates...
Prime the config file...
Installing script cli archive...

Downloading...
######################################################################## 100.0%
Checking archive integrity...
Extracting archive...
Copying archive contents...
Cleaning up...
Downloading...
curl: (22) The requested URL returned error: 404 Not Found
'curl --fail --location --progress-bar "$download_url" > "$sdkman_zip_file"': command failed with exit code 22.


Possible regression bugs from 6.8.0 to 6.9.X

Snakemake version: 6.9.0 and 6.9.1
Describe the bug

Running/Executing pipelines with the --wrapper-prefix and a local git repository works in 6.8.0 but not in 6.9.0 or 6.9.1
Note this wrapper repository is not the snakemake wrapper but a local repository.
If the input is a function, the code does not seem to work either (same versions)

Logs
Traceback (most recent call last):
  File "/home/cokelaer/miniconda3/envs/sequana/lib/python3.7/site-packages/snakemake/executors/__init__.py", line 593, in _callback
    raise ex
  File "/home/cokelaer/miniconda3/envs/sequana/lib/python3.7/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/home/cokelaer/miniconda3/envs/sequana/lib/python3.7/site-packages/snakemake/executors/__init__.py", line 579, in cached_or_run
    run_func(*args)
  File "/home/cokelaer/miniconda3/envs/sequana/lib/python3.7/site-packages/snakemake/executors/__init__.py", line 2460, in  run_wrapper
    raise ex
  File "/home/cokelaer/miniconda3/envs/sequana/lib/python3.7/site-packages/snakemake/executors/__init__.py", line 2441, in run_wrapper
    runtime_sourcecache_path,
  File "/home/cokelaer/Work/github/sequana_pipelines/fastqc/test/data/fastqc/fastqc.rules", line 219, in __rule_rulegraph
  File "/home/cokelaer/miniconda3/envs/sequana/lib/python3.7/site-packages/snakemake/wrapper.py", line 140, in wrapper
    runtime_sourcecache_path,
  File "/home/cokelaer/miniconda3/envs/sequana/lib/python3.7/site-packages/snakemake/script.py", line 1329, in script
     path, SourceCache(runtime_sourcecache_path), basedir, wildcards, params
  File "/home/cokelaer/miniconda3/envs/sequana/lib/python3.7/site-packages/snakemake/script.py", line 1249, in get_source
    path = infer_source_file(format(path, wildcards=wildcards, params=params))
  File "/home/cokelaer/miniconda3/envs/sequana/lib/python3.7/site-packages/snakemake/utils.py", line 430, in format
    return fmt.format(_pattern, *args, **variables)
  File "/home/cokelaer/miniconda3/envs/sequana/lib/python3.7/string.py", line 186, in format
    return self.vformat(format_string, args, kwargs)
  File "/home/cokelaer/miniconda3/envs/sequana/lib/python3.7/string.py", line 190, in vformat
    result, _ = self._vformat(format_string, args, kwargs, used_args, 2)
  File "/home/cokelaer/miniconda3/envs/sequana/lib/python3.7/string.py", line 200, in _vformat
    self.parse(format_string):
  File "/home/cokelaer/miniconda3/envs/sequana/lib/python3.7/string.py", line 284, in parse
    return _string.formatter_parser(format_string)
TypeError: expected str, got NoneType

Minimal example
This pipeline  https://github.com/sequana/fastqc/sequana_pipelines/fastqc.rules used to work a week ago with version 6.1 and works right now with 6.8.0. You'll need input data so a bit tricky but I believe this is a general issue.
Additional context

Is this fixed in recent releases?

I'm still seeing the problem in 7.8.5

Hi, sorry for missing this. Can you provide a minimal test case via a pull request? Also, please ping me on this via discord in case I miss the PR (I get a lot of github notifications, to many to parse them all).

I don't have the time to make a minimal test case right now, maybe you could add an help need label so that someone can create this? It shouldn't be too hard with all of pangolin being open source so one can go back and create minimal test case.

@cokelaer could you create a minimal example?

@johanneskoester
looks like it works with version 7.12.0 and 7.18.0 so I believe we can close the issue. Nevertheless, here is a simple example in case it is usefule for you.
samples = {'data': 'data.fastq'}
def getrawdata():
    return lambda wildcards: samples[wildcards.sample]

rule pipeline:
    input: expand("samples/{sample}/fastqc.log", sample=samples)

rule fastqc:
    input: getrawdata()
    output: "samples/{sample}/fastqc.done"
    log:
        "samples/{sample}/fastqc.log"
    threads:
        4
    params:
        options="",
        working_directory="samples/{sample}"
    wrapper:
        "main/wrappers/fastqc"

onsuccess:
    print('awesome')

You will need just one input file to be saved in a filename called data.fastq:
@HISEQ:426:C5T65ACXX:5:2301:5633:7203 1:N:0:GTGAAA
TTGCTTAATTCTATACTCATCCTCTACAGGGAGTTGGCAAGATTCAAAGACAACCAAAGAAGTCAACAAGGGATGTTCCACGCTTACCCCGTATTGGTAAG
+
CCCFFFFFHHHHHJJJIIHIHIJJJJJJJJI@HEHIJJJJIIJJJJIAHIIJIIJJIJIIJJIIHGIIIIDHHEHEDDFEFDDDDDDDDDD@BBDDDCDDD
@HISEQ:426:C5T65ACXX:5:2301:5815:7120 1:N:0:GTGAAA
TATCTGCGATTGGTTCCATCTTCCCGGGTGACTGTGCAGCTGACTCCCATGCCCACCATCCCAGAGTGAGTGATATGTCCACCAGATTCTGAGTCCGCAAG
+
CCCFFFFFFHHHGHGGHIIJIIIJIJJIDFHIJIIHHIIIIJJIJJJIHCHIJIIJJJJJHHHHFF;@BB6>ADDCECDDCC@DDDDDDDDED>C@BDDDD

That works with v6.8.0 using this command
snakemake  --wrapper-prefix git+file:///home/cokelaer/Work/github/forked/sequana-wrappers

You will need to install fastqc (e.g. from bioconda) and no other dependencies.


useHash

What new hook does?
Checklist

[x] Have you read contribution guideline?
[x] Does the code have comments in hard-to-understand areas?
[x] Is there an existing issue for this PR?

Issue

[x] Have the files been linted and formatted?
[x] Have the docs been updated?
[x] Have the tests been added to cover new hook?
[x] Have you run the tests locally to confirm they pass?

We decided today in #33 that location-related hooks will not be ported from react-use since they are commonly provided by routing libraries, which there are plenty available.
Thanks for your work anyway, we hope you keep contributing to this library!

:tada: This issue has been resolved in version 19.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Feature request: find out glibc version needed to run an AppDir

It would be cool if we could have a function to find out what glibc version is needed to run the contents of a particular AppDir.
Something along the lines of
https://github.com/AppImage/AppImages/blob/f92f77b928239528a9cc4ddad651635b4f616ce9/functions.sh#L137
find . -name *.so -or -name *.so.* -or -type f -executable  -exec strings {} \; | grep ^GLIBC_2 | sed s/GLIBC_//g | sort --version-sort | uniq | tail -n 1

but in libelfutil.

I would want to use this in appimagetool to throw at least a warning if the glibc version needed to run is higher than the one shipped by the currently oldest still-supported Ubuntu LTS release.

This is an ELF library. It doesn't do anything AppImage related, it's just providing an interface to ELF files.
Can you describe what you need in terms of ELF file handling? Then, this might be implemented here, and consumed by other applications.

I want to find out which version of glibc an application (and all the libraries it loads) needs in order to run.
This is not AppImage specific at all.

I'll rename the issue then. Thanks for the quick response.


Remove from _linux_dll_ignore_list libstdc++ for musl Linux(Alpine).

    # The C++ standard library can also be ABI specific, and can cause system
For linuxes without libstdc++ like Alpine it would be nice to include libstdc++ in build.
Example:

Install scipy in Alpine - it requires libstdc
Perform standalone build and move binary to new Alpine
import scipy.fftpack
ImportError: Error loading shared library libstdc++.so.6: No such file or directory

Tested the same with removed libstdc from ignored list and it is ok.

I am likely to put this one into the next hotfix of 1.2 even, it seems innocent enough.

This is part of the hotfix release 1.2.2 that I just made.

@kayhayen thanks a lot, will continue learning Nuitka =)

There should hopefully not be too much learning needed. I am trying to make things work out of the box, but obviously standalone on non-standard systems can need a bit more tweaking.


DS-676 - [App panel] There's no validation feedback in the dialog box when trying to create a variable that already exists.

This change ensures that a newly introduced variable is checked for existence before creating or updating it. If the variable is already present, we will display an error toast; otherwise, we will proceed with creation or update.

@jugnu-appveen


AM/PM is wrong for 12 noon

12 noon in the US is PM but the formatter reports it as AM.
This time<PHONE_NUMBER>321 is formatted as 26 Feb 2016 12:08:49 AM with format:
%-d %b %Y %-H:%M:%S %p
but it should be PM

Thanks for the report. Somehow I did not have a test covering that, I though I had one.
I cant use your test time to verify without knowing which timezone offset you are running in, however I reproduced the faulty behavior in my local zone.
I have also noticed the default dateTime string in Config_en_us was faulty to.
It should have been dateTime = "%-m/%-d/%Y %-I:%M %p"
For 12 hour Hour digits not 24 as it was before "%-H". not "%-I"
I have published a fix 2.1.1.


Nothing references teardown.sh

It looks like scripts/teardown.sh will uninstall Homebrew. Should this be documented in the README, or is it not ready for prime-time?

Not sure what it was for, but like you said, never referenced ever since its introduction. I think people can figure out how to remove homebrew. Deleted the file.


🚀 - Customize line numbers starting number

Which @codeimage/* package(s) are relevant/releated to the feature request?
Don't known / other
Description
It would be nice to let line numbers to start from a specific arbitrary positive number.
Thank you!

initial implementation: https://github.com/riccardoperra/codeimage/pull/613

Implementation ready in this pr https://github.com/riccardoperra/codeimage/pull/624.
Will be deployed in next update

Finally had the time to merge it 😄 Will be available soon in prod

Released with 1.6.0. Thanks 😄

P.S. Alla fine ti ho nominato anche nel changelog dentro l'app 😬 Prima o poi rilascero anche il resto


Docker build LLM workflow

TL;DR
Adds a basic GitHub Actions workflow to build Docker images for each example.  This PR builds an example image for the llm example only.  If image were generated based on the hash of the first commit in this PR, the following images would be pushed to Docker Hub:

mosaicml/examples:llm-latest
mosaicml/examples:llm-242a6b

Details
The workflow is triggered when:

A pull request is created or updated
The main branch is updated

When triggered by a pull requests, the workflow will only build a Docker image for each example specified in the workflow build matrix.  The images will not be pushed to the Docker Hub mosaicml/examples repository.
When triggered by a push to main, the workflow will build a Docker Image for each example specified in the workflow build matrix and push the images to the Docker Hub repository.  The images are tagged according to the following format in the mosaicml/examples repository:

mosaicml/examples:${{ matrix.example }}-latest
mosaicml/examples:${{ matrix.example}-${{ github.sha }}

To add examples, simply add the example folder name  to the list specified in the build matrix.
To Do

[ ]  Create mosaicml/examples Docker Hub repo: https://mosaicml.atlassian.net/browse/IT-1160

@abhi-mosaic @samhavens this is a very basic workflow.  My goal was to have a single workflow that we can easily extend to all the examples in the repo but finer grained customizations per example are possible if needed.  Let me know what you think.

This is exactly what I was imagining when I made that Slack comment 🎉

@bandish-shah is there a lot of overhead / PR wait time if we create a tag on every commit? Like would it make sense to build these only on release tags of mosaicml/examples? I just don't know what the overhead is like.
If it's fast enough to do this on every commit, then I'm down to just merge this PR and try it out for a few weeks, see how the research team uses the new images :)

@abhi-mosaic since it's triggered on push to main, this should only happen on PR merge... but yeah, usually a PR is for one example, so this will trigger some redundant builds. But, it shouldn't block anything, as I don't think this will run before merging

@abhi-mosaic I just wanted to get something quick in place.  It shouldn't be a problem now since this is just LLM for now.  Also once the Docker images are built they get cached so if there are no changes it should be pretty quick.  Right now it looks like no other CI runs in this repo so this should take more than a couple of minutes to run.
As we add more examples, we should add path based filtering so only the builds for examples that have changed run.  It's not straight forward to do with a matrix build but that is the next major enhancement to this flow.  Worst case brute force is to just have an independent build workflow for each example that triggers based on path.

@samhavens it does currently run on pull_request events for development purposes but let me add a TODO to remove that before the final merge so it only runs on push events to the main branch.


Expose the factory methods for query objects by primary key.

This ticket seems to suggest this was possible at one point, but possibly is not longer working for Swift 4.0:
https://github.com/realm/realm-cocoa/issues/3099
For the sake of this commit msg, consider our RLMObject subclass:
@objcMembers
class Order: RLMObject {
    dynamic var orderId: String = ""
    override class func primaryKey() -> String {
        return "orderId"
    }
}

Currently, the only way to retrieve an object by primary key on an RLMObject subclass in Swift (4.0), is to do this:
let order = Order.objects(in: realm, where: "orderId = %@", "someOrderId").firstObject()

This is because the RLMObject factory method + objectInRealm:forPrimaryKey is only available on the base RLMObject class as a convenience initializer. In other words, we get this if we try to use the converted init on the type Order:
Cannot invoke initializer for type 'Order' with an argument list of type '(in: RLMRealm, forPrimaryKey: String)'

In order to be able to call this factory method in Swift, we need to keep it as a factory method and not convert it to a init. To do this, we add the NS_SWIFT_NAME macro which then allows us to call the method as expected.
let order = Order.object(in: realm, forPrimaryKey: order.orderId) // Yay! It works!

Any reason that you cannot use Realm.object(ofType:forPrimaryKey:)?
API ref: https://realm.io/docs/swift/3.3.0/api/Classes/Realm.html#/s:FC10RealmSwift5Realm6objectu0_RxCS_6ObjectrFT6ofTypeMx13forPrimaryKeyq__GSqx_

@astigsen : Note that this is using the Obj-C Realm APIs within Swift. I'm not using RealmSwift in this project.

This appears to have been broken ever since Swift 3.0.


Add Index on member_profile.username to Accelerate Query

Missing index on table member_profile column username might make the underlying query issued via TeamService#fetchMemberProfileUsername slow. Since table member_profile is mostly read by guests and sometimes inserted or updated by admin, the performance impact for write operations is acceptable under common cases.

I want to work on this issue, being a beginner can you please assign this and guide me through it.


Add a License

License is CC-BY 4.0. I don't know if there's a license file I can use for that; GitHub certainly doesn't have a template for it.

http://creativecommons.org/licenses/by/4.0/legalcode.txt

Added.


Browsing: Objects should be intregrated within their parent study

Like other e.g. Study identifiers(s) or Study title(s) there should be a listing of Study object(s) to be opened by arrow down listing all objects and their details.

great! please add the feature also for logged users.

Perfect!


Symfony 7 support

We still have 2 months or so to go until the release of Symfony 7. So no need to rush yet :smile:
But it would be nice to look into this at some point soon. Maybe someone looked into it already? If not I'm also happy to take a look myself.

Thanks for merging @stof. Will you release new version soon?


CIS CentOS 7 - CIS-6.1.11

In the centos-7-level-1-scored-v1.yaml it is looking for the following conditions

The code is looking for "/etc/at/allow" instead of "/etc/at.allow". Due to this the audit is failing
https://benchmarks.cisecurity.org/tools2/linux/CIS_CentOS_Linux_7_Benchmark_v1.1.0.pdf

Looks like @avb76 is on it. Thanks for the report!

Fixed in #306


Add mix-blend-mode CSS property

Closes #5625
Inserted the property in the existing list, considering alphabetical order.

Thanks!

@adrianheine Thanks so much for the quick review & merge!


Re-implement teleportation cancelled sound and make it customisable

Regression from HuskHomes 1.5.x that I forgot to re-implement. With 2.1's customization sound feature this shouldn't be difficult to implement.

Closed in commit b10856baaeb16f10961593de891b0fadbb5493c5


Created the vertical_scaling.md file

Created the vertical_scaling.md file for vertical scaling definition it resolves #157

Let me know if there are any more changes :)

@JasonMorgan is it okay now ?

Yes it is! Great work!

Thank you :smile:


Omit unused model types from code generation

Slightly tweaked test swagger so that needed models are referenced.

Fixes https://github.com/Azure/autorest.go/issues/730
Partially addresses https://github.com/Azure/autorest.go/issues/757


 Thanks so much. :)
I'll leave the issue open for tomorrow (3am currently) and then I'll add it to the docs.

Added to the readme, and soon to be added to the docs. Credit granted. Thanks very much for your contribution! :)

Thanks Dayle!


New version: Jackett.Jackett version 0.22.850

Pull request has been created with WinGet Releaser :rocket:
Microsoft Reviewers: Open in CodeFlow

/azp run

[Policy] Needs-Manual-Merge


Gulp cordova plugin install return 'add' not in gulpfile

gulp --cordova 'plugin add org.apache.cordova.camera --save'
Returns for me
Task 'add' is not in your gulpfile
With gulp --cordova "plugin add org.apache.cordova.camera --save" It works BUT only with the new Syntax:
gulp --cordova "plugin add cordova-plugin-camera --save"
New Syntax: https://cordova.apache.org/announcements/2015/04/21/plugins-release-and-move-to-npm.html

I suppose you are on windows? https://github.com/mwaylabs/generator-m-ionic/blob/master/docs/guides/windows.md

I guess that means yes :) Happy Coding!

Thanks for the reminder. Maybe it'll be good to have the docs regarding this issue platform-independent as well!


[Question] Set y-Axis range regardless of entries

Hello,
I might be missing something but, I want to be able to display the y-axis on a line chart to a specific value, i.e. 100, when the entries within the chart do not reach 100, if that makes sense.
I set the chart's MaxValue to 100 but that didn't work (I think that's more of a cap on the value rather than how much to 'stretch' the y-axis).
See the below which should help explain.

Cheers

have you solved this issue?

I'm open to someone making a pull request, that's pretty much where this project is at, I'm slowly moving away from xamarin so I'm more here to keep things moving in terms of cutting a release, I'm hoping to see people jump in and contribute to the project.


[Event Request] Table 1003 "Job Planning Line".RecalculateAmounts

Describe the request
local procedure RecalculateAmounts(JobExchCalculation: Option "Fixed FCY","Fixed LCY"; xAmount: Decimal; var Amount: Decimal; var AmountLCY: Decimal)
var
    IsHandled: Boolean;
begin
    IsHandled := false;
    //------------------------------------------------------------OnBeforeRecalculateAmounts:BEGIN
    //OnBeforeRecalculateAmounts(Rec, xRec, AmountLCY, IsHandled);
    OnBeforeRecalculateAmounts(Rec, xRec, AmountLCY, IsHandled, Amount);
    //------------------------------------------------------------OnBeforeRecalculateAmounts:END
    if IsHandled then
        exit;

    if (xRec."Currency Factor" <> "Currency Factor") and
       (Amount = xAmount) and (JobExchCalculation = JobExchCalculation::"Fixed LCY")
    then
        Amount := ConvertAmountToFCY(AmountLCY, UnitAmountRoundingPrecisionFCY)
    else
        AmountLCY := ConvertAmountToLCY(Amount, UnitAmountRoundingPrecision);
end;

//------------------------------------------------------------OnBeforeRecalculateAmounts:BEGIN
[IntegrationEvent(false, false)]
//local procedure OnBeforeRecalculateAmounts(var JobPlanningLine: Record "Job Planning Line"; var xJobPlanningLine: Record "Job Planning Line"; var AmountLCY: Decimal; var IsHandled: Boolean)
local procedure OnBeforeRecalculateAmounts(var JobPlanningLine: Record "Job Planning Line"; var xJobPlanningLine: Record "Job Planning Line"; var AmountLCY: Decimal; var IsHandled: Boolean; var Amount: Decimal)
begin
end;
//------------------------------------------------------------OnBeforeRecalculateAmounts:END

Additional context
We need change amount from subscribeInternal work item: AB#492732

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 14606.


Update docstring links to use tsdoc link convention

Update rust links in docstrings to use tsdoc linking & camel cased names, i.e:
/// You can verify it against a Blakeb256 root hash by using [`Self::valid()`]

changed to
/// You can verify it against a Blakeb256 root hash by using {@link MerkleProof.isValid}

So links generated in TS/JS documentation work.
More info: https://tsdoc.org/pages/tags/link/

I want to work on this issue
Please assign it to me

Hey @ross-weir I want to work on this issue
In which file I need to change
I am unable to find the page

Hey @ross-weir I want to work on this issue In which file I need to change I am unable to find the page

Great! Thank you!
This file is the only one currently that needs updating: https://github.com/ergoplatform/ergo-lib-wasm/blob/main/packages/merkle-tree/src/lib.rs
This line, for example: https://github.com/ergoplatform/ergo-lib-wasm/blob/main/packages/merkle-tree/src/lib.rs#L76

I'll keep this open as a WIP, there'll be more to fix up as we migrate


Verification mails are not sent to @mail.ru addresses

Repro steps

Verify an account using<EMAIL_ADDRESS>
Expected Result
Verification mail arrives
Actual Result
Verification mail never arrives, not even in spam, I tested using other gmail mails to send to this mailbox and they all arrived.
System Info

Fixed after openkyc restart


Why 400 "Bad Request" if token exists but not decodable?

Hi,
thanks for the lib. I wounder if there is a specific reason behind returning HTTP-Status 400 when the token can not be decoded?
Background: If i request a resource with a expired token i currently get a 400 (instead of 401). But 400 has, at least for me, the meaning: the request has a bad syntax or can not be processed. I would assume a 401: something wrong with the authentification.
https://github.com/tuupola/slim-jwt-auth/blob/master/src/JwtAuthentication.php#L99
In the code you already checked if a token exists (and throw a 401 if not). After that you throw a 400, if it is noch decodable. Is this common practice?

There was a reason but I do not remember what is was anymore. That said, I now also tend to think response should be 401. See also #12.

It returns 401 now. Will do release soonish.

Thanks a lot.

We needed this fix aswell, because an expired token also gives the 400 bad request error.
However it would be better if you catch the exception types and render a response based on that, for example the ExpiredException should return a 401 unauthorized, yet the malformed, invalid token could return bad request.
This fix solves everything for us though ;) so no need for changes


Invalid media type & Unknown Video Info Key

2022-05-13 20:27:01.513 T:2785    ERROR <general>: NEWADDON Unknown Video Info Key "__no_default__"
2022-05-13 20:27:01.513 T:2785  WARNING <general>: Invalid media type "/category/korean-drama"

I keep getting this error over and over until Kodi crashes for me.
I'm on the latest OSMC (March2022) & Vero4K.

Fixed in 23e9183. Could you verify if this solves the issue.

yes it's fixed, thanks!


Print error logs from launcher during integration tests

What does this PR do?
Print error logs from launcher during integration tests
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[x] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.
@OlivierDehaene OR @Narsil

Hey @Atry thanks for the contribution.
Do you mind sharing a bit more about the problem this is trying to solve ?

I should be able to see error logs when running a test. The error logs are useful to debug failed tests.

But that's already the case with the -s pytest arg isn't it?

The -s option affects how to capture pytest's stdout and stderr, but it does not affect the launcher's stderr in the current implementation, because launcher's stderr is not forwarded to pytest.
This PR redirect the launcher's stderr to the launcher's stdout, which is already forwarded to pytest's stderr at https://github.com/huggingface/text-generation-inference/pull/480/files#diff-f99db5aabf9de5120c24409d7be1b79e635a807f5dc3c12de0116b66f5528be2R256, solving this problem.

Hi @OlivierDehaene, could you trigger the CI for this PR? You will see more logs than the main branch.


[#noissue] fix matcher type of GO_HTTP_CLIENT service type

Codecov Report

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

@@            Coverage Diff             @@
##           master    #7380      +/-   ##
==========================================
- Coverage   38.73%   38.73%   -0.01%
==========================================
  Files        3322     3322
  Lines       95992    95992
  Branches    11872    11872
==========================================
- Hits        37182    37178       -4
- Misses      55908    55912       +4
  Partials     2902     2902

Impacted Files
Coverage Δ

...rp/pinpoint/rpc/packet/ControlHandshakePacket.java
51.51% <0.00%> (-24.25%)
:arrow_down:

...tor/metric/datasource/DefaultDataSourceMetric.java
75.00% <0.00%> (-3.13%)
:arrow_down:

...pinpoint/profiler/monitor/DeadlockMonitorTask.java
18.60% <0.00%> (-1.17%)
:arrow_down:

...point/rpc/client/DefaultPinpointClientHandler.java
72.40% <0.00%> (-0.98%)
:arrow_down:

...orp/pinpoint/rpc/server/DefaultPinpointServer.java
79.44% <0.00%> (-0.70%)
:arrow_down:

...rp/pinpoint/rpc/server/PinpointServerAcceptor.java
88.74% <0.00%> (+4.63%)
:arrow_up:

...vercorp/pinpoint/rpc/packet/ServerClosePacket.java
68.00% <0.00%> (+16.00%)
:arrow_up:

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 2dec0d5...57194f6. Read the comment docs.


Adjustments to the exported data - values of data properties - should contain dataTypes as well

The exported data should contain also datatypes for predicates (relations) e.g. between the subject column and some literal (number).  Currently, if the value of the predicate is literal, there is no dataType for that literal.
Note: It should work in a way that if there is range for certain predicate, which is different from "xsd:String", then such datatype (range) should be attached to the produced literal.
Note: On the other hand, if the value of the predicate is String, there should be language tag (based on the language of the KB - in the config)
For that, the export service requires a simple service from KBProxy , which would give him some triples about the requested predicate (or just "range" of that predicate).

@josef-janousek Hi, could you look into it?
@josef-janousek @tomas-knap Concerning the proposed service: I think that the relation type is is something that should be filled during algorithm run and not queried later.

Ok, yes, that is also possible - to directly return by the algorithm. And it is not that much data, so lets do that in that way - to directly enrich what is returned by the algorithm (with e.g. domains/ranges)

For literal objects - datatype properties

I have added the two methods into the KBProxy (getPropertyDomains and getPropertyRanges). My commits are in the branch 325.

Closed by #376 .


Google Play Games Plugin plus Ads

So I'm currently using BFS Google Play games plugin (which sadly doesn't seem to have support anymore) and this Ad plugin.
For some reason, when I add the unity-plugin-library. jar file, my app will crash on launch. However, if I don't have it, the ads will not work.
It seems like there was a problem with the google-play-services.jar from the BFSGPG plugin not being the full jar file (missing the ad folders), but when I replaced them, the app would still crash on launch.
I can't really abandon the BFS plugin or else I would...

Closing due to non-response. If this is still an issue feel free to reach out at the support group.

Ram


Add Bicubic Interpolation

Summary
Adds Bicubic interpolation to the Keras backend.
K.resize_images(x, height_factor, width_factor, data_format, interpolation='bicubic')
PR Overview

[x] This PR requires new unit tests [y/n] (make sure tests are included)
[x] This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
[x] This PR is backwards compatible [y/n]
[ ] This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

Can someone assist me with the unit tests? I have no experience with unit testing and I'm confused about all the inner workings.

We need a discussion with @fchollet about this since it changes the API.
Also, let's note that this new feature is implemented only for the TF backend in this PR.


2.0/Big rework on ExpressionBuilder and CqlOperatorBinder to narrow down where conversions occur

Conversions are scattered everywhere and should preferably only happen in two places, binding to cql operator method when arguments are matched/converted against existing methods, and then again on the return type if necessary
This is a big task, so all work will be done across several PR's and on a side branch develop-2.0-bindings

Duplicate of #292


ensure the template is using latest version

the template should be using the latest release version, or at very minimal, only 1 major version behind

For sure, the goal is to update the templates after every release (to crates.io).
Currently the templates are not updated because we've run into some issues - the work can be tracked here.

All three templates are now being kept up to date - most recently, with version 1.13.0.


Interface derive does not support field renaming

Problem
This code:
#[derive(Interface)]
#[graphql(
    field(name = "id", type = "&AccountID")
)]
pub enum Account {
    User(User),
    Organization(Organization),
}

#[derive(SimpleObject)]
#[graphql(complex)]
pub struct User {
    #[graphql(name = "id")]
    account_id: AccountID,
}

Results in an error:
no method named `id` found for reference `&account::User` in the current scope

While using field(name = "account_id", type = "&AccountID"), works it ignores the renaming to "id".
Another attempt:
#[derive(Interface, Debug, Clone)]
#[graphql(
    field(name = "id", type = "&AccountID")
)]
pub enum Account {
    User(User),
    Organization(Organization),
}

#[derive(SimpleObject, Debug, Clone)]
#[graphql(complex)]
pub struct User {
    #[graphql(skip)]
    account_id: AccountID,
}

#[ComplexObject]
impl User {
    async fn id(&self) -> &AccountID {
        &self.account_id
    }
}

Results in a false error:
A GraphQL Object type must define one or more fields.

Error is not considering the id() method that makes object non-empty.
Specifications

Version: async-graphql 2.11.1
Platform: linux

Thanks for an amazing library!

Use the method attribute to specify the resolver method name.
#[derive(Interface)]
#[graphql(
    field(name = "id", method="account_id", type = "&AccountID")
)]
pub enum Account {
    User(User),
    Organization(Organization),
}

It worked, thank you @sunli829!

What about the second error caused by ComplexObject being ignored - A GraphQL Object type must define one or more fields.? I have the same issue.


Copy paste ~ ACS string (assignments)

Import assignments via ACS string

Do you mean NMR text https://pubsapp.acs.org/paragonplus/submission/acs_nmr_guidelines.pdf ? Please also check the standards by Chemotion Repo: https://www.chemotion.net/docs/repo/details_standards/analysis_test/nmr#inline-notation-for-1h-nmr-data
Best,
Tillmann


Adding callback option for fit function

Adding some callbacks would help to track the model better.
Example callbacks for wandb, tensorboard etc. would be nice.

WandB callback for automatic loss tracking is implemented by
5c1f94d  and callbacks are implemented in
c4690a4


Implement rest data loader

Description
Implement RESTDataLoader.
Checklist

[x] I am familiar with the Contributing Guidelines.
[x] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.

/merge


Report/lessons learned

I initally had severeal subsections, but condensed them into one subsection because the report is getting too long.

Also mention: Kubernetes made it really easy to change who hosts all of our stuff
Also mention: Infrastructure as Code (with Terraform) has made it really easy to deal with student credits and switching subscriptions on Azure. We simply rebuild all the infrastructure on a new subscription with a few commands.


store: Fixed intersect matching when one matcher filters out all series.

Fixes https://github.com/improbable-eng/thanos/issues/833
Signed-off-by: Bartek Plotka<EMAIL_ADDRESS>

Built and had a quick test, can confirm #833 is fixed and working as expected :)


Clean up transaction result code

Description
This PR intends to clean up the transaction result code:

Aligned the ReasonCode with the error code defined in the C interface;
Corrected the inappropriate usage of INTERNAL_ERROR;
Fixed the STATIC_MODE_ERROR misinterpretation.

Type of change
Insert x into the following checkboxes to confirm (eg. [x]):

[x] Bug fix.
[ ] New feature.
[x] Enhancement.
[ ] Unit test.
[ ] Breaking change (a fix or feature that causes existing functionality to not work as expected).
[ ] Requires documentation update.

I think this PR was supposed to be updated to go to the master-pre-merge branch


Unable to access to ArgoCD UI in Firefox: SEC_ERROR_BAD_DER

Deploying ArgoCD following the instructions in README.md with default values works smothly, however it is not possible to access to the UI using Firefox.
The following messages are showed in the browser without any capability to navigate:
Secure Connection Failed

An error occurred during a connection to argocd-server-labs-ci-cd.apps.cluster-e381.e381.sandbox1862.opentlc.com. security library: improperly formatted DER-encoded message.

Error code: SEC_ERROR_BAD_DER

    The page you are trying to view cannot be shown because the authenticity of the received data could not be verified.
    Please contact the website owners to inform them of this problem.

The same route works successfully in Chrome.
Environment:

OpenShift 4.8
ArgoCD Operator 0.0.15
Firefox 94.0 (64-bit)
Fedora 34

PS: Open this issue initialy here, as I am not sure if the issue is related with the Helm Chart, OpenShift 4.8 (provided by RHPDS) or finally it is an issue of Argo CD.

hi @rmarting  this is likely related to Firefox and self signed certs.
can you try:

about:config
switch network.stricttransportsecurity.preloadlist -> false
restart browser

I just tried with FF93, FC34 on a *.dev domain (HTST) - worked OK for me.

Also note that once the argocd operator v0.1.0 releases into the community via OLM, we can make use of this new feature:
feat: Enable service CA for reencrypt routes to argocd-server on OpenShift
https://github.com/argoproj-labs/argocd-operator/pull/326

@eformat It is not working in my environment (FF94, Fedora34), adding the following properties in my FF configuration (about:config):
security.tls.insecure_fallback_hosts = <argocd-dashboard-host>
network.stricttransportsecurity.preloadlist = false

So, it is related with FF. Maybe we could maintain this issue open to confirm and verify the new feature before to close it. Does it suit for you?

hi @rmarting i am going to close this for now. Firefox94.

One thing to note - i have recently pushed #211 - the GitOps operator on OpenShift now supports reencrypt Routes which will get around client browser config issues with real certs.


IndeterminateProgress has no "bufferColor" propType

is this on purpose?

Good catch.
I think that if you needed it, we can add it (or you, PR is welcomed :))


Where is shapely's "used by" GitHub widget?

@mwtoews you asked in another issue why Shapely doesn't have a "used by" widget on the top of its GitHub page. Some things I've read lead me to suspect that it's the lack of a requirements.txt file. All the other projects I work on and have the widget have a requirements.txt file and/or some requirements in the setup script. I've added an empty requirements.txt and will do a little more research and asking around.

It took contacting GitHub support for me to fix that for xarray. That might be worth a try...

Email sent :email:

GitHub has sorted it out.

Hello Sean,
Thanks, the package looks like it was assigned to the wrong repository. I have corrected this and can see the Used by on your repository now. Please let me know if there is anything else I can help you with.
Kind regards,
Laurel


Dependency updates

Updating dependency and browserify script, with the intention of first getting the repository in a functional state again before fixing the range notation (gt, gte, lt, lte instead of start, end).

Replaced level-browserify by level as the project indicated, because installing level-browserify wasn't possible anymore on a recent system (api changes according to node-gyp).
Fixed invalid stream closure in test/triple_unicode_store_spec and test/triple_store_spec (replaced listening to close event by end event).
Non-major updates on level-sublevel, multilevel, osenv, setimmediate & tmp

Are you using this module? Are you interested in maintaining this module going further?
I'm not actively maintaining this so I might consider giving you access.
(There is probably some work to do on CI, test are not being run).

I've used it in the past, and interested using it again.
Based on previous usage: This lib is still pretty much the only one that I can simply throw batch of triples into and it just works. Hexadb was inspired by this lib, but the queries are either broken (or I'm missing something). GunDB works, but has a different api style which doesn't readily support searching. I've tried others, but most are tied to RDF these days, trying to handle some magic edge-cases.
While converting to neo4j, jena or mongodb with lots of indexes would probably work, having a matcher on bare triples on top of a key-value store & then converting them freely between triples and objects gives a lot of freedom in storage.
TL;DR; I used to used it, looking to use it again & interested in maintaining for the bugs I come across (and if I have time left, perhaps even some open issues).

I have added you to the team! You can land this yourself. Open an issue/discussion when you would like to cut a new release and we'll organize that as well.

Awesome!
This specific PR should not generate a release in my opinion, based on the notion of up-coming changes.

Merging, not bumping the version number.


Update hardware-and-software-requirements-0.md

Add a note regarding pre-req software being supported as part of the 2013 lifecycle.

@Techwriter40 Hi Kirk, the note looks good. Could you please check and merge? Thanks!


fix: add python entry point

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

When running rerun on Windows I run into this error:
Fatal error in launcher: Unable to create process using '"D:\bld\rerun-sdk_1697490036427\_h_env\python.exe"  "E:\Developer\prefix\rerun\.pixi\env\Scripts\rerun.EXE" ': The system cannot find the file specified.

It appears the prefix D:\bld\rerun-sdk_1697490036427\_h_env was not properly replaced by the build. I think this behavior is overwritten when manually specifying an entry point which is what this PR adds.

I can verify that this now indeed works on Windows. :)


Sass documentation links are out-of-date

🐞 Bug report
Command (mark with an x)
- [x] new
- [ ] build
- [ ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Description
The documentation links for Sass are out-of-date. The documentation of the difference between the two syntaxes now lives at https://sass-lang.com/documentation/syntax.
🔬 Minimal Reproduction
Any invocation of ng new will reproduce this.
🌍 Your Environment

Angular CLI: 8.1.0
Node: 12.0.0
OS: linux x64
Angular: 8.1.0
... animations, cli, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.801.0
@angular-devkit/build-angular     0.801.0
@angular-devkit/build-optimizer   0.801.0
@angular-devkit/build-webpack     0.801.0
@angular-devkit/core              8.1.0
@angular-devkit/schematics        8.1.0
@ngtools/webpack                  8.1.0
@schematics/angular               8.1.0
@schematics/update                0.801.0
rxjs                              6.4.0
typescript                        3.4.5
webpack                           4.35.2

Thanks @nex3, for bringing this up


Change

There is a default float format (DefaultFloatFormat = %2f) I would like to change. There doesn't seem to be a good way to do it without recreating the whole chart.ContinuousSeries struct so you can change the ValueFormatter

I've added XValueFormatter and YValueFormatter fields on the struct, they are coalesced with the default formatter in GetValueFormatters()

Thank you! That was pretty speedy.
Overall, I'm really enjoying using this library for debugging. Some things are just easier to think about as graphs.

glad to hear it!


sourcemap: 'nosource'

Describe the feature
Can we add 'nosource', similar to 'hidden' as a sourcemap option? I don't want to publish the source code.
Additional information

[X] Would you be willing to help implement this feature?
[ ] Could this feature be implemented as a module?

Final checks

[X] Read the contribution guide.
[X] Check existing discussions and issues.

How would this differ from sourcemap: hidden? Is there a way to implement this in Vite (see docs)?

'hidden' means the sourcemaps are generated with full source code, but not referenced in minified file.
'nosource' means the sourcemaps are generated without full source code (only file, line numbers).
There's probably a use case for having both at the same time as well.
https://webpack.js.org/configuration/devtool/
Here's a comment on a request that has a possible solution: https://github.com/vitejs/vite/issues/15035#issuecomment-1818081430

Normally I would think this should be implemented at bundler level, but happy to consider it as it seems it can be implemented with rollup config. Setting this should enable the corresponding rollup option for both vite + nitro.

Try this:
// nuxt.config.ts

export default defineNuxtConfig({
  sourcemap: {
    server: true,
    client: true,
  },

  vite: {
    build: {
      rollupOptions: {
        output: {
          sourcemapExcludeSources: true,
        },
      },
      sourcemap: true,
    },
  },
})


Authentication Type: Active Directory  - Universal with MFA Support

Issue Type: Bug
Can't login with Active Directory + MFA (chrome, mac).  Was working previously but now when we go to login it tries to open a new tab in chrome (https://localhost:50801/signin?nonce=iclearedthisincaseitsenesitve)  and chrome displays: This site can’t provide a secure connection
Steps to reproduce (i previously went through these steps and saved):

Connection Type: Microsoft SQl Server
Server: xxx
Authentication Type: Active Directory  - Universal with MFA Support
Click Connect

Go to a query window and hit "reconnect" and then select the previously saved connection.  It opens up a new tab

and fails.
Azure Data Studio version: azuredatastudio 1.20.1 (fe6312306647441b30102a64bb43abb06ebcf2be, 2020-07-17T02:32:14.131Z)
OS version: Darwin x64 19.5.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7920HQ CPU @ 3.10GHz (8 x 3100)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 2

Memory (System)
16.00GB (0.25GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (10)

Extension
Author (truncated)
Version

admin-pack
Mic
0.0.2

admin-tool-ext-win
Mic
0.0.2

agent
Mic
0.47.0

dacpac
Mic
1.5.0

import
Mic
0.18.0

profiler
Mic
0.11.0

server-report
Mic
0.2.2

whoisactive
Mic
0.1.4

azdata-sanddance
msr
3.0.0

vscodeintellicode
Vis
1.2.1

(1 theme extensions excluded)

@aaomidi any ideas what's happening?  Or how to resolve?  Do you think this is fixed in the latest insiders build potentially?

the SSL error is due to something in your environment enforcing https for localhost.
To remove that rule:
Microsoft Edge: Visit edge://net-internals/#hsts Under "Delete domain security policies" put in localhost
Google Chrome: Visit chrome://net-internals/#hsts Under "Delete domain security policies" put in localhost

Hi @aaomidi!
Can you please clarify why disabling https for localhost is a valid solution in this case?

@marinvirdol Sure thing!
There is no way we can place a valid https cert on your localhost - so having HSTS on for localhost tells the browser to refuse to load localhost over http. Some dev environments end up enabling HSTS for localhost since they can create valid https certs for it, but it would be extremely intrusive of us to do so.


Config is not loaded from package.json

Describe the bug
The default search locations of cosmicconfig are overwritten and the package.json option is left out, so configs are not loaded from it.
The culprit: https://github.com/kamilkisiela/graphql-config/blob/master/src/helpers/cosmiconfig.ts#L71
The original: https://github.com/davidtheclark/cosmiconfig/blob/main/src/index.ts#L109
To Reproduce
Steps to reproduce the behavior:

Define a graphql configuration in package.json
Try to load it using loadConfigSync

A Codesandbox example where configurations are loaded from .#rc files, #.config.js files, but not from package.json:
https://codesandbox.io/s/graphql-config-ignoring-packagejson-nergw?file=/src/index.js
Expected behavior
The loadConfigSync function should load the graphql configuration from the package.json file, as per the graphql-config documentation and the default behavior of cosmiconfig
Environment:

OS: All
GraphQL Config Version: 3.2.0
NodeJS: All

v3.3.0


Make Search button more attractive in Gallery Section.

The Search button looks very simple. I want to make it more attractive. Please assign me this task under GSSoC'23.

I would like to contribute to this issue. Please assign me this task under GSSOC'23.


[Bug]  Cleanup workflow not working

Changes:

On Branch Creation Github event does not contain head_commit object. Added a conditional statement for same
Sometime workflow event is not triggered on branch creation, so added 'workflow_dispatch:' so that it can be manually triggered.

Fix #343
Test URLs:

Before: https://main--aem-boilerplate--adobe.hlx.page/
After: https://main--aem-boilerplate--gargadobe.hlx.page/

@tripodsan Please review this


Add JSON support to UUID

Adds the following methods to support for UUIDs being used directly in JSON mappings, as per issue #5548:

UUID.new(JSON::PullParser)
UUID#to_json(JSON::Builder)

This looks like a good feature.  Thanks :)


New Rule: Ensure ALB & NLB serve only HTTPS traffic

'''
#####################################
##           Gherkin               ##
#####################################
Rule Name:
  ELASTIC_LOAD_BALANCERS_SERVE_HTTPS

Description:
  Check whether SSL is enforced on the Amazon Application Load Balancers and Network Load Balancers either using HTTPS/TLS listeners or by redirecting HTTP traffic to HTTPS.

Trigger:
  Periodic

Reports on:
  AWS::::Account

Rule Parameters:
  HttpsRedirectionCheck
   Optional
   A boolean parameter to check if HTTP to HTTPS traffic redirection is enabled. Default is True.

Scenarios:
  Scenario: 1
    Given: Parameter HttpsRedirectionCheck is configured
    And: It is not either True or False
    Then: Return an error
  Scenario: 2
    Given: No Application Load Balancers or Network Load Balancers present in the account
    Then: Return NOT_APPLICABLE
  Scenario: 3
    Given: At least one Application Load Balancer or Network Load Balancer present in the account
    And: Load Balancer has TCP listener(s) only
    Then: Return NON_COMPLIANT with Annotation "{ALB/NLB_Name} has no HTTPS or TLS listeners configured"
  Scenario: 4
    Given: At least one Application Load Balancer or Network Load Balancer present in the account
    And: Load Balancer has HTTPS/TLS and HTTP listener(s)
    And: The parameter HttpsRedirectionCheck is False
    Then: Return NON_COMPLIANT with Annotation "{ALB/NLB_Name} has HTTP listener(s) configured"
  Scenario: 5
    Given: At least one Application Load Balancer or Network Load Balancer present in the account
    And: Load Balancer has HTTPS/TLS and HTTP listener(s)
    And: The parameter HttpsRedirectionCheck is True
    And: HTTP to HTTPS redirection is not enabled
    Then: Return NON_COMPLIANT with Annotation "{ALB/NLB_Name} serves HTTP traffic without redirecting to HTTPS."
  Scenario: 6
    Given: At least one Application Load Balancer or Network Load Balancer present in the account
    And: Load Balancer has HTTPS/TLS and HTTP listener(s)
    And: The parameter HttpsRedirectionCheck is True
    And: HTTP to HTTPS redirection is enabled
    Then: Return COMPLAINT
  Scenario: 7
    Given: At least one Application Load Balancer or Network Load Balancer present in the account
    Then: Return COMPLAINT
'''

@jongogogo Kindly review the Gherkin for any updates/changes

NLP has TLS terminaison, but I'd be suprized it has HTTPS redirections. I suggest we rescope the rule to ALB to get started. (+ the TLS terminaison feature on NLP date from Jan so it won't be in the lamba boto yet.)
Similarly ALB cannot have TCP listener only HTTP/HTTPS.
SSL should be replaced by TLS/SSL
HTTPS/TLS should be replaced by HTTPS
Need a parameter comma-separated list on the SSL policies: https://docs.aws.amazon.com/elasticloadbalancing/latest/application/create-https-listener.html#describe-ssl-policies
We discussed with Santosh, while we arrive to the conclusion than a single rule is best for redirection. I still not convinced. Can you chat with him and consider 2 rules, one 1) checking the HTTPS listener and their SSL policies, and 2) another one checking that the only HTTP listerners are redirections.

@jongogogo Do you think we should split this rule and make separate rule for ALB and NLB?

Let's start with ALB. We will see to extend later.

Modified the gherkin to fit only ALB. Also added support for a parameter where valid security policies can be defined

I am not making sense of scenario 9. How can we do SSL redirection without an SSL listener?
If it is not possible, then I suggest we don't check the HTTP listener existence, it is not our objective to check the redirection is configured properly. Which raise another point: shall we split into a new rule verifying that the redirection is done properly (including HTTP listener)? So far I am in favor of doing that.
Note: Added another default SSL policy.

Some more thought- @schirag1993 Each ALB listener can use multiple rules with different conditions (https://docs.aws.amazon.com/elasticloadbalancing/latest/application/load-balancer-listeners.html#rule-condition-types); if you are checking for redirection in this rule, you will need to check each listener rule. I agree with @jongogogo that we can build this rule to just check if HTTPS listeners with allowed security policies exist or not. This would be somewhat similar to AWS Config Managed rule elb-predefined-security-policy-ssl-check: https://docs.aws.amazon.com/config/latest/developerguide/elb-predefined-security-policy-ssl-check.html

Some more thought- @schirag1993 Each ALB listener can use multiple rules with different conditions (https://docs.aws.amazon.com/elasticloadbalancing/latest/application/load-balancer-listeners.html#rule-condition-types); if you are checking for redirection in this rule, you will need to check each listener rule. I agree with @jongogogo that we can build this rule to just check if HTTPS listeners with allowed security policies exist or not. This would be somewhat similar to AWS Config Managed rule elb-predefined-security-policy-ssl-check: https://docs.aws.amazon.com/config/latest/developerguide/elb-predefined-security-policy-ssl-check.html

Makes sense. Modifying rule to check if HTTPS listeners have valid SSL policies

Modified the gherkin to check only if valid security policy is used on ALB's HTTPS listeners

Looks good. I suggest we change the name to reflect: https://docs.aws.amazon.com/config/latest/developerguide/elb-predefined-security-policy-ssl-check.html

Rule name updated

Hey, I just found this https://github.com/awslabs/aws-config-rules/tree/master/python/ELB_ALB_PREDEFINED_SSL_CHECK
Update this one if required or we can close this issue.

Shall we go ahead and close this then @jongogogo?


Bug: Spacebar doesn't respond after scrolling on reading pane

I hit spacebar to move from one feed entry to the next.  If I have an entry that displays a lot of content, I will move the mouse over to the far right reading pane and scroll.
After this action, the spacebar will no longer allow me to move onto the next feed entry. Hitting command-/ will move to the next unread entry, and now the spacebar will work again.
I am running NNW 5.0 (2609) on a MacBook Pro 2013 running  10.15 Beta (19A546d).

It works on my 10.14.6 machine — there’s a good chance this is a 10.15 beta bug.

It probably could be.  I don't have my computer running 10.14 to test this.
I will say that I am getting this error on Version 5.0.1d2 (2612) under 10.15 Beta (19A546d)