[Feature Request] Support for Function Calling

Hello,
First of all, I want to extend my gratitude for creating such useful software.
To further enhance this software, I am proposing the addition of Function Calling support.
I have already pushed my implementation of this feature at https://github.com/iwamot/ChatGPT-in-Slack/commits/function-calling. In this implementation, the OPENAI_FUNCTION_CALL_MODULE_NAME environment variable is used to specify the name of the module that contains the functions. If not specified, the application will retain its current behavior.
May I proceed to create a PR for this proposed addition? I look forward to your feedback. Thank you very much.

Hi @iwamot, thank you for the excellent proposal! The addition of function calling is certainly something I've been looking forward to. I haven't gone through your local branch in detail yet, but at first glance, there seem to be several changes that might not be essential. When you submit a pull request, keeping the changes minimal would be very helpful. Thanks in advance!

Hi @seratch, thank you for your prompt and positive response. I am glad to hear that you were looking forward to the addition of Function Calling.
I agree wholeheartedly with your point that changes should be kept to a minimum, and I will try to reduce the diffs. Of course, if you or any other contributors could give me ideas for improvements, that would be very appreciated. It would be my pleasure to collaborate with you all on adding this feature.
Again, thank you for considering this proposal!

Just a quick update: the diff has now been reduced from '306 additions and 14 deletions' to '167 additions and 16 deletions'.

I think I have made reasonable improvements and will proceed to create a pull request. Thank you!
https://github.com/iwamot/ChatGPT-in-Slack/commits/function-calling-rebased

Merged at https://github.com/seratch/ChatGPT-in-Slack/pull/52. Thank you so much!


ARM images

Since almost all fission core images support ARM it would be nice to have the same for the connectors. By having a quick look at some of the connectors it seems there aren't any major roadblocks.

Image available now https://github.com/orgs/fission/packages?repo_name=keda-connectors


more fluid listing (like atmospherejs.com) with infinite scroll

:construction_worker:  I'll start work on this tomorrow


creating-a-composite-action.md template error

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/actions/creating-actions/creating-a-composite-action
What part(s) of the article would you like to see updated?
The section Testing out your action in a workflow has a template that throws an error when one due ambiguous/incomplete instructions about replacing the word 'action' with the user's username.
Additional information
No response

200000 в рублях или гривнах все аллах пн, 5 авг. 2024 г., 10:03 welcome[bot] @.>:
…
Thanks for opening this issue. A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines https://docs.github.com/en/contributing. — Reply to this email directly, view it on GitHub <#34196 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZNHMBPMHIO3OCJFICMBAO3ZP4PSTAVCNFSM6AAAAABL7VMR52VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRYGMZDCNZZHE . You are receiving this because you are subscribed to this thread.Message ID: @.>

Thanks for opening this issue. A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.

@misafu Thank you for opening an issue and linking it to your PR! ✨

This got resolved in #34197

@janbrasna Thank you for catching this! ✨


Unable to receive udp multicast messages in android

Hi,
I'm not able to receive any multicast messages in Android.
When running the UWP project everything is ok, but I can't get it to work when running the Android project.
This is my receiver code :
private async Task DiscoverServer()
{
        // typical instantiation
        if (receiver == null)
        {
            var port = 9121;
            var address = "<IP_ADDRESS>"; // must be a valid multicast address

            receiver = new UdpSocketMulticastClient();
            receiver.TTL = 5;

            receiver.MessageReceived += Receiver_MessageReceived;

            // join the multicast address:port
            await receiver.JoinMulticastGroupAsync(address, port);
        }

        IPAddresses.Clear();
    }

private void Receiver_MessageReceived(object sender, Sockets.Plugin.Abstractions.UdpSocketMessageReceivedEventArgs args)
    {
        var from = String.Format("{0}:{1}", args.RemoteAddress, args.RemotePort);
        var data = Encoding.UTF8.GetString(args.ByteData, 0, args.ByteData.Length);

        if (data == "SERVER")
        {
            string serverAddress = args.RemoteAddress;
            IPAddresses.Add(serverAddress);
        }
    }

These are my manifest permissions :
<uses-permission android:name="android.permission.INTERNET" />
<uses-permission android:name="android.permission.ACCESS_WIFI_STATE"/>
<uses-permission android:name="android.permission.CHANGE_WIFI_MULTICAST_STATE"/>
<uses-permission android:name="android.permission.ACCESS_NETWORK_STATE"/>

Why is this not working ?

Howdy, can you list out a few more things to help diagnose :shipit:

platform and network of the device sending the udp packets
platform, physical vs emulator and network of the UWP device
physical vs emulator and network of the android device
what does the code that calls DiscoverServer look like

Hi,
The sender is a server UWP application running on Windows IOT (Raspberry pi
3). It used your plugin to send a multicast message for clients to pickup.
The idea is that clients can find all servers in a network.
The client is either my development computer when I'm testing the UWP
client, or an Android cellphone when I'm testing the Android client.
The UWP client works, but the Android client is not receiving the udp
message. I do not use emulators. Both client and server are physical
devices on the same network.
When the user clickes a button the client app waits for server messages for
5 seconds and lists all IP addresses for incoming messages.
Below is the code that calls the DiscoverServer method
private void BtnDiscover_Clicked(object sender, EventArgs args)
{
RunDiscovery();
        Task doList = Task.Factory.StartNew(async () =>
        {
            await Task.Delay(5000);
            Device.BeginInvokeOnMainThread(() =>
            {
                lvFoundServer.ItemsSource = IPAddresses;
            });
        });
    }

    private async void RunDiscovery()
    {
        try
        {
            await DiscoverServer();
        }
        catch (AggregateException ae)
        {
            foreach (Exception e in ae.InnerExceptions)
            {
                if (e is TaskCanceledException)
                    Debug.WriteLine("Unable to compute mean: {0}",

((TaskCanceledException)e).Message);
else
Debug.WriteLine("Exception: " + e.GetType().Name);
}
}
}
Hope you can help.
Met vriendelijke groeten,
Kind regards,
Benjamien Toté
BETOWARE bvba
2016-12-12 23:01 GMT+01:00 Ryan Davis<EMAIL_ADDRESS>
Howdy, can you list out a few more things to help diagnose [image:
:shipit:]

platform and network of the device sending the udp packets
platform, physical vs emulator and network of the UWP device
physical vs emulator and network of the android device
what does the code that calls DiscoverServer look like

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rdavisau/sockets-for-pcl/issues/121#issuecomment-266566805,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AH0fZhorsVfmpgOUaLIij9qdAMq068vKks5rHcQ9gaJpZM4LLCQh
.

I'm having the same issue. The sockets-for-pcl works fine with Android 6.  My co has me testing our UDP discovery service on a Nexus 9 physical device running android 7.1, and the app eventually crashes.
Something in my output I've never seen before was this error:
"JNI detected error in application use of deleted local reference 0x1"
Ryan, have you tried your PCL sockets code running Android 7.1 on a physical device?
I'd like to help if I can. Perhaps I can grab latest and run the code locally to see where in your codebase the issue is.
We really need this fixed asap.
thanks

Try acquiring a multicast lock:
WifiManager wifi = (WifiManager) context.getApplicationContext().getSystemService(Context.WIFI_SERVICE);
WifiManager.MulticastLock multicastLock = wifi.createMulticastLock("multicastLock");
multicastLock.setReferenceCounted(true);

...

multicastLock.release();

Credits to https://www.b4x.com/android/forum/threads/enable-multicast.43321/#post-263242


Request: Search Dropdown onAddItem should optionally prevent onChange call

This is highly debatable and perhaps it should be customisable. My use case is that I have a list of objects and a user needs to search for, but if it's not in the list he'll have to create his own from the name he searched for and add additional required fields.
That makes the allowAdditions prop perfect. However what a user would enter in the search field might not be representative of an entire object, i.e. a "material object" might need additional fields such as cost.
So in my case I would show more fields (or rather remove the Dropdown and show a different set of inputs) when the user tries to add something. That functionality can easily be trigged from onAddItem.
However the problem I am facing is that onChange is called immediately after onAddItem, which causes awkwardness (for example onChange might be setting a materialId but when it's called after onAddItem its value would be the string the user entered.)
Could we make this behaviour customisable? callOnChangeAfterAdd is err... awkward but something along those lines you know?
I realise this issue is very verbose and contains no code examples, so apologies for that. If you give the go-ahead for this I could probably contribute the PR myself. :)

Yep, I'm actually in complete agreement with the goal here.  Though, rather than add a prop, I think we should just not callback at all because I don't think we should be automatically making this new item the active item at all.  I just covered this 4days ago here https://github.com/Semantic-Org/Semantic-UI-React/issues/1132#issuecomment-272928232.
The issue is that when an item is added, we call onAddItem, then we also update the state of the Dropdown setting the new item as the currently active item.  This immediately fires an onChange because the value did change from the previous value to the new item value.  Worse, the new item text is used for the text and the value which is not ideal for several reasons.
Instead, when an item is added, I propose we only call onAddItem and that is it.  Then users can implement whatever workflow they want after that.  If the current value should be updated (current behavior), then pass the new item directly back to the Dropdown value.  If something else should be done, then do that instead.
Any thoughts on this?

What you described is exactly the behavior I would want as well. It is a breaking change though of course.

Great, this one is ready to go if you'd like to pick it up.  Feel free to open the PR as early as possible, no need to wait for it to be ready to merge before opening.

I think that #2113 fully resolves problem, onAddItem is now called after onChange, so you can control new value without problem.

That does not solve the issue to my understanding because the onChange event doesn't know that it was triggered from an allowAddition action.

@nathanhannig if you feel this is still an issue, please open a new report. This is an extremely old ticket.


Any Requirements Tools to open Android Project.

Hi Nintendaii,
It was successfully working on my android Phone. Then when I was trying to edit the code using android studio it was not working when I try to build a bundle (.aar) file. I got an error I don't know yet if you use Android Studio or another version. Please advise what tools you used or a requirements
Thank you.

Hi, can you please say what exact error appears when you try to build your .aar file? I used Android Studio 4.1.2 for my builds


Update Reinstalling-and-Updating-Packages.md

added missing -ProjectName in command

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/Consume-Packages/Reinstalling-and-Updating-Packages.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Thanks for catching that!


Fix handling of optional IDispatch, IUnknown method args

Fixes https://github.com/Microsoft/visualfsharp/issues/907
Aligns F# compiler's treatment of these optional arguments with what C# and VB do.
For omitted obj optional args:

