Add submission dates to challenge showcases

Based on the git blame output of the old website repo, this PR automatically fills the submittedOn property on the contributions (challenges only).
[Update: I was also able to get the data for cabana videos]
Out of 1303 contributions, 1065 (~81%) could be automatically filled. 90 (7%) of them had to be manually filled due to moving of data / unable to match the contribution from old to new website / the contribution only existing after the new website.
Rest of the contributions already have the field.
In  this PR, all challenge submission dates are added.

Thanks again @dipamsen!


Adds subgraph support

Fixes #8

[x] Subgraph support
[x] Documentation

[x] API
[x] example

[x] Tests

Other than the .dot output format missing the proper tabbing, this worked perfectly for me! Thanks a lot

@SebastienGllmt thank you, I fixed that in my fork.

please resolve conflicts


[Immediate] Workaround for loading merged frameworks in immediate mode

Some frameworks that previously had a separate swift overlay have been merged in macOS 14. This is causing symbols to not resolve if using the new SDK but running on an older OS in immediate mode. Ideally we would handle this automatically in JITLink as is done by the system linker, but in the meantime add a workaround to load the correct libraries manually.
rdar://110371405

I tested this manually on a macOS 13 host using a 14 beta SDK; for each framework I found a symbol that migrated and ensured it did not cause JIT errors after this change.  If there's a reasonable way to test this in the lit suite, let me know, but I didn't come up with one.

@swift-ci please test


Only act on ingest sourceTypes

We don't want to create topics for remote or host sourceType datasets. Previously, we had logic to create topics for sourceTypes implicitly. We'll switch it to an explicit model with this card.
Acceptance criteria:

[ ] Create topics for ingest or stream sourceTypes
[ ] streisand and genesis should only care about stream sourceTypes
[ ] discovery-api completeness scoring continues to work as expected

 # BEFORE ----------------------------------------------------------------------------
  def handle_dataset(%Dataset{technical: %{sourceType: "remote"}}) do
    :ok
  end

  def handle_dataset(%Dataset{} = dataset) do
    subscribe(dataset)
  end

  # AFTER ----------------------------------------------------------------------------
  def handle_dataset(%Dataset{technical: %{sourceType: "ingest"}} = dataset), do: subscribe(dataset)
  def handle_dataset(%Dataset{technical: %{sourceType: "stream"}} = dataset), do: subscribe(dataset)
  def handle_dataset(%Dataset{} = _dataset), do: :ok

Will need to go to prod along with the other issue

reaper
forklift
voltron
carpenter
People:

Rob
Johnson
Mackenzie
Jessie
Jeff
Jarred

Go To Prod

Update all redis from batch to ingest
Deploy Apps
Check busses on a map
Check a batch > ingest type?


User request: customize move notation font size

Especially if you pop up the window in a separate monitor, change it only there.

For now I've added a new theme with a slightly larger font. Closing.


easy-jsend

easy-jsend/easy-jsend.d.ts
check list

[ ] is collect naming convention?

https://www.npmjs.com/package/easy-jsend
http://bower.io/search/?q=easy-jsend
others?

[X] has a test file? (easy-jsend/easy-jsend-tests.ts or others)
[ ] pass the Travic-CI test?

@DeadAlready thanks mate!


Add support for AMBASSADOR_ID in Helm chart

In order to support multiple Ambassador installs, the AMBASSADOR_ID env variable needs to be set within the Deployment. This change adds in an optional deployment.ambassadorId variable which sets AMBASSADOR_ID within templates/deployment.yaml.

This is also addressed in #657.

@rhs should this be closed now that #657 has been merged?

Yes it should be closed, thanks for the reminder! (And thanks for submitting the PR in the first place.)


hyper parameters

Could you please provide the hyper parameters you used for the other aspects? I have successfully achieved your reported performance on aspect 0 of beer view, but I failed on aspect 1 (see the screenshot). Thanks a lot!

Different aspects use different sparsity (e.g., 0.138 for beer aspect 1). See our appendix.

Yes, set the sparsity_percentage to 0.14 for beer 1.

Have you successfully reproduced the results?

I am sorry but I cannot. The results of beer 1 with  sparsity_percentage as 0.14 are as follows:

Try this:  python run_share_beer.py --lr 0.0001 --batch_size 256 --share 1 --sparsity_percentage 0.123 --sparsity_lambda 12 --continuity_lambda 10.  We refer to the implementation of DMR and the sparsity is loosely limited. Usually the preset s should be smaller than the final sparsity. More questions please contact<EMAIL_ADDRESS>

OK. I will have a try.

I have obtained similar results as you reported. Thanks a lot! It seems that more training epochs are essential.

Other hyper-parameters have been uploaded.


Training time?

Hi, I have the code built and have run the colmap conversation and the point cloud pre process.
(I used render scale = 0.25)
I am using the mipnerf360 garden data set .
Training has now been running for 12 hours and is up to epoch 106.
I see the config is set to 600 epochs, so I assume another few days of training time.
Is this the expected time to train? I am on a rtx 3090 desktop.
Thanks!

Hi,
that seems unusually long. How many points are in your point cloud? I have been working on an RTX4090 and training took more like 4h, I don't think the 3090 should be that much slower.

It's the mipnerf 360 garden, and I ran the point cloud pre process with a factor of 2.

...I am trying again without the pre process (points = 136412) so I will report back!

Running the standard garden dataset (without pre-processing) runs great until around epoch 105, then it slows considerably (0 -to 7% in 30 mins).
I am trying with batch size reductions as:
train --config configs/train_normalnet.ini --TrainParams.scene_names garden_test --TrainParams.batch_size 1 --TrainParams.inner_batch_size 1 --TrainParams.train_crop_size 256
I will see if that helps... Any other tips appreciated!

I was not able to figure this out directly, sorry!
I notice you seem to take the sparse output of COLMAP, our method requires the dense output btw. However this should not impact speed.
I will upload the mipnerf360 datasets and checkpoints we used soon (at least the 7 that are openly sharable), so you can try out the result at least :)

Thank you, that would be great.
Using the train --config configs/train_normalnet.ini --TrainParams.scene_names garden_test --TrainParams.batch_size 1 --TrainParams.inner_batch_size 1 --TrainParams.train_crop_size 256 with a render scale of 0.25 I was able to train, but he result is pretty patchy due to lowering the res! I will try a colmap dense cloud as well. Thanks again!

I have added the 7 scenes from the mipnerf dataset (outdoor and indoor half resolution as in our evaluation) as well as the boat to the supplemental: https://zenodo.org/records/10641253

Thank you! I have now computed the dense cloud with colmap, and running with:
train --config configs/train_normalnet.ini --TrainParams.scene_names garden_dense --TrainParams.batch_size 1 --TrainParams.inner_batch_size 1 --TrainParams.train_crop_size 256
and a render_scale of 0.25 fails after the 'test' run.
train --config configs/train_normalnet.ini --TrainParams.scene_names garden_dense --TrainParams.batch_size 1 --TrainParams.inner_batch_size 1 --TrainParams.train_crop_size 256
register neural render info
register TnnInfo
PyTorch version: 2.1.0
The cuDNN version is 8801
cuDNN avail? 1
The CUDA runtime version is 12010
The driver version is 12020
Loading Config File configs/train_normalnet.ini
Using Random Seed:<PHONE_NUMBER>
POINT GRADIENTS ARE COMPUTED.
torch::cuda::cudnn_is_available() 1
Render Mode epochs: DT -2 - Fullblend -2 - Fuzzyblend -2 - BilinearBlend -2 - FastBlend 0
Use NeAT recogarden_dense
====================================
Scene Loaded
  Name       garden_dense
  Path       D:\NERF\TRIPS\scenes\garden_dense
  Image Size 5187x3361
  Aspect     1.54329
  K          3844.9 3852.36 2593.5 1680.5 0
  ocam       5187x3361 affine(1, 0, 0, 0, 0) cam2world() world2cam()
  ocam cut   1
  normalized center 0 0
  dist       0 0 0 0 0 0 0 0
CAM model: CameraModel::PINHOLE_DISTORTION
  Points     7877153
  Colors     1
  Normals    1
  Avg. EV  0
  Num Images 185
  Num Cameras 1
Compute scene importance bounding box as 95% of points interval around center of mass
Starting Compute center of mass...center of mass:0.308283
2.15885
1.12378
 Done in 74.9361ms.
Starting Build range vec... Done in 46.5421ms.
Starting Sort range vec... Done in 889.808ms.
Starting Extend box... Done in 161ms.
Box: AABB: [-12.1599 -5.48379 -11.3698 ] [12.8644 6.71304 12.093 ]
====================================
Modulo stepsize: 8
Train(161): 1 2 3 4 5 6 7 9 10 11 12 13 14 15 17 18 19 20 21 22 23 25 26 27 28 29 30 31 33 34 35 36 37 38 39 41 42 43 44 45 46 47 49 50 51 52 53 54 55 57 58 59 60 61 62 63 65 66 67 68 69 70 71 73 74 75 76 77 78 79 81 82 83 84 85 86 87 89 90 91 92 93 94 95 97 98 99 100 101 102 103 105 106 107 108 109 110 111 113 114 115 116 117 118 119 121 122 123 124 125 126 127 129 130 131 132 133 134 135 137 138 139 140 141 142 143 145 146 147 148 149 150 151 153 154 155 156 157 158 159 161 162 163 164 165 166 167 169 170 171 172 173 174 175 177 178 179 180 181 182 183
Test(24): 0 8 16 24 32 40 48 56 64 72 80 88 96 104 112 120 128 136 144 152 160 168 176 184
cropped sampler 5187x3361 to 256 x 256 render scale 0.25
cropped sampler 5187x3361 to 256 x 256 render scale 0.25
full sampler 1296x840 render scale 0.25
full sampler 1296x840 render scale 0.25
POINT GRADIENTS ARE COMPUTED.
GPU memory - Point Cloud: 283.578MB
Pinhole Intrinsics: 3844.9 3852.36 2593.5 1680.5 0  -- 0 0 0 0 0 0 0 0
Tensor [1, 13] float cuda:0 Min/Max 0 3852.36 Mean 920.866 Sum 11971.3 sdev 1531.62 req-grad 0 Min-Coords: [0, 4, ]
GPU memory - Texture [4, 7877153] : 189.052MB
Using Mm Adam texture optimizer
optimizing texture with lr 0.1/0.004
optimizing response with lr 0.0001
optimizing exposure with lr 0.0005
Optimizing with my adam implementation.
optimizing 3D points with lr 0.0001
optimizing point size with lr 0.01
optimizing poses with lr 0.0001
no intrinsics optimizer
POINT GRADIENTS ARE COMPUTED.
Using MultiScaleUnet2dDecOnlySmall with filters:
32 32 32 32 32 32 32 32
RENDER NETWORK: NUM PARAMETERS: 22
Total Model Params: 59675
Loading VGG from: loss/traced_caffe_vgg_optim.pt

