statusline got wrong color.

this is my plugins
 plugins: [
    "hyper-tabs-enhanced",
    "hyperlinks",
    "hyperterm-alternatescroll",
    "hyper-statusline",
    "hyper-dracula"
  ],

BUT it got a statusline like this:

hyper: 1.4.2

problem solved,
theme should be placed above statusline.
 plugins: [
    "hyper-dracula",
    "hyper-statusline"
  ],

This trick doesn't work for extensions that are being developed (local plugins) :(.

This is more of a Hyper issue. Since statusline fetches colors from active theme, that theme needs to be loaded before it. That's the way Hyper handles plugins ✌


[$1000] Red dot remains visible on chat after a failed attachment is deleted

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Click on “Profile” avatar
Click on “Preference” link
Turn on “Simulate a network failure”
Open an existing chat
Click on “Actions” icon
Click on “Add attachment” link
Select a picture of file
Wait few seconds
Turn off “Simulate a network failure”
Click on “Delete” icon on the failed attachment
Observe that the failed attachment is deleted, and the corresponding error indicator ‘Red dot’ remains visible on the chat

Expected Result:
Red dot shouldn’t remain visible after a filed attachment is deleted
Actual Result:
Red dot remains visible after a failed attachment is deleted
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.2.93-4
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/43996225/229311733-d3e450eb-e37c-49b9-845e-399efde391b0.mp4
https://user-images.githubusercontent.com/43996225/229311740-f7a68b5f-2489-44e8-97a9-995256351019.mp4
Expensify/Expensify Issue URL:
Issue reported by: @Natnael-Guchima
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1680249331562699
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01365275cfe90a2203
Upwork Job ID:<PHONE_NUMBER>344288256
Last Price Increase: 2023-04-05

Triggered auto assignment to @bfitzexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

@bfitzexpensify Whoops! This issue is 2 days overdue. Let's get this updated quick!

Think this is valid for external

Job added to Upwork: https://www.upwork.com/jobs/~01365275cfe90a2203

Current assignee @bfitzexpensify is eligible for the External assigner, not assigning anyone new.

Triggered auto assignment to Contributor-plus team member for initial proposal review - @0xmiroslav (External)

Triggered auto assignment to @neil-marcellini (External), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Red dot in LHN (chat) item still shows after deleting the report action that cause the error indicator.
What is the root cause of that problem?
When we delete a report action, we never clear the errors.
https://github.com/Expensify/App/blob/4ee21e95a38858df28e46caf54dc97e1bca9bc7a/src/libs/actions/Report.js#L706-L712
What changes do you think we should make in order to solve the problem?

Add errors: null to clear the error in optimistic data and errors: reportAction.errors in failure data to revert the errors just in case the request fails.
https://github.com/Expensify/App/blob/4ee21e95a38858df28e46caf54dc97e1bca9bc7a/src/libs/actions/Report.js#L725-L737

Not overdue, melv

Proposal
Please re-state the problem that we are trying to solve in this issue.
Red dot remains visible after a failed attachment is deleted.
What is the root cause of that problem?
The root cause is here where we're getting the errors of the report. We don't exclude deleted (or pending deleted) report actions here so after the (errored) report action is deleted, it will still show the red dot.
What changes do you think we should make in order to solve the problem?

We need to add logic below this line https://github.com/Expensify/App/blob/62276baa62ffefbae4ccfc7ed5aacc61a7322737/src/libs/OptionsListUtils.js#L230 to filter only the report actions that are deleted (or pending deleted).
Then use that filtered list to calculate the list of errors of the report.
If we just set the errors to null when deleting the report action, it will not work for the case when we delete the report action from another device. This is because the errors is stored locally.
What alternative solutions did you explore? (Optional)

Another potential solution is to set the errors to null in optimistic data (of course will need to restore it in case of request failure) when deleting the action. And also set the errors to null when we detect that the report action has been deleted (to make sure it still works when deleting the report action from another device), this can be done by putting the logic inside a ONYXKEYS.COLLECTION.REPORT_ACTIONS Onyx listener for example.
While investigating this issue, I found another issue, not sure if it should be fixed in this GH but just mentioning for the sake of completeness: When there's error with posting the report action in 1st device, now the pendingAction is add, and then we delete that report action from a 2nd device. The report action will not disappear from the 1st device.
This is because in here https://github.com/Expensify/App/blob/e3597c68d03ad48b50b328daf5cf3dbd269b678d/src/libs/ReportActionsUtils.js#L209 we still show the report action if the report action has any pendingAction. In this case it has pendingAction is add from the initial report action creating attempt so it still shows even though already deleted.
To fix this we need to update the logic so that if the report action is deleted, it will only still show if the pendingAction is delete.

📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸

@0xmiroslav says he's on it

I kind of like this proposal. It makes sense to remove the errors optimistically and restore them if deleting fails. Thoughts @0xmiroslav? Can you verify that we don't clear the errors in the response or Pusher data from deleting a comment?

@0xmiroslav What is that don't work? If we want to handle when the user deleted the chat from other device, we should clear the errors from the Pusher.

Also IMO the errors of an action and whether the action is deleted are not related to each other. So it might not be good to assume an action does not/did not have an error after it's deleted, it causes data loss. If we don't want to show it in the UI it might be better to filter it out.

If we just set the errors to null when deleting the report action, it will not work for the case when we delete the report action from another device.

@tienifr that doesn't make sense to me, because how would we see the report action (message) on another device if it failed to add? From my local testing, the simulate failing network requests toggle only simulates a failure, but the message actually posts. Since there's no consideration for other devices we can go with @bernhardoj's proposal and we won't need a backend change. Sound good @0xmiroslav?

True the whole issue is an edge case, but if for some reason adding an attachment fails and the user decides to delete the comment rather than dismissing the error, we should clear the error. It's quite a simple front end change so we should do it.

📣 @bernhardoj You have been assigned to this job by @neil-marcellini!
Please apply to this job in Upwork and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑💻
Keep in mind: Code of Conduct | Contributing 📖

Thanks! Will open the PR in a few hours.

Reviewing label has been removed, please complete the "BugZero Checklist".

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[ ] [@0xmiroslav] The PR that introduced the bug has been identified. Link to the PR:
[ ] [@0xmiroslav] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
[ ] [@0xmiroslav] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
[ ] [@bfitzexpensify] Determine if we should create a regression test for this bug.
[ ] [@0xmiroslav] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.
[ ] [@bfitzexpensify] Link the GH issue for creating/updating the regression test once above steps have been agreed upon:

Offers sent out for payment

I think adding a regression test here is a good idea, perhaps in the Compose box - Attachment only (Offline Behavior) test

Offers sent out for payment

Accepted. Thanks 🙏.

@0xmiroslav please review the items in https://github.com/Expensify/App/issues/16852#issuecomment-1515074270 when you get a chance - thanks!

Bump @0xmiroslav

Thanks @0xmiroslav. I think we are good to close this one out now.


vis-network-9.0.4

Updating vis-network to latests version 9.0.4 and a minor error reporting modification that was helpful while creating https://github.com/qiita-spots/qp-spades

After finding the latest open issue reported here: https://github.com/qiita-spots/qiita/issues/2968; I don't think it's worth updating to the latests version of vis-network. Thus, closing this PR and opening a new one with the code modifications and the requested test.


Pretrained Model

This project is great. In paper, i see that Ecapa-TDNN model have two config (C=512, C=1024). Your pretrained model you uploaded has C = 1024. Could you share the pretrained model with C=512.
Thank you so much!

I did not save the model with C=512, sorry

Thank you


Clarify license of models / model+weights coming from net_download

