Display and From for String

Support turning a cron::Schedule back into a string as discussed in #46. That issue spoke of a to_string() method, but this PR instead implements From<Schedule> for String, which feels more Rust:y.
This is a naive implementation which tries to minimize the change impact while trying to adhere to maintainer's request of returning the original input rather than returning a canonical representation the Schedule. Since cron::Schedule is a pub struct, this is a breaking change, so in order not to force existing consumers to produce a string representation.

Spring cleaning. This was applied upstream in https://github.com/zslayton/cron/pull/54.


Sort Timeseries by Name

I'm submitting a ...

[ ] Bug report
[x] Feature request
[ ] Question / Support request

If you don't send bug report, remove sections below
For bug report please include this information:
Software Versions

Grafana
Zabbix
Grafana-Zabbix Plugin

4.4.3
3.2.6
3.6.1

Description
It would be nice to have a Metric Function that sorts timeseries by Name. this would make graphs much more readable when having multiple series within the same graph

No further work required - i already implemented this.
ill release it within the next few days.

Feature is included in #465

Feature has been merged.

Thanks for the contribution!


add option for clearance in all directions

The existing implementation of clearance only checks for obstacles along one diagonal.  This works for the problem as defined in the reference material, but it is not suitable for situations where one needs to check for clearance in a radius around the walkable point.  To address that need, this PR adds an option to check for clearance in all directions.  This change is backwards-compatible and preserves the existing check as the default behavior.

for those wondering it seems this PR was merged

Apparently not, according to the commit log.  See related comment on PR 63.


Fix issues related to MXCSR register

These are fixes for issues described in #1059.

Thank you @M-HT ! Can you add some tests to demonstrate the fixes?

The first two issues are missing definitions so tests don't make sense, because programs using them fail to compile without the fix.
As for the third issue, I don't know how to demonstrate it.

I'm not sure what meaningful test can be written which uses _MM_ROUND_MASK / _MM_GET_FLUSH_ZERO_MODE.
Also native-aliases.sh script only removes simde prefix and not SIMDE prefix, and it ignores macro "constants" like _MM_ROUND_MASK, _MM_SHUFFLE, ...

How about this test, is it acceptable ?
static int
test_simde_MXCSR (SIMDE_MUNIT_TEST_ARGS) {
  uint32_t original_mxcsr = simde_mm_getcsr();
  uint32_t masked_mxcsr = original_mxcsr & ~(SIMDE_MM_ROUND_MASK | SIMDE_MM_FLUSH_ZERO_MASK);

  simde_mm_setcsr(masked_mxcsr | SIMDE_MM_ROUND_NEAREST | SIMDE_MM_FLUSH_ZERO_OFF);
  uint32_t rm_nearest_off = HEDLEY_STATIC_CAST(uint32_t, SIMDE_MM_GET_ROUNDING_MODE());
  uint32_t fzm_nearest_off = HEDLEY_STATIC_CAST(uint32_t, SIMDE_MM_GET_FLUSH_ZERO_MODE());

  simde_mm_setcsr(masked_mxcsr | SIMDE_MM_ROUND_NEAREST | SIMDE_MM_FLUSH_ZERO_ON);
  uint32_t rm_nearest_on = HEDLEY_STATIC_CAST(uint32_t, SIMDE_MM_GET_ROUNDING_MODE());
  uint32_t fzm_nearest_on = HEDLEY_STATIC_CAST(uint32_t, SIMDE_MM_GET_FLUSH_ZERO_MODE());

  simde_mm_setcsr(masked_mxcsr | SIMDE_MM_ROUND_DOWN | SIMDE_MM_FLUSH_ZERO_OFF);
  uint32_t rm_down_off = HEDLEY_STATIC_CAST(uint32_t, SIMDE_MM_GET_ROUNDING_MODE());
  uint32_t fzm_down_off = HEDLEY_STATIC_CAST(uint32_t, SIMDE_MM_GET_FLUSH_ZERO_MODE());

  simde_mm_setcsr(masked_mxcsr | SIMDE_MM_ROUND_DOWN | SIMDE_MM_FLUSH_ZERO_ON);
  uint32_t rm_down_on = HEDLEY_STATIC_CAST(uint32_t, SIMDE_MM_GET_ROUNDING_MODE());
  uint32_t fzm_down_on = HEDLEY_STATIC_CAST(uint32_t, SIMDE_MM_GET_FLUSH_ZERO_MODE());

  simde_mm_setcsr(masked_mxcsr | SIMDE_MM_ROUND_UP | SIMDE_MM_FLUSH_ZERO_OFF);
  uint32_t rm_up_off = HEDLEY_STATIC_CAST(uint32_t, SIMDE_MM_GET_ROUNDING_MODE());
  uint32_t fzm_up_off = HEDLEY_STATIC_CAST(uint32_t, SIMDE_MM_GET_FLUSH_ZERO_MODE());

  simde_mm_setcsr(masked_mxcsr | SIMDE_MM_ROUND_UP | SIMDE_MM_FLUSH_ZERO_ON);
  uint32_t rm_up_on = HEDLEY_STATIC_CAST(uint32_t, SIMDE_MM_GET_ROUNDING_MODE());
  uint32_t fzm_up_on = HEDLEY_STATIC_CAST(uint32_t, SIMDE_MM_GET_FLUSH_ZERO_MODE());

  simde_mm_setcsr(masked_mxcsr | SIMDE_MM_ROUND_TOWARD_ZERO | SIMDE_MM_FLUSH_ZERO_OFF);
  uint32_t rm_zero_off = HEDLEY_STATIC_CAST(uint32_t, SIMDE_MM_GET_ROUNDING_MODE());
  uint32_t fzm_zero_off = HEDLEY_STATIC_CAST(uint32_t, SIMDE_MM_GET_FLUSH_ZERO_MODE());

  simde_mm_setcsr(masked_mxcsr | SIMDE_MM_ROUND_TOWARD_ZERO | SIMDE_MM_FLUSH_ZERO_ON);
  uint32_t rm_zero_on = HEDLEY_STATIC_CAST(uint32_t, SIMDE_MM_GET_ROUNDING_MODE());
  uint32_t fzm_zero_on = HEDLEY_STATIC_CAST(uint32_t, SIMDE_MM_GET_FLUSH_ZERO_MODE());

  simde_mm_setcsr(original_mxcsr);

  simde_assert_equal_u32(rm_nearest_off, rm_nearest_on);
  simde_assert_equal_u32(rm_down_off, rm_down_on);
  simde_assert_equal_u32(rm_up_off, rm_up_on);
  simde_assert_equal_u32(rm_zero_off, rm_zero_on);

  simde_assert_equal_u32(fzm_nearest_off, fzm_down_off);
  simde_assert_equal_u32(fzm_nearest_off, fzm_up_off);
  simde_assert_equal_u32(fzm_nearest_off, fzm_zero_off);

  simde_assert_equal_u32(fzm_nearest_on, fzm_down_on);
  simde_assert_equal_u32(fzm_nearest_on, fzm_up_on);
  simde_assert_equal_u32(fzm_nearest_on, fzm_zero_on);

  return 0;
}

Also native-aliases.sh script only removes simde prefix and not SIMDE prefix, and it ignores macro "constants" like _MM_ROUND_MASK, _MM_SHUFFLE, ...

Oh, I didn't see that. It would be good to add those SIMDE_ prefixed to the script, but I can see how that is a bit tricky with the private SIMDE_ constants and other defines..

How about this test, is it acceptable ?

I like them, thanks! Can you add them to this PR?

I looked at the how to fix the failing tests and noticed following:
In xmmintrin.h (from gcc, clang and visual studio), functions _mm_getcsr and _mm_setcsr use unsigned int datatype and _MM_GET_ROUNDING_MODE, _MM_SET_ROUNDING_MODE, _MM_GET_FLUSH_ZERO_MODE, _MM_SET_FLUSH_ZERO_MODE are either macros using _mm_getcsr and _mm_setcsr or functions using unsigned int datatype.
In simde, functions simde_mm_getcsr, simde_mm_setcsr, SIMDE_MM_GET_FLUSH_ZERO_MODE, SIMDE_MM_SET_FLUSH_ZERO_MODE use uint32_t datatype (which is an understandable change), but functions SIMDE_MM_GET_ROUNDING_MODE and SIMDE_MM_SET_ROUNDING_MODE use unsigned int datatype.
I think functions SIMDE_MM_GET_ROUNDING_MODE and SIMDE_MM_SET_ROUNDING_MODE should also use uint32_t datatype (to be consistent). Then fixing the failing tests would be simple.

I think functions SIMDE_MM_GET_ROUNDING_MODE and SIMDE_MM_SET_ROUNDING_MODE should also use uint32_t datatype (to be consistent). Then fixing the failing tests would be simple.

Makes sense to me, please implement that; thank you!

@M-HT Do you have time to file the bug report with emscripten/llvm? The emscripten people are pretty fast at fixing things
Directions: https://emscripten.org/docs/getting_started/bug_reports.html
Error log: https://github.com/simd-everywhere/simde/actions/runs/6172090486/job/16751643532?pr=1060#step:8:1076

I created this issue: emscripten-core/emscripten#20250


Support dynamically obtaining the capture count from Regex

extension RegexComponent {
    public var captureCount: Int { get }
}

Would this be on RegexComponent or Regex? I'm not sure when/where you'd want to ask in a generic context where it would be unwieldy to ask the var regex on the component, and it seems a little weird for it to appear on conforming types such as String.

I think RegexComponent, because any regex component can be used as a standalone regex to match some string (by calling String.wholeMatch, etc).

it seems a little weird for it to appear on conforming types such as String.

Would it still be weird if we named it regexCaptureCount instead?

Do we want 2 protocols? One that will be sufficient for generic algorithms and the other that will have a bunch of API dumped on it?

Would it still be weird if we named it regexCaptureCount instead?

It's a little weird if, without importing anything, code completion on a string produces regexCaptureCount. We can talk about hiding it away I suppose, but it's weird to have a slew of regex-specific API on all of these types.


feat: port Logic.Equiv.Fin

I'm going to merge #1741 into this branch, then fix errors.

Oops, I messed up the rebase. Will push a fix asap.

Thanks :tada:
bors merge


Added changes to other variable files. Succeeding PR #1778

After merging 1864, cannot accept this PR since this becomes outdated
Instead added it manually here https://github.com/GeekyAnts/NativeBase/commit/ac65694ec0d59f804ec40b42967f9236ab5f946c


1.3.2 Create work breakdown structure

Tasks:

[x] Create work breakdown structure

Summary:
created WBS based on Gantt chart
Duration: 1 day


[Bug][InnoDB] Setting up InnoDB cluster as Devlake DB

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
Hi all , we tried configuring InnoDB cluster as Devlake DB. Met with an issue , few tables dont have primary keys in them . Group Replication has a requirement to have primary key in them .
Error shown -
ERROR: The following tables do not have a Primary Key or equivalent column:
lake._devlake_locking_stub, lake._tool_teambition_connections, lake._tool_bitbucket_pipeline_steps

Group Replication requires tables to use InnoDB and have a PRIMARY KEY or PRIMARY KEY Equivalent (non-null unique key). Tables that do not follow these requirements will be readable but not updateable when used with Group Replication. If your applications make updates (INSERT, UPDATE or DELETE) to these tables, ensure they use the InnoDB storage engine and have a PRIMARY KEY or PRIMARY KEY Equivalent.
If you can't change the tables structure to include an extra visible key to be used as PRIMARY KEY, you can make use of the INVISIBLE COLUMN feature available since 8.0.23: https://dev.mysql.com/doc/refman/8.0/en/invisible-columns.html

