Send email to open pass holder with new event info

Notify them of data.world event being added to open pass

Happy Friday NC Open Pass holders,
Just a quick note to let you know that we've added two additional events to your pass! How about that to end the week on?
The first event is happening next week on September 7, 2017. It's a data.world demonstration and the event is called "Building a Social Network for Data Sharing." data.world has built a platform where the world's problem solvers can find and use a vast array of high-quality open data. Carolee Mitchell, Manager of User Operations, will demonstrate data.world's suite of data services.
If you would like to attend, please register on their form.
The second event is a customer value workshop designed to coach any teams that enter the open data competition. The workshop will be hosted by the NC State Entrepreneurship Clinic at HQ Raleigh on October 10, 2017. So mark your calender for that and we'll follow-up with more details soon.
Have a fabulous weekend and thank you for supporting the NC Open Pass event series. The NC Open Pass website has been updated if you want to share the news.
Jason Hibbets & our awesome planning team
cc @natalienoel1


PD uses cost estimation based on size instead of a hard coded value

/assign @bufferflies

Should be tracked in PD, close it now.


[SPARK-9912] [MLLIB] QRDecomposition should use QType and RType for type names instead of UType and VType

@hhbyyh

Merged build triggered.

Merged build started.

Test build #40672 has started for   PR 8140 at commit 454b27e.

Test build #40672 has finished for   PR 8140 at commit 454b27e.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class QRDecomposition[QType, RType](Q: QType, R: RType)

Merged build finished. Test FAILed.

The failed test is unrelated. This is a trivial change. So merged into master and branch-1.5.


Wrong link in README.md

Update documentation link in README.md

Gem version is 3.5.0 but docs link is 3.4.0
Thanks to the community on the Discord, I discovered why certain classes are missing in the documentation (wrong link)
Should I make a PR for this?

If you can take the time to make a PR, that'd be much appreciated. Also if you can search for anywhere else that this may have been missed.
On this note, we should create a release document that details the process


Feat req: Add ability for full width vertical menu to show when hovering over collapsed state

[x] I'm sure this does not appear in the issue list of the repository

Basic Info
What problem does this feature solve?
This feature would allow the full menu (and not just the bare bones cascading menu items) to appear when the user hovers over the collapsed state of the menu. https://arco.design/vue/en-US/component/menu#collapse-control
👀 Look at the behavior of this vertical menu on the Cloudflare dashboard  as an example.
What are the similar cases of this feature

Get, this will be added in subsequent version


Exercise Tutorial

Exercise Tutorial:
The current URL is: http://<IP_ADDRESS>:1111/tutorials/exercises/basic
Added "Tutorials" to the left navbar

An example of a tutorial/exercise page

Important to consider!
I don't really like the tutorials/exercises in the documentation section BUT for now, it's not possible to add it somewhere else easily. We'll work on another design for the website, in the meanwhile, we could simply leave this new thing/tutorial in the navbar. I don't think it will be a problem, also it will be temporal.

I merged it. Thank you.


P16.0 code drop

Build succeeded (check pipeline).

https://checklist.intgdc.com/job/compliance/job/gooddata-public-demos-oss-check/108/ : SUCCESS in 4s


Switch BUILD_AS_NON_ROOT as default to on.

It's been long enough, there should be no ports that don't build as root, and I'm tired of going over them every month and fixing them...

Yup. Need to add some logic about ccache since they don't work together. I'll handle that.
Regards,
Bryan Drewery

On Apr 11, 2016, at 05:43, Mathieu Arnold<EMAIL_ADDRESS>wrote:
It's been long enough, there should be no ports that don't build as root, and I'm tired of going over them every month and fixing them...
You can view, comment on, or merge this pull request online at:
https://github.com/freebsd/poudriere/pull/383
Commit Summary
Switch BUILD_AS_NON_ROOT as default to on.
File Changes
M src/etc/poudriere.conf.sample (4)
M src/share/poudriere/common.sh (2)
Patch Links:
https://github.com/freebsd/poudriere/pull/383.patch
https://github.com/freebsd/poudriere/pull/383.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub

Mmm, I've been using both for a long time, and do work just right together.

Mmm, I've been using both for a long time, and do work just right together.

Same here.

To be fair, there is one port that does not build as a user, it is devel/spark, I have not fixed it because I don't know 1) java 2) scala 3) maven, and don't want to :)
But I'm sure the fix is easy for someone who knows it. (And I really don't feel it is a blocker)

There's no way it can work by default since the CCACHE_DIR is going to be default root owned and building as user is not root, it is nobody. Nobody cannot write into the CCACHE_DIR. You must chmod a+w in the CCACHE_DIR at least.

The chmod also creates a security problem.

Using nobody is also not the best solution from security standpoint. Better to use dedicated user.

A dedicated user would mean that a user need to be added in the jail, which is not really a good idea, nobody is perfect for that.

A dedicated user would mean that a user need to be added in the jail, which is not really a good idea

I don't see any problems with that. We already add resolv.conf, make.conf and other stuff on jail startup, adding a user is just one more command.

nobody is perfect for that

nobody is actually not good for anything and should never be used. The fundamental problem with nobody is that more than a single service may run under it. The more services are run as nobody, the more privileged it becomes, ending up as another superuser.
What that basically means is that when poudriere ccache cache is owned by nobody, and any other service running as nobody is compromised, cache would get poisoned, malicious code would end up in the packages and all machines which fetch from that repository would be rooted.

We do already add the PORTBUILD_USER in the jail if not present. That's not a problem. It is added as uid 65532.
What is a problem is POLA and breaking Poudriere for users.  Sure you could setup ccache in the right configuration to build most posts with non-root, but not all.  Most Poudriere users don't know how or care to debug the obscure failures that come from broken ccache+non-root.  So it must just work by default.  Sharing a ccache dir can't work between Poudriere and root, which because of how Poudriere uses it, CCACHE_DIR vs CCACHE=yes, is going to be pretty common since the user is providing a directory that is likely being shared elsewhere.
What's still lacking is any documentation anywhere explaining how to do BUILD_AS_NON_ROOT and ccache together.  Until that time I will keep the default off BUILD_AS_NON_ROOT when using ccache.  I will add an override though.

I'm going to make the default no with CCACHE_DIR and give a warning when both are enabled.

User does need to do anything or care of anything, poudriere can and should do everything itself - create private directory for its ccache and properly chown it. Another directory for root's ccache, if you still want to support it. I don't understand what's the problem with this.

I'm repeating myself now.  Poudriere does not own or create the ccache dir, it is provided by the user.  Because the user provides it then Poudriere cannot make assumptions about what it may do to it.  It was never documented like that and it would be a POLA violation to go messing with it.

I'm repeating myself now. Poudriere does not own or create the ccache dir, it is provided by the user

I'm repeating myself as well, poudriere must create and own it's own ccache dir, and use it for buids. User is free to override it with his own directory.

Feel free to write your own tool.  FYI I've added you to my ignore list.


Make reward cards colourful again

The Chore
There are some grayed cards for an asset in a settled market.

Deployed to mainnet


Improve Documentation?

Every once in a while, we get a question like

How do I move the window?
How can I make the window frameless?
How do I style the window?
How can I hide the caption/title bar?
How do I use the startResize() method?
etc.

I think the documentation needs to be improved.
Please do suggest.

I made a website https://leanflutter.org , I will add more documentation and examples to this website
https://github.com/leanflutter/website


Enhancement: Weapon modding QOL

Since i seem to be a blind af dude, i tend to search the correct slot on the weapon. Some visual assist would be nice. Like if i drag'n'drop a pistolgrip onto the empty pistolgrip slot, it sometimes takes a while for me to find that slot, especially, when some other mods were being added, that add even more slots, so the icon/box/slot for the pistolgrip moved (even to the next line).
The idea is, that the compatible Slots would light up (just as compatible rigs/backpacks, if there is enough space). Also the slot, that is already taken, but is able to be swapped with the new e.g. pistolgrip would be nice.

Additionally INSANE would be, a visual on the preview-3d-model, for the modslot you are hovering above, where that slot belongs to. Usually no problem, since there is only one pistolgrip-, barrel-, etc. slot, but when there are multiple (usually tacitcal) slots, you don't know, to what physical place it belongs. (in the stash no problem, but midraid, when it takes time for the animation to dis-/assemble them). For empty slots it could be a bright/lit dot and for taken slots, the item, that is placed on the slot. (placed slots are much easier to distinguish though, obviously) But i guess that is not that simple and fast/responsive.

I am not sure, if i should create another issue for that, since it has a connection to the example above:

To the topic, updating 3d models, according to their modslots - it would be nice, if the weapons in raid (on dead bodies) would update, according to its slots. Ad-lib i am not sure, if the mods would update, which would be not that important to me, but the light/laser. Sometimes you get blinded or distracted from some dead bodies equipment, so i then open their weaponspecs and drop (via hotkey) their light or disable the light via (your) context-menu, but it won't be updated. You can 'force' an update, if you swap the weapon to a new slot on the body (if i recall correctly, directly discarding it, didn't update it - not sure), but it's kinda janky.

