NPM vulnerability complaint for moment

Thank you for this amazing library. While updating one of my plugins that uses this library, npm audit reports that the moment library version has known vulnerabilities.
So likely just needs a version update.
Tnx!

Oh, now I see it might be the obsidian library itself.


Support OpenId Connect via PAC4J

Seems like Pac4j 1.7 has started to support openid connect. Since CAS already takes advantage of this library for oauth, etc it would be good to dedicate perhaps another module to support openid connect  from the cas server perspective.
Tagging @leleuj so he's aware of the request.

Yes, I'm on it, it will go into the cas-server-support-pac4j module which is based on pac4j (I just released v1.7.0 which supports OpenID Connect) and supports "authentication delegation".

Thank you for the summary. This will be a valuable addition to CAS.

PR: https://github.com/Jasig/cas/pull/910

Handled by https://github.com/Jasig/cas/pull/910


Clean up of code and repo

I reshuffled the directory structure and made the whole workflow much simpler I think (simpler to document at least!) :)

The same file preprocessing.py is in two different folders. It would be good to only have it in one place or split the file in two. The latter would be preferable because I think it contains different functions for NER and SRL/word embeddings.

explore_ner.ipynb is also missing the seaborn import: e.g., import seaborn as sns

In word2vec-embeddings.ipynb, preprocessing the corpus takes a really long time (> 20 min). I am wondering whether this is really necessary because for me, preprocessing the entire text in R only took about 2 min?

I managed to run all the notebooks now, so here are a few general comments:

The code contains still quite a few commented out sections. Perhaps we should remove some of them before we wrap up?
Some cells of the notebook take quite long to run. A note on that would probably be useful!
The code still contains a few duplicated functions which could be put into one single location (I pointed out where).

In word2vec-embeddings.ipynb, preprocessing the corpus takes a really long time (> 20 min). I am wondering whether this is really necessary because for me, preprocessing the entire text in R only took about 2 min?

Yes, well I do additional preprocessing on top of what you do in R and that requires interacting with some files (replacing ngrams etc.) 20min indeed still seems like a lot but it takes around 7min on my laptop so I don't know why it takes that long for you...

The same file preprocessing.py is in two different folders. It would be good to only have it in one place or split the file in two. The latter would be preferable because I think it contains different functions for NER and SRL/word embeddings.

Fixed, now uses only one file in util/


README.md -  enable the matchers in Jest mode -

In order to get the expect-playwright matcher to work in Jest mode, you need to import the machers and extend the expect function.
This was missing for me?

Did setupFilesAfterEnv not work for you?

Closed since it seems stale and for other users it seems working.


Dont see line in the chart on android device if I use hideDataPoints

Description
With hideDataPoints on in android line i nchart is not visible.
Steps to reproduce

Implement line chart
Add hideDataPoints

Snack or a link to a repository
No response
version of react-native-gifted-charts
1.4.10
React Native version
0.74.0
Platforms
Android
Workflow
None

@SpiritMan77 please use the latest version of the library


Game crashes due to empty laser list

File "/home/priyansh/Python2/projects/space-invaders/main.py", line 268, in <module>     menu()   File "/home/priyansh/Python2/projects/space-invaders/main.py", line 260, in menu     main()   File "/home/priyansh/Python2/projects/space-invaders/main.py", line 248, in main     redraw_window ()   File "/home/priyansh/Python2/projects/space-invaders/main.py", line 201, in redraw_window     menu()   File "/home/priyansh/Python2/projects/space-invaders/main.py", line 260, in menu     main()   File "/home/priyansh/Python2/projects/space-invaders/main.py", line 239, in main     player.move_laser(enemies,4)   File "/home/priyansh/Python2/projects/space-invaders/main.py", line 106, in move_laser     self.lasers.remove(laser) ValueError: list.remove(x): x not in list

Error Generation:
The error is generated whenever you continuosly press the space bar to shoot the lasers.

Attempted Fixes :
Tried making if else statement so that the laser is not removed from the list when the size of the list is less than a particular size but this doesn't fix the issue.

I tried to force this bug to happen using your original code and I don't think it happens because you press continuously the space bar to shoot, because I wasn't able to get the error a lot of times and I was always pressing the space bar. I also think it doesn't have anything to do with if the list is empty or not.
Checking the error message and analysing the code to see why it would happen I have the theory that it happens when a laser hits more than one enemy at the same time, so in the for loop it gets removed once per enemy, but you can only remove it once, so the second time it gives the error. I'm not totally sure though.
However, the solution is simple and you don't have to worry about finding out what exactly is making the error happen, unless you want to find out for learning purposes. The only thing you have to add is an if statement that checks if that element exists in that list before removing it:
def move_laser(self,objs,vel) :
        for laser in self.lasers:
            laser.move(vel)
            if laser.off_screen(HEIGHT) :
                self.lasers.remove(laser)
            else:
                for obj in objs :
                    if (laser.collision(obj)) :
                        objs.remove(obj)
                            if laser in lasers:
                                self.lasers.remove(laser)

I'll add a commit to my main branch and make a PR. It also has other commit fixing typos etc.


Refactored Data Sources into a standalone package

This is a refactor of Data Sources into a standalone package, incorporating some API changes to allow for some new features and some future expansion.

A toga.sources.Datum is the end point value that is to be rendered.

Any object could be used as a datum; the __str__ representation of the datum will be what is displayed.
On some widgets, certain attributes of the datum have special meaning. For example, Tree and Table views will look for an icon attribute; if it exists, it will be added as an icon in the cell where the datum is rendered.
As the displayed value is determined by __str__, the datum can be an integer, float, or any other value that makes sense.
The datum can store data or values that aren't necessarily displayed.

A toga.sources.Row is a collection of Datum objects that are grouped in some way - for example, a row in a Table. However, a Row is not ordered - it's a dictionary type, and values are retrieved as attributes of the row.

On a widget where values are ordered (such as Table), the widget has an _accessors attribute that mirrors the headings. The accessors are the attribute names that should be retrieved. In simple cases, the user won't have to specify accessors - they are determined programatically from the heading names themselves (lower cases, stripped of special chars, etc)

A toga.sources.Node is a Row that has children. It is used by Trees in much the same way as Rows.
There is a toga.sources.ListSource and toga.sources.DictSource that are, as you'd expect, wrappers of lists and dicts.

At present, data manipulation operations (such as insert/remove) are all on the Source. This includes subtree operations on the DictSource.
This "attribute and dictionary" approach means a single data source can be used for different visualisations of the same data - change the headings on a table, and the data shown changes, too (in theory, at least - the signals aren't actually hooked up to do this)
It also makes ListDataSource more useful for DetailedList. DetailedList displays rows, but not in simple "columns"; a column based data source doesn't make much sense.

@Ocupe This is a big change; I still have to fix the broken tests, and add more tests explicitly for the new data source APIs, but I'd be interested in your opinions on the general direction.

Overall, I like that we're pulling data sources into their own top-level object, and I like that the complexity around the core widgets hasn't increased much, and has decreased in some cases.
Similarly, I think we're increasing internal complexity but giving the user a simpler interface for putting data in their application. This is the right direction to be moving in, although the internal complexity increase is noticeable, and I think might make things a little tougher for developers of BeeWare (but maybe our target is developers using BeeWare).
My biggest concerns are: testing, documentation, and interop.
The surface area has increased, but so has the testable surface area. Additionally it would be amazing to have some docs about "how to build data-driven applications", even if they're just a stub, or a heavily commented example.
As for interop, this PR drives home the need for people focused on the Windows and Android side of Toga. This is a breaking change in many ways, and will required the other platforms to catch up.
Not at all asking you to do this work, and it's far from the best use of your time, but still needs to be done before I'll be comfortable cutting a version.

@phildini Completely agreed about docs and testing. This is a work in progress, not intended as a patch ready to commit. I didn't want to write a whole bunch of docs if the basic premise was going to be rejected.
And yes - interop is a thing we need to deal with. To date, the Table and Tree widgets have been very ad-hoc; @Ocupe and others have done some great work recently on those two widgets, which is one of the reasons I thought we should take that work, spend the time to polish the data source APIs, and make sure we have a good foundation going forward. Hopefully, we're now homing in on stable APIs that can be reproduced on multiple platforms, and future incompatibilities will be minimised.

@freakboy3742 sorry if my feedback was too granular! I totally get this this is still in "proof of concept" stage.
For my $0.02, I do think this is roughly the right approach, especially since users don't have to care much about how the data sources work (for now).

If I may, I'd like to offer a few ideas (hardly suggestions and certainly not requests nor complaints).
The Source class appears to be an Observable (here a gist of what I would call an Observable), except it has _notify(), which calls a method with the given name on all the listeners (widgets, I believe), and that two "notifications" are in use: data_changed and refresh. data_changed is used many times (12), but refresh is only used once. Unless I'm missing something, both tell the widget implementation to do the same thing: re-render the widget, using self.interface.data. Is refresh necessary?
If _notify() can be replaced with something like notify() from the gist, then Source can be renamed Observable. End users could write their own data source types by inheriting from Observable and calling self.notify() each time they mutate data (such as in a custom insert() method, etc). To me, this has value since the observer pattern is already an established concept, while Source requires additional explanation/definition. It also eliminates the need to understand the functions of individual "notifications" (like data_changed or refresh), and implementation classes only need one "refresh-me-now" method.
To use this approach, self._data.add_listener(self._impl) would be replaced with self._data.add_listener(self._impl.data_changed). I know I've been quite wordy, but I hope I'm not adding confusion.
Another idea: ListSource could implement __iter__ and become for-loop friendly!
Again, these are only ideas; do with them what you wish. I am excited to see this underlying data API making headway, and it I look forward to seeing it develop and settle.

@DariusMontez Thanks for that input. The observer pattern is broadly what I was aiming at; the difference being that I wasn't sure if we were going to need more than one "notification" method. I've been trying to move away from refresh (due to ambiguity with "refreshing" a Detailed List), but I wasn't certain if we're going to need at least 2 signals - one for "update all the data" and one for "update this row". If that need disappears, then the "classic" observer that you've described makes perfect sense.
However, it's not accurate to say that Source "becomes" an observable. It implements observable, but Source has much more functionality than just Observable. A Source is effectively an "observable list", or "Observable Dict" - the "List" and "Dict" parts are just as important as the "Observable". Being observable is just how the data interchange takes place.
As for __iter__ - I'm not 100% sure if there will be a whole lot of use for it, but it's not hard to add, so why not :-)

