Training speed and results

Sorry to bother you, I have two question about this project. for training work, I use multi gpus, the speed about 1.79it/s in first two epoch, it takes one day. After that, the speed is so fast, I don't know why this happened. Also the result, the detections repeatability is more better than rpautrat/Superpoint's, but the descriptors: hpatches-i is 0.90. hpatches-v is 0.55. Compare with rpautrat/Superpoint's, the result is not good.

Thanks for your attention. I have no idea about the speed. It is similar to my training process.  I guess the detector loss causes this phenomenon
The default hyper-parameters may not work well, you have to adjust them several times to get better results. The latest version adds two important variables, i.e., positive_dist, negative_dist, to help you fine tune the model. The parameters lambda_d, lambda_loss, positive_margin and negative_margin are the most decisive ones. You need to adjust them to make positive_dist and negative_dist as small as possible. (The weight i released in the latest version can achieve 0.66 acc. on hpatches_v)

Thx much with your reply! I will try finetune  these parameters!  0.66 acc for hpatches-v is  your best result? Also, is the result use pretrain model(superpoint_bn.pth)?

Thx much with your reply! I will try finetune these parameters! 0.66 acc for hpatches-v is your best result? Also, is the result use pretrain model(superpoint_bn.pth)?

I trained the model without any pre-trained model, and it cost me several days to achieve this performance. Training the model really needs experience and tricks. And i'm failed to find a better hyper-parameters that can directly obtain a good model. 0.66 may not be the best, but this is the best model i can get right now.

Got it! thx again!

my result never reach 0.60 for hpatches-v when try several hyper-parameters, can you share hyper-parameters with superpoint_bn.pth? because I inference this model, the result hpatches-v is 0.66. Also if you get better result that can share the experience. I will so appreciate!

If you want to run with superpoint_bn.pth, remember to set eps=1e-3 for all BatchNorm2d func. in backbone.py and cnn_head.py. Moreover, another parameter, i.e., momentum , may also work for superpoint_bn.pth (set momentum=0.01). I didn't test this parameter, you can try.

Yes, I get 0.66result when set eps=1e-3  for inference.  But when I training without any pre-trained model, how to set those hyper-parameters?  Besides, the optimizer is Adam not SGD, how to set momentum=0.01?

momentum=0.01 and eps=1e-3 is for BatchNorm2d, and only work for superpoint_bn.pth, which is converted from rpautrat's superpoint.
If you want to train your model using this pytorch scripts, i suggest you remove the parameters eps and momentum in BatchNorm2d
sp_0.pth is the model i trained without any pretrained weight.
As far as i know, this pytorch version is very sensitive to the parameters lambda_d and lambda_loss.  A bigger lambda_d will make the model more stable，however，the performance is not good.

Thanks! Ihave a last question about README.md -> Steps. Is that means training model need two stage? The firsts stage need Commet the following lines in loss.py and Set base_lr=0.01 , than use trained models train stage two: base_lr=0.001 and fix hyper-parameters:lambda_d = 250  #balance positive_dist and negtive_dist lambda_loss = 10 #balance detector loss and descriptor loss
The 7.Start training again.(lambda_d and lambda_loss may need to be adjusted several times). is means fix hyper-parameters several times to train model?(maybe stage three, stage four?)

If I use your last version, should I train how many times?

Also,  converted from rpautrat's superpoint didn't have  same result, Is possible have bug like this: _https://github.com/rpautrat/SuperPoint/issues/117_
_https://github.com/eric-yyjau/pytorch-superpoint/issues/24_

Set eps=1e-3 for BN will achieve 0.66 acc. on hpatches-v. I'm not sure if you can achieve similar performances by setting eps=1e-3, momentum=0.01. Because the default parameters of BN between tf and torch are different, eps and momentum are two key parameters. Moreover, the conv2d is also not exactly the same. So we can only get similar performances as the rpautrat's sp.
I'm not sure how many times you need to train. Right now, i haven't got an effective training process. However, i strongly suggest you reading the descriptor_loss functions in loss.py and debugging some of the key variables like dot_product_desc, positive_dist, negtive_dist. I think this will greatly help you adjust the hyper-parameters

Thanks for your answer!

Hi litingsjj, I found that the parameters for photometric is different from  rpautrat's SuperPoint. This may affect the training performance.
I have update superpoint_coco_train.yaml according to rpautrat's SuperPoint.
···
random_brightness: {max_abs_change: 0.2}
random_contrast: {strength_range: [0.5, 1.5]}
additive_gaussian_noise: {stddev_range: [0, 10]}
additive_speckle_noise: {prob_range: [0, 0.0035]}
additive_shade:
transparency_range: [-0.5, 0.5]
kernel_size_range: [100, 150]
nb_ellipses: 20
motion_blur: {max_kernel_size: 3}
···
And,i'm checking to see if there are any other problems.

great! what's the results  when you update the *.yaml? Also, I can't reproduce your result for now

Maybe my trained magic model is different with yours,  so export coco dataset  make affect performance. can you provide your magic model?
last week , i use your project, the descriptor loss  have some comment, like better comment this at the begining of training. I'm confused about your train strategy

a. The magic point trained by this repo. is different from rp's superpoint. It seems that our magic point usually generate more keypoints than sp's mp. It may be caused by homography_adaptation in homo_export_labels.py. Or you can set a larger det_thresh in *.yaml when generate coco labels.
b. According to this issue, it is better to comment the following lines
dot_product_dest = F.relu(dot_product_dest)
dot_product_desc = torch.reshape(F.normalize....)
dot_product_desc = torch.reshape(F.normalize....)

This may be unnecessary.
c. The repository have been updated.

change the photo_aug strategy in photometric_augmentation.py
Apply homo_aug first, then photo_aug in coco.py and synthetic_shapes.py

Moreover, I can not achieve similar training performances with these improvements, keep on checking...

Hi, i uncommented the following 3 lines and set lr=0.001, and train superpoint on the coco dataset generated by rpautrat's magicpoint model. The performance on hpatches-v is 0.698 !
    dot_product_desc = F.relu(dot_product_desc)

    ##l2_normalization
    dot_product_desc = torch.reshape(F.normalize(torch.reshape(dot_product_desc, [batch_size, Hc, Wc, Hc * Wc]),
                                                 p=2,
                                                 dim=3), [batch_size, Hc, Wc, Hc, Wc])
    dot_product_desc = torch.reshape(F.normalize(torch.reshape(dot_product_desc, [batch_size, Hc * Wc, Hc, Wc]),
                                                 p=2,
                                                 dim=1), [batch_size, Hc, Wc, Hc, Wc])

However, the magic point model still generate more key points than rpautrat's model.

if you worried about that, maybe we can use  rpautrat's model to export coco dataset to train superpoint. Also, the dataset(coco2017) is different with rpautrat used(coco2014). And can i use your last version to reproduce your result?

The performance on hpatches-v get 0.725!

Great! Can you share your training method ? I'm generating the coco labels now

Great! Would you to share your training method ? This may help lots of people who follow this repo.

I trained magic point with rpautrat's project to get coco gt points, then use your version to train superpoint.

OK, thinks

I trained magic point with rpautrat's project to get coco gt points, then use your version to train superpoint.

Hello,sorry to bother you, rpautrat's project generate the lables is .npz while this project generate lables is .npy,how do you solve this question?

I trained magic point with rpautrat's project to get coco gt points, then use your version to train superpoint.

Hello,sorry to bother you, rpautrat's project generate the lables is .npz while this project generate lables is .npy,how do you solve this question?

Hi, it is easy to convert *.npz to .npy. And remeber to zoom the coco image to 240320 by function ratio_preserving_resize

I trained magic point with rpautrat's project to get coco gt points, then use your version to train superpoint.

Hello,sorry to bother you, rpautrat's project generate the lables is .npz while this project generate lables is .npy,how do you solve this question?

Hi, it is easy to convert *.npz to _.npy. And remeber to zoom the coco image to 240_320 by function ratio_preserving_resize

Is that mean rpautrat's project's image size is not 240320 and this project's image size has been resized to 240320?

I trained magic point with rpautrat's project to get coco gt points, then use your version to train superpoint.

Hello,sorry to bother you, rpautrat's project generate the lables is .npz while this project generate lables is .npy,how do you solve this question?

Hi, it is easy to convert *.npz to _.npy. And remeber to zoom the coco image to 240_320 by function ratio_preserving_resize

Though rpautrat's lables are .npz,it is one image to one npz file,so it seems easy to convert.

hi, could you email me the coco gt points you got in<EMAIL_ADDRESS>i tried to generate them as you said, but the results are not good.

I trained magic point with rpautrat's project to get coco gt points, then use your version to train superpoint.

hi, could you email me the coco gt points you got in<EMAIL_ADDRESS>i tried to generate them as you said, but the results are not good.


Async Loading Screen With World partition

Hi,
I am new to UE, and started to use world partition (not level streaming).
I have all in a same level, including main menu (like the Medieval demo for UE 5 on the market place)
When I do not use your plugin, what happen is that the level starts before getting entirely loaded, which is quite bad: i assume that Level with World Partition starts automatically without being loaded.
So when I use your plugin, I don't have any loading screen, instead it automatically start with a black screen (which I imagine that it's just my level being loaded).
Do you have any recommandation on how to use your plugin with a single world partition level  ?

The Async Loading Screen is only triggered when you use the OpenLevel node. World Partition is still a level streaming loading, so I think my plugin won't work with it. Please follow the plugin tutorial to understand how to use it.


Limit thruster system's input thrust cmd

Signed-off-by: Ian Chen<EMAIL_ADDRESS>🎉 New feature
Summary
Add two new params to the thruster system for configuring the min and max input thrust cmd:

<max_thrust_cmd>
<min_thrust_cmd>

This affects both PID and vel controlled thrusters.
Test it
Run the INTEGRATION_thruster test
Checklist

[x] Signed all commits for DCO
[x] Added tests
[ ] Added example and/or tutorial
[x] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] codecheck passed (See contributing)
[ ] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

what do you think about cheking if min is bigger than max ?

added check in d795d0098a071dd3e33bcaf88ea86c3cb79850f3

This pull request has been mentioned on Gazebo Community. There might be relevant details there:
https://community.gazebosim.org/t/new-ignition-releases-2022-03-01-citadel-edifice-fortress/1313/1

I just tried merging this forward to main and I see some conflicts with https://github.com/ignitionrobotics/ign-gazebo/pull/1269

I just tried merging this forward to main and I see some conflicts with #1269

never mind there is already a merge-forward PR in https://github.com/ignitionrobotics/ign-gazebo/pull/1367, but I think it also is struggling with these conflicts

This pull request has been mentioned on Gazebo Community. There might be relevant details there:
https://community.gazebosim.org/t/new-ignition-releases-2022-04-13-fortress-edifice/1367/1


Validate form elements with numbers

