PHOENIX-6120 Change IndexMaintainer for SINGLE_CELL_ARRAY_WITH_OFFSET indexes

@kadirozde @swaroopak @ChinmaySKulkarni @gjacoby126 @tkhurana

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
6m 26s
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.

_ 4.x-PHOENIX-5923 Compile Tests _

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

+1 :green_heart:
mvninstall
11m 43s
4.x-PHOENIX-5923 passed

+1 :green_heart:
compile
1m 29s
4.x-PHOENIX-5923 passed

+1 :green_heart:
checkstyle
2m  0s
4.x-PHOENIX-5923 passed

+1 :green_heart:
javadoc
1m 54s
4.x-PHOENIX-5923 passed

+0 :ok:
spotbugs
4m 25s
root in 4.x-PHOENIX-5923 has 1000 extant spotbugs warnings.

+0 :ok:
spotbugs
2m 54s
phoenix-core in 4.x-PHOENIX-5923 has 946 extant spotbugs warnings.

_ Patch Compile Tests _

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

+1 :green_heart:
mvninstall
6m  5s
the patch passed

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

+1 :green_heart:
cc
1m 29s
the patch passed

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

-1 :x:
checkstyle
2m  1s
root: The patch generated 140 new + 5206 unchanged - 37 fixed = 5346 total (was 5243)

+1 :green_heart:
prototool
0m  1s
There were no new prototool issues.

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

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

+1 :green_heart:
spotbugs
7m 42s
the patch passed

_ Other Tests _

-1 :x:
unit
183m 35s
root in the patch failed.

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

238m 16s

Reason
Tests

Failed junit tests
phoenix.end2end.IndexToolForNonTxGlobalIndexIT

phoenix.tx.TransactionIT

phoenix.end2end.SystemCatalogRollbackEnabledIT

phoenix.end2end.UpsertSelectIT

Subsystem
Report/Notes

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

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

Optional Tests
dupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile cc prototool

uname
Linux 1bd7751993ab 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
4.x-PHOENIX-5923 / 9f5a0bf

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-986/2/artifact/yetus-general-check/output/diff-checkstyle-root.txt

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

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

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

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

modules
C: phoenix-core . U: .

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

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.3 prototool=1.10.0-dev

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

This message was automatically generated.


chore: Updating the Changelogs

:tada: This PR is included in version 1.17.0-alpha.10 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.17.0-alpha.10 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.6.0-alpha.1 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Added asdf-secrets-manager to infrastructure

I thought about this again, but isn't the name of this repository a little too generic? I think something like the bitwarden secrets manager would be better.

That's right, I was thinking about it too, i'm gonna fix the naming all around and update pull requests

btw, do I also have to submit a PR to asdf-plugins repo to add the shortname or is it sufficient to follow the guidelines for this community repo?

Anyway, thanks @FIAV1!


Render error on /variant/2-169132617-A-G

What you did:

What you expected to see after you did that:

What you actually saw after you did that:

Hi @dcweeks , I had a look at this page and didn't see anything clearly wrong with it, do you have any more details about what you saw?

Yes it appears OK to me now too.
Seems like the page started to render but then failed — the start of rendering content went away  to some message encouraging one to report it.  Sorry I didn’t have a chance to add more info at the time but I was in a hurry.
I’m trying to remember but I think it cleared after I restarted the browser (chrome on macOS) and tried again.

On Oct 8, 2024, at 22:42, phildarnowsky-broad @.***> wrote:
Hi @dcweeks https://github.com/dcweeks , I had a look at this page and didn't see anything clearly wrong with it, do you have any more details about what you saw?
—
Reply to this email directly, view it on GitHub https://github.com/broadinstitute/gnomad-browser/issues/1637#issuecomment-2400784165, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIPO5MDVCQUIEM4CVJIPCWTZ2Q7UHAVCNFSM6AAAAABPNKG7ZSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBQG44DIMJWGU.
You are receiving this because you were mentioned.

Thanks for filing this issue,
Since we cannot replicate this, and it appears OK now to you, we believe this was a transient error, and will close this issue.
Thanks!


Update taxonomy.csv files to latest

For now - these should be extracted from the latest spreadsheet shared by @jeremymiller and placed in src/dendrograms following the naming convention used for the mouse taxonomy.
@jeremymiller:  Any changes in the mouse table should be reflected in an updated dendrogram JSON file.

@jeremymiller currently we have a tab called 'child_cell_set_accessions' in https://github.com/obophenotype/brain_data_standards_ontologies/blob/master/src/dendrograms/nomenclature_table_CCN202002013.csv that has the 'cell_set_accession' of all the child nodes - was wondering if you could provide something like that? current spreadsheet you have passed to us only has cell_set_label (RNAseq XXX) to label which are the child nodes. @dosumis just to check, we did get that file and it is not generated by us by reading off the excel sheet right?
Also, I've just noticed these nodes missing and are now in cell_set_additional_alias.
MY-projecting L5 ET		RNAseq 070-071	CS202002013_260
Non-MY-projecting L5 ET		RNAseq 072-073	CS202002013_261
It is correctly represented now in terms of the RNAseq child nodes (I rmbr requesting that change), but just want to double check that this is correct that the two accession should be deleted.

I've placed the new spreadsheet into a new branch:  https://github.com/obophenotype/brain_data_standards_ontologies/blob/16Jun21-Dendrogram-Update/src/dendrograms/cell_set_nomenclature.xlsx

Thanks.  Please split the cell_set_nomenclature sheet into individual csvs files following this naming scheme:
nomenclature_table_CCN202002013.csv
Thanks!

@dosumis I assume you mean for the other taxonomies?

For all - presumably nomenclature_table_CCN202002013.csv has changed

Commit for all other taxonomies here:
https://github.com/obophenotype/brain_data_standards_ontologies/commit/3b4bd3602853a9ae676e584be60baf3960c684ea
Currently still missing child_cell_set_accessions - will update once I get them

fixed with https://github.com/obophenotype/brain_data_standards_ontologies/commit/151b7569c948b51d2a4b092eeace982a70014c48


Add friends

Implement backend for sending reuests to users and a when accepted the users become friends.

hey @haseebzaki-07  the backend is already implemented for sending and receiving requests. I am not able to understand what you are trying to say ? can you elaborate further ? 😅

@AshuKr22 The backend is implemented but is not working as expected neither the users are displayed nor the request is going out on the frontend. I want to fix it

@AshuKr22 The backend is implemented but is not working as expected neither the users are displayed nor the request is going out on the frontend. I want to fix it

Hi beans, So our friend request works like linkedin.  A sent request to B. Then B would be having a pending friend request decision on pending tab. But the A would not be having any visuals where he can saw all the sent request. according to this, it's working fine, I hope it's working fine for your local machine. Do let us know anything you found interesting.

Okay I understood the functioning but there is a bug , User when sent out the request should see sent request for a particular user , instead it is showing send request. @sid0000007 @AshuKr22


BREAKING CHANGES: cheerio to linkedom

Reduce dependencies. Will break the transform function use cheerio.

@SettingDust ok, it looks like moving from jQuery to native DOM. I will check your pr now.

Why doesn't it close automatically?


A feature under the menu strip to be able to refresh package lists.

A feature under the menu strip to be able to refresh package lists. As I couldn't uninstall WinRAR from ChocolateyGUI and had to us the Windows Programs and Features to uninstall it. And ChocolateyGUI didn't update the lists as the uninstall didn't take place from the ChocolateyGUI itself.
Contributor Edit:
To sum, add button to refresh local package information. (RS)

Added a couple new buttons to provide the requested functionality:

@gep13 Yay! :D


Fixes the Issues

Issues In:

Alignment of Contact Us
Extra Padding in Footer section in Copyright block.

Issue Number : #210
Please add labels gssoc-24-ext and hacktoberfest

@mdazfar2 sir pls review my PR asap and Merge it

Sir pls add labels

done @rupeshv2121.

Hope you star this repo


Consider porting the codebase to ByteSpan

The whole codebase needs to be ported to ByteSpan. Otherwise it's not possible to make UTF8 encoding of small strings competitive with native code implementation, as there are too many calls to pin and unpin the byte[] buffers.
For this to happen, ByteSpan needs to be made public and BufferPool needs to probably use native memory instead of pinning. I am not sure if these tradeoffs are worth it.
Alternatively, I would need to wait for a real GC tracked Span before seeing great runtime perf.

I did this experiment. The performance improvement was very significant. Without this change, formatting lots of small value types is much faster than current CLR StringBuilder/TextWriter. But simply writing text to these is slower. After converting the codebase to ByteSpan, literally everything (for which I have perf tests) started to be faster.
But the change has a very significant side effect: it forces many APIs to be disposable and if they are not disposed it causes memory (native memory) leaks.
And so my conclusion is that we need a ByteSpan that can be backed by an interior pointer to a GC memory.
BTW, this was a conclusion reached by other similar investigations, so this is not a novel discovery, but rather another confirmation of the need of a real CLR-built-in array slice.


Treeview not rendering schema on 1.79.2

Expected Behavior
Tell us what should happen: I should see a collection of schemas that the Apicurio resistry shows in its UX
Current Behavior
Tell us what happens instead of the expected behavior: As expected, I see the 3 section titles in the left panel of VSCode. The top panel has no text, just a chevron that seems to open infinitely, but never shows any text for the item
I am running 1.79.2 of VSCode and the latests release version of Apicurio Schema Registry.
Image of empty treeview(s):

Hi snoop244
Apologies I missed the Github notification, I'm not working on delivery at the moment.
That's a strange behavior I've never seen before.
Let me re-install a dev env and check with the latest VSC and Apicurio image.
I'll keep you posted.

Hi,
I'm running VSC :
Version: 1.81.1 (Universal) Commit: 6c3e3dba23e8fadc360aed75ce363ba185c49794 Date: 2023-08-09T22:20:33.924Z (1 mo ago) Electron: 22.3.18 ElectronBuildId: 22689846 Chromium: 108.0.5359.215 Node.js: 16.17.1 V8: <IP_ADDRESS>-electron.0 OS: Darwin arm64 22.6.0
and apicurio :
2.4.7.Final
docker run -it -p 8080:8080 apicurio/apicurio-registry-mem
I face any issues at all, see bellow :

Could you give more context ? or procedure to reproduce the issue ?


Virus sfvip player?

The only way to realistically distribute software nowadays is to buy an EV code signing certificate. It only costs ~500€per year...
Some security providers will even charge you money to whitelist your code. That's their business model.
So that's what it is...

