Migrate CLOB columns to LOB (SMILE-3072)

Hibernate on Postgres has just awful behaviour when you use a column annotated with @Lob of type String. In that specific case, it creates a column of type text which has a stringified version of the largeobject ID corresponding to the string.
As far as I can tell, Hibernate have no intention of fixing this (it would be hard for them to fix retroactively admittedly). Various apps that use Hibernate have had to introduce workarounds, e.g. https://shibboleth.atlassian.net/browse/OSJ-135
This is just awful behaviour, since this isn't a reference that Postgres actually understands - Meaning that PG sees the CLOB content as an orphan and will reclaim it if you use the vacuumlo command.
This PR:

Adds a new hibernate dialect that extends the PG 9.4 dialect but add support for the PG oid datatype
Migrates all existing text CLOB columns to use OID instead

Additional note: Hibernate do finally seem to be moving on this issue. Ticket here:
https://hibernate.atlassian.net/browse/HHH-14211

Looks like this will fix #2434, right?

@tuomoa yup, it definitely should. Hibernate's default behaviour on Postgres is so weird...

Hey @jamesagnew, I am just bumping into this issue when upgrading hapi fhir to version 5.6.0. How can we make use of the fix described above?

@SimonedeGijt Please use this as the hibernate dialect: ca.uhn.fhir.jpa.model.dialect.HapiFhirPostgres94Dialect

Thanks, this worked indeed!

Hey, as far as I can see, this was already fixed in the latest Hibernate (5.6.2) -> https://hibernate.atlassian.net/browse/HHH-14211

Yup - That is just the weirdest timing. This issue has existed for ages in Hibernate, and the second we implement a workaround, they fix it. :)


Sample which uses onSaveInstanceState / onCreate(savedInstanceState)

Hi are you asking how to save the state of the flow using onSaveInstanceState? So we can retrieve it in onCreate?

I'm actually just making a note here that this is something that'd be nice for this lib and might look into doing if doesn't exist, I attended the talk today and that was what I was missing :D nice lib btw


Qiachip CB2S version (there are differenet version) just a simple relay BK7231N

this device: https://www.elektroda.pl/rtvforum/viewtopic.php?t=3874289&highlight=
qiachip_cb2s_beforePairingIthinkSo.zip

Profile added! Is it possible for you to test it on the stock firmware on your device?

bump, did this profile cause the device to show an A- ap after the exploit ran?

it looks like there is a dump of this device and a pinout in https://github.com/tuya-cloudcutter/tuya-cloudcutter/issues/128
if we are able to exploit it, is there a way to upload a firmware to it like https://github.com/kuba2k2/libretuya  OTA?
Thank you for all your help.

An update has been pushed that should include a profile for this device.  Please update your repo and give it a try.  Please see the instructions page ( https://github.com/tuya-cloudcutter/tuya-cloudcutter/blob/main/INSTRUCTIONS.md ) for any command or input changes.
If the new profile does not work, please re-open this ticket with details (including logs) of the failure.
Any feedback is appreciated.

Can you correct the naming please?
 > QIACHIP
   Qiachip

There are two entries with same name

Done.

Done.

Wow! So quick!
Thank you!
I was running the script for the second time and was wondering why only a single entry now :-)


[WIP] 3248 - Dashboard initial commit

Dashboard: Incremental Small Review Series - 1
General Description
Changed Home page redirecting behavior. When a user visits www.hydroshare.org, a browser loads

non-authenticated visitor:
the same web page as current release. But behind, url has  changed to www.hydroshare.org/general.
authenticated visitor:
login page /accounts/login. After login, the browser gets redirected to dashboard pages/dashboard.html

Code Changes:

urls.py:  url mapping, the "/" uses different a new view.
views.py added one new methods to handle routing, another method for handling 'general' url. The last one is to redirect dashboad template.
dropdownmenu.html Hardcoded "home" and link into "dropdownmenu.html". The links are going to be static soon. Please skip this file.
dashboard.html has fake content with a skeleton here. Content will be added in the next review. Please just take a look and move on.
base.html has a change from the old home '/' to the new '/general'.

Note:
On the web page, the new entry is called "HOME". The template and the view handler for that is actually "dashboard".

@cuahsimarko The code is let you folks know what I have done and seeking review and feedback. The code won't be merged to develop as it will lead to navigation behavior change.

closing this. A new PR is created to track latest progress.
https://github.com/hydroshare/hydroshare/pull/3302


UniProtKB shouldn't have KB in the CURIE prefix?

I just noticed that we use the prefix UniProtKB:, but it appears it should be UniProt:. Eg. Note how this URL doesn't resolve: https://identifiers.org/resolve?query=UniProtKB:Q5STB3, but this one does: https://identifiers.org/resolve?query=UniProt:Q5STB3
Can be reproduced here: https://arax.ncats.io/?term=TNF

Biolink chose UniProtKB:
https://github.com/biolink/biolink-model/blob/37e245f7755f5878bc1bab12d822b51ae72a88c4/biolink-model.yaml#L143
Pity that Biolink and identifiers.org are out of sync.
but I think we need to go with Biolink.

Ah, gotcha. So a nonissue that is kinda an issue; huh!


cxxrtl blackbox behavior issue / possible bug ?

Attached is an example showing the issue.  (sorry about tar.bz2 inside zip ... github doesn't want tar.bz2 attach :/ )
cxxrtl_bb.zip
It's a simple memory block implemented two ways : once using pure verilog  ( mem_sim_data_verilog.v ) and the other as a cxxrtl blacbox ( blackbox.cpp & mem_sim_data_bb.v ) and I expect them both to produce the same behavior, but they don't.
I didn't even get to test the write port but the read port appears as a "async" read instead of a "sync" one (i.e. data appears directly and not at the next cycle like I'd expect).

This is a bug; there is currently no way to make the blackbox version work.


feat: expose firmware version

With this PR the firmware version is exposed.

Is the already enough to get this done?


hardware: add enableRedistributalFirmware

Motivation for this change
Due the recent inclusion of broadcom-bt-firmware in enableAllFirmware,
it was required to set nixpkgs.config.allowUnfree to obtain the full
list. To make this dependency more explicit an assertion is added and an
alternative option enableRedistributalFirmware is provided to only
obtain firmware with an license allowing redistribution.
see https://github.com/NixOS/nixpkgs/pull/25567 for previous discussion.
Things done

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

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Minor gripe: Spelling
I think you meant Redistributable not Redistributal (which AFAICT is not a word)

fixed in e69758567559e3dea4577921364328b6f98e4ed3.


O/P Overlaps. Make output fit in terminal !!!

Pretty print the output. I know its just a small thing but everything overlapses. Would've done it myself but I'm a beginner.

I'm facing this myself. Besides removing the score columns and leaving only the final score, @flesch needs to improve its table layout module correct?

Yep, I've started some work to fix this.

@flesch I can help if necessary. Is there anything I can do?

@satazor @flesch I see that the description column is quite large.  Perhaps adjusting the size of this will be easiest?  Otherwise, removing one of (Quality, Popularity, Maintenance or Package Score) would be the next logical item.

Btw, out of topic, but did you guys saw my tweet about https://github.com/npms-io/npms-badges? I've released it yesterday.

@flesch is there any news on this?

Sorry for the disappearance - I do have some work on this I'll try to get in soon.
It will improve the table display, fix the description so it's just text (stripping any markdown and HTML that won't work in the terminal) and incorporate the unstable/insecure/deprecated flags.


Documents logic recently discussed in #964

@eirslett can you verify this is correct? cc @yurishkuro

Looks good to me!


Updated list of functions to include newly added ones

Added the list of functions below
Left
Trim
InStr
Word
Guid
IsNull
IsNullOrEmpty
IsPresent
IsString
CBool
CStr
ConvertToBase64
ConvertToUTF8Hex
BitAnd
Count
Join
Item
RemoveDuplicates
FormatDateTime
DateFromNum

@twimmers : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Michele, Tom's changes can be approved. I'll work to get him access to the other repo.

#sign-off

@twimmers @msmimart - this PR is too large for the public repo. We'll need to move it to the private repo in order to take advantage of the proofing, validation, and staging tools there. Before we can do that, however, the branch conflicts will need to be resolved. Can you resolve these?
Thanks.

Thanks @Jak-MS. @twimmers, let me know if you're able to resolve the conflicts, otherwise I can open a new PR in the private repo. Thanks!

@twimmers this one was replaced by https://github.com/MicrosoftDocs/azure-docs-pr/pull/98830/files and can be closed, correct?

@msmimart Correct, closing this out


Allow formatting of abbreviations

All abbreviations are currently lowercase
It is useful to be able to add them with formatting
This makes it possible

Just FYI, the casing of abbreviations is wholly determined by the contents of the csv at the moment. If you define eg.
TEST,stupid test
it'll output TEST.
My only real concern is a lot of the formatters don't really make sense for abbreviations (smash, camel, snake, dotted etc) at first glance, so this may end up defining a lot of potentially questionable commands that might result in more conflicts?

I'm aware that the casing is determined by the csv. The current default csv includes many abbreviations that are traditionally cased differently, so this seems like it was intended to be used this way? Consider "wip". I have the need to use abbreviations with different casing for variable names in code, so I would like this to work.

I think this is possibly duplicate (and more limited) behaviour of PR #636 which looks like it could be pushed along instead? fwiw I haven't tested either PR to confirm, as I'm just poking around on mobile, so maybe missing something.

🤔 maybe so. Unfortunately it seems that that pull request has conflicts now, so I'm not really sure how to push it forward.

🤔 maybe so. Unfortunately it seems that that pull request has conflicts now, so I'm not really sure how to push it forward.

np. I'll take a look to see if I can sort it out.

We discussed this in the community meet up. You should probably use the capture <user.formatters> instead of the list. Also we feel that we should probably move abbreviation and this new command to text.talon.


Add Support for new Software Types

This adds support to query by Software Types: asa, fmc, ftd and fxos

Adding support for additional software types.

Review for code style with the new endpoints added for GET /OS_version/OS_data and GET /platforms


VPAT-768 Added labels to regions in search grid

 VPAT-768 | FMG | All browsers | Forms Page | Make appropriate label to forms table role=navigation


Unrecoverable config error when a module [no longer] exists.

Based on #1119 - Mikaela brought up a valid point. What if users upgrade and their configuration is not up to date?
I put the idea to test, and was confident ZNC would handle this manner in a clean and elegant way, perhaps something like "This module [no longer] seems to exist, ignoring this config entry". Perhaps with a prompt "Start ZNC anyways?" and a --ignore-missing-modules flag.
[ .. ] Checking for list of available modules...
[ >> ] ok
[ .. ] Opening config [/home/zarthus/.znc/configs/znc.conf]...
[ >> ] ok
[ .. ] Loading global module [webadmin]...
[ >> ] [/home/zarthus/znc-build/lib/znc/webadmin.so]
[ .. ] Binding to port [+1111]...
[ >> ] ok
[ ** ] Loading user [zarthus]
[ ** ] Loading network [freenode]
[ .. ] Loading network module [garbage_module]...
[ !! ] Unable to find module [garbage_module]
[ ** ] Unable to find module [garbage_module]
[ ** ] Unrecoverable config error.

