Not respecting port with isPackagerRunning

Per https://github.com/facebook/react-native/blob/7e53ee1095ac956887dd6a20d19a403096472660/private-cli/src/util/isPackagerRunning.js#L23 it doesn't respect the port you can pass with the flags on CLI

For example, if I run my script with this:
react-native-webpack-server start -e index.android -P 9090 -p 9091 -w 9092

Then react-native run-android won't work since it is looking for port 8080.

This is a known issue with react-native: https://github.com/facebook/react-native/issues/1429
Unfortunately there are a bunch of places in RN where port 8081 is hard-coded; until that's resolved, you'll need to just use port 8081

See https://github.com/mjohnston/react-native-webpack-server/issues/65#issuecomment-151222398 I did a quick hack


NoSuchMethodError: java.nio.ByteBuffer.position(I)Ljava/nio/ByteBuffer;

Hello, I have an issue with Kodein DB 0.8.0-beta
The code is at https://github.com/LouisCAD/kotlin-libraries-playground
It works locallly on macOS but not on the CI so I'm a bit at a lost
The stacktrace is here https://scans.gradle.com/s/qztqjpf2oc23u/console-log?anchor=180
Or here https://github.com/LouisCAD/kotlin-libraries-playground/pull/103/checks?check_run_id=2658798807

This is due to a Java 1.8 / 1.9 compatibility issue: https://www.morling.dev/blog/bytebuffer-and-the-dreaded-nosuchmethoderror/
That's because your CI runs Java 8 while your MacOS probably runs Java 11.
I'll update Kodein-DB with a fix and publish ASAP.
If you don't want to wait, you can update your CI JDK to 11.

0.8.1-beta is out and fixes this issue.

That works, thanks!


edgeRenderer doesn't show the right color

Bugs

Bug repro on playground (line 51)
Expected result: get pur green edgeColor
Current result: kind of lighter color, but the pure green appears when activating the inspector!

Good try young padawan but this is not a bug:)
The edgeColor is a color4: https://www.babylonjs-playground.com/#K4S3GU#9


Use flexible syntax to specify browsers

Would you be interested in adding a more flexible syntax to specify the platforms/browsers/versions? For example:
{
  "platforms": [
    {
      "name": "internet explorer",
      "version": "oldest..latest"
    },
    {
      "name": "chrome",
      "version": ["oldest", "latest"]
    }
  ]
}

What I'm thinking is the same syntax used in zuul. The advantage is that you don't have to update the config if a new version of a browser is added in SauceLabs. The same applies for removed versions.
Converting this syntax to the required format is easy (the code can be extracted from zuul) and can be done in a separate npm module.
The only problem I see is that it isn't backward compatible.
Thoughts?

Sure, that would be cool. Out of curiosity, why do you say it wouldn't be backwards compatible? Presumably if it didn't find a .., it would just assume a single version, no?
Also, I assume this technique would need to query the Sauce Labs API to see what versions/platforms are available. Is that what zuul does?

Well currently you have an array of arrays. The new config will be an array of objects. Maybe we can check if the first element is an array and determine how to handle the config based on this info.
Yes zuul get all browsers from https://saucelabs.com/rest/v1/info/browsers/webdriver.

I published sauce-browsers. It works like this:
const browsers = [
  { name: 'firefox', version: 50, platform: 'Mac 10.9'},
  { name: 'chrome', version: ['oldest', 'latest'] },
  { name: 'safari', version: 'oldest..latest' }
];

sauceBrowsers(browsers).then((platforms) => {
  platforms = browsers.map((el) => [el.os, el.api_name, el.short_version]);
  // `platforms` can now be used in `easy-sauce`.
}).catch((err) => console.error(err.stack));

Support for https://github.com/ai/browserslist would be great. Tools such as https://github.com/babel/babel-preset-env use it.

@rtsao yeah I thought that but afaik there is no way to specify the OS in browserlist and it doesn't query Sauce Labs to actually know what browsers are available.


Add a spec for SizedQueue.new implicit integer conversion

I discovered that TruffleRuby doesn't accept anything but true Integer instances.
MRI call to_int on the argument, allowing Float and a bunch of other types.
cc @eregon

The CI fails, could you fix the code?

Arf, sorry... Should be good now.


English option

Hello can you add English option

This is a difficult job for me
you can use google translate to understand the project's wiki

hey, I was able to do that, I meant for the application,  all the titles are in Chinese any chance you can do spanish or english


feat(nav) change version navigation

To fix:
11:19:21 AM: Configuration file: jekyll.yml
11:19:21 AM:             Source: app
11:19:21 AM:        Destination: ./dist/
11:19:21 AM:  Incremental build: disabled. Enable with --incremental
11:19:21 AM:       Generating...
11:19:21 AM: jekyll 3.8.6 | Error:  Liquid syntax error (line 244): 'if' tag was never closed
11:19:21 AM: [19:19:21]   Liquid Exception: Liquid syntax error (line 244): 'if' tag was never closed in /_layouts/docs.html
11:19:21 AM: [19:19:21]
11:19:21 AM: 'jekyll' errored after 23 s
11:19:21 AM: [19:19:21]
11:19:21 AM: Error: Command failed: bundle exec jekyll build --config jekyll.yml --profile --destination ./dist/
11:19:21 AM:   Liquid Exception: Liquid syntax error (line 244): 'if' tag was never closed in /_layouts/docs.html
11:19:21 AM:     at ChildProcess.exithandler (child_process.js:281:12)
11:19:21 AM:     at emitTwo (events.js:126:13)
11:19:21 AM:     at ChildProcess.emit (events.js:214:7)
11:19:21 AM:     at maybeClose (internal/child_process.js:915:16)
11:19:21 AM:     at Process.ChildProcess._handle.onexit (internal/child_process.js:209:5)
11:19:21 AM: [19:19:21]


Order of default vs required

This is mostly a question and not really an issue - it's actually probably the correct behaviour but I would appreciate your thoughts. When a property is required AND has a default sub-schema, the validator first reports the error of missing required and then applies the defaults. A schema like
{
   "properties" : {
        "foo" : { "type" : "string", "default" : "me foo!" }
   },
   "required" : ["foo"]
}

with {} as input data produces a "missing required property foo"  error but the resulting object is (in my mind, correctly) set to { "foo" : "me foo!" }. I often resort running the validation twice - once to apply all defaults while ignoring errors and a second time to actually validated the input, completed with all default. It would be a useful feature to apply defaults first and then validate required constraints.

@atomictag
This behaviour is specified in the JSON Schema specification and the official test suite. The existence of default keyword must not change the result of validation, because support of default keyword is optional for validator implementation.

Thanks for the clarification!


No File Opened empty state hides summary tab

this empty state is wrong, it hides everything, when user is on home its probably wrong to hide everything

this bug will be fixed when navigation changes are done


when use cmd-L to select line, selection border will select one more line

I enabled selection border, if I select multiple lines by cursor, the border works fine; if I select line by using cmd-L, it will select one more line on the bottom.
maybe you can check the video: https://dl.dropboxusercontent.com/u/12302850/highlight-line-issue.mov

Here's of what visual of what @jesseminn is reffering to:

Details:
Selecting text with cursor properly displays the selection border as it should.
However, using the keyboard shortcut [cmd + l] to select lines adds the bottom selection line one line below of where it should be.


Documentation

The notebooks in the examples/ directory illustrate much of the functionality of datashader, but it would be very helpful to provide a more comprehensive set of documentation on the website at http://datashader.readthedocs.org/

I think this is broadly done (datashader.org), with just specific problems remaining.


close popup

Want to prevent the modal from closing during selecting text inside popup.
Don't close if it starts inside popup and then ends on backdrop.
https://watch.screencastify.com/v/WSgr37ivN9rgWkZLSCGU

@OlhaNikoliuk Is it fixed already?

fixed


a button icon and text not centered in 4.4.3

Hello, I recently updated metro to 4.4.3, and all "a" links have the icon and text stuck to the top of the a button (all browsers).
Is it a bug??

https://codepen.io/tommycorr/pen/zYKaMQR

    View

    View

Thank you Olton!

Thank you Olton!


Unable to compile core_affinity_rs on non-general-purpose operating systems

The current version supports only Linux, Windows and macOS.  PR  #15 should this issue.

This should be fixed with merging that PR.


[libs] WIP - Decrease Libs file size

To Do

[x] Test in real applications

So far I was able to decrease the size of movex by more then half, and movex-core-util by almost a half 🎉

This was tested in Chessroulette.
And also merged in #203


The project website's docs are out of date:

I noticed on the website, in the Events section:

afterChange | slick, currentSlide | Fires after slide change

But on the README:

afterChange | event, slick, currentSlide | After slide change callback

Happy to contribute a PR if I can be pointed in the right direction.

Additionally README.md for 1.8.0 has unmerged git merge chunks . the live one seems fine though https://github.com/kenwheeler/slick/blob/master/README.markdown


.NET 6: Cache in-memory

[Enter feedback here]

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

ID: 13f5f8f9-18dd-9c12-9ed1-fcc6b44ba7fd
Version Independent ID: 640b4f36-5886-7b69-1c95-cfc8574a02fd
Content: Cache in-memory in ASP.NET Core
Content Source: aspnetcore/performance/caching/memory.md
Product: aspnet-core
Technology: aspnetcore-performance
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@serpent5 I'll take this if you haven't started


Allow multiple toMatchSnapShots() per test case

In standard jest tests we can also do multiple snapshots.

yeah. it makes sense to have multiple toMatchSnapShots(). Indeed it makes sense to have multiple of everything since we should trust user that he knows how to chain steps together