the latest  <IP_ADDRESS> is very very worst ...:
https://www.virustotal.com/gui/file/6cfba998d314e3d5226692d3b479c33135cbae35df10c6257a6329d755c92795
->12  RED.....<-

23 security vendors and no sandboxes flagged this file as malicious
why the olders vers. has really not this very bad score...??...:(

Please check https://github.com/sebdelsol/sfvip-all/issues/10#issuecomment-1914278349

i have -portabilized- yr installer cause i really do want to use as much portable sw as i can,hope you understand,
so maybe this is the reason of the different md5...and i really do suggest you to offer also a portable way of yr tool....
and i re-suggest you to compile yr tools ,as in the past, in order not to have that so really bad virsutotal score...
sanyway the worst thing is that ALSO my av malwarebytes really do not ALLOW me to use yr tool in the new <IP_ADDRESS>....so this is why i'm trying to make you thinking deeply and carefully about this....
up to now i'm friendly use yr <IP_ADDRESS>...and in my -portable- way....
it is really all up to you if you want yr potentially users be confident in using yr tool
best regards

and i really do suggest you to offer also a portable way of yr tool....

You can build your own portable version as explained here.

i re-suggest you to compile yr tools,as in the past, in order not to have that so really bad virsutotal score...

This has nothing to do with compilation but with the signatures of exes produced by either Nuitka or Pyinstaller.
As it has already been explained both are leveraged by malware creators hence both suffer from lazy AV heuristics.
Those false positives are so random they often changes on a daily basis.

it is really all up to you if you want yr potentially users be confident in using yr tool

This repo is freely shared: What people want to do with it is entirely up to them.

up to now i'm friendly

You keep spamming the issues and now you're quite entitled: I've put a stop to that.


Unable to sent templated emails without locale using SMTP

The Freemarker template resolver adds _subject and _content suffixes to provided template name (should only add this if the locale was provided)

The temporary workaround I've used is to create _content and _subject ftl, in order to be able to send the emails.


RDP Wrap Win10 Home - Access denied

I need to remote access a computer for a course I'm attending, but Microsoft wants me to upgrade to Pro och Enterprise (not happening!)
I have installed RDP Wrap. Get all green on the RDPConf and when I run RDPCheck I get "Access denied".
I have:

disabled Win Defender
added the RDP Wrap file paths to my anti-virus exceptions
given full access to C:\Program Files\RDP Wrapper (including rdpwrap.dll).
Still nothing.

Could someone pls help?

See #391.

same wrong, doesn't help to me @binarymaster


Support containerd and nerdctl

Is your feature request related to a problem? Please describe.
containerd and nerdctl are alternative to moby and the docker cli.
Describe the solution you'd like
Add an option to allow users change the container engine between containerd and moby
Describe alternatives you've considered
Additional context
Rancher Desktop supports this.

You can install containerd and nerdctl in a Linux machine for now.


First try to build repository locally

Hi, this is my first attempt to rebuild locally the lab repository
the rebuild option of vs fails
'
1>Downloading runtime.native.System 4.3.1
1>Downloading System.Text.RegularExpressions 4.3.1
1>Something went wrong while downloading DevExpress.ExpressApp.TreeListEditors.Web 21.1.5
1>Message: The package DevExpress.ExpressApp.TreeListEditors.Web 21.1.5 can't be found in C:\Program. (The directory doesn't exist.)
1>Please check the feed definition in your paket.dependencies file.
'
Dx is installed in the default path, but paket restore is having some issues with the space in the path.

paket.dependecies needs to modified to match your sources
here how azure does it
https://github.com/eXpandFramework/eXpand/blob/230ec8a0a8b5ada2b18aa7c1d26513047cd10a53/.github/Azure-pipeline.yml#L33

so use your DXApiFeed instead

also to clarify a bit more for all, if the build does not pass on azure it most problably won't work in your local machine although it worked on mine

Closing issue for age. Feel free to reopen it at any time.
.Thank you for your contribution.


Postgres: relationship with hasMany can't be cleared

Link to reproduction
https://github.com/janisvb/payload-reproduction/tree/empty-relation
To Reproduce
export PAYLOAD_DATABASE=postgres
yarn dev _community

Describe the Bug
When having a relationship field with hasMany: true, it is not possible to clear the value:
{
  name: 'authors',
  type: 'relationship',
  relationTo: 'users',
  hasMany: true,
}

Clearing the value programmatically does not work
const post = await payload.create({
  collection: postsSlug,
  data: {
    text: 'example post',
    authors: [user.id],
  },
})

// not working
await payload.update<Post>({
  collection: postsSlug,
  id: post.id,
  data: {
    authors: [],
  },
})

// also not working
await payload.update<Post>({
  collection: postsSlug,
  id: post.id,
  data: {
    authors: null,
  },
})

Also manually clearing the value in the Admin UI does not work.
Payload Version
2.0

Thanks @Janisvb — on it now


Transducer loss leads to memory leak

Hi, I'm using your transducer loss codes and pytorch-lightning to train my model. But I found the 4D tensor which is used to calculate transducer will be accumulated in GPU, when I check the GPU memory in training step(before a batch starts), there are many 4D tensor(come from the previous batches) in the GPU memory. That will lead to CUDA out of memory finally. I don't know what went wrong.

gpu_tracker is used to check the GPU memory.

The loss in training step is from this.

This is the result of GPU memory usage in training step.
I try to use 'del', 'gc.collect()' and 'torch.cuda.empty_cache()' in everywhere, but they are all useless.

Torch should soon support natively the transducer loss and we will switch to it. However, this behavior shouldn't be expected. Are you sure that the increase in memory isn't due to sorting: ascending in your config file ?

Hi @TParcollet , thanks for your reply.
I have changed the codes in figure 1 to the codes in figure 2, and it works. I'm not sure if different versions of pytorch, lightning and so on lead to this problem.

figure 1

figure 2

Any insight here @aheba?

@YangHao97 did you see any speed degradation with your new code/ ?

Closing due to lack of activity here, but reopen if there's followups.


BrowserSync and Local by flywheel problems

npm starts browser sync is trying to access
https://localhost:3000/ and resulting in
localhost unexpectedly closed the connection.
I have
// Project options.
projectURL: 'alonlive.local', - that is my local domain and it works fine when accessing directly.
npm - 6.4.1
CLI version 3.9.1
Local version 3.9.1
Thanks!
If you'd like us to keep producing professional free and open source software (FOSS). Consider paying for an hour of our dev-time. We'll spend two hours on open source for each contribution. Yeah, that's right, you pay for one hour and get both of us (Awais & Maedah) to spend an hour as a thank you.
🚀 $99.99 — Support for one hour or more →
🔰 $49.99 — Support half an hour maintenance →
☕ $9.99 — Buy us lunch or coffee to keep us trucking #OpenSource →

SSL is known to cause such issues. Try to disable that?

@AKdigital  This is a known issue with the .local domains on OSX. For all new sites, I recommend using .test or local.yoursite.com.
Issue with .local is it has some loop back that causes connection issues.

I'm having issues with this also and would like to know how to fix it.

The BrowserSync is confusing. I'm using XAMPP at localhost/wp_vs_532 and put that in the config file. But hot reloads weren't working. I noticed the message about port 3000 and tried localhost:3000/wp_vs_532 and now it works. I know Apache is on ports 80 and 443 so it seems like wpgulp takes the input url and spits out updates over port 3000 of that url???

Updating Local to the latest version might help.
https://localwp.com/community/c/releases/11


[UI "Підтримай проєкт" modal window] Verify that after entering a donation amount equal to the amount shown on the button, the button turns red

Priority: low
Preconditions

The "Підтримай проєкт" modal window is opened.

Steps to reproduce                                  /                                               Expected result

Type the donation amount of 50              /                                               The 50 button is red
Type the donation amount of 100            /                                               The 100 button is red
Type the donation amount of 500            /                                               The 500 button is red

User story: #136 #78
Related bug:

Add the epic and the page where it should be checked at the beginning in the [].


Official curl ... | bash install not adding pixi to PATH when ~/.zshrc missing

Checks

[X] I have checked that this issue has not already been reported.

[X] I have confirmed this bug exists on the latest version of pixi, using pixi --version.

Reproducible example
curl -fsSL https://pixi.sh/install.sh | bash

Issue description
Running the official installation command on macOS is not adding pixi to the PATH.  I've tried on 3 different Macs tonight, some new (M1) some old, they all have the same issue.
Each one is using the default zsh shell.  Running curl -fsSL https://pixi.sh/install.sh | bash (or even curl -fsSL https://pixi.sh/install.sh | zsh) results in ~/.pixi/bin/pixi being installed, but the PATH environment variable isn't updated.
Debugging this a bit, I notice that machines where people haven't done much setup before don't have ~/.zshrc, so it's skipping updating PATH.
Expected behavior
I'd expect pixi to become executable.  I'm trying to get new users working with pixi, and this is a blocker for people who are new, and will assume the install failed.
Looking at the install script, I wonder if update_shell() should be improved to create the file if it's missing?
update_shell() {
    FILE=$1
    LINE=$2

    # Create the file if it doesn't exist
    if [ ! -f "$FILE" ]; then
        touch "$FILE"
    fi

    # Append the line if it's not already in the file
    if ! grep -Fxq "$LINE" "$FILE"
    then
        echo "Updating '${FILE}'"
        echo "$LINE" >> "$FILE"
    fi
}

He @humphd, good catch! feel free to make a PR with your fix!
Nice to hear that the first thing you do on a new system is install pixi :wink: !


Add-Migrations missing -IgnoreChanges

I have extracted an existing MySql database from a remote server following following Getting Started with EF Core on ASP.NET Core with an Existing Database
I am using EF Core 2.0 and EF Core Tools using VS 2017 on Windows.
I am now trying to add a table to this database following Migrations.
When I run
Add-Migration InitialExpanse
I get the error
Table 'already_existing_table' already exists
but
Add-Migration InitialExpanse -IgnoreChanges
Add-Migration : A parameter cannot be found that matches parameter name 'IgnoreChanges'.
At line:1 char:30
+ Add-Migration InitialExpanse -IgnoreChanges
+                              ~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidArgument: (:) [Add-Migration], ParameterBindingException
    + FullyQualifiedErrorId : NamedParameterNotFound,Add-Migration

How do I deal with this issue?

Okay, I have a workaround by commenting out the relevant code in the Up and Down methods in the migration.   But -IgnoreChanges would be nice for capturing existing state from an already existing database.

