Dark mode tries to save to realm

Describe the bug
Dark mode is trying to sync to the existing Realm schema with fields that don't exist. This means that drafts and boardings don't save properly.
To Reproduce
Steps to reproduce the behavior:

Login
Click the profile avatar in the upper right
Go into Dark Mdoe
Create a boarding and submit it.
see the error: "an error occurred while saving"

I see this error in the emulator. On my physical device, I don't see an error, however the boardings don't get saved into the database.
In the logs I see:
Ending session with error: failed to validate upload changesets: could not find primary key field "_id" in schema "DarkMode" (ProtocolErrorCode=212)
Expected behavior
In #390, our lead developer said "Probably better to just store it as local state in the device, whether outside of Realm or in a non-sync’d realm, as it’s reasonable that someone would want different values on say their phone and tablet."
That's what should happen; regardless of the dark mode status, the boarding reports should be able to save.

@thearavind I think you would have the most context to solve this - would you like to take a try at it?

Yes @Sheeri please assign this to me I can do it

Fantastic @thearavind ! I have assigned this to you.


idn_to_utf8: empty domain name in EmailAddress Validator

idn_to_utf8(): idn_to_utf8: empty domain name in ...\vendor\zendframework\zend-validator\src\EmailAddress.php on line 557
When a field has EmailAddress validator and its value is empty, this warning shows up.
The fields validator has required=true and also NotEmpty validator.
shouldn't a empty value considered false validation and return false before checking for domain name ?
Zend 2.5.1
php 5.6.16

Did you set the flag "break chain on failure" on your validator chain?

@Maks3w tnx, I used to think that it was set by default and i was confused why it wasn't stopping with the required or NotEmpty.
I set it and it works now.
But the problem still exist, EmailAddress validator should be able to validate empty fields as well, in case the email field is optional and there is no other validator.

Required/Optional is not responsibility of zend-validator. zend-inputfilter is the responsible of that.
zend-inputfilter try to be agnostic about what "empty" means (null, 0, '', [], 'This is what I understand as empty') so it only expects the field name does not appear as a key in the data provided.

All i am saying is that :
EmailValidator checks if $email isString but dose not check if its empty(a string with the length of zero)
idn_to_utf8 dose not accepts empty strings(a string with the length of zero) and gives warning when $email is a empty string.
if $email is empty string(a string with the length of zero) ,idn_to_utf8 should not be called at all.

    protected function idnToUtf8($email)
    {
        if(strlen($email) == 0)
            return $email;

        if (extension_loaded('intl')) {
            return idn_to_utf8($email) ?: $email;
        }
        return $email;
    }

Ok. Do you want to send a Pull Request with your patch (and tests)?

created the pull request.#94
Test case already exist for empty value
https://github.com/zendframework/zend-validator/blob/master/test/EmailAddressTest.php#L259
but this only happens when intl extension is active.

this can be closed as #94 merged.

Thanks @samsonasik


⚠ Bofrost.fr has degraded performance

In 9df5d38, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1398 ms

Resolved: Bofrost.fr performance has improved in 818423a.


[BUG] docker standalone image does not build

Describe the bug
I am trying to build the standalone docker image version, because the existing tooling does not allow to remap WEBSOCKER_PORT or other ports:.
To Reproduce
Steps to reproduce the behavior:

clone https://github.com/illacloud/deploy-illa-manually
cd to docker,
run make build-docker-image
See an error

illa-builder:build-self: cache miss, executing 3bebdfaecb22dad3
illa-builder:build-self:
illa-builder:build-self: ><EMAIL_ADDRESS>build-self /opt/illa/illa-builder/apps/builder
illa-builder:build-self: > pnpm run ts-check && vite build --mode self
illa-builder:build-self:
illa-builder:build-self:
illa-builder:build-self: ><EMAIL_ADDRESS>ts-check /opt/illa/illa-builder/apps/builder
illa-builder:build-self: > tsc --noEmit
illa-builder:build-self:
illa-builder:build-self: vite v3.2.4 building for self...
illa-builder:build-self: transforming...
illa-builder:build-self: Use of eval is strongly discouraged, as it poses security risks and may cause issues with minification
illa-builder:build-self: ✓ 3704 modules transformed.
illa-builder:build-self: Terminated
illa-builder:build-self:  ELIFECYCLE  Command failed with exit code 143.
illa-builder:build-self: ERROR: command finished with error: command (/opt/illa/illa-builder/apps/builder) pnpm run build-self exited (1)

command (/opt/illa/illa-builder/apps/builder) pnpm run build-self exited (1)
 Tasks:    59 successful, 60 total
Cached:    0 cached, 60 total
  Time:    2m4.549s

 ERROR  run failed: command  exited (1)
 ELIFECYCLE  Command failed with exit code 1.
The command '/bin/sh -c pnpm build-self' returned a non-zero code: 1

------------------------------------------------------------------------------------------------------

illasoft/illa-builder                                          latest                         74c8c548c35b   10 days ago      724MB

I tried updating npm, and purging some caches in the dockerfile
## clone frontend
WORKDIR /opt/illa/illa-builder
RUN cd /opt/illa/illa-builder
RUN pwd

RUN git clone https://github.com/illa-family/illa-builder.git /opt/illa/illa-builder/
RUN git submodule init; \
    git submodule update;

RUN npm cache clean --force
RUN npm install -g<EMAIL_ADDRESS>RUN npm cache clean --force
RUN npm install -g pnpm
RUN whereis pnpm
RUN whereis node

Expected behaviour

consistent usage of variables (this is high level issue I tried to fix myself)
npm actually build/install all the code.

 And could you paste the docker file here?

Closed due to no response.


Movies section

Maybe add a "Movies" section next to the "Series"?

The separate Series section has a few advantages over Series in the Categories section:

It has a specific sort order (defined by VRT), kind of "popular" / "must-see" / "newest" instead of A-Z sorting
It excludes certain categories ("nostalgie", duplicates with "audiodescriptie")
It is able to show the description
It just looks nice 😄

So I'll keep it as-is for now. I might move it to the Home tab in a later version to be more inline with the real VRT.NU website.


Popups don't show on form widgets hidden by branching logic