Checking instance configuration...

{
    "status": "error"
}

What do you expect to happen
InnoDB cluster should have recognised devlake mysql instances .
How to reproduce

Deploy devlake with a single mysql instance.
Deploy mysql-operator using helm-chart. (link)
Deploy InnoDB cluster using helm-chart. (link)
Try validating mysql instance of step-1 for innodb cluster .(Used - dba.checkInstanceConfiguration('Mysql instance'))

Anything else
We can bypass this issue by adding primary key to these tables for now .But it may culminate to become bigger problem when all datas are there on devlake and we try to upgrade Devlake to a higher version .
Version
v0.19.0-beta2
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

@pratiyush05  Confirmed, it's a bug, I'm working on it, and it will be fix in v0.20.0-beta*.

@pratiyush05 I have fixed missing primary keys in these table. Can you tell me your operations? When you meet this problem(bootstrap devlake or scale your database)?

Hi @d4x1 , we met this when we were trying to set up devlake with InnoDB cluster as its DB for the first time. Group Replication property of InnoDB cluster needs Primary Keys for its functioning .
We faced these compatibility issue -

We tried to include mysql instance to InnoDB cluster . Mysql Inst. was already having tables and old datas .Group replication was failing on it .
We later tried to first deploy InnoDB and then allow Devlake to create its tables . In this case , devlake couldnot create few of its tables ( cicd_deployment_commits table among few others ).

@pratiyush05  situation 2,cicd_deployment_commits has primary keys, so it should be created successfully theretically.
Can you paste some useful logs to help us to debug this issue?
I am afraid my fix cannot work if devlake cannot init its tables in innodb cluster, but my fix should works on situation 1.

@d4x1
I re-preformed step 2 . I dont see any warning or error while performing it .
Steps -

Installed mysql innodb along with mysql operator .
Install devlake v0.19.0-beta2 with innodb service .

I captured few logs -
Migration started on install devlake in lake container -

After completion , found few domain layer tables missing -

@pratiyush05  db migration is triggered in Config UI, have you triggered it manually? (just to make sure.)

@pratiyush05  ok.
I test it with a new inno db database(not a cluster), and all tables are inited without trigger manually.
And I think there is something wrong with you db cluster.  Does devlake connect to the master instance(read and write)?

Yes Devlake connects with InnoDB cluster via its primary instance but it cannot write all tables and remains stuck in migration indefinitely .

@pratiyush05 Can you share the status or screenshot of the config-ui?

Sure ,
Status -
status:
  phase: Running
  conditions:
    - type: Initialized
      status: 'True'
      lastProbeTime: null
      lastTransitionTime: '2023-11-24T09:22:06Z'
    - type: Ready
      status: 'True'
      lastProbeTime: null
      lastTransitionTime: '2023-11-24T09:22:13Z'
    - type: ContainersReady
      status: 'True'
      lastProbeTime: null
      lastTransitionTime: '2023-11-24T09:22:13Z'
    - type: PodScheduled
      status: 'True'
      lastProbeTime: null
      lastTransitionTime: '2023-11-24T09:22:06Z'
  hostIP: X.X.X.X
  podIP: X.X.X.X
  podIPs:
    - ip: X.X.X.X
  startTime: '2023-11-24T09:22:06Z'
  containerStatuses:
    - name: config-ui
      state:
        running:
          startedAt: '2023-11-24T09:22:13Z'
      lastState: {}
      ready: true
      restartCount: 0
      image: devlake.docker.scarf.sh/apache/devlake-config-ui:v0.19.0-beta2
      imageID: >-
        docker-pullable://devlake.docker.scarf.sh/apache/devlake-config-ui@sha256:15fb0426d0a81c8c69e1852241449b8279d8e138323ff7085baf0dd9bf4ee161
      containerID: >-
        docker://e4c0861ff29a5acb4f3a4d85c339e044d61c7ea56787a90d0a2dd1565b843745
      started: true
  qosClass: BestEffort

Screenshot of config-ui -

Hi , just asking another small issue of migration only -
Parallely we started trying  another HA mode of mysql for Devlake. And we came across this bitnami mysql chart. On using it for devlake, we faced something strange -
2023-11-24T09:26:31.278866932Z Version:  v0.19.0-beta2@
2023-11-24T09:26:31.292442173Z panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0x118d198]
2023-11-24T09:26:31.292453882Z
2023-11-24T09:26:31.292457310Z goroutine 1 [running]:
2023-11-24T09:26:31.292461198Z database/sql.(*DB).Close(0x0)

Devlake is also stuck in migration in here .
Devlake version used - v0.19.0.-beta2
If this innodb cluster issue gets resolve first , we will surely use it over this bitnami one .
If this needs to be raised as a separate issue , I will do the need full .

@pratiyush05

What does this front-end interface look like? Can it be displayed normally?

I want to see which step of migration execution went wrong or got stuck. Can you give me the screenshot, like this:

On browser I am seeing this -

On doing "Proceed Database migration" it is showing -

@pratiyush05  Add a custom_field6 field in the _tool_tapd_stories table with type text, then restart and perform the migration operation again.

If you still have problems with the above, you can also do this：

INSERT INTO _devlake_migration_history (created_at, script_version, script_name, comment)
VALUES
('2023-11-24 07:39:01.113',<PHONE_NUMBER>0004, X'6D6F64696679207461706420637573746F6D206669656C64206E616D65', X'74617064');
DROP TABLE IF EXISTS _tool_tapd_stories;
CREATE TABLE _tool_tapd_stories (
connection_id bigint unsigned NOT NULL,
id bigint NOT NULL,
workitem_type_id bigint unsigned DEFAULT NULL,
name varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
description longtext COLLATE utf8mb4_bin,
workspace_id bigint unsigned DEFAULT NULL,
creator varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
created datetime(3) DEFAULT NULL,
modified datetime(3) DEFAULT NULL,
status varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
owner varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
cc varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
begin datetime(3) DEFAULT NULL,
due datetime(3) DEFAULT NULL,
size smallint DEFAULT NULL,
priority varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
developer varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
iteration_id bigint unsigned DEFAULT NULL,
test_focus varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
type varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
source varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
module varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
version varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
completed datetime(3) DEFAULT NULL,
category_id bigint DEFAULT NULL,
path varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
parent_id bigint unsigned DEFAULT NULL,
children_id text COLLATE utf8mb4_bin,
ancestor_id bigint unsigned DEFAULT NULL,
business_value varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
effort float DEFAULT NULL,
effort_completed float DEFAULT NULL,
exceed float DEFAULT NULL,
remain float DEFAULT NULL,
release_id bigint unsigned DEFAULT NULL,
confidential varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
templated_id bigint unsigned DEFAULT NULL,
created_from varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
feature varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
std_status varchar(20) COLLATE utf8mb4_bin DEFAULT NULL,
std_type varchar(20) COLLATE utf8mb4_bin DEFAULT NULL,
url varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
attachment_count smallint DEFAULT NULL,
has_attachment varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
bug_id bigint unsigned DEFAULT NULL,
follower varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
sync_type text COLLATE utf8mb4_bin,
predecessor_count smallint DEFAULT NULL,
is_archived varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
modifier varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
progress_manual varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
successor_count smallint DEFAULT NULL,
label varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
custom_field_one text COLLATE utf8mb4_bin,
custom_field_two text COLLATE utf8mb4_bin,
custom_field_three text COLLATE utf8mb4_bin,
custom_field_four text COLLATE utf8mb4_bin,
custom_field_five text COLLATE utf8mb4_bin,
custom_field_six text COLLATE utf8mb4_bin,
custom_field_seven text COLLATE utf8mb4_bin,
custom_field_eight text COLLATE utf8mb4_bin,
custom_field_9 text COLLATE utf8mb4_bin,
custom_field_10 text COLLATE utf8mb4_bin,
custom_field_11 text COLLATE utf8mb4_bin,
custom_field_12 text COLLATE utf8mb4_bin,
custom_field_13 text COLLATE utf8mb4_bin,
custom_field_14 text COLLATE utf8mb4_bin,
custom_field_15 text COLLATE utf8mb4_bin,
custom_field_16 text COLLATE utf8mb4_bin,
custom_field_17 text COLLATE utf8mb4_bin,
custom_field_18 text COLLATE utf8mb4_bin,
custom_field_19 text COLLATE utf8mb4_bin,
custom_field_20 text COLLATE utf8mb4_bin,
custom_field_21 text COLLATE utf8mb4_bin,
custom_field_22 text COLLATE utf8mb4_bin,
custom_field_23 text COLLATE utf8mb4_bin,
custom_field_24 text COLLATE utf8mb4_bin,
custom_field_25 text COLLATE utf8mb4_bin,
custom_field_26 text COLLATE utf8mb4_bin,
custom_field_27 text COLLATE utf8mb4_bin,
custom_field_28 text COLLATE utf8mb4_bin,
custom_field_29 text COLLATE utf8mb4_bin,
custom_field_30 text COLLATE utf8mb4_bin,
custom_field_31 text COLLATE utf8mb4_bin,
custom_field_32 text COLLATE utf8mb4_bin,
custom_field_33 text COLLATE utf8mb4_bin,
custom_field_34 text COLLATE utf8mb4_bin,
custom_field_35 text COLLATE utf8mb4_bin,
custom_field_36 text COLLATE utf8mb4_bin,
custom_field_37 text COLLATE utf8mb4_bin,
custom_field_38 text COLLATE utf8mb4_bin,
custom_field_39 text COLLATE utf8mb4_bin,
custom_field_40 text COLLATE utf8mb4_bin,
custom_field_41 text COLLATE utf8mb4_bin,
custom_field_42 text COLLATE utf8mb4_bin,
custom_field_43 text COLLATE utf8mb4_bin,
custom_field_44 text COLLATE utf8mb4_bin,
custom_field_45 text COLLATE utf8mb4_bin,
custom_field_46 text COLLATE utf8mb4_bin,
custom_field_47 text COLLATE utf8mb4_bin,
custom_field_48 text COLLATE utf8mb4_bin,
custom_field_49 text COLLATE utf8mb4_bin,
custom_field_50 text COLLATE utf8mb4_bin,
created_at datetime(3) DEFAULT NULL,
updated_at datetime(3) DEFAULT NULL,
_raw_data_params varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
_raw_data_table varchar(255) COLLATE utf8mb4_bin DEFAULT NULL,
_raw_data_id bigint unsigned DEFAULT NULL,
_raw_data_remark longtext COLLATE utf8mb4_bin,
PRIMARY KEY (connection_id,id),
KEY idx__tool_tapd_stories_modified (modified),
KEY idx__tool_tapd_stories_raw_data_params (_raw_data_params)
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_bin;

Then restart and perform the migration operation again.

@abeizn  Why should users run such sql scripts?

@abeizn Why should users run such sql scripts?

Yes, I'm not sure what happened before that caused the database field to be deleted. It can only be modified manually. Or just delete the database and redeploy it.

Thanks @abeizn , it worked after I redeployed after re-creating the table .
For debugging , it shows ALTER TABLE command is run which finds unknown custom_field6 as shown in this picture above -

and the migration code linked to it is - link
but this code seems to be linked to table "_tool_tapd_tasks" only not "_tool_tapd_stories". Is this an intended workflow ?

@pratiyush05  https://github.com/apache/incubator-devlake/blob/0f990b109c52dfaf7103405f043d449452a751e8/backend/plugins/tapd/models/migrationscripts/20230411_modify_custom_field_name.go#L42.
There is a fallthrough here, so that when issuesName is "_tool_tapd_stories", the program will continue to execute the case of "_tool_tapd_tasks"

Any update in this ? Why migration is stucking for InnoDB cluster ?

@pratiyush05  I'm not sure why it crashed. Maybe there were changes in the database, maybe the migration failed during execution, or maybe a developer changed this value.


inference speed

720P 93 model, A800 8GPU, 55s one step. One video takes 1.5 hours, is that a normal time

bash scripts/text_condition/gpu/sample_t2v_ddp.sh

same question. how did you fix it ?

I wonder 93*480p model can inference videos less than 93 frames?


Replace remaining Helper.log calls with UI. ones.

Replace remaining Helper.log calls with UI. ones

This looks great, please squash the commits and then it's ready to be merged :tada:

@KrauseFx done

Perfect, thanks :+1:


Folder structure

@chemardes and @debdal6, I propose the following structure for this project:

A playground/ folder where we'll be doing experimental things
A pdesolvers/ (or however you would like to call it - find a fancy name) for your Python code; this will be a Python library
A LICENSE file (discuss with each other and agree on the licence you want to use)
A gpu/ folder with the GPU code
A .gitignore file (for Python and C/C++)
A .github/ folder because we need to set up GitHub actions for continuous integration; you can also put there issue tempates
A CHANGELOG.md (using SemVer)
A main README.md file

@chemardes so in playground we place all our prototype python scripts which is almost all the scripts we have at the moment
fancyLibraryName will be the folder where we place our packaged python libraries to be published


Non fixed EsfMeas [DEVINFRA-872]

I think this one need to be more like NavSat - https://github.com/swift-nav/ublox/blob/c3094069679b6341c35f6c01888559e5ed333cd3/ublox/src/ubx_packets/packets.rs#L596
EsfMeas should have a method that returns an iterator of EsfMeasInfoRefs, like how NavSat has an iterator that returns NavSatSvInfoRefs

I think this one probably needs to be more like NavSat -
https://github.com/swift-nav/ublox/blob/c3094069679b6341c35f6c01888559e5ed333cd3/ublox/src/ubx_packets/packets.rs#L596
EsfMeas should have a method that returns an iterator of EsfMeasInfoRefs, like how NavSat has an iterator that returns NavSatSvInfoRefs
but maybe I'm not understanding it correctly

yea... but the thing is they decided to not put the iterator at the end of the packet rather in the second last making the last u32 optional which is derived from the flag so had to somehow retrieve the flags as well...
NavSat has it at the end of the msg so all is well, they could just continuously iterate without actually getting numSvs
or maybe im doing this wrong not actually sure at this point, it is so weird.

or maybe im doing this wrong not actually sure at this point, it is so weird like NavSatSvInfoRef struct doesn't actually exist

It's generated by the macro so it exists but not in the source code - it can be super helpful to see the output
cargo install cargo-expand
cd ./ublox
cargo expand >out.rs

will expand everything then you can search for NavSatSvInfoRef

Can you use the numMeas part of the flags to do this?
pub struct EsfMeasInfo<'a> {
    flags: u16,
    id: u16,
    data: EsfMeasInfoDataIter<'a>,
    calib_tag: Option<u32>,
}

