Bug on Mac OS

prebuild-install@4 does not work on mac os v10.13.3
npm install rocksdb

><EMAIL_ADDRESS>install
> prebuild-install || node-gyp rebuild

Illegal instruction: 4  prebuild-install

I'm not on mac myself so difficult for me to verify this. Anyone else that can verify/reproduce this? cc @prebuild/builders

No Mac here either :/

Tried running npm install rocksdb on Mac OS 10.12.6, seems to have worked without issues.

Log
~/dev/lab/prebuild-repro ∴ npm install rocksdb

><EMAIL_ADDRESS>install /Users/ng/dev/lab/prebuild-repro/node_modules/rocksdb
> prebuild-install || node-gyp rebuild

/Users/ng/dev/lab/prebuild-repro
└─┬<EMAIL_ADDRESS>  ├─┬<EMAIL_ADDRESS>  │ └──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  ├──<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ └──<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ └──<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ ├─┬<EMAIL_ADDRESS>    │ │ ├──<EMAIL_ADDRESS>    │ │ └─┬<EMAIL_ADDRESS>    │ │   ├──<EMAIL_ADDRESS>    │ │   ├──<EMAIL_ADDRESS>    │ │   ├──<EMAIL_ADDRESS>    │ │   ├──<EMAIL_ADDRESS>    │ │   ├──<EMAIL_ADDRESS>    │ │   └──<EMAIL_ADDRESS>    │ ├──<EMAIL_ADDRESS>    │ ├─┬<EMAIL_ADDRESS>    │ │ ├──<EMAIL_ADDRESS>    │ │ ├──<EMAIL_ADDRESS>    │ │ ├──<EMAIL_ADDRESS>    │ │ ├──<EMAIL_ADDRESS>    │ │ ├─┬<EMAIL_ADDRESS>    │ │ │ ├──<EMAIL_ADDRESS>    │ │ │ └─┬<EMAIL_ADDRESS>    │ │ │   └──<EMAIL_ADDRESS>    │ │ ├─┬<EMAIL_ADDRESS>    │ │ │ └──<EMAIL_ADDRESS>    │ │ └──<EMAIL_ADDRESS>    │ └──<EMAIL_ADDRESS>    ├──<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ ├──<EMAIL_ADDRESS>    │ └─┬<EMAIL_ADDRESS>    │   └──<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ ├──<EMAIL_ADDRESS>    │ ├──<EMAIL_ADDRESS>    │ └──<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ ├─┬<EMAIL_ADDRESS>    │ │ └──<EMAIL_ADDRESS>    │ └──<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ ├──<EMAIL_ADDRESS>    │ ├──<EMAIL_ADDRESS>    │ └─┬<EMAIL_ADDRESS>    │   ├──<EMAIL_ADDRESS>    │   ├─┬<EMAIL_ADDRESS>    │   │ ├──<EMAIL_ADDRESS>    │   │ └──<EMAIL_ADDRESS>    │   ├──<EMAIL_ADDRESS>    │   └──<EMAIL_ADDRESS>    ├─┬<EMAIL_ADDRESS>    │ └──<EMAIL_ADDRESS>    └──<EMAIL_ADDRESS>
npm WARN enoent ENOENT: no such file or directory, open '/Users/ng/dev/lab/prebuild-repro/package.json'
npm WARN prebuild-repro No description
npm WARN prebuild-repro No repository field.
npm WARN prebuild-repro No README data
npm WARN prebuild-repro No license field.
~/dev/lab/prebuild-repro ∴ node -v
v8.10.0
~/dev/lab/prebuild-repro ∴ npm -v
3.10.10

Which version of npm are you on?

@heri16 was using node 8.10.0 and npm 3.10.10. Tried again with node 8.11.2 and npm 6.1.0, also without errors.
~/dev/lab/prebuild-repro ∴ node -v
v8.11.2
~/dev/lab/prebuild-repro ∴ npm -v
6.1.0
~/dev/lab/prebuild-repro ∴ ls -la
total 0
drwxr-xr-x   2 ng  staff    68 Jun 11 09:30 .
drwxr-xr-x  36 ng  staff  1224 Jun 11 08:55 ..
~/dev/lab/prebuild-repro ∴ npm install rocksdb

><EMAIL_ADDRESS>install /Users/ng/dev/lab/prebuild-repro/node_modules/rocksdb
> prebuild-install || node-gyp rebuild

npm WARN saveError ENOENT: no such file or directory, open '/Users/ng/dev/lab/prebuild-repro/package.json'
npm WARN enoent ENOENT: no such file or directory, open '/Users/ng/dev/lab/prebuild-repro/package.json'
npm WARN prebuild-repro No description
npm WARN prebuild-repro No repository field.
npm WARN prebuild-repro No README data
npm WARN prebuild-repro No license field.

+<EMAIL_ADDRESS>added 66 packages from 55 contributors and audited 112 packages in 4.518s
found 0 vulnerabilities

Edit: Maybe it is node.js itself and not your addon. I don't know how they build their osx stuff

I know :wink:
@mathiask88 We create prebuilt binaries on Travis. I think you're onto something with native modules. Maybe it's trying to run a binary which that somehow doesn't match the host system.
@heri16 Is there anything special with your environment?
Please post the result of npm i rocksdb --verbose and post that log to a gist.

Nothing special other than installing node v8.10.0 via NVM. and doing an npm update to npm@next

This works:
npm rebuild rocksdb --platform=linux --loglevel verbose
This does not:
npm rebuild rocksdb --platform=darwin --loglevel verbose
npm info it worked if it ends with ok
npm verb cli [ '/Users/heri/.nvm/versions/node/v8.10.0/bin/node',
npm verb cli   '/Users/heri/.nvm/versions/node/v8.10.0/bin/npm',
npm verb cli   'rebuild',
npm verb cli   'rocksdb',
npm verb cli   '--platform=darwin',
npm verb cli   '--loglevel',
npm verb cli   'verbose' ]
npm info using<EMAIL_ADDRESS>npm info using<EMAIL_ADDRESS>npm info readInstalled object
npm verb rebuild path, id [ '/Users/heri/work/eqh-middleware/packages/eqh-graphql/node_modules/rocksdb',
npm verb rebuild<EMAIL_ADDRESS>]
npm info build /Users/heri/work/eqh-middleware/packages/eqh-graphql/node_modules/rocksdb
npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm info linkStuff<EMAIL_ADDRESS>npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>
><EMAIL_ADDRESS>install /Users/heri/work/eqh-middleware/packages/eqh-graphql/node_modules/rocksdb
> prebuild-install || node-gyp rebuild

prebuild-install info begin Prebuild-install version 4.0.0
prebuild-install info looking for cached prebuild @ /Users/heri/.npm/_prebuilds/6f44f7-rocksdb-v3.0.1-node-v57-darwin-x64.tar.gz
prebuild-install info found cached prebuild
prebuild-install info unpacking @ /Users/heri/.npm/_prebuilds/6f44f7-rocksdb-v3.0.1-node-v57-darwin-x64.tar.gz
prebuild-install info unpack resolved to /Users/heri/work/eqh-middleware/packages/eqh-graphql/node_modules/rocksdb/build/Release/leveldown.node
sh: line 1:  8456 Illegal instruction: 4  prebuild-install
gyp info it worked if it ends with ok
gyp verb cli [ '/Users/heri/.nvm/versions/node/v8.10.0/bin/node',
gyp verb cli   '/Users/heri/.nvm/versions/node/v8.10.0/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js',
gyp verb cli   'rebuild' ]

This works:
npm rebuild leveldown --platform=darwin --loglevel verbose
npm info it worked if it ends with ok
npm verb cli [ '/Users/heri/.nvm/versions/node/v8.10.0/bin/node',
npm verb cli   '/Users/heri/.nvm/versions/node/v8.10.0/bin/npm',
npm verb cli   'rebuild',
npm verb cli   'leveldown',
npm verb cli   '--platform=darwin',
npm verb cli   '--loglevel',
npm verb cli   'verbose' ]
npm info using<EMAIL_ADDRESS>npm info using<EMAIL_ADDRESS>npm info readInstalled object
npm verb rebuild path, id [ '/Users/heri/work/eqh-middleware/packages/eqh-graphql/node_modules/leveldown',
npm verb rebuild<EMAIL_ADDRESS>]
npm info build /Users/heri/work/eqh-middleware/packages/eqh-graphql/node_modules/leveldown
npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>npm info linkStuff<EMAIL_ADDRESS>npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>
><EMAIL_ADDRESS>install /Users/heri/work/eqh-middleware/packages/eqh-graphql/node_modules/leveldown
> prebuild-install || node-gyp rebuild

prebuild-install info begin Prebuild-install version 4.0.0
prebuild-install info looking for cached prebuild @ /Users/heri/.npm/_prebuilds/067087-leveldown-v4.0.1-node-v57-darwin-x64.tar.gz
prebuild-install info found cached prebuild
prebuild-install info unpacking @ /Users/heri/.npm/_prebuilds/067087-leveldown-v4.0.1-node-v57-darwin-x64.tar.gz
prebuild-install info unpack resolved to /Users/heri/work/eqh-middleware/packages/eqh-graphql/node_modules/leveldown/build/Release/leveldown.node
prebuild-install info unpack required /Users/heri/work/eqh-middleware/packages/eqh-graphql/node_modules/leveldown/build/Release/leveldown.node successfully
prebuild-install info install Successfully installed prebuilt binary!

Rocksdb is a fork of Leveldown.
If leveldown works, it just means that precompiled rocksdb does not work on darwin? Or uses certain extensions that is not supported on my hardware? I just verified that it works on docker-for-mac on the same hardware after doing npm rebuild --platform=linux
Is this an downstream issue?

Moving to: https://github.com/Level/rocksdb/issues/65


Introduce Kotlin lint (detekt)

Is your feature request related to a problem? Please describe.
To make the code base to be consistent

Ooooo can i grab this issue?

Ooooo can i grab this issue?

Sure Kar. Let’s do it

Awesome! thank you!
So just to be clear its about bring detekt in as a linter.
Is there anything more? Like implement into CI/CD or something more?

Awesome! thank you! So just to be clear its about bring detekt in as a linter. Is there anything more? Like implement into CI/CD or something more?

Yeah adding it and being able to execute the command to see analyze is good eough. It would be nice if you can implement it into the GitHub action before build step. Many thanks Kar

Close as completed in #191


KeePassGroup functions should return objects of type PSKeePass.Group

The New-KeePassGroup and Update-KeePassGroup functions return objects of type “KeePassLib.PwGroup”.
It would be more convenient receiving objects of type “PSKeePass.Group”. This would allow using them directly as input for the Update-KeePassGroup function.

thanks I'll check it out.

included in latest v.<IP_ADDRESS>


chore: bdd-sbhg-import-1573040149 to 0.0.1

chore: Promote bdd-sbhg-import-1573040149 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


feature: Direct URL Retrieval for Current File and Line in gitbrowse

Did you check the docs?

[X] I have read all the snacks.nvim docs