Since net_download fetches files from external urls, the licence of these external files is currently unclear (even if we fix https://github.com/NifTK/NiftyNetModelZoo/issues/1), see e.g.:
https://www.dropbox.com/s/cnbsszxa418xc1x/anisotropic_nets_brats_challenge_weights.tar.gz
which is referred to from
https://github.com/NifTK/NiftyNetModelZoo/blob/master/anisotropic_nets_brats_challenge_model_zoo.ini
My suggestion would be to ask contributors to add a licence file in each of the external compressed files and recommend to use a cc-by or cc0 license.
Checking that a licence file is present could be a prerequisite to merge a new model in (at least at the time of submission).

cc-by license included in each of the model zoo entry's readme


Is there a way to retrieve or change the cover of a page?

I would like to create new items in a database, and for the items I create, I'd like to set a cover. Is this possible? I can't find it in the documentation or code.

Never mind, found it. Cover can be retrieved as such:
page.get("format.cover")

and be set as such:
page.set("format.cover", "http://link.to/my_image.jpg")

Maybe good to document it?

Glad you found it! There are a lot of these types of things, so I didn't try to document everything. If you see a good place to add it to the examples or docs, would welcome a quick PR! Thanks!

Hey, I have a weird issue. API seems to change the cover (I print the field after I set it) but in Notion the cover doesn't change.

Hey @OdysLam
I ran into the same issue, instead of .icon and .cover
it should be:
page.get("format.page_cover")

page.set("format.page_cover", "http://link.to/my_image.jpg")


Suggestion - traps

Your module has inspired me to think of a module that would be great, one of traps:
when a character passes through the marked area and stops, several events could occur:
First of all, an image will jump that is visible to all players (the image of a trap, a pit, rocks, spikes, etc...)
It would be great if the trap had a menu to set it: difficulty CD, damage, description.
So it will jump an event that will make the player throw a cd of the necessary category to avoid the trap.
In the chat a card should appear with the description of the trap, the cd needed to avoid it and below a direct roll of the trap's damage, for example: instead of showing you 2d6, but throwing the damage directly (12, 10,8,2,4,5, etc...)

Thanks for the suggestion! I also like this idea, and it's been suggested by a couple other folks too, so I'm thinking it'd be worth exploring.

There is a new feature in version 0.4.0 which allows you to execute an associated macro with the Hey, Wait! tile. Potentially you could have a journal entry prepared here, or something more complex, which is triggered by the macro. An example macro to show a journal entry could be:
JournalEntry.collection.getName("your_journal_entry_here").show();


Support cron expressions for custom periods

Checklist

[X] I have filled out the template to the best of my ability.
[X] This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
[X] This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.
All predefined periods (except hourly) are currently resetting at 12 midnight. E.g.: it is not possible to reset a daily period at 6PM.
Describe the solution you'd like
Support cron expressions for user to configure a custom period
Describe alternatives you've considered

Additional context

Solved in v0.8.0


Follow up on focus event helper function

@rwjblue there is one other point to chat about at some point too, the focus method in the existing test helpers has an issue with Firefox and focus on window or not. I was able to resolve that in this library as I added test running via Chrome, Firefox on Travis as well.
See…
https://github.com/cibernox/ember-native-dom-helpers/blob/master/addon-test-support/helpers.js#L11-L31
vs.
https://github.com/emberjs/ember.js/blob/7819409b7b6559e9fc0300a29e66a16356fb67a7/packages/ember-testing/lib/events.js#L8-L27

Tests to investigate:

https://github.com/emberjs/ember.js/blob/7819409b7b6559e9fc0300a29e66a16356fb67a7/packages/ember-testing/tests/helpers_test.js#L696-L728
https://github.com/emberjs/ember.js/blob/7819409b7b6559e9fc0300a29e66a16356fb67a7/packages/ember-testing/tests/helpers_test.js#L354-L425

These tests failed for us when running on Firefox or phantomjs…

https://github.com/cibernox/ember-native-dom-helpers/blob/master/tests/integration/fill-in-test.js
https://github.com/cibernox/ember-native-dom-helpers/blob/master/tests/integration/click-test.js

…the focus part was the issue


Retry transaction if not enough fees/gas

Is your feature request related to a problem? Please describe.

Transaction failure due to insufficient gas/fees is not handled
Describe the solution you'd like

@jbibla: if gas price is too low and transaction is dropped, show button for increasing gas price by x% so user can try again (in action modal) https://github.com/cosmos/voyager/issues/1721#issuecomment-466364606

Describe alternatives you've considered

Set gas-prices & estimated fees on UI based on simulation
Keep contents from tx on store
submit tx on sign action modal step
Check if tx failed and gas/fee was insufficient
a. Show back to set gas action modal step
b. Serror that the gas/fee was insufficient

Additional context

depends on #2238 and #1735

Consider this design also for the Tx Page


Search Endpoint use keyword params in trace view.

Please answer these questions before submitting your issue.

Why do you submit this issue?
[ ] Question or discussion
[x] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question

What do you want to know?

Bug

Which version of SkyWalking, OS, and JRE?

Which company or project?

What happened?
If possible, provide a way to reproduce the error. e.g. demo application, component version.

Requirement or improvement

Please describe your requirements or improvement suggestions.

Yes, there is a bug. Are you going to fix this?

@wu-sheng I will fix it later.


Scanner struggles to scan qr with more information

When I try to scan qr with more information it struggles to scan at little distance, I have to take my phone near to scan the qr.

I have also noticed with the latest release scanning a small datamatrix does not work on some devices, and not as well on others as the previous release did.

I am facing same issue: when I pick an image from the gallery, it does not scan that image, and the AnalyzImage method is always false on iOS physical devices.


backup multiclusterobservability under active data backup

https://issues.redhat.com/browse/ACM-4225
Move MultiClusterObservability to the active resource set since this resource once restored on the restore hub creates all obs resources using the backed secrets and configmaps; as a result it creates PV and PVClaims
We don't want to have these resources created until the very end, when the passive cluster becomes active

/cherry-pick release-2.7

/cherry-pick release-2.8


Replacements API documentation missing

Thanks for submitting a bug report, please fill in the following details.
Description
There is a link to the Replacements docs that appears to be broken
Reproduction Steps
Steps to reproduce the behavior:

Go to https://changie.dev/integrations/nodejs/
Click on the link to https://changie.dev/config/replacements/
See the 404 page

What happened
404 page with no docs on the replacement api
Expected behavior
I would expect to see some documentation of the code in core/replacement.go.

Hmm I think it's supposed to link here https://changie.dev/config/#config-replacements
Arguably not a very detailed section though.

That would have been really helpful since I spent about an hour trying to figure out why changie batch didn't run it 😢
I'll try to give this update a shot later today!


Cannot convert a symbolic Tensor (gru/strided_slice:0) to a numpy array.

System information

OS Platform and Distribution: Arch Linux
TensorFlow installed from package: python-tensorflow
TensorFlow version: unknown 2.4.1
Python version: 3.9.1
CUDA/cuDNN version: No GPU

Standalone code to reproduce the issue
from tensorflow.keras.layers import Embedding, Input, GRU

x = Input(shape=(None,))
x = Embedding(input_dim=50, output_dim=16, mask_zero=True)(x)
x = GRU(units=256)(x)

Other info / logs
Traceback (most recent call last):
  File "/home/jnphilipp/Nextcloud/Code/jnphilipp/deep_learning/test3.py", line 5, in <module>
    x = GRU(units=256)(x)
  File "/usr/lib/python3.9/site-packages/tensorflow/python/keras/layers/recurrent.py", line 660, in __call__
    return super(RNN, self).__call__(inputs, **kwargs)
  File "/usr/lib/python3.9/site-packages/tensorflow/python/keras/engine/base_layer.py", line 951, in __call__
    return self._functional_construction_call(inputs, args, kwargs,
  File "/usr/lib/python3.9/site-packages/tensorflow/python/keras/engine/base_layer.py", line 1090, in _functional_construction_call
    outputs = self._keras_tensor_symbolic_call(
  File "/usr/lib/python3.9/site-packages/tensorflow/python/keras/engine/base_layer.py", line 822, in _keras_tensor_symbolic_call
    return self._infer_output_signature(inputs, args, kwargs, input_masks)
  File "/usr/lib/python3.9/site-packages/tensorflow/python/keras/engine/base_layer.py", line 863, in _infer_output_signature
    outputs = call_fn(inputs, *args, **kwargs)
  File "/usr/lib/python3.9/site-packages/tensorflow/python/keras/layers/recurrent_v2.py", line 439, in call
    inputs, initial_state, _ = self._process_inputs(inputs, initial_state, None)
  File "/usr/lib/python3.9/site-packages/tensorflow/python/keras/layers/recurrent.py", line 859, in _process_inputs
    initial_state = self.get_initial_state(inputs)
  File "/usr/lib/python3.9/site-packages/tensorflow/python/keras/layers/recurrent.py", line 642, in get_initial_state
    init_state = get_initial_state_fn(
  File "/usr/lib/python3.9/site-packages/tensorflow/python/keras/layers/recurrent.py", line 1948, in get_initial_state
    return _generate_zero_filled_state_for_cell(self, inputs, batch_size, dtype)
  File "/usr/lib/python3.9/site-packages/tensorflow/python/keras/layers/recurrent.py", line 2987, in _generate_zero_filled_state_for_cell
    return _generate_zero_filled_state(batch_size, cell.state_size, dtype)
  File "/usr/lib/python3.9/site-packages/tensorflow/python/keras/layers/recurrent.py", line 3005, in _generate_zero_filled_state
    return create_zeros(state_size)
  File "/usr/lib/python3.9/site-packages/tensorflow/python/keras/layers/recurrent.py", line 3000, in create_zeros
    return array_ops.zeros(init_state_size, dtype=dtype)
  File "/usr/lib/python3.9/site-packages/tensorflow/python/util/dispatch.py", line 201, in wrapper
    return target(*args, **kwargs)
  File "/usr/lib/python3.9/site-packages/tensorflow/python/ops/array_ops.py", line 2819, in wrapped
    tensor = fun(*args, **kwargs)
  File "/usr/lib/python3.9/site-packages/tensorflow/python/ops/array_ops.py", line 2868, in zeros
    output = _constant_if_small(zero, shape, dtype, name)
  File "/usr/lib/python3.9/site-packages/tensorflow/python/ops/array_ops.py", line 2804, in _constant_if_small
    if np.prod(shape) < 1000:
  File "<__array_function__ internals>", line 5, in prod
  File "/usr/lib/python3.9/site-packages/numpy/core/fromnumeric.py", line 3030, in prod
    return _wrapreduction(a, np.multiply, 'prod', axis, dtype, out,
  File "/usr/lib/python3.9/site-packages/numpy/core/fromnumeric.py", line 87, in _wrapreduction
    return ufunc.reduce(obj, axis, dtype, out, **passkwargs)
  File "/usr/lib/python3.9/site-packages/tensorflow/python/framework/ops.py", line 852, in __array__
    raise NotImplementedError(
NotImplementedError: Cannot convert a symbolic Tensor (gru/strided_slice:0) to a numpy array. This error may indicate that you're trying to pass a Tensor to a NumPy call, which is not supported

+1! I'm getting the same issue.

@jnphilipp
I ran the code shared above it seems incomplete, please find the gist here [there is no error or output]
menwhile can you please refer to these resolved issues and let us know: link, link2

@Saduf2019  I know the code looks incomplete, but it is the minimum that I need to get the error message above.
As to the issues: I don't even import numpy.

i too am having this issue just now
i have been running the same scripts for months, and i just atrted having this isseu today when i re-reun them. Last time they worked was 7 days ago (i even have the git commit uploading the then freshly executed script)
the only thing that changed in between was an updated version of numpy

Me too, after the last update.

@jnphilipp I ran your code in TF2.4.1 and I don't see any error as you described. Please check the gist here. Am i missing something here?
@ZeroOut @ghylander Can you please share a standalone code to reproduce the issue? Thanks!

@jvishnuvardhan
yes
'numpy.version'
'1.19.3'
this problem arises with numpy version 1.20 and 1.21

@jvishnuvardhan
yes
numpy.__version__
1.19.3
this problem arises with numpy version 1.20 and 1.21

Thanks! The same issue just started happening on my setup, and downgrading to 1.19.4 solved.

@jvishnuvardhan no, you're not missing anything in the gist.

.

So, the issue was resolved for you.? Thanks!

@jvishnuvardhan no, you're not missing anything in the gist.

So, the issue was resolved for you.? Thanks!

@ZeroOut @ghylander As mentioned by you, looks like this was related to numpy version. I just checked that TF was built with 'numpy ~= 1.19.2' . Please check here to know all the packages and their versions that were used when building TF pip package2.4. Thanks!

Similar issues that are due to numpy version 1.20.
https://github.com/tensorflow/tensorflow/issues/47263
https://github.com/tensorflow/tensorflow/issues/47311

I met a similar problem when I run the BERT model (repo: https://github.com/google-research/bert).
I have downgrade Numpy from 1.20 to 1.19.2 and the problem solved.
Thanks!

I am closing this issue as this is more related to numpy version as mentioned above and not related core Tensorflow. Loading correct numpy version resolves the issue. Thanks!

Moved to #47691

Try making a new python environment and install tensorflow==2.4.0

@jvishnuvardhan yes numpy.__version__ 1.19.3 this problem arises with numpy version 1.20 and 1.21

Saved my day! Thanks!

[Solved]
I have the same error when training the data, even also try to downgrade the NumPy version, but it not be fixed.
Finally, I solved that issue. Try the below command:
!pip install numpy==1.17.4
!pip install pycocotools==2.0.0


fix(Request): extend error reasons based on missing values from spec

As the title indicated, adds two allowed error codes that are specified in the Chrome Developer Tools Protocol specifications but missing from the Puppeteer enum:
https://chromedevtools.github.io/devtools-protocol/tot/Network#type-ErrorReason

@googlebot Signed it.


Fixed #679 - getStrLn should fail with EOFException when underlying Reader …

…returns null
I was in two minds regarding the 'purity' of the following form but erred on the side of not paying for the additional flatMap...
IO.effect(SConsole.withIn(reader) {
   StdIn.readLine()
})
.flatMap {
   case s if s == null => IO.fail(new EOFException("..."))
   case s => UIO.succeed(s)
}
.refineOrDie ...

Sorry, made a bit of a mess of the commits there. I realise it's worth doing issues on their own branch for safety

Moved code relating to #651 onto separate branch while this is pull request is open

🙏🎉 Thanks for your work on this!

@jdegoes The pleasure was all mine :-)
I realise I've got a bit of a git deficiency I'll need to address in the future


Add manifold for Wishart distribution

I will look into it later, technically WishartFast is our internal construct, preferably we should project on Wishart?

Then we will need to define log partition, gradlogpartition and fisher for Wishart from distributions?

Hm, you are right, I will come back to this issue later next week


feat: add publisher for detailed stop reason

Description

Modify some planning modules to publish detailed reason of vehicle stop.
The additional message includes stop factor, stop reason, and state.
The purpose of this change is to enable external application to recognize the reason of vehicle stop at each stop point.
Modified modules are the following:

scene modules in behavior velocity planner, obstacle
obstacle stop planner
surround obstacle checker

Related links

Modify launcher for remap in this PR
Tests performed

Run planning simulator.
Put initial pose and goal.
Open another terminal and run the launcher : ros2 launch default_ad_api default_ad_api.launch.py
Run the command : ros2 topic echo /api/get/motion_factors

The topic should be published as follows:

header:
stamp:
sec:<PHONE_NUMBER>
nanosec: 500246668
frame_id: map
motion_factors:

pose:
position:
x: 3717.7839452345706
y: 73723.42854108683
z: 0.7956681883687983
orientation:
x: 0.0
y: 0.0
z: 0.23976677505715965
w: 0.9708305174327234
distance: 5.599999904632568
reason: 9
status: 2
detail: ''
pose:
position:
x: 0.0
y: 0.0
z: 0.0
orientation:
x: 0.0
y: 0.0
z: 0.0
w: 1.0
distance: .inf
reason: 3
status: 1
detail: ''

Notes for reviewers

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

Please resolve conflicts.

The conflict is resolved.

Mm, I'm sorry. I guess I can't spare time to review this for a while and that I'm not the right person to review this... :thinking:
Could you choose another appropriate reviewer?

@TakumiKozaka-T4 cc @isamu-takagi  Please share the API design document?

@.TakumiKozaka-T4 has left.
hould we continue to develop this PR?

@shmpwk
I'll take over this PR. And convert it a draft once to match this API.

This PR is taken over https://github.com/autowarefoundation/autoware.universe/pull/1985.


json: add "null"

Fixes #1366

Thanks. Do you have a name that we can add to AUTHORS, or would you prefer not to give one?

Thanks for considering this. I think, we could add a name, once i've contributed anything more significant.

We've the habit of adding names for everyone who contributes to AUTHORS, so I'd be happy to add yours. Anyway, I merged in this change.


APM/OTel relationship with AWS MQ Queue

Relevant information
For any service instrumented using Otel or New Relic APM, a service map is shown in the new relic dashboard which displays the relationship between the service in question and other entities it is dependent on.
In case a service calls a MQ Queue service of AWS, this relationship is not shown in the service map. Instead, it shows that the service calls an external entity.
With the code changes in this PR, the user can correctly see the relationship between the Otel/APM service and Amazon MQ Queue (ActiveMQ or RabbitMQ)
Checklist

[ ] I've read the guidelines and understand the acceptance criteria.
[ ] The value of the attribute marked as identifier will be unique and valid.
[ ] I've confirmed that my entity type wasn't already defined. If it is I'm providing an explanation above.

Hi @Sanjalee-NewRelic.
We will be closing PR's that haven't had any interaction in a month. If this PR is still needed please reopen the PR and answer the feedback requests.


starting camera position is different in Unity XR player vs. WebVR export

see this comment I made in @caseyyee's PR #295: https://github.com/mozilla/unity-webvr-export/pull/295#discussion_r194188654

something in here caused the default camera position to be different (compare it to the position in master)
with eb01ad3 (previously master)

in this branch

related issues:

#284 for XR player for Oculus

#102 for WebVR export for Windows Mixed Reality

#154 for general improvements to the camera, specifically, per @delapuente:

During design time, there should be only one camera in the scene hierarchy. The camera is doubled in gameplay.

Most of the items were implemented in #159 . Changing the camera rigging to a unique camera duplicated in runtime feels like a considerable change in the way the prefab is used. Furthermore, SteamVR Plugin also supports a stereo + mono camera rigging. I think this needs further discussion.

Seems to work for me now.  Reopen if you encounter this issue again!


Allow more CSS custom properties on cds-accordion-header

Please review our Contribution guide for more details about how to contribute effectively.
Summary
Describe the change or feature in detail. Try to answer the following questions.

What is the change?
Add --box-shadow, --icon-visibility, and --icon-margin CSS custom properties to cds-accordion-header.

Why should it go in Clarity?
This will allow Clarity users to extend the Clarity Core Accordion and build a stepper component with a look and feel that matches the Clarity Angular Stepper.

Does this change impact existing behaviors? If so how?
It will not change existing behaviors.

If this change introduces a new behavior, is this behavior accessible?
N/A.

Use case
Provide some specific details about how this change would improve your application and use case.
The added CSS custom properties will allow styling the Core accordion to match the Angular Stepper:

--box-shadow for adding panel status indicator color block on the left of the panel header

--icon-visibility for hiding the cds-button-expand in inactive/disabled panels

--icon-margin for aligning the cds-button-expand to the top

Examples
If possible, show examples of the change. You may create one yourself, or link to external sites that have the idea. It can also be very useful to prototype the idea in isolation outside of Clarity with a Stackblitz example.
API
Describe the intended API for the feature you want to add. This would include:

CSS classes and DOM structure for pure static UI contribution, and inputs/outputs, components, directives, services, or anything that is exported publicly for Angular contributions.
Add --box-shadow, --icon-visibility, and --icon-margin CSS custom properties to cds-accordion-header.

Examples of code snippets using this new feature.
.header {
  --icon-margin: var(--cds-global-space-3) 0 auto 0;
}

.panel.inactive .header {
  --box-shadow: inset var(--cds-global-space-5) 0 0 var(--cds-alias-status-neutral-tint);
  --icon-visibility: hidden;
}

Note very clearly if anything might be a breaking change.
No breaking change.

In the case of bug fixes or internal changes, there will most likely be no API changes.
Implementation plan
Describe how you plan to implement the feature, answering questions among the following or anything else you deem relevant.

What parts of the code are affected?
See https://github.com/vmware-clarity/core/pull/279

Will you introduce new services, components or directives?
No.

Can you describe the basic flow of information between classes?
N/A.

Does this introduce asynchronous behaviors?
No.

Will you need to refactor existing Clarity code?
No.

Will reasonable performance require optimizations?
No.

Will it need to access native elements (and be incompatible with server-side rendering)?
N/A.

...

Conclusion
Describe how long you expect it to take to implement, what help you might need, and any other details that might be helpful. Don't worry, this is obviously non-contractual. 😛
The change is implemented in https://github.com/vmware-clarity/core/pull/279. Looking for reviews and approvals.

Resolved with #279


Blur doesn't work on iOS 10.3.1

Describe the bug
Regular bottom toast with blurred background doesn't work on iOS 10.3.1
To Reproduce
Steps to reproduce the behavior:

open demo project
show regular bottom toast

Screenshots

iPhone (please complete the following information):

iOS Version: 10.3.1
Xcode Version 9.4.1
SwiftEntryKit Release # 0.5.1

@Banck - Just got to address this but I'm not able to reproduce it anymore. It's considered resolved IMO. Let me know if you think otherwise.


provide assertEquals for floating point values

Not a bug, but a feature request,
in Java there is an assertEquals for floats and doubles which lets you specify an epsilon value, so that imprecisions in calculations can be dealt with.
assetEquals(1.0,0.9999999999, 10e-10) would be true if the difference is <= the epsilon.

Yes, I have thought about it, but the question is just whether epsilon is really a good measure. There are other, generally better measures like relative error.

the epsilon variety is well known from junit, and it is really simple. It would not hurt of course to provide to different asserts. One for "dummies", anotherone for more complex scenarios.


[YouTube Downloader] "output format mp4 not available"

Extension
https://www.raycast.com/vimtor/youtube-downloader
Raycast Version
1.69.2
macOS Version
14.3
Description
I installed ffmpeg with brew and checked the filepath with "which ffmpeg", then copied that filepath into the extension, same with probe. Am able to launch ffmpeg from the terminal, but when I try to use the raycast extension, it downloads to 100%, then says "Encoding failed: Output mp4 not available"
How do I fix this?
Steps To Reproduce
Software versions above, base environment
Launch Raycast
Open YouTube downloader
paste in youtube URL
run
then the error appears
Current Behaviour
Encoding failed
Expected Behaviour
Download in specified folder

Found the issue here, implemented the "temporary fix" of unlinking and then linking ffmpeg. Now instead of saying the format is not available, it just finishes, but no download in the folder.

This was solved in https://github.com/raycast/extensions/pull/12583
@raycastbot close this issue


chanageable font size/type on file explorer

Description of the new feature / enhancement
Itd b nice 2 have a text option 4 explorer 2 increase/decrease size & maybe even chang the font? I hate that u cant change the size of text there, so if u cange font size in windows settings so its larger, explorer text seems waaaay smaller then- make me feel blinder than a bat
Scenario when this would be used?
nytime text is 2 small 2 c
Supporting information
No response

This is better for Windows Feedback hub.
Windows does already support this and if something isn't working correctly for how your system is configured, that is the best way to resolve it.  I tried on my Windows 11 box and font sizes did adjust appropriately.


Disable map click / find intersecting polygon functionality on report page

Don't react to map clicks when the chart and data table are visible. Just show the polygon of the currently selected AOI.

Resolved in #40.

Ignore my earlier comment. This has not been resolved yet.


Fix typo in index.md

hestitate -> hesitate

Coverage decreased (-0.04%) to 63.011% when pulling 2273c7d9937ff22767297d118fc60924a46a1ef9 on slel:patch-1 into 91fb46809d2376f1b81f521f8b9d7f4e7804d3c9 on dcjones:master.

Fixed in #879


Distribute nodes across Kafka partitions more evenly

The current mechanism to assign metrics to a Kafka partition
based on the hostname.hashCode() does not generate a distribution even
enough and create some hot spots in our Kafka cluster. By calculating a
CRC32 of the hostname and using that instead we create a more even
distribution, according to our tests.

@zfrank Sorry for the late approval. For some reason I'm not receiving notifications for updates here.


CUDA Compability 3.0 not working

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 16.04
TensorFlow installed from (source or binary): source
TensorFlow version: 1.13.1
Python version: 3.5.2
Installed using virtualenv? pip? conda?: pip3
Bazel version (if compiling from source): 0.21.0
GCC/Compiler version (if compiling from source): 5.5.0
CUDA/cuDNN version: 10.0/7
GPU model and memory: GTX770 4GB

Describe the problem
I compiled tensorflow from sources with compute compability 3.0 but after installation it still looks for 3.5

2019-05-31 15:06:10.209035: I tensorflow/core/platform/profile_utils/cpu_utils.cc:94] CPU Frequency:<PHONE_NUMBER> Hz
2019-05-31 15:06:10.209418: I tensorflow/compiler/xla/service/service.cc:150] XLA service 0x4042a70 executing computations on platform Host. Devices:
2019-05-31 15:06:10.209526: I tensorflow/compiler/xla/service/service.cc:158]   StreamExecutor device (0): ,
2019-05-31 15:06:10.252169: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:998] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2019-05-31 15:06:10.252663: I tensorflow/compiler/xla/service/platform_util.cc:194] StreamExecutor cuda device (0) is of insufficient compute capability: 3.5 required, device is 3.0
2019-05-31 15:06:10.252779: F tensorflow/stream_executor/lib/statusor.cc:34] Attempting to fetch value instead of handling error Internal: no supported devices found for platform CUDA

