Synthesised League item properties

This PR adds support for synthesised and fractured item properties.  The synthesised property is a simple boolean, while the fractured property is a boolean along with a list of string mods.  These mods are displayed before any explicit mods on the item. (Any synthesised mods simply show up as implicit mods.)
Note: I do not have the icon adorners yet, as we've only seen them in screenshots so far (e.g., synthesised and fractured).  I'll add the image files for those icons once I get my hands on them.
#965

From: https://www.pathofexile.com/forum/view-thread/2333648
Enchantments (such as those found from the Labyrinth) now have their own mod slot on items. They no longer use the Implicit mod slot.

I can support for enchantments once we have a bit more information on them (e.g., what are the JSON properties named, where do they sit in the tooltip relative to implicit mods, what color to use?).
I've guessed the URL for the fractured symbol is https://web.poecdn.com/image/item/popup/fractured-symbol.png .  However, I haven't figured out the URL for the synthesised symbol yet.

Oh for sure, was just a note so we remember :)

Was hoping to get this change in myself this week, but didn't get any time unfortunately. Thanks for getting on it.
Synthesis symbol is here: https://web.poecdn.com/image/item/popup/synthetic-symbol.png?1550034833000

https://www.pathofexile.com/forum/view-thread/2333275
Sample Synthesised and Fractured json snippets are in this forum post.
If you could add some Unit Tests that leverage these snippets that would be amazing!

Thanks for finding the location of the synthesised icon!  I've added both icons and a unit test to parse the sample JSON.


jib.extraDirectories.permissions to change folder/file permissions

Could anyone please let me know whether jib.extraDirectories.paths and  jib.extraDirectories.permissions params set in gradle jib build changes the directories permissions?
In my tekton pipeline, I have the below command inside the script param of Gradle 4.6 image (in Docker repo.)
gradle wrapper  build service:jib --no-build-cache --recompile-scripts -S -i --debug --scan  -x test
-Djib.container.user='1000:1000'
-Djib.container.user='gradle:gradle'
-Djib.extraDirectories.paths = ['/workspace/app', '/workspace/app/service']
-Djib.extraDirectories.permissions = ['/workspace/is360api': '775', '/workspace/app/service': '775'] \
Environment:
Tekton Pipeline , Kubernetes 1.18, Gradle 4.6 and Jib 4.0.4

Jib version: 4.0.4
*Build tool: Jib Gradle *

Description of the issue:
jib.extraDirectories.paths and  jib.extraDirectories.permissions params set in gradle jib build  doesn't seem to be working as expected.
Expected behavior:
jib.extraDirectories.paths and  jib.extraDirectories.permissions params changes the folder/file permissions when set in gradle build jib command.
Steps to reproduce:

Invoke the gradle build jib command (provided above) inside the Tekton Pipeline Step (in Kubernetes )  in script param of the Gradle 4.6 image

Error Message:
service-build-step] 09:16:33.389 [ERROR] [org.gradle.internal.buildevents.BuildExceptionReporter] Caused by: org.gradle.internal.service.ServiceCreationException: Could not create service of type CrossBuildFileHashCache using BuildSessionScopeServices.createCrossBuildFileHashCache().
[service-build-step] 09:16:33.389 [ERROR] [org.gradle.internal.buildevents.BuildExceptionReporter] 	at org.gradle.internal.service.DefaultServiceRegistry$FactoryMethodService.invokeMethod(DefaultServiceRegistry.java:857)
[service-build-step] 09:16:33.389 [ERROR] [org.gradle.internal.buildevents.BuildExceptionReporter] 	at org.gradle.internal.service.DefaultServiceRegistry$FactoryService.create(DefaultServiceRegistry.java:808)
[service-build-step] 09:16:33.389 [ERROR] [org.gradle.internal.buildevents.BuildExceptionReporter] 	at org.gradle.internal.service.DefaultServiceRegistry$ManagedObjectServiceProvider.getInstance(DefaultServiceRegistry.java:612)
[service-build-step] 09:16:33.389 [ERROR] [org.gradle.internal.buildevents.BuildExceptionReporter] 	at org.gradle.internal.service.DefaultServiceRegistry$SingletonService.get(DefaultServiceRegistry.java:669)
[service-build-step] 09:16:33.389 [ERROR] [org.gradle.internal.buildevents.BuildExceptionReporter] 	at org.gradle.internal.service.DefaultServiceRegistry$FactoryService.assembleParameters(DefaultServiceRegistry.java:821)
[service-build-step] 09:16:33.390 [ERROR] [org.gradle.internal.buildevents.BuildExceptionReporter] Caused by: org.gradle.api.UncheckedIOException: Failed to create parent directory '/workspace/app/service/.gradle' when creating directory '/workspace/app/service/.gradle/4.6/fileHashes'
Thanks in advance!

I'm sorry, I think I was confusing. And I am still confused. Seems like both you and I are confused. We need to make sure we are on the same page.
I thought you were using Jib to create another customized Gradle image based on gradle:4.6 and you were getting an error when you run a usual, non-Jib Gradle build when you run your new customized Gradle image. So, forget what I said in https://stackoverflow.com/a/61488918/1701388. I've updated the SO answer.
Rather, now I see this is what you are doing.

Inside a Tekton step, you use the official gradle:4.6.

That is, it is NOT that you are building another customized Gradle image from the base image gradle:4.6 while using Jib.

This Tekton pipeline is a general-purpose image builder; the purpose is to containerize any Java application given as input. And for doing containerization, you use Jib.

The only place you run Jib is this Tekton pipeline. That is, you are not building an image (by Jib or Docker) where you plan to run Jib inside the image. Jib just runs on the vanilla gradle:4.6.

The error you are getting is from running ./gradle jib inside the Tekton step.

It is NOT from when running an image built by Jib.
It is NOT from when running Gradle inside an image built by Jib.

If my understand is right, then these are meaningless, because it is the container configuration for the image being built by Jib. These don't affect the runtime environment (that is, gradle:4.6) where Jib is running.
-Djib.container.user='1000:1000'
-Djib.container.user='gradle:gradle'

And it is possible that these may be meaningless for the same reason. These are the settings for the image being built by Jib.
-Djib.extraDirectories.paths = ['/workspace/app', '/workspace/app/service']
-Djib.extraDirectories.permissions = ['/workspace/is360api': '775', '/workspace/app/service': '775'] \

I think there are serious confusions. From the command ./gradlew ... service:jib and the directory name /workspace/..., I think you meant these directories are the directories inside the gradle:4.6 in Tekton, not the directories to be put inside an image being built by Jib.
Therefore, I believe that running the following Gradle command without Jib inside this Tekton step will just fail as well.
gradle service:build

If my understand is correct, this is not a Jib issue and not something Jib can resolve. It's just that you cannot run Gradle with gradle:4.6 on Tekton due to the unfortunate requirement that Gradle has to run with the user gradle:gradle in gradle:4.6.

Closing this as this is not a Jib issue. Some related stuff is also explained in https://github.com/tektoncd/catalog/issues/289#issuecomment-621301966.
Here, the problem seems to be that running the gradle command inside the image gradle:4.6 on Tekton fails, as the gradle command cannot write files into /workspace/app/service inside gradle:4.6 due to permission/ownership issues. There may be multiple different ways to resolve this issue depending on how you set up your Tekton pipeline. For example, there may be a way to direct gradle not to create .gradle inside a project directory. Or, you may be able to configure the Tekton task so that /workspace/app/service is writable by a non-root user inside gradle:4.6.
Again, -Djib.container.user is for the user of the image that Jib will create; it doesn't/can't change the user running gradle:4.6 on Tekton. The same goes for -Djib.extraDirectories.
I am closing this, but feel free to update. I did have an issue of the same nature when I tried to create the Jib Gradle Tekton catalog sample.

FTR: issue resolved after @2020testuser configured a Tekton Task so that Gradle can create necessary files.
https://github.com/tektoncd/catalog/issues/289#issuecomment-621895563


WIP Add delete bitmap for DSIP-018

Proposed changes
Add delete bitmap for
DSIP-018: Support Merge-On-Write implementation for UNIQUE KEY data model
Problem Summary:
Checklist(Required)

Does it affect the original behavior: (No)
Has unit tests been added: (Yes)
Has the document been added or modified: (No Need)
Does it need to update dependencies: (No)
Are there any changes that cannot be rolled back: (No)

Further comments

@compiletheworld  the formatter failed, pls fix it.


Fix IBV subtype/genotype parsing issue in subtyping report generation

Fixes for issue https://github.com/CFIA-NCFAD/nf-flu/issues/32

PR checklist

[ ] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!

[ ] If you've added a new tool - ensure that you've added the version info to a versions.yml and added it to the ch_versions channel in the workflow you added the tool to.
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] If necessary, also make a PR on the CFIA-NCFAD/nf-test-datasets repo

[ ] Ensure the test suite passes (nextflow run . -profile test_{illumina,nanopore},docker).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

@nhhaidee The metadata table should contain a Genus field with information about whether the top matches for each segment are Alphainfluenzavirus (IAV) or not. Non-IAV will be a different genus like Betainfluenzavirus for IBV. I dropped the original species column since it was essentially uncleaned free-form text while Genus contained curated information that could be useful for further analyses.
The metadata table also contains a Segment field which may be useful for checking that the sample's segment designation lines up with the reference's segment.


repo.dev.wixpress.com is down

npm install fails due to this. I'm not sure where else to report this. Thanks!
$ npm install
npm WARN deprecated<EMAIL_ADDRESS>use 'getPackages' from @lerna/project
npm WARN deprecated<EMAIL_ADDRESS>The functionality that this package provided is now in @npmcli/arborist
npm WARN deprecated<EMAIL_ADDRESS>this library is no longer supported
npm WARN deprecated<EMAIL_ADDRESS>Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.
npm WARN deprecated<EMAIL_ADDRESS>request has been deprecated, see https://github.com/request/request/issues/3142
npm ERR! code ENOTFOUND
npm ERR! syscall getaddrinfo
npm ERR! errno ENOTFOUND
npm ERR! network request to https://repo.dev.wixpress.com/artifactory/api/npm/npm-repos/which-boxed-primitive/-/which-boxed-primitive-1.0.2.tgz failed, reason: getaddrinfo ENOTFOUND repo.dev.wixpress.com

This seems a bit odd. Why is which-boxed-primitive using a non open-code hosted version from wixpress vs the standard library? https://www.npmjs.com/package/which-boxed-primitive

@nirnaor @shmuelhizmi

@yhattav I'm on it

@Fmstrat can you retry now?