Hi,
thanks for this module, it adds a really useful functionnality to REDCap.
It's working fine, except when the linked text is in a form widget hidden by branching logic.
In this case, the form widget is not visible on page load, and the javascript code on of the module doesn't find it. I played around with the code and found it comes from this line (line 129 in main Class):
if(node.parentNode.nodeName == 'SCRIPT' || node.textContent.trim() == '' || !$(node.parentNode).is(":visible")){
Removing the last part of the test make the popups work even when under branching logic:
if(node.parentNode.nodeName == 'SCRIPT' || node.textContent.trim() == ''){
I wonder what this test is about and if it would be safe to remove it?

https://community.projectredcap.org/questions/44482/module-inline-descriptive-popup-popups-dont-like-b.html


Fix TopK output shape when k is not known

TopK validation left the "k" axis unchanged when k wasn't known

@tomdol I'm not sure how to make k a scalar in the python test. Could you fix it? Thanks.


Protos\greet.proto: No such file or directory

followed https://docs.microsoft.com/en-us/aspnet/core/tutorials/grpc/grpc-start?view=aspnetcore-5.0&tabs=visual-studio#create-a-grpc-service
created template gRPC prod in VS 2019, both with .net 5 and .net 3.1. neither worked and project won't build.
named project "GrpcGreeter".
the services class has multiple ref errors. (nb: the SSL security popups did not appear)

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

ID: 1d298a6c-8ddc-dbc8-d219-0fa7138442c2
Version Independent ID: b96be346-958a-cfcd-d80a-c352d64223e8
Content: Create a .NET Core gRPC client and server in ASP.NET Core
Content Source: aspnetcore/tutorials/grpc/grpc-start.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @JunTaoLuo
Microsoft Alias: johluo

@wadepickett  can you verify the tutorial works?

Thanks #SogoGolf for taking the time to show us what you were running into.
I followed the tutorial using  Visual Studio 2019 version 16.8.5 and the latest .NET SDK.  The resulting new project includes Grpc.AspNetCore 2.34.0.
#SogoGolf you were getting Grpc.AspNetCore package version 2.27.0.  I would guess you might be using an earlier version of Visual Studio than 16.8.5. In the tutorial the client instructions install the latest of the tools, which are also a newer version.  So there is some potential for version mismatch headaches.
For me the server app and the client console app compiled as expected and ran as expected for .NET 5 using those versions I mentioned.
That said this content does need help:  It has never been updated for version 5 and needs to point to the latest Visual Studio and SDK for starters.  I last updated this topic for 3.1 and then it has been updated by a few folks since but not for a 5.0 update.
There was an issue around renaming the namespace in the greet.proto to "GrpcGreeterClient" which was fixed in the doc but was not updated in the current client downloadable sample: The namespace inside the greet.proto file for the client sample project's namespace should be:
option csharp_namespace = "GrpcGreeterClient";
but incorrectly, the client sample was showing the original namespace for the server app greet.proto:
option csharp_namespace = "GrpcGreeter";
I will:

Update the topic for version 5 and fix the client app sample.
Clarify the instruction for the  SSL security popups.  You will not get those if they were set previously, but the instruction assumes you always would.
Fix the Client downloadable sample.

The client sample that doesn't run is a bit of blocker, along with the the mismatch between the 3.1 pre reqs being specified while the latest tooling is still installed.  So I set this to P1 in my queue.


Completion of the management of Custom Roles in the death Event

noted that I unintentionally activated a thing that removed all unnecessary spaces, and added some to the "if". this made the pool request unreadable sorry...
All changes :
Synapse3.SynapseModule/Config/SynapseTranslation.cs
Synapse3.SynapseModule/Events/PlayerEvents.cs
Synapse3.SynapseModule/Map/Rooms/RoomService.cs
Synapse3.SynapseModule/Patches/PlayerPatches/DeathPatches.cs the whole file has been changed

I would like to point out that I could not test the code because I had a patch error which seems to come from Nerone (and I don't think I'm causing this error whit my new code). The code should work perfectly I used it under Synapse2...

Use the Custom Synapse Logger Utility i've made, it uses the same Neuron Logger.

P.S I saw you also removed the Kick Event, the kick event is useful and can be used to kick players, you literally can't -1 Ban Event, its impossible. Add it back again.

I hopefully have time for a rewiev soon. The Harmony Error ist due to a Bug that we still havent fix (its also prevents us from using transpiler) but now to Ludens comments: im on mobile but I didnt See that He removed the kick Event and the Snypaselogger is optional when we Change to it as Default we have to do to it in the entire Project Not Just this random pr. Its okay to Help reviewing the Code but at the end we Decide about what will be included and what not

Use the Default Synapse Logger Utility i created, it uses the same Neuron Logger as default, but makes it easier to do the job.
for another pull request, because my specific pull request is modification of an Event.

PS J'ai vu que vous avez également supprimé l'événement Kick, l'événement kick est utile et peut être utilisé pour kicker les joueurs, vous ne pouvez littéralement pas -1 Ban Event, c'est impossible. Ajoutez-le à nouveau.
Un événement d'interdiction signifie simplement interdire, donner un coup de pied signifie simplement expulser quelqu'un du serveur, donner un coup de pied ne provoque pas d'interdiction, il expulse seulement celui d'un serveur et le joueur expulsé peut rejoindre à tout moment, tandis que l'interdiction bannit un joueur, fait ne pas laisser un joueur entrer sur le serveur pendant un certain temps ou de manière permanente
Et c'est pourquoi, le KickEvent est la fonction inverse de la fonction BanEvent, exactement la même chose que vous voulez faire.

are you sure of what you are saying? It does not seem that I have edited any other Event than the death event...
the only edition made on the kick event is a deletion of a tab performed by my IDE.

P.S I saw you also removed the Kick Event, the kick event is useful and can be used to kick players, you literally can't -1 Ban Event, its impossible. Add it back again.
A ban event simply means banning, kicking simply means to kick someone out of the server, kicking doesn't cause a ban, it only kicks the one out of a server and the kicked player can rejoin anytime, while Banning bans a player, does not let a player enter the server for a specific amount of time, or permanantly
And so that's why, the KickEvent is the reverse function of the BanEvent function, just the same you want to do.

are you sure of what you are saying? It does not seem that I have edited any other Event than the death event...
the only edition made on the kick event is a deletion of a tab performed by my IDE.

It shows as a whole deletion of the event function in the commit, check.

Can you send me the link of the line informing of this deletion. Because, I can't find it.

Although, its okay if you add the kick function in the banevent too, i wanted it to be separate so it made more sense too.

@warquys Its the line 650, where you deleted the KickEvent function along with the ChangeRole function.

No. Github cut it bc no modification was aplided, but you can show it clicking on the the double arrow.

No. Github cut it bc no modification was aplided, but you can show it clicking on the the double arrow.

I checked it again, and yes, it is fixed. There doesn't seems to be any problem with it now.


HTMLEditorField saving of image HTML tag does not convert to ShortCode if no parent HTML tag

Affected Version
4.13.12
Description
If you use an HTMLEditorField and insert an image NOT seating inside another HTML tag, the ShortCode conversion does not happen + the src attribute gets saved with the BASE_URL.
Normally, whether the img seats at the root or inside another HTML tag should not matter: The ShortCode conversion should happen + the src attribute should not contain the BASE_URL.
Steps to Reproduce

Add this code to _config.php: HTMLEditorConfig::get('cms')->setOption('forced_root_block', '');
Load an HTMLEditorField in the CMS and make sure it's empty of any content
Insert an image using the toolbar
Save the record
Check how the record is saved in the database and you should see that it has not been converted into the ShortCode format + The src path contains the BASE_URL.
Erase the HTML content.
Insert an h1 tag with some text.
Repeat the same image insertion process above but this time, insert the image inside the h1 tag and save.
Check how the record is saved in the database and this time the ShortCode conversion has happened + the src path does not contain the BASE_URL.

PR

https://github.com/silverstripe/silverstripe-asset-admin/pull/1395

Linked PR has been merged, it will be be auto-tagged by github actions


Added tests for editemp command

Coverage increased (+2.3%) to 54.849% when pulling 73a0c431ca3aeccaaf2e259604bf14503ef7d0d4 on kianhong95:JUnitTests into 2feed6f17b1e2ad045a383a186a6251cc7ad7021 on CS2113-AY1819S1-F09-2:master.


Heads-up: #[start] is being removed

We are planning to remove the unstable #[start] attribute from the compiler, see https://github.com/rust-lang/rust/pull/134299 for rationale and further details. This repo showed up in a search for users of the attribute. For all we know, there's no good reason to use this attribute and you can do the following instead:
#![no_main]

#[no_mangle]
extern "C" fn main(argc: core::ffi::c_int, argv: *const *const u8) -> core::ffi::c_int {
  // ...
}

If that for some reason does not work, please let us know!

Thanks for the heads up! This was only used in some test code that I had copied from elsewhere and I have no problems taking it out :smile:
Will be fixed with https://github.com/ipetkov/crane/pull/769


Quarterly balance sheet

Is there a way to get quarterly balance sheets / income statements.

I will upgrade the package ASAP.

The package is updated. Please install the latest version. Thank you for your advice.


Use symfony/process instead of shell_exec

The shell_exec calls reported that the log file was not found even if a PHP file_exists said that it was there. I guess there is something with the escaping of the shell argument, so I added symfony/process to project and used that instead of shell_exec and escapeshellarg. My experience with symfony/process is that the escaping works very well and it fixed the problem here.

Merged in 7e119aa8e3539167a9b81952d9f1f9cdb89a4d8a. Thanks!


Update nscplugin, sbt-scala-native, ... to 0.4.6

Updates

org.scala-native:nscplugin
org.scala-native:sbt-scala-native
org.scala-native:scala3lib
org.scala-native:scalalib
org.scala-native:test-interface

from 0.4.4 to 0.4.6.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-native" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "org.scala-native" }
}]

labels: library-update, early-semver-minor, semver-spec-patch, commit-count:1

Superseded by #1071.


Fix postgres persistence

Hi, first thanks for developing and sharing this project.
We are using with postgres 10.6, and we notice some problems

First when trying to delete (don't have now the message) it's fail because "job -> 'id'" has documented here '{"a": {"b":"foo"}}'::json->'a' gives {"b":"foo"} and we need filter by the id of the job... '{"a":1,"b":2}'::json->>'b' that give 2. We replace > with >> in get and delete
Second, save need check if exist, to update either to insert (as diferent as mysql where can use replace, hear postgres/json)

If i have more time, i'll make another PR with https://github.com/ory/dockertest for testing.
Thanks again

Hi, i get time for this... the error where
&pq.Error{Severity:"ERROR", Code:"22P02", Message:"invalid input syntax for type json", Detail:"Token "0ce70361" is invalid.", Hint:"", Position:"", InternalPosition:"", InternalQuery:"", Where:"JSON data, line 1: 0ce70361...", Schema:"", Table:"", Column:"", DataTypeName:"", Constraint:"", File:"json.c", Line:"1248", Routine:"report_invalid_token"} i upload the integration test (flaged with integration) job/storage/postgres/postgres_int_test.go

In addition... i'll use this branch to include job InitDelayDuration in GetAll, because without this... when kala restart with postgres persistence, jobs doesn't start... which doesn't happen in mysql (code copied and refactor for json table)

Seem that when the test works again in circleci https://issueexplorer.com/issue/hashicorp/raft-boltdb/21 this library is a bit older... upgrade library boltdb/bolt with go.etcd.io/bbolt

@tooolbox or @ajvb  now all green :) https://app.circleci.com/pipelines/github/ajvb/kala/97/workflows/f37a0256-1ee3-48a9-baf1-aafb2bc8ae9f/jobs/509


Change docker execution command from npx . to npx --yes . to prevent prompts on new container creation?

When I create a new docker container based on the antennas image, I get the following prompt/output:
Need to install the following packages:
  file:

To get the container to run, I need to attach to the container and send a 'y' to finish the setup.
Changing the execution command from npx . to npx --yes . seems to stop the prompt being required on startup. Maybe this could be added to the Dockerfile to make setup a bit smoother?

Hmm, I don't get this error on Docker for some reason, but happy to add that in to help.


