Display commit revision for latest Tag, or first commit

I am integrating convco into a monorepo that uses pants for the build of many artefacts (packages)
Pants, given a commit can detail with filters (paths, types) which packages are impacted.
It can also build the said packages with the same filters.
An example of the usage would be

pants --changes-since=$(convco version --tagrev --prefix my/lib/v --paths my/lib ) package list
Convco knows for a given paths (path/to/package) and for a given tag prefix path/to/package/v , what the latest tag is.
What I am looking for is to ask convco for the commit of that tag instead  (version --tagrev)
Currently I do this lookup in shell with calls to convco and to git log
It will be nice to do this only with convco
In scenario where there is no tag then convco should print the first introducing commit (if paths or the first commit if no paths)
Thoughts ?

I used --commit-sha in #158 , but i think it might work to add a flag --print-prefix as pants probably understands all kinds of git revisions for --changed-since
this is what it prints for this repository at e9d98f7ee3b1c08242b694afdd7c2712994d3e7b.
❯ git log --oneline -1 v0.4.3
feeea90 (tag: v0.4.3) chore: release v0.4.3

❯ cargo run -- version
    Finished dev [unoptimized + debuginfo] target(s) in 0.07s
     Running `target/debug/convco version`
0.4.3

❯ cargo run -q -- version
0.4.3

❯ cargo run -- version --commit-sha
    Finished dev [unoptimized + debuginfo] target(s) in 0.06s
     Running `target/debug/convco version --commit-sha`
feeea9022f2795d280602aaba4d3c24a8088f125

I also added an option to print the prefix convco version --print-prefix or convco version --pp

Very nice. Thanks !


Dixon bio and mugshot

Tasks

[ ]  write a bio
[ ]  attach an image you'd like to use for promotional materials

Added a pull request #20

Pull request #20 merged


Fix column not found when using additional_attributes w/ xMany relations

Fix for belongsToMany and hasMany relationship display with
additional_attributes. You can now use additional_attributes with those
relations.
https://github.com/the-control-group/voyager/pull/2885
The fix get the model before plucking it. It caused error because when
you pluck the query builder, it affects the mysql query. When you use
additional attribute, it doesn’t exist in the database and result with
a SQLSTATE[42S22]: Column not found: 1054 Unknown column ‘youradditionalattribute’ in 'field list’ error…

There is also a problem with pivot table on display, it prefix the already prefixed table. Which causing this error:
SQLSTATE[42S02]: Base table or view not found: 1146 Table 'mydatabase.myprefix_myprefix_mytable' doesn't exist [...]

