Update Code of Conduct and Security Policy

Updates the readme, code of conduct and security policy per OSPO request.

@anmenaga or @TravisEz13 are you able to merge this?


Improve the logic to get key annotations from HiltMapBinds.

It is improved to find key annotation to which MapKey annotation is applied from the existing method of finding key annotation based on Suffix.

resolved: https://github.com/Park-SM/HiltBinder/pull/14


firebase_messaging - update dependency needed

Hello,
i get this error when trying to use firebase_messaging latest with firebase_auth plugin
Because firebase_messaging >=7.0.3 <8.0.0-dev.7 depends on firebase_core ^0.5.0+1 and firebase_auth >=0.20.0 <0.21.0-1.0.nullsafety.0 depends on firebase_core ^0.7.0, firebase_messaging >=7.0.3 <8.0.0-dev.7 is incompatible with firebase_auth >=0.20.0 <0.21.0-1.0.nullsafety.0.
firebase_core needs to be updated

Hi @vytautas-pranskunas-
You are getting this message because you cannot mix and match the firebase plugin versions after the recent updates to the minimum ios constraint. You should use all of the latest versions for the plugins (you dont need to use nullsafety).
Upgrading to the latest firebase_messaging: ^8.0.0-dev.14 should fix the issue for you.
Closing since this isn't an issue with the plugins themselves and the versioning is described in the error message itself.
Thank you


java.lang.NullPointerException: Cannot get property 'error_loop' on null object @ line 1246

I noticed a few of these in my logs:
 java.lang.NullPointerException: Cannot get property 'error_loop' on null object @ line 1246

I'm still learning groovy and I've just started looking into the code, so correct me if I'm wrong, but isn't the loop starting at 1 the way it's implemented? So isn't it skipping lock 0 and going one past the end of the array? That'd make sense for the null.
def allCodesDone() {
  def i = 0
  def codeComplete = true
  theLocks.each { lock->
    i++

    if (state."lock${i}" && state."lock${i}".error_loop == true) {
      codeComplete = false
    }
  }
  return codeComplete
}

I tested moving the "i++" to the end of the loop, and I no longer see an exception.
def allCodesDone() {
  def i = 0
  def codeComplete = true
  theLocks.each { lock->

    if (state."lock${i}".error_loop == true) {
      codeComplete = false
    }

    i++
  }
  return codeComplete
}

Thanks!
https://github.com/ethayer/user-lock-manager/commit/c743a38262aca24fa85ac19a7bb89895d8b90a76


Showing all time wrong cpu load

Hello there.
I have been using you apk for some years. I own now a Xiaomi Redmi 4X and I have installed your apk but it is showing wrong values/ wrong graphs in notification area only, the colored lines are not moving at all.
My cpu is octa-core, my os is nougat 7.1.2 and your apk is reporting that hardware is qualcomm msm8940 .
Hope that this info will help you. This is the only apk I like.
I have attached two pictures so you can see details about my phone and also the notification area with the cpu stat icons.
Thank you.

How about v2.0.0 ?


JSDoc Support - @template doesn't work in prototype definition

VSCode Version: 1.5.3
OS Version: macOS 10.12

@template JSDoc comments are not recognized if applied directly to a prototype function definition.
/**
* @template T
* @param {T} bar
* @returns {T}
*/
function foo(bar) { }
foo({id: 1234}).    // Intellisense will provide a recommendation of id in this example

function Foo() { }
/**
* @template T
* @param {T} bar
* @returns {T}
*/
Foo.prototype.foo = function (bar) {
};
new Foo().foo({id: 1234}). // Intellisense will not provide any recommendations in this example

// -- However, it will if the prototype is defined this way -- //
Foo.prototype.foo = foo;
new Foo().foo({id: 1234}). // Intellisense will provide a recommendation of id in this example

Edit from @waderyan: added syntax specifier to code block

@benjaminmillhouse thank you for opening this issue. I have recreated on my own machine and will send this across to the TS repo.

This issue was moved to Microsoft/TypeScript#11492


Fixes documentation build docs #11972 issue

Issue #11972
Summary: Building the docs, while resulting in a working build, raises some errors due to missing Python reference targets.
Solution:
This PR has resolved some docs errors that are because of incorrect reference targets using the Sphinx reference directive (e.g. {class}).
Assumption:
Unresolved errors are because of fast forward reference and my unknown reason.
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.get_rendition:1: WARNING: py:class reference target not found: wagtail.images.models.Filter
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.get_rendition:1: WARNING: py:class reference target not found: wagtail.images.models.AbstractRendition
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.find_existing_rendition:1: WARNING: py:class reference target not found: wagtail.images.models.Filter
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.find_existing_rendition:1: WARNING: py:class reference target not found: wagtail.images.models.AbstractRendition
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.create_rendition:1: WARNING: py:class reference target not found: wagtail.images.models.Filter
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.create_rendition:1: WARNING: py:class reference target not found: wagtail.images.models.AbstractRendition
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.get_renditions:1: WARNING: py:class reference target not found: wagtail.images.models.Filter
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.get_renditions:1: WARNING: py:class reference target not found: wagtail.images.models.AbstractRendition
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.find_existing_renditions:1: WARNING: py:class reference target not found: wagtail.images.models.Filter
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.find_existing_renditions:1: WARNING: py:class reference target not found: wagtail.images.models.Filter
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.find_existing_renditions:1: WARNING: py:class reference target not found: wagtail.images.models.AbstractRendition
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.create_renditions:1: WARNING: py:class reference target not found: wagtail.images.models.Filter
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.create_renditions:1: WARNING: py:class reference target not found: wagtail.images.models.Filter
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.create_renditions:1: WARNING: py:class reference target not found: wagtail.images.models.AbstractRendition
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.generate_rendition_file:1: WARNING: py:class reference target not found: wagtail.images.models.Filter
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.generate_rendition_file:1: WARNING: py:class reference target not found: django.core.files.base.File
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.generate_rendition_file:1: WARNING: py:class reference target not found: django.core.files.base.File
wagtail/docs/reference/contrib/routablepage.md:: WARNING: py:attr reference target not found: wagtail.models.Page.url
wagtail/wagtail/models/__init__.py:docstring of wagtail.models.Page.route_for_request:1: WARNING: py:class reference target not found: HttpRequest
wagtail/wagtail/models/__init__.py:docstring of wagtail.models.Page.route_for_request:1: WARNING: py:class reference target not found: RouteResult
wagtail/wagtail/models/__init__.py:docstring of wagtail.models.Page.find_for_request:1: WARNING: py:class reference target not found: HttpRequest
wagtail/wagtail/models/__init__.py:docstring of wagtail.models.Page.update_aliases:1: WARNING: py:class reference target not found: PageRevision
wagtail/wagtail/models/__init__.py:docstring of wagtail.models.Page.update_aliases:1: WARNING: py:class reference target not found: optional
wagtail/wagtail/blocks/field_block.py:docstring of wagtail.blocks.field_block.FieldBlock:1: WARNING: py:class reference target not found: wagtail.blocks.base.Block
wagtail/wagtail/blocks/field_block.py:docstring of wagtail.blocks.field_block.ChoiceBlock:1: WARNING: py:class reference target not found: wagtail.blocks.field_block.BaseChoiceBlock
wagtail/wagtail/blocks/field_block.py:docstring of wagtail.blocks.field_block.MultipleChoiceBlock:1: WARNING: py:class reference target not found: wagtail.blocks.field_block.BaseChoiceBlock
wagtail/wagtail/blocks/field_block.py:docstring of wagtail.blocks.field_block.PageChooserBlock:1: WARNING: py:class reference target not found: wagtail.blocks.field_block.ChooserBlock
wagtail/wagtail/documents/blocks.py:docstring of wagtail.documents.blocks.DocumentChooserBlock:1: WARNING: py:class reference target not found: wagtail.documents.views.chooser.BaseDocumentChooserBlock
wagtail/wagtail/images/blocks.py:docstring of wagtail.images.blocks.ImageChooserBlock:1: WARNING: py:class reference target not found: wagtail.blocks.field_block.ChooserBlock
wagtail/wagtail/snippets/blocks.py:docstring of wagtail.snippets.blocks.SnippetChooserBlock:1: WARNING: py:class reference target not found: wagtail.blocks.field_block.ChooserBlock
wagtail/wagtail/blocks/static_block.py:docstring of wagtail.blocks.static_block.StaticBlock:1: WARNING: py:class reference target not found: wagtail.blocks.base.Block
wagtail/wagtail/blocks/struct_block.py:docstring of wagtail.blocks.struct_block.StructBlock:1: WARNING: py:class reference target not found: wagtail.blocks.struct_block.BaseStructBlock
wagtail/wagtail/blocks/list_block.py:docstring of wagtail.blocks.list_block.ListBlock:1: WARNING: py:class reference target not found: wagtail.blocks.base.Block
wagtail/wagtail/blocks/stream_block.py:docstring of wagtail.blocks.stream_block.StreamBlock:1: WARNING: py:class reference target not found:

wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.find_existing_rendition:1: WARNING: py:class reference target not found: wagtail.images.models.Filter
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.create_rendition:1: WARNING: py:class reference target not found: wagtail.images.models.Filter
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.get_renditions:1: WARNING: py:class reference target not found: wagtail.images.models.Filter
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.find_existing_renditions:1: WARNING: py:class reference target not found: wagtail.images.models.Filter
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.find_existing_renditions:1: WARNING: py:class reference target not found: wagtail.images.models.Filter
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.create_renditions:1: WARNING: py:class reference target not found: wagtail.images.models.Filter
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.create_renditions:1: WARNING: py:class reference target not found: wagtail.images.models.Filter
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.generate_rendition_file:1: WARNING: py:class reference target not found: wagtail.images.models.Filter

:point_up_2: I am not able to resolve this errors. Can you help me how to fix this, I tried a lot changing noindex to noindexentry
try lot of solutions like:

Give noindex/noindexentry  vice versa to duplicate objects
Try to add Filter to conf.py as autodoc_type_aliases ,
but nothing works

wagtail/wagtail/models/__init__.py:docstring of wagtail.models.Page.route_for_request:1: WARNING: py:class reference target not found: HttpRequest
wagtail/wagtail/models/__init__.py:docstring of wagtail.models.Page.route_for_request:1: WARNING: py:class reference target not found: RouteResult
wagtail/wagtail/models/__init__.py:docstring of wagtail.models.Page.find_for_request:1: WARNING: py:class reference target not found: HttpRequest

