Build fails including Logger.h

Using Xcode / clang, the build fails when including Logger.h, because there is a log level named DEBUG (build error is "Expected identifier" because XCode defines a DEBUG macro per default for the debug target). Renaming it to e.g. TDEBUG fixes the issue:
enum Level
{
TDEBUG = 0,
INFO = 1,
WARN = 2,
ERROR = 3
};
IMHO, the best would be to avoid DEBUG and rename the enums to something like LOG_DEBUG, LOG_INFO etc. Even though it will mean a breaking change in some code.

@benstadin I think it would indeed be fine to update these enum names


The process can't exit when use ocx in .NET(C# exe) or IE or microsoft webbrowser

thanks!

I kind of created it for VB6 and it has some VB6 related hacks. You might try to un-hacking DllCanUnloadNow in the code, it should work.


[Bug]: Missing Profile Names

What happened?
I am not seeing drop downs in the profile list for the following Show Types / LTM's. I know they keep adding new ones.

4th Anniversary Party
Frosty Frolics

Platform
None

We can add 4th Anniversary Party in the list but Frosty Frolics is a "creative rounds only" show => All these shows are merged in Fall Guys Creative.


Moving string parsing logic from CliOptions into string_util

For a more logical structure, and also because this can be re-used for setting knob values with textbox inputs in the UI.
Other changes

Added Split() helper function
Added support for comma-separated values

Thanks for the changes! Just wondering, was the change to include KnobDemo intentional? Since it wasn't there at first.

Thanks! I removed Split(), changed logic for SplitInTwo() and made the callers validate the result more, and updated the PR description. I think this is ready for merging


Improve search memory usage

Add a bytebuffer pool to improve performance and allow bounding of total active bytebuffers to constrain total memory usage.
Eliminate unnecessary storage of primitive reference values where they are only used to eval a parent function, e.g. substring.
Make countDistinct use HyperLogLog by default so there is no longer a need to store all values. It should be possible to force exact counts but perhaps limit how many items can be counted exactly. Exact counts could still use hashes if we think they are likely to be unique enough.
Add another flavour of joining that will concatenate unique values but only produce a string up to the max string length. Maybe this should be called joiningUnique.
Related to issue:  #3500

Also take a look at the way data is transferred to a table via the getData method in LmdbDataStore. 1000's of ItemImpl objects are added to the list without being needed by the consuming table.


add find_data_series and crop modeling methods

I think these look good to me. I think it would be much better for find_data_series to include an example input. I don't think it's very clear how to construct a query with just what's presented here.
@itsjohnward I think you said that's possible, but might take a bit of extra work to make sure the example isn't treated as a doctest

The way he did it here https://github.com/gro-intelligence/api-client/pull/140/commits/60dc0e9bdeae4ad19d6b4488f5182479a2f55364#diff-603e79178b8e522d0e366a9fca477b24R192 works. Would be picked up as a docstring if it were formatted like
>>> function(input)
output

But this isn't recognized as a valid docstring:
Example::
  function(input)

Outputs::
  output


Dependency com.barchart.udt:barchart-udt-bundle:2.3.0 too old

LittleProxy cannot be used on MacOS anymore, if you want to sign your executable.
notarytool complains that a binary uses an SDK older than the 10.9 SDK. The binary in question is libbarchart-udt-core-2.3.0-SNAPSHOT.jnilib which comes from the dependency com.barchart.udt:barchart-udt-bundle:2.3.0.
The project looks abandoned https://github.com/barchart/barchart-udt and I can't find an active fork.
A workaround may be to fork the project. rebuild the binaries and exchange the old dependency with the newly forked project.

@ChristianFischerJena I see that UDT support was removed in Netty 5.+ exactly for the same reason:

@deprecated The UDT transport is no longer maintained and will be remove

See https://stackoverflow.com/questions/56838184/why-is-udt-transport-deprecated-in-netty-4-1
So I suggest to remove UDT support from LittleProxy as well.
I personally don't use it, and I don't anybody who would use it.

@ChristianFischerJena Ive removed UDT support in https://github.com/LittleProxy/LittleProxy/pull/488
We can release LittleProxy 2.3.0 if you need it.

@asolntsev Thank you very much for your help! A version 2.3.0 with the dependency removed, would be very nice.

@ChristianFischerJena I've released LittleProxy 2.3.0, please try.


Downloading test data fails

From the download page on the website I can not download any of the sample models such as the armadillo.vdb.  The error message is:
<Error>
NoSuchKey
<Message>The specified key does not exist.</Message>
<Key>
releases/io/aswf/openvdb/models/armadillo.vdb/1.0.0/armadillo.vdb-1.0.0.zip
</Key>
<RequestId>C13281DD73D8E125</RequestId>
<HostId>
pFPxssALt+v4bL3FjyU+cdyP9FcjXePhmMiZhUOtjeLaB7/7sFIx4KhmCnJxt6uNfeckSQiHOcg=
</HostId>
</Error>

Can the links be fixed or can you provide an alternative download location?

Apologies for the inconvenience, we’re aware of this issue and are looking into it.

The following PR should resolve this:
https://github.com/AcademySoftwareFoundation/openvdb-website/pull/24

it didnt work for me. Can you say what exactly i should do please?

This has now been resolved and the download links should work again. Thanks for your patience.


Allow direct of upload/media manager for article image url

Allow direct upload or choose existing image for media manager.

Hey! We already have an issue about this feature: https://github.com/v17development/flarum-blog/issues/5
Hope we can implement this soon though :)
I'll close this issue for now


Is a HeaderContext to validate headers useful?

Hi there,
Can I make a pull request adding an HeaderContext with methods to validate headers?
Thanks!

Hi,
In fact we have already this kind of assertion in the behat2 branch :
https://github.com/rezzza/rest-api-behat-extension/blob/behat-2.x/src/RestApiContext.php#L108
We just currently never backported it on the master. So feel free to contribute ! I think we could add it to the RestApiContext, no need to create a new context.


[bug] spinner clear() leaves artifacts when used in a procedural input flow

More of a nit than anything, but I think this gif makes it pretty clear what's being left behind. Ideally I would like it to be as-if the spinner never existed:

@cylewitruk The fix turns out to be pretty simple: #69

@cylewitruk The fix turns out to be pretty simple: #69

Rediculously simple 😅 Good find, thanks!


[Feature request] Edit .bat files using windows cotext menu

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
Would be useful to be possible to select Notepads to edit .bat files. At the moment, when I right click a .bat file, there's no way to assign Notepads to edit
Describe alternatives you've considered
I tried to modify the registry in order to change the defaut edit software, but I can't assing notepads that way cause the software is PWA, so I couldn't finde the .exe file.
Additional context
NA

This is a known security limitation of UWP apps. This is not allowed by Windows 10. Nothing I can do here.

I tried to modify the registry in order to change the defaut edit software, but I can't assing notepads that way cause the software is PWA, so I couldn't finde the .exe file.

Which registry path did you use??

\HKEY_CLASSES_ROOT\batfile\shell\edit\command

Closing this since it is restricted by UWP app model policy.


Fix copy alpha type mismatch in FloatCLUTEval

One more alpha channel bug that somehow slipped my testing when preparing PR #271.
A missing type cast caused a blank canvas in Krita when using float color space in combination with a CLUT display profile, so still related to issue #247.
All good things come in threes...

Thanks for all the effort you are putting on this.


Incorrect device type Zemismart Roller Driver via Deconz

Zemismart Zigbee Roller Blind Driver (M515EGBZTN) connected via Deconz shows in Phoscon as a light (but useable to open and close blind), however homebridge Hue exposes as a Not Supported Battery device.
I cant find any known issues, although there are some over on the Deconz GitHub regarding the reasons Phoscon sees the device as a bulb.
homebridge-hue.json.gz

You don't expose /lights resources.  The blind is exposed as a /lights resource (except for the battery).

Ah yes! Now it appears as a blind in homekit and working, however, it is showing zero battery percentage and a low battery warning. Anything I can do about this?

deCONZ hasn’t received any battery percentage and reports it as 0 (according to your logfile).  You’ll have to ask over there why this is the case.

Thanks, i have asked, however, no solution at the moment. Any chance to incorporate a setting to hide battery level or make it show a custom value?

Don’t expose the ZHABattery resource through excludeSensorTypes In config.json or by creating a blacklist resourcelink.  See Wiki.

Thank you! Ive used excludesensortypes for now and working well to hide my blind battery!! In fact, all my other zigbee door/window sensor battery levels are still being reported to HomeKit. Is that how it should work?

Yes, for those the battery is exposed on the /sensors resource, only the blinds, which are exposed as /lights resources have a separate ZHABattery.

Ah ok. Thank you for the explanation. Great stuff!

On 21 Jun 2021, at 16:36, Erik Baauw @.***> wrote:

Yes, for those the battery is exposed on the /sensors resource, only the blinds, which are exposed as /lights resources have a separate ZHABattery.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Multiple authors squished together in dc: creator field

Attach (recommended) or Link to PDF file here:
http://mariovalle.name/SciViz/1-s2.0-S0097849310001846-main.pdf
Configuration:

Web browser and its version: Using pdf.js from Node.js through npm package pdfjs-dist
Operating system and its version: Windows 10 64bits
PDF.js version: 2.6.347
Is a browser extension: No

Steps to reproduce the problem:

Load the file in acrobat
In document properties look at the Author field: it is Samuel Silva; Beatriz Sousa Santos; Joaquim Madeira
Look at  Additional metadata > advanced > dc:creator. It is marked as seq container and contains the 3 authors.
Using the code below gives: "Author": "Samuel Silva" that is, only the first one. And "dc:creator": "Samuel SilvaBeatriz Sousa SantosJoaquim Madeira", that is the three authors squashed together without separators.

What is the expected behavior? (add screenshot)
In both fields the three authors as a single string with some sort of separator between them. Note that other pdf files that contains multiple authors already have them as a single string with names separated by commas.
The code used:
var pdfjsLib = require("pdfjs-dist/es5/build/pdf.js");
var pdfPath = "http://mariovalle.name/SciViz/1-s2.0-S0097849310001846-main.pdf";
var loadingTask = pdfjsLib.getDocument(pdfPath);
loadingTask.promise
  .then(function (doc) {
    var lastPromise;
    lastPromise = doc.getMetadata().then(function (data) {
      console.log(JSON.stringify(data.info, null, 2));
      if (data.metadata) {
        console.log(JSON.stringify(data.metadata.getAll(), null, 2));
      }
    });
  });