If it has MarshalAs(IUnknown), MarshalAs(IDispatch), or MarshalAs(Interface), emit a null default value
Else if it has IUnknownConstantAttribute emit UnknownWrapper(null)
Else if it has IDispatchConstantAttribute emit DispatchWrapper(null)
Else emit Type.Missing

See relevant bits of the C# compiler and VB compiler.

@latkin, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

@latkin, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

OK, looks good!

@dotnet-bot test this please

Thanks for taking care of this Lincoln, we miss you, and hope you are having a great time in your new Job.


Fix for passing aerospike timeout parameters

Codecov Report

Merging #87 into develop will increase coverage by 0.43%.
The diff coverage is 90.47%.

@@              Coverage Diff              @@
##             develop      #87      +/-   ##
=============================================
+ Coverage      87.06%   87.50%   +0.43%
  Complexity       230      230
=============================================
  Files             24       24
  Lines            804      816      +12
  Branches          70       70
=============================================
+ Hits             700      714      +14
+ Misses            71       69       -2
  Partials          33       33

Impacted Files
Coverage Δ
Complexity Δ

...nusgraph/aerospike/operations/BasicOperations.java
93.61% <66.66%> (ø)
22.00 <1.00> (ø)

.../janusgraph/aerospike/AerospikePolicyProvider.java
83.33% <85.71%> (+2.08%)
4.00 <4.00> (ø)

...m/playtika/janusgraph/aerospike/ConfigOptions.java
95.83% <100.00%> (+0.83%)
1.00 <0.00> (ø)

...usgraph/aerospike/TestAerospikePolicyProvider.java
100.00% <100.00%> (ø)
2.00 <1.00> (ø)

...raph/aerospike/operations/AerospikeOperations.java
91.17% <100.00%> (+1.52%)
14.00 <0.00> (ø)

...raph/aerospike/operations/BasicLockOperations.java
84.39% <0.00%> (-1.42%)
37.00% <0.00%> (-1.00%)

...aph/aerospike/operations/AerospikeKeyIterator.java
72.72% <0.00%> (+7.27%)
13.00% <0.00%> (+1.00%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9a8d4a1...5714ea2. Read the comment docs.


Added labels and moved alerts.

@Dakuan I've added the badges, however they are called labels in the bootstrap docs so I replicated that for consistency.

lovely :shipit:


Option to disable symbol demangling

Requested by a few people, including @dascandy

Also requested by @MalwareMinigun on twitter

Also now requested by @zygoloid ... we should really implement and soon...

Hoping to push today after testing other than C++

Hurray! Thanks @RabsRincon !

Site updating with demangling option


DateFormat.DateTimeWithTimezone.parse: further specify return value

OpenUI5 version: 1.111.0
Steps to reproduce the problem:
Currently, the return type of sap.ui.core.format.DateFormat.DateTimeWithTimezone.parse is simply an Array
// parseResults: any[]
const parseResults = DateFormat.getDateTimeWithTimezoneInstance({showTimezone: false}).parse(sCurrentValue, sTimezone);

What is the expected result?
A developer would benefit from a consistent and more strongly typed return value.

Hi,
Thank you for your pull request. I've created an internal record<PHONE_NUMBER>. The status of the issue will be updated here in GitHub.

Thank you for your request, I've created a change improving the documented return value of the method as followed:
@return {Array<Date|undefined, string|undefined>} the parsed values
This format was chosen since, the values in the array may be undefined depending on the format options you chose showDate: false, showTime: false or showTimezone: false.
For further information on the API feel free to checkout or [public API reference|https://sapui5.hana.ondemand.com/#/api/sap.ui.core.format.DateFormat.DateTimeWithTimezone%23methods/sap.ui.core.format.DateFormat.DateTimeWithTimezone.parse].

Hello BenReim,
after a deeper investigation I've found out my previously suggest solution is not applicable. Nevertheless, I was able to implement and submit the requested change in form of an overlay.
The requested documentation will be available in version 1.113.
Thanks again for your request and best regards,
Viktor


When I uninstall this extension the comments are still hidden.

I have no idea how to fix this. already restarted vscode.

This is because there is no uninstall trigger on VSCode. Make sure to remove the settings from your .vscode/settings.json file.

Unfortunately, VSCode does not provide an uninstall hook/event. There is a way to add a script, but this doesn't seem to run in the context of your project/VSCode workspace. I have added a new section on the README to explain the steps to take when resetting it to the original state: https://github.com/estruyf/vscode-hide-comments#removing-the-extension


Malformed http headers

Was running remi behind a proxy and saw that it was failing to parse the headers. Curling the webserver locally indeed shows malformed header with repeated stuff:
HTTP/1.0 200 OK
Server: BaseHTTP/0.6 Python/3.7.0
Date: Fri, 31 Aug 2018 22:54:48 GMT
Set-Cookie: remi_session=0
HTTP/1.0 200 OK
Server: BaseHTTP/0.6 Python/3.7.0
Date: Fri, 31 Aug 2018 22:54:48 GMT
Content-type: text/html

Notice the repetition of the first 3 lines.

The problem is send_response is called twice:
https://github.com/dddomodossola/remi/blob/2576f8737c7c0dfaa44eddd7019dab9cf38b5346/remi/server.py#L341
and
https://github.com/dddomodossola/remi/blob/2576f8737c7c0dfaa44eddd7019dab9cf38b5346/remi/server.py#L836

Thank you for your work. We're looking to possibly adding this to repl.it because users have been asking for gui and given we run in the browser this seems like a good fit. Look forward to the fix.

@amasad I made a fix, removing the duplicate http send_response request. Please let me know if this solves the "malformed http header" problem.
It would be cool to have remi on repl.it, different people asked me about the possibility to use remi on web devel solutions like heroku on so on. Let me know it this will be done, I will inform my contacts. ;-)

awesome! can we get a release so I can test it?
On Mon, Sep 3, 2018 at 12:35 AM Davide Rosa<EMAIL_ADDRESS>wrote:

@amasad https://github.com/amasad I made a fix, removing the duplicate
http send_response request. Please let me know if this solves the
"malformed http header" problem.
It would be cool to have remi on repl.it, different people asked me about
the possibility to use remi on web devel solutions like heroku on so on.
Let me know it this will be done, I will inform my contacts. ;-)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dddomodossola/remi/issues/235#issuecomment-418027173,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAj2_laqXzYtVTnqnqL0zzbU8ULago6Uks5uXNumgaJpZM4WV86E
.

I got something working but ran into #237. Here is a hosted remi app (without the https): http://remi-repl-demo--amasad.repl.co/
Source: https://repl.it/@amasad/remi-repl-demo

I will make a release soon, thank you for sharing the hosted app.

@amasad  Released version 1.2.2


Possibility to use on headless server/router?

Hi! Thanks for this amazing app. Is it possible to use the proxy on a headless Linux box or router to connect to it afterwards via IP and PORT?

Hi! Thanks!
Yes, it's possible. See this wiki page for example of autostart on Linux. Also you'll need to add -listen <IP_ADDRESS>:8080 option to OPTIONS in order to enable connectivity from other hosts.

It's saying this flag provided but not defined: -listen

I see thanks, I've tried this option and it doesn't seem to work. As well added all of the containers and in logs, I see no connectivity. The docker host is a clean Debian 10 host x86.  Any ideas on how I can troubleshoot this?

It seems to work on an arm box, so will check the firewall settings.

When running directly with the bind command all of them work fine, but with the docker container, I don't get connectivity.  =(

@tornadox what docker command do you use?

sudo docker run -d \

--security-opt no-new-privileges \
-p <IP_ADDRESS>:8080:8080 \
--restart unless-stopped \
--name hola-proxy \
yarmak/hola-proxy -country de

In this case use -p 8080:8080 instead of -p <IP_ADDRESS>:8080:8080

thank you! It worked!

I know this one is not related to windscribe proxy, but once started I get this
MAIN    : 2021/08/08 16:52:22 main.go:192: CRITICAL Cold init failed: bad http status: 400 Bad Request, headers: http.Header{"Access-Control-Allow-Headers":[]string{"Origin, X-Requested-With, Content-Type, Accept"}, "Access-Control-Allow-Origin":[]string{"*"}, "Cache-Control":[]string{"no-cache, no-store, max-age=0, must-revalidate"}, "Cf-Cache-Status":[]string{"DYNAMIC"}, "Cf-Railgun":[]string{"9a8ebb9c48 1.54 0.010990 0032 57da"}, "Cf-Ray":[]string{"67ba413878ea3d71-KBP"}, "Connection":[]string{"keep-alive"}, "Content-Type":[]string{"text/html; charset=UTF-8"}, "Date":[]string{"Sun, 08 Aug 2021 16:52:22 GMT"}, "Expect-Ct":[]string{"max-age=604800, report-uri=\"https://report-uri.cloudflare.com/cdn-cgi/beacon/expect-ct\""}, "Server":[]string{"cloudflare"}, "Strict-Transport-Security":[]string{"max-age=16000000; includeSubDomains; preload;"}}

MAIN    : 2021/08/08 16:52:25 main.go:122: INFO     windscribe-proxy client version v1.0.2 is starting...

MAIN    : 2021/08/08 16:52:25 main.go:189: WARNING  Failed to load client state: open /state/wndstate.json: no such file or directory. Performing cold init...


请问一下 有离线下载的接口吗? 支持在小程序里使用吗？

--------------^ Click "Preview" tab for a nicer view!
We use GitHub as an issue tracker and do not provide support here.
For usage and support questions, please check out resources below:

Read Leaflet documentation: https://leafletjs.com
For support questions like "How can I do X with Leaflet?" please search or ask questions on StackOverflow or StackExchange using Leaflet tag:

https://stackoverflow.com/questions/tagged/leaflet
https://gis.stackexchange.com/questions/tagged/leaflet

Thanks!

Am I right in understanding you want to integrate leafletjs into a native application? There's no libraries to render leafletjs to, for example, a C# application. What language are you using to program your application? If you're using C#, have a look at http://www.osmsharp.com/


Can't build TerraAngel

Whenever I try to run fast_start.bat it opens for a few seconds then closes and doesn't create a new folder

do you have the dotnet 6.0 sdk installed?

yes i downloaded that before trying to install it

did you restart your computer after installing it?

yes

hmm, open up command prompt and go to the directory where the fast start is, and run it from there

ok

you also need to add --branch dev-<IP_ADDRESS> when cloning if you're using Terraria <IP_ADDRESS>

you also need to add --branch dev-<IP_ADDRESS> when cloning if you're using Terraria <IP_ADDRESS>