It seems like there is a network issue that is preventing your computer from connecting to the https://repo.dev.wixpress.com/artifactory/api/npm/npm-repos/ URL. The error message "ENOTFOUND" is indicating that the hostname "repo.dev.wixpress.com" cannot be resolved. This could be caused by a number of issues including a problem with your internet connection or a firewall blocking the connection.
You can try resolving this issue by doing the following:
Check your internet connection and make sure it is stable and working properly
Try disabling any firewalls or anti-virus programs that may be blocking the connection
Try using a different network or internet connection to see if that resolves the issue
If none of the above steps work, you can try contacting the system administrator of https://repo.dev.wixpress.com/artifactory/api/npm/npm-repos/ or the minecraft support team for assistance.
It's also worth noting that some of the packages that you are trying to install are deprecated, so it's possible that this issue is related to the outdated packages you are trying to install. You may want to try updating these packages to their latest versions if possible.

Great thanks for sharing!

The issue you're encountering is an ENOTFOUND error during the npm install command. This error typically indicates a problem with resolving the network address or connecting to the required resources. To address this problem, you can try the following steps:
Verify your internet connection: Ensure that you have a stable and working internet connection.
Clear the npm cache: Run the following command to clear the npm cache, which can sometimes resolve network-related issues:
npm cache clean --force
Retry the installation: Run npm install again and check if the issue persists. It's possible that the error was transient and can be resolved by retrying.
Check your network configuration: Ensure that you don't have any proxy or firewall settings blocking the npm registry or the specific resource being accessed during the installation. Adjust your network configuration if necessary.
Use a different network or VPN: If you're on a restricted network, try switching to a different network or use a VPN to bypass any limitations or restrictions.
If none of these steps resolve the issue, you may want to reach out to the package maintainers or consult the npm support channels for further assistance, as it could be an issue specific to the package or your network environment.

@robert-micheal no need, just delete the lock file and reinstall, I'll make sure we will fix it from our side soon


read-only-properties-mixin lit v2

Hi! 👋
Firstly, thanks for your work on this project! 🙂
Today I used patch-package to patch<EMAIL_ADDRESS>for the project I'm working on.
This patch updates read-only-properties-mixin to lit v2, and should be added as part of our upgrade strategy
Here is the diff that solved my problem:
diff --git<EMAIL_ADDRESS><EMAIL_ADDRESS>index b6aba38..41bda1c 100644
---<EMAIL_ADDRESS>+++<EMAIL_ADDRESS>@@ -1,10 +1,8 @@
-// when we update to typescript 3.8
-// import type { LitElement } from "lit-element";
-import { LitElement } from "lit-element";
+import type { ReactiveElement } from 'lit';

 declare type Constructor<T = {}> = new (...args: any[]) => T;

