Add TypedMutableMapping and EventedDict

This PR adds two new container classes, namely TypedMutableMapping and EventedDict which are very similar to the list and set containers.
These were implemented as part of #2900  which still requires a bit of work. Since there are others like @alisterburt who might benefit from an EventedDict it makes sense to split it up.
Changelog

added TypedMutableMapping and EventedDict to the available containers
added interface tests

Type of change

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

How has this been tested?

[x] example: the test suite for my feature covers cases x, y, and z
[ ] example: all tests pass with my change

Final checklist:

[x] My PR is the minimum possible work for the desired functionality
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If I included new strings, I have used trans. to make them localizable.
For more information see our translations guide.

@alisterburt Thanks for the PR! I completely forgot to add it there 😄

Oops - i may have just been a little trigger happy here on merging. This PR has 4 tests that failed twice on being re-run. Is there maybe something going on here that needs to be fixed? Could it be a funny interaction between this PR and our tests? It looks like it was a threading test that was failing - cc @tlambert03

@sofroniewn ive tried to reproduce this one failing test on my machine without success..

@sofroniewn ive tried to reproduce this one failing test on my machine without success..

Yeah probably CI only, we'll look into it

Python 3.7 tests seem to be failing in 2 of my active PRs due to eventeddict not being hashable!

yeah I think this is a legitimate python  3.7 bug.  hashing behavior was a bit different then.  @lukasz-migas, if you test locally with 3.7 can you get it?  for example: tox -e py37-macos-pyqt
i think the issue is that, with the nested events, the object itself needs to be hashable.  You might need to add a __hash__ method on the class (but exactly how it should be implemented isn't immediately obvious to me, I'm not sure we want to just use id(obj))

@tlambert03 Indeed I could reproduce it in py37.
Not sure what is the best way to implement the __hash__  method but we could try something like this:
    def __hash__(self):
        return hash(frozenset(self))

which fixes the issue in 3.7 and doesn't break it in 3.8. We could actually make it conditional that the method is only added in py37 and not in 3.8+. Obviosuly above we are only hashing keys rather than keys and values which might not be what we want.
Here is the culprit if anybody is interested: bpo-34751:

We could actually make it conditional that the method is only added in py37 and not in 3.8+. Obviosuly above we are only hashing keys rather than keys and values which might not be what we want.

I don't mind, but if we do add without condition we should add a comment that it should be removed after dropping py37. Thoughts @tlambert03 on how best to proceed?

i like the conditional addition.  If possible, at class construction time though rather than every time __hash__ is called


Added support for submenu for ContextMenuMode.PopupMenu

Thanks a lot for this PR, it should have been done a long time ago. I will try to release it as soon as possible, although I have some problems with free time

Thanks a lot for this PR, it should have been done a long time ago. I will try to release it as soon as possible, although I have some problems with free time due to relocation

@HavenDV No problem, I added a fix today to resolve a last minute refactoring bug

Overall I don't see any issues with your code, it looks good. I'm having some issues running windows based code on mac m1, I'm looking into it, trying to build it with Parallels Studio/Windows 11 arm/VS 2022 and seeing some issues with source generators. After checking the console application I will accept this


an error with the training set direction

Hello, I recently installed this model to train a custom dataset. The environment setup is complete, and I first attempted to use the MOT17 dataset to test whether the training process works properly. However, during the training, I encountered some abnormal data, and I was wondering if you could provide any guidance on how to resolve this issue.
Currently, I have downloaded both the Crowdhuman and MOT17 datasets. However, while training, I noticed that all the loss values are zero, which seems to suggest that the data is not being properly loaded. To check the data loading path, I added the following line of code: print(f"Frame path: {frame_path}"). The result shows that the dataset is loading Crowdhuman, but when I issued the command, I set the dataset to MOT17. I'm not entirely sure where the problem lies—could you kindly take a look? Thank you very much.
Also, just to mention, my computer has only one GPU: an NVIDIA GeForce RTX 3060. Since my GPU is limited, do I need to modify any lambda functions in the code? I appreciate your help.
If there's anything that I didn't explain clearly, please feel free to let me know, and I will provide any additional details you may need.
Thank you again.

Here is the related information regarding the failed training.

Here is the place where I added the code I mentioned earlier.

The data loading process is not related to the GPU. Therefore, I think you should not modified any data loading function on your GPU (3060).
I have never seen this issue before. Have you run the data preparation scripts (like ./data/gen_crowdhuman_gts.py) before running the training script?

Hello, considering that there might have been some mistakes in my previous operations, I deleted the previously cloned model and cloned it again. This time, I only modified the GPU quantity, downloaded the pre-trained weights and datasets, and placed them in the designated locations. I also used the gen_mot17_gts.py and gen_crowdhuman_gts.py files to generate the necessary files.
The training process is now running, but the loss values seem abnormal (most loss values start at 0 during the initial training). I suspect that the images might not have been successfully read by the model for training. Could you kindly advise if I made any mistakes? If the training were running correctly, what would the expected behavior look like? Thank you very much.

To be honest, this is really weird. I need to wait until I have a spare GPU server to show what the correct logging looks like.
You could check the content of the data loaded into the training loop, after these lines:
https://github.com/MCG-NJU/MeMOTR/blob/7de13f4afee57f8d5fcf47567d890af401815174/train_engine.py#L192-L199
You can add code like these:
print(batch["infos"][0])                          # the GTs
print(batch["imgs"][0][0].shape)                  # the image's shape
# Or others

You could analyze it based on the results or upload it here.

Hello, I followed your suggestion and added the print statement. The current output is shown in the image below (Image 1). The prediction result tensor([], dtype=torch.int64) shows 0, and ids, areas, and labels all indicate that no objects were detected. I'm not quite sure why this is happening.
I noticed that in the train_mot17.yaml file, there is a setting "USE_CROWDHUMAN: True". I initially suspected that the issue might be due to training with MOT17 while having Crowdhuman included in the configuration. So, I changed "USE_CROWDHUMAN: True" to false, but this resulted in an error (Image 2).
I also tried some commands related to Submit and Evaluation, but I encountered a small issue. When using eval mode, I got the following error (Image 3), even though I don't have any related files. Could you please advise if this file is supposed to be generated automatically? If so, did I make a mistake somewhere?
Sorry for the multiple questions, and I truly appreciate your help.
Thank you very much.
(Image 1)

(Image 2)

(Image 3)

According to (Image 2), it seems that you did not successfully load any image and annotation from MOT17. You can add some breakpoints during the data loading process to determine where the problem is.
For example, here:
https://github.com/MCG-NJU/MeMOTR/blob/7de13f4afee57f8d5fcf47567d890af401815174/data/mot17.py#L59-L68 all GTs should be loaded.

Here’s a polite translation:
Hello,
I am currently encountering an issue during training where the images are found, but the corresponding bounding boxes are not correctly located. I have verified that the image and text file paths in mot17.py are correct. I have also tried various methods to resolve this issue, and I found that when generating text files using gen_crowdhuman_gts.py, the generated files are missing the v value in the line "i, x, y, w, h, v = map(float, (i, x, y, w, h, v))" during data reading.
Could I ask if you have encountered a similar situation before?
Thank you very much.


TSC implementation

Added a tsc driver implementation tested with U545 and external circutry for touch sensing

@kkoppul2 it would be really helpful if you PR'd an example on how to use if, if you have the time! :pray:

Will do


Overwhelming output while twig syntax error

I'm testing webpack-bundle with clean Symfony 3.2 project. When there is syntax error in *.html.twig, for example missing '}', dashboard is overwritten with error text. Error message is printed every second, it is also included in var/logs/dev.log:
[2016-12-03 23:37:14] app.ERROR: Twig_Error_Syntax: Unclosed "block" in...
[2016-12-03 23:37:15] app.ERROR: Twig_Error_Syntax: Unclosed "block" in...
It is possible to get nice error output with
maba_webpack:
     twig:
         suppress_errors: false

but it doesn't work well with bin/console maba:webpack:dev-server because server is not restarted on error.

Currently maba:webpack:dev-server PHP process starts webpack-dashboard process, which in turn starts webpack-dev-server process.
webpack-dashboard puts everything from stardard output of internal command into left top pane. All other things are taken from socket - server is ran from webpack-dev-server process which gives out statistics about compilation process.
For errors (and log messages in verbose mode) to be shown in that pane it would take to run PHP process from webpack-dashboard. This would require either to run node process in the first place instead of good old app/console (maybe some helper in bin folder is possible) or get something like PHP -> node -> PHP -> node, which is an overkill IMO for displaying log messages inside the console properly.
Furthermore, if PHP is ran from node process, it does not see TTY, which in turn (currently) disables dashboard plugin. Colors would not be available, too, as I remember.
You could test it by setting bin.webpack_server.tty_prefix to [] and running node node_modules/webpack-dashboard/bin/webpack-dashboard.js -- app/console maba:webpack:dev-server.

About printing every second - printing is done by Symfony itself as error is called on logger. I don't think that it would be good idea to extend interfaces so that error would not be logged if same one was logged in previous iteration - it would complicate all the code and logic for not so much gain.
Sleep timeout could be extended in case of an error, but again - it would mean that when the error is fixed, it would take more time to start compiling all over again.


Added new Youtube Channel Resources for Developers

@prisha-srivastava contribute here https://github.com/Ishaan28malik/Get-Dev-Resources


No longer working since Today

Yesterday the tesla API was raising 500 / 503 errors. Today things seem running healthy except that since then the works demo isn't showing the values anymore after logging in.
Other methods like https://tesla-info.com/tesla-token.php also are no longer working.
I'm using 2FA

Other methods like https://tesla-info.com/tesla-token.php also are no longer working.

Thanks, I see it too. Error from auth2/v3/token call is:
{"error":"invalid_request","error_description":"Invalid code_verifier","error_uri":"https://auth.tesla.com/error/reference/11d1efa0-894f-4782-99d2-f9b325ee8eed-1676455831381"}

Not sure what changed or how to fix it.

@tomhollander thanks for validating.

As per https://tesla-api.timdorr.com/api-basics/authentication, code_verifier was a random string. Looks like the requirements have somehow changed.
Also unfortunately this isn't just impacting 2FA users - seems to be broken for everyone.