struct EsfMeasInfoDataIter<'a>(core::slice::ChunksExact<'a, u8>);

impl<'a> EsfMeasInfoDataIter<'a> {
    fn new(data: &'a [u8]) -> Self {
        EsfMeasInfoDataIter(data.chunks_exact(4))
    }
}

impl<'a> core::iter::Iterator for EsfMeasInfoDataIter<'a> {
    type Item = u32;

    fn next(&mut self) -> Option<Self::Item> {
        Some(u32::from_le_bytes(self.0.next()?.try_into().unwrap()))
    }
}

impl<'a> EsfMeasInfo<'a> {
    fn new(bytes: &'a [u8]) -> Self {
        let flags = u16::from_le_bytes(bytes[0..2].try_into().unwrap());
        let id = u16::from_le_bytes(bytes[2..4].try_into().unwrap());
        let num_meas = 0; // some bit magic here with flags
        let data = EsfMeasInfoDataIter::new(&bytes[4..num_meas * 4]);
        let calib_tag = if flags & 0x8 != 0 {
            Some(u32::from_le_bytes(bytes[num_meas * 4 + 4..num_meas * 4 + 8].try_into().unwrap()))
        } else {
            None
        };
        Self {
            flags,
            id,
            data,
            calib_tag,
        }
    }
}

Can you use the numMeas part of the flags to do this?

yea... been trying to avoid making an iterator tho so easier for serialization (instead of nested things), made U32Iterator, works ok

yea... been trying to avoid making an iterator tho so easier for serialization (instead of nested things, guess we can't really avoid nesting either ways), made U32Iterator, works ok

yeah iterators are a little inconvenient to serialize with serde, because serialize takes a shared reference but .next() needs a unique/mutable one. I don't think it will be too bad (especially because you're generating most of it) but you could add a helper like this to make the serialization easier
struct SerializeIter<T>(core::cell::RefCell<T>);

impl<T> serde::Serialize for SerializeIter<T>
where
    T: Iterator,
    T::Item: serde::Serialize,
{
    fn serialize<S>(&self, serializer: S) -> Result<S::Ok, S::Error>
    where
        S: serde::Serializer,
    {
        serializer.collect_seq(self.0.borrow_mut().by_ref())
    }
}

then you SerializeIter(packet.iterator_field()).serialize(serializer)?;. the refcell lets us get a &mut from a &, at the cost of doing the borrow checking at run time. should be very small, and only effects the serde integration


[TypeScript]: Need to (F12/Goto definition) twice on destructured imports to get to actual definition

VSCode Version: 1.2.1
OS Version: Windows 8.1 x64

Steps to Reproduce:

Import a module member using destructuring syntax
F12 / Goto definition on the destructured import symbol

The first F12 takes you to the destructuring expression. The second F12 then takes you to the actual definition
See attached gif for an example

Possibly also related is that tooltips on destructured import symbols show the import statement instead of its definition

Compare with non-destructured version

Moving to TS team.

This issue was moved to Microsoft/TypeScript#9278


Not convert 0x302e31 bytes to uint

when i convert "0.1" string to bytes and then convert bytes to uint it's not converting. Is there any other way where i can convert "0.1" string to 0.1 uint value. plz help

0.1 cannot be represented as a uint... so I suspect that's the problem.

You should try the ethereum stack exchange site.  It's a good place to get answers to questions like this.


ellie (or erlang ssl) not sending intermediate certs - breaks the ssl chain.

I have setup a very simple Elli ssl server here:  https://paperlesssolutionsltd.com.ng:333
via https://www.sslshopper.com/ssl-checker.html#hostname=https://paperlesssolutionsltd.com.ng:333 I can see that the ssl chain is broken:

Please what am i missing in my setup?
def start_link(args) do
    :elli.start_link(
      port: 333,
      certfile: Application.app_dir(@app, "priv/cert.pem"),
      keyfile: Application.app_dir(@app, "priv/cert.key"),
      ssl: true,
      callback: __MODULE__,
      callback_args: args,
      max_body_size: 1_073_741_824
    )
end

was an issue with the Erlang version.
Upgrading to 23.2.4 solved it
https://elixirforum.com/t/making-ssl-tests-all-pass-for-phoenix-lets-encrypt/3507/30?u=charleso

Oh, good :)


Koios 1.2.0 alpha

Description

To-Do:

[x] update epoch_params section to include new governance params
[ ] Add deposit field to account_info
[x] Add delegated_drep field to account_info
[ ] Introduce governance section (add compatibility with BF endpoints for redundancy from lib providers pov)

[ ] /action_list
[ ] /action_info
[ ] /committee
[ ] /committee_votes
[ ] /drep_list
[ ] /drep_delegators
[ ] /drep_info
[ ] /drep_metadata
[ ] /drep_updates
[ ] /drep_votes
[ ] /proposal
[ ] /proposal_metadata
[ ] /proposal_parameters
[ ] /proposal_votes
[ ] /pool_votes
[ ] /committee_members

[x] Make bytecode optional in tx_info, closes #293
[x] Update tx_info to include various toggles in input (similar to block_tx_info , and in addition _bytecode)
[x] Update block_tx_info to include _bytecode toggle
[x] Update block_tx_info and tx_info to include treasury_donation field
[x] Add tx_cbor endpoint
[ ] Add tx_cbor2json endpoint (?)

Does it make sense to also add the _bytecode toggle to the asset_utxos and address_utxos endpoints? Right now, the bytecode is returned for all reference scripts. I don't ever need the bytecode from these endpoints either, but omitting them isn't as important to me as it is for the tx_info endpoints. I'm fine with whatever makes maintaining asset_utxos and address_utxos easier for you.


Linux branch’ binary is broken

./nheqminer_cpu -u t1JyivjnNwn35vsSYqeTQTyw1zH3DdHdM7q -t 6 -p x -l stratum.zcash.nicehash.com:3357
gives me

[16:34:55][0x00007f69baffd700] miner#5 | Starting thread #5 (CPU-XENONCAT-AVX2)
[16:34:55][0x00007f69c20c1700] stratum | resolve: Host not found (authoritative)
[16:34:55][0x00007f69c20c1700] stratum | Reconnecting in 3 seconds...
[16:34:58][0x00007f69c20c1700] stratum | Connecting to stratum server stratum.zcash.nicehash.com:3357:3357
[16:34:58][0x00007f69c20c1700] stratum | resolve: Host not found (authoritative)
[16:34:58][0x00007f69c20c1700] stratum | Reconnecting in 3 seconds...
[16:35:01][0x00007f69c20c1700] stratum | Connecting to stratum server stratum.zcash.nicehash.com:3357:3357

this has already been fixed, pull the updates ;)

@buzztiaan just did, still get

[16:41:26][0x00007fc7f4fa1700] stratum | Connecting to stratum server stratum.zcash.nicehash.com:3357:3357
[16:41:26][0x00007fc7edffb700] miner#5 | Starting thread #5 (CPU-XENONCAT-AVX2)
[16:41:26][0x00007fc7f4fa1700] stratum | resolve: Host not found (authoritative)

This is my first time using the miner and I'm getting this too.

Ok, now it works, just pulled again :D Has been fixed a couple of mins ago 👍

okay, I've just pulled, I'll compile again. Thanks!

Thank you. All good now after re-compile.


Add Ajayan Borys to list of Docs approvers

@ameowlia Hi. I'm a Tech Writer at VMware and just joined our CloudFoundry docs team. Thanks for your help!

I have successfully gone through the Easy CLA authorization. Not sure why it still shows as missing.

@rkoster, informally, I have been holding a different standard for new approvers for SWE v docs for the following reasons:

The reviewer requirements were written with software engineers in mind, not docs contributors.
The docs repos are separate from code repos.
Our docs teams have been extremely understaffed recently and are only recently beginning to be staffed.
Given their extreme understaffed-ness, there exist very few people who can approve PRs. Until recently, the number was practically 0 for several areas (there were people listed but they were no longer actually contributing).
Ultimately it is up to the WG leads to nominate an approver.

For those reasons I am happy to approve @HenryBorys's addition.
In the future we could...

Document an official different process for docs members.
Do nothing and keep handling them as one offs.

@rkoster, @Gerg -  I'm curious to hear your thoughts on the matter.


Build error with

This commit... https://github.com/shamazmazum/cl-forward-diff/commit/d66ce3fd4d2df5e861ce4544cbbf7da913e9d721
; caught ERROR:
;   READ error during COMPILE-FILE:
;
;     Lock on package SB-C violated when interning *COMPILATION-UNIT* while in
;     package CL-FORWARD-DIFF.

https://github.com/ocicl/cl-forward-diff/actions/runs/11231598146/job/31221366229