@freakboy3742 I saw that you removed the optional refresh from the ListSource.
def insert(...., refresh=True):
    if refresh:
        # call refresh on listeners

This is fine by me but I added it for a reason that we should keep in mind and address.
I'm not sure if you already addressed it but I just put it out there anyway :)
Example:
When we delete a row on iOS we get a nice animation by doing this on the native level:
tableView.beginUpdates()
self.interface.data.remove(item, refresh=False)
tableView.deleteRowsAtIndexPaths_withRowAnimation_([indexPath], UITableViewRowAnimationLeft)
tableView.endUpdates()

but if we do not defer or prevent the refreshing of the widget the animation gets cancelled.
So the problem is that iOS takes care of refreshing the view by it self and when we interrupt this mid way we can't take advantage of the nice animations (deleting row, adding row). I guess this is also true for other widgets and maybe even other platforms?

@Ocupe Well spotted :-)
I did remove it - because I didn't think it was appropriate for the data interface. The decision as to whether to actually refresh is something for the listener to worry about, not the data source. One of the listeners is the widget itself; and the widget is in a much better position to know whether updates need to be suspended.
In the particular case you've described, I'd imagine using a context manager to wrap the beginUpdates/endUpdates lifecycle, and also encompass setting an internal flag so if data_changed is triggered during the context manager's lifecycle, the widget won't be redrawn.
Does that make sense?

@freakboy3742 makes sense yes. I imagined that you already considered this ;)
I'm happy that you clean up after me :D otherwise we would end up with a entangled mess :/

@Ocupe @phildini I think this is getting close to mergeable. There might be a couple of tweaks to be made to the arguments for some of the notification methods, but I think the broad structure is sound now, and there are a couple of PRs (#282 from @DariusMontez, and one that I have queued up for DetailedList on Cocoa) that depend pretty heavily on these changes.
So - can I get a sign off on this broad API design so we can merge and tweak as new widgets reveal issues?


[blockchain] Exclude zero values from gas price metrics

Description
The maximum and average gas price metrics were disturbed by the zero values of the system contract transaction. With zero values adopted, they do not accurately reflect actual user gas prices.
The PR fixes this, by removing miner transactions from transaction count, and providing price bottom limit to metric collector when no user transactions are included.
Changes include

[x] Bugfix (non-breaking change that solves an issue)

Testing

[x] I have tested this code with the official test suite

Codecov Report

Merging #337 (28202d7) into dev (8b45344) will increase coverage by 0.11%.
The diff coverage is 31.81%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##              dev     #337      +/-   ##
==========================================
+ Coverage   47.37%   47.48%   +0.11%
==========================================
  Files         135      135
  Lines       20440    20451      +11
==========================================
+ Hits         9683     9711      +28
+ Misses       9919     9908      -11
+ Partials      838      832       -6

Impacted Files
Coverage Δ

blockchain/metrics.go
11.49% <0.00%> (ø)

blockchain/blockchain.go
46.21% <30.00%> (-0.40%)
:arrow_down:

blockchain/testing.go
60.46% <100.00%> (+0.18%)
:arrow_up:

... and 4 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Incompatible with @trivago/prettier-plugin-sort-imports

prettier version: 2.5.1
@trivago/prettier-plugin-sort-imports version: 3.1.1
tailwindcss version: 2.2.17
My .prettierrc.json
{
  "singleQuote": true,
  "jsxSingleQuote": true,
  "importOrder": [
    "<THIRD_PARTY_MODULES>",
    "^@/types$",
    "^@/types/(.*)$",
    "^@/api$",
    "^@/api/(.*)$",
    "^@/config/(.*)$",
    "^@/context/(.*)$",
    "^@/pages/(.*)$",
    "^@/components/(.*)$",
    "^@/hooks/(.*)$",
    "^@/helpers/(.*)$",
    "^@/images/(.*)$",
    "^@/locales/(.*)$",
    "^[./]",
    "^./tailwind.css$",
    "^.*.css$"
  ],
  "importOrderSeparation": true,
  "importOrderCaseInsensitive": false,
  "tailwindConfig": "./tailwind.config.js"
}

Only import sorting works

Hey! Thanks for reporting this. I've created a new issue (#31) dedicated to keeping track of Prettier plugins that are not compatible with this plugin. Closing this issue in favor of that 👍

PS — please feel free to share this as a comment in that thread, it would be great to have it there for quick reference.

@Mattinton thanks for the workaround, I had to make a small change because it was giving me TypeError: Cannot read properties of undefined (reading 'map').
const tailwindPlugin = require("prettier-plugin-tailwindcss");
const sortImportsPlugin = require("@trivago/prettier-plugin-sort-imports");

module.exports = {
  parsers: {
    typescript: {
      ...tailwindPlugin.parsers.typescript,
      preprocess: sortImportsPlugin.parsers.typescript.preprocess,
    },
  },
  options: {
    ...sortImportsPlugin.options,
  },
};

Update on this one — we've added support for this plugin in the latest (beta) version of our Prettier plugin. You can read more about that here: https://github.com/tailwindlabs/prettier-plugin-tailwindcss/issues/31#issuecomment-1312202554

In case anyone else runs into this. If you import a type with the same name as an exported function. Prettier will throw this exact same error.
Example:
import { RandomName } from './page'

// other code

export default function  RandomName () {
// more code
}


Godot resizing window to match primary display when requested to draw in secondary monitor

Godot version
3.3 stable
System information
Linux Mint
Issue description
I have two monitors:
External Monitor 1: 1920x1080 (External monitor)
Laptop Monitor 2: 1366x768 (laptop)
I have Godot running in Monitor 1 and running the game in Monitor 1 with Windowed mode with no scaling. Whatever I set as the height it is being reduced the maximum allowable size that the window (as below examle) can fit inside the viewable area of the laptop. In this example I set the game to be 1024x768 but the Window is show as 1024x714, i.e. 768 minus the task bar minus the window decoration.
In the editor settings I have run set to 'current window' and then tried explicitly 'monitor 1'. Either way it starts correctly in the external monitor but is resized to fit the screen minus task bar of the laptop.
If I turn off the display on my laptop and just have the external monitor, it works fine. If I set my external monitor as my 'primary', it works.
So, no matter what I do, Godot is taking the maximum size of my laptop primary display viewable area and resizing my game display, rather than taking the metrics of the display being requested.

Steps to reproduce
As above, I simply create a game to fit my monitor, but is larger than my laptop and it gets resized.
Minimal reproduction project
No response

I'm fairly sure this is a duplicate of another issue, but I can't find it right now.

@Calinou I've just updated, what is happening, if it helps, is it takes the resolution of the 'primary' display even when displaying in a secondary monitor.


option to only print red URLs?

Could there be an option to only print the URLs for which there was a failure? This way it'd be easier to go through them for a website that has many URLs?

They are all printed in summary at the end, but yes we could definitely have a print fails only option!

I think a better option here is to set some sort of print levels, or better print style something --prints-style that can take value like all, fails-only, maybe success-only too?

@SuperKogito I think this idea is spot on - after we review and merge https://github.com/urlstechie/urlchecker-python/pull/28 would you care to work on this for urlchecker-python? I think the variable might be something like --print-level to mirror the idea of logging levels.

I'm going to transfer this issue to urlchecker-python, as it needs to be implemented there. As a follow up it will need to be added here.


ombi: 4.22.5 -> 4.35.10

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for ombi is: Self-hosted web application that automatically gives your shared Plex or Emby users the ability to request content by themselves
meta.homepage for ombi is: https://ombi.io/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.35.10 with grep in /nix/store/h6s47ihp453cra10z6ibhyxgs8qyixqz-ombi-4.35.10
found 4.35.10 in filename of file in /nix/store/h6s47ihp453cra10z6ibhyxgs8qyixqz-ombi-4.35.10

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
ombi

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/h6s47ihp453cra10z6ibhyxgs8qyixqz-ombi-4.35.10 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A ombi https://github.com/r-ryantm/nixpkgs/archive/04aa417225803b968d7ad7b5a1db97f9b5149b71.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/h6s47ihp453cra10z6ibhyxgs8qyixqz-ombi-4.35.10
ls -la /nix/store/h6s47ihp453cra10z6ibhyxgs8qyixqz-ombi-4.35.10/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

ombi

Maintainer pings
cc @andreisergiu98 for testing.

Result of nixpkgs-review pr 218571 run on aarch64-linux 1

1 package built:

ombi


Size on Buttons

I understand that this is in the early stages of development, but I had a thought.
Let me know if we should leave you alone during this early phase
Setting the size using ShadcnButtonSize is ignored if a height is set on the inherited theme.
Arguments on a widget itself should take precedence over the inherited theme.
https://github.com/nank1ro/shadcn-ui/blob/9c3278988be3160d2dbce226b50f7f943a1827f2/lib/src/components/button.dart#L286-L310
P.S. It would be awesome if the preset values could be inherited from the Theme.

Hi there,
thanks for reporting the issue.
Just refactored the whole theme, because it didn't convince me.
I think how I implemented it now should be better.
I also changed the order in which the operations are performed to compute the size of the button, check
https://github.com/nank1ro/shadcn-ui/blob/main/lib/src/components/button.dart#L298C3-L307C6
I also moved some of the presets in the component theme.
Let me know what you think about it

Setting padding on the button theme won't work unless the user explicitly sets the theme size to null.
IMO I would expect an explicitly declared padding theme to take precedence over a default size.
EdgeInsets padding(ShadcnThemeData theme) {
   if (widget.padding != null) return widget.padding!;
   if (size(theme) != null) {  // <<< This by default will return true
     return defaultPaddingForSize(size(theme)!); // <<< Will be used instead of the themes padding
   }
   if (buttonTheme(theme).padding != null) {
     return buttonTheme(theme).padding!;
   }
   return defaultPaddingForSize(ShadcnButtonSize.$default);
 }

This is because by default the size is set to $default and the button uses the padding from size before the padding theme.
I would make a class to contains presets for each of the button sizes or change default size to null

I would go with whatever Material Mike has to say about implementing a ThemeData, he's done this the most.
Thanks for being so gracious ❤

Would you still have the single properties width, height and padding in the Button class?
These properties would be removed from the button theme and converted in this single class?
Feel free to open a PR if you have all in mind 💙
I personally don't like to have multiple sources of truth, for example the padding property is determined by both the padding and size properties, while it should be based on a single source of truth.

Would you still have the single properties width, height and padding in the Button class?
Yeah, using another class would here would seem strange.

These properties would be removed from the button theme and converted in this single class?
Yup

See what I did here:
https://github.com/nank1ro/shadcn-ui/pull/2

Right now the size property only affects width, height and padding., so that's what I moved into another class.
I could see how people would want to move other things into a subclass:
E.G. If a user want to have more rounded corners on a large button.
Another way could be having the entire main theme contain 4 different button themes.
This would make things much more complex, we would basically need to do what you've done with the types of buttons (primary, destructive...) for sizes too.

closed by https://github.com/nank1ro/shadcn-ui/pull/2


Document Upgrade to 0.5 rc1?

Are you planning on documenting the upgrade to 0.5 rc1?
I have read on slack that the location of the macaroons has changed ... so not sure it as simple as the upgrade to 0.4.2.

it's as simple as upgrading to 0.4.2, except you need to change the copy command
sudo cp /home/bitcoin/.lnd/data/chain/bitcoin/mainnet/admin.macaroon /home/admin/.lnd/data/chain/bitcoin/mainnet/

When I upgraded I had issues with different path for the macaroons, I eventually figured it out which essentially was the command above.

One more difference in 0.5rc (wasted a LOT of time on this one).
If you are running on testnet - commands like:
lncli getinfo
will fail with message like this:
unable to read macaroon path (check the network setting!): open /home/admin/.lnd/data/chain/bitcoin/mainnet/admin.macaroon: no such file or directory
solution is to add parameter: --network=testnet
lncli --network=testnet getinfo

I also can't get the auto unlock to work with 0.5.
I changed the code like this:
#!/bin/sh
# LND wallet auto-unlock script
# 2018 by meeDamian, robclark56

/bin/sleep 180s
LN_ROOT=/home/bitcoin/.lnd

curl -s \
        -H "Grpc-Metadata-macaroon: $(xxd -ps -u -c 1000 ${LN_ROOT}/data/chain/bitcoin/mainnet/admin.macaroon)" \
        --cacert ${LN_ROOT}/tls.cert \
        -d "{\"wallet_password\": \"$(cat /etc/lnd/pwd | tr -d '\n' | base64 -w0)\"}" \
        https://localhost:8080/v1/unlockwallet > /dev/null 2>&1

echo "$? $(date)" >> /etc/lnd/unlocks.log
exit 0

I'll definitely document the update process for the final 0.5 version. But thanks for all the valuable information already, that will help me quite a bit!

The automated RPC credentials, zmqpubrawblock and zmqpubrawtx settings did not work and had to be configured in lnd.conf.
parameter: debughtlc=true is not swallowed by 0.5-beta-rc1.
The macaroons are moved into .lnd/data/chain/bitcoin/mainnet. This must be considered when allowing access to non-bitcoin account. The same tree must be created under /home/admin/.lnd for it to be copied and given read-access.

RC1 was hogging RAM and CPU, RC2 is much more friendly.
System Overview script needs a small adjustment too, since macaroonpath is different. Omitting this while callin LNCLI works when account has read access, just like the bonus scripts for LNBalance and LNChannels....

@AmazingDroopy
I wrote the LNBalance & LNchannels scripts, that @Stadicus added to the bonus section. I have yet to upgrade to V0.5 rc1 or rc2.
After I get RC2 installed, my scripts will break ... so I will then fix them and issue Pull Request with the changes - unless someone beats me as it should be a pretty simple fix.
My assumption is that only one or both of the  lncli=xxxx  lines need editing:
# Set default (mainnet)
lncli='/usr/local/bin/lncli'
lnd_pid=$(systemctl show -p MainPID lnd | awk -F"=" '{print $2}')
chain='main'

# read cli args
for i in "$@"
do
case $i in
    --testnet*)
    lncli="/usr/local/bin/lncli  --rpcserver=localhost:11009"
    lnd_pid=$(systemctl show -p MainPID lnd_testnet | awk -F"=" '{print $2}')
    chain='test'
    shift # past argument=value
    ;;
