Creating a circuit of 1 element crashes "ortools" and "gurobi"

solver: "ortools" and "gurobi"
CPMpy: version v0.9.9 commit e79b3afedc934a9437c2ddb3a9f54d7e2d7bd3ee
minizinc-python release version 0.7.0 commit {https://github.com/MiniZinc/minizinc-python/commit/a195cf63fcfbc98665d70ab64efb5424db25bd7e)
Creating a circuit of 1 element crashes both "ortools" and "gurobi".
from cpmpy import *
x = intvar(0, 1, name="x")
m=Model()
m += Circuit([x])
m.solve(solver="ortools") # same with "gurobi"

expected behavior:
No crash like other solvers ("minizinc:ortools" ...).
bug found while working on my master thesis.
edit: wrong solvers in line 1

The bug appears when the "decompose" method of the expression is called, in _post_constraint(). The decompose method is called only in ortools and gurobi and that is why this happens there.
In the decompose method of Circuit, it is assumed that it contains more than one variables.
So, "order" is assumed to be an array of variables
       order = intvar(0,n-1, shape=n)
and then an index is used to define that the last one has no succesor
order[n-1] == 0
However, when only 1 variable is given, then n = 1, resulting in "order" not being subscriptable.
This seems an easy fix, just returning nothing when it contains only 1 (or even 0?) variable, skipping the decomposition.

Has been solved.


Why is persistenceavailable set to false

may need to be able to enable persistence. sorry if this was done for good reason, I'm new to redis.

That is to keep tests separate - if you want persistence we can create a setting for it?


⚠ Bofrost.at has degraded performance

In cd8e147, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 998 ms

Resolved: Bofrost.at performance has improved in 75385c2.


possible to rewrite in rustlang?

possible to rewrite in rustlang?

Sorry, unfortunately I will not have the time to write a Rust implementation of it.


Fixed the build on our jenkins

@bshaffer Thanks. I needed to migrate the e2e test into the cloudbuild step. PTAL

Alright, jenkins build is green now :)

@bshaffer Now we verify the composer installer. PTAL

The last failure was caused because the cloudbuild hit the GCS rate limitting for log injestion.
It happens when we run multiple builds (on circleci, and jenkins at the same time, etc)
Workarounds are

to lower log verbosity
to use different projects on different CI


Fix filename to feature gate: KubeletPodResourcesDynamicResources

Rename the file from
*-dynamice-*

to
*-dynamic-*

Also add some minor changes in text

Thanks for the fix. What I'd suggest is that we may want to rename all feature gate files based on the names of the gates. There is no reason to do this translation. For example, the gate named 'IPTablesOwnershipCleanup', then the file should be named as IPTablesOwnershipCleanup.md rather than ip-tables-ownership-cleanup.md. The current naming is as ridiculous as ip-tables-owner-ship-clean-up.md. We all know that iptables is the name of a technology, we should not split the word into ip-tables.

Sounds resonable. However, this would significantly impact more than 300 feature-gate files, potentially involving some cross-references. Adding localized pages would further double or even triple the workload.
We can open a new issue and resolve it gradullay.

Thanks for the fix. What I'd suggest is that we may want to rename all feature gate files based on the names of the gates. There is no reason to do this translation. For example, the gate named 'IPTablesOwnershipCleanup', then the file should be named as IPTablesOwnershipCleanup.md rather than ip-tables-ownership-cleanup.md. The current naming is as ridiculous as ip-tables-owner-ship-clean-up.md. We all know that iptables is the name of a technology, we should not split the word into ip-tables.

I'd rename ip-tables-ownership-cleanup.md to iptables-ownership-cleanup.md. The kebab case filenames match the way we name files on the rest of the site and folks seem to find them pretty easy to work with; I've only heard one objection.
If we want to lint - or enforce - that the filename is equivalent to the feature gate, we can; we don't have to have the exact same case convention. Computers can do a check even when the case doesn't match.

@windsonsea Whenever you get a chance, could you please take a look at the reviewers' feedback https://github.com/kubernetes/website/pull/46488#discussion_r1632538793 and respond to them? Thanks!


[Feature Request] Spinning visualizer and beatmap logo

It might be nice if we could have a visualizer and logo that can spin slowly like below:

it would be cool if they could be configured independently of each other
e.g. https://www.youtube.com/watch?v=QtDbzbQTLHQ


Update jaeger yaml to include the jaeger-agent svc

Per slack discussions and https://github.com/istio/istio/issues/18658.

@douglas-reid: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_installer
642a96b108b1595eb31c6c9f91d8819dc421b4d2
link
/test gencheck_installer

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@douglas-reid: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_installer
5154817f584ed6b240850a6373a8e77bf1853a08
link
/test gencheck_installer

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

In response to a cherrypick label: new pull request created: #515


[Sprint: 53] XD-3235 Test looks for specific files vs *

Also set up the FileJdbc tests have unique job names instead of ec2test3

LGTM.  Merged as 40bb9ba9815900d770137e2d4087f81ade243344


Added Perl installation instructions to NOTES-PERL.md for HPE NonStop.

CLA: Permission is granted by the author to the OpenSSL team to use these modifications.
Fixes #14931.
Signed-off-by: Randall S. Becker<EMAIL_ADDRESS>
Checklist

[x] documentation is added or updated

My confusion. I'll amend the commit tomorrow.

@rsbeckerca the content looks good, but please remove the CLA declaration from your commit message.

Done

24 hours has passed since 'approval: done' was set, but this PR has failing CI tests.  Once the tests pass it will get moved to 'approval: ready to merge' automatically, alternatively please review and set the label manually.

This pull request is ready to merge

Merged to master in 2de02e7dca0a875b1ae5b6a4a4f946705eb71edc, thank you!


what's the difference between .float_weight and .weight?

Hi,
I note that after quantization-aware training, the saved  weight file contains .float_weight and .weight for each layer. What's the difference between  them?
Thanks

".weights" are the quantized weights which you get after applying "quantization" function followed by "dequantization".
".float_weights" are the floating point shadow copy of the original weights

During forward pass -> apply quantization on ".float_weights"  i.e quantization + dequantization and  update the ".weights"
Calculate gradient on ".weights"  and update ".float_weights"

Closing due to inactivity. Please reopen if needed.


Refactor the assets resolution of the script setup

Close: #3543
with <script setup>, Assets(components/directives) will be tried to resolve from the setup state, this leads to naming conflicts between assets and ordinary variables, e.g.
<template>
  <div v-focus/>
</template>

<script setup>
const focus = ref(0);
</script>

The constant focus in setup will overwrite the globally registered directive with the same name. This is because the above code will be compiled as:
export function render(_ctx, _cache, $props, $setup) {
  return _withDirectives((_openBlock(), _createBlock('div', null, null, 512 /* NEED_PATCH */)), [
    // Will no longer perform asset resolution
    [$setup["focus"]]
  ])
}

with $setup["focus"], it will no longer perform asset resolution.
My idea is that in order to avoid name conflicts, we should always perform asset resolution, which makes globally registered assets have a higher priority, and at the same time, use setup state as a fallback. E.g.
export function render(_ctx, _cache, $props, $setup) {
  const _directive_focus = _resolveDirective(
    // resolve globally registered assets
    "focus",
    // warnMissing
    false,
    // fallback, when the asset resolution fails, use the fallback instead
    $setup["focus"]
  )

  return _withDirectives((_openBlock(), _createBlock('div', null, null, 512 /* NEED_PATCH */)), [
    [_directive_focus]
  ])
}

This behavior also applies to components resolution. And it only takes effect when there is a state with the same name in the setup result, without breaking anything.

I don't think we should prioritize global asset registrations over local variable resolutions. Local resolution should always have higher priority.
To reduce the potential shadowing of directives (which is way less obvious and more likely than components), I reverted it to require the variable to have a v prefix (RFC updates soon) in d35e0b14.


[Event Hubs] Improve prefetching

Packages impacted by this PR
@azure/event-hubs
Issues associated with this PR
https://github.com/Azure/azure-sdk-for-js/issues/27253
Describe the problem that is addressed by this PR
https://github.com/Azure/azure-sdk-for-js/issues/27253#issuecomment-1791749605
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
To be tested using stress testing framework.
UPDATE: The results are in and it is confirmed there is no more space leak!
Provide a list of related PRs (if any)
https://github.com/Azure/azure-sdk-for-js/pull/26065
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[x] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[x] Added a changelog (if necessary)

API change check
API changes are not detected in this pull request.

I reran the prefetching experiment from https://github.com/Azure/azure-sdk-for-js/pull/26065#discussion_r1218703744 to make sure prefetching is still working properly and that appears to be the case according to the graph below. Blue is this PR and the other is v5.11.2.

Thanks a lot @harsha-nalluru and @jeremymeng for the valuable discussions and helping to diagnose the issue and running bisecting experiments to close down on the culprit commit 👍


net/haproxy: [FR] Add Basic Auth functionality

Hi,
I want to replace an apache reverse proxy which is doing basic authentication via .htaccess before user can access a specific service.
@fraenki is there a way to implement this like described here:
https://serverfault.com/questions/239749/possible-to-add-basic-http-access-authentication-via-haproxy

I'll consider addings this, but I first need to finish #208 before adding new features.

Example/tutorial: https://forum.opnsense.org/index.php?topic=6597.0

I think OPNsense cannot handle the "401 Unauthorized" Pages yet.
(Though in this special case it should)
best
manu

I think OPNsense cannot handle the "401 Unauthorized" Pages yet.

Sorry, I don't get it, what's your point?

Therefore I guess the 401 option should be added to the option-box in the corresponding GUI error-templates-page.

Are you referring to HAProxy -> Settings -> Advanced -> Error Messages?
This feature does not support 401 according to the official documentation:
http://cbonte.github.io/haproxy-dconv/1.8/configuration.html#4-errorfile

yes I mean that page.
and are you referring to this?:
It is important to understand that this keyword is not meant to rewrite errors returned by the server, but errors detected and returned by HAProxy. This is why the list of supported errors is limited to a small set.
I'm sorry, I think I misunderstood, the err message that will be served when a basic auth failed, will be a 403, not 401, I guess.
never mind, m


Ruby NLP API side bar links broken

The side bar from this page has broken links to Project , Document  and Annotation

Thanks for finding this. These types were removed from the Natural Language API, but the nav for the google-cloud meta-package was never updated, although the nav for the google-cloud-language package was correctly updated. I'll fix it now. Please refer to the google-cloud-language docs for now.


fake quantized model

Hi, how to get a fake-quantized model?

Hello, you can follow the "Calibration" section in README to get a fake-quantized model. To reproduce results in the paper, you can download pre-trained transformations from the modelzoo.

How to save quanted model like Transformers' model, which can be directly used

For weight and activation quantization, FlatQuant needs to insert some online transformations as well as activation quantization operations in the fake quantized model, so we suggest using our pre-defined model class for inference. More details can be found in flatquant/model_tools.