Indeed it is...
$ ocicl list cl-forward-diff
cl-forward-diff(ghcr.io/ocicl):
 latest
 20241008-7804948
 20241003-0d6b3fa
 20240728-9ce122a


Add configmap watcher to watch configmap changes

Changes
Fixes : https://github.com/openshift-pipelines/pipelines-as-code/issues/603

Submitter Checklist

[ ] ♽  Run make test lint before submitting a PR (ie: with pre-commit, no need to waste CPU cycle on CI
[ ] 📖 If you are adding a user facing feature or make a change of the behavior, please verify that you have documented it
[ ] 🧪 100% coverage is not a target but most of the time we would rather have a unit test if you make a code change.
[ ] 🎁 If that's something that is possible to do please ensure to check if we can add a e2e test.
[ ] 🔎 If there is a flakiness in the CI tests then don't necessary ignore it, better get the flakyness fixed before merging or if that's not possible there is a good reason to bypass it. (token rate limitation may be a good reason to skip).

/retest

Codecov Report

Merging #785 (bdace40) into main (85114fb) will decrease coverage by 0.31%.
The diff coverage is 25.00%.

@@            Coverage Diff             @@
##             main     #785      +/-   ##
==========================================
- Coverage   66.01%   65.69%   -0.32%
==========================================
  Files          75       75
  Lines        4569     4530      -39
==========================================
- Hits         3016     2976      -40
- Misses       1269     1270       +1
  Partials      284      284

Impacted Files
Coverage Δ

pkg/reconciler/controller.go
0.00% <0.00%> (ø)

pkg/cmd/tknpac/resolve/resolve.go
67.39% <100.00%> (ø)

pkg/random/random.go
100.00% <0.00%> (ø)

pkg/sync/semaphore.go
100.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

/retest


More logging for MPD addon

Which addon?
MPD
Is your feature request related to a problem? Please describe.
MPD addon starts and stops without any errors in the log (except the RTIOThread one).
Describe the solution you'd like
More logging, so I can hopefully figure out what's going on.
Describe alternatives you've considered
Updated to the latest version (1.4.0); Restarted HA; Uninstalled & re-installed MPD addon. Issue persists.
Additional context
My log:
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] mpd-permissions: applying...
[fix-attrs.d] mpd-permissions: exited 0.
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] setup-folders: executing...
[11:13:13] INFO: Setup Folders
[cont-init.d] setup-folders: exited 0.
[cont-init.d] setup-settings: executing...
[11:13:13] INFO: Setup settings
[cont-init.d] setup-settings: exited 0.
[cont-init.d] done.
[services.d] starting services
[11:13:13] INFO: Start MPD
[services.d] done.
[cmd] /bin/ash exited 0
[cont-finish.d] executing container finish scripts...
[cont-finish.d] done.
[s6-finish] waiting for services.
exception: RTIOThread could not get realtime scheduling, continuing anyway: sched_setscheduler failed: Operation not permitted
[s6-finish] sending all processes the TERM signal.

System info
Raspberry Pi 4
Home Assistant 0.116.4
HassOS 5.2 (dev branch; was needed to support boot-from-usb)
Supervisor 249

A verbose option will be added with version 1.4.1. For more custom settings the custom config file is also an option.

Will be delayed, due a build system failure....


[FEATURE] Add isMake property to OTC trade model

Is your feature request related to a problem? Please describe.
A classification is needed to determine whether a transaction is Make or Take.
Describe the solution you'd like
Add isMake property to OTC trade model

my fault X(((((


Improve mapping display when multiple properties from one spec are mapped to spine

This, I believe is the result of mapping two properties from CEDS  (i.e. Organization Identification System and Organization Identifier) to one property in the spine.

It is really not clear that there are two names and two definitions on the CEDS side.  It gets worse when there are more propertties than two, e.g.

Using a linebreak separator would help (if that is a quick fix), however for most mappings each property should be in it's own field. (the possible exception is a "transform" mapping which aggregates several properties to one, but that is a bit of a niche case)

[ ] when there are several properties from a single schema mapped to the spine they each need to be shown separately each on their own "card", i.e. just as if they came from different schema (though of course they will have the same org, schema name etc.)

Checked on PhilQA, with CTDL Organization by mapping name and alternateName both to name in the spine

Looks much better, moving to QAPasses


Added Databricks support to MLflow Callback

Added a quick check to make integration easier with Databricks; another option would be to make a new class, but this seemed more straightfoward.
cc: @liangz1 Can this be done in a more straightfoward way?

@rithwik-db @liangz1 lemme know when ready for review

@baskaryan can you please take a look at this PR; It should be ready for review. Thank you!

makes sense to me, thanks @rithwik-db @liangz1!


Exp10.hpp msvc issue

I think the little adaptor isnt quite right on windows, the specializations dont exist
    TestExp.cpp
C:\build\PothosSDR_vc16\Source\PothosComms\math\Exp10.hpp(24,18): error C2039: 'exp10f': is not a member of '`global namespace'' (compiling source file C:\build\PothosSDR_vc16\Source\PothosComms\math\TestExp.cpp) [C:\build\PothosSDR_vc16\Build\PothosComms\math\MathBlocks.vcxproj] [C:\build\PothosSDR_vc16\PothosComm
s.vcxproj]
C:\build\PothosSDR_vc16\Source\PothosComms\math\Exp10.hpp(24,1): error C3861: 'exp10f': identifier not found (compiling source file C:\build\PothosSDR_vc16\Source\PothosComms\math\TestExp.cpp) [C:\build\PothosSDR_vc16\Build\PothosComms\math\MathBlocks.vcxproj] [C:\build\PothosSDR_vc16\PothosComms.vcxproj]
C:\build\PothosSDR_vc16\Source\PothosComms\math\Exp10.hpp(24,18): error C2039: 'exp10f': is not a member of '`global namespace'' (compiling source file C:\build\PothosSDR_vc16\Source\PothosComms\math\Exp.cpp) [C:\build\PothosSDR_vc16\Build\PothosComms\math\MathBlocks.vcxproj] [C:\build\PothosSDR_vc16\PothosComms.vc
xproj]
C:\build\PothosSDR_vc16\Source\PothosComms\math\Exp10.hpp(30,18): error C2039: 'exp10': is not a member of '`global namespace'' (compiling source file C:\build\PothosSDR_vc16\Source\PothosComms\math\TestExp.cpp) [C:\build\PothosSDR_vc16\Build\PothosComms\math\MathBlocks.vcxproj] [C:\build\PothosSDR_vc16\PothosComms
.vcxproj]
C:\build\PothosSDR_vc16\Source\PothosComms\math\Exp10.hpp(24,1): error C3861: 'exp10f': identifier not found (compiling source file C:\build\PothosSDR_vc16\Source\PothosComms\math\Exp.cpp) [C:\build\PothosSDR_vc16\Build\PothosComms\math\MathBlocks.vcxproj] [C:\build\PothosSDR_vc16\PothosComms.vcxproj]
C:\build\PothosSDR_vc16\Source\PothosComms\math\Exp10.hpp(30,18): error C2039: 'exp10': is not a member of '`global namespace'' (compiling source file C:\build\PothosSDR_vc16\Source\PothosComms\math\Exp.cpp) [C:\build\PothosSDR_vc16\Build\PothosComms\math\MathBlocks.vcxproj] [C:\build\PothosSDR_vc16\PothosComms.vcx
proj]

I swear, C/C++ programming is like Stockholm Syndrome sometimes. On it.


Update README.md - Deletion of Cleverbot : No Longer Public

Cleverbot : Deletion of API that is no longer free / public

Thank you for taking the time to work on a Pull Request for this project!
To ensure your PR is dealt with swiftly please check the following:
[ X ] Your submissions are formatted according to the guidelines in the contributing guide
[ X ] Your additions are ordered alphabetically
[ X ] Your submission has a useful description
[ X ] The description does not end with punctuation
[ X ] Each table column should be padded with one space on either side
[ X ] You have searched the repository for any relevant issues or pull requests
[ X ] Any category you are creating has the minimum requirement of 3 items
[ X ] All changes have been squashed into a single commit

Thanks!


Use install_yamls defaults to locate CRs to deploy instead of searching

As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running
[x] Appropriate documentation exists and/or is up-to-date:

/approve


Update socratic-46.md

Add recent blog posts for potential discussion

Hey @subsystem1125, I'm sorry I didn't see this before we did the event! Since we didn't end up discussing these links, I'm going to close this PR to preserve the record of what we talked about. I hope you had a good time at the event :)


Failing on some networks

Report from @avinashvarna (chrome+win) : https://pastebin.com/raw/AJXdK6jw
Prominent part:
sanskrit-coders.github.io/:1 Failed to load https://api.vedavaapi.org/couchdb/dict_entries/_design/index_headwords/_view/index_headwords?limit=500&start_key=%22r%22&reduce=true&group=true: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'https://sanskrit-coders.github.io' is therefore not allowed access. The response had HTTP status code 403.

This is strange, as CORS is enabled on the couchdb server (visible in the config file and the fauxton UI).

and indeed on my computer:

@avinashvarna - I wonder if there is some chrome setting you've made (honored across devices) that causes this? Can you check from an incognito window? Also, can you check firefox javascript console and open a new bug for it if needed?

Update from @avinashvarna - "I tried it on my wife's phone as well, out of curiosity. Didn't work there either (with wifi/cellular). Have you tried independently on a machine where you haven't logged in to vedavaapi ?
"
And an excerpt from an earlier message:

Laptop - FF (Win) + Chrome (Win)
Phone - FF (Android) + Chrome (Android) v 63.0.3239.111
चतुर्ष्वपि कार्यं नाकरोत् ।
यदा भवता प्रथमवारं sanskrit-programmers मध्ये सूचना प्रेषिता तदा मया प्रयुक्तम् आसीत् । तदा तु कार्यं करोति स्म इति स्मरामि ।

Confirmed to work from an incognito window on my computer.

On FF (Win), I get TypeError which seems to be covered by #2.
I don't sign into chrome, so it's unlikely that there is a setting remembered across devices. However, same result from incognito mode as well:

I think the main problem is the 403 Forbidden error. At that point, the server probably doesn't return the CORS header. Seems to be reproducible on all devices I have access to (including my wife's desktop and phone).

Same result (403 forbidden) in FF on ubuntu VM as well:

Not sure what's going on.  If it works for everyone else, then don't spend too much time on it.

I think this has to do with the server (api.vedavaapi.org) not supporting HTTPS correctly. For example, I can access http://api.vedavaapi.org/scala/swagger/index.html fine (if I type http explicitly), but https://api.vedavaapi.org/scala/swagger/index.html gives me a 403 error.
I have the HTTPS everywhere plugin installed in Chrome which redirects traffic to https if the website supports it.

The strange thing is that https://api.vedavaapi.org/scala/swagger/index.html works fine on my computer ..

Curioser and curioser. 😮 Not sure if the server logs can help us.

mitra, tvAm vedavApigaNe kasminchid adhunaivAmantritavAn; api cha https://github.com/vedavaapi/vedavaapi-servercfg/issues/6 ityasmin niyukto.asi

Looking at the logs: There were a lot of:
[Wed Dec 20 07:53:09.208854 2017] [authz_core:error] [pid 14052] [client <IP_ADDRESS>:57957] AH01630: client denied by server configuration: proxy:http://localhost:9090/swagger/index.html
[Wed Dec 20 07:53:10.268342 2017] [authz_core:error] [pid 14052] [client <IP_ADDRESS>:57957] AH01630: client denied by server configuration: /home/samskritam/vedavaapi-servercfg/apache2/static/favicon.ico, referer: https://api.vedavaapi.org/scala/swagger/index.html
Googling brought me to: https://wiki.apache.org/httpd/ClientDeniedByServerConfiguration

