Is the Limitations part of the README still accurate?

Environment
Current GitHub master page.
Current behavior
The current text is:

Currently, only IPv4 is supported. The library is incredibly verbose in its logging to help debug issues on new platforms in prep for a first release. This will change. The library is mostly interim in its structure. Please consider submitting PRs and helping make this work reliably across embedded devices.

Expected behavior
I was wondering about whether the incredibly verbose thing is still accurate, as well as preparing for a first release. Sounds like it might be a bit dated.

Wow, that text definitely isn't correct.


Add guidance for supported string lengths

See #509

@n3wscott Ha I didn't even realise I was doing that... Serves me right for editing in the browser!

lgtm

LGTM


block iframe and other domain that use my static file

hi,
first of all sorry because of my poor english language.
I have a node server that serve some static files such as images, css, js and etc.
and i want to block any domain to use my static file and block iframes to show my website.
how can i do that?
is it possible to use cors ?
with regards, Milad Aslani

No. This is because what you want it outside what the web technology CORS does. CORS stands for cross origin resource sharing and is designed to open the door further to unblock things the browser blocks by default. Since those are not blocked by default by browsers, it falls outside of the CORS specification.


Fix module name in README.md

Readme was using an old module name which can be misleading. I swapped it out for the updated name.

The readme will be replaced soon by the new package


DCOS-21921: Use standard-version (+refactoring)

⚠ This one depends on #2838

This PR adds standard-version to our stack.
To re-enable our Jenkins Release Builder in our Pipeline with sv, I had to do some refactorings. With these changes, we are able to cut releases from jenkins with a single click 👍
The change was necessary because standard-version changes the way we tag our code, that means we arent able to trigger jenkins like we did before.
I had to refactor the existing code "here and there" in order to decouple it and make it easier for you guys to review. it looks like a lot of changes, but it really is mostly spliting up files & cleaning up stuff. Also I added a lot of checks and configuration parts to make the bash scripts as solid as possible.
Alltogether, these changes should allow you to read & review every file on its own 🙏

Some explanations per file:
Jenkinsfile
not much changed here, just removed the semver regex and added a parameter, also changed the Release Latest and Release Version blocks.
scripts/ci/release-latest
Entrypoint for releasing a new latest build, basicaly only calls upload-build and update-dcos-repo
scripts/ci/release-version
Entrypoint for releasing a new version, this one creates a release and pushes it to our repo!
also calls upload-build, update-dcos-repo and create-bump-pr
scripts/ci/utils/upload-build
Uploads an already existing release.tar.gz to with the correct name to AWS.
scripts/ci/utils/update-dcos-repo
Updates the given DC/OS branch with the latest buildinfo.json & upstream.json
=> "Part 1" from the old update-github file
scripts/ci/utils/create-bump-pr
Creates a new Bump PR based on the given DC/OS Branch
=> "Part 2" from the old update-github file
scripts/ci/utils/generate-changelog
havent changed much here, just added standard-version support and moved the file.

Sadly Github doesnt display the moved files correctly in this PR, but you should be able to follow=up my changes through the differen commits, I tried to add as much commits as possible.

Lastly: I know that this is all bash, I'd love to see that we change these scripts script-by-script to something which is testable and "just better", but changing the language would have exeeded the scope of this even more, so this will happen in the future.

works already:

[x] upload latest build
[ ] upload release build
[x] update latest branch
[ ] create release branch
[ ] create release PR

Automatically created PR: https://github.com/dcos/dcos/pull/2718 🎉

rebased, reworded commit messages and addressed comments 👍

had to change a few things again 😅 last changes combined:
https://github.com/dcos/dcos-ui/compare/a473d8201bb3c0b60c87c9008181e178eba880a3...4d69370d50e8206a25744e875d6a4cd90fcf2489


[ETHOSN] Fix quantization parameters in test

The Ethos(TM)-N tests for addition had a poor choice of parameters for
the int8 case.

Thanks @Leo-arm !


Registry tool: YAML imports silently skip invalid and irrelevant YAML files

https://github.com/apigee/registry/pull/888#discussion_r1053866795
I think we need to figure out how to differentiate when files are intended for input (as resource descriptions) and invalid and when they are irrelevant for YAML resource input because they are part of (service config) or an entire (openapi.yaml) API spec.

I think we need to figure out how to differentiate when files are intended for input (as resource descriptions) and invalid and when they are irrelevant for YAML resource input because they are part of (service config) or an entire (openapi.yaml) API spec.

Let's report the number of files applied when registry apply completes and we can also report an error when a specific file is applied that isn't Registry YAML.


Support for Redis 6 ACL

Redis 6 introduced new ACL options which allows sysadmins to create multiple username / password pairs and have fine-grained control over what users can do.
https://redis.com/blog/getting-started-redis-6-access-control-lists-acls/
Currently this plugin doesn't support this (either via editing the host or setting the password to match the new AUTH syntax). It would be great if this plugin could support this style of authentication.

same situation here , any sulotions for this ?


Carthage installation fails

Carthage installation of 1.4.0 fails during build with
/Carthage/Checkouts/LifetimeTracker/Sources/UI/CircularDashboard/CircularDashboardViewController.swift:32:58: error: 'shared' is unavailable: Use view controller based solutions where appropriate instead.
    fileprivate var formerStatusBarStyle = UIApplication.shared.statusBarStyle
                                                         ^~~~~~
UIKit.UIApplication:5:20: note: 'shared' has been explicitly marked unavailable here
    open class var shared: UIApplication { get }
                   ^
/Carthage/Checkouts/LifetimeTracker/Sources/UI/CircularDashboard/CircularDashboardViewController.swift:162:46: error: 'shared' is unavailable: Use view controller based solutions where appropriate instead.
        formerStatusBarStyle = UIApplication.shared.statusBarStyle
                                             ^~~~~~
UIKit.UIApplication:5:20: note: 'shared' has been explicitly marked unavailable here
    open class var shared: UIApplication { get }
                   ^
/Carthage/Checkouts/LifetimeTracker/Sources/UI/CircularDashboard/CircularDashboardViewController.swift:163:23: error: 'shared' is unavailable: Use view controller based solutions where appropriate instead.
        UIApplication.shared.statusBarStyle = .default
                      ^~~~~~
UIKit.UIApplication:5:20: note: 'shared' has been explicitly marked unavailable here
    open class var shared: UIApplication { get }
                   ^
/Carthage/Checkouts/LifetimeTracker/Sources/UI/CircularDashboard/CircularDashboardViewController.swift:190:23: error: 'shared' is unavailable: Use view controller based solutions where appropriate instead.
        UIApplication.shared.statusBarStyle = formerStatusBarStyle
                      ^~~~~~
UIKit.UIApplication:5:20: note: 'shared' has been explicitly marked unavailable here
    open class var shared: UIApplication { get }

The framework target currently sets Require Only App-Extension-Safe API to Yes currently. UIApplication.shared isn't available in app extensions.
I tried to only use the shared application if the library isn't compiled / running in an app extension, but didn't find a good solution yet.
@krzysztofzablocki Should the UI actually support app extensions? I'm not sure if it would work in any app extension type. At least in the document provider it wasn't shown.

I don't think the UI part would work in app extension, one could log the messages using just the tracking part

should be ok with #46 I think, reopen if not


it is not working in crome and opera

this api is not working google crome and Opera it displaying
this issue

it is working fine it required https for using in chrome thanks a lot....


Fix Common URLs for different posts with same name.

Describe the bug
The URL for posts is not unique, which results in a conflict for posts with the same name. If there are multiple posts with the exact same name, then the post which is the oldest will be opened every time when you click on Read More on any of the same name posts.
To Reproduce
Steps to reproduce the behavior:

Go to the project website.
Create an account or log in.
Compose 2 posts with the exact same name but with different body contents.
Click Read More on the newer post of the two.
Notice how you have been redirected to the other post with the same name.

Expected behavior
Ideally, the website should have a unique URL for every post, even for posts with the same name. So that the user gets correctly redirected to the post that he clicks on.
How to proceed

Currently the express route parameters for posts/:postName takes postName which is the post's title as the argument which can be common between posts.
Instead, try to give the post id as the argument to the route parameter since MongoDB maintains a unique id for every post therefore it will make our URL unique.

Hint: Look into how the posts are rendered in /views/home.ejs.

/level2/


game

Environment data

VS Code version: XXX
Jupyter Extension version (available under the Extensions sidebar): XXX
Python Extension version (available under the Extensions sidebar): XXX
OS (Windows | Mac | Linux distro) and version: XXX
Python and/or Anaconda version: XXX
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): XXX
Jupyter server running: Local | Remote | N/A

Expected behaviour
XXX
Actual behaviour
XXX
Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

XXX

Logs

Output for Jupyter in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Jupyter)

XXX

import random
class Player:
def init(self, name, health=100, attack=20, defense=10):
self.name = name
self.health = health
self.max_health = health
self.attack = attack
self.defense = defense
self.position = (random.randint(0, 50), random.randint(0, 50))
self.inventory = []
self.score = 0
def move(self, direction):
    x, y = self.position
    if direction == "up":
        y = min(y + 1, 50)
    elif direction == "down":
        y = max(y - 1, 0)
    elif direction == "left":
        x = max(x - 1, 0)
    elif direction == "right":
        x = min(x + 1, 50)
    self.position = (x, y)

def attack_player(self, other_player):
    damage = max(0, self.attack - other_player.defense)
    other_player.health -= damage
    print(f"{self.name} attacked {other_player.name} for {damage} damage!")
    if not other_player.is_alive():
        print(f"{other_player.name} has been defeated!")
        self.score += 1

def is_alive(self):
    return self.health > 0