Updating broken "Azure Portal" link

Azure Portal Link was broken in text. Updating it to point to active-directory-integrating-applications, which seems to be most appropriate content in context of article.

Hi @reddobowen, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

@reddobowen : Thanks for your contribution to the Azure documentation! The author, @bryanla, has been notified to review your proposed change.

Hi @reddobowen - thanks again for your contribution and finding this broken link! I submitted another PR to your reddobowen:patch-1 branch, which has some additional changes. The problem was introduced by some changes to the reference-style links at the bottom of this article. I've corrected the reference so the link will resolve correctly (to the new Azure portal).
As soon as you can merge PR #1 in your fork, we can get this merged for you. Thanks again!

Thanks @BryanLa. - Haver merged these changes in for you.

Excellent, thanks @reddobowen. I will #sign-off so these should get merged soon, and likely be in the next build (this afternoon or tomorrow morning).


As a user, I do not want to see stacktraces when errors occur

Not urgent as we are still in development but I want to track this because we need to fix it before public deployment.
The default error page currently includes stack traces and info that we do not want to be publicly available when the service is published.
Need to create a user-friendly error page that indicates an error occurred. The error page should include a unique error code identifier so that service operators/developers can quickly search application logs using the unique error code to find the relevant stack trace.

Approach: Generate an UUID and log the error information associated with it.
Use Case: If the user reports the error UUID, we can use it as an index to search the logs and retrieve the actual stack trace on the back end.
Note: The stack trace should be recorded in the application logs and not displayed to the end user.


Error getting devices: Error: UserOpenTwoFactorAuth

Provided all necessary information in config.json but it's not working due to failing on two factor authentication.
here is the error message captured from the logs:
5/14/2020, 9:38:59 AM > [5/14/2020, 9:38:59 AM] [Wyze] Refreshing devices...
5/14/2020, 9:38:59 AM > [5/14/2020, 9:38:59 AM] [Wyze] Performing request: app/user/login
5/14/2020, 9:38:59 AM > [5/14/2020, 9:38:59 AM] [Wyze] API response: {"ts":<long-number-here>,"code":"<4-digits-here>","msg":"UserOpenTwoFactorAuth","data":{}}
5/14/2020, 9:38:59 AM > [5/14/2020, 9:38:59 AM] [Wyze] Error getting devices: Error: UserOpenTwoFactorAuth

2FA isn't currently supported, but I'll look into this in the future to see if I can implement it via refresh tokens. I'd suggest disabling 2FA for now if you want to use this plugin. Thanks!

i noticed that if the user provides all info you request in your constructor (except refreshToken which is not necessary) in the config.json he won't need to perform the login method. therefore, you can bypass it as an alternative. from what i saw in your code this is already the case.
i guess you just need to collect those in your getClient()

well.. good news for you.
in the PR i just submitted i added support for 2FA as well :)
took the chance to enhance a bit the error handling and logging here and there, nothing major.


Add onSomethingDown callbacks

Adds callbacks for the different clicks on mouse down.
This is useful for certain things like changing the position of the cursor in an Input component on mouse down.

You're probably right that it's not a common need. I was mainly thinking in terms of future improvements to the Input component. It needs the cursor to change on left click mouse down and it will eventually need a word to be highlighted on double click mouse down.
I don't think any component should use onMouseDown from a View because it is called for any type of mouse button. This leads to things like the Slider component moving with a right-click drag. My current thinking is just that anything that cares about mouse clicks should use a Clickable so that things like onMouseDown work only with left clicks and code checking for what mouse button was used does not need to be duplicated. It could also make the intent more clear (e.g. that Slider is meant to be clicked).
I guess it depends on how the Clickable component is meant to be used. Maybe there should be a different component?

I'm honestly not really sure what Clickable is supposed to be either, as the events it exposes could easily have been made available directly on View.

I don't think any component should use onMouseDown from a View because it is called for any type of mouse button. This leads to things like the Slider component moving with a right-click drag.

The mouse button is passed in the events args. That's how you're able to implement this. And this is analogous to how onMouseDown works on the Web. For something as rarely needed as this I personally think that's fine.
Left-only click, right click and double click is common enough that I think it warrants separate events, and they fit well with Clickable. But the rest I'm not so sure about design-wise. It would be nice to have a holistic idea and some overall guiding principles for where we're going with this I think.

Sounds good! A document with some guiding design principles would be quite nice, especially with regards to big things like #489.


Check for comparing 2 tables are identical (will be less efficient than metrics based check)

Hi @timgent - Is this check only for ddl's (column/datatypes) or will it compare the contents of entire table ?

@rishabhishere the intention was for it to be of the entire table, including decent messages about differences (hence why this check type would be less efficient than the metric based checks, as this would involve a join).
Comparison of DDLs could be added separately if that would be helpful - if it's a feature you'd like to see please raise an issue.


Financial Journal [P11300] - Change variables to protected

Hello,
<Page 11300 - Financial Journal>
We are having the problem that we can't update the fields  and   on our Financial Journal Page.
The function  is set Obsolete:

I would ask if it is possible to make the following Global Variables Protected:

BalanceLastStatement: Decimal;
StatementEndingBalance: Decimal;
TotalBalance: Decimal;
Balance: Decimal;

Kind regards,
Nicolas

You can use obsoleted function, in this case this is just information. Protected will be added.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Build(deps): Bump jszip from 3.8.0 to 3.9.0 (#3275)

Pull request opened by the merge tool on behalf of #3275

Merged in c0c06028e418720320c947045a160a6d667cb813


Add support for hardkernel supported kernel/u-boot base C2 machine

Add LCD startup scripts and sytemd scripts.
Refresh linux-stable 4.14 patchset
Downgrade mali to r6

thanks


The Searchbar crashes on '

Bug: The Searchbar crashes on '
Problem Description

The Searchbar crashes on '
It should not crash
this could be a symptom of more serious problems

Steps For Reproduction

go to search
search for '
it crashes

This is probably a duplicate  for https://github.com/hpi-swt2/bookkeeper-portal-blue/issues/169


automatic is an unsupported --view argument

dockutil --add ~/Documents --view automatic results in an error
Error processing options: unsupported --view argument

But the help page indicates automatic is a valid argument for --view
folder_options:
  --view [grid|fan|list|automatic]                              stack view option
  --display [folder|stack]                                      how to display a folder's icon
  --sort [name|dateadded|datemodified|datecreated|kind]         sets sorting option for a folder view

dockutil --add ~/Documents --view --automatic  does not result in an error

I still get the same error with --automatic
It's worth noting that I don't get any errors with any of the other folder view options of grid, fan or list

Hmm, I'll admit I may be using an older version and if it is a bug, it may have been introduced somewhere along the line. Curious, what version are you using?

$ dockutil --version
2.0.2

on my home computer running 2.0.0 the command executes without error, but no documents folder added to dock. I upgraded to v 2.0.2 and got the same error as you. Now I have to check my work mac were I ran it this morning without error and with success...

Sheez, just checked and at work I still have 1.1.4! (had to downgrade for early 10.10 support I seem to recall.) So you seem to have identified a bug.

try
dockutil --add ~/Documents --view auto
I had a look at the code and the issue is on line 195:
    elif opt == '--view':
        if arg == 'fan':
            showas = 1
        elif arg == 'grid':
            showas = 2
        elif arg == 'list':
            showas = 3

195=>            elif arg == 'auto':
as a test i edited the line... changing from "auto" to "automatic" and it works as the notes suggest.
to make it work without edits just use "--view auto"

Ah, good find. auto is probably a better keyword anyway (shorter), so maybe I can submit a pull request with updated documentation to resolve this. Thanks!

You're most welcome! It was a good dig, someday we'll find an undiscovered race of humans in all this code:-)


whereis lookup logic is wrong

The package should not fallback on whereis if which failed on Linux. See https://github.com/vvo/selenium-standalone/issues/96 for the example of mislocating java binary by trusting whereis.
See https://superuser.com/questions/40301/which-whereis-differences for the explanation of whereis command. It should come with additional checks to get binaries correctly.

I am happy to take a PR on this @techtonik

Not sure it will happen any time soon. As I don't code in JavaScript, there is bigger issue with understanding the whole Gratipay JS testing toolchai, and it will just take more time for me than for somebody who knows how npm works and deal with it daily.

@techtonik I think using https://github.com/isaacs/node-which is the way to go


Grid column not editable in popup

Hi,
I'm, facing this strange issue where I'm not able to show column as editable in spite of declaring it as editable column, if the grid is coming in pop up. The user simply cannot edit by double click in that column.
However if the same grid is not displayed as popup , the editable columns works fine.
Is this a known issue , if yes is it resolved yet.

I got the missing thing.
So I see that recid ie mandatory to declare in the records data, to perform the edit functionality correctly.


(2.3.9) Rim War index out of range

IMPORTANT: Please first search existing bugs to ensure you are not creating a duplicate bug report!
Describe the bug
!!! bug description here !!!
Not sure
Steps to reproduce the behavior (VERY IMPORTANT)

Go to '...'
Click on '....'
Scroll down to '....'
See error

Error Log
!!! Link to contents of Player.log or Error.log (MUST BE LINK) !!!
!!! Link to HugsLib log. (MUST BE LINK) !!!
https://gist.github.com/f1423b5eb7465b89538abe3a38c9af6a
Save file
!!! Please add a save file where the issue occurs to speed up testing and fixing of the issue !!!
Screenshots
!!! Add screenshots to help explain your problem here !!!
rim war.zip

Please retest with RT>=2.5.12 and include steps to reproduce. Closing until then.


Added more explaination about user.

Signed-off-by: dungeonmaster18<EMAIL_ADDRESS>
What this PR does / why we need it:
Added more explaination about $user.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
It's ready to merge.
cc @kmova

Hey can @kmova can you review this.


Fix #125: Move test files to test/files

Fix #125: Move test files to test/files so that they're not included in the pkg that's uploaded to hex.pm.

I like more this under test/ (btw not sure if we can ignore folders in the packages via config)

No idea.


swagger specification validate failed at:  types don't match expect map key string or int got: bool

When try to validate a swagger 2.0 spec in yaml with "n" as the property name like:
definitions:
MyModel:
properties:
n:
type: number
format: double
......
I got the error message,
2017/10/13 14:05:57 types don't match expect map key string or int got: bool
After use the quote on the property name as 'n', it works.   I don't understand why the swagger generator behaves like this.  Also, I suggest to add the following features:

When the validation error occurs, it's useful to print out the line number of the error in the spec file.  For a very large file (over 30K line), it's pain to debug.
Can the generator accept the swagger 2.0 JSON format?   The internal does convert the yaml to JSON during the parse.

Thanks.

this is a yaml thing.
You want to single quote your key name: 'n'