Proxying to a service with no explicit access in a location block.
ProxyPass /foo/ http://internal.example.com:8900/
ProxyPassReverse /foo/ http://internal.example.com:8900/

Applying the recommended solution for Apache 2.4:

Solution (2.4):
Location /foo>
Require all granted

solves the issue. Now I can access https://api.vedavaapi.org/scala/swagger/index.html, the dict UI works fine on chrome etc.
@vvasuki How was it ever working for you without this setting?

I have modified the conf file on the server. Should I update this repo? Didn't want to do it because the server conf and the github versions didn't match to begin with.

आह् - साधु!  कृपया  कोशे रक्षतु - सर्वदा तत्र स्यादेव प्रयोग्या व्यवस्थासञ्चिका।
                32.	  # Secured forward proxy - to access pesky but necessary government sites
		33.	  ProxyRequests On
		34.	  ProxyVia On
		35.	  <Proxy "*">
		36.

ಅಸ್ಯೋತ್ತರಮ್ ತು ನ ಜಾನಾಮಿ.

परिवर्तनेषु मया केवलं २०-२२, २७-२९ योजितम् । अत्र उदाहृतं परिवर्तनं (३२-३७) मया न कृतम् । अत एव

Didn't want to do it because the server conf and the github versions didn't match to begin with.

इत्यवदं खलु !

PR प्रेषितम् ।

Merged  - भविष्ये यथा भवान् साक्षात् रक्षितुं शक्नुयात् तथा व्यवस्था कृता।

एतदपि चित्रम् पश्यतु -

For example, I can access http://api.vedavaapi.org/scala/swagger/index.html fine (if I type http explicitly), but https://api.vedavaapi.org/scala/swagger/index.html gives me a 403 error.

इत्युक्तं खलु? https://github.com/vedavaapi/vedavaapi-servercfg/blob/7eeb5e5f11700b50bd09748a8df25973a9fcd84e/apache2/sites-available/vedavaapi.conf इत्यत्रैतावदेव +अदृश्यत -
  ProxyPreserveHost On
  ProxyPass /scala/ http://localhost:9090/ nocanon
  ProxyPassReverse /scala/ http://localhost:9090/

bug in apache? न जानामि ।


Convert some libraries from static to dynamic

Description and Context

When used as an external library in BACI, some linker issues occurred because of the use of a static library. Therefore, mirco_library target is converted to dynamic library and OpenMP dependency in added to it. The ideal case would be to make mirco as a dynamic library, but I was facing some issues regarding that. So I will open an issue regarding it. Merging this PR is required to move ahead with our BACI/MIRCO integration.
Also, -Werror compiler flag is added to treat warnings as errors in the pipeline.
Related Issues and Pull Requests

Closes
Blocks
Is blocked by
Follows
Precedes
Related to
Part of
Composed of

How Has This Been Tested?

ctest is working. mirco executable is working. mirco_library is used in BACI in my local, and it is working.
Checklist

[ ] My commit messages mention the appropriate GitHub issue numbers and are longer than one line where appropriate.
[ ] I have added/updated documentation where necessary.

Additional Information

Interested Parties / Possible Reviewers

@mayrmt

To avoid printing the Trilinos configuration messages while building BACI, I have moved them inside the TRILINOS_IN_MIRCO flag. These things will help us to integrate MIRCO into BACI; however, the CMakeLists in MIRCO need to be redesigned.

After the meeting with Sebastian Pröll, I think we can go ahead with the merge request as the current target mirco_library is enough for BACI. So that BACI is merge-ready as it does not have to care about the internal changes in MIRCO CMake. I am working on his comments to modernize MIRCO CMake.

Let's with this PR until PR #78 has been merged.


Add filter for group tags

Enable filter for 'Group-Tag' via preselection

fixed in ui-router branch


fix: openapi stuff

Summary
Fix some issues with openapi schema generation. See commits for details.
Related Issues / Discussions
https://discord.com/channels/1020123559063990373/1049495067846524939/1245141831394529352
QA Instructions
App should work, workflows should work.
Merge Plan
n/a
Checklist

[x] The PR has a short but descriptive title, suitable for a changelog
[ ] Tests added / updated (if applicable)
[x] Documentation added / updated (if applicable)

Note: the vast majority of the diff is regenerated types.


Anyone else getting memory issues?

❔Question
Hello! I wonder if anyone else is getting GPU memory errors even with the small model (yolos_small) ?
Additional context
I am on a 4 GPUs node with Geforce Gtx 1080 ti with 11gb memory each. I use batch size 1 as recommended. Both distributed and non-distributed versions throw the same error.
Tiny model trains smoothly without a trouble.
If there are any tips to reduce memory usage that would be awesome as well!

Hi @kilickaya, thanks for your interest in YOLOS.
We have actually answered here:https://github.com/hustvl/YOLOS/issues/5#issuecomment-867533669, --use_checkpoint may help you with that.


Disable GitHub packages (only for action repositories)

Should be disabled from the main repo, top right, gear icon

https://github.com/cli/cli/issues/8755


major changes to IPS tutorial

Hi Arianna, can you please review again.
new: https://developers-qa-blue.wcms-nonprod.c.eu-de-2.cloud.sap/tutorials/abap-environment-ips.html
old: https://https://developers.sap.com/tutorials/abap-environment-ips.html
Thank you a lot in advance
Ulrike

Hi Ulrike, all seem good! I will merge now :)


CARRY: Add workflow to release ODH/Kuberay with compiled test binaries

Why are these changes needed?
To speed up downstream tests.

Related issue number
https://issues.redhat.com/browse/RHOAIENG-3757

Checks

[x] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[x] Manual tests
[ ] This PR is not tested :(

PR check fail is unrelated as this PR adds a new workflow.

/approve


services/mirror: fix dropped test errors

This picks up two dropped test errors in the services/mirror package.

Codecov Report

Merging #9007 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #9007      +/-   ##
==========================================
+ Coverage   41.19%   41.19%   +<.01%
==========================================
  Files         546      546
  Lines       71306    71306
==========================================
+ Hits        29371    29377       +6
+ Misses      38208    38199       -9
- Partials     3727     3730       +3

Impacted Files
Coverage Δ

modules/log/event.go
64.61% <0%> (-1.03%)
:arrow_down:

models/repo_list.go
74.07% <0%> (+0.92%)
:arrow_up:

models/unit.go
67.56% <0%> (+5.4%)
:arrow_up:

modules/indexer/indexer.go
55.26% <0%> (+10.52%)
: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 149a9df...0df37b7. Read the comment docs.


Squash policy (github config)

Current behaviour
Once a PR is merged all the commits are in the main branch. If I have 15 commits they will be all in main
Expected behaviour
1 commit per PR on the main branch

Done, thanks ✅

@cicoub13 We can make squash merge the default but I wouldn't make that the only option. When I make 3 commits, all of them are atomic, with good commit messages and I still want to be able to revert only one of them easily. Sorry for not speaking up sooner.

OK with you but I would like the squash option the default one 🤔

https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/configuring-pull-request-merges/about-merge-methods-on-github

You can enforce one type of merge method, such as commit squashing or rebasing, by only enabling the desired method for your repository.

@cicoub13 I wouldn't enforce anything. It's up to us maintainers to tell apart PRs with commits that look like this:

main thing
oopsie cs
ah shit there's a bug with version X
maybe that will fix it
incorporate feedback from review

from PRs that look like this:

main thing
other unrelated small thing I'm fixing along the way, can be reverted independently


Add / Remove single IPs by using 'nic update'

As already discussed with @jasminSPC yesterday, nic update has a little issue.
    profitbricks nic update --datacenterid [dcid] --serverid [server_id] --id [nic_id] --ip <IP_ADDRESS>

The above mentioned command is causing very much trouble, because it's updating all IPs on this NIC to the given one from this command. This means, if you have this IPs set for example:
    Primary IP: <IP_ADDRESS>

    Additional IP: 251.123.321.168
    Additional IP: 251.123.321.222
    Additional IP: <IP_ADDRESS>
    Additional IP: <IP_ADDRESS>
    Additional IP: <IP_ADDRESS>

...the command would remove all IPs and just set the new primary IP <IP_ADDRESS>:
    Primary IP: <IP_ADDRESS>

This is fatal and can cause a downtime for multiple server, if the updated server is/was a router/firewall, which use NAT to forward all requests to the correct server. In my scenario for example, I've "shutdown" the full datacenter of my company for about 30 minutes.
The issue is caused by the REST API of Profitbricks, because the NIC update part requires a collection of IPs and not only a single IP: https://devops.profitbricks.com/api/rest/#update-a-nic
Well... Currently you have to get the list of all assigned IPs, add or remove your wanted IP from or to this list and run the nic update command on the CLI. Great... NOT!
I've already asked @jasminSPC , if you can add two further parameters to the CLI, which are doing this automatic. For example:
    profitbricks nic [update|add|remove]

Update should do something like updating the NIC or an IP, while add and remove should do how it's called for: Adding and removing something - in thise case single IPs. Following the logic, which I've already sent to @jasminSPC as file:

// Get list of assigned/set IPs
function getListOfSetIP(params) {
	var ipBlockList = processIpblock(list) // Save a current list of the IP Block(s)

	return ipBlockList
}

// Adds a further IP to NIC
function addIPtoNIC(params) {
    var ipBlockList = getListOfSetIP(params.datacenterid, params.serverid, params.id) // Call function to get current IP Block list

	var data = {}
    try {
        if (params.path) {
            data = JSON.parse(fs.readFileSync(params.path, 'utf8'))
        }
        else {
            data = {}
            if (params.name)
                data.name = params.name
            if (params.ip)
                data.ips = ipBlockList . [params.ip] // Append new/further IP to existing IP Block
        }
    }
    finally {
        pbclient.patchNic(params.datacenterid, params.serverid, params.id, data, helpers.printInfo)
    }
}

// Removes a single IP from NIC
function delIPfromNIC(params) {
    var ipBlockList = getListOfSetIP(params.datacenterid, params.serverid, params.id) // Call function to get current IP Block list
	ipBlockList = str_replace([params.ip], "", ipBlockList) // Remove IP Block from list

	var data = {}
    try {
        if (params.path) {
            data = JSON.parse(fs.readFileSync(params.path, 'utf8'))
        }
        else {
            data = {}
            if (params.name)
                data.name = params.name
            if (params.ip)
                data.ips = ipBlockList
        }
    }
    finally {
        pbclient.patchNic(params.datacenterid, params.serverid, params.id, data, helpers.printInfo)
    }
}

@jasminSPC has already contacted Profitbricks, to ask them, if there is an better solution for this problem or if they can implement something better for this to not slow down the CLI.
I'm waiting for the fix, thanks in advance!

Any updates, @jasminSPC ?

Nothing at the moment. I will keep you posted
On Mon, Jan 25, 2016 at 8:18 PM, TS3tools<EMAIL_ADDRESS>wrote:

Any updates, @jasminSPC https://github.com/jasminSPC ?
—
Reply to this email directly or view it on GitHub
https://github.com/profitbricks/profitbricks-cli/issues/20#issuecomment-174626869
.

--
Jasmin Gacić

You can achieve this by running one of following:
profitbricks nic update --datacenterid [dc] --serverid [server] -i [nicid] --addip <IP_ADDRESS>
profitbricks nic update --datacenterid [dc] --serverid [server] -i [nicid] --removeip <IP_ADDRESS>

Thanks, I'll test it!


Add missing build_deps.sh step to docs for static build on Linux

I had to call ./scripts/build_deps.sh before the ./configure step was satisfied.

Coverage remained the same at 85.248% when pulling a17d4c512ef04ada7f5bae14ed4a3bc5ce3e4678 on electricshaman:build-deps-step-static-linux into 4df286b1427b0dd3f4e621ff832e4864c57af656 on fhunleth:master.

Thanks for catching that!


Automated cherry pick of #1942: 华为 & ucloud fix

Cherry pick of #1942 on release/2.11.
#1942: 华为 & ucloud fix

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11@e972a40). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.11   #1945   +/-   ##
==============================================
  Coverage                ?   7.51%
