Usability: Unable to go back to the properties screen

Once the user has advanced to step 3, there does not appear to be any way to go back to step 2 to select, edit, etc., any of the properties.

This functionality can be accomplished by changing the status of the mapping on the "View Mappings" screen.  Admittedly this is a bit cumbersome, but the application is functioning according to spec at this point.  We can address this in a future phase.

We have a work around which we should document. Needs review of workflow and new functionality before we can action this.
Adding documentation tag and making priority 2

Fixed now


Mypy "Unexpected keyword argument" error when using alias

Initial Checks

[X] I have searched GitHub for a duplicate issue and I'm sure this is something new
[X] I have searched Google & StackOverflow for a solution and couldn't find anything
[X] I have read and followed the docs and still think this is a bug
[X] I am confident that the issue is with pydantic (not my code, or another library in the ecosystem like FastAPI or mypy)

Description
Using a Field with an alias results in following mypy issue:
$ mypy sample_v1.py
sample_v1.py:10: error: Unexpected keyword argument "foo" for "Sample"  [call-arg]
Found 1 error in 1 file (checked 1 source file)

When I use mypy plugin for pydantic the error changes to:
$ mypy sample_v1.py
sample_v1.py:10: error: Missing named argument "foo_alias" for "Sample"  [call-arg]
Found 1 error in 1 file (checked 1 source file)

Feels similar to #5070 but occurs for nwere version of both pydantic and mypy - regression?
Example Code
from pydantic import BaseModel, Field

class Sample(BaseModel, allow_population_by_field_name=True):
    foo: str = Field(alias="foo_alias")

Sample(foo="foo_value")

Python, Pydantic & OS Version
pydantic version: 1.10.13
            pydantic compiled: True
                 install path: /Users/pawelr/source/mypy_pydantic/dev_venv/lib/python3.8/site-packages/pydantic
               python version: 3.8.10 (v3.8.10:3d8993a744, May  3 2021, 09:09:08)  [Clang 12.0.5 (clang-1<IP_ADDRESS>)]
                     platform: macOS-14.4.1-arm64-arm-64bit
     optional deps. installed: ['typing-extensions']

Affected Components

[X] Compatibility between releases
[ ] Data validation/parsing
[ ] Data serialization - .model_dump() and .model_dump_json()
[ ] JSON Schema
[ ] Dataclasses
[ ] Model Config
[ ] Field Types - adding or changing a particular data type
[ ] Function validation decorator
[ ] Generic Models
[ ] Other Model behaviour - model_construct(), pickling, private attributes, ORM mode
[X] Plugins and integration with other tools - mypy, FastAPI, python-devtools, Hypothesis, VS Code, PyCharm, etc.

@rzepinskip,
Thanks for your question!
Check out this issue: https://github.com/pydantic/pydantic/issues/5893 and this note in the docs for more info.
Check out this section of the docs to learn how to enable the mypy plugin: https://docs.pydantic.dev/latest/integrations/mypy/#strict-optional.
We're no longer actively working on V1. Given that this is fixed in V2, closing this as resolved.


Sort entities for each suffix group in schema

Currently the list of entities associated with each group of suffix for a given datatype can be listed in any ways and do not have to match the  canonical BIDS entity order.
https://github.com/bids-standard/bids-specification/blob/master/src/schema/rules/entities.yaml
That file should be the final arbiter on bids entities order, but it would be more convenient for schema user and downstream tools if the list of entities were properly ordered for each suffix group.
I suggest adding some CI workflow for the schema to check that entities in suffix groups are in the right order.

duplicate of #1071


Voice Notes

Is your feature request related to a problem? Please describe.
No feature of sending voice notes
Describe the solution you'd like
I would like to add feature where crew mates can send voice notes to each other.
Describe alternatives you've considered
Additional context
Please assign me this issue under GSSoC'24.

Thanks for assigning!!

Hi @Chelseasingla1,
Any updates?

Hi @Chelseasingla1, Any updates?

yes  i will update it by today

Hi @Chelseasingla1, Any updates?

There is still some issue with voice notes function like how it will reach to another server maybe need some changes regarding that can you provide me some time for that, it is showing recording is on but not able to sent it in recording from


Add created field to containers page

Is your feature request related to a problem? Please describe.
We need to see at a glance the state of all our stacks (and their services) across many projects on our swarm. It's currently difficult to find out which stacks or services have tasks that are failing or stuck in a retry startup loop, without a lot of clicking into all the stacks and services.
Describe the solution you'd like
Adding a created or started (ideally this and maybe other fields would be selectable via the portainer UI on demand) would allow us to sort all containers across our whole swarm by timestamp, and if a service is stuck in a retry loop it would obvious as we would see multiple running/stopped containers for the that service near the top of the containers list.
Describe alternatives you've considered
Click into each stack or service and look at its task list, which is already sorted by "last update".

Related to #568

Hi @deviantony
I am new to portainer and loved what you guys are doing here.
I have added Created column at the end of the table and attaching a screenshot here.
What would be the appropriate order of the column?
Also, will it help if we use something like moment.js to make it human readable? Or will that be confusing while sorting on the column?

Hi @parag-datar

What would be the appropriate order of the column?
What do you mean?

Also, will it help if we use something like moment.js to make it human readable? Or will that be confusing while sorting on the column?

We're already using moment.js, have a look at the getisodate Angular filter that is used in other views.

Sorry for not being clear in my last message.

What would be the appropriate order of the column?

What I mean is, including the Created At column, there are 9 columns. Currently, I have placed it at the end, after the Ownership column. Is that okay or should I place it somewhere else?

We're already using moment.js, have a look at the getisodate Angular filter that is used in other views.

Thanks, will take a look 👍

Ah, IMO it should be located after the Status column and before the Quick Actions.
Actually, we're thinking that 9 columns is too much here...  we should solve #568 first.


Add simple concurrency support to HTTP20Connection

adds simple scheme for allowing HTTP20Connections to be used
concurrently by adding a couple of RLocks.

connection and stream creation is synchronized

reads and writes of frames on the connection and its streams are
synchronized

connection#request is synchronized

The current tox tests pass, but no new unit tests are added.
However a threading test is added to test_release.py that passes, e.g
$ py.test -s
test_release.py::TestHyperActuallyWorks::test_threaded_abusing_nghttp2_org

Thanks for this @tbetbetbe, this is a really good start! I've just pushed up the concurrency branch: feel free to target this PR against it.
We need to be really careful to guard against the risk of deadlock, where the big problem is the order in which we take locks, and ensuring that some of the recursive logic used by hyper doesn't bump into any of those conditions. Doing this is going to be really tricky, I think: fundamentally, we may have to analyse possible control flow paths by eye and confirm that they have no risk of deadlocking.

Closing for #189.


Remove the limit on number of concurrent streams

See #685 for details. We certainly need more than 256 streams. Having an upper limit close to that number leaves us vulnerable to having this issue again in the future.
So I'd rather not limit it and rely on performance metrics if we start seeing impact of too many streams open.
I also don't see an easy way to test this, so tests are not included :(

Have you confirmed that there is no limit by default? The HTTP/2 spec default is unlimited, but hyper might have its own default.

yea that's what hyper documentation states: https://docs.rs/hyper/latest/hyper/server/conn/struct.Http.html#method.http2_max_concurrent_streams


Update code of conduct pt-br to the latest version

Description
Update Code of Conduct page in Brazilian Portuguese to the latest English version.
Fixes #38404

/assign

/retitle [pt-br] Update community/code-of-conduct

/cc @MrErlison
Could you PTAL when you have some time?

/uncc @jailton @jhonmike

/approve
Thank you very much! 🎉
This PR will be merged after a second review with /lgtm is provided.

/label tide/merge-method-squash

/lgtm


DocumentDB: Adding support for listing connection strings

This is useful for MongoDB based DocumentDB accounts. Now referencing azure-mgmt-documentdb==0.1.1. Upgraded from azure-mgmt-documentdb==0.1.0

@dmakwana,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Awesome! Would it be possible for the output of the az documentdb create command to display the MongoDB connection string after completion (if --kind MongoDB was set)? This way the user doesn't need to immediately re-run list-connection-strings to retrieve it, and it circumvents them from potentially getting confused with the connection string that is displayed by default.

@lostintangent This is required also from REST/PowerShell; users need to call the list-connection-string operation manually. For parity, we will leave it like this for the time being

@dmakwana please rebase and resolve conflicts

@dmakwana please fix CI failures.

@troydai Is there any way to kick this off again? The failure doesn't seem related to my changes. It's a storage account test and it's only failing for python 3.5

@dmakwana I reset the tests.

@troydai Seems like the Travis CI server isn't co-operating. It failed but it won't populate the job log. Any idea?

Codecov Report

Merging #2580 into master will increase coverage by 0.73%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2580      +/-   ##
==========================================
+ Coverage   72.58%   73.31%   +0.73%
==========================================
  Files         374      397      +23
  Lines       20297    22798    +2501
  Branches     2971     3593     +622
==========================================
+ Hits        14732    16715    +1983
- Misses       4638     5071     +433
- Partials      927     1012      +85

Impacted Files
Coverage Δ

...tdb/azure/cli/command_modules/documentdb/custom.py
10.63% <ø> (ø)
:arrow_up:

...b/azure/cli/command_modules/documentdb/commands.py
93.75% <100%> (+0.41%)
:arrow_up:

...or/azure/cli/command_modules/monitor/validators.py
15.78% <0%> (-34.22%)
:arrow_down:

.../azure/cli/command_modules/vm/_template_builder.py
78.66% <0%> (-2.29%)
:arrow_down:

...li-batch/azure/cli/command_modules/batch/custom.py
82.06% <0%> (-0.72%)
:arrow_down:

src/azure-cli-testsdk/azure/cli/testsdk/patches.py
82.85% <0%> (-0.48%)
:arrow_down:

...ure-cli-sql/azure/cli/command_modules/sql/_util.py
90.19% <0%> (-0.03%)
:arrow_down:

...mmand_modules/azure-cli-role/azure/cli/__init__.py
100% <0%> (ø)
:arrow_up:

...vice/azure/cli/command_modules/appservice/_help.py
100% <0%> (ø)
:arrow_up:

...onitor/azure/cli/command_modules/monitor/params.py
100% <0%> (ø)
:arrow_up:

... and 50 more

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 4c8780a...8602384. Read the comment docs.

