Objective-C interop is too slow

I can see us spending quite a bit of time in querying the obj-c selectors on each message send.
Upstream issue - https://github.com/SSheldon/rust-objc/issues/49
One possible solution (that needs to be revived, cc @mystor): https://github.com/mystor/objc_methname

#![feature(const_slice_len)]

static string: &[u8; b"hello".len()] = b"hello";

fn main() {}

compiles on nightly; we just need to push to stabilize this feature (which should be trivial?)

For the time being, have you tried using for example lazy_static for selectors? Last time I tried it didn't seem to be much slower from having the system resolving them a launch, and way faster than resolving the selector at each call (as msg_send! does).
Something like that:
use objc::Message;

#[allow(non_upper_case_globals)]
mod selectors {
    use objc::runtime::Sel;

    lazy_static! {
        pub static ref isEqual_: Sel = Sel::register("isEqual:");
    }
}

fn is_equal(object: *mut objc::runtime::Object, compared_to: *mut objc::runtime::Object) -> bool {
{
    let ret_val: BOOL = unsafe {
        (*object)
            .send_message(*selectors::isEqual_, (compared_to,))
            .unwrap()
    };
    ret_val != 0
}

Also changing it later on to use link_section instead should be pretty straightforward.

@vincentisambart honestly, this is not as urgent to get in stable as it is important to just see what impact it's going to have. So while your suggestion is completely reasonable, I'm leaning towards rolling out support for it under "nightly" feature and checking out how it helps in the meantime. Those few of us who do the benchmarking could easily use the nightly, while for others it's not a blocker in any way.
Also, cc @pcwalton as requested.

@Gankro, do you mind filing an issue about stabilizing const_slice_len?

there's some issues with const fn stuff that means they're not quite ready for it; but nika wrote up a version that doesn't need it (kvark has it)


Add leptos-material, a component library.

This library automatically builds and bundles the Material Web Components. It includes Leptos component wrappers for the components, as well as some native Leptos components to fill in the gaps, such as Cards and Datepickers.

Awesome! Thanks for creating it, and thanks for adding it to the list!


Javascript and Typescrypt samples links 404

Hi Service Bus team
The two links Javascript Sample and Typescript samples (at the bottom of the page) are pointing to pages that doesn't exist anymore. I recommend to redirect the user to the latest samples in the github repository.
Thanks
Francesco De Liva

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

Note that publishing in this repository is suspended until Tuesday, May 25. Merging will resume Tuesday. Authors can still review and sign off on PRs in the interim. Thanks.

#sign-off

Hi @spelluru
Out of curiosity, when are you going to merge it?
Thanks
Francesco


Export whole books / chapters

Hey, I was wondering if it's possible to add a feature to export the whole book / some chapters?

For me as Student it would be a great feature!
So +1

This would be a great feature, it would allow us to keep one set of printable documentation for our clients.

Any idea if/when this feature might see the light of day?

@mayrmartin :smile:
What exports formats we thinking for this then?
Zip of HTML/Text files? Something else?

A single PDF would be nice

A single HTML file would be perfect for me...so I could convert it to a E-Book.

But also a single PDF file would be nice.

Also a zip archive of markdown files would allow to extract the content and keep it versioned in git. Interfacing with a git repo would be a killer feature.
Great project!

Would also love "export ALL" feature. I don't care about pdf/html/md. Any one would be good.

I created a command  to export a whole book to pdf. It is pretty dirty and not production-ready but works for what I needed it for. Maybe it helps somebody. Due to changes on master it only works with current release branch.
After loading in Kernel.php you may call php artisan book:export <bookslug>, where  has to be replaced with the slug of the desired book. This will create a bookslug.pdf in BookStacks root directory.

Okay, I've now created chapter/book export options based on the page exports that were available (PDF/HTML/TXT). This will be in the next release (v0.15) which I'm hoping to send out in the next day or two.
For other export options or other export improvements please open a new, specific issue.
Thanks @lommes for sharing that script to tide people over.


TypeError: this.color.join is not a function

Getting the following error on this file which worked previously.
TypeError: this.color.join is not a function
Stack trace:
TypeError: this.color.join is not a function
    at Object.CSG.Polygon.Shared.getHash (http://openjscad.org/csg.js:2944:31)
    at Object.CSG.fuzzyCSGFactory.getPolygonShared (http://openjscad.org/csg.js:4614:37)
    at http://openjscad.org/csg.js:694:47
    at Array.map (native)
    at Object.CSG.reTesselated (http://openjscad.org/csg.js:687:30)
    at Object.CSG.union (http://openjscad.org/csg.js:240:32)
    at union (http://openjscad.org/openscad.js:329:13)
    at motorXY (blob:http%3A//openjscad.org/df4f014f-28ce-4f4e-b2ea-b4df6453ae1d:503:46)
    at main (blob:http%3A//openjscad.org/df4f014f-28ce-4f4e-b2ea-b4df6453ae1d:1287:13)
    at Function.OpenJsCad.runMainInWorker (http://openjscad.org/openjscad.js:580:18)

Just to be sure... Browsers cache Javascript files. Please clear your browser cache and try again. This will make sure that all libraries are the most recent.
Thanks.

I have the exact same issue.  Cleared my cache.  Was there a resolution?

I did a little debugging for this.  The code is expecting color.join to be a function, which means it expects color to be an Array, but apparently in this case it is not.  So I set a conditional breakpoint in http://openjscad.org/csg.js:2944 , with the condition: (this.color.constructor !== Array)
Now it breaks and I can look at the value of color to find that it is set to "grey", a string instead of an Array.  This value is set by line 492 of the example code provided by quilford.
cube({size:[9,_nemaXYZ,20]}).setColor("grey"),

So the quick fix is to modify your line of code here to have actual numeric values as all your other setColor calls do.
However this value shouldn't be allowed to be set in the first place.
Looking into csg.js:setColor function, it calls out to CSG.Polygon.Shared.fromColor, which also only expects array type as input(when a single argument is passed), but the only check it does on single argument input is that the input has a length of 3 (r,g,b) or 4 (r,g,b,a).  Since the string "grey" happens to be 4 characters long, it assumes its a proper array and sets the value.  This function should have an additional type check for arrays and throw an exception at the time of setting if the input is not an array.

That's correct. setColor() expects an array of RGB values.
Solids can be given a color using the setColor() function. setColor() expects an array with 3 or 4
elements [r, g, b, [alpha]]. Beware: this returns a new solid,
the original solid is not modified! Faces of the solid will keep their original color when doing
CSG operations (union() etc). Color values range between 0.0 and 1.0 (not 255).
There's another function (OpenSCAD-like) called color() which takes a string, i.e. color name.

FYI @zdev I just updated csg.js with the latest version from OpenJsCad https://github.com/joostn/OpenJsCad/commit/183ec6684f5b7e7a25773af0130adec58bf6e112

Closing this issue?

Thanks for the help.


Increment version for template releases

Increment package version after release of com.azure azure-sdk-template

Not sure why this was closed but this should be merged as we released azure-sdk-template library to Maven and the version has to be incremented.
cc: @mitchdenny

Not sure why this was closed but this should be merged as we released azure-sdk-template library to Maven and the version has to be incremented.
cc: @mitchdenny

Generally the templates serve as test pipelines and also release prerelease version s based on the build number. So really bumping the version for template is not necessary.

Not sure why this was closed but this should be merged as we released azure-sdk-template library to Maven and the version has to be incremented.
cc: @mitchdenny

Generally the templates serve as test pipelines and also release prerelease version s based on the build number. So really bumping the version for template is not necessary.


Feature - travis pytest

Pull Request Test Coverage Report for Build 156

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 46.797%

Totals

Change from base Build 152:
0.0%

Covered Lines:
121

Relevant Lines:
224

💛  - Coveralls


arch/arm: Add CONTROL register bit field definition

Summary
and replace all hardcode value
Impact
code refector, no real change
Testing

Very nice! Much better without hard-coded values!


Add consistency check for ufs versions against their compiled modules

For new HDFS UFS versions, a new enum describing the version needs to be added to HdfsVersion.java as well as adding a new ufs module to compile the jar when building the tarball. It is easy to check for the UFS modules in a tarball by inspecting the lib directory of a tarball, but it is not easy to check that the corresponding version enum is defined. This check ensures that each HDFS UFS module has a corresponding HdfsVersion by checking that the compilation version matches version enum regex.

alluxio-bot, merge this please


add support to close the client connection

this is required so that we can open up a listener on the same port

:tada:


Add Link Formatting to Tiptap Editor

Description
Implement the ability to format selected text as a hyperlink in the Tiptap editor. The link should allow users to enter a URL and apply it to the selected text.
Requirements

Link Formatting:

When text is selected and the link option is chosen, a dialog should appear allowing the user to enter a URL.
The selected text should be formatted as a hyperlink pointing to the entered URL.

Menu Integration:

Add an option to the editor's bubble menu or toolbar to format selected text as a link.

Link Dialog:

The link dialog should look and behave like the screenshot below

Screenshot

Done ✅
https://github.com/user-attachments/assets/2fdef3ad-4116-4983-91a2-5210a7a23965


[FEATURE] Support RedKit's solution file

[ ] Support the detection of .w3edit files as a project manifest
The "scripts" directory is located at ./workspace/scripts

Resolved in #24


Detect BSD 2-Clause license via URL, just like the 3-Clause one

Fix #638

Thank you very much

Seems like I can not merge it, I think you need to do it :D


allow driving through private access gates to allow service vehicles …

…to access suburban gated communities from hell

Merging this, need to regenerate graphhopper files to allow for passing through gates


Missing a closing node

Instead of:
<ProjectReference Include="..\CS0433_1\CS0433_1.csproj">
  <Aliases>CustomTypes</Aliases>
<ProjectReference Include="..\CS0433_2\CS0433_2.csproj" />

it should be:
<ProjectReference Include="..\CS0433_1\CS0433_1.csproj">
  <Aliases>CustomTypes</Aliases>
</ProjectReference>
<ProjectReference Include="..\CS0433_2\CS0433_2.csproj" />

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

ID: cc727a99-200d-d34b-a2db-72f336f6f326
Version Independent ID: fa87a308-62df-2b4c-1685-2ff9372b5c46
Content: Compiler Error CS0433
Content Source: docs/csharp/language-reference/compiler-messages/cs0433.md
Product: dotnet-csharp
Technology: csharp-diagnostics
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Hii, Can I work on this issue? @lennybacon

As the PR is already merged, I think it should be good to close.

Thanks!

This was fixed in 3b61eeced55fcfe739b321a92ffb991216947ff3. Please close the ticket.

Thx


Add License

I'm interested in the many approached to making ebooks with github; I'm glad to see your work, and I encourage you to look at what others are doing. Adding an open license for your code would be a good thing.

Added MIT license.

Thanks!


Updating readme and change log to reflect major changes in 1.36

Updating Readme and Changelog

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 42.296%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
27752

Relevant Lines:
61357

💛  - Coveralls


22 allow specifying an array of validation functions

Closes #22
Changes

update README
allow withFormHandling optional onChange prop to be an array of functions

Steps to Test

[ ] try to define an input with an array of validation functions
[ ] confirm the first error is caught rather than the last one (if two always failing functions are provided, the error thrown by the first one should be passed to the input).

@mauleb conflicts


Make it reasonable to add packages to SBRP, as a downstream repo source-build champ who needs more

This involves:

Making the SBRP tooling public.
Documenting the SBRP tooling and/or making improvements to how it's used to make it reasonable for a repo's champ.

Related: https://github.com/dotnet/source-build/issues/3091, https://github.com/dotnet/source-build/issues/3089, https://github.com/dotnet/source-build/issues/3054, https://github.com/dotnet/source-build/issues/3044, https://github.com/dotnet/source-build/issues/3038, https://github.com/dotnet/source-build/issues/2804, https://github.com/dotnet/source-build/issues/1968, https://github.com/dotnet/source-build/issues/1958, https://github.com/dotnet/source-build/issues/1957, https://github.com/dotnet/source-build/issues/1956, https://github.com/dotnet/source-build/issues/1690.

I am making the assertion that the spirit of this issue has been completed during the .NET 8.0 release.  Of course there are more things that can be done to improve this process.  We are expecting repo developers to be adding their own SBRPs following the  documented instructions.  This process has been proven out via several contributions outside of the source-build team.


Add security package modules with unit tests

Check if there are differences between the Python modules in the following (sub)packages:
https://github.com/elixir-europe/WES-ELIXIR/tree/dev/wes_elixir/security
https://github.com/elixir-europe/proTES/tree/dev/pro_tes/security
https://github.com/elixir-europe/proWES/tree/dev/pro_wes/security
If there are differences, summarize and discuss differences. If not, add modules to archetype (adjusting import paths).

https://github.com/elixir-cloud-aai/proTES/tree/dev/pro_tes/security doesn't have decorators.py module. Do, we need to include this as a part of archetype?

Thanks @sarthak-sopho! It's a little complicated. I will need some time to look at this in detail to figure out what the most up-to-date version is

Okay @sarthak-sopho, I have just worked quite a bit on cwl-WES's version of security/decorators.py - which is now the most up-to-date version and so should be the one to use here. See elixir-cloud-aai/cwl-WES#151

Things to note here:

check for required parameters to pass through the decorator in order to guarantee generic use; also, set reasonable defaults to allow simple use
for unit tests relying on service/HTTP calls to identity provider, mock up responses in cases where valid token requirement is essential for the code to proceed

Closed by #30


Failed to import Liveportrait

Since some days, my Liveportrait doesn't work anymore so I wanted to reinstall it and I have always the same mistake when I try to import Liveportrait from the ComfyUI manager. I have also tried with git clone but always the same probleme. Someone has an idea about it ?

Since some days, my Liveportrait doesn't work anymore so I wanted to reinstall it and I have always the same mistake when I try to import Liveportrait from the ComfyUI manager. I have also tried with git clone but always the same probleme. Someone has an idea about it ?

That node shown there is not this node pack, it's another one: https://github.com/shadowcz007/comfyui-liveportrait
This one is:

I will try it. I dind't saw that it was the wrong node pack. Thank you very much.

Now it works again. It is probably because I have insisted trying to install the wrong node. 👍 Thank you for your help.


extract initial sample tasks to a script

move the initial 4 sample tasks out to a separate bash script. no tests appear to make use of them yet, and having them as a separate command will make testing more predictable.
cc @gammazero

fix #64


Discussion

when a user clicks a discussion icon, they'll go to a new page that shows the headers of the sending page and the discussion box.
so if a user is on the dictionary word detail page for 'Israel' and they click on the discussion icon for the 'Israel' entry, the discussion box will show a header for 'Dictionary' and a sub header for 'Israel'
if they click on a word definition for 'Israel', the discussion page will show 'Dictionary', 'Israel', and the word definition in the header

@lucky-gru helping out with caching


Does not seem to work on the beta curseforge site

vs

Hi, sorry for the late reply. It looks like curseforge changed something on their beta site which broke the extension. While the new site is still under development bugs like these might happen.


YargsError handling across whole lodestar-cli package

fixes #1099 by using YargsOptions across the whole lodestar-cli package

I would exclusively use YargsError in yargs handlers, so to files within lodestar-cli/src/cmds.

I would exclusively use YargsError in yargs handlers, so to files within lodestar-cli/src/cmds.

ah yeah i see what you mean.  going to push something for that in a sec


Aqara Temperature Sensor Reads 0 Then Immediately Updates Again to Correct Temp

Hi Mariano,
I came across your post here about how you were going to implement a possible fix like Veonua did.
However veonua's driver was showing the problem so I decided to install and try yours to see if you had it fixed.
I just had it flash to zero for an update and then immediately update again to the correct temp..
The problem is this is triggering some automations.
Do you know why it is showing zero and then updating correctly? Did your fix of ignoring zero values if the previous value wasn't around zero not work?
Thanks!

Hi,
The github code was not updated.
Nobody who asked me told me that it didn't work well, but when I reviewed the code I saw an error and I corrected it. Thanks
Only works for manufacturer== "LUMI" and model== "lumi.weather"
This driver version fix it
─────────────────────────────────────────────────────────
 Name         Zigbee Temp Sensor and Child Thermostat Mc
 Version      2024-01-04T10:04:05.829046739
─────────────────────────────────────────────────────────

I assume because manufacturer== "LUMI" this will not work for the Aqara/Xiaomi devices? Could you add in support for them too?

Aqara, xioami are trademarks. What the driver recognizes is the manufacturer and model recorded in its firmware, in this case LUMI
You can view the manufacturer and model code of your device at my.smartthings.com, advanced users.
As far as I know, the problem only affects the Aqara model, which I wrote to you

Oh, I understand about trademarks vs. manufacturer. Thanks for pointing that out. I checked under advanced users and I can confirm it shows as "lumi.weather" for model and manufacture code is "LUMI".
With that it seems like your manufacturer== "LUMI" will work. I see your commit/change and will test it and update this ticket.
Thanks.

I haven't had anymore false 0's since this change. I believe this has been fixed now. Thank you. This can be closed.


LUTECE-2009 : Fix tests compilation by using spring-test's MockHttpSe…

…rvletResponse

Thanks, so we should get rid of MokeHttpServletRequest as well ?

I think MokeHttpServletRequest should be rewritten as an extension of spring-test's MockHttpServletRequest


[$1000] Chat - Comments are deleted differently in mWeb/Chrome and Android App

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

Actual Result:
Comments are deleted differently in mWeb/Chrome and Android App. Label "Edited" is crossed out in the Android App but in mWeb/Chrome and Web is not crossed.
Workaround:
Unknown
Platforms:

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

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~016ac39b6311905f28
Upwork Job ID:<PHONE_NUMBER>730941440
Last Price Increase: 2023-05-04

Triggered auto assignment to @dylanexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

Current assignee @dylanexpensify is eligible for the External assigner, not assigning anyone new.

Triggered auto assignment to Contributor-plus team member for initial proposal review - @Santhosh-Sellavel (External)

Triggered auto assignment to @MariaHCD (External), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

This looks like a duplicate of https://github.com/Expensify/App/issues/18042.

Nice catch @akinwale!


Watching Functions in PostgreSQL 11

Hi there! I was trying to test PostgreSQL 11 in @OmniDB, but I found this error when I try to see the functions:
column p.proisagg does not exist
LINE 8:             where not p.proisagg
                              ^
HINT:  Perhaps you meant to reference the column "p.prolang".

Searching in the PostgreSQL documentation I found a breaking change in the system catalog table pg_proc, at least that's what I think. There's not proisagg anymore. There's a new field named prokind who enumerated this: "f for a normal function, p for a procedure, a for an aggregate function, or w for a window function". I see this here.
I think this new change reduce a lot of code for developing new tools, but at the same time this will affect not only this project, but A LOT OF PostgreSQL admin tools will be affected with this.

Hi @jeasoft ,
Can you please try OmniDB 2.11 and provide your valuable feedback?
Thank you very much in advance.
Regards,
William Ivanski

Hi @wind39.
I will do it on the weekend and let you know. Thanks for your great work guys!

GREAT! @wind39!

Thank you for your excellent work guys!


If OCM_CONTAINER_KERBEROS_USER is already set, prefer it

Prefer OCM_CONTAINER_KERBEROS_USER if it is already set in the environment, otherwise fall back to $OCM_USER (if set), and then finally fall back to $(whoami)
This helps in cases where the Kerberos user is different from the OCM user, for whatever reasons.

/lgtm
@iamkirkbater can you confirm? I don't have kerb on my local...
I want to merge because it looks so easy, but @iamkirkbater is one of the experts on this :-P

/lgtm
@iamkirkbater can you confirm? I don't have kerb on my local...
I want to merge because it looks so easy, but @iamkirkbater is one of the experts on this :-P

/lgtm
This looks fine to me.  I don't think it should cause any issues.

/lgtm
This looks fine to me.  I don't think it should cause any issues.


Update dependencies (testing PaC configured in Stonesoup)

I've deployed locally the image built by PaC and it seems it's working fine


ondataavailable is not triggered anymore in Chrome

Using the code shown here https://github.com/streamproc/MediaStreamRecorder#record-audiovideo to record audio and video the ondataavailable event is not triggered anymore making impossible to access the recorded blobs.
This issue happens using the MediaRecorder API
Also, any of the example demo are working anymore.
To recreate the issue:

open one of the demo, for examplehttps://www.webrtc-experiment.com/msr/video-recorder.html
open the console
start recording
stop recording
try to save
an error will appear in the console at line 110 of MediaStreamRecorder.js. Cannot access blobs object because undefined.

Any help or ideas?
Really appreciated!

As a workaround I forced the WhammyRecorder when browser is Chrome. This can be achieved with the following code:
var mediaRecorder = new MediaStreamRecorder(stream);
var isChrome = /* your preferred detection method */
if (isChrome) {
  self.mediaRecorder.recorderType = MediaStreamRecorder.WhammyRecorder;
}

Same problem here

it seems this issue does not happen anymore.

I am having this issue in Chrome with the A/V example. It worked for me in Firefox.

@stevula please test this demo:

https://www.webrtc-experiment.com/msr/video-recorder.html


Fix state duplication

The Problem

Either the domain entities are restricted not to reference other entities in 1-to-1 relationships
or the resulting json contains duplicate information.

Domain perspective
It makes everything more cumbersome for the domain if you start using Ids instead of directly referencing objects as domain models can con longer encapsulate all information necessary for their operation and it has to be provisioned separately.
Persistence perspective
References have to be resolved, otherwise updating an unresolved reference may result in the parent object containing obsolete information: If a Phase has a Loop and Loop.Length is updated, Phase.Loop.Length will not updated and it's not stuck with wrong data.
Solution
Do the leg work in the repositories. Unfortunately that does come with notable drawbacks:
-Repositories now have to implement actual update logic (previously the object was simply replaced, but that no longer works, because if a Loop is removed during update the reference to Phase breaks and they are serialized as separate objects)

Domain encapsulation is weakened, as in order to update an object instance the repo has to be able to set the object's properties. The lesser evil is to declare the setters as internal and use InternalsVisibleToAttribute in order to restrict outside intervention to only within the domain and persistence projects.

Example for domain solution and it's complications can be seen here

Fixed. WOOHOO!


How to implement unrolled generative adversarial networks in theano/keras?

After reading the ICLR 2017 submitted paper unrolled generative adversarial networks in http://<IP_ADDRESS>:3000/pdf?id=BydrOIcle, I'm curious about this technique for addressing the problem of mode collapse when training GAN models.
The key idea is that  we could update the generator after several simulative updating of discriminator, while the discriminator uses the original parameter before these simulations to complete updates.
At first sight, I think of the recurrent process using theano.scan(). However, since the parameters are theano.shared, how could we simulate one subset of them recurrently to update the others while keep the original subset being kept for another use?
Any ideas? Thanks !

I was testing out unrolled GANs in straight theano but I would definitely prefer to do so in Keras. I'm thinking you could do an unrolled GAN in Keras using theano.clone and replace. Use Keras to make the graph, then clone it replacing shared variables with updates. Seems tricky but doable. https://gist.github.com/kastnerkyle/29be76e78adaebaa2b69

Just got Keras unrolled GANs working in theano and tensorflow. Warning: compiling will crush your RAM.
https://github.com/bstriner/keras_adversarial
Basic idea is to get discriminator and generator updates, and substitute discriminator updates into generator updates k times.
This is the core code, but there is a lot more to integrate with Keras if you just browse the repo.
replace = {k:v for k,v in dupdates}
updates_t = gupdates
    for i in range(depth):
        updates_t = [(k, theano.clone(v, replace)) for k, v in updates_t]
return updates_t


Validate/test if cluster configuration property of type string can be un-set

The rpk cluster config set at the moment of writing has the following logic:
https://github.com/redpanda-data/redpanda/blob/c3cca097b01c250715fd4012bac175f09c26778e/src/go/rpk/pkg/cli/cmd/cluster/config/set.go#L85-L99
			if meta.Nullable && value == "null" {
				// Nullable types may be explicitly set to null
				upsert[key] = nil
			} else if meta.Type != "string" && (value == "") {
				// Non-string types that receive an empty string
				// are reset to default
				remove = append(remove, key)
			} else if meta.Type == "array" {
				var a anySlice
				err = yaml.Unmarshal([]byte(value), &a)
				out.MaybeDie(err, "invalid list syntax")
				upsert[key] = a
			} else {
				upsert[key] = value
			}

This mean that if any property of type string which has different default value then "" (empty string) can not be unset.
I didn't test it, but it might be worth checking the rpk cluster config import
https://github.com/redpanda-data/redpanda/blob/c3cca097b01c250715fd4012bac175f09c26778e/src/go/rpk/pkg/cli/cmd/cluster/config/import.go#L98-L223
Related issue https://github.com/redpanda-data/helm-charts/pull/395

I don't think there's a way for us to null out something. We depend upon the existence of the key/value in the dict in order to send something to config set. And I'm not sure there's a way from the yaml to say that a particular value for a given key can be null. or empty. or whatever.
  ansible.builtin.command: rpk cluster config set {{ item.key }} {{ item.value }} {{ redpanda_rpk_opts }}
  loop: "{{ configuration.cluster | dict2items }}"
  register: result
  changed_when: '"New configuration version is {{ config_version.stdout|int() }}." not in result.stdout'
  run_once: true
  no_log: true```

Thinking about this a bit more: since the command is exercised as a jinja2 template, we'd have to figure out how to get the {{ item.value }} to resolve to empty/nothing. Maybe we could do something like {{ item.value if item.value is defined and item.value|length  }}

To achieve this we'd likely need to write an RPK module for ansible and define state. We'd then use RPK to check the state of the config value, and remediate accordingly if state is set to present (set to the value provided) or absent (set to null).
This is a longer term project, so isn't something we can solve right away. It's necessary for most of the administrative playbooks people want us to put together so it is something on my personal todo list.
As it stands today, my recommendation for people who want to shift how their cluster works while using ansible would be for them to change up the playbook and then engage in manual remediation with RPK to ensure the changes are sound. That's the method most likely to lead to good outcomes today.


Do not cache of kernels when token is cancelled

Fixes #11240

Codecov Report

Merging #11325 (26347e0) into main (65b2c5d) will increase coverage by 0%.
The diff coverage is 66%.

@@          Coverage Diff           @@
##            main   #11325   +/-   ##
======================================
  Coverage     63%      63%
======================================
  Files        476      476
  Lines      33923    33930    +7
  Branches    5497     5499    +2
======================================
+ Hits       21532    21550   +18
+ Misses     10331    10329    -2
+ Partials    2060     2051    -9

Impacted Files
Coverage Δ

...rnels/raw/finder/localKernelSpecFinderBase.node.ts
79% <66%> (-2%)
:arrow_down:

.../standalone/survey/dataScienceSurveyBanner.node.ts
66% <0%> (-6%)
:arrow_down:

src/standalone/api/kernelApi.ts
75% <0%> (-2%)
:arrow_down:

src/notebooks/controllers/kernelRanking/helpers.ts
58% <0%> (-1%)
:arrow_down:

src/platform/common/utils/localize.ts
76% <0%> (-1%)
:arrow_down:

src/kernels/common/baseJupyterSession.ts
80% <0%> (+<1%)
:arrow_up:

.../notebooks/controllers/vscodeNotebookController.ts
82% <0%> (+1%)
:arrow_up:

src/kernels/jupyter/jupyterKernelService.node.ts
76% <0%> (+1%)
:arrow_up:

src/kernels/jupyter/baseKernelConnectionWrapper.ts
37% <0%> (+2%)
:arrow_up:

...otebooks/controllers/controllerPreferredService.ts
90% <0%> (+5%)
:arrow_up:

... and 2 more


[ADD]: Instagram Flutter Clone

Describe the project you want to add with tech stack
A completely Responsive Instagram App- Works on Android, iOS & Web!
Tech Stack:

Flutter

Expected behavior

A clone of Instagram.

@pranjay-poddar @MohitGupta121 @TusharKesarwani Can you please assign me this issue.


Multiple alternative experiments

This issue is primarily intended for discussion as we move towards adding CITEseq processing (filtering/normalization) into the workflow. While working towards this goal, I noticed some aspects of the pipeline which may have been written with a longer term goal of accommodating multiple alternative experiments, although offline discussion has suggested this is a feature (⚠ pun alert) that we don't implement.
I'm specifically looking at these places:

In the make_merged_unfiltered_sce process, we assign a name for the alternative experiment based on the technology metadata. This choice on its own does not mean that we are setting up for multiple experiments, but it could suggest that the altExp name should be sent into to later processes so that the right altExp can be pulled out to work with (for example, to be filtered on!).
https://github.com/AlexsLemonade/scpca-nf/blob/a6b87e474888cdfc4f11c137b5ab138b730c97ae/modules/sce-processing.nf#L30-L59

In the post process Rscript itself post_process_sce.R, we have this for-loop that implies the potential existence of multiple named altExps.
https://github.com/AlexsLemonade/scpca-nf/blob/a6b87e474888cdfc4f11c137b5ab138b730c97ae/bin/post_process_sce.R#L97-L105

To me, it seems the most immediate decision here is whether we need to be passing altExp names through the pipeline to be able to isolate the CITEseq one for processing, in the event there are multiple altExps. If we always only expect 1 altExp, its name is essentially moot so that metadata isn't needed. Along the way, we could also get rid of that for-loop.
On the other hand, if we think we might in the longer term want to handle multiple altExps, then the metadata name does need to be passed through to save us from our future selves, and we'd keep the for-loop.
Tagging @jashapiro @allyhawkins for discussion!

The name of the altExp are already passed along as meta.feature_type so there is no need for a change there at this time.
Updating the workflows to handle multiple AltExps is something we have not yet had to support, and will require substantial reengineering. The most likely scenario is that we will get data that has something like multiplexing and CITEseq or multiplexing and ATAC, in which case we would be adding a separate process and flow among processes. The merge script would have to change, as would how we designate those files in the technology column.
Which is basically to say, this a problem we can face when we actually have data with multiple altExps. I don't think there are any changes that need to be made at this time.

Thanks @jashapiro! Followup from your comment, I am maybe understanding something I hadn't before. Does this wrangling add feature_type to the metadata forever? I had thought it was maybe just scoped locally to change metadata within this process, but if this change to meta is "global", then :tada:!
https://github.com/AlexsLemonade/scpca-nf/blob/a6b87e474888cdfc4f11c137b5ab138b730c97ae/modules/sce-processing.nf#L42-L44

This was just an issue for discussion, and it has been discussed! Can be closed.


Fix #993 Draco UV export

Fix #993
Fix #1079
Fix #1006
Fix #998
Fix #917
Fix #916

Nice, six issues with three characters :)
You probably meant #988 instead of #998.

Yes, you're right. I edited
Too much numbers for me ;-)

@julienduroure do you know when this will be included to Blender ? This was merged almost a month ago, yet the new Blender release from 6 days ago (2.83.2) still has the problem. It's the only thing preventing us from switching to 2.83

@SBRK This is merged in 2.90 alpha/beta version, not in 2.83.x


support environment list from ENV variable

Closing since we have merge conflicts, and other feedback that still needs to be taken into account.


Bump Wireguardex to support older glibc

Pull Request Test Coverage Report for Build e931d02a42677350d10fa5ccc037137c9dc968c5-PR-906

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 67.521%

Totals

Change from base Build 72e0458a7353401fb4cda01c6a7c4e147825ea5f:
0.0%

Covered Lines:
1054

Relevant Lines:
1561

💛  - Coveralls


refactor: Adjust dialogue UI (make bigger, perhaps scale with screen)

Done as in 9dea3c41fbdf82c9bbee4f6757cf28cce3180e05 and 92264bea67d2c3687f8fcbc5f757e372be30af00


When shortmess contains F and we run :cdo, messages from autocommands are not always printed to the message-history

Steps to reproduce
Run vim --clean -S test.vim, with
vim9script

set shortmess+=F
mkdir('/tmp/vim/', 'p')
writefile(['test'], '/tmp/vim/1')
writefile(['test'], '/tmp/vim/2')
writefile(['test'], '/tmp/vim/3')

augroup test
    autocmd!
    autocmd BufWinEnter * echomsg $'BufWinEnter: {expand('<amatch>')}'
    autocmd BufReadPost * echomsg $'BufReadPost: {expand('<amatch>')}'
augroup END

grep! test /tmp/vim/1 /tmp/vim/2 /tmp/vim/3
cdo! substitute/test/TEST

Output:
BufWinEnter: /tmp/vim/1
(1 of 3): test
"/tmp/vim/2" 1 line, 5 bytes
BufReadPost: /tmp/vim/2
BufWinEnter: /tmp/vim/2
(2 of 3): test
"/tmp/vim/3" 1 line, 5 bytes
BufReadPost: /tmp/vim/3
BufWinEnter: /tmp/vim/3
(3 of 3): test

The current output is inconsistent. Why isn't the echomsg from BufReadPost for /tmp/vim/1 written to messages, but BufWinEnter is? In my opinion, there are two lines missing in the output:
"/tmp/vim/1" 1 line, 5 bytes
BufReadPost: /tmp/vim/1

If we remove set shortmess+=F and rerun the script, we get exactly that:
"/tmp/vim/1" 1L, 5B
BufReadPost: /tmp/vim/1
BufWinEnter: /tmp/vim/1
(1 of 3): test
"/tmp/vim/2" 1 line, 5 bytes
BufReadPost: /tmp/vim/2
BufWinEnter: /tmp/vim/2
(2 of 3): test
"/tmp/vim/3" 1 line, 5 bytes
BufReadPost: /tmp/vim/3
BufWinEnter: /tmp/vim/3
(3 of 3): test

Note that if we replace echomsg with popup_notification() in the script above, in total six autocommands are triggered as one would expect (BufReadPost and BufWinEnter for all three files that are edited).
Expected behaviour
When shortmess contains the F flag, either Vim shouldn't give any file infos at all when running :cdo, or it should print them all consistently for every file edited in :cdo.
Version of Vim
9.0.818
Environment
Linux
Logs and stack traces
No response

Hm,
I see the problem.
We reset 'shortmess' option when jumping to the next error message, but not when loading the very first quickfix match.
This is done on purpose, to avoid, that any possible messages are hidden:
https://github.com/vim/vim/blob/98aeb2100c2759111f93f0f0857e93d98afdc88a/src/ex_cmds2.c#L653-L654
I guess to make it consistent, we should reset p_shm also a couple of lines above, like this:
diff --git a/src/ex_cmds2.c b/src/ex_cmds2.c
index 9c00b80b7..b20d8b572 100644
--- a/src/ex_cmds2.c
+++ b/src/ex_cmds2.c
@@ -541,7 +541,13 @@ ex_listdo(exarg_T *eap)
                buf = NULL;
            else
            {
+               // Clear 'shm' to avoid that the file message overwrites
+               // any output from the command.
+               p_shm_save = vim_strsave(p_shm);
+               set_option_value_give_err((char_u *)"shm", 0L, (char_u *)"", 0);
                ex_cc(eap);
+               set_option_value_give_err((char_u *)"shm", 0L, p_shm_save, 0);
+               vim_free(p_shm_save);

                buf = curbuf;
                i = eap->line1 - 1;

Thank you, the patch fixes the above issue.

I see the problem.
We reset 'shortmess' option when jumping to the next error message,
but not when loading the very first quickfix match.
This is done on purpose, to avoid, that any possible messages are hidden:
https://github.com/vim/vim/blob/98aeb2100c2759111f93f0f0857e93d98afdc88a/src/ex_cmds2.c#L653-L654
I guess to make it consistent, we should reset p_shm also a couple
of lines above, like this:
diff --git a/src/ex_cmds2.c b/src/ex_cmds2.c
index 9c00b80b7..b20d8b572 100644
--- a/src/ex_cmds2.c
+++ b/src/ex_cmds2.c
@@ -541,7 +541,13 @@ ex_listdo(exarg_T *eap)
                buf = NULL;
            else
            {
+               // Clear 'shm' to avoid that the file message overwrites
+               // any output from the command.
+               p_shm_save = vim_strsave(p_shm);
+               set_option_value_give_err((char_u *)"shm", 0L, (char_u *)"", 0);
                ex_cc(eap);
+               set_option_value_give_err((char_u *)"shm", 0L, p_shm_save, 0);
+               vim_free(p_shm_save);

                buf = curbuf;
                i = eap->line1 - 1;

Can you make a PR with this?  And a test for it.
We'll have this change for 'shortmess' four times then, better move it
to a couple of functions.
--
GOD: That is your purpose Arthur ... the Quest for the Holy Grail ...
"Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

Fixed by https://github.com/vim/vim/commit/9aee8ec400fe617f6d82441c46a22d0cef6fa3e6.


Feature: Support for retrieving multiple result sets

Necessary when a stored procedure returns multiple result sets.
Useful for e.g. when combining 2 selects in one round trip to the server (SELECT x FROM x; SELECT y FROM y;)

This issue was moved to nanodbc/nanodbc#27


Unable to upload dataset to OpenNeuro

OpenNeuro does not accept the dataset as it is, raising errors. Currently, the dataset is stored in Google Drive.

Since there seems to be issues in the dataset (#12, #13), we should probably resolve those first before attempting to fix the problems preventing the dataset from being uploaded to OpenNeuro.

Is there any additional information about which files are not BIDS compliant?
For issue #12 : As mentioned in the issue (https://github.com/spinal-cord-7t/coil-qc-code/issues/12#issuecomment-1952473648), the notebook should retrieve all 3 DREAM scans and combine them. This means that the DREAM scans with 3 different voltages should be included in the OpenNeuro dataset. However, it doesn't look like the lower and higher voltage scans were converted to BIDS (see my comment here: https://github.com/spinal-cord-7t/coil-qc-code/pull/9#issuecomment-1952548438). This issue should not stop us from making sure that the data we already do have is properly BIDS formatted.
For issue #13 : The correct NYU scans can be sorted out independently of the BIDS conversion issue.

Unfortunately, OpenNeuro doesn't give us much information on what exactly is wrong about the naming. Expanding the error simply enumerates all the files (111 files) and reiterates the error message:

The second error is easily fixable by adding an appropriate dataset_description.json file however, which I will do.

But we know that the file that is causing the problem is "sub-MGH1_acq-coilQaSagLarge_SNR.nii.gz, yes?

Is the suffix "coilQaSagLarge_SNR" part of BIDS?

After reading the acq- entry https://bids-specification.readthedocs.io/en/stable/appendices/entities.html#acq, I see that "coilQaSagLarge" is your label, and that "SNR" is a suffix. This must be what is causing the problem. @jcohenadad had pointed out in this comment: https://github.com/spinal-cord-7t/coil-qc-code/pull/9#pullrequestreview-1811994832 that "SNR" is not a BIDS suffix and had advised you on how to proceed in order to decide how to handle this file.

But we know that the file that is causing the problem is "sub-MGH1_acq-coilQaSagLarge_SNR.nii.gz, yes?

This is just the first file for which this error applies, there are in total 111 files that have filenames that don't comply with the BIDS convention (according to OpenNeuro at least).

I have created a new topic on NeuroStars asking for feedback and listing the scans for which the filenames are detected as incorrect on OpenNeuro.

Closing this issue as I no longer have any problems uploading the dataset on OpenNeuro.


Payment system.

About the feature
On behalf of all Russian/Belarusian/Ukrainian/I'm telling Kazakh hosting companies, please support the P2P QIWI payment system or something similar.
Additional information
No response

P.s This is my addition to one person's request.

Do you have any links or resources for this? Because most searches are coming up Russian and I can't translate them.

Do you have any links or resources for this? Because most searches are coming up Russian and I can't translate them.

https://developer.qiwi.com/en/p2p-payments/#p2p-invoices


Can't save ImageWork in Hyrax branch

Need to remove :cartographic_projection from image work form.

@eliotjordan Is this closed?

It is.


Ensures that synctool exclusion rules are applied (in addition to the…

… initial walk)
on adding new watched files as well as when restoring from a backup.
Resolves: https://jira.duraspace.org/browse/DURACLOUD-1129

Hey @bbranan :  as an aside I tested these changes on the synctool as well as the synctoolui.

new pr coming soon without static reference refactoring.


Samples (csharp_dotnetcore/49.qnamaker-all-features): 'ComponentRegistration' is obsolete: 'Use BotComponent for new components.'

Context:
Sample: https://github.com/microsoft/BotBuilder-Samples/tree/main/samples/csharp_dotnetcore/49.qnamaker-all-features
This sample uses an obsolete class ComponentRegistration and I cannot find a working example with the correct implementation
Additional Context:
The botbuilder-dotnet package was updated to change component registration: https://github.com/microsoft/botbuilder-dotnet/pull/5347
Solution
Please provide an up to date sample of the botbuilder-dotnet BotComponent implementation

any update on this?

Hi, @dougbw and @rizified, it is OK to continue using the deprecated ComponentRegistration in this case, we will update the samples once we have a better implementation, but you should be OK


Roll Flutter Engine from b26e1b023cdb to 0e10f4cee636 (6 revisions)

https://github.com/flutter/engine/compare/b26e1b023cdb...0e10f4cee636
2024-05-29<EMAIL_ADDRESS>Roll Skia from d2469aaf5ab1 to 99966843c23f (1 revision) (flutter/engine#53086)
2024-05-29<EMAIL_ADDRESS>Roll Skia from 9ea5a1dd5270 to d2469aaf5ab1 (2 revisions) (flutter/engine#53085)
2024-05-29<EMAIL_ADDRESS>Roll Skia from eedef8763102 to 9ea5a1dd5270 (1 revision) (flutter/engine#53084)
2024-05-29<EMAIL_ADDRESS>Roll Skia from 0026a6047133 to eedef8763102 (1 revision) (flutter/engine#53081)
2024-05-29<EMAIL_ADDRESS>DisplayList forward opacity incompatibility through deferred saves (flutter/engine#53078)
2024-05-28<EMAIL_ADDRESS>Roll Skia from ac454b80130c to 0026a6047133 (1 revision) (flutter/engine#53076)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jacksongardner@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


package_type parameter seems to be ignored in policy evaluation

Is this a request for help?: No

Is this a BUG REPORT or a FEATURE REQUEST?
BUG REPORT

Version of Anchore Engine and Anchore CLI if applicable:
anchore-cli, version 0.7.0
Name: anchore-engine
Version: 0.7.1
Summary: Anchore Engine
Home-page: http://www.anchore.com
Author: Anchore Inc.
Author-email<EMAIL_ADDRESS>License: Apache License 2.0
Location: /usr/local/lib/python3.6/site-packages
Requires: Click, pg8000, pathlib, semantic-version, psycopg2-binary, Flask, prometheus-flask-exporter, treelib, pytz, prometheus-client, retrying, prettytable, Twisted, ijson, swagger-spec-validator, jsonschema, six, PyYAML, zope.interface, passlib, requests, python-keystoneclient, psutil, Authlib, zope.component, SQLAlchemy, boto3, watchdog, connexion, python-dateutil, toastedmarshmallow, typing, Werkzeug, yosai, python-swiftclient, urllib3

What happened:
Anchore policy evaluation shows STOP action for non-os packages vulnerabilities but policy parameter package_type with value os present.
See attached anchore.log.json.txt, JSON result of policy evaluation.
What did you expect to happen:
Only os vulnerabilities to be considered when evaluating the policy.

Any relevant log output from /var/log/anchore:
What docker images are you using: v0.7.1
How to reproduce the issue: See the attached result for the policy.
Anything else we need to know:

Hi @wilx , I see this only for GHSA vulnerabilities in that output. Have you seen any other matches (CVEs from NVD) in this case?

There are no more fixable os vulnerabilities in the image. There unfixable os vunlerabilities and also one additional nvdv2:cves (python) vulnerability in the results.

There are no more fixable os vulnerabilities in the image. There unfixable os vunlerabilities and also one additional nvdv2:cves (python) vulnerability in the results.

Ack. Thanks. That confirms my analysis as well.


Shaka Player repeatedly throws the warning "Assertion failed: SegmentReferences are incorrect" when playing certain DASH assets

Have you read the FAQ and checked for duplicate open issues?: Yes.
What version of Shaka Player are you using?: 2.3.7
Can you reproduce the issue with our latest release version?: Yes
Can you reproduce the issue with the latest code from master?: Yes
Are you using the demo app or your own custom app?:custom  portal that launches shaka player
If custom app, can you reproduce the issue using our demo app?: It is reproduceable
What browser and OS are you using?: The shaka player runs on a Set- Top-Box on Ekioh browser
What are the manifest and license server URIs?:
Manifest URL: http://<IP_ADDRESS>/live/eds/LRT_HD/DASH/LRT_HD.mpd

What did you do?
What did you expect to happen?
What actually happened?

segmentincorrect-5.log
attached logs showing the above warning message

More details about the issue:
Although the playback is not halted due to the warnings, logs get accumulated due to excessive warning statements which can slow down the system

Hi @ravics78,
Can you repro it in any other browser? I don't seem to get the assertion in either Chrome or Firefox.


Added underline mark

I have added underline mark.

Hello @hanspagel can you check this out?

Thanks Jason! ✌ Merged


Storage( dynamodb ) specified during amyplify init is not being generated

** Which Category is your question related to? **
CI / Headless
** What AWS Services are you utilizing? **
Amplify CLI
** Provide additional details e.g. code snippets **
I'm trying to initialize an amplify project using headless amplify CLI, part of this initialization requires a database creation using DynamoDb as the storage engine. I've used a fork version of this script but it seems I might be doing something wrong since the database isn't created, not to say that the whole --categories flag is being ignored.
According to the docs there is not such a thing as a documented  --categories flag for amplify init command but it exists on the sample scripts this script
Am I missing something? this is the content ( $CATEGORIES ) I pass to amplify init on the --categories--_ argument
{
  "storage": {
    "usersTable": {
      "service": "DynamoDB",
      "providerPlugin": "awscloudformation",
      "providerMetadata": {
        "s3TemplateURL": "https://s3.amazonaws.com/myapp-dev-20190731130334-deployment/amplify-cfn-templates/storage/usersTable-cloudformation-template.json",
        "logicalId": "usersTable"
      },
      "lastPushTimeStamp": "2019-08-15T14:00:04.271Z",
      "output": {
        "PartitionKeyName": "email",
        "Region": "us-east-1",
        "Arn": "arn:aws:dynamodb:us-east-1:048xxxxxxxxx:table/users-dev",
        "StreamArn": "arn:aws:dynamodb:us-east-1:048xxxxxxxxx:table/users-dev/stream/2019-08-12T18:35:51.674",
        "PartitionKeyType": "S",
        "Name": "users-dev"
      },
      "lastPushDirHash": "2J+8sTotqquf3dZYkXgZfk1KV8Y="
    }
  }
}

This is the command I'm running, there are no errors.
amplify init \
--amplify $AMPLIFY \
--providers $PROVIDERS \
--categories $CATEGORIES \
--yes

@oscarnevarezleal Headless creation of resources/categories is not supported yet. the categories flag is used to just pass in parameters to the cloudformation templates and not to create the resources. You would have to go through the CLI flow to add the resources once and then use the headless scripts (mentioned above) to automate your deployments.

Noted, thanks for clarifying this.


Using 'throw' with inside a catch or without try/catch causing inappropriate error.

When I used a throw statement inside a batch block or without having a try/catch block I am getting following error which is even not meaningful.
error in ballerina program:
at createTable(temp2.bal:71)
at SQLConnectorTestMysql(temp2.bal:54)

This is fixed now.


Build to dist folder

This allows us to create a package.json which differs from the source package.json, and be selective about the files included in the distributed bundle.
Fixes #42

Coverage remained the same at 100.0% when pulling e71f0baa16084d77085afe230f7529ba5e960992 on npm-dist into 3ce19f673fdde1ffff9b17855fe9540fd64e9fee on master.

Coverage remained the same at 100.0% when pulling e71f0baa16084d77085afe230f7529ba5e960992 on npm-dist into 3ce19f673fdde1ffff9b17855fe9540fd64e9fee on master.


⚠ Topfarmacia has degraded performance

In 3134639, Topfarmacia (https://www.topfarmacia.it) experienced degraded performance:

HTTP code: 200
Response time: 10949 ms

Resolved: Topfarmacia performance has improved in 5983dc7.


Nuget Push Timeout

Task "Nuget" command "Push".
Is it possible to add "Timeout" or "Arguments" input field to the task so that it will be possible to use it for big packages that takes more than 300 sec to upload? Field "Nuget Arguments" was available in the deprecated task "NuGet Publisher".
I know that I can use "custom" and provide any arguments and I already did it as an workaround, but after selecting "custom" we cannot use "Service connection" to external nuget server. Service connection is very helpful in our case - we have one server that is used from multiple build/release definitions, so no need to remember/change and expose apikey to this server. Sure, I can think about workaround again and keep apikey in "Library" even encrypted, but IMHO "ServiceConnection" and "TimeOut: in task Nuget Push will be very useful.
Thank you.

It will be very useful.

Hey thanks for the enhancement idea. Our stance with respect to tasks is moving more where we invest less in the client wrapping tasks and more in the authenticate tasks (à la Twine Authenticate but for nuget.exe). Customers would then own calling the clients in their yml pipelines which would allow for this level of flexibility. We are actively working on nuget.exe authenticate task.
As a result, it is unlikely we will get to feature enhancements for existing tasks.

The Nuget Authenticate task has been released. You can find the documentation for it here: https://docs.microsoft.com/en-us/azure/devops/pipelines/tasks/package/nuget-authenticate?view=azure-devops.


Use go-getter with secrets as well

Before this PR, only values can be used with go-getter and not secrets. The PR implement remote FS for secrets as well to maintain the consistency.
#{{ $configRepo := "git::https://example.xxx/yyy/zzz.git" }}

environments:
  '{{ .Environment.Name }}':
    values:
    #{{- if .Environment.Values }}
    - {{- .Environment.Values | toYaml | nindent 6 }}
    #{{- else}}
    - ../envs/_default.yaml
    - ../envs/{{ $env }}.yaml
    - '{{ $configRepo }}@/{{ $env }}.yaml?ref={{ env "CONFIG_REPO_REF" | default "main" }}'
    #{{- end }}
    secrets:
    #{{- not if .Environment.Values }}
    - ../envs/{{ $env }}.secrets.yaml
    - '{{ $configRepo }}@/{{ $env }}.secrets.yaml?ref={{ env "CONFIG_REPO_REF" | default "main" }}'
    #{{- end }}

@itscaro adding some unit tests will be better.

@yxxhero yes just PR to discuss about the feature first :)

@itscaro yeah. could you show an example?

@yxxhero just added an example in description, still testing on local :)

@yxxhero Humh any idea how we test this part, I found no existing test for go getter with either values or secrets

@itscaro I will see how to add a unit test. you can test it locally, and copy the process to the description.

@yxxhero I was trying to use e2e test to check for output, but there are random temporary directories in output, any idea to just check if output contains some string? I've not looked how snapshot_test.go works thoroughly.

@itscaro I think you can specify HELMFILE_TEMPDIR to make temp values file paths a bit more predictable and regex-replace the paths in the real output to make it stable enough for comparison with the snapshot.
https://github.com/helmfile/helmfile/blob/f7b3fb4432dba7889884a1a741626b5958496fa6/pkg/envvar/const.go#L7

@mumoshu thanks, I have thought of it too, not have time to test yet.

need to update docs.

This logger depends on how fast the output is capture we have one line log ou two-line log so the log prefix helm> differs :( any idea?

Is that due to that Helmfile runs multiple Helm commands concurrently? Then I can recommend setting concurrency to 1. It defaults to the number of releases in the helmfile.yaml so it's fully concurrent by default. Explicitly setting it to 1 completely disables the concurrency. We do the same in unit tests like apply_test.go.

with --enable-live-output, it seems the output is idempotent
@yxxhero  will update doc

[x] update doc
[x] add remote> prefix to logs in remote.go
[x] use --concurrency=1


feat(rust):add key validation during secret_import in ockam_vault

It looks like Noise XX Key Agreement test vectors were generated by using invalid predefined keys. Need to update those values before we can merge it
We'll need to update those values in elixir as well cc @hairyhum @mrinalwadhwa

Hey @SanjoDeundiak, may I do something about it ?

Hey @SanjoDeundiak, may I do something about it ?

Hey @devalain . If you want to update xx test vectors - you're welcome to do so. It would be great!

Hope I did it right.

#1695


Strip removable sources for gecko-dev

Summary so far:

Can remove googletest, as evident from the strip attempt here.
google/ruy is licensed Apache 2.0. It appears to rely on pytorch/cpuinfo entirely to detect platform [1, 2].
pytorch/cpuinfo is BSD licensed. appears to have a clog dependency.
pytorch/cpuinfo/deps/clog is BSD-2 licensed.

The workflow also strips extras folder, which contains tests etc.
It's possible to create a tarball from here for import into gecko-dev with the remaining sources, automating andre's request.

Merging at this checkpoint, to hand over harder dependency stripping (cpuinfo, etc.) to stronger, more experienced developers.


performance: templ parse takes a long time and uses high CPU when unclosed void elements are used

Describe the bug
With the code I show here templ takes VERY long to compile, about 29 seconds. If I add more cases to the switch case it takes even longer, to the point where it is completey unusable and locked.
If I just remove the inputs in the top of the ActionStep template it goes below one second again...
This seems like a really funkey scanner issue to me, but I am sure you are much faster in finding the actual issue...
the templ process also climbs to ridiculous CPU usage while trying to generate in this case...
To Reproduce
Repo: https://github.com/s00500/templ-switchcaseissue
Expected behavior
Fast compile, no lockup
Logs
❯ templ generate
(!) templ version check: failed to read go.mod file: open /go.mod: no such file or directory
(✓) Complete [ updates=1 duration=28.554246583s ]

templ info output
❯ templ info
(✓) os [ goos=darwin goarch=arm64 ]
(✓) go [ location=/usr/local/go/bin/go version=go version go1.22.1 darwin/arm64 ]
(✓) gopls [ location=/Users/lb/go/bin/gopls version=golang.org/x/tools/gopls v0.16.1 ]
(✓) templ [ location=/Users/lb/go/bin/templ version=v0.2.759 ]

Desktop (please complete the following information):
macOS (latest, sonoma 14.6.1)
❯ templ version
v0.2.759
(Masterbuild from today)

Here is the time it takes after adding one more case with a  to the template:
❯ templ generate
(!) templ version check: failed to read go.mod file: open /go.mod: no such file or directory
(✓) Complete [ updates=1 duration=48.298382166s ]
(increase by ~20 seconds)

It's the unclosed
 elements. The parser attempts to look for a close tag to decide what to do.
Once fmt has run, it doesn't need to do that any more. The fix would be for the parser to give up trying to look for the
 as soon as it sees another
 in the doc.

The parser is slow because, imagine this scenario ("element: void nesting same is OK" test).
People sometimes accidentally put elements inside void elements, so it's not clear what the intent is.

How should templ handle this?

Or...

Or an error complaining about a void element having nested content?
Or strip the nested content?
Or ignore closers for void elements?

Ah I see... ok...
Any chance this can led to a warning ? the behavior is understandable, but it is not intuitive. So having a warning shown on every  feels like it would help a lot, what do you think ?

If you run templ fmt it should auto-fix
 to
 (although it would take a long time first time time).
If you do <input>
</input> or similar, you should get a "void element  should not have child content" warning in the LSP warnings, e.g.:

Cool... I was able to get the message you show in the screenshot. but I dont get any if I just put a  without any closer...
Also templ fmt takes long as you mentioned
(never actually completed) so it is kind of not applicable here
Out of your comments from yesterday I feel that errors or warnings are the way to go. if the parser has difficulties with this kind of scenario that is fair, I just feel it would be great if the lsp could warn for that early :-)
(Amazing work on this project btw, finally a templating system for go that is mostly enjoyable to use)

In the PR, I think I've sorted the performance out. If you have time to build templ in the PR and try it out, that would be great.
The "install-snapshot" task in the README does the required stuff.

Awesome, I tested this on my little repo with the snapshot and it performs as expected, thanks a lot for the quick fix :-)


Dark Mode Footer: White Text and Background Causing Visibility Issue

Description
When the site is switched to dark mode, the footer section displays both the text and the background in white, leading to a lack of visibility for the content. This makes it difficult for users to interact with the footer in dark mode.
Steps to Reproduce

Switch the site to dark mode.
Scroll to the footer section.
Observe that both the text and the background are white.

Expected Behavior
In dark mode, the text in the footer should be a dark color (e.g., black or grey) while the background should remain a contrasting dark shade to ensure proper visibility and readability.
Actual Behavior
Both the text and background of the footer are white, leading to no visible content.
Screenshots

Additional Information
No response
What browser are you seeing the problem on?

[X] Firefox
[X] Chrome
[ ] Safari
[ ] Microsoft Edge

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

than you brother for assigning me this issue

Its been 2 weeks since i assigned you this issue i am closing this as its nkt completed


#212 fixed warning from PyCharm on FromComponent

Let's keep it open for a while in case Pycharm developers fix it on their end

Let's keep it open for a while in case Pycharm developers fix it on their end

These warnings have kept on several month yet. I don't think Pycharm devs are planning to fix it

@Forceres I also think JetBrains won't fix this issue soon. For example, this issue has existed for more than 2 years.
https://youtrack.jetbrains.com/issue/PY-54560/Support-PEP-681-typing.dataclasstransform

I'll give you another example. https://youtrack.jetbrains.com/issue/PY-20927/Support-standard-types-for-AsyncIterable-and-AsyncIterator-from-typing-and-collections.abc


Try adding the docker image to the repo

Instead of the organization

Cannot be done, creates an image named dai-lab-03/streaming-app instead


Replaced tntsearch with the laravel scout tnt driver

Should fix #195 since its using a native laravel implementation now
Whilst a new .env file variable can be set to override the search driver should the user want to use another search provider, by default it'll default to tntsearch even if the .env file hasn't been updated.
There should no breaking changes with this patch

Thanks for doing this @talvbansal! Does this mean that the application will no longer write the tags.index and posts.index files? Or does that still work the same?

Its literally exactly the same since the same backend driver is used, this just automatically updates the index files when new items are added or updated. :)

So the only thing that concerns me is that the unit tests take almost 5x the amount of time to complete as before. They used to take roughly 6-7 seconds, now they average about 25 seconds. Not sure if somehow this PR is affecting their routines. Thoughts?

Wow, i hadn't even noticed! Mine just took 6mins to run!! When i was testing these models in Easel i didn't get this sort of slow down, any idea what you might be doing differently?

@talvbansal Not sure at this point. I know that @RDelorier and @erikgall have some good experience in PHPUnit testing, perhaps they can offer some insight?

If you're updating the index on every model save then the migrations will probably cause that to run a lot, you can disable the auto indeed in test mode to help speed it up

You can also try switching TNT database to in memory, check the phpunit.xml for an example of :memory:

Fatal error: Uncaught ReflectionException: Class log does not exist in /Users/rdelorier1/Repos/canvas/vendor/laravel/framework/src/Illuminate/Container/Container.php:734
Stack trace:
#0 /Users/rdelorier1/Repos/canvas/vendor/laravel/framework/src/Illuminate/Container/Container.php(734): ReflectionClass->__construct('log')
#1 /Users/rdelorier1/Repos/canvas/vendor/laravel/framework/src/Illuminate/Container/Container.php(629): Illuminate\Container\Container->build('log', Array)
#2 /Users/rdelorier1/Repos/canvas/vendor/laravel/framework/src/Illuminate/Foundation/Application.php(697): Illuminate\Container\Container->make('log', Array)
#3 /Users/rdelorier1/Repos/canvas/vendor/laravel/framework/src/Illuminate/Container/Container.php(849): Illuminate\Foundation\Application->make('log')
#4 /Users/rdelorier1/Repos/canvas/vendor/laravel/framework/src/Illuminate/Container/Container.php(804): Illuminate\Container\Container->resolveClass(Object(ReflectionParameter))
#5 /Users/rdelorier1/Repos/canvas/vendor/laravel/framework/src/Illumina in /Users/rdelorier1/Repos/canvas/vendor/laravel/framework/src/Illuminate/Container/Container.php on line 734
PHP Fatal error:  Uncaught ReflectionException: Class log does not exist in /Users/rdelorier1/Repos/canvas/vendor/laravel/framework/src/Illuminate/Container/Container.php:734
Stack trace:
#0 /Users/rdelorier1/Repos/canvas/vendor/laravel/framework/src/Illuminate/Container/Container.php(734): ReflectionClass->__construct('log')
#1 /Users/rdelorier1/Repos/canvas/vendor/laravel/framework/src/Illuminate/Container/Container.php(629): Illuminate\Container\Container->build('log', Array)
#2 /Users/rdelorier1/Repos/canvas/vendor/laravel/framework/src/Illuminate/Foundation/Application.php(697): Illuminate\Container\Container->make('log', Array)
#3 /Users/rdelorier1/Repos/canvas/vendor/laravel/framework/src/Illuminate/Container/Container.php(849): Illuminate\Foundation\Application->make('log')
#4 /Users/rdelorier1/Repos/canvas/vendor/laravel/framework/src/Illuminate/Container/Container.php(804): Illuminate\Container\Container->resolveClass(Object(ReflectionParameter))
#5 /Users/rdelorier1/Repos/canvas/vendor/laravel/framework/src/Illumina in /Users/rdelorier1/Repos/canvas/vendor/laravel/framework/src/Illuminate/Container/Container.php on line 734

well shit

zzz had to do composer update instead of composer install

I found a way for the scout driver to be set to null in testing mode but its just as slow as before :\

I can get the tests to speed up but it's going to take a little work, I'll do it as a separate pill request tho

As @RDelorier said, the more times you touch/have to use the database the slower your tests are going to be. That is why in my eloquent-phpunit package I allow you to turn off database seeding, which makes the tests run around 3x faster.

there must be seperate pill requests

conflicting files must be recreated

Could it be possible to implement Scout, but let the user choose the driver?

@Jeroen-G absolutely, we're just shipping with TNTSearch out of the box so that everyone can use it initially with no need for access to a third party.
To change the scout driver just change the SCOUT_DRIVER value in your projects .env file.
I haven't forgotten about this PR. I'm hoping to find time to amend the code as per Taylors recommendation here

Forgot to comment on this, the issue with slow unit tests should be resolved now!

Awesome thanks @talvbansal!

@talvbansal Question about the composer.json file. If we are using Scout/TNT search driver, do we need the original TNT search package to still be pulled in? Or is this a drop-in replacement?

Ahh good spot, it should be able to be removed with no side effects the scout driver will automatically pull the package in as a dependency 😀

Done! :)

@talvbansal 2 things I noticed when pulling this down locally:

The sidebar navigation is gone on the admin/post/create page, not sure if other places as well

On the Tools page, there is a Reset Index function so that users can manually reset the index if they so choose. It throws a Scout error, wonder if you could look into that?

After that, I think it's ready to merge!

Not sure why the blank bar on the screen shot has happened, it works fine for me, have some of your assets been cached? If you clear them do you get the same issue?
So I looked into the reset index issue for scout and came across this within the scout code base:
    if ($this->app->runningInConsole()) {
            $this->commands([
                ImportCommand::class,
                FlushCommand::class,
            ]);
    ....

So those artisan commands are only registered when the application is running on the console. Which to me makes sense since they're so infrequently used that it makes no sense to register them for every request.
At this point since scout is a first party laravel product and we've opened up the option for multiple drivers to be used as a result of its integration the question becomes: is having the reset index tool one that is likely to be used a lot?
Models are auto indexed when they're updated now so the only reasonable case i can think for this is if you suffered some form of index corruption. If that were the case im sure you'd be on the command line trying to work out whats wrong anyway.
Just my thoughts on this.

I think you can fire console commands from a http app. Not sure how, but have seen it somewhere.

You can make artisan calls but only if the commands are registered, the above code shows that the commands are only registered when the app is run in the console.
But realistically how many people have reset indexes from the front end

@talvbansal Yup I do agree with you on that. It's probably very uncommon, I can take it out, especially if it's going to be happening automatically. For some reason still running into the blank sidebar issues, and also some JS stuff not working. Tried clearing out the asset caches, re-compiling everything but the issues still remain.

Sorry i was testing this out on another branch im working on, you're totally right, fixed in the latest commit and the references to resetting the index has been removed along with the unit test for it.

Awesome @talvbansal thanks for pushing all those commits. I think the internet is broken today because of all the DDOS attacks, so my merging this in may wait till everything is a little more stable. But really great work 👍


How to find the visible order of the next DockContent?

Is there any way to get the correct visible order of the tab to the direct right of the active one?  A user of my application asked that if he closes an active tab, the tab to the immediate right of the active tab should take focus.
So, if I close New_03, New_04 should take focus:

The code I'm using below works fine except when the user has moved the tabs around from their "natural" order:
Before:

After:

I've seen this, but it doesn't help because it explains how to sort tabs in a desired order... I'm trying to find out what the current visible order is, not sort the tabs.
Here's what I'm currently doing:
private IDockContent GetNextDocument(IDockContent startingItem, bool forward = true)
{

	var editors = MainDockPanel
						.Contents
						.AsQueryable()
						.Where(dockContent => dockContent is IMyEditorView)
						.ToList();

	int valToIncrement = forward == true ? 1 : editors.Count - 1;

	for (int x = 0; x < editors.Count; x++)
	{
		DockContent dockContent = (DockContent)editors[x];

		if (dockContent.Equals(startingItem))
		{
			int nextDoc = x;
			do
			{
				nextDoc = (nextDoc + valToIncrement) % editors.Count;
				dockContent = (DockContent)editors[nextDoc];

				//If we've gone through 1 full cycle, quit
				if (nextDoc == x)
					return null;

			} while (dockContent.IsHidden);
			return dockContent;
		}
	}
	return editors.FirstOrDefault() as IDockContent;
}

...which is a modified version of this.

You referred to #270, and I think it also has some relationship with #341 .


python3Packages.grandalf: 0.6 -> 0.7

Motivation for this change
Update to latest upstream release 0.7
Switch to pytestCheckHook
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 155094 run on x86_64-linux 1

1 package failed to build:

python310Packages.scmrepo

19 packages built:

dvc
dvc-with-remotes
python310Packages.aiohttp-retry
python310Packages.dictdiffer
python310Packages.flatten-dict
python310Packages.grandalf
python310Packages.mailchecker
python310Packages.python-benedict
python310Packages.python-fsutil
python310Packages.shtab
python39Packages.aiohttp-retry
python39Packages.dictdiffer
python39Packages.flatten-dict
python39Packages.grandalf
python39Packages.mailchecker
python39Packages.python-benedict
python39Packages.python-fsutil
python39Packages.scmrepo
python39Packages.shtab

gevent doesn't work with Python 3.10 at the moment.

nixpkgs-review ... --package-regex="python39" passes on macos 10.15 (everything python 3.10 seems broken on macos at the moment)
nixpkgs-review on nixos x86_64 and aarch64, only python310Packages.scmrepo fails

@ofborg build dvc


server: warn on admin mutations, GET requests, add mocking of messages

Description:
When a mutating admin-console endpoint is reached, verify that it's a POST.  For now, the operation succeeds but a warning log is printed.
To test this, log-mocking needed to be added, which was more involved than just checking the POST.  The logging sink sent to spdlog::logger now delegates to another sink, which can be changed during tests, and mocked.
This is a step toward fixing https://github.com/envoyproxy/envoy/issues/2763
Risk Level: Low
Testing: //test/...
Docs Changes:

Link to Data Plane PR]
[in progress, will update]

Release Notes: [in progress, will update]

If this change is user impacting you must add a release note via a discrete PR to
version_history.rst.

[Optional Fixes #Issue]
[Optional API Changes:]

Link to Data Plane PR]

Deprecated:
Deprecates supporting admin mutations with a GET.

per discussion with @htuch I added a snippet to the style guide about testing log warnings.

@jmarantz Would you please break this into 2 separate PRs, one for the logging changes, and one for the admin change?

@ggreenway sure -- I was thinking the logging changes would be dead code without the admin change, but I guess it's a pretty significant refactor on its own.  Will do.

 If so, I have no idea how to fix that...

egads -- I don't know how to solve that easier without erasing your comments by rebasing.  Actually i don't really know how to solve this with rebasing either :)
In any event, I have started #2930, with just the log infrastructure changes and new tests.  I will close this PR, at least temporarily, until that one is merged.


[Documentation]: Doc page 404

Is there an existing issue for this?

[X] I have searched the existing issues

What kind of documentation would you like added or changed?
the docpage on the homepage is 404.
https://github.com/towhee-io/towhee/blob/main
Why is this needed?
No response
Anything else?
No response

/assign @filip-halt


Update to use compile instead of build command

Update locations on Travis and Makefile, so don't get deprecation warning on the build.
I didn't change some references to work "Build the compiler", including the build target because I wasn't sure about the convention and was concerned about changed the semantics of the sentence.

Thanks! We can probably merge this after releasing 0.18.0. The problem is that the current compiler doesn't support this so it won't work. Problems one has to face with a bootstrapped compiler :-)

@asterite hahaha I tottaly forgot about this detail, seeing the build I saw your comment in practical :)


RocksDB version 6.26

Scope & Purpose
Enterprise companion PR: https://github.com/arangodb/enterprise/pull/785
This PR upgrades the bundled version of RocksDB to 6.26 (from 6.8).
For now, this should be a devel-only change. We may later decide to backport the upgrade if it proves to be stable in devel.
(in contrast to the branch name, the upgrade is actually to 6.26, not 6.25 - this is due to 6.26 being released after the upgrade branch was created)

[ ] :hankey: Bugfix (requires CHANGELOG entry)
[ ] :pizza: New feature (requires CHANGELOG entry, feature documentation and release notes)
[ ] :fire: Performance improvement
[x] :hammer: Refactoring/simplification
[x] :book: CHANGELOG entry made

Backports:

[ ] Backport for 3.9: (Please link PR)
[ ] Backport for 3.8: (Please link PR)
[ ] Backport for 3.7: (Please link PR)

Related Information

[x] Docs PR: https://github.com/arangodb/docs/pull/794
[x] Enterprise PR: https://github.com/arangodb/enterprise/pull/785

Testing & Verification

[x] The behavior in this PR was manually tested
[x] This change is already covered by existing tests, such as virtually all tests.

I pushed a small change that synchronises CMakeLists.txt between RocksDB and arangod;s RocksDBEngine;
The old CMakeLists set -DROCKSDB_LITE (because it thought Darwin on arm means IOS and that means, hey, ROCKSDB_LITE for reasons.)


Add tests to the new default session.

#245
Hey @mrshu, I added some tests, updated the tutorial and I think that I found a more elegant way to use the boto3 default Session.
What do you think?
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@igorborgest I am honestly torn on whether accessing DEFAULT_SESSION is better than depending on a private method, but this may be at least a bit more clear.
That said, this looks good to me -- thanks a ton for adding the tests and updating the tutorial as well!


Update smallweb.txt

Added my own blog and two additional blogs as per the rules.

Please sort them to correct position

@vprelovac sorted!

They are not properly sorted and one of them is a duplicate

I'm sorry, @vprelovac! I messed up again. I hope I've sorted everything this time. Thanks for your time and patience.


KAFKA-13133 Replace EasyMock and PowerMock with Mockito for AbstractHerderTest

Development of EasyMock and PowerMock has stagnated while Mockito continues to be actively developed. With the new Java cadence, it's a problem to depend on libraries that do bytecode generation and are not actively maintained. In addition, Mockito is also easier to use.KAFKA-7438
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@kkonstantine @rhauch It would be great to have a review from one of you two so that we can get this merged.

We should also remove this test from https://github.com/apache/kafka/blob/trunk/build.gradle#L381 before we merge this PR.

@wycccccc are you planning to address the comments?

Closing in favor of #12473.


Add savefig loss

Codecov Report
Attention: Patch coverage is 41.66667% with 7 lines in your changes missing coverage. Please review.

Project coverage is 34.18%. Comparing base (d42f0f5) to head (dcaa4d8).

Files with missing lines
Patch %
Lines

src/checkpoints.jl
0.00%
7 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #131      +/-   ##
==========================================
+ Coverage   34.14%   34.18%   +0.04%
==========================================
  Files          12       13       +1
  Lines         495      506      +11
==========================================
+ Hits          169      173       +4
- Misses        326      333       +7

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


IOS

hi there, i m built the app successfully for the android, while building for IOS i m stuck with the error "AppAuth.h" file not found. can you please help me out to resolve this issue.

when i build xcode project from unity, this file "com.elringus.unitygoogledriveios.mm" in plugins folder throwing error "AppAuth.h" file not found

Most likely you didn't install the required native dependencies, please read the readme carefully.


What context is passed to the model?

I'm loving CodeCursor. I am having a hard time understanding what exactly is getting passed to the model when I click Ask.
It seems that sometimes, the entire active file is passed as context, and sometimes only half the file. Is there any capability to have context from multiple files? Or a whole project?
I notice that my active selections are also passed. Is there any other context that is passed when I have an active selection? What is that logic?
Maybe it can be added to the readme, as it's important information to know how to use this.

Why are you so interested in the context ? Bring you unexpected response ?
I take a quick look at the source code, and only offer you these message according your Question:

pass the entire active file as context, not whole project;
active selections always should be passed, maybe follow the Cursor App action in this way;
the text before and after your selection also will be passed;
sometimes only half the file will be passed is not a normal action, may be a bug of CodeCursor or vscode API, you should provide more message, such as screenshot, http response and etc.

From my perspective, it's not necessary  to add these message to the readme, it doesn't block people using CodeCursor, but I feel it's better to make a new manual handbook in this project, put these message into this file.
I hope this could help you.
If your confusion is resolved, please close this question, thank you.

Knowing the information that was passed to the model is important to know what questions I can ask it. It is still not clear to me exactly what is passed. There is likely a size limit when asking questions on an entire file, what is that size limit?

Unlikely, considering the context length of GPT4 is 8000 tokens.

To tell the truth, I didn't know the context length of GPT4 before, now I know, but it's redundant for me. In my daily work, I usually use at most 300 tokens.
This plugin is based on Cursor App, so if you want to get more information about the context length and other limits, I think you'd better ask the maintainer of Cursor App.

Hi @tonydavis629, currently we don't have any idea how Cursor processes your inputs. The API request will encapsulate the whole document of your active tab and the selection (if any). They are sent to Cursor server as is along with your prompts. For more model-related questions, I would suggest you file an issue in https://github.com/getcursor/cursor. Thanks for your supports.


Unable to create website. Certificate error.

Put https://my.domain.s3.amazonaws.com/: x509: certificate is valid for *.s3.amazonaws.com, s3.amazonaws.com, not valkyrie.systems.s3.amazonaws.com

Results in.
2018/06/09 16:52:52 HOST s3.amazonaws.com
2018/06/09 16:52:52 HOST
Creating Bucket
Error creating S3 bucket
Put https://valkyrie.systems.s3.amazonaws.com/: x509: certificate is valid for *.s3.amazonaws.com, s3.amazonaws.com, not valkyrie.systems.s3.amazonaws.com

uname -a
Darwin Bradleys-MacBook-Pro.local 17.5.0 Darwin Kernel Version 17.5.0: Mon Mar  5 22:24:32 PST 2018; root:xnu-4570.51.1~1/RELEASE_X86_64 x86_64

Maybe related to #49

For me I get
Error creating S3 bucket (or) Error uploading
The request signature we calculated does not match the signature you provided. Check your key and signing method.

Downgrading to v1.3.1 resolved the issue.


Product Progress

First of all great work! I am building an integration for a Jura X8 machine with our own payment system and could use a lot of your work. Thank you!
Do you maybe have more information about the product progress data? I'd like to find out when a cleaning program is started and so far couldn't figure what the individual bytes in the reply of the machine mean.

Thank you for the kind words.
Regarding the cleaning program I think you can detect it with an alert using the get_alerts function but I did not added them yet since I was not fully sure. The product progress I am not sure it detects the cleaning but it might. Next time I am with the machine I will test to see if I can find something useful and let you know.

In the detection of the product progress you test byte #2 for ["3e" or "00"] and then use byte #3 for the chosen product. I've noticed that during the different cleaning processes, byte #2 is often "ff" and then a product is detected (in my case its int(33) or int(34)). However, I am not sure if "ff" has a specific meaning here. Need also to do more testing and logging.

Hello @karimj, sorry for the lack of feedback in the past month. I have been away from the coffee machine. I plan to start next week to do more tests and hopefully get more information regarding the cleaning programs. I think if you leave it reading from product progress and start the different programs, hopefully, a pattern will emerge that we can control.

@karimj, sorry for no new, did you get any progress on this? I would be glad to do some tests to help to find some pattern.


Return null for date_trunc(null) instead of panic

Which issue does this PR close?

Closes #6701.
Rationale for this change
see #6701

What changes are included in this PR?
Changed unwrap() for unwrap_or_default().
I've tried to use nano.map(|nano| truncate(nano)) like discussed in the issue. But I'm not sure to understand how I'm suppose to use it.

Are these changes tested?
yes

Are there any user-facing changes?
no

I know this push doesn't work, but I really don't get it. Now the unwrap() error is gone but I get this error.
Running "timestamps.slt"
External error: query failed: DataFusion error: simplify_expressions
caused by
Internal error: Optimizer rule 'simplify_expressions' failed, due to generate a different schema,
original schema:
DFSchema { fields: [DFField { qualifier: None, field: Field { name: "date_trunc(Utf8(\"year\"),NULL)", data_type: Timestamp(Nanosecond, None), nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} } }], metadata: {} },
new schema:
DFSchema { fields: [DFField { qualifier: None, field: Field { name: "date_trunc(Utf8(\"year\"),NULL)", data_type: Null, nullable: true, dict_id: 0, dict_is_ordered: false, metadata: {} } }], metadata: {} }.
This was likely caused by a bug in DataFusion's code and we would welcome that you file an bug report in our issue tracker
[SQL] SELECT DATE_TRUNC('year', NULL);
at tests/sqllogictests/test_files/timestamps.slt:840

You are pretty close to finish this.

Just a format issue:
             if nano.is_none() {
-                return Ok(ColumnarValue::Scalar(ScalarValue::TimestampNanosecond(None, tz_opt.clone())));
+                return Ok(ColumnarValue::Scalar(ScalarValue::TimestampNanosecond(
+                    None,
+                    tz_opt.clone(),
+                )));
             }

You can run cargo fmt locally or use it in your editor plugin so the code will be formatted automatically.

Thank you for your help, I really appreciate it

Thank you @BryanEmond


Accuracy Metrics

Hi I see that you are using the 'accuracy' metric in the neural networks.
When dealing with multilabel classification does it mean that if any one of the tags are correct, the prediction is treated as correct? i.e. Is the following case considered a correct prediction:
Truth label: [0, 1, 2]
Predicted label: [2, 3, 4, 5]
Would it be better for a custom metric like F1 score?

We use Keras' metrics, so in this example it will not be treated as fully correct - only as 1/6 correct, because just one out of 6 labels was correctly predicted. Here's the code for that.
But you're right, that there's no good reason for using accuracy in this case. You can compute custom metrics using the callbacks in the train function, but Magpie should probably provide an additional metric on top of the loss. F1 score sounds like a good suggestion.

Do you mean 1/4? Since only 4 labels are predicted.
If I'm not wrong, the binary_accuracy takes in a one hot vector for the truth labels. So in the example it would be:
Truth label:  [1, 1, 1, 0, 0, 0]
Predicted label after rounding: [0, 0, 1, 1, 1, 1]
Since only one is correct. K.mean gives us number of correct / number of classes = 1/6 (Is this what you were implying at the start?)
This is kind of weird because even if the prediction exactly matches the truth labels we would only be getting 3/6?

Oh wait my mistake. I forgot that the 0's are also considered as correct predictions. So exact match will give us 6/6.
Thanks for the help


Fix deps.

A few deps were listed as devdeps.

@noffle seems like your branch was out of date, mind rebasing?

I pulled in the changes in #51

Closing in favor of #51

Rad -- thanks.


Bump up to v0.3.3

I'll approve this after #105 is merged

@vbkaisetsu Ah, sorry. I was mistaken about https://github.com/daac-tools/vibrato/pull/105. I see.


AttachContext overrides previous AttachContext

Hi,
I have a scenario where my REST api service calls 2 middlewares. both of them use the AttachContext before calling ctx.Next().
After a long debug I realized that the second AttachContext is overriding the first one since the AttachContext key is the same one generated here .
Is this intentional?
Thanks in advance

Hi @ayeletdeRoos,
Yes, it is. The point of AttachContext is to save a second context in the RequestCtx, but only one! If you need to save more than one, you can create a new one from the first context, and reattach it to the request!
Something like this:
// NOTE: ctx => atreugo.RequestCtx

myCtx := ctx.AttachedContext()
myBothCtx := context.WithValue(myCtx, <key>, mySecondCtx)

ctx.AttachContext(myBothCtx)

When you attach the last context, all values are accesible via ctx.Value(...)

Hi @savsgio ,
Right, I did a similar workaround - thanks!
I do think a check for this "edge" case can be helpful (if a key is overridden in the ctx)
Thanks again for your reply :)