Just a note that 12/10/2018 -IgnoreChanges does not seem to work yet (i could be doing something wrong). But was following (basically) https://docs.microsoft.com/en-us/ef/ef6/modeling/code-first/migrations/existing-database which indicates to use -IgnoreChanges
If it helps anyone, commenting out the code INSIDE the up method and running update-database eventually can get the EF process into "sync" and then you can go from there. At least in Dev.


Verification

I'm one of the contributors to https://github.com/damus-io/damus and I'm excited to learn more about Bitcoin developments.
https://github.com/damus-io/damus/commit/9e9077bab518d73a3cdf9df6c32bb6aca834fa55

I received the ticket and attended the conference, so I'm closing this issue.


Help Intent and corresponding dialog

I think this has been done, but check.

It is already there


Automatic Mixed Precision does not work in hub.KerasLayer

According to the documentation, I tried to use Automatic Mixed Precision (AMP) in tensorflow 2.0 in keras style. Here is my code:
#!/usr/bin/env python
# coding: utf-8
from __future__ import absolute_import, division, print_function, unicode_literals
import tensorflow_hub as hub
import tensorflow.keras.mixed_precision.experimental as mixed_precision
import tensorflow.keras.layers as layers
import numpy as np
import tensorflow as tf

# we can use mixed precision with the following line
policy = mixed_precision.Policy('mixed_float16')
# policy = mixed_precision.Policy('float32')
mixed_precision.set_policy(policy)
print('Compute dtype: %s' % policy.compute_dtype)
print('Variable dtype: %s' % policy.variable_dtype)
num_samples = 1024
batch_size = 16
max_seq_len = 128
num_class = 16
epochs = 3
vocab_size = 30522

# BERT_PATH = 'https://tfhub.dev/tensorflow/bert_en_uncased_L-12_H-768_A-12/1'
BERT_PATH = '../input/bert-base-from-tfhub/bert_en_uncased_L-12_H-768_A-12'

def bert_model():
    input_ids = tf.keras.Input((max_seq_len,), dtype=tf.int32, name='input_ids')
    input_masks = tf.keras.Input((max_seq_len,), dtype=tf.int32, name='input_masks')
    input_segments = tf.keras.Input((max_seq_len,), dtype=tf.int32, name='input_segments')

    bert_layer = hub.KerasLayer(BERT_PATH, trainable=True)

    print('bert_layer._dtype_policy:', bert_layer._dtype_policy)
    print('bert_layer._compute_dtype:', bert_layer._compute_dtype)
    print('bert_layer._dtype:', bert_layer._dtype)

    _, bert_sequence_output = bert_layer([input_ids, input_masks, input_segments])

    print("bert_sequence_output.dtype:", bert_sequence_output.dtype)

    x = layers.GlobalAveragePooling1D()(bert_sequence_output)
    logits = layers.Dense(num_class, name="logits")(x)

    print("logits.dtype:", logits.dtype)

    # when using mixed precision, regardless of what your model ends in, make sure the output is float32.
    output = layers.Activation('sigmoid', dtype='float32', name='output')(logits)
    print('output.dtype:', output.dtype)

    model = tf.keras.models.Model(inputs=[input_ids, input_masks, input_segments], outputs=output)
    return model

# make dummy inputs
train_X = []
train_X.append(np.random.randint(0, vocab_size, size=(num_samples, max_seq_len)))  # ids
train_X.append(np.zeros(shape=(num_samples, max_seq_len)))  # masks
train_X.append(np.zeros(shape=(num_samples, max_seq_len)))  # segments
train_Y = np.random.randn(num_samples, num_class)  # labels

model = bert_model()
optimizer = tf.keras.optimizers.Adam(learning_rate=1e-4)
model.compile(loss="binary_crossentropy", optimizer=optimizer)
model.fit(train_X, train_Y, epochs=epochs, verbose=1, batch_size=batch_size)

What I expect :
bert_sequence_output.dtype should be float16, because it is the output of a layer (i.e. the bert_layer) that uses the mixed_float16 policy.
But what I actually get:
the code above tells me  bert_sequence_output.dtype is float32, and here is the full log:
ssh://xiepengyu@<IP_ADDRESS>:22/home/xiepengyu/miniconda3/envs/tf2/bin/python -u /home/xiepengyu/google_quest/scripts/multi_bert_aug_mixed_precision_test.py
2020-01-05 11:30:50.951010: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudart.so.10.1
2020-01-05 11:30:51.380306: W tensorflow/stream_executor/platform/default/dso_loader.cc:55] Could not load dynamic library 'libnvinfer.so.6'; dlerror: libnvinfer.so.6: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /home/xiepengyu/cuda/cuda-10.1/lib64:$LD_LIBRARY_PATH
2020-01-05 11:30:51.380387: W tensorflow/stream_executor/platform/default/dso_loader.cc:55] Could not load dynamic library 'libnvinfer_plugin.so.6'; dlerror: libnvinfer_plugin.so.6: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /home/xiepengyu/cuda/cuda-10.1/lib64:$LD_LIBRARY_PATH
2020-01-05 11:30:51.380399: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:30] Cannot dlopen some TensorRT libraries. If you would like to use Nvidia GPU with TensorRT, please make sure the missing libraries mentioned above are installed properly.
2020-01-05 11:30:52.292392: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcuda.so.1
2020-01-05 11:30:52.635553: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1555] Found device 0 with properties:
pciBusID: 0000:03:00.0 name: GeForce GTX 1080 Ti computeCapability: 6.1
coreClock: 1.582GHz coreCount: 28 deviceMemorySize: 10.92GiB deviceMemoryBandwidth: 451.17GiB/s
2020-01-05 11:30:52.635599: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudart.so.10.1
2020-01-05 11:30:52.637236: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcublas.so.10
2020-01-05 11:30:52.638264: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcufft.so.10
2020-01-05 11:30:52.638493: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcurand.so.10
2020-01-05 11:30:52.640188: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcusolver.so.10
2020-01-05 11:30:52.641278: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcusparse.so.10
2020-01-05 11:30:52.644628: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudnn.so.7
2020-01-05 11:30:52.650678: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1697] Adding visible gpu devices: 0
2020-01-05 11:30:52.650998: I tensorflow/core/platform/cpu_feature_guard.cc:142] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
2020-01-05 11:30:52.658229: I tensorflow/core/platform/profile_utils/cpu_utils.cc:94] CPU Frequency:<PHONE_NUMBER> Hz
2020-01-05 11:30:52.658878: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x562d05824cc0 initialized for platform Host (this does not guarantee that XLA will be used). Devices:
2020-01-05 11:30:52.658896: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): Host, Default Version
2020-01-05 11:30:52.871435: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x562d058cb200 initialized for platform CUDA (this does not guarantee that XLA will be used). Devices:
2020-01-05 11:30:52.871481: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): GeForce GTX 1080 Ti, Compute Capability 6.1
2020-01-05 11:30:52.875039: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1555] Found device 0 with properties:
pciBusID: 0000:03:00.0 name: GeForce GTX 1080 Ti computeCapability: 6.1
coreClock: 1.582GHz coreCount: 28 deviceMemorySize: 10.92GiB deviceMemoryBandwidth: 451.17GiB/s
2020-01-05 11:30:52.875109: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudart.so.10.1
2020-01-05 11:30:52.875137: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcublas.so.10
2020-01-05 11:30:52.875149: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcufft.so.10
2020-01-05 11:30:52.875161: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcurand.so.10
2020-01-05 11:30:52.875172: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcusolver.so.10
2020-01-05 11:30:52.875183: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcusparse.so.10
2020-01-05 11:30:52.875195: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcudnn.so.7
2020-01-05 11:30:52.876635: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1697] Adding visible gpu devices: 0
2020-01-05 11:30:53.444364: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1096] Device interconnect StreamExecutor with strength 1 edge matrix:
2020-01-05 11:30:53.444427: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1102]      0
2020-01-05 11:30:53.444436: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1115] 0:   N
2020-01-05 11:30:53.450671: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1241] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 10392 MB memory) -> physical GPU (device: 0, name: GeForce GTX 1080 Ti, pci bus id: 0000:03:00.0, compute capability: 6.1)
Compute dtype: float16
Variable dtype: float32
bert_layer._dtype_policy: <Policy "mixed_float16", loss_scale=DynamicLossScale(current_loss_scale=32768.0, num_good_steps=0, initial_loss_scale=32768.0, increment_period=2000, multiplier=2.0)>
bert_layer._compute_dtype: float16
bert_layer._dtype: float32
bert_sequence_output.dtype: <dtype: 'float32'>
logits.dtype: <dtype: 'float16'>
output.dtype: <dtype: 'float32'>
Train on 1024 samples
Epoch 1/3
2020-01-05 11:31:06.079381: W tensorflow/core/common_runtime/shape_refiner.cc:88] Function instantiation has undefined input shape at index: 1161 in the outer inference context.
/home/xiepengyu/miniconda3/envs/tf2/lib/python3.7/site-packages/tensorflow_core/python/framework/indexed_slices.py:433: UserWarning: Converting sparse IndexedSlices to a dense Tensor of unknown shape. This may consume a large amount of memory.
  "Converting sparse IndexedSlices to a dense Tensor of unknown shape. "
2020-01-05 11:31:08.348584: W tensorflow/core/common_runtime/shape_refiner.cc:88] Function instantiation has undefined input shape at index: 1161 in the outer inference context.
2020-01-05 11:31:18.719649: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libcublas.so.10
1024/1024 [==============================] - 34s 33ms/sample - loss: 0.0720
Epoch 2/3
1024/1024 [==============================] - 15s 15ms/sample - loss: 0.0185
Epoch 3/3
1024/1024 [==============================] - 15s 15ms/sample - loss: 0.0042

Process finished with exit code 0

When I change the policy to float32, the several print give me these info (other parts of log is the same as mixed_float16):
Compute dtype: float32
Variable dtype: float32
bert_layer._dtype_policy: <Policy "float32", loss_scale=None>
bert_layer._compute_dtype: float32
bert_layer._dtype: float32
bert_sequence_output.dtype: <dtype: 'float32'>
logits.dtype: <dtype: 'float32'>
output.dtype: <dtype: 'float32'>

According to the log, here is my conclusion:

The mixed_float16 policy dose work in other custom layer, e.g. the Dense layer named "logits", because its output has a dtype float16.
The policy of the Bert layer has been set to mixed_float16, but somehow it seems like it is not working judging from the dtype of bert_sequence_output.dtype is float32. Another evidence is the GPU memory usage (which is dominated by the variables in Bert layer) is nearly the same in both cases.