@dmakwana tests were reset and passed. There was an operational issue with Travis CI. In multiple cases I observed the missing logging issue.


fix: fallback handler type

What this PR fixes
For Safe's without Fallback Handler the fallbackHandler field is null.

Please also bump the version in package.json.

Released v3.5.5 📦


Possible to Use without React?

All the docs use React, but how would one use driftdb without a view library like react, simply using vanilla JS?
Please include some documentation for users who don't want to use Meta flux and related tools

Docs are now published here: https://driftdb.com/docs/js
I'll close this but feel free to follow up if you have other questions.


[controller][server] Add metric to track pub-sub admin operation failures

Add metric to track pub-sub admin operation failures
Added a metric: pub_sub_admin_op_failure_count to track the count of failed admin operations. This will allow us to monitor these failures and set up alerts.
How was this PR tested?

Does this PR introduce any user-facing changes?

[x] No. You can skip the rest of this section.
[ ] Yes. Make sure to explain your proposed changes and call out the behavior change.

Thanks a lot, @bharathkk for the review and discussions.


IEM to IMD

@egonw ; perhaps the name of this repo should also be changed? But I don't have access to that (yet).

We still need this subdomain.


[Feature Request] Inherited permissions

Describe the feature you'd like
Currently when creating a page and/or a chapter under a shelf you have to manually copy the permissions each time a new one is created.
Describe the benefits this feature would bring to BookStack users
The benefit helps all users having to perform one less repetitive task and stop human error (as we all forget from time to time)

Hi @athornfam2,
Please see #1091 and  #2652 for context around this request. Unless you have a solution to address the concerns or technical challenges I'll soon close this off.

Hello @ssddanbrown
After reading through both cases. Am I correct in understanding that Shelves do not have inherited permissions but books do?

@athornfam2 Yeah, Books and chapters will cascade their permissions down unless overridden.
Shelves do not do this due to the fact that a book can be in many shelves.

@ssddanbrown Understood. I'd say this request can be closed then.


OTR: Missing parent? ligand-dependent nuclear receptor binding/TF binding

Ligand-bound nuclear receptors are transcription factors https://en.wikipedia.org/wiki/Nuclear_receptor. Currently GO:0016922 ligand-dependent nuclear receptor binding is a child of receptor binding.
Could you add an Is_a relationship to: transcription factor binding GO:0008134
ligand-dependent nuclear receptor binding  GO:0016922

Hi @hattrill
I added the parent 'nuclear hormone receptor binding'. The term you mention 'GO:0016922 ligand-dependent nuclear receptor binding', refers to a binding that is 'ligand-dependent'; so it's in fact 'ligand binding-dependent nuclear receptor binding'. I am not sure this is how the term has been used. Is this something important for nuclear receptors (ie are there many protein bindings that are dependent on the ligand binding?
Thanks, Pascale

Ok, but we should review this term as part of the signaling project when we do these pathways.

I checked a few annotations and they were OK, so I will not create a ticket for this just yet.
Thanks, Pascale


⚠ Bofrost.de has degraded performance

In d13ba54, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1196 ms

Resolved: Bofrost.de performance has improved in 17ca96a.


feat: add user password in ldap server search result

fix #1508, and add user password in ldap server search result to support authentication in some ldap client like freeradius.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Shenyz seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@imshenyz plz fix:


Support more point values for grinder

0.1 not just 0.5

v42 will support 0.1 and default to 15.0 because it's a bit more annoying with this.
Grinder templates are going to be key here.


Add bokeh plot to ForcedResponse() class

This commit implements plotting using bokeh to generate magnitude and phase plot to ForcedResponse class.

Codecov Report

Merging #116 into master will decrease coverage by 0.41%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #116      +/-   ##
==========================================
- Coverage   61.32%   60.91%   -0.42%
==========================================
  Files           9        9
  Lines        1766     1778      +12
==========================================
  Hits         1083     1083
- Misses        683      695      +12

Impacted Files
Coverage Δ

ross/results.py
15.27% <0%> (-0.7%)
:arrow_down:

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 d589a35...68b44ca. Read the comment docs.


missing *.mat files

Cant find these files:
FXLMS.m:91:  x=importdata('SEC13R.mat');    % Input x(n) from data file

FXLMS_tool.m:91:x=importdata('SEC18R.mat');    % Input x(n) from data file

I don't know - is it necessarily files or not, originally it commented in sources:
FXLMS.m:69:%    "TF.mat" contains transfer function of P(z) and S(z)

FXLMS_tool.m:69:%    "TF.mat" contains transfer function of P(z) and S(z)


merbridge is a simple tool to acclerate your mesh with eBPF

https://github.com/merbridge/merbridge/
Merbridge: use eBPF to accelerate your mesh
https://landscape.cncf.io/card-mode?category=service-mesh&grouping=category&selected=merbridge
It is also included in CNCF landscape.
I'm not sure which part should I add this project to.

https://istio.io/latest/blog/2022/merbridge/

Yes, makes sense to add Merbridge. Section Projects Related to eBPF is likely the most appropriate. It's in no particular order, so you could append at the end or try to insert it close to something similar, if any.


Add myself to the list of fans

Hi @yegor256. Been long-time fan of your work. I've added myself to the list of fans. :)

@MrHadiSatrio thanks! :)


increase CI test coverage for connection target types

give the weird behavior we saw with connection strings and SqlPassword=True. @NandanHegde15 had the idea to have a separate Circle CI job would be verify that dbt debug works on all target configurations that are commonly used. We can either add this to #62 or in a new PR after #62 is merged.
The ones I can think of:

[ ] trusted_connection=True
[ ] windows_integrated=True
[ ] when Encrypt and TrustServerCertificate aren't specified
[ ] the various combos ofEncrypt, TrustServerCertificate, and SqlPassword
[ ] Active Directory

[ ] Password
[ ] Integrated (trickier as it would require a VM and an AD user in the same RG as the Azure SQL server)
[ ] Interactive

That would be really great. It would really help improve stability on releases to be confident that this doesn't break. I think after #62 is merged. To make the merges smaller.

@jtcohen6 could you help us think how to best set up running dbt debug against all permutations of target configs? for example, we want to make sure that the connections.py logic works whether:

the values are true or false, and if
the params aren't specified.

encrypt: true
trust_cert: true

we'd also need to do the same with all the AAD auth methods...

@swanderz I don't think dbt debug returns nonzero exit codes, but you could do it just as well by executing a dbt compile command with each permutation.
I think you could do that outside of the pytest framework to start: create a sample.profiles.yml with a whole range of targets. In your Circle steps, copy that profile and execute dbt compile --target {target} for each.

Is anyone working on this? I would really like to push a new version to pypi. But then we need to make sure that all functionality we are shipping are working as expected...

Is anyone working on this? I would really like to push a new version to pypi. But then we need to make sure that all functionality we are shipping are working as expected...

working on this today!

working on this today!


System76 Gazelle Gaze 18

Description of changes
trying to add support for the System76 Gazelle model Gaze18
Things done
added the start of a functioning nvidia configuration. Vulkan does not seem to be working.

[x] Tested the changes in your own NixOS Configuration
[x] Tested the changes end-to-end by using your fork of nixos-hardware and
importing it via <nixos-hardware> or Flake input

looks good to me, but I still haven't managed to make vulkan work.

@mergify queue


Documentation improvements for valid column types

Added uuid to the list of columns
Added jsonb to Postgres
Sorted column types alphabetically

IMO uuid should not be added to the list here as that is not universal.
It's already noted that uuid is available for the Postgres adapter
On Fri, Jul 24, 2015 at 5:00 AM, Leonid Mamchenkov<EMAIL_ADDRESS>
wrote:

Added uuid to the list of columns
Added jsonb to Postgres
Sorted column types alphabetically

You can view, comment on, or merge this pull request online at:
https://github.com/robmorgan/phinx/pull/608
Commit Summary

Added uuid to the list of supported columns
Added jsonb column type to Postgres
Sorted column types alphabetically
Duplicated changes into seeding.rst

File Changes

M docs/migrations.rst
https://github.com/robmorgan/phinx/pull/608/files#diff-0 (21)
M docs/seeding.rst
https://github.com/robmorgan/phinx/pull/608/files#diff-1 (21)

Patch Links:

https://github.com/robmorgan/phinx/pull/608.patch
https://github.com/robmorgan/phinx/pull/608.diff

—
Reply to this email directly or view it on GitHub
https://github.com/robmorgan/phinx/pull/608.

@w2pc to me it looks like the documentation is misleading, since it says that only Postgres supports UUID.  We are using UUID just fine with MySQL and SQL Server.  I took a quick look around and it seems that SQLite adapter supports it too (which makes it 4 out of 4).  PDO adapter, which all other ones are extending, supports it.  And the phinx type constant is defined in AdapterInterface, which all adapters implement.
To me it looks pretty universal.

But a UUID in MySQL is a column type of 16byte "binary" which is already in
the column types.
On Fri, Jul 24, 2015 at 6:34 AM, Leonid Mamchenkov<EMAIL_ADDRESS>
wrote:

@w2pc https://github.com/w2pc to me it looks like the documentation is
misleading, since it says that only Postgres supports UUID. We are using
UUID just fine with MySQL and SQL Server. I took a quick look around and it
seems that SQLite adapter supports it too (which makes it 4 out of 4). PDO
adapter, which all other ones are extending, supports it. And the phinx
type constant is defined in AdapterInterface, which all adapters implement.
To me it looks pretty universal.
—
Reply to this email directly or view it on GitHub
https://github.com/robmorgan/phinx/pull/608#issuecomment-124494117.

@w2pc not all databases support UUID natively.  Those that don't, are using char(36).  The adapters are mapping this difference pretty nice, so that from the point of view of migrations, you don't even need to know if the engine supports uuid column type.
Currently, the documentation suggests that uuid column type is only supported by Postgres, so you probably should (implicitly) check the database engine in your migration and use char(36) for everything that is not Postgres.  Which is not the case.
Just trying help people keep migrations simple and abstract. :)

Just checked to be sure, and MysqlAdapter uses char(36) for UUID:

Line 814
Line 871


Define object with props as an optional argument