This would essentially solve upgrading issues with module changes from refusing znc to start up.
I don't think this should alter the config, just ignore the entries that ZNC cannot make sense of, or are refusing to load because they don't exist. A warning should be thrown, yes, but I do not believe it should be the be-all-end-all of starting up ZNC. Especially when considering upgrading, and that it does not seem to be that there are tools.
Most users would probably prefer a way to ignore the warning or have ZNC sort it out, than to manually edit the possibly overwhelming config of ZNC.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/26917044-unrecoverable-config-error-when-a-module-no-longer-exists?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github).

Associated conversation in #znc-dev:
(11:01:41) <Zarthus> >When I upgrade from 1.4 to 1.7.0 or newer, will my ZNC start without issues or will it die to missing awaynick module?
(11:01:50) <Zarthus> I haven't tested, but if that makes znc straight up die I'd be surprised
(11:02:22) <Zarthus> let's see if i can mimic a config entry
(11:02:58) <Mikaela> at least in the past ZNC has died on start to not being able to load some module specified in config file.
(11:03:19) <Zarthus> that sounds more like a critical bug than something this pull request should worry about
(11:03:59) <Zarthus> but you're right
(11:04:00) <Zarthus> https://gist.github.com/Zarthus/3ae7314c2836b52e8549
(11:04:44) <Mikaela> and I believe this is why the boilerplate code was added.
(11:05:40) <Mikaela> I have seen requested on #znc for flag to ignore modules that aren't found which makes sense, but I don't want it default in case someone fails with compiling ZNC somehow and compiles it without modules, it's not nice if all modules of all users are unloaded

perhaps with a prompt "Start ZNC anyways?"

How would this work with systemd unit or crontab? I prefer the other suggestion

and a --ignore-missing-modules flag

which could be mentioned in znc instead of the prompt.
Also my concern as I said on #znc-dev is I have seen requested on #znc for flag to ignore modules that aren't found which makes sense, but I don't want it default in case someone fails with compiling ZNC somehow and compiles it without modules, it's not nice if all modules of all users are unloaded.
And to add to my worry, it would be even worse for ZNC to write config file with no modules.

How would this work with systemd unit or crontab? I prefer the other suggestion

I'm not sure if there's a solid way to detect if it's a user running the command or not, but the way I see it it's either prompting the user (which a lot of software does, separately offering a --no-interaction or --assume-yes flag (see --ignore-missing-modules in this case). Or ZNC flat out just erroring. In either case, ZNC would not start and the sysadmin would have to find out why.

which could be mentioned in znc instead of the prompt.

Do you mean the command znc? Are you expecting something like consider running znc with --ignore-missing-modules? I'm not entirely sure what you mean with this.
The way I see it, it's currently failing, and will still fail with my suggestion unless you explicitly tell it otherwise.

I mean instead of a prompt Start ZNC anyway? mention that it's possible to run ZNC with --ignore-missing-modules.

I'm not sure if there's a solid way to detect if it's a user running the command or not,

As far as I am aware there isn't and I fear cron would launch multiple instances of the prompt causing issues with resource usage in longer run.
Adding new flags for not running interactively is a big NO!, because people have always used crontabs and different initscripts and this change would break them. If there must be a new flag, make it --interactive instead, that wouldn't break anything.

I'd probably argue the current behavior is the best behavior.  This way, when a configuration file is loaded, the least amount of functionality is lost and there's no surprises.  This is especially important in the case of third-party modules being used.  If ZNC were to simply start up and skip the modules it can't load, that would reduce the amount of information in the configuration file.
That said, I wouldn't be opposed to a CLI argument to change this behavior.  The use of such an argument would also mean the user would have to pass it explicitly, and thus be fully aware of the implications.
If implemented, I might also propose adding a warning about how the LoadModule entries would be removed from the configuration file when ZNC writes to it again as it periodically does.  When that happens, ZNC would no longer attempt to load those modules the next time it starts up.
Ben (MrC @ Freenode)

Perhaps with a prompt "Start ZNC anyways?"

I'm thinking of a poor sysadmin booting up an upgraded znc instance with a couple of hundreds of users. When would the  "Start ZNC anyway?" prompt appear? After every error? -Frustrating. Only after first error? -Don't know what's coming up. After parsing the whole config?

Making it explicit, something like --ignore-module awaynick would be less error-prone ie. no accidentally lost modules.

(21:18:42) <Zarthus> i'm considering two alternative appproaches
(21:19:43) <Zarthus> the first one is rather simple: if something gets deprecated, do something about it on bootup, and not on the LoadModule function
(21:20:19) <Zarthus> the second is some sort of config parser that receives an older version and a newer one and checks for possibly irrelevant things (like a LoadModule awaynick statement)
(21:20:59) <Zarthus> the problem with both of these is that it does not really "fix" the unrecoverable config error, but I'm starting to see more and more why it's unrecoverable to begin with
(21:21:55) <Zarthus> it seems to be the best approach there is to letting the user know znc needs help fixing something and is unable to determine the next step
(21:22:32) <Mikaela> while not becomimg interactive with cron & init scripts
(21:23:11) <Zarthus> i'm thinking of something that hooks in make install to parse the configuration then
(21:23:35) <Mikaela> and distribution packages?
(21:24:04) <MrC> And there might not be any config file before, or after "make install".
(21:24:10) <Zarthus> doesn't have to be make install, but znc could detect if it's the first run since an upgrade
(21:24:12) <Mikaela> and global installs too
(21:24:38) <Zarthus> we don't really care about new installs, they'll have an up-to-date config
(21:24:48) <Mikaela> first run after upgrade is detected as it starts by backing up the config file
(21:25:27) <Zarthus> great, so then perhaps we can move some of the legacy-loadmodule-logic into there and *fix* it rather than work around it
(21:25:55) <Zarthus> it might even be an idea to allow for znc administrators to run a 'verify-config' command of some sort to do it interactively while znc is running

Making it explicit, something like --ignore-module awaynick would be less error-prone ie. no accidentally lost modules.

And with this the config file would be written without awaynick? Sounds good to me except that it seems to only accept single module.
How about e.g. --ignore-modules awaynick q nickserv?


[Feature Request] Rotate around non-center point

Downstream issue: https://github.com/streetcomplete/StreetComplete/issues/3101
In StreetComplete, when a quest form or "add note" form is open, the map is partially obscured, so the visible center is not the same as the screen center. In this case, operations should take place around this offset center, not the screen center. StreetComplete has some workarounds to enable this, but rotating the map to face north (when the user presses the compass button) was not easily possible:

https://github.com/streetcomplete/StreetComplete/pull/3122
It would be great if there were a way to set a different center for rotation (or movement!) events. I imagine this would be added to the CameraUpdate, but I'm not familiar enough with Tangram to say for sure.

So, basically the feature request ist to be able to define a pivot-point around which the camera update is rotated (and zoomed)

This would be very helpful for my application, too. Actually, it would be helpful if a margin could be defined that would shift the map to avoid the region at the current center from being obstructed by UI elements that might move or expand.

This is in-progress. The basic idea is to add a method on the map view of the form:
setPadding(int left, int top, int right, int bottom)

The "padding" values will determine the relative position of the map's center position in the view.
This functionality is working in the desktop app in my current branch, and I hope to get this into the mobile SDKs in the next few days.

Thanks, @matteblair! Amazing work, as always!

Resolved by https://github.com/tangrams/tangram-es/pull/2294


Music organization (kog patch 20)

readme created audio folders
rearranged the audio files
changed readme for general instructions of the game

It is.

I've fixed the readme title


[Proposal] RadioButton

From @jamesmontemagno on December 23, 2016 7:7
Details coming soon
Copied from original issue: FormsCommunityToolkit/Controls#2

I'd like to contribute.

Any help needed?

This project looks so dead.

If you would like to contribute then submit a PR

I wish they ever accept your PR...

Huh? There's only three PRs. The oldest is fine months but it is waiting for changes

@skendrot I wish you were here last July. I was really into submitting a PR, but getting through all the testing and effort with getting no response held me down. Unfortunately now I'm not on a XF project anymore. But hopefully I'll get back.
At least it's nice to see that this essential project is getting the basic maintenance and responsiveness it deserves. Thank you!

We're looking to get an official 1.0 together primarily based off of what currently exists in the project, but for a vNext/1.1, if we start adding in controls, a radio button might be worth looking into assuming a proper spec.

I've created a (Xamarin.Forms) control which is RadioButton-ish, which suited my need, and may be useful to other, but isn't quite a proper RadioButton.
Best explained with an example.  A person record has a gender field which is stored as "M" or "F".  I wanted a control the would display "Male" and "Female" as buttons, but bind "M" or "F" to the Value:
 <jmc:RadioButton  ItemSource="M=Male;F=Female" Value={binding Gender}"   />

in another place in the app, i use:
 <jmc:RadioButton  ItemSource="L=Leading;S=Supporting;F=Featured" Value={binding RoleType}" />

It works nicely in places where you'd otherwise have to use a Switch or a Picker.
Would there be any interest in that in the library??

@jamescurran Is there a limit of the number of buttons your control can handle? If not, it sounds interesting...

I put no arbitrary limit on the number of buttons.  Since I'm just stuffing them into a horizontal StackLayout, I imagine that there's is a practical limit somewhere.

RadioButton is now a control in Xamarin.Forms, this is therefore outdated


ADD: Type annotations for timing.js

fixed the type errors and type annotations for timing.js . Also tested with yarn run flow with no error.

Thanks @mayank4kathuria for taking a shot at this!

I'm going to ignore on this PR the first few commits, with setup docs -- let's discuss those on #2107.
Your diffs make a lot of extra changes because they add DOS-style, "CRLF" line endings. Please set up your editor so it doesn't do this; if you ask in the #mobile stream, we can help you figure out how.
Run yarn run test:lint to see the issues it flags; this includes the line endings (so it will help you make sure you've fixed that), and it finds one small other issue. We keep our lint error-free.
Please read our guide on writing clear, coherent commits. All these changes (again, ignoring the setup instructions) should be one commit; and take a look at what the guide says about the summary line of your commit message.
There's no need to write types redundantly, like repeating (key: string) for the variable right next to where it appears for the arrow function itself.
Generally I don't think we need to write void as a return type when the function definition is right there; it doesn't add much for the reader.
Adding the type number for what now returns is helpful, though! I'd be glad to merge a change that added that.
Generally the type Object doesn't add much information, or even any at all. What would be useful on timingMap and countMap is to add a type that says what types the keys and values in these maps should be -- see the Flow docs for how.

Would you try revising this PR with these points in mind? Thanks again!

@gnprice for the CLRF thing, we can use the zulip.get .gitattributes file, which prevents that from happening to people trying to develop on Windows.

@timabbott Yeah, good thought -- done, as 58c70ecc.

@gnprice  Sorry for the rookie mistake it was my first time making a PR and Thanks for the advice, will keep those in mind from my next PR onward.
i will submit another PR with all the mentioned points for the same file.

Cool, thanks @mayank4kathuria
I'll close this one then, and waiting for the new one 👍


nixos/sx: init

Motivation for this change
sx is a simple alternative to both xinit and startx for starting an Xorg server.
https://github.com/Earnestly/sx
this module is largely based on services.xserver.displayManager.startx
Things done

[x] 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 packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[x] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Not an Xorg user, not sure why you are requesting my review.

my bad, it was recommendded

resolved merge conflicts

resolved merge conflicts


BAEL-2901 Composite primary keys in JPA

Hi @jzheaux - I have formatted the code using the recommended 'formatter.xml' but I am unable to login to JIRA for some reason. Please review it.


Add Snowpack to build and enable full TypeScript strict mode

I was very disappointed with the fact that we had no types for React and were therefore not able to enable full TypeScript strict mode.
By adding SnowPack (which is definitely hot shit)

we can import all node_modules dependencies via /web_modules
they are pre-bundled for more efficiency
we can have CJS dependencies that are converted to ESM for us
it is finally possible to add actual types by using TypeScript "paths"

I will fix our test setup... :) Will take a moment, though

@andreashille I fixed the test setup, moved it from babel-jest to ts-jest and prepared it for using react testing library, please have another look.

On my machine, it can't even be installed. npm ci complains that the lock file is out of date and npm install failes with
npm ERR! Invalid dependency type requested: alias

On my machine (Linux) npm ci and npm install both fail.
npm ci complains that the lock file is out of date.
npm install fails with npm ERR! Invalid dependency type requested: alias


Does 'luks-setup.sh` can create password for multiple partitions?

I do not recall if password stored in tpm can be just one or many and was wondering if this use case which is in the subject is possible to achieve with the script, meaning I have 3 partitions which I would like to make encrypted.

Do you need to set a corresponding key for each partition? If so, the current luks-setup.sh doesn't support it. But you can patch this script, pass --key-slot argument to cryptsetup to specify different slot to store different key.
Here are some references from cryptsetup man page[1][2][3]:
--key-slot, -S <0-N>
For LUKS operations that add key material, this option allows
you to specify which key slot is selected for the new key.
--new-key-slot <0-N>
This option allows you to specify which key slot is selected
for the new key.
[1] https://man7.org/linux/man-pages/man8/cryptsetup-luksformat.8.html
[2] https://man7.org/linux/man-pages/man8/cryptsetup-luksOpen.8.html
[3] https://man7.org/linux/man-pages/man8/cryptsetup-luksAddKey.8.html

Hi @yizhao1,
thx for quick response and sorry for my long but I was on holidays.

Do you need to set a corresponding key for each partition? If so, the current luks-setup.sh doesn't support it. But you can patch this script, pass --key-slot argument to cryptsetup to specify different slot to store different key.

Yes that is what  I need it for. I can give a shot :)

