text:
fix: fix delete vector bug

The previous deletion logic was by index, and there would be a bug when deleting it after sorting

@Sherlocksuper what about deletion for other objects?

authors: Sherlocksuper hsluoyz

repo: casibase/casibase


text:
Kotlin standard library errors with kte and Gradle continuous build

Using 3.1.0, we are having a great experience with:

TemplateEngine.createPrecompiled
and also with

val codeResolver = DirectoryCodeResolver(CWD.server("src/main/jte"))
TemplateEngine.create(codeResolver, Paths.get("build/jte-classes"), ContentType.Html).apply {
  setBinaryStaticContent(true)
}

Precompiled runs great in prod, the DirectoryCodeResolver gives us great near-instant hotreload locally, with both .jte and .kte.
The only problem we have is with Gradle continuous build. We have a hotreload setup that uses continuous build to restart the server anytime our business logic changes. And for some reason, it is erroring out on .kte templates like so:
  rootCause=gg.jte.TemplateException: Failed to compile template, error at pages/Admin/adminTrialShow.kte:1
@file:Suppress("ktlint")
/Users/ntwigg/Documents/dev/diffplugdotcom/server/build/jte-classes-73184/gg/jte/generated/ondemand/pages/Admin/JteadminTrialShowGenerated.kt:1:7
Reason: Cannot access built-in declaration 'kotlin.Suppress'. Ensure that you have a dependency on the Kotlin standard library
	at gg.jte.compiler.kotlin.KotlinClassCompiler.compile(KotlinClassCompiler.java:45)
	at gg.jte.compiler.TemplateCompiler.precompileClasses(TemplateCompiler.java:122)
	at gg.jte.compiler.TemplateCompiler.precompile(TemplateCompiler.java:94)
	at gg.jte.compiler.TemplateCompiler.load(TemplateCompiler.java:50)
	at gg.jte.TemplateEngine.lambda$resolveTemplateOnDemand$0(TemplateEngine.java:354)
	at java.base/java.util.concurrent.ConcurrentHashMap.compute(ConcurrentHashMap.java:1916)
	at gg.jte.TemplateEngine.resolveTemplateOnDemand(TemplateEngine.java:347)
	at gg.jte.TemplateEngine.resolveTemplate(TemplateEngine.java:337)
	at gg.jte.TemplateEngine.render(TemplateEngine.java:228)

It's strange because the same code works great when we run it without continuous build. Not necessarily a JTE issue, but I wonder if setNoStdlib here ought to be false instead of true?
https://github.com/casid/jte/blob/f53fb9584e7d8526ebc7efba581efad378d3782f/jte-kotlin/src/main/java/gg/jte/compiler/kotlin/KotlinClassCompiler.java#L25-L28

Hmm, the documentation says Don't automatically include the Kotlin/JVM stdlib and Kotlin reflection into the classpath, which sounded good to me, when I intitially added Kotlin support. However, I'm not a heavy Kotlin or Gradle user, so that was rather a gut decision than a well backed one from experience.
Maybe you could try to change this line, make a snapshot build and see if it is working afterwards?

Hello @casid
I tried to set the noStdLib to false and rebuild. It did not fix the issue.

source: gharchive/issue

authors: casid nedtwigg ylemoigne

repo: casid/jte

license_type: permissive

license_source: github-api


text:
add widget for access_logging

Cherry-pick #256 with no conflicts

looks like an indentation issue on this and the original, otherwise 👍 lgtm

I fixed the spacing here. I'll submit a new PR to fix the indent on develop

source: gharchive/pull-request

authors: dereklwood mattwuenschel

repo: caskdata/cdap-ambari-service

license_source: bigquery


text:
Update README.md with latest net-ssh changes

This is temporary until everything can be modified to use bundle install and bundle exec and tested with packaging.

:+1: lgtm.  create https://issues.cask.co/browse/COOPR-781 for the permanent fix

repo: caskdata/coopr


text:
update synology-cloud-station to 3.2-3484

This pull request updates synology-cloud-station to the latest version 3.2-3484.

Thank you for the contribution. It was merged directly as 9bdac110d5629b299d3a0818b790a6ae3c10eb98 to keep commit history cleaner. Your contribution is still credited to you.

authors: victorpopkov wuman

repo: caskroom/homebrew-cask


text:
updated sabnzbd (1.0.0)

new build from sabnzbd for osx, see https://forums.sabnzbd.org/viewtopic.php?f=8&t=20459

Thank you for the contribution. It was merged directly as 6f45f6264548589a87c0e831dbafb44cb365bfff to keep commit history cleaner. Your contribution is still credited to you.

authors: elnappo victorpopkov


text:
Add cutter 1.0

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if adding a new cask:

[x] Named the cask according to the [token reference].
[x] brew cask install {{cask_file}} worked successfully.
[x] brew cask uninstall {{cask_file}} worked successfully.
[x] Checked there are no [open pull requests] for the same cask.
[x] Checked the cask was not already refused in [closed issues].
[x] Checked the cask is submitted to [the correct repo].

audit for cutter: warning
 - possible duplicate, cask token conflicts with Homebrew core formula: https://github.com/Homebrew/homebrew-core/blob/master/Formula/cutter.rb
Error: audit failed for 1 cask: cutter

Formula is unrelated.

Thanks for the PR!

authors: commitay ndaprela


text:
Add BusyCal

