Support for multiple tags!!

Hi,
Great work!! I have used the library for 3 Anchors - 1 Tag with Arduino Lilypad (USB). It works very well. It seems that the library doesn't support multiple tags yet. Is any upgrade under development currently?
Will adding the code for "if(_type==TAG)"  in "addNetworkDevices" in "DW1000RangingClass" be sufficient to extend the functionality to multiple tags? If not, can you please advice me on changes required to support multiple tags.
Thanks

Hi blue-orb !
This is great you give us a feedback for the 3 anchors, because i didn't test with it ! Thanks!
Yes, at the end, we want the library to support multiple tags.
It's not this simple..; In fact, it's the TAG which emit a message (the poll message) to all anchors in order to start ranging. This message is sent as a specific frequency. (#define DEFAULT_TIMER_DELAY 80 at the beginning of the DW1000Ranging.h) It means each message for starting a ranging cycle is send every 80ms.
(in fact it's the time for 1 anchor. When we have more anchors, the timer delay is changed dynamically in order to wait more because there is more message to be sent...)
So in conclusion, tags need to know how much tags are working together. In order to dynamically change the frequency. And the messages must be sent not in the same time. (that mean: TAG 1 need to init ranging with anchors, do ranging cycle. then TAG 2 need to init ranging with anchors, do ranging cycle.)
I didn't really think about all of that precisely, and i didn't find documentation from decawave...
Did you understand what i mean ? I hope i was enough clear ...
I will start again to work on the library soon !
cheers,
Léopold

That sounds really complex, why not use the same architecture as in the GPS technology?
where the satellites are sending information in a broadcast configurations and the receiver gets the dynamic position of each satellite and with that information can get the position.
I suggest the next architecture to create a web server system to manage multiple TAGs location

what do you think about this architecture?, do you think that is possible to create this system with the current library? Any feedback can be helpful.
Thanks

Flip that architecture around and I think you've got the beginnings of a solid TDoA setup which, according to DecaWave, can support up to 11,000 simultaneous tags (YMMV).
In the most simple terms, rather than your Anchors transmitting, they'll need to be setup to receive.
One Anchor is going to need to acting as a timekeeper and TWR the other anchors to get time of flight and timestamps.
This will allow you to calculate a time offset in the ns range for your Location Engine to measure by.
From there, you should be able to Blink your Tags periodically and then relay the receipt of that from your Anchors to the Location Engine which can then use the offsets generated by your TWR + the time-stamped receipt of the Blink from the "Master" Anchors and feed that into a multilateration algorithm to give you a position.
Or something like that anyways... I've been mulling over making an attempt at trying to put something like that together using this library but haven't found the time to get started.

HI
I found message from library about "for now let's start with 1 TAG" in DW1000Ranging.cpp
Do not yet two more TAGs are supported?

I see that you dynamically assign polling delays with the number of devices  (ANCHORS) present:
_timerDelay = DEFAULT_TIMER_DELAY+(int)(_networkDevicesNumber*3*DEFAULT_REPLY_DELAY_TIME/1000);
Where and how would be put in the check for a second TAG and dynamically assign delays for polling so that there are no conflicts?
Thanks

I also have the need for multiple tags, to work with at least one anchor. Although it may not be the most efficient, could you have the tags request a range operation before actually performing it? I'm thinking it would work something like this For tags A, and B, as well as anchor A... Tag A, and tag B both decide they'd like to do a range check at the same moment in time. They both send a request to anchor A. Tag A's request happens to get read first. Tag B's request is added to a queue of ranging requests to work through. Both tags immediately wait for a message back before starting a ranging operation. At this point, Tag A gets a message back, and does a ranging operation. Tag B however, has not heard anything back, so it assumes it has been queued and waits. Tag A's ranging operation completes and anchor A checks its queue to see that it should reply to tag B's ranging request. Tag B finally gets a message back and begins its ranging operation. Both Tags have now been ranged at the expense of one additional "ping" per range operation

I was wondering if this has been updated before I dig in and have a look?
Thank

closed in favour of  #181

I put three Anchors around the 2m*3m competition venue。
(0,0)
(2,0)
(1,3)
with height of 0.35m
And the tag is on the robot with height 0.35m。The picture below is one of the 3 Anchors

The test video is with only 3 anchors and 1 tag in channel_7.
https://drive.google.com/open?id=1TxSGrx_w0xiMKlBBKkp-kTj5YrbTJhDZ
It works great.
Then I am trying to implement 6 Anchors 2 tags system. 3 Anchors and 1 tag are all in channel_7 and the other set of 3 Anchors and 1 tag are all in channel_4. Both tags works well with around 3 minutes. Then one set of the tag starts not to receive one of the 3 Anchors in same channel.
Below picture is the two Anchors. I put 2 Anchors in same point.

(0,0) (0,0)
(2,0) (2,0)
(1,3) (1,3)
with height of 0.35m and 0.3m
I think maybe the problem is I put two anchors too close? And the electromagnetic wave is too strong, so it interrupt?
Hellow @leosayous21. How you solve your problem? Any advice is welcome
really needs help


allow updating all fields in upsert without re-declaring

Creating & Updating the same fields is very repetitive. It would be great to be able to simplify this.
_, err := r.DB.User.UpsertOne(
	db.User.Email.Equals("mock"),
).Create(
	db.User.ID.Set("user_1_mock"),
	db.User.Email.Set("mock"),
	db.User.PasswordHash.Set(r.PW.Hash("mock")),
	db.User.Name.Set("john"),
).Update(
	db.User.ID.Set("user_1_mock"),
	db.User.Email.Set("mock"),
	db.User.PasswordHash.Set(r.PW.Hash("mock")),
	db.User.Name.Set("john"),
).Exec(ctx)
if err != nil {
	return err
}

js equivalent:
const data = { id: "user_1_mock", ... }
prisma.user.upsert({
    where: { ... },
    data: data,
    update: data,
})

One suggestion: UpdateAll()?

Upsert?

This IS upsert – it's really just about minimising code if you want to update and create/update all fields in both cases.


Object.isAbstractMap()?

If no AbstractMap in prototype chain which means no o instanceof AbstractMap, maybe we need Object.isAbstractMap(o)?

I would not put this on Object, but perhaps something like this elsewhere.
At the tc39 meeting, @ljharb suggested that inserting an AbtractMap.prototype between Map.prototype and Object.prototype may be adequately compatible to be attempted. @ljharb suggested that we leave all on Map.prototype all methods currently there, but also duplicate the query methods onto AbstractMap.prototype.
We would need to attempt this on all relevant collections and either do all or none. I don't know how we'd test the web compatibility of this suggestion. But if we can determine it to be compat enough, I find it attractive.

I'll research how all the popular shims interact with collections and their prototypes over the coming weeks, which will hopefully make either an objective case that it's not web compatible, or a compelling enough case that it is likely to be to convince browsers to ship it.
It could even, perhaps, be shipped asap without exposing a global AbstractMap as a needs consensus PR, separate from this proposal, if the committee feels that this would be worth it.

It seems no reason to intentionally rely on Map.prototype.__proto__ === Object.prototype, so I am optimistic about adding AbstractMap.prototype. 😃
So I close this issue now. Could reopen if we do not have the fortune.


Default value of maxcount option of searchcount() is different from help

Describe the bug
Help file says default value of maxcount option of  searchcount() is 0, but it is actually 99.
To Reproduce
$ vim --clean
:call append(0, 'a' ->repeat(200))
:echo searchcount(#{pattern: 'a'})
" {'exact_match': 0, 'current': 100, 'incomplete': 2, 'maxcount': 99, 'total': 100}

Expected behavior
The default value of maxcount option is 0 and the following is echoed:
" {'exact_match': 0, 'current': 200, 'incomplete': 0, 'maxcount': 0, 'total': 200}

Environment

Vim 8.2.2150
macOS Catalina (version 10.15.7)
iTerm2

Let me update the help, the default value of 99 was intentional.


OpenFermion-Cirq merge: Library code

As discussed at the weekly OpenFermion sync, we have decided to merge OpenFermion-Cirq into OpenFermion. The main benefits are consolidation of functionality and removing the overhead of maintaining two separate libraries.
This pull request moves the library code from OpenFermion-Cirq, except for the variational and optimize modules, which are deprecated. I plan to move the examples in a separate pull request, but let me know if you'd like them in this one.

@kevinsung is there a way to make this draft open so I can submit commits to it?  I thought PR's are open by default but maybe not drafts?
Anyways, I updated this code to work with the new module structure.  It is now passing tests but is probably lack in coverage.
You can see the modifications on my branch of your draft PR. https://github.com/ncrubin/OpenFermion/tree/kevinsung-ofc-merge

I marked this draft ready for review; maybe that does it? In any case, I can add your commits myself. I'll also get the test coverage passing.

@ncrubin it looks like you deleted your branch. Would you mind restoring it so I can add your commits?

@kevinsung Thank you for doing this!


bird2: 2.0.5 -> 2.0.6

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/bird/versions.
meta.description for bird2 is: '"BIRD Internet Routing Daemon"'.
meta.homepage for bird2 is: '"http://bird.network.cz"

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 2.0.6 with grep in /nix/store/r13w1n53jvh4q86vn774wxp1m16iv7yj-bird-2.0.6

directory tree listing: https://gist.github.com/f95a23be8a0c27288abd0d48ce356394

du listing: https://gist.github.com/d920e866485523a8bd2ac05cf392bb7e

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)
1 package rebuild(s)
1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
bird2

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/r13w1n53jvh4q86vn774wxp1m16iv7yj-bird-2.0.6 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A bird2 https://github.com/r-ryantm/nixpkgs/archive/839d86715d164f2a51976cac155eb5357da947b2.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/r13w1n53jvh4q86vn774wxp1m16iv7yj-bird-2.0.6
ls -la /nix/store/r13w1n53jvh4q86vn774wxp1m16iv7yj-bird-2.0.6/bin

cc @fpletz @globin for testing.

@fpletz do we still need bird1 actually?

This should be considered a security issue as it fixes CVE-2019-16159. If 1.6.x isn't removed, it also needs to be bumped.

OK, I bumped both bird branches on master and 19.09 – only minor changes in these cases.  I'm not sure what to do with 19.03, as there are significantly older versions 1.6.3 and 2.0.2, and I have very little knowledge about running routing daemons.

Ditto. Thanks muchly.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/nixpkgs-update-partners-with-serokell-and-nlnet-to-add-cve-reporting/3577/5


build2: init at 0.13.0

Motivation for this change
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all packages 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/)
21.11 Release Notes (or backporting 21.05 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.

doesn't build on darwin, looks like build script requires g++ and stdenv defaults to using clang on darwin

Result of nixpkgs-review pr 136103 at 8aa632a1 run on x86_64-linux 1

1 package built successfully:

build2

1 suggestion:

warning: missing-phase-hooks
buildPhase should probably contain runHook preBuild and runHook postBuild.
Near pkgs/development/tools/build-managers/build2/default.nix:18:3:
   |
18 |   buildPhase = ''
   |   ^

There are some complicating factors which make build2 a nontrivial addition to nixpkgs:

it is built using itself, so needs a separate bootstrapping step
it is used to build other packages, so needs manual setup-hook wiring to be aware of buildInputs in the nix store (see e.g. cmake's setup hook, https://github.com/NixOS/nixpkgs/blob/6b369a4de7e6b5ed7d4519581de3bc3d7a3cd5be/pkgs/development/tools/build-managers/cmake/setup-hook.sh)

I have a local wip branch I've been toying with on and off for a couple months which attempts this. There are still some issues, such as extraneous back-references to -dev store paths. I'm happy to open a draft PR if you'd like to play around with it.

There are some complicating factors which make build2 a nontrivial addition to nixpkgs:
* it is built using itself, so needs a separate bootstrapping step

* it is used to build other packages, so needs manual setup-hook wiring to be aware of buildInputs in the nix store (see e.g. cmake's setup hook, https://github.com/NixOS/nixpkgs/blob/6b369a4de7e6b5ed7d4519581de3bc3d7a3cd5be/pkgs/development/tools/build-managers/cmake/setup-hook.sh)

I have a local wip branch I've been toying with on and off for a couple months which attempts this. There are still some issues, such as extraneous back-references to -dev store paths. I'm happy to open a draft PR if you'd like to play around with it.

I'd rather give it into your hands completely because I don't even understand the problem.
At the end of the day there are three binaries in the result and no trace of the bootstrapping...

Works for me TM

doesn't build on darwin, looks like build script requires g++ and stdenv defaults to using clang on darwin

does it work now?

i dont have a darwin machine, it was just an ofborg build log

i dont have a darwin machine, it was just an ofborg build log

upsie, my bad

I just tried to build on darwin-x86_64 and had no problem.
Feel free to ping me if you need a nixpkgs-review later on.

thanks all :]
I'm looking forward to @r-burns patches.


feat(types)!: introduce Mnemonic type that is ZeroizeOnDrop

Description of change
Consistent with the new Password type in #468 this PR adds a Mnemonic type, that makes sure sensitive data gets zeroed when the owner is being dropped.
Links to any relevant issues
None
Type of change

Enhancement (a non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)

How the change has been tested
Unit tests

Things we probably want to keep from this PR:
(a) The trimming of the input string,
(b) making the fns type safe (replace String with Mnemonic whereever possible)?

I agree, those aspects should be maintained.

Waiting for https://github.com/iotaledger/iota-sdk/pull/715 to be merged which will make their Mnemonic type available.

Changes of this PR were already done in #715


Please write a tutorial on the CRA example if possible

I appreciate that a CRA example has been included in the project, but I am wondering whether its README could be fleshed out? The README as it stands right now is a good resource and it is probably possible to recreate the CRA example following it, which is what I am going to try if this issue gets closed, but I'd feel more comfortable knowing I got it right if there was a checklist or a short guide to follow.
What I have in mind is a list of steps going from create-react-app to having the example as it is now in incremental steps. Reverse engineering this example is probably also possible, but again, it would be IMO better if this was provided in this repo as I personally and I am sure many others would feel more comfortable following such guide and knowing it was put together when rather than hacking it together ourselves and hoping our replication was correct.
What do you think?

Hello @TomasHubelbauer, improving documentation is always a good thing for me. I don't have time to write it but feel free to submit a PR about it.

I will try to put together something as I understand it for your review so you can confirm whether I got it right.

@TomasHubelbauer Were you able to put up this doc? Stuck on this for a while now :(

Unfortunately not, at the time it was a bit over my head and I've since not needed to revisit it. That said I would love to have a solution for this, too, just to know that when I do there will be something proven so I don't need to do the guess work again.

I just came across this: https://github.com/xfumihiro/jest-puppeteer-example
I won't look into it for the time being as I don't need this currently, but @dvc009 you might be interested.

A few more resources have cropped up, I am going to close the issue now and whoever lands here from a search engine can take a look at these:

https://blog.scottlogic.com/2020/01/22/react-app-puppeteer.html
https://kelvinmwinuka.com/testing-react-app-with-jest-puppeteer/
https://jestjs.io/docs/puppeteer


Move occ-models into occ

In core, we will move folder occ-models into folder occ.

This one after 523 ?

@WeizhengSap 433 has been merged


ControlSpacer

Tasks

[ ] Design, build and test the ControlSpacer component.

References

https://github.com/vime-js/vime/blob/master/core/src/components/ui/controls/control-spacer

Unnecessary.


Fixing experience calculation

line@27 - if some1 has got 1 exp in a bar then 1 / 17 is 0,058823529, casting that to (int) results in 0 exp in total, add 0.5 to fix this
line@115 - int / int does not result in a float value
+ a typo

Ohhai, haven't seen you in forever 0.o
Is this the only bug with exp code? I recall seeing someone complain about the XP tags too

It's probably the only one I could find,
And yeah quite some time :P mostly because of my studies and lack of time to work on my own plugins :P
But I needed to recall how to calculate exp (selling items for exp etc) so looked in here and fixed few things by the way :P

Fair enough. Thanks! :D
(And come on IRC sometime man)


Home Assistant Core crashes several times a day

Hi,
The last days my HA Core installation (latest version 2024.8.2) crashes several times a day.
In the home-assistant.log.1 file I see that's the last lines always are the polling instructions of the Whisperer 2 sensors (I have in total 9 and polling is every 10 minutes).
After I disabled the polling of the sensors in HA then crashing of the system seems to be gone.
This week I upgrades the HACS integration to the latest version, could this be related? It started after upgrading HACS.
At this moment I don't have any logs to show here but could this be checked that the cause could be HACS?
Thx
Bernd

Thx for your feedback. After a day or 2 without crashing HA after disable the polling of the Netro sensors I enabled it again without issues. Meanwhile there where some other integrations I updated and at this moment no problems anymore. I could be that this issue isn't Netro inegration related. Still on HA 2024.8.2, I update in the first days and let you know if this can be closed. Regards,
Bernd

This one can be closed. The rebooting of HA Core stopped several days ago. Reason unknown, but several integrations and HA itself was updated.


Add documentation on how to add 3rd Party code on Hybrid Application

Hello, I try to bootstrap a hybrid angular application with angular-cli.
Most stuff (just) works.
However integrating old code into my application seems to be a show stopper.
Currently these libraries are mostly tied to the AngularJS version and do not come with good typings.
What I tried is adding them to my package.json and run yarn install:
    "angular-i18n": "<2",
    "angular-ui-bootstrap": "^2.5.0",

This worked so far.
Then I added them to the scripts array in .angular-cli:
"scripts": [
        "../node_modules/angular-ui-bootstrap/dist/ui-bootstrap-tpls.js",
        "../node_modules/angular-i18n/angular-locale_de-de.js"
]

This looked working, too and I think it also added it to the scripts bundle.
However when I try to add 'ui.bootstrap' to my angularjs application it fails with:
Error: [$injector:nomod] Module 'ui.bootstrap' is not available! You either misspelled the module name or forgot to load it. If registering a module ensure that you specify the dependencies as the second argument.
http://errors.angularjs.org/1.6.4/$injector/nomod?p0=ui.bootstrap

Is there any way to make that work?
And also how can I ensure that the i18n package from the legacy application will be loaded?
Looks like these have the same problem:
    "angular-cookies": "<2",
    "angular-resource": "<2",
    "angular-sanitize": "^<2",

Hello, we reviewed this issue and determined that it doesn't fall into the bug report or feature request category. This issue tracker is not suitable for support requests, please repost your issue on StackOverflow using tags angular and angular-cli.
You should also mention it's an hybrid application, this is an important part of the question. It doesn't seem to me that using the CLI is affecting anything though, it sounds more like it's a problem configuring the injector in the hybrid app.
If you are wondering why we don't resolve support issues via the issue tracker, please check out this explanation.


Problem with reoccurring dates

Issue with RRULE:FREQ=WEEKLY;BYDAY=SU,MO,TU,WE,TH,FR
BEGIN:VCALENDAR
PRODID:-//Google Inc//Google Calendar 70.9054//EN
VERSION:2.0
CALSCALE:GREGORIAN
METHOD:PUBLISH
X-WR-CALNAME:Office Hours
X-WR-TIMEZONE:America/New_York
X-WR-CALDESC:
BEGIN:VTIMEZONE
TZID:America/Los_Angeles
X-LIC-LOCATION:America/Los_Angeles
BEGIN:DAYLIGHT
TZOFFSETFROM:-0800
TZOFFSETTO:-0700
TZNAME:PDT
DTSTART:19700308T020000
RRULE:FREQ=YEARLY;BYMONTH=3;BYDAY=2SU
END:DAYLIGHT
BEGIN:STANDARD
TZOFFSETFROM:-0700
TZOFFSETTO:-0800
TZNAME:PST
DTSTART:19701101T020000
RRULE:FREQ=YEARLY;BYMONTH=11;BYDAY=1SU
END:STANDARD
END:VTIMEZONE
BEGIN:VEVENT
DTSTART;TZID=America/Los_Angeles:20180506T100000
DTEND;TZID=America/Los_Angeles:20180506T103000
RRULE:FREQ=WEEKLY;BYDAY=SU,MO,TU,WE,TH,FR
DTSTAMP:20180722T165639Z
<EMAIL_ADDRESS>CREATED:20180721T064200Z
DESCRIPTION:
LAST-MODIFIED:20180721T065236Z
LOCATION:
SEQUENCE:1
STATUS:CONFIRMED
SUMMARY:Ok
TRANSP:OPAQUE
END:VEVENT
BEGIN:VEVENT
DTSTART;VALUE=DATE:20180129
DTEND;VALUE=DATE:20180130
DTSTAMP:20180722T165639Z
<EMAIL_ADDRESS>CREATED:20180130T220521Z
DESCRIPTION:
LAST-MODIFIED:20180130T220555Z
LOCATION:
SEQUENCE:1
STATUS:CONFIRMED
SUMMARY:All Day
TRANSP:TRANSPARENT
END:VEVENT
END:VCALENDAR

In the rule above I have Sunday, Monday, Tuesday, Wednesday, Thursday, Friday.
However the result I get from icalparser skips Mondays (the even started ON a Monday)

I figured out that this is a bug in freq.php
If you debug Freq.php (turn $debug = true in method findNext). You'll see this
O: Mon, 03 Dec 2018 00:00:00 -0800
START: Mon, 03 Dec 2018 10:00:00 -0800
EOP: Mon, 10 Dec 2018 00:00:00 -0800
DAY: Sun, 09 Dec 2018 10:00:00 -0800 A: 1
DAY: Mon, 10 Dec 2018 10:00:00 -0800 A: 0
DAY: Tue, 04 Dec 2018 10:00:00 -0800 A: 1
DAY: Wed, 05 Dec 2018 10:00:00 -0800 A: 1
DAY: Thu, 06 Dec 2018 10:00:00 -0800 A: 1
DAY: Fri, 07 Dec 2018 10:00:00 -0800 A: 1
OK

Notice "DAY: Mon, 10 Dec 2018" is A: 0. Thats because the EOP (End of period) is set to 12:00am. On Monday the Event goes until 10:00:00. So it's outside the range of the EOP. However it shouldn't be. The EOP should be the same time as START (not the same date).
I was able to fix this by making sure the EOP time is set to the time of START (I didn't change the date though)
Freq.php Line 279:
Replace:
$eop = $this->findEndOfPeriod($offset);
with
		//EOP needs to have the same TIME as START ($t)
		$tO = new<EMAIL_ADDRESS>new \DateTimeZone('UTC'));

		$eop = $this->findEndOfPeriod($offset);
		$eopO = new<EMAIL_ADDRESS>new \DateTimeZone('UTC'));
		$eopO->setTime($tO->format('H'),$tO->format('i'),$tO->format('s'));
		$eop = $eopO->getTimestamp();
		unset($eopO);
		unset($tO);

I still need to run through your unit tests to confirm this won't break anything

Thanks @tm1000 - that fix worked for me.

@andreas-globi  Thanks! I ran the unit tests against my fixes and they all worked so I am proposing PR #32

Hi everyone.
Hello tm1000 :)
There's an issue for this rule:
RRULE:FREQ=WEEKLY;INTERVAL=2;BYDAY=MO,TU,WE,TH,FR
The result if like:
RULE:FREQ=WEEKLY;INTERVAL=1:BYDAY=MO,TU,WE,TH,FR
the link for tests:
https://calendar.google.com/calendar/ical/reihrbu4mkcm9irjvr934r0rds%40group.calendar.google.com/private-8713c756bc73683538f110cbb2350616/basic.ics
I tested the calendar with example/index.php and it returns a wrong result.
If you could help me please?


Partially resolved issue #4687 by changing the improper status code for concurrent snapshot executions

Signed-off-by: Mani<EMAIL_ADDRESS>Description
Partially resolved issue #4687 by changing the improper status code for concurrent snapshot execution exception.
Issues Resolved
#4687
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@Bukhtawar Hi, for issue #4687, should I make a PR by resolving Repository exception status as well ?
Also, any comments on this one ?
cc: @andrross

Hi @Bukhtawar, is this ok ?
Please let me know your comments on this.
Thanks.

The code change looks good, while there is code format issue, running ./gradlew :server:spotlessApply will resolve the CI failure.
1: Task failed with an exception.
-----------
* What went wrong:
Execution failed for task ':server:spotlessJavaCheck'.
> The following files had format violations:
      src/test/java/org/opensearch/OpenSearchExceptionTests.java
          @@ -121,7 +121,7 @@
           ········exception·=·new·RemoteTransportException("test",·new·IllegalStateException("foobar"));
           ········assertThat(exception.status(),·equalTo(RestStatus.INTERNAL_SERVER_ERROR));

          -········exception·=·new·ConcurrentSnapshotExecutionException("testRepo","testSnap","test");
          +········exception·=·new·ConcurrentSnapshotExecutionException("testRepo",·"testSnap",·"test");
           ········assertSame(exception.status(),·RestStatus.TOO_MANY_REQUESTS);
           ····}

  Run './gradlew :server:spotlessApply' to fix these violations.

@baba-devv Would you please address @tlfeng's comments on fixing code formatting errors. You can also create a CHANGELOG.md entry to have a passing Changelog verifier task. Please let us know if you need any help with this PR.

@DarshitChanpura Sure, I'll close this shortly.

@dblock So there were incorrect status codes for two exception in #4687 .

ConcurrentSnapshotException
RepositoryException(bucket not found)
Missing S3 bucket belongs to the second one, what I've changed is for the first one, hence the name partial.

Looks like the rebase brought some other stuff in here (1700+ files changed 😅). Let us know if you need help?

@dblock yes, I did pulled & after raising the PR I got these issues. Sorry for that. What would you recommend in this case, pulling doesn't help as its not updating anything.

You should squash this, the change is simple but we are bringing in 18 commits, if it helps https://code.dblock.org/2015/08/31/getting-out-of-your-first-git-mess.html may be an easy way,.

Superseded by #8986


Proposal for quickstart update

@nikolasburk I took a quick stab at this - I focused on the very beginning to help people jump into it faster. I think the actual flow of the rest of the quickstart is great and at the appropriate depth. Even showing the tables in the sqlite database is IMO more useful than hiding them.

I think that looks great, thanks so much Matej! 🙏


Error: Attempting to watch missing directory

After installing the add-on and running ember server, I get the following error.
The Broccoli Plugin: [Funnel] failed with:
Error: Attempting to watch missing directory: C:\Program Files\Git\ember-panels
node_modules\sl-ember-components\node_modules\sl-ember-components\app
at EventEmitter.Watcher_addWatchDir [as addWatchDir](C:Program FilesGite
mber-panelsnode_modulesember-clinode_modulesbroccoli-sane-watcherindex.js:9
0:11)
at C:\Program Files\Git\ember-panels\node_modules\ember-cli\node_modules\emb
er-cli-broccoli\lib\builder.js:95:35
at lib$rsvp$$internal$$tryCatch (C:\Program Files\Git\ember-panels\node_modu
les\ember-cli\node_modules\rsvp\dist\rsvp.js:1036:16)
at lib$rsvp$$internal$$invokeCallback (C:\Program Files\Git\ember-panels\nod
e_modules\ember-cli\node_modules\rsvp\dist\rsvp.js:1048:17)
at C:\Program Files\Git\ember-panels\node_modules\ember-cli\node_modules\rsv
p\dist\rsvp.js:331:11
at lib$rsvp$asap$$flush (C:\Program Files\Git\ember-panels\node_modules\embe
r-cli\node_modules\rsvp\dist\rsvp.js:1198:9)
at _combinedTickCallback (node.js:370:9)
at process._tickCallback (node.js:401:11)
The broccoli plugin was instantiated at:
at Funnel.Plugin (C:\Program Files\Git\ember-panels\node_modules\sl-ember-co
mponents\node_modules\broccoli-funnel\node_modules\broccoli-plugin\index.js:7:31
)
at new Funnel (C:\Program Files\Git\ember-panels\node_modules\sl-ember-compo
nents\node_modules\broccoli-funnel\index.js:44:10)
at Class.module.exports.treeForVendor (C:\Program Files\Git\ember-panels\nod
e_modules\sl-ember-components\index.js:85:17)
at Class._treeFor (C:\Program Files\Git\ember-panels\node_modules\ember-cli
lib\models\addon.js:322:31)
at Class.treeFor (C:\Program Files\Git\ember-panels\node_modules\ember-cli\l
ib\models\addon.js:290:19)
at C:\Program Files\Git\ember-panels\node_modules\ember-cli\lib\broccoli\emb
er-app.js:446:20
at Array.map (native)
at EmberApp.addonTreesFor (C:\Program Files\Git\ember-panels\node_modules\em
ber-cli\lib\broccoli\ember-app.js:444:30)
at EmberApp._processedVendorTree (C:\Program Files\Git\ember-panels\node_mod
ules\ember-cli\lib\broccoli\ember-app.js:865:29)
at EmberApp._processedExternalTree (C:\Program Files\Git\ember-panels\node_m
odules\ember-cli\lib\broccoli\ember-app.js:895:21)

What version of Node are you running?

v5.7.1

Does running rm -rf node_modules && npm cache clean && npm install resolve the issue?

I ran the command but there was some errors during. Now when I run ember s, I get a different error:
Error: Could not require 'ember-cli-build.js': Cannot find module 'ember-cli/lib
/broccoli/ember-app'

same thing here, when I disabled check at index.js -> treeForVendor() (line 82)
var folder = ( Number( process.version.match( /^v(\d+)/ )[1] ) >= 5 ) ? this.name : '../';

so that the folder was '../', it worked.
ember-cli: 2.8.0-beta.1
node: 6.2.1
os: win32 x64

I have this issue too. I installed sl-ember-components at v0.12.3, cleared the npm cache, removed node_modules, and reinstalled everything, and it's giving me this error. I don't know why it's trying to watch this directory (node_modules/sl-ember-components/node_modules/sl-ember-components/app), because it doesn't make any sense for a package to have itself in node_modules anyway.

@jdhines i think this addon is abandoned, there were no major fixes for at least a year, in the end i just gave up and used ember-bootstrap + bunch of smaller addons.

@jdhines @vlascik This problem seems related to the same cause of https://github.com/softlayer/ember-cli-jsdoc/issues/19 and https://github.com/softlayer/ember-cli-jsdoc/issues/36.  I don't have time at the moment to test all of the iterations of the code but hope to get a fix in for this soon.
@jdhines This package does not have itself in node_modules, it is due to the problem described in the previously-listed issues.  watchman is not watching this directory for any changes, however it does access the directory in order to add LESS-generated CSS into the vendor tree of a consuming app to be imported in included().  More information about this can be found at https://github.com/softlayer/sl-ember-components#styling and https://github.com/softlayer/sl-ember-components/blob/master/index.js#L70-L77
@vlascik This project no longer has full-time development resources being dedicated to it.  Myself and @juwara0, who were the primary contributors no longer work at SoftLayer, and both SoftLayer and our present employer's focus have shifted away from this library.  I try and keep up with any bugs that are reported, such as making modifications last month to be able to support Ember 2.11.0+.  Both @juwara0 and myself remain very interested in the space and any efforts we contribute to it have been undertaken at https://github.com/ember-forge/ember-forge-ui, though you will notice even that project has not been able to be given full-time attention either.  If you, @jdhines, or others have interest in the promise of what https://github.com/ember-forge/ember-forge-ui can provide (of which a lot has already been written and the architecture proven out) please reach out and we can discuss collaboration.
Thank you for your interest in, and use of, this project.

I have been able to replicate this problem.
Serving an app when running

node: v7.8.0
ember-cli: v2.12.1
ember-cli-babel: v5.2.4
sl-ember-components: v0.12.3

there is no problem.  When I upgraded ember-cli-babel to v6.1.0 I encountered this error thougH
Error: Attempting to watch missing directory: /notmessenger/project/node_modules/sl-ember-components/node_modules/sl-ember-components/app
    at Watcher_addWatchDir [as addWatchDir] (/notmessenger/project/node_modules/ember-cli-broccoli-sane-watcher/index.js:142:11)
    at //notmessenger/project/node_modules/broccoli-builder/lib/builder.js:132:35
    at tryCatch (/notmessenger/project/node_modules/rsvp/dist/rsvp.js:539:12)
    at invokeCallback (/notmessenger/project/node_modules/rsvp/dist/rsvp.js:554:13)
    at /notmessenger/project/node_modules/rsvp/dist/rsvp.js:629:16
    at flush (/notmessenger/project/node_modules/rsvp/dist/rsvp.js:2414:5)
    at _combinedTickCallback (internal/process/next_tick.js:73:7)
    at process._tickCallback (internal/process/next_tick.js:104:9)


Update http4s-circe, http4s-client, ... to 1.0.0-M18

Updates

org.http4s:http4s-circe
org.http4s:http4s-client
org.http4s:http4s-core

from 1.0.0-M3 to 1.0.0-M18.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, semver-pre-release

Superseded by #40.


Deploy app

Need automate deploy app when have a correct push to main branch
More simple way is deploy to vercel

Configured automated deploy in vercel to main branch


FR: Improve German Localization

[x] I have verified that I am on the latest version of the Linter (1.19.1)

Describe the Bug
The german localization is very poor and partly not understandable.
One example is the word "Flusen".
How to Reproduce
Steps to reproduce the behavior:

Use Obsidian and the Linter plugin with german language

Expected Behavior
A understandable german ui.
Screenshots

Device

[x] Desktop
[ ] Mobile

Comment
I could help you with german translation (is my natural language).

Help with that translation would be greatly appreciated. I do not speak German, so what is there is just a google translate of the values.
There is some information on how to do the translation listed here on the new wiki which I am still linking into the plugin here: https://platers.github.io/obsidian-linter/contributing/translation/#addingupdating-values-for-an-existing-language

Hey @afischer211 , I wanted to reach back out and see if you had any suggestions around updating the German translation for the Linter. I know that you said you would be willing to help, so I thought I would reach out and ask. Feel free to let me know if anything needs better documentation or what wordings are and what they should be.

Hello,
I'm starting with the work and hopefully I can create a PR until end of the week.

Thanks @afischer211 . Feel free to reach out if you run into any issues.


chaining setters looses exact Widget type

I would like to chain some setters together with the initialization, e.g.:
auto vl = new VerticalLayout("vl").layoutWidth(FILL_PARENT).layoutHeight(FILL_PARENT);

Problem is, that vl now is of type widget, as layoutWidth and layoutHeight are inherited from Widget and return this (a Widget).
It would be great if there is a clever d-trick to let them return the correct subtype.
As a workaround the two liner:
d
auto vl = new VerticalLayout("vl");
vl.layoutWidth(FILL_PARENT).layoutHeight(FILL_PARENT);
````
always works, but it still would be nice.

not sure how to do it

trick: use DML for creation of layouts.
Below is equivalent of your code:
//import dlangui.dml.parser;
//import dlangui.widgets.layouts;
auto vl = parseML!VerticalLayout(q{ VerticalLayout { layoutWidth: fill; layoutHeight: fill }  });

nice. thanks ... i will give this a try. i also do not see a way to do this with d-lang tools.


Allows scalac ignores java errors.

Java errors are reported to SbtBuildReporter by java compiler
directly.
Commit removes a bug when java depends on scala code (see SbtInputs)
Fixes #1002738

LGTM


set-up size-plugin-bot 🤖

Items:

[x] Add rollup plugin rollup-plugin-size
[x] Add configuration file(.github/size-plugin.yml) for size-plugin 🤖
[x] Install Github App  👈@tannerlinsley

@tannerlinsley is Travis enabled on this repository? couldn't see any the checks

It is now
Tanner Linsley
On Nov 29, 2019, 9:30 PM -0700, anotherjsguy<EMAIL_ADDRESS>wrote:

@tannerlinsley is Travis enabled on this repository? couldn't see any the checks
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.


Move to Julia 1.1 support

BREAKING CHANGE: Drop support for Julia v0.6.
Moves the codebase to supporting Julia v1.1 (and up?). This requires updating how Lint.jl is installed in CI and the server to a method that is incompatible with the older versions. Currently since Lint.jl hasn't been updated in the Julia registry we need to install it from their master branch.
Also note that the specs have been changed since it seems that Lint.jl now reports a different line number for the error in question... that is even further off from what it should be.
As I know essentially nothing about Julia it would be great if somebody more familiar with it could review this and give it a sanity check. @TeroFrondelius? @TotalVerb?

Maybe switch to https://github.com/ZacLN/StaticLint.jl? I actually started looking into StaticLint.jl, but it's missing documentation and I couldn't easily figure it out.

The above method works, the issues with it are:
*Anyone updating to this version would need to run through the same installation steps currently to get a working version of Lint.jl

They published a release 3 days ago, but it's not in the Julia registry (yet?)
I don't know Julia, so the code for the server is largely a hack that works, but I don't know if there is a cleaner way to do it

I'm going to merge this as it gets the package into a usable state again, if it's not the correct way to do this it's likely to be a better base to start off of than the current state.


Millisecond precision timestamps seems to be hardcoded?

CREATE TABLE trade (
  xch TEXT,
  sym TEXT,
  px FLOAT,
  qty FLOAT,
  flag tinyint,
  ts TIMESTAMP(6),
  rts smallint,
  PRIMARY KEY(xch, sym),
  TIME INDEX(ts)
);

public=> desc table trade;
 Field |         Type         | Null | Default | Semantic Type
-------+----------------------+------+---------+---------------
 xch   | String               | YES  |         | PRIMARY KEY
 sym   | String               | YES  |         | PRIMARY KEY
 px    | Float32              | YES  |         | FIELD
 qty   | Float32              | YES  |         | FIELD
 flag  | Int8                 | YES  |         | FIELD
 ts    | TimestampMicrosecond | NO   |         | TIME INDEX
 rts   | Int16                | YES  |         | FIELD

	for i := 0; i < tickList.Len(); i++ {
		tck := tickList.At(i)
		px, _ := tck.P()
		qty, _ := tck.Q()
		dpx, err := decimal.NewFromString(px)
		if err != nil {
			return err
		}
		dqty, _ := decimal.NewFromString(qty)
		series := gt.Series{}
		series.AddTag("xch", topic[len(topic)-1])
		series.SetTimestamp(time.UnixMicro(int64(tck.Lts())))
		series.AddField("rts", tck.Rts())
		series.AddField("sym", m.Key)
		series.AddField("px", dpx.InexactFloat64())
		series.AddField("qty", dqty.InexactFloat64())
		series.AddField("flag", string(tck.Flag()))
		metric := gt.Metric{}
		metric.AddSeries(series)
		req := gt.InsertRequest{}
		req.WithTable("trade").WithMetric(metric)
		log.Debugf("%s %s %f %f %s %s", topic[len(topic)-1], m.Key, dpx.InexactFloat64(), dqty.InexactFloat64(), string(tck.Flag()), time.UnixMicro(int64(tck.Lts())))

		affectedRows, err := client.Insert(ctx, req)
		if err != nil || int(affectedRows) != 1 {
			return err
		}
	}

I get
rpc error: code = Internal desc = Failed to insert value to table: greptime.public.trade, source: Failed to operate table, source: Type of column ts does not match type in schema, expect Timestamp(Microsecond(TimestampMicrosecondType)), given Timestamp(Millisecond(TimestampMillisecondType))

hi, sorry for the inconvenience, but you can SetTimePrecision as you need.
By the way, there is no need to create the table via schema in advance, it will be created automatically and the data type will match.
And greptimedb-client-go is planned to adjust to the new version of proto, and it will behave better thereafter.

if datatype does not match, you can change your tinyint and smallint to int in your schema, and retry it. Any comment is welcome.


Add manager validation to an Ansible Service Template

Allow to validate that we have 1 credential, 1 repo, 1 playbook inside
our Ansible Tower provider
https://www.pivotaltracker.com/story/show/139469385

@miq-bot add_label enhancement, providers/ansible_tower
@miq-bot assign @bzwei
cc - @gmcculloug

@bdunne The workflow you described is different from the UI mock-up (coming from PM?) The spec requires such check before the form is rendered.
The logic proposed here have been evolved from multiple refactoring and tests. We can decide where to put this code and how to call it, but the code itself looks good and can be reused.

cc @h-kataria

@bzwei Can you or someone else describe the workflow?

@bdunne @bzwei is correct as soon as user selects an entry in the Drop down to try and add an Ansible Playbook type catalog item, before form is rendered we need to check if there are at least 1 Repository, 1 Playbook and 1 Machine Credential in VMDB, if not then just display flash message and do not try to render the form.

@h-kataria
Why does the UI need to check before the form is rendered?  Why can't it check after fetching the choices for the drop-downs?
How will the service UI do this?  They can't call the method.
In the future, when the user first needs to select the provider, will only the providers meeting these criteria be available for selection?


Infrastructure for training large models

e.g. with reference to deep speed code in TOFU codebase
What type of training? (full fine-tuning vs. PEFT etc.)

Cache directories (models, datasets, WandB, ...)


Revoke Button Text

The Revoke Button text should be editable, and not Cookie Policy, which doesn't explain what it's form.
I would recommend it say "Revoke Cookies" or even better, have it be a smaller, float-on-scroll button on the side of the browser with a + sign or maybe of a cookie with the "no smoking" circle/slash  that expands out to show "Revoke Cookies".
This would be a more familiar location for the tab, regardless of where the full alert is located and the text itself makes more sense than what it is now.

The Revoke Button text should be editable, and not Cookie Policy, which doesn't explain what it's form.
I would recommend it say "Revoke Cookies" or even better, have it be a smaller, float-on-scroll button on the side of the browser with a + sign or maybe of a cookie with the "no smoking" circle/slash that expands out to show "Revoke Cookies".
This would be a more familiar location for the tab, regardless of where the full alert is located and the text itself makes more sense than what it is now.

If you still wish to change it you do you by adding "policy":"Cookie Policy" to content options. Like
"content": {     "message": "This website uses cookies.",     "dismiss": "Close",     "link": "More Info",     "href": "https://www.patchworkoftips.com/cookie-policy/", 	"target":"_blank", 	"policy":"Cookie Policy"
Just change Cookie Policy to whatever text you like


[Bug] The timed task of the cloud function trigger is invalid

Search before asking

[X] I had searched in the issues and found no similar issues.

Environment
lafyun.com
CPU Architecture
amd64
Versions
预览版
What happened

一次不会执行
How to reproduce
设置了定时任务一次都不会执行
Debug logs
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Known issues will be fixed in the next release.


 Let's take a look error message.
message: "A network error (such as timeout, interrupted connection or unreachable host) has occurred."
I have no idea how to solve this. so, googling and find a issue.
https://github.com/angular/angularfire2/issues/581
After reading issue, I know that it's ok on firefox.
firefox is better than chrome?
I can't believe that. I love google. How can i resolve this issue on chrome?
force be with you and let me know soon, thanks.

@jmdobry ptal

@softcontext given the error message it looks like this may have been a temporary issue. Please let us know you are still experiencing this with the latest codelab and we'll re-open this.


Build docker image from codebase.

I suspect most folk expect to be able to build a Docker image with the
latest code from the Git repo rather than needing to wait for a release
being pushed to NPM.  This achieves that without changing much else.

Note that there is possibly some extra cleanup that can be done as the resulting container is slightly larger than the current one that just installs Nelson from NPM.
% docker images
REPOSITORY          TAG                 IMAGE ID            CREATED             SIZE
rosstimson/nelson   latest              1ea042349579        4 minutes ago       113MB
romansemko/nelson   latest              b38a4a7f726a        21 hours ago        83.4MB

I don't see that slight size increase as a blocker for this though.
Also, line 7 npm install -g ., I'm not that familiar with the NodeJS ecosytem, perhaps there is a better way to do this using Yarn.

Thanks! This will make the maintenance of the Dockerfile easier as well. I do not think that the slight increase in size is a blocker at the moment.
I think with yarn you do "yarn global install", but npm does practically the same.


Use static linking

This makes changes the way the binary is compiled, so it is statically linked.
This is good so we eliminate some system dependencies that weren't present on some basic images.

/lgtm
/approve


Vector#set doesn't work for Category vectors

v = Daru::Vector.new([1,2,3])
vc = v.to_category
vc.send(:set, 5, 6)
# NoMethodError: undefined method `<<' for nil:NilClass

I believe the reason is category vector, while imitating being the same vector, cleans up @data variable. Probably it is not the only broken method for category vectors.
Showcased by https://github.com/SciRuby/daru/pull/388
@lokeshh WDYT?

@zverok Yes, :set method is missing for category vector. We have to fix this.

I added these lines of code but it didn't work. Should I send it as a PR?


Slight cleanup of C++ EngineBuilder

Make generateConfigStr() const (to ensure that it produces the same output if called multiple times).
Add a small helper method for inserting custom filter configs into the template.
Risk Level: Low
Testing: No behavior change
Docs Changes: N/A
Release Notes: N/A

/retest

/assign @alyssawilk


Risk of using default password

Can anybody comment on the risk of using the default password? I realize that nobody can connect unless they hack my WiFi password. Other than that, are there any risks? I have already setup my iDevices with the default password and just don't want to go through the setup process again.

That password is only used for first time setup, after that, people cannot add the accessory using the direct add method, instead it has to use HomeKit's user sharing method. It shouldn't be a huge problem.


Add default value to fields in JobConfigurationPOJO

Fixes #1052 .
Changes proposed in this pull request:

Set maxTimeDiffSeconds default -1 in JobConfigurationPOJO
Complete test cases

+1


fix: allow users with read only permission to read APIs

:warning: This is a report of the bugfix of https://github.com/gravitee-io/issues/issues/6475, that has been done in 3.5 :warning:
:warning: But we didn't delivered it in 3.5 cause some customers are using this bug as a feature :warning:
:warning: Before merging, wait for confirmation that the new behavior is OK for all our customers :warning:
https://github.com/gravitee-io/issues/issues/7243
fix: allow users with read only permission to read APIs
The required permissions to access APIs from the console were set
to Create, Update or Delete, making user with the Read permission
attached to their role unable to access APIs, be it from a group
or a direct membership.

🚀 CI was able to deploy the build of this PR, so you can now try it directly here
Notes: The deployed app is linked to the management API of the Element Zero team's environment.

As discussed with CSM and product team :
This behavior should not change for now, or it will break several customers use-cases.
There is currently a "hack" that permits to see read-only APIs on console, by enabling any other create/update/delete permission.
This is a just a workaround, waiting for this feature to be implemented : https://github.com/gravitee-io/issues/issues/5414
So, I close the ticket cause there is nothing to fix actually.


Added Currency convertor

Title and Issue number
Currency Converter #1990
Close #1990
Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have created a helpful and easy to understand README.md

https://github.com/pranjay-poddar/Dev-Geeks/assets/133862694/f92c7ebd-2a75-4ca4-96aa-ddaa5dad5248

@MohitGupta121


CSE_ModbusRTU_ADU class

My project is about to control heat recovery ventilation unit in house. I am able to read some input registers now (ADU, CRC, send, receive on serial monitor is correct). I am trying to use class CSE_ModbusRTU_ADU. I have defined
CSE_ModbusRTU_ADU adu;

and then in setup(),
CSE_ModbusRTU_ADU();

and in the loop() I put this line,
unsigned int aduwrd = adu.getLength();

or getByte ..... but I get always 0x00 even though on serial monitor all ADU data is present and correct. Is it possible to explain how to use CSE_ModbusRTU_ADU class in some sample code? Thank you. My Mega 2560 board is client and ventilation unit is server.

Hi @PedroPiko. Sorry for the delay after our first interaction. I am editing the styling of your comment to make it easy to read.
The class CSE_ModbusRTU_ADU can be used to create standlone ModbusRTU packets. The usage is well explained in the API.md document. In order to read from the ADU, you will need to first write data to it. When the CSE_ModbusRTU_ADU class is used in conjunction with the CSE_ModbusRTU_Server and CSE_ModbusRTU_Client classes, they take care of writing to the ADUs.
So when using the CSE_ModbusRTU_ADU directly, you need to make sure that you add data to the object first.
I also don't understand why you have,
CSE_ModbusRTU_ADU();

in your setup() function. It is unnecessary. Global objects are instantiated where they are declared.
Please let me know if you have gained a better understanding of the library while I was away, and any other suggestions for improving the library and its documentation.


Optimize the Docker images

The size of the Docker images has a direct impact on the time that it takes to get the Airy Core Platform up-and-running. Also, since we package some of the images in the Vagrant box, this will have an impact on the size of the box and the time it takes for downloading it.
Currently used Docker images include:

Kafka, Zookeeper, Kafka schema registry
Postgres, Redis
Airy Core apps
Images required for Kubernetes to run properly

This is the current status of the images:
For the base services:
confluentinc/cp-kafka            598MB
confluentinc/cp-zookeeper        598MB
confluentinc/cp-schema-registry  1.21GB
postgres                         158MB
redis                            40.9MB

For the apps:
ghcr.io/airyhq/api/admin                       365MB
ghcr.io/airyhq/sources/twilio-events-router    361MB
ghcr.io/airyhq/sources/twilio-webhook          354MB
ghcr.io/airyhq/webhook/publisher               368MB
ghcr.io/airyhq/sources/facebook-sender         361MB
ghcr.io/airyhq/sources/google-events-router    361MB
ghcr.io/airyhq/sources/chat-plugin             366MB
ghcr.io/airyhq/sources/facebook-events-router  361MB
ghcr.io/airyhq/sources/google-webhook          349MB
ghcr.io/airyhq/sources/facebook-webhook        349MB
ghcr.io/airyhq/frontend/demo                   26.4MB
ghcr.io/airyhq/webhook/consumer                27.1MB
ghcr.io/airyhq/sources/twilio-sender           366MB
ghcr.io/airyhq/api/auth                        310MB
ghcr.io/airyhq/sources/google-sender           363MB
ghcr.io/airyhq/frontend/chat-plugin            21.9MB
ghcr.io/airyhq/api/communication               366MB

At the moment, we will focus the most at Kafka, Zookeeper and the Schema registry.

The image for Kafka and Zookeeper has been optimized. Both statefulsets at the moment use the same image, only with different arguments to the entrypoint of the container.
Still left is the Schema registry image.

The image for Kafka and Zookeeper has been optimized. Both statefulsets at the moment use the same image, only with different arguments to the entrypoint of the container.
Still left is the Schema registry image.

finished in #596 🎉

finished in #596 🎉


Backport: ABFS. HADOOP-17296 to 17404

Cherry-picked the following commits:
HADOOP-17296
HADOOP-17413
HADOOP-17422
HADOOP-17407
HADOOP-17347
HADOOP-17404

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
29m 20s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s
No case conflicting files found.

+0 :ok:
markdownlint
0m  0s
markdownlint was not available.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 15 new or modified test files.

_ branch-3.3 Compile Tests _

+1 :green_heart:
mvninstall
32m 34s
branch-3.3 passed

+1 :green_heart:
compile
0m 36s
branch-3.3 passed

+1 :green_heart:
checkstyle
0m 28s
branch-3.3 passed

+1 :green_heart:
mvnsite
0m 36s
branch-3.3 passed

+1 :green_heart:
shadedclient
16m 27s
branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 27s
branch-3.3 passed

+0 :ok:
spotbugs
0m 58s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
0m 56s
branch-3.3 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 29s
the patch passed

+1 :green_heart:
compile
0m 27s
the patch passed

+1 :green_heart:
javac
0m 27s
the patch passed

+1 :green_heart:
checkstyle
0m 19s
the patch passed

+1 :green_heart:
mvnsite
0m 31s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
xml
0m  3s
The patch has no ill-formed XML file.

+1 :green_heart:
shadedclient
15m 38s
patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 23s
the patch passed

+1 :green_heart:
findbugs
0m 59s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 28s
hadoop-azure in the patch passed.

+1 :green_heart:
asflicense
0m 33s
The patch does not generate ASF License warnings.

103m 57s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2618/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2618

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient xml findbugs checkstyle markdownlint

uname
Linux 52c65b4cc2eb 4.15.0-65-generic #74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-3.3 / 75f34a1

Default Java
Private Build-1.8.0_275-8u275-b01-0ubuntu1~16.04-b01

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2618/1/testReport/

Max. process+thread count
538 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-azure U: hadoop-tools/hadoop-azure

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2618/1/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
29m 20s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s
No case conflicting files found.

+0 :ok:
markdownlint
0m  0s
markdownlint was not available.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 15 new or modified test files.

_ branch-3.3 Compile Tests _

+1 :green_heart:
mvninstall
32m 34s
branch-3.3 passed

+1 :green_heart:
compile
0m 36s
branch-3.3 passed

+1 :green_heart:
checkstyle
0m 28s
branch-3.3 passed

+1 :green_heart:
mvnsite
0m 36s
branch-3.3 passed

+1 :green_heart:
shadedclient
16m 27s
branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 27s
branch-3.3 passed

+0 :ok:
spotbugs
0m 58s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
0m 56s
branch-3.3 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 29s
the patch passed

+1 :green_heart:
compile
0m 27s
the patch passed

+1 :green_heart:
javac
0m 27s
the patch passed

+1 :green_heart:
checkstyle
0m 19s
the patch passed

+1 :green_heart:
mvnsite
0m 31s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
xml
0m  3s
The patch has no ill-formed XML file.

+1 :green_heart:
shadedclient
15m 38s
patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 23s
the patch passed

+1 :green_heart:
findbugs
0m 59s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 28s
hadoop-azure in the patch passed.

+1 :green_heart:
asflicense
0m 33s
The patch does not generate ASF License warnings.

103m 57s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2618/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2618

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient xml findbugs checkstyle markdownlint

uname
Linux 52c65b4cc2eb 4.15.0-65-generic #74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-3.3 / 75f34a1

Default Java
Private Build-1.8.0_275-8u275-b01-0ubuntu1~16.04-b01

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2618/1/testReport/

Max. process+thread count
538 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-azure U: hadoop-tools/hadoop-azure

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2618/1/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

TEST RESULTS
HNS Account Location: East US 2
NonHNS Account Location: East US 2, Central US
HNS OAuth

[INFO] Tests run: 89, Failures: 0, Errors: 0, Skipped: 0
[WARNING] Tests run: 461, Failures: 0, Errors: 0, Skipped: 68
[WARNING] Tests run: 208, Failures: 0, Errors: 0, Skipped: 24

HNS SharedKey

[INFO] Tests run: 89, Failures: 0, Errors: 0, Skipped: 0
[WARNING] Tests run: 461, Failures: 0, Errors: 0, Skipped: 24
[WARNING] Tests run: 208, Failures: 0, Errors: 0, Skipped: 16

Non-HNS SharedKey
[INFO] Tests run: 89, Failures: 0, Errors: 0, Skipped: 0
[WARNING] Tests run: 461, Failures: 0, Errors: 0, Skipped: 249
[WARNING] Tests run: 208, Failures: 0, Errors: 0, Skipped: 16

TEST RESULTS
HNS Account Location: East US 2
NonHNS Account Location: East US 2, Central US
HNS OAuth

[INFO] Tests run: 89, Failures: 0, Errors: 0, Skipped: 0
[WARNING] Tests run: 461, Failures: 0, Errors: 0, Skipped: 68
[WARNING] Tests run: 208, Failures: 0, Errors: 0, Skipped: 24

HNS SharedKey

[INFO] Tests run: 89, Failures: 0, Errors: 0, Skipped: 0
[WARNING] Tests run: 461, Failures: 0, Errors: 0, Skipped: 24
[WARNING] Tests run: 208, Failures: 0, Errors: 0, Skipped: 16

Non-HNS SharedKey
[INFO] Tests run: 89, Failures: 0, Errors: 0, Skipped: 0
[WARNING] Tests run: 461, Failures: 0, Errors: 0, Skipped: 249
[WARNING] Tests run: 208, Failures: 0, Errors: 0, Skipped: 16

thanks.Checked out the branch locally and did the merge of individual changes; github only lets us squash a commit

thanks.Checked out the branch locally and did the merge of individual changes; github only lets us squash a commit


Make components hashable

Component instances are not immutable, so they cannot be "hashed".
Parameters can be changed before their geometry is created.
For example:
>>> from cqparts_misc.basic.primatives import Cube
>>> cube = Cube(size=10)
>>> cube.size = 20
>>> cube.local_obj  # returns a 20x20x20 cube

This provides a freedom, but can also be confusing...
>>> from cqparts_misc.basic.primatives import Cube
>>> cube = Cube(size=10)
>>> cube.local_obj  # returns a 10x10x10 cube
>>> cube.size = 20  # does NOT reset cube.local_obj

@fragmuffin What is the use case for unlocked parameters? Is it important to be able to update parameters later on in the same script?

I've found it useful when multiple parts are unioned to create a single part.
Specifically MaleFastenerPart : line 177, but that could be changed, and it's probably the only time I've used it.

Is it important to be able to update parameters later on in the same script?

So... I'm thinking "no"; perhaps it's not.
However, that doesn't change a component's behaviour with regard to changing local_obj.
In the case of changing its local_obj geometry, it would need to start unlocked, then be locked in order to make it hashable.
In which case... we may as well have the parameters changeable, but they do need to be more strict.


Chapter 4 - transport layer

Merged master into this branch to fix the CI.

All existing comments have been addressed it seems (except for the non-normative section formatting), so I am merging this now. If you have additional comments, please open tickets or send PR.


#11374: Removed the announcement knowledge panel customisability

Announcements widget will only show on landing page if  announcements are present

Describe your changes:
Fixes

I worked on ... because ...

Type of change:

[ ] Bug fix
[ ] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My PR title is Fixes <issue-number>: <short explanation>
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

@aniketkatkar97 lets finish up this and merge it in

@aniketkatkar97 is it possible to remove the border around announcements


guestOperationsManager managed object

Hi,
Reading the vSphere API 5.0 doc, the guestOperationsManager should be accessed by the ServiceContent class.
However, when I try to invoke it , it returns me "nil"
om = vim.serviceContent.guestOperationsManager
=> nil
(I'm already doing tasks like poweron/snapshot/etc and everything works fine)
I'm trying to figure out how to execute a program inside a windows VM, and reading the APIs it seems that guestOperationsManager should do what I'm looking for....
...any clues?
thank you in advance.
m.

What's the value of vim.rev? Can you paste the code you're using?

The value of vim.rev is 4.1 , does it mean that i'm not using the 5.0 APIs? I've just installed the rbvmomi gem, did I miss something?
the cose is this one:
require 'rubygems'
require 'rbvmomi'
vim = RbVmomi::VIM.connect(:host => '<IP_ADDRESS>', :user => 'root', :password=> 'password', :insecure => true)
dc = vim.serviceInstance.find_datacenter
vm = dc.find_vm("WinXPSP3")
thank you.

Ops, I just figured out that I was trying to use the gem with an ESXi 4.1 server :
Now I just updated it to 5.1, and when I call  vim.serviceContent.guestOperationsManager it correctly returns me that object.
I'm going to try it, if I get success, i'm looking forward to posting the full example here.
thank you!

Ok here is the new problem:
When I try to authenticate to the guest system, I have this error:
om = vim.serviceContent.guestOperationsManager
am = om.authManager
guestauth = {:interactiveSession => false, :username => 'admin', :password => 'password'}
am.ValidateCredentialsInGuest(:vm => vm, :auth => guestauth)
RuntimeError: unexpected property name username
Now, standing to the API,  guestauth should be considered as a NamePasswordAuthentication Data Object, which is an extension of GuestAuthentication one.  However, when I'm calling am.ValidateCredentialsInGuest it is expecting only the :interactiveSession attribute inside the auth value.
What I mean, is that it seems to be not able to read the extended attributes of NamePasswordAuthentication...
any clues?
thx you

Excerpts from m4rco-'s message of 2011-10-14 07:53:50 -0400:

Ok here is the new problem:
When I try to authenticate to the guest system, I have this error:
om = vim.serviceContent.guestOperationsManager
am = om.authManager
guestauth = {:interactiveSession => false, :username => 'admin', :password => 'password'}
am.ValidateCredentialsInGuest(:vm => vm, :auth => guestauth)
RuntimeError: unexpected property name username
Now, standing to the API,  guestauth should be considered as a NamePasswordAuthentication Data Object, which is an extension of GuestAuthentication one.  However, when I'm calling am.ValidateCredentialsInGuest it is expecting only the :interactiveSession attribute inside the auth value.
What I mean, is that it seems to be not able to read the extended attributes of NamePasswordAuthentication...

In the case of subtyping you need to explicitly create the data object.
Just do RbVmomi::VIM::NamePasswordAuthentication(:interactiveSession => false, ...).
I usually "include RbVmomi" to make that shorter.

Thank you rlane, it works!
soon i'll post here my code,
bye!
m.


fix: made the text mobile responsive.

🛠 Fixes Issue
Closes #156
👨💻 Changes proposed

added the ".output" div to the media queries for mobile phones
gave it a padding and reduced the fontSize by 1 rem

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers
I could change the order of the media queries so "important" won't have to be used, but if you're okay with this present solution then no problem.
📷 Screenshots

I would prefer you use the calc function to calculate based on the width of the container or screen

I would prefer you use the calc function to calculate based on the width of the container or screen

Alright, i will do that


Android : App does not launch after screen unlock

Using the latest version 2.0.3 I was able to have everything working like a charm on iOS.
In Android everything works except when the call is received while the screen is locked. The app doesn`t launch to foreground after the screen is unlocked. Have anyone faced this before? I appreciate any help on this!

Hello @alxferraz, I recently had this same issue on IOS.
My issue was when answering the call on an unlocked phone, it took me to the app screen as expected. But when the phone is locked and I answer the call, the app wakes, and the call starts in the background.
After troubleshooting and trying different applications' behavior, I found that the issue was with the call type.
If you set the call type as video, and answer the call in any state the call will start in the app in the foreground.


Blackduck importer fixes

Fixes

Parsing ZIP file now finds the correct files. (Solves always 0 findings)
Locations will cap the size of the various file paths at 4000 characters (so there is no sql error)
CVE's will be pulled out of the vulnerability id field (they will sometimes contain BDSA identifiers as well as CVE's)
There is no "Product id" column in the csv's, that is replaced with "Component id"

Let me know if you have some different version of blackduck that has a Product id in the csv instead of Component id, I could make it backwards compatible.
This PR refers to issue #1680.
Have also updated the sample scan zip at DefectDojo/sample-scan-files and created a PR there.
My first contribution
@madchap, thanks for the help. Please take a look. :)

I would like to upgrade this parser farther and overhaul it. Blackduck is less about the vulnerabilities in the included components for your projects, and more about those packages being at vulnerable versions in the first place.
For our team's use, I will be updating the parser and importer to support this way of presenting findings. An out-of-date and vulnerable component will be ONE finding, and it's various CVE's will be included in the description, the mitigation will simply be to update the vulnerable component to a secure version.
Another use case of Blackduck (the one that we care more about, honestly) is the License Risk, which isn't being considered with the current parser/importer logic. I will be updating it to create findings for License Risks as well.
Would this be of value to any Blackduck users? If not I can just keep those future changes locally.

Thanks @Apipia for working on this first contribution!
A few comments:

I would indeed retain the backwards compatibility. We do not want to break anything for other users in general and be able to address the use-cases of many.
Regarding your 2nd comment, you don't necessarily have to modify this very parser either. I feel it would then be overly complicated to maintain backward compatibility while trying to adapt to newer ways of presenting findings. You could also add a brand new Blackduck parser (e.g. different factory/class name) to handle the report in a novel way and eventually have the two options offered in parallel. Please use another PR for that.
For the license risks, it will be interesting to see what changes you propose ;-) It is probably better for you to work on this in a separate PR to keep things simple.

If you would like people to help test your PR at some point, please upload some test files to help. You can of course anonymize them as needed. It would be good to have them in the other repo at https://github.com/DefectDojo/sample-scan-files.

I have made the requested backwards compatibility changes.
Also I have a PR on the sample upload files here:
https://github.com/DefectDojo/sample-scan-files/pull/40
The data is completely random and will create 11 findings:
https://github.com/DefectDojo/sample-scan-files/pull/40

I am guessing it should yes, even though the way we test today is a bit naive.
That being said, @Apipia, it would be great if you could add a test with regards to this new file format so that it gets automatically tested for both these cases.

Thank you @Apipia for the contribution!


set environment

The action should also provide an option to set the target environment in the deployment event.

live in 1.3.1


Refactored EmbeddedServer creation Fixes #461

Hi and thanks for contributing!
I'm leaning towards not merging this PR.
The small gain from this change is that EmbeddedJettyFactory is created on demand. It's a class with no members and really doesn't do anything until create() is invoked i.e. an instance with very small footprint. The loss is that the code is no longer straight-forward. Now, there's a mix of switch and specific Map functionality to solve something that is very trivial. For me it's not worth it for saving the creation of one instance.
Your thoughts?

The reason for this change is described in #461. It is not the performance gain, it is the fact that I do not want to have to incude any jetty related jar since I am deploying my war on Tomcat.
IMO this change does not make the code any more complex. I suppose it is about the way each one has code in his mind.

I missed the #461 part of this. My bad. Let me have another look.
I would definitely say the code is more complex. Having just a map is simpler than having a map and a switch combined. However, if the gain is big enough than that can be worth it. I'll check if it can be done in another way.

I refactored tha code to remove the switch statement, since this is your concern.
The only thing that is changed now is that instead of all embedded servers being created in the static constructor, they are added once in the create method.
However, if you have 5 embedded servers, with your approach you add them anyway without actually knowing if anyone will use them. Most likely someone will use only one.

I've created an alternative solution. Haven't had time to try it out since I don't have the infrastructure setup on the computer I'm on now. Please have a look #466

We did almost the same thing, moved the initialization to a method, instead of tha static constuctor.
Either way is fine.

Yup!


Crash in 1.18.5

Toggling .lightlvl on crashes the game in version 1.18.5. Here is the crash log in my case:

Game Version: v1.18.5 (Stable)
6/6/2023 9:36:34 PM: Critical error occurred
Loaded Mods<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>System.MissingFieldException: Field not found: 'Vintagestory.API.Common.Entities.Entity.Pos'.
at easylightlevels.ELLCoreSystem.Run()
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
at System.Threading.ThreadHelper.ThreadStart()

The issue is probably in line 148 of Core.cs. Try changing the reference to player.EntityPos.AsBlockPos as referenced here.

This seems to have been fixed by a recompile. The offending field has been unchanged, so perhaps it was a problem with differing .NET versions (recompile updated it 4.6.1 to 4.8).


Bug: 0.18.2: Cannot find module './database'

The 0.18.2 release throws an error when trying to execute. On Windows as well as Linux.
Error: Cannot find module './database'
Require stack:
- /snapshot/sdk-for-cli/lib/commands/init.js
- /snapshot/sdk-for-cli/index.js
1) If you want to compile the package/file into executable, please pay attention to compilation warnings and specify a literal in 'require' call. 2) If you don't want to compile the package/file into executable and want to 'require' it from filesystem (likely plugin), specify an absolute path in 'require' call using process.cwd() or process.execPath.
    at Function.Module._resolveFilename (node:internal/modules/cjs/loader:933:15)
    at Function._resolveFilename (pkg/prelude/bootstrap.js:1819:46)
    at Function.Module._load (node:internal/modules/cjs/loader:778:27)
    at Module.require (node:internal/modules/cjs/loader:1005:19)
    at Module.require (pkg/prelude/bootstrap.js:1719:31)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Object.<anonymous> (/snapshot/sdk-for-cli/lib/commands/init.js:9:37)
    at Module._compile (pkg/prelude/bootstrap.js:1794:22)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1153:10)
    at Module.load (node:internal/modules/cjs/loader:981:32) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '/snapshot/sdk-for-cli/lib/commands/init.js',
    '/snapshot/sdk-for-cli/index.js'
  ],
  pkg: true
}

Downgrading to 0.18.1 works fine.

The 0.18.2 release throws an error when trying to execute. On Windows as well as Linux.
Error: Cannot find module './database'
Require stack:
- /snapshot/sdk-for-cli/lib/commands/init.js
- /snapshot/sdk-for-cli/index.js
1) If you want to compile the package/file into executable, please pay attention to compilation warnings and specify a literal in 'require' call. 2) If you don't want to compile the package/file into executable and want to 'require' it from filesystem (likely plugin), specify an absolute path in 'require' call using process.cwd() or process.execPath.
    at Function.Module._resolveFilename (node:internal/modules/cjs/loader:933:15)
    at Function._resolveFilename (pkg/prelude/bootstrap.js:1819:46)
    at Function.Module._load (node:internal/modules/cjs/loader:778:27)
    at Module.require (node:internal/modules/cjs/loader:1005:19)
    at Module.require (pkg/prelude/bootstrap.js:1719:31)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Object.<anonymous> (/snapshot/sdk-for-cli/lib/commands/init.js:9:37)
    at Module._compile (pkg/prelude/bootstrap.js:1794:22)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1153:10)
    at Module.load (node:internal/modules/cjs/loader:981:32) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '/snapshot/sdk-for-cli/lib/commands/init.js',
    '/snapshot/sdk-for-cli/index.js'
  ],
  pkg: true
}

Downgrading to 0.18.1 works fine.

Can confirm, Just posted in discord 10 mins ago.


ConduitDataCollection doesn't include time and cycle number

The ConduitDataCollection doesn't add the time and cycle number to the root file. They were both missing when I loaded the resulting root file into VisIt. I was able to fix this by adding the following lines to the SaveRootFile function inside the data collection:
  n_root["blueprint_index/mesh/state/time"] = time;
  n_root["blueprint_index/mesh/state/time_step"] = time_step;
  n_root["blueprint_index/mesh/state/cycle"] = cycle;

Thanks,
Patrick

Thanks @greene30, do you want to propose a simple PR to fix this?

let me check out something -- I think this might be fixed in conduit now (https://github.com/LLNL/conduit/pull/613), but I want to look at the MFEM useage.

I can setup the PR once @cyrush is done checking conduit (assuming it's still needed).

yes, since we are using blueprint::mesh::generate_index, the fix in (https://github.com/LLNL/conduit/pull/613) should help here. @greene30 -- If possible - can you let me know the git sha of conduit that you are using? conduit::about() should provide a quick way to get it.

also -- it won't hurt to make the change in mfem as well (it would just override the data) if you need to use an older version of conduit.

Here's the sha and version from conduit::about():
"version": "0.4.0",
"git_sha1": "63fb8f9889c468e2af630be7d2a3d2abb8947004"
This is the version ALE3D is currently using, but I noticed some one is in the process of updating to 0.5.1.

thanks @greene30  -- pre 0.5.1 is already old -- It seems like making the change to add the details to the root file in mfem would help, b/c you may not be able to use the newer conduit in the near term.  Again, it won't hurt anything when used with newer conduit.

I'm looking through your CONTRIBUTING.md document right now. I'll try to get the PR setup some time this week.

Thanks @greene30, let me know if you have any questions

Fixed by #1864


Communicate loading status more clearly

The initial loading phase - before anything is shown - can take quite long. Would be good to communicate this status more clearly to the user, e.g. by prominently placing some text "Querying endpoint..." or alike in the middle of the screen (i.e. visualization pane), and keeping it there until the first class (bubble) appears. Otherwise, the usr might get the impression that the tool failed to work (even if not).

If a user directly clicks on "Graph" in the menu (i.e. before entering any endpoint URL), no graph is shown. This is fine, but a short message in the middle of the visualization pane informing the user might also be useful in this case.

Much better now. Minor issue still remains, as the generic error message appears to be wrong in certain cases. It says "No SPARQL endpoint selected!" even in cases when a correct SPARQL endpoint was selected (but is not acceesible for some reason) or in cases when an endpoint URL was entered wrongly (with a type). Easy fix: change the text. Better fix: distinguish different types of errors.


Add support for zola

Closes #70

Thanks! Could you add zola to this line as well?
https://github.com/taiki-e/install-action/blob/6e270f6359e00e6e4b83c8edc6d769c1947b7f5b/.github/workflows/ci.yml#L102-L104

It fails the test on some of those containers if I add it there: https://github.com/taiki-e/install-action/actions/runs/4148390689

Oh, in that case, it is ok as is. (I will need to split the matrix into a glibc 2.31+ group and a 2.27+ group later.)

Published in 2.4.0.


Updated bangla language settings

I have updated cloudflare.json and abetterbrowser.js to reflect Bengali language texts. I am a native speaker of Bangla language and consider the texts suitable for official usage.

Thanks!

You welcome :-) This was my ever first commit to a another open source project.


Use encodeURIComponent for identity cookie encoding

The AboutAds opt out integration is pixel-based and will require that a server component write the identity cookie. I have found that cookie interop between java servlets and javascript works far better if the cookie is url encoded.
java:
URLEncoder.encode(Base64.getEncoder().encodeToString(raw), "UTF-8")

javascript
encoding: encodeURIComponent(btoa(JSON.stringify(value)))
decoding: JSON.parse(atob(decodeURIComponent(value)))

Released to production


renaming message.rs to api_client.rs so things look cleaner.

This was already implemented.  Updating naming of messages to api_client so it makes a little more sense how things are organized.


Illegal memory references on null tests when running cargo test --release

The following tests fail with invalid memory references (signal: 11, SIGSEGV: invalid memory reference):

tests::thin_array_ptr::check_null
tests::fat_array_ptr::check_null
tests::array_ref::null_test
tests::array_ref::ref_counting_test

These all invoke some variant of the is_null method as defined in the BaseArrayRef trait; ultimately, they all call this method defined in heaparray::mem_block:
impl<'a,E,L> MemBlock<'a,E,L> {

    // more code above...
    pub fn is_null(&self) -> bool {
        self as *const Self as usize == Self::NULL
    }
}

My guess is that on release builds, Rust assumes that references to objects cannot be null (or in this case, usize::MAX), and so it optimizes away my null check. Unfortunately, that assumption doesn't hold, so the rust code just blows past all the safety checks and seg-faults!

Fixed using criterion::black_box


Add support for the new cargo lockfile format

Recently cargo nightly started produces lockfiles with the new format (https://github.com/rust-lang/cargo/pull/7579).
Tarpaulin fails to compile tests, if it encounters such a log file. One instance can be found here. Tarpaulin fails with this message:
Error: "Failed to compile tests! Error: failed to parse lock file at: /home/travis/build/jonasbb/serde_with/Cargo.lock"

In my case I run Tarpaulin on a library and therefore do not commit the lockfile, which is why nightly now uses the new lockfile format.

There's now a new cargo_update branch this updates the tarpaulin cargo dependency to 0.40.0 and may solve the issue. I'll try it myself when I've got time to but in the meantime if you test it and report back I can fast track the merge/release :+1:

Same problem too

@xd009642 I started a try build with tarpaulin from the new branch. It passed on all Rust versions.
https://travis-ci.org/jonasbb/serde_with/builds/621365163
So this would fix the issue for me.

Awesome, well since it doesn't affect anything in terms of existing build functionality I've just pushed out another release. If you could try it out and confirm it now works this issue can be closed :smile:

It's now released in 0.10.0 so closing this


Incorrect display of title link after title update by F2

Does this issue occur when all extensions are disabled?: Yes/No
Yes

VS Code Version: 1.88.1
OS Version: Windows 10 Home 22H2 19045.4291

Steps to Reproduce:

Add the following code to the md-file:

- [Options](#options)

# Options

[Options](#options)

Go to the end of the header and press F2.
In the window that appears, add the Cyrillic characters, e.g. Настройки:

Press Enter.

Result (encoding: UTF-8):

Should already be fixed in the current insiders build. Fix will also go out in VS code 1.89


Add UserClientRequestId and GetElapsedClientLatency to CosmosDiagnostics

Pull Request Template
Description
This is adding UserClientRequestId and GetElapsedClientLatency to CosmosDiagnostics.

The elapsed client latency is useful for users to determine if they need to log the diagnostics.
UserClientRequestId allows users to correlate Cosmos DB requests with their application request id.
a. This is never sent to the service.
b. UserClientRequestId is blocked on bulk operations because there is no way to merge all different values into a single one.
Fixed bug where request options was not being sent in Database.ReadStreamAsync() and Database.DeleteStreamAsync()

Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Closing issues
closes #1218 #458
public abstract class CosmosDiagnostics
{
        public abstract string UserClientRequestId { get; }

        public abstract TimeSpan GetElapsedClientLatency();

        public abstract override string ToString();
    }
}

#458 update description to the scope of the change.

Batch diagnostics: Diagnostics duplication should solve it no?

Pulling in ClientReqeustId inside might actually confuse CX that its actually sent uses E2E and might be source of future CRI's.
If its for application tracing/correlation, then application can trace it explicitly (yes its more work but straightforward).


bug(badge): Wrong text color

Describe the bug
For all Badge variants, text is always 'contentPrimary' (black) but content colors are different depending on the variant.
It is a problem for Accent (especially in dark mode), Brand & Alert variant for accessibility reason
Here what we actually have
Dark mode:

Light mode:

Expected behavior
Here the Figma : https://www.figma.com/file/WkGTfR63Nf2AGHSoRvEhWC/Vitamin---Android-UI-kit-(v0)?node-id=4401%3A8975&t=d1AOnr3jeUSIb1Cu-0
For Badge variants, text color must be

Alert & Brand -> `contentPrimaryReversed' (white)
Accent -> 'contentAccent'
Default & Reversed -> 'contentPrimary'

Light

Dark

Closed by https://github.com/Decathlon/vitamin-compose/pull/106


Epic: refactor global context to simplify and improve

[x] separate languages #6
[ ] separate auth context #12
[ ] separate deck context

Per the discussion on #12 we're going to close this Epic, and count it as a success that we hardcoded the Languages and improved the rendering behavior of the global context object. (You might even consider #13 as a part of this epic, although it wasn't initially a goal.)


Unexpected 0x41 markers added during merge

Hi!
While working on https://github.com/solokeys/solo/pull/238 I have noticed a 0x41 marker on the APPLICATION_END_PAGE boundary, introduced during the all.hex merge. Is it needed anywhere, or is it just for debug purposes (e.g. to hand-check firmware dump)?
https://github.com/solokeys/solo-python/blob/e2a81a2c39bef61f98dca29154c5fdedff3a1044/solo/operations.py#L68-L69

I believe those were there originally for debugging purposes yes.  I can't find or remember any other reason.

OK! Let them be. Perhaps it would be worth making a comment to the setting lines in solo-python about their purpose. This will make the ticket actionable.

Closing for inactivity.


Directive usage errors need to appear in runtime too

There are cases like https://github.com/aspnet/Razor/issues/2539#issuecomment-416384015 when using directives as variables in cshtml results in a weird csharp error at runtime. Instead we should provide a Razor error like it does during design time.
cc @dougbu @NTaylorMullen

Closing as no customers complained about this.


Altitude of fileinfo

The altitude of fileinfo is 45000 in this page. The altitude of FileInfo in my computer (RS4 Win10 Enterprise, fltmc) is 40500. Which one is correct?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0a2d558e-1073-85d5-f863-5bdbf6e3ddc6
Version Independent ID: 62e4d735-803f-7608-154b-7032dc1ceb3d
Content: Allocated Altitudes
Content Source: windows-driver-docs-pr/ifs/allocated-altitudes.md
Service: unspecified
Product: windows-hardware
GitHub Login: @windows-driver-content
Microsoft Alias: windowsdriverdev

Thanks, @lawlxy
The writer for this area is on vacation through the middle of next week. If you need the info immediately, I suggest posting in the Windows Hardware WDK and Driver Development forum

Hi @lawlxy, sorry for the delay. I checked and 40500 is the correct altitude; this seems to be a typo. I'll update the documentation. Thanks!


connection refused - on-connect doesn't wait for server

Debian Squeeze - node-forever and redis
I have an /etc/init.d script which starts the app on startup.
Per documentation, on-connect or on-ready wait for redis server.  My node app auto-starts but won't connect.  I've added my own delays, but still no go.
================app.js=================
var http = require('http');
var https = require('https');
setInterval(simLoop, 3000);
var client, redis;
var waitToStart = 5;
function simLoop() {
waitToStart--;
if (waitToStart > 0) {
console.log('waiting to start ' + waitToStart);
return;
}
try {
if (redis == null)
redis = require("redis");
if (client == null) {
client = redis.createClient();
client.on("error", function(err) {
console.error("Error connecting to redis ", err);
});
}
getData();
} catch (e) {
return;
}
}
=========================app log file======================
tail simErr.log
Error connecting to redis  [Error: Redis connection to <IP_ADDRESS>:6379 failed - connect ECONNREFUSED]
Error connecting to redis  [Error: Redis connection to <IP_ADDRESS>:6379 failed - connect ECONNREFUSED]
Error connecting to redis  [Error: Redis connection to <IP_ADDRESS>:6379 failed - connect ECONNREFUSED]
Error connecting to redis  [Error: Redis connection to <IP_ADDRESS>:6379 failed - connect ECONNREFUSED]

The client is going to retry to connect to redis until either a hard timeout is reached or redis is back online. The errors just tell you that redis is not reachable yet.


Missing slash in sitemap URL scheme when upgrading from sphinx-sitemap 2.2 to 2.3

Hi!
I was on Sphinx<5 and sphinx-sitemap==2.2 and updated my project dependencies to Sphinx>5 and sphinx-sitemap>2.2. When I did so, I noticed that the URLs generated in sitemap.xml where missing a backslash before {lang}, which made the URLs incorrect. See the picture below for the diff:

I can solve this by setting sitemap_url_scheme = "/{lang}{version}{link}", but that doesn't look like a good solution to me. Do you know if there is any other source of error here, possibly within sphinx-sitemap? I've seen https://sphinx-sitemap.readthedocs.io/en/latest/advanced-configuration.html, but this error appears when just changing between sphinx-sitemap 2.2 to 2.3 (and keeping Sphinx>5 constant), so it looks like it was introduced by upgrading sphinx-sitemap.

It looks like you have Sphinx<5 on the left since you have {html_baseurl}/404.html on the left and {html_baseurl}en/404.html on the right. I don't see any code changes between 2.2 and 2.5 that look like they would change the behavior.
The {html_basurl} gets a forward slash automatically added here, so I'm guessing something might be happening with using the plus operator? (I need to convert to using format)
I don't have an immediate answer as to what is going, I'll have to test some more.

Thanks! I'll keep sitemap_url_scheme = "/{lang}{version}{link} for now, although I suspect this might break down the line.


“Select your county” button has an insufficient accessible name and lacks a role of button [iOS]

The “select your county” button found on the “NY COVID Data” screen is labeled by whichever county is currently selected (e.g., “Albany”). Without the visual information provided by the “filter” icon and the surrounding content, VoiceOver users are unlikely to understand the purpose of the button without activating it and exploring the next screen. The button also does not convey a role of button.

The accessibilityLabel attribute can be used to provide additional information to VoiceOver users. As the accessibilityLabel overwrites the visible text label for said users, the accessibilityLabel value must include the visible text. Use the accessibilityRole attribute on the interactive element (TouchableOpacity, Pressable, etc.) to convey a button role to VoiceOver users.
<TouchableOpacity
  onPress={…}
  accessibilityRole='button'
  accessibilityLabel='[Insert additional text here for VoiceOver users] All Counties in New York'
>
  <Text>All Counties in New York</Text>
</TouchableOpacity>

As per @willslone this needs to be addressed before launch. Upgrading severity to Critical accordingly.

@willslone Should be addressed in build 34 and after.


fix two bugs

emitting a notfound event when navigate to a unexists state when using a statename.
check whether the link target  is the same domain and avoid navigating.

Thanks! I will add some testcase later for this pr.


CHAOS REPORT:The sign-up and sign-in

Is there an actual issue here?

[X] I have verified this isn't part of ChaosWeb's intentional disorder

Describe the unexpected!
The signin and sign-up page are not opening when we click to them.
Confirm

[X] I understand ChaosWeb is intentionally designed to be messy
[X] I agree to follow this project's Code of Conduct
[X] I found a genuine bug amidst the chaos
[X] I'm enjoying the mess!
[X] I'm GSSOC'24 Extd registered Contributor
[X] I'm Hacktoberfest registered Contributor

@vansh-codes  brother , please assign this task to me. I have already made many sign up and login pages for different projects and quite experienced in it. I assure the ui of the pages will must be chaotic and  give anti-user experience. I have mentioned you about this issue earlier days, but maybe there exits same issue , so mine was closed that time.

I want the same issue
On Sun, 13 Oct, 2024, 7:37 pm sayanp607, @.***> wrote:

@vansh-codes https://github.com/vansh-codes brother , please assign
this task to me. I have already made many sign up and login pages for
different projects and quite experienced in it. I assure the ui of the
pages will must be chaotic and give anti-user experience. I have mentioned
you about this issue earlier days, but maybe there exits same issue , so
mine was closed that time.
—
Reply to this email directly, view it on GitHub
https://github.com/vansh-codes/ChaosWeb/issues/97#issuecomment-2408994179,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BKOHTVQDMSUVJJU3BNKSSYTZ3J5D7AVCNFSM6AAAAABP3EACJGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBYHE4TIMJXHE
.
You are receiving this because you authored the thread.Message ID:
@.***>