Here are some references from cryptsetup man page[1][2][3]: --key-slot, -S <0-N> For LUKS operations that add key material, this option allows you to specify which key slot is selected for the new key.
--new-key-slot <0-N> This option allows you to specify which key slot is selected for the new key.
[1] https://man7.org/linux/man-pages/man8/cryptsetup-luksformat.8.html [2] https://man7.org/linux/man-pages/man8/cryptsetup-luksOpen.8.html [3] https://man7.org/linux/man-pages/man8/cryptsetup-luksAddKey.8.html

Thx for informations.

Hi @yizhao1, thx for quick response and sorry for my long but I was on holidays.

Do you need to set a corresponding key for each partition? If so, the current luks-setup.sh doesn't support it. But you can patch this script, pass --key-slot argument to cryptsetup to specify different slot to store different key.

Yes that is what I need it for.  I was also thinking if it is worth doing? Isn't it sufficient to use just one key for all the partitions? If so than how can I re-use the key already in the TPM?

It is possible to use one key for all partitions. But it requires some tricks when using luks-setup.sh. When creating the first encrypted partition, you need to pass the -e parameter to luks-setup.sh to evict the existing key in the tpm and create a new key. When creating subsequent encrypted partitions without the -e parameter, these partitions will use the current key in the tpm.
//Yi

Here are some references from cryptsetup man page[1][2][3]: --key-slot, -S <0-N> For LUKS operations that add key material, this option allows you to specify which key slot is selected for the new key.
--new-key-slot <0-N> This option allows you to specify which key slot is selected for the new key.
[1] https://man7.org/linux/man-pages/man8/cryptsetup-luksformat.8.html [2] https://man7.org/linux/man-pages/man8/cryptsetup-luksOpen.8.html [3] https://man7.org/linux/man-pages/man8/cryptsetup-luksAddKey.8.html

Thx for informations.

Thanks, this is working.
I have one more question for you but it concerns another thread Could you please answer there?

Thanks, this is working.
I have one more question for you but it concerns another thread. Could you please answer there?

OK no need to answer in that thread. Actually I have forgotten a lot and thx to this blog I have recalled that PCR7 is responsible for Secure Boot State, so I turned it on in UEFI and it started to work...


Fix error thrown when using count(*) in ORDER BY clause of joins

Codecov Report

Merging #5372 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #5372      +/-   ##
============================================
+ Coverage      82.4%   82.42%   +0.01%
- Complexity    11256    11258       +2
============================================
  Files          1355     1355
  Lines         44132    44133       +1
  Branches       5560     5560
============================================
+ Hits          36367    36376       +9
+ Misses         5143     5142       -1
+ Partials       2622     2615       -7

Impacted Files
Coverage Δ
Complexity Δ

...a/io/crate/analyze/relations/RelationSplitter.java
96.66% <100%> (+0.02%)
33 <0> (+1)
:arrow_up:

...arch/indices/recovery/BlobStartPrefixResponse.java
66.66% <0%> (-33.34%)
3% <0%> (-2%)

...in/java/io/crate/lucene/match/CrateRegexQuery.java
43.47% <0%> (-30.44%)
4% <0%> (-4%)

core/src/main/java/io/crate/types/ArrayType.java
73.84% <0%> (-7.7%)
24% <0%> (-2%)

.../elasticsearch/action/bulk/BulkShardProcessor.java
72.46% <0%> (-4.35%)
44% <0%> (-3%)

...te/blob/pending_transfer/PutHeadChunkRunnable.java
76.71% <0%> (-4.11%)
12% <0%> (-2%)

...o/crate/planner/node/dql/join/NestedLoopPhase.java
74.69% <0%> (-3.62%)
20% <0%> (-2%)

...in/java/io/crate/breaker/RamAccountingContext.java
80% <0%> (-3.34%)
19% <0%> (-2%)

...ration/collect/collectors/LuceneBatchIterator.java
86.95% <0%> (-3.27%)
36% <0%> (-1%)

.../operation/udf/DropUserDefinedFunctionRequest.java
91.42% <0%> (-2.86%)
10% <0%> (-1%)

... and 27 more

Continue to review full report at Codecov.

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


Update LinkedIn URL

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Kindly update the title as data:<your-github-username>


Elliptical Arc can I disable in SVG out? Can I disable repeat end points?

Have been working on a WebGL contour triangle engine so was excited when I found your library as it  seems ideal to generate path data I can pass to my engine for testing so tried using  Dan's wrapper https://danmarshall.github.io/google-font-to-svg-path/ to create some svg.
My problem is that I have not yet implemented Elliptical Arcs and I wanted to be sure I needed them and for fills they maybe a problem.
https://github.com/nanjizal/Trilateral/blob/master/src/trilateral/justPath/SvgPath.hx#L160
Is it possible with your library to only output bezier curves and avoid Elliptical from the svg output, I could not work out where to modify that, I could not see it in your Path code, and I could not see it in Danmarshall's code which seems mostly a wrapper round yours?
Another issue I am having is that poly2trihx fill library seems unhappy about repeated points which happens at the end of a letters, I can extract them quite easily if I don't join letters together... but if I do then it's more tricky. Is there an option to disable that or do I need to deal with that?
The elliptical arc means that if I render the fills with hxPolyK at moment the 'o' does not get rendered at all, polyK does not deal with holes but seems more tolerant, the white outline is my Trilateral library still WIP, I am using haxe Kha cross toolkit but have similar demos working direct with haxe js WebGL, the clip is from the browser.