that might be the problem

maybe

so just git clone https://github.com/CEO-Chair/TerraAngel.git --recursive --branch dev-<IP_ADDRESS>

also make sure you clone in a place like your desktop, dont use C:\Windows\Sytem32

also make sure you clone in a place like your desktop, dont use C:\Windows\Sytem32

how do i change where it clones

eg, cd C:\Users\you\Desktop

and then clone

ok

that worked

thanks

yeah


Add links to source code spans in forc doc

Clicking source takes you to the span of the corresponding source code

This is basically similar to a go to def lsp request.


Update Notifications to 2.0

Signed-off-by: Mohammad Qureshi<EMAIL_ADDRESS>Description

Update Notifications to use OpenSearch/OpenSearch Dashboards 2.0 and update Gradle to 7.3
Update gradle dependency repositories used
Update GitHub Actions to not publish locally

Issues Resolved
#365
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

The Test and Build Notifications GitHub workflow is failing because 2.0 snapshot for common-utils has not been uploaded to maven yet.
Building Notifications locally after publishing the main branch of common-utils to mavenLocal succeeded:
qreshi@186590ce20fd notifications % ./gradlew build
=======================================
OpenSearch Build Hamster says Hello!
  Gradle Version        : 7.3
  OS Info               : Mac OS X 10.16 (x86_64)
  JDK Version           : 14 (OpenJDK)
  JAVA_HOME             : /Library/Java/JavaVirtualMachines/jdk-14.0.1.jdk/Contents/Home
  Random Testing Seed   : 273F468DA7F6E69B
  In FIPS 140 mode      : false
=======================================

...

BUILD SUCCESSFUL in 6m 41s


[WIP] Make OrchestrationTemplate self referential

The idea here is to support the concept of nested (linked) templates, i.e. templates with links to other templates. The "main" template would serve as the parent, with all other templates serving as children, potentially having children of their own.
Each child template would store a URL reference to its location that the parent could obtain, and since the children are using the same table, could store their content as well. How MIQ uses that content or URL is up to the app, but it would be readily available in a logical manner.
The ultimate goal here is to improve support for provisioning from nested templates, initially for AWS, but also for other providers. Currently you cannot access a nested template within ManageIQ, they are only URL's within the parent, which makes them difficult to manage.
With this in place you might have a template like this:
...
"name": "MyTemplate"
"resources": [
  {
      "apiVersion": "2015-01-01",
      "name": "linkedTemplate",
      "type": "Microsoft.Resources/deployments",
      "properties": {
        "mode": "incremental",
        "templateLink": {
          "uri": "https://www.contoso.com/AzureTemplates/newStorageAccount.json",
          "contentVersion": "<IP_ADDRESS>"
        },
        "parameters": {
          "StorageAccountName":{"value": "[parameters('StorageAccountName')]"}
        }
      }
  }
]

Assuming these had been added as two separate records, with the parent_id set for the linked template, you could do:
parent = OrchestrationTemplate.find_by_name("MyTemplate")
child = parent.children.first
child.url # "https://www.contoso.com/AzureTemplates/newStorageAccount.json"
child.content # Presumably the content of that URL
child.parent # reference back to the parent.

@miq-bot add_label wip

@Fryguy @carbonin They're applied at the database level. You can comment on tables, too. Yes, this is only meant for developers, and I do look at the columns in psql/pgadmin sometimes.
The comments can be used for doc generation to help developers. Consider our developer documentation for VM's:
http://manageiq.org/docs/guides/architecture/providers_database_architecture/vm_or_template
Most of the descriptions there are blank. I would very much like to know what some of those fields are. By attaching comments, you can help make the database, and thus the application, somewhat self documenting. This should help not only the team, but potentially outside contributors.
P.S. When I have any authority over a Rails project, these are mandatory. :)

@djberg96 The concept makes sense but I am wondering if we should be using ancestry to link to templates together.  @kbrock thoughts?

@gmcculloug As in the ancestry gem? I wasn't aware we were using it, though now that I look at it, that should work as well. Seems a bit heavy for what I'm doing here perhaps, but yeah, that looks like it will work.

We should have meetings to design this properly. Orchestration Template is about reusability. A template can be embedded in multiple parents or to be used standalone. There should not be a parent-child relationship.
Same is for the URL. We identify the uniqueness of a template by its content but the same template can physically exist in multiple places.

@bzwei First, you're on vacation, go away. ;)
I've updated the PR to use the ancestry gem, though it still keeps the hierarchical approach.
https://github.com/stefankroes/ancestry
We can talk about this some more this week.

@djberg96 per our previous discussion you may want to drop this PR for now.


🐛 Bug Report — Runtime APIs — AbortSignal.timeout throws un-catchable async DOMException

async function fail() {
  try {
    response = await fetch("https://example.com", {
      method: "POST",
      body: new FormData(),
      signal: AbortSignal.timeout(1),
    });
  } catch {
    console.log("caught error");
  }
}

expectation: when the promise is awaited, the fetch is aborted, and any error thrown is suppressed via the catch {} block
❌ reality: ✘ [ERROR] Uncaught (async) DOMException
I assume this is a bug in workerd
NOTE: this bug is ONLY reproducible when running wrangler locally. Running via --remote does NOT exhibit this behavior.
As a workaround, this works as expected:
async function failWorkaround() {
  const controller = new AbortController();
  const timeout = setTimeout(() => controller.abort(), 1);
  try {
    response = await fetch("https://example.com", {
      method: "POST",
      body: new FormData(),
      signal: controller.signal,
    });
  } catch {
    console.log("caught error");
  }
  clearTimeout(timeout);
}

✅ uncaught exception is suppressed by catch {}
wrangler 3.5.0

Hi @aroman,
Can you provide a more complete reproduction? The code snippets you provided are not actually syntactically valid, and my best guess at what you meant appears to work correctly. It would help to see a whole worker script that exhibits the problem.

@kentonv yes; apologies for the sloppy snippets above. I have provided a complete, reliably reproducible demonstration of the issue below.
of note — I cannot seem to repro this with GET requests, it seems that I need a POST request to trigger the behavior. the repro example below demonstrates the behavior clearly. note that I can only reproduce this in miniflare, not running via --remote.
// Run with: wrangler dev worker.js

export default {
  async fetch(req) {
    if (req.url.endsWith("/fail")) {
      await fail();
    } else if (req.url.endsWith("/fail-workaround")) {
      await failWorkaround();
    }
    return new Response("check the wrangler dev console");
  },
};

// Will log: ✘ [ERROR] Uncaught (async) DOMException
// Even though we're explicitly catching the error
// "caught error" will be logged
async function fail() {
  try {
    response = await fetch("https://example.com", {
      method: "POST",
      body: new FormData(),
      signal: AbortSignal.timeout(1),
    });
  } catch {
    console.log("caught error");
  }
}

// Will NOT cause the DOMException error to be logged
// "caught error" will be logged
async function failWorkaround() {
  const controller = new AbortController();
  const timeout = setTimeout(() => controller.abort(), 1);
  try {
    response = await fetch("https://example.com", {
      method: "POST",
      body: new FormData(),
      signal: controller.signal,
    });
  } catch {
    console.log("caught error");
  }
  clearTimeout(timeout);
}

Is it possible you're using an old version of Miniflare? Miniflare v2 tried to emulate Workers APIs in Node, whereas v3 actually uses workerd under the hood.
I haven't actually tested the above using raw workerd, but it does work correctly on cloudflareworkers.com and the underlying code is the same between them.

Is it possible you're using an old version of Miniflare? Miniflare v2 tried to emulate Workers APIs in Node, whereas v3 actually uses workerd under the hood.
I haven't actually tested the above using raw workerd, but it does work correctly on cloudflareworkers.com and the underlying code is the same between them.

@kentonv
I am using miniflare v3, which is bundled with the latest version of wrangler, 3.5.1. Per the release notes for that version, it seems it is specifically using miniflare version 3.20230814.1.
As I mentioned in the OP, this behavior is only reproducible running via miniflare, and NOT running on cloudflare's hosted workers environment (as in via --remote).

Thanks for your efforts to repro, @mrbbot! Good to know I'm not going crazy 😅
Is the --verbose flag on by default for wrangler dev? I've never invoked workerd directly, only via wrangler.

OK, I may have misunderstood what I was looking for when I tried this. So the exception isn't actually "un-catchable", it's just that there's a spurious log message?

OK, I may have misunderstood what I was looking for when I tried this. So the exception isn't actually "un-catchable", it's just that there's a spurious log message?

@kentonv yes, that is correct. even though the error is caught (i.e. the catch block runs), the message "✘ [ERROR] Uncaught (async) DOMException" still appears. i am not sure if this will actually bubble up as an actual uncaught promise rejection (as the log suggests), or the log is totally inaccurate, but either way, it's a bug and confusing.

@jasnell Apparently AbortSignal.timeout() generates a bogus unhandled rejection event, whereas using setTimeout() and controller.abort() to implement a timeout does not. Any idea why?

Err actually I'm not sure if it's an unhandled rejection event or just something like an unhandled rejection event, but either way...

Sorry just spotted this. will investigate

Ok, @kentonv the issue is not specific to AbortSignal.timeout() here, you'll see that in the case of calling controller.abort('foo') within the setTimeout() the error is also logged then. The reason is that within the AbortSignal we are using a kj::Canceler to actually cancel the underlying operation on the fetch which means we're creating another propagation path for the error that is being logged. That path operates separately from the try/catch that is in scope here so would be logged separately. I'll investigate further to see how we can suppress it.

Actually, let me take that back... it *is AbortSignal.timeout in terms of the way that it builds the error internally. Specifically, AbortSignal.timeout reports the reason for the disconnect using a kj::Exception converted to a JS exception, which gets converted back into a kj::Exception (this part can be cleaned up a bit). The code is set up to ignore logging for kj::Exception's that are marked as DISCONNECTION errors but in this case that's not reported. It ends up being no different than if AbortController.abort(...) is called with a user-supplied reason. Should be fairly straightforward to clean up

