Selecting "Delete cluster" while notebook is opened does not delete  the cluster

When I have a notebook opened and I select "Delete cluster", the following occurs:

Browser navigates to the Notebooks list

Nothing else seems to occur. I don't see any indication in the UI that the cluster is being deleted.
If I refresh the page the cluster panel continues to just show "Running". Now if I select "Delete cluster", the panel transitions to showing "NONE" for the notebook runtime.

Related to #1384

I'm closing this as a duplicate of #1384, which should be fixed soon. Thanks for the feedback!

Tested this and the behavior is great. Thanks!


hlvx and shadow stack pages

How does hlvx interact with shadow stack pages? The spec says
However, the shadow stack is readable by all instructions that only load from memory.

and
Instructions HLVX.HU and HLVX.WU are the same as HLV.HU and HLV.WU, except that execute permission takes the place of read permission during address translation. That is, the memory being read must be executable in both stages of address translation, but read permission is not required.

Does the hlvx requirement supersede the shadow stack text?

The HLVX instruction tests for execute permission in the page tables. Shadow stack pages are encoded with XWR=010 and attempts to access shadow stack pages using HLVX will lead to a load page fault.

Thanks for clarifying.


E5108: Error executing lua ...te-sshfs.nvim/lua/telescope/_extensions/remote-sshfs.lua:159: attempt to index local 'opts' (a nil value)

The telescope extension commands are started with no arguments instead of one argument that should be an empty (or not) table.
For instance,
M.find_files = function()
  require("telescope").extensions["remote-sshfs"].find_files()
end

should be
M.find_files = function(opts)
  require("telescope").extensions["remote-sshfs"].find_files(opts)
end

Then, either check for opts' existence and or set it to {} by default, and add it to the plugin doc.
Complete error stack :
E5108: Error executing lua ...te-sshfs.nvim/lua/telescope/_extensions/remote-sshfs.lua:159: attempt to index local 'opts' (a nil value)
stack traceback:
        ...te-sshfs.nvim/lua/telescope/_extensions/remote-sshfs.lua:159: in function 'find_files'
        ...te-sshfs.nvim/lua/telescope/_extensions/remote-sshfs.lua:485: in function 'find_files'
        ...are/nvim/lazy/remote-sshfs.nvim/lua/remote-sshfs/api.lua:27: in function 'find_files'
        [string ":lua"]:1: in main chunk
E5108: Error executing lua ...te-sshfs.nvim/lua/telescope/_extensions/remote-sshfs.lua:159: attempt to index local 'opts' (a nil value)

Thank you for the input and good catch! Added opts to the API layer.


[ldd-spectral] New LDDTool Flagging Warnings

Issue Type
Bug affecting build process
Describe the issue identified (if applicable)
The newer versions of LDDTool are checking things that have never been checked before and are flagging some minor issues that should be cleaned up - like attributes that are both nillable and optional.
Describe the solution you'd like
Build with the latest LDDTool and cleanup any warnings.
Describe alternatives you've considered
None
LDD Dictionary Version
All versions that are being built - the actual version was not reported.  All Spectral versions should be updated.
PDS4 IM Version
N/A
Need-by Date
Next build.
Additional context
Note that there are potentially several dictionary versions that should be updated and built for various IM versions.

New version of LDDTool is not longer flagging these warnings. Neither is the available 1D00 version of LDDTool. This makes it effectively impossible to correct whatever it was complaining about.  So I'm closing this for want of data until the error reporting is reinstated.


Couldn't inference on Gluoncv

Some of your models like Resnest_200 and Resnest_269 are not available for inference through gluoncv as it shows the model files are unavailable. Any way I can check the above models on ADE20K.
I even tried through encoding but it was showing error of enclib_gpu/enclib_cpu whenever I tried to import it.

For issues with GluonCV, please report it on their github repo.
You can also download gluon model from this repo. Please update to the newest version.


Liveness probes

Hi and thank you for this much needed library.
Have you considered, or are you planning to add, a way to drivers are able to communicate properly?
I'm thinking about exposing a function to ping redis or select 1 a database.
It would be great to have such functionality available on the BentoCache instance, to be called when needed or exposed by an http route.

hey, thanks !
i am not sure if that's the role of the library to do that
when you create your Bentocache instance, you must pass your ioredis instance, for example. so you can keep this instance around and use it for your route + health check. so yeah, nothing stops you from doing this without bentocache exposing this functionality. It just seems a bit out of scope to me
and also, there are drivers for which it wouldn't make sense to have this, like filesystem and in-memory.
let me know what you think

I agree with you that if a library like bentocache provides one with the ability to supply his own (redis or whatsoever) instances there should be no need to implement healthchecks. Still I think that given the aim of bentocache, a built in healthcheck would be nice!
Moreover, I don't see this capability with redis' bus driver, as in the constructor you can only provide ioredis options.
I could always wrap or extend the driver class if it wasn't that (and rightly so) underlying instances are private members.
As for memory or fs drivers, I still think that a healthcheck can [1] [2] be implemented, even if that may not make sense for common workloads. Also, testing won't be trivial.
With that being said, I still think it would be a nice feature but perhaps not as valuable as it is to me for other users.
Up to you! I would be happy to submit a pr should ever change your mind.


Email update css

high level what you did:

created css classes to try to dry up code. it's not perfect, and it broke the responsiveness of the email but its now only 700 lines of code instead of 3700!

Overall Checklist:

[ ] tests written for all methods added
[ ] test coverage above 90%
[ ] RSpec running clean
[x] ran rails s -- features working properly!
[x] pulled/merged main before submitting PR

Great work refactoring this email template!


localhost:3000 shows 2017 theme

Hi, I have a fresh install of wordpress then cloned the repository and install with npm, gone into the wp-admin using MAMP PRO as a host and changed the theme to FoundationPress.
ok no style so  I use 'npm install'
When I visit the site from here I get the FoundationPress theme, great
I want to use the browser sync so changed the url in gulpfile.js to 'localhost' and 'npm run watch' but when the localhost:3000 window pops up I get the 2017 theme?
So I trashed the 2017 theme thinking this will force it to show FoundationTheme, but now it's still showing the 2017 theme which is kind of weird
Any ideas?
Thanks
Andrew
Sierra 10.12.5
node v6.10.3
foundation-sites#6.3.1

You may have a vhost or another process running on port 3000 which is serving a WP install.
Have you tried changing ports?

where can I change ports in gulpfile.js

Around line 103 of gulpfile.js
    // port: PORT

Ah great thanks, tried that and it changes the port number but still no joy. It's not that it fails to load, it starts trying to load but ends up spinning

localhost right? Apache? so when you go to localhost it gives you the default apache info page?

I mean this is working fine and you can see there I've changed the port to 8080. But when I try to view localhost:8080 it doesn't load

'vhosts files to run an apache server on a specific folder?' I haven't done anything like this, just following the instructions in the docs

The only thing I can think of is to make sure FoundationPress is installed under the "wp-content/themes" folder of the working WP install. When you go to wp-admin FoundationPress is available under themes, right? and you can activate it? Other than that, I can't see why you're setup is not working

yeah thats all in place but had to change themes using MAMP PRO because obviously I couldn't login in using BrowserSync, thanks anyway

You really don't go to WP dashboard using browsersync, that's only supposed to give you a preview of your sites front-end. You login to the wp dashboard of the virtual host you set up using MAMP Pro. You can create any number of virtual hosts in mamp pro, so how did you set up the one that servers your WP?

Just the same as any host in MAMP Pro. Create a new host pointing to the wp directory and create a database with it, open up the host and in the browser finish the installation of wordpress

This is what I get when I run
npm run watch
[BS] Access URLs:
--------------------------------------
Local: http://localhost:3000
External: http://<IP_ADDRESS>:3000
--------------------------------------
UI: http://localhost:3001
UI External: http://<IP_ADDRESS>:3001
--------------------------------------
[BS] Watching files..

I just visit `http://<IP_ADDRESS>:3000` and I get my wp install with FoundationPress loaded.
If I want to change my theme or edit content, I visit http://themesite.dev/wp-admin and log in

This is my gulpfile.js entry at line 17
// Enter URL of your local server here
// Example: 'http://localwebsite.dev'
var URL = 'http://themesite.dev';`

ok yeah it loads when i use the  http://<IP_ADDRESS>:8080 but it still using 2017theme which is not even in the theme directory. The only theme in the theme directory  is FoundationPress

ahh, but your vhost is pointing to localhost. if clean2 is the directory, you should have a vhost in MAMP pro pointing to it, and you need to add in a new domain. Right now, localhost is just serving up the default mamp vhost, that's why you're not seeing the changes.

Success. Of course... it's pointing to local host because I told it to, which set up on MAMP PRO is just a fresh install of wp with 2017 theme... So i just change the gulpfile.js to point to 'clean2' . and it works. Got there in the end thanks for your help @linuxbastard.

Closing this. Nice help @linuxbastard! (sweet username too!)


Update AI pages with Edinburgh Update 2

Added new page (ChatGPT_Demo) to demo core chat experience.
Removed the "Copy Text" Snippet in the GPT Action Menu page in favor of the new action.

This should be good to go now, I fixed up all the internal links to make sure they work

@josephine-the-great  -  This looks good.  Wait for the release to occur and then Merge!!!


UT for deployments package

Add unit tests for deployments package - #41

Would prefer if we use the assert package here too.

@asobti it should be fine

@camilocot I merged your previous PR and added the assert package to the list of dependencies. Could you modify this PR to use that package. You might need to rebase on the master branch first.

@camilocot Sorry just saw your last comment + commit. Can we use the assert package in deployments_test.go as well where you're doing manual checks and t.Error() calls?

@asobti I forgot deployments_test.go file :) Could you check it again?

Looks great, thanks


Question: class can inherit module?

I have a small question.
I found you can inherit modules to class as if modules were a class.
The following works on the latest Goby build, but I just wonder if this behavior is OK.
module Foo
  def self.my
    999
  end
  def ten
    100
  end
  def five
     7
  end
end

class Bar < Foo
  def five
    5
  end
end

a = Bar.new
puts(a.ten * a.five ) # => 500
puts(Bar::Foo.my)  # => 999

If you run the code with Ruby, you receive "superclass must be a Class (Module given) (TypeError)."
If you change module to class in line 1, it works with Ruby.

One more question.
Are the embedded classes closed?
Looks like the embedded classes are not extensible so far: I cannot add or override methods against them, and cannot inherit them.
class String
  def my_method
    self + "47"
  end
end

a = "test"
puts(a.my_method)  # => undefined method `my_method' for 1