=== Epoch 0 ===
Scene Log - Texture: Tensor [4, 7877153] float cpu Min/Max 0 1 Mean 0.499994 Sum 1.57541e+07 sdev 0.288652 req-grad 1 Min-Coords: [0, 2355555, ]
  Background Desc:
  Confidence per point: Tensor [1, 7877153] float cpu Min/Max 0.5 0.5 Mean 0.5 Sum 3.93858e+06 sdev 0 req-grad 0 Min-Coords: [0, 0, ]
  Confidences under 0.5: 7877153
  LayerBuf per point: Tensor [7877153, 1] float cuda:0 Min/Max -8.03805 17.4942 Mean -5.37804 Sum -4.23637e+07 sdev 0.689841 req-grad 1 Min-Coords: [6001664, 0, ]
      softplus: Tensor [7877153, 1] float cuda:0 Min/Max 0.000322886 17.4942 Mean 0.00614902 Sum 48436.8 sdev 0.0173115 req-grad 1 Min-Coords: [6001664, 0, ]
  Poses: Tensor [185, 8] double cuda:0 Min/Max -1.98294 4.1659 Mean 0.33086 Sum 489.672 sdev 1.33716 req-grad 0 Min-Coords: [117, 5, ]
  Point Position: Tensor [7877153, 4] float cuda:0 Min/Max -81.9788 96.2361 Mean 0.903868 Sum 2.84796e+07 sdev 3.4264 req-grad 1 Min-Coords: [4570866, 2, ]
Eval  0 |   0% |                              |   0/161 [00:00:0000] [0.00 e/s] USING NEW IMPL
Eval  0 | 100% |##############################| 161/161 [00:58:0019] [2.77 e/s]  Cur=1510.437378 Avg=1545.123047
Best - Worst (Eval) [1461.39, 1650.78]
Loss:         garden_dense Param 0 VGG 1544.31 L1 0.363617 MSE 0.183437 PSNR 7.38558 SSIM 0.447156 LPIPS 0.966876 count 161
Test  0 | 100% |##############################| 24/24 [00:14:0136] [1.70 e/s]  Cur=1526.503418 Avg=1546.691284
Best - Worst (Eval) [1493.11, 1638.31]
Loss:         garden_dense Param 0 VGG 1545.88 L1 0.362421 MSE 0.182466 PSNR 7.40616 SSIM 0.447356 LPIPS 0.966075 count 24
Saving checkpoint...

I skipped the point pre-process step here as well. Is it simply a hardware issue? Can I downsample anything further?

Hi,
from your outputs, the only thing that seems wierd to me are your torch and cuda versions. Running train on my machine, the versions look like this:
PyTorch version: 1.13.1
The cuDNN version is 8302
cuDNN avail? 1
The CUDA runtime version is 11070

while you have PyTorch 2.1.0, cuDNN  8801 and CUDA runtime 12010.
Maybe that is the issue there?

Hi.,
I have a similar problem with training time. I used tt_playground, and train with default parameters:
build\bin\RelWithDebInfo\train.exe --config configs\train_normalnet.ini
It took 18 hours for 330 epoch, as shown in the image below.

My GPU: RTX 3090
Windows: 11
Cuda 11.8
Do you have any recommendations so that I can reduce training time for this data?
Thank you in advance


Column return_code is missing

Error writing merge history entry to database: table merge_history has no column named return_code

This is a bug that comes after an upgrade from an old version. See the wiki for solutions


Routing error while testing latest civic-server & civic-client

Hi:
Just doing a fresh integration test with latest civic-server & civic-client. Got the routing error below.
Thanks,
-b

fresh clone of civic-server civic-client
civic-server $ git log -1
commit f7126a2ae51c9df9094ab18eb1bf2a44954ab3e9
Author: Adam Coffman<EMAIL_ADDRESS>Date:   Thu May 28 12:49:17 2015 -0500

civic-client $ git log -1
commit 9ac055a524248555a314052f518d0815e39bf762
Author: nspies13<EMAIL_ADDRESS>Date:   Thu Apr 16 00:07:20 2015 -0500

Looks like your civic-client repo is out of date, the last commit on the master branch was yesterday (bf2a95cebb66befb5be39b1616da75de96138a0b). We've since switched to non-nested URLs for the REST endpoints, so if the variant ID is 1, to get its representation you'd request:
http://<IP_ADDRESS>:3000/api/variants/1

Closing.

Thanks.  I double checked and was indeed on an older version. Apologies for the interruption


Form is not responsive

components: form
reactstrap version 6.4.0
react version 16.5.2
bootstrap version 4.1.3

What is happening?
I have email form it isn't responsive for other devices. That's look like here. I describe my problem on stackoverflow, but no one helped me. Can someone help please?
What should be happening?
Form should be responsive also for smaller devices.
Thank you!

@TheSharpieOne you are my hero! Thank you!


sonatypePublishToBundle not found

If I put publishTo := sonatypePublishToBundle.value in my build file ( link ), I get
/home/vvasuki/gitland/indic-transliteration/indic_transliteration_scala/build.sbt:38: error: not found: value sonatypePublishToBundle
publishTo := sonatypePublishToBundle.value
             ^
[error] Type error in expression

What am I doing wrong?

sbt version: 1.7.3
sbt-sonatype version: 2.0

[IJ]sonatypeProfileName
[info] indic-transliteration
[IJ]sonatypeRepository
[info] https://oss.sonatype.org/service/local
[IJ]

Also, in ~/.sbt/1.0/sonatype.sbt , I have:
sonatypeProfileName := "com.github.sanskrit-coders" . Whence do I get indic-transliteration above?

The latest version of sbt-sonatype is 3.9.14. Try the latest one first

