Add links to tarballs on Linux download page

Related to https://github.com/git/git-scm.com/issues/416
I added an obvious link at the top as well as another in the RHEL-and-friends section, as it explicitly mentioned building from source as an option but didn't provide a link.

Looks like an obvious improvement to me. Thanks!

Looks much better, thanks!


Implement opacity prop for MapOverlay on Android and iOS (Google Maps)

Does any other open PR do the same thing?

No.
What issue is this PR fixing?
We needed the ability to control the transparency/opacity of Overlay components. For some reason, the transparency control for GroundOverlay was commented out for Android and was never implemented for iOS.
The transparency option was implemented as opacity as the rest of this project seems to prefer opacity, and both platforms implemented it differently as well.
How did you test this PR?

Tested on an android emulator and iOS simulator using<EMAIL_ADDRESS>and using the branch HEAD "react-native-maps": "git://github.com/Crop-R/react-native-maps.git#5ee80e8d1e5af9962937d5db578d333fef4d35cb" as a dependency. We pass an image url to the image prop of MapView.Overlay.

LGTM
🐽 @christopherdro
I don't really get why travis is  "yellow" the job seems to be executed with no error 😒

@JJStoker thanks for the PR 😻


feat: RTL support

feature for support dir="rtl" for latest version of bootstrap #24807

hi @dharmeshpipariya thanks for you PR. This has been discussed many times, the last one here: #23703


Remove unused KMS key

Describe the bug
The nested AISTACK creates a KMS Key SQSManagedKey that is not used or needed by LMA, and which can cause deployment failures in environments where creation of keys is restricted.

fixed in v0.1.8


I can't create new project

It stucks while i input the company's domain.

)
I use get_cli Version: 1.7.1

You hit enter and it's not working?
I tested it now on mac and it worked, but it could be a bug introduced in the last version for the windows platform

You hit enter and it's not working? I tested it now on mac and it worked, but it could be a bug introduced in the last version for the windows platform

Hello, @jonataslaw
When i hit enter, the cursor is moving to the left side of command prompt.

While i install it, it shows me this notification.

Did i miss something?
Thank you so much for your response,

Same error, any solution?

facing the same problem, any solution for this issue.

facing the same problem, any solution for this issue.

I had to go back to version 1.6.0 for it to work, I did that while waiting for the solution

facing the same problem, any solution for this issue. I had to go back to version 1.6.0 for it to work, I did that while waiting for the solution
1.6.0 is working fine, thanks for the solution.

/// --------------- error
import 'package:cli_dialog/cli_dialog.dart';
void main() async {
final dialog1 = CLI_Dialog(listQuestions: [
[
{
'question': "title",
'options': ['choices option 1', 'choices option 2']
},
'result'
]
]);
final answer = dialog1.ask();
final result = answer['result'] as String;
print(result);
await Future.delayed(Duration(seconds: 1));
final dialog = CLI_Dialog(
questions: [
['What is your name?', 'name']
],
);
final name = dialog.ask()['name'];
print(name);
}
//// -----------------------------------------
cli_dialog 0.5.0  this package not working as expected on windows os ...

/// --------------- error import 'package:cli_dialog/cli_dialog.dart';
void main() async { final dialog1 = CLI_Dialog(listQuestions: [ [ { 'question': "title", 'options': ['choices option 1', 'choices option 2'] }, 'result' ] ]);
final answer = dialog1.ask(); final result = answer['result'] as String; print(result); await Future.delayed(Duration(seconds: 1));
final dialog = CLI_Dialog( questions: [ ['What is your name?', 'name'] ], );
final name = dialog.ask()['name']; print(name); } //// ----------------------------------------- cli_dialog 0.5.0 this package not working as expected on windows os ...

Thanks for pointing that out!

Same problem.

same problem can't create project and its irritated me

any solution?

install get CLI previous ver 1.6
On Wed, 29 Dec 2021 at 02:39, Kevin Piazzoli Felix Perez <
@.***> wrote:

any solution?
—
Reply to this email directly, view it on GitHub
https://github.com/jonataslaw/get_cli/issues/146#issuecomment-1002282759,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ4664BEWDRDZ6HXHDWBL4TUTIRPDANCNFSM5J6QPTVA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

Any solution here ?

the only way I managed to run get cli was to uninstall the v1.7.1 and install the v1.6.0 package
flutter pub global deactivate get
flutter pub global deactivate getx
dart pub global activate get_cli ^1.6.0

please check this :
https://github.com/jonataslaw/get_cli/issues/148#issuecomment-1017157446

please check this : #148 (comment)

It solved my problem on 1.7.1. Thank you! @fahmi277

Same Issue

get_cli 1.7.1  not working, i tested on mac it's ok but not on windows 10

I'm using get_cli 1.8.0 same problem while creating the project (windows 10)

I'm using get_cli 1.8.0 same problem while creating the project (windows 10)

The issue is still not solved

Please check this
https://github.com/jonataslaw/get_cli/issues/148#issuecomment-1017157446

========Solution=============
First, uninstall your current version (1.8.0)
flutter pub global deactivate get_cli

After that install version 1.6.0
flutter pub global activate get_cli 1.6.0

Just press Ctrl + Z and hit enter, your options will appear

Same issue with version 1.8.1 on windows 11. Any solution ?

Check this
#148

Just press Ctrl + Z and hit enter, your options will appear

v 18.1 at  my windows solved in this way


[tracking] Setup CI job for stage 'functional tests'

https://github.com/RedHatInsights/insights-results-aggregator-mock/issues/79

Done in https://github.com/RedHatInsights/insights-results-aggregator-mock/pull/157


Fail on start with python-evdev 0.6.0

Previously ds4drv worked correctly with python-evdev 0.5.0. I upgraded python-evdev to version 0.6.0 and now ds4drv fails on start with this:
Traceback (most recent call last):
  File "/bin/ds4drv", line 9, in <module>
    load_entry_point('ds4drv==0.5.0', 'console_scripts', 'ds4drv')()
  File "/usr/lib/python3.5/site-packages/ds4drv/__main__.py", line 374, in main
    thread = create_controller_thread(index + 1, controller_options)
  File "/usr/lib/python3.5/site-packages/ds4drv/__main__.py", line 343, in create_controller_thread
    controller = DS4Controller(index, controller_options, dynamic=dynamic)
  File "/usr/lib/python3.5/site-packages/ds4drv/__main__.py", line 59, in __init__
    self.load_options(self.options)
  File "/usr/lib/python3.5/site-packages/ds4drv/__main__.py", line 117, in load_options
    self.fire_event("load-options", options)
  File "/usr/lib/python3.5/site-packages/ds4drv/__main__.py", line 62, in fire_event
    self.loop.fire_event(event, *args)
  File "/usr/lib/python3.5/site-packages/ds4drv/eventloop.py", line 70, in fire_event
    self.process_events()
  File "/usr/lib/python3.5/site-packages/ds4drv/eventloop.py", line 75, in process_events
    callback(*args)
  File "/usr/lib/python3.5/site-packages/ds4drv/actions.py", line 282, in load_options
    joystick = create_uinput_device(joystick_layout)
  File "/usr/lib/python3.5/site-packages/ds4drv/uinput.py", line 363, in create_uinput_device
    device = UInputDevice(mapping)
  File "/usr/lib/python3.5/site-packages/ds4drv/uinput.py", line 227, in __init__
    self.create_device(layout)
  File "/usr/lib/python3.5/site-packages/ds4drv/uinput.py", line 268, in create_device
    product=layout.product, version=layout.version)
  File "/usr/lib/python3.5/site-packages/evdev/uinput.py", line 107, in __init__
    _uinput.create(self.fd, name, vendor, product, version, bustype, absinfo)
SystemError: Objects/longobject.c:403: bad argument to internal function

Hitting that issue as well on Arch Linux. Can't find a way around it.

Same problem on Manjaro/Archlinux. Maybe the workaround is to get back on the 0.5.0 version of python-evdev, but i don't know how to do on Archlinux.

@Muskipet https://wiki.archlinux.org/index.php/downgrading_packages

:+1:

@Ape The problem with that is that ds4drv on install notices that there is a new evdev, and installs it as a depend. I can't see a way to block this from happening, or to force it to use an older installed version. I am sure it is easy enough, but I know nothing about Python, so out of my league.

It looks like the following change to python-evdev broke compatibility with ds4drv. There is now an extra index for absinfo corresponding to the current value for an axis. The new representation used for uinput device creation has a size of 6 while the one made from the data passed by ds4drv has a size of 5. Adding new items at index 1 for each absinfo entry (using 0 for testing) allows ds4drv to work.
https://github.com/gvalkov/python-evdev/commit/62969d7bca43d87799237590a460d4361012225c#diff-4837c124766bc823c51073050068c3ee
Also relevant: https://github.com/gvalkov/python-evdev/blob/master/evdev/device.py#L20

@Ryochan7 Could you send a fixing pull request to ds4drv?

I released version 0.5.1 which works with python-evdev 0.6.0.


can dropdown list cell bind datatable?

can dropdown list cell bind datatable ,just like combobox.
have valuefield and textfield
thanks

You can make your own drop-down cell with a standard .NET ListView or ListBox control inside, use the standard .NET ListView or ListBox to bind the data you want to show. For more details please refer to:  https://reogrid.net/walkthroughs/how-to-create-custom-drop-down-cell/


Loop "losing" one image after full loop due to duplicate logic

So after i loop through one full cycle, the next  tag loses its src value and is therefore blank. I see the "duplicate" logic behind the scenes creates another image tag. So if i have a gallery of 5 images, there is actually a 6th one created. That one misses some properties within its HTML element.
Original:

Duplicate when looping:

As additional info, my element within the swiper slide has this structure:
<swiper slide>
     <image container>
          <actual image/>
     <text bar below image>
          <some text/>
          <copyright text/>

The missing src properties are occuring within the actual images img tag.
I tried all i can on my side. Please fix.

Solution
So to solve this problem. I use the slideChange event only for loop mode.
For example:
I have 5 slides with the loop mode. The data can be visualized like this
5 1 2 3 4 5 1 // length: 7

As you can see to reach to the final slide. We have 2 ways
First is from 4 -> 5
We could check
if (prevSlide === n - 3 && currentSlide === n - 2) {
   // do something here
}

Second is from 1 -> 5
if (prevSlide === 1 && currentSlide === 0) {
   // do something here
}

is #394 related to this?

I have the same issue with next/image component? Do you have an example with this component?

checkout swiper the library this is a wrapper around natively supports react now
https://swiperjs.com/react


Is there a way to tap into the annotations saving process through this directive?

eg.:
<div annotorious-annotate on-save="saveFunction()">
</div>

in the controller:
$scope.saveFunction = function() {console.log('saving!')}

This would be useful for linking the annotations to a back-end, for example.

Hi! sure, now there is.
I have added an example under  'src/storage-providers/'
The file is:
https://github.com/igorlino/angular-annotorious/blob/master/js/storage-providers/annotorious-local-storage.provider.js
It will save and load annotations to/from the local browser storage. This is working code.
This will give you an idea how to do it for other type of storage providers.
If you create another storage provider, that ends up generic enough, you may like to contribute it for the angular-annotorious community. So then we could place it as an example also.