def add_item(self, item):
    self.inventory.append(item)
    if item == "Health Potion":
        self.health = min(self.max_health, self.health + 50)
    elif item == "Sword":
        self.attack += 10
    elif item == "Shield":
        self.defense += 5
    elif item == "Armor":
        self.defense += 10
    print(f"{self.name} picked up {item}")

def use_ability(self, other_player):
    # Implement special abilities here
    pass

class Item:
def init(self, name, description):
self.name = name
self.description = description
def generate_items(num_items):
items = []
item_names = ["Health Potion", "Sword", "Shield", "Armor"]
item_descriptions = [
"Restores 50 health points.",
"Increases attack by 10.",
"Increases defense by 5.",
"Increases defense by 10."
]
for _ in range(num_items):
name = random.choice(item_names)
item_names.remove(name)
description = item_descriptions.pop(0)
items.append(Item(name, description))
return items
def print_player_status(player):
print(f"Player: {player.name}, Health: {player.health}/{player.max_health}, Attack: {player.attack}, Defense: {player.defense}, Score: {player.score}")
def main():
num_players = int(input("Enter the number of players: "))
players = [Player(f"Player {i+1}") for i in range(num_players)]
items = generate_items(num_players * 2)
while len(players) > 1:
    for player in players:
        if len(players) == 1:
            break
        print(f"\n{player.name}'s turn:")
        print_player_status(player)
        action = input("Enter your action (move/up/down/left/right/attack): ").lower()

        if action == "move":
            direction = input("Enter direction (up/down/left/right): ").lower()
            player.move(direction)
        elif action == "attack":
            target = input("Enter player name to attack: ")
            for other_player in players:
                if other_player.name.lower() == target.lower() and other_player != player:
                    player.attack_player(other_player)
                    break
            else:
                print("Invalid target!")
        else:
            print("Invalid action!")

        # Check for item pickup
        for item in items:
            if player.position == (item_pos := (random.randint(0, 50), random.randint(0, 50))):
                player.add_item(item.name)
                items.remove(item)

    # Print player status after each round
    print("\nPlayer Status:")
    for player in players:
        print_player_status(player)
    print("\n")

print(f"\n{players[0].name} wins the game with a score of {players[0].score}!")

if name == "main":
main()

Closing due to lack of information.
If there's a bug, please provide more information on the bug, etc by filling in the issue template.


Improve Contact Section

The improvements that need to be made in the contact section include :

Implementing a Contact Form for collecting users' queries.
Add SMTP configuration in the settings.py module
Creating a new view for the contact page (template) that utilizes SMTP for sending contact form responses to the configured Gmail account.

I want to work on this issue.
And also where are we hosting this website.

I want to work on this issue.
And also where are we hosting this website.

@codeit13 , can't assign the issue before 21st , till then check out the codebase and get the project running locally :)
Currently the website is being hosted in the pythonanywhere server.
Click here to view the deployed app

@codeit13 , can't assign the issue before 21st , till then check out the codebase and get the project running locally :)
Currently the website is being hosted in the pythonanywhere server.
Click here to view the deployed app

OK. Thanks

OK. Thanks

I want to work on the issue.

I want to work on the issue.

I can work on it , we can also try  email verification to eliminate the spam user ...how it will be sir ?

I can work on it , we can also try  email verification to eliminate the spam user ...how it will be sir ?

@amanjha8100  , I am assigning this issue to you. Make sure you are able to solve it on time else the issue will be assigned to next participant.

@amanjha8100  , I am assigning this issue to you. Make sure you are able to solve it on time else the issue will be assigned to next participant.

ok bhaiya

ok bhaiya

@7saikat7 , If the issue is not solved by @amanjha8100 then you will be assigned this issue after a period of 3 days.

@7saikat7 , If the issue is not solved by @amanjha8100 then you will be assigned this issue after a period of 3 days.


How to get 50% KB

Hi Sun,
You didn't give the preprocessing code in kb_05,kb_03,kb_01. I want to know how you get the 50% KB. Whether you get it by dropping the fact triples in freebase randomly at 50% propability or not?
Wish for your reply！

Yes, we randomly drop facts with the probability of 50%.

On Apr 29, 2021, at 8:48 AM, vongyx @.***> wrote:

Hi Sun,
You didn't give the preprocessing code in kb_05,kb_03,kb_01. I want to know how you get the 50% KB. Whether you get it by dropping the fact triples in freebase randomly at 50% propability or not?
Wish for your reply！
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Thanks for your help!


Governance while yosh is out of office

Hey all,
Just wanted to share that I'll be out of office starting tomorrow until December 19th. In the interim: please feel free to keep filing issues, make PRs, and merge them too. I trust folks here to exercise their judgement on which things to merge, and make sure changes are reviewed, etc. So far things have been going pretty steady, so please feel free to things going the way they are.
Both @matheus-consoli and @eholk have commit rights and can merge PRs. And @eholk also has the ability to publish new releases in a pinch. I don't expect a new release to have to be made, but should there be some critical issue that needs a hot fix: it's at least possible if I'm not around.
For anyone new to the project: welcome! I ask you to please be respectful of people's time. Other than me are folks here are really just volunteering their time to help out on this. While they may be kind enough to help in a pinch, they are under absolutely no obligation to do so. Should there be a larger issues or design questions: I'll be back in a few weeks and I'll be happy to discuss them then.
Anyway, I figured I'd let people know so nobody wonders why I'm suddenly radio-silent. See y'all in a few weeks!

Back!

Welcome back!


Custom social parameter

Hello,
I am using your theme for a while now, however I can not seem to find a way to add a custom social parameter, I would appreciate it if you help me

Thanks for using Ezhil, you can configure social links by adding params.social config block to config.toml. Check the sample config file here.
For example if you are adding Linked profile then config will look like
[[params.social]]
name = "Linkedin"
icon = "linkedin"
url = "https://linkedin.com"

You can search for icon names in Feather icons site.


Setting ASP.NET Core TagHerlper Attribute Without Encoding

I want to add the integrity attribute to a script tag in my tag helper. It contains a + sign which I don't want encoded.
<script integrity="sha384-Li9vy3DqF8tnTXuiaAJuML3ky+er10rcgNR/VqsVpcw+ThHmYcwiB1pbOxEbzJr7"></script>

This is my tag helper:
[HtmlTargetElement(Attributes = "script")]
public abstract class MyTagHelper : TagHelper
{
    public override async Task ProcessAsync(TagHelperContext context, TagHelperOutput output)
    {
        // Omitted...

        output.Attributes["integrity"] = "sha384-Li9vy3DqF8tnTXuiaAJuML3ky+er10rcgNR/VqsVpcw+ThHmYcwiB1pbOxEbzJr7";
    }
}

This is the output of the above code, where + has been replaced by +:
<script integrity="sha384-Li9vy3DqF8tnTXuiaAJuML3ky+er10rcgNR/VqsVpcw+ThHmYcwiB1pbOxEbzJr7"></script>

How can I stop this encoding from taking place?

This is the key:
output.Attributes[IntegrityAttributeName] = new TagHelperAttribute(IntegrityAttributeName, new HtmlString("sha384-Li9vy3DqF8tnTXuiaAJuML3ky+er10rcgNR/VqsVpcw+ThHmYcwiB1pbOxEbzJr7"));


WAC Update Tab Error for Hyper-V 2012 (Not R2)

As best as I can tell from the documentation for Windows Admin Center (WAC), Microsoft Hyper-V Server 2012 is supported, but when I connect to a Hyper-V 2012 server and navigate to the Updates tab, the following error is generated in the UI:

Notification Details
Error
(1) RemoteException: The specified module 'ScheduledTasks' was not loaded because no valid module file was found in any module directory. (2) RemoteException: The term 'Get-ScheduledTask' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

I have confirmed that this module is not present on any of my Hyper-V 2012 servers, but per the documentation for Windows Server 2012, the module was introduced and is standard on this generation of server operating systems. The WAC prep instructions do not list any additional steps beyond install WMF 5.1 (which has been manually applied to these systems) for Server 2012 systems, and there are no specific instructions for Hyper-V 2012 (while there are for Hyper-V 2012 R2 and Hyper-V 2016). Searching for ways to manually install this module has not been fruitful.
Having update management in a central console like this would be a huge help to administer systems over the long term. I am happy to provide further information if required, let me know what is needed.

Any word on if this is expected or not? Anything that can be done about getting this bug resolved?

Thanks for catching this! I've created a bug to investigate.


Add cpan parser

Hi, i added parser for Perl's cpanfile

@timgluz The tests just went through, everything is green 👍

cool


MNT: rerender

Hi! This is the friendly automated conda-forge-webservice.
I've started rerendering the recipe as instructed in #895.
If I find any needed changes to the recipe, I'll push them to this PR shortly. Thank you for waiting!
Here's a checklist to do before merging.

[ ] Bump the build number if needed

Fixes #895

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.


Correct usage of MP @ConfigurationProperties

Same as https://github.com/quarkus-qe/quarkus-test-suite/pull/351

Failure is unrelated to this PR, merging.


Can Tensorflow (not lite) run on Android NDK C++

It looks like features like Dropout are not supported on Tensorflow Lite + Android NDK.
I've got a Keras model that uses GPU, Dropout, CuDnnLSTM layer and I want to convert it to a tflite file or run it on Tensorflow for Android.
Is this possible? Is there a guide that tells me what is and is not supported on tflite?

Yes. Here is an example of using TensorFlow from C++ on Android.
https://github.com/tensorflow/tensorflow/tree/master/tensorflow/tools/benchmark

Closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Jack 3,5mm doesn't fit into the PCB

My PCB (ordered on Tindle) doesn't have the right holes (not like on the picture) to put the 3,5mm audio jack !
Does anyone had the same problem ?
Thanks !

Are you sure ? Cause it's seems to be exactly the same ...