@aroman ... I have a PR that fixes the specific issue when using AbortSignal.timeout(), but I have a question for you.
Here's the set up:
When we have a fetch request that that has a request body payload (e.g. POST, PUT etc) to be transmitted to the server, there are actually two distinct async operations involved: The promise that is pending for the Response (which I'll call P1) and the promise that is pending for the actual transmission of the request body payload (which I'll call P2). The AbortSignal actually cancels both of these by passing along the reason that is given to the AbortSignal. This reason is typically an Error but can be any JavaScript value.
P1, the promise for the Response, is the thing that you are actually awaiting for when you call await fetch(...). That promise is resolved independently of the transmission of the request body payload (P2).  The other promise (P2) is handled internally and is NOT something user code can handle directly. The "unhandled exception" that you are seeing here is from the rejection of P2.
We currently have code in place that when you abort a fetch using an AbortController, and you do not pass in a reason explicitly (calling abortController.abort() with no arguments), the rejection of P2 is handled silently within the runtime. That is, you will NOT see the unhandled exception notice for P2. When you call abortController.abort(reason), where reason is any JavaScript value, you WILL see the unhandled exception notice for P2. Using AbortSignal.timeout(n) is currently equivalent to abortController.abort(reason). The PR that I have open in #1177 causes the AbortSignal.timeout(n) to also be handled silently, addressing the specific issue that you have reported here.
Here comes the question for you: Even after #1177 lands, calling abortController.abort(reason) will cause the unhandled exception error to be reported. This is a slightly different case than what you reported and it would be helpful to understand what you, as a user, would expect in this case.
Consider the following example. We are assuming that the server side of this request is duplexing their response... that is, they are sending the response headers before the client has completely finished transmitting the request payload. This would mean that the await fetch promise resolves before the request payload has finished. The abort in this case will end up stopping the transmission of the payload abruptly after the fetch promise has already resolved.
  const enc = new TextEncoder();
  const rs = new ReadableStream({
    type: 'bytes',
    async pull(c) {
      await scheduler.wait(100);
      // stream never ends...
      c.enqueue(enc.encode('hello'));
    }
  });
  const ac = new AbortController();
  try {
    const response = await fetch("https://example.com/", {
      method: "POST",
      body: rs,
      signal: ac.signal,
    });
    // we've received the response headers but the body payload is still running... let's
    // use the AbortController to interrupt
   ac.abort(new Error('canceled'));
  } catch {
    // never runs because the fetch promise itself resolved just fine....
    console.log("caught error");
  }

In this scenario, because we passed an explicit reason to ac.abort(...), the runtime will report in the logs that an unhandled exception occurred. This exception is not catchable by user code. If we do not log the exception, then you might not have any way of knowing why the request payload body was never transmitted. You might never actually know whether it was or was not actually transmitted at all.
PR #1177 does fix the problem you originally raised but it does not address this additional concern about ac.abort(new Error('canceled') in this case. The question to you is, would the unhandled exception log in such a case also be problematic for you? Would you prefer that we not emit any unhandled exception messages in this case? What would be your expectation here?


Un-join, remove players from waiting room

If someone accidentally double-joins, you have to create a new room at the moment

What's left to do:

There is no "You're not part of this game" page
Players who aren't part of a game can still start the game for the others
There should be a confirmation dialog asking if you really want to remove someone
Self-remove is not implemented (where should you be redirected if you leave?)


Fix #39

On first glance of the code from my phone, is the intention here that this will generate a JsonAdapter if you have one class annotated with @MoshiJsonAdapter?

Basiclly yes , and you can see the difference in this commit. What I do is set  "com.ryanharter.auto.value.moshi" dynamic.
For example,in your method createJsonAdapterFactory():
TypeSpec.Builder factory = TypeSpec.classBuilder(
ClassName.get("com.ryanharter.auto.value.moshi", "AutoValueMoshiAdapterFactory"));

and I just change it like this:
TypeSpec.Builder factory = TypeSpec.classBuilder(
ClassName.get(moshiAdapterFactory.packageName(), moshiAdapterFactory.simpleName()));

What about using annotation processor options for this?

I think what robxyy's issue is about the conflict between one module which refer to it and two or more modules which refer to it.

I'm pretty sure I understand the problem, I'm just not sure this is the best solution.
With this design, the user is forced to add an extra annotation to (an arbitrary) one of their AutoValue annotated classes.  In my case I have tens of AutoValue classes, so which one do I put it on.
You could say all, but that get's messy.  If I need to update a custom JsonAdapter package, I have to change it in 30 places, and hopefully not screw one of them up.
If I only put it on one of them, since that's all that's really required, when I want to update my custom JsonAdapter package, where do I change it?  Hopefully I remember which of the 30 I put the annotation on, otherwise I have to go through each one looking for it.
And what happens if there's more than one annotation?  Currently, it looks like it will just choose the first it finds, which could change and leaves you guessing.
All of this leads to non-deterministic behavior which I'm not a fan of.
That being said, I understand that this is a problem, AutoValue classes across multiple modules will each get their own com.ryanharter.auto.value.moshi.AutoValueMoshiAdapterFactory, which will override other module's adapter factories.  This is something I wrestled with a bit when creating it, but just settled on having a static classname.  We'll need to come up with some solution, but I don't know that this is it.
If you think I've overlooked or misunderstood anything, please let me know.

Specifying an annotation processor option in the build
apt {
    arguments {
            moshiAdapterName 'com.example.AdapterFactory'
    }
}

seems simple enough for this.
Otherwise maybe take inspiration from how AutoValue itself works!
@MoshiAdapterFactory
public abstract class Factory implements JsonAdapter.Factory {
  public static Factory create() {
    return new AutoValueMoshi_Factory();
  }
}

I think @JakeWharton's suggested solution would be really useful here. @rharter what do you think?


[stable/prometheus-pushgateway] Prometheus pushgateway is not scraped by prometheus by default?

Versions:
$ helm version
Client: &version.Version{SemVer:"v2.15.1", GitCommit:"cf1de4f8ba70eded310918a8af3a96bfe8e7683b", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.15.1", GitCommit:"cf1de4f8ba70eded310918a8af3a96bfe8e7683b", GitTreeState:"clean"}

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"16", GitVersion:"v1.16.2", GitCommit:"c97fe5036ef3df2967d086711e6c0c405941e14b", GitTreeState:"clean", BuildDate:"2019-10-15T19:18:23Z", GoVersion:"go1.12.10", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"14+", GitVersion:"v1.14.8-gke.2", GitCommit:"188432a69210ca32cafded81b4dd1c063720cac0", GitTreeState:"clean", BuildDate:"2019-10-21T20:01:24Z", GoVersion:"go1.12.11b4", Compiler:"gc", Platform:"linux/amd64"}

Am I correct in assuming that the installing the stable prometheus operator and pushgateway charts, should result in the pushgateway being scraped by default?
For example, doing this..
Install Prometheus Operator:
helm install stable/prometheus-operator

Install Prometheus Push Gateway:
helm install stable/prometheus-pushgateway --set serviceMonitor.enabled=true

And port-forwarding from my laptop to the kubernetes cluster, I can push a metric to the gateway:
echo 'test_metric 3.15' | curl --data-binary @- http://localhost:9091/metrics/job/pushgateway/instance/laptoptest

