S3 Bucket constantly calls PutBucketNotification

What happened?

The issue is similar to this issue, however, I've hardcoded the ID in the notifications queue configuration and we're using version 0.21.2 which should have this fix in place:
https://github.com/crossplane/provider-aws/issues/476
I've created a queue and it continuously tries to put in the notifications configuration, generating a test message every time.
How can we reproduce it?

Create a bucket with a notifications queue, check cloudtrail logs to see that it's making an api call every minute.
What environment did it happen in?
Crossplane version:

@empath-nirvana are you using queueArnRef to get the queue? If you so, this might have been fixed in #1161. Can you check again with the latest master?

Thanks @empath-nirvana. I was able to reproduce the bug with your YAML.
It seems the cause of this issue is that the prefix filter rule is converted to camel case Prefix which generates a diff:
  []types.QueueConfiguration{
  	{
  		... // 1 ignored and 1 identical fields
  		Filter: &types.NotificationConfigurationFilter{
  			Key: &types.S3KeyFilter{
  				FilterRules: []types.FilterRule{
  					{
- 						Name: "Prefix",
+ 						Name: "prefix",
  						... // 2 ignored fields
  					},
  				},
  				... // 1 ignored field
  			},
  			... // 1 ignored field
  		},
  		... // 2 ignored fields
  	},
  }

This seems to be an issue on AWS side:
{
    "QueueConfigurations": [
        {
            "Id": "...",
            "QueueArn": "...",
            "Events": [
                "s3:ObjectCreated:*"
            ],
            "Filter": {
                "Key": {
                    "FilterRules": [
                        {
                            "Name": "Prefix",
                            "Value": "some-prefix"
                        }
                    ]
                }
            }
        }
    ]
}

The SDK itself also only defines prefix and suffix as correct values.

Is there a way to work around this that doesn't involve waiting for a patch to the aws sdk for go?

Its a bit difficult because the controller uses go-cmp to compare the configurations and it does not have an ignore case option AFAIK.
@muvaf what would be the best way to circumvent this issue?

What happens if you change the enum to allow "Prefix" and "Suffix"?  Does s3 only accept lower case?

Given that aws-sdk has said they won't fix the issue, is there some way this can be fixed by the cross-plane provider?

Since its a bug at AWS it would be best to wait for them to fix it on their side.
On provider level you could only add a hacky workaround by converting the respective fields to lowercase strings or find a way to ignore case when doing cmp.Equal.

I talked to someone on the S3 team. It’s extremely unlikely they’re going
to modify S3 api behavior.
On Thu, Mar 10, 2022 at 6:32 AM MisterMX @.***> wrote:

Since its a bug at AWS it would be best to wait for them to fix it on
their side.
On provider level you could only add a hacky workaround by converting the
respective fields to lowercase strings or find a way to ignore case when
doing cmp.Equal.
—
Reply to this email directly, view it on GitHub
https://github.com/crossplane/provider-aws/issues/1165#issuecomment-1063959137,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUVYTL5PMTCDTPKHU6PIIUTU7HMTHANCNFSM5O7OYNCQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

--
“The world is not comprehensible, but it is embraceable.”

Facing same issue but with TopicConfiguration.Filter.Key.FilterRules.Name. Crossplane is infinitely calling the PutBucketNotification on our bucket. Seems the same fix need to be applied to TopicConfiguration as well.


Feature: ctrl tab shortcut

Description
Briefly, I added a ctrl-tab behavior to Bruno with a dedicated UI (a popup to show which tab to switch to)
To do so I had to store a stack of recent tabs in order and I added a component for the UI.
I had to extract the special tabs names map to allow usage in the ctl tab feature and tweaked the name of the collection setting tab which was (in my opinion) not descriptive enough.

Issue: https://github.com/usebruno/bruno/issues/736
https://github.com/usebruno/bruno/assets/28385941/009c1741-fb14-4782-b555-7ac668238ff2

One thing I'm thinking about is I only added the collection name in addition to the tab name, but we might want complete path (folders) as well
Contribution Checklist:

[x] The pull request only addresses one issue or adds one feature.
[x] The pull request does not introduce any breaking changes
[x] I have added screenshots or gifs to help explain the change if applicable.
[x] I have read the contribution guidelines.
[x] Create an issue and link to the pull request.

Publishing to New Package Managers
Please see here for more information.

I added folder path to ctrl Tab as it felt needed for large collections:
https://github.com/usebruno/bruno/assets/28385941/253e099b-827b-44ea-9232-bd00e6e68622

Hi @helloanoop,
just rebased this pr I made some time ago that add ctrl+tab to the Bruno electron interface since it's a feature I missed quite a lot when I switched from Insomnia to Bruno. Should be quite painless to merge, I tested it quite extensively again.
Cheers and a ton of strength, I just looked at the number of opened pull request :sweat_smile:

Sorry for being so late to review this @Jeremie-Chauvel !
We are reviewing this pull request. We are aligned with the UX updates that have been proposed and the Ctrl+Tab functionality.
We expect to have this merged in the next few days.

Thanks @Jeremie-Chauvel !
No need to rebase. I have merged them to a feature branch. There are a couple of improvements that we want to make on the state management of the tabs.  Will provide an update later today.


Selenium.WebDriverException: The HTTP request to the remote WebDriver server for URL when running the test using TFS "run functional tests" task

Hi,
We use "Visual Studio Test Agent Deployment" and "Run Functional tests" tasks in TFS to set up the test agent on a test machine and run selenium tests there. The test agent is set up not to run as "interactive process".
Before last week, all the tests run fine in this set up. However, a few days ago, all tests started failing with error message like this:
OpenQA.Selenium.WebDriverException: OpenQA.Selenium.WebDriverException: The HTTP request to the remote WebDriver server for URL http://localhost:19509/session timed out after 60 seconds.
On the same machine, I can run the tests fine if I invoke the tests using vstest.console.exe from command line. Running the tests in visual studio works as well.
I tried to use "interactive process" mode. The thing is I have to log on to the test server and keep the session open otherwise the set up agent process keeps restarting the test machine, account used to set up the test agent is an admin user account, We are not using CodedUI, just plain selenium.
We were able to run the tests fine without checking "interactive process" for months. We are on TFS 2015 update 1, no recently TFS or agent update. RunVisualStudioTestsusingTestAgent task version is 1.0.22. We specified the test agent location and "Update Test Agent" is not checked, so there is no update to the version of test agent right. There was no code change to tests themself either.
I couldn't find out why tests started failing with no obvious configuration or code changes. Does anyone see this issue before or know a solution?
Thanks,
Di

My guess would be the browsers got updated and need a newer version of the driver. Nothing has changed from the test agent side...


Add tests to geometry_utils

[x] Tests for are_touching
[x] Tests for neighbors_and_sides
[x] Tests for right_left_parts
[x] Tests for segments
[x] Tests for to_convex_parts
[x] Tests for to_graph
[x] Tests for triangulation
[x] Tests for is_on_the_left
[x] Tests for right_part
[ ] Tests for touching_sides
[ ] Tests for join_to_convex
[ ] Tests for is_convex
[ ] Tests for slope_intercept
[ ] Tests for slope_angle
[ ] Tests for rotating_splitter
[ ] Tests for rays_intersection
[ ] Tests for insert_between
[ ] Tests for to_tuple

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@1a479bb). Click here to learn what that means.
The diff coverage is 0%.

@@           Coverage Diff            @@
##             master      #1   +/-   ##
========================================
  Coverage          ?   9.09%
========================================
  Files             ?       3
  Lines             ?     176
  Branches          ?       0
========================================
  Hits              ?      16
  Misses            ?     160
  Partials          ?       0

Impacted Files
Coverage Δ

pode/geometry_utils.py
59.25% <ø> (ø)

pode/pode.py
0% <0%> (ø)

Continue to review full report at Codecov.

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

This PR is not anymore relevant after https://github.com/LostFan123/pode/pull/17 was merged with master.


[8.x] Add hasTables function

Sometimes we need to check the existence of lots of tables, and instead of using and operator between hasTable function, So we can refactor that using hasTables

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


[INFINITY-2005][HDFS] Name node test failure fix

The journal nodes might be out of quorum if they've gone through a restart or crash (which happens currently through the test_kill_all_journalnodes test). In a successive test (test_permanently_replace_namenodes), upon a replace of a name node pod, the name node bootstrap task can't bootstrap and goes into an infinite restart loop trying to contact the journal nodes.
This updates how we account for journal nodes being ready in order to retain quorum such that name nodes, as a result, can function correctly.

@elezar good with the changes?


Fix runner crashing on launching Keyboard Manager UI