Thanks for this plugin! I sleep much better now :)
I have 3 questions. Will split them in 3 issues. Here is first one.
Is it possible to define object with props as an optional argument?
export default function dummiesReducer(
  $state: $Map = $initialState,
  {
    type,
    index,
    errors,
  } : {
    type  : string,
    index : ?Array<number>,
    errors: ?{ code: number, data: Object }, // <--- if arg is passed - check its props
  }
) {
  // ...
}

@alexfedoseev thanks for the bug reports. Our support for object props / default args is not very good at the moment - if it works, it's mostly be happy accident. Really this whole plugin will need a refactor when babel6 comes out and I'd really like to tackle this then.

if it works, it's mostly by happy accident

Haha, waiting for Babel 6 then. Thanks for great work!

Looks like this workaround works:
function({ errors } : { errors: void | { code: number, data: Object } }) {}

Haha, waiting for Babel 6 then.

It just came out today, but I'm waiting a day or two for things to calm down before starting with it.

Great, thanks! :+1:


[Infra] Rotate ServiceAccount.json secret used by RC integration tests

b/229016700
#no-changelog

The console tests passed in CI so this is good to go.


EMNIST Dataset issue

Problem encountered on https://dotnet.microsoft.com/learn/ml-dotnet/get-started-tutorial/data
Operating System: windows
I wanted to use model builder for training my model, when i give it the emnist dataset it doesnt let me train the data, it just doesnt enable the next option to train the data, and it doesnt show the data preview! any help? any suggestions?

Can you share screenshots and the dataset?

Link to the dataset: https://www.kaggle.com/crawford/emnist
I cannot provide screenshots at the moment, the problem was that train data was not being enabled as i suspect the dataset wasn't being read properly, so there is an issue.

This isn't fixed. Loading the data preview visualization locks takes forever and locks VS.

The perf for large datasets has been improved! Please try our newest build https://marketplace.visualstudio.com/items?itemName=MLNET.07.
I've tested with datasets as large as EMNIST. Please give it a try and let me know if you run into problems. Thanks!


.ts file is being transpiled in different ways

TypeScript Version:  2.8.3

Search Terms:
same file different results
Code
I have this source file:
import chai = require('chai');
import AssertStatic = Chai.AssertStatic;

export {InjectParam} from "./test-suite-params/inject/inject-param";
export {EachHookParam} from "./test-suite-params/each-hook/each-hook-param";
export {AllHookParam} from "./test-suite-params/all-hook/all-hook-param";
export {TestCaseParam} from "./test-suite-params/test-case/test-case-param";
export {ITestSuite} from 'suman-types/dts/test-suite';
export {ItFn, ITestDataObj} from 'suman-types/dts/it';
export {IDescribeFn, IDescribeOpts, TDescribeHook} from "suman-types/dts/describe";
export {IBeforeFn} from 'suman-types/dts/before';
export {IBeforeEachFn} from 'suman-types/dts/before-each';
export {IAfterFn} from 'suman-types/dts/after';
export {IAfterEachFn} from 'suman-types/dts/after-each';
export {DefineObject} from "./test-suite-helpers/define-options-classes";
export {DefineObjectContext as DefObjContext} from "./test-suite-helpers/define-options-classes";
export {DefineObjectTestCase as DefObjTestCase} from "./test-suite-helpers/define-options-classes";
export {DefineObjectAllHook as DefObjAllHook} from "./test-suite-helpers/define-options-classes";
export {DefineObjectEachHook as DefObjEachHook} from "./test-suite-helpers/define-options-classes";

Sometimes it transpiles to this (this is the desired output):
"use strict";
Object.defineProperty(exports, "__esModule", { value: true });
var inject_param_1 = require("./test-suite-params/inject/inject-param");
exports.InjectParam = inject_param_1.InjectParam;
var each_hook_param_1 = require("./test-suite-params/each-hook/each-hook-param");
exports.EachHookParam = each_hook_param_1.EachHookParam;
var all_hook_param_1 = require("./test-suite-params/all-hook/all-hook-param");
exports.AllHookParam = all_hook_param_1.AllHookParam;
var test_case_param_1 = require("./test-suite-params/test-case/test-case-param");
exports.TestCaseParam = test_case_param_1.TestCaseParam;
var define_options_classes_1 = require("./test-suite-helpers/define-options-classes");
exports.DefineObject = define_options_classes_1.DefineObject;
var define_options_classes_2 = require("./test-suite-helpers/define-options-classes");
exports.DefObjContext = define_options_classes_2.DefineObjectContext;
var define_options_classes_3 = require("./test-suite-helpers/define-options-classes");
exports.DefObjTestCase = define_options_classes_3.DefineObjectTestCase;
var define_options_classes_4 = require("./test-suite-helpers/define-options-classes");
exports.DefObjAllHook = define_options_classes_4.DefineObjectAllHook;
var define_options_classes_5 = require("./test-suite-helpers/define-options-classes");
exports.DefObjEachHook = define_options_classes_5.DefineObjectEachHook;

but sometimes it transpiles to this instead: ( this is the undesired output )
"use strict";
Object.defineProperty(exports, "__esModule", { value: true });
var inject_param_1 = require("./test-suite-params/inject/inject-param");
exports.InjectParam = inject_param_1.InjectParam;
var each_hook_param_1 = require("./test-suite-params/each-hook/each-hook-param");
exports.EachHookParam = each_hook_param_1.EachHookParam;
exports.AllHookParam = all_hook_param_1.AllHookParam;
var test_case_param_1 = require("./test-suite-params/test-case/test-case-param");
exports.TestCaseParam = test_case_param_1.TestCaseParam;
var test_suite_1 = require("suman-types/dts/test-suite");  // !!! this is causing a runtime error
exports.ITestSuite = test_suite_1.ITestSuite;
var it_1 = require("suman-types/dts/it");
exports.ItFn = it_1.ItFn;
exports.ITestDataObj = it_1.ITestDataObj;
var describe_1 = require("suman-types/dts/describe");
exports.IDescribeFn = describe_1.IDescribeFn;
exports.IDescribeOpts = describe_1.IDescribeOpts;
exports.TDescribeHook = describe_1.TDescribeHook;
var before_1 = require("suman-types/dts/before");
exports.IBeforeFn = before_1.IBeforeFn;
var before_each_1 = require("suman-types/dts/before-each");
exports.IBeforeEachFn = before_each_1.IBeforeEachFn;
var after_1 = require("suman-types/dts/after");
exports.IAfterFn = after_1.IAfterFn;
var after_each_1 = require("suman-types/dts/after-each");
exports.IAfterEachFn = after_each_1.IAfterEachFn;
var define_options_classes_1 = require("./test-suite-helpers/define-options-classes");
exports.DefineObject = define_options_classes_1.DefineObject;
var define_options_classes_2 = require("./test-suite-helpers/define-options-classes");
exports.DefObjContext = define_options_classes_2.DefineObjectContext;
var define_options_classes_3 = require("./test-suite-helpers/define-options-classes");
exports.DefObjTestCase = define_options_classes_3.DefineObjectTestCase;
var define_options_classes_4 = require("./test-suite-helpers/define-options-classes");
var define_options_classes_5 = require("./test-suite-helpers/define-options-classes");
exports.DefObjEachHook = define_options_classes_5.DefineObjectEachHook;

Expected behavior:
I would expect that given the same input file, and given the same version of tsc, the file to be transpiled to the same output.
Actual behavior:
For some reason (probably a good one), the input is transpiled to different output.
The ultimate problem is that my program ends up trying to load .js files, but all there is to load is .d.ts files, so I get a
module not found error.

my only hypothesis at the moment, is that maybe in a certain scenario, tsc cannot load the .d.ts files, so it just assumes that they are .js files instead?

I am afraid this is pretty vague. please provide some concrete repro steps, configuration used, how you are building etc.. for us to be able to debug the issue.

@mhegazy sure here is my tsconfig.json file:
{
  "compilerOptions": {
    "outDir": "dist",
    "skipLibCheck": true,
    "declaration": true,
    "baseUrl": ".",
    "typeRoots": [
      "./node_modules/@types",
      "./node_modules/suman-types/dts"
    ],
    "target": "es6",
    "module": "commonjs",
    "noImplicitAny": false,
    "removeComments": true,
    "allowUnreachableCode": true,
    "lib": [
      "es2015",
      "es2016",
      "es2017",
      "dom",
      "es5",
      "es2015.promise"
    ]
  },
  "compileOnSave": false,
  "include": [
    "src"
  ]
}

and I am just building with tsc

It's possible that when it compiles, it is missing
"./node_modules/suman-types/dts"

the types should not change the generated output. so something else is going on. do you have a consistent repro?

yeah I have a consistent repro, it's happening when my Github repo is running on Travis.
Do you want me to try to trim it down to just a few files?

I think my original hypothesis is correct actually. I fixed it by including the
   "./node_modules/suman-types/dts"

in my build. Does that make any sense?

Do you want me to try to trim it down to just a few files?

that would be ideal. thanks!

I know what's going on, it is failing if "./node_modules/suman-types/dts" is missing, but in some cases on CI/CD platforms, node_modules gets cached so it will succeed on second try.
I don't think we need a repro since I know what's happening, we just need to discuss why the transpile would be wrong if .d.ts files are missing.

I don't think we need a repro

We do.

I can get you the repro, but I wanted to discuss first if this is simply expected behavior - it might very well be. If .d.ts files are missing from the build, will tsc sometimes assume they are .js files or .ts files instead of .d.ts files?

I'm not sure if this is related, but with 2.9.1 I'm running into an issue where previously working code, is now dropping a specific import causing the code to break.  I'm trying to narrow down to a better repro case.  The odd part, is the referenced variable from the import remains in the file, but the import clause does not.

If the export is type only, i.e. only an interface or a type alaise, and it is originating in a .d.ts file. the compiler needs to the .d.ts file to know whether to elide the export or not.
if this is the case you have, then that is expected. for anything else, no differences should be there.

@arciisine please file a new ticket and give us additional information about the issue you are running into.

I created a REPRO for this:
https://github.com/ORESoftware/ts_bug_repro
the problem is real. whether the problem is a bug or a feature is TBD

Basically in the readme file of the repro project I linked to above, I think tsc is doing the correct thing - if it can't find a .d.ts file, it then simply assumes that it is a missing .js or .ts file, so it generates the code to load that file. I think that if people care about this problem, they would simply listen to the tsc output for errors, and as long as tsc always complains that it cannot find a file, then it's the users fault for ignoring the tsc compilation error.