The way to create annotations is using the angular service interface.
Assuming you have an image  element that has a src attribute.
var  photo = jQuery(document).find(".your-image")[0];
You could call:
annotoriousService.makeAnnotatable(photo);

If you want to reset/reload all annotations in the page:
annotoriousService.reset();

If you want to set the design of the annotation:
annotoriousService.setProperties({
            outline: '#00f',
            stroke: '#ff0000',
            fill: 'rgba(255, 0, 0, 0.3)',
            'hi_stroke': '#00ff00',
            'hi_fill': 'rgba(0, 255, 0, 0.3)'
        });

I have updated a bit the https://github.com/igorlino/angular-annotorious/blob/master/README.md
To include this info for future users :)
If you would like to add further document that you think is useful for other developers, its very welcome.

That's great, thanks for the info. Also, thanks for updating the documentation :)

hey guys I wanted to try the storage-provider part but it wont work because of the outdated angular lodash module

Unable to find a suitable version for angular, please choose one by typing one of the numbers below:
1) angular#~1.3.4 which resolved to 1.3.20 and is required by angular-annotorious#1.0.7
2) angular#~1.4 which resolved to 1.4.14 and is required by ng-lodash#0.5.0
3) angular#>0.9.0 which resolved to 1.6.4 and is required by angular-uuid4#0.3.1
4) angular#>=1.3.0 which resolved to 1.6.4 and is required by ng-tags-input#3.2.0
5) angular#^1.x which resolved to 1.6.4 and is required by angular-local-storage#0.6.0

therefore i can't get the storage-provider working. Is there a workaround for that issue?

a newer version on angular lodash its compatible.
Simply declare the newer version in a resolution and bower won't complain.
https://jaketrent.com/post/bower-resolutions/


About page

fixing #302

sorry Jasper , you have made so many changes in a single pr , it's hard to track the changes and I have no idea what will be the effect on the website , please create an issue on which feature you want to work , I will asigne to you then you can start working on it, while making a pr please add ss also !


Cannot send messages larger than 2048 bytes

Bug Report
Cannot send messages larger than 2048 bytes.
Sending a message greater than 2048 results in an error， It's pretty certain。
buf := make([]byte, 2048) Unable to read the full message causing an error
This code needs to be modified to read all the message contents at once.

@bytemode Can you provide the reproduce step? or some logs.


Clean up the TRB_label_assist code

This will:

help us modify it and make it better if we want to experiment with different algorithms, including trajectory matching
allow us to integrate into our production system
make sure that it is reproducible before the related paper gets published

The code is in: https://github.com/e-mission/e-mission-eval-private-data/tree/master/TRB_label_assist
It works, but it requires us to use a custom fork of the e-mission-server repo to work
We should see the changes between the custom fork and the current e-mission-server master/gis branch
See if the changes are still necessary or whether there are implementations in e-mission-server that have superceded them
If they have not been superceded, we need to incorporate them in e-mission-server
So at the end, this should be reproducible against a core e-mission-server repo
Stretch goal: change this repo to work with docker containers that are built with the base e-mission-server container so that people don't have to install e-mission-server and set the EMISSION_SERVER_PATH and all the funky setup steps.
After we are done with this, we should integrate the random forest model + prediction into the e-mission server code for label assist.

either replace the existing clustering approach OR
better, keep both methods in the current plugin architecture and maybe even consider an ensemble down the road.

The current implementation of the mode building part of label assist code is at:
https://github.com/e-mission/e-mission-server/tree/master/emission/analysis/modelling/trip_model
There are previous implementations in https://github.com/e-mission/e-mission-server/tree/master/emission/analysis/modelling/tour_model and https://github.com/e-mission/e-mission-server/tree/master/emission/analysis/modelling/tour_model_first_only

tour model was written by a summer intern and is very bad code
tour_model_first_only was a last minute hack by me to salvage some of the interns code and get a feature working in production
trip_model is a rewrite of tour_model_first_only to be more modular

The custom branch uses some code from tour_model or tour_model_first_only since the TRB_label_assist work was being done in parallel with the trip_model rewrite.
So we should:

see where that functionality is in trip_model
if it exists, replace its usage in TRB_label_assist with the implementation from trip_model
if it doesn't exist, we add it to trip_model and then replace its usage in TRB_label_assist

Eventually we should take the new and improved model(s) from TRB label assist and put the into trip_model
For a high level summary of what the label assist model does, please see this poster:
https://www.osti.gov/biblio/1922617

Two files, Clustering.py and models.py needs to be updated for us to port the entire code to just our  main repo.
I had two questions:
Q1  Among the vast number of models in the models.py file, we are currently using  ClusterExtrapolationClassifier, ForestClassifier and NaiveBinningClassifier in our performance_eval.py file. Out of these three , only NaiveBinningClaasifier has dependence on the code from tour_model. But there's one another model ( which we are currently not using) RefactoredNaiveCluster  which is dependent on tour_model. Should we, for now, let model be dependent on tour_model or port this to trip model ?
Q2 Is there any way I  can check the the working of trip_model by running it ? Is there any module that's using it that I can/should run ??
NOTE :  regenerate_classification_performance_results.py. is currently standalone and is not being used by any of the notebooks.

@humbleOldSage I just want to clarify that, while we will eventually port the code around the new models in the e-mission-eval-private-data repo to the e-mission-server repo, the first goal is to port the code required by the special branch of e-mission-server that this repo to the master branch of e-mission-server.
clustering and models and files in e-mission-eval-private-data not the server.

Q2 Is there any way I can check the the working of trip_model by running it ? Is there any module that's using it that I can/should run ??
Trip model has unit tests, and a script to launch it against a database.
If you run it against the full CEO dataset, I would suggest launching over the weekend so that you can still use your computer 😄
$ grep -rl trip_model emission | grep -v __pycache__
...
emission/analysis/classification/inference/labels/inferrers.py
emission/analysis/modelling/trip_model/model_type.py
emission/analysis/modelling/trip_model/config.py
emission/analysis/modelling/trip_model/model_storage.py
emission/analysis/modelling/trip_model/run_model.py
emission/analysis/modelling/trip_model/greedy_similarity_binning.py
emission/tests/modellingTests/TestBackwardsCompat.py
emission/tests/modellingTests/TestRunGreedyIncrementalModel.py
emission/tests/modellingTests/TestRunGreedyModel.py
emission/tests/modellingTests/modellingTestAssets.py
emission/tests/modellingTests/TestGreedySimilarityBinning.py
...

NOTE : regenerate_classification_performance_results.py is currently standalone and is not being used by any of the notebooks.
That is by design - if you check the history:

Note that these results can take very long (> 2 days) to regenerate
Running them from a notebook will either not print logs, or will print so many
logs that the notebook buffers will be overwhelmed

Moving the computation code out to a separate script allows us to more easily
redirect the output to a file and track the progress of the execution

There looks like an indirect dependence of RefactoredNaiveCluster, so we'll have to work on it as well.

I have been working with the sample data that the script  generates for testing for now. It should be enough for my purpose. If it doesn't I'll run it on full data set over the weekend. Also, won't need to run all of them, just need 1 test file, the TestGreedySimilarityBinning.py one.

It should be enough for my purpose.

Curious - what are you running the existing tests for? Is it to test the new functionality that you are moving into trip_model? If so, you should really write a new unit test (which can use similar sample data) instead of ad-hoc testing.

While I move the functionalities, I am just using them to verify my understanding of the  code-flow and I/O to modules. That's why I felt a small randomly generated sample should serve the purpose. Once done, I do plan on writing tests new unit tests and may even test on the full CEO dataset.

The dependence of the 2 files as mentioned previously(clustering.py and models.py)are on the following 2 modules  in the custom branch:

bin_helper module in theSimilarity Class from  similarity.py file.
save_models and create_user_input_map modules from the build_save_model.py file.
fit module again from the Similarity Class from similarity.py file.

For i. , I probed the trip_model folder in the main branch and  we surely have this functionality already implemented in the main branch.
I can say this because to understand the functionalities and flow of data on the custom branch side, I ran the dependent notebooks on the CEO dataset. To match this understanding on the main branch side, I ran the existing unit test  specifically for the modeling branch using  the below command.
PYTHONPATH=. python -m unittest emission/tests/modellingTests/TestGreedySimilarityBinning.py
Both these functions were passing origin/destination latitude  and longitudes and rely on  od_similarity.py file for similarity computation. This helped me reach the conclusion that the functionality and the end result of these two had an overlap.
However, both these functions are working on different data-forms, i.e., custom branch uses data in data-frame format while processing whereas the main branch uses them in Event type wrapper.  On Further searching, I could figure that the data was being converted from Event  type to data_frame at some-point before visualizing them.

To begin with the changes in the clustering.py file , I can see that the end result of the "naive" algorithm in the add_loc_clusters function is a column being added to the data frame having  group labels ( numeric value ) as its value at line :
loc_df.loc[:, f"{loc_type}_{alg}_clusters_{r}_m"] = label
where {loc_type} -> start or end or trip
{alg} -> DBSCAN or naive or OPTIC or fuzzy or mean_shift.
{r } -> stands for the grouping radius, takes value 50,100 and 150 currently  .
These group labels are generated using the bin_helper function present in the eamts.Similarity class and stored in two places

Inside the eamts.Similarity class they are stores in self.start_bins or self.end_bins or self.trip_bins  depending on  value of loc_type variable.

In the dataframe, the same results is stored in one  of three column amongst start_bin/ end_bin /  trip_bin and the other two are filled with NaNs.

The additional column (columns if we pass multiple r values), that I referred to in the beginning, copies labels from point 1 (above) and appends into the data frame.
When same labels  are generated from the main branch, we'll be able to remove our dependence on the custom branch.

IF we are able to generate these labels from the existing trip_model in the main branch, we'll be able to remove our dependence on the custom branch.

And what would it take to generate these labels from the existing trip_model in the main branch?

In the main branch ,trip_model has a similar functionality implemented in the _assign_bins function in the GreedySimilarityBinning class in greedy_similarity_binning.py file.
However, rather than storing bin labels of each trip next to it, _assign_bins  groups trips by their bin labels. In general,  the data takes the form:
        {
            bin_id: {
                "feature_rows": [
                    [f1, f2, .., fn],
                    ...
                ],
                "labels": [
                    { label1: value1, ... }
                ],
                "predictions": [
                    { "labels": { label1: value1, ... }, 'p': p_val }
                ]
            }
        }

        where
        - bin_id:  str    index of a bin containing similar trips, as a string
                          (string type for bin_id comes from mongodb object key type requirements)
        - f_x:     float  feature value (an ordinate such as origin.x)
        - label_x: str    OpenPATH user label category such as "mode_confirm"
        - value_x: str    user-provided label for a category
        - p_val:   float  probability of a prediction, real number in [0, 1]

        :param config: if provided, a manual configuration for testing purposes. these
                       values should be provided by the config file when running OpenPATH.
                       see config.py for more details.