Thanks! I got past that problem, but now getting different errors. Trying with a simpler repository (https://github.com/sanskrit-coders/scala-utils/blob/d582dedfa08ce117d0bf3a9df788a7938b3a0da1/build.sbt ):
[IJ]sonatypeProfileName
[info] com.github.sanskrit-coders
[IJ]sonatypeRepository
[info] https://oss.sonatype.org/service/local
[IJ]

[IJ]publishSigned
[info] Wrote /home/vvasuki/gitland/sanskrit-coders/scala-utils/target/scala-2.13/scala-utils_2.13-1.24-SNAPSHOT.pom
[warn] versionScheme setting is empty; set `ThisBuild / versionScheme := Some("early-semver")`, `Some("semver-spec")` or `Some("pvp")`
[warn] so tooling can use it for eviction errors etc - https://www.scala-sbt.org/1.x/docs/Publishing.html
[error] gpg: WARNING: unsafe permissions on homedir '/home/vvasuki/.gnupg'
[error] stack trace is suppressed; run last publishSigned for the full output
[error] (publishSigned) java.io.IOException: Server returned HTTP response code: 403 for URL: https://oss.sonatype.org/content/repositories/snapshots/default/scala-utils_2.13/1.24-SNAPSHOT/scala-utils_2.13-1.24-SNAPSHOT-sources.jar.asc
[error] Total time: 3 s, completed Nov 5, 2022, 7:34:08 AM
[IJ]

And running the release command fails with 2022-11-05 07:36:19.868+0530  warn [SonatypeService] No staging repository is found. Do publishSigned first.  - (SonatypeService.scala:119).
For a fresh start I'd dropped the staging repo I found via the web interface:

How to get past this?

It's a permission error. See https://github.com/xerial/sbt-sonatype#homesbtsbt-version-013-or-10sonatypesbt
And also, your sbt-release configuration is out-dated. See https://github.com/xerial/sbt-sonatype#using-with-sbt-release-plugin

It's a permission error. See https://github.com/xerial/sbt-sonatype#homesbtsbt-version-013-or-10sonatypesbt

I doubt it because I have -
> cat ~/.sbt/1.0/sonatype.sbt                                                                                 ✔

credentials += Credentials("Sonatype Nexus Repository Manager",
  "oss.sonatype.org",
  "vishvas",
  "REMOVED")

// Your profile name of the sonatype account. The default is the same with the organization value
sonatypeProfileName := "com.github.sanskrit-coders"

// To sync with Maven central, you need to supply the following information:
publishMavenStyle := true

// License of your choice
licenses := Seq("APL2" -> url("http://www.apache.org/licenses/LICENSE-2.0.txt"))
homepage := Some(url("https://sanskrit-coders.github.io/"))

developers := List(
  Developer(
    id    = "sanskrit-coders",
    name  = "Sanskrit Coders",
    email =<EMAIL_ADDRESS>    url   = url("https://sanskrit-coders.github.io/")
  )
)%

(I've added some common settings there instead of (project root)/sonatype.sbt since multiple projects share the same developers, licenses, homepage etc. fields.

And also, your sbt-release configuration is out-dated. See https://github.com/xerial/sbt-sonatype#using-with-sbt-release-plugin

The below seems right as per the link?
releaseProcess := Seq[ReleaseStep](
  checkSnapshotDependencies,
  inquireVersions,
  runClean,
  runTest,
  setReleaseVersion,
  commitReleaseVersion,
  tagRelease,
  releaseStepCommand("publishSigned"),
  releaseStepCommand("sonatypeRelease"),
  setNextVersion,
  commitNextVersion,
  pushChanges
)

The "read staging respository" step seems to succeed -  Yet, I get the error:
[info] 	publish commited: moved /home/vvasuki/gitland/sanskrit-coders/scala-utils/target/sonatype-staging/1.25/default/scala-utils_2.13/1.25.part
[info] 		to /home/vvasuki/gitland/sanskrit-coders/scala-utils/target/sonatype-staging/1.25/default/scala-utils_2.13/1.25
[success] Total time: 2 s, completed Nov 5, 2022, 1:08:33 PM
2022-11-05 13:08:33.871+0530  info [SonatypeService] sonatypeRepository  : https://oss.sonatype.org/service/local  - (SonatypeService.scala:26)
2022-11-05 13:08:33.873+0530  info [SonatypeService] sonatypeProfileName : com.github.sanskrit-coders  - (SonatypeService.scala:27)
2022-11-05 13:08:33.874+0530  info [SonatypeClient] Reading staging repository profiles...  - (SonatypeClient.scala:100)
2022-11-05 13:08:47.276+0530  warn [SonatypeService] No staging repository is found. Do publishSigned first.  - (SonatypeService.scala:119)
2022-11-05 13:08:47.293+0530  info [SonatypeClient] Reading staging profiles...  - (SonatypeClient.scala:114)
2022-11-05 13:08:49.690+0530 error [Sonatype]
java.lang.IllegalStateException: No staging repository is found. Run publishSigned first

The releasing step has been changed a long time ago. publishSigned no longer creates staging repository at Sonatype. https://medium.com/@taroleo/sbt-sonatype-f02bdafd78f1
// This setting creates a bundle in the local directory if version is not a snapshot:
publishTo := sonatypePublishToBundle.value

  releaseStepCommand("publishSigned"), // This creates files in local repository
  releaseStepCommand("sonatypeRelease"), // This step uploads bundle and perform releasing steps at sonatype

sonatypeRelease has been changed to sonatypeBundleRelease:

Thank you! Upon fixing the above, I don't get the previous mentioned error. But now I get:

2022-11-08 14:27:40.442+0530  info [SonatypeClient]   Evaluate: profile-target-matching-staging  - (SonatypeClient.scala:384)
2022-11-08 14:27:40.442+0530  info [SonatypeClient] The close stage is in progress. Waiting for 10.13 sec.  - (SonatypeClient.scala:164)
2022-11-08 14:27:51.255+0530  info [SonatypeClient]     Failed: profile-target-matching-staging, failureMessage:Profile target mismatch: /default/scala-utils_2.13/1.25/scala-utils_2.13-1.25-sources.jar.asc  - (SonatypeClient.scala:384)
2022-11-08 14:27:51.255+0530  info [SonatypeClient]   Evaluate: signature-staging  - (SonatypeClient.scala:384)
2022-11-08 14:27:51.255+0530  info [SonatypeClient]     Passed: signature-staging  - (SonatypeClient.scala:384)
2022-11-08 14:27:51.256+0530  info [SonatypeClient]   Evaluate: no-traversal-paths-in-archive-file  - (SonatypeClient.scala:384)
2022-11-08 14:27:51.256+0530 error [SonatypeClient] [close] Failed  - (SonatypeClient.scala:176)
2022-11-08 14:27:51.256+0530 error [SonatypeClient] Activity name:close, started:2022-11-08T08:57:24.943Z  - (SonatypeClient.scala:464)
2022-11-08 14:27:51.257+0530 error [SonatypeClient]     Failed: profile-target-matching-staging, failureMessage:Profile target mismatch: /default/scala-utils_2.13/1.25/scala-utils_2.13-1.25-sources.jar.asc  - (SonatypeClient.scala:382)
2022-11-08 14:27:51.258+0530 error [Sonatype] [STAGE_FAILURE] Failed to close the repository.  - (Sonatype.scala:440)

Process finished with exit code 1

Build file is https://github.com/sanskrit-coders/scala-utils/blob/master/build.sbt  . What else is wrong?

It seems this error is shown if your uploaded artifact is not following Maven's conversion:
2022-11-08 14:27:51.255+0530  info [SonatypeClient]     Failed: profile-target-matching-staging, failureMessage:Profile target mismatch: /default/scala-utils_2.13/1.25/scala-utils_2.13-1.25-sources.jar.asc  - (SonatypeClient.scala:384)

The path includes /default, which should be removed somehow.

The path includes /default, which should be removed somehow.

I see, thanks! No idea where that is coming from - tried searching for "default" in the codebase - but turned up nothing.
Earlier messages look like this:
[info] Main Scala API documentation successful.
[info] 	published scala-utils_2.13 to /home/vvasuki/gitland/sanskrit-coders/scala-utils/target/sonatype-staging/1.25.0/default/scala-utils_2.13/1.25.0.part/scala-utils_2.13-1.25.0-sources.jar.asc
[info] 	published scala-utils_2.13 to /home/vvasuki/gitland/sanskrit-coders/scala-utils/target/sonatype-staging/1.25.0/default/scala-utils_2.13/1.25.0.part/scala-utils_2.13-1.25.0.jar
[info] 	published scala-utils_2.13 to /home/vvasuki/gitland/sanskrit-coders/scala-utils/target/sonatype-staging/1.25.0/default/scala-utils_2.13/1.25.0.part/scala-utils_2.13-1.25.0.pom
[info] 	published scala-utils_2.13 to /home/vvasuki/gitland/sanskrit-coders/scala-utils/target/sonatype-staging/1.25.0/default/scala-utils_2.13/1.25.0.part/scala-utils_2.13-1.25.0.jar.asc
[info] 	published scala-utils_2.13 to /home/vvasuki/gitland/sanskrit-coders/scala-utils/target/sonatype-staging/1.25.0/default/scala-utils_2.13/1.25.0.part/scala-utils_2.13-1.25.0-javadoc.jar.asc
[info] 	published scala-utils_2.13 to /home/vvasuki/gitland/sanskrit-coders/scala-utils/target/sonatype-staging/1.25.0/default/scala-utils_2.13/1.25.0.part/scala-utils_2.13-1.25.0.pom.asc
[info] 	published scala-utils_2.13 to /home/vvasuki/gitland/sanskrit-coders/scala-utils/target/sonatype-staging/1.25.0/default/scala-utils_2.13/1.25.0.part/scala-utils_2.13-1.25.0-javadoc.jar
[info] 	published scala-utils_2.13 to /home/vvasuki/gitland/sanskrit-coders/scala-utils/target/sonatype-staging/1.25.0/default/scala-utils_2.13/1.25.0.part/scala-utils_2.13-1.25.0-sources.jar
[info] 	publish commited: moved /home/vvasuki/gitland/sanskrit-coders/scala-utils/target/sonatype-staging/1.25.0/default/scala-utils_2.13/1.25.0.part
[info] 		to /home/vvasuki/gitland/sanskrit-coders/scala-utils/target/sonatype-staging/1.25.0/default/scala-utils_2.13/1.25.0
[success] Total time: 2 s, completed Nov 10, 2022, 5:27:30 PM
2022-11-10 17:27:30.122+0530  info [SonatypeService] sonatypeRepository  : https://oss.sonatype.org/service/local  - (SonatypeService.scala:26)
2022-11-10 17:27:30.125+0530  info [SonatypeService] sonatypeProfileName : com.github.sanskrit-coders  - (SonatypeService.scala:27)
2022-11-10 17:27:30.128+0530  info [SonatypeClient] Reading staging repository profiles...  - (SonatypeClient.scala:100)
[info] Preparing a new staging repository for [sbt-sonatype] scala-utils 1.25.0
2022-11-10 17:27:31.163+0530  info [SonatypeClient] Reading staging profiles...  - (SonatypeClient.scala:114)
2022-11-10 17:27:34.760+0530  info [SonatypeClient] Creating a staging repository in profile com.github.sanskrit-coders with a description key: [sbt-sonatype] scala-utils 1.25.0  - (SonatypeClient.scala:120)
2022-11-10 17:27:35.052+0530  info [SonatypeService] No previous staging repository for [sbt-sonatype] scala-utils 1.25.0 was found  - (SonatypeService.scala:99)
2022-11-10 17:27:42.551+0530  info [SonatypeClient] Created successfully: comgithubsanskrit-coders-1168  - (SonatypeClient.scala:133)
SLF4J: No SLF4J providers were found.
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See https://www.slf4j.org/codes.html#noProviders for further details.
SLF4J: Class path contains SLF4J bindings targeting slf4j-api versions 1.7.x or earlier.
SLF4J: Ignoring binding found at [jar:file:/home/vvasuki/.sbt/boot/scala-2.12.17/org.scala-sbt/sbt/1.7.3/log4j-slf4j-impl-2.17.1.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: See https://www.slf4j.org/codes.html#ignoredBindings for an explanation.
2022-11-10 17:27:42.613+0530  info [SonatypeClient] Uploading bundle /home/vvasuki/gitland/sanskrit-coders/scala-utils/target/sonatype-staging/1.25.0 to https://oss.sonatype.org/service/local/staging/deployByRepositoryId/comgithubsanskrit-coders-1168/  - (SonatypeClient.scala:294)
2022-11-10 17:27:44.986+0530  info [SonatypeClient] Finished bundle upload: /home/vvasuki/gitland/sanskrit-coders/scala-utils/target/sonatype-staging/1.25.0  - (SonatypeClient.scala:296)
2022-11-10 17:27:44.989+0530  info [SonatypeClient] Closing staging repository [comgithubsanskrit-coders-1168] status:open, profile:com.github.sanskrit-coders(55338d12f081b8) description: [sbt-sonatype] scala-utils 1.25.0  - (SonatypeClient.scala:192)
2022-11-10 17:27:45.662+0530  info [SonatypeClient] Activity name:open, started:2022-11-10T11:57:37.541Z, stopped:2022-11-10T11:57:42.514Z  - (SonatypeClient.scala:395)
2022-11-10 17:27:45.663+0530  info [SonatypeClient] repositoryCreated: id:comgithubsanskrit-coders-1168, user:vishvas, ip:<IP_ADDRESS>  - (SonatypeClient.scala:384)
2022-11-10 17:27:45.666+0530  info [SonatypeClient] The close stage is in progress. Waiting for 3.00 sec.  - (SonatypeClient.scala:164)
2022-11-10 17:27:48.911+0530  info [SonatypeClient] Activity name:close, started:2022-11-10T11:57:46.819Z  - (SonatypeClient.scala:395)
2022-11-10 17:27:48.911+0530  info [SonatypeClient]   Evaluate: id:8df11eec44baf2, rule:sources-staging, rule.disabled:javadoc-staging  - (SonatypeClient.scala:384)
2022-11-10 17:27:48.911+0530  info [SonatypeClient] The close stage is in progress. Waiting for 4.50 sec.  - (SonatypeClient.scala:164)
2022-11-10 17:27:53.645+0530  info [SonatypeClient]   Evaluate: checksum-staging  - (SonatypeClient.scala:384)
2022-11-10 17:27:53.645+0530  info [SonatypeClient]     Passed: checksum-staging  - (SonatypeClient.scala:384)
2022-11-10 17:27:53.645+0530  info [SonatypeClient]   Evaluate: signature-staging  - (SonatypeClient.scala:384)
2022-11-10 17:27:53.645+0530  info [SonatypeClient]     Passed: signature-staging  - (SonatypeClient.scala:384)
2022-11-10 17:27:53.646+0530  info [SonatypeClient]   Evaluate: pom-staging  - (SonatypeClient.scala:384)
2022-11-10 17:27:53.646+0530  info [SonatypeClient] The close stage is in progress. Waiting for 6.75 sec.  - (SonatypeClient.scala:164)
2022-11-10 17:28:01.308+0530  info [SonatypeClient]     Passed: pom-staging  - (SonatypeClient.scala:384)
2022-11-10 17:28:01.308+0530  info [SonatypeClient]   Evaluate: sources-staging  - (SonatypeClient.scala:384)
2022-11-10 17:28:01.309+0530  info [SonatypeClient]     Passed: sources-staging  - (SonatypeClient.scala:384)
2022-11-10 17:28:01.309+0530  info [SonatypeClient]   Evaluate: profile-target-matching-staging  - (SonatypeClient.scala:384)
2022-11-10 17:28:01.309+0530  info [SonatypeClient]     Failed: profile-target-matching-staging, failureMessage:Profile target mismatch: /default/scala-utils_2.13/1.25.0/scala-utils_2.13-1.25.0-javadoc.jar.sha1  - (SonatypeClient.scala:384)
2022-11-10 17:28:01.309+0530  info [SonatypeClient]   Evaluate: no-traversal-paths-in-archive-file  - (SonatypeClient.scala:384)
2022-11-10 17:28:01.309+0530 error [SonatypeClient] [close] Failed  - (SonatypeClient.scala:176)
2022-11-10 17:28:01.309+0530 error [SonatypeClient] Activity name:close, started:2022-11-10T11:57:46.819Z  - (SonatypeClient.scala:464)
2022-11-10 17:28:01.310+0530 error [SonatypeClient]     Failed: profile-target-matching-staging, failureMessage:Profile target mismatch: /default/scala-utils_2.13/1.25.0/scala-utils_2.13-1.25.0-javadoc.jar.sha1  - (SonatypeClient.scala:382)
2022-11-10 17:28:01.311+0530 error [Sonatype] [STAGE_FAILURE] Failed to close the repository.  - (Sonatype.scala:440)

Process finished with exit code 1

Tried asking at https://issues.sonatype.org/browse/MVNCENTRAL-7724

Got a solution there!
Earlier I had (as required in https://github.com/xerial/sbt-sonatype ):
sonatypeProfileName := "com.github.sanskrit-coders"
I was able to release, as soon as I added:
organization := "com.github.sanskrit-coders"
organizationName := "com.github.sanskrit-coders"
organizationHomepage := Some(url("https://sanskrit-coders.github.io/"))

Is this a bug in README.md?
ALso, in case organization is not set, rather than using default which would leave debuggers clueless,  it may be a good idea to set unspecifiedOrganization etc..


Delay on RGB camera data in v0.9.10

CARLA version:
Specifically 0.9.10
Platform/OS:
Windows 10
Problem you have experienced:
The lambda listening to a camera (sensor.camera.rgb) receives data with a large delay (about 500-1000ms). Other sensors seem to be unaffected.
What you expected to happen:
A lesser delay.
Steps to reproduce:
Spawn an RGB camera, set the lambda with .listen(), then display images as soon as they arrive. In 0.9.10, the data has a delay compared to the simulation spectator camera or other sensors' data.
Other information (documentation you consulted, workarounds you tried):
In other versions of CARLA (tested with 0.9.11 and 0.9.12) there is no noticeable delay.

I found this piece of information in the "Synchrony and time-step" documentation:

Data coming from GPU-based sensors, mostly cameras, is usually generated with a delay of a couple of frames. Synchrony is essential here.

What I experience in 0.9.10 is a far longer delay than what "a couple of frames" would imply. Anyways, looks like synchronous mode could solve this.
I won't have time to do proper testing in synch mode, but I might revisit this issue later. Until then, I'll be using version 0.9.12 (I don't need specifically 0.9.10).

@tamas-visy the delay from sensors depends heavily on hardware configuration. The delay of a couple of frames is presuming very high performance graphics hardware (and also CPU). If you are using high performance GPUs from older generations or commodity graphics hardware, this would explain why your delay might be longer. Even though we do make the best of effort to optimise, CARLA is necessarily highly demanding on graphics hardware.
Can you give us some details of your hardware configuration?

I'm using a regular CPU and GPU. However, please note that as per my original post

In other versions of CARLA (tested with 0.9.11 and 0.9.12) there is no noticeable delay.

The issue exists in specifically 0.9.10, in the newer versions I tested I did not encounter a noticeable delay. However, I did not find any patch notes about a related fix or update. Also, I do not recall such a delay in a pre-0.9.10 version on my configuration either.


file.py: Rename to local_target.py

Description

Rename file.py to local_target.py
Remove deprecated File class

Thanks for the catch @dlstadther, does it look good now? :)


Set sphinx-build --keep-going by default

Summary
Related to Qiskit/qiskit-terra#7940
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.09%) to 86.152%

Totals

Change from base Build<PHONE_NUMBER>:
0.09%

Covered Lines:
2812

Relevant Lines:
3264

💛  - Coveralls


added L-S test notebook

Used the same dataset as in the SuperSmoother example; finds the same period, if high-enough resolution.

Looks great!


Add info / links for website/API changes on new releases to FAQ

Ref this review on the initial FAQ document:
Originally posted by @aahlenst in https://github.com/AdoptOpenJDK/openjdk-build/pull/1457#discussion_r485525430

Covered in the FAQ.md changes in https://github.com/AdoptOpenJDK/openjdk-build/pull/2066/files


Thesis pseudo-code problem

Read the paper corresponding to the code and found that there is a problem in Algorithm 4.
18 for i = 1 t o l e n (P) do
19 if Pi.isDominated = True then
20 F ← F ∪ Pi;
21 end
22 end
Pi.isDominated = True   I think it should be changed to  Pi.isDominated = False
Because the solution that is not dominated is thePareto front set, called F。

Dear love381,
Yes, you are right. It should be Pi.isDominated = False in the paper.
However, this problem is corrected in the code in this repository. See line 66 of the file DeepPruningESWIN.py.
Best,
Francisco Fernandes

Dear Fernandes，
I will take a good look at the source code, thank you for your affirmation.
Best,
yameng zhu


Handle missing revocation property

Codecov Report

Merging #437 into master will not change coverage by %.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #437   +/-   ##
=======================================
  Coverage   88.90%   88.90%
=======================================
  Files         247      247
  Lines       12082    12082
=======================================
  Hits        10742    10742
  Misses       1340     1340


[WIP] Fix bastion IP provisioning script snippet (Issue 2605)

Small changes:

Update main.tf of OpenStack tree to fix provisioning of bastion IP in no-floating.yml
Minor clarification of OpenStack procedure

I was testing using command-line arguments to set the Terraform working directory and @kschroed points out the script will fail if Terraform is run from inventory/$CLUSTER as defined in the README, so I need to update that.

CI failure? Someone resolving this?

It needs to be changed also on the master and node right ?
I still run into the error when using bastion_node = 0

This needs rebase
PTAL


Move copying workflows to template to on push

In our current workflow, we're copying workflows from .github/workflows to `templates/github´.
https://github.com/bjerkio/create-bjerk-typescript/blob/92945134d26df4e65cff838aa5dfb34a9e271b64/.github/workflows/weekly-cron-jobs.yaml#L54-L80
I realize that v2.11.0 didn't get the GitHub Workflow updates that are in the changelog because we only updated .github/workflows. Would it make sense to always look for these changes on every push?

For reference, v2.12.0 that was just released includes changes that was supposed to be in v2.11.0 :)


Only allow patch releases for our dependencies

I hope this will help reduce future breakages as those with GRPC 1.8.0.

@ofrobots what do you think about this approach?
@schmidt-sebastian can you link the relevant issue from grpc that explains the breakage?

@schmidt-sebastian can you link the relevant issue from grpc that explains the breakage?

Sorry, I should have done that to begin with: https://github.com/grpc/grpc-node/issues/130
We saw high errors numbers on Google Cloud Functions and many user reports after we automatically pulled in this update. There are some links to these reports in the GitHub issue.

If I understand correctly, grpc messed up by making a breaking behaviour change in a semver-minor update. This is terrible, but it is still a mistake made by gRPC. We should provide this feedback to the grpc folks (/cc @murgatroid99) and let them fix. Locking down all dependencies using ~ dependencies seems overkill. If gRPC continues break the semver contract, we should absolutely stop trusting their updates, and perhaps escalate.
The problem with a locking down dependencies is that in the npm module ecosystem fixes (bug fixes and security fixes) are typically provided on the tip of the major branch. If you are restricting to through2 as ~2.0.3 then you're not going to get any fixes after<EMAIL_ADDRESS>happens. This will rapidly lead to the package being out of date and potentially insecure.
/cc @JustinBeckwith

@ofrobots Sounds good. I will update this PR to allow future versions of GRPC patch releases for 1.7.x only (right now we lock to 1.7.1).


Ability to specify startup window

As requested in #271.

Coverage increased (+0.04%) to 97.31% when pulling 427049079d3dad83f49c9ba65fabfd045f20eec9 on kerams:startup_window into f9e7089e65e8ad31e61f0f8c64b0ea778e0cb634 on tmuxinator:master.


Add dev version to version string

Summary
This commit adding dev versions to the qiskit.version string. If a
.git dir is found and we determine that the current head does not point
at a tag then we will append a dev version to the version string to
indicate that. If however we can't find a .git dir (likely meaning it's
a pip install) or the curret git HEAD is a tag, then we use the version
string from VERSION.txt.
Details and comments
Fixes #1796

Downloaded your branch, but it does not seem to work for me:

In [8]: get_version_info()
Out[8]: '0.9.0'

Actually, I guess the git info is '' when you download.

Yeah it will default to 0.9.0 (or whatever version is in the version.txt) if it can't find a .git dir. If you run
git checkout -b mtreinish-dev-version-str master && git pull https://github.com/mtreinish/qiskit-core.git dev-version-str in a terra repo checkout it should work fine. (or at least it does for me locally):
Python 3.7.3 (default, Mar 26 2019, 21:43:19)
[GCC 8.2.1 20181127] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import qiskit
dot - graphviz version 2.40.1 (0)
b''
q>>> qiskit.__version__
'0.9.0.dev0+64e5ca7'
>>>

``` (ignore the dot output, that's fixed by #2495 )

Yeah, I got it to work.  Does this work on Windows?

Umm, it should assuming git is in the windows PATH and there is a .git. In the worst case it will just fail to run git and set the version string to 0.9.0.dev0+Unknown

Ok this is failing because the basic aer sims use the terra version as their own. So need to mod that as well.

Ok, I spun up a windows vm to test this there. I got it working eventually (it did take more effort than I expected). In the process I did a find a few issues though, first we don't suppress stderr from the git commands, second we didn't catch an exception from the is this commit also a tag check, third we didn't handle running outside of the repo correctly. I've got fixes for those which I'll push up shortly.


Issue #4: Security posture of dependencies

This pull request addresses the following:

Removal of the phpjs dependency
Addition of the locutus dependency
Modification to the src/index.js to address the breaking API change from phpjs
to locutus for the php/network/{ip2long,long2ip} functions

I'm getting the same security errors, would appreciate this being merged.

@krg7880 any chance this PR can be merged?


ci: Generate SBOMs for all container images on release

This PR

generate SBOMs with Syft for all Keptn container images when a release is done
SBOMs are attached to the release as separate *.spdx.json files

Fixes #7163

Can we manually attach the sbom to the release v1.0.0?

yes, i'll do it this week

Can we manually attach the sbom to the release v1.0.0?

@thisthat done


Support passing env variables to TestKit build runs

My understanding is that passing env variables to build runs in TestKit is something desired but not prioritized yet. Based on the TestKit doc:
The TestKit does not expose a mechanism for fine grained control of environment variables etc. Future versions of the TestKit will provide improved configuration options
This ticket is to track feature request to add env variables support to TestKit / GradleRunner API. Can you confirm that it is a desired feature? I hope the answer is 'yes' :). It is impacting our ability to use TestKit at LinkedIn. We might be able to provide a pull request.

Friendly ping! We can offer a PR but need to know if this is a desired feature - passing env variables to TestKit build runs.

:+1: - I've also found myself wanting to do this recently, especially when building plugins that wrap CLI tools.

👍
Just started using the test-kit to try testing a build script and it was immediately obvious that we would want this.  It's not uncommon to replace old bash scripts with gradle scripts, and it's a natural to use env vars as the way to configure build scripts.
It looks like the underlying BuildLauncher already accepts envVars, so it's surprising that it wasn't just added to the TestKit API in the first place since it just would have to get passed down with the other args.

Hey Gradle team, are you OK with allowing the users to pass env variables to TestKit build runs?

Sounds good to me, the underlying Tooling API capability is already there. If you'd like to provide a PR, that would be great.

Can I get an early feedback on the PR #8089? It should be fairly small time commitment. Thank you!


BHBC-db-context: Add db user context pre-step and update db env vars

Overview
This PR contains the following changes

Update the old getDBConnection function in db.ts to now return an object that wraps the raw connection client.

This object has handful of obvious functions for opening a connection, performing queries, and committing/rollingback/releasing the connection
When used, it will automatically set the user context for you, so you dont have to worry about it.

This will make the background functions/triggers set up by Charlie happy

Updated the DB related env vars to be more obvious

This came up because the env var naming was based around 1 user, but now that we have 2 users, it was getting confusing to track which user was being used when (in the github workflows and docker compose, etc)

DB_USER/DB_USER_PASS come from the database-user/database-user-password secrets, and represent the biohubbc-user user
DB_USER_API/DB_USER_API_PASS come from the database-user-api/database-user-api-password secrets, and represent the the biohub_api user
The only exception to the above is in the db setup workflow

In this case, the DB_USER/DB_USER_PASS vars are set from the database-user-root/database-user-root-password secrets, as that is the only user that can currently run the migrations/seeding.  If the biohubbc-user gets those permissions later on, we can easily change this back to it, and we'd only have the 2 above users.

Added a new seed to get our dev team IDIRs added to the system_user table.

As per the comment in that file, that seed will become unneeded once we have the capability in-app to add users.  This is just a stop-gap so that no one testing in dev is blocked in making api calls.

This PR contains the following types of changes

[x] New feature (change which adds functionality)
[ ] Bug fix (change which fixes an issue)
[x] Misc cleanup / Refactoring / Documentation
[ ] Version change

Checklist
A list of items that are good to consider when making any changes.
Note: this list is not exhaustive, and not all items are always applicable.
General

[ ] I have performed a self-review of my own code
[ ] My code follows the style guidelines of this project
[ ] New and existing unit tests pass locally with my changes

Code

[ ] New files/classes/functions have appropriately descriptive names and comment blocks to describe their use/behaviour
[ ] I have avoided duplicating code when possible, moving re-usable pieces into functions
[ ] I have avoided hard-coding values where possible and moved any re-usable constants to a constants file
[ ] My code is as flat as possible (avoids deeply nested if/else blocks, promise chains, etc)
[ ] My code changes account for null/undefined values and handle errors appropriately
[ ] My code uses types/interfaces to help describe values/parameters/etc and inform future developers

Documentation

[ ] I have commented my code sufficiently, such that an unfamiliar developer could understand my code
[ ] I have added/updated README's and related documentation

Tests

[ ] I have added/updated unit tests for any new code I've added/updated
[ ] I have added/updated Postman tests to account for any new/updated code

How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Screenshots
Please add any relevant UI screenshots, if applicable.

I notice that the seeding isnt running (because the command in the dockerfile is only running the migrations).
I made some updates in invasives to toggle between migrations+seeding (for local and pr builds) and just migration (for dev/test/prod) builds. So I can bring that in as well.

I notice that the seeding isnt running (because the command in the dockerfile is only running the migrations).
I made some updates in invasives to toggle between migrations+seeding (for local and pr builds) and just migration (for dev/test/prod) builds. So I can bring that in as well.

Closing and re-opening to kick of fresh github workflows

Closing and re-opening to kick of fresh github workflows


NER returns words with padding chars #

Hi,
I'm using latest Weaviate 1.16.6 with NER module set via Docker
ner-transformers:
    image: semitechnologies/ner-transformers:dbmdz-bert-large-cased-finetuned-conll03-english
    environment:
      ENABLE_CUDA: '0'

I'm getting somewhat unexpected what I would call "partial" tokens as result for e.g. an input document
"Peaceful protest - India - Bengaluru"
it returns (relevant part only)
{
                    "_additional": {
                        "tokens": [
                            {
                                "certainty": 0.996863603591919,
                                "endPosition": 24,
                                "entity": "I-LOC",
                                "property": "name",
                                "startPosition": 19,
                                "word": "India"
                            },
                            {
                                "certainty": 0.9994908571243286,
                                "endPosition": 33,
                                "entity": "I-LOC",
                                "property": "name",
                                "startPosition": 27,
                                "word": "Bengal"
                            },
                            {
                                "certainty": 0.9993565678596497,
                                "endPosition": 36,
                                "entity": "I-LOC",
                                "property": "name",
                                "startPosition": 33,
                                "word": "##uru"
                            }
                        ]
                    }

Putting the same string into Huggingface model UI at least renders "Bengaluru" as one single token being NER tagged.

I asked in Slack and got response from @laura-ham

I think this has to do with entity grouping. There is a parameter introduced to the huggingface model called “grouped_entities”, which can be set to True or False. My guess is that this NER module in Weaviate doesn’t make use of this parameter, which is why the entities are split. See here for more info: https://github.com/huggingface/transformers/pull/3957

I checked the code and @laura-ham is right. One has to explicitly enable grouped_entities during pipeline creation or even better in latest API set the aggregation_strategy .
That alone is not enough: when going for latest version 4.25.1, the response format will slightly being changed when grouping was enabled: https://github.com/huggingface/transformers/blob/main/src/transformers/pipelines/token_classification.py#L202-L203
So, instead of entity the NER tag will be contained in entity_group key.

The question now is:

Go to latest 4.25.1
Should we enable grouping by default? - if not users will have to aggregate the words later on by their selves
Which aggregation strategy to prefer? There are currently five options:

The strategy to fuse (or not) tokens based on the model prediction.
- "none" : Will simply not do any aggregation and simply return raw results from the model
- "simple" : Will attempt to group entities following the default schema. (A, B-TAG), (B, I-TAG), (C,
                  I-TAG), (D, B-TAG2) (E, B-TAG2) will end up being [{"word": ABC, "entity": "TAG"}, {"word": "D",
                  "entity": "TAG2"}, {"word": "E", "entity": "TAG2"}] Notice that two consecutive B tags will end up as
                  different entities. On word based languages, we might end up splitting words undesirably : Imagine
                  Microsoft being tagged as [{"word": "Micro", "entity": "ENTERPRISE"}, {"word": "soft", "entity":
                  "NAME"}]. Look for FIRST, MAX, AVERAGE for ways to mitigate that and disambiguate words (on languages
                  that support that meaning, which is basically tokens separated by a space). These mitigations will
                  only work on real words, "New york" might still be tagged with two different entities.
- "first" : (works only on word based models) Will use the `SIMPLE` strategy except that words, cannot
                  end up with different tags. Words will simply use the tag of the first token of the word when there
                  is ambiguity.
- "average" : (works only on word based models) Will use the `SIMPLE` strategy except that words,
                  cannot end up with different tags. scores will be averaged first across tokens, and then the maximum
                  label is applied.
- "max" : (works only on word based models) Will use the `SIMPLE` strategy except that words, cannot
                  end up with different tags. Word entity will simply be the token with the maximum score.


Add script recommendations

Propose some lines about module scripts.
@netlify /docs/guidelines/components/modules

@nf-core-bot fix linting

@nf-core-bot fix linting

Can we reference the reason why not to use module binaries?
Personally I don't really like the interface to template, and I hate to see R scripts written with base R and the horribly clunky code to allow arguments in that way. I would suggest that we encourage the use of a seqera container with an appropriate interface package (e.g. argparse).
We could do with more explanation as to how to actualy use template, because I don't think we have that here (and the Seqera docs are almost non-existent on the topic).
I much prefer the use of in-line scripts where possible, what is the justification behind saying it can only be 20 lines?

Oh, and some examples pointing to other modules rather than deseq2/differential repeatedly would be good.

Can we reference the reason why not to use module binaries? Personally I don't really like the interface to template, and I hate to see R scripts written with base R and the horribly clunky code to allow arguments in that way. I would suggest that we encourage the use of a seqera container with an appropriate interface package (e.g. argparse). We could do with more explanation as to how to actualy use template, because I don't think we have that here (and the Seqera docs are almost non-existent on the topic).

A PR is in progress to address the gap in the Nextflow docs. This doesn't mean that nf-core can't have its own standards/recommendations, but it will hopefully add something that can be easily referenced.

Great, always happy to see more documentation on the Nextflow page itself.

Can we reference the reason why not to use module binaries?

Apparently you can only use binaries in cloud contexts if you use wave (not other containers), which I feel is overly restrictive.

Personally I don't really like the interface to template, and I hate to see R scripts written with base R and the horribly clunky code to allow arguments in that way. I would suggest that we encourage the use of a seqera container with an appropriate interface package (e.g. argparse). We could do with more explanation as to how to actualy use template, because I don't think we have that here (and the Seqera docs are almost non-existent on the topic).

Agree, more Nextflow docs/example on how to use would be nice, and again, would rather not 'force' Seqera Containers.

I much prefer the use of in-line scripts where possible, what is the justification behind saying it can only be 20 lines?

It's a rule of thumb suggested by @pinin4fjords (I've changed to 'approx' though)

Oh, and some examples pointing to other modules rather than deseq2/differential repeatedly would be good.

Suggest some then! :stuck_out_tongue_closed_eyes:

I suggest removing the repeated "approximately 20 lines" and instead describing this once: something like, "When a script extends beyond a readable length (approximately 20 lines)..."
Then just refer to a readable length for the rest of the text.
Readable length could be "Manageable size", "too verbose", "excessive length", or something like that

I suggest removing the repeated "approximately 20 lines" and instead describing this once: something like, "When a script extends beyond a readable length (approximately 20 lines)..."
Then just refer to a readable length for the rest of the text.
Readable length could be "Manageable size", "too verbose", "excessive length", or something like that

Done!

Much better - thanks!

Is everyone reasonably happy with the current PR? Are there any blockers to merging?


[BUG]: bot not firing?

Is there an existing issue for this bug?

[X] I have searched for this issue

Description of the bug
for some reason the bot doesn't seem to be working. I put 3 requests in, two of which have passed and should have fired. yet when I request a list, it says all three are still in progress.

Steps to reproduce
I used semma and via carota as the two restaurants, filled in reqD of 2.15 and 2.17 at midnight. now 3pm on 2.17 and neither have fired.
System Specs
macos ventura
Additional Context
No response

Did you run list twice? list is not an in-place operation, i.e., you must run the command again to see if the status of anything updates

Yes, of course.
Best,
Dora
914-523-7381
On Tue, Feb 20, 2024 at 12:40 PM Bruce Jagid @.***>
wrote:

Did you run list twice? list is not an in-place operation, i.e., you must
run the command again to see if the status of anything updates
—
Reply to this email directly, view it on GitHub
https://github.com/21Bruce/resolved-bot/issues/33#issuecomment-1954719093,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BGHN5645RBTFME7FXJ67CQDYUTNYDAVCNFSM6AAAAABDNRZQO2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJUG4YTSMBZGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

@DoraGor  Both me and the other dev have been really busy with school stuff the past two weekends. I'm unsure what's causing this new bug, could be resy using captcha or a bug in the new table type code. Either way, I think you should try cloning an older version of the bot. I think commit 60e88dd1e92d5a023485ee04f84b521b187e223e should be working if it were an issue with the table type code. If you try this and it doesn't work, then I'd really have to sit down and analyze the network response from the api layer


Set Up the SAP Launchpad Service

Tutorials: https://developers.sap.com/tutorials/cp-portal-cloud-foundry-getting-started.html
Step 2: Add yourself to the Launchpad_Admin role
Please change the steps to add Launchpad_Admin role. Because in  Security > Trust Configuration Default Identity provider link is disabled as you can see in the below screenshot.

Change those step to Security > Users > Select your user > Scroll to Role Collections > Click on 3 dots > Click on Assign Role Collection > Select Launchpad_Admin > Click Assign Role Collection button.
Thanks and Regards
Shivakumar P

This issue is fixed


Move the fixupLangCode helper function into the web/genericl10n.js file

This helper function was added almost two years ago, in PR #13696, and it still has only a single call-site. Furthermore, with the changes made in PR #16572 it also cannot hurt to reduce the size of the web/l10n_utils.js file slightly.

/botio integrationtest


html gallery file not found

Lost css/valiant360.css required file.

<IP_ADDRESS> - - [27/May/2017 23:59:12] code 404, message File not found
<IP_ADDRESS> - - [27/May/2017 23:59:12] "GET /css/valiant360.css HTTP/1.1" 404 -

Fixed.


Improve how tests handle restarting workers

Instead of relying on waiting for lost worker detection, force the workers to be lost.

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

jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/8653/
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/8654/
Test PASSed.

alluxio-bot, merge this please


caasp: Update CaaSP nodes before enrolling

This reduces some complexity as will rely on fewer parts of the velum
automation. It fixes the current deployment issues we are seeing in the
update-caasp playbook.
It should also speed up the deployment a bit since we can run e.g. the
reboot for all nodes in parallel.

@rhafer travis is complaining...

@rhafer travis is complaining...

Yeah. Let's see if the latest change is any better.

It is better indeed :)


OTP Reuse?

Does the two-factor authentication allow reuse of the OTP? I don't see anything in the database changing after signing in through 2FA. This was handled in devise-two-factor-auth by adding a consumed_timestep column in https://github.com/devise-two-factor/devise-two-factor/pull/43. See this post for details.
Should authentication-zero do something similar?
Great work on this gem btw!

Looking into it further, looks like rotp supports an after option. It looks fairly simple to add an otp_consumed_at column to user, pass that in to rotp verify, and update it whenever signing in with otp.

Yeah, I haven't implemented it as a matter of simplicity. So, yes, you can reuse the same OTP  inside the 30 seconds, I don't see it as a big issue because we use the OTP codes as 2FA, but I would be happy to merge a simple implementation for that.

Yeah, I haven't implemented it as a matter of simplicity. So, yes, you can reuse the same OTP inside the 30 seconds, I don't see it as a big issue because we use the OTP codes as 2FA, but I would be happy to merge a simple implementation for that.

I agree it isn't a serious security issue, and if the goal is simplicity I can understand intentionally leaving it out. I just wanted to raise it here in case it was missed. Feel free to close this issue.

I will re-evaluate it though...


Error when adding existing project with Paket to the solution

I've encountered an error message box in Visual Studio when trying to move project folder (having Paket already integrated) into a different directory and adding it again to the solution. Error message:

The imported project "<project_path>\...paket\paket.targets" was not found. Confirm that the path in the <Import> declaration is correct, and that the file exists on disk. <csproj_path>

My guess is that after adding an existing project with <Import Project="relative path to paket.targets" /> already set on, Paket won't try to readjust the path to the current solution, target project has been imported to. So if the project folder was moved or copied, path to paket.targets becomes invalid and an error message will be thrown.

Usually a run of "paket install" should fix that. Any chance for me to
reproduce?
On Oct 12, 2015 7:22 PM, "Bartosz Sypytkowski"<EMAIL_ADDRESS>wrote:

I've encountered an error when trying to move project folder (having Paket
integrated) into a different directory and adding it again to the solution.
Error message:
The imported project "<project_path>...paket\paket.targets" was not
found. Confirm that the path in the  declaration is correct, and
that the file exists on disk. <csproj_path>
My guess is that after adding an existing project with  already set on, Paket won't
try to readjust the path to the current solution, target project has been
imported to. So if the project folder was moved or copied, path to
paket.targets becomes invalid and an error message will be thrown.
—
Reply to this email directly or view it on GitHub
https://github.com/fsprojects/Paket/issues/1131.

The problem is that Visual Studio is rejecting adding existing project in the first place. I think that basic reproduce path could be made like that:
Given following directory structure:
root
    \.paket folder
    \src
        \folderA
            \A.csproj (with Paket <Import> set)
        \*.sln file

, try to move folderA with csproj like the following:
root
    \.paket folder
    \src
        \folderB
            \folderA
                A.csproj
        \*.sln file

To mirror changes, you'd need to remove A.csproj from the solution and add it again as the existing project. Then the error should occur (I'm using VS2015). Since path to paket.targets is set in the csproj file and it's relative to it, it's pointing to the invalid directory.

Just run paket install before you readd the project. Paket doesn't care if
the project is referenced in a sln. It should fix all paths and then you
can add it again.
On Oct 12, 2015 7:44 PM, "Bartosz Sypytkowski"<EMAIL_ADDRESS>wrote:
The problem is that Visual Studio is rejecting adding existing project in
the first place. I think that basic reproduce path could be made like that:
Given following directory structure:
root
.paket folder
\src
\folderA
\A.csproj (with Paket  set)
*.sln file
, try to move folderA with csproj like the following:
root
.paket folder
\src
\folderB
\folderA
A.csproj
*.sln file
To mirror changes, you'd need to remove A.csproj from the solution and add
it again as the existing project. Then the error should occur (I'm using
VS2015). Since path to paket.targets is set in the csproj file and it's
relative to it, it's pointing to the invalid directory.
—
Reply to this email directly or view it on GitHub
https://github.com/fsprojects/Paket/issues/1131#issuecomment-147472702.


Create CLI help arguments

Kind of essential. When the command is wrong or when the user uses --help, print the help dialog.

Added. Based them on the npm command line help.


Battery voltage incorrect PSM mode

Hi,
I'm using the 2022-10-22 version to create a firmware for my CC2530 modules.
In PSM mode the battery status is sent now in the newest version but it is incorrect. The firmware is sending battery: 100 and voltage: 3400 regardless of the power supply voltage. I tried voltages between 2.5V - 3.3V
Do you know what can be the issue?

I cannot confirm it. I've tested the firmware with 2.5 and 3.0V. It reports voltage correctly.


pipelines: Use variables to remove Windows workspaces

Ref: #2205 , #1956 , #2204 , #2400
The plan is to use a variable to remove the problematic Windows workspaces, as we have an awful lot of hard coded implementations now. PRs in draft until I can figure out what variables can actually be used!
ping @M-Davies , @Haroon-Khel , @sxa
Signed-off-by: Will Parker<EMAIL_ADDRESS>

run tests

run tests

Ah hang on- the Compile Groovy check has at least picked up that ${WORKSPACE} is undefined. Removing now

Ah hang on- the Compile Groovy check has at least picked up that ${WORKSPACE} is undefined. Removing now

Can you use the variable that the value of CYGWIN_WORKSPACE gets assigned to when it's defined?

Can you use the variable that the value of CYGWIN_WORKSPACE gets assigned to when it's defined?

I'll give it a go :-) the Groovy check will fail if the variable is undefined, so it's worth a go! :+1:

I'll give it a go :-) the Groovy check will fail if the variable is undefined, so it's worth a go! :+1:

Okay, setting ${workspace} didn't work. I'll try ${env.CYGWIN_WORKSPACE}.

Okay, setting ${workspace} didn't work. I'll try ${env.CYGWIN_WORKSPACE}.

run tests

Edit: Hmm, it appears I can't do that. @M-Davies would you mind? Also, is there a list of people who have access anywhere? I looked at the README.md you linked @Haroon-Khel , which alludes to it, but doesn't say where it is :-)

https://github.com/AdoptOpenJDK/openjdk-build/blob/master/pipelines/build/prTester/README.md#add-to-whitelist. The last bullet links you to the https://ci.adoptopenjdk.net/job/build-scripts-pr-tester/job/openjdk-build-pr-tester/ job: 🙂

Should you want the up to date admin or white list, check the configuration of the openjdk-build-pr-tester job. If you don't have the permissions to view the configuration, then try out the add to whitelist and run tests commands on a test PR to see if they work.

You might want to try env.CYGWIN_WORKSPACE and CYGWIN_WORKSPACE again on the pr tester itself. The Compile Groovy step only checks if the variables/functions exist in https://github.com/AdoptOpenJDK/openjdk-build/tree/master/pipelines/src/main/groovy and so are unrelated to what Jenkins variables we have on the server

run tests

Edit: Hmm, it appears I can't do that. @M-Davies would you mind? Also, is there a list of people who have access anywhere? I looked at the README.md you linked @Haroon-Khel , which alludes to it, but doesn't say where it is :-)

https://github.com/AdoptOpenJDK/openjdk-build/blob/master/pipelines/build/prTester/README.md#add-to-whitelist. The last bullet links you to the https://ci.adoptopenjdk.net/job/build-scripts-pr-tester/job/openjdk-build-pr-tester/ job: 🙂

Should you want the up to date admin or white list, check the configuration of the openjdk-build-pr-tester job. If you don't have the permissions to view the configuration, then try out the add to whitelist and run tests commands on a test PR to see if they work.

You might want to try env.CYGWIN_WORKSPACE and CYGWIN_WORKSPACE again on the pr tester itself. The Compile Groovy step only checks if the variables/functions exist in https://github.com/AdoptOpenJDK/openjdk-build/tree/master/pipelines/src/main/groovy and so are unrelated to what Jenkins variables we have on the server

add to whitelist

add to whitelist

run tests

run tests

So, judging the PR-Build Tester, specifically (https://ci.adoptopenjdk.net/job/build-scripts-pr-tester/job/build-test/job/jobs/job/jdk11u/job/jdk11u-windows-x64-hotspot/481/consoleFull) , it seems to have passed, however I can't find any output in the log indicating that it did what it was meant to :sweat_smile:

So, judging the PR-Build Tester, specifically (https://ci.adoptopenjdk.net/job/build-scripts-pr-tester/job/build-test/job/jobs/job/jdk11u/job/jdk11u-windows-x64-hotspot/481/consoleFull) , it seems to have passed, however I can't find any output in the log indicating that it did what it was meant to :sweat_smile:

FYI This may be rendered irrelevant by https://github.com/AdoptOpenJDK/openjdk-build/pull/2416

FYI This may be rendered irrelevant by https://github.com/AdoptOpenJDK/openjdk-build/pull/2416

In case the #2416 doesn't fix the issue, I'll mark this as ready to review, as I think this fixes the issue :-)

Yeah, the mention PR was merged, so I'll close this one :+1:


coreir distribution crashes when I try to use it?

@rdaly525 I've been experimenting with a JIT specializer for the CGRA in coreir.
I'm trying to benchmark it on kiwi and would like to get it building on travis so that I can debug linux compilation problems before running on kiwi (which is a pain to use via ssh from my mac).
I'm trying to download coreir as one of the dependencies. I can download the tarball, unpack it, and install coreir, but when I try to build the JIT on travis I get an error in the coreir json parser that my gcc version is not supported (even though I am setting COREIRCONFIG="g++4.9" in my travis file.
Any idea what might be wrong here?
Link to travis build

I think the issue is that your cmake command is finding/using gcc 4.8. If we inspect the output of the cmake command we see
$ cmake .
-- The C compiler identification is GNU 4.8.4
-- The CXX compiler identification is GNU 4.8.4
-- Check for working C compiler: /usr/bin/gcc
-- Check for working C compiler: /usr/bin/gcc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Check for working CXX compiler: /usr/bin/g++
-- Check for working CXX compiler: /usr/bin/g++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Configuring done
-- Generating done
-- Build files have been written to: /home/travis/build/dillonhuff/FlatCircuit

One work around for this is if you want to point CMAKE to a specific compiler, you can try
cmake -DCMAKE_C_COMPILER=/usr/bin/gcc-4.9 -DCMAKE_CXX_COMPILER=/usr/bin/g++-4.9 .

or you can try
CC=gcc-4.9 CXX=gcc-4.9 cmake .

When compiling your project, it tries pulls in the /usr/include/coreir/ir/json.h header file which has a gcc version requirement

I tried the CMake change and it worked. Thanks!


Refactor quality report generation to use wildcards

Change the generation of quality reports to a callback function that determines what FASTQC files must be generated. FASTQC files are generated one at a time, and one FASTQC job is spawned per raw or qced FASTQ file. Logic in the callback function allows for creating FASTQC files for adapter trimming and decontamination only when these steps occur. This addresses concerns brought up in https://github.com/rotary-genomics/rotary/pull/161#pullrequestreview-2087595363.

@jmtsuji This addresses the issue where FASTQC would be run on qc stages turned off in the config (e.g., adapter trimming and contamination filtering). This is determined inside the generate_fastqc_file_list callback function.
This code needs to be tested live; I've only tested it with a dry run.

I had to change the directory structure of the raw read folder by splitting it into short and long read sub-directories. This made it more compatible with wild cards.
@jmtsuji Note that there are two rules for running fastqc (raw and quality controlled) right now. I had trouble combining them and getting the wild cards to work. To merge the two rules, I would have to drastically change the names of the output QC files. After several attempts, I have decided to leave these rules as is.

@jmtsuji Looking forward to your review! Any feedback would be appreciated.

@LeeBergstrand This is a nice addition to allow some QC steps to be optional and still have FastQC / MultiQC work. The logic of generate_fastqc_file_list and its interactions with the previous and subsequent rules is pretty dense, so it's hard for me to gauge how smoothly the function will work without a test using real data. Because I don't see any clear issues with the code, I will go ahead and approve this. However, we should make sure to test this with a real dataset (e.g., with various QC steps toggled on/off) later, before merging to main, to confirm there are no issues. Sounds good? Thanks again for this code addition?
P.S. I think it's OK to leave the 2 separate functions for FastQC like you've currently done 👍

So FASTQC makes html and zip files. This function basically generates what FASTQC files will be produced depending on the QC stages ran for both short and long reads. It uses the wild cards to figure out if it is being invoked for short on long reads. If it is short, it creates each of the outputs for R1 and R2 files and if long only the outputs for long files. For the short files it creates additional output for the contamination and adapter trimming qc stages if they used. As indicated by global variables.

@jmtsuji Will merge in and we can test later. I've done some limited testing where I remove all the contamination references.

@LeeBergstrand Thanks for the additional background! Also, I'm glad this is working when all contamination references are removed.
Sounds good to merge for now. We can check this code in more detail via testing later on.


remove unwanted modules and dependencies

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


Why is my training process not complete？

while parsing CIF: Some fractional coordinates rounded to ideal values to avoid issues with finite precision.
  warnings.warn("Issues encountered while parsing CIF: " + "\n".join(self.warnings))
100%|█████████▉| 11351/11356 [09:20<00:00, 26.38it/s]

what should I do?

Besides， when I execute the 'stop run' command, the result is
forrtl: error (200): program aborting due to control-C event
Image              PC                Routine            Line        Source
libifcoremd.dll    00007FFC3D1EDF54  Unknown               Unknown  Unknown
KERNELBASE.dll     00007FFCA1D3B943  Unknown               Unknown  Unknown
KERNEL32.DLL       00007FFCA2F67034  Unknown               Unknown  Unknown
ntdll.dll          00007FFCA44A26A1  Unknown               Unknown  Unknown

If I execute the 'stop run' command, is the training process complete?


PathTemplate not escaping separators in complex resource IDs

PathTemplates using complex resource IDs does not escape complex separators.
The following code fails for version 2.2.1.
Code example
import com.google.api.pathtemplate.PathTemplate;
import org.junit.jupiter.api.Assertions;
import org.junit.jupiter.api.Test;

class PathTemplateTest {

    @Test
    void test() {
        PathTemplate template = PathTemplate.create("resources/{foo}_{bar}");
        Assertions.assertEquals("resources/abc%5F123_def%5F456", template.instantiate("foo", "abc_123", "bar", "def_456"));
    }
}

Stack trace
org.opentest4j.AssertionFailedError:
Expected :resources/abc%5F123_def%5F456
Actual   :resources/abc_123_def_456

Are you assuming that they should be escaped? Is that documented behavior somewhere?

Closing as "intended behavior". Please re-open if you disagree.


barcodeRead is always false even after scanning

To Reproduce
import React, { useRef, useCallback } from 'react';
import { View, StyleSheet } from 'react-native';
import { RNCamera } from 'react-native-camera';
import { BarcodeMask, useBarcodeRead } from '@nartc/react-native-barcode-mask';

function CameraScreen({ navigation }) {
  const camera = useRef(null);

  const handleBarcodeData = useCallback(
    barcodeData => {
      console.log('barcodeData', barcodeData);
      // navigation.goBack();
    },
    [navigation],
  );

  const {
    barcodeRead,
    onBarcodeRead,
    onBarcodeFinderLayoutChange,
  } = useBarcodeRead(
    navigation.isFocused(),
    barcodeData => barcodeData,
    handleBarcodeData,
  );

  console.log('barcodeRead', barcodeRead);

  return (
    <View style={styles.container}>
      <RNCamera
        ref={camera}
        style={{
          flex: 1,
          width: '100%',
        }}
        captureAudio={false}
        onBarCodeRead={onBarcodeRead}
        barcodeTypes={barcodeRead ? [] : [RNCamera.Constants.BarCodeType.qr]}
      >
        <BarcodeMask
          onLayoutChange={onBarcodeFinderLayoutChange}
          maskOpacity={0.5}
          animatedLineColor="#f00"
        />
      </RNCamera>
    </View>
  );
}

const styles = StyleSheet.create({
  container: {
    flex: 1,
    alignItems: 'center',
    justifyContent: 'center',
  },
});

export default CameraScreen;

Expected behavior
console.log() message should be
                                                  (before scanning)
barcodeRead false
                                                  (after scanning)
barcodeData 1234
barcodeRead true
                                                  (stop)

However barcodeRead is always false, so it does not change barcodeTypes prop.
                                                  (before scanning)
barcodeRead false
                                                  (after scanning)
barcodeData 1234
barcodeRead false
barcodeData 1234
barcodeRead false
barcodeData 1234
barcodeRead false
...                                               (repeat)

Smartphone (please complete the following information):

Device: iPhone X
OS: iOS 13
react-native 0.61, react-native-camera 3.16
1.1.6

@taeingyu Please check out the latest release to see if it works out for you. Please read the README regarding barcodeRead changed.

:tada: This issue has been resolved in version 1.1.7 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

@taeingyu Please check out the latest release to see if it works out for you. Please read the README regarding barcodeRead changed.

It works great! and the new barcodeReadDelay is just the feature I was looking for. Thanks!


Fixed show/hide cursor (windows)

These methods cause a panic due the the fact that ShowCursor and HideCursor are not methods in kernel32.dll
Additionally the initSyscall function was never being invoked.
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x20 pc=0x7ee824]

goroutine 1 [running]:
syscall.(*LazyProc).Find(0x1000000009826c0?)
        C:/Program Files/Go/src/syscall/dll_windows.go:245 +0x24
syscall.(*LazyProc).mustFind(...)
        C:/Program Files/Go/src/syscall/dll_windows.go:267
syscall.(*LazyProc).Call(0x0, {0x0, 0x0, 0x0})
        C:/Program Files/Go/src/syscall/dll_windows.go:285 +0x28
github.com/hedzr/progressbar/cursor.hideCursor()
        C:/dev/progressbar-orig/cursor/cursor_windows.go:109 +0x25
github.com/hedzr/progressbar/cursor.Hide(...)
        C:/dev/progressbar-orig/cursor/cursor.go:7
main.main()
        C:/dev/progressbar-orig/examples/schema/schema.go:66 +0x25

Pull Request Test Coverage Report for Build<PHONE_NUMBER>9
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
46

Relevant Lines:
673

💛  - Coveralls


[cloud_functions] Error being swallowed by Future

This is returning a promise that is not resolved, so it will never fall into that catch and it never returns a Cloud Functions Exception.
https://github.com/FirebaseExtended/flutterfire/blob/bdb95fcacf7cf077d162d2f267eee54a8b0be3bc/packages/cloud_functions/cloud_functions/lib/src/https_callable.dart#L42-L53|

I think this is a valid question - the code needs to be using await for the catch to throw. Right now it's being swallowed as an unhandled exception in the Future.

Exactly, catch is being ignored.

This was fixed in the reworked version last year. Thanks


SA1516 is reported for the declarator, not the identifer

If SA1516 should be reported for this code:
private static readonly bool foo = false;

Then "foo = false" is the location for the diagnostic. I think reporting on just "foo" would be better.

Agreed. In addition, if multiple variable declarators are included, the warning should probably only be reported for the first of them.
Another option is only reporting it for the field type and/or modifiers, since those appear before the first variable and only appear once.

Is this superseded by #562?


Hunger games asks me about french products

What

App settings: DE_de
When using hungergames I get asked about products which only have french photos
Guess we don't have or have broken localization

Duplicate with #4122.

Thanks @monsieurtanuki I'm lacking behind with my github notifications


Version for lumen

Package works with lumen, but only if composer.json dep "laravel/horizon": "^1.4", is changed from official horizon to "kinsolee/horizon-lumen": "^1.2" (It is port for lumen)
Is there possibility to make this package also ready for lumen ? Or should I create separate package called "lumen-horizon-rabbitmq" ?

It doesn’t look like the kinsolee/horizon-lumen is versioned against the official horizon package. I think the best option here is for you to create a lumen-horizon-rabbitmq fork so this package remains being safely versioned against the official laravel framework and laravel horizon packages.


Basically unusable for multiple tenants with multiple users

We have two different tenants in two different datacenters. I have two different users to go with that and I have to switch contexts all the time. When I switched from one datacenter to the other using kubectl the user experience with the azure auth plugin was that I need to log in once for both the first time I switch and then I can seamlessly switch between the two without re-authenticating every time, because the tokens were saved individually for both clusters, so kubectl always knew which  token to use for which context. That is impossible to set up with kubelogin. When I convert the config, everything is converted but the two contexts will use the same token, and only the one I was using at the time of convert will work, the other context will provide an authorization error. Kubelogin is unable to figure out that I had multiple users for the different tenants. To fix it, I need to log out, remove the tokens from kubelogin, re-authenticate, convert the config again. And I have to do these steps every single time I switch contexts. It's awful.
The azure auth plugin had sooo much better user experience... :(
Can something be done about this?

did you try to specify --token-cache-dir in convert-kubeconfig? i suppose you can create unique temp dir using mktemp to store the cached token

I'm sorry I don't understand how using a token cache dir would help me in this? From the documentation it is not clear to me.
What I have problem with is the following:

I have logged in to two different tenants with: az login --tenant A then, az login --tenant B,
az account set --tenant A
az aks get-credentials --name clusterA --resource-group rgA
kubectl get pods -> I get the pods
az account set --tenant B
az aks get-credentials --name clusterB --resource-group rgB
kubectl get pods -> I get the pods
kubectl config use-context clusterA, then get pods -> I get the pods
kubectl config use-context clusterB, then get pods -> I get the pods.

Now, when I try to use kubelogin I have to do the following:
10. az account set --tenant A
11. kubelogin convert-kubeconfig -l azurecli
12. kubectl get pods -> I get the pods
13. az account set --tenant B
14. kubectl config use-context clusterB, then get pods -> error: You must be logged in to the server (Unauthorized)
15. kubelogin remove-tokens
16. kubectl get pods -> I get the pods
So essentially instead of just switching seamlessly like in step 8 and 9, I have to call az account set and kubelogin remove-tokens every time I switch contexts, which is very annoying and also much slower like previously. I don't see how the token-cache-dir helps with this...

please look at this proposal, this would work, at least it works for me when i manually edited the kubeconfig

az login --tenant A
az login --tenant B,
az account set --tenant A
az aks get-credentials --name clusterA --resource-group rgA

az account set --tenant B

az aks get-credentials --name clusterB --resource-group rgB
kubelogin convert-kubeconfig -l devicecode --context clusterA --token-cache-dir $(mktemp -d)
kubelogin convert-kubeconfig -l devicecode --context clusterB --token-cache-dir $(mktemp -d)

after this, you can change contexts without interfering each other.
you can confirm this proposal by editing the kubeconfig to add unique token-cache-dir, but you have to use devicecode login

I have to side with @SebestyenBartha on this - kubelogin is almost useless. I can connect fine to AKS clusters but not to our legacy k8s 1.17, 1.18 local clusters nor the legacy clusters on Azure. For the legacy clusters the process is kubectl use-context... or kubectx clustername and k9s. Once kubelogin is used as minimally directed nothing can connect to the legacy clusters at all, including kubectl cluster-info.

@ScottS-byte would you mind elaborating? does your legacy k8s use AAD or not? if you use kubelogin convert-kubeconfig --context ..., it will only change the config for selected context. the rest should remain the same, and you can switch by using kubectl config use-context XYZ


MudTimePicker Value Parsing

Feature request type
Enhance component
Component name
MudTimePicker
Is your feature request related to a problem?
The current implementation for the OnGet() method on the MudTimePicker class is pretty limited on what it will support. End user experience is negatively impacted when they type in "2 pm" or "3:00 AM" and it throws an error such as: Conversion from String to Nullable`1 failed: String '3:00 AM' was not recognized as a valid DateTime.
Describe the solution you'd like
The following input should successfully be parsed as valid input no matter what the current TimeFormat or AmPm properties are set to:
[UserInput] => [ParsedTimeSpan]

2p => 14:00:00
5 am => 05:00:00
5:21 pm => 17:21:00
14:21 => 14:21:00
6 => 06:00:00
16:15 pm => 16:15:00, or error if that makes more sense.

It would be great to introduce a property of CoerceText to ensure that whatever the end user types in, the displayed text after they finish is shifted back to match the TimeFormat or AmPm properties.
This, for example, comes close to what I would expect:
private readonly static Regex pmAmRegex = new Regex("([^pa]+)(p|a)?", RegexOptions.IgnoreCase);

private TimeSpan? OnGet(string value)
{
    if (string.IsNullOrEmpty(value))
        return null;

    var match = pmAmRegex.Match(value);
    if (!match.Success)
        throw new Exception($"Could not extract time from provided input: {value}");

    var timePart = match.Groups[1].Value.Trim();
    // To avoid "1" being interpreted as 1 day:
    if (timePart.IndexOf(":") == -1)
        timePart += ":00";

    if (!TimeSpan.TryParse(timePart, out var timeSpan))
        throw new Exception($"Could not extract time from provided input: {value}");

    if (timeSpan.TotalHours < 12 && match.Groups[2].Success && match.Groups[2].Value.Contains("p", StringComparison.CurrentCultureIgnoreCase))
        timeSpan = timeSpan.Add(TimeSpan.FromHours(12));

    // Potentially assert that timeSpan.TotalDays < 1 here

    // If (CoerceText == true)
    //    Text = Converter.Set(timeSpan);

    return timeSpan;
}

Have you seen this feature anywhere else?
No response
Describe alternatives you've considered
I can pass in a custom converter, but then I have to pass additional context like the proper TimeFormat to the custom converter. This gets awkward if I want to use the MudTimePicker's default time format.
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I don't think the disk display/select mode is very good for pick time component.


Avoid using require to read package version

Using this library with Webpack and yarn PnP results in the following warning:
../../../.yarn/__virtual__/@google-cloud-logging-winston-virtual-bfe17c7235/0/cache/<EMAIL_ADDRESS>Critical dependency: the request of a dependency is an expression

Import trace for requested module:
../../../.yarn/__virtual__/@google-cloud-logging-winston-virtual-bfe17c7235/0/cache/<EMAIL_ADDRESS>../../../.yarn/__virtual__/@google-cloud-logging-winston-virtual-bfe17c7235/0/cache/<EMAIL_ADDRESS>
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[x] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[x] Code coverage does not decrease (if any source code was changed)
[x] Appropriate docs were updated (if necessary)

Fixes #762

Sounds great, thanks!


Add 'plugin:security/recommended' config to plugin

Usage:
{
  "env": {
    "es6": true,
    "node": true
  },
  "extends": [
    "eslint:recommended",
    "plugin:security/recommended"
  ],
  "rules": {
    "security/detect-object-injection": "off"
  }
}

Cool beans. We don't need to be explicit for the other rules?

All the rules in "plugin:security/recommended" will "warn" by default, I was just showing how you could override rules to off/error in your local ESLint configs since that rule seems to be the noisiest of the bunch.
xoxo,
-peter


Exceptions thrown during use (Listed below when found)

In KryptonDataGridView.cs line 1153 state = normal

In AccurateText.cs line 305
                            Color tmpColor = tmpBrush?.Color ?? SystemColors.ActiveCaptionText;

Wrong project, this is for the Components not the extensions


Removed duplicate function declaration. Error in Google Closure Compiler

This fix is commited on top of the 5.1.1 tag
Google Closure Compiler complains about this:
ERROR - Duplicate let / const / class / function declaration in the same scope is not allowed.
function badIndex(index, nestedProblems)
         ^^^^^^^^

The function badIndex is declared twice in the Native.Json module.
This looks like a PR merge mistake so I just removed the duplicate declaration.

This should be removed in the development branch of the library. Thanks for the report though!


Feature/extend config with stylelint

Extended eslint config with stylelint, typescript parser and prettier
Updated deps
Migrated to Eslint v9

This shouldn't be bundled with eslint as eslint is only for TS/JS code so I've made it it's own package https://github.com/innovixx/stylelint-config


multiple redirect components in Switch cause Uncaught Invariant Violation: Maximum update depth exceeded

Version
5.0.0
Test Case

Steps to reproduce
just click anywhere in the codesanbox app
Expected Behavior
Redirect is triggered and app changes route correctly to /
Actual Behavior
Uncaught Invariant Violation: Maximum update depth exceeded. This can happen when a component repeatedly calls setState inside componentWillUpdate or componentDidUpdate. React limits the number of nested updates to prevent infinite loops.
There was also a brief conversation on twitter about this bug with original author of the codesandbox: https://twitter.com/capajj/status/1110332407293665280

The multiple Redirects are causing each one to loop back on the other, so this makes sense. Instead, you should move them up outside of the Switch to one that wraps the whole animated component.


the confirm email link it is not working

the confirm email link it is not working, it direct me to undefined link;
http://undefined/new-verification?token=******

sorry about that i mostly focus on some section like

/
/settings
/login
/chat/copilot
/new
/:user
/:user/:repo
/:user/:repo/issue
/:user/:repo/issue/new
and more

but i need to focus on signup page that page contain many bugs and issue i want something deferent for that page that's why i not working on it now.
but today i starting working on it.
for now i give u permission to login to your account i edit email verification via DB changes.
thanks for visiting and make this issue, happy coding @AdelEddarai


Video support example shows Google Map example

Hello,
First of all, I'd like to congratulate you for the project. I really like it.
Since I don't have a ReadMe.io account (and I don't want it), I'm opening this issue to warn about the duplication: Video support example shows the same content from Google Map example in the docs page.