As I mentioned the reason I am running tsc before running npm install is because of this chicken and egg problem:
https://stackoverflow.com/questions/50616253/catch-22-situation-with-tsc-and-npm-install
any insight into how to best solve that S.O. issue would good to know. Because that is what caused me to not run npm install in the first place.

thanks @ORESoftware for the repro. looks like the issue is what i have described  in https://github.com/Microsoft/TypeScript/issues/24513#issuecomment-393692390. for type only exports to be elided, the compiler needs to have access to the declaration site.


Change /std:latest to /std:c++20 for affected STL topics

Waiting on some libraries, like format(), to have /std:c++20 applied to them.
Right now they are listed as /std:latest.

Tracking in quest


Question about the repository

This repository looks cool :)
I'd love to participate and help, but it's kind of hard to understand what it's really about. Is it a collection of functions? A set of interconnected classes with a major purpose? How can your code help people, exactly?
Best regards.

Thanks! I started this repo trying to put together tools/functions related to spatial data mining (trajectory data mining).
As I'm currently a master's student, this mostly reflects scientific works that my colleagues and I have been studying in our research or that somehow supports experiments I did.
Although it's a bit raw and very "academically driven", I think it would be cool if we could shape it and implement methods for real-world problems (I'm not sure if there is an open-source lib for that purpose).
What do you think?

I think it's an interesting idea.
I didn't think it was an academic project because I rarely see academic projects with meticulous coding practices, such as PEP8/Codacy and on-the-fly continuous integration tests.
The long-term shape of the repository, of course, would heavily depend on the initial, current and future needs of your team. But we can see what time will tell.
By the way, have you checked GeoPy yet?

Well, let's say I started it after having trouble finding (or even coding) other people's code and making it work with my data, so it's also sort of "a favor" for future students haha.
I've set up the tests and code style checks because I did think it's important to start caring about that stuff from the beginning, but we actually don't have any tests yet.
I'm not sure I knew about GeoPy, but it looks like it's mainly about geocoding.

I see. I mentioned GeoPy because it implements a Geodesic distance and I didn't know what it was. But I did a quick research and it seems to be the same as the Haversine distance, already implemented here.
Thank you for all the answers! By the way, I think a good way of showing what the project is about would be creating a few straightforward examples to display what kind of problems the lib can help solve. Of course, this takes time and your team has its own priorities.

Hi, @lucaspetry! I'm also a master's student and I will be dealing with semantic trajectories in my project. I found this library through the paper "Towards semantic‐aware multiple‐aspect trajectory similarity measuring", and considered your initiative really interesting!
I will use an ontology-based representation and explore some data mining tasks in my project. Now I am thinking about using Trajminer to do data manipulation, and as an abstraction layer of my project.
I was going to open an issue presenting myself, but then found this old topic. Let me know the best way to share my project's requirements and to contribute to this lib. May I just open a bunch of issues?


Add name parameter docs

Groups and Users info calls have included search by name for a number of versions already.
Channels would as well once pull 6827 is merged.

@graywolf336 Any chance for another review? I updated the commits a month ago.

I apologise for that, thanks for the reminder

Thanks for the pull request. I'll see about fixing the conflicts unless you want to.

@graywolf336 rebased and dropped the conflicting Users.info change. abrom added username there already.


Docker Compose

Consider using docker compose for stability and convenience.
Current behavior

Mount points for docker run -v <src>:<trgt> have to be defined by user.

