ModuleNotFoundError: No module named 'yolov4.tf'; 'yolov4' is not a package

Good evening, and thank you for your great work! I have an error ( it's the title of my issue ) while executing the script to run the conversion of YoloV4 to TFlite. Any ideas what I have been doing wrong? Sorry for the trouble. I am using Windows, and I have Python 3.7.0, using an anaconda environment with tensorflow-gpu/tensorflow version 2.3.0. Any help would be much appreciated.
Also quick question since I am a beginner; I have been trying to convert YOLOv3 and YOLOv4 weights to int8 quantized Tflite to run them to Coral for quite some time without success. Could I do it, at least for YOLOv4-tiny using this script? Again, I am a beginner and any help would be much appreciated!

Did you install yolov4?
python -m pip install yolov4

And, YOLOv4-tiny cannot be run on coral because its activation is leaky-relu.
Ref: https://coral.ai/docs/edgetpu/models-intro/#supported-operations
So, If you want, you should set the activation function to relu.
from yolov4.tf import YOLOv4

yolo = YOLOv4(tiny=True)

yolo.classes = "coco.names"

yolo.make_model(activation1="relu")
yolo.load_weights("yolov4-tiny-relu.weigths", weights_type="yolo")

convertion
Ref: https://github.com/hhk7734/tensorflow-yolov4/blob/master/test/make_edgetpu_tflite.ipynb

Thanks a lot for the quick reply my friend and for your time
I have indeed installed yolov4 as you said at your first reply. I also tried following your guide for tflite conversion, but this is where the error popped up. I will try it once more now and will report back in a minute. Can this procedure be done for yolov4 too or just its tiny version?
Thanks

I failed on yolov4. I think yolov4 model is too big.

So I tried running the script again but I renamed it from yolov4 to another name and now it miraculously worked :P ! I don't know if the name of the file was the problem or not. Now the only thing that remains is to put the correct path to the whole coco 2017 dataset and to provide a txt file. Do I have to make a txt file for every single image in the coco dataset?
I have tried yolov3 but I failed on quantization. Same for yolov4 of course. I hope this method works it is my last hope

coco.names, train2017.txt, val2017.txt are in https://github.com/hhk7734/tensorflow-yolov4/tree/master/test/dataset

Thanks a lot for the help. I have some issues extracting the dataset, but once I finish it I will report back asap

Ref:

https://wiki.loliot.net/docs/etc/project/yolov4/yolov4-training#dataset-files-and-formats
https://wiki.loliot.net/docs/etc/project/yolov4/yolov4-training#prepare-dataset

Good luck :)

Hello,
I tried running the script and provided the tiny relu model given, plus the information from the dataset folder. I also used the coco 2017 train dataset as of course specified in the code. I executed it, and it hanged for 20 minutes, ultimately giving me this error:
RuntimeError: Unsupported output type INT8 for output tensor 'Identity' of type FLOAT32.
This is how I edited the code to match my system, other than that I changed nothing. Maybe there is something wrong with the paths of mine, or the way I entered them at the code.
from yolov4.tf import YOLOv4

yolo = YOLOv4(tiny=True, tpu=True)

yolo.classes = "./coco/dataset/coco.names" #path to coco.names

yolo.make_model(activation1="relu")
yolo.load_weights("yolov4-tiny-relu.weights", weights_type="yolo") #weights file is in the same directory

dataset = yolo.load_dataset(
    "./coco/dataset/train2017.txt", #path to the train text
    training=False,
    image_path_prefix=r"C:\Users\Jim\Desktop\int8post\coco\train2017" #contains all the 2017 train images
)

yolo.save_as_tflite(
    "yolov4-tiny-relu.tflite",
    quantization="full_int8",
    data_set=dataset,
    num_calibration_steps=500
)

Install TF 2.2.0 and try again.
I failed to convert when using 2.3.0.

Thank you very very much, it all works successfully. I will test the model the next days, I really appreciate your help you can't find that everywhere


SwitchCase branches not correctly handled

If I have a switch case without break or return, istanbul reports some branches to be covered even if not tested.
For example :
switch (foo) {
  case 'bar': case 'baz':
    alert(foo);
    break;
  default:
    alert('Hello !');
}

will become
switch (foo) {
  case 'bar':
     __cov_ezf654zfe468['b'][0][1] ++;
  case 'baz':
     __cov_ezf654zfe468['b'][0][2] ++;
    alert(foo);
    break;
  default:
    alert('Hello !');
}

Thus, testing the 'bar' case will validate the 'baz' case too and branch coverage statistics will be wrong.
So I propose to test the value before incrementing :
switch (foo) {
  case 'bar':
    if (foo == 'bar') __cov_ezf654zfe468['b'][0][1] ++;
    alert(foo);
  case 'baz':
    if (foo == 'baz') __cov_ezf654zfe468['b'][0][2] ++;
    alert('Hello ' + foo);
    break;
}

closing per discussion in related issue. This is not as trivial as it might seem


Upgrade to platform 5.4.1-pre7

Brings m&m bugfix (https://github.com/Terracotta-OSS/terracotta-platform/pull/457).
Note: build will fail because 5.4.1-pre7 is still releasing.

test this please


[Bug]: Photo mode is broken with this mod

With this mod, the original in game photo mode is broken and give stripes pictures and not a correct screenshot.
Indeed this is not essential but it would be appreciate is this could be fix in a next update.
Keep up the gook work.
Thank you.

best workaround for it atm ig is to disable the ui and use prt scrn


Rename pythia configs to represent the total parameter count

[ ] Create configs/pythia folder and populate with cleaned pythia configs

If we’re going to continue to add config files for new models we train, perhaps we should name them pythia-70m.yml or pythia/70m.yml? If only because it seems highly likely that we’ll train two different models with the same number of parameters at some point.

If we’re going to continue to add config files for new models we train, perhaps we should name them pythia-70m.yml or pythia/70m.yml? If only because it seems highly likely that we’ll train two different models with the same number of parameters at some point.

I'm fine with this, since it would also solve the issue of having a single 125M.yml config, but it being unclear whether that's a GPT-style config or a Pythia-style config. I'll add a pythia folder under configs containing cleaned versions of the pythia configs (i.e. versions without cluster-specific paths and such).

Resolved by https://github.com/EleutherAI/gpt-neox/pull/881


Remove converter examples for deduplication

The example use an uncommon prefix (not xct) and thus likely don't work when copy-pasting. And the resource is probably getting forgotten too. The same example exists with the correct prefix and the resource definition in the linked help articles. Thus I opted to remove the examples, thus guiding the user via the links to those articles with the complete examples.

Docs Build status updates of commit cf6840e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/behaviors/eventtocommandbehavior.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

I don't see why you would want to remove these? Especially leaving the text above intact.
Why not update the namespace name or make the sample more verbose?

That was my initial reaction, then I clicked on the links to the converter help pages: They contain exactly that example. If at any point in the future something is changed on those converters and their help texts changed, most likely the example here will be forgotten and outdated. Thus I believes it's better to not have any example here and just the link to the appropriate location with the example (and explanation).
https://docs.microsoft.com/de-ch/xamarin/community-toolkit/converters/itemselectedeventargsconverter

Makes sense. Let's remove it for now and revisit once I invest some more time in these Docs. Thanks! :)


Added helper to estimate cost of query with YOLOPandas

Added a new utils package with a query_helpers script designed to add more functionality to YOLOPandas. In particular, I added the ability to estimate token costs per query so that users can better understand how expensive it is to run YOLOPandas on their data. I thought the function had the right tradeoff in simplicity to start.

Thanks for this @mturk24. I notice that utils is implemented outside of yolopandas/, and currently won't be installed via pip. What is your intent as to how users should access utils?

Thanks for this @mturk24. I notice that utils is implemented outside of yolopandas/, and currently won't be installed via pip. What is your intent as to how users should access utils?

No problem, I think what you've built is awesome and I'd love to help contribute! I just addressed this by moving utils within 'yolopandas' now.

Looks good, thanks @mturk24. I added a call to run_query_with_cost to integration tests. As-is it basically checks that get_openai_callback will run and has attributes for total_tokens, prompt_tokens, etc.

Awesome thanks!


Error while creating doclet

Sorry, something went wrong during creation of your doclet!
Error: Error: ENOENT: no such file or directory, open '/usr/src/app/lib/031eafb0-dbe3-11e5-9d55-ed04cc93f6aa'

As shown on my page for the repo/branch.  This is the first time I've pushed to this branch with a .doclets.yml
The config I used was quite simple:
dir: lib
packageJson: package.json

Some inconsistency in the docs regarding the doclet config, but I'll create a separate issue.
Edit: To note, I've declared all of the documentation in this repository out of context (it's not inline with other code, only jsdocs code is in the repo) and I'm declaring it within .coffee files, not .js.

Wow.  This is totally my issue.  Not yours.  Thanks.


Get the check-in count from the Batch Registry contract

Goal
There is a "To Be implemented" string in packages/nextjs/app/page.tsx that should show the number of builders that have checked in to the contract.
We want to show the real count (reading from the BatchRegistry contract)
For more info about the contract: https://github.com/BuidlGuidl/batch3.buidlguidl.com/issues/9

Will be working on this issue later today around 4:30PM GMT

Fixed on #12


[Feature] [Satellite] Prometheus Fetcher support to configure TLS for scrape in kubernetes servicediscovery situation

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
When the target service enabled SSL/TLS and deployed in k8s, we have to support users to configure SSL/TLS for prometheus scrape job.
Use case
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

No update in 26 days, if there is no one working on this, we could close this for now.


Select component with click anywhere inside component's box #1066

@sudhirverma The changes you have applied override the previous component layout.

The dotted border layout does not for already installed component. See attached image.

The component border when we hover is also missing as in the case we had previously.

The font for component description is also bold, which should be normal.

Please fix these changes.

Codecov Report

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

@@            Coverage Diff             @@
##           master    #1085      +/-   ##
==========================================
- Coverage   93.11%   92.69%   -0.43%
==========================================
  Files          39       39
  Lines        4273     4342      +69
  Branches      470      745     +275
==========================================
+ Hits         3979     4025      +46
- Misses        294      295       +1
- Partials        0       22      +22