Is your feature request related to a problem? Please describe.
I'm often in situations where I need to share a link to a specific line of code with my coworkers. Currently, gitbrowse requires me to open the file in the browser, find the line, and then manually copy the link. This process is a bit tedious when done frequently and disrupts my workflow. I’d love a more direct way to get a link to a specific line without needing to open a browser.
Describe the solution you'd like
A method like copyToClipboard in gitbrowse would allow me to retrieve the URL of the current file and line and copy it directly to the clipboard. An alternative could be enhancing get_url to return the URL for the current file and line without requiring a repository argument. This way, I could quickly generate and copy the link for easy sharing.
Describe alternatives you've considered

Manually opening the file in the browser, finding the line, and copying the link. This works but is time-consuming.
Writing a custom script outside gitbrowse to achieve the same functionality, but I’d prefer this as part of snacks to keep my workflow consolidated in Neovim.

Additional context
If this feature aligns with the goals for gitbrowse, I’d be happy to contribute to its implementation. Any guidance on the preferred approach or API design would be appreciated. Thanks again for creating snacks—it’s a fantastic addition to the Neovim ecosystem!

just pass open = function(url)...end as an option?

@folke Thanks for the suggestion!
I also want to use the open functionality to launch the URL in the browser. My current configuration is as follows:
open = function(url)
  vim.system({ 'explorer.exe', url })
end,

owever, I’m looking for a way to make this open function versatile, so it can both open the URL in the browser and optionally copy it to the clipboard. Right now, I’m considering some workarounds to achieve both actions, but they feel a bit hacky. If you have any thoughts on a clean approach to handle both actions within open, I’d really appreciate it!

I don't get what you mean.
don't configure that open globally to get your url.
Just do it once.

Snacks.gitbrowse.open({open = function(url)Snacks.notify (url) end})

lik this:
local function get_url()
  local ret
  Snacks.gitbrowse.open({open = function(url)
    ret = url
  end})
  return ret
end

That second option wont work when there's more than one remote.
Just use the first option, as a callback

@folke thanks for help , i did miss the Snacks.gitbrowse.open opts , i'm now able to copy the url using :
snacks.gitbrowse.open({
  open = function(url)
    vim.fn.setreg('+', url)
    vim.notify('Yanked url to clipboard')
  end,
})


Make version parameter optional

The readme states that the if I'm using the programmatic api, the version argument is optional.

version - String
The Electron version with which the app is built (without the leading 'v') - for example, 0.33.9. See Electron releases for valid versions. If omitted, it will use the version of the nearest local installation of electron-prebuilt.

I realised that if I do not run have a local installation of electron-prebuilt, an error will be thrown. Instead of this behaviour, I think it would be simpler for the user if the module makes a query to determine the latest version of electron and use that directly if it is not specified. I've made this implementation in my change, and at the same time, a positive side effect would be that the --version command line flag is also an optional parameter.

@jiahaog Hi and thanks for the PR. I am OK with this, but it would be nice if there were tests for this feature so we could catch regressions in the future in case something breaks this.

Apart from the style nit, I have the same comments as Max.

I'm not entirely sure how should I go about writing a test for this. Based on the test app within the ./test/fixtures/basic, it's my understanding that the test script executes npm install before proceeding to package the app. In this scenario, the automatic inference of the electron version that I've written is bypassed because the packager is able to find a version of the nearest local installation of electron-prebuilt.
Could you point me in the right direction on how to implement a test?

You would create a new (barebones) test fixture, without electron-prebuilt in the dependencies or devDependencies of the fixture's package.json.


Unable to change the font properties

I was trying to change the font properties font size material check box but I could not, kindly post your replies.

Well I found a solution from the internet in VB language
Dim pfc As New PrivateFontCollection
pfc.AddFontFile(Application.StartupPath & "\fnts\Roboto-Light.ttf") 'Your font location, make sure the font is in "*.ttf" format
Label1.Font = New Font(pfc.Families(0), 16)


More than two conditions

hello, the structure to look for more than two conditions, I think it is not working.
{"logic":"and","filters":[{"field":"cuenta.banco","operator":"startswith","value":"banco bb"},{"logic":"or","filters":[{"field":"medioPago.descripcionMedioPago","operator":"startswith","value":"efe"},{"field":"sucursal.descripcion","operator":"startswith","value":"mbb"}]}]}
// service
return selectFrom(iEgresoDao).where(filter).findPage(pageable);
just take the first and last condition.
Thank you.

Sorry, I already solve it thanks :)


When displaying content of a binary CodebaseResource, say the strings are extracted

When displaying content of a binary CodebaseResource, say the strings are extracted from binary.
This could came in the form a simple message after the filename such as (text content isextracted from binary) or an icon with this message as a hover.

Resource flags (binary, archive, media) added in details view in https://github.com/nexB/scancode.io/commit/f44fd36091672e2f76ad71632d19d0851e82b087


InteractiveMarkers Crashing On Rolling

I ran into an unhandled std::runtime_error when using a rolling build of rviz2 on Windows.
I don't think Windows has anything to do with it, but thought it would worthwhile to mention my platform.
It seems that recently the way frame_locked_ was computed in interactive_markers.cpp has changed.
See fe52932 for reference
The exception is generated by the following change in that commit (and similar changes).
@@ -97,9 +97,9 @@ void InteractiveMarker::processMessage(
     orientation.w = 1.0;
   }

-  reference_time_ = message.header.stamp;
+  reference_time_ = rclcpp::Time(message.header.stamp, RCL_ROS_TIME);
   reference_frame_ = message.header.frame_id;
-  frame_locked_ = (message.header.stamp == rclcpp::Time());
+  frame_locked_ = (reference_time_ == rclcpp::Time());

   requestPoseUpdate(position, orientation);
   context_->queueRender();

I didn't follow up on what exactly frame_locked_ is used for, but the exception is thrown on the comparison
reference_time_ == rclcpp::Time()

The reason being reference_time_ is explicitly constructed with RCL_ROS_TIME for it's rcl_clock_type_t, but the default constructed rvalue rclcpp::Time() will have a rcl_clock_type_t being RCL_SYSTEM_TIME. This difference in clock types will cause an exception in rclcpp::Time::operator==.
Constructor for rclcpp::Time :
RCLCPP_PUBLIC
explicit Time(int64_t nanoseconds = 0, rcl_clock_type_t clock_type = RCL_SYSTEM_TIME);

Comparison operator for rclcpp::Time :
bool Time::operator==(const rclcpp::Time & rhs) const
{
  if (rcl_time_.clock_type != rhs.rcl_time_.clock_type) {
    throw std::runtime_error("can't compare times with different time sources");
  }

  return rcl_time_.nanoseconds == rhs.rcl_time_.nanoseconds;
}

See both
https://github.com/ros2/rclcpp/blob/rolling/rclcpp/include/rclcpp/time.hpp
and
https://github.com/ros2/rclcpp/blob/rolling/rclcpp/src/rclcpp/time.cpp
for reference.
This PR documents why the change was made
https://github.com/ros2/rclcpp/pull/2293
And this comment on the PR mentions not knowing if RCL_ROS_TIME or RCL_SYSTEM_TIME is the correct choice.
https://github.com/ros2/rclcpp/pull/2293#issuecomment-1800852519
To mitigate issues for myself I switched the explicit clock type from RCL_ROS_TIME to RCL_SYSTEM_TIME on my local, but am unsure if this is the appropriate fix.

Hi @AustinMooreT,
Thank you for explaining the issue, do you mind to open a PR ? We can discuss there which one is the best option.

Hi,
This was an oversight on my end, thanks for taking a look at it! For fixing it, I'm not sure changing all the clocks to system time is the correct way to do it. Since the issue is with the comparison to default time of a message time stamp I would use the comparison operator of the builtin_interfaces::msg instead of rclcpp::Time eg.:
frame_locked_ = (message.header.stamp == builtin_interfaces::msg::Time());

This should've been my default way of handling comparisons of message headers stamps with the empty timestamp.
Let me know if this sounds good. I will also see if I can find other instances of this typo.

Should this issue be marked as closed, or how does the process go after merge of a fix?

Yeah, it should be closed.  Thanks for the report and the fix.


fix: ensure rotoscope injected

RE #60.  Original PR did not properly ensure deps.step() would work when $rootScope was not manually injected.  Because $rootScope was not injected, tests that did not inject $rootscope would all timeout.
Since none of the existing specs tested without injecting $rootScope, the PR passed all tests.
Have added step-spec test to ensure deps.step() works properly when $rootScope not manually injected.

Cool, thank you very much for noticing the problem and adding a unit test for it


Tidy up the stream module

Remove unused direct dependency on mio.
Inline the Capture::next_noblock method into stream module.
Use newtype wrapper of Capture that implements AsRawFd with
pcap_get_selectable_fd(), instead of new data type SelectableFd that
carries around the fd;
...in order to use AsyncFdReadyMutGuard::try_io which abstracts away
the details previously explicit in Capture::next_noblock.
(Drop the PacketCodec requirement of being Unpin.)
Add PacketStream::inner_mut to make it possible to inject packets
in-between awaiting packets.

This is a breaking change since it removes stream::SelectableFd and
stream::PacketStream::new from the public API.
Closes #192

From a quick I like this PR. It does appear much cleaner. I would like to take a closer look at it first though.
However, for that can I ask you to do two things:

Since we don't have tests covering this feature, could you please add an example of the issue you were trying to solve in #192.
Can you make sure the CI passes - that way in case it takes me a bit too long to get around to reviewing I won't have to ping you for some minor fixes.

Hmm, I do not see how the recent Windows build failure would be caused by this PR.

@axelf4 thanks for the ping. The windows CI has been lately failing to download winpcap occasionally and needs to be given a kick to run again.
Looks like all tests pass.
Thanks for the PR and the example! I'll have a look at the PR with your example some time soon.

Thanks for the PR. Code is nice and your example seems to work for me! I'll merge straight since I took my time and I don't want to block again, but if you have time, I'd appreciate if you could add a CHANGELOG entry for this PR.


spread OSDs across zones uniformly

issue:

consider we have three nodes in a zone and one of the nodes (bigger) which is cordoned has 5 OSDs running, two other nodes (smaller) are not running OSDs
assume the region has three such zones in same config
now if we evict OSDs from the cordoned node and have tsc at hostname level to satisfy the constraint all OSDs should be running on one of smaller nodes which isn't possible due to less resources
due to this we can't ever evict pods from the bigger node if tsc takes into account of cordoned nodes as well

rc:

we don't have a way to take tainted nodes into consideration in tsc calculations until k8s 1.26 kep

fix:

set tsc at zone level which effectively counts number of OSDs running per zone even with cordon nodes
as a result we can have 5 OSDs running in a zone irrespective of bigger/smaller nodes

Signed-off-by: Leela Venkaiah G<EMAIL_ADDRESS>

ptal @rohan47, not able to find your name in reviewers for some reason 🤔


Request for release

Do you have any plans for the next release? We're currently using the latest ref for the module instead of a forge release because of the raised stdlib dependency.
Thanks!

done in #309


Implement ChEMBL target class in new ETL

As part of the work in refactoring the target ETL, we want to include the ChEMBL target class field inside the target object. This is an action to ensure backwards compatibility to the association faceting by target class.
Based on my understanding, the data is retrieved from files PIS retrieves from ChEMBL API:

https://www.ebi.ac.uk/chembl/api/data/protein_class.json
https://www.ebi.ac.uk/chembl/api/data/target_component.json
A first action, would be to identify if these can be retrieved directly from ChEMBL ES.