fixed.

Thank you @ritz078


401 Unauthorized Errors

The site is set up on my local machine. I'm using Valet. Followed the installation commands and configured the .env. Can you help me figure this out? Thanks.

I got it working by not using Valet. Thanks.

I was wrong. Still getting the same issues. Sorry for closing and reopening.

Have you followed the instructions for Laravel Passport as well?

php artisan passport:install ? Of course. Is there something I'm missing? Am I required to authenticate with the voten.co public API to get this working?

Have you followed the instructions for Laravel Passport as well?

I'm having the same issue and i have not found any Passport instructions you are talking about, please link them?
Thank you.

@LEVATHIAN
php artisan key:generate
php artisan migrate
php artisan passport:install

php artisan passport:install

Still getting the same issue

php artisan passport:install

Still getting the same issue

sullyfischer dont help fix problem.

@amiller911 what?!

@LEVATHIAN please look it up as a "Laravel Passport" issue on the net

@LEVATHIAN please look it up as a "Laravel Passport" issue on the net

I have been trying to fix the problem for 2 weeks with no resulsts, i looked up all possible solutions.

Having the same issue, did try all the commands listed above and in the installation instructions.

Looks like this is the issue: https://github.com/laravel/passport/issues/795
TokenGuard.php in passport expects jwt token to be serialized, but laravel has serialize set to false by default, so there's an exception thrown from unserialize.