This is the good one to use !
http://www.conrad.fr/ce/fr/product/705866/Jack-35-mm-Embase-femelle-horizontale-Cliff-FC68133-Nombre-total-de-poles-4-stereo-noir-1-pcs/?ref=detview1&rt=detview1&rb=1

The one in Squaro Engineering's DigiKey shopping cart is made by CUI and has pins that are symmetric with respect to the center of the audio plug. These pins do not fit the published PCB. The one in the original BOM is made by Lumberg and has asymmetric pins.

I decided to order the Lumberg part instead. DigiKey didn't have it, but other suppliers (Newark, for example) do...

As mentioned above, the Cliff FC68133 fits my PCB not the Lumberg part.
http://uk.farnell.com/cliff-electronic-components/fc68133/stereo-jack-3-5mm-5pos-pcb/dp/2518190

The semi-assembled PCB that is offered by Squaro Engineering on Tindie? Then there must be multiple versions. It certainly didn't fit mine...

Yea this Audio jack is hard to find in the US, I found it here:
http://www.newark.com/lumberg/klbr-4/phone-audio-connector-socket-3/dp/97K2709?ost=97K2709&ddkey=http%3Aen-US%2FElement14_US%2Fsearch
@jditte I also got the semi-assembled PCB for Squaro and this fits.
It fits, BUT I still cannot get sound.  getting this error:
Playback open error: -16, Device or resource busy

yes there seems to be some discussion on the audio jack. I just received my PCB from Squaro (ordered from Tindie) and tyhe one I got from the supplied BOM does not fit at all.
I got this one (DOES NOT FIT)
https://be.farnell.com/pro-signal/mj-352w-0/connector-receptacle-3-5mm-phono/dp/1267400?CMP=i-bf9f-00001000
holes look different on my PCB then they do on the PCB assembly manual website though...

@kaosbeat As mentioned before, this is the one that worked on my PCB from Squaro (ordered from Tindie):
https://be.farnell.com/lumberg/klbr-4/receptacle-3-5mm-phono-tht-r-a/dp/1217016
but I don't know whether they have redesigned the PCB in the meanwhile...


feat(timeout): new property for the notification component

Related Ticket(s)
Fix #291
Description
This changes add the property timeout to the components inline-notification and toast-notification. This new property will close the notification if a time (in ms) is specified after that time pass.
Changelog
New

New property timeout for: inline-notification, toast-notification.

No breaking changes.

I have a little question on this PR @asudoh , in the next line. Meanwhile this initial setup for the test it works I think it's not the best approach, probably.
For this use case I saw that Jest has Timer mocks but I couldn't make them work properly I don't know if you have a little more experience working with them, Akira. Thanks in advance as always!


Add ibt-0041-0041 firmware for AX210 BT

To avoid an infinite loop failing to load firmware for AX210 bluetooth
for new EM321 model. Note that bluetooth is not likely to work with this
since we need a newer kernel with newer drivers for that.
Note that we avoid adding all of linux-firmware-intel and not even all ibt-* firmware to keep the image size under control.

Approved, but I would add more information to the commit message, like for which device do you need this particular firmware.

@rene added to commit and description.


[FIX] Fixed cleaning old files in the export_to method

Description
Modified the condition for file deletion to ensure files older than 1 minute are removed.

I've added one comment. Is it possible we could also add a changelog for this and maybe a spec?

I cant merge this as it needs specs to ensure its working correctly with this new change (which its not)
This PR will be replaced by #12


Javascript tutorial

🛠 Fixes issue #72
📇 Changes proposed

Added resources for Javascript