Closed by [#54]


Added kjossul's mods

Hope you like these mods! Feedback appreciated.

Thank you for your contribution!


A abend of microsoft.pythontools.parsing.badsouurceexception

when I open a python projcet,it show;

what should I do??who can help me

Can you check the .py files in your project to see whether you have specified coding comments at the start? If these are incorrect then we may observe this exception (though we're supposed to handle them all).

Didn't hear any response, so closing this. Feel free to reopen if you have new information.


Fix typo for the challenge Backwards

This PR fixes the typo in the challenge backwards, where aws is missed in the commands given in hints section.
Fixes - #17

Sure I'll do that. Perhaps it was a typo 😅
On Tue, 7 May, 2024, 14:52 David, @.***> wrote:

@.**** requested changes on this pull request.
Please remove this one extra 'aws'
In aws/challenges/Backwards/challenge.yml
https://github.com/BishopFox/cloudfoxable/pull/18#discussion_r1592103867
:

@@ -35,7 +35,7 @@ hints:

|-
If your stuck, try running the permissions command in cloudfox and grepping for the word secret:

cloudfox aws -p cloudfoxable permissions -v2 | grep -i secret

cloudfox aws -p cloudfoxable aws permissions -v2 | grep -i secret

Thanks for contributing! Did you mean to insert this 'aws' here? I think
this one is not necessary.
—
Reply to this email directly, view it on GitHub
https://github.com/BishopFox/cloudfoxable/pull/18#pullrequestreview-2042567024,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEIITPXYFXMDNCXIR7QM7CTZBCMMBAVCNFSM6AAAAABGK2NYQCVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDANBSGU3DOMBSGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Fixed.


Fixed #11

Adding MajorMinorPatch filtering to UseLatestSnapshots goal

@olamy Could you please review this?

does it fix #11?


Upgrade to msbuild

Addresses #57
cc @pranavkm

Hi @jbagga, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;


Multi search/POST Request

Hello,
Is there any possible way you could implement or extend current Search functionality to include multi search or at least so it can use POST instead of GET for specific case? My problem is because sometimes my query will throw an exception "Query string exceeds max allowed length of 4000" which I guess it's limitation of using GET for typesense search.
I really like this lib, and thank you for making it! I would really like if it's possible for it to include something like this that could solve the issue above.
Thanks!

Hi @questionablequestion,
I've now added multi-search #89 to the client and released a new version with the tag 4.2.0. The docs has been updated with examples of how to use it. :)
// Rune

Hey @runeanielsen, I just had a quick try but all the fields in hits have null value. Any idea on why is that? I have tried it via Insomnia (it's like Postman), and multi search is returning okay results

@questionablequestion actually I've an idea why it is happening, I'll make a new release, in a few mins, and you can try again.

@questionablequestion I've made a new version 4.2.1, it should be out on NuGet in a few mins, please try again and report back.

@runeanielsen You are awesome! Thank you so much for super quick update! Can I somehow buy you a coffee?

No problem, the appreciation is enough, have a nice day. :)


Updating footer section

The styling of the footer section can be still improved to make it look more attractive by increasing the space between horizontal columns and changing the styling of icons and links over there. Please let me if I can work on it.

Yes sure, go ahead! @aysal04

can i work on this issue @akshathar-navada already it's 3 days from issue allocation

I have almost done it. Will send the pr by tonight

can i work on this issue @akshathar-navada already it's 3 days from issue allocation

Yes @SasiVardhan247 , u can also work on it!!


Sybase provider

Before you tell me to look to SAP for this, consider the following:

They don't want to and they don't intend to.
If they did they would screw it up as much as they did with the full framework.
The benefit to Microsoft of developing a Sybase provider is that it serves as a bridge and an escape hatch for people who prefer MSSQL
TDS is still supported and the SQL syntax is very similar to MSSQL so it could probably use the MSSQL driver as a starting point
I don't have the expertise to contribute this myself

Is there a Sybase .NET Core ADO.NET provider? If not, it will be a huge task.

Well, kind of. They ship with SQL Anywhere, and also as NuGet packages, but changes with VS2017 make them all but useless because they don't show up in the provider lists in Visual Studio. If you manually write all the code you can use them and even create an EF model that uses them (without designer support).
https://www.nuget.org/packages/Sap.Data.SQLAnywhere/

Maybe this would provide a strong basis https://github.com/senseibaka/AdoNetCore.AseClient

@PeterWone We would be very happy to give guidance to an open-source community-driven effort to create a Sybase provider. However, we're not planning to create one ourselves. We haven't seen huge demand for it and in terms of prioritizing resources we feel they would be more valuable working on other areas.

I've been meaning to get back to you about this: apparently ODBC support has returned to dotnet Core. This surprised me as several sources led me to believe that OBDC would never be available on account of being specific to Windows. But apparently it's now a Unix thing too, so it's back.
I have verified that I can use this to access SQL Anywhere.
So rather than support for Sybase, an ODBC provider would be far more widely applicable. Time permitting I may even have a crack at writing an EF Core provider myself.
Thanks for at least considering it.

@PeterWone glad to hear. Yes, we enabled the ADO.NET provider for ODBC on .NET Core with the goal of unblocking connectivity to databases that don’t have their own .NET Core ADO.NET providers yet but have ODBC support which is very common. An EF Core provider for Sybase that uses ODBC for database connectivity sounds indeed like a feasible project.
Just to clarify, that would not be a general purpose ODBC provider for EF Core. For example, all the SQL generation would be Sybase-specific. And to your previous point, given the common lineage of SQL Server and Sybase databases, the SQL Server provider should be a good starting point for all the SQL generation and type mapping pieces.

I know that is closed but it is something that we need too. Anyone is willing to doing some effort with me on that? Which will be the effort? Where should we start?

@giorgiozoppi If you want to implement a .NET Framework only provider using the existing .NET Framework Sybase ADO.NET provider, I think you may find the series of commit to the develop branc in this provider useful (it is not fully completed  - Migrations not implemented) - this represents less than a weeks effort.
https://github.com/GibraltarSoftware/VistaDB.EFCore/commits/develop

Is it for .NET core 2.0?

@giorgiozoppi at the risk of undermining support for producing an EFcore solution, may I point out that you can use Dapper with the ODBC provider. The primary shortcoming of this approach lies with update and insert, for which you must handcraft update and insert SQL.

@PeterWone does it works with .NET Core? I will go for it.

@giorgiozoppi I have been using Dapper/ODBC to access SqlAnywhere 17 databases for the last year. It's fast and reliable but a lot more work than EF whenever I have to insert or update. It is also possible to manage transactions.

we moved to dapper but we still use .net 4.6 do you belive that it is possible to use with .net core as well? the db is on windows, do you mean that we can use the api on linux?

Well... the documentation says there are Sybase SqlAnywhere drivers for Unix. If you can obtain them and if they work, then yes you can use Dapper in dotnet Core to access SqlAnywhere from Linux.
I have not actually done this.
Good luck getting SAP to help you with ANYTHING.


Check definition type before adding statement

Fixes #504.

Looks like b6c7c36 has this already, closing.


Improved IPv6 filtering support

This commit uses crate ip_network to determine whether an IPv6 address is
link-local or unique local.
Note that this extra dependencies can be removed once rust-lang/rust#27709 is
stablized.
Also note that this commit doesn't added any test yet.

I'm really not good at writing test. They could be included in another PR.

This PR is incredibly well done. I love that you added such detailed comments and refactored the code like this to make the intent clearer. It's probably pretty easy to write tests now. 😄
#[test]
fn test_uri_is_loopback() {
  let uri = Uri::try_from("::1").unwrap();
  assert!(uri.is_loopback());
}

Adding a few of these would already be a great start.
Testing Filter::is_ip_excluded is a little more tricky.
We could add a Default impl and a couple of custom filters, which are only activated for tests:
#[cfg(test)]
impl Filter {
    fn exclude_loopback_ips() -> Self {
        let filter = Filter::default();
        filter.exclude_loopback_ips = true;
        filter
    }
}

// then in the tests...
#[test]
fn test_uri_is_loopback() {
    let uri = Uri::try_from("::1").unwrap();
    let filter = Filter::exclude_loopback_ips();
    assert!(filter.is_excluded(uri));
}

(this won't compile, but you get the idea. 😉)
Let me know if you want to add the tests in this PR. I can also help out if you like.

This should be good to merge now.


Optimize permissions for ordinary users to view cost analysis data

Version: main
UI: dev-4750b53
Commit ID: caefc39e2dde0381a705b83d4c0d0adc2c25bf5f
Env: MacOS 13.1, Apple M2
Test Steps:
An ordinary user with access to only a few projects. But it can see the cost data of all projects which is a bit weird.
Perhaps we can disable access to view cost analysis for ordinary users. And open access to them in a future feature through a custom role policy.

No more valid in the latest version


Message: Fix Bubble border-radius

Message: Fix Bubble border-radius

This update fixes the border-radius rendering for Chat Bubbles, when
used within a Scope.
The issue was the & was inheriting the full scope style path.

Coverage remained the same at 100.0% when pulling e0397efaf34ea2f68de88cea0053e6a803b3e901 on message-chat-border-radius into 308dd68186d0949430a97f0f1f3d9b149e04da80 on master.


huggingface_hub.utils._errors.LocalEntryNotFoundError:

Hi, I encountered this error when I tried to reproduce it, and I haven't found a way to solve this problem on the Internet. Can you provide me with some solutions? :）
As follows,
huggingface_hub.utils._errors.LocalEntryNotFoundError: Connection error, and we cannot find the requested files in the disk cache. Please try again or make sure your Internet connection is on.

Hi,
it looks like you do not have Internet connection to huggingface. Please check if you can successfully download and load the pre-trained CLIP weights as in this line
Best,

Hi,
appreciate so much for your quick reply,
pre-trained CLIP weights (convnext_large_d_320, laion2b_s29b_b131k_ft_soup) can not successfully download from that link, but another version (ViT-B/16, openai) can be downloaded normally.

Hi,
it looks like you do not have Internet connection to huggingface. Please check if you can successfully download and load the pre-trained CLIP weights as in this line
Best,

Hi,
appreciate so much for your quick reply,
pre-trained CLIP weights (convnext_large_d_320, laion2b_s29b_b131k_ft_soup) can not successfully download from that link, but another version (ViT-B/16, openai) can be downloaded normally.

Hi,
it looks like you do not have Internet connection to huggingface. Please check if you can successfully download and load the pre-trained CLIP weights as in this line
Best,

Hi,
appreciate so much for your quick reply,
pre-trained CLIP weights (convnext_large_d_320, laion2b_s29b_b131k_ft_soup) can not successfully download from that link, but another version (ViT-B/16, openai) can be downloaded normally.

Hi,
I just tried that it still work in my end. The difference could be that ViT-B/16 OpenAI weight is hosted by OpenAI, while the ConvNeXt weight is hosted by huggingface. In this case, I would assume you may need some proxy to download the weight, or you could download the weight  manually and change the pretrained weight path to your local files.
Best,

Hi,
it looks like you do not have Internet connection to huggingface. Please check if you can successfully download and load the pre-trained CLIP weights as in this line
Best,

Hi,
appreciate so much for your quick reply,
pre-trained CLIP weights (convnext_large_d_320, laion2b_s29b_b131k_ft_soup) can not successfully download from that link, but another version (ViT-B/16, openai) can be downloaded normally.

Hi,
I just tried that it still work in my end. The difference could be that ViT-B/16 OpenAI weight is hosted by OpenAI, while the ConvNeXt weight is hosted by huggingface. In this case, I would assume you may need some proxy to download the weight, or you could download the weight  manually and change the pretrained weight path to your local files.
Best,

The problem has been solved. Thanks a lot for your help. :)


drivers: spi: nrf: use dmm

dmm is a Nordic cache helper used to streamline the process of allocating DMA buffer in correct memory region and managing the data cache on nRF54H20: https://github.com/zephyrproject-rtos/zephyr/pull/71278
SPI driver should use it to support a case when CONFIG_SPI_NRFX_RAM_BUFFER_SIZE is set to 0 on nRF54H20.

Please ping me when https://github.com/zephyrproject-rtos/zephyr/pull/74290#discussion_r1639381089 is added to the code. I'll review it after.

Needs rebase
@nika-nordic coming back to this?


In a project with multiple binaries, Cereal's registration fails

AFAIK, Cereal creates a factory with types in each separate binary module and does not export it. So that in each module
CEREAL_REGISTER_TYPE
leads to a call to
InputBindingCreator struct
which calls
StaticObject < InputBindingMap < Archive > > ::getInstance().map
This StaticObject has CEREAL_DLL_EXPORT declaration, yet it does not work.
Are there some docs or could you please comment?

If you haven't read it yet, check out http://uscilab.github.io/cereal/polymorphism.html and see if it helps at all.  What OS are you building for?  Are you performing registration in headers or source files?  Try doing it in the header if you can and see if it resolves any issues.

Hi!
I've done my share of RTFM - all the docs, some of doxygen, some source code j-D
That's a cross-platform build. Problems, so far, are with windows. I am registering in header.
You have a StaticObject class singleton object. My polymorphic class's name ends up in its concretization:
|   auto & map = StaticObject<OutputBindingMap>::getInstance().map;
-- ( cereal/include/cereal/details/polymorphic_impl.hpp : 281)
Under Windows StaticObject has declspec(dllexport), that's what is compiled in, say, library A. The question is, how are you going to declspec(dllimport) it in dependent library B?
AFAIK, in windows dllexport and dllimport are paired.
IMO, for now it works because for every model StaticObject is created anew and not imported - so that you have 10 libraries and 10 StaticObjects (X binaries and X factory singletones). At the end, polymorphic types are registered in the StaticObject of their module and not in the global StaticObject.
And I doubt that dllexport works on template classes, for you can not export an incomplete type.

Maybe, my issue is connected with https://github.com/USCiLab/cereal/issues/107

Will try and look into this a bit more this week/weekend.  Everything related to DLLs in Windows was a nightmare so it won't surprise me if something is wrong.

The thing is, you have to dllimport and dllexport all StaticObject< OutputBindingMap > specializations: for every archive. And then include them somewhere in each project. It may be done through precompiled headers.

Write 4 dllimports and 4 dllexports in some header (say, cerealdefs.h) in most basic library,
link it in every project which uses Cereal
Include cerealdefs.h in all stdafx-s of all libraries
Remove dllexport from StaticObject template declaration: otherwise it won't compile and it is redundant

It works that way.
Furthermore, you can include cereal headers into cerealdefs.h so that they will be precompiled. That will improve compilation time: with Cereal it becomes very long.
If you figure out how to make all this happen through macros and support custom archives and bring this into new version of Cereal, it would be very cool.
For now, I've done as I wrote and it satisfies my needs.

I have a suggestion, AzothAmmo.
If me and my project buddies will rewrite Cereal a little bit in this particular aspect and test it for linux and windows, will you be OK to accept our patch into master branch of repo?
Do you make such decisions for Cereal or should I contact someone else?

I believe he is one of the maintainers.  This project has been very open to pull requests, so I imagine that if it didn't break things and was in general an improvement, that it would be acceptable.

We're open to any improvements.  A few things to keep in mind that help us with pull requests:

Work from the development branch (which is identical to master as of writing this)
Outline your changes.  Especially if this might significantly change the implementation of existing functionality, the more we can discuss it before seeing the result the easier it will be to integrate.
See http://uscilab.github.io/cereal/contributing.html

Thank you. My team and I will create a pull-request when we are ready. Can not say something specific on the concrete date, for now.

I believe I have the same problem.
I've fixed this before by putting the any registries in a cpp file that gets compiled into a DLL, thus since everything links the same DLL, everything gets registered to the same global registry.  However that would require either explicitly putting something in a cpp file to define where the global object should reside, or header sorcery that I am not privy.

I've also run into the same problem. Particularly my setup is this:

DLL: contains abstract base class A that is registered, in addition to a derived class B
EXE: a unittest executable that links against the DLL, this uses GoogleMock, so it has classes that derive from base class A. These mock classes are registered, but it appears they're registered in the StaticObject of the executable module. When attempting to serialize via the DLL, which has a different StaticObject, it fails to find the derived mock type.


Key Detail Stats Page UI

Check out from https://github.com/webb-tools/webb-dapp/pull/581, please view it then come back to this PR.
Commits

chore: move KeygenTable to page-statistics's container
chore: fixed the button icon spacing
uppercase key card
chore: LabelWithValue allowss passing value as react element
fix: spacing error of avatar group component
fix: scrollable for drawer
feat: add KeyDetali component
fix: pagination not display properly
parameterize the CardTable component
applied card table for key detail
chore: format table match the design

Summary of changes

Setup the UI for Key Detail Page

Reference issue to close (if applicable)

Closes NaN

Code Checklist

[x] Tested
[x] Documented

Screenshot
https://user-images.githubusercontent.com/60747384/191030712-1cbbc75a-6576-4bb9-9880-b6c503e604db.mp4

Looks good @AtelyPham  please resolve the conflicts and merge develop


Custom styles

Hi, I don't see anything in the docs about styling the timeline.
So far I've been able to style groups and items using itemRenderer and groupRenderer but this doesn't allow one to do much on styling.  For example, applying a background color to an item will only put a color on top of the default blue one, and some blue pixels will still be visible (margin and padding don't solve this):

Shouldn't there be some props to easily change the color when items are selected/unselected, etc ... ?
Same with the timeline's grid and header, how to change the header (red by default) or the weekend columns (pale yellow) color for example ?
Hopefully I'm not missing something obvious but I think this is essential.

Simply change the styles via CSS? They style are nested so they are hard to override, but it is possible. Inspect the elements and just copy/paste the selectors to your own CSS/Less/SCSS files.

Ok thanks i'll do that!


cherry-pick (4.12): Fix double evaluation and filter sensitive settings (#5641)

Description
Fix double evaluation and filter sensitive settings

Pull Request Test Coverage Report for Build 250860

0 of 0   changed or added relevant lines in 0 files are covered.
32 unchanged lines in 10 files lost coverage.
Overall coverage decreased (-0.009%) to 78.898%

Files with Coverage Reduction
New Missed Lines
%

/libraries/AdaptiveExpressions/BuiltinFunctions/GetNextViableTime.cs
1
93.94%

/libraries/Microsoft.Bot.Builder/DelegatingTurnContext.cs
1
88.24%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/SetProperties.cs
1
66.67%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Recognizers/EntityRecognizers/TextEntityRecognizer.cs
1
85.71%

/libraries/Microsoft.Bot.Builder.Dialogs/Prompts/DatetimePrompt.cs
1
77.5%

/libraries/Microsoft.Bot.Builder/Adapters/TestFlow.cs
2
78.32%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/SetProperty.cs
2
60.0%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/EndDialog.cs
3
72.0%

/libraries/Microsoft.Bot.Builder.Dialogs/Memory/Scopes/SettingsMemoryScope.cs
3
95.71%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/HttpRequest.cs
17
72.97%

Totals

Change from base Build 222931:
-0.009%

Covered Lines:
17700

Relevant Lines:
22434

💛  - Coveralls


Only do counting when count_only=true for repo dashboard

Ref: #29878

I was unable to create a backport for 1.21. @wxiaoguang, please send one manually. :tea:
go run ./contrib/backport 29884
...  // fix git conflicts if any
go run ./contrib/backport --continue


fix mis-spelling in README.md

Ⅰ. Describe what this PR did
Make the document more readable
Ⅱ. Does this pull request fix one issue?
None

Codecov Report

Merging #833 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #833      +/-   ##
==========================================
- Coverage   40.03%   40.02%   -0.02%
==========================================
  Files         108      108
  Lines        6359     6359
==========================================
- Hits         2546     2545       -1
  Misses       3606     3606
- Partials      207      208       +1

Impacted Files
Coverage Δ

supernode/daemon/mgr/scheduler/manager.go
24.06% <0%> (-0.76%)
:arrow_down:

Continue to review full report at Codecov.

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


[Node.js SDK] Skype calling keeps repeating the root dialog

Bot Info

Bot handle:
App ID: 577d561a-c560-463d-91dd-f7fabf1b95ed
SDK Platform: Node.js
SDK Version: "botbuilder": "^3.9.1",
"botbuilder-calling": "^3.0.1",
Active Channels: Skype
Deployment Environment: ec2

Issue Description
I'm trying to use the example code listed here: https://github.com/Microsoft/BotBuilder/blob/master/Node/examples/demo-skype-calling/app.js
The example runs fine when I call from Skype. However, when I say one of the options(e.g. "digits" or "record", the root dialog and the "demoDialog" repeat.
Code Example
https://github.com/Microsoft/BotBuilder/blob/master/Node/examples/demo-skype-calling/app.js
Reproduction Steps

simply run the example code

Expected Behavior
expecting the dialog to switch to the one spoken. For example, "dtmf" or "chat"
Actual Results

We've reproduced the issue and it's related to a change to our state service. You can work around the issue by either using the memory store:
bot.set('storage', new builder.MemoryBotStorage());

Or we should have a fix early next week.

@filanthropic, did @Stevenic's solution help you with the example?

I have updated the example code to use memory storage per @Stevenic fix instructions in previous comment.
https://github.com/Microsoft/BotBuilder/blob/master/Node/examples/demo-skype-calling/app.js#L77

@stevengum97 yes, it did. Thank you @Stevenic !
Now, I just have to figure out how to save the recordedAudio as a file. Any pointers?

@filanthropic Saving audio files is not a Bot Framework specific problem. Consult the Node.js docs on streams and buffers, and/or consult Stack Overflow.


Rasa-moodbot-demo evaluation error

Rasa NLU version:0.13.4
Operating system (windows, osx, ...):windows 10
Content of model configuration file:

language: "en"

pipeline:
- name: "nlp_spacy"                   # loads the spacy language model
- name: "tokenizer_spacy"             # splits the sentence into tokens
- name: "ner_spacy"                   # uses the pretrained spacy NER model
- name: "intent_featurizer_spacy"     # transform the sentence into a vector representation
- name: "intent_classifier_sklearn"   # uses the vector representation to classify using SVM

Issue:
I tried to replicate results from your rasa-moodbot-demo, but when I execute command pprint(interpreter.parse("doing great")) to evaluate Rasa NLU model performance, I get following error.

ValueError                                Traceback (most recent call last)
 in ()
----> 1 pprint(interpreter.parse("doing great"))
c:\users\xxxx\anaconda3\envs\chatbot\lib\site-packages\rasa_nlu\model.py in parse(self, text, time, only_output_properties)
355
356         for component in self.pipeline:
--> 357             component.process(message, **self.context)
358
359         output = self.default_output_attributes()
c:\users\xxxx\anaconda3\envs\chatbot\lib\site-packages\rasa_nlu\classifiers\sklearn_intent_classifier.py in process(self, message, **kwargs)
174             X = message.get("text_features").reshape(1, -1)
175             intent_ids, probabilities = self.predict(X)
--> 176             intents = self.transform_labels_num2str(intent_ids)
177             # predict returns a matrix as it is supposed
178             # to work for multiple examples as well, hence we need to flatten
c:\users\xxxx\anaconda3\envs\chatbot\lib\site-packages\rasa_nlu\classifiers\sklearn_intent_classifier.py in transform_labels_num2str(self, y)
110         :param y: List of labels to convert to numeric representation"""
111
--> 112         return self.le.inverse_transform(y)
113
114     def train(self, training_data, cfg, **kwargs):
c:\users\xxxx\anaconda3\envs\chatbot\lib\site-packages\sklearn\preprocessing\label.py in inverse_transform(self, y)
271         """
272         check_is_fitted(self, 'classes_')
--> 273         y = column_or_1d(y, warn=True)
274         # inverse transform of empty array is empty array
275         if _num_samples(y) == 0:
c:\users\xxxx\anaconda3\envs\chatbot\lib\site-packages\sklearn\utils\validation.py in column_or_1d(y, warn)
786         return np.ravel(y)
787
--> 788     raise ValueError("bad input shape {0}".format(shape))
789
790
ValueError: bad input shape (1, 6)
I see that the error is probably with sklearn classifier but I don't know how to fix it. Please help.
Thank you.

which version of scikit-learn have you got installed?

Hi,
I have the same issue with scikit-learn  0.20.0. I solved the issue by passing first element of intent_ids (a vector)
intents = self.transform_labels_num2str(intent_ids[0])
The rasa nlu version is 0.13.5

forcing scikit-learn==0.19.1 and retraining works for me

My version of scikit-learn is 0.20.0.
I downgraded it to 0.19.1 and it works.
Thanks a lot.


Use packages to find mesh at run time

Hello
Using packages instead of find makes the package searched in runtime and not at the compilation stage. Thanks to this, you can run visualizations on remote equipment where the file path is different.
Regards

@Serafadam Can I ask you for review?


Mention the remaining converters in the converters tutorial

Summary
I've added the remaining converters to the tutorial and standardized usage in a few places.  I was motivated to formulate a PR when I realized that LinearInequalityToPenalty is not the same thing as LinearEqualityToPenalty (it was added later, in #157).
Details and comments
Per https://github.com/Qiskit/qiskit-optimization/issues/84#issuecomment-1008211844, I'm not sure if it matters the order in which the converters that QuadraticProgramToQubo wraps are listed.

I wonder whether listing the full set here is a good idea - as I guess happened now it gets out of data. Maybe it would be better to say it has converters and list some of them to show what they are and say for the full set of convertors refer to the API ref here https://qiskit.org/documentation/optimization/apidocs/qiskit_optimization.converters.html

This PR started out as a single line where I added LinearInequalityToPenalty, which I think is worth mentioning alongside LinearEqualityToPenalty.  From there, I decided just to add everything that seemed to be missing, but I don't feel as strongly that all the other converters be listed.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4068

Relevant Lines:
4489

💛  - Coveralls

This PR looks good because it just adds the missing converters.
But, as @woodsp-ibm pointed out, the API reference has the full list already, the list in this tutorial might be redundant.
If we introduce more converters in the future, it might be good to remove this list and revise this tutorial.


Add virtuosoProps property to ReactSortableTree

Hi @nosferatu500 and @tianzhich, my original motivation for this PR was the need to track a ReactSortableTree's scroll position so that it could be restored after navigating away and back to its containing view. However, I just noticed #45, which should also be resolved by these changes.
As you'll see, I've included a new Storybook page to demonstrate my scrollTop-tracking use case, along with a couple of new rows in the README's Props table (for virtuosoProps as well as virtuosoRef).

I also needed to update react-virtuoso to the latest version in order to get access to the Virtuoso.getState() method, and this caused the yarn.lock file to be modified rather extensively. Since this was done on MacOS 12.6.5 with Node 16.15.1, and I've not tested these changes in any other environment, you might want to verify that a fresh yarn install still works everywhere you expect it too.
Finally, I'd like to get these changes published in a new release of this package (say, v4.4.0?) as soon as possible. Please let me know if there's anything I can do to help out with that.

Hi! Thank you for this PR!
LGTM


Truncate response data in debug log

The response data should be limited to a reasonable amount of data, otherwise, if a user who processing large messages would have a large log file pretty quickly.

Thank you. I think it makes the stdout cleaner


IP allocation doesn't allocate IPs according to IPv4 subnetting math

Loop through osdn-docker-run > 255 times and you'll see what I mean:
++ ip netns exec 32545 ip addr add <IP_ADDRESS>0/16 dev eth0
Error: an inet prefix is expected rather than "<IP_ADDRESS>0/16".

/cc @rajatchopra @mrunalp

/etc/openshift-env/used.log

Yep. That was a hack to get it going.
I didnt expect this to be run so many times :).
Will fix.

To be clear it is only a bug in the experimental branch sdn_no_lbr and sdn_internal_ports.
We do not need to track this bug/fix for openshift-sdn master.

agreed, sorry i should have noted that.

@jeremyeder We have now added an ip allocator that will be integrated into plugins soon.


Final v2 audit of animation

Looks like we're not always setting up animation to be disabled on some styles, including water.

This fix that... https://github.com/tangrams/tron/commit/49e0c87a6867cc59ece7c82ce06a84abe30f2a20


Draft poster for jamboree

Abstract and Notice of Intent (NOI) was submitted to the MSFC STO Jamboree. The next step is to draft the poster.
Note: This will likely take several sprints. Adding deadline of June 5. STI due June 11.
Tasks:

[ ] Create an outline/structure of the poster
[ ] Add content
[ ] Team reviews draft and makes edits/suggestions
[ ] Poster is polished and finalized

Acceptance Criteria:

[ ] Digital version of poster is finished

@ashleyriddle92 @cw0045 Let's try to keep track of how much time we spend on this so we can more accurately estimate the points.


delete rest.Response

Pull Request Test Coverage Report for Build 1234

13 of 47 (27.66%)  changed or added relevant lines in 7 files are covered.
31 unchanged lines in 6 files lost coverage.
Overall coverage decreased (-0.05%) to 62.544%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

client/rest/rest_client.go
11
12
91.67%

core/handler/bizkeeper_consumer_handle.go
0
6
0.0%

core/handler/fault_inject_handler.go
0
6
0.0%

core/handler/transport_handler.go
0
7
0.0%

pkg/util/httputil/httputil.go
0
14
0.0%

Files with Coverage Reduction
New Missed Lines
%

core/config/sd_config.go
1
0.0%

core/handler/bizkeeper_consumer_handle.go
1
38.6%

server/highway/highway_conn.go
2
79.05%

pkg/util/httputil/httputil.go
2
12.0%

core/registry/util.go
4
65.63%

core/registry/servicecenter/cache.go
21
36.24%

Totals

Change from base Build 1228:
-0.05%

Covered Lines:
4926

Relevant Lines:
7876

💛  - Coveralls


CQ-4345916 Capture DOR specific properties for the core components

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes and the overall coverage did not decrease.
[ ] All unit tests pass on CircleCi.
[ ] I ran all tests locally and they pass.

@rismehta , can you pls approve this? looking to merge this today.


Switch to 1.3.x of config-api in community

Do the other config-api's also need to be updated/released, more specifically the mongo.config-api, cassandra.config-api, neo4j.config-api, orientdb.config-api?

I don't think the NoSQL config APIs need to be bumped, as they are not used in product (right?). What needs to be bumped is the Keycloak config API.

Correct, the NoSQL config APIs are not in product.  Thanks for clarifying!

@scottmarlow : @Ladicek is right. Only the ones shared with the prod build need to be bumped to avoid conflicts.
And @Ladicek : What you've been asking for is here https://github.com/wildfly-swarm/wildfly-swarm/pull/807


Support for Cross correlating Model variants and Unique Ops Congiguration script

The script should do the following things below:

Run all models until specified compile depth
Export unique op requirements to the file (each model as it's own file)
Parse those op requirements and create output file that can be loaded up into google sheet. This output file should contain the listed models on on one axis and then list of ops on another axis. Elements in between should contain checkmark if desired op from X axis, exists in model on Y axis. Models should be sorted alphabetically and ops should be sorted by number of occurrence in the models

Buda PR: https://yyz-gitlab.local.tenstorrent.com/tenstorrent/pybuda/-/merge_requests/2331


Bluetooth: Controller: CIG: LL Rejects: Remote request being rejected

Is your enhancement proposal related to a problem? Please describe.
A clear and concise description of what the problem is.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or graphics (drag-and-drop an image) about the feature request here.

This is implemented as part of PR43023 and subsequent fixup PRs


fk_id support for belongs_to

Squash your commits into one commit -- this looks messy -- use git pull --rebase -- so that it doesn't generate a merge commit -- makes PRs look well...ugly.

@robbyoconnor That's not a problem, we can handle it on merge time. :)

@stanislas-m I think I'm just OCD in general about these things :)


Responsive styling on mobile

Migrated from GitLab: https://gitlab.com/meltano/hub/-/issues/27
Originally created by @DouweM on 2021-05-07 14:44:58

View 3 previous comments from the original issue on GitLab


[FLOC-1775] Deal with ignorance of containers or datasets

Thanks.  Code changes look pretty good.  I think this merits some some words in the API documentation.  Seeing some nodes go missing from these lists will be surprising and mysterious and should be explained so people don't think it's just due to random bugs in our software.
Merge when that's done and buildbot is however green it can be expected to be.


⚠ Turnos 24x7 has degraded performance

In 7b14900, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 807 ms

Resolved: Turnos 24x7 performance has improved in 7f34697.


[DCOS_OSS-618] Ignore core.ssl_verify for external requests

Otherwise, when a cluster is configured with a custom CA path, it will
try to use it instead of the system's CA bundle and external requests
will fail.
In such cases, setting it to None will let the requests package enabling
it by default.

Also, we need to add an integration test before we can call this done.

I've just pushed a commit according to your comments.
What kind of integration test are you thinking of? As it involves an HTTPS request to an external URL I don't see an obvious way to do it.


Add endpoint factory

Needed for https://github.com/ManageIQ/manageiq-providers-kubernetes/pull/38

@miq-bot add_label test

@durandom please review

@durandom addressed comments, please take another look

@durandom ready for merge?

@durandom done. Ready?


Help: Issue with cloudflare-dyndns chart

Sorry I am still a bit of a newbie working with Kubernetes and Charts. I am having an issue deploying this chart.
# cloudflare-dyndns-helm-values.txt
cloudflare:
  user: "$CLOUDFLARE_USER"
  token: "$CLOUDFLARE_APIKEY"
  zones: "$CLOUDFLARE_ZONES"
  hosts: "$CLOUDFLARE_HOSTS"
  record_types: "$CLOUDFLARE_RECORDTYPES"

# cloudflare-dyndns.yaml
---
apiVersion: helm.fluxcd.io/v1
kind: HelmRelease
metadata:
  name: cloudflare-dyndns
  namespace: default
  annotations:
    fluxcd.io/automated: "false"
spec:
  releaseName: cloudflare-dyndns
  chart:
    repository: https://billimek.com/billimek-charts/
    name: cloudflare-dyndns
    version: 1.0.0
  values:
    image:
      repository: hotio/cloudflare-ddns
      tag: stable-47b759b
    resources:
      requests:
        cpu: 100m
        memory: 64Mi
      limits:
        cpu: 500m
        memory: 128Mi
  valueFileSecrets:
  - name: "cloudflare-dyndns-helm-values"

The chart has defined the env variables like so:
...
    spec:
      containers:
        - name: {{ .Chart.Name }}
          image: "{{ .Values.image.repository }}:{{ .Values.image.tag }}"
          imagePullPolicy: {{ .Values.image.pullPolicy }}
          env:
            - name: CF_USER
              valueFrom:
                secretKeyRef:
                  name: {{ template "cloudflare-dyndns.fullname" . }}
                  key: cloudflare-dyndns-user
            - name: CF_APIKEY
              valueFrom:
                secretKeyRef:
                  name: {{ template "cloudflare-dyndns.fullname" . }}
                  key: cloudflare-dyndns-token
            - name: CF_ZONES
              valueFrom:
                secretKeyRef:
                  name: {{ template "cloudflare-dyndns.fullname" . }}
                  key: cloudflare-dyndns-zones
            - name: CF_HOSTS
              valueFrom:
                secretKeyRef:
                  name: {{ template "cloudflare-dyndns.fullname" . }}
                  key: cloudflare-dyndns-hosts
            - name: CF_RECORDTYPES
              value: "{{ .Values.cloudflare.record_types }}"
            - name: DETECTION_MODE
              value: "{{ .Values.cloudflare.detection_mode }}"
            - name: LOG_LEVEL
              value: "{{ .Values.cloudflare.log_level }}"
            - name: SLEEP_INTERVAL
              value: "{{ .Values.cloudflare.sleep_interval }}"
...

Do you think I should update the Chart to not use the valueFrom.secretKeyRef? It seems as though they can all just be regular values (see below) like since the valueFileSecrets will just drop them in.
          env:
            - name: CF_USER
              value: "{{ .Values.cloudflare.user }}"
            - name: CF_APIKEY
              value: "{{ .Values.cloudflare.token }}"
            - name: CF_ZONES
              value: "{{ .Values.cloudflare.zones }}"
            - name: CF_HOSTS
              value: "{{ .Values.cloudflare.hosts }}"
            - name: CF_RECORDTYPES
              value: "{{ .Values.cloudflare.record_types }}"
            - name: DETECTION_MODE
              value: "{{ .Values.cloudflare.detection_mode }}"
            - name: LOG_LEVEL
              value: "{{ .Values.cloudflare.log_level }}"
            - name: SLEEP_INTERVAL
              value: "{{ .Values.cloudflare.sleep_interval }}"

Again sorry and thanks for your help!

In addition to the cloudflare-dyndns-helm-values.txt file, do you have a secret created named the same (cloudflare-dyndns-helm-values) in the same namespace?  You will need a secret with the data json containing a values.yaml key with a base64-encoded set of values data to be merged in.
See this for a possible better way to go about doing this.

I do have secret there in the same namespace. I am just a little confused on how to mix secrets and the non-secret envVars.
Shouldn't the name property below be cloudflare-dyndns-helm-values? Or is that is what is interpolated?
            - name: CF_HOSTS
              valueFrom:
                secretKeyRef:
                  name: {{ template "cloudflare-dyndns.fullname" . }}

Closing this issue, I'll be able to submit something when I test and fix locally.


[BUG] - [VERY LITTLE :-] - use - for snake, not _

Hi,
First congratulation for this great work !!!
There is a little mistake in the snake case snippet, it mention '_' in exemples results but i think it might be a '-', take a look to following lines :

https://github.com/30-seconds/30-seconds-of-python/blame/master/snippets/snake.md#L21
https://github.com/30-seconds/30-seconds-of-python/blame/master/snippets/snake.md#L22
https://github.com/30-seconds/30-seconds-of-python/blame/master/snippets/snake.md#L23
https://github.com/30-seconds/30-seconds-of-python/blame/master/snippets/snake.md#L24

[BUG] use - for snake, not _
Expected Snippet Behavior
'-' in  exemples results
Current Snippet Behavior
'_'  in  exemples results
Possible Solution
'-' in  exemples results
Best regards,
Julien

Hi all, I also noticed that the camel case example is not working:
>>> snake('camelCase')
'camelcase'

Maybe transforming the original JS regexp didn't work for those. I will see what I can do or you can open a PR with a proposal to fix the code.


Switch statement?


add attribute_changed hook to event handlers

🚧 needs tests

@chef/client-core should be ready for review


[Rust] Several Rust idioms

BTW, you can publish fury-derive to hold the crate name.
Also, fury is taken by @carllerche so we may have a communication with them for possibly passing the name.

BTW, you can publish fury-derive to hold the crate name.
Also, fury is taken by @carllerche so we may have a communication with them for possibly passing the name.

Thanks for reminding this, seems rust fury hasn't been updating for three years, will talking to  @carllerche for naming issue


Error !!

I might be seeing a similar message to @wtchoga . @ktmeaton  do you know if https://github.com/phac-nml/rebar/blob/ae94b5327b5366ebfa80dfb60aac614b7d0d05cb/rebar/constants.py#L25 (url to https://github.com/corneliusroemer/pango-sequences) has from time to time excluded a lineage unintentionally?
I'm trying to track down why `rebar/dataset/sars-cov-2-latest/alignment.fasta' builds itself and skips from XCC to XCE, but I worry each one of us is working with a different pango-sequences pull

Hi @wtchoga and @GLTOhorsman! Thanks for documenting this, I've encountered the same thing. I'm working on a larger fix for this, and am aiming for a pre-release within the week. Part of this will include version controlling which pango-sequences pull was used (ex. commit hash/date), for reproducible analyses.
@GLTOhorsman, I had the exact same question yesterday about pango-sequences excluding lineages. In that repo's Caveats section, it's mentioned that lineages will only be included "if there are more than 3 sequences of a lineage available as open data (Genbank/RKI/COG-UK)". For example, at the time of writing, XCU and its specific parents (XBC.1.7.1 and FL.23.2.1) are not yet available, since they have too few sequences. I will include a warning if a population is expected to be present (ex. XCU is designated) but doesn't have a consensus sequence yet.


Update docs workflow

Use actions/deploy-pages for deployment
Update artifact name and extraction directory

Builds on #1.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

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

Totals

Change from base Build<PHONE_NUMBER>8:
0.0%

Covered Lines:
297

Relevant Lines:
297

💛  - Coveralls


Changing logic to get the context after we check for --help

Also adding a \n to the error message just in case.
Closes https://github.com/golang/dep/issues/125
@jessfraz I hope I can just open the PR here right? Let me know if I need to change anything!

awesome!!!

LGTM you rock!!


[REQUEST] A script to remove unused drivers

For example, if you just swapped out an AMD Radeon card for an Nvidia GeForce card, the script would detect that there is a driver that's currently unused by any hardware, and automatically remove it.
Check out the script here: https://theitbros.com/remove-old-unused-drivers-using-powershell/
This probably better configured off by default, since it may remove printer drivers when its powered off for example. Alternatively, it could also ask for confirmation for every driver removal beforehand.

Hi @ptrkhh , I'll take a look at it and see if it's a good fit for Tron. Thanks.

This is a great idea.
Though, prompting for each driver would be too much and also in many cases people won't know/won't be able to figure out what driver specifically does and may tend to accept it's removal (this is considering they are properly going through the whole script).
If we can figure out when the driver was last used/when that hardware was last plugged in and then set a threshold say 6 months (give an option to adjust the threshold) and remove all drivers not used in that time period. This is one approach.
All said, I still think there should be a smarter way to handle this.
Great Idea

Hi @ptrkhh , after testing and reviewing, I think I'm going to leave this out of Tron for the time being. Reasoning being that while driver removal is a good idea (and often saves a ton of space), it's a little more prone to causing errors than I'm comfortable with for the average user, so I'll leave this in the "up to the discretion of the tech" category. Thanks for the great suggestion.


Responsiveness issues

e.g. Try to open the website in a window 1300px wide, the dropdown disappears and the map is not shown correctly

yeah I had what I think are similar problems (using Firefox). The header text and the dropdown were not shown when the browser was at 100% - when zooming out to 80% or lower the entire thing was displayed.
If the text is too long, let me know, but I assume its on the programming side?


[firebase_auth]: Windows build is broken again

Is there an existing issue for this?

[X] I have searched the existing issues.

Which plugins are affected?
Auth
Which platforms are affected?
Windows
Description
C:...windows\flutter\ephemeral\cpp_client_wrapper\include\flutter\encodable_value.h(199,60): error C2665: 'std::variant<std::monostate,bool,int32_t,int64_t,double,std::string,std::vector<uint8_t,std::allocator<uint8_t>>,std::vector<int32_t,std::allocator>,std::vector<int64_t,std::allocator<int64_t>>,std::vector<double,std::allocator>,flutter::EncodableList,flutter::EncodableMap,flutter::CustomEncodableValue,std::vector<float,std::allocator>>::variant': no overloaded function could convert all the argument types [C:...build\windows\x64\plugins\firebase_auth\firebase_auth_plugin.vcxproj]
Reproducing the issue
Try to use firebase_auth on windows builds
Firebase Core version
Whatever 5.3.3 depends on or latest (transitive)
Flutter Version
3.24.5
Relevant Log Output
C:...windows\flutter\ephemeral\cpp_client_wrapper\include\flutter\encodable_value.h(199,60): error C2665: 'std::variant<std::monostate,bool,int32_t,int64_t,double,std::string,std::vector<uint8_t,std::allocator<uint8_t>>,std::vector<int32_t,std::allocator<int>>,std::vector<int64_t,std::allocator<int64_t>>,std::vector<double,std::allocator<double>>,flutter::EncodableList,flutter::EncodableMap,flutter::CustomEncodableValue,std::vector<float,std::allocator<float>>>::variant': no overloaded function could convert all the argument types [C:...build\windows\x64\plugins\firebase_auth\firebase_auth_plugin.vcxproj]

Flutter dependencies
literally firebase_auth
Additional context and comments
C:...windows\flutter\ephemeral\cpp_client_wrapper\include\flutter\encodable_value.h(199,60): error C2665: 'std::variant<std::monostate,bool,int32_t,int64_t,double,std::string,std::vector<uint8_t,std::allocator<uint8_t>>,std::vector<int32_t,std::allocator>,std::vector<int64_t,std::allocator<int64_t>>,std::vector<double,std::allocator>,flutter::EncodableList,flutter::EncodableMap,flutter::CustomEncodableValue,std::vector<float,std::allocator>>::variant': no overloaded function could convert all the argument types [C:...build\windows\x64\plugins\firebase_auth\firebase_auth_plugin.vcxproj]

I'm having the same issue. I have tried every troubleshooting step I can think of, no luck.

cc @russellwheatley

I believe this is a duplicate of: https://github.com/firebase/flutterfire/issues/16536#issuecomment-2518723562

@StephanTLavavej - thank you for the input. Closing out in favour of this issue: https://github.com/firebase/flutterfire/issues/16536


No Such Module 'TwilioVoice'

i am getting this error in Xcode 15.2 "No Such Module 'TwilioVoice'" , i have runned pod install, pod update everything but getting this error

Hi @wmt-raj-mobile
The quickstart is configured to reference the SDK using SPM. Once pod install is done you should be able to use the .xcworkspace, which has the SDK reference set up for you. Please check the Xcode project settings in case the app cannot find the SDK.

It was not working as per your solution. But when i updated the ruby from 2.7.0 to 3.3.0 it started working.


feat: started and completed feat/dvt-button

SUMMARY
started and completed feat/dvt-button
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[x] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

DvtButton.module.tsx dosya böyle olmamalı hepimiz aynı bir dosya kısa harflar olmalı. dvt-button.module.tsx

superset Licensed olan 3 dosya eklemeniz gerekiyor


Ensure border styles and z-levels match before doing border healing

Goal is to resolve https://github.com/JoshKarpel/counterweight/blob/f3d42f5fd24d4d060799be41430583810b7f4717/counterweight/border_healing.py#L42

#91 will mean we need to match on z as well.

I'm not 100% sure that we should match on z, actually - merging between levels does make some things look nicer...


Update version separated from setup

minor change
version separated from setup
print version in how to

 pip3 freeze  | grep iroha-ya-cli

Enough


Fix Sass partial changes not triggering recompiles in Dev

Changes
The @snowpack/plugin-sass plugin does not correctly trigger sass recompiling when partials past the first level of imports are changed during snowpack dev. Although Snowpack recognizes that the file has changed, it does not trigger the top file importing it as changed so nothing is recompiled.
The scanSassImports() function will now recursively find all the child imports. It will also reduce duplicates since it is possible that many partials are importing the same file like a _variables.scss partial. I also added support for @forward which is also used for importing partials https://sass-lang.com/documentation/at-rules/forward.
Related Discussions:
#1717
#2668
Testing
I first updated the marks a dependant as changed when an imported changes and isDev=true test to look for a child of a child dependency. This proved the bug by causing the test to fail. I then wrote the code to allow the test to pass. I also manually tested in a large project with multiple levels of nesting.
Docs
Bug fix only.

Just released in v1.3.1! 🎉


cmd/compile: Inline simple functions with dead sections

Using go1.5beta3.
Simple functions with dead sections that would normally be inlined should be inlined.
Example:
package foo

import "fmt"

const debug = false

func Foo(a, b int) int {
	if debug {
		// This section is never used
		fmt.Println()
	}
	return a + b
}

func Bar(a, b int) int {
	return a + b
}

However, only Bar is inlined:
$ go build -gcflags=-m
./foo.go:15: can inline Bar

This is #9274


fix wysihtml5 asset path

fix path for use in production with rails 4.2 and last spockets
Asset was linked to from an alias rather than its exact path. Alias resolving may not be available in production. Use "bootstrap-wysihtml5/index.css" instead of "bootstrap-wysihtml5.css"
Asset was linked to from an alias rather than its exact path. Alias resolving may not be available in production. Use "bootstrap-wysihtml5/index.js" instead of "bootstrap-wysihtml5.js"

This pull request don't fix the wysihtml5 in production. That works in development but not in production.

I updated my app to Rails 4.1.2 and Ruby 2.2.2 and I am having this very same issue. It started with:
Asset filtered out and will not be served: add `Rails.application.config.assets.precompile += %w( bootstrap-wysihtml5/index.css )` to `config/initializers/assets.rb` and restart your server

Following the advice didin't work, I got the message shown on entries above...
Any ideas?

I just tried these same changes and they work both on development and production, tried them on Heroku... I will submit the pull request in case someone wants to check it out…


[Dubbo-6694] Fix URL compatible with "default." prefix

What is the purpose of the change
Fix issue #6694、https://github.com/thubbo/jmeter-plugins-for-apache-dubbo/issues/125
Same PR #6702
Brief changelog
org.apache.dubbo.common.URL
Verifying this change
https://github.com/thubbo/jmeter-plugins-for-apache-dubbo/issues/125
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[x] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

@ningyu1 hi, thanks for your contribution.
I think the issue this pr willing to fix has been fixed in #7239.
Please feel free to reopen it if you have any question.


PowerToys Settings not opening :(

Microsoft PowerToys version
0.67.1
Installation method
GitHub, PowerToys auto-update, Microsoft Store
Running as admin
None
Area(s) with issue?
General
Steps to reproduce
Despite being installed and recently being updated, Why is it not opening?

✔ Expected Behavior
It should have opened!
❌ Actual Behavior
It is neither getting shown in installed apps at Store nor at Windows Search.
Hence, I am unable to run power toys and no power toy features are working.
Other Software
No response

/reportbug

Do you have the icon in the system tray?

When you say “open”, walk me through this

I'm getting a similar problem after the most recent update (0.67.1)

PowerTools "Runs" is showing up in the system tray.
But I cannot open a PowerTools window. And thus cannot change the settings.

None of the following work: Opening PowerTools from the start menu, opening it by clicking on the system tray, right clicking on the system tray icon and selecting "settings". I get the hourglass icon for a moment and then no further response.

Only some functionality seems to be working with the program in this state. My key remappings are working fine. But the Quick Accent feature is not.
Restarting my machine did not fix this problem.

PowerToysReport_2023-02-09-00-09-37.zip

@RMWinslow perfectly described what's happening!
Here are some more snaps of the system tray:

@RMWinslow , Looking at the installfolderstructure.txt from the bug report, it looks like some previous files from 0.67.0 were not upgraded correctly. Specifically PowerToys.Interop.dll
Would you mind:
1 - Uninstall PowerToys.
2 - See if the PowerToys install directory (default: C:\Program Files\PowerToys) is still there. If it is, delete it manually.
3 - Install PowerToys.
Please let us know if this works for you.
/needinfo
@stefansjfw , We've had other cases recently where this also happened for PowerToys.Interop.dll . Can you check if there's anything different about PowerToys.Interop.dll or add some logs we can see in the reports?

Should I also do what's been instructed to @RMWinslow ?

Should I also do what's been instructed to @RMWinslow ?

Yes, please.

Would you mind: 1 - Uninstall PowerToys. 2 - See if the PowerToys install directory (default: C:\Program Files\PowerToys) is still there. If it is, delete it manually. 3 - Install PowerToys.

I uninstalled 0.67.0
The install directory seems to be removed (or at least isn't in C:\Program Files).
I installed 0.69.1 and it seems to be functioning properly.

Thanks for the update. I'll close the issue for now. If it happens again, feel free to reopen it.

I am getting the same problem. Does not start, nothing in tray. No previous files in Program Files prior to install.
I tried via Store, Winget and scoop, surely scoop at least should start since it doesn't care about system files.
Windows 11, Insider, Release channel.

@yashamon Could you attach /bugreport?

How can I attach bug report? Nothing shows up in system tray. In fact, if I execute powertoys.exe in shell there is no output at all.

Sorry, forgot to mention the workaround for bug report. Go to Program files (or any dir Powertoys is installed in)/tools and run BugReportTool.exe. it should generate zip file on your Desktop. Thanks

Sorry for late reply I was travelling. I get: NativeCommandExitException:
Program "PowerToys.BugReportTool.exe" ended with non-zero exit code:
-1073740791.
On Tue, Jul 25, 2023 at 4:15 PM Stefan Markovic @.***>
wrote:

Sorry, forgot to mention the workaround for bug report. Go to Program
files (or any dir Powertoys is installed in)/tools and run
BugReportTool.exe. it should generate zip file on your Desktop. Thanks
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/23982#issuecomment-1650635845,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABQGIZUNI7N5XC7L4N6XPCTXSBAR7ANCNFSM6AAAAAAUVMFZBE
.
You are receiving this because you were mentioned.Message ID:
@.***>

I have a feeling that my dot net libraries are corrupted. How would I fix
that?
On Sat, Jul 29, 2023 at 1:52 PM yasha savelyev @.***>
wrote:

Sorry for late reply I was travelling. I get: NativeCommandExitException:
Program "PowerToys.BugReportTool.exe" ended with non-zero exit code:
-1073740791.
On Tue, Jul 25, 2023 at 4:15 PM Stefan Markovic @.***>
wrote:

Sorry, forgot to mention the workaround for bug report. Go to Program
files (or any dir Powertoys is installed in)/tools and run
BugReportTool.exe. it should generate zip file on your Desktop. Thanks
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/23982#issuecomment-1650635845,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABQGIZUNI7N5XC7L4N6XPCTXSBAR7ANCNFSM6AAAAAAUVMFZBE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Nevermind, after uninstalling powertoys everything works well. So it is not
my libraries.
On Sat, Jul 29, 2023 at 5:20 PM yasha savelyev @.***>
wrote:

I have a feeling that my dot net libraries are corrupted. How would I fix
that?
On Sat, Jul 29, 2023 at 1:52 PM yasha savelyev @.***>
wrote:

Sorry for late reply I was travelling. I get: NativeCommandExitException:
Program "PowerToys.BugReportTool.exe" ended with non-zero exit code:
-1073740791.
On Tue, Jul 25, 2023 at 4:15 PM Stefan Markovic @.***>
wrote:

Sorry, forgot to mention the workaround for bug report. Go to Program
files (or any dir Powertoys is installed in)/tools and run
BugReportTool.exe. it should generate zip file on your Desktop. Thanks
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/23982#issuecomment-1650635845,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABQGIZUNI7N5XC7L4N6XPCTXSBAR7ANCNFSM6AAAAAAUVMFZBE
.
You are receiving this because you were mentioned.Message ID:
@.***>

I've been using powertoys since 0.43 (21?) and its always been stable and a great tool! Since the upgrade to 0.7? powertoys or settings won't open. Any attempt to interact causes windows explorer to 'die' . Windows 11 is patched up to date and I've attached  a report.
I've uninstalled and checked the install directory is empty. then downloaded the lastest binaries from github manually . Normally I let powertoys update automatically .
PowerToysReport_2023-10-03-22-46-56.zip

This fixed by the last malwarebytes update

I have the same issue with version <IP_ADDRESS>: It seems like all of the Tools are working but I cannot open the PowerToys overview or its settings (when I click on it, nothing happens).
But I was able to generate the ZIP with the report:
PowerToysReport_2023-10-09-15-16-05.zip

I'm having a similar issue, I tried to open, I see the windows but it closes before I can interact with the window of PowerToys.
PowerToysReport_2023-12-29-12-10-28.zip

I'm having the same issue, using latest version as of this post. Uninstalled and Reinstalled several times but no luck. Whenever I try to launch settings from the Taskbar it never opens.
PowerToysReport_2024-02-21-09-03-13.zip

Same issue, the only way to solve it was to uninstall everything with Bulk Crap Unninstaller, deleting the Registry Keys and Power Automate, which has a different entry somehow.
Then installed from Microsoft Store, not the installer in the website. That solved the issue, tried everything suggested here and some other options I found online and nothing seemed to help but this.

Same issue, PowerToys v0.80.1 loads into the System Tray but I can't get the Settings to open. Downloaded from Microsoft Store.
PowerToysReport_2024-04-25-10-24-31.zip

Same issue
PowerToysReport_2024-07-07-14-37-23.zip

PowerToysReport_2024-07-22-09-13-39.zip
Same issue.  Used to work on Win 10, upgraded to Win 11, never seen PowerToys again except in system tray and pop-up menu.  Tried everything here.

Same Issue
PowerToysReport_2024-07-29-12-16-41.zip

I'm also experiencing this on my work machine. Application appears to be running, but settings and other screens won't open. The frame of the window appears, but then disappears before any content is shown.
Downgrading to 0.80.1 seems to fix the problem. At least the screens will open. Bug report from 0.80.1 attached.
PowerToysReport_2024-08-01-14-20-28.zip

If you installed powertoys in Microsoft store, the path would be "C:\Users(username)\AppData\Local\Microsoft\PowerToys".
I resolved not opening issue by following steps.

remove powertoys in the control panel.
delete the folder mentioned above.
reboot PC.
install 0.80.1 version.


Possible PIT implementation.

I have forked rxinu and implemented a basic PIT driver. The PIT, under this model, operates in Mode 3, using lobyte/hibyte configuration. I have experimented with moving the resched() call into the timer irq, and saw that a couple of test processes ran, printing to screen. However, after that it stopped, which is something I need to debug. Anyway, I thought you might like to check out my most recent few commits and give your opinion on the code so far.
Screenshot of rxinu running with the new model:

Thanks @too-r!  It looks like the PIT driver is working as intended.
Regarding the issue you saw, it looks like there were a few issues involving deadlocks in resched.  The resched method will deadlock if any scheduling related locks are still held when it is called.
Problem 1: Incorrect locking of ready_list in resched
Debugging
I found out where the kernel is deadlocking by running gdb, hitting the deadlock, and running info stack
Call #2 in the stack is the culprit:
#2  0x000000000010cdb2 in rxinu::scheduling::cooperative_scheduler::{{impl}}::resched (
    self=0x146538 <<rxinu::scheduling::SCHEDULER as core::ops::deref::Deref>::deref::__stability::LAZY+8>) at src/scheduling/cooperative_scheduler.rs:102

It points to the following following block:
// skip expensive locks if possible
if self.ready_list.read().is_empty() {
    return;
}

Click to expand debugging output
(gdb) info stack
#0  0x000000000012542b in core::sync::atomic::AtomicUsize::load (
    self=0x146568 <<rxinu::scheduling::SCHEDULER as core::ops::deref::Deref>::deref::__stability::LAZY+56>, order=core::sync::atomic::Ordering::Relaxed)
    at /home/rob/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/src/libcore/sync/atomic.rs:1081
#1  0x000000000012bc33 in spin::rw_lock::RwLock<alloc::vec_deque::VecDeque<rxinu::scheduling::process::ProcessId>>::read<alloc::vec_deque::VecDeque<rxinu::scheduling::process::ProcessId>> (
    self=0x146568 <<rxinu::scheduling::SCHEDULER as core::ops::deref::Deref>::deref::__stability::LAZY+56>) at /home/rob/.cargo/registry/src/github.com-1ecc6299db9ec823/spin-0.4.6/src/rw_lock.rs:176
#2  0x000000000010cdb2 in rxinu::scheduling::cooperative_scheduler::{{impl}}::resched (
    self=0x146538 <<rxinu::scheduling::SCHEDULER as core::ops::deref::Deref>::deref::__stability::LAZY+8>) at src/scheduling/cooperative_scheduler.rs:102
#3  0x00000000001232f4 in rxinu::arch::x86::interrupts::irq::timer::{{closure}} ()
    at src/arch/x86/interrupts/irq.rs:28
#4  0x0000000000115dcb in rxinu::arch::x86::interrupts::disable_interrupts_then<closure> (
    uninterrupted_fn=...) at src/arch/x86/interrupts/mod.rs:15
#5  0x00000000001232a7 in rxinu::arch::x86::interrupts::irq::timer (_stack_frame=0x14bbc8)
    at src/arch/x86/interrupts/irq.rs:27
#6  0x0000000000128571 in alloc::vec_deque::VecDeque<rxinu::scheduling::process::ProcessId>::grow_if_necessary<rxinu::scheduling::process::ProcessId> (
    self=0x146570 <<rxinu::scheduling::SCHEDULER as core::ops::deref::Deref>::deref::__stability::LAZY+64>)
    at /home/rob/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/src/liballoc/vec_deque.rs:1755
#7  0x00000000001289e1 in alloc::vec_deque::VecDeque<rxinu::scheduling::process::ProcessId>::push_back<rxinu::scheduling::process::ProcessId> (
    self=0x146570 <<rxinu::scheduling::SCHEDULER as core::ops::deref::Deref>::deref::__stability::LAZY+64>, value=...)
    at /home/rob/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/src/rust/src/liballoc/vec_deque.rs:1120