Any other info / logs
I followed official guides
https://www.tensorflow.org/install/gpu
https://www.tensorflow.org/install/source
for installation of NVIDIA packages and build process.
In configure step I chose CUDA only, set the paths and select version 3.0 (btw it was default). Everything else was default.
I built tensorflow from v1.13.1 tag.

You need to build with XLA off, if you want to use CUDA 3.0. This message was added in 1.14, but it not backported into 1.13.1
https://github.com/tensorflow/tensorflow/blob/v1.14.0-rc0/configure.py#L1021-L1023
        if ver < 3.5:
          print('WARNING: XLA does not support CUDA compute capabilities '
          'lower than 3.5. Disable XLA when running on older GPUs.')

That was it. Thanks.


Tests in llvm ir continuation

In this PR, more DifferentialFunctionComparator unit tests were rewritten using LLVM IR.
To do this, methods allowing testing certain DifferentialFunctionComparator methods in LLVM IR were added.
This allows to rewrite unit tests which looked like this:
TEST_F(DifferentialFunctionComparatorTest,
       CmpOperationsWithOpDiffTypes) {
    BasicBlock *BBL = BasicBlock::Create(CtxL, "", FL);
    BasicBlock *BBR = BasicBlock::Create(CtxR, "", FR);

    auto ConstL = ConstantInt::get(Type::getInt32Ty(CtxL), 2);
    auto AddL = BinaryOperator::Create(
            BinaryOperator::Add, ConstL, ConstL, "", BBL);
    auto CondL = ICmpInst::Create(llvm::Instruction::ICmp,
                                  llvm::CmpInst::ICMP_EQ,
                                  AddL,
                                  AddL,
                                  "",
                                  BBL);

    auto ConstR = ConstantInt::get(Type::getInt64Ty(CtxR), 2);
    auto AddR = BinaryOperator::Create(
            BinaryOperator::Add, ConstR, ConstR, "", BBR);
    auto CondR = ICmpInst::Create(llvm::Instruction::ICmp,
                                  llvm::CmpInst::ICMP_NE,
                                  AddR,
                                  AddR,
                                  "",
                                  BBR);

    ASSERT_NE(DiffComp->testCmpOperationsWithOperands(CondL, CondR), 0);
}

