Model.create() Values autocomplete is missing?

Am i missing something or is there no autocomplete on new Model({...}) ,Mode.create({...}) or Model.build({...})...
I added the columns to my Models as shown in the documentation, but when i go to create a Model and i start typing the Class members, they do not auto complete..
Shouldn't it work like this?

This guy did the sequelize tyescript integration manually and it seems like a lot of dirty work to make it work. If this is not something integrated with this package, would it be hard to implement so that we get auto complete?

It's a bit awkward, since a while ago sequelize-typescript offered said completions (0.6 I think), then they removed when Sequelize@5 added official Typescript support (but without completions) and now Sequelize@6 has full completions (using a hidden property called _attributes).
So currently, the maintainer of this library seems to be inactive, and you get better support of types sticking to the official (ironically).

I had the same issue, and I solved it by creating a type for my model maybe you need two one for TModelAttributes and another for TCreationAttributes, and then do like this (I forgot how it calls in typescript 😅). then the completion works fine.
type User = {
  id: number;
  username: string;
 //...
}
type CUser = Omit<User, 'id'>

class User extends Model<User, CUser> {
 @Column
 //..
}

@AbderrazzakB That's pretty much how sequelize@6 recommends it, and it properly fills the _attributes underlying property that feeds the autocompletion.
The only issue you'll run is when attempting to do relations, where the, for example, @BelongsToMany(() => Class) does not forward the _attributes information, and causes Typescript to throw a compilation error.
For now, I think the only solution is that, and cast to any to cover it up. For Example:
interface UserAttributes {
    id: string
    name: string
}

type UserCreationAttributes = Omit<UserAttributes, 'id'> // id is auto-generated.

@Table({ ... })
class User extends Model<UserAttributes, UserCreationAttributes> { // <-- we add the type info for Sequelize@6.
    @Column({ type: DataTypes.UUIDV4 })
    id!: string

    @Column({ type: DataTypes.TEXT, allowNull: true })
    name!: string | null

    // Here we cast to any so seq-typescript doesnt cry
    @BelongsToMany(() => Post as any, () => UserPost as any)
    readonly posts?: Array<Post>
}

You can check the types work in VSCode by doing
User['_attributes']['<autocomplete here>']

@AbderrazzakB That's pretty much how sequelize@6 recommends it, and it properly fills the _attributes underlying property that feeds the autocompletion.
The only issue you'll run is when attempting to do relations, where the, for example, @BelongsToMany(() => Class) does not forward the _attributes information, and causes Typescript to throw a compilation error.
For now, I think the only solution is that, and cast to any to cover it up. For Example:
interface UserAttributes {
    id: string
    name: string
}

type UserCreationAttributes = Omit<UserAttributes, 'id'> // id is auto-generated.

@Table({ ... })
class User extends Model<UserAttributes, UserCreationAttributes> { // <-- we add the type info for Sequelize@6.
    @Column({ type: DataTypes.UUIDV4 })
    id!: string

    @Column({ type: DataTypes.TEXT, allowNull: true })
    name!: string | null

    // Here we cast to any so seq-typescript doesnt cry
    @BelongsToMany(() => Post as any, () => UserPost as any)
    readonly posts?: Array<Post>
}

You can check the types work in VSCode by doing
User['_attributes']['<autocomplete here>']

No way that the lib that calls itself *-typescript does not provide proper types for most basic Model methods.
That's sad!


Special casing GpuAtomicMin / GpuAtomicMax for ROCm

The TF1.8 implementations for the float and double implementations of GpuAtomicMin and GpuAtomicMax routines assume the presence of   float and double versions of min and max __device__ routines. That assumption is true for CUDA but not for HIP,  and what ends up happening is an implcit cast from float/double to int leading to bad results!
Fixing this by special casing the ROCm implementation to call fminf / fmin / fmaxf / fmax

@deven-amd Even though the PR is a success I'd like to explore the possibility of amending HIP because these gaps ought better be residing in HIP instead of applications such as TensorFlow.


Change DurationInMillis's type from int to int64

Fix #61

Codecov Report
Base: 73.11% // Head: 73.11% // No change to project coverage :thumbsup:

Coverage data is based on head (6d3b5d7) compared to base (0c9041a).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #62   +/-   ##
=======================================
  Coverage   73.11%   73.11%
=======================================
  Files          43       43
  Lines        2232     2232
=======================================
  Hits         1632     1632
  Misses        532      532
  Partials       68       68

Flag
Coverage Δ

unittests
73.11% <ø> (ø)

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

Impacted Files
Coverage Δ

pkg/job/blueocean_type.go
100.00% <ø> (ø)

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

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


Remove initialize and _add_year_CIN methods in DialogueStateTracker?

The method initialize in DialogueStateTracker is defined but not implemented.
It appears that both methods (initialize and _add_year_CIN) are unused, should we remove it?

I would say yes. Less clutter == better.


JCenter is shutting down

JCenter is shutting down in 12 months
https://jfrog.com/blog/into-the-sunset-bintray-jcenter-gocenter-and-chartcenter/
Please transfer the snowplow tracker from jcenter to any other hosting service

Thanks. This is on our radar. We intend to have a solution in place for our next major release (which isn't too far away now 🎉). It's extremely likely we will be publishing to Maven Central.

We've now moved all existing releases that were on Bintray/JCenter to Maven Central (0.6.0 to 1.7.1).
https://search.maven.org/artifact/com.snowplowanalytics/snowplow-android-tracker/1.7.1/aar

Awesome thanks!!


What looks like a typo in fdisk.sh

Whats going on here?
https://github.com/mbrukman/stackexchange-answers/blob/ac1ef164d69c8ac7f2d78f148266b0be279b2fda/stackoverflow/24021214/fdisk.sh#L93-L104

Sorry for the delay in response. That's not a typo; those are all individual commands which are single-character, and they're on separate lines because they require Enter to be pressed after each one of them. The two blank lines are equivalent to pressing Enter twice in a row and accepting the default options that are presented.
If you want to see what this looks like interactively, just run the command manually:
fdisk -c -u /dev/sda

and see which keystrokes you input, and you should end up with the same list:

d Enter
n Enter
p Enter
1 Enter
Enter
Enter
w Enter

I was going to add a comment to document it in the script itself, but in fact, this is already documented in a detailed comment right above the text you quoted:
https://github.com/mbrukman/stackexchange-answers/blob/ac1ef164d69c8ac7f2d78f148266b0be279b2fda/stackoverflow/24021214/fdisk.sh#L87-L93


add quotes and sort=new to praw search

slight tweak to make reddit search an exact match sorted by newness.

thanks! i think we want the quotes in bridgy though, not here.
also, i found this: https://www.reddit.com/wiki/search#wiki_search_api . looks like we should maybe use site:domain.com instead? and also it'd be nice to use the OR operator mentioned there to combine all of the domains into a single get_activities search call in bridgy's Reddit.search_for_links(), instead of one per domain like now.

after some fiddling site:domain.com definitely doesn't work, as it will only pick up submissions that are of the url type.  (won't work if someone submits a post that has a url in it.)
Quotes and the OR seem to be a good idea and do work.  sending in a bridgy PR.

thanks! merging.
this also reminds me that we'll definitely want tests for get_activities soon! not urgent though.


AttributeError: 'Tensor' object has no attribute 'output'