Impacted Files
Coverage Δ

browser/model/devstudio-autoinstall.js
97.29% <0%> (-2.71%)
:arrow_down:

browser/model/helpers/hash.js
85.29% <0%> (-2.59%)
:arrow_down:

browser/pages/install/controller.js
79.85% <0%> (-1.73%)
:arrow_down:

browser/model/helpers/downloader.js
96.13% <0%> (-1.66%)
:arrow_down:

browser/main.js
95.58% <0%> (-1.39%)
:arrow_down:

browser/pages/account/controller.js
82.11% <0%> (-1.33%)
:arrow_down:

browser/pages/location/controller.js
91.13% <0%> (-1.27%)
:arrow_down:

browser/model/jbossfusekaraf.js
95.09% <0%> (-0.83%)
:arrow_down:

browser/pages/confirm/controller.js
62.01% <0%> (-0.78%)
:arrow_down:

browser/services/data.js
91.62% <0%> (-0.78%)
:arrow_down:

... and 17 more

Continue to review full report at Codecov.

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

Resolved in https://github.com/redhat-developer-tooling/developer-platform-install/pull/1087.


Incremental computing: Adapton?

I'm curious if you have seen the Adapton project? It could be a way to help optimize a notebook.
Apologies if this isn't the right way to provide suggestions; I didn't see a mailing list or other community information.

Raising issues if fine for now. I've been meaning to create a mailing list or two, but haven't gotten to it yet.
Thanks for the suggestion. I hadn't seen Adapton. It looks interesting. I'm unsure though how it would be of use to Evcxr. Are you able to explain more what you had in mind?

I have not dug into how execution works, but in the context of a notebook this example seems relevant: https://docs.rs/adapton/0.3.30/adapton/#demand-driven-change-propagation . It allows for only recomputing statements when a change is made, and the result is actually requested.

I'm just going through seeing with old issues I can close. I agree that demand driver change propagation is a good way for a compiler to operate. Unfortunately in the case of evcxr, we don't really have control over that. We just call out to rustc via cargo. If at some point enough of the compiler was broken out into separate crates, we could look at using those crates directly. At that point and with sufficient incremental compilation in the compiler, it might start to look a little like you describe. So I hope eventually evcxr, or an alternative REPL for rust operates something like you describe. For now though, I'll close this issue, as I can't see any progress in that direction in the short to medium term. Thanks for the suggestion though and sorry for being about 2 years late replying to your last message


Exception must be thrown outside the retry loop

Will merge this for now. We still have to work on a few things:

Full test suite
Should not delay emitting ConnectionStatus.Connecting


Remove stream_url_from_id API

This is controversial and I'll just put the code in the dehydrator
for now.
Closes: https://github.com/coreos/stream-metadata-rust/issues/2

@bgilbert reminderping


navbar: Fix background-color in Dark theme.

CZO: https://chat.zulip.org/#narrow/stream/9-issues/topic/top.20navbar.20bottom.20border/near/1598791
Fix the background color for the #userlist-toggle-button and the middle part of the navbar to match the overall navbar color in the Dark theme. This resolves the inconsistency in the background colors.

Fixes:

Screenshots and screen captures:
Before:

After:

Before:

After:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Opened #26133 to fix this


[FEAT] Adding translations for defaultColumnTypes'. filterOperators

Is your feature request related to a problem? Please describe.
I want to be able to add translations in the dictionary and make refine automatically use these for the default filterOperators' labels without needing to build my own filters from scratch. Since this is not possible, using the defaultOperators makes it that I get labels that don't translate along. I have to define my own currently anyway so that I can override the label with a translation, which defeats the purpose of having a default. Unable to find a way to do this currently. It makes a lot of sense to pull these by default from your translation provider and fallback to the labels that already exist currently.
Describe alternatives you've considered
No response
Additional context
Filters don't translate along:

Describe the thing to improve
Ideally you simply pass a list of the following type in for example de-DE/common.json:
"filters": {     "operators": {       "is": "ist",       "between": "zwischen",       "notBetween": "nicht zwischen",       "null": "ist null",       "notNull": "ist nicht null"     }   },   ...
Which makes it that all the filter labels are translated.

Hey @sohaibgv thanks for the issue, looks like a good idea. Would you like to work on this one after discussing the details?

Hi @BatuhanW, I'm interested to work on this issue. Can you assign this to me?

I found that these translations are actually already defined by the UI library you use and not by refine.
We were using MUI and these ones actually exist in
import { enUS, nlNL } from '@mui/x-data-grid/locales';
Some of the translations that were missing specifically came from
import { nlNL, enUS } from '@mui/x-date-pickers/locales';
and
import { enUS, nlNL } from '@mui/material/locale';
These imports are required to make the filters translate on locale changes.
You can look at the keys MUI defines and add these for any missing locales.


feat: add staging area

This change is

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

Project coverage is 64.64%. Comparing base (ac99c1c) to head (0ce25b9).

Additional details and impacted files
@@                                    Coverage Diff                                    @@
##           mohammad/priority-queue/add-thin-priority-transaction     #196      +/-   ##
=========================================================================================
+ Coverage                                                  64.57%   64.64%   +0.07%
=========================================================================================
  Files                                                         26       26
  Lines                                                        940      942       +2
  Branches                                                     940      942       +2
=========================================================================================
+ Hits                                                         607      609       +2
  Misses                                                       307      307
  Partials                                                      26       26

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


Hardened End Stone recipe conflict

Hardened end stone uses 4 end stone in a 2x2 pattern which conflicts with vanilla MC's end stone bricks recipe. It's probably a bad practice to overwrite default MC recipes without a good reason.
There are two logical resolutions to this:

Change the recipe. (For my server I updated it to 9 end stone)
Require a mod like polymorph as a dependency which can handle conflicting recipes. This is IMO sloppy and a bad solution, but would work.

I had uploaded a fixed version for 1.20x
Or do you want a version for 1.19x?

@Craftjakob Thanks for the quick response. I manually edited the recipe inside the jar file so I'm fine for now, I just wanted you to be aware so you could update it, appreciate it!
I assume you fixed the recipe, what was the end result? 9 end stone > 1 hardened end stone?

I have actually changed it to 8 end stone, because maybe someone have a mod installed, that adds compressed end stone.
Like that:

#  =  1 Hardened End stone

Or do you want more than one or another crafting recipe?

I don't have a strong opinion. The solution that made sense to me was 9, and in the vein of compressed (without being actually "compressed end stone") I added a de-crafting recipe that reverses the 1=>9.
In hind sight, a lot of the older mods used obsidian (powder?) for hardened items, so maybe 8 end stone and 1 obsidian would make more sense?
But again, no strong opinions, do what makes most sense for you! Thanks!


Hide and visible in password to improve UI

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
It is good to add seen  and unseen feature in sing in page . It give a choice to user to check its password before sign in
Expected Behavior
No response
Reproduction steps
1. Go to sign in page
3. Scroll down to password
4. See error there is no hid and visible icon.

Screenshots

Logs
No response
Browsers
No response
OS
No response

@Sukomal07

ok i will do it

@wanderlust

@GOURAVSINGH19 , kindly let us know the approach and ETA,
@Sukomal07 let us give @GOURAVSINGH19 a chance,
We can work on the auth ... like the missing things

@GOURAVSINGH19 where are you?

yes bro.

You were interested in taking this up right? @GOURAVSINGH19
Still interested? join discord

Yes I am interested..

How I can join your discord (any Link)..

Tell us your approach and ETA @GOURAVSINGH19

1.We added a passwordVisible state to manage the visibility of the password.
2.The togglePasswordVisibility function toggles the passwordVisible state when the button is clicked.
3.We replaced the type attribute of the input element with a dynamic expression based on the passwordVisible state, which changes between 'text' and 'password'.
4.Added a button next to the input field that triggers the togglePasswordVisibility function when clicked. The button's text changes dynamically between 'Show' and 'Hide' based on the passwordVisible state.

Sure @GOURAVSINGH19 go ahead. How much days will you take ?

today...

@GOURAVSINGH19 github readme , you'll find the discord link

Sorry for disturbing you But I need you help while trying push the change it show errors. like this

You have the answer there @GOURAVSINGH19

Assign me excited to contribute

@krishnaacharyaa  if there is no issue then  you can merge it .

There is issue and I have pinged you twice @GOURAVSINGH19 see the PR comments carefully

Can you please assign me this issue under GSSoC'24 i would like to work on it.

I am sorry but I can't found any pinged issue in my pr comments

@Dhevamuthu @Tarungupta18 have you heard of term code of conduct? If not try to understand

Can you please assign this issue to me ?


Update pipelines to run Unity 2018.4 instead of Unity 2018.3

Overview
This updates our pipelines to run on the installed Unity 2018.4 versions instead of 2018.3.
Internal machines have Unity 2018.4.12 and external machines have Unity 2018.4.6.
I also added a README to remind myself which pipelines were used where.

/azp run


No new data

It looks like we're running the job daily, but the last date I find data is from April, 18, 2016.
https://github.com/open-austin/construction-permits/blob/master/data/2016/2016-04-18.csv
Perhaps the site changed around that date? Or they are blocking our call?

Looks like they're sending back an .xls file now - but trying to get it with requests lib is giving me an empty html table for some reason. Weird.

I looked at this issue in the "Which of our featured projects are still working/up?" audit. Is it important to get the permit scraper working again and fill in the gap in the data? Or is the data.austintexas.gov dataset sufficient now? Is it time to turn off the scraper and stop updating this repo? What do you think @luqmaan @mateoclarke @davemcphee @spatialaustin @wilsaj?

I think it's ok to retire the permit scraper. Construction permit data is available on the open data portal and apparently updated on a weekly basis:
https://data.austintexas.gov/Building-and-Development/Issued-Construction-Permits/3syk-w9eu


Navigation issue with QRoute.withChild while using replaceLastName or replaceAllWithName