( In case your curious this is my current classes, you can switch to FillOnly from Fine if I just need fill data.
https://gist.github.com/nanjizal/b9909a6ed84bc8805e2e439e0ee7cc9a )
I realize it's not that practical rendering text as triangles, but feasible for a few letters.

Hi @nanjizal - this is probably not an opentype.js issue. In my code, I optimize for arcs. Can you close the issue here and open a new one at https://github.com/danmarshall/google-font-to-svg-path ? 👍

oh ok sorry Dan I did look through your code and mostly it seemed to use two other MIT projects


Add form sign up option in Engagement view

Issue #: https://github.com/bcgov/met-public/issues/2151
Description of changes:

refactor and break up the code in subscribe widget view to make it more maintainable
use Rich Text Editor to view the rich text
Link the form sign up to the cacform page

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met-public license (Apache 2.0).

Codecov Report

Merging #2153 (54f44dd) into main (c411991) will increase coverage by 0.02%.
The diff coverage is 47.05%.

@@            Coverage Diff             @@
##             main    #2153      +/-   ##
==========================================
+ Coverage   68.90%   68.93%   +0.02%
==========================================
  Files         463      466       +3
  Lines       15011    15044      +33
  Branches     1178     1177       -1
==========================================
+ Hits        10344    10371      +27
- Misses       4465     4472       +7
+ Partials      202      201       -1

Flag
Coverage Δ

metweb
61.45% <47.05%> (+0.08%)
:arrow_up:

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

Files Changed
Coverage Δ

...gagement/view/widgets/Subscribe/EmailListModal.tsx
31.11% <31.11%> (ø)

...agement/view/widgets/Subscribe/SubscribeWidget.tsx
42.42% <63.63%> (+15.39%)
:arrow_up:

...gement/view/widgets/Subscribe/EmailListSection.tsx
64.28% <64.28%> (ø)

...ement/view/widgets/Subscribe/FormSignUpSection.tsx
66.66% <66.66%> (ø)


Collapse all / expand level / expand all buttons?

Soulseek has this functionality when browsing a library, where everything is expanded by default but there are buttons that collapse all to see just the root level then progressively 'unfold' the menu.

Hi @mxmilkb!
Collapse & expand content on what page?
This repo already has userscripts for:

collapsing markdown by header
repo files filter
code folding
collapsing long code/quote blocks
diff files filter
toggle expanders like on the commits page

Please share a link to the page with content to which you are referring.

Oop, sorry, the ToC plugin

Oh, you can already do that...

Shift+ Click on the arrow next to the anchor to toggle all same level items.

from the docs - https://github.com/Mottie/GitHub-userscripts/wiki/GitHub-table-of-contents

Maybe not exactly what you're looking for, but it's served me really well. I am open to feedback on how it could be improved.

I'm guessing this issue has been resolved, so I'm going to close it. If you continue to have problems, please feel free to continue the discussion in this thread.


Diff Two Arrays Challenge passed using wrong results.

Diff Two Arrays

https://www.freecodecamp.com/challenges/diff-two-arrays
Issue Description

Wrong array consisting of excessive numbers of correct solution values being accepted as right.
Upon entering a wrong combination of .filter() and a for loop methods, which returns incorrect array (code below), all the tests and the challenge as a whole are considered successfully passed.
e.g. the first set of test arrays should return [4] and returns [4,4,4,4,4], which is of course something completely different, but still is accepted as a correct solution.
Browser Information

Browser Name, Version: Firefox51.0.1 (32-bit)
Operating System: Ubuntu 14.04
Mobile, Desktop, or Tablet:  Desktop

Your Code

function diffArray(arr1, arr2) {
  var newArr = [];
  arr1.filter(function(x) {
    for (var i=0; i<arr1.length; i++) {
      if (arr2.indexOf(arr1[i]) < 0) {
        newArr.push(arr1[i]);
      }
    }
  });

  arr2.filter(function(x) {
    for (var i=0; i<arr2.length; i++) {
      if (arr1.indexOf(arr2[i]) < 0) {
        newArr.push(arr2[i]);
      }
    }
  });

  console.log(newArr);
  return newArr;

}

diffArray([1, 2, 3, 5], [1, 2, 3, 4, 5]);

Screenshot

@achudoz thanks for the issue. This challenge's tests have been updated recently, which solve the issue you've brought up. You can check them out on the beta site for yourself. Closing as resolved in staging. Happy coding!


No longer raises an error if a map can not be encoded in JSON

Logging raising errors is problematic - it can mask real errors or break things if the logging level is changed.
Some things are not (by default) encodable with Jason. This falls back to using inspect/1 should a Map not be encodable using the string formatter.

Thanks @paulanthonywilson, version 1.0.2 published with this fix.

🎊


[FEA] Support max on single-level struct in aggregation context

Is your feature request related to a problem? Please describe.
I wish the plugin can support Max on Single-struct in aggregation context.
Additional context
Code example:
import org.apache.spark.sql.Row
import org.apache.spark.sql.types._

val data = Seq(
    Row(Row("Adam ","","Green"),1,"M",10,"Math:13,Gym:24,English:45",Map("hair"->"black","eye"->"black"),"{\"hair\":\"black\",\"eye\":\"black\"}"),
    Row(Row("Bob ","Middle","Green"),2,"M",20,"Math:55,Gym:24,English:37",Map("hair"->"yellow","eye"->"yellow"),"{\"hair\":\"green\",\"eye\":\"green\"}"),
    Row(Row("Cathy ","","Green"),3,"F",30,"Math:83,Gym:15,English:63",Map("hair"->"blue","eye"->"blue"),"{\"hair\":\"blue\",\"eye\":\"blue\"}")
)

val schema = (new StructType()
  .add("name",new StructType()
    .add("firstname",StringType)
    .add("middlename",StringType)
    .add("lastname",StringType))
  .add("low",IntegerType)
  .add("gender",StringType)
  .add("high",IntegerType)
  .add("score",StringType)
  .add("feature",MapType(StringType,StringType))
  .add("feature_json",StringType))

val df = spark.createDataFrame(spark.sparkContext.parallelize(data),schema)
df.write.format("parquet").mode("overwrite").save("/tmp/yl/tmpdatas")

val df2 = spark.read.parquet("/tmp/yl/tmpdatas")
df2.createOrReplaceTempView("df")

spark.sql("SELECT max(name) from df").show()

The example above is a reduction context. Currently on track of development, targetting for 22.02 release.


docs(zh-cn): update  github-pages.md

Check List
Please read and check followings before submitting a PR.

[ ] I want to publish my theme on Hexo official website.

[ ] I have read the theme publishing doc.
[ ] name is unique.
[ ] link URL is correct.
[ ] preview URL is correct.
[ ] preview URL web site is rendered correctly.
[ ] Add a screenshot to source/themes/screenshots.
[ ] Screenshot filename is same as value of name.
[ ] Screenshot size is 800 * 500.
[ ] Screenshot file format is png.

[ ] I want to publish my plugin on Hexo official website.

[ ] I have read the plugin publishing doc.
[ ] name is unique.
[ ] link URL is correct.

[ ] Others (Update, fix, translation, etc...)

Languages:
[ ] en English
[ ] ko Korean
[ ] pt-br Brazilian Portuguese
[ ] ru Russian
[ ] th Thai
[x] zh-cn simplified Chinese
[ ] zh-tw traditional Chinese

zh-ch docs keep up with  zh-tw;


Unable to activate any plugins "No plugin specified, or not a valid plugin"

Describe the bug
Currently running 1.7.2 and cannot activate any plugins. Even stock sample plugins.
Error "No plugin specified, or not a valid plugin"
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.
Environment (please complete the following information):

YOURLS version: 1.7.2

Plugins enabled: None

PHP version: 5.6
Other relevant details depending on the context (server OS & version, MySQL version, browser version, …)

Is it April First? Cause this seems like a joke!

No, this is not a joke, I cannot activate any plugins at all...

Apache or Nginx?
This could be the server block or .htaccess
Program Transfer Coruption
It seems the program is not finding your plugin directory.
You might want to download a NEW copy of YOURLS and then upload the program files again (overwrite).
I have in the past downloaded files that were corrupted along the way.  I have also uploaded files that were corrupted along the way.
/user/ directory
What directory is your /user/ directory in? Is this your DocumentRoot (Apache) or Webroot (Nginx)? Or is it in a subdirectory like /s/ or /e/ or /y/?

I have the same issue with Reset URLs plugin, but another plugins (like: Random Backgrounds, Random Keywords, ...) can be activated/deactivated successfully.
PHP version: 7.2
MySQL version: 8.0
Server/OS: IIS7 / Microsoft windows server 2008 r2

@AhmadKahlid If other plugins can be activated successfully then the issue is different. Be sure to put the plugin.php is a directory named with a simple chars and no space (eg. plugins/reset-urls/plugin.php).

Closing as inactive.


--disable-secret-rotation not a valid flag

[Enter feedback here]
I tried to disable autorotation and that is not an option
az aks addon update -g rg-scott -n aks-scott-akv -a azure-keyvault-secrets-provider --disable-secret-rotation
unrecognized arguments: --disable-secret-rotation

az version
{
  "azure-cli": "2.38.0",
  "azure-cli-core": "2.38.0",
  "azure-cli-telemetry": "1.0.6",
  "extensions": {
    "account": "0.2.2",
    "aks-preview": "0.5.91",
    "resource-graph": "2.1.0",
    "ssh": "1.1.0"
  }
}

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

ID: 409a3be9-4831-9401-0bb3-c65209bb3f42
Version Independent ID: 4cf81213-5989-f36c-7456-70accfe452c2
Content: Use the Azure Key Vault Provider for Secrets Store CSI Driver for Azure Kubernetes Service secrets - Azure Kubernetes Service
Content Source: articles/aks/csi-secrets-store-driver.md
Service: container-service
GitHub Login: @Nickomang
Microsoft Alias: nickoman

@rscottwatson Thank you for bringing this to our attention!  We are reviewing the documentation and will get back to you shortly.

@rscottwatson Thank you very much for bringing this to our attention!  We've updated the instructions for disabling autorotation:

To disable autorotation, first disable the addon. Then, re-enable the addon without the enable-secret-rotation flag.

Thank you again for helping to improve the documentation!  We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

Won't that affect the managed identity? If I granted permissions to the MI won't a new MI get created when the addon is enabled again which will obviously not have any permissions to access any vaults?

@rscottwatson Thank you for pointing this out.  I've looped in the document author and we are following up.


creating a new season doesn't add new rounds in round table

This breaks the series, matchups, standings pages on new divisions. We have a stopgap and we are all good for this season now, but we will want to fix this for future seasons

#172


[kugo] ril reports errors for oem commands

Seems transpower successfully detects device state and informs ril, only the latter returns an error, as seen in logcat -b radio:
05-09 13:40:44.230  2826  2826 E RILQ    : (0/2826): RIL[0][rild] qmi_ril_parse_oem_hook_header: Received un-handled/oem command id = 524489

ping @jzoran @jerpelea

Looking into this.

those errors can be ignored


Deleting a subscription feed on the current macOS version actually deletes the feed below it

Highlight an article; choose "Select  in sidebar..." from the secondary menu. Press the delete key or choose to delete the feed from the secondary menu and after confirmation, the feed below it disappears.

What version of the app are you running?
What type of account (On My Mac, iCloud, Feedbin, etc.) are you using? Do you have multiple accounts?
(I’m not able to reproduce this, so I’m looking for more information.)


Restructure output to be JSON

API Response:
keys: alt-text, long-description, structured-text
structured-text: return JSON with individual levels. Each level should be structured with nested json.
i.e.
{ level1: {
header1: {
... }
}
}

etc

@elizaan and I reviewed this today; what we probably want is structure but not at a "level" level.
Here's how this should look, and we want to possibly also format this information:
=====
UpSet Introduction
This is an UpSet plot that visualizes set intersection. To learn about UpSet plots, visit upset.app.
Dataset Properties
The dataset shows attributes of characters from the Simpsons. It contains 6 sets, and 44 elements, of which 6 sets are shown in the plot.
Set Properties
The largest set is Male, with 8 characters, followed by Duff Fan with 6, Evil with 6, School with 6, Power Plant with 5, and Blue Hair with 3.
Intersection Properties
The plot is sorted by size. There are 12 non-empty intersections, all of which are shown in the plot. The largest 5 intersections are School & Male (3 characters), the empty intersection (3), just Male (3), Duff Fan, Male & Power Plant (3), and Evil & Male (2).
Statistical Information
The average intersection size is z, the median is q. The 90th percentile is r, and the 10th percentile is q. The largest set, Male, is present in x% of all non-empty intersections. The smallest set Blue Hair is present in y% of all intersections.
=====
In addition, we want to add a single, very short alt-text line.
Should the text from above be just in one field and be formatted e.g., with Markdown?

This looks like a good change!! 😸

This looks like a good change! 👯♂ 😸


Fix random speed generation in type constructors

Defines a new rand_speed(motility) function to extract speed from motility types
Extends rand_vel to generate a velocity vector with speed defined by the distribution in the motility type
Fixes bugs in type constructors

Closes #33

Failed tests are due to floating point errors in the test; will need to fix the tests but this can be merged


Add 1×1 TinyNutrientDispenser.png

Add 1×1 TinyNutrientDispenser.png for preview, north, south, west, east and upload them on the retexture-branch onto the main branch. And then do a merge request to CLV-TinyWorkbenchs branch: main.

see https://github.com/emipa606/TinyWorkbenchs/pull/2


Infusion Tab Crashing BETA17

Hello, as you can tell from the title when ever I try to click on the infusion tab my entire game crashes. Has anyone encountered the same problem and managed to fix it? or can you tell me what could possibly be going wrong with my game? any support would be greatly apreciated.
crash-2018-07-09_11.25.08-client.txt

Update CTM to <IP_ADDRESS>, it's a known compatibility problem between CTM and newer forge versions and has nothing todo with Thaumcraft....
https://github.com/Chisel-Team/ConnectedTexturesMod/issues/63

Thanks, that fixed it so I'm good now.

please close the issue if it is fixed.


MCMMO Right click ability

MCMMO right click ability doesn't work with sword blocking module enabled

On my factions server, when using 1.9 - 1.12 client and right clicking with your sword, a shield appear in your offhand which makes McMMO ability "Serrated Strikes" unable to activate.

Using protocolsupport plugin for the multi version client support. Not sure if that helps or not

Thank you for the information. What version of the plugin are you using?

1.5.0

Mcmmo. Is 1.5.08

I have tried testing this and managed to get "Serrated Strikes" to work. I am using OCM 1.5.1 and the latest McMMO from here.
Can you upload a copy of your OCM config.yml to pastebin or similar service?
Could you provide more detail on what exact part of the ability doesn't work? Do you get the "ability activated" message but it then doesn't do anything?

the bug was given to me by a player on my network, i have no done extensive testing.
"Right clicking a sword will make a shield appear in your off-hand slot making 'Serrated Strikes' (McMMO) impossible"
This is the OCM config, perhaps there is a setting i need to change?
https://hastebin.com/kenanenili.bash

perhaps its protocolsupport blocking it?

@gvlfm78 Anything new about this issue?

I told you, serrated strikes works perfectly fine for me, so until someone documents a way in which it doesn't work I can't do anything, as I'd have no clue what the problem is

At the moment when shields are enabled, the only way to activate serrated strikes is through shift right clicking rather than just right clicking.

@KuraiKokoro Does that also happen when you're holding a shield in a server without OCM? i.e. is holding the shield the problem?

Just went ahead and disabled OCM, holding the shield is the problem.

I just gave up trying to use OCM due to the two major bugs we were having with it. .-.

@Ac3dUd3 @ridalarry @RainOfPain125 @KuraiKokoro To clarify, on a normal 1.9+ server with just McMMO, is this still an issue when using shields in a normal manner?


Allow to render diagrams from file

I want to render really big diagrams (BPMN, which is XML), so it's handy to put diagram code in separate file and link it from docs.

Hello @tsouvarev ,
thanks for your PR, I appreciate your effort and looking forward to integrate your extension.
It would be helpful for the users if the is something for troubleshooting/logging, when the file does not exists.
Could you please add some logging and maybe also some hint which will be rendered into to docs.

Sure, that makes sense. I will look into it.19 авг. 2022 г. 15:30 пользователь Benjamin Bittner @.***> написал:
Hello @tsouvarev ,
thanks for your PR, I appreciate your effort and looking forward to integrate your extension.
It would be helpful for the users if the is something for troubleshooting/logging, when the file does not exists.
Could you please add some logging and maybe also some hint which will be rendered into to docs.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

I've added info message before reading file and error if operation didn't succeed

Hello @tsouvarev ,
sorry for the long silence, the last month were no time left for any other stuff than daily business.
Thanks for your added changes, looks much better now. One more thing I would like you to add before I can merge the PR:
Please add the mkdocs-parameter 'docs_dir' as base-path for the file-picking and also include this part of the path in the log message. I would guess, this would help others to find the problem when the plugin can't find the file.

@b-bittner thanks for your time and valuable comment! fixed that


How do I use the partial shown in the README?

Following the examples from http://handlebarsjs.com, it seems like {{#calendarDate}}{{this.date}}{{/calendarDate}} would work but I get "Invalid date" in the rendered HTML.

You probably aren't in need of this answer any more, but {{calendarDate "January 1, 1991"}} worked for me.

Haha, yup. Hexo frustrated me so I wrote my own blogging engine. I'll close this issue since it seems you've solved it.


Handling of imported PropTypes with info addon

Bug or support request summary
When importing PropTypes from another file, babel-plugin-react-docgen will not follow the import. Example:
Alert.js:
// ...
export const AlertShape = PropTypes.shape({
// ...
});

Alerts.js:
import { AlertShape } from './Alert.js';
// ...
Alerts.propTypes = {
    // ...
    alerts: PropTypes.arrayOf(AlertShape)
};

This will result in a "custom" type (within the arrayOf type) that looks like this:
{
    name: 'custom',
    raw: 'AlertShape'
}

The info addon will then render the name property in the PropTypes table like so: [custom], which is not very informative as to what data the array is supposed to contain.
Since the variable name of the original import is contained in the raw property next to the currently in use name, I think it would make sense to check if raw is present (if name is set to custom), since I expect it will almost always be more informative than just printing custom, thus rendering a more informative [AlertShape].
Could also make it this behaviour opt-in with a setting, of course.
Steps to reproduce
See code above, or any other example that imports PropTypes from another file.
Please specify which version of Storybook and optionally any affected addons that you're running

@storybook/react 3.4.8
@storybook/addon-info 3.4.8

Affected platforms
All
I can supply a PR if this is something that you think is reasonable and would not break any behaviour I haven't thought of :)

FYI, I opened a similar issue some months ago: https://github.com/storybooks/storybook/issues/3493
It seems that we can show the name of the imported type instead of custom. I didn't take the time to try it, sorry 😕
Of course, having the name of the type  would be better than custom but for my need it won't be enough 😞

@mlegait oh I see - and I thought I had searched well enough :) Thanks for letting me know!
From reading through your issue, it definitely seems to be a (possible) problem for many people, and something out of the hands of storybook itself.
I will try and prepare a PR for at least displaying the raw value for now, as it seems actually getting the import resolved might be a longer thing.. I will close this and use the other issue as reference, I don't think we need it twice :)