[ISSUE #8496] Add more test coverage for ConsumeMessagePopOrderlyService

Fixes #8496 .

@yx9o A reminder from the Tianchi platform: it‘s better to submit your results to the official website asap.

link #8262


fix Issue 4149

use DWARF ref type

Based on patch in:
http://d.puremagic.com/issues/show_bug.cgi?id=4149

To do this right, every place TYnptr is used TYref probably also needs be there too, everywhere.  The tester shows that this pull fails on all 32 bit platforms.

The ref'ness was lost before it got into the dwarf stage, but also easy to fix,
assuming that Walter is right about the backend being ready to handle TYref's
now.

Yeah, taking that literal was a little naive. I will have a look when I get to test it on a Windows machine.


Add  the hint to register the resource provider Microsoft.Management before deployment

This PR fixes

@nielsophey - what is the issue you are encountering?
CC: @krnese

RP registration is done implicitly when deploying ARM templates including resources for a given namespace. What is the issue?

Hi If you deploy the ARM template without having the Resource provider registered you will get an Error...I have tried it out twice in different subscriptions. If Microsoft.Management is not registered the deployment will not run - so my idea was to write it into the prereqs...

@nielsophey thanks for adding details. Can you pls share repro steps, and point to template you deployed, how you deployed, as well as your permissions on the tenant root? The template will register management RP (to the tenant) when creating the management group structure.

@krnese - i took the template https://ms.portal.azure.com/?feature.customportal=false#create/Microsoft.Template/uri/https%3A%2F%2Fraw.githubusercontent.com%2FAzure%2FAzOps%2Fmain%2Ftemplate%2Fux-foundation.json and startet to deploy it into a quite new tenant without ever having a management group deployed. The tenant had only one subscription. The deployment starter and failed because Microsoft.Management resource prodiver was not found - once I registered the RP the deployment worked (still having the trouble to do the deployment twice, some policies needed time to be deployed). Can I share something else to you to make it more clear?

@nielsophey; we have verified that this is not by design and we are investigating with the team what is occasionally causing this.


No Relu Layer in squeezenet.py ?

I found there is a details in the paper, that is "• ReLU (Nair & Hinton, 2010) is applied to activations from squeeze and expand layers." But I can't find any use of ReLU in squeezenet.py. Are there some reason for without ReLU?

It actually is using relu, it's just not explicit.  It's the default activation function for the tf.contrib.layers.conv2d.
https://www.tensorflow.org/api_docs/python/tf/contrib/layers/conv2d

I got it, thanks!


[backend/age] Use age as a library

At the moment age is not useable as a Go library.
The author announced that this would be supported in the future.
Once it's available we should switch to using age as a library since this will simplify gopass setup and possibly unlock a few more use cases.

Hey, I have a draft of the API I am going to expose, could you have a look and tell me if it'd work for you in FiloSottile/age#63?
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

I finally got around to use the new API and I have to say that it's great.
Really clean and well thought of API.
My use of it isn't nearly as nice (yet), but it should serve as a good proof of concept.
Expect me to further iterate on it.

Hey @dominikschulz first of all thanks for the great work :muscle:
I cannot find the documentation on how to integrate age with gopass, could you point me in the right direction? Thanks!

It's very much experimental.
gopass init --crypto age should get you started.
More documentation is coming soon.


[bug] Auto rebuild not working in elements/docs folder

Describe the bug
To see any change to a doc in elements/docs, the local server needs to be restarted.
Which Component?
ux.redhat.com (not a component)
Steps to reproduce

Run npm start to start a local instance of ux.redhat.com.
Try making a change in any of the existing pages under the elements/docs folder or try adding a new page to a element's docs.
See if the preview automatically shows the change.
If it doesn't, restart the server. You should be able to see the change after that.

Expected behaviour
Once you have a local preview open, it should be watching for and showing changes as you save them.
Screenshots
No response
Element HTML
No response
Operating System (OS)
macOS
Browser
No response
More browser information
No response
Additional Info
No response

https://github.com/11ty/eleventy/issues/2909


vkMapMemory and vkBindImageMemory call ordering issue

Hi,
I am observing a problem while uploading image data to the GPU.  My case is as simple as to create a linearly-tiled image with host-writable and host-coherent backing memory; then to copy the pixel data and to draw some textured geometry. Pretty much the same as the well-known LunarG cube demo does.
If I do the work in the following order:

vkCreateImage
vkAllocateMemory
vkBindImageMemory
vkMapMemory + copy + vkUnmapMemory

everything works as expected. However, if I swap calls 3 and 4, so that the map+copy+unmap sequence appears before vkBindImageMemory, the result is completely black image.
I've tested both ways on a Windows+GTX1070 machine with 'native' Vulkan and had no issues whatsoever. According to the specs, there seems to be no ordering requirements for these calls.
This gist illustrates the change and may be used to reproduce the issue using the cube demo. Points of interest are marked with POI in comments.
This doesn't seem to happen with buffers, except texel buffers, which are apparently backed with images in MoltenVK.
I am using MoltenVK ver. 1.0.3 (bundled with LunarG SDK (vulkansdk-macos-<IP_ADDRESS>)), macOS 10.13.4. Hardware: MacBook Pro (Mid 2015) with Intel Iris Pro.
An easy way to encounter such a problem is to allocate resource memory via VulkanMemoryAllocator by calling vmaCreateImage and passing VmaAllocationCreateInfo with VMA_ALLOCATION_CREATE_MAPPED_BIT flag. This will cause the memory to be mapped before it is bound to the VkImage.

@ch45er
This has been fixed in PR #173. Please verify and close this issue if successful.

Hi @billhollings,
with the patches applied, the modfied cube demo works properly. However, I am still experiencing (seemingly the same) problem with the VMA (as described above). I would like to take some time to debug it and make sure it's (un)related.
Thanks!

@billhollings
I have an update. What is happening now:

vkAllocateMemory
vkBind*Memory
vkMapMemory + copy + vkUnmapMemory

sequence works no matter if vkBind*Memory appears before or after vkUnmapMemory.
But the underlying Metal resource update actually happens upon vkBind*Memory or vkUnmapMemory.
In my case, the VkDeviceMemory has VK_MEMORY_PROPERTY_HOST_COHERENT_BIT set and is intended to be permanently mapped. The memory is used for resource uploading and staging resources are sub-allocated from it. Since the memory is host coherent, I expect any writes to that memory to be instantly visible to the forthcoming commands (e.g. vkCmdCopyImage) as the spec says: "the host cache management commands [...] are not needed to flush host writes to the device".
What I am seeing however, is that synchronization is required with MoltenVK.
In the modified cube demo case, the writes are implicitly flushed by either vkBind*Memory or vkUnmapMemory. For example, if you "forget" to unmap the memory in demo_prepare_texture_image, you will see a completely black texture. This may be "fixed" though, by moving vkBindImageMemory down (to the second POI mark).
Should I open a new issue for this? Is host coherency in Vulkan terms even possible in Metal?

@ch45er
Unfortunately...you are correct in your last statement. Host-coherent texture memory is not natively supported under Metal.
However...it might be possible to deal with this if we limit mapped memory to linear textures that only support transfer operations (as per the Vulkan spec)...and then implement such a VkImage as a MTLBuffer instead of as a MTLTexture. A MTLBuffer can be coherently mapped via a permanent vkMapMemory() call.
I have opened a new issue #175 as a separate enhancement request for that...and I am closing this issue...since your original problem has been resolved.

@billhollings thanks for your effort and clarification.


Config.cleanupWikifierOutput = true injects extra  blocks in delayed-output macros

The issue was discovered (not by me) in a simple text where part of it was added later using the <<timed>> macro. Boiling it down to the smallest possible code, the following code creates markedly different results depending on the Config.cleanupWikifierOutput setting:
<<nobr>>Text <<timed 1s>>more text<</timed>>.<</nobr>>

Result with Config.cleanupWikifierOutput = false
Text more text.

Result with Config.cleanupWikifierOutput = true
Text
more text
.

Inside of the <<timed>> macro specifically, the Wikifier gets called without any options:
new Wikifier(frag, item.content);

Ref: https://github.com/tmedwards/sugarcube-2/blob/master/src/macros/macrolib.js#L3426
... but even if it was, there is currently no option to not surround the result with a
...
 block that would be passed on to the internal convertBreaks() method. The issue is compounded by the fact that a construct like this:
A starting paragraph <<timed 1s>>that is being extended...

And also split apart<</timed>>. Isn't it //fun//?!?

... would require the macro to split the surrounding
 DOM element into two ore more.

No generically applicable workaround aside of not using the setting exist yet, to my knowledge. For the use case of "inline" <<timed>> only, the following as a replacement of the above line of the <<timed>> macro works well enough:
let oldCleanup = Config.cleanupWikifierOutput;
Config.cleanupWikifierOutput = false;
new Wikifier(frag, item.content);
Config.cleanupWikifierOutput = oldCleanup;

Yes.  This is a thorny problem for multiple reasons.  This may need to be pushed to v3.


Improve navbar and make it fixed navbar at top.

[GSSOC'21] No its not a problem. I tried to enhancing the look and make it more easier to jump on one page to another through navbar.
Navbar should be fixed at top. Right now It is not. When scrolling it's not getting displayed to user and have to go up again to navigate through another section.
@urvashi-code1255 @Vishal-raj-1  I need to make that update allow me to do it and add the label of GSSOC'21.

@aashi2912 Check the website. This issue is already solved.


Consolidate auto-release workflow

what

Use cloudposse/github-action-auto-release in auto-release.yaml workflow

why

Solve old nodejs warning
Reduce duplication of code

@osterman done


[Dubbo-6963] fix bug: delete last config not take effect

fix #6963 ，修复删除最后一条动态配置不会生效的问题

@wikiwikiwiki Hi, thanks for your contribution.
Please merge the latest master branch to resolve confilcts.

@wikiwikiwiki Hi, thanks for your contribution.
Please merge the latest master branch to resolve confilcts.

already resolved

Codecov Report

Merging #6964 (f420df1) into master (68062c1) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #6964      +/-   ##
============================================
- Coverage     59.17%   59.13%   -0.04%
  Complexity      529      529
============================================
  Files          1076     1076
  Lines         43451    43451
  Branches       6352     6352
============================================
- Hits          25711    25696      -15
- Misses        14901    14919      +18
+ Partials       2839     2836       -3

Impacted Files
Coverage Δ
Complexity Δ

...e/dubbo/registry/integration/RegistryProtocol.java
57.94% <0.00%> (ø)
0.00 <0.00> (ø)

.../apache/dubbo/rpc/protocol/AsyncToSyncInvoker.java
62.06% <0.00%> (-10.35%)
0.00% <0.00%> (ø%)

.../dubbo/remoting/transport/netty4/NettyChannel.java
59.40% <0.00%> (-4.96%)
0.00% <0.00%> (ø%)

...a/org/apache/dubbo/monitor/dubbo/DubboMonitor.java
86.66% <0.00%> (-1.91%)
16.00% <0.00%> (ø%)

...pache/dubbo/registry/support/AbstractRegistry.java
78.14% <0.00%> (-1.49%)
0.00% <0.00%> (ø%)

...e/dubbo/remoting/transport/netty4/NettyClient.java
69.56% <0.00%> (-1.45%)
0.00% <0.00%> (ø%)

...ng/transport/dispatcher/all/AllChannelHandler.java
89.65% <0.00%> (ø)
0.00% <0.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 68062c1...f420df1. Read the comment docs.

LGTM.
@wikiwikiwiki Do you have any time to add some unit test cases for this? I will merge this pr later, and we can do it in a new pull request.

LGTM.
@wikiwikiwiki Do you have any time to add some unit test cases for this? I will merge this pr later, and we can do it in a new pull request.

ok


OpenAI API Response Error

OpenAI API Response was: TypeError: Cannot read properties of undefined (reading 'body')

@yohappy what function are you using?

+1

OpenAI API Response was: TypeError: Cannot read properties of undefined (reading 'message')

set api key
open codegpt chat
type for example Hello
get error above

fixed with the last version


operation response schema and response example behaves differently

Description
I am depening on response_schema and example_response extensively in my custom resolver. I am expecting them to give out schema and example for same operation in same fashion, but they are not.
Consider hello world yaml
from connexion.mock import MockResolver

class CustomResolver(MockResolver):
    def mock_operation(self, operation, *args, **kwargs):
        response, code = operation.example_response() # Returns "hello dave!", 200
        schema = operation.response_schema() #returns {}

I can understand that we need to specify status_code and content_type for the schema, however https://github.com/zalando/connexion/blob/master/connexion/operations/abstract.py#L283 seems to be using default content type in case one fails to specify. Why then, are we not using same for status_code? Something like
status_code = status_code or sorted(self._responses.keys())[0]

Expected behaviour
response_schema should give out default appropriate schema found, even when status_code is not mentioned (increasing status code can be used to determine first matching schema).
Actual behaviour
response_schema returns nothing when status_code is not mentioned. However, example_response does.
Additional info:
Output of the commands:

python --version
Python 3.7.4
pip show connexion | grep "^Version\:"
Version: 2.4.0

It makes sense to me that when we are trying to generate an example we can guess very liberally about what the result might be.
However, guessing at the response schema could cause all kinds of havoc - we might use it to validate the response.
My guess is that your change would cause serious breakage for folks using the "validate_responses" option, as connexion might suddenly start validating responses with guessed schemas. What do you think?

It would probably make more sense to make the status_code required, since every response schema needs to be under a status_code in both swagger 2 and openapi 3. You can still implement a mechanism to fetch the lowest response code outside of this function.

@RobbeSneyders The status_code is required yes it your comment makes sense. However, the same goes for responses' examples. They too are dependent the same way on status code as their schemas are.
This, in a sense, is breaking the uniformity between schema and examples. Either both have to have same signature/derivates, or the same should be expected from both. This is really confusing for me.


Contact SarthakJariwala and coordinate efforts, give credit.

I made a mistake. When I started down the path of making a nox plug-in for pyscaffold, I first searched pypi for a plug-in to use, assuming that someone had already made one. Seeing none, I assumed nobody had worked on one. So, created this repo, made the plug-in, and pushed it to pypi.
When I felt I had something worth sharing, I figured I would post to the pyscaffold community or to the nox community to see if there was community interest in the plug-in. That's when I learned about @SarthakJariwala and his efforts to make a nox plug-in for pyscaffold, through the issue  https://github.com/pyscaffold/pyscaffold/issues/287 and their plug-in efforts here: https://github.com/SarthakJariwala/pyscaffoldext-nox to create a nox plug in.
I would not have registered the pypi package if I had known of the prior effort to make a plug in, especially after seeing the discourse on the topic by the community already.
I welcome well-intentioned discussion on what I should do, especially if @SarthakJariwala has any opinions. Feel free to reach out publicly or privately.

Thanks for reaching out @jaustinpage. Nothing wrong on your end :-)
The extension was almost ready a while back, but I didn't push it to PyPI. I haven't worked on the extension for almost two years (I have switched to a poetry-based workflow for python package but still use pyscaffold for my data science projects) and pyscaffold has undergone some changes during that time, so it might not be compatible with the new API changes. I haven't tested it though.
If you find parts of my code useful, please feel free to use them. The project is permissively licensed (MIT). If you feel your project is ready, I am happy to archive mine and link people to the extension you are developing.
I think it would be great to have a nox extension for pyscaffold.

I am keeping MIT licensing for this plugin.
In general, I was working to get parity with tox for the noxfile.py, there are admittedly a few extras in the noxfile that probably are a bit overboard. I should probably see if they belong in mainline nox.
I think this one is ready... It would be a delight to see if it works for others.

I'll leave the question of archiving and linking up to you, I welcome it but only if you feel this is good enough to warrant it.


Update Readme.md, remove formatting in code

Coverage decreased (-0.3%) to 93.267% when pulling 68ef453a84ca3a243bc59627c41443db28c042bd on strathausen:patch-1 into a8d7388a174427c4a91679a9cf87845663377820 on vpulim:master.


Allow batchsize to be conditional

I'd like to set different batch sizes by conditions.
If I attempt to do this with bicep, I receive an error.

The value must be a compile-time constant. bicep(BCP032)

@batchSize(someParameter == 'none' ? 5 : 1)

However I can write this in ARM, it's totally valid and deploys
        "copy": {
          "name": "storagecopy",
          "count": "[parameters('storageCount')]",
          "batchSize" : "[if(equals(parameters('storageCount'),3),1,2)]",
          "mode" :  "[if(equals(parameters('storageCount'),3),'Serial','Parallel')]"
        }

is there an update to this bug?

@sundarkv and @Gordonby What is the use case that requires conditionally setting batch size?

I am facing the same issue. I am building a solution where I want the end user to set the batch size required when deploying VMs. Would be great if the @batchSize decorator can accept a paramter.


Add Env admin generic object Resource

Regarding: https://github.com/rancher/rancher/issues/10303
Added an envAdminGenericObject resource type which is only accessible by env admin role. It is stored in generic_object table with kind=envAdminGenericObject. EnvAdminGenericObject use generic object as the parent.
To avoid operating envAdminGenericObject via generic object API, a generic object resource manager filter is added(This is because resource data is available from parent type, e.g. given "container,parent=instance", you can get container data in instance API). To avoid database changes and reuse current DB table, kind=envAdminGenericObject is used to differentiate envAdminGO & GO.
@cjellick @yasker

Considering that this solution cannot achieve real isolation from env & envAdm(environment members can still get access from container shell), I'm gonna close this PR.


Implements end-to-end integration tests for Authorization in Rest Layer

Description
This change test Authorization in Rest Layer feature by testing NamedRoutes and legacy Routes. It does so by adding two plugins in security plugin's test framework with one registering NamedRoutes and other registering legacy Routes.

Category: Enhancement

Issues Resolved

Resolves #2928

Testing
Integration tests
Check List

[x] New functionality includes testing
[x] New functionality has been documented
[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.

Not sure why Codecov/project shows 27% decrease when this PR adds more tests.

Backport is currently blocked by: https://github.com/opensearch-project/security/issues/3230

No sure why the codecov/project report shows decrease of .01% when upon clicking the link to codecov it shows an increase of .01%

No sure why the codecov/project/plugin report shows decrease of .01% when upon clicking the link to codecov it shows an increase of .01%

Congrats you've found another source of non-deterministic test results - src/main/java/org/opensearch/security/configuration/ConfigurationRepository.java
I've updated https://github.com/opensearch-project/security/issues/3137, and you can update the codecov.yml to prevent false positives if you'd like

@DarshitChanpura Can you make sure to include this in the everything at once backport for AuthZ in the Rest layer?


[Dev][Archery] Use --arrow-ref instead of --arrow-sha in archery crossbow submit

Describe the enhancement requested
Because --arrow-sha accepts not only SHA but also tag.
See also: https://github.com/apache/arrow/pull/44577#pullrequestreview-2404322965
Component(s)
Archery, Developer Tools

--arrow-head may be a candidate because archery.crossbow.Target uses head: https://github.com/apache/arrow/blob/f5691d467c5638c6f6a07948de82369cc0ea282e/dev/archery/archery/crossbow/core.py#L797


persistence with user change and refresh for money button/ emoji drop

#131 #131 #110 #142

@masters3d Navigator


CAMEL-10340: Adjusted deleteIfFiltered behavior for aws-sqs component

Hello,
Please review my code by fixing CAMEL-10340.
Thanks,
Yi Yan

Thanks! The PR has been merged. Thanks for your contribution @yanbasic :-)
Can you close the PR?


[Backport] [1.x] [BUG] Fix org.opensearch.action.admin.cluster.node.stats.NodeStatsTests.testSerialization

Signed-off-by: Andriy Redko<EMAIL_ADDRESS>Description
Backport of https://github.com/opensearch-project/OpenSearch/pull/1390 to 1.x
Issues Resolved
Closes https://github.com/opensearch-project/OpenSearch/issues/1389
Check List

[ ] New functionality includes testing.

[X] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc 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.

Let's hold merging this PR before https://github.com/opensearch-project/OpenSearch/pull/1391 gets merged.

@owaiskazi19 rebased, thanks!


Add support for excluding list of exact column names

Description
This PR adds support for specifying a list of ExcludeName values to be excluded from classification per info_type.

Hey @ethan-cartwright can you please fix the lint failures ?
The commands are same as main datahub PR and are also mentioned here.

As per customer requirements, all changes looks good to me.


Peaks : Add scaling factor for jump in power when power difference is small.

What type of PR is this?
/kind bug

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

Pull request for scaling power jump value

I0313 09:02:32.454752       1 eventhandlers.go:218] "Update event for scheduled pod" pod="default/cpu-stress-test-65f6675556-bklbq"
I0313 09:02:32.467686       1 eventhandlers.go:126] "Add event for unscheduled pod" pod="default/cpu-stress-test-65f6675556-vzkq4"
I0313 09:02:32.467781       1 scheduling_queue.go:537] "Pod moved to an internal scheduling queue" pod="default/cpu-stress-test-65f6675556-vzkq4" event="PodAdd" queue="Active"
I0313 09:02:32.467851       1 schedule_one.go:80] "About to try and schedule pod" pod="default/cpu-stress-test-65f6675556-vzkq4"
I0313 09:02:32.467883       1 schedule_one.go:93] "Attempting to schedule pod" pod="default/cpu-stress-test-65f6675556-vzkq4"
Node : tantawi1 , Node cpu usage current : 67.3 , predicted : 73.54999999999998
Jump in power 7.64162509601929e-85  score -9223372036854775808
Node : tantawi2 , Node cpu usage current : 4.47 , predicted : 5.72
Jump in power 0.00037766007893825886  score 2133796
Nodescores : [{Name:tantawi2 Scores:[{Name:Peaks Score:0}] TotalScore:0} {Name:tantawi1 Scores:[{Name:Peaks Score:0}] TotalScore:0}]
I0313 09:02:32.468744       1 default_binder.go:53] "Attempting to bind pod to node" pod="default/cpu-stress-test-65f6675556-vzkq4" node="tantawi2"
I0313 09:02:32.471793       1 eventhandlers.go:171] "Delete event for unscheduled pod" pod="default/cpu-stress-test-65f6675556-vzkq4"
I0313 09:02:32.471825       1 eventhandlers.go:197] "Add event for scheduled pod" pod="default/cpu-stress-test-65f6675556-vzkq4"
I0313 09:02:32.471894       1 scheduling_queue.go:1120] "Event received while pods are in flight" event="AssignedPodAdd" numPods=1
I0313 09:02:32.471965       1 round_trippers.go:553] POST [https://<IP_ADDRESS>:443/api/v1/namespaces/default/pods/cpu-stress-test-65f6675556-vzkq4/binding](https://<IP_ADDRESS>/api/v1/namespaces/default/pods/cpu-stress-test-65f6675556-vzkq4/binding) 201 Created in 2 milliseconds
I0313 09:02:32.472080       1 cache.go:404] "Finished binding for pod, can be expired" podKey="fe0853c6-7bda-45c3-b051-5ffd74d1599a" pod="default/cpu-stress-test-65f6675556-vzkq4"
I0313 09:02:32.472126       1 schedule_one.go:286] "Successfully bound pod to node" pod="default/cpu-stress-test-65f6675556-vzkq4" node="tantawi2" evaluatedNodes=2 feasibleNodes=2
I0313 09:02:32.477854       1 round_trippers.go:553] POST [https://<IP_ADDRESS>:443/apis/events.k8s.io/v1/namespaces/default/events](https://<IP_ADDRESS>/apis/events.k8s.io/v1/namespaces/default/events) 201 Created in 5 milliseconds
I0313 09:02:32.486858       1 eventhandlers.go:218] "Update event for scheduled pod" pod="default/cpu-stress-test-65f6675556-vzkq4"
I0313 09:02:33.658544       1 client.go:94] received status code 200 from watcher
&{1710320521 {15m<PHONE_NUMBER><PHONE_NUMBER>} KubernetesMetricsServer {map[tantawi1:{[{ CPU Latest  67.3} { Memory Latest  5.350725243277423}] {} {}} tantawi2:{[{ CPU Latest  4.47} { Memory Latest  1.5227247946536684}] {} {}}]}}