==============================================
  Files                   ?     418
  Lines                   ?   64877
  Branches                ?       0
==============================================
  Hits                    ?    4876
  Misses                  ?   59590
  Partials                ?     411

Flag
Coverage Δ

#aFlag
7.51% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/guest_actions.go
0% <0%> (ø)

pkg/util/aws/region.go
0% <0%> (ø)

pkg/util/aws/vpc.go
0% <0%> (ø)

pkg/compute/models/cloudproviders.go
1.08% <0%> (ø)

pkg/util/ucloud/region.go
0% <0%> (ø)

pkg/util/aws/instance.go
0% <0%> (ø)

pkg/compute/models/snapshots.go
1.81% <0%> (ø)

pkg/util/ucloud/disk.go
0% <0%> (ø)

pkg/util/ucloud/snapshot.go
0% <0%> (ø)

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 e972a40...5225aaf. Read the comment docs.

/lgtm
/approve


Bump cats-effect to 3.0.0-RC2; port catnap+tests to CE3

Here's catnap upgrade I've been slowly doing over the past few weeks. Putting it out if you guys want to pick it up faster than I can do it (we should probably start a branch and make it a part of 4.x going forward since bincompat is going to be broken anyway).
A lot of our tests were relying on cats IO having a model where it executes stuff on current thread as much as possible, and that came to bite us of course. We should really consider writing tests using high-level pure-fp stuff, with cats-effect-testing or something.
The big change is that Async/Concurrent split is now very different and most of our stuff can built only using Async. No need for OrElse and such. And ContextShift and Timer are kinda gone. I'll write some more detailed comments in the upcoming days, feel free to ask if anything looks suspicious for now.

I changed the base to series/4.x.


Implement Icon priorities.

With this, a library can set a "default" icon, to be used only if the user doesn't provide a better one. Closes #1508.

Glad someone reviewed this before it was merged, even if that person was me five months later. Also I tested it more this time, as did CI.


ChrY variants on Female sample

Even thought I specified that the sample had sex female I see variants in ChrY, Is this normal?

Hi Konstantinos,
It is normal to see variants on the Y chromosome, even in a female sample. If you look in the multiqc report there should be some metrics regarding what the predicted genders of your sample are, so you can check if there is a swap. Even if there wasn't a swap, there are trace reads that align to Y t hat can be enough to call a variant there as well.

Hi Rory!
Why don't you just exclude Y chromosome when sex female is defined, so It is easier for downstream analysis?

The sample might not really be a female!
On Fri, Oct 18, 2019 at 8:08 AM Konstantinos Kyriakidis <
<EMAIL_ADDRESS>wrote:

Hi Rory!
Why don't you just exclude Y chromosome when sex female is defined, so It
is easier for downstream analysis?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/bcbio/bcbio-nextgen/issues/2983?email_source=notifications&email_token=AADFG6XR6KUS5URPYZOKVHTQPGRL5A5CNFSM4JB46L62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBUERDY#issuecomment-543705231,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADFG6XOMKH4NZWFL6TTJZLQPGRL5ANCNFSM4JB46L6Q
.

Hi Rory!
One more thing. I specified in my config file: effects: vep, but the line I get are like that:
chrY	56874973	rs6568306	C	G	141.0	PASS	AC=1;AF=1;AN=1;CALLERS=gatk-haplotype,strelka2;DP=4;FS=0;MLEAC=1;MLEAF=1;MMQ=60,45;MQ=45.62;MQ0=0;QD=28.08;SOR=0.693;rs_ids=rs6568306	GT:AD:DP:GQ:PL	1:0,4:4:99:151,0

The template I used is:
details:
- algorithm:
    aligner: bwa
    effects: vep
    ensemble:
      numpass: 2
      use_filtered: false
    mark_duplicates: true
    realign: false
    recalibrate: false
    save_diskspace: true
    tools_on:
    - vep_splicesite_annotations
    - noalt_calling
    variantcaller:
    - gatk-haplotype
    - strelka2
  analysis: variant2
  description: 178F
  files:
  - /mnt/36642bae-9ec9-4100-88a2-ac173a20ea16/WORKDIR/BRAZILIANS/178F/input/SRR6701493_1.fastq.gz
  - /mnt/36642bae-9ec9-4100-88a2-ac173a20ea16/WORKDIR/BRAZILIANS/178F/input/SRR6701493_2.fastq.gz
  genome_build: hg38
  metadata:
    sex: female
fc_name: 178F
upload:
  dir: ../final

There should have been VEP anntoation, right?


fonts load from de.serlo.localhost:4567 in dev

Does only occur in dev (since ASSET_PREFIX == NEXT_ASSET_PREFIX in production): the font isn't loaded correctly, since it is requested via NEXT_ASSET_PREFIX instead of ASSET_PREFIX

fonts get loaded multiple times, two times correctly and two times unprefix - no idea why this is happening

found out the reason, we need to pass assetPrefix to the client


WIP1

CH changes - Github pull-requests practice exercise - Deeksha Reddy Chikoti

will create a new pull request after changing some line in existing code


Feature: ++model++ to build additional dependencies

Feature
Feature description
When using the --models flag in conjunction with + only results in the downstream/upstream dependencies being built (and not their individual dependencies).  It would be useful to have a ++modelName++ syntax to build that path on the tree including the dependencies of the dependent/parent models.
Who will this benefit?
Anyone using selective builds in clean CI environments.

thanks for the suggestion @swettk! I'm super into the idea, but open to alternatives for the syntax. Ideally, we'd pick something 1) sensible 2) easy to parse 3) conflict-free with most terminals.


Requests don't respect forced https scheme

I have  \URL::forceScheme('https'); in app boot. However I noticed that requests still go to http://... and this doesn't seem to complete visually. If I refresh the page I can see that the POST or DELETE was successful but nprogress never completes and it and the page goes into a crawl state.

@iKlsR I'm just making assumptions but there's a good chance this is caused by ziggy. I remember also having issues with https and ended up adding this in app.blade:
@routes

@if(config('app.env') !== 'local')
     <script>window.Ziggy.baseProtocol = 'https';</script>
@endif

Long way from being perfect/nice, but it did the job.

@druc Thanks, seeing some success with this so it will do for now.

I'd recommend taking this issue up in the ziggy repo (https://github.com/tightenco/ziggy). The issue here is that the client-side doesn't realize that it should be HTTPS.


High CPU load with fail2ban in escaped docker processes

Subject

[ ] I would like to contribute to the project (code, documentation, advocacy, integration, ...)
[ ] I would like some help for integrating this container with another service (rainloop, PostFWD, ...)
[ ] I would like to configure a not documented mail server use case
[x] I would like some feedback concerning a use case
[ ] I have questions about TLS/SSL/STARTTLS/OpenSSL
[ ] Other

Description
I've recently had to restore a backup from Synology NAS where mail container is running. Opening the backup took ages and investigation showed high CPU load of >25. After stopping all docker containers including mail, problem persisted. ps aux and top showed that main culprit was still fail2ban from the mail container. Apparently fail2ban and other mail processes had survived stopping the docker container (I can confirm it was stopped and docker service shutdown).
Not sure if this is a problem with configuration of supervisord, supervisord itself or the docker version running on Synology but it would. be great if processes could be secured against escaping from container. fail2ban itself is useless in my case as it totally kills the machine.

If the processes survive stopping the container that sounds like a docker bug to me? Not much we can do about that as far as I know. As for fail2ban, don't run it if you don't need it. I don't run it personally, I prefer to run that kind of thing on the host. You can always mount the log files so that they can be accessed from the host.
If you have a suggestion for fixing this I'm all ears. If not I would suggest closing and checking if there is a bug fix for your docker version (or file a bug for it yourself).

Like @erik-wramner said, I too cannot image this to be a bug in the mailserver but rather a very rare bug in Docker itself. Can you kill fail2ban and restart the processes and redo what you described again to verify that this is reproducible?

I too cannot image this to be a bug in the mailserver but rather a very rare bug in Docker itself.

As I've said, might very well be supervisord or docker itself: https://blog.phusion.nl/2015/01/20/docker-and-the-pid-1-zombie-reaping-problem/

Can you kill fail2ban and restart the processes and redo what you described again to verify that this is reproducible?

Killing fixed the problem, I'll try to repro  when I  find time.

Alright. Please report back here.

@andig that was an interesting link. It was from 2015 though, so hopefully the Docker team has solved it by now. Let us know if you can reproduce the error. If so it would be nice with full versions for everything (OS, Docker, ...) as well.

Closing for time being. Have not observed this since.


module 'site' has no attribute 'getsitepackages'

When python setup.py install:
Building wheel kaolin-0.2.0+83002fe
--Building version 0.2.0+83002fe
Traceback (most recent call last):
File "setup.py", line 78, in
build_deps()
File "setup.py", line 43, in build_deps
sp = site.getsitepackages()[0]
AttributeError: module 'site' has no attribute 'getsitepackages'

Please comment out lines 42-45 in setup.py and try running the setup again. This will be addressed in an upcoming fix. Thanks for bringing it to our attention!

If inside virtualenv, site module is missing getsitepackages() method.
See https://github.com/pypa/virtualenv/issues/737#
Above link suggests using your system Python to find site packages:
python3 -c 'import site; print(site.getsitepackages())'

and editing file site.py inside your virtualenv by adding getsitepackages() method that returns the output of above.

getsitepackages() doesn't exist in venv in some versions of Python:
Python 3.7.6 (default, Dec 19 2019, 23:49:42)
[GCC 5.4.0 20160609] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import site
>>> site.getsitepackages()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: module 'site' has no attribute 'getsitepackages'
>>> site.getsitepackage()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: module 'site' has no attribute 'getsitepackage'
>>> site.getsitepackages()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: module 'site' has no attribute 'getsitepackages'
>>>

Your best bet is to iterate on sys.path, find one that you can write in (or one containing "site-packages" in the name) and put it on that.
If you do things like this, like a shot in the dark, we will have many back and forth on github. Your best bet is to write a unit test, and run it with all the Python version you support using tox. This will save you a lot of trouble.

Hi, thank you for using Kaolin! The library went through a major rewrite with the version 0.9.0, and this issue is no longer current. Please refer to our latest documentation:
https://kaolin.readthedocs.io/en/latest/


Is this still the way to go?

I've notice that the tRPC supports web sockets now, however i don't see any SSE support. Seems like this last update was 10 months ago, but may i ask if this way is still valid?
Thanks!

Hi @shirecoding! I just gave it a go with the latest tRPC release (10.45.x right now) without running into any new hiccups:
https://github.com/OutdatedVersion/trpc-sse-link/assets/11138610/a75bc156-7197-4632-9b58-61e8c9ec43f9
Probably already saw it when you were looking for WebSocket/SSE support but there's some movement on first-class stream support (WHATWG, not Node.js') in tRPC from October 2023 ... which seems related. Nothing past that.

Just want to pitch in here and say that I'm using a modified version of this repo's SSELink in prod with over 100 DAU, and so far so good. 😊


io: Remove blanket trait impls

