Detect collisions

A new system has been added that detects collisions between airplanes. Contrary to the previous version, the new implementation simply checks the distance between two airplanes and compares it to a predefined threshold.

Codecov Report

Merging #273 (c1c0ec0) into 0.4.0-dev (1ffb8b9) will decrease coverage by 0.39%.
The diff coverage is 35.71%.

@@              Coverage Diff              @@
##           0.4.0-dev     #273      +/-   ##
=============================================
- Coverage      48.51%   48.12%   -0.40%
=============================================
  Files             27       28       +1
  Lines            439      453      +14
=============================================
+ Hits             213      218       +5
- Misses           226      235       +9

Impacted Files
Coverage Δ

game/simulation/src/bus/event.rs
0.00% <ø> (ø)

game/simulation/src/system/detect_collision.rs
30.76% <30.76%> (ø)

game/simulation/src/state/running.rs
93.54% <100.00%> (+0.21%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


brew link doesn't work after running brew diy

Bug report
Please note we will close your issue without comment if you delete, do not read or do not fill out the issue checklist below and provide ALL the requested information. If you repeatedly fail to use the issue template, we will block you from ever submitting issues to Homebrew again.

[ ✔] ran brew update and can still reproduce the problem?
[✔ ] ran brew doctor, fixed all issues and can still reproduce the problem?
[✔ ] ran brew config and brew doctor and included their output with your issue?

What you were trying to do (and why)
I am trying to run brew link on a formula created with brew diy after installing the formula to the location that brew diy gave.
What happened (include command output)
I ran brew diy and it outputted --prefix=/usr/local/Cellar/libudfread/1.1.1. I then ran brew link libudfread and got the following.

Command output

Error: No available formula with the name "libudfread"

What you expected to happen
I expected it to link the formula.
Step-by-step reproduction instructions (by running brew commands)
brew diy
brew link
Output of brew config and brew doctor commands

`brew config`:
`HOMEBREW_VERSION: 2.5.8-56-g18c638d
ORIGIN: https://github.com/Homebrew/brew
HEAD: 18c638de30b20b037914023d92d693dd5608962c
Last commit: 3 hours ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: a642cfcc78f9fbb0c74905b013b225a59532f1be
Core tap last commit: 11 minutes ago
Core tap branch: master
HOMEBREW_PREFIX: /usr/local
HOMEBREW_CASK_OPTS: ["--colorpickerdir=/Library/ColorPickers", "--prefpanedir=/Library/PreferencePanes", "--qlplugindir=/Library/QuickLook", "--screen_saverdir=/Library/Screen Savers", "--dictionarydir=/Library/Dictionaries", "--fontdir=/Library/Fonts", "--input_methoddir=/Library/Input Methods", "--internet_plugindir=/Library/Internet Plug-Ins", "--audio_unit_plugindir=/Library/Audio/Plug-Ins/Components", "--vst_plugindir=/Library/Audio/Plug-Ins/VST", "--vst3_plugindir=/Library/Audio/Plug-Ins/VST3"]
HOMEBREW_DISPLAY: /private/tmp/com.apple.launchd.wpb6e8bkbi/org.macosforge.xquartz:0
HOMEBREW_MAKE_JOBS: 8
Homebrew Ruby: 2.6.3 => /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/bin/ruby
CPU: octa-core 64-bit kabylake
Clang: 12.0 build 1200
Git: 2.29.2 => /usr/local/bin/git
Curl: 7.64.1 => /usr/bin/curl
Java: 15.0.1, 13.0.2, 12, 1.8.0_272, 1.8.0_221
macOS: 10.15.7-x86_64
CLT: 1<IP_ADDRESS>
Xcode: 12.1
XQuartz: 2.7.11 => /opt/X11`

`brew doctor`:
`Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry or file an issue; just ignore this. Thanks!

Warning: A newer Command Line Tools release is available.
Update them from Software Update in System Preferences or run:
  softwareupdate --all --install --force

If that doesn't show you an update run:
  sudo rm -rf /Library/Developer/CommandLineTools
  sudo xcode-select --install

Alternatively, manually download them from:
  https://developer.apple.com/download/more/.

Warning: Some installed formulae were deleted!
You should find replacements for the following formulae:
  sudokut
  tfdocgen
  plib
  aplus
  tilp
  libticables
  lziprecover
  clamtk
  clamtk
  libticonv
  libtifiles
  gfm
  libudfread
  libudfread
  libticalcs
  OpenGL
  xdg-utils

Warning: "config" scripts exist outside your system or Homebrew directories.
`./configure` scripts often look for *-config scripts to determine if
software packages are installed, and which additional flags to use when
compiling and linking.

Having additional scripts in your path can confuse software installed via
Homebrew if the config script overrides a system or Homebrew-provided
script of the same name. We found the following "config" scripts:
  /Library/Frameworks/Python.framework/Versions/3.9/bin/python3.9-config
  /Library/Frameworks/Python.framework/Versions/3.9/bin/python3-config
  /Library/Frameworks/Python.framework/Versions/3.9/bin/pybind11-config
  /Library/Frameworks/Python.framework/Versions/3.8/bin/python3-config
  /Library/Frameworks/Python.framework/Versions/3.8/bin/python3.8-config
  /Library/Frameworks/Python.framework/Versions/2.7/bin/python2-config
  /Library/Frameworks/Python.framework/Versions/2.7/bin/python2.7-config
  /Library/Frameworks/Python.framework/Versions/2.7/bin/python-config
  /Library/Frameworks/Python.framework/Versions/3.5/bin/python3.5m-config
  /Library/Frameworks/Python.framework/Versions/3.5/bin/python3.5-config
  /Library/Frameworks/Python.framework/Versions/3.5/bin/python3-config
  /Library/Frameworks/Python.framework/Versions/3.7/bin/python3.7-config
  /Library/Frameworks/Python.framework/Versions/3.7/bin/python3.7m-config
  /Library/Frameworks/Python.framework/Versions/3.7/bin/python3-config
  /Library/Frameworks/Python.framework/Versions/3.6/bin/python3.6m-config
  /Library/Frameworks/Python.framework/Versions/3.6/bin/python3-config
  /Library/Frameworks/Python.framework/Versions/3.6/bin/python3.6-config
  /opt/perl5/perls/perl-5.32.0/bin/ppkg-config

Warning: You have unlinked kegs in your Cellar.
Leaving kegs unlinked can lead to build-trouble and cause brews that depend on
those kegs to fail to run properly once built. Run `brew link` on these:
  sudokut
  speedtest
  cmake
  perl
  aplus
  clamtk
  docker-compose
  libudfread
<EMAIL_ADDRESS>  berkeley-db`

Please paste the output of brew link libudfread --debug, thanks!

Error: No such keg: /usr/local/Cellar/udfread /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/delegate.rb:85:in call'
/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/delegate.rb:85:in method_missing' /usr/local/Homebrew/Library/Homebrew/cli/named_args.rb:209:in resolve_keg'
/usr/local/Homebrew/Library/Homebrew/cli/named_args.rb:149:in block in to_kegs' /usr/local/Homebrew/Library/Homebrew/cli/named_args.rb:148:in map'
/usr/local/Homebrew/Library/Homebrew/cli/named_args.rb:148:in to_kegs' /usr/local/Homebrew/Library/Homebrew/cmd/link.rb:42:in link'
/usr/local/Homebrew/Library/Homebrew/brew.rb:120:in <main>'

Did you name your formula udfread? Because that's what Homebrew derives the various paths from.

Just to clarify, some change between 2.5.7-36-gac8cc16 and 2.5.8-62 seems to break linking kegs without formulae. The OP had an issue due to name mismatch, but all the actual issues were reproduced on 2 separate 10.14 macos laptops that are otherwise a-ok.
That is, after recent brew update,  brew link started failing for me, but not because of wrong name like udvread (above). I see two modes of failure: (1.) cannot unlink then relink existing diy links, and (2.) cannot create links for new diy formula in most cases, even with formula name correctly matching name of folder in /usr/local/Cellar. And brew doctor is warning incorrectly about "deleted formulae" that are not formula at all, but diy links.
Below I details the steps I can use to reproduce issues on macos 10.14.
(Symtom 1.) Previously linked diys cannot be unlinked then re-linked after brew update to 2.5.8-62-g084f2e7
bash-5.0$ brew link docopts
Linking /usr/local/Cellar/docopts/0.6.3... 1 symlinks created
bash-5.0$ brew config
HOMEBREW_VERSION: 2.5.7-36-gac8cc16
ORIGIN: https://github.com/Homebrew/brew
HEAD: ac8cc16b9e7a7e8da9d1a180ff549a28dd96b64a
Last commit: 5 days ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core.git
Core tap HEAD: da7c16e4a10a88492284cd08b025d73d029c83cb
Core tap last commit: 4 days ago
Core tap branch: master
HOMEBREW_PREFIX: /usr/local
HOMEBREW_CASK_OPTS: []
HOMEBREW_MAKE_JOBS: 4
HOMEBREW_NO_INSTALL_CLEANUP: set
Homebrew Ruby: 2.6.3 => /usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.6.3_2/bin/ruby
CPU: quad-core 64-bit ivybridge
Clang: 10.0 build 1001
Git: 2.27.0 => /usr/local/bin/git
Curl: 7.54.0 => curl
Java: 1.8.0_242
macOS: 10.14.6-x86_64
CLT: <IP_ADDRESS>.1.1562985497
Xcode: N/A
XQuartz: 2.7.11 => /opt/X11
bash-5.0$ brew unlink docopts && brew link docopts
Unlinking /usr/local/Cellar/docopts/0.6.3... 1 symlinks removed
Linking /usr/local/Cellar/docopts/0.6.3... 1 symlinks created. ### NO PROBLEM with 2.5.7-36-gac8cc16
bash-5.0$ brew update
Updated Homebrew from ac8cc16b9 to 084f2e75e.
Updated 3 taps (homebrew/core, homebrew/cask and homebrew/services).
==> New Formulae
cfn-flip            k3sup               phive               vulture
fizz                ox                  tree-sitter
...
==> Deleted Casks
aspera-connect
bash-5.0$ brew unlink docopts && brew link docopts
Unlinking /usr/local/Cellar/docopts/0.6.3... 1 symlinks removed
Error: No available formula with the name "docopts"    ### NEW PROBLEM AFTER  `brew update` to 2.5.8-62
bash-5.0$ brew config
HOMEBREW_VERSION: 2.5.8-62-g084f2e7

Also after update, brew doctor issues incorrect warnings for "deleted formula": what the doctor now says are deleted formulae,  are, in reality  almost all my own diy-names, such as docopts
$ brew doctor
Warning: Some installed formulae were deleted!  #  better word(s) than "deleted": "retired" or "no longer available"
You should find replacements for the following formulae:
docopts
$ docopts --version
docopts v0.6.3-rc2 commit 651ff2c built at 2020-04-05T05:13:14Z
$ brew link docopts
Warning: Already linked: /usr/local/Cellar/docopts/0.6.3
To relink:
  brew unlink docopts && brew link docopts
$ brew unlink docopts && brew link docopts
Unlinking /usr/local/Cellar/docopts/0.6.3... 1 symlinks removed
Error: No available formula with the name "docopts"

(2.) Newly created diy software cannot be linked unless folder in /usr/local/Cellar is same name as existing formula.
At least as of today,2020-Nov-03, brew link usually fails with any new diy programs. For example, belowbrew link fails for diy-name hellognu , but brew link works  if I use diy-name hello, which is also the name of a formula that is not installed.
$ brew diy --name hellognu --version=2.10
$ cd /tmp && wget https://ftp.gnu.org/gnu/hello/hello-2.10.tar.gz
Error: Couldn't determine build system. You can manually put files into /usr/local/Cellar/hellognu/2.10
$ mkdir  /usr/local/Cellar/hellognu/
$ cd /usr/local/Cellar/hellognu/
$ tar zxf  /tmp/hello-2.10.tar.gz
$ mv hello-2.10.  2.10
$ cd 2.10
$ pwd
/usr/local/Cellar/hellognu/2.10
$ ./configure  --prefix="/usr/local/Cellar/hellognu/2.10"  && make && make install
$ /usr/local/Cellar/hellognu/2.10/bin/hello --version
hello (GNU Hello) 2.10
$ $ brew link gnuhello
Error: No such keg: /usr/local/Cellar/gnuhello

But below brew link hello works if I use create Cellar/hello which is name of existing formula hello (but brew install hello was never done.
$ which hello # not installed
$ brew diy --name="hello" --version=2.10
--prefix=/usr/local/Cellar/hello/2.10
$ mkdir  /usr/local/Cellar/hello/
$ cd /usr/local/Cellar/hello/
$ tar zxf  /tmp/hello-2.10.tar.gz
$ mv hello-2.10.  2.10
$ cd 2.10
$ pwd
/usr/local/Cellar/hello/2.10
$ ./configure  --prefix="/usr/local/Cellar/hello/2.10"  && make && make install
$ /usr/local/Cellar/hello/2.10/bin/hello --version
hello (GNU Hello) 2.10
$ brew link hello
$ brew link hello
Linking /usr/local/Cellar/hello/2.10... 168 symlinks created

I manually applied Mike's commit to link.rb to fix $9036 at https://github.com/MikeMcQuaid/brew/commit/1f971ba2a0575523300df2f4e63c03769ed3ad2a
And it worked for me! That is, using his  /usr/local/Homebrew/Library/Homebrew/cmd/link.rb I think it fixed all the issues I reported


Irregular billing on mainnet

There is some unexpected behavior for mainnet billing. Specifically, there is a repetitive pattern  every 3 hours as can be seen below. Additionally there are a couple of big spikes which can't really be explained,

(1 bar = 1 hour, 96 hour timespan)

Additional zoom on last 24 hours:

The spikes are because of contracts that were put to grace and funded again at that time.
The other irregularities are because 2 validators on mainnet are not running the offchain worker. We will get that sorted today

Initiated removal of validator: 5Eo9uD6bpcQq9LYCHegnM2ZkbFRiM5QW9hv6wU97gyj9NKAn
Let's purge those session keys and reinsert proper keys and start adding the validator again.

Validator 5Eo9uD6bpcQq9LYCHegnM2ZkbFRiM5QW9hv6wU97gyj9NKAn is creating blocks and billing contracts again.

Closing, Since blocker Validator has been removed and no more instance of the issue is being reporduiced


Configure Cognito verification email custom sender address

I have setted up auth using amplify cli. When users signs up, they receive verification code email from no-reply@verification email address.
I want to change that and use my gmail address. I tried searching on GitHub issues and on internet how to do that using amplify cli. The only method I could find was using AWS Cognito Userpool console. I heard that any manual changes made to console gets lost after next amplify push operation.
Will console method work or any other method using amplify cli is possible?
Thanks!

Thanks for the answer!

I could not set up custom sender email for default Cognito configuration.
My cognito user pool region : ap-south-1
My userpool Message customization section's SES Region sub-section shows these region

I went to SES and added two new email addresses for ap-south-1 region and US East Virginia (1st one in above image) regions.
Now in my userpool Message customization section's From Email address ARN sub-section , I can see two entries in dropdown - Default and other my email address added in US East Virginia (1st one in SES region , not the one added in ap-south-1 which is userpool region)
Below i kept it default :

Now in my userpool Message customization section's From Email address (not Email address ARN) sub-section :

Email address ARN - default
Email address - blank
When saved, still getting verification email from no-reply@verification
Email address ARN - one added in SES US East Viriginia
Email address - blank
When saved, still getting verification email from no-reply@verification
Email address ARN - default OR one added in SES (both selected one-by-one)
Email address - one added in SES
When saved,  getting error

Basically I want to use default Cognito config (50 emails per day) but custom sender email which I am not getting to work
Please help!
I have used aws amplify cli to add auth by amplify add auth command

H @mir1198yusuf - apologies that this is giving you some trouble.
In the top portion - Select the region and then enter then enter the email that is verified via SES.

Then, select the Yes, use Amazon SES option

Hello @siegerts  -
I followed the above steps and it really helped.
But the issue I am facing now is that after signing, I am receiving following error -

Cognito received the following error from Amazon SES when attempting to send email : Email address is not verified. The following identities failed the check in region US-EAST-1 : <new_signed_up_email>

My Cognito userpool is of ap-south-1 and SES verified email (custom sender email for Cognito) is of us-east-1
Do I have to move SES out of sandbox? or any other

Hi @mir1198yusuf , Yes, you'll need to verify the email and move it out of the sandbox.  The full details of this process are documented @ https://docs.aws.amazon.com/cognito/latest/developerguide/user-pool-email.html .
Also, from the docs -

Available regions for Amazon Cognito are us-east-1, us-east-2, us-west-2, eu-west-1, eu-west-2, eu-central-1, ap-northeast-1, ap-northeast-2, ap-south-1, ap-southeast-1, ap-southeast-2, and ca-central-1. Available Amazon SES regions are: us-east-1, us-west-2, eu-west-1.

Hope that this helps!

After moving out of sandbox for region us-east-1, it is working perfectly. Thanks!
Sidenote - Add this to Amplify Auth doc please

@siegerts -
Sorry to bother you again for unrelated thing. just one last thing, the verification code emails are going to Promotion folder.
I know this is not the right place to ask for this.
Can you redirect me where to file an issue on GitHub/etc ?
Any additional configs needed ?
Security details of email is
Signed by - amazonses.com


How do I run this network with my own data?

Dear all, thank you for publishing this work.
I am curious how to use this network on my own data. For example, say I have a PCD for chairs, I do not have the ground truth for this. I just want to run inference script to give me a list of segmented objects.
How would be the best way to progress using your network? Any help on this would be very much appreciated.
Thank you for the contribution !!

You can forward your own data using the model finetuned on ScanNet or S3DIS. The predictions correspond to 20 categories in ScanNet or 13 categories in S3DIS.


ResizeContainer() should be able to support resizing CPU as well

Currently only allows re-sizing memory

Commit 329b574 fixes this issue


Nextjs 13.4

Hello,
I saw the thread for Nextjs SWC however I am not that deep in to the TS word and don't understand what this means.
All I want to know is if I can use your library for server side type checking in a standard NextJS 13.4.
Thank you
EDIT: I searched more and found https://nextjs.org/docs/architecture/nextjs-compiler -> so nextjs 13.4 uses SWC and based on a previous thread it is not possible to use typia with NextJS 13.4. Thank you.

Not possible with SWC


Remove empty XML tag.

No offense, but did you just really go over the XML source of a SVG file with something that is not a graphics program? ;)

Going to just leave this. It isn't hurting anything to have a metadata tag in there. Thanks for your interest, though!


YouTube: filter out Shorts (But only for Subscriptions Page: https://www.youtube.com/feed/subscriptions)

What filter does this issue target?
https://letsblock.it/filters/youtube-shorts
What changes should be made?
Youtube Subscriptions Page is the only place where Shorts become unbearable. Youtube fixed the channel's page by creating different tabs for shorts, videos and livestreams. But Subscriptions Page is the only place where they didn't implement this.
Is it possible to block shorts only from Subscriptions Page: https://www.youtube.com/feed/subscriptions?
I want shorts to be available everywhere on youtube except the Subscriptions Page.
Any help would be appreciated. 😇😇
Did you write a uBlock filter solving this?
No response
Additional notes
No response

Hello @fossist, thanks for opening this issue!
Customization is at the core of the project's goals, but it comes with a significant maintenance overhead: make sure that rules can work separately and together, test the different combinations on every PR, ask for configuration when handling issues...
The youtube-shorts template is the most frequently updated template because of website changes, and the frequency of changes does not seem to slow down (right now the navbar rules are broken for some users, due to an ongoing design change). Once the situation settles for several months, I'll be happy to revisit that decision, but for now I prefer to avoid adding any configuration to this template.

@fossist try
www.youtube.com#?#ytd-browse[page-subtype="subscriptions"] .ytd-thumbnail[href^="/shorts/"]:upward(ytd-grid-video-renderer)
www.youtube.com#?#ytd-browse[page-subtype="subscriptions"] ytd-video-renderer:has(.ytd-thumbnail[href^="/shorts/"]):upward(ytd-item-section-renderer).
For those rules works.


GLES2: 3d renders all objects black

Godot version:

Current master: 4c169352a61fc97eff65867017910c94a6dbfde3
OS/device including version:
Windows 10 / Nvidia

Issue description:

Using GLES2 renderer in 3d all objects are rendered black (including the grid and handles)

Steps to reproduce:
Create an empty gles2 project.
Minimal reproduction project:

gles2_black_issue.zip

Can you enable the system console (Editor menu) and paste any error reported?

Yes; but I think there is no error reported:

Looks to be related to https://github.com/godotengine/godot/issues/31019
Delete the *.glsl.gen files in drivers/gles2/shaders and then rebuild. This is likely caused by shaders not rebuilding when you rebuilt from master.

yes; this was the issue here; Thx a lot and sorry for not seeing this.

No problem! It would be hard for you to know about that specific issue. :)

How i find these files?

@BaielComAcentoAgudo clayjohn's advice targeted to people compiling Godot from source. If you've downloaded a Godot binary, then you're not compiling Godot from source.
There's already a bug report about meshes turning black on old AMD/Intel graphics cards: https://github.com/godotengine/godot/issues/30330


[FR] Allow custom time with /restrict

That way we could also set the period we think it's best for the new user only use text in his/her message. For example, it would be nice if we could extent to a week, enough time to analyze if he/she went to spam the group or seek real help and so on.

24 hours is enough, 99.9% of spammers on Telegram are banned within 24 hours by Telegram.


Swap comment servers without going to settings page #7365

Fixes
Issue Number: https://github.com/lbryio/lbry-desktop/issues/7365

What is the current behavior?
What is the new behavior?
Other information

PR Checklist

Toggle...
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[ ] Documentation changes
[ ] Other - Please describe:

Please check all that apply to this PR using "x":

[x] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[ ] I added a line describing my change to CHANGELOG.md
[x] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

Good progress. Besides fixing styling, this should be at the top next to the refresh button.
The refresh button reloads all comments. The server selector changes the server for all comments.
Below, the comment field adds one comment.
Also, don't forget to make the selector only show up when the number of servers is > 1.

@ByronEricPerez What do you think if we could have option in app to integrate using multiple comments servers into single feed.
Aka at the same time user could see Odysee comments and also other comments server, or user can decide to limit to selected comments servers or just use single one.

Hi @jessopb how are you, I have already corrected the location, in addition to the fact that it only shows if there are 2 or more commentServer, I just need to add that it shows the comments. And finally some css.
Hi @kodxana , I think the idea is good, it would also be more practical to have the option of being able to show them all in one.

Hi @jessopb , I'm having trouble seeing the comments on all the videos.
The btn moved it to the correct component, it already synchronizes, I need to align it a little more. What do you think?

Although I comment I get an error asking me to restart, but I have that comment in my history.
I also don't see comments on any video. This error started appearing yesterday.

Fixed why it didn't show comments  , the error was in index.js

Put the refresh button last, and it's a bit too wide, I think.

@kodxana would you like to test this branch?


Adds ember-suave inline filters + Fixes more linting errors

https://github.com/samselikoff/ember-cli-mirage/issues/657#issuecomment-204349715
I think this solves https://github.com/samselikoff/ember-cli-mirage/issues/671#issue-144915155 as well.

Awesome, thanks!
What's the .jscsfilters file for, global filters? Could we keep it all file-specific? Feels like that's too easy to ignore

I just saw your comment in the other thread. Is it possible to just use comments at the top of files when we need to override the default settings?

the .jscsfilter is just a plain text file listing all the filters. I added it because thought  it is hard to track down all the filters if someone wants to review them. But of course I can remove it.
Also updated filters regarding to changes in https://github.com/samselikoff/ember-cli-mirage/commit/13405df8770e496529e19985e9c6c46aed4c82ba
Shall I rebase into 1 commit, or keep commits separeted?

One more concern: my npm test results
1..1094
tests 1094
pass 1094
skip 0
fail 0

while Travis build #1057 results
1..1030
tests 1030
pass  1030
skip  0
fail  0

So it seems in Travis still does not run all the tests.

Sorry if I wasn't clear. I want requireParenthesesAroundArrowParam to be false globally for the project, because I don't like that rule. But if we have individual files that need to omit a certain rule for some reason, I'd rather do it in the file than in a global file.
Are you saying you cannot have a global change and also apply file-specific overrides? That would surprise me.

I am saying you can define global rule in the .jscrc, but it will break your tests because https://github.com/DockYard/ember-suave/issues/94. That is why I removed all global rules. And wee need to apply them on file basis as inline filter.

Ok - so are you happy with this PR? I've been heads down on some other issues so I don't have time to trace through all the references here.

Yes, I think this is the best we can do in light of the ember-suave issue.

:+1: thank you


As of vscode 1.88.1 squiggles are offset too high.

Just started happening as of the latest version of vscode.
I've tried other themes and couldn't replicate the problem, also reverting to 1.87.2 fixes the issue.
Here's how it looks with the default dark theme in 1.88.1

Looks like https://github.com/microsoft/vscode/issues/209614


python3 --version Traceback (most recent call last):   File "", line 1, in  NameError: name 'python3' is not defined

python3 --version
Traceback (most recent call last):
File "", line 1, in
NameError: name 'python3' is not defined

Can you provide additional information?


Added date to release page

I just added the date to the release page, using the same format from the archives page.
Checklist

[ ] npm test passes
[ ] documentation is changed or added (if applicable)
[ ] permission has been obtained to add new logo (if applicable)
[x] contribution guidelines followed here

You've added a second release object which is breaking the linter.
We already store the release date in https://github.com/adoptium/adoptium.net/blob/main/src/js/releases.js#L84. In principle it should be a case of getting the handlebars to show the date

Okay thank you. Please how can I fix this issue?
On Wed, 13 Oct 2021 at 9:30 AM, George Adams @.***>
wrote:

You've added a second release object which is breaking the linter.
We already store the release date in
https://github.com/adoptium/adoptium.net/blob/main/src/js/releases.js#L84.
In principle it should be a case of getting the handlebars to show the date
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/adoptium/adoptium.net/pull/204#issuecomment-942060960,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHMKITMRNPAAVXAFZTBUB4LUGU7TJANCNFSM5F3P6K5Q
.
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.

--
Nneamaka Chalokwu


FastMatrix: Add tests covering its interface

I wrote these tests a few weeks ago in preparation for an optimization effort for our FastMatrix implementation. Maybe it would be nice to just merged them now since it looks like there are PRs coming in that touch this area
This PR contains:

[ ] Changelog
[ ] Documentation
[x] Tests

@kusma Would prefer to have these on master before https://github.com/fusetools/fuselibs-public/pull/148 is merged. I don't understand why the Travis step fails

@Duckers: It fails because of https://github.com/fusetools/uno/issues/1176


Improve perf by caching generators

Small change, order of magintude improvement. Tested on 185 page blog:
Pre-change - Generation time: 00:04:59.3250690
Post-change - Generation time: 00:00:19.2490206

Shouldn’t it also invalidate cache when the generator file is changed?

Oh yeah, I was only thinking about the build mode where it runs and exits. I'll take another look at the the watch mode.

Support cache invalidation in watch mode now.

We probably could go even further than this and invalidate cache only in some particular cases (invalidate all when config or any loader has changed, invalidate single entry when the generator has changed. ). But even this is great, thanks a lot!

I thought about that, and agree it would be a better solution I didn't do it for two reasons:

This bug and knowing where the route directory is makes a little tricky to be sure what the cache key related to the file would be: https://github.com/ionide/Fornax/issues/63
I don't think it would impact perf that much as all the files are regenerated anyway

Next logical step for better perf, for watch mode at least, is calculating which files have changed and just regenerating them.


How to make it deleting downloaded torrent files, after closing mpv?

Not something i will add. see #8 for workaround

See: https://github.com/mrxdst/webtorrent-mpv-hook/issues/1#issuecomment-1434562324


Add second E2E gateway failover scenario

Two gateway nodes configured with 1 submariner engine replica.

This change is

There are arguably too many implementation details in the test here, but it depends on the implementation of the engine pod anyway, so...

I assume you specifically mean waiting for a new pod and Endpoint instance. I agree that makes it a bit whitebox but I did that to eliminate possible timing issue where, after deleting the pod, we actually run the connectivity test against the previous engine instance before k8s has a chance to react to stop the process/container etc. Perhaps I don't need to check for new  a Endpoint instance but I mainly did that as a sanity check that a new engine instance is running. It also gives it a bit more of a cushion to avoid premature timeout in the connectivity test.

I assume you specifically mean waiting for a new pod and Endpoint instance. I agree that makes it a bit whitebox but I did that to eliminate possible timing issue where, after deleting the pod, we actually run the connectivity test against the previous engine instance before k8s has a chance to react to stop the process/container etc. Perhaps I don't need to check for new a Endpoint instance but I mainly did that as a sanity check that a new engine instance is running. It also gives it a bit more of a cushion to avoid premature timeout in the connectivity test.

Right, that’s what I was referring to, but I don’t think it’s much of a problem really. However the fact that it’s useful for timing doesn’t appear in the code; it might be worth adding a comment to that effect (so that in future, if the details change and someone fixes the test, they don’t spend ages figuring out what the point was).

I assume you specifically mean waiting for a new pod and Endpoint instance. I agree that makes it a bit whitebox but I did that to eliminate possible timing issue where, after deleting the pod, we actually run the connectivity test against the previous engine instance before k8s has a chance to react to stop the process/container etc. Perhaps I don't need to check for new a Endpoint instance but I mainly did that as a sanity check that a new engine instance is running. It also gives it a bit more of a cushion to avoid premature timeout in the connectivity test.

Right, that’s what I was referring to, but I don’t think it’s much of a problem really. However the fact that it’s useful for timing doesn’t appear in the code; it might be worth adding a comment to that effect (so that in future, if the details change and someone fixes the test, they don’t spend ages figuring out what the point was).

Added comments and rebased.


Cachix caches aren't added to the configuration

I'm not sure if I'm missing something, but I can't seem to find anywhere where cachix is included in the nixos configurations.
Expected Behavior
Building a host with devos should have the cachix binary caches included.
Current Behavior
Cachix is available in the cachix folder, but isn't wired up to be a part of the host.
Possible Solution
import cachix.nix somewhere in hosts/default.nix
Context
This could be intentional, I'm not sure but I can't find a reason for why it was removed.

must've been removed by mistake, definitely not intentional.


Can u please update ?

please update Kits and Uconomy
or tell me how to update it
I really need these 2 plugins
I already try to update myself for a few days but always not working
please help me
thanks

Yeah its not working for me either


Fix xbmc.translatePath not found in LE11

Kodi 20 (Nexus) no longer supports xbmc.translatepath.  Use xbmcvfs.translatepath instead.

This works well, I just tested it on RPi4 Libreelec 11 (specifically LibreELEC-RPi4.arm-10.95.0.img.gz)

Hi, is this a breaking change or does it still work with LE10? Thanks

Hi, is this a breaking change or does it still work with LE10? Thanks

It should still work on LE10

My understanding is that xbmcvfs is there since v19 (Matrix)
https://romanvm.github.io/Kodistubs/_autosummary/xbmcvfs.html
So it may not be backwards compatible for the old v18 (Leia)

Indeed, on LE 9 stick with the old version.


Allow the Creation of Configuration Extensions

This PR needs the PR https://github.com/willdurand/Hateoas/pull/234 to be merged.

So there is 2.10.0 now: https://github.com/willdurand/Hateoas/releases/tag/2.10.0.

Is it possible to drop PHP 5.4 support? Otherwise I'd have to fix a few syntax errors...
#55

@willdurand: Ping

@willdurand: Would you be able to tag the newest master?


build.sh: remove fetching dependencies

We're vendoring the dependencies now, there's no need to fetch them.

Tested with go1.{4,5,6}. LFAD.


Enable feature gate CSIInlineVolume for kube-controller-manager

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespaces from that line:
/kind api-change
/kind bug
/kind cleanup
/kind design
/kind documentation
/kind failing-test
/kind feature
/kind flake

What this PR does / why we need it:
Enable feature gate CSIInlineVolume for kube-controller-manager.
/kind failing-test
Which issue(s) this PR fixes:

Fixes #130
Special notes for your reviewer:
The other two feature flags are already enabled in aks-engine, we should preserve them too.
Release note:
NONE

/test pull-cloud-provider-azure-e2e

/test pull-cloud-provider-azure-e2e


Error when players connect

Error running system event handling function for resource esplugin_mysql: citizen:/scripting/lua/scheduler.lua:41: Failed to execute thread: citizen:/scripting/lua/scheduler.lua:762: No such export mysql_fetch_all in resource mysql-async
stack traceback:
[C]: in function 'error'
citizen:/scripting/lua/scheduler.lua:762: in metamethod '__index'
@mysql-async/lib/MySQL.lua:143: in field 'fetchAll'
@esplugin_mysql/server.lua:5: in upvalue 'handler'
citizen:/scripting/lua/scheduler.lua:219: in function citizen:/scripting/lua/scheduler.lua:218
stack traceback:
[C]: in function 'error'
citizen:/scripting/lua/scheduler.lua:41: in field 'CreateThreadNow'
citizen:/scripting/lua/scheduler.lua:218: in function citizen:/scripting/lua/scheduler.lua:182
Pelayori: sdsdsdsd
Sending heartbeat to live-internal.fivem.net:30110
Error resuming coroutine: @es_extended/server/functions.lua:122: attempt to index a nil value (global 'Async')
stack traceback:
@es_extended/server/functions.lua:122: in field 'SavePlayers'
@es_extended/server/functions.lua:133: in function 'saveData'

Delete your server's cache and try again.

SCRIPT ERROR: citizen:/scripting/lua/scheduler.lua:890: No such export mysql_fetch_all in resource mysql-async

@keketiger i got this error too, is there anyone fixed this?

i solve this problem,
i don't know exact cause of this problem
if you use fxserver creator, don't use it
just follow this doc ( https://docs.fivem.net/server-manual/setting-up-a-server/ )

@SungJJinKang Personally I just reinstall fivem-mysql-async and the problem was solved.

i have same problem..


feat: Integrate LitVar into Seqvar page (#421)

Cannot continue working, since the LitVar API's are not reachable.

Closing the stale PR


Different setting per folder

I have a project which consists of many submodules and each of them has its own JSHint and JSCS config files. In Sublime I want to have all these submodules as one projects (because they are rather small).
Is it possible to configure SublimeLinter3 so it uses different linter settings per folder? Or is it more a question about Sublime's than SublimeLinter's capabilities?
If I didn't miss anything and this is indeed impossible to achieve now, then could such feature be added to SublimeLinter or it doesn't seem to be feasible?

A linter executable will look for it's configuration starting from the file it is linting. So, this should "just work". If not, that's up to the linter, not SL.


Debugger not breaking at breakpoints with Jest in monorepo

VSCode Version: 1.43.2
OS Version: macOS Catalina 10.15.3
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Darwin x64 19.3.0

Steps to Reproduce:

Create file structure

think
|- mobile
  |- package.json
  |- __tests__
    |- placeholder.js
* `cd mobile && yarn add -D jest-expo`
* File > Open Workspace > `think/`
* Add launch.json config
```json
    {
      "type": "node",
      "request": "launch",
      "name": "Jest All",
      "program": "${workspaceFolder}/mobile/node_modules/.bin/jest",
      "args": ["--runInBand", "--config", "jest.config.local.js"],
      "console": "integratedTerminal",
      "internalConsoleOptions": "neverOpen",
      "disableOptimisticBPs": true,
      "cwd": "${workspaceFolder}/mobile"
    },

Add Expo's recommended "jest" config to think/mobile/package.json:

  "jest": {
    "preset": "jest-expo",
    "verbose": true,
    "transformIgnorePatterns": [
      "node_modules/(?!(jest-)?react-native|react-clone-referenced-element|@react-native-community|expo(nent)?|@expo(nent)?/.*|react-navigation|@react-navigation/.*|@unimodules/.*|unimodules|sentry-expo|native-base)"
    ]
  },

Add placeholder test in think/mobile/__tests__/placeholder.js with debugger statement and breakpoint set on next line:

describe('placeholder', () => {
  it('is a placeholder', () => {
    debugger
    expect(true).toBe(true)
  })
})

See output but breakpoint doesn't hit:

~/S/t/mobile>  cd /Users/gsquare567/Sites/think/mobile ; /usr/local/bin/node --inspect-brk=16004 node_modules/.bin/jest --runInBand --config jest.config.local.js
Debugger listening on ws://<IP_ADDRESS>:16004/1722fd72-9574-4084-be57-7dfcffd5030d
For help, see: https://nodejs.org/en/docs/inspector
Debugger attached.
Debugger attached.
 PASS  __tests__/placeholder.js
  placeholder
    ✓ is a placeholder (2ms)

Loading mobile babel.config.js
Test Suites: 1 passed, 1 total
Tests:       1 passed, 1 total
Snapshots:   0 total
Time:        1.344s
Ran all test suites.
Waiting for the debugger to disconnect...
fish: '/usr/local/bin/node --inspect-b…' terminated by signal SIGKILL (Forced quit)

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

@garrett-gottlieb have you tried the preview of the new JavaScript debugger?
Please see https://code.visualstudio.com/updates/v1_43#_new-javascript-debugger
Could you try if it works better for you?


Adds a new EventQueueFactoryAwareInterface for plugins

Plugins can now use the event queue factory to resolve connections to event queue instances, for processing asynchronous events without having to store event queue instances or play object ping-pong.
public function handleMyAsynchronousEvent(ConnectionInterface $connection, $message)
{
  $queue = $this->getEventQueueFactory()->getEventQueue($connection);
  $queue->ircPrivmsg($this->myOutputChannel, $message);
}

Coverage decreased (-0.6%) to 96.78% when pulling 5d218cbe073b5a8b89e2e171b75e7da06057addb on Renegade334:devel-eventqueue-aware into 2ed099cb73f1d49d2ca20e5f71a3756bcde9e319 on phergie:master.

Coverage increased (+0.69%) to 98.07% when pulling e1bd9061af1d1318f53f5974e0464cc030d0e9e1 on Renegade334:devel-eventqueue-aware into 2ed099cb73f1d49d2ca20e5f71a3756bcde9e319 on phergie:master.

Coverage increased (+0.69%) to 98.07% when pulling e1bd9061af1d1318f53f5974e0464cc030d0e9e1 on Renegade334:devel-eventqueue-aware into 2ed099cb73f1d49d2ca20e5f71a3756bcde9e319 on phergie:master.

Very nice work. We'll need to add this to the wiki and update any plugins that currently maintain their own storage of event queues, such as FeedTicker.

Hm, one minor issue that I missed and Scrutinizer picked up: setEventQueueFactory() uses $queueFactory in the method declaration and $factory in the docblock.
public function setEventQueueFactory(EventQueueFactoryInterface $queueFactory)

* @param \Phergie\Irc\Bot\React\EventQueueFactoryInterface $factory

:-/
Can you ret-con it yourself?

Yup, I'll get it. Thanks.

God bless GitHub Mobile.


Great application, I have some suggestions & issues

Suggestions
1- we need the ability to set default application for the file extension in properties.
2- files listing in the left panel like this concept, Source

3- we need a replace all button when pasting files already exist like the normal file explorer.
4- in the default file explorer when I sort files/folders by name first comes the folders and then the files but in files,
it's not, so I do love to get the option to get the same in files too.
Issues
one issue actually,
when I right click on any file/folder and click on properties and I close the properties tab immediately the app gets crashed..
English is not my native language..
And in the end I want to thank everyone worked on this application 😀.

https://files-community.github.io/docs/#/articles/replace-file-explorer.
We don't have plans for a tree view in the sidebar, however, you can subscribe to the feature request for a tree view layout #1928.
This was implemented in main and will be available with v2.
Do you have the option to list and sort directories with files turned on? It sounds like this could be why you see the wrong behavior.

Thank you for all your feedback!

in the first suggestions i didnt mean to replace the default file explorer
i meant a button like this button in the file explorer properties tab

and yes i have this option turned on

when I right click on any file/folder and click on properties and I close the properties tab immediately the app gets crashed..
it seems this problem only happens when i have this option in experimental turned on

in the first suggestions i didnt mean to replace the default file explorer
i meant a button like this button in the file explorer properties tab

I misread that, we are working on this option 🙂

it seems this problem only happens when i have this option in experimental turned on

This setting is being removed in v2 so it will longer be an issue.


Lack of documentation of fields in the Properties section

The document lacks the description of the field in the Properties section which are significant and contain the main content and information for the signin.

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

ID: a6abfd33-5740-865b-24a4-542a989bc399
Version Independent ID: 0139ee02-5fcd-1d2d-800e-d440131348ec
Content: Sign-in log schema in Azure Monitor
Content Source: articles/active-directory/reports-monitoring/reference-azure-monitor-sign-ins-log-schema.md
Service: active-directory
Sub-service: report-monitor
GitHub Login: @MarkusVi
Microsoft Alias: markvi

@xomemayol
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

The fields are documented here.
#please-close


AMBARI-24881 Add Service Request JSON (benyoka)

What changes were proposed in this pull request?
Implemented the JSON object for Add Service Request. Also ported the Configurable class from the 3.0 branch to support JSON serialization/deserialization for Configuration classes.
How was this patch tested?

Wrote new unit test for the AddServiceRequest class
Ported and adapted ConfigurableTest from the 3.0 branch.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/4509/
Test FAILed.
Test FAILured.

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/4511/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/4518/
Test PASSed.


feat(telemetry): collect SPDK node disk type

https://github.com/longhorn/longhorn/issues/6033

The change seems incorrect.
What we should do is:

Longhorn disk type: block (v2), filesystem (v1)
Node disk types: SSD, HDD (No SPDK, SPDK is just an upper layer, but the underlying device type is still the same)
Volume engines: v2, v1

cc @longhorn/dev-data-plane

The change seems incorrect.
What we should do is:

Longhorn disk type: block (v2), filesystem (v1)
Node disk types: SSD, HDD (No SPDK, SPDK is just an upper layer, but the underlying device type is still the same)
Volume engines: v2, v1

cc @longhorn/dev-data-plane

I see. Thank you for explaining.
We use the rotational file to determine the device type. So it appears that this also applies to the SPDK disks. Therefore, we have two options:

Either prefixing SPDK- to the SSD, HDD, etc., to differentiate the one that's created for SPDK, or
Close the original issue since there is no difference to the underlying devices.

WDYT?

The change seems incorrect.
What we should do is:

Longhorn disk type: block (v2), filesystem (v1)
Node disk types: SSD, HDD (No SPDK, SPDK is just an upper layer, but the underlying device type is still the same)
Volume engines: v2, v1

cc @longhorn/dev-data-plane

I see. Thank you for explaining.
We use the rotational file to determine the device type. So it appears that this also applies to the SPDK disks. Therefore, we have two options:

Either prefixing SPDK- to the SSD, HDD, etc., to differentiate the one that's created for SPDK, or
Close the original issue since there is no difference to the underlying devices.

WDYT?

After discussing with @innobead , we will implement the following:

Collect The number of Longhorn disk type at node level: block (v2), filesystem (v1)
Collect Volume engines at volume level: v2, v1

As for Node disk types: SSD, HDD, we will not do anything extra as the current implementation also applies to disks created for SPDK.

This PR LGTM
However, While walking through this code region, I doubt the old logic to calculate avgVolumeSize and avgVolumeActualSize:

We only add to the totalVolumeSize and  totalVolumeActualSize if the volume is v1: https://github.com/longhorn/longhorn-manager/blob/ae6b6aa99d0e2eb12fd0d868f5e67105bb6b685c/controller/setting_controller.go#L1570-L1571
However, we are dividing that total value to volumeCount which is the count of both v1 and v2: https://github.com/longhorn/longhorn-manager/blob/ae6b6aa99d0e2eb12fd0d868f5e67105bb6b685c/controller/setting_controller.go#L1634-L1642

This looks a bit off to me. Could we double check?


Merging more than two topics from quick moderation fails

https://www.elkarte.net/community/index.php?topic=5323
In this instance the error is triggered by  'notifications' => $this->_req->getPost('notifications', 'trim', ''), but notifications is a post array.  The HttpReq class does not currently deal with that fact when using the sanitation call in getPost or getQuery.
I think updatingfunction cleanValue() so it does a further validator setup will fix things
		// To the validator
		$this->_dataValidator->validation_rules(array());
		$this->_dataValidator->sanitation_rules(array($name => $sanitize));

		if (is_array($this->_param[$name]))
		{
			$this->_dataValidator->input_processing(array($name => 'array'));
		}

		$this->_dataValidator->validate($this->_param);

Since 1.1.5 is still open, should we add this back?


feat: DIA-940: Read Adala predictions back in once they are completed

Draft PR. needs to be rebased on top of DIA-939 and changes deduped. Still some TODOs and probable errors in the code because the testing setup isn't fully worked out.
Small db changes to support LSE PR, see that one for details.
Related PRs:

https://github.com/HumanSignal/Adala/pull/62 rework some Adala API endpoints for clarity, and statuses to match LS better
https://github.com/HumanSignal/label-studio-enterprise/pull/5391

will add the Prediction modelrun field in a sec, have to deconflict with the other updates to develop today

rebased on DIA-939 and deconflicted duplicate changes, but this is still NOT mergeable, because it includes conflicting changes to Prediction model that were reverted in d1021d643ff33e039505bae56999c68dbfabfecf on develop - need to figure out what the status of those changes is


FEAT string to JSON converter

Describe the solution you'd like
Add a converter using json.dumps() to make the string json formatted

Can you elaborate? What if the string isn't JSON?


GR12 - Uniswap x L2s

Prize Title
Uniswap x L2s
Prize Bounty
$5000 in UNI
Challenge Description
Uniswap v3 is currently deployed on two L2 scaling solutions (Aribitrum and Optimism) with likely third deploy to Polygon (if governance proposal passes). Even though lot of end users complain about high gas prices on L1 we still didn't see huge adoption of L2s.
From user feedback we got two most common reasons why they don't use L2s is lack of information and user experience of bridging funds. We would like to see how you tackle this issue.
Submission Requirements
Judging will happen once all submissions have been received. Please include the following:

Live demo or explainer (video is fine!)
Working code or readable document in a public github repo

The requirements for your submission must:

Be free and open source
Be impactful for Uniswap users
Be built during the hackathon
Be original
Have been fun to build 🤗

If you have any questions feel free to ping us on the discord or find us on twitter!

0x37a9caa906a07F9836dAfab2a650B0a0e5e4e0c0


App.hs refactor (3)

This PR is a bit more complicated.
Please review the other two i submitted ( #369 then #370 ) before reviewing this one so that the "diff" does not contain the code from the previous ones and it's easier to look at.
This PR should help with introducing the "Router" if it desired.

I looked at the generated diff here https://github.com/ruslantalpa/postgrest/commit/0c3545fa09bf6c297dc9a2dec43370503ec9ca3d and it's not nice (more confusion). I suggest looking directly at the source  (after looking a bit at the diff to get a sense what parts changed)

Looks good. I think after merging this with #368 we remove some of the intuition disconnect and end up making the app function much better.
It might be easier if I just cherry pick this commit into #368, would you like/mind that?

I am not sure i understand the  order of the things you suggest but my view is this:

Merge (1,2,3) with improvements if suggested
You create a fresh PR starting from master (without the previous commit attempts) that strictly contains the implementation of Router data type and 1-2 additional helper functions and see if there is improvement in readability of the "app" code added by the Router idea (no refactoring of other parts like error functions and such so that we can clearly see the impact and compare old style to new style)

One thing i did wrong in this PR is combine the refactoring of parseRequest + createStatemnet with the rewrite of the app function, the first part is a step in the right direction, the last part  (the app function, removing explicit GET and nesting of the core logic under ([table], _)) i am not 100% sure so if someone thinks that can be done in a better way i'll close this particular PR and open a separate one with just the parseRequest and createStatement

What I mean by ordering is that #368 and just the 3rd commit here might be better as one PR vs two.

The 3rd commit is about single routes in pg, you know my opinion there. The second commit (router) has value although i think it's better to implement it fresh rather then trying to rebase/merge

Sorry, I meant the 3rd commit in this PR 😊

@ruslantalpa let's break this into pieces like you say. This PR includes commits that were already merged so a clean PR would be good for that reason too (either that or rebasing your PR branch on new master).


Update dependencies & GH Actions

Update dependencies and GH Actions according to @dependabot.

The issue here is due to providers.optimade.org being down, which will be fixed in Materials-Consortia/providers#71. This can be merged as all pytest runs have passed.


do not call dispatcher for ND pilots

The call to the dispatcher to check the job status should only be for the "true pilot" jobs, that is the jobs where aCT fetches the job and pushes to the CE, but then the normal pilot workflow runs with staging, heartbeats etc. For the "NorduGrid" jobs where there is no proxy or network connection we shouldn't do the check of the status since the problem of rerunning the job doesn't happen. Sorry for not being clearer on this when we requested this feature.

Ok thanks David.


Attempt to synthesize a code section if non are available in order to suport ngen-generated PDBs

Based on suggestions I got in https://github.com/getsentry/pdb/issues/153 -- this PR attempts to synthesize a section if:

There is no section data
There is no OMAP From data
There is only one entry in the section map that has any executable ranges from the section contributions

I have no idea how correct this is, but it seems to work in my (currently limited) sample size. I can at least compare my resolution with Microsoft's for stack traces so I'll keep checking.
Without this, for these PDBs it's possible to get a list of the global symbols, but no address map translator can be created meaning that in practice it's not possible to do anything with those symbols.
Background here, from that linked issue:
I'm working to improve some profiling tools (samply specifically) that uses the pdb create under the hood. Part of what I need is being able to handle symbols for .NET, specifically symbols from Crossgen2-built Ready 2 Run assemblies. I think that's where things are coming from anyway, anyway -- things that end in .ni.pdb, I think written here with some comments about DiaSymReader and other: https://github.com/dotnet/runtime/blob/fc76b1cac3f02cc9729f6682d6850fd7982e9fe5/src/coreclr/tools/aot/ILCompiler.Diagnostics/PdbWriter.cs#L199
Here's an example of this type of PDB, from Microsoft's symbol server: dotnet.ni.dll Also just in case, this isn't a Portable PDB, it's a normal PDB, but I think written in a very limited way. It's just the symbol information.
When read by the pdb create, these pdbs show up as having no section information. Which means it can't get an address map, which means that I end up with no way of translating RVA addresses to symbols. Section contribution information is there though, e.g. here's dia2dump -x:
*** SECTION CONTRIBUTION

    RVA        Address       Size    Module
  00001000  0001:00000000  00275000  C:\Users\cloudtest\AppData\Local\Temp\5egk1fj3.cpa\dotnet.dll
  00276000  0002:00000000  00034000  C:\Users\cloudtest\AppData\Local\Temp\5egk1fj3.cpa\dotnet.dll
  002AA000  0003:00000000  00004000  C:\Users\cloudtest\AppData\Local\Temp\5egk1fj3.cpa\dotnet.dll

These section contributions map directly to the 3 sections in the actual code dotnet.dll. I have no idea where dia2dump is getting the RVA from above (especially the ones beyond the first section), as it's not in the section contrib information. I do see in PdbWriter.cs some places where sections are written, but I have no idea where that info is going!
If I parse it the section_map as a DbiSectionMap from syzygy I get:
DBISectionMapItem { flags: 13, section_type: 1, unknown_data_1: 0, section_number: 1, unknown_data_2:<PHONE_NUMBER>, rva_offset: 0, section_length: 2576384 }
DBISectionMapItem { flags: 13, section_type: 1, unknown_data_1: 0, section_number: 2, unknown_data_2:<PHONE_NUMBER>, rva_offset: 0, section_length: 212992 }
DBISectionMapItem { flags: 13, section_type: 1, unknown_data_1: 0, section_number: 3, unknown_data_2:<PHONE_NUMBER>, rva_offset: 0, section_length: 16384 }
DBISectionMapItem { flags: 8, section_type: 2, unknown_data_1: 0, section_number: 0, unknown_data_2:<PHONE_NUMBER>, rva_offset: 0, section_length:<PHONE_NUMBER> }

I may not be able to get to this any time soon -- I've left the role where this was important a few months ago and am working in a totally different area. Would need to context switch pretty hard :)


Mock Setup for API

Summary
As developer
I want to simulate certain situation with ease
So that I can build and test new features with ease.
Context
In the beginning, not all API feature will be implemented in the actual API. And spinning up the whole infrastructure may not be suitable for every situation.
For integration testing and for quick development iterations, it would be great to have a simple local mock API service.
Out of Scope
No response
Further links
No response
Acceptance Criteria
No response
Implementation Ideas
No response

I'd consider using a local Kind/K3d with "mock" CRDs applied.


plan to migrate away from wkhtmltopdf

Hi there. I was made aware by a person (who wants to move from WooCommerce to Sylius and build a SaaS-solution out of it) asking, why wkhtmltopdf is needed.
I told the person, that this should normally not be the case in modern and secure software.
In summarization I told them:

wkhtmltopdf is basically unmaintained for years (https://github.com/wkhtmltopdf/wkhtmltopdf, "This repository has been archived by the owner on Jan 2, 2023. It is now read-only.")
the latest release was 4 years ago (https://github.com/wkhtmltopdf/wkhtmltopdf/releases)
it contains a critical unpatched SSRF vulnerability with a CVSS score 9.8 / 10 (https://nvd.nist.gov/vuln/detail/CVE-2022-35583)
the upcoming Cyber Resilience Act requires that digital products (including software) shall not contain known vulnerabilities when released on the market (the SBOM documents would reveal that too)
PCI DSS certification may be at risk (https://security.stackexchange.com/a/140942)
(and a few more points regarding thirdparty software, that are often overlooked)

Hello,
I have a plan to use https://gotenberg.dev/ but we need to wait some weeks.


Remove broken support for macOS

It has been broken for quite some time (ex: https://github.com/cupy/cupy/issues/952#issuecomment-491635935), and CUDA has dropped support for Mac, so there's no reason to keep them around.

Jenkins, test this please

Jenkins CI test (for commit 7dc2c1f1f5fcd100d3256d9ddece0cbda8bcde69, target branch master) succeeded!

Jenkins, test this please

Jenkins CI test (for commit f475e6c725f20d9d06577cf94583a61a1d682bc6, target branch master) succeeded!

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

Hi,
My laptop is MBP2019.
Do you have a plan to support macOS?
If not, I will set it dual system.
Thx

@Photonico Unfortunately, ROCm (nor CUDA) supports Mac.


API delegering - 'Virksomheter det tidligere er delegert til:' when there are no organizations

Start API delegering
User should not have delegated API to any org
Go to page where you add organization
There you can see - Virksomheter det tidligere er delegert til: even if you haven't delegated API to any org
Wondering if we need this when there are no existing delegations

@rtafj @jonkjetiloye

@anishastphen personally I would think this was 'as designed', but from the figma design I see the header isn't even there to tell the user the list contains organizations which has already received delegations.
@hannekristin I guess this is more of a design review issue?

@hannekristin - jeg lukker denne. Protester hvis du er uenig :)


Parameterize Tuf object to deserialize custom metadata field

This would allow us to pass back a real structure and not be pinned to JSON.

This can't be parameterized in a nice way that doesn't wreak havoc with serde (or so it seems). The TargetDescription struct can just store it as raw bytes because the DataInterchange trait allows it to be serde'd to/from a) DataInterchange::RawData and b) bytes. This is good enough and someone external using the crate would be able to extract the bytes and do what they need with them.
Eventually someone might come up with something more clever than the couple of things I tried, but this is good enough for now.

This is mostly implemented. TargetDescriptions now capture the target custom field. We just need to make Client::fetch_target_description public.


Returns wrong thumbprint when using "Subject" as certificate identifier

Config:
        Name = 'MyWebiste'
        BindingInfo = MSFT_xWebBindingInformation
        {
            Protocol              = 'https'
            Port                  = '443'
            CertificateStoreName  = 'MY'
            CertificateSubject    = 'WebShop.contoso.com'
            HostName              = '*'
            IPAddress             = '*'
        }
    }

If multiple certificates with the same subject are found, the returned thumbprint is a merge of all found certificates.
Error message:

PowerShell DSC resource MSFT_xWebsite  failed to execute Set-TargetResource functionality with error message: Failure to add certificate to web binding. Please make sure that the certificate thumbprint
"0F26652D3DB7152F6B575E53D85B8B80BBDD3603 BC80F9BF4E861EA972F0B88644AFA5ECD90B4C2D" is valid. Error: "Value does not fall within the expected range.".
+ CategoryInfo          : InvalidOperation: (:) [], CimException
+ FullyQualifiedErrorId : ProviderOperationExecutionFailure
+ PSComputerName        : localhost

Come on Microsoft! How is this still not solved after almost two years?
@SteveL-MSFT @johlju  If I commit a fix to this problem, at what pace can I expect a review and new package release?
Suggested solutions:

Correct the functionality of Find-Certificate to return a single certificate (as name suggests):
https://github.com/dsccommunity/xWebAdministration/blob/master/source/Modules/xWebAdministration.Common/xWebAdministration.Common.psm1#L787:

 $certs = $certs | Sort-Object -Descending -Property 'NotAfter' | Select-Object -First 1

OR if the Find-Certificate is supposed to return multiple certificates, correct the issue directly in xWebSite.psm:
https://github.com/dsccommunity/xWebAdministration/blob/master/source/DSCResources/MSFT_xWebSite/MSFT_xWebSite.psm1#L1477

$Certificate = Find-Certificate @FindCertificateSplat  | Select-Object -First 1

You are more than welcome to send in a pull request. 🙂 I will review it as soon as I have time (depending on day job and family), As soon as a commit is merged to master a preview release will be released. We can release a new full version shortly after that.
For my reference, this was worked on here but was abandoned: https://github.com/dsccommunity/xWebAdministration/pull/334/files
Btw. The DSC module is not part of the DSC Community, no longer owned my Microsoft. That is in part why we can release so fast now. 😃

Oh, I suggest alternative 2 since it looks like Find-Certificate is meant to return more than one certificate. Looked at the function over at CertificateDsc and it does the same, so let's change xWebSite.

@johlju Thanks for your response. I've created a pull request where I have fixed the issue and created a new unit test to demonstrate the issue. Hopefully this can be reviewed and released in the near future :)

@vidarw Pushed a new release so should be available within the hour.

Thanks a lot for the quick responses and fast review process! :)


Collaborating on the project

Hi, I'd like to collaborate on this project so that HLearn gets usable.
Do you have a list of issues that need contributions? I'm looking to start with something simple.

There are two options:

Find some FIXME in the source code that sounds fun to you and fix that.  You can see  #41 as an example.

Add support for sparse linear algebra.  This will involve writing bindings to a BLAS library for sparse linear algebra.  There are currently three implementations of linear algebra in subhask.  The first thing to look at is is this example which gives an overview of the linear algebra operations.  This example uses the HMatrix bindings.  HMatrix is a Haskell library that provides bindings to standard BLAS libraries.  Two other implementations are the accelerate GPU backend currently under development (see #39), and a native haskell backend that is slow but does not require dependencies (see #32).

Thanks for the answer. Will have a look and come back with some extra questions!

There's a lot of undefined's also - filling these in or adding a fixme if it's unclear would be an easy entry to helping out.


Hackintool not showing any ports on z490 board

It did show ports but I clicked clear all and now hitting refresh does nothing.  No matter what I try I can't get my port list back.  The ports are working though...

ok, I have it showing ports but it doesn't show any devices on the ports even  though they are working.  This is using OpenCore .60 and I'm using usbinjectall

I don't have either of those.  I have SSDT-USBx.aml

This is what Hackintool looks like on my system. I have several USB devices plugged in.

I am running with USBInjectAll.kext right now.  My USB map is not enabled.

and I have the port limit quirk enabled.


Use git recordings for pull request validation

Thank you for the review! I removed the watcher changes to make this pull request self-contained and easy to revert if needed.


How to get no-reference indicator ?

Where is the code to get the values of UCIQE and UIQM indicators ?
Thank you !

I'm very sorry. In trying to organize the code to post on github, I rewrote the eval file and forgot about UCIQE and UIQM, but you can easily search github for python code provided by others. In fact, I have not had much time to update the code recently as I was away on business. I hope this didn't cause you any problems.

@TingdiRen
Thanks for your reply in your busy schedule. During our experiment, we found that different data division will affect the final results. Could you please provide us with your division txt?

@TingdiRen Thanks for your reply in your busy schedule. During our experiment, we found that different data division will affect the final results. Could you please provide us with your division txt?

I'm really sorry, I just noticed this response. However, I have already left my previous workplace where I had access to the server. When I asked today, the server I used has been formatted. Therefore, I'm sorry that I can't provide you with that partition. But in previous experiments, different partitions did not result in unexpected differences in performance metrics, so it should not significantly affect your reproducibility.


Fix typo

missing e in 'Stash only' command description: 'Crate a simple stash'

Thank you @janosh !


Adjust alignment of globe and lock icon.

discussion - https://chat.zulip.org/#narrow/stream/101-design/topic/globe.20icon.20for.20world-public.20streams

Merged, thanks @amanagr!


Index trait bound makes it impossible to return an owned value

I'm working on json diffing. I have some representation of two jsons in memory, like a serde_json::Value. Obviously these cannot be directly diffed, since they're not array-like. So to diff them I need to flatten them into an array of json-lines. I'd rather not actually construct the array, however: I don't want the memory usage. Instead, I'd like to generate the lines on the fly.
I can almost do this by implementing the Index trait. It doesn't quite work, however, since I need to return a reference. Ideally, I'd like to be able to pass in an FnMut(usize) -> T or something along those lines.
I'm not sure if anybody but me will find this useful, TBH, but I figured there's no harm in asking.

@rocurley Why can you not produce a &str (or similar). What the Index trait expands to does need to be a reference, but I did not find it massively to be a problem so far. Do you have an example where it does not work for you currently?

So my actual use case is diffing jsons. But you don't need the full complexity of json here to have this issue. Say you want to diff Vec<Vec<u8>>s. Now in principle you could just pass that in directly and be fine. But the diff will then consider the inner Vec<u8>s to be the unit of diffing, so the diffs will be too coarse. [[1,2]] -> [[1,2,3]] shouldn't say the entire input has changed, for example.
So one way to do this is to flatten the input into a vector representation that allows you to reconstruct the original input:
enum DiffElem {
    VecStart,
    Elem(u8),
}

fn to_diffable(xss : &Vec<Vec<u8>>) -> Vec<DiffElem> {
    let mut out : Vec<DiffElem> = Vec::new();
    for xs in xss {
        out.push(DiffElem::VecStart);
        out.extend(xs.iter().map(|x| DiffElem::Elem(*x)));
    }
    out
}

Now this will work, but requires making a copy of the entire input. But it doesn't seem strictly necessary. You could "index" into the diffable representation without ever constructing it:
// This implementation is ineffecient. To get a more effecient one, we'd
// annotate each subvector with its the index of its VecStart, and use
// binary search to find the subvector we care about.
fn index_diffable(mut i : usize, xss : &Vec<Vec<u8>>) -> Option<DiffElem> {
    for xs in xss {
        if i == 0 {
            return Some(DiffElem::VecStart);
        }
        if i < xs.len() {
            return Some(DiffElem::Elem(xs[i]));
        }
        i -= xs.len();
    }
    None
}

But you can't do this with the Index trait.

I agree that this case is not great. Generally I'm not a huge fan of the Index trait as base. I have already considered alternatives before but never with too much success.

I've been trying to implement a wrapper type to get ropey::RopeSlice to work with similar using the iterators it provides, but the problem is that they only return owned values.

The original issue I had with Index wasn't even that it returns references but that it can be hard to get the bounds checks reliably eliminated (#19). So originally I was trying to do that.
To address this particular thing with borrows I tried to have a custom trait but making both borrows and owned values work is surprisingly complex and I did not go very far with it because I did not at all like what it did to the API. I'm not sure if there are clever ideas I haven't been thinking of yet that would make both cases work.

@cessen the issue is that this Index based API is all over the place. This goes really deep in the library.

Ah, got it.  If I get some time and the mood strikes, I might take a crack at that as well.  If we're lucky, even if the API goes deep, it still might not be too hard to separate from the algorithm code itself.  If we're lucky... :-)

The other issue with the callback API is that it can't borrow. So it's not even that the callback based API could be the base of the API.

Perhaps a macro that could create two versions of the function; one that borrows and one that returns an owned value?

So one could move somewhat to improving this by going all in on the IdentifyDistinct type. So something like this could be done:
let old = Rope::from_str("...");
let new = Rope::from_str("...");
let h = IdentifyDistinct::<u32>::new_from_iter(old.lines(), new.lines());
let diff = capture_diff(
    Algorithm::Myers,
    h.old_lookup(),
    h.old_range(),
    h.new_lookup(),
    h.new_range(),
);

The new_from_iter is easy to add, but the returned ops are still annoying to work with. For instance iter_changes again requires an Index bound.

So now that there are GATs that issue might be worth toying around with again.


About file fixes

Made changes necessary to make the "about" files load properly. In particular, the new "about" files had incorrect IRIs, and all the "about" files referencing cache IRIs needed "master/latest/" inserted into them.
Important Note
I just had an important insight into one of Pawel’s comments that may throw a monkey wrench into what we've been trying to accomplish. If the “about” files really exist only for the build process, then they should be referencing only local files. If we were to wipe out everything, including the web server for our IRIs, and start a build from scratch, the build server wouldn’t find the files it needs in order to build that web server!

its the URL specified in the import, is there a specific keyword in the catalog that can tell it that it is an URL?

Lets approve, tag and force push later if needed


EBSBlockDevice supports KmsKeyId

Per AWS Documentation, EBSBlockDevice supports KmsKeyId now
https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-ec2-launchtemplate-blockdevicemapping-ebs.html

I approve

Thanks!


Fixes toString return type

fixes toString return type:
const value1: string = new TinyColor('#ccc').toString();
const value2: string = new TinyColor('#ccc').toString('rgb');
const value3: string | boolean = new TinyColor('#ccc').toString('name');

Codecov Report

Merging #214 (a64befa) into master (ce3b3df) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #214   +/-   ##
=======================================
  Coverage   97.79%   97.79%
=======================================
  Files          10       10
  Lines         635      635
  Branches      146      163   +17
=======================================
  Hits          621      621
  Misses         14       14

Impacted Files
Coverage Δ

src/index.ts
98.21% <100.00%> (ø)

Continue to review full report at Codecov.

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

Codecov Report

Merging #214 (a64befa) into master (ce3b3df) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #214   +/-   ##
=======================================
  Coverage   97.79%   97.79%
=======================================
  Files          10       10
  Lines         635      635
  Branches      146      163   +17
=======================================
  Hits          621      621
  Misses         14       14

Impacted Files
Coverage Δ

src/index.ts
98.21% <100.00%> (ø)

Continue to review full report at Codecov.

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


React Compiler

Will there be instructions on how to use the new React Compiler given that it's babel based?

For now the simplest way is to switch back to the Babel plugin. They working a Rust port, I hope this can be builtin in SWC in the future!

You should use vite-plugin-react

import { useState } from "react";
function Header() {
console.log("Header", Math.random());
return (

React Counter

);
}
function App() {
const [count, setCount] = useState(0);
return (
<>

{count}
<button onClick={() => setC
}

Will be there option to use react compiler with SWC in the future?

If the React team lands the Rust port of the compiler, I think SWC will try to integrate it directly. But the rust part didn't get updated since annouced: https://github.com/facebook/react/commits/main/compiler/crates


feat(Illustrations): Add Code Celebration illustration to gamut-illustrations

Overview

PR Checklist

[x] Related to designs: FIgma
[x] Related to JIRA ticket: GRW-171
[] I have run this code to verify it works
[ ] This PR includes unit tests for the code change
[ ] The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

PR Links and Envs

Repository
PR Link
PR Env

Monolith
Monolith PR
Monolith Env

Portal
Portal Link
Portal Env

Another Repo
Another Link
Another Env

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.27.3-alpha.831b0d.0@codecademy/gamut-kit@0.6.148-alpha.831b0d.0@codecademy/gamut-labs@32.2.1-alpha.831b0d.0@codecademy/gamut@48.0.5-alpha.831b0d.0@codecademy/monaco@0.6.54-alpha.831b0d.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://62b373c006e3db21e9dbc344--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.27.3-alpha.d74749.0@codecademy/gamut-kit@0.6.148-alpha.d74749.0@codecademy/gamut-labs@32.2.1-alpha.d74749.0@codecademy/gamut@48.0.5-alpha.d74749.0@codecademy/monaco@0.6.54-alpha.d74749.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://62b379da47ca7027f48ea103--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.27.3-alpha.79970c.0@codecademy/gamut-kit@0.6.148-alpha.79970c.0@codecademy/gamut-labs@32.2.1-alpha.79970c.0@codecademy/gamut@48.0.5-alpha.79970c.0@codecademy/monaco@0.6.54-alpha.79970c.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://62b47a004fb33900641bfe77--gamut-preview.netlify.appDeploy Logs

@dreamwasp thank you for looking at this! yes it looks like it was missing the viewBox property, it's looking good now 👍


Make Airflow connectionsTemplates and variablesTemplates work

What issues does your PR fix?

https://github.com/airflow-helm/charts/issues/618

What does your PR do?
Prior to this PR specifying variablesTemplates or connectionsTemplates did not actually get picked up by the python code. Additionally, in order for these variables to be injected to the right pods you had to specify at least 1 variable or connection, now this is not the case (i.e. it's consistent with the documentation specified here

https://github.com/airflow-helm/charts/blob/main/charts/airflow/values.yaml#L145-L148
https://github.com/airflow-helm/charts/blob/main/charts/airflow/values.yaml#L80-L95

Checklist
For all Pull Requests

[X] Commits are signed off
[X] Commits have semantic messages
[X] Documentation updated
[x] Passes ct linting

As requested in https://github.com/airflow-helm/charts/issues/618#issuecomment-1167429872, I am closing this.


Automatic decompression of HDFS files tagged with extended attribute using Snappy

281

 Quality gate: Failed
480 Issues

 32 Vulnerabilities
 448 Code issues

Details

 Quality gate: Failed
468 Issues

 32 Vulnerabilities
 436 Code issues

Details


Test failed due to missing files

https://download.copr.fedorainfracloud.org/results/remilauzier/zemeroth/fedora-rawhide-x86_64/02311687-rust-ab_glyph/builder-live.log.gz
Fedora as update there version of ttf-parser but the test failed due to missing files.
---- src/font.rs - font::Font::as_scaled (line 175) stdout ----
error: couldn't read src/../../dev/fonts/Exo2-Light.otf: No such file or directory (os error 2)
--> src/font.rs:178:36
|
5 | let font = FontRef::try_from_slice(include_bytes!("../../dev/fonts/Exo2-Light.otf"))?;
|                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= note: this error originates in a macro (in Nightly builds, run with -Z macro-backtrace for more info)
error: aborting due to previous error

Test fonts aren't uploaded to crates.io. but you can use the github tags
here instead of you want to run tests.
Including test assets would significantly bloat the crate for all
dependants. So I'd rather avoid them in crates.io. I'm not sure if there's
a better way here though.
On Thu, 1 Jul 2021, 18:58 Rémi Lauzier, @.***> wrote:

https://download.copr.fedorainfracloud.org/results/remilauzier/zemeroth/fedora-rawhide-x86_64/02311687-rust-ab_glyph/builder-live.log.gz
Fedora as update there version of ttf-parser but the test failed due to
missing files.
---- src/font.rs - font::Font::as_scaled (line 175) stdout ----
error: couldn't read src/../../dev/fonts/Exo2-Light.otf: No such file or
directory (os error 2)
--> src/font.rs:178:36
|
5 | let font =
FontRef::try_from_slice(include_bytes!("../../dev/fonts/Exo2-Light.otf"))?;
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= note: this error originates in a macro (in Nightly builds, run with -Z
macro-backtrace for more info)
error: aborting due to previous error
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/alexheretic/ab-glyph/issues/42, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARZHV3ZX743RV5WCIM2IEDTVSUD7ANCNFSM47VGIQFQ
.

I can't use tags since it would complicate packaging it for fedora that use rust2rpm to ease packaging.


Debug info should depend on the DEBUG var, not PROFILE

Currently, gcc checks if PROFILE is "debug" and enables debug info if so. I'm not sure if this was added recently, but Cargo appears to set the DEBUG env var to the debug setting in the active profile, which seems like a better variable to check.
If this is done, the compiled C code has debug info iff the Rust code has debug info (this isn't currently the case).

Agreed! I think at the time of inception the DEBUG env var didn't exist...


PublishableBehavior afterFind proposal

Taking into account that cakephp does not fire the beforeFind() callback for behaviors inside the related models, this is a first approach to remove the non published elements from hasMany relations that has the publisable behavior enabled.
/**
 * After Find Callaback
 * @param \Model $model
 * @param array $results
 * @param boolean $primary
 * @return array
 */
	public function afterFind(\Model $model, $results, $primary = false) {

		if ( ((bool)Configure::read('Publishable.disable')) === true) {
			return $results;
		}

		$childrens = $model->getAssociated('hasMany');
		if (array_key_exists($model->alias, $results)) {
			// Not contemplated
			die( "This case is not contemplated" );
		} else {
			foreach ($results as &$result) {
				foreach ($childrens as $m) {
					if (array_key_exists($m, $result)) {
						if ($model->{$m}->Behaviors->enabled('Publishable')) {
							foreach( $result[$m] as $key => $subresult ) {
								if ($subresult['published'] != true) {
									unset( $result[$m][$key] );
								}
							}
						}
					}
				}
			}
		}
		return $results;
	}

Could you send a PR to the develop branch instead? We'll discuss the new feature in the PR


Body ng controller

Could anyone spoonfeed me to how to Get a  to work ?

What have you tried? And what context do you need this applied?
We're using nested views, so you should be adding
's to your views & adding the ng-controller attribute to that.
Why not just follow the examples that exist in the current MEANJS framework? We don't add the ng-controller attribute to the views. These are specified in the angular UI-router configurations; e.g. Home view

Im trying to connect a controller to the body, to allow elements to be used on the whole site, and not be controlled by each invidivual page.
I am using the :

      {{{block "content"}}}

for the individual pages, but i need one other to control all the other elements outside of the section(s). if you got what i mean?

You can do something similar to this: https://github.com/meanjs/mean/blob/master/modules/core/server/views/layout.server.view.html#L39
Or you could add a directive to the the server layout view, that encompasses your functionality. I ended up doing this for client-side Notifications in my own project.

Could You share some more?
Btw, with the linked example, I could do something like that in the body too?

You could include a client-side template just like what we're doing with the header. Although, I would recommend using a directive over that approach. Does your functionality have persistent UI elements? Meaning you would have a html view.

Yes IT do.
How Would The direktive be ?

I do have 3 html sections befare The normal section snd 2 after in The view i Would need to manipulate

https://github.com/meanjs/mean/blob/master/modules/core/server/views/layout.server.view.html#L39
but that wouldnt be dynamically updated if i updated some settings in the Home controller?

What exactly are you trying to accomplish?
My guess is that a directive is exactly what you need. There are plenty of examples on the web for implementing directives, and we have some in this framework. It shouldn't be difficult.

Im trying to acconplish to frequently update a lot of the HTML elements
outside of the section up view .
In the normal case with body ng controller I would use it as a regular
controller to update elements in the body, that is not controlled by any of
the young views .
Made that any sense or do you want me to take a picture and explain in
details ?
Den lørdag 17. september 2016 skrev Michael Leanos<EMAIL_ADDRESS>følgende:

What exactly are you trying to accomplish?
My guess is that a directive is exactly what you need. There are plenty of
examples on the web for implementing directives, and we have some in this
framework. It shouldn't be difficult.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/meanjs/mean/issues/1511#issuecomment-247751373, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJHUrOunShFvEkj3_buPCAe6j2Jbxod-ks5qq3_LgaJpZM4J-Bky
.

example with what you said.
layout:

controller:
(function () {
  'use strict';

  angular
    .module('core',[])
    .controller('HeaderController', HeaderController);

  HeaderController.$inject = ['$scope', '$state', 'Authentication', '$rootScope','Socket'];
  function HeaderController($scope, $state, Authentication, $rootScope,Socket) {

    var aux = this;

    $scope.datatest = 200001;
    $rootScope.datatest = 900;

    $scope.varselmessage = 'varsel';
    aux.varselmessage = 'varsel2';

    vm.test = 22;
    $scope.test = 33;

    $scope.notifications = [{text: " melding" , image : "kapring"}];
    console.log("loaded , maybe possible4?");

  }
}());

view:

    <button class="navbar-toggle" type="button" ng-click="vm.isCollapsed = !vm.isCollapsed">
      Toggle navigation

    </button>
    MEAN.JS

{{ vm.test }}

on site: http://prntscr.com/ciyzlm

@frkwhiteangel if you do var aux = this within your controller, aux will be the scope. So when you do:
 var aux = this; // aux is bound to scope
...
$scope.datatest = 200001;
$rootScope.datatest = 900; // will overwrite 200001

$scope.varselmessage = 'varsel';
aux.varselmessage = 'varsel2'; // will overwrite varsel

vm.test = 22;
$scope.test = 33; //will overwrite 22

Read this: https://github.com/johnpapa/angular-styleguide/blob/master/a1/README.md#style-y031
To make things a little easier you should use controllerAs and use var vm = <somevar> so that when you're using the controller the connection is more visible. Only inject scope where you need to use it like to emit events or for timers etc

i did do include https://github.com/meanjs/mean/blob/master/modules/core/server/views/layout.server.view.html#L39
and made my own with a differnt name, so using two.
but one of them always fails
angular.js:13920 Error: [ng:areq] Argument 'dataController' is not a function, got undefined
(function () {
    'use strict';

    angular
        .module('core',[])
        .controller('dataController', dataController);

    dataController.$inject = ['$scope', '$state', 'Authentication', '$interval'];

    function dataController($scope, $state, Authentication, $interval) {

        var vm = this;
vm.test = 2;

    }
}());


Error in using object mapper if DAO param is DTO

cannot inject param in plugin cuz it was pojo, not map.


Messages app

Signed-off-by: Jorma Virkkunen<EMAIL_ADDRESS>

on line 36 this sentence in my opinion should have one more "to"

Added.


Project tweaks

Update Xcode project indent settings to match coding style
Fix .gitignore so it ignores build folders in repository root and w/in demo project folder

:+1:


update datapath to README

Pull Request Description
Describe what this PR does / why we need it
Does this pull request fix one issue?

Describe how you did it
Describe how to verify it
Special notes for reviews

Codecov Report

Merging #294 (3dca0af) into main (5543ccc) will decrease coverage by 0.19%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #294      +/-   ##
==========================================
- Coverage   41.39%   41.19%   -0.20%
==========================================
  Files          39       39
  Lines        2534     2534
==========================================
- Hits         1049     1044       -5
- Misses       1377     1380       +3
- Partials      108      110       +2

Flag
Coverage Δ

unittests
41.19% <ø> (-0.20%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/controllers/networking/quota_controller.go
80.00% <0.00%> (-8.00%)
:arrow_down:

...kg/controllers/networking/ipinstance_controller.go
85.18% <0.00%> (-7.41%)
:arrow_down:

pkg/controllers/networking/pod_ip_cache.go
42.30% <0.00%> (-5.13%)
:arrow_down:

pkg/controllers/networking/pod_controller.go
49.82% <0.00%> (+1.76%)
:arrow_up:

Continue to review full report at Codecov.

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


[Communication] - [sms] - Removed typo, extra comma in the code.

Description
Removed typo, extra comma in the code.
All SDK Contribution checklist:

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

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] 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

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

/azp run python - azure-communication-sms - tests

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-communication-sms


Worker reports failure getting next session

clickup: https://app.clickup.com/t/tvcdbk

Wrong repo :facepalm:


fix(TreeView): Fix invalid tree item children

Issue: none
What I did

MAST engineers reported a change in behavior from 4.6.0 to our prerelease. Items that had invalid children would get the expandable arrow when they shouldn't. In addition, I added unit tests so we don't break this accidentally in the future.
Screenshots:
All examples with the same code:
   <TreeView>
        <TreeItem label="Node 0 - fragment" itemId="item0" testId="item0">
          <></>
        </TreeItem>
        <TreeItem label="Node 1" itemId="item1" testId="item1">
          <TreeItem label="Child 1" itemId="item-child1">
            <TreeItem label="Grandchild 1" itemId="item-gchild1">
              <Tag>This is a tag as a child of Grandchild 1</Tag>
            </TreeItem>
          </TreeItem>
        </TreeItem>
        <TreeItem label="Node 2" itemId="item2">
          <TreeItem label="Child 2" itemId="item-child2">
            <TreeItem label="Grandchild 2" itemId="item-gchild2">
              <TreeItem label="Great-grandchild 2" itemId="item-ggchild2" />
              <TreeItem label="Great-grandchild 3" itemId="item-ggchild3" />
            </TreeItem>
          </TreeItem>
        </TreeItem>
        <TreeItem label="Node 3" itemId="item3"></TreeItem>
        <TreeItem label="Node 4" itemId="item4">
          Child of node 4 is just text
        </TreeItem>
      </TreeView>

In 4.6.0

In 4.7.0-next.50

Update from this PR

Checklist

[X] changeset has been added
[X] Pull request description is descriptive
[-] I have made corresponding changes to the documentation
[X] New and existing unit tests pass locally with my changes
[X] I have added tests that prove my fix is effective or that my feature works

How to test

Open CodeSandbox and try different kinds of tree structures. Ensure that the items that are expandable are the expected ones (ones with valid children, whether that's a single TreeItem or multiple).
Note: to test previous behavior, open CS and change the package version from 4.6.0 to the latest prerelease

MAST engineers checked this version and are no longer seeing this visual issue.


File error codes

Attempts to address #169 by adding simple error codes (1 and 2) to APPEND and WRITE.
(I had trouble writing tests for this, need to find a surefire way to make these things fail.)
Edit: I just re-read the original issue, going to write a test for a protected file now.
Edit2: Basic tests are in https://github.com/Lartu/ldpltest/pull/20
🕺

Wonderful! Thank you very much for this!