:point_up_2: These are unknown to me. Tried various ways, but nothing works to me. I think I am not able to find root cause in detail.
wagtail/wagtail/blocks/field_block.py:docstring of wagtail.blocks.field_block.FieldBlock:1: WARNING: py:class reference target not found: wagtail.blocks.base.Block
wagtail/wagtail/blocks/field_block.py:docstring of wagtail.blocks.field_block.ChoiceBlock:1: WARNING: py:class reference target not found: wagtail.blocks.field_block.BaseChoiceBlock
wagtail/wagtail/blocks/field_block.py:docstring of wagtail.blocks.field_block.MultipleChoiceBlock:1: WARNING: py:class reference target not found: wagtail.blocks.field_block.BaseChoiceBlock
wagtail/wagtail/blocks/field_block.py:docstring of wagtail.blocks.field_block.PageChooserBlock:1: WARNING: py:class reference target not found: wagtail.blocks.field_block.ChooserBlock
wagtail/wagtail/documents/blocks.py:docstring of wagtail.documents.blocks.DocumentChooserBlock:1: WARNING: py:class reference target not found: wagtail.documents.views.chooser.BaseDocumentChooserBlock
wagtail/wagtail/images/blocks.py:docstring of wagtail.images.blocks.ImageChooserBlock:1: WARNING: py:class reference target not found: wagtail.blocks.field_block.ChooserBlock
wagtail/wagtail/snippets/blocks.py:docstring of wagtail.snippets.blocks.SnippetChooserBlock:1: WARNING: py:class reference target not found: wagtail.blocks.field_block.ChooserBlock
wagtail/wagtail/blocks/static_block.py:docstring of wagtail.blocks.static_block.StaticBlock:1: WARNING: py:class reference target not found: wagtail.blocks.base.Block
wagtail/wagtail/blocks/struct_block.py:docstring of wagtail.blocks.struct_block.StructBlock:1: WARNING: py:class reference target not found: wagtail.blocks.struct_block.BaseStructBlock
wagtail/wagtail/blocks/list_block.py:docstring of wagtail.blocks.list_block.ListBlock:1: WARNING: py:class reference target not found: wagtail.blocks.base.Block
wagtail/wagtail/blocks/stream_block.py:docstring of wagtail.blocks.stream_block.StreamBlock:1: WARNING: py:class reference target not found: wagtail.blocks.stream_block.BaseStreamBlock

:point_up_2: For above errors, I have doubt, Should I remove references to non-documented parent class/base class by remove :show-inheritence flag or any other way.
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.generate_rendition_file:1: WARNING: py:class reference target not found: django.core.files.base.File
wagtail/wagtail/images/models.py:docstring of wagtail.images.models.AbstractImage.generate_rendition_file:1: WARNING: py:class reference target not found: django.core.files.base.File