Another QOL change, based on the modding in raid (with or without leatherman), would be being able to discard/drop any weaponmod on equipped (on me or dead enemy) weapons. Since you are able to drag'n'drop those mods (e.g. the pistolgrip) it would be nice to contextmenu- or hotkey-discard the taken slot. Which would be helpful for faster looting or modding, midraid, between two weapons. For swapping mods, that are not interchangeable (because of compatibility on both slots) or directly discarding mods - e.g. the mag (which does work) and pistolgrip - to reduce the weapons size, and take it to the backpack.
(btw. with props'/dvize' "Props In-Raid Modding (PIRM)" this is possible, just fyi, if there would be a solution to be found)

Done in next release.
Cool idea, but yeah insane work. A bit outside what I can do.
I can't repro this... dead body with a laser on, if I discard/move the laser it's gone. Might be another mod you have messing with it?
I want to follow the "can't modify in hands" rule, because bypassing that causes weird issues (normally you would never be able to equip an incomplete weapon and the game never expects it). But I've changed it to only count your hands, so you'll be able to modify guns on bodies.

Noice!
From what i have seen in your plugin so far, i think you eventually would have been able to 😁👍, but with way too much time and effort, i agree. Especially too much, for this tiny feature upgrade.
In case i described it poorly - i was talking about the weapon model on the map, on the body. Not about the preview model, in the inventory of the dead bot. But if we're on the same page - interesting, gotta check again then - maybe it's a fika thing, since i am usually session client (not host)
Yeah, though so, although the game could even handle the weapon in your hands equipped, without pistolgrip, i totally agree, that this should not work and is too 'dangerous' for any error/jank. This (only on other bodies) is a perfect trade-off!

Thanks, man!
Btw, instead of requesting stuff all the time, as if you owed to implement/change anything - i'd contribute myself, if my skills would be sufficient. E.g. i tried to recompile "Boop's TagEverything" mod (since it could be nice, e.g. for the "inventory organizing features" mod for backpacks and rigs, pulling items in the stash), but i couldn't figure out, why the for 3.9.8 compiled mod, didn't do it, despite booting without any error. If this is an interesting feature, for you, it could be nice, to be integrated in UIFixes. (Boop sadly is offline, since April)


No module named 'theano.gpuarray' while installing

When I want to install theanolm i got this error
theanolm train
Traceback (most recent call last):
  File "/usr/local/bin/theanolm", line 8, in <module>
    from theano.gpuarray.type import ContextNotDefined
ImportError: No module named 'theano.gpuarray'

I have installed theanolm on python3 correctly and I can import it without any problem

This should already be fixed in the develop branch. I'll make a new release soon. If you want, you can try the develop branch (or copy bin/theanolm from there) and let me know if it works.

I made a new release. I didn't test it with libgpuarray uninstalled, so please reopen the issue if the error persists.


AttributeError: 'BF16_Optimizer' object has no attribute 'cur_scale'

The error occurred when I tried to fine-tune in single_modality
Traceback (most recent call last):
File "run_finetuning.py", line 719, in
main(opts, ds_init)
File "run_finetuning.py", line 636, in main
train_stats = train_one_epoch(
File "/mnt2/xiayingxin/InternVideo/InternVideo2/single_modality/engines/engine_for_finetuning.py", line 92, in train_one_epoch
loss_scale_value = get_loss_scale_for_deepspeed(model)
File "/mnt2/xiayingxin/InternVideo/InternVideo2/single_modality/engines/engine_for_finetuning.py", line 22, in get_loss_scale_for_deepspeed
return optimizer.loss_scale if hasattr(optimizer, "loss_scale") else optimizer.cur_scale
AttributeError: 'BF16_Optimizer' object has no attribute 'cur_scale'

Can you provide your DeepSpeed version?

deepspeed==0.10.1


An Admin user can upload documents to the engagement page

An Admin user can upload documents to the engagement page
So that the public can read them
Acceptance Criteria
A public user can clearly open and read what is posted

This will be one of the "widgets". Should we create a user story to add the widgets, then a task for each widget to be added?
Who’s Listening?
Documents
Important links
Open House
Virtual Sessions

Hey team! Please add your planning poker estimate with Zenhub @djnunez-aot @jadmsaadaot @saravanpa-aot @VineetBala-AOT


[SPARK-39701][CORE][K8S][TESTS] Move withSecretFile to SparkFunSuite to reuse

What changes were proposed in this pull request?
This PR aims to move withSecretFile to SparkFunSuite and reuse it in Kubernetes tests.
Why are the changes needed?
Currently, K8s unit tests generate a leftover because it doesn't clean up the temporary secret files. By reusing the existing method, we can avoid this
$ build/sbt -Pkubernetes "kubernetes/test"
$ git status
On branch master
Your branch is up to date with 'apache/master'.

Untracked files:
  (use "git add <file>..." to include in what will be committed)
	resource-managers/kubernetes/core/temp-secret/

Does this PR introduce any user-facing change?
No. This is a test-only change.
How was this patch tested?
Pass the CIs.

Could you review this, @viirya ?

Thank you, @viirya . Merged to master/3.3.


Add control flow complexity limiter

This PR adds an additional control flow complexity limiter. We already have a limit of 2000 recursive invocations of getTypeAtFlowNode to avoid overflowing the call stack. With this PR we add a limit of 500000 invocations of getTypeAtFlowNode in total before we deem a function or module statement block too complex and disable control flow analysis along with reporting an error.
Fixes #29926.

Closing this since it is covered in #31003.


Fix-NPM scripts changed from nest to stencil [05-temporal-package]

Pull Request: Update Default NPM Scripts
Description
This pull request aims to address an issue where NPM scripts lead to an error when executed using the nest command as the default. The proposed changes update the default command in the package.json file from nest to stencil.
Files Changed

stencil/sample/05-temporal-package/package.json

Proposed Changes
The default NPM scripts in the package.json file have been updated from nest to stencil. This change ensures that the scripts execute correctly and aligns with the project's dependencies and tooling.
}

Will do it sir

Added the cmd for stencil-cli setup


docs(cloud_firestore_odm): Fix spelling mistake in documentation.

Description
see title
Related Issues
none.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[ ] All existing and new tests are passing.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] The analyzer (melos run analyze) does not report any problems on my PR.
[ ] I read and followed the Flutter Style Guide.
[ ] I signed the CLA.
[ ] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change.
[X] No, this is not a breaking change.

Closed: https://github.com/firebase/flutterfire/pull/8130

Please merge this PR first to fix broken web link: https://github.com/firebase/flutterfire/pull/8332


CAMEL-17879: remove deprecated component camel-hystrix

It might be better to wait until the whole set of tests run on my own CI before merging, so I think it would be better to wait until tomorrow for merge.

The build completely cleanly and, apparently, without any problems on my own CI tests. We should be good to merge this one.


code:404 msg: service not found: DEFAULT_GROUP@@localhost

Hello, why don't I report this error by adding localhost service in nacos? Is this plan reasonable? Is there any other plan?

Maybe  you can add a service which name is localhost in nacos server. I think this is an issue of Nacos，https://github.com/alibaba/nacos

也许你可以在nacos服务器中添加名为localhost的服务。我认为这是Nacos的问题，https：//github.com/alibaba/nacos
Hello, what do you mean by Nacos error? Did I introduce the version problem or was it his own problem?

yes, you can submit an issue in Nacos.

I also encountered the same problem, The problem was solved when I set ribbon.eureka.enabled=false:
before:ribbon.eureka.enabled=true
after:ribbon.eureka.enabled=false


Feature rate limiter

Feature: Network rate limiter
Description
Network rate limiter.
User is allowed to perform burstNb xhr requests in burstPeriod ms. After exceeding this rate, the limiter enters in cooldown mode : the next requests are scheduled evenly with cooldownNb requests per cooldownPeriod ms

Can you please merge this PR ?
This merge is now urgent.

@durandx No, I can't.
This cause problem as it is with CSRF token, since headers are pre-built at the moment of the fetch call.
I still need to find some time refactor this.
However, this can be integrated into a specific project easily with webpack alias, if the project is not using CSRF token.


pkg/math remove null checks or provide helpful messages to ArgumentError

In general we don't explicitly check for null in library methods.
If we do check args, though, the 'message' param to ArgumentError should be helpful. A null 'message' won't help the user of the function

@Kevmoo please triage this issue.
There is no Pkg-Math label, and the pkg directory does not seem to contain a 'math' directory.
I don't know where the sources for the math package are located.

Not sure what's going on here...


Correct Polish grammar

When it comes to dates, we use different naming form for moths.

Can I get a second opinion on this PR?

@jenssegers
here is the second opinion - from April 2015, year before this issue was reported
Just by an accident, and much to my dread I have found  that this date package was never updated the incorrect translation of month names. In Polish language in dates the name of the month must be in Genitive case!
The fact that even major portals such as blogspot use Nominative to display month names in dfh dates has no power over grammar rules of a language... It is simply wrong.
https://github.com/jenssegers/date/commit/b836c8134032b23a61362e0b9f44fa10a0cdd443
Please update the head of the repository with the proper translations. See the above link. ASAP!


chore: Pin the version of ruff used in the CI to 0.3.0

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 90.86%.  Consider uploading reports for the commit 9930557 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #167   +/-   ##
=======================================
  Coverage   90.86%   90.86%
=======================================
  Files          32       32
  Lines        4279     4279
=======================================
  Hits         3888     3888
  Misses        391      391

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


Is it worth raising the minimum version of Dart to 3?

Yeah only if you want to use some new language feature I guess


Testing in 5.1 type in form field by ID