Expected behavior
- Consider using `docker compose` yaml to provide easy runtime instructions as to avoid having users/developers do this with every `docker run` instance and to enforce target mount points inside containers.
- See [this](https://docs.docker.com/get-started/08_using_compose/) for more information on docker compose.
- Account for any changes to bind points that may arise from #12.

Docker is not used often as of now so closing this issue


MATCHED_VERTEX and ADJACENT_MATCHED_VERTEX cannot be used in a view filters

The OperationChain validation fails when checking the input classes of the filters.

Merged into master.


fix: hide age when installed is newer than recommended

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


Greengrass discovery sample appears to not be working after latest updates

I have deployed GreenGrass service v1.9.2 on a raspberry pi with the device sdk cpp v2 for a proof of concept of this technology and was able to successfully discover and publish to the GreenGrass group setup on the AWS cloud account (using commit 42ec5c5).
Since the latest version and code changes (14 days ago) i am unable to publish and am getting the following error: "Connection setup failed with error libaws-c-common: AWS_ERROR_FILE_INVALID_PATH, Invalid file path". This message is printed out from the code when it is unable to make a new mqtt client connection. I checked the paths that i am passing to the connection but cannot determine which one, if any are incorrect, since the same paths are passed in the previous commit of the code and it works (if that is what the error is referring to).
I also have a question about the distinction of the discover connection and the mqtt connection which i believe are on ports 8443 and 8883 respectively. If this is the case why is the mqtt connection created within the scope of the discover? I assume since discover is supposed to be once off that in a normal situation you would discover once and then create the mqtt connection outside of the scope of the discover connection or does this have to be done so that the pub/sub goes through GreenGrass (and not direct from the sdk to cloud)?
Could you please assist as soon as possible, as this is quite urgent.

Without a log trace, my best guess is that error is coming from the code trying to load the CA file into your local trust store. We did not previously have any validation or error cascade when the trust store import failed, but we do as of the most recent releases. Offhand, it seems like the CA might have already been in the trust store on the device in your prior tests, or at some point in the past, and is failing now?
Is there any way you can enable logging at trace level and paste a trace?

Thank you for the reply.
I am not certain how i would be able to provide with more logging information for the the included aws libraries. Although i did notice there was a slight change in the way you passed the CA file in the latest and previous samples. The latest code you just pass the path to the CA file and c_str() it whereas the previous commit uses an in-memory buffer. I dont think this makes any difference though, just for interest sake if i try to pass the path as a char i get the exception i mentioned above, if I try the ByteCursor i get a segmentation fault.
You mentioned in your comment about the CA already being in the trust store, in the sample you override the default trust store with the path to the new CA file, does this not ensure you read the new CA file?
I have actually been stuck with this since two weeks ago after the code change, are you able to confirm if the problem is with the latest commit. I would really like to be able to provide you with more logging details (and hope the error is some simple mistake on my side) but it seems to be in the mqttclientconnectionconfigbuilder (with certificateauthority) method when getting the CA.

When you initialize your Aws::Crt::ApiHandle, call InitializeLogging(...) before starting anything up. This will let you either log to stdout, or to a file, which you can then recover the errors from. It's going to be most informative to use LogLevel::Trace for diagnosing this issue. Once you have a log, please post it here, and we can try to diagnose what you're seeing.

So here is the last part of the trace (id and cert redacted) as i believe the rest is irrelevant.
It appears as indicated that the root CA failed to be read. A small observation, the cert indicated in this message is not the same as ATS root.ca.pem (is it not supposed to be?)
[TRACE] [2019-07-31T13:42:51Z] [1977611232] [tls-handler] - id=xxxxxxxxxx: Downstream window<PHONE_NUMBER>
[TRACE] [2019-07-31T13:42:51Z] [1977611232] [channel] - id=xxxxxxxxxx: acquired message xxxxxxxxxxof length 0 from pool xxxxxxxxxx. Requested size was<PHONE_NUMBER>
[TRACE] [2019-07-31T13:42:51Z] [1977611232] [tls-handler] - id=xxxxxxxxxx: Bytes read 2489
[TRACE] [2019-07-31T13:42:51Z] [1977611232] [channel] - id=xxxxxxxxxx: sending read message of size 2489, from slot xxxxxxxxxxto slot xxxxxxxxxxwith handler xxxxxxxxxx.
[TRACE] [2019-07-31T13:42:51Z] [1977611232] [http-connection] - id=xxxxxxxxxx: Begin processing message of size 2489.
[TRACE] [2019-07-31T13:42:51Z] [1977611232] [http-stream] - id=xxxxxxxxxx: Incoming head is done, body is next.
[TRACE] [2019-07-31T13:42:51Z] [1977611232] [http-stream] - id=xxxxxxxxxx: Incoming body: 2489 bytes received.
[DEBUG] [2019-07-31T13:42:51Z] [1977611232] [http-stream] - id=xxxxxxxxxx: Client request complete, response status: 200 (OK).
[ERROR] [2019-07-31T13:42:51Z] [1977611232] [file-utils] - static: Failed to open file -----BEGIN CERTIFICATE-----
MIIEFTxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx
-----END CERTIFICATE-----
with errno 36

I’m assuming this is the CA returned by greengrass? And so you’re taking the CA value and passing it as a const char* instead of as a byte cursor? Const char* overload is for file paths e.g. “/etc/ssl/certs”. The byte cursor api is for pem buffers (the one you want here)

But the reason it worked before vs now is purely accidental. In reality this never worked, you just happened to already have the CA in your default trust store and the connection happened to work. The difference here now is that we caught the error.

Thanks for the response.
So just to clarify I have tried passing in the byte cursor but i get a segmentation fault, so it doesn't appear to be working correctly. I am also not certain i understand what you mean by it accidentally worked before. The ATS root certificate for greengrass was installed on a new device. I ran the latest master commit of the sdk v2 and it failed, i reverted to the previous commit of sdk v2 and it worked. No change in adding CA to trust store (unless this is something to do with the code calling overridedefault trust store in the tlscontextoptions)?
I am also having a similar issue on a linux machine where a memory address exception is being thrown when the groupToUse.CAs->at(0) is accessed, regardless of commit version.
Please let me know how this can be resolved.

Can you show your code for bootstrapping the TlsContextOptions? Specifically all of the code leading up to OverrideDefaultTrustStore()?

So the code can be found here https://github.com/awslabs/aws-iot-device-sdk-cpp-v2/blob/master/samples/greengrass/basic_discovery/main.cpp, on the master branch. I am using it exactly as is, to test on the device before refactoring for my use case (the caFile is the greengrass root.ca.pem i.e. /greengrass/certs/root.ca.pem) .

Any thoughts on this yet? So to clarify there are two issues that appear to be related (unable to read root CA cert) on different platform architectures, the more urgent one is to get basic discovery working on the arm, thereafter to fix it for the x86_x64.

Apologies for the latency, we are still investigating. The sample runs fine for us in CI, so we're looking into what else could be going on here. We'll update the ticket when there's new info.

Should be fixed in latest commit

This doesn't appear to be fixed, i tried one more time and got a segmentation fault when the connection is built, indicating a problem with the CA again perhaps.
I am going to abandon this for now while using an older stable version until this is made generally available. If you have an opportunity please check this discovery sample again.

Yesterday's push contains some stability fixes that were affecting the discovery sample (along with http proxy support)

Any response on my previous comment yet?

Are you running the sample? Or your own code? Which version of greengrass are you running? Which QOS value are you using?

Thank you for replying.
So everything that i have mentioned before on this thread is still relevant.
I am still running the latest discovery sample, still using the same greengrass version 1.9.2, i have not modified the QoS so it should still be 0, but i am getting the above error when trying to run the discovery sample.
From the above comments you can see that various updates have been applied to the code but it doesn't appear to be working. I am using the only stable version (42ec5c5) that is still working but would like to use the latest to get access to the new device shadow functionality too.
I also have an unanswered question relating to the specific line that was previously failing (see above) with regards to the groupsToUse.CA() value. Where does this certificate value come from? If i want to create a connection with doing a discovery where do i get that certificate from, as its not the greengrass core certificate, nor root.ca.pem, nor device certificate???

What OS/version is your greengrass core running on?

Raspbian (i am aware of the greengrass container issue but the kernel was not updated which is why we can still use v1.9.2 of the GG core).

When you try the latest (and it fails for you), how are you building the sdk: are you using BUILD_DEPS=1 or are you building and installing the aws crt by hand?
Have you tried the discovery sample against v1.8 of greengrass core?

I have been testing the latest SDK on a pi with Raspbian, using the instructions in the readme with BUILD_DEPS=ON. I followed this same process for all versions since 42ec5c of the code. This was the only version that worked (subsequent versions have the errors that i have documented above in this thread).
On our development environment we build the dependencies separately.
I have not tried the discovery sample against v1.8, but we are now using the new raspian kernel so we are going to be upgrading to v1.9.3 soon.

Also, when using the latest version, can you enable Trace level logging and attach the log file from when you run the discovery sample?  It should just need a one-line addition after the ApiHandle declaration:
apiHandle.InitializeLogging(LogLevel::Trace, "/tmp/discovery.log");

Yes i will do that (i don't have access to the test environment at the moment so will send tomorrow).


Pin the docs requirements

Following chat on slack, I've introduced pip-tools to the docs requirements and generated a new requirements file to ensure pinned versions.

[x] I have opened this pull request against develop
[ ] I have added or modified the tests when changing logic
[ ] I have followed the conventional commits guidelines to add meaningful information into the changelog
[ ] I have read the contribution guidelines  and I have joined #workgroup-pr-review on Slack to find a “pr review buddy” who is going to review my pull request.

Sounds good for me! :+1:

I'm adding some docs to this PR


Common generate script

While creating a generator for acronym and noticed how each exercise had it own generate script. This leads to a lot of code duplication since each script is almost identical.
This PR would allow for a single generate script that takes an exercise name as the argument, and regenerates the exercise.  If the exercise is not found and error message will display.  I tested it out on all the exercises and I had to modify two generators to ensure consistent naming.
Example usage:
bin/generate raindrops
Let me know if this is something that would be helpful!

Nice @bmulvihill, This is something I'd been planning on doing when I found some time, but it looks like you beat me to it.

@kotp @insi the contributing doc would need to change slightly.  I can update it in this PR.  I can also remove all the old generate scripts.

That would be awesome!

@kotp @Insti Ok this should be good to go

Was blocking #454


fix: relax towards older crc32c

@johanneskoester if you have bandwidth, I am done with the intel MPI example but blocked here: https://github.com/snakemake/snakemake-executor-plugin-googlebatch/issues/18. I think there might be a bug with this example that I got from snakemake (the output is generated but empty).


Python

Hi! Add icon for *.py files, pleace

It's already added. Please, download the latest version of the plugin, then Disable and Enable the icons and restart vscode.


Access all search equations from Python code

v 1.3 Return equations from all parallel hillclimbers.

version 1.3.0


Daily mini-contest

At a specified time daily, put up a list of (say 5) problems to be solved as per cf rating. The earlier your AC the more points you get. Maintain leaderboard per rank. Essentially mini practice-contests within the server.

Daily is too optimistic? :)
Maybe once a week so that people get time to solve harder problems also.

Another major issue: how do you make sure that among all the members who will probably solve the weekly problems, they have not already solved this particular problem that you're gonna give them?
One way can be to propose three problems in each set and ask the user to solve any one of them. It's unlikely anyone solved all three. Or if they've already solved the problem they can request a new one.

Look at this pull request if yuo want to make something similar : https://github.com/cheran-senthil/TLE/pull/311


The select resources combobox is duplicating records

The select resources combobox is reordering the resources alphabetically but duplicating records.

Solved on https://github.com/WingLongitude/lontra-harvester/commit/779f45385363e96d97e21ea96e4758df754464a7


Misleading zip error message for broken links

I tried sdk install ant and got this output:
Downloading: ant 1.9.7

In progress...

######################################################################## 100.0%
  End-of-central-directory signature not found.  Either this file is not
  a zipfile, or it constitutes one disk of a multi-part archive.  In the
  latter case the central directory and zipfile comment will be found on
  the last disk(s) of this archive.
unzip:  cannot find zipfile directory in one of /home/Jonathan/.sdkman/archives/ant-1.9.7.zip or
        /home/Jonathan/.sdkman/archives/ant-1.9.7.zip.zip, and cannot find /home/Jonathan/.sdkman/archives/ant-1.9.7.zip.ZIP, period.

Stop! The archive was corrupt and has been removed! Please try installing again.

I looked up this message and found a number of old tickets about wrong version of zip. I'm using cygwin and I checked that I do have the zip and unzip packages from the archive category. I edited the sdkman-install.sh script to skip deleting the "corrupted" zip file and I found that what actually got downloaded was this:
<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">

<title>404 Not Found</title>

Not Found

The requested URL /apache/ant/binaries/apache-ant-1.9.7-bin.zip was not found on this server.

<address>Apache/2.4.10 (Debian) Server at mirror.vorboss.net Port 80</address>

I checked the download URL with curl and got this output:
$ curl -L -I https://api.sdkman.io/2/broker/download/ant/1.9.7/cygwin_nt-10.0
HTTP/1.1 302 Found
Server: nginx/1.11.5
Date: Sat, 07 Jan 2017 04:02:12 GMT
Content-Length: 0
Connection: keep-alive
location: http://mirror.vorboss.net/apache/ant/binaries/apache-ant-1.9.7-bin.zip

HTTP/1.1 404 Not Found
Date: Sat, 07 Jan 2017 04:02:13 GMT
Server: Apache/2.4.10 (Debian)
Content-Type: text/html; charset=iso-8859-1

So it appears the root cause of my issue is that the broker is redirecting to a URL that doesn't have the requested package, and the zip error is shown because what got downloaded from the server is some HTML instead of a zip file.
I made a couple of changes in sdkman-util.sh and sdkman-install.sh to detect the HTTP error and produce a more accurate message.

Please see my pull request #519 for the changes

Thanks for reporting this. Comments left in PR.

Hi, my issue seems related.
On a fresh install of minimal ubuntu server sdk install properly , sdk java isnall works, but when i did sdk install grails i got this.
machi@groovyMachi:~$ scren -ls
No command 'scren' found, did you mean:
Command 'screen' from package 'screen' (main)
scren: command not found
machi@groovyMachi:~$ screen -ls
There is a screen on:
2842.pts-0.groovyMachi  (06/27/2017 12:14:42 AM)        (Attached)
1 Socket in /var/run/screen/S-machi.
machi@groovyMachi:~$ sdk install grails
Downloading: grails 3.2.11
In progress...
############                                                              17.5%
curl: (56) GnuTLS recv error (-54): Error in the pull function.
End-of-central-directory signature not found.  Either this file is not
a zipfile, or it constitutes one disk of a multi-part archive.  In the
latter case the central directory and zipfile comment will be found on
the last disk(s) of this archive.
unzip:  cannot find zipfile directory in one of /home/machi/.sdkman/archives/grails-3.2.11.zip or
/home/machi/.sdkman/archives/grails-3.2.11.zip.zip, and cannot find /home/machi/.sdkman/archives/grails-3.2.11.zip.ZIP, period.
Stop! The archive was corrupt and has been removed! Please try installing again.
machi@groovyMachi:~$
Any workarounds? the whole sdk install grail is broken to me.

same issue here installing jetbrains java through sdkman
when installing sdkman on gitbash i created a symbolic link named "zip.exe"my to 7zip using powershell (as admin privilege)
New-Item -ItemType SymbolicLink -Path "C:\Program Files\Git\mingw64\bin\zip.exe" -Target "C:\Program Files\7-Zip\7z.exe"


Hexadecimal strings are coerced into (incorrect) numbers

Hi there,
First off: great work.
We've bumped into a bit of an issue when trying to pass long hexadecimal strings. When passing these, mongo-querystring will coerce this into a number. Example: 0xfbede9b15fbeec9e1579653ac3af52b0a54ec2a3725150c64e681264cb18cbf6 gets converted to 0, which is clearly not the intention of the caller.
Ideally (but this is debatable), I believe values in hexadecimal notation should/could be passed on as strings instead of numbers. Most of the time in Mongoland, this kind of value is stored as a string (eg a hash of some sorts).
Questions that remain could be:

How do we detect hexadecimal notation? (is 0x enough or do we want to go more in-depth?)
What if we want this to convert to binary instead (some store this kind of value as binary instead, but I'd like to think this is a rare case)
Should we strip the 0x or not? (guess this could be a user option)

Perhaps start a brief debate on this, and I'd be happy to provide a PR once you settle on a potential approach [if any, of course]? I'm also aware number parsing can be disabled altogether, but we have a need for parsing to numbers in other cases.
PS: for now, although horribly inefficient and not robust since it can result in false positives, this can be worked around by using the ~ operator.

This issue can also affect hexadecimal Mongo ObjectIds. Usually, an ObjectId will contain a mix of alpha and numeric characters, but it can happen that your timestamp, random, and increment each happen to encode to all numeric characters, in which case it mongo-querystring will convert it to a number.
There's not really a safe way for the library to detect what the user's intent is, as they may happen to be passing a 16-digit number in the URL, but the custom option to bypass parsing should cover the ObjectId case as well as the general hexadecimal case.
However, this approach would opt out of the parsing logic for arrays. If you definitely know your hexadecimal values will start with '0x', it seems you could subclass MongoQS and override parseStringVal to return an unmodified string, or strip the '0x', etc.


Update http4s-blaze-client, http4s-circe, ... to 0.21.16

Updates

org.http4s:http4s-blaze-client
org.http4s:http4s-circe
org.http4s:http4s-client
org.http4s:http4s-dsl

from 0.21.1 to 0.21.16.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Applied Migrations

dependency:v0_21@org.http4s:http4s-scalafix:0.21.5

Documentation:

https://github.com/http4s/http4s/releases/tag/v0.21.5

Files still referring to the old version number
The following files still refer to the old version number (0.21.1).
You might want to review and update them manually.
build.sbt

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, semver-patch, scalafix-migrations, old-version-remains

Superseded by #160.


Full-static mode querystring API

What problem does this feature solve?
Full-static mode is really useful !
However, when developing a list with query string like /articles?page=2, it is currently not possible to save one payload per page.
The only full-static solution that I found so far is to:

fetch all articles in asyncData / fetch (so there is one payload with all articles).
filter all articles in the mounted hook.

However, the page is re-rendered on page load.
What does the proposed changes look like?
To work around this, Nuxt could expose a payload API to allow developers to save multiple payloads per page.
In the  /articles?page=2 example, we could save one payload for one article page.
On page load, we would be able to choose the appropriate payload to load.
Or maybe Nuxt could expose an API to load a specific payload from the query strings values.
I looked into the full-static PR and did not find any way to do this. It seems that there is only one payload saved per page.
However, I found a reference to this issue in the RFC. @manniL commented about this issue but I'm not sure if a solution was found.
Thanks a lot for this great framework !

This feature request is available on Nuxt community (#c10864)

I went with completely avoiding query strings wherever possible.
Instead of /articles?page=2 I use sth like /articles/page/2 because it is more SEO-friendly, easier to pick up for the user and works with static site generation easily as they can be treated as dynamic pages where 2 would be the dynamic part of the URL.
I think fully supporting query string parameters for static site generation would be quite tough.

Thanks for the feedback @manniL !
Using dynamic pages instead of query parameters would totally do the job for simple use cases but I can see several limitations:

What if you want to have a structure with an /articles (list) page and /articles/:slug page ? You may have a route conflict if a slug is page.
What if you want multiple query parameters like ?page=2&order=asc or ?page=2&filter=category2 ? This seems like a common pattern. You could convert this to a custom slug format like /articles/page/page-1-filter-category2 but it does not seems really convenient / common pattern.

Also, query parameters seems to be indexed by search engines (not an expert in this field, I may be wrong).
I guess you're right : supporting query string parameters may be quite a huge task for Nuxt !
Would it be easier to expose some payload API to let developers handle their own logic of "which payload is saved in static and which is not" ?
There's also another concern that I missed in the original issue : does the crawler handles query parameters ?
For example, if I have a <router-link :to="{ query: { page: '2' } }" />, will the crawler pre-render this page ?
Thanks :)

* What if you want to have a structure with an `/articles` (list) page and `/articles/:slug` page ? You may have a route conflict if a slug is `page`.

A very rare edge case but true, that'd be a problem here.

* What if you want multiple query parameters like `?page=2&order=asc` or `?page=2&filter=category2` ? This seems like a common pattern. You could convert this to a custom slug format like `/articles/page/page-1-filter-category2` but it does not seems really convenient / common pattern.

There are a couple of options here: Either a custom slug format or still using query parameters for the order but sorting them on the front-end (decoupled from the API call).

Also, query parameters seems to be indexed by search engines (not an expert in this field, I may be wrong).

"It depends". If they are important for the content, Search Engines usually take them into account. This doesn't mean it is good for SEO though (and good for the user).

I guess you're right : supporting query string parameters may be quite a huge task for Nuxt !
Would it be easier to expose some payload API to let developers handle their own logic of "which payload is saved in static and which is not" ?

Hard to judge from my perspective. Maybe @pi0 has an opinion here?

There's also another concern that I missed in the original issue : does the crawler handles query parameters ?
For example, if I have a <router-link :to="{ query: { page: '2' } }" />, will the crawler pre-render this page ?

No, query parameters will be ignored for rendering. You can't really "represent" them when generating pages.

We are approaching the Nuxt 2 EOL date (June 30, 2024) - see this article for more information. I'm closing this issue as it's marked as a Nuxt 2 related enhancement and it's not a critical issue.
That doesn't mean it might not be relevant for Nuxt 3. If it is, please feel free to open a new issue (or just comment, and I can reopen it). 🙏
Thank you for your understanding and for your contribution to Nuxt! 🎉

I think would be really amazing (if possible) to support string parameters for SSG and SSR in nuxt 3

@LeonardoRick They are supported for SSR easily but not for full SSG/prerendering


Avoid getting stuck in empty nodes in the Pages tree when calling |Catalog_getPageDict| (issue 5644)

This is a tentative patch that fixes #5644.
The issue with the referenced PDF file is that the Pages dictionary contains a number of empty Kids, and we were getting stuck in those nodes before reaching the actual pages.
As far as I can tell this code is just an optimization, to avoid what is usually a number of unnecessary iterations, but isn't strictly necessary. The only way I could find to solve the particular issue referenced, and to avoid similar issues in the future, was to remove that code path. This might mean that the lookup becomes a tiny bit slower in very large files, but I unfortunately found no other way to fix the issue (hence why I'm labelling this as a tentative patch).
Note: I'm adding an eq test since, when attempting to fix the issue without removing the optimization, at one point the code would cause the first page to render on all pages.

/botio test

/botio-windows preview

/botio-windows preview

/botio-linux preview

/botio test

/botio test

/botio test

/botio test

/botio makeref

@brendandahl Good to merge, right?


Lead-acid discharge plots

Make script to produce discharge plots for the lead-acid papers

Have finished this, but now not sure if it's better to do this in the main PyBaMM repo, or in my own fork. What do you reckon @martinjrobins ?

I was thinking we should have a papers directory somewhere, where we can put scripts relevant to papers using pybamm. Ideally, these would be tested along with everything else, so that the research from the papers is always reproducible. Presumably something like this would live there, in the main repo?


Background canvas on mobile

Only when the user touch the screen, dots appear, not while they are dragging

Fixed


Add realtek rts5912 soc

This PR adds support for the RTS5912 EC Chip from Realtek.
The initial support for the board includes TIMER, GPIO, CLOCK_CONTROL, PINCTRL.
please help us review the pull request.
Thanks!

please, organize commit content properly, it seems that first commit contains e.g. pinctrl stuff which should not be there.

We have put those commit content to their own commit.

I can't see any changes regarding my previous request

We misunderstood the request and moved the [pinctrl+register] files to their own commit.
Now we have committed the device tree file into a single commit.
Is that correct?

@gmarull @nordicjm pinging.
Can you please review the changes if they make this pull request better.

Please, avoid combo commits like d20f604. DT bits need to be added progressively as support for its nodes is added.

Thanks for your advise. We have moved those DT bits to their own commits.

About z_arm_on_enter_cpu_idle,
RTS5912 will keep this method to not let cpu enter idle.

We add the realtek into MAINTAINERS.yml in latest commit
https://github.com/zephyrproject-rtos/zephyr/pull/75267#discussion_r1777281955

We have run ruff to format the img_gen.py.

Fix merging conflicts.


Implemented the "Include timestamp in log messages" example

I have given a stab in describing the example without passive voice. I'm afraid that it has become a little to wordy.
resolves https://github.com/brson/rust-cookbook/issues/98

Thanks! It's much better without passive voice.


Stopping / Pausing Audio playback once it's started

Is it possible to somehow stop/pause the playback, once it has begun?

VoiceClient->pause() will pause the current playing sound, VoiceClient->unpause() will unpause it. You can use VoiceClient->stop() to completely stop it.

You are amazing! <3
Only issue i'm having now is getting ffmpeg to handle a live audio stream, but i'll catch you on Discord about that at some point :)