to look like this:
TEST_F(DFCLlvmIrTest, CmpOperationsWithOpDiffTypes) {
    auto left = R"(define void @f() {
        %1 = add i32 2, 2
        %cond = icmp eq i32 %1, %1
        ret void
    })";
    auto right = R"(define void @f() {
        %1 = add i64 2, 2
        %cond = icmp ne i64 %1, %1
        ret void
    })";
    CREATE_FROM_LLVM(left, right);
    ASSERT_NE(testCmpOperationsWithOperands("cond"), 0);
}

The PR contains 15 rewritten unit tests, 14/15 of the rewritten tests were using/testing certain DifferentialFunctionComparator methods (e.g. cmpGEPs), and the last one was testing custom patterns.
Tests that are NOT rewritten:

tests using dbg informations (eg. CmpGepsRenamed),
tests which are already short/simple (eg. CmpAttrs),
tests that compare constants with something  (eg. CmpValuesCastFromUnion) because I did not find a good way of
rewriting/testing this.

Note: For now I left the rewritten tests at the same place (in DifferentialFunctionComparatorTest.cpp file) so it is easier to review, after that I will move it to DFCLlvmIrTest.cpp file.

I addressed the comments and moved the rewritten tests to the DFCLlvmIrTest.cpp file.


Infinite loading on internal server error

Describe the bug
https://share.getcloudapp.com/04uqpvwx When I try to go into a resource and server returns 500, I have a default error popup that something went wrong and afterwards the loader never stops. Is there some way to make loader stop and show a custom message?

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

can you paste the stacktrace from logs?

https://share.getcloudapp.com/WnuJovN8 I know that this loader never stops because of non existent user id, that's why server throws 500, but there should be a way to show a custom message instead of infinite loader in case of server errors

ok. noted. i think it is related to #580

closing in favour of #580


[FR] $GITHUB path for cloning repos?

Is there a way to set the path to the root of all local GitHub code and have it automagically clone to there?
This is what I'd like to do. Set a path with some environment variable:
export $GITHUBPATH=$HOME/src/github.com

Run hub clone:
hub clone oreillymedia/HTMLBook

And have hub put the code into ~/src/github.com/oreillymedia/HTMLBook, creating the necessary directories. This could even work regardless of my current working directory.
Of course this in inspired by go get, but without being specific to Go, or even requiring that Go be installed.
What do you think?

I'd like to +1 this feature request--it would be very useful to Go developers and anyone else who prefers this structure. If $GITHUBPATH is unset, it could do the normal git behavior (cloning into $PWD/project) thereby not affecting users who don't want this feature.
What's the roadmap for CLI extensibility? Is this feature being tracked somewhere?

We had it planned in our minds but now I've added it to our roadmap. It might take a while to develop. Suggestions how users could technically hook into hub and define their own behaviors is welcome. Ideally they shouldn't be required to write Go code.

Did something like this ever happen? All the linked issues are closed.
@homburg suggested I use https://github.com/homburg/git-get


Request release 2.5.1

Several of the merged PR's since the 2.5.0 release, particularly #443 and #492, resolve vulnerabilities. Can you please release a 2.5.1 or 2.6.0 version?  Thanks!

There is no reaction from the authors since months. Thus I would not bet on a new version ;-)
Maybe you think about switching to that fork:
https://relearn.netlify.app/

I've forked this project to McShelby/hugo-theme-relearn. The fork is meant to be a drop-in replacement of this theme.
See further statement in matcornic/hugo-theme-learn#442.


game crashes when loading with better animations collection

game crashes right after loading with better animations collection & puzzles lib

Do you have a crash log, or game log?

crash-2024-04-22_18.57.54-client.txt

Crash is from a failure to create a vanilla model. The log doesn't have any useful information as to why it failed. Would you be able to send the full run log? It's in the Minecraft directory under logs/

ill send the run logs from the day
2024-03-22-2.log.gz
2024-03-22-3.log.gz
2024-03-22-4.log.gz
2024-03-22-5.log.gz
2024-03-22-6.log.gz
2024-03-22-1.log.gz

I was unable to replicate this crash. Running the anims collection mod (+ puzzles lib) alongside changed with some performance mods (Rubidium, Oculus, Modernfix, and spark). I am uncertain to what is causing your issue, but it does not appear to be directly related to Changed.

I was unable to replicate this crash. Running the anims collection mod (+ puzzles lib) alongside changed with some performance mods (Rubidium, Oculus, Modernfix, and spark). I am uncertain to what is causing your issue, but it does not appear to be directly related to Changed.

im starting to think its optifine


composer up not working for yii2

After composer up --prefer-dist I get
Problem 1
      - Installation request for yiisoft/yii2 <IP_ADDRESS> -> satisfiable by yiisoft/yii2[<IP_ADDRESS>].
      - yiisoft/yii2 <IP_ADDRESS> requires cebe/markdown ~1.0.0 | ~1.1.0 -> no matching package found.

This problem I get after your new release 1.2. https://github.com/cebe/markdown/releases/tag/1.2.0
3 days ago composer up worked fine.

https://packagist.org/packages/cebe/markdown#1.1.0 Tags are in place. Have you updated composer itself recently?

Can you show your composer.json file?

Yes, I also see this tag. But it not working.
Today I also did  composer self-up
My composer file
{
  "name": "yiisoft/yii2-app-advanced",
  "description": "Yii 2 Advanced Project Template",
  "keywords": [
    "yii2",
    "framework",
    "advanced",
    "project template"
  ],
  "homepage": "http://www.yiiframework.com/",
  "type": "project",
  "license": "BSD-3-Clause",
  "support": {
    "issues": "https://github.com/yiisoft/yii2/issues?state=open",
    "forum": "http://www.yiiframework.com/forum/",
    "wiki": "http://www.yiiframework.com/wiki/",
    "irc": "irc://irc.freenode.net/yii",
    "source": "https://github.com/yiisoft/yii2"
  },
  "repositories": [
    {
      "type": "composer",
      "url": "https://asset-packagist.org"
    }
  ],
  "minimum-stability": "dev",
  "require": {
    "php": ">=7.0",
    "yiisoft/yii2": "<IP_ADDRESS>",
    "yiisoft/yii2-bootstrap": "~2.0.6",
    "yiisoft/yii2-swiftmailer": "~2.1.0",
    "yiisoft/yii2-authclient": "~2.1.0",
    "yiisoft/yii2-jui": "~2.0.0",
    "hiqdev/composer-config-plugin": "*",

    "samdark/hydrator": "~1.0.0",
    "samdark/yii2-webshell": "~2.0",

    "loveorigami/lo-core": "*",
    "bower-asset/flatpickr": "3.0.*"
  },
  "require-dev": {
    "yiisoft/yii2-debug": "~2.0.0",
    "yiisoft/yii2-gii": "~2.0.0",
    "yiisoft/yii2-faker": "~2.0.0"
  },
  "config": {
    "process-timeout": 1800
  },
  "scripts": {
    "post-install-cmd": "php init --env=Development --overwrite=n"
  }
}

... and if i deleted

    {
      "type": "composer",
      "url": "https://asset-packagist.org"
    },

result is same

for "yiisoft/yii2": "2.0.*" update is good, because fixed in https://github.com/yiisoft/yii2/issues/15899

Reported to Jordi and Nils via email.

For now drop vendor and run composer update

I suspect the following to be the problem:

you have composer.json with "yiisoft/yii2": "<IP_ADDRESS>", which depends on "cebe/markdown": "~1.0.0 | ~1.1.0".
before I released 1.2.0, there was a branch-alias in composer.json of cebe/markdown: "dev-master": "1.1.x-dev".
So if you have minimum-stabilty: dev as I see in your composer.json above, you get something installed that is now released as 1.2.0.
now after 1.2.0 release, I changed the branch alias to 1.2.x-dev
a composer update which was at the 1.1.x-dev branch now wants to upgrade but has the constraint to stay in the "~1.0.0 | ~1.1.0" constraint given by Yii. It can not upgrade to 1.2.0, but needs to downgrade from 1.1.x-dev to some stable 1.1.x version, but that seems to fail.

Not sure if what I described above is the cause of the problem but I think it is very likely.

Nils confirmed that packagist/composer are OK i.e. UI displays correct data.

@cebe Many Thanks!

Hi is there an actual solution to this problem?  I have having the same error message.

@tnguyen444 are you able to reproduce the problem in a minimal setup of composer? Would be worth reporting that as a bug.

I tried to rebuild this problem with a minmal set of packages but I am unable to reproduce it with composer 1.7.1, maybe it has been resolved already? I found something else that is weird when using minimum-stability: dev, reported that in https://github.com/composer/composer/issues/7544

could be related to https://github.com/composer/composer/issues/7329


PR for issue #2555

Follow this discussion
I create this PR. Keep APIs the same, modify the implementation of persist method.
pls review my code, thanks.