Here the jump in power for tantawi1 is less than tantawi2. However, pod is scheduled on tantawi2  (since both these values are rounded to 0 by the score computation function later).
Workaround:
int64 	A 64-bit signed integer. It has a minimum value of -9,223,372,036,854,775,808 and a maximum value of 9,223,372,036,854,775,807 (inclusive).
Our max value of jump in power can be <1000 most of the time (which depends on the node configuration).
So, we can multiply the jump in power (which is defined as int64) with 10^6, they won't be rounded to 0 by the score computation function later.
This fix resulted in the scheduler picking tantawi1 in the above example for which the log is shared above.
Note: Even through we don't know the increase in power for other node's pod while calculating the power for current node's pod, this workaround helps to address this issue since we are multiplying the jump in power for the pod on any node with the same value (hence the comparison is not affected).


App crash if I unmount the component faster than the end of the animation

Hi,
Thanks for that amazing component.
I found a bug which make my app crash if I navigate too fast.
The bug is here:
https://github.com/deanmcpherson/react-native-sortable-listview/blob/master/index.js#L449

  componentDidMount() {
    InteractionManager.runAfterInteractions(() => {
      setTimeout(this.measureWrapper, 0)
    })
}

If I go too fast and the component is Unmounted before the end of the animation, (or just at the end), this.measureWrapper is called on an unmounted component and this.refs is undefined.