When we create the relationship, we need to put the prefix out of the name of the pivot table. I did something locally but I don't know if i'm the only one having this problem.
I changed the same line here:
belongsToMany($options->model, str_replace(\DB::getTablePrefix(), '', $options->pivot_table)

Should I modify my pull request or create an other one and delete this one?

I need to do it in a branch, I will submit it back lather.


last update text climbs up the back button under the MAP

last update text climbs up the back button some times.
Lastly, There should not be a back button either as i have not clicked on a state.

@deep5050 Does this happen every time? I've only faced this intermittently. Let me know if you have any concrete steps to consistently reproduce it.

@shuklaayush  you are right , it happens between clicking a state and the getting the district view sometimes. I don't know the logic behind the implementation. But it could be due to slow internet connection or system lag / website poor performance.

Couldn't reproduce. Please reopen if you're seeing this issue consistently.


Fix render exception when biome ID is null

net.minecraft.util.registry.Registry<T>.getId(T var1) returns @Nullable Identifier, which means it can return null too, and this case was not handled, causing chunk rendering to crash, leaving blank space. Adding a simple if to check that it's not null, otherwise returning default colour, appears to fix the issue.
Fixes #10

1.18.2?

I tested this change merged with the updated 1.18.2 version of the mod and it completely fixes the rendering issue.

Rebased on https://github.com/2Retr0/BedrockWaters/commit/d42ba57f4467367e52f5a18c0cbfe62f2a1f3679.

I've fixed this issue in version 1.7.0. It should be noted, however, that this version only applies to 1.19.2--sorry for the late reply!


Bounty: Add login with metamask/walletconnect

Description
This PR adds these login methods:

Desktop web:

WalletConnect Modal QR Code
Metamask (if the extension is installed)

Mobile web and native app:

WalletConnect Modal QR Code and deep link

About # (link your issue here)
https://github.com/GoodDollar/Bounties/issues/18
How Has This Been Tested?
Checklist:

[x] PR title matches follow: (Feature|Bug|Chore) Task Name
[x] My code follows the style guidelines of this project
[x] I have followed all the instructions described in the initial task (check Definitions of Done)
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I have added reference to a related issue in the repository
[x] I have added a detailed description of the changes proposed in the pull request. I am as descriptive as possible, assisting reviewers as much as possible.
[x] I have added screenshots related to my pull request (for frontend tasks)
[ ] I have pasted a gif showing the feature.
[x] @mentions of the person or team responsible for reviewing proposed changes

Is this bounty still open?

@3scava1i3r not anymore thanks

@sirpy how about this PR: https://github.com/GoodDollar/GoodDAPP/pull/3802/files ? could it be closed now ?

@sirpy i've resolved conflicts. could this PR be merged ?


Restore new org-agenda bindings

These relatively new bindings (#3082) were lost during the rebase performed in #3043. This PR restores these, and also unifies the org-agenda bindings for deadline/schedule with the ones from org-mode.

Ack, my bad. Thanks for restoring them!


Add options for a reducing amount of temp2 space used at any given time

Related to #819 and #805
By using a RAM block cache as small as 40GB, almost 50% of writes to a temp2 SSD volume can be prevented.
If would be great if this concept could be extended to adjust chia_plot to restrict the amount of data it is processing at any given time in the temp2 dir such that it might completely fit into a user's RAM block cache or RAM disk.
This kind of feature would be very valuable for the Chia community, since it could give users with less system memory the ability to do more plotting in memory without degrading SSD's. It would also help to negate some of the bad press that Chia gets for it's reputation as an "SSD destroyer".

Tu ya configuraste para que con 40GB de Ram funcione?
Si es así, me podrías explicar porfavor? mi placa no soporta 128 gb de RAM, pero 64 si es una opción

@nikobart I used PrimoCache on Windows to configure a 40GB L1 cache (Strategy: Write Only,  Defer Write period: infinite) for my temp2 SSD volume.  This reduced SSD drive writes by about 50%. I believe the Linux equivalent would be to use bcache

@nikobart Solía PrimoCache en Windows para configurar una caché L1 de 40 GB (Estrategia: Sólo escritura, Defer periodo Comentario: infinito) para el volumen de mi SSD Temp 2. Esta unidad SSD reduce las escrituras en aproximadamente un 50%. Creo que el equivalente de Linux sería usar bcache. Mi sistema solo tiene 64 GB de RAM, por lo que esto dejó suficiente memoria libre para el trazador Chia y otras tareas en segundo plano.

Muchas Gracias!
última duda, la velocidad de trazado se vio afectada?

Nice idea!
However, at least for me, be careful with PrimoCache....latest Win10 x64 and after reboot, during restart, had multiple BSODS and PrimoCache is not working, unistalled.
The equivalent Linux version would be rapiddisk, but it is quite coumbersome to build/install/use (recompiling kernel etc..)
Would be nice to have a guide for Linux with bcache (or eventually LVM too since it supports caching).
I'll start with few commands as example:

installing bcache
sudo apt-get update
sudo apt-get upgrade
sudo apt-get install bcache-tools
creating & mounting ramdisk
sudo mkdir /mnt/ramdisk
sudo mount -t tmpfs -o size=2G ramdisk /mnt/ramdisk
sudo mount -a
creating bcache device
sudo wipefs -a /dev/ssd -> this will wipe ssd
sudo make-bcache -B /dev/ssd -C /mnt/ramdisk --discard --writeback
sudo mkfs.ext4 /dev/bcache0
deleting bcache device
sudo umount -v /dev/bcache0
sudo su - -c "echo 1 >/sys/fs/bcache/device_uuid/unregister"
sudo su - -c "echo 1 >/sys/block/bcache0/bcache/stop"

the problem im facing is that, after creating the ramdisk, I launch sudo make-bcache and get the following error: Cannot use "/mnt/ramdisk", it is a folder"....Any idea?

PrimoCache results:

win10 latest
1x nvme as tmpdrive & tmpdrive2 (21G, write only, defer time infinite, block size 4k)
Total write req 1.12TB (host)
Total disk write 0,99TB

this is about a 11% of saved writes per plot. Plot time barely affected (48min ubuntu, 51min win10)

the problem im facing is that, after creating the ramdisk, I launch sudo make-bcache and get the following error: Cannot use "/mnt/ramdisk", it is a folder"....Any idea?

You can't use bcache with a tmpfs ramdisk. You need block device.   brd module does it.

new test done:

E:\ tempdrive1 | D:\ tempdrive2
cache enabled on D:\
cache info: 25GB, block size 4kb, write only, defer write infinite, mode native, empty data written from cache
1.12TB / 1.43TB -> 21.7% saved ssd writes -> sbout 300GB per plot
6% faster plot time compared to ubuntu

PrimoCache results:

cache enabled on tempdrive2
cache info: 55GB, block size 4kb, write-read , defer write infinite, mode native, empty data written from cache.
tempdrive2 total write 991GB / write to disk 394GB -> 594Gb  saved ssd writes
plot time drop from 48 min to 40 min

@wavepacket0 Any luck making it work with bcache in Ubuntu ?


Magnitude criterion requires abs

Magnitude pruning prunes weight according to the absolute value of the weight |w_i|. The current implementation in
https://github.com/intel/neural-compressor/blob/master/neural_compressor/compression/pruner/criteria.py takes the weight with sign w_i as a score for pruner, therefore large negative weights will be pruned in the first order. However, these weights are quite important and their pruning will change output of the model noticeably. This can be fixed as follows:
    def on_step_begin(self):
        """Calculate and store the pruning scores based on magtinude criterion."""
        with torch.no_grad():
            for key in self.modules.keys():
                p = self.modules[key].weight.data # add .abs() here
                self.scores[key] = p

@YIYANGCAI please follow this issue.


Add "log off" option, which will allow for several accounts (e.g. Personal and Business)

Feature Description
I can't find the "log off" option in the Bitcoin Keeper app (iOS v1.3.1)

If the log off option exists, it would be great to make the button more visible.
If the log off button doesn't exist, I would be grateful if you add it.

There are several reasons for why users would want the ability to log off, and log in to several Bitcoin Keeper accounts.
But the ability to have both a personal and a business vault is definitely one important use case.
Pre-requisites/ dependencies

iOS v1.3.1 (latest update)

Thanks for raising the issue. Keeper currently doesn't offer this option, but we have it in mind and discussed an approch for it in #5575
I will make sure to update here as we progress on it!


rename legacy to hybrid

https://github.com/tariqkurd-repo/riscv-cheri/pull/new/rename_legacy_mode

I think the new extension name makes a lot of sense, just not quite sure whether we want to use it for the name of the mode as well. I have a slight preference for "Integer Mode" over "Hybrid Mode" but I'm happy with either.
CC: @jrtc27 @swm11 @rwatson

oh I see, I thought we were talking about both.
Probably better resolve this one next! I'm not keen on integer mode, as it also does FP, Vector etc. so I don't think is really covers it.
Hybrid makes more sense to me because it does the existing stuff, and some new stuff too.

discussed with @PeterRugg - we're now trying out default mode and capability mode
Opinions?

discussed with @PeterRugg - we're now trying out default mode and capability mode Opinions?

How about "compatibility mode"?

discussed with @PeterRugg - we're now trying out default mode and capability mode Opinions?

How about "compatibility mode" and "capability mode"? After all, the mode is a trick to make standard RISC-V software work well on a machine with CHERI support (among other things!)

then both modes begin with C. Also this one came up in the meeting yesterday as an option IIRC

discussed with @PeterRugg - we're now trying out default mode and capability mode Opinions?

How about "compatibility mode" and "capability mode"? After all, the mode is a trick to make standard RISC-V software work well on a machine with CHERI support (among other things!)

then both modes begin with C. Also this one came up in the meeting yesterday as an option IIRC

Sure, but we've never used C to refer to Capability mode because that letter is used for a few other things already.

Default isn’t necessarily the default

true. I think I should put the name of the mode in a variable!
What are we going to call it?

unsafe mode?
compatibility mode
coarse grained mode, and cap mode is fine grained
default mode
riscv mode
integer pointer mode, and cap mode is capability pointer mode
thin pointer mode, and cap mode is fat pointer mode
any other options?

We need to choose a name before friday! Opinions?

We need to choose a name before Friday! Opinions?

My preference is still "compatibility" and "capability" mode, even though both start with the same heavily overloaded letter!

I'm in favour of using names that have a very clear meaning, so like "capability pointer" and "integer pointer" names for modes.

I tend to agree with Simon as well; "capability pointer" mode and "integer pointer" mode are most clear.  Maybe we can come up with an abbreviation for them.

Idle thoughts: I was wondering to what extent only the extension needed a name, and we are either in/entering capability mode or not in/leaving capability mode.
Otherwise, I also like @swm11 's suggestion.

I think the suggestion by @swm11 is very clear. It might be a bit too long for the common usage. Would "int pointer mode" and "cap pointer" be a sensible abbreviation?


docs: update GOVERNANCE

You create an GitHub ISSUE in the community Repo, recommending the Maintainers consider your proposal to change the Governance doc. (you don't need to detail your changes in the doc). We need an ISSUE as it is our core reference to this possible event.

Another suggestion is to do it on a PR so everyone can comment inline, rather than a plain issue. I'm not sure how we can convey changes effectively on an issue, perhaps a PR would be better suited.

This is why we must have an ISSUE vote first. A PR has a different more relaxed approval process where only a 'reasonable number of reviewers' need to approve a PR. Which is too unsafe for a change to our Governance doc

As long as the same rule is in place, not sure what makes it unsafer. I'd argue a PR is safer, because reviewers would get to see whatever is being modified as opposed to reaching some kind of consensus only for it to diverge in the actual PR.
Another approach could be an RFC-line process.

As for your PR... I think jumping to the PR step first, is an accelerating the process a little bit too quickly. As is taking general public feedback/vote in the PR "Review comments" where the Maintainers haven't had a chance to review the conceptual changes first... and refine the language (in a closed safe session).

Ok...

For Governance doc changes, we need to be extremely carful & vigilant.

I recommend we put this PR on hold/draft until we achieve steps 1, 2, 3 & 4.

Which is why I raised it as a draft to begin with 👍

Closing until we have an established procedure to update the Governance


[docs] Fix sphinx warnings

It's currently hard to write new docs because there are so many warnings in the Sphinx output about bad markup that you can't see if any new documentation contains bad syntax. This PR fixes up all the markup errors and other Sphinx problems, such that the docs build without warnings. (Well, only one, about sympy, but now you can see any new warnings much more easily.)

@simoncozens Just so you know I'm updating the documentation in designspaceLib as part of #2436. I see that you've started touching a tiny bit the designspace files, in order to fix warnings there. That's fine and won't be an issue to merge but if you could not change too much that part of the documentation that would be helpful. Can I also do anything on my side to prevent conflicts later? (I'll make sure not to add more warnings)

It's no problem. Designspace documentation is already pretty good :-) I'm focusing on the TrueType tables, which are the really useful but really undocumented bit of fontTools...


How to create a cluster using the image?

How to create a cluster using the image?

Hi Jasper!
This image was not designed with clustering in mind.  HBase's ZooKeeper can get kind of grumpy if you don't configure it "just-so."  If you figure out a way to do it via the image, I will happily test and merge a pull request that enables the functionality!


Eufy Security addon cannot be found in HACS

Please check Services, Known Bug / Issues and Troubleshooting over here first: https://github.com/fuatakgun/eufy_security/blob/master/README.md
Describe the bug
On Step 6 of the instructions, I cannot find the "Eufy Security" addon in HACS:
To Reproduce
Steps to reproduce the behavior:

Go to HACS
Tap "Explore + Download repositories"
Type "eufy"
notice no results

Expected behavior
Step 6 of install instructions indicate i should be able to find this?
Screenshots

Please double read the instructions.
Add-ons are meant to be used inside supervisor.
Integrations are meant to be used inside HACS.

I think @tmchow  meant "repository could not be found in HACS". I had the same issue, but just waited a couple of minutes and then it appeared in the list.

HACS is refreshing all the repositories in a scheduled manner and because of github's limiting, it can take time to get the whole list. It is how HACS is working but you can always install it manually but too much hassle :-)

I made a mistake in my original issue report, @ciryon was right in what I meant. It's now showing, I wasn't waiting long enough.

Please double read the instructions.
Add-ons are meant to be used inside supervisor.
Integrations are meant to be used inside HACS.

What if I don't use supervisor? Is there a way to install add-on somehow manually?


Not running the tests

My file at /home/user/dev/app/test/sum.test.js
const sum = require('../src/sum');

test('adds 1 + 2 to equal 3', () => {
    expect(sum(1, 2)).toBe(3);
});

My file at /home/user/dev/app/src/sum.js
export const sum = (a, b) => {
    return a + b;
};

Error message:
[user@fedora34 app]$ npm run test

><EMAIL_ADDRESS>test
> jest src --config jest.config.json

No tests found, exiting with code 1
Run with `--passWithNoTests` to exit with code 0
In /home/user/dev/app
  11 files checked.
  testMatch: **/__tests__/**/*.[jt]s?(x), **/?(*.)+(spec|test).[tj]s?(x) - 1 match
  testPathIgnorePatterns: /node_modules/ - 11 matches
  testRegex:  - 0 matches
Pattern: src - 0 matches

Ok. I created a new example installation found what I was doing wrong. Looks like I need to name my file *.spec.js and have it in the src folder. It's working now. Thanks anyway. :+1:


SettingsList not taking full width

Can't figure out why this is.
Example:

Code:
Column {
  SettingsList(...)
  SettingsSlider(...)
}

It's working here.
Maybe you have to add Modifier.fillMaxWidth() ?

Anyway, that extra "space" on the right is because of action param, that you can use to add a custom icon (or text, or whatever) as a separate actionable item

I'm confused... action should just be the action that's performed when you make a new selection right? Why would that need to be a composable thing like a button? For a settings list, all you care about is the dialog with your list options, I don't see why it needs a 2nd icon.

Check this example from bluethoot settings.
Action here is the gear part of the settings tile, so you can jave different parts, main setting and secondary

Where is that example code?

You can check the app sample, and there check on MenuLinks


Series REST API support

Series REST API support:

[x] Series list
[x] Series begin
[x] Series forget
[x] Show details
[x] Refactored series and CLI plugins to avoid code duplications
[ ] Tests

Might be a good idea to consider implementing the plugin apis in a separate modules as these are already very large. Maybe plugins/api/series.py which then import series plugin as a dependency?

I was thinking the exact same thing but @stevezau convinced me otherwise. I think it's a good idea myself, same as CLI

Yeah i think we should seperate it as well. I was doing this originally but @gazpachoking convinced me otherwise. I guess it makes sense considering the size of some of the plugins.

To summarize, it's all @gazpachoking fault

I fully agree.

I think it's more restful if instead of using url params you can forget episodes like DELETE /<seriesid>/episodes/<episodeid>/ Maybe we should also add the release level at /<seriesid>/episodes/<episodeid>/releases/

Maybe something at <showid>/begin/<episodeid> for setting begin episode.

I'll try to summarize:

Num
Action
Endpoint
Description

1
GET
/series
Get all series data (with additional params)

2
GET
/series/<name>
Search for series via name

3
POST
/series/<name>
Create a new show and set begin (via body)

4
GET
/series/<show_id>
Get series begin

5
PUT
/series/<show_id>
Set series begin (via body)

6
DELETE
/series/<show_id>
Forget series

7
GET
/series/<show_id>/episodes
Get all series episodes

8
DELETE
/series/<show_id>/episodes
Forget all series episodes

9
GET
/series/<show_id>/episodes/<ep_id>
Get episode details

10
DELETE
/series/<show_id>/episodes/<ep_id>
Forget episode

11
GET
/series/<show_id>/episodes/<ep_id>/releases
Get episode releases

12
DELETE
/series/<show_id>/episodes/<ep_id>/releases/<release_id>
Delete episode release

How does that look? Probably should have started with a table like this to begin with...

With regard to all /episodes endpoints, I really do not have to get show_id as well since episodes have their own IDs. Should I just ignore show_id for those methods or validate it (that the episode received from ID realtes to the show received from ID)?

With regard to all /episodes endpoints, I really do not have to get show_id as well since episodes have their own IDs. Should I just ignore show_id for those methods or validate it (that the episode received from ID realtes to the show received from ID)?

You aren't using SxxEyy as the episode id?

@gazpachoking good point.. Shouldn't we do something like
/show_tv/episodes/season_id/ (list all eps within that season?)
/show_tv/episodes/season_id/ep_id (show ep info)

I don't think that's needed. Season are not an entity in our system and the episodes Metadata by itself is very little. I think it would add unnecessary complexity and lose some of the current uniformity.

Maybe I can add another endpoint that will allow for such lookup. Something like 'series/lookup'

what is the ep_id then? is that the id of the database row?

Yes, it's the db ID. Returned from all the relevant GET actions

I think it should be accessed via sX/eX as that's what makes sense in the real world. No one will care about the internal db primary key.. IE.. someone should be able to go directly to /series/<show_id>/episodes/sXX/eXX without knowing the internal primary key?
I'm assuming show_id is also the internal ID? might need to re-think that as well

I beg to differ. An API is not meant to be used as a user UI and I don't see the need to add complexities that arise from supporting those standards. The user could get all the relevant IDs in one call, why is supporting another standard is even needed?
Don't forget that the SxxExx is only one style of episode ID, there are dates and sequence too.
I honestly think it's more elegant to only support our internal ID, but if I'm in the minority on this I'll gladly change it.

Hmm, this needs to somehow be in a plugins folder rather than in core. Maybe plugins/api until we figure out the full restructure we were discussing.


It seems just support android 4.4 and up.

Is that true? Can you make it to support android 4.0 and up?

It is true, because this tool uses android 4.4 framework features.


docs: Add missing .env instructions to README

Also, prepare project to have another source of API keys than LastPass by replacing <FROM LASTPASS> with <API KEY>, and add note that API keys could be obtained from LastPass.

Actually, all references to POSTNORD_KEY should be removed. It is not used any more.

$ grep -r POSTNORD_KEY
k8s/base/stack/engine-server.yaml:            - name: POSTNORD_KEY
k8s/base/stack/engine-server.yaml:                  key: POSTNORD_KEY
packages/engine-server/lib/parcel.js:const postnordKey = process.env.POSTNORD_KEY
infra/README.md:  kubectl create secret generic postnord-api-key --from-literal=POSTNORD_KEY=<FROM_LASTPASS> -n <NAMESPACE>
README.md:POSTNORD_KEY - for engine-server

Sorry no. It actually is used in engine-server. But then again, I don't think engine-server works with the changes we've made to engine and it should probably all be deleted, together with everything that's not the api, engine or route-optimiser


Two progress bars

Use two progress bars. The first (primary) progress bar is the current one which updates after each successfully exported layer.
The secondary progress bar would be the one that is used by the file format export procedures to continuously update the progress for each file.
The secondary progress bar would be considerably smaller than the primary bar. I did try to make it work, but I didn't manage to disable the text displayed over the secondary bar, which would "leak" outside given the bar's small size.

Dropping this, since it is currently not possible to implement this properly and there is no compelling reason to change the way it works now anyway. The user sees both the overall progress in the dialog and the progress of the current layer at the bottom of GIMP.

It is, after all, possible to implement the secondary progress bar while suppressing the text, by using gimp.progress_install.


Consider doing a video

Might be worth doing a video YouTube introduction to the proposal that you attach to the proposal if you guys have time. There is a lot of very official language and this might soften it with your intentions more clearly defined.

Or if there is already a video consider attaching it to the main readme.md file.

For sure and agreed. This idea also occurred to me and some other folks suggested it. This will happen.


PHOENIX-6444 Extend Cell Tags to Delete object for Indexer coproc

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

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

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

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch.

_ Patch Compile Tests _

-1 :x:
mvninstall
1m 17s
root in the patch failed.

+0
hbaserecompile
12m 10s
HBase recompiled.

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

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

-1 :x:
checkstyle
0m 49s
phoenix-core: The patch generated 25 new + 1115 unchanged - 18 fixed = 1140 total (was 1133)

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

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

+1 :green_heart:
spotbugs
3m 31s
the patch passed

_ Other Tests _

-1 :x:
unit
115m 45s
phoenix-core in the patch failed.

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

154m 44s

Subsystem
Report/Notes

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

GITHUB PR
https://github.com/apache/phoenix/pull/1197

Optional Tests
dupname asflicense javac javadoc unit spotbugs hbaserebuild hbaseanti checkstyle compile

uname
Linux 022c12ba7eb6 4.15.0-128-generic #131-Ubuntu SMP Wed Dec 9 06:57:35 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
master / 958ec62

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

mvninstall
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/1/artifact/yetus-general-check/output/branch-mvninstall-root.txt

mvninstall
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/1/artifact/yetus-general-check/output/patch-mvninstall-root.txt

checkstyle
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/1/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/1/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt

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

Max. process+thread count
10389 (vs. ulimit of 30000)

modules
C: phoenix-core U: phoenix-core

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

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.3

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

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

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

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch.

_ Patch Compile Tests _

-1 :x:
mvninstall
1m 17s
root in the patch failed.

+0
hbaserecompile
11m 53s
HBase recompiled.

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

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

-1 :x:
checkstyle
0m 50s
phoenix-core: The patch generated 18 new + 1122 unchanged - 11 fixed = 1140 total (was 1133)

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

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

+1 :green_heart:
spotbugs
3m 29s
the patch passed

_ Other Tests _

-1 :x:
unit
117m 23s
phoenix-core in the patch failed.

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

156m  1s

Subsystem
Report/Notes

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

GITHUB PR
https://github.com/apache/phoenix/pull/1197

Optional Tests
dupname asflicense javac javadoc unit spotbugs hbaserebuild hbaseanti checkstyle compile

uname
Linux 7f0734794b4b 4.15.0-128-generic #131-Ubuntu SMP Wed Dec 9 06:57:35 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
master / 958ec62

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

mvninstall
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/2/artifact/yetus-general-check/output/branch-mvninstall-root.txt

mvninstall
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/2/artifact/yetus-general-check/output/patch-mvninstall-root.txt

checkstyle
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/2/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/2/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/2/testReport/

Max. process+thread count
10877 (vs. ulimit of 30000)

modules
C: phoenix-core U: phoenix-core

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/2/console

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.3

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

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

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

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
17m  2s
master passed

+0
hbaserecompile
24m  1s
HBase recompiled.

+1 :green_heart:
compile
1m  3s
master passed

+1 :green_heart:
checkstyle
0m 48s
master passed

+1 :green_heart:
javadoc
0m 51s
master passed

+0 :ok:
spotbugs
3m 17s
phoenix-core in master has 960 extant spotbugs warnings.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
9m 53s
the patch passed

+0
hbaserecompile
20m 12s
HBase recompiled.

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

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

-1 :x:
checkstyle
0m 50s
phoenix-core: The patch generated 18 new + 1122 unchanged - 11 fixed = 1140 total (was 1133)

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

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

+1 :green_heart:
spotbugs
3m 28s
the patch passed

_ Other Tests _

-1 :x:
unit
116m 22s
phoenix-core in the patch failed.

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

174m 15s

Reason
Tests

Failed junit tests
phoenix.end2end.PermissionsCacheIT

phoenix.end2end.index.IndexMetadataIT

Subsystem
Report/Notes

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

GITHUB PR
https://github.com/apache/phoenix/pull/1197

Optional Tests
dupname asflicense javac javadoc unit spotbugs hbaserebuild hbaseanti checkstyle compile

uname
Linux 2e28931b7fed 4.15.0-128-generic #131-Ubuntu SMP Wed Dec 9 06:57:35 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
master / 84d317c

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

checkstyle
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/3/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/3/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/3/testReport/

Max. process+thread count
11269 (vs. ulimit of 30000)

modules
C: phoenix-core U: phoenix-core

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/3/console

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.3

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

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

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

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+0 :ok:
mvndep
4m 50s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
12m 41s
master passed

+0
hbaserecompile
24m 27s
HBase recompiled.

+1 :green_heart:
compile
1m 33s
master passed

+1 :green_heart:
checkstyle
1m  0s
master passed

+1 :green_heart:
javadoc
1m  6s
master passed

+0 :ok:
spotbugs
3m 18s
phoenix-core in master has 960 extant spotbugs warnings.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 12s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
9m 58s
the patch passed

+0
hbaserecompile
16m 55s
HBase recompiled.

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

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

-1 :x:
checkstyle
0m 49s
phoenix-core: The patch generated 5 new + 1118 unchanged - 15 fixed = 1123 total (was 1133)

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

+1 :green_heart:
xml
0m  1s
The patch has no ill-formed XML file.

+1 :green_heart:
javadoc
1m  7s
the patch passed

+1 :green_heart:
spotbugs
3m 28s
the patch passed

_ Other Tests _

-1 :x:
unit
115m 27s
phoenix-core in the patch failed.

+1 :green_heart:
unit
2m 15s
phoenix-server in the patch passed.

-1 :x:
asflicense
1m  1s
The patch generated 1 ASF License warnings.

178m 34s

Subsystem
Report/Notes

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

GITHUB PR
https://github.com/apache/phoenix/pull/1197

Optional Tests
dupname asflicense javac javadoc unit spotbugs hbaserebuild hbaseanti checkstyle compile xml

uname
Linux fb179d0badc1 4.15.0-128-generic #131-Ubuntu SMP Wed Dec 9 06:57:35 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
master / 84d317c

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

checkstyle
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/4/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/4/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/4/testReport/

asflicense
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/4/artifact/yetus-general-check/output/patch-asflicense-problems.txt

Max. process+thread count
11696 (vs. ulimit of 30000)

modules
C: phoenix-core phoenix-server U: .

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/4/console

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.3

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

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

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

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
9m 57s
the patch passed

+0
hbaserecompile
20m 17s
HBase recompiled.

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

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

-1 :x:
checkstyle
0m 49s
phoenix-core: The patch generated 5 new + 1118 unchanged - 15 fixed = 1123 total (was 1133)

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

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

+1 :green_heart:
spotbugs
3m 25s
the patch passed

_ Other Tests _

-1 :x:
unit
116m 11s
phoenix-core in the patch failed.

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

175m 31s

Subsystem
Report/Notes

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

GITHUB PR
https://github.com/apache/phoenix/pull/1197

Optional Tests
dupname asflicense javac javadoc unit spotbugs hbaserebuild hbaseanti checkstyle compile

uname
Linux 0da2e36e54a6 4.15.0-128-generic #131-Ubuntu SMP Wed Dec 9 06:57:35 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
master / f4ee13b

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

checkstyle
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/5/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/5/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/5/testReport/

Max. process+thread count
12502 (vs. ulimit of 30000)

modules
C: phoenix-core U: phoenix-core

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/5/console

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.3

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

This message was automatically generated.

@gjacoby126  Could you please review the patch ?
All the unit tests are succeeding but unit test phase is failing with the following error.
[INFO] --- jacoco-maven-plugin:0.8.5:report (report) @ phoenix-core ---
[INFO] Loading execution data file /home/jenkins/jenkins-agent/workspace/enix-PreCommit-GitHub-PR_PR-1197/yetus-general-check/src/phoenix-core/target/jacoco.exec
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 01:55 h
[INFO] Finished at: 2021-04-19T16:52:02+00:00
[INFO] Final Memory: 78M/1632M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.jacoco:jacoco-maven-plugin:0.8.5:report (report) on project phoenix-core: An error has occurred in JaCoCo report generation. Error while creating report: malformed input around byte 2 -> [Help 1]
[ERROR]

@shahrs87 - could you please check that IndexMetadataIT passes locally.

[INFO]
[INFO] --- maven-surefire-plugin:2.22.2:test (default-test) @ phoenix-core ---
[INFO]
[INFO] -------------------------------------------------------
[INFO]  T E S T S
[INFO] -------------------------------------------------------
[INFO] Running org.apache.phoenix.end2end.index.IndexMetadataIT

 [INFO] Tests run: 21, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 214.302 s - in org.apache.phoenix.end2end.index.IndexMetadataIT
[INFO]
[INFO] Results:
[INFO]
[INFO] Tests run: 21, Failures: 0, Errors: 0, Skipped: 0

IndexMetadataIT succeeded locally. @gjacoby126

Thank you @gjacoby126  for the review and merge !!


Cannot Attach to Backend

When I pull from GitHub to a new machine I get errors when doing commands such as:
awsmobile pull
awsmobile console
Where I receive:
backend awsmobile project unknown
Attempting to do:
awsmobile init
Gives an Invalid Signature error
Any ideas?

Curious what you mean by "pull from Github". Could you elaborate.
awsmobile pull and awsmobile console can only be run in a valid awsmobile project. You generally want to run awsmobile init prior to running these commands.
Could you provide a more verbose error output from the Invalid Signature error you are receiving.

Please make sure you are using the same credentials to access aws.
You can do a side by side comparison by
awsmobile configure aws -l
on the two machines and see if they are the same

I have exactly the same credentials with awsmobile and aws cli along with the same npm versions of aws-amplify and aws-amplify-react, what else can I check?


✨[Feature Request]: Enhancement of Blood Bank.html page

issue #580
Description
[Please include a brief description of the changes or features added]
Related Issues
[Cite any related issue(s) this pull request addresses. If none, simply state “None”]

Closes #580

Type of PR

[ ] Bug fix
[X] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
https://github.com/Anishkagupta04/RAPIDOC-HEALTHCARE-WEBSITE-/assets/154777864/b5f6165a-49f7-4189-8a32-b1c1f0b4c496
Checklist

[X] I have gone through the contributing guide
[X] I have updated my branch and synced it with project main branch before making this PR
[X] I have performed a self-review of my code
[X] I have tested the changes thoroughly before submitting this pull request.
[X] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[X] I have commented my code, particularly in hard-to-understand areas.

@varshith257 please review the changes , when we click on the Getlist button in blood.html file , we get details in the page of list.html file.

@varshith257 please review the PR.

@varshith257 please review the PR

Will review it by EOD
On Tue, 18 Jun, 2024, 7:24 am Lakshmi Pavananjali, @.***>
wrote:

@varshith257 https://github.com/varshith257 please review the PR.
—
Reply to this email directly, view it on GitHub
https://github.com/Anishkagupta04/RAPIDOC-HEALTHCARE-WEBSITE-/pull/606#issuecomment-2174763244,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4BF3HC6ZUU6WBKG4SXM7TLZH6HMXAVCNFSM6AAAAABJMZZS26VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZUG43DGMRUGQ
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

@varshith257 okay

Thankyou for your contribution

@Anishkagupta04 also add GSSOC and level labels

@varshith257


Contact List

Hi,
First of all great work !I
I want to list all contact (Hydrogen bonds, ...) from a contact representation. This is referring to issue #782. I have added a comment, but this is not reopening the issue, sorry if I did wrong.
I am not familiar with js, I am wondering is it possible to use the getContactData() function with something like the<EMAIL_ADDRESS>- interactive/ligand-viewer from the gallery.
Here is what I tried :
$ contactRepr.repr.getContactData()

But i have an error "t is undefined"
So I tried :
contactRepr.repr.bufferList[0].picking.contacts.contactStore

I have access to interaction type and indexes of atoms, but I don't know how to link the indexes to the structure indexes, to get for each contact the atom names and residues involved.
Thanks. Samuel

Long overdue reply for historical reference:
stage.loadFile("rcsb://1ajx").then(c => {
  var reprElement = c.addRepresentation('contact')
  console.log(reprElement.repr.getContactData(c.structure))
})


Option to change default sorting

Hi,
It would be great if you can allow changing default sorting (Column, Direction) per each view. Ideally, custom sorting should be saved in a state file so next time I open e1s everything's sorted how I left it.

Thank you! That's true!
A more customizable sorting is preferred. However, k9s-like sorting experience could require some hours to implement. If getting more requests about sorting, I'm open to it.


Doesn't work in Firefox

See https://github.com/AurelienLourot/ghuser.io/issues/42#issuecomment-403884577

Fixed by https://github.com/AurelienLourot/ghuser.io/commit/9b29141f4f16c1a83f79526d37b25013fc735c23


kubectl: init at v1.8.1

Motivation for this change
kubectl is needed to administer kubernetes clusters from the command line.
For google cloud users, it isn't installable through the google-cloud-sdk
derivation - this pulls the precompiled binary and installs it.
As far as I can tell there is no hosted tarball archive for this binary, hence
this binary install approach.
Things done

[ ] 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
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Trying to understand why you're packaging the prebuilt binary vs. building the code yourself. Can you elaborate on that?

As far as I can tell there is no hosted tarball archive for this binary, hence this binary install approach.

Do you mean that they don't host the source anywhere? https://github.com/kubernetes/kubernetes/tree/master/pkg/kubectl would probably work fine with a fetchFromGitHub subdirectory or the like. We also have a full k8s package from @offlinehacker and others that might also include kubectl.

@copumpkin  I just wanted this tool, not the entire kubernetes install as I am simply administering a kubernetes cluster on google container engine.
This is my first nixpkgs PR. I thought seeing as there were a bunch of other basically pre-compiled tools in the nixpkgs set, that downloading this one (despite not being compressed) would also be fine. Happy to build the kubectl binary from the github-hosted source as well if that is the preferred means of building these kinds of derivations.

@jamesthompson we don't have any explicit rule against binary packages but generally prefer builds from source wherever possible.
On the necessity of this vs. the full k8s package, I'm wondering if we could just split the outputs from the main k8s package so folks like you wouldn't need to get the full thing, but we wouldn't be incurring additional maintenance by having two packages with essentially the same tool (but with separate version numbers needing to be bumped regularly, probably getting out of sync, etc.).
@offlinehacker @NeQuissimus any opinion on splitting kubectl into a separate output on the main k8s derivation?

I also need kubectl and currently I'm using
pkgs.kubernetes.override { components = [ "cmd/kubectl" ]; })
and it works without problems.
It's slightly annoying that I have to compile and it takes a long time and lots of memory (I had to add a swapfile just to be able to compile this). But I think it'd be better if we just add this line to all-packages.nix and get binary caches from Hydra without depending on the prebuilt binary.

Could we maybe use another output in the kubernetes derivation to expose kubectl? Then we wouldn't have to rebuild so much go code.

Ping :) I'd love to see kubectl as a separate output that I can pull in without the rest of k8s... Any takers? :D

Is the protocol reasonably stable?

Reasonably, yes...
It is possible to use a newer cluster with our kubectl derivation.
I have something similar to @utdemir that I use because I like to keep the versions in-sync:
    ((pkgs.kubernetes.override { components = [ "cmd/kubectl" ]; }).overrideAttrs (oldAttrs: {
      version = "1.9.0";
      name = "kubectl-1.9.0";
      src = fetchFromGitHub {
        owner = "kubernetes";
        repo = "kubernetes";
        rev = "v1.9.0";
        sha256 = "03383j5n66r5qv3y03g3v9vmvmzhpj3fzparsz1s22x1zzr4qg5a";
      };
    }))

We could just throw that into top-level so that Hydra builds it, I suppose.

Oh, I hadn't seen this PR when I made #34126, but I did basically exactly what @NeQuissimus and @copumpkin said. 😅

I'm going to close this out - given https://github.com/NixOS/nixpkgs/pull/34126 solves the basic need of this PR.


policy=“translation” is failed

Hi， I test you code with two framework, and two dataformat, and the same result is "translation" policy is failed!
Envs:
Python: 3.7
Ubuntu: 1804
CUDA 10.1
Pytorch: 1.4.0
Test1:
I use your code in BMSG-GAN repo and train 3-channel dataset(102 -flower) but only using 755 samples for checking, when the policy included "translation" , and failed to train.

RuntimeError: CuDNN error: CUDNN_STATUS_NOT_SUPPORTED. This error may appear if you passed in a non-contiguous input

but if policy I just used "color,cut", it work fine!
Test2:
In my own GAN  with 1-channel dataset, the performance is the same, if not use "translation" policy it work fine, otherwise it failed

When I use Python: 3.5 this bug is disappeared!!!

I'm trying to fix it, but I cannot replicate the error. Could you help me to verify whether the updated code works now on Python 3.7? Thank you!

maybe we should mention PyTorch/Tensorflow version in the README.

The DiffAugment operations are supposed to work on all common PyTorch/TensorFlow versions. If it doesn't work properly on some version, just let me know!

@zsyzzsoft  this code work fine on Python 3.5， but not work on Python 3.7， Did you try on Python 3.7??

I'm using Python 3.7.6. Have you try the updated code? Could you share me the complete error message, including which line yields the error?

ok， if I occur same problem， I will report full log here ， and I just close this issue temporarily

Has anyone solved this issue? I'm seeing this CuDNN error using Python 3.6.8.

Has anyone solved this issue? I'm seeing this CuDNN error using Python 3.6.8.

Fixed the issue by calling .contiguous() at the end of the rand_translation() function.

Has anyone solved this issue? I'm seeing this CuDNN error using Python 3.6.8.

Fixed the issue by calling .contiguous() at the end of the rand_translation() function.

Thanks! Fixed in master.

Thanks! Fixed in master.


WIP jsonalchemy: clean up of MARC fields definition

See #1722, #2075, #2348, #2651

@egabancho can you please make consistent usage of ' and " in field definition?

Please move contrib inside Invenio package (invenio/contrib).

Unnecessary, see https://github.com/inveniosoftware/jsonalchemy
Only a double check is needed to verify that all the fields and subfields included in this PR are represented.


Automated Package Upgrade

:tada: This PR is included in version 5.0.74 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Obtain dependencies through RequestServices, not through options

I can see that dependencies like ITicketStore, ISystemClock are being set through the options and I am not sure why they are not being set and requested through the RequestServices.
Besides this being the sensible approach, one legitimate reason is the below scenario:

I would like to replace ITicketStore with my own implementation and that implementation rely on another service which is scoped per http request inside the services collection.

:-1: for ITicketStore (as resolving it from the DI container would affect every instance of the cookies middleware in a same application) but :+1: for ISystemClock :smile:

@PinpointTownes you have hit my second frustration: https://github.com/aspnet/HttpAbstractions/issues/492
To me, that should be the part that needs fixing too and ITicketStore should be retrieved from RequestServices unless there is another compelling reason.

This part is far trickier to "fix" than you might think :smile:
We had a very interesting discussion with @MartinJohns a while ago about this specific question and we ended up creating a new set of extensions to support such scenarios (Isolate and IsolatedMap). https://github.com/aspnet-contrib/AspNet.Hosting.Extensions/blob/dev/src/AspNet.Hosting.Extensions/HostingExtensions.cs
We use Isolate in OpenIddict to run MVC 6 in an isolated environment, with its own options (that won't interfere with the default app options): https://github.com/openiddict/core/blob/dev/src/OpenIddict.Mvc/OpenIddictExtensions.cs#L51.
Sadly, since it uses its own environment, using the services defined by the parent builder is not easy and we have to use some "services bridges" to make that work: https://github.com/openiddict/core/blob/dev/src/OpenIddict.Mvc/OpenIddictExtensions.cs#L97-L105

@PinpointTownes yes, it's not easy to change this right now, I get that but it doesn't mean it shouldn't.

ISystemClock is for internal testing purposes only and so we'll ignore that for now.
Regarding DI support in general, that's a good idea, but it's a very large change so we'll revisit this in a future release.

We can revisit this now in 2.0, since Auth 2.0 has moved everything over to DI, ISystemClock for example is now a service.  We could do something similar for ITicketStore.  Clearing backlog for re-triage

We could do something similar for ITicketStore.

Which would necessarily imply that you couldn't use a different store per scheme. A better approach would be to have both an option property and DI injection if the option property is null.

It's not clear which items can really be services because some of them are auth-handler-specific, so resolving them from DI would prevent that. We'll have to see for which specific items this is actually possible.

Consider making cookie validators services?

Moving to 2.1 as this isn't critical for 2.0

Closing because we are not planning on doing this.


[GE] [NME] [WebGL1] 'round' : no matching overloaded function found

For those of us unfortunate enough to still be stuck with WebGL1, round is not a function, and results in the GUI Editor not working, as well as the Round node in the Node Material Editor producing an error.
I'm not sure what the best approach would be, other than perhaps adding to the helperFunctions, as it is included in shaders generated by both
float round(float a){
return floor(a + 0.5);
}
vec2 round(vec2 a){
return vec2(floor(a.x + 0.5),floor(a.y + 0.5));
}
etc.

Hello please use the forum to start discussion about issues (as stated in the issue template)
cc @DarraghBurkeMS


fix gh actions

This reverts #75 and #76 and provides the fallback as proposed in #73
In the future, if there are failing feeds, use the || to shortcut on error
and report an EXIT_STATUS > 1 to the healthchecks.

re-enable help-wanted and lessons in make
allow help-wanted and lessons to error with status 2

Thanks @zkamvar We know it's the lesson feeds that fail the most, but should we consider adding this line to each make line?
I had started to look to see if there is a way to do this for all lines without adding it to each line.

We know it's the lesson feeds that fail the most, but should we consider adding this line to each make line?
I had started to look to see if there is a way to do this for all lines without adding it to each line.

Yes. I think that would be worthwhile.

We know it's the lesson feeds that fail the most

It's the incubator feed that fails the most. The lessons feed has been consistent for the most part until now.

Unfortunately, this PR has reached its rate limit for the hour, so I will revisit this in an hour.


vscode-extensions: updates and new packages

Motivation for this change
Updating versions and adding some new extensions.
Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.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

[x] Fits CONTRIBUTING.md.

Also can we please do something to make this less manual? Having a list of all this metadata would be a start - probably 99% of the extensions have an identical definition structure, others could be appended in a nix file manually. Having this, we could make scripts that automate updating all of this in a git-friendly way.
Happy to implement this if it sounds like a good idea.

Also can we please do something to make this less manual? Having a list of all this metadata would be a start - probably 99% of the extensions have an identical definition structure, others could be appended in a nix file manually. Having this, we could make scripts that automate updating all of this in a git-friendly way.
Happy to start implementing this if it sounds like a good idea.

Long term, I would like to do something similar to the vimPlugins ecosystem
just have a manifest, and it's able to fetch the latest, determine the correct sha's. Then render a generated output. We can then apply an overrides overlay to modify the packages as need to make them nix friendly.


Update README.md

Need to CD into the "vagrant" folder to start it up

good catch

closed as requested

Reminding you to add the web.xml file missing from inside the /vagrant/configs/ folder. I can test once it's in there.

Thanks for the bump...

Ok @evagoras I've added the web.xml and updated Lucee.  Please pull a fresh copy and try again.


python-bindings for Visualization: Capture images during RL training (equivalent to physics.render() von dm_control)

Hey all,
I am currently working on an RL problem for which I developed a Testing Env in MuJoCo and would like to document during the training process the actions e.g. after 10000 steps, 1M steps etc.
The dm_control library had a simple function to basically capture an image of the viewpoint at a certain state ( physics.render() ) which returns an image.
Is their any equivalent way to do it with just the python bindings?
I tried to import the Camera Class from dm_controll which contains the render function, but unfortunately some functions the bindings physics-equivalent do not overlap.
If not, any other ideas how to accomplish the capturing of parts of the training?
Cheers from Spain

The new MuJoCo bindings don't have a convenience class like Camera, so you'll need to use the lower-level MuJoCo API. We'll work on a Colab notebook to demonstrate this.
In the meanwhile, you can look at the test code for python rendering, which shows how to use mujoco.GLContext and the rendering functions: https://github.com/deepmind/mujoco/blob/main/python/mujoco/render_test.py

Awesome thanks!


Ensure ngOnInit has executed before calling updateTitle().

I came into an issue where updateTitle() was sometimes being called before ngOnInit.

@softsimon Thanks a lot, could you push to npm again?


Raspberry PI 5 aarch64: exec /usr/local/bin/docker-entrypoint.sh: exec format error

All images fail to start (hanging on  Waiting for Outline server to be healthy ....)  with this error:
pi@rpi:~ $ docker logs shadowbox
exec /usr/local/bin/docker-entrypoint.sh: exec format error
exec /usr/local/bin/docker-entrypoint.sh: exec format error
exec /usr/local/bin/docker-entrypoint.sh: exec format error
exec /usr/local/bin/docker-entrypoint.sh: exec format error
exec /usr/local/bin/docker-entrypoint.sh: exec format error`

I thought you supported arm64/aarch64?

I got same error too


Update faq.yml

Service has been GA for 12 months

#label:"aq-pr-triaged"

#please-close
Content updated in the private repo for faster publishing.


REFACTOR-#1531: move default_to_pandas into base query_compiler class

Signed-off-by: Anatoly Myachev<EMAIL_ADDRESS>
What do these changes do?

[x] first commit message and PR title follow format outlined here

NOTE:  If you edit the PR title to match this format, you need to add another commit (even if it's empty) or amend your last commit for the CI job that checks the PR title to pick up the new PR title.

[x] passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] signed commit with git commit -s
[x] Resolves #1531
[ ] tests added and passing
[x] module layout described at docs/development/architecture.rst is up-to-date

@modin-project/modin-core is it possible to restart the check of docs?

@dchigarev ready for merge

I don't think we need to move default_to_pandas into the base QC. The base QC should be generic for all storage formats. I think the method should belong to the PandasQC or DefaultPandasQC (D2PandasQC) as we thought earlier.

The base QC should be generic for all storage formats.

default_to_pandas implementation is generic for all QCs, so nothing is broken here I guess.

the method should belong to the PandasQC or DefaultPandasQC

But is there Default2PandasQC though? It seems that currently, the BaseQueryCompiler is the one that handles falling to pandas logic.

So far we don't have a Default2PandasQC but we thought about it at some point. I would prefer to have default_to_pandas in that class because there might be <Default>PyarrowQC that has a default_to_arrow.

So far we don't have a Default2PandasQC but we thought about it at some point. I would prefer to have default_to_pandas in that class because there might be <Default>PyarrowQC that has a default_to_arrow.

I don’t see any options that some of the Query Compilers will not support defaulting to pandas, because for sure not a single execution will be able to implement all the functionality that is in pandas and it will have to redirect some of the functions to pandas.

PyarrowOnEngine, NumpyOnEngine.

PyarrowOnEngine, NumpyOnEngine.

PyarrowOnEngine - This is just an implementation detail of the pandas interface. Or don't you think so?
NumpyOnEngine - Are you talking about the implementation of the NumPy interface? In this case, we will need a separate hierarchy of QueryCompilers for it. Because now this class aimed to work only with Pandas interface: This class defines common query compilers API, most of the methods     are already implemented and defaulting to pandas. That is, it is not basic for everything, but only for the pandas.

PyarrowOnEngine, NumpyOnEngine.

PyarrowOnEngine - This is just an implementation detail of the pandas interface. Or don't you think so?

I don't think so. Pyarrow is another storage format in Modin, which should follow the logic/idea of the storage format concept.

PyarrowOnEngine, NumpyOnEngine.

PyarrowOnEngine - This is just an implementation detail of the pandas interface. Or don't you think so?
NumpyOnEngine - Are you talking about the implementation of the NumPy interface? In this case, we will need a separate hierarchy of QueryCompilers for it. Because now this class aimed to work only with Pandas interface: This class defines common query compilers API, most of the methods are already implemented and defaulting to pandas. That is, it is not basic for everything, but only for the pandas.

We will likely need to implement a QC for the NumPy storage format. Again, at some point we wanted to split the BaseQC into a D2PQC and a Base/GenericQC but haven't done anything on that front yet.

PyarrowOnEngine, NumpyOnEngine.

PyarrowOnEngine - This is just an implementation detail of the pandas interface. Or don't you think so?

I don't think so. Pyarrow is another storage format in Modin, which should follow the logic/idea of the storage format concept.

I see no contradiction to what I wrote.

PyarrowOnEngine, NumpyOnEngine.

PyarrowOnEngine - This is just an implementation detail of the pandas interface. Or don't you think so?
NumpyOnEngine - Are you talking about the implementation of the NumPy interface? In this case, we will need a separate hierarchy of QueryCompilers for it. Because now this class aimed to work only with Pandas interface: This class defines common query compilers API, most of the methods are already implemented and defaulting to pandas. That is, it is not basic for everything, but only for the pandas.

We will likely need to implement a QC for the NumPy storage format. Again, at some point we wanted to split the BaseQC into a D2PQC and a Base/GenericQC but haven't done anything on that front yet.

Likely. I feel like we need a separate issue for this, where we will continue the discussion. Please create it with all your thoughts on this.

What I think Iaroslav is trying to express is that BaseQueryQC should mostly be an abstract class which has nothing to do with exact syntax of library being implemented.
I think we should treat Modin like "LLVM for ML", so modin.pandas is frontend, query compilers are backend (and IR to some extent). So it's basically different abstraction layers, and Modin per se is not designed to be limited to only Pandas API.
So PandasQueryCompiler having a .default_to_pandas() method is some implementation details which are not applicable for other input APIs.
cc @devin-petersohn

What I think Iaroslav is trying to express is that BaseQueryQC should mostly be an abstract class which has nothing to do with exact syntax of library being implemented.
I think we should treat Modin like "LLVM for ML", so modin.pandas is frontend, query compilers are backend (and IR to some extent). So it's basically different abstraction layers, and Modin per se is not designed to be limited to only Pandas API.
So PandasQueryCompiler having a .default_to_pandas() method is some implementation details which are not applicable for other input APIs.
cc @devin-petersohn

Almost all functions in BaseQueryCompiler class are related to pandas. To do this, it will be easier to rename it to something like BasePandasQueryCompiler and create a separate class that will be the base for everything.

https://github.com/modin-project/modin/issues/3636


added optional $namespace param for auth route registration

Useful for anyone wishing to store their Auth Controllers under a different namespace such as /Auth/Frontend. Can be used in routes.php by calling Route::auth('Namespace\To\Controllers');

I would suggest just copying the routes in this case and adding your own namespace. This method is meant to be highly opinionated.

Thanks Taylor. I had a similar thought as well, what are your feelings on an optional argument in the make:auth command to dump the complete routes rather then the shortcut method?


Android Basics: Introduction to Coroutines in Kotlin Playground

Hello,
Hello. I found an error in the 2nd paragraph in the code area.

Hello @NeutrinoWS,
The codelab content and the code snippets were updated, please refer to the link below:
https://developer.android.com/codelabs/basic-android-kotlin-compose-coroutines-kotlin-playground#3


Refactored so endpoint details are managed in-controller

👋 Hello, I was playing around with the codebase, using this wonderful little app to learn some dart/flutter.
The big if/else blocks, and different arrays, caught my eye when browsing since they looked like something that might be tricky to manage over time, with references as strings in multiple places.
As a little task to learn dart, I thought I'd have a go at implementing an alternative option to managing the endpoints/name of the controllers, so that details only have to be managed in a singular place in a somewhat abstract way.
This PR changes things up so that the controllers themselves manage their name and provided methods.
There's then an Endpoints class that manages things at a high level, with a list of controllers.
With this setup, additional endpoints can be added all within the controller. No changes to others files are needed.
If there's a new set of endpoints (with a new controller), it just needs to be added to the array Endpoints class.
Seems to work from my quick bit of testing in an Android emulator. Have tried to keep everything else (including the endpoint/category order) the same as before. Thought I'd create a PR after doing this in-case you'd like this kind of setup.
Please don't feel any pressure to merge this PR though if you're not fond of this approach, this kind of thing can be a matter of preferences, and this is your project. I'm just happy to have learnt some dart via this process.
Also, I have no clue on dart/flutter conventions, so I may not have followed them. If I've done anything you don't understand, feel free to ask.

Hey Dan,
Thanks a lot for looking into this! I was actually trying to achieve something similar to what you've coded, but I only spent a few hours on it, mainly focusing on the API functions. Your PR is a big help, and I'll give it a try. Once I've tested it and everything seems good, I'll go ahead and approve this PR. Appreciate it!

It looks good! Except for one little wrong url. In the chapter controller. I fixed it!
THANK FOR YOUR NEW VERSION! I LOVE IT!


[question] PIXI.AnimatedSprite by V4.X, how to create framesAnimatedSprite

I see the API from PIXI.AnimatedSprite, that use linear array to create AnimatedSprite.
but i want to use  two-dimensional array to create AnimatedSprite.
Official API documentation:

// linear array
let alienImages = ["image_sequence_01.png","image_sequence_02.png","image_sequence_03.png","image_sequence_04.png"];
let textureArray = [];

for (let i=0; i < 4; i++)
{
     let texture = PIXI.Texture.from(alienImages[i]);
     textureArray.push(texture);
};

let animatedSprite = new PIXI.AnimatedSprite(textureArray);

what I want
let framesImages = [
// The first frame
 ['a1.png', 'b1.png', 'c1.png'],
// The second frame
 ['a2.png', 'b2.png', 'c2.png']
]
let animatedSprite = new PIXI.AnimatedSprite(framesImages );
animatedSprite.play()

eg:

thanks!

I would recommend creating your own custom class to handle this. Your custom class can have 2 instances of AnimatedSprite - which all have their own array of textures. And your custom class can then sync up playing both animated sprites at the same time, stopping them, swapping out... whatever you want to do!
But it's not something that can be done just within 1 Animated Sprite

i use canvas to render['a1.png', 'b1.png', 'c1.png']  to png
after use PIXI.Texture.fromCanvas(canvas) to create a texture and use texture  to create new PIXI.Sprite(texture)
last , new PIXI.AnimatedSprite(SpriteList)
function createMergePng (texturesList) {
   let canvas = document.createElement('canvas')
   let context = canvas.getContext('2d')
   texturesList.forEach((layerItem, index) => {
    if (index === 0) {
        canvas.setAttribute('width', layerItem.width)
        canvas.setAttribute('height', layerItem.height)
    }
   context.drawImage(layerItem, 0, 0)
 })
 return canvas
}

let framesPng = []
let framesImages = [
// The first frame
 ['a1.png', 'b1.png', 'c1.png'],
// The second frame
 ['a2.png', 'b2.png', 'c2.png']
]
// 创建帧
framesPng = framesImages.map((pngs) => createMergePng(pngs))
// 创建实体Textures
let framesTextures = framesPng.map(canvas =>PIXI.Texture.fromCanvas(canvas))
let animatedSprite= new PIXI.extras.AnimatedSprite(framesTextures )


addon_before or addon_after does not work on bootstrap4

I have seem a case #72 where this was confirmed working with slight css issue but on bootstrap4 addon text doesn't show at all

Forgive me, it is my fault this actually works


Binds not working and restarting service raises secure keyboard entry error

Environment: OSX Sierra 10.12, zsh; I usually use iTerm2 but have been trying this in Terminal.app just in case. I installed kwm 4.0.2 and khd 1.1.0 through Homebrew and started/enabled them with brew services.  I do not have secure keyboard entry enabled, and kwm and khd are both allowed to control the computer. My configuration is copied verbatim from the kwm examples (including khdrc).
kwm is running; the border is present and follows mouse focus as expected. When I first boot, khd is also running, /tmp/khd.err is empty, and /tmp/khd.out shows it processing the config file. It does cut off suddenly a bit over halfway through, although I'm not sure if that means anything:
Khd: Using config '/Users/dianfay/.khdrc'
Token_Comment:  Enable kwm compatibility mode
Token_Comment:  set color of default mode
Prefix Color: 0xFFD5C4A1
...
Token_Modifier: cmd
Token_Modifier: alt
Token_Literal: 3
Token_Command: kwmc display -f 2
Token_Comment:  Swap Focused Window
Token_Modifier: ctrl
Token_Modifier%

The problem is that none of the binds in khdrc seem to register. I've been trying cmd+ctrl+r to rotate windows and cmd+alt+vi keys to move focus to no avail (although cmd+alt+h is hiding other windows in the current desktop). No further output is produced in either khd.out or khd.err. I did sudo launchctl config user path /usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin from #9 since I'm using an alternative shell, but it does not appear to have affected anything.
The kicker: when I brew services restart khd, Khd: Secure keyboard entry is enabled! Terminating. appears in khd.err, sometimes multiple times. khd.out is empty.

I had the same issue. I was able to fix by restarting, then enabling khd in System Preferences > Security & Privacy > Privacy > Accessibility. I made sure that khd was enabled with brew services start khd before restarting.

That hasn't worked for me, unfortunately. I gave it another try, removing khd from the Accessibility preferences and restarting; when it came back up khd requested access and I granted it, but it failed to start and dumped several secure keyboard entry warnings into the error file. Restarting again returned it to the state where khd is running but not catching key events, with the output file again cutting off at the same place.

I didn't think so, but it turns out that WebRoot SecureAnywhere enables it globally by default. Toggling that option off got the binds working. Thanks!


Some bugfixes

Hi there Balthazar!
First of all, thank you so much for the module, has been very useful for my project!
I´ve made some changes and bugfixes to the module, be free to make the changes you desire. I´m sending to you to be shared with Github´s community ;)
Thanks!

Thanks


Error styling using StyleSheet for Label component

package version :

react : 16.2.0
react-native : 0.52.0
native-base : ^2.3.10

when I create style with StyleSheet in Label component it throws :
TypeError: in this environment the sources for assign MUST be an object. This error is a performance optimization and not spec compliant.
but if I use inline styling it just fine.

@yofriadi unable to reproduce. Attaching a gif.
Gif

@akhil-geekyants here the gif of mine. it will error after I clicked on it.

Fixed with 2.4.1


Find Projects Page - Volunteers needed

When sorting by volunteers needed, shows projects that are not looking for volunteers:

https://helpwithcovid.com/projects/222-spira-s-sms-covid-19-tracker

I can take this one @prophesy604 🙌  I'm signing off now, will start work on it tomorrow

Hey @prophesy604. Looking at the code, this is expected behaviour: When you click 'Volunteers Needed' in the 'Sort by' section, the page loads all the projects sorted by the number of volunteers each project has (ie projects with fewest/0 volunteers will appear towards the top).
I'm guessing we should keep this functionality for users who want to search for projects who need the most help (ie have fewer volunteers).
I'm also assuming what you need as an admin is a filter ie an extra option in the 'Filter by' section to filter in/out projects that have no volunteers. If I'm correct, shall we make this a feature that only admins can use?

It should check if the project has closed volunteers aka volunteers filled. if it has, it should not show those projects when you sort by volunteers needed cause they do not need volunteers. in addition to the existing behaviour of showing fewest/0...
does that make sense? @shen-sat

I just verified that the project details page looks at the person's skillset and shows whether the project is accepting volunteers.
My profile did not have the funding skillset, now that I've added it - the project details page allows me to volunteer:

So since the find projects page sort by "volunteers needed" is using the expected behaviour based on the count.  i don't believe it should go by the skills indicated in the profile on this page. closing ticket.


Feature request: forknames, chainid as options

We have ETC classic team who need this feature. They use different forknames and chainid.
would be nice to have a config file options that specify following:

t8n tool to use (path or smth)
chainid to generate transactions with.
forknames map (testName -> actualName that is sent to t8n)

or support it as cmd options when running the test generation.
this is a request feature low priority.

This would require a separate implementation of the ethereum_test_forks package in order to add the required logic for each fork and adding an interface to select which implementation to use. We discussed internally and decided this is not something we'll add to this repo, but anyone is welcome to fork and add the required logic for their chain.


Implementation in Albert

Hi Moradnejad,
WHile implementing it in ALBERT i am facing the following error
dtype=float32)> has shape (768, 16384), but the saved weight has shape (768, 3072).

Hi,
I assume you are not feeding the same amount of rows.


FAS-50 protected endpoint

Added JWT protected endpoint .../users/me

returns 401 response is the supplied token is invalid (expired or null username)
returns information following UserBase schema if the token is properly decoded

Build: http://localhost:6006/job/FAPI/job/PR-24/1/ finished with status SUCCESS


Docs(README): Add Nix and NixOS install instructions

I sort of closed to other pr somehow.
This has only the two most common ways to use ctpv on NixOS or through Nix, I removed the redundant ones.

Thank you! Sorry for a long response.


Generating an "EtAl" string separately from authors' names with :select filter

I would like to produce an "EtAl" string for titles with multiple authors separately from the authors' names. The expression
[authors1:select=2,1]
works as expected for multiple authors and gives me "EtAl" if the first of the authors’ last names does not contain spaces. However, when there are spaces in the last name, I need to use something like
[authors1:condense:select=2,1]
but this, in turn, omits or 'swallows' the "EtAl" for last names with no spaces.
Report ID:
H99QI4YM-euc

In this case I'd sooner add an [etal=2,EtAl] function to directly do what you want, or a [split=.] after which select would work.

Yes, my earlier attempts are crude hacks. A clean solution with a proper function is certainly to be preferred.

Ugh wait, there's also [editors1], and [etal= would have to know whether you want etal for all authors, or just the editors. [..:split=.] would work though.

Ideally for every title there is a list either of authors or of editors. If there are both, the list of authors takes precedence. If the list contains more than one name (with or without spaces, periods or any other character), I would like to access an "EtAl" string which I can customize. Doable?

Sorry, I only realize now that you are, of course, right. A [etal] function would have to account for both lists (authors and editors). But to avoid further complication, it could be made so that the argument given to the function applies equally to both lists.

I'm thinking it'd be better to just extend auth/ed functions. I could make them auth.etalN etc where auth.ethal is just equivalent to auth.etal2, and auth.etalN+noauthor to do the same but just produce etal when the conditions are met. auth.etal2+noauthor with two authors would produce nothing per the current behavior. I do not want to change the existing behavior. I took the original patterns from JabRef without thinking about them and there are a few that in retrospect I shouldn't have adopted, but I don't want to break existing unless it's a bug.
If you have an idea that generalizes over all these etal functions, that'd be good too.

I thnk it is not so easy to reproduce my custom pattern with the auth / ed functions:
[Last name of first author or editor (normalized, lower case)]
[Initials for first and middle name of first author or editor (lower case, prefixed by ".")]
["ea" in case of multiple authors or editors (prefixed by "."]
[Year or date of publication (no periods, dashes or anything, prefixed by ":"]
[Disambiguation by numbers 1, 2, 3... (prefixed by ":"]
Honestly, I am not sure it is really worth all the work. I am not a heavy Zotero user anyway. I just thought it would be neat to sync my 2000+ BibTeX *.bib file with Zotero and have my usual citation keys. You don't need to go the extra mile just for me.

authors(max=2).discard + authors(1) + authors([2,2],name='%(i)s').prefix('.') + date('%Ym%d').prefix(':')
|
authors(1) + authors([2,2],name='%(i)s').prefix('.') + '.ea' + date('%Ym%d').prefix(':')


[RFC]: improve README examples of math/base/tools namespace

Description
This RFC proposes to improve the README examples of the math/base/tools namespace package. Currently, namespace package READMEs only include minimal examples which are not particularly informative and do not showcase namespace functionality.
Historically, namespace examples were not given much effort due to the rapidly changing nature of the project; however, now that development and organization has settled, it would be good to revisit these packages and add proper examples to better communicate to users how to use namespace contents.
For those wanting to contribute to this issue, you can use the following steps:

Browse through the contents of the math/base/tools namespace and identify interesting functions and packages to showcase.
Update the example code block in the "Examples" section of the README for the math/base/tools namespace and open a PR.
When updating the README, examples should follow existing project conventions (e.g., ES5).
Examples may require functionality from other packages to help make examples more interesting.

Related Issues
None.
Questions
No.
Other
No.
Checklist

[X] I have read and understood the Code of Conduct.
[X] Searched for existing issues and pull requests.
[X] The issue name begins with RFC:.

@Planeshifter I would like to work on this issue


chore: bump Node version to node20

https://github.com/slimtoolkit/slim    , I am read the code  to solve my  problem . "next no find"


Error: NOT SUPPORTED: keyword "id", use "$id" for schema ID

Hello,
I'm currently trying to run the schematics @oktadev/schematics:add-auth command on a React Native project.  However I receive an error: Error: NOT SUPPORTED: keyword "id", use "$id" for schema ID. This is about 3 steps into the follow guide: https://developer.okta.com/blog/2019/11/14/react-native-login, after installing the oktadev/schematics and the angular schematics CLI.

It should work if you use the latest version without a version.
npm i -D @oktadev/schematics

Closing due to lack of response. Please re-open if this is still an issue with the latest version.


Service & method autocomplete sorting

Autocomplete for services & methods are strictly sorted alphabetically. Instead, certain props should be prioritized in the sort based on their popularity. This would reflect the same autocomplete behavior you see in Roblox Studio and is far more intuitive.
Services are sorted as follows:
Players
RunService
TweenService
ReplicatedStorage
ContextActionService
ServerStorage
UserInputService
ServerScriptService
StarterPlayer
Selection
ReplicatedFirst
Workspace
StarterGui
HttpService
HeightmapImporterService
KeyframeSequenceProvider
Teams
CoreGui
TeleportService
DataStoreService
MessagingService
PluginGuiService
LocalizationService
Chat
Stats
Visit
Debris
Hopper
Studio
Geometry
Lighting
AdService
VRService
GuiService
LogService
LuaSettings
StarterPack
TestService
TextService
UserService
VirtualUser
AssetService
BadgeService
GameSettings
GroupService
LoginService
MouseService
SoundService
TimerService
DebugSettings
DraftsService
FriendService
HapticService
InsertService
JointsService
LuaWebService
NetworkClient
NetworkServer
PointsService
PolicyService
ScriptContext
ScriptService
SocialService
StudioService
TaskScheduler
BrowserService
CommandService
CookiesService
DraggerService
GamepadService
PackageService
PhysicsService
PublishService
RenderSettings
SpawnerService
TextBoxService
ContentProvider
DebuggerManager
GamePassService
KeyboardService
LanguageService
MaterialService
NetworkSettings
PhysicsSettings
TextChatService
FlagStandService
UserGameSettings
VoiceChatService
CollectionService
ControllerService
HttpRbxApiService
MemStorageService
TeamCreateService
TouchInputService
VisibilityService
VoiceChatInternal
AssetDeliveryProxy
ClusterPacketCache
MarketplaceService
MemoryStorageService
PathfindingService
PermissionsService
PluginDebugService
AvatarEditorService
GuidRegistryService
ILegacyStudioBridge
MeshContentProvider
NotificationService
ScriptEditorService
VideoCaptureService
VirtualInputManager
ChangeHistoryService
RemoteDebuggerServer
RuntimeScriptService
UGCValidationService
AnimationClipProvider
VersionControlService
ConfigureServerService
HSRDataContentProvider
ProximityPromptService
IncrementalPatchBuilder
PluginManagementService
CacheableContentProvider
ShorelineUpgraderService
SelectionHighlightManager
SolidModelContentProvider
VisibilityCheckDispatcher
SmoothVoxelsUpgraderService
GoogleAnalyticsConfiguration
ProcessInstantPhysicsService
FacialAnimationStreamingService
AnimationFromVideoCreatorService
CoreScriptDebuggingManagerHelper
FacialAnimationStreamingService

Instance methods:
FindFirstChild
WaitForChild
GetActor
Destroy
GetDescendants
GetPivot
IsA
GetChildren
PivotTo
Clone
isA
MoveTo
Remove
GetDebugId
MakeJoints
BreakJoints
GetFullName
GetAttribute
IsAncestorOf
SetAttribute
GetAttributes
IsDescendantOf
ClearAllChildren
FindFirstAncestor
FindFirstDescendant
FindFirstChildOfClass
FindFirstChildWhichIsA
FindFirstAncestorOfClass
GetPropertyChangedSignal
FindFirstAncestorWhichIsA
GetAttributeChangedSignal

I would recommend to anyone with some extra time to review these and ensure no missing props.

Removed services that most probably won't want/need
Removed services that are unlikely to be contested in the sort

Players
RunService
TweenService
ReplicatedStorage
ContextActionService
ServerStorage
UserInputService
ServerScriptService
StarterPlayer
Selection
ReplicatedFirst
Workspace
StarterGui
HttpService
Teams
CoreGui
TeleportService
DataStoreService
MessagingService
Chat
Debris
Lighting
GuiService
LogService
TextService
GamepadService
PhysicsService
TextChatService
VoiceChatService

Could Instance:FindFirstChild() be prioritized above almost everything?
Reasons:

Instance:FindFirstChild is the only practical way to guarantee a child and not a property.
Instance:WaitForChild is not necessary for Instances created locally.
Instance:FindFirstChild is usually necessary for practically checking if a named child exists.

I'd hypothesize it's the most commonly called method on Instances.

Could Instance:FindFirstChild be prioritized above almost everything?
Reasons:

Instance:FindFirstChild is the only practical way to guarantee a child and not a property.
Instance:WaitForChild is not necessary for Instances created locally.
Instance:FindFirstChild is usually necessary for practically checking if a named child exists.

I'd hypothesize it's the most commonly called method on Instances.

Totally agree. FindFirstChild is also sometimes necessary when used in tandem with type checking. I've updated the list to reflect this.


Please update this to refer to NCRONTAB expressions and not CRON expressions

The timer trigger page has been updated recently to remove the confusion regarding this.
https://docs.microsoft.com/en-au/azure/azure-functions/functions-bindings-timer#ncrontab-expressions

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

ID: 24523dbf-010f-b844-5ff0-88192eac8849
Version Independent ID: d7ef5962-d94c-96d0-bf08-1efffc6d402b
Content: Run Background tasks with WebJobs - Azure App Service
Content Source: articles/app-service/webjobs-create.md
Service: app-service
GitHub Login: @ggailey777
Microsoft Alias: glenga

Thanks for the feedback! We are currently investigating and will update you shortly.

#assign:ggailey777

Related Functions article fix was made here: https://github.com/MicrosoftDocs/azure-docs-pr/pull/83428

Thanks @ggailey777.

Did this get fixed or not? The PR link is 404.


[WC-1959]: Update DG2 docs related to MultiSelect

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@samuelreichert why was this closed?

@ConnorLand We have created another story to update the docs for data widgets, including several topics, so this update will also be part of it.


Global inputs for server-routes

🆒 Your use case
Sometimes, we require the ability to use the same input in different route or retain the input's value after restarting the server.
🆕 The solution you'd like
which of the following is better in terms of use?

method A: we can implent the global inputs with the local input and by clicking on add button user can select the type of input

method B: we can implent the global inputs in a drawer and have all the global inputs in one place and user can select the inputs tab(query, params, e.g.) from there

thanks
🔍 Alternatives you've considered
No response
ℹ Additional info
I think method B is more helful, maybe. method A is more beautiful though it can be a pain for someone that doesn't use global inputs, it needs to select local each time that wants to add input

I think we could go with method B and:

Group by type, instead of selecting the type. So we will have multiple Add buttons in each section
We could list global params in normal request's params with disabled state, to indicating they are inherit from the global state

@antfu Sure! great idea, thanks


Added basic development enviroment setup instructions to USAGE.md

I am testing Tilex locally and I wrote this on the way. Thanks for sharing this project with us ;)

Hey! Thanks a lot for this pull request and for the kind words.
Most if this information is contained in the README: https://github.com/hashrocket/tilex/blob/master/README.md. We’d like to keep that as the place where setup instructions are maintained.
Was that file difficult to find? If so, would you be willing to add a link to it in lieu of these changes? If you did find it, was there a particular part of the instructions you found lacking?
Thanks again!

Hey man, thanks for replying. You're right, I just realized that after writing this thing down...and I understood what happened: When I firstly cloned the repository I saw this USAGE file with a text like "Here's a basic guide", I immediately thought "here, where?", so I wrongly just assumed that it did not exist and started to write it down while I was doing the setup...
I think the actual PR would put a link to README.md in USAGE.md and probably only add the part about changing the database name in config/dev.exs file when needed.
I will close this! Thanks!


Not sure what's going on with the text colors since the switch to the custom color picker

What

Not sure what's going on with the text colors since the switch to the custom color picker (they all turned black)

Part of

#1171

Screenshot/Mockup/Before-After

Seems to be a regression after https://github.com/openfoodfacts/smooth-app/pull/1881


bug: problems with broken links crawler workflow

Describe the bug
(A) I tried to apply it to the current contents of the branch named main, and nothing happened. See https://github.com/kubestellar/kubestellar/actions/runs/8484373434 . Perhaps it is because the condition at https://github.com/kubestellar/kubestellar/blob/fc7cebf2f21ffa640064e021b9604a5226af97bf/.github/workflows/broken-links-crawler.yml#L20 fails when the workflow is manually dispatched.
(B) Currently the latest run is https://github.com/kubestellar/kubestellar/actions/runs/8484036554, which I think happened after the conclusion of the latest run of the build-and-push workflow. But the crawler issued complaints that seem invalid. For example, the crawler complained:

Error: ClientResponseError: 404 - https://docs.kubestellar.io/main/direct/examples/user-guide.md found by navigating through: https://docs.kubestellar.io/main/ -> https://docs.kubestellar.io/main/direct/examples/

but I can not find a link to "anything/user-guide.md" when I view the source of https://docs.kubestellar.io/main/direct/examples/ . For example:
mspreitz@mjs13 kubestellar % curl https://docs.kubestellar.io/main/direct/examples/ | grep user-guide
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  111k  100  111k    0     0   952k      0 --:--:-- --:--:-- --:--:--  960k
mspreitz@mjs13 kubestellar % echo $?
1

Steps To Reproduce
See above
Expected Behavior
(A) The workflow can be manually dispatched.
(B) The right errors are reported.
Additional Context
No response

@clubanderson


Cannot create new organization in Docker image with TomEE

[Version]
OSCM 17.4.0
Docker image with TomEE
[How to reproduce]
Login as Platform Operator
Go to Create organization
Fill up all of neccesary fields
Click Save button
[Expected result]
New organization is created
[Observed result]
Red message "The operation failed."
The organization is not created
[Details]
I have tried a few different configuration of "Organization role". In each cases I have received the red communicat.
I have not noticed any error messages in the log files.

The problem is caused by not possibility to send an emails. After proper configuration of email server it should be fixed.
@kwodzynski test it again please.

Tested on Docker image PRIVATE-BUILD-ON-DEC4B557CD2B2017/09/28 (OSCM 17.4.0).
If the email server is properly configured, the problem does not occur. Closing.