Is it possible to find a form field by ID? As in the laracasts/intergrated package?
I am getting the following error:
InvalidArgumentException: Unreachable field "newadvisername"

But it is there, as I checked for it with see
$this->visit('/advisers')
			 ->seePageIs('/advisers')
			 ->see('Manage Advisers / Introducers')
			 ->see('id="newadvisername"')
			 ->type('Sample new name', '#newadvisername');

Stack trace
/home/vagrant/Projects/HDC/vendor/symfony/dom-crawler/FormFieldRegistry.php:89
/home/vagrant/Projects/HDC/vendor/symfony/dom-crawler/FormFieldRegistry.php:126
/home/vagrant/Projects/HDC/vendor/symfony/dom-crawler/Form.php:76
/home/vagrant/Projects/HDC/vendor/laravel/framework/src/Illuminate/Foundation/Testing/CrawlerTrait.php:588
/home/vagrant/Projects/HDC/vendor/laravel/framework/src/Illuminate/Foundation/Testing/CrawlerTrait.php:568
/home/vagrant/Projects/HDC/vendor/laravel/framework/src/Illuminate/Foundation/Testing/CrawlerTrait.php:556
/home/vagrant/Projects/HDC/tests/AdvisersTest.php:22

Ask on the forums please.


Transcribe mask ROM uart functions with bedrock2

Notes: https://hackmd.io/@klQsRT6uTf6T8Ht0igQq-g/BkwtWo5Fd

@googlebot I consent.

@jadephilipoom @satnam6502 I have finished transcribing the functions of maskrom uart, and tested them by manually replacing them in opentitan main repo. I will automate this in the next PR. If that's okay, this PR is ready to go.

Great! I'll defer to @jadephilipoom and @samuelgruetter for the code review.

@jadephilipoom @satnam6502 I have finished transcribing the functions of maskrom uart, and tested them by manually replacing them in opentitan main repo. I will automate this in the next PR. If that's okay, this PR is ready to go.

Great! I'll merge in main so the CI can run again. I've looked over the changes since last time and my approval stands, so you're all good to merge once the CI passes.


SystemUiController.setStatusbarColor not changed icons color in status bar

Describe the bug
SystemUiController.setStatusbarColor not changed icons color in status bar if android:windowLightStatusBar is "true" in themes.xml. If  android:windowLightStatusBar is "false" SystemUiController.setStatusbarColor works correct. Is it correct behavior?
To Reproduce
Steps to reproduce the behavior:

Go to themes.xml and add <item name="android:windowLightStatusBar">true</item> to current theme which selected in AndroidManifiest.xml
Try to change icons color in status bar to light:

SideEffect {
        systemUiController.setStatusBarColor(
            color = statusBarColor,
            darkIcons = false
        )
    }

Icons in status bar still dark.

Expected behavior
Icons color should be light
Screenshots?

Environment:

Android OS version: Android 11
Device: OnePlus 9R, OnePlus 3
Accompanist version: v0.21.2-beta

Additional context

In my case, i set statusBarColor Transparent, and no extra xml setting, but icons are always white.
systemUiController.setSystemBarsColor(color = Color.Transparent,
                                      darkIcons = true)

Android 11

In my case, i set statusBarColor Transparent, and no extra xml setting, but icons are always white.
systemUiController.setSystemBarsColor(color = Color.Transparent,
                                      darkIcons = true)

Android 11 v0.20.2

were you able to get it worked? I'm still facing this issue. I'm also having status bar transparent.


Terraform and hcl files in a directory with a symlink?

I'd like to do something like this
root
|- commons
     | - *.tf
     | - *.hcl
|-dev
     | - parent.hcl
     | - symlink to commons
|-test
     | - parent.hcl
     | - symlink to commons

When I try running terragrunt plan-all it fails because can't find any terraform conf file (Module /.../dev does not have an associated terraform configuration and will be skipped.)