Hi @SchabanBo, there is an issue with the replaceLastName or replaceAllWithName while using with QRoute.withChild.
Let suppose say we have some onBoardingPages & some dashboardPages. Now as user has signed in or signed-up we will take the user to dashboard page & as user don't want to come back to onBoardingPages once signed up or sign-in, So I am using replaceLastName/replaceAllWithName to replace the onBoardingPagesCompletely with DashboardPages. Now in dashboardPages there is an logout option so clicking on it we will use replaceAllWithName(to delete complete stack no matter how may pages) & send the user to onBoardingPages. Now at this point after logout if user navigates between the pages of sign-up/sign-in/forgot-password pages then the below errors will pop up
Errors:-

Take user to dashboard page by clicking on login option & clicks on logout option. Now you will be in sign-in page, now click on sign-up button to navigate to sign-up page then error come as "Unhandled Exception: RangeError (index): Invalid value: Valid value range is empty: -1". But the correct way is user should navigate between sign-in/sign-up pages smoothly without any errors.
Take user to dashboard page by clicking on login option & clicks on logout option. Now you will be in sign-in page, now click on forgot-password button. You will be navigated to forgot password back & at this point click on back button then error comes as "Unhandled Exception: Bad state: No element". But the correct way is user should go back to sign-in page.

Below is the reproducible code
import 'package:flutter/material.dart';
import 'package:qlevar_router/qlevar_router.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  final routes = [
    QRoute(
      name: 'sign-in',
      path: '/sign-in',
      builder: () => const SignInPage(),
    ),
    QRoute(
      name: 'sign-up',
      path: '/sign-up',
      builder: () => const SignUpPage(),
    ),
    QRoute(
      name: 'sign-up',
      path: '/sign-up',
      builder: () => const SignUpPage(),
    ),
    QRoute(
      name: 'forgot-password',
      path: '/forgot-password',
      builder: () => const ForgotPassword(),
    ),
    QRoute.withChild(
      name: 'dashboard',
      path: '/dashboard',
      initRoute: '/home',
      children: [
        QRoute(
          name: 'home',
          path: '/home',
          builder: () => ElevatedButton(
            onPressed: () => QR.navigator.replaceAllWithName('sign-in'),
            child: const Text('Logout'),
          ),
        )
      ],
      builderChild: (router) => Dashboard(router: router),
    ),
  ];

  MyApp({super.key});

  @override
  Widget build(BuildContext context) => MaterialApp.router(
        routeInformationParser: const QRouteInformationParser(),
        routerDelegate: QRouterDelegate(routes, initPath: '/sign-in'),
        theme: ThemeData.dark(),
      );
}

class SignInPage extends StatelessWidget {
  const SignInPage({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) => Scaffold(
        appBar: AppBar(
          title: Text('Sign In'),
          centerTitle: true,
        ),
        body: Column(
          children: [
            TextField(),
            ElevatedButton(
              onPressed: () => QR.toName('forgot-password'),
              child: const Text('Forgot-Password'),
            ),
            ElevatedButton(
              onPressed: () => QR.navigator.replaceLastName('sign-up'),
              child: const Text('Sign-Up'),
            ),
            ElevatedButton(
              onPressed: () => QR.navigator.replaceLastName('dashboard'),
              child: const Text('Login'),
            ),
          ],
        ),
      );
}

class SignUpPage extends StatelessWidget {
  const SignUpPage({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) => Scaffold(
        appBar: AppBar(
          title: Text('Sign Up'),
          centerTitle: true,
        ),
        body: Column(
          children: [
            TextField(),
            ElevatedButton(
              onPressed: () => QR.navigator.replaceLastName('sign-in'),
              child: const Text('Sign-In'),
            ),
            ElevatedButton(
              onPressed: () => QR.navigator.replaceLastName('dashboard'),
              child: const Text('Login'),
            ),
          ],
        ),
      );
}

class ForgotPassword extends StatelessWidget {
  const ForgotPassword({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) => Scaffold(
        appBar: AppBar(
          title: Text('Forgot password'),
          centerTitle: true,
        ),
        body: Column(
          children: [
            ElevatedButton(
              onPressed: () => QR.back(),
              child: const Text('Go back'),
            ),
          ],
        ),
      );
}

class Dashboard extends StatelessWidget {
  final QRouter router;
  const Dashboard({Key? key, required this.router}) : super(key: key);