And now my test metric correctly shows up on the pushgateway at http://localhost:9091/metrics
However, the pushgateway does not appear in the list of targets in prometheus (in http://localhost:9090/targets ). I assume it should? I also can't see the test_metric data either.
Out of the box, is more configuration needed?

I am having the same trouble.

hi @billtohara i know this bug and have a pending PR https://github.com/helm/charts/pull/18877
due to the absence of the former owner i took ownership for this PR, unfortunately I cant /lgtm my own PRs so as soon I find someone to do so this should be fixed

Should be fixed

Thank you @cstaud
So I should be able to pick this up from stable, using the same commands? I'm still using helm 2.15.1 and get the errors below. Should I upgrade to 2.16.1 or 3.0.0 to get your fix?
$ helm install stable/prometheus-operator

Error: release ringed-crocodile failed: Internal error occurred: failed calling webhook "prometheusrulemutate.monitoring.coreos.com": Post https://garish-parrot-prometheus-o-operator.default.svc:443/admission-prometheusrules/mutate?timeout=30s: no endpoints available for service "garish-parrot-prometheus-o-operator"

# Don't expect this to work, but fwiw, it complains about the namespace.
$ helm install stable/prometheus-pushgateway --set serviceMonitor.enabled=true
Error: release youthful-jackal failed: namespaces "monitoring" not found

As the error mentions the monitoring namespace is missing. The serviceMonitor resources needs to be created in the prometheus namespace (this is often monitoring, and used as default) so prometheus can watch for the services to monitor. Eighter you create a serviceMonitor in the desired namespace or you provide an custom namespace ate the values file you provide.
Have a look at the documentation, serviceMonitor.namespace
Note: The prometheus-operator charts also offers a parameter to specify a namespace where it should look for serviceMonitor, I think its called serviceMonitorNamespaceSelector

@billtohara you can provision it directly in the values.yaml or manually run it during run-time

@cstaud @billtohara the below is set by default, still the pushgateway is not scraped by the prometheus server
serviceMonitor:
  honorLabels: true


[main] Upgrade to latest dependencies

v1.3 Release prep -xtreme-sameer-vohra
/cc knative-sandbox/networking-wg-leads
/assign knative-sandbox/networking-wg-leads
Produced by: knative-sandbox/knobots/actions/update-deps
Details:
bumping knative.dev/networking 9ba6cf4...e8b2eb9:
  > e8b2eb9 Update community files (# 629)

/cc @carlisia @kvmware @xtreme-sameer-vohra
I think it is fine to merge but tagging release-leads just in case as today is the release day.

/lgtm
/approve


Use k8s session affinity for istio-telemetry

Redo https://github.com/istio/istio/pull/8733

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: bianpengyuan
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: geeknoid
If they are not already assigned, you can assign the PR to them by writing /assign @geeknoid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kyessenov What is the right place to add test for code change in mixer pilot plugin?

@bianpengyuan: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
d1fc854e80e5e028773983d892267e43aecad799
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/hold

@bianpengyuan I don't think there is a unit/integration test covering the modified code. The tests in mixer/test/client/pilotplugin* might be useful as a starting point, but I believe they are hardcoding mixer upstream.
IMHO, this should be solved by core pilot instead of a plugin. Destination rule should provide a way to opt-out of EDS and use DNS since it appears to work better for affinity in mixer case.


Crucible has incorrect name

When placed, the crucible has the name "Unfired Crucible"

Fixed in b1a4bbfa1f9caf00c2d9b4697341a73b8b158348


AttributeError exception when trying to get information on draft

newPost = project.post(headline="test draft", blocks=blocks, draft=True)
print('Check out your post at {}'.format(newPost.url))

"'NoneType' object has no attribute 'url'
The draft posts but the program stops here. Should this exception be handled?

So - this is on the top of my head, without having the code directly in front of me to validate, so like... mountains of salt ahead.

Last time I looked into this issue, it seems like a limitation of Cohost(* ish, more on this later). The short answer is there is Cohost itself doesn't have an API endpoint for retrieving drafts, and drafts are rendered on the server side (from what I can tell!).

Slightly longer:
Unfortunately, this is all kind of to do with how posts are made. Our current way of making a post involves firing some data off to Cohost, uploading the attachments, and then editing the post to include attachments, and then setting that post to public. If you have no attachments, we just fire it all up, no questions asked.
The problem with this is, the response we get back does include a post ID, but, doesn't include a URL, and, doesn't match up with the data get when we ask Cohost for posts (more on this later!). As such, we're left with two, both equally kinda sucky scenarios:

Look up the posts on the current project, and just assume the most recent post is the one we just posted
Try and ram all the data we assume we posted to Cohost into a post object, and hope it all works

We currently take approach 1, which, works fine enough.
The problem comes in when you make a draft post however. Cohost has (last time I checked) no API to pull in drafts. Instead, the server actually renders the drafts page in one. There might well be some API I could use, but, last time I checked I couldn't find one. As such, there's no way to actually retrieve back the post we got from Cohost, so this just returns None.
You can actually a reference to this inside project.py, specifically:
if not draft:
    return self.getPosts()[0]  # this will be what we just posted
return None # TODO: Get drafts working!

To fix this, we could do some hackery by trying to do approach 2 (manually construct a fake post object), however, I don't really want to do this silently. My fear is we never want to assume everything we sent on the client will be identical to what Cohost actually did, as that will be frustrating to debug.

History over, onto solutions -
This isn't to say this isn't solvable, but, moreso that as of right now this is kind of intended behaviour. However, in restrospect, I don't think a None was the right type to return here, as you might want some idea of what was actually posted. I'll change this out to instead return the raw post ID, or perhaps a Draft object, which you can then make an informed decision how to handle this in your own code.
Thanks for reporting! 🥳

Update on the Draft Problem!
I haven't got round to work on this yet, but, draft pagination in the recent Cohost update should make this somewhat easier. The issue is the endpoint still returns rendered pages, just rendered in batches. However, I think because of this, I could do some hackery by only requesting the most recent draft, to reduce load on Cohost while still retrieving the actual post delivered to the site.
As it's still rendered, I've accepted I just need to build some parser to take in a post's HTML, and generate content blocks from it. This shouldn't be too difficult, but the next couple of months for me are a little hectic, so I may not find the time to do this. But! If I find a quiet evening, I might be able to get round to this ✨


Fix the titus-sshd docker build for 2021

I think the gentoo approach is OK. It does give us
the benefit of being able to precisely control how
sshd is built, and keep all the features of a modern
ssh daemon (port forwarding, ssh CA auth, etc).
The downside of course is... gentoo and build times.
But this does work, and maybe it is just worth it to
fight the build once every couple of years.

I still have a lot of extra junk in the dockerfile that is "temporary", so I'm going to remove that one piece at a time, make itest each time to ensure it still works.

Again, just pushing this up here in case my computer explodes. make itest does work on this image this very second. I'll ask the gentoo team to not change anything :)

Ok. Green build and I'm happy with the state of the dockerfile.
If shipit I'll merge, push to docker hub, and carefully upgrade titus-sshd via FP across our infra.
This brings us from OpenSSH_7.7p1 > OpenSSH_8.1p1. (cc @JackOfMostTrades )


Better error message for World::resource_scope

Objective

Fixes #5365
The assert!() when the resource from World::resource_scope is inserted into the world is not descriptive.

Solution

Add more context to the assert inside of World::resource_scope when the FnOnce param inserts the resource.

bors r+


sort by actual value

@rultor release, tag is 0.0.46

@rultor release, tag is 0.0.46

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 0.0.46

@yegor256 Done! FYI, the full log is here (took me 15min)


Entries view error

Description
I've found that on the entries section, it shows me 100 - #entries entries.

Shows 6 entries when total number of entries is 94.
5 if 95.
7, 93
and so on...

Screenshots

I think there's no problem with the .json file. Visualisation and Journal sections show all entries properly and I've opened and beautified the .json and found no syntax error.

What is your view value set to? I just tested it with a blank profile and the entries are numbered normally

Here's my config:
"config": {
    "ui": {
      "bg": "#202020",
      "colour": "#f8f8f8",
      "accent": "#eb4e32",
      "colourMode": "sector",
      "font": "sans-serif",
      "view": 30,
      "tableView": 50,
      "stat": "decimal"
    },
    "system": {
      "calendar": "gregorian",
      "timeFormat": "24"
    }
  }

Same issue here! :)


alua: Fix MANAGEMENT PROTOCOL IN command sense code handling

The error check here is actually the mistake by CDB data from scsi
client, should not be locally in target.
Signed-off-by: Xiubo Li<EMAIL_ADDRESS>

Thanks.


DetGadget Hat

About the PR

Adds the DetGadget Hat, a hat that spawns in the detective's locker. This fancy new hat stores items that you can call upon at will using a catchphrase and stating the item you want. The default catchphrase is "go go gadget".
Why / Balance

:)
Technical details

Creates the ItemSlotVoiceControlSystem and the ItemSlotVoiceControlComponent. The VoiceTriggeredEvent is raised by TriggerOnVoice, piggybacking on that system for the catchphrase features. Adds some tags to a couple items for the ItemSlotComponent whitelist.
Media

https://github.com/user-attachments/assets/a4465907-55e6-470d-a5ff-bf4f0e4f9f20

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

add: The detective can now get the DetGadget hat, a hat for storing all the useful things the detective needs

I was going to say you should probably pick a different phrase for legal reasons, which might still be a good idea to do to maintain our usual cautiousness and also the suspension of disbelief in the game, but it does appear the Inspector Gadget trademark was abandoned.... so...
https://trademarks.justia.com/858/96/inspector-85896205.html

I don't think you can search these item slots if they're used to hide things.
What happens if someone near you says the phrase?

What happens if someone near you says the phrase?

Other people around you can take the items from the hat. I thought about removing their ability for other people from doing that, but then I thought it could be a cool way to steal things from the detective. 2 of the thief steal objectives can be kept in the hat if a thief is clever they could walk near the detective, find out their catchphrase, then steal items by speaking the words... I uh... need to reduce the hearing range though.

Hah, this is one of my favorite silly items from SS13!
It probably should be locked to the wearer's voice though. It might be fun if you could trick it with a voice mask too.

how many items can it store

This has the potential of being a strong item to the point of being mandatory for any Detective player, as anything that greatly expands available storage is of high value in the game.
The total storage space if I've understood it correctly is 9 slots, limited to a single instance of the items it can store (lighter, flashlight, cigarettes, forensic scanner, bodybag) with it likely expanding to 13 with the camera (2) & recorder (2).
On the other hand cigarettes, lighter and bodybag can be seen as superfluous items (bodybag has some uses, but not really much for you as a Detective), so that reduces the "useful" slots down to 4 (8 with unimplemented items).
That isn't terribly strong and is in line with players who choose to use the vest instead of the Detective coat. As long as the number of items that can fit in the hat are kept in check I think that this item won't cause any unbalance in Detective playstyles.
Fun!

how many items can it store

A body bag, a flashlight, a pack of cigarettes(you pull the cigs when you use the command), a forensic scanner, and a lighter.

Hah, this is one of my favorite silly items from SS13!
It probably should be locked to the wearer's/detective's voice though. It might be fun if you could trick it with a voice mask too.

The issue with that is the detectives lighter. Thief's can't really access the hats inventory if they don't have a mask. Maybe if it counts as the objective stolen when the lighter is in the hat and the hat is in the thief's inventory? I would need to test that but I believe that would work, it could be confusing for players though.

Hah, this is one of my favorite silly items from SS13!
It probably should be locked to the wearer's/detective's voice though. It might be fun if you could trick it with a voice mask too.

The issue with that is the detectives lighter. Thief's can't really access the hats inventory if they don't have a mask. Maybe if it counts as the objective stolen when the lighter is in the hat and the hat is in the thief's inventory? I would need to test that but I believe that would work, it could be confusing for players though when a hat they can't open finishes a steal object for a lighter.

Give the hat a telling description and have screwing it dump the item slots. That would solve the search issue.

"go go gadget nuclear fission explosive (nt-457129)"

how does punctuation work with this?

how does punctuation work with this?

Fine, what it's doing is testing for your set words then looking at the whole message for the first item. You can say "bodybag go go gadget!!!" if you want or any other set of words. "DEPLOY FLASHLIGHT!!!" Or something like that. I don't think capitalization matters, it's been a while since I tested the pr

this is so funny


Crystal and Cuda code p001-p003

Here's a language that generally falls under the radar that my hipster university buddies really liked back in 2015 (More info on the wiki)
With regards to Cuda, the code works but currently it's a bit spaghetti-like. If anyone has any suggestions on how to make the initialisation of inputs, weights and bias a bit cleaner and more readable I'll be happy to include the change (especially for p002 and p003 because they are the worst offenders)

Love it lol


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-yml-archive-non-versioning-archive/pull/323 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-yml-archive-non-versioning-archive.git

Docs Build status updates of commit 9611d19:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/index.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-3.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/sql-database-engine.png
:white_check_mark:Succeeded

docs/to-be-archive/level2/article3-2-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-3.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Use type exports where appropriate