This makes sense, because terraform isn't being called in dev/commons, but rather at dev and terraform doesn't automatically read the tf files when it is one directory under.
With that said, I am not 100% sure if symlinks would still work even if you pointed to commons/main.tf in dev. Terragrunt verbatim copies the files to the working directory (.terragrunt-cache, so I would expect the symlinks to be broken when it is copied over.

Ok, makes sense now. If I copy the commons dir before running the plan/apply it works properly so that must be the problem. Thanks!

Thanks for closing the loop! Closing this issue now.


Private answers

Hey guys.
Would it be possible to enable an option where I can block the bot from responding privately?

You can probably do it in src/message/index.ts by putting an early return like
if (!chat.isGroup) return streamingReply.edit("bot doesnt respond in private chats");

But I am not gonna add this since I think it's out of scope and similar results can be achieved by the block and while list

It worked by putting this code
Thanks @Luisotee


[Theme Switching] 🐛 "Use Device Settings" not working on IOS

https://expensify.slack.com/archives/C049HHMV9SM/p1702574550326249

@chrispader

Partially caused by this: https://github.com/Expensify/App/blob/072efdd0185e1d912844de89127f274442f2fec3/ios/NewExpensify/Info.plist#L123-L124
Also looking into this
https://stackoverflow.com/questions/68243384/dark-mode-usecolorscheme-always-returns-light-on-android/68249064#68249064


Add screen to view the current status of 'sibling' rooms

Ie. other rooms in the same roomlist

Closed by 7bff5c2f75c61fc1430febc167fabf3078a83742.


cmdLineTester_SCCommandLineOptionTests_1 fail (jdk8/j9)

https://github.com/AdoptOpenJDK/run-aqa/runs/1354982566?check_suite_focus=true#step:4:50735
Testing: Run !printallcallsites
Test start time: 2020/11/04 21:36:14 Coordinated Universal Time
Running command: /opt/hostedtoolcache/jdk-8-openj9/1.0.0/x64/bin/jdmpview -core j9core.dmp
Time spent starting: 2 milliseconds
Time spent executing: 689 milliseconds
Test result: FAILED
Output from test:
[OUT] DTFJView version 4.29.5, using DTFJ version 1.12.29003
[OUT] Loading image from DTFJ...
[OUT]
[OUT] Could not load dump file and/or could not load XML file: null
[OUT] For a list of commands, type "help"; for how to use "help", type "help help"
[OUT] > DDR is not enabled for this core file, '!' commands are disabled
[OUT] >
https://github.com/AdoptOpenJDK/run-aqa/runs/1354982566?check_suite_focus=true#step:4:53130
[OUT] nameOption2: Expected to find the cache name
[OUT] TEST PASSED
[ERR] JVMSHRC806I Compressed references persistent shared cache "runner" has been destroyed. Use option -Xnocompressedrefs if you want to destroy a non-compressed references cache.

Looks like cmdLineTester_callsitedbgddrext_openj9 passed in recent running. https://github.com/AdoptOpenJDK/run-aqa/runs/1476255594?check_suite_focus=true#step:4:33468. Not sure if it's intermittent or fixed.

Updating this issue to remove cmdLineTester_callsitedbgddrext_openj9 from the heading.  That test was failing consistently a few weeks ago, but has not failed for a while.  I do not see an obvious PR/fix that would change behaviour, but we can raise it as a separate issue if its turned into something intermittent.

Update cmdLineTester_SCCommandLineOptionTests_1 failures on linux with jdk8/11/15 ( Original one is the failure of cmdLineTester_callsitedbgddrext_openj9):
Testing: nameOption2
Test start time: 2020/12/02 20:42:49 Coordinated Universal Time
Running command: sh nameOption2.sh /opt/hostedtoolcache/jdk-8-openj9/1.0.0/x64/bin
Time spent starting: 1 milliseconds
Time spent executing: 272 milliseconds
Test result: FAILED
[OUT] Hello, world!
[OUT] TEST PASSED
[OUT] nameOption2: TEST FAILED
[OUT] nameOption2: Expected to find the cache name
[OUT] TEST PASSED
[ERR] JVMSHRC806I Compressed references persistent shared cache "runner" has been destroyed. Use option -Xnocompressedrefs if you want to destroy a non-compressed references cache.

Success condition was found: [Output match: TEST PASSED]
Failure condition was found: [Output match: TEST FAILED]
Failure condition was not found: [Output match: Error:]
Failure condition was not found: [Output match: Unhandled Exception]
Failure condition was not found: [Output match: Exception:]

...
Testing: nameOption4
Test start time: 2020/12/02 20:42:50 Coordinated Universal Time
Running command: sh nameOption4.sh /opt/hostedtoolcache/jdk-8-openj9/1.0.0/x64/bin
Time spent starting: 8 milliseconds
Time spent executing: 276 milliseconds
Test result: FAILED
[OUT] Hello, world!
[OUT] TEST PASSED
[OUT] nameOption4: TEST FAILED
[OUT] nameOption4: Expected to find the cache name
[OUT] TEST PASSED
[ERR] JVMSHRC806I Compressed references persistent shared cache "_runner_docker" has been destroyed. Use option -Xnocompressedrefs if you want to destroy a non-compressed references cache.

Success condition was found: [Output match: TEST PASSED]
Failure condition was found: [Output match: TEST FAILED]
Failure condition was not found: [Output match: Error:]
Failure condition was not found: [Output match: Unhandled Exception]
Failure condition was not found: [Output match: Exception:]

cmdLineTester_callsitedbgddrext_openj9 reopened in #50

Test nameOption2 and nameOption4 get the username from the environment variable LOGNAME
export TESTUSER=$LOGNAME
to match the shared Cache generated by NAME=_%u.
In github runner ubuntu environment _%u is runner and $LOGNAME turns out be empty ( github runner macos $LOGNAME  is runner, so test passes with macos).
This actually should also affect test nameOption, which set DEFAULT_CACHE_NAME="sharedcc_$TESTUSER" . However as TESTUSER is empty so grep "$DEFAULT_CACHE_NAME" $TESTSCRIPT.out will return 0. Kind of luck, test is actually fragile.
Next step would need to figure out why $LOGNAME is empty on ubuntu runner, and be runner on macos runner. Is this expected?

LOGNAME is not in the Ubuntu GitHub Actions Environment Variables List. Instead RUNNER_USER is the environment for the user runner. There’s always a chance that the list will get outdated as soon as a new container is configured. We will put a warning if it happens and update our fixes.


Encoder could not be initialized.

Couldn't import Carla egg properly
pygame 2.1.2 (SDL 2.0.18, Python 3.8.10)
Hello from the pygame community. https://www.pygame.org/contribute.html
INFO:  Found the required file in cache!  Carla/Maps/Nav/Town07.bin
Encoder could not be initialized.
Exit

I tried Carla 0.9.8. it still gets this error
Couldn't import Carla egg properly
pygame 2.1.2 (SDL 2.0.18, Python 3.8.10)
Hello from the pygame community. https://www.pygame.org/contribute.html
WARNING: Version mismatch detected: You are trying to connect to a simulator that might be incompatible with this API
WARNING: Client API version     = 0.9.13
WARNING: Simulator API version  = e32e6bff

might be the problem of CUDA or Pytorch, in my case I just reinstalled these two with the follow command conda install pytorch==1.13.1 torchvision==0.14.1 torchaudio==0.13.1 pytorch-cuda=11.7 -c pytorch -c nvidia which works

has someone solve it? i have the same problem in windows @nsliem745


reports with brackets get opening bracket deleted!

assert v_no_beats_per_burst <= 256
report("burst_size_to_axi_len: burst size shall be smaller or equal to 256 beats of data. In this case it would be "&integer'image(v_no_beats_per_burst)&" beats")
severity error;

solved


Look for cookbook name in metadata.rb when checking forks

A lot of the cookbooks on github are named "chef-cookbook-name" or something like that instead of the actual cookbook name.
When forking I prefer to preserve the repo name so there's less confusion for someone else looking at fork.
When Chef-Guard is checking the cookbook being uploaded it's trying to find cookbook name in forks and it fails with
=== Cookbook Compare errors found ===
You are trying to upload 'cookbook-name' version '1.0.0' which is a
non-existing version of a community cookbook! Make sure you are using
an existing community version, or a fork with a pending pull request.
=====================================

The problem with this is the amount of time and effort needed to find the correct repo (if it even exists). We would need to scan each and every repo in your folk organization to see if the repo has a metadata.rb. And for every repo that has a metadata.rb we would need to parse it to see if it matches the cookbook we are searching for....
So I get your point, but I doubt if it is a big enough issue to move from the single HTML GET call we have to do now, to the full blown search logic (which makes at least one call per repo in your fork org and needs to parse all metadata.rb files it encouters) needed to do what you are suggesting...

Thought about this one for a while, but for the time being I'm not going to add this kind of logic.
So closing this as a won't fix for now, feel free to reopen the issue if there are any new compelling reasons that might change my point of view.


Can we have Signal3 and Signal4?

The intent is to make a VO(Value Object) if you need more than 2 variables.
var vo:MyVO = new MyVO();
var signal1:Signal1 = new Signal1<MyVO>();


[HOLD for payment 2023-05-25] Allow admin deleting of comments

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

Problem
When offensive or spammy comments are posted in a room, admins should be able to delete the messages right away. We already allow them to remove messages after they are flagged in the moderation process.
Importance
We want to maintain a pleasant community in NewDot and we can't do that if unwanted messages are hanging around and ruining it for everyone else. This is a top priority for upcoming large conferences like RENDERATL.
Solution
Allow admins of a room to delete comments
Platforms:

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

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

Version Number: N/A
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:

Expensify/Expensify Issue URL:
Issue reported by: @quinthar
Slack conversation: https://expensify.slack.com/archives/C04QEB4MJEQ/p1683650689339049?thread_ts=1683583595.780049&cid=C04QEB4MJEQ
View all open jobs on GitHub

I'll be working on this internally as my top priority

Jumping in here to get this across the line while Neil is OOO

I'm heading OOO from May 17-23rd so I re-added the Bug label in case any action is needed from the BZ team during that time. If this is still open on May 24th, I'll take it back from you @dylanexpensify!
This is entirely internal I believe, so BZ likely isn't needed.

Sounds good!! Have a great time off @lschurr!!

The App PR was just merged and will be CPed to staging.

@dylanexpensify Can you please assign me to the issue. I reviewed the associated PR.

@sobitneupane Assigned you!

I'm heading OOO tomorrow for about a week so adding another BZ member while I'm away to help keep the train moving! 🚂  Thanks @sonialiap!!!

I'll take this one back since I'm back from OOO. Thanks @dylanexpensify!

2023-05-29

@sobitneupane - Can you apply for the job? https://www.upwork.com/jobs/~01ef0a0d267f7aff18

Offer sent @sobitneupane :)

Payment made. Do we need a regression test for this one @sobitneupane?

Regression Test Proposal

Set up a policy with a user and an admin
Send few messages from the user, and a message from the admin in the #announce room
As the user, verify you can delete your comment
As the user, verify you cannot delete the admin's comment
As the admin, verify you can delete the user's comment
As the admin, verify you can delete your own comment

Do we agree 👍 or 👎

Great, thanks @sobitneupane! Created GH for adding regression test. Closing this one out.


docs: updating landing site with some new marketing copy

#68  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Doubt about internal function of royalties

What is the purpose of this function?
thank you
function _updateAccount(uint256 _id, address _from, address _to) internal {
        uint length = royalties[_id].length;
        for(uint i = 0; i < length; i++) {
            if (royalties[_id][i].account == _from) {
                royalties[_id][i].account = payable(address(uint160(_to)));
            }
        }
    }

https://github.com/rarible/protocol-contracts/blob/787c51853265a70e8e9c84dbc7f24f80d8003a25/royalties/contracts/impl/AbstractRoyalties.sol#L22

Hi, Thank you for your interest in the project.
Sorry for the long answer.
The main purpose of this method is to change the royalties account address to a new one specified in the parameter _to

Please, if there are no other questions. Please close issue, thank You.


Authentication Failed using Private Key,Public Key

Hello
We are trying to connect our Mac and aws servers using the private key. but every time we got authentication failed.
Also, we have checked both keys on another application and it's working fine.so I think the problem is in the library. can you please help?
using this method
.byPublicKeyFromFile(username: self.username, password: "", publicKey: "", privateKey: pvtfilePath?.relativePath ?? "")

but not working can you please help us

Can you show me the complete code for how you configure your client?

Here is my code.
Please help us
var session: NMSSHSession?
session = NMSSHSession(host: "<IP_ADDRESS>", port: Int(connection.port), andUsername: "Jigar D")
var pubfilePath = Bundle.main.url(forResource:"Jigarid_rsa", withExtension: "pub")
            var pvtfilePath = Bundle.main.url(forResource:"Jigarid_rsa", withExtension: "")
            session?.authenticate(byPublicKey: pubfilePath?.relativePath ?? "", privateKey: pvtfilePath?.relativePath ?? "", andPassword: "")

Without knowing the NMSSHSession code it's very hard to be more helpful. Is that code available anywhere?

We are using this file please check
Archive 3.zip

@Lukasa We already generate issue but not provide any solution.I think this library also used LibSSH2 because without that how can we connect server.

swift-nio-ssh does not use libssh2, it is a ground-up reimplementation of the SSH wire protocol.

NMSSHSession and swift-nio-ssh are entirely unrelated. swift-nio-ssh and libssh2 are entirely unrelated. I cannot support you with issues with libssh2.

Yes, swift-nio-ssh supports SSH private key authentication.

No, NMSSHSession and swift-nio-ssh are unrelated. You would use entirely different API calls.

@Lukasa if possible can you please provide link or demo ?

There is an example in this repository. Alternatively, you could investigate Citadel.


[Issue]: v20240823 update - UEI & TIN adjustments

Why is this issue important?
In July we learned of changes to the field validations for UEI & TIN fields and adjustments are needed for upload validation to reflect these
Current State
If entity type = subrecipient, UEI is required and TIN is optional; error will generate if UEI is blank
If entity type = contractor, either a UEI or TIN is required; error will generate if both are blank
If entity type = beneficiary, either a UEI or TIN is required; error will generate if both are blank
IAA is an entity type that did not exist before July 2024
Expected State
Recognize v20240823 as the current version number, and the version warning should generate if the version number of the upload file is before or after the current version:
“Upload template version ([uploaded template version]) is [older/ newer] than the latest input template ([latest template version])”
Updates to UEI & TIN requirements:
Current state requirements are correct and should remain unchanged when entity type = contractor or subrecipient. However, the following changes should be made:
if IAA is selected as the entity type, UEI & TIN are both optional. No error message will generate if both are blank
If Beneficiary is selected as the entity type, TIN is required and UEI is optional. If TIN is blank, the following error message will generate: "You must enter a TIN for this subrecipient."
Implementation Plan
The following functions need to be added...
Relevant Code Snippets
No response

@as1729 only for new beneficiaries


Bug fixes!

Download: false for share
Fix module reference (no bugs yet but noticed this wasn't fixed)
Fix share stats (old commit that was missed in transfer?)

👍

Not sure why travis isn't running but everything is passing locally.


support for fastify v5

Great job. Just want to say thanks for putting this out. I just upgraded your plugin and fastify. I love seeing all my dependencies upgraded, and everything just works! Thank you!

thank you for the comment @ryanxcharles ❤ - i finally had some time to get it over the line


⚠ Fortnite has degraded performance

In 90e63e4, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1328 ms

Resolved: Fortnite performance has improved in 55c54da.


Blog Tile: "Changes to Multi-Tenancy in Node-RED Dashboard 2.0"

Related Issue
https://github.com/FlowFuse/website/pull/2181
Description
Considerations:

Blog Tile

Inspirations:

Doesn't need to be too fancy, security themed, with user icons thrown in there too I guess?

Hi @joepavitt would this work?

Ideal - thanks @Yndira-E


Exec format error

After I run docker compose up it prints:
Container ark_... created
Attaching to asa-server-1
asa_server-1 exec /tini: exec format error

Probably because of an architecture mismatch.


Enlarge the Error Raise in yaml2nxdl

I realized that many features in yaml are not actually checked when creating an nxdl, e. g. if one mistypes the units, no error is raised.
We should address this soon

Yes, there is also an related issue that if there is a units attribute in a base class and it is not explicitly written in a base class the reader throws a warning that an attribute w/o documentation is written. Our solution in NXmpes was to just write the units attributes in the appdef again to suppress this warnings. Probably, this is not only related to units but also throws warnings for attributes in general.

Yes, there is also an related issue that if there is a units attribute in a base class and it is not explicitly written in a base class the reader throws a warning that an attribute w/o documentation is written. Our solution in NXmpes was to just write the units attributes in the appdef again to suppress this warnings. Probably, this is not only related to units but also throws warnings for attributes in general.

I think this is actually separate from what @aalbino2 mentions here. What you mention is actually related to the way we handle inheritance in the dataconverter. We can meet for a meeting and put these thoughts together. I want to handle this once correctly. I can explain you in more detail over zoom why it works how it does currently and what I am planning to do to sort this out.

Please pin me when you will discuss this @shabihsherjeel @domna, thank you.
And, yes, Sherjeel is right, I was talking about catching more and more possible errors contained in the yaml file of an app def or base class


feat: Enable tab viewes for swiping

Type: feat
Platform: all
This is like a way popular patter at least in android apps when you swipe view to the left/right and you get the tab switched. That would be a very nice feature.
With something like
<ion-tabs swipable='true' >

to activate it.

Greetings @hiquest!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

Great suggestion! +1

Moving this to our our trello board.
https://trello.com/c/dnCyi4wI/85-scrollable-tabs

+1

+1

+1

+1

I guess guys, since this moved to trello there's no reason +1 this here, instead go to the trello board and upvote there


Handle undefined values in Transformer

I am getting an error when a Point object comes in a query result when the z property is undefined.
See here
The error happens here because:

the Point value is transformed recursively as an Object
isPlain() returns false on undefined
code reached isRelation() but rel is undefined.
undefined.identity fails.

TypeError: Cannot read property 'identity' of undefined

This is related to #122  since Spatial objects are not properly handled either.

Hey there again, thanks for the pull request. I just updated the commit message to ensure it triggers a release and fixed one code style issue. Once I get the build to run I'll merge this :)

:tada: This PR is included in version 5.0.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Bug] Missing error in debugger and incorrect toast message

Description
System throws "unexpected error" instead of "missing row limit field/row offset field" on data source query page for Google Sheets - debugger only shows error for missing row limit
Steps to reproduce the behaviour:

Go to Datasource and click + to create a new Google Sheets datasource
Do not enter details for row limit and row offset and click 'Run'
Observe the error displayed and check debugger. Debugger only shows error for missing row limit and not for row offset. User needs to view browser console to view actual error logs

Important Details

Version: Cloud
OS: Win10
Browser: Chrome
Environment: Production

Will be covered by #6991


magic-enum: fix build issues on gcc 12

Description of changes
disable tests to fix build on gcc 12 by disabling tests
can be enabled after issiu fixed
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, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

fixed it


rkt/run: added --set-env-file switch and priorities for environments

--set-env-file gets an environment variables file path in the format "VAR=VALUE\n...".
This switch may be used multiple times on the command line to provide multiple
environment files. When it used multiple times, it merges each environment file
in a single environment and it returns an error if the same variable is set multiple
times (either in different files or in the same file).
This patch implements priorities among the various way to specify
environment variables, from highest to least priority:

Command line (--set-env)
From files (--set-env-file)
Image Manifest
Inherit (--inherit-env)
A variable in a group with an higher priority will override the value
of a variable with the same name in groups with lower priority without
producing any error. Variables with unique names will be always preserved.

Fixes https://github.com/coreos/rkt/issues/647

ok to test

Patch updated.

LGTM if green


password

Hi everybody,
I'm new to nodejs. I'm using this module to authenticate users for a Microsoft Active Directory. I'm not sure how secure the communication with the active directory is. Are the passwords sent in plaintext to the active directory ???
Thank you!!!

This IMHO depends on the protocol you're using. Stick with LDAP and you should be good to go. However latest release of ldap-js (a dependency of this project) has an issue which is only solved in current master. If you use a fork I've made recently, you will be fine. Look for repositories in "improvisio software" company.

2015 13:43, 13:43, yosser22<EMAIL_ADDRESS>napsal/a:

Hi everybody,
I'm new to nodejs. I'm using this module to authenticate users for a
Microsoft Active Directory. I'm not sure how secure the communication
with the active directory is. Are the passwords sent in plaintext to
the active directory ???
Thank you!!!

Reply to this email directly or view it on GitHub:
https://github.com/gheeres/node-activedirectory/issues/92

So sorry but I'm really a beginner. The URL of the active directory is  'ldap://domain.de'. Does it mean that the communication is secure and the passwords are not transmitted in plaintext??
thank you :)