Somehow I still can't figure out how to fix this. Any hints?

Somehow I still can't figure out how to fix this. Any hints?

Upgrade to Passport 7.0 and Laravel 5.7.x.

"laravel/framework": "5.7.*",
"laravel/passport": "^7.0",
"pbmedia/laravel-ffmpeg": "^3.0.0",
"php-ffmpeg/php-ffmpeg": "dev-master#c11b79ab5b0174aa1a56c54c67491169e78a4c17",

This worked perfectly for me! Thanks @BoKKeR


jfrog cli error "M2_HOME is not set" in GitHub action/workflow

Describe the bug
This notice for GitHub workflow/action virtual runners says the M2_HOME env variable is no longer getting set: https://github.com/actions/virtual-environments/issues/2600
Unfortunately, the command below fails with an error saying it is required:
jfrog rt mvn clean install --batch-mode -s settings.xml mavenBuildConfig.yaml --build-name=my_build --build-number=0.0.1-SNAPSHOT
Error message: Error:  M2_HOME environment variable is not set
To Reproduce
Run the command above in a github action/workflow. (But I haven't tried it in a scaled down workflow.)
Expected behavior
It should run maven
Screenshots
If applicable, add screenshots to help explain your problem.
Versions

JFrog CLI version: jfrog version 1.44.0
JFrog CLI operating system: Ubuntu 20.04
Artifactory Version: n/a

Additional context
Add any other context about the problem here.

When I add this to a previous step in the workflow, it does not fail:
echo "M2_HOME=/usr/share/apache-maven-3.6.3" >> $GITHUB_ENV

@leemeador,
Having the M2_HOME environment variable set has always been a requirement of the jfrog rt mvn command. Nothing changed on JFrog CLI.
I assume something changed on the workflow that runs the command. I understand that adding the env var resolves the issue.

@eyalbe4  Exactly. Thanks for the quick reply.
That's why I included the link to GitHub's issue where they were removing M2_HOME.
I'm asking for you guys to change JFrog CLI to not need M2_HOME so that we don't have to set it and everyone that is new to writing actions doesn't have to search all over the internet to not find any info about it. I do know that lots of maven related tools do not use that variable OR maybe they use it if its there and do something else if its missing.

@eyalbe4 - this issue of CLI expecting M2_HOME is seriously affecting us. Is there a fix in progress?

It may be that the image you are using doesn't have maven installed or that M2_HOME needs to be set. Here is what I did to fix the issue.
.artifactory:
image: $ARTIFACTORY_IMAGE
stage: deploy
services:
- docker:dind

before_script:
# Install curl
- export DEBIAN_FRONTEND=noninteractive
# Configure Artifactory instance
- jfrog c add artifactory --artifactory-url=$JFROG_ARTIFACTORY_URL --access-token $JFROG_ARTIFACTORY_ACCESS_TOKEN --interactive=false
# Configure maven
- apt update
- apt -y install maven
- mvn -v
- echo "$M2_HOME"
- export M2_HOME=/usr/share/maven
- echo "$M2_HOME"


Using Razorpay in my react app

Hi,
I tried integrating the razorpay as per your documentation and I got the below error
Access to fetch at 'https://api.razorpay.com/v1/orders' from origin 'http://localhost:8000' has been blocked by CORS policy: Response to preflight request doesn't pass access control check: It does not have HTTP ok status.
Code that I used is as follows
const razorpay = new Razorpay({
  key_id: "some_id",
  key_secret: "some_scret",
})

const orderData = {
  amount: '100',
  currency: "INR",
  receipt: 'XYZ-123',
  payment_capture: "1",
}

const orderCreationResponse = await razorpay.orders.create(orderData)

The Orders API is not meant to be used from the frontend. Any API that needs your key_secret is not meant to be used from the frontend.

@umanghome thank you. Will use this in the backend and try. :)


What offsets do i have to update?

I just wanted to ask what im suppost to update becouse this sln is weird and hard to understand, like why is the offset file called monkeware keys??? H E L P !

You need update everything, dont use this paste thanks :), its hard for the beginner people. Its just for learning but is so missed up.