Test case for this feature is here.

Sure, I will try.


Remove dependency on commons-validator

We have benchmark, sample, and test code that utilizes commons-validator:commons-validator:1.6. This dependency is brought into our build system here:
[INFO] --- maven-dependency-plugin:2.8:tree (default-cli) @ azure-cosmosdb-commons ---
[INFO] com.microsoft.azure:azure-cosmosdb-commons:jar:2.4.5
[INFO] \- commons-validator:commons-validator:jar:1.6:compile
[INFO]    \- commons-beanutils:commons-beanutils:jar:1.9.2:compile

Notice thatcommons-validator depends on commons-beanutils:1.9.2 and that Snyk reports this version of commons-beanutils contains a security vulnerability:
High severity vulnerability found in commons-beanutils:commons-beanutils
  Description: Arbitrary Code Execution
  Info: https://snyk.io/vuln/SNYK-JAVA-COMMONSBEANUTILS-30077
  Introduced through<EMAIL_ADDRESS>  From<EMAIL_ADDRESS>><EMAIL_ADDRESS>><EMAIL_ADDRESS>><EMAIL_ADDRESS>
A fix to this issue is under development but is not yet released:
https://snyk.io/vuln/SNYK-JAVA-COMMONSBEANUTILS-30077
We should update our dependencies as soon as the fix is available. Here is the full list of direct dependencies on commons-validator reported by maven.
 com.microsoft.azure:azure-cosmosdb-commons:jar:2.4.5
[INFO] \- commons-validator:commons-validator:jar:1.6:compile
[INFO]    \- commons-beanutils:commons-beanutils:jar:1.9.2:compile

com.microsoft.azure:azure-cosmosdb-commons-test-utils:jar:2.4.5
[INFO] \- com.microsoft.azure:azure-cosmosdb-commons:jar:2.4.5:compile
[INFO]    \- commons-validator:commons-validator:jar:1.6:compile
[INFO]       \- commons-beanutils:commons-beanutils:jar:1.9.2:compile

com.microsoft.azure:azure-cosmosdb-gateway:jar:2.4.5
[INFO] \- com.microsoft.azure:azure-cosmosdb-commons:jar:2.4.5:compile
[INFO]    \- commons-validator:commons-validator:jar:1.6:compile
[INFO]       \- commons-beanutils:commons-beanutils:jar:1.9.2:compile

com.microsoft.azure:azure-cosmosdb:jar:2.4.5
[INFO] \- com.microsoft.azure:azure-cosmosdb-commons:jar:2.4.5:compile
[INFO]    \- commons-validator:commons-validator:jar:1.6:compile
[INFO]       \- commons-beanutils:commons-beanutils:jar:1.9.2:compile

I don't see any code using org.apache.commons.validator
(base) moderakh@JOFORNAR-X1CARB:~/github/azure/azure-cosmosdb-java$ grep "org.apache.commons.validator" -r .
(base) moderakh@JOFORNAR-X1CARB:~/github/azure/azure-cosmosdb-java$
Does any code uses it? if not maybe we can remove that instead.

Root cause: com.microsoft.azure.cosmosdb.internal.directconnectivity.EndpointMock was using commons-collections, not commons-collections4 which is what we use elsewhere. All is well after changing this EndpointMock import from:
import org.apache.commons.collections.map.HashedMap;

to:
import org.apache.commons.collections4.map.HashedMap;

I will port this change in my current working branch to PR #137.

The fix to this issue is now being tested in PR#137.


add golang openapi implementation

add golang openapi implementation

Hey @wzshiming Thanks for the PR.  It would be great if you could add to your Readme that you are supporting OpenAPI V3.


Markdown Preview fails to install due to missing pygments.formatters module

While installing the latest Markdown Preview using Package Control, I run into the error that the pygments.formatters module cannot be found. I'm installing on the latest Sublime 3.0, build 3143, and am running on Windows 10.
The console shows:
reloading plugin Markdown Preview.helper
reloading plugin Markdown Preview.markdown_settings
reloading plugin Markdown Preview.markdown_wrapper
reloading plugin Markdown Preview.MarkdownPreview
Traceback (most recent call last):
  File "C:\Program Files\Sublime Text 3\sublime_plugin.py", line 109, in reload_plugin
    m = importlib.import_module(modulename)
  File "./python3.3/importlib/__init__.py", line 90, in import_module
  File "<frozen importlib._bootstrap>", line 1584, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1565, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1532, in _find_and_load_unlocked
  File "C:\Program Files\Sublime Text 3\sublime_plugin.py", line 915, in load_module
    exec(compile(source, source_path, 'exec'), mod.__dict__)
  File "MarkdownPreview in C:\Users\Rick\AppData\Roaming\Sublime Text 3\Installed Packages\Markdown Preview.sublime-package", line 57, in <module>
ImportError: No module named 'pygments.formatters'

It seems that this dependency was not correctly installed.

Restart Sublime and see if there is still an issue.  Sometimes a restart is necessary after installing a plugin.  I can't see enough of the log to see if Pygments actually got installed, but if restarting does not work, you can try to run the command Package Control: Satisfy Dependencies (but watch the console to see when it finishes), and then restart.

It works now. I don't know what happened, because I'm sure I restarted Sublime Text multiple times, as well as reinstalled the plugin and ran Satsify Dependencies, but after doing it again it's OK now. I must have done something wrong. Apologies for taking your time.

Package control can some times get broken which is why I suggested additional steps besides just restarting, but glad it's working now.


Render proposal create error messages

Render error message when the transaction is fails.

We could rework the styling of the balance component, @ianbach @dgamboa would appreciate feedback on that.


High level fee document

tbd

We want some additional intuition for developers on Celestia on how to reason about fees and the fee market. This section is a good start but doesn't build intuitions on things like limits and why the fees are the way they are.

we still have a follow up in https://github.com/celestiaorg/docs/issues/1003
but this can be closed with https://github.com/celestiaorg/docs/pull/1009


More External Search Options

Added search for:
AniDB
FilmCrave
MyAnimeList
TasteDive

@eckomega I'm not seeing any changes in here. Also, please base your changes off of the most recent copy of the master branch of this repo (you may need to pull down from here, and then push up to your fork), as the commits in your PRs are starting to stack up.

@kannibalox It looks like there may have been an issue when overwriting the previous file.
To pull down from here and push up to my fork... do I need to delete the copy of the script that's currently on my fork? Like starting fresh?


Add entrypoint for add-number

MkDocs plugin have the naming convention mkdocs-<name>-plugin but for entrypoint, leave out the "mkdocs-" and "-plugin".
We can add another entrypoint to setup.py, e.g. from:
    entry_points={
        'mkdocs.plugins': [
            'mkdocs-add-number-plugin=mkdocs_add_number_plugin.plugin:AddIndexPlugin'
        ]
    }

to:
    entry_points={
        'mkdocs.plugins': [
            'mkdocs-add-number-plugin=mkdocs_add_number_plugin.plugin:AddIndexPlugin',
            'add-number=mkdocs_add_number_plugin.plugin:AddIndexPlugin'
        ]
    }

And then update the README.md appriopriately.

Oh and while we're at it, we can change the plugin class AddIndexPlugin to AddNumberPlugin, it will help users identify the plugin on any errors

Happy to submit PR for this..

OK, just do as you say.
There is no environment for development because of the outbreak in China. So you can go and realize your idea.And thank you for your contribution to this plugin.

Oh, are you based in Wuhan? Must be crazy out there. I'm from the Netherlands, not too many cases here yet but a lot of attention from the news

No, I'm in other provinces, not a disaster area.Although the outbreak has been brought under control, I have not been allowed to return to work.The virus is very contagious, must wear a mask when you go out.

I haven't seen anyone wearing a mask here, yet. Impressive that the outbreak has been controlled.. huge effort from China!


Use remotes and re-document

Coverage remained the same at 37.092% when pulling f9e0f9c3ed6fa8f3c98a89ed16ab109902399d75 on use_remotes2 into d1901df050a4024c5b8f1524e7021a5c8b16bce0 on 0.6-dev.


Issue with Documentation: kubelet-authentication-authorization

This is a...

[ ] Feature Request
[x] Bug Report

Problem:
https://kubernetes.io/docs/admin/kubelet-authentication-authorization describes default behavior that is the opposite of how the current Kubernetes 1.10 release behaves.

The documentation says ...
The actual 1.10 behavior is ...

By default, requests to the kubelet’s HTTPS endpoint that are not rejected by other configured authentication methods are treated as anonymous requests
Mapping  authenticated requests to user:anonymous is disabled in kubelet 1.10

To disable anonymous access and send 401 Unauthorized responses to unauthenticated requests: start the kubelet with ....
Anonymous authenticated access is disabled by default in 1.10, so no action is necessary.

Start the kubelet with the --anonymous-auth flag
The --anonymous-auth flag doesn't exist in kubelet 1.10.  It's been replaced with the authentication:anonymous:enabled <bool> YAML v1beta1 KubeletConfiguration record, which defaults to false.

To enable X509 client certificate authentication, start kubelet with the --client-ca-file flag
The --client-ca-file flag doesn't exist in kubelet 1.10. It's been replaced with the clientCAFile: <string> YAML  configuration record, which is empty by default.

The default authorization mode is AlwaysAllow, which allows all requests.
It's not AlwaysAllow in kubelet 1.10. The default authorization mode is Webhook in 1.10.  Achieving the documented behavior involves setting the authorization:mode: AlwaysAllow YAML v1beta1 KubeletConfiguration record

Start the kubelet with the --authorization-mode= <string>  flag
The --authorization-mode flag doesn't exist in kubelet 1.10. It's been replaced with the authorization:mode:  YAML v1beta1 KubeletConfiguration record, which is set to "Webhook" by default.

Proposed Solution:
Correct the documentation:

Replace the above CLI flags with their new corresponding YAML keys and values
Describe the new default behavior:

Authenticated access not rejected by other configured authentication methods will be denied and a "401 Unauthorized" result sent
Subdivide access to the kubelet API via Webhooks is now the default, therefore concise documentation should be provided describing how exactly webhooks (both api-server-side and kubelet client-side) can be configured such that kubectl actions like exec and logs actually succeed.

Page to Update:
https://kubernetes.io/docs/admin/kubelet-authentication-authorization

Kubernetes Version: 1.10

if you specify a config file, the defaults are different (and more secure), but if you simply invoke kubelet, all of the documented behavior still holds (--anonymous-auth=true, --authorization-mode=AlwaysAllow, etc)

cc @mtaufen for confusion from disappearing deprecated flags

closing this for now... we're not going to remove the description of the insecure defaults until running with a config is required and the defaults are actually secure
/close

if you simply invoke kubelet, all of the documented behavior still holds (--anonymous-auth=true, --authorization-mode=AlwaysAllow, etc)

Those flags don't exist in the current 1.10, release.  They did in prior releases, but they've been removed, therefore the documentation currently describes flags that don't even exist.

Those flags don't exist in the current 1.10, release. They did in prior releases, but they've been removed, therefore the documentation currently describes flags that don't even exist.