Hi everyone:)
I'm trying to use keras_explain in colab for my 3dcnn network
the visualization method I want is LRP
but I'm ending up "AttributeError: 'Tensor' object has no attribute 'output'" error
I really do need to solve this problem
my imports are like this:
import tensorflow
import keras
from keras import layers
and also I have to use tensorflow 1 because with tensorflow  2 I'm getting another error
so the versions are:
tensorflow 1.15.2
keras 2.1.6 (it's the version that installed by keras_explain)
I also have keras-vis-0.5.0
this is the model I have:
def get_model(width=80, height=80, depth=80):
    """Build a 3D convolutional neural network model."""

    inputs = keras.Input((width, height, depth, 1))
    x = (inputs)

    x = layers.Conv3D(filters=8, kernel_size=5, activation="relu", padding="same")(x)
    x = layers.MaxPool3D(pool_size=2)(x)
    x = layers.BatchNormalization()(x)

    x = layers.Conv3D(filters=16, kernel_size=3, activation="relu", padding="same")(x)
    x = layers.MaxPool3D(pool_size=2)(x)
    x = layers.BatchNormalization()(x)

    x = layers.Conv3D(filters=32, kernel_size=3, activation="relu", padding="same")(x)
    x = layers.MaxPool3D(pool_size=2)(x)
    x = layers.BatchNormalization()(x)

    x = layers.Flatten()(x)

    x = layers.Dense(units=512, activation="sigmoid")(x)
    x = layers.Dropout(0.05)(x)

    outputs = layers.Dense(units=1, activation="sigmoid")(x)

    # Define the model.
    model = keras.Model(inputs, outputs, name="3dcnn")
    return model

this is the error I get:

AttributeError                            Traceback (most recent call last)
 in ()
----> 1 exp = explainer.explain(img, 0)
1 frames
/usr/local/lib/python3.6/dist-packages/keras_explain/lrp.py in explain(self, image, target_class)
31
32         # retrieve outputs of all layers
---> 33         outputs = self.get_layers_outputs(model, image)
34
35         relevances = {}  # key: layer name, value: relevance of input tensor
/usr/local/lib/python3.6/dist-packages/keras_explain/lrp.py in get_layers_outputs(self, model, input)
130             #s_recon_layer = Lambda(lambda x: K.squeeze(x, 2))(recon_layer)
131
--> 132             out_fun = K.function([model.layers[0].input], [layer.output])
133             layer_output = out_fun([input[None, ...]])[0]
134             activations[layer.name] = layer_output
AttributeError: 'Tensor' object has no attribute 'output'

@shakilashj
I tried your code with version as specified in the image

@shakilashj
I tried your code with version as specified in the image


Index out of bounds: the len is 26 but the index is 30

Description
File content(at the bottom should be attached raw, not formatted file - github removes some non-printable characters, so copying from here may not work):
#let apa-bibliography = (() => {
let dict-from-entries = entries => {
	for (key, value) in entries {
let n(x, default, f) = if x == none { default } else  f(x) }
	}
}
let months = (
)
let parse-date(x) = {
(works, include-all: true) => {
	let use-workkey) = [#used-works.update(x => x + (key,))]

command
timeout -v 100 typst compile TEST___FILE.typ

cause this
thread 'main' panicked at crates/typst-syntax/src/parser.rs:838:7:
index out of bounds: the len is 26 but the index is 30
stack backtrace:
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

##### Automatic Fuzzer note, output status "Some(101)", output signal "None"

compressed.zip
Reproduction URL
No response
Operating system
Linux
Typst version

[X] I am using the latest version of Typst

Seems to be caused by #3463. If I disable the memoization in expr_with_paren, then the panic doesn’t occur.

I think this happens because the parser needs to parse a parenthesized expression at the beginning of the line containing (works, include-all: true) => { in two different situations, with two different lists of accumulated syntax nodes in p.nodes. Not sure how to fix this; we could memoize by (p.current_start, p.nodes.len()) instead of by p.current_start only, but that might hide a deeper issue.

Good news! This was resolved by the parser refactor (#5310) and should ship with the next version update. In particular it was commit 5 (16cc7eb), which refactored the memoization system, and mainly this comment in restore_memo_or_checkpoint:
// It's important that we don't truncate the nodes vector since
// it may have grown or shrunk (due to other memoization or
// error reporting) since we made this checkpoint.

In this case, the number of errors increases between memoization steps, and the old version incorrectly truncated the nodes vector, leading to an out-of-bounds access plus panic when we report the final unclosed curly-brace.
The most minimal reproduction I could make is:
#{
  (
    if x {} else {}
    { () = 2
  ) = 3
}

Let's go over each of the parts:

Setup

We use a code-block just so we can write an assignment expression
We write an assignment expression with parentheses on the left side, which will cause a mispredict in the paren expression parser
Inside the parens, we use the if-statement to create a case that has fewer errors when parsed as a parenthesized expression vs. as the lhs of an assignment. (keywords aren't allowed at all in destructuring assignments, so it errors early)
We start an unclosed curly-brace expression
After the open curly-brace we add another assignment expression so the paren parser will mispredict, causing a reload of the nodes vector.

Parsing

In the first pass, the parser creates the unclosed-delimiter error message with no problems
But when the parser reaches the outer equals sign, ) = 3, it reparses the inside as a destructuring pattern.
This time, the parser generates more errors (so the node length is longer) and the index of the open curly-brace is further in the nodes vector
When we reach the inner equals, () = 2, we load the value for it from the memo and the old version would incorrectly truncate the nodes vector when loading the parsed state
When we try to create the unclosed-delimiter error, we access the opening delimiter by it's new, longer, index
But this access is out-of-bounds because the nodes vector was truncated
So Rust panics

What a lovely bug!
:boot:
:bug:

I think this can be closed now.


[sysrst_ctrl] Update the debouncer behavior

This makes the debounce behavior more similar to the initial design, as it corresponds better to the intended programming model. That means that the debouncer logic is implemented using an FSM + counter, rather than using an instance of prim_filter.
Design-wise however, this still uses the approach that reuses one common detector module for all debounce and detect use cases within sysrst_ctrl.
The PR also adds an FPV target to prove embedded assertions.

@cindychip: I have added you mainly for the FPV part.

CC @patel-madhuri: apologies for the back and forth on this. I hope this is the last alignment that is needed.

Thanks @tjaychen. I'll hold off merging this until we clarified https://github.com/lowRISC/opentitan/issues/10663 - just to be sure this is implemented as intended.

Thanks @tjaychen. I'll hold off merging this until we clarified #10663 - just to be sure this is implemented as intended.

EDIT: I merge this in now in order to unblock DV. However we should still double check on #10663 whether the current implementation does indeed fulfill our requirements.


Initialise the client with access token

Hi
We need to call drive api for file download and upload and hence we wanted to use this java sdk. But it seems that it expects a client_secret.json file to initialise the client prior to calling any api and while doing so it launches the browser asking for user's consent.
Our use case is we have already authenticated the user by our CDN/frontend using gapi client side library (for javascript) and thus we dont need a prompt again by our service (which this library will do while initialising the client) . Can' t we simply pass the drive access token to this sdk to initialise the client and calling drive api's

Exactly what i am searching for :)

I'm not sure which is the question here, but I will go ahead on what I understood.
Are you looking for this kind of credential initialization in order to provides directly an Access-Token?
```
return new GoogleCredential.Builder()
.setJsonFactory(this.factory)
.setTransport(this.transport)
.build()
.setAccessToken(this.accessToken);

Thanks Nicola
Yes i was looking for the same thing , i wanted to initialise the drive object using acess_token directly  instead of going through oauth2.0 flow. i tried you snippet , it worked ! thanks. you may close this issue now
`public class DriveApiManager {
private static HttpTransport httpTransport;
private staticfinal JsonFactory JSON_FACTORY = JacksonFactory.getDefaultInstance();
private static final String APPLICATION_NAME = "DRIVE";
private static Drive drive;
public void getFiles() throws GeneralSecurityException, IOException{
httpTransport = GoogleNetHttpTransport.newTrustedTransport();
Credential credential = new GoogleCredential.Builder()
    .setJsonFactory(JSON_FACTORY)
    .setTransport(httpTransport)
    .build()
    .setAccessToken("ya29.Gl3pBdeZQmE38oR85JF9ysnT5heE3_FAvZK-DNbnAqzBmlFfdVsMI1LOErv7cc1a_rTFJ2hBBLZY_hBNPZAzu4WNs5xMl_5yHHzzPyUp284wAoXGPLoPIZnWj9dFM8g");

drive = new Drive.Builder(httpTransport, JSON_FACTORY, credential).setApplicationName(
    APPLICATION_NAME).build();
FileList FileList = this.retrieveAllFiles(drive);
System.out.println(FileList);

}`

For security reasons, please remove tour access-token.

That is fine , we will be sending valid access token from our cdn to service

Thanks for answering the question @NicolaSpreafico!


Helm Chart

I am in need of a helm chart for zeebe-simple-monitor.
Would it be approved if I create it in this repo?

Not a maintainer here, but do check out the camunda-helm repo - it's available under https://helm.camunda.cloud/ and has all the necessary publishing infrastructure already set-up.

@michelefa1988 thank you for raising this up. Yes, I would welcome any contributions for helm charts :+1:
There is a similar request for ZeeQS that contains a previous used helm chart: https://github.com/camunda-community-hub/zeeqs/issues/217

I am willing to contribute on creating a helm chart


any tips on how to correctly set salt/lenght?

Hello
do you have any tips on how is to safety generate the 'salt'?
Thank you

Just use laravel's str_random to generate it, or whatever. TBH, hash ids arn't "secure", anyway, so you might as well just write a human readable name for the salt.

thanks for explaining. thought it is more problems :)


Add Block User features

Get User Block List
Block User
Unblock User

Ref: #74

Pull Request Test Coverage Report for Build 765101783

23 of 29 (79.31%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 86.718%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

users.go
23
29
79.31%

Totals

Change from base Build 762685143:
-0.2%

Covered Lines:
790

Relevant Lines:
911

💛  - Coveralls


PanelContext: Eventbus should not filter out local events

The pie chart legend hover events are not working. They highlight slices on the same panel and using the standard DataHoverEvent. The panel relies on being able to subscribe to local events. But the scenes PanelContextEventBus filters out any local events.
Removing this filtering, panels filter out local events they don't care about using:
 if (eventBus === evt.origin) {
   return;
  }

Trying to make the needed changes to get that to work. But when testing the cursor sync demo I am unable to get it to work (with or without these changes). All cursors show on all panels. Think uPlot cursor sync scope key has broken.
We could also try to revive https://github.com/grafana/grafana/pull/71866

ah, the first demo scene has global scope, got confused when panel titles said Panel 1 - syncs tooltip with Panel 2'


Which is LRU or LRUCache correct?

When trying to build the latest version, LRUCache function seems to be undefined.
https://github.com/pissang/qtek-model-viewer/blob/e9400a4c5ead79f70ed63bb3e6a28b689724e8db/src/graphic/helper.js#L105
In QMV.js (2017.11.12) in the dist folder of this repository, the LRU function is used. Which is LRU or LRUCache correct?

Try npm install zrender to update zrender version.

Hmm. I Updated zrender with 'npm install zrender', however LRUCache seems to be undefined.

Weird, I removed the dependencies add just now in https://github.com/pissang/qtek-model-viewer/commit/7081142b680335d54ba296efbe0970f15ad9eca9 . Not sure if this commit can fix this issue.  Also, the build files are updated.

I downloaded the latest repository (https://github.com/pissang/qtek-model-viewer/archive/master.zip) and tried npm install and npm run build again However, it did not resolve LRUCache's problem :(

BTW, the problem with LRUCache is an event from several days ago. So, I think that it is not that today's fix is affecting.

Understand. LRU in dist folder is generated by rollup, which is the name used in the LRU module. I think it's a side effect of scope hoisting. Do you get any issue on this LRUCache stuff?

Do you get any issue on this LRUCache stuff?

When LRUCache uses the generated version, the following error is displayed when displaying the glTF model. (I depend on Google Translate, so I am sorry if I do not seem to understand the question correctly.)
QMV.js:27732
Uncaught ReferenceError: LRUCache is not defined
    at Object.helper.loadTexture (QMV.js:27732)
    at Object.helper.createAmbientCubemap (QMV.js:27841)
    at SceneHelper.updateAmbientCubemapLight (QMV.js:29692)
    at Viewer.setAmbientCubemapLight (QMV.js:32757)
    at Task.<anonymous> (index.js:94)
    at Task.trigger (QMV.js:146)
    at Viewer.<anonymous> (QMV.js:32478)
    at Viewer.wrapper (QMV.js:211)
    at sub.trigger (QMV.js:146)
    at afterLoadBuffer (QMV.js:18506)

Seems your build is different with mime. What rollup version do you use? I've encountered a similar issue when using rollup >=0.51.0

My version of rollup is below.
C:\home\edu\javascript\library\qtek-model-viewer-master>rollup -version
rollup version 0.51.2

I updated rollup to the latest version (0.51.5) and it seems the problem has been resolved.
LRU is now used instead of LRUCache.
    var textureCache = app.__textureCache = app.__textureCache || new LRU(20);

Well, I also have this issue when using 0.51.2. And it is fixed in latest version 0.51.5, And earlier version 0.50.0 also don't have this problem.

@cx20 Great. Closing this issue.


chore(master): release 0.4.5

:robot: I have created a release beep boop
0.4.5 (2023-09-06)
Bug Fixes

buffer: avoid sync conflicts during startup (#1414) (0b3ab8f)
buffer: the output document should be modifiable (#1416) (bc7e09b)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/vscode-neovim/vscode-neovim/releases/tag/v0.4.5 :sunflower:


feat: set lower y limit to zero

1-D Histograms were zero suppressed, causing improper display of plots.

What's the current status? Can we bump the minimum plotext?

What's the current status? Can we bump the minimum plotext?

Just did that with this latest push. I asked for clarification on the lower bound (https://github.com/piccolomo/plotext/issues/112#issuecomment-1282475742) but I think this is good to go, but I'll let @amangoel185 confirm as well as I assume he had some test files he was checking with.

Thanks!

@henryiii @amangoel185 How do you feel about a patch release with this and PR #79 in it?

I was just thinking that. :)

Nice. :sunglasses:

In about a week Textual CSS will land, so getting a patch release out first before we move to the new version might be best. Also, not sure what the versioning / backward-compat will be, might be worth checking out with the CSS branch to see if we need a limit.

Sounds like a good plan. @amangoel185 would you have time to do this?


Parametric management of FTP parameters

Hello and congratulations for the good work done.
I tried the library and everything works great.
Now however, I would like to make the access parameters to the FTP area parametric.
I planned to read these parameters from SD-CARD and assign them to the variables:
char ftp_server[] = "";
char ftp_user[] = "";
char ftp_pass[] = "";
for example :
String FTP_SER;
String FTP_USR;
String FTP_PWD;
these variables are read from SD-CARD.
Then I try to do:
char ftp_server[] = FTP_SER;
char ftp_user[] = FTP_USR;
char ftp_pass[] = FTP_PWD;
but I can't get it to work no matter what I try.
It's true that I'm not very good with the Arduino language but it seems strange to me not to be able to manage something apparently so simple.
Would you be so kind as to give me a hand.
Thank you for any collaboration.

You can't direct assign a String to a char array. I believe you need https://www.arduino.cc/reference/en/language/variables/data-types/string/functions/tochararray/

On the code shared I do not see where it would print “ Connecting to:”
It is recommended to paste the full output of the terminal.
this is not related to this library so I’d suggest you post your question on arduino forum.
without spending more time on it, it seems like your char ftp_server etc is not big enough to contain the string.

Hello and thanks for the prompt response. I'll try to explain the problem differently:
I define the char variables:
char ftp_server[] = "";
char ftp_user[] = "";
char ftp_pass[] = "";
I associate the values with String variables:
FTP_SER = "ftpserver";
FTP_USR = "ftpuser";
FTP_PWD = "ftppassword";
after using toCharArray I get:
ftp_server = ftpserver;
ftp_user = ftpuser;
ftp_pass = ftppassword
So far everything is fine and nothing strange.
Now, as soon as I start the function:
ESP32_FTPClient ftp (ftp_server,ftp_user,ftp_pass, 5000, 2);
The following message appears on the Arduino IDE serial monitor:
Connecting to: passwordftp
I put dummy values as FTP parameters so as not to display the real ones, but the result doesn't change.
The strange thing, which I don't think depends on Arduino, is this exchange of parameters. Which doesn't happen using your example.
If you spend some time testing the code you might find something that wasn't expected.
Thanks anyway for your great work.
Greetings.

As I wrote earlier, you’re declaring the variables wrong.
try
char ftp_pass[64]…. Etc

Thanks for your patience, everything works perfectly now.
Good work


Use RunContext as Run is deprecated

I'm removing this Deprecated method in knative.dev/pkg since it's been a few years

/assign @lionelvillard @zroubalik

/approve
/lgtm


Return Hive Table object information in getTable

Merged build finished. Test FAILed.

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

jenkins, test this please

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/5665/
Test PASSed.

alluxio-bot, merge this please.


Feat: Added managing contact messages from admin successfully issue 426

Pull Request Resolves [#426 ]
Title: Added managing contact messages from admin successfully.
Description

In admin backend, created routes and controller to fetch all the contact messages from the database.
In admin frontend, created a typescript interface for contactmessages and fetching all the messages in the ContactMessages.tsx.
Also given two button to so that admin can Read the whole message (when click on read button a beautiful pop-up model appears) and also Reply it in the gmail (with email address add in the mail).
I also fixed the css of the buttons in manage post section.
The design is responsive.

Related Issues
Fixes #426
Changes Made
New features: Added managing contact messages from admin.
Screenshots

Video
https://github.com/user-attachments/assets/f9d589b2-8a6e-4c3d-982e-dfd323647bd7
I certify that I have carried out the relevant code of conduct and provided the requisite screenshot for validation by submitting this pull request.
Thank You for this contribution.

@Ultimateutkarsh11 please add label


New database does not recognized by Agnet

Hi;
When I create New database, GUI does not show newly added database until restart the agent service,
regards

It runs the databases collection every 1hr by default or when the service is started.   Restarting the service is an option if you want to trigger the collection to run immediately or you can wait until the next scheduled run time.  If hourly collections are not frequent enough you can change the schedule.


Changes since 6-19-2018

most changes were merged from develop into master since 6-19-2018 in commit 837bbebc7c5b2abb5323692cde5a19783a3fb80c [837bbeb]

@rniswon @murphyg Is this pull request still relevant?  I can't see what the differences are in the PDF file so I'm not sure if this has already been addressed in a different update


Basic v1.0 Intune documentation

Microsoft Intune
General Guidance for Using Baseline with Microsoft Intune
Groups in Intune (especially dynamic groups) are slow to update as group data is stored in AzureAD and there is a delay in syncing between Intune and AzureAD. For time-sensitive deployments (e.g. zero-touch) it's recommended to scope to All Users or All Devices and narrow scope using device filters if possible. See Microsoft's documentation here.
Many thanks to Luke Hawkins, Chris McFall, and Johan Kjellman in the MacAdmins Slack for providing the foundation of most of this documentation.
Configuration Profile
Customize the mobileconfig file as desired. Signing the profile is not required. Then upload it to Intune:

Navigate to Devices - macOS - Configuration Profiles.
Click Create, click "New Policy", and select the "Templates" profile type.
Click "Custom" and click "Create".
Enter a name and description and click Next.
Enter a name for the configuration profile, deploy to Device channel, upload the configuration file, and click Next.
Assign as desired and click Next.
Click Create.

Zero Touch
For a successful zero-touch experience, use a script to deploy Baseline. Due to delays when Intune deploys packages, Baseline is unlikely to run when needed if deployed as a package.
Deploy the script version of Baseline

Download Baseline.sh and customize as desired. Be sure to add an exit condition. Here is an example from Johan Kjellman:

#################################
# Look for Company Portal.app #
# Abort deploy if it exists #
# This portion written by Johan Kjellman #
#################################
Company_Portal="/Applications/Company Portal.app/Contents/Info.plist"
echo "Checking if computer is already deployed"
if [ -f "$Company_Portal" ]; then
echo "Already deployed"
exit 0
else
echo "Continuing deployment"
fi

In Intune, navigate to Devices - macOS - Shell scripts.
Click Add. Enter a name and click Next.
Upload the script.
Change "Run script as signed-in user" to "No".
Change "Max number of times to retry if script fails" to 3 and click Next.
Assign to all users and click Next.
Review settings and click Add.

Use a script to download and run a pkg version of Baseline
If you wish to repackage the pkg version and add additional pkg files: Customize as desired, then upload to your desired location (cloud/local network storage).
Create a shell script that downloads and installs the pkg from Github or the location of your customized pkg.
On-Demand/Self-Service
Company Portal
By default, Baseline does not install anything to the device. Therefore, Intune has no way of detecting if Baseline successfully "installed". Customize the pkg file so it "installs" an app to put in the detection rules.
To run Baseline on-demand via the Company Portal, complete the following steps:

Download Baseline.pkg and customize it as desired.
In Intune, navigate to Apps - macOS and click Add.
Select "macOS app (PKG)" as the app type and click Select.
Upload the pkg file, fill out the information, and click Next.
Add pre or post-install scripts and click Next.
Set the minimum OS. Baseline is compatible with macOS 11 or newer, but if you added additional pkg files, they may have higher minimum OS requirements.
Set the detection rules based on how you've customized the package.
Assign as desired and click Next.
Click Create.

Also FYI, Intune is in the "looking for documentation" sidebar twice:

Microsoft Intune
General Guidance for Using Baseline with Microsoft Intune
Groups in Intune (especially dynamic groups) are slow to update as group data is stored in AzureAD and there is a delay in syncing between Intune and AzureAD. For time-sensitive deployments (e.g. zero-touch) it's recommended to scope to All Users or All Devices. Narrow scope by using Intune-native device filters (Microsoft documentation).
Many thanks to Luke Hawkins, Chris McFall, and Johan Kjellman in the MacAdmins Slack for providing the foundation of most of this documentation.
Configuration Profile
Customize the mobileconfig file as desired. Signing the profile is not required. Then upload it to Intune:

Navigate to Devices > macOS > Configuration Profiles.
Click Create, click New Policy, and select the Templates profile type.
Click Custom and click Create.
Enter a name and click Next.
Enter a name for the configuration profile, deploy to Device channel, upload the configuration file, and click Next.
Assign as desired and click Next.
Click Create.

Zero Touch
For a successful zero-touch experience, use a script to deploy Baseline. Due to delays when Intune deploys packages, Baseline is unlikely to run when needed if deployed as a package. When deploying via a script, ensure there is an ExitCondition specified in the configuration profile.
Deploy the script version of Baseline

Download Baseline.sh and customize as desired.
In Intune, navigate to Devices - macOS - Shell scripts.
Click Add. Enter a name and click Next.
Upload the script.
Change Run script as signed-in user to No.
Change Max number of times to retry if script fails to 3. Leave the other options unconfigured and click Next.
Assign to all users and click Next.
Review settings and click Add.

Use a script to download and run a package version of Baseline
If you wish to add additional pkg files to the package version: Download Baseline.pkg, customize, then repackage and upload to your desired location (cloud or local network storage).
Create a shell script that downloads and installs the default package from Github or the customized package from your network location.
On-Demand/Self-Service
Company Portal
By default, Baseline does not install anything to the device. Therefore, Intune has no way of detecting if Baseline successfully "installed". Customize the pkg file so it "installs" an app to specify in the detection rules (these cannot be blank).
To deploy Baseline for on-demand use via the Company Portal, complete the following steps:

Download Baseline.pkg and customize it as desired.
In Intune, navigate to Apps > macOS and click Add.
Select macOS app (PKG) as the app type and click Select.
Upload the pkg file, fill out the information, and click Next.
Add pre or post-install scripts if desired and click Next.
Set the minimum OS. Baseline is compatible with macOS 11 or newer, but if you added additional pkg files, they may have higher minimum OS requirements. Cllck Next.
Set the detection rules based on how you've customized the package and click Next.
Assign as desired and click Next.
Click Create.

Thank you again for this!
https://github.com/SecondSonConsulting/Baseline/wiki/Microsoft-Intune


Update pulumi-terraform reference

This looks like it's missing a new generated file tables.py, did it get generated when you ran tfgen locally?

Good catch.  It was missing.  Added it to the PR and repushed.


Alert channel should not start with #

I received a 404 otherwise.  The readme indicates that a # prefix is required.

Also, I had to remove the quotes around the channel, but my setup may be nonstandard in that respect.


Standardize css names

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

18 of 18 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.06%) to 81.073%

Totals

Change from base Build<PHONE_NUMBER>:
-0.06%

Covered Lines:
202

Relevant Lines:
244

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

18 of 18 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.06%) to 81.073%

Totals

Change from base Build<PHONE_NUMBER>:
-0.06%

Covered Lines:
202

Relevant Lines:
244

💛  - Coveralls


Nostr Fund Page Git Handle Link Fix

https://opensats.org/projects/opensats -> https://github.com/OpenSats

2 other commits are in this request that should not be. One was already merged, and the other is irrelevant. Only one pull request is legit. Closing this pull request to open another clean one for easier review and merge process.


Added Word_en.yml

To fix blocks named "air" on /jobs info [job]
I changed all blocks named "air" to their real names

This is not a fix for materials are named with air. If you using oldest minecraft version of your server then some materials and some IDs are missing so it will cause this showing air. We can't fix this issue because the Mojang don't release IDs to the materials where is missing. Changing the configs not fix this issue.

I didn't fully understand this, but I can tell you surely that it worked for me. A lot of materials were showing "air" in /jobs info [job] AND in the Word_en.yml. So I just replaced all the "air" names by their real names in the Word_en.yml and it worked.

Yes, because you just renamed them, but it won't do anything for the job. This change is only for names and not for proper operation. If you try to do something about the job with a changed material, it will not gain income.

Yes that's the problem I (and many others as I understood) experienced, blocks are working fine but names were showing "air" in the /jobs info. So that's why I corrected Word_en.yml, so that now /jobs info shows correct blocks and items instead of "air"


Update README addressing issue #4

Clean up markdown with shields.io image and cocoapods link

Ready to merge

I meant more that the readme has to be current with instructions, details, etc before launch... but this removing the invalid badges is needed too :)


samples/net/http_server: Failed to respond back to CURL command on http Client

I was trying to test http_server from sample app made it to run on frdm_k64f boards.
The steps I have followed for http_server:
cd zephyr/samples/net/http_server
mkdir build && cd build && cmake -GNinja -DBOARD=frdm_k64f ../
ninja
cp -vr outdir/frdm_k64f/zephyr.bin /run/media/MBED/
•Here I have used frdm_k64f as http_server & the no modification is done in configuration file.
I am getting the execution failure after executing the curl command on Linux machine.
curl -vvv  http://<IP_ADDRESS>/
Output :-

* Rebuilt URL to: http://<IP_ADDRESS>/
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 80 (#0)
> GET / HTTP/1.1
> Host: <IP_ADDRESS>
> User-Agent: curl/7.55.1
> Accept: */*
>
< HTTP/1.1 200 OK
< Content-Type: text/html
< Transfer-Encoding: chunked
<

    <meta charset="UTF-8">
    <title>Zephyr HTTP server sample</title>

It works!

* Illegal or missing hexadecimal sequence in chunked-encoding
* stopped the pause stream!
* Closing connection 0
curl: (56) Illegal or missing hexadecimal sequence in chunked-encoding

ERROR-IN-SHELL

ERROR-IN-SHELL

what does this mean?

@jukkar ping

The curl error print is fixed by #9435

@tarunkum, @jukkar : Should this be closed then?

tested, works fine.


Update code and build scripts to use latest quiche(0.22.0) release.

This bumps the quiche version from the build deps scripts and update the code to use latest quiche version which changed some of their API:

quiche_conn_stream_recv
quiche_conn_stream_send

which now takes an extra param to get the  stream application error codes(if any).
Note:
error_code is not used for now. A new PR should add logic to deal with this outcome.
PS: : This needs CI to update quiche to 0.22.0. CI will fail till then.
fixes: https://github.com/apache/trafficserver/issues/11486

Will not merge this till we have rocky image updated.
Thanks,

[approve ci autest rocky]

[approve ci autest]

Cherry-picked to v10.0.x


Mathematical symbols identification

i am trying to identify mathematical symbol in an image( if 3+2=5 is a image then i need to identify + and + symbol) ie just like object detection. i there any methods to identify, then please let me know....

Please refer to #8


Fix wrong option to skip groups

There was a mistake in the skip group option:

As you see, the real name of the option is:

This pull fixes the issue

Thanks ^_^


👽 🔥 Fix/remove jwt

fix #131
my understanding is that we only remove the jwtoken used to set cookie with not the jwtoken used for the oauth state/{nonce} param.

@hillairet do we know of any app that's using the cookie thing from here? Just wonder if we need to flag this as a breaking change. thanks

Yeah that's a breaking change. I don't think it's used though.


Failed train_rpn.py

Hello, I've tried to run train_rpn.py , but I got some error like this. Maybe you can explain it to me
Using TensorFlow backend.
Traceback (most recent call last):
  File "train_rpn.py", line 18, in <module>
    from keras_frcnn import config_anc9 as config
ImportError: cannot import name 'config_anc9'

sorry, that was my test configuration.
by fixing line 18 to
from keras_frcnn import config

it should run properly. let me fix this.

I can run it now, but it's always running so long at epoch 1. Am I wrong at some steps?
{'retak_aligator': 179, 'retak_garis': 178, 'lubang': 174} Training images per class: {'bg': 0, 'lubang': 174, 'retak_aligator': 179, 'retak_garis': 178} Num classes (including bg) = 4 Config has been written to config.pickle, and can be loaded when testing to ensure correct results Num train samples 360 Num val samples 0 WARNING:tensorflow:From /usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/op_def_library.py:263: colocate_with (from tensorflow.python.framework.ops) is deprecated and will be removed in a future version. Instructions for updating: Colocations handled automatically by placer. loading weights from ./resnet50_weights_tf_dim_ordering_tf_kernels_notop.h5 loaded weights! Starting training Epoch 1/250 399/400 [============================>.] - ETA: 0s - loss: 3.9409 - rpn_out_class_loss: 3.4894 - rpn_out_regress_loss: 0.4515

hi @syifalaili26
it will depend on how large your dataset is.
You might want to test with a smaller dataset? and 250 epoch might be too long.. 20~50 should be enough.
it will be slow if you don't have GPU acceleration too.

I use 360 datasets, how many epoch normally should I use?

20 epochs should be enough.
you should toggle few learning rates and find out which one works the best

Thank you, I'll try it again

I still can't train RPN, always loading at epoch 1, I've used 20 epoch as you suggest but still not working. Can you help me?

@syifalaili26
What is the problem you are getting? if it's a error, it would help if you post the message.
Or do you get stuck at the end of epoch 1?
Starting training Epoch 1/250 399/400 [============================>.] - ETA: 0s - loss: 3.9409 - rpn_out_class_loss: 3.4894 - rpn_out_regress_loss: 0.4515

instead of your custom datasets, can you try running with known datasets like VOC2007 or KITTI?


Find a better organizational scheme for inter-language integration / protocol tests and integration tests between Apache Arrow and third party projects

Some integration tests with 3rd party projects have gotten moved to https://github.com/apache/arrow/tree/master/integration which doesn't look right to me. I suggest we either find a new home in the codebase for the protocol integration tests or move the 3rd party integration tests somewhere else
Reporter: Wes McKinney / @wesm
Related issues:

[Python/CI] Add integration test with kartothek (is related to)

Note: This issue was originally created as ARROW-3975. Please see the migration documentation for further details.

This is quite outdated. The 3rd party integration tests are now handled through Crossbow.


jasper migration to python3

Hi,
How we can migrate jasper to python3? currently jasper runs uisng python jasper.py while python3 gives error related to yaml module not found.

As python 2 is no longer getting any security fixes, migration to python 3 would be most welcome!

Since Python 2 is being dropped, might be a good idea to migrate to Python 3.x
The current state of Jasper Voice Control can't be installed when it's trying to look for python2-tzlocal.
AUR Issues : → https://aur.archlinux.org/packages/jasper-voice-control-git
a list of issues makes it impossible to install, because it asks for packages that no longer exist or provide any support for.
Failed to prepare transaction:
could not satisfy dependencies:
- unable to satisfy dependency 'python2-tzlocal' required by python2-apscheduler
- unable to satisfy dependency 'python2-apscheduler' required by jasper-voice-control-git
- unable to satisfy dependency 'jasper-voice-control' required by jasper-tts-festival
Resolving dependencies...
Warning: cannot resolve "python2-tzlocal", a dependency of "python2-apscheduler"
Warning: cannot resolve "python2-apscheduler", a dependency of "jasper-voice-control-git"
Warning: cannot resolve "jasper-voice-control", a dependency of "jasper-tts-festival"
Failed to prepare transaction:
could not satisfy dependencies:
- unable to satisfy dependency 'python2-tzlocal' required by python2-apscheduler
- unable to satisfy dependency 'python2-apscheduler' required by jasper-voice-control-git
- unable to satisfy dependency 'jasper-voice-control' required by jasper-tts-festival

the dependencies need to resolve by pointing to current Python libraries for 3.x

Open to help w porting to python3: https://github.com/appeacock/jasperbuster

Hey guys, I have my own Jasper to Py3 project. Please help - https://github.com/Akul2010/jasper-client-for-python-3-


ERROR:browser_gpu_channel_host_factory.cc(103)] Failed to launch GPU process.

It's unclear why this is happening. This is on a push of the master branch as of today's date. Host's config.txt:
disable_splash=1
dtparam=i2c_arm=on
dtparam=spi=on
dtparam=audio=on
enable_uart=1
gpu_mem=160

Forgot to say that this is an rpi3 model B, running on an HDMI display.

@brownjohnf can you paste more logs around the error please?

also, what's the output on the screen after you see this error?

@curcuz Output on the screen is just white/light gray. Prior to the startup/error, the screen is the default resin display with the ASCII Resin.io on the screen.
27.02.18 22:08:50 (-0800)  main  Systemd init system enabled.
27.02.18 22:08:50 (-0800)  main  systemd 215 running in system mode. (+PAM +AUDIT +SELINUX +IMA +SYSVINIT +LIBCRYPTSETUP +GCRYPT +ACL +XZ -SECCOMP -APPARMOR)
27.02.18 22:08:50 (-0800)  main  Detected virtualization 'other'.
27.02.18 22:08:50 (-0800)  main  Detected architecture 'arm'.
27.02.18 22:08:50 (-0800)  main  Set hostname to <cb0459c-cb0459c>.
09.04.18 14:56:00 (-0700)  main
09.04.18 14:56:00 (-0700)  main
09.04.18 14:56:00 (-0700)  main  X.Org X Server 1.16.4
09.04.18 14:56:00 (-0700)  main  Release Date: 2014-12-20
09.04.18 14:56:00 (-0700)  main  X Protocol Version 11, Revision 0
09.04.18 14:56:00 (-0700)  main  Build Operating System: Linux 4.9.0-4-armmp-lpae armv7l Debian
09.04.18 14:56:00 (-0700)  main  Current Operating System: Linux cb0459c-cb0459c 4.9.59 #2 SMP Wed Apr 4 15:17:24 UTC 2018 armv7l
09.04.18 14:56:00 (-0700)  main  Kernel command line: 8250.nr_uarts=1 bcm2708_fb.fbwidth=1680 bcm2708_fb.fbheight=1050 bcm2708_fb.fbswap=1 vc_mem.mem_base=0x3ec00000 vc_mem.mem_size=0x40000000  dwc_otg.lpm_enable=0 console=tty1 console=ttyS0,115200 root=/dev/mmcblk0p2 rootfstype=ext4 rootwait
09.04.18 14:56:00 (-0700)  main  Build Date: 16 October 2017  09:26:27AM
09.04.18 14:56:00 (-0700)  main  xorg-server 2:1.16.4-1+deb8u2 (http://www.debian.org/support)
09.04.18 14:56:00 (-0700)  main  Current version of pixman: 0.33.3
09.04.18 14:56:00 (-0700)  main         Before reporting problems, check http://wiki.x.org
09.04.18 14:56:00 (-0700)  main         to make sure that you have the latest version.
09.04.18 14:56:00 (-0700)  main  Markers: (--) probed, (**) from config file, (==) default setting,
09.04.18 14:56:00 (-0700)  main         (++) from command line, (!!) notice,(II) informational,
09.04.18 14:56:00 (-0700)  main         (WW) warning, (EE) error, (NI) not implemented, (??) unknown.
09.04.18 14:56:00 (-0700)  main  (==) Log file: "/var/log/Xorg.0.log", Time:Mon Apr  9 21:56:00 2018
09.04.18 14:56:00 (-0700)  main  (==) Using system config directory "/usr/share/X11/xorg.conf.d"
09.04.18 14:56:11 (-0700)  main  [177:0409/215611.457673:WARNING:audio_manager.cc(293)] Multiple instances of AudioManager detected
09.04.18 14:56:11 (-0700)  main  [177:0409/215611.458306:WARNING:audio_manager.cc(254)] Multiple instances of AudioManager detected
09.04.18 14:56:12 (-0700)  main  [177:0409/215612.729990:ERROR:browser_gpu_channel_host_factory.cc(103)] Failed to launch GPU process.
09.04.18 14:56:12 (-0700)  main  [177:0409/215612.732822:ERROR:browser_gpu_channel_host_factory.cc(103)] Failed to launch GPU process.
09.04.18 14:56:12 (-0700)  main  [177:0409/215612.734807:ERROR:browser_gpu_channel_host_factory.cc(103)] Failed to launch GPU process.
09.04.18 14:56:12 (-0700)  main  [177:0409/215612.747641:ERROR:browser_gpu_channel_host_factory.cc(103)] Failed to launch GPU process.
09.04.18 14:56:12 (-0700)  main  [177:0409/215612.749737:ERROR:browser_gpu_channel_host_factory.cc(103)] Failed to launch GPU process.

I'm getting the same issue as well, same pi version
23.04.18 18:25:04 (+0100) X.Org X Server 1.18.4
23.04.18 18:25:04 (+0100) Release Date: 2016-07-19
23.04.18 18:25:04 (+0100) X Protocol Version 11, Revision 0
23.04.18 18:25:04 (+0100) Build Operating System: Linux 4.4.26-v7+ armv7l Raspbian
23.04.18 18:25:04 (+0100) Current Operating System: Linux 4056ff1 4.9.50 #2 SMP Wed Nov 1 16:03:39 UTC2017 armv7l
23.04.18 18:25:04 (+0100) Kernel command line: 8250.nr_uarts=0 bcm2708_fb.fbwidth=800 bcm2708_fb.fbheight=480 bcm2708_fb.fbdepth=16 bcm2708_fb.fbswap=1 vc_mem.mem_base=0x3f000000 vc_mem.mem_size=0x3f600000 dwc_otg.lpm_enable=0 console=null root=/dev/mmcblk0p2 rootfstype=ext4 rootwait vt.global_cursor_default=0
23.04.18 18:25:04 (+0100) Build Date: 11 November 2016  11:59:59AM
23.04.18 18:25:04 (+0100) xorg-server 2:1.18.4-2+rpi1 (https://www.debian.org/support)
23.04.18 18:25:04 (+0100) Current version of pixman: 0.33.3
23.04.18 18:25:04 (+0100)       Before reporting problems, check http://wiki.x.org
23.04.18 18:25:04 (+0100)       to make sure that you have the latest version.
23.04.18 18:25:04 (+0100) Markers: (--) probed, (**) from config file, (==) default setting,
23.04.18 18:25:04 (+0100)       (++) from command line, (!!) notice, (II) informational,
23.04.18 18:25:04 (+0100)       (WW) warning, (EE) error, (NI) not implemented, (??) unknown.
23.04.18 18:25:04 (+0100) (==) Log file: "/var/log/Xorg.0.log", Time: Mon Apr 23 17:25:04 2018
23.04.18 18:25:04 (+0100) (==) Using system config directory "/usr/share/X11/xorg.conf.d"
23.04.18 18:25:10 (+0100) Application is already running 'registry2.resin.io/v2/99aff053459dd9b3fd9f9f872d7d176b'
23.04.18 18:25:23 (+0100) [187:0423/172523.770681:WARNING:audio_manager.cc(293)] Multiple instances ofAudioManager detected
23.04.18 18:25:23 (+0100) [187:0423/172523.771234:WARNING:audio_manager.cc(254)] Multiple instances ofAudioManager detected
23.04.18 18:25:25 (+0100) [187:0423/172525.438505:ERROR:browser_gpu_channel_host_factory.cc(103)] Failed to launch GPU process.
23.04.18 18:25:25 (+0100) [187:0423/172525.438830:ERROR:browser_gpu_channel_host_factory.cc(103)] Failed to launch GPU process.
23.04.18 18:25:25 (+0100) [187:0423/172525.439068:ERROR:browser_gpu_channel_host_factory.cc(103)] Failed to launch GPU process.
23.04.18 18:25:25 (+0100) [187:0423/172525.439239:ERROR:browser_gpu_channel_host_factory.cc(103)] Failed to launch GPU process.
23.04.18 18:25:25 (+0100) [187:0423/172525.439379:ERROR:browser_gpu_channel_host_factory.cc(103)] Failed to launch GPU process.

Anyone know what's causing this?

+1

Following worked for me:

changed dependencies to: "electron": "~1.6.16" and "electron-rebuild": "^1.7.2"
in main.js add app.disableHardwareAcceleration(); to get rid of the browser_gpu_channel_host_factory.cc(103)] Failed to launch GPU process. error

also v2.0.0-beta.8 works (with added bonus of loading pdf's in iframes) see https://github.com/resin-io/resin-electronjs/issues/70


Rename Mutex#destroy -> Mutex#finalize.

Remove redundant pointerof in ConditionVariable.

Thanks! Some time ago passing self didn't work (compiler error), it seems that's fixed now. I wonder in what commit that happened, but in any case, yes, passing self is easier now :-)


Refactor

Much prefer the architecture of func main of server to signer , which was changed by @cyli  , it's more simple and clear.
In my option it's a nicer implementations that we pass the config to each component rather than use the global variable mainViper. That makes notary easier to run, to embed, to reuse and to test.
I think maybe we can do the same things on the signer side  just like what the server does.

Can one of the admins verify this patch?

Jenkins, test this please (Oh come on J-God, If you can hear me , I'll buy you a beer)

The -4.48% seems caused by the Yubikey tests.

Jenkins, please test this

please test this

Jenkins, test this please.

@riyazdf thanks :)   LGTM!  Thanks for the cleanup @HuKeping!

We have some config changes pending in the rethinkdb PR (just adding rethink setup). We want to get that feature in this week so this will need to be updated once we get that PR merged. Otherwise, looks good so far.

OK, I'll rebase it once that was ready

Jenkins, test this please

Jenkins, test this please

Jenkins, test this please

Jenkins, test this please

@HuKeping it took a little longer than we hoped but rethinkdb is finally merged. If you can rebase and fix any conflicts we'll get this PR moving :-)

@endophage sure, will do it later

rebased

Jenkins, test this please.

:|  Codecov seems to have some issues with things coming in late I guess?  Even though the status hasn't been updated, clicking on the codecov link gives us a percentage of 74.65% for this PR.

LGTM!  Thanks @HuKeping

yeah I don't know why codecov isn't updating but this LGTM!


About Tanks ang Temples

Hello, thank you very much for your excellent work, I recently reproduced your work following the tutorial you posted, but there is a little problem, I need your help.

The quantitative results in the DTU dataset are basically the same as the results presented in your paper, but the evaluation effect in the TNT dataset is very poor, not even half of the data you present, I would like to ask if there is any good solution. I would appreciate it if you could tell me how
Is it possible that there is a problem with my steps, I will now roughly describe it, you see if it is appropriate. Firstly, the DTU dataset is used for training, the trained model is put into the blended dataset for fine-tuning to obtain the blended training model, and finally the blended training model is used as a checkpoint to test the TNT dataset.

Hello, your training or fine-tuning steps are correct.
Due to the limited information provided so far, I am not sure what the specific problem is.
However, the result of this anomaly is likely to be a problem with the data set, you can check it and download the data set we provided here.
And you can first use the checkpoint we provided to test whether you could get reasonable TNT quantitative results.

I just need to change the TNT test set, or all the data sets, my DTU evaluation is similar to yours
---Original---
From: "Tianqi @.>
Date: Sun, Jul 7, 2024 16:32 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [TQTQliu/ET-MVSNet] About Tanks ang Temples (Issue #22)
Hello, your training or fine-tuning steps are correct.
Due to the limited information provided so far, I am not sure what the specific problem is.
However, the result of this anomaly is likely to be a problem with the data set, you can check it and download the data set we provided here.
And you can first use the checkpoint we provided to test whether you could get reasonable TNT quantitative results.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Hello author, I'm sorry to bother you again, I followed the method you provided, used the dataset you provided, and even used the pre-trained model you provided, and still did not get effective visualization results, I have not yet evaluated. Is there any other solution, could it be a problem with the environment, the python version I use is 3.8.18

Hello author, I'm sorry to bother you again, I followed the method you provided, used the dataset you provided, and even used the pre-trained model you provided, and still did not get effective visualization results, I have not yet evaluated. Is there any other solution, could it be a problem with the environment, the python version I use is 3.8.18

Hello, can you show the visualization results?

Hello, thank you very much for replying to me again! According to your advice, I downloaded the TNT data set you recommended, and then tested it with the pre-training model you gave, but the effect was still poor, there was noise around the point cloud. Today, I saw that you mentioned the environment problem in your reply to others, I tried to reinstall the environment, and found that some packages you provided did not support the python version 3.10.8. Below is my point cloud visualization, tested with the training model you gave.
---Original---
From: "Tianqi @.>
Date: Tue, Jul 9, 2024 13:36 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [TQTQliu/ET-MVSNet] About Tanks ang Temples (Issue #22)
Hello, can you show the visualization results?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

Hello, thank you very much for replying to me again! According to your advice, I downloaded the TNT data set you recommended, and then tested it with the pre-training model you gave, but the effect was still poor, there was noise around the point cloud. Today, I saw that you mentioned the environment problem in your reply to others, I tried to reinstall the environment, and found that some packages you provided did not support the python version 3.10.8. Below is my point cloud visualization, tested with the training model you gave.
---Original---
From: "Tianqi @.>
Date: Tue, Jul 9, 2024 13:36 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [TQTQliu/ET-MVSNet] About Tanks ang Temples (Issue #22)
Hello, can you show the visualization results?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Hello, thank you very much for replying to me again! According to your advice, I downloaded the TNT data set you recommended, and then tested it with the pre-training model you gave, but the effect was still poor, there was noise around the point cloud. Today, I saw that you mentioned the environment problem in your reply to others, I tried to reinstall the environment, and found that some packages you provided did not support the python version 3.10.8. Below is my point cloud visualization, tested with the training model you gave.

Based on the visualizations presented, I think the results are reasonable and you can submit to get quantitative results. Looking forward to your feedback.

Hello author I read your personal profile, very impressed, I am also a graduate, convenient to leave personal contact information, I may have a lot of questions to consult
---Original---
From: "Tianqi @.>
Date: Tue, Jul 9, 2024 14:12 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [TQTQliu/ET-MVSNet] About Tanks ang Temples (Issue #22)
Based on the visualizations presented, I think the results are reasonable and you can submit to get quantitative results. Looking forward to your feedback.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Hi, this is my reproduction, using my own training model, slightly worse than the data in your paper. How can this final figure be the same or very close to yours
---Original---
From: "Tianqi @.>
Date: Tue, Jul 9, 2024 14:12 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [TQTQliu/ET-MVSNet] About Tanks ang Temples (Issue #22)
Based on the visualizations presented, I think the results are reasonable and you can submit to get quantitative results. Looking forward to your feedback.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Sporadically failing CI testcases

We have some sporadically failing testcases, which blocks CI.
Has been discussed in #101 among other PRs. But I'm opening an issue to keep the discussion in one place.
I'm able to reliably reproduce the failure on my machine like so:
I=1; while cargo nextest run test_read_line; do I=$(($I+1)); echo "PASS $I"; done; echo "FAIL $I"

    Starting 1 tests across 1 binaries (19 skipped)
        FAIL [   0.207s] rexpect session::tests::test_read_line

--- STDOUT:              rexpect session::tests::test_read_line ---

running 1 test
test session::tests::test_read_line ... FAILED

failures:

failures:
    session::tests::test_read_line

test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured; 19 filtered out; finished in 0.21s

--- STDERR:              rexpect session::tests::test_read_line ---
thread 'session::tests::test_read_line' panicked at 'assertion failed: `(left == right)`
  left: `"hans"`,
 right: `"hanshans"`', src/session.rs:437:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

   Canceling due to test failure: 0 tests still running
------------
     Summary [   0.207s] 1 tests run: 0 passed, 1 failed, 19 skipped
        FAIL [   0.207s] rexpect session::tests::test_read_line
error: test run failed
FAIL 270

It failed after 270 runs of the test. I'm running Ubuntu 20.

So, I removed the code that turn echo off:
https://github.com/rust-cli/rexpect/blob/52edda700d39edabc1a7abcfa2d45ddb14e629c0/src/process.rs#L121-L123
That results in test_bash and test_bash_control_chars consistently failing. But it seems to remove the flakyness from test_read_line.
Very odd to me. I guess bash and cat set some flags inside their own process that affects us differently?


Feature Request: Allow ignoring errors by code

The Rust compiler emits a very large number of different error messages under the same code rustc(dead_code).
I want to ignore all of them but at the moment I have to specify the text of each one in errorLens.exclude and new messages are added from time to time.
I would be nice to be able to just ignore the entire code and not worry about it.

Using code without source doesn't make sense. I've added ability to exclude source/code pair to the "errorLens.excludeBySource" setting:
"errorLens.excludeBySource": [
	"eslint(padded-blocks)",
],


[v6] Upgrade Solidity to 0.8.17

Upgrade Solidity to 0.8.17 (foundry doesn't know 0.8.18 yet)

Utils (library, abstract etc.) uses ^0.8.0
Interfaces uses >=0.8.0

Upgrade openzeppelin, forge-std to latest
Categorize contracts into abstracts, interfaces and libraries for clarity (instead of a single big utils)
Remove v5 files (contracts, docs etc.)
Remove unused node packages

Categorize contracts into abstracts, interfaces and libraries for clarity

I'm not sure abstracts category is going to help as we don't operate with the awareness of "which contract is an abstract contract". Or more broadly speaking, separating them into abstracts, interfaces and libraries carry meaning to a compiler but not so much to a developer so it does not help much in my opinion.
After looking at the contracts in abstracts, maybe we can just treat them as library (if not utils)? which means contracts in libraries do not necessary have to be Library type.

Originally a Lib prefix is added to library files in order to help importer to determine what type of imported contract. I think @pilagod is trying to separate them by path so no prefix is needed now.

I think abstract contract and library do have differences that developers must be aware. For abstract contract we can only use it by inheritance, while library is just like a namespace for a collection of utils.
It's clearer to me to separate them. When importing things from abstracts folder, we will know them should be inherited by contract at the beginning. And when importing things from libraries, we will know them could be used directly or augment other types.
And just like @charlesjhongc said, we can also get rid of the prefix/suffix like BaseXXX and LibXXX, which adds no much meaning to the util.
import { A } from "contracts/abstracts/A"
import { B } from "contracts/libraries/B"

contract Strategy is A {
    using B for type;

    constructor() A() {}

    action() external {
        B.action()
    }
}

But this would require developers to remember which contract is abstract, which is not right? Because neither the name EIP712, Multicall or SignatureValidator strike me as an abstract contract (more like library). But if number of contracts is kept low then I guess it won't be too much of a problem.

Developer doesn't need to remember, they can just go through these two directories to see which function he needs (just like we go through single utils folder). For example, when he wants to use Multicall, he can find and see it in the abstracts folder, then he knows it is a abstract contract, and he has to inherit it before usage.
Current popular projects is also not favor of one single utils folder with prefix/suffix to distinguish files, rather they group utils by meaning or characteristic. For example, in Uniswap libraries only for libraries, and base folder for abstract contracts. Another example is Openzeppelin, they structure folders purely by meaning (presets, extensions), and without any file prefix/suffix.
To summarize, imo one big utils folder is not a good way to manage a growing project in the long run. A more meaningful structure would help developers to understand and develop in a clear way, and I personally prefer Uniswap structure more than Openzeppelin.

Developer doesn't need to remember, they can just go through these two directories to see which function he needs (just like we go through single utils folder).
To summarize, imo one big utils folder is not a good way to manage a growing project in the long run. A more meaningful structure would...

I think that's where I would disagree because the structure is not meaningful in my opinion. I definitely prefer not having one big folder but if today we separated files into different folders and I still have to "go through every folder to find the file I'm looking for" then the separation is doing the opposite of helping.
I don't have a better idea of how to categorize the files right now and it's not a big problem so let's go with this design (abstract/libraries) now and see how it plays out.

I'm positive with this direction. At least I think having the libraries path make sense to me now. But for the abstract path, perhaps we can separate contracts by their function or subject instead of their type of contract? It can be adjusted later and I think this PR is good to go.

OK, let's go for current structure at this moment. Feel free to open PR to refine it if there is any good ideas 💪

@charlesjhongc @NIC619 I also remove L2 relevant interfaces for now. If we need them later, we can move them back from v5 branch. Check if you guys have other questions, or we can be ready to merge it to start building our new v6 protocol 💪


Flux executable supports generic git bootstrap

the flux executable will need to support bootstrap git in addition to the existing bootstrap github.
currently the flux executable package only supports bootstrap github.
we need to add a layer under this method which looks at the FluxConfig, checks which provider is present (github vs git), and then executes a sub-command like fluxBootstrapGit or fluxBootstrapGithub, passing in the relevant configuration values.
We should consider setting up the options pattern for flux so that we can pass options to a generic method like fluxBootstrap(...opts FluxBootstrapOpts) which sets the command opts for flux.
the ultimate goal of this ticket is to set us up so that if the git field of the FluxConfig is non-nil we properly process it and run the right bootstrap command.

/close


How to configure XLSForm conversion service with central-backend

I'm trying to connect the ODK central backend to a XLSForm conversion server (https://github.com/getodk/xlsform-online) to allow upload of XLSForm formatted forms.
When I enter the corresponding localhost server and port in the configuration at config/default.json and try to submit a form through the frontend I encounter an error SyntaxError: Unexpected token < in JSON at position 1 on the backend thrown by lib/util/xlsform.js.
In debugging the response body further it looks like the conversion server is returning an HTML error page mentioning the resource cannot be found:

Not Found

The requested resource was not found on this server.

How can I successfully configure central-backend to work with a conversion service? Am I just missing something big and using the wrong project to try to integrate conversions?

Hi @drkatz! I believe that https://github.com/getodk/xlsform-online is for XLSForm Online, which is a standalone web app. ODK Central uses https://github.com/yanokwa/pyxform-http as an XLSForm-to-XForm conversion API. Let us know if you have trouble with pyxform-http!
@issa-tseng, how about we link to pyxform-http from the readme?

Thanks @matthew-white! This pointed me in the right direction and things are working as they should now.
A link to the repository in the readme would definitely be helpful so users like myself can find the project as it's one of @yanokwa's personal repos (I was searching around the getodk organization)

@matthew-white yeah that makes sense. probably we can add an external interfacing section with the additions coming for block 9

I think pyxform-http should officially join the ODK organization and I've asked the TSC to consider this in our next meeting. https://forum.getodk.org/t/requesting-for-pyxform-http-to-become-an-odk-tool/26237.


Will you add support for HTTPS?

Just add HTTPS

You can add your own HTTPS support downloading, compiling and moving OpenSSL include files to your VStudio include folder and modifying the header to define HTTPS support.
Also you can search for pre compiled include OpenSSL files for your system.

Now you can refer to this repository，https://github.com/Nevermore1994/http-request


Only warn about protoc version if explicitly set to <3.2.x

There are some cases where it does not make sense to specify the protoc version - for example, when compiling with buf. (See https://github.com/bufbuild/buf/issues/1244 and https://github.com/bufbuild/buf/issues/1736). Furthermore, the compiler version is not read elsewhere right now.
This change updates the behavior to only warn if the compiler version is explicitly set and is < 3.2.x.

So the version is used, to provide the warning that the descriptor set being generated from may not have the correct support to get properly working JSON from the Swift library.
Skimming the links you provided, I think maybe the purpose of the version number might be misunderstood.
The interface between protoc and the plugins is serialized descriptors. The protobuf format provides some degree of api safety because descriptors do (old versions and parse newer version, etc.). Since protoc to a plugin is a cross process api boundary, the version provides the plugin additional knowledge about the calling process, a way to know what other things might have been done to the data being received.
In the case of this warning, the conversion from proto field names to the json strings wasn't formally documented, and for a while protoc was always providing the strings in the descriptors, so the warning is reporting when a developer is likely using some old version so we can't guarantee that what the swift plugin will generate will interoperate correctly since we rely on protoc making those strings.
If that tool really wants to replace the role protoc normally provides, it sorta has to ensure it is providing all of the validations/normalization/etc. that protoc does, and if it is stepping up to that, then it seems like it should be setting the protoc version to what it is guaranteeing to match to maintain the plugin api contract.  Future changes to protobuf could well evolve descriptor.proto and introduce new ways plugins might need to use the version to realize when protoc did/didn't do something a plugin relies one.
So if Apple wants to drop this warning, that's fine, but as I mention, that version is providing plugins information about the contract, and so it might be needed in the future to know about other differences.
One footnote here as to why Swift might seem a little different - the Swift plugin is written in Swift, but a lot of other plugins get written in C++. The ones written in C++ get a little extra safety net because they get the official protobuf C++ code that is always used within protoc to re-validate/normalize the descriptor sets as they read them in. Since none of that is officially documented, no plugin not written with in C++ really has a way to ensure they are doing the same things and instead have to rely on the thing build the binary descriptor sets to have done it, and that version number is the only really want to check. Without checking it, developers would be left on their own to try to figure out when something generated doesn't work because the normal checks relied on didn't happen.

We should not remove this check.  I commented on the buf thread about this issue to point out that by not setting this field, buf is explicitly choosing to proclaim compatibility with pre-3.2 protoc.  And in that case, the plugin has no choice but to assume that the JSON data is incomplete.

We've replied over there too - we'd respectfully disagree that choosing not to set this field is declaring pre-3.2 protoc compatibility, instead it is not making any such claim at all. We'd argue a more faithful implementation of theCodeGeneratorRequest API for SwiftProtobuf to make would be "if unset, do not warn, if set and pre-3.2, warn, else do not warn", however reasonable people can disagree here.

I updated the PR with the logic suggested by @bufdev, as I agree that it's probably a more faithful representation of the intent

If the buf developers want to provide some other way to recognize the buf version, then I'd consider recognizing that.  Of course, I'd be even happier if you negotiated with the protoc folks to make sure that there was a single uniform way for plugins to recognize the front-end being used.  I think that information would be very useful to include in the generated output sources as well.

Funny and somewhat-unrelated story, one of the original reasons we stopped producing this value was because plugin authors added the version number in their generated sources :-) It ended up causing many diffs with no functional change for many of our users, and we came to the opinion that it was an anti-pattern that was causing developer misery via unnecessary diffs.
Anyways, I think the fundamental disagreement here is that, in our view, plugins should not alter their behavior based on what version of protoc or buf produces a CodeGeneratorRequest for the plugin, and that the behavior of the plugin should remain the same, regardless of if protoc, buf, some other compiler, or even a handwritten CodeGeneratorRequest serialized to a file and piped to the plugin via stdin, is the source of the CodeGeneratorRequest. The compiler_version field (in our view) should be treated as entirely optional metadata. In that vein, it seems very reasonable to warn if you explicitly detect that the CodeGeneratorRequest was produced by a compiler version the plugin author knows will result in certain undesirable behaviors - for example, if the plugin knows that certain (by definition optional) fields on the FileDescriptorSet will not be set for protoc <3.2, it seems reasonable to warn about this. In the absence of this optional metadata, however, the warning likely should not be printed.
Even better would be for SwiftProtobuf to handle the case where <3.2 versions of protoc do not produce FileDescriptorSets that contain whatever they need for proper JSON handling, but this is beyond the scope of my understanding of this specific plugin - I know Protobuf, but I definitely don't know Swift or this plugin implementation, and I'm sure you thought of this heh.

Closing.


Engine uses wrong directory for appdata on Windows

Godot version
not version specific
System information
Windows 10
Issue description
According to the documentation engine uses the %appdata% location to store application data. This location points to ..user/AppData/Roaming/ directory, intended specifically for the data that supposed to be synchronizable between multiple user PCs (that share Microsoft account). For Godot using this directory makes no immediate sense, and looks more like a mistake. In fact with no synchronization strategy in place it would only work correctly so long as there is no synchronization taking place.
The appropriate location for the local-only application data is %LocalAppData% .
Steps to reproduce
n/a
Minimal reproduction project
n/a

While Local would make more sense the majority of games are actually using LocalLow on Windows.

Roaming folder contains data that can move with the user server profile from computer to computer
Local folder contains data that cannot move with a user profile.
LocalLow folder same as Local but includes restricted low-level access.

On my system the balance between Roaming / Local / Locallow is like 6 / 17 / 87 games.
It makes sense for the Godot Editor to place its own data into the Roaming folder.
It makes mostly no sense for a Godot game project to place its data in the Roaming folder. It should be Local or LocalLow.
The LocalLow folder adds an extra level of security for programs that know they are vulnerable, e.g. due to multiplayer and web connect. So this way they can start themselfin in "low" access mode so that they can only write into those folders and not risk the rest of the hard drive.

Some games like Xonotic use the elusive Saved games directory too. This directory has the upside of not being an hidden directory, but it's still difficult to discover. It may be the least well-known standard Windows library folder.
There's also the informal standard Documents\My Games, but this one won't be internationalized as it's a hardcoded name.

There is a way to change "Saved Games" location. It's stored in the registry.

reg query "HKEY_CURRENT_USER\Software\Microsoft\Windows\CurrentVersion\Explorer\User Shell Folders"

https://www.elevenforum.com/t/move-or-restore-default-location-of-saved-games-folder-in-windows-11.8717/


Add a parameter in write_report to control console output

When using markdown (and probably Jupyter and similar tools) for generating reports, the write_report statement generates a text output providing a lot of details. It is displayed after the code chunk like any console output.
Although this log could be useful in production (maybe it would be better redirected to logr if a log is opened), it would be great if we could instead display the table as TXT while in development, i.e. adding a parameter to write_report as console_output = c("log", "table", "both", "summary", "none") with "log" as default.
Depending on the development stage, we could switch between  [console_output="table", preview=1] and [console_output="log", preview=NULL].
"none" or "summary" would be interesting when generating several formats (typically RTF and PDF) of the same table; "summary"  would not output the details of the table and could be used for subsequent generations.
This shouldn't change the file creation itself, just what happens in the console.
When "table" is selected, we could display it in rstudio Viewer.

You can prevent writing to the console by sending the result to a variable.  Like this:

The write_report() output does go to the logr log automatically, if it is open and the autolog feature is turned on.  If you don't want it to go to the log, you can turn it off with log = FALSE on write_report(), or turn off autolog.
For the "table" option, there is the proc_print() function from the procs package.  This will send any data frame to the viewer.  It won't show the entire report, but you can at least see the data.
These comments somewhat address everything except the "summary" option.  Don't have anything like that currently. Will consider.

What is the point of the RMD?  Are you trying to get several TFLs into the same document, or are you creating an SAP or CSR where you are combining the tables with some other content?

I'm using it like a notebook (like I use Jupyter with python), mainly as a
reporting tool for ad-hoc analysis or other clinical presentations for
conferences or internally (board...). Usually a lot of graphs and small
tables, but it's always good to link or integrate the full corresponding
listings, so people could check etc... We are currently complementing some
CRO deliverables and Reporter helped us to easily keep the same format.
So, to answer your question, I guess that's a little of both.
On Sun, Dec 3, 2023 at 7:02 PM David J. Bosak @.***>
wrote:

What is the point of the RMD? Are you trying to get several TFLs into the
same document, or are you creating an SAP or CSR where you are combining
the tables with some other content?
—
Reply to this email directly, view it on GitHub
https://github.com/dbosak01/reporter/issues/320#issuecomment-1837763907,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABUR24OQNILWHTKMFBEX4ZDYHU4K7AVCNFSM6AAAAABAFBM3DOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZXG43DGOJQG4
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

I see.  What you need is issue #161.  The idea was to stream the table content back without creating a written file.  I can create both text and HTML reports.  But with the current functionality I don't know how to get them back to the RMD nicely.  Let me play with this and see what I can come up with.  Also I can look at #161 again and see how hard it is to implement.  If it is not too hard, I can do it over winter break.

This was an interesting experiment.  Thanks for making me explore it.  I was able to get the HTML into the RMD pretty cleanly using the includeHTML() function from htmltools.  Example attached.
MyDoc.txt
And here is what I'm getting:

There might be some interference with the style sheets.  But this will at least get you started.

Got it.  The page_header object always aligns to the container, either a page or browser.  But the titles can align to the container, the content, or a specific width.  Check out the "width" parameter on the titles() function.  If you attach the titles to the content, by default they will align to the content they are attached to.  Also, you can put multiple titles on an object.  Also there is a "columns" parameter that allows you to split the titles into 1 to 3 columns.  All this means you can achieve what you are trying to do by putting the page header information in a separate titles() function instead of the page_header() function.  Then set the columns to 2 and put your page header strings in there.
Here is what it looks like:

And here is the code:
MyDoc2.txt
Note that for footnotes, you'll also want to attach them to the content, and don't use the page_footer().


By default, a new QR template should have color black

The default color of a new QR template should be black instead of white.

Looks like it was already fixed the moment this issue was made and I reverted it


avoid release branches and release from main branch itself

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 75.239%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
6223

Relevant Lines:
8271

💛  - Coveralls


Add options to layout

Description
Adds options to layout:

hideHeader
hideBreadcrumbs
hideTOC

Also modifies some margin/padding to make this possible, but likely needs a good look to make sure the changes are not brittle.
Checklist

[ ] For UI or styling changes, I have added a screenshot or gif showing before & after
[x] I have added a changeset
[ ] I have added to the docs where applicable
[ ] I have added to the VS Code extension where applicable

Added another adjustment for TOC when header is hidden to push it up in line with the top of the sidebar.
Think this is good to go for now.
Other things this has made me think about re: layout control:

If someone hides the sidebar, will they expect to have the mobile menu available to navigate? Or would they want to be able to configure that? (e.g., something like hideSidebar={true} showHamburger={true})
We may need to expose control over the margins of the page. Especially for embedded use cases, if someone turns off all the nav options (header, sidebar, breadcrumbs, TOC) I think they will want fine grain control over the edges of the window
With hideHeader, we lose access to page options like show/hide queries, print, settings, etc.

I think there's a need for a dev toolbar for cases like this, and there's probably other stuff we can add to that toolbar to make it a really useful helper for people as they're developing in Evidence


S3E6

fix a rotating image

Why do you need to re-create the entire PR process to modify a rotated image? I guess you didn't update the local fork branch, causing all your modifications to be based on the local branch of the last pull. Please close the current PR and re-pull the local branch to submit the non-redundant part. Thank you.

The best path is as follows: update the local branch to the latest, delete the wrong picture and add the correct picture (keep the picture name consistent).


Improve numerics of abs jvp (and softplus)

After receiving reports of mysterious numerical issues with fourth-order grad of softplus, we merged #1315 as it seemed to resolve them, but didn't have a good story for what was actually going on.
Unfortunately, this led to a different numerical problem—the previous implementation of softplus as np.logaddexp(x, 0) was much better for large x, since it's implemented as max(x, 0) + log1p(exp(-abs(x))) (or, essentially, select(x > 0, x + log1p(exp(-x)), log1p(exp(x)))). But the version with max and abs wasn't behaving exactly like the one with select, since that had neither numerical issue while max+abs still had bad fourth-order grads.
Eventually I narrowed the grad issue down to the JVP rule for abs, which is jvp(abs)(x, g) = real(g * conj(x)) / abs(x), i.e. multiplying and then dividing by a number that might be very large or small. Since that is only needed for the complex case, this PR switches to a simpler JVP implementation for reals and reverts softplus to the more numerically-stable logaddexp-based implementation.
So now we have both grad(grad(grad(grad(nn.softplus))))(1e-8) = -0.125 and nn.softplus(89.) = 89.0.
CC @yasamanb @Sohl-Dickstein @mcoram

Nice! Can you add test cases for all these numerical conditions? We want to notice any regressions on them, and it sounds like they're straightforward to test.

Added tests for the specific issues noticed. I guess I should maybe also test the abs jvp directly?


Overlapping rendering

Possibly compromised data?

Something wrong with coordinates creation?

https://github.com/BlackHawkSigma/universal-defect-explorer/blob/a41c99e7b1474a58feac3e6698840a9a665c37af/web/src/utils/convertRawData.ts#L21-L23


Fallback value should be close to original value

Describe the bug
fallback value for "SystemControlBaseHighAcrylicElementBrush" is "SystemBaseHighColor" which has a completely opposite contrast as the acrylic color (white in acrylic as compared to black in solid color)
Steps to reproduce the bug
Use "SystemControlBaseHighAcrylicElementBrush" as background color and have text on top of it and set to battery saver.
Expected behavior
The text should be visible and readable
Screenshots
 - without battery saver

with battery saver

All devices

As a safe check, I would also check Fallback values for all the acrylic brushes and colors


Throw an error if vpnPort is NaN

We currently apply the default vpn port number if this is the case,
which might be confusing.
See https://github.com/resin-io/resin-device-config/pull/26#discussion_r52496039

revalidator appears to have a cast option ( https://github.com/flatiron/revalidator#available-options ), would that be a better way to do this?

@Page- Nice, works fine!

LGTM


Avatar image tap event

Would you create an avatar image event delegate or callback? Or is there any easy way you suggest to add a listener to avatar images?

@ebsaral: Thanks for creating the very first issue :+1:  and I'm sorry for this late response, I somehow missed the notification email.
I'm not sure what type of event you are in need of, so I'm assuming you are talking about touch events.
Ideally, MessageCellNode should support a delegate with various methods, including tap on avatar, tap on bubble, etc. I haven't implemented it yet, but it is definitely high in my todo list. Any help on this is very welcome.
For the time being, I guess the easiest way is to add touch handling directly to the avatar image node. Some useful methods can be found here.


Fix LSTM and GRU layers gradient calculations

Description
Fix for LSTM and GRU layers without DNNL enabled give wrong gradients #17898
For bidiractional LSTM with number of layers > 2 input gradient calculation was incorrect.
Reason of wrong calculations was overwriting y derivative (dy) tensor by
calculated x derivative (dx) tensor before right2left layer could use dy for own
gradient calculations.
For GRU with number of layers > 2 i2h_weight gradient for
layers in the middle (all except last and first) was incorrect.
Wrong caluculations were caused by assigning output pointer to
input instead of calculating new input pointer.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] Changes are complete (i.e. I finished coding on this PR)
[x] To the best of my knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Thanks for your contribution. CC @pengzhao-intel @TaoLv @ciyongch
@bgawrych Could you please rename the title to a more concise and specific one? When the PR merged, the title will become the commit message. Thanks.

Thanks for your contribution. CC @pengzhao-intel @TaoLv @ciyongch
@bgawrych Could you please rename the title to a more concise and specific one? When the PR merged, the title will become the commit message. Thanks.

Of course, I overlooked this. Sorry :)

Thanks @bgawrych yes, we need to cherry pick this to 1.x.

@zixuanweeei  @ciyongch  please help take a review :)

@bgawrych  let me know when all internal cases and performance test passed and then I will merge the PR

@mxnet-bot run ci [windows-gpu]

@pengzhao-intel
I think it's ready. I don't see any performance drops in internal CI and all test passed.

@zixuanweeei @ciyongch @pengzhao-intel Please take another look at the new commits and also make sure your comments are addressed. Thanks!

Please also cherry-pick to 1.x branch.

@bgawrych , please cherry-pick to both v1.7.x and v1.x branch (as these two branches are diverged now), so that this patch will be included in the upcoming 1.7.0 release.


Support for increase/decrease columns in update.

Is it just me or currently the only way for increasing/decreasing values in column is $model->setCol($model->getCol + 1) which is prone to the error since it's possible that original value has been changed meanwhile.
Is it possible (or would it be worth to implement if not) to use something like sql update
update table set col=col+x still using object notation?
I.e. something like
$movie->setLastView(time())
           ->increaseNumViews(1)
           ->save()

It would be a great help for every currency related project

Even with an increment in the update SET directly is prone to race conditions since you need to hold the lock in both cases to make it atomar. A better way of dealing with this is optimistic locking.

In the concrete case of a counter column you dont need optimistic locking but a per statement atomicity is enough (you just need to protect against information loss, order of statements doesnt matter)

Ah, now after thinking more about I guess I get your point: You only want to increase the number and you don't care about the current value. So yeah, then the update method you proposed is a good way, although already not so sexy possible. So I'd give +1 for a method like incrementFieldX or decrementFieldX().

Yes marcj that's exactly what I meant.
Plus I guess that in case an incrementFieldX or decrementFieldX is used we should also launch reload action on that model (or include a recommendation  to do that in function docs) since it would be the only way to know original value.

@Alekc yes absolutely. Would you mind creating a pull request? :)

I am not sure I would be the best person to do that since my knowledge of Propel's internals is quiet limited :( (adding such a feature without having the whole picture in mind is not such a great idea).
I've read your last message again and noticed (just now) that you were speaking about the QueryClass, while I was thinking more about the Entity (Base) class.
I.e.
$user->increaseLoginCounter(1)
        ->setLastLoginTime(time())
         ->save()

Although this kind of approach obviously has it's disadvantages, such as:

Possible conflicts (what if user do php $user->setLoginCounter(10)->increaseLoginCounter(1).) Can probably be worked around if we add counters code to the end of the query. I.e.

CREATE TABLE User
    (`id` int, `loginCounter` int)
;
INSERT INTO User
    (`id`, `loginCounter`)
VALUES
    (1, 1)
;
update User set loginCounter=20, loginCounter=loginCounter+1 where id=1;

This would work on Mysql, not sure about Postgres
Or (not sure how that would work in propel base code), when calling for increaseLoginCounter(xx) it will use ->setLoginCounter(IncrementObjectInstanceOfSomeKind), in that case increase/set will override previous values.

As I said earlier, we will need to enforce reloading of an instance in case increment/decrement has been used.