Returned "path" of HTTPReader and GDriveReader diverges

HTTPReader returns the URL for the "path"
https://github.com/pytorch/data/blob/13b574c80e8732744fee6ab9cb7e35b5afc34a3c/torchdata/datapipes/iter/load/online.py#L46
while GDriveReader returns the file name
https://github.com/pytorch/data/blob/13b574c80e8732744fee6ab9cb7e35b5afc34a3c/torchdata/datapipes/iter/load/online.py#L129
Since OnlineReader determines at runtime whether to call the HTTP or GDrive download
https://github.com/pytorch/data/blob/13b574c80e8732744fee6ab9cb7e35b5afc34a3c/torchdata/datapipes/iter/load/online.py#L198-L203
the "path" of the yielded tuples is impossible to predict:
from torchdata.datapipes.iter import IterableWrapper, OnlineReader

dp = IterableWrapper(
    [
        "https://raw.githubusercontent.com/pytorch/data/main/LICENSE",
        "https://drive.google.com/uc?export=download&id=1GO-BHUYRuvzr1Gtp2_fqXRsr9TIeYbhV",
    ]
)
dp = OnlineReader(dp)

for path, _ in dp:
    print(path)

https://raw.githubusercontent.com/pytorch/data/main/LICENSE
torchvision.txt

We should align the two. My vote is out to align based on the file name. Still, returning the URL could also be useful if redirect logic as discussed in https://github.com/pytorch/vision/pull/6060#pullrequestreview-980646299 is added to the HTTPReader.

I agree that we should align the two and the default being the file name is likely better than URL.

I am not sure about it. If any url contains directory path, idk this is the idea behavior. For example
dp = IterableWrapper(
    [
        "https://abc.com/folder1/file1.txt",
        "https://abc.com/folder2/file1.txt",
    ]
)

Returning filename by default becomes a problem for users as there will be duplicate filenames.
So I would prefer option 2 but not enabled it by default for HttpReader. And, in order to align the behavior within OnlineReader, we might have to provide this default functions. This makes the behavior more deverged.
I do have a question about the use case of OnlineReader. Do we actually have such Dataset held in different remote sources? @pmeier
One potential use case might be multi-model. But, for multi-model with different data sources, they also need different pipeline to run pre-processing. Then, IMO, it makes more sense to have a separate Reader for each data source and run a few operations after each Reader, then combine these pipelines together.

Another thing is the other related DataPipes are returning URL not filename such as FSSpec and IoPath.
I understand Google Drive is special case because the URLs don't contain any file name or file path. So, in order to have an aligned result from OnlineReader, it seems we have to return filename and provide a function to retrieve file path/name for urls not for google drive.

I do have a question about the use case of OnlineReader. Do we actually have such Dataset held in different remote sources?

I don't think so, no. But that shouldn't be the issue TBH. OnlineReader is about convenience. The contract the user enters is "Here is a datapipe of URLs. Please give me back a datapipe of streams of the URL data as well as some information identifying each stream (URL / path / ...)". As a user I'm not interested what is happening in the backend, i.e. if I need a different functionality for downloading from GDrive rather than a plain HTTP object.
This is the same argument I'm making in https://github.com/pytorch/vision/pull/6060#issuecomment-1134297103 for loading of archives. In both cases as user I'm willing to trade specific control for convenience.
Note that I'm not saying that we shouldn't have the individual classes. If one for example only wants to perform plain HTTP requests, they can use the HttpReader which has no special handling for GDrive.

Understood about the convenience for users. I am more concern about how to maintain this OnlineReader.
So, to achieve a common ground, we might need to:

Add option for users to define filepath_fn for urls returned by all Readers (disabled by default due to BC)
Add the same option for OnlineReader but provide a default method to return filename. Since we can extend OnlineReader to handle more types of URLs such as S3, do we want to let users to provide a dictionary of functions to extract files based on URL types?
Change the document
Add warning for users the behavior is going to be changed for Readers


need setup include path for rc.exe (wtl fails to build)

Repro:

vcpkg integrate install
vcpkg install wtl
cd installed\x86-windows\share\wtl\samples\MemDlg
Add CMakeLists.txt as following:
add_executable(MemDlg WIN32 MemDlg.cpp MemDlg.rc)
Update stdafx.h and MemDlg.rc to use e.g. #include <wtl/atlapp.h> instead of <atlapp.h>
cmake, followed by msbuild

Result:

cpp builds fine, but rc file fails to find atlres.h

Cause of the failure:

In wtl the header file is needed for rc.exe
scripts\buildsystems\msbuild\vcpkg.targets sets up AdditionalIncludeDirectories for ClCompile but not for ResourceCompile.

fixed in pull request #915


Upgrade to Liquibase 3.10.1 (also liquibase-hibernate5)

This finally fixes the excessive index / foreign key constraint recreations
Fixed tests by extracting changeLog part (Due to polluted outputCapture)
Fixes GroovyChangeLogSpec if default JVM langauge is not en.
String comparisons are now done without spaces in tests
fixes #95 #100 #159

Tested successfully with Grails 4.0.3 in a fairly complex project.

Thank you for the PR, can you please remove the extra spacing from the changes?

I can. Isn't it better readable when indention is correct? With the PR it does not matter anymore, as spaces are removed before comparison anyway.

It probably would be easier not to change some of the indentations like the Specs where the quoted multiline strings have extra indentation. This affects the final string output.

It doesn't matter as I've written, as all blanks are removed from the fixtures and the output before comparison to make the tests more robust, but I will change it back on your request.

I am not very sure but I think there are issues while tracing the history or resolving conflicts where the whitespaces/indentation creates some confusion. So, I would prefer to avoid the changes from this specific PR, we can create another PR specific to those changes.

Done. If you like me to squash commits, feel free to comment.

I will squash it directly while merging, thanks!


make build failed!

hi,
When i build this source code,   after i enter the make build cmd,   it tells me following error:
undefined reference to<EMAIL_ADDRESS>And i have updated the glibc to 2.34,  it shows me the same error!
Can you give me some suggestions?
And now i am trying to compile this app on the arm64 platform， what do i need care about？

This usually happens if your linux distribution is older than we support. We currently target Ubuntu 22.04 for both amd64 and arm64.