Using the code below gives: "Author": "Samuel Silva" that is, only the first one.
console.log(JSON.stringify(data.info, null, 2));

That data comes from the Author field in the /Info dictionary of the PDF document, and that entry only contains one name; hence this part works as expected.

And "dc:creator": "Samuel SilvaBeatriz Sousa SantosJoaquim Madeira", that is the three authors squashed together without separators.
console.log(JSON.stringify(data.metadata.getAll(), null, 2));

Starting with PDF.js version 2.7.570 (fixed by PR #12838), please see https://github.com/mozilla/pdf.js/releases/tag/v2.7.570, the dc:creator now returns an Array; hence this issue is already fixed.

Perfect! When it will be available as npm package?
Less important, but the same problem seems to happen with "dc:subject" field in the provided file. In this case the "Keywords" field is correct,

Perfect! When it will be available as npm package?

https://www.npmjs.com/package/pdfjs-dist?activeTab=versions

but the same problem seems to happen with "dc:subject" field in the provided file.

See the PR linked above.

For future reference: Please open separate issues for different bugs, since tracking multiple different issues in one bug can make tracking things somewhat difficult in general.
Also, if possible, try and test against the development version of PDF.js, see e.g. https://github.com/mozilla/pdf.js#online-demo, since we generally don't backport fixes to already existing releases.


Showcase Submission for 178-climate-spiral

Thank you Grem for your contribution! A member of the Coding Train team will review it shortly.

Robo MO
Grem

Thank you for moving this @fturmel!


[performance] RxList's methods lead to rerender

Is your feature request related to a problem? Please describe.
When I attempt to modify an RxList using methods( e.g. List.add, List.removeWhere), it always causes all items to be re-rendered.
Describe the solution you'd like
Is it possible to update only the changed items? Thank you.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

This issue is unverified and requires further validation. It has been temporarily closed.


(wip) add informational popover base component and sample

What type of PR is this? (check all applicable)

[ ] Refactor
[x] Feature
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update
[ ] Community Node Submission

Have you discussed this change with the InvokeAI team?

[x] Yes
[ ] No, because:

Have you updated all relevant documentation?

[ ] Yes
[ ] No

Description
Adds a new common component IAIInformationPopover that composes JSX to be rendered within a popover as a tooltip. We were not able to use the Tooltip component provided by chakra because you cannot interact with elements within those (at least not that I could get working).
This just a sample over positive prompt. We need content from @hipsterusername and @Millu before we can roll this out.
Related Tickets & Documents

Related Issue #
Closes #

QA Instructions, Screenshots, Recordings

Added/updated tests?

[ ] Yes
[ ] No : please replace this line with details on why tests
have not been included

[optional] Are there any post deployment tasks we need to perform?

This is sweet! Feel like this will be a big QoL improvement for folks
Experiencing an issue where I'm having to click on a dropdown to select it, then click again to expand the dropdown, but II can't tell if this is due to the popover or something else
https://github.com/invoke-ai/InvokeAI/assets/7254508/d90f0aab-d531-4c1e-bbf3-f8c7a8918f48

Are you having this issue on the main branch as well? I'm not able to replicate it on firefox or chrome

I'm only seeing it on this branch, unfortunately


Organize git and preliniary content

In anticipation of more tutorial content, I moved the git and Jupyterlab to a dedicated section with an index page that summarizes the learning objectives.
Also separated required from supplemental content under the preliminary section and remove the schedule from the SWC workshop and replaced with a list of covered content.

Just added a small eye-candy checklist to the preliminary as well
https://deploy-preview-37--snowex2022.netlify.app/preliminary/index.html#required-setup


Action for triggering validation repository

This is related with https://github.com/poliastro/validation/issues/5. It adds a github actions file for triggering the validation repository every time a new merge in main branch is done or a release is published.

The branch needs to be created from the actions tab, sorry. Closing this...


New exchange - C-CEX

For now it supports only public data:

Order book
Trades

BTW, to update a PR, you just need to push the new changes. No need to close and resubmit!


Improve the DataTable component

Some discussions already occurred in #105, but I'd like to recap everything here so it's easier to track.
Here are a few things have in mind

[x] Move back the selectable prop inside the DataTable, and also add onSelectionChanged and selectedRows
[x] Create a specific component for TableColumn with props (name, numeric, tooltip, cellFormatter).
[x] Adds a Sortable table
[ ] Fix the selected rows when the table receives new props
[ ] Write tests

The usage of a specific TableColumn, or the columns prop on the table will result in the exact same result. With a real component, it will be easier to set a specific styles, or width, or even to know if the props are valid.
The target is to keep compatibility with the current implementation as well.
cc @janpieterz @FoxxMD

Sweet proposal! Sorry, I've been a bit off of projects using MDL lately.

I started the modifications. I didn't push it yet. I'm still thinking about how to build the Sortable thing. If I do a prop, or a HoC component over the current table component.
I'm almost sure I'll go with the HoC component, so I can keep the "sortable state" out of the current component.

+1 for HoC. I don't know how you are writing the sortable functionality but providing a way to override your default functionality (per-column?) would be super helpful if the data was more complex than a plain value.

I did something, I still have to adjust the tests.
Basically, I created a new component called SortableTable which takes exactly the same props as the default Table, and we can specify a custom sortFn on the table headers.

perfect

Should this just be a prop option for the datatable vs a new component? That's what we're doing for our component library at Inova. Just a thought, not trying to argue anything. You know your repo better than I :)

At first I did it like that, but it also adds a lot of state management in the base table component and I think it's better in this case to have it in another component.
But I agree, because selectable is a prop, maybe it would be more consistent to have sortable as a prop :/

For our component library with MDL, we bounced back and forth with that same problem. I think that's an architectural problem that get's ironed out with maturity. Really, depends on the masses here vs for us, private. I've been enjoying this repo with comparison on how we are doing it here, at Inova. Either way, great work @tleunen

I'm having an issue with the Selectable feature...
When the component receives new props, with new rows.. How should behave the selection? Should we reset everything all the time? It's probably a bad idea, but at the same time, I cannot really know if the row is fully new or not since I only store its id :s I may need to store everything after all :/

I would like to check whether the following improvement will support custom header that required colSpan as the screenshot below?

@zeskysee you mean the row with the "add" and "remove" buttons?
Nop, it won't be part of the table at this stage. But I'm pretty sure you could easily build it outside of it.

@tleunen so you mean add the buttons outside the table?

Yep, with the new component (not released yet), you'll be able to get the selected rows so you can enable/disable the buttons outside of the table

May I know will the new data table (not released yet) will be working if i did not specify any table header?

It won't work without headers. Headers are used to know in which order the data have to be displayed.

After re-read the material design's data tables guideline, it is correct to make the assumption that the header is mandatory

A data table contains a header row at the top that lists column names, followed by rows for data.

Sorry for my ignorance question. Thanks for the quick response and answer :) Looking forward to your new release data table.

Hopefully, it should be this weekend :)

@michaelguild13 I've decided to change the way I was thinking it. So in my latest attempt, both the selectable and sortable are props. And I also put the specific code in each a Selectable and Sortable HoC component so I can keep the initial Table component quite easy to read.
No changes for the end user except now when importing the Table, he will both import the sortable/selectable code, even if he never uses them. But I guess that's ok...

PR merged. Closing this ticket


debugging: forgetting to put resourceId in side

I just spent a while debugging why my app was crashing,
THREE.Object3D.add: object not an instance of THREE.Object3D. T…E.Shape {curves: Array[1], autoClose: false, actions: Array[2], holes: Array[0], userData: Object…}

Even though I was copying the examples. It turned out to be that my <shape resourceId="" /> wasn't in a <resources> tag. Is there any way to give a nice error message if this is the case? if not this tickent can be closed

Thanks for the report, I will try to fix this. Could you please upload a sample repro for it?


Issues with value attribute not being set

I have the following CSHTML:
<input asp-for="Customer.LastName" class="form-control" />
@Html.EditorFor(a => a.Customer.LastName)
Last Name: @(Model.Customer.LastName)

which is rendered as:
<input class="form-control" type="text" data-val="true" data-val-maxlength="The field Last Name must be a string or array type with a maximum length of '25'." data-val-maxlength-max="25" id="Customer_LastName" name="Customer.LastName" value="">
<input class="text-box single-line" id="Customer_LastName" name="Customer.LastName" type="text" value="">
LastName: Bean

Customer is defined as
        public CustomerCreate Customer { get; set; }
and CustomerCreate, partially, as
    public class CustomerCreate
    {

        [Display(Name = "QB_LASTNAME")]
        [RequiredFromStage("CustomerFieldNeeded", 1, ErrorMessage = "ERROR_QB_SURNAME_REQUIRED")]
        [MaxLength(25)]
        public string LastName { get; set; }
    }
}

As you will see the value attribute is not being set for the first two but is present in the POCO object Customer (which is a POCO object of type CustomerCreate - all mine not MVC)
There are no Editor Templates in play for strings.
This is in MVC Core 2.1.6
Has anyone any ideas?
Thanks
Mark

As you will see the value attribute is not being set for the first two but is present in the POCO object Customer (which is a POCO object of type CustomerCreate - all mine not MVC)

How are you setting LastName? Via controller? Form post?

Both in a way!
The form starts off as a view served from a HttpGet method on a controller. At this point Customer is null.
The user then submits the form a number of times (think of a wizard design pattern) incrementing a page counter passed back and forth as a hidden value. As they navigate through (effectively, create a booking,  enter some key customer attributes, searching for an existing customer and if found direct that this record be used otherwise create a new one). In the use case here we have searched for a record and are creating a new one so the Customer record is populated with a skeleton record in the controller that is not persisted to the database at that point but used to pass the values to the view.
Thanks
Mark

@MCFHTAGENTS could you provide a small repro without a database?