They do exist, and are functional, but were (unfortunately) masked from the flag help because they are marked as deprecated in favor of starting with a config file.
This was resolved in https://github.com/kubernetes/kubernetes/pull/63448 and the flag help in 1.10.4 now displays properly again.

Yes, I encountered this straight away on the 1.10.0 release.
Glad it was fixed in 1.10.4!


Fix search icon on search input

When edit --pico-form-element-spacing-horizontal
Exemple (set to 1px) :
From :

To :

@benizz0, thank you for spotting it and for the PR.
Note that you did not generate all the CSS files.
I can do it, but if you want to do it, here's how to do it:

Install the dependencies: yarn
Lint all the files, build and minify all the CSS files: yarn build.

You should end up with 233 changed files 🙂

Hello @lucaslarroche,
it's done !

Thanks. Merged, it will be released this weekend with v2.0.1


Add vcpkg installation instructions

Nana is available as a port in vcpkg, a C++ library manager that simplifies installation for nana and other project dependencies. Documenting the install process here will help users get started by providing a single set of commands to build nana, ready to be included in their projects.
We also test whether our library ports build in various configurations (dynamic, static) on various platforms (Windows: x86, x64) to keep a wide coverage for users.
I'm a maintainer for vcpkg, and here is what the port script looks like. We try to keep the library maintained as close as possible to the original library.
I'm trying to add vcpkg installation instructions but I couldn't find the related docs in code item except for on Wiki page.
Could you please help add the following contents to this port?
Installing and building nana via vcpkg
You can download and install raylib using the vcpkg dependency manager:
  git clone https://github.com/Microsoft/vcpkg.git
  cd vcpkg
  ./bootstrap-vcpkg.sh
  ./vcpkg integrate install
  vcpkg install nana

The nana port in vcpkg is kept up to date by Microsoft team members and community contributors. If the version is out of date, please create an issue or pull request on the vcpkg repository.
Thanks,
Nancy

On obtainng and building Nana as a beginner to Nana:
Thank you for vcpkg instructions. I would like to share my experience so contributors can help make this better.
Visual Studio Community 2019
Windows 10 x64
Nana 1.7.4 (current version at the time of this post)
There two methods that I attempted to build Nana locally on my computer. One worked very well, the other has some difficulty building the libaray from source.
In nana/build/vc2019, the Visual Studio project is made ready for us. Thank you for that. It is great. In the build configurations, we can freely select the target platform x84, x64, ARM., etc. Thank you again for having it arranged for us, too.
Cloning Nana from Github repo, This method has error when VS2019 "build".
Downloading from nanapro.org main page, upper right "DOWNLOAD" we get a .zip file. After unzipping, open this VS2019 project into Visual Studio, "build" worked beautifully. In nana/build/bin I have nana_v142_Release_x64.lib. Wouldn't that be v174?
Of these methods obtaining Nana, the vcpkg method would appear a good method to try.


bump rand and num-traits

Bump num-traits to 0.2, and rand to 0.4. No changes needed.
Additionally, force num-complex to the minimum version needed for new num-traits, so that nobody picks a different version.
Fixes #30.

Thanks!


implement empty listener for registerGDPRConsent hook

Questions
Answers

Description?
From now on registered hooks must have a corresponding hook listener, the registerGDPRConsent is a special case, we implement an empty listener.

Type?
improvement

BC breaks?
no

Deprecations?
no

Fixed ticket?
Partly fixes #19230

How to test?
Check that the module can still be installed without error

Thanks @matthieu-rolland


Admin text-models limit is wrong if Free tier is used

The code is hard-coded based on the workspaces limit of the Standard tier

There is an explanation of why the limits are wrong here
https://github.com/IBM/taxinomitis/blob/0f30eed534d6fb0a97caffd3ab120e8a3df8e9ec/public/components/help/help.html#L343-L387

The place where it is hard-coded is here:
https://github.com/IBM/taxinomitis/blob/0f30eed534d6fb0a97caffd3ab120e8a3df8e9ec/src/lib/restapi/users.ts#L203-L204
The number of Watson Assistant credentials is multiplied by 20 (because each Standard tier service instance supports 20 workspaces)
The number of Visual Recognition credentials is multiplied by 1 (because each Free tier service instance supported 1 classifier)
(Although, newly created credentials for VR now allow 2 classifiers, so that's wrong for new classes)

I don't think there is an API that would let us query the tier / models-limit dynamically, so I suspect the only way to fix this is to start storing the tier with the service credentials
https://github.com/IBM/taxinomitis/blob/0f30eed534d6fb0a97caffd3ab120e8a3df8e9ec/src/lib/training/training-types.ts#L5-L14


WaveNet as a Virtual instrument

Hi Guys,
I'm currently working to adapt this project to work as a virtual instrument using
midi vectors as local conditioning as outlined here:
https://arxiv.org/abs/1806.09905
https://arxiv.org/abs/1810.12247
I'm using Solo Cello recordings from the musicNet dataset
and have written a pre-processing script to generate the conditioning vectors.
I have run into a couple issues getting the wavenet too train. I am at 20K steps
with an avg loss of 5.5 (tensorflow log attached) and am getting noise.
I'm worried that this is due to the fact the data is only split across 10 files of about 5 mins log each so it goes through epochs very quickly, this is certainly slowing down training considerably.
the audio printed to checkpoints\audio\stepXXXX_predicted seems to sound good though. am I missing something here?
I'm doing this for a piece of university research, however, the deadline is in a few weeks so i would be very grateful if anyone has any comments on this.

here are my hparams
name="wavenet_vocoder",
# Convenient model builder
builder="wavenet",

# Input type:
# 1. raw [-1, 1]
# 2. mulaw [-1, 1]
# 3. mulaw-quantize [0, mu]
# If input_type is raw or mulaw, network assumes scalar input and
# discretized mixture of logistic distributions output, otherwise one-hot
# input and softmax output are assumed.
# **NOTE**: if you change the one of the two parameters below, you need to
# re-run preprocessing before training.
input_type="raw",
quantize_channels=65536,  # 65536 or 256

# Audio:
sample_rate=44100,
# this is only valid for mulaw is True
silence_threshold=2,
num_mels=127,
fmin=125,
fmax=7600,
fft_size=1024,
# shift can be specified by either hop_size or frame_shift_ms
hop_size=275,
frame_shift_ms=None,
min_level_db=-100,
ref_level_db=20,
# whether to rescale waveform or not.
# Let x is an input waveform, rescaled waveform y is given by:
# y = x / np.abs(x).max() * rescaling_max
rescaling=True,
rescaling_max=0.999,
# mel-spectrogram is normalized to [0, 1] for each utterance and clipping may
# happen depends on min_level_db and ref_level_db, causing clipping noise.
# If False, assertion is added to ensure no clipping happens.o0
allow_clipping_in_normalization=True,

# Mixture of logistic distributions:
log_scale_min=float(np.log(1e-14)),

# Model:
# This should equal to `quantize_channels` if mu-law quantize enabled
# otherwise num_mixture * 3 (pi, mean, log_scale)
out_channels=10 * 3,
layers=24,
stacks=4,
residual_channels=512,
gate_channels=512,  # split into 2 gropus internally for gated activation
skip_out_channels=256,
dropout=1 - 0.95,
kernel_size=3,
# If True, apply weight normalization as same as DeepVoice3
weight_normalization=True,
# Use legacy code or not. Default is True since we already provided a model
# based on the legacy code that can generate high-quality audio.
# Ref: https://github.com/r9y9/wavenet_vocoder/pull/73
legacy=True,

# Local conditioning (set negative value to disable))
cin_channels=127,
# If True, use transposed convolutions to upsample conditional features,
# otherwise repeat features to adjust time resolution
upsample_conditional_features=True,
# should np.prod(upsample_scales) == hop_size
upsample_scales=[11, 25],
# Freq axis kernel size for upsampling network
freq_axis_kernel_size=3,

# Global conditioning (set negative value to disable)
# currently limited for speaker embedding
# this should only be enabled for multi-speaker dataset
gin_channels=-1,  # i.e., speaker embedding dim
n_speakers=7,  # 7 for CMU ARCTIC

# Data loader
pin_memory=False,
num_workers=2,

# train/test
# test size can be specified as portion or num samples
test_size=0.0441,  # 50 for CMU ARCTIC single speaker
test_num_samples=None,
random_state=1234,

# Loss

# Training:
batch_size=1,
adam_beta1=0.9,
adam_beta2=0.999,
adam_eps=1e-8,
amsgrad=False,
initial_learning_rate=1e-3,
# see lrschedule.py for available lr_schedule
lr_schedule="noam_learning_rate_decay",
lr_schedule_kwargs={},  # {"anneal_rate": 0.5, "anneal_interval": 50000},
nepochs=2000,
weight_decay=0.0,
clip_thresh=-1,
# max time steps can either be specified as sec or steps
# if both are None, then full audio samples are used in a batch
max_time_sec=None,
max_time_steps=8000,
# Hold moving averaged parameters and use them for evaluation
exponential_moving_average=True,
# averaged = decay * averaged + (1 - decay) * x
ema_decay=0.9999,

# Save
# per-step intervals
checkpoint_interval=10000,
train_eval_interval=10000,
# per-epoch interval
test_eval_epoch_interval=100,
save_optimizer_state=True,


Fix a malformed __all__

Description
As it says on the tin.
Status

[x] Ready to go

I looked over the rest of the __all__'s, they appear to be correct. Some of them seem unnecessary, but likely wont harm anything (e.g. schema_getters) unless the point was to mask the other things the schema_getters.py file imports.

Codecov Report

Merging #45 into master will not change coverage.
The diff coverage is 100%.

This pull request fixes 1 alert when merging 3d30a59230c35a8cb1a472ac7e26779480827cd0 into 3ec89035affd4e0b77bde772ac52514f3b297b90 - view on LGTM.com
fixed alerts:

1 for Explicit export is not defined

Comment posted by LGTM.com


makemkv: 1.14.0 -> 1.14.1

Motivation for this change
Previous URL is unreachable
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

@c0bw3b I just pushed a fix

@nyanloutre thank you for this


Wrong Helm repository name

Hi
Thanks again Razvan.
I noticed that there are pages with different repo names when using helm. For instance:

https://docs.stackable.tech/home/getting_started.html:

helm repo add stackable-stable https://repo.stackable.tech/repository/helm-stable/

https://docs.stackable.tech/druid/stable/installation.html

$ helm repo add stackable https://repo.stackable.tech/repository/helm-stable/

As you can see, the name is different leading to the following error when you try to install something: Error: INSTALLATION FAILED: failed to download "stackable/druid-operator"

This has been mostly fixed already by the looks of it.
I only found references in the release notes to a repo called "stackable" and I'll submit a PR


Refactor of the KCFGViewer

Going in a more step-by-step fashion compared to https://github.com/runtimeverification/pyk/pull/460
Rolling out our own key bindings was messing up with the defaults of textual and adding a lot of unecessary boilerpalte. This PR also closes it.

Dedicated classes for each widget in the NodeView (Term, Constraint, Info, Custom)
Relevant ones all inherits from ScrollableContainer and can be focused. Adds native key bindings and can be selected using Tab or Tab+Shift to loop the other way around
using an absolute path for the css file so that the class APRProofViewer inherits KCFGViewer's css. @import is not something that works with textual, but as we may want in the future to add stuff only for the APRProofViewer css.

if we want to add a css file specific to the APRProofViewer, we could use https://textual.textualize.io/guide/CSS/#using-multiple-css-files

I'm having a hard time uploading a GIF that fits the size limit

I assume you tested that hiding/unhiding individual views works? Asknig because it looks like some container IDs changed, and so we need to make sure it's still able to apply the hiddent attribute.

Yes @ehildenb that does work!


When selecting background image and selecting a non JPG/PNG type, it is selected anyway.

Repro steps

Select a png
Now change it to a txt
Press OK on the warning

Expected Result
The png is still listed in the input
Actual Result
The txt is selected
System Info

https://github.com/threefoldtech/freeflowconnect_frontend/pull/52

Still happens

https://github.com/threefoldtech/freeflowconnect_frontend/pull/71


[AutoTVM][Fix] Fix wrong axis names of data_vec

This PR is trying to fix the wrong axis names of data_vec. As the data_vec is nchwc format, the axis names should be batch, ic_chunk, ih, iw, ic_block, but not batch, ic_chunk, ih, ic_block, iw.
Although the following code does not use these last two axises, so it does not cause some bugs for now. But I think we should fix this.

cc: @yzhliu, @tkonolige

@NaNAGISaSA have to tested performance before and after this change? The change in names is correct but I wonder if you need to change the reorder code below it too.

Hi, @tkonolige , thank you for your review.
I think we do not need to change the reorder code below, here is the benchmark result and code analysis:
Benchmark

Model: ResNet50 With Quantization
Backend: Intel(R) Core(TM) i7-9700 CPU @ 3.00GHz
Threads: 4
Benchmark Before Change:

Execution time summary:
 mean (ms)   median (ms)    max (ms)     min (ms)     std (ms)
  586.5478     586.6723     587.4650     585.2788      0.7180

Benchmark After Change:

Execution time summary:
 mean (ms)   median (ms)    max (ms)     min (ms)     std (ms)
  584.4176     585.1300     586.2465     580.8972      1.8741

So we can see that the benchmark result for ResNet50 before and after changing the code is equal.
Code Analysis
In the context of the changes, for iw variable, it is not used in the following code in its function. And for ic_block variable, it is reassigned immediately with the ic_block axis for kernel_vec in the following line 170 and 308, so this change will also not influence the following code.

The benchmark result is acquired with module.benchmark(dev, number=100, repeat=5)


Refs #158 - don't hide selected columns

Fixes #158
If this change is accepted, it can be really usefull for our DBAs, who are reviewing our pull requests which contain updates on perf rec files :)