Summary of the Pull Request
What is this about?
This PR fixes a crash which occurs on launching the keyboard manager UI from settings.
The error message that appears is Exception thrown at 0x00007FFFA254A799 (KernelBase.dll) in PowerToys.exe: WinRT originate error - 0x8000FFFF : 'WindowsXamlManager and DesktopWindowXamlSource are supported for apps targeting Windows version 10.0.18226.0 and later.  Please check either the application manifest or package manifest and ensure the MaxTestedVersion property is updated.'. Exception thrown at 0x00007FFFA254A799 in PowerToys.exe: Microsoft C++ exception: winrt::hresult_error at memory location 0x000000A9C3AFCFC8.
This appears because the maxVersionTested attribute (removed in #6095 ) is required in the executable's manifest (in this case in runner), as described here https://docs.microsoft.com/en-us/windows/apps/desktop/modernize/host-standard-control-with-xaml-islands-cpp#create-a-desktop-application-project.
As mentioned on this part of the same doc, there is this warning which is a known issue manifest authoring warning 81010002: Unrecognized Element "maxversiontested" in namespace "urn:schemas-microsoft-com:compatibility.v1". I couldn't find a way to suppress or disable this warning, so we will be able to get rid of it only after migrating to WinUI 3.
I have added a comment with details in the manifest file.
PR Checklist

[X] Applies to #6183
[X] CLA signed. If not, go over here and sign the CLA

Validation Steps Performed
How does someone test & validate?
Validated that keyboard manager UI gets launched.

I was mislead by https://docs.microsoft.com/en-us/windows/win32/sbscs/application-manifests that states it's optional.
Pretty crazy that an external application crashes because it's not set in the runner.

rerunning CI

rerunning CI

I tried that once already. I made an issue for the failing test (#6186). It's failing locally as well.

Oddly enough the CI passed, but the test did fail.

I'll go ahead and merge this PR either way since it's unrelated.


chore: extract DEFAULT_DEV_PORT and DEFAULT_PREVIEW_PORT

Description
In the packages/vite/src/node/server/index.ts file, 5173 is used 3 times, and I personally think that 5173 should be extracted as a constant
What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[x] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the PR Title Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

Maybe we also need to re-export it from https://github.com/vitejs/vite/blob/main/packages/vite/src/node/index.ts to make it available outside (via import ... from 'vite'), do we?
/packages/vite/src/node/index.ts file will export packages/vite/src/node/publicUtils.ts. so it can be accessed externally(via import ... from 'vite')


jetbrains.rider: fix internal dotnet runtime/SDK

Motivation for this change
Jetbrains have upgraded their internal runtime to .NET 5 (in 2021.1 I think).
Without this, Rider crashes when opening any .NET solutions.
Things done

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

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

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

Result of nixpkgs-review pr 120589 at 92b42b47 run on aarch64-linux 1

1 package failed to build:

jetbrains.rider

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement.


Consider removing suppressed and submitted_in from Nominations

If #8197 gets merged, or generally we get full slash reversal, we no longer use neither submitted_in nor suppressed if the Nomination struct in pallet-staking. Consider removing these in favour of state-space.

I'd like to work on this. Anything special to consider?

This is just a migration (if any at all) and very small code changes, and the PR that is blocking it might not get merged anytime soon, so I'd recommend waiting on it.

Ah, nevermind. I missed that #8197 was unmerged.

meh, should stay around for now.


Next release date

I know your working hard fixing all the issues that people are putting here and probably want to get most fixed before you make a new released version, but I would really like to get around to opening that 2GB asset file.. Do you have an expected date on when you will release the next version?

All right, I will release the new next week

released


id3v2 parameter -R not recognized

Hello, there seem to be different versions of id3v2 floating around: on Opensuse 15.3 id3v2 does not recognize the parameter -R where that from, say, Debian does. They output the same version information, but --help shows that -R is not listed on the Opensuse build. Maybe a compile time option.

@Konfekt I'm not really using this tool anymore, so I don't really plan on making any changes/fixes.
If you'd like to submit a PR, I'll review it and merge it in.

Also, see #4

Oh, thank you, I did not  look at the resolved issues. Is there a better program to mass edit tags in an editor now?

No idea. It's just not something I do much anymore. I usually just write a quick bash loop.
And tagedit only supports mp3 and ogg, but I almost exclusively use aac and opus now.
Sorry I can't be more help.

Is there a better program to mass edit tags in an editor now?

I recommend beets which uses the MusicBrainz database to automatically tag your music. I used to be meticulous about hand editing my music tags, but I've found that I am much better off this way in terms of time and fixing metadata in the MB database helps out other people too. That said, it has a plugin for mass tag editing if you find yourself needing it or prefer not to venture into autotagging. I use it sometimes to make fixes (in Vim) prior to them being accepted by MB.
The only downside is that initially "importing" your collection with beets (i.e.: tagging) may suggest a lot of changes to your collection (like Unicode characters replacing ASCII ones, label data added that often needs the release country hand modified, etc), but you can import your music "as-is" and then just use beets for mass tag editing. If you're concerned, always use the "timid" flag (or add it to the config).
The autotagging has turned up some pretty big discrepancies in my collection though.. Such as track names being rather wrong. Then one album had a track that I had been listening to for years with a completely wrong title. It will let know if you have missing tracks and (depending on the MB database quality) figures out whether you have some kind of deluxe release with extra tracks, etc, etc..


about the prompt

How can I generate the .npz file? Is it from soundStream?

sorry, as mentioned in https://github.com/suno-ai/bark/issues/1 this is a step that can only be done on our side. This is intentional to block people from voice cloning sources without explicit permission and help mitigate risks of voice technologies.


update to version 0.5.0

And also update meta.yaml syntax to "new style"

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

Whenever possible python packages should use pip. See https://conda-forge.org/docs/meta.html#use-pip

@conda-forge-admin please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@ocefpaf what am I doing wrong with this one? Seems like it should be something obvious...

@ocefpaf what am I doing wrong with this one? Seems like it should be something obvious...

@ocefpaf what am I doing wrong with this one? Seems like it should be something obvious...

GitHub is flaky today so I guess e cannot do much until they are back only :-/

Thanks for having a look. Works for me locally too....

See #6


ngResource instance methods fail when isArray is set to true.

Instance methods fail when isArray is set to true.
For example:
I am attempting to build a generic filter which needs to be able to handle, based on configuration parameters both a resource instance, and a service.
I have the following factory code:
angular.module('sample').factory('Sample', function($resource) {
  var methods = {
    search: {
      method: 'POST',
      isArray: true
    }
  }
  var Sample = new $resource('/sample', methods);

  Sample.prototype.elasticFilter = function (search) {
    var query = {
      query: {
        filtered: {
          query: {
            wildcard: {
              name: '*' + search.toLowerCase() + '*'
            }
          }
        }
      }
    }
    return this.$search(query);
  }
  return Sample;
});

Now if I do the following in my directive:
scope.search = function (search) {
  Sample.elasticFilter(search).$promise.then(function () {
    handle
  });

this results in
  TypeError: Cannot read property 'then' of undefined

printing out Sample.elasticFilter(search), the result is directly a `Promise.
modifying the directive to read directily off Sample.elasticFilter(search).then results in:
TypeError: value.push is not a function.
Drilling into the code, this is happening as interplay between:
https://github.com/angular/angular.js/blob/master/src/ngResource/resource.js#L556 (isInstanceCall is true)
and
https://github.com/angular/angular.js/blob/master/src/ngResource/resource.js#L592 (trying to push onto resource).
While I understand that this may be in fact "working as intended (don't use resources in this manner)" I would point out that this could use a significantly more informative error message and/or a mention in the documentation

I don't think it is an issue with ngResource. You seem to be using it in the wrong way.

There are also a lot of other errors in the code you provided, but I assume they are copy/paste errors.
In any case, I would suggest going over the docs and making sure you understand the difference between "class" methods and "instance" methods.

--
It's not clear how you intend to use elasticFilter, but if you are trying use it as a "static" function (i.e. Sample.elasticFilter), it should look like this:
.factory('Sample', function SampleFactory($resource) {
  var methods = {...};

  var Sample = new $resource('/sample', {}, methods);
  Sample.elasticFilter = function (search) {
    var query = {...};
    return Sample.search(query);
  };

  return Sample;
})

[Live demo][1]

--
So, afaict, this is working as expected.

[1]: http://codepen.io/ExpertSystem/pen/oXXYXj?editors=101


meilisearch: fixes build on darwin

Description of changes
Fixes a build issue on x86_64-darwin as stated in #282283. Thanks @happysalada.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Related PR #281316
Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 282334 run on x86_64-darwin 1

1 package built:

meilisearch


[BuschJaegerSysAp] Invalid message received.

Hi, first kudos for your great plugin!
I started using your plugin about 3 months ago and everything worked perfectly.
However I think after updating my SysAp, whenever I try to toggle a "Sensor/ Schaltaktor 1/1-fach" switch via HomeKit is see the below in the homebridge-out.log
How can I debug this issue in more detail?
e..g how can I log the invalid message being returned from the home api?
Thank you for any help on this matter
➜  logs tail homebridge-out.log [1/5/2019, 7:38:02 AM] [BuschJaegerSysAp] Received a message from websocket [1/5/2019, 7:39:02 AM] [BuschJaegerSysAp] Received a message from websocket [1/5/2019, 7:40:02 AM] [BuschJaegerSysAp] Received a message from websocket [1/5/2019, 7:41:02 AM] [BuschJaegerSysAp] Received a message from websocket [1/5/2019, 7:42:02 AM] [BuschJaegerSysAp] Received a message from websocket [1/5/2019, 7:42:53 AM] [BuschJaegerSysAp] raw/ABB700C8734F/ch0003/idp0000/0 [1/5/2019, 7:42:53 AM] [BuschJaegerSysAp] Received a message from websocket [1/5/2019, 7:42:53 AM] [BuschJaegerSysAp] Invalid message received. [1/5/2019, 7:42:53 AM] [BuschJaegerSysAp] Received a message from websocket [1/5/2019, 7:43:02 AM] [BuschJaegerSysAp] Received a message from websocket

Hi,
thanks. No problem.


Cannot get it working

Hi @huan / anyone,
I cannot get this puppet-discord working, maybe because of something very simple, as both ts-node and esm are new to me.
Can you provide a fully working project that I can just run it please?
I've done as much as I can for you --
https://gist.github.com/suntong/3dc1ec6c3a167340351577c2c5f8e09f
Hopefully it'll be an easy fix for you based on that.
Here is what I'm getting:
$ node -r ts-node/esm index.ts
internal/modules/cjs/loader.js:948
    throw new ERR_REQUIRE_ESM(filename);
    ^

Error [ERR_REQUIRE_ESM]: Must use import to load ES Module: /.../nodejs/wechaty/puppet-discord_test/node_modules/ts-node/esm.mjs
    at Module.load (internal/modules/cjs/loader.js:948:11)
    at Function.Module._load (internal/modules/cjs/loader.js:790:14)
    at Module.require (internal/modules/cjs/loader.js:974:19)
    at Module._preloadModules (internal/modules/cjs/loader.js:1244:12)
    at loadPreloadModules (internal/bootstrap/pre_execution.js:467:5)
    at prepareMainThreadExecution (internal/bootstrap/pre_execution.js:70:3)
    at internal/main/run_main_module.js:7:1 {
  code: 'ERR_REQUIRE_ESM'
}

BTW, if I run node index.js, it'll complain that Cannot find module .../src/mod.js, as I can see only src/mod.ts there.
thx for your help.

And if I try to do npm install wechaty-puppet-discord, I'll get:
$ npm install wechaty-puppet-discord
npm ERR! code E404
npm ERR! 404 Not Found - GET https://registry.npmjs.org/wechaty-puppet-discord - Not found
npm ERR! 404
npm ERR! 404  'wechaty-puppet-discord@*' is not in the npm registry.
npm ERR! 404 You should bug the author to publish it (or use the name yourself!)
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

PS. this nodule is under construction

Yeah, I understand.
I included my 3rd attempt above only to show that I've tried everything that I could. BTW, this is the first time I'm trying to make things work from git repo too. Last time when I tried, which was the first time that I started to get wechaty working years ago, I failed as well and never tried again.
I know you've been able to make it work before, so if you could make a working project first, before you moving on to add more features, that'll be really appreciated. thx.

Thanks for the feedback, I have added a notice in the README and linked it to this issue.


NTR: reverse gyrase

Hi,
Reverse gyrase is a type of topoisomerase activity for which there is no GO term. It is currently annotated with "GO:0003918, DNA topoisomerase II activity (double strand cut, ATP-hydrolyzing) activity", which I think is wrong.
The enzyme introduces positive supercoils into closed DNA in an ATP-dependent fashion. This is the only protein known to introduce positive supercoils, it is only found in thermophiles and is the unique hallmark of thermophiles.
I will attach a small table with comparative data for reverse gyrase, topoisomerase I and topoisomerase II.
Topoisomerases-table.docx
Reverse gyrase is almost always a monomer, binds and cleaves single-strand DNA, changes the DNA linking number by +1 (like topoisomerase I), but uses ATP to introduce positive supercoils (like topoisomerase II). Topo II binds and cleaves double-stranded DNA and changes the linking number by +2, another reason its EC and activity are not correct for reverse gyrase; see the table. I am asking for a new EC number for this activity and I think GO should follow suit.
Suggested term label, (it copies the term labels for topo I and topo II)
Reverse gyrase (single strand cut, ATP-hydrolyzing) activity
Definition:
Catalysis of a DNA topological transformation by transiently cleaving one DNA strand at a time to allow passage of another strand; changes the linking number by +1 per catalytic cycle, is driven by ATP hydrolysis.
Reference; PMID:25013168
Alas there is not a more recent review.
the GN for these enzymes is rgy
It is a child of GO:0003916, DNA topoisomerase activity.
There no terms I know of that should be its children. There are no synonyms, as I said above I am asking for a new EC number because neither of the EC numbers for topo I (<IP_ADDRESS>) or topo II (<IP_ADDRESS>) are correct. Rhea won't make a term (for now?), it would be text only.
Cross-references:
Hopefully a new EC will be forthcoming
https://en.wikipedia.org/wiki/Reverse_gyrase

It appears that the reverse gyrase has also RNA-topoisomerase activity [WIKI]. Should we have then "reverse gyrase activity" to be IS_A "isomerase activity". Alternatively, we can have "DNA reverse gyrase activity" IS_A "DNA topoisomerase activity".

@raymond91125 I dont see the RNA gyrase in the wiki entry - it seems to mention RNA topoisomerase for ' type IA topoisomerase class', and while the reverse gyrase belongs to this family, I dont see where it is mentioned that is also acts on RNA.
Thanks, Pascale

The RNA substrate part may not be entirely clear but there are several related mentions in the literature. If the proposed MF is DNA-specific, we may as well make that explicit, I think.

Wiki: More specifically, reverse gyrase is a member of the type IA topoisomerase class; along with the ability to relax negatively or positively supercoiled DNA[5] (which does not require ATP[6][7]), type IA enzymes also tend to have RNA-topoisomerase activities.
PMID:25013168 Overall, the helicase domain of reverse gyrase thus undergoes a similar conformational cycle as DEAD-box proteins (65–67) (reviewed in (58)), alternating between open and closed states with different DNA affinities (Figure (Figure2B).2B). While closing is coupled to the cooperative binding of ATP and DNA/RNA in reverse gyrase and DEAD-box proteins alike,
PMID:11823434 A search with DALI (Holm and Sander, 1993) revealed that H3 is structurally homologous to residues 1–46 of the E.coli rho transcription terminator. This region in rho is involved in RNA binding (Dombroski and Platt, 1988; Dolan et al., 1990) and may play an analogous role in reverse gyrase,
PMID:20851892 In addition, because reverse gyrase is able to bind RNA,3 it might also remove secondary structures in DNA-RNA hybrids.

@AndreaAuchincloss What do you think? Thanks.

Hi,
I've probably read most of the papers in the field, and I've not seen any mention of an RNA topoisomerase activity; then again once people decide it doesn't have that activity, they no longer look for it. However I do not think it is appropriate for this particular protein, we could add a phrase along the lines of "only known to be active on DNA" if you want to, although as a child of DNA topoisomerase activity that may be evident(?).

Type IA enzymes tend to have RNA-topoisomerase activity is not proof for a domain function in reverse gyrase.
The mention in PMID:25013168 refers to DEAD box helicases as I read the sentence.
PMID:11823434 postulates the H3 domain to be involved in DNA-binding.
PMID:20851892 they say RG can bind RNA with a tiny 3 as unpublished results. If they haven't published the data since 2010 I don't know if I think it is worth mentioning.

I can't find any evidence of an RNA gyrase activity, to add it to the GO term would be misleading in my opinion.

Is "RNA topoisomerase even physiologically possible? I thought it was to relax supercoiling and relax linking number in a topological transformation.

@AndreaAuchincloss Thanks for the clarification.
+[Term]
+id: GO:0160097
+name: reverse gyrase activity
+namespace: molecular_function
+def: "Catalysis of a DNA topological transformation by transiently cleaving one DNA strand at a time to allow passage of another strand; changes the linking number by +1 per catalytic cycle, is driven by ATP hydrolysis." [PMID:25013168]
+xref: WIKIPEDIA:Reverse_gyrase
+is_a: GO:0003916 ! DNA topoisomerase activity
+property_value: term_tracker_item "https://github.com/geneontology/go-ontology/issues/26019" xsd:anyURI
+created_by: rynl
+creation_date: 2023-09-08T16:47:25Z


Remove dependency on findbugs Nullable

Any proxied configuration method may now return null if no default provided.

archaius-pull-requests #312 SUCCESS
This pull request looks good

NetflixOSS » archaius » archaius-pull-requests #143 SUCCESS
This pull request looks good

archaius-pull-requests #322 FAILURE
Looks like there's a problem with this pull request

NetflixOSS » archaius » archaius-pull-requests #156 FAILURE
Looks like there's a problem with this pull request

NetflixOSS » archaius » archaius-pull-requests #157 FAILURE
Looks like there's a problem with this pull request


cuSPARSELt: Support CUDA 12

Depends on issue ( https://github.com/conda-forge/staged-recipes/issues/21382 )

@leofang, is there anything in particular we need to do here?

Sounds like the existing version should already work. Will set up a PR to add that build

@conda-forge-admin, please re-render


Summaries Are Missing From Source

The summaries for each chapter aren't in the repository, yet they are in the PDF available on the website.

I assume you mean the chapter summaries in "Sets, Logic, Computation"?  They are in https://github.com/rzach/phil379 (see the include directory).

Ah, thanks for letting me now. Sorry for the mistake.


fix: fix varbinary for cursor

Codecov Report

Merging #214 (e15333e) into 3.0 (9619472) will decrease coverage by 0.05%.
The diff coverage is 90.90%.

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

@@            Coverage Diff             @@
##              3.0     #214      +/-   ##
==========================================
- Coverage   80.82%   80.78%   -0.05%
==========================================
  Files          20       20
  Lines        2675     2685      +10
==========================================
+ Hits         2162     2169       +7
- Misses        513      516       +3

Files Changed
Coverage Δ

taos/field_v3.py
93.33% <90.00%> (-0.42%)
:arrow_down:

taos/cinterface.py
81.16% <100.00%> (-0.10%)
:arrow_down:

... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


As an experienced golfer with a group of novice friends interested in golf, I want a simple way of tracking our games that we all can use together while I introduce them to the sport.

Closed with last pull request (#38):
https://github.com/ChicoState/Tee-Together/pull/38


Question about LCA

Thank you very much for this nice tool!
If I understand correctly, Emu does not apply a lowest common ancestor (LCA) like approach, right? This was not entirely clear for me when reading the paper.
For instance, when Kraken2 cannot confidently assign a sequence to a species level, it will assign it to the lowest taxonomic rank possible, which is usually the genus level.
Cheers,
Paul

Hi @Pstege,
Apologies for the delay. This is correct, Emu does not use LCA. Instead each read may be assigned multiple species with the likelihood percentage of each. These values are used directly when calculating the community profile, so a read is not forced to be classified as only a single species. To see a breakdown of this probability distribution for each read, you can use the keep-read-assignments flag.


Add C compatible usages

First patch is just to not have to call cargo fmt after each python call.
Second patch adds a C compatible output following https://gitlab.freedesktop.org/libevdev/udev-hid-bpf/-/blob/main/src/bpf/hid_report_helpers.h
The idea is to replace the description of the report descriptor in hid-recorder by something we can directly integrate into a C program.

Thanks for the patches, I've taken the first patch and merged it as part of #11. The second patch is too specific to the #defines in another repo which means the same comments as in https://github.com/hidutils/hid-recorder/pull/24#issuecomment-2224254274 apply.


Use pkg-config to find the gssapi system library

This uses pkg-config to locate the gssapi system library and the necessary include paths.  This can be particularly useful when gssapi has interface dependencies (mit has comerr includes in it's public interface, for example) that are installed in different prefixes as is often the case with third party package managers.

Closing to do some additional testing to make sure I've covered the necessary use cases

Sorry for the churn, this good to go.  I thought I had missed something but it was an unrelated issue in my downstream application.


needs sudo ?

env: MacOS
some needs sudo to bypass the permission issue
sudo npm install -g @offckb/cli
sudo offckb node

I've also encountered this issue. Granting the current user write access to the node_modules directory can resolve the problem.
sudo chown -R $(whoami) /usr/local/lib/node_modules
npm install -g @offckb/cli

added the solution from @sunchengzhu  in the readme:

https://github.com/RetricSu/offckb/pull/77


DCP generation fails for some tests

Upon updating the RapidWright version containing the fix for https://github.com/chipsalliance/fpga-interchange-tests/issues/94, some tests started to fail at the DCP generation from the physical and logical netlists in RapidWright.
These tests were not failing earlier and there was no change in the toolchain, therefore it might be caused by changes in RapidWright.
Steps to reproduce

Download murax.zip
unzip murax.zip
cd murax && RAPIDWRIGHT_PATH=<path to RW> make

This should output the following:
Exception in thread "main" java.lang.RuntimeException: java.util.concurrent.ExecutionException: java.lang.RuntimeException: java.util.concurrent.ExecutionException: java.lang.RuntimeException: java.util.concurrent.ExecutionException: java.lang.NullPointerException
        at com.xilinx.rapidwright.util.ParallelismTools.joinFirst(ParallelismTools.java:221)
        at b.b(Unknown Source)
        at b.a(Unknown Source)
        at com.xilinx.rapidwright.design.Design.writeCheckpoint(Unknown Source)
        at com.xilinx.rapidwright.interchange.PhysicalNetlistToDcp.main(PhysicalNetlistToDcp.java:69)
Caused by: java.util.concurrent.ExecutionException: java.lang.RuntimeException: java.util.concurrent.ExecutionException: java.lang.RuntimeException: java.util.concurrent.ExecutionException: java.lang.NullPointerException
        at java.base/java.util.concurrent.FutureTask.report(FutureTask.java:122)
        at java.base/java.util.concurrent.FutureTask.get(FutureTask.java:191)
        at com.xilinx.rapidwright.util.ParallelismTools.joinFirst(ParallelismTools.java:219)
        ... 4 more
Caused by: java.lang.RuntimeException: java.util.concurrent.ExecutionException: java.lang.RuntimeException: java.util.concurrent.ExecutionException: java.lang.NullPointerException
        at com.xilinx.rapidwright.util.ParallelismTools.get(ParallelismTools.java:149)
        at com.xilinx.rapidwright.util.ParallelismTools.join(ParallelismTools.java:244)
        at S.a(Unknown Source)
        at b.a(Unknown Source)
        at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
        at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
        at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
        at java.base/java.lang.Thread.run(Thread.java:829)
Caused by: java.util.concurrent.ExecutionException: java.lang.RuntimeException: java.util.concurrent.ExecutionException: java.lang.NullPointerException
        at java.base/java.util.concurrent.FutureTask.report(FutureTask.java:122)
        at java.base/java.util.concurrent.FutureTask.get(FutureTask.java:191)
        at com.xilinx.rapidwright.util.ParallelismTools.get(ParallelismTools.java:147)
        ... 7 more
Caused by: java.lang.RuntimeException: java.util.concurrent.ExecutionException: java.lang.NullPointerException
        at com.xilinx.rapidwright.util.ParallelismTools.joinFirst(ParallelismTools.java:221)
        at S.a(Unknown Source)
        at S.a(Unknown Source)
        at com.xilinx.rapidwright.util.b.a(Unknown Source)
        at S.a(Unknown Source)
        ... 4 more
Caused by: java.util.concurrent.ExecutionException: java.lang.NullPointerException
        at java.base/java.util.concurrent.FutureTask.report(FutureTask.java:122)
        at java.base/java.util.concurrent.FutureTask.get(FutureTask.java:191)
        at com.xilinx.rapidwright.util.ParallelismTools.joinFirst(ParallelismTools.java:219)
        ... 8 more
Caused by: java.lang.NullPointerException
        at com.xilinx.rapidwright.device.Node.a(Unknown Source)
        at S.a(Unknown Source)
        at S.a(Unknown Source)
        at S.a(Unknown Source)
        at java.base/java.lang.Iterable.forEach(Iterable.java:75)
        ... 7 more

Versions
RapidWright: 57ffdef175dd669d66fb0a1d0261b888e64c48b6

Testcase link not currently working.

@eddieh-xlnx I think there was an issue with GH, I have uploaded a new zip which should work now

Thanks! The issue should be fixed with the latest RapidWright, can you please give that a try?

@eddieh-xlnx Bumping RW fixed the problem, thanks for the quick fix! Closing.


I have no idea how to use this tool .just can't figure out where to start.can anyone help pls

I have no idea how to use this tool .just can't figure out where to start.can anyone help pls

Download from here: https://github.com/jvolker/miq-nz-booking-tools/releases/latest
Read the user guide: https://github.com/jvolker/miq-nz-booking-tools/blob/master/MIQ-Booking-Assistance/README.md
If you have more specific questions feel free to reopen this issue.

Hi  there
Thanks for email.
Unfortunately I'm not tek savey with this stuff. Saw the github site from a
NZ stuff news report  and thought it might help me get my isolation room .
I clicked on the link in your email but cant not see where or how to
download wat ever it is I have to download.
My phone is a old samsung S4 .
Just need some guidance please.
Hope you can help help
Owen
Uk
On Wed, 28 Jul 2021 23:03 Jeremias Volker, @.***> wrote:

Download from here:
https://github.com/jvolker/miq-nz-booking-tools/releases/latest
Read the user guide:
https://github.com/jvolker/miq-nz-booking-tools/blob/master/MIQ-Booking-Assistance/README.md
If you have more specific questions feel free to reopen this issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jvolker/miq-nz-booking-tools/issues/97#issuecomment-888650585,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVAFAEMZK4EEGVIOKMDFVTLT2B5DVANCNFSM5BFGX7NQ
.

Hey Owen,
This is a not a mobile application, so you will need to download and install the miq-nz-booking-assistance-Windows-v0.10.3.exe file on to a windows computer. Then you should be able to follow along with the user guide.

Hi Jacob
Thanks for the information.
Unfortunately I only have my phone and chromebook laptop.
I've been trying to secure a isolation room for months but never seem to be
in time or quick enough. Seems so unfair as I only want to get home to NZ.
 At least
everyone trying to get back would have some sort of plan and time frame to
make plans but this allocation system is pants.
Thanks again for your help
Owen
On Thu, 29 Jul 2021 18:13 Jacob Thorne, @.***> wrote:

Hey Owen,
This is a not a mobile application, so you will need to download and
install the miq-nz-booking-assistance-Windows-v0.10.3.exe file on to a
windows computer. Then you should be able to follow along with the user
guide.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jvolker/miq-nz-booking-tools/issues/97#issuecomment-889318927,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVAFAELARZHXC4OBBKEFFUTT2GD2JANCNFSM5BFGX7NQ
.


Add cobwebs, strings, and placeable tripwires (no function)

Could I get a review on this please

Just tested fall damage negation, works perfectly

Could someone review this please

Cobwebs are missing physics for non player entities

Any idea how I can do that?

How would i do the physics here for non player entities?

i say we merge this because water doesn't have physics for non player entities either


Check for warnings before build

Fix for #394.
Checks whether warning count > 0 before generating Yail.
Will throw an error -

@halatmit @josmas @afmckinney

@shrutirij What about blocks with red error triangles? Are they covered in this change?

@jisqyv, no it doesn't. I think errors show up later on, while compiling and the build doesn't succeed.
I can add it to the change, though. It should be just like checking for warnings.
-----Original Message-----
From: "Jeffrey Schiller"<EMAIL_ADDRESS>Sent: 06-11-2015 20:29
To: "mit-cml/appinventor-sources"<EMAIL_ADDRESS>Cc: "Shruti Rijhwani"<EMAIL_ADDRESS>Subject: Re: [appinventor-sources] Check for warnings before build (#601)
@shrutirij What about blocks with red error triangles? Are they covered in this change?
—
Reply to this email directly or view it on GitHub.

@jisqyv, turns out there are some cases where errors aren't detected at compile time.
I've added the error check to this PR.
@halatmit

Note: I'll finish this up.

@jisqyv Shall I finish this over the weekend?

Sure.
-Jeff
On Friday, January 22, 2016, Shruti Rijhwani<EMAIL_ADDRESS>wrote:

@jisqyv https://github.com/jisqyv Shall I finish this over the weekend?
—
Reply to this email directly or view it on GitHub
https://github.com/mit-cml/appinventor-sources/pull/601#issuecomment-173821863
.

--

Jeffrey I. Schiller
Information Systems and Technology – MIT App Inventor
Massachusetts Institute of Technology
77 Massachusetts Avenue  Room 32-386
Cambridge, MA 02139-4307
617.910.0259 - Voice
<EMAIL_ADDRESS>http://jis.qyv.name
http://appinventor.mit.edu

Warnings are just that and should allow the build.


Initial draft on validation

@dougbu when you get a chance to take a look

@rachelappel @danroth27 @shirhatti do we need even one sample in the aspnet/Docs repo? This section follows the model I've seen elsewhere, including 94 files when maybe 5 relate specifically to validation.
In my opinion, the answer is no. We have separate samples repos, samples in the individual product repos, and more. It would be way simpler and require much less maintenance if this section (and the repo in general) included only the files relevant to the topic. (Inline code is even better :smiley_cat:) No need for a runnable sample anywhere in this repo.

@dougbu We want samples that we can build and run so that the code snippets in our docs don't rot and break. I don't want docs with inlined code that doesn't compile. I do agree that our samples shouldn't have unnecessary files. We should try to make this sample much more focused. Most samples should only be a handful of files.

@dougbu @danroth27 removed dead weight and pushed relevant samples

:watch:

:watch:

@danroth27

Hi @danroth27, you are mistyping your colleague's name. It is @dougbu not @doubu - not the extra g. Thanks.

@dougbu Please review

:watch: mostly for small comments. Looks good overall.

:watch:

@dougbu @danroth27

:watch: please ping me e.g. using an :up::date: comment when above comments are addressed.

:up::date: @dougbu

:watch:

:up::date: @dougbu

:shipit: from me with two caveats:

There's an open question I've asked @danroth27 that would be good to answer before closing the PR.
The PR still has a few open comments and I added one or two more above. Suggest refreshing the browser on the Conversation tab to see everything GitHub thinks hasn't been done. Most of these items are polish but please at least address this one.

c16274761db4156f46860f0fdc95c1bc3b1305a5


Configurable label font

It would be great if labels could have configurable fonts. One of my projects needs labels with monospaced text, but I don't want all the text in the UI to be monospaced. From what I can tell this isn't possible right now since at the moment Label's get_layout gets the font name from the theme. Probably the most elegant solution would be an Option<String> with the theme being used as the default if the font is None.

I agree this makes sense, and a PR would be welcome; font should work the same as color and size.
cc @futurepaul who has been thinking about this lately.

I haven't tried anything out yet but ideally you can follow the example of the builder / setter pair we have for size and color. That way Label::new sets the font as theme::FONT_NAME and then the builder or setter updates that value, no need for an option. I'm curious if we can do KeyOrValue here as well, but I haven't used that yet with a String so no guarantees.

theme::FONT_NAME is the key for the environment, not the actual font name. If we don't use an Option then we either store the env key, meaning we couldn't directly set the font name, or directly store the font name and grab the theme font name once during creation, neither of which are good ideas IMO.


Pandas to Dask transformation

Basic source-to-source transformation of Pandas operations into Dask.
This is a starting step to demonstrate how such a transformation would be done and is not comprehensive yet.

sorry this sat around a while, can you rebase off master and resolve the conflicts?


Added get_statistics variable to buckets

Added a boolean variable to bucketers to turn off calculation of bucket tables and summary. These are not called on in the BucketingProcess class, nor by the Skorecard class. Turning off these redundant calculations results in a speed increase.

Thanks for the contribution. And yes, this makes sense!

Released to pypi also https://github.com/ing-bank/skorecard/releases/tag/v1.6.4


Fix elaboration of case and support as patterns

Can we non-squash merge this PR?

Can we non-squash merge this PR?

Of course. I squash-merge PRs when some of the commits messages are like "update test output" and "fix". If the commits are clean, I prefer keeping them.

As discussed, we'll just squash-merge this one.


Fixes #1045 -Take UNC path from query instead of uri

Description
When UNC path is used for a repo the gitUri is formatted without the double slash.
The following example is from a repo on //aesrocc/home/ROCCTEST
gitlens://0cee44d/aesrocc/home/ROCCTEST/fldedit/invfe53gst.et?{"path":"//aesrocc/home/ROCCTEST/fldedit/invfe53gst.et","ref":"0cee44d88243c4180f17837d9adf3b43a231ead2","repoPath":"//aesrocc/home/ROCCTEST"}
\_____/   \_____/\__________________________________________/ \___________________________________________________________________________________________________________________________________________/
   |        |            |                                                                                |
scheme     authority       path                                                                 query

The uri strips the //
uri.path = /aesrocc/home/ROCCTEST/fldedit/invfe53gst.et
but the path is still correct in the path element  of the query string and should be reliable on all file systems
data.path = //aesrocc/home/ROCCTEST/fldedit/invfe53gst.et (from query)
Checklist

[x] I have followed the guidelines in the Contributing document
[x] My changes follow the coding style of this project
[x] My changes build without any errors or warnings
[x] My changes have been formatted and linted
[x] My changes include any required corresponding changes to the documentation
[x] My changes have been rebased and squashed to the minimal number (typically 1) of relevant commits
[x] My changes have a descriptive commit message with a short title, including a Fixes $XXX - or Closes #XXX - prefix to auto-close the issue that your PR addresses

closing to use separate branch


Improve the interface of the ledger-bitcoin python library

Listing the issues found while working on the Electrum integration:

[x]  The Transport interface doesn't allow to specify the path when using hid mode.
[X]  The sign_psbt should allow to pass a str or bytes as a psbt, and return the same type; we don't want to bound callers to use the custom PSBT class.
[X] Add __version__ to the module.

Done in #79.


when the user clicks on grid row that particular values should be populated in pop-up

As the application starts one pop-up(parent pop-up) will appear on the webpage and it consists of buttons,textboxes etc.
and when the user clicked on the button another pop-up(child pop-up) will appear on the screen and this pop-up consists of grid and some textboxes,buttons etc.
My requirement:-is when the user clicked on grid-row that particular row data should be shown in the textboxes of first(previous or parent pop-up) how we can bind that??

I think you can use, as starting point, the BasicSamples/CustomLinksInGrid sample.
To se data in current dialog, you can use Serenity.EditorUtils.setValue() method

yes  it is working on  grids which are placed on main page,but
my grid is not populated on the main page but it is populated on one dialog box.


Testing PR DO NOT MERGE

Update tutorial-url-redirect-cli.md.
Added two book mark links for testing purpose.

@kexugit : Thanks for your contribution! The author, @vhorne, has been notified to review your proposed change.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-26-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3413

Questions ?
Please refer to the Backport tool documentation


Postgres connection invalidated in DBeaver Windows version 5.1.1

After installing Windows version 5.1.1 64-bit I started to encounter issues with database connections to PostgreSQL databases. DBeaver regularly shows a popup window telling me the connection was invalidated even though the query is still running. Dbeaver then restores the connection which in itself is great.
However, as a result of losing the connection I also lost a temporary table I was creating (temporary tables get dropped after the session ends). I have a feeling this is related to some kind of timing setting because it seems to occur especially in situations where a query is running for more than a couple of minutes.
I decided to re-install version 5.0.4, which was the latest version I used before installing 5.1.1. In the older version things work again as expected. The issue has gone.
Please let me know if you need any more information to resolve this issue,

What is your database? Maybe DBeaver detect connection loss error incorrectly.
Is there anything interesting in debug log? (https://github.com/dbeaver/dbeaver/wiki/Log-files)
You can disable automatic invalidation in 5.1+:

Hi
Thanks for your e-mail and sorry for the late response. I connect to Postgres version:
EnterpriseDB <IP_ADDRESS> on x86_64-pc-linux-gnu, compiled by gcc (GCC) 4.4.7 20120313 (Red Hat 4.4.7-16), 64-bit
Version 5.0.4 is okay but I will try using the latest version with below suggestion. Unfortunately I do not have old log files anymore.
Best regards,
John
Op 30 jun 2018 om 16:44 uur uur schreef Serge Rider<EMAIL_ADDRESS>What is your database? Maybe DBeaver detect connection loss error incorrectly.
Is there anything interesting in debug log? (https://github.com/dbeaver/dbeaver/wiki/Log-files)
You can disable automatic invalidation in 5.1+:
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@johnrijs could you find a solution? i've same problem

I get the same error randomly, usually it's at the beginning of day if I was using the connection the evening before.
I'm assuming it's not a problem, but it's annoying being distracted by a notification that may be pointless.

I think it has a problem. Because, when I run long scripts setting temporary session parameters, this live connection gets broken and Dbeaver reconnects with a new session and triggers same query. But this time all my session parameters are lost.


password unlock when sending SWT

Location
/confirm sending
Expected behavior
after confirm sending got to password unlock, then to transaction processing
Actual behavior
after confirm sending goes directly to transaction processing
Steps to reproduce the problem
my-wallet > send SWT > confirm sending> ok

fixed in branch #600


docs(input): add type description

🔗 Linked issue

❓ Type of change

[x] 📖 Documentation (updates to the documentation or readme)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Add description for input types

📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

LGTM ;)

Thanks 😊


feat: Adds error handling for async operations, moves boilerplate to factory function

#111
@driskull
Hey Matt, I don't expect this to be merged directly but I thought I would put it here since you are in the process of rewriting this boilerplate. It's just some refactoring that made sense in the context of how I'm setting up the embedded group gallery app.
Most of the boilerplate logic is the same as before, but it's moved from a class to a factory function that returns the boilerplate results as an object literal. I tried to clarify the asynchronous requests a bit and also added some error handling (turns out my original issue this morning was with the PortaItems not firing then()).
The only other big change is that it provides a scaffold for structuring the application using the js api 4.x widget framework.

very cool @nicksenger.
I'll go through and take the things that I haven't done yet. Although I rewrote almost all of it and I think its structured a lot better now. Still a few things to do though.

The only other big change is that it provides a scaffold for structuring the application using the js api 4.x widget framework.

I was debating doing this. I'll see what you got.

Most of the boilerplate logic is the same as before, but it's moved from a class to a factory function that returns the boilerplate results as an object literal

Was debating this as well but sticking with a class for now. Since querying for group items still needs a method.

Yeah I'm not really sure why i switched to a closure - more familiar with them I guess. You can simulate public methods by tacking a function on the exported object, but in the end it's 6 of one, half dozen of the other :]

@nicksenger could you look at the latest 4masterts and let me know your thoughts?


View config not found for name text

I did everything in documentation but got this error.

Could you give me your data and show me your code?


Make Koans PEP8 compliant?

This is not a super issue, but I noticed that there are small things (indentations, spaces, names) that are not PEP8 compliant. I noticed this in the Python3 folder (Had not checked the Python2 folder)

We've had PRs in the past to fix the PEP8 compliance but inevitably it keeps falling out of whack over time. We're always happy to take new PEP8 fix PRs.


workbox is not defined

Library Affected:
workbox-webpack-plugin
Browser & Platform:
all browsers
Issue or Feature Request Description:
I am using workbox-webpack-plugin to setup my service worker. I added the InjectManifest pointing to my service-worker file, however, when I load the webpage I get an error saying "workbox is not defined". This is because the service worker file has no reference to the workbox object. The docs claim that the workbox object will get auto injected during build, but this does not seem like the case (or am I totally wrong?); However, when I use a CDN in my service-worker file, it works. Could someone please point me what I am doing wrong here?

Before using the workbox object, you need to import the workbox-sw loader package, which you can read how to do here: https://developers.google.com/web/tools/workbox/guides/get-started#importing_workbox
In short, you need to add something like the following to the top of your service worker file:
importScripts('https://storage.googleapis.com/workbox-cdn/releases/5.0.0/workbox-sw.js');

// ...

@philipwalton and when we are using workbox-webpack then too ?

Facing the same issue. any solution?

@manjunathgk22 are you using webpack? If so just install the required packages and import it in your service-worker file.
https://developers.google.com/web/tools/workbox/guides/using-bundlers


docs: Add initial docs on how to use node local storage

This adds basic documentation on how to use node local storage using
local persistent volumes in filesystem mode.
Other sections are mentioned, although empty for now, on other ways to
use it that we may want to expand later.

@pothos @invidian thanks a lot for the detailed review! And the commit suggestions!
I think it is much improved now, PTAL @pothos @invidian :)

@surajssd PTAL. I've addressed the all comments, except the ones I asked you something, and adapted to use /mnt/node-local-storage as path, as that was recently changed in master

Node local storage was moved back to /mnt in this PR https://github.com/kinvolk/lokomotive-kubernetes/pull/72 recently merged, so we need to adjust this doc to reflect that

@rata do you have time to take care of this PR?

@invidian not really, and is kind of outdated now. I'll try to keep an eye on this during next week, but no promises :(

I'm taking a look into it to get it merged.

IMHO, it looks fine besides the two open comments (the one I just posted and suraj comment from long ago)

@surajssd do you think it looks good to approve the PR? Can you approve it, then? :)
@invidian can you approve too? (not sure, it's weird, you addressed the last comments at least. But I can't approve as it is my PR :))
@pothos or maybe can you take a look and approve if it seems okay? :)
PS: I'd merge with squash and merge, if we are all ok with this (don't see value in the split commits)

@surajssd do you think it looks good to approve the PR? Can you approve it, then? :)

@surajssd already approved it, but it got dismissed by my push. Let me squash the commits then.

Merging on behalf of @invidian ? :joy:

@invidian thanks a lot for driving this forward! :)


DatePicker breaks with "date.clone is not a function" when "selected" date is passed down as a prop

If I initialize a DatePicker by directly providing the selected prop as a moment object, everything works fine:
const WrapperComponent = ({ startDate, onDateChange }) => (

        <DatePicker
                selected={moment()}
                onChange={onDateChange}
            />

)

However if i try to initialize the component by passing the "selected" prop externally as a startDate prop from WrapperComponent like this:
const WrapperComponent = ({ startDate, onDateChange }) => (

        <DatePicker
                selected={startDate}
                onChange={onDateChange}
            />

)

...the DatePicker breaks by moment js, even if the startDate provided by a predefined value of the same moment() object:
date_utils.js:167 Uncaught TypeError: date.clone is not a function
    at safeDateFormat (date_utils.js:167)
    at DatePicker._this.renderDateInput (datepicker.js:388)
    at DatePicker.render (datepicker.js:474)
    at ReactCompositeComponent.js:795
    at measureLifeCyclePerf (ReactCompositeComponent.js:75)
    at ReactCompositeComponentWrapper._renderValidatedComponentWithoutOwnerOrContext (ReactCompositeComponent.js:794)
    at ReactCompositeComponentWrapper._renderValidatedComponent (ReactCompositeComponent.js:821)
    at ReactCompositeComponentWrapper.performInitialMount (ReactCompositeComponent.js:361)
    at ReactCompositeComponentWrapper.mountComponent (ReactCompositeComponent.js:257)
    at Object.mountComponent (ReactReconciler.js:45)

What am I doing wrong? Can't I use a DatePicker in a stateless component, passing down it's props?

Please see https://github.com/Hacker0x01/react-datepicker/issues/491

@timfisher it does not solve my issue, i also pass a moment() object: if i give it directly, it works, if i pass a moment() object as prop, it doesn't.

The issue corresponds to https://github.com/Hacker0x01/react-datepicker/issues/862 and a workout i used is to use a react class component instead of a stateless function as react-datepicker needs an internal state. The downside is that this way the external redux state and the internal this.state.startDate, etc needs to be reconciled somehow, but this is the question of controlled forms and redux which redux-forms answers for example.

The valuable you passed to selected props should be a moment object, a plain Date object cause the error.

+1

This may help
`
import React from 'react';
import DatePicker from 'react-datepicker';
import moment from 'moment';
import 'react-datepicker/dist/react-datepicker.css';
// CSS Modules, react-datepicker-cssmodules.css
// import 'react-datepicker/dist/react-datepicker-cssmodules.css';
class Example extends React.Component {
constructor(props) {
super(props)
this.state = {
startDate: moment()
};
this.handleChange = this.handleChange.bind(this);
}
handleChange(date) {
    this.setState({
        startDate: date
    });
}

render() {
    return <DatePicker
        selected={this.state.startDate}
        onChange={this.handleChange}
    />;
}

}`

<DatePicker
onChange={(e) => { this.SaveDateOfDeposit(e, i) }}
                                    selected={ this.state.dateOfDeposit==""?moment():moment(this.state.dateOfDeposit,"DD/MM/YYYY")}

                                    peekNextMonth
                                    showMonthDropdown
                                    showYearDropdown
                                    dropdownMode="select"
                                    maxDate={moment()}

                                    dateFormat="DD/MM/YYYY"

                                />

SaveDateOfDeposit(date,i) {

    let DOB = date.format("DD/MM/YYYY");

    this.setState({
        dateOfDeposit: DOB,
    })

}

Note:
this.state: {
dateOfDeposit=""
}

Hi. you must add DateAdapter to your providers in your Module like that:
import {  MAT_DATE_FORMATS, DateAdapter} from '@angular/material/core' ;
@NgModule({
... ,
providers:[{ provide: DateAdapter}, { provide: MAT_DATE_FORMATS}]
...
})
also if you want to use custom format :
@NgModule({
... ,
providers:[{ provide: DateAdapter, useClass: CustomComponent},
{ provide: MAT_DATE_FORMATS, useValue: CUSTOM_FORMAT}],
...
})

I'm still getting this same error. I'm just experimenting with code from the docs, so I'm not sure why it won't work
`const [startDate, setStartDate] = useState(new Date());
const [endDate, setEndDate] = useState(null);
const onChange = (dates) => {
const [start, end] = dates;
setStartDate(start);
setEndDate(end);
};
`
Getting this error:
TypeError: date.clone is not a function
at localizeDate (webpack-internal:///./node_modules/react-datepicker/lib/date_utils.js:354:15)
at Calendar._this.localizeDate (webpack-internal:///./node_modules/react-datepicker/lib/calendar.js:111:43)
at new Calendar (webpack-internal:///./node_modules/react-datepicker/lib/calendar.js:356:19)
at constructClassInstance (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:13308:18)
at updateClassComponent (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:16985:5)
at beginWork$1 (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:18505:16)
at HTMLUnknownElement.callCallback (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:347:14)
at Object.invokeGuardedCallbackDev (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:397:16)
at invokeGuardedCallback (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:454:31)
at beginWork$$1 (webpack-internal:///./node_modules/react-dom/cjs/react-dom.development.js:23217:7)


Dimmer support

Device description
Eco-Dim/Namron push-button rotary dimmer
Zigbee2mqtt page
https://github.com/Koenkk/zigbee2mqtt.io/blob/master/docs/devices/Eco-Dim.07.md
MQTT message with device information
Nothing in zigbee2mqtt/bridge/config/devices, but this came from zigbee2mqtt/bridge/log:
{'dateCode': '20191105', 'description': 'Zigbee & Z-wave dimmer ', 'friendly_name': 'Living Room Dimmer', 'hardwareVersion': 1, 'ieeeAddr': '0x086bd7fffe5cd8ed', 'lastSeen':<PHONE_NUMBER>565, 'manufacturerID': 4714, 'manufacturerName': 'Namron AS', 'model': 'Eco-Dim.07', 'modelID': 'Dimmer-Switch-ZB3.0', 'networkAddress': 20946, 'powerSource': 'Mains (single phase)', 'softwareBuildID': '3.04', 'type': 'Router', 'vendor': 'EcoDim'}

MQTT messages from device
'zigbee2mqtt/Living Room Dimmer', payload {'brightness': 64, 'linkquality': 23, 'state': 'ON'}

**MQTT state change commands**
'zigbee2mqtt/Living Room Dimmer/set'
{
  "state": "ON",
  "brightness": 200
}
{
  "state": "OFF"
}

Please upgrade to the latest plugin version from the master branch (v. 3.0). This device should work fine

Please upgrade to the latest plugin version from the master branch (v. 3.0). This device should work fine

Thanks! Looking forward to trying it out.
On Fri, 15 Jan 2021, 21:11 Stanislav Demydiuk<EMAIL_ADDRESS>wrote:

Please upgrade to the latest plugin version from the master branch (v.
3.0). This device should work fine
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stas-demydiuk/domoticz-zigbee2mqtt-plugin/issues/535#issuecomment-761173172,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQS4JRENUKI4WDOQGE4HUSLS2COORANCNFSM4VODL2XA
.

Thanks! Looking forward to trying it out.
On Fri, 15 Jan 2021, 21:11 Stanislav Demydiuk<EMAIL_ADDRESS>wrote:

Please upgrade to the latest plugin version from the master branch (v.
3.0). This device should work fine
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/stas-demydiuk/domoticz-zigbee2mqtt-plugin/issues/535#issuecomment-761173172,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQS4JRENUKI4WDOQGE4HUSLS2COORANCNFSM4VODL2XA
.

Hi,
Same Ecodim.07 dimmer here, after updating the plugin domoticz log is filled with the following errors:
Previous version of the plugin was less then a week old, did not behave like this.
2021-01-17 12:14:30.735 Error: Python Plugin System: (Zigbee2MQTT) 'onMessage' failed 'TypeError':'get_numeric_value() takes 3 positional arguments but 4 were given'.
2021-01-17 12:14:30.735 Error: Python Plugin System: (Zigbee2MQTT) ----> Line 279 in '/home/pi/domoticz/plugins/zigbee2mqtt/plugin.py', function onMessage
2021-01-17 12:14:30.735 Error: Python Plugin System: (Zigbee2MQTT) ----> Line 124 in '/home/pi/domoticz/plugins/zigbee2mqtt/plugin.py', function onMessage
2021-01-17 12:14:30.736 Error: Python Plugin System: (Zigbee2MQTT) ----> Line 167 in '/home/pi/domoticz/plugins/zigbee2mqtt/mqtt.py', function onMessage
2021-01-17 12:14:30.736 Error: Python Plugin System: (Zigbee2MQTT) ----> Line 199 in '/home/pi/domoticz/plugins/zigbee2mqtt/plugin.py', function onMQTTPublish
2021-01-17 12:14:30.736 Error: Python Plugin System: (Zigbee2MQTT) ----> Line 54 in '/home/pi/domoticz/plugins/zigbee2mqtt/devices_manager.py', function handle_mqtt_message
2021-01-17 12:14:30.736 Error: Python Plugin System: (Zigbee2MQTT) ----> Line 64 in '/home/pi/domoticz/plugins/zigbee2mqtt/adapters/base_adapter.py', function handle_mqtt_message
2021-01-17 12:14:30.736 Error: Python Plugin System: (Zigbee2MQTT) ----> Line 126 in '/home/pi/domoticz/plugins/zigbee2mqtt/devices/device.py', function handle_message
2021-01-17 12:14:30.736 Error: Python Plugin System: (Zigbee2MQTT) ----> Line 104 in '/home/pi/domoticz/plugins/zigbee2mqtt/devices/device.py', function get_device_args
2021-01-17 12:14:30.736 Error: Python Plugin System: (Zigbee2MQTT) ----> Line 41 in '/home/pi/domoticz/plugins/zigbee2mqtt/devices/light/dimmer.py', function get_numeric_value
Please let me know if additional info is needed, I am new to this..

Hi,
Same Ecodim.07 dimmer here, after updating the plugin domoticz log is filled with the following errors:
Previous version of the plugin was less then a week old, did not behave like this.
2021-01-17 12:14:30.735 Error: Python Plugin System: (Zigbee2MQTT) 'onMessage' failed 'TypeError':'get_numeric_value() takes 3 positional arguments but 4 were given'.
2021-01-17 12:14:30.735 Error: Python Plugin System: (Zigbee2MQTT) ----> Line 279 in '/home/pi/domoticz/plugins/zigbee2mqtt/plugin.py', function onMessage
2021-01-17 12:14:30.735 Error: Python Plugin System: (Zigbee2MQTT) ----> Line 124 in '/home/pi/domoticz/plugins/zigbee2mqtt/plugin.py', function onMessage
2021-01-17 12:14:30.736 Error: Python Plugin System: (Zigbee2MQTT) ----> Line 167 in '/home/pi/domoticz/plugins/zigbee2mqtt/mqtt.py', function onMessage
2021-01-17 12:14:30.736 Error: Python Plugin System: (Zigbee2MQTT) ----> Line 199 in '/home/pi/domoticz/plugins/zigbee2mqtt/plugin.py', function onMQTTPublish
2021-01-17 12:14:30.736 Error: Python Plugin System: (Zigbee2MQTT) ----> Line 54 in '/home/pi/domoticz/plugins/zigbee2mqtt/devices_manager.py', function handle_mqtt_message
2021-01-17 12:14:30.736 Error: Python Plugin System: (Zigbee2MQTT) ----> Line 64 in '/home/pi/domoticz/plugins/zigbee2mqtt/adapters/base_adapter.py', function handle_mqtt_message
2021-01-17 12:14:30.736 Error: Python Plugin System: (Zigbee2MQTT) ----> Line 126 in '/home/pi/domoticz/plugins/zigbee2mqtt/devices/device.py', function handle_message
2021-01-17 12:14:30.736 Error: Python Plugin System: (Zigbee2MQTT) ----> Line 104 in '/home/pi/domoticz/plugins/zigbee2mqtt/devices/device.py', function get_device_args
2021-01-17 12:14:30.736 Error: Python Plugin System: (Zigbee2MQTT) ----> Line 41 in '/home/pi/domoticz/plugins/zigbee2mqtt/devices/light/dimmer.py', function get_numeric_value
Please let me know if additional info is needed, I am new to this..

@SjoerdvN, added the fix, could you please update plugin and try again

@SjoerdvN, added the fix, could you please update plugin and try again

Confirmed to work just fine, thanks!

Confirmed to work just fine, thanks!


Add deprecated function in MultiIndex

There are many functions deprecated in Pandas 0.26 API, so I add these functions based  Pandas 0.26 API in case of unnecessary coding.
https://dev.pandas.io/docs/reference/indexing.html

Codecov Report

Merging #1140 into master will decrease coverage by 2.53%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1140      +/-   ##
==========================================
- Coverage   95.18%   92.64%   -2.54%
==========================================
  Files          35       35
  Lines        7060     7060
==========================================
- Hits         6720     6541     -179
- Misses        340      519     +179

Impacted Files
Coverage Δ

databricks/koalas/missing/indexes.py
100% <100%> (ø)
:arrow_up:

databricks/koalas/usage_logging/__init__.py
24.32% <0%> (-72.98%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50% <0%> (-50%)
:arrow_down:

databricks/koalas/__init__.py
78.72% <0%> (-6.39%)
:arrow_down:

databricks/conftest.py
92% <0%> (-6%)
:arrow_down:

databricks/koalas/frame.py
93.63% <0%> (-3.19%)
:arrow_down:

databricks/koalas/groupby.py
89.62% <0%> (-2%)
:arrow_down:

databricks/koalas/namespace.py
85.97% <0%> (-1.53%)
:arrow_down:

databricks/koalas/plot.py
93.33% <0%> (-0.96%)
:arrow_down:

databricks/koalas/testing/utils.py
78.51% <0%> (-0.75%)
:arrow_down:

... and 2 more

Continue to review full report at Codecov.

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

@RainFung, I think it's pandas 1.0.0 BTW :-)

Closing this. I think this is done at pandas 1.0.0 support. https://github.com/databricks/koalas/pull/1197 and https://github.com/databricks/koalas/pull/1299


no method named replace found for struct PathBuf in the current scope

Current master (commit 00cd266520e8518d2ff272664b5c24003c0e86e5) doesn’t build on Linux:
error[E0599]: no method named `replace` found for struct `PathBuf` in the current scope
   --> src/utils/src/lib.rs:166:19
    |
166 |         src = src.replace(r"\", "/");
    |                   ^^^^^^^ method not found in `PathBuf`

For more information about this error, try `rustc --explain E0599`.
error: could not compile `utils` due to previous error

(I’d suggest using portable methods like PathBuf::join and Path::file_name so you don’t need all these platform-specific cases.)

@andersk I forgot to mention, I'm in the middle of a large refactor, so the code's going to be broken for a bit. Since I'm by no way  a rust professional, I'm cleaning up things as I'm learning the language 😁 .
A few fixes addressed by the refactor coming up are:

using PathBuf or Path instead of using String for handling paths.
correct usage of anyhow, and removing those unwrap() calls
generally cleaner code and better variable naming


Mesh region enhancements

Close  #4551

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

Project coverage is 34.40%. Comparing base (2eb1acd) to head (c458185).
Report is 6 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #4566       +/-   ##
===========================================
- Coverage   81.51%   34.40%   -47.12%
===========================================
  Files         110      110
  Lines       53745    53750        +5
===========================================
- Hits        43811    18492    -25319
- Misses       9934    35258    +25324


Add more details of the http request

Use nt/http/httptrace for getting more detailed metrics from the http probes

Expose time taken for each phase dns resolution, connection, tls handshake, request send and reciept of first byte.
use end misc latency times from net/http/httptrace callbacks directly
Backward compatible with original http probe

cc @manugarg  this adds more fine-grained metrics

@dhananjaysathe Sorry for the delay in reviewing this PR. Unfortunately, adding more metrics affects everyone, so we'd want to enable them only through a configurable option. But, before you make any further changes, let me take a look at it.


Improve database constraints

Package.Id: Required, max length of 128
Package.VersionString: required
PackageDependencyGroup.Id: Required, max length of 128 , case insensitive on Sqlite

See https://github.com/NuGet/NuGetGallery/blob/7e0f3c7af27a0a12cdec012a39f78fafab28b0a4/src/NuGetGallery.Core/CoreConstants.cs#L10

Done by d5f957c87fb21990b1ed500fb5a59a798c47f1fd


Need to update the readme

readme is too short.

wrongly added creating here raising pull request.

closed for reversing issue on pull request.


Choose a better decision tree for slide 23

Slide link: https://gdi-orlando.github.io/gdi-featured-web-concepts/#/23/1
This decision tree is hard to explain because the content is a little esoteric. Maybe we can find an image with a more relatable example. :)

I've selected a few flow charts that might be helpful here. They are all from other websites, so it'd be nice to give credit;

https://meeraacademy.com/system-flow-diagram-for-online-shopping-system/: This has a little typo: "Regiter", instead of "Register" 🤦♂
https://quarknet.fnal.gov/projects/templates/flowchart.shtml
https://www.startuprocket.com/articles/how-to-create-a-user-experience-flow-chart-ux-flow-chart

What do you think?


Ensure 100% user story coverage in use case doc

US07.02.01, US07.04.01, US08.09.01, US09.01.01 are missing, thanks @therabidsquirel

completed by 281bb279d89a8a8cceab8ceb7fd84c0298a6a68b


Granda Mento - activate on staging & run env test

Following the success of CR5 on staging (tracked here), run the activation governance proposal on staging and run the GrandaMento env test.

Tentative staging deployment is Tuesday Aug 17


chore(main): release 0.5.0

:robot: I have created a release beep boop
0.5.0 (2024-04-16)
Features

add and document io-ts codec and zod scheme (27e3b55)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/jozan/secret/releases/tag/v0.5.0 :sunflower:


activeClassName not working

using React 0.12.0 when navigating, class to selected link is not added.

@ryanflorence is there a way to set the activeClassName for the containing  ? its working fine so I didn't feel it was worth opening a new issue but say for bootstrap pills navigation, you set the active class on the  not the , which means that by default the bootstrap active class won't work. there is https://github.com/insin/react-router-active-component component which is supposed to do that but it seems it doesn't work as of react-router v1.0


Fix Dialyzer error

Adding a required parameter would make Dialyzer fail with "The created
fun has no local return".  This is true: Maru creates a fun that
handles the case when the parameter has not been passed in, which is
supposed to throw an error.  Let's make Dialyzer not warn about it by
declaring it as "generated code".
Fixes #100.

I came across this issue with<EMAIL_ADDRESS>It works well with version 0.14.0-pre.1, which was released a long time ago.


Just an Idea: Implement sourceDateMargin in source code

It would be useful to have the possibility of having in the source of a source member the date of the row displayed at the right end of the row itself

Like the Source Dates in Gutter setting? (it's on the left though)
https://codefori.github.io/docs/developing/sourcedates/#source-date-settings


Function Delete unit tests

Instead of using testify for mocking I used gomock for both mocking and generating mock for lambda API. Though I still use testify for assertions because it's more concise.
Let me know if it's ok for you.

awesome! thanks


Add unsignedLong type

This patch adds unsignedLong type as follows

The unsigned long type is an unsigned integer type that has values in the range [0,<PHONE_NUMBER>].

ISSUE=#158

we have to add ISSUE=#ISSUE_NUM or none at end of PR message.
refer to https://github.com/lunchclass/bacardi/pull/153

and please use capital case at the first letter of PR title.

I have modified what you mentioned, can you check whether I did correctly?

I've checked the issue with your patch. So, this happens due to camelcase filter.
We expect the result of the filter with unsigned short as follows:
UnsignedShort
but actually got
Unsigned short
/cc @yjaeseok

@usefulhyun FYI, the camelcase(pascalcase) issue was resolved. https://github.com/lunchclass/bacardi/issues/168
Could you please rebase this patch and retry to fix again?

@usefulhyun You should run ./bacardi format to format your changed source code. :)

Thank you all for your help.


fix relative paths in the composite spec for compute

@amarzavery So you change the semantic of relative path :(
Expect the first build on Compute to fail, until I change it into SwaggerToSdk.
That being, I agree that it makes more sense this way....

No modification for NodeJS

No modification for Python

No modification for Ruby

No modification for Python

No modification for NodeJS

No modification for Ruby


Containing getrlimit is not implemented in jnr-posix issue

Containing issue on SLES/Linux:
Exception in thread "Thread-1" java.util.ServiceConfigurationError: com.orientechnologies.orient.core.engine.OEngine: Provider com.orientechnologies.orient.core.engine.local.OEngineLocalPaginated could not be instantiated
        at java.util.ServiceLoader.fail(ServiceLoader.java:232)
        at java.util.ServiceLoader.access$100(ServiceLoader.java:185)
        at java.util.ServiceLoader$LazyIterator.nextService(ServiceLoader.java:384)
        at java.util.ServiceLoader$LazyIterator.next(ServiceLoader.java:404)
        at java.util.ServiceLoader$1.next(ServiceLoader.java:480)
        at com.orientechnologies.orient.core.Orient.registerEngines(Orient.java:369)
        at com.orientechnologies.orient.core.Orient.startup(Orient.java:293)
        at com.orientechnologies.orient.core.Orient.startUp(Orient.java:209)
        at com.orientechnologies.orient.core.Orient.instance(Orient.java:195)
        at com.orientechnologies.orient.core.db.OrientDBInternal.remote(OrientDBInternal.java:87)
        at com.orientechnologies.orient.core.db.OrientDB.<init>(OrientDB.java:188)
        at site.ycsb.db.OrientDBClientV311.initAndGetDatabaseUrlAndDbType(OrientDBClientV311.java:134)
        at site.ycsb.db.OrientDBClientV311.init(OrientDBClientV311.java:95)
        at site.ycsb.DBWrapper.init(DBWrapper.java:86)
        at site.ycsb.ClientThread.run(ClientThread.java:91)
        at java.lang.Thread.run(Thread.java:748)
Caused by: java.lang.IllegalStateException: getrlimit is not implemented in jnr-posix

by exception handling and fallback to default values. Requires subsequent down port to 3.1.x code line.

Codecov Report

Merging #9334 into develop will decrease coverage by 0.01%.
The diff coverage is 32.50%.

@@              Coverage Diff              @@
##             develop    #9334      +/-   ##
=============================================
- Coverage      54.19%   54.18%   -0.02%
+ Complexity     35536    35533       -3
=============================================
  Files           2423     2423
  Lines         191988   191997       +9
  Branches       35020    35022       +2
=============================================
- Hits          104047   104031      -16
- Misses         73001    73033      +32
+ Partials       14940    14933       -7

Impacted Files
Coverage Δ
Complexity Δ

...java/com/orientechnologies/common/jnr/ONative.java
31.51% <32.50%> (-0.61%)
22.00 <0.00> (ø)

...ntechnologies/orient/server/OClientConnection.java
70.55% <0.00%> (-4.30%)
49.00% <0.00%> (-4.00%)

...ogies/orient/core/metadata/sequence/OSequence.java
67.68% <0.00%> (-3.05%)
56.00% <0.00%> (ø%)

...logies/orient/server/OClientConnectionManager.java
57.35% <0.00%> (-2.27%)
42.00% <0.00%> (-1.00%)

...re/storage/cache/chm/readbuffer/StripedBuffer.java
48.91% <0.00%> (-2.18%)
17.00% <0.00%> (-1.00%)

...logies/orient/core/db/viewmanager/ViewManager.java
67.06% <0.00%> (-1.19%)
51.00% <0.00%> (-1.00%)

.../orient/core/storage/cache/chm/AsyncReadCache.java
66.12% <0.00%> (-0.81%)
44.00% <0.00%> (-1.00%)

...s/orient/core/metadata/schema/OSchemaEmbedded.java
66.49% <0.00%> (-0.51%)
77.00% <0.00%> (ø%)

...nologies/orient/core/schedule/OCronExpression.java
22.16% <0.00%> (-0.36%)
50.00% <0.00%> (-1.00%)

...etwork/protocol/binary/ONetworkProtocolBinary.java
58.93% <0.00%> (-0.20%)
85.00% <0.00%> (ø%)

... and 8 more

Continue to review full report at Codecov.

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


Font sizing commands

Here is a simple test illustrating two bugs in \normalsize:
\text{a}, \text{\normalsize a}, \frac{\text{a}}{\text{\normalsize a}}

First, there's a space before a in \text{\normalsize a} (in the denominator).  Second, the size isn't actually changed!  Here's how it's supposed to render, in LaTeX: (note larger a in denominator)

I'm worried that I don't understand how sizing nodes are meant to behave in KaTeX.  This comment worries me a little:
groupTypes.sizing = function(group, options) {
    // Handle sizing operators like \Huge. Real TeX doesn't actually allow
    // these functions inside of math expressions, so we do some special
    // handling.

How are sizing commands supposed to work in math mode in KaTeX, given that they don't work in LaTeX?  In any case, the way they're working in text mode doesn't match LaTeX, so that's presumably a bug.

Font size in KaTeX is controlled by two kinds of CSS class, explicit sizing classes like .size5 and styling classes like .scriptstyle. The “sizing” code thinks that resetting the size class suffices to get a desired size. It doesn't suffice because a surrounding span might have a style class.

See #719 for a fix.

Closing since #719 has been merged.


Add Textbook and Show CurrentlyListed for user

Closed due to duplicate commit found https://github.com/The-Sciences-and-Engineering-Squad/Marketext/pull/63


Using pretrained weights

I have trained my model
The weights are now stored in unet_membrane.hdf5
I want to use these previously-stored weights for testing it on images now.
How to do it?

Use model.load_weights("pretrained_model.h5_here")

Use model.load_weights("pretrained_model.h5_here")

thank you


Not pulling on all machines

After playing with this plugin for a few days now, I have noticed that while it pulls the system and application logs on ~80% of the systems, there are some that it just won't pull the logs from and thus the plugins section remains blank. Even issuing the manual plugin eventlog pull command doesn't seem to do anything.

Windows 10 systems ~90% successful, the 10% suspecting it may have something to do with Antivirus/Firewalls (AV provider FW's).

Windows 2016 - 100% successful.

Windows 2012 R2 ~95% successful.

Windows 2008r2 0% successful, it may have something to do with how logs are accessed on older systems.

If there are any logs specifically we need to collect to help debug this please let us know.

Thanks for the report! Glad to see someone's trying it out!
The plugin relies on the Get-WinEvent Powershell command currently (which I believe was introduced back in Powershell v3). If you haven't updated on your older machines (such as the 2008R2- then this would be the reason why. Can you confirm this with the 2008R2 machines?
As for the other 90+% machines- do they not report anything back at all for the machine (live and/or history)? Does history come in sometimes and not others?
A debug flag is coming soon. I've made several changes for an upcoming version, but the release will likely be timed around when Ylian (from the MeshCentral project) gets back from his travels, as I'm trying to integrate a GUI to interface with plugins via MeshCentral to offer more choices.

You would guess it has to do a lot with the PowerShell version. 2008R2 were only shipped with Powershell V2.  I will see if I can upgrade it to 3 or even 4, but we will see the outcome.

The 2012R2's and Win10 machines may be the same problem. Interested to see what you find!

Just wanted to check in on your findings here, otherwise I'll close this out in a couple days and assume it was the Powershell versions causing the issue. Thank you.


Android support

Hello NVIDIA Team!
It was nice to see to find a link to this sample on NVIDIA Developer portal.
This moment we're actively engaging in Vulkan migration from OpenGL ES for our AR apps development.
And you mentioned "The demo is available on launch day for Windows, Linux and Embedded (L4T) and will be available in the coming days for Android."
But I've not found this.
So, can you confirm that Android support is still in your plans. Or something changed?
Best,
Dzmitry

Hi Dzmitry,

So, can you confirm that Android support is still in your plans. Or something changed?

It is up-to Google to decide if they want to allow Vulkan video on Android or not. I guess this is based on the customer's interest in these extensions. Currently, IHVs are disallowed to expose Vulkan video on Android.
Please note that L4T is Linux-Ubuntu (not Android)-based. NVIDIA has plans to release support for Vulkan video with L4T, but I'm not able to currently disclose any plans for that. If you have any specific interests in L4T, please send me an email with your required APIs (decode/encode/both) and use case, and we may be able to provide pre-release Vulkan video drivers.
Thanks,
Tony

Hello, Tony,
Thank you for the answer!
We'd like to develop Vulkan-based apps on Magic Leap 2, so our main
intention is to find representative examples which could be built on " Android
Open Source Project (AOSP), Android 10 (Q) API level 29".
So if you have any instruction, tutorials how might it be done, I'd be
appreciated for that.
Thanks,
Dzmitry
On Wed, May 1, 2024 at 12:11 AM Tony (Atanas) Zlatinski <
@.***> wrote:

Hi Dzmitry,
So, can you confirm that Android support is still in your plans. Or
something changed?
It is up-to Google to decide if they want to allow Vulkan video on Android
or not. I guess this is based on the customer's interest in these
extensions. Currently, IHVs are disallowed to expose Vulkan video on
Android.
Please note that L4T is Linux-Ubuntu (not Android)-based. NVIDIA has plans
to release support for Vulkan video with L4T, but I'm not able to currently
disclose any plans for that. If you have any specific interests in L4T,
please send me an email with your required APIs (decode/encode/both) and
use case, and we may be able to provide pre-release Vulkan video drivers.
Thanks,
Tony
—
Reply to this email directly, view it on GitHub
https://github.com/nvpro-samples/gl_vk_chopper/issues/28#issuecomment-2087335600,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGDGJHW6MNPJEZIJQ5DDTBTZAACIVAVCNFSM6AAAAABDE7CFQOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBXGMZTKNRQGA
.
You are receiving this because you authored the thread.Message ID:
@.***>


test: update beacon api spec to v2.5.0

Motivation
Reflect latest beacon-APIs spec changes.
Fixes #6170
Fixes #6187
Fixes #6206
Fixes #6167

:tada: This PR is included in v1.17.0 :tada:


Update issue templates

New issue template for opening bugs

@KhalilGarali before merging can you make a ticket for it, and rename your last commit following our commit convention (using the ticket number)?
To rename your last commit, follow this:

Type the command git commit --amend
Type i and the using your arrows move around the file to rename your commit
Type esc
Type : followed by wq and then type Enter
Lasty, push with git push --force-with-lease

@KhalilGarali before merging can you make a ticket for it, and rename your last commit following our commit convention (using the ticket number)?
To rename your last commit, follow this:

Type the command git commit --amend
Type i and the using your arrows move around the file to rename your commit
Type esc
Type : followed by wq and then type Enter
Lasty, push with git push --force-with-lease

Renamed the last commit, no need for another ticket since this is already linked.


Support for htmldjango files

The language server gets confused in htmldjango files and expands snippets as if they are css files. Would be great to use emmet-ls in htmldjango files as well.

Same for PHP Nette Framework and it's template engine called Latte. It's basically HTML with embeeded template syntax.
I looked at emmet-ls source code, but I cannot find a way to tell that also files with .latte extension should be treated as html.
It's same issue as @milkias17 reported.

@JanValiska  I got it working so what you need to do is in the emmet-ls/out/server.js there is a line that says: if languageID === "html" so just add to that conditional the filetype like if languageID === "html" || languageID === "htmldjango" I would do a pull request but couldn't get the master branch working.


double preceision floating point support

it is essential to parse Lat and Lng in double floating point rather than single float point in centimeter level high accuracy operations. its a good idea to support minmea_double style method

Closing due to inactivity but we'd love to see a PR that implements this improvement if you'd be so inclined.


use window.getComputedStyle

Thanks for this useful lib!
Because the global in NodeJS is not window, it would be better if we use  window.getComputedStyle instead of using getComputedStyle directly.
I ran into this issue when I ran my tests in jsdom. Currently, I use gobal.getComputedStyle  = window.getComputedStyle before my tests to fix it.

Very nice. Thank you :)

Thanks for merging

@wnr would you mind to release v1.1.1 for this modification?

Yes, of course. Sorry for the delay, it has been published now :)

Thank you very much :)


added back to top button

Pull Request Checklist

[x] I have added screenshots and videos to show before and after the working of my code.
[x] I have ensured that the screen size is set to 100% while making the video.
[x] I have synced the latest fork with my local repository and resolved any conflicts.
[x] I have mentioned the issue number which I created before making this PR .(format to mention issue number is : fixes #(issue number) )
[x] I understand that if any the above conditions are not met , my PR will not be MERGED .

added a back to top button on all pages. resolves issue #1586
here's what it looks like. at the bottom right corner there's a blue colored back to top button

@ANSHIKA-26 please check

Refork and resync

@ANSHIKA-26
@hiteashgupta1
please check now