Yes the generator works with yaml and json so you can use either.
I agree on line number and column position however default go tooling doesn't provide the right means to do so

Can you tell me why the n is a reserved keyword in the Swagger spec?  When I use the JSON file, it has to have the 'n' as well.   Thanks.

As for the reason, YAML treats these strings as boolean:

thanks.

http://yaml.org/type/bool.html


Attempting to fix rendering problems on jujucharms.com (again)!

I also changed some paragraphs to be less than 80 columns.
I added some words on Ingress and what it is.
Fixed some other spots.

I addressed your comments. Please re-review.


All file-based data sources should have a process method similar to CzmlDataSource

The current implementation of GeoJsonDataSource does not give any way to dynamically load features in multiple batches, because it always clears the entity collection when load is called.  CzmlDataSource offers a process method which simply calls the underlying (private) load method without clearing existing entities.
This method should be added to GeoJsonDataSource and KmlDataSource for consistency.

I put together a simple PR adding this to GeoJsonDataSource, because it was trivial.  I'm reasonably confident that the current implementation will handle processing various kinds of input without inducing any weird behavior.  I added one test, to call load with some features, and then process with additional features, and that works.  More tests could be added to poke at edge cases but I suspect it's fine.  (The team could of course have insight to the contrary!)


[ODB] fail to get uploadUrl for an existed file name

Category

[ ] Question
[ ] Documentation issue
[x] Bug

Step 1.
Upload a file to OneDrive for Business named "test.file"
Step 2.
Create a upload session for the same file "test.file"
Step 3.
Request for the uploadUrl and then got an error.

GET uploadUrl

{
    "error": {
        "code": "-1, Microsoft.SharePoint.Client.InvalidClientQueryException",
        "message": "運算式 \"drive/items('01VY6VLMTBMCZYLMN7MBFYZO5UMNEADZHE')/getUploadStatus(guid='70bc8977-30ae-42be-91c9-659c5c5fb987',path=)\" 無效。"
    }
}

Step 4.
More information is below
request-id →dbcf559f-80da-b000-5d96-b47ee56d2b78
date →Mon, 25 May 2020 07:51:43 GMT

I also try different options like rename | fail | replace, and all of them are the same.
Thank you for help.

We made a fix recently that stopped include the parent id in the upload URL for a file update, however it looks like this regressed the GET scenario. We've disabled the change for now - can you confirm that the issue is resolved for you?

Hi @ificator ,
Thanks for reply and have quick fix, everything works fine right now.

Thanks for confirming @joewutaiwan - I'm going to close this issue out but feel free to reopen if you notice anything strange.


onItemUpdating previousItem not correct when using rowClick function

When implementing onItemUpdating, expect args.previousItem to contain original item data.
If rowClick is implemented, args.previousItem in onItemUpdating contains updated data, not the original.  If I remove rowClick function and revert to in-place update, args.previousItem has expected original item data.

Could you provide an example in jsfiddle (or any other online editor) reproducing the issue?


Fix unsupported backend error when using s3 in django-storages

the storages backend doesn't implement path() as it should be, but filer requires that.

awesome, thanks!


feature request: unit tests!