Regarding arm64, we support this platform (see https://github.com/spacemeshos/go-spacemesh/blob/develop/.github/workflows/release.yml) - I don't think any special steps are required to build for it.

its because it curl's a pre-compiled binary .so file that is meant for a specific OS version
the makefile should build post-rs from source, specific git commit hash, and use that .so file for linking the program


Interactive Demo

Can you setup an interactive demo using your examples so we can see it in action?

exactly... when looking for libraries its useful to have a demo so we can quickly filter through what's useful or not. having to download or clone to run a demo is not ideal!

You're right. I've published the demo http://babotech.github.io/react-ingrid/


[AVM Module Issue]: Plan fails because for_each loop on diagnostic setting is empty due to Log Analytics workspace ID not known until Apply

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Issue Type?
I'm not sure
(Optional) Module Version
v0.1.4
(Optional) Correlation Id
No response
Description
I am trying to create a Virtual Network and Log Analytics workspace using their respective AVMs. I am trying to pass the ID of the LAW as part of the diagnostic settings configuration for the vnet but the Terraform Plan fails with the following error;

Error: Invalid for_each argument
on main.tf line 70, in resource "azurerm_monitor_diagnostic_setting" "example":
70:   for_each = {
71:     for key, value in var.diagnostic_settings : key => value
72:     if value.workspace_resource_id != null || value.storage_account_resource_id != null || >value.event_hub_authorization_rule_resource_id != null
73:   }
├────────────────
│ var.diagnostic_settings is map of object with 1 element

Example code:
resource "azurerm_resource_group" "name" {
  location = "uksouth"
  name     = "rg-test"
}

module "law" {
  source                                             = "Azure/avm-res-operationalinsights-workspace/azurerm"
  location                                           = azurerm_resource_group.name.location
  name                                               ="law-test"
  resource_group_name                    = azurerm_resource_group.name.name
}

module "vnet001" {
  source                        = "Azure/avm-res-network-virtualnetwork/azurerm"
  location                      = azurerm_resource_group.name.location
  name                          = "vnet-001"
  resource_group_name           = azurerm_resource_group.name.name
  virtual_network_address_space = ["<IP_ADDRESS>/24"]
  subnets = {
    "subnet-001" = {
      address_prefixes = ["<IP_ADDRESS>/24"]
    }
  }
  diagnostic_settings = {
    sendToLogAnalytics = {
      name                           = "sendToLogAnalytics"
      workspace_resource_id          = module.law.resource.id
      log_analytics_destination_type = "AzureDiagnostics"
    }
  }
}

Expected behaviour:
The AVM module should respect the "(known after apply)" availability of values passed as inputs.
Or am I doing something wrong? Thanks

I have resolved my issue, as per the technique described in @matt-FFFFFF video (https://youtu.be/6OkdaPDphz4?si=Mgq9-Ml5WupKb4xc).
The actual code experiencing the issue differs from the original example, as I was passing the LAW ID into a local module that then create a collection of resources, including the diagnostic setting.
So, instead of;
module "common-001" {
  source                     = "./modules/common"
  environment                = var.environment
  application                = var.application
  location                   = var.primary_location
  address_space_prefix       = "10.1"
  tags                       = local.tags
  key_vault_role_assignments = local.common_role_assignments
  log_analytics_workspace_id = resource_id = module.law.resource.id
}

I now do this;
module "common-001" {
  source                     = "./modules/common"
  environment                = var.environment
  application                = var.application
  location                   = var.primary_location
  address_space_prefix       = "10.1"
  tags                       = local.tags
  key_vault_role_assignments = local.common_role_assignments
  log_analytics_workspace_id = {
    resource_id = module.law.resource.id
  }
}

Glad you found the video useful!!


[SP-5275] Backport of PPP-4444 - Random issues with Karaf - Race Cond…

…ition is still occurring  (8.3 Suite)

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -f 'pom.xml' -pl 'engine,core' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine,core' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  core/src/main/java/org/pentaho/di/core/Const.java
  engine/src/main/java/org/pentaho/di/kitchen/KitchenCommandExecutor.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.job.entries.evaluatetablecontent.JobEntryEvalTableContent

Branch Change: -1.1% :small_red_triangle_down:
Complexity Change: -1.1% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.job.entries.getpop.MailConnectionMeta

Branch Change: -1% :small_red_triangle_down:
Complexity Change: -1.5% :small_red_triangle_down:
Instruction Change: -0.3% :small_red_triangle_down:

org.pentaho.di.kitchen.KitchenCommandExecutor

Complexity Change: -0.4% :small_red_triangle_down:
Instruction Change: -1.1% :small_red_triangle_down:
Line Change: -1.8% :small_red_triangle_down:
Method Change: -4.2% :small_red_triangle_down:

org.pentaho.di.trans.step.StepMeta

Branch Change: -0.6% :small_red_triangle_down:
Complexity Change: -0.5% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:
Line Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.datagrid.DataGridMeta

Branch Change: -2% :small_red_triangle_down:
Complexity Change: -1.7% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -2.2% :small_red_triangle_down:
Complexity Change: -2.5% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:
Line Change: -0.5% :small_red_triangle_down:

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Complexity Change: -1.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.sort.SortRowsMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceupdate.SalesforceUpdateMeta

Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceupsert.SalesforceUpsertMeta

Instruction Change: -0.1% :small_red_triangle_down:


Visual Diffs

Revise Visual Diffs page, including:

Organize into sections:

Setting up Visual Diffs
Using/Viewing Visual Diffs

Add note about visualdiffs service key (linked to Setting up Services -> Service Attributes -> visualdiffs
Pull in some of the info from the visualdiffs attribute writeup to the Visual Diffs page
Add a section about the visual diff config being in whatever you're building (See info from Ben)
Add note about using visualdiff under php vs. apache, etc. (see note from James)

Added content specified here to Visual Diffs page, added more troubleshooting info to the Troubleshooting page around visual diffs, and pushed the changes to new-docs.


Feature ACP: basic forums management

I skipped for now deleting forum and editing forums order.

Nice, I’ll review this tomorrow. Thanks!

On 8 Jan 2017, at 18:54, Matslom<EMAIL_ADDRESS>wrote:
I skipped for now deleting forum and editing forums order.
You can view, comment on, or merge this pull request online at:
https://github.com/mybb/mybb2/pull/275 https://github.com/mybb/mybb2/pull/275
Commit Summary
Merged branch master into master
Initial forums management
More forums management
Merged branch master into master
Merged branch master into feature/ACP-forums-management
File Changes
M app/Database/Models/Forum.php https://github.com/mybb/mybb2/pull/275/files#diff-0 (2)
M app/Database/Repositories/Decorators/Forum/CachingDecorator.php https://github.com/mybb/mybb2/pull/275/files#diff-1 (59)
M app/Database/Repositories/Eloquent/ForumRepository.php https://github.com/mybb/mybb2/pull/275/files#diff-2 (72)
M app/Database/Repositories/Eloquent/TopicRepository.php https://github.com/mybb/mybb2/pull/275/files#diff-3 (8)
M app/Database/Repositories/ForumRepositoryInterface.php https://github.com/mybb/mybb2/pull/275/files#diff-4 (48)
M app/Database/Repositories/TopicRepositoryInterface.php https://github.com/mybb/mybb2/pull/275/files#diff-5 (8)
A app/Http/Controllers/Admin/Forums/ForumsController.php https://github.com/mybb/mybb2/pull/275/files#diff-6 (210)
M app/Http/Controllers/Admin/Users/WarningsController.php https://github.com/mybb/mybb2/pull/275/files#diff-7 (2)
M app/Http/Controllers/ForumController.php https://github.com/mybb/mybb2/pull/275/files#diff-8 (2)
A app/Http/Requests/Forums/CreateForumRequest.php https://github.com/mybb/mybb2/pull/275/files#diff-9 (47)
M app/Http/routes.php https://github.com/mybb/mybb2/pull/275/files#diff-10 (26)
M app/Presenters/ForumPresenter.php https://github.com/mybb/mybb2/pull/275/files#diff-11 (16)
A app/Twig/Extensions/AdminDeleteButton.php https://github.com/mybb/mybb2/pull/275/files#diff-12 (47)
A app/Twig/Extensions/ListAllForums.php https://github.com/mybb/mybb2/pull/275/files#diff-13 (56)
M config/twigbridge.php https://github.com/mybb/mybb2/pull/275/files#diff-14 (2)
A resources/lang/admin/en/forums.php https://github.com/mybb/mybb2/pull/275/files#diff-15 (32)
M resources/lang/admin/en/general.php https://github.com/mybb/mybb2/pull/275/files#diff-16 (1)
A resources/views/admin/forums/add.twig https://github.com/mybb/mybb2/pull/275/files#diff-17 (12)
A resources/views/admin/forums/edit.twig https://github.com/mybb/mybb2/pull/275/files#diff-18 (12)
A resources/views/admin/forums/forum-item-select.twig https://github.com/mybb/mybb2/pull/275/files#diff-19 (8)
A resources/views/admin/forums/forum-item.twig https://github.com/mybb/mybb2/pull/275/files#diff-20 (10)
A resources/views/admin/forums/list.twig https://github.com/mybb/mybb2/pull/275/files#diff-21 (11)
A resources/views/admin/partials/delete_button.twig https://github.com/mybb/mybb2/pull/275/files#diff-22 (5)
A resources/views/admin/partials/forums/forum_form.twig https://github.com/mybb/mybb2/pull/275/files#diff-23 (114)
M resources/views/admin/warnings/warning_types/list.twig https://github.com/mybb/mybb2/pull/275/files#diff-24 (6)
M resources/views/layouts/admin.twig https://github.com/mybb/mybb2/pull/275/files#diff-25 (8)
Patch Links:
https://github.com/mybb/mybb2/pull/275.patch https://github.com/mybb/mybb2/pull/275.patch
https://github.com/mybb/mybb2/pull/275.diff https://github.com/mybb/mybb2/pull/275.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/mybb/mybb2/pull/275, or mute the thread https://github.com/notifications/unsubscribe-auth/AAttcWzWOf3Ky5ynSc54XDZnXITs0YNWks5rQTDsgaJpZM4Ldw7G.

Really good work @Matslom

Yes! good job @Matslom

Tanks @ATofighi for review.


Update to new forge version and mappings

I felt that the forge version and mappings should be updated. Remember to rerun gradlew!

I did update it but forgot to publish it to GitHub


fix: SQL statements use standard client-side read methods now

Tests fail most of the time. The server will core dump because of grpc.


⚠ Bofrost.be has degraded performance

In 6453489, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1082 ms

Resolved: Bofrost.be performance has improved in ed19310 after 6 minutes.


jsonnet: unify internal configuration field name

Before this PR we were merging default configuration and user-provided configuration into a field or a variable with name config or cfg. This PR provides a unified way to store configuration in the form of hidden _config field. Such field is a de-facto standard for this particular case and should prevent name collision (we had it in grafana component).
Changes:

exposed _config for prometheus-operator component
renamed cfg to _config for grafana.libsonnet
renamed config to _config for other components
adjusted platforms/eks.libsonnet
adjusted addons/weave-net/weave-net.libsonnet

Good catch! Fixed it and rebased to fix merge conflicts.

Rebased to fix merge conflicts and CI issues.

ping @prometheus-operator/kube-prometheus-reviewers :)

is there a particularly strong reason for using underscore?

Yes, without it the field can cause conflicts. This can be seen for example in grafana component, where we used cfg just to prevent name conflict with config.
Since underscore is already a somewhat established way of configuring "private" fields I used the same here.

Would this change affect downstream? e.g., someone who might be overriding config fields?
It is excellent to stick to a pattern though, I really like this change!

Would this change affect downstream?

It might, but this is also the best time to introduce this change as this is not present in release-0.7 and release-0.8 is due to be released soon.


New feature: allow configurable entrypoint instead of requiring config.js

Motivation:

I have a typescript codebase and am adding storybook to it
I want hot-reloading and no intermediate build step

To do this, my entrypoint into the stories should be a typescript file, which will be loaded with the typescript loader.
I have a working example of this in https://github.com/amacleay/demo-storybook-ts-hotreload:

Add config.ts from the story configuration directory to the entrypoints: https://github.com/amacleay/demo-storybook-ts-hotreload/blob/eb88b22e600547536ae951185c388ab81316d0c2/webpack.config.js#L11
Create an empty config.js file in the config directory: https://github.com/amacleay/demo-storybook-ts-hotreload/blob/eb88b22e600547536ae951185c388ab81316d0c2/config.js

I did not find a better way to have a pure-TS config entrypoint.
Possible solution:

Add a new configuration parameter, entrypoint, which ttakes the name of the entrypoint file to use in place of config.js

I am not sure about a new parameter solution though

I'm definitely open to other ideas; I am not terribly fond of a parameter for an entry point myself, but I think it would get the job done.

I've rather resigned myself to the fact that, with storybook, one needs to write the code that storybook hooks into (config.js, addons.js) using the particular babel-ified flavor that storybook expects.
FWIW, I think this is a very sub-optimal interface because, in the general case, it ties the consumer tightly into the babel configuration of storybook itself without any clear way to know what that babel configuration is and how it will behave. I think that ship has long since sailed, though.

Released as (4.0.0-alpha.15)[https://github.com/storybooks/storybook/releases/tag/v4.0.0-alpha.15]


Can't use shortcut Alt + A to click "Don't save" on Brazilian Portuguese

Issue Type: Bug

Use Brazilian Portuguese translation
Create new file and insert any text
Close tab. A dialog will appear with options "Salvar", "Não salvar" and "Cancelar".
Try triggering "Não salvar" using the keyboard shortcut Alt + A (letter A is underlined, indicating the shortcut)

The problem occurs because the a letter has a tilde on it (and it should have), which makes it impossible to use only the A key.
A simple solution would be to underline 'N' instead of 'ã', producing the Alt + N shortcut

We should adjust that in the translation. The character to use is usually denoted with a &

Fix implemented - Note: it will take a few days to make it into the build
Resource ID:	0;"Xliff file"\0;"Xliff Resources"\0;"src/vs/workbench/services/dialogs/browser/abstractFileDialogService.dontSave"
Relative File Path:	vscode-loc-drop/loc/en/src/en/vscode-workbench/vs_workbench_services_dialogs.xlf.lcg
Project:	VSCode
Subtenant:	VSCode

fix should be in the build now


[MS4] Modernization #2

We need to add some code formatting standards and address issues detected by multiple static analysis tools:
Code formatting standards can be addressed by:

[x] Spotless

Regarding static analysis we plan to:

[x] #1849
[ ] #1850
[x] solve Dependabot alerts as soon as possible,
[x] solve problems detected by CodeQL,
[x] solve problems detected by Scorecard.

Reports by the three tools integrated into Github (Dependabot, CodeQL, Scorecard) have been addressed.
We can not achieve the CII-Best-Practices badge, until we introduce a dynamic analysis tool (fuzzing).

Commits related to this issue:

da836b16d6daf87fac7ca80a5123150fb92669f3

We introduced a deterministic formatter in apache/logging-parent#59.
The process of choosing the right formatter started on September 20th in this dev thread. The choice fell on palantir-java-format, because:

it has an excellent Java 8 (lambdas) support (compared to google-java-format),
it is largely compatible with our previous Java Style Guide. Compared to the Google Java Format, it uses an indentation of 4 spaces instead of 2 spaces,
compared to the Eclipse Java formatter it has no formatting options. This is actually an advantage, since it prevents discussing each one of the hundreds Eclipse options. Moreover some Eclipse options render the formatting non-deterministic,
it provides support for both our build environment (Spotless) and IntelliJ IDEA editor,

In order to provide support for the Eclipse IDE, we ported Google's Java Format Eclipse plugin to palantir-java-format: palantir/palantir-java-format#949.


arch-riscv: Implement Zcmt

Implement table jump (cm.jt and cm.jalt) instructions
According to spec, when jump to the JVT entry address. We need to take two fetch steps. The first is fetching the zcmt instruction. The second is fetching the JVT entry. The JVT entries is considered the extension of table jump instructions. The JVT entries read from icache_port and enable get them in I-cache, so we can't follow the generic way to load JVT entry from memory helper.
In gem5 implementation we will get two StaticInst for cm.jalt. The first is move the PC to address of JVT entry, and mark the pc_state to zcmt fetch mode. When in zcmt mode, we will get the StaticInst that jumps to the JVT entry, and leaves the zcmt fetch mode.
If there is fault during second fetching, xEPC is set to instruction address, xCAUSE is set to expect type of fault, xTVAL is set to address of JVT entry
Spec: https://github.com/riscv/riscv-isa-manual/blob/main/src/zc.adoc#zcmt
Change-Id: I8e0caca35a22a51386677d70fa09c3dc97e4610a

Looks like the checkpoint tests are failing. Should this affect checkpointing?

I updated the checkpoint https://resources.gem5.org/resources/riscv-hello-example-checkpoint?database=gem5-resources&version=4.0.0, you would need to update the following line to use version 4.0.0 and then the CI test will use the new checkpoint
https://github.com/gem5/gem5/blob/63d25922a2db14643f132c0f62a224c7f7991c49/configs/example/gem5_library/checkpoints/riscv-hello-restore-checkpoint.py#L94

@Harshil2107, can you look into or explain why the checkpoint test is still failing? Please do what you can to get this change merged.


Remove HttpContext access from constructor

This PR removes the HttpContext access from the constructor(s). This is needed to prevent the DI tree from breaking when verifying it with SimpleInjector.

Thanks for your contibution @Spaggel

2.1.1 has been published https://www.nuget.org/packages/Core.Flash/2.1.1

Thank you!


[WIP] Spark: Implement an action to remove orphan files

This WIP PR adds a Spark action that removes orphan files that can be left in some edge cases like executor preemption. At this point, the action only cleans up data files by comparing the content of the data location with the all_data_files metadata table. Later, we should be able to clean up orphan metadata files as well.
TODOs:

Ignore custom metadata locations while cleaning up data files.
Documentation.

Thanks @aokolnychyi for the PR. Based on what I see on PR, you are trying to clean up files that are not referenced by metadata table and all_data_files table. I am actually going to verify it, as I get to use this work in our project, but do the files which fail to get deleted during cleanup after commit here, is also cleaned up with this work? My point is whether the dataFile which is deleted as part of an expired snapshot belongs to all_data_files table or not.

@mehtaashish23, this action should remove all orphan files including those that we failed to delete while expiring snapshots. The all_data_files metadata table gives us currently referenced files in metadata.

@aokolnychyi, this looks almost ready to me. The parallel file listing looks like it needs to be updated, and we need javadoc. Otherwise I think the other points can be done as follow-ups.

I'm merging this since it's large and the remaining comments are minor. That avoids needing to re-read the whole commit for small updates. Thanks for adding this, @aokolnychyi! I think it is going to be really useful.


Prettier don't follow the same rules of ESlint : split the html attributes to multiple lines in .vue page

Hi, very clean code this project.
The App.vue  after yarn format command:
<img alt="Vue logo"<EMAIL_ADDRESS>/>

No changes in the page, after CTRL + S  (save command) on vscode :
<img alt="Vue logo"
<EMAIL_ADDRESS>/>

How to force Prettier to NOT split the html attributes to multiple lines ?

Below my config files.
tsconfig.json : (no changes)
prettierrc.js : (no changes)
package.json :   (updated with pnpm up)
{
  "name": "vite-vue-typescript-starter",
  "version": "0.0.0",
  "scripts": {
    "dev": "vite",
    "build": "vue-tsc --noEmit && vite build",
    "deploy": "vue-tsc --noEmit && vite build --base '/vue-ts/'",
    "preview": "vite preview",
    "format": "prettier --write .",
    "lint": "eslint . --ext .js,.jsx,.ts,.tsx,.vue",
    "test": "jest"
  },
  "dependencies": {
    "vue": "^3.2.37"
  },
  "devDependencies": {
    "@types/jest": "^28.1.2",
    "@typescript-eslint/eslint-plugin": "^5.29.0",
    "@typescript-eslint/parser": "^5.29.0",
    "@vitejs/plugin-vue": "^2.3.3",
    "@vue/test-utils": "^2.0.1",
    "@vue/vue3-jest": "^28.0.0",
    "eslint": "^8.18.0",
    "eslint-config-prettier": "^8.5.0",
    "eslint-plugin-jest": "^26.5.3",
    "eslint-plugin-vue": "^9.1.1",
    "jest": "^28.1.1",
    "jest-environment-jsdom": "^28.1.1",
    "jest-transform-stub": "^2.0.0",
    "prettier": "^2.7.1",
    "ts-jest": "^28.0.5",
    "typescript": "^4.7.4",
    "vite": "^2.9.12",
    "vue-eslint-parser": "^9.0.2",
    "vue-tsc": "^0.38.1"
  },
  "//devDependencies": {
    "vue-tsc": "do not update to 36+ because of DefineComponent issue"
  }
}

Thanks in advance.

@clabnet Hi. Thanks. Are there any changes to .editorconfig in your project? Or maybe you have multiple config files for prettier. Can you show me ls -la of the folder containing package.json?

@clabnet This looks like an issue with your VSCode. Sorry, but I do not use it, so I can't help you with that. Try disabling some plugins and see which one is the culprit.


graph blank, no bpmn data?

ok - got it running on a mac, but the graph doesn't seem to render anything
there's data in the sentence_data.json file, and that looks ok
and here's the model output - also looks good
model_output.json
but the graph is blank? the bpmn structure is empty etc....
fwiw i'm trying to graph business process in the city https://www.charlottesville.gov/216/Starting-a-Small-Business

@alibama Sorry for the late reply, been working on a new version of this app, you can check it out: https://github.com/jtlicardo/bpmn-assistant


update

due to removing the Copy method and making Value immutable

Current coverage is 96.25% (diff: 100%)

Merging #7 into master will not change coverage

@@             master         #7   diff @@
==========================================
  Files            10         10
  Lines          2188       2188
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           2106       2106
  Misses           35         35
  Partials         47         47

Powered by Codecov. Last update 22a8a16...ebebd17


Setup Colours

Setup colours for the landing page
Checklist:

[ ]  setup custom colours in tailwind.config
https://tailwindcss.com/docs/customizing-colors#using-custom-colors

Colour Palette:

Colours
HEX
RGB

Base
#001236
0, 18, 54

Primary
#523FF4
82, 63, 244

Secondary
#8375FF
131, 117, 255

Text
#CEDFFF
206, 223, 255

Accent
#12B3C2
18, 179, 194

Colour Palette Reference:

instead of setting the colours in theme, it should be extended


🚀 Performance: Incremental Highlighting

At the moment each keystroke triggers highlighting for the whole document. While this is not an issue for small documents this might bring big performance losses on larger documents.
Though there needs to be some kind of padding of the changed text range since a single character cannot be parsed to anything. Ideally this would be the entire current line.

[ ] Highlight only current line on text input.
When this results in an error there should be two options:

[ ] When no further editing is done (i.e. some small timeout), try highlighting the whole document.
[ ] When further editing is done, do nothing.

[ ] When selected text (multi-line) is deleted, also include previous and following line.
[ ] When multi-line text is inserted highlight those lines.

From what I understood, a key advantage tree sitter provides is highlighting in the scope of the change, not necessarily the current line itself. Is this something we could instead do?


durable objects's documentation is missing?

Is this feature implemented?

Ok, I found it, it is doNamespace


Error in termux

Whin i click on play that show
Error type 3
Error: Activity class {is.xyz.mpv/is.xyz.mpv.MPVAc

Closing due to inactivity since August 26.


Exoplayer: Video Effects Not Applied When Using MediaItem with Image URI

Version
Media3 main branch
More version details
Library Version: androidx.media3:media3-transformer:1.4.1
Android Version: Android 13 and other emulators.
Device Model: Tested in Moto Edge 40 and pixel emulators
Devices that reproduce the issue
Device Model: Tested in Moto Edge 40 and pixel emulators .
Devices that do not reproduce the issue
NA
Reproducible in the demo app?
Yes
Reproduction steps
I've been exploring the Media3 APIs for applying video effects, and I've encountered an issue when trying to preview effects on image-based MediaItem. The effects are successfully applied and previewed when the MediaItem is a video, but the same effects are not applied when the MediaItem is an image (e.g., a JPEG file).
Working Case (Video):
Create a MediaItem with a video URI.
Apply video effects such as zoom or rotate.
The effects are successfully applied and previewed using ExoPlayer.
val mediaItem1 = MediaItem.Builder().setUri(videoUri).build()

val player = ExoPlayer.Builder(context).build()
player.playWhenReady = true
player.setMediaItem(mediaItem1)
player.setVideoEffects(
    ImmutableList.of(
        createRotateEffect(3000),
        createZoomEffect(3000, 0.5f, 1f),
        createZoomEffect(3000, 1f, 0.5f)
    )
)
player.prepare()
playerView.player = player

Not Working Case (Image):
Create a MediaItem with an image URI and set a duration using setImageDurationMs.
Apply the same video effects.
The effects are not applied when previewing using ExoPlayer.
val mediaItem1 = MediaItem.Builder()
    .setUri(getUri(context, "image1.jpg"))
    .setImageDurationMs(3000)
    .build()

val player = ExoPlayer.Builder(context).build()
player.playWhenReady = true
player.setMediaItem(mediaItem1)
player.setVideoEffects(
    ImmutableList.of(
        createZoomEffect(3000, 0.5f, 1f)
    )
)
player.prepare()
playerView.player = player

Expected result
The video effects should be applied and previewed correctly, even when the MediaItem is an image.
Actual result
The effects are not applied when the MediaItem is an image.
Media
Screen_recording_20240902_200249.webm
Bug Report

[ ] You will email the zip file produced by adb bugreport to<EMAIL_ADDRESS>after filing this issue.

Please note that the same is working for export using Media transformer api.Same has been discussed in https://github.com/androidx/media/issues/1658 and working as expected now.

Hi @droid-girl  Could you please help with this query.

@claincly  I see this restrict option for compositionPlayer. Will this be a restriction for other apps to use this ?

It's added because it's currently under construction and may be less tested, but you can try it out no problem.

Thank you @claincly !!
I’ve tested CompositionPlayer, and it seems to be working well . Given that it is currently marked with the @RestrictTo(LIBRARY_GROUP) annotation and noted as being under construction, is it recommended to use it in a production app at this stage ? Are there any potential issues or limitations that we should be aware of before proceeding?. Thanks.

I guess that depends on what you want to achieve with it and what features do you need, like what is your typical set up, what you want to play with it?

These features should all be well-supported, one note on the TextOverlay, we don't support Composition level effects yet so you have to add it to every EditedMediaItem

Ok. Thank you very much for all the details.


Initial implementation of dashboard

This PR includes a HTTP server to serve the dashboard.
A frontend using vue.js is also implemented.

I also added an implementation of the frontend to this PR.
See example: https://ossf-malware-analysis.storage.googleapis.com/index.html
and https://ossf-malware-analysis.storage.googleapis.com/index.html#/package?ecosystem=pypi&name=mst-ssh&version=0.1.2


access token not generated

Starting new HTTPS connection (1): android.clients.google.com
Starting new HTTPS connection (1): android.clients.google.com
Access token not generated
Session not created successfully

I installed and did everything y'all said to do but ^ im running windows 10

I get the same on Ubuntu 16.04


WinRM::WinRMWebService (NameError) If Newer Gem Versions Are Used

The following ruby gem versions must be used or the below error occurs on job runtime:

winrm - 1.8.1 - May 6, 2016 (43.5 KB)
winrm-fs - 0.4.3 - June 21, 2016 (30 KB)

/var/lib/rundeck/libext/cache/rd-winrm-plugin-1.4.1/winrmexe.rb:97:in `': uninitialized constant WinRM::WinRMWebService (NameError)

Seems the new major version of one or both of these gems broke a reference.
As I cannot install v1.0.0 of winrm-fs without installing v2.0.1 of winrm I cant be sure which one breaks, but winrm is a safe guess.

Fixed by #30


Don't work for resized window

Don't work for resized window, can triggered by resize unity window

Hi, thanks for opening. Is this for Windows or Mac? If Windows, could you checkout and try https://github.com/marijnz/unity-toolbar-extender/tree/potential_windows_fix?

Hey. So by not working, you mean it's not updating while you're resizing right? After it, it snaps to the right position and works as usual? So in other words, it's just the visual glitch?

Yes,it's a visual glitch like these.

Fixed in https://github.com/marijnz/unity-toolbar-extender/pull/7 which got merged to master. You can double check and close the issue if all is good.


Add possibility to extend the Home Assistant template engine

Description
Add support for extending the Home Assistant template engine.
Motivation and Context
It would be cool if Spook could extend the Home Assistant template engine with custom functions/tests.
How has this been tested?

Screenshots (if appropriate):

fnmatch is not an existing template method in Home Assistant, but injected by Spook.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Other

Checklist

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

You are just AWESOME Frenk - Thanks you so much for Sppok (I do not use it and of course never would :) )


Security.ServerVulnerabilityAssessment API failed to delete the Server Vulnerability Assessment

After tested, API failed to delete the Server Vulnerability Assessment and threw below error message. After confirmed with service team, seems it's a bug. Hopes it could be fixed by service team asap since we're blocked on this issue.
DELETE https://management.azure.com/subscriptions/xx-xx-xx-xx/resourceGroups/xxxxxx/providers/Microsoft.Compute/virtualMachines/xxxxx/providers/Microsoft.Security/serverVulnerabilityAssessments/default

Error message:
security.ServerVulnerabilityAssessmentClient#Delete: Failure responding to request: StatusCode=202 -- Original Error: autorest/azure: error response cannot be parsed: "" error: EOF

This is a swagger bug - this API appears to be an async operation (otherwise it would not return 202 as response code), but it is not marked as long-running-operation in the swagger which results in an SDK with issues.

This is a swagger bug - this API appears to be an async operation (otherwise it would not return 202 as response code), but it is not marked as long-running-operation in the swagger which results in an SDK with issues.


arch/arm64: syscall SYS_switch_context and SYS_restore_context use tcb as parm

sys_call2(SYS_switch_context, (uintptr_t)rtcb, (uintptr_t)tcb)
sys_call1(SYS_restore_context, (uintptr_t)next)
Summary
Update this section with information on why change is necessary,
what it exactly does and how, if new feature shows up, provide
references (dependencies, similar problems and solutions), etc.
Impact
Update this section, where applicable, on how change affects users,
build process, hardware, documentation, security, compatibility, etc.
Testing
Update this section with details on how did you verify the change,
what Host was used for build (OS, CPU, compiler, ..), what Target was
used for verification (arch, board:config, ..), etc. Providing build
and runtime logs from before and after change is highly appreciated.

[Experimental Bot, please feedback here]
No, this PR does not meet the NuttX requirements. The provided information is essentially placeholder text copied from the template.  It lacks critical details about the actual changes being made. Specifically, the following sections are incomplete:

Summary:  Completely missing the why, what, and how of the change. The two lines of code provide no context.
Impact:  All impact assessments are left as placeholders.  Without knowing the change, impact cannot be assessed.
Testing:  Completely missing build host, target details, and before/after testing logs.  The current "testing logs" are just function calls and not actual logs.

The submitter needs to fill out these sections completely with specific details about their change before the PR can be considered. Just including the function calls sys_call2(SYS_switch_context, ...) and sys_call1(SYS_restore_context, ...) doesn't explain anything about the purpose or effect of the change.

@pussuw could you review?

@pussuw could you review? I can't add you to the reviewer list, do you join github Apache group?

Yes, maybe there is something I forgot to do. I'll check later today.

d arch/arm64/src/common/arm64_syscall.c be updated too ?

done


Can't login on Docker version

I tried several versions of the Docker container, but I can't login to any of them.
I'm running an older version on CentOS, and I want to reuse the database. This seems to work, Status page shows green checks.
However, I can't login. It goes back to empty fields with a 'Create' button.
Adding a new user via CLI doesn't help, either.
Log shows this;
[Wed May  2 07:38:45 2018] [error] Problem with execution of sql statement select sha2(password,256) from users where username = ? FUNCTION seccubus.sha2 does not exist at lib/Seccubus/DB.pm line 156.

This should be fixed now with the new docker images.


[Android] App crash on android search assistant opening

Description
App crashes on opening android search assistant when modal is visible
Crash log
03-29 17:43:44.600 18602 18602 E AndroidRuntime: FATAL EXCEPTION: main
03-29 17:43:44.600 18602 18602 E AndroidRuntime: Process: com.modalcrash, PID: 18602
03-29 17:43:44.600 18602 18602 E AndroidRuntime: java.lang.NullPointerException: Attempt to invoke virtual method 'void android.view.View.dispatchProvideStructure(android.view.ViewStructure)' on a null object reference
03-29 17:43:44.600 18602 18602 E AndroidRuntime: 	at android.view.ViewGroup.dispatchProvideStructure(ViewGroup.java:3080)
03-29 17:43:44.600 18602 18602 E AndroidRuntime: 	at android.view.ViewGroup.dispatchProvideStructure(ViewGroup.java:3080)
03-29 17:43:44.600 18602 18602 E AndroidRuntime: 	at android.view.ViewGroup.dispatchProvideStructure(ViewGroup.java:3080)
03-29 17:43:44.600 18602 18602 E AndroidRuntime: 	at android.view.ViewGroup.dispatchProvideStructure(ViewGroup.java:3080)
03-29 17:43:44.600 18602 18602 E AndroidRuntime: 	at android.view.ViewGroup.dispatchProvideStructure(ViewGroup.java:3080)
03-29 17:43:44.600 18602 18602 E AndroidRuntime: 	at android.view.ViewGroup.dispatchProvideStructure(ViewGroup.java:3080)
03-29 17:43:44.600 18602 18602 E AndroidRuntime: 	at android.app.assist.AssistStructure$WindowNode.(AssistStructure.java:422)
03-29 17:43:44.600 18602 18602 E AndroidRuntime: 	at android.app.assist.AssistStructure.(AssistStructure.java:1335)
03-29 17:43:44.600 18602 18602 E AndroidRuntime: 	at android.app.ActivityThread.handleRequestAssistContextExtras(ActivityThread.java:2833)
03-29 17:43:44.600 18602 18602 E AndroidRuntime: 	at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1667)
03-29 17:43:44.600 18602 18602 E AndroidRuntime: 	at android.os.Handler.dispatchMessage(Handler.java:102)
03-29 17:43:44.600 18602 18602 E AndroidRuntime: 	at android.os.Looper.loop(Looper.java:179)
03-29 17:43:44.600 18602 18602 E AndroidRuntime: 	at android.app.ActivityThread.main(ActivityThread.java:6101)
03-29 17:43:44.600 18602 18602 E AndroidRuntime: 	at java.lang.reflect.Method.invoke(Native Method)
03-29 17:43:44.600 18602 18602 E AndroidRuntime: 	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:865)
03-29 17:43:44.600 18602 18602 E AndroidRuntime: 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:755)
Reproduction Steps and Sample Code
Simple app that can help to reproduce
https://github.com/sooth-sayer/RNModalCrash
Additional Information

React Native version: 0.42.3
Platform: Android

+1
We need an urgent for fix for this bug

Happening for me using 0.45.1

Hi there! This issue is being closed because it has been inactive for a while. Maybe the issue has been fixed in a recent release, or perhaps it is not affecting a lot of people. Either way, we're automatically closing issues after a period of inactivity. Please do not take it personally!
If you think this issue should definitely remain open, please let us know. The following information is helpful when it comes to determining if the issue should be re-opened:

Does the issue still reproduce on the latest release candidate? Post a comment with the version you tested.
If so, is there any information missing from the bug report? Post a comment with all the information required by the issue template.
Is there a pull request that addresses this issue? Post a comment with the PR number so we can follow up.

If you would like to work on a patch to fix the issue, contributions are very welcome! Read through the contribution guide, and feel free to hop into #react-native if you need help planning your contribution.

Happening for me using 0.47.1

Please, reopen the issue, because trouble still exist.

@Dmytro96, new issue here https://github.com/facebook/react-native/issues/15932


Not load fieds

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Platform: [e.g. Chrome extension, mozilla addon, desktop app, npm package, snap]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

The same problem, Altair doesn't display a list of fields in the docs.

have the same issue. I think it appeared after last update.

But auto-completion in the query editor is working and these fields are available

Can you click that icon and select an option? That should show "None" by default.

Can you click that icon and select an option? That should show "None" by default.
when i select any item, nothing happens

Weird. Any errors in the browser dev console?

Also what specific version of Altair are you using?

Latest release is 5.0.6. just to make sure I'm debugging the right version.

Weird. Any errors in the browser dev console?
No, queries work as they should, right now I'm working with one of them.
Also what specific version of Altair are you using?
5.0.6. Everything works on previous version, I checked with a friend

I can't figure out what the issue is with your setups (it works fine for me), but I've updated it to always return the fields.

Let me know if this is fixed in the latest version.

the problem has been fixed in version <IP_ADDRESS>


Schedule meeting with Alan to discuss MapLibre contributions and possible roadblocks

@alanchenboy from GrabMaps has been making code contributions to MapLibre. We want to schedule a meeting and define what should be part of the Milestone to upstream GrabMaps' bugfixes.

Meeting set for 9 Feb 9:00 - 10:00 (CET), 16:00 - 17:00 (UTC+8)

-_- Glad to be a contributor.


Desktop Application

Cross-platform desktop apps for the major operating systems such as Windows, Linux, and MAC with electron

🎉 A new version of camera.ui
A new version of camera.ui is currently under active development. An initial alpha/beta release and previews are coming soon. Stay tuned for exciting updates: https://github.com/seydx/camera.ui/issues/448 .
This version will no longer be developed / fixed. The new version contains many novelties and fixes (most important is for HKSV recording).


Arbitration & Transaction Fixes

This adds the shared conversation to the arbitration component and makes some other repairs.

I'm getting the dreaded {code: -32603, message: "Internal JSON-RPC error."} when I try to accept an offer. (On candy maple, accepting as seller as account 2, after offer made by account 1)
Will keep trying...

@wanderingstan on sample listings? You can't accept them since we added token to the equation. Franck is working on it now.

I'm unable to get messaging working locally. Is that supposed to work?

@wanderingstan plz try again 🙏


ERC20WithPermit token

Generic implementation of ERC20 token with permit functionality, used
already in keep-network/tbtc-v2 and keep-network/coverage-pools.
User can authorize a transfer of their token with a signature conforming
EIP712 standard instead of an on-chain transaction from their address.
Anyone can submit this signature on the user's behalf by calling permit
function, as specified in EIP2612, paying gas fees, and possibly
performing other actions in the same transaction.
By the time of implementing this contract, to my knowledge, there is no
other audited EP2612 token implementation other than Uniswap V2 on which
this code is based on. OpenZeppelin has quite recently added permit to
their ERC20 but this code is still in a draft state, was released in the
most recent version of the library, and to my understanding was not
audited yet by another firm, and could have breaking change in the
future.
The implementation mostly mirrors UniswapV2ERC20. It adds some tweaks
from OpenZeppelin ERC20 implementation such as more meaningful revert
messages and Approval event emitted on calls to transferFrom. This
allows applications to reconstruct the allowance for all accounts just
by listening to the event. Additionally, this token implements
approveAndCall pattern.
Unit tests were based on OpenZeppelin ERC20 unit tests with our own
tests implementation for EIP2612 and EIP712. The token code in
keep-network/tbtc-v2 this is is a ctrlc/ctrlv was audited by ToB.

Oh boy, I'm sorry for spamming. I was using VSC plugin for the review and I was 100% sure the comments are being added to the pending review so I will submit them all at once 🤦🏻

Let's open a separate issue for test improvements. All tests but permit-related were ported from OpenZeppelin and already reviewed in coverage-pools PR. If we want to do some modifications, let's not hold PRs that wait for this one.


Fix Browsersync html task reload issue

Fixes #251
Only reload once at the end of the stream to keep browser from reloading too early

Sounds good. Thanks for figuring that out!


Verify blocks between checkpoints

Basic checkpoint verifier implementation.

I rebased this PR on #538, to pick up the test cleanup changes.

I made some progress on shared mutable data (the current checkpoint height). I'll be able to do something similar to share the cached blocks between futures. (I also rebased, because there were conflicts.)
I'm going to read up on tokio::sync::oneshot, and then try to implement chained verifies:
https://github.com/ZcashFoundation/zebra/issues/429#issuecomment-646338626
I expect to revise all the new functions, so I'm not sure there's much point in anyone reviewing them.
I would appreciate some help getting the &self lifetime right on methods. I couldn't work out how to do that, so I turned the methods into functions, and passed them the shared data directly:
https://github.com/ZcashFoundation/zebra/pull/524/files#diff-ec898ca6f6f9c2940a1e5e1aa27f44b5R138-R225

I would appreciate some help getting the &self lifetime right on methods. I couldn't work out how to do that, so I turned the methods into functions, and passed them the shared data directly:
https://github.com/ZcashFoundation/zebra/pull/524/files#diff-ec898ca6f6f9c2940a1e5e1aa27f44b5R138-R225

After thinking about this a bit more, I'd like to create an inner struct for all the mutable state. Then CheckpointVerifier can wrap that struct in Arc<Mutex<...>>. The inner struct can have methods that depend on the mutable state. And the CheckpointVerifier can have methods that depend on immutable state.
(And if we need immutable data in the mutable state methods, we can call the CheckpointVerifier methods and pass the results in. That's not ideal, but it's better than multiple mutexes.)

I don't think it's necessary to create an inner struct for all the mutable state or to use an Arc<Mutex<_>> to share access to it, because the checkpoint design here doesn't require sharing any state, it hands the rx end of the oneshot to the caller as the response future.  It looks like the need to share state comes from trying to adapt the previous stub code to solve the checkpointing problem, but the two designs aren't really  structurally similar.

I've implemented the general structure, but there are some bugs in the code (or the tests). At the moment, most of the checkpoint tests hang.

Oh, I know what it is… the code doesn't handle the parent hash field  of the genesis block.
The lookup function returns "None", but it should return a hard-coded value instead. Is the parent hash of the genesis block all zeroes? It's not mentioned in the spec.

The lookup function returns "None", but it should return a hard-coded value instead. Is the parent hash of the genesis block all zeroes? It's not mentioned in the spec.

It's "null", which in bitcoin is all zeroes.

I fixed some bugs and got the tests working again.
I want to refactor the code a bit, then I'll submit this PR for review.

I'm happy with the current state of this PR, and I'm ready for a bigger review. I have restructured this code a lot, so it might be easier to review the overall changes. (Rather than each commit.)
There are a bunch of things we could do to improve this code, but I think we have the basic functionality we need to move forward.
I didn't know if I should squash the commits, because we've already done some initial reviews. But we should definitely squash them before merging.

My next step is to create and parse the checkpoint lists (#564).

I've made some of these changes, but I still have a few big refactors and refinements to go.
I'd like to wait until I've finished all the changes for a review.

Note to self:
If we implement Drop on QueuedBlock, then our drop() function gets even simpler.

In this case, the second interval of pending requests will stall until a future invocation of call, even though they are ready now.

Hmm, I see how this could have cause a nasty hang, if the final block is in the second-last interval. We would refuse any more blocks, but never check the final interval.

If we implement Drop on QueuedBlock, then our drop() function gets even simpler.

Turns out we can't implement Drop on QueuedBlock, because tx isn't Copy, and doesn't implement Default.

However, this means that the verifier will only ever process a single interval at a time, regardless of how many blocks are queued. Consider a scenario such as:
|   |   |   |   (checkpoints)
|VVV|   |   |   (previously verified blocks)
|VVV|P P|PPP| (new blocks arrive, are queued pending verification)
|VVV|PPP|PPP| (last block required arrives)
|VVV|VVV|PPP| (only one interval is flushed)

In this case, the second interval of pending requests will stall until a future invocation of call, even though they are ready now.
I think there is a way to prevent this and also simplify the range check logic, by:
1. scanning forward from the beginning of the interval to the first gap:

let mut pending_height = verified_height;
for (&height, _) in queued.range((Excluded(pending_height), Unbounded)) {
    if height != pending_height + 1 {
        break;
    }
    pending_height = height
}
// Now pending_height is the height of the longest unbroken chain
// starting from the verified height.

Done in target_checkpoint_height() in 272a9b27bd59930858bf2ff056b82d2e29137524.

1. finding the highest checkpoint below the `pending_height`:

let checkpoint = checkpoint_list
   .range((Excluded(verified_height), Included(pending_height)))
   .cloned()
   .next_back();

Done in target_checkpoint_height() in 272a9b27bd59930858bf2ff056b82d2e29137524.

1. if that checkpoint exists, processing all pending entries up to that height:

if let Some((checkpoint_height, checkpoint_hash)) = checkpoint {
    let range = (Excluded(verified_height, Included(checkpoint_height));
    // process entries.
}

Done in process_current_checkpoint_range() in 272a9b27bd59930858bf2ff056b82d2e29137524.
There are some edge cases in this new algorithm, which I tried to cover:

verifying checkpoints that are inside the current range

the previous algorithm did not support multi-checkpoint ranges

a queue entry with a side-chain block, but no main chain block

the previous algorithm checked for side-chain blocks before sending block verifications
we need to do this check to maintain our "continuous blocks" state invariant

There's one edge case I didn't handle:

side-chain failure, in a multi-checkpoint range

ideally, we would restart verification with a shorter range
currently, we wait for the next block, which is guaranteed, because we just rejected a block height

Side-chain failures and multi-checkpoint ranges should be rare, so I don't think this is a high priority.

I've completed most of the changes that @hdevalence and @yaahc asked for, except for the mass code movement.
I'd like to do that in a follow-up PR, I'm keeping a list of tasks in #582.

As a follow-on, it might be helpful (after this is merged) for us to go through and add tracing calls so that we can inspect it running.

I opened #611 to add tracing to CheckpointVerifier.
We might also want to add tracing to the other verifiers, but they should have separate tickets. (They aren't as complex yet, so they might be a lower priority.)

Let's wait for CI to pass, then anyone can squash and merge.


Allow getting the token using email and password

The IAuthenticationProvider should be flexible for those scenarios where the user may not login using the username but an email and password.
I know that with the current implementation I can just make the clients send the email in the "username" and it will work but it is not nice for the clients to do that.

The username can be used with an email as well, is up to the implementation validating looking for username, email or other identification


npm run build:package:windows after running thia it shows resource folder is not their

Hi @rakeshsarkar9711 Do you want to build from source or your just testing out the app?
You can download the app from LiSA.
If you want to build it from the code, you can check https://github.com/Cosmicoppai/LiSA#building-from-source
Make sure you are installing all the python dependencies prior running the build command.

@rakeshsarkar9711 Make sure you're in the virtual env.
Also, It would be great if you can provide pip freeze output :)

after this when i open the application the application pages stays blank

@rakeshsarkar9711
Can you provide the output of pip freeze and provide some SS or logs of build, cuz I'm unable to reproduce it.
Make sure to include python & node version along with system specs.

aiobotocore==2.5.4
aiohttp==3.8.5
aioitertools==0.11.0
aiosignal==1.3.1
altgraph==0.17.4
anyio==3.7.1
async-timeout==4.0.3
attrs==23.1.0
beautifulsoup4==4.12.2
boto3==1.28.17
botocore==1.31.17
certifi==2023.7.22
charset-normalizer==3.2.0
click==8.1.7
colorama==0.4.6
exceptiongroup==1.1.3
frozenlist==1.4.0
fsspec==2023.6.0
h11==0.14.0
idna==3.4
iso8601==2.0.0
jmespath==1.0.1
m3u8==3.5.0
multidict==6.0.4
pathvalidate==3.1.0
pefile==2023.2.7
progress==1.6
pyathena==3.0.6
pycryptodome==3.18.0
pyinstaller==5.6.1
pyinstaller-hooks-contrib==2023.8
python-dateutil==2.8.2
pywin32-ctypes==0.2.2
s3fs==2023.6.0
s3transfer==0.6.2
six==1.16.0
sniffio==1.3.0
soupsieve==2.4.1
starlette==0.31.0
tenacity==8.2.3
typing_extensions==4.7.1
urllib3==1.26.16
uvicorn==0.23.2
websockets==11.0.3
wrapt==1.15.0
yarl==1.9.2
py = 3.11.5
node = 30.7.0

Host Name:                 CODEZERO
OS Name:                   Microsoft Windows 11 Home Single Language
OS Version:                10.0.22621 N/A Build 22621
OS Manufacturer:           Microsoft Corporation
OS Configuration:          Standalone Workstation
OS Build Type:             Multiprocessor Free
Registered Owner<EMAIL_ADDRESS>Registered Organization:   N/A
Product ID:                00356-24591-91075-AAOEM
Original Install Date:     8/29/2023, 11:19:08 AM
System Boot Time:          9/27/2023, 12:39:45 AM
System Manufacturer:       LENOVO
System Model:              82NL
System Type:               x64-based PC
Processor(s):              1 Processor(s) Installed.
[01]: Intel64 Family 6 Model 165 Stepping 2 GenuineIntel ~2496 Mhz
BIOS Version:              LENOVO G8CN23WW, 6/7/2023
Windows Directory:         C:\Windows
System Directory:          C:\Windows\system32
Boot Device:               \Device\HarddiskVolume1
System Locale:             en-us;English (United States)
Input Locale:              en-us;English (United States)
Time Zone:                 (UTC+05:30) Chennai, Kolkata, Mumbai, New Delhi
Total Physical Memory:     8,100 MB
Available Physical Memory: 2,121 MB
Virtual Memory: Max Size:  17,828 MB
Virtual Memory: Available: 7,470 MB
Virtual Memory: In Use:    10,358 MB
Page File Location(s):     C:\pagefile.sys
Domain:                    WORKGROUP
Logon Server:              \CODEZERO
Hotfix(s):                 4 Hotfix(s) Installed.
[01]: KB5029921
[02]: KB5030310
[03]: KB5028756
[04]: KB5030508
Network Card(s):           2 NIC(s) Installed.
[01]: Intel(R) Wi-Fi 6 AX201 160MHz
Connection Name: Wi-Fi
DHCP Enabled:    Yes
DHCP Server:     <IP_ADDRESS>
IP address(es)
[01]: <IP_ADDRESS>
[02]: fe80::1435:1ea7:152:f714
[02]: Realtek PCIe GbE Family Controller
Connection Name: Ethernet
Status:          Media disconnected
Hyper-V Requirements:      A hypervisor has been detected. Features required for Hyper-V will not be displayed.

The issue, #44, has been resolved. You can try building the project after pulling the latest changes.
Please note that the watchList and Manga Integration features are still under development.
However, you are welcome to use the other available features :)

HI @Cosmicoppai why no anime is been ready to watch?

@hemanth-kumar-boddeda
It's been a while, we are no longer actively maintaining this repo.
The Site scraper might have broken because of the following reasons:
Site schema changes.
Crawler Blocked due to some reason.
Let's hope, someday we'll able to get some time from our schedule to maintain this project.
In the meantime, Please feel free to dig into issue 😉.

@hemanth-kumar-boddeda Linking the crawler code: https://github.com/Cosmicoppai/LiSA/blob/main/backend/scraper/scraper.py


raftstore: apply early

What have you changed?
A request will go through following process: broadcasting -> write logs -> broadcasting commit -> write commit logs -> apply logs -> calling callbacks. Apparently, we can apply logs and write commit logs at the same time, which should save one write time for each request.
Unresolved Problems:

[x] When it's not gracefully shutdown during the process, it may fail the constraint that apply index <= commit index.

What is the type of the changes?

Improvement

How is the PR tested?

Unit test

Does this PR affect documentation (docs) or should it be mentioned in the release notes?
No.
Does this PR affect tidb-ansible?
No.

/bench

amazing!!!

/bench
workload:
  threads: 512
  prepare: false
  repeat_time: 3
  benchtime: 60

/bench
workload:
  threads: 512
  repeat_time: 3
  benchtime: 60

/bench +ycsb

I meet a panic when I insert another table while sysbench prepare.
process 1: sysbench oltp_update_non_index --config-file=config.40 --threads=16 --tables=16 --table_size=4000000 prepare
process 2: sysbench new_insert --config-file=config.32 --threads=2 --tables=8 --time=500 --table_size=6400000 run

new_insert.lua is similar to oltp_insert.lua but it has different table name.

region 151 is splited out at the time of creating tables of new_insert.

/release make dist_release

/bench +ycsb

/bench

/bench +ycsb

/bench +ycsb

/bench +ycsb

/bench +ycsb

@Little-Wallace @gengliqi @overvenus PTAL

I find the last_commit_index is the latest commit_index written to the disk now.
Why not set last_commit_index to the lastest index of committed entries that have already written to disk because we only send these committed entries to apply fsm for early apply?
If so, the order is applied_index <= last_commit_index <= last_index, commit_index <= last_index.
After restarting, we can just set commit_index = max(commit_index, last_commit_index) to recover all things.
And the logic is exactly the same to the previous logic.
We don't need to worry about applied index is greater than commit_index.
Then the code such as https://github.com/tikv/tikv/blob/a36c8d89e92660d7948f826791db6076a77f97ec/components/raftstore/src/store/peer.rs#L1426-L1431 can be removed.

After restarting, we can just set commit_index = max(commit_index, last_commit_index) to recover all things.

No, it can't. Suppose a log is marked as (term, index). A follower's log can be sequence S1 (3, 4), (3, 5), (3, 6), (3, 7). After leader overwrites follower's log, sequence becomes S2 (3, 4), (4, 5), (4, 6) and commit index is 6. If it loses all the updates and is reverted to S1, setting commit index to 6 unconditionally is wrong.

After restarting, we can just set commit_index = max(commit_index, last_commit_index) to recover all things.

No, it can't. Suppose a log is marked as (term, index). A follower's log can be sequence S1 (3, 4), (3, 5), (3, 6), (3, 7). After leader overwrites follower's log, sequence becomes S2 (3, 4), (4, 5), (4, 6) and commit index is 6. If it loses all the updates and is reverted to S1, setting commit index to 6 unconditionally is wrong.

According to raft Leader Completeness Property, the leader always holds all of the log entries committed in previous terms.

After restarting, we can just set commit_index = max(commit_index, last_commit_index) to recover all things.

No, it can't. Suppose a log is marked as (term, index). A follower's log can be sequence S1 (3, 4), (3, 5), (3, 6), (3, 7). After leader overwrites follower's log, sequence becomes S2 (3, 4), (4, 5), (4, 6) and commit index is 6. If it loses all the updates and is reverted to S1, setting commit index to 6 unconditionally is wrong.

Once entries are committed in memory, it means that they won't be overwrite even if they elect a new leader.

 I think you may misunderstand two points of the case I described.
First, the log is overwritten before they are committed. Hence in sequence S1, the commit index is not larger than 4, in sequence S2, the commit index is 6. There is no commit index descrease, the situation is possible and allowed.
Second, "loses updates and reverted to S1" is totally possible when there are bugs in the file system, disk or even in our code. We have met not just once that commit index or last index is decreased in production system.

Once entries are committed in memory, it means that they won't be overwrite even if they elect a new leader.

@Little-Wallace logs are overwritten before they are committed.

@BusyJay But if  logs are overwritten before they are committed, the last committed entry would have a lager term than the last entry's term, which has been pesistent on disk.

 I think you may misunderstand two points of the case I described.
First, the log is overwritten before they are committed. Hence in sequence S1, the commit index is not larger than 4, in sequence S2, the commit index is 6. There is no commit index descrease, the situation is possible and allowed.
Second, "loses updates and reverted to S1" is totally possible when there are bugs in the file system, disk or even in our code. We have met not just once that commit index or last index is decreased in production system.

Once entries are committed in memory, it means that they won't be overwrite even if they elect a new leader.

@Little-Wallace logs are overwritten before they are committed.

We don't need to write commit_index to disk to ensure these logs are committed.
Only when we know a log is committed, it is impossible to be overwritten to another log.
So it's safe to forward the commit index to the previous commit index which is received before.
If the already saved data lost due to file system bugs, there are too many panic cases. Such as cannot get committed logs for applying, decreasing commit index and so on.

...So it's safe to forward the commit index to the previous commit index which is received before.

No. Moving forward the commit index means to swallow a fatal error and assume everything is OK. Not to say that if all nodes do the same to swallow the errors and getting committed logs lost silently in the end, at lease trying to move forward the commit index on one node is just wrong because commit term is not recorded. Moving it forward means commit index is pointing at an old log. If it becomes leader and try to replicate logs to other followers, data consistency is totally ruined.
Given the complexity and uncertainty introduced by swallowing the error, panic is the correct, clean and safe way.

If the already saved data lost due to file system bugs, there are too many panic cases. Such as cannot get committed logs for applying, decreasing commit index and so on.

Yes. So it's safe to forward the commit index to the previous commit index which is received before.

No. Moving forward the commit index means to swallow a fatal error and assume everything is OK. Not to say that if all nodes do the same to swallow the errors and getting committed logs lost silently in the end, at lease trying to move forward the commit index on one node is just wrong because commit term is not recorded. Moving it forward means commit index is pointing at an old log. If it becomes leader and try to replicate logs to other followers, data consistency is totally ruined.
Given the complexity and uncertainty introduced by swallowing the error, panic is the correct, clean and safe way.

If the already saved data lost due to file system bugs, there are too many panic cases. Such as cannot get committed logs for applying, decreasing commit index and so on.

Yes.

What's the fatal error?
I don't understand why forwarding the commit index to the previous commit index received before can swallow a fatal error.

@Little-Wallace @gengliqi PTAL


warn if kubenetes version isn't supplied its the same as upgrade to s…

…ame version
Related command

az aks upgrade
Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

AKS

@paulgmiller Could you please resolve the conflicts?

Could you please refer to the document Submitting Pull Requests to refine the PR title

Please note that since we will launch the release of this sprint the day after tomorrow. If you cannot address all the comments tomorrow, we will have to postpone the release of this PR to the next sprint (02-07)

Pushed a merge

Running flake8 on modules...
ERROR: /mnt/vss/_work/1/s/src/azure-cli/azure/cli/command_modules/acs/custom.py:2120:1: W293 blank line contains whitespace
/mnt/vss/_work/1/s/src/azure-cli/azure/cli/command_modules/acs/custom.py:2120:1: W293 blank line contains whitespace
2     W293 blank line contains whitespac

@paulgmiller Could you please resolve the code style issue today? If not, we will postpone this PR to the next sprint (02-07)

I ran out of time to fix the whitespace and honestly I'm a little confused by the line numbers
2120 in the difff and locall is
"Node pool {} doesnt exist, use 'aks nodepool list' to get current node pool list".format(nodepool_name))
        which is neither blank nor has  trailing whitepae.

So I just went through all added blank lines and found it.

/azp run

/azp run


Made database accessible through controller to improve tests

This pr seeks to improve testability of napps by making the database accessible through the kytos controller. Making the db accessible through the kytos controller means we can extend the existing controller mock to include a database mock, rather than having to patch the functions used to access the database.

@Ktmi once again, thanks for your initiative with this PR. I mapped the discussion to be continued here https://github.com/kytos-ng/kytos/issues/322
I'll go ahead and close this PR.


fixes #211

Description
What was the problem
Refer #211
How is it resolved?
Tweaked the signature of navigate function to add another paramter
@param isRemoveOp a flag that tells whether the operation that invoked is remove or not

Using this additional piece of information, the navigate function can take a call to exit early from the function for remove operation if the path doesn't exist
    if (!schema[subPath] && isRemoveOp)
        return false;
    schema = schema[subPath] || (schema[subPath] = {});

How can we test the change?
Refer #211 description
Changes include

[x] Bugfix (non-breaking change that solves an issue)

Closes issue(s)
Resolve #211
Checklist

[x] I have tested this code
[x] I have added unit test to cover this code
[x] I have updated the Readme
[x] I have followed the contributing guide

@asutosh97 the new version is released.


Revert "Update semver validation regex to be more flexible and allow the prerelease label to contain a "feature." string"

Now that we have published the docs for 1.1.0-pnp.beta.2, reverts the temporary workaround from Azure/azure-sdk-for-c#1497
If we want to support <feature>.<beta>.<rev> versioning long term, we should follow the eng/common workflow and make such a fix again using that approach, starting in azure-sdk-tools.

@weshaggard I think we are going to need to revise our versioning guidelines to accommodate scenarios where we have a 1:1 mapping between package and repository. The 1.0.0-beta.1 format doesn't leave room for teams to stabilize new features on a branch and ship previews to customers.
We wouldn't want to merge a feature onto master until we are confident that it wouldn't stop subsequent releases from the master branch (the alternative is long lived diverging servicing branches which is probably less desirable).
We are likely to have the same issue with the iOS repository.

Thanks for reverting these as these changes are definitely incomplete. There is a teams conversation going on about potential solutions for this issue but until we come up with that plan we should avoid release any versions that don't match our versioning guidelines otherwise we will be fighting a lot of different things not just these 2 changes.


Unknown issue

Hi, I have recently install latest version of PS 2021(22.3.1) and copy the plugin to plugin folder as usual
But now I'm having issue with the images and I'm not able to edit/make change of it. The transparency has gone or having weird colors/texture. However when I opened the images in chrome or other image viewers, it seem fine.
I have tried re-install PS but its not working. It's working fine in previous version.
Here's what it look like.

It should be image of crosshair and having transparent background.
Image are working fine in the program or view in other apps. ignore the white background as I'm open it in chrome.

Thank you for your report.
Could you provide a few more details:

Your operating system version
The problematic source image
The exact steps to reproduce the issue

On my side, with Photoshop 22.4.2 Release and WebPShop 0.3.3 on Windows Server 2016, creating a 210x150 px transparent image, saving it as WebP and opening it, no problem occurs.


Add Raspberry Pi/ARM support

This is something that is close, but I've intentionally put on the back-burner. Why? Because the x86 version of Solar2DTux needs to be released officially on GitHub first. No point adding more complexity to something that isn't release worthy.
We're fast approaching the first official release of Solar2DTux though, after a long period of public beta testing on discord.
As soon as the official release is out, I will go back to getting this done.

@DannyGlover This is VERY interesting if it ever materializes.
But let me point that Linux != (Linux + X11 Desktop). Some of us (in fact ALL game-oriented distros on the Raspberry Pi) use Linux without an X11 server.
For that, SDL2 is the solution: if the engine uses SDL2 and lets SDL2 initialize it's window, audio and input, you will see this engine running literally everywhere.
So, does Solar2DTux run on SDL2?

@DannyGlover This is VERY interesting if it ever materializes.
But let me point that Linux != (Linux + X11 Desktop). Some of us (in fact ALL game-oriented distros on the Raspberry Pi) use Linux without an X11 server.
For that, SDL2 is the solution: if the engine uses SDL2 and lets SDL2 initialize it's window, audio and input, you will see this engine running literally everywhere, X11 or not.
So, does Solar2DTux run on SDL2?

Hello.
Solar2DTux currently uses WxWidgets. Gutting that out is a non trivial task.
I am not fully decided on if WxWidgets will be the backend for the long haul or not, but there are no plans to change backends until Tux gets to complete feature parity with its Windows/MacOS counterparts.
Thanks for the suggestion.


Do capabilities support key rotation?

Should invoker be limited to DIDs or DID_URLS ?
See https://github.com/decentralized-identity/confidential-storage/pull/142/files#r541971789

Discussed on July 22, 2021 WG call:

Action item: change the example so the question doesn't come up.
Use the term "controller" instead of "invoker"
The value of the controller property should just be a DID.
Should it be a did:key did? -> Use did:example
Concerns about using rotating keys from OCAP community

Proposal: leave the ability to do key rotation (by specifying DIDs), until concerns are raised from OCAP community

Clarify that we're only using DIDs here for the key rotation ability.
Raise the possible concerns against key rotation in the ZCAP spec.

Addressed by PR https://github.com/decentralized-identity/edv-spec/pull/75 (invoker changed to controller, using a DID for controller)


You cannot use the same root element (body)

Using karma I always get this error:
Setup failed on Works as link when popups disabled: Assertion Failed: You cannot use the same root element (body) multiple times in an Ember.Application

I don't know exactly but seems like that's because karma do run many tests in the same time. So module setup() maybe called twise before module teardown() gets called.

this seems no longer relevant, closing due to age


Add endsentence linter

This adds a linter that checks if the documentation ends with a period. Since
this is a very small check and most people don't care about details like this,
it is disabled as default.
I'm not sure if you want this or not but I figured I'll give it a shot.
Thanks!

I think this is of too-narrow use to include TBH. Thanks though.

Thanks! 👍


Not compatible with Expo SDK 52

Describe the bug
When you upgrade to Expo SDK 52 this module is not working anymore. When you start the app, it shows the error Invariant Violation: Tried to register two views with the same name RNSScreen
To Reproduce
Steps to reproduce the behavior:

Create a project with create-expo-app
Install and implement react-native-magic-modal
Start the app and open it with Expo Go App
See error

Expected behavior
The app should start without the error message and when you execute magicModal.show, it should display the modal.
Dependencies
"@expo/vector-icons": "^14.0.2",
"@react-navigation/bottom-tabs": "^7.0.0",
"@react-navigation/native": "^7.0.0",
"expo": "~52.0.18",
"expo-blur": "~14.0.1",
"expo-constants": "~17.0.3",
"expo-font": "~13.0.1",
"expo-haptics": "~14.0.0",
"expo-linking": "~7.0.3",
"expo-router": "4.0.11",
"expo-splash-screen": "~0.29.18",
"expo-status-bar": "~2.0.0",
"expo-symbols": "~0.2.0",
"expo-system-ui": "~4.0.6",
"expo-web-browser": "~14.0.1",
"react": "18.3.1",
"react-dom": "18.3.1",
"react-native": "0.76.5",
"react-native-gesture-handler": "~2.20.2",
"react-native-magic-modal": "^5.1.20",
"react-native-reanimated": "~3.16.1",
"react-native-safe-area-context": "4.12.0",
"react-native-screens": "~4.1.0",
"react-native-web": "~0.19.13",
"react-native-webview": "13.12.5"

Exact same issue in my project after adding
import { MagicModalPortal } from "react-native-magic-modal";
export default function TabTwoScreen() {
// Render UI
return (

);
}

Thanks for reporting! Can you install react-native-magic-modal@experimental and see if it solves your issue?

Reopening to keep track.

Thanks for reporting! Can you install react-native-magic-modal@latest and see if it solves your issue?

Hello, I can confirm that "react-native-magic-modal@experimental" resolved the issue in my project.
Thanks

Nice! Closing. Thanks - feel free to use latest as the fix is already merged.


update the README.md to 0.2.1

pls rebase

rebased

Oh, I guess these changes will be out of date as soon as we merge this pr #171

Ideally, we would setup a frontpage public server and use that in the README.md too

Something else is I'm not sure we should include the whole generated code. Maybe just a small sample to explain what the compiler is doing under the hood but having all these lines seems overkill.

Maybe link to generated code instead
On Feb 10, 2017 2:50 AM, "Martin Bonnin"<EMAIL_ADDRESS>wrote:

Something else is I'm not sure we should include the whole generated code.
Maybe just a small sample to explain what the compiler is doing under the
hood but having all these lines seems overkill.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/apollographql/apollo-android/pull/176#issuecomment-278879546,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEUX3CkuIn8jpcu5LbuE7YjiEd0VXgD6ks5rbBa_gaJpZM4L8cIi
.

Yeah, generated code is pretty much verbose, I guess we need to cut it.
We can't put a link as it's generated code and not under the git

Hey so my last pr was more of a bandaid. Could you rebase this as you had some great changes too.

I just forced pushed a new version (not sure this is the correct way to do...). It adds the "custom scalar types" and "generateClasses" section. @digitalbuddha  I think they were the only things not in your pr already.


Toast Notifications

Changed alerts to toast notifications that disappear on their own. Also added ability to play and pause track

Added 2 small libraries, remember to npm install


CIMPL6 Import

Changes needed to support CIMPL6.  The two most significant changes to the CIMPL6 model are:

Replaced code, Coding, and CodeableConcept with new concept primitive
Removed RefValue in favor of using a field's "entry-ness" to determine if it should be a reference

To support the new concept primitive (and removal of the other code-ish classes), I created a Concept class that is output with (and used by) the generated code.
All unit tests should pass.  For information on running w/ other required modules and CIMPL6 source see:
https://github.com/standardhealth/shr-cli/pull/207
Marking as a DRAFT PR because all versions will need to be bumped to non-beta versions upon approval.

@dehall -- I've addressed your comments (well, the short-term ones anyway).  We tried to shield most of the changes from the rest of the framework, but the replacement of CodeableConcept, Coding, and code, as well as the removal of RefValue, was pretty much impossible to happen transparently.  So... here we are.
It's worth noting that Mark has changed a lot of the base classes and such as well, so I don't think the grammar change is going to be the only challenge...


Add row based virtualization to pivot table

This PR adds row based virtualization to the pivot table. Column virtualization will be added separately.
TanStack's virtualization library and flexRenderer are not performant enough for our use case and will need to be replaced.

@djbarnwal Responded to your questions and updated the comments.


Remove deprecated methods from JodaCompatibleZonedDateTime which are called by scripts

Signed-off-by: Andriy Redko<EMAIL_ADDRESS>Description
There are number of deprecated methods in JodaCompatibleZonedDateTime which are still available in scripts. Those should be removed since in some cases they pose significant performance overhead (due to deprecation warnings).
Issues Resolved
Closes https://github.com/opensearch-project/OpenSearch/issues/3156 and https://github.com/opensearch-project/OpenSearch/issues/3345
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

start gradle check

@reta precommit is failing because of a forbidden API:
Forbidden method invocation: java.time.format.DateTimeFormatter#ofPattern(java.lang.String) [Uses default locale]

  in org.opensearch.script.JodaCompatibleZonedDateTimeTests (JodaCompatibleZonedDateTimeTests.java:191)

@kavilla its fair. Ideally deprecated method usage should have been migrated. But we agree it was not the right practice we removed it after launching RC1.
@reta we'll go revert this in 2.0, 2.x line and let it go out in 3.x.
Let me know if you strongly feel otherwise.

@saratvemulapalli sounds good, sorry for that @kavilla

@kavilla I will create an issue on OpenSearch Dashboards, the deprecated methods introduce significant overhead because of the deprecation logger (vs actual useful work the methods do), we have to migrate the scripts to use the non-deprecated variants.


Any release date for 2.0.0?

Hi! First of all, thanks for this module, it saves me hours of run time by running on 25000 instances in a breeze with multithreading, starting job would've cost 600 ms per instance while your thread jobs takes 25 ms. I love it!
I have seen in https://github.com/PaulHigin/PSThreadJob/pull/31 that the 2.0.0 will be coming on Powershell Gallery soon, and I would need that version for the bump on the ThrottleLimit range (I would use more than 50, basically https://github.com/PaulHigin/PSThreadJob/issues/24). Therefore, I'm wondering when is it going to be up on Powershell Gallery?
Thanks in advance!

It should be soon.  Possibly this week but certainly next week.  I have spent a lot of time recently refactoring the repo to support a build pipeline, so that I can publish a signed module.

ThreadJob 2.0.0 has been published.
https://www.powershellgallery.com/packages/ThreadJob/2.0.0


feat: add replicatedhq/kots

replicatedhq/kots

https://github.com/replicatedhq/kots
KOTS provides the framework, tools and integrations that enable the delivery and management of 3rd-party Kubernetes applications, a.k.a. Kubernetes Off-The-Shelf (KOTS) Software

Thank you for your contribution!

v0.14.1 has been released 🎉
Thank you for your contribution!


Typescript definitions

I'd love to include this in a typescript project, any chance for typescript definition support?

We will be looking to do this in milestone 2 (or will offer a bounty for someone who would like to offer this service!).


themeimporter: fix not being able to init with theme

Fixes the issue where a user cannot run a jambo init with the
theme specified.
The usage of the theme importer as a constructor was originally
incorrect in the repository scaffolder; the change is made here
to use it correctly.
TEST=manual
Created a new directory locally, cd-ed into it, and ran
jambo init --theme answers-hitchhiker-theme. Saw that the
jambo repo was initialized correctly and that the theme was
also imported into the repo.

We should make a hotfix branch for Jambo. That branch will be the target of this PR. Under gitflow, we don't commit directly to master unless it's a new release.

Done!


can't change mongodb port in docker compose

I think you need to use docker networks.

Honestly I don't use docker a lot myself. I set that stuff up back in the day but am not super familiar with it and it's not high on my priority list but I would definitely accept a PR with some updates if you know what to do.

I got it working but the app doesn’t allow me to upload a video. It just eventually returns a 404 for the video
Anthony
http://profullstack.com

On Dec 30, 2019, at 12:04 PM, mayeaux<EMAIL_ADDRESS>wrote:

Honestly I don't use docker a lot myself. I set that stuff up back in the day but am not super familiar with it and it's not high on my priority list but I would definitely accept a PR with some updates if you know what to do.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Do you see anything come through the logs?

i see a lot of errors in devtools. nothing on the node side

Can you check the /uploads folder and see if everything is turning up there?

one small showed up there. but i couldn't figure out how to open it in the
browser.
On Mon, Dec 30, 2019 at 4:05 PM mayeaux<EMAIL_ADDRESS>wrote:

Can you check the /uploads folder and see if everything is turning up
there?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mayeaux/nodetube/issues/33?email_source=notifications&email_token=AAAGV5PFVNUH4ZFS4ROCYDTQ3KED7A5CNFSM4KBKLCVKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEH3PEJY#issuecomment-569831975,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAGV5NUQZ4OSDDZOWKVGX3Q3KED7ANCNFSM4KBKLCVA
.

--
Anthony Ettinger
http://anthony.ettinger.name
<EMAIL_ADDRESS>+1<PHONE_NUMBER>

Basically the folder that is loaded via process.env.SAVE_AND_SERVE_FILES is served through express.static, line on 195 app.js . So if there is an ./uploads folder with those files they should be served up by express


Variables in SVG Link

Hello,
I'd like to be able to use variables in the SVG link to build a more dynamic panel.
For instance, I have a Grafana variable of $device_type, I'd like to be able to specify the URL in the SVG field using that variable.
For instance, I'd want the following to work:
https://example.com/${manufacturer}/${device_type}.svg

Sure, I'll get that added. I only added it to the yaml links originally as my svg's are very 'data-query' specific.

this is actually proving a bit more involved than I'd hoped so it won't make the next build. but hopefully the one after that.

change landed so will be in next build

built into ver 1.12.0 with grafana for review


Remove RBK branding in the chaise/viewer app

We have RBK branding leaking into the chaise/viewer. This needs to be removed.
https://github.com/informatics-isi-edu/chaise/blob/recordset/viewer/index.html.in#L4

@robes I changed the title to just "Image Viewer"; will we need a dynamic title like "Image Viewer | [consortium name here]"?

We have a config variable for the title tag for other index.html.in files (using chaise-config.js) so I can ask Serban to do the same for this file. So "Image Viewer" as a default is fine.

Can you point Jessie to it?

Serban used this in scripts/controller/ermrestInitController.js
if (chaiseConfig['headTitle'] !== undefined) {
	document.getElementsByTagName("head")[0].getElementsByTagName("title")[0].innerHTML = chaiseConfig['headTitle'];
}

And this in record/assets/javascripts/app.js
if (chaiseConfig['headTitle'] !== undefined) {
var title = document.createElement("title");
title.innerHTML = chaiseConfig['headTitle'];
document.getElementsByTagName("head")[0].appendChild(title);
}

Awesome, well I've added the snippet in the Viewer and Data Entry apps.


Logging

I can't turn off logging of SDWebImage without modifying the code....

@troypayne are you talking about the prefetcher?

@mythodeia yes

check this https://github.com/rs/SDWebImage/pull/1106

Hmm perhaps it works in theory? I tried doing #define SD_LOG_NONE 1 in my bridge.h file (i'm using swift) above my import of the prefetcher and i'm still getting the logs

i do not know if swift is causing this or not but since the actual logging is taking place in 3 lines i think you can manually comment them for now


Unable to add new package after creating a project with "create-turbo"

What version of Turborepo are you using?
1.0.7
Describe the Bug
Can't create a new package and import it, seems like an error reading a tsx file?

Expected Behavior
Should be able to create another package and import it easily?
Am I missing some steps? there is no documentation on how to create a new package in the monorepo so I expected it to be as simple as that.
To Reproduce
After using npx create-turbo@latest, to create a new monorepo, I simply try to create a new package.

I copied the "ui" package and named the copy "btn".
changed the component to have different text
changed the package name in the packages/btn/package.json to "btn".
added "btn" as a dependency in apps/docs/package.json
change the import of the button in apps/docs/pages/index.tsx from 'ui' to 'btn'.
run yarn again to link my module
run yarn dev/build and get this error

now webpack (which is hidden by turbo to next somewhere) cannot figure out how to import the file.

Ok found it, it's an issue with next that requires an extra step to transpile dependencies, not with turbo...
Adding this line in next.config helps
"const withTM = require("next-transpile-modules")(["ui", "btn"]);"

it's strange that "ui" works out of the box but not a new package. Do you know why @liron-navon ?

Oh my bad, "ui" is included in the boilerplate


Get the locations of each object

Is there any C++ api to get the locations (internal state) of each object (e.g. coin, agent, saw  in coinrun)?
Thank you.

I'm afraid there is no API for that.  I experimented with a "save state" feature, but it serialized the state into a string that can only be interpreted directly by the C++ code.
If you want to do this, you can modify the "info_space" to put information in the info dictionary returned by the gym API.  There is actually already some game information in this dictionary: https://github.com/openai/procgen/blob/master/procgen/src/vecgame.cpp#L300
Once you define an info space there, you can put information into it like level_complete here: https://github.com/openai/procgen/blob/master/procgen/src/game.cpp#L154
For something like coinrun, you'd probably want to do that in https://github.com/openai/procgen/blob/master/procgen/src/games/coinrun.cpp#L492, so assuming you've defined a space like:
    {
        struct libenv_space s;
        strcpy(s.name, "coin_x");
        s.type = LIBENV_SPACE_TYPE_BOX;
        s.dtype = LIBENV_DTYPE_FLOAT32;
        s.shape[0] = 1;
        s.ndim = 1,
        s.low.float32 = -1e9f;
        s.high.float32 = 1e9f;
        info_spaces.push_back(s);
    }

You'd probably want something like
*(float*)(info_bufs[3]) = (float)(coin_x);

Note that the index 3 there is the index of the info space in the info_spaces vector, so you'll probably want to define constants so those don't get confusing.
Also I didn't see the coin_x value being saved, so you'd have to save it at https://github.com/openai/procgen/blob/master/procgen/src/games/coinrun.cpp#L405

Thanks a lot for your help !

@kaixin96 Did you ever end up implementing this? If so, would you mind uploading the code somewhere (e.g. in a fork)? I need the underlying state for a bunch of different environments in the python API and this seems like the first step.


Meow.

Moved entity spawning and related code out to its own class.
Added getInstance static method to Vanish, so we can easily get the plugin instance.
Added ocelot entity spawning effect.
Hopefully followed existing code formatting well enough (let me know if not...)

Remind me about this later! I completely forgot.

Should I like, squash this or something?

Thanks!
https://github.com/KittehOrg/Vanish/commit/ebd97f4dd5905d6a22544122e6711aafd3bf82f9


Added examples in readme for azurestackhci

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
API changes are not detected in this pull request.

@NikitaSingh-ms:
Our automation is identifying you as a community contributor.  Please review the Azure SDK onboarding documentation and ensure that your GitHub organization membership and permissions are correct.

Azure SDK onboarding (Microsoft internal)
Azure SDK onboarding assistance (Microsoft internal)

Please also be sure to add yourself to CODEOWNERS for this library, if you will be maintaining it going forward.