Thanks for your report, can you verify if my fix works?

@nihgwu It works ! Thanks a lot for your reactivity.


How to get the hyperparameters?

Thanks for you wonderful work. I have trying inferring using your code, but it works  not good. I'm wondering how to find the best hyperparameters? Have you implemented the learning procedure?

I have not implemented the learning procedure, but am happy to accept PRs implementing it, as mentioned in the README.
Besides that, finding hyperparameters is an open question in ML, so nothing I can help with in a github issue :smile: Use a validation-set and do search.


[Bug] Tooltip in radar shows '-' if axisName.show = false

Version
5.3.1
Link to Minimal Reproduction
No response
Steps to Reproduce
Earlier I reported #15915 and it was fixed for radar.name.show = false. I tried this with 5.3.1 (as it was fixed in 5.3.0) and noticed the same issue exists when radar.axisName.show = false.
Current Behavior
Setting axisName.show = false shows '-' in the tooltip for the axis names.
Expected Behavior
The tooltip should show the actual name.
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
Note that the reason why I am trying to disable the axisName is I display a bunch of radar charts together and I don't want the axis names showing up and overlapping and creating clutter.

Actually it seems like #15915 didn't get fixed as I am able to reproduce it with the exact steps given there.
However, I noticed that according to the documentation the radar option doesn't have a "name" node. It has "axisName" node. So, perhaps, the fix should take care of axisName.show being false and the tooltip still showing the names.

Seems to be a bug.

@dirslashls I still want to confirm if it works for you in v5.3.0 as I tested it in 5.3.0, it can't work.

@plainheart , it didn't work in 5.3.0 or 5.3.1.

Thanks for your feedback. We just fixed it in #16764 and sorry for this. I'm afraid you have to wait for the next release v5.3.3.

@plainheart, no worries. Thanks for taking care of he fix.


Update config.yml

I just have users manually change the directory here.

looks like the variables defined in congif.yml have changed. I assume this would require corresponding changes to the Snakefile. Is that the case? If so, would be good to include those changes (as a commit) in the same PR. Please let me know!

It looks like the Snakefile changes are in their own pull request: https://github.com/dunhamlab/yevo_pipeline/pull/10
Merging either one of these alone would break the pipeline but merging both should fix.


No Types for the directive or mixin

Please add type definitions for the directive and mixin.

@Kaelten Open for PRs for this addition.


OpenSSL 3.0.0-beta-1 ACVP GCM Internal IV generation Queries

"algorithm": "ACVP-AES-GCM",
"revision": "1.0",
"isSample": true,
"testGroups": [
{
"tgId": 1,
"testType": "AFT",
"direction": "encrypt/decrypt",
"keyLen": ...,
"ivLen": ...,
"ivGen": "internal",
"ivGenMode": "8.2.1",
"payloadLen": ...,
"aadLen": ...,
"tagLen": ...,
"tests": [
{
"tcId": 1,
"pt/ct": "",
"key": "...",
"aad": "...",
"iv": ""(for decryption only)
}
]
}
]
As the Generation for IV is internal and IV is also an input for decryption so do we need to generate a new IV?
what API should I use for Generation of IV for ACVP in GCM?

Is the answer you seek in the acvp test?
https://github.com/openssl/openssl/blob/master/test/acvp_test.c
This file is a demonstration for the FIPS lab of the way to do what's required.  It includes GCM testing.

acvp_test.c contains GCM test in which IV is external.
I wanted to know how to generate a new IV.
and In case ivGen is internal do we need to generate a new IV for decryption as input test vectors also contains IV as input

A different IV will change the encryption, so you must decrypt with the same IV as was used for encryption.
If you don't set an IV, one will be generated as part of the encryption.  Read and save this to decrypt.

And which API can I use for new IV generation for encryption

You don't call any API.  The IV is generated automatically.


Data Attribute Selector

More of a question than an issue, but why use "$=" for the attribute selectors instead of "=".

In prep for 1.7, which will include all of the states.


attempted use of backend-webrtc-rs fails

Trying to use backend-webrtc-rs instead of backend-go-pion but it won't compile.
This is running cargo build --release --target aarch64-apple-ios, but maybe it also happens for normal targets. Not sure yet, haven't checked.
error[E0433]: failed to resolve: use of undeclared crate or module `imp`
  --> /Users/connor/.cargo/registry/src/github.com-1ecc6299db9ec823/tx5-0.0.1-alpha.17/src/back_buf.rs:13:10
   |
13 |     imp: imp::ImpWriter,
   |          ^^^ use of undeclared crate or module `imp`

error[E0433]: failed to resolve: use of undeclared crate or module `imp`
  --> /Users/connor/.cargo/registry/src/github.com-1ecc6299db9ec823/tx5-0.0.1-alpha.17/src/back_buf.rs:22:18
   |
22 |             imp: imp::ImpWriter::new()?,
   |                  ^^^ use of undeclared crate or module `imp`

error[E0433]: failed to resolve: use of undeclared crate or module `imp`
  --> /Users/connor/.cargo/registry/src/github.com-1ecc6299db9ec823/tx5-0.0.1-alpha.17/src/back_buf.rs:65:21
   |
65 |     pub(crate) imp: imp::Imp,
   |                     ^^^ use of undeclared crate or module `imp`

error[E0433]: failed to resolve: use of undeclared crate or module `tx5_go_pion`
  --> /Users/connor/.cargo/registry/src/github.com-1ecc6299db9ec823/tx5-0.0.1-alpha.17/src/back_buf.rs:76:33
   |
76 |     pub(crate) fn from_raw(buf: tx5_go_pion::GoBuf) -> Self {
   |                                 ^^^^^^^^^^^ use of undeclared crate or module `tx5_go_pion`

error[E0433]: failed to resolve: use of undeclared crate or module `imp`
  --> /Users/connor/.cargo/registry/src/github.com-1ecc6299db9ec823/tx5-0.0.1-alpha.17/src/back_buf.rs:78:18
   |
78 |             imp: imp::Imp::from_raw(buf),
   |                  ^^^ use of undeclared crate or module `imp`

error[E0433]: failed to resolve: use of undeclared crate or module `imp`
  --> /Users/connor/.cargo/registry/src/github.com-1ecc6299db9ec823/tx5-0.0.1-alpha.17/src/back_buf.rs:87:18
   |
87 |             imp: imp::Imp::from_slice(slice)?,
   |                  ^^^ use of undeclared crate or module `imp`

error[E0433]: failed to resolve: use of undeclared crate or module `imp`
   --> /Users/connor/.cargo/registry/src/github.com-1ecc6299db9ec823/tx5-0.0.1-alpha.17/src/back_buf.rs:102:18
    |
102 |             imp: imp::Imp::from_json(s)?,
    |                  ^^^ use of undeclared crate or module `imp`

error[E0425]: cannot find value `on_new_conn` in module `endpoint`
   --> /Users/connor/.cargo/registry/src/github.com-1ecc6299db9ec823/tx5-0.0.1-alpha.17/src/config.rs:320:55
    |
320 |                 .unwrap_or_else(|| Arc::new(endpoint::on_new_conn));
    |                                                       ^^^^^^^^^^^ not found in `endpoint`

@Connoropolous - webrtc-rs isn't implemented at all, I just wanted to make sure the feature switching logic was in place to ensure the availability of that path when it's ready.
There are some issues I'm keeping my eye on:

https://github.com/webrtc-rs/webrtc/issues/426
https://github.com/webrtc-rs/webrtc/issues/103
https://github.com/webrtc-rs/webrtc/issues/119

In general the lib doesn't feel ready for prime-time. There are anecdotally many performance issues with it, and lots of churn / immature aspects that just need some time to sort out.
That being said, if someone had time, we do want to see it implemented so we can start to get some apples-to-apples comparisons and be able to make an informed decision about when we/it are ready to switch to as a default.

I see. Ok, thanks.
Do you have any immediate ideas or perspective on the apparent tx5 panic mentioned here?
https://hackmd.io/_CFnGaWvQg6Ys3DxZblmug#May-28
It seems the next step is to track it down

It seems likely to stem from an unwrap as there's no error specific error message, true?
https://github.com/search?q=repo%3Aholochain%2Ftx5 .unwrap&type=code

@Connoropolous - oo, that actually looks like a panic in the GO code... unfortunately both languages use that term.
I have a highest level panic catch in the go lib, but for this case I think I might need to add a lower-level one so the error can be returned with the more relevant call. Well... I'll look into it in any case, and create a new issue for this.
Thanks for finding all these issues!!

Ok fantastic.

if you need anything to construct steps to reproduce, let me know

https://github.com/holochain/tx5/issues/40

@Connoropolous - If it's okay, I'll leave this open for actually implementing the webrtc-rs backend. I changed the title : )

Ok cool

@neonphog I have no idea whether we'd be up for the challenge of this at all, without a sense of the scope... can you give me an impression of the work involved in doing this?

is it kind of like implementing the equivalent of the things that are marked with the backend-go-pion feature?
https://github.com/search?q=repo%3Aholochain%2Ftx5 backend-go-pion&type=code

just cause I was curious, I tried something, can you check if its in the right direction.
#41


[HUDI-390] Add backtick character in hive queries to support hive identiter as tablename

What is the purpose of the pull request
This pull request adds support for keyword as tablename in hive sync
Brief change log

Add back tick character to escape keywords in hive queries

Verify this pull request
This pull request is a trivial rework / code cleanup without any test coverage.
Committer checklist

[x] Has a corresponding JIRA in PR title & commit

[x] Commit message is descriptive of the change

[x] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

thanks for reviewing @leesf :)


Porting Whisper model to n300

[x] Measure and record current performance.
[x] Rebase the model to main, ensure the PCC = 0.99
[x] Port functionality to n300 card (single device)
[x] Provide Op Report
[x] Check Model into CI
[x] Implement Demo
[ ] #13400

Whisper Model is ported to n300 with the following torch ops:

Slice op - #13672
Add op - #13464
Conv1d op - #13529

Model Pcc is 0.989.
Demo Results :

Accuracy for WhisperForAudioClassification - 0.75
Accuracy for WhisperForConditionalGeneration - 0.73

Single Card Demo Pipeline - #8339


Automatically delete spark-warehouse folder or set its location on config file

sparklyr creates a folder named "spark-warehouse" in working directory during read process of CSV files, for instance. Is there any way to automatically delete spark-warehouse folder after reading process or set its location on config file?
Thanks.