How does this look? The trial version now functions as the full version (see also #5779).

Looks perfect. Thanks you.

@ndr-qef, my pleasure. I'm just trying to figure out why Installer.app is still running after install. Just ran this on a different machine and noticed. I don't think I've seen this behaviour in any other casks I've installed, would this be something with a known workaround?

Also after running brew cask uninstall busycal I'm still seeing some kind of busted BusyCal.app in /Applications. Maybe this needs some special treatment for both install and uninstall.

You are right; I can reproduce the behavior in a Mavericks VM. Moreover, the pkg installer opens a Finder window at /Applications, which is likely undesirable from our perspective.

I didn't notice the Finder window, I see it now as well. That must be part of the installer scripts because I'm seeing it when running the installer manually. When running the installer manually it does exit Installer.app at the end.

authors: Cottser ndr-qef


text:
Mainnet error frequency rule

Mainnet error frequency rule
At least 1 events occurred between 2021-10-19 17:28 UTC and 2021-10-19 17:43 UTC
@timestamp: 2021-10-19T17:43:59.197Z
@version: 1
_id: aRKnmXwBaqnV04emJGsu
_index: mainnet-casper-node-rs-2021.10.19
_type: _doc
agent: {
   "ephemeral_id": "36b182fe-7e77-4b51-be80-6706dc9ef001",
   "hostname": "casperlabs-casper-node-1",
   "id": "51290c0e-7161-481e-a829-4a1f549fa3e6",
   "name": "casperlabs-casper-node-1",
   "type": "filebeat",
   "version": "7.10.1"
}
cl-sre-metadata: {
   "env": "mainnet",
   "node_hostname": "casperlabs-casper-node-1",
   "node_version": "1.0.0"
}
cl_log_source: casper-node
cloud: {
   "account": {
       "id": "107469540579"
   },
   "availability_zone": "us-east-2a",
   "image": {
       "id": "ami-0d5d9d301c853a04a"
   },
   "instance": {
       "id": "i-0b3992bfd47f941b1"
   },
   "machine": {
       "type": "t3.xlarge"
   },
   "provider": "aws",
   "region": "us-east-2"
}
ecs: {
   "version": "1.6.0"
}
fields: {
   "message": "could not send response to request down oneshot channel"
}
host: {
   "architecture": "x86_64",
   "containerized": false,
   "hostname": "casperlabs-casper-node-1",
   "id": "ec2a247454e5d7806dde28f3b3a5db74",
   "ip": [
       "10.22.5.226",
       "fe80::d4:57ff:feb3:ec46"
   ],
   "mac": [
       "02:d4:57:b3:ec:46"
   ],
   "name": "casperlabs-casper-node-1",
   "os": {
       "codename": "bionic",
       "family": "debian",
       "kernel": "4.15.0-1051-aws",
       "name": "Ubuntu",
       "platform": "ubuntu",
       "version": "18.04.3 LTS (Bionic Beaver)"
   }
}
input: {
   "type": "log"
}
level: ERROR
log: {
   "file": {
       "path": "/var/log/casper/casper-node.log"
   },
   "offset": 36265235
}
log_filename: casper-node.log
message: {"timestamp":"Oct 19 17:43:58.676","level":"ERROR","fields":{"message":"could not send response to request down oneshot channel"},"target":"casper_node::effect"}
num_hits: 3
num_matches: 1
tags: [
   "beats_input_codec_plain_applied",
   "syslog_parsed"
]
target: casper_node::effect
timestamp: Oct 19 17:43:58.676

Closing this for now, as this is expected to not be an issue anymore once #2207 is fixed.

authors: marc-casperlabs piotr-dziubecki

repo: casper-network/casper-node


text:
[Request] Responses returning expressions for lists by using Json Path

Hello, my friends! I'd like to suggest expressions returning lists that are values of json attributes. For example: I have a json object in the request that contains an attribute with a list as its value. In the response, there's the same list, so in order to not hardcoding it, I would like to return the request list in the response too, as you can see in the examples below:
Request:
"items": [
      {
          "sku": "Gasolina Superior",
          "name": "Gasolina Superior",
          "value": 10000,
          "quantity": 1
      }
]

In the response, I tried the following expression on Castlemock:
"items": ${BODY_JSON_PATH(expression="$.salebasket.items[*]")

The expression I got by using this website, but the response I'm getting when I make the request to Castlemock is the following:

You can see that it returns not a list, but the first object in it. If a pass two objects in the list, the same occurs:

Is there a way to return the list as it is passed in the request? If there isn't, what do you guys think of making this possible?! It would be a lot appreciatted.
I searched in the expression docs, but didn't find something, so I'm opening this request.
Thank you very much!

Hi @wedla.
Thanks for reporting this! Great issue report!
I took a look at this and I can see that Castle Mock only returns one values. However, i have updated this so that it can retrieve multiple values. The fix will be part of next release (Which I hope can be released during the weekend).
Thanks!

authors: karldahlgren wedla

repo: castlemock/castlemock


text:
Integrate CACM collection

The CACM collection is small enough that we can include it in the repository... so we can have indexing/retrieval experiments completely integrated in with the system.

Won't integrate CACM in this repo.
Instead, we put it on another repo: https://github.com/castorini/Anserini-data/tree/master/CACM to avoid possible license issue.

authors: Peilin-Yang lintool

repo: castorini/Anserini


text:
Replicate Anserini Core18 experiments

Can we have someone outside the project replicate the Anserini TREC 2018 CORE runs?
You'll need to be an active participant in TREC 2018 because otherwise you won't have access to the topics and qrels...

Fetch the topics and qrels per https://github.com/castorini/Anserini/blob/master/docs/experiments-core18.md#Retrieval

Change the collection path at
https://github.com/castorini/Anserini/blob/master/src/main/resources/regression/core18.yaml#L16

Run

nohup python src/main/python/run_regression.py --collection core18 --index >& log.core18 &

@isoboroff if you have a some time?

Sure, it worked fine with only the two changes you mentioned:

2018-11-30 21:42:43,052 - regression_test - INFO - ==========Verifying Results==========
2018-11-30 21:42:43,175 - regression_test - INFO - {"actual": 0.2487, "collection": "core18", "expected": 0.2487, "metric": "map", "model": "bm25", "topic": "All Topics"}
2018-11-30 21:42:43,251 - regression_test - INFO - {"actual": 0.364, "collection": "core18", "expected": 0.364, "metric": "p30", "model": "bm25", "topic": "All Topics"}
2018-11-30 21:42:43,354 - regression_test - INFO - {"actual": 0.2911, "collection": "core18", "expected": 0.2911, "metric": "map", "model": "bm25+rm3", "topic": "All Topics"}
2018-11-30 21:42:43,431 - regression_test - INFO - {"actual": 0.4087, "collection": "core18", "expected": 0.4087, "metric": "p30", "model": "bm25+rm3", "topic": "All Topics"}
2018-11-30 21:42:43,536 - regression_test - INFO - {"actual": 0.2919, "collection": "core18", "expected": 0.2919, "metric": "map", "model": "bm25+ax", "topic": "All Topics"}
2018-11-30 21:42:43,612 - regression_test - INFO - {"actual": 0.4033, "collection": "core18", "expected": 0.4033, "metric": "p30", "model": "bm25+ax", "topic": "All Topics"}
2018-11-30 21:42:43,713 - regression_test - INFO - {"actual": 0.2504, "collection": "core18", "expected": 0.2504, "metric": "map", "model": "ql", "topic": "All Topics"}
2018-11-30 21:42:43,787 - regression_test - INFO - {"actual": 0.362, "collection": "core18", "expected": 0.362, "metric": "p30", "model": "ql", "topic": "All Topics"}
2018-11-30 21:42:43,896 - regression_test - INFO - {"actual": 0.2754, "collection": "core18", "expected": 0.2754, "metric": "map", "model": "ql+rm3", "topic": "All Topics"}
2018-11-30 21:42:43,970 - regression_test - INFO - {"actual": 0.3773, "collection": "core18", "expected": 0.3773, "metric": "p30", "model": "ql+rm3", "topic": "All Topics"}
2018-11-30 21:42:44,075 - regression_test - INFO - {"actual": 0.2976, "collection": "core18", "expected": 0.2976, "metric": "map", "model": "ql+ax", "topic": "All Topics"}
2018-11-30 21:42:44,151 - regression_test - INFO - {"actual": 0.4067, "collection": "core18", "expected": 0.4067, "metric": "p30", "model": "ql+ax", "topic": "All Topics"}
2018-11-30 21:42:44,151 - regression_test - INFO - All Tests Passed!

Awesome, thanks @andrewyates !

sure, #521

authors: andrewyates lintool


text:
Attachment seeker when run by spatial transformer not finding any attachments

I have a feeling with the new logic the attachment tables are not being added to the geodatabase so the attachment seeker is not finding anything to extract. Double check this is the case and fix it.

Change has been made for this in the commit: https://github.com/cat-cfs/twobilliontoolkit/commit/8024ce6be6ccc93738c219eab226aaf5221dc87d
Changes include:

fixed the way the gdb path was join in the arc export function causing it to not function properly and now extract attachments

👍

repo: cat-cfs/twobilliontoolkit

license: MIT


text:
Test individual archivers

Background
Right now the individual archivers (inheriting from AbstractDatasetArchiver) are largely untested. There's been discussion in #43 about the appropriate way to test these archivers. In the long run, we will want to have new archives tested through nightly builds to fully verify the entire workflow. This, however will likely be future work once we are actually generating new archives regularly, and have decided how we want to integrate those archives into PUDL. In the immediate future, there are likely helpful tests that we can create at the unit/integration level to perform some basic sanity checking.
Scope
This issue will only track the development of unit/integration tests, and will leave full integration into the nightly builds process for future work. Things we can check for at this level include:

[ ] Are all of the links we attempt to download files from valid?
[ ] Is the structure of the archive what is expected by PUDL (partitions are formatted correctly, and filenames look like what we'd expect

@zschira Are there unit or integration tests that you imagine falling under this issue that wouldn't be covered by what's described in #70?

authors: zaneselvans zschira

repo: catalyst-cooperative/pudl-archiver


text:
Are some CEMS UTC datetimes off by an hour?

In the CEMS transformation step, the reported operating hour is converted from local time to UTC time based on each plant's local timezone reported in plants_entity_eia. However, I am wondering if in some cases we are currently changing the data to the wrong UTC time. For the Open Grid Emissions project, we convert datetime_utc back to datetime_local based on each plant's reported local timezone from plants_entity_eia so that we can assign a monthly report_date to each column that can be used to match the CEMS data with the monthly totals reported in EIA-923. Because we want a complete timeseries for each unit-month, we drop any observations in unit-months with incomplete hourly data. During this process, I've found that in many cases, incomplete months contain only a single hourly value at the very end of the month. This made me suspicious (why would a plant only report the last hour of a month?), and made me wonder if maybe the datetime is off by an hour.
I would have to look into specific examples to see if this is actually the case, but here is my hypothesis about what might be happening:
Because the raw CEMS data is reported by state-month, I am wondering if the op_hour column represents the local time for each plant, or a uniform local time for the entire state. In states that span multiple time zones (AZ, OR, ID, ND, SD, NE, KS, TX, KY, TN, IN, FL), this could make a difference. If the reported operating hour is uniform for the entire state, then the transform step would need to first convert local reported time to local plant time, then to UTC.
To check into this I need to:

Examine specific examples of unit-months with only a single hour reported at the end of the month
Find out where these units are located
Look at the raw CEMS data to see if these single hours are present there
Try to look into the CEMS documentation about what the operating hour represents

Other possible explanations:

Maybe some issue with DST?
Is the version of plants_entity_eia used to convert to UTC different from the one I'm using to convert back?

See https://github.com/singularity-energy/open-grid-emissions/issues/194
So after looking into this, I discovered that the reason that I was seeing months with single hour datapoints for some units is because the raw CEMS files from the FTP site report all data in local standard time (not local prevailing time). So when these data are converted to local prevailing time, there are cases (generally for plants that only report during the ozone season and have data through Sept 30 at 11pm) when there is a single hour of data for October 1 at Midnight in local prevailing time.
I also did not find any evidence that there were a lot of plants located in the states that have 2 time zones that were off by an hour. However, I still want to double check with EPA the process they use for converting timezone data for each plant.

I accidentally closed this issue when I just meant to comment so re-opening.
One thing I'm noticing is that in pudl.transform.epacems, it assumes that the reported operating hour is the plant's local standard time (rather than the state's local standard time). This is something I want to confirm with EPA before closing this.

Ugh. Timezones. 😵💫

According to email communication with the EPA CAMPD team, "OP_HOUR represents each plant's local standard time. Therefore, the OP_HOUR data, including the FTP datasets, correspond to each facility's local standard time even if the data set is for a state which uses two different time zones."
This confirms that the current transformation steps for the CEMS data are correct.

Whew!

authors: grgmiller zaneselvans

repo: catalyst-cooperative/pudl

license: mit


text:
pydoc.locate cannot locate modules in local dir

Sometimes pydoc.locate fails to locate modules in local dir. Probably because local directory changes to something else.
HotFix:
PYTHONPATH=.:$PYTHONPATH catalyst-dl run -C config.yml

https://github.com/python/cpython/blob/main/Lib/pydoc.py#L1721 -> https://github.com/python/cpython/blob/af9ee57b96cb872df6574e36027cc753417605f9/Lib/importlib/_bootstrap.py#L1167

repo: catalyst-team/hydra-slayer


text:
[Discussion] Extension's Structure Proposal

This issue is to discuss the extension structure.
With the first proposal I try to keep in consideration all the ideas in #1, the metadata in other standards as suggested in #3 and some issues reported in nwb-schema:
(1)[https://github.com/NeurodataWithoutBorders/nwb-schema/issues/538]
(2)[https://github.com/NeurodataWithoutBorders/nwb-schema/issues/517]
(3)[https://github.com/NeurodataWithoutBorders/nwb-schema/issues/431]
(4)[https://github.com/NeurodataWithoutBorders/nwb-schema/issues/406]
(5)[https://github.com/NeurodataWithoutBorders/nwb-schema/issues/343]
(6)[https://github.com/NeurodataWithoutBorders/pynwb/issues/1736]
NB:

The schemes are still WIP: I will add more details on a written form later
Reference arrows have been left out of the schema for a more compact visualization (they will be included in the final version)
OptogeneticSeries link to Microscope and LightSource represented in the schema for the MicroscopySeries

Ophys Acquisition

Optogenetics
it also take into account the extensions for patterned photostimulation: https://github.com/catalystneuro/ndx-holographic-stimulation, https://github.com/histedlab/ndx-photostim

I have been meaning to start this discussion for the last month but it was not until last week that I met wit the members of the Cladinin group.
Recently, the Cladinin lab came with a proposal for doing imaging registration to standard atlases called Bifrost. While the details of how to do registration are complex I think that we can move the NWB format further by having at least the ability to express the most basic of spatial coordinates transformations: the affine transform.
In fmri neuroimaging the transformation is used to express how the data in voxel space (i.e. the data as it is) could be transformed into some real world coordinates. A good summary can be found in the following tutorial but the idea is simple, every image / video has a matrix of the following kind:

That is used to express the rotations and translations that would be necessary to express the data as it is (the voxel or pixel space) into lab coordinates.
Right now, I think that the current elements of the ImagingPlane map to the following elements of the Affine matrix formulation for expressing real world coordinates:

origin_coords: maps to the translation part of the matrix
grid_spacing: maps to the diagonal part of the matrix (the scaling).

As you see we are missing the ability to express rotations that usually affect the non-diagonal terms of the matrix (and also the diagonal). That is, the having the concept of the affine matrix will generalize two our fields and expand the expressive power of proposed ImagingSpace concept.  I think this is a useful concept to have.

Another concept that maybe needs refinment is the use and scope of the reference frame field. Currently we have reference_frame that expresses as a description of what the origin is (e.g. Bregma). For example, here is the origin as Bregma in the Paxinos atlas.

Which is what I think inspired the field at the time. However, in neuro imaging it seems more common to agree that the axis are given anatomically (right-left, posterior-anterior, inferior-superior) and then there are different conventions for the signs of the given axes. Check this image:

That is, if we are gona use the reference_frame field or something else as a description of the what do the axes mean maybe we should update the string to contain a set of working examples as the current one is not very descriptive in my opinion.

With regards to the light source, what are the intended values to record under power and intensity? Is it the power/intensity of the light source itself, or the power/intensity measured under the objective, which in most cases are the most meaningful to record? I assume a description of this would be added in the doc, but maybe it could be made more explicit by for example consider names like for example source_power or target_power/ power_at_target

For the topic of coordinates a good discussion can be found here:
https://github.com/ome/ngff/pull/138

repo: catalystneuro/ndx-microscopy


text:
Deathemonic rofi launcher does not close when esc is pressed

Im using the launcher Deathemonic bin on Hyprland 0.41.2-3
If i open it with my shortcut (super + R) it does not close until i open an app, even when esc is pressed
If this is a Rofi issue im very much ready to go repor there, but I dont really know what the Deathemonic bins even are so im asking here first

It can be closed with alt+f1, seems like you can change it in bin/launcher

oh yep, i see. I might suggest possibly removing that line (so that its consistent with the runner) or adding a documentation mention for that. I'm willing to draft a small pr for either of those.

Try using rofi-wayland instead of the regular rofi for wayland compositors.

This was on rofi-wayland. using normal rofi through xwayland didnt work very well for me.

authors: Iris-TheRainbow NotMugil xirzo

repo: catppuccin/rofi


text:
🛑 Artfos ERP is down

In 1245aac, Artfos ERP (https://intranet.artfos.com.ar/equipo/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Artfos ERP is back up in b4955fc.

repo: cavalicenti/upptime


text:
🛑 Janax Soft is down

In 9b558ae, Janax Soft (https://www.janaxsoftware.com.ar/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Janax Soft is back up in 0f9a227.


text:
NextPath is broken on Postgres

Description
https://discourse.cayley.io/t/path-save-lost-some-data-when-using-sql-backends/1192
Steps to reproduce the issue:

Initialize a clean PG instance
Load test data
Execute the query:

g.V("<dani>", "<bob>").Save("<follows>", "target").All()

Received results:
{"id":"bob","target":"fred"}
{"id":"dani","target":"greg"}

Expected results:
{"id":"bob","target":"fred"}
{"id":"dani","target":"bob"}
{"id":"dani","target":"greg"}

Environment details:
Backend database: postgres

Related to a new optimizer that flattens NodesFrom. Because of this, NextPath stops working - it's not implemented for the base Select iterator.

repo: cayleygraph/cayley


text:
chore: bdd-sbhg-1584371985 to 0.0.1

chore: Promote bdd-sbhg-1584371985 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


text:
chore: bdd-spring-1586226233 to 0.0.1

chore: Promote bdd-spring-1586226233 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


text:
chore: bdd-nh-1588804450 to 0.0.1

chore: Promote bdd-nh-1588804450 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


text:
Implement apS function

Hi!
I implemented the apS function from the paper.  I wasn't sure if it was deliberately omitted or not?  Maybe because you had in mind modelling rigid and non-rigid Selectives separately, or perhaps there was some other reason?
If you think this might be a useful contribution, let me know and I'll expand on the testing a little further before submitting something for review.  But if not, don't feel obliged: I'm just satisfying my own curiosities!

Sorry, I completely missed this until now!

@icmurray Sorry for the slow review. This looks great. Is there a reason why it’s still a draft PR? Are you still working on it?

@cb372 thanks for reviewing!
I left the PR in a draft state because I had the niggling feeling that something was missing!  But it's not come to me, so I guess either it's good enough or it wasn't important.

My drive-by, unsolicited suggestion: assume good intentions and merge.
"Merge first and ask questions later", if you will. 😄

authors: cb372 dwijnand icmurray


text:
Can't write Assert.xyz for a Map where the expected value of a key is 'null'

6.14.3
Expected behavior
The provided test case shouldn't fail - at least I think it should not fail.
Actual behavior
The test case fails.
Is the issue reproducible on runner?

[x] NetBeans

Test case sample
@Test
public void thisSucceeds() {
	var actual = new HashMap<String, String> ();
	actual.put("key", "1");
	var expected = new HashMap<String, String>();
	expected.put("key", "1");

	Assert.assertEqualsDeep(actual, expected);
}
@Test
public void thisFailsButShouldAlsoSucceed() {
	var actual = new HashMap<String, String> ();
	actual.put("key", null);
	var expected = new HashMap<String, String>();
	expected.put("key", null);

	Assert.assertEqualsDeep(actual, expected);
}

Problem [Assert.assertEqualsDeep, Assert.java#L2063]
The method trys to retrieve the data type of the expected value expectedValue.getClass() which fails since the expected value of the key is null.

The latest released version of TestNG is 7.7.1. Please retry using that and let us know if the problem exists.

Sorry, cant test 7.7.1 since swapping the libs shipped with netbeans 17 (6.14.3) does not allow me to run any tests...
Exception in thread "main" org.testng.TestNGException:  Couldn't find a constructor in class org.testng.reporters.VerboseReporter 	at org.testng.internal.objects.InstanceCreator.newInstance(InstanceCreator.java:57) 	at org.testng.ITestObjectFactory.newInstance(ITestObjectFactory.java:10) 	at org.testng.TestNG.setListenerClasses(TestNG.java:679) 	at org.testng.TestNG.configure(TestNG.java:1519) 	at org.testng.TestNG.privateMain(TestNG.java:1405) 	at org.testng.TestNG.main(TestNG.java:1378)
The method assertEqualsDeep(Map<?, ?> actual, Map<?, ?> expected, String message) has changed from 6.14.3 to 7.7.1 but both contain the expectedValue.getClass() call without checking that expectedValue is not null.
Method:
https://github.com/cbeust/testng/blob/89dc5845fcb46c26af187e50ea907a7382d06e72/testng-asserts/src/main/java/org/testng/Assert.java#L2143
https://github.com/cbeust/testng/blob/89dc5845fcb46c26af187e50ea907a7382d06e72/testng-asserts/src/main/java/org/testng/Assert.java#L2164
This method has the same problem, altough its much more unlikely that a set will contain a 'null' value...
https://github.com/cbeust/testng/blob/89dc5845fcb46c26af187e50ea907a7382d06e72/testng-asserts/src/main/java/org/testng/Assert.java#L2044
https://github.com/cbeust/testng/blob/89dc5845fcb46c26af187e50ea907a7382d06e72/testng-asserts/src/main/java/org/testng/Assert.java#L2063

I've created a maven test project with netbeans and added testng 7.7.1 and my testcase... and its running fine 😂 dont know why my ant-project throws exceptions at me when i swapped the libs, but maven works fine. and the testcase too.
Objects.equals(...) in 7.7.1 handles the map stuff (and also null values) while back in 6.13.4 there was just actual == expected so execution continues and reaches the problematic code section.
closed as not reproducible/fixed in 7.7.1

@mictru - Thank you for the confirmation. Since you mention ant, am guessing that somewhere in your libs there perhaps is TestNG already available (and its an older version) or maybe you have a jar in your libs that contains a shaded TestNG jar within it. All these are just naive guesses since I have never used ant as a build tool.

authors: krmahadevan mictru

repo: cbeust/testng


text:
Drop ant from the project

As the project clearly moved to Maven long time ago, I think ant reference and files could be removed from the project.
Of course, useful ant targets must be moved in maven.

+1

Agreed, ant needs to go.

Done

authors: FibreFoX cbeust juherr


text:
is this project finish ?

i try registration but i take error.
Error:

An exception occurred while executing 'INSERT INTO fos_user (username, username_canonical, email, email_canonical, enabled, salt, password, last_login, locked, expired, expires_at, confirmation_token, password_requested_at, roles, credentials_expired, credentials_expire_at) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)' with params ["oguzhan", "oguzhan", "oguzhantasc@gmail.com", "oguzhantasc@gmail.com", 1, "gmtlydkij6gckwkggk00cgoswgs0osc", "bhm55APgIomOiNvDLJt9LhFVJIkq7e8FNA1llWW\/U7f2+mFbIfYDWO3iFJTQ6uLF00YblBB3Y\/0bUHzUG49x5w==", null, 0, 0, null, null, null, "a:1:{i:0;s:11:\"ROLE_LEADER\";}", 0, null]:

Hi dear,
That's already published.But the error about u'r document's doesn't exits.U have to add , app/config/parameters.yml . and u'r config.I'm adding sql query's.Therefore u have to update your mysql db.
Merhabalar,
Bu proje tamamlanmıştır.Fakat sizin elinizdeki dosyaların içerisinde parameters.yml dosyası eksiktir.Kendi parameters.yml dosyanızı açıp içerisine kendi ayarlarınızı yazmanız ve veritabanınızı güncellemeniz gerekmektedir.

hi again, this error no your site - http://www.gorev.li -

Let's check it out together, username : testt pass: 12345 and we can test together on this link.
http://www.gorev.li/dashboard/#&togetherjs=7r8u6E04Qc

Log-in after click this link http://www.gorev.li/dashboard/#&togetherjs=7r8u6E04Qc

nice work :+1:
lastly, u fix this area then everything is nice. http://hizliresim.com/mP3nyP


text:
Add support for properties

Hi, great tool, thank you for your work !
Would it be possible to also search for property terms ?
I see in term_collector.py that you filter out PropertyClass terms:
if not isinstance(ontology_term, PropertyClass) and ontology_term is not Thing \                     and ontology_term is not Nothing:
Would it be possible to provide arguments to control this behavior ?

Hi @nleguillarme — Thank you, we are glad to hear that!
Yes, we can certainly support search for property terms. To help us design a suitable solution, would you mind giving us a general idea of your use case? For example, are there scenarios where you would want to search for only properties, or would an argument to search property terms (in addition to classes) suffice?

Thank you for the feedback, this is very helpful. We are working on a solution which will be included in a next release. I'll update you here once we have a pre-release implementation that could be experimented with.

Hello,
We have added the functionality as requested in the newest pre-release (2.2.0). Please take a look and let us know if the functionality and documentation is sufficient. We will add it to PyPi in the next release, if we have your approval.

Hello
I've just tested it and it seems to work as expected.
Thank you for adding this functionality.

authors: nleguillarme paynejason rsgoncalves

repo: ccb-hms/ontology-mapper


text:
chatgpt bugbounty usage

Adding repo github prompts for bugbounty and pentesting to experimental list

Thank you for the fantastic contribution.

Thank you for the repository

authors: cckuailong tin-z

repo: cckuailong/awesome-gpt-security


text:
How to set vina score to dock complex with metals?

I heard that there is some sf than can be used in metal ions(Mg,Se,Zn...) docking in AutoDock Vina, But I couldn't find it in manual.
Is there anything to do with  v = Vina(sf_name='vina')?

The vina scoring function, which is the default (e.g., v = Vina()), handles metals. There's zinc specific potentials but that is for the autodock4 scoring function.

Though the default scoring function can deal with metals, the protein.pdb cannot be converted into pdbqt file if they have Zn,Mg...
for example, 1ew9_pocket.pdb consists of Zn and Mg, and the Error is like:
/ADFRsuite_x86_64Linux_1.0/bin/prepare_receptor -r 1ew9_pocket.pdb -A hydrogens -o 1ew9_pocket.pdbqt
'  ' apparently composed of not std residues. Deleting
Traceback (most recent call last):
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/AutoDockTools/Utilities24/prepare_receptor4.py", line 216, in <module>
    dict=dictionary)
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/AutoDockTools/MoleculePreparation.py", line 558, in __init__
    version=version, delete_single_nonstd_residues=delete_single_nonstd_residues)
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/AutoDockTools/MoleculePreparation.py", line 124, in __init__
    self.repairMol(mol, self.repair_type_list)
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/AutoDockTools/MoleculePreparation.py", line 174, in repairMol
    self.newHs = self.addHydrogens(mol)
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/AutoDockTools/MoleculePreparation.py", line 187, in addHydrogens
    HB.addHydrogens(mol)
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/MolKit/hydrogenBuilder.py", line 61, in addHydrogens
    babel.assignHybridization(mol.allAtoms)
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/PyBabel/atomTypes.py", line 136, in assignHybridization
    self.valence_three()
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/PyBabel/atomTypes.py", line 236, in valence_three
    elif self.count_free_ox(a) >= 2: a.babel_type="Cac"
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/PyBabel/atomTypes.py", line 167, in count_free_ox
    self.count_heavy_atoms(bonded_atom) == 1:
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/PyBabel/atomTypes.py", line 157, in count_heavy_atoms
    if bonded_atom.babel_type[0] == 'H': count = count + 1
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/MolKit/molecule.py", line 409, in __getattr__
    raise AttributeError('member %s not found'%member)
AttributeError: member babel_type not found

Though the default scoring function can deal with metals, the protein.pdb cannot be converted into pdbqt file if they have Zn,Mg...
for example, 1ew9_pocket.pdb consists of Zn and Mg, and the Error is like:
/ADFRsuite_x86_64Linux_1.0/bin/prepare_receptor -r 1ew9_pocket.pdb -A hydrogens -o 1ew9_pocket.pdbqt
'  ' apparently composed of not std residues. Deleting
Traceback (most recent call last):
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/AutoDockTools/Utilities24/prepare_receptor4.py", line 216, in <module>
    dict=dictionary)
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/AutoDockTools/MoleculePreparation.py", line 558, in __init__
    version=version, delete_single_nonstd_residues=delete_single_nonstd_residues)
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/AutoDockTools/MoleculePreparation.py", line 124, in __init__
    self.repairMol(mol, self.repair_type_list)
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/AutoDockTools/MoleculePreparation.py", line 174, in repairMol
    self.newHs = self.addHydrogens(mol)
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/AutoDockTools/MoleculePreparation.py", line 187, in addHydrogens
    HB.addHydrogens(mol)
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/MolKit/hydrogenBuilder.py", line 61, in addHydrogens
    babel.assignHybridization(mol.allAtoms)
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/PyBabel/atomTypes.py", line 136, in assignHybridization
    self.valence_three()
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/PyBabel/atomTypes.py", line 236, in valence_three
    elif self.count_free_ox(a) >= 2: a.babel_type="Cac"
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/PyBabel/atomTypes.py", line 167, in count_free_ox
    self.count_heavy_atoms(bonded_atom) == 1:
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/PyBabel/atomTypes.py", line 157, in count_heavy_atoms
    if bonded_atom.babel_type[0] == 'H': count = count + 1
  File "/data2/rjli/ADFRsuite-1.0/ADFRsuite_x86_64Linux_1.0/CCSBpckgs/MolKit/molecule.py", line 409, in __getattr__
    raise AttributeError('member %s not found'%member)
AttributeError: member babel_type not found

If so, how can we use vina to dock protein with metals? We even cannot get their pdbqt files!

@diogomart  Dear developer, I met the same questions. When there is metal ions such as Mg, the ADFR can't convert it into pdbqt file.
So it is impossible to use vina to dock complex with metal.

Hi @Kerro-junior and @Dadiao-shuai,
You should be able to get passed the issue if you are not adding hydrogens in prepare_receptor. Please see an example here: https://github.com/ccsb-scripps/AutoDock-Vina/issues/241#issuecomment-1734266302
If you think your problem is structure-specific, it would be really helpful if you could your files so others can reproduce the error.

authors: Dadiao-shuai Kerro-junior RJ-Li diogomart rwxayheee

repo: ccsb-scripps/AutoDock-Vina


text:
Update pom for release

Fixes #32
Changes made in small increments for better review.
Please squash all changes before merging.

@aalmiray - sorry, it looks like this may need a rebase now.

It's OK. I'll take care of it in a few.

authors: aalmiray afrittoli

repo: cdevents/sdk-java


text:
books 如何在本地进行测试？

编辑了 books 下的 py 文件之后，想在本地测试一下是否能够成功生成想要的书籍，请问应该如何操作？

安装GAE SDK，它就有一个本地调试环境，用它打开kindleear工程，然后在launcher的 extra flags里面加入 module-worker.yaml，就可以调试了，或也可以使用dev_appserver.py打开本地工程app.yaml/module-worker.yaml，一样的，我都是使用dev_appserver.py，至于将生成的书籍发送处理，有多种方法，要不你就建本机SMTP调试服务器，要不就使用sendmail（移植于linux）方案。
我用的是本机SMTP调试服务器方案。
如果嫌麻烦的话，还是上传到GOOGLE服务器吧，多上传几次多调试几次就好。
好像限额是一天1000次上传还是多少，足够你用的。

好的，谢谢，我还是上传调试吧 :joy:

@cdhigh 大神，因为我想修改一下前端，所以试图使用 Launcher 本地调试 KindleEar，按照你说的配置倒是可以访问了。但是页面显示“not found”，Log 中显示：
INFO     2016-04-28 15:11:45,813 module.py:787] worker: "GET / HTTP/1.1" 404 9
这是怎么回事儿呢？还需要设置什么地方吗？

worker模块和default模块的端口地址是不一样的，你输入了worker模块的端口号而出错。

@cdhigh 原来如此，感谢点拨，已搞定。

authors: cdhigh miaowm5 runbing

repo: cdhigh/KindleEar


text:
feat(deps): upgrade runtime dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-runtime-dependencies-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1403

Questions ?
Please refer to the Backport tool documentation


text:
chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-k8s-26-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 2835

Questions ?
Please refer to the Backport tool documentation


text:
chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dependencies" workflow

AWS CodeBuild CI Report

CodeBuild project: GitHubPipelineAutoBuildProj-XK8ItpxnSow5
Commit ID: 0dd0ca3aae177ff66b685b2cfab6ca19e8104166
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

repo: cdklabs/aws-delivlib


text:
Enhance survey filters in Django admin

Summary | Résumé
Trello
https://trello.com/c/eDGRMZDD/868-refine-filters-for-survey-send-feature

New filters on registrant including new columns
Default hardcoded 14 day filter

 Refine filters for survey send feature

repo: cds-snc/covid-alert-portal


text:
chore: disable Salesforce feature flag

Summary
Update the Staging environment variables to disable the Salesforce feature flag.  This is being done to confirm that the API and admin still work as expected with the feature disabled.
What happens when your PR merges?

Prefix the title of your PR:

fix: - tag main as a new patch release
feat: - tag main as a new minor release
BREAKING CHANGE: - tag main as a new major release
[MANIFEST] - tag main as a new patch release and deploy to production
chore: - use for changes to non-app code (ex: GitHub actions)

Alternatively, change the VERSION file - this will not create a new tag, but rather will release the tag in VERSION to production.

What are you changing?

[ ] Releasing a new version of Notify
[ ] Changing kubernetes configuration

Provide some background on the changes

Give details ex. Security patching, content update, more API pods etc

If you are releasing a new version of Notify, what components are you updating

[ ] API
[ ] Admin
[ ] Documentation
[ ] Document download API

Checklist if releasing new version:

[ ] I made sure that the changes are as expected in Notify staging
[ ] I have checked if the docker images I am referencing exist

[ ] api lambda (requires Notification-Production / AdministratorAccess login)
[ ] api k8s
[ ] admin
[ ] documentation
[ ] document download API

Checklist if making changes to Kubernetes:

[ ] I know how to get kubectl credentials in case it catches on fire

After merging this PR

[ ] I have verified that the tests / deployment actions succeeded
[ ] I have verified that any affected pods were restarted successfully
[ ] I have verified that I can still log into Notify production
[ ] I have verified that the smoke tests still pass on production
[ ] I have communicated the release in the #notify Slack channel.

Diff
$ make diff-staging
Archive:  .previous.env.zip
  inflating: .env
Archive:  .env.zip
  inflating: .env
36c36
< FF_SALESFORCE_CONTACT=true
---
> FF_SALESFORCE_CONTACT=false

repo: cds-snc/notification-manifests


text:
Modify Templates Table to Include UpdatedAt Field

Acceptance Criteria

[ ] templates table includes UpdateAt timestamp field which is updated the record is modified.
[ ] modify return from database so that this field is available in the form client
[ ] clean migration with no loss of data

@thiessenp-cds I think this was implemented as part of the work you have done around /myforms. Is it correct? If yes then we could close the ticket.

@thiessenp-cds I think this was implemented as part of the work you have done around /myforms. Is it correct? If yes then we could close the ticket.

Yup, updatedAt was added to the Prisma templates file. So you can mark that off as complete :)

authors: Moro-Code craigzour thiessenp-cds

repo: cds-snc/platform-forms-client


text:
A few broken text for Logic

Description
There's some areas with broken text around the logic functionality - see screenshots.
Steps to reproduce
Area one:

Go to "Add a rule"
Scroll down to the bottom of the Modal
See error

Area two:

Go to the Branching tab
See error in panel

Details

Browser: Chrome
Operating system: Mac
Environment: Staging
Language: English

Expected behaviour
Plain language text
Screenshots or videos
When you open up the modal:

Navigating to a Page in Logic view:

Opening up a new form and navigating to the Logic tab:

A few more instances:

authors: Abi-Nada anikbrazeau


text:
[Snyk] Upgrade @aws-sdk/client-sqs from 3.74.0 to 3.150.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @aws-sdk/client-sqs from 3.74.0 to 3.150.0.

:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 31 versions ahead of your current version.
The recommended version was released 23 days ago, on 2022-08-15.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Closed by #1001

authors: bryan-robitaille sastels


text:
nature.com

apparently it was not using all of nature , only "open search" in the past , so..
https://www.nature.com/search?q=autism&title=autism&order=relevance&date_range=last_30_days

https://www.nature.com/search?q=autism&title=autism&order=date_desc

https://github.com/ceberous/AutismMastadonBot/commit/409783bd9398df18ce0b9cd95819e610bdf9dd30

repo: ceberous/AutismMastadonBot


text:
🛑 tanecni-divadlo.cz is down

In 8b979ca, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in 3382909.

repo: cebreus/upptime


text:
🛑 tanecni-divadlo.cz is down

In 730ca20, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in c777d88.


text:
🛑 tanecni-divadlo.cz is down

In 47f2f0c, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in 71e6c50.


text:
🛑 tanecni-divadlo.cz is down

In 77d7bca, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in 59b0f8f.


text:
🛑 tanecni-divadlo.cz is down

In db9df2f, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in 946a7bb.


text:
🛑 tanecni-divadlo.cz is down

In e0d63c1, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in ffdb4de.


text:
🛑 tanecni-divadlo.cz is down

In c56dfaf, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in 118ce13.


text:
🛑 tanecni-divadlo.cz is down

In 30161d1, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in 9a425c2.


text:
🛑 tanecni-divadlo.cz is down

In 39caa36, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in 346fa79.


text:
🛑 tanecni-divadlo.cz is down

In 29d7d87, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in ab86fb8.


text:
🛑 tanecni-divadlo.cz is down

In a0bcee0, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in a122df8 after 474 days, 17 hours, 41 minutes.


text:
🛑 tanecni-divadlo.cz is down

In 0fea204, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 520
Response time: 10334 ms

Resolved: tanecni-divadlo.cz is back up in b2700e8 after 476 days, 2 hours, 32 minutes.


text:
🛑 tanecni-divadlo.cz is down

In 318521b, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in 21812a2 after 479 days, 21 hours, 22 minutes.


text:
🛑 Bali 2017 is down

In b29855b, Bali 2017 (https://bali-2017.cebre.us/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Bali 2017 is back up in 9ca43e6 after 3 minutes.


text:
🛑 tanecni-divadlo.cz is down

In 034140f, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in 64cd8f1.


text:
🛑 tanecni-divadlo.cz is down

In f3c5076, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in 81a7ee1.


text:
🛑 tanecni-divadlo.cz is down

In c651f04, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in 383ad47.


text:
🛑 tanecni-divadlo.cz is down

In 0430e11, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in 9c49064.


text:
🛑 Bali 2017 is down

In d5d65d3, Bali 2017 ($SITE_BALI) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Bali 2017 is back up in f7198d7.


text:
🛑 tanecni-divadlo.cz is down

In 456beb9, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in 6b6738d.


text:
🛑 tanecni-divadlo.cz is down

In cad2e01, tanecni-divadlo.cz (https://www.tanecni-divadlo.cz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: tanecni-divadlo.cz is back up in f882089.


text:
Strange change of longitude values in test

In test:
tests/test_wps_subset_cru_ts.py::test_wps_subset_cru_ts_check_min_max[wet-1951-01-01-2005-12-15-area0]

...as follows...
data:

 lat = -89.75, -39.75, 10.25, 60.25 ;

 lon = -179.75, -129.75, -79.75, -29.75, 20.25, 70.25, 120.25, 170.25 ;
}
(roocs) [root@localhost flamingo]# fg
python -m pytest tests -v

'wet'
(Pdb) wps_ds[variable].coords
Coordinates:
  * lat      (lat) float32 10.25 60.25
  * lon      (lon) float32 1.0 51.0 101.0 151.0 201.0 251.0
  * time     (time) datetime64[ns] 1951-01-16 1951-02-15 ... 2005-11-16
(Pdb)
FAILED     [ 87%]

Might relate to changes in roocs-utils and clisops - need to check.

repo: cedadev/flamingo


text:
Updating to breaking upstream changes

Issue #, if available:
N/A
Description of changes:
Just updating to some breaking API changes in 3.0
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I'm pretty sure this actually reverts to the old (pre-3.0) APIs

Maybe my local env is messed up

repo: cedar-policy/cedar-spec


text:
Add roundtrip targets for entity data

Fixes for #472 by adding two new targets to check that we can always parse entity data after serializing to JSON. Local runs haven't found any failures

These passed an overnight local test run, so I'm hopeful they won't turn up anything in the nightly dashboard tests


text:
Improves the screen size normalization of the gizmo

Now the widget size doesn't depend of the width-height ratio and the canvas size.
Stops reading directly width and height attribute on canvas too (for
performance).

I don't think anyone used the index triangle since it just couldn't work properly

first the kdtree is re-ordering the triangles and uses its own indexes https://github.com/cedricpinson/osgjs/blob/develop/sources/osg/KdTree.js#L92
second not sure how to handle multiple primitiveSet

repo: cedricpinson/osgjs


text:
absorb rust-lorawan lorawan-device's trait impl locally

Typically, a trait implementation is not committed to the upstream repo. Was there a specific reason this one was?

This should probably be done on embassy-rs/lora-phy and not on this fork

Yea I didn't realize it was a fork. I'm just going to maintain this impl in the rust-lorawan crate for now.

authors: lthiery lucasgranberg

repo: ceekdee/lora-phy


text:
fix bug of verifyMultiRowRootsToDataRootTupleRoot

The original _root parameter is not verified before being used in verifyMultiRowRootsToDataRootTupleRootProof, it will allow arbitrary _rowRoots to be proved valid.

Summary by CodeRabbit

Refactor

Updated the verification process in the application to enhance data integrity checks.

Thanks for reporting this 🙏
Can you provide test cases that fail without this fix?

I don't have a test case, I found this problem when code review.

@zhiqiangxu I confirm the bug, would you be able to add the tests? Also, this fix should also be done for: verifySharesToDataRootTupleRoot and verifyRowRootToDataRootTupleRoot. Would you be down to updating this PR to include fix + tests for those too?
If not, I'll gladly pick this PR up and add the necessary changes :D

@zhiqiangxu I confirm the bug, would you be able to add the tests? Also, this fix should also be done for: verifySharesToDataRootTupleRoot and verifyRowRootToDataRootTupleRoot. Would you be down to updating this PR to include fix + tests for those too?
If not, I'll gladly pick this PR up and add the necessary changes :D

The last commit should already fix them. And the existing tests are updated, basically removing the redundant root parameter.
Feel free to add more tests:)

@zhiqiangxu I confirm the bug, would you be able to add the tests? Also, this fix should also be done for: verifySharesToDataRootTupleRoot and verifyRowRootToDataRootTupleRoot. Would you be down to updating this PR to include fix + tests for those too?
If not, I'll gladly pick this PR up and add the necessary changes :D

I fixed the mentioned functions, please take a look, and feel free to add more tests:)

authors: rach-id zhiqiangxu

repo: celestiaorg/blobstream-contracts


text:
Test for a specific size instead of multiple of the share size

in the current cli integration test, we are checking to see that emitted events contain a multiple of the share size instead of checking for a direct size. We should update that test to check for a specific size.

can close as this was completed https://github.com/celestiaorg/celestia-app/blob/6b86e91eea1063c27c1ae461eddbe505f778df08/x/payment/client/testutil/integration_test.go#L118

repo: celestiaorg/celestia-app


text:
test: testground sanity check with knuu

In replace of https://github.com/celestiaorg/celestia-app/pull/3184
Closes #3189

Introduces the capability to customize the resource allocation for each Node and TxSimNode individually, allowing for a diverse range of resource specifications rather than applying a uniform resource allocation across all instances.
Enables running txsim as a knuu instance.

Have you had any preliminary results yet as to whether having it as a separate instance enables greater load on the network?

Not yet, was awaiting reviews on the PR before moving forward with the test. I will conduct it and share the results soon.

Migrating this PR to this one


text:
Fix typos

Hello
I fixed several typos
Hope it helps.

we'll fix these alongside further documentation updates, we're not inclined to merge automated PRs, thanks

repo: celestiaorg/celestia-node


text:
[DNSResolver] Raise SocketError when there’s no connection.

In short, what I tried to fix is the following behaviour when trying to resolve a hostname when offline:
irb(main):001:0> require 'celluloid/current'
=> true
irb(main):002:0> require 'celluloid/io'
=> true
irb(main):003:0> Celluloid::IO::TCPSocket.new("www.google.com", 80)
NoMethodError: undefined method `"\u0000\u0002\u0001\u0000\u0000\u0001\u0000\u0000\u0000\u0000\u0000\u0000\u0003www\u0006google\u0003com\u0000\u0000\u0001\u0000\u0001"' for nil:NilClass
	from /Users/eloy/.gem/ruby/2.3.0/gems/celluloid-io-0.17.3/lib/celluloid/io/dns_resolver.rb:44:in `resolve'
	from /Users/eloy/.gem/ruby/2.3.0/gems/celluloid-io-0.17.3/lib/celluloid/io/tcp_socket.rb:101:in `create_socket'
	from /Users/eloy/.gem/ruby/2.3.0/gems/celluloid-io-0.17.3/lib/celluloid/io/tcp_socket.rb:53:in `initialize'
	from /Users/eloy/.gem/ruby/2.3.0/gems/celluloid-io-0.17.3/lib/celluloid/io/socket.rb:39:in `new'
	from /Users/eloy/.gem/ruby/2.3.0/gems/celluloid-io-0.17.3/lib/celluloid/io/socket.rb:39:in `new'
	from (irb):3
	from /Users/eloy/.rubies/ruby-2.3.0/bin/irb:11:in `<main>'

What happens is that @socket remains nil here which means that you end up sending the hostname as a NilClass#send message here.

With stdlib it raises the following exception, although that one is actually not raised by the name resolver, but by Socket:
irb(main):001:0> require 'socket'
=> false
irb(main):002:0> TCPSocket.new("www.google.com", 80)
SocketError: getaddrinfo: nodename nor servname provided, or not known
	from (irb):2:in `initialize'
	from (irb):2:in `new'
	from (irb):2
	from /Users/eloy/.rubies/ruby-2.3.0/bin/irb:11:in `<main>'

The one raised by Resolv is:
irb(main):001:0> Resolv.getaddress("www.google.com")
Resolv::ResolvError: no address for www.google.com
	from /Users/eloy/.rubies/ruby-2.3.0/lib/ruby/2.3.0/resolv.rb:95:in `getaddress'
	from /Users/eloy/.rubies/ruby-2.3.0/lib/ruby/2.3.0/resolv.rb:45:in `getaddress'
	from (irb):1
	from /Users/eloy/.rubies/ruby-2.3.0/bin/irb:11:in `<main>'

I’m not sure which one I should use, any thoughts?

probably the one from socket?

perhaps a new exception would make sense... SocketNameResolutionError < SocketError?

Sure, I can do that, and keep the same error message, yeah?

@ioquatix Like so?

Yeah that seems fine.

@tarcieri still waiting for travis issues to be fixed, boop :)

@ioquatix that was on nio4r. The test failures look like the same on master (we never got celluloid-io back to green)

ah okay I guess I'll take a look at the issues here.

authors: alloy ioquatix tarcieri

repo: celluloid/celluloid-io


text:
Cofactor Trait

We should have a trait for scaling by cofactor as well. We use more efficient methods for G2, and will possibly use a more efficient methods for G1 on BLS12 curves as well.
Originally posted by @kobigurk in https://github.com/celo-org/bls-zexe/pull/140

We decided to focus on Celo's use case for the time being.

authors: gakonst kobigurk

repo: celo-org/celo-bls-snark-rs


text:
Clean the README

Clean the Readme.md file

Thanks @Ankush263. This is fixed on PR #182

repo: celo-org/celo-composer


text:
[Suggestion] Reset security PIN option should be shown as if user forgot the created security PIN they won't change it in the app.

Frequency: 100%
App version: Pilot build v1.5.1
Repro on:  Google Pixel XL (8.1), Redmi Note 7 Pro (9.0)
Repro Steps:

Launch the app.
Proceed for payment.
The user forgot the password and enter the wrong PIN “Incorrect PIN” error is shown.
The user is not able to change the created PIN.

Impact:  User is blocked to proceed for transaction.
Investigation: User is able to change the device PIN if they forgot it.
Current Behavior: Reset security PIN option is not shown.
Expected Behavior: Reset security PIN option should be shown

Hi @nityas can you please confirm is it wont fixed for now?

repo: celo-org/celo-monorepo


text:
Exemplar: Use generic interface for attachment values.

Updates #1058.
Similar to Java PR https://github.com/census-instrumentation/opencensus-java/pull/1779. Though in Go I think we can take advantage of the flexibility on interfaces so I just used interface{} for generic attachment values. WDYT?

FYI this PR doesn't contain user-visible changes. Will merge by EOD today if there're no other comments/concerns.

repo: census-instrumentation/opencensus-go


text:
Fix : vulnerabilities in resolveHostName.php

Description
Fixed vulnerabilities in host form
Fixes # MON-15395
Type of change

[x] Patch fixing an issue (non-breaking change)
[ ] New functionality (non-breaking change)
[ ] Breaking change (patch or feature) that might cause side effects breaking part of the Software

Target serie

[ ] 21.04.x
[x] 21.10.x
[x] 22.04.x
[x] 22.10.x (master)

 How this pull request can be tested ?

Go to “Configuration > Hosts > Hosts”
Edit/Create a host
Define a “ IP Address / DNS” like www.google.com
Check the DNS is replaced by IP address

Checklist
Community contributors & Centreon team

[x] I have followed the coding style guidelines provided by Centreon
[ ] I have commented my code, especially new classes, functions or any legacy code modified. (docblock)
[ ] I have commented my code, especially hard-to-understand areas of the PR.
[x] I have rebased my development branch on the base branch (master, maintenance).

migrated from https://github.com/centreon/centreon/pull/12036

repo: centreon/centreon-gha


text:
fix check nmapexist on debian

command is a shell-buitin, so command -v nmap must use shell module
Signed-off-by: Shengjing Zhu zsj950618@gmail.com

Why? Can we see the error? command should work fine since there are no pipes or/and special characters.

@leseb unlike centos, there is no /usr/bin/command in debian. command is a shell built in.
zsj@debian ~ $ which command
command: shell built-in command

authors: leseb zhsj

repo: ceph/ceph-ansible


text:
Fix rolling update playbook

Running ansible-playbook rolling_update.yml to upgrade a Ceph cluster from 14.2.6 to 14.2.7 failed due to the three issues addressed by the three commits in this PR:

Wrong dashboard.yml playbook import path;
norebalance flag remains set after upgrade;
ceph.conf.j2 template fails to render if mon_host_v1 is disabled.

Hm, the playbook syntax check is unhappy with the new import path, it cannot find the dashboard.yml playbook anymore.
It's a bit ugly, but I could create a symlink in infrastructure-playbooks pointing to ../dashboard.yml; that would make the CI happy, and would still allow rolling_update.yml to be run from the root of the repository without issues.

Thanks for the review! I've replied to your comments inline below.

Two of the three issues mentioned are wrong

Wrong dashboard.yml playbook import path;

that's because you're running the rolling_update.yml playbook from the infrastructure-playbook directory which is wrong.
You should use either the full or relative path to the playbook
$ ansible-playbook -vv -i hosts infrastructure-playbook/rolling_update.yml
or
$ ansible-playbook -vv -i /path/hosts /path/ceph-ansible/infrastructure-playbook/rolling_update.yml

Actually, I'm running it as described in infrastructure-playbooks/README.md (i.e. I copied rolling_update.yml to ceph-ansible's root directory, and ran ansible-playbook rolling_update.yml), which doesn't work. I had also tried ansible-playbook infrastructure-playbook/rolling_update.yml, but it failed too (for a different reason, I think because it didn't find my group_vars and host_vars directories in that case, but I'm not sure).
So either it's a documentation issue and the README needs to be updated, or the playbook import path does need to be updated.

norebalance flag remains set after upgrade;

This flag isn't present on master so I assume you're talking about the stable-4.0 branch (due to nautilus).
So this was already fixed by [1]

You're absolutely right, I was using the stable-4.0 branch and didn't realize that the issue didn't apply to master.

ceph.conf.j2 template fails to render if mon_host_v1 is disabled.

This looks like a valid issue
[1] 675b678

Actually, I'm running it as described in infrastructure-playbooks/README.md (i.e. I copied rolling_update.yml to ceph-ansible's root directory, and ran ansible-playbook rolling_update.yml), which doesn't work. I had also tried ansible-playbook infrastructure-playbook/rolling_update.yml, but it failed too (for a different reason, I think because it didn't find my group_vars and host_vars directories in that case, but I'm not sure).
So either it's a documentation issue and the README needs to be updated, or the playbook import path does need to be updated.

It's a documentation issue as I mentioned in https://github.com/ceph/ceph-ansible/pull/5001#issuecomment-579573413

@dsavineau I tried again running ansible-playbook infrastructure-playbook/rolling_update.yml, and I can confirm that it doesn't work. It fails with
TASK [ceph-facts : set_fact _current_monitor_address]
*******************************************************************************
Thursday 06 February 2020  07:19:56 +0000 (0:00:01.105)       0:02:20.912 *****
fatal: [mon-01]: FAILED! =>
  msg: '''_monitor_addresses'' is undefined'
fatal: [mon-02]: FAILED! =>
  msg: '''_monitor_addresses'' is undefined'
fatal: [mon-03]: FAILED! =>
  msg: '''_monitor_addresses'' is undefined'
fatal: [osd-01]: FAILED! =>
  msg: '''_monitor_addresses'' is undefined'
fatal: [osd-02]: FAILED! =>
  msg: '''_monitor_addresses'' is undefined'
fatal: [osd-03]: FAILED! =>
  msg: '''_monitor_addresses'' is undefined'

(I have monitor_interface defined in group_vars/all.yml.)
If instead I copy the playbook to the root of the repository and run ansible-playbook rolling_update.yml, it works fine; so I think the README is correct, it is still required to copy those playbooks, otherwise Ansible looks for group_vars and host_vars in the wrong place.
Are you seeing a different behavior?

@dsavineau I tried again running ansible-playbook infrastructure-playbook/rolling_update.yml, and I can confirm that it doesn't work. It fails with
TASK [ceph-facts : set_fact _current_monitor_address]
*******************************************************************************
Thursday 06 February 2020  07:19:56 +0000 (0:00:01.105)       0:02:20.912 *****
fatal: [mon-01]: FAILED! =>
  msg: '''_monitor_addresses'' is undefined'
fatal: [mon-02]: FAILED! =>
  msg: '''_monitor_addresses'' is undefined'
fatal: [mon-03]: FAILED! =>
  msg: '''_monitor_addresses'' is undefined'
fatal: [osd-01]: FAILED! =>
  msg: '''_monitor_addresses'' is undefined'
fatal: [osd-02]: FAILED! =>
  msg: '''_monitor_addresses'' is undefined'
fatal: [osd-03]: FAILED! =>
  msg: '''_monitor_addresses'' is undefined'

(I have monitor_interface defined in group_vars/all.yml.)
If instead I copy the playbook to the root of the repository and run ansible-playbook rolling_update.yml, it works fine; so I think the README is correct, it is still required to copy those playbooks, otherwise Ansible looks for group_vars and host_vars in the wrong place.
Are you seeing a different behavior?

I don't see any behavior on my side because the {group,host}_vars directories and the inventory are correctly configured.
Your {group,host}_vars directories aren't loaded because those directories should be either:

in the playbook directory
in the inventory directory

See [1] for more information
So I suppose you have your group_vars directory in the main ceph-ansible directory and the inventory file somewhere in /etc/ansible/hosts (as you're not using -i).
This is working on initial deployment because the main playbooks (site.yml and site-container.yml) are already in the root ceph-ansible directory.
See also https://github.com/ceph/ceph-ansible/issues/4968 for a similar issue
[1] https://docs.ansible.com/ansible/latest/user_guide/intro_inventory.html#organizing-host-and-group-variables

authors: BenoitKnecht dsavineau


text:
nfs: make use of latest sidecars in the deployment

The sidecars in the NFS deployment has latest versions which is
also updated for RBD and CephFS drivers. This commit update
the versions in the NFS deployment too.
Additional Note for reviewer:
provisioner update has been taken care as part of #3186
Signed-off-by: Humble Chirammal hchiramm@redhat.com

/retest ci/centos/k8s-e2e-external-storage/1.21

/retest ci/centos/mini-e2e-helm/k8s-1.21

/retest ci/centos/mini-e2e-helm/k8s-1.21

@Mergifyio refresh

/retest ci/centos/mini-e2e-helm/k8s-1.21

/retest ci/centos/mini-e2e-helm/k8s-1.21

@Mergifyio rebase

/retest ci/centos/mini-e2e/k8s-1.23

@Mergifyio rebase

/retest ci/centos/mini-e2e-helm/k8s-1.21

/retest ci/centos/mini-e2e-helm/k8s-1.21

@Mergifyio refresh

@Mergifyio rebase

ci/centos/mini-e2e-helm/k8s-1.23

/retest ci/centos/mini-e2e/k8s-1.21

/retest ci/centos/mini-e2e-helm/k8s-1.23

@Mergifyio refresh

ci/centos/mini-e2e-helm/k8s-1.22

ci/centos/mini-e2e-helm/k8s-1.23

/retest ci/centos/mini-e2e-helm/k8s-1.23

/retest ci/centos/mini-e2e-helm/k8s-1.23

@mergifyio rebase

/retest  ci/centos/mini-e2e/k8s-1.23

/retest ci/centos/mini-e2e/k8s-1.23

authors: Rakshith-R humblec

repo: ceph/ceph-csi


text:
ceph_deploy/hosts/suse/install.py: SUSE zypper ref

zypper may not pull in latest packages if the repository cache is not updated
so we should do a 'zypper ref' before installing packages.
Signed-off-by: Owen Synge osynge@suse.com

this patch has caused ceph testing to break dont know why though

repo: ceph/ceph-deploy


text:
Provide a mechanism for setting "ceph.conf" configuration values so they will be in effect during "cephadm bootstrap" phase

Nowadays, Ceph has a MON store for cluster configuration, but certain options still have to be set in ceph.conf before the cluster is bootstrapped.
One example is osd crush chooseleaf type = 0. If this is not provided on the cephadm bootstrap command line via the -c option, the initial CRUSH map created by cephadm bootstrap will have the failure domain set to "host" and there is no easy way to change that.
It's possible that there are other options like this one, which must be set via cephadm bootstrap -c in order to properly take effect.
Therefore, I am proposing that ceph-salt provide a mechanism for setting these options.

UPDATE: I found another situation where this is needed (and I think it's quite likely that there are more):
If someone needs to run cephadm bootstrap with MGR debugging turned up, the only way to do this is via cephadm bootstrap -c.

Hm, that should not end up in the ceph.conf. Instead, I'd put this into the mon store. In general there is only a very limited amout of config that should live within the ceph.conf.

@sebastian-philipp @trociny Allow me to elaborate a little.
On a single-node cluster, we typically want the failure domain to be "osd", not "host". I can achieve this by doing:
echo -en "[global]\n        osd crush chooseleaf type = 0\n" > /root/ceph.conf
cephadm bootstrap -c /root/ceph.conf

After this command runs, there is a MON, a MGR, a pool, and a CRUSH map (with failure domain "host"), although there are still no OSDs at this point.
Once the CRUSH map is created, there is no easy way to modify it. Hence this issue.
Therefore, I do not understand what you mean by "put this into the mon store".

I will edit the issue description to emphasize that this mechanism would/should only be used for ceph.conf settings that must be made prior to bootstrapping, like osd crush chooseleaf type = 0

https://tracker.ceph.com/issues/44284
you're right. the workaround looks ok to me.

If someone needs to run cephadm bootstrap with MGR debugging turned up, the only way to do this is via cephadm bootstrap -c.

@smithfarm  Can you please share what exactly has to be added to the ceph.conf?

If someone needs to run cephadm bootstrap with MGR debugging turned up, the only way to do this is via cephadm bootstrap -c.

@smithfarm Can you please share what exactly has to be added to the ceph.conf?

Nevermind, ceph-salt should be flexible enough to allow the user to set any option.

repo: ceph/ceph-salt


text:
github: update PR template with new checklist item and helpful links

Add a new checklist item reminding contributors to run make api-update to track new APIs. Add two short paragraphs that cover some of the common things I copy and paste or need to look up when working with contributors' PRs.

@Mergifyio rebase

repo: ceph/go-ceph


text:
Mock zfs Provider

Issue Description
In order to complete adequate testing of #159 Status Pusher, several mock providers must be made.  The scope of this card is to create a mock zfs provider.  The real provider is being handled in #168.
Acceptance Criteria (How do we know when this is done?)
Functioning well enough to complete testing on #159.
Related Tickets
#159

duplicate of #168

repo: cerana/cerana


text:
Error with default sorting on relationship field

Hello!
I'm using version 0.8.1 (also Rails 4.2.5.1 and Ruby 2.1.10) and following code
class A < Resource
    attributes :name
    has_one :b

    def self.default_sort
      [{field: 'b.name', direction: :asc}, {field: 'name', direction: :asc}]
    end
end

class B < Resource
    attributes :name
end

results with following error, when I'm trying to make any request to A's resource.
ERROR -- : Internal Server Error: undefined method `model' for #<Class:0x000000042533a8> /vendor/bundle/ruby/2.1.0/gems/activerecord-4.2.5.1/lib/active_record/dynamic_matchers.rb:26

It is happening in apply_sort method, when try to get records.model.to_s:
associations = _lookup_association_chain([records.model.to_s, *model_names])

Is it a bug or something is wrong with my code?

Also happens in Rails 5.1.1 w/ ruby 2.3.3 with a has one when relationship fields are added to sorts like this:
has_one :account ...   def self.sortable_fields(context)     super + [:"account.name"]   end

authors: begor nbarthel

repo: cerebris/jsonapi-resources


text:
update release command to be correct.

I had the command in the docs wrong. Had to run a different command to update the package.

:+1:

repo: cerner/canadarm


text:
[time-input] Remove Unsupported Translation Files

Issue Description
At this point we only support these i18n languages: ['ar', 'en', 'en-US', 'en-GB', 'es', 'es-US', 'es-ES', 'de', 'fi-FI', 'fr', 'fr-FR', 'pt', 'pt-BR']
In the time-input component, it seems a couple unsupported translation files were added. These should be removed until these locales are supported:

nl
sv

Issue Type

[ ] New Feature
[ ] Enhancement
[ ] Bug
[x] Other

We've recently added nl and nl-BE locales and will be adding sv in the near future.

This a non-issue then?

authors: bjankord emilyrohrbough yuderekyu

repo: cerner/terra-core


text:
Add wdio tests for dynamic grid

Summary
Replaced nightwatch tests with wdio tests
Addresses https://github.com/cerner/terra-core/issues/1302

Lets move all these test examples to the new dir structure to drop reference to nightwatch all together
/component
    /examples
        /test-examples
            ...examples

@emilyrohrbough Updated: https://github.com/cerner/terra-core/pull/1471/commits/604ef0032d7de375d395254be59a0d8d12bf8c14

I think the generate-config script needs ran as well

authors: bjankord emilyrohrbough


text:
cmctl as downloaded for 1.8.0 shows canary as its version

Describe the bug:

I just downloaded cmctl from this repository and running cmctl version doesn't show the command actual version (1.8.0). I was expecting 1.8.0 but instead is shows canary.
This is the current output
Client Version: util.Version{GitVersion:"canary", GitCommit:"", GitTreeState:"", GoVersion:"go1.17.8", Compiler:"gc", Platform:"linux/amd64"}
Server Version: &versionchecker.Version{Detected:"v1.7.1", Sources:map[string]string{"crdLabelVersion":"v1.7.1"}}

If I run the same using version 1.7.2, then this is fine and the correct version is displayed
Client Version: util.Version{GitVersion:"v1.7.2", GitCommit:"2e0bfc87d0c63c473c31a17f4c8c65e89806dc16", GitTreeState:"clean", GoVersion:"go1.17.8", Compiler:"gc", Platform:"linux/amd64"}
Server Version: &versionchecker.Version{Detected:"v1.7.1", Sources:map[string]string{"crdLabelVersion":"v1.7.1"}}

Expected behaviour:

cmctl version shoud show version 1.8.0.
Steps to reproduce the bug:

download cmctl from https://github.com/cert-manager/cert-manager/releases/download/v1.8.0/cmctl-linux-amd64.tar.gz
unpack the archive via tar -zxf cmctl-linux-amd64.tar.gz
run ./cmctl version

Anything else we need to know?:
Environment details::

Kubernetes version: 1.22
Cloud-provider/provisioner: bare metal
cert-manager version: 1.8.0 (client), 1.7.1 (deployed)
Install method: e.g. helm

/kind bug

Hi! That is strange, I thought I had fixed this in https://github.com/cert-manager/cert-manager/pull/4968. 😞
I will investigate now.

It seems like we can't have multiple -ldflags in a go build command (source).
The make command
make bin/cmctl/cmctl-linux-amd64

runs go build with two consecutive -ldflags. The second one "cancels" the first:
GOOS=linux GOARCH=amd64 CGO_ENABLED=0 GOMAXPROCS= \
  go build -o bin/cmctl/cmctl-linux-amd64 -trimpath \
  -ldflags '-w -s -X github.com/cert-manager/cert-manager/pkg/util.AppVersion=v1.8.0-6-gd212165c8da228 -X github.com/cert-manager/cert-manager/pkg/util.AppGitCommit=d212165c8da228437e71a74c2e117ae9d62c7f24' \
  -ldflags '-X "github.com/cert-manager/cert-manager/cmd/ctl/pkg/build.name=cmctl" -X "github.com/cert-manager/cert-manager/cmd/ctl/pkg/build/commands.registerCompletion=true"' cmd/ctl/main.go

The right command would look like this:
GOOS=linux GOARCH=amd64 CGO_ENABLED=0 GOMAXPROCS= \
  go build -o bin/cmctl/cmctl-linux-amd64 -trimpath \
  -ldflags '-w -s -X github.com/cert-manager/cert-manager/pkg/util.AppVersion=v1.8.0-beta.0-4-g312baa4d7e0a25 -X github.com/cert-manager/cert-manager/pkg/util.AppGitCommit=312baa4d7e0a25222571f8cdb1c95356a28a70d3 -X "github.com/cert-manager/cert-manager/cmd/ctl/pkg/build.name=cmctl" -X "github.com/cert-manager/cert-manager/cmd/ctl/pkg/build/commands.registerCompletion=true"' cmd/ctl/main.go

I'll open a PR to fix this. (cc @SgtCoDFish)

bumping this issue so it doesn't auto-close.

bumping this issue so it doesn't auto-close.

@aureq Thank you for bumping - this issue totally flew under the radar and your bump helped me realise that! Thank you for getting involved :raised_hands:  :grin:

authors: SgtCoDFish aureq maelvls

repo: cert-manager/cert-manager


text:
Add global image repository value to helm values to facilitate private repo and eiliminate need to maintain individual repo image paths

Is your feature request related to a problem? Please describe.
because we use a private artifactory repo, we are forced change the fqdn portion of XXX.image.repository for each container image in our helm values.  the problem presents itself when there is an image name change like in the current v1.15.0 release for apistartup image causing manual upkeep of those image helm values.
Describe the solution you'd like
add to helm values a global image repository fqdn value that overrides all the image fdqns.
Describe alternatives you've considered
add fdqn value to each XXX.image block
Environment details (remove if not applicable):

Kubernetes version: 1.29
Cloud-provider/provisioner: AWS EKS
cert-manager version: v1.15.0
Install method: helm via argocd

/kind feature

We've had a short discussion on that topic during this morning's open standup. Erik said "I find this type of top-level value useful as a user, but Go templates makes it painful to maintain".
It seems like a reasonable request.
/good-first-issue

Do we need some kind of helper chart to reduce code duplication across all charts (cert-manager, trust-manager, approver-policy, etc.)?

/remove-lifecycle stale


text:
Reason for spawning a new process in authentication and authorization extensions ?

I have been using docker auth authorization and authentication extension for a project. I have observed that for each and every authentication request, a new process is spawned. This result  in higher resource consumption. Is there a particular reason for implementing like this, without giving the developer to add a plugin (Or at least communicate through UNIX socket) ? Introducing a plugin may avoid spawning a new process.
Thank you ..!

ext_authn/autz is meant for quick hacks / low volume usage cases.
next step currently is implementing the Authenticator or Authorizer interface in the source.
i agree that there is a place for an intermediate option for a shared library or socket interface. the reason it doesn't exist is that no one had the need or time to implement it yet. you are welcome to do so.

I have implemented a new custom authentication and authorization mechanism where the developers can add their own plugins to existing docker auth implementation. As you mentioned in the comment, I implemented the authenticator and authorizer interfaces in doing so. I did it like that to make this backward compatible with the existing users. I have tested this new authentication and authorization implementations with my authentication and authorization plugins.
Please review this PR and merge
Highly appreciated if you can review this quickly and merge since I am going to use docker auth implementation for a production project at WSO2.
If we can get this merge I will write the necessary documentation and examples to use the new implementation as the already existing docs in your repo.
Thank you ..!

I have fixed the comments for the PR. I had a question there. Please review those changes and let me know further changes.
Thank you ..!

Hi @rojer
It would be great if you can help us to get this merged and released since one of our solutions needs this improvement.

@hasinthaindrajee i assume you were talking about https://github.com/cesanta/docker_auth/pull/254
reviewed, still some work necessary to get it in.

Hi @rojer
I have done the remaining necessary fixes. Can you please review it and merge?

Hi @rojer , Thanks for helping out to get the PR merged. Any idea when we can get the repo released with this change ?

let me take a look at some other PRs and i'll push latest

built and pushed cesanta/docker_auth:latest

authors: hasinthaindrajee rojer tharindulak


text:
🛑 echo-server.madacluster.tech is down

In a1af8c7, echo-server.madacluster.tech (https://echo-server.madacluster.tech) was down:

HTTP code: 0
Response time: 0 ms

Resolved: echo-server.madacluster.tech is back up in 04ddbce after 14 minutes.

repo: cesarempathy/upptime


text:
Update environment variable

As per the official Android documentation, the $ANDROID_HOME environment variable is now deprecated -> ANDROID_HOME, which also points to the SDK installation directory, is deprecated.
This PR simply replaces every occurrences of $ANDROID_HOME with $ANDROID_SDK_ROOT. It also modifies a bit the Notes section to directly link to the official documentation.
Let me know

Hi @cesarferreira any news on this?

Thank you for your contribution

Thank you for your contribution

My pleasure. It's just a small thing 😊

repo: cesarferreira/dryrun


text:
Přesměrování data.cesko.digital na GitHub

Kdyby se v tom někdo vrtal, ať ho to přesměruje na GitHub repo s dokumentací.

Je to celé trochu hack (servírovat legitimní obsah jako chybovou stránku), ale dává mně to smysl, přidám – díky!

@zoul, tak redirect status kód asi není možný. Nechat tedy 403 nebo změnit na 200?
Mimochodem tyhle následné opravy, které nelze odhalit předem, jsou opravdu zábavné. 😄 😕

Jáj 🤦😁 Pokud je možné 200, bral bych 200, sémanticky to líp odpovídá. Pokud ne, tak vraťme 403 a zapomeňme 😄

authors: HormCodes zoul

repo: cesko-digital/assets


text:
feat: navigator adjustments

https://cesko-digital.atlassian.net/browse/LOON-273 + změny z komentů z https://github.com/cesko-digital/loono/pull/43
status: WIP (není ready na review)

@mzdm nechci nijak tlacit, ale jak je to s timhle prkem? je tam jen dost zmen, tak aby se to pripadne resolvnulo co nejrychleji

Jj do konce týdne snad (ještě tam jsou nutný nějaký clean upy)

@mzdm promin ze s timhle furt prudim, ale jak to vidis? pripadne to muzem proste zavrit a rict ze se to udela nekdy jindy, ale jde mi o to at tu nevisi neco takoveho podivneho

@killalad  yep toto zavírám, teď to nestihám

authors: killalad mzdm

repo: cesko-digital/loono


text:
1.6beta2 on OSX

I can't seem to get reflex to work.. perhaps I'm using it incorrectly.. im doing reflex -g * -- echo hi I've also tried reflex -g ./* echo hi .. nothing happens when I change those files

Thanks for the report. I'll take a look.

Hi @pkieltyka,
The way you're invoking reflex, the * is expanded by your shell, so if you have (for example) a.txt and b.txt in your current directory, reflex sees
reflex -g a.txt b.txt -- echo hi

So then if you change b.txt, nothing happens; if you change a.txt, then reflex will try to run b.txt, which is definitely not what you intended.
If you really want to run a change after any file changes, you shouldn't provide -g (or -r) -- by default, reflex considers all file changes in the current directory.
reflex echo hi

If you want to use a glob pattern with a *, you'll want to quote it:
reflex -g 'foo*' -- echo hi

Finally, use -v if you're ever not sure what reflex is doing -- it will print out exactly the configuration that it's using so you can see what's going on.
Hope this helps!

thanks it certainly helps! and it works perfectly.

authors: cespare pkieltyka

repo: cespare/reflex


text:
Inner types in prototype style

Inner types are currently only partly functional in prototype style. For instance, this doesn't work:
class Outer() {
    class Inner() {
        shared String s = "hello";
    }
    shared String test() { return Inner().s; }
}

In the generated JS code test can't access Outer.Inner() properly because it is defined inside Outer and is not exported. That could be fixed of course. But perhaps the real problem is that we mix prototype style and capture style here: while methods are defined separately and added to the prototype, inner types are defined inside the body of the outer type, creating a closure. It seems that this mixing of concepts is not completely trouble-free.
It would be more consistent, and probably work out better in the long term, to define inner types in the same way as other class members in prototype style. It would also definitely be more efficient because currently inner types are completely initialized every time an instance of the outer class is created.
Implementing this is definitely not uncomplicated either though.

After playing around with some JavaScript code I think it should be possible to move the inner types out of the class body. I'll need to refactor the generated code a bit first.

Inner types in prototype style are now added to the prototype instead of being defined in the class body. All the tests still pass, and correct code is generated for the example given above. As a nice side effect the prototype style code is now more readable because the members are grouped properly.
But there are probably some cases that still don't work, for both prototype style and capture style, in particular if inner and outer types have identical names.

Ok, this seems to work pretty well now. The problems with identical names of inner and outer types are not directly related to this, so I'll open a separate issue for that.

repo: ceylon/ceylon-js


text:
Wrappers for covariant Java Collections

As discussed here.

@jvasileff Thanks!
One question: why doesn't CeylonMutableXxxx extend CeylonXxxx?

Laziness, I guess :)
Should be fixed.

Thank you!

authors: gavinking jvasileff

repo: ceylon/ceylon-sdk


text:
Figure out the welcome/new user email notification situation

I want to hook into this: https://codex.wordpress.org/Plugin_API/Action_Reference/user_register

v6.3.0 - https://github.com/cferdinandi/gmt-wordpress-for-web-apps/pull/64

repo: cferdinandi/gmt-wordpress-for-web-apps


text:
Need help getting it work in React

Hi guys! So far i've managed to import the js file into page and add ID's to links.
But still can't get it to work with the last step, adding the var scroll = new SmoothScroll('a[href*="#"]');
Can you guys help me to set it up correctly in React JS?
Thanks in advance!

I'd need to see a demo link to understand what's not working/why.

Does the div you want to scroll to have an id of id?

@cferdinandi yes of course. See my previous message. I just don't know where and how to add
var scroll = new SmoothScroll('a[href*="#"]');

Ah. Anywhere after you've loaded/imported smooth-scroll.js. Header, footer, doesn't matter.

Don't know if you've used react, i'm doing it so
    componentDidMount () {
        const script = document.createElement("script");

        script.src = "../utils/scroll.js";
        script.async = true;

        document.body.appendChild(script);
    }

and in console/network i don't see it loading.

Without a working demo I can look at, I'm afraid I've reached the limit of the debugging I can help you with. Let me know if/when you get something setup that I can look at.

BTW, this may help you out? https://stackoverflow.com/a/42848407/1293256

Alright, i was using gatsby and managed to fix it by adding each script as a separate file. Thanks!

authors: cferdinandi ziyafenn

repo: cferdinandi/smooth-scroll


text:
Configuration

I'm planning on running this for prod, how do I configure it? (i.e. different port, etc)

Added description of .env to README at 063ce36617b259a3cf7cdd70a5a083ee6ae9264f.
In case you are interested, here is a note on how to write .env in prod.
Create .env.production or .env.production.local and write the following
CK=[your consumer key for twitter API]
CS=[your consumer secret for twitter API]
TW_OAUTH_CALLBACK=[your server's URL]/twitter_callback # e.g. https://hogehoge.com/twitter_callback
MK_CALLBACK=[your server's URL]/misskey_callback # e.g. https://hogehoge.com/misskey_callback

authors: ThatOneCalculator cffnpwr

repo: cffnpwr/caffe-bruncher


text:
Added testing for web-storage-proxy.js

Added testing for web-storage-proxy.js
Additions

Added tests to increase coverage to 100% across the board

Changes

Refactored code to be less complex and allow for easier testing
Updated code to utilize storage object instead of boolean to allow for storage options outside the window object

Testing

Run gulp test:unit:scripts see the green across the board for this file

Review

@KimberlyMunoz
@anselmbradford
@sebworks

Screenshots
Before
=============================== Coverage summary ===============================
Statements   : 26.51% ( 321/1211 )
Branches     : 21.84% ( 90/412 )
Functions    : 22.49% ( 56/249 )
Lines        : 26.75% ( 321/1200 )
After
=============================== Coverage summary ===============================
Statements   : 28.37% ( 343/1209 )
Branches     : 24.27% ( 100/412 )
Functions    : 24.5% ( 61/249 )
Lines        : 28.54% ( 343/1202 )
Notes

Skipping tests of internal private functions

Checklist

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

LGTM, other than the one comment.

:+1:

Code comment at the top of web-storage-proxy.js needs to be updated or removed.

authors: anselmbradford jimmynotjim sebworks

repo: cfpb/cfgov-refresh


text:
Clean up the Hero and Summary code blocks

Clean up the Hero and Summary code blocks to be more semantic and remove extra containers
Additions

Created .hero_preview block to combine .hero_card-bg and .hero_card-padding styles
Added header and footer blocks to .summary block
Added last-child hook to remove margin from last child within summary no matter what elem

Removals

Removed extra .hero_card-bg and .hero_card-padding blocks
Removed extra whitespace

Changes

Moved hero card url condition to outside of the markup
Swapped divs for more semantic article where applicable

Testing
Fetch branch

navigate to /blog page
navigate to /newsroom page

Review

@anselmbradford
@sebworks

Preview

No visual differences

Notes

The image container is needed to suppress loading of the images until they are needed
Fixes #257

If I can get a merge I'll integrate these changes into the Sprint-21 Hero and events list I'm currently working on.


text:
Search type selector doesn't work properly

The "Things to search for" selector in the search sidebar doesn't seem to work properly. If you select "HTML" or "Components", the resulting search still goes to the "Links" view:

The search should modify the search_type parameter in the query string -- you can manually trigger the HTML/components search by setting ?search_type=html or ?search_type=components.

Yes! This needs to be fixed. Currently it's two separate forms. Very dumb.
Couple options I see:

Combine the forms at the HTML level by having all the form elements contained within a single <form> element and have two submit buttons. This is a pain due to the page's layout.
Continue having two separate forms but use JS to sync the forms by adding type=hidden form elements that get updated whenever an option or search term is changed.
Remove the Update button and use AJAX to update the results in real-time when sidebar options are changed. This is what iRegs does (try changing a sidebar option). There's a separate endpoint that returns partial HTML that the front-end fetches and injects into the DOM.

iRegs has a no-JS fallback that shows an Update button for when JS is disabled by the user. Our template is based on iRegs but has no AJAX functionality, hence our ugly update button.
Option 2 above is easiest to implement so I'll start on that right now.

authors: chosak contolini

repo: cfpb/crawsqueal


text:
Additional Print Styles

Made some adjustments on print styles to reduce the number of pages being printed.
Since Chrome prints with mobile styles and Firefox/IE prints on desktop styles, the styles need to be flexible enough to work with both.
These aren't perfect, but they're a huge improvement, especially for Firefox/IE where a lot of text was being cut off.
Testing

Review and Print Preview the following pages:

http://localhost:8000/owning-a-home/explore-rates/
http://localhost:8000/owning-a-home/closing-disclosure/
http://localhost:8000/owning-a-home/loan-options/
http://localhost:8000/owning-a-home/loan-options/FHA-loans
http://localhost:8000/owning-a-home/process

Amazing.

authors: KimberlyMunoz contolini

repo: cfpb/owning-a-home


text:
Support native HTTPS  in application

Currently in order to target new clients we route HTTPS via nginx to strip it to HTTP before sending it to our application.
This is not very portable so I'd like to support HTTPS natively.

The main issue with this currently is that Java wants you to use JKS and we'd prefer to allow PEM to prevent use of some proprietary format.

Looking into this issue Java doesn't appear to budge with the standard server sockets and we are forced to use JKS for the time being. I will be looking into a replacement for this in due time as the preferred certificate should be PEM.