Looking for GHL marketplace app SSO implementation documentation

I wouild like to apologize by creating an issue to get your attention. I'd be happy to send you a virtual coffee or beer for your help.
I am simply looking for any documentation that discusses how GHL is implementing SSO for marketplace apps. I see your flow diagram, but it's missing the key point of how the session is encrypted so that I can understand how to decrypt it. I thought decryption would be a bit more straight forward, similar to your NodeJS example, but it is not as easily implemented in Python.
I'm using the pycryptodome library and the AES module that it provides. I'm base 64 decoding the SSO key and session payload into their byte equivalents. The data I'm seeing does line up with AES specifications for key size and block size on the payload. I've tried all the AES modes as well and I cannot get the decryption process to yield anything but gibberish. I'm assuming I should be seeing byte data that looks like ASCII json to be parsed as a last step, similar to your example.
Any help would be appreciated, as I'm stuck here. Let me know what I can do to show my appreciation.

Here's a bit more detailed walkthrough of the flow and how GHL implements SSO session encryption.
https://www.youtube.com/watch?v=3rveQDuVlR0

For anyone finding this issue later on, give this blog post by @seankerr a look:
https://seankerr.dev/posts/decrypting-highlevel-sso-session-using-python/
Closing this now but feel free to reopen it if needed.


World gas metrics and optimizations

continuation of https://github.com/latticexyz/mud/issues/1133

[ ] ResourceSelector
[ ] AccessControl
[ ] Call
[ ] World
[ ] EphemeralRecordSystem
[ ] AccessManagementSystem
[ ] table libs via world

less important:

[ ] StoreRegistrationSystem
[ ] WorldRegistrationSystem

some things from https://github.com/latticexyz/mud/issues/444:

[ ] Optimize implementation of ReverseMappingHook (it's very naive right now, see https://github.com/latticexyz/mud/pull/487)
[ ] Revisit calling unimplemented hook methods, see https://github.com/latticexyz/mud/pull/487


devstats: Add small helper to collect commit statistics

The tool works on local git repositories and creates them if they don't
exist yet. It uses "git shortlog" and thus doesn't know about any other
form of contribution, e.g., through GitHub issues and comments. The
association with a company is derived from the email address. Currently
the script does not make stats per company for non-Kinvolk/Microsoft
commits. The repos ignition, afterburn, and ign-converter were excluded
because they are upstream projects.
How to use/Testing done
./devstats 2021-06-01 2022-05-31   # set SKIP_FETCH=1 on rerun

I have an idea to move the reporting inside the repo loop. This should allow us to add a repo list for each author.

I have an idea to move the reporting inside the repo loop. This should allow us to add a repo list for each author.

done


DRILL-6321: Lateral Join and Unnest - star column prefix

Add prefix to unnest's star column
add tests for output column verification

DRILL-6321 is marked as resolved with 1.14 Drill version. But it involves several PRs.
@chunhui-shi What is the purpose of that? Do we need reopen it or to open a new one for this PR?


Drag multiple resources onto exported array variable at once

Addresses proposal https://github.com/godotengine/godot-proposals/issues/718
This allows multiple resources to be dragged onto exported array variables at once, instead of opening the array and doing it one at a time.
Drops are allowed if all files are of the array type. Undo and redo works. Initialises array if it has not been done yet. Additional drops are appended to the current array items.

Note that most of the code was adapted from editor_properties.cpp (as below). I didn't write very much of this, just pieced it together.
https://github.com/godotengine/godot/blob/32f95c2f8782f7e524310b4c040f86cb475fdab2/editor/editor_properties.cpp#L2718-L2808

Good, but there are a lot of commits right now, so please squash them: https://docs.godotengine.org/en/stable/community/contributing/pr_workflow.html#mastering-the-pr-workflow-the-rebase

Ah, whoops! My last contribution only had 1 commit so I didn't need to do that. Now I better understand how the workflow should be for PR's, thanks.

Thanks!

@akien-mga Should this be doco'd somewhere? I'm thinking in https://docs.godotengine.org/en/3.2/getting_started/scripting/gdscript/gdscript_exports.html#exporting-arrays

Thats a good idea yeah.


Update enable-authentication-angular-spa-app.md

the property is missing for the authority.

#sign-off

@ShannonLeavitt
I can't sign-off PRs?
I see this response:
Invalid command: '#sign-off'. Only Microsoft employees can use this command.

@kengaderdus - The sign-off issue looks like it was a transient bug. Thanks for letting us know!


typeError: navigationStore.getStateForAction is not a function

in examples/react-native
"dependencies": {
"@babel/runtime": "^7.0.0",
"prop-types": "^15.6.2",
"react": "16.5.0",
"react-native": "0.57.1",
"react-native-button": "^2.3.0",
"react-native-message-bar": "^2.0.10",
"react-native-router-flux": "^4.0.3",
"react-native-screens": "^1.0.0-alpha.12",
"react-navigation": "2.16.x",
"react-navigation-deprecated-tab-navigator": "^1.3.0",
"react-navigation-stack": "^0.6.0"
},
always get this error
typeError: navigationStore.getStateForAction is not a function

Should be fixed with 4.0.4

me too.version: 4.0.4!

code:

@aksonov this seems still broken - the old API was not passing navigationStore but params.

Could you fix it by removing of navigationStore as param at all?

On 30 Sep 2018, at 17:47, Davis Z. Cabral<EMAIL_ADDRESS>wrote:
@aksonov this seems still broken - the old API was not passing navigationStore but params.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Please check 4.0.5 version

When migrating from v3 to v4.0.6, the solution for me was
import { combineReducers } from 'redux';
import NormalReducer from './normal-reducer';
import RouteReducer from './route-reducer';

const appReducer = combineReducers({
    normal: NormalReducer,
    route: RouteReducer
});
export default (state, action) => {
    return appReducer(state, action);
}

change the bottom code
export default (state, action) => {
    return appReducer;
}

I'm getting this error, for 4.0.6.  I wonder what could be the cause

any update?

4.2.0, 4.3.0, the same.

when upgrad react-native-route-flux.
<Router createReducer={reducerCreate} backAndroidHandler={this.onBackPress.bind(this)} panHandlers={this.panResponder()}>
should be remove attribute createReducer and panHandlers
like this
<Router backAndroidHandler={this.onBackPress.bind(this)}>


⚠ Bofrost.lu has degraded performance

In da07efe, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1282 ms

Resolved: Bofrost.lu performance has improved in 6d490c7 after 28 minutes.


CODESPACES: Resource Not Accessible by Integration when creating a PR

Extension version: 2021.2.1982
Codespaces: Production(e035230)
OS: Windows 10

Linking https://github.com/microsoft/vssaas-planning/issues/2176
Steps to Reproduce:

Fork a public repo
Add the parent as an upstream
Create a branch
Make a change
Stage & Commit
Push branch
Select "yes" when asked if you would like to create a PR
Select the upstream from the dropdown
Take the default title
Click "Create"
1

What Should Happen
A PR should be created for the branch against the upstream repo
What Happens instead
An error is shown in the extension that says "Resource not accessible by integration"

@joshaber @matthewisabel It looks like the codespaces token is missing permissions for this - the call it's making is just to create the PR, it shouldn't be trying to fetch anything from the parent.

That's true but since the token is scoped to only the one repository it won't allow access to the parent to create the PR. This is the opposite side of, e.g., https://github.com/github/codespaces/issues/1585.

@burkeholland Can you reopen this in the github/codepsaces repo? I'm not able to transfer it there

https://github.com/github/codespaces/issues/2243


Valid {{setting.}} url

/cc @zendesk/vegemite
If we have a setting we can't check the url for validity, because we don't know how the url is setup.
This will validate any url setting that has a `{{setting.`` in it.
url: "{{setting.url}}"
url: "http://{{setting.domain}}.my_server.com/iframe.html"
url: "/assets/{{setting.page}}.html"

Can we validate that signed and/or signedUrls is false?

what is signed_urls if have never heard of that before.

signedUrls makes all locations signed.


Make it easy for people to run the website

Not even our good friend @jlongster could figure it out! 😆
The way I see it, there are a few options:

add everything inside website/package.json to devDependencies in the root (probably easiest)
do a cd website && npm install as part of npm start (not ideal, would take extra time to boot the server)
put everything in a separate repo and deploy the site separately (not ideal, code and site would need to be versioned separately)

Maybe use Create React App? 😄

I busted the website right before release 🙄 will fix soon, but Tim is right, use create react app and then copy/paste any example into App.js.

fwiw, I did try to get it working! I did npm install from the website directory and npm start but http://localhost:9000 just showed me a blank page.

Yeah I busted it right before release and then just went ahead and released.

No worries at all! Thanks for continuing to work on this project.

OK the site works for now. I'd like to build the site statically inside of our docs directory and just serve it via GH pages. Working on that now.

related - started work on my fork on a CONTRIBUTING.md file https://github.com/threepointone/react-router/blob/v4/CONTRIBUTING.md

whoops, looks like there's a previous one? I'll edit this to be closer to the original one soon.

Fixed in 640ddf35e.


Implement something similar to RuntimeNamePropertyAttribute

Implement a mechanism that links x:Name with a property in the model (for instance, FrameworkElement.Name is linked to x:Name using an attribute).
This will have implications with the full implementation of name scopes.

Done!

It's implemented via Metadata (like the dependencies between  properties)


hold ended flows

Recently (in #1532), we started deleting flows immediately once we detect a TCP FIN/RST flags.
However, such flows can still see new packets, and thus we propose to add a "hold" duration to keep these flows in the flows table a bit longer.
Another change I made here is not report flows as "expired" if they are already reported as ended by other means (such as TCP FIN/RST).

How can you change be verified? Please provide a sequence that can prove its' working - preferably add a unit test.

@safchain It was suggested by @hunchback that we add another ticker that will run every flow.hold seconds (default 15), and will delete those flows that were detected as finished (TCP FIN/RST) on the last update iteration (they will be added to some dedicated "finishedFlows" array, instead of going over the entire flow table).
Your view on that suggestion?

@orozery the approach suggested by @hunchback seems to be good. Indeed it will limit the amount of flow.

@safchain I've implemented @hunchback 's suggestion.
Also added a unit test.

I will shortly fix the test.

@nplanel thanks for your comments! I made code changes.

@safchain so should I revert the change suggested by @hunchback to add the new ticker?

@orozery I discussed with @nplanel and I think he is right. As we push the flow in the hold list only in the update function I think that is better to remove the ticker to keep it simple. With this approach we will keep a bit longer the flow in the flow table (per update ticker) but it was potentially the same with the ticker.

run skydive-functional-tests-backend-orientdb


"Backend Vars cannot be accessed on the frontend" falsely triggered

Describe the bug
Subclassing states with backend variables raises exception since commit
https://github.com/reflex-dev/reflex/commit/f9d219407fa30558bf4eba9d39664efc46eba0b9
To Reproduce
import reflex as rx

class State(rx.State):
    _hello: str = "No hello, Reflex!"

class SubState(State):
    pass

app = rx.App()

Expected behavior
Compile
Specifics (please complete the following information):

Python Version: 3.12.2
Reflex Version: main
OS: arch
Browser (Optional): Any

problematic merge got reverted, closing this


NativeBase icon problem

Hi,
I've upgrade to the last version (0.15 to 0.17) in my project, and after upgrade it with breaking changes, i didn't succeed to make it work with nativeBase's icons..
<DialogContent>
<List>
  <ListItem icon>
    <Left>
      <Icon name="info" />
    </Left>
    <Body>
      <Text>textButton</Text>
    </Body>
  </ListItem>
</List>
</DialogContent>

Styles from nativeBase are ignored too.
In 0.15 version, it worked great :(
React Native Environment Info:
System:
OS: Windows 10
CPU: (8) x64 Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz
Memory: 3.45 GB / 15.86 GB
Binaries:
Yarn: 1.5.1 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
npm: 6.2.0 - C:\Program Files\nodejs\npm.CMD
IDEs:
Android Studio: Version  <IP_ADDRESS> AI-182.5<IP_ADDRESS>14842
React-native 0.59.1

I think the native-base theme mecanism is not progated into the class Dialog or DialogContent.


[Inquiry] A missing error assignment in QPeek.c at the Minimal folder

Hello, I found that an error assignment might be missed in QPeek.c at the Minimal test folder.
https://github.com/FreeRTOS/FreeRTOS/blob/d7667a0034841f2968f9f9f805030cc608bfbce1/FreeRTOS/Demo/Common/Minimal/QPeek.c#L367

@kaizsv,
Thanks for bringing this to our attention, I will investigate this.

I have now merged the above PR, thank you again for this report.


Question/Error: 'Can’t find variable: URL'

Hi, my User class is defined like this:
import { Type } from 'class-transformer'
import 'reflect-metadata'

export class User {
  readonly id: string
  readonly firstName: string
  readonly lastName: string

  @Type(() => URL)
  readonly photoURL: URL

  @Type(() => Date)
  readonly creationDate: Date
}

But when running (React Native/Typescript) I get the following error:
Can’t find variable: URL. Any idea how to solve this one?

You have to import your URL type to use it. Like
import { URL }  from './path/to/url';

If you want to use the built-inURL object that wont work, because class-transformer has no info about it. (And there is no way to add it as far as I know.)

Thanks! So following question (I’m new to TS): how come with Date it just works?

Date is so common it's specially handled.
https://github.com/pleerock/class-transformer/blob/7343716b96f466d54e905fabc50371bbc71c56ea/src/TransformOperationExecutor.ts#L65-L73

Got you. In your opinion, shouldn’t the built-in URL type should be supported as well?

Feel free to create a PR. :)


Multi-threaded encoding/compression?

I don't have any hands-on experience with laz-perf so this is more of a question than an issue.
I'm interested in multi-threaded LAZ-writing and I do get the impression this API might facilitate
multiple threads each encoding separate chunks, and a main thread "gathering" and outputting
those in the appropriate order.
Just curious if there are any known obstacles in the way of doing that?
In the context of garden variety Windows and Linux binaries.

These three projects should be inspirational in regards to laz-perf usage:

https://github.com/hobuinc/untwine
https://github.com/connormanning/copc.js
https://github.com/PDAL/PDAL/


blog: add "feature flags are just if statements, right?" post

This PR

Adds a blog post

Can I request a re-review on this please?

I've addressed the feedback and made it more generic but I don't agree that removing references to flagd (entirely) is appropriate.
Flagd is an open source, OpenFeature compliant, reference implementation. As such, IMO, it is entirely appropriate to direct people to use it (if they aren't already using feature flags).
I entirely agree that we should never say "flagd is the way" but "flagd is one way" is entirely appropriate, especially for brand new users.


Add new project / Multi step form

Project title

Multi step form
Project Description

It is the form where user can fill his/her information through multiple steps
Web stack

HTML / CSS / JS
Screenshot(mandatory)

Additional Context(mandatory)
Are you contributing under any open-source program ?

Yes / Codepeak & KWoC

@AaadityaG We already have a form project in our repo.
Sorry , but this issue can't be assigned.


Feature: Print version on startup

Is your feature request related to a problem? Please describe.
When running in cluster we should be able to clearly see the version of kor on startup
Describe the solution you'd like
Print the version on startup
e.g. v0.2.7

Can i work on it ? and can you point me in some direction? Thank you

Can i work on it ? and can you point me in some direction? Thank you

Sure.
Take a look at https://github.com/yonahd/kor/blob/main/pkg/utils/banner.go
And https://github.com/yonahd/kor/blob/main/cmd/kor/root.go#L48

Hey, i just wanted to ask where can i get the current version number of kor running
@yonahd

This comes from the git tag.
It would need to be embedded during build time possibly in the goreleaser

Sorry, it is taking me some time, as i have some tests coming up in my class. I will try my best to fix the issue, i need some time

No rush

@yonahd
I plan to update the .goreleaser.yml to add
builds:
  - id: OS
    ldflags:
      - -X github.com/yonahd/kor/pkg/utils.version={{.Version}}
snapshot:
  name_template: "{{ .Version }}"

that way I guess I can access the version from the git tag
Does this look good?
I wanted to ask where should I print the version and the format of it ?

@yonahd I plan to update the .goreleaser.yml to add
builds:
  - id: OS
    ldflags:
      - -X github.com/yonahd/kor/pkg/utils.version={{.Version}}
snapshot:
  name_template: "{{ .Version }}"

that way I guess I can access the version from the git tag Does this look good?
I wanted to ask where should I print the version and the format of it ?

I would need to test this.
I want to add it in the utils just like the printing of the kor banner

is this formatting okay ?

Perfect


Unsupported keras layer type Conv3DTranspose

HI! I cannot load my keras CVAE model to java since I am getting the following error: “Unsupported keras layer type Conv3DTranspose”. Why is it not possible to load Conv3DTranspose layers with DL4J? What do I need to do mitigate this problem?

Fixed here: https://github.com/eclipse/deeplearning4j/commit/75899208427bb755f8629d6a17d594162a078fa4


Create admin setting to toggle contextual help

The contextual help can be an optional feature. Create a system config setting to allow toggling its activation. An admin panel setting would be ideal, but the util/config tool is a sufficient first iteration.

Done


internal server error

[Is this likely due to my firewall? I'm on Windows 8, with ZoneAlarm firewall.]
With this PDF:
http://4thavenueyoga.com/wp-content/uploads/sites/3/2015/07/4thAveYogaSchedule_23jul2015.pdf
I got this error:
Tabula API version: 1.0.0
Filename: 4thAveYogaSchedule_23jul2015.pdf
Internal Server Error (500)
    Request Method:
    POST

    Request URL:
  http://<IP_ADDRESS>:34555/pdf/b0d3cc2e7f21a2e0faeac3ecddbb00df51741603/data

(ArgumentError) comparison of Float with nil failed

Hi @LiamDGray,
Thanks for your report. It's a known issue that'll be fixed on our next release due later this week.

Liam,
Your pdf doesn't seem to be a text pdf but an image. For example, if you try to search for the term 'class' or 'avenue' you don't find it.
With this version, OCRized, it works with me.
https://drive.google.com/file/d/0B1YVmQyWrFA8U1VpcjlGV3Ywalk/view?usp=sharing

Thanks, @hpiedcoq .


A9 complete

No passing unit test actions at https://github.com/cu-ecen-aeld/assignment-9-anuh7/actions


Hour hand doesn't move as it should... fix included

If you're finding the hour hand in the factory demo to be a bit wonky... here's the fix.
The hour hand doesn't increment in fractions, so if it's 5 to 3, it looks almost like 5 to 2 because the hour hand hasn't moved on!
The fix shows the hour hand correctly positioned between two numbers for half past the hour:

The fix adds on the angle of the minutes scaled to 1/12 of a circle to the angle of the hour hand... which is simply multiplying the minutes by 5.
lvgl_gui.cpp
void update_sensor(lv_timer_t *timer) {
  float volt = (analogRead(PIN_BAT_VOLT) * 2 * 3.3) / 4096;
  char buf[20];
  snprintf(buf, 20, "bat_volt:%.2f V", volt);
  lv_label_set_text(bat_label, buf);

  struct tm timeinfo;
  if (getLocalTime(&timeinfo)) {
    // The line that fixes the hour hand.
    lv_img_set_angle(hour_img, ((timeinfo.tm_hour) * 300 + ((timeinfo.tm_min) * 5)) % 3600);
    lv_img_set_angle(min_img, (timeinfo.tm_min) * 60);

thanks for your reminder.Please submit a pull request and I'll merge this.

https://github.com/Sarah-C/T-QT/commit/2980144b879a394d9c9399973fc774039b7895f6
Micky513673326 has been nice enough to submit the pull request. Thank you!
Issue closed.

Closed!


Netflix Sign in not working

I'm Signed in to Netflix, Scrobble, sync and automatic sync is on, but the extension doesn't work and keeps saying I'm not signed in.

Fixed in v0.4.1


Does not work on command prompt

tried the same command on both cmd and pwsh 7
works on only powershell
i also tried in various other formats like
"D:\\compres_OUT"
'D:\\compres_OUT'
'D:\compres_OUT

This is not a bug -- you've escaped your own quote and passed D:\compres_IN" -o D:\compres_OUT as a single argument.

See Why does trailing backslash followed by quote behave so strangely in cmd? for more background on this Windows behavior.


allow QuantumChannel as target in process_fidelity?

Hi all,
I've been looking at your process_fidelity method along with your Process Tomography tutorial.
I noticed that in process_fidelity, you do not allow users to pass in a noisy QuantumChannel for target like you do for the channel argument:
https://github.com/Qiskit/qiskit-terra/blob/7af29f44581a5f2d16069ac24d284be45c8deb9f/qiskit/quantum_info/operators/measures.py#L85-L91
If I try, I get the exception:
QiskitError: 'Channel cannot be converted to Operator representation'

The reason I ran into this is because in the process tomography tutorial:
https://github.com/Qiskit/qiskit-tutorials/blob/master/legacy_tutorials/ignis/6b_process_tomography.ipynb
it passes in numpy matrices instead of Choi instances which ends up giving the wrong fidelities (and you see it's passing in the noisy Choi data for the 2nd argument which is target):
print('fit fidelity (process):', np.real(process_fidelity(choi_ideal, choi_lstsq.data, require_cptp=False)))

I came up with a little test program to explore this:
from qiskit import *
from qiskit.quantum_info import Statevector, process_fidelity, purity, state_fidelity, average_gate_fidelity
from qiskit.quantum_info.operators.channel import PTM, Chi, Choi, SuperOp
from qiskit.quantum_info.operators import Operator
from qiskit.quantum_info import process_fidelity
from qiskit.ignis.verification.tomography import process_tomography_circuits, ProcessTomographyFitter

def get_choi(circuit, with_noise=False):
    if with_noise:
        # Generate process tomography circuits and run on qasm simulator
        qpt_circs = process_tomography_circuits(circuit, circuit.qregs[0])
        job = qiskit.execute(qpt_circs, Aer.get_backend('qasm_simulator'), shots=500)

        # Extract tomography data so that counts are indexed by measurement configuration
        qpt_tomo = ProcessTomographyFitter(job.result(), qpt_circs)

        choi_lstsq = qpt_tomo.fit(method='lstsq')
        return choi_lstsq

    job = qiskit.execute(circuit, Aer.get_backend("unitary_simulator"))
    ideal_unitary = job.result().get_unitary(circuit)

    TOL = 1e-14
    ideal_unitary.real[abs(ideal_unitary.real) < TOL] = 0.0
    ideal_unitary.imag[abs(ideal_unitary.imag) < TOL] = 0.0
    choi_ideal = Choi(outer(ideal_unitary.ravel(order="F")))
    return choi_ideal

def superop_pf(channel, target):
    # this alters qiskit's process_fidelity to use SuperOp for target
    channel = SuperOp(channel)
    input_dim, output_dim = channel.dim
    target = SuperOp(target)
    channel = channel.adjoint() @ target
    fid = np.trace(channel.data) / (input_dim**2)
    return float(np.real(fid))

def chow_eq8(Rexp, Rideal):
    nq = np.log(Rexp.shape[0]) / np.log(4)
    d = 2.0 * nq
    F = np.trace(Rexp.T @ Rideal)/(d*d)
    return F.real

circuit = QuantumCircuit(1)
circuit.rx(np.pi, 0)
choi_pi = get_choi(circuit)
ptm_pi = PTM(choi_pi)

rows = []
for theta in np.linspace(-1,1,9):
    circuit = QuantumCircuit(1)
    circuit.rx(theta*np.pi, 0)
    choi_theta = get_choi(circuit, with_noise=True)
    ptm_theta = PTM(choi_theta)

    eq8 = chow_eq8(np.matrix(ptm_pi.data), np.matrix(ptm_theta.data))

    #errors:
    #pf=process_fidelity(choi_pi, choi_theta, require_cp=False),

    rows.append(dict(
        theta=theta,
        tutorial=process_fidelity(choi_pi.data, choi_theta.data, require_cp=False),
        pf=process_fidelity(choi_theta, choi_pi, require_cp=False),
        superop_pf=superop_pf(choi_pi, choi_theta),
        eq8=eq8
    ))

pd.options.display.float_format = '{:,.4f}'.format
df = pd.DataFrame(rows)
df

and I get:

It seems like you could allow users to pass in a noisy QuantumChannel for target if you add code to convert it to a SuperOp instead of always an Operator.

it looks like this was fixed at some point:
https://github.com/Qiskit/qiskit-terra/blob/a2193a5c3df131775e1b2d0f2f8fe31dc035ae0c/qiskit/quantum_info/operators/measures.py#L70-L72
that is, it looks like a QuantumChannel is now an option for both channel and target.


Swift 4 Compiler Error

HI, could you push the 3.4 tag to access your swift 4.2 Migration from Cocoapods ?

Second! Much appreciated.

done


Augmentation types

Hi,
I was wondering if rotation and transformations are the only 2 types of augmentations that applied in your pipeline.
https://github.com/devnkong/FLAG/blob/7e48d9194d3a7f515335cc351a663d65e09c2e1c/deep_gcns_torch/utils/data_util.py#L76
Have you tried other augmentation transformations?

Hi Xiaoyang,
Actually these augmentations were implemented by the DeepGCN paper and never used in our work.
All of our augmentations are implemented here: https://github.com/devnkong/FLAG/blob/main/ogb/attacks.py
Let me know if you have other questions.


re-enable the chatops for "acceptance tests"

Re-enable the chat-ops that enables acceptance tests for PRs from a forked repository.

Added to epic https://github.com/pulumi/pulumi-kubernetes-operator/issues/586


create null FK option for non-compound foreign keys

A possibly common app schema will be a foreign key that is nullable. That is, the child field is allowed to be null, and if its null than the FK is not violated. The child-parent connection is enabled  only when the child field is non-null, in which case it must match at least one parent field entry.

This can be handled via an advanced row predicate, so I think no need to keep as open issue.


Programmatically resize column width

😯 Problem to Solve
There is a need to provide a method of adjusting column widths in a table from outside of directly interacting with the table.
The most notable use case is within SLE where columns need to be resized from loading a saved/stored view.
Solution details
The implemented solution will allow the previously mentioned use case to work with the nimble-table
## Related PRs
- [ ] #1002

This was completed via #1098


Test execution affecting the processor behavior in other tests

I've built some APT processor that verifies types based on whether they implement certain base types. To do that, I perform assignability checks like this one.
I have tests verifying the verification (hah!) and they succeed if I execute the tests methods individually (inside my IDE for example). If I execute the entire test class, however, those tests involving the assignability checks fail. In other words, a type that was checked to implement a particular interface apparently does not implement that anymore, at least in the type-space that's considered during the test run.
I am not sure whether this is a problem in the contextual setup that Cute creates, or whether there's something wrong with my Processor implementation.
To verify:

Clone https://github.com/xmolecules/jmolecules-integrations
Switch into the jmolecules-apt module
Enable the two disabled test cases in JMoleculesProcessorUnitTests.
Run mvn test -Dtest=JMoleculesProcessorUnitTests#detectsInvalidAggregateRootReferenceInImplementingAggregate -> succeeds
Run mvn test -Dtest=JMoleculesProcessorUnitTests#detectsInvalidAggregateRootReferenceInImplementingEntity -> succeeds
Run mvn test - Dtest=JMoleculesProcessorUnitTests -> fails

Hi, i will take a look at it.
But, Cute does provide a fresh Compile File Manager and Compiler that is initialized inside the executeTest method.
So it's unlikely that Cute is mixing up stuff between tests.

One more question, which IDE are you using?

I have checked your Tests,
your processor is a wildcard processor handling all annotations, but your source files of those two tests just have a sinfle Override annotation present on a method.
Your processor code
rootElements.stream()
				.filter(TypeElement.class::isInstance)
				.map(TypeElement.class::cast)
				.flatMap(it -> {

					TypeElementWrapper wrap = TypeElementWrapper.wrap(it);
					return verifications.stream().map(verification -> verification.verify(wrap));

				}).reduce(true, (l, r) -> l && r);

		return true;
	}

is basically ignoring that ExecutableElement so compilation succeeds.

I am using Eclipse.
But wait, which behavior is correct? The succceding or the failing one? Because depending on the mode of execution, the test result is different. Why is that? The @Override annotation is present in both cases.

Usually you need some kind of annotation you can rely on to make the processor work.
This could be for example an annotation placed on the package-info.java.
This would allow you to test all TypeElement residing in those package.
But be warned: it's not possible to access sub-packages via the Processor api. So you would have to add that annotations in all packages you  want to check.

The succeding is correct. The processor only gets the methods ExecutableElement via the Overwrite annotation. So the element is filtered out and the processor does nothing. So the compilation succeeds.

Well, in both standalone and multi-execution, the processor is invoked with a TypeElement of MyAggregateRoot. In the standalone execution, that element apparently implements Identifiable (via AggregateRoot and Entity), in the multi-execution it does not. I don't understand why that is. I also do not see the processor not being invoked at all, as you suggest. I even see it working when used in an example. It's solely the combined test execution that it apparently fails in.

The good thing is that those tests allow you to debug your processor. Just set some break points and you can check whats happening internally.

The processor is invoked. But not for a TypeElement, just for the annotated methods ExecutableElement - so it's filtered out by the stream filter expression.

One more thing, you are using getRootElements  which seems to be odd. I have never used that myself but according to Javadoc it can alead to altering behavior between compilations.
Usually you would  use getElementsAnnotatedWith() and use/loop over the annotations passed into the proccessor method.

Please consider to replace that wildcard mechanism by a kind of anchor annotation like i have mentioned before. Then you have a fixed entry point for checks and don't rely that there are any kind of other annotations present, which aren't guaranteed to be there. Package annotations could be good for that as i have mentioned before.

I unfortunately can't do that as I have to support arrangements in which no annotations are at play at all.
That said, I was able to resolve the problem by moving away from static TypeMirror instances for the reference types in JMoleculesDddVerification. I assume that the reason for the issue is that the instances being static keeps the originally initialized ones around between processor runs that each initiate a new Types world. It thus would result in a mismatch for the second processor invocation. The first one sees the same Identifiable that the MyAggregateRoot references in its type hierarchy, as it stems from the same type pool the processor works with. In the second round, the one referred to from the type hierarchy is a different one than the instance initialized in the first run.
I'll close this. Thanks for your support here! 🙇

Ok, great. Forget about the things I mentioned about  getRootElements. It should work.  It will always provide a TypeElement or PackageElement if a package-info.java file exists. So you have to add a testcase for existing package info files.


[Suggestion] Search only after pressing Enter

I don't know if this is still being supported but hopefully yes...
Searching with each key press can make the game lag a bit (mainly if the game just started or became inactive for a time), a cool config option would be where it searches the item only after you press Enter, meaning if HEI's prev search was "dirt" it will stay in there (if i clear the search bar and press enter, it will show all item as normal) then i type "chest", it will only show the results if i press Enter otherwise it just stay in the HEI's prev search aka the "dirt" one.

Are you using the ultra low memory mode? (check in configs)

Why?

Normally the lag is noticeable when you use ultra low memory mode.

Nope


Remove depency on deprecated package 'crypto'.

As documented on npm's page for the crypto package, this package is deprecated as there is now a Node-native library. The native library happens to be API compatible (at least for the purposes of gatsby-source-prismic anyway) so the dependency simply had to be removed from package.json.

Awesome, thanks for the PR! 👍


Changed CLI list to return all records

Formerly, the CLI list command was broken and inconsistent. This change
ensures the all record api is used for lists.
closes #259

denominator-pull-requests #437 SUCCESS
This pull request looks good

NetflixOSS » denominator » denominator-pull-requests #62 SUCCESS
This pull request looks good


Fix flaky bundle tests.

They are flaky because Android SDK does not guarantee all progress callbacks are complete before the task resolves.
Whether or not it should guarantee this is another topic.

They are flaky because Android SDK does not guarantee all progress callbacks are complete before the task resolves.
Whether or not it should guarantee this is another topic.

Is this the issue that will be fixed in the next Android release? If so, we should revert this PR after that fix goes in (and we update to that dependency).

They are flaky because Android SDK does not guarantee all progress callbacks are complete before the task resolves.
Whether or not it should guarantee this is another topic.

Is this the issue that will be fixed in the next Android release? If so, we should revert this PR after that fix goes in (and we update to that dependency).

We have not decided whether or not we will change Android yet..there are arguments against it. Will revert this PR if that is what we end up doing.


Unable to find cookie file

Fill in this template completely. Issues not following this template will be closed and ignored.
Check the boxes below by filling [ ] with an x so it looks like [x].
Use the Preview button to ensure the template is filled in correctly.

[x] I am using the latest version of PyInstaLive: 3.2.0.
[x] I have installed either Python 2.7.x or 3.5+: 2.7.17
[x] I have read the README.
[x] I have read the FAQ.
[x] I have checked all existing issues, none of which could solve my issue.

I am using:

[ ] Windows 10
[ ] Windows 8.1
[ ] Windows 8
[ ] Windows 7
[ ] Linux (distribution: )
[x] macOS (version: 10.15.3)
[ ] Other (device & OS name: )

To report a bug, fill in the information below.
PyInstaLive information
Run pyinstalive --info and paste its output below. Don't forget to omit your username and password.
Example:
$ pyinstalive --info
---------------------------------------------------------------------------
[I] PYINSTALIVE (SCRIPT V3.2.0 - PYTHON V3.6.3) - 01-02-2019 07:00:17 PM
---------------------------------------------------------------------------
[I] To see all the available arguments, use the -h argument.

[I] PyInstaLive version:        3.2.0
[I] Python version:             2.7.17
[I] FFmpeg framework:           Available
[I] Cookie files:               None found
[I] CLI supports color:         Yes
[I] Command to run at start:    None
[I] Command to run at finish:   None
[I] Config file contents:

    [pyinstalive]
    username = xxxxx
    password = xxxxx
    download_path = /Users/Jeffrey/Downloads
    download_lives = True
    download_replays = True
    download_comments = true
    show_cookie_expiry = True
    log_to_file = True
    ffmpeg_path =
    run_at_start =
    run_at_finish =
    use_locks = True
    clear_temp_files = False
    do_heartbeat = True
    proxy =
    verbose = False
    skip_merge = False

[I] End of PyInstaLive information screen.
---------------------------------------------------------------------------

Command used
pyinstalive -d “<xxxxx>”
Behavior
---------------------------------------------------------------------------
[I] PYINSTALIVE (SCRIPT V3.2.0 - PYTHON V2.7.17) - 03-16-2020 08:08:06 PM
---------------------------------------------------------------------------
[W] Unable to find cookie file: xxxxx.json
[I] Creating a new one.
---------------------------------------------------------------------------
[E] Could not login:
---------------------------------------------------------------------------

Question
I have tried to reinstall pyinstalive but it still doesn’t work.

It can't find a login cookie because an error occurs while trying to login, thus no cookie file storing your login is created. The problem though is that Instagram doesn't seem to be returning a proper error response in your case and as such no error information is displayed.
Normal error would look like this.
---------------------------------------------------------------------------
[I] PYINSTALIVE (SCRIPT V3.2.0 - PYTHON V2.7.17) - 03-16-2020 09:23:13 PM
---------------------------------------------------------------------------
[W] Unable to find cookie file: username.json
[I] Creating a new one.
---------------------------------------------------------------------------
[E] Could not login: {"status": "fail", "error_title": "Incorrect password for username", "error_type": "bad_password", "buttons": [{"action": "dismiss", "title": "Try Again"}], "message": "The password you entered is incorrect. Please try again.", "invalid_credentials": true}
---------------------------------------------------------------------------

You could try using Windows (Boot Camp?) and a different internet network to ensure those aren't the cause of the problem.
But given the lack of error information there's nothing I can do to help solve the problem.

@crazzzyjeff I did some digging through older issues, another user had the same problem on MacOS which apparently had to do with SSL certificate issues. I suggest checking out this issue for more information and how solve the issue.
https://github.com/notcammy/PyInstaLive/issues/68#issuecomment-516445500

Problem is solved after changing the python in my terminal to 3.8.2!
It was working fine at first, but I think I messed up some files and stuff.
This happened when I reinstalled it, I don't know what to do.
---------------------------------------------------------------------------
[I] PYINSTALIVE (SCRIPT V3.2.0 - PYTHON V3.8.2) - 03-17-2020 05:20:30 AM
---------------------------------------------------------------------------
[I] Successfully logged into account: xxxxx
[I] Cookie file expiry date: 2020-06-15 at 06:01:23 AM
---------------------------------------------------------------------------
[E] Could not get user info for '“xxxxx”': 'ascii' codec can't encode character '\u201c' in position 18: ordinal not in range(128)
---------------------------------------------------------------------------

The problem was solved when I install python version 3.8.2 in Terminal and reinstall pyinstalive.
Thanks a lot!!!

``Problem is solved after changing the python in my terminal to 3.8.2!
It was working fine at first, but I think I messed up some files and stuff.
This happened when I reinstalled it, I don't know what to do.
---------------------------------------------------------------------------
[I] PYINSTALIVE (SCRIPT V3.2.0 - PYTHON V3.8.2) - 03-17-2020 05:20:30 AM
---------------------------------------------------------------------------
[I] Successfully logged into account: xxxxx
[I] Cookie file expiry date: 2020-06-15 at 06:01:23 AM
---------------------------------------------------------------------------
[E] Could not get user info for '“xxxxx”': 'ascii' codec can't encode character '\u201c' in position 18: ordinal not in range(128)
---------------------------------------------------------------------------

@crazzzyjeff are you still having problems? I received the above in my email but can't find the comment here on GitHub. In any case I suspect it's your input that's incorrect, '“xxxxx”' indicates your username input is being handled including the double quotes, and \u201c is a left double quote. Make sure to input commands as per the examples given here https://github.com/notcammy/PyInstaLive/blob/master/README.md#example
Note that “ is not the same as ", Python is pretty finnicky when it comes to character encoding

Yes, it was because of the quotation marks. I figured it out afterwards so I deleted the comment.
But I ran in to another problem when I was using the program today. It seems that the program have only recorded 300 comments and stopped recording comments afterwards during the livestream.

It can't save at a faster rate, and not all comments that are sent actually show up in someone's livestream especially with lives that have a lot of people watching/commenting. It's beyond my control. The ClientError can be ignored by the way

OK :) Thanks a lot for your help and effort!!!

OK :) Thanks a lot for your help and effort!!!

No problem.
This script is by no means perfect so don't hesitate to create a new issue should you have any more questions or have a bug to report.


[BYON]: Impossible to update custom notebook image source

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Once an image has been imported, there is no way to update its source.
When a custom image is updated in its source repo (eg quay.io), like with a new tag, there is no way to update its definition in the UI to use this new tag. Using a "latest" tag and update it in the source repo is not an option as:

You may not want to use the tag "latest", which has some impact on not always exactly knowing what you're running.
Even if you move the tag "latest" in the source repo, it is not picked up automatically by the ImageStream. Once the sha signature of the source image has been imported in the ImageStream, it is never updated until you force it.

Expected Behavior
Ability to modify the image source for a custom image in the UI.
Steps To Reproduce

Import a custom image
Want to modify the image source, like pointing to a new tag
No way to do that in the UI

Workaround (if any)
Modify the image source (tag) directly in the image stream.
Btw, I understand the need for unicity in the byon Image Streams, but could we simply append a uuid to the name of the image when naming the Image Stream instead of totally obfuscating the name? When someone will have tens of custom images, or even two, and want to check the ImageStream, good luck finding which is which...
OpenShift Version
No response
Openshift Version
No response
What browsers are you seeing the problem on?
No response
Open Data Hub Version
No response
Relevant log output
No response

Moved to feature/byon project

@andrewballantyne Could you move this to jira please?

Moved to Jira: https://issues.redhat.com/browse/RHOAISTRAT-43


Fix bounding box mess

We should receive a min_lon, min_lat, max_lon, max_lat. Also take a look at the code since there's a mess when processing the bbox coordinates.

Fixed in 0.0.5


Doesn't work in node > 0.10

Fails to build/install, on OSX at least (the actual error message is a rather abstruse Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.). I believe this is due to the dependency on https://github.com/jeremyfa/node-exec-sync (which itself depends on ffi). exec-sync is now a built-in feature for node>=0.12 (https://nodejs.org/api/child_process.html#child_process_child_process_execsync_command_options).

I probably should have done it asynchronously, but it was my first Node project so I didn't know better. It was just a Hello World type project so I'm not maintaining it any more. You might have more luck with https://www.npmjs.com/package/generator-license. Thanks for your interest though.

Makes sense, and agreed there are alternatives - just wanted to let you know. If you don't intend to fix (which is fine) then I'd suggest maybe updating the README to note that this isn't actively maintained. Thanks!

Done, thanks.


why I Change the file path use the ardrive App, but when I use CLI commend get json data, the path is not update ?

why I update the file path use the ardrive App, but when I use CLI commend get json data, the path is not update ?

`import datetime
import os
import json
import time
strs = []
pp = "%s,%s\n"
def list_file(id: str, keyWord: str) -> None:
command = "ardrive list-folder --parent-folder-id %s"
d = os.popen(command % id)
data = json.load(d)
for f in data:
    if f['entityType'] == "file":
        if (keyWord == "") or (keyWord in f['path']):
            strs.append(pp % (f['path'], f['dataTxId']))
            print(pp % (f['path'], f['dataTxId']))
        continue
    elif f['entityType'] == "folder":
        list_file(f['folderId'], keyWord)

list_file("My folderId",
keyWord="")  # Root
`

strs = []
pp = "%s,%s\n"
def list_file(id: str, keyWord: str) -> None:
command = "ardrive list-folder --parent-folder-id %s"
d = os.popen(command % id)
data = json.load(d)
for f in data:
if f['entityType'] == "file":
if (keyWord == "") or (keyWord in f['path']):
strs.append(pp % (f['path'], f['dataTxId']))
print(pp % (f['path'], f['dataTxId']))
continue
elif f['entityType'] == "folder":
list_file(f['folderId'], keyWord)

I am sorry but I do not follow what you are trying to accomplish.
May you kindly explain with detail what are you trying to do and why do you think it is a CLI issue?
Is there any reason for using the CLI instead of https://github.com/ardriveapp/ardrive-core-js ?
If your issue is regarding how-to use the app, I suggest to use our official support channel: https://ardrive.zendesk.com/hc/en-us


Port 80 already in use

failed to bind port <IP_ADDRESS>:80/tcp: Error starting userland proxy: listen tcp4 <IP_ADDRESS>:80: bind: address already in use.
Is it possible to fix this?

I think it has to do with nginx also using port 80. But I'm using nextcloud on this server and unfortunately I need it to work. I will greatly appreciate if you could provide me a solution. Thank you.

have you found any solution to this? I need to be able to access a web server over https on my VPS, which I also set up as a proxy server using this script.
The proxy service is running on ports 80 and 443 and I can't set up nginx to run on those ports because they're already taken...
And the letsencrypt documentation says port 80 needs to be available

Letsencrypt must use port 80 to validate the hostname.
You need to update the configuration of other services to use ports other than 80


Layout Recommender

Add the ability for the system to auto recommend a layout based on density/nodes/etc.

I did a first pass on this but we need to improve it later on.


getSize returns wrong value

I tested getSize() with the MB85RC64TAPNF chip which is 64k in size and getSize returns "8".
I can confirm that the chip is actually 64k by writing and reading from address 0xFFFF.

Thanks for reporting this issue.
Will look into it asap,
Do the other functions work as expected?

Found the datasheet - https://nl.mouser.com/datasheet/2/1113/MB85RC64TA_DS501_00044_3v2_E-2329194.pdf

According to that datasheet getSize() should return 3 and not 8
So it might be a slightly other (older?) type without the density bits..
What does the chip return on getManufacturerID() and getProductID() calls?
(top of the output of testFRAM.ino sketch)

OK, it took some time to wake up.😁
getSize() does return the number of kiloBYTE, and the number 64 in the type of the FRAM is the number of kiloBIT.
64 kilobit == 8 kilobyte.
The reason you can write to address 0xFFFF is that the device just truncates the address bits.
The library does not protect address range
To see this you need to write 0x00 to all addresses.
Then write 0xFF to 0xFFFF
Then read all address and see where the 0xFF pops up.
If my hypothesis is correct 0xFF should pop up at 0x1FFF - 0x3FFF - 0x5FFF - 0x7FFF - 0x9FFF - 0xBFFF - 0xDFFF and 0xFFFF

I will already improve the documentation a bit to make the getSize() return value more explicit.

Thank you very much for a fast reply!  As you pointed out, I incorrectly assumed that the getSize function should return 64, but indeed 8 makes sense had I read the docs more carefully to realize it returns BYTES not BITS.  I apologize for any confusion and appreciate you taking the time to clarify this.
The chip returns manufacturer ID 0x0A, Product ID 0x0358
Thank you again for a fast and helpful response!

you're welcome. took some time for me too to see the light.
I will update the library (mostly documentation) today, so it is more explicit.

The chip returns manufacturer ID 0x0A, Product ID 0x0358

looks perfect!
If there are no further issues you may close this one

FYI
Pull Request is opened (develop branch) with updated documentation and function getSizeBytes().


Error minimizing application

When minimizing the application, and then returning to it, I see that the screenshot is able to be captured again.

Hello @orafaelpedrosa, thank you for your feedback.
Please confirm how to reproduce this issue.
Platform:
Package version:
Error observed:
Device OS version:
And any other information you think can help.

Closing! Please reopen if you face that issue again @orafaelpedrosa


Add bitbucket-power extension

Description

Screencast

Checklist

[ ] I read the extension guidelines
[ ] I read the documentation about publishing
[ ] I ran npm run build and tested this distribution build in Raycast
[ ] I checked that files in the assets folder are used by the extension itself
[ ] I checked that assets used by the README are placed outside of the metadata folder

Hi 👋
We already have extensions for bitbucket in the store, maybe it could fit into one of them?

Hi 👋
We already have extensions for bitbucket in the store, maybe it could fit into one of them?

Yes, I use it but it didn't meet my needs.  Does not have pipelines functionality for example.  Should I attempt a PR on that package instead?

Do you think it will fit in @Lp-Francois

Do you think it will fit in @Lp-Francois

@pernielsentikaer yes it makes more sense, like the Github extension groups PRs, pipelines, etc in the same extension :)

Totally agree, could you look into moving it into that extension @skuio


Project restructure! (issue #3)

Restructured the project according to the resources you provided, unfortunately I got carried away and put everything into one massive commit, but below is a relatively accurate summary of everything that was changed (copied verbatim from the commit message with slight edits), please have a look over the code aswell, I'd love any feedback you may have, thanks!
Notes:

semver wasn't increased cuz im not sure if it should be yet (because the api interface and whatnot is still the same)
api compatability with old code was tested, tests are also all successful
formatted with the provided rustfmt config
as outlined in the edits, some fields were made public because I couldn't figure out how to make them work without doing so, they aren't visible in the docs but i'm still not sure.

Edits:

it will now compile on stable rust
in order to make it compile on stable rust a new function that mimicked the assume_init function had to be made: "assume_init_box" (in common.rs)
all the code was separated into 3 modules, common, encoder and decoder (visibilities and api were kept as is, old source code should still be compatible with the restructure)
function "new" added for OggPacket to make stuff cleaner
VorbisInfo's (in common/vorbis_data.rs:20-21) field was made public
VorbisComments's (in common/vorbis_data.rs:196-197) field was made public
instead of collect_into, extend was used (decoder/decoder_data.rs:39-41)
VorbisEncodingState's (encoder/encoder_data.rs:13-18) fields were made public
all occurences of .assume_init() being used on Box objects were replaced with calls to the new "assume_init_box" function

Splendid work! I genuinely appreciate the effort you put into making such a well-written PR! :heart:
I'm a bit busy at work now, but I'll arrange the proper time and attention to review your PR as soon as possible.


Adds --disable_quotes options

Fixes #17
Added options to Enable/Disable quotes

Hi @janish1999 . Apparaently some xml files have crept up in this PR. Please remove them.

I would also appreciate if you made the changes in separate commits, each commit describing smaller sub-changes, as is the recommended way. Thanks!

I would suggest creating a new PR once #18 is merged. Otherwise, we may face unnecessary merge conflicts. Sorry for the inconvenience.
Also, please don't edit the Pipfile manually in anyway.

Yeah sure.
I'll create a new pull request after #18 is merged

Thanks a lot! :+1:


build minio server elsewhere

it's currently built inside a docker build (https://github.com/deis/minio/blob/master/Dockerfile) and then docker cp-ed to the right place. it makes more sense to do the build inside a docker run

shouldn't be too hard to implement if you mount the local dir :)
docker run -v $PWD:/go/src/github.com/deis/minio go:1.5.1 sh -c 'cd /go/src/github.com/deis/minio && make build && mv /go/bin/minio /go/src/github.com/deis/minio/rootfs/bin'

...or something like that

yep :+1:


feat: Dom Helpers for more concise block code

Context
This PR adds a small JS file to help with DOM elements generation.
We've been using this in our project and it speeds up a lot the development, code reviews and testing, so I thought it might be useful for new projects as well.
https://github.com/hlxsites/moleculardevices/pull/161
Why?
The following Dom Helper functions allow for:

a more concise code
composable structure
dom like syntax structure to easily visualise the resulting HTML when looking at the code
faster development and code review of block code
while maintaining  100 LHS for performance
100% vanilla JS / no compilation required
with minimal overhead - a few ifs and function calls

Main Usecases

DOM rendering of dynamic data from spreadsheets and indexes
Adding new DOM elements during block decoration to what Franklin provides, based on the word document (e.g. adding buttons for a carousel, where the elements are coming from the word document)

Examples
It allows rewrite from:
function renderItem(item) {
  const newsItem = document.createElement('div');
  newsItem.classList.add('blog-carousel-item');
  const newsItemLink = document.createElement('a');
  newsItemLink.href = item.path;
  newsItem.append(newsItemLink);
  const newsThumb = document.createElement('div');
  newsThumb.classList.add('blog-carousel-thumb');
  newsThumb.append(createOptimizedPicture(item.image, item.title, 'lazy', [{ width: '800' }]));
  newsItemLink.appendChild(newsThumb);
  const newsCaption = document.createElement('div');
  newsCaption.classList.add('blog-carousel-caption');
  newsCaption.innerHTML = `

${item.title}

${item.description}

      Read More

  `;
  newsItemLink.appendChild(newsCaption);
  return newsItem;
}

To:
function renderItem(item) {
  return (
    div({ class: 'blog-carousel-item' },
      a({ href: item.path },
        div({ class: 'blog-carousel-thumb' },
          createOptimizedPicture(item.image, item.title, 'lazy', [{ width: '800' }]),
        ),
        div({ class: 'blog-carousel-caption' },
          h3(item.title),
          p({ class: 'blog-description' }, item.description),
          p({ class: 'button-container' },
            a({ href: item.path, 'aria-label': 'Read More', class: 'button primary' }, 'Read More'),
          ),
        ),
      ),
    )
  );
}

More examples from our project code:

https://github.com/hlxsites/moleculardevices/blob/main/templates/blog/blog.js#L57-L74
https://github.com/hlxsites/moleculardevices/blob/main/blocks/related-assay-data/related-assay-data.js#L33-L41
https://github.com/hlxsites/moleculardevices/blob/main/blocks/social-share/social-share.js#L94-L106
https://github.com/hlxsites/moleculardevices/blob/main/blocks/citations/citations.js#L83-L110
https://github.com/hlxsites/moleculardevices/blob/main/blocks/social-share/social-share.js#L26-L34

Other projects
The reason why I thought it would be a good addition to the boiler plate is that I found similar/partial implementations, some repeating between projects, some independently written.

Adobe.com: https://blog.adobe.com/blocks/block-helpers.js
Volvo Trucks: https://github.com/hlxsites/vg-volvotrucks-us/blob/main/scripts/scripts.js#L40-L53
Caesars: https://github.com/hlxsites/caesars/blob/main/caesars-palace/scripts/scripts.js#L244-L263
Airlessco: https://github.com/hlxsites/airlessco/blob/main/scripts/scripts.js#L119-L134
WKND: https://github.com/hlxsites/wknd/blob/main/scripts/scripts.js#L50-L69

Credits
This is of course inspired by the JSX (React) syntax, articles and code pieces explaining its internals.

👍 - @solaris007 and I are working on something similar (arguably not as sophisticated as this proposal) in [0]. I think it would make some sense to have a standard way to create nested Dom structures in a readable way.
[0] https://github.com/hlxsites/mammotome/pull/67/files#diff-a5ee87ff9063f921ba01933977e6164e5729551986030f0ed7eb747bcffb740fR228

@andreituicu I'm curious what limitations in template literals made you come up with this approach?
One benefit I can see is it's easier to attach event listeners and maybe less need for innerHTML, but besides that I'm not sure I see what else you get.
I would 100% agree that it's much easier to read and maintain than imperative DOM code. But on the flip side I think the same can be done with string template literals (and still have it be concise. composable and easy to visualize).
function renderItem(item) {
  return /* html */`

          ${createOptimizedPicture(item.image, item.title, 'lazy', [{ width: '800' }])}

${item.title}

${item.description}

            Read More

  `;
}

To me, this looks like a templating language and templating languages should be a user choice, not a framework default.
The most vanilla thing that I could imagine is based on the HTML template element https://developer.mozilla.org/en-US/docs/Web/HTML/Element/template – but it's not very widely adopted yet.

@dylandepass

Correct, my bad, I through the example together quickly without thinking too much about it. I solved this in the past by using outerHTML as you mentioned.

Sure, I should've maybe been explicit: I only pointed it to show that, although possible with string literals, the developer experience when writing it could be a little bit more seamless from my point of view.

One note, you can fix the syntax highlighting and code completion issue in vscode with this extension.

Noted! Was not aware! Thank you!

I just wanted to better understand the pain points here and see if there was something else I was missing.

Happy to discuss and thank you very much for the feedback!

While I see the point on not putting this into boilerplate, as @trieloff pointed out it should be a user's choice.
There seems to be a need for this. I'm aware two other projects built helpers similar like this one. We might add this to the block party instead of boilerplate.

@mhaack Thank you for the feedback!

We might add this to the block party instead of boilerplate.

you are referring to https://github.com/adobe/helix-block-collection , correct?
I can open a PR there if that's a more appropriate place for this and contribute a doc entry to https://www.hlx.live/developer/block-collection#block-collection to make it easier discoverable.
@trieloff does that sounds like a good approach to you?

I'm aware two other projects built helpers similar like this one.

just checking: are they captured in the Other Projects section in this PR? if not could you please share them so I can add those as well?

i agree that the best place would be "block party" mentioned by @mhaack to get started.
the "block party" is a new concept we came up with for community contributed blocks and snippets. i don't think the code should be in any particular repos but it lives where it is... i don't know the latest on process for including things in the block party. maybe it makes sense to track those as issues in https://github.com/adobe/helix-website.
(cc: @amol-anand)

https://github.com/hlxsites/block-party seems like the best place for this.

should we move this issue to block-party and close this PR?

@davidnuescheler @trieloff Thank you! Created https://github.com/hlxsites/block-party/pull/1 .
Do you mind if I also keep this PR open as well for discoverability reasons? At the moment, not that many people know about the Block Party repository, while I think many check the boilerplate pull requests for possible improvemennts.

To solve the discoverability issue, you could submit a PR that updates the README with links to block collection and block party.

@trieloff just to clarify I understood correctly: You mean a PR to the README of this repository (the boilerplate)? Sure, will do!


Change Platform importing to not be dynamic

I added an class variable to the API (_platform) and now _set_platform_dir_class works by building a dict from all subclasses of PlatformDirsABC where the key is the classvar _platform and the value is the actual implementation class.
The change is that new implementation need to be imported into the __init__.py, but nothing else needs to be changed.
fixes #55
*Did not run the tests as I don't have experience with tox.

I believe this was done this way for performance reasons

Correct.
@Giddius Can you try keeping the conditionals?

I'm personally -1 on this change and prefer the current system. Not sure what actual problem this is trying to solve besides reducing some perceived complexity 🤔

Only for info, the current system can not be compiled with nuitka because of the dynamic imports. see #55

Nukita has special options to manage this

Fixed by https://github.com/platformdirs/platformdirs/pull/68


Remove validation errors for form fields

Currently the removeValidator() method provided in the validationService removes the validation message from the form validation summary but the error still continues to be shown next to the validated field.
On removing validator the error attached to field should also be removed.

It might be nice to have some of your help sometimes, instead of just posting issues... that is what a community is for, helping each other.

Looking at my code, it already does the effect that you are asking for. You can see and test this on my demo sample in the Service page. If you really still have issues, then make a plunker describing your problem.

I was wrong. I had some code changes at my end because of which i was getting this issue. Once i tried to recreate the issue from scratch i realized i had made a mistake.
Sorry. You can please go ahead and close this issue.


Skip ConfigMap update when there are no changes

During E2E tests there are a lot of no-op reconciliation
since chaosduck kills the leader pod and every time
a new replica starts we trigger a global resync that
most of the time doesn't actually change the ConfigMap.
For example, the ConfigMap diff looks like this:
  	"""
  	{
- 	  "generation": "1",
+ 	  "generation": "2",
  	  "resources": [
  	    {
  	... // 15 identical lines
  	"""
  )

This leads to an unnecessary reconciliation loop on the data plane
and increase update conflicts of the ConfigMap.
Proposed Changes

Skip ConfigMap update when there are no changes

Release Note

:broom: Update or clean up current behavior
Skip ConfigMap update when there are no changes

/kind cleanup

/assign @slinkydeveloper

/lgtm
/approve
Sounds like a nice improvement!


Fix regression for non-unique crash IDs

Fix regression introduced in 3409913f78837a955c081df6cc237673db5c7bab, which made crash report IDs non-unique, as the 32 MSB's were zeroed out ((int64_t)foo & ~0xffffffff is always 0), leading to only 9 bits of entropy

Good catch!


how to set close connection res Code and description

webSocket.close(RES_CODE,description);
This is possible in okHtttp, is it possible with this library...?

with a response code and description you mean the option to send a data body along with the close action? I'm not sure about the meaning of the okHttp rescode and description because that's a different specification but what I can do is to add the option to send data inside the close frame, and your app will be responsible of giving a meaning for that data


Release new version to support Django 4.2

Noticed support for Django 4.2 has been committed, but no release has been released to PyPi.

@uri-rodberg @claudep is there anything blocking a new release with the recent improvments?

@JanMalte I'm waiting for PR https://github.com/jazzband/sorl-thumbnail/pull/740 to be approved. But Regarding Django 4.2, it's not needed. I think if we release a new version now it should support Django 4.2.

Hi,
You can write in your requirements file the following line:
-e git+https://github.com/uri-rodberg/sorl-thumbnail.git@9339cc250328f7c5d00212877d920a69bf7e740a#egg=sorl_thumbnail

Or, manually run pip install -e git+https://github.com/uri-rodberg/sorl-thumbnail.git@9339cc250328f7c5d00212877d920a69bf7e740a#egg=sorl_thumbnail. This will install sorl-thumbnail from my fork with the updated commits (except one commit) and including my PR which is not merged yet. If you install this version, you can run it with Django 4.2.

@jezdez could you please step in and help to get a new release packed?

I wonder why noone proposed a patch to update the Changelog :roll_eyes:

I wonder why noone proposed a patch to update the Changelog 🙄

Added support for Django 4.2.
Added support for Python 3.11.
Dropped support for Python 3.7.
And also, if you accept my PR (https://github.com/jazzband/sorl-thumbnail/pull/740), write something about it.

#746 for the CHANGES.rst updates.

@claudep @uri-rodberg if you are ok with the current master branch, I could try to make a new release.

Thanks @JanMalte ! I tried to trigger a release by pushing a tag, but as almost each package on which I contribute has a different release workflow, I don't remember if it was the right thing to do :disappointed:

@claudep @uri-rodberg if you are ok with the current master branch, I could try to make a new release.

Hi @JanMalte. Actually I was hoping that my PR will be merged before the new release, but I understand we need to talk about it more, so I guess it's OK to release the master branch as it is now. I don't know how to release sorl-thumbnail, but good luck!

Oh, the release is now available on https://jazzband.co/projects/sorl-thumbnail, but I cannot push to PyPI as I'm not a project lead. Ping @jezdez or @camilonova

@claudep Done.


allow native modules to be created and require()'d

I need to extend phantomjs w/ some C++ code but I don't want to modify phantomjs directly and have to rebuild it constantly (the modifications will be running on travis). So I created this pull request which lets you build and require native modules (ie, shared libraries).
Notes:

Native modules must end with the .phantom extension and must expose an extern "C" createNativeExtension function that returns a QObject * to expose to JavaScript.
I don't think the tests will pass on windows. It doesn't seem to be possible to specify a custom prefix when building a shared library w/ qmake. Or at least I can't find a way.

Let me know what you think.

I can see some value in this: I have been toying with the idea of creating a zeromq (http://zeromq.org/) binding for phantom (as it would allow a nice range of client/server patterns to be implemented).
I can appreciate that it wouldn't necessarily be something that would make it to phantom's core, so wrapping it up in a requireable native module could work well.

Not relevant anymore.


Path to my spec files insn't complete

When i have in my src directory some directories they named like as the followed one's

src/core
src/collections
src/conditionals

Is the name inside of my Vitest explorer like

re/result/result.spec.ts
llections/dictionary/dictionary.spec.ts
nditionals/match/match.spec.ts

My fist asumetion is that every directory is starting with the characters co.
If ranemed all directories like

src/sore
src/sollections
src/sonditionals

or

src/sire
src/sillections
src/sinditionals

This has the same effect, that the path isn't complete, inside of the Vitest explorer
To Reproduce

Rename all directories inside of your src folder with the same two characters. co, si, fo
Click on Vitest to see the path to your spec files
2.1. Vitest view settings Sort by Location and View as Tree

Expected behavior
I can see the full path to my spec files
Screenshots

Environment
info.txt
{
  "System": {
    "OS": "macOS 13.5.1"
  },
  "Binaries": {
    "Node": {
      "version": "16.17.1",
      "path": "~/.nvm/versions/node/v16.17.1/bin/node"
    },
    "Yarn": "Not Found",
    "npm": {
      "version": "8.15.0",
      "path": "~/.nvm/versions/node/v16.17.1/bin/npm"
    }
  },
  "IDEs": {
    "VSCode": {
      "version": "1.81.1",
      "path": "/usr/local/bin/code"
    }
  },
  "npmPackages": {
    "vite": {
      "installed": "2.9.16",
      "wanted": "^2.9.9"
    },
    "vitest": {
      "installed": "0.12.10",
      "wanted": "^0.12.6"
    }
  }
}

vite.config.ts
{
  "System": {
    "OS": "macOS 13.5.1"
  },
  "Binaries": {
    "Node": {
      "version": "16.17.1",
      "path": "~/.nvm/versions/node/v16.17.1/bin/node"
    },
    "Yarn": "Not Found",
    "npm": {
      "version": "8.15.0",
      "path": "~/.nvm/versions/node/v16.17.1/bin/npm"
    }
  },
  "IDEs": {
    "VSCode": {
      "version": "1.81.1",
      "path": "/usr/local/bin/code"
    }
  },
  "npmPackages": {
    "vite": {
      "installed": "2.9.16",
      "wanted": "^2.9.9"
    },
    "vitest": {
      "installed": "0.12.10",
      "wanted": "^0.12.6"
    }
  }
}

vitest.config.ts
import tsconfigPaths from 'vite-tsconfig-paths';
import { defineConfig } from 'vitest/config';

export default defineConfig({
	plugins: [tsconfigPaths()],
	test: {
		globals: true,
		exclude: ['**/node_modules/**'],
		include: ['**/*.spec.ts'],
		coverage: {
			provider: 'istanbul',
			reporter: ['html'],
			reportsDirectory: './.tests/coverage',
		},
		reporters: ['default', 'html'],
		outputFile: './.tests/index.html',
	},
});

tsconfig.json
{
	"compilerOptions": {
		"target": "ES2020",
		"useDefineForClassFields": true,
		"module": "ESNext",
		"lib": ["ES2020"],
		"skipLibCheck": true,
		"baseUrl": "./src",
		"paths": {
			"core": ["./core/."],
			"conditional": ["./conditionals/."],
			"collections": ["./collections/."]
		},

		/* Bundler mode */
		"moduleResolution": "bundler",
		"allowImportingTsExtensions": true,
		"resolveJsonModule": true,
		"isolatedModules": true,
		"noEmit": true,

		/* Linting */
		"strict": true,
		"noUnusedLocals": false,
		"noUnusedParameters": false,
		"noFallthroughCasesInSwitch": true
	},
	"include": ["src"]
}

package.json
{
	....
	"type": "module",
	"files": [
		"dist"
	],
	"main": "./dist/std.umd.cjs",
	"module": "./dist/std.js",
	"types": "./dist/index.d.ts",
	"engines": {
		"npm": ">=8.15.0 <=9.6.7",
		"node": ">=16.17.1 <=18.17.1"
	},
	"exports": {
		".": {
			"import": "./dist/std.js",
			"require": "./dist/std.umd.cjs"
		}
	},
	"scripts": {
		"build": "tsc && vite build",
		"serve:tests": "vitest --coverage --ui",
		"lint:check": "eslint src --ext .ts --ext .tsx && tsc --noEmit",
		"postinstall": "husky install"
	},
	"devDependencies": {
		"@types/node": "20.5.9",
		"@typescript-eslint/eslint-plugin": "6.5.0",
		"@typescript-eslint/parser": "6.5.0",
		"@vitest/coverage-istanbul": "0.34.3",
		"@vitest/ui": "0.34.3",
		"eslint": "8.48.0",
		"eslint-plugin-check-file": "2.6.2",
		"eslint-plugin-no-null": "1.0.2",
		"husky": "8.0.3",
		"prettier": "3.0.3",
		"typescript": "5.2.2",
		"vite": "4.4.9",
		"vite-plugin-dts": "3.5.3",
		"vite-tsconfig-paths": "4.2.0",
		"vitest": "0.34.3"
	}
}

OS: macOS Ventura 13.5.1
VSCode version: 1.81.1 (Universal)
Node.js version: 16.17.1
Vitest version: 0.34.3
Vitest plugin version: v0.2.42

UPDATE
Even when i update node to the latest version, still exists this issue

It appears that the test explorer truncates file names where the path matches for all test files, even if that is part way through a folder name.
For example for this folder structure:

It truncates everything up to the end of ../outer:

So if we were to add another test file at the top level:

Then less of the path would be the same between all test files, so more gets displayed:

I think this is just the behaviour of the test explorer, and as far as I can tell there isn't a way to configure this to behave differently.

This should be fixed in pre-release 0.5.0 and higher. Note that the extension now requires Vitest 1.4.0 or higher.


Cargo.toml: extend SVSM kernel lints to whole workspace

Move required lints in kernel/src/lib.rs to Cargo.toml so they apply to all crates in the workspace.

@joergroedel manual apply failed again. Closing.


Merge master branch into cert-manager-feature-branch

Merged the master branch into cert-manager-feature-branch

Do we know why the unit test is failing @prafull01 ?

I have rebased the branch with cert-manager-feature-branch to fetch all the code changes and now it is working. @keith-mcclellan


LIBERO evaluation issue

I failed using LIBERO to evaluate the results
(openvla) root@autodl-container-852f45bfea-b1111753:~/znz/openvla#  python experiments/robot/libero/run_libero_eval.py
--model_family openvla
--pretrained_checkpoint /root/autodl-tmp/openvla-7b-finetuned-libero-spatial
--task_suite_name libero_spatial
--center_crop True
2024-12-25 21:32:38.947795: I tensorflow/core/util/port.cc:113] oneDNN custom operations are on. You may see slightly different numerical results due to floating-point round-off errors from different computation orders. To turn them off, set the environment variable TF_ENABLE_ONEDNN_OPTS=0.
2024-12-25 21:32:38.979465: E external/local_xla/xla/stream_executor/cuda/cuda_dnn.cc:9261] Unable to register cuDNN factory: Attempting to register factory for plugin cuDNN when one has already been registered
2024-12-25 21:32:38.979500: E external/local_xla/xla/stream_executor/cuda/cuda_fft.cc:607] Unable to register cuFFT factory: Attempting to register factory for plugin cuFFT when one has already been registered
2024-12-25 21:32:38.980982: E external/local_xla/xla/stream_executor/cuda/cuda_blas.cc:1515] Unable to register cuBLAS factory: Attempting to register factory for plugin cuBLAS when one has already been registered
2024-12-25 21:32:38.987062: I tensorflow/core/platform/cpu_feature_guard.cc:182] This TensorFlow binary is optimized to use available CPU instructions in performance-critical operations.
To enable the following instructions: AVX2 AVX512F AVX512_VNNI AVX512_BF16 FMA, in other operations, rebuild TensorFlow with the appropriate compiler flags.
2024-12-25 21:32:39.681459: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Could not find TensorRT
[robosuite WARNING] No private macro file found! (macros.py:53)
[robosuite WARNING] It is recommended to use a private macro file (macros.py:54)
[robosuite WARNING] To setup, run: python /root/miniconda3/envs/openvla/lib/python3.10/site-packages/robosuite/scripts/setup_macros.py (macros.py:55)
2024-12-25 21:32:43.137142: W tensorflow/core/common_runtime/gpu/gpu_device.cc:2348] TensorFlow was not built with CUDA kernel binaries compatible with compute capability 9.0. CUDA kernels will be jit-compiled from PTX, which could take 30 minutes or longer.
GenerateConfig(model_family='openvla', pretrained_checkpoint='/root/autodl-tmp/openvla-7b-finetuned-libero-spatial', load_in_8bit=False, load_in_4bit=False, center_crop=True, task_suite_name='libero_spatial', num_steps_wait=10, num_trials_per_task=50, run_id_note=None, local_log_dir='./experiments/logs', use_wandb=False, wandb_project='YOUR_WANDB_PROJECT', wandb_entity='YOUR_WANDB_ENTITY', seed=7)
[] Instantiating Pretrained VLA model
[] Loading in BF16 with Flash-Attention Enabled
:283: DeprecationWarning: the load_module() method is deprecated and slated for removal in Python 3.12; use exec_module() instead
Loading checkpoint shards: 100%|███████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 4/4 [00:01<00:00,  3.33it/s]
Loaded model: <class 'transformers_modules.openvla.openvla-7b.31f090d05236101ebfc381b61c674dd4746d4ce0.modeling_prismatic.OpenVLAForActionPrediction'>
Logging to local log file: ./experiments/logs/EVAL-libero_spatial-openvla-2024_12_25-21_32_51.txt
[info] using task orders [0, 1, 2, 3, 4, 5, 6, 7, 8, 9]
Task suite: libero_spatial
0%|                                                                                | 0/10 [00:00<?, ?it/s][Warning]: datasets path /root/znz/openvla/LIBERO/libero/libero/../datasets does not exist!
[Warning]: datasets path /root/znz/openvla/LIBERO/libero/libero/../datasets does not exist!
Task: pick up the black bowl between the plate and the ramekin and place it on the plate                                                                                | 0/50 [00:00<?, ?it/s]
Starting episode 1...
Floating point exception (core dumped)
bash: --task_suite_name: command not found

why does OpenVLA encounter Floating point exception (core dumped) ? I have debug and it failed soon after passing through the model

It doesn't have any problem when in the finetune process, and can calculate the loss conveniently. But failed in the eval

I have lost the --task_suite_name first time. But still same problem
(openvla) root@autodl-container-852f45bfea-b1111753:~/znz/openvla# python experiments/robot/libero/run_libero_eval.py
--model_family openvla
--pretrained_checkpoint /root/autodl-tmp/openvla-7b-finetuned-libero-spatial
--task_suite_name libero_spatial
--center_crop True
2024-12-25 21:45:36.559080: I tensorflow/core/util/port.cc:113] oneDNN custom operations are on. You may see slightly different numerical results due to floating-point round-off errors from different computation orders. To turn them off, set the environment variable TF_ENABLE_ONEDNN_OPTS=0.
2024-12-25 21:45:36.591817: E external/local_xla/xla/stream_executor/cuda/cuda_dnn.cc:9261] Unable to register cuDNN factory: Attempting to register factory for plugin cuDNN when one has already been registered
2024-12-25 21:45:36.591851: E external/local_xla/xla/stream_executor/cuda/cuda_fft.cc:607] Unable to register cuFFT factory: Attempting to register factory for plugin cuFFT when one has already been registered
2024-12-25 21:45:36.593344: E external/local_xla/xla/stream_executor/cuda/cuda_blas.cc:1515] Unable to register cuBLAS factory: Attempting to register factory for plugin cuBLAS when one has already been registered
2024-12-25 21:45:36.599599: I tensorflow/core/platform/cpu_feature_guard.cc:182] This TensorFlow binary is optimized to use available CPU instructions in performance-critical operations.
To enable the following instructions: AVX2 AVX512F AVX512_VNNI AVX512_BF16 FMA, in other operations, rebuild TensorFlow with the appropriate compiler flags.
2024-12-25 21:45:37.280173: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Could not find TensorRT
[robosuite WARNING] No private macro file found! (macros.py:53)
[robosuite WARNING] It is recommended to use a private macro file (macros.py:54)
[robosuite WARNING] To setup, run: python /root/miniconda3/envs/openvla/lib/python3.10/site-packages/robosuite/scripts/setup_macros.py (macros.py:55)
2024-12-25 21:45:40.520420: W tensorflow/core/common_runtime/gpu/gpu_device.cc:2348] TensorFlow was not built with CUDA kernel binaries compatible with compute capability 9.0. CUDA kernels will be jit-compiled from PTX, which could take 30 minutes or longer.
GenerateConfig(model_family='openvla', pretrained_checkpoint='/root/autodl-tmp/openvla-7b-finetuned-libero-spatial', load_in_8bit=False, load_in_4bit=False, center_crop=True, task_suite_name='libero_spatial', num_steps_wait=10, num_trials_per_task=50, run_id_note=None, local_log_dir='./experiments/logs', use_wandb=False, wandb_project='YOUR_WANDB_PROJECT', wandb_entity='YOUR_WANDB_ENTITY', seed=7)
[] Instantiating Pretrained VLA model
[] Loading in BF16 with Flash-Attention Enabled
:283: DeprecationWarning: the load_module() method is deprecated and slated for removal in Python 3.12; use exec_module() instead
Loading checkpoint shards: 100%|███████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 4/4 [00:00<00:00,  4.14it/s]
Loaded model: <class 'transformers_modules.openvla.openvla-7b.31f090d05236101ebfc381b61c674dd4746d4ce0.modeling_prismatic.OpenVLAForActionPrediction'>
Logging to local log file: ./experiments/logs/EVAL-libero_spatial-openvla-2024_12_25-21_45_49.txt
[info] using task orders [0, 1, 2, 3, 4, 5, 6, 7, 8, 9]
Task suite: libero_spatial
0%|                                                                                | 0/10 [00:00<?, ?it/s][Warning]: datasets path /root/znz/openvla/LIBERO/libero/libero/../datasets does not exist!
[Warning]: datasets path /root/znz/openvla/LIBERO/libero/libero/../datasets does not exist!
Task: pick up the black bowl between the plate and the ramekin and place it on the plate                                                                                | 0/50 [00:00<?, ?it/s]
Starting episode 1...
0
1
Floating point exception (core dumped)


Modified Drum kit and change its theme

Related Issue

Info about Issue or bug

Closes: #[issue number that will be closed through this PR]
Describe the changes you've made
A clear and concise description of what you have done to successfully close your assigned issue. Any new files? or anything you feel to let us know!
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
A clear and concise description of it.
Checklist:

[ ] My code follows the style guidelines of this project.
[ ] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[ ] New and existing unit tests pass locally with my changes.

Screenshots
Put any screenshot(s) of the project here.

I made some changes in Drumkit. I changed the drum theme and it make a different different sound by clicking on the particular drum. I hope this changes is really helpful for this issue.

Hi @aditidewangan Please resolve the conflicts.

Hello @aditidewangan Kindly reply in 2 days or we need close this issue.

Okay @harshgupta20 I try to resolve the conflicts.

What is the problem in my changes?

Actually someone else was also working who changed few things around to add some new feat.
What you need to do is to accept both changes i.e your changes + current changes. So that your changes reflect without affecting current changes or you could change also.

Okay but how can I accept the changes because this issue is not made by me.

what you can do pull the latest changes in main branch merge in your current branch. Or directly pull to current branch. After that you will be able to see merge conflict.

Okay @991rajat


very confusing UI to customize the blacklist

I am trying to use the GUI to customize the blacklist but this is very confusing. what does checking a box do? does it mark it as something that needs to be removed or mark as something that needs to be kept?

Answered Here: #336

Answered Here: #336


Do not store segment in error, only segment id

fix fixture: add segment id
only store segment_id in error, I think whole segment is not needed
clean up some outdated comments

Shall we do the same for SegmentRule?
I was thinking about it, but decided against it, because essentially we print everything in this


Add IPv4 Configuration File Access Helper

Summary
This commit adds support to access an IPv4 Configuration file similar to the Linux dhpc.client ipfcg file.  This version, tailored for deeply embedded systems supports several options to tailor the file and file access to different environments.  It supports:

Writable as well as read-only configuration files.
ASCII human readable files as well as smaller binary files.
It supports using character driver access to constrained media (such as EEPROM).

Impact
Since nothing uses the IPv4 Configuration file at present, there should be no impact from this addition.
Testing
To be provided after initial review.

@patacongo - Thank you!

I just added another option to use character devices to support writing at an offset into the device.
@davids5 So you are going to test it?  I was thinking of adding an examples/ipcfg to just to a simple read/write test.  But I am sure your testing would be better and we don't really need another just example.

@patacongo - yes see top post:

I need to finish some other tasks and will test this ASAP (Please give me a day or 2).

@davids5 I did add an example/ipcfg so that I can do testing as well.  But it sounds like you are ahead of me.

@patacongo yep. Here is the last change needed https://github.com/patacongo/incubator-nuttx-apps/commit/8e3d080ae149a164f8a1ea42eb1285db290b7ecd

@patacongo - If you are done. We can merge this. Then I can fixup the 2 PR I had to use this.

@davids5 I final completed testing using examples/ipcfg.  It is good to merge from my point of view.

@patacongo

@davids5 I final completed testing using examples/ipcfg. It is good to merge from my point of view.

I tested the ASCII as well it is AOK.
Once CI is happy I will merge it.
Greg, Thank you for taking the time to do this! It made it clear to me what was needed. Some time I just have to see it to understand. Than you again!


Using ToString on DateTime object after using AddDays produces incorrect month

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
PS>  $d = [DateTime]::ParseExact('2020-01-01', 'yyyy-mm-dd', $null)
PS> $d = $d.AddDays(31)
PS> $d; $d.ToString('yyyy-mm-dd'); $d.GetDateTimeFormats()[0]
Expected behavior
February 1, 2020 12:01:00 AM
2020-02-01
2020-02-01

Actual behavior
February 1, 2020 12:01:00 AM
2020-01-01
2020-02-01

Error details
No response
Environment data
Name                           Value
----                           -----
PSVersion                      7.2.0-preview.10
PSEdition                      Core
GitCommitId                    7.2.0-preview.10
OS                             Microsoft Windows 10.0.19043
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

'yyyy-mm-dd' -- year-minutes-days

Thanks. I had assumed that "mm" would be month since "%m" is for month.


AseDataReader.RecordsAffected doesnt return correct value

I've noticed the RecordsAffected property doesn't seem to behave as expected. Does anybody else experience the same problem?
AseDataReader.RecordsAffected returns 0 for all Update, delete and insert statements, returns row count for select statements
according to other implementations of DataReader including Sybase documentation it should return The number of rows changed, inserted, or deleted; 0 if no rows were affected or the statement failed; and -1 for SELECT statements.
Environment

Core 2.2
AdoNetCore.AseClient 0.16

Hi @sgwyn good find. It appears our implementation of RecordsAffected is incorrect.
I'll add a fix to the next release

@sgwyn - heard you had a fix for this. Can you send a PR?


Feature Request: API for Dictionary registration

Hi,
i've created a custom dictionary to for Advanced Slides syntax. I would like to ship this dictionary with the current build of advanced slides and register it dynamically when the user also has installed various complements plugin. Therefore it would be be cool when provide me such an api for integration

Hi, @MSzturc :)
Does that mean it is needed to add the path(URL) to the Custom dictionary paths via the API? (ex: addCustomDictionaryPath(path: string) )

Yes exactly, with a path like .obsidian/plugins/obsidian-advanced-slides/autoComplete/dict.md. I would call it when advanced slides get's initialized so you have to check if the path exists in custom dictionary list

OK. LGTM 👍
Probably, I will add the API such a ensureCustomDictionaryPath(path: string, state: "present" | "absent").

@MSzturc
I have released v6.1.0-beta1 🚀
Please confirm whether it works as you expected via BRAT.
P.S.
This is my first time creating a public API as the Obsidian plugin, so if you know a better implementation, It would be great to hear about it :)

Awesome, works as expected! The implementation is fine. One think i would introduce with v.6.1 is an API Method isFeatureSupported(featureName: string) => boolean With this method every user of the API can check if the installed variousComponents Version supports a feature. This allows a more robust implementation since it could be used to handle version mismatches.
Thx ;-)

Got a prototype running ;-)
https://user-images.githubusercontent.com/4275753/166114827-b7402f6b-d51e-4ff4-9b8c-70fc4d9b29f8.mov

isFeatureSupported(featureName: string) => boolean

Thank you! I'll add the function 👍

@MSzturc
I have released v6.2.0-beta1 🚀
There is a bit of breaking change. Please fix the code as follows.
- app.plugins.plugins["various-complements"].ensureCustomDictionaryPath("./your-dictionary.md", "present")
+ app.plugins.plugins["various-complements"].api.ensureCustomDictionaryPath("./your-dictionary.md", "present")

done.

I noticed that the current version couldn't have settings for each custom dictionary file.

I think we need to embed the settings in custom dictionary files. So I'll support JSON format as a dictionary file that includes the settings.

Ah... it doesn't need settings except for Caret location symbol after complement in the case of JSON file format :)

I noticed that the current version couldn't have settings for each custom dictionary file.

That would be a nice feature since there are some Settings to get the dict working properly:
settings.maxNumberOfSuggestions = 15; //this one is optional but more than 5 feels better using it
settings.enableCustomDictionaryComplement = true; // this could be activated automatically when ensureCustomDictionaryPath was called
settings.insertAfterCompletion = false; //feels better, otherwise you have to move the cursor back after many completion (would be cool when this one would be dict specific
settings.caretLocationSymbolAfterComplement = "<CARET>"; //would be cool if it's dict specific

A dict based on a JSON file would make a hierarchically dictionary easier to implement since we wont have to parse a md file

settings.caretLocationSymbolAfterComplement = ""; //would be cool if it's dict specific

I'll add this option certainly. However I think that will not add other settings for now.

A dict based on a JSON file would make a hierarchically dictionary easier to implement since we wont have to parse a md file

Yes. Originally I don't use Custom dictionary (I do now), so I selected the specifications based on the IME dictionary, but already the expressions are pushing the limits 😅

@MSzturc
Mostly done. However, I am afraid of degradation. Therefore it would be great if you try it for a few days after I release it as v6.1.0-beta4 🙏
https://user-images.githubusercontent.com/9500018/166485820-2f1150fd-8e0b-46e4-901f-62a935639be4.mp4

Support properties.
{
  "caretSymbol": "[[CARET]]",
  "ignoreSpaceAfterCompletion": true,
  "words": [
    { "value": "custom dictionary1" },
    { "value": "custom dictionary2", "description": "hogehoge" },
    { "value": "custom dictionary3", "aliases": ["w3", "word3"] },
    { "value": "```json\n[[CARET]]\n```", "displayed": "code-json" }
  ]
}

settings.insertAfterCompletion = false; //feels better, otherwise you have to move the cursor back after many completion (would be cool when this one would be dict specific
settings.caretLocationSymbolAfterComplement = ""; //would be cool if it's dict specific

Uhhh... looks neat! Will test it tomorrow and give you feedback. Thx

Will v6.1.0-beta4 support dict hierarchy (as described in #119 )?

No it doesn't because of https://github.com/tadashi-aikawa/obsidian-various-complements-plugin/issues/119#issuecomment-1114161098.

@MSzturc
I have released v6.1.0-beta4 🚀

Played a bit with it! Works like intended! Thx man

@MSzturc
Thanks for conforming!
By the way, can I remove the API of this issue by any chance? Because Advanced Slides for Obsidian seems not to use this feature.
Best regards.

Yeah, if you want you may remove it. I experimented yesterday a bit with it and i was not comfortable with my dictionary without the hierarchy feature, Advanced Slides is too complex for a basic auto complete, so i hacked an own suggester... but i have to rewrite it after its stable.. the code is too complex

OK! Thanks for reply 👍


Question: parsing a xml string back into an XMLDocument

Searched through a lot of topics, but couldnt find a solution to my problem.
I have a string that is already made up out of contents that an XMLElement standardly has, with subelements and its syntaxes and such. How would I go about parsing this string into another XMLElement as a XMLElement child?

Solution to future readers: use XMLDocument.Parse()


no such container

no such container
404
no such id: docker-ubuntu-vnc-desktop

Sounds like a simple restart of kitematic should do it - I would also recommend upgrading to the latest Docker Toolbox if you haven't already done so.


Add a way to specify that Forge is Mojang mapped

@Jab125 any idea when this will be pulled in? Think I'm running into exactly what this is supposed to fix 😄
Fails during remapping with following message Cannot get mappings service with Mojang mappings without Mojang merging enabled!

When can this be merged as it's an issue I keep running into


tower_grafana_dashboards

Type of repository request
Whether this request is a creation of a new repository, migration of existing repository, or deletion of a repository
A: New Repository
Requested name for the repository
Name of the repository to create or location of existing repository to migrate.
A: ansible-tower-grafana-dashboards
Repository Description
Description applied to the newly created repository.
A:  Playbook for installing node_exporter on rhel8 and dashboards to import into Grafana and monitor Ansible Tower metrics.
Admin Access
Name of the team(s) and/or user(s) that should have admin access on the newly created repository.
A: leoaaraujo
Write Access
Name of the team(s) and/or user(s) that should have write access on the newly created repository.
A: leoaaraujo
Read Access
Name of the team(s) and/or user(s) that should have read access on the newly created repository.
A leoaaraujo
Additional details
Any additional information or context to provide for this request.
A: the link to this repository will be made available in an article that will be published on the ansible blog to download the dashboards and playbook.

Review request:
cc: @redhat-cop/container-cop-core

@jlozadad Any chance that we can create the repo?

Repository has been created as requested

Thanks @sabre1041 and @liquidat


The limit on text can be at most 1GB on Postgres

So the migration doesn't work

I reduced text limit to 256Mb, it should be more than enough for any page and far than Postgresql limits.


Resume from save

Adds an option to resume from a save point without interfering with the building list.

I like the thought behind this, but I think I will call it Start instead - as in start the steps here. :) Thanks for the help. I hope you will come with more valuable feedback.