Hi
The reason the selected columns are collapsed is because they change frequently in applications when migrations add/remove columns. Also most DB's that Django natively supports are row-based in their default configurations, so there is not much of a performance penalty for selecting extra columns, except from certain cases such as one column being quite large and thus consuming extra network bandwidth. Have your DBA's given any reasoning behind wanting to see them?
I also think this change isn't great for 99% of users, who probably don't have humans inspecting the column list and appreciate the lower churn and more compact output when columns are added/removed from tables. So it would have to be a default-off option at least if were to add it.

Hi, thank you for your feedback.
You're right, I will add a default-off option for this. I thought about it last week, and I forgot ... :)

Have your DBA's given any reasoning behind wanting to see them?
I asked to one of my colleague to answer. He should respond soon.

Yes, no problem. I planed to extend the defaults:
    defaults = {
        'MODE': 'once',
        'HIDE_COLUMNS': True,
    }

(hide_columns True or show_columns False ? What do you prefer ?)

Oh, I guess HIDE_COLUMNS defaulting to True makes sense?

Ok for me, thanks :)

Hi, I'm the requested DBA :)
I'll only speak about PostgreSQL and Oracle, to be sure I don't say anything I'm not sure of, but what follows is probably valid also at least with SQL Server and DB2. And focus on some very obvious queries for the sake of example, but there are many more complicated occurrences where having the columns will help me. The examples will be with PostgreSQL, but Oracle is very similar for these cases.
There are many reasons why the columns returned are important, but here are the firsts that come to my mind:

If all the requested columns are in an index, there is no need to go check the table (that's called an index only scan). For instance, this query changes completely:

create table test (a int, b varchar, c varchar);
insert into test select (random()*1000000)::int,i::text, repeat (i::text,30) from generate_series(1,1000000) g(i);
create index idx_test on test(a,b);
vacuum analyze test;

=# explain (analyze,buffers) select a,b,c from test where a < 100000;
                                                          QUERY PLAN
-------------------------------------------------------------------------------------------------------------------------------
 Bitmap Heap Scan on test  (cost=2314.84..31673.91 rows=99925 width=190) (actual time=65.428..109.311 rows=100283 loops=1)
   Recheck Cond: (a < 100000)
   Heap Blocks: exact=27483
   Buffers: shared hit=27870
   ->  Bitmap Index Scan on idx_test  (cost=0.00..2289.86 rows=99925 width=0) (actual time=57.979..57.979 rows=100283 loops=1)
         Index Cond: (a < 100000)
         Buffers: shared hit=387
 Planning time: 0.247 ms
 Execution time: 114.687 ms
(9 rows)

=# explain (analyze,buffers) select a,b from test where a < 100000;
                                                            QUERY PLAN
----------------------------------------------------------------------------------------------------------------------------------
 Index Only Scan using idx_test on test  (cost=0.42..3289.11 rows=99925 width=10) (actual time=0.092..42.901 rows=100283 loops=1)
   Index Cond: (a < 100000)
   Heap Fetches: 0
   Buffers: shared hit=388
 Planning time: 0.256 ms
 Execution time: 56.436 ms
(6 rows)

The difference in runtime isn't that great here, because it all fits in the RAM. But the first query accesses 100 times more blocks, as it has to visit the table in a random order instead of only scanning a well ordered index. On large data sets, performance can vary several orders of magnitude.

Some data may not be inlined in the table, even if it is a row-based database. Oracle and Postgresql sometimes store data out of line, if it is too big to fit in a block. A big text field for instance, or a bytea.

create table test (a int, b varchar);
insert into test
-- create a big garbage column ... it has to be quite big, or, as PG will compress it, it may still fit
select (random()*1000000)::int,
           array_to_string(array ( select
                  substr('abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789',
                              trunc(random() * 62)::integer + 1, 1)
FROM   generate_series(1, 1000000)), '') from generate_series(1,1000) g(i);
Time: 1,788 ms
=# explain (analyze,buffers) select a,b||'a' from test ;
                                               QUERY PLAN
--------------------------------------------------------------------------------------------------------
 Seq Scan on test  (cost=0.00..19.50 rows=1000 width=36) (actual time=1.624..396.937 rows=1000 loops=1)
   Buffers: shared hit=130383
 Planning time: 0.098 ms
 Execution time: 397.124 ms
(4 rows)

Time: 397,731 ms
=# explain (analyze,buffers) select a from test ;
                                             QUERY PLAN
-----------------------------------------------------------------------------------------------------
 Seq Scan on test  (cost=0.00..17.00 rows=1000 width=4) (actual time=0.044..0.851 rows=1000 loops=1)
   Buffers: shared hit=7
 Planning time: 0.096 ms
 Execution time: 1.238 ms
(4 rows)

Time: 2,126 ms

I had to cheat a bit on this second one: as explain analyze doesn't return the row, PG is smart and wont access b if I don't force it to do something on b. If I did return the data to a client, PG would have accessed b, like in this example (see the difference in block hits)

Another one that comes to my mind is ordering/hashing: memory costs are very different depending on the selected colums:

create table test (a int, b varchar);
insert into test select i,repeat((i::text),10) from generate_series(1,100000) g(i);
marc=# explain analyze select a,b from test order by a;
                                                    QUERY PLAN
-------------------------------------------------------------------------------------------------------------------
 Sort  (cost=10325.82..10575.82 rows=100000 width=53) (actual time=167.657..194.548 rows=100000 loops=1)
   Sort Key: a
   Sort Method: external sort  Disk: 6240kB
   ->  Seq Scan on test  (cost=0.00..2021.00 rows=100000 width=53) (actual time=0.050..29.298 rows=100000 loops=1)
 Planning time: 0.206 ms
 Execution time: 199.617 ms
(6 rows)

Time: 200,561 ms
marc=# explain analyze select a from test order by a;
                                                    QUERY PLAN
------------------------------------------------------------------------------------------------------------------
 Sort  (cost=10325.82..10575.82 rows=100000 width=4) (actual time=72.151..81.108 rows=100000 loops=1)
   Sort Key: a
   Sort Method: quicksort  Memory: 7760kB
   ->  Seq Scan on test  (cost=0.00..2021.00 rows=100000 width=4) (actual time=0.041..33.913 rows=100000 loops=1)
 Planning time: 0.136 ms
 Execution time: 84.537 ms
(6 rows)

Time: 85,161 ms

There i chose a work_mem (the memory used for sorts) so that the change is obvious... there is more to sort when you ask for all the columns, so here it has to sort on disk. The same applies for hashes used for aggregates or joins for instance.

Fair enough. But in my experience optimizers work very differently in production compared to test environments, as the dataset is much bigger. Any instrumentation you can get from production is tends to be worth 10x.
django-perf-rec is mostly intended to catch more obvious problems, like N+1 queries and unintended joins, rather than such relatively nuanced performance problems. It can be hard to figure out how many queries the Django ORM will even execute, let alone which tables they will touch :)
Still I will accept this feature when it's optional, default-off.

The feature is optional, default-off (or default-on, depending on the wording :) )
https://github.com/adamchainz/django-perf-rec/pull/159/files#diff-c3a36757e150957275f1de9b38bd100cR11

Hey @zebuline , sorry this took some time to come back to. I've merged and released in 3.1.0: https://pypi.org/project/django-perf-rec/ . See #169 for some post-commit tidy up I did, instead of asking you for those changes through review. I wrote a detailed description there to walk through the changes I thought were necessary, hope it helps as educational reading. Thanks for your contribution, hope you can make more here or on other projects in the future - don't be discouraged by my slow response time here!


CE-NONE: various code improvements

This is a bunch of small changes I implemented while reading the code in preparation for CE-6909.

Thanks for the feedback! I addressed all the comments in the recent commits.


Invoke-PaginateSearch retrying on bad requests