Too difficult to augment values which do not export their type

Let's take this file (react-router/lib/Route.d.ts):
import * as React from 'react';
import Router from './Router';
import { Location } from 'history';

declare const self: self.Route;
type self = self.Route;

declare namespace self {
    interface RouteProps extends React.Props<Route> {
        path?: Router.RoutePattern;
        // ...
    }
    interface Route extends React.ComponentClass<RouteProps> {}
    interface RouteElement extends React.ReactElement<RouteProps> {}
}

export default self;

Let's say that I wanted to write a module augmentation for this.
There is no way (that I can think of) to reopen the namespace or any of the interfaces.

Local namespace/interface merging is not allowed with imports.
There is no way I can reopen a default export in a module augmentation.

As a result, I have to rewrite react-router so that I can make it augmentable because the original author did not think far ahead enough to support it.

I should actually take back what I said in part - it is not so much that the author didn't think far enough ahead to support augmentations, but rather, that the module is available using a default export. The author wanted to export all of the types through the default export as well, but as mentioned above, there is no way to reopen default exports.

looks like a duplicate of https://github.com/Microsoft/TypeScript/issues/14080


space-cadet-pinball: unstable-2021-12-02 -> unstable-2021-12-29

Motivation for this change
Version bump
In release 2.0.1, the two lines in the cmakelists.txt file contained absolute paths. This was fixed in https://github.com/k4zmu2a/SpaceCadetPinball/pull/115. Alternatively we could stay on release 2.0.1 and just patch the file using substituteInPlace. I am unsure which is preferred.
Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

In release 2.0.1, the two lines in the cmakelists.txt file contained absolute paths. This was fixed in k4zmu2a/SpaceCadetPinball#115. Alternatively we could stay on release 2.0.1 and just patch the file using substituteInPlace. I am unsure which is preferred.

Generally in this case it would be better to point to release 2.0.1 and backport the patch fixing the issue. For example: https://github.com/NixOS/nixpkgs/blob/master/pkgs/applications/accessibility/squeekboard/default.nix#L41-L48
We can get the patch here: https://patch-diff.githubusercontent.com/raw/k4zmu2a/SpaceCadetPinball/pull/115.patch

In release 2.0.1, the two lines in the cmakelists.txt file contained absolute paths. This was fixed in k4zmu2a/SpaceCadetPinball#115. Alternatively we could stay on release 2.0.1 and just patch the file using substituteInPlace. I am unsure which is preferred.

Generally in this case it would be better to point to release 2.0.1 and backport the patch fixing the issue. For example: https://github.com/NixOS/nixpkgs/blob/master/pkgs/applications/accessibility/squeekboard/default.nix#L41-L48
We can get the patch here: https://patch-diff.githubusercontent.com/raw/k4zmu2a/SpaceCadetPinball/pull/115.patch

Result of nixpkgs-review pr 153013 run on x86_64-linux 1

1 package built:

space-cadet-pinball

Result of nixpkgs-review pr 153013 run on x86_64-linux 1

1 package built:

space-cadet-pinball


question: it is optional the use of minio?

In the case I want to keep using Ceph or the Ceph-less option from V1

I know in some components (like builder) we are using the minio client to communicate with the object storage but we have open issues to change that to the AWS Go SDK so we can plug in and use any object storage like S3 or ceph. See https://github.com/deis/builder/issues/80 :)

@aledbf, the mc client that we use can communicate with many object storage systems that support the Amazon S3 API, so Minio is optional as long as you have an S3 compatible object store. I've only tested with Minio and S3 as of now, but I understand Ceph supports the S3 API.
Finally, to expand on @bacongobbler's comment, deis/builder#80 will enable us to expand our support for S3 compatible object stores by supporting multiple signature versions.

@aledbf I'm going to close this for now. Can you please re-open or file a new issue if you have further questions here?

@arschles no need. I use minio just to pass the tgz between workflow and builder. I'm using a custom builder that end with a docker image, not a tgz and that solves my problems. Thanks

Great, thanks for letting us know @aledbf


Implement Files API

Implement dl-api's File functionality:
https://github.com/doubleleft/dl-api-android/blob/master/lib/dlapi/src/main/java/com/doubleleft/api/Files.java