TestChildPOCO.zip

The attached zip file creates the output on my machine as presented by the attached image

@MCFHTAGENTS you're not @addTagHelpering the Mvc.TagHelpers so the input tag helper isn't running. For example, if you view source:

Note the asp-for is still in the output.
However, if at the top of your file you add @addTagHelper *, Microsoft.AspNetCore.Mvc.TagHelpers Everything works as expected:


[BUG] Looking for MQTTX to log messages for longer duration

Discussed in https://github.com/emqx/MQTTX/discussions/1074

Originally posted by unnabrt September 16, 2022
I have been using MQTTX application for last 4 months. It is very useful tool to connect to my MQTT broker and works as a logger. But the MQTT connection in the tool doesn't last for more than 2 hours. It started successfully with the following messages
[2022-09-16 15:52:34] [INFO] MQTTX client with ID 58c90fdb-eaa7-4f5a-b3e0-d45361e80f11 assigned
[2022-09-16 15:52:34] [INFO] Connect client xxx_demo, MQTT/SSL connection: mqtts://mqtt.myportal.com:8883
[2022-09-16 15:52:34] [INFO] xxx_demo connect success, MQTT.js onConnect trigger
after running for nearly 1.5hrs, it ended with the following error message:
[2022-09-16 17:33:29] [INFO] xxx_demo message arrived: message added "message_63d3c492-b016-4ad1-a60c-04c6a6c2f895" added to topic "server/L00006/pub_sensor_reading", payload: "{"UID":"L00045","OBJ":"32769","TS":"1663320805","SNS":["28.88","-4.88","33.50"]}" MQTT.js onMessageArrived trigger
[2022-09-16 17:33:30] [ERROR] xxx_demo connect fail, MQTT.js onError trigger, Error: read ECONNRESET
at TLSWrap.onStreamRead (internal/stream_base_commons.js:209:20)
[2022-09-16 17:33:30] [INFO] xxx_demo connect close, MQTT.js onClose trigger
Kindly help if there is any setting to prolong the duration of usage.

Btw, the OS and the versions:
MQTTX Version: v1.8.2
on Windows 10 Pro, v 21H2
thanks

@unnabrt Thank you for the information.


Allow Array Responses

check_authentication method explodes when the json response is an array.
This adds a validaton to avoid fail on to_h call.

@JuanCrg90 puedes corregir los conflicts?

@enortham the branch is up to date.  could you help me to merge the PR and publish it to rubygems?

Listo
On Mon, May 18, 2020 at 11:06 AM Juan Carlos Ruiz<EMAIL_ADDRESS>wrote:

@enortham https://github.com/enortham the branch is up to date. could
you help me to merge the PR and publish it to rubygems?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/easybroker/easy_meli/pull/1#issuecomment-630282277,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABKOD4TQALGN3VM4IOT5BLRSFMIVANCNFSM4NCVXSYQ
.

--
Eric Northam
CEO, EasyBroker https://www.easybroker.com/


Envoy docker images on ECR

Title: Public envoy docker images on ECR
Description:
Now that dockerhub is enforcing rate limits for unauthenticated pulls, pulling envoy images from dockerhub can cause issues for downstream projects. It would be great to have official envoy images published on AWS ECR as an alternative. This would let us (and probably others) avoid creating a dockerhub -> ECR mirror under our own namespace.

Hi there, I'm a Consul PM here (one of a few). We have found it a bit challenging to support AWS ECS for our use case with Envoy, and would love to see if the same Envoy image could be mirrored on the ECR repository. I just wanted to provide a bit more color here around this ask:
Currently external users of Envoy Proxy within a Service Mesh will be rate limited quickly by Docker Hub when pulling Envoy images (resulting in errors during deployment) because docker pulls from AWS Fargate share the same IP. In addition, ECR provides the ability for Amazon customers to pull images from private subnets without a NAT Gateway, which can bring down the cost of deploying a Service Mesh on ECS since NAT Gateways can be prohibitively expensive.

I don't think any projects here would object, but I suspect it's an issue of 1) doing the work and 2) making sure there's funding (I assume anything with unlimited downloads isn't free but correct me if I'm wrong).  If you folks are up for doing (1) and can confirm if (2) is  thing I can reach out to CNCF folks about budgeting.

Hey @alyssawilk, we are up for doing the work! We'd just need to set up a new AWS account to host the images in ECR, claim the namespace, setup the repo, and update the CI pipeline to tag and push to the repo.
As for pricing, ECR offers "50 GB-month of always-free storage for your public repositories. You can transfer 500 GB of data to the internet for free from a public repository each month anonymously (without using an AWS account.) If you sign up for an AWS account, or authenticate to ECR with an existing AWS Account, you can transfer 5 TB of data to the internet for free from a public repository each month..." Beyond that, storage is "$0.10 per GB-month for data stored in private/public repos." [1]

I checked with our awesome CNCF folks and they told me that docker has ratelimit exceptions for OSS projects.  He said they'll look into getting this sorted out, so hopefully we can fix without a bunch of work and switching download platforms.  I'll update here when I hear back from them, but feel free to ping back if you haven't heard in a week or so.

cc @phlax

FWIW just head back from the dockerhub folks - while we haven't had the project reassessed yet we're making progress :-)

From docker: Welcome to the Docker Open Source Program!  We are very excited to have you as a part of our great community.  We have whitelisted your namespace "envoyproxy" and this should come into effect in the next week or so. With this whitelisting, the Docker data pull rate policies that went into effect last November, will not apply to the users pulling images from your namespace.
@mdeggies let us know if you have problems after a week or so, otherwise we'll let this stale out as "fixed"

Thank you @alyssawilk I will sync with @mdeggies on this and our internal team! This is very exciting!

@alyssawilk you can go ahead and close this issue! I've confirmed that this works for our team!

Oh nice!  I guess their week or two is more like a day or two :-)

Thanks very much @alyssawilk ! :)

Sorry to dig up an ancient issue. Is the envoy namespace no longer in the allowlist for open source? I got this error:

Is the envoy namespace no longer in the allowlist for open source?

i think that may be the case. I will follow up, but it might take a bit of time.

Thanks. It seems like the allowlist is decided every 12 months based on their documentation.

this should be resolved (again) - we have been accepted back onto the OS program

thanks for the heads up @aykutersoyrmn - ill take a look at our Docker acc and see if there is any obvious reason

doesnt appear to be any issue with our account - still showing as Docker-sponsored OSS
ill follow up and see if i can find out more

@phlax Thanks for looking into this super fast 🙌
We're not block on this as our workaround works pretty well, we might actually keep it forever 😄

not sure if related - we are migrating some CI and likely increased our usage of the docker image (i started seeing the rate limit issue in ci) - ive opened a PR to resolve that here #36032


Gap between display and playing for audio element (firefox mac)

Hi,
When I start reading (from 0) and then advance time (for 5s by example), display is correct but not the sound reading and finally for a track of 3min45, the total time increase to 3min50.
It's possible to reproduce it on http://mediaelementjs.com/ with audio example.
If I start directly in track I have no problem.
It's with firefox 43.0.4 on mac OSX 11.2, no problems on windows.
Is it a problem of Firefox ?

Hi,
Now in firefox 44.0.2 the problem disappeared.

@flobookdo Can this issue be closed?


Folder actions children of view directory

Folder actions are specific to directory views so a directory view should have these actions in a toolbar of some sort:

New File
New Folder
Upload Files
Upload Folder
Paste Item

It would also be nice to have consistent view action locations across different view types.
For example:

Save Document
Toggle bulb
Toggle Line wrapping
Use Tabs/Spaces
Indent Unit

Fixed in https://github.com/silverwind/droppy/commit/67e75c6e40db31128f8e8108f5ac969ea25e3fe9 et al.


Order By Operator not supported

Hi Nat,
it seems to me that we are currently not supporting JSONata's  Order By operator.
https://docs.jsonata.org/path-operators#---order-by
E. g.: https://try.jsonata.org/YKVa54Iu_

Correct, we don't currently support this. It is documented at the bottom of the README.md file.

So now we could support this soon 👍

Thank you again Martin -- I really appreciate your contributions. I've released v2.4.0 today. It may take a couple of hours to reach the Maven Central, but the Sonatype repo should have it now.


fix(HitsPerPage): compute selected from isRefined

Summary
fixes https://github.com/algolia/instantsearch/issues/5397
Slight improvement in the HitsPerPage + SortBy tests, as it was relying on Vue state instead of the DOM
I wanted both to be exactly the same, but SortBy doesn't have isRefined on the items apparently.
Workaround in the example to make HitsPerPage responsive to the URL is also removed.

Result

This makes HitsPerPage responsive to the URL, as expected like other widgets.

Yes, I doubted to write a common test here, but I was juggling too many other things to write it just yet


Issue with  k8s-extension

Related command
az extension add --name k8s-extension
Extension name (the extension in question)
k8s-extension
Description of issue (in as much detail as possible)
When 'k8s-extension is being installed using az command error occurred
To Reproduce
Steps to reproduce the behavior:
Go to power shell where azure cli is installed
type az extension add --name k8s-extension
See error "No matching extensions for 'k8s-extension'. Use --debug for more information."
Expected behavior
The Extension is successfully installed
Environment (please complete the following information):
OS: Windows 10
Shell: powershell
Az CLI version: 2.0.80
Python version (if pip installed): 3.9.13

route to CXP team

@jereesh I can confirm this does work. Could you try upgrading to the latest Azure CLI version (2.37.0) and try again?


notebook driver

lscpi | grep VGA
01:00.0 VGA compatible controller: Advanced Micro Devices, Inc. [AMD/ATI] Chelsea LP [Radeon HD 7730M] (rev ff)
where can i find the right driver

your GPU is Verde.
Please refer to issue #49


Handle topics_widget being undefined.

discussion: https://chat.zulip.org/#narrow/stream/9-issues/topic/Recent.20conversations.20not.20loading

Merged, thanks @amanagr! @evykassirer FYI for post-commit review.


Go to the settings tab.
Check the box to select the Google Drive add on and press submit.
Scroll down to configure the Google Drive add on and import your API token.
Scroll back up and uncheck the Google Drive box and press submit.
Go to the User settings page.
On the left hand side of the page go to configure add ons.
Scroll Down to the Google Drive add on.