feat: rich text

What it solves
Adds richText to the BaseBlock type. One step closer to how CMS store rich text.
Solves the need to have links in between text blocks.
Screenshot

Good idea!
You could take it even further by allowing Markdown in all content types. Most md compilers allow allowlisting possible HTML tags, so it should be safe to insert as HTML.

Definitely! I will create an issue for that to implement it more generically.
I would however like to keep the scope of this PR minimal to allow embedding the link in the screenshot I shared (the new governance framework link). WDYT?

I've approved it, but tbh this solution is pretty hacky.

Thanks 🙏
I will refactor it when tackling https://github.com/safe-global/safe-homepage/issues/246

You're right. I'm closing this PR for https://github.com/safe-global/safe-homepage/pull/247
Content types markdown support will follow on https://github.com/safe-global/safe-homepage/issues/246


Playoffs Schedule and Updated Roster for Teams

Couldn't find a place in the API where I can get Playoffs Schedule. Do you have plans on adding it?
Also - roster data for teams is not up to date through this endpoint: "http://data.nba.net/json/cms/noseason/team/wizards/roster.json"
Is there another endpoint one can get the updated roster from?

Hello,
This API is only a gateway to data that the NBA provides, it is not the provider of the data itself. As for the roster you may want to look at http://nbasense.com/nba-api/Stats/Stats/Roster/TeamRoster which accesses http://stats.nba.com/stats/commonteamroster/


Add support for mocking server timestamp

I would like to propose the use of the clock package, which would make it possible to mock the server timestamp. Now the code uses Timestamp.now(), which in return uses DateTime.now(), which can not be changed from the outside.

Hi @AndrasTimar , would you mind share more information about the clock package?

Hi @AndrasTimar , would you mind share more information about the clock package?

Sure, you can read about it here: https://pub.dev/packages/clock

That sounds great! Would you be able to contribute a PR for it?

I created a PR for this.
https://github.com/atn832/fake_cloud_firestore/pull/287
Sorry, I am late........very late.

Thanks to @ming-chu for implementing this. I just published it to https://pub.dev/packages/fake_cloud_firestore/changelog#246.


Cleanup stale files.

Also update to a modern version of Hugo.