For eg below is one such groups saved by _assign_bins on a dummy data :
{'0': {'feature_rows':[[-4.958129127344314e-05, 8.768060347937052e-06, 0.999720312837768, 1.0001227632472012],
                       [-0.0002885431635348521, 4.2737498370605324e-05, 1.0002114424380646, 0.9998622709144951],
                       [0.00021187675902559317, -0.00031421641939745657, 0.9998863315113582, 1.0000619592251845],
                       [0.00041447629708705905, -0.0002639625392563102, 1.0001666270256024, 0.9995808207008069],
                       [-2.651656515482855e-05, 0.0004016395427976644, 0.9999933052671837, 0.9999978850795779]],
            'labels': [{'mode_confirm': 'transit', 'replaced_mode': 'drive', 'purpose_confirm': 'school'},
                       {'mode_confirm': 'transit', 'replaced_mode': 'drive', 'purpose_confirm': 'work'},
                       {'mode_confirm': 'bike', 'replaced_mode': 'drive', 'purpose_confirm': 'work'},
                       {'mode_confirm': 'transit', 'replaced_mode': 'drive', 'purpose_confirm': 'school'},
                       {'mode_confirm': 'bike', 'replaced_mode': 'drive', 'purpose_confirm': 'school'}],
             'predictions': []},

 '1': {'feature_rows': [[0.06332302074892233, 0.02417860787080751, 1.0195018436957517, 0.9714679398747385]],
            'labels': [{'mode_confirm': 'bike', 'replaced_mode': 'drive', 'purpose_confirm': 'work'}],
             'predictions': []},

 '2': {'feature_rows': [[0.055288906505610205, 0.016151126202667912, 1.0453033636120435, 0.9712560053027818]],
          'labels': [{'mode_confirm': 'walk', 'replaced_mode': 'drive', 'purpose_confirm': 'home'}],
           'predictions': []},

'3': {'feature_rows': [[0.0023381228366961128, -0.008375891067784268, 0.9628717886188353, 0.9681751183057479]],
           'labels': [{'mode_confirm': 'transit', 'replaced_mode': 'drive', 'purpose_confirm': 'school'}],
          'predictions': []},
.
 .
.
.
}

Here 0,1,2 and 3 are bin labels which have trips ( actually they are features of trips ) belonging to that bin, grouped as a list.
We need _assign_bins function to also give the trips with their labels.

One possible way is ( to be included in PR ) , the following line gets the matching bin for the point.
bin_id = self._find_matching_bin_id(trip_features) here
Beyond this, grouping of trips a.t. labelBins take place. I  introduce a self.binLabel class variable of type list that'll append  the bin Id for the trip being processed. Once we iterate all the trips, we'll have their labels in a list which we can return to TRB_label_assist and can  append to the data frame.

Changes that need to be done in the clustering.py in TRB_label_assist module to link this with trip_model would be  :

to initiate a GreedySiilarityBinning type model with configs as :

            model_config = {
                "metric": "od_similarity",
                "similarity_threshold_meters": r,  # meters,
                "apply_cutoff": False,
                "incremental_evaluation": False
            }

Pass the data to the fit function  in the required format ,i.e.,  List[ecwc.Confirmedtrip]

Currently, add_loc_clusters has data in df form. Need to get data in Confirmedtrip format.

retrieve labels stored in self.tripLabels in GreedySiilarityBinning class

After discussions on this PR, particularly this suggestion  focuses on reducing load on production side less. So we'll move the label computations to clustering.py.
More precisely, our aim would be to retrieve

trip feature --> binId( or label or bin No)

like mappings  from

binId ---> [list trip_features]

like structure which GreedySimilarityBinning class provides us. Then further,  map every  trip_feature( along with the binID) to the correct trip in dataframe.
For e.g. :
From grouped bins which look like this
{'0': {'feature_rows':[[-4.958129127344314e-05, 8.768060347937052e-06, 0.999720312837768, 1.0001227632472012],
                       [-0.0002885431635348521, 4.2737498370605324e-05, 1.0002114424380646, 0.9998622709144951],
                       [0.00021187675902559317, -0.00031421641939745657, 0.9998863315113582, 1.0000619592251845],
                       [0.00041447629708705905, -0.0002639625392563102, 1.0001666270256024, 0.9995808207008069],
                       [-2.651656515482855e-05, 0.0004016395427976644, 0.9999933052671837, 0.9999978850795779]],
            'labels': [{'mode_confirm': 'transit', 'replaced_mode': 'drive', 'purpose_confirm': 'school'},
                       {'mode_confirm': 'transit', 'replaced_mode': 'drive', 'purpose_confirm': 'work'},
                       {'mode_confirm': 'bike', 'replaced_mode': 'drive', 'purpose_confirm': 'work'},
                       {'mode_confirm': 'transit', 'replaced_mode': 'drive', 'purpose_confirm': 'school'},
                       {'mode_confirm': 'bike', 'replaced_mode': 'drive', 'purpose_confirm': 'school'}],
             'predictions': []},

 '1': {'feature_rows': [[0.06332302074892233, 0.02417860787080751, 1.0195018436957517, 0.9714679398747385]],
            'labels': [{'mode_confirm': 'bike', 'replaced_mode': 'drive', 'purpose_confirm': 'work'}],
             'predictions': []},

 '2': {'feature_rows': [[0.055288906505610205, 0.016151126202667912, 1.0453033636120435, 0.9712560053027818]],
          'labels': [{'mode_confirm': 'walk', 'replaced_mode': 'drive', 'purpose_confirm': 'home'}],
           'predictions': []},

'3': {'feature_rows': [[0.0023381228366961128, -0.008375891067784268, 0.9628717886188353, 0.9681751183057479]],
           'labels': [{'mode_confirm': 'transit', 'replaced_mode': 'drive', 'purpose_confirm': 'school'}],
          'predictions': []},
.
 .
.
.
}

we retrieve trip with theirs bin ids as  below.
[-4.958129127344314e-05, 8.768060347937052e-06, 0.999720312837768, 1.0001227632472012]  ---> {0}
[-0.0002885431635348521, 4.2737498370605324e-05, 1.0002114424380646, 0.9998622709144951]. ---> {0}
[0.00021187675902559317, -0.00031421641939745657, 0.9998863315113582, 1.0000619592251845],  --->{0}
[0.00041447629708705905, -0.0002639625392563102, 1.0001666270256024, 0.9995808207008069],  --->{0}
[-2.651656515482855e-05, 0.0004016395427976644, 0.9999933052671837, 0.9999978850795779]],  --->{0}
[0.06332302074892233, 0.02417860787080751, 1.0195018436957517, 0.9714679398747385] ---> {1}
[0.055288906505610205, 0.016151126202667912, 1.0453033636120435, 0.9712560053027818] ---> {2}
[0.0023381228366961128, -0.008375891067784268, 0.9628717886188353, 0.9681751183057479 ---> {3}

Once we have trip-bin mappings, we'll have to search for each trip in the datagram loc_type and then add bin label in that row.

In the clustering_examples.ipynb, the data is loaded in 'Entry' format and then converted to df .This df is passed around the modules for computations and visalisations.
On the other hand , the trip_model's fit function in greedy_similarity_binning.py accepts  List of ecwc.Confirmedtrip type data.
Looks like we can connect these two by additionally passing around  Entry type data (along with df ) inside the notebook and then passing only Entry type data (for generating bin labels to) fit function . Once we get bin labels, we follow the steps here  to add bin labels to respective trips.

In the process to link them ,  I find that the load data cell,  esta.TimeSeries.get_time_series is called, which invoke an object of BuiltinTimeSeries .  Then get_data_df from  BuiltinTimeSeries is called, which called find_entries  and to_data_df.  to_data_df is responsible for converting Entry to df by calling _to_df_entry.
Once  back from these calls in the load_data cell,  at this point we have dataframe. We can additionally get the Entry type data as well, but filter_labeled_trips and expand_userinputs are working specifically on df in the notebook before passing df to tour_model_extended. It is better to get ConfirmedTripType data right before tour_model_extended's similarity function call.

After searching for a while, I can conclude that there is no functionality implemented  to convert df to ConfirmedTrip type. However, one way I can think of that can help us fulfill our goal using already implemented functionalities would be to generate a dummy entry using create_fake_entry function in /emission/core/wrapper/entry.py and then replace dummy entries with our data.
This is still a hypothesis and needs to be tested.

To test this hypothesis, I can check the column names in trips generated by generate_mock_trips in TestGreedySimilarityBinning.py (which are unit tests basically) and match if I have same columns in dataframe in clustering.py.

@humbleOldSage one of the entries in get_data_df is the object id (stored as _id).
So you should be able to, in the analysis code, keep the list of trip entries around, and once the filtering is done, filter the list of entries based on the _ids retained in the filtered df. That way, you can pass in real confirmed trip objects instead of fiddling around with conversions - confirmed_trip -> data_df -> fake confirmed trip.
If we are going to use dataframes extensively, we can also consider changing trip_model to take in a dataframe instead of a list of confirmed trips. I think that Rob structured the module this way so that the feature extraction could be modular. But if we pass in a dataframe, then the feature extraction would be column selection and/or new column creation from existing columns, possibly by using apply

If we do decide to  move to df in trip_model, it would just be column extraction (columns with sat_lat, start_lon , end_lat and end_lon). However, we'll have to rewrite fit , which includes calls to _apply_cutoff() ( current configs pass apply_cutoff = False so not being used but is part of fit function)  and _assign_bins ( which includes feature extraction as well). We can keep this option open in case nothing else works out.
Using object id seems like a good alternate to conversions. I'll pick this route for now and move forward.

Then I think it makes more sense to switch to df based processing from now itself. I'll rollback on the Trip changes I did on my end and take the data-frame way.

@shankari  I might be wrong here , but does the following make sense ?
From my understanding, I feel if we take the df way, then  from TRB_label_assist 's POV, this essentially mean we can replace trip_model ( of the main branch) to tour_model_Extended (from hlu09's( custom) branch ) . This is because all the operations in custom branch ( even data structures declared) are already using data-frames.  If data-frame friendly operations  in trip_model are what we aim to achieve, then  currently they are in tour_model_extended . No need to reimplement all of trip_model's function in df-friendly format to make 'TRB_label_assist' work . Simply replace  'tour_model_Extended' in place of trip_model.
However, It is ESSENTIAL to point out that my understanding is from TRB_label_assist's POV.  Meaning, the implementations in tour_mode_extended will make TRB_label_assist to work. But ..
It might be possible that there are dependencies on trip_model from modules other than TRB_label_assist ( which I am not aware of currently). These dependencies might be using trip_model in ways  which might not even be implemented in custom branch's tour_model_extended.   So, if we decide to replace trip_model with tour_extended_model, we'll have

add df versions of every function in trip_model to 'tour_extended_model'  and call this  upgraded_tour_extended_model .
atleast  change 'Entry'/ 'ConfirmedTrips' type data to df  before df-version-of trip_model (or upgraded_tour_extended_model) can use it.