Expected
My project would no longer be listed underneath the add on.
Actual
My project is still listed as having the Google Drive add on configured and, when you try to remove the add on through the user settings page by pressing the x next to the project title, a pop up message lets you know that an error has occurred and that the project could not be deauthorized. The only way to get rid of the add on completely is to either delete the google drive access token on the user settings page or by going to the project settings page, adding Google Drive to the project, and then scrolling down and deauthorizing Google Drive.

@lyndsysimon, after looking into this issue it seems that PR #2231 takes care of it.

OK we'll 'hold' this for now and check post PR #2231 deployment. Thanks.

@lbanner I am still having this issue


Improved diagnostic message for enumeration invalid key/value

Before attempting to pass an invalid value in an enumeration would result in an error referencing a keyset-value mapping with no mention of the enumeration itself. This should more obviously point to the fact that the given value is not present in the enumeration.
DAFFODIL-2607

Small change and really surprised we didn't have any existing tests that checked for this error situation where we attempt to use an invalid value for an enumeration.  I checked against the dfdl-regression test suite and it seems that pretty much everything passes.  I believe there may be a couple tests in P8 that need updating to not check for "keyset-value" anymore.

Comments should be addressed now.  I actually had an unparser test case before, but it was triggering the same error message so I didn't think it was adding any additional coverage.  I was expecting to get the "key not found" error, but I think the only way that happens is when we are expecting a different type (ie expecting an int and get a string).


Support_SGX1-machine-with-SGX-in-tree-driver: patch does not apply

Hello, I'm trying to apply the patch for SGX SDK and PSW on non-FLC platforms. I have Ubuntu 18.04 and Linux Kernel 5.11.
When I try to apply the patch this is the output:
~/linux-sgx$ git apply ~/inclavare-containers/hack/no-sgx-flc/Linux-SGX-PSW-2.13-Support-SGX1-machine-with-SGX-in-tree-driver.patch
error: patch failed: psw/enclave_common/sgx_enclave_common.cpp:815
error: psw/enclave_common/sgx_enclave_common.cpp: patch does not apply

What should I do?
Thanks in advance.

This patch is only applicable to sgx sdk & psw 2.13. What is the version of your patched sgx sdk & psw?


EID-1682: Add in pipeline to murder the proxy node on que

⚠ ⚠ ⚠ This creates a pipeline that can nuke the proxynode. ⚠ ⚠ ⚠

Its set as paused when its deployed.


feature: add a public method for checking if file is in dotnet repo

context:
I would like to call keymaps conditionally depending on the language I am working with. My current solution is to work with filetype, but the problem with that is that I cannot run commands when I am on a file which I havent defined (some files will appear on many repos of different languages, like gitconfig, .env, etc...)
The solution would be to define the keymaps not going by filetype by by condition.
Also, instead of having this:
      {
        ft = "cs",
        "<leader>oB",
        function()
          require("easy-dotnet").build_solution()
        end,
        desc = "Build .NET App",
      },

It would be nice to have a function like require("easy-dotnet").is_dotnet_project():
      {
        cond=require("easy-dotnet").is_dotnet_project(),
        "<leader>oB",
        function()
          require("easy-dotnet").build_solution()
        end,
        desc = "Build .NET App",
      },

Then, the function require("easy-dotnet").is_dotnet_project() will recognize if the current file, independent of the filetype, is part of a dotnet project

Making the is_dotnet_project() function is rather easy but it might not work as you expect. It wont check the current file, it will rather check cwd and subfolders for sln and csproject definitions. This is how basically all of the logic inside the plugin works. Are there cases where checking relative to the file instead of vim cwd is necessary? I would love to hear your thoughts. My biggest reason for using cwd is that its simpler and for the way I use it, it will always work as expected.

Hi, awesome how fast you develop!
Yes, I didn't express myself correctly. The idea was exactly that: not need to check the current file but the whole path. I have some other places in mind where I will be adding this check. Let me add this function and check if checking the cwd is a problem.
(e.g. I am using git-worktree from primeagen and I will copy my own appsettings from one worktree to another which will not push)


Adds bundle analyzer

No asset should be larger than 3 MB
Adds script to verify asset size
Updates build job to parallelize test runs
On Error:

Signed-off-by: Bipul Adhikari<EMAIL_ADDRESS>

/lgtm

/test odf-console-e2e-aws


DEC_HELIX configuration fails to boot

With only DEC_HELIX and BLUETFT defined, ESP32 (doit devkitv1 clone, 30 pin) is stuck in a reboot cycle.
Typical output:
rst:0x3 (SW_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:2
load:0x3fff0030,len:1184
load:0x40078000,len:13104
load:0x40080400,len:3036
entry 0x400805e4
ets Jul 29 2019 12:21:46

Please advise how to provide additional debugging information.
Filesystem has been uploaded and is known good from DEC_VS1053 configuration.

I was having the same problem and I found this old issue. I was using the Github "release" which is out dated. The problem went away when I used the up to date code by cloning the repository.


Use lang with api queries

Use lang with api queries
This pull request does the following:

Uses the new items query over the deprecated itemsByType query
Adds the lang: attribute to the items API query based on the user's selected language
Adds a helper function for getting two digit lang codes -> src/modules/lang-helpers.js
Adds a prebuild script that creates trader icons from their respective translation

resolves: https://github.com/the-hideout/tarkov-dev/issues/107
Example 📸

.deploy to development

We have another problem, not only images are now in ru, but also the links... I'll suggest reverting this change until we have the API with trader ids


Drop terraform things from dependabot

These generally require a bit of hand-holding anyway and trigger rebuilds of the world, so don't automate them.

As part of a Chainguard effort to reduce code duplication and provide higher quality images, we have stopped accepting pull requests for changes to images in this repository. Due to this, we are closing all pull requests today in order to start fresh.
If you think this pull request was closed in error, please feel free to re-open this pull request or open an issue and we will try to help you out. Thank you!


Add company-create endpoint

Description of change
This adds an endpoint that creates a company using data from the dnb_service given a duns_number.
Following PRs would primarily deal with extracting more data from the DNB response to create richer company records in DataHub.
Checklist

[x] Has a new newsfragment been created? Check changelog/README.rst for instructions
[ ] Do any added or updated endpoints appear in the API documentation? See docs/Maintaining the API documentation.md for more details
[ ] Have any relevant search models been updated?
[ ] Have any relevant fixtures (fixtures/test_data.yaml) been updated?
[ ] Have any relevant select-/prefetch-related field lists in the views and search apps been updated?
[ ] Has the admin site been updated (for new models, fields etc.)?
[ ] Has the README been updated (if needed)?

TY for the early feedback @currycoder
I would do the following:

Rebase.
Refactor search_dnb to take the query parameters instead of a request.
Re-introduce duns_number validation in the company-create view.
Fail when/if dnb-service returns multiple results for a duns_number.

@alixedi did you see this comment?

Also can't see any logic around created_by and modified_by – would be good to add tests for those and update the logic as needed

@alixedi did you see this comment?

Also can't see any logic around created_by and modified_by – would be good to add tests for those and update the logic as needed

Missed it. I've put something in now.


Add an offset

Hi,
Thanks for the lib.
I am currently using the angular2-contextmenu in association with the angular2-tree-component.
On my page I have a fixed header at the top (height : 50px) and the main content of the page is in an absolute position.
When I right click on a node of tre tree the context menu shows, but its position is moved 50px to the bottom.
I would like to have the possibility to add an offset (or maybe two : one for top and one for left directions) (possibly by overriding a param in options) to update the position of the context-menu according to the design of the page.
What do you think ?
Thanks !
Julien

Can you give me a plunker or git repo that reproduces this?  (Or you could paste your html/css code directly into the issue.)  I want to make sure I fully understand your situation.
Also, you can add your own offset in css directly like this:
.angular2-context-menu > .dropdown-menu {
  margin-top: 50px;
  margin-left: 5px;
}

Hi,
I manage to make it work by adding
.angular2-context-menu > .dropdown-menu {
  margin-top: -50px;
}

This way, the context-menu is shifted to the top.
As the value is exactly equals to the height of my header, the user experience is very smooth now !
Thanks for pointing out the solution ;-)
Julien

By the way, the date of version 0.7.0 in the changelog refers the same date than the version 0.6.0.
Woul be nice if you can change it.
Thanks.
Julien

Yes, good catch.  Fixed.


Display tooltip instantly on warning button press in Scene Tree Editor

This PR replaces the AcceptDialog warning in favour of showing the warning tooltip instantly when the Tree button is pressed.

I personally find it so much more lovely. Definitely faster than clicking to display the warning in a popup and then closing it by pressing OK.
Requires https://github.com/godotengine/godot/pull/64796 to be merged first.
With very wide Trees, it may not behave as expected until https://github.com/godotengine/godot/pull/64771 is adjusted and merged first.

I do not understand why that one Linux debug build would fail...

Check the log :)
https://github.com/godotengine/godot/runs/7979613143?check_suite_focus=true
Control.show_tooltip --- executing with 0 parameters []
GDSCRIPT CODE:     Control.new().show_tooltip()
scene/gui/control.cpp:3045:41: runtime error: member call on null pointer of type 'struct Viewport'
scene/gui/control.cpp:3045:41: runtime error: member access within null pointer of type 'struct Viewport'

================================================================
handle_crash: Program crashed with signal 11
Engine version: Godot Engine v4.0.alpha.custom_build (18fbde9f8c828f48b57e97c7ce5820ef1e5d9853)
Dumping the backtrace.

Oooooh, it's trying to get the Viewport but that can't be done if the Viewport isn't set. I see now.

We discussed this in a PR review meeting. We're not strongly opposed to the idea but also not really convinced that this is an improvement. Notably, removing the AcceptDialog actually takes away a feature, as there's no longer a way for users to display the tooltips in a stable way - any mouse movement will discard the tooltip, and a mouse movement following a click is not unlikely. It also makes it impossible to select/copy the warnings (not sure if possible in the AcceptDialog but it could be in theory) or screenshot.
Overall, it's a new UX language to show tooltips on click which seems a bit confusing/unnecessary.


