yt-dlp with ffmpeg H.264 Nvidia NVEnc and H.265 Nvidia NVEnc

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2022.11.11 (update instructions) or later (specify commit)
[X] I've searched the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
I place yt-dlp.exe and ffmpeg.exe in the same folder to download videos and use ffmpeg.exe to convert videos to different formats. Is there a way when after yt-dlp.exe finishes downloading a video, to have ffmpeg.exe use H.264 Nvidia NVEnc or H.265 Nvidia NVEnc when converting video formats to other video formats? Example webm to mp4 webm to mkv mp4 to mkv etc?
Thanks
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

to have ffmpeg.exe use H.264 Nvidia NVEnc or H.265 Nvidia NVEnc when converting video formats to other video formats

Shouldn'y u be asking this in some ffmpeg forums? Once u figure out the answer to that, simply passthrough the relevent args using --ppa


Bug in make_s7_df() when running on the full sample

Error as below:
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
[/home/u5708159/python/Modules/ppxftests/scripts/s7/Examples](https://vscode-remote+ssh-002dremote-002bmisfit-002eanu-002eedu-002eau.vscode-resource.vscode-cdn.net/home/u5708159/python/Modules/ppxftests/scripts/s7/Examples) - S7.ipynb Cell 5 line 1
----> [1](vscode-notebook-cell://ssh-remote%2Bmisfit.anu.edu.au/home/u5708159/python/Modules/ppxftests/scripts/s7/Examples%20-%20S7.ipynb#X12sdnNjb2RlLXJlbW90ZQ%3D%3D?line=0) make_s7_df(
      [2](vscode-notebook-cell://ssh-remote%2Bmisfit.anu.edu.au/home/u5708159/python/Modules/ppxftests/scripts/s7/Examples%20-%20S7.ipynb#X12sdnNjb2RlLXJlbW90ZQ%3D%3D?line=1)             eline_SNR_min=3,
      [3](vscode-notebook-cell://ssh-remote%2Bmisfit.anu.edu.au/home/u5708159/python/Modules/ppxftests/scripts/s7/Examples%20-%20S7.ipynb#X12sdnNjb2RlLXJlbW90ZQ%3D%3D?line=2)             eline_ANR_min=3,
      [4](vscode-notebook-cell://ssh-remote%2Bmisfit.anu.edu.au/home/u5708159/python/Modules/ppxftests/scripts/s7/Examples%20-%20S7.ipynb#X12sdnNjb2RlLXJlbW90ZQ%3D%3D?line=3)             correct_extinction=True,
      [5](vscode-notebook-cell://ssh-remote%2Bmisfit.anu.edu.au/home/u5708159/python/Modules/ppxftests/scripts/s7/Examples%20-%20S7.ipynb#X12sdnNjb2RlLXJlbW90ZQ%3D%3D?line=4)             metallicity_diagnostics=["N2Ha_PP04",],
      [6](vscode-notebook-cell://ssh-remote%2Bmisfit.anu.edu.au/home/u5708159/python/Modules/ppxftests/scripts/s7/Examples%20-%20S7.ipynb#X12sdnNjb2RlLXJlbW90ZQ%3D%3D?line=5)             nthreads=1)

File [~/python/Modules/spaxelsleuth/spaxelsleuth/loaddata/s7.py:681](https://vscode-remote+ssh-002dremote-002bmisfit-002eanu-002eedu-002eau.vscode-resource.vscode-cdn.net/home/u5708159/python/Modules/ppxftests/scripts/s7/~/python/Modules/spaxelsleuth/spaxelsleuth/loaddata/s7.py:681), in make_s7_df(eline_SNR_min, eline_ANR_min, correct_extinction, gals, sigma_gas_SNR_min, line_flux_SNR_cut, missing_fluxes_cut, line_amplitude_SNR_cut, flux_fraction_cut, sigma_gas_SNR_cut, vgrad_cut, metallicity_diagnostics, nthreads)
    679 colnames = res_list[0][1]
    680 eline_list = res_list[0][2]
--> 681 df_spaxels = pd.DataFrame(np.vstack(tuple(rows_list_all)),
    682                           columns=colnames)
    684 # Cast to float data types
    685 for col in [c for c in df_spaxels.columns if c != "ID"]:
    686     # Check if column values can be cast to float

File [~/.conda/envs/py310/lib/python3.10/site-packages/numpy/core/shape_base.py:289](https://vscode-remote+ssh-002dremote-002bmisfit-002eanu-002eedu-002eau.vscode-resource.vscode-cdn.net/home/u5708159/python/Modules/ppxftests/scripts/s7/~/.conda/envs/py310/lib/python3.10/site-packages/numpy/core/shape_base.py:289), in vstack(tup, dtype, casting)
    287 if not isinstance(arrs, list):
    288     arrs = [arrs]
--> 289 return _nx.concatenate(arrs, 0, dtype=dtype, casting=casting)

ValueError: all the input array dimensions except for the concatenation axis must match exactly, but along dimension 1, the array at index 0 has size 85 and the array at index 1 has size 99

Fixed.

Accidentally closed the wrong issue...

The issue was because some galaxies are missing LZIFU outputs for certain emission lines and for other products (specifically for 3C278 and one of the PKS sources). This introduced a secondary bug where in some cases a truncated list of emission lines was being passed to add_columns(). It has been fixed by monitoring the number of columns returned by _process_s7().


STOR-1988: add WIF support

Prerequisites:

Configure GCP infrastructure for WIF: https://docs.google.com/document/d/1U1032SG_pTK5FyF8vOB-vGtZGCZ0Mq9FnQdRkd2k998/edit#heading=h.9e0hzxyge5hl
NOTE: If cluster was installed in WIF mode there should already be one workload identity pool configured and can be reused.

Testing:
Before installing Filestore no credentials request is created:
$ oc -n openshift-cloud-credential-operator get credentialsrequest/openshift-gcp-filestore-csi-driver-operator -o yaml
Error from server (NotFound): credentialsrequests.cloudcredential.openshift.io "openshift-gcp-filestore-csi-driver-operator" not found

Install Filestore operator via OperatorHub, let operator create credentials request:

NOTE: Make sure we have audience, poolID, providerID and serviceAccountEmail which indicate successful WIF mode detection:

oc -n openshift-cloud-credential-operator get credentialsrequest/openshift-gcp-filestore-csi-driver-operator -o yaml
apiVersion: cloudcredential.openshift.io/v1
kind: CredentialsRequest
metadata:
  annotations:
    include.release.openshift.io/self-managed-high-availability: "true"
    include.release.openshift.io/single-node-developer: "true"
    operator.openshift.io/spec-hash: 7cda3595ea3dc856afec7afd60c7cd73b1bc77e019b40ccc1828f36999d7aa5d
  creationTimestamp: "2024-09-27T12:44:06Z"
  finalizers:
  - cloudcredential.openshift.io/deprovision
  generation: 1
  name: openshift-gcp-filestore-csi-driver-operator
  namespace: openshift-cloud-credential-operator
  resourceVersion: "140918"
  uid: e3e0ef67-4e87-4312-a6e5-5f72b4ee9db5
spec:
  providerSpec:
    apiVersion: cloudcredential.openshift.io/v1
    audience: //iam.googleapis.com/projects/1043659492591/locations/global/workloadIdentityPools/rbednar-cluster-03-9psw9/providers/rbednar-cluster-03-9psw9
    kind: GCPProviderSpec
    poolID: rbednar-cluster-03-9psw9
    predefinedRoles:
    - roles/file.editor
    - roles/resourcemanager.tagUser
    providerID: rbednar-cluster-03-9psw9
    serviceAccountEmail<EMAIL_ADDRESS>    skipServiceCheck: true
  secretRef:
    name: gcp-filestore-cloud-credentials
    namespace: openshift-cluster-csi-drivers
  serviceAccountNames:
  - gcp-filestore-csi-driver-operator
  - gcp-filestore-csi-driver-controller-sa
status:
  lastSyncGeneration: 1
  lastSyncTimestamp: "2024-09-27T12:44:31Z"
  provisioned: true

Secret for Filestore is created by CCO:
oc -n openshift-cluster-csi-drivers get secret/gcp-filestore-cloud-credentials -o yaml
apiVersion: v1
data:
  service_account.json: XXXXXXXX
kind: Secret
metadata:
  annotations:
    cloudcredential.openshift.io/credentials-request: openshift-cloud-credential-operator/openshift-gcp-filestore-csi-driver-operator
  creationTimestamp: "2024-09-27T12:44:12Z"
  labels:
    cloudcredential.openshift.io/credentials-request: "true"
  name: gcp-filestore-cloud-credentials
  namespace: openshift-cluster-csi-drivers
  resourceVersion: "140698"
  uid: dc7ba206-712e-4e66-bbe3-3e2bd264687b
type: Opaque

Create SC with correct network name:
oc create -f - << EOF
kind: StorageClass
apiVersion: storage.k8s.io/v1
metadata:
  name: filestore-csi
provisioner: filestore.csi.storage.gke.io
parameters:
  connect-mode: DIRECT_PEERING
  network: rbednar-cluster-03-9psw9-network
allowVolumeExpansion: true
volumeBindingMode: Immediate
EOF

Create ClusterCSIDriver object:
$ oc apply -f - <<EOF
 apiVersion: operator.openshift.io/v1
 kind: ClusterCSIDriver
 metadata:
    name: filestore.csi.storage.gke.io
 spec:
    logLevel: Normal
    managementState: Managed
    operatorLogLevel: Trace
EOF

Create PVC (takes a while since min size is 1TiB)
oc -n openshift-cluster-csi-drivers get pvc/pvc-1
NAME    STATUS   VOLUME                                     CAPACITY   ACCESS MODES   STORAGECLASS    VOLUMEATTRIBUTESCLASS   AGE
pvc-1   Bound    pvc-64ef838e-278d-4ac7-87d0-6815e735f379   1Ti        RWO            filestore-csi   <unset>                 5m

Check driver logs:
I0927 12:49:18.618014 1 controller.go:316] CreateVolume succeeded: volume:<capacity_bytes:1099511627776 volume_id:"modeInstance/us-central1-c/pvc-64ef838e-278d-4ac7-87d0-6815e735f379/vol1" volume_context:<key:"ip" value:"<IP_ADDRESS>" > volume_context:<key:"volume" value:"vol1" >

/hold
Wait for https://github.com/openshift/library-go/pull/1805 to merge and bump library go here.

/retest-required

/retest-required

/unhold

Had to add unrelated commit due to changes pulled by library-go bump: https://github.com/openshift/library-go/pull/1787 + https://github.com/openshift/library-go/pull/1824

/lgtm

Web console testing is blocked by https://issues.redhat.com/browse/OCPBUGS-43780

@chao007 this "warning" is expected, it means the console detected WIF mode correctly. On the next page after clicking "Install" you will be presented with the form for inputting the required values. I tested this a while ago and could configure the operator without any issues this way.

@RomanBednar Thanks for your clarification. I pasted this pic in order to say that filestore operator has support wif now. At least the features.operators.openshift.io/token-auth-gcp: "true" worked now.

/label qe-approved

/cherry-pick release-4.17


Bug: [eslintperfectionist/sort-switch-case] incoherent error message Expected "default" to come before "default".

Describe the bug
Get an incoherent eslint error message  for eslintperfectionist/sort-switch-case:
Expected "default" to come before "default".

Code example
Project (for testing – remove the eslint-disable comment): https://github.com/OlivierZal/melcloud-api/blob/fd854ab272e3b818b0570878d9815e938481d7ec/src/facades/device_ata.ts#L48
#getTargetTemperatureRange(operationMode = this.setData.OperationMode): {
    max: number
    min: number
  } {
    switch (operationMode) {
      case OperationMode.auto:
        return {
          max: this.data.MaxTempAutomatic,
          min: this.data.MinTempAutomatic,
        }
      case OperationMode.cool:
      case OperationMode.dry:
        return { max: this.data.MaxTempCoolDry, min: this.data.MinTempCoolDry }
      case OperationMode.heat:
      default:
        return { max: this.data.MaxTempHeat, min: this.data.MinTempHeat }
    }
  }

ESLint version
v9.7.0
ESLint Plugin Perfectionist version
v3.0.0
Additional comments
Here are screenshots:

Validations

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.

Thank you for your issue!
I was able to reproduce the problem. I'll look into fixing it.

Thank you for your issue!
This problem was fixed in<PHONE_NUMBER>52be679f0f5099f9f488d5f4704542 and released in v3.1.0.


valueClass not working with JSON object

I'm having an issue working with the valueClass property. If I add a property to the data object to store the value of the class string (i.e., node.data.className), it doesn't appear in the class attribute of the span that contains the value of the column.
After looking at the code where valClass is defined (line 648 in jstreegrid.js), I think that t.attr(col.valueClass) would always be undefined as that property is stored in the objData object rather than t. Is that correct? Forgive me if I've missed something or am adding the property in the wrong place.
If I replace the valClass declarative with something closer to the way val is defined a bit more previously in the code (see code below), it seems to work. Also, you could have the opportunity to pass a function as a value the as per the value property.
Thanks in advance for your help
Jaime
if (col.valueClass !== undefined && col.valueClass !== null) {
   if (objData.data !== null && objData.data !== undefined && objData.data[col.valueClass] !== undefined) {
      valClass = objData.data[col.valueClass];
    } else {
      valClass = "";
    }
}

Why, yes, I think you are right. It should affect a few things in those lines LL647-660.
I want to run some tests....

Hi Avi,
Thanks for getting back to quickly. Let me know the outcome of your
testing.
Thanks again!
Jaime
On Jun 1, 2015 8:45 AM, "Avi Deitcher"<EMAIL_ADDRESS>wrote:

Why, yes, I think you are right. It should affect a few things in those
lines LL647-660.
I want to run some tests....
—
Reply to this email directly or view it on GitHub
https://github.com/deitch/jstree-grid/issues/96#issuecomment-107428358.

Very nice catch. I am pushing it out now. Thank you.


Up z-score to 30

So that we have fewer alerts

Quick note: I think Jon did look at the last ~10 PRs in  arrow/arrow and determined that the false-positive alerting rate is obviously too large.
So, this really is not fine-tuning. This is acknowledging that PR authors basically ignore the feedback on arrow PRs, and this is a rather brutal step towards alerting less often, with the hope to change the trend.

Thank you both for looking into this.  There are two variables at play:

The false positive rate (addressed by this PR)
The fact that regression is alerted via a comment on a closed PR

This PR addresses the first, but still doesn't address the second.  This is what my notification feed looks like:

I see about 20-40 rows per day.  Processing these notifications is a considerable chunk of my day.  If the row starts with a purple icon then I dismiss it without further thought.  Unfortunately, this purple icon does not mean "This notification is telling you that a PR was closed" but it reflects "the current state of the PR is closed".
So a "a regression was detected" notification (which is interesting) is identical to "a PR was closed / merged" notification (which is not).
I'm not sure how best to address the problem.  One possibility would be to locate the issue associated with the PR, re-open it, and then comment on the issue.


Github Actions

This workflow builds mathlib docs for each commit pushed to any branch and additionally deploys the built docs to the web for each commit to master. (This way we don't have to manually trigger a build on mathlib to update the docs.)

I think it is a good idea to have CI here as well.  I'm not so sure I like the duplication of the deployment logic and responsibility.  I'm pretty sure we'll forget to update one of them in the future.

Good point. As far as reducing deployment logic, I can try to make one script (merging ci.sh here and deploy_docs.sh in mathlib) that we can run both here and in mathlib's workflow. Would it be better to host that here or in mathlib?

OK, this now uses the deploy_docs.sh script from mathlib.


Several patches

This is a all of the changes that I have been working on for this module. There are 4 major things: support for binary messages, a bugfix for partial replication (or deletes) that can leave nodes stuck, remove v2 (> 3 years old) and better documentation. I'll try and split these changes into smaller PRs, so don't merge this, wait for the others.

All done


Added preferences option for the UI style

Currently simply a fixed set of paths to the CSS files. A theme class or
map can be made later on. Likewise a set of all the existing stages so
you don't have to hard-code replacing the style of each stage.
(That's what the commit says already.)

Looks fine, and what about a reload button to support live css editing?

Can you test if it reloads properly by switching the style twice?
On my system JavaFX loads the old CSS file until I restart the application completely.

That works fine for me, maybe you should update your system...


Fix force parameter

I plan to add more tests, so marking this WIP for now.

I guess it's no longer WIP! :)  I was working on the provider tests, but that started getting quite involved.  I'll open a new PR when I get some more time on it.


OpenBSD ld: error: unable to find library -ldl

Which operating system does the issue occur on?
OpenBSD 6.6.
error: linking with `cc` failed: exit code: 1
  |
  = note: "cc" "-Wl,--as-needed" "-Wl,-z,noexecstack" "-m64" "-L" "/usr/local/lib/rustlib/x86_64-unknown-openbsd/lib" "/home/j/code/github.com/jwilm/alacritty/target/release/deps/alacritty-7b69b3e2c744c6f6.alacritty.2i7timb7-cgu.4.rcgu.o" "-o" "/home/j/code/github.com/jwilm/alacritty/target/release/deps/alacritty-7b69b3e2c744c6f6" "-Wl,--gc-sections" "-pie" "-Wl,-zrelro" "-Wl,-znow" "-Wl,-O1" "-nodefaultlibs" "-L" "/home/j/code/github.com/jwilm/alacritty/target/release/deps" "-L" "/home/j/code/github.com/jwilm/alacritty/target/release/build/libloading-0bc66433ed08a8e8/out" "-L" "/usr/X11R6/lib" "-L" "/usr/X11R6/lib" "-L" "/usr/local/lib/rustlib/x86_64-unknown-openbsd/lib" "-Wl,-Bstatic" "/tmp/rustctmtoWF/liblibloading-e0f747f82c29590c.rlib" "-Wl,--start-group" "/tmp/rustctmtoWF/libbacktrace_sys-0805c4ce0b1f9ea4.rlib" "-Wl,--end-group" "/usr/local/lib/rustlib/x86_64-unknown-openbsd/lib/libcompiler_builtins-c11725f82f107574.rlib" "-Wl,-Bdynamic" "-lfontconfig" "-lexpat" "-lfreetype" "-lz" "-lfreetype" "-lz" "-lexpat" "-lxcb" "-lxcb" "-lxcb" "-lxcb" "-lxcb-render" "-lxcb-shape" "-lxcb-xfixes" "-ldl" "-lutil" "-lpthread" "-lc++abi" "-lc" "-lm" "-lutil" "-lcompiler_rt"
  = note: ld: error: unable to find library -ldl
          cc: error: linker command failed with exit code 1 (use -v to see invocation)

I'd assume you've installed Xenocara, Rust and Python already?
It sounds like you're missing a linking library.

I'd assume you've installed Xenocara, Rust and Python already?rustc 1.37.0, Python 3.7.4,

The current version I'm using is 0.3.3 (707f429). I just double checked and I'm still able to build at commit 707f429 but receive the error when building master.

Could you pin down which commit specifically broke it for you?

Closing, since alacritty can be build on OpenBSD(at least I've tested it on OpenBSD 6.7).


How about the robust of this method

same dataset, if I have select different numbers of high GINI value gene (pvalue 0.01, and 0.001), cannot get the same (similar) rare celltype group.

Hi gerrardmai,
Thanks for your question. The default value for the parameter p-value for UMI based RNA-seq data is 0.0001. With the data in our hands, the clustering results are quite robust for a range of epsilon. Of cause, you can tune the parameters and it is can be expected that you will get different results, especially for p-value. By the way, may I ask which platform your dataset is generated from?
Best,
Lan

Hi Jing
Thank you for using GiniClust. All the datasets can be downloaded directly from the original publications, as described in our GiniClust paper.
Best
Guo-Cheng

On Apr 17, 2019, at 5:57 AM, Jing Gong<EMAIL_ADDRESS>wrote:
Hi, Lan,
Thank you for the work. In the sample directory you have provided two data set (GBM and qPCR), does it possible to also upload the other three data sets you used in the GiniClust paper?
Thanks,
Jing
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

That's great, I'll look into the details, thanks.
Best,
Jing


Added log_all_trials param to NeptuneCallback

Added a log_all_trials param to NeptuneCallback to allow disabling logging of all trials, similar to what log_all_trials does in log_study_metadata()

@Raalsky - when you have a while, could you check the failure with Deepdiff? I am not able to reproduce it locally on Windows

@SiddhantSadangi Try to pin deepdiff to deepdiff==6.6.1 (or just <6.7).

@SiddhantSadangi Try to pin deepdiff to deepdiff==6.6.1 (or just <6.7).

@Raalsky - no luck

Let's .... skip the test :wink:. You should be able to use something like pytest.mark.skip + please post something to Jira about it :wink:

@Raalsky - fixed 🎉

@Raalsky - keeping the branch in case there's further feedback from the user before we publish a stable release


[Firebase/Performance] GULObjectSwizzler crashes on objc_disposeClassPair.

[REQUIRED] Step 1: Describe your environment

Xcode version: 13.1
Firebase SDK version: 8.10
Installation method: Any
Firebase Component: Firebase
Target platform(s): iOS

[REQUIRED] Step 2: Describe the problem
Steps to reproduce:
When using Firebase Performance along with Detox Sync (a framework used by Detox), there is a double ISA Swizzling by the components. This causes a crash when NSZombiesEnabled=NO.

Firebase Performance does ISA Swizzling (using GULObjectSwizzler).
Detox Sync does ISA Swizzling, with a dynamic subclass of the generated class, using DTXSwizzlingHelper.
GULObjectSwizzler calls to dispose its own generated subclasses using objc_disposeClassPair().
Crash with the error message: objc_disposeClassPair: class 'fir_<some uuid>_<class name>' still has subclasses, including 'fir_<some uuid>_<class name>(__detox_sync_<class name>)'!

This issue seems to be related: https://github.com/google/GoogleUtilities/issues/15.
Also, another explanation of a similar issue, with zombie objects: https://github.com/firebase/firebase-ios-sdk/issues/8321#issuecomment-900321941 (instead Zombie objects, we have DetoxSync objects).

This branch of DetoxSync reproduces the issue: https://github.com/asafkorem/DetoxSync/tree/feature/reproduce-detox-sync-crash: build and run ExampleApp/TimersTest.xcodeproj with NSZombiesEnabled=NO.

@paulb777 just to note this is of intense interest in react-native-firebase repo, our test harness is based entirely on Detox, and last time I did a survey of available e2e test tools, Detox still won, even with the horrific ios e2e crash frequency we're taking now. Would :heart: to see some sort of fix or remediation or something here :-)

Thanks @mikehardy, and sorry about the inconvenience around this thus far. Looks like we're really getting somewhere now.

@asafkorem @mikehardy Thanks for the detailed report and background. We'll take a look.  It's one of our priorities to overhaul how Firebase uses swizzling. However, a resolution may still be months out.
In the meantime, is omitting FirebasePerformance when Detox is used a viable workaround?

It is possible but is equivalent to saying omit performance from e2e testing for us

Thanks for the response @paulb777, as @mikehardy mentioned it is possible but we'd rather avoid that, I think it would be better as a workaround to set some compiler flag that will only omit the deallocation part that triggers this crash. This flag can be used when building the app for Detox tests.

@asafkorem is there any reason we can't ride-along with NSZombiesEnabled=1 behavior - that is, make GULObjectSwizzler think we are running the zombies instrument, and thus avoid de-allocation.
Stated differently, perhaps the "NSZombiesEnabled" check could more properly be thought of as a "don't dispose generated classes" check, just with a too-specific name, but it would serve us as well :thinking: . Have you tried it?

@mikehardy yes, I thought about it too and already tried this approach.
I wasn't able to set this environment variable using xcodebuild command :/

Oh, this makes more sense now.. thanks @mikehardy. I'll try this soon

cc: @visumickey

It appears to be working for me, if not completely, then "well". The first run I did I took a SIGSEGV in DetoxSync again, but then I was 10/10 on further runs (an incredible result, I was at maybe 1-2/10 success prior). I am using the special method of passing env vars through simctl as mentioned above. I'm about to make a commit to react-native-firebase that will have a github cross link so others may examine in case it's useful. Thanks for this trick, at least preliminary results show it appears workable to me

I'm a little concerned about the SIGSEGV. Was that just a freak accident? At least with the DetoxSync crash there was some transparency into what it was. With SIGSEGVs I dislike that they're usually a disaster to debug

I'll need to wait to collect quite a few more results I think, before I want to spend anyone's time in this repo, possibly chasing ghosts.

Hmm, sadly I think statistics is a thing and I simply had an improbably long run of good results. With quite a few more runs throughout the day / evening, I still definitely have a problem with Detox-current on firebase-ios-sdk-current on iOS-simulator-current on both local macOS x86-64 VMs and up on github actions macOS runners :shrug: - really seemed like an improvement, and I have a hunch statistically it still might be, but there's no silver bullet with the NSZombiesEnabled=1, at least the way I'm setting it https://github.com/invertase/react-native-firebase/runs/4511769043?check_suite_focus=true :shrug:

Yeah can confirm still seeing Signal 11 with SIMCTL_CHILD_NSZombiesEnabled=1. Is there an easy way of confirming that this property is being injected into the simulator build?

@mikehardy Adding SIMCTL_CHILD_NSZombiesEnabled=1 before the xcrun simctl launch command doesn't seems to solve this (NSZombiesEnabled is still false).

Really sorry I took everyone down the wrong path with my plural-zombies typo! Thanks @asafkorem for seeing that and clearing it up. Excited to have this mitigated, my CI is running now in react-native-firebase based off your PR there Asaf

@asafkorem  Nice catch!

@paulb777 @ncooke3 @visumickey
Although passing the NSZombieEnabled flag does solves this problem as a workaround, I would have preferred to have a dedicated flag for disabling this class-disposing from firebase-performance and not to ride on the zombie-enabled flag, which might cause other side-effects like ignoring real memory issues on detox tests or unnecessary memory consumption, especially if a clean solution may only arrive in months. 🙏🏼

.. I would have preferred to have a dedicated flag for disabling this class-disposing from firebase-performance and not to ride on the zombie-enabled flag, which might cause other side-effects ..

I've opened a PR for this: https://github.com/google/GoogleUtilities/pull/66

Thanks @asafkorem!  GoogleUtilities 7.7.0 is now published to CocoaPods and SPM with your PR that disables class disposing when the GULGeneratedClassDisposeDisabled environment variable is set.

Thanks @paulb777!
We'll replace the zombie flag with this one on Detox tests execution command.
FYI @mikehardy.

Sample size of 10 runs which - for this bug, in the react-native-firebase harness - is strongly statistically significant already. Looks great, thanks for the ping @asafkorem

Although enabling this flag is a bit workaround, it's a pretty decent one. In my opinion, we can definitely live with that, considering the fact that this crash happens when running the apps on Detox tests environment, and this kind of memory consumption (not disposing the generated classes) should not particularly worry us in this case.
I opened this issue in the context of this bug, and I think we can close it now since it was resolved. Thanks guys!

Thanks for the fixing PR @asafkorem !


Stability score for a sequence

Hi,
Thank you for sharing this great tool! I was wondering if there is a simple and quick way to use ThermoMPNN to extract a single score that would represent the overall stability of my protein. The heatmap provided as the output is very informative, but at the moment, a score that could indicate how stable the protein is on a scale going from "highly stable" to "not stable at all" would be fantastic.
Thank you in advance!

Hello,
Unfortunately, ThermoMPNN is trained only on relative stability scores, so it is not really designed to score the wild type protein on an absolute scale.
To do this kind of holistic evaluation, I would recommend using something with an empirical energy function like Rosetta or FoldX. Hope this helps!


[WIP] Initial version for oc-react-template

Note: CSS not yet supported

@matteofigus all the dependencies now follow this convention:

compileDependencies: modules needed to compile an oc-component based on the template, to be used with the cli on oc dev
devDependencies: template level development/testing modules

This goes with https://github.com/opentable/oc/pull/480


(#61) Sending notification to the scheduled user

For #61
Duplicate of #236 which was mistakenly merged. #236 had been reverted.
Changes include:

Using Laravel's default notification to send mail to interviewer set for the new round. This is only triggered if an application round is marked as confirmed.

Checked this PR :

[ ] For duplication.
[ ] For performance.
[ ] Usage of recursive methods, loops and algorithms.
[ ] For readability and understanding.
[ ] For functions with multiple responsibilities.
[ ] For scope of refactoring.
[ ] For coupling
[ ] For alternative approaches to the implementation.
[ ] For how the change impacts other parts of the system.
[ ] For validations. client side and server side validations are present.

For more insights on code reviews checkout our guidelines.

@bohraji we don't have queue enabled on Production. I think we will go for AWS SQS. Let me know if you have any preference on this.
To keep track of enabling queues, I've created an issue #247

@rathorevaibhav Amazon SQS is good, but will come at extra cost. To start with we can go with REDIS or Database based queues, and scale as need arises. Thoughts?

@bohraji I think Redis will require some installation effort on our primary servers. Not sure if it is already installed. If it is, that would be helpful.
In any case, I'll have to compare the installation effort first and then will proceed with whichever is more efficient for now.

@bohraji review feedback implemented. ready for review again.

@bohraji changed the service provider name to DatabaseServiceProvider. Let me know if there's anything else.


Add vfsStreamWrapper::unregister() to unregister the URL wrapper.

It would be nice to unregister the URL wrapper again, just for the sake of restoring the global state.
I ran into that issue because of a code base which has existing tests which do register a handler for vfs:// as well and I need a sane way to reset vfsStreamWrapper::$registered.

Thank you, just merged.


support for non https remotes

hey, thx for this awesome plugin ❤
we have a gitlab instance running without https -> don't ask!
here is the required change to support the http protocol (and keep the default https; works for all remotes)
config sample:
{
    "gitlens.remotes": [{ "domain": "gitlab.xxxx.at", "type": "GitLab", "protocol": "http" }]
}

let me know if there is something to change/adapt.

@hjanuschka Thanks for the contribution, I will get this merged into GitLens 6 which is likely coming out tomorrow!


Bulk remove users from group via CSV file

Is your feature request related to a problem? Please describe.
For educational institutions especially, there are a lot of movements between Google groups as students move between classes. However, Google Workspace currently only supports bulk addition of users to a group (which is group agnostic - that is, you can upload multiple users to multiple groups in a single file). GAM expands this feature since they allow bulk update or remove users from a group via a file, but this is not group agnostic; So there's no way to remove multiple users from multiple groups via a single file.
Describe the solution you'd like
I would like GAM to allow users removal from multiple groups in bulk, via a single command and a single file. So for example, gam update groups remove file <file name> would take a CSV file that for each line has the user email and the group email, and would remove the specified user from the specified group.
Ideally, I would love to see an option to move users from one group to another, so that GAM could take a file with user email, origin group, and destination group and transfer the users from one group to the other (for example gam update groups move file <file name> ) - but that may be an overly specific feature, although it would be amazing to have.
Describe alternatives you've considered
This issue could be solved by manually dividing the single file into multiple files, one for each group (or just doing it manually); but this is a cumbersome solution. Another solution would be writing a specialized code to do this, but this is also not an ideal solution.

This is simple with Advanced GAM
Make a CSV file RemoveGroups.csv with two columns: email,group
gam redirect stdout ./RemoveGroups.txt multiprocess redirect stderr stdout csv RemoveGroups.csv gam user "~email" delete group "~group"
Make a CSV file ChangeGroups.csv with three columns: email,origin,destination
gam redirect stdout ./ChangeGroupsRemove.txt multiprocess redirect stderr stdout csv ChangeGroups.csv gam user "~email" delete group "~origin"
gam redirect stdout ./ChangeGroupsAdd.txt multiprocess redirect stderr stdout csv ChangeGroups.csv gam user "~email" add group "~destination"
See: https://github.com/taers232c/GAMADV-XTD3/wiki/Users-Group-Membership https://github.com/taers232c/GAMADV-XTD3/wiki/Users-Group-Membership
Ross
Ross Scroggs
@.***

On Aug 26, 2023, at 8:22 AM, elorrtu5746 @.***> wrote:
Is your feature request related to a problem? Please describe.
For educational institutions especially, there are a lot of movements between Google groups as students move between classes. However, Google Workspace currently only supports bulk addition of users to a group (which is group agnostic - that is, you can upload multiple users to multiple groups in a single file). GAM expands this feature since they allow bulk update or remove users from a group via a file, but this is not group agnostic; So there's no way to remove multiple users from multiple groups via a single file.
Describe the solution you'd like
I would like GAM to allow users removal from multiple groups in bulk, via a single command and a single file. So for example, gam update groups remove file  would take a CSV file that for each line has the user email and the group email, and would remove the specified user from the specified group.
Ideally, I would love to see an option to move users from one group to another, so that GAM could take a file with user email, origin group, and destination group and transfer the users from one group to the other (for example gam update groups move file  ) - but that may be an overly specific feature, although it would be amazing to have.
Describe alternatives you've considered
This issue could be solved by manually dividing the single file into multiple files, one for each group (or just doing it manually); but this is a cumbersome solution. Another solution would be writing a specialized code to do this, but this is also not an ideal solution.
—
Reply to this email directly, view it on GitHub https://github.com/GAM-team/GAM/issues/1657, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCTYL3KR7GFBQYUQSQT25TXXIIERANCNFSM6AAAAAA37VEMFU.
You are receiving this because you are subscribed to this thread.


Make the property AutoAcknowledge available to be changed, and added function to stop consumers.

Fixes #23 and #24

Something failing here - please check the tests

Also, if you make separate PRs for each issue it will be easier to approve them one at a time

@wagslane Fixed the testing:

Also I deleted the other branch, so that's why I pushed both commits at the same time,  if that's a problem I can recreate the branch with the other code, it's not that much.

@wagslane any update?


calcHist() works for ushort but this is not documented

Documentation for OpenCV 3.0 states that calcHist() function takes as input arrays of depths CV_8U or CV_32F. But from source code  in OpenCV 3.1.0 it seems the function can be used for 16-bit arrays. I tried and it works just fine. It would be great if you documented the feature.
16-bit image sample_image.zip
#include "opencv2/highgui.hpp"
#include "opencv2/imgproc.hpp"
#include <opencv/cv.h>

using namespace cv;

int main(int argc, char *argv[])
{
        Mat img = imread("out1.tif", IMREAD_UNCHANGED);

        if (img.depth() == CV_16U)
        {
            int histSize = 2048;
            float range[] = {0, histSize};
            const float* histRange = { range };
            bool uniform = true; bool accumulate = false;

            Mat hist;
            calcHist( &img, 1, 0, Mat(), hist, 1, &histSize, &histRange, uniform, accumulate );

            double hmin, hmax;
            minMaxLoc(img, &hmin, &hmax);

            //plot the histogram
            Mat canvas = Mat::zeros(200, histSize, CV_8UC1);
            for (int j = 0, rows = canvas.rows; j < histSize-1; j++)
                line(canvas, Point(j, rows), Point(j, rows - (hist.at<float>(j) * rows/hmax)), Scalar(255,0,0));

            imshow("hist", canvas);
            waitKey(0);
        }
        else
            return 1;

        return 0;
}

This issues should be posted to the main repository issue tracker.


Unable to resove module util in react native ......please help to solve this issue!!

@yogeshmoradiya121 util is a native Node.js module and is not available in the browser.
You might be able to solve your issue by installing the util npm package npm install --save util but I cannot promise that it will work.

@yogeshmoradiya121  did you fix it?
@dlebech  please note, this question is not related to browser apps, react-native is for building native iOS and Android apps, rather than browser apps.
Your point is probably still valid of course:  are you saying that this use case would not be supported by this library?
Would you recommend people fork this library and adapt as necessary for usage in different contexts?
a scan of the code suggests the use of util seemed to be in 1 logging line, I'm not sure what other issues there are likely to be.

@Jon007 Thanks for the clarification. We created the library to be used with the Node runtime. I'm not familiar with react-native, but it seems that it is not generally compatible with Node and can indeed be considered a different platform/runtime from Node, so your assessment is correct: supporting other JS-based platforms would probably require a fork. For example, even if we removed the util package, I'm not sure that would be enough since you would also need the crypto and http packages of Node to perform the requests.
There might be another option though. Forgive me if you already know this, but our library is not the official WooCommerce-for-Node library (it's this one). We (at Conversio) were just faster to market so to speak than the WooCommerce team which is why we got the woocommerce name for npm.
I don't know if the official Node client will work better for a react-native use case, but at least they are probably more likely to create support for react-native.


Scoller.tableView not always displayed on first viewing

On 'some' FilterController->Scroller->TableSelection setups the Scroller.tableView does not display until a window resize event occurs.
When the resize occurs, the table is drawn but the error listed below is also generated for each table row.
Just noticing that I have notes in the filterController setup regarding two styles of inner tableView setup with one not honouring popup height/width. Perhaps this is related or helps with the investigation.
If you require/want a test setup, I can package the app up for you.
Thanks
Joel
Setup:
            name: 'filterController',
            class: 'foam.u2.ViewSpec',
            factory: function() {
                var fc = this.FilterController.create({
                    textSearch: false,
                    allowAddingFilters: false,
                    data$: this.filteredDAO$,
                    tableView: this.Scroller.create({
                        of: this.Fieldworker,
                        data$: this.filteredDAO$,
                        //
                        // NOTE: this creation style ignores popup height/width
                        // while the next honours the popup height/width
                        // tableView: this.TableSelection.create({
                        //     of: this.of,
                        //     data: this.dao,
                        //     columns: this.tableColumns
                        // }, this)
                        //
                        tableView: {
                            class: 'foam.u2.TableSelection',
                            view: {
                                class: 'foam.u2.TableView',
                                columns: this.tableColumns
                            },
                            selectionQuery$: this.fieldworkerSelectionQuery$
                        }
                    }, this)
                }, this);
                return fc;
            }

Error for each Table row on window resize:

Uncaught TypeError: Cannot read property 'getBoundingClientRect' of null
at Object.onResize (foam-bin.js:28066)
at l (foam-bin.js:5358)
at frameFired (foam-bin.js:6933)

translates to:
  listeners: [
    {
      name: 'onResize',
      isFramed: true,
      code: function() {
        // Determine the height of the table's space.
        var height = this.el().getBoundingClientRect().height;    <-----
        this.scrollHeight_ = height;
        this.scrollExtent_ = Math.floor(height / this.rowHeight);
      }
    },

If you require/want a test setup, I can package the app up for you.

That would be helpful.
It appears that onResize() is being called either before it's been added to
the DOM or after it's been removed. More defensive code would just return
from onResize() if this.el() is null, but it would be good to figure out
why this is happening.
On 13 March 2017 at 12:21, Joel Hughes<EMAIL_ADDRESS>wrote:

On 'some' FilterController->Scroller->TableSelection setups the
Scroller.tableView does not display until a window resize event occurs.
When the resize occurs, the table is drawn but the error listed below is
also generated for each table row.
Just noticing that I have notes in the filterController setup regarding
two styles of inner tableView setup with one not honouring popup
height/width. Perhaps this is related or helps with the investigation.
If you require/want a test setup, I can package the app up for you.
Thanks
Joel
Setup:
        name: 'filterController',
        class: 'foam.u2.ViewSpec',
        factory: function() {
            var fc = this.FilterController.create({
                textSearch: false,
                allowAddingFilters: false,
                data$: this.filteredDAO$,
                tableView: this.Scroller.create({
                    of: this.Fieldworker,
                    data$: this.filteredDAO$,
                    //
                    // NOTE: this creation style ignores popup height/width
                    // while the next honours the popup height/width
                    // tableView: this.TableSelection.create({
                    //     of: this.of,
                    //     data: this.dao,
                    //     columns: this.tableColumns
                    // }, this)
                    //
                    tableView: {
                        class: 'foam.u2.TableSelection',
                        view: {
                            class: 'foam.u2.TableView',
                            columns: this.tableColumns
                        },
                        selectionQuery$: this.fieldworkerSelectionQuery$
                    }
                }, this)
            }, this);
            return fc;
        }

Error for each Table row on window resize:
Uncaught TypeError: Cannot read property 'getBoundingClientRect' of null
at Object.onResize (foam-bin.js:28066)
at l (foam-bin.js:5358)
at frameFired (foam-bin.js:6933)
translates to:
listeners: [
{
name: 'onResize',
isFramed: true,
code: function() {
// Determine the height of the table's space.
var height = this.el().getBoundingClientRect().height;    <-----
this.scrollHeight_ = height;
this.scrollExtent_ = Math.floor(height / this.rowHeight);
}
},
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/foam-framework/foam2/issues/257, or mute the thread
https://github.com/notifications/unsubscribe-auth/AHHXs7eMgcnD19PkXEegKG1by5yPaFLwks5rlWz8gaJpZM4Mbe7u
.


add hint regarding style and script tag locations in layout components

Description (required)
This PR adds a paragraph to the layout component page, explaining how <script> and <style> tags should be included in a component containing an <html> shell.
Related issues & labels (optional)

Closes #8603
Suggested label: add new content

Discord: lukets4328

Hello! Thank you for opening your first PR to Astro’s Docs! 🎉
Here’s what will happen next:

Our GitHub bots will run to check your changes.
If they spot any broken links you will see some error messages on this PR.
Don’t hesitate to ask any questions if you’re not sure what these mean!

In a few minutes, you’ll be able to see a preview of your changes on Vercel 🥳

One or more of our maintainers will take a look and may ask you to make changes.
We try to be responsive, but don’t worry if this takes a few days.

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

en
basics/layouts.mdx
Source changed, localizations will be marked as outdated.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


[raylib:wasm32-emscripten] build failure

Host Environment

OS: Windows
Compiler: Latest EMSDK, Visual Studio 2019, Ninja 1.10.2.git.kitware.jobserver-1

Package: raylib:wasm32-emscripten
Vcpkg version: 2021-08-03-1a6d67b8150e94b7f473c6ab1c0cfdd794316a64
To Reproduce
Steps to reproduce the behavior:
./vcpkg install raylib:wasm32-emscripten
Failure logs
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:127 (message):
Command failed: ninja -v
Working Directory: C:/Users/chknm/TestingGrounds/vcpkg/buildtrees/raylib/wasm32-emscripten-rel/vcpkg-parallel-configure
Error code: 1
See logs for more information:
C:\Users\chknm\TestingGrounds\vcpkg\buildtrees\raylib\config-wasm32-emscripten-out.log
Call Stack (most recent call first):
scripts/cmake/vcpkg_configure_cmake.cmake:356 (vcpkg_execute_required_process)
ports/raylib/portfile.cmake:39 (vcpkg_configure_cmake)
scripts/ports.cmake:141 (include)
config-wasm32-emscripten-out.log

What is the image it just says uploading image

What is the image it just says uploading image

Ah sorry, that's a wrong operation, is this issue still exist?

I do not know I have changed projects and computers since then.
On Fri, Jan 28, 2022, 9:47 AM Jack·Boos·Yu @.***> wrote:

What is the image it just says uploading image
Ah sorry, that's a wrong operation, is this issue still exist?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vcpkg/issues/19515#issuecomment-1024459509,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANBYAT7NTYB6A43OVAVEFG3UYLJEXANCNFSM5CANDL4Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

Okay, please reopen this issue if it still bother you.

I have checked and the error still persists. The issue should be reopened.

Could NOT find X11 (missing: X11_X11_LIB)

X11 is not supported on Windows in vcpkg now.

This issue hasn’t been updated in 3 month, if it is still an issue, please reopen this issue.


Empty list message

Hi there
I am having a problem with the message shown when a list is empty.
Although i have set the contentsWhenEmpty parameter,
it is ignored and the default message "Empty list " is shown,
when either of the two list is empty.
It only shows the requested message,  when both of the list are empty.
I have tried this on Android and linux, with the same result.
Thanks

Having the same issue. @arionargo Did you find any solutions to this?

Unfortunately, i haven't found any solution yet.
Although, by looking the code in lib/drag_and_drop_list.dart at line 161,
this should be working

@arionargo I just fixed this issue. I was earlier setting the contentsWhenEmpty to the DragAndDropLists. Instead, you have to add the contentsWhenEmpty to DragAndDropList
(see the slight diff; It's setting to the single DragAndDropList and not DragAndDropLists)

thanks @pampas93 , i will try it and let you know. An idea what is the difference between using it(contentsWhenEmpty) at DragAndDropLists and DragAndDropList ?

DragAndDropLists is the container for DragAndDropList. And digging into the lib code, contentsWhenEmpty within DragAndDropList class made sense

@arionargo I just fixed this issue. I was earlier setting the contentsWhenEmpty to the DragAndDropLists. Instead, you have to add the contentsWhenEmpty to DragAndDropList
(see the slight diff; It's setting to the single DragAndDropList and not DragAndDropLists)

Confirming that this is the correct solution, thanks for figuring this out!


Oracle Linux 8.9 GA

Link to product page on endoflife.date
https://endoflife.date/oracle-linux
Could you update the page to reflect new release?
What is the source website for the product and for its version information?
https://docs.oracle.com/en/operating-systems/oracle-linux/8/relnotes8.9/#Oracle-Linux-8
Additional context
Add any other context or screenshots about the incorrect details.

Hello @cashfit,
The data comes from DistroWatch and the version 8.9 does not appear (yet) in the page, it's the reason why EOL does not mention this version.
The page has been updated on 2023-12-22 and the most recent version announced is

2023-11-17: Distribution Release: Oracle Linux 9.3

Don't know the workflow on DistroWatch but it may appear in the coming days and so be reflected here :confused: ?

Thanks for notifying us @cashfit, and thanks for those information @romainx.
Just like it was done for 8.8 I manually updated the page to update the latest 8 to 8.9 in #4401 (used https://blogs.oracle.com/linux/post/oracle-linux-8-update-9 as the release date).
Unfortunately DistroWatch does not always mention all versions. But some automation based on it is still useful as it allows us to catch at least some versions.


Add completeLoginViaTransaction and beginLoginViaTransaction functions in Frontend Client Package

Ported over completeLoginViaTransaction and beginLoginViaTransaction functions from notifi-react-hooks package into notifi-frontend-client
To Do:

Remove walletPublicKey in configuration and update dappAddress name to tenantName

Looks great! Just a few points of feedback:

Please remove .DS_Store (add it to .gitignore?)
In general, we should expose as much of the underlying graphql input params as we can
For the SDK, we need much stricter type safety than we do in the Notifi Hub.

Thank you for the comments! I removed DS_Store using these steps: https://gist.github.com/lohenyumnam/2b127b9c3d1435dc12a33613c44e6308 and added it to the .gitignore file. I also reviewed what the DS_Store file: https://buildthis.com/ds_store-files-and-why-you-should-know-about-them/.


How to pretrain in imagenet1k?

Your work are awesome, the gratest part is pretrained in imagenet-1k that could provide a good basis for transfer learning in other task. However, I couldn't reproduce the training procedure only by Table 17 ,I'm wondering if you could provide the code for pretraining in imagenet-1k. Thank you for your response beforehand.

Sure. Thanks for your attention to our work. For the pretrain code, you can refer to https://github.com/VCIP-RGBD/RGBD-Pretrain .

I don't notice you had provide the information in readme, I apologize for my mistake. I appreciate your message, thank you a lot.

Never mind 😃 . If you have any other questions about our work, feel free to contact me.


#CLI 2.9.1 Upgrade on Mac in Atom

Hello,  i have problem with CLI, when i open Atom and PlatformIO i see on Home Screen CLI 2.9.0 and button Upgrade NOW!. And if upgrade i see in terminal:
You're up-to-date! PlatformIO 2.9.1 is currently the newest version available.
My hardware:
OS X 10.11.5 Beta
Atom 1.7.3
Where is problem ?
I sorry for my English.

Looks like you have a few PlatformIO CLI installed on your machine. Please open system Terminal and run pip uninstall platformio.

Thanks it's work now :) i'm uninstall on brew platformio.
Now is close inssue.


look into script type="ld+json"

It's just schema.org in json pyjamas, so it ought to be easy to add this.

It's been added for a while now, closing.


Add send and receive commands

# send a message:
$ mq send /some-queue 'Message!;

# receive a message:
$ mq receive /some-queue
> Message!

This makes manual testing of individual parts much easier!


Confirm that Google Realtime API supports unauthenticated users

It's not clear from quick glances whether or not Google's Realtime API that we want to build on top of supports unauthenticated users. We want users to be able to:

type in "uxgraph.io" in Incognito mode and immediately be able to create a new uxgraph without logging into Google
if the unauthenticated user remembers the URL he had, he should be able to open up the same later
if the unauthenticated user has the URL to someone else's uxgraph, he should be able to open that up too (if he has permission to view)

Once you confirm that the Realtime API supports these use cases, this issue is done.
In the event that Realtime API does not support these use cases, we'll have to investigate switching to another system like Firebase...

Going to continue to investigate, but I'll leave this here for now...

You can use the API in in-memory mode while unauthenticated, which means the collaboration won't work, but you can still create a document, modify it, and serialize it out. Which means we'd have to have our own server for storing this data. This quickstart has an example.
So we can support use cases 1 and 2. We can also support some version of 3, if we're cool with it just being a view-only snapshot. But this would mean we'd have to store snapshots of all shared docs on our own servers.

Does case 3 even make sense? How can an unauthenticated user receive permission to view a doc?

The owner of the uxgraph would have to make it public to everyone

Yeah, if the doc is public, then unauthorized users should be able to see it too.

Looks like all @tessa3/owners are satisfied with Realtime's "in-memory mode" support for unauthenticated users so I'm closing this issue.


Zero length array not supported in other programming languages.

Find way to replace zero length array of uint8_t in struct RE and single element array of char in struct SIZED_STRING to uint8_t* and char*.
That structs passed as parameter when invoking modules functions.

Can you provide more details about what do you want to achieve? Do you want to write modules in other programming languages and link them into YARA?


Use dasherized name for get-select-options in fixtable-grid.hbs

Currently can get an error like this:

WARNING: Attempted to lookup "helper:getSelectOptions" which was not found. In previous versions of ember-resolver, a bug would have caused the module at "/helpers/get-select-options" to be returned for this camel case helper name. This has been fixed. Use the dasherized name to resolve the module that would have been returned in previous versions.

Should fix this by dasherizing the name in fixtable-grid.hbs.

Fixed in commit 10d9f329e19d199d42936980fc1160924932c2e6


avoid and/or silence common warnings

A handful of pvlib functions often cause spit out runtime warnings for things like dividing by 0 or comparing the size of a nan value to a float. We should avoid or silence the warnings that we know are ok. They also make the test output a pain to read.
Here's a possibly incomplete list:

[ ] irradiance.perez divide by 0
[ ] irradiance.perez comparison with nan
[ ] clearsky.simplified_solis comparison with nan
[ ] pvsystem.sapm divide by 0 in log
[ ] pvsystem.ashraeiam comparison with nan
[ ] pvsystem.physicaliam comparison with nan
[ ] pvsystem.adrinverter comparison with nan
[ ] test_weather_irradiance_input irradiance parameter

I have some code ready for addressing the warnings in the perez model implementation. I'll post it soon.

I think the readability of the code suffers (in terms of being able to quickly see what math is being done) with some of the changes you propose.  I sometimes disable certain warnings at the start of my scripts when they get on my nerves.  Could that be adequate for other users?  This suggestion could be put in the doc string for functions that are particularly annoying.


Show values on a summary/legend/chart

SQL Operations Studio Version:

Steps to Reproduce:

When exporting the chart to an image, maybe should show the values on summary/legend/plot

Cleaning up old issues.  I'm going to close this as "won't fix" for now since we don't plan to address in the next few milestones.


Unsoundness when mutable objects are used (false has type true)

I think this is due to a problem with unification: (Tested on v0.80.0)
const a = {x: true};         // A mutable object.
const b:{x: true} = a;       // Works
const c:{x: boolean} = a;    // Also works...
c.x = false;                 // Uh oh. This can't be good...
const derp:true = b.x;       // Egads! false:true

Try it
I was trying to explain how in a Hindley-Milner style type inference system, subsequent type annotations will influence the types inferred earlier through unification, but it wasn't working as I expected. Ordering shouldn't matter of course, but changing the order of b and c above does produce a type error as would be expected of a sound type system.
#5895 may or may not be more than tangentially related...

Just to be clear, this yields coerce:
// @flow
function coerce<T, U>(t: T): U {
  class Unique {}
  type Box<E> = {x: E};
  const box = {x: new Unique()};
  const xob = (box: Box<U | Unique>);
  (box: Box<mixed>);
  box.x = t;
  if (xob.x instanceof Unique) throw new Error("Unreachable.");
  return xob.x;
}

(Thanks for posting.)


upon inserting a new row, sometimes errors will be thrown saying duplicate primary keys exist

As title says. Funnily enough, on exactly the same code base, this does not not happen to me but does happen to a colleague

Reopen if this is still an issue, with more details such as code to reproduce.


[Persistence] Update pocket/smt to have support for a Merkle Sum Trie

Objective
Update the SMT to (optionally) support "Merkle Sum" tree behaviour
Origin Document

Relay Chains & GeoZones presentation
Relay Mining Document
Plasma Core Merkle Sum Tree

Goals

Enable the core utility of M3 in pocket-v1

Deliverable

[ ] Add a configuration that enables the SMT to behave as a Merkle sum tree per Plasma Core's specification
[ ] Tests to prove the integrity of the new changes
[ ] Benchmarks to show the performance impact
[ ] Update the documentation where needed

Non-goals / Non-deliverables

Improve the performance of the SMT (memory and compute-wise)
Integrate the changes into pocket-core/pocket

Creator: @Olshansk
Co-Owners: @h5law

Completed in #845


Querying Collections fails when upgrading JDK

Querying a collection works on JDK 1.8
		FilterDescriptor qFilter = alias(FilterDescriptor.class, "filter");
		List<FilterDescriptor> filters;
                     ...
		from($(qFilter), filters).where($(qFilter.getField())
					.eq("USER_ID"))
					.singleResult($(qFilter.getValue()));

but fails on JDK11 with the following exception:

java.lang.RuntimeException: java.lang.UnsupportedOperationException

at jdk.compiler/com.sun.tools.javac.api.JavacTaskImpl.handleExceptions(JavacTaskImpl.java:163)
com.mysema.codegen.JDKEvaluatorFactory.compile(JDKEvaluatorFactory.java:73) ~[codegen-0.6.8.jar:?]
Caused by: java.lang.UnsupportedOperationException
at com.mysema.codegen.MemFileManager.getJavaFileForInput(MemFileManager.java:91) ~[codegen-0.6.8.jar:?]

It seems getJavaFileForInput actually throws an exception.
querydsl version is 3.7.4
It might be linked to this issue:
https://github.com/cglib/cglib/issues/129
Is querydsl locked in with JDK version?

Is querydsl codegen supported on JDK 11 or even JDK 15?

QueryDSL 3.7.4 is very old no longer maintained. QueryDSL 4.4.0 supports Java 8 and Java 9+ with some modifications. QueryDSL Collections 4.4.0 uses a really old cglib version and only works when using a JDK and not when using a JRE. QueryDSL 5 will improve support for Java 9+ but is currrently unreleased.

Hi @jwgmeligmeyling
I’m using Querydsl Collections 4.4 with Spring Boot Data Key Value (or at least trying).
You mean I could try to use another version of cglib in my project, or is there at the moment no real solution using Querydsl Collections from JDK 11 onwards until Querydsl 5 is released?

JDK 11 is pretty much unsupported for QueryDSL 4.4.x but most users have been able to get it to work with minor tweaks like updating a couple dependencies.
QueryDSL 5.x compiles and tests against JDK 11 just fine. I am not sure about JDK 15 yet, it simply hasn't been tested, but as the changes between 11 and 15 are much less drastic than between 8 and 9 I suppose it will be fine (except maybe codegen that might struggle a little with Records).

Thanks for the feedback @jwgmeligmeyling !

@jwgmeligmeyling regarding minor tweaks like updating a couple dependencies., if you know which one, can you provide some info? Because i can't use Collections due to the same error, I use Java 11 + Spring Boot 2.3.9 and tried with QueryDSL 4.2.2, 4.4.0, 4.3.1, Whithout success.  Aslo i use apt-maven-plugin for annotation processing.

For Java 11 you'll need Querydsl 5.0.0.M1 minimally.

@jwgmeligmeyling for java 11 i receive another
Compilation of public class Q_0911015504_1275614662_1275614662_1195259493_01114099497 failed  exeption,/Q_0911015504_1275614662_1275614662_1195259493_01114099497.java:3: error: package com.mypackage.dto does not exist, so i suppose in some reason compilator doesn't see packages
Mb you have working pom file for java 11 and qdsl?

Querydsl compiles against Java 11 😉  So any example in the Querydsl project is a working example.
Perhaps your combining old and new versions for querydsl-core, querydsl-codegen, querydsl-collections, querydsl-jpa, and/or querydsl-apt`?

@jwgmeligmeyling  i use only querydsl-jpa, also same version of querydsl-apt. Q calsses are generated fine. The problem is - build CollQueryFactory.from(some, some).where(predicate).fetch() i receive exeption that i mentioned above. If i use it without where, all works fine. Also i use lightweight tomcat, is it can be issue?

Could you share the full stacktrace and the output of mvn dependency:tree?

Trace.log - i hope this is fine for you?
Regarding "classpath mode" what does it mean ? I use embeded tomcat, jar instead of war builded.
mvn dependency tree is too big but, [INFO] +- cglib:cglib:jar:3.3.0:compile
[INFO] |  +- com.querydsl:querydsl-jpa:jar:5.0.0.M1:compile
[INFO] |  |  - com.querydsl:querydsl-core:jar:5.0.0.M1:compile
[INFO] |  |     - com.mysema.commons:mysema-commons-lang:jar:0.2.4:compile
[INFO] |  +- com.querydsl:querydsl-collections:jar:5.0.0.M1:compile
[INFO] |  |  +- com.querydsl:querydsl-codegen:jar:5.0.0.M1:compile
[INFO] |  |  |  +- com.querydsl:codegen-utils:jar:5.0.0.M1:compile
[INFO] |  |  |  |  - org.eclipse.jdt:ecj:jar:3.25.0:compile
[INFO] |  |  |  +- javax.inject:javax.inject:jar:1:compile
[INFO] |  |  |  - io.github.classgraph:classgraph:jar:4.8.108:compile
[INFO] |  |  - com.querydsl:querydsl-apt:jar:5.0.0.M1:compile
[INFO] |  +- com.github.vineey:rsql-querydsl-all:jar:2.0.0.RELEASE:compile
[INFO] |  |  +- com.github.vineey:rsql-api-all:jar:2.0.0.RELEASE:compile
[INFO] |  |  |  +- com.github.vineey:rsql-api-core:jar:2.0.0.RELEASE:compile
[INFO] |  |  |  +- com.github.vineey:rsql-api-select:jar:2.0.0.RELEASE:compile
[INFO] |  |  |  +- com.github.vineey:rsql-api-filter:jar:2.0.0.RELEASE:compile
[INFO] |  |  |  +- com.github.vineey:rsql-api-sort:jar:2.0.0.RELEASE:compile
[INFO] |  |  |  - com.github.vineey:rsql-api-page:jar:2.0.0.RELEASE:compile
[INFO] |  |  +- com.github.vineey:rsql-querydsl-page:jar:2.0.0.RELEASE:compile
[INFO] |  |  +- com.github.vineey:rsql-querydsl-select:jar:2.0.0.RELEASE:compile
[INFO] |  |  +- com.github.vineey:rsql-querydsl-filter:jar:2.0.0.RELEASE:compile
[INFO] |  |  - com.github.vineey:rsql-querydsl-sort:jar:2.0.0.RELEASE:compile
[INFO] |  |  +- org.springframework.data:spring-data-jpa:jar:2.3.9.RELEASE:compile

Compilation of public class Q_0911015504_1275614662_1275614662_1195259493_01114099497 failed exeption,/Q_0911015504_1275614662_1275614662_1195259493_01114099497.java:3: error: package com.mypackage.dto does not exist, so i suppose in some reason compilator doesn't see packages is caused by another issue than the issue reported here.
Unfortunately the issue is hard to reproduce without more context. Perhaps you could try recreating the issue in the Querydsl integration test domain? Existing tests for querydsl collections can be found in: https://github.com/querydsl/querydsl/tree/master/querydsl-collections/src/test/java/com/querydsl/collections


Add @dibyom as an OWNER! 🙌

Changes
@dibyom has been contributing significant work including variable path
resource substitution https://github.com/tektoncd/pipeline/pull/877 and
his continued work on one of our oldest and most sought after features:
conditional execution (https://github.com/tektoncd/pipeline/pull/27)!
He has also reviewed 44+ PRs in the Pipelines repo:
https://tekton.devstats.cd.foundation/d/46/pr-reviews-by-contributor?orgId=1&var-period=d&var-repo_name=tektoncd%2Fpipeline&var-reviewers="dibyom"
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[n/a] Includes tests (if functionality changed/added)
[n/a] Includes docs (if user facing)
[x] Commit messages follow commit message best practices

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task and TaskRun to build and release this image

Adding a hold here so the other owners can review ( @ImJasonH @vdemeester @afrittoli @dlorenc @abayer ):
/hold

/lgtm

/lgtm

/lgtm
/meow space

That is a very creepy cat. I apologize.

/lgtm
:tada:

/hold remove

/hold cancel

/test pull-tekton-pipeline-unit-tests


PUT operations in OAM layer should return 204 (Implementation)

Problem description
For PUT request in OAM layer , after successful updation , server is returning 200 OK as the ResponseCode.
As per the specification and generic concepts, this is not correct.
Proposed solution
Since the PUT operation doesn't return any value in the ResponseBody , the ResponseCode should be 204 NO_CONTENT.

FIxed as a part of the pull request#31


record identifying information with each experiment

This makes it easier to keep track of experiment configuration and parameters automatically. Note that "git diff HEAD" will include both staged and unstaged changes.
Example:
~/dqn/test-runs/version_info_10-02-07-56-18_0p00025_0p99$ cat cmdline
./run_nature.py --experiment-prefix /home/dsj/dqn/test-runs/version_info --rom breakout
~/dqn/test-runs/version_info_10-02-07-56-18_0p00025_0p99$ cat git-rev-parse
71061e912c138c1a00bac65e41d15634e14f0585
~/dqn/test-runs/version_info_10-02-07-56-18_0p00025_0p99$ cat git-diff
diff --git a/deep_q_rl/ale_agent.py b/deep_q_rl/ale_agent.py
index fc3b04b..ea0f5a8 100755
--- a/deep_q_rl/ale_agent.py
+++ b/deep_q_rl/ale_agent.py
@@ -57,8 +57,6 @@ class NeuralAgent(object):
             f.write('\n')
         with open(os.path.join(self.exp_dir, 'git-rev-parse'), 'w') as f:
             f.write(subprocess.check_output(['git', 'rev-parse', 'HEAD']))
-        with open(os.path.join(self.exp_dir, 'git-status'), 'w') as f:
-            f.write(subprocess.check_output(['git', 'status']))
         with open(os.path.join(self.exp_dir, 'git-diff'), 'w') as f:
             f.write(subprocess.check_output(['git', 'diff', 'HEAD']))

This is a good idea, but it's duplicating some of the functionality from PR #44 against the dev branch.
I've been trying to confine code changes that involve significant reorganizations or changes to the interface to the dev branch, only updating the master branch directly if there is an important bug-fix or other non-disruptive change.    That's fallen by the wayside a bit as there have been a stream of critical improvements going directly into master, and I haven't had the resources to vet and merge the changes in dev.   I'll merge PR #44 as soon as the author fixes some minor issues with it.   Maybe at that point you could re-target this against dev with just the git version stuff?


chore: prepare v6.0.0

Description
This PR prepares the codebase for the upcoming v6.0.0 release. The changes included are:

Bumped the go module from github.com/desmos-labs/desmos/v5 to github.com/desmos-labs/desmos/v6.
Added missing changeset entries, and fixed the messages on existing ones.
Updated the on-chain upgrade workflow to test the upgrade from v5.2.0 to v6.0.0 using the chain state exported from the testnet

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[x] included the correct type prefix in the PR title
[ ] added ! to the type prefix if API or client breaking change
[x] targeted the correct branch (see PR Targeting)
[ ] provided a link to the relevant issue or specification
[ ] followed the guidelines for building modules
[ ] included the necessary unit and integration tests
[ ] added a changelog entry to CHANGELOG.md
[x] included comments for documenting Go code
[x] updated the relevant documentation or specification
[x] reviewed "Files changed" and left comments if necessary
[x] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[x] confirmed the correct type prefix in the PR title
[x] confirmed ! in the type prefix if API or client breaking change
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[x] manually tested (if applicable)

Test on chain upgrade CI is failed because of timetout. Tested manually and it works properly. I think we should increase the timeout time to 1 hour or longer.

@dadamu I have no idea why the workflow is failing. I have incremented this to 1 hour but it still has problems. I would guess something related to GitHub for some reason. Anyway, I would like to merge this if you approve it

@RiccardoM Overall looks great, I think it is ready to go after this missing upgrade handler is added:
https://github.com/desmos-labs/desmos/pull/1212#discussion_r1299682669

@dadamu Thank, I missed that comment. I have now merged your code using #1216 :pray:


Add contact info for SpankChain

<EMAIL_ADDRESS>is indeed your one-stop shop for all security outreach :dancer:
Thanks for doing this, great idea!

Thanks! I am going to merge this myself, as it looks like you backported a fork of our repo.

https://github.com/trailofbits/blockchain-security-contacts/commit/173a7a9147f5c9e9f1c86f08db2a49df88d61fb4


Fix scala rules if scala compiler plugins provided

In BUILD.bazel if you have:
scala_library(
name = "testapp",
plugins = ["@org_spire_math_kind_projector_2_12"]
)
It fails with following error:
File "..../io_bazel_rules_scala/scala/private/rule_impls.bzl", line 123, in paths.append
p.scala.outputs.jar
'struct' object has no attribute 'jar'
Seems like jar attributed has been changed to jars some time ago. This pull request fixes this problem.

I signed it!

+1 to a test.
@johnynek this is a bit aside to this PR as my comment is bigger in scope but I'd really like to kill the scala attr

+1 to killng the scala attr. Maybe bazel has finally added all the things and fixed all the bugs making that possible.
The last time I tried, we could not.

Guys, I would like to add tests for it as well, but I have no idea where to start. Could you please give me an example of your tests?

The tests are here:
I thought we already may have had a test with a plug-in, e.g. wart remover, but I couldn’t find it on my phone.
Maybe try kind projector as you suggest above and make sure it builds and your code compiles?

@johnynek Added tests, please check. Most likely you didn't see this issue, because you use maven_jar rule, not scala_maven_import_external. I've noticed that scala_maven_import_external rules triggers the path with hassattr['scala']

//cc @ittaiz

btw, if you'd rather I do this check please let me know and I'll do it

Sure, let me check

Yep, it fails with:
ERROR: Analysis of target '//test/src/main/scala/scalarules/test/compiler_plugin:compiler_plugin' failed;
build aborted: Analysis of target '//test/src/main/scala/scalarules/test/compiler_plugin:compiler_plugin' failed; build aborted

@ittaiz So I removed the e2e test. Please, let me know if any other help needed.

@ittaiz , @johnynek any updates on when could it be released?

Sorry for the review latency here. I really appreciate your help.

Thanks!

Indeed.
Thanks!
On Sat, 4 May 2019 at 11:43 Dmitry Naumenko<EMAIL_ADDRESS>wrote:

Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_scala/pull/719#issuecomment-489307589,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAKQQFYXFHF7QLICWYVPX3TPTVEBZANCNFSM4HAEOVMA
.


rpmlib: Dependency sets (i.e. comparing RPM versions)

This issue tracks support for using rpmlib to compare RPM versions, a.k.a. dependency sets. Presently no support is provided.

C API: https://docs.fedoraproject.org/ro/Fedora_Draft_Documentation/0.1/html/RPM_Guide/ch15s04s02.html
Python API: https://docs.fedoraproject.org/ro/Fedora_Draft_Documentation/0.1/html/RPM_Guide/ch16s05.html

RPM has all sorts of complicated and fancy rules for comparing package versions and determining which ones are newer. Quoth the Fedora RPM Guide:

You could compare the version numbers directly, calling headerGetEntry to get the version and release tags, converting these strings to numbers and then comparing, but this would cause problems. The custom comparison is not as exact as the code in this section, especially since many packages have version numbers that are not true numbers, such as 1.12.4, with one too many decimal points. This makes the comparisons harder. In addition, there is more than just the version number to take into account. You need to deal with the Epoch value, as well as the release, too.

So, tl;dr: if we ever want to compare RPM versions, we should really use rpmlib's comparison logic to do it.

~~~We really gotta get #11 fixed!~~~

Update: #11 is fixed, and #19 added transaction set support.

There are two levels of version comparison: rpmvercmp() and rpmds* dependency sets.
Note that rpmvercmp() needs to be applied to each element in the tuple (E,V,R) iteratively. rpmvercmp is NOT a means to compare two EVR strings.
The Python bindings are overly complex and encumbered with ancient politics having to do with the "mysterious epoch" and missing values, which you likely should avoid these days in new bindings.


extended view with session for field visibility stage

What changes are proposed in this pull request?
Note: Last thing I am doing is cleaning up some of the _cls, cls, field_names, etc to be more consistent

[x] Core extended view functionality
[x] Remove all SelectedPaths code now that we use session atom purely with ExcludedPaths
[x] bugs

[x] Video broken some fields stay highlighted
[x] Changing dataset and opening the field visibility
[x] Check group dataset
[x] bookmark icon issue
[x] Include nested fields in Filter Rule tab issue
[x] unit tests fixes

How is this patch tested? If it is not, please explain why.
(Details)
Release Notes
Is this a user-facing change that should be mentioned in the release notes?

[ ] No. You can skip the rest of this section.
[ ] Yes. Give a description of this change to be included in the release
notes for FiftyOne users.

(Details in 1-2 sentences. You can just refer to another PR with a description
if this PR is part of a larger change.)
What areas of FiftyOne does this PR affect?

[ ] App: FiftyOne application changes
[ ] Build: Build and test infrastructure changes
[ ] Core: Core fiftyone Python library changes
[ ] Documentation: FiftyOne documentation changes
[ ] Other

Thank you @benjaminpkane! I'll pull the branch and do some testing on my side and merge 🎉


Decode-Url Function

Q
A

Bug fix?
no

New feature?
yes

New sample?
yes

Related issues?

What's in this Pull Request?
Decode-Url Function to decode a url encoded string.

Hey @P3N-101 i was looking forward to process this pull request today. I hope that you'll resubmit another one?
Let me know if I can help in any way


Can't install on my mac

Can't install with this log:
/usr/local/bin/cordova-res -> /usr/local/lib/node_modules/cordova-res/bin/cordova-res

><EMAIL_ADDRESS>install /usr/local/lib/node_modules/cordova-res/node_modules/sharp
> (node install/libvips && node install/dll-copy && prebuild-install) || (node-gyp rebuild && node install/dll-copy)

info sharp Downloading https://github.com/lovell/sharp-libvips/releases/download/v8.7.4/libvips-8.7.4-darwin-x64.tar.gz
ERR! sharp EACCES: permission denied, open '/usr/local/lib/node_modules/cordova-res/node_modules/sharp/16245-libvips-8.7.4-darwin-x64.tar.gz'
info sharp Attempting to build from source via node-gyp but this may fail due to the above error
info sharp Please see https://sharp.pixelplumbing.com/page/install for required dependencies
gyp WARN EACCES user "root" does not have permission to access the dev dir "/Users/mauro/.node-gyp/10.16.0"
gyp WARN EACCES attempting to reinstall using temporary dev dir "/usr/local/lib/node_modules/cordova-res/node_modules/sharp/.node-gyp"
gyp WARN install got an error, rolling back install
gyp WARN install got an error, rolling back install
gyp ERR! configure error
gyp ERR! stack Error: EACCES: permission denied, mkdir '/usr/local/lib/node_modules/cordova-res/node_modules/sharp/.node-gyp'
gyp ERR! System Darwin 18.6.0
gyp ERR! command "/usr/local/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /usr/local/lib/node_modules/cordova-res/node_modules/sharp
gyp ERR! node -v v10.16.0
gyp ERR! node-gyp -v v3.8.0
gyp ERR! not ok
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `(node install/libvips && node install/dll-copy && prebuild-install) || (node-gyp rebuild && node install/dll-copy)`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/mauro/.npm/_logs/2019-06-25T14_34_41_300Z-debug.log

Some help ?

You have a lot of permissions issues, likely from using a mix of sudo and non-sudo npm commands.
See these docs: https://docs.npmjs.com/resolving-eacces-permissions-errors-when-installing-packages-globally
You can also use a tool like nvm: https://github.com/nvm-sh/nvm

I've installed nvm and node but similar error..
Maybe the problem is other.. :(

You can try installing Node 10 LTS. Node 8 is getting old.

I’using 10 lts as you can see on log

same :(

I solve with command: npm i -g cordova-res --unsafe-perm

I solve with command: npm i -g cordova-res --unsafe-perm

thanks.

I am having the same issue, but unfortunately using --unsafe-perm isn't changing the outcome and I still have the error that 'vips/vips8' file not found in common.cc:25:10.

I am having the same issue, but unfortunately using --unsafe-perm isn't changing the outcome and I still have the error that 'vips/vips8' file not found in common.cc:25:10.

Did you get the solution of the issue

I added 'sudo' in front of the suggested 'npm i -g cordova-res --unsafe-perm' so:
sudo npm i -g cordova-res --unsafe-perm
which worked for me!

I am having the same issue, but unfortunately using --unsafe-perm isn't changing the outcome and I still have the error that 'vips/vips8' file not found in common.cc:25:10.

i'm still struggling with the problem...have you found a solution

I solve with command: npm i -g cordova-res --unsafe-perm

thank you it works

yeah it works

@atapias @swennijboer you guys are legends!

for me its did work :(

You save my day , Cheer

I am having the same issue, but unfortunately using --unsafe-perm isn't changing the outcome and I still have the error that 'vips/vips8' file not found in common.cc:25:10.

i'm still struggling with the problem...have you found a solution
this worked for me :)

I added 'sudo' in front of the suggested 'npm i -g cordova-res --unsafe-perm' so:
sudo npm i -g cordova-res --unsafe-perm
which worked for me!

Thank you，成功解决了我的问题。

It is not working for me on windows environment, I know the thread is about Mac, but if anyone have solved it on windows, please help me out.

I added 'sudo' in front of the suggested 'npm i -g cordova-res --unsafe-perm' so:
sudo npm i -g cordova-res --unsafe-perm
which worked for me!

It works

Thanks, It works,

npm i -g cordova-res --unsafe-perm

Work like a charm!


yarn start Error

Describe the bug
After I start the client app with  yarn start
I get this error, Please help me
yarn run v1.22.4 $ react-scripts start Could not find a required file.   Name: index.html   Searched in: C:\Users\acer\Desktop\cuckoo\client\public error Command failed with exit code 1. info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
Expected behavior
The client app should start
Additional context
Add any other context about the problem here.

The public folder was missing, my bad. Added it. Should work properly now.


Repeating arrows in component details breadcrumb

The example below shows multiple breadcrumb arrows, which is incorrect. Currently, the arrows should only show as expected in a breadcrumb, as you drill down through the dependencies list.

@andreasn I grabbed this from one of your comments in a PR. I'm not able to reproduce this in my Centos VM. Can you provide details about your set up? Does this issue always happen for you?

I haven't seen it again, so it might have been a temporary glitch.

I just ran into it again.

My setup is this script running on top of the RHEL test vm from the Cockpit tree.
See the file itself for how to run it on top of a Linux system. Not sure how to execute it on a mac though :/

Thanks!

@jgiardino I can't reproduce this. I suggest to close it and if we are able  to show a straightforward reproducer we can open it again.


EGNN explodes for larger problems

I'm going to look into this further... closing.

This needs further research I think. Closing for now.

I've worked quite a bit with this neural network layer now, and I think it's probably the least stable thing I've ever tried to train! It's almost impossible to keep it from exploding, unless you train it with a tiny learning rate. I suspect the algorithm itself is broken, or maybe not yet properly described in the paper. I've tried the latest fix, which helps a bit, but on training with random sized graphs of length < 100 nodes, I managed to get past 58 epochs of training (loss going down) before it started exploding to NaN. Looking at the algorithm, I just think adding an unbounded weighted sum of relative coordinates to the existing coordinates is never going to be stable for anything other than a toy problem. The weights needed for 100 nodes are never going to comparable in magnitude to the weights needed for 10 nodes, say. Maybe the original authors meant to normalize the weights in some way, or perhaps norm the relative coordinate vectors - or perhaps they just never tried it on larger messier problems.

@denjots thanks for letting me know! Have you tried any of those solutions you listed? Norming the relative vectors for weighting seems like a good idea

Have you, by chance, tried https://github.com/lucidrains/geometric-vector-perceptron yet?

@denjots there's also a lot of other things to try, if you are willing. It just seems too promising a technique to let go entirely

@denjots do you want to try setting norm_rel_coors = True for your larger experiments? normalizing the coor_weights may be a good ideas as well

I have to admit I have played with those things and others e.g. using tanh to squash the coordinate weights. It's a balancing act between basically getting it to be stable but unwilling to learn anything and unstable. Neither situation is good, obviously. Adding some normalization works for graphs up to 100, but larger graphs produce NaNs again. Applying more normalization stabilizes it again, but then it learns very very slowly (if it is indeed learning anything). It just feels like a neural net model that really wants to be unstable! Also, I'm a little worried that some tweaks will cause equivariance failures, but I think normalization at least should not do so.
I'm currently trying to rescale the weights by various functions of the graph size - maybe that will work?

@denjots are you using EGGN or EGGN_sparse. perhaps only restricting the operation to local neighborhoods on the biomolecule would work better?

@denjots yea, I mean, instability isn't really something foreign to us deep learning folks. it's just a problem to be solved

@denjots yea, i think the three things to try are (1) restricting updates only from local atoms (2) norm relative vectors (3) norm the coordinate weights coming out of the coordinate MLP

I haven't tried the sparse version - too many dependencies, some that don't install out of the box on my system. I did try some early experiments with a distance dependent mask in the basic EGGN - maybe I will revisit them. It's a shame, because EGGNs looked so simple, but I guess things can be too simple sometimes...

@denjots i wouldn't let up yet, there's another similar paper from recent ICLR that shows good results https://github.com/lucidrains/geometric-vector-perceptron I think it is worth trying to localize, or at least clamp the relative vector norms.
the alternative is to use SE3-Transformers, which, although it works, is an order of magnitude slower

@denjots https://github.com/lucidrains/En-transformer I have another version of EGNN that you could try, that is free from the PyG dependencies. you can restrict to local neighbors with num_nearest_neighbors keyword

I have tried SE-3 transformers - both Fabian's and yours - and they are both stable. Obviously the speed and simplicity of EGGN was very attractive, but you can't have everything. Thanks for the pointer to the EGNN-transformer - I had noticed it, but was keen to get the basic EGNN to work first - I will queue the transformer for testing and see how it goes, however.

@denjots thank you for relaying your results! this matters a lot to me, as if this approach can be made to work, it may make Alphafold2 replication a lot simpler

@denjots I'll ready a version of EGNN without PyG for you to try by later this evening, also restricting to nearest neighbors

@denjots I've added the feature in 0.0.20, so you can use it like so https://github.com/lucidrains/egnn-pytorch/blob/0.0.20/tests/test_equivariance.py#L22-L37

Thanks - will have a look tomorrow.

I think I'm happy enough with where it's at now. I went back to my own masking in the end - mainly becaise the idea of a fixed number of neighbors wasn't very realistic - so I just mask the coordinate weights with a specified distance threshold now. This, combined with the new initialization, stabilizes training up to at least 8 layers now, which is good enough for me. It's not learning much of value so far, but that's likely more a science problem I need to look at than a software problem. Thanks a lot for your help!

Glad to hear you got it working!

Maybe helpful?
https://arxiv.org/abs/2009.03294
You are not doing any norm in the eGNN?


Update package.json

Version 1.1.0 of stacked used the now deprecated res.headerSent. Yesterday I submitted a pull request to the maintainer to update to res.headersSent which has been merged in. Updating to version 1.1.1 will remove all of the deprecation warnings one would normally see while running superstatic. Cheers.

Thanks! Published as 1.1.7.


Fix add host and allow_unsafe_werkzeug arguments to sockerio.run()

When starting the swap-it-dashboard with all installed dependencies as specified in the Readme.md, i receive the following error:
Server initialized for threading.
Traceback (most recent call last):
File "/dashboard/application.py", line 274, in
socketio.run(app, port=8080)
File "/usr/local/lib/python3.11/dist-packages/flask_socketio/init.py", line 641, in run
raise RuntimeError('The Werkzeug web server is not '
RuntimeError: The Werkzeug web server is not designed to run in production. Pass allow_unsafe_werkzeug=True to the run() method to disable this error.
Error can be fixed by adding the allow_unsafe_werkzeug=True argument.
Besides, adding the host argument to socketio.run() enables the use of the dashboard within a docker-compose project, since custom hosts can be specified for docker-compose

Could you check that the Flask and Flask SocketIO version equal the fixed versions in requirements.txt? I'm not getting this error message, even in a fresh venv. I want to ensure it is not an issue with the versions.

received the error when starting the application from a new docker image.
Here's the content from my Dockerfile that you can re-use. The dependencies are installed as described in the ReadMe.md
docker build -t swap-it-dashboard_test -f Dockerfile .
docker run -p 8080:8080 swap-it-dashboard
FROM debian:latest
RUN apt-get update -y
RUN apt install nodejs npm -y
RUN apt-get install python3 -y
RUN apt install python3-pip -y
RUN rm -f /usr/lib/python3.11/EXTERNALLY-MANAGED
WORKDIR /
COPY . ./dashboard
WORKDIR /dashboard
RUN pip3 install --no-cache-dir -r requirements.txt
RUN npm install
EXPOSE 8080
CMD ["python3", "application.py"]

I changed the underlying server to "eventlet", since "Werkzeug" is only for development purposes. This changes fixed the issue for me, please verify that it works for you as well!

looks good to me :)


put back the transactions, fi used in SetupCommand

The observable events for models are fired before the transaction is committed. In order to make this work again we can;

create a custom event to listen to
put the observable post-(not pre-) events for these models in a pool to be fired once the transaction is committed

No longer valid for 3.x


[WIP] Fix loadMetadata bug related to ACL

This PR fixes issues uncovered during autobot testing.

ls failed to load files when acl profile is turned on (due to a masked exception)
ACL inheritance should also respect the umask. Basically the inherited acl should not violate any umask.

Merged build finished. Test FAILed.

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

jenkins, test this please

Merged build finished. Test FAILed.

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

Merged build finished. Test FAILed.

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

jenkins, test this please

Merged build finished. Test FAILed.

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

jenkins, test this please

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

Merged build finished. Test FAILed.

Merged build finished. Test FAILed.

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

jenkins, test this please

@gpang PTAL, thanks

Merged build finished. Test PASSed.

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

Merged build finished. Test PASSed.

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

@gpang comments addressed, PTAL. Thanks.

Merged build finished. Test PASSed.

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

@gpang Could you merge this one? I have a subsequent PR that relies on this to work.


Migrate to new testing APIs

This is still WIP, as the support for the new testing APIs is not yet released in ember-mocha (using the master branch for now). So don't merge yet!
But all green it seems, so 🎉 /cc @Turbo87

@andreasschacht this is now updated to the real release, and should be ready to be merged!


promote_similar for output data columns in merge

Sometimes some chunks may not contain missing values in some columns, but may in others. Or some chunks may be all Integers and some may have floats. In such cases we need to promote the output columns type while doing a merge.

Updated to also promote index columns similarly. Travis failure seems to be due to missing native dependency on OSX.


Add book download

References #4228

Adds PDF download option for books
Instead of passing a iiif image url to the Download component we now pass an array of download options
The download options come from the iiifPresentationManifest if we have one, else they are generate from the iiif imageUrl if we have that
N.B. this does mean making a second API call for the manifest, but can't see a way around that
(but since we're doing this we should send if down the pipe, then the IIIFPresentationPreview won't need to request it again - will do this in separate PR)

Next PR:

getting the appropriate licensing information
maybe using context for the download props?

closing in favour of #4255


change: remove static/dynamic directory distinction

What's in this PR?
Removes the static/dynamic subdirectories for layout files. All layout files are dumped into one directory: layouts/!
Resolves #870
Affects #848

Codecov Report

Merging #871 (fc2b555) into main (3ca62a0) will increase coverage by 0.06%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #871      +/-   ##
==========================================
+ Coverage   81.76%   81.83%   +0.06%
==========================================
  Files         105      105
  Lines        4229     4228       -1
  Branches      605      605
==========================================
+ Hits         3458     3460       +2
+ Misses        660      658       -2
+ Partials      111      110       -1

Impacted Files
Coverage Δ

.../CreeDictionary/CreeDictionary/paradigm/manager.py
93.82% <100.00%> (-3.74%)
:arrow_down:

src/CreeDictionary/CreeDictionary/views.py
81.08% <0.00%> (+1.80%)
:arrow_up:

src/CreeDictionary/API/models.py
87.84% <0.00%> (+2.20%)
:arrow_up:

Continue to review full report at Codecov.

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

@andrewdotn, given the overwhelming response in #870, I'll leave it up to you if you think this should be merged. For me, it makes documenting the whole thing a lot simpler! 😄