The task, I believe, then MIGHT translate to upgrading 'tour_model_extended' to be an exact df version of trip_model . Every function of trip_model implemented in df format.

A visual explanation :

The figures generated after 97406c437f5fedb37abf96f65c04480c6c6f78b7 , are different from the paper. Below are the screenshots . To the left are screenshots from the commit I mentioned. To the right are the ones from the paper
SUBURBAN AREA. :
50m

100m

150m

COLLEGE CAMPUS :
In case of college campus, I am unable to determine what area of the map is the screenshot (in the paper) from.  However, even  if I compare the old custom branch  and the recent commit ones, they differ.

I think one of the reason, why the labels are different is  due to the format of the input to the ecc.calDistance function. The custom branch passes the points in [lon,lat] format, the main branch is using [lat,lon] format for this. However, by itself, this doesn't seem to have solved the issue. I am still looking for what else could be doing this.

The other thing that was causing the issues is the  similar function in eamms.SimilarityMetric module. It calculates similarity based on both origin and destination when we might require origin/destination/origin-and-destination.

It works for college as well.

In general, for this, it is fine to make the change in the master branch because we don't have to use it in production, but we can if we want to. We should just make sure that it is configurable so we can continue using O-D on production but use the other options for evaluation only for now.
In the timeseries, we did not want to make the change in master because there isn't only one set of entries we want to read so the change was not modular enough.
In this case, trip_model is set up to be modular, so we can just use that by adding new modules.

With the changes in fit function, till now, there are dependencies in almost all notebooks and files that needs to be changed.

classification_performance.ipynb :

has dependence on  cv_for_all_algs from performance_eval, which calls to fit functions of ClusterExtrapolationClassifier , which calls RefractoredNaivebinning, which is dependent on eamts.Similarity of the custom branch.
I've shift this to eamtg.GreedySimilarityBinning ( as done in clustering.py). In doing so, I'll change the parameters fit function of all three ClusterExtrapolationClassifier, ForestClassifier, and NaiveBinningClassifier to receive the Entry type data of the dataframe from the notebook.

While running regenerate_classification_performance_results.py , the following message occurs while running NaivebinningClassifier
running cross validation for model: fixed-width (O-D)
2023-08-28 03:45:35,441 INFO ------ START: predictions for user 00db212b-c8d0-44cd-8392-41ab4065e603 and model <class 'models.NaiveBinningClassifier'>
2023-08-28 03:45:35,454 DEBUG num trips 724
2023-08-28 03:45:35,471 INFO ----- Building model <class 'models.NaiveBinningClassifier'> for fold 0
2023-08-28 03:45:35,472 INFO PERF: Initializing NaiveBinningClassifier
2023-08-28 03:45:35,512 INFO About to fit the model <class 'models.NaiveBinningClassifier'>
2023-08-28 03:45:35,512 INFO PERF: Fitting NaiveBinningClassifier
2023-08-28 03:45:43,605 INFO About to generate predictions for the model <class 'models.NaiveBinningClassifier'>
2023-08-28 03:45:43,605 INFO PERF: Predicting NaiveBinningClassifier
2023-08-28 03:45:43,669 DEBUG getting key model_type in config
2023-08-28 03:45:43,669 DEBUG getting key model_storage in config
2023-08-28 03:45:43,691 DEBUG no GREEDY_SIMILARITY_BINNING model found for user 00db212b-c8d0-44cd-8392-41ab4065e603
2023-08-28 03:45:43,691 DEBUG In predict_cluster_confidence_discounting: n=-1; returning as-is
.
.
.
.
.

2023-08-28 03:45:44,545 DEBUG getting key model_type in config
2023-08-28 03:45:44,545 DEBUG getting key model_storage in config
2023-08-28 03:45:44,549 DEBUG no GREEDY_SIMILARITY_BINNING model found for user 00db212b-c8d0-44cd-8392-41ab4065e603
2023-08-28 03:45:44,549 DEBUG In predict_cluster_confidence_discounting: n=-1; returning as-is
2023-08-28 03:45:44,549 DEBUG getting key model_type in config
2023-08-28 03:45:44,550 DEBUG getting key model_storage in config
2023-08-28 03:45:44,554 DEBUG no GREEDY_SIMILARITY_BINNING model found for user 00db212b-c8d0-44cd-8392-41ab4065e603
2023-08-28 03:45:44,554 DEBUG In predict_cluster_confidence_discounting: n=-1; returning as-is
2023-08-28 03:45:44,556 ERROR skipping user 00db212b-c8d0-44cd-8392-41ab4065e603 due to error: ValueError('attempt to get argmax of an empty sequence')
Traceback (most recent call last):
  File "/Users/ssaini/Documents/GitHub/e-mission-eval-private-data/TRB_label_assist/performance_eval.py", line 243, in cv_for_all_users
    min_samples=min_samples)
  File "/Users/ssaini/Documents/GitHub/e-mission-eval-private-data/TRB_label_assist/performance_eval.py", line 179, in cross_val_predict
    pred_df = model_.predict(test_trips)
  File "/Users/ssaini/Documents/GitHub/e-mission-eval-private-data/TRB_label_assist/models.py", line 192, in predict
    proba_df = self.predict_proba(test_df)
  File "/Users/ssaini/Documents/GitHub/e-mission-eval-private-data/TRB_label_assist/models.py", line 742, in predict_proba
    axis=1)
  File "/Users/ssaini/miniconda-4.8.3/envs/emission-private-eval/lib/python3.7/site-packages/pandas/core/frame.py", line 8861, in idxmax
    indices = nanops.nanargmax(self.values, axis=axis, skipna=skipna)
  File "/Users/ssaini/miniconda-4.8.3/envs/emission-private-eval/lib/python3.7/site-packages/pandas/core/nanops.py", line 71, in _f
    return f(*args, **kwargs)
  File "/Users/ssaini/miniconda-4.8.3/envs/emission-private-eval/lib/python3.7/site-packages/pandas/core/nanops.py", line 924, in nanargmax
    result = values.argmax(axis)
ValueError: attempt to get argmax of an empty seque

On investigating, it seems model was fit and saved correctly. when predict_proba is called in NaiveBinningClassifier , which has a call for predict_cluster_confidence_discounting in emission.analysis.classification.inference.labels.inferrers .   Inside this is a call for predict_labels_with_n which is initiating Greedy similarity binning call. However a different model was used for fitting. This seems to be causing the error.

To tackle this, we can use Greedy similarity from trip_model (with fixed radius) for fitting and save that model. And then load would work similarly.

The other and better way is to check if the model_ is of instance ClusterExtrapolationClassifier and then call a different fit function for it.

I am not sure that this is better. Your instinct is to have if/else everywhere. that is not necessarily always correct.
Consistent interfaces are sometimes better than avoiding new code (but not always)

To tackle this, we can use Greedy similarity from trip_model (with fixed radius) for fitting and save that model. And then load would work similarly.

So have you actually done this yet?

Yeah. This  is already done in commit : a34836faf364f9548d3ab5373c8efd0d1e729a53.
Before the commit, this files wouldn't run and threw error. Its now working and generated the Cross validation CSVs .
Let me run the classification_performance.ipynb again put in the performance graphs here.

Plots below are from the paper and from the latest run of the code . They match.
Accuracy  and Weighted F-score for each model :
Plot from the paper

Plot from classification_performance.ipynb after the changes

Weighted F-score for purpose prediction
Plot from the paper

Plot from classification_performance.ipynb after the changes


How to get the 3d data not the 2d Xray after registiration

I want to use the elastix for my surgery registration based on introperation Xray(2D) and preoperation CT scan(3D). I am studing the registration following http://elastix.bigr.nl/wiki/index.php/Par0013. However, the out data is 2d xray which is not what I want for my navigation.
Maybe I have a mistake in the method. Please give me some advice for my goal. Thanks.

@  @timeanddoctor ,I have the same problem? you solve it ?

Thanks for submitting this issue. Did you find a solution? If not, maybe you can get help from https://groups.google.com/g/elastix-imageregistration  For now, we'll close this issue.


Tests failing on main

This happened after #325 but it is probably caused by #307

Fixed via #308


Confusion between currency and locale formats

It seems this lib only allows one format per currency.
However, the format, in particular symbolOnLeft, usually depends on the locale.
For example, I think Canadians speaking French will use 5$ while English speakers write $5.
In a similar way, if I'm targeting brits traveling in Europe, I would write €5 instead of 5 €.
Is there something I'm missing?

The locale specific formatting is a bit tricky.
@edorivai created a solution for us to be able to format currencies based on user's locale but you won't be able to format USD currency for en-GB locale for example. It will show the £ symbol in that case. Checkout #29
Some examples:
currencyFormatter.format(1000000, { locale: 'de-DE' });
// => '1 000 000,00 €'

currencyFormatter.format(1000000, { locale: 'nl-NL' });
// => '€1.000.000,00'

Our locale formatting data is very limited at the moment. You can add the missing locales here localeFormats.json and send a PR.
Another solution is to use the Intl APIs. More specifically the NumberFormat. The only problem with the Intl APIs is that it's not consistent across different browsers.

I created the locale based formatting, only for the use case of formatting the currency that corresponds with that specific locale. Frankly, I didn't even think about formatting a different locale. However, the current setup still allows you to use a locale based format for a different currency!
@smirzaei is right, the locale formatting data is very limited, I only added the nuances in rendering Euro values, and am hoping for others to help us out here by adding more locale formats.
To give you a bit more info into how it works; the locale formats in localeFormats.json are used to overwrite the default formatting for a currency. If you don't specify a specific currency code, but you do specify the locale, it will try to find the appropriate currency for that locale. For example { locale: nl-NL } will automatically select EUR as the currency. The default formatting for EUR looks like this:
	{
		"code": "EUR",
		"symbol": "€",
		"thousandsSeparator": " ",
		"decimalSeparator": ",",
		"symbolOnLeft": false,
		"spaceBetweenAmountAndSymbol": true,
		"decimalDigits": 2
	},

And the locale formatting for nl-NL looks like this:
	{
		"symbolOnLeft": true,
		"spaceBetweenAmountAndSymbol": false,
		"thousandsSeparator": "."
	},

The formatting data is applied like so:
assign({}, defaultCurrency, findCurrency(code), localeFormat)

Which means that the fields in the locale format for nl-NL will overwrite those the defaults for EUR. This results in the following formatting object:
	{
		"code": "EUR",
		"symbol": "€",
		"thousandsSeparator": ".", // overwritten
		"decimalSeparator": ",",
		"symbolOnLeft": true, // overwritten
		"spaceBetweenAmountAndSymbol": false, // overwritten
		"decimalDigits": 2
	},

This mutates an original format of 1 234,56 € to €1.234,56.
So this system should work fairly well for your use case. All you need to do is add the formatting rules for the locales you need, and you should be good to go.
One last thing; this issue actually brought up a problem with the current version of the locale formats. I've already submitted a PR in #38, so you might want to wait until that is merged ;).