esac
done

Hopefully, I address all these topics in the now updated guide. Please help me make sure that there are no evil pitfalls and feel free to open issues or create a pullrequest if you stumble upon an error.
Thanks for your contributions!


Add move notice to previous repository

Hi @sponce : Could you add a quick move notice to https://gitlab.cern.ch/sponce/cpluspluscourse/ so that people who still have the old link end up here?

Yes, sure. I'll "close" it and redirect.

I've archived the project, changed the title to "C++ course - outdated", changed the description to points here and added a readme pointing here. Should be enough :-)


Mardown feature to print Heading

I wanted to add h1 Heading feature from the markdown

Looks great!
Thank you!


[DO NOT MERGE] Update roadmap

⚠ This repo is Continuously Deployed: make sure you follow the guidance ⚠
DO NOT MERGE awaiting final copy
What / why
Update the roadmap page. Mostly content changes, with a few layout changes, specifically:

moves the visit counter to the bottom of the page
adds three links to other pages with images at the top of the page
adds sharing links at the bottom

Visual changes

Before
After

When you review, please also check that the sharing links at the bottom of the page work as expected.

@andysellick if this is to be shared on social media I think we should use metadata to improve the way it would be previewed.
As it is currently, if shared on Facebook it would look a little something like this:

Which is fine, but perhaps a little bit generic? I wonder if we should be more deliberate with it.
If shared on Twitter it would simply appear as a link, as none of the metadata to generate a twitter share card is there.
This definitely needs some custom Open Graph meta tags (my bad for missing this in the original implementation of this page)!

@danacotoran I've updated the copy today and added the new links for sharing. I don't have either of these accounts (twitter, linkedin) so can you test the links work properly please?

This definitely needs some custom Open Graph meta tags (my bad for missing this in the original implementation of this page)!

Have added some of these as well.

Checked the social links and this looks good to me now, but it might be good to get a content designer to check the copy updates properly as I'm not the best at spotting issues with that.
One thing that stood out to me visually was that on some breakpoints the text in only one of the  "Programme updates" columns wraps onto the next line which does not look great but also not the end of the world if it's not an easy thing to fix.


Replace travis-ci with gh actions

This PR replaces the Travis-CI configuration with GitHub actions to re-enable CI for the repo. Clicking through the badge on the README, it leads to https://travis-ci.org/github/db-migrate/node-db-migrate, which has been roughly disabled for a year to the day with a message to migrate to travis-ci.com. However, Travis-CI has drastically altered how open source works on the .com domain, along with the recent stream of security leaks that makes it no longer the recommended choice of CI.

Assuming fine with moving to gh actions, I can put up similar PRs for the other repos within the organization.

Sure please do
Sent from Nine
--
Best
--
Tobias Gurtzick
CEO
WizardTales GmbH
Additional Contacts:
@github | @.***
Website: https://www.wizardtales.com
@.***
nmlpm: -- coming soon & open source --
Please tag your mails with "request", "issue" or "proposal", unless none of these fit or there already exists a thread.
-- Why should one reach for what one may be capable reaching, that's awfully boring and after all one still be ones yesterday being. --
Sitz der Gesellschaft: Ratingen
Kokkolastr. 5
40882 Ratingen
Germany
Registergericht: Amtsgericht Düsseldorf, HRB 84140
Geschäftsführer: Tobias Gurtzick