  @override
  Widget build(BuildContext context) => Scaffold(
        appBar: AppBar(
          title: Text('Dashboard'),
          centerTitle: true,
        ),
        body: router,
      );
}

Hi @SchabanBo, any updates on this?

Hi @tejHackerDEV,
Thanks for asking, I forget this totally.
This is fixed now in dev and will be out with the next release soon.
Btw: Great reproducible code and great explaining, thanks a lot.

@SchabanBo, thanks for the fix. Also I was thinking to contribute the project once I get free from my office works. So are you accepting any maintainers for now? Because I will be happy to do so once I get free (which will happen after some weeks)

Actually, that would be very good. I would be happy with any help anytime 😊.
I am struggling too to find time for my open source projects.

Hi @SchabanBo, first of all thanks for the fix & I am happy to take maintainer ship of the project once I get free from my office works. Thank you


replace golang purge script with powershell script

Description
Swap from GraphAPI-based data cleanup (which doesn't allow
us to access hidden folders like Audit retention or delete-restoration)
to powershell script-based cleanup (which allows us to make
SOAP requests against legacy exchange apis).
Type of change

[x] :bug: Bugfix
[x] :robot: Test

Issue(s)

#1266

Test Plan

[x] :muscle: Manual
[x] :green_heart: E2E

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: ryanfkeepers:x: gmatevYou have signed the CLA already but the status is still pending? Let us recheck it.


Replace the usage of commons-collections to commons-collections4

Instructions:

The PR has to be tagged with at least one of the following labels (*):

feature
bugfix
performance
ui
backward-incompat
release-notes (**)

Remove these instructions before publishing the PR.

(*) Other labels to consider:

testing
dependencies
docker
kubernetes
observability
security
code-style
extension-point
refactor
cleanup

(**) Use release-notes label for scenarios like:

New configuration options
Deprecation of configurations
Signature changes to public methods/interfaces
New plugins added or old plugins removed

Codecov Report
Attention: Patch coverage is 0% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 34.49%. Comparing base (59551e4) to head (23539cc).
Report is 161 commits behind head on master.

Files
Patch %
Lines

...pache/pinot/core/util/PeerServerSegmentFinder.java
0.00%
1 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #12711       +/-   ##
=============================================
- Coverage     61.75%   34.49%   -27.26%
+ Complexity      207        6      -201
=============================================
  Files          2436     2381       -55
  Lines        133233   130758     -2475
  Branches      20636    20253      -383
=============================================
- Hits          82274    45108    -37166
- Misses        44911    82318    +37407
+ Partials       6048     3332     -2716

Flag
Coverage Δ

custom-integration1
?

integration
<0.01% <0.00%> (-0.01%)
:arrow_down:

integration1
<0.01% <0.00%> (-0.01%)
:arrow_down:

integration2
0.00% <0.00%> (ø)

java-11
34.49% <0.00%> (-27.22%)
:arrow_down:

java-21
<0.01% <0.00%> (-61.63%)
:arrow_down:

skip-bytebuffers-false
34.49% <0.00%> (-27.26%)
:arrow_down:

skip-bytebuffers-true
<0.01% <0.00%> (-27.73%)
:arrow_down:

temurin
34.49% <0.00%> (-27.26%)
:arrow_down:

unittests
46.09% <0.00%> (-15.65%)
:arrow_down:

unittests1
46.09% <0.00%> (-0.80%)
:arrow_down:

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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


apply: better define what "early-boot" means

I submitted the initial service unit I'm using in https://github.com/coreos/torcx/pull/34. As part of system integration, we need to define to better define the "early-boot" story and capture it at in devdoc.
I'm currently using this very weak anchoring:
WantedBy=sysinit.target
RequiredBy=basic.target

Other datapoints:

torcx.service is a oneshot unit, so it'll block targets progression till the final sealing is done (or the service fails)
some addon services handled via torcx will be part of the dependency graph (docker, containerd, rkt-api) so it would be better to extract them as early as possible
torcx-apply doesn't fetch, so it doesn't strictly need a network'd target
torcx-apply doesn't depends on dbus service/socket
torcx-apply needs local-fs to be in place, in particular it relies on /tmp, /var and /run
some users may provide images via a NAS volume, thus depending on remote-fs.target. I think this usecase can and should captured via an user drop-in

The outcome of this ticket would likely be an upper and lower bounds (as in targets and services) to place torcx.service in-between.
/cc @euank @crawford @marineam

This is going to take some careful thought.
For example, the sockets.target starts very early in the boot process. We're basically going to want to start before any other target, so we can install any "wants".

A couple things, ordering is managed by Before/After, not wanted/required.
Changing service dependencies during boot, although possible, is a very dangerous. We abandoned coreos-cloudinit for this very reason.
It sounds to me like this is a sufficently constrained job that it may be practical to run it as a generator instead of a service. Generators run right when systemd starts so you can only depend on /etc and /run (you will be given a set of paths in /run as arguments where the app can write service files and create .wants links.) Be warned, at this point even /tmp is not reliably available. The key is running so early is you ensure systemd actually respects the changes you make. Making dependency and service charges any later is essentially a sure fire way to have an endless stream of bugs.

One thought that I had while Luca and I were staring off in to space: Perhaps all packaged services need to be wanted by a special torcx-package.target. We might be able to constrain the dependency changes sufficiently that we don't find ourselves in such a world of hurt.
And, maybe not.

Current status is that we are slowly adapting torcx to be a generator (see https://github.com/coreos/torcx/pull/37), with a small twist.
As we don't plan to handle runtime upgrades, generated units are written directly to the runtime directory only on the first run before sealing the system. On subsequent daemon-reload, generator detects that the system is already sealed and just stops there. We also rely on the fact that inignition and cl-initrd already set up / and /usr, to pick up vendor packages and other bits from there. Apart from that, we only ever unpack/propagate files to /run.

Just a final update: torcx is now completely run as a generator; this is done.


Selecting multiple error codes is not obvious to the user - Publisher - Endpoint Advance Configuration

It's a better user experience if we can provide checkboxes with the dropdown.

We update from the component given with the MUI documentation.
https://v4.mui.com/components/selects/#multiple-select

Fixed with PR https://github.com/wso2/apim-apps/pull/209


Add WebNN-native binding for Node.js to expose WebNN JavaScript API

webnn-polyfill has 600 test cases to test APIs, ops and models. To avoid porting of these tests to C++, it would be useful to run these JS tests with node.js. The WebNN-native binding is to bind WebNN-native C++ API with addon.
This PR depends on https://github.com/webmachinelearning/webnn-native/pull/10.

Thanks much @fujunwei!
Current webnn-polyfill@5c5e9f6 has 615 test cases (614 passing / 1 pending by WebNN Polyfill API):

API tests  total: 84, passing 84
Ops tests  total: 266, passing 266
Models tests total: 15, passing 14, 1 pending
CTS tests total: 250, passing 250

Following node/README.md, I verified this PR:

built webnn-native by DML backend, when executing npm test command to run all tests of webnn-polyfill,
got 386 passing / 1 pending / 228 failing
built webnn-native by OpenVINO backend, when executing npm test command to run all tests of webnn-polyfill,
got 385 passing / 1 pending / 229 failing

Let's look at pass status of running Ops tests and Model tests:

Backend
Ops tests (npm run test-ops)
Model tests (npm run test-models)

DML
128 passing 138* failing
14 passing 1pending

OpenVINO
126 passing 140* failing
14 passing 1 pending

Above '14 passing 1 pending' of running Model tests are same as test result by WebNN polyfill API.
And 138 failing ops tests by DML backend and 140 failing ops tests by OpenVINO backend are most with unimplemented op features likes conv2d support transpose option, and those unimplemented ops (div/exp/gru/instance_norm/max/min/pad/pow/reduce/resample/sigmoid/slice/split/squeeze/sub/tanh/transpose) by WebNN native.

Based on the feedback from WebML WG Teleconference – 24 June 2021, both the webnn-polyfill tests and webnn-native implementation on native ML APIs are useful to inform the spec development. This PR specifically enables running the webnn-polyfill tests on native ML API implementation. With that, I am going to merge this PR and unlock the development of other features, like new API change and ops.
Thanks much @fujunwei and @BruceDai !


Speech recognition

Use DeepSpeech to generate a file with text and timestamps of each word (like in #3)

DeepSpeech alternatives to check:

https://github.com/flashlight/flashlight
https://github.com/pytorch/fairseq

https://github.com/pytorch/fairseq/tree/master/examples/speech_to_text

https://huggingface.co/transformers/model_doc/speech_to_text.html

https://github.com/espnet/espnet
https://github.com/alphacep/vosk-api


Add rendering captures in documentation examples

Include captures that show the expected rendering for the examples exposed in the documentation.

First pass of captures added in https://github.com/metanorma/metanorma.com/pull/286

@ronaldtse , @opoudjis
I have added the renderings I consider necessary to metanorma.com, except for the requirement blocks from Requirement, Recommendation, and Permission blocks. I didn't add those because I'm not sure if the rendering is really finished.
For example, this code:
[.requirement]
====
[.specification]
--
This is a formal specification
--

[.measurement-target]
--
This is a measurement target
--

[.verification]
--
This is a verification step
--

[.import]
--
This is a code stub
--

====

results in:
DOC output:

@opoudjis , is this the final result?

@ronaldtse will need to confirm, but my understanding is, yes.


After pause or resume the page is stucked

Hi,
Referring to pause and resume: https://github.com/felixmosh/bull-board/issues/214
After pressing pause or resume I can see the popup for confirmation, the queue get paused or resumed but the page get stucked.
For example I paused the queue "{null}-alert-error-tor" and then pressing on the queue "...external-notification" (or everything else) nothing happens:

I'm using version 3.5.3, browser both chrome and firefox.
Thanks for the package and the support
ciao
Sam

Thank you for reporting this issue 🙏🏼
Can you open the dev tools and check if there is any error in the console?

No messages in console.
I temporary published this url:
https://ghsrrfkiujqpdofyxhqf.cfapps.eu10.hana.ondemand.com/jobs-monitor
It's a sandbox, you can try to pause and resume as you want.

Confirmed, thank you, some weird bug with radix-ui.
I'll investigate it.

Thanks, can I remove the url?

Yeap

Removed, let me know how I can help, I'm glad to contribute to it.
Thank you
Sam


Should string mode be removed? It kind of seems unnecessary

String assignment can already be done with literal mode. The string mode allows to not put quotes ; this essentially saves 2 characters at the expense of readability.

Conclusion: yes


Feature/custom branding

Coverage decreased (-1.5%) to 77.926% when pulling fca68726cc254c3d6ed93f009eb95bd0724b59ed on feature/custom-branding into a075b6e22ee4b4755fec23cdeb62985921d9c944 on master.


Gender buttons doesn't switch

https://colorlib.com/polygon/gentelella/form_wizards.html

It's not a formal bug. There is no source code to handle this switch.
Male button has a class btn-primary which is incorrect. The submit button has such a class too. They share the same color.
I'll propose removing class replacing class btn-primary with btn-default and removing abstraction classes like data-toggle-passive-class. After this visual marker will be class 'active' which is bootstrap default class.
If it's ok I can handle a PR. The same in form.html

Buttons are switching but because of the background-color you cannot say exactly which is which in terms of selection , I will make this better , thank you @lordspace for reporting this.

Cool


Add Ruby 3.3 testing

Summary
Provide a detailed description of all the changes present in this pull request.
Additional Context
Add any additional context about the problem here.

[ ] Root cause and the steps to reproduce. (If applicable)
[ ] Thought process behind the implementation.

Related Issues (if any)
Mention any related issues or pull requests.
Checklist

[ ] 🟢 Spec tests.
[ ] 🟢 Acceptance tests.
[ ] Manually verified.

Hey @bastelfreak, as a team we have had a chat and we plan on continuing to test in line with the versions of Ruby that the agent is shipped with (https://www.puppet.com/docs/puppet/7/platform_lifecycle#about_agent) to keep in sync.
LTS: 7.29.1 Ruby 2.7.8
Latest: 8.5.2 Ruby 3.2.3
We will update the versions accordingly if and when required, however we have agreed that adding Ruby 3.3 to the test matrix would not be beneficial enough for us to warrant the additional load across our suite of tools. If you have any strong feeling about this please do not hesitate to reach out to myself or the team.

Hi @pmcmaw ,
I understand that it makes sense to use the Ruby versions used in the Puppet Agent AIO packages for unit tests. Vox Pupuli does the same. However I think that doesn't make sense for gems like puppet-lint. puppet-lint isn't used during unit tests. puppet-lint is used for static code validation in CI and locally. And my impression is that people install puppet-lint locally with their system Ruby, so puppet-lint should support it. Ruby 3.3 got released almost half a year ago and distributions start to ship it or prepare it. Also it doesn't make sense / doesn't bring any benefit to use an old Ruby version in CI for static code analysis. In contrast the old versions are often even slower compared to new MRI versions.
And does that add so much more load on the CI setup? And the jobs run on the public GitHub runners, does Puppet/Perforce pay per runner?

@pmcmaw I think as a developer you should always be forward looking. We're not asking you to track Ruby development versions, but at least be compatible with the latest tooling available. Fedora 40 will ship it with Ruby 3.3 and puppet-lint is a developer tool. As a developer I want to be able to use the latest and greatest.
As a bonus, once Puppet does want to update its bundled version, you're already prepared for it. It won't be a big deal since over time you've done the work. If you don't update, it becomes a big deal and you're more likely to push off to the future.

@ekohl @bastelfreak I am following up on this particular request and wanted to drop a comment to acknowledge that I am not ignoring yous. 👍


Lookahead manual calculations

Add a comparison with the correct answer for current implementation of lookahead algorithm

Didn't pull main repository, made calculation with average, not median. Need to recalculate

@Ksenia-C ветка с Lookahead и другими алгоритмами вмержена в main, перенесите ваши правки туда.


go build error

[root@MyCloudServer coredns]# go build
middleware/etcd/etcd.go:16:2: cannot find package "github.com/coreos/etcd/client" in any of:
/root/goc/src/github.com/miekg/coredns/vendor/github.com/coreos/etcd/client (vendor tree)
/root/go/src/github.com/coreos/etcd/client (from $GOROOT)
no etcd in coredns

did you go get?

Just did a clean build on a new systems. Just works.


topn with granularity regression fixes

Description
changes:

fix issue where topN with query granularity other than ALL would use the heap algorithm when it was actual able to use the pooled algorithm, and incorrectly used the pool algorithm in cases where it must use the heap algorithm, a regression from #16533
fix issue where topN with query granularity other than ALL could incorrectly process values in the wrong time bucket, another regression from #16533

This PR has:

[x] been self-reviewed.
[x] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[x] been tested in a test Druid cluster.

Since @gianm comments were non blocking, going ahead with merge since this blocks 31.0.1 release.


Make sure the storage keyword is not valid in scripts.

Storage declarations are now forbidden in non-contracts (see https://github.com/FuelLabs/sway/pull/1874). The reset follows naturally.


Replace the Fetch API by Apollo Client

Apollo Client would give a better and more consistent experience

Actually we could leverage Fetch and a handler (ctx).


Replace the terminology between shared non-conforming slave and master

This PR improves further the new Mesh::FaceInformation by associating location and conformity to both elem1 and elem2 for a given face, this should make the Mesh::FaceInformation even easier to process by a human.
This PR also proposes to replace the concepts of slave and master faces for two main reasons:

These terms are politically loaded, especially in the US due to its history,
This terminology is hard to grasp, indeed slave and master refer to how the faces are created in the code, therefore understanding how the code works is a prerequisite to understand the documentation (?!). Whereas replacing these concepts by fine and coarse describes what the faces are from a mathematical/geometrical perspective. Furthermore, slave and master are not sufficient to describe all the possible face configurations, and thus a third concept of ghost faces (also not trivial to grasp) is needed to resolve the ambiguity. However, when describing faces in term of coarse and fine, all the possible face configurations can be described (non-ambiguously) without requiring a third concept.

This PR should be reviewed after #2195 , or #2769 should be merged in #2195 .

I'd like to edit this PR and start its review, and I generally agree that renaming slave/master to fine/coarse may be a good idea, but I am confused -- there are way to many changes here which look much more significant than renaming.
What am I missing?
Was this PR intended to be towards a different branch than master (maybe yohann/pa-dg-ncmesh)?

I'd like to edit this PR and start its review, and I generally agree that renaming slave/master to fine/coarse may be a good idea, but I am confused -- there are way to many changes here which look much more significant than renaming.
What am I missing?
Was this PR intended to be towards a different branch than master (maybe yohann/pa-dg-ncmesh)?

@tzanio, see the comment here: https://github.com/mfem/mfem/pull/2767#issuecomment-1013558033
Basically this PR is the same as #2769, however, its target is the master branch.
I prefer to merge #2769 into #2195 (after resolving the discussions there) and then that into master. So this PR can be closed.

This PR should be reviewed after #2195 , or #2769 should be merged in #2195 .

@tzanio Yes, this was the reason for that comment.

Can this PR be closed in favor of #2769?


Color simple polymers

To better distinguish e.g. the 4 chains of a more complex antibody format it would be quite helpful to assign different colors to the 4 different simple monomers. This would override the default light blue that represent amino acids in the HELM editor.
Another useful alternative but only optional setting is the coloring of protein simple monomers using a color code for the physico-chemical properties of the amino acids.

solved on antibody level by HELM Antibody Editor


⚠ ClamAV has degraded performance

In 1deea51, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 174 ms

Resolved: ClamAV performance has improved in 2d82e14.


Test access to NetSuite for C+

Can you please:

Sign into 1Password to view the credentials for NetSuite
Sign into NetSuite
Do something to see if you're able to test NetSuite like you have in the past when you worked on the integration
Provide feedback in a comment on this post.

Thx

✅ I confirm 1Password works for both sign-in into NetSuite and also for displaying NetSuite credentials for Expensify (see attached video for the latter).
https://github.com/user-attachments/assets/d26947c3-0d33-4ecb-bf5c-70f1f0b10baa

Thanks @ikevin127 , I deleted the vid cuz token/credential details were showing (unsure if that matters but erring on the side of caution)

The NetSuite credentials worked fine for me and I was able to connect a workspace successfully.

Sweet, looks like we're good here. If anyone has issues, please post about here or in #contributor-plus.   Thx!


impl pg array append

impl array_append
allow non nullable range - can be discarded if we don't want the polymorphism
typo - the CI pointed out the typo and I think it isn't worth opening a new PR for that.

Ah I thought there was only one failing but there were 2... Probably that will fail again then...
It's only on Ubuntu so I would suspect an ubuntu-related package upgrade...

Ah I thought there was only one failing but there were 2... Probably that's not random will fail again then...
It's only on Ubuntu so I would suspect an ubuntu-related package upgrade...
That's the difference between the two runs during PG install:

Nothing striking...
We'd have to get the segfault's backtrace. In any case it seems very unlikely that it's related to this PR so we can probably merge this anyway.


Remove redundant FamPlex-MeSH predictions and mappings

This PR removes FamPlex-MeSH predictions and mappings that are available via FamPlex directly (as a "primary" source) and should therefore not be duplicated here. It also adds some CHEBI-MeSH curations. Fixes #28.

two questions:

We can make mesh-pubchem.compound then pubchem.compound-chebi equivalences with PyOBO. If it's two hop, should we call them predictions?
When we make PRs for curations, should we do second curator agreement? Or should we use PRs just as a place to check in?

The changes look good to me from the surface

Good questions,

That's an interesting point. I guess it depends on what our criterion is for something being added to Biomappings. Until now, I thought that we would consider a mapping here redundant if the same is available from a specific primary source. For example, in this PR, the fplx-mesh mappings were removed because these same mappings are available from FamPlex. The category you're bringing up is a little bit different in that it implies that series of mappings available from distinct primary sources combined by a "secondary" source like PyOBO could also be considered redundant. I think that's definitely reasonable but we should then try to automate the process of flagging/removing these redundancies somehow since they are non-trivial to detect.

In the specific case of pubchem.compound-mesh-chebi, I think some mappings will be missing and can therefore be curated here. Randomly looking at some examples, I found that e.g.,
mesh	D001455	Bambermycins	skos:exactMatch	chebi	CHEBI:28908	bambermycin

can be reproduced via the chebi-pubchem-mesh path but e.g.,
mesh	D003358	Cosmetics	skos:exactMatch	chebi	CHEBI:64857	cosmetic

cannot.

I don't think second curator agreement is necessary, I think the automated testing, the open PR review process, plus the fact that the resource can be directly/publicly edited should be enough to weed out issues. Having said that, I think it might be useful to flag certain predictions for a "second opinion" where another curator could come in and weigh in on the right choice; the list of such predictions could even be exposed for crowdsourcing in innovative ways (twitter, Slack, etc.). I think most predictions are pretty straightforward though and one curator can confidently decide on it without requiring additional labor.


Improve expander animations

Is your feature request related to a problem? Please describe.
See the following FluentAvalonia animation:

It looks okay, however not as nice as the settings app animation:

Describe the solution you'd like
Updated animations to more closely match the settings app.

It was an intentional decision not to include the animation within SettingsExpander. Because of the way Avalonia implemented the animation of the actual expander control, it doesn't work quite right in all situations with this control (particularly if you want the implicit composition animations to make the rest of the UI move with the expansion/collapse).
You can still enable it yourself with a Selector that targets the Expander in the template of the SettingsExpander to set the ContentTransition property.

@amwx Thanks for the reply! If I may ask, is there an Avalonia issue to track this as an issue, or is it inherent to the design?


Session Replay masking does not mask input placeholders

Expected Behavior
All text displayed in an input should be masked if using conservative or amp-mask class
Current Behavior
Placeholder text is not masked. This is an issue for searchable dropdown components which may update the placeholder value to the selected option.
Steps to Reproduce

create an input with placeholder text and apply amp-mask

Environment

"@amplitude/session-replay-browser": "1.13.9"
"react": "17.0.2",

Seems reasonable. We have an internal ticket to track this. We'll update this Github ticket accordingly. Thanks for the feedback!

Hey @GxDesign as I was going through the implementation, which I'd like to get merged upstream I saw this issue being raised for rrweb. If it gets accepted we'll pick up the upstream change. If it gets rejected I may go ahead with my implementation. Hope that helps!


XML instance inspection: Primitive fields under collection parent has duplicate collection flag

/root/firstArray<>/Value and /root/firstArray<>/secondArray<>/value themselves should not have CollectionType.LIST although their parent is collection. This issue causes wrong nest level.

XML nested collection
<?xml version="1.0" encoding="UTF-8"?>
<root>
  <firstArray>
    <value>firstArrayValue0</value>
    <secondArray>
      <value>secondArrayValue0-0</value>
      <thirdArray>
        <value>thirdArrayValue0-0-0</value>
      </thirdArray>
      <thirdArray>
        <value>thirdArrayValue0-0-1</value>
      </thirdArray>
    </secondArray>
    <secondArray>
      <value>secondArrayValue0-1</value>
      <thirdArray>
        <value>thirdArrayValue0-1-0</value>
      </thirdArray>
      <thirdArray>
        <value>thirdArrayValue0-1-1</value>
      </thirdArray>
      <thirdArray>
        <value>thirdArrayValue0-1-2</value>
      </thirdArray>
    </secondArray>
  </firstArray>
  <firstArray>
    <value>firstArrayValue1</value>
    <secondArray>
      <value>secondArrayValue1-0</value>
      <thirdArray>
        <value>thirdArrayValue1-0-0</value>
      </thirdArray>
      <thirdArray>
        <value>thirdArrayValue1-0-1</value>
      </thirdArray>
      <thirdArray>
        <value>thirdArrayValue1-0-2</value>
      </thirdArray>
    </secondArray>
    <secondArray>
      <value>secondArrayValue1-1</value>
      <thirdArray>
        <value>thirdArrayValue1-1-0</value>
      </thirdArray>
      <thirdArray>
        <value>thirdArrayValue1-1-1</value>
      </thirdArray>
    </secondArray>
  </firstArray>
</root>

Interestingly ..thirdArray<>/value doesn't have CollectionType.


Implement support for multi-polygons

Like spherical_geometry and shapely do

@cdeil - I wonder if we could say that PixelPolygonRegion should be a single region and this could be implemented with composite regions?

Are compound regions of disjoint polygons supported through CompoundPixelRegion and CompoundSkyRegion?


Problem : type error at reading CSV header

Problem : type error at reading CSV header
already defined COLUMN_TYPE = OrderedDict() and header = keys(COLUMN_TYPE)
in function download_data(...)
julia> data = CSV.File(read(r.files[1]); delim="\t", header, select)
ERROR: TypeError: in keyword argument header, expected Union{Vector{String}, Vector{Symbol}, Integer, AbstractVector{<:Integer}},
got a value of type Base.KeySet{Symbol, OrderedDict{Symbol, DataType}}
Stacktrace:
 [1] top-level scope
   @ ~\Documents\Julia\Kaggle\BirdCLEF2023\geoinfo.jl:52

it is ok to do this by converting keys into vector.
header = collect(keys(COLUMN_TYPE))
julia> data = CSV.File(read(r.files[1]); delim="\t", header, select)

(related to  #12 (comment))

True, I will fix this in the next version, along with #12. Thank you for documenting these issues.


grep 2.25

Merged in https://github.com/Homebrew/homebrew-dupes/commit/85039293439d5be2c304edd1e0a817dcd2071143. Thank you @chdiza!


Add support for babel-macros

This takes care of at least some of https://github.com/threepointone/glamor/issues/301.
Wanted to start playing with babel-macros so figured I'd work on adding support here.
Not sure if the usage is as intended, so maybe also pinging @kentcdodds for guidance. Especially for the way I introduce the import of the actual runtime. Had to reach out to program for this.
Also another question I have is when to perform the hoisting, since we don't want that during development I suppose. Maybe use process.env.NODE_ENV? Maybe BABEL_ENV? Maybe babel-macros should have an env that gets passed to the macros?
One more thing, is that this won't catch the css attribute in JSX elements, as babel-macros only gets us the Identifier nodes that are the references from the macro import. I could reach out to the Program node to perform the hoisting in the entire file, but that feels like abusing babel-macros. Maybe babel-macros should also give us
Also needs a lots of testing, will try and see how to add some.
Even if this is still far from what you were thinking, maybe it helps a bit as a starting point for your experiments. Hope it's at least a bit helpful!

I'll wait for kent's comments on how best to import the lib

I think it's fine for the the hoisting to always happen, unless you can think of a problem?

I expected the hoisting to not happen for css props, and that's fine for now, it shouldn't be a responsibility of this macro (and we'll revisit it when we figure out how to doreader macros for babel-macros)

thanks for the PR, excited to get this in!

(I also wanted to take this opportunity to use SC/emotion's css parser instead of our homegrown one, but that can wait I suppose)

Thanks for the quick feedback!
The only concern I can think of for always hoisting is weird behavior when debugging. If some call gets hoisted, you may have problems trying to set a breakpoint in there. Not sure how often that happens, though.
I'm currently working on avoiding traversing by just calling the transformations directly on the paths we're given by babel-macros. This will also make it work if we alias the identifier to something else, like import { css as whatev } from 'glamor/macro'

iirc the hoisting should only hoist the input object, not the actual css() call, so breakpoints still work, but I'll have to verify. in any case, don't let that stop you. if I had to pick, I'd do it on NODE_ENV, not BABEL_ENV.

Not sure if the usage is as intended, so maybe also pinging @kentcdodds for guidance. Especially for the way I introduce the import of the actual runtime. Had to reach out to program for this.

Yep, that's the way to do that kind of thing. It's not supposed to be easy to modify areas of the code that your macros isn't touching :wink:

Also another question I have is when to perform the hoisting, since we don't want that during development I suppose. Maybe use process.env.NODE_ENV? Maybe BABEL_ENV? Maybe babel-macros should have an env that gets passed to the macros?

I think you could just handle that yourself in your macro. Check process.env.NODE_ENV and do whatever you need.

One more thing, is that this won't catch the css attribute in JSX elements, as babel-macros only gets us the Identifier nodes that are the references from the macro import. I could reach out to the Program node to perform the hoisting in the entire file, but that feels like abusing babel-macros.

Yes, this is a limitation and honestly I don't think that there's anything I can do for that... Maybe you could do: <div {...[css]: {}} /> but that'd be way weird...

Ok I've added support for custom aliases when importing the macro.
I wanted to avoid traversing the paths babel-macros give us to convert the template strings with the visitor (and just perform the path transformation), but I was getting an error with nested calls to css. This was due to the parser grabbing the stubs from the source using start and end positions, which get messed up when doing intermediate transformations (https://github.com/threepointone/glamor/pull/312/commits/d26497c8594a7acc9732e13850e576d9f871fe41#diff-e09bf259105b708ada46f5bdeb5f932fR132).
Basically I wanted to do this:
references.css.forEach((path) => {
  if (path.parentPath.type === 'TaggedTemplateExpression') {
    transformToCallExpression(path.parentPath);
  }

  if (parentPath.type === 'CallExpression') {
    hoistCallExpression(parentPath);
  }
});

But had to resort to:
references.css.forEach((path) => {
  path.parentPath.parentPath.traverse(babelPlugin.visitor) // This converts all the subtree (so, including nested expressions)

  if (parentPath.type === 'CallExpression') {
    hoistCallExpression(parentPath);
  }
});

Maybe that was one of the reasons you wanted to use the other parsers? Haven't looked at them yet, but maybe they just manipulate the AST without relying on the source and this won't be a problem.

Ok I've added support for custom aliases when importing the macro.

If I understand you correctly, you want to support:
import {css as myCSS} from 'glamor/macro'

If that's what you mean by this then you should probably know that babel-macros supports this out of the box, you shouldn't have to do anything to handle this.

@kentcdodds yes! Actually the fact that babel-macros supported it out of the box was the reason I decided to support it. The babel plugin and the hoist plugin were using hardcoded names for the function, so that's part of the work I did.

Ah, gotcha :+1: coolio!

I know this stretches the scope a bit, but would you also like to take a crack at automatically adding labels to the rules (based on var being assigned to, if available, or any other idea you might have)? no worries if you don't, but would make this so compelling to use.

Yeah sure I can take a stab at it. Do we want to make the current babel plugin to also have this functionality? Also, do we want to add labels also in the object-style calls?
I'm thinking we can do what you suggest. Another one that may be helpful could be adding labels when applied to a JSX element, we can combine the element name with "className", or just the element name, for cases where you inline the css call inside it either with spread or assigned to the className.
Regarding the macro itself, anything else you see missing? What do you think about the approach I had to take because of https://github.com/threepointone/glamor/pull/312#issuecomment-320036417?

sorry have to ignore this PR today, just a bit swamped with work and stuff, will reply over the weekend. but tldr - your approach sounds good.

Sure! no rush with this!

I know you may still be busy with some other stuff, this is just for you to read it whenever you have time.
I've added some code to automatically add labels when you have a variable declaration. I tried doing it for css directly inside the className of a jsx element, but realized it wouldn't be very useful as the label would be "div", "span", "a", ie not very helpful. I can add any other strategy we can come up with, I'll think about it a bit more.
Should we disable the labels when compiling for prod?
Also, I want to add some tests for this, and wanted to use, as @kentcdodds suggests in babel-macros, the babel-plugin-tester, but that suggests to use jest so we can benefit of snapshot testing. So I'll wait for your "approval" or not to use these libraries in this repo. If not, I can probably add some regular mocha tests.
Again, no rush on this, will be around when you have time to answer

Hey folks! babel-macros 1.0.0 has been released. You'll probably want to check the docs. Pretty much all you'll need to change is wrap your macro in createMacro from const {createMacro} = require('babel-macros')
Cheers!


Getting no test targets to run

I am getting -
Bad state: no test targets to run
#0      FlutterTestRunner.run (package:patrol_cli/src/features/drive/flutter_test_runner.dart:105:7)
#1      DriveCommand.execute (package:patrol_cli/src/features/drive/drive_command.dart:310:39)

#2      CommandRunner.runCommand (package:args/command_runner.dart:209:13)

#3      PatrolCommandRunner.runCommand (package:patrol_cli/src/command_runner.dart:291:18)

#4      PatrolCommandRunner.run (package:patrol_cli/src/command_runner.dart:235:18)

#5      patrolCommandRunner (package:patrol_cli/src/command_runner.dart:52:14)

Duplicate of https://github.com/leancodepl/patrol/discussions/795


docs(typography): Improve documentation for Typography page

Rephrase sentences and adjust structure for several sections.

Codecov Report

Merging #199 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #199   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          20     20
  Lines         379    379
=====================================
  Hits          379    379

Continue to review full report at Codecov.

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


Groovy-7574: Ranges: Make Range members final. Also improve javadoc.

Else they can easily be changed in Groovy, which leads to undefined states
supercedes #109

merged, thanks


SSFO: Dump when smartforms is locked

If a smartforms can't be enqueued during deserialization a dump occurs.

Suggest to catch CX_SSF_FB.
https://github.com/larshp/abapGit/blob/f14ac831bacd969082ac4cf7451ff581949ca808/src/objects/zcl_abapgit_object_ssfo.clas.abap#L255-L269

also see https://github.com/larshp/abapGit/issues/437


Set IP public/private classifier to be compliant with RFC 1918.

Per #151. Also broke out the "private IP check" as its own function (refactoring related checks) with tests to ensure that not all 172- and 192- prefix IPs are classified as public.  Resubmit of previous PR #153 because a function was mis-documented.
Preserved prefix checking for determining private/public rather than these approaches in order to preserve similarity to Axes module's approach.

Very nice, thank you.


"The Unity environment took too long to respond..." error when running examples.

I am unable to get any of the example agents or the simulation environment to open on my machine and am not entirely sure why. I am running Ubuntu 16.04 and I have attempted to create a Dockerfile with all of the necessary components (per the Installation Instructions) as follows:
FROM tensorflow/tensorflow:1.12.3-gpu
MAINTAINER Justin VanHouten ()
RUN add-apt-repository ppa:deadsnakes/ppa
RUN apt-get update
RUN apt-get install -y nano
RUN apt-get install -y wget
RUN DEBIAN_FRONTEND=noninteractive apt-get install -y xorg openbox
RUN apt-get install -y git-core
RUN apt-get install -y python3.6
WORKDIR ..
RUN wget https://bootstrap.pypa.io/get-pip.py
RUN python3.6 get-pip.py
RUN pip install opencv-python
RUN git clone https://github.com/beyretb/AnimalAI-Olympics.git
WORKDIR AnimalAI-Olympics
WORKDIR animalai
RUN pip install -e .
WORKDIR ..
WORKDIR examples/animalai_train
RUN pip install -e .
WORKDIR ..
WORKDIR ..
RUN wget https://www.doc.ic.ac.uk/~bb1010/animalAI/env_linux_v1.0.0.zip
RUN unzip ./env_linux_v1.0.0.zip -d env
If I then run a container based on this Dockerfile, cd into /examples and use the command "python3.6 [any agent python script] configs/[any premade YAML]" I get the error "The Unity environment took too long to respond...".
If I cd into /env and enter command "./AnimalAI.x86_64" it appears to run but nothing happens; I can Ctrl-Z out of it but it doesn't throw any errors.
When I open the log file, the only line says that "Desktop is 0 x 0 @ 0 Hz" which indicates to me that it isn't detecting a display. My questions are:

Is my assumption about not detecting a display correct?
Can I rectify this problem through certain package installations? Is there a way to discern which packages I would need?

Thank you in advance for any help or guidance you can provide; I've been over the Installation and Quick Start guides a dozen times and am not really sure what I'm missing.

Hello,

Is my assumption about not detecting a display correct?
Correct, you need to emulate a virtual screen which on linux can be done using xvfb.

Can I rectify this problem through certain package installations? Is there a way to discern which packages I would need?
I would recommend having a look at the submission docker file which you need to build for submitting your agent. It can also be used for training on your machine or in the cloud.

Don't forget to set docker_training=True when creating your environment so that the environment can launch via xvfb

closing this issue due to inactivity


IMF data request error

I'm trying to get pandasdmx (v0.9, using pandas 0.23 and python 3.7) up and running for some IMF data requests. In particular, accessing the BOP (BPM6) series.
#!/usr/local/bin/python
import pandas as pd
import pandasdmx

countries = {'Australia': 'AU', 'Austria': 'OE', 'Belgium': 'BE', 'Canada': 'CA', 'Hong Kong': 'HK', 'Denmark': 'DK', 'Euro area': 'EM', 'Finland': 'FN', 'France': 'FR', 'Germany': 'BD', 'Greece': 'GR
', 'Ireland': 'IR', 'Israel': 'IS', 'Italy': 'IT', 'Japan': 'JP', 'Netherlands': 'NL', 'New Zealand': 'NZ', 'Norway': 'NW', 'Portugal': 'PT', 'Singapore': 'SG', 'Spain': 'ES', 'Sweden': 'SE', 'Switzer
land': 'CH', 'United Kingdom': 'UK', 'United States': 'US'}

imf = pandasdmx.Request('IMF_SDMXCENTRAL')
bop_flow = imf.dataflow('BOP_BPM6')
dsd = bop_flow.dataflow.BOP_BPM6.structure()

# Get a list of regions/areas from country list
areas = bop_flow.write().codelist.loc['CL_REF_AREA']
areas = areas[areas['name'].isin(countries.keys())]

# Get  a list of indicators for each of the specific BOP series of interest
codes = bop_flow.write().codelist.loc['CL_INDICATOR']
codes = codes[codes['name'].str.contains('Balance of Payments', case=False) & codes['name'].str.contains('Financial Account', case=False) & codes['name'].str.contains('Portfolio Investment', case=False) & codes['name'].str.contains('Net acquisition of financial assets', case=False) & codes['name'].str.contains('US Dollars', case=False)]
codes = codes[codes.index.str.contains('BP6')]

# Pull down data into a dataframe
for code in list(codes.index):
resp = imf.data(resource_id='BOP_BPM6', key={'INDICATOR': code, 'REF_AREA': list(areas)}, dsd=dsd)
data = resp.data
quarterly = (s for s in data.series if s.key.FREQ == 'D')
df = resp.write(quarterly)

The error I get is:
requests.exceptions.HTTPError: 404 Client Error:  for url: https://sdmxcentral.imf.org/ws/public/sdmxapi/rest/data/BOP_BPM6?detail=serieskeysonly

from this line:
Traceback (most recent call last):
  File "./IMFdata.py", line 59, in <module>
    resp = imf.data(resource_id='BOP_BPM6', key={'INDICATOR': code, 'REF_AREA': list(areas)}, dsd=dsd)

Sorry for the delayed response.
A high-level research suggests that the IMF has made changes to the SDMXCentral API. See the user guide. This guide does not talk about data requests at all. Only structure queries seem to be supported. I read in another guide which may or may not be up to date, that data requests are still possible. But I found it only through google, not the SDMXCentral website itself. That guide, however, talks about user authentication. There may be a chance that data requests may be restricted to registered users now.
This topic needs further research.

I am also running into an 404 Client Error.
import pandasdmx as sdmx

print(sdmx.__version__)

data_source = "IMF"
src = sdmx.Request(data_source, backend="sqlite", fast_save=True, expire_after=600)

src_msg = src.dataflow("NAG_NSTD")

src_flow = src_msg.dataflow.NAG_NSTD

dsd = src_flow.structure

keys = {"DATA_DOMAIN": "NAG", "INDICATOR": "SUR_NGDPVA_R_ISIC3_TOTAL_G_XDC"}
params = {"StartPeriod": "2000"}

out = src.data("NAG_NSTD", key=keys, params=params, dsd=dsd)

Traceback:
1.0.0
Traceback (most recent call last):
  File "<input>", line 1, in <module>
  File "C:\Program Files\JetBrains\PyCharm Community Edition 2019.3.3\plugins\python-ce\helpers\pydev\_pydev_bundle\pydev_umd.py", line 197, in runfile
    pydev_imports.execfile(filename, global_vars, local_vars)  # execute the script
  File "C:\Program Files\JetBrains\PyCharm Community Edition 2019.3.3\plugins\python-ce\helpers\pydev\_pydev_imps\_pydev_execfile.py", line 18, in execfile
    exec(compile(contents+"\n", file, 'exec'), glob, loc)
  File "C:/Users/andi/PycharmProjects/sdmx/sdmx.py", line 17, in <module>
    out = src.data("NAG_NSTD", key=keys, params=params, dsd=dsd)
  File "C:\Users\andi\AppData\Local\Programs\Python\Python38\lib\site-packages\pandasdmx\api.py", line 390, in get
    response.raise_for_status()
  File "C:\Users\andi\AppData\Local\Programs\Python\Python38\lib\site-packages\requests\models.py", line 941, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 404 Client Error:  for url: https://sdmxcentral.imf.org/ws/public/sdmxapi/rest/data/NAG_NSTD/NAG..SUR_NGDPVA_R_ISIC3_TOTAL_G_XDC...?StartPeriod=2000

In the docs, it is mentioned that IMF Supports series-key-only and hence dataset-based key validation and construction. I don't fully understand the consequences. Any hint how the code should look like to take this into account is highly appreciated.

The source of this issue is not understood yet. It is a high priority
to fix this, now that v1.0.0 is out.
Any help would be appreciated.
On 25/05/2020, Andreas Vester<EMAIL_ADDRESS>wrote:

I am also running into an 404 Client Error.
import pandasdmx as sdmx

print(sdmx.__version__)

data_source = "IMF"
src = sdmx.Request(data_source, backend="sqlite", fast_save=True,
expire_after=600)

src_msg = src.dataflow("NAG_NSTD")

src_flow = src_msg.dataflow.NAG_NSTD

dsd = src_flow.structure

keys = {"DATA_DOMAIN": "NAG", "INDICATOR":
"SUR_NGDPVA_R_ISIC3_TOTAL_G_XDC"}
params = {"StartPeriod": "2000"}

out = src.data("NAG_NSTD", key=keys, params=params, dsd=dsd)

Traceback:
1.0.0
Traceback (most recent call last):
  File "<input>", line 1, in <module>
  File "C:\Program Files\JetBrains\PyCharm Community Edition
2019.3.3\plugins\python-ce\helpers\pydev\_pydev_bundle\pydev_umd.py", line
197, in runfile
    pydev_imports.execfile(filename, global_vars, local_vars)  # execute the
script
  File "C:\Program Files\JetBrains\PyCharm Community Edition
2019.3.3\plugins\python-ce\helpers\pydev\_pydev_imps\_pydev_execfile.py",
line 18, in execfile
    exec(compile(contents+"\n", file, 'exec'), glob, loc)
  File "C:/Users/andi/PycharmProjects/sdmx/sdmx.py", line 17, in <module>
    out = src.data("NAG_NSTD", key=keys, params=params, dsd=dsd)
  File
"C:\Users\andi\AppData\Local\Programs\Python\Python38\lib\site-packages\pandasdmx\api.py",
line 390, in get
    response.raise_for_status()
  File
"C:\Users\andi\AppData\Local\Programs\Python\Python38\lib\site-packages\requests\models.py",
line 941, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 404 Client Error:  for url:
https://sdmxcentral.imf.org/ws/public/sdmxapi/rest/data/NAG_NSTD/NAG..SUR_NGDPVA_R_ISIC3_TOTAL_G_XDC...?StartPeriod=2000

In the docs, it is mentioned that IMF Supports series-key-only and hence dataset-based key validation and construction. I don't fully understand the
consequences. Any hint how the code should look like to take this into
account is highly appreciated.
--
You are receiving this because you were assigned.
Reply to this email directly or view it on GitHub:
https://github.com/dr-leo/pandaSDMX/issues/119#issuecomment-633688486

I can confirm that IMF SDMXCentral unfortunately no longer supports data queries. It continues to support queries for structural metadata though, however useful this may be to average data consumers.
I will update the docs and possibly pandasdmx.sources package accordingly to avoid the misleading Impression that IMF data can be queried with pandasdmx.
TODO:

consider alternative channels to obtain IMF data.


[Extension API] Allow determining whether the current user is following another participant

In order to support extensions that contribute unique and collaborative user interfaces (e.g. a whiteboard, integrated browser previews), they need to be able to detect if a participant is following someone, so that they can focus the appropriate UI in response to navigations by the participant being followed (e.g. launching a browser preview window).
Additionally, since the “Participants” list in the Live Share view displays each user’s active location (e.g. a file position), extensions should be able to update this location to refer to non-file-based documents to reflect each user’s actual focus (e.g. viewing a whiteboard, browsing a web page).

We’re not able to prioritize this issue over the other higher-impact issues we receive every week, based on the votes and comments from others in the community and our understanding of the issue. However, rest assured that we love your input. If you feel it deserves to stay open, then clarify your use case and contact us to let us know how severe it’s for you.


[Docs] Add material-ui-chip-input to related projects

This adds material-ui-chip-input to the list of related projects, as suggested in #4993. I'm not sure about adding it directly to the Chip's documentation, so I'm not pull requesting that. ;)

Looks good to me!

@leMaik Thanks, great project!


 I think there are many benefits from returning exception structs when returning {:error, _} tuples:

They can be raised with raise/1
They're easier to pattern-match on
They can be documented very well

See an example of PR similar to this here: https://github.com/elixir-gettext/expo/pull/74
What do you folks think @NickNeck? cc @wojtekmach

Pull Request Test Coverage Report for Build ff87f35693304703f365b6c556c65594851697db-PR-9

54 of 55 (98.18%)  changed or added relevant lines in 9 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.07%) to 98.846%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/datix.ex
15
16
93.75%

Totals

Change from base Build 761bead1df49b0fe52477a77388885d101889411:
0.07%

Covered Lines:
257

Relevant Lines:
260

💛  - Coveralls

I was gonna suggest a similiar change, thank you!

I like this changes. I had changed some of my other projects after reading the blog post you mentioned.


client: Fix compilation with Solana 1.14

anchor-client uses an error variant that was added in solana-client 1.16 which cause compilation errors when using 1.14(https://github.com/coral-xyz/anchor/pull/2488#discussion_r1200952015). It was added in commit https://github.com/coral-xyz/anchor/pull/2488/commits/25a708eead151cc8f30cfef3dec8ef6ad6c38b7f and tests missed this issue because it's just after the 1.16 upgrade. We will most likely drop the 1.14 support after Metaplex crates upgrade to 1.16 but it doesn't hurt to make it compatible now.

@acheroncrypto looking forward to buy you a beer in breakpoint 🙏


Value Extractor for all digit in one value

I am here to request a new feature which extract all digit in one number from string .
Here is an example .
String  "13-Feb-2023 [00:06:36]"
Value "132023000636"

13-Feb-2023 [00:06:36] -> 13Feb2023000636

i need only digit<PHONE_NUMBER>36


Roll Engine from 9520bb15b3d2 to 2536a3485940 (7 revisions)

https://github.com/flutter/engine/compare/9520bb15b3d2...2536a3485940
2021-06-22<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from krpEDaT56... to 8F5w2I_W6... (flutter/engine#26873)
2021-06-22<EMAIL_ADDRESS>Roll Dart SDK from 77bc0eb4b39a to b260adefae51 (4 revisions) (flutter/engine#26872)
2021-06-22<EMAIL_ADDRESS>Roll Skia from 64751750f474 to 9f73b04b437d (10 revisions) (flutter/engine#26871)
2021-06-21<EMAIL_ADDRESS>update gcloud credentials (flutter/engine#26868)
2021-06-21<EMAIL_ADDRESS>Revert "Issues/80711 reland (#26813)" (flutter/engine#26859)
2021-06-21<EMAIL_ADDRESS>Roll Skia from 9a4824b47c03 to 64751750f474 (1 revision) (flutter/engine#26851)
2021-06-21<EMAIL_ADDRESS>Roll Dart SDK from debd5e54b544 to 77bc0eb4b39a (1 revision) (flutter/engine#26853)
Also rolling transitive DEPS:
fuchsia/sdk/core/mac-amd64 from krpEDaT56jlQ to 8F5w2I_W6Nk0
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

build_ios_framework_module_test test prepped for this roll with https://github.com/flutter/flutter/pull/85051.  Closing this one to pick up that test change.

Commit queue failed; closing this roll.


Docs don't point to the updated v4.0.0 ones

The ReadMe links to this page: https://purescript-spec.github.io/purescript-spec/
However, the actual docs for things like Hooks (https://github.com/purescript-spec/purescript-spec/blob/master/docs/writing-specs.md) aren't included.

@JordanMartinez Thanks for pointing this out. I've rebuilt and published the docs. Please verify that you're seeing the latest documentation at https://purescript-spec.github.io/purescript-spec/.

Yup! Thanks for fixing that so quickly!


Update to C++ SDK 2.5 in .NET SDK

Support the latest 2.5 release, which is currently 2.5.1.
See https://github.com/DolbyIO/comms-sdk-cpp

Small correction, should be updated to C++SDK 2.5.3


[Removal]: Alert Linen Wallaby

Hotspot Name
Alert Linen Wallaby
Hotspot b58 Address
11txzPQ9mQkhNRX8XRjC2bCyEPjppFoaC18cHjSaVbwxe5ncibd
Discord Handle
PeterMonita
Hotspot Manufacturer
Cotx x3
Removal Reason
I bought the hotspot 2 hand and in the app cotx say on deny  list
Modifications
No
Extra forwarders
No
Extra antennas
No
Additional Information
I bought this Cotx x3 hotspot yesterday, transferred to my wallet, Cotx app says on deny list, can't see in helium app or Explorer. In Explorer the hotspot doesn't appear in hex 881969d6e9fffff. Bought this hotspot for my family on this location. I thought I transfer the hotspot again to a new wallet I created for them, doesn't seem to help. Can someone please help me?
. The hotspot is now installed with the original antenne and saw filter.

It is no longer on deny list.

It is no longer on deny list.

Thanks for the reply, but in explorer the hex is not getting green, it's now 1.5 days later 881969d6e9fffff. Strange

In Ihub.global helium track and hotspotty hotspot doesn't show up in the map. 881969d6e9fffff. It look like there is no Hotspot.

GitHub is for hotspots on the denylist only.  You are not on denylist. I recommend going to helium discord channel. ``

Working


Fix: SVG icons do not show on native

Details

A numeric attachmet's source can represent either an icon (ex: SVG) or a static image. A static image must be rendered with AttachmentViewImage and an icon with Icon. This causes a problem that on native, SVG icons do not show. This PR seperates these two cases with numeric source.
Fixed Issues

$ https://github.com/Expensify/App/issues/33682
PROPOSAL: https://github.com/Expensify/App/issues/33682#issuecomment-1870733381
Tests

Open any workspace/room with default avatar
Press header >> Press ws/room avatar in details page
Verify the avatar shows
====
Repeat step 1 - 3 for a ws/room with custom avatar
====
Go offline
Create a distance request
Go to request details report
Verify the generic receipt thumbnail shows
Tap on the receipt thumbnail
Verify the distance request placeholder shows

[x] Verify that no errors appear in the JS console

Offline tests

NA
QA Steps

Open any workspace/room with default avatar
Press header >> Press ws/room avatar in details page
Verify the avatar shows
====
Repeat step 1 - 3 for a ws/room with custom avatar

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

Sorry for the delay @tienifr. I will try to get to it asap. But I am working partially. So, I could not prioritize this PR. Can you please resolve the merge conflict.

@tienifr Let's resolve the merge conflict and we can get this merged.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/arosiclair in version: 1.4.35-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/mountiny in version: 1.4.35-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


assistance with code embedding

 I am looking for assistance with a code to embed 3d models on my website.

This is the issue tracker for VS Code, we cannot offer support beyond of that.


Y24-185 - Move Sequencescape API v1 calls over to API v2

User story
Sequencescape API v1 will be shut down in the near future and has been deprecated for the past 5 years.  We should switch usage of the API over to v2 by installing and making use of the json_api_client gem.
Who are the primary contacts for this story
Stuart McHattie
Who is the nominated tester for UAT
PSD
Acceptance criteria
To be considered successful the solution must allow:

[ ] No more API v1 calls are made to Sequencescape
[ ] All traces of sequencescape-client-api and its config have been removed.

Dependencies
Any new endpoints required in Sequencescape will have to be created as they are discovered to be missing.  A new story should be created under that repository if there are indeed endpoints to create.
Additional context
The calls made during the last two week period are as below, with counts for each endpoint:

GET    - /api/1/: 33

GET    - /api/1/tag_layout_templates: 4

GET    - /api/1/tag_layout_templates/2: 4

GET    - /api/1/{uuid}: 45

GET    - /api/1/{uuid}/qcables: 12

POST   - /api/1/qcable_creators: 4

POST   - /api/1/{uuid}/first: 17

POST   - /api/1/{uuid}/lots: 4

Reviewing the code, there are v1 endpoints being called in a lot of places, which is not reflected in the endpoints actually being hit on Sequencescape as shown above.  The working assumption is that Gate Keeper is not used as much any more and only some parts of it are in active use.  To track, though, the obvious uses of v1 endpoints are for:

Creating state changes.
Finding assets by barcode.
Creating QC decisions.
Finding lot types by various criteria.
Finding plates by ID.
Finding lots by ID.
Creating QC Creators.
Finding robots by barcode.
Creating stamps.
Find QCAbles by barcode.
Finding robots by UUID.
Finding robots by barcode.
Finding order templates and creating orders from them.
Creating submissions.
Finding tubes by ID.
Finding users by swipecard.
Finding barcode printers.
Finding lots by batch ID.
Finding transfer templates.
Getting all tag layout templates.
Getting all tag2 layout templates.
Creating bulk transfers.
Finding tag/tag2 layout templates and then creating a tag layout from it.
Creating plate conversions.
Creating plates.
Creating specific tubes.
Creating tubes from tubes.
Getting all plate templates.
Finding plate purposes.
Finding tube purposes.
Fetching studies and projects.

These may not be exhaustive, but I think they are.  The stakeholders will be consulted on this as this would be a lot of work to switch over to V2 and it looks like most of this isn't actually used.


New Unit Name in Build not Loadout

Off hand do you have any idea why these aren't pulled from the same place?

The stats all seem to be right.

Interestingly this goes away if/when you refresh.

This needs to be (re)verified


Fix update apt cache

Resolves https://github.com/GovTechSG/terraform-modules/issues/169.
Tested a total of 20 packer runs on all consul-server, nomad-server, nomad-client, vault, no more errors found during the installation of td-agent gcc package.
Also no need to wait for 10s, since the completion of cloud-init should ensure that the instance is ready to go.

I removed the 10s pause because it didn't seem necessary, since the first provisioner can only successfully start when the SSH service of the packer building EC2 instance is ready. Typically the cloud-init adds around 10-20s to the Ansible provisioner based on what I observed so far.
But were there other reasons as to why the pause was added? I am okay to add back the pause onto the shell provisioner if there were some reasons.

Okay to the timeout just in case it never completes. Hope it can squeeze into the one line nicely haha

OK without the pause.
Try to see if the timeout can be done with Packer.

https://github.com/hashicorp/packer/issues/4819, waiting for someone to implement haha :stuck_out_tongue:
So gotta do more shell magic instead for now


Adding Menu, MenuItem and MenuLabel

In the Menu Component, the unique problem I found is that the MenuItem seems like it's not rendering the icon... I don't know if I did something wrong but would be great if someone can validate it ;

To test the component you can use the reproducible example below:
import dash
from dash import dcc, html, Input, Output
import dash_mantine_components as dmc

app = dash.Dash(__name__)

app.layout = html.Div(
    className="text-center",
    children=[
        html.Div([
            html.Div("Menu and sub Menu Components test"),
            dmc.Menu(children=[dmc.MenuLabel(children="Label test"),
                               html.Div(children=['"Div Option"']),
                               dmc.MenuItem(children="MenuItem1"),
                               dmc.Divider(),
                               dmc.MenuItem(children="testing"),
                               dmc.MenuItem(children=["Testing with div"])]),
        ]),
    ],
    style={"padding": "64px"},
)

if __name__ == "__main__":
    app.run_server(debug=True, port=9966)

What about the callbacks? Can you please also include how this component will be used in the apps?

Hey @snehilvj I did some modifications and now it's a good alternative to create links to redirect the user as a button;