Ugly loop in detailed routing

While the design passes detailed routing fine, magic later finds an issue with one of the metal1 holes:
[Removed: see https://github.com/The-OpenROAD-Project/OpenROAD/issues/1690#issuecomment-1067179723 ]
See https://github.com/RTimothyEdwards/open_pdks/issues/236 for Tim's image
This is a blocker for mpw-5.

Relevant to another issue: https://github.com/The-OpenROAD-Project/OpenLane/issues/954
Metal3 minimum area < 0.24um^2 (met3.6)

Where in the layout is this hole to be found?

The linked issue says
The attached screenshot was taken at position (181.675um, 880.160um), (181.860um, 880.315um)
which makes no sense as the design isn't that large:
UNITS DISTANCE MICRONS 1000 ;
DIEAREA ( 0 0 ) (<PHONE_NUMBER>45 ) ;

That location is outside the DIEAREA.

@RTimothyEdwards would you respond to the above about error location

That's on me, I uploaded a different layout with the same issue. @maliberty @RTimothyEdwards
I'll upload a new one in a bit.

@donn whichever is smaller is better :)

I can reproduce it now and will take a look.

There is no MINENCLOSEDAREA rule in the LEF for met1 so there is no reason for the router to think this is illegal.  Why does magic think so?  @RTimothyEdwards @donn

@Stephanommg this is some ugly routing (even if not illegal per LEF):

Would you take a look when you have a chance.  It isn't urgent.  Be sure to use the reproducible.tar.gz above.

@maliberty :  (1) There is a MINENCLOSEDAREA rule in the LEF file for metal1 if you are using the technology LEF file from open_pdks.  (2) Regardless of whether or not the LEF file declares MINENCLOSEDAREA, it's still a SkyWater DRC rule, and magic flags it.

@Donn:  (@maliberty ) :  The problem is that with the change to min/max/nom versions of the technology LEF file, the MINENCLOSEDAREA patch to the technology LEF file somehow vanished.  The issue is in open_pdks somewhere and I am investigating.  Please feel free to close this issue, as it is not an OpenROAD problem.

@donn :  I'm still not sure why the patch failed, but I'm testing a fix now.

I'll leave this open for the routing quality concern but will not worry about the holes issue.

@maliberty I am also facing this issue with latest openlane tool set with PDK=sky130B and same thing passes with PDK=sky130A
My Repo: https://github.com/dineshannayya/riscduino_qcore
Failing Macro : make ycr4_iconnect
Failing Signature:
Min area of metal1 holes > 0.14um^2 (met1.7)
175.695um 265.440um 175.880um 265.595um
142.115um 221.920um 142.300um 222.075um
183.975um 205.600um 184.160um 205.755um
91.055um 33.605um 91.240um 33.760um
219.395um 1081.440um 219.580um 1081.595um
193.635um 1418.720um 193.820um 1418.875um
207.435um 1364.320um 207.620um 1364.475um
211.575um 1339.205um 211.760um 1339.360um
225.835um 1266.400um 226.020um 1266.555um
207.895um 1817.925um 208.080um 1818.080um
203.755um 1766.880um 203.940um 1767.035um
[INFO]: COUNT: 11
[INFO]: Should be divided by 3 or 4
Openlane Tool Version:
Kernel: Linux v5.13.0-52-generic
Distribution: ubuntu 20.04
Python: v3.8.10 (OK)
Container Engine: docker v20.10.17 (OK)
OpenLane Git Version: f9b5781f5ef0bbdf39ab1c2bbd78be8db11b27f2
pip: INSTALLED
pip:venv: INSTALLED
---
PDK Version Verification Status: OK
---
Git Log (Last 3 Commits)

