faq contrinution Nov

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?
FAQ contribution for Amit

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65607823d04daa008eba8c77--harness-developer.netlify.app


add support for volume plugins

This change adds the capability to do the equivalent of docker volume create and docker volume delete.
volume create will happen int two scenarios:

If we receive an explicit storage.volume.activate event
If we receive a compute.instance.create event that contains volume data in the volumesFromDataVolumeMounts field. That field is an array of volumes that should be on the host for the container. We iterate through it and create a volume appropriately.

LGTM

Addresses https://github.com/rancher/rancher/issues/2686


model contributions more in line with tf-slim

There are a number of good model implementations in third party repositories, and I think keras-contrib may be the most sensible place for them to go. However, each will need an example and a way to train, or a single script that works for all models. For that reason I propose a tf-slim style setup for vision based classification and semantic segmentation that can eliminate some of that overhead for users and provide more general utility than the keras/applications folder.
The following may be aspects of a model folder:

[ ] have a script equivalent to train_image_classifier.py explained in the tf-slim readme.
[ ] train_image_classifier might support both image labeling and semantic segmentation
[ ] model test folder, perhaps in a folder models placed in https://github.com/farizrahman4u/keras-contrib/tree/master/tests/keras_contrib

One such example model implementation is keras-resnet. The author is interested in submitting a pull request to keras-contrib.
I know that deep-learning-models exists but there are benefits keras-contrib could eventually provide:

provide way to train the models from scratch
community administrators (doesn't rely on availability of fcollet)
code and documentation to convert trained models from other deep learning apis like caffe

Is there interest in this type of contribution?
Would the ideal place to put such a thing be in kerascontrib/keras-contrib/models?

I have mixed feelings about this. While centralization can be a good thing, I think a model library might go beyond the intended scope of this repository. I had imagined this to be more of a testing ground for cutting edge features from recent literature. Perhaps it would be better to expand on fchollet's deep-learning-models if he's willing to grant permissions? What do others think?

This repo is intended for testing new features (new layers, backend functions, optimizers, etc), as well as examples that use such features. If you have models that use only existing Keras features, it would be better to PR to @fchollet's deep learning models repo.

We all agree that there should be an official keras repository for the community to contribute new models. Ideally it should be in an separate repo such as @fchollet 's deep learning models repo. However, there will be some models that may use new features which only exist in keras-contrib, so some of the new models MUST be pushed to keras-contri while the others goes to the model repo. Therefore I think we can just make it simple and use this repo as the only one repo for all models so that the contributors don't need to wonder which model repo they should pushed their models to.

Would it be possible to use @fchollet 's deep learning models repo and add a folder keras-contrib-models with a readme. It could be explicit enough to advise the user that he should use the keras-contrib repo with these models?
I could also be possible to explain how to set up a virtualenv and provide a Dockerfile so it will be easy to work in a clean environment?
I guess the reviewers from the keras-contrib repo should review the models associated with the features merged here.
What do you think?

Allow me to tweak my suggestion so that as you specified, I'll be proposing a specific new feature as well as examples of using that feature:

Define a standard class to create the most prominent published models of various types that can generate each model and configure the model including:

single class, multi class, segmentation, and instance segmentation, where applicable
network size for various hardware and applications (i.e. the many different scales of resnet/densenet that are available, for example)
provide callbacks or attachment points in this API for appropriate attachment points to add/train additional data

Create a new training class (or extend the existing one) to meet needs not met by the current one and automatically handle the standard class in (1).

Provide a built in multi-gpu training queue
Support deeply supervised learning
Provide hyperparameter search on these models (may be part of model API).
Automate training of a set of models/configurations for evaluation

Provide a simple applications folder app that initializes these utilities and demonstrates their use:

trains, saves, and evaluates models from the command line
loads and run a selection of datasets
provide a simple folder layout you can use to supply your own dataset

As requested, I believe this meets the standard @farizrahman4u is looking for of testing new features, as well as creating examples of using such features. However, this isn't a complete design yet, just a request for comment.

In other words, I'm trying to bootstrap some of the tools needed for applied R&D in a way that is much more disciplined than the quirky model/weight libraries in other deep learning projects.

I see, in my opinion (that could be wrong) it would definitely be possible to impose a simple style guide for the examples and maybe normalize the existing ones. The whole applied research pipeline is really interesting and important but it seems hard to integrate in this repo (API, weights management, experiment planning, etc.). I imagine it could be done in a separate project.


ConPTY changes to support Windows

Summary of the Pull Request
Changes for ConPTY to support bootstrapping Warp
@abhishekp106 I realized I didn't make a branch for the current ConPTY changes we've made to support bootstrapping Warp. This is the the branch if you need to make any changes while I'm gone.

Thanks!


config: set pull as required check in docs-cn and docs

Set pull as required check in docs-cn and docs.

/ok-to-test

/merge


Project not listing in Public observer

we created project in local,  but not listing on below public url
https://app.climatewarehouse.chia.net/#/projects?orgUid=all

Hi Deenapixel,
This is expected behavior as the public URL is owned by the World Bank. In order to have your data appear on that public URL, the World Bank would have to "subscribe" to your project data, and then it would be displayed.
Please note: anyone in the world may subscribe to and see the data you've submitted. But the data will not be listed on the public website until the World Bank subscribes to your data.
I'd love to understand more about your goals engaging with the Climate Warehouse software so that I may support you better! Are you wanting to share your registry climate data with the ecosystem?
Anyways - happy to help in any way possible. We can communicate with this ticket, or if you'd prefer to take it to a private channel, you may email me at<EMAIL_ADDRESS>or use the secure messaging app 'Keybase' to send me a message at ccornick.

Thank you for the clarification - ccornick.

If there is anything else you need, please feel free to open another ticket or reach out via one of the above methods!


UserDAO fails on testRegisterUser test

There is a problem with ids not matching in test, after persisting the new User entity.

Fixed by adding config = @SqlConfig(transactionMode = SqlConfig.TransactionMode.ISOLATED to annotation @Sql in UserDAOTest class.

Fixed by adding config = @SqlConfig(transactionMode = SqlConfig.TransactionMode.ISOLATED to annotation @Sql in UserDAOTest class.


Fix war_extras and webxml_webapp_extras examples

moved the existing war_extras example up under options:war_extras, then took a stab at an example for webxml_webapp_extras from a working copy in my project

Thank you for the contribution!

Thank you again.
Even though no code changed I published an update to NPM.
Bob.

On Apr 7, 2016, at 10:32 PM, Paul Thaden<EMAIL_ADDRESS>wrote:
moved the existing war_extras example up under options:war_extras, then took a stab at an example for webxml_webapp_extras from a working copy in my project
You can view, comment on, or merge this pull request online at:
https://github.com/MorrisLLC/grunt-war/pull/17 https://github.com/MorrisLLC/grunt-war/pull/17
Commit Summary
Fix war_extras and webxml_webapp_extras examples
File Changes
M README.md https://github.com/MorrisLLC/grunt-war/pull/17/files#diff-0 (5)
Patch Links:
https://github.com/MorrisLLC/grunt-war/pull/17.patch https://github.com/MorrisLLC/grunt-war/pull/17.patch
https://github.com/MorrisLLC/grunt-war/pull/17.diff https://github.com/MorrisLLC/grunt-war/pull/17.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub https://github.com/MorrisLLC/grunt-war/pull/17

My very first pull request ever!  Thanks to you for a great grunt task!


Update jquery.sliphover.js

Add one option for fix the direction.

it's better to list the possible value for the new added option direction in the code comment

Thanks for reply.
Now the possible value could be 0/1/2/3 or 'top'/'right'/'bottom'/'left'.

if the value's not number https://github.com/wayou/SlipHover/pull/23/files#diff-d60e13701f6a88a6aeb682913c43e2dbL60 this line may throw error.

Thank you for your advice!
Now added another switch statement to solve that problem !
Appreciating your guidance!


remove broken facts

Agree that the version must be determined from the binary itself as opposed to determining from a package.


index_to_string_table_from_file can use tf.string as vocabulary file

Fix #14505
How to test

[x] add test case
[ ] pass all tests

Can one of the admins verify this patch?

Jenkins, test this please.

Thanks for your approval, @martinwicke. Since the master branch often is broken recently, it might be not easy to make PR pass all tests.

I know. I'm making sure that failures are unrelated or pre-existing.


Update virtualbox and qqbrowser

In the future, please submit separate pull requests for each cask, as more often than not this type of pull request brings problems with one or more of them, and it halts the inclusion of the other ones.


Datetime tick precision

Introduced by #728 and #782.
DateTime(Y,M,D,h,m,s,fff) ignores tick precision. This also might behave the same way for FormulaValue.New().
FormulaValue.New( WithoutSubMilliseconds( datetime ) )

The issue here is: Do DateTimeValues in the Power Fx language carry tick precision?
The JS implementation does not, and the DateTime() function does not currently accept it.
But C# DateTime does.
This becomes an issue when serializing/deserializing a DateTime - if we do allow ticks, we don't want to lose them when round-tripping through a serializer.

I would posit that business applications would rarely (never?) need sub-millisecond precision. Is there any reason (besides "C# supports it") that we would need this kind of precision? If not, then the consistency across all platforms would probably make being a millisecond-precision language a better choice.


Update meow to v9 and devDependencies

Updating dependencies to help resolve https://github.com/mapbox/polyline/issues/84. Per https://github.com/sindresorhus/meow/issues/204#issuecomment-1017609191, I updated meow to v9 rather than to the latest since later versions require ESM.

@tristen it looks like tests using Node 10 are failing, which I believe is due to the update to eslint. Do you think it would make sense to update node_js in https://github.com/mapbox/polyline/blob/master/.travis.yml to more recent versions?

@dpdiliberto that seems like a sensible change to me 👍. Maybe we can replace 10 and 12 with more recent 16 and 18 versions.

Thanks @tristen , I updated the Node versions, and also upgraded the Travis build environment (per https://travis-ci.community/t/the-command-npm-config-set-spin-false-failed-and-exited-with-1-during/12909/7), since I was running into the following build error:
$ nvm install 18
node: /lib/x86_64-linux-gnu/libm.so.6: version `GLIBC_2.27' not found (required by node)
node: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.25' not found (required by node)
node: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.28' not found (required by node)
node: /lib/x86_64-linux-gnu/libm.so.6: version `GLIBC_2.27' not found (required by node)
node: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.25' not found (required by node)
node: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.28' not found (required by node)
The command "npm config set progress false" failed and exited with 1 during .

All checks are passing now 👍 .

Thanks @dpdiliberto and @tristen , please help publish a new release to NPM.

@leslieli done<EMAIL_ADDRESS>

Thanks @tristen .

Looks upgrading the meow to v9 doesn't fix the issue https://github.com/mapbox/polyline/issues/84, it is still referencing the<EMAIL_ADDRESS>not v7.5.2+
└─┬<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ └──<EMAIL_ADDRESS>    └─┬<EMAIL_ADDRESS>      └─┬<EMAIL_ADDRESS>        └─┬<EMAIL_ADDRESS>          └──<EMAIL_ADDRESS>

👋 @leslieli want to put a pull request together that addresses the issue? I'd be happy to review and cut a new release.

@tristen I am not quite familiar with the project and would like to suggest someone in your team could help on this. If possible, could you provide the estimation - when we can expect the fix release? Thanks.


[Schema Registry] Cross language design review

Ongoing discussion:

Design: API

what should the method return?

  - Schema:
      - content
      - SchemaProperties
 - SchemaProperties
     - id
     - version
     - ...

Design: API: SRAvroSerializer.serialize(..., schema):

what's the expected type of the parameter schema?
if string/bytes, then it's the SDK's duty to do normalize -- remove space from the input (\n\t, etc.) or the service would handle it
Laurent: good with bytes and string for p1

Design: Naming

response type(object)

SchemaId: in JS it's called SchemaIdResponse, probably we should consider a better/different/ name? what's naming convention here
Schema: same to the question above

content/string/schema

Option: all return the same object type e.g.: SchemaProperties

parameter name

Design: SchemaProperties dict mixin support?
Impl: Encoding

big/small endian problem to id/format identifier
Shall we use struct.pack/unpack to construct payload?
Ask service team/other languages how they impl this

Impl: Dependency

which packages are required for user (dependency)?

Laurent: postpone aio implementation later, only do the sync avro serializer now

Others

generate schema from class/type? the input being a object, is it pythonic?
future discussion, not now, but protobuf probably need to support this

Eng: Doc

Doc auto generation, need to ask Scott
Samples and sample readme release to official ms website
API reference

Finished discussion:

Impl: Parsing in sr and avsr

should we remove all the space (regular expression "\s") in the schema string user passes into our sdk?
it's the service's duty

Design: Typing

serialization type: string vs enum vs both
class a(str, Enum)

auto register schema for SR Serializer?

data collected and moved into onenote page, will spawn separate issues for each task


Hide captured pokemon when "Hide pokemon" is checked

I added a small feature to allow people to hide captured pokemon.
I put it under the search bar, but not sure if you can think of a better place for it to go.
A few things that probably need to be done:

[ ] Fix styling issues, it looks kinda ugly, probably need to make it nicer
[ ] Probably save the setting in local storage - it currently gets cleared, just like the seearch query, but this probably makes more sense to persist

In terms of how this works, I made a few design decisions. Feel free to tell me I'm wrong with any one them.
Firstly, the default view looks exactly the same, just with the "hide completed" checkbox appearing under the search box.
When you check this, those pokemon are hidden from view. We just don't render them.
You'll notice that it doesn't show all 30 spaces in this screenshot. It would defeat the purpose of this feature to do so, as you'll jsut end up with boxes of 30 empty, which makes you scroll as much as you would before. So I've basically modulo-ed the boxes to 6. So if there are 28 in it, you'll get 30 spaces, but if there's 1, you'll get 6, 8, you'll get 12, etc.
This also lowers the whitespace for the alola/galar sections, they take up fewer rows now.
Next, when you have a box fully done, it'll completely hide from view. I thought this made most sense as possibly seeing a box with no pokemon would be a bit confusing, and if you are selecting "Hide Completed" it'll be to save space and mental calculations on looking through the list. So this made most sense to me.
Thanks for the website, I really like it, and thought I'd add the code for this feature that I'd really like. Happy to modify things as you want. Let me know if you want this feature or not. As shown above, there are a couple of things I need to do before this would be ready for merge anyway, but thought I'd get it up now before I spend too much time pouring over CSS.

hey @McInkay, thanks for the contribution! if you check out our roadmap, we’re actually going to be working on this feature soon, but we were planning on implementing a bit differently than the route you’ve chosen to go with the boxes. as such, i’m going to close this PR for now but we may use it for some code inspiration when we’re implementing our version :slightly_smiling_face:
in the future if there are some changes you want to make, it’s probably a good idea to comment on the issue for it to see how it aligns with our plans for the site and/or make sure it isn’t something we’re already in the middle of working on. if it isn’t, and there’s room for an external contribution, then a comment will helps us guide you on what we were thinking for the functionality and ui/ux. this will help us both out to make sure were all on the same page before any time is spent!

I mean, I'm happy to make the changes if there's something in how I did it you disagreed with. This was my first version of the feature, and was just the way I thought it should work. Happy to polish it up and change some of the logic if you want it done differently.
To be honest, it was quicker to implement this than to comment and then wait to see what happened. I didn't polish it up deliberately so I didn't spend too long working on it to find out you didn't want it that way, that's fine.
I did notice it was on your roadmap, but as far as I could tell it wasn't currently in progress, so I thought I could help out. There's no point in duplicating the work that is already done in it, unless this is hugely different from how you want it done.


add 'xmlns' as a default svg attribute

When you call makerjs.exporter.toSVG() and specify useSvgPathOnly: false the resulting svg by default does not have a xmlns attribute.  This causes the browser to display it as XML (not SVG).
Without this PR, you can add it as an svgAttrs in the exportOptions, but it seems like a sensible default.
{
  useSvgPathOnly: false,
  svgAttrs: {
    xmlns: "http://www.w3.org/2000/svg"
  }
}

var makerjs = require('makerjs');
var square = new makerjs.models.Square(100);
var svg = makerjs.exporter.toSVG(square, {useSvgPathOnly: false});
console.log(svg);

Before

<?xml version="1.0" encoding="UTF-8"?>
<svg width="100" height="100" viewBox="0 0 100 100">
   <g id="svgGroup" stroke-linecap="round" fill-rule="evenodd" font-size="9pt" stroke="#000" stroke-width="0.25mm" fill="none" style="stroke:#000;stroke-width:0.25mm;fill:none">
      ...
   </g>
</svg>

After

<?xml version="1.0" encoding="UTF-8"?>
<svg xmlns="http://www.w3.org/2000/svg" width="100" height="100" viewBox="0 0 100 100">
   <g id="svgGroup" stroke-linecap="round" fill-rule="evenodd" font-size="9pt" stroke="#000" stroke-width="0.25mm" fill="none" style="stroke:#000;stroke-width:0.25mm;fill:none">
      ...
   </g>
</svg>

I think we may want to add some client-side tests to verify that the Playground will still work, as I believe that this will cause any DOM manipulations such as element.getAttribute() to only work with element.getAttributeNS.

I was wondering if there might have been a reason this was left out and maybe the Playground is it. Maybe we can add something to the documentation instead if you decide this feature can't be merged.   It will help someone who is struggling with XML being displayed instead of the image.

@danmarshall - Are there existing "client-side" tests that you would like me to enhance or do I need to create new ones?
I did some testing in chrome console and document.getElementById('svgGroup').getAttribute("style"); is working.
I also found this commit, which might be related: https://github.com/microsoft/maker.js/commit/585a73cfe019125a63d865d1306d8fc60ac1fa76
I am happy to add tests to get this PR finished, so let me know what you are specifically looking for.

Hi @boomer196 I'd say that my main concern would be breaking code in someone else's project if they have come to depend on the previous functionality.
Other than that, I agree with you that it is probably a sensible default.

I'd say that my main concern would be breaking code in someone else's project if they have come to depend on the previous functionality.

Just chiming in with my 2¢ here:

This package is still in v0 land. Semver Rule #4 states that backwards compatibility isn't guaranteed when < v1.
Nonetheless, this change definitely deserves a minor version bump (from v0.14 to v0.15, or similar).
The xmlns attribute is referenced all throughout the SVG spec. It's effectively mandatory.
If someone does need the old behavior, they can always lock themselves to v0.14 (that's what package.json is fo), or set xmlns: null by overriding their own svgAttrs to retain the old behavior.

Please do merge this. It's a very sensible default.

Thanks for chiming in @rmm5t , those are good points. Appreciate the thoughtful discussion! I think this is good to merge :)

0.15.0 published


False positive membership with new swift Sets

When checking whether or not an instance of a protobuf object is contained in a swift set, I'm getting false positives (says the value is in the set when it shouldn't be).
I can provide an example if useful.

Do you have example with descriptions of problems?

i'll put some together tomorrow


Server crashes when starting without a data.json file

There is currently no sample data.json in the /dashboard-api directory. Since there are multiple dependencies on the file within the directory, a server would simply crash. I think we need to provide a sample data at the very least so that anyone can spin up the local server.
As @RandellDawson mentioned, that solution is not ideal because one cannot test the most up-to-date data. We could potentially write the scripts to pull the most up-to-date from the production server once things are settled.

I just added a sample_data.json which one can make a copy and name it data.json to have something.


Parser Error: Could not find type "Camera3DResource" in the current scope when altering C# scripts

Issue description
When trying to run my project in the editor after making various changes to my C# code (completely unrelated to PhantomCamera), I sometimes get the error 'Parser Error: Could not find type "Camera3DResource" in the current scope.' This only seems to happen after re-compiling my altered C# scripts; once the error appears, re-running the project resolves it.
Here's what the error looks like:

I don't reference Camera3DResource in any of my C# scripts, the error seems to point to phantom_camera_3D.gd at line 101. For some reason the project can't seem to find Camera3DResource when initially compiling, but then finds it after running again.
I'm working in the .NET version of Godot 4.2.1. Could this error somehow be related to using C# scripts?
Appreciate any help looking into this!
Steps to reproduce
In my project with PhantomCamera set up:

Open a random C# script.
Alter the code in some way (could be as small as entering a new line).
Run the project in Godot.
The error appears at an unpredictable frequency (maybe 20% of the time)

It's difficult for me to provide a project since I have many addons that break the project when trying to reduce it to a Minimal reproduction project, but if there's any interest I can help make a project that reproduces this.
(Optional) Minimal reproduction project
No response

Have a hunch this will be resolved in the 0.7 release, as it will fundamentally change the property system, but will run some tests and see if I can replicate it still.

Have a hunch this will be resolved in the 0.7 release, as it will fundamentally change the property system, but will run some tests and see if I can replicate it still.

Cheers thanks for looking into this! Let me know if you'd like me to test any potential fixes on my end, happy to help how I can. :]

If you have updated to the latest release, does that resolve your issue?

I had the same bug before 0.7v, which seems to fix it (testing several times by repeating the method described above).
Thanks for the fix !

Thanks for testing @Natorkawl!
Closing issue, but feel free to reopen if it appears again.


Remove teads-curve from if-unofficial-plugins and replace with generics pipeline

Why: Sub of #546
What: Remove the teads-curve from if-unofficial-plugins and replace with generics pipeline
Scope of work:

[ ] Implement acceptance criteria
[ ] Unit tests includes positive and negative test cases in if reposiroty
[ ] unit tests have 100% coverage and passing
[ ] Create PR
[ ] Incorporate PR feedback

Acceptance criteria:

[ ] TBC

@jmcook1186 please see the issue and update Scope of Work and AC as needed 🙏

note - this will stay blocked until the interpolation plugin is shipped


Fixed typos in french translation.

@mariannems, Can I reach out to you in the future for reviewing changes in French translation?

Sure!


404 when running curl install.sh

Environment, CPU architecture, OS, and Version:
x86_64, MacOS Monterey
Describe the bug
When trying to install LocalAI i get "curl: (22) The requested URL returned error: 404 ".
I have updated curl and made sure its allowed internet. Also tried running with "sudo".  I tried pinging https://localai.io , unable to resolve host. However google.com works fine.
Cheers

We currently don't support x86_64 mac, but that's a good point - likely we can add back support to it (as we used to).

Thanks for your response. Hope to see support added again.

Thanks for your response. Hope to see support added again.


Virtual hardware buttons cover footer

Ionic version:  (check one with "x")
[ ] 1.x
[x] 2.x
[ ] 3.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
Current behavior:
Our client has ordered this phone for their users (https://www.amazon.com/Motorola-Moto-XT1650-32GB-BLACK/dp/B01IW7N900). The phone features what we're calling 'virtual hardware buttons'. That is, the hardware buttons are rendered on screen, they aren't physical buttons on the phone. When the app loads, the hardware buttons cover the footer content. This is causing confusion among the users.
Is there a way to account for these virtual buttons (beyond adding additional padding)? If not, we can add more padding, just wondering if anyone came across an elegant solution.

Hey @bryantmakesprog , this is definitely an interesting issue. My nexus 6 also has software buttons (as they are standard in stock android), would you be able to share a repo i could use to reproduce this?

I'll see if I can throw together something to reproduce the issue. We're scrounging for a device to replicate on :stuck_out_tongue_closed_eyes:

😄 sounds good!

I have a HTC One M8 test device that also uses the "soft" hardware buttons but I don't have any issues with overlap over the footer in my app they are always underneath where they should be.
@jgw96  If there is a test repo I am also happy to spin it up on my device and provide results if that is helpful.

@bryantmakesprog any luck on reproducing this? @ghenry22 I'd gladly share a repo you can use to reproduce this issue if I had one, but I have not been able to repro this issue at all haha

Yeh I can't reproduce in my personal project either

@jgw96 @ghenry22
My apologies, I haven't had a moment to generate something. Due to time constraints, we've just moved the footer content to somewhere other than the footer :stuck_out_tongue_closed_eyes:
If y'all aren't seeing it I'd probably close this issue. It is and issue, but my issue is resolved :+1:


Fix ambiguous types in dotnet

This PR closes #1437
Changes include: -

Adds support for escaping only types by taking advantage of the existing ReplaceReservedNames function
Adds support for escaping strings used in XML comments to reduce likelihood of warning in generates SDKs
Fix unnecessary of escaping Enum members in dotnet as they will be capitalized
Fixed unnecessary use of fully qualified type names in Dotnet as the new  function DoesTypeExistsInTargetAncestorNamespace would also match types in the same namespace.
Fixed CSharp and Shell tests that had set up orphaned instances of CodeClass which led to test failures.

Successful generation in dotnet can be verified at the link below
https://dev.azure.com/microsoftgraph/Graph Developer Experiences/_build/results?buildId=71393&view=results

Thanks @baywet


Contrast in code blocks and inline code is too weak in Dark theme in Safari on macOS and iOS

Describe the bug
When using Microsoft Docs with the Dark theme, code blocks and inline code use a light gray text on a near-white background. This is difficult to read due to the weak contrast between those two colors.
Also note that it doesn't meet the web accessibility contrast ratio requirements: https://webaim.org/resources/contrastchecker/?fcolor=CCCCCC&bcolor=F3F3F3
To Reproduce
Steps to reproduce the behavior:

Go to https://docs.microsoft.com/en-us/aspnet/core/mvc/views/overview?view=aspnetcore-2.1
Click on the "Dark" link in the top right, if the Light theme is currently enabled.
Scroll down to the "Creating a view" section.
Observe the CSHTML code block, as well as the handful of inline code instances in the subsequent paragraph, and notice the weak contrast.

Expected behavior
The contrast of the code background and foreground colors should be stronger. Since this is a dark theme, I would expect the background to be dark and the text to be light, like the rest of the theme. The near-white background used for code is jarring when reading the inline code instances.
Screenshots
Here is a screenshot of the section referred to in Step 3.

Desktop

OS: macOS
Browser: Safari
Version: 12.0

@ShevaDas @kubeaver Can you take a look at this feedback?

Adding @dend for code samples

@kubeaver I don't have access to that page; will it be fixed relatively soon?

@mrlife it's in the queue. Checking on it now. I don't yet have an exact ETA as there are a few other things before it.

@mrlife the link points to an internal bug tracker, so you won't be able to access it. We are working on getting this fixed. I will update the issue once we resolve the underlying bug.

Looks fixed, thank you!

@mrlife Thanks for the report! I'll close this out now. Feel free to open a new issue if other things crop up.


Fix mismatch of shared console log file extension with its content type

Issue description
Currently, the console contents are shared as a file with the .txt file extension when tapping on "..." -> "Share console" in the Tabris.js iOS client, although they have HTML contents.
Proposed solution
The console contents should be shared as a file with the .html extension when tapping on "..." -> "Share console".
Alternative solutions
Alternatively, the contents of the console logs could be changed to simple plain text, bringing the behavior in line with the Android client.

When sharing console as an activity item we provide an instance of NSAttributedString.
That, later on, is interpreted by the UIKit in UIActivityViewController that offers to transfer it as plain text through AirDrop. We cannot change that.
We could make a temporary HTML file with content generated from NSAttributedString and use it as an activity item. In this case, we would share a file with an HTML extension, but you would lose the possibility to simply copy the contents of the console to paste it later on in any other text editor.
I logged a few messages with console.jsx snippet to see how it looks now and I haven't found HTML tags in the file I AirDropped to my MacBook. text-282D6EA537B0-1.txt
This might mean that NSAttributedString is not always converted to HTML, just in some specific cases. Do you remember in which case you got HTML code instead of plain text?


Not grown plants give money

I have a farmer job where you have to break full grown crops to earn money.
The jobconfig.yml looks like this :
Break:
  BEETROOT_BLOCK-3:
    income: 1
    points: 1.5
    experience: 4
  CROPS-7:
    income: 2
    points: 1.5
    experience: 5
  CARROTS-7:
    income: 1
    points: 1.0
    experience: 4
  POTATOES-7:
    income: 1
    points: 1.0
  COCOA-2:
    income: 0.5
    points: 4
    experience: 3.5

But when i break not grown crops (just planted / first stage) it gives me money anyway. I thinks the data values don't work in 1.13 because of the new IDs system without data values.

Did you try with ID instead of names?

Yep, same problem using IDs.

_Fixes, commit_

ac31db0927384f66c1862582c5e91081f86bedbf


[corecheck/snmp] Refactor SNMP corecheck

Regression test: https://github.com/DataDog/integrations-core/pull/9970
What does this PR do?
1/ Refactor SNMP Corecheck to make it more modular
2/ Makes it easier to implement SNMP Autodiscovery https://github.com/DataDog/datadog-agent/pull/8996
Motivation
Required before implementing https://github.com/DataDog/datadog-agent/pull/8996
Additional Notes
Related PRs:

https://github.com/DataDog/datadog-agent/pull/8996
https://github.com/DataDog/datadog-agent/pull/8971
https://github.com/DataDog/integrations-core/pull/10079

Describe how to test your changes
1/ Check that the refactor doesn't introduce any regression using e2e tests
ddev env test --base --new-env snmp --agent datadog/agent-dev:alex-snmp-corecheck-refactor

2/ Check that the refactor doesn't introduce any regression using some manual testing
3/ Check all tests are still there looking at CI builds or running locally using and comparing with Agent main branch:
inv test --targets ./pkg/collector/corechecks/snmp

Checklist

[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] The need-change/operator and need-change/helm labels has been applied if applicable.
[ ] The appropriate team/.. label has been applied, if known.
[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] The config template has been updated if applicable.

Note: Adding GitHub labels is only possible for contributors with write access.

As an improvement for reviewers experience on next refactoring PRs I suggest editing the commits history to drive reviewers through the changes. Reviewing such a big PR slows the approval time, plus can lead to missing potential errors in code.


Added anyType template option

Added anyType template option to be able to replace, for example Any or [String: Any] with AnyCodable and [String: AnyCodable].
Did not find a way to do it using only template.

I think it’s a great addition ! I did often do it in my template too.
Please consider updating the Changelog and Readme files.
I will even propose it to enable by default.
Thanks for tour contribution @tapoton.

Hi @tapoton, thanks for the PR! As @mackoj noted could you please also add:

a changelog entry
an entry to the template (with a value as Any) https://github.com/yonaskolb/SwagGen/blob/master/Templates/Swift/template.yml
some documentation in the readme in the Swift Template section https://github.com/yonaskolb/SwagGen/blob/master/README.md#swift-template-1

Hi!
Added necessary changes, enabled Any -> AnyCodable override in default template

Looks like all changes made by FixturesTests are valid. Please, take a look

please @tapoton could you consider moving it default value to Any as @yonaskolb requested and I think you will be in a good place ;)

My bad, didn't get that at first

@tapoton should your feature use the same option as codableResponses ? Because the goal of codableResponses is to made the model Codable.

I think this can stay separate to codableResponses


DNN-8639

Hello.
I deleted the references to missed files in Website project.
I didn't sign Contributor License Agreement, because I didn't find where i can do it.
Regards,
Richard

@zyhfish can you please clean this up and submit a new PR. We will close this one.

merged, and remove un-need changes.


Release 0.6.0

Changelog:

Allow creating snapshots from a single C source file
Add extended analysis statistics
New pattern for matching reordered binary operations
New pattern for variable grouping
Nix flakes support for building and development
Unification of applied optimizations for various build commands
Fix comparison of PHI nodes and union types

Unfortunately, we still don't support LLVM 17 so we won't be able to build this release to newer Fedora versions. Still, I'd do the release since there's a lot of new changes coming. We may still do a feature release once we add the LLVM support.

The version number should probably be changed also here
https://github.com/diffkemp/diffkemp/blob/e6cded9aefd2e51ada26c972f499fe08d442d789/flake.nix#L20

Nice catch! Fixed.

, otherwise I think we can do the release.
Originally, I encountered two problems when I tried the current (respectively previous) RPM package on Fedora, but after further investigation it should be fine:
[...]

If you wouldn't mind, I'll ping you when the new RPMs are ready and you could test if the above issues are resolved.

If you wouldn't mind, I'll ping you when the new RPMs are ready and you could test if the above issues are resolved.

Sure, I can test it.

@PLukas2018 here's the list of artifacts from the newest RPM scratch build (for Fedora 39, i.e. LLVM 17):
https://download.copr.fedorainfracloud.org/results/viktormalik/diffkemp/fedora-39-x86_64/07475809-diffkemp/
If you could try to download the RPM, install it, and check if the errors that you previously encountered disappeared, that would be great. I had to do some changes to the specfile, see #339 (these are already included in the above scratch build).
Thanks!

If you could try to download the RPM, install it, and check if the errors that you previously encountered disappeared, that would be great. I had to do some changes to the specfile, see #339 (these are already included in the above scratch build).

So I tried it (I tried build, build-kernel, view commands) and did not find any problem, everything worked well.

Thanks! v0.6.0 is out:
https://github.com/diffkemp/diffkemp/releases/tag/v0.6.0


Update requirements.txt for CatCmaSampler

Contributor Agreements
Please read the contributor agreements and if you agree, please click the checkbox below.

[x] I agree to the contributor agreements.

[!TIP]
Please follow the Quick TODO list to smoothly merge your PR.

Motivation

To avoid installing cmaes from GitHub.
Description of the changes

I've just released cmaes v0.11.0 with CatCMA support, thus we can install cmaes via PyPI.
TODO List towards PR Merge

[ ] Copy ./template/ to create your package
[ ] Replace <COPYRIGHT HOLDER> in LICENSE of your package with your name
[ ] Fill out README.md in your package
[ ] Add import statements of your function or class names to be used in __init__.py
[ ] Apply the formatter based on the tips in README.md
[ ] Check whether your module works as intended based on the tips in README.md

Please let me know if I need to check the todo list :pray: This PR simply updates requirements.txt though.


Put on hackage please.

There's already an sdl2-ttf package on Hackage, it just doesn't work with the current version of the sdl2 package at the moment. You can find its repo here. It differs greatly in its API from my version.
I wouldn't feel comfortable with uploading another sdl2-ttf binding to Hackage. There's been a lot of duplication of effort and I should have just contributed to @mankyKitty directly. The best option would probably be to fix Sean's sdl2-ttf so it works with sdl2 again.

Actually, scratch that, I think Sean's sdl2-ttf works with sdl2 now.

I'm currently not dog-fooding my own library, but I do pay attention when people send PRs and issues my way. So if there are things that you'd like to see then please shout out and I'll do what I can. :)

Thanks for your responses.
I have a problem with @mankyKitty's version. I want to call this function:
SDL.createTextureFromSurface
  :: transformers-<IP_ADDRESS>:Control.Monad.IO.Class.MonadIO m =>
     SDL.Renderer -> SDL.Surface -> m SDL.Texture

Which takes an SDL.Surface. But I get an SDL.Raw.Types.Surface.
Font.renderTextSolid
  :: Graphics.UI.SDL.TTF.FFI.TTFFont
     -> String -> Color -> IO (GHC.Ptr.Ptr SDL.Raw.Types.Surface)

If it helps, this is how my version creates a Surface. You seem to be doing the same thing when you define textSurface.


Typo

Typo in the answer for first question
Is Azure Synapse Link supported for all Azure Cosmos DB APIs?
Azure Synapse Link is supported for the Azure Cosmos DB API for NoSQL and MongoDB. The support for API for Gremlin is now in preview.
Correction: Azure Synapse Link is supported for the Azure Cosmos DB SQL API and MongoDB. The support for API for Gremlin is now in preview.

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

ID: 3ba968c1-aa44-fe88-5574-ee2ab007bf09
Version Independent ID: 4c9a84cb-5e7b-0291-babe-b3fa6d41e1e3
Content: Frequently asked questions about Azure Synapse Link for Azure Cosmos DB
Content Source: articles/cosmos-db/synapse-link-frequently-asked-questions.yml
Service: cosmos-db
GitHub Login: @Rodrigossz
Microsoft Alias: rosouz

@vee73
It would be great if you could add a link to the documentation you are following for these steps? This would help us redirect the issue to the appropriate team. Thanks!!

https://learn.microsoft.com/en-us/azure/cosmos-db/synapse-link-frequently-asked-questions

@vee73
Thanks for your feedback! We will investigate and update as appropriate.

@vee73 The SQL API just got renamed to NoSQL API.

#please-close

@vee73 Thanks for the heads up. The fix will be live soon.


Implementing CIS supervision and establishment timeout

This implements CIS supervision timeout as well as CIS establishment timeout.
It changes also the way supervision timeout is stored and accessed for ACL. This in order to not have to do time vs. interval math for individual CIS's
This is work that has been in use for two months, with which just over 500 EBQ tests are passing.

rebased


feat: DH-16737 Add ObjectManager, useWidget hook

Hook for loading a widget that makes it easy to use
ObjectManager context allows for implementation to handle loading the object

On Core side, we have a very simple ObjectManager, as we just have one connection
On Enterprise side, the ObjectManager provided to the context manager will need to handle fetching from queries, and will be able to handle more scenarios (such as when a query is restarting)

Use with https://github.com/deephaven/deephaven-plugins/pull/502

I think the status messages are fine (as long as they're constants). I've used tanstack query some which has each state as a boolean value in the returned object. When I was using it, it seemed to be a fine pattern as well. In the case of tanstack query it's ajax/browser fetch requests with optional refetch intervals/events.
https://tanstack.com/query/latest/docs/framework/react/reference/useQuery


Validate importer to ingestion takes <2s

Import a 20k+ record file data through the new importer into Pinot.
Verify the roundtrip time <2s

Tested this with a records file from Performance environment containing 24667 records.
Testing just one transaction took 665 ms

So After further Investigation the new importer is taking most of the time and actual Kafka->Pinot->Rest Query takes about 290ms.


[Bug] There is a problem with the distribution strategy of the orca unionall operator.

Cloudberry Database version
master
What happened
The orca optimizer currently returns the ANY policy for the first child of a unionall-like node, which will result in Gather Motion for the downstream chi
ldren and a 1:n Redistribution for the upstream.
for example:

    ->  Redistribute Motion 1:3  (slice2)
       ->  Append  (cost=0.00..863.91 rows=18001 width=12)
          ->  Finalize Vec Aggregate
            ->  Gather Motion 3:1  (slice3; segments: 3)
              ...
       ->  Gather Motion 3:1  (slice4; segments: 3)
          ->  HashAggregate

What you think should happen instead
remove child gather motion and upstream redis motion, just add 1:n to first child.
How to reproduce
tpcds 167 query
Operating System
centos7
Anything else
No response
Are you willing to submit PR?

[X] Yes, I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct.

gpdb fix this change, https://github.com/greenplum-db/gpdb/commit/0cd056a0a3d3c30a1d6d4479e67802b6673118c7


(GEM) version bump for beaker-rspec 5.0.1

Merged build triggered.

Merged build started.

Merged build started.

Merged build triggered.

Merged build finished.

:red_circle: Test failed.
Refer to this link for build results: http://jenkins-beaker.delivery.puppetlabs.net/job/Beaker-rspec Combined Smoketest/6/

Known puppetlabs-inifile regex matching failure (not related to beaker-rspec code change).

:red_circle: Test failed.
Refer to this link for build results: http://jenkins-beaker.delivery.puppetlabs.net/job/Beaker-rspec Combined Smoketest/7/

Merged build finished.


Latest version 7.1.4 still can't download files

If I grab the URL from the Log, paste in browser - it downloads just fine. Now with the new version I put the cab file in the temp folder and it still says it can't read, and now I have no HP models listed. :(

======== Errors(s) Occurred ========	DriverAutomationTool	7/1/2022 10:55:14 AM	26940 (0x693C)

[Error] - The handle is invalid. (Exception from HRESULT: 0x80070006 (E_HANDLE))	DriverAutomationTool	7/1/2022 10:55:14 AM	26940 (0x693C)

DriverAutomationTool	7/1/2022 10:55:14 AM	26940 (0x693C)

Warning: Failed to find HP XML file: HpCatalogForSms.latest.cab	DriverAutomationTool	7/1/2022 10:55:14 AM	26940 (0x693C)

I tried this on a different machine in a different OU with administrative software not installed - and everything worked as I would expect. Problem is on my end.


How to export creation.Timestamp and last.TransitionTme of each node and pod

I would like to have every time stamp of each node and pod in a .csv file any guidance?

Hello! @iqbalkhan82
Thanks for your interest in this product.
Sorry, 😢  currently, our simulator does not provide any specific method to get retrieve the fields for resources.
But the simulator opens the port 3131 to access the api-server.
For example, you can list all Pod data on the specified namespace, like below.
Therefore, it is possible to achieve the objective by using a similar API with a generic api-server.
(But you may need to parse json or some client library...)
$ curl http://localhost:3131/api/v1/namespaces/default/pods
{"kind":"PodList","apiVersion":"v1","metadata":{"resourceVersion":"271"},"items":[{"metadata":{"name":"pod-dp4wp","generateName":"pod-","namespace":"default","uid":"1499876a-f563-49b3-83c6-4053a4036ad5","resourceVersion":"221","creationTimestamp":"2023-01-18T16:56:31Z","annotations":{"scheduler-simulator/filter-result":"{\"node-dq5f6\":{\"AzureDiskLimits\":\"passed\",\"EBSLimits\":\"passed\",\"GCEPDLimits\":\"passed\",\"InterPodAffinity\":\"passed\",\"NodeAffinity\":\"passed\",\"NodeName\":\"passed\",\"NodePorts\":\"passed\",\"NodeResourcesFit\":\"passed\",\"NodeUnschedulable\":\"passed\",\"NodeVolumeLimits\":\"passed\",\"PodTopologySpread\":\"passed\",\"TaintToleration\":\"passed\",\"VolumeBinding\":\"passed\",\"VolumeRestrictions\":\"passed\",\"VolumeZone\":\"passed\"}}","scheduler-simulator/finalscore-result":"{}","scheduler-simulator/postFilter-result":"{}","scheduler-simulator/score-result":"{}"},"managedFields":[{"manager":"simulator","operation":"Update","apiVersion":"v1","time":"2023-01-18T16:56:31Z","fieldsType":"FieldsV1","fieldsV1":{"f:metadata":{"f:annotations":{".":{},"f:scheduler-simulator/filter-result":{},"f:scheduler-simulator/finalscore-result":{},"f:scheduler-simulator/postFilter-result":{},"f:scheduler-simulator/score-result":{}},"f:generateName":{}},"f:spec":{"f:containers":{"k:{\"name\":\"pause\"}":{".":{},"f:image":{},"f:imagePullPolicy":{},"f:name":{},"f:resources":{".":{},"f:limits":{".":{},"f:cpu":{},"f:memory":{}},"f:requests":{".":{},"f:cpu":{},"f:memory":{}}},"f:terminationMessagePath":{},"f:terminationMessagePolicy":{}}},"f:dnsPolicy":{},"f:enableServiceLinks":{},"f:restartPolicy":{},"f:schedulerName":{},"f:securityContext":{},"f:terminationGracePeriodSeconds":{}}}}]},"spec":{"containers":[{"name":"pause","image":"k8s.gcr.io/pause:3.5","resources":{"limits":{"cpu":"100m","memory":"16Gi"},"requests":{"cpu":"100m","memory":"16Gi"}},"terminationMessagePath":"/dev/termination-log","terminationMessagePolicy":"File","imagePullPolicy":"IfNotPresent"}],"restartPolicy":"Always","terminationGracePeriodSeconds":30,"dnsPolicy":"ClusterFirst","nodeName":"node-dq5f6","securityContext":{},"schedulerName":"default-scheduler","priority":0,"enableServiceLinks":true,"preemptionPolicy":"PreemptLowerPriority"},"status":{"phase":"Pending","conditions":[{"type":"PodScheduled","status":"True","lastProbeTime":null,"lastTransitionTime":"2023-01-18T16:56:31Z"}],"qosClass":"Guaranteed"}}]}

/close
/kind support
As @196Ikuchil said, you can fetch any resources from kube-apiserver.

I'll close it but feel free to reopen it if you have anything further.


A Parser-blocking, cross-origin script, http://localhost:3000/dist/bundle.js, is invoked via document.write. This may be blocked by the browser if the device has poor network connectivity.

when i run   nam run dev, and the app is running ,but it show error message like this
A Parser-blocking, cross-origin script, http://localhost:3000/dist/bundle.js, is invoked via document.write. This may be blocked by the browser if the device has poor network connectivity.
why? I use mac os 10.11.5, it cannot work,
and  i use window, it ok,
how can i do??

This is not really a problem. The boilerplate app should still work.

@amilajack but why it report cross-origin  error in mac os?

It is because creating inserting a script dynamically at runtime is terrible for performance. Browsers want to be able to walk the DOM tree without having to evaluate JS to insert scripts that will pull in other scripts.

@amilajack  and how to do ?

The message can be ignored.

https://developers.google.com/web/updates/2016/08/removing-document-write

Reopening because I think this actually may have some noticeable performance.

@amilajack  thank you , solved!

http://blog.dareboost.com/en/2016/09/avoid-using-document-write-scripts-injection/
Dynamic script injection will be disallowed in Chrome 54. We definitely need to migrate from this soon.

I am already seeing errors from this using electron 1.4.3. Not sure when this happened but in my haste, I downgraded to electron 1.3.4 and my errors went away. Will have to slowly increment down in electron versions to find the latest compatible version.
I get the Parser-blocking error both in terminal and in console of electron app.

Are they warnings or errors?

Last night there were errors that I assumed were related. After downgrading electron, I did a bit more work. I just updated to the latest electron again though and now I'm just getting the Parser-blocking warning. So perhaps the error was unrelated as I cannot reproduce now.

I have this too with this repo. Downloaded about an hour ago, npm i && npm run dev show me

Chrome Version 54.0.2840.71 (64-bit) on MacOs

Its probably caused by something else. The first two errors are just warnings

@Ugarz Did you see #400

I replaced
document.write(link.outerHTML) with document.body.appendChild(link)
and document.write(script.outerHTML) with document.body.appendChild(script)
The resulting output is the same and it gets rid of the errors.

Made https://github.com/chentsulin/electron-react-boilerplate/pull/604 for this. Would be great if you could review it just to confirm


bcachefs: EOL kernel, out of date, unstable.

Checklist
This is a thread to raise discussion on how to proceed with maintenance with this package.
I've experienced several file system corruptions.
The kernel that this build pulls in is end of life, and there's no other option but to use it or hack on nixpkgs, since this package is not a patch to NixOS kernels.
On disk format has changed and requires the use of the compat-branch tools to fsck and upgrade your disk format, this is only a major issue if you run bcachefs on root, as your system will fail to boot without first upgrading the disk format. compat-branch will need to be included as a transitional tool for a while (1-2 years?).
My proposal:

Adapt to a patch on NixOS kernel, this way we can get the latest version of the kernel and not be stuck on x.x.0

Remove hard dependency on specific kernel package, so users can select what kernel package they want to use.

package both compat and master branches of bcachefs-tools.

Use evilpiepirate git source because it's more up to date.

[x] Checked the nixpkgs master branch

[x] Checked the nixpkgs pull requests

Project name
nix search name: bcachefs-tools, linuxPackages_testing_bcachefs

current version: 5.9.0 kernel, 2020.11.20
desired version: flexible kernel, open to suggestions on maintaining
Notify maintainers

maintainers: @davidak @chiiruno

https://discourse.nixos.org/t/systemd-random-seed-service-constantly-fails/12137
@hyperfekt made this script here which we can probably adapt to be a more robust solution.

Frankly if it was my choice I would have removed bcachefs support from NixOS and instead maintained a wiki page with exact instructions on how to set it up. Having it available normally implies a level of stability that simply isn't provided by upstream; and I don't see how someone would run bcachefs without closely watching what happens upstream anyway.

Kent Overstreet (the developer of bcachefs) has been talking about getting it more stable and implying upstreaming into the maintained Linux kernel isn't too far away, and if that's true this problem may solve itself without much intervention from our side and not worth a lot disruption right now.
The question of course is how close those things actually are, which is very hard to estimate.

I think you're right, I don't think it should have been merged in to nixpkgs, especially with how much of a "hack" it is at the moment, maybe NUR would be a better place.
I wouldn't mind maintaining something on NUR but I can't even get my kernel to compile with the diff applied. And have zero clue how to inspect the failed build to figure what the problem is.
  CC [M]  drivers/gpu/drm/nouveau/nouveau_led.o
  LD [M]  drivers/gpu/drm/nouveau/nouveau.o
  AR      drivers/gpu/built-in.a
  AR      drivers/built-in.a
make: *** [../Makefile:185: __sub-make] Error 2
note: keeping build directory '/tmp/nix-build-linux-5.10.30.drv-0'
builder for '/nix/store/73rbks2yv3sxm4zmcpxkx9daa1bg6hkp-linux-5.10.30.drv' failed with exit code 2

@YellowOnion thanks for bringing this topic up!

Adapt to a patch on NixOS kernel, this way we can get the latest version of the kernel and not be stuck on x.x.0

this would break it in unexpected ways and add maintenance burden since we would move away from upstream. it would break the main use case of the package (testing what upstream provides and report issues)

Remove hard dependency on specific kernel package, so users can select what kernel package they want to use.

this does not work. you have to wait until bcachefs support is upstreamed to mainline linux

package both compat and master branches of bcachefs-tools.

i have not looked into it lately, but sounds resonable. we also need documentation how it is supposed to be used

Use evilpiepirate git source because it's more up to date.

maybe. github provides more efficient download i think

Having it available normally implies a level of stability that simply isn't provided by upstream

the package is called linuxPackages_testing_bcachefs for a reason. of course we can't provide better software than upstream
i have planned to add documentation. that should make it more clear
it is just for testing it and reporting issues upstream until it is stable enough and get upstreamed

I don't think it should have been merged in to nixpkgs, especially with how much of a "hack" it is at the moment

i disagree. it is not a hack. the package provides integration of the bcachefs upstream kernel at the time of it's last update. it has tests and everything
since it's just for testing, it does not have a high priority and is only updated from time to time. feel free to help!
the package makes it easy to try it out for users. this is not the case with wiki instructions or NUR. i suggest contributing to nixpkgs instead!

If you end up choosing to remove the bcachefs kernel from nixpkgs, it could be nice to do so before 21.05 is released.

i hope you have reported them upstream, so they can get fixed.

The fixes are in master, and this is why I brought up this thread.
When you specify boot.supportedFilesystems = [ "bcachefs" ]; it pulls in Kent's 5.9.0, which hasn't had any commits in 5 months, it's on a kernel that's even older, this is what I mean by "hardcoded", I don't know if NixOS has a way for a package to "provide" a specific feature, while another one demand it, so we can get more flexibility.
It should be easy enough to provide patches for 5.9.x, 5.10.x, and 5.11.x, with the NixOS kernel on the various branches Kent has provided.
I'm new to NixOS so just struggling to understand the plumbing of a lot of this stuff.
I did eventually get my patch to compile last night, the problem was the error happened far earlier (like a good 5mins earlier) I don't know why the kernel keeps building uselessly.
@symphorien I think this would break too much stuff for people who have come to depend on this package.
I guess I should go investigate how to contribute to nixpkgs.

While generally the bcachefs patchset applies just fine throughout all patches of a minor (in semantic version speak), it is specific to that minor and usually needs work to be adapted to it. Kent only maintains a singular minor on top of which bcachefs is developed. All older minors would either have a severely outdated version of bcachefs or would need rebasing work.The latter is true for minors newer than the current as well.
The current state of the bcachefs upstream however is that it is branched off from when a new minor is released, meaning it does not receive changes from newer patch versions. What I have done for myself is to apply the bcachefs patchset on top of the most recent patch version of that minor in nixpkgs and it has never not applied cleanly, or produced issues that did not also occur on the upstream version.
In conclusion I recommend that if nixpkgs continues to provide a bcachefs-enabled kernel it adopt the same method.
My problem with the bcachefs enabled kernel in nixpkgs is twofold:

It gets pulled in automatically by specifying bcachefs as a filesystem, which does not allow for the user to be educated on the risk they are taking in using that filesystem.
A newer version of the bcachefs patchset often renders filesystems unmountable, leading to serious disruption. So any updates to the version of the bcachefs patchset should be undertaken by the user, and not happen automatically simply by changing their revision of nixpkgs.

When you specify boot.supportedFilesystems = [ "bcachefs" ]; it pulls in Kent's 5.9.0, which hasn't had any commits in 5 months, it's on a kernel that's even older, this is what I mean by "hardcoded",

@YellowOnion i see. unlike other distros, nixos provides tools to customize packages. so you can update the package in your config without changing nixpkgs. i'm not sure what the best method is right now, i used nixpkgs.config.packageOverrides last time. maybe the documentation helps you or have a look in the forum, chat or wiki https://nixos.wiki/wiki/Overlays#Examples_of_overlays

It should be easy enough to provide patches for 5.9.x, 5.10.x, and 5.11.x, with the NixOS kernel on the various branches Kent has provided.

i don't think we have the resources to maintain 3 kernels when the one we have is that outdated ;)

I'm new to NixOS so just struggling to understand the plumbing of a lot of this stuff.

it's actually quiet easy. i have never compiled a custom kernel and was able to package the initial package. have a look at the commits https://github.com/NixOS/nixpkgs/commits/a8758fdce5c95a1d20f599b4a6b11f862db1252f/pkgs/os-specific/linux/kernel/linux-testing-bcachefs.nix

I guess I should go investigate how to contribute to nixpkgs.

yes. i think most users will come to the point where they become contributors, even it the contributions are small, like updating a package. i think such contributions are easier here than in other distros. just create a pull request on github
it is extremely empowering when you can just fix the issue that bothers you yourself or just update the package that wasn't touched by the maintainers for 6 months ;) everyone is allowed to improve packages, not only the official maintainers
and when you are a user of bcachefs and regularly update it, feel free to add yourself as a maintainer and help keep it up to date :)

i don't think we have the resources to maintain 3 kernels when the one we have is that outdated ;)

the branches for 5.9, and 5.10 are stagnant, once it works it works. We don't have to "maintain" anything, just master.
we just need automation, leavaging: https://github.com/NixOS/nixpkgs/blob/master/maintainers/scripts/update.nix
I already have patches for ever signification push Kent made here: https://github.com/YellowOnion/bcachefs-patches

Why would anyone want to use a bcachefs kernel that is not the latest available?

For now I've updated the patchset and made the package follow revisions of the respective minor in #123765.
Frankly I expect bcachefs to be close enough to stabilization now that most people who will be bitten by it probably already have been.

AFAIK, the parts of bcachefs that need to be mainlined into the kernel are well on their way.
Doing some cleanup on bcachefs and related issues, do we still need this open?


about how to get shape and appearance code

Hi , thanks for your great work. I want to know about how to get shape and appearance codes, are these encoded from an image when training, and generate randomly when testing?

Hi @gulinsong , they are randomly sampled when testing; We have not added the option to obtain a shape / appearance code for a given input image. This could be done with test-time optimization of the latent codes, or adding an encoder during training.


Use safer tar extract

Modulus Pull Request
Description
Closes #127
Specifying the members converts the severity to low as per bandit docs
Checks the members before extracting such that the members:

Don't try to traverse up the directory structure.
Don't specify an absolute path to target arbitrary locations on the file system.
Will be extracted within the intended target directory, even after resolving symbolic links and redundant path elements.

Checklist

[x] I am familiar with the Contributing Guidelines.
[x] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.
[x] The CHANGELOG.md is up to date with these changes.
[x] An issue is linked to this pull request.

Dependencies

/blossom-ci

Neat! didn't realize this would be a security issue.

Neat! didn't realize this would be a security issue.

Thanks @loliverhennigh . Yes, this was flagged by our CD process. Hopefully, after this issue is fixed: https://github.com/NVIDIA/modulus/issues/120, this can be flagged during the CI itself.


Image Gallery - Keyboard Control

Keyboard Only User
To Do:
Add functionality such that images in an image gallery can be navigated using keyboard controls.
Notes

Medium Priority
Image Gallery should be navigable by users using only keyboard inputs.
Included in sprint due to coupling with other keyboard-only User Story (see Related Links, below).

Size

Small

Related Links

Issue: Keyboard Only User
User Story: Image Gallery, Keyboard

Task Branch Name
feature/image-gallery
Work performed
Added a basic template for an image gallery in Bootstrap. Has a semi-functional modal image display.
Work remaining
Need to implement a Lightbox modal that allows scrolling through images in the gallery (in lieu of the modal currently in place).
Still does not actually allow keyboard scrolling through the images.


The 'await' operator can only be used when its containing method is marked with the 'async' modifier

I was using the latest Toasts.Forms.Plugin in a shared asset project, trying the sample code you provided.
var notificator = DependencyService.Get<IToastNotificator>();
bool tapped = await notificator.Notify(ToastNotificationType.Error,      "Error", "Something went wrong", TimeSpan.FromSeconds(2));
But this gives me the following error,
Error CS4033: The 'await' operator can only be used when its containing method is marked with the 'async' modifier (CS4033) (iRemote.iOS)
However, when I change the code to:
var notificator = DependencyService.Get<IToastNotificator>();
var tapped = notificator.Notify(ToastNotificationType.Error,      "Error", "Something went wrong", TimeSpan.FromSeconds(2));
It works fine, although the dismiss on click doesn't work on Android (works fine in iOS).
Could you please direct me as to what I've done wrong?

It's an async implementation so if you don't call wait it will not wait till the function finished, it will run asynchron.
I just had a quick look into the code but I didn't actually tried to reproduce your problem. I think there is no implementation for an automatic dismiss on click in Android. It just returns true if it's clicked, on iOS the message gets also removed. Anyway, it shouldnt be that hard to implement. You can probably forward the click event to the manager in the Android implementation and then send the REMOVE_CROUTON message with the sender crouton.

Thanks for replying @TorbenK . The await problem was because I hadn't defined my showToast() method as async.
For the dismiss onClick in android, I actually downloaded your source code and modified the Crouton.cs file to hide it in the onClick(). I'm a newbie to xamarin and c#, could you elaborate/redirect me to a resource to do this in the way you have suggested?

I added it, I just did minor testing. You can get it here:
https://github.com/TorbenK/Toasts.Forms.Plugin

Thanks a lot @TorbenK !

@TorbenK could you release your package on Nuget?

it is on nuget, but without the android dismiss functionality.
i can update the package next days.
https://www.nuget.org/packages/Toasts.Forms.Plugin.Xamarin.1.3.1/

I updated my package on the nuget to <IP_ADDRESS> with dismiss functionality, @TorbenK thanks for handling issues while I was offline :+1:


how to keep it 24/7

nice work but how do i keep it 24/7. thanks

nice work but how do i keep it 24/7. thanks

I guess in a host or not

Can it run in a host 24/7

use https;//bot-hosting.net


bug(mat-expansion-panel): Expansion Toggle missing in versions 10, 11, 12

Reproduction
As visible in the official documentation
https://material.angular.io/components/expansion/overview
The mat expansion toggles for the first expansion are missing since version 10 (version 9 and below looks ok)
This particularly a concern when there is only 1 expansion panel used it cant be seen by end user that panel expands on click
Expected Behavior
Expansion panel toggle on all panels unless hidden with hideToggle
Actual Behavior
Expansion panel toggle when hideToggle not set
Environment

Angular: 10.x, 11.x, 12.x
CDK/Material:
Browser(s): Firefox, Edge
Operating System (e.g. Windows, macOS, Ubuntu): Windows, Linux

ignore, missed panel setup


ISO/IEC 7812 card number support

Add ISO/IEC 7812 card number support.

It would be better to include this feature in another library.
In a pure SEPA processing platform, we does not need to deal with Credit Card number.

@jycr, thank you for your feedback.
JBanking is meant to address general banking concerns, it is not only a library for SEPA. So I still think this feature can have its place in this library.


There is no route invalidation occurring.

Hello Dears,
Could anyone help me? There is no route invalidation occurring.
The DB_UPDATE_FREQUENCY is set to 5 seconds (default), I add a route on node A, the route works on node A, but on node B e C the route does not work.
I do GET curl http://kong:8001/routes on node B and the route added route_new appears, but when I do curl -k https://kong:8443/route_new the route does not work:
{"message": "no route matched with those values"}.
I enabled the debug and the event below does not occur:
[lua] cluster_events.lua:248: [cluster_events] new event (channel: 'invalidations')
After manual kong reload on node B and C, the route works.
My scenery:
3 Kong CE v1.2.1 running on Docker connected to 1 Postgres 9.6
1 Konga v0.14.3 running on Docker.
Host 1: 1 Kong A, 1 Konga e 1 Postgres.
Host 2: 1 Kong B
Host 3: 1 Kong C
Communication between the Kongs and Postgres is working.

I tried to reproduce, but I could not reproduce this, at least easily, on my own. Can you provide a full example? E.g. commands you use to start the Kong with pointers to any non-default configuration commands, and the commands you use to add routes. Does it happen always, or just sometimes?

Hello Aapo Talvensaari,
Sorry for the delay.
It happens always. I use Konga to create routes.
The configuration of the containers and the tcpdump of POST and GET are attached.
Details.txt
Regards,
Reginaldo

Adicionalmente, quando eu crio rota, a tabela cluster_events não é atualizada, em um outro ambiente que está funcionando, atabela é atualizada:
kong=# select * from cluster_events ;
id | node_id | at | nbf | expire_at | channel | data
----+---------+----+-----+-----------+---------+------
(0 rows)

Hello Aapo Talvensaari,
Here's the solution: do not change the timezone of the containers.
I have identified error in the postgres container log:
docker logs compose_kong-database_1 -t
2019-07-04T21:45:43.449633091Z ERROR:  relation "cluster_events" does not exist at character 116
2019-07-04T21:45:43.449682217Z STATEMENT:  SELECT id, node_id, channel, data,
2019-07-04T21:45:43.449701337Z         extract(epoch from at) as at,
2019-07-04T21:45:43.449728076Z         extract(epoch from nbf) as nbf
2019-07-04T21:45:43.449755533Z  FROM cluster_events
2019-07-04T21:45:43.449774333Z  WHERE channel IN ('invalidations','balancer:targets','balancer:post_health','balancer:upstreams','proxy-cache:purge')
2019-07-04T21:45:43.449791153Z    AND at >  to_timestamp(1562276573.372000)
2019-07-04T21:45:43.449807591Z    AND at <= to_timestamp(1562276743.434000)
Imagine what it might be, so I remembered that I had added timezone configuration in the containers to solve the wrong time problem in access.log.
After removing the lines below the docker-compose.yml and re-creating the containers the problem stopped:
volumes:

/etc/localtime:/etc/localtime:ro

You have now updated the cluster_events table:
kong=# select * from cluster_events ;
id                  |               node_id                |             at             | nbf |         expire_at          |    channel
|                      data
--------------------------------------+--------------------------------------+----------------------------+-----+----------------------------+------------
---+-------------------------------------------------
4d6a894e-66fc-4401-aedd-055076b51cff | b423c27e-d087-41ed-b903-7d1a64bdc287 | 2019-07-04 23:53:15.319+00 |     | 2019-07-05 00:53:15.319+00 | invalidatio
ns | routes:db3a724d-6144-4b07-89b2-1d7869d8ec3f::::
68a1508e-60af-4919-a3f3-30a564b54919 | b423c27e-d087-41ed-b903-7d1a64bdc287 | 2019-07-04 23:53:15.323+00 |     | 2019-07-05 00:53:15.323+00 | invalidatio
ns | router:version
(2 rows)
Now the routes are being invalidated / updated in the Kong B and C.
Thanks for the help dear.

I am glad that you got your problem solved. Sorry for not having more time to deal with this. I have done recently some work to make cluster events more robust and to deal with possible clock skew. So all nodes will now share the db clock on cluster events.


Fixed issues to mfast works with Elasticia exchange issue#108 &110

This change is not baed on the standard encoding rule and would break the interoperability with other exchanges. If you really need this, I would suggest to use macro to enable it when needed.


Support absolute paths in the classPaths. Support classPath string arrays.

This pull request makes two changes to solve an issue I was having while including node-java-caller in one of my libraries.
First, a new parameter is added to JAVA_CALLER_OPTIONS called useAbsoluteClassPaths.  To preserve the existing behaviour, it defaults to false. When true, the paths in the classPath are taken as they are without prepending the rootPath to the beginning.
Second, multiple classPaths can be optionally specified in a string array, rather than a string with the : separator.  This helps when multiple absolute paths are used in a Windows environment.  Without it, the : after the drive letter is treated as a separate path.

I should note, I reduced the memory requirement for one of the other tests.  I had a lot of things running at the time, 😄

A backgrounder on my library, and why I made these changes.
I'm working on a package called node-docushare.  Basically it's an unofficial wrapper around a Java API for our DocuShare document management system.  I bundle a few helper libraries with the package, but the licensing around the actual DocuShare API package is unclear, and varies with each DocuShare version.
Since that one JAR file is not included, I give my package users the ability to set the path to their JAR file, which could be anywhere, not likely within a specific folder in my package.  I then append that path to the classPath.
Absolute paths work great for this purpose, hence the useAbsoluteClassPaths change.
The option of using a string array for the classPath, that's to support absolute paths in a Windows environment.
Thanks for the consideration.

Sorry for the fixes not related to your PR, there is some time a new version of java-caller has not been published and Mega-Linter had a lot of updates since !
You can copy-paste the .cspell.json provided in artifacts and push again ^^

End of the workday here. Will have to look at where and how to copy and paste .cspell.json tomorrow morning, Eastern Standard Time.

Click on Details near Mega Linter

Then on artifacts -> Download Mega-Linter reports

Then you'll see .cspell.json file, and you can replace the existing .cspell.json file in your local repo, then commit -> push :)

@dangowans released in v2.3.0 :)

Nice!  My day has been made.  Thank you.


Fix table scrolling, add selectrow event.

This fixes a bug and adds a small feature, both related to ListTables:

The auto scrolling in ListTables does not take into account the fixed header row, thus alwas being one off.
Emit an event when a table row is selected (existing events are not triggered consistently)

@Frando PR has been merged here https://github.com/embark-framework/neo-blessed/pull/4


Capture stderr of tests

I'm currently having a problem with testing libraries that use log with either env_logger or flexi_logger in their tests. In the "real world", they should log to stderr, but Cargo doesn't capture stderr from tests, so I have to jump through hoops to get these libraries to log to stdout instead.
Would it be possible to add an option to cargo test that allows it to capture stderr from tests?

Thanks for the report! This is more of a rust-lang/rust issue (where libtest lives) rather than a Cargo issue though, could you file an issue on that repository?

OK, will do.


Resolve Several Reported False Positive

This PR resolves many of the reported false positives.

Can one of the admins verify this patch?


[Connectores] Koberose

Describe the issue:
Provided kerberos client project should be update.

Document,  has mention a private repo.

Project is outdated and not support with latest .net frameworks.

Need to check the possibility to provide this client as an extension.

Related issues:
https://github.com/wso2/product-is/issues/18462

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Ability to toggle file column labels on results view

https://github.com/codefori/vscode-db2i/blob/89482f86c5616ea1470d4d2c65ba6626277d01eb/src/views/results/resultSetPanelProvider.ts#L49


Add the BlueBench benchmark

BlueBench is an open-source benchmark developed by domain experts to represent required needs of Enterprise users.
It is constructed using state-of-the-art benchmarking methodologies to ensure validity, robustness, and efficiency by utilizing unitxt's abilities for dynamic and flexible text processing.
As a dynamic and evolving benchmark, BlueBench currently encompasses diverse domains such as legal, finance, customer support, and news. It also evaluates a range of capabilities, including RAG, pro-social behavior, summarization, and chatbot performance, with additional tasks and domains to be integrated over time.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 3 committers have signed the CLA.:x: Shachar Don-Yehiya Shachar.Don-Yehiya@ibm.com:x: perlitz:x: shachardonShachar Don-Yehiya<EMAIL_ADDRESS>seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @haileyschoelkopf @baberabb @lintangsutawika , can you take a look please? thanks!

Hi @shachardon. Just need you to resolve the merge conflicts.


Revert to ADT Atom type

A while back, we decided that an existential data type for "Atom" would be beneficial because it would allow users to compile in their own atom type, perhaps in a separate module. This strategy never fulfilled its promise because all serialization/deserialization code necessarily hardcodes detection for each type. In these places we can't even use pattern matching, so the user would not be alerted to missing implementations.
Now with the JSON interface adding another serialization location, the existential datatype is even more annoying.
We have since allowed for datatypes which can be created at runtime via the ConstructedAtom datatype, so the need for new datatypes at compile-time is reduced.
I propose reverting the Atom datatype to the original primitives (Int, Text, Day, etc.). Users will still be able to add their own primitives, if they like, but the compiler will give them sensible warnings if they miss a coding spot (through incomplete pattern matches).
The existential datatype experiment was interesting, but it seems it that its costs now outweigh its benefits.

This is now available on the master branch. The change does indeed simplify a lot of logic and increases confidence that all types are properly handled.


Prefer rest path over Function/Source/Sink Config values

Motivation
When users create/update sink/source/function, they call the rest endpoint that has a tenant/namespace/name components and pass it a Function/Source/SinkConfig. These configs also have tenant/namespace/name parameter that might be different from the rest path.
This pr gives preference to the rest path by overwriting the config fields.
Modifications
Describe the modifications you've done.
Result
After your change, what will change.

rerun cpp tests

rerun integration tests


leetcode solution for problem 105 : Construct Binary Tree from Preorder and Inorder Traversal

Related Issue
[Cite any related issue(s) this pull request addresses. If none, simply state "None”]
#867
Description
I have contributed solutions for the LeetCode problem "Construct Binary Tree from Preorder and Inorder Traversal" to the open source project. This problem involves constructing a binary tree from given preorder and inorder traversal arrays. The solutions have been implemented in four programming languages: C++, Python, Java, and JavaScript.
Type of PR

[ ] Bug fix
[ ] Feature enhancement
[X ] Documentation update
[ ] Other (specify): _______________

Checklist

[X ] I have performed a self-review of my code.
[ X] I have read and followed the Contribution Guidelines.
[ X] I have tested the changes thoroughly before submitting this pull request.
[ X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[ X] I have commented my code, particularly in hard-to-understand areas.

Additional Context
[Include any additional information or context that might be helpful for reviewers.]
Resources for Guidance
Here are some resources that may be helpful as you contribute to CodeHarborHub:

Code Harbor Hub Community Features
Docusaurus Documentation
React.js Documentation
Markdown Guide
MDX Documentation
Mermaid Documentation

Resources for DSA Features on Our Site:

Tabs: Switch Languages
Line Numbering
Math Equations

@Ajay-Dhangar please check the pr


Thaumonomicon can't progress past first steps

I have done the vis crystals quest, slept, got the book, crafted salis mundus and created a thaumonomicon, but the game wont let me right click on a crafting table with salis to create an arcane crafting bench. I don't know if i missed something, but I can't seem to find anything i did wrong
I have attached a copy of what my thaumonomicon looks like if it would help

something looks out of place did you open that first icon (book) at the top? that should get everything rolling


How to add groups / projects !?

Downloaded the newest project viewer files - and now the "right click" doesn't work anymore. How can I edit / add / delete groups and / or projects!?

ok found.. Menu Packages > Project-Viewer > Open editor...
but why no right click anymore? :(

will make it available @OhDerAlex! there is a shortkey also for now. sorry

Cool, thx.
And I like the changes - nice work!

been asked to implement an alike open in finder / explorer, what do you think? I still need to investigate this as well

@OhDerAlex next release will have feature back plus one more

@OhDerAlex check it on 1.0.7 release please

Hey. Updated now. Thanks a lot! :)


Support matching suggestions on spacebar

Use-case/Problem
https://github.com/draft-js-plugins/draft-js-plugins/issues/1048
Implementation
Pass a matching function(with suggestions and lastSearchValue as arguments) from config as a prop to MentionSuggestions component. Use this function after spacebar is pressed to find a matching solution if there is one.

nevermind, you can't: https://github.com/npm/npm/issues/2974
Although there is a mention of gitpkg in a similar issue for yarn: https://github.com/yarnpkg/yarn/issues/4725 see commit https://github.com/yarnpkg/yarn/issues/4725#issuecomment-496683978 Might be good to investigate, but we might need yet-another repo from @afahy if we want to push the built package into it's own lib :)

fun fact, that issue for npm is 7 years old, so I don't think anything will change soon.

@ndrjkvc Anyhow, if we just want to move on, you would have to install the whole draft-js-plugins and if you want to use your built library should be able to just include:
import createMentionPlugin from "draft-js-plugins/draft-js-mention-plugin"

@ndrjkvc Anyhow, if we just want to move on, you would have to install the whole draft-js-plugins and if you want to use your built library should be able to just include:
import createMentionPlugin from "draft-js-plugins/draft-js-mention-plugin"

That does not work, the plugins subfolders are empty after draft-js-plugins install (not really empty, they contain changelog, license, and readme)

But there may be something we can write into package.json to let npm know it should download that folder fully. @szist

But there may be something we can write into package.json to let npm know it should download that folder fully. @szist
you should add/create a .npmignore file, so that it doesn't use the .gitignore to remove all deps. See https://github.com/stocktwits/web-shared/blob/develop/.npmignore vs https://github.com/stocktwits/web-shared/blob/develop/.gitignore

do we still need the lib/*.js in git and the files in package.json ?

No


Update doc to clarify Linked Events only support cloud-mode destinations

Proposed changes
Just checking with engineers, we currently support Linked Event for actions-based destinations only in cloud-mode. Web destinations like Wisepops and GA4 Web are not supported.
Slack - https://twilio.slack.com/archives/C05BMCB3P2P/p1707979990521649
Merge timing
ASAP once approved
Related issues (optional)
https://segment.atlassian.net/browse/KCS-1097

Hi @joeynmq - The Linked Events PM also asked me to add similar verbiage to the docs. Let me know when this is ready to review!

Hey @joeynmq - closing this out to open #6116. I was pinged by the PM to also expand this to the Linked Audience docs, and want to get these updates out today.

Thanks @rchinn-segment!


pagerTabItem got hidden after navigation link

after navigate to other screen by using NavigationLink, the previous screen's pagerTabItem got all dissapeared.

Hi all, this issue was fixed in #56. If you still experience the same issue then please reopen with additional details or open a new issue!


chatbot not working

I just get an empty response

Here's the console output:

Same here. Even installing and running the repo fails, pretty silently.

the repository not having been touched since the first commit 3 months ago does not look good. @zahidkhawaja

Its because, its a boiler plate replicating the chatGPT UI in nextjs, the backend has to be implemented and the request goes to the backend server URL that handles the processing done in history, question. Request captures question, history in UI, we have to implement backend or use langchain chat FAST API endpoints to have it connected!

I'm trying to integrate this with Berri.ai's API endpoint but can't figure out how to submit the necessary parameters.  They've forked your repo but no where does it describe how to add the necessary parameters.  https://docs.berri.ai/api-reference/endpoint/query


Error on insertData Function

Error

The insertData function has a bug during running the program.😉
when I am pressing the add button without filling the fields of the form it is not showing any message that I should know what was happened 🥺🙄.

Ok I will solved the error.😏


Test-AdfLinkedService return an object

When you specify an array of linkedServices it would be nice to receive an object (eg hash-table) over the result (passed/failed/total).
It could be implemented as follows:
`function Test-AdfLinkedService {
:
:
Write-Host "Test connection result:"
Write-Host "Passed: $ok"
Write-Host "Failed: $($all-$ok)"
Write-Host "Total : $all"
$result = [ordered]@{Passed = $ok; Failed = ($all-$ok); Total = $all}
return $result
}`

Thanks. That good idea. In addition to that, would be good to:

return list (hash table) of tested objects with their result
add unit test of this method

I just finished #151 and used updated Test-AdfLinkedService CmdLet in the automated test process and I have a couple of questions about this.

Should Cmdlet print anything by default if not using -Debug and/or -Verbose flags?

I had to use 6> /dev/null to to ignore Write-Info level because I wanted to print single "." after every Test-AdfLinkedService invocation

I will create something (without unit tests) on top of #151 quickly because this is something I needed yesterday. PR coming later.

Yes, please. I believe every test should be logged separately as it does help with troubleshooting significantly.


code for yolo

I just want to Pass an Image and Get the Inference ...How Should I Do so...

This project does not really do that. I would suggest doing an Android tutorial (https://www.udacity.com/courses/android) and trying to understand what the code is doing.


TAS: Support Kubeflow.

What type of PR is this?
/kind feature

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

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

NONE

/test all

/test all

/cc @mimowo

/cc @mimowo

ack

/lgtm
/approve
In a follow up please mark all the integration tests (except for one kubeflow Job) as redundant.
/hold
in case @tenzen-y has some more comments

/hold cancel
To unblock other PRs pending for the release. @tenzen-y please feel free to still comment on the PR, and we will address any comments in follow up PRs.

/hold
as I just got a ping on slack, let's wait for the comments

/hold cancel
ok I got info @tenzen-y will review later today. In the meanwhile I think it is safe to merge to unblock https://github.com/kubernetes-sigs/kueue/pull/3449 and so we can start preparing the release notes


backup: create external storage per-backup reqeust.

fixed #11112

What problem does this PR solve?
Issue Number: fixed #11112
Problem Summary:
What is changed and how it works?
What's Changed:
Create external storage at handle_backup_task.
Check List
Tests

Manual test (add detailed scripts or steps below)
For a workload with 30K tables, backing up to S3 storage:

with this PR
master

Release note
None

/cc 3pointer kennytm

/run-test

For reference, the comment was about #7236, which has been taken out in #9415.
Unfortunately, YuJuncen has tested that applying this PR brings #7236 back. According to hyperium/hyper#2649, this is likely due to the use of block_on_external_io, which spawns and kills a fresh tokio runtime in every write call.
We think that refactoring ExternalStorage::write to actually return a future (i.e. making it async fn) is the easiest approach.
[2021/10/21 21:48:40.623 +08:00] [FATAL] [lib.rs:465] ["dispatch dropped without returning error"] [backtrace="   0: tikv_util::set_panic_hook::{{closure}}
   1: std::panicking::rust_panic_with_hook
             at /rustc/2faabf579323f5252329264cc53ba9ff803429a3/library/std/src/panicking.rs:626:17
   2: std::panicking::begin_panic::{{closure}}
   3: std::sys_common::backtrace::__rust_end_short_backtrace
   4: std::panicking::begin_panic
   5: <futures_util::future::try_future::MapOk<Fut,F> as core::future::future::Future>::poll
   6: hyper::client::client::Client<C,B>::send_request::{{closure}}
   7: <core::future::from_generator::GenFuture<T> as core::future::future::Future>::poll
   8: rusoto_core::request::http_client_dispatch::{{closure}}
   9: <core::future::from_generator::GenFuture<T> as core::future::future::Future>::poll
  10: <core::future::from_generator::GenFuture<T> as core::future::future::Future>::poll
  11: <core::future::from_generator::GenFuture<T> as core::future::future::Future>::poll
  12: <core::future::from_generator::GenFuture<T> as core::future::future::Future>::poll
  13: <core::future::from_generator::GenFuture<T> as core::future::future::Future>::poll
  14: <core::future::from_generator::GenFuture<T> as core::future::future::Future>::poll
  15: <aws::s3::S3Storage as cloud::blob::BlobStorage>::put
  16: backup::writer::Writer::save_and_build_file
  17: backup::writer::BackupWriter::save
  18: backup::endpoint::BackupRange::backup
  19: core::ops::function::FnOnce::call_once{{vtable.shim}}
  20: <yatp::task::callback::Runner as yatp::pool::runner::Runner>::handle
  21: std::sys_common::backtrace::__rust_begin_short_backtrace
  22: core::ops::function::FnOnce::call_once{{vtable.shim}}
  23: <alloc::boxed::Box<F,A> as core::ops::function::FnOnce<Args>>::call_once
             at /rustc/2faabf579323f5252329264cc53ba9ff803429a3/library/alloc/src/boxed.rs:1572:9
      <alloc::boxed::Box<F,A> as core::ops::function::FnOnce<Args>>::call_once
             at /rustc/2faabf579323f5252329264cc53ba9ff803429a3/library/alloc/src/boxed.rs:1572:9
      std::sys::unix::thread::Thread::new::thread_start
             at /rustc/2faabf579323f5252329264cc53ba9ff803429a3/library/std/src/sys/unix/thread.rs:91:17
  24: start_thread
  25: __clone
"] [location=/data4/juncen/.cargo/registry/src/github.com-1ecc6299db9ec823/hyper-0.14.11/src/client/conn.rs:316] [thread_name=bkwkr-1]

/lgtm cancel

/run-br-integration-tests

/run-integration-br-tests

/unhold the i/o part of external storage is asynchronous.

/hold cancel

/run-integration-br-tests

/test
https://ci.pingcap.net/blue/organizations/jenkins/tikv_ghpr_test/detail/tikv_ghpr_test/11050/pipeline = #9309?

test_resolve::test_joint_confchange 🤔
/test

test_resolve::test_joint_confchange => https://github.com/tikv/tikv/issues/11325
/test

/merge

/test test_node_multiple_rollback_merge

/test test_server_transfer_leader_safe

/test test_sending_fail_with_net_error

/test test_unsafe_recover_update_region + test_sending_fail_with_net_error

seriously the number of retries to get it work is really frustrating.


API should contain the latest version for every container.

We should provide in the API and the web the lastest version for a package as requested in the issue: https://github.com/BioContainers/specs/issues/74

Done


Fix bug in Safari when calculating text coords of hidden nodes

Hi @marijnh, while working on a component which uses CodeMirror I noticed some errors being raised in Safari v14.0.3.
It doesn't seem to be a fatal issue, but logs a TypeError: undefined is not an object (evaluating 'h.width').
Looking into it a bit more, it seems that the issue is related to how I render the parent DOM element housing the editor.
Depending on the user supplied prop, the CodeMirror instance can start hidden, and later toggled to be seen.
The component can be seen here

What happens is that range.getClientRects() (linked below) returns an empty array (see screenshot from safari stopped at the breakpoint. Apologies for the minified source code). Trying to access this array with an index then returns undefined.
https://github.com/codemirror/view/blob/1b92b55c62697087aed79101801ef4c41a6c3d28/src/inlineview.ts#L146-L148

A simple fix in this case was to check the existence of the rect variable before accessing the width property. However I couldn't quite decide what should happen regarding the fallback on line 148.
There's a bit of type coercion going on, and in this case the function might return undefined I think. Should I stub a Rect object with zero dimensions in this case?
Please let me know if you have any suggested changes, if this PR is out of scope, or if a rambled too much and failed to explain the issue.
Thanks for your time!

Thanks for the report. I think returning a zero rectangle is indeed the way to go here—I've implemented that in attached patch.

Would it be easy to publish a release for this fix? We're running into the same issue and would love to incorporate it.

Sure. I've tagged 0.17.8