The lack of tests makes it somewhat difficult to contribute to this repo because I'm worried that by changing things, I'd be breaking other things :(

Implemented in v 0.5.0


move from cats 0.9 to 1.0

the needed cats-effect change was made in https://github.com/typelevel/cats-effect/pull/61 (thanks @djspiewak!), but we can't unfreeze cats until https://github.com/monix/monix/issues/366 is resolved

This cannot be fixed until Cats releases packages for master, because it's a binary compatibility issue, e.g...
[info] - Sync[Coeval[Int]].sync.forEffect consistent flatMap *** FAILED ***
[info]   Exception raised on property evaluation. (Checkers.scala:37)
[info]   > ARG_0: <function0>
[info]   > ARG_1: <function0>
[info]   > Exception: java.lang.AbstractMethodError: cats.effect.laws.SyncLaws$$anon$1.forEffectConsistency(Ljava/lang/Object;Ljava/lang/Object;)Lcats/laws/IsEq;

[info] - Sync[Coeval[Int]].sync.monadError ensure consistency *** FAILED ***
[info]   Exception raised on property evaluation. (Checkers.scala:37)
[info]   > ARG_0: <function0>
[info]   > ARG_1: java.lang.Error
[info]   > ARG_2: <function1>
[info]   > Exception: java.lang.AbstractMethodError: cats.effect.laws.SyncLaws$$anon$1.monadErrorEnsureConsistency(Ljava/lang/Object;Ljava/lang/Object;Lscala/Function1;)Lcats/laws/IsEq;

[info] - Sync[Coeval[Int]].sync.monadError ensureOr consistency *** FAILED ***
[info]   Exception raised on property evaluation. (Checkers.scala:37)
[info]   > ARG_0: Coeval.Once(<function0>)
[info]   > ARG_1: <function1>
[info]   > ARG_2: <function1>
[info]   > Exception: java.lang.AbstractMethodError: cats.effect.laws.SyncLaws$$anon$1.monadErrorEnsureOrConsistency(Ljava/lang/Object;Lscala/Function1;Lscala/Function1;)Lcats/laws/IsEq;

Basically AbstractMethodError happens when binary compatibility is broken. And in this case it is broken because the cats-laws package has new methods in those traits.
Hopefully we'll fix this once Cats releases the scheduled 1.0.0-MF version.
/cc  @djspiewak

I'll try to put in some work this weekend to push on some of the 1.0-MF stuff.  @kailuowang has really been driving that effort, but I think he's blocked on other people now.

now that cats 1.0.0-MF is out, is it a plausible time to unfreeze, or to at least move to newer frozen versions?

Hey, so the master in Monix is fixed and actually depends on Cats 1.0.0-MF.

well, let's take a stab at this and just see what happens: https://scala-ci.typesafe.com/job/scala-2.12.x-integrate-community-build/1743/consoleFull

@SethTisue apparently that build failed, any ideas why?

[info] >>> The dbuild result is------------: FAILED (failed: fs2-cats, doodle)

we have doodle forked, but I see they are already on cats 1.0.0-MF on their master branch, so I assume that  if we refreshed our fork (which is still on cats 0.9), the problem would go away
is there someone reading this who could take care of upgrading fs2-cats to cats 1.0.0-MF? note that http4s depends on fs2-cats. @rossabaker what's the status of http4s moving to 1.0.0-MF...?

http4s on cats-1.0.0-MF is on the release-0.18.x branch.  I don't think that branch has users yet, but runs green.

Tagging @mpilquist for the fs2 questions.

new test run with http4s's release-0.18.x branch: https://scala-ci.typesafe.com/job/scala-2.12.x-integrate-community-build/1753/consoleFull

Why was this aborted?  Did it hang?

I killed it so I could use the Jenkins node on something more pressing. I'll be returning to this soon.

notes on work in progress:

doodle already moved to cats-1.0.0-MF on their develop branch, so we probably just need to refresh our fork
Michael already fixed fs2-cats
jawn-fs2 needs fs2-io now (there's probably no reason we weren't building fs2-io)

@rossabaker currently blocked on jawn-fs2 which still uses fs2.time which was removed from fs2 between 0.10.0-M2 and 0.10.0-M5. can you update jawn-fs2 to the newer fs2 version?

@rossabaker working around it for now by excluding these subprojects: "jawn", "argonaut", "json4s"

jawn-fs2 is now based on fs2-0.10.0-M5 on master, and is released as jawn-fs2-0.12.0-M1.

@rossabaker I see https://github.com/rossabaker/jawn-fs2/commit/e313fefcd98e5d58943d4b3056843c06422c3518 exists, but it isn't on master

Oops. 😳  Now it's on master.

@rossabaker thanks! the http4s part of this is now green, I'm just making sure something didn't regress elsewhere before merging.

@rossabaker note that we're not running http4s's tests, because org.http4s.CookieSpec org.http4s.EntityDecoderSpec are failing


Create Atlas example

Create an example using Atlas from https://github.com/RobotLocomotion/models/tree/master

#48

Atlas stands, but struggles to maintain its initial orientation. Likely just a tuning problem with the objective.

Closed with #77


chore: readme interpolation

This PR introduces a script for interpolating markdown files, and restructuring existing readmes to extract some common content (features list).
The way the script works is by taking .template.md files, interpolating the contents of the imported markdown files, and writing .md files. Both files (template and final) are version controlled, so when modifying a certain readme we should be modifying the template files always, since the other one is generated.
The invocation of this script is performed as a prepackage script on the root package, since that's the entry point for packaging and distributing our modules.
Closes #401

Codecov Report

Merging #405 (1d18af1) into development (d34fe2e) will decrease coverage by 0.05%.
The diff coverage is n/a.

:exclamation: Current head 1d18af1 differs from pull request most recent head f44cc0e. Consider uploading reports for the commit f44cc0e to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@               Coverage Diff               @@
##           development     #405      +/-   ##
===============================================
- Coverage        55.73%   55.68%   -0.05%
===============================================
  Files              175      175
  Lines             4464     4464
  Branches           735      735
===============================================
- Hits              2488     2486       -2
- Misses            1744     1745       +1
- Partials           232      233       +1

Impacted Files
Coverage Δ

src/utils/index.ts
80.00% <0.00%> (-6.67%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@kanej ready for re-review!


PPX support

Naturally, I only use 4.02, as most of my work includes usage of extension points.

I was pleasantly surprised to find out that with merlin 2.1.2 some ppx extensions seem to be working by just adding the relevant ocamlfind package (for example cconv.ppx).
Is this feature still experimental or is it stable enough to be mentioned in the README?

Should be stable as it uses the same logic as toplevel.

Composing PPX extensions seems to still have issues.  The example in ppx_import's README mixing ppx_import and ppx_deriving (https://github.com/whitequark/ppx_import#combining-with-deriving) does not work in vim even if I manually :Use ppx_import and then :Use ppx_deriving.std.  The same works from vanilla ocaml, utop or normal compilation.

merlin applies ppx ordered by their name… I don't know what would be the best way, probably by order of appearance but then there is some ambiguity when mixing ppx from .merlin and from :Use.
Also, it is still unclear to me what is the correct way to specify ppx options.

A cascading model would suggest that .merlin preprocessors be applied prior to :Use preprocessors.

@hcarty : should be fixed in latest git.

master works for me under 4.02.1 @def-lkb.  Thank you!


ng-add Path "undefined" does not exist

Is there an existing issue for this?

[X] I have searched the existing issues

Which Transloco package(s) are the source of the bug?
Schematics
Is this a regression?
Yes
Current behavior
ng add @ngneat/transloco does not work in angular 17.
In a new Angular project (ng new test), running ng add @ngneat/transloco gives the error Path "undefined" does not exist
Expected behavior
ng add @ngneat/transloco shoudl work
Please provide a link to a minimal reproduction of the bug, if you won't provide a link the issue won't be handled.

Transloco Config
No response
Please provide the environment you discovered this bug in
Transloco: 6
Angular: 17
Node: 18
Package Manager: npm
OS: macOS

Browser
No response
Additional context
No response
I would like to make a pull request for this bug
No

Same thing happening to me, clean project created with ng new angular-frontend, me too using Angular 17, Node 18 and Transloco 6 (but Transloco 5.0.10 same thing).

The same thing happens to me too and I don't know why
Angular CLI: 17.0.1
Node: 18.18.0
Package Manager: npm 9.8.1
OS: win32 x64

Anyone have a solution?

I'll try to take a look today

I'll try to take a look today

Thanks Shahar ^^ !

Hello, any news on this topic ?
I have the same issue on my side.
Transloco: 6
Angular CLI: 17.0.3
Node: 21.2.0
Package Manager: npm 10.2.3
OS: Debian

I've tested on both v16 and v17 projects, and it works as expected 👍
LMK if there are any issues

Shahar, thank you so much 😇😇😇☺☺☺☺

hi i have been the same issue at moment of use "ng add @angular/pwa" to resolve I updated the version of angular cli and  dissapear the error.

Im also experiencing this issue "Path "undefined" does not exist.". The only thing I can imagine is that I don't really have a standard project structure.
"newProjectRoot": "resources",
    "projects": {
        "frontend": {
            "projectType": "application",
            "root": "./",
            "sourceRoot": "resources/frontend",

Maybe this has something to do with it?

I've tested on both v16 and v17 projects, and it works as expected 👍 LMK if there are any issues

Hi it's not work for me with V17
Node v18.13.0
Mac
"@ngneat/transloco": "^6.0.4",
Nx workspace

@shaharkazaz

@davisriska @dzcora Please share a reproduction as the original issue was already resolved.

Still getting this issue with Angular 18.1.4.

@berkon I might be missing something, but how is running the angular/pwa schematics related to this issue or Transloco?

@shaharkazaz, you are right, I was just looking at the title and my error message is exactly the same. I didn't notice that it was about transloco.
BTW: My findings showed that it looks like this issue happens when there is something not compliant to the standard Angular workspace/project settings/file structure etc. . When generating a clean project from scratch it worked well. Unfortunately the error message is really useless.


Robot allocator update should happen after world sim update

From #45, when RA and WS are not in sync, undefined timing behavior happens because robot updates may happen while planning is going on.
Orange bars are when RA updates happen
Blue bars are when WS updates happen

Want RA updates after
Separate todo is to figure out what takes so long for RA updates and fix

Using 0mq to notify RA when to update, WS sends publish when finished with update, RA subscribes to it.


MapRoulette: Allow working on a specific hidden challenge

Description
I tend to set MapRoulette Challenges to "invisible" whenever I want to test them but also when I only want to work on them with a specific group of people.
However, it looks like there is no way to access a non-public challenge in Rapid.
(Right now, 45242 is public again, to allow testing it.)

I was hoping for something like …

When no challenge ID is given, show all public challenges
Wen a specific challenge ID is given, show that one regardless of it's public state

I've asked the MapRoulette devs what they think of allowing this little override, I'm in favor.


Add account management UI info to user guide

Added two pages to document account management tasks, covering billing and access, to assist with common customer questions. Team based access will be updated in a future commit to avoid creep. The pages link to each other, which is why I made the decision to bundle them in one commit.
An additional change was to move the account management section to the top of the user guide, based on feedback.
Future changes will likely be needed as questions arise, and these pages are not meant to be comprehensive at this time.
QA steps taken: quick pass with vale linter, site rendered as html and visually inspected. Ran link-check. No issues were detected.
This commit applies to FFTK 1305 and FFTK 1611
Signed-off-by: Katrina Prosise<EMAIL_ADDRESS>Readiness

[x] Merge (pending reviews)
[ ] Merge after date or event
[ ] Draft

Overview
Why merge this PR? What does it solve?
Checklist
Optional. Add a 'x' to steps taken.
You can fill this out after opening the PR. "Did I..."

[x] Run spelling and grammar check, preferably with linter.
[x] Avoid changing any header associated with a link/reference.
[x] Step through instructions (or ask someone to do so).
[x] Review for wordiness
[x] Match tone and style of page/section.
[x] Run make linkcheck.
[x] View HTML in a browser to check rendering.
[x] Use semantic newlines.
[x] follow best practices for commits.

[x] Descriptive title written in the imperative.
[x] Include brief overview of QA steps taken.
[x] Mention any related issues numbers.
[x] End message with sign off/DCO line (-s, --signoff).
[x] Sign commit with your gpg key (-S, --gpg-sign).
[x] Squash commits if needed.

[x] Request PR review by a technical writer and at least one peer.

Comments

Docs for a9c3cd70ad58e6c4abc3862640e76b39b685cfd6 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1668/docs/artifacts/html/index.html

There's some other information I think we can explore, like each member can manage their devices, and the factory owner/admin can manage all devices, also who has source code access and can push changes.
But I don't know if you'd like to address this in a different section of the docs (not related to account/billing).

Docs for 41850b9ab048c48b6683bc7b2c246d90ed10c9f3 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1674/docs/artifacts/html/index.html

Docs for 2f6c150df810cd42bc4242315620e1d487492df2 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1685/docs/artifacts/html/index.html


Icon

Please accept this small – selfmade so don't worry about a license – gift as a token of my appreciation!

Thank you very much Koen, appreciated!


Enhancement: Support for Direct Serialization and Deserialization of Quantity Objects

Description:
Currently, the library does not support direct serialization and deserialization of Quantity objects. This limitation prevents the use of Quantity as a standalone object in endpoints, resulting in a NullPointerException. An example of the issue is shown below:
@PostMapping
public Quantity<Length> readQuantity(@RequestBody Quantity<Length> payload) {
    return payload;
}

The following exception is thrown:
java.lang.NullPointerException: Cannot invoke "com.fasterxml.jackson.databind.BeanProperty.getType()" because "property" is null

Improvement Proposal:
I propose adding support for the direct serialization and deserialization of Quantity objects. This would allow developers to use Quantity in RESTful APIs and other contexts without wrapping it in a POJO.
Personal Experience:
I have been exploring the library and its capabilities, and I like what I have seen. However, I believe that we can improve it even more.
Implementation Offer:
If it is acceptable, I am willing to implement this feature. Please let me know if this is okay, and I will be happy to start working on it.
Thank you for considering this enhancement.

Good point, the library is not designed to handle this case.
I will take a look on how this can be implemented!


fix: ImageRepo spec.image.name is not a "name"

The spec.image.name field for ImageRepository resources was defined as a K8s "name" field but it isn't, it actually need to be able to contain slashes which are typically found in container image names.

/lgtm


Overlay network can't be created

When I do $ docker network create --driver overlay isolated_lemp on my CoreOS host, I've got the following error message:
Error response from daemon: failed to parse pool request for address space "GlobalDefault" pool "" subpool "": cannot find address space GlobalDefault (most likely the backing datastore is not configured)
$ docker-compose --x-networking --x-network-driver=bridge up works without problem.
docker version:
Client:
 Version:      1.9.1
 API version:  1.21
 Go version:   go1.4.3
 Git commit:   9894698
 Built:
 OS/Arch:      linux/amd64

Server:
 Version:      1.9.1
 API version:  1.21
 Go version:   go1.4.3
 Git commit:   9894698
 Built:
 OS/Arch:      linux/amd64

docker info:
Containers: 0
Images: 78
Server Version: 1.9.1
Storage Driver: overlay
 Backing Filesystem: extfs
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 4.4.0-coreos
Operating System: CoreOS 928.0.0
CPUs: 1
Total Memory: 997.3 MiB
Name: core-1
ID: VTHP:4CUA:LQKE:NWGW:ZEHY:VLTS:ZHLD:YLI2:OOD6:3LAI:DEFA:X4GL

uname -a:
Linux core-1 4.4.0-coreos #2 SMP Fri Jan 15 03:58:19 UTC 2016 x86_64 Intel(R) Core(TM) i7-2630QM CPU @ 2.00GHz GenuineIntel GNU/Linux

This environment is under Virtualbox.
I want to create an isolated network for my docker-compose LEMP stack. Since --link will be deprecated in the future, I ditched it and try to implement an isolated network.
My problem was that Wordpress cron jobs didn't worked in bridged network so I gave net: "host" to every container in the docker-compose file, but I've got the error message link will not work with net command and it will be deprecated in the future. So I trying to implement this new networking feature, right now wihout luck.
Maybe not connected with this issue, but my MariaDB inside the stack become dead and sometimes I cannot removed the old containers. When I restarted the whole service with a systemd file, my entire webserver static files deleted together with the old containers.
According to this helper fellow on StackOverflow he experienced the same behavier and lost all of his webserver files too, which were shared into the container via volume: http://stackoverflow.com/a/34873462/1442219

UPDATE:
I just got noticed that you cannot create overlay network on a single node.


Pause music when computer is restarted

It's a nice touch that Hermes pauses music when your computer sleeps, but in OSX Lion if you shut down and choose to "reopen windows when logging back in", when you start the computer again, Hermes starts playing immediately. It would be better if Hermes started up in a paused state.

It doesn't play on restart, but I'm not sure it does exactly what I'm expecting (yes, I have high expectations). With the new version, on restart, Hermes isn't playing any station. I would expect it to stay on the same station (and preferably even the same song) on restart, but it is in a paused state. Ideally, after a restart, you can just press the play/pause key and it'll be as if you never restarted.
I'm not sure how exactly the APIs work, but I'm not sure you would even need to refer to Lion APIs – just switch Hermes to a paused state when the computer is about to shut down, and then Lion should automatically restore this paused state when you restart.

Pandora's API gives a link to a song, and this link expires after an amount of time. This means that when you reboot, you're not guaranteed that the link will work.
I can see, however, that it might be possible to try its best at restoring the previous state. It could certainly save off where
you were in the song and the list of future songs/station/whatnot. On restart it would then read off all this information and try to restore state based off of that.
It's very likely this restore will fail, however, because all of the links will have expired. You can't ask Pandora for a link to a specific song, so you'd have to throw away the saved songs and continue playing song on the current station. Does that sound reasonable?

That sounds very reasonable. I'm not sure how long this link lasts, but in the current stable version of Hermes I'm fairly certain I left my computer off for around 12 hours and it resumed playing when I turned it back on. I don't know if this is because Pandora's link still hadn't expired, or the whole song was just cached and Lion restored this cache on startup.

How does this build look to you?

Works great!


Fix blueprint typo: "metrics-adapter[s]"

Came across this little blueprint typo while working on some custom adapters.

Normally I try to always add tests for PR's, but not sure if/how blueprint file-gen code is tested..
Thanks for a sweet library! Refactoring our spaghetti pre-Ember.Service analytics util right now :D

Thanks!


⚠ Bofrost.fr has degraded performance

In 8d94a9e, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1383 ms

Resolved: Bofrost.fr performance has improved in 292ac58 after 10 minutes.


Support optional/configurable IAMEndpoint for Minio Client (#32581)

Targeting issue #32271
This modification allows native Kubernetes + AWS (EKS) authentication with the Minio client, to Amazon S3 using the IRSA role assigned to a Service account by replacing the hard coded reference to the DefaultIAMRoleEndpoint with an optional configurable endpoint.
Internally, Minio's credentials.IAM provider implements a discovery flow for IAM Endpoints if it is not set.
For backwards compatibility:

We have added a configuration mechanism for an IamEndpoint to retain the unit test safety in minio_test.go.
We believe existing clients will continue to function the same without needing to provide a new config property since the internals of Minio client also often resolve to the http://<IP_ADDRESS> default endpoint that was being hard coded before

To test, we were able to build a docker image from source and, observe it choosing the expected IAM endpoint, and see files uploaded via the client.

by the way, "app.example.ini" and related documents https://gitea.com/gitea/docs/pulls also need to update 🙏

Wait for config example update before merge. If there is any difficulty, maintainers could also help.

@wxiaoguang thank you for the feedback, I'll take care of the documentation PR today!

Added the requested updates for app.example.ini and opened https://gitea.com/gitea/docs/pulls/103
Let me know if there is any feedback on documentation structure or anywhere I missed that I also need to update. Thanks!

@mowoc-ocp please fix the merge conflicts. :tea:


Issue with 'sqlite3_adapter'

I am having trouble with an issue with 'sqlite3_adapter'.  I'm currently running Ruby 2.3 and Sqlite 1.3.11 on Windows 10 x64.  I have the railsinstaller environment basically replicated but with each component installed invidually (Ruby from RubyInstaller, Git, Bundler, etc.).  I have tried to run the server for both an older, Rails 4.1 application as well as a new Rails 5 application I created for testing.  Each time, I am stuck by the error:
Could not load 'active_record/connection_adapters/sqlite3_adapter'. Make sure that the adapter in config/database.yml is valid. If you use an adapter other than 'mysql2', 'postgresql' or 'sqlite3' add the necessary adapter gem to the Gemfile.
My config/database.yml has the correct adapter.  I've checked, and I have the correct gem.  Sqlite itself is installed.  I'm not sure what the issue is here.  Is this something unique to Ruby 2.3 on Windows?  Will I need to do a fresh install? Do I need to downgrade to Ruby 2.2?  I'm just looking for more info before I spend another day on this.

This is a duplication of #185 .
The sqlite3 binary gem doesn't support RubyInstaller-2.3 so far. It's probably best to stay at Ruby-2.2 on Windows, until #186 is merged and a new version is released.

I would just like to point out that this isn't quite a duplication - while I did have the same environment as #185 , the error I'm getting is different.

Try all answers here (or don't be on Windows because Ruby support sucks): https://stackoverflow.com/questions/17643897/cannot-load-such-file-sqlite3-sqlite3-native-loaderror-on-ruby-on-rails

Unless something explicitly does not support a platform, saying to pick something else because support is lacking is, to be frank, a sucky response.  Regardless, this actually isn't a problem anymore for me given that I no longer use Railsinstaller.
Given that the corresponding fix has been merged, I would say this issue could be closed.  However, I will leave that up to the discretion of the maintainers.

I would rather say the state of support is somewhat off topic here (partly because I don't enough details to draw conclusions of any projects in particular), hence I removed that part shortly after posting. Regardless if the latest major OS version and the most common practice to setup Ruby/Rails simply fails (on many levels not just here) then I'd say it's fair to point that out and maybe even to draw general conclusions.

Except this is an issue from mid-2016, an older version of Ruby and the SQLite gem, and the proposed fix was merged in mid-2016.  Not to mention that this is no longer the most common practice to setup Ruby and Rails on Windows 10 - that's now through the Windows Subsystem for Linux.
You tried to be a smart-ass under the guise of providing a solution.  Next time, just provide the solution if it's even still needed.

If you would've thought to actually look at the information I provided you would've seen that it's relevant in regards to platform, version and time frame. And that it's also an issue in 2018. How interesting is it that..
But I guess it's my mistake to not realize that an open widely generic could not load issue with no valuable information from 2 patches down the line with no PRs referenced is exclusive to you.
And instead of valuing the information or just simply acknowledging that it might be useful for others you decided to whine about something that's literally not even there. Wish you further good luck with open source.

@rlgreen91 @thisismydesign Have you tried RubyInstaller-2.4+ especially the new version with built-in DevKit? For my taste it has gotten much better compared to the previous RubyInstaller versions.
The sqlite3 gem isn't well supported, so that it currently doesn't work with RubyInstaller-2.5 out of the box, but I posted a simple solution here.

@larskanis From my (and many others') experience the issue can be resolved by going through the answers in the referenced StackOverflow thread. I don't plan to downgrade since I found a workaround with RubyInstaller-2.5 but thanks a lot for #229 it's good to see that there're plans to fix it.

@larskanis thanks, but I'm no longer using RubyInstaller.

We're working on shipping precompiled native gems as well as increasing test coverage for people who want to use system libraries. If you're curious please follow #320.


Addition of models fro Idera ER/Studio

Overview
Links to Jira tickets

{List all applicable Jira tickets}

Description of relevant changes

addition of models fro Idera ER/Studio

PR Checklist
A list of items that are good to consider when making any changes.
Note: this list is not exhaustive, and not all items are always applicable.
Code

[ ] New files/classes/functions have appropriately descriptive names and comment blocks to describe their use/behaviour
[ ] I have avoided duplicating code when possible, moving re-usable pieces into functions
[ ] I have avoided hard-coding values where possible and moved any re-usable constants to a constants file
[ ] My code is as flat as possible (avoids deeply nested if/else blocks, promise chains, etc)
[ ] My code changes account for null/undefined values and handle errors appropriately
[ ] My code uses types/interfaces to help describe values/parameters/etc, help ensure type safety, and improve readability

Style

[ ] My code follows the established style conventions
[ ] My code uses native material-ui components/icons/conventions when possible

Documentation

[ ] I have commented my code sufficiently, such that an unfamiliar developer could understand my code
[ ] I have added/updated README's and related documentation, as needed

Tests

[ ] I have added/updated unit tests for any code I've added/updated
[ ] I have added/updated the Postman requests/tests to account for any API endpoints I've added/updated

Linting/Formatting

[ ] I have run the linter and fixed any issues, as needed
See the lint commands in package.json
[ ] I have run the formatter and fixed any issues, as needed
See the format commands in package.json

SonarCloud

[ ] I have addressed all SonarCloud Bugs, Vulnerabilities, Security Hotspots, and Code Smells

🫡


Important Bug with ControlledEnvironment and Drag&Drop.

Discussed in https://github.com/lukasbach/react-complex-tree/discussions/104

Originally posted by gpisano97 July 19, 2022
Hello, i'm using your component on React,js (17). I'm working with a big tree (with many nested children), and (with my team mates) we implemented a Controlled Environment (because we need to add and remove items), but the drag and drop logic is broken. We have filled the onDrop event and the "putting an item inside another" mechanism works. If we don't expand any Item also the reordering mechanism works, buf if we expand some Items and we try to move some child became all broken! The "target" element is wrong and in some cases (with the elements on bottom of the tree) also the "item" array is wrong!!! On "bestofreactjs.com" there is the issue #15, and seems to be similar. We are in trouble, this problems is making lose a big amount of time! How can we fix it ? Thanks all for the answers.
Best Regards.

As @madhums mentioned in the discussion, it would be helpful to see a codesandbox as an example of the specific issue, or some other details what makes the element wrong and under what circumstances this behavior can be reproduced.

I'll close this as I don't have any actionable details on this. Please note that I recently fixed some issues regarding dnd targets, which will be released in the following days, so there is a chance that might be fixed alongside this release.


Clean the crawled content before saving it to mongodb

The cleaned content, especially headings have lot of newline and trailing and leading backspace, which might introduce unnecessary padding vectors in word embeddings. Hence clean them before saving it to the database.
Additionally, scrape additional contents like some tags etc if they will be useful for searching in elasticsearch database.

I would like to take up the issue
@Tushar98644 however, i would suggest to wait till #17 is completed because this issue might create merge conflicts with that mentioned issue.

@Tushar98644  apologises for the delay you can start working on the issue now

@Tushar98644 any updates?

@Tushar98644 any updates?

haven't started working on it , will update in the next 2 days


FEAT: gutting q2-comp to re-write as R wrapper for ANCOM/ANCOMBC/SECOM

Closing this and opening a new PR (same code changes) to be merged into the ANCOM-wrapper feature branch


Cluttered JSON output in KH v0.5.2 run as K8s pod

What happened
Integrated KH with Starboard so it runs with JSOn output format as K8s pod. Then we parse the logs. I realised that in v0.5.2 praising JSON output fails due to stack trace added by KH before the actual JSON content:
2021-06-26 14:08:07,371 ERROR kube_hunter.modules.discovery.kubernetes_client Failed to initiate Kubernetes client
Traceback (most recent call last):
  File "/usr/local/lib/python3.8/site-packages/kube_hunter/modules/discovery/kubernetes_client.py", line 13, in list_all_k8s_cluster_nodes
    kubernetes.config.load_incluster_config()
  File "/usr/local/lib/python3.8/site-packages/kubernetes/config/incluster_config.py", line 118, in load_incluster_config
    InClusterConfigLoader(
  File "/usr/local/lib/python3.8/site-packages/kubernetes/config/incluster_config.py", line 54, in load_and_set
    self._load_config()
  File "/usr/local/lib/python3.8/site-packages/kubernetes/config/incluster_config.py", line 73, in _load_config
    raise ConfigException("Service token file does not exists.")
kubernetes.config.config_exception.ConfigException: Service token file does not exists.
{"nodes": [], "services": [], "vulnerabilities": [{"location": "Local to Pod (359a3672-3ed9-40df-a3bd-b24c1cf585a6-qr95w)", "vid": "None", "category": "Access Risk", "severity": "low", "vulnerability": "CAP_NET_RAW Enabled", "description": "CAP_NET_RAW is enabled by default for pods.\n    If an attacker manages to compromise a pod,\n    they could potentially take advantage of this capability to perform network\n    attacks on other pods running on the same node", "evidence": "", "avd_reference": "https://avd.aquasec.com/kube-hunter/none/", "hunter": "Pod Capabilities Hunter"}]}

Expected behavior
Error or valid JSON output. Not both. Otherwise it's hard to determine whether KH failed or succeeded.
I did not bump into such issue with previous v0.4.1

@danielpacak
Thats because of a bug in a new feature weve added.
But anyways, by your question I understand you run kube-hunter with --log error. A better practice that will also solve this issue is running with --log none.
This way even if we have some error internally with kube-hunter it will not mess up the report output

Got it. Thanks for the hint. We'll update the log level appropriately.


bug(MatExpand): Newly added item to panel not toggle able

Reproduction
Use StackBlitz to reproduce your issue: https://stackblitz.com/edit/angular-tn6hwr
Steps to reproduce:

Open console then type window.foo()

Expected Behavior
The last added panel is not toggle-able on click of the header, you must click outside for it to toggle properly.
Actual Behavior
Should toggle on click of header
Environment

Angular: 10.1.3
CDK/Material: 10.2.2

You might why call window.foo()
window.foo() is simulation data coming from external BT device.

It's because foo is being run outside of the Angular zone. You can fix it by changing it to this:
constructor(private cdr: ChangeDetectorRef, ngZone: NgZone) {
  window["foo"] = () => ngZone.run(() => this.add());
}


Test the shell installer

The default installer doesn't seem to work well, you will need to revalidate with a fresh install

Fix in aed7098


fix(typescript): export declare the Logger class

Coverage remained the same at 98.413% when pulling af6e012c1a7296934cebf72df26ab1f9f845899c on export-declare-logger-class into 73f542449560b98ac2efb5d9a9adf734cf2406f2 on master.


[DNM] SIR-686 - Bump build number

This bumps the build number to trigger a build in prefect, verifying that teh secret change in SIR-686 works.

Linter check found the following problems:
The following problems have been found:
ERROR: clone/recipe/meta.yaml:0: missing_tests: No tests were found.
ERROR: clone/recipe/meta.yaml:0: missing_section: The requirements section is missing.
ERROR: clone/recipe/meta.yaml:15: missing_summary: The recipe is missing a summary
ERROR: clone/recipe/meta.yaml:15: missing_documentation: The recipe is missing a doc_url or doc_source_url
WARNING: clone/recipe/meta.yaml:15: missing_description: The recipe is missing a description
ERROR: clone/recipe/meta.yaml:15: missing_dev_url: The recipe is missing a dev_url
Errors were found


[Feature request] Last executed command from command launcher

Hi,
As Command launcher became my primarily way to execute commands with Vscode, I would find handy to have to possibility to have the last executed command from Command launcher listed always on top of the Command launcher list (for easy access).
This isn't an essential feature but I think it would still be handy
I'm currently writing an extension of my own (to manipulate files) and if you're not interested in developing this suggestion, I may at some point in the future develop that feature and create a pull request . Would that be a feature you may accept for your extension in the future ?
Thanks

Hi BachiMjavanadze.
I'll answer for your feature request and question.
In conclusion, your feature request and pull request is rejected.
Both reasons are due to the test code.
The test code was also the reason for the rejected of the previous feature request.
It saddens me to say this, but the test code in this repository is terrible.
It takes the same amount of time as implementation, maybe even more.
But we also consider it important because of the frequency of VSCode's upgrade and the variety of operations.
If you are going to submit a pull request, test code is a must.
But it will be impossible. I am sure of it, I am sick of it.
Thanks for reporting the issue.
If there are no additional questions, I'll close this issue in about week.

Ok no problem 👍
(I'm not BachiMjavanadze even though I wrote [Feature request] the same way ^^)


EOS-25818: State machine integration

Problem Statement

Problem statement

Design

For Bug, Describe the fix here.
For Feature, Post the link for design

Coding
Checklist for Author

[ ] Coding conventions are followed and code is consistent

Testing
Checklist for Author

[ ] Unit and System Tests are added
[ ] Test Cases cover Happy Path, Non-Happy Path and Scalability
[ ] Testing was performed with RPM

Impact Analysis
Checklist for Author/Reviewer/GateKeeper

[ ] Interface change (if any) are documented
[ ] Side effects on other features (deployment/upgrade)
[ ] Dependencies on other component(s)

Review Checklist
Checklist for Author

[ ] JIRA number/GitHub Issue added to PR
[ ] PR is self reviewed
[ ] Jira and state/status is updated and JIRA is updated with PR link
[ ] Check if the description is clear and explained

Documentation
Checklist for Author

[ ] Changes done to WIKI / Confluence page / Quick Start Guide

View rendered README.md
View rendered Testing_Verification_History.md

Can one of the admins verify this patch?

@mssawant  for now I am closing this . It is a draft PR and task also is open.


Segfault when null expression given to cmsketch_frequency

create stream s (x integer);
create continuous view v as select cmsketch_agg(x) from s;
insert into s (x) values (10);
select cmsketch_frequency(cmsketch_agg, null) from v;

Segmentation fault (PID 20124)
version: PipelineDB  at revision  on x86_64-pc-linux-gnu, compiled by gcc (Ubuntu 4.9.2-0ubuntu1~14.04) 4.9.2, 64-bit
backtrace:
pipelinedb: derek derek [local] SELECT(debug_segfault+0x25)[0x7213e9]
/lib/x86_64-linux-gnu/libpthread.so.0(+0x10340)[0x7f8610503340]
pipelinedb: derek derek [local] SELECT(DatumToBytes+0x4b)[0x786f55]
pipelinedb: derek derek [local] SELECT(cmsketch_frequency+0xec)[0x97123a]
pipelinedb: derek derek [local] SELECT[0x69832b]
pipelinedb: derek derek [local] SELECT[0x698d42]
pipelinedb: derek derek [local] SELECT[0x69f642]
pipelinedb: derek derek [local] SELECT(ExecProject+0x305)[0x69fc4d]
pipelinedb: derek derek [local] SELECT(ExecScan+0x18a)[0x69ffa6]
pipelinedb: derek derek [local] SELECT(ExecSeqScan+0x22)[0x6bf324]
pipelinedb: derek derek [local] SELECT(ExecProcNode+0x2b2)[0x6946a1]
pipelinedb: derek derek [local] SELECT(ExecutePlan+0x65)[0x691830]
pipelinedb: derek derek [local] SELECT(standard_ExecutorRun+0x16b)[0x68f5e9

Since adding NULL to a cmsketch is a noop, the only question is what we return for:
cmsketch_frequency(cms, NULL);

0 would be correct, but not necessarily intuitive since cmsketch_agg silently accepts NULL. The other option is to return NULL.

Another option is to disallow NULL for both cmsketch_agg and cmsketch_frequency.


Switch to use static metrics for more metrics

[x] Waiting for PR merge: https://github.com/pingcap/rust-prometheus/pull/166
[x] Waiting for PR merge: https://github.com/pingcap/rust-prometheus/pull/169
[x] Benchmark

Inspected around 3% improvement in every storage ops.

Cool

how about context switch?

@overvenus @hicqu PTAL

Too old. Will open a new PR.


Support prometheus metrics

Closes #89

Closing this PR, because the code is already present in a different PR (https://github.com/strongdm/log-export-container/pull/102)


Support non-fixed schema for indexed document

First step: I added a document {text: 'something', author: 'rua'}
Second ste: I added a document with new field {text: 'something', author: 'rua', type: '1'}
Now, I query http://localhost:20220/v1/indexes/candidates/search?q=author:rua&fetch=*, I got 503 Service unavaiable
I think I can update rest of document with new field, but that is not a best way.

Got it, will check & fix.

This happens when the engine is restarted without "cleaning" the persisted index data.
When you start the engine and there is previous index data, it could (they match) pick up the old documents and those are not in the in-memory storage, so an exception is raised when trying to fetch said documents.
Can you try the same test with a clean index (target/base should be removed before launching the engine)?
I will also update that exception's visibility.

Dear adrani,
Thanks for quickly reply. I'm also sorry for my basic English. But in my scenario, I did not restart server, I just added documents one by one. Just the second document had new field.

This is what I've tried to verify the issue doesn't reproduce (unless the server is restarted):
curl -d "{"docid":"post2", "fields":{"text":"I love Fallout"}}" -v -X PUT http://localhost:20220/v1/indexes/idx/docs
curl -d "{"docid":"post1", "fields":{"text":"I love Planescape Torment"}}" -v -X PUT http://localhost:20220/v1/indexes/idx/docs
curl -d "{"docid":"post1", "fields":{"text":"I love Planescape Torment", "type": "1"}}" -v -X PUT http://localhost:20220/v1/indexes/idx/docs
Then this queries:
http://localhost:20220/v1/indexes/idx/search?q=love&fetch=*
http://localhost:20220/v1/indexes/idx/search?q=I+love+Planescape+Torment&fetch=*
With a fresh index, that works. If your test is substantially different please add the full details to reproduce it.

Dear adrani,
You're right. Many thanks.
But if I delete target/base directory, so

How can I remain my index data? Are there any options when start index?
What are difference between Standalone RestAPI and Thrift api? I mean that:  class:com.flaptor.indextank.index.IndexEngine and class: com.flaptor.indextank.api.Launcher
How can I backup index data?

Regards,

Index data can be retained by replacing the in-memory storage with a persistent implementation. I'm working on providing that option.
By design, a user access an index through the REST API. Across the platform stack we've used Thrift to integrate the engine. The standalone API option skips the Thrift layer and exposes the REST API directly.
Gracefully: index structure is persisted with a shutdown hook.
Ungracefully: a recovery could take place (ex.: fetching documents from the persistent storage mentioned before).

Awesome! Thanks for your help.


long files are leading to memory crashes (on gpu)

I am trying to transcribe long files. When it gets to about an hour or so, there's about a 50% chance that it will crash with the error CUDA failed with error out of memory. I have a Nvidia RTX 3070 which has eight gigabytes of VRAM which should be sufficient, right? When I am transcribing, VRAM usage stays below 65%. It is very odd that it says that it is running out. I have tried both int8 and float16 but it doesn't seem to change anything. I'm using the large-v2 model. (Sorry if something like this already exists I couldn't find anything.)
Update: It is very random, it appears that the time does not matter. Sometimes it'll transcribe a two-hour file completely fine and other times it will fail on a 20-minute file.

Try --best_of=1.

The fact that the error CUDA failed with error out of memory happens randomly makes me suspect there might be some other programs running on the background that happen to use the gpu. There are too many possibilities that can lead to that.
What I can propose is perhaps splitting the audio manually into smaller files so that each time it crashes you don't lose as much?
Alternatively, perhaps you can attempt to do the transcription with a smaller model?
Note that VRAM usage do spikes at times.

Try best_of=1.

Thanks for the suggestion but it didn't seem to help.

The fact that the error CUDA failed with error out of memory happens randomly makes me suspect there might be some other programs running on the background that happen to use the gpu. There are too many possibilities that can lead to that. What I can propose is perhaps splitting the audio manually into smaller files so that each time it crashes you don't lose as much? Alternatively, perhaps you can attempt to do the transcription with a smaller model? Note that VRAM usage do spikes at times.

I have thought of that and I've tried closing all other apps on my PC to no avail. The spike seems to be undetectable by task manager so I have no idea what could be causing it.

Where and how do you run it?

You could say that from the start... Try to update wsl to latest, run this in console: wsl --update --pre-release

That actually worked, thanks. Such as silly issue though, probably should have checked before I made a report.


Extrusion

Objective

Adds a basic Extrusion<T: Primitive2d> shape, suggestion of #10572

Solution

Adds Measured2d and Measured3d traits for getting the perimeter/area or area/volume of shapes. This allows implementing .volume() and .area() for all extrusions Extrusion<T: Primitive2d + Measured2d> within bevy_math
All existing perimeter, area and volume implementations for primitves have been moved into implementations of Measured2d and Measured3d
Shapes should be extruded along the Z-axis since an extrusion of depth 0. should be equivalent in everything but name to the base shape

Caviats

I am not sure about the naming. Extrusion<T> could also be Prism<T> and the MeasuredNd could also be something like MeasuredPrimitiveNd. If you have any other suggestions, please fell free to share them :)

Future work
This PR adds a basic Extrusion shape and does not implement a lot of things you might want it to. Some of the future possibilities include:

[ ] bounding for extrusions
[ ] making extrusions work with gizmos
[ ] meshing

Thank you to everyone involved with the authoring or reviewing of this PR! This work is relatively important and needs release notes! Head over to bevyengine/bevy-website#1312 if you'd like to help out.

I think it would make a lot of sense to merge either #13478 or #13346 first, because as it stands you really cannot do anything with extrusions


Could not find shared library libiperf.so.0 (on OSX)

This is a great wrapper and I'd love to use it on my Mac Book Pro (OSX Sierra 10.12.16).  Are there any workarounds/fixes for the fatal error on OSX where it can't find libiperf.so.0 ?

@jmack51 Thanks to @Austinpayne this should now work in the latest release on PyPi.


Add an info field to transcript effect record

As stated in issue https://github.com/ga4gh/schemas/issues/623, the current transcript effect model loses fidelity for some information generated by SnpEff. This PR adds an info field in a spirit similar to the rest of the API which maintains key:[value] pairs for modeling information that do not have named fields.
This change allows a wider variety of data to be modeled and induces minimal changes to existing implementations.
Here is the mapping provided in #623 that shows which fields are not being represented. With this PR, all data presented by a ? would be appended to the info field.

Allele -> alternateBases
Annotation -> effects[0].term (may we have several terms in this list?)
Annotation_Impact -> ?
Gene_Name -> ?
Gene_ID -> ?
Feature_Type -> ?
Feature_ID -> featureId
Transcript_BioType -> ?
Rank -> ?
HGVS.c / HGVS.p -> hgvsAnnotation.genomic / hgvsAnnotation.protein
cDNA.pos / cDNA.length -> cDNALocation.start / cDNALocation.end
CDS.pos / CDS.length -> CDSLocation.start / CDSLocation.end
AA.pos / AA.length -> proteinLocation.start / proteinLocation.end
Distance -> ?
ERRORS/WARNINGS/INFO -> ?

@david4096 - I missed the earlier discussion, but a number of these values are attributes of the gene or transcript which I expect would be retrieved via the sequence annotations API using the transcript id rather than duplicated in TranscriptEffect.
That said, there is no harm in an additional info field, as long as important, commonly used 'info's are identified and promoted to named attributes.

@sarahhunt agreed! @GuilhemSempere had provided the above list. I believe the fields that are not currently described in feature messages would be Distance,
ERRORS/WARNINGS/INFO, and Annotation_Impact. This provides a consistent way to continue to interchange these data without enforcing them as fields.

@sarahhunt can I get your +1 ?

+1. Any other comments @pcingola, @andrewjesaitis ?

+1
The only thing that jumped out to me is in the #623 comment. I believe it should be HGVS.c -> hgvsAnnotation.transcript, since this is the annotation relative to the transcript and not to the genome. Everything else looks good.
I agree on the data duplication point that @sarahhunt brought up, but since these info fields are just an arbitrary map, it can be seen as an implementation detail (that is adding/checking what is a "sequence annotation" field and adding the rest here).

Thanks @andrewjesaitis! Good point- I'd not noticed the HGVS.c ->hgvsAnnotation.genomic misassignment.


[Subtask] Support tag pre event to Gravitino server

Describe the subtask

Add corresponding TagPreEvent
Generate pre-event in TagEventDispatcher

Parent issue
#5899

@TungYuChiang, Hi, I would like to try to work on this one. Would you be able to assign this to me? Thank you!

hi @schoolLychee Thank you for your interesting this issue, I assign this issue to you.
Please go ahead, Thanks.

@xunliu, sorry, I used the wrong account. Could you kindly reassign to this one?


Add User field in Filter Nomenclature task

This was discussed in the most recent webinar for UCD@TW.
It would be helpful for administrators for the UCD project (and doubtless others) to have a "User" field in the Filter Nomenclature task.  This would allow us to audit the work of particular users for quality control purposes.

@jlpereira We need to unify the creator/update/user/date facet across the filters.


Improve Hash formatting in EnglishPhrasing.list

We're currently treating all Enumerables as an Array in this method,
resulting in a Hash being formatted as a list of two-element key-value
arrays.
This results in unexpected formatting in the default failure message for
a custom matcher when the expected value is a Hash.
This commit updates EnglishPhrasing.list to no longer format a Hash as
if it were an Array.
Fixes #1192.
Before:
Failure/Error: expect(nil).to custom_match({a: 1, foo: 'bar', bang: Object.new})
  expected nil to custom match [:a, 1], [:foo, "bar"], and [:bang, #<Object:0x00007fb89542bad0>]

After:
Failure/Error: expect(nil).to custom_match({a: 1, foo: 'bar', bang: Object.new})
  expected nil to custom match {:a=>1, :bang=>#<Object:0x00007f9f363b0b28>, :foo=>"bar"}

Thank you!
You're on the wall of fame now.


Update object

Related to PR #85.
Runs a function for the given object and sets the result on the object.
I did a couple of things different from the original PR. I made the default value optional. If it's provided, it's passed to objectPath.get. I also removed the modifyDefault parameter, as I wasn't super sure what it was used for.

Coverage increased (+0.009%) to 99.731% when pulling 03623c7f8fa160232bc09d1187881fc267c45642 on j-rewerts:update-object into d680fe7a7db142e3754c83a80225453fef19b5fb on mariocasciaro:master.

Coverage increased (+0.009%) to 99.731% when pulling 03623c7f8fa160232bc09d1187881fc267c45642 on j-rewerts:update-object into d680fe7a7db142e3754c83a80225453fef19b5fb on mariocasciaro:master.

Coverage increased (+0.009%) to 99.731% when pulling 03623c7f8fa160232bc09d1187881fc267c45642 on j-rewerts:update-object into d680fe7a7db142e3754c83a80225453fef19b5fb on mariocasciaro:master.


NATS single node guarantees?

Hi all,
I'm wondering if, in a single node setup of NATS, the following guarantees hold:

a publisher will be notified if they failed to publish a message if their Flush() operation failed (the publisher will know if their connection was interrupted to NATS during publish)

a subscriber will always receive a Flush()ed message or encounter a connection error (the message may be lost, but the subscriber will know it is lost)

I understand NATS only has at-most-once delivery guarantees, but I am wondering if it has these "once-or-error" semantics when running as a single node – and if not, under what scenarios will this assumption be violated.

Hi,
Flush() sends a PING to the server, forces a flush of the underlying
buffered TCP connection, and awaits a PONG response, thus ensuring that any
previously buffered messages on the connection have been sent to the
server. An error is generated if no PONG is received within the timeout
period (default 60s in most clients - use FlushTimeout() if you want to
tweak that).
So yes, if Flush() succeeds, the message will have made it to the server
successfully. If Flush() fails, you have no such guarantee, although
technically it’s still possible the message made it, but the PING didn’t.
On the subscriber side, the subscriber should receive the message if it is
not a “slow consumer” (i.e. it’s servicing its connection — draining
messages —  in a timely fashion). If the subscriber is deemed a slow
consumer by NATS, messages may be lost or the connection may be dropped.
The client will not be notified that a specific message has been dropped,
however, since NATS messages have no uniquely identifynig info (unlike NATS
Streaming).
If you want to be notified of a slow consumer problem, register an async
error handler for your connection; this error handler (callback) will be
invoked when a slow consumer problem is encountered.
Make sense?
-Larry
On January 25, 2017 at 1:42:35 AM, rfw<EMAIL_ADDRESS>wrote:
Hi all,
I'm wondering if, in a single node set up of NATS, if the following
guarantees hold:

a publisher will be notified if they failed to publish a message if
their Flush() operation failed (the publisher will know if their connection
was interrupted to NATS during publish)
a subscriber will always receive a Flush()ed message or encounter a
connection error (the message may be lost, but the subscriber will know it
is lost)
I understand NATS only has at-most-once delivery guarantees, but I am
wondering if it has these "once-or-error" semantics when running as a
single node.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nats-io/go-nats/issues/256, or mute the thread
https://github.com/notifications/unsubscribe-auth/AGq4LONZZSfKi9KIr3pXNTbmYlSxpfC8ks5rVwr7gaJpZM4LtOaC
.

@rfw, Thank you for your interest in NATS!
Regarding the Flush() operation, should the server goes down as you are calling flush, you'll get a connection error (ErrInvalidConnection, ErrConnectionClosed), or a timeout error (ErrBadTimeout).  Note, this indicates the flush may have failed, not necessarily that it did with respect to what happens in the server.
Flushing does not guarantee a message will be received by a subscriber.  From the producer side, a successful flush only indicates that the server has received the message, not that subscribers have received it.  Consider the case where the server has received the published message, buffered the message to send to the subscriber, and has responded to the producer's flush, but is terminated before the message can be sent to the subscriber.
To achieve a guarantee for your application, I suggest you check out NATS streaming, or you can verify through request/reply that all of your data has been received.
Can you share more of your use case?
Thanks,
Colin

Hi Colin and Larry,
Thanks for your replies. My use case is to optimistically broadcast updates of my data store to subscribers – I don't care if messages are lost or really even received out of order, I just need to make sure that if a message was lost, the client can resubscribe and also re-fetch the latest state of all data they're interested in from the store.
If the server was terminated before the message was sent to the subscriber, would the subscriber know that the server was terminated and therefore they would have missed an update?
NATS streaming seems a little overkill for the guarantees I need – I don't particularly need persistence, durability, or replay here.

@rfw, Absolutely, happy to help!

If the server was terminated before the message was sent to the subscriber, would the subscriber know that the server was terminated and therefore they would have missed an update?

Yes, the subscriber can detect a broken connection through callbacks setup on the NATS connection:
// Setup callbacks to be notified on disconnects, reconnects and connection closed.
nc, err = nats.Connect(servers,
    nats.DisconnectHandler(func(nc *nats.Conn) {
        fmt.Printf("Got disconnected!\n")
    }),
    nats.ReconnectHandler(func(_ *nats.Conn) {
        fmt.Printf("Got reconnected to %v!\n", nc.ConnectedUrl())
    }),
    nats.ClosedHandler(func(nc *nats.Conn) {
        fmt.Printf("Connection closed. Reason: %q\n", nc.LastError())
    })
)

In your case, this should be very straightforward - when a reconnect handler is invoked, you know there was an interruption of service, but the connection has been re-established, and you can then fetch any missing data.

Perfect, thanks for your help!


Sticky scroll: folding icons missing from some sticky headers

Type: Bug

Issue troubleshooting has identified that the issue is with Visual Studio Code - Insiders.

If the sticky scroll header shows only one line, that line has no folding indicator.
If it shows 2 or more lines, only the last line has the indicator.

This is working correctly in 1.82.0 Stable.
VS Code version: Code - Insiders 1.83.0-insider (5a400e53e985dc5e24f6ee574b07ab23943841c5, 2023-09-07T17:35:42.740Z)
OS version: Windows_NT x64 10.0.22621
Modes:

Hi @gjsjohnmurray, thank you for filing the issue. I was looking at this issue right at the moment and was about to file a bug but you did it before me. I think this issue may have been caused by https://github.com/microsoft/vscode/pull/188499, I will work on a fix.


keep_quoted is not a supported option

So I'm trying to use gulp-terser and which to use the keep_quoted like::

function terser() {
    return gulp.src('js/**/*.js')
        .pipe(gTerser({
            keep_fnames: false,
            keep_quoted: true,  // <====== doesn't work here
            mangle: {
                 keep_quoted: true,  // <==== neither here
                properties: true,
                keep_fnames: false,
            }
        }))
        .pipe(gulp.dest(`${destStatic}/js/`))
}

but sadly keep getting error:  keep_quoted is not a supported option

You can try this configuration：
function terser() {
    return gulp.src('js/**/*.js')
        .pipe(gTerser({
            keep_fnames: false,
            mangle: {
                properties: {
                    keep_quoted: true
                },
                keep_fnames: false,
            }
        }))
        .pipe(gulp.dest(`${destStatic}/js/`))
}

Thanks for fast reply Duan, that solves the issue!


Add OWNERS file

Onboarding this new component into the SR-IOV network operator.
Refs

https://docs.ci.openshift.org/docs/how-tos/onboarding-a-new-component/
https://issues.redhat.com/browse/CNF-11148

/lgtm
/approve

/approve

/approve
/lgtm
If this doesn't work you have to add the OWNERs file to openshift/release
E.g. https://github.com/openshift/release/pull/48085 @zeeke / @SchSeba
If you think you will need either me or @bn222 . You can add us to the owners as well.

We need @vrindle in the maintainers too. Also add @SalDaniele for good measure.

/approve /lgtm
If this doesn't work you have to add the OWNERs file to openshift/release E.g. openshift/release#48085 @zeeke / @SchSeba
If you think you will need either me or @bn222 . You can add us to the owners as well.

Thanks @wizhaoredhat . I already made changes to https://github.com/openshift/release/pull/48448 , but the very first commit of OWNERS has to be merged by some admin. (asking internally)
Updating the PR with you all, thanks for jumping in

/lgtm
/approve

/approve


All AR/ARIMA are disabled

AR/ARIMA models refit each validation block prediction that may be time consuming. It needs to be fixed and AR/ARIMA models should be enabled.
Also test.unit.api.test_presets.test_presets_time_series should be fixed at TODO mark.

Update. AR is switched because it is fast (https://github.com/aimclub/FEDOT/pull/1153).


Fixing Dewback Rider instructions

Ability name "SHOCK LANCE:" is missing from Dewback Rider instructions.
1 of the dewback rider bonus effects is mentioning this ability name, hence why it needs to be added to the instructions:
https://github.com/GlowPuff/ImperialCommander2/blob/bf899b0866435be75644233fc2ca9928bde223a8/ImperialCommander2/Assets/Resources/Languages/En/bonuseffects.json#L77C4-L77C100

@Noldorion as FYI as it may need to be checked (and possibly fixed) for other languages as well.


viewer introduces spurious composite binding sites

The issue on the Intact Editor at http://www.ebi.ac.uk/intact/jira/browse/EDITOR-35 is actually a bug in the viewer.
The viewer is introducing a composite binding site where there shouldn't be one.

#85 should fix this. Will wait for confirmation everything's OK before closing.
The viewer still shows a genuine composite binding site (as in EBI-10828997) correctly.
However, the code cannot deal with a composite binding site that is across two different participants - I think its going to be v2.0.0 before that's fixed.

I think that's working now.

OK, lets try to fix #86 also before doing another release,
C
On 22/02/16 11:44, Birgit wrote:

I think that's working now.
—
Reply to this email directly or view it on GitHub
https://github.com/MICommunity/interaction-viewer/issues/84#issuecomment-187137511.

The University of Edinburgh is a charitable body, registered in
Scotland, with registration number SC005336.

I wasn't sure if they were linked so made it a new issue just in case.

they're not linked...
On 22/02/16 11:55, Birgit wrote:

I wasn't sure if they were linked so made it a new issue just in case.
—
Reply to this email directly or view it on GitHub
https://github.com/MICommunity/interaction-viewer/issues/84#issuecomment-187140837.

The University of Edinburgh is a charitable body, registered in
Scotland, with registration number SC005336.


Issue with PyInstaller

According to http://pyinstaller.47505.x6.nabble.com/Help-Failed-to-execute-script-td2705.html apparently there is an issue with workalendar's init that makes it incompatible with PyInstaller.
I tried their "very hacky fix"  and it works. It would be nice if you could fix your init.
I tried with python 3.9.5 and workalendar 15.3.0.
Thank you in advance.
SD

Would it be ok if we replaced this __init__ with an importlib_metadata with a fallback and a dependency to importlib_metadata only on older Pythons ?

(Also, if yes, would you be interested in doing it ?)

I have no clue how to fix things. I use python as a tool for data analysis and I don't have enough experience.
If needed I could help testing solutions (but as I said I lack of experience).
Thanks

Ok if @brunobord agrees, I'll do the modification, which I believe is a good idea anyway, and once it's released, you can try again and tell us if it solved your problem.

Thank you for your fix. I tried the new version but the problem is still there. I guess the fix will be merged in future releases. Thank you again for your time.

Yes, the PR above hasn't been merged yet.
There are multiple ways to test this:

wait for the PR to be merged and a subsequent release of workalendar
build the package wheel yourself from my branch and use it to test
build the package yourself and upload it to a dedicated index that you'll use (if you want some more info for this, let me know)

I builded the package from your branch and it works!
Thanks again,
SD

Hm, I suggest we reopen until my PR is merged...

I think I'm going to make a Release Candidate and publish it on PyPI.
Then you'll be able to use it and test if nothing's broken in the meantime.
Are you okay with this?

That would be great. Thank you for the support.
SD

I confirm that in the branch release-candidate/15.5.0rc1 the pyinstaller issue is solved.

@SimoneD89 that's super cool. If I may, could you please make one tiny last test using the release candidate files? https://pypi.org/project/workalendar/15.5.0rc1/

The compiled scripts work fine with workalendar 15.5.0rc1 installed from pypi. Thank you for your support!
Summary:
workalendar 15.4.0: compiled scripts are not executing
workalendar 15.5.0rc1: pyinstaller issue solved and compiled scripts work fine


Phar link is broken

Link from readme (https://phar.phpunit.de/foal.phar) contains 404 Not Found.

This link will work once a (1.0.0) release has been made.


SP2 failing all scenarios

On May 19th, all scenario player runs at SP2 failed with the same/similiar message:
Traceback (most recent call last):
  File "/scenario-player/scenario_player/tasks/base.py", line 140, in __call__
    return_val = self._run(*args, **kwargs)
  File "/scenario-player/scenario_player/tasks/execution.py", line 36, in _run
    task()
  File "/scenario-player/scenario_player/tasks/base.py", line 140, in __call__
    return_val = self._run(*args, **kwargs)
  File "/scenario-player/scenario_player/tasks/execution.py", line 36, in _run
    task()
  File "/scenario-player/scenario_player/tasks/base.py", line 140, in __call__
    return_val = self._run(*args, **kwargs)
  File "/scenario-player/scenario_player/tasks/api_base.py", line 59, in _run
    f'HTTP status code "{resp.status_code}" while fetching {url}. '
scenario_player.exceptions.RESTAPIStatusMismatchError: HTTP status code "402" while fetching http://<IP_ADDRESS>:39457/api/v1/channels. Expected 201: {"errors": "The account balance is below the estimated amount necessary to finish the lifecycles of all active channels. A balance of at least<PHONE_NUMBER>102393600 wei is required."}

The SP wallet on SP2 has currently 80+ goerliETH

This happens on both SP1 and SP2 now.

We are giving 0.3 ETH to each node, but the errors indicate that more ETH (1.03 in the case of the error in the top message) is needed.

Goerli had frequent congestion issues recently. With that in mind, theses errors might be reasonable, depending on the actual gas prices at the time. I can't find a data source for goerli gas prices, so this is still mostly speculation.

@karlb, @konradkonrad is this issue still relevant?

This really was just the goerli network having problems.


Unable to mvn build sample application

Framework version: XX
Implementations: Spring Boot

Scenario
Attempting to test Lambda functionality using quick start instructions in wiki.
Expected behavior
When I run mvn clean install, the application should build itself into a jar and/or zip file ready to upload to Lambda.
Actual behavior
Maven gives the following error on build:
Failed to execute goal org.apache.maven.plugins:maven-install-plugin:2.5.2:install (default-install) on project <artifactId-name>: The packaging for this project did not assign a file to the build artifact -> [Help 1]
Steps to reproduce

Run the command in the wiki:

mvn archetype:generate -DgroupId=my.service -DartifactId=my-service -Dversion=1.0-SNAPSHOT \
       -DarchetypeGroupId=com.amazonaws.serverless.archetypes \
       -DarchetypeArtifactId=aws-serverless-springboot-archetype \
       -DarchetypeVersion=1.4

cd into the created directory where the pom.xml file is stored.
Run mvn clean install.

Thanks for the report @lyle-partsavatar - we'll look into it for the next release. Does mvn clean package work?

Yeah, mvn clean package worked.  Once I read the README and actually did it then it worked XD . Sorry for being dumb.

No problem. I'll leave this issue open to make sure that we fix mvn install in the archetypes for the next release.

Switched this to wontfix since it doesn't make sense to enable install for applications such as these. We'll update the documentation to clarify that the command is mvn clean package.