Remove the blanket impls of Read, BufRead, and Write. Replace them with just the impls required to get rust-bitcoin building.
Note, the TcpStream stuff is used in examples/handshake.rs.
Fix: #2432

@apoelstra I did this instead of trying the trait idea we discussed - my aim is to make forward progress, by tooth and nail. This makes the crate non-broken and as such releasable. It may be missing functionality which seems reasonable for a v0.1.0.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

-6 of 10 (40.0%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 84.12%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

io/src/lib.rs
4
10
40.0%

Files with Coverage Reduction
New Missed Lines
%

io/src/lib.rs
2
70.92%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
19329

Relevant Lines:
22978

💛  - Coveralls

Just noticed I missed doing everything in https://github.com/rust-bitcoin/rust-bitcoin/issues/2432#issuecomment-1928116934

@TheBlueMatt is this PR going to rug you?

I've implemented for BufWriter as requested. Also Write for TcpStream since we do Read. Also I've added generics to the BufRead and BufWrite impls.

Please merge #2352 before this one.

Rebased and improved commit message to mention the macro changes. No other code changes.

No sweat, lets get this first release done - rust-bitcoin devs, gotta ship.

@apoelstra I suggest let's wait for Tobin to address my comments before merging. I mainly ACK so you can merge if Tobin doesn't want to change it because I think it's acceptable.

Happy to change, all suggestions seem good to me. I'll hack it up.

All done except the sink stuff. I think the sink is correct, it currently is:
/// A sink to which all writes succeed. See [`std::io::Sink`] for more info.
///
/// Created using `io::sink()`.
pub struct Sink;

impl Write for Sink {
    #[inline]
    fn write(&mut self, buf: &[u8]) -> Result<usize> { Ok(buf.len()) }

    #[inline]
    fn write_all(&mut self, _: &[u8]) -> Result<()> { Ok(()) }

    #[inline]
    fn flush(&mut self) -> Result<()> { Ok(()) }
}

/// Returns a sink to which all writes succeed. See [`std::io::sink`] for more info.
#[inline]
pub fn sink() -> Sink { Sink }

92cb9026111924b3af44300afdce8621d9b8353a looks good except that it removes the std::io::Write impl for our Sink ... but I think we can/should keep this impl.

Woops, will put it back in. Hard to get good help huh.

Add the impl back in and made the impl of our Write unconditional i.e., always implement our Write and implement std::io::Write if "std" is enabled. Same as what the macros do.

FTR we don't actually use the Read trait since we use BufRead and don't provide our own BufReader. So we could actually remove the Read trait and move helpers on top of BufRead. :) Not sure if it's worth the effort though.

Fair points, lets leave it as is for now and see where the io crate goes. At the moment it is doing its job.


Wrong type hint for Then.validate_data() method

Hello!
In all your examples Then.validate_data() method accepts name of the class, but the type hint is BaseModel and type checker reports error. Perhaps it should be replaced with Type[BaseModel].

Hello!
Thank you for pointing that out! Fixed in https://github.com/olxxi/ReqFlow/commit/1aef53fa7bda73e922d17bb166bcb2b1881fd4c3 and pushed to PyPI.
pip install ReqFlow==1.0.5a0


xtt tool organization

The current xtt appears to me (i.e., this is my opinion!) like an arbitrary collection of commands, not a cohesive xtt utility.
Take the six actions today:

genkey (ecdsa)
gen509cert (x509 cert)
wrapkeys (ASN.1 wrapped keys)
genrootcert (root cert)
genservercert (server cert)
infocert (any cert)

I see the following inconsistencies (not an exhaustive list):

One of the commands specifies something about the data format (x509); the rest do not.
The gen cert commands are split by type (x509, server, and root); the info cert command is combined.
The genkey help string mentions the underlying crypto details (ecdsa).  The wrapkeys and various cert commands do not mention the underlying crypto details.
None of the commands reference the terminology used in our discussions, e.g., provisioning key (DAA) vs identity key (formerly ed25519, now ecdsa).
5 of the command names are a combined verb and noun (gen/wrap and key/cert).  The 6th is two nouns, info and cert.

I think it's worth discussing an organization for these commands.
Should the "types" of cert and key be based on the usage (e.g., root, identity, server) or should they be based on structure (e.g, ecdsa, x509, ed25519).
Should the commands be organized beyond just a single action?  E.g., "xtt cert show", "xtt cert generate", "xtt keys generate", "xtt keys show --public".
What additional commands are planned or desired?  Getting those documented (perhaps as strawman usage strings in this issue thread) would be helpful.

Command
Component
Structure
Usage

Generate
Key
ECDSA
Root

Wrap
Certificate
x509
Server

GetInfo

ASN.1
Identity

(Xaptum Type)

ECDAA

These commands could be laid out perhaps into four parts.
One slightly difficult part in naming these is that commands, right now, do not necessarily use all 4 parts. For example, the x509 certificate does not have a specific usage(root, identity, server), and the server and root certificates do not have a "structure" that we have a name for--since no one else uses this structure. Can/should we name this Xaptum specific structure?


[FLINK-2662] [dataSet] Translate union with multiple output into separate unions with single output.

Fixes FLINK-2662 by translating Union operators with two (or more) successors into two or more Union operators with a single successor.
In the optimizer union operators with two (or more) successors caused problems, when these successors had different partitioning requirements and some of these successors were other Union operators. In certain situations, the UnionMerging post pass would fail because of a non-forward shipping strategy between two subsequent union operators.
This fix does only adapt the program translation and does not change the API.

Another possibility would be to simply add a NoOp operator behind a union with multiple outputs with different partitioning schemes. That prevents the unions from being merged.
Would that not be better, in terms of not duplicating records (as it happens when having multiple instances of the one union operator)?

Adding a NoOp would prevent the unions from being merged, but the output of the NoOp would still need to be replicated because it needs to be served to two different operators.
if you have something like
1 ------------\
               >-> U_2 -> X
2 -\          /
    >-> U_1 -<
3 -/          \-> Y

Duplicating U_1 would would temporarily result in

1 --------------\
                 >-> U_2 -> X
                /
2 -\-/-> U_11 -/
    X
3 -/-\-> U_12-> Y

The generated plan with merged unions would be
1 --\
     >->-> X
    / /
2 -/-/--\
    /    >-> Y
3 -/----/

With adding a NoOp the plan would be:
1 -----------\
              >-> X
2 -\         /
    >-> NO -<
3 -/         \-> Y

This plan would also duplicate record (the output of NoOp) and in addition add serialization overhead due to the additional operator.

I think you are right.
+1 to merge then

Thanks, merging then


Type Mismatch: Negative to Unsigned

Summary
The function in ec_asn1.c is declared to return an unsigned value, but on line 72 it returns a negative value.The function is declared to return an unsigned value, but in some cases it returns a negative value.
What version of gRPC and what language are you using?
0.0.3
IOS
What operating system (Linux, Windows,...) and version?
MAC- 10.15.3
What runtime / compiler are you using (e.g. python version or version of gcc)
Xcode
What did you do?
Explanation
It is dangerous to rely on implicit casts between signed and unsigned numbers because the result can take on an unexpected value and violate weak assumptions made elsewhere in the program.

The file ec_asn1.c is under third_party/boringssl-with-bazel


JFK3 departure not acting as it should

For aircraft assigned the JFK3 departure, I get no assigned altitude or fix.

Looks like it was a syntax issue with the runway property, it was entered with an extra colon, as "rwy:":, instead of "rwy":. Will be resolved by
"JFK3": {
"icao": "JFK3",
"name": "KENNEDY THREE",
"rwy:": {
  "31L": ["CRI", ["_CRI3", "A25+"]],
  "31R": ["CRI", ["OGY", "A25+"]],
  "04L": ["_RWY04L3", ["_RWY04L3to100", "A50+"]],
  "04R": ["_RWY04L3", ["_RWY04L3to100", "A50+"]],
  "13L": ["_RWY13LDEP"],
  "13R": ["_RWY13RDEP"],
  "22L": ["_RWY22LDEP"],
  "22R": ["_RWY22RDEP"]
},

etc...

Will be resolved by #442.


Configurable scene loader (e.g. glTF options)

Currently configuring loader options is cumbersome and error prone. If you wanted to adjust a glTF loader option as well as the MSFT_lod extension options for example, your code would look something like this:
const pluginActivatedObserver = SceneLoader.OnPluginActivatedObservable.add((loader) => {
  if (loader.name === "gltf") {
    const glTFLoader = loader as GLTFFileLoader;
    glTFLoader.skipMaterials = true;
    glTFLoader.onExtensionLoadedObservable.add((extension) => {
      if (extension.name === "MSFT_Lod") {
        const msftLodExtension = extension as MSFT_lod;
        msftLodExtension.maxLODsToLoad = 5;
      };
   });
  }
});

// We want to configure the glTF loaded by this call
await SceneLoader.ImportMeshAsync(...);

// Oops we also configured the glTF loaded by this other call
await SceneLoader.ImportMeshAsync(...);

pluginActivatedObserver.dispose();

This is somewhat difficult, especially the deeper the options you want to set (such as glTF 2.0 extension options). It is also not super type safe as it requires casting plugins and extensions to known types. Also the configuration requires modifying global/static state, which means it is possible to inadvertently configure other imports besides your own.
While thinking about solutions to this problem, it occurred to me that a solution to this problem is more obvious when we think about it in the context of another problem, which is eliminating our dependency on side effects. Currently loader plugins are registered via side effects, glTF version loaders are registered via side effects, and glTF 2.0 extensions are registered via side effects. If we were not relying on side effects, then the user would do something explicit to opt in to the loaders, glTF versions, and glTF extensions they actually want to support. If they are already doing that, then it would present and obvious place to put configuration for those plugin/loader/extensions. Given this, my proposed changes in this PR work towards solving both of these problems by making providing the option of explicitly choosing what plugin/loader/extensions you want, and configuring them while you are at it. The replacement code for the above would then be:
const sceneLoader = new SceneLoader({
  plugins: [
    GLTFFileLoader.Configure({
      skipMaterials: true,
      loaders: {
        ...GLTF2.GLTFLoader.Configure({
          extensions: {
            ...GLTF2.Loader.Extensions.MSFT_lod.Configure({
              maxLODsToLoad: 5,
            }),
          },
        }),
      },
    }),
  ],
});
await sceneLoader.importMeshAsync(...);
sceneLoader.dispose();

The advantages of this are:

Conceptually easier to understand (subjective).
Fully type safe, no casting (you are explicitly interacting with the plugin/loader/extension types and therefore their associated options.
No global state modified, the options you specify apply to your own SceneLoader instance (the Configure calls just return factories that can instantiate the plugin/loader/extension with options).
You opt in to only the stuff you care about (in the example above, only glTF 2.0 and only the MSFT_lod extension).

This approach is also fully backward compatible, as it simply introduces the option to instantiate a SceneLoader and configure it exactly the way you want (leveraging the existing factory function logic). I think this approach is inline with a future CoreScene/CoreSceneLoader/CoreSceneLoaderPlugin as well, so we could likely rework it to be built on top of that layer when it is created.
High level notes on the changes:

Static state on SceneLoader was mostly moved to instance state, with a private default instance that the static contract calls into. The default instance behaves the same as the original static API.
Options introduced at each layer (plugins, glTF loaders, glTF 2.0 extensions).
Configure functions added that take the options and return a factory (compatible with existing approach).
In some cases, and OptionsBase class was introduced to maintain back compat (options directly on plugin/loader) while avoiding duplicating the list of options across multiple types.

Completing this approach will require:

[ ] Possibly adding default configurations that match the exhaustive configurations we have today (all plugins, all loaders, all extensions), to make it easy to still have that but then tweak a few options.
[ ] Add code doc comments to new APIs.
[ ] Add documentation explaining the option of instantiating SceneLoaders.
[ ] Add unit tests.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

I do like it! And yes we protect backward compat for now! (as we wont remove side effects)
This is a great idea!

A side-note - I find the plugin configuration a bit complicated (syntax-wise). Adding it to importMesh would make more sense TBH, but we will probably have an issue with back-compat.

I initially thought about passing options directly to importMesh, but there are a number of complications. Things like how do we route the options through to the right plugin/loader/extension in a type safe and non-error prone way, and what if someone passes through options for a plugin/loader/extension they have not opted in to use?
Configuring the plugin/loader/extensions as part of opting in to using them doesn't face these same problems. Since SceneLoader doesn't actually hold much state, and instance is basically (as you said) a specific configuration of loading logic, so it is quite light weight and creating multiple instances shouldn't be a problem. It also has the advantage that once you setup your desired configuration, you can just make import calls on that instance many times without having to pass through a complex configuration for every call.

@ryantrem should we close this one?


WIP: Add missing query import to README example

Clicked the "Update branch" button in GitHub, but it might be better if you do a rebase and I can merge it.
PS! It's perhaps better for the future, if you have many minor changes to the README, that we do them as one PR.

Also, if you have a work in progress PR that you don't want me to merge right away, you can do a DRAFT PR.

https://github.blog/2019-02-14-introducing-draft-pull-requests/

Sorry for spamming I can keep this one open till end of the week if I find more typos in documentation.

NP. We should probably create a contribution guide at some point 👍

@mishak87, if you want to squash your commits, I can rebase and merge.

Rebased and squashed typos. I found the last one by accident.
Finally after two months I am getting back to programming 😆

Rebased & merged :-)


chore(install): add codecov utility

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@205edb6). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1385   +/-   ##
=======================================
  Coverage        ?   46.58%