Oh, and if it wasn't clear already, after adding your locale formatting, just use it like so:
currencyFormatter.format(1234.56, { code: 'USD', locale: 'fr-CA' });
// => '1 234.56 $'
currencyFormatter.format(1234.56, { code: 'USD', locale: 'en-CA' });
// => '$1 234.56'

Output will depend on your implementation of the locale format of course :)

Thank you for considering this issue.
In the meantime, I found the currencyFormatter.js lib from OSREC.
While some features are missing, the currency and locale formats can be set separately.
FYI you can find my fork here, adding the ability to display prices as integers:
https://github.com/Gwened/currencyFormatter.js

@Gwened thanks for the link to currencyFormatter.js. That's a really cool lib, especially their base of currencies and locales. I'm gonna see if I can reuse their config base in this lib. It seems that they don't support code splitting (in other words; only including the currencies and locales that you want to support), which is something we're looking into for v2.


Get cmake.buildDirectory directly from cmake tools

Get cmake.buildDirectory from cmake tools directly via command. If the cmake tools extension is not present, we will fallback to the content of the cmake.buildDirectory in the settings.json file as before.
Closes #28
Big shout out to @Shatur95 🙏

Thank you a lot, works fine!

Nice. I've deployed V0.6 with all the changes, so you can use the stable version.
Thanks for your support 🙏

Thank you!


Metadata

Hi,
I unfortunately lost the data after having downloaded it. I am more interested in the metadata than the images. Would it be possible to provide a package of just the metadata and no images please?

For internal reasons, I separated UnrealEgo into 2 different sets (See the current version of README).
“UnrealEgo 01-06” contains original data, such as RGB images, Depth images, and JSON files (~1.2TB in total).
“UnrealEgo impl 01-03” contains preprocessed NumPy files (~1TB in total), such as resized RGB images, 2D heatmaps, and pelvis-relative 3D keypoints, which can be used to try our implementations without downloading original data.
We might consider restructuring the dataset in the future.


Beheim/release0.7

Fixing typos and grammar.

Docs Build status updates of commit 8bd6d97:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/quantum-docs-pr, working directory: W:\r2dw-s, target branch: master, target branch commit: 85d179061eea2cd542740ea14cecd245de4c33ba, source commit: 8bd6d97f755608e39ff029a3c45d85b772a86985.

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

Docs Build status updates of commit 1634563:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/language/type-model.md
:warning:Warning

Details

articles/language/statements.md
:white_check_mark:Succeeded

articles/libraries/index.md
:white_check_mark:Succeeded

articles/libraries/numerics/index.md
:white_check_mark:Succeeded

articles/libraries/numerics/installation.md
:white_check_mark:Succeeded

articles/libraries/numerics/numerics.md
:white_check_mark:Succeeded

articles/libraries/numerics/toc.yml
:white_check_mark:Succeeded

articles/libraries/toc.yml
:white_check_mark:Succeeded

articles/language/type-model.md

[Warning] 1 invalid cross reference(s) "microsoft.quantum.language.statements".

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

Docs Build status updates of commit e53abc5:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/language/type-model.md
:warning:Warning

Details

articles/language/statements.md
:white_check_mark:Succeeded

articles/language/type-model.md

[Warning] 1 invalid cross reference(s) "microsoft.quantum.language.statements".

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


I refreshed my local repo and now get this error, Not finding libcublas.so

llm/llama.go:58:12: pattern llama.cpp//build//bin/*: cannot embed file llama.cpp/ggml/build/cuda/bin/libcublas.so.: invalid name libcublas.so

Resolved by deleting the local folder and rebuilding from scratch.


remove wants on synch targets in stop-instructions

openbmc/phosphor-state-manager#21 highlights an architecture issue with
OpenBMC's use of synchronization targets. When a service, such as
<EMAIL_ADDRESS>runs both in a standard power off
target, as well as in other paths (like the host quiesce path), there
is an issue.
The service starts the synchronization targets in the quiesce path and
this causes them to already be running on the power off, resulting in
the synchronization targets not actually coordinating the power off.
The direction this commit takes OpenBMC is that if a service needs to
run outside of the standard power on or off path, then they can not
have a Wants or Requires clause in the service file.
The following commit was done a while back to address this issue:
https://gerrit.openbmc.org/c/openbmc/phosphor-state-manager/+/40026
That is that we ensure the primary power on and off targets start the
synchronization targets so services requiring them can just use a
Before or After clause.
The piece that was never done was to go and fix the services which fell
into this bucket.
Tested:

Did multiple boots, reboots, and host crash tests and saw no issues

Signed-off-by: Andrew Geissler<EMAIL_ADDRESS>Change-Id: Ida68d83e2c2c18484eb4f28bc55c91fa5ff94930

Changes LGTM.
@rfrandse  1020 branch is used for fw1020.10 ?  Incase need any emergency fix in fw1020 , how are going to handle?
@geissonator


Improve the commit process and remove the CHANGELOG

Right now its getting difficult to track what issues got fixed in different branches. Sometimes we don't put ISSUE in the commit message, sometimes its BUG- and sometimes there are multiple commits for fixing an issue.
We need to follow some conventions for the commits like

Mandatorily have ISSUE- in the commit  message
strictly one commit per issue
Preferably have some script that can squash and validate the restrictions and merge.

This will make it easy to track the issues that got fixed in different branches and compare the branches.

Also consider integrating with build (travis) since we often see build failures after patches are merged to master.

 We may need to discuss on whether github repo should be maintained as root and gerrit as mirror of that to allow non horotnworks committers to commit directly into root repo.

Basically I would like to craft merge script which pulls the branch, and squashes commits into one, and merge without merge commit. (We can borrow Spark merge script and modify a bit to achieve it easily.)
There may be some considerations about merge script:

strictly one commit per issue

As I talked with @arunmahadevan we could allow exceptional cases: a PR has multiple authors. In this case we should respect authorship, so shouldn't squash commits or should squash commit per author (without breaking commit sequence). Not sure we would want to consider this at now and make it automatically.


[WIP] Imageio

Picking up an old suggestion from @Eric89GXL this uses imageio for saving movies. Imageio downlaods and manages the FFMPEG binary automatically so this would provide considerable simplification.

Coverage increased (+3.9%) to 76.21% when pulling 08d538aa65e8d3f8a9cdbe9e9658428946a39634 on christianbrodbeck:imageio into 40621da655859fd73404a2e6018953a4ad3daf10 on nipy:master.

nice to see these red lines without reducing features :)

Coverage increased (+3.8%) to 76.14% when pulling 10135dad09dcb82427f1e077de84be535c356229 on christianbrodbeck:imageio into 40621da655859fd73404a2e6018953a4ad3daf10 on nipy:master.

@agramfort I turned it into an optional dependency

@mwaskom @Eric89GXL any objection?

Otherwise LGTM

Coverage increased (+3.8%) to 76.14% when pulling c5ecdb723688bdcb2a7dacb3ccc4d40a0948c32b on christianbrodbeck:imageio into 40621da655859fd73404a2e6018953a4ad3daf10 on nipy:master.

Coverage increased (+3.9%) to 76.19% when pulling f2190219a961bda9b3525fea6486cb0b8dd51667 on christianbrodbeck:imageio into 40621da655859fd73404a2e6018953a4ad3daf10 on nipy:master.

Coverage increased (+3.9%) to 76.19% when pulling f2190219a961bda9b3525fea6486cb0b8dd51667 on christianbrodbeck:imageio into 40621da655859fd73404a2e6018953a4ad3daf10 on nipy:master.

LGTM