This is reasonable, marking as feature request.
Currently we only set the warehouse directory for windows OS, see https://github.com/rstudio/sparklyr/blob/ff105bcf10378c683c116480256404316f9aed2a/R/install_spark.R#L196. Reason being that the default path is c:\tmp\hive which causes install issues for users with no permissions to work outside their user folder. I think a good fix here would be to expose the hive-site.xml configuration in the sparklyr config.yml file with a prefix of "hivesite." and let users set this to a different location as well as override other hive-site.xml configuration settings


Replace video URL with the playlist URL for the series 30 minutes of code

Thank you!

Thanks. I am not sure if the script handles playlist well. But let's see

Thanks. I am not sure if the script handles playlist well. But let's see

If it does not and you're OK with it, I can open a new PR every time a new episode is live!


Trying to upload a file using . and drive root as destination fails with 500 error

Describe the bug
While trying to upload content to paths . and drive root, the server return 500 Internal Server Error
Steps to reproduce
curl -XPUT "https://localhost:9200/remote.php/dav/spaces/<drive-id>" -d"lorem" -uadmin:admin -vk
curl -XPUT "https://localhost:9200/remote.php/dav/spaces/<drive-id>/." -d"lorem" -uadmin:admin -vk

< HTTP/1.1 500 Internal Server Error

Expected behavior
405 or 400?
Actual behavior
500 Internal Server Error

TODO: Cleanup the expected failures in a reva  https://github.com/cs3org/reva/pull/4768/files#diff-d06f0265b57142c2bbad97953c887323699f80e366ee85ea9173c241345c1d41

Cleanup the expected failures in reva

That should happen when the next ocis test commit id bump happens to reva (PR like https://github.com/cs3org/reva/pull/4770 )
@saw-jan @Salipa-Gurung it would be nice to get someone to do the commit id bump in reva, since there are expected-failures that should be able to be removed.

https://github.com/cs3org/reva/pull/4773

https://github.com/cs3org/reva/pull/4773 passed and merged.


Bump rails html sanitizer and parser

For: https://trello.com/c/X6CWuGOX/134-rails-security-vulnerability-sanitize-463-libxml-292
Bump rails-html-sanitizer for CVE-2018-3741 and parser to avoid using a yanked version.

Looks good to me, @h-lame.
I'd been lazy and included the bump to parser in https://github.com/alphagov/asset-manager/pull/523 but I'll remove it from there once this is merged :-).


Updating SDK versions to 2.12.7

This updates the AWS_SDK_VERSION to 2.12.7 across all Podfiles. Currently the process is manual and in the future, heard that @kneekey23 may be picking up something down the road to automatically do this.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I see we're using "compatible with" (~> 2.12.7) in some cases and "exact match" in others. We should be consistent with our usage to avoid unexpected surprises down the road. In particular, the "compatible with" flag means we don't have to manually update the version on patch version upgrades to the SDK, as long as we do a pod update and commit the resulting Pods & Podfile.lock

it looks like they are using "~>" in all cases, for example when we define the constant AWS_SDK_VERSION, it is being used as "~> AWS_SDK_VERSION"

this PR is outdated now and we can update to take ~> 2.13.0 to get all patch versions of 2.13. a minor version will last a few months


feat(ac-markdown)Support markdown for adaptive cards

Description of changes made (Required)

Added  function for handing markdown by using an existing function
Added unit test

https://www.npmjs.com/package/adaptivecards#supporting-markdown
Associated issue number/tracking link (Required)
94103
The following criteria should be met (Required)

[X] commit guidelines were followed
[X] Tests have been added for new functionality and all existing tests pass.
[X] pushing guidelines are followed

Reviewers (Required)
@InteractiveTimmy @akoushke @adamweeks

@mokandre could you please rebase this on the latest master as well, we have some pipeline changes that need to take effect.


Sub menu query fails: [Error 299] Only part of a ReadProcessMemory or WriteProcessMemory request was completed.

When I try accessing items from a submenu, the query fails. I've tried the following on both 32-bit and 64-bit Python versions. The application is 32-bit, and I'm using a Japanese locale. The Windows version is 8.1 64-bit.
import time
from pywinauto.application import Application

app = Application().connect(path="rpg2000.exe")
menu= app.TFormLcfMain.menu_item("Project")
time.sleep(1)
smenu = menu.sub_menu()
time.sleep(1)
print(smenu.items())

Traceback (most recent call last):
  File "rpg.py", line 14, in <module>
    print(smenu.items())
  File "C:\Program Files\Python36\lib\site-packages\pywinauto\controls\menuwrapper.py", line 366, in __repr__
    return "<MenuItem " + self.text() + ">"
  File "C:\Program Files\Python36\lib\site-packages\pywinauto\controls\menuwrapper.py", line 223, in text
    mem.Read(s, address)
  File "C:\Program Files\Python36\lib\site-packages\pywinauto\remote_memory_block.py", line 263, in Read
    raise ctypes.WinError()
OSError: [WinError 299] Only part of a ReadProcessMemory or WriteProcessMemory request was completed.
> python
Python 3.6.1 (v3.6.1:69c0db5, Mar 21 2017, 18:41:36) [MSC v.1900 64 bit (AMD64)] on win32

Hmm... Probably this menu can't be automated using Win32 API.
Did you try Application(backend='uia')? MS UI Automation should be more reliable for such a menu.


ASSERTION_FAILED: ZCL_ABAPGIT_GUI_PAGE_CODE_INSPCP #1912

zcl_abapgit_gui_page_code_insp "catches all actions" in zif_abapgit_gui_page~on_event with WHEN OTHERS.
abapgit_home must be ignored
didn't finde any other page class with this behaviour.

#1912

#1912


feat(@angular-devkit/schematics): add UpdateBuffer2 based on magic-string

This PR adds UpdateBuffer2 which should eventually replace UpdateBuffer. UpdateBuffer2 internally uses the magic-string library.
UpdateBuffer and related symbols have been marked as deprecated.
Closes #21110

This is an initial implementation of the magic-string changes discussed in #21110. I'm open to suggestions for improvement.
Also while this is not a breaking change to the API, it does change internal behavior (see changed unit tests). Would this be considered a breaking change?

Thank you for the contribution.
I'll do a more thorough review soon but I wanted to address the breaking change question. Looking at the changes to the tests, this would indeed be breaking.  As-is, this would need to go into a major version (v13 being the next in Fall 2021).  To get this in sooner, both modes would need to be supported and controlled via an option.  This option could potentially be exposed via beginUpdate but if there were a more appropriate place that would work as well.  This would allow the current behavior to remain the default for 12.x but still allow the new behavior for those who need/prefer it.  The default could also potentially be swapped in v13+ and then followed by an eventual removal.

This fix is not urgent for me, so landing it in v13 would be fine by me. I can change the deprecation notices accordingly.
As for providing it side-by-side; A possibility would be to pass a parameter through NodeWorkflowOptions => BaseWorkflowOptions => HostTree => beginUpdate.
I'm blanking a bit on the potential names. useMagicString for parameter and UpdateRecoderMagicString for the UpdateRecorder implementation?

For the option name, preference would be to treat magic-string as an implementation detail and keep it encapsulated if possible.  Maybe a name such as compatibilityMode or similar (open to other ideas) with a default to true which would use the existing methodology.

An alternative for the configuration may be an environment variable (e.g. NG_LEGACY_UPDATE_RECORDER, NG_SAFE_UPDATE_RECORDER or simply NG_UPDATE_RECORDER_V2), as there are already precedents like NG_PERSISTENT_BUILD_CACHE.
This would avoid the need for the propagation.

@clydin I have updated the implementation. I have reverted previous changes to the existing UpdateBuffer and created an additional UpdateBuffer2, which now both extend UpdateBufferBase. I have implemented a switch based on the environment variable NG_UPDATE_BUFFER_V2.
I'm still very much open to suggestions for improvement.

@clydin I don't know whether the change to the goldens is acceptable in this case. I can revert the changes to HostSink, if it is a problem.

@clydin No problem (in regards to the delay). I have rebased the branch.
One question; Should I switch to usage to default to UpdateBuffer2 (e.g. rename NG_UPDATE_BUFFER_V2 to NG_LEGACY_UPDATE_BUFFER and adapt accordingly) or would you like to continue using UpdateBuffer for v13?

Let's leave the default with the existing behavior for now.  We may switch it for v13 but we would like to do some analysis regarding the magnitude of breakage before changing it.
Thank you again for the contribution.

Thank you for the review and other feedback. 😃


Serialize ENHS, ENHC

addressing #269

thanks, looks good, I'll do a few tests tomorrow and then merge if successful

hi,
While doing a pull from repository
https://github.com/larshp/CLAS
I get an error: "Error occured while checking ENHS: Enhancement object ENHS ZSPOT cannot be read", in the exists method:
  METHOD lif_object~exists.
    DATA: lv_spot_name  TYPE enhspotname,
          lv_message    TYPE string,
          li_cx         TYPE REF TO cx_root,
          li_spot_ref   TYPE REF TO if_enh_spot_tool.

    lv_spot_name = ms_item-obj_name.

    TRY.
      li_spot_ref = cl_enh_factory=>get_enhancement_spot(
                    spot_name = lv_spot_name ).
    CATCH cx_enh_root INTO li_cx.
      lv_message = `Error occured while checking EHNS: ` && li_cx->get_text( ).
      _raise lv_message.
    ENDTRY.

it looks like exception CX_ENH_IO_ERROR should be caught and return 'rv_bool = abap_false.'
can you have a look?

merging, will track issues in #269

Hi Lars, sorry for silence - touch schedule on my project began. My collegue said that the issue was because of param dark = Х, but I see that you fixed already. Sorry for late responce.
P.S. abapmerge is awesome ;)

no problem :smile:
I did not test all corners of ENHS/ENHC, but I think a few issues has been solved


feat:DocsGPT-Telegram-Bot

Feature: Adds Telegram bot extension to enable users to interact with DocsGPT via Telegram.

This change was needed to provide users an additional platform to interact with DocsGPT. By adding Telegram bot functionality, users can easily ask documentation-related questions and receive answers directly in their Telegram app. This enhancement improves accessibility and user experience.

This PR was initially submitted for Hacktoberfest, but throughout the process, I grew fond of the project and hope to contribute to it in the future as well.

Im am really looking forward to you other contributions! Please create an issue before PR next time, one of our team members will help you and warn you of anything specific if needed.

Im am really looking forward to you other contributions! Please create an issue before PR next time, one of our team members will help you and warn you of anything specific if needed.

oh my bad! give me some time to check how slack extensions work and I will create the issue soon. :)


Crash with Custom Traits and Potion Core

Not sure who's at fault but I figured I would report it here because I made the trait using content tweaker.
here's the trait: https://pastebin.com/GmWdHPVJ
here's the crash report: https://paste.dimdev.org/waqisekucu.mccrash

I'd rather think either target or attacker is null, but in any case, you could check using isNull(attacker) for example.
SkySom
If what you said where true he'd gotten a warning in his CrT log about not being able to resolve the bracket (which may be true, since he did not include his CrT log)
But since the Bracket handler is nothing but a method call and this function is executed during the game I don't think that's the cause (unless he misspelled the bracket)


