Improved visualization of the OpenDRIVE Standalone mode

Description
The code changes improve the visualization of the road mesh for a given .xodr file. For one a texture gets applied to the generated road mesh and additionally a separated mesh for the lane markings gets generated. This is necessary since only one texture can be applied to one mesh. Currently only the marking types "solid" and "broken" are supported and for anything else will default to "solid".
The code for the mark mesh follows the same scheme as the generation of the road mesh, i.e. a ChunkedGeneration iterating over every Road -> every Section -> every Lane. The differences follow from the fact that "broken" markings are more complex to handle, since one needs to remember if currently a "line" or a "gap" has to be generated and the remaining length of said line/gap as well. A HashMap is used to track this information for a pair of <RoadId, LaneId>, I will refer to this as the "mark_info" in the following. This mark_info is particular important if one road ends and another starts. Thus the successor(s) of a road are needed. If the successor is a junction the information about the actual RoadIds cannot be accessed in MeshFactory.cpp. Hence an additional HashMap gets generated in Map.cpp that maps a RoadId to a vector of RoadIds i.e. the successors of the road. Thus the order in which the roads are handled is important, currently this is done in ascending order of their RoadId. The reason for this decision follows from the fact that in general the road network can be viewed as a cyclic graph and I am not aware of an algorithm that can order such a graph. Hence the assumption "for every road x the Id(s) of the successor road(s) are bigger than x.Id" was made. The code still works in case this assumption is violated, but might generate lines/gaps of varying length around transitions of roads. The overall length of the line/gap of a broken mark are defined in road_params.
When GenerateMark is called on a lane the RoadInfoMarkRecord gets looked up first, to acquire the information about the type of marking (solid, broken) and the width. Then a key with the current RoadId and LaneId is used to check if there is already a mark_info in the HashMap and if not then a new entry is generated. The 3d points of the marking gets calculated similarly to the points of a road. In a loop the corner points of the lane at the current position are calculated and depending on the LaneId one or the other is used as the center for the marking. Using this point and the width of the marking the actual two points are calculated and recorded. Additionally if the remaining length of a line/gap is less then the road_param.resolution then the end points of the line/gap are calculated and the mark_info gets updated, i.e. switch line to gap and vice versa and set the new remaining length. Finally depending if it is the end of a section or the end of the road the mark_info gets recorded for the current key or for the successors.
Fixes #
Where has this been tested?

Platform(s): Ubuntu 20.04
Python version(s): 3.8.5
Unreal Engine version(s): 4.24

Possible Drawbacks
None that I could think of, as only visuals are added.
Screenshots
Before:

After:

This change is

@Axel1092 as main feature reviewer.

Hi @fitnessflo, sorry for the late response.
Thank you for this pull request, this looks a very nice addition to the standalone mode. We will start revising and revising it and give you feedback soon.

Hi @fitnessflo, I left a few comments on specific lines but overall I don't really understand why you keep track of successors and the mark_lane_memory variable. It seems to me that for a lane part of s1 to s2 you only need the current information to paint a solid line or a broken one.
In addition, the algorithm seems to be a bit broken:

In this example I used Town10HD (CarlaUE4/Content/Carla/Maps/OpenDrive/Town10HD.xodr) and only a few marking are present while most are missing.

Hi @Axel1092 , I also want to draw lane mark in this mode. Do you have any other advises? I have already tried to modify the source code refering this website, but still struggling with some bug in Make launch. Thank you for your time!

Hello @fitnessflo. I wish to draw lane mark in this mode. Could you give me some advice? I study your code for some time, which inspired me. But I am still having some trouble on building it. Did you have other update recently? Thank you!

@fitnessflo I am sorry  for the late reply, and i sincerely appreciate your reply! Thank you!  Now, I can rebuild your work, some changes may take into consideration, since the version of Carla is 0.9.13 now. If you want to know the change, i can update it for you. Here is my email<EMAIL_ADDRESS>I am a undergraduate who will go to Singapore for EE master from China.


[SPARK-39960][BUILD] Upgrade mysql-connector-java to 8.0.30

What changes were proposed in this pull request?
Upgrade mysql-connector-java from 8.0.29 to 8.0.30
Why are the changes needed?
This version brings some bugs fixes, the release note as follows:
https://dev.mysql.com/doc/relnotes/connector-j/8.0/en/news-8-0-30.html
Bugs Fixed, eg:

Historically, MySQL Server has used utf8 as an alias for utf8mb3. Since release 8.0.29, utf8mb3 has become a recognized (though deprecated) character set on its own for MySQL Server and to make things consistent, in release 8.0.30, any collations prefixed with utf8_ are now prefixed with utf8mb3_ instead. To go with that change, Connector/J has updated its character set and collation mapping accordingly in this release, and users are encouraged to update to Connector/J 8.0.30 to avoid potential issues when working with MySQL Server 8.0.30 or later. (Bug #34090350)

The description for the connection property rewriteBatchedStatements has been corrected, removing the limitation that server-sided prepared statements could not take advantage of the rewrite option. (Bug #34022110)

DatabaseMetaData.getTypeInfo always returned false for AUTO_INCREMENT for all data types. With this fix, Connector/J returns the correct value for each data type. Also, the missing types DOUBLE UNSIGNED and DOUBLE PRECISION UNSIGNED have been added to the ResultSet. (Bug #106758, Bug #33973048)

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass GA.

thanks, merged to master


Improve performance of preprocessor

Bootstrap Compilation takes 78 seconds.
Preprocessing takes 74 seconds.
So improving preprocessor would bring compile times down to under 5 seconds. That was the original performance goal, and now it is within reach.

Preprocess a b not preprocess a preprocess b

22s without any major structural changes.


un-mineable yellorium with iguana tweaks

after changing the mining level of yellorium with iguana tweak. it says on the tool tip that it can mine it but when i mine it, it will show you that you mined it but them pop back into place.

Let me get this straight, you changed the block with another mod and its not working, so therefore this mod is to blame?
It sounds more like an Iguana Tweak issue, as there seems to be nothing special in the ore code here.


[py2py3] make WMCore/Configuration.py suitable for CRABClient

Investigation related to https://github.com/dmwm/CRABClient/issues/4974
Status
In development
Description
CRABClient needs to be shipped with CMSSW environment that do not ship python-future, and CRABClient relies on WMCore.Configuration. I am investigating if it is feasible to remove the dependency on python-future from WMCore.Configuration. I do not think this is the correct way to go, i am opening this PR just to have an opinion from jenkins. After it, we can discuss this approach openly. A full run of jenkins unittests is worth 100h of speculation.
Is it backward compatible (if not, which system it affects?)
As always: it should be, but we will only discover with certainty after we tested it
Related PRs
not yet
External dependencies / deployment changes
nope (WMCore in general will continue to depend on python-future!)

This experiment failed badly! We will go back to the original plan, closing this. sorry for the noise!


[release-1.9] Switch logic for injection (#30013)

Switch logic for injection

Add release note

bikeshed

(cherry picked from commit 2364ce8940402ed6a6b0c3d31ed1e62bef29031e)
Unlike the master PR, this is disabled by default.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio_release-1.9
f2cbf2b4eb1a7d4eea4f381cd056bc1902102071
link
/test lint_istio_release-1.9

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio_release-1.9
f2cbf2b4eb1a7d4eea4f381cd056bc1902102071
link
/test lint_istio_release-1.9

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio_release-1.9
806e29467dbb64e5a89d11f5355cc9bdd0d8e822
link
/test lint_istio_release-1.9

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio_release-1.9
806e29467dbb64e5a89d11f5355cc9bdd0d8e822
link
/test lint_istio_release-1.9

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@howardjohn: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio_release-1.9
8b6952971b75198b159ea5d23df8bf04c5b67144
link
/test lint_istio_release-1.9

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio_release-1.9
8b6952971b75198b159ea5d23df8bf04c5b67144
link
/test lint_istio_release-1.9

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Page templates etc. in package

Either make page templates, or make an SQL dump available that has the needed templates. An SQL dump may be the only thing to do as many of the templates require multilingual support, which are quite difficult or impossible to deal with in the migration tool.

Look at issue #5

Should be fixed in part by 223f46de062bc4582235e268601f1bb7cc767c60

Still doesn't install correctly

Fixed by 1ad1199220cf8ba048235e82794b9bb17b7ceeef !
Sample content now works!

Fixed by 1ad1199220cf8ba048235e82794b9bb17b7ceeef !
Sample content now works!


New Release with module support?

Could we have have a release cut with go1.11 module support?

@Koshroy This is done now: https://github.com/piprate/json-gold/releases/tag/v0.1.1


improve document docs

One other question I had while trying to migrate my code base to this and which I think would be nice to clarify in the docs: Can I rely on DocumentDeserialize visiting my fields in the same order they were serialize into the document store?
If I cannot match field positionally between serialization and deserialization, I would need access to the relevant Field values when deserializing a structure field by field, i.e. something like DeserializeSeed to pass in a collection of Field values.

From the API contract there is no such guarantee, e.g. we may choose whatever is better compressible in the future.

But then we need to extend the API with a seed as otherwise, I do not see how I can deserialize the compile-time fields of struct without knowing their runtime field ID.

From the API contract there is no such guarantee, e.g. we may choose whatever is better compressible in the future.

But then we need to extend the API with a seed as otherwise, I do not see how I can deserialize the compile-time fields of struct without knowing their runtime field ID.

#2362 should provide what is required pass the runtime field ID to a deserialize implementation.


Interactive elements

Describtion
To increase the interactivity and user engagement of the Dashboard / App Builder, I propose the addition of easy-to-use and customizable buttons (interactive elements). This feature would enable users to quickly create buttons that can perform specific actions or trigger events within the dashboard, making the overall experience more interactive and dynamic.

Allow me to suggest to add a simple text link (or the ability to set one to a label) as one of the baseline components (I assume labels are going to be an integral part of this).
When linking to external pages (or even to set an anchor to jump to other parts of the dashboard), something that is less obtrusive than a button is often desirable. My typical use case is to add an external link to documentation/reference as part of a form.

We could add this functionality to the "ui_text" widget that's already available? Would provide the same formatting options, etc. that would likely be shared across the two components (if a separate component) anyway.

Closing this higher level item out as we have most of the interactive elements built, any remaining items have individual trackers/issues with their own respective priorities attached.


Rework Platform-related machinery

This PR adds the Platform class hierarchy (CPU64, Intel64, KNL, ... ARM, ..., Power, ...) so that the compiler can make platform-specific choices when it comes to optimization.
The nice thing now is that if you want to cross-compile (note: I'm talking about the code that devito generates, not the actual machine code) for a certain target architecture, now you can do it with DEVITO_PLATFORM=... and for example knl rather than hsw. On KNL, for example, nested parallel loops are collapsed and nested parallelism is enabled.
By default, we stick to the cpu64 mode, which is essentially what we have in master
This PR also drops now uselss environment variables: DEVITO_DLE_OPTIONS and DEVITO_ISA. In particular, the highest ISA now is bound directly to a Platform, so no need to set it up explicitly.

Codecov Report

Merging #792 into master will decrease coverage by 16.01%.
The diff coverage is 61.5%.

@@             Coverage Diff             @@
##           master     #792       +/-   ##
===========================================
- Coverage   88.93%   72.92%   -16.02%
===========================================
  Files         104      104
  Lines       11098    11204      +106
  Branches     2202     2211        +9
===========================================
- Hits         9870     8170     -1700
- Misses        949     2734     +1785
- Partials      279      300       +21

Impacted Files
Coverage Δ

devito/logger.py
83.09% <ø> (-1.84%)
:arrow_down:

devito/profiling.py
76.81% <ø> (-0.17%)
:arrow_down:

devito/yask/__init__.py
0% <0%> (-94.65%)
:arrow_down:

devito/yask/dle.py
0% <0%> (-100%)
:arrow_down:

devito/yask/operator.py
0% <0%> (-92.57%)
:arrow_down:

devito/yask/wrappers.py
0% <0%> (-86.39%)
:arrow_down:

devito/yask/utils.py
0% <0%> (-100%)
:arrow_down:

devito/core/__init__.py
100% <100%> (ø)
:arrow_up:

devito/ops/__init__.py
100% <100%> (ø)
:arrow_up:

devito/dle/parallelizer.py
90.67% <100%> (-1.69%)
:arrow_down:

... and 50 more

Continue to review full report at Codecov.

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

lscpu only works on linux system

indeed

so this gonna break on osx at least

no, or, I mean, this is not the point. That piece of code attempts to leverage lscpu -- we just hope it's available on the underlying system. But if it's not, we proceed silently (see Popen's stderr gets captured). IOW, the platform autodetection attempts a number of approaches and then if they all fail we still to the fallback CPU64, representing "a generic 64-bit cpu architecture"

all sorted out and green light given offline, so merging

merged.


[FEATURE REQUEST] Sidebar has high width

Description
I found that the sidebar has greater width and half of the space is empty.

Describe the solution you'd like
I would make it bit short.
Additional Notes (optional)
I would like to work on this issue under GSSOC'23

this is fixed.


Filter for haskell_binary|library|test

Closes https://github.com/google/hrepl/issues/10
These are the only kinds of targets that produce the compile_info_output_groups that hrepl requires. This is to avoid errors if non-Haskell targets are picked up as intermediate dependencies between targets specified on the command-line, e.g. through data dependencies. See #10.
This also enables users to call hrepl on wild-card targets. E.g. in the rules_haskell repository
hrepl //tests/binary-with-lib:all

Now no longer fails on with an error of the form
hrepl<EMAIL_ADDRESS>openBinaryFile: does not exist (No such file or directory)

Furthermore, this should help when running on big directories that otherwise leads to
haskell-check: exec: resource exhausted (Argument list too long)

Rebased on master.


bug: gap in tooltip arrow

What version of daisyUI are you using?
2.51.6
Describe your issue
I just noticed an issue I didn't notice before, so this might be a regression: there is a gap between the tooltip bubble and its arrow.

The screenshot above was taken from Chrome 113.0.5672.127 on Windows 10.
What browsers are you seeing the problem on?
Chrome
Reproduction URL (optional)
https://daisyui.com/components/tooltip/

I just checked an older build and the gap isn't there:

daisyUI v2.45.0

@saadeghi I think I found the root cause: the default screen settings in Windows 10 for my laptop was set to 125% scale:

Firefox shows the same gap on my laptop, so it might be the way Windows (or my laptop's Nvidia graphics card) is rendering the CSS at 125% scale. The issue went away after changing the scale factor to 100%.

Thanks for additional info.
Currently the arrow is using px values (unlike other styles which are using rem) and that is why it's not adapting the default zoom. I will fix this soon.

Hopefully you'll find a way to make it work because I just tried rem unit and still see the gap:

I was checking the docs and noticed that the gap is still there. Not only on top and bottom, but on the left side as well but not on the right side.

I'm using MS Edge with windows 11 scaled to 125%. Scaling to 100% makes everything too small to notice.

Is this still on the roadmap? Happens on Edge.


docs.kontena.io footer has random CSS sometimes

seen many, many times by me and others.
where should "cloud" issues be put?

seeing this almost daily still


Easing for view parameters

Allows interpolated transitions between view states
TODO:

[x] Document eased setters in tangram.h
[x] Document eased setters in MapController.java
[x] Last call to setter (with or without easing) determines final parameter value

Hey @ecgreb can you do an API thought experiment for me? If you have a function setMapPosition(x, y) that immediately moves to (x, y) and a function setMapPosition(x, y, t) that moves to (x, y) over t seconds, what would you expect to be the final positions of these sequences:
A
setMapPosition(x0, y0, 2);
setMapPosition(x1, y1, 1);

B
setMapPosition(x0, y0, 1);
setMapPosition(x1, y1);

@blair1618 I would expect the final position to be (x1, y1) in both scenarios. Do I win a prize?

@ecgreb Cool, just wanted to check my reasoning about expectations.
You may collect your high five next time you're in the office :)

Hey @hjanetzek can you look over this and merge if it seems good to you?

Looks good!
I wonder though why quint is the default? - It feels to start a bit too delayed for the move-on-tap.
A few additions that would be nice to have, if you want to continue with this :)

more ease types
A function to cancel all ongoing eases.
Have the pinch gesture use eases (on double tap)
chaining of eases for fly-to animations for zoom-out and back in while moving. (Or generally for longer running animations it should be split into a start-, going- and end-phase)

I experienced with vtm that one often wants to coordinate between position changes from interactions and from animations. So maybe InputHandler and easings could be combined into a CameraController at some point.

Cool, thanks!
I just picked quint because I like the feel of the continuous 2nd derivative, but we should reconsider once we have some feedback from the app. I'm open to more ease types too!
Users can stop an ease right now just by calling the setter again without easing; I had imagined that this would cover most needs but an explicit cancel could be convenient too.
I had forgotten about double-tap zooming! That would be a good candidate for easing, but I'm also not sure we want to keep that as built-in library behavior :\
I had actually been working on a fly-to animation, but I stashed it for later because it got a bit complicated and I wanted to get the other easing functions merged first.


Secondary Axis missing in Charts

For column or bar charts, often we end up having secondary axis to depict additional features. I tried various ways, but couldn't figure out a way to add Secondary-axis to either a column or bar charts. Is there any possible way or work-around to add this.
any help would be highly appreciated!

I also want to know how to add a secondary axis in python-pptx. If it's not yet implemented then please let us know the other way or can share his/her code .help would be highly appreciated!
@scanny please help us

This is not implemented yet, but you can add it to build\lib\pptx\chart\axis.py
I suggest just copying the a chart that already has a secondary axis and replacing the values.

Duplicate of #141.


Darwin's TFO impl was not support now.

TCP Fastopen impl for darwin is not support syscall with TCP_FASTOPEN_CLIENT.
Need to instead with CONNECTX.

if anyone need it, i may submit PR for this.

Instead of trying to fix this mess, maybe consider switching to a modern alternative instead:

https://github.com/database64128/shadowsocks-go
https://github.com/SagerNet/sing-box

Both of these use my https://github.com/database64128/tfo-go for TFO support on Linux, Windows, macOS and FreeBSD.


Node.js can become unkillable even with kill -9 on Linux 6.1.113 or 6.6.57

Describe the bug
A Node.js program can get stuck after running for a while.
Only reproduceable on Linux kernel 6.1.113 or 6.6.57 (or later versions of 6.1 or 6.6).
Steps To Reproduce

[!WARNING]
If you reproduce the bug, your computer cannot be shut down normally because the unkillable process prevents it.

In configuration.nix, set the kernel version to 6.1 or 6.6:
# boot.kernelPackages = pkgs.linuxPackages_6_1;
boot.kernelPackages = pkgs.linuxPackages_6_6;

Also, install the node package by adding it to either users.users.xxx.packages or environment.systemPackages.
Set the root nixpkgs to a rev no earlier than f142a76870a94e9917305f29b2311e0bf2a72a59 (if using Linux 6.6) or 0dffe83747699260fe128a8ccd3d58ed279acd59 (if using Linux 6.1) (they are the exact first bad commits that I found by bisecting nixpkgs). You can do this by running
sudo nix-channel --add https://github.com/NixOS/nixpkgs/archive/0dffe83747699260fe128a8ccd3d58ed279acd59.tar.gz nixos
sudo nixos-rebuild boot --upgrade

(or by simply upgrading to the latest nixos-24.05).
Reboot. Run uname --kernel-release and check that kernel version satisfies 6.1.113 <= version < 6.2 || 6.6.57 <= version < 6.7.
Run the following command (the Node.js version is 20.15.1, but I think it should be reproduceable with any Node.js 20.x):
node --input-type=module <<'JAVASCRIPT'
import { readdir, readFile } from "fs/promises";
let i = 0;
setInterval(async () => {
  console.log(i++);
  (await readdir("/proc")).forEach(pid => +pid > 0 && readFile(`/proc/${pid}/cmdline`, "utf8"));
}, 5000)
JAVASCRIPT

Wait for a while until it outputs around 100 numbers or more.
Hit Ctrl+C to try to stop it. If it cannot be killed this way, you reproduce the bug. It is unkillable even with kill -9 if you try that.
Expected behavior
The process gets killed by Ctrl+C.
Additional context
I first talked about this symptom in https://github.com/NixOS/nixpkgs/pull/351774#issuecomment-2442052855.
I thought this is an upstream bug of arRPC, so I opened OpenAsar/arrpc#120. There are some people commenting in that issue saying that they experience the same problem and that they all use NixOS. Therefore, I thought this may be a bug that only affects NixOS.
I tried to reproduce the bug on Arch Linux VM (installing the kernel via the package linux-lts), and it was not reproduced.
I then read the source code of arRPC and tried to find a short program that can reproduce this bug. The node command pasted above is the reproducing command that I found.
Notify maintainers

Metadata

system: "x86_64-linux"
host os: Linux 6.6.59, NixOS, 24.05 (Uakari), 24.05.6258.dba414932936
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.18.8
channels(root): "nixos-24.05, nixos-hardware, nixos-unstable"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Add a :+1: reaction to issues you find important.

Could you double check if it's possibly some instance of https://github.com/NixOS/nixpkgs/issues/353709 I understood from that issue that the causing code is from nodes and triggers a bug in the kernel (at least the lts<6.6.60), not sure about older than 6.6, but maybe it's worth a shot if it goes away on most recent 6.6ers

Tested with on a xanmod-6.6.60 with the provided example and that exists fine on my machine with nodejs v20.18.0, can you double check with a said 6.6.60?

Yes 6.6.60 fixes it.


Allow ordering by expression as well as string

https://github.com/wix/angular-tree-control/pull/112/files?w=1

Converting order-by to be two-way bind would be a breaking change. I opted to add an additional optional parameter order-by-expression that you can use to pass a predicate function.


DeprecationWarning on MongodB

Please update new connection script using new version of mongodb . Thank you

This is a bug in Mongoose and will be fixed in a new version, nothing to be done other than ignore the warning until new versions are available I think. Discussed in this thread here; https://github.com/Automattic/mongoose/issues/5304

module.exports = function(){
var config = require('../configurations/config');
var mongoose = require('mongoose');
//Mongo DB Database Connection
mongoose.connect(config.mongoDBconnect, { useMongoClient : true }  )
.catch(err => console.error(err));
};

Change line 7 in server/api_router/loginDBConnection.js
mongoose.connect(config.mongoDBconnect, function(err) {
to
mongoose.connect(config.mongoDBconnect, { useMongoClient: true }, function(err) {

I had the exact same issue and that suggestion worked


High level API for Client

The current Rpc monad does not capture RECV_STATUS_ON_CLIENT well.
Should maybe check for each Rpc operation whether the status has been received, and if so, throw exception to end the execution of Rpc.
Too easy to implement buggy behaviour, if the user doesn't expect an exception and wanted to do more operations?

I'm tempted to use exceptions instead of an Rpc monad. It'd be easier to write code that has two clients that need to interact.


When you reply to a github issue it's sent back as a webmention

Hello, when I tested brid.gy GitHub publish integration I got a strange webmention that I didn’t expected.
This is my original content https://abisso.org/stream/2020/10/19/tor-in-vagrant/ and it received 2 reply webmentions.
One is the legitimate reply from the issue author (shown as comment on my original post) and another one that is a reply pointing to my own comment (that I filtered out on the web page).
Is this behavior expected?
Thank you for this wonderful project :pray:
(Originally published at: https://l0g.in/59S74i)

sorry for the trouble, and thanks for filing! looks like this is a bug. your post's u-syndication correctly points to https://github.com/rootzoll/raspiblitz/issues/1671#issuecomment-712288548 , but we then lose the fragment at some point, so we think your post is for the original issue, and so every comment then turns into a webmention, including yours. will fix.
relevant snippet from the poll log where this posse post discovery happened:
processing permalink: https://abisso.org/stream/2020/10/19/tor-in-vagrant/
requests.head https://abisso.org/stream/2020/10/19/tor-in-vagrant/
u-syndication links on the h-feed h-entry: ['https://github.com/rootzoll/raspiblitz/issues/1671#issuecomment-712288548']
requests.head https://github.com/rootzoll/raspiblitz/issues/1671
saving discovered relationship https://github.com/rootzoll/raspiblitz/issues/1671 -> https://abisso.org/stream/2020/10/19/tor-in-vagrant/
discovered relationships {'https://github.com/rootzoll/raspiblitz/issues/1671': [SyndicatedPost(key=Key('GitHub', 'nolith', 'SyndicatedPost',<PHONE_NUMBER>573120), created=datetime.datetime(2020, 10, 19, 17, 53, 50, 853789), original='https://abisso.org/stream/2020/10/19/tor-in-vagrant/', syndication='https://github.com/rootzoll/raspiblitz/issues/1671', updated=datetime.datetime(2020, 10, 19, 17, 53, 50, 853795))]}

😆 😢
https://github.com/snarfed/bridgy/blob/b472fd7bf7469755f63d707d4538699a9752d57e/github.py#L27-L39

> I’m lazy, though, so I’m leaving this as is for now.
Thank you for your answer and the funny excerpt from the code comment 😆
May I ask you what’s the meaning of OPD?
(Originally published at: https://l0g.in/59SMNU)

My guess: OPD stands for original_post_discovery (as this is a common pattern in Python).

fixed, deploying now. thanks again for filing!


Configurable tmp folder location

Currently Stryker creates a new dot folder in the current working directory which is not in a default temp location https://github.com/stryker-mutator/stryker/blob/8e0e6b016b513a5fbd1f24fc7a08499ef3151dba/packages/stryker/src/utils/StrykerTempFolder.ts#L9, which means every user of stryker needs to edit their .gitignore file.
It would be nice if this location could be configured in the stryker.conf.js file, and by default live in ./tmp/stryker instead of .stryker-tmp

Thank you @viamin  for reporting this issue.
The position of your .stryker-tmp folder is indeed not ideal. We actually used your system's temp folder earlier, but we ran into a problem: require searches for node_modules relative to your current file. For example: const chai = require('chai'); won't work if you copy your files to your systems temp folder. This is why we ended up with this solution.
In other words: your systems temp folder will only work for browser-based test runners like karma.
If we would add a configuration item for this it will be one more way users can make mistakes in configuring Stryker. It's already kind of hard to do. Would you still want this option?

I was not referring to system /tmp, but the project root ./tmp, which would be in the same directory as node_modules. I'm proposing a project with these files:
/path/to/project_root/tmp/stryker/ (replacing /path/to/project_root/.stryker-tmp/)
/path/to/project_root/node_modules
/path/to/project_root/package.json
etc.

Ah i see what you mean. That should work indeed. Do you want to take a shot at implementing this? Would you only allow relative path locations? If we allow for the systems tmp folder for example, we should warn people about the node_modules.

I think this one can be implemented after https://github.com/stryker-mutator/stryker/pull/361 or with it.

Solution created in https://github.com/stryker-mutator/stryker/pull/475

We're closing this issue for now. If someone wants to fix this, let us know!


Outputting plan file to specific path fails

Describe the bug
The following commands fails to redirect the plan file
tg_command: plan -out {{ github.workspace }}/terragrunt.tfplan

To Reproduce
name: Deploy customer changes
on: push

env:
  TF_VERSION: 1.5.1
  TG_VERSION: 0.55.21

jobs:
  terragrunt-plan:
    runs-on: ubuntu-latest
    steps:
      - name: Checkout code
        uses: actions/checkout@v4

      - name: Plan
        uses: gruntwork-io/terragrunt-action@v2
        with:
          tf_version: ${{ env.TF_VERSION }}
          tg_version: ${{ env.TG_VERSION }}
          tg_dir: ${{ env.WORKING_DIR }}
          tg_command: plan -out {{ github.workspace }}/terragrunt.tfplan

Expected behavior
No changes. Your infrastructure matches the configuration.

Terraform has compared your real infrastructure against your configuration
and found no differences, so no changes are needed.

Actual behaviour
│ Error: Failed to write plan file
│
│ The plan file could not be written: open
│ /home/runner/work/<path>/terragrunt.tfplan:
│ no such file or directory.
╵

Other notes
The following command works
tg_command: plan -out $(pwd)terragrunt.tfplan

But I would expect one of these commands to work instead
tg_command: plan -out {{ github.workspace }}/terragrunt.tfplan
tg_command: plan -out "{{ github.workspace }}/terragrunt.tfplan"
tg_command: plan -out $(pwd)/terragrunt.tfplan

Nice to have

[X] Terminal output
[ ] Screenshots

Versions

Terragrunt Action version: terragrunt-action@v2
Environment details (Terragrunt version, Terraform version, etc.):

  TF_VERSION: 1.5.1
  TG_VERSION: 0.55.21

Related
Might be related to #40

Hi @Mornor, the issue is the forward slash, right?

Yes, it should work with a forward slash (so a full path) like so
tg_command: plan -out {{ github.workspace }}/terragrunt.tfplan
tg_command: plan -out "{{ github.workspace }}/terragrunt.tfplan"
tg_command: plan -out $(pwd)/terragrunt.tfplan

But it doesn't (Error: Failed to write plan file ...).
However, this works (without forward slash)
tg_command: plan -out $(pwd)terragrunt.tfplan

IMO, we should be able to give a full path for the plan file as output.

@Mornor, -out is a terraform flag, Terragrunt simply passes this flag to Terraform when running the terraform plan command. You should create this issue in Terraform issues.

The corresponding commands (terraform plan -out terraform.tfplan) works fine.
Locally issuing following command works as well, so this make me think the issue is related to the Terragrunt GH Action itself
$> rm -rf $(pwd)/terragrunt.tfplan
$> terragrunt plan -out $(pwd)/terragrunt.tfplan
No changes. Your infrastructure matches the configuration.

$> terragrunt show $(pwd)/terragrunt.tfplan
No changes. Your infrastructure matches the configuration.

However, it seems the terragrunt command itself doesn't work as expected. Locally, this does not produce any plan.
$> terragrunt plan -out terragrunt.tfplan

But this does
$> terragrunt plan -out $(pwd)/terragrunt.tfplan

However, not with Github actions (Error: Failed to write plan file [...]  terragrunt.tfplan: no such file or │ directory
This might be related to a core Terragrunt issue.

Hi,
AFAIK, *workspace variables in case of runners point to the wrong locations, use /github/workspace path to save/load plan files
https://github.com/actions/runner/issues/2058

Hi @denis256,
Big up to you, as I managed to make it work with a combination of your solution and some mv commands:
    - name: Plan
      uses: gruntwork-io/terragrunt-action@v2
      with:
        tf_version: ${{ inputs.tf_version }}
        tg_version: ${{ inputs.tg_version }}
        tg_dir: ${{ inputs.customer_dir }}/${{ inputs.customer }}/${{ inputs.dir_to_deploy }}
        tg_command: plan -out /github/workspace/terragrunt.tfplan

     - name: Move plan file
        working-directory: ${{ github.workspace }}
        run: mv terragrunt.tfplan ${{ inputs.customer_dir }}/${{ inputs.customer }}/${{ inputs.dir_to_deploy }}/terragrunt.tfplan

I don't really like or understand the combination of {{ github.workspace }} and /github/workspace/  (and I still think there's something not working properly under the hood), but at least it works!

Closing this issue as it is related to how the GitHub action sets up the workspace directory.


Synthesize a minimal landing page for combined archives

Bug/issue #, if applicable: rdar://79160385
Summary
This makes the docc merge command synthesize a minimal "landing page" for the combined documentation.
It also adds two new flags --synthesized-landing-page-name and --synthesized-landing-page-kind to customize small details about this page. The existence of the pair of these flags is indicted by a new entry in "features.json"
Dependencies
None
Testing

Build documentation for two targets with dependencies. For example: Inner.doccarchive (zip) and
Outer.doccarchive (zip).

Default name and kind

Merge the two archives into a combined archive using
swift docc merge /path/to/Inner.doccarchive /path/to/Outer.doccarchive

Preview the combined documentation. For example:

Run cd Combined.doccarchive followed by ruby -run -ehttpd . -p8080

Open a browser and navigate to localhost:8080/documentation/

The landing page should display "Documentation" for the title in the navigator and display name on the page
The landing page should display "Package" for the title heading.
The landing page should display links to both modules (in alphabetical order)

Custom name and kind

Remove the previous combined archive:
rm -rf Combined.doccarchive

Merge the two archives into a combined archive using, passing a custom name and/or kind
swift docc merge \
  /path/to/Inner.doccarchive /path/to/Outer.doccarchive \
  --synthesized-landing-page-name "My Custom Combined Title" \
  --synthesized-landing-page-kind "My Custom Landing Page Kind"

(unchanged) Preview the combined documentation. For example:

Run cd Combined.doccarchive followed by ruby -run -ehttpd . -p8080

Open a browser and navigate to localhost:8080/documentation/

The landing page should display your custom landing page title for the title in the navigator and display name on the page
The landing page should display your custom landing page kind for the title heading.
(unchanged) The landing page should display links to both modules (in alphabetical order)

Navigate the combined archive

While still previewing the combined documentation:

Click a few elements in the navigator hierarchy:

You should navigate to that page
The navigator hierarchy should continue to display the landing page and both modules

Click the landing page title at the top of the navigator hierarchy:

You should navigate to the landing page
(unchanged) The navigator hierarchy should continue to display the landing page and both modules

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran the ./bin/test script and it succeeded
~[ ] Updated documentation if necessary~ This feature is still experimental and may change

@swift-ci please test

Is there a reason why we are displaying the API collections in a grid format? since this page is synthesised the author will not be able to customise the header image on each tile. Could we consider using a list instead?

Trying to merge two tutorials will raise an error because both contains a table of contents. Tutorial.doccarchive' and 'Tutorial2.doccarchive' both contain '/data/tutorials/table-of-contents/
A possibility to mitigate this is to disambiguate the names if two urls from the merged archives collide.

Trying to merge two tutorials will raise an error because both contains a table of contents. Tutorial.doccarchive' and 'Tutorial2.doccarchive' both contain '/data/tutorials/table-of-contents/
A possibility to mitigate this is to disambiguate the names if two urls from the merged archives collide.

That is the expected behavior. It's the developer's responsibility to not name the top-level content they merge the same.

@swift-ci please test

Is there a reason why we are displaying the API collections in a grid format? since this page is synthesised the author will not be able to customise the header image on each tile. Could we consider using a list instead?

Yes, the page looks rather sparse as a list.
The developer authors each modules page image in the markup for that module extension file so they are able to customize that image.

A good enhancement that I think it should be part of the PR is giving the developer the possibility to customise the topic section style via a CLI option, instead of using the detailed grid as a fixed value.

I added a new flag for this in https://github.com/swiftlang/swift-docc/pull/1011/commits/5be3a6657000b321a87d77bd60158d423a97dd1f

@swift-ci please test


set nullglob when looking for PORT files

When deploying an app without a web proc and $NO_VHOST set, dokku will not generate a file that matches this glob and thus will yield error output as indicated in #1678.
It's also probably good code hygiene to guard nullglobs like this anyway. We do it almost everywhere else except for backup-export/import triggers but we know we are going to sort those out soon anyway.
NOTE: This is not a fatal error and does not cause dokku to fail deployment.
closes #1678

Whats the original bug?

@josegonzalez updated the PR description

Got it, seems legit :)


Some operations after DataFrame.sample do not work.

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): MacOS
Modin installed from (source or binary): binary
Modin version: 0.7.0
Python version: 3.7.6
Exact command to reproduce:

import string
import numpy as np
import modin.pandas as mpd

chars = [i+j+k for i in string.ascii_letters for j in string.ascii_letters for k in string.ascii_letters]

mdf_chars = mpd.DataFrame({'Random_Char': chars, "Random_Int": np.random.randint(0, len(chars)), "Random_Float": np.random.random(len(chars))})

mdf_chars.sample(100000).reset_index()
mdf_chars.sample(10000).describe()

Describe the problem
Operations like reset_index and describe after the sample method may run forever or crash with below error message.
Source code / logs
UserWarning: Distributing <class 'dict'> object. This may take some time.
F0220 22:58:56.524958 230891520 direct_task_transport.cc:154] Lost connection with local raylet.
We have been seeing a number of these types of failures with Ray lately. Does the issue persist if you use the Dask engine? You may have to run pip install modin[dask]. Our Dask implementation is less mature than the Ray engine, but I haven't been seeing these crashes lately.
I have created an issue in Ray ray-project/ray#7263 to track this. Thanks again for reporting!

@devin-petersohn I'm not sure whether I'm facing an error of the similar nature (maybe here I need to open a separate issue if you deem the root cause different):
I'm able to process a complete parquet file using my current script in some 6 minutes using modin. The parquet file fits in mem, and all the operations I do follow reading the parquet files don't take more than a couple of minutes (e.g. set_index, merge, etc.)
Whenever I summon df.sample(), the workers start dying out-of-memory and my process gets killed soon after. I'm aware that this is Dask error, and I find this issue to look fairly similar to mine.
I just wanted to double-check here since sample() is showing similar weirdness when called in the script whether it's related in any way to this issue and whether we have a work-around insight to proceed.
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux (Remote Compute)
Modin version: 0.7.0
Python version: 3.7

@jadhosn Let's open a new issue and continue discussion there. This issue was more specific to Ray limiting the amount of code that can be executed in a task.

This was fixed in Ray, and I just tested on the latest release with no issue. Closing as resolved, please reopen if the issue was not fixed for you!


removes container service role from policy

Removes the AmazonEC2ContainerServiceforEC2Role role from the requirement scripts since this is not required by the integration.

Integration reporting without this permissions:
https://onenr.io/0BQrKdrApwZ
https://onenr.io/0qwyD4n5eQn

Yes thanks for asking, i check for those but nothing there.


Plugin Assets and Media Folder are not copied

The Assets and media Folders(https://getkirby.com/docs/guide/plugins/custom-plugins#plugin-assets) inside of Plugins appear to be missing in build.
Expected Behavior
the Assets in the assets Folder of a plugin should be available at /media/plugins/$PLUGIN_NAME/**
So, basically Copy the Assets from The Plugin into the folder under /media/plugins/$PLUGIN_NAME
Current Behavior
The Assets are not Available in the Build Folder
Possible Solution
For now, copy the assets' folder manual to the build output. For Fixing scaning the Plugin Folder For assets Folders and copy them to the correct Path
Steps to Reproduce
Create a test plugin, create an assets' folder, put in a File and build the Site. Check build folder for the Plugin under /media/plugins
Here is a Repo that should showcase what I mean: https://github.com/dustsucker/PluginAssetsNotInBuild
Context
Kirby Version: 4.1.2
OS: Linux(Nixos)(Should not mater here)
Plugin Instalation: Git submodule / Manual
Plugin Usage: inside a generate.php file with these basic comands and some customRoutes:
$staticSiteGenerator = new D4L\StaticSiteGenerator($kirby);
$staticSiteGenerator->skipMedia(false);
$staticSiteGenerator->setCustomRoutes($customRoutes);
$files = $staticSiteGenerator->generate("./build");
Thank you for your good Work.

Hello, thank you for reporting this! I wasn't aware that plugins could expose their own assets, this was a quick one to add. Let me know if it works as expected and have a great weekend!


Is the training code released?

Seems like only the evaluation code is released in this repo?

Hi,
For the mask proposal module training code/instructions, please refer to Mask3D codebase.
Best,
Ayca


how to use dbscan.c for 2d  data points

is there any way?

tanks i find sol in your doc
we are using three-dimensional data
points; however, we can also use any n-dimensional data points
as long as a distance function can be defined between any
two points in D.


Unable to use DNS on local network for some reason

During install, I made sure to choose the option for use DNS in /etc/resolv.conf
My ubuntu installation is set to use a 192.168 pi hole DNS that I have setup.
I can connect using the openvpn PC client fine, but the second I try to go to a website, it will fail with a DNS error. Anyone know why?

Can you try doing the same as described in #142?

Also in server.conf for dns put the pi-hole ip directly.

Thank you! Your second suggestion worked a treat :)
Any idea if this behavior or the option to choose your own DNS during setup can be made in the future? Thanks again.

For sure in the future we will have a wiki explaining this type of setup.


ETH Lisbon Safe Apps SDK

Background:
We have a hackathon coming up and unfortunately, we are a little light on guides for the Safe Apps SDK. Fortunately, the documentation is excellent.
Task:
Please write a guide which shows a developer how to:

integrate with the Safe Apps SDK
Batch 2 or more transactions into one
Propose the Transaction

I managed to follow the guide and actually get it working. So if I can do it, then it's good to go. https://docs.gnosis-safe.io/tutorials/build-a-safe-app-that-allows-transferring-assets-out-of-the-safe-in-1-transaction


Ability to specify service name

Now we specify naming of services in my dict by its name in mesos. But when we have two services in different groups that names ident we can't discover both of them.
So we need to specify it like this:
services: [
 {
        "myname": "memcached_sessions"
        "name": "memcached",
        "group": "sessions.production"
},
{
        "myname": "memcached_cache"
        "name": "memcached",
        "group": "cache.production"
}
]

I don't like myname. It is bad keyword :)
How about nickname?

myname is just example
nickname is always about a person. We should use other name.


Add from_numpy method to TH2

Adds ability to write 2D histograms created using np.histogram2d.
Changes:

from_numpy() method added to classes.TH2
Necessary additions to convert.towriteable() for directly writing to file

f = uproot.create("file.root")
data = np.random.normal(0, 1, (10000, 2))
f['hist']= np.histogram2d(data[:,0], data[:,1], bins=(50, 50), range=((-5, 5), (-5, 5)))
f.close()

Thanks, @rob-tay!
@reikdas, does this work with your developments in file-writing? (You added a superclass TH for multi-dimensional histograms—is anything in this PR redundant with that or needs to be changed to work with that? Or is it consistent and just adds to what you've done?)

I added a comment about a possible redundant check. Apart from that looks great! :)

Maybe the explicit TH2 check is redundant after the TH check, and if so, it should probably be removed.
Does this require a particular combination of uproot and uproot-methods versions to function properly? That is, will any combinations that are allowed by the version constraints in setup.py cause mysterious exceptions, rather than just NotImplementedError if you're using a version that doesn't have the TH2 handling yet?
Do we need to synchronize new version constraints between uproot and uproot-methods to get this feature? (Hopefully not!)

Does this require a particular combination of uproot and uproot-methods versions to function properly? That is, will any combinations that are allowed by the version constraints in setup.py cause mysterious exceptions, rather than just NotImplementedError if you're using a version that doesn't have the TH2 handling yet?

Hmm, yeah this will require uproot 3.5.1 which adds the uproot.write.objects.TH.TH class. If using a previous version then I think this will result in a ModuleNotFoundError when importlib is used to import the TH class

Okay, then for safety, I'll coordinate the inclusion of this PR with the next semi-major release (uproot 3.6).
I'm also revamping the lazy-array implementation—replacing uproot's custom LazyArray class with one constructed from awkward's ChunkedArray of VirtualArrays. The lazy-array making methods will have slightly altered parameter lists, which suggests a new semi-major release. All of this will have to go in before an uproot HATS next week.


[RFE] Rancher AWS Node Driver -> IAM Role Enhancement Request

Is your feature request related to a problem? Please describe.
Currently, when setting up Ranchers integration w/AWS, there are external requirements that have to be done in the AWS Platform to allow Rancher to do what it requires. We would like to see Rancher take over all of these external requirements, primarily IAM roles.
Describe the solution you'd like
As a Rancher AWS user, I want Ranchers AWS Node Driver to create, manage and administrate IAM roles so that Rancher can do the full management of the Infrastructure.
Requirements:
Must haves:

Rancher would accept a full admin Access Key & Secret Key to AWS Commercial/GovCloud/C2S/SC2S or an Access Key & Secret Key specially provisioned to create new IAM roles to allow minimal attack surface area.
Rancher creates required IAM roles for all Rancher and AWS resources and applies to them using ARN format the Service Accounts providing only the exact permission the service account requires.
Provisioning can only be done by Cluster-Admins

Nice to haves:

Direct integration with EKS pod identity webook
Rancher should dynamically build IAM roles that are more granular to the resources instead of full wide open permissions
** EG: IAM roles should be "instance-id::ec2::aws-region" vs "*::ec2::aws-region"

This is still valid, please reopen.
SURE-7249

not stale


Support for Swift 3.0

Hi,
Is this library going to update to support Swift 3.0?
Thank you.

yes they're busy with it
#455

Duplicate of #455


Instructions in ReadMe seem to be incorrect

Hi there,
It seems the instructions in readme are incorrect, particularly around the symlink, and ensuring it is in onboot.d .
Unless I am doing it completely wrong, the symlinking is not linking where it should be for the booting, and even the initial run.
Thanks for looking into this!

You absolutely correct :-) Thank you for pointing it out. I've fixed the readme.


Modify queryRequest to support custom headers

We recently added OTP based flow as an alternative for scheduling appointments, however this requires setting token in the headers (currently done through fetch request rather than queryRequest).
queryRequest (src/Utils/request/query.ts) currently doesn't support custom headers; add logic to do so.

Hi @Jacobjeevan , Kindly assign me this issue so that I can start working on it. Thanks


drivers: flash: flash_stm32: usage fault

Hi,
I found a bug in the flash driver of the stm32. In most cases or
configurations, the bug won't show up. It's an issue of flash_stm32.c in combination with
its device-tree configuration. stm32f3.dtsi states:
[...]
                flash-controller@40022000 {
                        compatible = "st,stm32f3-flash-controller";
                        label = "FLASH_CTRL";
                        reg = <0x40022000 0x400>;
                        interrupts = <4 0>;

                        #address-cells = <1>;
                        #size-cells = <1>;

                        flash0: flash@8000000 {
                                compatible = "soc-nv-flash";
                                label = "FLASH_STM32";

                                write-block-size = <2>;
                                erase-block-size = <2048>;
                        };
                };
[...]

Later, the control registers are loaded in flash_stm32.c:256,
flash_stm32_write_protection() tries to load the base address of the
control registers:
#define DT_DRV_COMPAT soc_nv_flash
[...]

static struct flash_stm32_priv flash_data = {
        .regs = (FLASH_TypeDef *) DT_INST_REG_ADDR(0),

FLASH_TypeDef *regs = FLASH_STM32_REGS(dev);

regs will point to 0x8000000, which is surely not the base address of
the control registers. This is typically the vector table base address,
and in most cases, by luck, only content of registers is read, as
further checks of the registers evaluate 'gratefully'. However, I found
this bug as the content of my vector table triggers the issues that
happens later in flash_stm32.c:272:
if (regs->CR & FLASH_CR_LOCK) {
        regs->KEYR = FLASH_KEY1;  /* Bad memory access */
        regs->KEYR = FLASH_KEY2;
}

The condition is true, and regs->KEYR writes, trying to access the vector
table and triggers a fault.
Big thanks to @rralf for helping debugging this issue.
CC:
@galak
@superna9999

@rckTom, does PR #24059 fix these faults?

Yes, PR #24059 fixes the fault.

Tom, you then might want to add your Tested-by to that PR. At least we had a really funny remote-ssh-screen debugging session. ;-)


S-Group: Ketcher interaction with non-listed Functional Groups

Is your feature request related to a problem? Please describe.
If the S-Group is not in the FG Ketcher list, then the structure is added with the expanded S-Group. Also the user cannot contract or remove abbreviation for editing.
https://user-images.githubusercontent.com/62464241/209813954-0148fa93-03db-45c8-b39d-3c9bc7cda35d.mp4
Describe the solution you'd like
Add to the Ketcher support for interacting with S-Groups that are not in the Ketcher's list. User should can contract, expand or remove abbreviation.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Steps for Reproduce

Launch Ketcher
Open browser console
Paste in

await ketcher.setMolecule(`
  Ketcher<PHONE_NUMBER>D 1   1.00000     0.00000     0

 11 11  0     0  0            999 V2000
    5.3803   -1.2588    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
    5.3803   -2.2588    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
    6.2463   -2.7587    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
    7.1124   -2.2588    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
    7.1124   -1.2588    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
    6.2463   -0.7588    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
    7.9783   -0.7588    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
    8.8443   -1.2588    0.0000 C   0  0  0  0  0  0  0  0  0  0  0  0
    8.3444   -2.1248    0.0000 H   0  0  0  0  0  0  0  0  0  0  0  0
    9.7104   -1.7588    0.0000 H   0  0  0  0  0  0  0  0  0  0  0  0
    9.3443   -0.3927    0.0000 H   0  0  0  0  0  0  0  0  0  0  0  0
  8  9  1  0     0  0
  8 10  1  0     0  0
  8 11  1  0     0  0
  7  8  1  0     0  0
  1  2  2  0     0  0
  2  3  1  0     0  0
  3  4  2  0     0  0
  4  5  1  0     0  0
  5  6  2  0     0  0
  6  1  1  0     0  0
  5  7  1  0     0  0
M  ISO  3   9   2  10   2  11   2
M  STY  1   1 SUP
M  SLB  1   1   1
M  SAL   1  4   8   9  10  11
M  SBL   1  1   4
M  SMT   1 CD3
M  END
`)

Try contract S-Group or Remove abbreviation

Duplicate of https://github.com/epam/ketcher/issues/2052


why is 'next' in createApplication needed?

function createApplication() {
  var app = function(req, res, next) {
    app.handle(req, res, next);
  };

I deleted it in my local env,it turns out fine.
And It seems no way to assaign it , how to use this 'next' here?

Hi @ginobilee it is used when you mount apps on each other. For example:
var app  = express()
var app1 = express()

app.use(app1)

Thank you !
I just begin learning express.I know it too little to ask this question,almost regret just after i post it .
Thank you again!

It's not a problem at all, @ginobilee

any example？


Weirdness when mocking out already existing directory

it('should foo', function () {
    return expect(function () {
        console.log(fs.readdirSync('/tmp'));
    }, 'with fs mocked out', {
        '/tmp': {}
    }, 'not to error');
});

The output is a list of all the crap in my /tmp:
[ '.ICE-unix',
  '.X0-lock',
  '.X11-unix',
  ...
  'unity_support_test.0' ]

Oddly, if I add a trailing slash to the fs.readdirSync call I get a different result:
it('should foo', function () {
    return expect(function () {
        console.log(fs.readdirSync('/tmp/'));
    }, 'with fs mocked out', {
        '/tmp': {}
    }, 'not to error');
});

Output:
[ 'home', 'tmp' ]

I realize this is probably one or more bugs in mountfs, but I still wanted to report it.

Seems to be a general problem with readdir.
it('/tmp', function () {
    return expect(function () {
        expect(fs.readFileSync('/tmp/foo', 'utf-8'), 'to equal', 'bar');
        expect(fs.readdirSync('/tmp'), 'to equal', [ 'foo' ]);
    }, 'with fs mocked out', {
        '/tmp': {
            'foo': 'bar'
        }
    }, 'not to error');
});

     UnexpectedError:
expected
function () {
    expect(fs.readFileSync('/tmp/foo', 'utf-8'), 'to equal', 'bar');
    expect(fs.readdirSync('/tmp'), 'to equal', [ 'foo' ]);
}
not to error
  threw: expected
  [
    '.ICE-unix',
    'tmpjl3efa',
    'tmpu2vlbh',
    'tmpw_mfh6',
    'tmpy79kjn'
  ]
  to equal [ 'foo' ]

  [
    '.ICE-unix', // should equal 'foo'
                 // -.ICE-unix
                 // +foo
    'tmpjl3efa', // should be removed
    'tmpu2vlbh', // should be removed
    'tmpw_mfh6', // should be removed
    'tmpy79kjn' // should be removed
  ]
      at Context.<anonymous> (test/index.spec.js:153:24)

I think that I might have destroyed readdir sync when trying to get https://github.com/unexpectedjs/unexpected-fs/blob/master/test/index.spec.js#L96 working.
We might want to make them override the existing folders completely instead of attempting to augment them?

Oh, hmm, good question.
I'm also leaning towards overriding existing folders rather than "layering" over them, but I'm assuming that you had a use case for it?

https://github.com/unexpectedjs/unexpected-fs/tree/issue10

I landed the work we did yesterday, that fixed most of the oddities with readdir. The home and tmp folders come from this piece of code in mock-fs:
https://github.com/tschaub/mock-fs/blob/6dde8fb8709c4d97212387174d7ce97faaf8a907/lib/filesystem.js#L39-L56
I've opened an issue trying to explain our use case and ask him to make it configurable.


(maint) Add issue and PR auto responses

Description
Add auto response messages to issues and PR's that are submitted. This will help provide an understanding of what people should expect.
Motivation and Context
There are a number of PR's being submitted that are wrong, and issues being raised where people expect the community team to do all the work. I hope this helps to address some of this (it won't fix it but every little bit helps).
How Has this Been Tested?
This is currently working over on my GitHub repository.
Screenshot (if appropriate, usually isn't needed):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Migrated package (a package has been migrated from another repository)

Checklist:

[x] My code follows the code style of this repository.
[ ] My change requires a change to documentation (this usually means the notes in the description of a package).
[ ] I have updated the documentation accordingly (this usually means the notes in the description of a package).
[ ] I have updated the package description and it is less than 4000 characters.
[ ] All files are up to date with the latest Contributing Guidelines
[ ] The added/modified package passed install/uninstall in the Chocolatey Test Environment(https://github.com/chocolatey-community/chocolatey-test-environment/). Note that we don't support the use of any other environment.
[ ] The changes only affect a single package (not including meta package).

:white_check_mark: Package verification completed without issues. PR is now pending human review

@AdmiringWorm this is ready for review again. Thanks for the suggestions - 'created' is a much better option!

:white_check_mark: Package verification completed without issues. PR is now pending human review

@pauby thanks for getting these workflows added. Should help with the incoming issues and PRs.

It looks like I misremembered the available types for issues, and created is only for PRs.
I'll get that updated.


Should setting an image to empty string remove it?

Just a simple question: Should setting an image to empty string remove it? Or can it only be removed using Carrierwave's remove_xxx field?

it can be only removed by using remove_xxx, there's a present? check for the new value in the adapter.rb

Thanks!


  I'd assume the seed file comes from the neuralhash_128x96_seed1.dat file, but I'm really not sure how you converted it to get to that point.  Same for the model, I'd assume that's from the .net, .shape, and .weights file but I have no idea how.
I'm really new to ML space so I'm not super familiar with the tools, but I'm very curious! I was never able to get the ONNX model that the original repo shows how to make to work on my machine, I believe because of a hardware incompatibility, but I was able to get the tensorflow model you have to work, so I was thinking it would be ideal to be able to convert that in the future in order to keep up to date if Apple changes their model or seed.

Hey @csandman, thanks for your interest in this repo!
It's been awhile so I might be wrong but here's what I remember:
For seed.js, like you mention you'd first need neuralhash_128x96_seed1.dat.
Then you should run something like
import numpy as np

seed1 = open('neuralhash_128x96_seed1.dat', 'rb').read()[128:]
seed1 = np.frombuffer(seed1, dtype=np.float32)
seed1 = seed1.reshape([96, 128])
# seed1 will be the array that you see in seed.js

As for the model - you first need the ONNX model, then I believe I opened it in Torch and re-exported it again into tensorflow-js compatible formats.

Thanks a bunch for responding!  So for the seed, I see that you basically just saved the output of that part of the original script as a JS/JSON object and used that in your own script, that makes a lot of sense haha.  I'm curious though, I see the reshape part of the script is run both in the original script and in your script, is that correct?  Or did you save the output of loading the seed before you ran reshape in the original script?
This might also be due to a lack of understanding on my part, I'm not entirely sure what the reshape part of the script is doing.


Manager Users

Context

Add helper methods to retrieve invitation type and invitation status

Codecov Report
Base: 53.94% // Head: 53.97% // Increases project coverage by +0.03% :tada:

Coverage data is based on head (20570a4) compared to base (07ee629).
Patch coverage: 65.38% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #339      +/-   ##
===========================================
+ Coverage    53.94%   53.97%   +0.03%
===========================================
  Files          205      205
  Lines        11657    11674      +17
  Branches      1506     1513       +7
===========================================
+ Hits          6288     6301      +13
- Misses        5369     5373       +4

Impacted Files
Coverage Δ

trade_remedies_api/invitations/models.py
59.50% <ø> (+0.82%)
:arrow_up:

...rade_remedies_api/cases/services/v2/serializers.py
73.26% <50.00%> (ø)

trade_remedies_api/config/viewsets.py
77.14% <60.00%> (-2.86%)
:arrow_down:

...emedies_api/invitations/services/v2/serializers.py
88.23% <68.42%> (-11.77%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Migration fails on startup

Using bonukai/mediatracker:latest the migration on startup fails and the app doesn't start. There seems to be something wrong with my database, any idea what is wrong or how I can resolve it?
Logs that keep repeating:
mediatracker  | info:  Server timezone: Europe/Amsterdam
mediatracker  | info:  Server time: 7/9/2024, 3:20:31 PM
mediatracker  | info:  MediaTracker 0.2.3 listening at http://d771386f6374:7481
mediatracker  | info:  Updating metadata for 283 items
mediatracker  | info:  Updating: The Walking Dead (last updated at: 8/25/2023, 11:18:20 PM)
mediatracker  | error: error: update "episode" set "releaseDate" = $1, "description" = $2, "episodeNumber" = $3, "id" = $4, "seasonId" = $5, "seasonNumber" = $6, "title" = $7, "tmdbId" = $8, "tvShowId" = $9, "isSpecialEpisode" = $10, "seasonAndEpisodeNumber" = $11 where "id" = $12 - duplicate key value violates unique constraint "episode_tmdbid_unique"
mediatracker  | info:  Updating: The Haunting of Bly Manor (last updated at: 4/15/2024, 1:55:00 PM)
mediatracker  | error: error: update "mediaItem" set "genres" = $1, "id" = $2, "imdbId" = $3, "lastTimeUpdated" = $4, "mediaType" = $5, "network" = $6, "numberOfSeasons" = $7, "originalTitle" = $8, "overview" = $9, "releaseDate" = $10, "tmdbRating" = $11, "source" = $12, "status" = $13, "title" = $14, "tmdbId" = $15, "url" = $16, "needsDetails" = $17, "tvdbId" = $18, "externalPosterUrl" = $19, "externalBackdropUrl" = $20, "posterId" = $21, "backdropId" = $22, "platform" = $23 where "id" = $24 - duplicate key value violates unique constraint "mediaitem_tvdbid_unique"
mediatracker  | info:  Updating: Star Trek: Strange New Worlds (last updated at: 4/13/2024, 2:55:00 PM)
mediatracker  | error: error: update "episode" set "releaseDate" = $1, "description" = $2, "episodeNumber" = $3, "id" = $4, "seasonId" = $5, "seasonNumber" = $6, "title" = $7, "tmdbId" = $8, "tvShowId" = $9, "isSpecialEpisode" = $10, "seasonAndEpisodeNumber" = $11 where "id" = $12 - duplicate key value violates unique constraint "episode_tmdbid_unique"
mediatracker  | info:  Updating: Suits (last updated at: 5/31/2023, 8:35:00 PM)
mediatracker  | error: error: update "episode" set "releaseDate" = $1, "description" = $2, "episodeNumber" = $3, "id" = $4, "seasonId" = $5, "seasonNumber" = $6, "title" = $7, "tmdbId" = $8, "tvShowId" = $9, "isSpecialEpisode" = $10, "seasonAndEpisodeNumber" = $11 where "id" = $12 - duplicate key value violates unique constraint "episode_tmdbid_unique"
mediatracker  | info:  Updated 0 items
mediatracker  | error: Failed to update 4 items

Fixed itself after a while, probably fkxed on tmbd side?


Fixed alignment of MeshMap access popup

Description:
Fixed alignment/positioning issues in the MeshMap access popup card.
Signed commits

[X] Yes, I signed my commits.

Current:

Updated:

CITY
😁


Please tag releases

Given that there are already 1-2 projects using this library it would help packagers to have releases.

I second that, obs-studio is now released and contrary to bundling rnnoise like e.g. mumble does, it requires the library being present: https://github.com/obsproject/obs-studio/blob/master/cmake/Modules/FindLibrnnoise.cmake
Which makes it hard to guess under which version we should actually package rnnoise.

Having release tarballs and tags would also make it easier to package rnnoise in Distributions.  I've asked for a Debian package in https://bugs.debian.org/980839 and did not know what to put in the version entry.  Just a minor issue but exemplify the use of a version number.

Having release tarballs and tags would also make it easier to package rnnoise in Distributions.  I've asked for a Debian package in https://bugs.debian.org/980839 and did not know what to put in the version entry.  Just a minor issue but exemplify the use of a version number.

I am also interested in this for packaging in Void Linux (void-linux/void-packages#31203)

I'm packaging this for Gentoo, since it's a new dependency of telegram-desktop. Please tag suitable releases, as that'd help a ton.

 #

Same, I'd like to package it for alpine and it's not tagged. It's a problem.

While I can tag a release, I'm also looking for a maintainer (or co-maintainer) for help.


"node" is called "nodejs" on ubuntu, hence the build fails

steps
Compile sbt on a plain Ubuntu box with nodejs installed. Compilation fails at:
[sbt212] [info] > node ./node_modules/vscode/bin/install
[sbt212] [error] sh: 1: node: not found

problem
"Because of a conflict with another package, the executable from the Ubuntu repositories is called nodejs instead of node . Keep this in mind as you are running software."
expectation
sbt compiles.
notes
The offending line is in vscode-sbt-scala/client/package.json.
The code should either work with nodejs as well (for instance by calling a stub script), or alternatively the documentation should direct Ubuntu/Debian users who want to compile sbt to install the additional package nodejs-legacy, which provides the necessary symlink.
sbt version: current 1.1.x

The Scala community build just uses this test, which could be integrated here:
if hash nodejs 2>/dev/null; then
    export NODE=nodejs
else
    export NODE=node
fi


Remove unused injected controller

Readiness checklist

[ ] I added/updated tests.
[x] I ensured that the PR title is good enough for the changelog.
[x] I labeled the PR.
[x] I self-reviewed the PR.

Hmmmmm, why is this unused??

It was added in #212 . Maybe @britsta used in for some kind of validation and then moved it elsewhere and forgot to remove it. 🤔 And we didn't see it in our review. I am also surprised that we don't get some kind of error.

There's an error popping up?

There's an error popping up?

Where?

There's an error popping up?

Where?

You wrote I am also surprised that we don't get some kind of error, so you started to talk about an error 😅

Yes, but I said there is no error popping up. And then you asked There is an error popping up?. 😅
I wish there was an error popping up. Do you know why there isn't?

Oh, lol. Yeah eslint doesn't tracks these kind of unused vars.


Merging Develop into Master 9.19.2019 - Design Library updates

Description of the change:
Reason for the change:
Link to original source:

Closes #

Hello @Whatthefoxsays,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


Support for Kraft mode

First of all, wanted to say big THANKS for this initiative.
This project is easy to use, easy to understand.
I am using this image, compared with regular kafka, and observed many advantages.
Just wanted to reach out asking for an enhancement request.
Kafka supports Kraft mode (zookeeper-less)
While this project supports native, it seems not to support Kaft.
Would it be possible for this project to support Kraft mode on top of native?
Thank you

Would it be possible for this project to support Kraft mode on top of native?

It does support KRafft already, that's it is default behaviour.
See: https://github.com/ozangunalp/kafka-native?tab=readme-ov-file#configuring-the-kafka-broker

THANKS!


fbp-generator: accept strings like "INT32_MIN" for int/float default values v2

Differences from v1:
- remove useless loop.
This should solve the DBL_MIN being platform dependent.
Signed-off-by: Ricardo de Almeida Gonzaga<EMAIL_ADDRESS>

@barbieri ?

looks good, if it works then merge it.

merged, thanks!


Crux doesn't show any logs if verbosity is set above 3

Crux doesn't show any log if verbosity is set above 3.
What I would expect is to have the maximum logging facility in such case.

I bumped into this issue independently. I submitted a PR with somewhat different behavior, also added to the help text
https://github.com/blk-io/crux/pull/55
Not sure which behavior they'd prefer (they seem to not be using all the logging levels), but I agree this definitely needs fixing.


Slot filling option for modules (flow)

Something that I have really dream about in the night between the 22nd to the 23rd of February.
Feature Use Case
A good example could be the game of "guess the number".

User: Leon, let's play the guess the number game
Leon: Alright, I've chosen a number, you guess.

Feature Proposal
Be able to have a dedicated "flow" for each module. As long as the user does not say "stop" or the module does not exit via a trigger, then the flow continue for this specific module.
Dedicated "flow" for each module. Module with an option flow true or false. When true, do not exit the current flow until the user say "stop" or the module exit via a trigger.

 ✨ [1.0.0-beta.3] Dedicated modules flow


Hooks.WallpaperSetter: Preserve aspect ratio while scaling images

Description
Preserve the aspect ratio of wallpapers that xmonad sets. When previous versions would distort images to fit the screen size, it will now find a best fit by cropping instead.
I don't understand the imagemagick command invocation very well, so this was put together through trial and error. I've tested it with good results though, in a dual-head setup where each monitor has a different aspect ratio (1920x1080 and 1920x1200).
Checklist

[X] I've read CONTRIBUTING.md

[X] I tested my changes with xmonad-testing

[X] I updated the CHANGES.md file

Thanks!


Battery percentage always 100% on Kobo Clara BW

First time using this so I'm not sure if this is specific to the Kobo Clara BW. I've installed this mod and I enabled the battery indicator but it always says 100%. This is with and without any usb cable connected. The clock works fine though.
Here is my config:
[General]
Margin=Auto

[Battery]
BatteryType=Level
Enabled=true
LevelTemplate=%1%
Placement=Footer
Position=Right

[Clock]
Enabled=true
Placement=Footer
Position=Left

Hi, thanks for the report. I believe Kobo are using different power management hardware from what I've previously known about. A Libra Colour user has confirmed there's changes.
I'll look into fixing this later today or tomorrow. I may have an alternative method that is more hardware agnostic.

I've pushed a potential fix. Can you please try the following version?
https://github.com/shermp/NickelClock/actions/runs/8936925578/artifacts/1470101681


rearrange de branches to sate clippy

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (c237ce1) 80.96% compared to head (5bd87d9) 81.20%.

Files
Patch %
Lines

src/de.rs
91.66%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #53      +/-   ##
==========================================
+ Coverage   80.96%   81.20%   +0.24%
==========================================
  Files           3        3
  Lines        1077     1075       -2
  Branches     1077     1075       -2
==========================================
+ Hits          872      873       +1
+ Misses        144      142       -2
+ Partials       61       60       -1

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


Fixed outdated copyrights and org name

Description

Fixed copyright lines in header to say SpotHero, Inc. instead of SpotHero, Parking Panda, or anything else.
Updated all projects to use the org name SpotHero, Inc..

Danger has errored
[!] Invalid Dangerfile file: Not a directory @ rb_sysopen - .zinc/Dangerfile/Dangerfile. Updating the Danger gem might fix the issue. Your Danger version: 6.0.7, latest Danger version: 6.0.9
 #  from Dangerfile:3
 #  -------------------------------------------
 #
 >  danger.import_dangerfile(path: ".zinc/Dangerfile")
 #  -------------------------------------------

  Generated by :no_entry_sign: Danger


golang load .dll file, get "failed to load xxx.dll: the specified module could not be found" error

I am familiar with C, C++, PYTHON.
I started to learn golang about one month ago.
I also started to learn rust in the same time.
Right now, I am wondering which language should I continue to learn.
So I am trying to rewriting my CAN tool with golang.
Here is the code:
canDll, err := syscall.LoadDLL("./gylinker-x64-dll/VCI_CAN.dll")
if err != nil {
log.Fatal("Error during loading .dll file,", err)
}
defer canDll.Release()
proc, err := canDll.FindProc("VCI_InitCAN")
if err != nil {
	log.Fatal("Error during finding procedure,", err)
}

proc.Call() // don't know how to pass parameter yet

I replaced the .dll file with another one from a big company. The error is gone.
canDll, err := syscall.LoadDLL("./zlg-x64-dll/zlgcan.dll")
if err != nil {
log.Fatal("Error during loading .dll file,", err)
}
defer canDll.Release()
proc, err := canDll.FindProc("ZCAN_InitCAN")
if err != nil {
	log.Fatal("Error during finding procedure,", err)
}

However, the code above seems to work correctly on my laptop (go 1.22), but it crashes when I run it on my PC at office(go 1.19).
So, I have two question:

why the code above can load one .dll file correctly, while cannot load the other one?
why the code above have different behavior on different computer?

Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For questions please refer to https://github.com/golang/go/wiki/Questions


Disable managed memory for nvc++

Using managed memory breaks certain tests like test_thread_sort at least on windows / WSL
Disable it uniformly to be on the safe side and let the user deside whether they want the fast footgun

The change looks fine to me.  I think the same effect could be achieved by changing -stdpar to -cuda in cmake/CubCompilerHacks.cmake.  The biggest difference between -stdpar and -cuda is that -stdpar turns on managed memory and -cuda does not.
This section right next to where the line was added is old.  The NVC++ part of the comment no longer applies.  This can be removed.
    # Today:
    # * NVCC accepts CUDA C++ in .cu files but not .cpp files.
    # * NVC++ accepts CUDA C++ in .cpp files but not .cu files.
    # TODO: This won't be necessary in the future.
    list(APPEND cxx_compile_options -cppsuffix=cu)


HDDS-2655. Use pre-compiled Pattern in NetUtils#normalize

What changes were proposed in this pull request?
Replace String#replaceAll in NetUtils#normalize with a pre-compiled Pattern, which is better for performance.
https://issues.apache.org/jira/browse/HDDS-2655
How was this patch tested?
Added unit test (TestNetUtils).

Thanks @ayushtkn and @elek for the review, and @elek for committing it.


Create Dockerfile

Add directory "action-a" and create "Dockerfile" in it

Review changes, verify actions


Feature/update description

You must change the basis branch. You are merging this to main instead of dev.


GOSA1 - UI for Registration with email and with google auth

Description
GeeksforGeeks Andriod App registration with the help of email and other credentials or with help of google authentication. Create text fields to take relevant user data and create a user in user_data field in firebase. And also create a button to activate google authentication.

kindly assign it to me

I want to work on it

I can work on it

@muskan-888 You can start working on this issue
If you have any doubts or queries, you can comment here or ask us on the WhatsApp group

plz assign it to me!

@singhtaran1005 thank you for the contribution. you have recieved 5 points for the contribution. Keep it up


fixes to pkg-config

Some small tweaks to make yajl.pc get installed to the correct location and include the necessary include directory.

sorry, further testing is obviously required. my double -I doesn't appear to work with all versions of pkg-config either. closing for now, i'll come back when i have something better tested


Activate newly installed extensions immediately

Based on https://github.com/microsoft/vscode-azureresourcegroups/pull/146#discussion_r838925631. If an extension is installed that would have already been activated by the activation manager (had it been installed), then we should immediately activate it.

How does this "listen" for newly installed extensions?

How does this "listen" for newly installed extensions?

How does this "listen" for newly installed extensions?

It already was listening for new extension installs, but it wasn't previously immediately activating those that it would have activated had they been installed already. Line 34 (in the new PR version, line 37) adds a listener to vscode.extensions.onDidChange. Unfortunately VSCode doesn't offer any info on what changed (add, remove, upgrade...no data at all), so we just have to scan all the extensions again. Fortunately that takes very little time--2ms at most.

I was able to do a test by building a VSIX of the Docker extension with a fake onResolve listener. Everything worked 👍


fix(testing): Work around kokoro image changes

Recent changes to Kokoro have caused this initial git command to error, resulting in no tests being run.
While we're skipping the check here, it is still done in testing/kokoro/system_test.sh, so I do not anticipate any cost increase over out previous baseline (prior to the breakage).

Instead of commenting this out, can we instead try to mark this as a safe repo with something like git config --global --add safe.directory .... I think this will allow us to run the commands if I understand correct

I don't believe this is a safe.directory issue, because the error message is not in a git repository in the failing build, rather than the safe.directory message of fatal: detected dubious ownership in repository.
I tested safe.directory fix just to verify, and got the same error

@muncus Hmm, kk thanks. Can you file an issue to undo this so we make sure this eventually gets restored?

@codyoss filed #2811 to track.
The check is still, AFAICT being performed in the system_test.sh script that kokoro runs - is there a reason we need it in both places? Feel free to follow up on that bug with any other changes you'd like to see.


I had to remove that line cause, apparently, it was trying to delete my entity before the children.

We look at this row and the reason it's failing for us is because of the auto-flush in NHibernate before potentially dirty (out of sync) reads, which is the case for the listing below the line in question in this pull request. See our pull request. /Olle, Henrik


Support HTTP2 by using Jetty 9.4.8

Increase number of default thread in the pool to 13
Add a connector for http2
Use Conscrypt for ALPN Processor

As has been discussed on previous threads about this, the problem with upgrading beyond Jetty 9.2.x is that this drops support for Java 7, which is problematic both for enterprises with slow upgrade paths and Android devs.
I'd be happy to accept a solution that doesn't drop JDK7 support. Two options I can think of to achieve this are:

Add another JAR artifact to the build that's Java 8+ only and uses the latest Jetty version. This would probably involve splitting the source tree up into core and version-specific parts, which the build would include/exclude for the various output JARs.
Add some runtime detection of the environment i.e. are we on Java 8+ and Jetty 9.4+, and load the HTTP/2 code only under these conditions, such that NoClassDefFoundErrors and the like aren't thrown when running on Java 7.

👍
Do you have any preference ?
The first one seems to be more involved but potentially cleaner. The second one could be pretty straightforward but will clutter the code base a little bit...

I'm leaning towards the second at the moment. I made some changes a while back that allow wiremock to work under jetty 9.2 and 9.4 (partly by doing ringtone detention) so it would be an extension of that.

Ok that sounds good, I'll get started on that. If you have done something similar in the past, do you have any pointer or commits that I can look at ?

This is the code the dynamically detects which Jetty API we're using and acts accordingly, to give you a flavour of what I mean:
https://github.com/tomakehurst/wiremock/blob/master/src/main/java/com/github/tomakehurst/wiremock/jetty9/JettyUtils.java#L53

@tomakehurst i believe this concerns can be resolved by dockerlize this? i'm more than happy to contribute after this PR is accepted.

the problem with upgrading beyond Jetty 9.2.x is that this drops support for Java 7, which is problematic both for enterprises with slow upgrade paths and Android devs.

@samuel-soubeyran are you planning to add GRPC support after this PR? perhaps we can work together as well. 👯♂

@sunnycmf I'm not sure Docker would be much help in this instance, since it can't be used by Android devs or many people in slower-moving environments like banks.

Yes docker wouldn't solve the problem,
and yes the goal is to be able to use Wiremock to mock services over GRPC.


how to include js library files

As in plugins and so on:
just update them frequently? or do more submodules? or?

dont bother just copy/paste from another local project. Need bower/npm in time


The script property get pluralized in the inspector when selecting several nodes

Godot version
4.0 beta 1
System information
linux
Issue description
https://user-images.githubusercontent.com/71937306/190901680-a2de38ef-3fc3-4f57-94b5-27f10ef2c2a4.mp4
As you can see the other properties don't change.
I'm not sure if this is wanted behavior (to make it explicit that several properties are being edited, in which case it's incomplete) but at least the tooltip is lying (Node has no scripts property).
Steps to reproduce
Select several nodes
Minimal reproduction project
No response

Can you reproduce this on 3.5?

Can you reproduce this on 3.5?

This happens too in 3.5.stable.official.991bb6ac7 with a twist:

1 node selected
several nodes selected

setting the scripts property works as expected (sets the script property on each node).

tooltip says this property is called "scripts"

setting the Reference/script property sets the script property on each node but its value remains empty.

tooltip says this property is called "script"

https://user-images.githubusercontent.com/71937306/191741433-73f7782a-42a9-4953-aa99-86094d1f238f.mp4


fix(minifier): keep template literals with expressions

Merge activity

Oct 24, 4:00 AM EDT: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.


[Bug] Shift does not end caps word when achordion is in use

Describe the bug
When I'm in caps word mode, and I press my mod-tap shift, or type a shifted character, it doesn't end caps word mode when achordion is enabled. I am not 100% sure this isn't something to do with my own customization, but I've been trying to rule it out and haven't so far. If anyone can confirm this works fine for them I'll keep digging in to my own code.

e.g. I am trying to type CAPSWord
I hit <capsword>caps<shift>w<shiftup>ord
I end up with just CAPSWORD

My keymap
Information
Do the keys involved use any of the following features?

[x] Achordion (from this repo)
[ ] Auto Shift
[x] Combos
[x] Key Overrides
[x] Mod-tap or Layer-tap keys
[ ] Other custom userspace code:

Aha, seems this is only when using shift that has eager mods applied. I'll try to see what the fix is. Looks like we already have a workaround for caps word when invert on shift is enabled, maybe we need something similar for when it isn't enabled.

I just expanded the exclusion to always disable eager mods during caps word and everything works like I expect now.
- #if defined(CAPS_WORD_ENABLE) && defined(CAPS_WORD_INVERT_ON_SHIFT)
+ #if defined(CAPS_WORD_ENABLE)

Thank you for catching this and for suggesting the fix! I'll push a commit in a moment to do that.


Fixing Quest Progress

Subject
Fixing Quest Progress Component
Description

Too much white space.
Rounded corners on progress bar to make it uniform

Are you willing to participate in the development of this component and create a pull request?
None

Hi, @illionillion I'm interested in solving this issue, also could you please give me more info about the white space.
TIA

Hi, @illionillion I'm interested in solving this issue, also could you please give me more info about the white space. TIA

Hi, @RudhraBharathy !!
I leave it to you!!
Good Luck!!


fix(deps): update all node modules

This is to fix save-state and set-output commands, that use environment variables now.
https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/#patching-your-actions-and-workflows

Remember to squash and merge, there are commits with very large changesets on this one.


Fix issue 6 https://github.com/PrestaShop/ps_mbo/issues/6

QA approved, we just need a CR before merge

By the way, this repo needs a dev branch

It already has one, I can see it, maybe I need to do something else for the branch to be visible by everybody ?


Added LSM copy tool

Hi, I've created a first version of LSM fetch. The structure was copied from my GFAL2 module.
Will be happy to hear a feedback during testing.

Thanks a lot Pavlo.


⚠ Telly has degraded performance

In 3b3968b, Telly ($TLY) experienced degraded performance:

HTTP code: 200
Response time: 1324 ms

Resolved: Telly performance has improved in e73f74d.


remove data with normal label in training set

I noticed that you removed all normal labeled instances from training set( In your kddcup99 notebook input [5] X_train, y_train = X_train[y_train ==0], y_train[y_train==0] ). However, in my opinion, for an unsupervised learning method, you ought not to use known labels  before testing, they are just for testing your model performance. But when I tried to remove this code, I met this problem after running 6 epochs :
InvalidArgumentError: Cholesky decomposition was not successful. The input might not be valid.
	 [[Node: GMM/Cholesky = Cholesky[T=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:GPU:0"](GMM/add)]]

I wonder the reason of this circumstance. Could you give me some ideas? thanks a lot!

It seems that I missed something mentioned in the paper :
and only data samples from the normal class are used for training models.

Sorry for my carelessness = =


[plot legend widget] Legend is blinking when many curves are updated

Feedback from @mguijarr. After few fixes to use item.setData, his application is still blinking.

open a plot 1d
open the legend widget
create 2 curves
update curves data every seconds
The legend widget should blink, while there is nothing to update on its side

Maybe it still could exists a plot.clear on his application side. But we could check if we can reproduce the problem.

I'm closing it: duplicate of  #3345, PR #3346 should fix it.