Some type exports are erased by TypeScript and break when using Deno as the runtime:
error: Uncaught SyntaxError: The requested module './types.ts' does not provide an export named 'PartialCheckIssue'
export { PartialCheckIssue } from './types.ts';
         ^
    at <anonymous> (https://jsr.io/@valibot/valibot/0.33.1/src/actions/partialCheck/index.ts:3:10)

This PR fixes those exports

Congratulations on becoming our 100th contributor! 🍾


You need to install the imagick extension to use this back end

Description
How I can enable imagick extension on Vessel?

you can just install image magick in the dockerfile.
sudo apt install php-imagick


Include  when building Wayland support

Fixes #1666

Thanks! wayland isn't supported so this codepath is likely to rot as it is not tested, PRs like this are welcome for any fixes.


Broken on macOS

ping @sdondley
https://github.com/azawawi/raku-file-homedir/blob/c635ebb5333a519b5a43cbb6e84ae9193dd57c32/lib/File/HomeDir.rakumod#L15
https://github.com/azawawi/raku-file-homedir/blob/c635ebb5333a519b5a43cbb6e84ae9193dd57c32/META6.json#L10

Fixed in https://github.com/azawawi/raku-file-homedir/commit/9ff6b6eeca2af585dfc2d56a8801b14100ceca25. Also GitHub test workflow is now working on all platforms.
@ugexe  Thanks. 👍


Refinement overload added to 'every' functions

I have added Refinement overloads to the every function of ReadonlyArray,  ReadonlyRecord, ReadonlySet and their mutable counterparts.
Example
import { every } from 'fp-ts/ReadonlyArray'
import { isNumber } from 'fp-ts/number'

declare const as: ReadonlyArray<string | number>
every(isNumber)(as) // ReadonlyArray<number>

Thanks @mlegenhausen, released.


Website Release - Content Enhancement [Indonesian] - 12-04-2022

Changelog
Additional:

content/id/docs/Pengetahuan Umum
content/id/docs/Pengetahuan Umum/_index.md
content/id/docs/Pengetahuan Umum/crypto-wallet.md
content/id/docs/Pengetahuan Umum/proof-of-stake.md
content/id/docs/Seputar Cardano
content/id/docs/Seputar Cardano/_index.md
content/id/docs/Seputar Cardano/parameter-parameter-Cardano.md
content/id/docs/Seputar Cardano/serba-serbi-staking-ada.md

Minor Revisions:

content/id/docs/_index.md
content/id/docs/kegiatan-eastern-townhall-indonesia.md

I think you also have some commits today.
Please publish to the website :)

It works!


Archive pipeline on destroy

The Github Terraform provider has an archive_on_destroy argument for its github_repository resource, which allows you to optionally archive a Github repository when you remove the resource from your Terraform program, rather than completely deleting it.
Since Buildkite pipelines can also be archived, it would be convenient to add similar functionality to the Buildkite Terraform provider. This would be very useful for those that have auditing needs that require pipeline history, artifacts, etc. to be retained.

Ohh this is a really good idea @christophermaier! It probably ties into #177 as well but might need some changes. I'll have a further think/look into this

Closing this issue as completed, support for this is addressed in https://github.com/buildkite/terraform-provider-buildkite/pull/296


Node manager periodically prints to stdout

https://github.com/bacalhau-project/bacalhau/blob/297f1eaded9495e65c6e025c1290a2bf29d89448/pkg/node/manager/node_manager.go#L103
This should be a log message.
https://github.com/bacalhau-project/bacalhau/blob/297f1eaded9495e65c6e025c1290a2bf29d89448/pkg/node/manager/node_manager.go#L146
Ahem.

Fixed in https://github.com/bacalhau-project/bacalhau/pull/3585, somehow missed in review.


Documentation for set_task is for set_schedule

The documentation for set_task is describing set_schedule. Would be nice to have a description of set_task as well.
https://github.com/quantum-elixir/quantum-core/blob/92ad935a916a7e2109448a2755de77f584d255fa/lib/quantum/job.ex#L133

Fixed via #396


Add integration test for untested situation

I noticed one line of my implementation test didn't get tested by both my own unit tests and these integration tests.

This should fix that.
Source code: github.com/madewithlove/illuminate-psr-cache-bridge

Thank you for this test!


Main TourGuide CLI script.

This PR adds the main TourGuide CLI script.  All the commands but help and version will be scripts available inside the cli directory and will be added as separate pull requests and will depend on this one.

LGTM


[Automation] Generate SDK based on TypeSpec 0.15.16

[Automation] Generate SDK based on TypeSpec 0.15.16

API change check
API changes are not detected in this pull request.

/check-enforcer evaluate


Token indices sequence length is longer than the specified maximum sequence length for this model (3000 > 512). Running this sequence through the model will result in indexing errors

Token indices sequence length is longer than the specified maximum sequence length for this model (3000 > 512). Running this sequence through the model will result in indexing errors
Traceback (most recent call last):
File "", line 1, in
File "F:\PyCharm 2020.2.1\plugins\python\helpers\pydev_pydev_bundle\pydev_umd.py", line 197, in runfile
pydev_imports.execfile(filename, global_vars, local_vars)  # execute the script
File "F:\PyCharm 2020.2.1\plugins\python\helpers\pydev_pydev_imps_pydev_execfile.py", line 18, in execfile
exec(compile(contents+"\n", file, 'exec'), glob, loc)
File "E:/Documents/PycharmProjects/bert/getBertWordvec.py", line 7, in
outputs = model(input_ids)
File "F:\Anaconda3\envs\dnabert\lib\site-packages\torch\nn\modules\module.py", line 889, in _call_impl
result = self.forward(*input, **kwargs)
File "F:\Anaconda3\envs\dnabert\lib\site-packages\pytorch_transformers\modeling_bert.py", line 707, in forward
embedding_output = self.embeddings(input_ids, position_ids=position_ids, token_type_ids=token_type_ids)
File "F:\Anaconda3\envs\dnabert\lib\site-packages\torch\nn\modules\module.py", line 889, in _call_impl
result = self.forward(*input, **kwargs)
File "F:\Anaconda3\envs\dnabert\lib\site-packages\pytorch_transformers\modeling_bert.py", line 252, in forward
hi, my input data length is 3000, so the error has happened. And could I fix it through changing your code such as changge Token indices sequence length?

Hi,
To process long sequences, please use --model_type dnalong, and set the max sequence length as a multiple of 512 (e.g., 3072). Then the model should work well.

Ok，I will try，Thanks！

Closed #18.


Print download status in terminal

Issue
There should be more status messages:

When peer connects
# of incoming files
Download status

Download status
Peers will need to send/receive num of files before packets are streamed. Will also need to track when files are done.

Issue has been solved in PR #9.


Fedora 31 dotnet-sdk-3.0 checksum error during installation

I'm trying to install dotnet-sdk-3.0 on Fedora 31 and got a checksum error on both 3.0.100-1 and 3.0.101 versions when dnf tries to install netstandard-targeting-pack-2.1.0.

[MIRROR] netstandard-targeting-pack-2.1.0-x64.rpm: Downloading successful, but checksum doesn't match. Calculated: 45e199caf1151a750d11d9c5c7669d6e23d8dbb33da54d1d90bde55745996480(sha256)  Expected: fa194a8c12913dcec85c8d24bfc91686a7b93128b4c05346df16e2c641e2ba41(sha256)

I have the same issue, on Fedora 30.  What was your resolution @oldmalk?

@suvayu the suggested workaround is manually install the package.

dnf install https://packages.microsoft.com/fedora/30/prod/netstandard-targeting-pack-2.1.0-x64.rpm

After this, the installation runs fine.

@oldmalk Thanks.
I tried installing that package individually with just the name, it still failed, but using the url like you suggest worked.  I think the metadata in the repo is incorrect, and a direct install by url bypasses the checksum verification (which would actually be a bug in dnf).  Maybe it's worthwhile to leave this issue open, otherwise it won't get fixed.

Do you still have this issue? I know we had some issues with our package repos, but I expect they've been resolved by now -- CC @dagood @leecow


[hotfix][docs] Fix the formatting errors of some Chinese documents.

What is the purpose of the change

This is the English document：

This is the Chinese document：

According to the 2 pictures above, we can clearly find that the Chinese document is not formatted correctly.

Brief change log

Fix the formatting errors of some Chinese documents.

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@MartijnVisser , Sorry to bother you, can you help me review this pr? Thank you.

@MartijnVisser ,Thank you very much for your review.


Configure coverage

For coveralls, use istanbul-coveralls [--no-rm]

https://github.com/shinnn/istanbul-coveralls


Investigate and fix unit test result report

Some workflow doesn't show unit result report. For example https://github.com/nashtech-garage/yas/actions/runs/5109422529
But others do: https://github.com/nashtech-garage/yas/actions/runs/4728321212

I think it has shown up unit result but in another ci https://github.com/nashtech-garage/yas/runs/13827799338.
This issue is similar to this issue https://github.com/dorny/test-reporter/issues/67.

I think these workflows all show unit test result reports. But we export them with the same name Unit-Test-Results so they override each other.
Let me try to rename each of them with a specific name.

After set specific name for each report, they all show but centralize into the first ci:

:joy: then we should investigate further

"Check Runs are created for specific commit SHA. It's not possible to specify under which workflow test report should belong if more workflows are running for the same SHA. Thanks to this GitHub "feature" it's possible your test report will appear in an unexpected place in GitHub UI"
It seem that we can not do any thing further


Update CTO grammar for Import Aliasing

[ ] Update Parsing Expression Grammar to generate aliased import types within the AST
[ ] Generate the new parser
[ ] Add tests for import aliasing to the parser test suite

I have prepared a rough design of grammar if we are going to follow the syntax in issue, also considering the metamodel changes, i.e. a separate concept for importAliasTypes.
The grammar is close to the existing grammar for the ImportTypes
AliasedIdentifier = name:$Identifier _ ('as' _ aliasedName:$Identifier _)?
 {
	const result = {
				name,
				aliasedName
				};
	return result;
 }

commaSeparatedAliasedIdentifiers  = head:$AliasedIdentifier _ tail:(',' _ @$AliasedIdentifier _ )*
  {
    return [head, ...tail];
  }

ImportAliasTypes
  = ImportToken __ ns:QualifiedNamespaceDeclaration ".{" _ types:commaSeparatedAliasedIdentifiers _ "}" __ u:FromUri? {
    const result = {
          $class<EMAIL_ADDRESS>          namespace: ns,
          types,
      };
      u && (result.uri = u);
      return result;
  }

After parsing using the above, we will get a list of objects (object of type aliased imports) that contain the optional field aliasedName.
This could be one approach to the grammar design.

We are dropping the concept for the AliasedTypes in the metamodel and integrating it as an optional array in the ImportTypes concept.
Thus, we have dropped the grammar rules for the ImportAliasTypes and parsed aliased types in the ImportTypes under the new field aliasedTypes, while the non-aliased types fall under the types.
So, we now have the following structure.
imports:[
{
 '$class'<EMAIL_ADDRESS>      namespace: '',
      types: [],
      aliasedTypes: []
 }
]

In Technology-wt, @mttrbrts discussed the backward compatibility of the ast-generated; when we split the types based on which are aliased and which are not, we are changing the definition of the old array types; thus, the ast-generated won't be backwards compatible if any field was aliased.
So the suggestion is to keep all the types (aliased or not) in the types array if possible to prevent breaking change in the ast-generated.


topbar optional?

It would be nice to allow to remove the #topbar so the map can be included in other pages.
TIA,

If you put it in an iframe the top bar will disappear

Great thanks!


Fixing UI/UX

Issue Verification