#8  0x000000000010cd61 in rxinu::scheduling::cooperative_scheduler::{{impl}}::ready (
    self=0x146538 <<rxinu::scheduling::SCHEDULER as core::ops::deref::Deref>::deref::__stability::LAZY+8>, id=...) at src/scheduling/cooperative_scheduler.rs:96
#9  0x000000000010d5e1 in rxinu::syscall::process::create (new_proc=0x10dc20 <rxinu::test_process>,
    name=...) at src/syscall/process.rs:9
#10 0x000000000010daea in rxinu::rust_main (multiboot_information_address=2585288) at src/lib.rs:58
#11 0x000000000010a0b3 in long_mode_start ()
Backtrace stopped: Cannot access memory at address 0x14c000

Background
The reason that resched deadlocks when locks are held is that our context switch to a new process is an inner call in resched.  Here's how the stack call looks when we run a new process:
new_process
rxinu::arch::context::switch_to
rxinu::scheduling::cooperative_scheduler::resched
rxinu::interrupts::irq::timer
rxinu::rust_main

Note that resched will always remain in the call stack until we get back to the null process.
The next question is "Why are locks held during the call to resched?"  The way Rust implements locks is that they're held until drop is called on them.  There are two ways to make sure drop is called for a lock: 1) Reach the end of the scope or 2) drop is manually called on the lock.
I debated whether to use solution 1) or 2) and ultimately decided to stick with solution 1).  I reasoned that Rust as a language wants to limit manual management as much as possible.  That's why Rust uses scoping and lifetimes by default.  So I decided that rXinu will use scoping to free held locks.
Solution
So the issue is that I didn't properly scope the checking of the ready_list in resched.  Because the ready_list lock is still active when we call context::switch_to, there will be a deadlock the next time we try to read from or write to ready_list.
Here's my fix:
-        // skip expensive locks if possible
-        if self.ready_list.read().is_empty() {
-            return;
+        {
+            // skip expensive locks if possible
+            if self.ready_list.read().is_empty() {
+                return;
+            }

Previously, we didn't run into this issue by chance.  Thanks to your PIT driver, bugs in my scheduling code will be triggered more easily and can be fixed sooner!
Problem 2: Interrupting critical scheduling code
Now that we're using an irq handler to call resched, we need to ensure that the PIT irq handler doesn't invoke resched at incorrect times in our kernel.
Right now, important code, such as scheduling code using locks, can be interrupted.  We need to disable interrupts while holding locks and re-enable interrupts when we are done.
My solution was to wrap the important scheduling components in the disable_interrupts_then function.
Problem 3: Resetting the PIT_TICKS value after resched
Currently, your PIT irq handler increments after every timer interrupt.  It then checks to see if we've had 10 ticks, and if so we call resched.
The issue is that we need to reset PIT_TICKS to 0 after calling resched.  Otherwise, PIT_TICKS will always be greater than 10 and we will call resched every clock tick.
diff --git a/src/arch/x86/interrupts/irq.rs b/src/arch/x86/interrupts/irq.rs
index 295108d..8d97885 100644
--- a/src/arch/x86/interrupts/irq.rs
+++ b/src/arch/x86/interrupts/irq.rs
@@ -18,12 +18,17 @@ fn trigger(irq: u8) {
 }

 pub extern "x86-interrupt" fn timer(_stack_frame: &mut ExceptionStack) {
+    use arch::x86::interrupts;
+
     pic::MASTER.lock().ack();

     if PIT_TICKS.fetch_add(1, Ordering::SeqCst) >= 10 {
+        PIT_TICKS.store(0, Ordering::SeqCst);
         unsafe {
-            SCHEDULER.resched();
-        }
+            interrupts::disable_interrupts_then(|| {
+                SCHEDULER.resched();
+            });
+        }
     }
 }

Once I made the above changes, the scheduler worked as intended.  I stress tested the scheduling code by creating a process cycle where process_a creates process_b which creates process_a until memory runs out.  We are able to create and run 12 thousand processes before running out of memory.
However, there seems to be a significant performance hit by introducing the PIT interrupt handler.  It takes 2-3 times as long to create and run the 12 thousand processes than when we ran resched in the null process.
It's not a serious issue since rXinu doesn't emphasize performance, but it is definitely something I will track in the issue tracker once your code is in.  I believe it's related to wasted cycles either in the PIT driver or somewhere in the scheduling code.

Yep, those changes to do with storing 0 to the ticks and wrapping reschedule calls in disable_interrupts_then() were things that I'd implemented in my own system but not in my commits to rxinu, oh well. I'll make the changes you've noted above, and then probably PR this since PIT functionality is critical to progress with the usermode PR. The performance issue is something I've noticed with my own code, and is definitely worth looking into.

Sounds good 👍 Thanks again for the help!
I haven't researched PIT drivers too much, so I'm interested in finding out how frequently your PIT driver triggers an interrupt.  Is it every millisecond? Or every 0.5 seconds?  This answer could explain why context switching takes so long.

Triggers roughly every 2ms. It is certainly possible to make the PIT interrupt more frequently, however I've read that generally OS's use a divisor where the frequency is roughly 100Hz, or 10ms, more than what I'm using.


Hep.B Antigen

Hep.B Antigen missing from tablet.
I am assuming it's missing from the feed from the server.
Please check.

Hep. B exists on the tablet under the vaccines category of commodities, not a server issue


Key Discovery

We've scoped Notary v2 to not support TOFU.
How will clients acquire the public keys for different registries or repos?

Per the TUF design, if we can distribute root keys that do namespaced delegations for a registry or registries, we can reduce the number of keys that need to be distributed to end users. This reduced number of keys could be shared with client systems when they are provisioned using existing secret distribution methods, like SPIFFE/SPIRE.

For SCITT (see draft spec), the proposal is to use DID both as mechanism to establish issuer identifiers and to do key discovery (via DID resolution), while not being constrained to a single identity system but having extensibility through DID methods. For Notary to support this, signing envelopes would have to include the DID as issuer in the protected header plus a key ID.

Key discovery/distribution can be done outside of Notary. Additionally, I believe we have discussed, that users can inspect a signature using Notation to start their discovery process.

Re-opening, as key discovery is a critical component to notary usability. We are currently focusing on the notation inspect notation #238 command to provide the discovery method, for how to find the public key.

Steve - I believe we need to fill in more details on 238 because it is currently vague and not connected to this purpose as it is written.  Key discovery is vital for verification of course and we need to make this easy for users and producers.  Simply getting various metadata back about a signature not necessarily important in general, especially considering we have oras.

The focus here is we at least need to know where the public key is located. Then, users can download the public key, configure their trust stores and have an e2e story.  See https://github.com/notaryproject/notation/issues/172, and perhaps we can either copy the content from there, or re-use that issue for tracking.

I forgot about 172 and I believe this captures the notation flavor/idea intent to implement key discovery.  However - @letmaik's comment above  is worth considering that isn't covered in issue 172.

As discussed in the community meeting, move this issue to future milestone.


Always add Vary header with Origin

After enabling the CORS middleware in Laravel, it was found that the "Vary: Origin" header is always added regardless of whether it is a CORS request or not.
I don't think it is necessary if it is not a CORS request.
https://github.com/fruitcake/php-cors/blob/v1.3.0/src/CorsService.php#L218

The Vary header is there to make sure that the response is only cached for that specific Origin. If it does not have an Origin header, it needs to return a different response than when it has an Origin that is allowed. If you allow all origins (wildcard) it will always add that, and does not need the Vary header. If you have just 1 allowed origin, that's also the case. But for multiple origins,  we need to make sure it does not get cached for different origins, so the header is added always.

More information refers to the comments of PR https://github.com/fruitcake/php-cors/pull/25


[Improve] read flink on k8s deploy log improvement

[Improve] read flink on k8s deploy log improvement
What changes were proposed in this pull request

Removed MoreFeature, added a new class for only one method called , added complexity in my opinion. Needs to be simplified
read flink on k8s deploy log optimization
Removed LogService, LogServiceImpl

Brief change log

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Does this pull request potentially affect one of the following parts

Dependencies (does it add or upgrade a dependency): no

cc @MonsterChenzhuo @1996fanrui  PTAL in your free time, thanks~

cc @MonsterChenzhuo PTAL if your have free time. thanks.

Hi @wolfboys , just to managed the expectation, as I said before: Sorry, I'm not familiar with flink on k8s related code, so I just feedback some technical comments.
My comments are just code specification, I didn't review the core logical of this PR. So I cannot approve this PR after my comments are resolved. It still needs some k8s reviewers.
cc @MonsterChenzhuo

@wolfboys @1996fanrui  When I run gh login, I can't check pr without a github login jump, so I can't test accordingly. I don't know if it's a problem with the local network or with github

@wolfboys @1996fanrui When I run gh login, I can't check pr without a github login jump, so I can't test accordingly. I don't know if it's a problem with the local network or with github

Let me help you, come on

Hi @wolfboys , just to managed the expectation, as I said before: Sorry, I'm not familiar with flink on k8s related code, so I just feedback some technical comments.
My comments are just code specification, I didn't review the core logical of this PR. So I cannot approve this PR after my comments are resolved. It still needs some k8s reviewers.
cc @MonsterChenzhuo

Thanks for your professional opinions. very good 👏👏👏

LGTM


Learn from events for current week

If creating a new event for the week, and call it "A", and there is already event called "A" this week, and it's in the calendar "B", then if there is no autocompletion rule that says which calendar "A" should go into, it should go into the "B" calendar

Should be implemented by fetching data from API, or by scrapping the webpage
If scrapping the webpage, that would make https://github.com/maxpatiiuk/calendar-plus/issues/76 more feasible too


Added TypeScript declaration file

Made these types for my project using this module, sharing is caring!

Would you guys prefer I close this and add the types to definitelyTyped?


 It should work with the server in Pull #3103 with only setting the agent.server value

/assign @mattfarina
As this is related to #3103  I thought it best to assign to the same person.

/assign @timothyclarke

@mattfarina I was requested to implement WebPageTest by my current client. They are happy for project like this to be fed back to the community. As such I created this PR as well as #3103 which starts a webpagetest server with a default config and provides the option to also use Amazon images.
I am not part of the WebPageTest project. I have done minimal hardening by proving an example of locking down the main UI and providing an alternate host header with a restricted uri path. I created a pull request to the WebPageTest project to make use of their existing bootstrapping of the ami instance which forms the core of the docker webpagetest server pod.
The default config allows for the docker (agent) container to connect to the server and run tests.
I choose to push these two pull requests against incubator as they have enough to get a working setup in place. That setup can be entirely k8s by just setting host values in both this chart and the server chart, or optionally it can have more than linux - chrome/firefox by making use of AWS AMI's.

/assign mattfarina

@timothyclarke Once you are done working with this client (will it be soon), who would be able to maintain the chart? You, someone on the webpagetest team, no one?

I'm happy to maintain this going forward. Ideally I'd prefer someone from webpagetest to pick it up for what I hope is obvious reasons.

@timothyclarke please put yourself down as a maintainer. We want that prior to merging.


chore: remove unreachable codes

Codecov Report

Merging #326 (a548670) into main (ec0d628) will decrease coverage by 0.27%.
The diff coverage is 81.81%.

@@            Coverage Diff             @@
##             main     #326      +/-   ##
==========================================
- Coverage   64.08%   63.81%   -0.28%
==========================================
  Files          41       41
  Lines        7328     7168     -160
==========================================
- Hits         4696     4574     -122
+ Misses       2251     2210      -41
- Partials      381      384       +3

Impacted Files
Coverage Δ

pkg/agent/datapath/multiBridgeDatapath.go
59.89% <0.00%> (-0.53%)
:arrow_down:

plugin/tower/pkg/controller/endpoint/controller.go
78.00% <100.00%> (-0.25%)
:arrow_down:

plugin/tower/pkg/controller/global/controller.go
90.40% <100.00%> (+16.27%)
:arrow_up:

plugin/tower/pkg/controller/policy/controller.go
83.09% <100.00%> (-3.18%)
:arrow_down:

pkg/agent/controller/policy/policy_controller.go
78.78% <0.00%> (+0.75%)
:arrow_up:

pkg/controller/group/group_controller.go
70.93% <0.00%> (+2.16%)
:arrow_up:


Exception from JobMonitor

Traceback (most recent call last):
  File "vistrails/gui/vistrails_window.py", line 1946, in method
    getattr(obj, method_name)()
  File "vistrails/gui/vistrail_view.py", line 1084, in execute
    view.execute()
  File "vistrails/gui/pipeline_view.py", line 3603, in execute
    self.controller.execute_user_workflow()
  File "vistrails/gui/vistrail_controller.py", line 451, in execute_user_workflow
    self.jobMonitor.finishWorkflow()
  File "vistrails/core/vistrail/job.py", line 505, in finishWorkflow
    if self.callback:
ReferenceError: weakly-referenced object no longer exists

I don't have the exact steps to reproduce yet, this seems to have something to do with closing workflows.

Ran into this again!

Fixed the ReferenceError with 2e6d0d4c918c89287da88aa175dddf328d00b844. But this should not happen. Are there running jobs missing in the job list? The callback refers to a Running Job widget so I don't think deleting the spreadsheet sheet should make a difference.

Deleting the sheet ensures that the weakly referenced object dies, allowing the GC to make it invalid.
Maybe what you meant is "self.callback is not None"? I don't really know what happens there.

This should be fixed by c882102c75db3bcda0259729694eb269b3983690. The reference error was probably caused by the wrong widget being deleted.

I did a pass on the weakref usage in job.py with dcca2109. But really, I'm not sure why that reference need to be weak at all.

I think the weakref can be removed. The jobs used to be saved at shutdown where the view may have been released but that is not the case anymore.


[#167571906] Update BOSH's UAA URL

What
Updates BOSH's UAA url to the fqdn as it was unable to access UAA when deploying cf.
How to review

Deploy to your dev env

Who can review
Not me

Works in my dev-env