The files API will be deprecated soon. :(


A Django webservice with user login

We need a Django webservice that supports user registration and login.

Done.


Duplicate CSS rules

Could the default CSS rules be extracted into the .iconify declaration?

cc @antfu

Currently, this is how Iconify generates those data. I don't consider this a very big issue as gzip should do the work already.
Also mention some of the trade-off if we going to do so:

Properties like width and height can be different for different icons.
Rendering mode can be different for different icons (here, use mask for most monochrome icons), but there are also colored icons where we need to use a background image (read https://antfu.me/posts/icons-in-pure-css)
.iconify is too general to be a common class, as it is also shared with SVG mode. We might introduce a .iconify-css and put the common props there. Which we also add that class name to every icon.

So, in the end, it might still be useful to have such optimization, but I would expect somewhat around ~10% size improvements after gzip - if that still matters to you, PR is greatly welcome :)


Remove timeout from AsyncSerial

Find a way to have a cancelled task cancel a blocking Queue read/write

Probably can do this either with an async generator (then use shutdown_asyncgens()) or by creating a new event loop that can be stopped.


Wagmi Connectors: useWalletClient not working

I'm successfully running the Wagmi Demo: packages/wagmi-test-demo
I'm doing a small modification and adding in App.tsx
const { data: client } = useWalletClient()

and changing the sendTransactionOnButtonPress to
const sendTransactionOnButtonPress = async () => {
    client?.sendTransaction({
        to: "0xe0a942ff2e1724A2fe10627728bE327a43fE8C26",
        value: 100n,
        chain: sepolia,
    });
};

I'm getting this error.
TransactionExecutionError: The method does not exist / is not available.

I think I find the underlying issue in @wagmi/core
https://github.com/wevm/wagmi/issues/3640

Hey! It seems to be working, closing this


Question API Design

Hey there,
Thanks for publishing easypwned under MIT. Great help for my own project.
However I have one question because of the design: Why do you send the whole hash or password to the API?
That would open the usage up to errors, because people do log things and logs are wrong configured every so often.

I know it's because of the bloom filter (if I get it right.) But the problems are still valid.

Thanks for the answer, I adhd´ed the issue /o\


Problem: mkdocs highlighted line range is incorrect

Solution: Fix the highlighted line range.
Test plan: Verified running mkdocs locally.
Screenshot:

Thank you!


fix get partitioned topics for non-persistent topics

Motivation
When calling getting the a list of partitioned topics for non-persistent topics.  The topic domain for all the topics return in the list is not correct.

rerun java8 tests and cpp tests

rerun java8 tests
rerun cpp tests

run java8 tests

rerun java8 tests

rerun java8 tests


Runtime for BunJS

BunJS is a new JS runtime that is really fast. I will be working on adding support for it in OpenRuntimes.

Oh! I didn't see that.


rollup-shader-chunks.mjs: simplify + fix template string bug

Fixes bug indirectly introduced via https://github.com/playcanvas/engine/pull/5602 (it wasn't a bug before because the filter was broken :sweat_smile: ):

The reasoning is quite simple, we used JSON.stringify to add " around the string, but that renders every occurance of ${...} useless afterwards (hence they landed in the build folder like " ... ${decode} ...".
Affects every build target:

Fixes examples: Paint Mesh + Reflection Box
@epreston If you have time, can you review?
I confirm I have read the contributing guidelines and signed the Contributor License Agreement.

JSON.stringify has always been used by this plugin, even before the change.  I'll take a look.

JSON.stringify has always been used by this plugin, even before the change. I'll take a look.

Yep, the trick is to include the backtick in the regex replace and simply keep it as it is dropping JSON.stringify, to not change the string kind (turning template string into normal string).

Only one fragment is using string template literal replacement.
src\scene\shader-lib\chunks\common\frag\reproject.js at line 58.
${decode}
${encode}

JSON.Stringify ensures that the contend assigned to var or const is faithfully stored / reproduced.  I'll need another hour to review.

I think so.  I'm just doing a quick check on why other libraries go to pains to avoid template literals in the final build project.

Once this is merged, I want to add to this.
With the correction in this PR, this will produce the following for ES6:
var shader_fragment_begin = `
  // shader code
`;

Other tools I use produce the following:
const shader_fragment_begin = (
  /* glsl */
  `
  // shader code
  `
);

The parens ensure it gets evaluated, evaluated once, const ensures it is optimal and dependable. It still has the benefits of the template literal and supports the use case of that file.


Cloud config cherry pick

Cloud Config File
Description
Implements the ability to enter, store and retrieve API keys nad a default TranscriptionEngine. Configs are stored in a JSON file in the appdata mlvet folder as "cloudConfig.json".
The currently stored key is displayed in the CloudConfigView text box upon opening.
The text for the CloudConfigView information changes depending on whether it is the user's first time entering the API key.
Why is this change needed?
Allows users to enter their custom API key, use it to transcribe and have it saved for multiple sessions.
OS

[x] Windows
[x] Mac
[ ] Linux

I've tested it out locally, it works and I found a couple of things.

If you add an api key, click save (go to the next view), then go back, it save button is then disabled. It should be enabled as the api key entered by the user is persisted.
The back and save button in the cloud config view and not longer on the same level as the next view. See this video

https://user-images.githubusercontent.com/86769131/187671070-0cb218a1-1ed9-421d-be07-cf837fabb8ec.mp4

Look really good but when when there already is a key saved and I create a new project, it doesn't let me move forward without changing the transcription key.
The save button is disabled and that's the only way to move to the next view

New commit should fix this issue

Also - the screen to enter a key should not appear in the new project flow if the user has already provided a key.

I've tested it out locally, it works and I found a couple of things.

If you add an api key, click save (go to the next view), then go back, it save button is then disabled. It should be enabled as the api key entered by the user is persisted.
The back and save button in the cloud config view and not longer on the same level as the next view. See this video

misalignment.mp4

New commits should fix both issues

@exquisitebin @Taz17 @chloebrett It looks like you can't paste the key into the text field - which is a huge pain to have to type it out...
I believe it's because we have custom copy/paste actions. How difficult is this going to be to fix?

Chloe and I found it's a Mac problem, as it works fine on my Windows.

Also - the screen to enter a key should not appear in the new project flow if the user has already provided a key.

The current issue is that its managed by the useEffect in the ModalContainer which only gets called once while you're at the home page. Therefore if you complete making a new project and go back to the homepage (or restart the app) it no longer appears.


Fix .ftpignore issue

Small clear code
All file use LF
Fix .ftpignore issue

Thanks for your PR, but I've got a few comments:
This is a big PR, in the future please split your PRs into small feature-based ones (e.g. one for the .ftpignore commit and one for the code beautifying) because one feature is hard to review when there are changes in virtually all files.
The problem with your ftpignore approach is that you load the .ftpignore file only once at the clients initialization. If someone modifies the .ftpignore file during coding, he has to restart Atom (or reload the Window) for the changes to take effect (which is uncomfortable and not expected). That's why the .ftpignore file has been read every time again when a sync was triggered before...
Also I think the file ignoring should not be done manually in every function in remote-ftp.js, but in the core upload and sync functions in client.js.
So, I would be happy if you could resubmit a PR with just the code formatting you've done.
For the ftpignore thing, I agree a global ignore utility function is better and the .ftpignore file should have effect on all upload methods, not just the sync-local -> remote feature but the .ftpignore file should at least be checked for any updates. This is also on my todo list but if you have time to think about this and implement it I am happy to accept your PR :)


Pod Disruption Budget only allows 1 pod offline across entire installation

We have a setup that results in us having multiple clickhouse pods on one kubernetes host from different clusters but the pod disruption budget being generated only allows one pod to be offline which means we can't evict the pods when doing maintenance on those nodes. Simplified below but from a service perspective any of the nodes should be safe to take down but with the disruption budget it is not possible to evict all clickhouse pods from any node.
| Node1  | Node2  | Node3  |
| main-0 | main-1 |        |
|        | int-0  | int-1  |
| ext-0  | ext-1  | ext-2  |

I can't seem to find anyway to configure how the budgets are generated, am I missing something? We'd like to have them per cluster (I guess per shard would be more appropriate but we're not using sharding at the moment)

Hi @widders , PodDisruptionBudget has the following selector:
spec:
  selector:
    matchLabels:
      clickhouse.altinity.com/app: chop
      clickhouse.altinity.com/chi: <chi_name>
      clickhouse.altinity.com/namespace: <chi_namespace>
  maxUnavailable: 1

It should work across different CHI instances. Do you mean you have several clusters inside one CHI, and this is what causes issues?

Yes exactly, we deploy three clusters from one ClickhouseInstallation so those selectors match across all three clusters.
FYI The main cluster feeds to internal and external with replicated tables.

+1 Would like to see pod disruption budget on per-shard bases.

Fixed in 0.20.1

https://github.com/Altinity/clickhouse-operator/releases/tag/release-0.20.1


spconv/src/spconv/indice.cu 125

Hi,
I was tring to train with confif file "nusc_centerpoint_voxelnet_01voxel.py" with 1 GPU and sweep=1.
I encountered the crash during training. Kindly help.
File "/home/Nuscene_Top/CenterPoint/tools/train.py", line 128, in main
logger=logger,
File "/home/Nuscene_Top/CenterPoint/det3d/torchie/apis/train.py", line 381, in train_detector
trainer.run(data_loaders, cfg.workflow, cfg.total_epochs, local_rank=cfg.local_rank)
File "/home/Nuscene_Top/CenterPoint/det3d/torchie/trainer/trainer.py", line 538, in run
epoch_runner(data_loaders[i], self.epoch, **kwargs)
File "/home/Nuscene_Top/CenterPoint/det3d/torchie/trainer/trainer.py", line 405, in train
self.model, data_batch, train_mode=True, **kwargs
File "/home/Nuscene_Top/CenterPoint/det3d/torchie/trainer/trainer.py", line 363, in batch_processor_inline
losses = model(example, return_loss=True)
File "/home/anaconda3/envs/centerpoint/lib/python3.6/site-packages/torch/nn/modules/module.py", line 493, in call
result = self.forward(*input, **kwargs)
File "/home/Nuscene_Top/CenterPoint/det3d/models/detectors/voxelnet.py", line 47, in forward
x = self.extract_feat(data)
File "/home/Nuscene_Top/CenterPoint/det3d/models/detectors/voxelnet.py", line 24, in extract_feat
input_features, data["coors"], data["batch_size"], data["input_shape"]
File "/home/anaconda3/envs/centerpoint/lib/python3.6/site-packages/torch/nn/modules/module.py", line 493, in call
result = self.forward(*input, **kwargs)
File "/home/Nuscene_Top/CenterPoint/det3d/models/backbones/scn.py", line 364, in forward
ret = self.middle_conv(ret)
File "/home/anaconda3/envs/centerpoint/lib/python3.6/site-packages/torch/nn/modules/module.py", line 493, in call
result = self.forward(*input, **kwargs)
File "/home/anaconda3/envs/centerpoint/lib/python3.6/site-packages/spconv/modules.py", line 123, in forward
input = module(input)
File "/home/anaconda3/envs/centerpoint/lib/python3.6/site-packages/torch/nn/modules/module.py", line 493, in call
result = self.forward(*input, **kwargs)
File "/home/anaconda3/envs/centerpoint/lib/python3.6/site-packages/spconv/conv.py", line 155, in forward
self.stride, self.padding, self.dilation, self.output_padding, self.subm, self.transposed, grid=input.grid)
File "/home/anaconda3/envs/centerpoint/lib/python3.6/site-packages/spconv/ops.py", line 89, in get_indice_pairs
stride, padding, dilation, out_padding, int(subm), int(transpose))
RuntimeError: /home/Nuscene_Top/spconv/src/spconv/indice.cu 125
cuda execution failed with error 2