I don’t think you want to add all those .idea files

Oops. Gone.
Thanks.


Remove EnsurePSANotPrivileged check

What this PR does / why we need it:
OpenShiftPodSecurityAdmission feature gate is not enabled by default in 4.17 any longer. We need to stop enforcing that PSA is privileged
https://github.com/openshift/api/blob/release-4.17/features/features.go
https://github.com/openshift/api/pull/2018
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/ok-to-test

/retest-required

@csrwng ptal

/retitle NO-JIRA: Remove EnsurePSANotPrivileged check

/approve


Update Composer dependencies (2024-02-13-00-18)

Visual regression test passed!
View the visual regression test report


Purge-Revision Cli

Adds a cli to remove revisions:
ocis revisions purge

Fixes https://github.com/owncloud/ocis/issues/9472

This needs a rebase to get a green CI


NO definition -  UDP

This report identifies the volume of TCP versus UDP usage reporting on the latest month for audio, video, and video-based screen sharing (VBSS).
I can't find any previous reference to UDP on this page.  How is the learner supposed to know what is meant here?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 2a04f5de-2679-37c7-7d00-607471ea516d
Version Independent ID: c96ec0af-910a-d9f8-9d3b-95564b39d7fa
Content: Use CQD to manage call and meeting quality in Microsoft Teams - Microsoft Teams
Content Source: Teams/quality-of-experience-review-guide.md
Service: msteams
GitHub Login: @SerdarSoysal
Microsoft Alias: serdars

@drewjames1234 Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@drewjames1234 thank you for reaching out. This part of the article mentions UDP - https://docs.microsoft.com/en-us/MicrosoftTeams/quality-of-experience-review-guide#tcp. Is there any other specific information you are looking for?

Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Remove useless memory allocations in algs.

Lines like https://github.com/vais-ral/CCPi-Framework/blob/master/Wrappers/Python/ccpi/optimisation/algs.py#L76
allocate memory at every iteration.
Memory should be allocated once and reused.

x[:] = g.prox(u,invL)

closed by #136


Support custom query param for the authentication code

When a user is redirected back to the app callback during the authentication flow, the code to exchange for tokens is always returned using a code query param. It would be useful to be able to specify a different query param to pass the code in in cases when there are layered systems and code may already be a reserved query param (ie handled in a different way).
Currently
The application generates a url to redirect the user to as so:
const params = new URLSearchParams()
params.append("redirect_uri", redirect_uri)
const response = await fetch(`https://api.redactive.ai/api/auth/connect/${provider}/url?${params.toString()}`, {
    method: "POST",
    headers: {
        "Content-Type": "application/json",
        // Your API key should be stored securely and not commited to source control.
        "Authorization": "Bearer test_1234567"
    }
})

And the user is then returned to the application with the code to continue the auth flow available under the code param:
const currentUrl = window.location.href;
const urlParams = new URLSearchParams(new URL(currentUrl).search);
const sign_in_code = urlParams.get('code');

Proposed
The application is able to specify a query param it would like the code to be returned with:
const params = new URLSearchParams()
params.append("redirect_uri", redirect_uri)
params.append("return_code_query_param", "a_different_value")
const response = await fetch(`https://api.redactive.ai/api/auth/connect/${provider}/url?${params.toString()}`, {
    method: "POST",
    headers: {
        "Content-Type": "application/json",
        // Your API key should be stored securely and not commited to source control.
        "Authorization": "Bearer test_1234567"
    }
})

And the user is then returned to the application with the code to continue the auth flow available under the a_different_value param:
const currentUrl = window.location.href;
const urlParams = new URLSearchParams(new URL(currentUrl).search);
const sign_in_code = urlParams.get('a_different_value');

If return_code_query_param is not specified in the initial request, then code will be the default.

Feature has now been implemented by #18 in the python SDK. As well as release v1.4.45 internally. On the POST /api/auth/connect/${provider}/url there is an optional query parameter called code_param_alias if this is set the returning code at the end of the OAuth flow will be available under whatever the code_param_alias key is.


[ez] Export missing default model parsers

[ez] Export missing default model parsers
Now we can do from aiconfig import ClaudeBedrockModelParser for example. I also deleted the hf.py file since we now have aiconfig_extension_hugging_face for this

Based on comments:

Not removing core model parser for HF. Task for removing it in https://github.com/lastmile-ai/aiconfig/issues/1116
Not defining model parser for AzureOpenAIParser


Upgrade angular version to 13

Codecov Report

Merging #58 (6442ec6) into main (b268487) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #58   +/-   ##
=======================================
  Coverage   28.00%   28.00%
=======================================
  Files           3        3
  Lines          25       25
=======================================
  Hits            7        7
  Misses         17       17
  Partials        1        1

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 b268487...6442ec6. Read the comment docs.


Editing services description does not save

After modifying the description of a service using PUT API and attempting to save, the original value is still displayed.

@parthp2107 and @darth-coder00, is this a UI issue or a backend issue?

@harshach this is because a non-emtpy description is not updated by PUT operation because we don't update description to retain user written descriptions over ingestion. Given services are not ingested, we can simply allow updating description for some of the entities.

@vivekratnavel, we can pass a flag from PipelineUpdater class to the superclass EntityUpdater to allow description update in updateDescription method.

@vivekratnavel, we can pass a flag from PipelineUpdater class to the superclass EntityUpdater to allow description update in updateDescription method.

@sureshms services can be created from ingestion as well . At least we are not adding description for services but a service can be derived from ingestion config such service_name and connection details and gets created using the put.

@harshach, is there an easy way to distinguish the ingestion user? If that is the case, we have all the information in EnityUpdater to only disallow description updates during ingestion. That would be the best and easiest fix.


Shaders using Texture.Load do not produce valid GLSL shader

I'm encountering this error on Vulkan instance/runtime version 1.1.82:
'texelFetch' : required extension not requested: GL_EXT_samplerless_texture_functions
Tested on Slang 0.10.35, and 0.11.x
Does not appear to occur on VK version 1.1.77.

I'm a bit confused, since this seems to be an error message coming from glslang, and because Slang bundles its own version of glslang into its build I'm not clear on how changing the version of the Vulkan runtime could affect things.

PRs #685 and #686 should resolve the regression that was introduced.

Should be fixed in v0.10.36 and v0.11.3 when it goes live.


User's avatar link is broken

For example, clicking my avatar gives the link: https://twitter.com/https://pbs.twimg.com/profile_images/478951034820112384/I1MuO9IE_normal.png/
This should be: https://twitter.com/freddinator

Fixed


Add "JSON" to the "Open settings file" command names

[x] Closes #13225

@msftbot merge this in 1 minute


Error while cloning from 'git clone https://github.com/DevExpress/AjaxControlToolkit.git'

After installing git and when i execute commands in git console
git clone https://github.com/DevExpress/AjaxControlToolkit.git
I am getting below error fatal: unable to access 'https://github.com/DevExpress/AjaxControlToolkit.git/': failed to connect to github.com port 443: Timed Out
Please find the attached screenshot

Your question is off-topic. It's about Git, not about AJAX Control Toolkit.
Try searching or asking at Q&A sites like stackoverflow.com or superuser.com.
P.S. This thread looks related: https://stackoverflow.com/q/18356502


[DO NOT MERGE] Auto PR to update dependencies of pilot

This PR will be merged automatically once checks are successful.
none

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/pilot-presubmit.sh
73b037b873e393e41799a7b8abbd0f131cca5cc1
link
/test pilot-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


[Jet Chat]:  Show pop up with explicit token like @ for user mentions # for hashtags

Is there an existing issue for this?

[X] I have searched the existing issues

Is this a feature request for one of the samples?

[X] Yes, this is a specific request related to this samples repo.

Sample app
Jetchat
Describe the problem
I want to tag users or mention hashtags when typing in text field. This is similar to something most chat apps have like twitter and facebook. Have used https://github.com/linkedin/Spyglass for my use case using xml layouts.
Describe the solution
Would love to have the solution where user can tag others or hashtags with explicit token like @ or # in textfield.
Additional context
No response
Code of Conduct

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

Got the idea. Implemented a raw solution for my use case.

@raghunandankavi2010 Could you please post here how you implemented this feature ?

Yes, please, @raghunandankavi2010 . We would all love to view and learn that. Thanks, @iRaam , for the request.


BakerMillerPink is causing an exception.

This seems to be a missing case in your switch statement.
I'm iterating over all values of ColorName because I'm using it to find the nearest color name for a user-input color.

The issue is at https://github.com/iamartyom/ColorHelper/blob/95321f41495b17473ddef5c8b452c857e26a4049/ColorHelper/Color/ColorNames.cs#L1032-L1035

It's the only one missing as far as I can tell.

Hi @lyssieth.
Thanks for your contributing.
I will merge your changes and create new release now with version 1.6.1.


Votes should be hashed according to EIP-712 #335

Fixes: #335

Created struct VoteMessage
Created struct Vote
Added Hashing logic as per EIP-712

Please note:  In this  PR bytes20 coreIdentifier is still part of struct VoteMessage.
As the votes are for both origin and auxiliary chain, we need to have the coreIdentifier information while generating the voteHash.

Did you check whether it is possible to calculate the vote hash only once?
Yes, that was missed in the initial commit, I committed that part now, thanks!!!


cilium cli upgrade breaks eks anywhere

Hi,
I am testing eks anywhere with cilium integration, eks anywhere deployed cilium 1.9.10, but I want to upgrade cilium to newer version, ran cilium cli upgrade in  https://github.com/aws/eks-anywhere/issues/362#issuecomment-932395310, it appears breaking the cluster network as I was unable to get k8s API response from kubectl command.
wonder if this is related to kind that eks anywhere is using, and wonder how to trouble shoot this kind of issue, is there cilium cli upgrade debug flag to use?

shot in the dark, wonder if this is in same area of issue that  https://github.com/cilium/cilium/pull/17513 is addressing or have already addressed in most recent cilium


update: update domain name for my website

I recently changed my github username/domain name,
thought I would PR a fix

@purarue thanks for the update!


Dead package

Is this package dead? Should we migrate to another package?

I think it's pretty dead...


kola: skip Docker torcx tests in case of Edge channel