Above errors, I am able to solve by adding
autodoc_type_aliases = {
    'File' : 'django.core.files.File',
}
``` :point_up_2:to conf.py. I also tried various ways to solve this, but this works. Did I correct?

wagtail/docs/releases/1.2.rst:118: WARNING: py:meth reference target not found: wagtail.models.Page.copy
 :point_up_2:
  Above error, I don't know why it is not referring to method, I think it is correct?

Most above errors, I think they are because of typehinting and fast forward references. For example,
this code
https://github.com/wagtail/wagtail/blob/main/wagtail/images/models.py#L637-L639
if we remove "Filter" and return type hint, it works fine. Problem is with autodoc is unable to understand where is custom type belongs to
and I tried googling for this issue and may be our issue is also related to sphinx autodoc extension issue. I might be wrong.

I am able to solve
wagtail/wagtail/blocks/field_block.py:docstring of wagtail.blocks.field_block.FieldBlock:1: WARNING: py:class reference target not found: wagtail.blocks.base.Block wagtail/wagtail/blocks/field_block.py:docstring of wagtail.blocks.field_block.ChoiceBlock:1: WARNING: py:class reference target not found: wagtail.blocks.field_block.BaseChoiceBlock wagtail/wagtail/blocks/field_block.py:docstring of wagtail.blocks.field_block.MultipleChoiceBlock:1: WARNING: py:class reference target not found: wagtail.blocks.field_block.BaseChoiceBlock wagtail/wagtail/blocks/field_block.py:docstring of wagtail.blocks.field_block.PageChooserBlock:1: WARNING: py:class reference target not found: wagtail.blocks.field_block.ChooserBlock wagtail/wagtail/documents/blocks.py:docstring of wagtail.documents.blocks.DocumentChooserBlock:1: WARNING: py:class reference target not found: wagtail.documents.views.chooser.BaseDocumentChooserBlock wagtail/wagtail/images/blocks.py:docstring of wagtail.images.blocks.ImageChooserBlock:1: WARNING: py:class reference target not found: wagtail.blocks.field_block.ChooserBlock wagtail/wagtail/snippets/blocks.py:docstring of wagtail.snippets.blocks.SnippetChooserBlock:1: WARNING: py:class reference target not found: wagtail.blocks.field_block.ChooserBlock wagtail/wagtail/blocks/static_block.py:docstring of wagtail.blocks.static_block.StaticBlock:1: WARNING: py:class reference target not found: wagtail.blocks.base.Block wagtail/wagtail/blocks/struct_block.py:docstring of wagtail.blocks.struct_block.StructBlock:1: WARNING: py:class reference target not found: wagtail.blocks.struct_block.BaseStructBlock wagtail/wagtail/blocks/list_block.py:docstring of wagtail.blocks.list_block.ListBlock:1: WARNING: py:class reference target not found: wagtail.blocks.base.Block wagtail/wagtail/blocks/stream_block.py:docstring of wagtail.blocks.stream_block.StreamBlock:1: WARNING: py:class reference target not found: wagtail.blocks.stream_block.BaseStreamBlock
by removing :show-inheritence: flag in wagtail/docs/reference/streamfield/blocks.md to classes responsible for errors. Should I do this?

ok @zerolab, Thanks for informing.

Will close this for now, but rest assured you will be credited when #12044 is merged.
Thank you so much for your efforts!


Its not letting me launch minecraft

Whenever 'shulker-dupe_1.18.x_1.1.1.jar' is inside the mods folder Minecraft crashes with an error stating that Fabric isn't installed and any version of fabric is needed, but fabric is installed as it is the version that is being run.

install fabric api

I have, it is the version/instance I was running minecraft but when I added the mod it either would cause minecraft to craft with the error above or the mod wouldn't load

You have to instal the 1.19 Fabric API, which can be downloaded on CurseForge. I too had this problem, and it was fixed my doing this.

Make sure you are downloading the right version of the mod (can be sub-release dependant) and your fabric version and fabric api need to be updated, some mods are incompatible with this so you will need to remove those.


Using a different name for users table

I see that I can customize prefixes for the tables used by this library. But my "users" table already has a different name, and I really don't want to rename it and go through all my code to change references to this table. Does it have to be named users, or is there a way that I can specify a different table name for PHP-Auth to use?
Thanks.

Right now, there is no way to customize the name of the users table yet. Sorry!
Yes, we have allowed for customization of the prefix. That makes sense because it allows you to have two instances (e.g. two separate sites) use the same database (whenever that actually makes sense) and to adjust the table names to your own naming scheme (which may include a prefix). And, finally, it was easy to implement.
Sure, we could also allow the entire name of the users column to be changed. And then the names of all the other tables.
But does your schema for the users table really match the one required by this library? It just has a different name? That would certainly be a rare case.
Usually, the table having a different name is a reliable indicator for the fact that the whole schema for that table does not match. And supporting customized column names, missing indexes, different constraints and other data types, on top of custom table names, is certainly something we’ll never want to do, and no library will event want to do, I guess.
After all, you could always go through the source code of this library in your vendor directory and replace all occurrences of ' . $this->dbTablePrefix . 'users  (with a trailing space!) with your custom table name. That would require you to re-apply this patch after every update of the library, of course.
Does that help?

Closing due to inactivity


Add a command to display TTC alerts

TTC... part of the life of the Toronto office.
It'd be nice to have a command that can display TTC alerts, and perhaps even throw warnings when a line is closed, without being prompted.

Already added by @emmettu


Unexpected Performance Regression

Hi, I hope that this repo is still maintained or at least open for questions. :)
My use-case:
I have a code-base which utilizes the C++ wmma template API. For specific reasons I need to perform the m16n16k16 operations and I wanted to find a quick way to switch it out for the newer mma instructions using your awesome library in the following way:
#define USE_MMA
#ifdef USE_MMA
#include "wmma_extension/detail/common.hpp"
#include "wmma_extension/tcec/detail/simt/mma_simt.hpp"
#else
#include <mma.h>
#endif

// ..

#ifdef USE_MMA
  namespace w = mtk::wmma::mma_simt;
  namespace wmma = nvcuda::wmma;
#else
  namespace w = nvcuda::wmma;
  namespace wmma = nvcuda::wmma;
#endif

and then later on use the namespaces as follows:
w::fragment<nvcuda::wmma::matrix_a, 16, 16, 16, VALUE_TYPE, wmma::row_major>
...
w::fill_fragment(frags[i][j], (VALUE_TYPE)0);
// etc.

Basically the USE_MMA flag switches between this library and the built-in nvcuda wmma C++ API.
I have two question:

First of all, is this even the intended use-case for this library?
I observe a significant performance regression when I attempt to do this. Is this also expected? I was under the impression that the underlying mma instructions will outperform the wmma API.

I can post more details, if necessary, of course (in case I didn't horribly missuse/missunderstand the library internals).
Thanks,
Elvir

Whoops, seems like this code path doesn't invoke mma instructions (please correct me if I'm wrong). I guess I could get away with just having multiple m16n8k16, but I am still wondering if there is a more convinient way of doing this?

Hi @elvircrn, thank you for using this library. As you mentioned, mtk::wmma::mma_simt does not invoke the MMA instruction. Instead, there are two ways to use the m16n8k16 instruction to compute m16n16k16.

As you mentioned, invoke the m16n8k16 instruction twice in your code.
Use the mtk::wmma::tcec API as shown in the code below. This way is available only when the fragment type of matrix C and D is float.

#include <wmma_extension/tcec/tcec.hpp>

// Policy: use mma.m16n8.k16 without the single-precision emulation
using mma_policy = mtk::wmma::tcec::Policy<mtk::wmma::tcec::op_mma , mtk::wmma::tcec::without_ec, 16, 8 , 16>;

constexpr std::uint32_t N = 16;
mtk::wmma::tcec::fragment<nvcuda::wmma::matrix_a, N, N, N, half, nvcuda::wmma::col_major, mma_policy> frag_a;
mtk::wmma::tcec::fragment<nvcuda::wmma::matrix_b, N, N, N, half, nvcuda::wmma::col_major, mma_policy> frag_b;
mtk::wmma::tcec::fragment<nvcuda::wmma::accumulator, N, N, N, half, void, mma_policy> frag_c, frag_d; // Specify 'half' here, even though the actual data type is 'float'

//...
mtk::wmma::tcec::load_matrix_sync(frag_a, smem, N);

//...
mtk::wmma::tcec::mma_sync(frag_d, frag_a, frag_b, frag_c);

//...

The mtk::wmma::tcec API is for emulating single-precision matrix multiplication on Tensor Cores. But, you can use it for your purpose this time by changing the Policy of the fragments to disable single-precision emulation mode. In this example code, the mtk::wmma::tcec::mma_sync function invokes the mma.m16.n8.k16 instruction twice internally.
Feel free to let me know if you have any questions.
Thanks


Fieldset to group related elements

Current suggestion made by Simon Cox.
Require a fieldset module in Drupal 8 to group related elements for better clarity in creating a new CatalogRecord.
Elements suggested for grouping:

RDF File

Requested URI

In general, each section should have a fieldset to better structure the page.

Should be RDF File or Access URI: these are for a resource to be accessed for review. Requested URI is of the PID to be assigned.

I have added a Fieldset to group the now renamed fields of Resource RDF File and Resource Access URI together when a new entry is being created. It doesn't seem to make too much difference though, given the cluttered nature of the Drupal form.


Add options for ReferenceStore

This pull request adds two options for the ReferenceStore:

overrides to specify fetch options like auth headers

Should these same overrides also be used in the fetch call within ReferenceStore.fromUrl?

preferTarget to denote that the opts.target URL should be used regardless of whether a URL is present in the reference JSON

I would also be open to removing this change if it is unfavorable as it can alternatively be achieved in user-land by the user manually null-ing the URLs within the contents of the ReferenceEntrys before passing into fromSpec

I would also be open to removing this change if it is unfavorable as it can alternatively be achieved in user-land by the user manually null-ing the URLs within the contents of the ReferenceEntrys before passing into fromSpec

Handling a special null to fallback to a target is already a bit of a deviation from the reference specification. I'm open to adding something like preferTarget, but worry dynamically trying to handle entries is challenging and likely hard to handle edge cases with a consistent API. I think I'd rather encourage "pre-processing" an existing spec if this behavior is desired (but open to being convinced otherwise).
One thing that comes to mind is that file references can have multiple URLs (e.g., a Zarr could virtually combine several TIFFs). Right now target only overrides if an entry is null, but in a reference where there is null and other urls it could get complicated.
Maybe a solution would be to make refs public, and let someone modify the refs:
let store = await ReferenceStore.fromUrl("https://example.com/spec.json")
for (let entry of store.refs.values()) {
  if (Array.isArray(entry)) entry[0] = null;
}

I see, I hadn't considered those edge cases, I think then best to leave up to the user prior to fromSpec. I have removed that change here


Update openeo_driver dependency and fix errors with get_log_entries

Recently the openeo-python-driver was updated to support a minimum log level in the views and functions to retrieve batch jobs logs. Also the openeo-python-client and openeo-geopyspark-driver were updated for this change.
These changes were made to alleviate the problems in issue https://github.com/Open-EO/openeo-python-driver/issues/170 .
However, this change in the python driver is causing some unit tests to fail in the aggregator because the back ends are getting a method argument they don't recognize.
The version of the dependency needs to be upgraded, and some changes will also need to be made in the aggregator to make it compatible.
One example of the relevant exception:
self = <openeo_driver.testing.ApiResponse object at 0x7ff1b311ea00>, status_code = 200

    def assert_status_code(self, status_code: int) -> 'ApiResponse':
        """Check HTTP status code"""
        if self.status_code != status_code:
            message = f"Expected response with status code {status_code} but got {self.status_code}."
            if self.status_code >= 400:
                message += f" Error: {self.json}"
>           raise ApiException(message)
E           openeo_driver.testing.ApiException: Expected response with status code 200 but got 500. Error: {'code': 'Internal', 'id': 'r-f532ca98d7734704a219d63c9c9e2355', 'message': 'Server error: TypeError("get_log_entries() got an unexpected keyword argument \'level\'")'}

venv/lib/python3.8/site-packages/openeo_driver/testing.py:119: ApiException

The list of failing tests from the pytest report:
================================================================================= short test summary info ==================================================================================
FAILED tests/test_views.py::TestBatchJobs::test_get_logs - openeo_driver.testing.ApiException: Expected response with status code 200 but got 500. Error: {'code': 'Internal', 'id': 'r-f532ca98d7734704a219d63c9c9e2355', 'message': 'Server erro...
FAILED tests/test_views.py::TestBatchJobs::test_get_logs_not_found_on_backend[th3j0b] - openeo_driver.testing.ApiException: Expected response with status code 404 but got 500. Error: {'code': 'Internal', 'id': 'r-aeeb197b39014f1696607d6570cf226f', 'message': 'Server erro...
FAILED tests/test_views.py::TestBatchJobs::test_get_logs_not_found_on_backend[th-3j-0b] - openeo_driver.testing.ApiException: Expected response with status code 404 but got 500. Error: {'code': 'Internal', 'id': 'r-878834ccb786447a957de7eb459ea922', 'message': 'Server erro...
FAILED tests/test_views.py::TestBatchJobs::test_get_logs_not_found_on_backend[th.3j.0b] - openeo_driver.testing.ApiException: Expected response with status code 404 but got 500. Error: {'code': 'Internal', 'id': 'r-b603beda0c434d109da8f13ca9c893db', 'message': 'Server erro...
FAILED tests/test_views.py::TestBatchJobs::test_get_logs_not_found_on_backend[th~3j~0b] - openeo_driver.testing.ApiException: Expected response with status code 404 but got 500. Error: {'code': 'Internal', 'id': 'r-ff9ef618d5fa4f599c4c83d8a6eed7c0', 'message': 'Server erro...
FAILED tests/test_views.py::TestBatchJobs::test_get_logs_not_found_on_aggregator - openeo_driver.testing.ApiException: Expected response with status code 404 but got 500. Error: {'code': 'Internal', 'id': 'r-6b43758641e34abda4686d74a97b080c', 'message': 'Server erro...
FAILED tests/partitionedjobs/test_api.py::TestFlimsyBatchJobSplitting::test_get_logs - openeo_driver.testing.ApiException: Expected response with status code 200 but got 500. Error: {'code': 'Internal', 'id': 'r-613609735e11476188654d6b81a19253', 'message': 'Server erro...
FAILED tests/partitionedjobs/test_api.py::TestFlimsyBatchJobSplitting::test_get_logs_wrong_user - openeo_driver.testing.ApiException: Expected response with status code 404 but got 500. Error: {'code': 'Internal', 'id': 'r-00b807c6de474964a4c7b034771c9c36', 'message': 'Server erro...
======================================================================= 8 failed, 545 deselected, 1 warning in 1.35s =======================================================================

thanks!


Ability to prevent CSS from being extracted from the manifest

Is your feature request related to a problem? Please describe.
The problem is a FOUC due to the order of the files and position of the files in the template, the FOUC is due to the stylesheet being loaded at the end of the body. I need to be able to only output script there and inject the css in the head once.
Right now I can use craft.vite.asset('app.css') to inject into the head which solves this problem, but there is a duplicate entry at the bottom of the page.
Describe the solution you would like
I would like to be able to specify as an option to skip processing CSS files in the vite.php config, this could read the config and just not execute the part of the extractManifestTags function where it is calling self::extractCssFiles(self::$manifest, $manifestKey, $cssFiles);
Describe alternatives you have considered
Forking the project
Additional context
None needed, CSS is being injected in bottom of page with script tags with no ability to remove.

There will be no duplicate entry if you include the CSS as one of the entrypoints from your vite.config.ts rather than importing it via Javascript
See the discussion here: https://github.com/nystudio107/craft-vite/issues/31
There also would be no FOUC if you used the Critical CSS pattern


Archiving an iOS app using Xcode 10.2 and latest version of this SDK takes hours

After updating to Xcode 10.2, with or without Swift 5 updates and consuming the stellar-ios-mac-sdk 1.6.0 cocoapod our iOS app archive times have gone from less than 2 minutes to upwards of 5 hours.

FWIW, it looks related to this Swift 5 bug:
https://bugs.swift.org/browse/SR-10209
After I removed a significant portion of the words in each array in WordList.swift, the archiving time dropped significantly. It looks like they fixed the Swift issue but I'm not sure when it will make it into an Xcode build. Might be worthwhile to pull in the words from a resource file to avoid this issue?

Hi cstephens-phunware,
thank you for providing this solution. It is now available in v. 1.6.3.
Best Chris.


expandDocStrings property is not working in cluecumber-maven

Describe the bug
expandDocStrings property is not working in cluecumber-maven
To Reproduce
Steps to reproduce the behavior:

Go to your pom.xml and set this the property to true: <expandDocStrings>true</expandDocStrings>
Run the plugin mvn cluecumber:reporting
Check the logs:

14:14:53.321 INFO: - fail pending/undefined scenarios : false
14:14:53.321 INFO: - expand before/after hooks        : false
14:14:53.321 INFO: - expand step hooks                : false
14:14:53.321 INFO: - expand doc strings               : false
14:14:53.321 INFO: - expand attachments               : false

See error: Open the report, the doc strings are not expanded.

Expected behavior
The doc strings should be expanded in the report

Thanks for reporting, will check it!

Fixed in 3.2.1 (deploying now, should be available on Maven Central today)


VS code icons is distorted when invoked from the command line

Invoke vscode from the command line with "code ."
Icons are distorted.
When I start VScode from the desktop and open the project all is fine.

VSCode Version: Code 1.13.0 (376c52b955428d205459bea6619fc161fc8faacf, 2017-06-08T16:34:53.678Z)
OS Version: Darwin x64 16.6.0
Extensions:

Extension
Author
Version

vscode-docker
PeterJausovec
0.0.15

vscode-eslint
dbaeumer
1.2.11

vscode-npm-source
dkundel
1.0.0

docthis-customize-tags
dweizhe
0.4.5

vscode-babel-coloring
dzannotti
0.0.4

gitlens
eamodio
3.6.1

vscode-npm-script
eg2
0.2.0

vscode-tickscript
mattyjones
0.0.4

ava
samverschueren
0.4.0

vscode-react-native
vsmobile
0.3.2

JavaScriptSnippets
xabikos
1.4.0

Steps to Reproduce:

Start vscode with command: code .

Same problem here...

This is happening for me too

+1

https://code.visualstudio.com/updates/v1_12#_macos-blurry-icons

@bpasero The fix doesn't seem to be helping and it has only started happening after updating to v1.13 on my end. 🤔

@scholtzm does it only reproduce when you run from the command line or also from dock?

Only from the command line.

@scholtzm can you send me the contents of your environment, I wonder if there is another language specific setting that causes this. Email is also fine.

I have sent you an e-mail with my env. Considering this issue has been only opened yesterday, this might be a new bug.

We should always set process.env.LC_NUMERIC = 'C' as we did in 1.12.

This looks fixed for me now, after upgrading to 1.13.1


Make the generated module public

This patch will make the module generated by DbEnum public.
This adds greater flexibility for the library users.

Hi, can you explain what you are trying to achieve with this PR? The generated module only contains trait implementations, which are automatically 'public'.

Doh, my mistake. I was in a hurry when I made this workaround and figured it would be useful upstream. However as you point out it is wholly unnecessary.


aursync: clone/pull destination

Right now it just dumps stuff to $PWD.

https://github.com/rmarquis/pacaur/issues/434


fixed links again

per @srveit 's recommendations

Closing as a duplicate of https://github.com/CenturyLinkCloud/PublicKB/pull/1541


test: Add github actions for conformance test harness

Closes #789 .
I initially based this on the script at https://github.com/solid/conformance-test-harness/blob/main/scripts/css.sh which uses a docker container, but since the action is already contained anyway the script now just installs locally. This saves us a few steps and doesn't require the https setup. I still have the version that uses docker should that be preferred.
Example output at https://github.com/solid/community-server/runs/3068099528 . The relevant step is "Run the test harness". The logger is configured to also output JSON at the end should we want to parse these results. Currently nothing is done with these results yet though.

Since you are not running 2 container I don't think you need the docker network so I would have missed out --network="host" entirely. That certainly works when I do a similar thing locally though I haven't tried that in a github action.

The network="host" is necessary so the docker container can access the server running on the host. With the default network setup the host has access to the container but not the reverse.

Are we sure we want to run this periodically rather than per PR?
Or should we 1) run a specific version of the test suite per PR and 2) the latest version periodically?

Are we sure we want to run this periodically rather than per PR?
Or should we 1) run a specific version of the test suite per PR and 2) the latest version periodically?

I'll defer to @edwardsph for this one for what's best here.
We could fix the docker container version to a specific one of which we know that it succeeds on CSS? But will be more work to make sure that we stay on a relevant version then.

I'll defer to @edwardsph for this one for what's best here.

I think the answer to this may change as we move forwards with the tests. Currently, at build time, the test harness image embeds the tests available in https://github.com/solid/specification-tests (inc configuration on which to run). We will update the image either when the harness changes or when there are a set of new tests available. Alternatively, if you want to always run the latest tests, you can clone the test repo and replace the embedded ones in the CI script.
Moving forwards, we will want to version the test repo so you can choose to run a defined set of tests or the latest.


Windows support through ConPTY

Windows now supports creating PTYs though their ConPTY API https://devblogs.microsoft.com/commandline/windows-command-line-introducing-the-windows-pseudo-console-conpty/

Dup of #95


onValueChanged recive many times.

connect to ble device
setNotifyValue, and listen onValueChanged
turn off the Bluetooth device
reconnect to device and do step 2 again

After that, it will recive two time  onValueChanged when device write data to app.

When disconnect, must cancel value change subscription like :
_normalValueChangedSubscription?.cancel();
_normalValueChangedSubscription = null;


Update MD file on new branch 2017-12-05_19-47-18

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.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.


feat: add NoFuzzing() testing option

The new testing option allows to run assertions on test.Assert without fuzzing. My use case for running tests without fuzzing is in gadget where some errors panic to simplify the API.
Additionally, I was not sure if we should omit fuzzing when the tests are run in short mode? Should we?

maybe it shouldn't be ran actually with "PRoverSucceeded" call. The point of this method is to test constraint system solver against the big.Int test engine -> if a witness fails/succeeds with one, it should with the others.
What about just calling that separately on our integration tests?

Yup, removing fuzz testing in ProverSucceeded and then fuzzing separately in integration tests makes sense.
On a similar note -- maybe also remove marshalling tests from ProverSucceeded and move them into a new method? And add assert.AllTests etc. method for doing marshalling, compiling/solving, fuzzing etc.

(converted to draft until figuring what direction to take)

@gbotrel, do you have suggestion what to do with this PR? Someone stumbled against a similar problem why this PR was created (#336).


lib Types and Documentations Fix

Fix #49773
Follow-up fix #29721
Fix #24509 as a side-effect
Besides the fixes from the above issues, this PR also fixes a lot of unclear, missing, or broken documentations (I found that originally some of them are just copied from the spec, without any elaboration). I believe this PR will bring additional enhancements to TypeScript and the community.
For details and the track list about the changes, please see #49773 mentioned above.

Although the number of changed files appears to be large, most of them are just tests with only lib line number or definition changes.
I have carefully checked all of these files. I also created 6 more tests in addition, and 2 were amended with supplements related to the changes.
This PR is now ready for a review.

There are a whole lot of unrelated changes in here.  Looking at a smattering of them, it seems that some of them fix minor bugs, some of them are minor usability improvements, and some of them reflect personal preference.  I came here from #17002, and I'm having trouble finding the relevant fix for that particular issue among the many other things addressed here.
I've got no stake in the game here, other than as a TypeScript user, but based on my experience, I think the only way you'd get anyone to even consider accepting a change of this magnitude would be if you had a very high reputation with the project stakeholders, and even then, on a large public project like this, with many, many production consumers, it would require careful, timely, and costly review and testing, which would likely be prohibitive given the relatively minor benefits the changes bring.
Please don't think I'm crapping all over your work.  Most of what I've looked at so far appear to be good changes, and I appreciate the hard work you've put into them.  I'd like to see some of these changes make into a future release.  Also, I can relate, because I've certainly been known to push though similar sweeping cleanup efforts in my own projects or those in which I have a high degree of ownership.  But in those cases, the risks and costs of mistakes, as well as the time to fix them, were comparatively low.  But as we've seen in the past, minor, seemingly obvious changes can sometimes have relatively large unintended consequences, and considering this is one of the top ten most used programming languages in the world, the cost of such consequences can be very high indeed.
I would advise making several, much smaller PRs, each addressing a highly targeted change, so that the risks, benefits, and justifications can be weighed for each individually.  I would expect that at least some of them would be accepted in short time, and others might receive helpful feedback to make them even better.
Just my ₿ 0.000042.

@P-Daddy Thank you for your faithful advices. This PR is dedicated to address #49773. The change to isArray is committed to this PR recently, just because I am confident to this solution. I understand that I shouldn't have done this, so I will separate it into another PR if the Team asks me to do so.
On the other hand, if I split #49773 into multiple PRs by myself, there will be too many – maybe it'll reach 50. This is not a problem, the problem is that I will need to resolve conflict one by one for each opening PR. Doing some maths, 50×51÷2=1275 will be too many for me. (Since the testcases contain information about line numbers in lib files, almost all PRs will conflict with each other.) Therefore, I will only separate PR as per the reviewers' request.
Moreover, in #49773, @RyanCavanaugh commented (https://github.com/microsoft/TypeScript/issues/49773#issuecomment-1175553736): (Emphasis added by me)

Some of these seem OK and some are a little bit dicier. We can take a PR and evaluate case-by-case -- please be sure to include testcases that demonstrate what's being fixed.

By the way, I understand that the Team is busy, but is there any timeline as to when this PR will be reviewed? Thanks in advance.

@P-Daddy
OK, because everyone is telling me to separate PR, I had just been exploring some possible ways to split this.
So here they are: #50449, #50450, #50451, #50452, #50453, #50454.
I am not going to close this for the time being, unless the Team wishes.

@P-Daddy
Yes, it seems that benefits do outweigh scale – even such a small change in #42566 takes a year and a half to review.

OK, because everyone is telling me to separate PR

From what I can see only one person asked you to split up the PR, and they're not even a TS maintainer.

@fatcerberus
Because in my past experience, I was told to split every time I open a PR with a considerable size, and I suspect this one is also no exception.
I'm usually reluctant to do this, but if this can speed things up, it’s worth changing my mind to do so.

In that case, for what it’s worth I don’t think the way you’ve split it (part 1, 2, 3, etc.) is useful - the purpose of splitting it should have been so the unrelated changes could be isolated and described individually.  Splitting it into “chapters” doesn’t feel like it will make maintainers’ jobs any easier.
Also I noticed in the separated PRs you strongly recommend reviewing this one instead - which entirely defeats the purpose of the split (i.e. allowing the separate smaller parts to be reviewed in isolation).
Just my $0.02 as well - I’m not a TS maintainer either.

Well, the 6 parts can be more or less described individually (I know I should have given them better titles) – there will be line number conflicts after each merge though – and that's the best I could do: to let the Team choose the way they preferred. The recommendation of reviewing this PR is just my personal preference, honestly.
In either way, I am still hoping that I could bring benefits, but not troubles, to the community. My apologies if I did do so.

Great, TypeScript bug discovered! (Temporarily fixed with a non-null assertion)


Modal docs update

No ticket
This PR does the following:

Goal is to improve the Modal Storybook examples and documentation

How has this been tested?

Accessibility concerns or updates

Checklist:

[ ] I have updated the Storybook documentation accordingly.
[ ] I have added relevant accessibility documentation for this pull request.
[ ] All new and existing tests passed.

Front End Review:

[ ] Review the Vercel preview deployment once it is ready.

@bigfishdesign13 First pass, the section I added is a response to what I always get questions about/mess up in my own code


v3: MathJax.typesetPromise() fails after overwriting .innerText of a previously typeset element

https://jsbin.com/sutikuxefi/edit?html,output
I've pasted the same code below.
I'm following the docs for v3 to make a simple dynamic page: change the page content and ask MathJax to re-render it.
http://docs.mathjax.org/en/latest/advanced/typeset.html
http://docs.mathjax.org/en/latest/web/typeset.html
The automatic typeset on page load renders math in '#target". Later (setTimeout 2 seconds):
const e = document.getElementById('target');
// MathJax.typesetClear();
e.innerText = '\\(x^2 - y\\)';
MathJax.typesetPromise();

does not render the new math. It prints an error in the browser console:
Chromium 76: Uncaught (in promise) TypeError: Cannot read property 'appendChild' of null
Firefox 69: TypeError: t is null
Fiddling with it before reporting this issue, I've noticed that calling MathJax.typesetClear(); before mutating the DOM solves the problem. The new math expression is rendered and the browser console shows no error.
If I understand correctly Version 3 is the current stable version. I don't know what is the proper fix. But I suggest mentioning something in the docs soon so other new users won't run into this same problem and will be able to get a minimal working example of editing and re-typesetting a dynamic page.
I've stumbled over MathJax.typesetClear(); by accident.
The complete minimal example which reproduces this at v3.0.0:
<!DOCTYPE html>

  <meta charset="utf-8">

\(a^2+b\)

Sorry, I just realized that I submitted a fix for the problem over a week ago, but neglected to make a comment here.  First, thanks for reporting the issue.  You are correct that you should use MathJax.typesetClear() if you have removed previously typeset math from the page.  (In version 3, there is information stored about the math in a list of typeset expressions, and if you remove typeset math from the page and replace it with new math, that list will hold pointers to math that no longer exists in the page.  That is what is causing the error you are seeing, and can cause MathJax to use more memory than necessary (since it holds onto the math that is no longer needed).
The commit listed above prevents the error message in the case where you don't call MathJax.typesetClear().

A similar thing happens with MathJax.typeset() after changing the page. In this case the error in the console is:
TypeError: Q is null  tex-svg.js:1:255722.
calling MathJax.typesetClear() first fixes the problem.

What is the consequence if I call MathJax.typesetClear() which takes no arguments but, for efficiency, only MathJax.typeset(specificElements) ?
The math in other parts of the document are still typeset but they will no longer have information stored "in a list of typeset expressions."  Is this a problem?

You are correct that the math that was previously typeset will remain typeset, but that the list of math will no loner contain that earlier typesetting information.  This is not a problem in most situations, but if someone were to use the MathJax contextual menu to switch renderers, for example, only the math currently in the mat list will be re-rendered using the new renderer.  Or if they were to turn on accessibility features, like speech text generation, only the ones in the list will be affected.  That could cause confusion, and would make the earlier math inaccessible.  So there are potential consequences to working the way you have suggested.


Update build script for compatibility with JitPack.io & enable IR JS compiler

This PR means that:

You can now fetch the library from JitPack.io:

repositories {
    mavenCentral()
    maven("https://jitpack.io")
}

sourceSets {
    commonMain {
        dependencies {
            implementation("com.github.nwillc.ksvg:ksvg:master-SNAPSHOT") // WILL NOT WORK UNTIL PR IS MERGED! To test use com.github.harry1453.ksvg:ksvg:master-SNAPSHOT
        }
    }

    val jvmMain by getting {
        dependencies {
            // No dependency required!
        }
    }

    val jsMain by getting  {
        dependencies {
            // No dependency required!
        }
    }
}

The version string master-SNAPSHOT can be replaced with a tag name or commit hash. I have updated the README accordingly. Once this is merged, it would be good to create a tag to allow users to get the library via a normal-looking version number.

It enables the compatibility mode of the Kotlin/JS compiler which builds 2 JS libraries, one for the legacy compiler and one for the IR compiler. This means it is now possible to use this library with the IR compiler.

@nwillc can you push a git tag for the current version, the latest tagged commit is 2.x, but the bintray releases were dependent on were more recent on 3.x

Check it out tomorrow

On Jan 12, 2022, at 10:07 PM, Brad Murray @.***> wrote:

@nwillc can you push a git tag for the current version, the latest tagged commit is 2.x, but the bintray releases were dependent on were more recent on 3.x
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.


Delete unnecessary checks before calls of the function "free".

The function "free" is documented in the way that no action shall occur for a passed null pointer.
Redundant safety checks can be avoided.

Thanks! This has been merged.


Storage methods do not work when minified/uglified

Following up on the discussion in this Slack thread: https://blockstack.slack.com/archives/C074LC7RC/p1510523398000023
It appears that minifying/uglifying blockstack.js caused problems with the bitcoinjs dependency. Not sure of a good way around this without addressing the issue in the dependency lib.
At the very least, this should probably be documented someplace central/accessible so people know of the issue - this makes using this package in create-react-app or similar environments difficult.
I discovered when trying to use putFile and getFile in minified code

I agree with documenting the issue somewhere, especially since the issue is related to a dependency and not the package itself.
The solution is documented in the README for bitcoinjs-lib
A relevant excerpt:

NOTE: If you uglify the javascript, you must exclude the following variable names from being mangled: BigInteger, ECPair, Point. This is because of the function-name-duck-typing used in typeforce.

As far as using create-react-app goes, I believe that for a production build you need to use the eject command given by create-react-app and modify the config/webpack.config.prod.js.
You'll need to add except: ['BigInteger', 'ECPair', 'Point'] inside of the mangle object in the UglifyJSPlugin call. (It should be around line 275)


Send an invite url link message when add contact to room which room member count less than 40

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Send an invite url link message when add contact to room which room member count less than 40.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Add a parameter invite for method room.add() to trigger send invite url message or not when the room member count less than 40.
/*
  invite, default: false
    - true: send invite url message
    - false: add to room directly
*/
function add(contact: Contact, invite: boolean = false)

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
[enhancement]

Maybe forceInvite would be better for our case? Since this new parameter is changing the standard way of what WeChat or WeCom behaves.
When this is false or undefined, we are falling back to the standard logic, if the member count is greater than 40, invitation will be used, otherwise, the person will be added to the room immediately.

Conclusion
Change method add() parameter name invite to inviteOnly.


[improvement](group commit) Group commit insert into can be executed on observer fe

Proposed changes
The group commit insert can be executed on both follower and observer fe because:

it does not write edit log such as begin transaction
the table schema version is compared on BE and if version does not match, there is retry on FE.

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Can not move rows in UITableView

Describe the bug
You can not move rows in UITableview
Demo Project
https://github.com/StefaniOSApps/Example
Run Demo
Expected behavior
Can move rows to each position.

How can we check if a UITableViewCell is moving in scrolling view ?
I think the solution we can found in
open func gestureRecognizerShouldBegin(_ gestureRecognizer: UIGestureRecognizer) -> Bool {
        guard let gestureRecognizer = gestureRecognizer as? UIPanGestureRecognizer else { return true }
        let velocity = gestureRecognizer.velocity(in: gestureRecognizer.view)
        return abs(velocity.y) > abs(velocity.x)
    }

how can we can proof if a UITableViewCell is moving ?

Checkout latest commit

Reopen it
 if let tableView = scrollableView as? UITableView, tableView.isEditing {
            return
        }

it is not the best solution, because the UITableView with tableView.isEditing = true, does not allow to hide the NavBar (always).
I hope you know that tableView.isEditing is not the same one as moving UITableViewCell
It must be asked directly if a cell is being moved at this moment

i have been updated the demo project with the current solution
https://github.com/StefaniOSApps/Example
check it out

I could, but the starting issue remains, I'm not aware of a public API that tells you whether a cell is being dragged or not. I'll do some digging and let you know.

Maybe checking if a single cell is both in editing and highlighted? 🤔

I had already tested everything. I also checked the alpha value of the subviews, but without success.
I have solved this problem:
ADD
/**
     Defaults to `true`
     */
    open var showAlwaysNavigationBarWhileEditing = true

CHANGED
if let tableView = scrollableView as? UITableView, tableView.isEditing, showAlwaysNavigationBarWhileEditing {
            return
}

SET
[(ScrollingNavigationController *)self.navigationController setShowAlwaysNavigationBarWhileEditingmode:true];

i have been updated the demo project with the current solution
https://github.com/StefaniOSApps/Example
check it out

That's fine, but if the library user doesn't set that flag, the editing behavior will be broken like in your starting project. I rather have a more robust solution, I'll let you know if I can find it, if I can't I'll add that flag.

No, I do not think so. The function gestureRecognizer?.cancelsTouchesInView = false already enough, so that everything works.
The user can set in his own Project
[ScrollingNavigationController setShowAlwaysNavigationBarWhileEditing:true]; (default is true).
I think this solution is the best and my fork (branch master) use this way.

Ok, sounds reasonable. Added shouldScrollWhenTableviewIsEditing in the latest commit.

thanks


OneDrive for Business App Only token error with download

I have successfully configured an App Only token, for use on a test site, and used it to list, create folders, move items, delete items and upload files.  However when trying to download a file I do not get the documented 302 status but instead I get a 404 not found, followed by a html body that informs me "Sorry, you cannot access this document. Please contact the person who shared it with you."
If I use the @content.downloadUrl from the meta data I get the same response.
Going into Sharepoint and providing everyone read access allows me to download the file.
In Azure I have given the app, associated with the App Only token, all the access available.
I am confused since I can do writeable actions but not read the contents of files.
I have tried both url approaches and many other permutations of configuration on Azure AD, all to no avail:
https://{tenant}-my.sharepoint.com/personal/{user}/_api/v2.0/drive/items/01CGGKSYALIMOHMA23JBFKIGR4EJJS7NRG/content
and
https://{tenant}-my.sharepoint.com/personal/{user}/_api/v2.0/drive/root:/test.txt:/content
Thanks in advance
Paul
Confirmation of consent granted.

PS. All folder/files I create are marked as being created by SharePoint App, I would have expected either the owner or then name of the App to be used instead.

We can repro this issue and are investigating a fix now.

We've determine what the issue is and are trying to decide on a solution. No ETA yet I'm afraid.

There is a workaround available. You can fall back to the SharePoint REST API for downloading files in this scenario (when using app tokens). There's an app token sample I've been working on that isn't completely ready for prime-time yet, but you can see the DownloadFile method, which translates from OneDrive API to SharePoint REST to be able to download a file.

I can confirm the bug is still open but the suggested workaround is working well.
Btw. @rgregg the sample app fails when trying to list the user files

I can also confirm that although the bug is still open the workaround is fine
Thanks for the help :-)

Have you tried using DriveId as I was successful with accessing file content through the following API request with App-Only token.
https://{tenant}-my.sharepoint.com/personal/{user}/_api/v2.0/drives/{DriveId}/items/{ItemId}/content

I could not get DriveId to work since /Drives is not supported in OneDrive for Business?
However I did try ItemId, although that did not mp to our file model, but it failed anyway
The workaround works and I'm hoping for a fix soon :-)

Did it only work with ItemId?
I tried all of these with no luck
//			String url = oneDriveApi+userId+"/_api/v1.0/drive/root:"+encodedPath+":/content";
//			String url = oneDriveApi+userId+"/_api/v2.0/drive/root:"+ encodedPath +<EMAIL_ADDRESS>//			String url = oneDriveApi+userId+"/_api/v2.0/drive/root:"+ encodedPath + ":/content";
//			String url = oneDriveApi+userId+"/_api/v2.0/drive/items/01CGGKSYALIMOHMA23JBFKIGR4EJJS7NRG/content";
So if I wanted root: what is its DriveId?
Also does it work with an encodedPath since I have no storage in my model for the ItemId and can only use the path
Cheers
Paul

This is the exact request I was submitting to get Item metadata from any drive:
https://{tenantid}-my.sharepoint.com/_api/v2.0/drives/{driveId}/items/{itemId}/
For root folder I run:
https://{tenantid}-my.sharepoint.com/_api/v2.0/drives/{driveId}/root/
I would expect this will work with path queries also, as the relative part to the root and access token attached are the most important.
I'm not sure if that helps, but this is me answering myself after few days on SO http://stackoverflow.com/questions/35819627/download-shared-file-programmatically-using-onedrive-for-business-api :)

This link https://dev.onedrive.com/odb-preview/release-notes.htm#enumerating-drives says I cannot use /drives however that aside  I wonder if it the itemID that makes the API call work?
Also your SO says you use client secret but i though that dd not work for AppOnly token, I tried that first, so I shifted to certificates.
Also looking at your call there is no UPN in the REST API so how are you using an AppOnly token to access all the different user's accounts with a single AppOnly token?

I did not intend to enumerate drives, what I used /drives command to is accessing files on different drives within a tenant.
UPN is the driveId. App-only token has 'FullControl over Tenant' permissions and from my trial-error it appears that I can use this token to access any OneDrive for Business site through SharePoint or OdB APIs. Look at this article where Azure WebJob customizes users OneDrive for Business sites http://blogs.msdn.com/b/vesku/archive/2015/01/05/customizing-onedrive-for-business-sites-with-app-model.aspx using App-only token.

When I tried to get the token using client secret it failed to work and this link, http://www.andrewconnell.com/blog/user-app-app-only-permissions-client-credentials-grant-flow-in-azure-ad-office-365-apis, among others, showed how to use it along with the need for the admin to grant access.
I'm not running anything in Azure/Sharepoint, only using Azure as the placeholder for my app details, and accessing OneDrive from our server service. However your approach looks interesting :-)
We have a prototype, in Java, with the workaround, working now, so I guess we will just be waiting for the AppOnly cert method fix for our production code.
Does the client secret method need any admin grant confirmation?

The client secret is provided during SharePoint add-in registration. When this SharePoint add-in is actually installed on the SharePoint Online site collection in your tenant, this is when your app-only token gets the permission specified in SharePoint add-in manifest.
After installation you are free to use following code to retrieve valid Access Token, with TokenHelper being Visual Studio class generated by SharePoint Add-in project.
 Uri url = new Uri("https://{tenant}-my.sharepoint.com/personal/{UPN}/");          string realm = TokenHelper.GetRealmFromTargetUrl(url);  var token = TokenHelper.GetAppOnlyAccessToken(TokenHelper.SharePointPrincipal, url.Authority, realm).AccessToken;

We don't produce a Sharepoint Add-In. Instead we use a native server app to access One Drive for Business. Perhaps that is the confusion as to why your approach works and we we're having problems with the download?
The native app, app only token, approach only works with a certificate signing approach, and not a client secret, it is interesting though that you use the drive id and not the UPN, probably worth us investigating but as I said the workaround is working for us and a fix is promised :-)

@rgregg, @Schweigi, do you have any news about this issue?
Using GetFileByServerRelativeUrl might be difficult in some cases, like as file names in Hebrew.
For example, how to make correct url for the "New Text Document.txt" in Hebrew?
סמך טקסט חדש.txt

It's been a year now, any progress?

While the Sharepoint REST API workaround works, it is incomplete in that it does not support byte range requests

Hi there,
is there any progress on this issue? The workarounds suggested in this and surrounding threads doesn't work for me.
Any reasonable suggestions?
Thanks.

There are also issues with the workaround in that files that can be uploaded can't be downloaded either due to invalid characters or path length.

The download links for app-only should now be functional - is anyone still encountering issues? If so, it would be useful to know the number of characters in the query string of one of the URLs that is failing.

No still failing with 404.
First part returns 302 and url as expected. When trying to access the returned location url returns 404 not found.

I was able to download a file through an app-only token

Download still fails for me @ificator .  Total length of url is 2457 which is over the 2048 typical limit.  I imagine this is why it is failing.

I did find a workaround where if you extract the "access_token" from the querystring and instead add an "Authorization" header for the HTTP request as "Bearer {accessToken}" the download will work.

I've found a workaround to resolve the issue. If you authenticate by using service account - you should use your access token, not token in downloadUrl. Your download url should look like: "https://tenant.sharepoint.com/_layouts/15/download.aspx?UniqueId=12345678-90ab-cdef-0123-4567890abcde" and header "Authorization: Bearer accessToken".

We've been working to reduce the size of URLs with attached tokens and latest changes should now be fully deployed. Hopefully download URLs will now no longer arbitrarily fail due to URL length, but please let us know if any of you continue to see such issues.

@ificator : Hopefully I understand this issue and the status correctly, so I'm in the right place.
I'm facing the error 404 filenotFound when accessing a graph resource with App-only tokens. The URL works in the graphExplorer. Since I'm getting a 404 and not an access denied I assume my app has the required rights to get the file (not 100% sure). The URL is very long (getting a chart from an excel file in SP).
https://graph.microsoft.com/v1.0/sites/tenant,site-id,web-id/drives/long-id/items/long-id/workbook/worksheets('Grafieken')/charts('Grafiek 1')/Image(width=0,height=0,fittingMode='fit')
Am I in the right place, and is accessing an excel file in sharepoint supported with app-only credentials, or is it possible that the URL is too long..?

Hi @JWiersema, this is where some Graph magic comes into the picture. While the URL makes it look like it's hitting just OneDrive, it's actually hitting a couple of different services. If you make the following query do you get a 404?
https://graph.microsoft.com/v1.0/sites/{siteid}/drives/{driveid}/items/{itemid}

If you don't, then OneDrive itself is fine and there's most likely a downstream issue. In that case I'll see if I can track down someonw that will be able to help.

Hi, I've been trying to create a web application that allows users to access their OneDrive  Accounts and upload and download files. So far I have managed to get to a point where by a user can navigate through the drive and get information on the file and folders in said drive (size, name etc), but I am failing hopelessly when it comes to the core function of the app upload download.
I make a request "https://api.onedrive.com/v1.0/drive/items/{item_id}/content" trying to download an item but I get a 401 error saying invalid auth token [Bearer realm="OneDriveAPI", error="invalid_token", error_description="Invalid auth token"]. I'm not too sure where I'm going wrong and I would greatly appreciate some assistance


added more tricks

added more tips and format the text and changes made in the background to make it more attractive

made first ever pull request

@prachisharma02 Which issue is it related to? Please provide the issue link here.

There was no issues i just added some more tricks as a contribution
On Wed, Oct 5, 2022, 8:05 AM Tapas Adhikary @.***>
wrote:

@prachisharma02 https://github.com/prachisharma02 Which issue is it
related to? Please provide the issue link here.
—
Reply to this email directly, view it on GitHub
https://github.com/atapas/html-tips-tricks/pull/11#issuecomment-1267837013,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2UNOQ53MCXAMCVXNOMMI7TWBTSP7ANCNFSM6AAAAAAQ4OEUU4
.
You are receiving this because you were mentioned.Message ID:
@.***>

added more tricks please check it

@prachisharma02
Could you please list down the changes you have made?
If you have an issue for it, please update that and get me the link.

https://github.com/atapas/html-tips-tricks/issues/39#issue-1403136640 -issue link
added two new tips

figure
tooltip
also changed the background and heading font

please review it

please provide a feedback


Pull Request for implementation of the Blue/Green deployment support.

In a  Continuous Delivery pipeline, the services if deployed using the application push function provided by the plugin, existing application  will be either deleted and recreated or restarted after pushing the updated version.
This results in application downtime when new versions of the applications are being deployed.
In a highly available environment, applications has to be deployed in  Blue/Green manner, with zero down time during new deployments.
The Blue Green deployment support is implemented as an extension  to the earlier option of 'Reset app if already exists'.  This option has been converted to 'Handle Existing Application' section with three options.

Blue/Green deployment
Delete and Recreate
Push App and Restart

The Blue/Green deployment implementation follows the processing outlined at http://docs.pivotal.io/pivotalcf/devguide/deploy-apps/blue-green.html .
The algorithm implemented in the plugin is indicated below.

The 'Handle Existing Application' configuration options along with the help text is as given below:

While implementing the changes, the CloudFoundryPushPublisher::processOneApp was refactored to separate out handling existing application, updating deployment info with relevant details which will be used later.
Apart from the implementation of Blue/Green deployment support, the following changes were done to the code base .
Fixed Test Failures in order to start Development. B/G deployment feature was predominantly done in TDD so the new tests (which are at feature level, as with other existing test cases) cover all the changes done for adding ‘Handle Existing Application’.

Dependency on SSH_CONNECTION env variable in the Hello Java apps used for tests
Registered for app logs immediately after app is created.
Earlier, this was done after pushAppBits is invoked, but in that case the first line of log was consistently missing in the web socket notification
This was the line that tests were looking for “Building app package” - and thus all those tests failed.
Patched the sample Hello-Java apps to have the manifest.yml define dummy SSH_CONNECTION environment variable
The sample apps check VCAP_APP_HOST and if its value is <IP_ADDRESS>, the value of SSH_CONNECTION was being parsed to retrieve the IP
In Cloudfoundry, VCAP_APP_HOST will be <IP_ADDRESS> for DEAs. So all the apps were throwing null pointer exception while attempting to parse the value of SSH_CONNECTION
Modified ManifestChoice.getDefailtManifestConfig to include the SSH_CONNECTION env var.
Built 1.1.3 version of cloundfoundry-client-lib-shaded and used the same instead of 1.0.6
The 1.0.6 version was not binary compatible with the Jenkins plugin version being used (v1.532.3) or later versions.
Any error in the CF rest api invocations caused Jenkins job to hang – the errors were not correctly handled and java.lang.VerifyError used to be thrown.
This was found to be due to binary incompatability of v1.0.6 shaded lib built using common-codec v1.3 vs the v1.8 of common-codecs packaged along with Jenkins
All negative test cases were stuck due to this and upgrading to v1.1.3 of cloud foundry-client-lib  resolved this issue – v1.1.3 has dependency on common-codecs v1.6 and it seems to work fine with run time dependency of v1.8
The cf-java-client/cloudfoundry-client-lib/pom.xml modifications are done from a forked version of cf-java-cllient (jayaramsankara\cf-java-client)
Maven shade plugin config is modified to include spring and codehaus classfiles relocation.

Even after the above changes, the following tests in CloudFoundryPushPublisherTest were not successful, which seems to be due to the assumption the tests make on the target cloud foundry environment.

testPerformServicesNamesManifestFile, testPerformCreateService, testPerformResetService
The service plan ‘free’ was not configured / found in the test cloud foundry instance.
testPerformEnvVarsManifestFile
The python app did not get staged since no build pack was detected.
The python-buildback was present in cloud foundry

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

Thanks for the pull request! Sorry for not getting back to you earlier, this is a pretty big change so it took me a while to go through it.
Unfortunately I will not be able to accept this. Here's a non-exhaustive breakdown:

In total 15 new methods and 1 new inner classes were added, which is overly complex. Some of those methods (such as postProcessBGDeployment) only call another function after a condition check. It would be better to do the condition in the main processOneApp method.

DeploymentInfo:

The added attributes of the DeploymentInfo class use getters and setters unlike the existing attributes, and cannot be set in the constructor like the other attributes. Some of those attributes are also unused (such as isBlueGreenInitiated or isNewAppToBeCreated).
DeploymentInfo is for attributes of the deployment of a single app, identically to what you'd find in a manifest.yml file. Something dealing with an existing app (such as the previous variable resetIfExists, and the newly added inner class ExistingAppHandler) is more something to put in the main CloudFoundryPushPublisher class.

Separating the log stream from the alternate method of getting the staging logs is a good idea. Registering for the log stream as soon as possible is better, and I can see how doing it right after the app push instead of right before might leave some logs missing on slow targets. It would be better as a seperate pull request though, since this fix is not related to the Blue/Green deployment feature.

Relying on a CloudFoundryException to see if an app doesn't exist seems error prone. An unrelated CFException could happen, and would be wrongly caught and interpreted as a non-existing app, instead of being handled by the catches from the calling method.

Some old code removed from the service creation update (v1.4) was added back, such as line 418, and the comments on lines 406+

Tests added:

The 4 tests added to CloudFoundryPushPublisherTest use an extra 9 functions and 1 inner class, which is overly complex.
The test added to DeploymentInfoTest is in the middle of an existing test, doesn't use the JUnit framework, and only checks if "get" is correct after "set".

Resources modified:

I'm not sure why adding dummy SSH_CONNECTION env var to the test manifests, and to the code of some tests, was needed. The hello-java sample has a null check and should work fine even is SSH_CONNECTION is not set. All tests pass for me using a pivotal.io target, and other users are also able to pass the tests on different targets.
I'm not sure if updating the shaded cf-java-client was necessary, since I'm not getting the common-codec incompatibility you're seeing. However this update was probably needed sometime in the future, so this change is good. It would probably be better in its own pull request, since it's not intrinsically linked to the Blue/Green deployment feature.

Syntax inconsistencies:

Some added lines use spaces, while some others use tabs. This project uses 4 spaces indentations.
Other minor code style inconsistencies such as newlines before brackets, or the lack of spaces after commas.
Auto-generated Javadoc for some functions, but not others. The Javadoc that is present is mostly redundant (such as the one for checkAndRestoreRoute which just repeats the name of the method)

This is quite an extensive list, so you might not have the time to continue working this. There are a couple of workable items noted above which I would be happy to accept as separate pull requests.
Thanks,
William Gautier

Code is submitted with tabs for indentation, but existing code uses spaces.

Thanks for the comments.
Let me split this in to multiple pull requests and get back.
On the need to add SSH_CONNECTION env var, the "hello Java " app was using the the value of SSH_CONNECTION when the IP read from VCAP_APP_HOST was not <IP_ADDRESS> and throws null pointer exception.


Update readme with clasp create gifs

[ ] npm run test succeeds.
[ ] npm run lint succeeds.
[x] Appropriate changes to README are included in PR.

Pull Request Test Coverage Report for Build 130

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

Totals

Change from base Build 128:
0.0%

Covered Lines:
116

Relevant Lines:
745

💛  - Coveralls

Pull Request Test Coverage Report for Build 130

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

Totals

Change from base Build 128:
0.0%

Covered Lines:
116

Relevant Lines:
745

💛  - Coveralls

Pull Request Test Coverage Report for Build 130

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

Totals

Change from base Build 128:
0.0%

Covered Lines:
116

Relevant Lines:
745

💛  - Coveralls


Fill in details for LP position page

The MVP code is now up, with a skeleton of the LP position page: https://dex-explorer.testnet-preview.penumbra.zone/lp/plpid1zh5ngkkhcnlzugaeztk74umu7r7ytq9ghurz6v25n6m7mzyzvruqa3fdve
Brainstorming an initial list of changes to fill in data:
Position Status Box

[x] Styling for the position ID: this should be in a monospace font, with a copy button. Could be good to have a dedicated component for rendering position IDs that takes the proto and handles formatting internally.
[ ] Reuse the web frontend's ValueViewComponent to display token amounts if possible. This is blocked on being able to use those components from external applications, which might have some complexity
[ ] Use this as an opportunity to harmonize on a common Penumbra asset registry (also needs coordination), so we can share token symbols and icons. In the web frontend we've avoided showing the raw denoms and instead use only the symbol field (which is always unset on-chain, hence we only show "known" assets and treat all others as "Unknown Asset").
[x] Use no more than 6 significant figures for the price (e.g., 2.19450 instead of 2.194503208363691)

Timeline
The visual design language for the timeline is that the left-aligned boxes are actions by the user who controls the position, and the right-aligned boxes are actions by the DEX engine interacting with the position (currently this is just filling against it)
General thought on the "timeline"-ness: can we put explicit time markers on the side of the current content? Maybe putting block heights on one side and date time on the other, in a greyed out font that blends with the background?
User Actions

[x] Position opening, corresponding to EventPositionOpen. This should show:

[x] The transaction hash that opened the position (with copy button). Later this could link out to another explorer but just being able to copy the tx hash is good for now.
[x] The initial reserves for the position
[x] The block height and time when the position was opened. As noted above this could maybe be beside / out the box.

[ ] Position closing, corresponding to EventPositionClose. This should show:

[x] The transaction hash that closed the position (with copy button)
[ ] Add the final reserves to the event (currently this is not included)
[x] The block height and time when the position was closed. As above this could be outside the box.

[x] Position withdrawal, corresponding to EventPositionWithdraw. This should show:

[x] The transaction hash that closed the position (with copy button)
[x] Show the sequence number of the withdrawal -- in the state model, it's now possible to withdraw multiple times, if some future change to the protocol created a facility for adding more tokens to the reserves after the initial withdrawal. Currently this isn't done in the protocol or in the client software but multiple sequential withdrawals are possible and should be able to be rendered.
[x] Show the reserves withdrawn from the position
[x] The block height and time when the position was closed. As above this could be outside the box.

DEX Actions
There may be many, many executions connected to a particular position. It would be good to have some way to optionally collapse these, maybe using some heuristics.
One concrete suggestion: if there are more than 2*N executions, show only the first N and last N trades, with a vertical ellipsis in between (maybe also with a count of how many items are elided). Clicking on the ellipsis shows all of the intervening executions. The user can restore the collapsed view by refreshing the page. Then, set N = 1, so we only show the first and last trades, but have code where we could easily change to show more or less if we change our mind.

[ ] Position execution, corresponding to EventPositionExecution. This should show:

[ ] The trade (A of X for B of Y) using reserves information in the event

Need https://github.com/penumbra-zone/penumbra/issues/3844 to land to do this
In the meantime, can display just the updated reserves, to exercise the pipeline of doing stuff with the event

[x] The height and time when the execution happened. Note that a position can be executed against multiple times per block.

Additional TODO for tracking:

[ ] It appears some LP views may show duplicate data items, investigate and resolve. Ex. https://dex-explorer.testnet.penumbra.zone/lp/plpid10yv7qrnrqjx3u9rdmrex8rcy2tavx5eaz480xh3dazjdtxxmcvjsj6rjwm

The EventPositionExecution events now include the previous reserves state (allowing complete trade display) as well as a context field with the end-to-end pair (allowing display of something like "while routing C -> D")


Multilang support

Please add multilang support.

sure! added to my to-do list

@ViBE-HU Added language support. However, since I only speak Russian/English, I didn't risk making translations myself. Currently, only Russian and English are available. Localization files can be found in the repository: po/ru.po, po/en.po. If you have 5 minutes to translate a dozen phrases into your language, I'd be very grateful (and will add the localization right away).

No worries. The community will gather up and do their part.
My translation is on the way. But I wait the update first. I would like to check my work before I add the commit.

Not sure what's going on, but I still don't get the update. I'm on Ubuntu 24.04. Are these related maybe?

@ViBE-HU fixed, reviewed!

Nice. Tested. I think it's time to close the issue. Thanks!


[IAC-1786] IAC 0.181.0 Release Notes

Description
What
IACM's first release notes entry. Previous entries will be backfilled about 6-8 weeks after the first entry approval.
JIRA: IAC-1786
PR lifecycle
PRs must meet these requirements to be merged:

[x] Successful preview build.
[x] Code owner review.
[x] No merge conflicts.
[x] Release notes/new features docs: Feature/version released to at least one prod environment.

 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.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66268c613783f11c8792999a--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6626bfce75565f30fad21f85--harness-developer.netlify.app


Create pool of streams to LNS

Will be done in https://github.com/helium/helium-packet-router/issues/237


swapValues error on nested fields

Minimal example here:
https://codesandbox.io/s/react-form-basic-form-hooks-only-6muwk?fontsize=14&hidenavigation=1&theme=dark
To reproduce:

Add two nested "Nicknames"
Try to move them up/down using the controls, which calls swapValues on their indexes
Observe str.replace is not a function error

It looks like the fields get passed as an array like ['nicknames', '0'] and ['nicknames', '1'] in swapFieldValues:
https://github.com/tannerlinsley/react-form/blob/master/src/hooks/useForm.js#L386
But getBy calls makePathArray which expects a string here:
https://github.com/tannerlinsley/react-form/blob/master/src/utils.js#L35
https://github.com/tannerlinsley/react-form/blob/master/src/utils.js#L101
I'm not sure if the correct solution is passing in a string from swapFieldValues or to modify makePathArray to check if it's already an array.

Hey all, I'm closing this issue as it appears to be related to a version of the old non-current @tanstack/react-form and friend APIs. We're looking to 1.x soon enough, and I want to more easily filter down to more recent issues.
Thanks for being early adopters of our tooling! We know that it can lead to issues like this being unresolved sometimes, but we're working hard on the new iteration of Form.
Maybe take a look and let us know what you think?
https://tanstack.com/form
Best,

Corbin


New post: 'There are more types than classes'

For posting on Monday, Feb 13.

This is fantastic. I'm going to put the URL on a hotkey. Didn't see any typos, although I'm kind of sleepy. 👍 👍 👍

Great article! I had a few pedantic comments -- which you are free to ignore if you don't think they add anything useful.

@S11001001 Would you be inclined to run this through tut? Recently, this blog gained the ability to do so automatically. See this post for an example.

So awesome. I remember seeing something similar on Robert Harper's blog.

This is great, but can't we get there a lot more quickly?
Each class has a unique, finite, name (including path) which corresponds to a finite character string. There are countably many finite strings so, by correspondence, there are countably many classes, ie. ω.
Now consider the set of unary function types A => B for all classes A and B. The size of this set is ω^ω, which is bigger than ω by the usual Cantorian diagonal argument.
QED.

@milessabin not sure if you're trolling, but surely you're being impredicative. Consider the set of unary function classes for all classes A and B. The size of this set is also omega^omega. You then arrive at a contradiction because the set of classes contains a set larger than itself.

Not trolling. There isn't a set of unary function classes. There can't be, because all classes have names and there aren't enough names to go around.

If there's only one unary function class and an infinite number of unary function types, does that not get you to your conclusion immediately?

So awesome. I remember seeing something similar on Robert Harper's blog.

@edmundnoble Yep! I've deliberately borrowed his terminology here, too. I wanted to target a different audience and work in more typeish goodies along the way.

This is great, but can't we get there a lot more quickly?

@milessabin Totally. This is the scenic route, with lots more good photo opportunities.

Totally. This is the scenic route, with lots more good photo opportunities.

I agree with that sentiment.

One thing I’ve noticed is that when I, as a reader, observe an incomplete statement, I tend to skip over the rest of the article in my dire need to provide feedback to the author, letting the world know that he’s wrong :-)
Classes are relevant for OOP languages and are (also a) runtime thing because the OOP (subtype) polymorphism works at runtime, for any call-site the JVM needing to know (1) the interface + method you try to call and (2) the object’s vtable in order to link that method with an actual implementation, according with Java's single dispatching rules.
Such a reader, familiar with what OOP means and Java, will nod starting from the first section where it is said more or less that types are about variables and not objects, which isn't true, unless you also think dynamic languages have a single type, which IMO is a subjective opinion and does not help in making a distinction at all. A more useful comparison is to talk about what gets checked and inferred at compile time and what gets deferred to runtime, which is like a slider that gets moved closer to compile time for languages like Haskell and ML and closer to runtime for languages like Lisp.

That is why ... total type erasure is a desirable feature rather than a design flaw—plugging these leaks gives the programmer as much freedom to abstract by information-hiding as she wishes.

I've already heard the opinion from @travisbrown that he'd prefer total type erasure, but I don't agree. Even C++ has RTTI, allowing it to safely perform casts, for an OOP language you need casting and if you have casting you need isInstanceOf checks as well, otherwise you can have memory vulnerabilities. If you think isInstanceOf checks are not useful in the presence of type erasure for generics, I personally disagree, because those checks are going to happen anyway, done by the runtime and as a developer I'd rather have access to such checks, especially for doing optimizations.

Would you be inclined to run this through tut?

@larsrh I'll play with it this week, but would rather publish today with what exists now.

misunderstandings about classes vs types, and also type conformance vs type equality. (You touch on the latter point, though I think there's probably enough material for another blog article on this.)

@propensive I think comparing Leibniz and Liskov is the best approach to understanding the difference. Alas, such a treatment also requires delving further into introductory variance, a well which has by now run dry for me. Perhaps one of Typelevel's many fans of subtyping might like to have a go?

Given the title, I might also be tempted to mention, as an aside, the existence of types which don't (and can't possibly) have values, but which the compiler can still talk about

@propensive shh :)

unless you also think dynamic languages have a single type

@alexandru Yep! That's me. I mention this later in the article.

I'll play with it this week, but would rather publish today with what exists now.

Fine with me :+1:

@tpolecat @non @larsrh @edmundnoble @milessabin @alexandru Thank you very much for your comments on the draft.


Fix regression with shadows rendering

This should fix #100850.
Draft for now as I couldn't test it properly yet.
Will get back to it tomorrow.

I cherry-picked this pr but seems the same error persists.

Should be okay now.
The fix ensures the shadow far distance never goes below near (= 0.005).
This guarantees the projection matrix stays inversible.

Thanks!


Some BUG about CUDA KERNEL

If set matrix dimension N smaller than 512(e.g. 256, 128) when running benchmark, the result of the xnor_gemm has error.

If set matrix dimension N smaller than 512(e.g. 256, 128) when running benchmark.
the result of xnor_gemm has error.

Yeah, it was really just a proof of concept cuda kernel.
Not really optimized or useful for production.


Fix includeSpringEnv can not work

What is the purpose of the change
Fix https://github.com/apache/dubbo/issues/9207
Brief changelog
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #9259 (6e4a206) into 3.0 (3b9d6af) will decrease coverage by 0.13%.
The diff coverage is 83.78%.

@@             Coverage Diff              @@
##                3.0    #9259      +/-   ##
============================================
- Coverage     64.76%   64.63%   -0.14%
+ Complexity      329      328       -1
============================================
  Files          1206     1209       +3
  Lines         51802    51839      +37
  Branches       7716     7687      -29
============================================
- Hits          33551    33505      -46
- Misses        14637    14710      +73
- Partials       3614     3624      +10

Impacted Files
Coverage Δ

...g/apache/dubbo/config/spring/ConfigCenterBean.java
28.12% <0.00%> (ø)

...zeDubboConfigBeanInAdvanceApplicationListener.java
92.85% <92.85%> (ø)

...pache/dubbo/config/spring/util/DubboBeanUtils.java
77.77% <100.00%> (+1.85%)
:arrow_up:

...DubboConfigurationPropertiesAutoConfiguration.java
100.00% <100.00%> (ø)

...rationPropertiesToBeanDefinitionPostProcessor.java
100.00% <100.00%> (ø)

.../remoting/api/SingleProtocolConnectionManager.java
61.53% <0.00%> (-23.08%)
:arrow_down:

...ache/dubbo/remoting/transport/AbstractChannel.java
75.00% <0.00%> (-12.50%)
:arrow_down:

...dubbo/common/status/support/LoadStatusChecker.java
50.00% <0.00%> (-11.54%)
:arrow_down:

...mmon/threadpool/support/fixed/FixedThreadPool.java
80.00% <0.00%> (-7.50%)
:arrow_down:

...ng/transport/dispatcher/all/AllChannelHandler.java
62.06% <0.00%> (-6.90%)
:arrow_down:

... and 47 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 3b9d6af...6e4a206. Read the comment docs.

Replaced by https://github.com/apache/dubbo/pull/9266


[fix] null char and empty/whitespace string

The previous custom padding scheme was not always reversible if the secret contained null characters. Use Crypto.utils pad/unpad (PKCS padding) instead which is reversible.

This change is not compatible, but I did not rev the engine version because I was told we're the first to use the new version

Remove logic for not encrypting empty/whitespace strings as it is not necessary and they are still valid secrets

Accepting this merge request now. It will become part of encdec version 3, which is not officially supported.
The first official Github release of qpylib will have encdec version 4 containing further changes. At that point we will officially support 2 encdec versions: 4 and pre-3.


Add a Rate Us Option

Description
Hello, my name is A.A.D.Sailaja. I am a Tech Enthusiastic and i'm looking forward to contribute in open source projects. Here in this project i would like to add a rate us option it will allow the user to rate our website and also he can suggest us to improve if any
Can you please assign me this issue under gssoc'24
Screenshots
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)

Congratulations, @sailaja-adapa! 🎉 Thank you for creating your issue. Your contribution is greatly appreciated and we look forward to working with you to resolve the issue. Keep up the great work!We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

You can go ahead with this, but where and how do you plan to keep this? @sailaja-adapa

I will place this Rate Us option in footer when user click on that it will head to other page @pooranjoyb

I will place this Rate Us option in footer when user click on that it will head to other page @pooranjoyb

Do one thing, instead of doing that, write something like, support out project and then redirect them to the github repository to star the repo.
Do this change on the same PR for #315