Personally, I think this is due to the layer definded in Bert have been hard-coded to have a dtype float32, so we cannot use mixed_float policy to change its behavior. Is it right? What else could have caused the problem and how to fix it?
Thanks to all kindly help in advance!

Hi @iamxpy, thanks for reaching out!
Yes, you are right that loading a SavedModel (via hub.KerasLayer or otherwise) brings in its ops with the data types set at serving time, so it cannot be changed on the fly by tf.keras.mixed_precision policies (although model publishers could offer various precisions).
I have updated our documentation in commit https://github.com/tensorflow/hub/commit/27357e928160f0a29890de6aa56c656919547aa7 to reflect this.


feat: deepsource analyzer init command

Signed-off-by: Siddhant N Trivedi<EMAIL_ADDRESS>

Oops..Please don't review as of now. Will add more tests.


move scheduled tasks out of ResourceAdmin and into their own modules.

Moving scheduled tasks away from ResourceAdmin.
This PR makes scheduled tasks live inside their own modules.
# config.exs
config :kaffy,
  # other settings
  scheduled_tasks: [
    MyApp.Kaffy.Tasks
  ]

# tasks module
defmodule Bakery.Kaffy.Tasks do
  def task_cache_product_count() do
    %{
      name: "Cache Product Count",
      initial_value: 0,
      every: 300,
      action: fn _v ->
        count = Bakery.Products.count_products()
        {:ok, count}
      end
    }
  end

  defp get_product_count() do
    Bakery.Products.count_products()
  end
end

Any function inside the module that starts with task_ is considered a scheduled task.

Well done!
I like this is separated, that would avoid to overload resource Admin with too much code.

Part of the refactoring process. I'm changing how the current :resources option works partly following the discussion on #37 . Felt that tasks don't necessarily belong to schemas.


[Bug][Flink-Submit][dev-2.1.4]Fixed security issues with submitting external applications to manipulate the Jvm

…xternal applications to manipulate the Jvm  FLINK

What changes were proposed in this pull request
Issue Number: close #xxx

Brief change log

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Does this pull request potentially affect one of the following parts

Dependencies (does it add or upgrade a dependency): (yes / no)

reference ：https://github.com/apache/incubator-streampark/pull/3659


deps: update generator dependency to 0.8.4

Fixes: https://github.com/sdd/kiddo/issues/182

Pulls in the changes from https://github.com/Xudong-Huang/generator-rs/issues/64


Reuse Vault setup from other projects

Describe the issue
We need a good cloud master account management system with roles we can assign out.
Which Sprint Priority is this issue related to?
The 'Milestone' should clearly list Sprint Priorities - which one is this issue related to?
Additional context
Add any other context, attachments or screenshots
Definition of done
Vault setup code customized for cloud team needs
Settings stored in GitHub
TFA pattern established (eg only BC Gov phones)

Notes from discussion with Todd.  Requires refinement in order to estimate.  Might be unnecessary if we go the Terraform Cloud path and use their hosted Vault.


Dumb typo: AppLayout.objects not object

This message isn't used anywhere, but realized this mistake while implementing the server code

@prbot approve

👌


Optimistic updates

Hi!
I'm new to Svelte and Liveview, and I'm wondering if the current architecture of live_svelte would allow for easy handling of optimistic updates (which, from what I read, the lack of is one of the main downsides of pure live view due to high latency risks affecting the UX).
Thanks!

Yes it's possible
Let's say you have a variable x that's synced with LiveView. So you'd have export let x. Then you can update that variable in the client by doing x = 1. But note that won't be synced to the server. So you'd have to update it with by sending a message to liveview. For example: live.pushEvent("set_x", {x: 1})
The first step, updating it in the client, would be the optimistic update. After pushEvent is handled by LiveView, it would override the client again with the new variable, which hopefully would be the same as you updated in the client so the user doesn't see it change.
Hopefully that makes sense

Thanks for elaborating!


Feature request: reuse an existing browser tab when ng serve -o is run

Bug Report or Feature Request (mark with an x)
- [ ] bug report -> please search issues before submitting
- [x] feature request

Command (mark with an x)
- [ ] new
- [ ] build
- [x] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Current behaviour
When ng serve -o is run, it always opens a new browser tab.
Desired functionality

If there is an existing localhost tab, ng serve -o could reuse it instead of opening a new one, similar to what create-react-app does (OS X Chrome-only).
I'd love to try to implement this feature, so let me know if you're interested 😃

