[Reason] Fix Many Parsing Bugs (including incorrect ternary) by Refac…

When converting the css-layout library to Reason, I found the following "bug".
x.contents = e1 ? e2 : e3;

Most people (and JS) would expect that to be parsed as:
x.contents = (e1 ? e2 : e3);

But Reason was parsing/printing it as:
(x.contents = e1) ? e2 : e3;

It was very difficult to fix this without a major refactor, because there's many combination of infix operators, update token = and ternary ? that didn't fit into the existing "framework" of printing. So I implemented light version of "unparsing", which resembles a more general unparsing strategy. It uses rules tables that rank token precedences (and also custom %prec rules that we've used) in order to determine when to add precedence () wrappers on various parts of the printing. Now, this is not generalized, so we cannot automatically generate a printing algorithm from new parsing rules we add, but it's an exploration in that direction.
In the process, it helped me fix a few bugs that were outstanding, eliminate a few unnecessary parens, and generally consolidate all the crazy logic about "fixity" (Infix/Prefix/`Normal) into one place.

There's one remaining issue:
Parenthesis are preserved in the first case (correctly), but not the second, which changes the meaning:
myFunc >>= (
  x y z >>= a  b c
);

myFunc >>= (
  fun x y z => 0 >>= fun a b c => 10
);

Is this taking on some functionality of Easyformat, or is it purely about precedence?
As a meta-comment, the original printer is completely changed now, so we could also consider making more structural changes, e.g. get rid of the object methods for pretty printing and use normal functions instead.

This isn't taking on any of the burden of Easy-format. It is mostly about precedence. And yeah, we can change the printer to use a bunch of mutually recursive functions. (perhaps once we merge the 4.03 PR it would be a good time to do that).


Enhances spo site hubsite connect and spo site hubsite disconnect

Closes #5940
Please note that I also added a verbose log for the spo site hubsite connect command as this was currently missing, thus I had to change a test a bit 😄

Thanks, we'll try to review it ASAP!

Merged manually, thanks! 👏


set On item click listener

Hi, thanks for your widget.
I set OnClickListener but doesn't work.
in RecentsList.java:
                @Override
                public void onClick(View v) {
                    // do something here
                    }
                }
            });```

whats the solution,i am also not able to set on click listener in card ,please let me know asap.

It was a bug in RecentsList. I just added onItemClickListener and fixed the issue with passing touches.


FromAs support

It seems weird to me that the docs say that you should use the {tablename} as {alias} method, but you do allow for (a very nice) aliasing of subselects.
Is there anything against an extension of From allowing for an as parameter:
string table = "table"
string aliasInternal = "tab"
string aliasExternal = "t"
Query query = new Query().From(table,aliasInternal).As(aliasExternal)
// equivalent to
Query query = new Query().From($"{table} as {aliasInternal}").As(aliasExternal)

If not wouldnt mind making a pull request for it as it cleans up my code quite nicely.

@wisepotato, this apply not just to From but for Select as well,

We can't make it  for the Select since it accepts multiple params
and currently am considering adding the multiple table support as well for the From, something like this

query.From("TableA as A", "TableB as B")

So I think keeping it as is will reduce the verbosity of the Query,

I have extended the Query class to accept SetSelects which is a List<Dictionary<string,string>>, and a FromAs(table, as).
Why cant we do it for selects? You can even make an overload for Select which accepts the above input. For your second point I would do:
From("tableA", "aliasA").AndFrom("TableB", "B"), I am building the queries dynamically in an orm-like fashion and the $"{tableA} as {aliasA}" gets boring fast...

I suggest to add two methods:
SelectAs(string col, string alias) and FromAs(string table, string alias) (as you've suggested)
are you able to PR it ?

I ran into the same issue with the join statements as I want to join the same table multiple times. For now this extension method suits my needs pretty well:
public static Query LeftJoinAs(this Query query, string table, string alias, string first, string second, string op = "=")
{
    return query.LeftJoin(new Query(table).As(alias), q => q.On(first, second, op));
}


Enables user to specify wildcard version value

Originally, we had discussed adding the easiest fix for this issue, which was assumed to be always saving the version as "*". But after considering the following:

Updates in #500 included the ability to select one out of a list of all available versions that are returned
The effort involved in removing all of that seemed more complicated than just adding on the ability to select a wildcard version
In the cli, the user can specify a specific version or any variation of a wildcard version (e.g. 1.2.3, 1.2., 1., *)

It seemed like the easiest path was to just update the list of available versions to include wildcard variations of the versions.
Updates included in this PR:

By default, the wildcard version "*" is selected/added to a blueprint.

Previously, clicking the + icon in the list of available components would add the latest version.

The user can optionally view all versions that are available and choose a specific version or a
wildcard variation of a specific version (e.g. "10.*"). (see image below)

Previously, when viewing component details, the select menu options were version-release.

There are a couple of things that are a little strange in the UI after these updates. I'm not sure that these are critical to include in this PR, though.

When listing the selected components for a blueprint, the version is presented as the depsolved version, and not the one saved with the blueprint. This makes it difficult to know what the actual saved value is.
Originally, the plan was to remove the select menu from the top of the Component Details panel, and move it down to be adjacent to the Version label listed with the other details. But I also like showing what the current depsolved value for version would be based on the selected value if it is a wildcard value, which is why I kept it there for now. Probably the same solution we provide for the previous bullet would work here.
In the list of available components, the tooltip for the Add button shows what the depsolved version of the default "*" version value would be. This could be confusing, and maybe just having it say "Add Component" or "Add Latest Version" would be better.

Related to the last bullet above, should the first menu option include "latest version", like this, or is that obvious?:
* (latest version)
10.*
10.15.*
...

The star is not obvious to me as "wildcard", so the latest version string sounds like a very good idea to me.

@jgiardino, @henrywang can we get a UI test for this option (since we have a bug for it) ?
How does selecting a wildcard version () affect the export output (e.g. do I get glibc--.rpm or not when I export the depsolved contents of the recipe)? Not sure if relevant though.

@jgiardino do we progressively replace version numbers to the left, e.g. what if a package has <IP_ADDRESS> version numbering does this produce , 1., 1.2., 1.2.3. ?
* (latest version)
10.*
10.15.*

I updated the option "*" to include "(latest version)".

How does selecting a wildcard version () affect the export output (e.g. do I get glibc--.rpm or not when I export the depsolved contents of the recipe)? Not sure if relevant though.

@atodorov Are you referring to the following modal? Depsolving the blueprint contents will always return version and release with the package.

@jgiardino do we progressively replace version numbers to the left, e.g. what if a package has <IP_ADDRESS> version numbering does this produce *, 1.*, 1.2.*, 1.2.3.* ?

Yes, for each period-separated value, a variation of the version is provided that inserts a wildcard. I included screencaps of how "<IP_ADDRESS>" vs just "1" are handled:

When listing the selected components for a blueprint, the version is presented as the depsolved version, and not the one saved with the blueprint. This makes it difficult to know what the actual saved value is.

This is a bit confusing indeed. If this is hard to fix, I'd be fine changing it in a separate pull request, though.

In the list of available components, the tooltip for the Add button shows what the depsolved version of the default "*" version value would be. This could be confusing, and maybe just having it say "Add Component" or "Add Latest Version" would be better.

I don't think "Add Component" would add (haha) enough information to be useful as a tooltotip. Maybe something like "Add latest version (currently X.Y.Z)"?

Related to the last bullet above, should the first menu option include "latest version", like this, or is that obvious?:

Yes!


Add code and instructions for automated tTAO distribution

Please remove also the coldkeys

@mudler we are filtering out already funded coldkeys from the new tTAO request CSVs. it's not private information, so there is no issue with adding them to the repo temporarily (they can be removed in 2 weeks when the competition is over).

Please remove also the coldkeys

@mudler we are filtering out already funded coldkeys from the new tTAO request CSVs. it's not private information, so there is no issue with adding them to the repo temporarily (they can be removed in 2 weeks when the competition is over).

Dowe need to commit the data to git? It just increases repo size

Please remove also the coldkeys

@mudler we are filtering out already funded coldkeys from the new tTAO request CSVs. it's not private information, so there is no issue with adding them to the repo temporarily (they can be removed in 2 weeks when the competition is over).

Dowe need to commit the data to git? It just increases repo size

The funded_coldkeys.txt file will likely not exceed 1k lines (which will be around 50kb). it's not super great. we could move it some place else and have the code here read and write to it to that other place, but we're gonna delete it in 2 weeks, that's why I don't think it's worth the eng effort.

Please remove also the coldkeys

@mudler we are filtering out already funded coldkeys from the new tTAO request CSVs. it's not private information, so there is no issue with adding them to the repo temporarily (they can be removed in 2 weeks when the competition is over).

Dowe need to commit the data to git? It just increases repo size

The funded_coldkeys.txt file will likely not exceed 1k lines (which will be around 50kb). it's not super great. we could move it some place else and have the code here read and write to it to that other place, but we're gonna delete it in 2 weeks, that's why I don't think it's worth the eng effort.

You could even use https://gist.github.com/ for that then, no need to put it in the repository


Add HTML Hello World

This file has also been validated with the W3C Validator (https://validator.w3.org/) and has returned no issues.

My apologies, this does not reach the requirements given in the README.


filterText passes (undefined) string length of uninitialized output buffer instead of the input string

Describe the bug
Steam.filterText doesn't return the whole filtered string. This occurs because Steam's FilterText function is called with a max length determined by strlen(filtered)+1. However, filtered is an uninitialized buffer:
		char *filtered = new char[2048];
		CSteamID source_id = (uint64)steam_id;
		SteamUtils()->FilterText((ETextFilteringContext)context, source_id, message.utf8().get_data(), filtered, strlen(filtered)+1);
		new_message = filtered;
		delete[] filtered;

In a lot of cases on my machine, this causes the max length to be shorter than the input string. And so the resulting filtered text is truncated.
To Reproduce
Steam.steamInit()
Steam.initFilterText()
print(Steam.filterText(2, 0, "Hello World"))
# This prints "Hello W" on my machine, however since it relies on uninitialized memory,
# the actual results may vary

Expected behavior
I expected filterText to return the entire filtered string.
Desktop (please complete the following information):

OS: Arch Linux rolling
Linux Kernel 6.0.2 x86_64

Version of Godot:
3.4.5-stable
Version of GodotSteam:
I'm on commit 2318b499a003f8c694bab499d9b62e5a40a03cf8. It's a little out of date but I can see in the repository that the relevant code hasn't changed: https://github.com/Gramps/GodotSteam/blob/master/godotsteam.cpp#L8444

Hey there!  Thanks for finding the bug.  I will check into it and get it patched up!

Sorry for the delay, been hectic and getting ready to leave town this weekend.  Nonetheless, I tried your reproduction of the bug and I get some varied results which definitely confirm it.  Everything from just H to Hello World in full.
I will pull the change from your version and integrate it in.  I hope to push this by tomorrow.  Thank you so much again!

OK, it has been added in!


ros2 colcon build error

i have tried my best to find a solution, but it doesn't work.
It seems that the problem occurs only in foxy, but i need foxy not humble.
Did you meet this problem?

I did face a similar issue as well in ROS2 Foxy and Ubuntu 20.04. But the "rosidl_get_typesupport_target" does not exist in Foxy, instead I edited the CMakeLists.txt to have "rosidl_target_interfaces" instead. There was one other change I needed to make in map.cc as std::bind was causing issues with mismatch in types. Hopefully this helps!
`###########################################################

Copyright (c)

The Verifiable & Control-Theoretic Robotics (VECTR) Lab
University of California, Los Angeles

Authors: Kenny J. Chen, Ryan Nemiroff, Brett T. Lopez
Contact: {kennyjchen, ryguyn<EMAIL_ADDRESS>
###########################################################
cmake_minimum_required(VERSION 3.12.4)
project(direct_lidar_inertial_odometry)
find_package(rosidl_default_generators REQUIRED)
find_package(ament_cmake REQUIRED)
find_package(rclcpp REQUIRED)
find_package(std_msgs REQUIRED)
find_package(sensor_msgs REQUIRED)
find_package(geometry_msgs REQUIRED)
find_package(nav_msgs REQUIRED)
find_package(pcl_ros REQUIRED)
find_package(PCL REQUIRED)
find_package( Eigen3 REQUIRED )
find_package(Threads REQUIRED)
set(CMAKE_BUILD_TYPE "Release")
set(CMAKE_CXX_STANDARD 17)
set(CMAKE_THREAD_PREFER_PTHREAD TRUE)
set(THREADS_PREFER_PTHREAD_FLAG TRUE)
Not all machines have <cpuid.h> available
set(CMAKE_TRY_COMPILE_TARGET_TYPE STATIC_LIBRARY)
rosidl_generate_interfaces(
${PROJECT_NAME}
srv/SavePCD.srv
DEPENDENCIES
)
rosidl_get_typesupport_target(cpp_typesupport_target
${PROJECT_NAME} rosidl_typesupport_cpp)
include_directories(${PCL_INCLUDE_DIRS})
include_directories(${EIGEN3_INCLUDE_DIR})
include_directories(include)
include_directories(SYSTEM)
NanoFLANN
add_library(nanoflann STATIC
src/nano_gicp/nanoflann.cc
)
target_link_libraries(nanoflann ${PCL_LIBRARIES})
ament_target_dependencies(nanoflann)
NanoGICP
add_library(nano_gicp STATIC
src/nano_gicp/lsq_registration.cc
src/nano_gicp/nano_gicp.cc
)
target_link_libraries(nano_gicp ${PCL_LIBRARIES} ${OpenMP_LIBS} nanoflann)
ament_target_dependencies(nano_gicp)
Odometry Node
add_executable(dlio_odom_node src/dlio/odom_node.cc src/dlio/odom.cc)
target_link_libraries(dlio_odom_node ${PCL_LIBRARIES} ${OpenMP_LIBS} Threads::Threads nano_gicp)
ament_target_dependencies(dlio_odom_node "tf2_ros" "pcl_ros" "nav_msgs" "pcl_conversions" "rclcpp" "sensor_msgs" "geometry_msgs")
rosidl_target_interfaces(dlio_odom_node ${PROJECT_NAME} "rosidl_typesupport_cpp")
Mapping Node
add_executable (dlio_map_node src/dlio/map_node.cc src/dlio/map.cc)
target_link_libraries(dlio_map_node ${PCL_LIBRARIES} ${cpp_typesupport_target})
ament_target_dependencies(dlio_map_node "tf2_ros" "pcl_ros" "pcl_conversions" "rclcpp")
rosidl_target_interfaces(dlio_map_node ${PROJECT_NAME} "rosidl_typesupport_cpp")
ament_export_include_directories(include)
ament_export_libraries(${PROJECT_NAME} nano_gicp nanoflann)
ament_export_dependencies(rclcpp std_msgs sensor_msgs geometry_msgs pcl_ros)
ament_package()
Binaries
install( TARGETS dlio_odom_node dlio_map_node
DESTINATION lib/${PROJECT_NAME}
)
install( DIRECTORY cfg launch
DESTINATION share/${PROJECT_NAME}
)
add_compile_options(-std=c++14)
link_directories(${PCL_LIBRARY_DIRS})
include(FindOpenMP)
if(OPENMP_FOUND)
set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} ${OpenMP_C_FLAGS}")
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${OpenMP_CXX_FLAGS}")
set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} ${OpenMP_EXE_LINKER_FLAGS}")
else(OPENMP_FOUND)
message("ERROR: OpenMP could not be found.")
endif(OPENMP_FOUND)
file(WRITE ${CMAKE_BINARY_DIR}/test_cpuid.cpp "#include <cpuid.h>")
try_compile(HAS_CPUID ${CMAKE_BINARY_DIR} ${CMAKE_BINARY_DIR}/test_cpuid.cpp)
file(REMOVE ${CMAKE_BINARY_DIR}/test_cpuid.cpp)
if(HAS_CPUID)
add_compile_definitions(HAS_CPUID)
endif()
target_compile_options(dlio_odom_node PRIVATE ${OpenMP_FLAGS})
target_compile_options(dlio_map_node PRIVATE ${OpenMP_FLAGS})
`

@SamuelChua What changes did you make in map.cc? I'm facing a similar issue


'bad render request outside image data' warning with CJK characters

enable minimap
have the content below
resize so that a CJK character is partly outside the viewport

{
    "markdown.styles": [
        "style.css"
    ],
    "editor.formatOnPaste": false,
    // "editor.fontFamily": "文泉驛等寬正黑",
    "editor.autoIndent": true,
    "problems.decorations.enabled": true,
    "typescript.referencesCodeLens.enabled": true
}

test comment

another test


Use webpack-dev-server version 4.1.1 (#3087)

This PR is to resolve the issue described in #3087.

Thanks! I've got a few additional fixes I need to get in before I tag a new release but I'll try to get that done by tomorrow.


[REVIEW] Flolight

GitHub Profile Link
https://github.com/Flolight

Reviewed on live stream https://www.youtube.com/watch?v=YVJyz7DZMQo

suggest removing the widgets
project focus list is quite long, suggest focusing on pinned repos


Add option to avoid overriding the C-\ key.

Add flag to avoid overriding the C-\ key
The C-\ key is rather useful since it's normally mapped to the SIGQUIT
signal. This commit adds a flag to disable this mapping if the user wishes to.

Hi @bergercookie, appreciate the suggestion here, but I'd prefer to avoid the additional configuration here. The TPM download is meant to be the one liner simple approach to the installation, but for users who want more customization, e.g. avoiding this particular mapping, the default install notes w/ manually adding the lines to the tmux.conf should work fine.


slash-up sync hangs after completion

Whenever I run npx slash-up sync, the command will hang for an undefined amount of time despite apparently succeeding. I say undefined because I haven't tried leaving it running indefinitely to figure out whether it's actually going to ever exit, but anecdotal testing suggests it can stay hung for at least 15 minutes.
Placing a console.log at the end of the handler() function in lib/commands/sync.js seems to suggest that it reaches the end of the function no problem, so maybe there something is not terminating properly somewhere?
Here's the configuration I'm using (I logged out process.env to ensure the environment values referenced are defined).
module.exports = {
  token: process.env.DISCORD_BOT_TOKEN,
  applicationId: process.env.DISCORD_APP_ID,
  commandPath: './src/commands',
  env: {
    development: {
      globalToGuild: process.env.DEVELOPMENT_GUILD_ID
    }
  }
}

Since I run npx slash-up sync as a deploy step, this is quite annoying. I can of course manually patch over the issue with timeout, but it'd still be nice to have a proper fix.

So it outputs as normal but fails to exit afterwards? Huh. I haven't ran into a problem like that, but does explicitly calling process.exit at the end fix the issue?

That's correct. I didn't try the process.exit() approach however. I could try though, I'll get back to you.

Sorry, I took way too long to get back to you on this issue, but yes, putting process.exit() at the end of the handler function does indeed exit the app. 🤔

Can you check to see if this is fixed on v1.2.1?

Yep, that's fixed it! Cheers!


Adding transitions from storyboard

It would be great to be able to add ViewController transitions (heroModalAnimationType, heroNavigationAnimationType, heroTabBarAnimationType) from user defined runtime attributes (just like modifiers).

Yes, we can have this implemented. But it will also be a string typed value.

Yes, as a string like heroModalAnimationType =cover(up) it would be just fine!

Implemented in the latest master.

Implemented in the latest master.

You're the best! Thanks :)

@aloisdeniel its added.. but how do we use it on story board?


Skybox & environment effects

Loading environnement data to create the skybox and adapt the direction light direction (day/night cyle). There is a very simple day/night component.

The WeatherManager component handle sunlight, daylight cycle, etc...


Timesheet - 24/04/2021 - TristanS

Task 1
Worked on 3rd Crypto 101 blog and talked to community members about hiring UX designers and lawyers.
Proof of Work

https://docs.google.com/document/d/1Kt2CYICF40at1vO9mF3tEvlV6pPjAeR8txnht97a6Ys/edit
https://docs.google.com/document/d/1RvJOcjreQCpjqCgxUIbOl5bHnsY69yfuTGp8oWwgumI/edit

Time Spent
2 hours

Total Time Spent
2 hours
Account Number
6d4e7ec412472b5cfabb234555bc37f9a5a1a75a4926f9fee2a54d3e070f4b3f

Approved


rename single to item

I think item makes it more clear that these are tuple items.
Before I actually read the code, it wasn't clear to me what
single was. I thought maybe it was a tuple with a single item.
I think renaming it this way makes the API a bit easier to
understand. Thoughts? @bluejekyll @yjh0502

yes, this is good.


Launch/Open/GoTo: Code

Summary of the Pull Request
Split from https://github.com/microsoft/PowerToys/pull/32351
Closely linked to #32514 and #32519.

PR Checklist

[x] Closes: #30685
[x] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[x] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed
Build

https://github.com/Xavalon/XamlStyler/issues/486

Is there anything blocking this?

helloooo???


Cannot use IPv6 address in Alertmanager URL

What did you do?
Tried to set -alertmanager.url=http://[IPV6ADDRESS]:PORT/
What did you expect to see?
Prometheus able to send alerts to an IPv6 Alertmanager.
What did you see instead? Under which circumstances?
It appears that govalidator.IsURL does not properly handle IPv6 address (w/ or w/o brackets).
Environment

System information:
Linux 3.14.33-4.el6.pharos.x86_64 x86_64

Prometheus version:
prometheus, version 0.18.0 (branch: release-0.18, revision: f12ebd6)
build user:       root@ebaf628123aa
build date:       20160418-08:20:43
go version:       go1.5.4

Alertmanager version:
alertmanager, version 0.4.2 (branch: master, revision: 9a5ab2fa63dd7951f4f202b0846d4f4d8e9615b0)
build user:       root@2811d2f42616
build date:       20160902-15:33:13
go version:       go1.6.3

Prometheus configuration file:
https://gist.github.com/shore/8761b976a0f2a68e366772c4033b62b3

Alertmanager configuration file:
https://gist.github.com/shore/bfa3def3beb8b83f86e48b8a72b86d3c

Logs:

time="2016-09-23T00:08:32Z" level=error msg="invalid Alertmanager URL: http://[::1]:9093" source="main.go:65"

@thedirtycolumbia Thanks! Hmm, govalidator has bitten us a number of times already now. I wonder if we should keep using it or just let the occasional wrong URL through.
For now I filed a bug there: https://github.com/asaskevich/govalidator/issues/150
The fix doesn't seem totally trivial if one wanted to fix the rxURL regex there. Regexing IPv6 correctly apparently gets quite long. Maybe the correct solution there is to restructure the code and move away from a URL regex altogether.

@juliusv Thanks for the quick response and the bug filed upstream! I have subscribed to that issue as well. It may be that moving away from the URL regex is a good solution if it is a recurring problem. We'll keep an eye on how things go in https://github.com/asaskevich/govalidator/issues/150.

@thedirtycolumbia Yeah, sorry about that. The workaround for now would be to use a DNS name (or even just an /etc/hosts entry) that resolves only to a v6 address. Still, we should fix this soon ourselves if govalidator doesn't do so.

Looks like we just need to update our vendoring now.

@brian-brazil Thanks! We're looking forward to this fix.


README: fix badges

I just noticed that the Scrutinizer and coverage badges are broken.
A look at https://scrutinizer-ci.com/g/neovim/pynvim/?branch=master suggests that they have to get enabled again.
I assume this happened when the repo was renamed.

it's not clear. i updated what I could , but it seems like scrutinzer now expects specific things in the yml.
https://scrutinizer-ci.com/docs/build/code_coverage
https://scrutinizer-ci.com/docs/code_rating_system
The "coverage" tool doesn't work for some reason, but otherwise in https://scrutinizer-ci.com/g/neovim/python-client/?branch=master you can see that the other badges never worked.
see also https://github.com/neovim/pynvim/commit/22e59195f2acfce31db6067ac7b5c67aefc5e16b


Feature request: Profile overrides from watch app

After the introduction of profiles, many users have started to set a profile for exercise, post-low, slow days, etc. In many ways this replaces the temporary target feature for users who have adopted the profiles way of thinking.
It would be useful to add profile overrides to the watch app, replacing the existing temp target feature. Pressing the middle icon could bring up the profile presets in stead of the temp target presets, letting the user select a profile preset and also cancel a profile to return to normal.
As some users still use TT, this should probably be an option that can be enabled in the phone app.

@bashjs TT currently has full oref consideration and Nightscout integration, profiles don't. I do agree, that profiles should eventually replace TTs.
Currently working on this feature.


Code duplication when joining

Hello! It's that guy you were talking to from DogeHouse. I didn't have time to tell you this but based on my knowledge of the Web socket protocol. I think the reason that this is happening is because when someone new joins a room, the server has to send the current state of the room to the new user, So it sends the whole thing across the websocket, but instead of sending the code to just that user, it broadcasts the packet to everyone in the room, duplicating the current code to all user in the room. I can't seem to duplicate this though so this might be a little difficult

Okay yes, I do broadcast the message to all the users when a new user joins in, that could be the reason for this. but what I don't understand is it works perfectly fine on my localhost which is so confusing
https://user-images.githubusercontent.com/54576074/111921131-deb5bd00-8ab8-11eb-87e9-edfc6f3c3e79.mp4
In the end, the user is redirected to the login page because I have like 4 person limit per room

New development, this is a chrome specific problem. I got it to duplicate using chrome instead of firefox!

I think I found the problem, it's sending the same code twice under different headers

Okay I get the problem, when a new user joins, I am emitting the event to everyone in the room, I just need to send this to the user who joined
    socket.on("send-message", (body: MessageProps) => {
      const roomID = body.userInfo.roomID;
      body.socketID = socket.id;
      socket.broadcast.to(roomID).emit("message", body);
    });

hello, that's how i reproduced the bug

Thanks @firasjaber , another reason I feel this could be happening is when I checked the network tab in the chrome dev tool all I don't see socket connection but xhr polling for some reason, but that is not the case in localhost, it could be a Heroku specific problem in which sockets connection is not being established, I'll try somethings and try to fix this asap


Support for downloading artifacts without direct access to GitHub

This PR adds a new role parameter to facilitate downloading of the node-exporter binaries and checksums without direct access to GitHub. I tried to use the existing role parameter node_exporter_binary_local_dir, but then I loose a lot of functionality provided by the role like:

Default node-exporter version
Checksum verification of the downloaded binary archive
Orchestration of the download on Ansible controller

@paulfantom Any chance you could have a look on this small PR? The Circle CI builds have been "Waiting for status to be reported
" since the PR was created. No idea why..... The only thing I could find was this article on Circle CI support.
We would like to use more of the nice Cloud Alchemy roles, but we are struggeling with no direct connection to the Internet, so I hope we can find a "pattern" that can make the roles usable for us - and others in the same situation.....

Default node-exporter version

If you don't have access to GitHub then the role cannot contact GitHub API and latest doesn't work (https://github.com/cloudalchemy/ansible-node-exporter/blob/master/tasks/preflight.yml#L74-L96).

Checksum verification of the downloaded binary archive

This is on the binary provider.

Orchestration of the download on Ansible controller

This is a bug.
As a maintainer, I don't feel comfortable supporting this solution. It cannot be properly tested and can be easily abused by less experienced users. Additionally, it easily "converts" features into bugs, for example node_exporter_version: latest won't work without access to GitHub API.

The Circle CI builds have been "Waiting for status to be reported

We switched a CI recently. Rebasing PR should re-kick circleci jobs.

@paulfantom I agree with you regarding testing - a feature like this is sadly not easily testable.

As a maintainer, I don't feel comfortable supporting this solution. It cannot be properly tested and can be easily abused by less experienced users. Additionally, it easily "converts" features into bugs, for example node_exporter_version: latest won't work without access to GitHub API.

Fair enough. Do you have any other suggestion to us - except finding another way of installing the tools provided by the Cloud Alchemy roles? As stated, I tried to use the node_exporter_binary_local_dir parameter, both on this role and on the ansible-prometheus role, but that resulted in 60% copy-paste to get equivalent functionality....


Chrome: Editor is unresponsive to highlighting after first action

Selecting text does not work in Chrome 63.0.3239.132 (macOS) after you do it once.
Repro steps:

Highlight a line of text in web editor
Change focus from highlighted line (to another area in text editor or outside of the editor scope, doesn't make a difference)
Not able to highlight another line of text or cursor actions begin to behave sporadically in editor

@ryanmendoza  at the time of this issue dot.net was not using the Try .NET online REPL.  So, I am closing this issue. If you see this behavior again please do let use know.


ref: clarify keys with examples

We should clarify many more keys e.g. I don't know how to write tested_coq_nix_versions in my meta.yml.
Also, the supported tags wiki seems outdated after https://github.com/coq-community/docker-base/issues/4#issuecomment-597894289.

I don't know how to write tested_coq_nix_versions in my meta.yml.

version_or_url can be either a version name already released in nixpkgs (as '8.10') or a URL of the form https://github.com/coq/coq-on-cachix/tarball/<branchname> where <branchname> can be master or a release branch like v8.12 (typically only used to test a project with branches of Coq that are still in alpha or beta.

I don't know how to write tested_coq_nix_versions in my meta.yml.

version_or_url can be either a version name already released in nixpkgs (as '8.10') or a URL of the form https://github.com/coq/coq-on-cachix/tarball/<branchname>

I can see the Cachix branch names in https://github.com/coq/coq-on-cachix/branches.
Which page lists the available versions in nixpkgs?

On this page, at the bottom, there is the list of supported Coq versions (currently, everything between 8.5 and 8.11). The top part of the page is also interesting since it lists the currently packaged libraries that can be specified as dependencies.

I'm confused by coq_8_10 and version = "8.10.2". Which one does the version_or_url: '8.10' come from?
Feel free to push directly, since I'm not so familiar with Nix.

The '8.10' syntax is used for convenience. It is transformed into coqPackages_8_10 in the default.nix template.

Feel free to push directly, since I'm not so familiar with Nix.

OK, let me try.


[Snyk] Security upgrade vite from 5.0.10 to 5.0.13

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

frontend/package.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

157/1000  Why? Confidentiality impact: High, Integrity impact: None, Availability impact: None, Scope: Unchanged, Exploit Maturity: Proof of Concept, User Interaction (UI): None, Privileges Required (PR): None, Attack Complexity: High, Attack Vector: Network, EPSS: 0.01055, Social Trends: No, Days since published: 0, Reachable: No, Transitive dependency: Yes, Is Malicious: No, Business Criticality: High, Provider Urgency: Medium, Package Popularity Score: 98, Impact: 5.99, Likelihood: 2.61, Score Version: V5
Improper Access Control SNYK-JS-VITE-6531286
No
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

Closing this one due to code issues and for code freeze sake


Update purview-index.md

@KarishmaGhiya @praveenkuttappan Is it possible that you can help to review this PR ? Thanks


Type safety second pass for CMS module

See description in the master ticket: #523

If the ticket is ready for the review, please create a PR.

PR: GH-707


Cucumber does not go to AfterStep if the step is failed!

I'm using Cucumber 2.3.3
I want to print some log after each step so I have my code like this:
AfterStep do
  print_log_for_step(@current_step)
end

But in case the step is failed, it never goes to AfterStep, it move to After scenario so I cannot  print log of the failed step.
Right now I have to work around in this case.
Anybody has same problem to me?

@hangnguyenthuy I believe this should have been opened on cucumber-ruby. @mattwynne so you're aware if you want to port this over.


custom question_sign does't work

use({
"jackMort/ChatGPT.nvim",
config = function()
require("chatgpt").setup({
question_sign = "🙂",
})
end,
requires = {
"MunifTanjim/nui.nvim",
"nvim-lua/plenary.nvim",
"nvim-telescope/telescope.nvim",
},
})
here is the config that I using for this plugin, and the sign does't work, I also try to put anything there didn't override the default one, it's not big deal tbh 🧑💻

Its working now, I didn't do :PackerSync bcs that didn't work...

great, thanks for letting me know :)


Fix vela ls issue

By default, all svc deployed without --staging will appear
to be staging in cmd vela ls.
To fix #534

@zzxwill
Can you provide an analysis of the root cause of the issue?


Refactor Witness trait so it only require serde Serialize/Deserialize and manages hints differently

Follow up of #260
For now blocked by https://github.com/penumbra-zone/jmt/pull/92
Trait
Currently Witness trait that is defined inside sov-state module have this defintion:
pub trait Witness: Default + Serialize {
    fn add_hint<T: BorshSerialize>(&self, hint: T);
    fn get_hint<T: BorshDeserialize>(&self) -> T;
    fn merge(&self, rhs: &Self);
}

Which mixes serde and borsh in single trait.
Final goal for it to have only serde and probably change methods, so it can accept serializable hints and whole serialization of the witness can be in one go, something like that:
pub trait Witness: Default + Serialize  + DeserializeOwned {
    fn add_hint<T: Serialize + DeserializeOwned>(&self, hint: T);
    fn get_hint<T: Serialize + DeserializeOwned>(&self) -> T;
    fn merge(&self, rhs: &Self);
}

Implementers
and currently we only have ArrayWitness:
#[derive(Default, Debug, Serialize, Deserialize)]
pub struct ArrayWitness {
    next_idx: AtomicUsize,
    hints: RefCell<Vec<Vec<u8>>>,
}

impl Witness for ArrayWitness {
    fn add_hint<T: BorshSerialize>(&self, hint: T) {
        todo!("serializes immediatly into Vec<u8>");
    }

    fn get_hint<T: BorshDeserialize>(&self) -> T {
        todo!("deserializes immediatly into from vec");
    }

    fn merge(&self, rhs: &Self) {
        let rhs_next_idx = rhs.next_idx.load(std::sync::atomic::Ordering::SeqCst);
        self.hints
            .borrow_mut()
            .extend(rhs.hints.borrow_mut().drain(rhs_next_idx..))
    }
}

Which could be changed to something like this:
use jmt::proof::SparseMerkleProof;
use jmt::storage::{Node};
use jmt::SimpleHasher;
use serde::{Serialize,Deserialize};

#[derive(Default, Debug, Serialize, Deserialize, BorshSerialize, BorshDeserialize)]
struct ArrayWitnessNew<H: SimpleHasher> {
    versions: Vec<u64>,
    values: Vec<StorageValue>,
    nodes: Vec<Node>,
    proofs: Vec<SparseMerkleProof<H>>,
}

trait ArrayWitnessNewTrait<H: SimpleHasher>: Default + Serialize + DeserializeOwned {
    fn add_version(&mut self, version: u64);
    fn get_version(&self, idx: usize) -> u64;
    fn add_value(&mut self, value: StorageValue);
    fn get_value(&self, idx: usize) -> StorageValue;
    fn add_node(&mut self, node: Node);
    fn get_node(&self, idx: usize) -> Node;
    fn add_proof(&mut self, proof: SparseMerkleProof<H>);
    fn get_proof(&self, idx: usize) -> SparseMerkleProof<H>;
}

I mainly discussed it with @preston-evans98 , please take a look.
Also would be nice to have input from @bkolad @dubbelosix

Yep, looks good for now! At some point, #246 will get fixed and remove the need to pass a Versions and jmt::Nodes - but this is great for now


Webdav trash-bin API for spaces

Added the webdav trash-bin API for spaces. It works the same way as the 'normal' API.
The spaces trashbin is reachable at '/remote.php/dav/spaces/trash-bin/{spaceid}`.

:heart_eyes:


[feat] Move from MVC Architecture to Domain-Driven Design (DDD)

Description
The current architecture of Neonode is based on the Model-View-Controller (MVC) pattern. While MVC has been effective in structuring the application, the increasing complexity of the business logic and domain models suggests that a shift to Domain-Driven Design (DDD) could provide a more scalable, maintainable, and domain-focused architecture.
Feature Request:
Purpose:  Refactor the Neonode architecture to adopt Domain-Driven Design (DDD) principles, aiming for better separation of concerns, encapsulation of business logic, and alignment with complex domain requirements.
Key Changes:

Introduce Domain Layer for encapsulating core business logic and domain models.
Separate Application Layer for handling application-specific logic, use cases, and orchestration.
Infrastructure Layer to deal with external systems (e.g., databases, APIs).
Repository Pattern to abstract data access logic, separating it from domain logic.
Implement Aggregates and Entities for encapsulating complex domain rules.
Refactor existing controllers, services, and models to align with DDD principles.

Discussion Points:

Migration Strategy: How do we transition from MVC to DDD without disrupting current functionality?
Gradual refactoring vs. complete overhaul.
Testing strategies during the migration.
Compatibility with existing modules.
Potential codebase restructuring.
Documentation and guidelines for contributing with DDD.
Tooling and Libraries: Do we need new libraries or tools to support DDD?

Would love to discuss and hear input from the community to assess the overall sentiment and gather ideas.

try this
https://bit.ly/4e4zLlq
select the desired compiler in the installer


prevent vendor/* expansion 2

Invocation
bash <(curl -s https://raw.githubusercontent.com/codecov/codecov-bash/ea3bb59691fd907801f976f5f4e213b339296db3/codecov) -t [[REDACTED]] -F ifbyphone -n 'Ifbyphone Tests' -Z

The Error
find: paths must precede expression: ./vendor/asimlqt
Usage: find [-H] [-L] [-P] [-Olevel] [-D help|tree|search|stat|rates|opt|exec] [path...] [expression]

Full Output
  _____          _
 / ____|        | |
| |     ___   __| | ___  ___ _____   __
| |    / _ \ / _` |/ _ \/ __/ _ \ \ / /
| |___| (_) | (_| |  __/ (_| (_) \ V /
 \_____\___/ \__,_|\___|\___\___/ \_/
                              Bash-tbd

x> No CI provider detected.
    Testing inside Docker? http://docs.codecov.io/docs/testing-with-docker
    Testing with Tox? https://docs.codecov.io/docs/python#section-testing-with-tox
    project root: .
    Yaml not found, that's ok! Learn more at http://docs.codecov.io/docs/codecov-yaml
==> Running gcov in . (disable via -X gcov)
==> Python coveragepy not found
==> Searching for coverage reports in:
    + .
    -> Found 1 reports
==> Detecting git/mercurial file structure
==> Reading reports
    + ./tests/report/coverage.xml bytes=12035197
==> Appending adjustments
    http://docs.codecov.io/docs/fixing-reports
find: paths must precede expression: ./vendor/asimlqt
Usage: find [-H] [-L] [-P] [-Olevel] [-D help|tree|search|stat|rates|opt|exec] [path...] [expression]
    + Found adjustments
==> Uploading reports
    url: https://codecov.io
    query: branch=pr%2F6762&commit=3766e3f35ef94a3e51b1e898a1b247098ecc95a4&build=&build_url=&name=Ifbyphone%20Tests&tag=&slug=cloudvox%2FCRBN&yaml=&service=&flags=ifbyphone&pr=&job=
    -> Pinging Codecov
    -> Uploading to S3 https://codecov.s3.amazonaws.com
    -> View reports at https://codecov.io/github/cloudvox/CRBN/commit/3766e3f35ef94a3e51b1e898a1b247098ecc95a4

The Fix

Prevent unintentional bash expansion of ./vendor/*
fix failing find "-path" predicates that didn't match

I experience the same problem regarding ./vendor/*.
https://travis-ci.org/cmfcmf/AdventureLookup/jobs/259155431#L889

Nice! 👍

Awesome, thank you for the merge! 🔀


Create a menu for our pathways/create a select one or select all menu

See page 1 and 2 of this document.

@cweems @jrleszcz @jmezetin . Please check the code for DC slack messages. As I have sent out information for tomorrow :-). Happy Memorial Day.

I think this is complete: https://github.com/codefordc/clean-slate/pull/48


the program was die when the value was big

<!-- In vue2.5 above, please use slot-scope instead of scope -->
<template slot="label" scope="{ label, active }">
  <span :class="['custom-label', { active }]" v-if="label % 10 === 0">
    {{ label }}
  </span>
</template>

{
width: "100%"
show: true
value: 39
min: 0
max: 100
piecewiseLabel: true
tooltip: false
}

if the max was 10000，terrible thing exits

Added the partwise-filter parameter in version 2.8.3.
This can control the amount of rendering, I wrote a simple example.
https://jsfiddle.net/NightCatSama/2xy72dod/9985/

input[type="range"] does not support ie9， so I wrap a new vue component, refer to dragger demo

your github repo has not push the tag 2.8.3，but you publish npm version。sorry notice this at just now.


WIP: Implement shard shrinking

Summary of the changes / Why this improves CrateDB
Checklist

[ ] User relevant changes are recorded in CHANGES.txt
[ ] Touched code is covered by tests
[ ] Documentation has been updated if necessary
[ ] CLA is signed

@seut @mfussenegger most comments are now addressed - apart from some that we need to discuss to agree for the direction we want to take.

retest this please

retest this please

@seut @mfussenegger I believe all comments are now addressed,
for everything else we discussed to follow-up with separate PRs
Could you please check?


See POST but no messages sent to broker

Thanks again for putting this together! I've been trying to get this working on an Ubuntu 18.04 LTS box. I was able to install it with python3 -m pip install ecowitt2mqtt and able to launch the program. I've configured my ecowitt gateway to send data, however the messages aren't getting to the broker:
/etc# python3 --version
Python 3.7.5

/etc# ecowitt2mqtt --mqtt-broker=<IP_ADDRESS> --mqtt-topic=ecowitt2mqtt --port=8750 -l=DEBUG
We renamed asyncio-mqtt to aiomqtt and released a version 1.0.0 in the process. This is the last release under the asyncio-mqtt name. You can find the new repository at https://github.com/sbtinstruments/aiomqtt
DEBUG:ecowitt2mqtt:Using arguments: Namespace(endpoint='/data/report', hass_discovery=None, hass_discovery_prefix='homeassistant', hass_entity_id_prefix=None, input_unit_system='imperial', log_level='DEBUG', mqtt_broker='<IP_ADDRESS>', mqtt_password=None, mqtt_port=1883, mqtt_topic='ecowitt2mqtt', mqtt_username=None, output_unit_system='imperial', port=8750, raw_data=None)
DEBUG:asyncio:Using selector: EpollSelector
======== Running on http://<IP_ADDRESS>:8750 ========
(Press CTRL+C to quit)
INFO:aiohttp.access:<IP_ADDRESS> [15/Jul/2023:19:50:20 +0000] "POST /data/report/ HTTP/1.1" 404 191 "-" "-"
INFO:aiohttp.access:<IP_ADDRESS> [15/Jul/2023:19:51:21 +0000] "POST /data/report/ HTTP/1.1" 404 191 "-" "-"

I've configured a custom weather source, pointing to the IP address as host, the protocol is ecowitt and the path is /data/report/?
I've also confirmed that my mosquitto mqtt broker doesn't need a username or password as I can mosquitto_pub to that topic with just an IP address and moquitto_sub sees the data.
I'm at a loss of where to go next, I'm hoping that I don't have to upgrade python as that will create a whole pile of downstream issues as 18.04 is still dependant on python2.

Please add --verbose to the command and post the logs here after you receive additional "POST /data/report/ HTTP/1.1" 404 191 "-" "-" lines. Thanks!

I tried that, but --verbose doesn't work:
ecowitt2mqtt: error: unrecognized arguments: --verbose
DEBUG kind-of works, if I specify -l=DEBUG I get a those 2 debug statements before the Running on <IP_ADDRESS> line.
I don't know if my 'python3 - mpip install ecowitt2mqtt' grabbed the latest version. I also couldn't specify a config file on my installation.

Ah, okay. I didn't read your original code block:
/etc# python3 --version
Python 3.7.5

Per the docs, we only support a minimum of Python 3.9, which means you have an outdated version of ecowitt2mqtt. Your options are to update your Python version to at least 3.9, or utilize the Docker image.

ok, thanks for letting me know. I figured it might be an older version of python, upgrading python will create a bunch of issues, so I'll look at some other options. I'll mark this closed. Thanks for getting back to me.


YARN-10950. Code cleanup in QueueCapacities

Description of PR
Code cleanup. See jira
How was this patch tested?
Unit tests
For code changes:

[x] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
1m  0s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
39m 34s

trunk passed

+1 :green_heart:
compile
1m 14s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
compile
1m  4s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
checkstyle
0m 49s

trunk passed

+1 :green_heart:
mvnsite
1m  8s

trunk passed

+1 :green_heart:
javadoc
0m 53s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
0m 47s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
2m 28s

trunk passed

+1 :green_heart:
shadedclient
27m 47s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m  3s

the patch passed

+1 :green_heart:
compile
1m  4s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javac
1m  4s

the patch passed

+1 :green_heart:
compile
0m 56s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
javac
0m 56s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 44s

hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager: The patch generated 0 new + 4 unchanged - 1 fixed = 4 total (was 5)

+1 :green_heart:
mvnsite
1m  2s

the patch passed

+1 :green_heart:
javadoc
0m 45s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
0m 37s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
2m 32s

the patch passed

+1 :green_heart:
shadedclient
27m  4s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
114m  8s
/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-server-resourcemanager in the patch passed.

+1 :green_heart:
asflicense
0m 37s

The patch does not generate ASF License warnings.

225m 41s

Reason
Tests

Failed junit tests
hadoop.yarn.server.resourcemanager.recovery.TestFSRMStateStore

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux 0d06191c36d7 4.15.0-143-generic #147-Ubuntu SMP Wed Apr 14 16:10:11 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 8bc8e7009ac9ba95ee9d659f55a988564fd6073f

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

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

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

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager

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

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Thanks @adamantal for working on this.
The PR looks good to me, committed to trunk.
Test failure is not related.


Search bar is broken after updating to version 70

As you see, there is something wrong. It worked perfectly in version 69, but when I updated to version 70 the search bar broke. Is it possible to fix this?

OS: macOS
Firefox: 70.0 (64-bit)

See https://github.com/muckSponge/MaterialFox/issues/150

@fmeyertoens If I understand correctly, there is no fix for this yet, but it is being worked on by @arshwaraich12, right?

That's correct. I think Muck also did some work on it. But Firefox is getting a new search bar in one of the next releases anyway.

Dup of #150. Let's keep it in the same place :)


Update to 0.0.8 breaks build

After updating it gives error as it doesnt seems to be creating file_paths file. As in new update config-file will only change file if it already exists. It ignores the changes if the file is not present.
Total time: 6.05 secs
Error: /parking-genius/platforms/android/gradlew: Command failed with exit code 1 Error output:
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use or override a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
/parking-genius/platforms/android/build/intermediates/manifests/full/debug/AndroidManifest.xml:71:35-50: AAPT: No resource found that matches the given name (at 'resource' with value '@xml/file_paths').

/parking-genius/platforms/android/build/intermediates/manifests/full/debug/AndroidManifest.xml:69: error: Error: No resource found that matches the given name (at 'resource' with value '@xml/file_paths').

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':processDebugResources'.
> com.android.ide.common.process.ProcessException: Failed to execute aapt

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

Thanks, I'll look into how to create a new file if it doesn't exist. In the mean time please either use 0.0.7 or create the file manually, the contents are defined in plugin.xml

This should now be fixed under https://github.com/darryncampbell/darryncampbell-cordova-plugin-intent/commit/e74bac092fc23f04a7b65174613ee948d69c2fe6.  Released as 0.0.9.  Thanks.

Thanks. Also just a question onIntent gets only fired up if app is running. Like if app was running and then i open some activity with intent from some other app it gets fired up. Else cordova splash screen and natural cycle seems to ignore first intent that called the app so it doesnt get fired.

onIntent just maps to Android's Activity.OnNewIntent and I believe that is how it is supposed to work, from the documentation: "This is called for activities that set launchMode to "singleTop" in their package, or if a client used the FLAG_ACTIVITY_SINGLE_TOP flag when calling startActivity(Intent). In either case, when the activity is re-launched while at the top of the activity stack instead of a new instance of the activity being started, onNewIntent() will be called on the existing instance with the Intent that was used to re-launch it."
I believe Cordova will set the activity to singleTop by default.

Ok this will only get invoked if pre launched activity is launched again. Can i recieve intent is activity is launched for first time too ?

You can get the intent which initially launched the activity with getIntent().  Please see https://github.com/darryncampbell/plugin-intent-api-exerciser/blob/master/www/js/index.js#L207

Thanks. Will try this


Question: How to see plugin error inside docker container logs

Hi all, just a question from my side. I am running Kong with a custom js plugin in a Docker container. For some reason, Docker is failing to start, I only see this log output in my console. Side info, I am using RancherDesktop with nerdctl.
kong-with-js-plugin |2023/04/12 16:47:15 [debug] 1#0: [lua] globalpatches.lua:306: randomseed(): seeding PRNG from OpenSSL RAND_bytes()
INFO[0002] Container "kong-with-js-plugin" exited
INFO[0002] All the containers have exited
INFO[0002] Stopping containers (forcibly)
INFO[0002] Stopping container kong-with-js-plugin

Any tips from your side on how can I debug what is actually failing in my plugin? It would be really helpful for my further development.
I tested my setup with the small js plugin and it works, which confirms that my plugin paths are ok. Currently what I am trying to do is to build a ts plugin and compile it to js using tsc. So far worked. A problem occurred when I created an additional ts file with helper functions inside. Importing those functions inside my-plugin.ts brakes my setup.
Here is how it looks like
/usr/local/kong/js-plugins/
  node_modules
  package.json
  my-plugin.js
  libs/
     helpers.ts

# my-plugin.js
import { sayItOutloud } from "./lib/helpers";

async access(kong: any) {
  const test = new sayItOutloud();
  console.log(test.sayHello("Max"));
}

# helpers.ts
export class sayItOutloud {
  sayHello(user) {
    return "Hello " + user + "!";
  }
}

Any tip would be appreciated! Thank you

Ok I manage finally to output entire logs by doing docker compose logs -f -t on a separate console after the container crashes, and I saw this error: Error: Cannot find module '/usr/local/kong/js-plugins/lib'
Full log you can find in here:
kong-with-js-plugin |2023-04-13T09:40:31.786106542Z Error: Cannot find module '/usr/local/kong/js-plugins/lib'
kong-with-js-plugin |2023-04-13T09:40:31.786181542Z Require stack:
kong-with-js-plugin |2023-04-13T09:40:31.786240542Z - /usr/local/lib/node_modules/kong-pdk/server.js
kong-with-js-plugin |2023-04-13T09:40:31.786278542Z - /usr/local/lib/node_modules/kong-pdk/bin/kong-js-pluginserver
kong-with-js-plugin |2023-04-13T09:40:31.786381542Z     at Function.Module._resolveFilename (node:internal/modules/cjs/loader:1075:15)
kong-with-js-plugin |2023-04-13T09:40:31.786783542Z     at Function.Module._resolveFilename.sharedData.moduleResolveFilenameHook.installedValue [as _resolveFilename] (/usr/local/lib/node_modules/kong-pdk/node_modules/@cspotcode/source-map-support/source-map-support.js:811:30)
kong-with-js-plugin |2023-04-13T09:40:31.786840542Z     at Function.resolve (node:internal/modules/cjs/helpers:116:19)
kong-with-js-plugin |2023-04-13T09:40:31.786898542Z     at Server.loadPlugins (/usr/local/lib/node_modules/kong-pdk/server.js:66:33)
kong-with-js-plugin |2023-04-13T09:40:31.786946542Z     at new Server (/usr/local/lib/node_modules/kong-pdk/server.js:50:12)
kong-with-js-plugin |2023-04-13T09:40:31.786992542Z     at Object.<anonymous> (/usr/local/lib/node_modules/kong-pdk/bin/kong-js-pluginserver:18:12)
kong-with-js-plugin |2023-04-13T09:40:31.787029542Z     at Module._compile (node:internal/modules/cjs/loader:1254:14)
kong-with-js-plugin |2023-04-13T09:40:31.787078542Z     at Object.Module._extensions..js (node:internal/modules/cjs/loader:1308:10)
kong-with-js-plugin |2023-04-13T09:40:31.787127542Z     at Module.load (node:internal/modules/cjs/loader:1117:32)
kong-with-js-plugin |2023-04-13T09:40:31.787167542Z     at Function.Module._load (node:internal/modules/cjs/loader:958:12) {
kong-with-js-plugin |2023-04-13T09:40:31.787215542Z   code: 'MODULE_NOT_FOUND',
kong-with-js-plugin |2023-04-13T09:40:31.787253542Z   requireStack: [
kong-with-js-plugin |2023-04-13T09:40:31.787288542Z     '/usr/local/lib/node_modules/kong-pdk/server.js',
kong-with-js-plugin |2023-04-13T09:40:31.787325542Z     '/usr/local/lib/node_modules/kong-pdk/bin/kong-js-pluginserver'
kong-with-js-plugin |2023-04-13T09:40:31.787360542Z   ]
kong-with-js-plugin |2023-04-13T09:40:31.787560542Z }
kong-with-js-plugin |2023-04-13T09:40:31.798525542Z 2023/04/13 09:40:29 [error] 1#0: init_by_lua error: ...al/share/lua/5.1/kong/runloop/plugin_servers/process.lua:150: Not a plugin info table:
kong-with-js-plugin |2023-04-13T09:40:31.798597542Z /usr/local/bin/kong-js-pluginserver --plugins-directory /usr/local/kong/js-plugins --dump-all-plugins

Why is a folder considered a module? And also, do you have any idea how could I decouple my code into multiple files like in the approach I took in my question?

At this point, the JS plugin server only supports JavaScript and requires the specific directory layout described in the sample plugin.  We realize that this is restrictive, but we don't currently have plans to fix this ourselves, but you are welcome to send a pull request.


ImageCropView Pan & pinch gestures not work in XF3.5

ImageCropView Pan & pinch gestures not work in XF3.5

I am also seeing this problem with XF3.6 and the latest FFImageLoading. When reverting FFImageLoading and the Transformations nuget back to <IP_ADDRESS>0, the crop / zoom transformations work again.
However, I don't want to use such an old version of FFImageLoading due to the bug fixes that have been added in newer versions.

I ran into this issue when I upgraded to XF <IP_ADDRESS>4457 and FFImageLoading <IP_ADDRESS>3 and saw the image updated on a gesture then swiched back to it's original size/position almost immediately and never seemed to update as it did in the past, always falling back to the original image. After some digging I found if I commented out the line of code that sets the CacheKey in the source for CustomCachedImage (which is part of the ImageCropView.cs file)
            protected override void SetupOnBeforeImageLoading(TaskParameter imageLoader)
            {
                //imageLoader.CacheKey(_isRefined ? _refinedCacheKey : _cacheKey);
                base.SetupOnBeforeImageLoading(imageLoader);
            }

It now seems to work again. I haven't had a chance to try to dig any further into why this was causing an issue, but for now I can get it working as it was before I updated.

I also see what is a bug on clearing the cache in the SetSource method of CustomCacheImage where it clears the wrong cache key when the Source is set:
                    if (!string.IsNullOrWhiteSpace(this.refinedCacheKey))
                    {
                        await ImageService.Instance.InvalidateCacheEntryAsync(this.cacheKey, FFImageLoading.Cache.CacheType.Memory, true).ConfigureAwait(false);
                    }

Considering that the if statement is for refinedCachceKey the next line should be invalidating the refinedCacheKey, not the cacheKey, but unfortunately this in itself doesn't fix the problem either.

It work ~ thx

any news on this? @rex232   how did you mange make it work?

I follow lee parrish first answer to change the source, then it work
在 2019年7月23日 週二 22:59，shoeIT<EMAIL_ADDRESS>寫道：

any news on this? @rex232 https://github.com/rex232 how did you mange
make it work?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/daniel-luberda/DLToolkit.Forms.Controls/issues/268?email_source=notifications&email_token=AAOOCIULKGMMAYJGMJL364LQA4MGVA5CNFSM4HOQS3CKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2TM6SI#issuecomment-514248521,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOOCIVHZ4B6N24FRIPW7SDQA4MGVANCNFSM4HOQS3CA
.


change default rule creation behavior to have status=ACTIVE

see #636
PG1 85 approved.

@starpit This changes the default state to enable/active when a rule is created right?
This means we need two more things:

Update docs (example says that it needs to use --enable so the rule is enable on creation)
https://github.com/openwhisk/openwhisk/blob/master/docs/triggers_rules.md
With this change CLI users will not be able to create a rule that is disable anymore, the --enable flag on create doesn't make sense anymore, it means a new flag needs to be introduce (i.e. --disable) for wsk rule create --disable rulename trigger action

$ wsk rule create -h
usage: wsk rule create [-h] [-u AUTH] [--shared [{yes,no}]] [--enable]
                       name trigger action

positional arguments:
  name                  the name of the rule
  trigger               the trigger
  action                the action

optional arguments:
  -h, --help            show this help message and exit
  -u AUTH, --auth AUTH  authorization key
  --shared [{yes,no}]   shared action (default: private)
  --enable              enable rule after creating it

cc @mdeuser

@rabbah suggests handling the client update separately. How generally do you handle docs changes? Rolled in, or separate PR (I would imagine that docs updates have a distinct set of reviewers, which led me to limit this to the controller change, but am happy to go with the standard practice)

Yeah that's OK other team members can take care of the doc and go-CLI changes.
I was trying to note what were the impacts of these PR, and the fact that CLI changes have not being discuss before.
Any other impacts from this PR?

@csantanapr thanks for the help with this! i agree with the need to update the docs and the CLI. the third piece here is the UI. i have already discussed this with the UI team.

Is there an issue for the required CLI updates?

@mdeuser not yet. we should discuss the terminology there. e.g. is it --no-enable? --no-start (this is the terminology of cf)?

@starpit @mdeuser I created issues #1014 (docs) and #1015 (go-cli)
My proposal for CLI changes are in #1015
No need for a --no-enable, user can just run wsk rule disable after creation making things more simple.

I'ld suggest to at least have the doc changes in this PR. Having self-contained changes in a PR is pure goodness.

but the docs changes refer to the CLI (after all, this is not an API schema updates, this is a semantic update). so by this reasoning, we also need to roll he CLI changes into this PR?

@markusthoemmes i just pushed a change to the docs. for sequential updates like this, do i need to rebase the two commits into one?

I would preferred to have everything in one PR as general rule (code, tests, doc), but that's just a preference.
If @starpit to include the doc and CLI updates in this PR, that's OK by me.

The API currently ignores the "status" property when creating/updating a rule (PUT). An alternative strategy is to allow PUT to specify the desired state of the rule. The default can still be enabled but this would at least allow one to create a rule in an inactive state which we otherwise can't do after this change.
If we were to do that then --enable in the CLI can be set as the default with a --disable added for symmetry.

i'm trying to think of cases where an atomic create-but-disabled is important. if you aren't ready for your rule to be live, then don't create it.
cf's "no-start" option is important for them, because their push command doesn't not accept all of the necessary arguments for starting. e.g. environment variables and service bindings. so you push the app, which then lets you associate env vars and services... then you start it.
i don't see any reason why we would need to have that kind of two-phase startup, for openwhisk?

I haven't come up with a compelling reason.
So if a rule is created always in enabled state, is the required CLI change to just remove --enable?

Yes @rabbah check #1015

ok, i've updated the WskBasicTests rule test. it now

tries to update the rule, and expect CONFLICT
disables the rule
tries to update the rule, and expect SUCCESS

PG1 99 passed


add CONTRIBUTING.md

We have one for this repo. Could we be more specific on what the ask is here?
cc: @boris-drazic


Optimize and fix up some SVGs

Addresses some icons I missed in my SVG optimization PRs. A few of them are small fix-ups, like tiny lines in the middle of arcs. This saves about 5 MB.
Also I changed the opacity of some icons slightly. Many icons used alphas like 50 or 100. When doing 50/255 for example, it can get thrown off and become something like 0.19608 when opened in an editor. The alpha values 51, 102, 153, and 204 result in no precision inconsistencies (as they are 0.2, 0.4, 0.6, 0.8) so I used them where the change isn't noticeable. Even if it's noticeable, it's probably for the better, as some people have complained about low opacity icons. Where not applicable (e.g. an alpha value of 20, which isn't close to any of these), I changed to use at most 3 digits of precision.
^ I don't think this needs to be enforced strictly, it's just a nice thing to do once IMO, and also artists probably often copy the opacity of existing icons. (or maybe it's just me doing things like that? :sweat_smile:)

This saves about 5 MB.

I think you meant 5 KB? :stuck_out_tongue:

Fixed!

Of course this went on to become more of a dumping ground for more tweaks and improvements, so it's more like 8 kB now.

Thanks!


Vue v2.6+ nextTick 原理已经不一样了，希望注明或更改文档

next-tick.js
/* @flow */
/* globals MutationObserver */

import { noop } from 'shared/util'
import { handleError } from './error'
import { isIE, isIOS, isNative } from './env'

export let isUsingMicroTask = false

const callbacks = []
let pending = false

function flushCallbacks () {
  pending = false
  const copies = callbacks.slice(0)
  callbacks.length = 0
  for (let i = 0; i < copies.length; i++) {
    copies[i]()
  }
}

// Here we have async deferring wrappers using microtasks.
// In 2.5 we used (macro) tasks (in combination with microtasks).
// However, it has subtle problems when state is changed right before repaint
// (e.g. #6813, out-in transitions).
// Also, using (macro) tasks in event handler would cause some weird behaviors
// that cannot be circumvented (e.g. #7109, #7153, #7546, #7834, #8109).
// So we now use microtasks everywhere, again.
// A major drawback of this tradeoff is that there are some scenarios
// where microtasks have too high a priority and fire in between supposedly
// sequential events (e.g. #4521, #6690, which have workarounds)
// or even between bubbling of the same event (#6566).
let timerFunc

// The nextTick behavior leverages the microtask queue, which can be accessed
// via either native Promise.then or MutationObserver.
// MutationObserver has wider support, however it is seriously bugged in
// UIWebView in iOS >= 9.3.3 when triggered in touch event handlers. It
// completely stops working after triggering a few times... so, if native
// Promise is available, we will use it:
/* istanbul ignore next, $flow-disable-line */
if (typeof Promise !== 'undefined' && isNative(Promise)) {
  const p = Promise.resolve()
  timerFunc = () => {
    p.then(flushCallbacks)
    // In problematic UIWebViews, Promise.then doesn't completely break, but
    // it can get stuck in a weird state where callbacks are pushed into the
    // microtask queue but the queue isn't being flushed, until the browser
    // needs to do some other work, e.g. handle a timer. Therefore we can
    // "force" the microtask queue to be flushed by adding an empty timer.
    if (isIOS) setTimeout(noop)
  }
  isUsingMicroTask = true
} else if (!isIE && typeof MutationObserver !== 'undefined' && (
  isNative(MutationObserver) ||
  // PhantomJS and iOS 7.x
  MutationObserver.toString() === '[object MutationObserverConstructor]'
)) {
  // Use MutationObserver where native Promise is not available,
  // e.g. PhantomJS, iOS7, Android 4.4
  // (#6466 MutationObserver is unreliable in IE11)
  let counter = 1
  const observer = new MutationObserver(flushCallbacks)
  const textNode = document.createTextNode(String(counter))
  observer.observe(textNode, {
    characterData: true
  })
  timerFunc = () => {
    counter = (counter + 1) % 2
    textNode.data = String(counter)
  }
  isUsingMicroTask = true
} else if (typeof setImmediate !== 'undefined' && isNative(setImmediate)) {
  // Fallback to setImmediate.
  // Techinically it leverages the (macro) task queue,
  // but it is still a better choice than setTimeout.
  timerFunc = () => {
    setImmediate(flushCallbacks)
  }
} else {
  // Fallback to setTimeout.
  timerFunc = () => {
    setTimeout(flushCallbacks, 0)
  }
}

export function nextTick (cb?: Function, ctx?: Object) {
  let _resolve
  callbacks.push(() => {
    if (cb) {
      try {
        cb.call(ctx)
      } catch (e) {
        handleError(e, ctx, 'nextTick')
      }
    } else if (_resolve) {
      _resolve(ctx)
    }
  })
  if (!pending) {
    pending = true
    timerFunc()
  }
  // $flow-disable-line
  if (!cb && typeof Promise !== 'undefined') {
    return new Promise(resolve => {
      _resolve = resolve
    })
  }
}

文章已经注明了分析的版本是  Vue.js 2.5.17-beta.0，
不过未来有时间我会更新文章的。

文章已经注明了分析的版本是 Vue.js 2.5.17-beta.0，
不过未来有时间我会更新文章的。

你好，vue-router分析的时候是用的哪个版本的代码？


Server error 500

Hi, hope you are doing great, I really like the idea behind this package, always wanted a static site generator with django. I try to integrate coltrane to a simple django project (the project was generated with adamchainz simple-core template). The project is really simple, there is no custom code, I created the content directory at  the root and add two simple markdown files (index.md and simple-text.md), when I try to visit the corresponding urls I get a 500 error (nothing in the console to help), coltrane seems to be working though, when I visit a non existent route I get a 404. The project code is here, I would love if you could help me solve this issue. Thanks

@Tobi-De I made a PR at https://github.com/Tobi-De/dotfm/pull/1, let me know if it works for you.

@Tobi-De I made a PR at Tobi-De/dotfm#1, let me know if it works for you.

Thanks so much, It's working now, I will continue to play with it to see if there are other issues. I never paid attention to the LOGGING setting, now at least I know exactly what this configuration is for.


pull request

i did some easy stuff

Pull request rejected for non-compliance with the help command. Please add your command shortcuts to the help menu. Everything else looks good, open a new request when compliant.


Contributor fetch

Closes: #670
Description of Changes

Fetch all the contributors data using axios api
Properly added the image and name of the contributros
Now the teamData.js file is not required anymore as the data is dynamically fetched from github repository

Checklist:

[ x] My code adheres to the established style guidelines of this project.
[x ] I have conducted a self-review of my code.
[ x] I have included comments in areas that may be difficult to understand.
[ ] I have made corresponding updates to the project documentation.
[x ] My changes have not introduced any new warnings.

Screenshots

Original
Updated

Please provide any necessary screenshots to illustrate the changes made.

Hey @anmode Please review it

Please keep those member too which you commented out!

Please keep those member too which you commented out!

Its already include them


Auto Aggregate Reflection

As of C++20 it's possible to auto-magically get the names of the members of aggregates on all major compilers (in addition to auto-magically getting the values and types - which was already possible); integrate support for auto-reflecting aggregates when on C++20 & using them with the regular RareCpp interfaces.

Auto-reflection of aggregates was released in 2.3.0, see documentation
This is only available when compiling against C++20 or higher and has many limitations that the macros do not as described here.
example1: https://godbolt.org/z/K75dhj5K4
example2: https://godbolt.org/z/GcKharfxj
struct Item
{
    int id = 0;
    std::string name {};
};

struct Collection
{
    std::string description {};
    std::vector<Item> items {};
};

int main()
{
    Collection collection { "my collection", {{0, "first"}, {1, "second"}} };
    std::cout << Json::pretty(collection);
}

Output:
{
  "description": "my collection",
  "items": [
    { "id": 0, "name": "first" },
    { "id": 1, "name": "second" }
  ]
}


GPU Support

Apparently GPU support has been removed again recently. There are still the inactive feature branches gpu-support and nvidia-support but commit 4a1ccd1 removed it from main.
I cannot find any information though about the reasons behind this. Could the developers clarify the issue, if this is still on the roadmap and if and what help you would need?

Hi @janmuennich
Yes, we're about to add GPU support again. ETA is some time next week.

v1.2.0 is out now with NVIDIA GPU support


Can't set breakpoint in webworker

I'm using browserify and webworkify with sourcemaps but any breakpoint I set in the sourcemap'd file in Chrome is not hit.
My gulpfile task looks like this:
gulp.task('processjs', function () {
  var b = browserify({
    entries: 'app/js/app.js',
    debug: true,
    transform: [cacheify(babelify, babelifyCache), cacheify(ngAnnotate, ngAnnotateCache)]
  });

  return b.bundle()
    .on('error', errorHandler)
    .pipe(source('all.js'))
    .pipe(buffer())
    .pipe(sourcemaps.init({loadMaps: true}))
    // post-browserify transform tasks go here
    .pipe(uglify())
    .pipe(sourcemaps.write('.'))
    .pipe(gulp.dest('dist'));
});

are you sure your transforms are not the problem?

I have no idea, I'm kinda new to this.
However, when an error occur in the webworkified worker, the stopping point in the Chrome debugger is in an embedded data URI. I.e. the source of the worker has been converted into a data URI. So, it's not actually executing the source that was transformed by browserify, which would have sourcemap annotations.

Also, I can set breakpoints in other code which was processed by the same set of transforms.

in the mean time you can use debugger

Could you elaborate a little on what you mean by debugger?

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/debugger

I see, thanks.

Also running into this issue. The debugger statement does work, but it would be nice to be able to set breakpoints as well. Anyone had any luck getting breakpoints to work?

This is something that's not possible to fix because worker sources are created dynamically (with blob URLs), so breakpoints you set on a generated source don't persist after reloading a page.
debugger statements are a simple and effective workaround, so I'm closing the issue as not actionable.

See also http://stackoverflow.com/a/24198720/52207


helper.js add check package.json existence before require

Bug Report

[x] I’ve read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I’ve read and agree to the Code of Conduct.
[x] I’ve searched for any related issues and avoided creating a duplicate issue.

What happened
Some folders and files like lib, node_modules in package folder are in .gitignore, so when you switch branches these folders stays on place. When you collect folders(e.g. modules) for further usage these "empty" module also gets into result. This provides an error when you try to require package.json from it.
Error: Cannot find module '/var/www/github/design-system-components/packages/file-upload/package.json'
    at Function.Module._resolveFilename (module.js:555:15)
    at Function.Module._load (module.js:482:25)
    at Module.require (module.js:604:17)
    at require (internal/module.js:11:18)
    at Object.json (/var/www/github/design-system-components/scripts/helper.js:642:26)
    at Object.init (/var/www/github/design-system-components/scripts/helper.js:625:20)
    at Object.HELPER.init (/var/www/github/design-system-components/scripts/helper.js:1233:19)
    at Object.<anonymous> (/var/www/github/design-system-components/scripts/helper.js:1254:8)
    at Module._compile (module.js:660:30)
    at Object.Module._extensions..js (module.js:671:10)

What I expected to happen
No error :)
Reproducing

Module name: scripts/helper.js

Steps to reproduce:

Checkout new branch
Init, build and commit new component
Switch back
cd into the any existing package folder
run npm run watch

@toxamiz I've left some feedback in #697 but I admit that I don't think I fully understand what you are talking about? lib/ and node_modules/ certainly have a place in the .gitignore like you mention but your referring to references to ignored components not updating?

You can follow steps to reproduce.
I have a branch with new component and some commits. Then i switch to develop.  Git updates files to this state, so files from my component no longer presents. But files from .gitignore stays in place.
// my branch

my-component
- lib
- node_modules
- tests
- src
package.json
...other

// switch to develop, these folders stays in place

my-component
- lib
- node_modules
- tests

In helper.js you collect all folders, and then require package.json from each. But in my-component  package.jsondoesn't exists, it is in other branch. Folder exists, package.json - not. That causes an error.

This is expected behaviour? When switching from a branch with a new component to something like develop, the files will be ignored as they are not on the targeted branch...
I would suggest looking at options like git stash, see: https://git-scm.com/docs/git-stash when switching between branches and commits


主线合并

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 56.308%

Files with Coverage Reduction
New Missed Lines
%

<EMAIL_ADDRESS>1
16.07%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1417

Relevant Lines:
2471

💛  - Coveralls


test(clerk-js): Upgrade @testing-library packages

Description
This PR addresses the following deprecation warning in our unit tests

Checklist

[ ] npm test runs as expected.
[ ] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[ ] 🐛 Bug fix
[ ] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

@dimkl the latest @testing-library/react requires ^18.0.0 which is the same as the minimum supported react version we also support for both clerk-js and clerk-react.


Cannot build android on version 2.1.3

Expected Behaviour
the project will build on android
Actual Behaviour
the project will not build
Reproduce Scenario (including but not limited to)
install push plugin and try to build
Platform and Version (eg. Android 5.0 or iOS 9.2.1)
android 7.0.0
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
any
Cordova CLI version and cordova platform version
cordova --version                                    8.0.0
cordova platform version android                     7.0.0

Plugin version
cordova plugin version | grep phonegap-plugin-push   2.1.3

Logs taken while reproducing problem
(node:79798) UnhandledPromiseRejectionWarning: Error: Source path does not exist: google-services.json

As the error says, you need the google-services.json file

@jcesarmobile I didn't need that file before this plugin. Could you tell me what I need to put in this file? Or could your plugin create this file on installation?

You have to download it from firebase console, it’s a requirement for push notifications, it contains information about your project and google provides it after project creation (on firebase console)


mitochondrial_targeting_signal (SO:0001808)

SO term name and accession
Please give the term name and accession. Example: rRNA_gene (SO:0001637)
mitochondrial_targeting_signal (SO:0001808)
A polypeptide region that targets a polypeptide to the mitochondrion.
Suggested new parent term name and accession
Please give the term name and accession of the parent you suggest for the term. Example: ncRNA_gene (SO:0001263)
transit_peptide (SO:0000725)
The transit_peptide is a short region at the N-terminus of the peptide that directs the protein to an organelle (chloroplast, mitochondrion, microbody or cyanelle).
Reason for the change
A mitochondrial_targeting_signal (SO:0001808) is a transit_peptide (SO:0000725)
if that is not the case , we would need a grouping term that covered both mitochondira targeting signal and (mitochondrial) transit peptide
Relevant Publications
Please list a number of relevant publications that support the name, definition, and ontological location of the term you are requesting. It is best if you can include a PubMed ID or the last name of the first author along with the publication year, journal-title and potentially the article title.

ignore


[9.x] Added env_as_array helper.

This PR adds env_as_array helper.
Sometimes it's useful to be able to set array values in an env variable. Here is an example of usage in a config file:
'foo' => env_as_array('FOO'),
or with a defined values separator and a default value
'foo' => env_as_array('FOO', '|', ['bar', 'baz']),
And in .env file it looks like that:
FOO=bar,baz

After that, I can foresee someone will want to return collections :)

$array = explode(',', env('KEY', ''));

What if trim of each array value is required?

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

I find it fine to just put explode(',', env('FOO') ?? '') in my config files.


Only extract product-info.json of downloaded archive

This is in regard to issue #1.
We are preventing double space occupation on the hard drive of the machine extracting the release archive.
There are multiple ways to achieve this:
This is the ansible way:
        remote_src: true
        include:
          - */product-info.json
        extra_opts:
          - --strip-components=1

Which would translate to
            "/usr/bin/gtar",
            "--extract",
            "-C",
            "/tmp/ideaIU-2023.3.1",
            "-z",
            "--show-transformed-names",
            "--strip-components=1",
            "-f",
            "/tmp/ideaIU-2023.3.1.tar.gz",
            "*/product-info.json"

In this setup * would match everything including /.
I am not 100% sure if --wildcards somehow gets implicitly set, because the manual for gnu tar (https://www.gnu.org/software/tar/manual/html_section/wildcards.html) says that default for inclusion is --no-wildcard
My solution --no-anchored just searches for the given filename anywhere in the archive. This should be the same behaviour as described above in a less ansible way, but more reliable functional.
Of course this does not work zip archives etc. There unarchive's include would be more suited to do the job. But since we are working with tar files only for now (extension tar.gz) and therefore only use tar, I would say it is okay to use that option.
Also --strip-components is tar-only AFAIK.

Also I do not know what those molecule test do, here are the CI runs :D
https://github.com/func0der/ansible_collection_diademiemi.jetbrains/actions

This seemed to also have fixed CI then! I could never get the CI working in GitHub Actions (They did work locally, but took too long, I think GH shuts them down if they use too much CPU).
The molecule tests just check whether every task runs successfully on 9 distros, and does a rerun to check for idempotency.
Thank you! I'll merge this PR now!

No, thank YOU for taking the time to make this amazing collection :D


Remove an unnecessary string allocation

This removes an potentially unneeded allocation when processing messages. Previous to this change, the processMsg function always converted the subject from a []byte to a string, thus allocating, and used it to lookup or set a cache entry.
By moving the conversion for the cache lookup into the map access, we can use an optimization introduced in Go 1.3 that avoids allocating the string for the map lookup.
In case where we insert into the cache we still need to allocate, because we need a string for the Match function and also because setting the map key still needs to allocate, if the key isn't already in the map (if I remember correctly).
These are the results from the publish benchmarks from github.com/nats-io/nats/test:
benchmark                          old ns/op     new ns/op     delta
BenchmarkPublishSpeed-4            394           339           -13.96%
BenchmarkPublishSpeedViaChan-4     2005          2013          +0.40%

benchmark                          old allocs     new allocs     delta
BenchmarkPublishSpeed-4            1              0              -100.00%
BenchmarkPublishSpeedViaChan-4     4              3              -25.00%

benchmark                          old bytes     new bytes     delta
BenchmarkPublishSpeed-4            3             0             -100.00%
BenchmarkPublishSpeedViaChan-4     169           167           -1.18%

Both old and new number are the best of 3 runs. The slow down in BenchmarkPublishSpeedViaChan is just noise and fluctuates between + and - on the machine I used to test.

Coverage decreased (-0.09%) to 79.785% when pulling 4bf0b47e7437fbad6e0ccbe9dfb60b467125fcdc on nuss-justin:remove-allocation-1 into de48a4b4a5e6fceb45fe4f25e0ff44a23e05f267 on nats-io:master.

Nice catch.


ColumnDefinition with Star sizing and MinWidth leads to clipping

Describe the bug
Given a Grid layout with 2+ columns that all have star sizing, if the first column has MinWidth set to a significant value then it is possible to produce clipping when resizing the window. This issue only occurs when the first column has MinWidth set.
This bug has even more complicated behavior when interacting with nested Grids (see Screenshots for details).
Steps to reproduce the bug
The bug can be reproduced using the following layout in a UWP project:
<Grid>
    <Grid.ColumnDefinitions>
        <ColumnDefinition Width="*" MinWidth="320" />
        <ColumnDefinition Width="2*" />
    </Grid.ColumnDefinitions>
    <Rectangle
        Grid.Column="0"
        HorizontalAlignment="Stretch"
        VerticalAlignment="Stretch"
        Fill="Firebrick" />
    <Image
        Grid.Column="1"
        HorizontalAlignment="Stretch"
        VerticalAlignment="Stretch"
        Source="/Assets/StoreLogo.png"
        Stretch="Uniform" />
</Grid>

Resizing the window will clip the image in the second column.
Expected behavior
Columns that don't have MinWidth set should scale properly without clipping.
Screenshots
Using the sample layout above, the bug seems to occur once the second column width goes down to 320 * 2 = 640

When dealing with nested Grids, the bug only occurs when there is not enough space to display everything in the child Grid. Using CommandBar for example:

Another behavior when the CommandBar spans across 2 columns

NuGet package version
Microsoft.UI.Xaml 2.7.0
Windows app type

[X] UWP
[ ] Win32

Device form factor
Desktop
Windows version
21H2 (22000.376)
Additional context
No response

@gegao18 thoughts on whats going on here?


[DOCS] Add README.md file

Add a README.md file with instructions to. start the application

Hi, can you assign this issue to me?

I can add pictures of the homepage/frontend so devs know what it looks like when up and running

@HKAgrwl good idea but let's add the homepage/landing image later as it's not ready yet. But could you add icons for the tech stack? like you can add Nextjs/React icon beside the React/Nextjs and Nodejs icon beside the nodejs text.

add this discord link too - https://discord.gg/M2BMPdku

OK

@HKAgrwl assigned it to you. please work on it and raise a PR

@HKAgrwl could you please let me know if you can resolve the comments in the PR #15 ?

this issue is open to be worked. if anyone interested please create a PR. Thanks


Add Meghan's profile to syllabus

@nquarder @eculler Added profile!  Typo Concentration to Cert.

nice work @megsmedes !!


'least' cross-entropy also used in classifier;  'closest-exp' relaxes same concept requirement.

Cross-entropy and relaxed concept restriction issues addressed.  Will put 'coverage' study in its own repository.

New 'least' function uses cross-entropy from Markov chain LM to detect borrowings.
1.1. More importantly, it's integrated into 'classifier' (SVM) as another function.
1.2. Least is complementary with 'closest' and together they give best overall performance.
Cleaned up 'cognate' to drop several lexstat references and some un-useful model variants.  Can always add if we move in that direction in subsequent steps.
New 'closest_exp' offers choice of concept matching restrictions: target (default), central, nested (target [central]), and none. Verified that 'target' restriction gives same results as 'closest', so regression to current version OK.

Updates to cldf/ and README maybe not necessary since they are replaced by the install/activation. But I realized the versions in main were not current.

@fractaldragonflies, the conflicts are minimal, as far as I can see. you just have to re-run the cldf creation code after merging. This does not touch the main cldf conversion and other command, so it is still fine by me to merge this. We also have a release with the data that was used in the paper, so no worries, just merge.


Separate RunTime package into One for FMX and one for VCL

I recently made use of your project to replace the TWebBrowser with the Edge engine.  Thank you for open sourcing this project.
When it came time to deploying, I discovered that despite the fact that my application is for the VCL, I now need to deploy the FMX BPL (~12Mb) because the run-time package includes the FMX browser and related classes.  I have removed those references and compiled a different run-time package, but as a result, it makes it hard to manage your repo as a Git sub-module and keep everything up to date.
Would you please separate the run-time package into one for the VCL and one for FMX and have the Design-time package use both of them?

Thanks for reporting this issue!
I'll create separate packages.

All your changes are merged now.
Thanks again!


API stream false doesn't seem to work

I am trying to use the rest api and I am posting the following
{   "model": "mistral",   "prompt":"tell me a fancy joke", "stream": false }
And I get the following response.  Doesn't stream false disable partial responses?  Which platform are you on? It may be the case that you're not hitting the API endpoint of the container
Here's what I see from the latest Docker image (ollama/ollama:0.1.2):
curl -X POST http://localhost:11434/api/generate -d '{"model": "llama2", "stream": false, "prompt": "hello"}'

{"model":"llama2","created_at":"2023-10-13T16:10:13.407234042Z","response":" Hello! It's nice to meet you. Is there something I can help you with or would you like to chat?" ...}

I'll close this for now since I think there's a version mismatch, but please do re-open it if the issue keeps happening 😊

Hi @jmorganca  I faced the same problem on Mac. Should I install from the source if I want to get the latest Ollama?

For those that are still having this issue, this is what solved it for me
docker pull ollama/ollama:latest


[stable/fluent-bit] Add Pod Security Policy

Signed-off-by: Fernando Mendoza<EMAIL_ADDRESS>
What this PR does / why we need it:
Add the option to create a Pod Security Policy.
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
[x] Title of the PR starts with chart name (e.g. [stable/chart])

@kfox1111
@edsiper
@hectorj2f

/assign
/ok-to-test
/lgtm

/lgtm

/lgtm