class MyString < String
  def my_method
    self + "47"
  end
end

i = MyString.new("string")
puts(i.my_method)  # => Constant String is not a class. got=CLASS

I'm just unsure if this behavior is the expected one, or unimplemented. This depends on your plan or specification.

I don't think we have implemented self-defined method for built-in class, so defining extra methods  for String is not working. And yes, from a design perspective module should not be inherited to a class. It would be great to implement those features if you like to try :)

Unless @st0012 got a different idea on this

Thank you for the answer!
FYI: now I found that built-in class can be extended and overload methods by including a module:
module MyModule
  def my_method
    "47"
  end

  def to_i
    48
  end
end

class String
  include(MyModule)
end

i = String.new("string")
puts(i.my_method)  # => 47
puts(i.to_i)       # => 48

In Ruby, puts(i.to_i) returns 0.

Just a mutter to myself: Module's behavior is a little bit different between current Goby and Ruby, but this implies that built-in classes could be extended with modules that are written in Goby.
(Actually, I've been wondering if built-in classes such as Array and Hash can be extended with Enumerable module written in Goby, which might make Goby extensible more easily. To do this, perhaps I need Enumerator class first :-p. )

@hachi8833 @adlerhsieh I am not intended to make module inheritable, I just haven't forbidden such usage (please send a PR if you want to fix this). I also don't want built in classes to be extended, just haven't deal with it yet.
And thanks for the report, you really found some interesting behaviors I didn't expect.

Thank you for the answer! I just wanted to check your plans and intentions for modules and classes before touching codes.

@hachi8833 Do you want to fix this as your first contribution 😁?

Thnx! I'd dig it if I could fix the one.
To fix: modules should not be inherited to class with < operator on class definition.
(I'd notify that I'm slow in writing codes)

You should do this on the vm side and RClass struct has a flag determine if it's a module. So you just have to find a place to block the inheritance.

#131 under construction might be the key for the fix.

Nope, I think this check should happen on vm side, cause in the future I might remove module statement and use class statement for module too.

Thanks for @hachi8833 this has been fixed 😄


[3.x] PopupMenu rework and enhancements

Backport of #41640 for 3.x (cc @EricEzaM)
From my tests I didn't encounter any major breaking changes but there might be details about the popup behavior that I've missed.
I've done my best to port the PR with minimal changes, but also avoid having to massively change the existing codebase to accommodate for it. The only major deviation I had to do was adding in the variable max_size for setting the size, since that variable comes from Window, which  Popup was refactored to be a child of it (#37317)

It's a mix. I started by cherry-picking your commit and fixing conflicts, but then I compared with both master and 3.x, and did a few changes, some to fix bugs and others to keep current 3.x behavior.

Thanks!


[with-firebase] Ensure server credentials aren't bundled with client code

This is a great catch. Thanks! Could you fix this: https://travis-ci.org/zeit/next.js/jobs/228780445#L377
Then I'll merge 👌

@timneutkens fixed - habit sorry!

No worries 😄


Extend initial map to 64M

This will allow us to include a big u-root and that u-root can
contain u-root and ALL harvey binaries, i.e. all of amd64/bin.
We're going to want a better compression than just gzip.
-rw-r--r-- 1 rminnich rminnich 70192397 Aug 31 20:43 /tmp/initramfs.plan9_amd64.cpio.gz
-rw-r--r-- 1 rminnich rminnich 23180974 Aug 31 20:25 /tmp/initramfs.plan9_amd64.cpio.lzma
But we are, definitely, going to be able to have pretty much anything we need ... in the initramfs.
This is going to be pretty slick. Boot harvey, get the full graphical environment, THEN mount the
server from 9p. This would include rio etc. This is gonna be very nice.
Graham, tmpfs needs lzma added.
Signed-off-by: Ronald G Minnich<EMAIL_ADDRESS>

I did try to write a loop, but ...


Use pointer parameters in Variant function pointers

Instead of references. This is needed because those function pointers are used in GDNative which needs to work with plain C, which doesn't support passing parameters by reference.
Also fix a logic issue in the has() function to check available keys in objects.

Thanks!


Prepare MHC support by removing NHC dependencies

Remove NHC dependencies in resource and lease manager in order to make it usable for MHC support.
Will make the upcoming MHC PR less complex.

/lgtm
Giving others a chance to review as well, feel free to unhold
/hold

/retest

depends on #268

/override  e2e-k8s
There are more changes at these code places in the MHC PR in the meanwhile. However merging this as it is ist still better than nothing I guess 🤷🏼♂

/lgtm

/hold cancel


"not implemented" message with jupyter notebook under python-3.8.0a4 (due to switch to "proactor" loop on windows)

see  https://github.com/jupyter/notebook/issues/4613
maybe Jupyter-Server could be a place to experiment solutions for cheap ?

The code has been updated to fix that error. At least tests do not fail on CI (python 3.8 on windows, see https://github.com/jupyter/jupyter_server/blob/master/.github/workflows/main.yml)

Closing since it looks like the issue was fixed in jupyter_server. If you're still having issues, feel free to reopen.


Add Hive statements

This PR adds hive specific syntax for CREATE TABLE statements following the Impala syntax.
Hive style databases have an even more diverse set of SQL statements. I have tried to integrate them into the current syntax as smooth as possible. I have created a new node called table_settings with should include most of the new statements. Some of them are also supported by MySQL and Postgres (e.g. PARTITION|PARTITIONED BY). I also modified a bit the TABLEPROPERTIES since the TBLPROPERTIES of are quite similar.
Here is the reference statement that I tried to implement:
CREATE [EXTERNAL] TABLE [IF NOT EXISTS] [db_name.]table_name
  (col_name data_type
    [constraint_specification]
    [COMMENT 'col_comment']
    [, ...]
  )
  [PARTITIONED BY (col_name data_type, ...)]
  [SORT BY ([column [, column ...]])]
  [ROW FORMAT row_format]
  [STORED AS file_format]
  [LOCATION 'hdfs_path']
  [CACHED IN 'pool_name' [WITH REPLICATION = integer] | UNCACHED]
  [TBLPROPERTIES ('key1'='value1', 'key2'='value2', ...)]

Note: i have excluded COMMENT and SERDE_PROPERTIES for now
Note2: Snowflake, Impala, AWS Athena, Hive, Datafusion (maybe BigQuery) share alot of these statements in slightly different fashion. I only have access to Impala, Athena and Snowflake for testing purposes.
Fixes #69
Sorry for this huge PR. Next time I will try to cut it down into smaller pieces.
References:
https://impala.apache.org/docs/build/html/topics/impala_create_table.html

I think blubb.sql was added by mistake?

Oh, sorry. Yeah I added some stuff that was not meant to be added. I will fix this right away. I have to do a rebase anyway now.

@DerekStride rebased & squashed


Make the buttons in the summary bar not selectable

These buttons don't actually do anything, so making them not selectable makes sense.

Didn't even realize these were selectable. Thanks.


OpenBSD: EPROTO available since 6.2

Hello,
I was recently trying to use the latest version of glium (v0.23) on OpenBSD, which depends on a newer version of wayland-client (v0.21), that depends on v0.12 of nix, but it unfortunately does not build. While I realize that v0.13 compiles and works correctly on OpenBSD, forcing glium to use a newer version of wayland-client that depends on nix v0.13 does not fix the issue because wayland-client makes use of EPROTO, which is not provided by the nix crate on OpenBSD. EPROTO was added around OpenBSD 6.2 (https://man.openbsd.org/OpenBSD-6.2/errno), so it is safe to make it available via this crate on OpenBSD also.
Many thanks,
Rob

Ah, I see this is already in discussion: https://github.com/nix-rust/nix/pull/1036

Closed due to #1036 being merged.


[stable/chaoskube] Add pod annotations

Signed-off-by: Rahul Varghese<EMAIL_ADDRESS>Add the ability to specify pod annotations through
values.yaml.
Tested and verified that annotations are included in the existing pod.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ x] DCO signed
[x] Chart Version bumped
[ x] Variables are documented in the README.md

/ok-to-test

/lgtm


Fix cupy.clip to match numpy

Fixes #6901
Fixed the 'only a_max or a_min is given' issue by removing the default value of None for a_min and a_max.
Fixed the 'a_max < a_min' issue by tweaking the cythonic implementation for the function.
Added test cases for the same.

/test mini

The test failures seem unrelated to this PR. LGTM!


Issue50426-2

https://github.ibm.com/IBMPrivateCloud/roadmap/issues/50479

Do we need this change in project k?


Update GetRootContainer (bootstrapper).json

making the hosting url and sharing url non-empty string.
The previous empty string was making partners think optional values should be "" while they should just be omitted. Adding real values to make it more clear.

@karvell I'm OK with this change to the example response, but you do realize that both of these properties are actually optional, right? The only required property in a CheckContainerInfo response is Name.

+Trevor
Yeah it is optional. But I think the preferred approach to optional is to omit the response entirely, and not set it as empty string - do you agree with that? Empty string could be considered a valid response. We had a partner who explicitly set optimal responses to empty string because that’s what we had in the example.
From: Tyler Butler<EMAIL_ADDRESS>Sent: Monday, June 20, 2016 12:11 PM
To: Microsoft/Office-Online-Test-Tools-and-Documentation<EMAIL_ADDRESS>Cc: Karvell Li<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/Office-Online-Test-Tools-and-Documentation] Update GetRootContainer (bootstrapper).json (#230)
@karvellhttps://github.com/karvell I'm OK with this change to the example response, but you do realize that both of these properties are actually optional, right? The only required property in a CheckContainerInfo response is Name.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/Office-Online-Test-Tools-and-Documentation/pull/230#issuecomment-227239470, or mute the threadhttps://github.com/notifications/unsubscribe/AQKUYuiLw6n5WBF8nUB4rF1hDSmd1W2gks5qNuWmgaJpZM4I6ATv.

All WOPI JSON property strings default to the empty string, so while responding with an empty string is weird and not preferred, it's not technically wrong, either.
I think we're on the same page. Making the examples clearer is great; I just wanted to make sure that the client wasn't making assumptions that these were truly required properties. I assume there are validator tests already or in progress that will help ensure implementations are correct.


support for timestamp in milliseconds (CSVAdapter)

My CSV files are with timestamps in milliseconds which would be converted to Date directly. (without parsing).
Is there a way at the moment?

I'm afraid it is not possible at the moment.
I have been considering enhancing the csv adapter by reusing some of the features developed for the log files adapter, and amongst those is a much more powerful editor for expressing parsing rules for timestamps, that could fit the bill.
I'm going to leave this one open as a feature request, but if and when that happens will depend on me having some spare time (or someone else pitching in) so I can't give you any kind of an ETA.

Has there been an progress?  This would be a great feature add!   Either way, thank you for this fantastic tool.

The CSV plugin now uses the parsing dialog from the log file adapter, with offers a lot more flexibility when defining the parsing pattern, and also allows users to save, export and import those patterns.

For the specific requests expressed in this issue, you could simply use the built-in "Milliseconds from 01/01/1970" format:

Chances are your milliseconds timestamps are not Unix style epoch, so the date itself might be wrong, but at least the times in ms should be correct.
I've published a SNAPSHOT build that you can try out here: https://github.com/binjr/binjr/releases/tag/v3.9.0-SNAPSHOT

Support added in v3.9.0


Prevent removal of other *.dart.js script tags

The following script tag is incorrectly removed by the transformer:
<script src="/js/test.dart.js" type="text/javascript"></script>

This PR will more specifically target the dart.js provided by the browser package.
@sethladd

Thanks!

pushed up the new version. Thanks!

+1

+1


Scrolling with wheel broken in Firefox

You can drag the scrollbar to scroll through list-view, but if you attempt to use the mouse wheel Firefox snaps the scroll location back to the top.

Can you provide an example. Does this affect both virtual and non virtual. Which a versions of ember/list-view does this affect

Yeah seems to be my fault. My bad.

Maybe you can share the cause. So future travelers can solve there issue easiser

So I have spend a bunch of time investigating this and it is related to #229
I am using list-view in a popup and don't want the page to scroll when it runs out of items. So I watch the mousewheel event for the element so I can prevent the default scroll event and do it manually via the scrollTo method on list-view.
In the latest version of list-view some of the internals must have changed such that calling scrollTo in rapid succession causes it to have unexpected behaviour. On firefox it scrolls to the top and in Chrome it causes the scroll event to bubble up to the page and scroll it.
I am still looking for a better way to do this. In firefox watching the DOMMouseScroll event and setting cancelBubble to true does what I want. Have not been able to find a simple solution for Chrome or Safari.

I will also add  that Chrome seems to have a lot worse performance than other browsers. Scrolling really fast shows the rows empty for a split second before the label shows up whereas that does not happen in Firefox or Safari.


Is it possible to specify the config file on the command line?

I tried to use both -c and --config but Ly always picks up the hard-coded config file.

I just found out that it is not possible in v0.4.0 and was added to Ly only recently.


[Feature] Implement Locking to Prevent Simultaneous Runs of the Same dbt Project Across Hosts

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature
I would like to request a feature that enables dbt-core to prevent simultaneous executions of the same dbt project from different hosts. In multi-host environments or when using multiple orchestrators (e.g., Airflow, dbt Cloud, or custom CI/CD pipelines), it is currently possible for multiple instances to trigger the same dbt project, which can lead to conflicts, redundant processing, and potential data integrity issues.
Describe alternatives you've considered
No response
Who will this benefit?

Teams using dbt in production environments with multiple users, orchestrators, or CI/CD pipelines.
Users who want to avoid conflicts or race conditions in dbt runs across multiple hosts.

Are you interested in contributing this feature?
No response
Anything else?
This feature could be implemented with an optional flag (e.g., --enable-locking), allowing users to opt in for environments that require this level of control.

+1!


[Question]: vue-plaid-link or other bindings?

Hi, I'm wondering if you ever plan to also have official supported libraries for frameworks other than react, like Vue in my case?
I found 2 unofficial community libraries but both are still on vue 2. I just ported react-plaid-link (just the LinkToken one, not the deprecated one) to Vue 3 for a side project of mine, so I can already provide you with a start if you want to, but I have no time whatsoever to maintain and update a vue 3 community fork.
Just wondering, since I already made a port :)
Cheers!

@jclaessens97 Thanks for asking! We don't currently have any plans to add additional supported libraries for other frameworks. If you'd like to provide a Vue3 repo for consideration as a third-party wrapper we could link to, that'd be great!

Hi, @phoenixy1, finally found some time.
Please find it here: https://github.com/jclaessens97/vue-plaid-link
and as npm package: https://www.npmjs.com/package/@jcss/vue-plaid-link
Also note that oauth and web3 aren't implemented (yet), and only the non-deprecated flow is implemented.

@jclaessens97 thanks for adding this! There's no need to add web3 support as that flow has been deprecated. However, at this time most user accounts are connected via OAuth flows, so we probably wouldn't link to this package until OAuth support has been added. Once you've got that in, let us know!

Hi, I looked at it again and I already implemented the redirectUri. I just added extra examples for using composables and oauth so now it should be inline with react-plaid-link (apart from the deprecated web3 and public_token flows ofcourse).

@phoenixy1 I also added tests now so I think it should be ready :)