protein_class.json contains the different classes by protein_class_id
  "protein_classes": [
    {
      "l1": "Enzyme",
      "l2": null,
      "l3": null,
      "l4": null,
      "l5": null,
      "l6": null,
      "l7": null,
      "l8": null,
      "protein_class_id": 1
    },
    {
      "l1": "Adhesion",
      "l2": null,
      "l3": null,
      "l4": null,
      "l5": null,
      "l6": null,
      "l7": null,
      "l8": null,
      "protein_class_id": 2
    },

target_component.json contains the annotation of each target target component.
Both datasets can be mapped using accession and protein_classification_id == protein_class_id .
The current target object has the information in the next shape:
 |    |-- classes: array (nullable = true)
 |    |    |-- element: struct (containsNull = true)
 |    |    |    |-- l1: struct (nullable = true)
 |    |    |    |    |-- id: long (nullable = true)
 |    |    |    |    |-- label: string (nullable = true)
 |    |    |    |-- l2: struct (nullable = true)
 |    |    |    |    |-- id: long (nullable = true)
 |    |    |    |    |-- label: string (nullable = true)
 |    |    |    |-- l3: struct (nullable = true)
 |    |    |    |    |-- id: long (nullable = true)
 |    |    |    |    |-- label: string (nullable = true)
 |    |    |    |-- l4: struct (nullable = true)
 |    |    |    |    |-- id: long (nullable = true)
 |    |    |    |    |-- label: string (nullable = true)
 |    |    |    |-- l5: struct (nullable = true)
 |    |    |    |    |-- id: long (nullable = true)
 |    |    |    |    |-- label: string (nullable = true)
 |    |    |    |-- l6: struct (nullable = true)
 |    |    |    |    |-- id: long (nullable = true)
 |    |    |    |    |-- label: string (nullable = true)

Instead, we want a data model like the next one:
|-- targetClass: array (nullable = true)
|    |-- element: struct (containsNull = true)
|   |    |-- level: long (nullable = true)
|   |    |-- label: string (nullable = true)
|   |    |-- id: long (nullable = true)

I was able to process the PubChem export into Python's networkx / nxontology. Can read this in Python like:
from nxontology import NXOntology
url = "https://github.com/related-sciences/nxontology-data/raw/71cf538dc5c258ada880d58663b0205b7b7f8561/087_chembl_target_tree.json"
nxo = NXOntology.read_node_link_json(url)

Running nxo._get_name_to_node_info() logs the following warnings about nodes with duplicate names:
Node name duplicated for nodes 500 & 501: 'N-formyl methionyl peptide receptor'
Node name duplicated for nodes 535 & 536: 'Glycohormone receptor'
Node name duplicated for nodes 541 & 542: 'Protease-activated receptor'
Node name duplicated for nodes 583 & 584: 'Parathyroid hormone receptor'
Node name duplicated for nodes 585 & 586: 'Vasoactive intestinal peptide receptor'
Node name duplicated for nodes 587 & 588: 'Corticotropin releasing factor receptor'
Node name duplicated for nodes 592 & 593: 'Calcium sensing receptor'

Something perhaps to be aware of. Also would like to get in touch with ChEMBL to learn more, so please let me know if you're aware of the right contact there.

Sorry for the slow response @dhimmel. I shared your comments with ChEMBL. Hopefully they will get back to you soon

ChEMBL came back with some comments:

The ChEMBL protein family classification is a bespoke classification, reflecting some of the preferred or widely used community classifications for different protein families. It does not aim to be comprehensive, but is focussed around the main families that have been historically important for drug discovery. The classification is maintained manually, with new classes being added as/when needed (usually when a significant number of targets in a particular family appear in ChEMBL, or where they are of particular significance e.g., targets of approved drugs/candidates).
Kinases - Manning et al kinome paper
GPCRs - Guide to Pharmacology/IUPHAR, GPCRDB
Transporters - Mostly from TCDB
Epigenetic  - domain-based classification from SGC Chromohub (many epigenetic proteins have multiple domains, so multiple classifications)
Proteases - MEROPS
Enzymes – EC number
UniProt keywords may have been used to classify other groups e.g. secreted proteins.

They commented the classification could be better maintained and it might not reflect the most modern targets.
We will include this in the Platform documentation for future reference cc @HelenaCornu

Thanks @d0choa. ChEMBL's response is helpful to wrt which sources are used to construct the taxonomy. Perhaps I can follow up directly with ChEMBL with some of my more data related questions. Is there a GitHub repo / email address / other method of contact that is best?


Datapack Issue

Describe the Bug
I throw create into my mod pack and it stops loading because of some datapack issue. yeah idk
fabricmodloader 0.13.1
Reproduction Steps
https://www.technicpack.net/modpack/the-limit-reborn-1181.1905659
add create-mc1.18.1_v0.4d+394.jar
into the mods folder and it happens
Expected Result
To load into the world and test create on fabric
Screenshots and Videos
No response
Crash Report or Log
https://pastebin.com/BWkk65Qg
Operating System
windows 10
Mod Version
0.4.0c
Minecraft Version
1.18.1
Other Mods
No response
Additional Context
No response

its actually not 0.4.0c that was the latest version that I could select

does this still happen in the latest build? does it happen without create installed?

if it still occurs please reopen.


RIS live stream disconnects when mixing batch and real-time resources

When reading a mix of real-time and historical batch resources, the processing of batch resource can take a long time, causing ris live stream terminate the connection. Currently, libbgpstream does not handle disconnection well.

Related to issue #139 (ris error message handling). This issue is about handling the disconnections from ris-live stream source, which may or may not have error messages from ris. #139 is about handling error messages that can also be triggered by other types of errors.

RIS-live disconnection should be handled in the http transport (see #82).


{compute} az vm extension set: Add warning for AHBForRHEL/AHBForSLES Azure VM Extension installation process

Description

Warning for AHBForRHEL/AHBForSLES Azure VM Extension installation process https://github.com/Azure/azure-cli/issues/21649

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

compute


refactor: rename Variable to Identifier

coverage: 80.744%. remained the same when pulling 0ce16920351953098569f34b6092a57485be153f on rename-variable into 2ccb2fa879f174b8a55467a7624b90b829fe72eb on master.


Disable socket descriptor sharing with subprocs.

On Unix systems, sockets are like file descriptors, and file descriptors
are usually shared among child processes.
This means, that if we spawn a subprocess (or we fork) like we do in the
editor, open file descriptors will leak to the new process.
This causes issue with sockets as they might remain open and bound
(listening) when the original process closes.
Fixes #32533

Thanks!


GROOVY-7686: AIC: place local variable references above super ctor call

https://issues.apache.org/jira/browse/GROOVY-7686

Merged. Thanks!


Adding Selection Sort in JavaScript

Have you read the Contributing Guidelines on Pull Requests?
Yes
(Write your answer here.)
Description
Adding selectionSort.js to JavaSript/sort folder
(Write your answer here.)
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#2505
(Write your answer here.)

@HarshCasper Please review this PR


cmd/go: vcs warning leads to spurious vanity import violation

My copy of rsc.io/x86 contained an hgrc that pointed to a copy of codereview.py that has moved. hg commands on rsc.io/x86 thus yielded warnings. The go command interpreted the warning as part of the output, leading to a spurious import path violation:
$ go get -u rsc.io/pprof_mac_fix
package rsc.io/pprof_mac_fix
	imports rsc.io/x86/x86asm: /Users/jbleechersnyder/go/src/rsc.io/x86 is from *** failed to import extension codereview from /Users/jbleechersnyder/src/go/tip/lib/codereview/codereview.py: [Errno 2] No such file or directory: '/Users/jbleechersnyder/src/go/tip/lib/codereview/codereview.py'
https://code.google.com/p/rsc.x86, should be from https://code.google.com/p/rsc.x86

The fix is probably to only use stdout, not stdout+stderr. I have confirmed that the warning came on stderr and the useful output on stdout.

An even safer option might be to bail when stderr from the vcs command is non-empty.

The problem is in:
// run1 is the generalized implementation of run and runOutput.
func (v *vcsCmd) run1(dir string, cmdline string, keyval []string, verbose bool) ([]byte, error) {

... which merges stderr and stdout.

I'd like to fix this, but it's too late to be making fundamental changes like
separating stderr from stdout in these command invocations.
I just don't know what will break.
Possibly Brad's fix for #11381 will make this easier.

Missed this again.

I think unless there are more instances of this problem, we can just leave this alone. Git seems not to have this particular problem, and I can only assume that the hg codereview extension is dying out.
Reopen if you have a case that you believe is still compelling.
Thanks.


perlPackages.ListMoreUtilsXS.x86_64-darwin is broken

Relevant log:
Checking whether pureperl is required... no
Checking for cc... ld: unknown option: -mmacosx-version-min=10.10
Can't produce loadable XS module at inc/Config/AutoConf/LMU.pm line 73.

builder for '/nix/store/d7ls8w6maqiwa3rmjj93xix6wwfbipr6-perl-List-MoreUtils-XS-0.428.drv' failed with exit code 2

Don't know enough about Perl stuff to fix it.

Seems to be fixed.


Added functionality to pass a directory

I guess I never saw this. Looks interesting in hindsight. Thanks


Add Kinopoisk & Yandex Music icon

Description

Added icon for Kinopoisk (ru.kinopoisk), (ru.kinopoisk.tv).
https://play.google.com/store/apps/details?id=ru.kinopoisk
https://play.google.com/store/apps/details?id=ru.kinopoisk.tv
This is my second pull request, i really want to hear feedback about this :>

Type of change

:x: Bug fix (non-breaking change which fixes an issue)
✅ Icon addition (non-breaking change that adds/modifies Lawnicons's icons)
:x: General change (non-breaking change that doesn't fit the above categories like copyediting)

Icons addition information

Icons added

Kinopoisk (ru.kinopoisk)

Icons linked

Kinopoisk (linked ru.kinopoisk to @drawable/kinopoisk.xml)
Kinopoisk (linked ru.kinopoisk.tv to @drawable/kinopoisk.xml)

Closing as icons are in #945. Read the comment on #945 and don't include the same icons in new PRs for the future as there's no need


mac下编译失败

administratordeMacBook-Air:relay administrator$ make relay
/bin/sh build/prepare.sh
go build -ldflags -s -v -o build/bin/relay cmd/lrc/*
vendor/github.com/ethereum/go-ethereum/common/mclock/mclock.go:23:2: case-insensitive import collision: "github.com/loopring/relay/vendor/github.com/aristanetworks/goarista/monotime" and "github.com/Loopring/relay/vendor/github.com/aristanetworks/goarista/monotime"
vendor/github.com/ethereum/go-ethereum/accounts/accounts.go:23:2: case-insensitive import collision: "github.com/loopring/relay/vendor/github.com/ethereum/go-ethereum" and "github.com/Loopring/relay/vendor/github.com/ethereum/go-ethereum"
cmd/lrc/nodecmd.go:33:2: case-insensitive import collision: "github.com/loopring/relay/vendor/github.com/ethereum/go-ethereum/accounts" and "github.com/Loopring/relay/vendor/github.com/ethereum/go-ethereum/accounts"
cmd/lrc/accountcmd.go:29:2: case-insensitive import collision: "github.com/loopring/relay/vendor/github.com/ethereum/go-ethereum/accounts/keystore" and "github.com/Loopring/relay/vendor/github.com/ethereum/go-ethereum/accounts/keystore"
vendor/github.com/ethereum/go-ethereum/interfaces.go:25:2: case-insensitive import collision: "github.com/loopring/relay/vendor/github.com/ethereum/go-ethereum/common" and "github.com/Loopring/relay/vendor/github.com/ethereum/go-ethereum/common"
vendor/github.com/ethereum/go-ethereum/common/types.go:26:2: case-insensitive import collision: "github.com/loopring/relay/vendor/github.com/ethereum/go-ethereum/common/hexutil" and "github.com/Loopring/relay/vendor/github.com/ethereum/go-ethereum/common/hexutil"
vendor/github.com/ethereum/go-ethereum/crypto/crypto.go:32:2: case-insensitive import collision: "github.com/loopring/relay/vendor/github.com/ethereum/go-ethereum/common/math" and "github.com/Loopring/relay/vendor/github.com/ethereum/go-ethereum/common/math"
vendor/github.com/ethereum/go-ethereum/event/subscription.go:24:2: case-insensitive import collision: "github.com/loopring/relay/vendor/github.com/ethereum/go-ethereum/common/mclock" and "github.com/Loopring/relay/vendor/github.com/ethereum/go-ethereum/common/mclock"
vendor/github.com/ethereum/go-ethereum/interfaces.go:26:2: case-insensitive import collision: "github.com/loopring/relay/vendor/github.com/ethereum/go-ethereum/core/types" and "github.com/Loopring/relay/vendor/github.com/ethereum/go-ethereum/core/types"
vendor/github.com/ethereum/go-ethereum/core/types/bloom9.go:24:2: case-insensitive import collision: "github.com/loopring/relay/vendor/github.com/ethereum/go-ethereum/crypto" and "github.com/Loopring/relay/vendor/github.com/ethereum/go-ethereum/crypto"
vendor/github.com/ethereum/go-ethereum/accounts/keystore/keystore_passphrase.go:41:2: case-insensitive import collision: "github.com/loopring/relay/vendor/github.com/ethereum/go-ethereum/crypto/randentropy" and "github.com/Loopring/relay/vendor/github.com/ethereum/go-ethereum/crypto/randentropy"
vendor/github.com/ethereum/go-ethereum/crypto/signature_cgo.go:27:2: case-insensitive import collision: "github.com/loopring/relay/vendor/github.com/ethereum/go-ethereum/crypto/secp256k1" and "github.com/Loopring/relay/vendor/github.com/ethereum/go-ethereum/crypto/secp256k1"
vendor/github.com/ethereum/go-ethereum/common/types.go:27:2: case-insensitive import collision: "github.com/loopring/relay/vendor/github.com/ethereum/go-ethereum/crypto/sha3" and "github.com/Loopring/relay/vendor/github.com/ethereum/go-ethereum/crypto/sha3"
vendor/github.com/ethereum/go-ethereum/accounts/accounts.go:26:2: case-insensitive import collision: "github.com/loopring/relay/vendor/github.com/ethereum/go-ethereum/event" and "github.com/Loopring/relay/vendor/github.com/ethereum/go-ethereum/event"
vendor/github.com/ethereum/go-ethereum/trie/proof.go:25:2: case-insensitive import collision: "github.com/loopring/relay/vendor/github.com/ethereum/go-ethereum/log" and "github.com/Loopring/relay/vendor/github.com/ethereum/go-ethereum/log"
vendor/github.com/ethereum/go-ethereum/core/types/transaction_signing.go:27:2: case-insensitive import collision: "github.com/loopring/relay/vendor/github.com/ethereum/go-ethereum/params" and "github.com/Loopring/relay/vendor/github.com/ethereum/go-ethereum/params"
vendor/github.com/ethereum/go-ethereum/crypto/crypto.go:34:2: case-insensitive import collision: "github.com/loopring/relay/vendor/github.com/ethereum/go-ethereum/rlp" and "github.com/Loopring/relay/vendor/github.com/ethereum/go-ethereum/rlp"
vendor/github.com/ethereum/go-ethereum/core/types/derive_sha.go:24:2: case-insensitive import collision: "github.com/loopring/relay/vendor/github.com/ethereum/go-ethereum/trie" and "github.com/Loopring/relay/vendor/github.com/ethereum/go-ethereum/trie"
vendor/github.com/ethereum/go-ethereum/log/handler.go:11:2: case-insensitive import collision: "github.com/loopring/relay/vendor/github.com/go-stack/stack" and "github.com/Loopring/relay/vendor/github.com/go-stack/stack"
vendor/github.com/ethereum/go-ethereum/accounts/keystore/key.go:35:2: case-insensitive import collision: "github.com/loopring/relay/vendor/github.com/pborman/uuid" and "github.com/Loopring/relay/vendor/github.com/pborman/uuid"
vendor/github.com/ethereum/go-ethereum/trie/trie.go:27:2: case-insensitive import collision: "github.com/loopring/relay/vendor/github.com/rcrowley/go-metrics" and "github.com/Loopring/relay/vendor/github.com/rcrowley/go-metrics"
vendor/github.com/ethereum/go-ethereum/accounts/keystore/watch.go:25:2: case-insensitive import collision: "github.com/loopring/relay/vendor/github.com/rjeczalik/notify" and "github.com/Loopring/relay/vendor/github.com/rjeczalik/notify"
vendor/github.com/ethereum/go-ethereum/accounts/keystore/keystore_passphrase.go:43:2: case-insensitive import collision: "github.com/loopring/relay/vendor/golang.org/x/crypto/pbkdf2" and "github.com/Loopring/relay/vendor/golang.org/x/crypto/pbkdf2"
vendor/github.com/ethereum/go-ethereum/accounts/keystore/keystore_passphrase.go:44:2: case-insensitive import collision: "github.com/loopring/relay/vendor/golang.org/x/crypto/scrypt" and "github.com/Loopring/relay/vendor/golang.org/x/crypto/scrypt"
vendor/github.com/ethereum/go-ethereum/accounts/keystore/account_cache.go:34:2: case-insensitive import collision: "github.com/loopring/relay/vendor/gopkg.in/fatih/set.v0" and "github.com/Loopring/relay/vendor/gopkg.in/fatih/set.v0"
vendor/github.com/ethereum/go-ethereum/trie/sync.go:24:2: case-insensitive import collision: "github.com/loopring/relay/vendor/gopkg.in/karalabe/cookiejar.v2/collections/prque" and "github.com/Loopring/relay/vendor/gopkg.in/karalabe/cookiejar.v2/collections/prque"
cmd/lrc/accountcmd.go:33:2: case-insensitive import collision: "github.com/loopring/relay/vendor/gopkg.in/urfave/cli.v1" and "github.com/Loopring/relay/vendor/gopkg.in/urfave/cli.v1"
make: *** [relay] Error 1

👍


feat(impl):[TRI-1063] impl discovery service

Discovery service rest impl + mock impl.
Returning EDC base url or default address if not found

Merged by accident, wanted to merge my own PR, sorry!


Updated heroku.md

The changes in package.json and tsconfig.json neccesary for deploying on heroku were higlighted and comments were added to catch the eye of the reader on instance.
Though not a big update but i propose this change to ease the reader's eyes and give attention to the changes that they need to do instead of going deep into text and sometimes confusing.

@KnorpelSenf Please have a look on this PR , it's a small one and might be useful for the documentation

@roj1512 is maintaining the website, I'll leave the review to him


feat(design): create compactable interface

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/graycoreio/daffodil/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?
compact input is not normalized.
Part of: #763
What is the new behavior?
Create DaffCompactableinterface so thecompact` property will be consistent throughout @daffodil/design
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

@xelaint can you sign this commit?

@xelaint can you sign this commit?

@damienwebdev signed


MuscleTorqueWithVaryingBSpline

Hello PyElastica,
I really appreciate you guys for taking the time to resolve our queries. After taking the steps you mentioned in my issue titled " Generating Data From Trained Model ", I wanted to use MuscleTorqueWithVaryingBSpline and generate some data, and to do that I am using the code in the uploaded file.
Practice1.txt
But I run into the following error :
Traceback (most recent call last):   File "Practice1.py", line 199, in <module>     integrate(timestepper, ElasticaSim, final_time, total_steps)   File "C:\Users\reach\anaconda3\envs\MyEnv\lib\site-packages\elastica\_elastica_numba\_timestepper\__init__.py", line 79, in integrate     time = do_step(StatefulStepper, stages_and_updates, System, time, dt)   File "C:\Users\reach\anaconda3\envs\MyEnv\lib\site-packages\elastica\timestepper\symplectic_steppers.py", line 83, in do_step     SystemCollection.synchronize(time)   File "C:\Users\reach\anaconda3\envs\MyEnv\lib\site-packages\elastica\wrappers\base_system.py", line 158, in synchronize     feature(self, time)   File "C:\Users\reach\anaconda3\envs\MyEnv\lib\site-packages\elastica\wrappers\forcing.py", line 84, in __call__     ext_force_torque.apply_torques(self._systems[sys_id], time, *args, **kwargs)   File "C:\Users\reach\Desktop\Thesis\Elastica-RL-control-main\Case1\MuscleTorquesWithBspline\BsplineMuscleTorques\muscle_torques_with_bspline_numba.py", line 145, in apply_torques     self.max_rate_of_change_of_activation, ValueError: unable to broadcast argument 1 to output array File "C:\Users\reach\anaconda3\envs\MyEnv\lib\site-packages\numba\np\npyimpl.py", line 228,
Any advice into this will be of great help
Thank you.

Hi @ajaigit ,
You need to update the action arrays before running the simulation. For example, you define spline_points_func_array_normal_dir = [] during setup, so you are only passing an empty array of actions to MuscleTorqueWithVaryingBSpline which is why you are getting the error. Adding
spline_points_func_array_normal_dir[:] = np.zeros(spline_points_func_array_normal_dir)
spline_points_func_array_twist_dir[:] = np.zeros(spline_points_func_array_normal_dir)
spline_points_func_array_binormal_dir[:] = np.zeros(spline_points_func_array_normal_dir)

before you run the simulation will set all the actions to zero (or set the arrays to whatever you want).

Thank you for the response. I replaced the empty list with a random vector of size (6,1).  Unfortunately, that did not work out for me and the execution of the script ended up with an error. I am pretty sure that the error is probably because of my poor understanding about the working of Muscle Torques being used  so I still have a couple of questions about MuscleTorqueWithVaryingBSpline:

What purpose do spline_points_func_array_normal_dir and similarly spline_points_func_arrary in other direction ?
Does the RL algorithm randomly choose the magnitude of torque required or does it select from a predefined set of torques that it can choose from?
Is it possible to manually select the control points and then apply a torque of a given magnitude say 2 N-m and then observe the effect on the robot change in configuration?
Thank you.

The following script works for me. All I did was update the arrays with zeros. However, I noticed that you need to use pyelastica<=0.1.0.post5. The recent updates to v0.2.0 lead to breaking changes. They are minor so I will look into pushing a fix.
Practice1.txt
To answer your questions:

spline_points_func_array_normal_dir are where you put the appropriate actions for the torques in that direction. In 3D you  have a total action space of 2*num_control_points. You then need to place half the actions into 1 array and half into the other so pyelastica can use them to compute the torque on the arm. Pyelastica then computes a torque spline that is distributed along the arm and applies this torque to the arm.

This depends on what RL algorithm you use. In our paper, we used multiple continuous action algorithms from stable baselines so each action could be continuous between -1 and 1. However, Pyelastica doesn't really care if the value is continuous or not so if you implement a discrete action space method I don't foresee any issues.

Your Practice.txt script can do this very easily. In the updated script, the action arrays are all zeros so no torque is being applied. Just change one or more index to between [-1, 1]. The total torque applied is scaled by alpha, so if you want a max torque of 2 Nm, just set alpha=2.

@ajaigit we updated the main branch so you should now be able to use it with the latest version of pyelastica if that is useful. I am going to close this issue, but if you have any more questions please open a new issue and ask!
best,
Noel


Add Hoover, AL

Emails:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>note: The city of Hoover website has taken down many public emails, it seems. So these emails are mostly specific emails for members on the city board and financial/budgeting board members. No email exists for the mayor specifically.
A Message from a Concerned Citizen
Dear X,
My name is [YOUR NAME] and I am a resident of [NEIGHBORHOOD/BOROUGH/CITY]. I am emailing you today to demand that the Hoover Police Department (HPD) be defunded and that the budget and resources of the HPD be allocated to social service programs that are more effective at promoting community safety. Police presence has been proven time and time again to be ineffective at protecting the citizens of Hoover. Often, police presence results in the death of an innocent person of color (POC). I am outraged by the incompetence of Hoover city officials and their lack of insight to see the HPD’s use of excessive force against the people of Hoover, especially POC. Police have terrorized my community for too long, and I am asking that you put a stop to it. Defund the Hoover Police Department and allocate those funds to social service programs that better protect our community and are more effective at promoting community safety.
 I hope that you choose the side of the people.
Black Lives Matter,
[YOUR NAME]
[YOUR ADDRESS]
[YOUR EMAIL]
[YOUR PHONE NUMBER]

local specific info needed

@CWalker1901 Can you add some detailed information that's specific to Hoover? Is there a recent event that highlights the flaws of your police department? Or can you include budget numbers for the police department?
See our style guide for more ideas: https://github.com/defund12/defund12.org/blob/gh-pages/EMAIL_TEMPLATE_STYLE_GUIDE.md Thanks!

Hello, I found the police budget for Hoover, AL at $24 million which accounts for 31% of the city's total budget. Only $146,050 is spent on Heath & Welfare  and $165,000 for Medical & Nutrition Services
Source: http://hooveralabama.gov/143/Budgets
http://hooveralabama.gov/DocumentCenter/View/4596/FY-2020-Original-Budget?bidId=


Use the latest kubespawner release in hub image

Follow-up to https://github.com/2i2c-org/infrastructure/issues/1825
Also,  because this PR wasn't opened by dependabot on Monday, I believe it means dependabot doesn't work for git pip packages :/?
So maybe it's worth reopening https://github.com/2i2c-org/infrastructure/issues/1589?

dependabot doesn't work for git pip packages

I don't think it does and I believe that is out of scope for dependabot anyway. We should be using officially released versions anyway, so pinning to a specific commit (and the consequent revert back to a released version) should be a manual process, IMO. Because we are overriding the behaviour of how versioning works.

We should be using officially released versions anyway, so pinning to a specific commit (and the consequent revert back to a released version) should be a manual process, IMO. Because we are overriding the behaviour of how versioning works.

Agree!


How do I mutate an object inside of an array?

Is mutating an item inside of an array, like this todoStore.todos[index].setDone(true) supported?
export class TodoModel extends Model({
  todos: prop<Todo[]>(() => []).withSetter(),
})

I'm finding that this isn't consistently working, especially after first adding a new Todo to the model

This works, but the above indexing doesn't:
        done: true,
      }))```

todoStore.todos[index].setDone(true) should work

Aha - I wonder if it's related to this: https://github.com/xaviergonz/mobx-keystone/issues/173
My todo was created like
export class Todo extends Model({
  done: prop<bool>().withSetter(),
}) {
  static defaultTodo() {
    return new Todo({ done: false })
  }
}

Sounds like you found it :)

Yes, but unsure why a static factory method wouldn't work!
On Mon, 17 Jun 2024, 18:34 Javier Gonzalez, @.***>
wrote:

Sounds like you found it :)
—
Reply to this email directly, view it on GitHub
https://github.com/xaviergonz/mobx-keystone/issues/545#issuecomment-2173961494,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJWOW3EGF6KAM4SOISRURLZH4MYRAVCNFSM6AAAAABJOMQEFSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZTHE3DCNBZGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

It works if you use anything other than babel to transpile the legacy decorators (e.g. typescript compiler) and potentially it might work if you use the new decorators.

Hrmm, I'm not consciously using Babel - I've never even heard of it. Can
you point me at something to read up?
On Mon, 17 Jun 2024, 19:14 Javier Gonzalez, @.***>
wrote:

It works if you use anything other than babel to transpile the legacy
decorators (e.g. typescript compiler) and potentially it might work even
with babel if you use the new decorators.
—
Reply to this email directly, view it on GitHub
https://github.com/xaviergonz/mobx-keystone/issues/545#issuecomment-2174031977,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJWOW3FSNG6CZUUNGHSVDDZH4RRJAVCNFSM6AAAAABJOMQEFSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZUGAZTCOJXG4
.
You are receiving this because you authored the thread.Message ID:
@.***>

https://babeljs.io/


Device lab Flight School move (try pool)

Type of Request
infra task
Infrastructure Environment
devicelab
What is happening?
Move try pool bots to new Flight School lab:
Phase 1:

[ ] flutter-devicelab-linux-26
[ ] flutter-devicelab-linux-53
[ ] flutter-devicelab-mac-1
[ ] flutter-devicelab-mac-12
[ ] flutter-devicelab-mac-14

For each above machines:
Connect to network switch-1
Verify swarming bot is up and running
Verify that builds have run successfully after move
Create ticket to update hostadmin entry to point to new VLAN config (flightschool_1) (Similar to ticket t/225374063)
Verify DNS is correct and host is ssh'able
Verify salt states can be applied correctly
Phase 2:
Once phase 1 is completed successfully, move the remaining following machines:

[ ] flutter-devicelab-mac-17
[ ] flutter-devicelab-mac-19
[ ] flutter-devicelab-mac-38
[ ] flutter-devicelab-mac-45
[ ] flutter-devicelab-mac-46

For each above machines:
Connect to network switch-1
Verify swarming bot is up and running
Verify that builds have run successfully after move
Create ticket to update hostadmin entry to point to new VLAN config (flightschool_1) (Similar to ticket t/225374063)
Verify DNS is correct and host is ssh'able
Verify salt states can be applied correctly
Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

ticket for VLAN config for Phase 1 has been opened: t/225588176

Waiting for AC in the new lab going online and DNS entries update then will move Phase 1 Bots to new lab.

@kentnguyen99 DNS entry flutter-devicelab-linux-53.mtv.corp.google.com should be complete.
Note: it is expected the remote ssh to the updated machines will not work till they are physically moved to flight school.
Please perform the following tasks to see the effect of the change before they are physically moved:

[ ] reboot the machine
[ ] perform salt state apply
[ ] verify that the swarming server is still up and running

As expected, remote ssh to flutter-devicelab-linux-53.mtv.corp.google.com  is now down due to the DNS change.

All check and ready to go.

linux-53 has been moved to Flight School lab. Machine is up and running, I can login remotely from my end; However, it failed command "salt-call state.apply" due could not resolved DNS lookup.
flutter@flutter-devicelab-linux-53:~$ sudo salt-call state.apply
[sudo] password for flutter:
[ERROR   ] DNS lookup or connection check of 'salt-flutter.endpoints.fuchsia-infra.cloud.goog' failed.
[ERROR   ] Master hostname: 'salt-flutter.endpoints.fuchsia-infra.cloud.goog' not found or not responsive. Retrying in 30 seconds

Opened t/225700129 to track the DNS entry issues. Potentially missing DNS server entries in the new VLANs.

linux-53 is working, states applied fine, and build ran and succeeded.

lab temperature has been fixed and is cooling currently. Please resume with Phase 1 move of machines.

Phase 1 machines have been moved to Flight School, all machines online and ready for service.

opened t/225882455 for phase 2 dns changes.

Purchased some network cables, and phone cables (usb and lightning) for the new lab.

VLAN changes have been completed (t/225882455)

Phase 2 machines have been moved and the bots are up, but remote ssh is not working. Ticket for the VLAN changes has been re-opened to look at the issue.

hostadmin entries have been updated.


Update README

add "copy to clipboard icon" in installation section.
We cherrypicked PR #369  for this.

thanks @mateuslatrova !
Can I just be nitpicky about one little thing? sorry about that haha Could you reword the commit so that in its body it has reference to the original PR and a mention of the original author? apparently when you cherry picked the commit and pushed it you became the author.
My instructions were lacking since you did exactly as I requested, I will try to figure out how to preserve authorship for future PRs but for the time being could you just reword the commit? I would like for us to be transparent about it and avoid possible open source drama haha

implemented on PR #402


Youtube player parameters

Hi.
Can you add new editor option to set params string for all youtube embed videos. It will be useful to have ability add something like "?rel=0&modestbranding=1" to embed videos.

Hi, @realdigger
Thank you for your feedback  : )
It's Updated in v2.11.1

Thank you for SunEditor and it's features.


Missing activation events

Some commands do not activate the extension. Need to fix this.

No longer an issue in recent vscode versions


Feat/on chain json validation rebase/111

Goal
The goal of this PR is to add Json model validation on chain
Closes #111
Discussion
Null or empty Json schemas will be invalidated and not registered (even though serde-json does not consider them invalid)
Checklist

[ ] Chain spec updated
[ ] Updated the js/rpc code if a custom RPC added/changed
[ ] Design doc(s) updated
[x] Tests added
[ ] Benchmarks added
[ ] Weights updated

Comments from previous PR:
[x] Add negative json validation test
[x] Add utf 8 encoding test
[x] Clean up files changed
[x] Move serde to schema pallet since it needs to be no-std
Discussion:
Do I need to provide feedback to the user trying to register their schema as to why their schema failed to register?
The only error code that exists on the schema pallet currently is 'invalid schema'


Not able to create a query for a person name with '

Describe the bug
We are looking for searching an email/calendar detail of a person (Specifically a person whose name contains a single quote eg. O'Brien)
We use the MicrosoftGraph SDK on C# for Bot Framework. The query throws an error while we create it. Would you be so kind to assist us with this issue?
AB#8358

@aishwaryaumachandran I am not sure I understand, is this a query you are running on the Graph Explorer website?


Add LICENSE and license comments

@jeanconn - I'm planning to do this kind of change for all the Ska packages in skare/cfg/python_modules.cfg.  Add LICENSE.rst and put the license text in every .py file.  The text is always after existing comment lines starting with # in the file.  Do you see any problems?
tables3_api
ska_path
parse_cm
Chandra.Time
Chandra.ECF
Ska.arc5gl
Ska.astro
Ska.engarchive
Ska.File
Ska.ftp
Ska.Shell
Ska.Numpy
Ska.CIAO
Ska.DBI
Ska.Table
Ska.ParseCM
Ska.Matplotlib
Ska.tdb
Ska.TelemArchive
cosmocalc
Quaternion
Ska.quatutil
Ska.Sun
Chandra.Maneuver
Chandra.cmd_states
kadi
agasc
pyyaks
asciitable
pyger
Chandra.taco
Ska.report_ranges
xija
chandra_aca
chandra_models
find_attitude
mica
hopper
maude
cxotime
starcheck
testr

BTW the funniness in derived/pcad.py is because of a leading <U+FEFF> in the file that Amanda left there.

The only potential problem I note is conflict with pre-existing text.  I know we have a license in the top of Quaternion.py.

Good point, I will do some grepping to look for existing licenses.

I also note that Ska.Sun may need a BSD-2 to agree with IDL astro for the sun position stuff (right now there is just a URL mentioning the source).
https://idlastro.gsfc.nasa.gov/idlfaq.html


Blank page when creating a table with tuples as values

Hey, I'm using Nim 1.4.0 and Karax 1.2.1 and I have some very strange behavior, which I suspect to be a bug.
When I try the following code:
include karax / prelude

proc createDom(): VNode =
  buildHtml(tdiv):
    text "Hello World"

setRenderer createDom

I see the "Hello World" as expected (at least after modifying the html that is generated by karun so that it points to "app.js" instead of "/app.js" and opening the html file in the Browser. If I just use karun I'll only see a white page)
After then modifying the code to look like this:
include karax / prelude
import tables

let tab = toTable {"foo": (1, 2)}

proc createDom(): VNode =
  buildHtml(tdiv):
    text "Hello World"

setRenderer createDom

I'll only see a white page again. The problem seems to be a table where the key-type is a tuple of ints. If you create a table with string values everything continues to work. I compile via "nim js -o:app.js foo.nim" to prevent my modified html from getting overwritten

Can confirm.
Here is a sample browser err msg:
app.js:1065 Uncaught TypeError: src_1470024.slice is not a function
    at nimCopy (app.js:1065)
    at nimCopyAux (app.js:6041)
    at nimCopyAux (app.js:6046)
    at nimCopy (app.js:1040)
    at toTable_12811022 (app.js:6658)
    at app.js:6674
nimCopy @ app.js:1065
nimCopyAux @ app.js:6041
nimCopyAux @ app.js:6046
nimCopy @ app.js:1040
toTable_12811022 @ app.js:6658
(anonymous) @ app.js:6674

Can confirm.
Here is a sample browser err msg:
app.js:1065 Uncaught TypeError: src_1470024.slice is not a function
    at nimCopy (app.js:1065)
    at nimCopyAux (app.js:6041)
    at nimCopyAux (app.js:6046)
    at nimCopy (app.js:1040)
    at toTable_12811022 (app.js:6658)
    at app.js:6674
nimCopy @ app.js:1065
nimCopyAux @ app.js:6041
nimCopyAux @ app.js:6046
nimCopy @ app.js:1040
toTable_12811022 @ app.js:6658
(anonymous) @ app.js:6674

This isn't actually a Karax issue, but is instead a Nim JS issue. I tested with Nim 1.4.4 and was able to reproduce, however the error goes away on Nim. Closing for now, if there are further issues that you find, probably want to reopen at  https://github.com/nim-lang/Nim/issues if you can reproduce it without Karax
Version of the compiler that doesn't have the issue:
Nim Compiler Version 1.5.1 [Linux: amd64]
Compiled at 2021-04-14
Copyright (c) 2006-2021 by Andreas Rumpf

git hash: fe77c3c72fe1fde4f41286158ba612c02083dfe2
active boot switches: -d:release


Make links in schema text/content clickable

The links that appear in schema element descriptions, etc. will now be clickable.
Fixes #682

To see a preview of how this works, look at the below pages:

https://sujaypatil96.github.io/mixs/term_list/ (look at the description column)
https://sujaypatil96.github.io/mixs/0001215/
https://sujaypatil96.github.io/mixs/0001179/


chore(main): release queue 1.0.0

:robot: I have created a release beep boop
1.0.0 (2023-03-19)
Features

queue: add next() method to run next item in queue menually (a86bb58)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/remedyred/snickbit.js/releases/tag/queue-v1.0.0 :sunflower:


Killed before writing to markdown

I'm running pshtt against a file with ~13k domains. These are the final lines of the debug output:
Pinging https://www.zunheboto.nic.in...

Starting new HTTPS connection (1): www.zunheboto.nic.in
HTTPSConnectionPool(host='www.zunheboto.nic.in', port=443): Max retries exceeded with url: / (Caused by ConnectTimeoutError(<urllib3.connection.VerifiedHTTPSConnection object at 0x7f1c9bc4e940>, 'Connection to www.zunheboto.nic.in timed out. (connect timeout=10)'))

-------------------------

Printing Markdown...

Killed

The invocation was:
pshtt  domains.csv --markdown --debug --cache  --output output.md
The output.md file is empty. I've tried running on shorter domain lists and it works fine (No Killed in output, and the file is generated correctly).

You can download the raw CSV file from https://git.captnemo.in/nemo/pulse/ (domains.csv)

@captn3m0 This isn't quite enough information to debug the problem. The word "Killed" is never outputted by any of our code. One possibility is that your system may be running out of memory to process the entire list.
You could try out https://github.com/18F/domain-scan as an alternative parallelizer for pshtt, that makes use of Python 3 concurrency primitives to manage a rolling pool of threads over a given CSV. If you don't  use the --sort flag, I believe it never holds the entire dataset in memory, and should be time- and memory-efficient.
If you see the same problems, then I would also look at running the tool on a vanilla EC2 server or something like that, to eliminate the possibility of novel interference from something in your local environment.


Category chip

Description
Implements the CategoryChip component to render categories / category hierarchies. This also installs the sci-components package so that we can leverage the science design system and also evaluate it at the same time.
Demo
https://cat-chip-frontend.dev.imaging.cziscience.com/chip-test

@liaprins-czi thank you for the feedback! i'll start working on fixes. as for the tooltip, i forgot to mention that @kne42 is currently working on an implementation that we'll be able to use for both the preview page and categories, so i won't be working on styling at the moment (but i will look into the tooltip disappearance timing) 😄

@klai95 this implements the chip you'll be using for https://app.shortcut.com/imaging/story/172045/category-pills-for-search-plugin-pages

hey @liaprins-czi i've pushed my fixes! when you get a chance, could you take a look at it pls?
https://cat-chip-frontend.dev.imaging.cziscience.com/chip-test


Modify querymetrics to be exposed, not a private sealed class.  Maybe .GetQueryMetrics?

Is your feature request related to a problem? Please describe.
QueryMetrics provides very useful information, but here is how I have to get it using the newest cosmos SDK.
traces
| where message has "Retrieved Document Count"
| extend Query_Metrics_ = tostring(parse_json(tostring(parse_json(tostring(parse_json(tostring(parse_json(tostring(parse_json(tostring(parse_json(message).children))[1].children))[1].children))[0].children))[0].data)).["Query Metrics"])

Look at how many levels deep that is!!!
Describe the solution you'd like
I can already get Diagnostics like this:
var result = await this.container.UpsertItemAsync(entity, entity.ResourceIdentity.GetPartitionKey());
                    this.logger?.LogDebug("{operation} of record {entity} elapsed time {elapsed} query incurred charge of {resourceUnits} RUs",
                        "Update",
                        entity.Id,
                        result.Diagnostics?.GetClientElapsedTime(),
                        result.RequestCharge);

Simply expose QueryMetrics as well, via Diagnostics.GetQueryMetrics(), instead of having it be a very, very, very deeply nested property.
Describe alternatives you've considered
I began to just import the QueryMetrics and other types into my own project and write an extension method to do this, but the code style doesn't mesh well with our project and style-cop was causing issues.
I also tried to write an extension method to handle the conversion from Diagnostics.ToString(), but it looked really gnarly.
Additional context
Add any other context or screenshots about the feature request here.

@neilsh Sounds replicate of https://github.com/Azure/azure-cosmos-dotnet-v3/issues/3357

Very similar, it's still an issue in V3, and was like this as well in V2.


Actionbar not working with platform bukkit 1.20.5+

Simply run
plugin.adventure().player(player).sendActionBar(*insert component here*);

It does work everywhere except for Spigot & Paper 1.20.6 when using platform

Cannot reproduce on Spigot 4179-Spigot-342f493-bdb40c5 or Paper 1.20.6-130-master@5d8e53d. using implementation("net.kyori:adventure-platform-bukkit:4.3.3")

Ensure you're running on the latest build as it looks fine


Flutter Web: Incorrect Encoding of Asset URLs Containing Spaces and Special Characters"

Steps to reproduce
Add an image asset that contains spaces or special characters in its filename.
Example: Add an image named "Image with Space.jpeg" to your assets directory.
flutter:
  assets:
    - assets/images/Image with Space.jpeg

Use Image.asset() in your app to load the image.
Widget generateImage() {
  var imagePath = "assets/images/Image with Space.jpeg";
  return Image.asset(imagePath, fit: BoxFit.cover);
}

Expected results
The image "Image with Space.jpeg" should load successfully without any errors.
The network request for the image should show the URL as assets/images/Image%20with%20Space.jpeg.
Actual results
The image fails to load, and the console logs an error indicating a 404 HTTP status.
The network request for the image shows the URL as assets/images/Image%2520with%2520Space.jpeg, indicating that the space characters have been doubly encoded.
Error Message: Error while trying to load an asset: Flutter Web engine failed to fetch "assets/images/Image%2520with%2520Space.jpeg". HTTP request succeeded, but the server responded with HTTP status 404.

Code sample
Code sample
Widget generateImage() {
  var imagePath = "assets/images/Image with Space.jpeg";
  return Image.asset(imagePath, fit: BoxFit.cover);
}```

</details>

### Screenshots or Video

_No response_

### Logs

_No response_

### Flutter Doctor output

<details open><summary>Doctor output</summary>

```console
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 3.19.6, on Microsoft Windows [Version 10.0.22631.3447], locale de-DE)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[X] Android toolchain - develop for Android devices
    X Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[√] Chrome - develop for the web
[X] Visual Studio - develop Windows apps
    X Visual Studio not installed; this is necessary to develop Windows apps.
      Download at https://visualstudio.microsoft.com/downloads/.
      Please install the "Desktop development with C++" workload, including all of its default components
[!] Android Studio (not installed)
[√] VS Code, 64-bit edition (version 1.86.2)
[√] Connected device (3 available)
[√] Network resources

! Doctor found issues in 3 categories.```

</details>

Thanks for the report @hogdamian
I tried below code sample with which the image was loaded properly without the said exception you reported.
@override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        body: Center(
          child: Image.asset('assets/images/medical form.jpg', fit: BoxFit.cover)
        )
      ),
    );
  }

I am on mac machine and ran this code using latest master.


Add Profiles api and deprecate strategies/policy

This adds the profiles api as defined in the enhancement (https://github.com/openshift/enhancements/pull/548) and removes any functionality from the old tech preview API fields (which includes strategies, flags, and image. policy was briefly added but never included in a release, so that can just be removed)
Fixes #152

/hold cancel
enhancement has merged

/lgtm

/hold


PHPCS code formatting problems solved

I have the EditorConfig plugin in Sublime That's why the LSP always respects the 4 spaces and replaces all the tabs that are left there.

in the source code it does not automatically format unless you commit only the formatting...in this PR I did all the formatting by hand to reduce the size of the lines  since phpcs complains about lines longer than 120 characters

finally I applied phpcbf but it didn't do anything because almost everything was already formatted


Ensure that environment is available before checking for BCS

Fixes: https://github.com/godotengine/godot/issues/91239
I wrongfully assumed that doing a post process meant we had the environment available. But we also use the post pass for resolving MSAA, so this code can be reached even if there is no environment in the scene.
Simple fix is to check for the environment before enabling BCS.

Thanks!


Verbose pydocstyle

Codecov Report

Merging #64 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #64   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           9      9
  Lines         270    270
=====================================
  Hits          270    270

Coverage remained the same at 100.0% when pulling c012a495724bcaf790f882fb320091a57d60d048 on verbose-pydocstyle into 7028cac07596d0ed44176b85b43b73fa68866214 on master.

Coverage remained the same at 100.0% when pulling c012a495724bcaf790f882fb320091a57d60d048 on verbose-pydocstyle into 7028cac07596d0ed44176b85b43b73fa68866214 on master.

Coverage remained the same at 100.0% when pulling 87141376b940062623a464ff5174d583141378b3 on verbose-pydocstyle into 7028cac07596d0ed44176b85b43b73fa68866214 on master.


Type aware ed method

Added a type EdType to every Container, so that the ed method can return the appropriate type, rather than Container[_] (which the user would have to cast. However, this doesn't make the Stack.build hack any less hacky.

Yes, I approve of this change.
Why, thank you.


thanos: 0.35.1 -> 0.36.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for thanos is: Highly available Prometheus setup with long term storage capabilities
meta.homepage for thanos is: https://github.com/thanos-io/thanos
meta.changelog for thanos is: https://github.com/thanos-io/thanos/releases/tag/v0.36.1
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.36.1 with grep in /nix/store/zah974q0ivg81a3rlp6pmwbcxy2qfgif-thanos-0.36.1
found 0.36.1 in filename of file in /nix/store/zah974q0ivg81a3rlp6pmwbcxy2qfgif-thanos-0.36.1

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

thanos

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/zah974q0ivg81a3rlp6pmwbcxy2qfgif-thanos-0.36.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A thanos https://github.com/r-ryantm/nixpkgs/archive/e86a7c5986650d4ac621ebf2592ea85947b48c60.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/e86a7c5986650d4ac621ebf2592ea85947b48c60#thanos

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/zah974q0ivg81a3rlp6pmwbcxy2qfgif-thanos-0.36.1
ls -la /nix/store/zah974q0ivg81a3rlp6pmwbcxy2qfgif-thanos-0.36.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

thanos

Maintainer pings
cc @basvandijk @anthonyroussel for testing.

Add a :+1: reaction to pull requests you find important.

https://github.com/thanos-io/thanos/compare/v0.35.1...v0.36.1

Result of nixpkgs-review pr 335887 run on x86_64-linux 1

1 package built:

thanos


History per playlist

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

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

Checklist

[X] I'm requesting a feature unrelated to a specific site
[X] I've looked through the README
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
Hello,
Is it possible to add download archive per playlist?
Something like this:
#
# yt-dlp config
#

# download location
-o "downloads/%(playlist_title)s/%(id)s/%(title)s.%(ext)s"

# quality selection
-f "bv*[ext=mp4][vcodec^=avc1]+ba*[ext=m4a]"

# exec
--exec "./convert.sh 'downloads/%(playlist_title)s/%(id)s/'"

# enable metadata
--write-description --write-info-json

# download archive
--download-archive "downloads/%(playlist_title)s.lst"

# playlist links
## Playlist link 1
## Playlist link 2

Thanks in advance
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

Duplicate of #953


Request: Early storage in spawn without convenient swamp

I've got a solo world and the spiders have been good to me.  I'm carrying 20 spider string.  This is freaking me out that I might lose all this.  I've got extreme hills and jungle on my flanks.  No swamps.  No sugar cane.  I'd like some other way to create single item storage without having to worry about a swamp biome in the early days.

Fyi, jungle can still spawn sugar cane, although it's obviously harder to get.
But yes, this is already something I was interested in looking at, both to help early game storage, and also make the jump to chests less jarring.

Proposal:
1 wool knit + 1 stick = wool panel
4 wool panel = woolen basket
woolen basket = 4 woolen panel
8 woolen panel + plank = woolen hamper
So basically it's wicker with wool.

Not sure I would go into a jungle to look for something to hold the gear I might possibly lose going into a jungle.  Maybe a lucky gen could be at the edge.

We we're thinking bark baskets, for 1x1 storage, and move the current storage space up to 2x2 and 3x3, put some extra cost to regular chests, replace regular chests in the world with the new 3x3 hampers. pretty much what yany's thought of with his mod

To get regular chests is already 11 iron ingots.  Replacing the chests in the world with 3x3s sounds nice.  I find string too valuable early game to ever spend it on storage.  I guess in playthroughs where I find myself flush like the one that motivates this thread it would be nice.
I always thought the point of BTW automation was to get rid of what was an earlier grind.  Pushing chests further back just means I have to grind 3x3s and make a huge storage room.  For me a linear progression of storage from 1 to 4 to 9 to 20 or so is not that order of magnitude relief you feel moving to ovens or the saw (with chests as icing on the saw cake).  Will be interesting to see how it feels during play.


Add missing code coverage

Properly cover several code paths that were not tested.
Fix two bugs in exception-related code.

Closes #26.


Ship dotnet-aspnet-codegenerator once signing issue with CLI shim is resolved

This was pulled from preview2 because the executable generated on install is not signed.
Changes required:

Revert https://github.com/aspnet/Universe/pull/1022
We may need to bundle and sign our own executable shim in the dotnet-aspnet-codegenerator package. Still waiting on a design from @wli3 et. al.

FYI @muratg @prafullbhosale @Eilon - tentatively putting this issue in the rc1 milestone

Makes sense, thanks.

Blocked until the tool can ship its own shim, which we would be able to sign.

We have an initial design for this now. It looks like we are unblocked on this now and should probably assign someone to work on this soon.

Part of the details are still internal. I'll add you to the email thread.

Hi @seancpeters, I was told you're the new scaffolding owner!  Could you please take a look into this?
You'll need to send a shiproom approval this week. Next week will be escrow.
cc @vijayrkn

Reminder: ask mode ends today.

Fixed by: https://github.com/aspnet/Scaffolding/pull/7

@prafullbhosale wrong link?

Updated.

Can you send a PR to Universe then to revert https://github.com/aspnet/Universe/commit/fbdb3c558f04a056732a87a2e1864786603ac201 on the release/2.1 branch?

Done.


Thisyahlen/chore: bump up version and add othercfdaccount hook

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
21

Relevant Lines:
21

💛  - Coveralls


Option to force color output on/off

When running cargo fmt -- --write-mode diff | less -R to get an overview of changes the colorization of the diff is disabled (presumably through terminal/pipe detection). It would be useful to have a --color=always|never|auto flag to be able to force colored output on or off for situations like this, while retaining the output detection by default.

Hi, can I give this a crack? Thanks.

@clippered yes, please do! I'm not actually sure how to tackle this because I'm not familiar with terminal code. I would start by looking in https://github.com/rust-lang-nursery/rustfmt/blob/master/src/rustfmt_diff.rs

Thanks @nrc ! Same with me, but I'll try to understand the terminal code, and practice Rust as well. I'll let you know if I hit some bumps.


Installation of jhipster-uml

The current doc recommends global installation but I've had issues when using apps with different versions of JHipster (typically v3.x vs v2.x).
So my workflow is to install jhipster-uml locally with npm install jhipster-uml --save-dev and then use it from the project dir with $(npm bin)/jhipster-uml myJhipsterApp.jdl
Then it's a matter of finding the good jhipster-uml version corresponding to the generator-jhipster version of the app. It would be great to have a compatibility matrix for that !

Hi Christophe. Yes it would be. I'll try to get it done tomorrow or during the week-end.

A PR has been made, so I'm closing this. By the way, JUML doesn't deal with JDL files any more, it just exports them from any supported XMI file.


Background view not visible after subclassing FUIAuthPickerViewController

What
I subclassed FUIAuthPickerViewController in order to add a background image (UIImageView) behind the Google / FB sign-in buttons but I can't get it to display. This started happening after updating Firebase pods to the latest version (was working fine on the previous versions). See Details section for info.
Env

XCode 10.1
Swift 4.2
iOS 12.1.4
CocoaPods 1.6.0

Details
Issue description, code snippets, pod & sdk versions & screenshots on StackOverflow, since I initially didn't think it could be a bug.

If you print out the view hierarchy of the auth picker controller's view, what does it look like?

Ah, that sounds like it was introduced in https://github.com/firebase/FirebaseUI-iOS/pull/590. We can fix this by exposing the scroll view publicly so subclasses can add and remove subviews as they desire.

Sounds good! Will this come out in the next release?

If you are creating a subview and inserting it, you can also insert it at 1 instead of 0 which is a temporary fix until the scrollview is explicitly exposed.
 view.insertSubview(subview, at: 1) }

Tried that; I can see the background but not the sign in buttons. See my SO link for a screenshot.

Sorry, my use case was slightly different. I think if you do view.subviews[0].insertSubview(subview, at: 0), that should work. I also needed to set the background to clear as well for the UI View that houses the auth buttons.
 let selectorView = view.subviews[0].subviews[0]
 selectorView.backgroundColor = UIColor.clear

Before:

After:

Thank you, this actually works as a temp fix;
self.view.subviews[0].subviews[0].backgroundColor = UIColor.clear
self.view.subviews[0].insertSubview(imgView, at: 0)

The top area of the scrollbarview is still visible though.
Any thoughts? Initially I mistakenly thought it was the navigational status bar.

@polis80cy The new auth picker selector has a scrollview while previous version doesn't. The scrollview is added because some apps cannot display all their sign in options in a small screen (such as iPhone SE). We added the scroll view with backward compatibility in mind, and should be able to make most of developers' costumed views work. But in case a developer is doing something we didn't expect (such as utilizing some internal behaviors of our previous code), it may cause layout issues.
In your case, as far as I understand, you're trying to make a custom view with a background image. I believe you are fully able to do that with the new version. In our sample app, we have a FUICustomAuthPickerViewController (and you can test that by using the Demo app, and turn "Custom Authorization UI" switch on in the setting section). That view has a "knobColor" background.
To do add an image background manually, you can add the following code to the custom class:

@implementation FUICustomAuthPickerViewController {
  UIView * _backgroundView;
}

- (void)viewDidLoad {
  [super viewDidLoad];
  _backgroundView = [[UIView alloc] initWithFrame:CGRectZero];
  _backgroundView.layer.contents =  (id)[UIImage<EMAIL_ADDRESS>  [self.view addSubview:_backgroundView];
  [self.view sendSubviewToBack:_backgroundView];
}

- (void)viewDidLayoutSubviews {
  [super viewDidLayoutSubviews];
  _backgroundView.frame = self.view.frame;
}

I just tested and it correctly renders the image as the costumed background.

@Yue-Wang-Google I tried that and it still doesn't work I'm afraid.
This is on the latest version and all I'm doing is subclassing FUIAuthPickerViewController as per the readme file on GH. I'm not utilizing anything from previous codebase versions.
It's in Swift, no custom xib files.
Trying your suggestion:
class AuthViewController: FUIAuthPickerViewController {
    var backgView: UIView!

    override func viewDidLoad() {
        super.viewDidLoad()

        let backgroundImage = #imageLiteral(resourceName: "MainBackground")
        backgView = UIView(frame: CGRect.zero)
        backgView.layer.contents = backgroundImage.cgImage
        self.view.insertSubview(backgView, at: 0)
    }

    override func viewDidLayoutSubviews() {
        super.viewDidLayoutSubviews()

        backgView.frame = self.view.bounds
    }
}

And this is my delegate (which triggers the above class to fire) in AppDelegate.swift:
func authPickerViewController(forAuthUI authUI: FUIAuth) -> FUIAuthPickerViewController {
        return AuthViewController(authUI: authUI)
}

The result:

Trying the little hack as suggested by @cwoolum works but leaves that top grey scrollbar area visible as you can see from the screenshots in my previous comment. This is how:
class AuthViewController: FUIAuthPickerViewController {
    var imgView: UIImageView!

    override func viewDidLoad() {
        super.viewDidLoad()

        let backgroundImage = #imageLiteral(resourceName: "MainBackground")
        imgView = UIImageView(image: backgroundImage)
        imgView.contentMode = .scaleAspectFill

        self.view.subviews[0].subviews[0].backgroundColor = UIColor.clear
        self.view.subviews[0].insertSubview(imgView, at: 0)
    }

    override func viewDidLayoutSubviews() {
        super.viewDidLayoutSubviews()
        imgView.frame = self.view.bounds
    }
}

And this is the result (you can see the grey top part):

If there's a way to get rid of that grey top part (I've added some screenshots about that in my previous comment), I can definitely live with this. That's all I need at this point.

@polis80cy  please try the samples in /samples/objc/FirebaseUI-demo-objc/Samples/Auth/FUICustomAuthPickerViewController.{xib,h,m}. They should work well.

I tried the sample code with the latest FirebaseUI (6.2.1), without custom .xib file:
`@implementation FUICustomAuthPickerViewController {
UIView * _backgroundView;
}

(void)viewDidLoad {
[super viewDidLoad];
_backgroundView = [[UIView alloc] initWithFrame:CGRectZero];
_backgroundView.layer.contents = (id)[UIImage<EMAIL_ADDRESS>[self.view insertSubview:_backgroundView atIndex:0];
}

(void)viewDidLayoutSubviews {
[super viewDidLayoutSubviews];
_backgroundView.frame = self.view.bounds;
}`

And still does not work. The _backgroundView is behind the _scrollView.

@zhanetpaneva please try it with the nib. it should be working. you can remove the top close button and label in the nib.

@polis80cy try this workaround:
override func viewDidLoad() {
        super.viewDidLoad()

        let scrollView = view.subviews[0]
        scrollView.backgroundColor = .clear
        let contentView = scrollView.subviews[0]
        contentView.backgroundColor = .clear

        let background = UIImage(named: "image")
        backgroundImageView = UIImageView(image: background)
        backgroundImageView.contentMode = .scaleToFill
        view.insertSubview(backgroundImageView, at: 0)
}

And set backgroundImageView's constraints to the superview's edges. It worked in my case.

@zhanetpaneva this works beautifully, thank you!

@polis80cy try this workaround:
override func viewDidLoad() {
        super.viewDidLoad()

        let scrollView = view.subviews[0]
        scrollView.backgroundColor = .clear
        let contentView = scrollView.subviews[0]
        contentView.backgroundColor = .clear

        let background = UIImage(named: "image")
        backgroundImageView = UIImageView(image: background)
        backgroundImageView.contentMode = .scaleToFill
        view.insertSubview(backgroundImageView, at: 0)
}

And set backgroundImageView's constraints to the superview's edges. It worked in my case.

this caused error for me when cancling on email authui screen

Does anyone know how to make it full screen?
Even If I try to make the Nav Transparent the image doesn't cover the UIScreen.main.bounds.height of the screen, making it cut at the navBar.
I got this:
at LoginVC
/// Delegate function that allows us to create a custom FUIAuthPickerViewController
///
/// - Parameter authUI: FUIAuth instance
/// - Returns: The custom FUIAuthPickerViewController
func authPickerViewController(forAuthUI authUI: FUIAuth) -> FUIAuthPickerViewController {
// Create an instance of the FirebaseAuth login view controller
let customLoginViewController = FUIAuthPickerViewController(authUI: authUI)
    customLoginViewController.navigationItem.leftBarButtonItem = nil

    // Use the previous values to build a CGRect for the drawable's view's frame
    let drawableFrame = CGRect(x: self.view.frame.origin.x, y: self.view.frame.origin.y, width: self.view.frame.width, height: UIScreen.main.bounds.height)

    let drawableImageView = UIImageView(frame: drawableFrame)
    drawableImageView.image = UIImage(named: "Logging")
    drawableImageView.contentMode = .scaleAspectFill

    customLoginViewController.view.subviews[0].subviews[0].backgroundColor = UIColor.clear
    customLoginViewController.view.subviews[0].insertSubview(drawableImageView, at: 0)

    return customLoginViewController
}

This at AppDelegate:
didFinishLaunchingWithOptions.... {
window?.rootViewController = LoginViewController()
    // Remove the bottom line for the UINavigationBar
    UINavigationBar.appearance().shadowImage = UIImage()
    UINavigationBar.appearance().setBackgroundImage(UIImage(), for: .default)

    //Clear NavBar for a fullscreen ImageView
    UINavigationBar.appearance().isTranslucent = true

}

How I disable the scrolling or the bounce? how can I remove the navBar title?
I have tried this code
/// Delegate function that allows us to create a custom FUIAuthPickerViewController
///
/// - Parameter authUI: FUIAuth instance
/// - Returns: The custom FUIAuthPickerViewController
    func authPickerViewController(forAuthUI authUI: FUIAuth) -> FUIAuthPickerViewController {
        // Create an instance of the FirebaseAuth login view controller
        let customLoginVC = FUIAuthPickerViewController(authUI: authUI)

        customLoginVC.navigationItem.leftBarButtonItem = nil

        customLoginVC.navigationController?.isToolbarHidden = true
        customLoginVC.navigationController?.isNavigationBarHidden = true

        //Original Frame
        let drawableFrame = CGRect(x: 0, y: -44, width: UIScreen.main.bounds.width, height: UIScreen.main.bounds.height)

        let drawableImageView = UIImageView(frame: drawableFrame)
        drawableImageView.image = UIImage(named: "pdf2")
        drawableImageView.contentMode = .scaleAspectFill

        //vc
        customLoginVC.view.subviews[0].subviews[0].backgroundColor = UIColor.clear
        customLoginVC.view.subviews[0].insertSubview(drawableImageView, at: 0)

        customLoginVC.view.backgroundColor = .black

        let statusBar = UIApplication.shared.value(forKeyPath: "statusBarWindow.statusBar") as? UIView
        statusBar?.backgroundColor = UIColor.clear

        return customLoginVC
    }

Unfortunately, toolBar is not hidden, navBar is not hidding or setting it's text even if I set another title e.g. navBar.title = "hi" or even nil. background also doesn't change and it is white.

The unique thing that I'm able to modify is the back button which I'm already setting in up as nil and it works but the remaning code it is just like ignoring it... any thoughts? I'm all ears.

https://www.youtube.com/watch?v=d_byFfP3cTw&feature=youtu.be solved


Version (make sure you are on the latest version before reporting):

Minecraft:1.10.2
Forge: twitch...
Refined Storage: 1.2.26

Does this issue occur on a server? [yes/no] Yes
If a (crash)log is relevant for this issue, link it here: no crashes
[pastebin/gist/etc link here]

what is "invalid"


Use psr/container

This replaces the now-deprecated container-interop/container-interop with psr/container in what-should-be-a BC way.
(The exceptions are now a different type, is that acceptable?)

see #999 and more specifically @stof's comment

Here is what we need to end up with:

we need to require interop 1.2+ rather than 1.1+ (to have the version using PSR)
any place consuming a container should accept the PSR interface
any place defining container-related things (the container itself and the exceptions) should use the interop class (which extends PSR) to keep backward compatibility

Postponed till FIG actually accepts PSR-11. Currently it is in review:
http://www.php-fig.org/psr/

The website doesn't seem to be up to date, see https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-11-container.md
It is categorized as "accepted"


Make it a package

You actually created a plugin instead of a plugin. Packages can be used at any platform, but plugins can only be used on the platforms specified in pubspec.yaml.
Here, it should show web, windows, macos and linux as supported platforms, but it's only showing Android and iOS

Thanks! I just fixed it, now it's possible to use in any platform.

Nice. The android and iOS folder can be deleted


Portrait image from camera is not showing with correct aspect ratio

I'm getting this output when I take a picture using the camera on portrait:

And I swear that my head is taller than that :-)
Here's the image I got on self.image at this point at JSQPhotoMediaItem
- (UIView *)mediaView
{
    if (self.image == nil) {
        return nil;
    }

    if (self.cachedImageView == nil) {
        CGSize size = [self mediaViewDisplaySize];
        UIImageView *imageView = [[UIImageView alloc] initWithImage:self.image];
        imageView.frame = CGRectMake(0.0f, 0.0f, size.width, size.height);
        imageView.contentMode = UIViewContentModeScaleAspectFill;
        imageView.clipsToBounds = YES;
        [JSQMessagesMediaViewBubbleImageMasker applyBubbleImageMaskToMediaView:imageView isOutgoing:self.appliesMediaViewMaskAsOutgoing];
        self.cachedImageView = imageView;
    }

    return self.cachedImageView;
}

PS: Sorry for the grumpy face LOL

thanks @betzerra !
closing as a duplicate of #740


Export toQuery function

It would be nice if toQuery was also exported in index.js. There have been times that the only change a component has is a simple css property, to avoid multiple renderings it would be useful to just be able to change that property with a css media query.
Here is the ideal usage:
import { toQuery } from 'react-responsive';
export const MediumQuery = toQuery({ minWidth: 640 });

Then in your styled components you could just import MediumQuery and use like this:
@media ${MediumQuery} {
    ....
}

Interesting idea - want to send a PR that adds a few tests and exposes it?
Workaround is to import react-responsive/dist/lib/toQuery


Add locations for attribute names and values

This allows consumers of the HTML language service API to more easily manipulate attributes in documents without doing any extra parsing work. Locations are a separate record from attribute values to preserve the existing API surface.

@microsoft-github-policy-service agree

HTML Documents can be large and contain a lot of nodes. In order to keep the memory footprint minimal, the attribute locations as well as other ranges such as the tag name range are not kept. Instead, consumers that need them can easily get them by using the scanner.
If you want, you can add a function to the node that does that.