Yes, for my app it is also broken :(
Using my app login with the refresh token is still working, but no app can create this token right now...

Seems to work again here:  https://tesla-info.com/tesla-token.php

Seems to work again here: tesla-info.com/tesla-token.php

@Urkman Yeah, that seems to work again but TeslaAuth isn't showing the values. I don't have time ATM to analyze this.

There's some discussion on this at https://github.com/timdorr/tesla-api/discussions/689. If/when we figure out the solution I'll try to get an updated library ASAP (PRs welcome if anyone beats me to it).

I don't have a solution, but I have some things that didn't work with the TeslaAuthHelper class.  I'm going to post this here because it was specific attempts at improving this code base, and also didn't solve the problem.

Changing from 86 characters in the code verifier to 43 or 64.
Limiting the code verifier to only lowercase alphabetic characters
URL encoding the Code Challenge.
Changing the State length from 20 to 16 characters, per some other working implementations.
Adding ou_code to the state requested for the token, like the Tesla web site does (although it's using /oauth2/v1/authorize, not v3).
Adding locale=en-US

@brianflex
This is how I fixed it for my implementation.
code_verifier = <alphanumeric random string 86 characters long>
create the code_challenge like so in Python:
hashed = hashlib.sha256(code_verifier.encode()).digest()
encoded = base64.urlsafe_b64encode(hashed)
code_challenge = encoded.decode('ascii')[:-1]  # remove the last = that is guaranteed to be there.

The change for me was getting the digest from the sha256 hash instead of the hexdigest and removing the last "=".

Sounds like we have some leads now. I won't have time to look at this for the next 12 hours, but if anyone can raise a PR I can merge and publish to NuGet. Or I can try to fix tonight.

Thanks @mavese I got a fix.  Your hint about the digest vs. hexdigest commands sent me on a weird tangent on investigating Python's crypto library, but in the end I realized some of our problems could be coming from a sloppy comingling of bytes & chars, with extra conversions potentially in a way that depends on machine state.  I threw out what we were doing and rewrote it to avoid the excess conversions, and it solved the problem.
We did also have to add in the URL safe encoding handling for a few special characters.    The [:-1] trick you are using seems like an incomplete workaround because you'll miss other characters.  Check this out.
'''
byte[] code_challenge_SHA256 = ComputeSHA256HashInBytes(loginInfo.CodeVerifier);
loginInfo.CodeChallenge = Base64UrlEncode(code_challenge_SHA256);
'''
Then the Base64UrlEncode method:
'''
public static string Base64UrlEncode(byte[] bytes)
{
String base64 = Convert.ToBase64String(bytes);
String encoded = base64
.Replace('+', '-')
.Replace('/', '_')
.Replace("=", String.Empty)
.Trim();
return encoded;
}
'''
Note: You might want to look at what encode does, and whether it should be done before or after hashing.

Thanks so much @brianflex! PR merged, and now published to Nuget as v2.3.0.


Fix space app admin panel

🛠 Changelog

Fixes admin panel crashing when loading an individual space.
Refactors the spaces admin panel to individual files,
Removes unused models from admin panel (i.e. Building, RealEstate, ...)

🧪 Test plan

Automated tests

🚧 Dependencies

None

🎫 Tickets

None

Merged https://github.com/City-of-Helsinki/tilavarauspalvelu-core/pull/1256 to this

/fast-forward


Update sbt-typelevel, sbt-typelevel-site to 0.5.0

About this PR
📦 Updates

org.typelevel:sbt-typelevel
org.typelevel:sbt-typelevel-site

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

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

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.typelevel" }
}]

labels: sbt-plugin-update, early-semver-major, semver-spec-minor, version-scheme:early-semver, commit-count:1

Superseded by #70.


Odd modal h1 margin

_modal.scss line 110 is odd. I'm not sure what was intended.
margin-bottom: 0 0 0.6em 0;

Thanks @tom-- this is now fixed!


Map.Entry class can not complete!

Map.Entry can not complete method!
Map. ===> Map.class
Map.Entry.===> no result

Yeah, nested classes still not implemented. It requires some time.

there's no hurry about that, Your project has done very well.

https://github.com/Valloric/ycmd/issues/45#issuecomment-138011085

I added basic support for nested classes, please, can you try?

ok  let me have a look

The static import not yet supported.
Second and third errors now should be fixed.

great it works well

#116


Support Tensor creation from lists and nested lists without numpy

This is a draft for the second task #2649.
I'm trying to understand the code base; my apologies if this is completely missing the mark.
I'm not trying to lock a bounty, but rather understand by contribute.
This can be extended, but would get some input whether this goes into the correct direction.
This goes via a Python native route when creating tensors from lists. It doesn't remove any support for the creation from np arrays.
Approach:
We can create the pack the data via struct.pack and create a flat, (mutable) memory view on it (mutability required by https://github.com/tinygrad/tinygrad/blob/26f49869f47841817f066aa3357fd9d6aed4f32c/tinygrad/helpers.py#L294)
There's currently no support for bfloat16 (as not exposed in struct.pack), but this could maybe be implemented manually or using a library (as only serialization / deserialization is needed).
The way it is currently implemented is that it walks down the (potentially) nested input and gathers all elements in a flat list (and check that the shape is homogeneous). This is then serialized with struct.pack_into a bytesarray.
One could pack and walk at the same time, but this would assume that we know the dtype beforehand. This is currently the case, but you might want to support inferring the dtype (as numpy does) as well in case no dtype is specified.
Inferring requires looking at all data beforehand. So we could do a second walk there instead of collecting it in a flat list.
Are there any preferences?
The NumpyAllocator (used in the CPU device) assumes an np.ndarray as the buffer, so there's a np.frombuffer view around it in fromNative in lazy.py to avoid a copy.
Considered alternatives:

Use array.array. But that doesn't support float16 and bfloat16.

closing as stale


Fail to build Gitbook after recent update

Hi
After reinstalling v1.1.0 I get an error when running gitbook build
info: loading plugin "intopic-toc"... ERROR

PluginError: Error with plugin "intopic-toc": Cannot find module './dist/index.js'

Should be fixed with release 1.1.1
Please try again using the new version and feel free to open a new issue if any other problems occur.

Works like a charm. Thank you for the very quick fix


@keyframes not prefixed in chrome and not working

will be fixed in https://github.com/jsstyles/jss-vendor-prefixer/issues/1

@kof is there an example of using @keyframes animation?
I can't make it work (it renders as [object Object] instead of keyframe rule), so a working example would be helpful.


reduce cdc latency

Development Task
Cdc latency is determined by several factors. On the TiKV side the key is cdc.min-ts-interval, which is 1s by default. If we can set it to a small value, it will be helpful to reduce cdc latency.
However some issues should be resolved:

whether it will utilize too much CPU resource or not; tso thread count can be a bottleneck;
whether it will utilize too much CPU resource on the CDC side or not.

Test shows that when redo-log is enabled, cdc latency increases to infinit. I think some there are dead locks in cdc. Here is a log sector:
[2022/06/16 17:39:23.944 +08:00] [WARN] [table_actor.go:408] ["send fails"] [msg="{\"Tp\":3,\"Command\":null,\"PolymorphicEvent\":null,\"BarrierTs\":433945860749918508}"] [tableName=`test`.`sbtest13`] [tableID=78] [error="[CDC:ErrMailboxFull]mailbox is full, please try again. Internal use only, report a bug if seen externally"]
[2022/06/16 17:39:23.944 +08:00] [WARN] [table_actor.go:408] ["send fails"] [msg="{\"Tp\":3,\"Command\":null,\"PolymorphicEvent\":null,\"BarrierTs\":433945860749918508}"] [tableName=`test`.`sbtest1`] [tableID=89] [error="[CDC:ErrMailboxFull]mailbox is full, please try again. Internal use only, report a bug if seen externally"]
[2022/06/16 17:39:23.944 +08:00] [WARN] [table_actor.go:408] ["send fails"] [msg="{\"Tp\":3,\"Command\":null,\"PolymorphicEvent\":null,\"BarrierTs\":433945860749918508}"] [tableName=`test`.`sbtest12`] [tableID=99] [error="[CDC:ErrMailboxFull]mailbox is full, please try again. Internal use only, report a bug if seen externally"]
[2022/06/16 17:39:23.944 +08:00] [WARN] [table_actor.go:408] ["send fails"] [msg="{\"Tp\":3,\"Command\":null,\"PolymorphicEvent\":null,\"BarrierTs\":433945860749918508}"] [tableName=`test`.`sbtest15`] [tableID=71] [error="[CDC:ErrMailboxFull]mailbox is full, please try again. Internal use only, report a bug if seen externally"]
[2022/06/16 17:39:23.944 +08:00] [WARN] [table_actor.go:408] ["send fails"] [msg="{\"Tp\":3,\"Command\":null,\"PolymorphicEvent\":null,\"BarrierTs\":433945860749918508}"] [tableName=`test`.`sbtest7`] [tableID=93] [error="[CDC:ErrMailboxFull]mailbox is full, please try again. Internal use only, report a bug if seen externally"]
[2022/06/16 17:39:23.944 +08:00] [WARN] [table_actor.go:408] ["send fails"] [msg="{\"Tp\":3,\"Command\":null,\"PolymorphicEvent\":null,\"BarrierTs\":433945860749918508}"] [tableName=`test`.`sbtest2`] [tableID=97] [error="[CDC:ErrMailboxFull]mailbox is full, please try again. Internal use only, report a bug if seen externally"]
[2022/06/16 17:39:23.944 +08:00] [WARN] [table_actor.go:408] ["send fails"] [msg="{\"Tp\":3,\"Command\":null,\"PolymorphicEvent\":null,\"BarrierTs\":433945860749918508}"] [tableName=`test`.`sbtest3`] [tableID=72] [error="[CDC:ErrMailboxFull]mailbox is full, please try again. Internal use only, report a bug if seen externally"]
[2022/06/16 17:39:23.944 +08:00] [WARN] [table_actor.go:408] ["send fails"] [msg="{\"Tp\":3,\"Command\":null,\"PolymorphicEvent\":null,\"BarrierTs\":433945860749918508}"] [tableName=`test`.`sbtest14`] [tableID=74] [error="[CDC:ErrMailboxFull]mailbox is full, please try again. Internal use only, report a bug if seen externally"]
[2022/06/16 17:39:23.944 +08:00] [WARN] [table_actor.go:408] ["send fails"] [msg="{\"Tp\":3,\"Command\":null,\"PolymorphicEvent\":null,\"BarrierTs\":433945860749918508}"] [tableName=`test`.`sbtest8`] [tableID=95] [error="[CDC:ErrMailboxFull]mailbox is full, please try again. Internal use only, report a bug if seen externally"]
[2022/06/16 17:39:23.944 +08:00] [WARN] [table_actor.go:408] ["send fails"] [msg="{\"Tp\":3,\"Command\":null,\"PolymorphicEvent\":null,\"BarrierTs\":433945860749918508}"] [tableName=`test`.`sbtest9`] [tableID=73] [error="[CDC:ErrMailboxFull]mailbox is full, please try again. Internal use only, report a bug if seen externally"]

Same problem even if set cdc.min-ts-interval to 1s.


Add Jisho.org dictionary on Raw Manga(Japanese)

Describe your suggested feature
For instance, when reading raw manga(purely Japanese) there would be a tool that you can use to search for certain Japanese words this would help a learner of Japanese to quickly search, and thus, studying Japanese while reading manga would be a great help.
Other details
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I will fill out all of the requested information in this form.

i don't think i will add this feature anytime soon but you can check out https://github.com/Rattlehead15/tachiyomiOCR if you don't know it already.


Wrong applicationID

FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:processDebugGoogleServices'.

No matching client found for package name 'com.example.firebase_user_avatar_flutter'

You need replace in android/app/build.gradle :
line 41 :
    applicationId "com.example.firebase_user_avatar_flutter"

by
applicationId "com.codingwithflutter.firebase_user_avatar_flutter"

Thanks for reporting this.
It's actually the other way around. Both the build.gradle and AndroidManifest.xml use com.example.firebase_user_avatar_flutter as the package name.
I have updated this on the README.

Thanks you


Problem with the bottom navigation bar

Steps to reproduce
Create a page with a botton navigation bar and add a few pages to the body
Expected results
Be able to navigate between pages
Actual results
I have error
Code sample
Code sample
  return Scaffold(
      appBar: AppBar(
        title: const Text('Caisse'),
      ),
      body: IndexedStack(
        index: currentIndex,
        children: pages,
      ),
      bottomNavigationBar: NavigationBar(
          selectedIndex: currentIndex,
          onDestinationSelected: (value) {
            setState(() {
              currentIndex = value;
            });
          },
          destinations: const [
            NavigationDestination(icon: Icon(Icons.home), label: 'Accueil'),
            NavigationDestination(
                icon: Icon(Icons.line_axis), label: 'Depense'),
            NavigationDestination(
                icon: Icon(Icons.show_chart), label: 'Statistique'),
            NavigationDestination(
                icon: Icon(Icons.show_chart), label: 'Statistique')
          ]),
    );

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
════════ Exception caught by widgets library ═══════════════════════════════════
The following assertion was thrown building _ScrollNotificationObserverScope:
'package:flutter/src/widgets/framework.dart': Failed assertion: line 5207 pos 16: 'owner!._debugCurrentBuildTarget == this': is not true.

Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.22.2, on Microsoft Windows [version 10.0.22631.3958], locale fr-FR)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
[√] Chrome - develop for the web
[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.10.4)
[√] Android Studio (version 2024.1)
[√] VS Code (version 1.92.1)
[√] Connected device (3 available)
    ! Device 08977372CN007404 is offline.