f9b5781 2022-07-01T16:04:31+02:00 Fix a bug with `-overwrite` (#1171) - Anton Blanchard -  (HEAD -> master, origin/master, origin/HEAD)
abb9d59 2022-07-01T13:46:02+02:00 Reimplement support for multiple power domains (#1175) - Marwan Abbas -  ()
83b6145 2022-06-26T18:19:11+02:00 Fix #1163 (#1164) - Mohamed Gaber -  ()

@maliberty this block is same one which you have suggested fix in during below issue analysis
https://github.com/The-OpenROAD-Project/OpenLane/issues/1106

@RTimothyEdwards last comment "I'm` still not sure why the patch failed, but I'm testing a fix now." - was this actually fixed?
I believe Stephano did fix a loop case in drt as well.
In any case you need to package a proper routing test case and not just point at a repo.

@maliberty As this does not fails at routing phase, but fails Magic DRC check phase, i need to see how to extract both stage.

@donn was going to deal with this but the standard workaround is to put a call to "exit -1" in the relevant script to force a failure.
Based on the slack discussion this is an open_pdk issue not an OR one.

@dineshannayya are you sure this is the same issue, and not: https://github.com/RTimothyEdwards/open_pdks/pull/262 ?

@maliberty Recent update: https://github.com/The-OpenROAD-Project/OpenLane/blob/master/docs/source/using_or_issue.md

@antonblanchard based on a slack discussion this is an open_pdk issue and not an OR one.  I don't see any point in keeping this open anymore.


有人利用该开源软件用于商业行为进行收费

Download url: https://www.apowersoft.cn/gif

https://www.apowersoft.cn/store/gif.html?apptype=ads-bd&appvisitor=71103fa172c662ba369fb93f25946c53

Yes, it's a copy of my work. It's know, thank you.

Ok,you're welcome~


Bluetooth: Host: hci.h header include depends on conn.h

Describe the bug
Samples, application and controller implementation needing hci.h header include has dependency on conn.h which makes host connection data types be included in source code compilations.
Dependency introduced in 0f06c7d
Refer to https://github.com/zephyrproject-rtos/zephyr/pull/58154#discussion_r1203243070
To Reproduce
NA
Expected behavior
hci.h should not include conn.h .
Impact
annoyance. stable API change?
Logs and console output
NA
Environment (please complete the following information):

OS: Linux
Toolchain: Zephyr SDK,
Commit SHA or Version used: 5181b53

Additional context
NA

The fix for this issue is that hci.h only has typedefs and struct definitions. hci.h should be agnostic about the implementation
Create a new header file (for ex. hci_host.h) that contains the function prototypes


rr: encode gdb dep with wrapProgram instead of propagatedBuildInputs

Description of changes
Except in special wrapper environments like python3.withPackages, where all build dependencies are also runtime dependencies, propagatedBuildInputs doesn't actually encode runtime dependencies.
It's meant to propagate build dependencies to dependants, so the dependants don't have to explicitly add them to their buildInputs.
With wrapProgram, you can now run rr without having gdb installed (eg. nix run nixpkgs#rr replay).
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Oh wait lol - this was already discussed in e1456b291c53ab34f475c36092df07d6ddcb60ab and I forgot I referenced it like 4 years ago 😅


[Airbrake] [Production] undefined method `to_date' for nil:NilClass

Airbrake error: #1218
Airbrake project: zeitkit
Error type: ActionView::Template::Error
Error message: undefined method `to_date' for nil:NilClass
Where: teams#show
Occurred at: May 06, 2019 08:30:44 UTC
First seen at: May 06, 2019 08:30:44 UTC
Occurrences: 0 (0 since last deploy on <no information>)
Severity: error
URL: https://www.zeitkit.com/teams/1?utf8=%E2%9C%93&team_aggregator%5Bstart_date%5D=01%2F02%2F2019&team_aggregator%5Bend_date%5D=31%2F02%2F2019&team_aggregator%5Bspecific_range%5D=&commit=Submit
File: /PROJECT_ROOT/app/services/team_aggregator.rb
Backtrace:
/PROJECT_ROOT/app/services/team_aggregator.rb:89:in worklogs
/PROJECT_ROOT/app/services/team_aggregator.rb:77:in generate_result_data
/PROJECT_ROOT/app/services/team_aggregator.rb:67:in block in results
[GEM_ROOT]/gems/activerecord-4.2.10/lib/active_record/relation/delegation.rb:46:in map
...
Caused by NoMethodError: undefined method `to_date' for nil:NilClass

I forced this error manually.


{{#x}}{{#x}}{{/x}}{{/x}} breaks

You cannot work with objects such as the following due to pystache's reliance on regular expressions instead of a grammar:
{documents: [{name:1, children:[{name: 2, children:[...]}]}]}
Here's a test case that causes pystache to raise an exception:
http://pastebin.com/4eAYfyKD

This is now fixed in the development branch.  Thanks for the report!


Do not use Lenient SSL if custom cert is specified

Following this problem : https://stackoverflow.com/q/55354081/10791639 , a solution already discussed in https://github.com/intuit/karate/issues/385 was found.
It should be possible to either use SSLConnectionSocketFactory or LenientSslConnectionSocketFactory depending on the situation.

Build is OK, and it works in our projects.
Is it possible in some case to have keyStore != null and at the same time the need to use LenientSslConnectionSocketFactory?

@MercierAdrien I don't remember the details - but if SslTest is working now, we can get rid of LenientSslConnectionSocketFactory. Are you saying all tests in all maven modules pass ? You can submit a PR

Tests are passing for me. I didn't remove LenientSslConnectionSocketFactory, but it is only used when keyStore is null.

released 0.9.3


Build Error from the main core file.

Description
when I run build command for this error comes.
I have searched but nothing found to solve.
$ pnpm run build

<EMAIL_ADDRESS>build F:\working project\target-line
next build

▲ Next.js 14.2.5

Environments: .env

Creating an optimized production build ...
✓ Compiled successfully
Linting and checking validity of types  ...Failed to compile.
./node_modules/.pnpm/next-intl@3.17.2_next@14.2.5_@babel+core@7.24.9_react-dom@18.3.1_react@18.3.1__react@18.3.1__react@18.3.1/node_modules/next-intl/dist/types/src/navigation/react-client/createSharedPathnamesNavigation.d.ts:4:57
Type error: Type parameter declaration expected.
2 | import { Locales } from '../../routing/types';
3 | import { SharedNavigationRoutingConfigInput } from '../shared/config';

4 | export default function createSharedPathnamesNavigation(input?: SharedNavigationRoutingConfigInput): {
|                                                         ^
5 |     Link: (props: Omit<Omit<Omit<Omit<Omit<React.AnchorHTMLAttributes, keyof {
6 |         href: string | import("url").UrlObject;
7 |         as?: string | import("url").UrlObject;
 ELIFECYCLE  Command failed with exit code 1.

Verifications

[X] I've verified that the problem I'm experiencing isn't covered in the docs.
[X] I've searched for similar, existing issues on GitHub and Stack Overflow.
[X] I've compared my app to a working example to look for differences.

Mandatory reproduction URL
App Router
Reproduction description
Steps to reproduce:

Open reproduction
Click on …
See error: …

Expected behaviour
I have checked all the ts ignores file and eslint but non of them worked for me.

This error is because of Ts version if you want to use next-intl with next 14 or higher your ts should be higher than 5, it will fix that.


Delimiter selector updated

Delimiter selector made more robust and usable for all languages
Added a lot of language specific play/test files. They have very little in them right now but the purpose is that each language should have a file with all the common structures making it easier to test while developing.

Now I think everything is fixed :)


Full Content Loading

add the prop to set all classes for the full content loading

Proposed Changes
Add a prop to set all classes for the full content loading
Potential Breaking Changes
nothing
Acceptance Criteria

[ ] The proposed changes are covered by unit tests
[ ] The potential breaking changes are clearly identified
[ ] README.md is adjusted to reflect the proposed changes (if relevant)

Codecov Report

Merging #1258 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1258      +/-   ##
==========================================
+ Coverage   99.49%   99.49%   +<.01%
==========================================
  Files         356      356
  Lines        8968     8969       +1
  Branches     2126     2126
==========================================
+ Hits         8923     8924       +1
  Misses         45       45

Impacted Files
Coverage Δ

...ges/react-vapor/src/components/loading/Loading.tsx
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


tab is a confusing word

Description
On this page there is this block of text:
https://github.com/ros-planning/moveit2_tutorials/blob/foxy/doc/quickstart_in_rviz/quickstart_in_rviz_tutorial.rst
The planning scene robot using the Show Robot Visual checkbox in the Scene Robot tab.
The planned path using the Show Robot Visual checkbox in the Planned Path tab.
The start state using the Query Start State checkbox in the Planning Request tab.
The goal state using the Query Goal State checkbox in the Planning Request tab.

The word tab seems wrong as these are not tabs (and there are other things that are tabs in the UI).  We need a better word.

I believe those would be tree menus or tree views.

Thank you @stephanie-eng for fixing this.

np


Apollo Unable to generate classes for operations with Default Arguments on v3.2.0

Summary
Hi again! I'm running into an issue where Apollo is unable to generate classes when there are default arguments in my Query/Mutation. It specifically throws this error:
Caused by: java.lang.IllegalArgumentException: couldn't make a guess for [com.apollographql.relocated.cl](http://com.apollographql.relocated.cl/)

Version
3.2.0, things were working as expected on 3.1.0
Description
These queries don't work
Mandatory Arguments, no defaults
mutation MyMutation($firstName: CustomScalar!, $lastName: CustomScalar!) {
  my_put_operation(firstName: $firstName, lastName: $lastName)
}

Hardcoded default at field level w/ mandatory arguments
mutation MyMutation($firstName: CustomScalar!, $lastName: CustomScalar!) {
  my_put_operation(firstName: "MyDefault123" lastName: $lastName)
}

The following don't work
Default for non-mandatory argument for mutation
mutation MyMutation($firstName: CustomScalar = "MyDefault123", $lastName: CustomScalar!) {
  my_put_operation(firstName: $firstName, lastName: $lastName)
}

Non-Mandatory argument, default value hardcoded in field
mutation MyMutation($firstName: CustomScalar , $lastName: CustomScalar!) {
  my_put_operation(firstName: "MyDefault123" lastName: $lastName)
}

Here's the full stack trace for more context:
FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':subsystem:myFeature:generateServiceApolloSources'.
> couldn't make a guess for com.apollographql.relocated.cl

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 34s
53 actionable tasks: 3 executed, 50 up-to-date

Publishing build scan...

ERRORS (1)

FAILED TASK: :subsystem:myFeature:generateServiceApolloSources
  Description: org.gradle.api.tasks.TaskExecutionException: Execution failed for task ':subsystem:myFeature'.

    Caused by: java.lang.IllegalArgumentException: couldn't make a guess for com.apollographql.relocated.cl
      // ...
      at com.apollographql.relocated.d4.a(SourceFile:7)
      at com.apollographql.relocated.g4.a(SourceFile:217)
      at com.apollographql.apollo3.compiler.codegen.kotlin.helpers.DataClassKt.makeDataClass(SourceFile:13)
      at com.apollographql.apollo3.compiler.codegen.kotlin.file.OperationBuilder.typeSpec(SourceFile:844)
      at com.apollographql.apollo3.compiler.codegen.kotlin.file.OperationBuilder.build(SourceFile:3)
      at com.apollographql.apollo3.compiler.codegen.kotlin.KotlinCodeGen.write(SourceFile:206)
      at com.apollographql.apollo3.compiler.ApolloCompiler.write(SourceFile:790)
      at com.apollographql.apollo3.gradle.internal.ApolloGenerateSourcesTask.taskAction(SourceFile:275)
      // ...

Let me know if there's anything else I could add that would be helpful!

Hi! 👋  Thanks for reporting this.
For now, I couldn't reproduce the issue and the message "couldn't make a guess" doesn't ring a bell.
I have created a very simple project here, maybe you can try and tweak it to show how to reproduce the issue? Thanks a lot!

Actually we've found the culprit (thanks @martinbonnin!), it is due to using addJvmOverloads which unfortunately doesn't work as expected due to relocation.
This will need a fix on our part, in the meantime you can disable addJvmOverloads and this should resolve this issue.

Ahh good find.  I wonder why the tests included as part of #3907 didn't fail originally 🤔

@theBradfo This is because these don't depend on the relocated version of the plugin/compiler.

Just confirming that removing addJvmOverloads does remedy the issue, however I would like to use addJvmOverloads eventually so a fix would be much appreciated :)

Fixed in just released 3.2.2


Update CLI client code and add tests

Description
This change is a major update to the CLI's client code for interacting with the Radius API.
The client grew organically over time as we added features. Since it's client code it wasn't well tested because testing it would require a lot of mocking. Unfortunately in some places we do have complex logic in this code and unfortunately we also have some bugs.
I'm working on another fix to address #7520, and encountered a lot of limitations with the client code so I decided to fix it.
This change addresses the following:

Reviewed the API and made updates for consistency.
Added tests for ALL of the functions on the client.
Updated each API to accept either a resource ID or a resource name (needed for #7520).

Type of change

This pull request is a minor refactor, code cleanup, test improvement, or other maintenance task and doesn't change the functionality of Radius (issue link optional).

Looks like there's another bug for me to fix that's blocking functional tests. This is the downside of mock-based testing, you could have bugs in the code that's mocked out.

FYI - I found https://github.com/radius-project/radius/issues/7597 while troubleshooting this code and added a workaround for it.

I'm going to merge this. @vinayada1 - if you have additional feedback for me to address I'll do it in a follow-up 👍


Responsive radio

Github Issue or Trello Card
This PR addresses this issue: https://github.com/palmetto/palmetto-components/issues/437
What type of change is this?
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Updating documentation.
[ ] Updating deployment/build pipeline.
[ ] Updating Project Dependencies
[ ] Improving or adding to Test Coverage

Completeness Checklist

[X] TESTS: My changes maintain the baseline required test coverage, as specified by code climate analysis.
[X] DOCS: All new component work is covered in that component's Overview docs.
[X] DOCS: All new component work is covered in a component Playground.
[X] DOCS: All new visual changes or options are covered under relevant components' VisualTests.
[X] My code has no linting or typescript compile warnings.
[X] My work is tied to a Github issue and satisfies the acceptance criteria (if applicable) of the corresponding issue.

Quality Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes

UI Checklist

[X] I have conducted visual UAT on my changes/features.
[X] My solution works well on desktop, tablet, and mobile browsers.

@nathanyoung can you still replicate the issue with mixed state across inputs? I am unable to replicate it at the moment.

@nathanyoung can you still replicate the issue with mixed state across inputs? I am unable to replicate it at the moment.

yes, it's because the ids are not unique. There are accessibility warnings too.

 Responsive sizes design and implementation for RadioGroup


Underscore key support for environment variables

Unfortunately the env back-end makes asthetic variable naming a bit.. tricky. This is due to all _ being replaced with /, as well as the key upper being compared to the envar raw(no room for casing).
#567 attempts to solve this by through the key naming mechanism, however a "gotcha" was pointed out as well when using key prefixes. I would like to propose an alternative approach:
__Dunder__
We can replace only single _ in the envars with /, and shorten dunder __ to underscore _ for comparisons.
APP_SERVER_BIND__PORT => app/server/bind_port
I believe this also has the potential for a negligable impact on backwards compatibility as previously a dunder would have been converted to a das.. // 😄
The primary motivation for me on this is to allow for a more "conventional" looking naming scheme for environment variables while also playing well with alternative backends. I would be happy to submit a PR if this approach sounds good.

It is not only underscore _ that is causing the problem, many other characters like - and : are not supported in environment variables. We can only say that this is a very limited backend. Currently I'm using YAML file instead - put a YAML file completely in the environment variable (e.g. when creating a container), save it to a YAML file in a starting script, and use confd file backend.

Yes, I can agree it is certainly more limited in keyspace than most(all?) of the other backends. However, I think more can be done than simply pointing this out 😄
In my case I'm distributing docker images for others to use outright or build on top of. While cramming a document into an environment variable may work for my own deployment purposes, it's not such a good look for a distributed image.
What I would like is a way for people to start out with a Docker Hub image, try it out using conventional configuration means(environment variables and the odd volume where absolutely necessary), and then have a clear path forward for more advanced configuration as their needs and deployment infrastructure mature.
As it is, environment variables are sort of the "defacto", familiar configuration API for docker containers and basic compose setups. While APP_JAAS_MODULE_JETTY__CACHING__LDAP is slightly unconventional, it is much nicer than APP_JAAS_MODULE_JETTYCACHINGLDAP.

Different people may have different feelings for this... Some may think that two underscores are more "separated" than one underscore, so they may feel strange that why APP_MYAPP_CACHE__BACKUP corresponds to /app/myapp/cache_backup but not app_myapp_cache/backup. Env backends should be very straight forward, so I'm afraid that this design - although works quite well - may be confusing for users.


Problem with sqlalchemy .

async def search_offertypes(order: bool = False, parameter: OffertypeList = Depends(),
db: Session = Depends(get_db)) -> Any:
return paginate(db.query(OfferType))
output:
TypeError: object of type 'Query' has no len()

How route registration looks like?

I cann't see how search_offertypes is registred at offer_router

import time
from fastapi import APIRouter, Depends
from typing import Any, List
from sqlalchemy.orm import Session
from offertype.core.utils import get_db
from offertype.models import OfferType
from fastapi.responses import JSONResponse
from offertype.schemas import OffertypeCreate, OffertypeList, OffertypeUpdate
from fastapi.requests import Request
from fastapi.exception_handlers import HTTPException
from fastapi_pagination.ext.sqlalchemy import paginate
offer_router = APIRouter()
@offer_router.post("")
async def create_offertype(offertype: OffertypeCreate, request: Request, db: Session = Depends(get_db)):
try:
    offer = OfferType(**offertype.dict())
    offer.dt_create = time.ctime()
    db.add(offer)
    db.commit()
    db.refresh(offer)
    return offer
except Exception as e:
    print(e)
    return JSONResponse(status_code=403, content={'Error': 'Não foi possível cadastrar a offertype'})

@offer_router.post("/teste/")
def search_offertypes(db: Session = Depends(get_db)) -> Any:
return paginate(db.query(OfferType))

Try update code to (bassically you should set response_model):
from fastapi_pagination import Page

@offer_router.get("/teste/", response_model=Page[Any])
def search_offertypes(db: Session = Depends(get_db)) -> Any:
    return paginate(db.query(OfferType))

I i use any Pydantic models inside Page, this happen

No problem, happy you that you resolve your issue)


High blob count optimizations part 1 (to master)

These are the optimizations discussed.
The most important is to allow prefix-based searches.  I realized we didn't need any new parameters or syntax at all for that, can just treat it as an optimization of the existing include-pattern.
The other is to back off the progress reporting frequency at high file counts.  Because otherwise, looping through all the transfers to count those at the various statuses becomes expensive. (It still will be expensive if there are skipped or failed files, since as per known bug, they are re-listed every time. That bug is not fixed here.  I.e. this will be good for cases where there aren't too many skips or fails).

Disregard testing failure, it's just the macos build (which will always fail on master until we merge dev, as it has the change fixing the macos image version)


tisdk-default-image: matrix-gui packagegroup removed in meta-arago

We no longer need to remove it in this layer, given that it is dropped
from meta-arago entirely 1.
Signed-off-by: Aniket Limaye<EMAIL_ADDRESS>

@aniket-l
I'm working on few more cleanups, post which I would create a PR.
Does that sounds fine?

@cshilwant Yeah that works for me. Can reject this I guess

@aniket-l Sounds like a plan!


export: Ensure zulip->zulip export covers all tables that should be exported

I just completed the TODO project we'd for a long time had at the top of ALL_ZERVER_TABLES (well, now it's ALL_ZULIP_TABLES) to validate the list of tables that we export against the list of tables that are actually present; see 42aea68df3cc9e58c8e8ac857aab5bc36022129a.  I suppose this probably increases the priority of making sure we call sanity_check_output in CI (or at least refactor so the first piece is run there).
Aside from that bit of tooling work, the TODO list of things to fix is this:
    # The fact that these are not exported is a bug
    'zerver_botstoragedata',
    'zerver_botconfigdata',
    'zerver_mutedtopic',
    'zerver_realmauditlog',
    'zerver_pushdevicetoken',
    'zerver_service',
    'zerver_usergroup',
    'zerver_usergroupmembership',
    'zerver_userhotspot',

A few of these are larger projects (like botstoragedata / botconfigdata), just because we may need to add some to populate_db to make testing it convenient.  But I think most of these are pretty straightforward and we should just aim to fix quickly.  Some notes:

user groups are present in populate_db
hotpots are present in populate_db
muted topics are easy to make in the UI
realmauditlog entries are easy to make, but this one make require a bit of work just because of its foreign key pieces.
push device tokens might need to be made manually (see test_push_notifications.py).
service comes from making an outgoing webhook or embedded bot

And then medium priority is to also take along the analytics tables:
    'analytics_fillstate',
    'analytics_realmcount',
    'analytics_streamcount',
    'analytics_usercount',

(Check out the docs at http://zulip.readthedocs.io/en/latest/subsystems/analytics.html for how to get a analytics.ds realm with good test fixture data for this)

@rheaparekh FYI, this is the follow-up I'd mentioned :)

@zulipbot claim

@zulipbot I am still working on this.

This is fixed via db1260fb939feaa01da4570d5bc0dc3f567b6e4b, though we haven't done the analytics tables (which I'm not sure are really important to do; worth doing if they're easy, I guess).


[#812] [model] Optimize model input/output

Optimize models extra input/output memory allocation counting towards peak memory allocation.
Memory is allocated with for input of input layer and output/gradient of output layer.
However, that memory is never used as train_run() allocates new buffer and passes it to the
input layer/loss layer.
This patch takes the already allocated memory from input/loss layer to be used to collect input/label data.
This patch also removes the extra parameters from forwarding/backwarding and with corresponding
with_val functions. Further, two types of forwarding in loss layer has been merged to just 1 function.
Now, loss layer and input layer does not need to be distinguished and can be treated as a regular layer.
Signed-off-by: Parichay Kapoor<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #829. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.

Moved to #836 .


Update parameter management notebook

Update the parameter management notebook to show how to create a chemistry dict from your own parameters, e.g.
like this
Chen2020 = {
    "chemistry": "lithium_ion",
    "cell": "LGM50_Chen2020",
    "negative electrode": "graphite_Chen2020",
    "separator": "separator_Chen2020",
    "positive electrode": "nmc_Chen2020",
    "electrolyte": "lipf6_Nyman2008",
    "experiment": "1C_discharge_from_full_Chen2020",
    "sei": "example",
    "citation": "Chen2020",
}

but passing your own folder names.
This will make it easier for people who want to add an entirely new set locally, but using the cell, negative electrode, .... file structure.

I just wanted to point out that in my Spyder this had worked using ' ' instead of " " for some reason. Using " " returned an error...

I also wanted to ask a follow up question on this:
-> Within the custom parameter set folders, it is recommended to have a readme. file. I am not familiar with these. Along with the structure of Chen2020, I have used the readme. files in my custom parameter folders but did not edit these, is this a problem or is this fine?

The readme just gives some information on where the parameters came from (e.g. gives references and any other notes)

Closing this as parameters are now provided more easily in a single file (see #2342). Third party sets can be added using entry points .


Added solution link for Challenge 1

We checked out your repository. We're thrilled to say your solution is now listed in our Hall of Fame
Thank you for your contributions!
This PR is now closed! 🎉


Add custom format description to clarify one can use momentjs format

Adds a small description to the custom format panel options so it is clear users can use momentjs .format syntax
Related to https://github.com/grafana/clock-panel/issues/170

Might be because I'm lacking context here, but feels like the information is still a bit sparse. Do we describe how momenjs format syntax work in the plugin docs?

We have a link to the momentjs documentation but I suppose having to go and read the plugin readme while working on the panel options is not very common.


{KeyVault} Fix tests fail when running in serial due to keyvault name is duplicated in global in-momery cache

Description
KeyVault tests:

test_create_for_rbac_with_new_kv_cert
test_create_for_rbac_with_existing_kv_cert
would fail if run in serial but would succeed if running separately. The 1st one would pass, the 2nd would fail.

The fail reason is like https://github.com/Azure/azure-cli/pull/13002.
Details of why describe in this issue.
Testing Guide
Will fail if run pytest -v -x --log-level WARN src/azure-cli/azure/cli/command_modules/role/tests/latest/test_role.py
History Notes

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.

draft


Improve Seo

Improve the SEO by doing this

Adding alt text in every image on the website
Adding the remaining meta tag
Reduce the load time of the website
Format the structure of a webpage
Remove unwanted CSS
Add sitemap

I want to Work on this issue, can you please assign this to me.

I want to work on this issue @Intellimixiopce. Can you please assign it to me.


Suggestions for improving "https://libera.chat/guides/connect" for new users

I am completely new to IRC and found the documentation for getting connected to libera.chat confusing.  I used the URL in the above title to attempt connect to Libera.Chat using "web.libera.chat/gamja".  I chose it because it was stated to be "lightweight" and this hinted that it would be the simplest IRC client to use.  I then moved on to "Helping you connect" which immediately led to "Connecting to Libra.Chat".  I have the following suggestions on this page.

There needs to be paragraph explaining an overview of the connection process.  Jumping right into "Connect to Libera.Chat with TLS" is akin to describing how to wind a watch instead of explaining that it is used to tell time.  "Accessing Libera.Chat Via TLS" should be an aside of the general connection process.
Consider focusing on getting new users on board as simply as possible while pointing out avenues for additional capabilities and their associated requirements instead of jumping into a barrage of overwhelming details without a mental model upon which to hang the detail.  It seems like you have to have a mental model to understand the barrage of detail about using WebChat, SASL and/or CertFP so that you can form a mental model.  This complexity leads to confusion and I suspect leads many to give up on using Libera.Chat.
It seems to me that the description of accounts, passwords, nicknames and channels ought to be the primary information described along with why they are used.  Thus a simple but useful model of how IRC works can be learned and used by the newcomer.  Your objective should be to create a pattern or model of registration and use.  Give simple clear actionable definitions with indications that more power and complexity are enhancements to these definitions.
Examples are better than abstract descriptions.  Use Gamja or Kiwiirc as the example and generalize the named action in the descriptive text.
Avoid using specialized IRC terms without a clear definition.  For example the term "cloak" appears in regard to the term "user/nickname" with no foundational definition for either to the newcomer.  The term "Joining or talking in certain channels" is first used with no definitions for "Joining", "talking" or "channels".  The assumption that all or even most readers understand the technical actionable meaning of these words is exclusionary to new users such as myself. It is ok to repeat the definitions, while occasionally tedious, reinforces the concepts.
"Linking or Embedding" is a side issue of getting connected and really does not belong  here.  It appears to be aimed at a specialized subset of the general IRC population.  It seems more appropriate to come in a later part of the guide about expanding the use of  IRC.
In "Finding Channels" it would be good to first describe what channels are and how they are used.  The assumption that your new readers know what a channel is is problematic.  There are several common and many specialized area meanings of channels and IRC adds yet another meaning.  It appears that a channel is a discussion group focused on a semantic area.
"User Modes"  Again the assumption that your readers know what "User Modes" are with no definition is problematic.  One has to read the entire "usermodes" page to construct a "user mode" definition.  What is the "FOSS" community?

These suggestions may help to improve the clarity of the guides and therefore help newcomers to  more quickly be able to contribute to community discussions.

I want to know how can I register a nickname while I can't connect to libera.chat because SASL auth failed.

I want to know how can I register a nickname while I can't connect to libera.chat because SASL auth failed.

+1
when enter /connect libera, it replys that i need to register nickserv, where you had to /connect first. it's a dead loop, how would that happen.

+1
Not being able to register from https://libera.chat/guides/sasl#sasl-access-only-ip-ranges is very lame.

a possible alternative is a web registration form

+1

Yea I'm in the same boat now. Can't login because I haven't registered, and can't register because I'm not logged in.


Using Saxon-JS XSLT Task with space characters in the command line

Space characters in paths are converted to %20 by Saxon-JS (2.1) xslt3 and therefore execution fails.
This is a known issue in Saxon-JS - see: https://saxonica.plan.io/issues/4925

The Saxon-JS 2.2 release fixes this issue.


Bas Relief from 3D Models

Hi @marcomusy
What are the methods in vedo to convert a 3D model to a Bas relief Model by compressing the depth of model as shown in
https://www.youtube.com/watch?v=9zurRXrntJs
I want to transfer a 3D model to Planer surface as very low depth as

to

It is not so straightforward. You can work out a depth map porting your image to a numpy array and then make a mesh out of it.
Check out: https://github.com/marcomusy/vedo/blob/master/examples/basic/picture2mesh.py


Params validation

Currently the commands assume that the env variables exist. Some validation would be neat.

I will take this one :-)

Hi @woltsu how would you like the validation to be handle. if the env var doesn't exist what happens

I think that a clear warning message should be enough. We don't want the whole build to fail in my opinion.

@woltsu  i have been searching and i haven't found a way of doing this. Is there a resource that you can point me to that i can perhaps read that will help :-)

For now I think that the solution would be to echo a warning message to the user if some of the parameters are empty or null. You need to expand the command itself in here. Here's some discussion on how to figure out whether a variable is null/empty in bash: https://stackoverflow.com/questions/3061036/how-to-find-whether-or-not-a-variable-is-empty-in-bash

@woltsu i don't have a bot, is there a way i can test this locally?

No problem! Unfortunately I have not found a way yet to test this locally. But, it's very easy to create a bot of your own by following the instructions on the bottom of the README file. Then, just add your own environmental variables into the CircleCI project configuration. More info about that here: https://circleci.com/docs/2.0/env-vars/#setting-an-environment-variable-in-a-project.

There does seem to be a way to run circleci jobs locally, but I haven't familiarized myself with that yet: https://circleci.com/docs/2.0/local-cli/#run-a-job-in-a-container-on-your-machine

Awesome i will have to because i saw a couple of issues you raised that i would like to take a stab at
In the mean time i have created a PR check it and revert before i move to the other issues.


How to set "tabindex" on monaco editor programmatically or otherwise?

monaco-editor version: 0.10.1
Browser: Chrome
OS: Windows 10
I asked this question on Stack Overflow a few days ago (https://stackoverflow.com/questions/47897989/how-to-set-tabindex-on-monaco-editor-programmatically-or-otherwise) and since I didn't get an answer there, I am posting it here.

I'm wondering if there's a way to set the "tabindex" on the editor programmatically or otherwise. In my application, monaco editor is part of a rather complex form (lying somewhere between regular HTML form field elements) where I need to define tabindex attributes on form field elements. I believe that the tabindex on the editor is set as "0". The problem it creates is that once I tab out of the editor, it jumps to an HTML element out of the order.
For example, please look at image below.

Ideally, when I press the tab key in the editor, the focus should go on the textbox for "LOCKED UNTIL" (1) and when I press shift+tab key in the editor, the focus should go on the textbox for "MESSAGE ID" (2). However, pressing the tab key puts the focus on the splitter (3) as both the splitter and the editor has the tabindex of "0".
Currently to fix this problem, I am capturing the keyDown event on the editor and setting the focus manually.
  editor.onKeyDown((evt) => {
    if (evt.keyCode === monaco.KeyCode.Tab) {
      if (evt.shiftKey) {
        $('#MessageId').select();
        evt.preventDefault();
        evt.stopPropagation();
      } else {
        $('#MessageLockedUntil').select();
        evt.preventDefault();
        evt.stopPropagation();
      }
    }
  });

I'm wondering if there's a better way of handling this?

We could have a tabIndex property that we would put on our "hidden" textarea.

This is very needed... any solution or work-around it? I appreciate your help.

Any update regarding this? In my case I would like to navigate through multiple Monaco editors using tab.


Newtonsoft v9 compatibility

I have created a test app with the latest version <IP_ADDRESS>, I then updated all the nugget packages except for selectjs2 and rebuild and run the app which runs fine, however when I transform the t4 templates it produces an error that Newtonsoft V8 cannot be found. I assume I need to update a reference somewhere, any ideas? The offending line is "Serenity.JSON.Parse[T](String input)" but I don't know where to find or fix this.

To get it to work with v9, you need to add an assembly binding redirect in devenv.exe.config. Probably not worth it and not exactly portable to other developers on your team/build machine.

Ok Thanks.
Just to sure, it's both selectjs2 and Newtonsoft V9 that I don't update.
On Fri, Aug 12, 2016 at 12:25 PM, wldkrd1<EMAIL_ADDRESS>wrote:

To get it to work with v9, you need to add an assembly binding redirect in
devenv.exe.config. Probably not worth it and not exactly portable to other
developers on your team/build machine.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/volkanceylan/Serenity/issues/1058#issuecomment-239492657,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASwgMUcV-1ddUSZtzQJX5X96kq0P02cSks5qfJ5xgaJpZM4JjDu2
.

Select2.js, Newtonsoft, jQuery.UI.Combined and jQuery should not be updated.

Hi,
I have made a simple Excel Import that will take an Excel sheet and import
it into a Serenity table. I called it a wizard (although it's not yet) to
avoid naming issues. It's a stripped down version of a generic importer I
wrote that may be useful . The files are attached to this email in zipped
format.
I am sending them to you this way so that you can "Serenetize" them and
post them according to your specifications. There are a number of methods
that I'm sure you have better hooks for than what I did.
I hope this helps,
Trevor


Only a couple escape sequences are valid: https://golang.org/ref/spec#Rune_literals

It's because gore enters line-continuation mode if it fails to parse input. What is your expected behavior?
As you may already know, for string literals you must surround them by "s:
gore> "hello\t"
"hello\t"

It's because gore enters line-continuation mode if it fails to parse input. What is your expected behavior?

That explains it :P I was expecting an error.

Got the point. I have currently no idea with this problem.

The reason I ask is because I'm writing a CLI app too, and I wanted to see how you handle backslashes. For my app, all backslashes outside of quotes are literals, and backslashes within quotes always escape the character after, which you can see here. Where do you parse CLI input?

One night passed and I think we can use go/scanner to detect illegal characters in the source, though I have not tested it.
FYI, I'm parsing input using go/parser.ParseFile here and if the error was go/scanner.ErrorList, gore prompts to continue line here.

This issue is resolved now.


DKT-F:Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu!

Hi , I am currently working with DKT-F and encountered the following error: "Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu!"

Hi , I am currently working with DKT-F and encountered the following error: "Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu!"

Hi, this issue is caused by the tensors are not on the same device. Could u provide more details about your errors？Such as which line did the error occur?

Hi , I am currently working with DKT-F and encountered the following error: "Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu!"

Hi, this issue is caused by the tensors are not on the same device. Could u provide more details about your errors？Such as which line did the error occur?

File "/home/gzd/Desktop/pykt-toolkit/pykt/models/dkt_forget.py", line 56, in forward
rgap, sgap, pcount = self.rgap_eye[rgap].to(device), self.sgap_eye[sgap].to(device), self.pcount_eye[pcount].to(device)

.rgap_eye[rgap].to(device), self.sgap_eye[sgap].to(device), self.pcount_eye[pcount].to(device)

The tensors of self.rgap_eye,   self.sgap_eye and   self.pcount_eye are not on GPUs. The following codes may work.
self.rgap_eye = torch.eye(num_rgap) --> self.rgap_eye = torch.eye(num_rgap).to(device)
self.sgap_eye = torch.eye(num_sgap) --> self.sgap_eye = torch.eye(num_sgap).to(device)
self.pcount_eye = torch.eye(num_pcount) --> self.pcount_eye = torch.eye(num_pcount).to(device)

您好，我运行最新的sparseKT模型，也遇到了类似的错误，而且报错的行很多。

您好，我运行最新的sparseKT模型，也遇到了类似的错误，而且报错的行很多。

您好，我也遇到了您同样的情况，请问您解决了么？


Correct calculation of GK boundary fluxes with curvilinear geometry

This PR addresses issue #538 . This PR also goes with PR 50 of gkylcas: https://github.com/ammarhakim/gkylcas/pull/50
In this work:

We give eqn objects a new boundary flux term which is like boundary flux but has an extract factor of $|e^i|$ where e^i is the dual vector along the i-th directin.
We generated these new boundary flux terms for GK only, since other models don't have support for curvilinear geometry.
Implemented a new boundary_flux updater, which calls these kernels and is similar to ghost_surf_calc, but is more fine grained because it only does one boundary at a time (the app can create one of these for each boundary if desired).
Plug in this new boundary_flux updater to the GK app.

In an initial test with mirror geometry and Boltzmann electrons, we see that initial potential has the same sheath potential with uniform and nonuniform z grid in this branch, which was not the case before.
We also ran the affected tests, rt_gk_ltx_boltz_elc_1x2v and rt_gk_mirror_boltz_elc_1x2v, and produced similar shape final potentials but they were shifted due the shift in the sheath potential with the new boundary fluxes (and the number densities remained the same).

This fix helped clarify the differences between non-uniform and uniform geometry in the potentials. See how the mc2p_fix and mc2p_fix_uniform geometry line up? The line for mc2p without the fix in non-uniform geometry is labeled mc2p. There is a significant change in the densities and temperatures, too. This compares a 144 cell non-uniform grid to a 288 cell uniform grid, initialized with uniform densities. It's very promising that these two plots line up after this bug fix is implemented.

@tnbernard. It's most important that the potentials agree. This means that the sheath BC is producing the correct potential in both uniform and non-uniform geometry cases after the fix. However, the density and temperature profiles are unchanged with this fix. It means that the non-uniform geometry produces the same moment profiles after the fix. I wouldn't expect the results from the non-uniform grid to be the same as the uniform grid because they are on two different grids.


Achieve CII Passing status

What would you like to be added:
Work towards and achieve the CII Passing level.
https://bestpractices.coreinfrastructure.org/en/criteria
Why is this needed:
As a part of making sure we are a healthy project and a welcoming project for contributors.

Bump

We still want to do this, go away Stale Bot.

bump

Can I put you forward as our security champion, @skitt?

The project MUST have at least one primary developer who knows how to design secure software. (See ‘details’ for the exact requirements.)
This requires understanding the following design principles, including the 8 principles from Saltzer and Schroeder:
economy of mechanism (keep the design as simple and small as practical, e.g., by adopting sweeping simplifications)
fail-safe defaults (access decisions should deny by default, and projects' installation should be secure by default)
complete mediation (every access that might be limited must be checked for authority and be non-bypassable)
open design (security mechanisms should not depend on attacker ignorance of its design, but instead on more easily protected and changed information like keys and passwords)
separation of privilege (ideally, access to important objects should depend on more than one condition, so that defeating one protection system won't enable complete access. E.G., multi-factor authentication, such as requiring both a password and a hardware token, is stronger than single-factor authentication)
least privilege (processes should operate with the least privilege necessary)
least common mechanism (the design should minimize the mechanisms common to more than one user and depended on by all users, e.g., directories for temporary files)
psychological acceptability (the human interface must be designed for ease of use - designing for "least astonishment" can help)
limited attack surface (the attack surface - the set of the different points where an attacker can try to enter or extract data - should be limited)
input validation with allowlists (inputs should typically be checked to determine if they are valid before they are accepted; this validation should use allowlists (which only accept known-good values), not denylists (which attempt to list known-bad values)).
A "primary developer" in a project is anyone who is familiar with the project's code base, is comfortable making changes to it, and is acknowledged as such by most other participants in the project. A primary developer would typically make a number of contributions over the past year (via code, documentation, or answering questions). Developers would typically be considered primary developers if they initiated the project (and have not left the project more than three years ago), have the option of receiving information on a private vulnerability reporting channel (if there is one), can accept commits on behalf of the project, or perform final releases of the project software. If there is only one developer, that individual is the primary developer. Many books and courses are available to help you understand how to develop more secure software and discuss design. For example, the Secure Software Development Fundamentals course is a free set of three courses that explain how to develop more secure software.

At least one of the project's primary developers MUST know of common kinds of errors that lead to vulnerabilities in this kind of software, as well as at least one method to counter or mitigate each of them.
Examples (depending on the type of software) include SQL injection, OS injection, classic buffer overflow, cross-site scripting, missing authentication, and missing authorization. See the CWE/SANS top 25 or OWASP Top 10 for commonly used lists. Many books and courses are available to help you understand how to develop more secure software and discuss common implementation errors that lead to vulnerabilities. For example, the Secure Software Development Fundamentals course is a free set of three courses that explain how to develop more secure software.

https://bestpractices.coreinfrastructure.org/en/projects?q=submariner

We're at 93%. I thought there was a way to allow other people to edit our answers, but I don't see it now that I'm trying to add folks.
https://bestpractices.coreinfrastructure.org/en/projects/4865

Regarding sharing editing permissions, I see:

Note - if your repository URL is on GitHub, anyone who can commit to the repository will be able to edit the badge information.

https://bestpractices.coreinfrastructure.org/en/projects/new
Digging through their GitHub, I also see:

Additionally, To minimize calls to github also check if a GitHub user owns the repo of a badge, if so allow the user to edit the badge regardless of badge project "owner."

https://github.com/coreinfrastructure/best-practices-badge/pull/1419
I think that means you should be able to edit the badge information, @skitt?

Additionally, To minimize calls to github also check if a GitHub user owns the repo of a badge, if so allow the user to edit the badge regardless of badge project "owner."

coreinfrastructure/best-practices-badge#1419
I think that means you should be able to edit the badge information, @skitt?

I’ve logged in with my GitHub account but I can’t edit anything :-(.

I wonder if that’s because the repository entry on the badge points to the organisation on GH, not a specific repository.

I see we also meet a number of silver level criteria...

I wonder if that’s because the repository entry on the badge points to the organisation on GH, not a specific repository.

Yep, that was it, thanks @skitt! Everyone who can edit the main repo should now be able to edit the CII badge.

Lots of PRs to add the CII badge to READMEs (and clean up other badges):

https://github.com/submariner-io/submariner/pull/1343
https://github.com/submariner-io/submariner-operator/pull/1370
https://github.com/submariner-io/admiral/pull/221
https://github.com/submariner-io/enhancements/pull/31
https://github.com/submariner-io/lighthouse/pull/538
https://github.com/submariner-io/releases/pull/140
https://github.com/submariner-io/shipyard/pull/559
https://github.com/submariner-io/submariner-charts/pull/142

These are the two remaining CII questions:

The security mechanisms within the software produced by the project MUST generate all cryptographic keys and nonces using a cryptographically secure random number generator, and MUST NOT do so using generators that are cryptographically insecure.
A cryptographically secure random number generator may be a hardware random number generator, or it may be a cryptographically secure pseudo-random number generator (CSPRNG) using an algorithm such as Hash_DRBG, HMAC_DRBG, CTR_DRBG, Yarrow, or Fortuna. Examples of calls to secure random number generators include Java's java.security.SecureRandom and JavaScript's window.crypto.getRandomValues. Examples of calls to insecure random number generators include Java's java.util.Random and JavaScript's Math.random.

A cryptographically secure random number generator may be a hardware random number generator, or it may be a cryptographically secure pseudo-random number generator (CSPRNG) using an algorithm such as Hash_DRBG, HMAC_DRBG, CTR_DRBG, Yarrow, or Fortuna. Examples of calls to secure random number generators include Java's java.security.SecureRandom and JavaScript's window.crypto.getRandomValues. Examples of calls to insecure random number generators include Java's java.util.Random and JavaScript's Math.random.
This criterion does not suggest enabling assertions during production; that is entirely up to the project and its users to decide. This criterion's focus is instead to improve fault detection during dynamic analysis before deployment. Enabling assertions in production use is completely different from enabling assertions during dynamic analysis (such as testing). In some cases enabling assertions in production use is extremely unwise (especially in high-integrity components). There are many arguments against enabling assertions in production, e.g., libraries should not crash callers, their presence may cause rejection by app stores, and/or activating an assertion in production may expose private data such as private keys. Beware that in many Linux distributions NDEBUG is not defined, so C/C++ assert() will by default be enabled for production in those environments. It may be important to use a different assertion mechanism or defining NDEBUG for production in those environments.

This is done, thanks for all the help @skitt! :partying_face: :heavy_check_mark:


Work towards a configurable fips setup

This PR is supposed to replace the feature/fips branch. Notable updates:

rebase to latest master branch
build the k8s-dqlite and kubernetes services with the right flags for fips
introduction of env files to link to the right libcrypt libraries.

Moved to feature/fips branch


Use Alpine instead of Ubuntu for Docker Image

This could reduce the image size from 1.6GB (or 900MB) down to 250MB.
There is a libmesos bundle that should work with Alpine.

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-4239. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.

Note: This issue has been migrated to https://jira.mesosphere.com/browse/MARATHON-4239. For more information see https://groups.google.com/forum/#!topic/marathon-framework/khtvf-ifnp8.


[ISSUE #676]♻Refactor RemotingCommand struct🚀

Which Issue(s) This PR Fixes(Closes)

Fixes #676
Brief Description

How Did You Test This Change?

🚀Thanks for your contribution🎉. CodeRabbit(AI) will review your code first🔥


/v1/register-application circular dependency problem for new application registration.

Following is the procedure to register a new application in registry office ,

v1/register-application service in registry office should be addressed by the new application.
In order to address this service , the new application requires the correct operation-key.
But It is not possible to obtain the correct operation key until a new application is provisioned within the micro service framework.

We have to check possible solutions to overcome this circular dependency problem.

The registry-office :: /v1/register-application service shall not request for a proper operation-key value.
Instead registry-office shall be open for all registration attempts and assume to have received a deregistration request at the end of a time period (which can be configured as a Integer-Profile), if it didn't receive an approval message from the TypeApprovalRegister in meantime.
The security: apiKeyAuth: [] stereotype will be be removed from the registry-office :: /v1/register-application service with the next release of the specification.


Chartboost.start(...) -> Exception

I use the Chartboost extension in my iOS libGDX project. But if I call [i]Chartboost.start(...)[/i] I always get the following exception.
	at org.robovm.objc.ObjCObject$Marshaler.protocolToNative(ObjCObject.java)
	at org.robovm.bindings.chartboost.Chartboost.$m$startWithAppId$appSignature$delegate$(Native Method)
	at org.robovm.bindings.chartboost.Chartboost.start(Chartboost.java)
	at com.example.game.IOSLauncher.didFinishLaunching(IOSLauncher.java)
	at com.badlogic.gdx.backends.iosrobovm.IOSApplication$Delegate.$cb$application$didFinishLaunchingWithOptions$(IOSApplication.java)
	at org.robovm.apple.uikit.UIApplication.main(Native Method)
	at org.robovm.apple.uikit.UIApplication.main(UIApplication.java)
	at com.example.game.IOSLauncher.main(IOSLauncher.java)```

The Chartboost binding is subject to be deleted. Please use the new iOS RoboPod for Chartboost: https://github.com/robovm/robovm-robopods


Type of class member functions does not contain this type

It appears as though the this type of a class member function is not being reported as part of the type definition for the function and instead it is inferred as unknown even though usage of this within the function behaves appropriately.
TypeScript Version:
3.7.5
noImplicitThis
strictFunctionTypes

Search Terms:
Member Function This Type
Extract This Type
Code
class Foo {
    value: number = 0;
    func1() {
        this.value++;
    }

    func2(this: Foo) {
        this.value--;
    }

    func3 = function() {
        // can't access this.value
    }
}

type FuncType1 = Foo['func1'];
type FuncType2 = Foo['func2'];
type FuncType3 = Foo['func3'];

type InferThisType<T> = T extends (this: infer THIS, ...args: any) => any ? THIS : unknown;

type This1 = InferThisType<Foo['func1']>;
type This2 = InferThisType<Foo['func2']>;
type This3 = InferThisType<Foo['func3']>;

Expected behavior:
type FuncType1 = Foo['func1']; // (this: Foo) => void
type FuncType2 = Foo['func2']; // (this: Foo) => void
type FuncType3 = Foo['func3']; // () => void

type This1 = InferThisType<Foo['func1']>; // Foo
type This2 = InferThisType<Foo['func2']>; // Foo
type This3 = InferThisType<Foo['func3']>; // unknown

Actual behavior:
type FuncType1 = Foo['func1']; // () => void
type FuncType2 = Foo['func2']; // (this: Foo) => void
type FuncType3 = Foo['func3']; // () => void

type This1 = InferThisType<Foo['func1']>; // unknown
type This2 = InferThisType<Foo['func2']>; // Foo
type This3 = InferThisType<Foo['func3']>; // unknown

Playground Link:
https://www.typescriptlang.org/play/?ssl=20&ssc=120&pln=20&pc=1#code/MYGwhgzhAEBiD29oG8BQ0PQG5hAVwFMAuaAOzwFsAjAgJ2gF5oAGAbnUwDM9TgBGABQBKFB0yYALgAsAlhAB0OfAQDUK9uIC+qMdG68ATAOlySCeCLTjxJhUsIBaBxszbd+4AGZGensAky8KTCotaYAPTh0MBgpADkEtBgwMAEUNC2iriEutpuEgCeAA4EcH4AKsUEfD7mANpxHnxxALqs0JHQIQwAfNjwMgAmqIUlZbyVJQa1iA0eBq3tncayEGaIIr39QyNV48CTBN5M9Y1+nosdUd19WAPDu2MAkqScdOWrhwA85X1M5dACAAPCQEUiDGArUzQGSvOjQcoACSeAGUADTQeRYsC0ADmaySpAKmz6sQK0AA-AjkSjoCQeABrUjwADupHYj1KHzkNSYLzetG5EG+pyarR6SyijOZbM5CNW0z5cMFnyqX1FfgWLQlVzgiDlQuO0H571VJXVszOvAu2sl0GlrNIqCAA

Motivation
I am trying to create a mechanism for creating a type which excludes member-function properties, but my plan was foiled due to not being able to infer the this type for the function.
type InferThisType<T> = T extends (this: infer THIS, ...args: any) => any ? THIS : unknown;
type IsMemberFunction<T, P extends keyof T> = InferThisType<T[P]> extends T ? true : false;

type NonMemberFunctionPropertyKeys<T> = {
   [P in keyof T]: IsMemberFunction<T, P> extends true ? never : P
}[keyof T];

type ExcludeMemberFunctions<T> = Pick<T, NonMemberFunctionPropertyKeys<T>>;

I'd like to use this to provide stronger typing guarantees for angular bindings as well as the benefits of code-completion, etc.
type AngularBindings<T extends angular.IComponentController> =
  Partial<
    Record<
      Exclude<NonMemberFunctionPropertyKeys<T>, keyof angular.IComponentController>,
      '@' | '<' | '=' | '&' | '@?' | '<?' | '=?' | '&?'
    >
  >;

class MyController implements angular.IComponentController {
  public value: number;
  public callback: Function;

  public memberFunc() {
    this.value++;
    this.callback();
  }
}

class MyComponent implements angular.IComponentOptions {
  public bindings: AngularBindings<MyController>;
  public controller: angular.Injectable<angular.IControllerConstructor>;
  public template: string;

   constructor() {
     this.bindings = {
       value: '<',
       callback: '<',
       // memberFunc would produce compile error as would any typo.
     };
     this.template = html;
     this.controller = MyController;
  }
}

Related Issues:

Additionally, it would seem that fixing this could catch a class of errors when grabbing functions off of a class instance and calling them without call or apply
const f = new Foo();
const f1 = f.func1;
const f2 = f.func2;

f1(); // Should produce an error as calling f1() will throw.
f2(); // Error: The 'this' context of type 'void' is not assignable to method's 'this' of type 'Foo'. (2684)

https://www.typescriptlang.org/play/?ssl=1&ssc=1&pln=17&pc=6#code/MYGwhgzhAEBiD29oG8BQ0PQG5hAVwFMAuaAOzwFsAjAgJ2gF5oAGAbnUwDM9TgBGABQBKFB0yYALgAsAlhAB0OfAQDUK9uIC+qMdG68ATAOlySCeCLTjxJhUsIBaBxszbtqYPFIQJexmQIAdzhEYXZPb19OPn9OeX1+cK8fPQNY+J5gA3ZUaLDcoyFWIA

We don't implicitly add this to methods because it ends up making every function generic, which caused a 20%+ performance hit for compilation times in class-heavy codebases. You can "opt in" to methods which are in danger of being pulled off.


Use a better noRange

When there is a catastrophic failure to load a cradle (e.g. #610) the consequence is that every file is highlighted in red everywhere. If you want to continue developing with something like Ghcid it's super annoying to have to try and see the code under this sea of red. Solution is to treat cradle errors as being diagnostics spanning the first line of every file. It's still clear the error is there, you can still look at it, but it's not all you have to look at. There's no good reason to highlight an entire file in red.

LGTM!


Apt Repo cert issue

I'm getting intermittent TLS certificate errors (I can't confirm all are the same but the last one at least was claiming the cert had expired) trying to access the repo. I can't reliably reproduce the issue yet but it's happened several times in the last couple of hours.

The same happens for the website. Sometimes it says the certificate expired on May 27th but force reloading will usually fix it.

Thanks for bringing this up. I'm checking on this and will post back asap.

Yep this is very odd.
I came across it during a vagrant box build (i.e. vm image build, using packer) - the initial attempt to fetch the .deb during the automated setup failed at least half a dozen times - but even pausing the setup, connecting to the console of the in-process VM and trying to reproduce it, I always got a valid response.
Happy to help if you need more info/tests/whatever.

Please give it a shot, I think the webserver had some orphaned processes with old versions of the certificate. Not sure yet why it didn't kill those processes so hopefully the webserver won't keep triggering this issue. Otherwise I may need to look further into this.

I've made several builds in the last couple of days and none hit this error.

Awesome, thanks for reporting back! Closing this for now.


Create Docker image for ART

Description
Tried to address as many setup issues as possible to make things clearer for new comers. Also Dockerised the ART project to make sure we have a consistent reproducible environment across platforms. Updated README documentation accordingly.
Fixes #370
Type of change
Please check all relevant options.

[ ] Bug fix (non-breaking)
[x] New feature (non-breaking)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes

Had to rebase the commit graph to fixed an issue with a sign off missing on a commit. But this fails to update the current PR commit graph so I'm closing this PR and reopening another one with all the commits signed off

Continuing on #386


docs: improve the documentation for Aggregate code

Which issue does this PR close?
N/A
Rationale for this change
As part of reviewing the PR for https://github.com/apache/datafusion/pull/12269 from @jayzhan211 I found some
documentation that would make the code easier to understand.
What changes are included in this PR?

Update documentation

Are these changes tested?
By docs CI
Are there any user-facing changes?
No functional changes, only docs

I would add an example or something on group keys/values, its not very obvious imho

Done!

Thanks @alamb @comphead


Fix pluralization issue in Counter component

Hey there,
I've made a small fix to the Counter component in the react.dev site blog. The issue was related to the pluralization of the word "times" in the button text. When the count is 1, it should display "time" instead of "times".
Here's the changed code snippet:
`import { useState } from 'react';
export default function Counter() {
const [count, setCount] = useState(0);
function handleClick() {
setCount(count + 1);
}
return (

You pressed me {count} {count === 1 ? 'time' : 'times'}

);
}
`
This small modification ensures that the button text displays the correct singular or plural form based on the count value. Please review and merge this pull request if everything looks good.
Thanks!
Maneth

thank you

This is designed to be a very simple example and is intentionally simplified.

Ok thats all right 😇😇
On Sat, Sep 23, 2023 at 9:34 AM Sophie Alpert @.***>
wrote:

This is designed to be a very simple example and is intentionally
simplified.
—
Reply to this email directly, view it on GitHub
https://github.com/reactjs/react.dev/pull/6261#issuecomment-1732202907,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A565QICSACZ7BYWIGGVXM43X3ZNTZANCNFSM6AAAAAA37SISDQ
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Text expander

Description of the new feature / enhancement
I don't know if it is in your plans but a text expander would be a great addition to power toys.
Scenario when this would be used?
On all applications.
Supporting information
No response

Duplicate of #5074

Thanks for your suggestion! This issue is a duplicate.
I'm linking this issue to #5074 to centralize the discussion thread, please go to that thread for discussion.
Thanks for your help to make PowerToys a better piece of software!


[Bug] strict null checks for data flow resilience

PR checklist

[ X] Ensure you have added or ran the appropriate tests for your PR.
[ X] DCO signed

What type of PR is this?
bugfix for broken UI on asymmetric columns
What this PR does / why we need it:
UI breaks when comparison report doesn't have matching columns
Which issue(s) this PR fixes:
N/A
Special notes for your reviewer:

Does this PR introduce a user-facing change?:
Yes. Fixes UI breaking bug in comparison reports

Fixed issue where user gets a blank screen viewing comparison reports when base/target columns are non-matching

Note that tsconfig.json's strictNullChecks is now removed, so typescript will respect undefined and nullproperly, especially when it comes to optional argument parameters(arg?: ) => {}`


Compare cockpit and Webmin

Decide what is the most appropriate web admin interface.
desired features:

manage ngix to redirect hosts to containers/ports

decided for webmin


Fix asyncio exception

When using asyncio an exception is raised:
Error doing job: Task exception was never retrieved
Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/pysnmp/carrier/asyncio/dispatch.py", line 56, in handle_timeout
yield asyncio.From(asyncio.sleep(self.getTimerResolution()))
AttributeError: module 'asyncio' has no attribute 'From'

This PR fixes this.
Related to: https://github.com/home-assistant/core/issues/71368

I have read the CLA Document and I hereby sign the CLA

I have read the DCO Document and I hereby sign the DCO

recheck


User Research

What is it and who's it for?
Wellcome researchers would like to formulate the most valuable remote research process
User story
In order to participate in user research with Wellcome
As a "researcher"
I need/want/expect to be able to self apply for research opportunities online
Acceptance criteria
User research participants should be able to show their interest in Wellcomes research process
Implementation
Measurement

Number of research participants undertaking remote usability testing

Other considerations
Content design, messaging is important
What do the users want to gain from participating

@alicerichmond Please:

Assign this to yourself
Add this to the Project 'Experience Planning Board' and move to the 'In progress' column

Link to Miro with usability testing work flow, along with search user journey
https://miro.com/welcomeonboard/XSgZhTVfDjv8exClXEBfIvqPXM0QxsZihJFpdkLGVNYlZrBe1wRcqV1dbbdtBHeR

Confirmed with Aoife from the Trusts user research team that Wereby is a suitable research product for usability testing

Calendly can be used for booking times with us for research sessions

Still awaiting confirmation from compliance from Airtable, should be confirmed by the end of this week


pkc start currently is hardcoded to always start the autobackup service

Sometimes people don't want the autobackup service to be enabled to save RAM.

It would need to read the settings of the PKC and check if autobackup was disabled.


fix imports

I tried to solve this issue
https://github.com/magicuidesign/magicui/issues/185

thanks for this! just added a small fix to main to address this issue


Cannot find symbol new HotspotModule()

During run time it is throwing an error can't find symbol " new HotspotModule()".
Just from my end
"import reactnative.hotspot.HotspotPackage" this statement is importing HotspotPackage and in the later part it is adding  "new HotspotModule()" instead of "new HotspotPackage()".
Later tried to manually change this but during runtime RN is reverting them .
RN - 0.61.5
SDK 26

Hi, I have the same issue. Here is the solution. This worked for me.
https://github.com/assemmohamedali/react-native-wifi-hotspot/issues/10#issuecomment-539955334


Website Redesign: "About Us" Page Header Project Widgets- Desktop- UI

User Story
AS a New Orcahome user I WANT to see an easy to read webpage when I navigate to the Orcasound "About Us" page SO THAT I can easily learn about Orcasound's current projects.
Description
The intent of this story is to implement the UI finalized design from the Orcasound UX/UI team for the website redesign of the "About Us" Page. This Issue covers only the project widgets and text (see image below).
NOTES:
-Header/Footer already completed on different pull request #14
-This is the UI only, the pages for the Project Widgets will be created separately and can be linked to later.
-Use Material UI for the styling
Acceptance Criteria
GIVEN I open the Orcasound homepage WHEN I look at the page THEN I see a UI that matches the attached UX/UI design.
https://www.figma.com/file/YdvbAjHETokV2x1LBHaElj/Orcasound-Website-Redesign-2021?node-id=298%3A290

@MohitSharma-21 - Here is the other user story for the bottom half of the about page. You can work on it or unassign yourself and work on something else!

I can work on it @hgordon6634 .


Added eugodr

Hello! I was inspired by your post about meta pages and added my own a few days back, so I thought I'd join :)

Great, welcome!


read functions should return insightful errors

The read_cpt and read_bore functions have some "automagical" logic that infers the content of the file argument. The user can provide an object of types io.BytesIO | Path | str and with "engine"="auto", the content type is inferred automatically. This can result in confusing errors when erroneous input is provided.
Some examples:
Providing a non-existing path results in XMLSyntaxError
Input:
from pygef import read_cpt
read_cpt(file="non/existing/file.gef")

Response:
lxml.etree.XMLSyntaxError: Start tag expected, '<' not found, line 1, column 1

The expectation is to get a FileNotFoundError
Providing a non-existing path and engine="gef" results in ValueError
Input:
from pygef import read_cpt
read_cpt(file="non/existing/file.gef", engine="gef")

Response:
ValueError: The selected gef file is not a cpt. Check the REPORTCODE or the PROCEDURECODE.

The expectation is to get a FileNotFoundError
Providing an erroneous gef file results in XMLSyntaxError while gef can be parsed when forced
Input:
from pygef import read_cpt
read_cpt(file="path/to/erroneous.GEF")

Response:
lxml.etree.XMLSyntaxError: Start tag expected, '<' not found, line 1, column 1

Input:
from pygef import read_cpt
read_cpt(file="path/to/erroneous.GEF", engine="gef")

Response:
CPTData(bro_id=None, research_report_date=None, ...

The expectation is to get an error that the gef file is invalid, and this response should be consistent no matter the value for engine.

I see two options now, both breaking changes:

Require more content about the file argument (e.g. with a string_content=["path","file"] argument, but that would only have a meaning for file arguments of type str (kinda ugly)
As pandas.read_csv() does: Accept str | io.StringIO  types for file and always infer the str type as a Path, and the io.StringIO as file content. This would require the user to convert a string to StringIO object

I like the second one. we can set the type of filepath_or_buffer to Path |   io.StringIO. than we dont use strings and can use the Path.is_file() method.

When a string has the form of a path separated by slashes or even a single short word it can never be a valid XML or GEF file right? All XML files need to start with a < character, so that's easy, and all GEF files have the form of key: value, so I think a proper heuristic would be:
Is almost certainly path if:

Is the length <= 255 characters
Does it end with case-insensitive .gef or .xml
Does it not contain any < or : characters
Can it be opened from disk


Aws s3 bucket conversion has missing  unknown property 'transitionDefaultMinimumObjectSize' among [rules bucket expectedBucketOwner];

What happened?
when converting   https://github.com/terraform-aws-modules/terraform-aws-s3-bucket
PCL conversion works fine but when converting to a language we get
error: could not generate output program: :0,0-0: unknown property 'transitionDefaultMinimumObjectSize' among [rules bucket expectedBucketOwner];

This is meant to be part of BucketLifecycleConfigurationV2
Removing this line in the PCL causes the conversion to go over smoothly.
Example
na
Output of pulumi about
na
Additional context
No response
Contributing
No response

Upon investigation, this was added in the upstream provider at commit f6fda8c746d2b6951ae59d7a20d33dbaafe0d70f
It exists in our docs: https://www.pulumi.com/registry/packages/aws/api-docs/s3/bucketlifecycleconfigurationv2/#inputs
but when I run pulumi package get-schema aws it is not in there

Closing, updating my provider worked.  We should probably do something about this when a user is converting to make sure the versions match even when across pulumiverse versions, at the very least print a message.


Typechecking with Proptypes translated

Typechecking with Proptypes translated

Hi @klymenkoo and @ApolllonDev,
Thank you for reviewing.
I've applied the changes.

@vburlai Awesome 🔥🔥🔥
Thank you so much 🙌


Cannot ctrl-d on Bone Ring

Item Class: Rings
Rarity: Rare
Brimstone Whorl
Bone Ring
Requirements:
Level: 35
Item Level: 48
Minions have +10% to all Elemental Resistances (implicit)
+29 to Strength
Adds 1 to 2 Physical Damage to Attacks
+37 to maximum Mana
Minions have +13% to all Elemental Resistances

Language is set to English, league is set to Kalandra

Thank you for reporting this issue. It'll be fixed once 3.19 support is release in the next version of poe overlay (v0.8.20)


Tryout the APK getting started guide

Description:
Tryout the getting started guide in [1].
[1] https://github.com/wso2/apk/blob/0.0.1-m1/README.md#getting-started

Running into an error when starting Rancher. Had a chat with Sampath and Amali. Yet to resolve the issue.


Thumbnail test

Testing one of the possible solutions for the thumbnail.

I got a lot of conflicts, please check if I resolved them correctly.


Assign tas final

Proposed Changes
Add functionality to assign TAs to specific sections:
Add a new button to the graders page called "Assign by Section".
When the button is clicked, open a new modal that has a form that displays a list of all course sections, and next to each section is a dropdown with a list of TAs (plus an "empty" option). So, each section can be assigned at most one TA using this form. [If an instructor wants to assign multiple TAs to the same section, they'll need to submit the form multiple times.]
When the form is submitted, the controller assigns the TAs to all groups in their assigned sections. Note that all existing TA assignments remain unchanged; this form only adds new TA assignments.
...

Screenshots of your changes (if applicable)

Associated documentation repository pull request (if applicable)

Type of Change
(Write an X or a brief description next to the type or types that best describe your changes.)

Type
Applies?

🚨 Breaking change (fix or feature that would cause existing functionality to change)

✨ New feature (non-breaking change that adds functionality)

🐛 Bug fix (non-breaking change that fixes an issue)

🎨 User interface change (change to user interface; provide screenshots)

♻ Refactoring (internal change to codebase, without changing functionality)

🚦 Test update (change that only adds or modifies tests)

📦 Dependency update (change that updates a dependency)

🔧 Internal (change that only affects developers or continuous integration)

Checklist
(Complete each of the following items for your pull request. Indicate that you have completed an item by changing the [ ] into a [x] in the raw text, or by clicking on the checkbox in the rendered description on GitHub.)
Before opening your pull request:

[x] I have performed a self-review of my changes.

Check that all changed files included in this pull request are intentional changes.
Check that all changes are relevant to the purpose of this pull request, as described above.

[ ] I have added tests for my changes, if applicable.

This is required for all bug fixes and new features.

[ ] I have updated the project documentation, if applicable.

This is required for new features.

[x] If this is my first contribution, I have added myself to the list of contributors.

After opening your pull request:

[x] I have updated the project Changelog (this is required for all changes).
[x] I have verified that the pre-commit.ci checks have passed.
[x] I have verified that the CI tests have passed.
[x] I have reviewed the test coverage changes reported by Coveralls.
[x] I have requested a review from a project maintainer.

Questions and Comments
(Include any questions or comments you have regarding your changes.)
I can't figure out how to get the right query to filter groupings by sections

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

4 of 29 (13.79%)  changed or added relevant lines in 2 files are covered.
1887 unchanged lines in 61 files lost coverage.
Overall coverage decreased (-39.7%) to 51.827%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

app/assets/javascripts/Components/graders_manager.jsx
2
7
28.57%

app/assets/javascripts/Components/Modals/section_distribution_modal.js
2
22
9.09%

Files with Coverage Reduction
New Missed Lines
%

app/assets/javascripts/Components/graders_manager.jsx
1
37.4%

app/lib/markus_logger.rb
1
71.93%

config/dummy_lti_config.rb
1
66.67%

config/initializers/01_init_app.rb
1
50.0%

spec/factories/groups.rb
1
66.67%

config/initializers/resque.rb
2
83.33%

spec/factories/annotations.rb
2
92.86%

spec/factories/flexible_criteria.rb
2
84.62%

spec/factories/grouping_starter_file_entries.rb
2
80.0%

spec/factories/marking_scheme.rb
2
77.78%

Totals

Change from base Build<PHONE_NUMBER>8:
-39.7%

Covered Lines:
3821

Relevant Lines:
7182

💛  - Coveralls


Add frontend architecture

Ok just checked it out.

Looks good to me.

Other questions about the frontend remain. How does it communicate with the backend? Where is the logic handled? etc

Is it using Om or reagent? Why Om? Why reagent?


Add channel names to Spatial plot

Fixes #1516
Background
See issue description
Change List
Modified SpatialSubscriber.js component to:

Retrieve names and colors of channels, when they exist
Display the channel names at the top right corner of Spatial plot, using the same color for the text as the one used for the respective channel.

See examples of how the changes look like with some imaging demo datasets:

Checklist

[ ] Ensure PR works with all demos on the dev.vitessce.io homepage
[ ] Open (draft) PR's into vitessce-python and vitessce-r if this is a release PR
[ ] Documentation added or updated

At the end, I didn't go for the proposed implementation approach written in the issue descr ("Implement new DeckGL CompositeLayer of TextLayers for the Spatial view."). I tried it, but the text was very difficult to reliably ensure that it will stay in a good position for every different example, also the position of the text is changing as the user is zooming in and out and moving the plot, which seemed annoying.

Thanks, this approach makes sense. Can we position the text on the bottom left instead of top right? (In the top right it might overlap with the gene expression legend)

Thanks, this approach makes sense. Can we position the text on the bottom left instead of top right? (In the top right it might overlap with the gene expression legend)

Yes, positioning to bottom left sounds good. I have done it


Net6.0 versioned MVC API incorrectly applying versioning info in SwaggerUI

When following the examples laid out in the docs I have hit an issue re. swagger UI incorrectly asking for a version to be passed as a parameter:

and when the parameter is passed, it doesn't seem to reflect which versions are exposed by the API; the only controller exposed version is 1.0, but the parameter can be set to any number, resulting in a 200 response received.
I have attached an example bare bones application that reflects this behaviour, I've been struggling with this for hours now and I've tried lots of different ways to resolve it, but the fountain of ideas has definitely run dry.
Versioned.API.zip

Leaving a solution here for anyone else encountering the same issue, I found I was missing a .AddMvc() call between .AddApiVersioning() and .AddApiExplorer(..). As I had a call to AddMvc(...) later in startup I thought it would be ok, but I'm guessing the order these things occur is critical.

Thanks for reporting this. I'm going to categorize it as a 🐞 because it is not what you expected. AddApiExplorer should call AddMvc automatically if you don't. That will eliminate ordering issues, which you are not supposed to have to worry about. This will be fixed in the next patch release.
The example projects should show complete and correct working setups. Please call it out if they are incorrect.


Disable all animations with NoopAnimationsModule

For e2e/screenshot tests, it's nice to be able to disable animations. However, even when installing NoopAnimationsModule, some material animations still run e.g. the loading spinner, the focus animation on text fields.
This is a feature request to have NoopAnimationsModule disable all animations in Material.

@crisbeto I think that dialog needs to be done also, I can't see a PR for that

@benb7760 all the dialog animations are done through Angular which means that the NoopAnimationsModule works automatically already.

Still displaying animation after importing 'NoopAnimationsModule', any solution/alternative could be very welcome.


[FEAT]: IMPROVE SEO BY ADDING IMPORTANT META ELEMENTS 🚀 🔥

Description

The  Scene-Movie-Platform repository currently lacks sufficient meta elements to improve SEO performance.
The existing meta is limited and does not provide search engines with adequate information to understand and rank the website's content effectively.
To enhance the website's visibility and search engine rankings, it is crucial to add essential meta elements.
To address this issue, we need to add additional and important meta elements, such as title tags, meta descriptions, Open Graph tags, Twitter Card tags, structured data markup (Schema.org), canonical tags, and optimized image alt tags.
These additions will improve search engine crawlability, increase organic traffic, and optimize content sharing on social media platforms. By incorporating these necessary meta elements, we can significantly enhance the website's SEO and overall performance.

Hey there @sourabhsikarwar 👋🏻
Please make sure to assign this issue to me to make the work done, Thank You!


Make PrimitiveJob serializable

Summary
In qiskit-ibm-runtime we want to add job methods to our QiskitRuntimeLocalService (Issue https://github.com/Qiskit/qiskit-ibm-runtime/issues/1607). In order to save PrimitiveJob objects locally we need a way to serialize them. It's currently not serializable due to the use of ThreadPoolExecutor:
https://github.com/Qiskit/qiskit/blob/35d0954578d5312a63fb702f29bbd3d47ff25cc5/qiskit/primitives/primitive_job.py#L47-L48
Credit to @t-imamichi
Details and comments
Also fixes #12787

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

15 of 21 (71.43%)  changed or added relevant lines in 1 file are covered.
9 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.008%) to 89.684%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/primitives/primitive_job.py
15
21
71.43%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
3
93.23%

crates/qasm2/src/parse.rs
6
97.15%

Totals

Change from base Build<PHONE_NUMBER>5:
-0.008%

Covered Lines:
67525

Relevant Lines:
75292

💛  - Coveralls

It looks good overall. Could you add a reno to describe that both DataBin and PrimitiveJob become serializable?


self-notifications are displayed

I don't know if this is desired behavior necessarily, but I see notifications when I comment in a thread that I originally started. These notifications feel like they're clogging up what I actually want to be seeing; just what other identities are posting.
Don't want to attach a screenshot b/c don't want to dox, but happy to elaborate more!

This is the correct behavior logically, but I see how it's not great UX. Is your preferred behavior that you only get notifications when anyone who is not one of your identities interacts with any post from your identities?


Tests are an important thing, right?

Began using project fuzzball to write tests for the autocomplete!
Also, ran the new version of elm-format on everything.
Time to hook into travis...
Todos

[ ] Test every branch of the update function
[x] Integrate w/ travis
[ ] Investigate testing views / shallow rendering

I'll make each of these todo's into separate issues/prs


security vulnerability in kind-of dependency

You need to update to Kind-of 6.0.3 or later.
https://github.com/advisories/GHSA-6c8f-qphg-qjgp

Should be fixed in the most recent update.


Tutorial: How to infer stellar parameters from periodograms using asteroseismology?

This notebook will form part of the series: Science Examples - Stellar Physics.
In this fourth notebook, we will focus on the asteroseismic capabilities of Lightkurve contained in the seismology sub-module. The user will have already learned how to prepare data foru se in this sub-module in Tutorial #13.
I wrote a lot of this up in my PhD thesis, so there is alot of material that we can use in this tutorial without having to reinvent the wheel.
We can draw inspiration from existing Lightkurve tutorials, and tutorials written for the TASC5 conference:

How to characterise a Sun-like star using asteroseismology?
Rudy (Kepler) Example
TESS Example
K2 Example

Learning Goals:

How can we use the 2D ACF method to estimate numax and deltanu?
What are the seismic scaling relations?
How do we calculate stellar parameters using Lightkurve and asteroseismology?

Love it! :+1:

I propose naming this tutorial
How to estimate a star's mass and radius using asteroseismology
instead of
How to infer stellar parameters from periodograms using asteroseismology?
to make it sound more catchy!

I completed my review of this notebook.
I did not make fundamental changes, but did take the liberty to attempt to clarify the phrasing in a few places.
The file name is How-to-Estimate-a-Stars-Mass-and-Radius-using-Asteroseismology.ipynb.

@ojhall94 and @barentsen I finished copy editing this notebook "How to Estimate a Star's Mass and Radius Using Asteroseismology" and it was in excellent shape! I made very minor copy edits throughout. This notebook is all set on my end and ready to go!

Thanks @lglattly! :+1:

This notebook has successfully been merged into spacetelescope/notebooks.  Thanks everyone!  Closing the issue...


CAMEL-12552: Sent an error reason in the response body for HTTP 400 e…

…rrors.
Please, check CAMEL-12552 out for more.

Thanks guys, closing.


[Feature] Expose parent route for subroute

It would be very nice to have the parent route exposed into route object for subroutes.
Common use case is allowing to route to parent route (on validation by example) without using history.back() which is a different case.
It would be even better if route vm is exposed into the route object so we can write:
data() {
   this.something = this.$route.$parent.vm.something;
}

I think the first use case can be solved with named routes; the second use case sounds like a bad idea (directly accessing parent data from child views) - instead it's probably better to pass down data explicitly using props, or using some global state management pattern.

Yest, it can be solved with named routes but it is a burden to maintain it manually. The $parent would allow for implementation of a back button that goes one level up in the hierarchy or simple breadcrumbs. It would be awesome to have.
Is there any option to revisit this?
P.S. Since we have access to the $parent on the Vue instance, it would be only logical to have it here as well.

+1, have same need for somewhat complex app gathering routes from separate packages.

The $parent would allow for implementation of a back button that goes one level up in the hierarchy or simple breadcrumbs. It would be awesome to have.

For those coming up against this, I'm solving it thus:
From: https://router.vuejs.org/api/#the-route-object  (scroll to $route#matched

An Array containing route records for all nested path segments of the current route. Route records are the copies of the objects in the routes configuration Array (and in children Arrays):

The $route has a matched property which is an array of route matches (tree-like) with the current route being at the end of the array and thus the parent being second-last element in the array.
So to get any route's parent (better check if it has a parent by length >= 2 ... ?)
this.$route.matched.slice(-2).shift()
Because, per the docs, the element is a 'route record' - it contains name, params, query, meta, etc.


beanstalkd: 1.10 -> 1.11

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/beanstalkd/versions.
meta.description for beanstalkd is: '"A simple, fast work queue"'.
meta.homepage for beanstalkd is: '"http://kr.github.io/beanstalkd/"
Compare changes on GitHub

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
/bin/sh passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 1.11 with grep in /nix/store/jzi7x16vcp7hq3b85rgcvi6zyd29zi0w-beanstalkd-1.11
directory tree listing: https://gist.github.com/26213bb36b92f7db36a34ab295612d73
du listing: https://gist.github.com/de9f0bf44e62f0f780fd3956c79c2379

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

4 total rebuild path(s)
1 package rebuild(s)
1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
beanstalkd

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/jzi7x16vcp7hq3b85rgcvi6zyd29zi0w-beanstalkd-1.11 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A beanstalkd https://github.com/r-ryantm/nixpkgs/archive/dc08a411a8d17bf1c8e89d7a9913873831d66d0d.tar.gz

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

cc @zimbatm for testing.

@GrahamcOfBorg test beanstalkd


Backport "[Issue 15987] Nicer error message in case a derived method has an explicit term param" to LTS

Backports #21332 to the 3.3.5.
PR submitted by the release tooling.
[skip ci]

No regressions detected in the community build up to backport-lts-3.3-21857.
Reference


[bug bash] typo in 55.teams-link-unfurling/README.md

Github issues for C#
Sample information

Sample type: samples
Sample language: dotnetcore
Sample name: 55.teams-link-unfurling

Describe the bug
Typo in the 'Interacting with the bot' section in the README.md file.
Screenshots

fixed


How to destroy an SnmpEngine() object

I have a memory leak problem: when I create several SnmpEngine() objects in my restart loop case, apparently, GC not handles SnmpEngine() memory.
So how can I remove/destroy the previous SnmpEngine() object?
Is there any case like the below code:
snmp_engine = SnmpEngine()
snmp_engine.destroy()

Try this:
snmp_engine.unregisterTransportDispatcher()

Thanks for the response, but it did not solve my memory leak problem.

Also, I tried it snmp_engine.transportDispatcher.closeDispatcher()  but problem was remain.

del(snmp_engine)
seemed to work for me.