Emit integration fields in JSON if at a valid value that is also their default value.

This change will cause fields to be emitted for JSON integration even if they are their default value. For example: before this change a valid data rate of 0 would be omitted from the messages from the integration JSON messages. With this change a valid value of 0 will not be omitted anymore.
This change is related to issue #43

Thanks @TomSievers :+1: , I agree this is useful.


[Feature Request] Dynamic add section

This is for integrate with other plugins
e.g. To add a status line section for plugin runs unit test:
require'windline'.add_section({get_test_progress={get_test_progress(), 'white'}})

And once test completed
require'windline'.remove_section('get_test_progress')

Or add animation
animation.add_animation({get_test_progress = {
    timeout = 3000,
    delay = 200,
    interval = 150,
    effect = efffects.list_text(get_test_progress),
    on_tick = function(value)
       get_test_progress()
    end
}})

And remove animation on test finish callback
animation.remove_animation('get_test_progress')

hi I add some functions to add or remove components.
sample here
https://github.com/windwp/windline.nvim/blob/d4d628f23e19845754b97db96fac7fa75a00ce84/lua/wlsample/test_add_component.lua#L282
about add animation I still not figure out how to do it a simple way because the component can have a child component with highlight and the animation need to understand the structure of the component.

I am trying to call  lua require'wlsample/test_add_component' But did not see 🧛 shows in status bar.
Anything wrong?

hi you need to press key leader-x to add and leader-z to remove

Cool, I see how it works. Thanks!
Is it possible to add text animation as well?  (e.g. a progress bar)

https://github.com/windwp/windline.nvim/blob/master/lua/wlsample/job_test.lua#L69
you can try this. It run command sleep 10 .when command is running it display spinner and after that command finish you can get the text and display on status
I think the easy way is to write your component animation inside the text function and set  variables on buffer to display animation and after your task finish you can change those variables to display a result

job_utils.job_event(
        'sleep 10',
        'BufEnter',
        'job_sleep',
        function(data)
            if data.is_load then
                return 'event ' .. data.loading_text
            end
            return 'bufenter done ' .. (data.data or '')
        end
    ),

job_event require a system command. But lots of time I may run an async function from lua. e.g.   require"go.format".import()  And it will use underlie lsp async request.   So sleep 10s will not be accurate.  Is it possible to create a hook or event-driven way to update the status line?
Also. let say run yarn test I still prefer my plugin control the workflow and output. That means I will not run it through  job_event. If the user installed windline, the output may rout to windline if it is enabled by config.

yeah i know it that why i suggest to use a buffer variable

Not sure how it going to work.
If I am going to add a status show status of unit test. I will have a
|spinner|status indicator|message
Spinner will need to auto-update every 50ms until test finish
status indicator and message will be based on output of unit test.
I can update the global variable or buffer variable in the plugin.
How will I achieve this with job_event?

https://github.com/windwp/windline.nvim/blob/be5ae176a57be9191095e848261bf7f6156997ba/lua/wlsample/job_test.lua#L92
I add a sample here just change vim.g.tmp_job_state and that component will change.

Nice! That looks like the one I am looking for. Thanks!


Add tap()

tap() calls the given $callback with the given $value, which, in quite a few cases, makes the code read more fluent and "functional":
tap(User::create('John Doe'), function (User $user) {
    UserService::sendWelcomeEmail($user);
})->login();

// or

tap(DisposableEmail::create('From Nigerian Prince with luv'), function (Email $email) {
    EmailManager::send($email);
})->destroy();

This function is inspired by Laravel, which is in turn inspired by Ruby.

TBH I wasn't aware of with(). From the look of it, it seems the two have a fundamental difference: with() return the callback's result, when tap() returns the passed-in $value itself. As a result, they should serve different purposes.

Makes sense, let’s add it

Nice! I've updated the code. Btw, maybe the repo could use a .editorconfig file?

Sure, .editorconfig sounds good. Can you check again, tests still contain spaces apparently.

Can you check again, tests still contain spaces apparently.

Hmm, I'm not sure what spaces you're talking about this time. Can you elaborate?
Re: .editorconfig, I'll open another PR.

Hmm, I'm not sure what spaces you're talking about this time. Can you elaborate?

The tests have tabs, is that true?

The tests have tabs, is that true?

Don't think so – I think they're all spaces now.

Thank you!


"exceeds maximum batch" size when storing vector db

Describe the bug
I'm using Cheerio Web Scraper to fetch a website's content, and then I'm getting an error when storing it into a vector library.
Got error "exceeds maximum batch" from chroma db.
I checked the information and found that the vector libraries all have limitations, Like chroma,pinecone...
I tried to reduce the number of chunks and increase the size of every chunks, but I got a new error: "413 Request Entity Too Large" from vector db.
The I checked the code in packages/components/nodes/vectorstores/Chroma/Chroma.ts.
 const flattenDocs = docs && docs.length ? flatten(docs) : []
            const finalDocs = []
            for (let i = 0; i < flattenDocs.length; i += 1) {
                if (flattenDocs[i] && flattenDocs[i].pageContent) {
                    finalDocs.push(new Document(flattenDocs[i]))
                }
            }

            const obj: {
                collectionName: string
                url?: string
                chromaApiKey?: string
            } = { collectionName }
            if (chromaURL) obj.url = chromaURL
            if (chromaApiKey) obj.chromaApiKey = chromaApiKey

            try {
                await ChromaExtended.fromDocuments(finalDocs, embeddings, obj)
            } catch (e) {
                throw new Error(e)
            }

I found this finalDocs too big. Because a website's knowledge base can be very large,Is it possible to split into multiple chunks for cyclic storage here? Or is there any other way to solve this problem?
Screenshots
The flow

Chroma logs

But the finalDocs are the splitted chunks, it shouldnt be the culprit thats causing 413 Request Entity Too Large

But the finalDocs are the splitted chunks, it shouldnt be the culprit thats causing 413 Request Entity Too Large

But the length of the chunks is still a problem.such as chroma has a maximum length of 166, I printed the length of finalDocs is 419
I tried to change the following code and the problem was solved, but I don't know if there is any problem with this method.
packages/components/nodes/vectorstores/Chroma/Chroma.ts  >  upsert

// try {
//     await ChromaExtended.fromDocuments(finalDocs, embeddings, obj)
// } catch (e) {
//     throw new Error(e)
// }

// Split array
 function chunk(array: any[], size: number) {
                const chunkedArray: any[] = []
                for (let i = 0; i < array.length; i += size) {
                    chunkedArray.push(array.slice(i, i + size))
                }
                return chunkedArray
            }

            const chunkedArray = chunk(finalDocs, 100)
            for (const arr of chunkedArray) {
                try {
                    await ChromaExtended.fromDocuments(arr, embeddings, obj)
                } catch (e) {
                    throw new Error(e)
                }
            }


Added SecurityClient APIs

Description
Added SecurityClient APIs
Issues Resolved
Related to #435
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@nhtruong, I notice "body" and "path params" descriptions are missing for these APIs in OPenAPI spec. Can you please confirm? Thanks :)

@saimedhi Looks like they dont.
You can also check it our yourself. All files related to security are in this folder: https://github.com/opensearch-project/opensearch-api-specification/tree/main/model/security on API repo.
For example, to check if the create_user endpoint has any description for the body, inspect the @output in structures.smithy file in the create_user subfolder: https://github.com/opensearch-project/opensearch-api-specification/blob/main/model/security/create_user/structures.smithy#L22

Hi :slightly_smiling_face:
The security client is already implemented in this MR https://github.com/opensearch-project/opensearch-py/pull/399.
Is anything missing ?

@florianvazelle The code in this PR was generated from OpenSearch API. Moving forward, all new API endpoints will be first added to the API repo then the API code is generated for each client like this. This is to assure consistency across clients, and speed up the deployment of new features. This draft is also a chance for us to fine tune the generator and plug holes in the API specs.

Closing this pull request as SecurityClient is already present in plugins. @dblock Should the Security Client be left in plugins or added to opensearchpy/client as like cat client?

@saimedhi that PR that added security endpoints are for this repo.

@saimedhi @nhtruong I think I'm missing context, you guys decide what to do with this :)
for license headers, the rule of thumb is that all existing code needs to keep existing headers, and all new code needs only the SPDX header


fix: file object info for delta fs handler

Description
Fixes an error in the DeltaFileSystemHandler, when reading file metadata from remote storages. Due to an inconsistency between the behaviour object stores when invoking list operations on a path that points to a file, we incorrectly returned an Directory type for files in case of object stores. The bug only surfaced when using pyarrow < 9, since we used the call only when getting the file size, which we avoid when using more recent pyarrow versions.
@tustvold - I seem to vaguely remember discussing this at some point, but am not sure anymore. Is this something we should look into in object-store?
Related Issue(s)
closes #1109
Documentation

I seem to vaguely remember discussing this at some point, but am not sure anymore. Is this something we should look into in object-store?

object_store should not return directories in listing responses, it follows the object-store model and not a filesystem model. If one of the implementations is returning directories that is a bug.

From what I could tell happend. If you have a store with a single object file at the root. Using list (or list with prefix) like
path = Path::from("file")
store.list(Some(path))

I think the remote stores will return a list containing that single object, whereas LocalFileSystem will return an empty list.

turning this back into a draft, since the upstream fixes are in the works ...


Typos in monitoring application

Prometheis should be Prometheus
OpsGenie should be Opsgenie

The plural of Prometheus is actually Prometheis, so that unfortunately is correct as weird as it is.
We should correct OpsGenie to Opsgenie based on: https://www.atlassian.com/webinars/incident-management/intro-to-opsgenie

Fixed


Make max utilization  optional to support Serverless NEG backends

Similar to what we had to do with health checks to support Internet NEGs, it would appear that serverless NEGs cannot have a max util set.
Error: Error creating BackendService: googleapi: Error 400: Invalid value for field 'resource.backends[0].maxUtilization': '1.0'. Max utilization is not supported for Serverless network endpoint groups., invalid