Can you tell me your torch, cuda, spconv version? Also what other changes(if any) did you make to the code?

cuda execution failed with error 2 Uhm, I am not 100 percent sure. But the cuda error 2 seems to mean that you are out of memory.

Sorry for late reply.
I was using torch version 1.1
spconv 1.0 vwrsion.
I replicated the same setup as you mentioned in the installation instructions.
I tried your pointpillars succesfully without any hurdles.
But the config file uses spconv module and spconv module is crashing.
Moreover iam traing with sweeps=1. Hence i commented lines 87-97 in https://github.com/tianweiy/CenterPoint/blob/master/det3d/datasets/pipelines/loading.py
Apart from this change, there is no change in code.
Kindly help

cuda execution failed with error 2 Uhm, I am not 100 percent sure, but the cuda error 2 seems to mean that you are out of memory.

You don't need to comment loading files. Just change the config nsweep field to 1. Also I suspect it is gpu out of memory issue from the error log, can you check this ?

cuda execution failed with error 2 Uhm, I am not 100 percent sure, but the cuda error 2 seems to mean that you are out of memory.

You don't need to comment loading files. Just change the config nsweep field to 1. Also I suspect it is gpu out of memory issue from the error log, can you check this ?

how to reduce memory usage in test phase?

@AbdeslemSmahi  the simplest way is to add a --speed_test flag during testing. This will by default use batch size 1. Not sure how to go beyond this

@AbdeslemSmahi the simplest way is to add a --speed_test flag during testing. This will by default use batch size 1. Not sure how to go beyond this

Even that didn't work.

probably need to get a larger gpu then... or try pointpillars model which take less memory.

close for now. Feel free to reopen if you still have questions.

@AbdeslemSmahi  @tianweiy
Hi，I also encountered the same question with config nusc_centerpoint_voxelnet_dcn_0075voxel_flip_circle_nms.py, but it works fine with config nusc_centerpoint_pp_dcn_02voxel_circle_nms.py, have you solved this problem?
All the training process is on 2 Titan V(4 Titan V also tested, failed either), and I noticed that the first GPU seem to use more GPU mem than the second GPU.
is there any chance that the distributed launch assigns the dataloading only to the first GPU?

Hi, 0075 voxelnet will definitely take much more memory than pp. Can you train the 0.1 voxel size model ? You can also decrease the batch size a bit, I don't think this matter much for performance.
Also it seems spconv(voxelnet) is quite weird for Titan v. Basically, I try to train voxelnet on Titan xp, Titan rtx, 2070/2080, v100, Titan v. All other gpu works but for Titanv I can't use even batch size 2 for a kitti model. I feel this is a bug with spconv. Do let me know if your titanv works well with spconv

@tianweiy  Thank you for your reply. I followed your advice and the results are:

voxel_size=0.1, batch_size=4, Titan V, nproc_per_node=2, failed (cuda execution failed with error 2)
voxel_size=0.1, batch_size=4, Titan V, single Titan V, failed (cuda execution failed with error 2)
voxel_size=0.075(nusc_centerpoint_voxelnet_dcn_0075voxel_flip_circle_nms.py), batch_size=4, Titan xp, nproc_per_node=2, failed (GPU out of memory)
voxel_size=0.075(nusc_centerpoint_voxelnet_dcn_0075voxel_flip_circle_nms.py), batch_size=4, Titan xp, nproc_per_node=2, succeed

it seems that spconv cannot work on Titan V(when voxelnet involved), and it indeed takes large amount of memory to run the config with small voxel size. But now I reduce the bacth size to 2 and it worked, so nothing wired happen for the moment.
Thank you again for your timely and detailed reply!

Sure, good luck with your project.


Update Killercoda scenarios from deprecated k3s profiles

Email from Killercoda:
We're writing you because you've some scenarios based on K3s environments.
As recently announced these have been deprecated. Please check here for more details and how to migrate.
We suggest to go ahead and change your imageid to "kubernetes-kubeadm-1node" or "kubernetes-kubeadm-1node-4GB".
From the 20.01.2024, all K3s scenarios will automatically be run on corresponding Kubeadm environments. If you don't rely on K3s specifically, nothing should break.
You could also use our "ubuntu" environments and install K3s yourself in foreground or background scripts.

It looks like the scenarios on https://killercoda.com/kubevirt are currently broken.
I'll remove KubeVirt from the Killercoda main page for now, will add again once fixed.
Will follow this issue and repo.
Let me know if you need help!

Looking into it this morning, it seems my assumptions about how k3s installs were broken by a change. I've changed my "Is k3s installed yet?" test to look for a kubeconfig, not the binary. Will PR shortly.


Fix subnet name resolution for ELBs

This work fixes a bug where the subnet_names variable is not populated correctly
due to an incorrect variable reference.  Here we change that variable name to
ensure that the subnet_names are returned correctly.

Thanks @ahenroid


Generate types for Query fields

v0.19 used to generate exclusive types for fields
export type XxxQuery = {
  __typename?: "Query";

  field: Maybe<XxxField>;
};

export type XxxField = {
  ...
};