No, this way i'm afraid its not secure. Try switching to ldaps://same-url (default port is 636) and if it works, you're safe ;-)

2015 13:59, 13:59, yosser22<EMAIL_ADDRESS>napsal/a:

So sorry but I'm really a beginner. The URL of the active directory is
'ldap://domain.de'. Does it mean that the communication is secure and
the passwords are not transmitted in plaintext??
thank you :)

Reply to this email directly or view it on GitHub:
https://github.com/gheeres/node-activedirectory/issues/92#issuecomment-118615553

Thank you !!!!


feat: Add support for signInMessage method

Description
This PR is about adding support for the signInMessage method into the wallet-selector API.
This new method/function(way) will allow dApp users to "sign-in/login" into a dApp without a contractId (without adding LAK) by signing a message in the wallet based on the NEP413 standard for signing messages.
Once the message is signed and verified wallet selector will set the state as "signed-in"
Closes: https://github.com/near/wallet-selector/issues/921
This PR includes

Added interfaces for signInMessage method in core package and exposed them.
Added support for signInMessage in the core package.
Persisted the original message and the signedInMessageAccount into localStorage and rxjs store.
Added a new signInType() function to the selector API.
Added signInMessage support for all wallets that currently support signMessage.
Added new helper function verifyMessageNEP413 to core helper functions.
Added a signInMessage method to the API of the modal, so we can open the modal in a state to handle signInMessage e.g modal.signInMessage(params)
Updated examples, added a new button to trigger signInMessage
Updated docs for the changes in this PR.

Wallets that support signInMessage in this PR

Meteor Wallet
Here Wallet
Near Snap
Nightly Wallet
WELLDONE Wallet
NearMobileWallet
MyNearWallet
Sender

How to use/trigger signInMessage in a dApp

Note: All the examples below are documented in this PR.

Setup Wallet Selector & Modal
import { setupWalletSelector } from "@near-wallet-selector/core";
import { setupModal } from "@near-wallet-selector/modal-ui";
import { setupMeteorWallet } from "@near-wallet-selector/meteor-wallet";
import { setupHereWallet } from "@near-wallet-selector/here-wallet";

// Setup Wallet Selector
const selector = await setupWalletSelector({
  network: "testnet",
  modules: [
     setupMeteorWallet(),
     setupHereWallet(),
     // ...you can include other wallets too.
  ],
});

// Setup Modal
const modal = setupModal(selector, {
  contractId: "guest-book.testnet" // nothing changes here the `signIn` will use this contractId as always.
});

Trigger signInMessage

// Construct the message
const message = "test message for verification";
let nonceArray: Uint8Array = new Uint8Array(32);
nonceArray = crypto.getRandomValues(nonceArray);
const nonce = Buffer.from(nonceArray);
const recipient = "myapp.com";

modal.signInMessage({ message, nonce,  recipient }); // Choose the wallet from the modal to sign-in via signInMessage

Subscribe/listen to events
// Listen to `signedInMessage` event
const subscription = selector.on("signedInMessage", ({ walletId, accounts, message, signedInMessage }) => {
   console.log({ walletId, accounts, message, signedInMessage });
});

// Unsubscribe.
subscription.remove();