@jclaessens97 excellent! I have a docs update PR in progress that adds this to our community libraries in the docs; I anticipate it will be published this week.

@phoenixy1  Is there a way to follow the progress on your docs pr or is it on a private repo?

@jclaessens97 it's in a private repo. I'll update here when it's published!

Added to the docs: https://plaid.com/docs/api/libraries/#link-client-sdks

Super! Thanks for the update.


hamster fixes + entitystorage tweaks

About the PR

tweaks entitystorage so the storagefill actually properly adds the InsideEntityStorageComponent
tweaks the InsideEntityStorageComponent logic to be more broadly applicable.
Actually gives suffixes to the hamster cages
Makes the hamster cages not airtight so that they can chill in them.
make this one random datafield on entitystorage work

Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame
[ ] This PR does not require an ingame showcase

Changelog

no cl no fun

ok it work better go wild


Update akka-stream, akka-stream-testkit to 2.6.12

Updates

com.typesafe.akka:akka-stream
com.typesafe.akka:akka-stream-testkit

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

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

labels: library-update, semver-patch

Superseded by #243.


Add 'swiftisa' argument attribute [NOT FOR CHECKIN]

@atrick @rjmccall @aschwaighofer behold the madness

Is this for class-method invocation functions?  @atrick and I were talking about this earlier.

@rjmccall It's one possible strategy for implementing resilient class method invocation in a way that works for 'super' calls, yeah.

Yeah, the concerning things about this approach are (1) adding yet more CC complexity and (2) needing to slow down the presumably-much-more-important fast path of dynamic dispatch.

@rjmccall by 'fast path' you mean the case where the method has no overrides and the thunk jumps directly to the implementation without going through the vtable? We do that that unnecessary 'isa' pointer load in that case, or are you thinking of something else?

Oh, I think I was imagining something a little different, where the isa argument would be optional, with nil indicating a normal dispatch.  I'm not sure why I was thinking of that instead of the more obvious "caller always loads the isa first", although ugh, the code-size impact of that.

Anyway, the alternative I was thinking of was maybe having a module/class-combination-specific lookup function that would be keyed by the addresses of the invocation functions published by that module for that class.  The lookup function wouldn't be the most efficient thing, but it's not clear that it would need to be, since this is just used for resilient peer/super dispatch.


Get original exception object in shouldSendCallback

I want to access the original exception object in shouldSendCallback, so I can choose how to log it.
Could we add this?

@OliverJAsh iirc things should actually re-throw by default. How do you have things integrated? i.e. if you're using try/catch you'd effectively want to throw afterwards.

Yeah, rethrowing happens by default from captureException, and it'd be awkward to rethrow inside shouldSendCallback.

For context: https://github.com/getsentry/raven-js/blob/master/src/raven.js#L249

My mistake!

Actually we don't rethrow the error for captureException: https://github.com/getsentry/raven-js/blob/master/src/raven.js#L240-251
(We only throw if the exception is different from the original, i.e. one produced by the internals of TraceKit.report.)
So my original request remains: I would like access to the original error object from inside of shouldSendCallback.

It would also be nice to know if the error was caught or uncaught, i.e., if it came from window.onerror then it is uncaught. This would allow me to add logging for caught exceptions, i.e. by raven.context/wrap:
shouldSendCallback: function (data) {
    if (data.isCaught) {
        console.log('Caught error:', data.originalError.stack);
    }
}

I'm going to close this out as it's very unlikely we'd provide the error object at this point. The use-case is abusing a function which is only intended to say "should I send this upstream or not". More importantly, the error object should long be gone by the time this code path is hit, and if there needed to be something related to that it should happen much earlier in the process.