Since this module uses the Google terraform provider's google_compute_backend_service resource, I believe this will get fixed when this issue is fixed: https://github.com/hashicorp/terraform-provider-google/issues/6876

This should now be unblocked as per the latest beta provider
https://github.com/hashicorp/terraform-provider-google-beta/releases/tag/v3.37.0

to be clear, https://www.terraform.io/docs/providers/google/r/compute_region_backend_service.html should be created when max util is omitted, or something of the sort (such as a regional flag)

This seems unblocked now for serverless NEGs. You can set the field to null.
I'm in the process of creating a submodule specifically for creating LB for Serverless NEGs. It will not have the unsupported fields such as this.


fix line break in example 2 code

Docs Build status updates of commit 7ef2159:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/windows/remotedesktop/Set-RDCertificate.md
:white_check_mark:Succeeded

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


Make header references conform with man-pages(7) in all manuals

Details from man-pages(7) that are used:
Formatting conventions (general)
   ...
   Filenames (whether pathnames, or references to header files) are always
   in italics (e.g., <stdio.h>), except in the SYNOPSIS section, where in‐
   cluded files are in bold (e.g., #include <stdio.h>).  When referring to
   a standard header file include, specify the header file  surrounded  by
   angle brackets, in the usual C way (e.g., <stdio.h>).
   ...

This one seems obviously simple, but I wonder how many of these conformance fixups could be put into find-doc-nits?
For filenames, something like:
err("Bad filename reference $1") if /(F<[^>]\.h>)//

This one seems obviously simple, but I wonder how many of these conformance fixups could be put into find-doc-nits?
For filenames, something like:
err("Bad filename reference $1") if /(F<[^>]\.h>)//

Hmmm, it's a little more complex than that...  I suggest we do that find-doc-nits mod in a separate PR.  But otherwise, it's not a bad idea.

Hmmm, it's a little more complex than that...  I suggest we do that find-doc-nits mod in a separate PR.  But otherwise, it's not a bad idea.

It makes complete sense to do the find-doc-nits mods separately.  Might be worth thinking about all your recent changes and what could be easily automated while it's still fresh.

It makes complete sense to do the find-doc-nits mods separately.  Might be worth thinking about all your recent changes and what could be easily automated while it's still fresh.

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged
879365e6d4a53d80e83bbe468fcf2cdd02d30ba1 Make header references conform with man-pages(7) in all manuals

Merged
879365e6d4a53d80e83bbe468fcf2cdd02d30ba1 Make header references conform with man-pages(7) in all manuals


updated the readme

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

 No worries, but I couldn't merge this in unfortunately ;)


Update sidebar.ts

we dont seem to have a link on the left hand navigator to mask rules when looking at processing rules (https://help.sumologic.com/docs/send-data/collection/processing-rules/).
We also dont have the left hand bar when looking at the Mask rules page (https://help.sumologic.com/docs/send-data/collection/processing-rules/mask-rules/) ….
Slack Message

Fixed in source.


Revert v0.1.2 state

What was the need for this PR?

Accidentally I pushed a very old state of the repository while I was performing the release

Ok


add deepseek-v2, yi-large and gpt-4o pricing to token_counter

User description
for usage calculation

PR Type
enhancement

Description

Added new model pricing details to the token counter for gpt-4o, gpt-4o-2024-05-13, yi-large, and deepseek-chat.
Updated the token counting function to support the newly added models.

Changes walkthrough 📝
Relevant filesEnhancement

token_counter.pyAdd Pricing and Token Counting for New Models

metagpt/utils/token_counter.py
Added pricing details for gpt-4o, gpt-4o-2024-05-13, yi-large, and deepseek-chat models.  Updated token counting logic to include new models gpt-4o-2024-05-13 and gpt-4o.

+6/-0

💡 PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 70.20%. Comparing base (7057ace) to head (1047abf).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1274   +/-   ##
=======================================
  Coverage   70.20%   70.20%
=======================================
  Files         316      316
  Lines       18860    18860
=======================================
  Hits        13240    13240
  Misses       5620     5620

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


Fix ENS resolution if mainnet not supported

Found issue:

Check issue 350 reported by @SonOfMosiah

Proposed solution:

Add useEnsFallbackConfig hook to easily guarantee mainnet access to all ENS-related hooks, regardless of the chains supported by the dapp.

@lochie 👀

Btw @lochie, if you need some help from external reviewers, don't hesitate to reach me. Always happy to contribute to quality open source projects when I have time ✌

Btw @lochie, if you need some help from external reviewers, don't hesitate to reach me. Always happy to contribute to quality open source projects when I have time ✌

Thanks @philogicae, I really appreciate that! Will tag you if anything comes up 😊

This feature is now available in ConnectKit 1.7.3 🫡


Controlnet is not enabled. How to make it works?

This is my code.
import webuiapi
import json
import requests
import io
from PIL import Image, PngImagePlugin
import os

# create API client with custom host, port and https
api = webuiapi.WebUIApi(host='<IP_ADDRESS>',
                            port=7860,
                            sampler='Euler a',
                            steps=20)

options = {}
options['sd_model_checkpoint'] = 'SD1.5/v1-5-pruned.ckpt [e1441589a6]'
api.set_options(options)

# read image
r = api.txt2img(prompt="photo of a beautiful girl with blonde hair", height=512, seed=100)
img = r.image
img = img.resize((512, 512))

unit1 = webuiapi.ControlNetUnit(input_image=img, module='canny', model='control_sd15_canny [fef5e48e]')

r = api.txt2img(prompt="photo of a beautiful girl",  controlnet_units=[unit1], seed=300)
r.image.save(out_final)

r2 = api.txt2img(prompt="photo of a beautiful girl", seed=300)
r2.image.save(out_final2)

I found that r and r2 is the exact same image which mean that the controlnet is not activate.  How to activate the controlnet with the api? In the AUTOMATIC111 there is a "enable" check box for controlnet, is there anything similar to that in the api?

If your extensions (controlNet on SD-WEBUI) is "behind" the latest version on Automatic1111, function from sdwebuiapi will did not work, You can click 'check for update' for your extensions then restart UI for update latest version.

update the stable diffusion webui

Updating the Automatic1111 did solve this issue. Thank you very much


RequiredArgsConstructor on 2016.3.5

Hi,
On intelliJ 2016.3.5 I have a problem with @RequiredArgsConstructor in conjunction with @AllArgsConstructor. I have a class that has one final field and one non-final field and both annotations are on the class. It looks as follows:
@RequiredArgsConstructor
@AllArgsConstructor
public abstract class Command {
    protected final Type type;
    protected UUID uuid;
}

I expect two constructors, Command(type) (from the RequiredArgs) and Command(Type, UUID) (from the AllArgs). But what I get in the current version is only the Command(Type, UUID) constructor.

My coworker sees both constructors on IntelliJ 2016.3.4 and if I remember correctly I also saw them before I updated.
Regards
David

I have the same issue with Android Studio 2.4 p2, and I can confirm that on AS 2.3 (IntelliJ 2016.2) everything works correctly.


typeNumber doesn't work 1-40

Hi All,
I'm struggling to find option to generate a QRcode smaller, we can specify the size, but the QR code stays with high version. Trying to change the typeNumber option doesn't change it between 1-40, wondering if there is a bug here.
I want to generate a small version 2 QR code, actually not possible, what should be the option?
here example of my code (with lots of options tests xD):
$j('#qrcode').qrcode({
render	: "canvas", // canvas, table, png
width: 128,
height: 128,
typeNumber: -1,
correctLevel: 1, /* seems to reduce a bit the type, previous option doesn't seem to work from 1-14*/
/* pixelsPerTile: 10, --> doesn't seem to work*/
padding: 0,
border: 0,
background: "#ffffff",
foreground: "#000000",
text	: url
});
As I understand, to solution my problem I should enter "typeNumber: 2" --> but QR code is not generated and there is an error in the console log: code length overflow. (1012>272)
EDIT: So what I want to achieve is control or have proper version generated depending on the size: https://www.qrcode.com/en/about/version.html

I've just got something, I'm trying to generate a QR code for an URL: "https://support-dev.epfl.ch/incident.do?sys_id=7bc293aadb10081079f593c8f49619f3&sysparm_domain_restore=false&sysparm_stack=no" --> I think I cannot do a version 1 with such a string. reducing the size of the URL presented already solves a bit the problem. (if we put "toto" as text, the version generated is 1).

As described in my last comment, the size / version of QR code cannot be so small as my string is too long.


Snakemake 8 not usable with Slurm clusters offering GPUs

As previously reported

https://github.com/snakemake/snakemake/issues/2701
https://github.com/snakemake/snakemake-executor-plugin-slurm/issues/18

Snakemake 8.9 still fails to submit jobs to a Slurm cluster when requesting GPU resources.
profiles/default/config.yaml
executor: slurm

default-resources:
  ...
  slurm_extra: "'--gres=gpu:1'"

Select jobs to execute...
InputFunctionException in rule [obfuscated] in file [obfuscated]/Snakefile, line 6:
Error:
  WorkflowError:
    Failed to evaluate resources value '--gres=gpu:1'.
        String arguments may need additional quoting. E.g.: --default-resources "tmpdir='/home/user/tmp'" or --set-resources "somerule:someresource='--nice=100'". This also holds for setting resources inside of a profile, where you might have to enclose them in single and double quotes, i.e. someresource: "'--nice=100'".
    SyntaxError: invalid syntax (<string>, line 1)

Versions

Freshly installed Conda https://docs.anaconda.com/free/miniconda/#quick-command-line-install
Snakemake installed with Mamba as described here https://snakemake.readthedocs.io/en/stable/getting_started/installation.html
Snakemake 8.9
<EMAIL_ADDRESS>Python 3.12
Slurm 23.11.4

I'm afraid I could reproduce this issue.
We are working on a fix - actually this has been fixed, should be covered by a test case and still, it reoccurred. My apologies.

Thank you for your patience - I was absent and could not test.
Please update to the latest version of Snakemake: The parsing is now stable.


[Map] AS-UP_Manor

Duplicate of https://unrealarchive.org/maps/unreal-tournament/assault/U/as-up_manor_92aa4c2c.html

URL: https://unrealarchive.org/maps/unreal-tournament/assault/U/as-up_manor_dc4f8b49.html
Hash: dc4f8b49834af005d397dfd45fad2ce1661c730c
Current name: AS-UP_Manor

Updated!


error in CI builds

[INFO]  error  in ./src/components/quiz/testCreation/EditQuestion.vue
[INFO]
[INFO] Module Error (from ./node_modules/eslint-loader/index.js):
[INFO]
[INFO] /home/runner/work/skills-service/skills-service/dashboard/src/components/quiz/testCreation/EditQuestion.vue
[INFO]   117:3  error  `vee-validate` import should occur before import of `@/components/quiz/QuizService`  import/order

PR: #2344


0.1-dev-autoreconf

starting foundation to support rapid scaffolding in main. commits are self explanatory, however can be squashed upon request.

ci.yml: probably minimalize this, a lot of core-only options / flags that don't need to be in there. Keep most of the archs, except maybe -experimental and -sha512. remove intrinsics references, and wallet references.

done

So I think this is OK; there may still be some extraneous stuff but that got rid of the most of it.

i could certainly do more atomic commits/PR's but these 4 essentially address a singular function of having a working build/testing system. up to yall.


Module details are not available in Ballerina Central

Package details and list of functions are not available in Ballerina Central modules.
See [1] for example. However, this is available in the archived version [2].
[1] - https://central.ballerina.io/wso2/gmail
[2] - https://archive.central.ballerina.io/wso2/gmail

This issue is now fixed. Hence closing the issue.


Refactor Feature: String Concatenation to Interpolation

Search Terms
refactor, string, concatenation, interpolation
Suggestion
A small but useful refactoring that could be offered is to turn one or more string concatenations into an interpolated string. The inverse operation is also potentially useful (turning an interpolated string into concatentations)
Examples
let the = "the";
let foo = the + " quick" + " brown" + " fox" + " jumps" + " over " + the + " lazy" + " dog";

//Having cursor to left or right of + sign allows refactoring to convert to interpolated string ->

let foo = `${the} quick brown fox jumps over ${the} lazy dog`;

//Running the inverse operation on the result (converting an interpolated string into concatenation), would result in this -->

let foo = the + " quick brown fox jumps over " + the + " lazy dog";

Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript / JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. new expression-level syntax)

Duplicate of https://github.com/Microsoft/TypeScript/issues/18267


[Bug Fix] fix slice bug in LlamaRotaryEmbedding

PR types
Bug fixes
PR changes
Others
Description
In LlamaRotaryEmbedding, self.cos_cached and self.sin_cached are both the shape of [1, max_seq_len_cached, 1, self.dim], but forward function wrongly does slice operation on dimension 2 which is the dimension with the constant value 1. We update dimensions which should be slice actually.

可以帮忙 review 一下吗 😃  @wawltor @ZHUI


delete_project(not_exist_ok) default value should be False

This is a better user experience for beginners.

Resolved in 1e7b5f0.


Remote acess whith go2rtc

I'm using the following configuration on the card to use the microphone, but unfortunately, when I access it outside my LAN network, the card doesn't work, I have no sound.
cameras:
  - camera_entity: camera.front_door
    live_provider: go2rtc
    go2rtc:
      modes:
        - webrtc

It only works if I have my VPN to my HA server active.
Is there anything I can do to work around this issue?

https://docs.frigate.video/configuration/live#webrtc-extra-configuration
For external access, over the internet, setup your router to forward port 8555 to port 8555 on the Frigate device, for both TCP and UDP.

[Issue cleanup] If you still have issues with this, please try the latest release (https://github.com/dermotduffy/frigate-hass-card/releases/tag/v6.0.0-beta.4) first and re-open this. Thanks and good luck!

Do you use cloudflare because your network is CGNAT? If so, opening ports won't do anything. :(

Do you use cloudflare because your network is CGNAT? If so, opening ports won't do anything. :(

I don't know if I am using CGNAT or not, but I am using Cloudflare to access my Home Assistant externally...

Access this website, what is the result?
https://tomchen.github.io/symmetric-nat-test/

If you are behind Symmetric NAT, your best bet is to use NGROK:
https://github.com/AlexxIT/go2rtc?tab=readme-ov-file#module-ngrok

I was testing my home network and found an issue with incorrect port configuration on the wrong router. I had to enable bridge mode on the ISP's router and open port 8555 for WebRTC on the TP-Link Deco router. I also tested for CGNAT, and it returned "Normal NAT".
Unfortunately, I don't have audio from the streaming, and I can't communicate via the microphone to the camera. From what I've read, 2-way audio only works through an 'https' link, but it's not working.
I'm using the TP-Link Tapo C210 cameras with firmware version 1.3.11.

Here is how I have the Frigate card configured.
type: custom:frigate-card
cameras:
  - camera_entity: camera.cam_fg_quarto_lourenco
    live_provider: go2rtc
    go2rtc:
      modes:
        - webrtc
live:
  auto_unmute:
    - visible
    - selected
    - microphone
  auto_play:
    - selected
    - visible
  controls:
    builtin: false
    thumbnails:
      mode: none
    ptz:
      hide_zoom: true
      hide_home: true
      hide_pan_tilt: true
  preload: true
  lazy_unload:
    - unselected
    - hidden
  draggable: false
  zoomable: true
status_bar:
  style: none
menu:
  style: hover-card
  position: top
  alignment: top
  buttons:
    mute:
      enabled: true
      alignment: opposing
      priority: 4
    microphone:
      enabled: true
      alignment: opposing
      priority: 3
      type: toggle
    media_player:
      alignment: opposing
      priority: 2
    fullscreen:
      alignment: opposing
      priority: 1
    cameras:
      enabled: false
    clips:
      enabled: false
    snapshots:
      enabled: false
    timeline:
      enabled: false
    frigate:
      enabled: false
    live:
      enabled: false
performance:
  style:
    box_shadow: false

Here is how I have the Frigate camera configuration.
# Restream configuration
go2rtc:
  webrtc:
    candidates:
      - localhost:8555
      - stun:8555
  streams:
    cam_fg_quarto_lourenco:
      - ffmpeg:rtsp://user:pw@192.168.X.XXX:554/stream1#video=copy#audio=aac
      -<EMAIL_ADDRESS>


Proper bracket documentation for Brent's search.

Please, have a look at https://github.com/Horta/scipy/blob/01ec8969cb713f6bd52e9953583f553e5dab8f62/scipy/optimize/optimize.py#L1802.

I don't get how this is more correct the way you changed it, can you explain?

@Horta could you please answer @dlax's question?

Sorry for the delay, @dlax . You are right, there is no difference: it is just a matter of notation (a>b<c is equal to b < a, c).
I think this one can be closed.


Mutagen recipe is broken

The recipe for mutagen is outdated, the links in HOMEPAGE and SRC_URI are dead.
Do we need this tool in meta-bistro?
If yes, should we write a new recipe that refers to https://pypi.python.org/pypi/mutagen ?

I think we can remove it. We can add it back again if anyone needs it.


Translate Contribution Guide into English.

Currently, Contribution Guide was written in Korean. There not only has been foreigner contributors in the past but also it may be in the future. So, we should rewrite the guide to English.

i'll take it!


fix: ignore duplicate record errors on add key

This is a bandaid solution to prevent routing keys from being stored for multiple DIDs. It would be nice to correct behavior of the DIDDoc class to not turn routing keys into values in the publicKey list and then claiming that the DID is the controller (with no evidence). However, given the impending replacement of this behavior with did:peer, it doesn't seem worth it to go through that effort right now.
cc @swcurran @WadeBarnes and others

I'm working on some additional unit tests but assuming existing tests pass as I expect, testing this in your regression test setup would be helpful @WadeBarnes

Actually, I may have spoken too confidently. Sorry! I'm not seeing the results I expected. I'll continue testing.

I didn't see the results I expected because I initially tried to get the code to misbehave in a way I no longer permitted; worked around this in my test by circumventing the protections I added to recreate the conditions I believe we saw in those regression tests. If I've correctly deduced those conditions, I think this is now ready for testing.

Reiterating that while this is a bandaid, this will no longer be an issue once we get did peer fixes in. If we choose to migrate old records for stored DID documents, I think we should also reconstruct the key-to-DID mapping to get rid of old records of routing keys getting mapped to DIDs when they shouldn't have been. @Jsyro @swcurran

Daniel -- when StorageDuplicateError is thrown, is it going to cause a problem with processing?  I notice it is a warning, so presumably the service keeps moving along?

Daniel -- when StorageDuplicateError is thrown, is it going to cause a problem with processing? I notice it is a warning, so presumably the service keeps moving along?

@swcurran as far as I understand the circumstances of that led to the duplicate records errors, I believe ignoring the error and raising a warning will not have a negative impact on message processing.
The reason being that it is the routing keys that have duplicate records. These routing keys really have no reason to be stored in the key-to-DID lookup since we will never receive a message from a connection with those keys. If we did receive a message from those keys, prior to this fix we would have failed to process it anyways because we would have received that same duplicate record error. So on the whole, there's no change in how this scenario is handled when compared to the behavior before the introduction of #2409

Perfect — thanks!  -rc2 on the way.

@dbluhm - Looks like we're not out of the woods yet. In AATH, the Aries Framework Go tests that had been passing failed, as did the Findy Tests. I'll create a new issue and try to get the data needed to review them.


Consider changing color ramp for permafrost plate, for clarity

The color ramp for the permafrost plate is a bit confusing - it doesn't go dark to light. The more permafrost, the darker the color should be. Blue works, but the order of the shades is not intuitive.

Not bad… but discontinuous and isolated look pretty similar? At least to me…
On Tue, Jun 21, 2022 at 11:18 AM Charlie Parr @.***>
wrote:

Maybe something like this?
[image: pf_extent]
https://github.com/assets/7774271/62a3ed33-b860-4758-9264-7bff4cdbd52c
—
Reply to this email directly, view it on GitHub
https://github.com/ua-snap/arctic-eds/issues/27#issuecomment-1162146771,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABHC4BMQNE2PR3LVY5DTMFTVQIBPXANCNFSM5WNTXFYQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Carolyn Rosner | Designer
Scenarios Network for Alaska + Arctic Planning
University of Alaska Fairbanks
760-920-3721

Possibly, remove the borders (I don't see them on the production version -- @charparr what do you think remains to be done here?)

Looks good here, closing.  It's a weird mix of continuous data w/ 2 categorical slots (glacial, water) so it's gonna be a funny one but I like how it looks now.


Upgrade to node20

See similar PRs in k8s-set-context:

https://github.com/Azure/k8s-set-context/pull/84
https://github.com/Azure/k8s-set-context/pull/83

this will need updated release workflow to make a v5 release when this merges


First iteration progress

Coverage decreased (-22.4%) to 67.886% when pulling a4f1f3516b857c090a159706abbd35a7508886a0 on katie7r:master into 555a114dea693a4c5a05b4233e3405efe7b355b9 on cuttlesoft:master.


before_all hooks

I'm working on speeding up the discourse tests using let_it_be. We need a way to run code that sets up global variables before each before_all block is invoked. I can do this by providing a custom BeforeAll adapter within discourse, but if you would like, I could make it more general and submit a PR here. What would you prefer?

Hey @danielwaterworth!

We need a way to run code that sets up global variables before each before_all block is invoked

Yeah, I see (https://github.com/discourse/discourse/pull/7414/files#diff-e11bdaf9116be3911e9557b69e242370R213).

I could make it more general and submit a PR here

Yeah, that would be great.
We already have an example where it could be useful: https://github.com/palkan/test-prof/blob/a783083988e6b461f7ad32aa891ad57962a1b97d/lib/test_prof/before_all/adapters/active_record.rb#L15
It could have the following API:
Before.configure do |config|
  config.setup_before_all { }
  config.teardown_before_all { }
end

Closed by #140


Update copy-clan-usernames

Changed my username in Github, so this has to be changed aswell.

There's no change needed as GitHub redirects requests to your new username. If you have plugin updates, go ahead and update the username at the same time once you submit those updates.


Problem using the library on Android.

I first added this to gradle.kts, without any problem:
implementation("io.github.thoroldvix:youtube-transcript-api:0.3.6")
But when I added this code inside my main instance:

// Create a new default YoutubeTranscriptApi instance
YoutubeTranscriptApi youtubeTranscriptApi = TranscriptApiFactory.createDefault();

It gave me the following error. I added some screenshots.

Could you tell me what I'm doing wrong?

I've already managed to get it working.


My delete  action is not working

My delete action is not working. When I click on delete icon it shows me a confirmation box. After click on the confirm it does nothing.
How to fix this?

You may have an exception on a foreign key.
I would recommend to install Laravel DebugBar and check exceptions tab for more info.

I am using Exception Reporter and it has no error regarding this issue.

how can I remove this laravel Debugger ? It is very odd looking . :/

If you're using Chrome, refresh the screen and press Ctrl + Shift + I and go to the console. After you try to delete again, the error that is preventing it from working will probably appear. This delete action calls a DELETE route through a JQuery, so it will show an error in the console. Take a print and show it here.
But you're probably having an exception to some relationship. Is this record you're deleting related to another table?

No, I was trying to delete in the same table.
By the way, I have figured out my problem. It was a problem with routing. Thank you all. ^_^

In my case, I found it was a bug. When I use material design, it sends extra "plugin_ripples" parameter in data at src/Actions/Action.php file in the core of the library. unsetting it in both Action.php and interactor/Dialog.php files works perfectly.


Lockable Suit Storages & Paramedic Suit Storage

About the PR

Makes suit storages lockable, and adds a Paramedic suit storage & hardsuit filled locker to bring them in line with the rest of the jobs that have a suit in their locker.
This would have been a great opportunity for this PR to be useful, but oh well.
Why / Balance

Very silly that these couldn't be locked given how precious these suits are. It feels like a direct downgrade to map them without lockability.
Technical details

Basically just YML changes, Floofers supplied me with the sprites.
Media

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

Changelog

rd suit stealers will seethe

woooo!

I used to take the suits on a whim because i thought "if they aren't locked it must be for a reason".
Until i realized how much it shafted late join paramedics. I wonder what was the original idea behind not locking them.

I used to take the suits on a whim because i thought "if they aren't locked it must be for a reason". Until i realized how much it shafted late join paramedics. I wonder what was the original idea behind not locking them.
Maybe the point was to allow regular doctors to assist spaced patients? But they can just get a normal EVA if they are in such an emergency.

Yeah, pretty annoyed that the pr that properly added paramedic access was closed for basically no reason. I would vastly prefer that paramedics got their own access. Latejoining to find your suit gone, and its ALWAYS gone, makes paramedics into regular doctors but without chemistry access. It sucks.

the paramedic suit isnt a hardsuit

also you can give the para suit storage externals access instead since regular doctors dont have externals access

But then that means that so many other jobs like cargo, engineering etc can open it as long as they're in med (med is freely open so its likely that someone will open it)

the paramedic suit isnt a hardsuit

wrong, its a hardsuit. In the games files at least, in-game its just called a void suit since it looks cool.

a hardsuit has a togglable helmet, does the paramedic suit?

So in the files, should I call it a softsuit, a void suit...?
 Paramedic should just have their own access. I am absolutely not making it Externals so any ham-and-egger can come along and nab it too.

Yeah, pretty annoyed that the pr that properly added paramedic access was closed for basically no reason.

We’ve explained why in that PR. I spoke with Emisse (the head mapper) as well as other maintainers beforehand and they agreed it was unnecessary as well, so it really wasn’t closed for no reason.
On a different note, I believe this will break every currently existing map as they have the non hardsuit versions mapped yet no suit storage. Might be wrong though, and I don’t know what maps did when suit storages were initially added, so maybe it isn’t an issue.

Yes, the Paramedic suit storages will need to be mapped just like last time. I don't play on upstream maps so I'm not modifying any maps here.

Never said that was necessary- I’m fine with parameds having a suit storage. Just saying that there should probably be some solution to existing maps just not having parameds hits (other than the one in the bend) after this PR is merged and before every map is updated

I'm just gonna remove the Paramedic suit storages from this PR since Paramedics can't have shit in wizdetroit.

Never said that was necessary- I’m fine with parameds having a suit storage. Just saying that there should probably be some solution to existing maps just not having parameds hits (other than the one in the vend) after this PR is merged and before every map is updated

It's probably out of scope for this PR if mapping changes have to happen. I'd rather just get the lockers locking first, since that already works. It's like two yaml changes to add the paramedic storages back in anyway.

Never said that was necessary- I’m fine with parameds having a suit storage. Just saying that there should probably be some solution to existing maps just not having parameds hits (other than the one in the vend) after this PR is merged and before every map is updated

It's probably out of scope for this PR if mapping changes have to happen. I'd rather just get the lockers locking first, since that already works. It's like two yaml changes to add the paramedic storages back in anyway.

i could potentially do it myself as its not a hard change after this PR, else emisse would likely be willing to.


use new TxnGroup API to build a write pipeline

[ ] Update Gopkg.lock after badger merged corresponding PR

Want to replace #91 and #83


Added Set Operations Calculator

Fixes Issue🛠
Closes #1362
Description👨💻
Added Set Operations Calculator that takes input of two sets and performs set operations when you select one of the options shown that are, Union, Intersection, Cartesian Product, Difference, Power Set and Cardinality, and show the result.
Type of Change📄

[x] New feature (non-breaking change which adds functionality)

Checklist✅

[x] I am an Open Source contributor
[x] I have performed a self-review of my code
[x] My code follows the style guidelines of this project

Screenshots/GIF📷

@Rakesh9100 please review it


Added reference metadata + updated scripts.

Work in progress.
Adds metadata to the reference object by associating a JSON file with it.
Datadriven tests need to be fixed and extended.

Is there a reason we are using json instead of yml?  yml is a bit more human readable/writable, which is what these config files are, right?

I went with JSON over YML, as I'm starting to think of these files as something machine generated rather than user config files. The overall vision for the future is that we write a data importer program which will layout the files in the right way. Since we're using JSON a lot, this seems like a good option for this.

Pushed an update @dcolligan, I think it's ready for review now. This will cause some problems as it now requires the existance of the JSON files. It seemed like more trouble to optionally have them present, and to figure out how to handle md5s in this case.

If this is going to have the server choke on any data dir without json files, then we should do either one of two things:

update the example data with json files
have a server config option like "mandate reference metadata json files" that is off by default
I think just doing 1 is fine, since presumably there is no one using this version that isn't a dev.

I agree we'll need to update the example data, but we should wait until your readgroup stuff comes through first. The only people this will really annoy are the compliance suite people; they should get plenty notice this is in the pipeline, and perhaps a tool to generate the files.

So, maybe we should wait for the readgroupset stuff to land before merging this?

@dcolligan --- this has been updated, and should be mergable. I think we should merge this much and take on the remaining reference stuff afterwards, as this PR is big enough all ready.

+1


Books and categories components

[x] Books page is the default view:
Displays the list of books
Create a component called Book for displaying a single book
[x] Create a form for adding a book
[x] Create a separate component for this form
[x] Categories page:
Should display a button "Check status" only.

Hi @Draxeytina ,
Your project is complete! There is nothing else to say other than... it's time to merge it :shipit: Congratulations! 🎉
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear. Please, remember to tag me in your question so I can receive the notification.
As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

thank you @MeqdamAlqudah


Update cobra version

Update Cobra version in order to use the native cobra shell completion command
Add Go build in GitHub Actions workflow

Thanks for this!


Stuck Http probing non standard ports

Http probing non standard ports lasting hours
To Reproduce
Steps to reproduce the behavior:

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.

I thought it was just at  me, now I’m experiencing the same thing...  At this point the scanning stops.

Hey have you checked the process running on your machine? Keep in mind that this process tries to perform http probing on multiple ports and depending the number of subdomains found on your target this could take a long time

As a total process it tooks around 2 days to finish one scanning! I guess i modified all according a well scanning!

Well, this depends on multiple factors like your bandwith, the number of assets to scan, etc, not seems to be a bug tbh

How can be modified to minimize this time or scanning ports.. as well as for nuclei it tooks around 1 day also!