Equivalent in current stable
export type XxxQuery = (
  { __typename?: 'Query' }
  & { field: Maybe<(
...

Is there a way to use original behaviour so I can use type XxxField? Or do I have to use XxxQuery['field']?
Thanks!

I think compatibility plugin is the key but why is the new format preferred? Typing XxxQuery['field'] feels softer than XxxField.

It's been a long time since 0.x versions, and a lot has changed.
As @simPod said, you can use typescript-compatibility in order to generated those and migrate easily to 1.6.


SPRX, change serialization logic

see discussion in #2430
This changes the amount of data in each SPRX file, one header will now match one file. Change should be backwards compatible.

okay, all the SPRX are now deserialized, however the INTF and WEBI are not generated, as when the SPRX are deserialized it might not have enough information to build the proxy, and there is no diff for the SPRX objects the INTF and WEBI deserialization are not triggered again
Guess the part with
    IF mv_object = 'INTF'.
      generate_service_definition( lt_sproxdat_new ).
    ENDIF.

has to be moved to the INTF object

there is also some issues during uninstall, however triggering uninstall multiple times solves the issue, guess the ddic dependency graph for SPRX has to be calculated

tested ok with https://github.com/abapGit-tests/SPRX/pull/1
for https://github.com/abapGit-tests/SPRX_server_proxy I get the same error as with the old code
I think the file contents is correct now, however there are still problems during deserialization and uninstall, but I think the serialization problems are fixed

@flaiker can you try testing this change
@christianguenter2 would you like to review this change?

I cannot test deserilization for this project unfortunately.


global is not defined (ex: using @coinbase/wallet-sdk package)

Environment
Nuxt CLI v3.0.0-27467056.93c4dfe                                                                                14:15:17
14:15:19
Nuxt project info:                                                                                14:15:19

Operating System: Darwin
Node Version:     v16.13.1
Nuxt Version:     3.0.0-27467056.93c4dfe
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      head, css, plugins, components, buildModules, modules, build, strapi
Runtime Modules:  -
Build Modules<EMAIL_ADDRESS>
Reproduction
npm i @coinbase/wallet-sdk
In a .ts file
ex. file ./plugins/ethers.client.ts

import CoinbaseWalletSDK from "@coinbase/wallet-sdk";

async function connect() {
    // Initialize Coinbase Wallet SDK
    const coinbaseWallet = this.CoinbaseWalletSDK({
      appName: APP_NAME,
      appLogoUrl: APP_LOGO_URL,
      darkMode: false,
    })
}

Throws error
Uncaught ReferenceError: global is not defined
    at node_modules/readable-stream/lib/_stream_readable.js (_stream_readable.js:48:21)
    at __require (chunk-36VY3QWH.js?v=e31a8745:25:50)
    at node_modules/readable-stream/readable-browser.js (readable-browser.js:1:28)
    at __require (chunk-36VY3QWH.js?v=e31a8745:25:50)
    at node_modules/keccak/lib/api/keccak.js (keccak.js:1:23)
    at __require (chunk-36VY3QWH.js?v=e31a8745:25:50)
    at node_modules/keccak/lib/api/index.js (index.js:1:22)
    at __require (chunk-36VY3QWH.js?v=e31a8745:25:50)
    at node_modules/keccak/js.js (js.js:1:18)
    at __require (chunk-36VY3QWH.js?v=e31a8745:25:50)

Describe the bug
Installing a package (@coinbase/wallet-sdk) on a Nuxt3 project throws global is not defined.
Does Nuxt3 support pointer to window ?
Additional context
No response
Logs
No response

It's not solution to include whole NodeJs environment in browser :o why was this closed ?


Icon buttons used in toolbar is not getting 'tabindex' value '0' when clicked

🐛 Bug Report

When using buttons with icons within the nimble-toolbar and clicking on the icon, the button does not receive a tabindex=0. By default, the first button in the toolbar possesses a tabindex=0, whereas the remaining buttons have a tabindex=-1. Consequently, upon pressing the Tab key to shift focus from the previously selected field to the buttons, the focus does not transition to the button that was most recently clicked.
💻 Repro or Code Sample

Add nimble buttons with an icon in it to a nimble-toolbar.
Inspect the buttons and verify the default tabindex value.
Click the icon in the button which has tabindex=-1.
Notice the tabindex value in the dev tools.

🤔 Expected Behavior

The button which is clicked should be updated to tabindex=0. Also, the button which has tabindex=0 previously should be updated to tabindex=-1.
😯 Current Behavior

The button which is clicked is not updated to tabindex=0 when clicking on the icon. However, when clicking the button and not on the icon, the tabindex=0. However, upon clicking outside the icon, the tabindex gets updated.

💁 Possible Solution

🔦 Context

In nimble-rich-text-editor component, for footer formatting options we use nimble-toolbar with the nimble-toggle-button just the icon enabled with it. For more details, see: https://github.com/ni/nimble/pull/1416#discussion_r1282725591 (issue 2)
🌍 Your Environment

OS & Device: Windows on PC
Browser Google Chrome
Version 115.0.5790.170

I believe the issue is related to the clickHandler in FAST's toolbar. Within the clickHandler, the toolbar attempts to update its activeIndex, which represents which item in the toolbar should be focused. However, the check is that the item clicked is one of the focusable elements within the toolbar, not whether or not the item clicked is within one of the focusable elements. As a result, when the icon is clicked, the check doesn't pick up that the icon is within the focusable toolbar button, and the activeIndex doesn't get updated.
See: https://github.com/microsoft/fast/blob/218cfc014f5c76464493725f83bd883e8b07defa/packages/web-components/fast-foundation/src/toolbar/toolbar.ts#L134

I created https://github.com/microsoft/fast/issues/6819 to track the underlying issue in the FAST toolbar.
However, we should also evaluate whether adding pointer-events: none to some of our nimble styles makes sense, such as the start and end slots of our button styles.

Marking as blocked to reflect the dependency on the FAST issue.


The Shortcut key(Ctrl+R) isn't available after upgrade to v1.15.1

VSCode Version: 1.15.1
OS Version: WIN 10 /Ubuntu

Steps to Reproduce:

Choose the File-->Open Recent-->More(Ctrl+R), it will show the all recent open project.
But I use the shortcut key(Ctrl+R),  nothing for me.  and the status bar show the Ctrl+R was pressed, Waiting for the second key.....

Reproduces without extensions: Yes/No

@dengyue
It sounds like you may have a key binding utilizing a chord (Control+R + AnotherKey)
You can search in the key findings editor for 'Control+R' and see all bindings using this key.

@dengyue can you check whether you run into the problem @cleidigh (thanks!) mentioned above?


Wrong link on Storage Classes page

Fixes #36290
Updated page Storage Classes.

/retitle Fix wrong link on Storage Classes page


CNI Plugin

Add CNI Plugin

There's a bunch of TODOs here. And there's no background GC process, but it's a start in the right direction, IMHO.
If you want to try it out, drop a file in /etc/cni/net.d/10-sargun.conf (make sure there's no lower files)
with the contents:
{
  "cniVersion": "0.3.0",
  "type": "titus",
  "name": "sargun",
  "KubeletAPIURL": "https://localhost:10250/pods"
}

(I have absolutely no idea what the name does)

@hankjacobs @rgulewich I'm also thinking of splitting out the bits under vpc/service/ into a separate PR. What do you think?

Also, I'm sorry. This wasn't supposed to be 1000 lines. I have no idea how it became 1 thousand lines. :(.

+1 on splitting the vpc/service stuff into a separate PR. Will make it easier to review

@hankjacobs Already split: https://github.com/Netflix/titus-executor/pull/362


Make configurable db.SetMaxOpenConns, db.SetMaxIdleConns and db.SetConnMaxLifetime

I think it would be useful to be able to change db.SetMaxOpenConns(X), db.SetMaxIdleConns(Y) and db.SetConnMaxLifetime(Z) in the configuration file. What do you think?
I will first try to do this in my backend, and then I will send PR (if someone does not get ahead of me).
PS: https://www.alexedwards.net/blog/configuring-sqldb
and http://techblog.en.klab-blogs.com/archives/31093990.html

Is this for the SQL driver? If so, perhaps the same result can already be
configured via the DSN, see all the options here
https://github.com/go-sql-driver/mysql/blob/master/README.md#dsn-data-source-name
On Thu., 15 Aug. 2019, 17:47 Lord Alfred<EMAIL_ADDRESS>wrote:

I think it would be useful to be able to change db.SetMaxOpenConns(X),
db.SetMaxIdleConns(Y) and db.SetConnMaxLifetime(Z) in the configuration
file. What do you think?
I will first try to do this in my backend, and then I will send PR (if
someone does not get ahead of me).
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/flashmob/go-guerrilla/issues/174?email_source=notifications&email_token=AAE6MP2MWKJCPP2HSNVOQWLQEUJYTA5CNFSM4IL4JM7KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HFMKGBQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAE6MP6CORYSSZS5SYEJGL3QEUJYTANCNFSM4IL4JM7A
.

Yes, for SQL driver. But this driver dont provide DSN options for change this parameters: https://github.com/go-sql-driver/mysql/blob/master/README.md#connection-pool-and-timeouts
And in article on alexedwars blog (link above) says:

By default sql.DB allows a maximum of 2 idle connections to be retained in the connection pool. You can change this via the SetMaxIdleConns() method

All right, can be a config option then. PR welcome.
On Thu., 15 Aug. 2019, 23:31 Lord Alfred<EMAIL_ADDRESS>wrote:

Yes, for SQL driver. But this driver dont provide DSN options for change
this parameters:
https://github.com/go-sql-driver/mysql/blob/master/README.md#connection-pool-and-timeouts
And in article on alexedwars blog (link above) says:
By default sql.DB allows a maximum of 2 idle connections to be retained in
the connection pool. You can change this via the SetMaxIdleConns() method
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/flashmob/go-guerrilla/issues/174?email_source=notifications&email_token=AAE6MPZRUBSUH5I7AC6CVSDQEVSCZA5CNFSM4IL4JM7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4L66LY#issuecomment-521662255,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAE6MPZ26GY2MYJRGQ7V2PTQEVSCZANCNFSM4IL4JM7A
.

Don’t forget to add info to the wiki :)
I do not close issue so that there is a reminder of this. Close it yourself later.

Wiki updated.


Fix explanation in v2.9

Checklist

[X] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[X] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Just fixing a comment, no new build needed.


Allow Rails 8

Alternative to other open PR -- see comment here re: version thing - https://github.com/devise-two-factor/devise-two-factor/pull/286/files#r1806997538
While in there, converted the version list in CI config to one per line, so that someone-of-the-future can come in there and remove/add ruby/rails versions with single-line changes.
Will need continued updates/followup as betas/release moves through with either of these PRs to keep the appraisals/ci aligned.
Happy to alter gemspec dependency version format if desired.

Just noticed there's an 8.0.0.rc1 out today ... updating the Appraisal/gemfile here for that.

Release as v6.1.0. Thank you to all who contributed!


NSStringAPI: test fix for hasPrefix and hasSuffix

The previous version wrongly cached a result, which in turn masked a bug in the bridged hasSuffix().
(reported as https://bugs.swift.org/browse/SR-243)
This version also trips hasPrefix, but that involves \u{0}; the correct behaviour there is less obvious.

This bug fix surfaces two cases that always failed, but were masked.
They are now both annotated with xfail and links to bug reports.

Can one of the admins verify this patch?

Rebased after the changes in https://github.com/apple/swift/pull/1131

@swift-ci test

@gribozavr please review

@tkremenek The swift-ci failure was unrelated to these changes; test again?

Drive-by test restarting:
@swift-ci Please test OS X platform

@gribozavr This ok to merge?

@glessard Sorry, I don't understand what this patch changes -- it looks like a pure refactoring to me.  I see that you are removing the xfail predicates from the test list, but you're adding them back with the map() function.  Why is this making a difference?  I'm probably missing something, I would be glad if you could explain to me.

The bug fix is https://github.com/apple/swift/pull/629/commits/21d5da7a768c1a36089525de2f9b4b4cbff86215, the rest is dealing with the consequences of it (and of https://github.com/apple/swift/pull/1131, since they have distinct sets of xfails).
In the original version of 'checkHasPrefixHasSuffix', a pair of expected results is calculated, and then are used twice each. This wasn't correct. In particular, if the first character of lhs is combining, ("abc"+lhs).hasSuffix(rhs) should not be equal to lhs.hasSuffix(rhs), yet the test expected that to be true (see https://github.com/apple/swift/pull/629/commits/21d5da7a768c1a36089525de2f9b4b4cbff86215#diff-f7824440e4e317d38fc0554573876bccL261). The original test passed the ("\u{0301}", "\u{0341}") case by matching a bug of NSString/CFString (https://bugs.swift.org/browse/SR-243). Without that NSString bug, the original formulation of the test wouldn't have passed.
The fix simply ensures the expected result is recalculated for every single use, by moving the extra suffix and prefix tests to a separate call of checkHasPrefixHasSuffix. (I stumbled upon this while porting the tests for https://github.com/apple/swift/pull/440)

@gribozavr Did the above response clarify the patch?

Let's try to revive this:
@swift-ci please smoke test macOS platform.

@swift-ci please smoke test.

I think I'm out of nitpicks if this was pure refactoring.  Thank you @glessard.

Once you've added a bit of docs, please squash this down to a patch or two.

@CodaFi I think this should do it; thanks for the feedback

@swift-ci please smoke test.

macOS
Linux
// @shahmishal

This issue looks like:
https://github.com/janinko/ghprb/issues/150
https://github.com/janinko/ghprb/issues/284
going to retrigger the build to see if it picks up the new sha.
@swift-ci Please smoke test

I think we're good to go here.  Merging.

Ah, @gribozavr, did you have any remaining comments?

LGTM, thanks!


Cursor flickers when selecting next or previous item in custom menu

This is a really minor issue, but I thought I would create an issue and see if there is anything that could be done with this.
When selecting the next and previous item in the custom menu, the cursor flickers a bit. See the video below:
https://user-images.githubusercontent.com/5160701/136559314-ac47d32f-d9be-4d81-a69e-d6259e3f02cd.mov
This does not happen with the builtin one. See video below for comparison:
https://user-images.githubusercontent.com/5160701/136559414-e1b7047b-6062-4ad3-a3fc-3c006952e9ff.mov
Like I said, this is not a big problem, but it was something I noticed that bothered me a little bit. If you feel like this will be too hard to fix, or just doesn't bother, feel free to close the issue!
Awesome work with the custom menu btw!

@seblj I will try to fix it.
@captainko You can use native menu or other completion engine :)

I am getting this as well.
I sporadically tested the development branch a couple times during the past few weeks, and I don’t recall seeing this issue (though I could be mistaken). Maybe a recent regression?
I am back to native menu, hopefully not for long.

I think the implementation is changed to support repeat . feature.

I think it should be solved by set lazyredraw.
I will investigate the workaround. But I think it can't fix perfectly.

Is it fixed now...?

Is it fixed now...?

After trying today's update, I no longer experience any flickering, thanks!

Is it fixed now...?

Yes.
No more flickery/jumpy cursor. Much much nicer.
Superb.

Yes, sorry for late response, but it also is fixed for me! Thank you!

Hi, All of users.
If set lazyredraw will be the required option (for avoiding flicker), do feel it annoying?

Yes, it causes problems.
Years ago I tested lazyredraw and concluded it had side effects, such as X delete backwards having weird rendering effects (from memory).
lazyredraw is not the default, probably for good reason.
My 2cents.

Maybe I've supported undos/macros/dot-repeat with avoiding flicker. I've removed the breaking change announcement.
If anyone has another issue, feel free to create new issues.

Thank you @hrsh7th , the cursor flicker is now fixed in commit 7a36a31.
Excellent work and genuine thanks..