-declare module 'lit-element/lib/updating-element' {
+declare module '@lit/reactive-element/reactive-element' {
   interface PropertyDeclaration {
     readOnly?: boolean;
   }
@@ -22,6 +20,6 @@ export interface ReadOnlyClass {
 /**
  * Enables the nofity option for properties to fire change notification events
  */
-export declare function ReadOnlyPropertiesMixin<T extends Constructor<LitElement>>(superclass: T): T & Constructor<ReadOnlyClass>;
+export declare function ReadOnlyPropertiesMixin<T extends Constructor<ReactiveElement>>(superclass: T): T & Constructor<ReadOnlyClass>;

 export {};

This issue body was partially generated by patch-package.

Conclusion: just break forwards for lit 2,

breaking changes:

breaking types for read-only mixin
remove spread helpers (in favour of lit-labs)


import 'generated_plugin_registrant.dart'; from first_app|lib/MAIN/python_web_entrypoint.dart at 5:1

import 'generated_plugin_registrant.dart'; from
first_app|lib/MAIN/python_web_entrypoint.dart at 5:1

It's not really clear what the issue is, because you haven't filled out any of the template. Please file a new bug with more details.


Vulkan funcs should return unsafe.Pointer instead of uintptr

VulkanCreateSurface(instance interface{}) returns an uintptr which is a dangerous behaviour hence uintptr is not updated by the garbage collector. It should return the unsafe.Pointer instead which is updated.
In VulkanGetVkGetInstanceProcAddr() it is done correctly.
if you use VulkanCreateSurface() and try to cast it to a vkSurfaceKHR, you need to cast it back to unsafe.Pointer which then causes a warning when call "go vet": "possible misuse of unsafe.Pointer"

Hi @Nightwulf, thanks for the feedback! I've updated the function to return unsafe.Pointer instead.

Hey @veeableful,
thank you for the fast reply and change. Go vet no longer moans now!


Update inventory to 0.17.1.Final

Note that the itests still fail for me, due to failures with agent tests. That's because agent has not been updated in services to either use the new API or use the deprecated endpoint on its new location.

Closing this in favor of #34.


add extra step

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


BatchNormalization: no error on wrong shape or data type of the input

If the batch normalization layer uses cuDNN and either the input dimension does not match or the dtype of the input data is not the right one, no error is raised, BUT the outputs are inf or nan. If one executes the code on the CPU (set GPU parameter to -1) or you set self.use_cudnn = False manually in line 84 in chainer.functions.batch_normalization, then the error (ValueError: Broadcasting failed or TypeError: Type is mismatched [...]) is displayed.
In my case it was also even worse: the data on the GPU was altered and no error was rised there as well. My program crashed, since the data was used as indeces and they were pointing outside the array. Unfortunately, I cannot reproduce my scenario, but I think it would be a nice to have some checks whether the shape and data type are correct.
Runtime Info:
Chainer: 4.2.0
NumPy: 1.14.3
CuPy:
  CuPy Version          : 4.0.0
  CUDA Build Version    : 9000
  CUDA Driver Version   : 9010
  CUDA Runtime Version  : 9000
  cuDNN Build Version   : 7102
  cuDNN Version         : 7102
  NCCL Build Version    : 1000

Code to reproduce:
import chainer
import chainer.links as L
import numpy as np

######## PARAMS ########
GPU = 0
INPUT_TYPE = np.float32
INPUT_DIMS = 1024 # or 512 to make it work
BN_DIMS = 512
########################

class Model(chainer.Chain):
	def __init__(self):
		super(Model, self).__init__()
		with self.init_scope():
			self.bn = L.BatchNormalization(BN_DIMS)

	def __call__(self, X):
		return self.bn(X)

# std=2, mean=1
data = np.random.randn(16, INPUT_DIMS) * 2 + 1

model = Model()
var = chainer.Variable(data.astype(INPUT_DTYPE))

if GPU >= 0:
	var.to_gpu(GPU)
	model.to_gpu(GPU)
	chainer.cuda.get_device(GPU).use()

out = model(var)

print("Input  Mean: {: 8.4f} Std: {: 8.4f}".format(
	float(var.array.mean(axis=1).mean()),
	float(var.array.std(axis=1).mean()))
)
print("Output Mean: {: 8.4f} Std: {: 8.4f}".format(
	float(out.array.mean(axis=1).mean()),
	float(out.array.std(axis=1).mean()))
)

Hi, @BloodyD

Could you check the version of installed cupy?
I tried to run your snippet (thank you for sharing), but I cannot install cupy==4.0.0 because it is incompatible with chainer==4.2.0.

some checks whether the shape and data type are correct.

From v5, Chainer checks shapes and dtypes like this.
https://github.com/chainer/chainer/blob/3391a4ad4952134f463d811dfd0a2df58d9ec093/chainer/functions/normalization/batch_normalization.py#L62-L84

Hi @crcrpar,
thank you for the fast reply! Sorry, that I'm answering so late.

My installed cupy version is really 4.0.0 and it works quite fine with chainer 4.2.0. Are there some known issues, that may cause some problems?

The same code is actually already present in chainer 4.2.0. This made me curious. And of course the main source of errors sits right behind the monitor >.< In my environment I have disabled type checks to speed up some things:

export CHAINER_TYPE_CHECK=0

After removing this chainer behaves correctly! I'm closing the issue, since it was my own fault.


chore(examples/components): fix typo, update component type

In fact, it would be interesting to start rebuilding and updating them.

That's exactly what I meant – most of the examples still use text-based commands, which are discouraged nowadays and should be moved over to slash commands instead.


Cache multiple paths and add glob pattern support

This adds the ability to specify multiple items to cache in a single run (#16, #180). Additionally, the items to cache can be located with glob patterns. Paths/patterns are specified as new-line delimited input to the path parameter.
Because this adds multiple directories to the tarball that is cached, each path is made relative to the GITHUB_WORKSPACE env variable. If paths are outside of this directory, the tarball can contain a backtraced path (e.g. ../foo/bar). Because of this the -P flag is used with tar when unpacking the cache.
I have a couple of open questions:

If no paths are found (e.g. a glob pattern found no files/directories), should the action fail intentionally?
Because the tarball entries are now relative to GITHUB_WORKSPACE older cache entries will not be compatible with these changes. Is there a preferred way to add a salt value to the cache key?

Some examples:
    - name: Restore multiple relative paths
      uses: actions/cache@v3
      with:
        path: |
          node_modules
          dist
        key: ${{ runner.os }}-${{ hashFiles('**/package-lock.json') }}

    - name: Restore multiple absolute paths
      uses: actions/cache@v3
      with:
        path: |
          ~/foo/bar
          /users/mona/cache-me
        key: ${{ runner.os }}-${{ hashFiles('key.txt') }}

    - name: Restore from glob patterns
      uses: actions/cache@v3
      with:
        path: |
          **/*.ts
          **/node_modules
        key: ${{ runner.os }}-${{ hashFiles('key.txt') }}

Thanks for working on this!
We should look at using the version field in the server API.  The idea is you compute a hash of any important settings used by the cache action, which in this case is the path, so when the path changes the version changes.  Otherwise, if someone were to say add a new line to the path but forgot to change the key, you would get the old cache entry that does not include the new path.

The version field sounds like a perfect fit - is it exposed as a url param here?
Also, what should the initial value be? 1.0?

This is great! Thank you @ethanis!

Can you change to pass the path as Collections? It will be more yaml-like.

@imbsky Lists are currently not supported, see https://github.com/actions/toolkit/issues/184.  Ideally, once that is fixed, we would update this to support collections (as well as remain backwards compatible with multiline strings)

I see. Well, I don't think there is a big problem so this looks good.

@joshmgross is there anything else that needs to be changed in this PR?

this finally makes this actions usable without thousands of unncecessary lines when in a yarn monorepo.

@joshmgross is there anything else that needs to be changed in this PR?

I'll take a look later today, but should be all good

I would like to have one example added as part of this change that uses multiple paths.  @boredland perhaps you have one in mind?

@chrispat I think the lerna example would be helpful:
- name: restore lerna
      uses: actions/cache@v3
      with:
        path: |
          node_modules
          */*/node_modules
        key: ${{ runner.os }}-${{ hashFiles('yarn.lock') }}

@boredland do lerna projects have multiple yarn.lock files? If so, should the key should be based on all of them?
- name: restore lerna
   uses: actions/cache@v3
    with:
      path: |
         **/node_modules
      key: ${{ runner.os }}-${{ hashFiles('**/yarn.lock') }}

Lerna and yarn workspace have only one lock file at root.

@boredland, @chrispat I added an example for lerna here

Thanks for the great work! I'm using Yarn and Lerna, looking forward for v2.
@imbsky Lerna uses yarn.lock at root but also in every package.
@ethanis Is it possible to add in the doc that the v2 is not released yet. I spent some time trying to understand why the example wasn't working.

Thanks @gcourtemanche! Fixing that here: https://github.com/actions/cache/pull/242

Dose is support ignore path somehow ?
      - name: restore yarn workspaces cache
        uses: actions/cache@master
        with:
          path: |
            node_modules
            */*/node_modules
           !packages/notyou/
          key: ${{ runner.os }}-${{ matrix.node }}-${{ hashFiles('**/yarn.lock') }}

Dose is support ignore path somehow ?
      - name: restore yarn workspaces cache
        uses: actions/cache@master
        with:
          path: |
            node_modules
            */*/node_modules
           !packages/notyou/
          key: ${{ runner.os }}-${{ matrix.node }}-${{ hashFiles('**/yarn.lock') }}

Ignoring paths should be supported. The paths are evaluated by the @actions/glob package, see https://github.com/actions/toolkit/tree/master/packages/glob#exclude-patterns


Suggestion to Simplify Endpoints for Follow and Unfollow Operations in User Service

Hello @SohamRatnaparkhi, great job on your project! I have a suggestion regarding the User service API. Instead of using the /follow?toFollowId= endpoint, could we replace it with /follow?id=, and similarly, replace /unfollow?toUnfollowId= with /unfollow?id= I believe using toFollowId and toUnfollowId after follow and unfollow is redundant. Additionally, I think that using /follow?id= and /unfollow?id= would be more concise and meaningful.

Okay cool!
Will refractor the code soon.
Thank you!


Some questions of the pipeline

After reading the documents, I have other questions in main.sh.
1.What is the difference about CARD gene_homology and CARD gene_variant. The two same command only make same results.
2.What to do next when we get CARD_gene.bam file.
3.Author also need to warn the user to use the latest samtools version.
4.The command "samtools view -bS $OUT_DIR/$sra.CARD_gene.sam | samtools sort - $OUT_DIR/$sra.CARD_gene.bam" need to be changed "samtools view -bS $OUT_DIR/$sra.CARD_gene.sam | samtools sort -o $OUT_DIR/$sra.CARD_gene.bam"

Thank you for your interest in this project.  I have updated the documentation in my fork.

Please refer to the CARD database for detailed description of their data
In the updated version, we made some suggestions about possible next steps, which is beyond the scope of this project.
3&4) Yes and a Docker container is available for this reason.

I am sorry. I do not understand the possible next steps. Docker is just a Container which have some software, but what I need is command sets.  How to convert the bam file to output format file and dot plot? Could you provide the example which you deal with ERR1600439 and ERR1600437 data?


Update get_all Method in db.py to Retrieve All Items from DynamoDB table

This update modifies the get_all method to ensure that all items from the DynamoDB table are retrieved during a scan operation.
Issue #, if available:
#639
Description of changes:
Update get_all Method in db.py to Retrieve All Items from DynamoDB Table
Reason for Change:

This update modifies the get_all method in the db.py file to ensure all items from the DynamoDB table are retrieved during a scan operation.
It addresses an issue in Lab 4 of the Personalization workshop, where discrepancies between reranked and unranked product lists occur due to incomplete data retrieval.

Updated get_all method:
def get_all(self):
    items = []
    response = self.table.scan()
    while 'LastEvaluatedKey' in response:
        items.extend(response['Items'])
        response = self.table.scan(ExclusiveStartKey=response['LastEvaluatedKey'])
    items.extend(response['Items'])
    return items

Context:
In Lab 4, unranked_product_ids are pulled from the featured items table, subsequently used to obtain a reranked list of products from "all" products via AWS Personalize. The existing get_all method's use of DynamoDB table scan operation only retrieves a limited subset of items by default, thus the reranked list may contain fewer items than the unranked list.

Description of testing performed to validate your changes (required if pull request includes CloudFormation or source code changes):
Test:
all_products_resp = requests.get('http://{}/products/all'.format(products_service_instance))

all_products = all_products_resp.json()
print(len(all_products))

Output is 2465 representing all products.

Further tests were performed to ensure that both reranked and unranked lists now contain the same number of items when fetched for side-by-side comparison in Lab 4.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks for the contribution !

Thank you @BastLeblanc !


<IP_ADDRESS>.0

This PR updates the partner version to <IP_ADDRESS>.0 for release.
Can not merge.  Do not merge.

Released: https://github.com/ChartBoost/chartboost-mediation-android-adapter-unity-ads/releases/tag/v<IP_ADDRESS>.0


docs: fix simple typo, represention -> representation

There is a small typo in transcrypt/modules/org/transcrypt/compiler.py.
Should read representation rather than represention.
Semi-automated pull request generated by
https://github.com/timgates42/meticulous/blob/master/docs/NOTE.md

Merge conflicts with newer version. Now fixed in separate branch.


Use separator in stacktrace layout renderer even when format is raw

NLog version:  4.6.3
Platform:  Xamarin Android / Xamarin iOS
I don't know if it's been coded like that by design, but when the ${stacktrace} layout renderer is in format=Raw, the separator property isn't used. You can clearly see that in the source code:
https://github.com/NLog/NLog/blob/dev/src/NLog/LayoutRenderers/StackTraceLayoutRenderer.cs#L127
This feature request is to enable the separator property even when format=Raw.
Thanks

Good one! I think the separator option is added later.
I propose to change this in 5.0 (next non-patch release)

Is the default of " => " maybe wierd in combination with raw?


Set the header (at the day view) when the minSplitWidth is set to a high number to follow the vertical scroll

Hey @antoniandre!
Thanks for this great calendar. The calendar is just what I needed. Great job :)
Just one question about the day view in vue-cal. Is there a way to the header follow the vertical scroll when there is more splits than the screen can support (in this case, the horizontal scroll is enable)?
I am attaching images to exemplify my need.
This first image is showing the calendar with 3 splits (lanes). In this case, the minSplitWidth is set to 0 (zero), so the header with the labels Quadra 1, Quadra 2 e Quadra 3 scrolls with the page normally. So if I have an event at my calendar, I can easily identify which court (quadra) the event is tied to.

But when I set the minSplitWidth to 80, for example, the horizontal scroll is enabled (and it's awesome. It's just what I want) and the header (Quadra 1, Quadra 2, ...) doesn't follow the vertical scrolling.

I am making a Nativescript app and this is the best calendar with resources!
Thanks in advance.

Hi @giulianimartini,
Thanks for your feedback.
I know what you are talking about, I tried to do it at the beginning of the development of Vue Cal but was unsuccessful because of all the different options possible in vue-cal.
There is a way in CSS3 using the sticky position, this will not work on IE though.
Other than CSS which I think is great, doing this via Javascript would certainly be overkill.
Hope it helps, let me know if you figure out, I could add it in Vue Cal. ;)

Hi @antoniandre,
Thanks for your time.
I am a bit busy with this app development right now, but I will give it a try, maybe the CSS way, since I'm using the vue-cal in a webview (with chromium, not IE).
BTW, your app looks great!
Thanks, but the beauty here is this awesome calendar :D

Hi @antoniandre,
I tried the sticky position, but it simply does nothing (lastest Chrome).
.vuecal__weekdays-headings {   position: sticky; }
Do you have any other idea?

Hi @antoniandre,
I have experimented and researched a lot at the weekend. I think there is no pure CSS solution.
Position sticky has a problem: When the parent element has any overflow rules, it stops working. The only way is to set a height to that element (other than 100%), which requires some JS.
But even with that sticky solution there are some ugly glitches, like the scrollbar is hidden behind the header, issues with the time column and so on.
Alltogether the sticky solution feels somehow makeshift, in opposite to the rest of the calendar, which is incredible clean and tidy.
IMHO the only reasonable solution would be to bring back the weekday-headings including the split-days-headers to their place, the header.
Also the the all-day-bar needs to be there (or, even better, in a separate container between header and scrollable area).
Then a few lines of JS would be needed to sync the x-axis. But that's not more than 10-20 lines or so, I think.
With this modification, the calendar with split enabled would work like all calendars (Outlook Calendar, Apple Calendar, Google Calendar).
Do you think there is a chance that you could go in this direction? I could try to support you with some contributions.

Here are some links:
Overflow issue: https://uxdesign.cc/position-stuck-96c9f55d9526
He suggests to use the syncscoll library, which is indeed overkill and 1k minified.
A lengthy discussion on w3c, but there I found the hint with the fixed height: https://github.com/w3c/csswg-drafts/issues/865
A JSFiddle with a super compact JS solution (3 lines): http://jsfiddle.net/sgcer/1884/ (scroll the right column, the left column will follow)

Hi @paul-gaspar,
Right, I forgot to remove the dependency after removing the JS part.
You can retry the Codepen now, it is free of Javascript.
It does have some CSS3 variables which are not supported on IE, but you could probably get around it with a fixed height. But again, not easy to implement on Vue Cal.
Let me know how your test goes, that will also help @giulianimartini!

I am closing this as there seems to be no solution without JS.
If anyone finds a way, I am happy to add some CSS in Vue Cal's codebase!

Thanks @antoniandre!
As soon as I have some time, I'll try to check that out.

have anybody here found any solution?

Hi Leonardo,
Unfortunately not.
I'm out of time at the moment, but as soon as I get back to programming in
my project, I'll try to do something about it.
It is on my to-do list.
Any news, please let me know.
Best Regards.
Em qua., 7 de jul. de 2021 às 02:00, Leonardo @.***>
escreveu:

have anybody here found any solution?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/antoniandre/vue-cal/issues/286#issuecomment-875280334,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC674IRH2IYKF3CMHW3XB2LTWPNNZANCNFSM4NIT7IVA
.

--
Ricardo G. Martini, PhD.
ALGORITMI Research Centre
@.***
Lattes CV: http://lattes.cnpq.br/8710772034085679


[Task] explain the different ways of using templating in the docs

In principle, both should lead to the same result, however, that's clearly not the case here.

[ ] bug: #390 The file should not be renamed by bt

Choosing one or the other command depends on your use case. U understood correctly how they are intended to work. With that said, rt needed to be implemented per type and we never got around to that - means there are types for which it wasn't implemented.
Bt on the other hand relies on you executing retrieve first and then works for all types you downloaded. It's more generic because it didn't need to handle downloading a single file.
If you want to use templating then you have essentially 3 ways:

Use the commands rt, bd(b), d
Use the commands r, bt, bd(b), d
Use r, then manually copy & adapt files from retrieve/ to template/, then run bd(b), d

Option 4 is also available and relies on you using a git repo and the command createDeltaPkg & deploy together. That's mostly for CI/CD enabled projects but I have seen teams use it manually as well.
Originally posted by @JoernBerkefeld in https://github.com/Accenture/sfmc-devtools/discussions/385#discussioncomment-3214911

done in https://github.com/Accenture/sfmc-devtools/wiki/6.-Commands#templating and


BUG: Failed wheel upload for h5py

After most of the h5py binaries uploading successfully last night the 3.11 one failed:
https://github.com/h5py/h5py/actions/runs/8445531348/job/23133109268#step:5:408
Warning:  Distribution "h5py-3.10.0-cp311-cp311-manylinux_2_17_x86_64.manylinux2014_x86_64.whl" already exists. Removing.
Creating package "h5py"
Creating release "3.10.0"
Uploading file "scientific-python-nightly-wheels/h5py/3.10.0/h5py-3.10.0-cp311-cp311-manylinux_2_17_x86_64.manylinux2014_x86_64.whl"

  0%|          | 0.00/5.17M [00:00<?, ?B/s]
  0%|          | 16.0k/5.17M [00:00<01:04, 83.8kB/s]
  2%|▏         | 128k/5.17M [00:00<00:10, 507kB/s]
  9%|▉         | 496k/5.17M [00:00<00:03, 1.63MB/s]
 22%|██▏       | 1.14M/5.17M [00:00<00:01, 3.26MB/s]
 61%|██████▏   | 3.17M/5.17M [00:00<00:00, 8.93MB/s]
5.18MB [00:00, 6.13MB/s]
standard python located at:
  https://anaconda.org/scientific-python-nightly-wheels/h5py

Error:  ("Invalid dist_id ObjectId('6603800020b28178c3fabc66') does not exist", [404](https://github.com/h5py/h5py/actions/runs/8445531348/job/23133109268#step:5:405))

Any ideas on what the problem could? Could the upload be made more robust somehow? It removed the wheel built the previous night prior to trying to upload the new wheel, so I ended up with a failed --pre install on a Linux 3.11 CI.

@larsoner Looks like you didn't try to rerun the workflow. Please do that (by which I mean just select the rerun button from the GitHub UI).

No permissions to restart myself so I'll probably just wait 24h for the next nightly :)
Closing since it does seem to be an upstream bug!

I opened up https://github.com/Anaconda-Platform/anaconda-client/issues/712 for this.

Yeah I think @tacaswell or @takluyver should be able to restart this job (copied from above for convenience) to fix the incomplete nightly wheel upload:
https://github.com/h5py/h5py/actions/runs/8445531348/job/23133109268#step:5:408

I've hit restart, it seems to have succeeded now. :+1:


Explicit requirments

will enforce installation of requests[security] by requiring requests and it's extra requires pyOpenSSL and cryptography.

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] ~Reset the build number to 0 (if the version changed)~
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Hi @conda-forge/staged-recipes, can this be pushed?


integrate https://github.com/athena-team/athena-decoder

start a new branch to integrate athena-decoder into athena project. @godjealous

Currently, we can consider the athena-decoder as a seperate project, and install it using pip.
In athena, we only use the provided interface in athena-decode.
Evaluate the decoder from two aspects: the speed, and the CER
In the future, we may update athena-decoder and athena simultaneously,
@cookingbear please help hanyang to support this, Thanks.

OK, I have design the interface for decoder and implement it for transformer model. please help me accomplish other models according to the interfaces @cookingbear

please finish this in two weeks @cookingbear @godjealous


unable to invoke the shareFile method

E/flutter (14539): MissingPluginException(No implementation found for method shareFile on channel channel:me.alfian.share/share)
Hi, this error appeared when I tried to share the image of QR code.May i know is it any additional plugins or code required? btw, thanks so much for writing the Medium article. Really appreciate it. Hope you will notice this issue.

I'm having the same issue in flutter.
some body coud get a simple solution?

Same error getting


add METATYPE_IMAGE (plus a few fixes)

See discussion under PR #7.

I will give it some thought on how to simplify the API to support images. I feel we are on the right track here. Ideally I would like the user to just pass in a QImage and not worry about setting the MetaType explicitly. Then we sould also reuse the existing QMetaType value:
QMetaType::QImage | 70 | QImage

Hope to find some free time this week to implement the changes, complie flags, etc.

Please consider this superceded by https://github.com/juangburgos/QUaServer/pull/13 , until we have something better.


Warn users on mobile and unsupported browsers/systems about unsupported features and any other issues

Story
As a user visiting Vega dapps on a mobile device or from an unsupported browser or platform
I would like to be informed about any limitations and how to avoid them (e.g. by moving to desktop, using different browsers/tools, downloading an app, whatever…) when I first arrive at a dapp that either doesn't work at all, is unsupported, or requires a Vega wallet or other software/features that aren't available on mobile
So that I can decide whether or not to continue my journey on the mobile device and avoid wasting time
Acceptance Criteria

[ ] I am informed of limitations on each dapp that users a Vega wallet or isn't supported by my configuration immediately or soon after visited
[ ] I am informed if the dapp is untested on mobile or the UI is not designed for small screens and likely to be difficult or impossible to use
[ ] The information includes or links to accurate and easy to follow instructions to work around the limitations and/or about other ways to access the dapp that aren't limited
[ ] The information include details of things that are still possible on mobile

Tasks

[ ] Explore and sketch
[ ] Team and stakeholder review
[ ] Visual Design
[ ] Team review
[ ] Etc.

Sketch
Additional details / background info

Based on the browser list:
last 1 Chrome version
last 1 Firefox version
last 2 Edge major versions
last 2 Safari major version
last 2 iOS major versions
Firefox ESR
not IE 9-11 # For IE 9-11 support, remove 'not'.

This should be included as part of the mobile review


Any plan on supporting mobile devices?

Tried to use it on my phone but seems not to work.

Yes Jorge, it’s on my roadmap.


Consumer param fetch_message_max_bytes should be a soft limit

We hit a production issue with a stuck pykafka-based consumer that I tracked down to a single message that was larger than the consumer's fetch_message_max_bytes, but smaller than the broker's message.max.bytes (meaning the broker didn't reject the message when it was originally produced).
In the Java reference implementation, the fetch.max.bytes param is a soft limit on both the broker and the new consumer: https://kafka.apache.org/documentation/#newconsumerconfigs
It needs to be a soft limit to avoid exactly this scenario of a single message that is smaller than the broker's message.max.bytes but larger than the consumer's fetch.max.bytes stopping the consumer.
So the broker returned the message, but then when pykafka tried to process the message it blew up here: https://github.com/Parsely/pykafka/blame/c80f66c0d0b11d830aa333ed486967b5f242bc2f/pykafka/protocol.py#L383
I think pykafka should mimic the upstream behavior of treating this as a soft limit rather than a hard limit. Perhaps emit a warning, but still process the message.

One design consideration when treating this as a soft-limit is the memory pressure on the client. I didn't dive deep enough into the code to check if the decode buffer is fixed size or dynamic. It needs to be dynamic for this to be a soft-limit.

@jeffwidman Likewise, thanks for this -- assigning to @emmett9001.

Thanks @jeffwidman for the investigation, and for opening this as a separate ticket from #697.

Pykafka treats fetch_message_max_bytes as a hard limit specifically because the decode buffer is statically allocated by BrokerConnection on initialization. fetch_message_max_bytes is used as a safeguard against overflowing this buffer. It's a bug that this implementation leads to the problem mentioned by @jeffwidman above in which a message from the broker is larger than fetch_message_max_bytes but smaller than the broker's message.max.bytes.
If my understanding is correct, this problem only happens when fetch_message_max_bytes < message.max.bytes. If this condition doesn't hold, the bug will never appear. Thus, the workaround in current master is to ensure that fetch_message_max_bytes is always at least message.max_bytes.
There may have been a pull request at one point to dynamically allocate the decode buffer, but I can't find it. I remember we ran into some issues last time we tried to make that change.

Thus, the workaround in current master is to ensure that fetch_message_max_bytes is always at least message.max_bytes

Yes, I implemented that as a temporary workaround. But that isn't ideal in the long term in certain scenarios, such as worrying about memory usage on the client.

Update: I noticed that this behavior changed upstream as part of KIP-74 / KAFKA-2063, so likely the original implementation was actually the expected behavior at the time in the Java world.

@jeffwidman It's hard for me to tell from the breadcrumb trail of KIPs - does pykafka's current behavior now match the reference implementation?

Afraid not. The original issue description is still correct, my addendum comment was just that at the time of implementation pykafka matched the expected behavior but no longer does. It should now be a soft limit.


Maintenance of open_data module

Module to maintain
Website: https://school-brainhack.github.io/modules/open_data/
Code: https://github.com/school-brainhack/school-brainhack.github.io/tree/main/content/en/modules/open_data
Work to be done
Please ensure:

[ ] links, resources and exercises reflect the state-of-the-art
[ ] videos are of good quality and reflect the materials.

If any update / re-write of the module is needed, feel free to suggest alternative material in a comment!
If you think major changes are needed, we suggest you open an issue for new modules.

Links, resources, and exercises:

In the Jupyter Notebook, two of the chunks threw errors when I tried to run them (at least, in the binder). I've fixed both, but not sure where I should I make the pull request - the original repo (https://github.com/surchs/bhs_nilearn_example)?

Video:
Thought it was clear! Good quality and reflected the materials.
Other:

The github repo link at the top of the module page just links to the module page again.
Minor, but there's no image in the github repo, so the module page shows a broken image icon.

This is great, thank you! For the PR, yes please make it for the original repo and hopefully Sebastian will merge it.
I'll take a look at the link and upload an image.

Plan B @ryancyeung: I have forked Sebastian's repo here, please can you make the PR to that repo? Apologies for this! Once it's merged I can then add a new binder link, at the moment it fails to launch, possibly due to the errors you've caught. If you also wanted to upload it to binder please go ahead! Thank you!

Sounds good! I made the PR here, let me know if anything looks off. I'm also happy to upload to Binder and share the link after it's merged.


Sample code correction in migrate-to-policy.md

Corrected code sample so that the correct combinations of source address, source IP Group,Destination Address, Destination IP Group are applied in the if statements.

@isjwuk : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

#sign-off


Feature?/Bug: Doesn't work inside SalesForce HTML text box

I'm able to use this extension to paste an email template into Salesforce email box but only the text-only version. If I switch to HTML I can no longer get the templates to paste. My organization requires us to use an HTML template that adds the letterhead and contact information to the email, with a text box in the middle for typing our emails-- which is why I'd really appreciate if we could get it working in the Salesforce HTML box!
Video showing text-only box usage vs. HTML box usage: https://imgur.com/a/y60zi6f

Thanks for posting the bug! Ah hmm, interesting. I don't have access to
Salesforce, so I might need to ask you for some additional details. I'll
try to look at this this weekend.

Thanks for posting the bug! Ah hmm, interesting. I don't have access to Salesforce, so I might need to ask you for some additional details. I'll try to look at this this weekend.

You are only replying to what you want to it seems? The last update you put out made it completely useless for me and others but I guess that's not important. It's lucky I know how to roll back and use other versions or I'd be pretty damn annoyed.

Hey Klyde, I don't think there's been an update put out for a couple years,
so I'm not sure what you're talking about. Sorry that things haven't been
fixed for you yet, to be honest, this was just a fun side project for me
and I get paid nothing and I have another full time job, so I can't really
promise anything in terms of timeline for fixes. I also don't have time to
check the GitHub issue reports so I don't get to all items that are posted,
but I'll try to take a look at your issue when I'm able to, thanks.
.C
On Tue, Nov 5, 2019, 11:44 klyde<EMAIL_ADDRESS>wrote:

Thanks for posting the bug! Ah hmm, interesting. I don't have access to
Salesforce, so I might need to ask you for some additional details. I'll
try to look at this this weekend.
You are only replying to what you want to it seems? The last update you
put out made it completely useless for me and others but I guess that's not
important. It's lucky I know how to roll back and use other versions or I'd
be pretty damn annoyed.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/carlinyuen/ChromeAutoTextExpander/issues/60?email_source=notifications&email_token=AADHHXKIUZS7X43X4L2G56DQSHEIDA5CNFSM4JEHD7P2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDECLPA#issuecomment-549987772,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADHHXPUB22Q257QSD7BFATQSHEIDANCNFSM4JEHD7PQ
.


Bugfix/home page typo

 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.:x: vanierreddy sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Delegate job printing to the DriverExecutor since it involves argument resolution

This gets rid of the last two ArgsResolvers outside of the executor

@swift-ci test

@swift-ci test


screenshots

in action

Raspberry - PI - GPIO Driver. This one will:

Add/New Block/***
Add options to Command
Parse commands driver side

Ribbon Bar re-implemented


Recent pip install -r  reqs_optional/requirements_optional_gpt4all.txt  fails on Mac with apple silicon

Ignoring llama-cpp-python: markers 'platform_system != "Linux" and platform_machine != "x86_64" and platform_system != "Windows" and platform_system != "Darwin"' don't match your environment
ERROR: llama_cpp_python_cuda-0.2.23+cu118-cp310-cp310-manylinux_2_31_x86_64.whl is not a supported wheel on this platform.

I fixed the particular problem.  However, for Apple Silicon you will have to uncomment the last line to get that.  I don't know of a trigger in requirements to choose that.

Actually maybe that is fixed now.  I was basing off what oobabooga did, but maybe it'll work now.


windows_x64_vs2013: h2_full_nosec_test.exe disappearing_server

https://grpc-testing.appspot.com/job/gRPC_portability_master/1015/language=c,scenario=windows_x64_vs2013/testReport/junit/(root)/tests/vsprojects_x64_Debug_h2_full_nosec_test_exe_disappearing_server/
D0320 04:04:54.491000000  2880 test_config.c:276] test slowdown: machine=1.000000 build=1.000000 total=1.000000
I0320 04:04:54.497000000  2880 disappearing_server.c:194] disappearing_server_test/chttp2/fullstack
D0320 04:04:54.509000000  2880 chttp2_transport.c:698] got goaway [0]: 53 65 72 76 65 72 20 73 68 75 74 64 6f 77 6e 'Server shutdown'
E0320 04:04:54.509000000  2880 tcp_windows.c:152] ReadFile overlapped error: An established connection was aborted by the software in your host machine.

E0320 04:04:54.509000000  2880 disappearing_server.c:174] assertion failed: status == GRPC_STATUS_UNIMPLEMENTED

One more occurrence: https://grpc-testing.appspot.com/job/gRPC_master/16053/


Finish an MVP of the Meta page and get it public

Is your feature request related to a problem? Please describe.
The BHL does not have a centralized, multi-project page on Wiki. It needs one.
Describe the solution you'd like
The Wikimedian-in-Residence should create this page as part of their work.
Additional context
Add any other context or screenshots about the feature request here.

@JJDear83 Hi! I don't think I have the permission to add Issues to the BHL-WIKI project. Should I be able to do so?

HI @lubianat I think you have access. You are on the collaborators list for the project. hmmm. What is not working on your end?

Also, maybe we should set up a weekly 1:1. Sorry we haven't had enough time to connect, but in the New Year I intend to be more focussed on Wiki. Do you want to meet-up soon? use this link to find a time for us to connect?


Rename ApplicationData in Registration ?

In Registration we have a way to attach "custom" data.
The idea :
When a client tries to register,  a Leshan server could need to call external services to know if client is authorized to register.
This is done in Authorizer. During this authorization step, you could get some data (collected from that services) that you want to attach to registration (kind of optimization to avoid to recall that services later).
(See :https://github.com/eclipse/leshan/pull/1018 and  https://github.com/eclipse/leshan/issues/1293 and for more details)
Currently this looks like :
public class Registration {
     // ...
    /**
     * @return Some application data which could have been added at Registration by the {@link Authorizer}
     */
    public Map<String, String> getApplicationData() {
        return applicationData;
    }
    // ...
}

/**
 * This class is responsible to authorize up-link request from LWM2M client.
 */
public interface Authorizer {

    /**
     * Return an approved {@link Authorization}, if this request should be handled by the LWM2M Server. when a
     * declined{@link Authorization} is returned the LWM2M server will stop to handle this request and will respond with
     * a {@link ResponseCode#FORBIDDEN} or {@link ResponseCode#BAD_REQUEST}.
     *

     * Some Application Data could be attached to on approved {@link Authorization} using :
     *
     *

     * return Authorization.approved(myAppData);
     *

     *
     * @param request the request received
     * @param registration the registration linked to the received request.

     *        For register request this is the registration which will be created

     *        For update request this is the registration before the update was done.
     * @param senderIdentity the {@link Identity} used to send the request.
     *
     * @return an {@link Authorization} status.
     */
    Authorization isAuthorized(UplinkRequest<?> request, Registration registration, Identity senderIdentity);
}

The issue:
ApplicationData is maybe not clear enough, see : https://github.com/eclipse/leshan/issues/1237#issuecomment-1514178935
Proposed alternatives :

UserData
UserDefinedData
MetaData

What should we do ?
Do you think we should change that name ? or is ApplicationData clear enough ?
If you want to change it let us know which name you prefer ?
(If you want to add more alternatives name let us know too)
Note that this will break the API and maybe the redis serialization but if we think changing the name will make the code clearer, we should not be stopped by this.

My current opinion, I don't like MetaData because I feel this is not really registration metadata.
UserData / UserDefinedData is  OK to me.
I don't know what is the name generally used for this kind of data ? If you have some example do not hesitate to share.
@jvermillard, @msangoi, @JaroslawLegierski , @Warmek  any opinions ?

The name "UserDefinedData" sounds good.


Change how globals are passed in GraphQL queries.

Instead of using a separate globals field, use variables to add a __globals__ object containing the global variables.
Fixes #5863

I've added tests explicitly trying the scenario where new and deprecated globals don't match, but one of them is an empty dict to make sure that it's still an error.

Let's merge this for 4.x


Tests fail with Convex.jl backend

By any chance can you take a look at the convex.jl implementations and update them to the latest Convex.jl release? It should be a one line change.
I am having trouble testing locally because of some artifact download errors, but we need to address this issue soon and release a new version. If you can take a look, that would be great.

It is unfortunate that Requires.jl doesn't allow compat versions. We will be constantly falling into this issue whenever an optimization backend has breaking changes.

Hopefully this will be supported by future releases of Pkg.jl: https://github.com/JuliaLang/Pkg.jl/issues/1285

Fixed on master.

Thanks for the fix!


Identity 2.0

Hi,
I created a fork of your project and I implemented everything for Identity 2.0(based on ef implemenation). If you're interested I can create a pull request. The only thing is I still use the string Id of V1.
Thanks for the initial project.
Laurent

Are there any plan to make a NuGet package out of this branch, or to fork the Identity 2.0 branch? I realize the above comment says it's simple to do, but I would prefer to go through NuGet if at all possible - if not I will just do as Inu suggested.

I understand but it's not my work in the first place. Probably the easiest would be to merge the changes into the original repo but there is no response from the owner.

@lnu just submit a pull request. /cc @milesibastos

The pull request has been merged, the version updated to match Asp.Net Identity release and the new version pushed to nuget.
Thanks @milesibastos

thks to u for a great job!


Toggle Theme

This is in response to the issue #52 .
https://github.com/user-attachments/assets/13e76a8c-11e9-4f8a-9b4d-4dee7c6b7845

@TejasNasre

@inkerton the issue is ui/ux enhancement so make the whole website change the color light and dark  after that in light theme add whitish creamy color not the whole white corresponding change the whole website its level 3 issue

https://github.com/user-attachments/assets/5ab034e4-fa97-4989-a8e0-7b2cdc813387
The state is being saved and all pages are changing theme if there is any page which is not changing theme then let me know I will see for any issue.

you are updating other changes ?

you are updating other changes ?

Konse other changes bhai toggle button and start vala tha daal diya vo maine and koi issue ya khin problem h toh do let me know . Mujhe genuinely smjh nhi aaya what other changes are you talking about.
Global state is being saved . Theme is applied to the whole website is alawa kya changes the?

should show the star count here the ui not good

were are the borders and the event card should text white after open details page the borders are not there

border are not there

@inkerton i think you are ui/ux enthusiast so rewrap the whole components 😀add your creativity

Yeah I did not add any env variables so I could not check these pages. By any chance can you provide me with the env variable values?
@TejasNasre

Make upadtes asap with all changes its level 3 issue you don't need to add any Functionality and work with backend things just adding you do it fast 😄

Make update asap with all changes its level 3 issue you don't need to add any Functionality and work with backend things just adding you do it fast 😄

Working on it will do it in an hour

https://github.com/user-attachments/assets/151e4a79-639a-42fb-ba57-dbe8c14b22ea
@TejasNasre Done all completed

i am merging it but with level2. Thanks for your contribution

please write code that covers all the edge cases here the ui breaks at mobile device

Can You make It a another issue I will adjust the Button . You can make It a level 1 issue . I'll do it right away.

Can You make It a another issue I will adjust the Button . You can make It a level 1 issue . I'll do it right away.

please update here

or don't update now whole ui thing will fix later need to add functionality first

k let me know when you need it


Additional data from a session

The explicit table of most of my datasets from pimanager studies have a few participants with multiple (usually two) rows of the same question. Sometimes, with different responses. As if the participant was able to respond to the same question twice. Further, it also happens with data saved to the explicit table using API.save. So, it is not the user who responded twice, but the task that somehow saved twice, and sometimes with a different value.
I will follow this message with a few examples from my data, and @andrewdzik will follow up with a similar issue (we suspect it is the same) that was found in James Rae's IAT data.
To clarify: this is rare. About once in 200-400 sessions.

Today I started looking at the data of a new study that was launched last Thursday. So far, the data include about 210 participants.
In the IAT data file (from the IAT table), there was one session with more trials than expected (392 instead of 196). The data is not duplicated.
That session also had duplicate explicit data (but not the same values, as is clear from the latency values). There was another session with duplicate explicit data.
The session data of those two sessions don't look unusual.
I'm attaching the three data files (IAT, explicit, sessions):
dups.iatatt1.zip
User agents of these two sessions is identical:

Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/52.0.2743.116 Safari/537.36
Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/52.0.2743.116 Safari/537.36

The study is here:
https://app-prod-03.implicit.harvard.edu/implicit/showfiles.jsp?user=yba&study=iatatt1

Lets try to get all our clues together here:
My take on why it is not probable the problem is client side
We know that sometimes people that have a double record are randomised into different starting conditions. These conditions are sometimes set in JS code that is not part of the player which suggest that the problem is not with the player per-se.
This still leaves an option that the module can be loaded twice by the same page (though this would be a violation of the AMD standard - I wouldn't know how to do it if I wanted to...).
Is there any chance that we get duplicate records when users refresh the page?
I gather that duplicate records always require the completion of the first round (@yba?). So that leaves only refreshing on the debriefing page. Also, I don't see why a user would go on to complete an additional questionnaire if they already tried to escape it by refreshing...
Where else can we look
So far most of the problems that I've seen happen around midnight for some reason, I would like to check up that option, though there are some instances where the problem happens at other times.

By "first round" do you mean a whole session?  It will be a good clue if it happens only to participants who completed a session. @andrewdzik might have that information. If not, I can look at some recent data.

I looked at data from the study yba.ybalab.gss2.0001 from the last 7 days.
These session ids have duplicates in the explicit table:
9406036 9406926 9406942 9406945 9407230 9408147 9408277 9408394 9408552 9411660 9412103 9413258 9417195 9417603
Some background: This is study has dozens of between-participant conditions (i.e., participants are assigned to very different conditions and starting the study again is not likely to look much similar to the participants).  Because of that, the duplicates are in the explicit data saved from the manager.
It is saved with this code:
		realstart : [{
			inherit:'instructions', name:'realstart', templateUrl: 'intro.jst', title:'Introduction',
			piTemplate:true, header:'Welcome',
			load: function()
			{
            	API.save({
            		domain:global.cat1Label, imageSet:imageSet,
            		EPITICond:EPITICond, whichPrime : whichPrime, measure:measure
            	});
			}
		}],

The files are zipped here:
dups.gss2.zip
Here are the stories of a few of the sessions, as much as I could understand from those files:
9406036: Dropped out onthe 4th page (the task 'ep'). Did not complete any questionnaire.
9407230, 9408147, 9406926, 9408394, 9408552, 9412103, 9417195, 9417603: Completed the study. Has data only from one questionnaire.
9406942: Dropped out on the 4th page (the 5th task_id: 'ep'). Did not complete any questionnaire. The 4th task_id is instructions for a questionnaire, but the 5th task is a different task (the task 'ep'). So, perhaps the session re-started after the 4th task_id, could not save the task_id after the restart, until it reached the 5th task_id (not recorded previously). Then, the particiant decided to drop out.
9408277: Dropped out on the 4th page (the task_id 'amp'). Did not complete any questionnaire. However, the 3th page was instructions to one task (task_id 'instep') but the 4th page was actually a different task (the task_id 'amp'). So, perhaps restarted the study after 'instep' and got all the way to 'amp' until dropped out.
9406945: Dropped out on the 5th page (6th task_id). Completed the questionnaire once.
9411660: Completed the study. Importantly: this participant completed the questionnaire twice. This is important because it means that the problem is not some kind of restart of the manager. It is an actual restart of the study. That participant also completed two reaction-time tasks (EP and IAT). So, that participants seems like they completed the study twice (can't know for sure that they reached the debriefing twice).

I've found the cause of the bug.  From the debriefing page, if you leave the study and then use the back button, you will restart the study.  The browser loads the initial get request for the study entirely from the cache, meaning that the server logic that prevents refreshing the page is never reached since the server is never hit.
I'll work on updating the server code to keep the request from being cached.

But, what about all the sessions with duplicate data without any indication they reached the debriefing?

You can leave the study in the middle, press OK, and then use the back
button to restart it also.
On Tue, Nov 8, 2016 at 4:11 AM, baranan<EMAIL_ADDRESS>wrote:

But, what about all the sessions with duplicate data without any
indication they reached the debriefing?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ProjectImplicit/PIquest/issues/140#issuecomment-259083793,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AED7_MxsqeJbvicApsP_5u9pM4v3S3Ayks5q8DzDgaJpZM4J-AXT
.

Ah, nice. So, the fix you're proposing is that clicking 'back' will assign the participants to a new study instead of restarting the same study?

It will send them to the timeout page.  Unless there is a different
behavior that you want.
On Tue, Nov 8, 2016 at 8:03 AM, baranan<EMAIL_ADDRESS>wrote:

Assigned #140 https://github.com/ProjectImplicit/PIquest/issues/140 to
@andrewdzik https://github.com/andrewdzik.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ProjectImplicit/PIquest/issues/140#event-851448054,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AED7_DMZzHnlO7SMD5v22ajDuvpGFMynks5q8HMHgaJpZM4J-AXT
.

In general, when people click Back, an error page is not a great user experience. However, I am not sure that assigning them to a new study is also great because they probably want to return to the same study, which we can't allow them. The error page provides a link to the homepage, and that's probably fine.

Maybe we should replace the error page for timeout/back with an information
page.
It should include an explanation of why the user got there and some common
links back to where they would like to go. (and absolutely no mention of
errors or red text :P).
On Tue, Nov 8, 2016 at 4:05 PM, baranan<EMAIL_ADDRESS>wrote:

In general, when people click Back, an error page is not a great user
experience. However, I am not sure that assigning them to a new study is
also great because they probably want to return to the same study, which we
can't allow them. The error page provides a link to the homepage, and
that's probably fine.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/ProjectImplicit/PIquest/issues/140#issuecomment-259144403,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AByZF-1xdInaflNIK0N4vk1x3Iby0xUKks5q8IGQgaJpZM4J-AXT
.

Sounds good. @andrewdzik , would it be difficult to implement and deploy with your fix to the bug?

@andrewdzik can we close this?

Yes, I have the fix working for it.
On Thu, Nov 10, 2016 at 5:44 AM, eladzlot<EMAIL_ADDRESS>wrote:

@andrewdzik https://github.com/andrewdzik can we close this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ProjectImplicit/PIquest/issues/140#issuecomment-259658559,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AED7_Pb3W7St9AYFE5BkT_5cseKnwpoQks5q8vWAgaJpZM4J-AXT
.


Building DependencyCheck from source requires Administrator privileges on Windows

When building DependencyCheck from source on Windows for the first time (with a clean git workspace) the build fails on mvn install if you aren't running as Administrator.
Error message:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-resources-plugin:3.0.2:testResources (default-testResources) on project dependency-check-core: D:\Git\DependencyCheck\core\target\test-classes\nodejs\node_modules\dns-sync\node_modules\.bin\shjs: A required privilege is not held by the client. -> [Help 1]

This is coming from the dns-sync NPM package. If you then open up a Powershell window as Administrator and run mvn install again, it succeeds.
Reproduction steps:

From Powershell as a non-administrator, run git clean -dxf to remove all files and folders including those that are ignored by .gitignore from your workspace.
Run mvn install. Build fails.
Open a new Powershell window as an administrator in the repo root.
Run mvn install. Build succeeds.

For obvious reasons it would be ideal if building DependencyCheck from source did not require Administrator privileges 😄
Apache Maven 3.3.9
Java 1.8.0_144
NPM 5.6.0
Node 8.9.0
Windows 10.0.16299

Interesting - thanks for the report. I do not have a current windows 10 machine to debug this atm. It might be related to the blocked property (see Unblock-File).  But in reality, the file in question is never actually used/parsed as part of the test cases. We can safely remove the shjs files.

As I have a Windows 10 system at hand I just had a quick try:

open PowerShell (as a standard user)
get a fresh git clone https://github.com/jeremylong/DependencyCheck.git
run mvn install

I cannot reproduce the issue, the build completed successfully.


Request for distribution

Greetings,
I am the developer of ArkOS (https://github.com/christianhaitian/arkos/wiki).  An Operating System distribution designed to work on specific RK3326 and RK3566 gaming handhelds.  The primary emulator backend for most systems in the distribution is Retroarch.  I recently received a request from a community member to have this shader included in this distribution.  Is it ok for me to include this in the distribution?
Thanks,
Christian_Haitian

Hello, Christian_Haitian.
It's a delight to have my work distributed, especially in such a great project as yours. You are more than welcome to include the shaders in ArkOS, and thank you for your continued contribution to the retro handheld scene!
Thanks,
Woohyun Kang-----Original Message-----From: christianhaitian @.>To: Woohyun-Kang/Sharp-Shimmerless-Shader @.>Cc: Subscribed @.***>Sent: 2024-07-11T22:57:27+09:00Subject: [Woohyun-Kang/Sharp-Shimmerless-Shader] Request for distribution (Issue #1)
Greetings,
I am the developer of ArkOS (https://github.com/christianhaitian/arkos/wiki https://github.com/christianhaitian/arkos/wiki).  An Operating System distribution designed to work on specific RK3326 and RK3566 gaming handhelds.  The primary emulator backend for most systems in the distribution is Retroarch.  I recently received a request from a community member to have this shader included in this distribution.  Is it ok for me to include this in the distribution?
Thanks,
Christian_Haitian
—Reply to this email directly, view it on GitHub https://github.com/Woohyun-Kang/Sharp-Shimmerless-Shader/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATYKYSNXQIEUB6PKCMVSKYLZL2FMPAVCNFSM6AAAAABKXBPYC2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGQYDGMRWGIYDMNY.You are receiving this because you are subscribed to this thread.Message ID: @.***>

Thank you for the permission.  Take care and be blessed!


DCOS-60342, DCOS-60344: Add refresh rate configuration

Testing
Go to ui Settings,
Change the setting to something different then 2 seconds  ( bigger is better )
Reload the page.
Open network tab.
Open services page.
Check in the network tab that the endpoints are only requested at about the changed refresh rate setting.
You can also check this on various pages in the app.
Trade-offs

You have to hard reload the page to use the new refresh rate on streams

Dependencies
n/a
Screenshots
n/a
CLOSES DCOS-60342
CLOSES DCOS-60344

It looks like this to me:

Investigating ... Thanks and sorry for not testing the i18n part ...

@Poltergeist It should take effect right after selecting the new value. If we need to refresh the page to achieve it please refresh the page forcibly.
Right now I'm selecting the new value and it keeps calling the API every two seconds even when I navigate to other pages. Only reloading the browser helps

the services-tab seems to have a minimum wait time of the chosen value.

It looks like we have a race condition which makes for a waiting time of a little over the chosen value in 80% of my service tree visits.

i added an ugly fix that rerenders the whole ServicesContainer in case it is in loading-state and DCOSStore has new data. this is necessary as the serviceTree is not passed into our component but retrieved during render...

:tada: This PR is included in version 2.152.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Added configuration for XFCE4 Terminal

I've added a configuration file for the XFCE4 Terminal that uses the ('dark') solarized colour pallette.

@altercation - I've created a new standalone repos for this at https://github.com/sgerrand/xfce4-terminal-colors-solarized
Let me know how you'd like to proceed.


CCHub showcase event

CCHub is having an event this morning, by 11AM. Kindly update the readme with the link below
RSVP LINK

@Jemeni11 lemme know if you'd love to take this up

stale

CCHub is having an event this morning, by 11AM. Kindly update the readme with the link below
RSVP LINK

@Jemeni11 lemme know if you'd love to take this up

Me just seeing this now:

It was too sudden. I get 😃
You can try fixing the other issues

Okay :+1:

It was too sudden. I get smiley
You can try fixing the other issues


Unblocking files

Preflight Checklist

[x] I have installed the latest version of Storage Explorer.
[x] I have checked existing resources, including the troubleshooting guide and the release notes.
[x] I have searched for similar issues.

Problem
I am unable to remove the blocked flag or give a file a Zone.Identifier value other than 3 for files downloaded directly to an Azure file server
Desired Solution
It would be great to be able to remove the blocked flag.
Alternatives and Workarounds
No response
Additional Context
No response

@joel74 we're not quite sure what you are talking about/referring to. Can you provide some screenshots which show us the problem you are having in Azure Storage Explorer?

Windows often blocks downloaded files because it considers them unsafe. When one tries to open one of these blocked files, you get a security alert:

If the file was copied to an Azure file server before it was unblocked, the option to unblock the file in File Explorer is lost. (Also, unblocking via PowerShell command doesn't appear to work either.)

It would be nice if Azure File Explorer recognized when a file was blocked and provided an option to unblock it.


arm_arch_timer: Too many clock announcements with CONFIG_TICKLESS_KERNEL=n on SMP

https://github.com/zephyrproject-rtos/zephyr/blob/95cddd4df7e63375f27d6308dd9b31ffd4d8ab97/drivers/timer/arm_arch_timer.c#L46
If CONFIG_TICKLESS_KERNEL is not set this always announces a tick. With SMP only one core is expected to announce a tick even if all cores receive the IRQ, therefore this announces too many ticks. For example, this causes applications which use k_msleep to be woken up too early.
The code works correctly in a tickless kernel because last_cycle is shared and the cores that execute the critical section subsequently calculate delta_tick=0.
Unrelated: I believe last_cycle does not need to be volatile because it is protected by the lock.

out of curiosity, why do you need to have a non-tickless kernel?

@galak I think we can merge #33909 to close this

@galak I think we can merge #33909 to close this

Looks like PR still needs to address or come to resolution with @andyross comments.


Documentation

Thanks for the awesome library!
This project would be more awesome if the documentation was better. A link to a demo, installation instructions and a bower package would make a big difference!
Thanks

A quick doc is here but the rest is yet to be done -
http://www.codeproject.com/Articles/576246/A-Shopping-Cart-Application-Built-with-AngularJS?msg=4573566#xx4573566xx
Demo
http://demo.componentone.com/Wijmo/Angular/ShoppingCart/ShoppingCart/#/store
regards


python3Packages.adafruit-platformdetect: 3.18.0 -> 3.19.1

Motivation for this change
https://github.com/adafruit/Adafruit_Python_PlatformDetect/releases/tag/3.19.1
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

2 packages built:

python38Packages.adafruit-platformdetect
python39Packages.adafruit-platformdetect


Topology managed health checks

The health checks are currently built out using MHC resources manually, this should be modified to be using the topology so they can be managed by it.

this is done.


temporary downgrade for spl-token dependency

Hi there, did you mean to put in this PR? I don't understand the purpose. It also seems to use much older dependencies.


2.6.1

https://github.com/AeonLucid/POGOProtos/commit/942e841d9ceda5844354a3d4616334ba65bcf54a

2.7.0 incoming

Closed in favor of https://github.com/Necrobot-Private/POGOProtos/pull/23


upgrade: Improve error handling for database step

For connect option, the status was not handled properly in case
of database connection problems. In addition the formats were adjusted
to match other places.

 (3) UI: Generic error handling (connect to DB)

 Save upgrade errors in consistent format (bsc#1019298)


MultiStickWithPiCamera example using custom mobile net ssd model results error

Device:  Raspberry Pi 3 B+
CPU Arch.:  armv7l
OS:  Raspbian
I have trained a custom 1 class mobile net ssd v2 network using Tensorflow Object Detection API and successfully converted to to IR model using OpenVino 2019 R1.1. Then I substituted the model files in the MultiStickWithPiCamera.py example with my custom model. The predict_async thread raises an error.
Error message:
line 174, in predict_async
cnt, dev = heapq.heappop(self.heap_request)
IndexError: index out of range

Update:
I have resolved the problem. The reason is that, I am using a tensorflow implementation of Mobilenet SSDv2. Therefore after the conversion with the openvino model optimizer, the input layers and output layers are actually different from the model here.
In my case, I have changed the lines above in the MultiStickSSDwithUSBCamera_OpenVINO_NCS2.py
# Lines 147-154
def image_preprocessing(self, color_image):

    prepimg = cv2.resize(color_image, (300, 300))
    # Commented out the below lines to match the input format of my model. Edit yours accordingly.
    # prepimg = prepimg - 127.5
    # prepimg = prepimg * 0.007843
    prepimg = prepimg[np.newaxis, :, :, :]  # Batch size axis add
    prepimg = prepimg.transpose((0, 3, 1, 2))  # NHWC to NCHW
    return prepimg

and
# Line 179
# For my model, output dict returns with "DetectionOutput" key. You can find yours and replace accordingly.
out = self.exec_net.requests[dev].outputs["DetectionOutput"].flatten()


Does not work on front-end in Next.js projects

Even after adding fs via yarn add fs, I still get this error. ):
Screenshot:

Yes, this is a node module. fs and path are not available in the browser.
You can get around that fairly easily. vocab.bpe needs be saved as a json.
const bpe_file = fs.readFileSync(path.join(__dirname, './vocab.bpe'), 'utf-8');
fs.writeFileSync('./vocab_bpe.json', bpe_file)

Then the two jsons can be loaded directly.
//const fs = require('fs')
//const path = require('path');

//const encoder = JSON.parse(fs.readFileSync(path.join(__dirname, './encoder.json')));
//const bpe_file = fs.readFileSync(path.join(__dirname, './vocab.bpe'), 'utf-8');

const encoder = require('./encoder.json')
const bpe_file = require('./vocab_bpe.json')

Modifying the module itself didn't work for me. I get errors around process not being defined in node_module/path even through it's no longer using path and doesn't have any dependencies that do. Coping the code with the modification above into react with import { encode } from './gpt-3-encoder-react/Encoder' works perfectly fine. Tested on react 18.2.0.
Zip with both jsons and modified code.

If interested, a temporary fix would be to use https://www.npmjs.com/package/@nem035/gpt-3-encoder until the maintainers merge https://github.com/latitudegames/GPT-3-Encoder/pull/33

You can also use my fork gpt-tokenizer. See the playground here that works fully in the browser.

@syonfox - getting this in a frontend svelte project with the latest release - 1.1.4.
p.js:16 ReferenceError: __dirname is not defined
    at<EMAIL_ADDRESS>(Encoder.js:5:54)
    at __require (chunk-ENY474MH.js?v=4b935346:5:50)
    at<EMAIL_ADDRESS>(index.js:1:28)
    at __require (chunk-ENY474MH.js?v=4b935346:5:50)
    at index.js:6:1

is that not built in node if your in the browser environment you will need node_modules/gptoken/browser.js copy to relevant location in build
just include this as a regular file and use let c = gptoken.count(str)
also npm install gptoken


docs: Change labelGain description

The description of labelGain hyper-parameter was not correct and has been corrected. Actually, it looked like the one corresponding to alpha hyper-parameter.
No change in behaviour. Just a small change in documentation

/azp run


stream_from occasionally returns None

Example:
from steem.blockchain import Blockchain
from steem.steemd import Steemd

steemd = Steemd()
b = Blockchain(mode='head')
h = b.stream_from(
    start_block=14500000,
    full_blocks=True,
)
for block in h:
    if not block or 'block_id' not in block:
        raise Exception("invalid block: {}".format(block))

    num = int(block['block_id'][:8], base=16)
    print("Got block {} at {} with {} txs -- ".format(num,
        block['timestamp'], len(block['transactions'])))

Will sometimes fail like:
Got block 14577978 at 2017-08-14T21:41:36 with 19 txs
Got block 14577979 at 2017-08-14T21:41:39 with 17 txs
Got block 14577980 at 2017-08-14T21:41:42 with 15 txs
Traceback (most recent call last):
  File "hive/indexer/core.py", line 412, in <module>
    run()
  File "hive/indexer/core.py", line 400, in run
    listen_steemd()
  File "hive/indexer/core.py", line 354, in listen_steemd
    raise Exception("stream returned invalid block: {}".format(block))
Exception: stream returned invalid block: None

stream_from should never return None. If there is some issue it should throw an exception.
Output from the node indicates this case may have been due to double-production:
2502307ms th_a       application.cpp:507           handle_block         ] Got 15 transactions on block 14577980 by xeldal -- latency: 307 ms
2503812ms th_a       websocket_api.cpp:88          on_message           ] message: {"jsonrpc": "2.0", "id": 0, "params": ["database_api", "get_dynamic_global_properties", []]
2503813ms th_a       websocket_api.cpp:88          on_message           ] message: {"jsonrpc": "2.0", "id": 0, "params": ["database_api", "get_block", [14577980]], "method": "call"}
2505942ms th_a       application.cpp:507           handle_block         ] Got 20 transactions on block 14577981 by fubar-bdhr -- latency: 942 ms
2505995ms th_a       database.cpp:538              _maybe_warn_multiple ] Encountered block num collision at block 14577981 due to a fork, witnesses are: [["fubar-bdhr","2017-08-14T21:41:45"],["fubar-bdhr","2017-08-14T21:41:45"]]
2505996ms th_a       application.cpp:507           handle_block         ] Got 20 transactions on block 14577981 by fubar-bdhr -- latency: 996 ms
2506977ms th_a       websocket_api.cpp:88          on_message           ] message: {"jsonrpc": "2.0", "id": 0, "params": ["database_api", "get_dynamic_global_properties", []], "method": "call"}
2506978ms th_a       websocket_api.cpp:88          on_message           ] message: {"jsonrpc": "2.0", "id": 0, "params": ["database_api", "get_block", [14577981]], "method": "call"}

Fixed in https://github.com/Netherdrake/steem-python/commit/3f0a8b5e406a892609990ab5932ccce5a893eb1b


Missing t?

Probably PEBCAK but I copied the src and renamed it to Llama. Upon start I got :
18:16:42.032  [18:16:42] [Store] [INFO] Global store is ready  -  Server - Logger:15
18:16:42.033  t is not a valid member of ModuleScript "ReplicatedStorage.Libraries.Llama"  -  Server - copy:4
18:16:42.033  Stack Begin  -  Studio
18:16:42.034  Script 'ReplicatedStorage.Libraries.Llama.Dictionary.copy', Line 4  -  Studio - copy:4
18:16:42.034  Stack End  -  Studio
18:16:42.034  Requested module experienced an error while loading  -  Server - Dictionary:2
18:16:42.035  Stack Begin  -  Studio
18:16:42.035  Script 'ReplicatedStorage.Libraries.Llama.Dictionary', Line 2  -  Studio - Dictionary:2
18:16:42.035  Stack End  -  Studio
18:16:42.035  Requested module experienced an error while loading  -  Server - Llama:2
18:16:42.035  Stack Begin  -  Studio
18:16:42.036  Script 'ReplicatedStorage.Libraries.Llama', Line 2  -  Studio - Llama:2
18:16:42.036  Stack End  -  Studio
18:16:42.036  Requested module experienced an error while loading  -  Server - PlayerManager:16
18:16:42.036  Stack Begin  -  Studio
18:16:42.037  Script 'ServerStorage.ModuleScripts.PlayerManager', Line 16  -  Studio - PlayerManager:16
18:16:42.037  Stack End  -  Studio
I found the type t under modules but should I copy that as well?

Yea, Llama expects t to be under the Llama module. That's my bad though, I didn't put that in the guide! I will do that now.


Trouble installing java8

General troubleshooting steps

[n/a] I have retried my command with --force and the issue is still present.
[yep] I have checked the instructions for reporting bugs (or making requests) before opening the issue.
[???] None of the templates was appropriate for my issue, or I’m not sure.
[yep] I ran brew update-reset && brew update and retried my command.
[yep] I ran brew doctor, fixed as many issues as possible and retried my command.
[yep] I understand that if I ignore these instructions, my issue may be closed without review.

Description of issue
Space in java8 package name causing install to fail.
Output of your command with --verbose --debug
It is pretty clear without these flags, but I can rerun it with them.
==> Satisfying dependencies
==> Downloading http://download.oracle.com/otn-pub/java/jdk/8u162-b12/0da788060d494f5095bf8624735fa2f1/jdk-8u162-macosx-x64.dmg
Already downloaded: /Users/brysgo/Library/Caches/Homebrew/Cask/java8--1.8.0_162-b12,0da788060d494f5095bf8624735fa2f1.dmg
==> Verifying checksum for Cask java8
==> Installing Cask java8
==> Running installer for java8; your password may be necessary.
==> Package installers may write to any location; options such as --appdir are ignored.
==> installer: Package name is JDK 8 Update 162
==> installer: Installing at base path /
==> installer: The install failed (The Installer encountered an error that caused the installation to fail. Contact the software manufacturer for assistance.)
==> Purging files for version 1.8.0_162-b12,0da788060d494f5095bf8624735fa2f1 of Cask java8
Error: Command failed to execute!

==> Failed command:
/usr/bin/sudo -E -- /usr/sbin/installer -pkg /usr/local/Caskroom/java8/1.8.0_162-b12,0da788060d494f5095bf8624735fa2f1/JDK 8 Update 162.pkg -target /

==> Standard Output of failed command:
installer: Package name is JDK 8 Update 162
installer: Installing at base path /
installer: The install failed (The Installer encountered an error that caused the installation to fail. Contact the software manufacturer for assistance.)

==> Standard Error of failed command:

==> Exit status of failed command:
#<Process::Status: pid 85954 exit 1>

Output of brew cask doctor
==> Homebrew-Cask Version
Homebrew-Cask 1.5.10
caskroom/homebrew-cask (git revision 1fc48; last commit 2018-03-14)
==> macOS
10.13.3
==> SIP
Enabled
==> Java
9.0.4, 1.8.0_162, 1.8.0_161, 1.8.0_152, 1.8.0_151, 1.8.0_144, 1.8.0_131, 1.8.0_112
==> Homebrew-Cask Install Location
<NONE>
==> Homebrew-Cask Staging Location
/usr/local/Caskroom
==> Homebrew-Cask Cached Downloads
~/Library/Caches/Homebrew/Cask (2 files, 629.2MB)
==> Homebrew-Cask Taps:
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-cask (3932 casks)
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-drivers (145 casks)
/usr/local/Homebrew/Library/Taps/caskroom/homebrew-versions (173 casks)
==> Contents of $LOAD_PATH
/usr/local/Homebrew/Library/Homebrew/cask/lib
/usr/local/Homebrew/Library/Homebrew
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/gems/2.3.0/gems/did_you_mean-1.0.0/lib
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/site_ruby/2.3.0
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/site_ruby/2.3.0/x86_64-darwin9.0
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/site_ruby/2.3.0/universal-darwin9.0
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/site_ruby
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/vendor_ruby/2.3.0
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/vendor_ruby/2.3.0/x86_64-darwin9.0
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/vendor_ruby/2.3.0/universal-darwin9.0
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/vendor_ruby
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/2.3.0
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/2.3.0/x86_64-darwin9.0
/usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.3.3/lib/ruby/2.3.0/universal-darwin9.0
==> Environment Variables
LC_ALL="en_US.UTF-8"
PATH="/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/Homebrew/Library/Homebrew/shims/scm"
SHELL="/bin/bash"

Space in java8 package name causing install to fail.

I don't understand how this would cause installer to fail after running the installer -pkg command.
Install seems to work.
==> Satisfying dependencies
==> Downloading http://download.oracle.com/otn-pub/java/jdk/8u162-b12/0da788060d
######################################################################## 100.0%
==> Verifying checksum for Cask java8
==> Installing Cask java8
==> Running installer for java8; your password may be necessary.
==> Package installers may write to any location; options such as --appdir are i
Password:
==> installer: Package name is JDK 8 Update 162
==> installer: Installing at base path /
==> installer: The install was successful.
🍺  java8 was successfully installed!

If it doesn't fail for you I will rerun it with the verbose flag. Maybe I misdiagnosed it.

Verbose will probably have similar output.

installer: The install failed ...

This usually means a problem with the pkg itself but as it installs for me and we haven't had any other reports I think it is something specific to your setup.
Have you tried installing the pkg manually?

I will do that now.

Okay, after manually installing, I found that jdk-8u162-macosx-x64 fails but jdk-8u161-macosx-x64 succeeds.


RetryableEurekaHttpClient - Request execution failed with message: java.net.UnknownHostException:

RetryableEurekaHttpClient - Request execution failed with message: java.net.UnknownHostException:

Please include a concise but complete description and a minimal repro to reopen.


Feature: Make configuration location (path) variable.

Problem description:
While deploying the tool in an embedded system which uses an immutable /etc/ path that is rolled-back after a reboot, I find it rather tricky to point the tool to another directory to find the configuration files.
Using a symlink would probably be enough, but that is rather hacky imho.
Implementation description:
A flag should be introduce to specify the location where the tool should look up for configuration. If it's not provided it should default to the current path /etc/wireguard. This should ensure backwards compatibility.
Can you implement this by yourself and make a pull request?
Yes.

done


image and labels mismatch

Hi. Thanks for open-sourcing this wonderful work. But I got confused when reading and playing with the codes.
I tried tools/visualize.py to visualize the train split of the nuscenes dataset(with configs/nuscenes/seg/camera-bev256d2.yaml) and found that the images and the labels did not match in uv space, such as

But it was OK for the val split. So I dig into the train_pipeline and found that in ImageAug3D you affine transform the images but leave alone the labels and in RandomFlip3D you randomly flip the labels but leave alone the images. Wouldn't this be a problem?
Looking forward to getting a reply from you. Thank you.

Thanks for bringing about the discussion. The logic behind ImageAug3D and RandomFlip3D is that ImageAug3D is only supposed to apply transformations to images, so the LiDAR bounding boxes will not change. In contrast, RandomFlip3D is a LiDAR-only augmentation, so it will not influence the images for the same reason.
The reason why you observe such misalignment of GT boxes and camera images is that we assume no augmentation in the visualization code (see here). The transformation matrix is assumed to be the original lidar2image matrix from the dataset loader, while if you have augmentations you need to consider LiDAR and image augmentation matrices.

Closed due to inactivity. Please feel free to reopen if you feel it necessary.


Add in-place substitution option for linkchecker.py

Changes
Add a new flag -w to enable experimental in-place substitutions for Markdown links only.
This is intended to reduce the amount of repetitive work required to update out-of-date links.

I'm a noob to sig-docs so I can't comment on the need for enforcing the LANG/ prefix in these links.

Hi there, @mboersma! 👋
It's not so much about enforcing the prefix, but more so updating the links to point to the localized content when available. Sometimes when a localized page is created the other pages that it points to are not yet, so the idea here is mainly reduce the lift of updating bits that were out-of-date.
I'm going to incorporate a change though because links to English content are not supposed to get updated (and I forgot about this edge case 😅).
Thanks for reviewing this! I don't have a lot of experience with Python and really appreciate your suggestions 🙏

Do we document the usage of this anywhere? If so, do we need to update for this new functionality?

links to English content are not supposed to get updated

I see. Your changes look good to me then!
@nate-double-u asked:

Do we document the usage of this anywhere?

Good question–I see linkchecker.py documented in this README.md file.

@nate-double-u @mboersma good catch -- I'll add a brief explanation of -w in the README.

/label tide/merge-method-squash
/lgtm


[Maven Cleanup] Remove managed-ledger and zk-utils test-jar dependencies when possible

Currently, many modules depend on managed-ledger-test.jar just because they want to use MockBookkeeper and MockZookeeper.  This made module dependencies hard to understand and track.
This PR introduces a new testmocks module and pulls all mocks from managed-ledger tests into the new module.

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

@sijie PTAL

Add label release/2.5.1