[X] I have verified that this is a new bug report.
[X] I'm interested in working on this issue
[X] I'm part of GSSOC organization

Describe the bug
Changes to be done :

Adding about and contact page , which is missing now.

Expected behavior
No response
Steps to reproduce
No response
Additional Context
No response

Please work on one issue only, which one or you could create another issues for the remaining?

1 bug, 1 issue

Ok , I am working on this .

Please do assign gssoc label to this.


added abstract method decorator to authorization database

Yes. I ran the tests in my local Postgresql db and all the tests passed.

@rajivsunar07 Everything is fine. Thank you very much for your contribution.

You're welcome.


Update docs to use the term "Klaw Cluster API" correctly

Description
"Klaw Cluster API" should always be written in that way - capitalised and together.
Occurrences of other usages (e.g. just "Cluster API" or "Klaw cluster API" etc. should be replaced with the correct one.
In order to help find occurrences, we added a style rule in our Vale config (branding-temp.xml)
Todo:

cut and past the line related to Klaw Cluster API ("(?<!Klaw )cluster api": Klaw Cluster API) from our temporary branding-warning config and add it to branding.yaml

run npm run spell:error

This will give you a report of errors and help you find occurrences of the wrong use in the text.
The log will show you the file where the word was found as well as the line number:

update the usage to correct branding in text in markdown files

The npm run spell:error should be green before making a PR.

can I work on this issue?

can I work on this issue?

Hi @Devrishi-Dutta  I'll assign this to you :)
@Rawshansharma - I would ping you in case the issue gets stale.

@programmiri I have created a pr for this issue . Please review and let me know if something is missing.

Done with #125


Air Gap HA Install (stable) does not work

I am using version 1.13 of kubectl and version 1.11 of kubernetes (deployed per instructions from Rancher docs by RKE). I can get Rancher 2.x to deploy and work no problem via the non-air gap install method. Unfortunately the air-gap method (stable) does not work and seems to need fixing.
Edit: forgot to mention this on RHEL 7.5 operating system.
The command to deploy the certificate manager, 'kubectl -n kube-system apply -R -f ./cert-manager', fails as show below. I was able get it to deploy following instructions from helm, which seems to fix the issue by applying required Custom Resource Definitions to kubernetes. I also tried applying the change for fetching a specific version of cert-manager discussed in issue 18283, but that did not seem to make any difference.

fatal: [localhost]: FAILED! => {"changed": true, "cmd": "kubectl --kubeconfig=$(echo "/home/ec2-user/kube_config_20190314-rancher02_rke_config.yml") -n kube-system apply -R -f ~/cert-manager", "delta": "0:00:00.507408", "end": "2019-03-14 20:30:49.463995", "msg": "non-zero return code", "rc": 1, "start": "2019-03-14 20:30:48.956587", "stderr": "unable to recognize "/home/ec2-user/cert-manager/charts/webhook/templates/pki.yaml": no matches for kind "Issuer" in version "certmanager.k8s.io/v1alpha1"\nunable to recognize "/home/ec2-user/cert-manager/charts/webhook/templates/pki.yaml": no matches for kind "Certificate" in version "certmanager.k8s.io/v1alpha1"\nunable to recognize "/home/ec2-user/cert-manager/charts/webhook/templates/pki.yaml": no matches for kind "Issuer" in version "certmanager.k8s.io/v1alpha1"\nunable to recognize "/home/ec2-user/cert-manager/charts/webhook/templates/pki.yaml": no matches for kind "Certificate" in version "certmanager.k8s.io/v1alpha1"\nerror validating "/home/ec2-user/cert-manager/charts/webhook/templates/validating-webhook.yaml": error validating data: [ValidationError(ValidatingWebhookConfiguration.webhooks[0].clientConfig): missing required field "caBundle" in io.k8s.api.admissionregistration.v1beta1.WebhookClientConfig, ValidationError(ValidatingWebhookConfiguration.webhooks[1].clientConfig): missing required field "caBundle" in io.k8s.api.admissionregistration.v1beta1.WebhookClientConfig, ValidationError(ValidatingWebhookConfiguration.webhooks[2].clientConfig): missing required field "caBundle" in io.k8s.api.admissionregistration.v1beta1.WebhookClientConfig]; if you choose to ignore these errors, turn validation off with --validate=false", "stderr_lines": ["unable to recognize "/home/ec2-user/cert-manager/charts/webhook/templates/pki.yaml": no matches for kind "Issuer" in version "certmanager.k8s.io/v1alpha1"", "unable to recognize "/home/ec2-user/cert-manager/charts/webhook/templates/pki.yaml": no matches for kind "Certificate" in version "certmanager.k8s.io/v1alpha1"", "unable to recognize "/home/ec2-user/cert-manager/charts/webhook/templates/pki.yaml": no matches for kind "Issuer" in version "certmanager.k8s.io/v1alpha1"", "unable to recognize "/home/ec2-user/cert-manager/charts/webhook/templates/pki.yaml": no matches for kind "Certificate" in version "certmanager.k8s.io/v1alpha1"", "error validating "/home/ec2-user/cert-manager/charts/webhook/templates/validating-webhook.yaml": error validating data: [ValidationError(ValidatingWebhookConfiguration.webhooks[0].clientConfig): missing required field "caBundle" in io.k8s.api.admissionregistration.v1beta1.WebhookClientConfig, ValidationError(ValidatingWebhookConfiguration.webhooks[1].clientConfig): missing required field "caBundle" in io.k8s.api.admissionregistration.v1beta1.WebhookClientConfig, ValidationError(ValidatingWebhookConfiguration.webhooks[2].clientConfig): missing required field "caBundle" in io.k8s.api.admissionregistration.v1beta1.WebhookClientConfig]; if you choose to ignore these errors, turn validation off with --validate=false"], "stdout": "apiservice.apiregistration.k8s.io/v1beta1.admission.certmanager.k8s.io created\ncronjob.batch/cert-manager-webhook-ca-sync created\njob.batch/cert-manager-webhook-ca-sync created\nconfigmap/cert-manager-webhook-ca-sync created\nserviceaccount/cert-manager-webhook-ca-sync created\nclusterrole.rbac.authorization.k8s.io/cert-manager-webhook-ca-sync created\nclusterrolebinding.rbac.authorization.k8s.io/cert-manager-webhook-ca-sync created\ndeployment.apps/cert-manager-webhook created\nclusterrolebinding.rbac.authorization.k8s.io/cert-manager-webhook:auth-delegator created\nrolebinding.rbac.authorization.k8s.io/cert-manager-webhook:webhook-authentication-reader created\nclusterrole.rbac.authorization.k8s.io/cert-manager-webhook:webhook-requester created\nservice/cert-manager-webhook created\nserviceaccount/cert-manager-webhook created\ncustomresourcedefinition.apiextensions.k8s.io/certificates.certmanager.k8s.io created\ncustomresourcedefinition.apiextensions.k8s.io/clusterissuers.certmanager.k8s.io created\ndeployment.apps/cert-manager created\ncustomresourcedefinition.apiextensions.k8s.io/issuers.certmanager.k8s.io created\nclusterrole.rbac.authorization.k8s.io/cert-manager created\nclusterrolebinding.rbac.authorization.k8s.io/cert-manager created\nserviceaccount/cert-manager created", "stdout_lines": ["apiservice.apiregistration.k8s.io/v1beta1.admission.certmanager.k8s.io created", "cronjob.batch/cert-manager-webhook-ca-sync created", "job.batch/cert-manager-webhook-ca-sync created", "configmap/cert-manager-webhook-ca-sync created", "serviceaccount/cert-manager-webhook-ca-sync created", "clusterrole.rbac.authorization.k8s.io/cert-manager-webhook-ca-sync created", "clusterrolebinding.rbac.authorization.k8s.io/cert-manager-webhook-ca-sync created", "deployment.apps/cert-manager-webhook created", "clusterrolebinding.rbac.authorization.k8s.io/cert-manager-webhook:auth-delegator created", "rolebinding.rbac.authorization.k8s.io/cert-manager-webhook:webhook-authentication-reader created", "clusterrole.rbac.authorization.k8s.io/cert-manager-webhook:webhook-requester created", "service/cert-manager-webhook created", "serviceaccount/cert-manager-webhook created", "customresourcedefinition.apiextensions.k8s.io/certificates.certmanager.k8s.io created", "customresourcedefinition.apiextensions.k8s.io/clusterissuers.certmanager.k8s.io created", "deployment.apps/cert-manager created", "customresourcedefinition.apiextensions.k8s.io/issuers.certmanager.k8s.io created", "clusterrole.rbac.authorization.k8s.io/cert-manager created", "clusterrolebinding.rbac.authorization.k8s.io/cert-manager created", "serviceaccount/cert-manager created"]}

the command 'kubectl -n cattle-system apply -R -f ./rancher' results in this:

fatal: [localhost]: FAILED! => {"changed": true, "cmd": "kubectl --kubeconfig=$(echo "/home/ec2-user/kube_config_20190314-rancher02_rke_config.yml") -n cattle-system apply -R -f ~/rancher", "delta": "0:00:00.397727", "end": "2019-03-14 20:32:17.150395", "msg": "non-zero return code", "rc": 1, "start": "2019-03-14 20:32:16.752668", "stderr": "error: error validating "/home/ec2-user/rancher/tasks/main.yml": error validating data: invalid object to validate; if you choose to ignore these errors, turn validation off with --validate=false", "stderr_lines": ["error: error validating "/home/ec2-user/rancher/tasks/main.yml": error validating data: invalid object to validate; if you choose to ignore these errors, turn validation off with --validate=false"], "stdout": "clusterrolebinding.rbac.authorization.k8s.io/rancher created\ndeployment.apps/rancher created\ningress.extensions/rancher created\nissuer.certmanager.k8s.io/rancher created\nservice/rancher created\nserviceaccount/rancher created", "stdout_lines": ["clusterrolebinding.rbac.authorization.k8s.io/rancher created", "deployment.apps/rancher created", "ingress.extensions/rancher created", "issuer.certmanager.k8s.io/rancher created", "service/rancher created", "serviceaccount/rancher created"]}

If I turn off validation I simply get an json error, "cannot unmarshal array into Go value of type unstructured.detector".

@scotthesterberg could you post Rancher version you are hitting the issue on.

This would be the latest stable rancher, which from my online install appears to be 2.1.7.
I have not tried kubectl 1.11, but I encountered this exact same issue running on a machine with kubectl 1.5 (against a kubernetes 1.11 cluster), so I doubt it is a kubectl issue. I can give it a try if it would help though.

@jiaqiluo ever hit this error in your air gap testing? can you spot anything obviously wrong?

@alena1108 tried downgrading to kubectl 1.11, same error. Running kubectl from an Amazon linux 2 machine, against Redhat hosts configured by RKE.

There is no difference in airgap vs regular install besides the fetching archive and manually template and deploy in airgap vs that all automated in Helm/Tiller itself normally.
The first error still seems to refer to newer version being applied, as that needs custom CRDs before applying. Possibly the cluster was not cleaned/reset before re-applying? The second error seems to apply rancher/tasks/main.yml which is not in the chart itself, probably the playbook files.
If you can share the content of the files and reproduce using the commands/steps in https://rancher.com/docs/rancher/v2.x/en/installation/air-gap-high-availability/install-rancher/, let me know so we can reopen and investigate.


Potential Security Vulnerabilities

While it is obvious that this isn't a production project, it may be used as a template for real project. It may be an option to state in the readme, that the project contains potential security vulnerabilities and serves only as an example of implementing some techniques.
Summary
IdentityWithoutEF is vulnerable to Cross-Site Scripting (XSS) and Cross-Site Request Forgery (CSRF) that may lead to the elevation of privileges and per-user denial of service (DoS).
Details
Issue 1: XSS (CVE-2018-0784)
The application doesn't have the fix for CVE-2018-0784 that was found in ASP.NET Core templates. It is vulnerable to XSS if the logged-in user is tricked into clicking a malicious link like https://localhost:44315/manage/EnableAuthenticator?AuthenticatorUri=%22%3E%3C/div%3E%00%00%00%00%00%00%00%3Cscript%3Ealert(%22XSS%22)%3C/script%3E and enters an invalid verification code. Mode details are available in this blog post.
Impact
This issue may lead to the elevation of privileges.
Remediation
Modify the code according to the instructions from the advisory.
Issue 2: CSRF (CVE-2018-0785)
The application doesn't have the fix for CVE-2018-0785 that was found in ASP.NET Core templates. It is vulnerable to CSRF. A logged-in user with enabled Second Factor Authentication (2FA) may lose their recovery codes if they are tricked into clicking a link like https://localhost:44315/manage/GenerateRecoveryCodes or visit a malicious site that makes the request without the user's consent. As a result the user may be permanently locked out of their account after loosing access to their 2FA device, as the initial recovery codes would no longer be valid.
Impact
This issue may lead to the per-user DoS.
Remediation
Modify the code according to the instructions from the advisory.

Thanks @JarLob. I certainly don't want anyone using this as a template for a real project, but you are right that people may do so. I'll have a look into fixing the vulnerabilities or updating the readme. Thanks for filing this issue - much appreciated.


service action enable does not work on Linux Mint 17

When using the 'postgresql' (https://supermarket.chef.io/cookbooks/postgresql) cookbook, chef-solo (v11.16.4) fails with:
Error executing action `enable` on resource 'service[postgresql]'
File '/etc/init/postgresql.conf' does not exist

This is because chef assumes that Linux Mint uses upstart; for the postgresql-9.3 package, it's still using init.d.
So the action [:enable] attribute does the wrong thing.
A similar problem occurs with the 'rabbitmq' cookbook (https://supermarket.chef.io/cookbooks/rabbitmq), which makes me think that this needs fixing in chef, rather than in the individual cookbooks.

No, those are bugs against the individual cookbooks, they should already have an if statement around the platform_version when the platform is ubuntu and they need another one around the platform_version when the platform is linuxmint.

Cool, so how do I report a bug against the individual cookbooks? I might be being dim, but I can't find any link from (e.g.) https://supermarket.chef.io/cookbooks/rabbitmq where I might do that.

There should be a link to the github sources somewhere.  If not google github cookbook rabbitmq and its the top hit https://github.com/jjasghar/rabbitmq

Also https://github.com/opscode/supermarket/issues/962.


#V2 Fix podspec import

Remove sub module and use podspec instead and link SwiftAudio library

Hi, can you add
  s.swift_version = '5.0'

to the podspec, it is causing errorrs
[!] Unable to determine Swift version for the following pods:

- `react-native-track-player` does not specify a Swift version and none of the targets (`avid`) integrating it have the `SWIFT_VERSION` attribute set. Please contact the author or set the `SWIFT_VERSION` attribute in at least one of the targets that integrate this pod.

Hi @alfonsodev done

Hi @benjipott sorry for my comment it was too precipitated, I'm actually not sure about that anymore.
Currently I'm testing with your branch and pod install works without any warning about swift version 👍
But the XCode build fails with the following errors

do you have any idea how to fix it ?

I can confirm after running react-native link react-native-swift your branch builds correctly

@Guichaguri any idea when this pr can be merged in v2 ?

@benjipott
does react-native run-ios --configuration Release work for you ?
I'm getting
fatal error: module map file '/Users/alfonso/Library/Developer/Xcode/DerivedData/avid-dzwhofcxokyzvbgrircdbpntyevu/Build/Products/Debug-iphonesimulator/SwiftAudio/SwiftAudio.modulemap' not found

error

@alfonsodev no problems on my side

@Guichaguri any chances to merge this or upgrade npm ?

@benjipott Thanks for your work on this. I referenced it heavily in https://github.com/react-native-kit/react-native-track-player/pull/904. Just wanted to get this into dev first. Changes are being applied to v2 as well.


crash when use probeZS

logs and crashreports
I found that in yeelp/distinctdamagedescriptions/integration/crafttweaker/capabilities/CTResistances.java calls ed a zenclass "mods.ddd.IDDDDamageType", but the correct is "mods.ddd.damagetypes.IDDDDamageType" in yeelp/distinctdamagedescriptions/integration/crafttweaker/types/ICTDDDDamageType.java this will cause game crash when use probeZS

https://github.com/friendlyhj/ProbeZS/issues/10

friendlyhj/ProbeZS#10

friendlyhj/ProbeZS#10

That's pretty interesting. I'm surprised the CraftTweaker integration works at all even though this typo exists. Nice catch!

Offending line in question:
https://github.com/yeelp/Distinct-Damage-Descriptions/blob/1432b5f2c907c472ecc2f9a1904d7c2c4b3e1dad/src/main/java/yeelp/distinctdamagedescriptions/integration/crafttweaker/capabilities/CTResistances.java#L22


Compile on Ubuntu

Hello,
i try to install the Casinocoin-Wallet in my Ubuntu Linux over console.
The Ubuntu Version is 14.04.4 LTS. When i compile the wallet with  sudo make -f makefile.unix
I got following error
g++: error: unrecognized command line option '-std=c++14'
g++: error: unrecognized command line option '-std=c++14'
make[1]: *** [db/builder.o] Error 1
make[1]: Leaving directory `/home/csc/casinocoin-<IP_ADDRESS>/src/leveldb'
make: *** [leveldb/libleveldb.a] Error 2
Thank you for your help.

Make sure you have GCC 5 installed. Have you followed the wiki at https://github.com/casinocoin/casinocoin/wiki/Compile-Linux-CLI

Op 3 jul. 2016 om 11:20 heeft zimbo123<EMAIL_ADDRESS>het volgende geschreven:
Hello,
i try to install the Casinocoin-Wallet in my Ubuntu Linux over console.
The Ubuntu Version is 14.04.4 LTS. When i compile the wallet with sudo make -f makefile.unix
I got following error
g++: error: unrecognized command line option '-std=c++14'
g++: error: unrecognized command line option '-std=c++14'
make[1]: *** [db/builder.o] Error 1
make[1]: Leaving directory `/home/csc/casinocoin-<IP_ADDRESS>/src/leveldb'
make: *** [leveldb/libleveldb.a] Error 2
Thank you for your help.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hello,
thanks for your Reply. But i want install it on a Server, i don't have an graphical Interfaces.

Op 4 jul. 2016 om 00:30 heeft zimbo123<EMAIL_ADDRESS>het volgende geschreven:
Hello,
thanks for your Reply. But i want install it on a Server, i don't have an graphical Interfaces.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

OK, thanks, i found the mistake. I forgot the following steps
cd casinocoin/src/leveldb
make libleveldb.a libmemenv.a


EM Rifle Sets NPC's on Fire With Patch 0.0.3 Activ

During the mission when Terrormorphs attack the station i went to the NAT station and they supplied an EM weapon to take down the NPC civilians without lethal. It was an Assassin's EM Advanced Orion. EM 13. Long Barrel, Medium Scope, Stealth Lasers, Electromagnet Beams. With the Starfield Community Patch 0.0.3 active it would set NPC's on fire like the laser perk can do. With the patch deactivated it worked as normal (No NPC's Took Fire Damage). NG+(35)

Game Version
<IP_ADDRESS>
Language
English (EN)
New Game
No
Reproduction Steps
Take elevator to NAT Station, Pick up supplied EM weapon and attack hostile NPC's. Have all perks in Laser.

Is this the ONLY mod you have loaded? It doesn't sound like something any of our changes could cause.

I noticed this happening after the community patch update that integrated a lot of the fixes from Skill Fixes mod. Laser weapons that have been converted to EM are still causing targets to burn. Max level in Laser and EM skills. It's as if converted laser-to-EM weapons are still causing part of the laser skill's bonus to be applied, even though that weapon is no longer supposed to be considered a laser.

This will be fixed in the next release


Change style slightly

Thoughts? I tried to reduce the thick and heavy gray borders. Sections outside these screenshots haven't been affected.
Live demo: https://npmgraph-fregante.vercel.app
More can be done, but it's a start, if you'd like to entertain updates.
Before

After

Also the loader now:

Ping if you're interested

Sorry for the radio silence on this one.  I'm okay with some of this, but the change from splitter -> little knob thing is actually a regression.  I had that UI originally and it just didn't feel right to me.  To... one-off.  I opted for the more conventional splitter, even though it takes up a little more room.

The thing is I realized that that’s what made it look ugly in #30: The “boxing” of the UI rather than the colors. Rather than having a thin clickable line I opted for a toggle with a wide square area, and it helps with the style too

What if I restore the full-height toggle but leave it white/transparent unless hovered?

🙌


Add several useful goals

in_environment can read the current substitutions;
success is always successful;
failure is never successful;
isValue checks if a term is a Value;
isVariable checks if a term is a Variable;
isList checks if a term is a List;
isMap checks if a term is a Map.

I have not added tests, but there are comments.

I'll merge your pull request if you fix the naming inconsistency introduced with the in_environment goal. So as to remain consistent with the remainder of the library, it should be spelled inEnvironment.

Done.