=======================================
  Files           ?      117
  Lines           ?     6966
  Branches        ?        0
=======================================
  Hits            ?     3245
  Misses          ?     3582
  Partials        ?      139

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Callouts and fonts are not rendered correctly

Issue 1: Callouts
This is a great plugin. I love it. The issue here is that the callouts are not rendered correctly.
MDX Preview:

Live preview in Obsidian

Issue 2: Fonts
This is my font setup

While previewing in MDX Preview I get the interface font in the display while it must have been displayed in the  Text font

I get the correct font in Obsidian live preview

I was able to fix the second issue with the below CSS appended to styles.css file
#text,
p,
li {
    font-family: Optima !important;
}

But I doubt if it covers everything

Hi @Marin-Kitagawa, thanks for your contribution 😃
At least for now, I don't have the plan to make MDX preview plugin support Obsidian syntax.
The basic function of this plugin is the same as https://mdxjs.com/playground/
The long-term goal is to implement a live editor for MDX, where the Obsidian syntax would be supported.


Add EventEmitter module

This PR provides a fairly straightforward implementation of the EventEmitter bindings. I tried very hard to come up with a implementation in which Stream.t is a subtype of EventEmitter, but there are a lot of complications with that.
The most important problem is establishing some degree of type safety with respect to emitters, listeners, and events.  Each emitter should only be allowed to register listeners for its own set of events (not just any Event.t). Each event should also have exactly one valid listener signature. Therefore we use the encoding Event.t('listener, 'emitter) to try to uphold those invariants.
I explain most of this stuff in a doc comment in the Event module.
Here's a working example of the API in use:
module MyEmitter = {
  include EventEmitter.Make({});
  module Events = {
    let greeting: Event.t(string => unit, t) = Event.fromString("greeting");
    let insult: Event.t(string => unit, t) = Event.fromString("insult");
  };
};

module Test = {
  let emitter = MyEmitter.make();
  let result = {
    open MyEmitter;

    emitter
    ->on(Events.greeting, Js.log)
    ->on(Events.insult, Js.log)
    ->emit(Events.greeting, "Hello")
    ->ignore;

    emitter->listenerCount(Events.greeting);
  };
  Js.log(result);
};

As you can see, because the Event.fromString & Event.fromSymbol functions have polymorphic return values, each event must have an explicitly annotated type. This is how you define event listener signatures.
It's still possible to shoot yourself in the foot by using the same string or symbol to create multiple events with different listener function types, but I think it's acceptable. We can just warn about it in the docs.
Let me know what you think!

I really like this solution

Tests are passing, merged.


Create joyshockmapper.json

btw to clarify i added this here instead of scoop-extras bc betterjoy is here

I tested this manifest locally and I'm getting this when starting Joyshock Mapper, either through a command prompt or using the Start Menu:

it seems to be an issue with the /current/ junction link that scoop does, i'll contact the main dev

btw i'm kinda new to making scoop stuff so thanks for the help !

oops sory

This PR appears to have fallen apart (and I keep getting notifications due to CI force-pushing updates), so I'll close it for now.
Feel free to open another PR with the manifest if you wish :slightly_smiling_face:


application for dynamic QR codes for SMS WhatsApp etc

When I click to create static QR it gives me different options such as WhatsApp SMS PayPal etc. When I click to create dynamic QR it only allows me to create url redirection url
I wish you could also create dynamic QR code for SMS WhatsApp and all the other things that can be created for static QR code

This might be a cool feature, unfortunately I haven't time for develop that.I would appreciate if you could contribute to this project by developing this feature.


Separate config interface into parts

What does this PR do?
Separates the config interface into three parts: reader, writer, and loader
Motivation
Simplifies the interface definition for the config/sysprobeconfig components
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

Blocked by #14024


add counts next to percentages

Users want to see the counts for crfs in addition to percentages- should have toggle for this - maybe replace display cell annotations control

Heather informed me that Fractions are desired not counts


Obsolete parent terms

Hello! During our build of our EFO 3 beta, we have come across a couple of terms that are subclasses of obsolete terms in MONDO:

MONDO:0006280 is a subclass of an obsolete term MONDO:0003221
MONDO:0016612 is a subclass of obsolete term MONDO:0000559
MONDO:0016982 is a subclass of obsolete term MONDO:0005329

oh dear, sorry, I thought we were checking these. My recollection is that we had our own SPARQL check on release for this, while we were waiting for this functionality in report - it seems not.
But in any case, this is now bundled into the report option in the latest robot, we'll configure the workflow so report is always executed which should prevent this in future.
For now, these have all been fixed in the edit version and will be reflected in next release

@cmungall @nicolevasilevsky
FYI, I see that all 3 errors reported by Zoe are fixed now, so this ticket could be closed (I won't do that myself now as I'm not sure what your preferred policy/procedure is re closing tickets).


Enable some const in std.uni

Slightly hackish but is a huge enabler for compile-time stuff.

Turns out I can live without this at least for now, so closing it


Bug - Jasper Big block RGH3 issue

Describe the bug
Hello Octal450
Thank you for amazing job with J-runner with extras. But I found a bug with Big block 512mb jasper.
Everything works until you try to write BB (64mb) nand. Instead of 15 min write time it will finish in 3 min (like 16mb nands do).
When you write this nand with older j-runner version, it will write full BB 64mb nand and console works.
This problem occurs with XeBuild image and stock nand too. I didn't try any other BB nands or 48mb(corona 4gb) nands yet.
I found it immediately as I saw write time. But there are a lot of other not so experienced people out there, that won't notice something is wrong. I hope you can fix that in latest update.
If you need some additional info or nand dump, feel free to contact me.
Have a nice day.
Nightmare0

Hi,
What device are you trying to write with?
Kind Regards,
Josh

Hello
I am using JR-P v2 for many years without any issues.

hi,
Thanks, I'll look into it.
-J

Confirmed. Fix is on the way.
J

That was fast. Do you need to test it? I still have that console.

I'll message you once I send the release out. Few little bugs to squash.
J

If I will still have that console I will test it for you. Nevertheless, good job. Have a nice day.

It will be within the next few hours. Can that work?
You too!
J

It's 0:20 here. Going to work at 6. If you don't mind, I would do it tomorrow after work.

That is fine, thank you!
J


Support logos

Also please add CHANGELOG entry.

@normanjaeckel Done.

@normanjaeckel Ping.

Rebased and resolved changelog conflic.

@normanjaeckel Please check again.

Successfully retested.


PHPCS is a directory.

Hello,
I am trying to use this hook on my windows with gitbash.
While committing the files it prompts me  as
"9ceafc16be2f0e3d13d80456028bcfef94f0b1d1
.git/hooks/pre-commit: line 106: C:/xampp/php/PEAR/PHP/CodeSniffer/Standards/PHP
CS: is a directory"
Can anyone help me in same?
Thanks in advance

While committing the files it prompts me as
9ceafc16be2f0e3d13d80456028bcfef94f0b1d1
.git/hooks/pre-commit: line 106: C:/xampp/php/PEAR/PHP/CodeSniffer/Standards/PHP
CS: is a directory

My guess is that you've set the path to your phpcs executable to the PHPCS ruleset directory in your config file. I don't know where it's installed on a Windows machine but you could probably determine the path by running where phpcs in the command shell or which phpcs in gitbash.


Improved Python Error message

Expected Behavior
We receive the exact location of the message
 self.SetHoldings("SPY", 1 / None)
                         ~~^~~
 in main.py: line 11

Actual Behavior
We read the following:
self.SetHoldings("SPY" in main.py: line 11

Potential Solution
Review the error parser to process Python 3.11 new format.
Reproducing the Problem

# region imports
from AlgorithmImports import *
# endregion

class CasualYellowGreenPenguin(QCAlgorithm):

    def Initialize(self):
        self.SetStartDate(2022, 10, 8)
        self.AddEquity("SPY")

    def OnData(self, data: Slice):
        self.SetHoldings("SPY", 1 / None)

Checklist

[x] I have completely filled out this template
[x] I have confirmed that this issue exists on the current master branch
[x] I have confirmed that this is not a duplicate issue by searching issues
[x] I have provided detailed steps to reproduce the issue

There is another case where we receive this message:
Runtime Error: 'NoneType' object is not iterable
for:
class EnergeticBlackOwl(QCAlgorithm):
    def initialize(self):
        self.set_start_date(2024, 5, 1)
        self.add_equity("SPY")
        self.add_alpha(MyAlphaModel())

class MyAlphaModel(AlphaModel):
    def update(self, algorithm, data):
        return

the bug here is that update must return a list, and we don't have the full stack trace which we can only  see in the syslog:
2024-05-08T12:12:48.4386968Z ERROR:: Extensions.SetRuntimeError(): Extensions.SetRuntimeError(): RuntimeError at 05/01/2024 13:31:00 UTC. Context: OnData Python.Runtime.PythonException: 'NoneType' object is not iterable
   at Python.Runtime.PythonException.ThrowLastAsClrException() in \src\runtime\PythonException.cs:line 52
   at Python.Runtime.PythonException.ThrowIfIsNull(NewReference& ob) in \src\runtime\PythonException.cs:line 470
   at Python.Runtime.PyIter.GetIter(PyObject iterable) in \src\runtime\PythonTypes\PyIter.cs:line 58
   at Python.Runtime.PyObject.GetIterator() in \src\runtime\PythonTypes\PyObject.cs:line 770
   at QuantConnect.Algorithm.Framework.Alphas.AlphaModelPythonWrapper.Update(QCAlgorithm algorithm, Slice data)+MoveNext() in /Algorithm/Alphas/AlphaModelPythonWrapper.cs:line 81


Multiple GPUs

Can cn24 take advantage of multiple GPUs?

The only way would be to run multiple instances of CN24 and using the config files to select different GPUs. There is no support for SLI/CrossFireX in CN24.