Von: Matthew Peveler @.***>
Gesendet: Mittwoch, 15. Juni 2022 20:35
An: db-migrate/node-db-migrate
Cc: Subscribed
Betreff: Re: [db-migrate/node-db-migrate] Replace travis-ci with gh actions (PR #785)
Assuming fine with moving to gh actions, I can put up similar PRs for the other repos within the organization.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Thank you for your contribution :tada: !
This was actually on our ToDo for quite some time.


VM-accelerated emulator is possible inside VMs that support nested virtualization

Important
You can't run a VM-accelerated emulator inside another VM, such as a VM hosted by VirtualBox, VMWare, or Docker. You must run the Android emulator directly on your system hardware.

The above info is incorrect. You can run an accelerated Android emulator inside a VM if that VM host supports nested virtualization. As of right now, VMWare Fusion, Parallels Pro, and Parallels Business all support nested virtualization. I've personally tested HAXM accelerated emulators on VMWare Fusion and Parallels Pro.

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

ID: 4e37d642-2051-8d70-9968-dfb484122a6a
Version Independent ID: 178ea80c-2b99-0c44-eeb6-9db319439b33
Content: Hardware Acceleration for Emulator Performance (Hyper-V & HAXM) - Xamarin
Content Source: docs/android/get-started/installation/android-emulator/hardware-acceleration.md
Product: xamarin
GitHub Login: @conceptdev
Microsoft Alias: crdun

Thanks for the feedback @ryanoberleitner - we'll clarify the language.

Can Azure VMs with nested virtualization support (Dv3 series) currently run VM-accelerated emulator?

Can Azure VMs with nested virtualization support (Dv3 series) currently run VM-accelerated emulator?

Faced with the same question as @kszicsillag. Probably someone has already configured Android emulator in such environment and could describe the personal experience or provide a useful manual for configuration.

I am using VM which supports nested virtualization, though I faced issues with opening the emulator with x86 processor- We've encountered an error with your Windows Hypervisor Platform (WHPX) configuration that is preventing us from running your emulator accelerated.
Please ensure WHPX is properly installed, then perform a factory reset on the emulator and try again.
But was able to run emulator with API-25, armeabi-v7a, given it is very slow.

I am using VM which supports nested virtualization, though I faced issues with opening the emulator with x86 processor- We've encountered an error with your Windows Hypervisor Platform (WHPX) configuration that is preventing us from running your emulator accelerated.
Please ensure WHPX is properly installed, then perform a factory reset on the emulator and try again.
But was able to run emulator with API-25, armeabi-v7a, given it is very slow.

Equal behavior for my Azure VM with nested virtualization (see the screen with error bellow).

Also, was able to launch emulator with API-25, armeabi-v7a, but it worked drastically slow.

Yeah, Is there any reason why x86 wont work on the VM ?
Also, I want to run UI tests on the emulator, but it doesn't run. not able to decide if it is because of the VM constraint..
UI tests run fine with armeabi-v7a processor on my laptop hardware.

Hi,
I have to develop an android application in virtual machine (VmWare Horizon). Operating system of VM is windows 10 and I could installed android studio 3.5 there and successfully imported code. But I couldn't ran emulator in virtual machine. So is it possible to run emulator in virtual machine? Any workaround? or can you please tell me how to run emulator in VM? Thanks in advance
VM details:
Processor: Intel(R) Xeon(R) CPU E5-2699 v4 @ 2.20GHz 2.20 GHz
System type : 64-bit Operating System, x64-based processor
Emulator Error:
Unfortunately, the Android Emulator can't support virtual machine acceleration from within a virtual machine.
Here are some of your options:

Use a physical device for testing
Start the emulator on a non-virtualized operating system
Use an Android Virtual Device based on an ARM system image(This is 10x slower than hardware accelerated virualization)

Thanks for submitting an issue and we apologize for the delayed response. The requested updates haven't been made since the creation of the issue, and the timeline for resolution may vary based on resourcing, so the issue is being moved to an internal repo to better track it.


spotify: use default libgcrypt / libpng

Instead of overriding libgcrypt with the (insecure) libgcrypt_1_5 and libpng with libpng12, use the defaults for those two packages.
spotify was changed to use libgcrypt_1_5 instead of libgcrypt in commit 165cb05ea5f42866a5b38521a8266f10045d48b2 by @monocell in PR #8157 to address #8156, which found that:

the current spotify client seems to depend on libgcrypt.so.11. Pretending with libgcrypt.so.20 produces an error like:
... libgcrypt.so.11: version `GCRYPT_1.2' not found ...

Given that the relevant Spotify client is more than 5 years old, I don't think we have to worry about this any more. :)
Built and checked on my NixOS desktop machine because @dotlambda said he wouldn't check proprietary software in #111215.
This commit helps #106203, but doesn't close it because libgcrypt_1_5 is still used in staruml.
Motivation for this change
libgcrypt_1_5 has a heap overflow vulnerability; removing it is blocked on removing it from spotify and staruml.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Backport in 3c0ade192abdbc40256dcc5e72418c92303ee84c. I also only built it on the release branch.

Backport in 3c0ade192abdbc40256dcc5e72418c92303ee84c. I also only built it on the release branch.


Mission takeoff with rotary wing rejected due to (horizontal) acceptance radius

With a "high" (20 meters) NAV_ACC_RAD setting, takeoff for multicopter missions gets annoying. The Mission Feasibility Checker rejects takeoff altitudes below 21 meters, which is not always desired.
We should use the altitude acceptance radius NAV_MC_ALT_RAD for this check.

Fixed.


refactor!: remove runtime semantics of readonly

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

50 of 51 (98.04%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.08%) to 95.932%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Tuple.ts
22
23
95.65%

Totals

Change from base Build<PHONE_NUMBER>2:
-0.08%

Covered Lines:
1225

Relevant Lines:
1272

💛  - Coveralls


[Video Player]: Media viewer onClick enhancement functional specifications

User Story

As a [user role below]:
DDS developer

I need to:
have the Video Player functional specs

so that I can:
so that I can properly code the Video Player component that meets the Design functions and intent

Additional information

Overview and scope of the engineering work should be included in the linked epic (#5245 )
Development issue (#5247 )
Visual Designs & Functional Design Specs Box folder link(TBD)

Acceptance criteria

[ ] Documented Video Player functional specifications changes
[ ] Sent and/or presented the specs to Design and Engineering for their review and comments
[ ] Additional stories are created, if needed, for any work to be done in future sprints

Updated: https://ibm.box.com/s/y8py6laddi9jslvxbt30kvinkvfwfalp


CI: Workaround recently broken add-apt-repository on GHA

Hopefully adding the sources manually still works.
Follow-up to https://github.com/godotengine/godot/pull/83147, which didn't help.
Another attempt at working around https://github.com/orgs/community/discussions/69720.

You should be able to revert workarounds now; I believe I've fixed it properly in Launchpad.


Add base.html to meet_balls_app/templates

Desired Outcome
create a simple base.html file for other future templates to inherit
Implemented Changes
add base.html file to meet_balls_app/templates
in the file you will find links to bootstrap cdn and super basic Navbar implementation.
Connected Issue/Story
Resolves #64

In addition to the description, I will explain that the changes I made were only to create the base file and change the home.html existing file so that it inherits from the base, thereby checking that it really works.
one more thing i would like to know if tests are required for this kind of PR, if so what should i test for ?

In addition to the description, I will explain that the changes I made were only to create the base file and change the home.html existing file so that it inherits from the base, thereby checking that it really works.
one more thing i would like to know if tests are required for this kind of PR, if so what should i test for ?

What you can do is to add a test case for a get call on the main page and assert on the templates being called (as done on the class this week). I don't know however if this will list the base template or not. It will be good to have it anyhow as an example for testing templates usage.


TypeScript syntax highlighting wrong after decorator

When adding a decorator to a variable declaration i TypeScript, both public/private and the name of the variable is plain white, while if removing the decorator both are colored correctly.

Reproduced with
@MyAnnotation(1) private var foo = 'a';
Please add your example as well...
This needs to be filed against https://github.com/Microsoft/TypeScript.


Cannot move or copy instance of a class exposing Slot without breaking callback

The intended use of ass for class decoupling will not invoke the callback of the expected instance when when copied or moved.
e.g.
class Popup {
public:
    Slot<> show = Slot<>(this, &Popup::render);
private:
    void render() {
        Popup * pointer = this;
    }
};

Popup popup;
Signal<> signal;
signal.connect(popup.show);

Popup movedPopup = std::move(popup);

signal.emit();

Here, calling emit() will trigger the instance of Slot belonging to movedPopup, which is as expected. However the value of this in the callback method render() will be that of popup.
This is because the callback function in Slot (which contains the this value of popup) was moved into the Slot instance in movedPopup - this is as intended for Slot copy/move semantics, but when that function contains a pointer to this we get unexpected results, like this.

Just for context, pushing this onto the client could look something like this
class Popup {
public:
    Slot<> show = Slot<>(buildShowCallback());

    Popup() = default;

    Popup(Popup &&other) noexcept {
        this->show = std::move(other.show);
        this->show.setCallback(buildShowCallback());
    }
private:
    std::function<void()> buildShowCallback() {
        return [this]() { render(); };
    }
    void render() {
        Popup * pointer = this;
    }
};

Popup popup;
Signal<> signal;
signal.connect(popup.show);

Popup movedPopup = std::move(popup);

signal.emit();

😞


Unnecessary fields in docker compose files

I recently did some tests and it seems that some fields inside the docker compose files are unnecessary: privileged, everything concerning XAUTHORITY and QT_X11_NO_MITSHM
Does anybody know why they were set in the first place?
Additionally, it seems that Intel graphics cards (not our case) need an additional volume to be shared with the host to be able to use the GPU: /dev/dri

I have no idea about this. I would say that we should remove whatever is useless.

Apparently the XAUTHORITY related field are there to be able to see Webots' GUI from an SSH connection (implemented in part in simulation_server.py) as explained in this tutorial. The problem is that currently this mechanism doesn't work because the Docker containers do not have access to the correct ports to display the GUI (ports 6000 + DISPLAY if I understood ssh -X's  DISPLAY mechanism correctly).
A way of "correcting" this would be to add the equivalent --net host from the run command to the docker file with the field network_mode: host but that would be a very bad idea security-wise.
Another way would be to change the docker compose files to open the correct port is there is an SSH connection but that doesn't seem easy to do.
Or we could remove all XAUTHORITY related fields from the "production" docker compose files and have a separate "debug" docker compose file that would have all of them + the network_mode: host field for debugging purposes.

Yes, the last option seems to be the best and the simplest to implement. We may simply add a comment in the code explaining how to re-introduce the display of the Webots GUI from a SSH connection.


concurrent.futures in Python 2

I see in #59 that it was stated that concurrent.futures should not be reported because it is in the standard library. For Python 2, this is not the case. It has to be installed as futures. Whether this special case should be addressed depends on what level of support for Python 2 is wanted now.

Fixed in #80.


Chromium zTranslate & overflow:hidden

Do we know if either of these issues have been addressed and if these still impact engine?

https://github.com/Famous/famous/issues/493
https://github.com/Famous/famous/issues/621

I know the first issue is marked as fixed, but I am still able to reproduce the issue locally in Chrome 43.

https://code.google.com/p/chromium/issues/detail?id=421946
https://code.google.com/p/chromium/issues/detail?id=456601

It is my assumption, neither issue has been addressed and still impacts the engine in the same manner as the old library.

@jd-carroll
I will make tests for these but my initial assumptions are that
https://github.com/Famous/famous/issues/621 has been fixed
https://github.com/Famous/famous/issues/493 has NOT been fixed

#493 has been fixed.  See code here https://github.com/michaelobriena/famous-zindexTest
In a container here https://api-te.famo.us/codemanager/v1/containers/1ba2b818-ea4e-4af9-896c-ccfd555478a9/share

#621 has been fixed as the seed project is working.

@jd-carroll I will leave this open for a bit so you can confirm if you choose to.

Ok, I trust you so I'll go ahead and close.  I am still seeing an overflow issue with the old library, so I'll try porting some of that code to see how it responds.  If I am having any issues, I'll repost here. Thanks!

@jd-carroll could you fix your issue with the old library?


Hi i dont know if this is a issure or not

i just want to ask if how to do knex.js in models, ive try to create my own but it fails, it doesnt recognized by the controller, can you give samples on how to do it, also how classes to be extend from another model or controller, thanks

@Jamesoncy Can you be more specific with the issue. Share some code of what exactly you are trying to achieve

thanks for the reply
HomeController
'use strict'
const User = use('App/Model/User')
class HomeController {

index (request, response) {
response.json(User.getUsers())
}

}
module.exports = HomeController
Model
'use strict';
let Lucid = use('Lucid'),
Database = use('Database');
class User extends Lucid {
getUsers(){
let query = yield Database.table('users').select('*');
return query;
}
}
module.exports = User
i just wanna run some knex.js queries in the model but seeems i cannot retrieve it, asking if im doing it right or wrong, thansk for the feedback

Okay you are following the wrong approach here. Models can run queries on themselves, so there is no need to import Database into a model.
Model
const Lucid = use('Lucid')

class User extends Lucid {

}

module.exports = User

Now inside your controller you need to do
const User = use('App/Model/User')

class HomeController {

  * index (request, response) {
      const users = yield User.all()
      response.json(users.toJSON())
  }

}
module.exports = HomeController

Also I would suggest you to read more about ES2015 Generators https://medium.com/javascript-scene/the-hidden-power-of-es6-generators-observable-async-flow-control-cfa4c7f31435.
And read some docs on Lucid http://dev.adonisjs.com/docs/3.0/lucid

ah ... ok ahhahahaha im just trying if that can be happened, ok thanks
ahhahah so i just nid to stick with the lucid, also with the join queries ?
On Tue, May 24, 2016 at 11:55 PM, Harminder Virk<EMAIL_ADDRESS>wrote:

Okay you are following the wrong approach here. Models can run queries on
themselves, so there is no need to import Database into a model.
Model
const Lucid = use('Lucid')
class User extends Lucid {
}
module.exports = User
Now inside your controller you need to do
const User = use('App/Model/User')
class HomeController {

index (request, response) {
const users = yield User.all()
response.json(users.toJSON())
}

}module.exports = HomeController
Also I would suggest you to read more about ES2015 Generators
https://medium.com/javascript-scene/the-hidden-power-of-es6-generators-observable-async-flow-control-cfa4c7f31435
.
And read some docs on Lucid http://dev.adonisjs.com/docs/3.0/lucid
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/adonisjs/adonis-framework/issues/168#issuecomment-221317345

Yes


Repeated headline and section at release notes page

Hi.
At page Visual Studio 2022 version 17.8 Release Notes, the headline and the section of "Visual Studio 2022 version 17.8.1" have repeated twice.

This appears to be fixed now.


Getting few errors on build

I get errors,
Failed building wheel for ansible
Failed building wheel for PyYAML
Failed building wheel for MarkupSafe
Failed building wheel for pycparser
Fixed it though by adding
RUN pip install wheel
before
RUN pip install ansible

@spekki - Interesting; I haven't run into these issues last time I did a build, which was a few days ago (to get Ansible 2.7.0 on the container—see https://hub.docker.com/r/geerlingguy/docker-ubuntu1804-ansible/tags/). Are you making any modifications, or running docker build directly on this project's Dockerfile?

@spekki - Interesting; I haven't run into these issues last time I did a build, which was a few days ago (to get Ansible 2.7.0 on the container—see https://hub.docker.com/r/geerlingguy/docker-ubuntu1804-ansible/tags/). Are you making any modifications, or running docker build directly on this project's Dockerfile?

Didn't do any modifications before that wheel installation line. No idea why it failed as i said, it's working now. Just leaving this here in case someone else bumps into the same problem.

@spekki - I'll chalk it up to weird chance or networking :) Thanks for posting the update!


Create a way to distinguish lines in ConfigQuotes

Is your feature request related to a problem? Please describe.

Quotes are separated by new lines.
Since wrap occurs, it is not possible to distinguish lines

Describe the solution you'd like

Add a '-' to describe the line

Fixed in 8276ba2 (incorrect commit message)


oauth changes contenttype, adds body as formdata

Fixes #272

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Adrian Ernst seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks!


Tabs top dropdown changed to "Processor"

it's interesting, but note that it's not obvious which tab in CSS / JS, "Processor" every where doesn't convey a lot of information

Hmm. Interesting. The 2nd person to note an issue with this.
I'll give it a rethink. It was changed because many didn't know that this
was how to change the processor, and the information as to what language it
was is replicated. I get your point specifically regarding CSS and
JavaScript in the empty state.
On Wed, 10 Aug 2016, 17:08 Cyril Auburtin<EMAIL_ADDRESS>wrote:

[image: image]
https://cloud.githubusercontent.com/assets/3134574/17561198/48d60d3a-5f25-11e6-8f9a-958e56b7e223.png
it's interesting, but note that it's not obvious which tab in CSS / JS,
"Processor" every where doesn't convey a lot of information
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jsbin/jsbin/issues/2843, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAA1hHKLqTRm89kjouHeALJkaIiF2JW1ks5qefdvgaJpZM4JhT1O
.

ah I see.. meh, normally the   JavaScript▾  explicitly tell to click to see more.
If it were me I'd even group those options in the top toggle buttons :), saves place, but potentially confusing

Noticed it as well. Would be nicer, something like Processor: SELECTED_PROCESSOR.

No intention to make the label longer than it has been historically.
A settings cog is the one item that's come up before (but it would be next
to the panel name)
On Fri, 12 Aug 2016, 10:06 Eugene Serkin<EMAIL_ADDRESS>wrote:

Noticed it as well. Would be nicer, something like Processor:
SELECTED_PROCESSOR.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jsbin/jsbin/issues/2843#issuecomment-239397618, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAA1hJiPnm1ICEyxYdruk6VVNKgAG_Jhks5qfDeUgaJpZM4JhT1O
.

There is no need to make it longer. There is always a choice to use title attribute for example or add some sort of marking in dropdown list so that it would be clear, what is selected at the moment. All those options don't make label longer.


Bump to ver. 1.2.2

Logging fixes and optmizations

Pull Request Test Coverage Report for Build 122

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 120:
0.0%

Covered Lines:
942

Relevant Lines:
942

💛  - Coveralls


open() and toggle() not working

I am trying to trigger Flatpickr from the click event of a different element using open() or toggle() functions, however they are not causing the calendar to display.
Steps to Reproduce (for bugs)

Create an  element, create a Flatpickr instance with it and assign it to a variable
Create a  element, and (I'm using JQuery) attach an event handler to the 'click' event
In the function of the click event for the div, call open() or toggle() on the variable object

var planDatePickr;
planDatePickr = flatpickr("#planDateBack");
$("#planDate").click(function () {
    planDatePickr.toggle()
});

If you want help, you need to provide the necessary details:
Your Environment

Version used:
Browser Name and version:
Operating System and version (desktop or mobile):
Link to your project:

Apologies, if I thought that information was pertinent I would have added it.

Version: Latest (2.0.8)
Browsers: Chrome (54.0.2840.71), IE (11)
OS: Win 7 x64
File: calendar.txt

Quite puzzling at first. This is why it happens:

The div, whose onclick is used to trigger the calendar, is regarded as an outside element - flatpickr closes when you click outside the calendar
flatpickr doesn't know that the div is used to open the calendar

Two ways around this:

https://chmln.github.io/flatpickr/#example-strap
https://jsfiddle.net/0a3f6Lsf/

I had exactly the same problem.
Just to be clear, for those in the future, the important part in the fiddle provided above, is the setTimeout().
Also, the first link to the example is broken.

As I was looking to solve a similar setup (use an icon for toggling the datepicker) and ended up here when googling, here's an updated link to the documentation: flatpickr + external elements


Fixed incorrect unpackDir expression

Remove whitespace between dirs as which would cause copying fail in app packaging.

@fujunwei, PTAL, thanks!

LGTM, merge it.


Feature/packet input ways

Multiple packet input method support added: hex (default), file, interactive
User can select method by setting --packet-input-method to hex, file, interactive
If user select --packet-input-method file the --packet-file option must also be supplied
File Mode
the --packet-file option gets a JSON file with this format:
{
  "packet1": {
    "hex": "...",
    "handshake": false
  },
  "packet2": {
    "hex": "...",
    "handshake": true
  }
}

packet2 key can be committed entirely
Interactive Mode
with --packet-input-method interactive an ipython console is opened and user can craft his packet there and finally store it into variable p
This feature currently requires ipython and doesn't work with default python console.

I changed entire packet reader class with factory methods
the class is a context manager so can manage add/remove of iptables rules (we still need cleantracevis.py from some redundant codes)
also tired to change error handling to work with Exceptions as much as possible (still we need some error checks and handlers)

Now all command line arguments are saved with reports as a file with same name and .conf extension
This file can be loaded later using --config-file


pd_client: add reconnect metrics

What problem does this PR solve?
Issue Number: related #9799
What is changed and how it works?
pd_client: add reconnect metrics
Related changes

Need to cherry-pick to the release branch

Check List
Tests

No code

Side effects
Release note

No release note

/rebuild

/merge

/rebuild


chore(release): Trigger a release of previously merged changes

Preview: https://react-catalog-view-pr-catalog-view-64.surge.sh


Multi-image viewer with navigator: clipping an image re-centers navigator

As discussed here:
https://github.com/cuberis/openseadragon-curtain-sync/issues/4#issuecomment-637681675
… In a multi-image viewer (where the images are stacked directly on top of each other) with a navigator, clipping one of the images causes the entire navigator to reposition its contents. The position of the navigator contents should be based on the entire stack of images, not just one of them. If they all get clipped, it makes sense that they would get repositioned, but with just one of them clipped, the overall content space is still the same and shouldn't be repositioned.
Here's an example before clipping:

… And after clipping:

It kind of looks like it's trying to center the intersection of the images, rather than the union of them.

Hi! I have been working on this issue, but I do think this is not an issue of openseadragon, but of another library. I'm not sure if I'm breaking any rules linking a pull request of another project, but in any case, I think I may solved it.
Link to pr: https://github.com/cuberis/openseadragon-curtain-sync/pull/9

@RammasEchor awesome! Yes, thank you for posting a note here about the fix! I shall close this issue :)


Install rule issue

Hi,
I have a Travis job for a package that depends on libpointmatcher  (ros-indigo-libpointmatcher) which fails building with the following error :
No rule to make target `/tmp/binarydeb/ros-indigo-libpointmatcher-1.2.3/contrib/yaml-cpp-pm/include'

The Travis job can be found here.
Any idea ?

Was is working before and now it fails? Is so, it would help to know when it crashes to track the problem.

No it never worked as it is a job that I very recently set up. I'll see if I can reproduce the error on my machine and/or get more detailed info.

I couldn't reproduce exactly the same error logs on my machine but it might be due to the fact that I also depends on pointmatcher_ros which misses a dependency on libpointmatcher.
See pointmatcher_ros PR 1.


Object property can't be set as nullable: true

Used versions:
Swashbuckle.AspNetCore 5.2.0
Swashbuckle.AspNetCore.Newtonsoft 5.2.0
Steps to reproduce:
I created response model with object property, expecting offer to be nullable. I tried two different approaches, without results:

allowing null property

    public class DashboardGetResponse
    {
        [JsonProperty(Required =  Required.AllowNull)]
        public OfferGetModel Offer { get; set; }
    }

allowing null directly inside object object

    [JsonObject(ItemRequired =  Required.AllowNull)]
    public class OfferGetModel
    {
        public Guid OfferId { get; set; }
    }

Actual result:

neither DashboardGetResponse nor OfferGetModel schema suggests that offer field is nullable

"OfferGetModel": {
        "type": "object",
        "properties": {
        "offerId": {
            "type": "string",
            "format": "uuid"
          }
        }
 }

 "DashboardGetResponse": {
        "required": [
          "offer"
        ],
        "type": "object",
        "properties": {
          "offer": {
            "$ref": "#/components/schemas/OfferGetModel"
          }
     }
}

Expected result:

nullable: true property in OfferGetModel swagger schema

"OfferGetModel": {
        "type": "object",
        "properties": {
        "offerId": {
            "type": "string",
            "format": "uuid"
          },
        },
       "nullable": true
      },

OR

DashboardGetResponse referencing that offer object can be of nullable type

This is a complicated one to explain, particularly if you're not familiar with the Swagger/OpenAPI Spec, but I'll give it a go.
By design, object properties are created as "reference" schema's rather than "inline" schemas. This means that the property schema will hold a reference to a shared schema that's defined in the components section of the generated document. Now, if that "shared" definition had the nullable property set, then that would mean it applies to all uses of the schema. In most cases this would be OK but not all. For example, if the schema was used as a response model, then you're documentation will state that the response from an operation MAY be null, and this could be very misleading.
Long story short, the nullable flag (as well as some other schema properties) are "contextual" and therefore applying them to a "shared" schema definition ends up being very problematic. So, out-of-the-box, SB just doesn't do it.
In fact, this is a known issue/consideration when working with Swagger/Open API documents. The recommended approach is to create "inline" schemas that "extend" the shared schema by using the allOf keyword. Swashbuckle DOES support this approach but it's opt-in. That is, if you apply the UseAllOfToExtendReferenceSchemas setting, it will structure the generated Swagger slightly differently, in a way that should resolve your issue.

I've read the discussion on the OpenAPI repository (https://github.com/OAI/OpenAPI-Specification/issues/1368). In fact it led me here. Good summary. I've tried your suggestion and it works :) Thank you.