Hi,
I have a textField for numbers, and need to perform simple validations on it.
sort of like, @mytextField.value > 100
The above fails because the value is a string and not a number.
Are there any conversion functions to convert form values from string to int and double ?
Or is there a better way to achieve the same validation ?
TIA.

I was thinking, if we can add 4 functions to the expression language:

bool isValidInteger(String) - Returns if a string successfully parses into an Int
bool isValidDouble(String) - Returns if a string successfully parses into a Double
int toInteger(String, int) - parses string into a Int, returns the value 2nd parameter if parsing fails.
double toDouble(String, double) - parses string into a double, returns the value 2nd parameter if parsing fails.

I will be happy to submit a PR if you are Ok with this.

I am not sure, but there should be expression method for parsing int like Integer.parse or int.parse did you tried?

Integer.parse / int.parse didn't work.

Hi,
I looked at the source code and there really is no string to integer conversion at the moment.
One workaround right now is to write custom component and define the property
of type Property<int> instead of Property<String>. You would then need to make sure only the ints will be assigned to the property (that can be done in the renderer part). However, it is not very ideal, because you may need String values in that property as well - in that case, you would need to have two properties and assign both on the input change.
I think your proposal looks good, those methods will definitely be handy and I will happily accept that PR! There are just 2 obstacles you may hit when you start implementing it.
With each new function, you will need to implement a visitor method on CloneExpressionVisitor  and TraversalExpressionsVisitor.
And the second thing is that internally we use custom number types Integer and Decimal so we can have a common Number ancestor. It gets automatically converted to regular int and  double if you use it in the component properties. So it may make more sense instead of isValidDouble to have isValidDecimal and instead of toDouble to have toDecimal. Additionaly you can also have Number toNumber(String, Number) and Number isValidNumber(String) to benefit from the common Number ancestor. There is already a parse method on the Number data type that you can use. See https://github.com/OndrejKunc/flutter_dynamic_forms/blob/2f03512d4aac8952a100570a1299f34775f069c1/packages/expression_language/lib/src/number_type/number.dart#L244
I hope that doesn't scare you from implementing it :). I know the documentation is really needed - at least for all the available functions. In the future, I would like to revisit the expression_language implementation and maybe find a way how to make it more customizable, so you would be able to write custom functions and inject them directly from your project.

Great!
Thanks, that really helps.
I'll go with a custom component with a numeric property for now.

Are we moving towards support for custom expression_language function?
I find some of my expression logic gets very redundant and the same expression is used across multiple entries.  It would be convenient to be able to use shortcuts like how now(), and in some cases functionality can't be achieved until custom expressions are supported.

Hi @sharpsan, it is the next big feature I would like to implement. I didn't have much time lately, but right now it looks like this project will be used in my current workplace, so I will probably have some capacity to do it soon.

@sharpsan I just released a new version of packages, which contains the possibility to implement custom expressions. See https://github.com/OndrejKunc/flutter_dynamic_forms/tree/master/packages/expression_language#writing-custom-expressions or check the example project https://github.com/OndrejKunc/flutter_dynamic_forms/tree/master/packages/flutter_dynamic_forms_components/example


Test_Format.test_roundtrip_big_integer test fails with Python 3.10.7+, 3.9.14+, 3.8.14+, 3.7.14+