@dcramer How would you propose logging the error’s captured by Raven (along with their stack traces)?

@OliverJAsh we could add a hook for it, but being that generally we re-raise they should get logged by default in environments. Envs where upstream is catching them would be able to implement their own logging via an abstraction.

Another reason you might want access to the original exception in this callback is because the error might contain metadata which you need to use in your shouldSend logic.
We often catch errors to report them with extra metadata, but we then want to throw them again (to let the browser handle it as an uncaught exception) but prevent Raven from logging the error again when it reaches onerror.

For example:
                        raven.captureException(err, {
                            tags: {
                                feature: 'weather'
                            }
                        });
                        err.reported = true;
                        throw err;

I second this.. our error objects contain metadata which are needed to determining whether to prevent the error from being sent to sentry.

+1

https://github.com/getsentry/raven-js/issues/980


A-Frame style declarative abstraction layer

Hi,
Exciting project!
I really like how you're making this lib reactive and framework agnostic off the bat. It occurs to me though for folks wishing to transition to this from A-Frame it could help to have a declarative abstraction layer over it?
I've been working on an experimental build system for A-Frame using TypeScript, Webpack, and custom elements v1: https://github.com/edsilv/aframe-ts-webpack
Perhaps this could consume ecsy to provide all of the underlying logic, maintaining separation of concerns and giving A-Frame users a way to pick it up quickly?

Also:

Systems have just behaviour, without storing state

We're using Redux to manage state and reactively rendering A-Frame tags in jsx based on that. Seems like your approach would lend itself to that?

I'd be really interested to explore the possibility for a "WebECS" standard HTML/JSON encoding for this and A-Frame at the presentation level. Maybe there's the potential to create a working group. It's probably something the game and web development industries could mutually benefit from.

Closing it. We just updated the goals in this repo https://github.com/MozillaReality/ecsy#goals and even so we think this is a really cool project to build around ecsy, it won't be part of ecsy "core" as is, but instead should be build outside this repo in another one like ecsy-three, ecsy-babylon or so.


Prepare for LLVM 3.7

3.7 is coming out, no major changes to the API, so should be good. Need to publish nugget package and update Generated.cs here

Final RC4 has been tagged: https://www.nuget.org/packages/LLVMSharp/3.7.0-rc4-final

LLVMSharp 3.7 released: https://www.nuget.org/packages/LLVMSharp/3.7.0


[Docs]: Remove hotjar script

Is there an existing issue for this?

[X] I have searched the existing issues

Documentation Link
No response
Discord/slack/intercom Link
No response
Describe the problem and improvement.
We now have smartlook recordings for Documentation. So remove the hotjar script
Remove - microsoft style guide and set readability to 60.

Fixed as part of #1085


build.rs: fix with rustc not built from Git

I couldn't build hyperx because my rustc (from Nixpkgs) has a version output looks like this:
rustc 1.34.2

There's no Git information in parentheses after the version. I assume that this is because it was not built from a Git checkout.
To fix this, I made it so that if no space occurs after the "rustc " prefix, the whole remaining string is used as the version.

Thank for the report and patch. Just to confirm something hadn't more fundamentally changed, I rustup installed 1.34.2 I still get this:
rustc 1.34.2 (6c2484dc3 2019-05-13)
Is Nixpkgs actually building rust from source (and out of git tree?), or is it downloading/verifying a binary from something like static.rust-lang.org?

Yeah, it’s building from an official release tarball.
https://github.com/NixOS/nixpkgs/blob/915c2de37091518daa06f1f9248c8239715ccab9/pkgs/development/compilers/rust/rustc.nix#L22-L25


NeoPixels on PyGamer/PyBadge light up on their own

In the Beta of MakeCode Arcade something changed today and now the NeoPixels on PyGamer and PyBadge are all lighting up bright white on their own (even when Light extension isn't used). This is with both 3.5.0 and 3.6.0 bootloaders. It is ignoring Light calls to try to turn them off.
Interestingly, the PyBadge NeoPixels seem to turn off if USB is disconnected and only battery power is used, but same is not true on PyGamer.

Hey @jedgarpark,
We're on it!

@jedgarpark this is in dev

We should have a build soonish.

Excellent, thanks for the quick fix.


PyTorch implementation

@serengil thanks for this amazing library. Since you know that PyTorch is becoming the mainstream in ML, do you have any plans to implement this library in PyTorch?

No. DeepFace will be tf based library always.


do you know how we can double check the transaction status ?

Hi is there any UPI api for checking the recived payment status ????

i am wondering that how could we doble check the payment status and pamyent related information not other than just transaction id

Unfortunately, I don't think it's possible to fetch the transaction details using transactionID alone without keeping the records in your personal database. I don't think banks will make such information available so easily to public and is the reason why payment gateways exists. Maybe I am wrong but this is my conclusion.


[staging-22.05] systemd: 250.4 -> 250.7

removed the pull/22174.patch, it has been backported in v250,
see https://github.com/systemd/systemd-stable/pull/195
src/test/test-load-fragment.c got some /bin/echo invocations
in https://github.com/systemd/systemd-stable/commit/9727b9ee7b90afb8fa0e6328dcb6c34b1522d4fd
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

I tried building this, but the openldap build/test was broken. I rebased it on the current staging-{,next]}22.05 branches, and the build succeeded there.
I successfully ran nixosTests.systemd. Let's get this in.

thanks


[AMR Heatmap] Add "Metric" filter and "View Level" options to AMR Heatmap

Description
Adds a header and dropdown menu to the AMR Heatmap, with options to change between two metrics (depth and coverage) and select between two view levels (gene and allele). The heatmap will update instantly when a new option is selected.
Notes

Some state is lifted from AMRHeatmapVis to AMRHeatmapView, and so is the request for sample data

Tests

Added Rspec tests for index method (amr_heatmap endpoint)
Client-side code tested manually through local machine

🌶 🗺 !!!
A lot of these comments are based off my own coding preferences so feel free to take them with a grain of salt! 🧂

I appreciate them all the same :)

Looks like it's at a good state now, time for review!

^ For manually testing, you could write a little test plan like "Go to this URL, click the dropdown, change the filter to coverage, see coverage data" for the interactions.

Made some comments. Looks great overall!

Made some comments. Looks great overall!

awesome heatmap! 🎉


v4-beta.18 - Error: Template parse errors: 'ion-tab' is not a known element

Bug Report
Ionic version:

[x] 4.x
Current behavior:

Update from beta.17 to beta.18
Expected behavior:

Site would work
Steps to reproduce:

Related code:
A sample application via GitHub
Ref: https://github.com/peterennis/aetabs/commit/ed7286315edc575b5bfa7691c710868903ee1b6e
Current site running beta.17 : https://aetabs.adaept.com

Result running beta.18

Other information:

I am using tabs with sidemenu.
There is no official sample.
I read breaking changes, tried the suggestions, but could not get it to work.
@manucorporat
Possibly related: https://github.com/ionic-team/ionic/issues/16343
Ionic info:

C:\ae\adaept.com\aetabs>ionic info

Ionic:

   ionic (Ionic CLI)             : 4.5.0 (C:\Users\peter\AppData\Roaming\npm\node_modules\ionic)
   Ionic Framework               : @ionic/angular 4.0.0-beta.18
   @angular-devkit/build-angular : 0.11.3
   @angular-devkit/schematics    : 7.1.3
   @angular/cli                  : 7.1.3
   @ionic/angular-toolkit        : 1.2.0

Cordova:

   cordova (Cordova CLI) : 8.0.0
   Cordova Platforms     : none
   Cordova Plugins       : no whitelisted plugins (2 plugins total)

System:

   NodeJS : v10.14.1 (C:\Program Files\nodejs\node.exe)
   npm    : 6.4.1
   OS     : Windows 10

ion-tab is not a known element in beta.18 / beta.19. Remove this and read the breaking changes again -> https://github.com/ionic-team/ionic/blob/master/CHANGELOG.md#angular-tabs
There you see that you have to remove the ion-tab. I close this here but you can still comment here if you don't get it work

Sorry, I miss something here for the angular part: following your advice, I get this error:
ERROR Error: Uncaught (in promise): Error: Cannot match any routes. URL Segment: 'tabs/tab1'
Error: Cannot match any routes. URL Segment: 'tabs/tab1'
Did you change something respective to path and outlet ?
const routes: Routes = [
{
path: 'tabs',
component: TabsPage,
children: [
{
path: '',
redirectTo: '/tabs/(tab1:tab1)',
pathMatch: 'full',
},
{
path: 'tab1',
outlet: 'tab1',
component: Tab1Page
},
{
path: 'tab2',
outlet: 'tab2',
component: Tab2Page
},
{
path: 'tab3',
outlet: 'tab3',
component: Tab3Page
}
]
},
{
path: '',
redirectTo: '/tabs/(tab1:tab1)',
pathMatch: 'full'
}
];

Thanks for your response! I can still tell you to look at the breaking changes :) https://github.com/ionic-team/ionic/blob/master/CHANGELOG.md#angular-tabs
Remove all outlets and fix the redirectTo

ok my mistake, was looking at this:
https://github.com/ionic-team/ionic/blob/master/angular/BREAKING.md


Review logging in SSETransport on the client

catch and log exceptions in receive loop (and rethrow)
log in readCancellationRegistration
log before _application.Out.TryComplete();

We have another issue for improving client


"Compiled dbt preview" seems not work