Since the docker 1.12 profile was removed in Edge, many kola tests started to fail to run, because of the missing docker 1.12 profile. Ideally we should be able to make a different set of profiles for each channel. Unfortunately the current Jenkins pipeline is not able to do that. It takes into account only flatcar-master branch.
To work around the issue, read directly a version from /etc/os-release to detect if the image is Edge. If the semver's minor version is 99, then it means Edge, so skip the tests.
Note, the code is only a quick-and-dirty approach. In the future, we should create a generic way to make channel names available for individual tests.

Testing done:

ran docker.torcx-flag-file.cloud-config and docker.torcx-flag-file on Edgel Alpha, Beta, and Stable. (only on Azure, but results should be repeatable on all other platforms)
Tests passed for Alpha, Beta, and Stable. Tests were skipped for Edge.

LGTM!


no config file generated?

I dont think a config file generated, can someone just send me theirs or tell me how to generate it?

https://github.com/workspacer/workspacer/blob/master/src/workspacer/Config/workspacer.config.template.csx

reading the wiki is also advisable

hi, it made the .workspacer file on its own in %userprofile%.
I tried copying in another users config because thats the only way I knew how to and it seems to responding to the config so I'm currently working on figuring out all the binds.
Are the colors not as customiseable as dwm? are there patches i need to apply?

hi, it made the .workspacer file on its own in %userprofile%.

That's not it.
Like @jshield  says you need to create a ..workspacer directory inside %userprofile%. And that directory you need to create the config file, and name it workspacer.config.csx.

There's this whole webpage here which might serve as a good reference:
https://workspacer.org/config/

Also for reference, here are the default keybindings: https://workspacer.org/keybindings/

Closing as solved!


feat(table): sticky header

What Changed
Now the table has a sticky header. It will behave as you expect, sticking to the top when needed.
Why
The content of the table could be longer than expected, as a sequence of logs.

What might be impacted
Styles could be broken for some browsers.
Firefox & Chrome support sticky headers.
Todo:

[X] Add tests
[X] Add docs

Hey @tylerkrupicka / @adierkens, tagging you as I don't know the process to contribute to the project. Do I need to generate any artifacts? I've seen other PRs automatically generate storybook previews. How can I trigger that job here?
Thank you for this fantastic work!

Hey @tylerkrupicka / @adierkens, tagging you as I don't know the process to contribute to the project. Do I need to generate any artifacts? I've seen other PRs automatically generate storybook previews. How can I trigger that job here?
Thank you for this fantastic work!

Hey @tylerkrupicka @adierkens, another bump. I hope this time I am more successful.

Hey @mormubis, thanks for the contribution and sorry for the delay responding! Something looks broken with the CircleCI build for tests, so I'm taking a look at that to get your change in.

:rocket: PR was released in v1.2.0 :rocket:

@mormubis thank you for your contribution! I made some fixes to the CSS and height detection, and released this feature in https://github.com/intuit/devtools-ds/releases/tag/v1.2.0.


App doesn't launch unless webview is uninstalled

The app wouldn't launch unless any webviews installed on the device is uninstalled. Doesn't work with Android system webview or Bromite webview installed.

rom issue

Works on my machine :TM:

Bro wad ☠

React Native depends on WebView


Monitor updates to pr and post CI progress

Background and further details here: https://github.com/nodejs/build/issues/236
I'd be happy to chip in here.

@phillipj Looking at @jbergstroem's work for the build team in that linked issue above, I see he/they are using the Github Status API. That seems like a cool way to go... maybe incorporate that in to the work you have in pollTravis?

Yeah we should probably push to the github status API. That way, it also displays at the merge button and you don't need to find the latest comment.

Related github module API: http://mikedeboer.github.io/node-github/#statuses, looking into it.

@williamkapke trying to use to status API sounds awesome! One step closer to mimic Travis completely :dancers:

moving the status api discussion to https://github.com/nodejs-github-bot/github-bot/issues/14

@jbergstroem how would we know when to start polling? Could Jenkins do a HTTP requestagainst the bot, notifying there's a PR build that just started? Or should the bot listen for PR comments matching "CI: http://ci.nodejs.org/*"?

I owe a few of you a post -- sorry for the delay. I'm a bit backlogged at the moment; will get back to you shortly.

Seems the ../api/json endpoint would be perfect for polling build status. E.g. for https://github.com/nodejs/node/pull/6342: https://ci.nodejs.org/job/node-test-pull-request/2363/api/json

Landed with #35


Indication of pending PR on a component

We still need to decide what the right thing to display is, but when a user submits a contribution on a component and the PR is opened, we want to display to that user and other users that there is a PR open on that component. We may also want to give the user an indication of the states if the PR closed or the PR merged as well.

Closing, this is negated by https://github.com/clearlydefined/website/issues/187


Add Allocator support to executors

This adds support for custom allocators to our executors. We design follows the malloc/free function pair, with base types for all executor memory spaces to avoid users accidentally using incompatible Executor/Allocator pairs. I also used this opportunity to clean up the executor.* files and separate forward-declarations, streams, allocators and executors into different files, and deprecate the device_reset flag, which is no longer necessary for tests.
Why model malloc/free and not more allocator-friendly interfaces?
Without an interface-break, we cannot change the raw_alloc/raw_free interface, so we need to stick with the minimum that can be callsed from raw_alloc(size) and raw_free(pointer). Additionally,  none of the allocator wrappers I implemented can profit from any additional size information or even an opaque allocator info struct.

Since there were also other design options for the allocator interface, would you mind giving a very brief explanation, why you chose this approach over the others?

@MarcelKoch Good point, I added an explanation - basically interface stability. Also none of the allocator wrappers I implemented can profit from any additional information.

I agree with your suggestion to sync with application users, I'll see if I can find somebody with suitable experience. ATM, I have two possible interface variants in mind:

stateful allocators with allocate(size_type)/deallocate(void*) that store stream handle and maybe device ID to allow unified CPU/GPU allocation directly. Can lead to mismatches, which we may need to check for at executor construction time?
(by default) stateless allocators with allocate(stream_handle, size_type)/deallocate(stream_handle, void*) for GPUs and without stream_handle for CPU. This makes mismatches impossible, but makes it harder to use the same allocator for CPU and GPU.

ATM, I am leaning towards 1. i.e. adding get_device_id and is_stream_ordered/get_stream functions so that executors can check at construction time whether the allocator was set up correctly. What do you think @MarcelKoch?

I was too subtle with my remark, sorry about that. I just meant that @pratikvn should take a look at it, because he mentioned that he has a use case for custom allocators.

I think 1. is a suitable approach. But then we should add the checks to the executor constructor.
Another approach could be to have intermediate classes which have a function like std::unique_ptr<Allocator> create_allocator(const Executor*), but that might complicate the class hierarchy a bit and thus make it harder for users to provide custom allocators.


fix(package): we had the wrong url which broke semantic-release

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
14

Relevant Lines:
54

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
14

Relevant Lines:
54

💛  - Coveralls


added a popup we wait on when you did not agree to the experimentals

Changelog Description
this branches away from  pinning file support and adds the popup.
Additional info
Testing notes:

I don't think we can do it that way - addons are initialized even on headless machines (render machines) so any code importing QtWidgets will fail there. Also the import in addon.py is unnecessary?

yes we need an function that tells us if qt is availalbe so that we can guard the imports.
but i was not able to figure out how to do that without the linter hating me.
if i do (code out of the back of my head)
HEADLESS_MODE = True
try:
import qt
HEADLESS_MODE = False
except:
print("info")
if not HEADLESS_MODE:
import qt related things
if not HEADLESS_MODE:
run the function that creates the popup.
then he errors because there are things about the top layer import.
btw iLLiCiTiT already marked it but the addon cant start headless because lakefs gui import also needs qt.
but i was not able to find out how to do it.

@antirotor you brought up this idea
HEADLESS_MODE_ENABLED = os.getenv("AYON_HEADLESS_MODE") == "1"
in the other branch.
do we know it headless mode is a guarantee? because there are no Docs for it what so ever.

I actually don't know - I don't think that there is really any qt based check - this env var is set if you explicitly run ayon with --headless flag and that we should definitely respect. Maybe @iLLiCiTiT know if we check for the UI capability somewhere?

We don't, we can't. There is function is_headless_mode_enabled. If that returns True you should never get to point that you import anything from Qt.

Please bear in mind @mkolar 's comment:

This is overkill. Give a notice in the settings or whereever they are guaranteed to see it at least once and that's it. I wouldn't be adding explicit checkboxes.

The important bit in that point is the "must see at least once". I'd argue that with how the system is currently designed (and one being required to configure the LakeFS settings) that one MUST access the settings page regardless. So, we may be best off when that checkbox is disabled to just log a command-line warning and documentation and live with that.
Whether that suffices to what @dee-ynput wanted here - I'll let the battle commence here.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/120 is merged into live branch.
Auto archive content to https://github.com/OPS-E2E-PPE/docs-archive-test-target.git

Docs Build status updates of commit 379b4d1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

archiveSourceFolder/content/someContent.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Check machine configs have an SSH key for upgrade

This was already an implicit need, due how the capi spec generation is written, but the error produced wasn't clear and it also happen too late in the upgrade workflow. Now we throw an error during preflight validations.
Issue #, if available:
Related to #5420
Description of changes:
This was already an implicit need, due how the capi spec generation is written, but the error produced wasn't clear and it also happen too late in the upgrade workflow. Now we throw an error during preflight validations.
Output in case of failure:

❌ Validation failed    {"validation": "SSH Keys present", "error": "[m-cluster-etcd VSphereMachineConfig is invalid: it should contain at least one SSH key, m-cluster-cp VSphereMachineConfig is invalid: it should contain at least one SSH key]", "remediation": "Please include at least one SSH key per machine config. If your keys were autogenerated during the create cluster operation, make sure you include them in your cluster config for all lifecycle operations"}

Testing (if applicable):
Ran CLI manually.
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


Implement frontend for maximum_wk_srs_level_to_review option

As detailed in https://github.com/Kaniwani/kw-backend/pull/435
simple profile setting:
maximum_wk_srs_level_to_review which operates in the same fashion as the extant minimum_wk_srs_level_to_review, in that it is a partial inequality. Setting it to GURU will include all reviews up to and including guru. <=

Added in 2.6.0


Shaking Note Modchart

What is your question?
what's the lua code for shaking notes modchart

???


Path

Deployment using the deploy to azure button doesn't work cause the master branch was renamed to main

Fixed with #95