Problem Description
According to the release notes for 3.10.7:
https://docs.python.org/3/whatsnew/changelog.html#python-3-10-7-final
There's now a size limit on integers / strings when converting between them to address CVE CVE-2020-10735.

        This function generates the tnetstring representation of the given value,
        pushing chunks of the output onto the given deque instance.  It pushes
        the last chunk first, then recursively generates more chunks.

        When passed in the current size of the string in the queue, it will return
        the new size of the string in the queue.

        Operating last-chunk-first makes it easy to calculate the size written
        for recursive structures without having to build their representation as
        a string.  This is measurably faster than generating the intermediate
        strings, especially on deeply nested structures.
        """
        write = q.appendleft
        if value is None:
            write(b"0:~")
            return size + 3
        elif value is True:
            write(b"4:true!")
            return size + 7
        elif value is False:
            write(b"5:false!")
            return size + 8
        elif isinstance(value, int):
>           data = str(value).encode()
E           ValueError: Exceeds the limit (4300) for integer string conversion

Steps to reproduce the behavior:
Running tox -e py with any of the newly released Python versions triggers this test failure.
System Information
$ mitmproxy --version
Mitmproxy: 9.0.0.dev (+102, commit a5f75a0)
Python:    3.10.7
OpenSSL:   OpenSSL 3.0.5 5 Jul 2022
Platform:  Linux-5.19.8-200.fc36.x86_64-x86_64-with-glibc2.35

FYI the limit can be changed:
https://docs.python.org/3.11/library/sys.html#sys.set_int_max_str_digits
https://docs.python.org/3.11/library/stdtypes.html#int-max-str-digits


Duplicate reports in onTelemetriesFound()

I discovered that onTelemetriesFound() method is possibly reporting the same value over and over again even though no new telemetry packet is received.
Here is what I tried:
Estimote Location beacon is set to advertise Estimote Telemetry packet with 2570 ms. period.
BeaconManager is configured with 250 ms. foreground scanning period with no wait (setForegroundScanPeriod(250, 0)).
When onTelemetriesFound() method returns at every 250 ms, it repeats the last received telemetry packet since it is impossible for the beacon to transmit a new packet that frequently. If my suspicion is correct and these are indeed duplicates, is there a way for the API to filter these or do I have to implement some filtering scheme myself?
Here is the output of my logcat:
02-08 09:26:51.831 14844-14844/telemetry D/Line 47: MainActivity onTelemetriesFound(): beaconID: [XXXXX] rssi: -54 accelerometer: (-0.02,0.00,1.02)
02-08 09:26:52.084 14844-14844/telemetry D/Line 47: MainActivity onTelemetriesFound(): beaconID: [XXXXX] rssi: -54 accelerometer: (-0.02,0.00,1.02)
02-08 09:26:52.330 14844-14844/telemetry D/Line 47: MainActivity onTelemetriesFound(): beaconID: [XXXXX] rssi: -54 accelerometer: (-0.02,0.00,1.02)
02-08 09:26:52.580 14844-14844/telemetry D/Line 47: MainActivity onTelemetriesFound(): beaconID: [XXXXX] rssi: -54 accelerometer: (-0.02,0.00,1.02)
02-08 09:26:52.830 14844-14844/telemetry D/Line 47: MainActivity onTelemetriesFound(): beaconID: [XXXXX] rssi: -54 accelerometer: (-0.02,0.00,1.02)
02-08 09:26:53.085 14844-14844/telemetry D/Line 47: MainActivity onTelemetriesFound(): beaconID: [XXXXX] rssi: -54 accelerometer: (-0.02,0.00,1.02)
02-08 09:26:53.331 14844-14844/telemetry D/Line 47: MainActivity onTelemetriesFound(): beaconID: [XXXXX] rssi: -54 accelerometer: (-0.02,0.00,1.02)

Helo @arslanmustafa!
Take a look at our freshly released version 1.0.0 - your problem should be fixed :)
Let me know if this works for you now :)

Hi @pawelDylag,
I regret to say that the problem persists. I made sure I updated my SDK version to 1.0.0 in my gradle file. My little experiment is as follows:
I have flip to sleep enabled on my beacon, so when I flip it on its back it stops broadcasting telemetry packets (verified using a secondary receiver laptop using node.js). In my Android phone (running Android 5.1), I simply log the received telemetry packets to the console from within the onTelemetriesFound() method.
public void onTelemetriesFound(List list) {
for (EstimoteTelemetry tlm : list) {
Log.d("Line 71", "MainActivity onTelemetriesFound(): " + tlm.deviceId.toString() + "," + tlm.rssi + "," + tlm.ambientLight + "," + tlm.magnetometer);
}
}
I turn my beacon "on" and wait till I get one packet printed on my log. After that, I immediately turn the beacon upside down, but my log keeps printing that same first packet for as long as 20 seconds (while the node.js receiver stops getting packets).   We will add some fixups to avoid caching packets when using discovery methods. I'll inform you when the fixes are introduced - probably next release :)
Have a wonderful day!

Hey,
This problem should be fixed in sdk version 1.0.1.
Please notice that the data will not be reported duplicated now, but it won't be reported on every scan cycle either. TLM packes are splitted into smaller packets which we need to merge internally. Read more here under Single packet paragraph )
Let me know if it works for you right now - I will close this issue then.
Cheers!


Feature/backend user serializer modification

Changed user serializer to include access request primary keys
Added status field to access request (Active, Accepted, Rejected)

@Orlando-RL I changed opinions. Don't think access requests should be deleted, so I included a status on each of them instead. That way we can track if a request has been completed or not.


haskell-updates: update Hackage, in particular Cabal-<IP_ADDRESS>

Description of changes
produced with
./maintainers/scripts/haskell/update-stackage.sh
./maintainers/scripts/haskell/update-hackage.sh
./maintainers/scripts/haskell/regenerate-hackage-packages.sh

I am trying to get revision 2 of Cabal-<IP_ADDRESS> into the package set: https://hackage.haskell.org/package/Cabal-<IP_ADDRESS>/revisions/
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Looks like we are in good shape and haskell-updates can be merged in the next days. After that we'll  start another regular rotation. We are seemingly all a bit short on time at the moment, so things are going a bit slower than usual.

@sternenseemann , thanks, can we include
       pname = "Cabal";
       version = "<IP_ADDRESS>";
       sha256 = "0236fddzhalsr2gjbjsk92rgh8866fks28r04g8fbmzkqbkcnr3l";
       revision = "2";

https://github.com/NixOS/nixpkgs/blob/645175db2b2ecb4032005b2c24ff023d44cb6320/pkgs/development/haskell-modules/hackage-packages.nix#L2910-L2913 before haskell-updates is merged?
Should I slim down this PR to just this single change?

See #217111/#219027

Thanks


add render node utility

Summary
Motivation is to create a simple utility to reuse the validation to render a node from any component.
Changelog
[General] [Added] - Add render node utility to be used with component props
Test Plan
Added test with react-test-renderer to validate the same output for a simple test component

Please let me know if you are interested in my work and I can resolve the merge conflicts


CLOUDSTACK-8711: public_ip type resource count for an account is not decremented upon IP range deletion

Problem:
When you add an IP range and associate it to an account then resource count of public_ip will be updated to the range length.
After some time try to delete this range and the resource count of public_up for this account is not descremented and is causing account not to add any more public IPs to it once it reaches the resource limit.
RCA:
We were not decrement the count while deleting the IP range that was associated to an account.
Fix:
Up on deletion we are decrementing the resource count for public_up now.

Build failure is not related to this change. Code LGTM.

LGTM


Tick labels & locations

Currently there is no way to specify tick labels and locations.  Especially in subplots and date plots, this causes issues. with overlapping tick labels and overly-dense grids.  This should be addressed soon.

I have used the mpld3.plugins method as mentioned above.I was able to change the tick labels  by using
       ax.axis.tickFormat(function(d, i){             return SN_[i] //"SN#1 SN#2, etc depending on the tick value - 0,1,2,3,4"         });
And then calling it via
plugins.connect(fig,HelloWorld(SN_unique))
where SN_unique provides a Serial Number list for the x indices.
I cannot get to get my head around as to how to rotate the tick labels using d3 axes objects?


Nav item refactoring

@AlexeySachkov 's refactoring of Nav items. I would like to tweak this just a bit first.

This is too far out of date now. A lot has changed.


Added support to new runtime permissions model of Android M

What are your thoughts on somehow re-using the runtime permissions code in card.io-Android-Source native implementation ?

I am getting issues on android 6 where it doesn't open the camera. Is this the behaviour you were seeing?

Hi @jaypatel512, it's interesting Card.io already supported in some way the new Android permission model, I've just tested and it is working when you actually call the scan method. The problem I was seeing is that they didn't implement request for permissions for canReadCardWithCamera method (used by canScan method of the Cordova plugin).
It it's implemented in Card.io then there is no need to add it to the plugin, I'm going to report the issue to the other repo.

Yeah I've just noticed!
I'm going to close this pull request then and wait for the issue to be resolved in Card.io.
Thanks


Assignment: c1:i:-1

This bug has been observed in files from another issue.
Workflow:
prophyle download bacteria viruses plasmids
prophyle index -k 31 ~/prophyle/bacteria.nw ~/prophyle/viruses.nw ~/prophyle/plasmids.nw idx_bac_vir_plas
prophyle classify -LAP idx_bac_vir_plas/ ../workdir/NBC01_trimmed.fq > result.sam
prophyle analyze -f sam idx_bac_vir_plas/ NBC01_Prophyle_BVP NBC01_Prophyle_BVP.sam

Files: https://github.com/karel-brinda/prophyle/issues/248#issuecomment-327819938
Corresponding FASTQ file: original_reads.zip

After re-reading our source codes, I realized that it's an expected behavior rather than a bug. After distorting the computed assignments using LCA, we loose some information and -1 serves as N/A. Nevertheless, it would be better not to show N/A tags.


Add consumption-time value of __cplusplus to User-Agent

Closes #5661.
This PR is not ready. We will need to make sure that the preprocessor part is not being generated by the SDK clients in .cpp files.

@LarryOsterman, I just thought, it won't be as easy, but it is still doable.
We will need to review where the SDK clients are instantiating the policies. If they do it in .cpps, we will be getting C++14 we need to somehow move that to header files, and ideally it should be lightweight. I am thinking, if we put it into ClientOptions.Telemetry, that might work, because the options are instantiated by customer code.

@LarryOsterman, what do you think of this idea?
I also updated EventHubs.

To find places where we might instantiate options in the SDK .cpp files, I'll temporarily add something that explodes in compile-time when you try to instantiate these options, and compile our SDK without tests and samples.

Why do we need the different constants/macros for SDK source, test, and samples? Don't we only care about the SDK source for this telemetry?

@ahsonkhan, we will be able to answer the questions about the amount of telemetry coming from SDK, tests, samples, and customer code. As far as differentiation between customer's environment where they are free to set any C++ version they like, and building our SDK code, where it is us who set C++ standard to C++14, there is a criterion for the version reported to be >0. Our values reported truly are 2014 anyways, we know that, and we don't want to add that to the C++14 weight in our telemetry to not distort the picture.

I might be misunderstanding the goal here. I thought the primary question we are trying to answer here was, what is the C++ standard version our customers are using. That's the only one that's relevant.
I understand you want to differentiate "our/SDK" code versus customer code, to get an accurate picture of what customers use, which makes sense. By creating that distinction, we can focus on the C++ standard version that isn't coming from the SDK code.
With that, I see no reason or benefit to create sub-categories in the SDK code. I don't see why we'd ever care that the C++ standard version we are seeing from our code is SDK source or sample or tests, all of which should be the same anyway (C++ 14)
Therefore, I recommend we have one value for SDK code (say < 0), and one for customer code, whatever they pick. And simplify our solution immensely. We don't need the distinct _azure_BUILDING_SAMPLES, _azure_BUILDING_SDK, _azure_BUILDING_TESTS. They seem nice, in theory, but I see no use for it, in what we are trying to achieve here. Am I missing something?

/check-enforcer evaluate


Experiment with Parallelized Parsing

Before TypeScript can respond to any semantic query (accurately), the entire program has to be loaded into memory, parsed, and bound. Parsing today ends up taking a surprising amount of time, and even if we don't have to do any path resolution. One of the things we could exploit is that parsing any two files should (ideally) be independent operations - and so one could imagine spinning up multiple processes or workers to divide and conquer the problem.
Unfortunately, program construction is not an embarrassingly parallelizable step because we don't always know the full list of files in a program ahead of time; however, one could imagine some work-stealing scheme with single orchestrator.
A glaring problem for TypeScript is that it is currently synchronous from end-to-end, and most worker capabilities are built with an expectation of asynchronous communication (and often depend on the surrounding environment - Node.js main thread, UI thread in the browser, non-UI worker threads, etc.). Another problem is that while we might be able to divide and conquer, the overhead of moving data between workers might be more than we'd have anticipated. And one last concern I'll mention is that while running multiple workers gets more work done faster, it's not a free lunch - it has UX tradeoffs because multiple threads can degrade the responsiveness of the overall machine.
So in the coming months, we'll be investigating here, finding out what works, and seeing if we can bring it into TypeScript itself.

You would need an intermediate representation for the parsed AST, since the Nodes as they currently exist with methods and stuff, are not transferable. So that would basically implement #26871 which also enables caching the parse result.

In the Node.js enviroment there is a hack to do async computing with a sync API, I first read it here. It's a combination of worker_thread, SharedArrayBuffer and Atomics.wait

In the Node.js enviroment there is a hack to do async computing with a sync API, I first read it here. It's a combination of worker_thread, SharedArrayBuffer and Atomics.wait

Right; this is the same thing that's done for a synchronous filesystem in TypeScript's vscode.dev integration. It is one tool in my hypothetical toolbox when I look into this.

It's unfortunate that TypeScript can't leverage runtimes that are built for minimal overhead between threads/workers. I'll just leave that comment at that :)

Er, why leave it at that? What runtimes?

Just wondering if these tools are to solve the same performance problem:

https://github.com/kaleidawave/ezno
https://github.com/dudykr/stc

We're currently using drizzle ORM which relies a lot of union and infer, the VSCode typescript intellisense is super bad where it crashes from time-to-time which someone already reported here.
A genuine question: I see that the TS team introducing a handful of new features over the recent versions, has performance impact been part of the consideration when releasing a new feature? Reason I'm asking is because when a new feature comes out, the OSS projects tend to start using it and if the feature doesn't perform well, we have a huge ecosystem slowing down our projects as we upgrade towards newer version.
Thanks.

Just wondering if these tools are to solve the same performance problem:

https://github.com/kaleidawave/ezno
https://github.com/dudykr/stc

We're currently using drizzle ORM which relies on a lot of union and infer, the VSCode typescript intellisense is super bad where it crashes from time-to-time which someone already reported here. Prisma ORM faces the same issue as we can't even complete tsc -b with the types it generates, it just crashes.
A genuine question: I see that the TS team introducing a handful of new features over the recent versions, has performance impact been part of the consideration when releasing a new feature? Reason I'm asking is because when a new feature comes out, the OSS projects tend to start using it and if the feature doesn't perform well, we have a huge ecosystem slowing down our projects as we upgrade towards newer version
Thanks.

Performance impact is absolutely a thing used to determine feasibility of features, and indeed many features have been rejected due to performance cost.

Referencing previous talk https://github.com/microsoft/TypeScript/issues/30235


FR: Query threads of notes

In essence, a "thread" is a sequence of thought that should be treated as a "body" of knowledge. For example, when we search for a keyword in a forum, we get search results to an entire thread, pin-pointing the exact location (sequence) of the post within that forum thread. Likewise, if we retweet a twitter post, we can also retrieve the entire forum thread.
In a conversation with Robert Halsfield, he made a really good point that these threads of thought should be query-able as an entire body. It's easy enough to pull up the search and find the particular post, however, displaying it within a query on a synthesis page, for example, is a different issue. Wondering if it's possible to query for a keyword which then displays a result for the thread containing that note? Ideally it should be able to show the entire thread while pinpointing the exact location of the note.

Yup. Idk I generally look at "page atomicity" as "contains all of the words I might want to search together." If Breadcrumbs is just a navigational tool, that doesn't change too much for my concept of page atomicity. However, if I can graph query threads and trees, that would unlock a ton in Breadcrumbs for me.

This thread in the forum goes into detail about how the Roam and Obsidian paradigms are different, and gets at how blocks and indentation work to chain together units of information in Roam. My impression is that a lot of people use Breadcrumbs as an alternative to that sort of indentation.

I don't know if we are asking the same. I would like to make queries within a Breadcrumbs notes branch. Is this possible right now?
And, perhaps, add other search criteria such as go up, siblings...

Could you give a specific example? I kind of get the "theory" of the request, but can't see how it would work in practice


Pod库 swift4.2报错

BLTNInterfaceBuilder类中titleLabel.accessibilityTraits |= UIAccessibilityTraits.header报错Binary operator '|=' cannot be applied to two 'UIAccessibilityTraits' operands

Hi! Could you share this information so I can help:

What dependency manager you use (CocoaPods / Carthage / manual)
What version of the framework you use (2.0.2 is the only one that can be used with Xcode 10)

Thanks!

I have error compiling on Swift 4.2 @alexaubry Please advise

Hi @alexaubry - first off great work - love the framework.
I can confirm what @yarodevuci is seeing.
I'm using cocoapods on 2.0.2 and this is the only framework left of all that I use that seems to have this deprecation warning:

I've tried running this through the conversion tooling in Xcode but end up with some very strange errors - will try to take a look and post more detail here but suspect if you try to convert to Swift 4.2 syntax you will see the same.
Thanks in advance for your attention!
Andrew

This has been fixed in the latest release. You can all update the library to 3.0 in your Podfile for full support of Swift 4.2 😄

Awesome - Thanks!

From: Alexis Aubry<EMAIL_ADDRESS>Sent: Monday, December 17, 2018 8:37:13 AM
To: alexaubry/BulletinBoard
Cc: Andrew Lombard; Comment
Subject: Re: [alexaubry/BulletinBoard] Pod库 swift4.2报错 (#123)
Closed #123https://github.com/alexaubry/BulletinBoard/issues/123.
―
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/alexaubry/BulletinBoard/issues/123#event-2030262206, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AJxUH-1eC5GRihO6Y1LWLzMjjCvwR3Epks5u554JgaJpZM4XUeqL.


Adding go binding.

Adding bindings for golang.
Checklist

[x] All tests pass in CI
[x] There are enough tests for the new fix/feature
[x] Grammar rules have not been renamed unless absolutely
necessary (x rules renamed)
[x] The conflicts section hasn't grown too much (x new conflicts)
[x] The parser size hasn't grown too much (master: STATE_COUNT,
PR: STATE_COUNT) (check the value of STATE_COUNT in
src/parser.c)
[x] npm run generate ensure you include everything in the src/*

Hi @pnx thank you, your contribution. Very much appreciated!


Repeater row content XSS

Include script tag in data passed to rows of the repeater. The script will execute.
<script>alert('attack!');<script>

will result in an alert message
i would provide a codepen but it has problems with the script tag

See comments in #1974


[BUG] Live output blank and cannot be closed

Please confirm these before moving forward

[X] I have searched for my issue and have not found a work-in-progress/duplicate/resolved issue.
[X] I have tested that this issue has not been fixed in the latest (beta or stable) release.
[X] I have checked the FAQ section for solutions.
[X] This issue is about a bug (if it is not, please use the correct template).

UniGetUI Version
3.1.1
Windows version, edition, and architecture
Windows 11 Pro 10.0.22631.4037 x64
Describe your issue
When performing some software updates and doubling clicking on the specific update being installed the Live Output opens blank and I cannot close it. The Close button does nothing. The only to regain control is to close UniGetUI.

Steps to reproduce the issue
I can't reproduce it every time but have attached logs when it did. The update I double clicked on was Wireshark.
UniGetUI Log
[8/31/2024 1:01:42 PM]    __  __      _ ______     __  __  ______
                         / / / /___  (_) ____/__  / /_/ / / /  _/
                        / / / / __ \/ / / __/ _ \/ __/ / / // /
                       / /_/ / / / / / /_/ /  __/ /_/ /_/ // /
                       \____/_/ /_/_/\____/\___/\__/\____/___/
                           Welcome to UniGetUI Version 3.1.1
[8/31/2024 1:01:42 PM]
[8/31/2024 1:01:42 PM] Version Code:  3.11
[8/31/2024 1:01:42 PM] Encoding Code Page set to 437
[8/31/2024 1:01:42 PM] Using bundled GSudo at C:\Program Files\WingetUI\Assets\Utilities\gsudo.exe
[8/31/2024 1:01:43 PM] Loaded language locale: en
[8/31/2024 1:01:43 PM] Randomly-generated background API auth token for the current session: tfmfjty674q1h5yp0o5z93yj70gpu9kjor0wi4ztufbslgk90t0557p0nev7trvf
[8/31/2024 1:01:44 PM] Api running on http://localhost:7058
[8/31/2024 1:01:44 PM] Lang files were updated successfully from GitHub
[8/31/2024 1:01:44 PM] UniGetUI is up to date
[8/31/2024 1:01:45 PM] Command npm was not found on the system
[8/31/2024 1:01:45 PM] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                       █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                       █ Name: Npm
                       █ Enabled: True
                       █ Found: False
                       █ THE MANAGER WAS NOT FOUND. PERHAPS IT IS NOT
                       █ INSTALLED OR IT HAS BEEN MISCONFIGURED
                       ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[8/31/2024 1:01:45 PM] Downloaded new icons and screenshots successfully!
[8/31/2024 1:01:47 PM] Command pwsh.exe was not found on the system
[8/31/2024 1:01:47 PM] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                       █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                       █ Name: PowerShell7
                       █ Enabled: True
                       █ Found: False
                       █ THE MANAGER WAS NOT FOUND. PERHAPS IT IS NOT
                       █ INSTALLED OR IT HAS BEEN MISCONFIGURED
                       ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[8/31/2024 1:01:47 PM] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                       █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                       █ Name: .NET Tool
                       █ Enabled: True
                       █ Found: False
                       █ THE MANAGER WAS NOT FOUND. PERHAPS IT IS NOT
                       █ INSTALLED OR IT HAS BEEN MISCONFIGURED
                       ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[8/31/2024 1:01:47 PM] Command scoop was not found on the system
[8/31/2024 1:01:47 PM] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                       █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                       █ Name: Scoop
                       █ Enabled: True
                       █ Found: False
                       █ THE MANAGER WAS NOT FOUND. PERHAPS IT IS NOT
                       █ INSTALLED OR IT HAS BEEN MISCONFIGURED
                       ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[8/31/2024 1:01:48 PM] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                       █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                       █ Name: Pip
                       █ Enabled: True
                       █ Found: True
                       █ Fancy exe name: pip
                       █ Executable path: C:\Users\chris\AppData\Local\Microsoft\WindowsApps\python.exe
                       █ Call arguments:  -m pip
                       █ Version:
                       █
                       ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[8/31/2024 1:01:50 PM] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                       █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                       █ Name: Winget
                       █ Enabled: True
                       █ Found: True
                       █ Fancy exe name: winget.exe
                       █ Executable path: C:\Users\chris\AppData\Local\Microsoft\WindowsApps\winget.exe
                       █ Call arguments:
                       █ Version:
                       █   System WinGet CLI Version: v1.8.1911
                       █   Microsoft.WinGet.Client PSModule version: ""
                       █   Using Native WinGet helper (COM Api)
                       ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[8/31/2024 1:01:50 PM] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                       █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                       █ Name: Chocolatey
                       █ Enabled: True
                       █ Found: True
                       █ Fancy exe name: choco.exe
                       █ Executable path: C:\Users\chris\AppData\Local\UniGetUI\Chocolatey\choco.exe
                       █ Call arguments:
                       █ Version:
                       █   2.2.2
                       ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[8/31/2024 1:01:53 PM] ▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄▄
                       █▀▀▀▀▀▀▀▀▀▀▀▀▀ MANAGER LOADED ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
                       █ Name: PowerShell
                       █ Enabled: True
                       █ Found: True
                       █ Fancy exe name: powershell.exe
                       █ Executable path: C:\WINDOWS\system32\windowspowershell\v1.0\powershell.exe
                       █ Call arguments:  -NoProfile -Command
                       █ Version:
                       █   Name                           Value
                       █   ----                           -----
                       █   PSVersion                      5.1.22621.3958
                       █   PSEdition                      Desktop
                       █   PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
                       █   BuildVersion                   10.0.22621.3958
                       █   CLRVersion                     4.0.30319.42000
                       █   WSManStackVersion              3.0
                       █   PSRemotingProtocolVersion      2.3
                       █   SerializationVersion           <IP_ADDRESS>
                       ▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀▀
[8/31/2024 1:01:55 PM] Resetting administrator rights cache for process id 36124
[8/31/2024 1:01:56 PM] Microsoft.Powershell.Client detection has been forcefully skipped as the module is not required on the current context
[8/31/2024 1:01:56 PM] Dependency WinGet PowerShell Module for manager Winget is present
[8/31/2024 1:01:56 PM] LoadComponentsAsync finished executing. All managers loaded. Proceeding to interface.
[8/31/2024 1:01:57 PM] Found 0 available updates from Pip
[8/31/2024 1:01:57 PM] Found 0 installed packages from Pip
[8/31/2024 1:02:00 PM] Found 0 installed packages from Chocolatey
[8/31/2024 1:02:00 PM] Found 0 available updates from Chocolatey
[8/31/2024 1:02:04 PM] Found 1 installed packages from PowerShell
[8/31/2024 1:02:05 PM] Found 0 available updates from PowerShell
[8/31/2024 1:02:06 PM] Found 7 available updates from Winget
[8/31/2024 1:02:07 PM] Found 363 installed packages from Winget
[8/31/2024 1:10:49 PM] Non-MSStore WinGet Native Icons have been forcefully disabled on code
[8/31/2024 1:10:49 PM] Loaded icon with URL=UniGetUI.Core.IconEngine.CacheableIcon for package Id=WiresharkFoundation.Wireshark
[8/31/2024 1:10:49 PM] Icon for package WiresharkFoundation.Wireshark stored on C:\Users\chris\AppData\Local\UniGetUI\CachedMedia\Winget\WiresharkFoundation.Wireshark.png
[8/31/2024 1:10:49 PM] Non-MSStore WinGet Native Icons have been forcefully disabled on code
[8/31/2024 1:10:49 PM] Loaded icon with URL=UniGetUI.Core.IconEngine.CacheableIcon for package Id=Zoom.Zoom
[8/31/2024 1:10:49 PM] Icon for package Zoom.Zoom stored on C:\Users\chris\AppData\Local\UniGetUI\CachedMedia\Winget\Zoom.Zoom.png
[8/31/2024 1:13:37 PM] Resetting administrator rights cache for process id 36124

Package Managers Logs
Manager WinGet with version:
System WinGet CLI Version: v1.8.1911
Microsoft.WinGet.Client PSModule version: ""
Using Native WinGet helper (COM Api)

——————————————————————————————————————————

Logged native task on manager Winget. Task type is ListSources
Process start time: 8/31/2024 1:01:50 PM
Process end time:   8/31/2024 1:01:50 PM

-- Task information
 ...

The task reported success

——————————————————————————————————————————

Logged subprocess-based task on manager Winget. Task type is RefreshIndexes
Subprocess executable: "C:\Users\chris\AppData\Local\Microsoft\WindowsApps\winget.exe"
Command-line arguments: " source update --disable-interactivity"
Process start time: 8/31/2024 1:01:53 PM
Process end time:   8/31/2024 1:01:57 PM

-- Process STDOUT
 ...

Return code: SUCCESS (0)

——————————————————————————————————————————

Logged native task on manager Winget. Task type is ListInstalledPackages
Process start time: 8/31/2024 1:01:54 PM
Process end time:   8/31/2024 1:02:07 PM

-- Task information
 ...

The task reported success

——————————————————————————————————————————

Logged native task on manager Winget. Task type is ListSources
Process start time: 8/31/2024 1:01:56 PM
Process end time:   8/31/2024 1:01:57 PM

-- Task information
 ...

The task reported success

——————————————————————————————————————————

Logged native task on manager Winget. Task type is ListUpdates
Process start time: 8/31/2024 1:01:57 PM
Process end time:   8/31/2024 1:02:06 PM

-- Task information
 ...

The task reported success

——————————————————————————————————————————

Relevant information
No response
Screenshots and videos
No response

Pressing Esc doesn't close it either?


Failed to execute unzip

Hello,
Anyone has an idea what this means?
  - Updating wsdltophp/packagegenerator (3.0.3 => 3.0.4): Loading from cache    Failed to execute unzip -qq  '/Users/rf/workspace/pmp/vendor/wsdltophp/packagegenerator/fe74a9f75cc1acfe86690cb4b72ec034' -d '/Users/rf/workspace/pmp/vendor/composer/08c2d881'

error:  cannot create /vendor/composer/08c2d881/WsdlToPhp-PackageGenerator-ab2db90/tests/resources/generated/Valid?????????????.php
        Illegal byte sequence

    The archive may contain identical file names with different capitalization (which fails on case insensitive filesystems)
    Unzip with unzip command failed, falling back to ZipArchive class

Same issue for me. Removing vendor doesn't work. I'm unable to install this package via composer. I don't think using cyrillic characters in the filename is a good idea.

Are you on Windows ? Mac ? Or Unix ?

macOs: 10.14.6, composer: 1.9.0, php: 7.3.7

@Gemorroj, any though on this matter ? If I recall right, you're using cyrillic characters :)

The test files aren't required in the composer archive, are they? They could still exist in this repository to run the test suite, but be excluded for composer consumers. Just an idea.

#50
Yes, the use of Cyrillic in names is considered very bad form.
Perhaps this bug should be addressed to https://github.com/composer/composer

#50
Yes, the use of Cyrillic in names is considered very bad form.
Perhaps this bug should be addressed to https://github.com/composer/composer

Ok so I should rename the files containing Cyrillic to English names :)

Perhaps this bug should be addressed to https://github.com/composer/composer

It's seems to be an issue in unzip:

https://github.com/adamhathcock/sharpcompress/issues/315
https://unix.stackexchange.com/questions/251969/how-can-i-correctly-decompress-a-zip-archive-of-files-with-hebrew-names

fixed in 3.2.3 and 2.11.3 :wink:


Please help me

ubuntu@vibrio:~$ poppunk -h
/home/ubuntu/.linuxbrew/opt/python/lib/python3.7/site-packages/sklearn/externals/joblib/externals/cloudpickle/cloudpickle.py:47: DeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses
import imp
usage: PopPUNK [-h]
(--easy-run | --create-db | --fit-model | --refine-model | --assign-query)
[--ref-db REF_DB] [--r-files R_FILES] [--q-files Q_FILES]
[--distances DISTANCES]
[--external-clustering EXTERNAL_CLUSTERING] --output OUTPUT
[--plot-fit PLOT_FIT] [--full-db] [--update-db] [--overwrite]
[--min-k MIN_K] [--max-k MAX_K] [--k-step K_STEP]
[--sketch-size SKETCH_SIZE] [--K K] [--dbscan] [--D D]
[--min-cluster-prop MIN_CLUSTER_PROP] [--pos-shift POS_SHIFT]
[--neg-shift NEG_SHIFT] [--manual-start MANUAL_START]
[--indiv-refine] [--no-local] [--model-dir MODEL_DIR]
[--previous-clustering PREVIOUS_CLUSTERING] [--core-only]
[--accessory-only] [--microreact] [--cytoscape] [--phandango]
[--grapetree] [--rapidnj RAPIDNJ] [--perplexity PERPLEXITY]
[--info-csv INFO_CSV] [--mash MASH] [--threads THREADS]
[--no-stream] [--version]
How i can install a old library for poppunk using pip.
thank you

I'm not sure what problem you are having (looks like it's running ok) -- could you describe in more detail?
The message is a warning that a feature in scikit-learn will be removed in future, and won't affect the run or results
It is possible to use older versions, though we wouldn't recommend it.


Wrong created_at values in commits.csv

Looks to me that the most recent dump (2019-06-01) has wrong time stamp. Below is how we may reproduce the error.
In http://ghtorrent-downloads.ewi.tudelft.nl/mysql/mysql-2019-06-01.tar.gz, I found the following,
$ grep "2038-01-18" commits.csv
239581704,"1e954ef83c79ccccb8f570d350ea0ff934650fbb",4498938,4498938,18281400,"2038-01-18 23:00:00"
242299182,"0ebc3ea059dfcc8124135b94dfaa40ba7d49e5b2",1340592,1340592,18533076,"2038-01-18 23:00:00"
606676461,"d4cbfbc31940a514f2974405b579ff8989f9a0b6",33691743,33691743,55773517,"2038-01-18 00:00:00"

I loaded the CSV files to a PostgreSQL database, and then do a query. These projects are,
ghtorrent=# SELECT id,url FROM projects WHERE id=18281400 OR id=18533076 OR id=55773517;
    id    |                              url
----------+---------------------------------------------------------------
 18281400 | https://api.github.com/repos/cirosantilli/test-streak-puppet2
 18533076 | https://api.github.com/repos/phy1729/illacceptanything
 55773517 | https://api.github.com/repos/nhm150030/jfox
(3 rows)

Having compared the dates in the CSV file with the commit log in these projects. The dates are indeed wrong. Similarly, I also found these,
$ grep "2028-12-31 23:00:00" commits.csv
239657882,"182ef308602eabf227265eb2a5034ede567dccfa",4498938,4498938,18281400,"2028-12-31 23:00:00"
242561682,"cd952389edd54aa00cd3a98382da1193a1fc6d36",1340592,1340592,18507496,"2028-12-31 23:00:00"

In fact, if we group by the duration of commits of projects where the duration is the years between the oldest and the newest commits of a project and count the number of projects in a group, I got the following,
 duration | count
----------+-------
        0 |   492
        1 |   632
        2 |   414
        3 |   363
        4 |   325
        5 |   321
        6 |   277
        7 |   214
        8 |   202
        9 |   150
       10 |   102
       11 |   106
       12 |    89
       13 |    82
       14 |    93
       15 |    60
       16 |    63
       17 |    51
       18 |    49
       19 |    38
       20 |    33
       21 |    24
       22 |     8
       23 |    11
       24 |    14
       25 |    13
       26 |     8
       27 |     4
       28 |     1
       29 |     2
       32 |     1
       33 |     1
       34 |     3
       36 |     1
       38 |     2
       40 |     3
       42 |     1
       43 |     1
       44 |     9
       45 |     7
       46 |    15
       47 |    10
       48 |     2
       49 |     7
       58 |     1
       68 |     2
(46 rows)

where count is the number of projects whose duration is given on the left column.

From the FAQ page:

I've seen weird commit timestamps
Git records the commit timestamp on the developer's workstation. If the clock is misconfigured, timestamps will be weird.
We have seen timestamps such as 0000-01-01 00:00 or 2034-12-31 23:59. GitHub and GHTorrent do not process the timestamps in any way.

One of the projects you referenced has future dates set against the authored at timestamps e.g. 2046-09-18T00:00:00Z
This problem affects all data on GH itself - here's a test commit I just made in 2021.
Unfortunately, I don't think there's anything we can do about it. At least instances of this should be pretty rare.


Some metrics are missing from https://paperswithcode.com/sota/machine-translation-on-wmt2014-english-german

Similar to https://github.com/paperswithcode/sota-extractor/issues/25, check https://paperswithcode.com/sota/machine-translation-on-wmt2014-english-german, but found:

They show up correctly now, closed.


Weird behaviour when calling toggle() when tray set to not open on start

When I start a debugging session, the sidebar opens as expected. In my case I have set the tray to not open on start, so it does not appear with the sidebar:
dapui.setup({
  ...
  sidebar = {
    open_on_start = true,
    ...
  },
  tray = {
    open_on_start = false,
    ...
  },
  ...
})

When I call require("dapui").toggle() to hide the sidebar, the sidebar goes but the tray opens up.
This is technically correct, since the tray visibility has been toggled. But ideally, I would like to disable the tray becoming visible on toggle, since open_on_start is false.
Is this something that can be addressed in the plugin, or should users come up with a workaround for cases like these in personal configs?

This is expected behaviour and I don't think I want to change it. If you just want to toggle the sidebar you can pass "sidebar" to the toggle. Is that what you're looking for?

Great point - I didn't realise I could pass the sidebar as an argument here. Thanks!


Remove dependabot configuration

Should have been part of
https://github.com/Ableton/ansible-role-jenkins-jcasc/pull/222

@nre-ableton what is going on with the diff in the commit message and the PR description?

@nre-ableton what is going on with the diff in the commit message and the PR description?

Dammit, fixed. This happens sometimes in vim when you re-indent and the leading # characters get shifted around.

@ablbot rebase


Making overload_t's copy constructor have priority

So currently the copy constructor for the overload_t does not take
precendent over the other templated constructor.  This means a simple
function like the following fails:
void foo()
{
	auto a = hana::overload(f1,f2);
	auto b(a); //Error
}

To fix this I added a enable_if to turn off the constructor when not
applicable.........
Note: This (allegedly) fixes issue 412

I am not quite sure why the automated builds failed......(I don't have much experience with appveyor or travis-ci).....when I did a local make check nothing failed........does someone have a clue as to what is going on here?

It looks like it is detecting trailing white-space. I remember the error message for that being a little better.
The Appveyor build failure looks like a config issue with MSVC.


Interactive shell with charmrun in ssh mode does not work

Launching an interactive shell with charm.start(interactive=True) does not work when using charmrun in ssh mode. Fixing this requires changes to charmrun (in Charm++).

Closed by #94 and https://github.com/UIUC-PPL/charm/commit/cca7e4a99c30b498351c1ae3704a382f7969731d


The predicted scores are all zero.Have you seen this problem？

I used my own glove word2vec embedding data for my own dataset and vocabulary_glo.txt you provided.  I trained 1 epoch with 10000 iterations. The loss is just about 5. But when I use gzsd.py to predict and I found all the scores are zero.

From my experience, I faced such problems when the predicted confident scores are not high enough to pass NMS. You can find a score_threshold during NMS process. Moreover, I suspect your training needs to perform more epochs. In my experiments, loss usually moves down below 1.

From my experience, I faced such problems when the predicted confident scores are not high enough to pass NMS. You can find a score_threshold during NMS process. Moreover, I suspect your training needs to perform more epochs. In my experiments, loss usually moves down below 1.

Thanks for your reply. Now I am trying to train the network with more epochs. I hope I can get a good performance. Your work is really inspiring.


Fix initial state possibly being modified in simulation

Fixes #2240

Automerge cancelled: A status check is failing.


Autogen table stuff?

Currently debating whether to add in options to automatically generate stuff like loot tables, blockstates, and/or models for tables using Artifice. It'd be opt-in for devs who just don't want to have to deal with creating everything from scratch. Hell, it could possibly even have a template for its own textures using tint indices. Not entirely sure it's worth it, but I'll see what @AK9Official thinks.

There's now an option for generating assets! It's false by default, just in case pack devs wanna use something else.


feature request: add a context menu element for Copy Selection as Markdown Quote along with the existing Copy Selection as Markdown

Context
As far as my (pretty heavy) usage is concerned, the option Prepend quote (>) to selection is the one I switch on and off the most often — with the both config states (on and off) being very useful to me.
Actual behavior
Every time I need to switch the quote feature on and off, I need to

Right click on the extension's icon
Choose Manage extension
Choose Preferences
Check or uncheck the option
Click Save
Go back to the tab where my text is.

It's an excruciating chore.
Expected behavior
I'd like to have both options — Copy Selection as Markdown Quote along with the existing Copy Selection as Markdown — as two separate context menu commands.
Thank you

Please 🙏


Resize ops input is limited to 4D

At this moment all resize ops suppose they have 4D input, but:

That's wrong. 3D inputs should be acceptable, just make internal reshape to [1, x, y, z]
There's not input rank validation. That's clearly a bug.

cc @shugeo

Implemented and merged. Thanks to @shugeo


Dry latest closing just after launch

Hi,
When I launch the latest version of dry, it close immediately after. I tried to install it manually and with the dry-up.sh script, without success. But if I install v0.9-beta.3 it works normally.
With v0.9-beta.5:
$ dry
INFO[0000] Launching dry
INFO[0000] Bye

My configuration (Xubuntu Bionic):
$ docker -v
Docker version 17.12.1-ce, build 7390fc6

$ uname -a
Linux XXXXX 4.15.0-30-generic #32-Ubuntu SMP Thu Jul 26 17:42:43 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Thanks for this tool !

I will close the issue then ;)

What about docker version 1.12.6? I checked this chart and tried DOCKER_API_VERSION='1.24' dry but still
❯ docker -v
Docker version 1.12.6, build 78d1802

~
❯ DOCKER_API_VERSION='1.24' dry
INFO[0000] Launching dry
INFO[0000] Bye

@moncho


Fixed crash when calling setqflist()

This PR fixes a crash (read at address 0) in vim-8.0.1406
and older reproducible with:
$ vim -u NONE -c 'call setqflist([], " ", {"nr" : $XXX_DOES_NOT_EXIST_XXX})' -cq
Vim: Caught deadly signal SEGV
Vim: Finished.
Segmentation fault (core dumped)

Bug was found using afl-fuzz.
I'm not sure whether the fix is a workaround or a proper fix.
Bug happens when accessing an environment variable which
does not exist.  I wonder whether there could be other similar
bugs lurking.

Hi Dominique,
On Mon, Dec 18, 2017 at 1:42 PM, Dominique Pellé
<EMAIL_ADDRESS>wrote:

This PR fixes a crash (read at address 0) in vim-8.0.1406
and older reproducible with:
$ vim -u NONE -c 'call setqflist([], " ", {"nr" : $XXX_DOES_NOT_EXIST_XXX})'
-cq
Vim: Caught deadly signal SEGV
Vim: Finished.
Segmentation fault (core dumped)
Bug was found using afl-fuzz.
I'm not sure whether the fix is a workaround or a proper fix.
Bug happens when accessing an environment variable which
does not exist. I wonder whether there could be other similar
bugs lurking.

A similar check is needed in the qf_get_properties() function. Otherwise the
following commands will crash Vim:
cexpr ""
echo getqflist({'nr' : $XXX_DOES_NOT_EXIST_XXX})

Regards,
Yegappan

You can view, comment on, or merge this pull request online at:
https://github.com/vim/vim/pull/2464
Commit Summary
Fixed crash in setqflist()
File Changes
M src/quickfix.c (2)
M src/testdir/test_quickfix.vim (5)
Patch Links:
https://github.com/vim/vim/pull/2464.patch
https://github.com/vim/vim/pull/2464.diff

@Yegappan wrote:

A similar check is needed in the qf_get_properties() function.
Otherwise the following commands will crash Vim

Thanks for finding this other crash. I've just updated with
a fix and a test.

Codecov Report

Merging #2464 into master will decrease coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2464      +/-   ##
==========================================
- Coverage   74.56%   74.55%   -0.02%
==========================================
  Files          92       92
  Lines      132831   132832       +1
  Branches    29135    29135
==========================================
- Hits        99050    99029      -21
+ Misses      33777    33772       -5
- Partials        4       31      +27

Impacted Files
Coverage Δ

src/quickfix.c
92.86% <100%> (ø)
:arrow_up:

src/libvterm/src/screen.c
46.65% <0%> (-6.7%)
:arrow_down:

src/if_xcmdsrv.c
83.99% <0%> (-0.36%)
:arrow_down:

src/gui.c
47.49% <0%> (-0.06%)
:arrow_down:

src/window.c
81.77% <0%> (+0.03%)
:arrow_up:

src/channel.c
82.76% <0%> (+0.04%)
:arrow_up:

src/gui_gtk_x11.c
47.8% <0%> (+0.04%)
:arrow_up:

src/os_unix.c
54.06% <0%> (+0.13%)
:arrow_up:

src/if_py_both.h
76.59% <0%> (+0.14%)
:arrow_up:

src/gui_beval.c
63.09% <0%> (+0.85%)
:arrow_up:

Continue to review full report at Codecov.

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


If infrastructure catalog is off in setting, get a 404 error when selecting orchestration engine

Rancher Version: master 10/4
Steps to Reproduce:

Create a new environment
Make sure infrastructure catalog is disabled
Go to Stack's page
Click on Configure for any orchestration

Results: 404 error

Expected:  Do not show Infrastructure catalog items if it is disabled.

Buttons shouldn't show up if infra is off now.

Version - master 10/10
Buttons not showing if catalog is off.
Verified fixed


Adding the feature of Auto rotation in testimonial section

issue #836
Description
Add auto-rotation functionality to the Testimonial component
This pull request adds an auto-rotation functionality to the Testimonial component in order to automatically rotate through the testimonials without user interaction. The testimonials will advance to the next slide every 2.5 seconds and when the user move cursor to the testimonial it stops auto-rotating so that the user can read slowly.
Changes Made

Modified the Testimonial component to include auto-rotation functionality.
Added a useRef hook to reference the Slider component.
Added a useEffect hook to automatically advance to the next slide every 2.5 seconds.

https://github.com/MonalikaPatnaik/UMatter/assets/103810119/4ffff508-1086-43a5-b8ad-bdd57d528384

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.

Hii, @Prajwal0225, the testimonial slider has been added..
Thankyou!

Can you please add gssoc and level tag so that my contribution get counted under gssoc'23 😅


format

Codecov Report
Attention: 25 lines in your changes are missing coverage. Please review.

Comparison is base (5de6207) 91.98% compared to head (e14f8f6) 0.16%.

Files
Patch %
Lines

src/odeproblemparsetter.jl
0.00%
10 Missing :warning:

src/util_nums.jl
0.00%
8 Missing :warning:

src/util_componentarrays.jl
0.00%
4 Missing :warning:

src/odeproblemparsetterconcrete.jl
0.00%
2 Missing :warning:

ext/MTKHelpersMethodOfLinesExt.jl
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #50       +/-   ##
==========================================
- Coverage   91.98%   0.16%   -91.82%
==========================================
  Files          16      16
  Lines         599     592        -7
==========================================
- Hits          551       1      -550
- Misses         48     591      +543

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


[INLONG-1544] Add replica consumption logic in broker node

Fixes #1544

Codecov Report

Merging #1549 (acd8059) into INLONG-1524 (b53d09e) will decrease coverage by 0.05%.
The diff coverage is 0.81%.

@@                Coverage Diff                @@
##             INLONG-1524    #1549      +/-   ##
=================================================
- Coverage          12.21%   12.16%   -0.06%
  Complexity          1048     1048
=================================================
  Files                392      393       +1
  Lines              32743    32843     +100
  Branches            5157     5172      +15
=================================================
- Hits                4001     3994       -7
- Misses             27977    28082     +105
- Partials             765      767       +2

Impacted Files
Coverage Δ

...ng/tubemq/server/broker/msgstore/MessageStore.java
0.00% <0.00%> (ø)

...mq/server/broker/msgstore/MessageStoreManager.java
0.00% <0.00%> (ø)

...bemq/server/broker/msgstore/disk/MsgFileStore.java
0.00% <0.00%> (ø)

...tubemq/server/broker/nodeinfo/ReplicaNodeInfo.java
0.00% <0.00%> (ø)

...ong/tubemq/server/broker/utils/DataStoreUtils.java
8.00% <0.00%> (-1.10%)
:arrow_down:

.../inlong/tubemq/server/common/TServerConstants.java
0.00% <ø> (ø)

.../tubemq/server/common/utils/WebParameterUtils.java
18.69% <ø> (ø)

...tubemq/server/broker/msgstore/mem/MsgMemStore.java
35.66% <5.88%> (-0.70%)
:arrow_down:

.../java/org/apache/flume/sink/tubemq/TubemqSink.java
51.42% <0.00%> (-4.00%)
:arrow_down:

Continue to review full report at Codecov.

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


[Event Request] codeunit 6620 "Copy Document Mgt." - OnBeforeRecalculateAmount

Can you add event OnBeforeRecalculateAmount in procedure CopySalesDocLine?
        if ExactCostRevMandatory and
           (FromSalesLine.Type = FromSalesLine.Type::Item) and
           (FromSalesLine."Appl.-from Item Entry" <> 0) and
           not MoveNegLines
        then begin
            if RecalculateAmount then begin
                // >>>>>>>>>>>>>>>>>>>>>>>
                IsHandled := false;
                OnBeforeRecalculateAmount(FromSalesLine, ToSalesLine, Currency, IsHandled);
                if not IsHandled then begin
                    // <<<<<<<<<<<<<<<<<<<<<<<
                    ToSalesLine.Validate("Unit Price", FromSalesLine."Unit Price");
                    ToSalesLine.Validate("Line Discount %", FromSalesLine."Line Discount %");
                    ToSalesLine.Validate(
                      "Line Discount Amount",
                      Round(FromSalesLine."Line Discount Amount", Currency."Amount Rounding Precision"));
                    ToSalesLine.Validate(
                      "Inv. Discount Amount",
                      Round(FromSalesLine."Inv. Discount Amount", Currency."Amount Rounding Precision"));
                    // >>>>>>>>>>>>>>>>>>>>>>>
                end;
                // <<<<<<<<<<<<<<<<<<<<<<<
            end;
            ToSalesLine.Validate("Appl.-from Item Entry", FromSalesLine."Appl.-from Item Entry");
            if not CreateToHeader then
                if ToSalesLine."Shipment Date" = 0D then
                    InitShipmentDateInLine(ToSalesHeader, ToSalesLine);

    [IntegrationEvent(false, false)]
    local procedure OnBeforeRecalculateAmount(FromSalesLine: Record "Sales Line"; var ToSalesLine: Record "Sales Line"; Currency: Record Currency; var IsHandled: Boolean)
    begin
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


[v0.3.4] Fix trackmania.constants documentation

trackmania.constants and trackmania.api removed entirely from documentation because they only contain internal classes.


Wrong notation for some notes

Using a sawtooth synth for example, I need to use a2 in order to get an a3.
You can reproduce by setting up the bass line to Muse's Plug in Baby like I did:
<Synth
    type="sawtooth"
    steps={[
        [0, 2, "f#2"],
        [4, 2, "f#3"],
        [7, 1, "f#3"],
        [8, 2, "f#2"],
        [10, 2, "f#2"],
        [12, 2, "c#3"],
        [14, 2, "f#3"],

        [16, 2, "f#2"],
        [20, 2, "f#3"],
        [23, 1, "f#3"],
        [24, 2, "f#2"],
        [26, 2, "f#2"],
        [28, 2, "c#3"],
        [30, 2, "f#3"],

        [32, 2, "g2"],
        [36, 2, "g3"],
        [39, 1, "g3"],
        [40, 2, "g2"],
        [42, 2, "g2"],
        [44, 2, "d3"],
        [46, 2, "g3"],

        [48, 2, "g2"],
        [52, 2, "g3"],
        [55, 1, "g3"],
        [56, 2, "g2"],
        [58, 2, "g2"],
        [60, 2, "d3"],
        [62, 2, "g3"],

        [64, 2, "d2"],
        [68, 2, "d3"],
        [71, 1, "d3"],
        [72, 2, "d2"],
        [74, 2, "d2"],
        [76, 2, "a2"], // should be a3
        [78, 2, "d3"],

        [80, 2, "d2"],
        [84, 2, "d3"],
        [87, 1, "d3"],
        [88, 2, "d2"],
        [90, 2, "d2"],
        [92, 2, "a2"], // same here
        [94, 2, "d3"]
    ]}
/>

Nevermind, I forgot octaves increment on C and not A.


fix:  removed unused variables

Description
Added missing variable usage in the codebase to address the issue of 247 occurrences of declared but unused variables. This pull request aims to enhance the codebase's cleanliness and efficiency by ensuring that all declared variables are utilized appropriately.
PR fixes following issue(s)
Fixes #7442
Type of change
Bug fix (non-breaking change which fixes an issue)
Testing
How Has This Been Tested?

[ ] Manual

@gorlenavinkumar this PR seems to be outdated as compared to release.


Alpine Expression Error: Unexpected template string

The component generates a console error on the wire:key generation :

I'm sorry. In the meantime I switched to TallStackUI components.


x706—As a PO (Minal), I want the cassette printer to print directly from STAN to reduce errors and save money and time

[ ] Printing software links a template set up on computer, that STAN creates a CSV type file to send the information to to print
[ ] Need to know what information is needed on the template:  STAN barcode, Donor ID, tissue type - short code (starting with stage), Spatial location number, replicate
[ ] Computer cassette printer is linked to is 114561

CSV order: Barcode, Donor, Tissue description (including SL), Replicate

Directory: //file01-s0/sprint_files/cassette

this is working really nicely now I have things aligned :-) is there a way to still print the labels as well?

is there a way to still print the labels as well?

Maybe. If not, you might want to add another story to not hold up the current release slate.


how do you document a C++ library?

I'd love to see some default documentation suggestions for a header-only library. We could use documentation.js, but that might be odd since the JSDOC syntax is specific to JS usage, not how to use functions from a C++ lib.
cc @springmeyer @GretaCB

I added a JSDOC comment to the exclaim function as a starter. I can see this working better than expected - but only for standard functional programming. Not sure how it will work with C++ classes.
/**
 * Add an exclamation point to the end of a string
 * @memberof hello_world
 * @name exclaim
 * @param {std::string} message
 * @returns {std::string} message!
 * @example
 * std::string value = exclaim ("hello");
 * cout << value; // => 'hello!'
 */
std::string exclaim (std::string message)
{
    std::string response = message + "!";
    return response;
}

Going to stick with documentation.js since we already have a package.json for publishing.
We can also link to some other options if we want, such as doxygen.

Unfortunately, the {std::string} syntax doesn't play with with docsjs. It just skips over it instead of just placing an unknown type.

Unfortunately, the {std::string} syntax doesn't play with with docsjs. It skips over it instead of just placing an unknown type.

Ugh, okay. Hrmm... starting to feel like this is the wrong approach. Probably best to next understand how http://www.stack.nl/~dimitri/doxygen/ works and probably recommend it.

Sounds good @springmeyer - I'll keep digging and make updates to #6

Tried cldoc to generate html documentation and seeing some positive results! I'm bummed there's no markdown output, and the HTML output is quite verbose, with some minified scripts and XML data sources. That being said - it's nice to be able to write incline documentation.

cc @springmeyer @GretaCB

Closed in #6


Adjust default_params initialization for cpp compatibility to avoid - …

…Designator member outside aggregate error, when compiler is cpp.

@mergify backport foxy main galactic


As a user, I want to enter my own name so that the records will reflect that I am the serving doctor

This issue is closed as it has been implemented in #98

Verified, thank you for the work!


ibc: audit fixes

Describe your changes
This PR contains a set of fixes for the higher sev issues from the recent audit of the penumbra-ibc codebase, that are specific to the state machine side of things (as opposed to ibc-types, there will be a follow up PR for ibc-types and a new release for the relevant fixes there).
Checklist before requesting a review

[x] If this code contains consensus-breaking changes, I have added the "consensus-breaking" label. Otherwise, I declare my belief that there are not consensus-breaking changes, for the following reason:

closes #4421
closes #4420
closes #4419
closes #4418

We believe that this PR may resolve a lingering blocker for interchaintest, where timeouts were not being handled properly. Let's include this for Testnet 76 (#4402), so downstream can consume it, and press on with the testing integration. Before I mash the merge button, @avahowell could you respond to @cronokirby's comments above? It'd be helpful to have the narrative when we refer back to this changeset.


comments, likes, posts, notifications and messages two way reference is not needed

What version of Node.js are you using?
v14.17.5
What version of Yarn are you using?
1.22.15
What browser are you using?
chrome
What operating system are you using?
windows
Describe the Bug
Since the corresponding models already exist and you can just query the collection to retrieve a user's respective data, this 2 way reference is not needed and will not scale. Mongodb documents have a size limit of 16mb, if a user has thousands of likes, posts, etc... then everything is gonna break. Also, another downside is sending unnecessary large amount of json data when the user is being queried if only the full name is needed for example.
Applies to following and followers as well.
Expected Behavior
User model should be scalable
Steps to reproduce
Create a popular social network

Hi @kamalyusuf, thanks for your information. I have difficulty understanding why everything will break if there are thousands of posts or likes.
I understand that the current database architecture is not perfect, but I don't think it will have problems on a smaller scale.
However, if you can provide detailed information about the problem and the solution you are thinking of. Then, I'd be glad to discuss it with you and finally merge your PR.  If it's just a theoretical issue and you are not going to work on it, let's close it because we already have a similar one that nobody is working on: https://github.com/ElevenSymbols/orca/issues/181
In general, if somebody will be using Orca on millions of user scale, I'll be expecting their contribution to the project.


chore: bdd-spring-1588429798 to 0.0.1

chore: Promote bdd-spring-1588429798 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Fix one more linux build break in IC Annotation

I also can't find anywhere that featurizer_thread is used (inside of a #ifdef __APPLE__ or otherwise).

Looks like m_image_feature_extraction_sarray and m_writer should also be moved inside the #ifdef in the hpp file.

Thanks @TobyRoseman -- for now I'm going to take this fix (since it gives us a passing Linux build) and plan to refactor this whole class soon.


Release @google-cloud/automl v0.1.3

This pull request was generated using releasetool.

Release is at https://github.com/googleapis/nodejs-automl/releases/tag/v0.1.3

The release build has started, the log can be viewed here. :sunflower:

:egg: You hatched a release! The release build finished successfully! :purple_heart:


Fixed #25950 -- Added support for GEOSisClosed.

https://code.djangoproject.com/ticket/25950

Merged in 5d348bba3178f57bf018aee08d243a92933c5ce0, thanks. I corrected some indentation in the documentation that caused some incorrect rendering. The GIS docs still contain some 3 space indentation while we have standardized on 4 spaces elsewhere. If you wanted to clean that up, it would be welcome.


Add options for building examples and tests.

Also move zlib test sources to test/zlib.
New attempt at options. CMake is cleaner now which makes it easier.

This got merged already


Support POST method for Etherscan.io

lets just always POST then, less logic

changed to POST all

I do think it was more useful with an option :)

easy to add in when necesary, im resistant to feature bloat


Presence last_update_id API

Implements a prototype for the API discussed in https://chat.zulip.org/#narrow/stream/378-api-design/topic/presence.20rewrite/near/1586819
Commit 1 Adds the necessary last_update_id column and PresenceSequence model, commit message elaborates on the migration plan
Commit 2 The backend+frontend parts of the system using the above data structure.
This is WIP, so no tests and only tested in the webapp.
What this has:

Adds the last_update_id param and plumbing in the API through the backend and frontend
Implements what is intended to be an efficient locking mechanism inside do_update_user_presence, updating PresenceSequence and the related UserPresence update in a single SQL query, at the very end of the transaction, ensuring the hot lock only gets held as briefly as possible.
Verified the core mechanism seems to work; the webapp only fetches the recently updated presence data and updates its buddy list with the incremental info

What this does NOT have:

[x] Doesn't handle the discussed case of getting presence info about a user the webapp doesn't know yet (https://chat.zulip.org/#narrow/stream/378-api-design/topic/presence.20rewrite/near/1586828) EDIT: Fixed
[ ] Doesn't remove the user_presence query worker, which we probably want to do, moving presence updates in-process
[x] PresenceSequence for the realm doesn't get created in the import codepath. EDIT: Fixed

Responded to comments and pushed a commit with a proposal for handling the main race:

the discussed case of getting presence info about a user the webapp doesn't know yet (https://chat.zulip.org/#narrow/stream/378-api-design/topic/presence.20rewrite/near/1586828)

This is becoming fairly complete, with now just some frontend test coverage missing, so removing the WIP and prototype claims.

This now has test coverage and is passing CI. The commits are currently structured as a sequence of refinements on top of the main commit to keep things reviewable in logical chunks; feel free to squash if it's easier to review as one huge commit though.

I opened https://github.com/zulip/zulip/pull/30283 to integrate the migrations part of this, with the following changes.
diff --git a/zerver/migrations/0526_user_presence_backfill_last_update_id_to_0.py b/zerver/migrations/0526_user_presence_backfill_last_update_id_to_0.py
index 7083776df6..babf436730 100644
--- a/zerver/migrations/0526_user_presence_backfill_last_update_id_to_0.py
+++ b/zerver/migrations/0526_user_presence_backfill_last_update_id_to_0.py
@@ -17,12 +17,14 @@ def backfill_user_presence_last_update_id(
     BATCH_SIZE = 10000
     lower_bound = 0

-    # Make sure we run past the end in case of new rows created while we run.
+    # Add a slop factor to make it likely we run past the end in case
+    # of new rows created while we run. The next step will fail to
+    # remove the null possibility if we race, so this is safe.
     max_id += BATCH_SIZE / 2

     while lower_bound < max_id:
         UserPresence.objects.filter(
-            id__gt=lower_bound, id__lte=lower_bound + BATCH_SIZE, last_update=None
+            id__gt=lower_bound, id__lte=lower_bound + BATCH_SIZE, last_update_id=None
         ).update(last_update_id=0)
         lower_bound += BATCH_SIZE

diff --git a/zerver/migrations/0527_presencesequence.py b/zerver/migrations/0527_presencesequence.py
index 222fd3b9c0..76a172c06c 100644
--- a/zerver/migrations/0527_presencesequence.py
+++ b/zerver/migrations/0527_presencesequence.py
@@ -20,7 +20,10 @@ def create_presence_sequence_for_old_realms(
     BATCH_SIZE = 2000
     lower_bound = 0

-    # Make sure we run past the end in case of new rows created while we run.
+    # Add a slop factor to make it likely we run past the end in case
+    # of new rows created while we run. Races with realm creation are
+    # pretty unlikely, and should throw an exception, so we should
+    # catch them.
     max_id += BATCH_SIZE / 2

     while lower_bound < max_id:

The first commit says This builds on top of the previous commit which is no longer the case, since the commit in question was merged in #30283, and there have been other things merged since then.

Squashed and split thing into two deployable commits - first just backend API support, and then the frontend actually using it.

The first commit says This builds on top of the previous commit which is no longer the case, since the commit in question was merged in #30283, and there have been other things merged since then.

Thanks, fixed that message to refer that commit specifically by its hash.

Pushed a commit tweaking the details for last_update_id+slim_presence interactions a bit, to fit https://chat.zulip.org/#narrow/stream/378-api-design/topic/presence.20rewrite/near/1817372 It should be squashed into the backed implementation commit, just leaving it separate to easily show what's changed.

Squashed the backend/API commits to merge as #30343.

Probably we can actually merge the rest of this too, I'll come back to that tomorrow.

Merged, thanks @mateuszmandera!


Release Google.Cloud.BigQuery.V2 version 3.6.0

Changes in this release:
New features

Implement int64-based timestamps (commit 9677fcb)

Created release for Google.Cloud.BigQuery.V2-3.6.0


math: new 'huge' trig tests failing on s390x

Presumably the s390x assembly implementations, like the old Go implementations, weren't designed to produce the correct result for 'huge' input values.
--- FAIL: TestHugeCos (0.00s)
    all_test.go:3114: Cos(1.329227995784916e+36) = NaN, want -0.9258790228548379
    all_test.go:3114: Cos(1.7668470647783843e+72) = NaN, want 0.9360104259335379
    all_test.go:3114: Cos(3.1217485503159922e+144) = NaN, want -0.2828277764019379
    all_test.go:3114: Cos(1.8919697882131776e+69) = NaN, want -0.14616431394103618
    all_test.go:3114: Cos(2.514859209672214e+105) = NaN, want -0.7945605821067141
    all_test.go:3114: Cos(1.7976931348623157e+308) = NaN, want -0.9999876894265599
--- FAIL: TestHugeSin (0.00s)
    all_test.go:3124: Sin(1.329227995784916e+36) = NaN, want 0.377820109360752
    all_test.go:3124: Sin(1.7668470647783843e+72) = NaN, want -0.35197227524865776
    all_test.go:3124: Sin(3.1217485503159922e+144) = NaN, want 0.9591707089436872
    all_test.go:3124: Sin(1.8919697882131776e+69) = NaN, want 0.9892603263702362
    all_test.go:3124: Sin(2.514859209672214e+105) = NaN, want -0.6071848823564695
    all_test.go:3124: Sin(1.7976931348623157e+308) = NaN, want 0.00496195478918406
--- FAIL: TestHugeTan (0.00s)
    all_test.go:3144: Tan(1.329227995784916e+36) = NaN, want -0.40806638884180424
    all_test.go:3144: Tan(1.7668470647783843e+72) = NaN, want -0.37603456702698074
    all_test.go:3144: Tan(3.1217485503159922e+144) = NaN, want -3.391359650547799
    all_test.go:3144: Tan(1.8919697882131776e+69) = NaN, want -6.76813854009065
    all_test.go:3144: Tan(2.514859209672214e+105) = NaN, want 0.7641769501660493
    all_test.go:3144: Tan(1.7976931348623157e+308) = NaN, want -0.00496201587444489
FAIL
FAIL	math	0.014s

CC @griesemer @bmkessler for https://golang.org/cl/153059.

The s390x implementations are inaccurate. We either disable the tests for s390x or we switch to the non-assembly implementation.

I'm on this.


CQL evaluation always returns null in $apply operation

CQL evaluation always returns null in $apply operation for PlanDefinition action.condition and ActivityDefinition dynamicValue expressions.
I am working through test cases to try an identify the issue.  The same Library resources and CQL work successfully in CDS Hooks configuration, but not when executing PlanDefinition/xxx/$apply.  A CarePlan is returned, but no CQL expressions are executed successfully while evaluating action conditions or when setting ActivityDefinition dynamicValues in a CommunicationRequest result.
Any known issues with this, or suggestions for debugging?  If not, then I'll narrow down a simplified test scenario and continue logging interim values.  I am working in a fork that I also used to submit a PR for improving ActivityDefinition dynamic values.
Thanks!

In CqlExecutionService, evaluateInContext() creates this source statement:
library LocalLibrary using FHIR version '4.0.1' include FHIRHelpers version '4.0.1' called FHIRHelpers include ProstateCancerScreening called ProstateCancerScreening parameter ActivityDefinition ActivityDefinition parameter "%context" ActivityDefinition define Expression: PSA risk statement 1
The expression to be evaluated is "PSA risk statement 1".  Does this execution not support quoted CQL expressions?
Thanks,
Dave

It should be quoting that, this is a bug in the code that creates the library wrapper for evaluation.

@brynrhodes Just trying to get some clarity on this issue. Does anything need to be done in this codebase or has this issue already been resolved?


Custom Dot Cursor [React]

Reference
Check out the same in the master branch code

please assign me this issue it will be helpful for me

sure @nik464, go ahead.

Hi
I am new to this project can you please guide me @smaranjitghose
If this issue is not resolved

any updates @nik464

ya . I have done it basically i have to increase cursor outline width and size because initially it was very small

i have done it. you can check my pull request.

please add lgm soc to this issue. I am lgm soc participant.

please add lgm soc to this issue. I am lgm soc participant.

done


配置好的任务如何进行海豚调度

Before asking

[X] I have read the How to ask for help (我已阅读如何寻求帮助指引文档).

Your environment

Linkis version used: 1.1.2
Environment name and version:

cdh-5.14.2
hdp-3.1.5
hive-2.1.1
spark-3.2.1
scala-2.12.2
jdk 1.8.0_121
....

Describe your questions

Q1.  ...
Eureka service list

eg:
Some logs info or acctch file

linkis-xxx.log:

<!--日志文字贴到这里-->

log file:

linkis-xxx.log

Maybe you can get some info here https://github.com/WeBankFinTech/DataSphereStudio-Doc/blob/main/README-ZH.md


Requirements Gathering

Basic Information
This task is about liasing with the client to gather the requirements
This will be a collective task, however he/she will be the main point of contact whenever communicating.
Other Information
The goal that we have here is to consolidate what needs to be done for the project. Main points are:

Aim and Scope (the problem we are trying to solve, and the issues there are in the current system)
The features that are considered as MVP (minimum viable product)

Here's an example of an artefact that will result from this liasing https://aser-docs.systemhealthlab.com/developer/requirements/, the requirements document
More about MVP
Identifying MVP is important to determine which features delivers the most amount of value. This means that, if there are "fancy features" that the client doesn't even need, we can cut that in the development. Saving us a lot of time
After Requirements
We need to decide properly on what tech we are going to pick with a discussion as to why we are building one (if we are choosing to build rather than use an existing design solution)

Problem
Queue management (priority 1)
Service in the library as a dropin to get support in any element

research
writing
referencing
study

Start up quite slow in the semester, but it gets really busy. Student leader, sign in service. Problems whenever theres turnover of lectures
Data usage reporting (priority 2)
report

when the busy times
the topics of the busy times

Usage report

how many students
what do they ask
where the busy times are

Note: This is "nice-2-have"
Current Tech

uses Excel and shows (will be sent by Adam)

Current solution
Triage point

client
self-service

Agreement is to make it as low-barrier at a time. Maybe both having a self-service point + client.
Other notes
Usual cap time is around 15-20 minutes per students.
Similarity to Computer Lab sessions:

see as many student as can withtin the allotted time

Existing design solution
Unihub drop platform

does not need the same level

This has to cover both:

scope of work
project acceptance test

Risk register CLE = R (consequence * likelyhood * exposure = risk)

@Alexkszzz  need to follow up on the sample excel file to track students in the queue

Just going to put this here https://lucid.app/lucidchart/invitations/accept/inv_1d4b0ed7-fb98-4a2a-b338-c7a1513e9716?viewport_loc=-423%2C96%2C2219%2C1108%2C0_0 (this contains the Entity Relationship Diagram)

This has been done as part of scope of work and is uploaded in onedrive


cci org info and cci org browser should use default org

If you have a default org set, you shouldn't need to specify it in the CLI for cci org info and cci org browser.  What is a bit tricky is that Click requires args to be passed.  Need to investigate how to have an optional arg or something similar.  Perhaps the commands should match task run and use --org <name> if you want to specify an org other than the default?

dupe #378

related to https://github.com/SalesforceFoundation/CumulusCI/issues/242


"Module parse failed: Unexpected token" when using Lenis-React

Describe the bug
I get this error when using anything from lenis react:
Module parse failed: Unexpected token (3:5)
| import Lenis, { LenisOptions } from '@studio-freight/lenis';
| import React, { ForwardRefExoticComponent, PropsWithoutRef, ReactNode, RefAttributes } from 'react';

type LenisEventHandler = (lenis: Lenis) => void;
| interface LenisContextValue {
|     lenis: Lenis;

Import trace for requested module:
<EMAIL_ADDRESS>./src/components/containers/drawer.tsx
./src/app/(index)/(sections)/(topics)/section.tsx
Although, looking at the file, everything seems correct.

Reproduce Repo
https://github.com/PixelPage-YT/lenis-error

Oops, i figured out my import was messed up. Now i get this error: Error: (0 , react__WEBPACK_IMPORTED_MODULE_1__.createContext) is not a function

it works everywhere else 🤷♂

Yes since it even works in my project we can close this ig
thank you

you need 'use client' on top of the file


Fix max-uids decrease test and add Jira ticket commit msg script

Description
Fix a broken test which prevented cargo-husky from installing a new git hook.
Include new git hook for prepare-commit-msg which inserts the current jira ticket's tag into your commit message based on branch name.

Checklist

[X] Tracked rustfmt changes
[X] No new compiler warnings

this is very nice!! i will add this to some repositories i have! very nice   @Rubberbandits

out of curiosity, how the final message looks like?
Check out the messages in the commits of this PR! The first line gets the branch inserted automatically with the colon, then you just type your normal commit message and push.


Issue endpoint has incorrect description for "held_by_pokemon"

The documentation for v2 says this regarding the held_by_pokemon field on the item endpoint:

While in reality the structure is a bit different:

@M-Zuber Whoops. Thanks for pointing this out. Might not be able to fix this before the code push aimed for this weekend but we'll get it done.