@danielriodejaneiro, not yet, as there are considerations I'd like to discuss first.
First of all, this feature will only be available to OS X Chrome users (AFAIK it's the majority of devs). Second, not everyone uses Chrome for development, so we need a way to specify their preferred browser, e.g. with --browser option. For example, if the option is not present, we do our best to reuse an existing tab in Chrome, otherwise fallback to the current behaviour.
So I'd like to get some feedback on these points from maintainers (and from anyone else) before I start working on this. Is it worth the effort?

I'm not too keen on this myself for a couple of reasons:

it's not intuitive to expect -o works differently in OSX
making this the default can disrupt existing workflows that have multiple localhosts
adding a new flag to control this seems excessive (we try to avoid new flags as much as possible)
the alternative isn't so bad (ng serve and refresh)

I can imagine that it could be handy to do something like ng serve -o -r, but have a hard time thinking it's worth it with the caveats and maintenance.
Do you have some insight on how that feature is implemented, and if it could support other OSs?

Both of those unfortunately only add specific behavior on OSX and then only on chrome.
Ideally you would run ng serve --open and leave the server running for an extended period of time.  I agree with @filipesilva that the limited browser/OS support combined with the long maintenance cost strongly limit the overall utility of such a feature at this point in time.


Moved laravel/framework to require-dev

@lucadegasperi I don't understand why you would want to always require laravel/framework. This choice prevents us from using this great library with Lumen, simply because you cannot install both Lumen and Laravel in the same project. I've moved laravel/framework under require-dev to solve this problem.
This might also fix issue #351 (at least it fixes the composer dependency error) but I have yet to put it all together so I cannot say for sure.

Because when I started working on this version L5 was still in early development and stuff changed names too quickly. Putting L5 in the require-dev section doesn't solve the problem.

@lucadegasperi how drastic changes would be required to use this library with Lumen?

Don't know yet, haven't had the chance to test it. anyways, right now i'm updating the composer.json file to pull in only the illuminate components needed.

Improved the composer.json file.

It still seems that laravel is part of the required packages.

It is not, only illuminate components are.

Oh sorry, must have been cache. It seems to be fine now.


Browser support

The official driver runs in the browser. Is there any way to get rethinkdbdash to run in the browser, perhaps with Browserify?
Currently I get 'Error: Cannot find module '/node_modules/rethinkdbdash/lib/helper.js' when requiring rethinkdbdash and building with Browserify.

Perhaps a library akin to minimongo would be more appropriate for browsers. I'd even be happy with the client side version allowing you specify a query handler which would let you handle queries however you want
// actual api should be something that looks more natural than this
r.setQueryHandler(function (query) {
   return sendQueryToBackEndAndReturnPromise(query); // query being a JSON serialized version of any ReQL query after `.run` is called
});

Server side:
app.post('/api/', function (req, res) {
    var query = r.reqlFromJSON(req.query);
   query.run(); // you would ideally do some security checks on req.query before you unserialize and run it
});

This issue is about a browser directly connecting to RethinkDB via HTTP instead of using simple TCP connections.
If you want a mini-mongo, you can hook https://github.com/neumino/reqlite with rethinkdbdash. Reqlite implements most of the term baring the geo-spatial ones and changefeeds I think.

Ah sorry I misread the issue! If I'm understanding correctly rethink only supports HTTP officially via the endpoint for the admin interface, correct? If that's the case I'd say still have the driver generate whatever queries it wants to execute and let the user define a function that sends the actual request. Opens the door for other transports (web sockets, backend that does some additional security filtering or proxies the request safely to the backend ajax endpoint without exposing the web interface to the world, etc).
I know that's a tall order and really would require making the "transport" a pluggable part of the driver but I think it could lead to some interesting things. Transport being a single function that takes a query and returns a promise for the result. Would even make it easy to add support for weird environments like Chromecast via the message bus API or something

Closing as duplicate of #11


Moving workflow file into templates dir

Notes for Reviewers
This PR fixes #26

[ ] Yes, I signed my commits.

Good work, @suhail34 👍

Looks Good to Me . Will merge once the dependent change is merged in meshery-cloud and released.

Here you go: https://github.com/layer5io/meshery-cloud/pull/1136/files

Changes merged in Meshery-Cloud, now just after the release, this PR is good to merge.


setup tailwind css

add tailwind css to the application with all possible imports and tailwind configurations.
bonus addons: example to show how can we override tailwind configurations globally

@vkushal28 bro, @devham236 is gonna add a form using Tailwind css.
@devham236 if you want, you can raise a separate PR for this as well!

@Keerat666 I have already integrated tailwind css in the project while making the rating component #28 Kindly review that

Can you assign this to me!


Add Power Architecture Specific IPMI Tests.

Reviewed 6 of 6 files at r1.
Review status: all files reviewed at latest revision, 9 unresolved discussions, some commit checks failed.

common/OpTestIPMI.py, line 566 [r1] (raw file):

    def ipmi_get_bmc_golden_side_version(self):
        print "IPMI: Getting the BMC Golden side version"
        l_rc = self._ipmitool_cmd_run(self.cv_baseIpmiCmd +

will this command boot the bmc from golden side immediately or does it only set the bmc to boot from golden side when the user issues the reboot command?
Please specify what it returns (Example should suffice)

common/OpTestIPMI.py, line 579 [r1] (raw file):

    def ipmi_get_pnor_partition_size(self, i_part):
        print "IPMI: Getting the size of %s PNOR Partition" % i_part
        if i_part == "NVRAM":

"NVRAM" should be a constant

common/OpTestIPMI.py, line 582 [r1] (raw file):

            l_rc = self._ipmitool_cmd_run(self.cv_baseIpmiCmd +
                                          BMC_CONST.IPMI_GET_NVRAM_PARTITION_SIZE )
        elif i_part == "GUARD":

same comment as above

common/OpTestIPMI.py, line 585 [r1] (raw file):

            l_rc = self._ipmitool_cmd_run(self.cv_baseIpmiCmd +
                                          BMC_CONST.IPMI_GET_GUARD_PARTITION_SIZE )
        elif i_part == "BOOTKERNEL":

same comment as above

common/OpTestIPMI.py, line 590 [r1] (raw file):

        else:
            l_msg = "please provide valid partition eye catcher name"
            print l_rc

please specify an example in the doygen tag for l_rc. Helps in understanding the code

common/OpTestIPMI.py, line 597 [r1] (raw file):

    # @brief This function is used to check whether BMC Completed Booting.
    #
    # @return l_res @type str: output of command or raise OpTestError

example

common/OpTestIPMI.py, line 609 [r1] (raw file):

    #        Fault RollUP LED      0x00
    #
    # @return l_res @type str: output of command or raise OpTestError

example

common/OpTestIPMI.py, line 699 [r1] (raw file):

        l_rc = self._ipmitool_cmd_run(self.cv_baseIpmiCmd +
                                      BMC_CONST.IPMI_ENABLE_FAN_CONTROL_TASK_THREAD)
        print l_rc

not necessary to print everything. Let the user decide what to print and what not to print

common/OpTestIPMI.py, line 719 [r1] (raw file):

    #        Ex: ipmitool raw 0x3a 0x13
    #
    # @return l_state @type str: raise OpTestError when fails

example of l_state

Comments from Reviewable

Review status: 0 of 6 files reviewed at latest revision, 9 unresolved discussions.

common/OpTestIPMI.py, line 566 [r1] (raw file):
Previously, saqibkh wrote…
> will this command boot the bmc from golden side immediately or does it only set the bmc to boot from golden side when the user issues the reboot command?
>
> Please specify what it returns (Example should suffice)
>

This command won't change any BMC boot functionality, It will just retrieve BMC golden side firmware version

common/OpTestIPMI.py, line 579 [r1] (raw file):
Previously, saqibkh wrote…
> "NVRAM" should be a constant

All these are addressed in latest revision.

Comments from Reviewable

Reviewed 6 of 6 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Shall we move all instructions on "How to Set Up X" to a single place?

Things like:
How to set up docker/odk
How to install ROBOT
How to install Protege with ELK 0.5
are useful in their own rights, and IMO should be added in a single place, rather than scattered around the course units. Is there any objection to this? @beckyjackson @jamesaoverton @nicolevasilevsky

no objections!


Python 3 / Deprecation Warning removal / Django 1.9 support

Why did you write Django 1.9? The newest version of Django is 1.8 now.

On Tue, Apr 7, 2015 at 9:23 AM, shellfly<EMAIL_ADDRESS>wrote:

Why did you write Django 1.9? The newest version of Django is 1.8 now.

Django-master is on 1.9
--
http://yourlabs.org http://blog.yourlabs.org
Customer is king - Le client est roi - El cliente es rey.


style(runtime): set aria-hidden to true to the splite for the accessi…

…bility reason
The sprite is inserted immediately below the body element, but all elements immediately below the
body must be either landmarks or invisible for the blind users. Please see this document for the
detail. https://dequeuniversity.com/rules/axe/3.2/region?application=axeAPI
What kind of change does this PR introduce? (bugfix, feature, docs update, improvement)
improve accessibility
What is the current behavior? (You can also link to an open issue here)
The splite is inserted immediately below the body element and is not marked as aria-hidden  so that blind people's experience goes bad and some accessibility tools (react-axe) cause a warning like below.

What is the new behavior (if this is a feature change)?
Set aria-hidden=true to the splite
Does this PR introduce a breaking change?
No
Please check if the PR fulfills contributing guidelines
Linted, tested and followed the rules of commits 😺

@kisenka Do you have an indication on when this PR could be reviewed and potentially merged?
I was trying to add this via plugin configuration, however the sprite is not updated with the aria-hidden attribute or the config given in the plugin.
new SpriteLoaderPlugin({
      plainSprite: true,
      spriteAttrs: {
        'aria-hidden': 'true',
      },

But when giving a step backwards thought to this, the issue happens to everyone using this loader, and ideally the sprite is hidden from screen readers for everyone, otherwise the screen reader user will hear image, image, image, image and so on, for all the elements inside the sprite.

Hi @kisenka what do you think about it?
Please merge and release this accessibility improvment.


Fix possible bug in Resampler

Hello, thansks for your excellent work.

Hello, thanks for your excellent work!
When we read the code, we found that the calculation of Resampler's cross attention, k  seems to be only position embedding without information of the input images. We suppose this may be counterintuitive.

At the same time, we referred to some other code implementations about Resampler, and we found this process may require image information to participate in the calculation of cross attention.

We are not sure if this is a bug, if not, please ignore this PR.


feat: update /member-permissions link to /security in docs site

the member-permissions path is changing in the main app to /security. The content is still the same as the old page, but the path is updated.
Updating the links in docs so we don't break anything 😁
For more context, see this PR: https://github.com/buildkite/buildkite/pull/12278
This Bascamp post with response from Buzz: https://3.basecamp.com/3453178/buckets/11426308/messages/6184949967

so confused how did we miss this https://github.com/buildkite/docs/pull/2614/files

closing this as @JuanitoFatas opened another PR resolving this.

geez sorry, I clean forgot about this 😓


Please do not put AppImage into a gz or other archive

Please do not put PowerChat-linux.glibc2.14-x86_64.AppImage into a .gz
While it may be tempting to avoid users having to set permission, this breaks desktop integration with the optional appimaged daemon, among other things. Besides, the beauty of the AppImage format is that you never need to unpack anything. Furthermore, packing an AppImage into some form of archive prevents the AppImage from being added to the central catalog of available AppImages at https://github.com/AppImage/AppImageHub.
Reference:
https://github.com/AppImage/AppImageKit/wiki/Creating-AppImages#common-mistake

thank you.
I put the appimage on another site: https://gitee.com/rocket049/releases, it will be damaged after download.
So, I put it into a gz to avoid damage.

What?! Please explain. Putting the AppImage on another site should not damage it.

Can you upload a pure AppImage so that I can test please? Thanks.

you can try it now:  https://gitee.com/rocket049/powerchat/releases/v1.3.18

@probonopd It seems they fix the probrom, the new appimage is not damaged.

It needs a login to download, hence cannot run this automated test...

@probonopd I will upload appimage ,as you wish.

Getting
powerchat: symbol lookup error: powerchat: undefined symbol: gtk_menu_popup_at_pointer

On which distribution was this built? Can you build on Ubuntu trusty or xenial?

@probonopd try powerchat-debian，it can run on debian stable.

@probonopd The new appimage  has replace gtk_menu_popup_at_pointer with gtk_menu_popup.
You can try it again.

Thank you 👍
Can you test what happens when the machine is not connected to the internet? It should at least show a graphical dialog box saying you need to go online. Thanks.

You can use command line to read error message.

Not if you double-click the icon, then you can't see the message.


Add PLaMo-13B model as an LLM example

Thank you for developing this amazing library, MLX. This PR adds an example of operating a Llama 2-based model, named PLaMo using MLX, which is trained on both Japanese and English languages in a balanced manner. PLaMo is open source, and its weights are distributed on Hugging Face. This example includes scripts for downloading and converting the weights of PLaMo, and for generating text using the converted model. Please see the detail of PLaMo here: https://tech.preferred.jp/en/blog/llm-plamo/
This example is created in a directory separate from llms/hf_llm and llms/llama because PLaMo includes some differences from Llama implementation in several aspects. For example, replacing the rotary positional encoding used in PLaMo with nn.RoPE implemented in MLX did not work properly, so I had to implement it with more primitive operators (it implies something is different from the original RoPE also in the original PLaMo code). Additionally, the implementation of grouped query attention was slightly different from what is present in llms/llama/llama.py, and PLaMo adopted the "parallel layers" technique in its transformer block, so the architecture itself is slightly different from the original Llama 2. It was also necessary to modify the names of variables within the model to directly load the weights distributed on huggingface.
Due to these reasons, although much of the code is common with code in llms/llama, I created new scripts for this model based on the original implementation: modeling_plamo.py in a separate directory at llms/plamo.
I would appreciate it if you could kindly review this.

This is cool! Thanks for the contribution! At this point I think it makes more sense to start putting new models in mlx-lm rather than as standalone models. Would you be open to moving it there?

Sure! Just as I started working on this PR, the introduction of mlx-lm coincided in timing, so I adopted a strategy of separating it into a different directory. However, now it seems more appropriate to include it under mlx-lm. I will make such changes to this PR as well.

Thank you!! Let me know when it's ready for review!

@awni I have updated it to a state ready for review. Could you please take the time to review this PR? I would greatly appreciate it.

This looks great, and thanks for moving it to MLX LM.
Just to recap @mitmul, @mzbac will add the tokenizer config option in #312 to support the Qwen model in MLX LM after the new Transformers package is out.
After that you can rebase this on #312 after its merged and then we can land this as a quick follow up!
Let me know if I missed anything! Will be great to have these two new models, especially excited about the multilingual support 👍

Hi! #312 has landed! Would you mind rebasing this on main and then we can merge it also?

Hi! #312 has landed! Would you mind rebasing this on main and then we can merge it also?

Thank you for letting me know it, @awni . I merged the latest main and this PR became a simple one that almost just adds PLaMo model code and update README. I'd appreciate it if you would check this PR again.


code refine some pipelineX code

Proposed changes
The basic functionality of pipelineX has been completed. Next, we need to remove unused code, add comments, to enhance readability.
test pr
https://github.com/apache/doris/pull/28622
https://github.com/apache/doris/pull/28632
https://github.com/apache/doris/pull/28651

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


staging-next 2023-05-02

According to 23.05 schedule, this this iteration is the last one allowed to contain breaking changes (except desktop environments).
Helpful links

workflow docs: https://nixos.org/manual/nixpkgs/unstable/#submitting-changes-commit-policy
important jobs:
https://hydra.nixos.org/job/nixpkgs/staging-next/unstable#tabs-constituents
https://hydra.nixos.org/job/nixos/staging-next-small/tested
jobsets:
https://hydra.nixos.org/jobset/nixpkgs/staging-next
https://hydra.nixos.org/jobset/nixos/staging-next-small
nix-review reports: https://malob.github.io/nix-review-tools-reports/
previous staging-next: https://github.com/NixOS/nixpkgs/pull/228476

Mass breakages
(will be edited based on progress)

[x] python311Packages.pytest-benchmark (all platforms but 3.11 only)
[ ] notmuch
https://hydra.nixos.org/build/218564125#tabs-buildsteps

Perhaps notmuch maintainers could look into its build regression?  @flokli @puckipedia

T670-duplicate-mid: Testing duplicate message ids
 PASS   at most 1 thread-id per xapian document
 PASS   search: first indexed subject preserved
 BROKEN First subject preserved in notmuch-show (json)
        --- T670-duplicate-mid.3.expected       2023-05-05 08:31:05.952133649 +0000
        +++ T670-duplicate-mid.3.output 2023-05-05 08:31:05.952133649 +0000
        @@ -2,6 +2,7 @@
             [
                 [
                     {
        +                "crypto": {},
                         "date_relative": "2001-01-05",
                         "excluded": false,
                         "filename": [
        @@ -12,7 +13,7 @@
                         "headers": {
                             "Date": "GENERATED_DATE",
                             "From": "Notmuch Test Suite ",
        -                    "Subject": "message 1",
        +                    "Subject": "message 0",
                             "To": "Notmuch Test Suite "
                         },
                         "id": "XXXXX",
 PASS   Search for second subject
 BROKEN Regexp search for second subject
        --- T670-duplicate-mid.5.EXPECTED       2023-05-05 08:31:05.980133836 +0000
        +++ T670-duplicate-mid.5.OUTPUT 2023-05-05 08:31:05.984133862 +0000
        @@ -1,3 +0,0 @@
        -MAIL_DIR/copy0
        -MAIL_DIR/copy1
        -MAIL_DIR/copy2
 PASS   search for body in duplicate file

So the gnupg patch-update (in 122e7746ee7) certainly does break notmuch build (on aarch64-linux).  No idea why.  There were at least two different notmuch breakages in the bisected span (even though it's just over several days), so it's perhaps more complicated.

notmuch: relevant parts of test logs (thanks to Lily):
T350-crypto: Testing PGP/MIME signature verification and decryption
 PASS   emacs delivery of signed message via fcc
 PASS   emacs delivery of signed message via fcc and smtp
 PASS   signed part content-type indexing
 PASS   signature verification
 PASS   detection of modified signed contents
 PASS   corrupted pgp/mime signature
 PASS   signature verification without full user ID validity
 PASS   signature verification with signer key unavailable
FATAL: /build/notmuch-0.37/test/T350-crypto.sh: interrupted by signal 15

[...]
T357-index-decryption: Testing indexing decrypted mail
FATAL: /build/notmuch-0.37/test/T357-index-decryption.sh: interrupted by signal 15

Well, the bad tests are skipped in notmuch now, but I think it would be still nice if someone looked into that...

[ ] sway test times out when waiting for gpg to show "Passphrase"  # Test gpg-agent starting pinentry-gnome3 via D-Bus (tests if
  # $WAYLAND_DISPLAY is correctly imported into the D-Bus user env):
  swaymsg("exec gpg --no-tty --yes --quick-generate-key test")
  machine.wait_until_succeeds("pgrep --exact gpg")
  machine.wait_for_text("Passphrase")
  machine.screenshot("gpg_pinentry")
  machine.send_key("alt-shift-q")
  machine.wait_until_fails("pgrep --exact gpg")


Allow to scan range of frequencies

Original firmware allows scan only entire band. I think it will be neat feature to scan band partially.
Example:
https://baofengtech.com/scan-a-range-of-frequencies-with-your-uv-5x3/

See how that works for you. You need to set upper and lower frequency as beginning and end of a scan range. Long press 5, long press *.
https://github.com/egzumer/uv-k5-firmware-custom/suites/18540315300/artifacts/1076405937

Fixed by force push

It does not, but there were other changes that raised the memory usage.

That was fast. Thanks! Found minor bug in UI with scan range (#158)


Update README.adoc

Merged.  Thanks!
I think developers' productivity is the correct form for a possessive plural but I tweaked the wording after the merge anyway to make it clearer. Thanks again!


How to use it on vue js ? Please help

use a method inside your script tags that is triggered by a button or something like that


Use a config file for aws creds or read them from an existing ~/.aws dir used for aws cli

Avoid having credentials used as cli parameters that will potentially be stored in shell history presenting a security risk.
Have the option to use a config file or fetch creds from any existing aws cli ~/.aws dir configs
https://docs.aws.amazon.com/cli/latest/userguide/cli-chap-getting-started.html

Or does boto handle this already?

Haven't confirmed if ~/.aws/credentials work, but env vars do so would bet this is closable.

Can confirm this already works.


Alpha 3.0.0-next.68 test --coverage enters watch mode

Is this a bug report?
Yes

Did you try recovering your dependencies?

Yes.
yarn --version:
1.15.2
Which terms did you search for in User Guide?

alpha test --coverage enters watch mode
Environment

Environment Info:
System:
OS: Linux 4.15 Ubuntu 18.04.2 LTS (Bionic Beaver)
CPU: x64 Intel(R) Core(TM) i7-7600U CPU @ 2.80GHz
Binaries:
Node: 10.15.3 - /usr/bin/node
Yarn: 1.15.2 - /usr/bin/yarn
npm: 6.4.1 - /usr/bin/npm
Browsers:
Chrome: 73.0.3683.103
Firefox: 66.0.3
npmPackages:
react: ^16.8.6 => 16.8.6
react-dom: ^16.8.6 => 16.8.6
react-scripts: 3.0.0-next.68 => 3.0.0-next.68+4b5b76b7
npmGlobalPackages:
create-react-app: 1.5.2
Steps to Reproduce

In terminal
npx create-react-app@next --scripts-version=3.0.0-next.68 app-name
cd app-name
yarn test --coverage

Check terminal output
Expected Behavior

Run a jest test coverage report.
Actual Behavior

Looks like it is doing a coverage report, but seems to enter jest watch mode.

Reproducible Demo

https://github.com/jkdowdle/creact-react-app-3.0.0-next.68-test-coverage-issue

https://github.com/facebook/create-react-app/issues/6475#issuecomment-484208109
https://github.com/facebook/create-react-app/issues/6475#issuecomment-484208700

Likely related to this change: #4176

Hi @jkdowdle, Thanks for testing the latest alpha.
This is an intentional change.  jest did not support watch mode and coverage at the same time last year, so the previous behavior of yarn test --coverage did not run in watch mode.
Now that jest supports --coverage flag in watch mode, the following behaviors are supported:
yarn test --coverage' which runs jest in watch mode and prints coverage of the touched files yarn test --coverage --watchAll=false` which runs jest in non-watch mode and prints coverage of all files.

In addition what @bugzpodder said the watch mode is disabled in CI so it should be a fairly safe change

@bugzpodder @petetnt
Oh ok I didn't realize that was a feature Jest would introduce. But if this is expected behavior, awesome!
I will close the issue. Appreciate the quick responses on this!


Better validation of configuration

There should be a JSON schema validation for configuration properties. We need to validate and provide the best possible feedback to wrong values/properties to users. I think JSON schema can be a good way to achieve this, but it will be different for each format (yaml...). I believe the first step should be improving validation on the abstract configuration structures and second step could be validation of configuration files since the first step provides the feedback the value is not correct but it doesn't say where the value error is located or if the value is missing where that value should be defined.

+1
Also, there are two formats of flow.json with no clear description of their differences.

+1
Also, there are two formats of flow.json with no clear description of their differences.

There is an old flow.json format that if you try using with updated CLI it shows a warning. Is that what you have in mind?

@sideninja I am not sure which one of flow init or flow project init is old one. If I recall correctly, it may sometimes require different formats for different tasks or fail without warning on using wrong format.

Oh yeah the flow project init is getting deprecated, there is warning in the new version - I suggest you update it.


Tabular View Doesn't Show Some Metrics

By default (no benchmark filter), Tabular View should show all metrics. As shown in the screenshots below, it doesn't shows only one metric (i.e. footprint) for pingperf-quarkus-startup benchmark and misses the other one (i.e. startup time). When we use the benchmark filter to explicitly selected Quarkus benchmarks, then we can see both the metrics for pingperf as shown in the screenshots below.
We noticed this issue after #230 was merged and the server was updated with the latest changes.
Without Benchmark Filter

With Benchmark Filter

FYI @sophiaxu0424


Does TVM support multiple GPUs or multiple streams?

Hi,
Just as title. In some cases, some compute nodes in a graph are not dependent on other nodes, so we can put two set of nodes into two CUDA streams and let them run in GPU independently.  Can TVM calculate how many independent computation set there are in the graph and put each set into different streams?
Thanks!

We don't support now.
We could move this topic to discuss.tvm.ai


Modified test configurations for #oops/2685

Description
Please see https://github.com/JCSDA-internal/oops/pull/2685.
Issue(s) addressed
None.
Dependencies
build-group=https://github.com/JCSDA-internal/oops/pull/2685
Impact
None.
Checklist

[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[x] I have run the unit tests before creating the PR

Won't be needed due to changes in OOPS PR.


Added addTitles and listSales commands

Allows users to add their own data to the registry. Also updated the submit command to allow for command line input.
Once a submit is requested, a sales record is created which can be displayed using the listSales command.

 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.


Revert "MX records need ability to set priority"

{"rdapConformance":["rdap_level_0"],"notices":[{"title":"Terms of Use","description":["Access to AFILIAS RDAP information is provided to assist persons in determining the contents of a domain name registration record in the Afilias registry database. The data in this record is provided by Afilias Limited for informational purposes only, and Afilias does not guarantee its accuracy.Â Â This service is intended only for query-based access. You agree that you will use this data only for lawful purposes and that, under no circumstances will you use this data to(a) allow, enable, or otherwise support the transmission by e-mail, telephone, or facsimile of mass unsolicited, commercial advertising or solicitations to entities other than the data recipient's own existing customers; or (b) enable high volume, automated, electronic processes that send queries or data to the systems of Registry Operator, a Registrar, or Afilias except as reasonably necessary to register domain names or modify existing registrations. All rights reserved. Afilias reserves the right to modify these terms at any time. By submitting this query, you agree to abide by this policy."],"links":[{"href":"https://rdap-pilot.afilias.info/rdap/terms_of_use.html","type":"text/html"}]}],"remarks":[{"title":"EPP Status Codes","description":["For more information on domain status codes, please visit https://icann.org/epp"],"links":[{"href":"https://icann.org/epp"}]},{"title":"Whois Inaccuracy Complaint Form","description":["URL of the ICANN Whois Inaccuracy Complaint Form: https://www.icann.org/wicf"],"links":[{"href":"https://www.icann.org/wicf"}]}],"objectClassName":"entity","handle":"303","roles":["registrar"],"publicIds":[{"type":"Registrar IANA ID","identifier":"303"}],"links":[{"value":"https://rdap-pilot.afilias.info/rdap/entity/303","rel":"self","href":"https://rdap-pilot.afilias.info/rdap/entity/303","type":"application/rdap+json"}],"vcardArray":["vcard",[["version",{},"text","4.0"],["fn",{},"text","PDR Ltd. d/b/a PublicDomainRegistry.com"],["org",{},"text","PDR Ltd. d/b/a PublicDomainRegistry.com"],["role",{},"text","registrar"],["adr",{},"text",["","",["Unit No 501, 5th floor and Unit IT Building No 3","NESCO IT Park","Western Express Highway, Goregaon (East)"],"Mumbai","Maharashtra","400063","IN"]],["tel",{"type":["voice"]},"uri","tel:+1.8322951535;ext=7747"],["tel",{"type":["fax"]},"uri","tel:+1.3202105146"],["email",{},"text","registrar@logicboxes.com"]]],"events":[{"eventAction":"registration","eventDate":"2001-07-30T02:13:21.000Z"},{"eventAction":"last changed","eventDate":"2019-08-07T16:02:43.572Z"},{"eventAction":"last update of RDAP database","eventDate":"2019-08-12T11:51:54.684Z"}],"entities":[{"objectClassName":"entity","vcardArray":["vcard",[["version",{},"text","4.0"],["role",{},"text","abuse"],["tel",{"type":["voice"]},"uri","tel:+1.2013775952"],["email",{},"text","abuse-contact@publicdomainregistry.com"]]],"roles":["abuse"]}]}

@paranornfff - Can you provide a summary as to why this is needed?


Perhaps all the public C++ classes should be in their own namespace?

I'm opening this as an issue because I wanted to start a little dialog before just doing something and making a PR.
In the end perhaps it's not very important, but it seems like we should wrap everything in our own namespace.
I know that we've got a ZDW namespace in there already for a couple things.
Perhaps we want to just wrap everything in that namespace?
Or perhaps in something like adobe_zdw or adbe_zdw?  Or even a multi-level namespace like adobe::zdw?  (Maybe I should see if I can find any other adobe open source C++ code and see what's done elsewhere?)
Thoughts?  Disagreements?

Thanks for proposing the idea, Scott.  I like the direction and your idea to check out whether any other open source C++ code has a convention already being established.

Sadly, I couldn't find much by the way of Adobe open source C++ projects...
That being said, I'm kind of liking adobe::zdw.
That matches protobuf, which uses: google::protobuf (and they have a google::protobuf::internal namespace also, which we might want to follow)

I too like adobe::zdw, and maybe the ::internal namespace also.  Let's do it!

Thanks for the changes!
Merged your PR.
Closing this issue.


Button click does not seem to work

I'm trying to use this library for a custom presence app and although I can get buttons to show up in the presence, clicking on them seems to do nothing. No browser window opens or anything like that.
I have only tested this on macos with Discord client version 0.0.268.
Just for some concrete examples, I've tried several different urls, such as Button::new("google", "https://google.com") and Button::new("steam", "https://steampowered.com").
Can you confirm that button clicks are currently working as intended? Any ideas on what the problem might be?

You can't click on your own buttons, if others can click on them and it opens the url then it means it's working

You can't click on your own buttons, if others can click on them and it opens the url then it means it's working

Ah, that would explain it then. Thanks for the response!

Just to add a note to this: you can also test that the buttons work by checking the presence on Discord on mobile (where the buttons are clickable, for your own profile). At least this works on iOS. It's a little strange that it does't work on the desktop client and that the buttons aren't simply disabled or something.


docs(cookbooks/dependency-injection): document and samples ready for QA

Thanks, Istvan! Closing because we have one now.


C:\Users\vicky\Documents\hq>git clone https://github.com/Exaphis/HackQ-Trivia.gi t 'git' is not recognized as an internal or external command, operable program or batch file.

Do you have python installed?

ya 3.6

Is python in your path?

ya i am done this with computer properties

i change path for python

hello
please help me

Solved in #115


[ALLUXIO--2374]improve javadoc in hadooputils

https://alluxio.atlassian.net/browse/ALLUXIO-2374

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/11994/
Test PASSed.


dts: Remove mcuboot.overlay

As we want to remove dts dependency on Kconfig, we had a case based on
CONFIG_BOOTLOADER_MCUBOOT.  From a DTS point of view that was just
getting the chosen property 'zephyr,code-partition' set.  We can easily
move this to the actual dts files and remove the mcuboot.overlay.
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

I am concerned that with this strategy the amount of Device tree code we are going to maintain is going to be O(number_of_boards * number_of_features). Whereas before it was O(number_of_boards) + O(number_of_features).
Do we have any alternative strategies?

Closing this PR for now, but it helped generate some good ideas on how to address.


How to use twigphp/cache-extra?

Hey,
I am trying to use the twigphp/cache-extra (https://twig.symfony.com/doc/3.x/tags/cache.html) extension in TwigBridge.
In app/config/twigbridge.php I added in extensions->enabled section an entry:
TwigExtraCacheExtension::class,

When I try to use the cache block, I get an error:
ErrorException (E_ERROR)
Unable to load the "Twig\Extra\Cache\CacheRuntime" runtime.

The twig documentation says:
If you are not using Symfony, you must also register the extension runtime:

use Symfony\Component\Cache\Adapter\FilesystemAdapter;
use Symfony\Component\Cache\Adapter\TagAwareAdapter;
use Twig\Extra\Cache\CacheRuntime;
use Twig\RuntimeLoader\RuntimeLoaderInterface;

$twig->addRuntimeLoader(new class implements RuntimeLoaderInterface {
    public function load($class) {
        if (CacheRuntime::class === $class) {
            return new CacheRuntime(new TagAwareAdapter(new FilesystemAdapter()));
        }
    }
});

However, I have no idea where in the application to add such a loader. From what I see, extensions are added in this file:
vendor/barryvdh/laravel-form-bridge/src/ServiceProvider.php
Does this mean I should overwrite this service provider and use a custom one?

@rcrowe any help available here?

Just add it to AppServiceProvider for example:
use Symfony\Component\Cache\Adapter\FilesystemAdapter;
use Symfony\Component\Cache\Adapter\TagAwareAdapter;
use Twig\Extra\Cache\CacheRuntime;
use Twig\RuntimeLoader\RuntimeLoaderInterface;
...
class AppServiceProvider extends ServiceProvider
{
	public function boot() {
			$twig = app(\Twig\Environment::class);
	        $twig->addRuntimeLoader(new class implements RuntimeLoaderInterface
	        {
	            public function load($class)
	            {
	                if (CacheRuntime::class === $class) {
	                    return new CacheRuntime(new TagAwareAdapter(new FilesystemAdapter()));
	                }
	            }
	        });
	}
}


Make the "Sources and Uses of Funds" table responsive/mobile friendly

See the table under the "Sources and Uses of Funds" section of the Financial Plan page.

This table was created using Markdown. Instead, the table might have to be created with appropriate  Foundation markup and CSS classes.
This might be worth looking into: http://zurb.com/playground/responsive-tables
Also, it might be a good idea to create a CMS "Tag" Content item for tables in general.

Foundation v6 has support for responsive/mobile-friendly tables with its addition of responsive modifiers. I updated the content in the CMS to use full HTML markup and appropriate modifiers so the table is now responsive.


frontend: extend subsidized transfers with missing functionality

There are still no tests for this file as it requires some effort.

I aligned with Gabriel and due to the issues of overwriting the request manager address, I reimplemented the SubsidizedTransfer. This can be reviewed again now.

The way you implemented it looks perfect!
After thinking about the other possible ways to do it i realized we can avoid repeating ourselves (overriding the call... functions) by creating an abstraction around the functions inside the request-manager.ts. This abstraction will contain a contractAddress property that will reference to the on-chain contract that is initialized in the constructor. With this, we will only carry one instance of this class inside the Transfer or SubsidizedTransfer classes and call all the functions via that instance instead of using these functions as separate units. This way, the SubsidizedTransfer will also look way leaner. If i am not wrong, it is also more memory efficient. Looking forward to hear your thoughts on this.
I anyways accept your implementation as it is perfect. If you like my idea too, we can create an issue and optimize it in the future.

@GabrielBuragev
This would mean, that we create a class inside the request-manager.ts that would wrap the transaction functions in order to hold state (the contractAddress), right? The transaction functions were written in a functional programming style on purpose in the first place. The reasons were simplicity and easier testing.
There are even cases when we want to use different addresses of the request manager for different functions (e.g. fee sub). I think we would run into the same problem as in my comment above regarding the checkAndUpdateState function of the Transfer.
In my opinion, it is better to have this separation: Transfer / SubsidizedTransfer holds state, request-manager functions (and other contract functions) are pure.

Ah ok that makes more sense. But still we would have the issue about the checkAndUpdateState function. It should not use the fee sub address. Or how would that be solved if you only have one instance of the new class?

Ah true, somehow missed that. You are right, sorry for the confusion.


Remove "experimental" from odo preferences

What versions of software are you using?
Operating System: all
Output of odo version: master (2.0.5)
How did you run odo exactly?
odo preference view
odo preference set -h
Actual behavior
odo preferences mention Experimental mode that was added during the run up to v2 release
Expected behavior
Experimental mode is not being used any more. So odo should remove it.
/good-first-issue
/kind bug

/status blocked
Docker support is currently experimetnal, before we  can remove it from preference we need to decide on its fate.
https://github.com/openshift/odo/issues/4500

/reopen
/remove-lifecycle rotten
still a prolem

▶ odo preference view
PARAMETER             CURRENT_VALUE
UpdateNotification
NamePrefix
Timeout
BuildTimeout
PushTimeout
Experimental
Ephemeral             true
ConsentTelemetry      false

There is still Experimental config option in odo preference. This option should not be used for anything.

I'm happy to take this on for Hacktoberfest, if it's still available

@dimitraz Thank you for your interest, let me confirm it and get back to you.
@kadel I think we can move this forward, I don't see any reference to Experimental except in its code implementation and some help docs.

@valaparthvi  Should we be adding some hacktoberfest related label to this issue and its PR?


PodeWebContainer resize

Hi,
is it possible to resize New-PodeWebContainer ?
for example 😊:

Not sure about what you mean, resize Container?  What your example appears to show is Containers inside of Grid Cells?  What you tried that?

Hi @jbaechtelMT,
The PodeWebContainer has a standard size and stretches to the full length of the page. I mean change the size of the container, for example, make them two, three in a row so that you can display brief information there. Is it possible? or can it only be done by changing the css ?
Thanks

The following example should create a single Container (the entire page) and then creates a Grid of Cells (3 across) inside of that Container.  Then inside each of the Cells, it creates Containers (left, middle, right) in each of those Cells.  Then inside each of those Containers, Text is displayed.  Hope it Helps:
New-PodeWebContainer -Content @(                       # Outer Page Container
New-PodeWebGrid -Cells @(
New-PodeWebCell -Content @(                  # Left Grid Cell
New-PodeWebContainer -Content @(        # Left COntainer
New-PodeWebText -Name 'LeftCellContainerText' -Value "Left Cell Container Text"
)
)
New-PodeWebCell -Content @(                  # Middle Grid Cell
New-PodeWebContainer -Content @(        # Middle Container
New-PodeWebText -Name 'MiddleCellContainerText' -Value "Middle Cell Container Text"
)
)
New-PodeWebCell -Content @(                  # Right Grid Cell
New-PodeWebContainer -Content @(        # Right Container
New-PodeWebText -Name 'RightCellContainerText' -Value "Right Cell Container Text"
)
)
)
)

@jbaechtelMT,
thank you so much, everything is amazing. Thanks 👍👍👍


Display version number in Footer

@EinfachGustav Can you please take a look at the two security hotspots?


Support xinghuo(spark) LLM model

The Spark Cognitive Model, developed by iFLYTEK, represents a significant leap in the field of artificial intelligence.
https://www.xfyun.cn/doc/spark/HTTP调用文档.html#_1-接口说明

[ ]  Support chat model
[ ]  Support embedding model
[ ]  Support image model

@liugddx It seems this model may have compatibility with the OpenAI API, so it might not need to be added separately—similar to the considerations discussed in #1712. Just a thought!

@liugddx It seems this model may have compatibility with the OpenAI API, so it might not need to be added separately—similar to the considerations discussed in #1712. Just a thought!
It is not fully compatible with openai, you can check its official documentation. And there is no official statement that it needs to be compatible with openai.


昇腾NPU推理glm4-9b-chat出现'NoneType' object has no attribute 'do_sample'

Reminder

[X] I have read the README and searched the existing issues.

System Info

Platform: Linux-4.19.36-vhulk1907.1.0.h619.eulerosv2r8.aarch64-aarch64-with-glibc2.28
Python version: 3.10.14
PyTorch version: 2.2.0 (NPU)
Transformers version: 4.41.2
Datasets version: 2.20.0
Accelerate version: 0.32.1
PEFT version: 0.11.1
TRL version: 0.9.6
NPU type: Ascend910PremiumA
CANN version: 8.0.RC1.alpha003

Reproduction
ASCEND_RT_VISIBLE_DEVICES=4 llamafactory-cli chat glm4_9b_chat.yaml

Expected behavior
Traceback (most recent call last):
  File "/root/miniconda3/envs/llm/bin/llamafactory-cli", line 8, in <module>
    sys.exit(main())
  File "/root/project/LLaMA-Factory/src/llamafactory/cli.py", line 81, in main
    run_chat()
  File "/root/project/LLaMA-Factory/src/llamafactory/chat/chat_model.py", line 125, in run_chat
    chat_model = ChatModel()
  File "/root/project/LLaMA-Factory/src/llamafactory/chat/chat_model.py", line 44, in __init__
    self.engine: "BaseEngine" = HuggingfaceEngine(model_args, data_args, finetuning_args, generating_args)
  File "/root/project/LLaMA-Factory/src/llamafactory/chat/hf_engine.py", line 58, in __init__
    self.model = load_model(
  File "/root/project/LLaMA-Factory/src/llamafactory/model/loader.py", line 159, in load_model
    patch_model(model, tokenizer, model_args, is_trainable, add_valuehead)
  File "/root/project/LLaMA-Factory/src/llamafactory/model/patcher.py", line 120, in patch_model
    if not gen_config.do_sample and (
AttributeError: 'NoneType' object has no attribute 'do_sample'

Others
No response

模型文件未做任何修改，包含config，generation_config.json，configuration.json


Maintain test of Get-IshBackgroundTask which on large systems can result in System.OutOfMemoryException

On large/historical systems the cmdlet Get-IshBackgroundTask, especially with parameter -UserFilter All can result in huge data sets. By itself the cmdlet is okay, on large systems this can happen and hence there are filters like -ModifiedSince and -MetadataFilter to reduce the data set.
Context Get-IshBackgroundTask.Get-IshBackgroundTask failed
   FaultException`1: [-1] Exception of type 'System.OutOfMemoryException' was thrown.
   at <ScriptBlock>, C:\GITHUB\ISHRemote\Source\ISHRemote\Trisoft.ISHRemote\Cmdlets\BackgroundTask\GetIshBackgroundTask.Tests.ps1:86

Tests affected are

[x] GetIshBackgroundTask.Tests.ps1
[x] GetIshMetadataField.Tests.ps1

Possible solution is to allways include a date-based -MetadataFilter  that starts on today and iteratively goes to today-1, today-2 until the minimum required results to complete the tests are present.

This was fixed over #180 PR - forgot about this separate issue to track it. Fixed through assist function GetBackgroundTasks added in the Pester tests BeforeAll code block.


Refactor of MGA code to support low thrust legs

Hi Miguel, I've merged all the pull requests related to this, very nice!


🚀 Feature: Indent all lines on message

As discussed at https://github.com/BadgeLabs/mocha-eslint/issues/38#issuecomment-197735716 mocha only indent the first line of the report message, making it ugly since it gets unaligned with the mocha output. I've also seen that the code to indent the message is duplicated in several of the reporters, so if you like the idea I could try to unify them and implement it myself.
/cc @Turbo87

Can you give an example where formatting is broken? E.g. this seems correct to me:
describe('suite', function() {
  it('test', function() {});
  it('test2', function() {
    throw new Error('foo');
  });
});

$ mocha example.js

  suite
    ✓ test
    1) test2

  1 passing (10ms)
  1 failing

  1) suite test2:
     Error: foo
      at Context.<anonymous> (example.js:4:11)

@danielstjules the issue is visible if you have a multi-line error message:
describe('suite', function() {
  it('test', function() {});
  it('test2', function() {
    throw new Error('foo\nbar\nbaz');
  });
});

$ mocha example.js

  suite
    ✓ test
    1) test2

  1 passing (11ms)
  1 failing

  1) suite test2:
     Error: foo
bar
baz
      at Context.<anonymous> (/private/tmp/example.js:4:11)

Yes, the problem is the one that @Turbo87 has described, if the error message has multiple lines they get unindented. It would be as simple as split the message on new lines and apply the indention to all of them.

Hi there. I checked out exactly the same issue is still present. I would like to fix this problem.

This feels to me like something that should be behind an option. It's useful to be able to select & copy the exact characters of an error in some cases.

Coming back to this: per #5027 we're trying not to make too many big changes, and there are already a lot of options in Mocha. This issue has received, on average, about one positive interaction every 1.5 years since it was filed. I don't think it makes the cut.
If you really want this to happen, please do ask to re-open. We certainly can. But please also provide a concrete proposal for what the option would look like. 🙂

please also provide a concrete proposal for what the option would look like. 🙂

I think @Turbo87 already showed the actual issue...

I mean, the option. Like, what CLI option would be added to Mocha.

I don't think it needs an option, or if any, it should be enabled by default and an opt-out option.


my token

Token Address
https://tomoscan.io/address/0x739388A9A3c9621868bbcc751B011cEa8c107F58
Token Symbol
SLT
Token Name
Sl thunder
Token Logo

Token Description
A clear and concise description of the token.
Website
Link to the website that will use the token.
Social links
e.g Github, telegram, twitter

Hi @kienmegaltr ,
we update receiving address here
https://github.com/BuildOnViction/tokens#2-request-our-team
Please check and proceed if you still wish to update token information
Thanks
(This content is created via Gittool)
On behalf of bobcoin98

ThanksĐược gửi từ iPhone của tôiVào ngày 16 thg 11, 2023, lúc 10:59, Bob Jr @.***> đã viết:
Hi @kienmegaltr ,
we update receiving address here
https://github.com/BuildOnViction/tokens#2-request-our-team
Please check and proceed if you still wish to update token information
Thanks
(This content is created via Gittool)
On behalf of bobcoin98
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>


Image info from file or IIIF

Populate Image resource with info from either a file or IIIF info.json:
https://github.com/iiif-prezi/iiif-prezi/blob/e5956d5ab929ff8149a6429c6145790bb4bd59e5/iiif_prezi/factory_11.py#L822
Would require PIL if we are going to open images.

Helper issue for hackathon: https://github.com/iiif-prezi/iiif-prezi3/issues/55


No documentation on some static methods in Bloodhound e.g. tokenizers.whitespace

There are quite a few important methods in Bloodhound that is not documented in:
https://github.com/twitter/typeahead.js/blob/master/doc/bloodhound.md
such as:

Bloodhound.tokenizers.nonword
Bloodhound.tokenizers.whitespace
Bloodhound.tokenizers.obj.nonword
Bloodhound.tokenizers.obj.whitespace

Where is the documentation for these static methods (or am I just missing something)?

I have now documented them. Please accept pull request: https://github.com/twitter/typeahead.js/pull/1372


Pedestrian Paths & Curbs

Hi David,
A possible enhancement to put on the wish list:
When using CE streets to create pedestrian paths, the usual trick is lowering down the street width to 0.
This way we end up with just two contiguous sidewalk segments.
However, when we try to do this applying the CSR, we can't get an homogeneous result because of the concrete curbs that are always set at the sidewalk limit (see screenshots).
Would it be possible to add an attribute to get rid of the curbs when not needed?
In my opinion, best would be an option to decide if we want to have right, left or both curbs for each sidewalk.

I am highly resistant to edits that add attributes...I think paths need a dedicated rule personally.
Until that happens, I could remove curbs when the ROW (street width) is set to zero. It would accomplish the same thing right?

That's true, paths often require quite specific design. But it would be great if we could achieve a simple result without adding another rule to the project.
Removing it for zero ROW values sounds like an effective alternative to get rid of the curbs. It would be great to have that possibility.

I got something that works with the small issue I mentioned. This does work though. I will merge into main if you think it works.

I did some more testing. There is some oddity with the shape. I did test, but let me know if you have any issues.

Sorry for the delay,
It worked as expected.
Some screenshots:

Thank you for implementing this so quickly!