Cancel authn input if the user presses Ctrl+C

Previously the system would wait for the user's input and not cancel input if the user pressed Ctrl+C. This is because the signal was intercepted and used to cancel the root context, but this context isn't used by the terminal reading code and so had no effect.
The solution is to run the input in a separate goroutine and select on the routine's output or the context cancellation.
Alas, there's no way to give the same power to Ctrl+D, because interpreting EOF input is very much the purview of the standard library package we're using. But users can still just press Enter to submit empty input.
Resolves #3411.

@coderabbitai review


A rough outline for the SVG learning page.

Let me know on what more you would like and/or changes to what is already there.

I have made the changes you request, though I wasn't sure if you were wanting me to keep the old requirements with your additions or only use your requirements in the comments.


Error during doom upgrade

Trying to upgrade Doom Emacs and getting an error.

I've run doom up multiple times
I've run purge, clean, sync, and compile.

❯ doom up
> Preparing to upgrade Doom Emacs and its packages...
✓ Doom is already up-to-date!
Doom is up-to-date!
> Synchronizing your config with Doom Emacs...
  > Regenerating envvars file at "~/.emacs.d/.local/env"
    ✓ Successfully generated "~/.emacs.d/.local/env"
  > Installing packages...
    - No packages need to be installed
  > (Re)building packages...
    - No packages need rebuilding
  > Updating recipe repos...
    x There was an unexpected error
      Message: Symbol's function definition is void
      Data: (void-function . straight--process-buffer)
      Backtrace:
        (straight--process-buffer)
        (let* ((buffer (straight--process-buffer)) (start (save-current-buffer (se
        (let ((alist (let* ((buffer (stra")) (ref (straight-vc-get-commit typcurre
        (let* ((--cl-rest-- (append (list :recipe recipe) recipe)) (recipe (car (c
        (while --dolist-tail-- (setq recipe (car --dolist-tail--)) (let* ((--cl-re
        (let ((--dolist-tail-- recipes) recipe) (while --dolist-tail-- (setq recip
        (let* ((recipes (delq nil (mapcar (doom-rpartial #'gethash straight--repo-
        (let ((doom-output-indent (+ doom-output-indent-increment doom-output-inde
        (if doom--cli-updated-recipes nil (straight--make-build-cache-available) (
    ! Extended backtrace logged to doom.error.log

Backtrace: https://pastebin.com/D6hFateg
Additional details:
Config: https://github.com/michaelknowles/dotfiles/tree/master/doom.d
Steps to reproduce:
doom up
System information:
generated    May 03, 2021 10:56:16
distro       "Arch Linux" Linux 5.11.16-arch1-1 x86_64
emacs        28.0.50 makepkg 0c75d7d08 ~/.emacs.d/ -> ~/.emacs.d/
doom         2.0.9 HEAD -> develop e68c83fd4 2021-05-03 13:27:20 -0400 ~/.doom.d/ -> ~/.doom.d/
shell        /usr/bin/zsh
features     ACL CAIRO DBUS FREETYPE GIF GLIB GMP GNUTLS GPM HARFBUZZ JPEG JSON LCMS2 LIBOTF LIBSYSTEMD LIBXML2 MODULES NATIVE_COMP NOTIFY INOTIFY PDUMPER PGTK PNG RSVG SOUND THREADS TIFF TOOLKIT_SCROLL_BARS XIM XWIDGETS GTK3 ZLIB
traits       batch server-running envvar-file
modules      :completion company (ivy +fuzzy +icons) :ui doom doom-dashboard doom-quit hl-todo indent-guides modeline ophints (popup +defaults) ligatures treemacs vc-gutter vi-tilde-fringe workspaces :editor (evil +everywhere) file-templates fold (format +onsave) snippets :emacs (dired +icons) electric undo vc :term eshell :checkers syntax spell :tools (eval +overlay) lookup lsp magit pdf :lang clojure data emacs-lisp (go +lsp) json javascript latex markdown org (rust +lsp) sh (web +lsp) yaml :config (default +bindings +smartparens)

Duplicate of #4950


Not Work for Wagtail 3.0

AutocompletePanel does not work any more if Wagtail version is upgraded to 3.0.

Fyi: I've created a quick PR #116 which fixes the compatibility with wagtail 3.0 for us.

Thanks @th3hamm0r
I'm wondering whether there will be a new release compatible with Wagtail 3.0 soon.

Please update, I'm working with Wagtail 3.0 now :(

I just worked on our Wagtail 3 upgrade to fix a couple of items that were holding this back.
#121
Thanks @th3hamm0r for your #116 that helped a lot 👍

The pull request was merged two weeks ago, but the latest release is from 3 month ago. How can I get the version for Wagtail 3? ... I am jumping in circles on GitHub. Thanks for the guidance in advance.

@andre-fuchs as with every python package on github, you can always try to install a package from a git repository (see https://stackoverflow.com/a/13754517/1394330). Sometimes there are some required build-tasks (eg. generation of css/js files), then you will have to build it yourself and deploy that result somewhere. wagtail-autocomplete already contains the built files in the repository, so the following installation should work.
E.g. to install the merged commits from above, you could try to install it like:
pip install git+https://github.com/wagtail/wagtail-autocomplete.git@03f46a0c256989690d0a43fc21fe0f37f9ede765

I'm currently using a branch of my fork, which contains #116, but also #118 which fixes a coloring issue:
pip install git+https://github.com/th3hamm0r/wagtail-autocomplete.git@73a18bd43a5b08367d87061f152372abe486c821

Of course, as soon as a new release is published, you should better switch back to a clean installation from pypi.

This ticket can be closed @aliceni81, wagtail 3+ is supported since the release 0.9.0 (https://github.com/wagtail/wagtail-autocomplete/issues/125#issuecomment-1288554542).

Thanks, @th3hamm0r. Closing now.


Add kid-sk to contributors

kid-sk contributed to this repo but hasn't been added to contributors.
Follow up #92 to see instructions

@kid-sk I am working on it.

Closed by #126


fix: support deno

Summary
Deno cannot directly run Biome via deno run -A npm:@biomejs/biome check.
import child_process from 'node:child_process';
import process from 'node:process';

child_process.spawn(process.execPath, ['bar.js'], {
	stdio: [process.stdin, process.stdout, process.stderr]
});

deno can not run when env  have a null or undefined value
May be it is the bug of deno.  https://github.com/denoland/deno/issues/20373
But it can fix this error.

Test Plan
It run correctly when I changed it.

Wow! Thanks!


WIP: Add parse and validate functions

Fix: #726
Split out parse and validate and then re-use them in all execute functions.
This is a draft, please let me know if this direction is acceptable. Also, the current approach does not actually work in case of execute and resolve_into_stream because of lifetime issues.
I would be very happy to get some guidance on this, if this approach works, I'll add:

[ ] Tests
[ ] Docs
[ ] Examples

Glad to see someone taking this on! At a high level, I have a few thoughts here, aimed mainly towards simplifying the API:

This is up for debate, but I personally don't think the ValidDocument type is actually necessary. If a user of the library wants to skip validation that should be fine.

An invalid request is still technically executable, and will always produce a stable result as defined by the algorithms in the Execution section

– https://spec.graphql.org/June2018/#sec-Validation
The spec goes on to say that execution "should" only occur for valid requests, but I'm not sure we need to enforce that.

It's seems pretty off to me for parsing to require the schema as an argument. In every other GraphQL implementation I've looked at, the parsing process just takes in a string and outputs an AST.
Juniper's parser currently requires the schema to convert scalar tokens to custom scalar values, but I don't see why this has to be done in the parser. Maybe we could move that into the validator and executor instead? If we did that, I don't think Document would even require the ScalarValue parameter any more.

@ccbrown Thank you for looking into this!

don't think the ValidDocument type is actually necessary.
I was basing this on the fact that the execute functions are named execute_validated_query*. I would then suggest to have 2 APIs:

Validate + Execute validated (named just execute). This would have a type like ValidDocument in between.  It would be the default use case.
Just execute without validation. This would then be named execute_unvalidated_query* or similar. To suggest you're doing something unsafe (since the spec says you might end up with ambiguous or surprising results for unvalidated requests). This is probably not what you want most of the time.

Seems like a reasonable idea. I will have to look into it more in-depth, but I would make a separate PR for that.

This is a lower-level API that only people who know what they're doing should be using to begin with, so I think the hand-holding is unnecessary, but if we do think a type like ValidDocument is needed, let's hold off on the second execute_unvalidated_query API. That can come in a later PR if someone has a use-case for it. My goal with the suggestion was to simplify the API, not complicate it.

Btw, these changes don't actually have to be in the juniper crate's API. We could satisfy #726 with public APIs in juniper::parser, juniper::validation, and juniper::executor. That would keep the public juniper API narrow and make it less likely that someone would stumble upon these and try to use them for something inappropriate.

I agree with these comments :-)

Okay, thanks! That makes sense to me. I'll close this and create new PRs for the 3 new public APIs.
I spent my evening yesterday investigating the ScalarValue type parameter and I understand now what you meant with removing it from Document.

I think you can delay the parsing and so remove the type parameter.
For validation, you should just be able to pass in a validator of sorts that would try to parse the unparsed scalars
I believe it should also be possible to remove the parameter from execution as well, but I need to look deeper into it

Let me know what you think.


Update ediscovery-search-and-delete-teams-chat-messages.md

Made changes to the document as the URLhttps://graph.microsoft.com/v1.0/security/ediscovery/cases does not work and gives 400 bad request error.
Changing it with https://graph.microsoft.com/beta/compliance/ediscovery/cases works fine
fixes#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/10603

@JohanFreelancer9 please copy review

@tiburd Please merge this Pr. Thanks

close > reopen


Be stricter about the camera restarting after a property change.

The ZWO camera kept on crashing after exposure time changes. This is a bit slower, but as we're not changing exposure times that often, this is a worthwhile tradeoff to make.
The camera is not stopped for every property change, except for offset_x and offset_y.
Tested on hardware.

Something got fixed in the meantime, not really happening anymore. Closing this issue.


[BUG] Appwrite 0.15 not working

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]
Used dataProvider [e.g. custom, Strapi, Airtable]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]
Used dataProvider [e.g. custom, Strapi, Airtable]

Additional context
Add any other context about the problem here.

Hey @rospisey ,
Thank you for contacting us! We'll try it with<EMAIL_ADDRESS>and let you know!

Hey @rospisey ,
Can you share some more details about the error?

Hello @omeraplak
Appwrite error with auth_provider and data_provider with package @pankod/refine-appwrite. You can just change these function to support new appwrite version. Appwrite V0.15 has new database.

Hey @rospisey ,
We've updated our Appwrite API to 0.15 and it's working as an expected. How can we reproduce the issue?
Example: https://refine.dev/docs/examples/data-provider/appwrite/
Multi-tenancy example: https://refine.dev/docs/examples/multi-tenancy/appwrite/


daysOfWeekDisabled is not working

daysOfWeekDisabled option is not working .. i am trying to send an array of weekend [5,6] to make them disabled but
it did not work

Some options are not fully work. We will add and fix more options in the upcoming version
Thanks


[ai notebooks] new image segmentation tutorial

Here is a new tutorial about image segmentation with the BraTS2020 dataset. The aim is to segment brain tumors in MRI Scans.

Need some typo fixes before merge. Working on it


v1.49 creates debug.log automatically

VSCode Version: 1.49
OS Version: Windows 10 Enterprise - 1903

Steps to Reproduce:

Open react native/react js project with VS Code 1.49.
After project is loaded, debug.log is created automatically in root directory.

The debug.log file is created in almost source directories.

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

Does this problem also occur with all extensions disabled?

Nevermind, this is a duplicate of #105191

/duplicate #105191


Add Firefox Ubuntu Snap support #325

Hi,
Just wondering if there are any updates on this PR? I'm currently affected by this.
Alternatively - is there a workaround? Maybe a shell command I could run to install the cert into the Snapped Firefox?
Many thanks for your work on this very useful tool!!

This is also affecting me on Ubuntu 21.10.

@FiloSottile friendly bump


core.lock: Add coreutils dependency

...and use realpath from there.
Should close https://github.com/nix-community/dream2nix/issues/930

Thanks


If edit in table is enabled in production app, Manager is not enabling edit in table in devext

Describe the bug
You can see the behavior in this test app, This is a production app but we are using devext code to run this app.
https://devext.arcgis.com/apps/instant/manager/index.html?env=arcgis4localgov2.maps&appid=daedc3c3d1c04f94b55fcdb4a8ac9e07
Actual production app: https://arcgis4localgov2.maps.arcgis.com/apps/instant/manager/index.html?appid=daedc3c3d1c04f94b55fcdb4a8ac9e07, you can edit in table without any issue.
Expected Behavior
It should allow edit in table.
Reproduction Steps & Sample
Open the test app and try to edit a value from the table and notice the behavior
Other Relevant Info
No response

@azizaparveen
This issue is related to the enableInlineEdit configuration.
The production app does not have a configuration key for enableInlineEdit, so when it is loaded with the devExt app, it does not allow inline editing.
Config Json without enableInlineEdit

Config json with enableInineEdit

@jmhauck - Seems like this could be a backward compatibility issue with the new configuration added for enableInlineEdit, Could you please check and let us know your inputs on this

@sumitzarkar @jmhauck  it has the Edit in Table option on, only thing I can think of is the property has moved from Manager tab - app level to map level, so you have to click on Edit individual map to see this property

@sumitzarkar @azizaparveen we can work around this in the Instant app. I will add a change for this.

https://devtopia.esri.com/WebGIS/arcgis-portal-app-templates/pull/10971

This looks good


Falling back to pure SSR

Hello, is it force SSR on all links based on a condition? For example, say a Bing searchbot visits (user agent contains " bing "), can we force all links to hit the server (SSR)? This could also be a nice fallback on text based browser as well.

More broadly, it would be nice to have an api to say "hey Next, render the next route as SSR" (and forget all the components you've downloaded). Basically like forcing a browser refresh but only upon entering the next route.

That's how it works already. If you want only SSR and no client side, use <a href> instead of <Link>

Hmm.. maybe a mod to the Link component then. Basically you want CSR unless text only browser or a search bot. As far as I know Bing doesn't render Javascript.

Using  instead of  mean ALL traffic will be SSR, which isn't the desire. Only in certain situations.

@babakness you could use  when a BOT is using your site. You might have ways to identify a bot user from the client side.
Anyway, I don't think that's a good SEO practice.
If there's no JS, all <Link> tags will be <a> tags.
So, what you are asking is already there.


Add body-parser to rpc-controller to allow JSON request bodies

Proposed Changes

Allow the use of a raw JSON request body for the query and proofs requests, as opposed to multipart/form-data only.

@Aescwine thanks for opening this PR and effort, adding body-parser is one of next task on our lists, but it cannot be done on one place only, there are multiple places in rpc controller, and requires very good testing to be done after implementing this feature. So I'm afraid we cannot approve this

@NZT48 no worries, thanks for the response and good to know it's on the list. If I weren't a JS noob I'd have a go at doing the proper implementation myself :)


Selecting an item in the circular wheel will not lead to an action if the action is assigned with modifier key (ctrl, alt, shift)

I want to use MineMenu as a convenient access to rarely used but necessary actions. In large modpacks there are many more actions than can physically fit on the keyboard, so the rarest of them (which do not need to be pressed during combat or walking) would be good to assign in combination with Ctrl. For example, instead of pressing u, press ctrl+u.
But it doesn't work. I have Whyla - a fork of Waila, this mod adds hotkeys that change the information displayed on the screen. I assigned one of them to Ctrl+F8. MineMenu does not perform this action. I reassigned to F8 - MineMenu performs this action.
version: MineMenu-1.12.2-1.6.11-universal.jar

Known issue, can't be fixed unfortunately.
1.12.2 have also not been supported for years

Anyway 1.12.2 is best version at march 2024. You're going the wrong way.

No, no it's not. 1.12.2 was released in 2017

1.12.2 has many more available mods than any other version.


Improved reliability of Speed Index calculation

Apologies for the size of the PR. I can attempt to break it apart, though it may not be neccessary.
Summary of the changes:

Fix various definitions of beginning/time origin. In a few places, we used the first frame's timestamp as if that were the time origin. Now the first event's timestamp is used as the time origin all the way through. (A later PR will change this)
Added test cases for traces with 1,2,3. Added tests for them. The assertions confirm the end speedIndxes; I can imagine we assert something less end-to-end, like progress of the 1st and 2nd frames. wdyt?
Speed Index definition is updated. It's now, as Parvez described here, SI = firstPaint + sum(fP to VC){1-VC%}. This addresses the one-frame case completely, and improves accuracy in other places.
 It has to be the same size as the rest, but we do this for accuracy, as calculatePerceptualProgress finds our minPreceptualProgress which must be for a blank/default baseline rather than a frame with content.
Removed the dependency on babel. Aside from import statements, there was nothing left that isn't supported in node 5, so the improvement to dev iteration speed seemed worth it.

Fixes #21, #28

Still to do: #27 - allow for a custom time origin (or find navStart ourselves)
Also I reported a bug in babar which leads to bad plots: https://github.com/stephan83/babar/issues/2  Always doublecheck the individual frame progress %s.

Nice job. LGTM!


Modify Light hook

Added ModifyLight hook for ModLiquid

What is the reason for adding in x, y?

I was recommened that i add it by absoluteaquarian. Its to control where the liquid can light up.
Say if you wanted to make lava 2 where it would glow more intensly on certain frames then you can using the x and y


Allow 3 chars for all analyzer rules i.e. PTE1234

Hi Freddy,
This very small PR adds the ability to parse custom 3 character issue codes from alc output. I tried to find a gap for PTE, but without luck. Please see this as a compatibility PR for custom analyzers :).
Some years back, we decided to implement MCP which is not fully covered by the current implementation.
Thank you,
Carsten

Thank you!


Desktop support for USB, HID and Serial device access

This is a new feature to add USB, HID and Serial device access to the desktop by extending the existing WebUSB, WebHID and WebSerial functionality already offered in the web version #152310.
Since Electron version 25, Web-* device access has been possible using the functionality described here: https://www.electronjs.org/docs/latest/tutorial/devices
This ability is awesome because it allows access to functionality in Chrome already shipped as part of VS Code and means extension developers no longer have to include external packages such as node-usb or SerialPort. In many cases this approach should be more stable and reliable, too (e.g. chrome uses winusb for device access over libusb as used in node-usb).
To implement this addition, I've wired up a QuickPick dropdown for the user to choose the requested device leveraging Electron's APIs. Unfortunately the cleanest way I've been able to find to communicate between the electron-main and browser process is via the globals API. Happy to hear of a better way.
USB Selector:

Serial Selector:

HID Selector:

Please note: As this utilises the existing features in chrome, obtaining a concrete device after authorising it is only possible in the desktop web extension host. Therefore this feature is only available to pure web extensions (those without a main entrypoint in package.json).

Anything required to progress this PR?

Is there an associated issue with sufficient upvotes and motion to justify this change? And an explainer about the scenarios this would enable?

Is there an associated issue with sufficient upvotes and motion to justify this change? And an explainer about the scenarios this would enable?

I wasn't aware an issue is needed, I can copy-paste the original text above to a new issue if discussion is better on issues?

There are many conflicts in this PR btw.

There are many conflicts in this PR btw.

TBF, there weren't conflicts when I opened it a year ago and I got tired of keeping it up to date with no feedback.
If there is interest in this feature I'll revisit the PR.

There are many conflicts in this PR btw.

Resolved

The electron events (select-usb-device, select-hid-device, select-serial-port) which handle calls to the navigator device requests need to be wired into a UI for selection.
As these run in different parts of VS Code, I've had to wire an IPC channel in to send a device list for rendering and return a selected item. There isn't an existing pattern to do this, so advice on my proposal would be appreciated.


RuntimeError: inverse_cuda: For batch 0: U(2,2) is zero, singular U.

Great work!
We meet an issue caused by the computation of inverse_cuda We have pasted the error information as shown below,
m_inv = (sts+torch.eye(sts.size(-1)).to(sts.device).unsqueeze(0).mul(lam)).inverse() # way, d, d
RuntimeError: inverse_cuda: For batch 0: U(2,2) is zero, singular U.

May I repalce inverse() withcholesky_inverse()?

Looks like someone else also came across the same error: https://github.com/Tsingularity/FRN/issues/4#issuecomment-903588885
I think this seems to be an issue due to pytorch's version difference. Could u please try using our provided conda environment and re-run the code again? https://github.com/Tsingularity/FRN#code-environment This is the environment I used to develop this codebase and I never came across this runtime error before. Just let me know if this doesn't work for u.
For replacing inverse() with cholesky_inverse(), personally, I never used this function before so I cannot guarantee anything to be honest lol. But feel free to take a try and see how it goes.
Feel free to let us know if there's anything else we can help with.

Seconding that this is likely the same pytorch version issue.
Unfortunately, replacing inverse() with cholesky_inverse() isn't straightforward, as (to my knowledge) cholesky_inverse() doesn't currently support batched inputs, and implementing this sequentially could cause significant slowdowns.

closing this for now. feel free to re-open it if you need more help.

I have same issue.
when I follow your readme.md,  I got this error.
Traceback (most recent call last):   File "<stdin>", line 1, in <module>   File "/home/mars/anaconda3/envs/FRN/lib/python3.7/site-packages/torch/__init__.py", line 189, in <module>     _load_global_deps()   File "/home/mars/anaconda3/envs/FRN/lib/python3.7/site-packages/torch/__init__.py", line 142, in _load_global_deps     ctypes.CDLL(lib_path, mode=ctypes.RTLD_GLOBAL)   File "/home/mars/anaconda3/envs/FRN/lib/python3.7/ctypes/__init__.py", line 364, in __init__     self._handle = _dlopen(self._name, mode) OSError: /home/mars/anaconda3/envs/FRN/lib/python3.7/site-packages/torch/lib/../../../../libcublas.so.11: undefined symbol: free_gemm_select, version libcublasLt.so.11
so i found the method solving this problem here (https://github.com/pytorch/pytorch/issues/51080)
and then, i get this problem
Traceback (most recent call last):   File "train.py", line 34, in <module>     tm.train(model)   File "../../../../trainers/trainer.py", line 181, in train     iter_counter=iter_counter)   File "../../../../trainers/frn_train.py", line 101, in pre_train     log_prediction = model.forward_pretrain(inp)   File "../../../../models/FRN.py", line 137, in forward_pretrain     recon_dist = self.get_recon_dist(query=feature_map,support=self.cat_mat,alpha=alpha,beta=beta) # way*query_shot*resolution, way   File "../../../../models/FRN.py", line 77, in get_recon_dist     m_inv = (sts+torch.eye(sts.size(-1)).to(sts.device).unsqueeze(0).mul(lam)).inverse() # way, d, d RuntimeError: inverse_cuda: For batch 7: U(6,6) is zero, singular U.
I think your environment.yml does not working well now.
Can you check this?
Sorry for my bad English.
Thank you.


Adds draft package

This package implements a type that is meant to be a generic Buildpack
Plan entry resolves as well as a LayerType merger

The dotnet-core-runtime buildpack is now using a regular expression to evaluate the version-source matching a priority declaration: https://github.com/paketo-buildpacks/dotnet-core-runtime/blob/11b7d02648c9749d91356f3c7fa7b1419eca7242/plan_entry_resolver.go#L60-L62
Can we add support for that?

@ryanmoran There are some issues with doing a regular expression match. I would be interested in hearing what you would propose for the implementation.

I've taken a stab at a simple implementation that supports regular expressions: https://github.com/paketo-buildpacks/packit/pull/86/commits/92d1894e186c7592d2c134e258212f9c3f6b383b.
The main design choice was to assume that all strings passed as the priorities are regular expressions.

@sophiewigmore maybe this implementation would be better? In this case, we are allowing strings for exact matching and regexp for fuzzier matching cases.

@ryanmoran Oh wow, yeah. I definitely prefer that to requiring regexp only.

@ryanmoran @sophiewigmore LGTM


Cannot select text

I have an intermittent issue when using this theme where I cannot select some text with my mouse. Even if I double-click the word, the text does not get selected and the cursor does not move. I have ruled out possible issues with plugins by disabling all plugins. This issue was resolved by switching to another theme.

Strange Joleen. Thanks for noting this. If possible, can you take a
screenshot of when it happens.
On Sun, Mar 5, 2023 at 12:35 AM Joleen Bothma @.***>
wrote:

I have an intermittent issue when using this theme where I cannot select
some text with my mouse. Even if I double-click the word, the text does not
get selected and the cursor does not move. I have ruled out possible issues
with plugins by disabling all plugins. This issue was resolved by switching
to another theme.
—
Reply to this email directly, view it on GitHub
https://github.com/nickmilo/LYT-Mode/issues/55, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIOFMTPSV4AFBGTKAHDTG7LW2RF6XANCNFSM6AAAAAAVQBDMNA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I recorded a quick video of my screen.
https://user-images.githubusercontent.com/40122335/223049591-4e5e3c37-423a-4991-83c9-91813126ddbd.mov

Perfect, thanks for doing that. I don't have a quick solution, but
hopefully somehow out there does??
On Sun, Mar 5, 2023 at 11:50 PM Joleen Bothma @.***>
wrote:

I recorded a quick video of my screen.
https://user-images.githubusercontent.com/40122335/223049591-4e5e3c37-423a-4991-83c9-91813126ddbd.mov
—
Reply to this email directly, view it on GitHub
https://github.com/nickmilo/LYT-Mode/issues/55#issuecomment-1455632923,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIOFMTIYXPSBCMFSAUE4VBDW2WJOBANCNFSM6AAAAAAVQBDMNA
.
You are receiving this because you commented.Message ID:
@.***>


Module 'dialogflow' has no attribute 'SessionsClient'

Using dialogflow-0.1.0
import dialogflow

session_client = dialogflow.SessionsClient()

Error message:
Traceback (most recent call last):
  File "dialogflow.py", line 1, in <module>
    import dialogflow
  File "dialogflow.py", line 3, in <module>
    session_client = dialogflow.SessionsClient()
AttributeError: module 'dialogflow' has no attribute 'SessionsClient'

Tested on both Windows 10 and Ubuntu 16.04 (Windows subsystem linux)

Same issue here.

@samhains found a way out that it worked for me.
Change
dialogflow.SessionsClient()
for
dialogflow_v2.SessionsClient()
And also for any other use of dialogflow.(something) within your code.

you must import dialogflow_v2

is not working by import dialogflow_v2


The Signature header

I've had a discussion with some colleagues and it seems that the client and server are supposed to send different headers - Authorization for the client, Signature for the server, where their content is the same(?) except for the fact that the Authorization header has a prefix. Examples from the draft:

Authorization: Signature keyId="Test",algorithm="rsa-sha256",
headers="(request-target) host date content-type digest content-length",
signature="jgSqY(...)78w="

Signature: keyId="Test",algorithm="rsa-sha256",
headers="(request-target) host date content-type digest content-length",
signature="jgSqY(...)78w="

Signature.toString() assumes we want to send the Authorization header and, hence, always appends the "Signature " prefix, which makes it impossible to use directly for the Signature header.
The simplest way to add functionality to create the Signature header as well, is to change the scope of Signature.normalize() from private to public, and use that one on the computed signature string. It is not the cleanest way, but it would be backwards compatible. Another way is to rewrite and split the toString() method and make a clear distinction between the two uses.
What do you suggest?

Authorization for the client, Signature for the server

No.

Authorization is probably exclusively a client header and used for HTTP Authentication. This is typically seen in OAuth2 flows, or directly on a target endpoint to authenticate the client (ie prove they have permission to perform the action).
Signature can be generated by either the client or server. It is intended to be used primarily to ensure the integrity of the content - that it was generated by the client or server and has not been altered beyond the signer's original intention. It can also be used to authorise the client, but it is not uncommon to see both headers in this case that serve their respective purposes.

This is explained in the Introduction to draft version 11 of the protocol. If it is not clearly explained there, do raise an issue with your comments/suggestions to improve.

Not an issue as explained above.


fix: add missing methods to the interface iaccount

fix: add missing methods to the interface iaccount

https://github.com/pepperize/cdk-organizations/runs/5049121636?check_suite_focus=true#step:5:39
See https://github.com/pepperize/cdk-organizations/blob/main/API.md#identifier--5