const signInType = selector.signInType();
console.log(signInType); // "message"

Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

Type of change.

[ ] FIX - a PR of this type patches a bug.
[x] FEATURE - a PR of this type introduces a new feature.
[ ] BUILD - a PR of this type introduces build changes.
[ ] CI - a PR of this type introduces CI changes.
[ ] DOCS - a PR of this type introduces DOCS improvement.
[ ] STYLE - a PR of this type introduces style changes.
[ ] REFACTOR - a PR of this type introduces refactoring.
[ ] PERFORMANCE - a PR of this type introduces performance changes.
[ ] TEST - a PR of this type adds more tests.
[ ] CHORE - a PR introduces other changes than the specified above.

Resolving the conflicts introduced a bug in the meteor-wallet implementation, it seems that meteor-wallet-sdk - https://github.com/Meteor-Wallet/meteor_wallet_sdk should be updated, so it can work properly with the new changes introduced, more info can be found in here - https://github.com/Meteor-Wallet/meteor-public/issues/25.


Fixed mountpoint check

Replaced unused and non-existing variable PREFIX with SA_INSTALL_PREFIX. This caused RKE2 to be installed in the default location /usr/local even though it was a mount point, which could result in RKE2 not being started after a reboot.
https://github.com/rancher/rancher/issues/38106
https://github.com/rancher/rke2/issues/3001

Thank you for the contribution!


Fixing "double bias" in LSTM modules

Probably not a permanent solution - something like LinearNoBias should be in a general-purpose package like nnx - but I'm using it for now, and figured you might want to as well

Thanks!


Performance WGPU: Improve Reduce kernels