[Idea] urn of shadows / spirit vessel

Can we implement a considering when bot want to buy spirit vessel / urn of shadow?
If player have already bought a urn of shadow, the charge will be granted to the nearest hero. This will make either player / bot doesn't get any charge and the item will be unusable
Could you please show me where and how can i implement this? I would be happy to create the PR after i know how to make an exception on item buy list

Modifying item purchasing logic can easily cause problems if you are not familiar enough with the code base of how any changes can impact different heroes unexpectedly.
I'd suggest look into the item_purchase_generic file first, get familiar with it. From there you can add logic/strategites to check if any items on the list of items the bot is trying to purchase have been purchased by any other ally heroes, and then replace that item to a different item.
So you will need to know what to replace the urn for that won't cause any potential issues later like you don't want the bots to purchase another redundant item.
Also you need to know the bot may be half way through completing the urn or such items so if you stop the bot from buying other components of the item, the bots will end up having junk component items on its item slots and block the bot from buying other things due to slots get full quickly. So you need to know what items are good to sell in case you replaced an item in the middle of the item build. Also note the item build is not static - the components can change in newer Dota versions so please do not hardcode any components to sell.

I see... That makes sense, but i can learn about how the item bought by bot with this issue
Thank you! I will close this issue

I was trying to bring up my concerns for potential discussion and yes I don't personally have the plan to do this implementation. As my focus is to make a better bot team not for bots to work seamlessly with human players in the same team - they can almost NEVER satisfy a human player that plays bot games since human players will keep asking more and more from bots: help stacking creeps, dynamic roaming, use all abilities to assist the kill not stealing any kills, die for human, dynamically warding for human in all good spots possible, help human to get more time to farm, help human to do Roshan etc when human feel it's the time, help, help to turn the game around even when bots give up all resources for human but still not seem to win the game.
When working with a partner, some human players start to think the partner should do excellent things like a really pro player. Just like in ranked games core players don't think their supports are doing good enough, and supports complain about their noob cores. So we know even a pro player cannot satisfy some players, I don't want to bother spending the effort to do the improvement now.
But I do think it's possible to do, just require fair amount of effort.

Hmm i got your point there, so we should improve this bot script by functionality rather than making it like a human player
Is there any things on your mind that i can contribute here? Maybe some minor / intermediate issues to make me understand this script more

First let me apologize for hold my personal opinion too hard on the suggestion you made, it's a pretty good idea.
I believe you have the most motivation to do things if you are already interested about it, so please don't directly stop thinking about that idea, and don't hesitate to ask me for more details when you try to ramp up on the item purchase logic.
Things can be improved gradually. So you can start by doing relatively easier changes to the code by checking if the team has more than one same items, then purchase all alternative like purchase something similar either with healings or other common items.
I think if we make this logic in a good way, this can also apply to the case where we can make dynamic-ish decisions for bots to purchase/replace items that counter the other team better like MKB vs BF/PA.
I'm not sure how much experience you have for coding in general, but just a reminder this isn't a simple change and it will/can be quite interesting/extendable for bot performance in general. So I'm glad you showed interests on this and please don't give up on your ideas.

No need to apologise, i know your good intentions to keep this code efficient, i appreciate and understand completely.
In my opinion, there is no similar item on this game, that's why i don't see any solution to this item thinking. Like if we want to replace the urn / vessel with other items, there are too many things to be considered

If we want to replace it with dagon, we should consider is dagon already on the build list? Is dagon compatible with the hero?
If we want to replace it with silver edge (let's say vessel can counter DK's regen, so the alternative is break), we should consider is the hero physical / magical? Is it already on build list?
Or we should replace it with refresher orb to burst down the regen of the target? Does the bot need burst the target or does the bot need a DoT from the urn / vessel (like Invoker's cold snap)?

It would create endless possibilities and it's not worth to code when there are many simpler things to be done.
Even buying MKB vs Butterfly / PA / Radiance is a little bit complicated in my head, what item should be replaced with MKB? when should the bot buy MKB? Is MKB worth it to be bought (consider buying alternatives like silver edge / bloodthorn)?
I need a huge helps if we want to code this dynamic item thinking, i don't have much experience on LUA files but i code as a job so adapting is not that hard
With a new major patch (and new hero) on the horizon, i think we should focus on that thing instead of this itemization.
Maybe if you have some spare times, could you please create some tickets or milestones here so that i can pick a task to work on it? Right now i don't see any major problems on the script, but maybe you know some bugs and need to be helped, i'm willing to help.

I added a bunch of [Feature request] in Issues. Please take a look which one you want to help with.


store session in signed/encrypted cookies

This crate is suppose to replace the axum-sessions crate - which is now deprecated. But as far as I can tell, axum-sessions supports storing sessions in signed cookies (https://docs.rs/async-session/latest/async_session/struct.CookieStore.html), but this library does not.
My axum application does not have any kind of database, and uses oidc to authenticate. Adding a database just to keep track of the login would appear to be an overkill.
At the present, it would appear that the MemoryStore is my only solution, but presumably that will lose all sessions on restarting my program.

tower-sessions doesn't provide signing because no data is stored in the cookie. In other words, the cookie value is a pointer to the data stored server side.
MemoryStore is not intended to be used in production. Instead, for an in-memory cache, I would suggest MokaStore. If you need persistence then you can use a database or implement SessionStore for any arbitrary persistence method.

I think to accommodate this use case, the save() function in the SessionStore trait would have to return at least the session identifier (or an Option wrapping one), and for the middleware to use the Set-Cookie header whenever the implementation returns such a session identifier
That would allow a client-side storage implementation to fit the trait, by serialising and signing the session data into a new identifier

That would allow a client-side storage implementation to fit the trait, by serialising and signing the session data into a new identifier

I think this is in conflict with a core design consideration of tower-sessions: it's intentional that the cookie doesn't carry session data with it.

There are use cases where the design and operation of the server-side is much simpler if client-side sessions are a possibility, i.e. if there is no DB and having server-side sessions is the only reason a DB would be needed
But, it's understandable if such use cases are firmly out of scope for this project

i.e. if there is no DB and having server-side sessions is the only reason a DB would be needed

SessionStore needn't be implemented exclusively for a database. For instance, Django provides a file store. The key capability is loading and saving and so long as you can do that then your store can be anything you like. Redis might be a reasonable compromise for folks in this situation: it should require less ops overhead (hopefully) compared to a fully-fledged RDBMS which you otherwise don't need.
Returning to Django for a moment, it does provide serialization to the cookie, but it's discouraged: it represents a serious security risk and comes with other caveats, like cookie size. Looking back on axum-sessions, I think it was a mistake (and at least one other person agreed) to encode data in the cookie. I understand that it can be convenient and there may be use cases where it's absolutely fine to do so, the trouble is it's not generally the right solution.

For the record, the Elixir session frame work recommends storing sessions in cookies, and it is the default: https://hexdocs.pm/phoenix/1.3.0-rc.1/sessions.html#cookies
If this is a security issue or not, not sure. I guess it depends on what the session is used for, and what the consequences would be if an attacker got access. e.g. the risk of somebody stealing a not-expired token from a logged out session in order to access my web site that allows turning on/off my lights is pretty remote IMHO.
But perhaps it is risky that the Pheonix docs don't even mention any of the limitations of the default method. So I could imagine people are using it inappropriately without realising. e.g. if I logged out of my bank on a shared computer, I would be somewhat stunned and surprised if somebody could still access my bank account using the already obtained token.
Regardless, I plan to upgrade my app to use a database. At least attempt to do so :-). If this bug report is closed with "working as designed" I would be slightly sad, but this is OK with me.

I haven't really looked at this closely, but async-session implemented a cookie store. We might be able to do the same. (I'm not sure it should be baked into this crate, again regarding security.)


[Chat] Stabilizing Inline Image and Image Gallery and keep File Sharing in beta

What

Stabilizing Inline Image and Image Gallery
Separate file sharing from inline image
Update storybook and remove beta banners
Update SDK dependencies
Make Enum changes
Why

How Tested

Process & policy checklist

[ ] I have updated the project documentation to reflect my changes if necessary.
[ ] I have read the CONTRIBUTING documentation.

Is this a breaking change?

[ ] This change causes current functionality to break.

Under /workspaces/communication-ui-library/common/config/babel/features.js I think we can remove the teams-inline-images-and-file-sharing feature flag.
And update /workspaces/communication-ui-library/packages/react-composites/tests/app/chat/FakeAdapterApp.tsx to stop using that feature flag and use the file-sharing flag instead


apic instance chart

This looks like a good starting point.
One of the recent developments has been to use Helm Parameters to override specific values in a chart similar to https://github.com/cloud-native-toolkit/toolkit-charts/tree/main/stable/ocp-subscription.  We should look to parameterize the different options assuming that the current set of configurable values within values.yaml is not an extensive list.


Subject line has to start with an upper case letter

Hello,
i have a problem with CaptainHook.
When i commit i get the information
Subject line has to start with an upper case letter
What is wrong
Your sincerly
Stephan

The first line of the commit message must start with an uppercase char A-Z


Error while setting up hvac_group platform for climate

System Health details
System Information

version
core-2024.1.0

installation_type
Home Assistant OS

dev
false

hassio
true

docker
true

user
root

virtualenv
false

python_version
3.11.6

os_name
Linux

os_version
6.1.70-haos

arch
x86_64

timezone
Australia/Brisbane

config_dir
/config

Home Assistant Community Store

GitHub API
ok

GitHub Content
ok

GitHub Web
ok

GitHub API Calls Remaining
4754

Installed Version
1.33.0

Stage
running

Available Repositories
1390

Downloaded Repositories
43

HACS Data
ok

Home Assistant Cloud

logged_in
false

can_reach_cert_server
ok

can_reach_cloud_auth
ok

can_reach_cloud
ok

Home Assistant Supervisor

host_os
Home Assistant OS 11.3.rc2

update_channel
beta

supervisor_version
supervisor-2023.12.1

agent_version
1.6.0

docker_version
24.0.7

disk_total
125.3 GB

disk_used
66.0 GB

healthy
true

supported
true

board
ova

supervisor_api
ok

version_api
ok

installed_addons
Samba share (12.2.0), Filebrowser (2.23.0_10), AdGuard Home (5.0.1), Advanced SSH & Web Terminal (17.0.1), FTP (4.7.3), Firefox (1.0.7), Gitea (1.21.3), Guacamole Client (testnoupload-4), Home Assistant Google Drive Backup (0.112.1), MariaDB (2.6.1), Node-RED (16.0.2), Rclone Backup (3.0.1), ResilioSync (2.7.3-12), Simple Scheduler (2.2.1), Syncthing (1.17.0), wgeasy (9), ESPHome (2023.12.5), TasmoAdmin (0.28.0), Mosquitto broker (6.4.0), Uptime Kuma (0.10.1), Zigbee2MQTT (1.35.0-1), Terminal & SSH (9.8.1), Home Panel (2.7.0), Apache2 Minimal with MariaDB Client (2.7.8)