✅ Check List (Mark all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[ ] This PR contains changes in the documentation
[ ] This PR contains changes in the Code of the project
[x] The title of my pull request is a short description of the requested changes.

📷 Screenshots

Hey, @Rishav1707 I would like to request some more resources from you to add in sequences (Beginner to expert level). It could be anything such as blog posts which can be useful for beginners

Hey, @Rishav1707 I would like to request some more resources from you to add in sequences (Beginner to expert level). It could be anything such as blog posts which can be useful for beginners

Okk

Hello @Rishav1707 ?

Hello @Rishav1707 ?

Yes @hasnainmakada-99

Have you made the changes @Rishav1707

Have you made the changes @Rishav1707

Yeah, the changes were already made.

See the review which I have started @Rishav1707 , The changes are not made

See the review which I have started @Rishav1707 , The changes are not made

which review? and from where?

Look out in the files changed tab

Look out in the files changed tab


Team 7 animals

Description
Improved the bullethell phase projectiles and difficulty
Fixes / Closes # (issue)
https://github.com/UQcsse3200/2024-studio-1/issues/430
More tests added
Type of change
Please delete options that are not relevant.
Feature Improvement
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Tests written for task are passing and playtested both bosses with the changed functionality
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

It'd be nice to see those 12 new issues resolved... they appear to all be in code this PR is responsible for, in particular the one with Random() appears to be a bug.

LGTM


[CS2103T-F12-1] HelloFile

PR subject: HelloFile
Description: By being able to tag frequently used files/folders with a short nickname, CS students are able to manage their files with ease. Moreover, they are able to identify files which are related, further enhancing efficiency when navigating file structures.

Codecov Report

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

@@            Coverage Diff            @@
##             master      #43   +/-   ##
=========================================
  Coverage     72.31%   72.31%
  Complexity      400      400
=========================================
  Files            70       70
  Lines          1228     1228
  Branches        124      124
=========================================
  Hits            888      888
  Misses          308      308
  Partials         32       32

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 6389d85...60ee593. Read the comment docs.

Codecov Report

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

@@            Coverage Diff            @@
##             master      #43   +/-   ##
=========================================
  Coverage          ?   72.31%
  Complexity        ?      400
=========================================
  Files             ?       70
  Lines             ?     1228
  Branches          ?      124
=========================================
  Hits              ?      888
  Misses            ?      308
  Partials          ?       32

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 6389d85...78effbe. Read the comment docs.


Added a sponsorships page

Added a good bunch of changes, please give it a read-through for typos and such or if I missed anything.


Update Discord Server Invite Link

Assign this issue to me.
The Discord Server invite link provided in the README is not functional.
I will replace the invalid Discord Server invite link (https://bit.ly/heyfoss) with a valid one.

The link is functional. Please check properly.


Pin sphinx-autodoc-typehints version to 1.11.1

Our docs build fails with 1.12.1: https://readthedocs.org/projects/jax/builds/13514541/
reading sources... [ 50%] _autosummary/jax.numpy.linalg.pinv

Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/jax/envs/latest/lib/python3.8/site-packages/sphinx/events.py", line 111, in emit
    results.append(listener.handler(self.app, *args))
  File "/home/docs/checkouts/readthedocs.org/user_builds/jax/envs/latest/lib/python3.8/site-packages/sphinx_autodoc_typehints.py", line 417, in process_docstring
    type_hints = get_all_type_hints(obj, name)
  File "/home/docs/checkouts/readthedocs.org/user_builds/jax/envs/latest/lib/python3.8/site-packages/sphinx_autodoc_typehints.py", line 271, in get_all_type_hints
    rv = backfill_type_hints(obj, name)
  File "/home/docs/checkouts/readthedocs.org/user_builds/jax/envs/latest/lib/python3.8/site-packages/sphinx_autodoc_typehints.py", line 312, in backfill_type_hints
    obj_ast = ast.parse(textwrap.dedent(
  File "/home/docs/.pyenv/versions/3.8.6/lib/python3.8/ast.py", line 47, in parse
    return compile(source, filename, mode, flags,
  File "<unknown>", line 12
    max_rows_cols = max(a.shape[-2:])
                                    ^
IndentationError: expected an indented block

This may be caused by https://github.com/agronholm/sphinx-autodoc-typehints/pull/166

For what it's worth, https://github.com/agronholm/sphinx-autodoc-typehints/pull/168 would have prevented this from breaking our doc build (IndentationError is a subclass of SyntaxError)


Order and line status verbose names

Fixes #1232

First of all - thanks for all your excellent hard work.
Forgive my OCD, but I'd prefer you to write commit messages as per this guide: http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html
Eg:

Keep the first line short and start it with a capital letter
Use the present tense rather than the past tense

You don't need to include the number of the issue you're fixing in every commit message - better to just put "Fixes #..." in the PR description. It would be useful to have more informative titles and descriptions in the PRs too.
I'll look at the code now...

Couple of things to correct - but we should be able to squeeze this into 1.1

@mvantellingen per our latest discussion - do we want to rework status pipeline on top of django-fsm?

Per latest discussion with @mvantellingen, we'll experiment with django-fsm in order to rework statuses pipeline in the next versions. Thus, closing this PR since none of this changes will be used in the refactoring mentioned above.


Getting error when trying to generate SBOM for my docker image

I'm trying to get the SBOM for my docker image that already located in my local with
cdxgen docker-image-name -o /data/sbom_generator/cdxgcen/docker.json -t docker
Giving Error:
(node:3420611) UnhandledPromiseRejectionWarning: TypeError: fs.rmSync is not a function
at Object.getOSPackages (/usr/local/lib/node_modules/@cyclonedx/cdxgen/binary.js:315:12)
at createMultiXBom (/usr/local/lib/node_modules/@cyclonedx/cdxgen/index.js:3566:48)
at Object.createBom (/usr/local/lib/node_modules/@cyclonedx/cdxgen/index.js:4266:27)
at runMicrotasks ()
at processTicksAndRejections (internal/process/task_queues.js:97:5)
at async /usr/local/lib/node_modules/@cyclonedx/cdxgen/bin/cdxgen:157:22
(node:3420611) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag --unhandled-rejections=strict (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 2)
I've also tried the same with
docker save -o /tmp/my.tar docker-image-name
cdxgen /tmp/my.tar -o /tmp/bom.json -t docker

This also gave the same error.
Thanks!

v12.22.9

Thanks @boravinod145. 8.2.1 includes a fix. Could you try and let me know how it looks?

Thanks @prabhu, after upgrading the node package version it's working fine.


saveMessages as HTMLText onClick events on buttons not working

hi, @FredrikOseberg I am implementing a chatbot that can save history. OnClick event any of the buttons are not working after closing and reopening the chat window. I am using the same widget to dynamically create chat messages, so using (messages) is not working. Please help
saveMessages = (messages,HTMLString)

const saveMessages = (messages, HTMLString) => {
localStorage.setItem('HTMLString', HTMLString);
};
const loadMessages = () => {
const messages = localStorage.getItem('HTMLString');
return messages;
};

I also want to know about saving conversation.
If you solve the issue, could you let me know?

Hi Guys....!! Did you solved it?

here is the solution.
[(https://github.com/FredrikOseberg/react-chatbot-kit/issues/146#event-9880393910)]

But how I can merge it in my project?


[SR-237][build-script] Migrate Ninja build to Python

What's in this pull request?
More work on [SR-237] Merge build-script-impl into build-script.
Migrate Ninja build to Python.
Also, some functional changes:

In OS X, use user configurable toolchain.cxx instead of xcrun --find clang++ to build Ninja.
When the system installed Ninja is not found, try to build Ninja if --foundation is requested as well as cmake_generator == "Ninja".
To build Foundation, even without --build-ninja, use build-script detected Ninja path instead of just ninja,  because the system installed Ninja can be named ninja-build.
Added debug trace output for commands used in build-script

Clean build directory rm -rf {build_root}
Create build directory mkdir -p {build_root}
Archive symbols package: tar -c -z -f ...

Possible regression:

Existence check of products source directories has been postponed until after Ninja was built.
Report stdlib targets, e.g. Building the standard library for: ..., after Ninja was built.
(Depends #2497)

Please tell me these changes are unacceptable.
Migrated impl args:
--build-ninja
--darwin-deployment-version-{osx,ios,tvos,watchos}
Removed impl args:
--build-ninja: Not used in build-script-impl anymore
Added impl args:
--ninja-bin: To propagate just built ninja path to build-script-impl
--
To make it testable, cherry-picked (and enhanced) dry-runnable shell interface from #2241.
Resolved bug number: (Partially SR-237)

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

@swift-ci Please test

@rintaro Thank you very much for the change and tests!

@swift-ci Please test

@rintaro LGTM, just a few small comments.

Fixed. And added missing xcrun.sdk_path test case.
Omitting stderr=subprocess.PIPE mess up python -m unittest discover output, but it's harmless.

@swift-ci Please test


feat: support different sizes for standard and trial instances in the create Kafka dialog with sizes

What this PR does / why we need it:
This addresses @jgiardino comment here:

This might be a bad design, but the original design was to accommodate this use case:

If I don't have a subscription and can only create a trial instance, then I can get some idea about what options are available with a subscription.

In this use case, the size option isn't meant to show me what trial sizes are available, but rather what sizes are available with a subscription. This is why in the example above, we show that there are two options, 1 SU and 2 SU, along with a message that size options are only available when you have a subscription.
I realize that how we're trying to convey these options in the UI isn't a clean map to what the API gives us, and it would be good to refactor the UI to better reflect what we're getting from the API. But when we chatted with Bilgin last week, it sounded like we likely have half a year to refactor this UI. I also would like to better understand what the next evolution of trial instances are and any expectations regarding how we represent trial instances to the user before refactoring this UI.
For this upcoming release, I would suggest we keep what's currently implemented in Quota Available - Trial, since this matches the design as expected.
But also, we definitely should not use the concept of a streaming unit when describing a trial instance. When we do refactor the UI, we need to remove the mapping of sizes to a "streaming unit" and just display the displayName property of a size, and maybe not use a slider at all.

Verification steps
Trial instances show the sizes available for standard instances in the slider, but the limits on the right pane are those for the trial instance.

Standard instances still work as they should

The answer to both comments is that the UI defined its own data model which might differ from the one used by the API. We do that to enforce stricter types and to decouple the UI from the API.

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


Updated documentation to show usage of the ssml methods.

@DanElbert : Thanks for merging the previous pull request. Please have a look at this documentation update and merge if all looks good to you.
Thanks


Improvement for k8s.io/docs/concepts/workloads/controllers/cron-jobs/

Incorrect apiVersion used in example (application/job/cronjob.yaml):
Currently used: apiVersion: batch/v1
Correct one: apiVersion: batch/v1beta1
Thanks for fixing it in the documentation (https://kubernetes.io/docs/concepts/workloads/controllers/cron-jobs/).

batch/v1 is used since 1.21 and batch/v1beta1 will be deprecated since 1.25 https://kubernetes.io/docs/reference/using-api/deprecation-guide/#cronjob-v125 So, I guess its correct to reference on batch/v1 in all docs from the 1.21 version.

/triage invalid
@cloudsapiens you must read the docs for the version of Kubernetes that you are running, otherwise you might assume the latest version's docs are incorrect when they are actually accurate.

/triage unresolved
/close
Nothing to fix here.


express.js use fluent-logger reports ECONNREFUSED <IP_ADDRESS>:24224

microk8s inspect tarbar inspection-report-20200616_060335.tar.gz
Ubuntu 20.04, microk8s 1.18/stable, addon enabled fluentd dns
Hi, In my express.js app, I followed the singleton example from the fluentd-logger, https://github.com/fluent/fluent-logger-node
var logger = require('fluent-logger')
logger.configure('tag_prefix', {
   host: 'localhost',
   port: 24224,
   timeout: 3.0,
   reconnectInterval: 600000 // 10 minutes
});
logger.emit('label', {record: 'this is a log'});

I provisioned fluentd's configMap, kubectl edit configmaps/fluentd-es-config-v0.1.5 -n kube-system to make sure the fluentd pods listens at 24224 port

But in my express.js app's log, it produced the following error,

Can anyone help on the issue? Is my way to modify the configMaps the correct way to enable listening?

The thing is localhost isn't pointing to the fluentd pod.
You probably need to

expose the port 24244 in the fluentd daemonset.
Example:

       ports:
        - name: http-port
          containerPort: 24224
          protocol: TCP

Create a Service that can be linked to the fluentd-ds.

apiVersion: v1
kind: Service
metadata:
  name: fluentd-service
  namespace: kube-system
spec:
  selector:
    k8s-app: fluentd-es
  ports:
    - protocol: TCP
      port: 80
      targetPort: 24224

From your code, instead of using localhost, use fluentd-service.kube-system.

Example:
var logger = require('fluent-logger')
logger.configure('tag_prefix', {
   host: 'fluentd-service.kube-system',
   port: 80,
   timeout: 3.0,
   reconnectInterval: 600000 // 10 minutes
});

This is assuming your expressjs app is running as a pod.

Thank you so much balchua, your method works.
Recap of the system

A very simple express.js server app for testing microk8s fluentd add-on. It integrates with fluent-logger and is deployed as a pod

In mk8s,  fluentd's configMap is provisioned to start listening at 24224 port

In mk8s, fluentd's daemonSet is provisioned to expose port 24224

In mk8s, a fluentd service is created to expose the fluentd pod(s) for the express.js app to connect. When configure the fluent-logger, the host is set as the service's address.

I still have difficulty to see the express.js app's log in the kibana; but I think it's my own problem.
Sidebar
To just play around the fluentd with node.js OUTSIDE of mk8s, you can

create your own version fluentd.conf with the following and save it in your host machine

<source>
  @type forward
  port 24224
</source>
<match **.*>
  @type stdout
</match>

launch docker

use -v to mount the host machine's fluentd.conf to a fluentd-elasticsearch container
expose 24224 for TCP (or UDP)
use fluentd argument -c <path to custom fluentd.conf> to specify a custom config

docker run -it -p 24224:24224 -p 24224:24224/udp -v /home/ubuntu/fluentd/etc/:/etc/fluentd/config.d -e FLUENTD_ARGS="--no-supervisor -c /etc/fluentd/config.d/fluentd.conf"  k8s.gcr.io/fluentd-elasticsearch:v2.2.0


[CSL-1572] Add tests to 'PureToss'

@rockbmb I want to do something with this PR. Can you squash your commits (because 3 commits basically fix the first one) and rebase on master?

@gromakovsky Oh, I thought this email was about the rocksdb-haskell PR when I saw it on my phone.
Yes, I can.

@gromakovsky Done 👍
Also, let me know if what you're doing is something I can help with.

@gromakovsky it appears adding this set of tests causes Travis to hang. In any case, I would need to temporarily disable them, were this PR to be merged immediately, because they aren't entirely correct (there is an explanation why here: https://github.com/input-output-hk/cardano-sl/blob/master/lib/test/Test/Pos/Update/PollSpec.hs#L208)
I'll fix this problem, and then I'll deactivate them for now.

@rockbmb if you're talking about

No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.

then it's not because of your tests, we have this problem in other branches, too.

@gromakovsky if that's the case, then I won't disable them, and tests are fixed now.


Optimize MSM pippenger: Trees of optional multi-scalar multiplications

Implement trees of optional multi-scalar multiplications, as described here:
Faster batch forgery identification
For a reference implementation, see Gnark:
This needs to be benchmarked against the old implementation

Moved to #730


try data tree instead of group

due to issues with the grouping, decided to try out the Data Tree module  for the Categories table instead

thought we only had cases where there is no subcategory, just main category -> goods, which this data tree approach actually solves, but there seem to be cases where a main category does have subcategories, but there are also goods in the main category that don't belong to any subcategory...
This might be fine for the table, but causes problems for the category barchart
Also, we need a custom filter for the first column, because child nodes are not matched. For the others everything works like expected, you get the parent node and any matching child nodes


External authentication cookie not saved in Identity.External

[x] I read and understood how to enable logging

Issue
AspCore.Identity uses the "Identity.External" scheme to retrieve the external login provider: https://github.com/aspnet/Identity/blob/dev/src/Microsoft.AspNetCore.Identity/SignInManager.cs#L557
However IdentityServer sets the external authentication cookie into the idsrv.external scheme, resulting in a null response.

I did not find the part where IdentityServer sets the cookie into the idsrv.external namespace, or else I would have greated a pull request.
As a workaround I created a custom SignInManager that uses the IdentityServerConstants.ExternalScheme to retrieve the LoginInformation.

If you're using ASP.NET Identity and its SignInManager, then you need to code your UI to use that (as opposed to our QuickStart UI which uses our external scheme). IOW, you need to make those consistent.

What do you mean by that? I was using the Quickstart template and it uses
the SigninManager which does not work
On Sat 2. Dec 2017 at 19:36, Brock Allen<EMAIL_ADDRESS>wrote:

If you're using ASP.NET Identity and its SignInManager, then you need to
code your UI to use that (as opposed to our QuickStart UI which uses our
external scheme). IOW, you need to make those consistent.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/IdentityServer/IdentityServer4/issues/1826#issuecomment-348711127,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAuGfyExvZJn9WKSQ2VfAzyBIcAVL5B_ks5s8ZjFgaJpZM4QzVKv
.

So then you see this behavior in the ASP.NET Identity quickstart?

Yes, this is the behavior out of the box when using the Asp Identity, Ef, or combined quickstart.

I just ran the ASP.NET Identity QS 6 in chrome and it worked fine.

For me it does not work. I cloned the QS 6 and ran it. var info = await _signInManager.GetExternalLoginInfoAsync(); is null.

https://github.com/IdentityServer/IdentityServer4.Samples/blob/release/Quickstarts/6_AspNetIdentity/src/IdentityServerWithAspNetIdentity/Controllers/AccountController.cs#L327

So I also just ran the other two you mention (QS 8 and the combined one) and they also work.
It sounds environmental to me. I'd suggest debugging more into the external cookie and see which one you do have.

You are right. When I open the page in incognito mode everything works. Seems to be some caching issue.

I agreed with max-buecker, it resolve my problem when options.SignInScheme as leaving default.
.AddOpenIdConnect("IS4", "OpenID Connect", options =>
             {
                 **//**options.SignInScheme = CookieAuthenticationDefaults.AuthenticationScheme;


sansuj1982 file

Created a branch name as my_slide and created a file in the post directory. This file includes the layout for the portal. now i created a pull request for review and feedback.

Updated  file


Uncaught Exception: EBUSY: resource busy or locked, open 'C:\Users\bruto\AppData\Roaming\Google Play Music Desktop Playe

An uncaught exception was reported.
Info:
OS: win32
Arch: ia32
GPMDP Version: 3.3.0
Time: Fri Jul 08 2016 12:31:39 GMT-0600 (Mountain Daylight Time)
Error:
EBUSY: resource busy or locked, open 'C:\Users\bruto\AppData\Roaming\Google Play Music Desktop Playe
Error: EBUSY: resource busy or locked, open 'C:\Users\bruto\AppData\Roaming\Google Play Music Desktop Player\json_store\.settings.json'
    at Error (native)
    at Object.fs.openSync (fs.js:634:18)
    at Object.module.(anonymous function) [as openSync] (ELECTRON_ASAR.js:158:20)
    at Object.fs.writeFileSync (fs.js:1337:33)
    at t.value (C:\Users\bruto\AppData\Local\GPMDP_3\app-3.3.0\resources\app\build\main\utils\Settings.js:9:2252)
    at tryOnTimeout (timers.js:224:11)
    at Timer.listOnTimeout (timers.js:198:5)

This issue was created automatically inside the uncaughtException handler

We should implement some kind of error catching and retry on L70 of Settings.js


Investigate why some files are not injected into PhEDEx

Creating this issue so we do not forget to investigate and fix it once and for all.
Everytime we need to redeploy a new agent, we find lots of files that are not injected into PhEDEx (it seems they are only injected into DBS), and then we need to run the well known phedex-fix.sh script.
This is the query that always returns lots of files not injected:
mysql> select lfn from dbsbuffer_file where in_phedex=0 and lfn not like '%unmerged%';

 Investigate why some files are not injected into PhEDEx


update linux.md

Hello @Gyan-Singh
Thanks for your contribution.
Let's connect ->
GitHub -> @Suryansh1720001 (Follow)
LinkedIn -> https://www.linkedin.com/in/suryanshprajapati17
Also, Star the repo ⭐


Crashing when hovering over data>export in the text editor

Smithbox - Version 1.0.23
Object reference not set to an instance of an object.
at StudioCore.Editors.TextEditor.FmgImporter.MenubarOptions() in ./src/StudioCore/Editors/TextEditor/Data/FmgImporter.cs:line 45
at StudioCore.TextEditor.TextEditorScreen.EditorUniqueDropdowns() in ./src/StudioCore/Editors/TextEditor/TextEditorScreen.cs:line 190
at StudioCore.Smithbox.Update(Single deltaseconds) in ./src/StudioCore/Smithbox.cs:line 512
at StudioCore.Smithbox.Run() in ./src/StudioCore/Smithbox.cs:line 353
at Smithbox.Program.Main(String[] args) in ./src/Studio.App/Smithbox/Program.cs:line 36

Also with another project i am able to use the export, the project file that breaks is clevers moveset with like 5 other params edited for fp regen

It happens if you don't select a File or Text File I believe. It's been fixed for the next release.


adding readme to provide information on how to run the example

what
Adding a readme to provide information on how to run the example/complete.
why
The example is not self contained and needs to be run in parts using terraform -target=  construct.

/terratest


[Ubuntu] Add Microsoft.Graph module

Description
Add Microsoft.Graph module on Ubuntu Server 18.04/20.04.
Related issue:
https://github.com/actions/virtual-environments/issues/5255
Check list

[x] Related issue / work item is attached
[x] Tests are written (if applicable)
[x] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run ubuntu1804, ubuntu2004


Created sheet for FHTAGN RPG fhtagn-rpg.de

Created sheet for FHTAGN RPG fhtagn-rpg.de
created markup and styles for first page
created markup and styles for first page
version 0.1.0
bugfixing and thumbnail
Changes / Comments
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

✅   Does the pull request title have the sheet name(s)? Include each sheet name.
❌   Is this a bug fix?
Does this add functional enhancements (new features or extending existing features) ?
Does this add or change functional aesthetics (such as layout or color scheme) ?
If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
It's a completely new sheet, so these do not apply
✅   If this is a new sheet, did you follow Building Character Sheets standards ?


Mb/feature/info

Close #233

Update:
Current output for:
static Function Test7()
	variable i
	for(i = 0; i < 7; ++i)
		INFO("Loop: %d", n0 = i)
		INFO("Is smaller than 5: %d", n={ i < 5 })
		CHECK(i < 5)
	endfor
End

static Function Test8()
	INFO("empty testcase")
End

  Start of test "Unnamed"
  Entering test suite "Proc0"
  Entering test case "mymodule#Test7"
  0: is false. Assertion "CHECK(i < 5)" failed in Test7 (Proc0, line 55)
    ⓘ Loop: 5
    ⓘ Is smaller than 5: 0
  0: is false. Assertion "CHECK(i < 5)" failed in Test7 (Proc0, line 55)
    ⓘ Loop: 6
    ⓘ Is smaller than 5: 0
  Leaving test case "mymodule#Test7"
  Entering test case "mymodule#Test8"
  Test case "mymodule#Test8" doesn't contain at least one assertion
  Leaving test case "mymodule#Test8"
  Failed with 5 errors
  Leaving test suite "Proc0"
  Test finished with 5 errors
    ▶ Assertion "CHECK(i < 5)" failed in Test7 (Proc0, line 55)
    ▶ Assertion "CHECK(i < 5)" failed in Test7 (Proc0, line 55)
    ▶ Test case "mymodule#Test8" doesn't contain at least one assertion
  End of test "Unnamed"

@Garados007 Can I skip this one and give that to @MichaelHuth for review? Thanks.

Can I skip this one and give that to MichaelHuth for review? Thanks.

Yes. It wasn't your turn anyway. 😉

Can we have an example experiment/tests which trigger all the cases where the INFO should be ouptutted and/or reset?
Another example in the documentation?

I have implemented all suggestions.


fix bedrock inference region

The Claude 3.5 Sonnet v2 model doesn't work unless I SSH into the container and update this line to the US inference profile ID
If I don't change this then I get error saying:
Error code: 400 - {'message': 'Invocation of model ID anthropic.claude-3-5-sonnet-20241022-v2:0 with on-demand throughput isn’t supported. Retry your request with the ID or ARN of an inference profile that contains this model.'}

It's a blocker for me to use bedrock as well.

@buryhuang try manually editing the loop.py file until a change is made in the docker image


I need TensorFlow 2.2.0 but it is removed how to find it?

I need to install TensorFlow 2.2.0
why? because this repo (https://github.com/GantMan/nsfw_model) is requesting it and now matter what I tried can't make it work with newer TensorFlows
How can I install TensorFlow 2.2.0  on Windows 10 and Python 3.10?
The error I am getting is and I am not able to fix it
(venv) G:\nsfw_model>python a.py
2023-08-14 00:33:39.437427: I tensorflow/core/platform/cpu_feature_guard.cc:151] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX AVX2
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2023-08-14 00:33:40.148302: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1525] Created device /job:localhost/replica:0/task:0/device:GPU:0 with 21643 MB memory:  -> device: 0, name: NVIDIA GeForce RTX 3090 Ti, pci bus id: 0000:01:00.0, compute capability: 8.6
2023-08-14 00:33:40.149891: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1525] Created device /job:localhost/replica:0/task:0/device:GPU:1 with 9603 MB memory:  -> device: 1, name: NVIDIA GeForce RTX 3060, pci bus id: 0000:05:00.0, compute capability: 8.6
Traceback (most recent call last):
  File "G:\nsfw_model\a.py", line 13, in <module>
    print(predict.classify(model, 'test'))
  File "G:\nsfw_model\nsfw_detector\predict.py", line 67, in classify
    probs = classify_nd(model, images, predict_args)
  File "G:\nsfw_model\nsfw_detector\predict.py", line 77, in classify_nd
    model_preds = model.predict(nd_images, **predict_args)
  File "G:\nsfw_model\venv\lib\site-packages\keras\utils\traceback_utils.py", line 67, in error_handler
    raise e.with_traceback(filtered_tb) from None
  File "G:\nsfw_model\venv\lib\site-packages\keras\engine\training.py", line 1997, in predict
    raise ValueError('Unexpected result of `predict_function` '
ValueError: Unexpected result of `predict_function` (Empty batch_outputs). Please use `Model.compile(..., run_eagerly=True)`, or `tf.config.run_functions_eagerly(True)` for more information of where went wrong, or file a issue/bug to `tf.keras`.

predict.py
#! python

import argparse
import json
from os import listdir
from os.path import isfile, join, exists, isdir, abspath

import numpy as np
import tensorflow as tf
from tensorflow import keras
import tensorflow_hub as hub

IMAGE_DIM = 299   # required/default image dimensionality

def load_images(image_paths, image_size, verbose=True):
    '''
    Function for loading images into numpy arrays for passing to model.predict
    inputs:
        image_paths: list of image paths to load
        image_size: size into which images should be resized
        verbose: show all of the image path and sizes loaded

    outputs:
        loaded_images: loaded images on which keras model can run predictions
        loaded_image_indexes: paths of images which the function is able to process

    '''
    loaded_images = []
    loaded_image_paths = []

    if isdir(image_paths):
        parent = abspath(image_paths)
        image_paths = [join(parent, f) for f in listdir(image_paths) if isfile(join(parent, f))]
    elif isfile(image_paths):
        image_paths = [image_paths]

    for img_path in image_paths:
        try:
            if verbose:
                print(img_path, "size:", image_size)
            image = keras.preprocessing.image.load_img(img_path, target_size=image_size)
            image = keras.preprocessing.image.img_to_array(image)
            image /= 255
            loaded_images.append(image)
            loaded_image_paths.append(img_path)
        except Exception as ex:
            print("Image Load Failure: ", img_path, ex)

    return np.asarray(loaded_images), loaded_image_paths

def load_model(model_path):
    if model_path is None or not exists(model_path):
    	raise ValueError("saved_model_path must be the valid directory of a saved model to load.")

    model = tf.keras.models.load_model(model_path, custom_objects={'KerasLayer': hub.KerasLayer},compile=False)
    return model

def classify(model, input_paths, image_dim=IMAGE_DIM, predict_args={}):
    """
    Classify given a model, input paths (could be single string), and image dimensionality.

    Optionally, pass predict_args that will be passed to tf.keras.Model.predict().
    """
    images, image_paths = load_images(input_paths, (image_dim, image_dim))
    probs = classify_nd(model, images, predict_args)
    return dict(zip(image_paths, probs))

def classify_nd(model, nd_images, predict_args={}):
    """
    Classify given a model, image array (numpy)

    Optionally, pass predict_args that will be passed to tf.keras.Model.predict().
    """
    model_preds = model.predict(nd_images, **predict_args)
    # preds = np.argsort(model_preds, axis = 1).tolist()

    categories = ['drawings', 'hentai', 'neutral', 'porn', 'sexy']

    probs = []
    for i, single_preds in enumerate(model_preds):
        single_probs = {}
        for j, pred in enumerate(single_preds):
            single_probs[categories[j]] = float(pred)
        probs.append(single_probs)
    return probs

def main(args=None):
    parser = argparse.ArgumentParser(
        description="""A script to perform NFSW classification of images""",
        epilog="""
        Launch with default model and a test image
            python nsfw_detector/predict.py --saved_model_path mobilenet_v2_140_224 --image_source test.jpg
    """, formatter_class=argparse.RawTextHelpFormatter)

    submain = parser.add_argument_group('main execution and evaluation functionality')
    submain.add_argument('--image_source', dest='image_source', type=str, required=True,
                            help='A directory of images or a single image to classify')
    submain.add_argument('--saved_model_path', dest='saved_model_path', type=str, required=True,
                            help='The model to load')
    submain.add_argument('--image_dim', dest='image_dim', type=int, default=IMAGE_DIM,
                            help="The square dimension of the model's input shape")
    if args is not None:
        config = vars(parser.parse_args(args))
    else:
        config = vars(parser.parse_args())

    if config['image_source'] is None or not exists(config['image_source']):
    	raise ValueError("image_source must be a valid directory with images or a single image to classify.")

    model = load_model(config['saved_model_path'])
    image_preds = classify(model, config['image_source'], config['image_dim'])
    print(json.dumps(image_preds, indent=2), '\n')

if __name__ == "__main__":
	main()

Any help is appreciated
@vatsalkachhia @SuryanarayanaY @tilakrayal @sun1638650145 @pjpratik

found the issue
it requires python 3.8


feat(errors): capture errors thrown in redirect callback in onError

Resolves  #3201
This PR allows exceptions to be caught in router.onError.

Hi @posva ! Yes you are right, try catching router.match should be enough. I just thought it might be a good idea to also catch errors in the confirmTransition.
I will make the changes later, thanks for the feedback :)


Function request: LightBox

Requestion for a function to add an Lightbox function to images. (Click on image to view an large version)

thnx.
Please add an z-index to:
.ejs-lightbox{
z-index:5000; //increase if lightbox is not on top
}


Add scrollex

Will continue once the size of the lib will be smaller...
Though it looks really neat now!


human redable flag for bytes

Hi there. As the title says, what about to add an additional flag (-h maybe?) for an human readable output in the total byte size section? It's a lot more confortable reading file sizes using KB,MB,GB,TB etc...

no problem!
I will implement your request in the next version.
thank you for your request!

I just implemented your request in version 1.0.3, please find the new version product.

@tamada thank you for the ultrafast implementation. Seems to work good!


React example using ES6 classes

Simply the CheckboxWithLabel example re-written with ES6 classes<EMAIL_ADDRESS>

Sorry, forgot about the CLA. I've signed now.

cc @sebmarkbage
Mind reviewing this really quickly?

lgtm. I'll leave it to @jeffmo to decide if we want semi-colons in the examples or not.

I'm personally not a big fan of semicolons but if you guys prefer to have them I'll add them, no problem :)

In general I don't care either way. My only concern is that n00bs coming to this page might think that something has changed with regard to semi-colons - incorrectly attributing the choice to ES6.

Yep, I see your point and I agree :)
Just let me know what do you guys prefer then.

Review by committee! Let's add the semicolons. All of our code uses them.

@sebmarkbage @zpao alright, semicolons added :)


Metadata URL resource

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
Pingfederate has the ability to consume metadata endpoints for SAML providers and these can then be associate to a specific SAML connection inside PingFederate

New or Affected Resource(s)

pingfederate_metadata_url

Potential Terraform Configuration

resource "pingfederate_metadata_url" "example_metadata_url" {
  name = "example_organisation"
  url  = "https://example.com/FederationMetadata.xml"
  x509_file {
    file_data = file("example.pem")
  }
  validateSignature = true
}

References

pf-admin-api/v1/metadataUrls

This has been released in v0.2.0


StandardMaterial3D Distance Fade: Object Dither mode not working as expected (problem with model origin)

Tested versions
v4.2.1.stable.official [b09f793f5]
System information
Godot v4.2.1.stable - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce GTX 1060 with Max-Q Design (NVIDIA; <IP_ADDRESS>76) - Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz (8 Threads)
Issue description
I'm trying to make a third person character controller. I want that if the camera is very close to the character, it will become translucent. But when using Distance Fade in StandardMaterial3D, specifically the ObjectDither mode, it does not work like other modes (PixelAlpha, PixelDither).
It looks like ObjectDither mode uses the origin of the model when calculating the distance to the camera. Which leads to incorrect display if the origin of the model is not in its center.
Visual demonstration (all modes with the same settings, Min Distance = 0m, Max Distance = 2m):

https://github.com/godotengine/godot/assets/4838367/63b6d1df-517f-4db3-9bbf-3531505a83bb
https://github.com/godotengine/godot/assets/4838367/a45ef66b-a32b-4c6c-b231-62a42e2b95af
https://github.com/godotengine/godot/assets/4838367/b4e90d87-1575-446e-9173-49adbe342b5d
As you can see, only in ObjectDither mode when the camera flies above the player, the dithering effect does not occur because the origin of the model is not in its center, but below.

Steps to reproduce
In attached example launch the scene, then in World -> Player -> Visuals -> capsule -> Circle -> Surface Material Override play with the Distance Mode.

Just in case, I attached a capsule model made in Blender with the required origin to reproduce the bug. The scene uses its local copy.
Minimal reproduction project (MRP)
StandardMaterial3D_ObjectDither.zip

This isn't a bug. This is exactly what object dither does. The dithering is based on the distance from the camera to the origin of the object.

The dithering is based on the distance from the camera to the origin of the object.

That's the problem. I understand that from a software point of view, such an implementation seems correct, but this is not what the user expects. Because the origin point may be located outside the model geometry. And in this case, object dithering does not work correctly (see attached videos in the first message).
Quote from the Blender documentation that origin can be located outside the model:

The origin does not have to be located in the center of the geometry (e.g. mesh). This means that an object can have its origin located on one end of the mesh or even completely outside the mesh.

Much more expected behavior for the user will be when the object dithering depends on the distance of the camera to the visible geometry. Like in pixel alpha/dither, but only so that it is applied evenly to the entire model.
Therefore, please reopen this issue because Godot does not handle common standards properly. Maybe not as a bug, but as an enhancement if you think it's more correct.
For example, in character models the origin is often placed at the bottom (see attached videos in the first message), and although the Godot documentation recommends using Object Dither to hide the character, in practice this does not work correctly.
Godot documentation:

To hide a character when they get too close to the camera, consider using Pixel Dither or better, Object Dither (which is even faster than Pixel Dither).

Therefore, please reopen this issue because Godot does not handle common standards properly. Maybe not as a bug, but as an enhancement if you think it's more correct.

Feature proposals are tracked on https://github.com/godotengine/godot-proposals, so please open an issue there.


Todo list: Review best practices

[Dynamic Todo List]
[Read Tooo List from Local Storage]
[Add a New Item Todo List]
[Edit an Item from The Todo List]
[Remove an Item from The Todo List]
[Mark an Item of Todo List as Completed]
[Remove All Items of Todo List that have marked as Completed]
[Reset Todo list indexes]
[Sort Todo list by index]

Status: Approved 🟢 🟢 🟢
Hi, @BB-Simon 👋,
Highlights💡
Good commit message.✔
Good PR Summary.✔
Great UI design.✔
App functions as it should.✔
Your project is complete! There are the following suggested changes before you merge in order to make your code Clean and DRY!
Congratulations! 🎉
Optional Changes ♻
Check the comments under the review.


#117: Import through SASS URL does not work

#177
Regex was changed in the last version of Next.JS  - https://github.com/vercel/next.js/blob/master/packages/next/build/webpack/config/blocks/css/index.ts#L17

Hi, @cyrilwanner, could you please merge and publish a new version?

hey guys, any updates on this? @cyrilwanner @astorozhevsky it would help a lot on our project here, because right now we are using backgroundImage with style tag and require as workaround 😢


Fix "data-preparation-and-post-processing.ipynb"

Description
In "data-preparation-and-post-processing.ipynb", the file path "./kaggle-competition/train.txt" does not lead to the correct directory. Also, there's a variable "b" that has not been declared and it causes error.
changes

[ ] Change ""./kaggle-competition/train.txt" to "../kaggle-competition/train.txt" to correct the file path
[ ] remove an undeclared variable "b"

LGTM.
The notebook was made for a quick prototype. IMHO, we should migrate what inside the notebook to source code files. So, it's easier to cross-check and reproduce results.
@bact if you don't have any objection, i'm going to merge this PR.


Calling entangled without arguments should open help

Kinda new to entangled, but it looks good so far!
Unless entangled without arguments is already being used for something else, it should probably be used to print the help doc. This just makes it quicker to experiment with entangled for new users.

Thanks! I agree, this should be a quick fix.


Please invite me to the GitHub Community Organization

Name
Fábio Almeida
Discord Username (if applicable)
No response
Additional Context
Hello! My Name if Fábio (aka FabIn).
I meet Eddie though other GitHub friend! I would like to join for contribute with others devs and I hope help improve the OpenSource projects here. I like the approach about of the Eddie in relation the collaboration.

Welcome :D


feat: add activity directive metadata fields to activity form

Partially resolves AERIE-1952

Fieldset styling remains an issue here @camargo because of nested fieldsets so the styling will look odd until we resolve this in a separate PR. I could also throw in some temporary style overrides if desired.


Logo and Banner Area

The logo could be resized to better fit the header. Currently, the size seems slightly overwhelming, and more spacing between the logo and the navigation links might improve balance.

Logo size whould be small


Creating a loop (not submitting workflow) causes an uninterruptible error.

https://github.com/comfyanonymous/ComfyUI/issues/2077

fixed in 4.3.5

still has this bug

Please post workflow, as this might be correct behaviour if there is a loop

@chrisgoringe
Sorry, My bad. I found out.
I had linked wrong VAE port so that efficient node shows warning ,but it didn't shows up when I was installed cg-use-everywhere plugin.  So I googled & found some answer form here.
This is wrong.json
bug.json
Screenshot (without installed cg-use-everywhere plugin):

The mistake is wrong linked with VAE port from efficient node.
Screenshot (with cg-use-everywhere plugin):

efficient node's warning is missing, just popup like this.
When I linked VAE correctly from efficient node, whether or not everywhere is installed is correct.


Enhance Navbar Responsiveness and Design

Issue Description:

Currently, our Navbar is not responsive for tablet and mobile views, and it's only visible on larger screens. Additionally, it lacks a modern and visually appealing design. To provide a better user experience and a sleek look, we need to make the Navbar responsive, add a maximum width, and incorporate a Glass morphism effect.

Requirements:

Screen Size Responsiveness: Implement a toggle menu for tablet and mobile views to ensure that the Navbar adapts to smaller screen sizes. Users should be able to access the menu easily on these devices.

Maximum Navbar Width: Fix the maximum width of the Navbar for larger screens to maintain a consistent layout. This prevents the Navbar from extending too wide on larger displays.

Glass Morphism Effect: Apply the Glass morphism effect to the Navbar elements. This effect adds a modern and visually appealing touch to the Navbar's design.

These improvements will enhance the usability and aesthetics of our Navbar across various screen sizes. If you have any questions, suggestions, or ideas for the implementation of these enhancements, please feel free to share them in the comments.
Thank you for your commitment to improving our website's user interface and design!

Hello, @ajaynegi45! May I join in and work on this?

@ajaynegi45 hi, i can fix this for you, i would like to work on this issue, please assign this to me, notice that i will not start workin on this issue before getting assignment, thanks for your understanding

hello @ajaynegi45 , i would like to contribute to this issue, please assign me, thanks

Can I work on this?

Hello @anubhav1206 @Sky-De @waleedkhano @Raghu1124,
Thank you all for your interest in contributing to this task. I appreciate your willingness to help improve our navbar
To make this process more collaborative, I'm assigning this task to all of you. You can each work on your own version. In the end, we will evaluate the contributions and choose the one that best aligns with our project's needs and goals.
Please coordinate among yourselves and feel free to discuss any questions or ideas you may have during the process. Once you've completed your contributions, please submit them for review.
Looking forward to seeing your work and thank you for your valuable contributions! ❤


Refacto codes

I refactored codes based on our discussion yesterday.

Pull Request Test Coverage Report for Build 38

0 of 1 (0.0%) changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 62.978%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

Common.py
0
1
0.0%

Totals

Change from base Build 36:
0.04%

Covered Lines:
949

Relevant Lines:
1399

💛  - Coveralls


Add Station

Tech
Description
When a new station was added to the system, this is indicated on the GUI.
Trigger
A new station is available
Precondition
The system is running in normal mode.
Result
The new station will be displayed and values may be entered.
Postcondition
New station is integrated like any other station in the
overall system
Basic flow

The system creates randomly a new station.
The system is notifying all display components of the
new station
The user can select new station and
can use it.

Closed with issue #14


V 0.2.1: special characters are not displayed correct, %-ASCII

Describe the bug
After Update to version 0.2.1 the Textmessages are displayed not correct. spaces and German Umlaute (like Ä,Ü,Ö,..) seems to be in a html eroding.
To Reproduce
Steps to reproduce the behavior:
Nothing changed but I used the adapter to inform me via adapter device reminder. so before I run the update, the text was correct.

start e.g. washing Maschine
text message via device reminder to whatsapp is sendet
check text message
used Text: "Waschmaschine wurde gestartet"

Expected behavior
Text are displayed as configured, "Waschmaschine wurde gestartet"
Observed  behavior
"Waschmaschine%20wurde%20gestartet"
Screenshots & Logfiles

Versions:

Adapter version: 0.2.1
JS-Controller version: 4.0.21
Node version: 14.19.0
Operating system: ubuntu 21.10 (proxmox lxc)

Additional context
in device reminder are 2 users configured (my wife and me), but my wife uses telegram. the text is for sending via WhatsApp and telegram is set up onetime and both messenger adapters uses them. the Error occurs only in WhatsApp adapter, text in telegram are still fine.

Fixed on GitHub already ... 0.2.2 on it's way to github. repo update then later
Thank you for reporting


We should use consistent naming prefix for internal maps

https://github.com/vert-x3/vertx-hazelcast/blob/master/src/main/resources/default-cluster.xml#L77
As discussed.

+1

BTW, was there a resolution on that clustering issue you were looking into before the 3.1 release?

It seems to me that all internal maps use the __vertx. prefix in master.


Update virtual-network-service-endpoints-overview.md

change "private IPv4" to "private IPv6" as per what I just validated happens

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

Reviewing with PM.  A service endpoint to my knowledge doesn't change the public IPV4 address to a private IPv6 address.  Will check accuracy of the changes and change in another PR.
#please-close


Index not updating

Hello.
I've bound onMomentumScrollEnd and there I'm console logging the ref to my Swiper.
The state.index is always set to 0.
Is that a bug or is there another way to find out which node is viewed?

Seems like if I don't bind onMomentumScrollEnd, the index updates.
So when binding onTouchStartCapture everything works better.
Still not sure if that was a bug, but this works for me.

Thanks @subpublicanders
First update index, then call onMomentumScrollEnd, is this helpfully? or I am very pleased to follow up your problem.
/**
   * Scroll end handle
   * @param  {object} e native event
   */
  onScrollEnd(e) {

    // update scroll state
    this.setState({
      isScrolling: false
    })

    let offset = e.nativeEvent.contentOffset
    this.updateIndex(offset, this.state.dir)

    // if `onMomentumScrollEnd` registered will be called here
    this.props.onMomentumScrollEnd && this.props.onMomentumScrollEnd.call(this)
  },

Well, I solved it by not using onMoementumScrollEnd at all.
If i bind onMoementumScrollEnd in my Swiper-jsx, then the onScrollEnd is never even called.
I set a console.log in your code to try it. Could it have something to do with the default render (line 389)? That it gets overridden.

oh yes, onMoementumScrollEnd be overridden, thanks, i will fix it later.

@leecade when im loading content from some rest service, onMomentumScrollEnd and onIndexxhanged related is not working till the response returns back to the application from API. can you have any idea on this ???


[Bug]: Form default values not displayed in dialog component

Reproduction
https://stackblitz.com/edit/vitejs-vite-kztmwi?file=src%2FApp.vue
Describe the bug
Description
Default values are not displaying in forms within the dialog component after the first opening.
Steps to reproduce

Click to open the dialog.

Close the dialog.
Open the dialog again.

Expected behavior
The default values within the form should remain visible upon subsequent openings of the dialog.
System Info
System:
    OS: Linux 5.0 undefined
    CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 18.18.0 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 10.2.3 - /usr/local/bin/npm
    pnpm: 8.15.3 - /usr/local/bin/pnpm
  npmPackages:
    @vueuse/core: ^10.9.0 => 10.9.0
    radix-vue: ^1.6.1 => 1.6.1
    vue: ^3.4.21 => 3.4.21

Contributes

[ ] I am willing to submit a PR to fix this issue
[ ] I am willing to submit a PR with failing tests

@A-kirami Hi
in Dialogs it's better to use the vee-validate Form component
stackblitz.com/edit/vitejs-vite-4h9oqf

Thank you for the prompt response and solution!
I would like to suggest adding documentation about this issue and its resolution to the project's documentation. It would be helpful for users who might encounter similar issues in the future.

@A-kirami Hi
in Dialogs it's better to use the vee-validate Form component
stackblitz.com/edit/vitejs-vite-4h9oqf
If you want to use useForm composable instead, you need to create a custom hard-coded form component based on your form fields

Sorry, I'm not quite sure what a custom hard-coded form component is. Could you provide documentation or a demo? Thanks!

@A-kirami Sure will update stackblitz

@A-kirami https://stackblitz.com/edit/vitejs-vite-b87ysk

@A-kirami stackblitz.com/edit/vitejs-vite-b87ysk

Thank you for providing the demo! This resolves the issue for me.


Update Examples FILE_TYPE comma

Some of the examples did not have a comma after specifying the FILE_TYPE parameter. This was inconsistent within the page and incorrect syntax - a comma is required following the FILE_TYPE parameter for the credential to be correct parsed.
Unsure if D should read FILE_FORMAT = 'myFileFormat', to ensure it is correctly understood that it must be quoted.

@lmoloney : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Idagio Music streaming service

It'd be neat to have support for Idagio's music streaming.
https://www.idagio.com/

Please, fill free to provide your Pull Request for support of this service or expect someone else to see this issue and do the job. Maintainers do not add services by request.


Allow removing font size attribute from text

New changes have been great, but the problem is once you set a font size, you cannot unset the font size to no font size attribute in the delta.
With this change, if a font size of 0 is used, then the font size attribute is remove from the text.  This would be a totally optional usage, as the default list of font sizes does not include an option of 0, but a user can add it to a custom font size list if they wish

No, use clear button

No, use clear button

But that would remove everything, this would work similar to other buttons in the toolbar.
For example, if you have some text with bold and italic and then click the 'B' it would only remove the bold and leave the italic.
If you use had font size and bold and italic, you would have to reset the bold and italic afterwards.
This would follow the logic of the other buttons in the tool bar that can be 'on' or 'off'.  Maybe if 0 is a bad idea, you could use a different variable?

For the least please format your code

For the least please format your code

Sorry I edit everything in GitHub and hard, I will try to see if I can make it better :)

Ok I just updated it, I think the code is better?

I have to say, you should improve coding
https://github.com/singerdmx/flutter-quill/commit/b401cc5518de56ab9f4168b5f1f91d33de41aa0c

@mark8044 Also you probably can add similar feature for font family as below

Sample JSON:
  {
    "attributes": {
      "font": "sans-serif"
    },
    "insert": "Sans Serif"
  },
  {
    "insert": " "
  },
  {
    "attributes": {
      "font": "serif"
    },
    "insert": "Serif"
  },
  {
    "insert": " "
  },
  {
    "attributes": {
      "font": "monospace"
    },
    "insert": "Monospace"
  },

@mark8044 Also you probably can add similar feature for font family as below

Yes I would love this as well.  I can start on this in a few days.
I originally had the font size drop down as a generic drop down so it could be re-used for this, but now its refactored it to be specific for font size.  Should I make a new drop down specific to font size, or refactor the font size to be a generic drop down that can be re-used for font family and font size?

No need for generic use

@mark8044  Did you start anything? If not I am going to implement this


RFR edit_inline, edit_thumbnail and fixEmbed

does this fix #60 ?

It removes the need to directly query the database for a specific role when a reaction is added/removed (i.e. the RFRMessageEmojiRoles table), but it doesn't remove the need for that table to exist entirely when a person reacts and they aren't allowed to since a query to that table is needed to fix that situation, nor does it remove the need for that table to exist as part of fixEmbed.

One test is broke because of dpytest, i'll sort it dw

Don't worry @JayDwee, I can sort that out now

1 error in rfr tests fixed with https://github.com/CraftSpider/dpytest/pull/42
in the meantime @Unknownlocal can fix what Viraj has specified in his review


Feat: Add Meta Titles for All Pages

This PR:
Fixes: #565
Adding meta title to the following pages:

Homepage
Auth pages
Invitation page

@0xJamin We have added prettier format to standardise formatting across the source files. Could you please merge main and then run npm run format.

@deepakprabhakara Thanks for the info.
I'm on it.

@deepakprabhakara  files formatted


docs(yard): show the default value of enum type props & skip it in code example

This is great!
We should toggle (hide) imports as well when the default value is used. For example, the default button example does this:
import { Button, KIND, SIZE, SHAPE } from "baseui/button";

but those enums are not used.
Also, it would be great to add some code-generator unit test for this.


convert to SSG

Issue Name Here
Description
Converts site to use 11ty.
Please include a summary of the change and which issue is fixed.

#31

routing based through html template files + build process

#29

decomposes web page html into partials used by templates

Type of change

[ ] Bug fix
[x] New feature
[ ] Documentation
[x] Infrastucture
[ ] Other (please clarify): _____

Change log

Install, configure 11ty
Decomposes existing html into re-usable partials
Refactors styles, scripts to be compatible with build process
Converts existing index file into template
Creates index page
Stubs out pages:

about
code of conduct
contact
events
media
404
contribute (hidden in /drafts folder)
resources (hidden in /drafts folder)
team (hidden in /drafts folder)

Adds placeholder directory for potential blog/announcements content
pulls over existing favicons, assets, manifest

Testing

Deployed working branch to Netlify as a staging environment

Checklist

[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generate no new console warnings.
[x] I manually tested to prove my fix is effective or that my feature works.
[x] I have assigned this PR to an owner.

@marcusolsson

I think this is great as a base to build upon. First time hearing about Liquid though. What's the benefits over something like Nunjucks and Handlebars?

I don't think any one of the templating languages have much of an advantage over the other. This is one of those cases where the same type of tech was developed by different groups to satisfy their needs rather than coalescing around 1 tool.
My own use of Liquid is simply based on familiarity - it was the default templating for Jekyll when I started web dev and I have not had a reason to reach for anything else in JS. If you prefer another templating language, we can definitely go that way.

It was the only one I hadn't heard of before so just curious. I don't mind it, especially if you're already familiar with it. As you say, they're all pretty similar. Just curious :)


added powerpoint

This powerpoint was in the old archive.
Updated it to current archive and added it!
We want to use it at the workshop next week!

I wouldn't store binary blobs in our github repository. Can't we use any other service for this purpose (Google docs, Dropbox, One Drive)?

@pjljvandelaar Why do we want this file to be part of our repository?
If it is enough to make it available for people who want to learn about CustomersOrders example, then we can use GitHub to store our files and put a link in our existing MD files: CustomersOrders.pptx
I can do the same thing for powerpoint files of LuckyPeople and StimulusResponse examples, too.

I know git is not meant for binaries ...
I would like one place to have all documentation available.
Otherwise we increase the risk of making changes only to a part of the documentation.

@keremispirli Where does GitHub store binaries?

When you attach a file to any comment/issue, it's uploaded to GitHub and saved under your project. Then you can use the link to it anywhere you want. This can be used to host the file for users instead of including it in our repository.
In order to actually manage our documentation, though, we need a better solution. We already have DMSF, but it is internal which means every time we update a file, we will have to re-upload to GitHub and update links in the repository.
Ideal solution would be a place where we can store files and have public links to them. It is normally possible in OneDrive but TNO doesn't allow it. Maybe we can ask for a special OneDrive account for TorXakis that it's allowed? Or maybe there's a way to enable public links to files in DMSF?

@keremispirli let's just merge this branch and make an issue about what to do with all pptx files / binary files


Taiga integration test

this line doesn't return desired output:
status1 = tests.Wait(webip, "22")
as it's trying to dial webip at port 22 it keeps failing and retrying for 3 minutes and it fails but it returns true anyway.
trying to access webip outputs Bad Gateway so pinging webip returns that it's reachable however it's not running properly
and pinging webip grid3taiga.gent02.test.grid.tf without deploying works fine so that check doesn't affect the test:
out1, _ := exec.Command("ping", webip, "-c 5", "-i 3", "-w 10").Output()
assert.NotContains(t, string(out1), "Destination Host Unreachable")

changed the function name and it now uses tcp instead of ping to be more efficient


Fix Authenticate docs to include more kotlin

The Content Tab feature means that only java had any code samples for responseCount.

@swankjesse Bit worried about the effect of the whitespace changes.  Is there an easy way to test this in CI?

To verify https://github.com/square/okhttp/issues/7041#issuecomment-1027036179