I imagine the retry functionality is configurable and will look into it. I'd expect 400's to not retry... as the results won't change.
The following example produces the debug output that follows:
`$DebugPreference = 'Continue'
try {
$Search = @{
	queryType = "SAILPOINT"
	indices = @('identities')
	query = @{
		query = 'pat.smith'
	}
	sort = @(
		'id'
	)
}

$Search | ConvertTo-Json | Write-Output

Invoke-PaginateSearch -Increment 50 -Limit 100 -Search $Search -Verbose

} catch {
# Write-Host ("Exception occurred when calling Invoke-ListAccounts: {0}" -f ($.ErrorDetails | ConvertFrom-Json))
Write-Host ("Exception occurred when calling Search-Post:" + $.Exception)
Write-Host ("Response headers: {0}" -f ($_.Exception.Response.Headers | ConvertTo-Json))
}`
DEBUG: Getting Access Token
DEBUG: https://company2825-poc.api.identitynow-demo.com/oauth/token
DEBUG:
DEBUG:
DEBUG: Token Exp: 03/12/2023 22:07:28
DEBUG: Token is valid
VERBOSE: HTTP/1.1 POST with 404-byte payload
VERBOSE: Retrying after interval of 5 seconds. Status code for previous attempt: BadRequest
VERBOSE: Retrying after interval of 5 seconds. Status code for previous attempt: BadRequest
VERBOSE: Retrying after interval of 5 seconds. Status code for previous attempt: BadRequest
VERBOSE: Retrying after interval of 5 seconds. Status code for previous attempt: BadRequest
VERBOSE: Retrying after interval of 5 seconds. Status code for previous attempt: BadRequest
VERBOSE: Retrying after interval of 5 seconds. Status code for previous attempt: BadRequest
VERBOSE: Retrying after interval of 5 seconds. Status code for previous attempt: BadRequest
VERBOSE: Retrying after interval of 5 seconds. Status code for previous attempt: BadRequest
VERBOSE: Retrying after interval of 5 seconds. Status code for previous attempt: BadRequest
VERBOSE: Retrying after interval of 5 seconds. Status code for previous attempt: BadRequest
{"detailCode":"400.1.0 Required data missing or empty","trackingId":"7e8b3213dbb441208d62f9ff6557e135","messages":[{"locale":"und","localeOrigin":"REQUEST","text":"At least one of the fields "query, aggregations" must be specified."},{"locale":"en-US","localeOrigin":"DEFAULT","text":"At least one of the fields "query, aggregations" must be specified."}],"causes":[]}
Exception occurred when calling : @{detailCode=400.1.0 Required data missing or empty; trackingId=7e8b3213dbb441208d62f9ff6557e135; messages=System.Object[]; causes=System.Object[]}
Response headers: [
{
"Key": "Date",
"Value": [
"Sun, 12 Mar 2023 15:40:48 GMT"
]
},
{
"Key": "Connection",
"Value": [
"keep-alive"
]
},
{
"Key": "Server",
"Value": [
"nginx"
]
},
{
"Key": "Vary",
"Value": [
"Origin",
"Access-Control-Request-Method",
"Access-Control-Request-Headers"
]
},
{
"Key": "SLPT-Request-ID",
"Value": [
"7e8b3213dbb441208d62f9ff6557e135"
]
},
{
"Key": "Access-Control-Expose-Headers",
"Value": [
"SLPT-Request-ID",
"Content-Type"
]
},
{
"Key": "X-Robots-Tag",
"Value": [
"noindex"
]
}

Hello @rob-buskens-sp,
The library that our SDK generator is using relies on the Invoke-WebRequest Cmdlet. We are taking advantage of the built in retry logic with the two variables to configure retries, MaximumRetryCount and RetryIntervalSec. In looking closer at the functionality it appears that the Invoke-WebRequest Cmdlet will retry on status code 304 and status codes between 400-599.
https://github.com/PowerShell/PowerShell/blob/master/src/Microsoft.PowerShell.Commands.Utility/commands/utility/WebCmdlet/Common/WebRequestPSCmdlet.Common.cs#L1828

Tyler,
Fair enough, I find it curious though. Why retry things that won’t work ever? (rhetorical).
Bad requests would mostly be caught in testing. TCP has data integrity checks
Seems inefficient.
Not particularly configurable.
Good enough, I understand the behavior.
Thanks.
Rob Buskens
Solution Architect
SailPoint Certified IdentityNow Professional
@.*** @.***>
Join the #SailPointCrew https://www.sailpoint.com/company/careers/
[https://i.xink.io/Images/Get/N9791/s481.png]https://www.sailpoint.com/
From: Tyler Mairose @.>
Date: Wednesday, March 22, 2023 at 12:09 PM
To: sailpoint-oss/powershell-sdk @.>
Cc: Rob Buskens @.>, Mention @.>
Subject: Re: [sailpoint-oss/powershell-sdk] Invoke-PaginateSearch retrying on bad requests (Issue #5)
Hello @rob-buskens-sp, The library that our SDK generator is using relies on the Invoke-WebRequest Cmdlet. We are taking advantage of the built in retry logic with the two variables to configure retries, MaximumRetryCount and RetryIntervalSec.
ZjQcmQRYFpfptBannerStart
This Message Is From an External Sender
This message came from outside your organization.
ZjQcmQRYFpfptBannerEnd
Hello @rob-buskens-sphttps://urldefense.com/v3/__https:/github.com/rob-buskens-sp__;!!MsNKLpFGsw!LP2BOaM02IudbIP4OkZB_59Z1pvRgTaoMFhTEyxnL8zhNyCCt6Dgl3bwj4O897vASkILWQzMzj-6hjgEpTPRGH9bD7yM$,
The library that our SDK generator is using relies on the Invoke-WebRequest Cmdlet. We are taking advantage of the built in retry logic with the two variables to configure retries, MaximumRetryCount and RetryIntervalSec. In looking closer at the functionality it appears that the Invoke-WebRequest Cmdlet will retry on status code 304 and status codes between 400-599.
https://github.com/PowerShell/PowerShell/blob/master/src/Microsoft.PowerShell.Commands.Utility/commands/utility/WebCmdlet/Common/WebRequestPSCmdlet.Common.cs#L1828https://urldefense.com/v3/__https:/github.com/PowerShell/PowerShell/blob/master/src/Microsoft.PowerShell.Commands.Utility/commands/utility/WebCmdlet/Common/WebRequestPSCmdlet.Common.cs*L1828__;Iw!!MsNKLpFGsw!LP2BOaM02IudbIP4OkZB_59Z1pvRgTaoMFhTEyxnL8zhNyCCt6Dgl3bwj4O897vASkILWQzMzj-6hjgEpTPRGBYS2bX8$
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/sailpoint-oss/powershell-sdk/issues/5*issuecomment-1479952537__;Iw!!MsNKLpFGsw!LP2BOaM02IudbIP4OkZB_59Z1pvRgTaoMFhTEyxnL8zhNyCCt6Dgl3bwj4O897vASkILWQzMzj-6hjgEpTPRGKhYf4_E$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/AZH2GPNV6F23CLHNDRKVM4LW5MW3RANCNFSM6AAAAAAVYF4N6E__;!!MsNKLpFGsw!LP2BOaM02IudbIP4OkZB_59Z1pvRgTaoMFhTEyxnL8zhNyCCt6Dgl3bwj4O897vASkILWQzMzj-6hjgEpTPRGKZhu_ts$.
You are receiving this because you were mentioned.Message ID: @.***>

Closed as won't do. See https://github.com/sailpoint-oss/powershell-sdk/issues/5#issuecomment-1479952537


cli: add command to list usage of deprecated APIs

Hey, I just made a Pull Request!
Making this a separate command rather than a built-in lint rule. It would significantly slow down lint execution, since it brings in TypeScript compilation.
The purpose of this command is to run it occasionally in order to detect usages of deprecated APIs. In case one wants to hook it up to some form of automation there's also a --json option that switches the output to JSON.
I added it to the repo group of commands for now, since it scans the entire repo. May be that we want to put it somewhere else though. Potentially some form of housekeeping group together with #10138. Output looks something like this:

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Oh wow so running this after versions:bump might actually be really valuable huh

Oh the json thing made me think. Could be that this class of command should have a fail-on-errors flag. Such that if no depreciation were found it prints nothing (or just a short ok message unless in silent mode) and returns 0, while if it finds depreciations it can use stderr and return an error code. Good for scripts and CI, without having to parse the output json

Just made it exit if a code in there are any deprecations, because why not? :grin: Then up to the caller if they want to care about that or not. If the output isn't important you can > /dev/null, although tbh you'd probably always want to print that too


Last Evaluation label is not visible in case of too many evaluations in the chart

Environment

Client OS (e.g., Linux, macOS, Windows):
Keptn Version (keptn version):
Kubernetes Cloud Provider (e.g., GKE, AKS):
Kubernetes version (kubectl version):

Affected Component

Describe the bug
There are 50 number of evaluations generated on my Keptn instance. In this case the label of last generated evaluation is not visible (see screenshot.)
To Reproduce
Steps to reproduce the behavior:

Generate evaluations via API 50 or more
open Evaluations chart and take a look to the label of the last evaluation

Expected behavior
The label of last evaluation should be displayed successfully.
Current behavior
The label of last evaluation is not displayed.
Screenshots

Looks fine to me.

When you reduce the width of the window every nth label is shown, which is also fine I think.

But I fixed a small bug I encountered that is trying to read something of undefined. The heatmap is therefore not loading properly.

I understand that the chart shows the label only for every nth result.
Question: Would it be better if we show the same way but started from the end and not the beginning? Which means the last result would always have a label.
I think this is important since customer would be interested more often in the last result and the label accordingly would be needed.
This is also important for e2e automation tests.

@Gevorgisk I filed a new issue regarding the change: https://github.com/keptn/keptn/issues/3943


did a bunch of minor changes

basically rearranged some things and changed the visibility of certain attributes so the UML isn't completely cluttered

bruh I removed this because I thought someone merged it already


TTransportException from our platform

Hi..
I have a big problem.
After my cassandra and application(platform) start, running on good condition.
but four or five days later, my application has the problem from random TTransportException. (see below)
Case 1
[WARN]     : Skip-host failed      [FailoverOperator.java.operate():104]
Caused by: java.net.ConnectException: Connection refused
at java.net.PlainSocketImpl.socketConnect(Native Method)
at java.net.PlainSocketImpl.doConnect(PlainSocketImpl.java:333)
at java.net.PlainSocketImpl.connectToAddress(PlainSocketImpl.java:195)
at java.net.PlainSocketImpl.connect(PlainSocketImpl.java:182)
at java.net.SocksSocketImpl.connect(SocksSocketImpl.java:366)
at java.net.Socket.connect(Socket.java:529)
at java.net.Socket.connect(Socket.java:478)
at org.apache.thrift.transport.TSocket.open(TSocket.java:180)
Case 2
[WARN] 	   : org.apache.thrift.transport.TTransportException: java.net.ConnectException: Connection timed out
Case 3
[WARN]     : Got a HectorTException from <IP_ADDRESS>:45201. Num of retries: 0; message: org.apache.thrift.transport.TTransportException: Cannot write to null outputStream; cause: org.apache.thrift.transport.TTransportException: Cannot write to null outputStream (thread=TP-Processor544)  [FailoverOperator.java.operateSingleIteration():217]
this is our condition.
Cassandra version : 0.6.3
Hector version : 0.6.0-18
Cassandra cluster : has 2 node
Client Server (HTTP Server) : has 3
Traffic :
RESPONSE-STAGE : 532062 per hour
ROW-READ-STAGE  : 492055 per hour
Cassandra configuration :
2
org.apache.cassandra.locator.RackUnawareStrategy
Consistency Level : QUOTUM
I heared about this the epic race condition problem. and this problem will be fixed on 0.7.x.
but We cannot upgrade right now. (because 0.7.x is not stable, and upgrade issue)
Thank you for your corporating.^^

It is not answer！


Update factors.C

Comma and Semicolon missing issue addressed

@kaustubhgupta, please review and merge

Thanks for your contribution!