Dashboards

dashboards
2

resources
28

views
5

mode
storage

Recorder

oldest_recorder_run
December 26, 2023 at 7:37 PM

current_recorder_run
January 4, 2024 at 9:26 AM

estimated_db_size
350.90 MiB

database_engine
sqlite

database_version
3.41.2

Checklist

[X] I have enabled debug logging for my installation.
[X] I have filled out the issue template to the best of my ability.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] This issue is not a duplicate issue of currently previous issues..

Describe the issue
Hello. Thanks for making this. I'm keen to give it a whirl. I've tested it on HA 2023.12 and 2024.01 and it doesn't work on either.
I set up the HVAC Group helper and it then shows a red exclamation icon next to it. Checking log shows errors. No climate entity is created to control the group.

Reproduction steps

Install integration via HACS
Set up helper
...

Debug logs
DEVICE_CLASS_TEMPERATURE was used from hvac_group, this is a deprecated constant which will be removed in HA Core 2025.1. Use SensorDeviceClass.TEMPERATURE instead, please create a bug report at https://github.com/tetele/hvac_group/issueshttps://github.com/tetele/hvac_group/issues

Logger: homeassistant.components.climate
Source: helpers/entity_platform.py:360
Integration: Climate (documentation, issues)
First occurred: 9:31:05 AM (1 occurrences)
Last logged: 9:31:05 AM

Error while setting up hvac_group platform for climate
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 360, in _async_setup_platform
    await asyncio.shield(task)
  File "/config/custom_components/hvac_group/climate.py", line 109, in async_setup_entry
    heaters=hvac_actuator_entity_ids[CONF_HEATERS],
            ~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^
KeyError: 'heaters'

Logger: homeassistant.config_entries
Source: config_entries.py:406
First occurred: 9:33:09 AM (1 occurrences)
Last logged: 9:33:09 AM

Error setting up entry House Climate for climate
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 406, in async_setup
    result = await component.async_setup_entry(hass, self)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/climate/__init__.py", line 213, in async_setup_entry
    return await component.async_setup_entry(entry)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 188, in async_setup_entry
    raise ValueError("Config entry has already been setup!")
ValueError: Config entry has already been setup!

Logger: homeassistant.components.sensor
Source: helpers/deprecation.py:204
Integration: Sensor (documentation, issues)
First occurred: 9:28:29 AM (2 occurrences)
Last logged: 9:28:29 AM

Diagnostics dump
No response

The main purpose of this was to group together a heater and an AC, so I never thought to allow the usage of only coolers, such as in your case.
Anyway, it's not a complicated fix, expect a new release in the next week or so. Until then, if you want to test things out, you can create a group containing a heater, too. Mind you, an AC unit which can do heating can be both a heater and a cooler, in case you don't have any other climate entities.

Ah I see, that makes sense. Yes my use case is similar to what climate_group offers. But I see now this project is quite different. Very cool what it can do though.

Until then, if you want to test things out, you can create a group containing a heater, too.

I tried that and it's working now, thanks.


Excellent work, when will the code be released?

Great work. May I ask when will the full code will be released?

Hi. Thanks for your interest in our work.
Sorry that I am very busy recently. Hopefully, the code will be released next month.

Hi. Thanks for your interest in our work. Sorry that I am very busy recently. Hopefully, the code will be released next month.

I also appreciate your work very much. Can you update the code if you have time?


templates: fix V2_MetaFunction import

Follow-up of #6005
This import should come from the runtime, not the renderer

I think importing from react is fine here because this is also a client-side concern.  The thing exported from server-runtime is V2_ServerRuntimeMetaFunction.  I'm not sure why it's named differently offhand but it looks close if not identical.  Eventually we want to align some of these duped types so I think I'd vote to hold off on changing this until we do that?

I think importing from react is fine here because this is also a client-side concern. The thing exported from server-runtime is V2_ServerRuntimeMetaFunction. I'm not sure why it's named differently offhand but it looks close if not identical. Eventually we want to align some of these duped types so I think I'd vote to hold off on changing this until we do that?

name difference might be so that it doesn't show up in vscode's autocomplete

@brophdawg11 Before #6005, we always imported this type from the runtime & if I remember correctly the plan was to have an extra package that contains more types/functions like this (for example json & redirect), so react is definitely not the correct place. The most correct place we currently have is our runtimes

I missed that the adapters rename the export (V2_ServerRuntimeMetaFunction as V2_MetaFunction), at first I thought this change wouldn't compile 😬
I don't think we can say react is definitely not the correct place since meta runs both on the server and the client, unlike json/redirect which only run on the server.  So while we do want to dedupe some types eventually I don't think that necessarily means json and meta will land in the same place.
I don't have the background here but as Logan pointed out in #5785 it seems it was a specific choice to import from the react package?  I do know we're considering some API changes to meta/links in v2 (behind a v3 future flag) and these may just be heading in that direction so I'm inclined to leave this as is for now

Closed by #6123


PMM-2639: Use "%%" in place of "%" to specify a single percent sign.

https://www.freedesktop.org/software/systemd/man/systemd.unit.html

Coverage decreased (-0.02%) to 7.838% when pulling 6eea19bbac696e0d2047c52f165effe26772549e on PMM-2639_systemd_ubuntu_bionic into 13b8f7cee7c65569ea0c79878aabed4956416853 on master.


Netherite sub-blocks burn in lava

They need to float above lava just as the Netherite Block they're made from.

fixed


Czech translation for v1.6

PR to adobe:release branch

Now that 1.6 is out I think this is not needed anymore.

@ficristo It is needed for v1.7 (see https://github.com/adobe/brackets/pull/11834#issuecomment-149140183). But I don't understand, why there is a call for translation PRs on Google Groups, if devs finally don't merge some of them.

I think is not needed because you have already #12046 for 1.7.
Am I wrong?

@ficristo yes, you are wrong, #12046 is the same as this, just into master branch (this is into release branch).

@dvorapa Thanks for the change. Apologies for not merging this for 1.6.
Closing in favor of https://github.com/adobe/brackets/pull/12046 .


Add ability to configure SwaggerJson via params

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
✔/❌

Fixed issues
-

@rustamwin need a line for CHANGELOG since it's a released package.


Box drawn between line and line background with chart.js  2.9.3

Sorry, can not work out if this is a bug or a feature request. Feel free to reopen with more info if still valid.


affected/package: text/template

What version of Go are you using (go version)?

$ go version
go version go1.18 linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"

What did you do?

https://go.dev/play/p/Br7TxggWjtr
What did you expect to see?
with a different template name, we should get the same result as when the template name is the same as the file name
t, err := template.New("tmpl").ParseFiles("tmpl.txt")
What did you see instead?
when the template name is different from the file name, no result is obtained.
to fix it I modified the parseFile function in the file "/src/text/template/helper.go" at line 85 like this:
var tmpl *Template
		if t == nil {
			t = New(name)
		}
		if name == t.Name() {
			tmpl = t
		} else {
			tmpl = t.New(name)
			// what I added here
			t = tmpl
		}
		_, err = tmpl.Parse(s)
		if err != nil {
			return nil, err
		}

from ParseFiles:

Since the templates created by ParseFiles are named by the base names of the argument files, t should usually have the name of one of the (base) names of the files. If it does not, depending on t's contents before calling ParseFiles, t.Execute may fail. In that case use t.ExecuteTemplate to execute a valid template.

Closing as this is working as documented


COMDOX-372: Test

Purpose of this pull request
This pull request (PR) tests the project automation workflow.

Test was successful. The Actions workflow added this PR to the Commerce - Pull Requests project as expected.


Displays turned black

Microsoft PowerToys version
<IP_ADDRESS>
Installation method
Microsoft Store
Running as admin
No
Area(s) with issue?
General
Steps to reproduce
C:\Users\mi-wei\AppData\Local\Microsoft\PowerToys\PowerToys Run\Logs\<IP_ADDRESS>\2024-01-29.txt
Version: <IP_ADDRESS>
OS Version: Microsoft Windows NT 10.0.19045.0
IntPtr Length: 8
x64: True
Date: 2024-01-29 08:57:52
Exception:
System.Runtime.InteropServices.COMException (0x80263001): {Desktop composition is disabled} The operation could not be completed because desktop composition is disabled. (0x80263001)
at Standard.NativeMethods.DwmExtendFrameIntoClientArea(IntPtr hwnd, MARGINS& pMarInset)
at System.Windows.Shell.WindowChromeWorker._ExtendGlassFrame()
at System.Windows.Shell.WindowChromeWorker._UpdateFrameState(Boolean force)
at System.Windows.Shell.WindowChromeWorker._HandleDwmCompositionChanged(WM uMsg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at System.Windows.Shell.WindowChromeWorker._WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at System.Windows.Interop.HwndSource.PublicHooksFilterMessage(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
✔ Expected Behavior
No crash
❌ Actual Behavior
Crash (Displays became black in some seconds, then things starting popping up again).
Other Software
No response

/dup #30507


Raspberry Pi 2, Openelec and ws2801

Using OpenELEC-RPi2.arm-5.0.8.img, added to config file dtparam=spi=on param
"device" :
{
"name"       : "MyPi",
"type"       : "ws2801",
"output"     : "/dev/spidev0.0",
"rate"       : 250000,
"colorOrder" : "rgb"
},
"effects" :
{
	"paths" :
	[
		"/storage/hyperion/effects"
	]
},

while restarting the pi, the leds start working, now seems like it is working just fine i can see the color changes and blinking.
what should i do to make it into "production ready" for kodi? should i stop anything? to use another settings?
what do i miss here?

@Yariv-h
Could you please tell, what do you mean with "production ready". All is working fine?

Will close it
thank you


Self-managing queue (WIP)

This is a work in progress implementation of a self-managing queue. The loadMoreTiles call has been repurposed to function as a way to set a max loading threshold.
It will be interesting to see whether this implementation provides for faster tile downloading.
How it works:
The tile queue has a private method (loadMoreTiles_) which is invoked every time a tile is queued as well as every time a tile loads. This method ensures that the queue is always operating at its maximum capacity. The queue's capacity is set by the repurposed loadMoreTiles(maxLoading, maxNewLoads) method.
As tiles are dequeued, the queue checks for:

the tile's priority.
the tile's state

If the priority is DROP, the dequeued tile is ignored, and the queue moves on to the next one. If the state is anything other than IDLE, the tile is ignored as well.
Repurposing the loadMoreTiles call is temporary and was done to not impact other code. It is to be noted, however, that there is only one occurrence of a call to this method (in Map) [in the raster branch there are 2] so the change is minimal.
This implementation would also allow for exposing methods such as "pause" and "resume" on the queue, which could provide interesting functionality.

@tschaub

Nice @aisaacs.  Would be good to have a branch based on 7094f65ef74a06ee377afbb962a9612396c4b1d3 (so that it could make a pr for ol3/master).