Expected behavior
When I click on "Compiled dbt preview" dbt compiles the opened model file and shows the compile output in a new window.
Actual behavior
When I open a model file in VS code and press on "Compiled dbt preview" nothing happens. No hint for any errors is visible. It simply doesn't open the preview window. For example "Run dbt model" works as expected.
Steps To Reproduce
Function never works for me.
Log output/Screenshots
No response
Operating System
Windows 10 Version 21H2 (OS Build 19044.2130
dbt version
Core:   - installed: 1.3.0      - latest:    1.3.0 - Up to date!  Plugins:   - sqlserver: 1.3.0 - Up to date!
dbt Adapter
dbt-sqlserver
dbt Power User version
0.8.1
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

There should be an error if the Python process can't be started.

I had a similar problem on 0.8.4: clicking the preview button would do nothing and a few seconds later an error would pop up about not being able to start the Python bridge. I installed 0.8.3 instead and the compiled dbt preview now works.

There should be another error before that indicates why the python process couldn’t be started. Please provide that error.

Closing as not reproducible. If you have more details, feel free to reopen it.

I searched in the extension logs folder and found this error message. Does this help?
2022-11-09 08:50:41.363 [error] Error: cannot open query-preview:/c%3A/Development/gpc/dbt/dbt-demo-sqlserver/models/dimensions/DM_D_CostType.sql. Detail: Unable to resolve text model content for resource query-preview:/c%3A/Development/gpc/dbt/dbt-demo-sqlserver/models/dimensions/DM_D_CostType.sql
at a.$tryOpenDocument (vscode-file://vscode-app/c:/Program%20Files/Microsoft%20VS%20Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2779:76347)
at process.processTicksAndRejections (node:internal/process/task_queues:96:5)

This error message seems about the query preview feature and not about the compilation feature.
Can you record a video of the extension starting and the actions you perform to validate if things are working.

Here you are. I hope this helps.
https://user-images.githubusercontent.com/20833767/200820143-b7032f50-027e-4d6b-9e6d-043743cab8ed.mp4

I don't think you are running the latest version. There shouldn't be a console window popping up. Can you verify that?

I'm running 0.8.6. Details below
VS Code Version:
PS: > code --version
1.73.0
8fa188b2b301d36553cbc9ce1b0a146ccb93351f
x64
Extension Versions:
PS: > code --list-extensions --show-versions
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Python version:
PS: > python --version
Python 3.10.8

Fixed in 0.8.7
What's weird is that you still get the console window. I actually tested that on windows and was able to let it dissapear by making it a daemon process. Could you try to get some process information about that Python console window?

Version 0.8.7 fixed the issue. Now preview is working again. Thanks!
And here the info about the python processes. Actually two processes are started because I am using a virtual environment. I piped process info to the attached python.json.txt file.
PS: > Get-Process python | select -Property * -ExcludeProperty Parent, Modules, Threads | ConvertTo-Json -Depth 5 | Out-File python.json.txt
python.json.txt
Can it be related to the fact that I am running on Windows and in my workspace settings.json is set to "dbt.dbtPythonPathOverride": "../dbt-env-sqlserver-stable/Scripts/python.exe" ?
For me the path in the prompt title looks a little weired because it is mixed with / and \

Could you try selecting the python environment with ctrl shift p -- select python interpreter and remove the dbt.dbtPythonPathOverride?

That worked.
I removed dbt.dbtPythonPathOverride from my workspace settings and and selected the interpreter with ctrl shipt p. When I entered the path manually and browsed for the interpreter I needed to make sure to select pythonw.exe instead of python.exe.
When I selected the recommend python it worked also.

Yes but do you still get the popup console window?

https://stackoverflow.com/questions/9705982/pythonw-exe-or-python-exe
Ok that explains it. Thx for helping to investigate!


[AutoPR cost-management/resource-manager] [Hub Generated] Review request for Microsoft.CostManagement to add version stable/2019-01-01

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/5612

(message created by the CI based on PR content)

Codecov Report

Merging #4825 into restapi_auto_cost-management/resource-manager will decrease coverage by <.01%.
The diff coverage is n/a.

@@                                Coverage Diff                                @@
##           restapi_auto_cost-management/resource-manager    #4825      +/-   ##
=================================================================================
- Coverage                                          53.38%   53.37%   -0.01%
=================================================================================
  Files                                              10458    10454       -4
  Lines                                             219472   219461      -11
=================================================================================
- Hits                                              117159   117148      -11
  Misses                                            102313   102313

Impacted Files
Coverage Δ

...azure/mgmt/hanaonazure/models/hana_instance_py3.py
100% <0%> (ø)
:arrow_up:

...-luis/azure/cognitiveservices/language/__init__.py

...suggest/azure/cognitiveservices/search/__init__.py

...ledge-qnamaker/azure/cognitiveservices/__init__.py

azure-mgmt-advisor/azure/mgmt/__init__.py

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 8cb1468...8d90f65. Read the comment docs.

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/2924


Setup and documentation

I finally found the time to take a deeper look. I tried to run it on Debian 11 without using Docker (as this is what we run our infrastructure on). These are my findings so far:

libgdal-dev is needed to run, we maybe should add that to the documentation if Docker is not used (together with libpq-dev)
We also need postgresql-13-postgis-3-scripts (or the respective package for other postgresql versions)
I don't like the use of default users (okay) and passwords (definitely not okay). Not only may it lead people to use them. It also creates the danger of accidentally forgetting to set/reconfigure any of the default values which may cause security problems. We should not provide default passwords and enforce the explicit setting of any value required for authentication.
Making the database user a super user just to enable the extension is overkill. I think we should manually enable the extension when the database is created and the database user should not be a super user (neither in production nor testing).
I tried to scrape but Frankfurt is currently broken (if you just converted our old modules that's no surprise, it has been broken for a while now). I noticed that this seems to break the whole scraping process. This should not happen. Sometimes there are regularly problems with some data sources, this should not impact the scraping of any other sources.
The jsonschema version referenced in web/scrapers/builtin/requirements.txt doesn't seem to exist. According to PyPI, only 3.1.1 and 3.2.0 exist, but not 3.2.1. Is there anything that prevents us from using the latest version (4.3.2)? I tried that and for the things I tested, it worked.

Hi jk, glad you eventually jumped in ;)

Yes, the gdal and other utilities are needed. The README links to the install docs on the django page but i'll add the mentioned libraries explicitly.

I'll remove default user/password settings and actually will rewrite the configuration using python-decouple. They support little .env files and overwriting via command-line params.

Yeah, i don't like the postgres superuser thing either. For the real database, whether in production or local dev, it's no problem to enable the postgis extension once and the leave it. The only purpose for the superuser was to ease the unittest run.  It's possible to run ./manage.py test --keepdb though. I'll check that and update the REAMDE.

I copied the Frankfurt code and made some adjustments, mainly the source url. I think they are rotating the resource uuid or something. Possibly the scraper needs to get current url from their webpage.
The scraping of other pools should not be affected, though. When running ./manage.py pa_scrape scrape the stacktrace is just printed to the console but the other scrapers work. You can verify with ./manage.py pa_stats -t 2m to see the number of errors and successful  data collections within the last 2 minutes (-t 2m ).
Btw. the ffh pool does also currently fail and i think it can be removed completely as they only show cities which are already covered. And without any metadata.

jsonschema could sure be the newest version, don't remember how i came to the version number in the requirements. Actually now that i see that my venv uses 4.2.1 it's quite obviously a typo.

One more thing. If docker is not used in our production i might as well remove the Dockerfile and docs for now. I'm not happy with the ubuntu image and there should also be a setup to dockercompose with the database container. It takes some work and is not immediately relevant.

Like the commits say:

decouple is now managing the environment settings and there's a little example file with helpful comments.
no default user and password settings
no postgres superuser required

The setup requires to copy and adjust a few lines of postgres statements. Maybe we should supply a setup file similar to your ./bin/parkapi-setupdb
I also removed the Docker part because i'm in no mood right now to test and verify it

The Frankfurt data source simply does not deliver data right now (https://offenedaten.frankfurt.de/dataset/parkdaten-dynamisch).
Please check if everything regarding this issue is working so far and then close it if you like. Next time i'll put everything on a branch.
Wünsche frohe Festtage und so :D
:milky_way:

Haha, have put the wrong number on the commits, they are all supposed to relate to issue #4.

So, after a long time, I tried to run the server again ;)
I'm able to run the server now. I like the solution to use environment variables instead of default credentials. Yet I had to figure may way through the error messages of missing parameters so it might make sense to have them documented somewhere. E.g. I didn't really know what DJANGO_SECRET_KEY is used for.
Unfortunately I wasn't able to run the scraper itself. After installing all dependencies from the requirements.txt files it failed with the following error:
./manage.py pa_scrape scrape
module 'builtin' scraping all pools

ERROR in module builtin:
 Traceback (most recent call last):
  File "/.../ParkAPI2/web/scrapers/builtin/scraper.py", line 187, in <module>
    main(**parse_args())
  File "/.../ParkAPI2/web/scrapers/builtin/scraper.py", line 135, in main
    scrapers = get_scrapers(pool_filter=pools)
  File "/.../ParkAPI2/web/scrapers/builtin/scraper.py", line 77, in get_scrapers
    module = importlib.import_module(module_name)
  File "/usr/lib/python3.9/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 986, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 680, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 790, in exec_module
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "/.../ParkAPI2/web/scrapers/builtin/new/bahn.py", line 12, in <module>
    from decouple import config
ModuleNotFoundError: No module named 'decouple'

I tried installing decouple from pip but then got the following error:
Traceback (most recent call last):
  File "/.../ParkAPI2/venv/lib/python3.9/site-packages/django/core/management/__init__.py", line 237, in fetch_command
    app_name = commands[subcommand]
KeyError: 'pa_scrape'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/.../ParkAPI2/web/./manage.py", line 22, in <module>
    main()
  File "/.../ParkAPI2/web/./manage.py", line 18, in main
    execute_from_command_line(sys.argv)
  File "/.../ParkAPI2/venv/lib/python3.9/site-packages/django/core/management/__init__.py", line 419, in execute_from_command_line
    utility.execute()
  File "/.../ParkAPI2/venv/lib/python3.9/site-packages/django/core/management/__init__.py", line 413, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/.../ParkAPI2/venv/lib/python3.9/site-packages/django/core/management/__init__.py", line 244, in fetch_command
    settings.INSTALLED_APPS
  File "/.../ParkAPI2/venv/lib/python3.9/site-packages/django/conf/__init__.py", line 82, in __getattr__
    self._setup(name)
  File "/.../ParkAPI2/venv/lib/python3.9/site-packages/django/conf/__init__.py", line 69, in _setup
    self._wrapped = Settings(settings_module)
  File "/.../ParkAPI2/venv/lib/python3.9/site-packages/django/conf/__init__.py", line 170, in __init__
    mod = importlib.import_module(self.SETTINGS_MODULE)
  File "/usr/lib/python3.9/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 986, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 680, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 790, in exec_module
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "/.../ParkAPI2/web/park_api/settings.py", line 14, in <module>
    from decouple import config
ImportError: cannot import name 'config' from 'decouple' (/.../ParkAPI2/venv/lib/python3.9/site-packages/decouple/__init__.py)

Do you have any idea what I'm missing?

Okay..., what might be the problem here?
The requirements.txt lists python-decouple which is the correct package. Also all versions are pinned so when installing the env with pip install -r requirements.txt it should be more or less the same on all systems.
For the .env parameters: The .env.example file contains links to the django documentation to explain what the django-specific settings are.
Just from outside i'd argue you did not follow the steps in the README exactly (:wink:)
I did a clean install just to test it. Apart from the wrong version number for jsonschema (which we discussed earlier) all seems fine. i fixed the requirements.txt file.
Some scrapers have problems though (heilbronn, hanau, hamburg, and frankfurt still!).

Okay, I just checked what is installed and the error is really weird. I actually have python-decouple installed and I can import it in a python shell. I even added the import statement to manage.py and that doesn't generate an error. It seems that specifically the import in ParkAPI2/web/scrapers/builtin/new/bahn.py generates this error however it's not clear to me why. I also checked the interpreter and it's using the correct one (I'm working in a virtual env). Which python version do you use?

Weird indeed! I'm using python 3.8 locally. The only thing i can imagine is that pa_scrape is calling the wrong python executable. There is a new option to show the executed commands:
If calling ./manage.py pa_scrape scrape -p bahn -v 2 it displays:
running .../ParkAPI2/env/bin/python3.8 scraper.py scrape --pools bahn in directory .../ParkAPI2/web/scrapers/builtin
which is the correct enved python in my case. It's determined in park_api/management/commands/pa_scrape.py:159 using sys.executable.
My env was created with virtualenv -p python3.8 env (it's an older ubuntu where i have to state python3 explicitly).
Generally, without calling source env/bin/activate, i'm still able to use the environment (e.g, from a cronjob) via
ParkAPI2/web$ ../env/bin/python manage.py pa_scrape scrape -p bahn -v 2 and it works.
Can you check that at least the raw scraper is working (without the subprocess inside pa_scrape.py), e.g. with:
ParkAPI2/web/scrapers/builtin$ python scraper.py scrape -p bahn

After setting the project up twice now for testing by only using the README the only issue I encountered is fixed in #15. For the time being I don't see any deal breakers so I think we can close this issue.


A11y - Autocomplete does not announce edit box when wrapping through options

Bug, feature request, or proposal:
Bug: Screen reader accessibility Edit box focus issue.
What is the expected behavior?
The up/down arrows will wrap through the text box when the top or bottom of the list is reached. For example, pressing the down arrow when the last choice is highlighted will move focus back to the textbox, pressing down again will move focus to the first item in the list (Alabama). Likewise, with focus in the textbox and the list displayed, pressing up arrow will move focus to the last item in the list (Wyoming).
What is the current behavior?
Currently the Edit box cannot get focus when pressing the arrow keys.
What are the steps to reproduce?
Use the JAWS screen reader and navigate by using the keyboard only (disconnect the mouse).
What is the use-case or motivation for changing an existing behavior?
WCAG standard keyboard controls provide for a consistent behaviour and allow screen reader interaction.
Which versions of Angular, Material, OS, browsers are affected?
Windows10 browsers and screen reader used for testing.
What else should we know
A11y Issue:
The screen reader appears never to receive Edit box focus. The arrow keys move focus to the first and last list item, bypassing the Edit box. Although the user can type at any time, it is not obvious that the focus is on the Edit box at any time.

@DaveBest99 Let's say there is an "A" in the edit box, and you've navigated to the last option in the list. When you hit the DOWN arrow, what would you expect to hear?  Would it be "A, 4 items in list" ? Or "State edit box" ?

Some developers allow the Edit box to be included in the List box rotation; between the last list item and the first list item (WAI ARIA1.0 best practices):

When on the last item in the list Down arrow will place focus on the Edit field.

Pressing Down arrow again moves focus to the first item in the list.

I recommend the Github issue 3416 solution (WAI ARIA1.1 best practices):

When focus is on the last item in the list pressing Down arrow moves focus to the first item in the list.

Pressing Alt+Up_arrow, Enter key, or Escape key moves focus to the Edit field.

Github3416 steps:

Pressing Down arrow or Alt+Down_arrow should open the List box with the focus placed on the first item in the list "Alabama" (the Edit input field is no longer visible on the braille display, but instead "Alabama 1 of 4" and the speech output "Alabama 1 of 4").

Pressing the Down arrow the screen reader speech/braille should announce "Alaska".

Pressing Down arrow again the user should hear/see "Arizona".
Pressing Down arrow again the user should hear/see "Arkansas".
Pressing Down arrow again the user should hear/see "Alabama" (focus moves from last item to the first item).
Pressing Up arrow the user should hear/see "Arkansas" (focus moves from first item to the last item).
Pressing Home key should move focus to the first List item.
Pressing End key should move focus to the last List item.
6A. Pressing the Enter key on any list item should close the List box, move focus to the Edit input field, and place the selected item in the Edit input field. The Edit input field should have focus and the screen reader will speak the content of the field and show it on the braille display. The user can press Left and Right arrow keys to read the text character by character.
6B. Or, if the user presses the Escape or Alt+Up_arrow key, then the List box is closed and the focus is moved back to the Edit input field with the originally typed characters. No List item is selected, and the speech out reads the typed characters and shows them on the braille display with the cursor at the end of the character string. The user can press Left and Right arrow keys to read character by character.
Does this make any sense? Let me know and we can discuss.
David
From: Kara<EMAIL_ADDRESS>Sent: March 3, 2017 07:22 PM
To: angular/material2
Cc: David Best; Mention
Subject: Re: [angular/material2] A11y - Autocomplete does not announce edit box when wrapping through options (#3291)
@DaveBest99 https://github.com/DaveBest99  Let's say there is an "A" in the edit box, and you've navigated to the last option in the list. When you hit the DOWN arrow, what would you expect to hear? Would it be "A, 4 items in list" ? Or "State edit box" ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/angular/material2/issues/3291#issuecomment-284106978 , or mute the thread https://github.com/notifications/unsubscribe-auth/AVPBNm-63GmgChzg8AYR-yJB3jPbFfC0ks5riK6rgaJpZM4MLoVT .  https://github.com/notifications/beacon/AVPBNrD2FpdZgeE6jEyU5mNrMUA3HWslks5riK6rgaJpZM4MLoVT.gif


Consider dropping sjs 0.6.x support

This would simplify the build further.

If we drop sjs 0.6.x, we have to drop Scala 2.11 too. Would be fine, but then we'd have to drop Scala Native too 😞
I guess that's the trend, so we should.


Fix issue-585: Remove duplicate stat.h constants defined in constants.js

Fix issue-585
After removing
  S_IFREG: 0x8000,
  S_IFDIR: 0x4000,
  S_IFLNK: 0xA000,

all tests are passed. Because I fixed the both issue-588 and issue-585, I found out that when I have the code without the fix issue-588, when I run npm test command, it cannot launch Firefox, as show in image below:

So I think maybe have a relation between them. Just let you know that.

You need to run npm install and also have Firefox installed to run Firefox headless.

Fixed the callers in src/node.js to the correct version, let me know if I have done something wrong.


Scope hygiene

Massive Hygiene Simplification
So @mflatt of Racket fame has recently been working on a simplified hygiene algorithm for Racket that uses scope sets rather than marks and renames. His notes outlining the approach are here. It's crazy good.
Rather than do what I really should be doing (writing my thesis) I've hacked it up in sweet.js.
Previously sweet.js was using the standard Scheme/Racket approach of marks/renames and definition contexts. This was always hard to understand how it worked and we had a number of bugs that I could never track down. resolve was also super slow.
The basic idea of scope sets is that every syntax object is marked with the set of scopes it is in and each binding form (e.g. function) creates a new scope:
function foo(x^{a}) {
    function bar(x^{a, b}) {
        return x^{a, b};
    }
}

There is a global map that associates each identifier and scopeset with a new binding so the above might have a map like:
{
    x: [{
        scopeSet: [a],
        binding: 1
    }, {
        scopeSet: [a, b],
        binding: 2
    }]
}

and fully expanding it with resolve would then give
function foo(x1) {
    function bar(x2) {
        return x2;
    }
}

There are a few more details about macro invocation, recursive macro definitions, and let/var but that is the basic idea.
The big win here is that hygiene is mostly understandable now (check out how tiny resolve is now!) and faster (even without the memoization we were doing in the old resolve).
A few other things snuck into this PR.
New names for macro definitions
The current let name = macro {} is bad. I proposed some new syntax ideas in #437 and this is the start of implementing them. Internally there is now only one macro definition form stxrec which as the name suggests is recursive. On a suggestion from @mflatt the non recursive form is then built from the recursive form (implementation is here).
So right now the forms look like:
stxrec m {
    rule {} => {}
}
stxnonrec m {
    rule {} => {}
}

The stxnonrec is obviously a terrible name and just a placeholder. I think we want:
stxrec m = macro {
    rule {} => {}
}
stx m = macro {
    rule {} => {}
}
// using the `x = macro {}` is suggestive with normal `let`/`var`
// and also allows us to smoothly use the same syntax
// to put things other than macros into the compiletime env
stx x = 42;
stx x = function(x) { return x; };
// with maybe a convenient macro declaration form
macro m {
    rule {} => {}
}

Macro definitions no longer break hygiene to capture match
Probably fixes #430 and others.
So sorry I ever did that :(
macroclass currently doesn't work because it was breaking hygiene and I haven't updated it yet.
localExpand takes a stop list
At the moment it works by matching the name and the next delimiter token.
Phasing
Racket has begin-for-syntax that let's you define stuff for use inside a macro without going to the trouble of pulling it out into a separate module and importing it for syntax. I've added forPhase to do the same:
forPhase 1 {
    stxrec id {
        rule { $x } => { $x }
    }
}
stxrec m {
    case {_ } => {
        var x = id 42;
        return [makeValue(x, #{here})];
    }
}

There's probably some cleaner syntax we could come up with.
You can import into an arbitrary phase now:
import { m } from "mod.js" for phase 3;

That sounds absolutely amazing :-).

This is looking great! Bikeshed time.
Maybe instead of forPhase we could have a declaration sort of like export var or export function.
phase 1 var foo = 42;
phase 2 function foo() { return 42 }

It also mirrors the import syntax. I personally dislike nesting declarations in {} that can be used outside of it, because it feels like it breaks lexical scope. But that may just be me.
I also don't really like stx for a keyword, especially since we commonly use it as an identifier in macros. Could we just use the full word?

Bikesheds are the best!
Agreed forPhase indentation is the worst.
Some thoughts on phase:
// good but...
phase 1 var foo = 42

// we have a convenient for phase 1 import form
import fs from 'fs' for macros

// kinda odd
macros var foo = 42
// also odd
for macros var foo = 42
// definitely odd
macros macro m {
    ...
}

// crazy?
var foo for phase 1 = 42
var foo for macros = 42
macro m for macros {
    ...
}
macro m for phase 1 {
    ...
}
macro m phase 1 {
    ...
}
var foo @ phase 1 = 42, bar @ phase 3 = 100;
var foo @ macros = 42
var @ phase 1 foo = 42;
var @ macros foo = 42;

Yeah I've gone back and forth on stx vs syntax. stx is nice since it has the same length as var and let. Also stxrec is nice but syntaxrec feels pretty long. Not really sure where I'm landing on this one right now.
stx foo = 42;
stxrec foo = 42;

syntax foo = 42;
syntaxrec foo = 42;

Another phase thought, if we go with the phase as export form then macros could be an alias to 1. Makes it a bit more declarative.
phase macros var foo = 42;
phase templates var foo = 42;
// maybe not so bad?
phase macros macro m {
   ...
}
phase templates macro m {
  ...
}

You can just lose the s.
import foo from 'foo' for macro;
import goo from 'goo' for template;
import hoo from 'hoo' for phase 5;

macro foo {
}

macro var bar = 42;
macro function baz() {
}

template var a = 12;

phase 5 var b = 88;

But then to put a macro declaration in phase 1 you have macro macro m { }.

Or maybe that's just not allowed? Have macro stx foo = macro {} (or syntax if we go with that) but no standalone macro m {} form?

Superseded by #485


Oppgrader storybook og react-test-renderer

Coverage remained the same at 46.106% when pulling 5702c65cc98a7b11777d4dd054b861047b425cc3 on bjornreppen:story-up into 9aa8b77885af3aaa0114f35d021923937d2ab3bb on Artsdatabanken:master.


i18n(de): update basics/project-structure.mdx

Description (required)

update basics/project-structure.mdx

Related issues & labels (optional)

#9572
#8953
#8239
#7182
#6620
#5880
#4166
#3898
#3237: the assets section has been removed again, i guess?
#2546
#2339
#2133
#2178 (ASCII art is very funny Chris)

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File
Note

de/basics/project-structure.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


feature(dspy): Add MyScale in Retrieve

Add MyScaleRM to support retrieval for MyScaleDB:

MyScaleDB: https://github.com/myscale/myscaledb

Hi @usamajamil43, thanks for the contribution! left a few comments in the PR. Noticed some todos so feel free to mark the PR as ready to merge once the changes are made.
Couple of additional changes:

can you run ruff check . --fix-only to fix the failing test?
could you add documentation for MyScaleRM to the Retrieval Model Clients API? Feel free to follow some of the example RM documentation for reference.

Hey @arnavsinghvi11 , I have fixed the issues and added the documentation. Kindly, take a look

Thanks @usamajamil43 ! LGTM. Unfortunately, I cannot test the caches for this myself, but could you verify that with the existing behavior before pushing? Also, the PR has a few merge conflicts to resolve but should be ready to merge following that!

Hi @usamajamil43 , could you run ruff check . --fix-only and commit again? Ready to merge after that!

Hi, @arnavsinghvi11 Done!

Hi @usamajamil43 , can you rebase to the latest version on main? should fix the failing ruff test and ready to merge.

Hey @arnavsinghvi11,
I've tried to resolve the conflicts, but I'm still getting conflicts in files like lm.py and evaluate.py, which I haven't worked on. I'm not sure which changes to keep and which to discard in these files.

Hi @usamajamil43 , I believe if you rebase to the latest commit on the main branch and then run ruff, this should fix the existing error. I tried doing this by pulling your branch locally which resolved it, but I don't have access to push changes.

@arnavsinghvi11  Done!

@arnavsinghvi11  Can you please take a look? I was busy, so @ahsansaeed878 did the merge.

Thanks @usamajamil43 and @ahsansaeed878 for your patience with the PR! merged.


[SPARK-39546][K8S] Support ports definition in executor pod template

What changes were proposed in this pull request?
This pr changes the fabric8 k8s method used in BasicExecutorFeatureStep about setting blockmanager ports. Because It will override port definitions in executor pod template file.
Why are the changes needed?
We use spark.kubernetes.executor.podTemplateFile to specify executor pod definition. However, we cannot set port definition to pod. For example, this is my executorPodTemplate.yaml
apiversion: v1
kind: Pod
spec:
  containers:
    - name: spark-kubernetes-executor
      ports:
        - containerPort: 8090
          name: jmx-exporter
          protocol: TCP

When executors start, the pod definitions only show default blockmanager port.
      ports:
        - containerPort: 7079
          name: blockmanager
          protocol: TCP

It looks like a bug, because the BasicExecutorFeatureStep uses withPorts to override all port definitions.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Add unit test

Can one of the admins verify this patch?

cc @dongjoon-hyun could u take a look? thanks

Hi @dongjoon-hyun , please take another look

Welcome to the Apache Spark community, @fanyilun . I added you to the Apache Spark contributor group and assigned SPARK-39546 to you. Do we need this patch for driver pod template? If then, please feel free to make another JIRA and continue.

Thanks, driver pod template already supports ports definition. It works for me.


Create Github Actions to test/lint Docker container for application

Need to run hadolint and some security scanning on the container.

There is a action for hadolint https://github.com/marketplace/actions/hadolint-action

This is probably okay now.


Analyzer Proposal - Delegate Registration should have an As

Based on the current Autofac guidance, here, delegate registrations should have an As<TService> call to inform the limit type.
Proposed Behaviour
Where a registration is declared from a delegate, using one of the two Register extension methods, if that same registration is determined to not have an As call, a warning will be raised.
For example:
class MyClass : IMyService { }

interface IMyService { }

static void Main(string[] args)
{
    var builder = new ContainerBuilder();

    // Warning Generated on this registration call.
    builder.Register(c => new MyClass());

    // No Warning Generated on this registration call.
    builder.Register(c => new MyClass()).As<IMyService>();
}

Possible Intelligent Detection

[ ] Only raise the warning if the class derives from an abstract class or interface?

@nblumhardt, @tillig for comments.

OK, re-reading the documentation with an adequate increase in attention, I see that the intent is more about how we can't figure out if it's IComponent or Component that you're registering with a lambda like
// Component implements IComponent
builder.Register(c => new Component);

that is, it's implicitly AsSelf() and folks may not realize that. I'm guessing we probably had some frequent questions or confusion about that at some point, though it's been a while so I don't remember.
I think the idea about being more intelligent is reasonable - if it derives from an abstract class or interface, being explicit is reasonable (either include As<T>() or AsSelf()) where if it doesn't, the implicit AsSelf() stands.

Ok, if it derives from an abstract class or an interface, we'll encourage people to use As<T> or As<TSelf>. That should be pretty doable.

Sounds great 👍

(Probably need to whitelist IDisposable)

(Probably need to whitelist IDisposable)

Can you elaborate slightly? You mean that returning an IDisposable from the lambda should not require AsSelf/As? Since IDisposable doesn't derive from any interfaces, the standard rule should cover it?

I think the intention of applying the check for components that derive from an ABC or interface is that the ABC or interface might represent a service; since IDisposable isn't used as a service, the As call shouldn't be needed if it's the only interface a component supports. HTH!

Oh, yes, I understand now. Better add IAsyncDisposable to that as well.


Lack of recognition of the best pasta shape

Fusilli should be fully recognised as the finest pasta shape that there is. Therefore, the background should be a picture of a singular fusilli.

Penne rigate is underrated -- consider this change as well, since the ridges in penne rigate absorbs the sauce.


HBASE-25806 Backport the region location finder initialization fix in…

… HBASE-25802

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 44s
Docker mode activated.

_ Prechecks _

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

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

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

_ branch-2.3 Compile Tests _

+1 :green_heart:
mvninstall
5m  1s
branch-2.3 passed

+1 :green_heart:
compile
4m 10s
branch-2.3 passed

+1 :green_heart:
checkstyle
1m 28s
branch-2.3 passed

+1 :green_heart:
spotbugs
2m 46s
branch-2.3 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 27s
the patch passed

+1 :green_heart:
compile
3m 55s
the patch passed

+1 :green_heart:
javac
3m 55s
the patch passed

+1 :green_heart:
checkstyle
1m 31s
the patch passed

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

+1 :green_heart:
hadoopcheck
19m 17s
Patch does not cause any errors with Hadoop 2.10.0 or 3.1.2 3.2.1.

+1 :green_heart:
spotbugs
2m 26s
the patch passed

_ Other Tests _

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

55m  6s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3196/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3196

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile

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

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.3 / e6cb7f211a

Default Java
AdoptOpenJDK-1.8.0_282-b08

Max. process+thread count
96 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3196/1/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 50s
Docker mode activated.

-0 :warning:
yetus
0m  6s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2.3 Compile Tests _

+1 :green_heart:
mvninstall
4m 45s
branch-2.3 passed

+1 :green_heart:
compile
1m 16s
branch-2.3 passed

+1 :green_heart:
shadedjars
5m 16s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 39s
branch-2.3 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 13s
the patch passed

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

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

+1 :green_heart:
shadedjars
5m  3s
patch has no errors when building our shaded downstream artifacts.

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

_ Other Tests _

+1 :green_heart:
unit
138m 10s
hbase-server in the patch passed.

163m  7s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3196/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3196

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux c6712a41ab68 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.3 / e6cb7f211a

Default Java
AdoptOpenJDK-1.8.0_282-b08

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3196/1/testReport/

Max. process+thread count
3745 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3196/1/console

versions
git=2.17.1 maven=3.6.3

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

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
2m 45s
Docker mode activated.

-0 :warning:
yetus
0m  5s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2.3 Compile Tests _

+1 :green_heart:
mvninstall
5m 50s
branch-2.3 passed

+1 :green_heart:
compile
1m 22s
branch-2.3 passed

+1 :green_heart:
shadedjars
7m 13s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 48s
branch-2.3 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
5m 20s
the patch passed

+1 :green_heart:
compile
1m 17s
the patch passed

+1 :green_heart:
javac
1m 17s
the patch passed

+1 :green_heart:
shadedjars
7m 12s
patch has no errors when building our shaded downstream artifacts.

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

_ Other Tests _

+1 :green_heart:
unit
200m 39s
hbase-server in the patch passed.

235m 35s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3196/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3196

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 2823edee871d 4.15.0-126-generic #129-Ubuntu SMP Mon Nov 23 18:53:38 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.3 / e6cb7f211a

Default Java
AdoptOpenJDK-11.0.10+9

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3196/1/testReport/

Max. process+thread count
2921 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3196/1/console

versions
git=2.17.1 maven=3.6.3

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

This message was automatically generated.


MathiasMagnus SAXPY tutorial

The SAXPY tutorial.

IMAGE OF FINAL ALGO is missing

IMAGE OF FINAL ALGO is missing

Fixed.

Added language suggestions from @lpaoletti. Review the diff if needed 2b4a8be

@lpaoletti Can I get an approval?

The reduction tutorial PR: #3513

Should we remove the tab sync?
I don't like how the page jumps around when you select a tab due to content lengths changing everywhere at the same time.

Should we remove the tab sync?
I don't like how the page jumps around when you select a tab due to content lengths changing everywhere at the same time.

Probably we should. I am thinking what we can do with it.

I think we should keep it. I think a user is going to pick their flow in the first tab, and the page will update the content to display their flow. I don't think people will be flipping back and forth between flows except to see what differences there might be.

Looks good

Thanks for the review.


fix($parse): handle null targets at assign

When assigning to a sub property of a property that its value is null then write an empty object to the property that will contain the sub-property

Just checking that the value is falsy will also cause false/0/''/... to be replaced with {}. Is that correct? Seems weird when 0 will be replaced but 1 wouldn't...

@jbedard this follows the same logic that we had before, and the change in the logic is causing some regressions. We need this as this regression is blocking the next beta. Later we can tweak the logic to whatever we want.

Yea it is the same as before, so lgtm. Might be something worth changing in 1.4 though...

landed as 2e5a7e52a0385575bbb55a801471b009afafeca3


Validate TypedDict

Support of creating TypedDict instances (from both mypy_extensions and typing) from normal Dict.
Do not break compatibility with 3.6
Do not add new dependencies

What do you mean about 'creating'? Give an example please


Add --no-aliases and --no-names switched to node list command

To be able to produce the same listings like the old CLI tools we should add options for hiding the node names or the node aliases within the node listing.

Fixed with #32


Format file

i'm trying implement library
here is me eslintrc.js
module.exports = {
  root: true,
  extends: '@react-native-community',
  plugins: ['eslint-plugin-import'],
  rules: {
    'import/order': [
      'error',
      {
        groups: ['builtin', 'external', 'internal'],
        pathGroups: [
          {
            pattern: 'react',
            group: 'external',
            position: 'before',
          },
          {
            pattern: 'react-native',
            group: 'external',
            position: 'before',
          },
          {
            pattern: 'react*',
            group: 'external',
            position: 'before',
          },
          {
            pattern: '@react*',
            group: 'external',
            position: 'before',
          },
        ],
      },
    ],
  },
  settings: {
    'import/resolver': {
      node: {
        extensions: ['.js', '.jsx', '.tsx'],
      },
    },
  },
};

is it possible to auto format file or it's just notification?

Some rules are autofixable, some are not (this is true for all eslint plugins).
Run eslint with --fix to autofix what can be autofixed; you'll have to manually fix the rest.

@ljharb got it, thanks


Migrate bundling to esbuild

Adds esbuild bundling to bundle.js, server.js now in the repo.
VScode tasks.json includes two watch tasks (tsc and bundle), that can be run in dev mode. Error matcher $esbuild-watch is available through vscode extension.
Main.js was modified to prevent require resolution by esbuild and proper possible exception handling.
Made minified esbuild bundle with external source maps in build mode.

Not sure that it's right that this now bakes in dependencies into the js file published on npm (I believe) but those are still listed as dependencies.
I also suppose they are small and duplication doesn't matter

Didn't think about it. Bundlers are so bundlers. There is an option --external-imports for modules not to include, but an additional build script will be needed probably.
Otherwise, dependencies could be moved to devDependencies.


Add Rares Matei to Contributors list

@rarmatei, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I accepted it?

It felt great! I always wanted to start contributing, and your guide pushed me past that first step.
I in my day to day job I use a Pull Request Workflow, so it wasn't that part specifically that kept from creating my first PR, but it was more to do with the projects available and finding an issue I could start working on without spending days getting familiar with a project.
I tried before to setup this project: https://github.com/CoderDojo/community-platform
But had a lot of difficulties with PostgreSQL, and their guide didn't go into that much detail around this.
After I finished your guide however, I went to FreeCodeCamp and they have an amazing guide for setting up the project. I also found a really easy issue I could solve as my first PR. I got a response almost immediately to my PR, with a thank you, and some explanation around what I could change. All in all, it's been a really great experience, and I'll probably continue looking for ways to contribute to that project.
Really appreciate your work around setting up this repo, and hope it's going to help a lot more people!

That's brilliant @rarmatei. I'm super happy to learn about your experiences.
As I explained earlier, we started off this project to address various problems faced by first time contributors. Setting up dev environment is definitely one of them. We're still not sure how to solve this.
As an experiment, we've started a slack team. You can join the team by going to https://firstcontributions.herokuapp.com
You can probably help someone who is struggling to set up or contribute to FreeCodeCamp.

Cool, thanks! I've joined that!


mwc-node, mwc-node, mwc713 should use Nested Errors

Currently we are loosing errors history, as a result it is hard to understand the root cause if the issue.
Errors need to keep reference to underline error or at least to it's string representation.

This kind of update cannot be tested regularly. It is covered as part of regression testing.


Syllabus: How do we give some sense of "order" learners should follow?

Feature description
Let's take the following example:

How do learners know which order to follow across the 7 chapters?
Additional context
CC: @lerongil, @gracelindsell, @techtolentino

There's a few solutions that we discussed earlier:

Add numbers to the chapters: Yep, this helps order the Textbook chapters...but the external links have no order
Allow external links to inline "chapter" link: This is something we talked about from MVP, but we could add a "create custom chapter" link creator?

We should include this for user testing, along with the following ideas:

Chapter / Reading order
External link issues
Overview description

CC: @JRussellHuffman, @gracelindsell

yep. agreed. This is a good one

This repository is no longer maintained. It has been superseded by IBM Quantum Learning.


⚠ Bofrost.lu has degraded performance

In 8f33201, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1521 ms

Resolved: Bofrost.lu performance has improved in ea2a439 after 46 minutes.


Improve settings management

Currently, the bot token is setup using an environment variable, the log channel and prefix are hard-coded, and everything else is only configurable in-channel. This should be simplified such that it's all handlable through a single configuration file. @RyanEHenderson assuming that I'm not going to get throat-punched for asking this... do you have a preferred config file format? JSON? YAML? ...shudders... INI?

I typically use YAML for configs, probably stemming from how I started programming with Java for Minecraft and they usually use a YAMLConfiguration

I typically use YAML for configs, probably stemming from how I started programming with Java for Minecraft and they usually use a YAMLConfiguration

That doesn't sound terribly convincing... is that "I want to use YAML" or "I usually use YAML but..."

That doesn't sound terribly convincing... is that "I want to use YAML" or "I usually use YAML but..."

I'm most used to YAML but I'm open to other options, just not as familiar with them for configs. Should probably just go YAML

I'm most used to YAML but I'm open to other options, just not as familiar with them for configs. Should probably just go YAML