The reduction of dimension is not fully parallelized, since one thread reduces a whole dimension alone with a for loop. To improve the performance of shaders, we should use a tree-based approach similar to what is described in the Nvidia CUDA documentation (https://developer.download.nvidia.com/assets/cuda/files/reduction.pdf).
Reduce one dimension

[ ] Sum dim
[ ] Mean dim
[ ] Argmax
[ ] Argmin

Reduce full tensor to a scalar

[ ] Sum

I can take it.
I think we don't have any benchmark for the reduction of dimension, so I guess it makes sense to add a separate task for it

Hey there! I'm interested in this issue as well. I've crudely implemented Mark Harris' algorithm for parallel reduction with WGSL and wgpu, and I'm doing some experiments/benchmarks to improve performance. Here's my github repo in which I perform the experiments.
I'm in vearly early experimentation, and I've only implemented sum-reduce with f32. For now, one result that I got with my hardware (a laptop with an NVIDIA RTX 2060) is that my implementation is about 1.66x faster with an input size of 2^26 (i.e. 67,108,864 elements). Here's the plot generated with Criterion for three different input sizes (2^24, 2^25, 2^26) and comparing between CPU, Burn and my own wgpu implementation:

The above plot is overly optimistic and cherry-picked, evidently. Another benchmark, done from input sizes ranging from 2^20 to 2^28 (the max size for my GPU apparently) shows a less optimistic picture:

For maximum reproducibility, the benchmarks above were done under the implementation of this commit in my repo.
I believe this improvement over a narrow range can be explained by the fact that my implementation chooses a "work per thread" size of $k \cdot \lceil 2^{\log_2 {\lceil \log_2(n) \rceil}} \rceil$, i.e. $k$ times the smallest power of two greater than or equal to $\lceil \log_2(n) \rceil$. In the benchmark above, I use $k=8$.
My rationale was based on Mark Harris' slides that @louisfd linked above, in which he says: "Brent’s theorem says each thread should sum $O(\log n)$ elements [but] In my experience, [it is] beneficial to push it even further". He cites three reasons for this:

Possibly better latency hiding with more work per thread
More threads per block reduces levels in tree of recursive kernel invocations
High kernel launch overhead in last levels with few blocks

These are the parameters for n=2^24 to 2^28 elements w/ my implementation:

n
workgroup_size
work_per_thread
workgroups

2^20 = 1048576
1024
256
4

2^21 = 2097152
1024
256
8

2^22 = 4194304
1024
256
16

2^23 = 8388608
1024
256
32

2^24 = 16777216
1024
256
64

2^25 = 33554432
1024
256
128

2^26 = 67108864
1024
256
256

2^27 = 134217728
1024
256
512

2^28 = 268435456
1024
256
1024

Help on this is greatly appreciated! I'm still a beginner with GPGPU and I definitely missed many optimization mentioned by teh CUDA webinar.

@louisfd has worked on this issue in the past month. I think we now have two different algorithms for sum-dim and mean-dim. Not sure we have something new for sum and mean.
We also have benchmarks in burn-wgpu/benches/reduction.rs, so you could compare the new algorithms.

Oh great! I hadn't seen it. I'll look into it ASAP


How do I do Edge and Chrome flavors?

Testing #220989
For workspace.externalBrowser...
I don't really wanna put in an absolute path but I wanna do like edge-dev edge-canary chrome-canary or whatever and that "just works".

That's the syntax of js-debug and I have a package to do that: https://www.npmjs.com/package/@vscode/js-debug-browsers If there's interest I could make that slightly more generic for usage here.

Since we are using https://github.com/sindresorhus/open here, I would suggest to open a PR to them to support this. For the time being I suggest to use the path to the executable.


Respect batch_size everywhere to support RNN?

I tried to use a Keras model with a stateful RNN, but did not manage to get it to work. The idea was to introduce previous internal state like an additional recurrent input. Unfortunately recurrent units require a fixed batch_input_shape.
One part of the problem is that ExperienceReplay.get_batch() does not always return a fixed batch shape (but batch_mem = batch_size is not enough). On the other hand the dimensions of states from environment are not handled in same shape (maybe KerasModel.values()).

I believe you will have to write your own memory class to correct that problem actually.
You can get your batch size to be always the same by initializing the memory with zeros. You can do that for example, by calling remember in a for loop adding zeros states.
But, note that memory samples previous experiences randomly. So, even though you are using state-fulness you may not be getting the continuity that you may be expecting.
In any case, I hope that helps.

Oh, you are right. This won't work the way I thought and has to be approached differently. Thanks,


Migration to .NET Standard 2.0 + .NET 4.5

I have migrated TickSpec to .NET Standard 2.0. I have made all projects to output both .NET Standard 2.0 and .NET 4.5 binaries.
This resolves #8
The solution doesn't support debugging feature files when .NET Standard assembly is used, it is still supported with .NET Framework assembly.

Please fix the builds. Feel free to consider migration to FAKE 5 if it makes it easier for you.

Just wow; congrats.
Minor side points:

The release notes need an update and a version bump (For me it warrants 2.0).
The commits seem reasonably purposeful - would not argue with a non-squashed merge.
I'm seeing a failing test in CustomContainer on my local 15.9.0 Preview 2 when I run all tests OOTB in Debug Message: System.InvalidOperationException : Cannot decide which constructor to use. The type has multiple constructors with the same maximum number of parameters: System.IntPtr

I can confirm that I am able to see locally the same issue as @bartelink regarding the failing test. It seems to me like that the AutoFac wiring is not working properly (just a guess).
Moreover, when I press run all tests, VS claims that 70 tests passed, 1 failed and 1 not runned (I don't understand why, it is in the CSharp.NUnit and it was runned). The tests from xUnit.FSharp projects are missing completely.

Yes, I know about the issue with CustomContainer test doesn't work and you can see that I am intentionally skipping the test in fake build.
It is really broken in the path where IL generation isn't used, so .NET tests succeeds and .NET Core fails. However, it has nothing to do with .NET Core, it was broken before when you use the path of code where IL generation isn't used.
When I was looking at the code it felt to me that problem is that the default container is already taken when custom container is replaced - so it uses default container instead of custom one. I expected that we will fix this in separate pull request as it may be harder, feel free to fix it.
Regarding the missing test, VS behaves weirdly, when you open solution and build for first time it will correctly show just 71 tests but when you run the tests it will add one additional test which should not be there and which isn't executed, it is the CSharp.NUnit test which is called Test. It can be something in NUnitWiring.cs which I rewrote simplified way for NUnit 3 but I don't see the issue, any idea?

Have you seen also the missing xUnit.FSharp tests?
I have no idea why the extra "not runned" test appears in test explorer. It somehow looks like an issue in the Test Explorer but maybe we are triggering it somehow as it does not happen in other cases. I can try to research it tomorrow.
Regarding the test - let's create an issue and solve it later. You are right that this pull request is big enough. However, do you have any suggestions how to ensure that both methods will work? Like running the tests in both variants? Or, are you already doing that?

Ah, you removed the project which used the xUnit1 - I am fine with accepting the same argument as for nUnit2. The xUnit2 project is the same, it just uses the latest xUnit. However, in that case please rename xUnit2 project to just xUnit because the number does not have any meaning.
The failed test is fixed in branch bugfix/DelayReadingOfServiceProviderValue. I will start pull request as soon as you merge these changes, so the test can be fixed.
The issue with the not-started test seems like an issue in Test Explorer. Seems like VS tries to guess which test is there without compiling and then it does not remove it when running. Still, in my opinion it is not anyhow a blocker to merge this. Even if it would be an issue, we can handle it later.

I have done the changes:

made nuget packaging to again take version and release notes from release notes file (requested by @mchaloupka)
renamed xUnit2 project to xUnit (requested by @mchaloupka)
updated release notes to version 2.0.0 and put there this upgrade as the change (requested by @bartelink )

All comments were addressed, so I merged the changes. If we find any issues then we will address them separately.


Which version of Pytorch should we use ?

PyTorch 1.0, 0.3 and 0.4 all gave me annoying "compatibility" bugs.
For example, Pytorch 1.0 gave me a
pggan-pytorch/custom_layers.py", line 114, in forward
    return x + self.bias.view(1,-1,1,1).expand_as(x)
RuntimeError: expected type torch.cuda.FloatTensor but got torch.FloatTensor

Pytorch 0.4 gives me a RandomSampler based error, and after fixing it, I get :
  File "torch/nn/functional.py", line 1537, in
    return lambd_optimized(input, target, size_average, reduce)
RuntimeError: input and target shapes do not match: input [32 x 1], target [32] at /pytorch/aten/src/THCUNN/generic/MSECriterion.cu:15

Similarly with Pytorch 0.3 (it was a .item() error). So which version do you want us to use ?

Try PyTorch 0.4.1 thats what im currently using to run it.


Timing tasks in the range of Microseconds, using QuarkTs with Omnetpp

Hi,
Recently I came across a scenario where I may need to time tasks below 1 Millisecond - mainly within a simulation framework like Omnetpp. I somehow like the idea of using QuarkTS with Omnet as it would allow me to reuse a lot of code I will write for the simulation than in the actual firmware.
The plan is to use Omnetpp Self-Messages to simulate the ticks for QuarkTs. As I won't use a custom IDLE function, I thought about executing the QuarkTs Run Function after every Tick call and/or events. This should then execute all timed and event-driven tasks when I remember the kernel code correctly.
Coming to my questions:

What is your opinion on using a non-preemptive OS for timing below a millisecond?
Do you think the simulation would work with the proposed solution?
Without deep-diving into the code: is scheduling below a millisecond even possible with the current implementation? I normally use Q_SETUP_TIME_CANONICAL

Thanks for your effort!

Hi @HpLightcorner,
I don't have much knowledge of Omnetpp, but basically what you need for QuarkTS to meet temporary specifications is that you have to feed its internal tick from a context running in the background by invoking the qOS_SysTick () function. In embedded systems this is accomplished with a timer interrupt. This tick can be less than 1mS, but keep in mind that the maximum times you can define will be less.
For example:

For 1mS tick, the maximum time you can define is 49.7 days.
for 1uS tick the maximum time you can define is 71.58 minutes.

you can also use different time bases, for example 50uS

for 50uS tick the maximum time you can define is 59.65 hours.

If you do this, all the specifications should be aligned to that base.
for example, if you want to create a task that runs every 2sec
/ * Im assuming that your are using Q_SETUP_TIME_CANONICAL * /
#define TICK_TIME (50uL) / * uS * /
#define DESIRED_TASK1_PERIOD ( 2uL ) / * Sec * /
#define TASK1_PERIOD_SPEC              ( (DESIRED_TASK1_PERIOD * 1e6) / TICK_TIME )
qOS_Add_Task (& Task1, Task1Callback, qMedium_Priority, TASK1_PERIOD_SPEC , qPeriodic, qEnabled, NULL );
and the qOS_Run() function must be executed in a base context without any type of time control, basically at the maximum of the CPU (at full speed) or at least at a time significantly faster than the Tick that you are using as time-base.
it would be impossible to ask a task for 1uS if the tick is also running at 1uS.
Even if we ask 5us to a task and the tick is 1uS it would also be difficult to achieve, we must bear in mind that there are also other tasks that must be verified and it is possible that the required time specification cannot be met.
I hope that in this last you have understood my point.
ok, so going to your questions:
What is your opinion on using a non-preemptive OS for timing below a millisecond? totally possible
Do you think the simulation would work with the proposed solution? its posible, but you also need to call both, the qOS_SysTick and the qOS_Run. Just keep in mind what I mentioned before.
Without deep-diving into the code: is scheduling below a millisecond even possible with the current implementation? I normally use Q_SETUP_TIME_CANONICAL . Yes, but the maximum times that you can define will be less
just to complement, I think this kind of discussions can be moved here  :) ,  https://github.com/kmilo17pet/QuarkTS/discussions
let me know if you need more info.
Best,


chore: Add rapid mode to skip new app creation in cypress test

Pull Request Template
Use this template to quickly create a well written pull request. Delete all quotes before creating the pull request.

Description

Please include a summary of the changes and which issue has been fixed. Please also include relevant motivation
and context. List any dependencies that are required for this change
Links to Notion, Figma or any other documents that might be relevant to the PR

Add a TL;DR when description is extra long (helps content team)

Fixes # (issue)

if no issue exists, please create an issue and ask the maintainers about this first

Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Type of change

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Chore (housekeeping or task changes that don't impact user perception)
This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions, so we can reproduce.
Please also list any relevant details for your test configuration.
Delete anything that is not important

Manual
Jest
Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=d340762

/ok-to-test sha=d340762

/ok-to-test sha=f39fec8

/ok-to-test sha=3c69bcb

/ok-to-test sha=6aa351b

/ok-to-test sha=6aa351b


haunted house announcement card

📝 Changed routes:

/

(view full size)

Commit 72a24b2a82165e16f21bbfd202dc6f2a04bade62 (https://hackathons-i3l63nmdo.hackclub.dev).

@devramsean0 we kept there b/c all the old ones were there too, like epoch! But what do you recommend?

not entirely sure, it just feels wrong. but maybe near where the hackathon grant card is?

It's also there, so people notice it. We tried a lot of diff placements, but they all looked odd

heres one

*here's one we tried out


Allow when key for step configs

For simple on/off of the config based on the context it will be much more readable than using 'adjust' and having to think about the final ordering of the configs.
No changes to the config will be allowed (no modification of values) just 'enable' config if when is evaluated true
Compare:
adjust:
-  when: distro == fedora
   prepare+:
      - name: Fedora specific stuff
        how: shell
        script: <some script>
--snip more adjust rules and discover/provision/...---
prepare:
 - name: run this each time
   how: shell

with
prepare:
  - name: run this each time
    how: shell
  - name: Fedora specific stuff
        how: shell
        script: <some script>
        when: distro == fedora

Interesting idea. Sounds quite useful. And concise. And should be easy to implement. From the top of my head I don't see anything this approach would break.


different region support for different account

Hi Team,
I am using 1 instance scheduler to support multiple accounts.
Is it possible to enable different set of region for different account ?
for example: I want to enable instance scheduler for Frankfurt, Ireland region for account A and Ireland, Paris region for account B.
Current config entry in dynamoDB will enable the mentioned regions for all the accounts. There is no option to specify regions specific to a particular account.
Thanks,
Amol

Hi @luteamol You can deploy multiple stacks of instance scheduler and manage the accounts/regions in different combinations?

Thanks @gockle !
Managing multiple instances of instance scheduler would make it very complicated.
I would request to take this as a feature request.  A new config entry in Instance-Scheduler-ConfigTable should be able to identify regions for any account.
For example, entry-1 would define certain regions and related account list, while entry-2 will define another set of regions and its account list.
Thanks,
Amol

Hi @luteamol At this point there are no planned releases for this solution, the requirement mentioned above poses some unique changes,

The solution currently loads all the configs and starts multiple lambda executions for each service based on the account-region combinations in the file cloudwatch_event_handler.py the method handle_request and _run_schedulers_as_lambda.
If the solution has to implement a requirement where each account has different combinations of regions, the solution has to be modified extensively in the UI (CloudFormation template) and the config (DynamoDB table) where the data is stored along with modifications for the python file.

This would entail making changes which will create issues for all scenario where the stacks have to be updated to the latest version from 1.4.0. Hence it's unlikely this feature can be implemented due to both the effort and the unlikely situation that the back compatibility is broken due to incompatible update options.
Recommendations:

Run multiple stacks each stack will have unique combinations of regions and accounts
Modify the cloudwatch_event_handler.py file in your stack both the handle_request and _run_schedulers_as_lambda should be modified.

Thanks @gockle for detailed explanation !
I am closing this issue.


[Bug] Some functions from Templater are called twice on creation

Hello, thanks for the great plugin.
I'm using Templater and tp.system.prompt in my templates and when I try to create new note using QuickAdd prompts to enter are showing 2 times and only last value is used in the final note.
Example of template (test.md):
<% tp.system.prompt('hello templater') %>

Settings for template in QuickAdd:

Hey! Thank you for your detailed report. This should be fixed in the latest version (0.2.8). Please confirm. =D

@chhoumann, I've tested latest version and I confirm the issue is fixed, thank you


Docker quick start file not found

Pulled newest autoware docker, runing with error:

localization can not be complete.

[Errno 2] No such file or directory: u'/home/autoware/Autoware/ros/install/runtime_manager/share/runtime_manager/scripts/setup_tf.launch'

Hi, I have the same issue, and it turns out that the path is wrong in docker. See my post here:
https://discourse.ros.org/t/autoware-1-11-0-quick-start-demo-failed-to-load-the-map/9019
I also tried source build in my ubuntu 16 machine, and the path is actually correct for source build, I wonder why it is different in docker?

@YangLiu14  What do you mean path correct in ubuntu 16.04? Dokcer path has anything to do with host machine?

Don't worry about the thing I said about Ubuntu 16, I just tried to source build Autoware there.
I just meant the in the my_localiztion.launch file the path for finding the setup_tf.launch is ".../runtime_manager/script/setup_tf.launch".   This works fine, when I source build the Autoware from master-branch in my host.       But, when I run things in docker, the path is incorrect now because setup_tf.launch is located in ".../runtime_manager/launch_files/setup_tf.launch" inside the docker.
So I guess, it might simply because they didn't update the path in launch file inside the docker.

@YangLiu14 Right. anyway, the newest docker image even the default rviz setting were not right (it does not using base_link or world but map). Why they build docker image without any test........

Thanks for using Autoware and for your question. However the issue tracker is for posting confirmed bugs and feature requests.
We ask that you please ask questions at the ROS Answers website following our support guidelines: https://github.com/autowarefoundation/autoware/wiki/Support-guidelines#ros-answers. Please pay particular attention to the information we ask you to provide.
We are aware of the several issues with paths in the 1.11 release. They are being fixed in the repository. Contributions of bug fixes and testing are always welcome.

it is a bug, pls confirm

Yes, it is a bug, and it has been fixed.

Please ask your question at ROS Answers.

It seems this issue can be closed since (1) the bug is fixed, and (2) ROS Answers is introduced.


json file and elasticsearch

Hi
Im exploring elasticsearch and kibana and found out this excellent parser when i was looking around in github.
It would be nice import facebook messages to elasticsearch and draw a nice graphs and stats about conversations, maybe :P
One big problem is elastic does not like this kind json formating what this script makes..
Is there any simple way to make this work or it's not so easy ? I was just thinking it would be nice dataset to play around. If my idea is stupid then just ignore me :D
I tried to import and modify fb parser json using this sample data as example but did not get any great success.
https://www.elastic.co/guide/en/kibana/current/tutorial-load-dataset.html

Hi there! That's a good idea! I'm not familiar with elasticsearch really, but it shouldn't be too difficult to add another output format. However, looking at the documentation you sent, shouldn't it just be a matter of writing a mapping?

Going to close this due to inactivity. Please feel free to re-open if you have more questions.


fix: event hubs track2 perf coverage check fail

Description
After changing all perf test pom, now the eventhubs track2 perf CI is failed on main branch.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

Hi Alan, eventhubs track2 perf CI on main branch is failed now.  According to the old code eventhubs perf should not require jacoco test.

/azp run java - eventhubs

CI fails without this https://dev.azure.com/azure-sdk/public/_build/results?buildId=1275792&view=results


Update xml-crypto dependency to include fix

Update the xml-crypto dependency so that it uses a version of xml-crypto that contains the fix for the AT&T SAML issue.
Link to commit fe47b7ffac6c0f460d3802ba578c8a340218a28f.
https://app.asana.com/0/815188709102173/1192410228611211

Similar unit test failures happened on the last commit on master, so I'm gonna ignore them for now.


fix: Use UTC time in AWS Lambda integration

We use UTC throughout the SDK, the Lambda integration was the only
exception, now fixed.
Explicitly setting the timezone to UTC fixes a problem when loading the
AWS CloudWatch Logs console, where using local time is unreliable.

@untitaker can I ignore https://travis-ci.com/github/getsentry/sentry-python/jobs/377422532#L1164 ?

Yes. I think this may be a new release of Hypothesis causing the error. I will likely end up pinning hypothesis.


fix: react-navigation disable buggy automatic cleanup mechanism

Details

In this PR we:

explicitly disable react-navigations cleanup mechanism. It was added here to clear the navigation state when switching navigators, which is what we don't want (its what's causing the bug we aim to fix with this PR)
However, when we sign out we expect the state to reset (so that when we login we are not back to where we've logged out), so we explicitly reset the navigation state there

Fixed Issues

$ https://github.com/Expensify/App/issues/44600
PROPOSAL:
Tests

[ ] Verify that no errors appear in the JS console

Offline tests

QA Steps

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[ ] I linked the correct issue in the ### Fixed Issues section above
[ ] I wrote clear testing steps that cover the changes made in this PR

[ ] I added steps for local testing in the Tests section
[ ] I added steps for the expected offline behavior in the Offline steps section
[ ] I added steps for Staging and/or Production testing in the QA steps section
[ ] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[ ] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[ ] I included screenshots or videos for tests on all platforms
[ ] I ran the tests on all platforms & verified they passed on:

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

[ ] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[ ] I followed proper code patterns (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[ ] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I followed the guidelines as stated in the Review Guidelines
[ ] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] I verified that if a function's arguments changed that all usages have also been updated correctly
[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[ ] I verified that all the inputs inside a form are aligned with each other.
[ ] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/1e4c2c61-2e09-479b-b4b3-28370c5f4f97

MacOS: Desktop

Note: I realised that we shouldn't need to have this additional logout > clear navigation state logic.
When we log out react-navigation should clear all state from routes that don't exists in the newly mounted navigator.
I believe this is a bug in react-navigation which should be straight forward to fix. I started implementing a PR here, will finish the fix tomorrow and then update this PR!

https://github.com/react-navigation/react-navigation/pull/12128


Creation of Configuration Component

@GerritBurgerDev - Failing  build..? Need some help..?

Hey sorry got a little hectic first week will jump on it on Monday
On Fri, 04 Feb 2022, 10:18 ubergeoff, @.***> wrote:

@GerritBurgerDev https://github.com/GerritBurgerDev - Failing build..?
Need some help..?
—
Reply to this email directly, view it on GitHub
https://github.com/ubergeoff/flingo/pull/19#issuecomment-1029749754, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHFN2SL763G72U2JSBHKDFLUZODU5ANCNFSM5M5JKI3A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>


Update plex-media-server to <IP_ADDRESS>35-2d20185b0

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if updating a cask:

[ ] sha256 changed but version stayed the same (what is this?).
I’m providing public confirmation below.

Thanks for the PR!


fix(IAM Identity): update service after recent API changes

PR summary
This PR updates the IAM Identity service - including integration tests and examples - after recent changes to the API definition.
PR Checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Current vs new behavior

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

:tada: This PR is included in version 0.17.15 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Nil node in edge.rb creates NoMethodError

There is a bug where dependencies listed as ".", which indicates the root package, returns a NoMethodError in getNodeNameAndPort( node ) from lib/graphviz/edge.rb because the node is a nil.
NoMethodError: undefined method `id' for nil:NilClass

I added a temporary if-block to bypass this.
    def getNodeNameAndPort( node )
      name, port = nil, nil
      if node.class == Hash
        node.each do |k, v|
          name, port = getNodeNameAndPort(k)
          port = v
        end
      elsif node.class == String
        name = node
      else
        if node.nil?
          name = "." # I added this if-block
        else
          name = node.id
        end
      end

      return name, port
    end
  end

Thanks for catching that one @wildmaples 👍  I'll push the fix up later today 🚢


Re-run comparison to PSI + some minor fixes

I was looking at resolving the discrepancy in the result compared to PSI and as part of that I re-ran the Jochens Amor code to re-create the reference files that I was comparing against.
Surprisingly the results are now much closer.
There are two possible reasons:

I made a slight change in the command line arguments used to create the files, I don't think it should make a difference, but it might.
The Amor reduction workflow has changed on their end and the changes made the results more similar.

In any case, I uploaded the new reference files to our file-server.

Comparison between the old and the new results:
Old

New

This is the commit that seems to have been the cause of the change: https://github.com/jochenstahn/amor/commit/140e3192ddb7e7f28acee87e2acaee65ce1332aa


pdksync - [CAT-1606] : Adding release_prep gem group

[CAT-1606] : Adding release_prep gem group
pdk version: ``

Not required anymore.


Question: Displaying embeded HTML in markdown

I'm working on a project that pulls data from a third party API. The Markdow contains HTML content that is not rendered with juicy-markdown. For example:
To access the free version of this course, click the blue button that says Access Course Materials. is rendered with HTML markup instead of rendering italics for em
Developing Android Apps. Following Android's Core App Quality Guidelines, we'll identify the production criteria that still needs to be implemented, then tackle each problem until Sunshine is ready to publish on the Google Play Store. In the process, we'll introduce you to a host of tools and concepts, including: material design approaches, image-handling, performance profiling, and more!
Is there a workaround for this?

Sorry, but I didn't get how would you liek this to work. Could you provide jsbin/jsfiddle with sample code?

it should render the html. For example:
Developing Android Apps.
Should render as a hyperlink and not put the code in the markdown.  If I use marked-element component it renders the markup as a link.

Sorry, maybe it's because it's quite late at my timezone, but I'm still not sure If I get your problem correctly.
AFAIK for marked-element you need to provide content inside


LUCENE-9879: Skip errorprone on non-nightlies.

Let's give others some time to express their opinion and I'll commit it in. If we decide a separate option is good then I'll add it, no prob.

+1, why not try it out? If it becomes annoying in our nightly builds (e.g. many failures or something), we can revisit and put it somewhere else. But I think it'd be great to get this out of compileJava and compileTestJava


"Sălcioara" is listed in the drop-down menu

Description:
"Sălcioara" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Sălcioara" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Sălcioara, CĂLĂRAȘI (Curcani)".


The submissionData is shown as Google Maps Static Panorama using lng, lat, pitch, heading

https://github.com/sopra-fs24-group-12/PeekSeek-client/commit/a1e806f14dbf7edd008e1b6e37e7f27d37205b08