[√] Network resources

• No issues found!

Could not reproduce it on My device, could you please provide more information?
Here is my code snippets.
class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key});

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int currentIndex = 0;

  @override
  Widget build(BuildContext context) {
    final List<Widget> pages = [
      const Text('Accueil'),
      const Text('Depense'),
      const Text('Statistique'),
      const Text('Statistique'),
    ];
    return Scaffold(
      appBar: AppBar(
        title: const Text('Caisse'),
      ),
      body: IndexedStack(
        index: currentIndex,
        children: pages,
      ),
      bottomNavigationBar: NavigationBar(
          selectedIndex: currentIndex,
          onDestinationSelected: (value) {
            setState(() {
              currentIndex = value;
            });
          },
          destinations: const [
            NavigationDestination(icon: Icon(Icons.home), label: 'Accueil'),
            NavigationDestination(
                icon: Icon(Icons.line_axis), label: 'Depense'),
            NavigationDestination(
                icon: Icon(Icons.show_chart), label: 'Statistique'),
            NavigationDestination(
                icon: Icon(Icons.show_chart), label: 'Statistique')
          ]),
    );
  }
}

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, 3.24.0-1.0.pre.440, on macOS 15.1 24B5009l darwin-arm64, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2024.1)
[✓] VS Code (version 1.92.0)
[✓] Connected device (4 available)
[✓] Network resources


scrcpy not working on windows 10

Hello,
i have Win 10 64 bit and downloaded newest scrcpy.
I added SCRCPY_SERVER PATH and ADB to Path Environment Variable but i get the following Error:
PS C:\Program Files\ScrCpy> scrcpy
INFO: scrcpy 1.10 <https://github.com/Genymobile/scrcpy>
ERROR: Command not found: [C:\Program Files\ScrCpy], [push], ["C:\Program Files\ScrCpy"], ["/data/local/tmp/scrcpy-server.jar"]
ERROR: (make 'adb' accessible from your PATH or define its fullpath in the ADB environment variable)
ERROR: Could not execute "adb push"
Press any key to continue...

Did i forget something?

I added SCRCPY_SERVER PATH and ADB to Path Environment Variable

You should not set ADB if you use the one from the scrcpy folder.
If you really want to set it, it must point to the adb executable, not its folder: https://github.com/Genymobile/scrcpy/issues/769#issuecomment-533495032

thank you! got it working again!


Use time.Time instead of uint64 for MempoolTransaction

Current dependencies on/for this PR:

feature/proof-of-stake

PR #1042   👈

This stack of pull requests is managed by Graphite.


jets new throws error

I follow the installation requirements and perform jets new demo and receive the following error:

.rbenv/versions/2.5.0/lib/ruby/gems/2.5.0/gems/aws-sdk-core-3.30.0/lib/aws-sdk-core/plugins/regional_endpoint.rb:42:in `after_initialize': missing region; use :region option or export region name to ENV['AWS_REGION'] (Aws::Errors::MissingRegionError)

Guessing that it has to do with the ~/.aws/config region not being set and Jets assumes it does right now. Jets should report a more friendly error. Traveling right now but will take a look soon.


[Parser] Correct the fixit for protocol composition's syntax change

When migrating, we found our fixit to replace the old protocol composition syntax, namely Protocol<A, B>,  to the new syntax, A & B, does not preserve the trailing content after >. For instance, we replace Protocol<A, B>? with A & B. This patch fixes the issue by inserting whatever after > in the old syntax to the replacement string. I consider this as a hack; the root-cause fix should be in the lexer to smartly separate > and ? as two tokens instead of one.

Resolved bug number: (SR-)

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

A smoke test on macOS does the following:

Builds the compiler incrementally.
Builds the standard library only for macOS. Simulator standard libraries and
device standard libraries are not built.
lldb is not built.
The test and validation-test targets are run only for macOS. The optimized
version of these tests are not run.

A smoke test on Linux does the following:

Builds the compiler incrementally.
Builds the standard library incrementally.
lldb is built incrementally.
The swift test and validation-test targets are run. The optimized version of these
tests are not run.
lldb is tested.

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci smoke test

@benlangmuir could you review this?

@swift-ci smoke test

LGTM.

@swift-ci smoke test


Missing Operation? Unlatch

Hi, thanks for your effort creating this API and the home assistant Integration. I have a yale linus L2 (Europe).
With your home assistant Integration I can easily create an automation to e.g. automatically lock the door in the evening and unlock it in the morning.
However, I'm not able to open the door by unlatching the lock.
The yale android app has the three operations lock, unlock, unlatch.

I've just tested it. By replacing lock/unlock with unlatch, the lock unlatches and the door opens.
E.g.: /remoteoperate/{ID}/unlatch?v=2.3.1&type=async&connection=persistent

I've created a PR https://github.com/bdraco/yalexs/pull/101

Merged https://github.com/bdraco/yalexs/pull/101


Support mock trading？

Hi, this is a pretty good project.
As I am plan to do live trading, would like to run a strategy in simulation way.
Do you have a plan?

Yep that's fully supported. In fact, there is little difference between the live trading and paper-trading in this framework. You may simply change the mode in the configuration to paper-trading to get started, as mentioned in the doc.
To ensure we are on the same page -- it is the QMT that handles order processing and account management, so you will need to install 2 QMTs on your windows server, one for live trading and the other for paper trading

That's very nice !
A little confused about the 2 QMTs,

Since simulation doesn't need perfrom buy/sell actually, why need QMT ?
Or do you mean deploy two TradePy servers ?

Hmmmm I am not sure whether we are fully synced on the terminologies here 😂 In the general sense, there are 3 modes:

backtest: No QMT involved, just re-playing the history stock data and apply a strategy to decide buying & selling (is this the "simulation" you talked about?)
live-trading: Need QMT to receive orders. The doc has a section for guidance on how to set up a live trading environment.
paper-trading: 99% like the live trading, except that the QMT client you are running should be one that supports the paper-trading / mock-trading, i.e., it still process orders as if it is the live-trading, but using your paper-trading account.

You should have two QMT accounts and client side software, one for live trading and the other for paper trading.

Oh, I undertood. Since I didn't seen QMT client have paper-trading / mock-trading .. thanks for instruction.
By the way, how do you deploy QMT on Mac ? I tried Parallels Desktop with Win 11 ARM version,  not able to install QMT client ..

Ah I haven't tried deploying QMT on Mac before 👀 not sure whether QMT actually supports the ARM platform...

Yeah maybe Arm is the problem, my securities manager say that they can detect virtual machines though I don't beleve it😂


Quiz exercise: Refactor Quiz Exercise Service

Checklist
General

[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] I followed the coding and design guidelines.
[x] I documented the Java code using JavaDoc style.

Motivation and Context

The implementation of Quiz Pool API will reuse the functionalities of Quiz Exercise API. Therefore, QuizExerciseService is refactored so the functionality can be reused later by Quiz Pool.
Description

Create QuizService class
QuizService is an abstract class that contains the common methods of QuizExerciseService and QuizPoolService (will be implemented on separate PR).
Move the save, saveCorrectMappingsInIndices, saveCorrectMappingsInIndicesShortAnswer, restoreCorrectMappingsFromIndices, and restoreCorrectMappingsFromIndicesShortAnswer methods to QuizService.
The methods, which are used by QuizExerciseService, will be used by the QuizPoolService as well.
Create QuizConfiguration Interface
QuizConfiguration is an abstraction that contains the common operation of QuizExercise and QuizPool (will be implemented on separate PR)
Move the reconnectJSONIgnoreAttributes method to QuizConfiguration
The reconnectJSONIgnoreAttributes method, which is used by QuizExercise, will be used by the QuizPool as well.
Refactor QuizConfiguration.reconnectJSONIgnoreAttributes and QuizService.save method

Steps for Testing

Prerequisites:

1 Instructor
1 Course

Log in to Artemis
Navigate to Course Administration
Select the Course
Navigate to Exercises
Create a new Quiz Exercise
Verify that the Quiz Exercise can be created successfully.
Edit the newly created Quiz Exercise
Verify that the Quiz Exercise can be updated successfully.

Review Progress

Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Test Coverage

Class/File
Branch
Line

QuizConfiguration.java
78%
100%

QuizExercise.java
73%
91%

QuizExerciseService.java
92%
100%

QuizService.java
82%
94%

AnswerCounter.java
%
%

AnswerOption.java
%
%

DragAndDropMapping.java
%
%

DragAndDropQuestion.java
%
%

DragItem.java
%
%

DropLocation.java
%
%

DropLocationCounter.java
%
%

MultipleChoiceQuestion.java
%
%

QuizQuestion.java
%
%

QuizQuestionStatistic.java
%
%

QuizQuestionStatisticComponent.java
%
%

ShortAnswerMapping.java
%
%

ShortAnswerQuestion.java
%
%

ShortAnswerSolution.java
%
%

ShortAnswerSpot.java
%
%

ShortAnswerSpotCounter.java
%
%

Screenshots

https://user-images.githubusercontent.com/10885503/230736567-c00948a3-d60f-4a82-af8a-fa79f23c7643.mov

Codecov Report

Merging #6426 (d67f0e2) into develop (0d1ae45) will increase coverage by 0.00%.
The diff coverage is 88.64%.

:exclamation: Current head d67f0e2 differs from pull request most recent head 422571f. Consider uploading reports for the commit 422571f to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #6426   +/-   ##
==========================================
  Coverage      80.46%   80.46%
- Complexity     13033    13047   +14
==========================================
  Files           2323     2325    +2
  Lines          89243    89229   -14
  Branches       13190    13152   -38
==========================================
- Hits           71805    71801    -4
  Misses          9610     9610
+ Partials        7828     7818   -10

Impacted Files
Coverage Δ

.../tum/in/www1/artemis/domain/quiz/QuizQuestion.java
91.48% <ø> (ø)

...va/de/tum/in/www1/artemis/service/QuizService.java
83.62% <83.62%> (ø)

.../tum/in/www1/artemis/domain/quiz/QuizExercise.java
78.57% <84.61%> (+0.02%)
:arrow_up:

...in/www1/artemis/domain/quiz/QuizConfiguration.java
89.18% <89.18%> (ø)

...m/in/www1/artemis/service/QuizExerciseService.java
97.82% <96.55%> (+7.78%)
:arrow_up:

...tum/in/www1/artemis/domain/quiz/AnswerCounter.java
100.00% <100.00%> (ø)

.../tum/in/www1/artemis/domain/quiz/AnswerOption.java
93.54% <100.00%> (ø)

...n/www1/artemis/domain/quiz/DragAndDropMapping.java
90.00% <100.00%> (ø)

.../www1/artemis/domain/quiz/DragAndDropQuestion.java
70.50% <100.00%> (+0.43%)
:arrow_up:

...a/de/tum/in/www1/artemis/domain/quiz/DragItem.java
87.50% <100.00%> (ø)

... and 9 more

... and 1 file with indirect coverage changes

Flag
Coverage Δ

client
76.48% <ø> (+<0.01%)
:arrow_up:

server
84.92% <88.64%> (+0.01%)
:arrow_up:

Continue to review full report in Codecov by Sentry.

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


chore: update copyright year

Chore

[x] Update copyright year to 2022

Changes made

[x] Updated copyright in LICENSE.md from 2021 to 2021-2022 based on how I saw Nodejs & Angular's MIT License. (screenshots below)
[x] Updated copyright in apps/showcase/src/app/app.component.html from 2021 to 2022 based on how I saw it on This Dot's website. (screenshot below)

If the format I used is incorrect, I can update this PR or if this update isn't needed I can close this PR.
Thank you! 😄
Nodejs MIT License

Angular MIT License

This Dot's copyright on website

Thank you! 😄


Need to limit resources used during deployments.

We seem to have reached a limit with Private Cloud, where during a deployment of cfprojects the Controllers are getting overloaded. This is causes scheduling issues as well as DNS error rates to increase significantly.
Looking through CloudWatch metrics the controller seems to be able to handle:
2200 requests per min
Normal operation appears to be about
  700  requests per min
Kit appears to be polling every 3 secs for each namespace. The plan is to increase this rate to slow down a number of requests. We also need to add metrics for each request type to data-dog.

Add ENV for defining polling rate. Increase to 10sec
Add DD metrics for API requests
Benchmark deployments of cfprojects vs other services for api request rates

ELB load for API controller:

https://console.aws.amazon.com/ec2/v2/home?region=us-east-1#LoadBalancers:search=private-cluster-public-api-elb

@c1freitas If these are only spiking during cfprojects deployments, maybe we just need to target our optimizations around services that have fast rollbacks enabled? What do you think?

That might be nice. I would have to check with @chesleybrown to see about targeting a specific type. Maybe we could pass in the polling delay? It would be good to see metrics for the various types of deployments.

In looking at the timing, it appears the spikes happen for each deployment, not just cfprojects.
Metrics around the number of API calls will be important in trying to track down where the increased load is coming from (kit vs k8s)

Hey Ches, a little extra context - this evening during a deployment Juan noticed that the controllers were maxed out. We quickly paused deployments so we could investigate, resulting in this ticket.
I'd like for us to do some research to see if it really is the kit traffic causing this, and then tackle this and get back to a stable state as quickly as possible. Thanks!

The good news is we do already have metrics on the number of API request that kit-server makes and also by service.
https://app.datadoghq.com/dash/132876/deploys?live=true&page=0&is_auto=false&from_ts=1503316154238&to_ts=1503488954238&tile_size=s&tpl_var_service=*

It does seem that cfprojects and cfprojects-api consistently hit around 400 requests during a deploy. But I also see that surface has hit 400 too. Generally the number of requests will be determined by how long the deploy takes and how many clusters it's being deployed to.
I think the best and quickest solution right now is to increase the polling. I'll work on making it a configurable ENV var now and I'll default it to 10 seconds.

It's worth noting that fast-rollbacks vs rolling-update should have negligible differences. We make a couple of extra requests before the deployment starts and after the deployment is done when using fast-rollbacks. That should be the only real difference between the strategies. They both would have the same amount of polling requests which is what makes up the 99% of API requests made by kit-server.

This has been deployed. We have noticed a decrease in API requests after increasing the polling to 10. We've also added a monitor to notify us if things exceed our limit.
Going to close this.


Using patrol on Firebase Test Lab

This package looks very promising and easy to use - thanks for open sourcing it!
Are there plans to make patrol compatible with running integration tests on Firebase Test Lab?
If I understand correctly, patrol can be used on local machines by running patrol drive, but I haven't found any instructions on whether it's possible to use patrol on CI/CD as well. I'm thinking of something similar to the integration_test package which makes it possible to run integration tests on CI/CD with Firebase Test Lab (see the following instructions: https://docs.flutter.dev/testing/integration-tests#testing-on-firebase-test-lab).

Hi @Giuspepe! Congrats on opening the first external issue in this repository! 🥳

Are there plans to make Patrol compatible with running integration tests on Firebase Test Lab?

We definitely want to make Patrol run on as many CI providers as possible. I don't have much experience using Firebase Test Lab, so I quickly skimmed through its docs and from what I see, they let you upload the APK of the app under test (app-debug.apk) and the instrumentation APK (app-debug-androidTest.apk). Unfortunately it looks like they don't offer a possibility to run an executable, like patrol, so it's definitely gonna require some research.
Until now, we've made Patrol run integration tests on GitHub Actions. Check out the run-integration-tests.yaml workflow to see an example of this.

Hello,
@bartekpacia could you give some hint how did you integrate patrol tests on your CI?

What CI are you using?
Are you using real devices or simulators to run tests?
How do you handle failed tests?

Hi @pawlowskim,

What CI are you using?

GitHub Actions. You can see the workflow files in the .github/workflows folder in this repository.

Are you using real devices or simulators to run tests?

Currently, we only use Android emulators.
We want to run tests on iOS simulators and physical iOS devices as well, but that'd require us to use some device farm. We didn't have time for this yet.

How do you handle failed tests?

Short answer: we don't. We run a single test and if it fails, patrol exits with a non-zero exit code, which fails the job, and makes the workflow run red. It's a very, very basic form of error reporting, I admit.
I can assure you though that will focus on this area in the near future :)
PS Another problem we're facing in GitHub Actions is the flakiness of connecting to the VM service running on the device. See #272 to learn more.

I would like to understand why we need to call patrol drive and why this is not a standard flutter integration test.
I assume that this is because of the AutomatorServer that needs to be started, but I would love to get a more detailed explanation of the problem 🤔
Maybe you can guide me to some of the files that are important to fully understand this problem.
We are working with firebase test lab and I might be able to give some input on the problem and find a potential fix 👍🏽

I would like to understand why we need to call patrol drive and why this is not a standard flutter integration test.
I assume that this is because of the AutomatorServer that needs to be started

Exactly. We need to run the native automation server, which enables interaction with native UI, and there's no way to do this by using only flutter test.

@bartekpacia
I just looked into the sources and I was wondering if we can solve this issue by starting the AutomationServer in a custom test driver that is registered on the native side.
For example:
/*imports*/
-@RunWith(FlutterTestRunner.class)
+@RunWith(PatrolTestRunner.class)
public class MainActivityTest {
    @Rule
    public ActivityTestRule<MainActivity> rule = new ActivityTestRule<>(MainActivity.class, true, false);
}

This might be a naiv impl for starting the server.
class PatrolTestRunner extends FlutterTestRunner {

    public PatrolTestRunner(Class<?> testClass) {
        super(testClass);
    }

    @Override
    public void run(RunNotifier notifier) {
        startPatrolServer();
        super.run(notifier);
        stopPatrolServer();
    }
}

We can also move this into a Github discussion if you like. Or we can have a quick chat/call 👍🏽

@JulianBissekkou This looks very interesting – thank you! I'll share this with the team and get back to you.
I think there's no need for a separate Discussion – this issue is the right place for this.

On iOS this might be a little tricky because we can't just subclass FLTIntegrationTestRunner.
(Maybe I am missing something)
I created an Issue in the flutter repository: https://github.com/flutter/flutter/issues/113519

@JulianBissekkou, we're working on this in #661 :)

I'm happy to say that basic support is almost ready 🎉 All that's left is writing good docs (and maybe squashing some bugs) :)
See also this.

@bartekpacia  very exciting

Look no further! Here are the docs!
Please report issues in case there are problems :) But Firebase Test Lab is now officially supported.

Really cool! Good job guys 💪🏽
Is the AutomationServer now started in PATROL_INTEGRATION_TEST_IOS_RUNNER  and PatrolTestRule ?

Yes!

Just got the time to integrate patrol into my app. When running the tests from the IDE we still have to use patrol drive, right?
By default the IDEs will execute test using flutter test...

Oops, forgot to respond because :) Sorry!

When running the tests from the IDE we still have to use patrol drive, right?

Currently, you can use patrol drive (to run the tests with flutter_driver), or you can use the new patrol test run the tests natively: with Gradle on Android and Xcode/xcodebuild on iOS. But patrol drive is deprecated and we plan to remove it in 1.0.

By default the IDEs will execute test using flutter test. This will also be fixed in https://github.com/leancodepl/patrol/issues/661, right?

flutter test by default runs widget/unit tests under test/, so it shouldn't be a problem.
But flutter test integration_test won't work with Patrol's native automation feature. You have to use patrol test.


TypeError in no-unused-styles rule

When I'm working in Atom with the linter-eslint plugin, I often encounter a TypeError that originates in the no-used-styles rule while I'm typing.  I have linter-eslint configured to lint as I type.
I have found that the code that triggers the error contains the fragment <View style>.  So, I'm in the process of adding style information, but before I can add the = {}, I get the type error.
I'm using eslint-plugin-react-native 2.0.0.
Here's the stack trace:
TypeError: Cannot read property 'type' of null
    at Object.astHelpers.hasArrayOfStyleReferences (/path/to/myProject/node_modules/eslint-plugin-react-native/lib/util/stylesheet.js:312:11)
    at Object.astHelpers.collectStyleReferences (/path/to/myProject/node_modules/eslint-plugin-react-native/lib/util/stylesheet.js:157:20)
    at EventEmitter.JSXAttribute (/path/to/myProject/node_modules/eslint-plugin-react-native/lib/rules/no-unused-styles.js:46:35)
    at emitOne (events.js:95:20)
    at EventEmitter.emit (events.js:182:7)
    at NodeEventGenerator.enterNode (/path/to/myProject/node_modules/eslint/lib/util/node-event-generator.js:40:22)
    at CodePathAnalyzer.enterNode (/path/to/myProject/node_modules/eslint/lib/code-path-analysis/code-path-analyzer.js:608:23)
    at CommentEventGenerator.enterNode (/path/to/myProject/node_modules/eslint/lib/util/comment-event-generator.js:97:23)
    at Controller.enter (/path/to/myProject/node_modules/eslint/lib/eslint.js:895:36)
    at Controller.__execute (/path/to/myProject/node_modules/eslint/node_modules/estraverse/estraverse.js:397:31)

Thanks for your detailed explanation, this might be addressed by https://github.com/Intellicode/eslint-plugin-react-native/pull/59, I hope to have a fix for this soon!

Sounds good, thanks!

This appears to have been fixed in 2.2.0 by #59.  But see #109 for a similar error that now appears in the no-inline-styles rule.


Following gettings started steps in README gives me to "NO SSH connection to target" error

I've tried having a working Meterpreter reverse_tcp shell open in parallel to Backdoorme too, but it doesn't help neither.
So currently this program is unusable, sadly (or readme is incomplete).
Thanks!

The readme is only about getting Backdoorme ready. It still requires a target to backdoor, without it, no connection can be made.

Yes: I said I have a linux/x64/meterpreter/reverse_tcp session open and working.

Just having a meterpreter session isn't enough - does the target have SSH open?
It'd be an interesting next step to have this working over meterpreter.

Um, no, it hadn't SSH open. Ok, then.
Thanks!


WIP: For issue #97 find_packages finds namespace packages.

I don't think this approach is suitable. It will have too many false positives, creating packages for docs and scripts and examples in many projects. Furthermore, #915 provides another proposal under consideration.

I wrote a note in #97 about checking this assumption about false positives(I guess there's already a tool for checking setuptools changes on existing packages?). I think the false positives can be avoided in a similar manner to how I avoided site-packages folders in this patch. I also think it's worth weighing up any potential breakage over the breakage the bug is already causing to valid python packages that fail to be packaged.


Programming Languages - Added "Java, JavaScript and Python Design Patterns" resources

Description
Hi @jfmartinz  I have added Resources of Design Patterns for Java, JavaScript and Python in Programming Languages Directory
please take a look and review it, Thank You!!
Type of change

[x] Adding a new Resource
[x] Adding a new Category
[x] Documentation enhancement or fixes

Checklist:

[x] I read carefully CONTRIBUTING.md
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation

Solve the conflict

done @jfmartinz


Fixed import error I was getting when importing keras

I ran into an error like this stackoverflow question when running the notebook. Changing keras.preprocessing to keras_preprocessing in preprocessing.py fixed the issue.
https://stackoverflow.com/questions/72479044/cannot-import-name-load-img-from-keras-preprocessing-image
Let me know if this change causes problems on your setups.

You are good to merge. I tried it on my system and it worked


refactor(content-docs): clean up sidebars logic; validate generator returns

Motivation
Tweak the sidebar loading order again, after we have a much more complicated logic. This clears all the TODOs. I have also written some brief internal docs on the holistic views of these functions.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
All tests still pass.

It will be quite hard for me to review this PR 😅 next time try to do it in 2 steps:

first refactor without changing any test/behavior => verbose but easy to accept
then make some behavior changes => easy to review the little differences (I do see some little changes here, but probably not all)

If you think it's safe to merge, then I trust you considering overall it looks fine

first refactor without changing any test/behavior => verbose but easy to accept

I didn't change any behavior; just shifted the steps. If I don't reorganize the workflow there's basically no meaningful refactor.
And, the end behavior is exactly the same. Evidence: no snapshots from outside the sidebars folder have changed

Understood 👍 go ahead then
I did notice one change, and if there is one I suspect I might not notice all 🤪
For example this snapshot looks different

That does not look bad to me to do this change anyway

Yes, all the default values are applied during post-processing now. This means the generator can return a shape that's very similar to user config and it can still be correctly normalized


How is network key supposed to be set?

The original code from @mvillalba had a NetworkKey object that held the number and key, and was passed to setNetworkKey.
Your code does away with the object, but setNetworkKey still tries to access properties of the network variable.

Similar problems with assigning a channel.

I think it's a regression.
https://github.com/SamyCookie/python-ant/commit/efe8694874c3c3029c5e8a4aca58f6a3a522ad5e#diff-3f8cf9f5f1e5d5381a14d76efe2b0617  is the bad commit.

Or not. I have not access to my frontend code, but in fact the 'key' parameter seems to have a really bad name.
'setNetworkKey' seems to get a 'Network' object for its second parameter, not directly a key.
try for example:
network = Network(key, name)
node.setNetworkey(number, network)

Huh, I didn't see the Network object. I thought you had removed the entire class. I don't know why I missed it, it's right at the top of the file!

I made a small cleaning commit to rename 'key' variable to 'network'. It is now less confusing.
22e85dc6f145c52b4c8e9bada3591b40e87ea194


Feature: Better propogation of reasoned patterns

In the case of lazy reasoning we keep a record of the pattern we were reasoning for so we don't need to re-reason on future queries.
However, during the optimization process we can actually discover that we may be making a few other patterns available at the same time. If this is the case then that should be output as metadata from the rule-optimization actor so we can include that in the record of patterns that have been reasoned for.

We may also wish to include a flag that indicates if we are in a 'degenerate' case (such as querying over the pattern ?s ?p ?o ?g) where we need to undertake full reasoning anyway - and hence just apply the 'full' reasoning flag.


[CMWG] List the different org affiliation metrics

The Common Metrics WG is going to start listing the different org affiliation (and related) metrics in this Google doc, before splitting them up into separate issues.

Thanks Tobie!
I added related metrics from the README.


pull request for e2e test at 12/02/2016 15:45:05

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

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


Question: Performance messagepack vs. json in browser?!?

Exist there some measured performance value that compares MessagePack.js with json ?

Yes, there will. Look here: https://github.com/uupaa/MessagePack.js/wiki/Benchmark

Thanks. I'm interested in a compare between JSON/messagepack in browser. Because all browser support JSON native.
I ask because of: https://github.com/pypyjs/pypyjs.github.io/issues/4#issuecomment-117808915


Fix data unavailability tutorial

It was broken for a few main reasons:

There was no guarantee that important system data wouldn't end up on
the two nodes that we're bringing down, since we were only using two
localities.
Even if we used 3 localities, the pattern of starting up a 3-node
cluster first and then adding more nodes means that the system ranges
may not be properly spread across the diversities, since in v1.1 we
don't proactively move data around to improve diversity.
A read-only query isn't guaranteed to hang even if a range is
unavailable. If we only kill the 2 non-leaseholders, the leaseholder
will still be able to keep extending its lease (via node liveness)
and serve reads.

To fix #1, I've modified this to spin up a 9 node cluster across 3
localities.
To fix #2, I've spun up all the nodes before running cockroach init.
We can go back to the old way of doing this once the labs use v2.0.
To fix #3, I've switched from demo-ing a SELECT to using an INSERT.

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/8e8d48f80cdb2eb108bdb33d5fee1cc195076e33/

Looks like I need to regenerate a couple screenshots. Will have those up in a few minutes.

Screenshots updated.

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/37837918666a819284824da0a1eb0aa990837a48/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/87eca7ca405e0926c2adfece29a0396705780182/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/e98080b0171d8a799328ce6ee5ab08c98d88612b/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/16ad166a7ff525c5857742216d74fee508aa9be8/

Review status: 0 of 7 files reviewed at latest revision, 2 unresolved discussions, some commit checks failed.

training/data-unavailability-troubleshooting.md, line 194 at r1 (raw file):

    | Start Key | End Key | Replicas | Lease Holder |
    +-----------+---------+----------+--------------+
    | NULL      | NULL    | {3,6,9}  |            9 |

Shouldn't this be 7,8,9.

training/data-unavailability-troubleshooting.md, line 214 at r1 (raw file):

    --insecure \
    --port=26264
    ~~~

In other modules, we've told the trainees to ctrl-c the appropriate nodes. Any reason to do this differently here?

Comments from Reviewable

Review status: 0 of 7 files reviewed at latest revision, 2 unresolved discussions, some commit checks failed.

training/data-unavailability-troubleshooting.md, line 194 at r1 (raw file):
Previously, petermattis (Peter Mattis) wrote…
Shouldn't this be 7,8,9.

That might be clearer, but it wouldn't reflect reality given that we're starting all the nodes before running init. And we have to do that due to point #2 in the commit message.

training/data-unavailability-troubleshooting.md, line 214 at r1 (raw file):
Previously, petermattis (Peter Mattis) wrote…
In other modules, we've told the trainees to ctrl-c the appropriate nodes. Any reason to do this differently here?

Because the instructions in step one start up all processes in the background using & to avoid needing 10 terminals for this lab (9 for the nodes, 1 for other commands). If you think using 10 terminals would be less confusing, I can change it back. Using --background isn't an option because it doesn't work until after init has been run.

Comments from Reviewable

Review status: 0 of 7 files reviewed at latest revision, 2 unresolved discussions, some commit checks failed.

training/data-unavailability-troubleshooting.md, line 194 at r1 (raw file):
Previously, a-robinson (Alex Robinson) wrote…
That might be clearer, but it wouldn't reflect reality given that we're starting all the nodes before running init. And we have to do that due to point #2 in the commit message.

Ah, the nodes here won't be deterministic. You'll have to call that out in the presentation. I was wondering why you recommend visiting the nodes report. Now it makes sense.

training/data-unavailability-troubleshooting.md, line 214 at r1 (raw file):
Previously, a-robinson (Alex Robinson) wrote…
Because the instructions in step one start up all processes in the background using & to avoid needing 10 terminals for this lab (9 for the nodes, 1 for other commands). If you think using 10 terminals would be less confusing, I can change it back. Using --background isn't an option because it doesn't work until after init has been run.

Ah, I missed the &. Carry on.

Comments from Reviewable

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/3d388175da698e4d383d40cfd5a9749060dded1c/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/fb1ecc89da96ca8fb5c900af2512dc9f8b02c564/


Python version

Hi what Python Version does this run on?
I had to install alfworld on 3.6 for this issue
But python3.6 can only install openai max at 0.8.0, which breaks the code. I have also tested python3.7 and 3.8 and does not work with AI2Thor. I am on ubuntu 20.04. Can u share your python version?

Hi just an update, for people who might need it.
Basically my script/check_thor.py and run_eval.py both got stuck due to https://github.com/askforalfred/alfred/issues/130
The output of the log file looks like this:
`
SocketException: The socket has been shut down
at System.Net.Sockets.Socket.Send (System.Byte[] buf) [0x00000] in
at AgentManager+c__Iterator3.MoveNext () [0x00000] in
at UnityEngine.SetupCoroutine.InvokeMoveNext (IEnumerator enumerator, IntPtr >
(Filename:  Line: -1)
All I need to do ispip install werkzeug-2.0.3`. Alfworld does not require python3.6 to be able to run. I'm running this repo on python 3.8, ubuntu 20.04


samples: set client endpoint

Fixes https://github.com/googleapis/nodejs-storage/issues/1781 🦕

@ddelgrosso1 @Breathtender will write tests after I get confirmation that this sample looks good to you in terms of structure / things i'm printing

Thanks @shaffeeullah yes this looks good

Thanks @shaffeeullah yes this looks good

@Breathtender awesome. Just added tests. Please take another look


do we have to train model for new faces?

Suppose that we have new faces to apply this method and recognize each one. In this scenario do I have to train the network on these new faces or the pretrained model will give the sufficient performance?
I ask this question because I can't get my desired accuracy , so I guess there are two possibilities 1- the pretrained model is not proper for new faces 2- I can't use caffe c++ properly.
Thank you for your great work and support.

Thanks a lot,
I will try this out.

Hi Lixin Liu - what do you recommend?
First question:
Training the sphereface model with m = 1 on the new data-set first and then take the trained sphereface model to train sphereface-plus network with the same m = 1?
or
Training the sphereface model with m = 4 on the new data-set first and then take the trained sphereface model to train sphereface-plus network with the same m = 1?
Second question:
Does the chronology in training the network matter? - When I say chronology, I mean training m = 1 first then take the trained (m = 1) model to train m = 2  model and the same procedure for m = 3 and m =4 training.

@Maharshi-Aupera

Just train the sphereface model with m=4 on the new dataset and take the trained model to train sf-plus network with m=4.
I think such chronology will not work, you can have a try.


Adjust forms-generation-tool to properly release the cli for Linux, macOS and Windows

Releasing on NPM with a bin entry should be enough for people to be able to use it... All agree?

@tiagobento @porcelli would you mind to add more detailed description, what are acceptance criteria. i see you discussed in comments, but still not sure . Is there any existing librabry on npm repository that can be like example of demanded result?

@jomarko The build-env is an example of package that has a bin entry on its package.json. With the bin entry it'll be possible to directly use it in the CLI after the package is installed. Now, to use without node it would require to release the binaries, and for that we need to add steps in our CI as we have for the extended-services and kn-plugin-workflow packages.

@porcelli  @tiagobento We completed all the tasks related to form-generation-tool, should we start to work on this ticket too?
//cc @pefernan

+1 from me

Hello team, I just wonder what are the 'acceptance criteria' here.

what should be the cli command name? should it be form-generation-tool? or should the name somehow reflect apache, or kie ?
am I correct user will be intended to firstly execute npm install -g @kie-tools/form-generation-tool to make this new cli command available on his/her machine?

I was able to prepare a packages/form-generation-tool/src/cli/cli.js file:
#! /usr/bin/env node
/* License omitted */
require("../../dist/index");

then I prepared update in packages/form-generation-tool/package.json:
  "bin": {
    "form-generation-tool": "./src/cli/cli.js"
  },

and was able to execute it as npm run form-generation-tool  from the packages/form-generation-tool folder
However I somehow feel, this is not enough verification before some npm publish is done.
Could somebody help me with better understanding acceptance criteria? How to be sure prepared changes are expected?

what should be the cli command name? should it be form-generation-tool? or should the name somehow reflect apache, or kie ?
+1 using form-generation-tool as command name

am I correct user will be intended to firstly execute npm install -g @kie-tools/form-generation-tool to make this new cli command available on his/her machine?
I guess this is inevitable, I mean, the user needs to install the library before using that. I don't see alternatives.

@pefernan @tiagobento can you please share your thoughts too?

@yesamer thank you for your input. I prepared a PR when I follow this naming - form-generation-tool.
Anyone, please feel free to have a look on the PR

FYI, this feature is cancelled with https://github.com/apache/incubator-kie-issues/issues/1493


List View Library can be added to Edit View, but not List

Describe the bug
When creating a library entry from a List View, that library entry appears as an option for Edit Views, but not other List Views
To Reproduce
Steps to reproduce the behavior:

Open a List View
Select the top-most ListColumn from the Layers
Click Library > Add
Name and Icon the Library Entry as normal
Open configuration on a new List View
Open the Library section
Observe that your new Library Entry does not appear
Open Configuration on an Edit View
Open the Library section
Observe that your List View Library Item can be added to the Edit view

Expected behavior
List View Library Items should be visible on List Views and not on Edit Views
Screenshots
List View Library Item added to an Edit View:

Same Library Item not visible on a List View:

fixed in #2774


Consider variables without a default required

This commit enforces a separation of concern between the notion of a "required" value (declarative) versus a value that is allowed to be empty (parser trait).
The following rules apply:

A value without a default declaration is considered a required value.

It's an error if there's no environment variable present that matches a required value.

An environment variable that contains an empty string fulfills the requirement. It also overrides a default.

A default is used only when there's no matching environment variable, or when the parsing of a value fails (results in a warning).

This change is a little hard for me to read. Am I correct in thinking that the only real change is that it no longer considers "empty" and "unset" to be the same, and the rest is just cleanup?


Error should be Fail for Apex error "Property X not valid in version #"

This build is marked as Error but should be Fail since the problem is in the Apex code.
https://mrbelvedereci.herokuapp.com/builds/6069

Here's another Error that should be Fail:
https://mrbelvedereci.herokuapp.com/builds/7313 "Method does not exist or incorrect signature"
Maybe there is a way we can handle all these generically.


Draft: Add cluster option to result backend

This is a draft that adds redis cluster option for result backend.
This PR contains set up of redis cluster in CI, which can be used in further PRs (such as adding cluster mode support to broker).
This is still work in progress, I'll ask for review when I think it's ready.

I'd like to ask for CI (GitHub actions) approval. Otherwise it's really hard to fine tune the tests.

Sure. Anywa. I have a question regarding self.redis_cls.from_url. Does this thing create a connection pool inside or it maintains a single connection?

Sure. Anyway. I have a question regarding self.redis_cls.from_url. Does this thing create a connection pool inside or it maintains a single connection?

AFAIK RedisCluster maintains connection pool internally and it's important to create just single instance of it (not using it as a context manager, which would destroy the connection pool after each __aexit__). It's not possible to provide RedisCluster with external connetion pool upon instantiation. I suspect this is because cluster constructs the connection pool from fetched replicas addresses that are not known when calling RedisCluster.__init__.
I used the same approach for single Redis instance as well. The other possible approach would be to create separate result backend class for cluster mode. I'm not really sure which of these two approaches is better though. I'd appreciate input. If we added a new class, the original result backend could stay as it is.

@stinovlas, let's create a completely separate backend for now. It might be called like RedisClusterResultBackend. Also it would be easier to maintain.

@stinovlas, let's create a completely separate backend for now. It might be called like RedisClusterResultBackend. Also it would be easier to maintain.

I separated the cluster backend to a new class. There's quite a lot of duplication (also in tests), but let's call it a proof of concept for now. I believe that tests could be deduplicated, at least to an extent.

For tests. Let's create a docker-compose with cluster and non-cluster redis. with correct ports exposed.

For tests. Let's create a docker-compose with cluster and non-cluster redis. with correct ports exposed. Because github services is a bit limited.

I was able to put together single-node cluster without docker-compose. It seems to work. I can setup full-fledget six node cluster, but I'm not sure it's worth it for test (after all, we test interaction with the cluster, not the cluster itself).
If you insist on docker-compose, I'll need a bit of help, since I'm not that familiar with GitHub actions workflow. I can put together a docker-compose.yml, but then what? Can I call docker-compose up -d directly from CI, or is there some other setup that needs to be done?

I insist on docker-compose for two main reasons.

Local testing is the same as testing in CI.
It's easy to test everything locally;

Because currently there's no setup of a test deployment locally. One redis container is fine, but cluster setup is a bit of a pain to setup.

Looks good to me. I may change docker-compose file a bit later. But anyway. Thanks for your contribution. I really appreciate this.

Looks good to me. I may change docker-compose file a bit later. But anyway. Thanks for your contribution. I really appreciate this.

Thanks for communicating so promptly :-). I'll probably take a stab at ListQueueBroker which could provide cluster alternative as well, since it doesn't use potentially problematic PUB/SUB.

Yes. Seems like it's possible. But I'm not sure how well does redis cluster handles lists in terms of atomacity.


Warp Level Multisplit

This implementation is brilliant and simple. I've tried reading through the cub implementation and wow. This is so much better.
I was hoping you could answer a question for me about the block of code below.

uint4 bits = countbits(firstOffsets << LANE_MASK - LANE);
        int index = (firstKeys.x & RADIX_MASK) + (WAVE_INDEX << RADIX_LOG);
        uint prev = g_waveHists[index];
        if (bits.x == 1)
            g_waveHists[index] += countbits(firstOffsets.x);
        firstOffsets.x = prev + bits.x - 1;

Here is where you are calculating the wave offsets by using the WaveActiveBallot to check whether the thread pulled a key with a set bit.
But what is the code block above doing? I don't understand - for example - why you check to see if the counted bit is 1. The lane mask in the first few lines is also puzzling to me as it seems to shift away all of the bits in the lowest lanes.
Would you mind if I try my hand at implementing a version of your code in Metal? MSL is sorely lacking in freely available algorithms such as this one and your implementation is really impressive.
Originally posted by @bentoboxlimited in https://github.com/b0nes164/ShaderOneSweep/issues/1#issuecomment-1702007152

Feel free to implement the code in Metal, the only thing I ask is that you credit me, link to this repo, and include the original BSD-3 license from CUB as Nvidia is still the copyright holder of the algorithm.
Now, to answer the first question:
Our sort is a least-significant-digit radix sort (LSD). That means we performing counting sort passes starting from the least-significant digit, up to the most significant digit (MSD). Unlike MSD sorts. an LSD sort must be stable to function. That means that keys of an equivalent digit value must be scattered in the same relative order as in the original input during a pass.
So here is the issue: To save on register memory requirements, each warp shares a portion of shared memory to accumulate digit counts for the counting sort, we call this the warp histogram. Now, let's imagine, we have an Nvidia card which has a warp-width of 32 an each thread in our warp has the following keys:
0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5

Each of the first sixteen threads have a key of value 0, and the last 16 threads have a key value of 5. In order for the sort to be stable, each thread needs to know if any lower rank thread in the warp also has a key with the same value, so that the keys are scattered in the same relative order. To solve this, we use a modified version of Ashkiani et. al.'s Multisplit. I highly recommend reading the paper and attempting to implement the algorithm yourself, otherwise what I'm saying won't make much sense. To put it simply, all multisplit does is create a warp-width value where each bit indicates whether a thread shares a key with other threads of the warp. So going back to our example, the bitwise value produced by multisplit for key value 0 would be
00000000000000001111111111111111

because the first sixteen threads share the same value, 0. Because each thread knows its own index in the warp, we can calculate the number of preceding keys that share the same value by shifting away all higher rank bits, then counting the remaining bits. So for example, we want to get preceding count for thread number 5:
First we shift away 31 - 4 = 27 bits like so (this is firstOffsets << LANE_MASK - LANE):
00000000000000001111111111111111 << 27
//equals<PHONE_NUMBER>0000000000000000000000

Now, by counting the bits that remain, we know that there are 4 preceding threads, and so the correct, stable offset for this thread should be 4:
countbits(11111000000000000000000000000000) - 1
//equals 4

By counting the number of unshifted bits we get the total digit count for the warp, which a single thread then adds to the warp-histogram
countbits(00000000000000001111111111111111)
//equals 16

So to recap multisplit does two things:

It determines the correct per-thread offset if threads share a key value.

It determines the total count of a given key value in a warp, and safely increments the histogram in shared memory.

In Ashkiani's original implementation multisplit was performed for every digit of the radix. That means for an 8-bit radix, the WaveActiveBallot operation was performed for all 256 digits. Resulting in:
$$\left\lceil \frac{digitCount}{warpWidth} \right\rceil \log_2(digitCount)$$
WaveActiveBallot operations. However, this is wasteful. By the pigeonholing principle we know that at most, there can only be warp-width number of keys present, meaning that at most we should only be performing:
$$\left\lceil \frac{warpWidth}{warpWidth} \right\rceil \log_2(digitCount) = \log_2(digitCount)$$
WaveActiveBallot operations.
The issue is accomplishing goal number two of multisplit: determining the total count of the warp. Since we are no longer checking every key value, how do we know which value to increment in the histogram if we have the count? And how do we make sure that only single thread increments the histogram value?
The trick is to realize that we can determine the correct key value to increment in the warp histogram simply by retrieving the key value from a thread's register. The second trick is to realize that only one thread will have a shifted bit count of 1, for any given key, and so by checking if
if(bits == 1)

we guarantee that only a single thread increments the histogram for a given key value, thus ensuring it is properly updated.

Fantastic thanks for the detailed explanation. I'll definitely add a Metal implementation to my do to list and link back here.

Good luck! If you haven't done so already, make sure to pull the latest version of the repo because I refactored the code, so it should be significantly more readable now!


Load time still long

We're better than typescript, but still:

I think there's some really low-hanging fruit here. I think we can delay stuff by not require so much in the main file top?
For an IDE this is ok, but I'm using atom as a main text editor and load windows from command line all the time, and seconds to load is not ok. Many utils is add only like a few ms to load time.

Some progress:

Getting somewhere…

However, I realize I need to really start to be more critical on what plugins I have installed. And the once I really want that are slow I should probably try to help profile. This is actually really low hanging fruit sometimes, but sometimes the code gets a bit ugly. I gotta think about it a bit.
For one thing I'm no node expert so I think I first gotta look into how fast the require cache really is. If it's just super super quick I think the easiest way would be to simply move all requires until needed without any own caching.

Browser-plus and emmet gotta go for now :)

I think we're down with the cool kids now:

Still way to slow with 2 secs startup. It's actually more. Just counting doing atom .  on a small folder it's at about 4 sec until I can edit.


Undefined is not an abject (evaluating _reactnativeuikitten.RkTheme

Hi,
I just upgrade my react-native version to latest and also packages associate with that. I upgrade react-native-ui-kitten version  from 3.0.1 to 4.3.2 there is error will show relate to UI kitten. So can u provide the proper documentation for changing project version from 3.0.1 to 4.3.2. And also i have different theme for different entity in my project.

@Subramani058 v3 is not compatible with v4 as it was completely rewritten and there is no migration guide for this.

Can i upgrade to latest react-native version with<EMAIL_ADDRESS>Will it support?

Depends on what version of react-native you're installing.
We don't maintain v3 since react-native 0.57, so you should see if react-native brings any breaking changes starting from that version. It's changelog is here.


separate data only interface

It is only cleanup to clarify data interface and agent interface.

CI fails somehow.


[Feature]: bind python and c++ through tools other than pybind11

🚀 The feature, motivation and pitch
As vLLM goes into a fast release schedule (currently one release every two weeks), we will quickly hit the project-wide limit of pypi (around 5GB per project). One solution, as pointed out in https://github.com/pypi/support/issues/3792#issuecomment-2099941677 , is to build one wheel for all python versions (Python 3.8+).
I have figured out the procedure https://github.com/pypi/support/issues/3792#issuecomment-2101360740 , but pybind11 does not support this Python Limited API protocol.
One possible solution is to replace pybind11 with some other tools, so that the binding procedure can be used with Python Limited API.
Possible solutions:

Nanobind (seems to support it starting from Python 3.12 only: https://github.com/wjakob/nanobind/pull/561 )
register ops through pytorch directly https://pytorch.org/tutorials/advanced/torch_script_custom_ops.html

Alternatives
No response
Additional context
No response

cc @simon-mo

close as it is completed by https://github.com/vllm-project/vllm/pull/5047


Make owner an optional flag in the runners command

Description

Make the owner flag optional for the runner command. All runners for a customer are returned when the flag is not set.
The owner flag could be used to filter runners for a specific customer. The Actuated control plane will need to be updated to support filtering runners.
Motivation and Context

The owner flag was required but ignored. The command always returns all runners for orgs the access token owner is a member of.
How Has This Been Tested?

Verified the command can be used without the owner flag and runners for the correct orgs are returned.
Verified the command is working against an updated version of the controller that supports filtering.

Types of changes

[x] 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:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I've read the CONTRIBUTION guide
[x] I have signed-off my commits with git commit -s
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

I've done the work on this.


Windows - search_path crash when %Path% has ";";

Version: 1.13.1
Platform: Windows

I have a node snippet but I directly maps to uv_spawn -> seach_path
d:\code\node-cur$ node
> process.env.Path = String.raw`";";`
'";";'
> require('child_process').spawnSync('node.exe -i')

d:\code\node-cur$ echo %ERRORLEVEL%
-1073741819

Now the even XP is EOL is there a reason not to use WIN32's SearchPath

@refack great find. I've already started making a patch for that for libuv

@bzoz any reason not to use WIN32's SearchPath?

TBH, I don't know SearchPath.
If you want go ahead and fix this. I have a test that dies with current implementation:
TEST_IMPL(spawn_quoted_path) {
#ifndef _WIN32
  RETURN_SKIP("Test for Windows")
#else
  char* quoted_path_env[2];
  options.file = "not_existing";
  args[0] = options.file;
  args[1] = NULL;
  options.args = args;
  options.exit_cb = exit_cb;
  options.flags = 0;
  /* We test if search_path works correctly with semicolons in quoted path. */
  /* We will use invalid drive, so we are sure no executable is spawned */
  quoted_path_env[0] = "PATH=\"xyz:\\test;\";xyz:\\other";
  quoted_path_env[1] = NULL;
  options.env = quoted_path_env;

  /* We test if libuv will not segfaul */
  uv_spawn(uv_default_loop(), &process, &options);

  MAKE_VALGRIND_HAPPY();
  return 0;
#endif
}

TBH, I don't know SearchPath.
If you want go ahead and fix this. I have a test that dies with current implementation:
TEST_IMPL(spawn_quoted_path) {
#ifndef _WIN32
RETURN_SKIP("Test for Windows")
#else
char* quoted_path_env[2];
options.file = "not_existing";
args[0] = options.file;
args[1] = NULL;
options.args = args;
options.exit_cb = exit_cb;
options.flags = 0;
/* We test if search_path works correctly with semicolons in quoted path. /
/ We will use invalid drive, so we are sure no executable is spawned */
quoted_path_env[0] = "PATH="xyz:\test;";xyz:\other";
quoted_path_env[1] = NULL;
options.env = quoted_path_env;
/* We test if libuv will not segfaul */
uv_spawn(uv_default_loop(), &process, &options);
MAKE_VALGRIND_HAPPY();
return 0;
#endif
}

Cool.
Let's do both, replacing the exsisting logic with SearchPath might be semver-major

Yep, this should land in master.


Use wp user capabilities

We should use wp capabilities to decide if a logged in user is allowed to edit the tournament scores. Probably also split up administration (edit team data) and moderation (edit scores only).

Implemented in 6d9faca79c40701a1ffb8138029cbd8f5406ffd2 and ddc763635eee5a296773828234cda0f9dc55dcd4.


Add eslint-config-azuretools package

I tried exporting an ESLint config file from the dev package but ESLint doesn’t currently support loading a module as a config file. There's an open issue to support that but it doesn't seem like it's getting traction: https://github.com/eslint/eslint/issues/14170
So I'm proposing we make a little npm package that's just our ESLint configuration. This is the recommended way to share ESLint configs (source).
To use this in each extension, we would add "eslint-config-azuretools" as a dependency and ".eslintrc.js" would be simplified to:
module.exports = {
    extends: "azuretools"
};

(ESLint assumes the "eslint-config-" part)
If this is the direction we want to go, I'll add the rest of the files needed to make this a for-real package (license, readme, etc.)

After doing a little bit of my own local testing, here's what I found.

If I try to export all the properties we need in the index.js of vscode-azuredevextension, then the ESLint ConfigValidator.validateConfigSchema` throws an error about having an invalid ESLint configuration.
Based on the documentation, if we want to extend the eslint module via npm, I think that module.exports has to live in the index.js.
I was able to connect it locally with a file I named eslint.js and it worked fine.  I think if we wanted it to work across our other extensions, we could refer to the eslint.js file path directly with a relative path that would look like "./node_modules/vscode-azureextensiondev/src/eslint/eslint.js".   I imagine if we published a new dev package, we could refer it to the same way, but it's definitely less clean than just hosting a npm package dedicated to the eslint rules.

As discussed in standup Friday - the plan is to publish our shared config as an npm package for both production & test configs

It turns out you can put multiple configs in one package so I went ahead and added a shared config for tests


Can I add a component to input hour/minute/second?

It would be very useful to add it to the calendar.
Like the one in the browser.

That's outside the scope of this lib. The intention is that any UI beyond the calendar gets added by devs. You could add two selects or two inputs for minute and hour yourself
Closing for now, but happy to continue the discussion


Use Theme Extensions

Problem
With Flutter 3.0 Google announced a new Feature called "Theme extensions" which allows ThemeData to be extended with custom values. As we have a lot of custom functions for handling our various stlyes we whould make use of this feature and should unify all this.
https://docs.google.com/document/d/1LbD4JqBgAfHex02oR3r2jyu9lTBBNBmyec2ovT59Kr8/edit#heading=h.pub7jnop54q0
Non blocking, we can take care of that after the release
Part of

#525

As a UI/UX Designer and a Flutter developer I believe I can work on this. So, can I take up this issue?

Sure @BhuvanAde but as this is a relatively old issue and things have changed a lot since then, what did you have in mind

The initial idea was to have some constants for padding and such in the ThemeData class, but we managed to solve this problem using global constants in .\packages\smooth_app\lib\generic_lib\design_constants.dart  and I think we can leave it like that. The names will get way too long if we put them in the ThemeData class.
Though I've seen your work in your website, what suggestions do you have for the app

Yes, I understand! I went through the app. I noticed that we can make the profile section more enticing by making minor UI/UX changes. Please let me know if you are interested.

Hello @M123-dev, could you please suggest some issues for me to work on?

Sorry for my late answer @BhuvanAde I didn't had much time to contribute to this project.
You can check out these lists:

https://github.com/openfoodfacts/smooth-app/issues/525
https://github.com/openfoodfacts/smooth-app/issues/3045

Besides that here is a list ranked by priority:

https://github.com/orgs/openfoodfacts/projects/7/views/1

And if you are more interested on working UI/UX mocks at least from my personal opinion has the settings tab potential to improved. If you have ideas for this feel free to open a issue with them or share them on Slack 😄

Thanks @M123-dev


config api compatibility

🤔 This is a ...

[ ] New feature
[ ] Bug fix
[ ] Performance optimization
[ ] Enhancement feature
[ ] Refactoring
[ ] Code style optimization
[ ] Test Case
[ ] Branch merge
[ ] Workflow
[ ] Other (about what?)

🔗 Related issue link

💡 Background and solution

📝 Changelog

☑ Self-Check before Merge
⚠ Please check all items below before requesting a reviewing. ⚠

[ ] Doc is updated/provided or not needed
[ ] Changelog is provided or not needed

I tested it with nyanpasu. The api call is sucessfully. However, the backend seems not refresh successfully?

do you see any logs?
the data you posted seems to be from the subscription url and displayed by the client, not necessarly anything to do with the backend

Yes, I can see reloading info in log:
2023-12-26 22:11:04 INFO - [clash]:   2023-12-26T14:11:04.236512Z  INFO clash_lib: reloading config

2023-12-26 22:11:04 INFO - [clash]:     at clash_lib\src\lib.rs:305

2023-12-26 22:11:04 INFO - [clash]:

2023-12-26 22:11:04 INFO - [clash]:   2023-12-26T14:11:04.256364Z DEBUG clash_lib: reloading dns resolver

2023-12-26 22:11:04 INFO - [clash]:     at clash_lib\src\lib.rs:314

2023-12-26 22:11:04 INFO - [clash]:

2023-12-26 22:11:04 INFO - [clash]:   2023-12-26T14:11:04.256663Z DEBUG clash_lib: reloading mmdb

2023-12-26 22:11:04 INFO - [clash]:     at clash_lib\src\lib.rs:320

2023-12-26 22:11:04 INFO - [clash]:

2023-12-26 22:11:04 INFO - [clash]:   2023-12-26T14:11:04.256695Z DEBUG clash_lib::common::mmdb: mmdb path: C:\Users\a632079\.config\clash-verge\Country.mmdb

2023-12-26 22:11:04 INFO - [clash]:     at clash_lib\src\common\mmdb.rs:26

2023-12-26 22:11:04 INFO - [clash]:

2023-12-26 22:11:04 INFO - [clash]:   2023-12-26T14:11:04.260301Z DEBUG clash_lib: reloading cache store

2023-12-26 22:11:04 INFO - [clash]:     at clash_lib\src\lib.rs:330

2023-12-26 22:11:04 INFO - [clash]:

2023-12-26 22:11:04 INFO - [clash]:   2023-12-26T14:11:04.260556Z DEBUG clash_lib: reloading outbound manager

2023-12-26 22:11:04 INFO - [clash]:     at clash_lib\src\lib.rs:339

2023-12-26 22:11:04 INFO - [clash]:

That page is fetching data from GET /proxies, and seems config not be  applied to api endpoint.

ok i think it's a bug. let me take a look

Let me have a look later

The new listen port is :9090 does it not respond?

The new listen port is :9090 does it not respond?

no, remain old data yet.

@greenhat616 i think the backend is reloaded but somehow UI is not notified.
i tried to figure out the point when the UI refresh the proxies, but I can only find the auto refresh here https://github.com/keiko233/clash-nyanpasu/blob/main/src/components/proxy/use-render-list.ts#L28
do you know if there's any other case the proxy list gets refreshed?

Confirmed. It seems SWR cache issue. Clash Nyanpasu use GET / providers/proxies when core is Clash Premium or Mihomo/ Mihomo Alpha, which return a chunked encoding( it might break SWR cache). I supposed to draft a pr to fix this issue in Clash Nyanpasu.


NPSP Data Import Failure When Updating Existing Account with Address

If existing Account with an Address is imported via the NPSP Data Import tool, an error is thrown: SObject row was retrieved via SOQL without querying the requested field: Account.BillingCity
Steps to Reproduce:

Turn off organization address management in NPSP Settings
Import new Account with Address.
Import same Account with Address via NPSP Data Imports tool.

**lurch: add

Tracking W-021287

Included in beta release 3.115 (Beta 6)

Included in production release rel/3.115

@jesslopez-sf I'm still hitting this bug in Production when importing new Account records (not update to existing records). Are there any work arounds? I waited several days for fix and need to finish this import for a client. Thx for any help! @mrbelvedere

The release with this fix is being pushed to production orgs tomorrow evening.  You can get it sooner if you want:
Push Schedule
Sandbox orgs: 2017-11-22
Production orgs: 2017-11-28
Use the Installation URL below to install this release before the scheduled push dates.
Installation URL
https://login.salesforce.com/packaging/installPackage.apexp?p0=04t1Y000001I8xg

Thanks @davidhabib I installed the update and the error is resolved. Many thanks!!


ngSticky doesn't work if applied to the nearest element to .

Have this:

ABC

123

ABC doesn't stick to top 123 does.

+1 having a very similar issue, did you find a way around this?

@ccooper93 I assume you must are using bootstrap right? I figured out once bootstrap was applied, then this issue was reproduced. I have no time to investigate which CSS rule actives this issue, I inserted a blank div above the first div element with a very small width and height for the temporary solution.

ABC

123

@neesonqk Im using foundation 6 must cause the same/similar issue. Thanks for the suggestion, i'd added another sticky element and hidden it. Thats a nicer workaround!


Add ETCR

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

[x] Filename in the lowercase format
[x] Image format: png
[x] Image size: 256 by 256 px

hi sir, why my pull request always failed? i follow the tutorial from your youtube channel
thanks

Image already exist


Update User validations for slug-name split

Slugs are validated like names used to be (except not necessarily present), and names are much looser than previously (only length and presence are required)

Okay I just rebased and the merge test is still failing while the push test passes ?????
cc @vevix I'm so confused


Peer-to-Peer code review

Congratulations William, your code is very nice and structured and looks beautiful.
Some suggestions:

Reconsider if you need to map over the array as all this looping might be slowing down performance
Since the project is small and simple, there might be an easier solution
The page is loading quite slow so think about whether this can be optimized.

Thanks, @Anjacodes for the Issue, and very good structure @williamrolando88. Keep it up. 🔥🔥


Server Defean and mute

When you are server defeaned or muted there is the slightest color change and its hard to tell. Make it bright green or a different color

.container-3baos1 .redIcon-2uP7Ej {
color: #b075ff;
}
.strikethrough-1n4ekb {
color: #b3aeff;
}
Normally muted

Server deafened and muted

i'll look into it

I was looking here the second you said that, it was weird

Fixed, it should be included in a future update.