I might make sense to update the save_movie parameters to give better access to the imageio options (http://imageio.readthedocs.io/en/latest/format_ffmpeg.html#parameters-for-saving) but that could be a second PR

Yeah a second PR would be good. Thanks @christianbrodbeck

Thanks!


No metrics for: dnstap_responses_refused

Hi there,
I didn't find any metrics which can be related to: dnstap_responses_refused
Could you please add it?

added in release 4.0

added in release 4.0


Change the documentation to describe the "Clone" or github icon

Take screenshots for the docs.

Lets move this for now and leave the instructions as is.  We can update the "clone" icon at a later date, maybe even after the new toolbar is built.


New Feature Request

Feature details
Is there a way I can access any code that I have done in one desktop to a laptop without having to restart my laptop?
Additional information
No response

Hey @erdabravest2001! Sorry for the unacceptable response time here. As it currently stands, this is an issue that we are aware of (i.e., being able to access your progress in the codebook on any given device) and want to address in the future.
I will leave this issue open, as we don't yet have a fix.


[QUESTION] Other Apps

[ ] Custom Content

Are there any plans to include these apps: Project, Visio, To-Do, Delve, Publisher
We've got a lot in here, I have users asking for the content for these apps as well.

Assigning to someone who can provide an answer

powerbi would also be a nice include.

More on Stream too - lots of questions from business on how to manage who can view it

@ec-alm You can, there are various ways of doing so from custom sub categories to custom content packs, I suggest you watch the overview video for a complete rundown of all the ways to add custom content or build your own content. https://www.youtube.com/watch?v=-sLXl_rKGxQ

I'm going to close this issue as answered. If you have further issues, please feel free to create a new issue and reference this one.

I'm going to close this issue as answered. If you have further issues, please feel free to create a new issue and reference this one.


Crypto component for Azure Key Vault

Spin-off from #2237
Includes a new SubtleCrypto component for Azure Key Vault, which uses keys stored within the vault (as keys, so the key material never leaves the vault and all operations are performed within the vault).
Conformance tests are enabled.

Note that conformance tests are enabled for asymmetric ciphers only. Although Azure Key Vault supports symmetric ciphers too (AES), they are only available in the "Managed HSM" tier, which is insanely expensive and not suitable for our CI.

/ok-to-test


Sync-DbaLoginPermission, fix unused var

Type of Change

[x] Bug fix (non-breaking change, fixes #)
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose
Somewhere in history sa check was removed, so that piece is not used anymore

failed test is unrelated ✔


Installing from Github repo causes "An internal server error has occurred" error

Describe the bug
When attempting to access the Org Check app after cloning from GitHub and pushing to sandbox org, only the Home tab loads, the superhero dog image with "Please wait while Org Check is loading..." never goes away, and this message shows up at the bottom of the page: "An internal server error has occurred Error ID:<PHONE_NUMBER>-000000<PHONE_NUMBER>)". Enabling Lightning debug mode did not illuminate the issue.
To Reproduce
Clone the SalefsorceLabs/OrgCheck github repo. Deploy the force-app folder to a sandbox environment. Assign the "Org Check User" permission set to the running User in your sandbox. Attempt to access the Org Check App, note that the app tabs do not load correctly and the home page never loads.

Note: We don't want to install the managed package because three tabs cause our app to crash; we want to be able to hide those tabs to prevent this situation. The unlocked package link is far out of date and is still using visualforce pages rather than LWC.

Thank you for this issue.
I will try to reproduce the issue next week.
It is true that the Beryllium version (with LWC) is available on the main
branch of this repo. But the previous version (with Visualforce pages) is
still available on the appexchange.
I put the beryllium version on the main branch maybe too early in the UAT
process.
I am curious what is the error you have when installing the app from AppExc
or unlocked package.
Le sam. 20 avr. 2024 à 01:08, lrockford @.***> a écrit :

Describe the bug
When attempting to access the Org Check app after cloning from GitHub and
pushing to sandbox org, only the Home tab loads, the superhero dog image
with "Please wait while Org Check is loading..." never goes away, and this
message shows up at the bottom of the page: "An internal server error has
occurred Error ID:<PHONE_NUMBER>-000000<PHONE_NUMBER>)". Enabling Lightning
debug mode did not illuminate the issue.
To Reproduce
Clone the SalefsorceLabs/OrgCheck github repo. Deploy the force-app folder
to a sandbox environment. Assign the "Org Check User" permission set to the
running User in your sandbox. Attempt to access the Org Check App, note
that the app tabs do not load correctly and the home page never loads.
—
Reply to this email directly, view it on GitHub
https://github.com/SalesforceLabs/OrgCheck/issues/385, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA47R23JOCC4NMLB5OJB7BLY6GPVFAVCNFSM6AAAAABGP6FQ52VHI2DSMVQWIX3LMV43ASLTON2WKOZSGI2TIMRQGMZDSOA
.
You are receiving this because you were assigned.Message ID:
@.***>

Understood! Maybe I can use the Visualforce version in the meantime, I made a bad assumption that the managed app was on lightning so I thought the unpackaged app was out of date. I will gather the issues we faced in our org and share with you, it was mainly around tabs crashing, likely due to the volume of some of our metadata.
The three pages I recall crashing were:

Public Groups (never loads anything)
Users (loads but crashes quickly after load, can't click on any links or do any scrolling, browser popup will pop and ask if we want to wait or reload after a while)
Custom fields (same as users)

I'll try to provide more details in a follow up, would you prefer I open a new issue for these?

yes please


Add temporary mixlib-shellout constraint.

Fixes https://github.com/portertech/kitchen-docker/issues/129
Details here: https://github.com/portertech/kitchen-docker/issues/129

Closing this PR due to inactivity. Please submit a new PR if this issue still exists. Thanks!


us-nc-warren

warren county
http://www.warrencountync.com/gis.aspx
http://maps2.roktech.net/warren/

Well, we added it anyhow in https://github.com/openaddresses/openaddresses/pull/2816.

Apparently statewide NC hasn't worked in years:  https://results.openaddresses.io/sources/us/nc/statewide
I'll see if I can find it again.

Thank you!
On Fri, Apr 7, 2017 at 3:00 PM, Stephen K Hess<EMAIL_ADDRESS>wrote:

Apparently statewide NC hasn't worked in years:
https://results.openaddresses.io/sources/us/nc/statewide
I'll see if I can find it again.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/openaddresses/openaddresses/issues/548#issuecomment-292662370,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0EOy9ah8Wdy68mrzKXycc-KfK7ZYJbks5rtrIHgaJpZM4CwMku
.

Found it here:  https://services.nconemap.gov/secure/rest/services/NC1Map_Location/MapServer/0
I can't tell for certain but it doesn't look like it's been updated since 2014.  I'm going to keep pushing forward on individual counties in the hope that their data is updated more often.


NTR: RNA ligase (GTP) activity

Please provide as much information as you can:

Suggested term label:
RNA ligase (GTP) activity

Definition (free text)
Catalysis of the joining of an RNA with 3'-phosphate or 2',3'-cyclic-phosphate ends to an RNA with 5'-hydroxy ends according to either (i) a 3'-end 3'-phospho-ribonucleotide-RNA + a 5'-end dephospho-ribonucleoside-RNA + GTP = a ribonucleotidyl-ribonucleotide-RNA + diphosphate + GMP; or (ii) a 3'-end 2',3'-cyclophospho-ribonucleotide-RNA + a 5'-end dephospho-ribonucleoside-RNA + GTP + H2O = a ribonucleotidyl-ribonucleotide-RNA + diphosphate + GMP + H+.

Reference, in format PMID:#######
PMID: 21311021

Gene product name and ID to be annotated to this term
Worm rtcb-1 (P90838)
Human RTCB (Q9Y3I0)
fly RtcB (Q9VIW7)

Parent term(s)
RNA ligase activity (GO:0008452)

Synonyms (please specify, EXACT, BROAD, NARROW or RELATED)
3'-phosphate/5'-hydroxy nucleic acid ligase activity (EXACT)

Cross-references
EC:<IP_ADDRESS> (exactMatch)
RHEA:68076 (narrowMatch)
RHEA:68080 (narrowMatch)
MetaCyc:RXN-17905 (narrowMatch)
MetaCyc:RXN0-6566 (narrowMatch)

Any other information

There is a related EC/GO term, which seems to have caused some confusion:
id: GO:0003972
name: RNA ligase (ATP) activity
namespace: molecular_function
def: "Catalysis of the reaction: ATP + ribonucleotide(n) + ribonucleotide(m) = AMP + diphosphate + ribonucleotide(n+m)." [EC:<IP_ADDRESS>]
synonym: "poly(ribonucleotide):poly(ribonucleotide) ligase (AMP-forming)" RELATED [EC:<IP_ADDRESS>]
synonym: "polyribonucleotide ligase activity" RELATED [EC:<IP_ADDRESS>]
synonym: "polyribonucleotide synthase (ATP) activity" RELATED [EC:<IP_ADDRESS>]
synonym: "ribonucleic ligase activity" BROAD [EC:<IP_ADDRESS>]
xref: EC:<IP_ADDRESS>
xref: MetaCyc:RNA-LIGASE-ATP-RXN
is_a: GO:0008452 ! RNA ligase activity
Note the def for GO:0003972 should be updated to the current EC def:
"ATP + (ribonucleotide)n-3'-hydroxyl + 5'-phospho-(ribonucleotide)m <=> (ribonucleotide)n+m + AMP + diphosphate"
Which then also makes it clear that this activity acts on 3' hydroxyl and 5' phospho ends (in contrast to EC:<IP_ADDRESS>, which is the other way round)
So there may be some related annotation issues to be checked and maybe fixed too.
E.g.

Human RTCB (Q9Y3I0) is annotated to EC:<IP_ADDRESS>/RHEA:68076/RHEA:68080 in UniProt from PMID: 21311021 but is annotated with 'RNA ligase (ATP) activity (GO:0003972)' (=EC:<IP_ADDRESS>) from same paper. @
Worm rtcb-1 (P90838) is annotated to EC:<IP_ADDRESS>/RHEA:68076/RHEA:68080 in UniProt from PMID: 25366321 but is annotated with 'RNA ligase (ATP) activity (GO:0003972)' (=EC:<IP_ADDRESS>) from same paper.
"IPR027513 RNA-splicing ligase RtcB homologue, eukaryotic" has associated EC of EC:<IP_ADDRESS> but is annotated with  'RNA ligase (ATP) activity (GO:0003972)' (=EC:<IP_ADDRESS>)'

This also affects PTN000122119, but fixing the above human and worm annotations will fix that.

I also see there are these related BP terms, that look like they describe MFs? - not sure they should be kept?
id: GO:0000970 tRNA exon ligation utilizing GTP as source of linkage phosphate
=> 0 annotations
GO:0000969 tRNA exon ligation utilizing ATP as source of linkage phosphate
=> 0 annotations
GO:0000971 tRNA exon ligation utilizing 2',3' cyclic phosphate of 5'-exon as source of linkage phosphate
=> 1 EXP annotation

I added RNA ligase (GTP) activity and updated the definition for GO:0003972. I left the GO:0000970, GO:0000969, and GO:0000971 alone since it seems like these obsoletions should probably be made into new tickets.

Thanks @edwong57
Would you please you create an annotation review ticket for the incorrect annotations you spotted above?
(maybe all annotations need to be reviewed, I let you judge).
Thanks, Pascale

Hi @pgaudet What do you think about the 3 related BP terms I mentioned at the bottom of the original comment - would those be considered MFs?  I didn't want to make a separate obsoletion ticket for those without some feedback.

Realised that the def of the parent term 'GO:0008452 RNA ligase activity' also needed tweaking:
def: "Catalysis of the formation of a phosphodiester bond between a hydroxyl group at the end of one RNA chain and the 5'-phosphate group at the end of another."

def: "Catalysis of the formation of a phosphodiester bond between a hydroxyl group at the end of one RNA chain and the phosphate group at the end of another."
I just made that change.

So there may be some related annotation issues to be checked and maybe fixed too.

Human RTCB (Q9Y3I0) is annotated to EC:<IP_ADDRESS>/RHEA:68076/RHEA:68080 in UniProt from PMID: 21311021 but is annotated with 'RNA ligase (ATP) activity (GO:0003972)' (=EC:<IP_ADDRESS>) from same paper. @

Worm rtcb-1 (P90838) is annotated to EC:<IP_ADDRESS>/RHEA:68076/RHEA:68080 in UniProt from PMID: 25366321 but is annotated with 'RNA ligase (ATP) activity (GO:0003972)' (=EC:<IP_ADDRESS>) from same paper.

"IPR027513 RNA-splicing ligase RtcB homologue, eukaryotic" has associated EC of EC:<IP_ADDRESS> but is annotated with 'RNA ligase (ATP) activity (GO:0003972)' (=EC:<IP_ADDRESS>)'

This also affects PTN000122119, but fixing the above human and worm annotations will fix that.

I disputed the human annotations in P2GO and Sylvain has updated these.

@vanaukenk - can you review the rtcb-1 (P90838) annotation from PMID: 25366321 - I think it should be changed to the new term GO:0170057 RNA ligase (GTP) activity, or else the parent 'RNA ligase activity (GO:0008452)' term.

IPR027513 issue has been fixed in https://github.com/geneontology/go-annotation/issues/5152

PAINT issue is documented in https://github.com/geneontology/go-annotation/issues/5166

Yep, as noted above!

I made checkboxes - that's easier for me ;)

This also affects PTN000122119, but fixing the above human and worm annotations will fix that.

I am less sure about that ; the PAINT annotation cannot be automatically changed in primary annotations change. The only automatic change is if there is an obsolete with replace_by.
Do we need a ticket for that one?

Yes! From my comment above:

PAINT issue is documented in https://github.com/geneontology/go-annotation/issues/5166

I've updated the C. elegans rtcb-1 annotation.


how to take a photo ?

how to take a photo ,,using button?

I had not implemented it. This is just a demo to show how to use shader to implement some filters.

Can you please add the code to take a photo?

@johan-schoeman It is not a hard work. Ok, I will add it quickly.

That will be great! Thanks a lot - really appreciate it!

@muafakul @johan-schoeman  Take a look at the lastest 1.5 version. I have added new feature to caputre the image.

Thank you very much! Will test it a bit later today and will let you know.

I have not optimize it because I am now focusing more on the 2.0 version development. So please keep the issue open util I finished the optimization.


Minor UI changes

What does this PR do?
closes #61
How can this be tested
preview link: https://deploy-preview-62--blissful-mcnulty-82f253.netlify.app/

@AbhinavThukral97  can you confirm if changes are as per your suggestions?


Random advertiseement pop up when execute the test

The problem
I have a site https://master.nst.com.my/sports. There is advertisement pop up display randomly for certain menu options. There is no rules regard this. How to detect when there is pop up display then i just close the pop up.
https://drive.google.com/file/d/1FZMdu1Er_ny9issuzXC8MLdHU9q0-1tY/view?usp=sharing
https://drive.google.com/file/d/1uqPDzPdKX4qrl_nSKeOsOz2kzj93bAjB/view?usp=sharing
Details
I have a site https://master.nst.com.my/sports. There is advertisement pop up display randomly for certain menu options. There is no rules regard this. How to detect when there is pop up display then i just close the pop up.
Tell us about your environment

Selenide Version: 6.14.0
Chrome\Firefox\IE Version: 112
Browser Driver Version:
Selenide-Appium Version:
IOS\Android\WEB version:
Selenium Version: 4.8.3
OS Version: CentOS 9

Code To Reproduce Issue [ Good To Have ]

Interesting point here.

@nicholaswkc34 I just leave it here :)
https://selenide.org/2019/12/02/advent-calendar-how-to-abuse-selenide/


DDIM time delay

It seems the time delay in DDIM is at wrong place, as the delay is applied after releated operations are done.
https://github.com/lucidrains/bit-diffusion/blob/7a9b3ef518d405eea2b2a04de1c7515447a9286b/bit_diffusion/bit_diffusion.py#L501-L531

oops, yes, should be fixed! thanks!


Semi-sync is not enforced on primary when it restarts and when failover doesn't happen

Hi Shlomi!
Hope you're doing well.
I came across the following behaviour:
When semi-sync is enforced at orchestrator level, orchestrator does not set the primary node with rpl_semi_sync_master_enabled=1 on primary restart (when failover doesn't happen).
Orchestrator configuration for semi-sync:
"DetectSemiSyncEnforcedQuery": "SELECT @@global.rpl_semi_sync_slave_enabled AND @@global.rpl_semi_sync_master_timeout >= 1000000;"
All of our nodes have this enforced:
MariaDB [(none)]> SELECT @@global.rpl_semi_sync_slave_enabled AND @@global.rpl_semi_sync_master_timeout >= 1000000; +-------------------------------------------------------------------------------------------+ | @@global.rpl_semi_sync_slave_enabled AND @@global.rpl_semi_sync_master_timeout >= 1000000 | +-------------------------------------------------------------------------------------------+ |                                                                                1 | +-------------------------------------------------------------------------------------------+
We deploy our clusters with rpl_semi_sync_master_enabled=0 as default and set rpl_semi_sync_master_enabled=1 at runtime in the current primary. Whenever there's a failover (manual or automatic), orchestrator takes care of setting rpl_semi_sync_master_enabled=1 at runtime in the new primary and rpl_semi_sync_master_enabled=0 to the old primary.
However, if primary is restarted and there's no failover, orchestrator does not enforce rpl_semi_sync_master_enabled=1 on the primary.
Do you think it makes sense for orchestrator to be routinely enforcing rpl_semi_sync_master_enabled=1 on the primary?
Thanks,
Pedro.

Ouch, completely missed this.

Hopefully some discussion to follow on https://github.com/openark/orchestrator/issues/1360


enable WebSerial start button twice

What I have done:

make requestSerialButton disabled while measuring WebSerial power
make serial port closed after stopping measurement

I checked it works in the following situation;

start(serial) -> stop, disconnect: close serial port
start(usb) -> stop, disconnect: close usb port

After both situations, power can be measured again.

I fixed comments!

Thank you hikalium, I just fixed the position of comments for now.
By the way, I have one question about your first advice. Which does the situation mean formatting a file (e.g. formatting index.ts) or change style for the project overall (e.g. adding gts)?

As I chatted offline, "the project-wide coding style fixes" and "functional changes" are usually separated logically, so they should be filed as separated pull-requests to make the review easier. Anyway, now your change looks good for me, so I merge it into the main branch. Thanks for the great work!!


Fix authorisation

Sorry, thought I'd done this a while ago.
This pull request does two things:

Fixes authorisation where xbmc requires user name and password
Adds __nonzero__ method so user can test if xbmc is active when initialising object.

Can confirm, this fixes authorization issues I encountered on master.

Closing as not relevant anymore


Feature: update helm chart images tag to v0.1.0.beta

Signed-off-by: Jim Ma<EMAIL_ADDRESS>
Description
Update helm chart images tag to v0.1.0.beta
Related Issue

Motivation and Context

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Documentation Update (if none of the other choices apply)

Checklist:

[ ] Code compiles correctly.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Codecov Report

Merging #395 (2bfa6b2) into main (8cd293b) will decrease coverage by 0.17%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #395      +/-   ##
==========================================
- Coverage   41.82%   41.65%   -0.18%
==========================================
  Files          73       72       -1
  Lines        5640     5637       -3
==========================================
- Hits         2359     2348      -11
- Misses       2943     2949       +6
- Partials      338      340       +2

Flag
Coverage Δ

unittests
41.65% <ø> (-0.18%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/util/net/iputils/ip_utils.go
61.22% <0.00%> (-4.09%)
:arrow_down:

client/daemon/peer/peertask_stream_callback.go
62.50% <0.00%> (-2.50%)
:arrow_down:

client/daemon/peer/peertask_stream.go
56.98% <0.00%> (-1.14%)
:arrow_down:

client/daemon/peer/peertask_base.go
33.49% <0.00%> (-0.98%)
:arrow_down:

client/daemon/peerhost_stub.go


Correctly change the working directory in python

This is implementing #295 and https://github.com/neovim/neovim/issues/3636.

@Leandros can you include the fix @bfredl mentioned? Would like to move this forward

nvim.command('call rpcrequest({}, "python_chdir", {"cwd": getcwd()}', async=True)

seems to be a chicken-egg problem, instead I went with this:
os.chdir(nvim.eval('getcwd()', async=False))

Pushed an update to @Leandros branch, will merge after CI.

This appears to cause problems with initial use of :py3 for me: https://github.com/neovim/python-client/issues/337.


Move the deployment configuration to master branch

Is your feature request related to a problem? Please describe.
We need to use the master branch as the deployment branch
Describe the solution you'd like
A clear and concise description of what you want to happen.
Move the configuration to the master branch
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
N/A
Additional context
Add any other context or screenshots about the feature request here
N/A

Fixed by #154


Fix missing deno version on deploy

Using latests version of deno, amqp fail at runtime because Deno.version is undefined.
This is the detail of the error that I get using that library on supabase edge function based on deno.
TypeError: Cannot read properties of undefined (reading 'deno')
at https://deno.land/x/amqp@v0.21.0/src/amqp_connection.ts:12:36

Here some additional references
https://github.com/denoland/deploy_feedback/issues/98

I'm running into this exact same Issue. Can't get this running on the Supabase Edge Functions because of:
TypeError: Cannot read properties of undefined (reading 'deno')
    at https://deno.land/x/amqp@v0.23.1/src/amqp_connection.ts:12:36

The Official Deno APIs:
https://deno.com/deploy/docs/runtime-api#deno-apis

This should be fixed now: https://github.com/denoland/deploy_feedback/issues/98#issuecomment-1621831497


New Feature: handle deprecation of sniffs natively

Is your feature request related to a problem?

In PHPCS itself, this was most recently done in PHPCS 3.3.0 for the Squiz.WhiteSpace.LanguageConstructSpacing sniff and in PHPCS 3.4.0 for the Generic.Formatting.NoSpaceAfterCast sniff.
It can also be said that the MySource standard + all CSS/JS sniffs are effectively deprecated as they will be removed in PHPCS 4.0.
However, deprecating sniffs is not something which is unique to PHPCS itself. External standards will also, at times, want to deprecate sniffs.
Current options for deprecating a sniff
At this moment, when deprecating a sniff, there are effectively two options:

Soft deprecation, i.e. deprecation in name only via a mention in the changelog.
This will largely go unnoticed and end-users will be confronted with the removal in the next major.
Hard deprecation, i.e. deprecation by throwing a PHPCS warning from the sniff when the sniff is used.
This potentially breaks builds when CI is set to fail on warnings, which makes the impact of a deprecation too large for it to still be called a deprecation.

All in all, neither option is very satisfactory.
Describe the solution you'd like
Now to solve this conundrum, I'm thinking of introducing a new DeprecatedSniff interface which extends the existing Sniff interface and adds one additional method getDeprecationMessage().
interface DeprecatedSniff extends Sniff {
    /**
     * Retrieve an arbitrary deprecation notice.
     * @return string
     */
    public functon getDeprecationMessage();
}

When a sniff gets deprecated, the implements Sniff would then be swopped out for an implements DeprecatedSniff and the getDeprecationMessage() method would need to be added.
final class MySniff implements DeprecatedSniff {
    public function register() { ... }

    public function process() { ... }

    public function getDeprecationMessage() {
        // Example for a sniff which is deprecated with replacement:
        return 'This sniff is deprecated since [VERSION] and will be removed in [VERSION]. Use the [REPLACEMENT] sniff instead.';

        // Example for a sniff which is deprecated without replacement:
        return 'This sniff is deprecated since [VERSION] and will be removed in [VERSION].';
    }
}

In the PHPCS framework, when the ruleset gets loaded, it could then be detected if a sniff is deprecated and all sniff deprecation messages could be collected and displayed above the report output. Similar to messages about the ruleset containing a reference to a non-existent sniff or to a property which doesn't exist.
These deprecation messages would not influence the exit code of a PHPCS run and would therefore be non-breaking.
When the -q (quiet) option is used, these messages would also be silenced.
Opinions ?
Open questions:

Should the DeprecateSniff interface extend Sniff or should it be a stand-alone interface ?
A stand-alone interface would possibly allow it to be used more easily in combination with arbitrary abstract sniff classes.
Should there be just a getDeprecationMessage() method which allows for an arbitrary message or should the message be standardized more ?
If the message should be more standardized, this could be done by having multiple methods: getDeprecationVersion(), getRemovalVersion(), getReplacement(); or by a getDeprecationInfo() method which would expect an array with this info in three predefined keys.

/cc @kukulich @wimg @GaryJones @dingo-d @fredden @michalbundyra @stronk7
Additional Context
In a future iteration, I can imagine another interface which would allow for PHPCS natively handling the deprecation of one or more public properties in a sniff.

[x] I intend to create a pull request to implement this feature.

Should the DeprecateSniff interface extend Sniff or should it be a stand-alone interface?
Should there be just a getDeprecationMessage() method which allows for an arbitrary message or should the message be standardized more ?

I prefer standalone interface with just one method.

I think it's also better to have a single interface. When a sniff is deprecated you can just implement multiple interfaces (which to me sounds like a better OOP practice anyhow - composition over inheritance). That way it's not directly coupled to the Sniff interface.
And some standardization in the message output would be beneficial IMO, as that would bring more consistency across the standards. Imagine using 3-4 different standards, each having different depreciation messages, I think that would increase the cognitive load on the end user.
With a standardized depreciation message, users know what to expect, no matter which standard (bundled with PHPCS or not) they are using.

Notes to self - additional ideas to make this feature complete:

Mark deprecated sniffs as such in the sniff list which can be displayed when the -e option is used.
Display a notice that a sniff is deprecated when displaying the documentation for that sniff (--generator=... options).

I tend to agree with @dingo-d about the standardization of the messages, so my preliminary implementation is a stand-alone interface with three methods.
Here's screenshot of what the output could look like (work in progress, feedback welcome):

Next iteration - as before, feedback welcome. I expect to open a PR for this change later today.

And here's a screenshot of the output for "explain" when a standard contains deprecated sniffs:

Q: Is there any way to show the deprecation details in the "explain" report?
Other than that, it looks nice to me!

PR #281 is now open for this feature. Reviewing and testing would be appreciated.

Notes to self - additional ideas to make this feature complete:

Mark deprecated sniffs as such in the sniff list which can be displayed when the -e option is used.
Display a notice that a sniff is deprecated when displaying the documentation for that sniff (--generator=... options).

Of the above, the first has been included in PR #281. The second has not as it would involve more extensive changes. This could potentially still be added at a later date.

Unless anyone leaves a comment/feedback on the PR, I will merge it tomorrow.


Tag new release

Hey,
you didn't tag a new release for the 2.0. Can you do that or are you waiting the update of machuga package?

And if I use 2.0.*@dev I got this error:
composer update
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for efficiently/authority-controller 2.0.*@dev -> satisfiable by efficiently/authority-controller[2.0.x-dev].
    - efficiently/authority-controller 2.0.x-dev requires stayallive/authority-l5 ~2.4.1 -> no matching package found.

Potential causes:
 - A typo in the package name
 - The package is not available in a stable-enough version according to your minimum-stability setting
   see <https://groups.google.com/d/topic/composer-dev/_g3ASeIFlrc/discussion> for more details.

Read <http://getcomposer.org/doc/articles/troubleshooting.md> for further common problems.

Hi Romain,

you didn't tag a new release for the 2.0. Can you do that or are you waiting the update of machuga package?

I wait for @machuga package update then I'll release a stable version of Authority-Controller 2.0. But it doesn't mean it's unstable, you can used it in production.
For your error, read this, please:
https://github.com/efficiently/authority-controller/issues/19#issuecomment-82201011
Cheers,
Tortue Torche

Thanks for your quick answer! :+1:


Is there a Swagger definition of the API?

Where can we find a swagger (or other) definition of the API behind huginn?

Hi @deardooley,
we currently have no documentation of the API, most (if not all) calls accept and return JSON but there is no central place where its documented. To properly support an API it would probably be best to expose it via a separated (versioned) URL prefix. I would be open for pull requests either documenting the current API or separating it from the "internal" application calls.

I'm going to close this issue.  Please reopen if you want to discuss further!


Fix mavsdk test failure by increasing back motor constants of the iris

Describe problem solved by this pull request
Previously SITL tests were failing because the hover thrust was too low, which was decreased in https://github.com/PX4/sitl_gazebo/pull/575
Now, SITL tests have been failing since we decreased the motor constants on the iris since the hover thrust was too high
I think this points to the issue that the default configs need the hover throttle to be in a certain range, but this needs to be fixed on the flight control side, not the simulation.
@bresch FYI
Additional Context

https://github.com/PX4/sitl_gazebo/pull/584 is included. Once this PR is merged, will update the sitl_gazebo submodule to master and make this PR ready to review

Thanks @Jaeyoung-Lim


bug: generation max token limits

Currently generated Pydantic model has max_new_tokens set to 4096.
This value is not valid anymore and has been recently removed from the OpenAPI schema. We want to regenerate the schema and remove such limitations.
Workaround
from genai.schema import TextGenerationParameters

TextGenerationParameters(max_new_tokens=8000) # ❌
TextGenerationParameters.model_construct(max_new_tokens=8000) # ✅

Released in version 2.3.0.


Prevent spamming emails of registered users?

 Is there a way to prevent spamming the email of registered user by the random attacker?
Any suggestions for writing error handling would be appreciated!
I use express 4.X.

Hi seganus!
I think the easiest way is to store the IP address of the requests. And start to delay or entirely prevent sending new emails after the third or x try.

Hi
Let me recommend a slight amendment to @NoNameProvided 's proposal: Rather than blocking the IP address I would likely store the time of last request for each email address and delay/block on that basis. IP blocking could be another step but is a difficult topic as proxys / TOR / VPN are all resulting in shared IP addresses of several users... Blocking limits would hence have to be much higher if based on IPs.
Cheers

Thanks for the suggestions. I think express-brute might fit well. I have another question, is there a way to pass token in ejs as <%= token %> after successful accepting the token?

Can you detail what exactly you try to do? :)
Cheers

(1) I have a POST request handled by AJAX Jquery that should work when the value of "x-access-token" is set up by the server side call. Actually, I use every token generated to assign roles to specific users.
(2) I have an API route that passes data to some device (IOT). Since its used outside the main app, I have to set a token for it to work.
If it was a session based authentication I would generate a CSRF and pass that always for POST request. Since here I already have a token just need to pass it to work.
Thanks again for the valuable comments/suggestions.

For step 2: That request originates directly from the client? Why do you need a token? Can't you just push the data to the device?

Lets assume that the remote client is not a browser and only understands JSON.
(1) Before Token based authentication
app.get('/data/:id, require('somecode).GETJSON);
On GET /data/35246523654623545 - I great raw JSON stream
(2) After Token based authentication
var restricted = passwordless.restricted({ failureRedirect: '/login' });
app.get('/data/:id, restricted, require('somecode).GETJSON);
On GET /data/35246523654623545 - I great html content for /login
(3) Solution i think
var restricted = passwordless.restricted({ failureRedirect: '/login' });
app.get('/data/:id, restricted, require('somecode).GETJSON);
On GET /data/35246523654623545?token=VALUE&uid=VALUE I great raw JSON stream
`
(4) Offcourse i can implement this directly in ejs
<a href="/data/<%= id %>?token=<%= token %>&uid=<%= uid %>">Dummy data stream </a>
I figured <%= uid %> but not sure how to get <%= token %>?


[RFC] Link to discourse server

Discourse is a somewhat popular open source forum software. For example,
rust uses it: https://users.rust-lang.org/ .
I think it'd be nice if we had a bpftrace forum so commonly asked questions
can be catalogued and searched. It seems like discourse also supports
mailing lists so email-lovers can continue with that if they'd like. It also looks
like discourse has support for admin/moderator boards, so we can have
more private contributor discussions there instead of via email.

Maybe we should add a question/issue  issue template too that asks people to go to discourse when its not a bug/feature?

Maybe we should add a question/issue issue template too that asks people to go to discourse when its not a bug/feature?

Yeah that's a good idea

It looks like this now:


[Accessibility] AutoSuggestBox and NumberBox PlaceholderText fails Accessibility Insights Pass

Describe the bug
The PlaceholderText element for AutoSuggestBox and NumberBox fails Accessibility Insights Pass. Test failures are only observed for OS Build 19h1 and below.
PlaceholderText is temporarily removed from AXE test pages from #3764

Steps to reproduce the bug

Go to AutoSuggestBox and NumberBox test page in MUXCTestApp
Run Accessibility Insights over any element that has placeholder test.
Observe failed tests.

Expected behavior
Tests are supposed to return no errors.
Screenshots
No response
NuGet package version
Microsoft.UI.Xaml 2.7.0
Windows app type

[ ] UWP
[ ] Win32

Device form factor
Desktop
Windows version
May 2019 Update (18362)
Additional context
No response

If this is only present on 19H1 and below, could it be that this is an OS issue and not a WinUI 2 issue? If so, would it make sense to close this since there likely isn't much WinUI 2 could do to fix this?

The 19H1 statement appears to be incorrect. The AutoSuggestBox is getting flagged as an error in Windows 10 21H2.


Handle IEEE754 overflows as ±Infinity instead of raising errors

Highlighted by @nst in Parsing JSON is a Minefield 💣:

According to RFC 7159, "A JSON parser MUST accept all texts that conform to the JSON grammar" (section 9). However, according to the same paragraph, "An implementation may set limits on the range and precision of numbers."

Most parsers just handle overflows as ±Infinity.

Any opinion on this @JamesNK?

I ended up disabling the custom float parse logic so this change is no longer relevant.

Actually, converting overflow to infinity could still happen by catching OverflowException. Would you consider another pull request to handle overflows as ±Infinity?


Added more Frontend projects you can contribute

Fixes Issue
#584

Changes proposed
Gatsby
Twbs Bootstrap

Screenshots

Note to reviewers

https://github.com/WeMakeDevs/roadmaps/issues/584#issue-1783780442

I think your commit has some problem, so Git is unable to detect the existing lines properly. It may be helpful if you can add the new stuff in description as well.

Hey, @soumil1 it seems you might have edited all the existing resources as well. We are unable to figure out which ones were added by you. Either recreate a branch and re-commit the changes and open a new PR or you can mention the resources which you have added

Hey, @soumil1 it seems you might have edited all the existing resources as well. We are unable to figure out which ones were added by you. Either recreate a branch and re-commit the changes and open a new PR or you can mention the resources which you have added

Hey, @siddhant-khisty sorry from my end. Actually, on the first line, I mistakenly changed "## - 2nd level heading" to "# - 1st level" on Open Source Frontend projects to which you can contribute, which made it appear as if I had changed the entire Readme, but I haven't. I only added the "Gatsby" and "Twbs Bootstrap" links

I think your commit has some problem, so Git is unable to detect the existing lines properly. It may be helpful if you can add the new stuff in description as well.

Can I mention the new changes I made to the description now?

I think your commit has some problem, so Git is unable to detect the existing lines properly. It may be helpful if you can add the new stuff in description as well.

Can I mention the new changes I made to the description now?

Yes

Edited the description. Can you please check

Hey, @soumil1 these projects that you've included do not really fit under the frontend category. Imo they would be better suited as backend projects. Yes they are frontend frameworks, but a contributor would need a good amount of frontend as well as backend knowledge to contribute to them. I could be wrong tho. @AnuragThePathak @kaiwalyakoparkar what are your opinions on this issue?

Bootstrap one is frontend ig and Gatsby too (but these are not web apps, except the Bootstrap website).

So what do I do now? @siddhant-khisty @AnuragThePathak
Can I create a new pull request and update both of these links in backend development?

@soumil1 The bootstrap is one fine I guess. But please move the gatsby one to backend. You can simply push more commits to this branch to update this PR as well

@soumil1 please fix the merge conflicts and we can merge

Sorry for causing a problem. I accidentally force-pushed the commit instead of deleting the conflicts. Is there any way to recover from this disaster :(
@siddhant-khisty

Do you still have the commits with you? If yes we can recover from the disaster.


net/frr: simplify BGP API Controller

CC @mimugmail

@fichtner with that, the FRR plugin should be completely migrated and the plugin should be much smaller now. Should we do a release?

if you want why not... I'll move it to 18.1.2 then