undefined reference to `pthreadpool_compute_2d_tiled'

When I compile caffe2 using make, I got the following error (ubuntu 16.04):
libCaffe2_CPU.so: undefined reference to `pthreadpool_compute_2d_tiled'
Here is the summary of configuration in my system:
-- ******** Summary ********
-- General:
--   Git version           : v0.7.0-47-g964ffa9-dirty
--   System                : Linux
--   C++ compiler          : /usr/bin/c++
--   Protobuf compiler     : /usr/local/bin/protoc
--   CXX flags             :  -fopenmp -std=c++11 -fPIC -Ofast -ftree-vectorize -ffast-math -funsafe-math-optimizations -ftree-vectorize -fomit-frame-pointer -Wno-narrowing
--   Build type            : Release
--   Compile definitions   : CAFFE2_USE_EIGEN_FOR_BLAS;CAFFE2_USE_GOOGLE_GLOG;CAFFE2_USE_GFLAGS;EIGEN_MPL2_ONLY;CAFFE2_FORCE_FALLBACK_CUDA_MPI;CAFFE2_NO_BUILTIN_CPU_SUPPORTS
--   BUILD_SHARED_LIBS     : ON
--   BUILD_PYTHON          : ON
--     Python version      : 2.7.12
--     Python library      : /usr/lib/aarch64-linux-gnu/libpython2.7.so
--   USE_CUDA              : OFF
--   USE_NERVANA_GPU       : OFF
--   USE_GLOG              : ON
--   USE_GFLAGS            : ON
--   USE_LMDB              : ON
--     LMDB version        : 0.9.17
--   USE_LEVELDB           : ON
--     LevelDB version     : 1.18
--     Snappy version      : 1.1.3
--   USE_OPENCV            : OFF
--   USE_ZMQ               : OFF
--   USE_ROCKSDB           : OFF
--   USE_MPI               : ON
--   USE_NCCL              : OFF
--   USE_NNPACK            : ON
--   USE_OPENMP            : ON
--   USE_REDIS             : OFF
--   USE_GLOO              : OFF
-- Configuring done
Any idea of how to resolve it?
Thanks,

I'm having a similar error on Arch Linux x86_64 when using nnpack from system libraries (external nnpack installation):
libCaffe2_CPU.so: undefined reference to `pthreadpool_compute_1d_tiled'
libCaffe2_CPU.so: undefined reference to `pthreadpool_compute_2d_tiled'
libCaffe2_CPU.so: undefined reference to `pthreadpool_compute_2d'
libCaffe2_CPU.so: undefined reference to `pthreadpool_create'
collect2: error: ld returned 1 exit status

My external nnpack and pthreadpool are compiled with -fPIC in cflags and cxxflags.
Interestingly, when uninstalling nnpack and letting caffe2 to build nnpack from source during the compilation, I have no error.

@kaishijeng @dbermond
I modify Depencies.cmake
add :+1: list(APPEND NNPACK_INCLUDE_DIRS "/usr/local/include")
list(APPEND NNPACK_LIBRARIES "/usr/local/lib")
#set(NNPACK_LIBRARIES /usr/local/lib/libnnpack.a /usr/local/lib/libpthreadpool.a)
link_libraries("/usr/local/lib/libnnpack.a" "/usr/local/lib/libpthreadpool.a")
this is ok

@zfc929 Thank you for sharing this solution. It works.
The relevant part though is setting the NNPACK_LIBRARIES cmake internal variable. It will define which nnpack libraries will be used when linking:
https://github.com/caffe2/caffe2/blob/05ba3e954cd86b7c1cfbb001a675a46412948adb/cmake/Dependencies.cmake#L56
libpthreadpool is needed during caffe2 linking stage. This can be confirmed by taking a look at the caffe2 building process when the user chooses to build nnpack from source:
https://github.com/caffe2/caffe2/blob/05ba3e954cd86b7c1cfbb001a675a46412948adb/cmake/External/nnpack.cmake#L151
This libpthreadpool linking part is missing from cmake when the user chooses to use an external nnpack installation (from system libraries):
https://github.com/caffe2/caffe2/blob/05ba3e954cd86b7c1cfbb001a675a46412948adb/cmake/Modules/FindNNPACK.cmake#L28

@kaishijeng
This issue was fixed by commit f070eb99a4a36f12abbdb9af98bb68ff485c840b. At least for me it's fixed.
I think it can be closed.

Thanks for confirming @dbermond.


Add reporter feature

I change the structure of ParseResult.
This is big change.
And, I fix some parsers and add.
I will include more changes.  However, there is very long term, so, I want to update version.

status ok


An suggestion

Can you make the digital demo room open source on github?

Going to ping @cfgammie for this one, since the demo room predates me by... more than a decade, it looks like.  Also transferring this since it isn't really a KHARMA issue, specifically.


Any progress here?

Hello!
I would love to see this project up and running! I was wondering if you were still planning to create this?

Started to work on the preprocessor implementation a week or so ago, here's the result so far (a comparison of output between it and GCC's):


Bugfix: Scheduler crashes for some configs

When the 'disable_hard_timeout' is set but 'disable_failures' is not.
The fix is simply to follow the "good" coding practice to not allow
IntParameters to have a None-value as default.

lgtm


Suggestions: Add Role ForceJoin To Allow Spectators

I play on a large server that often has many people joining as a spectator to the game and to listen to all the discussion. Would it be possible, now that you have ,forcejoin and ,joinall to have the ,joinall specifically allow for a Role Name. This way spectators aren't lumped into the ,joinall command.

If there isn't a way to ,joinall a role, maybe a link could be generated for each player to click and it automatically add them to the game and allow them to emoji (as I have seen this suggestion) click to select their color. This way Admins can build a channel that only a specific role can see to see the link. It would need to work where the player role does NOT have access to message as we wouldn't want them interfering with the BOT commands.

This can definitely be implemented. How about an argument in the joinall command, like ,joinall [role]?


Newer Versions of Beat Saber

This isn't an issue but more of a question. I was wondering if there are any plans on updating this mod so it is compatible with new versions of Beat Saber? I know that most other quest mods have or are in the process of updating to 1.19.1.
Thank you for reading my question

Hello BlazeFolf,
Our Multiplayer mods are being updated, it just takes time due to the frequent changes in the games multiplayer code and the pc multiplayer mods counterpart.
So please bear with us as the BeatTogether Team will do it's best to get the multiplayer mods updated as fast as we can while ensuring the best possibly stability.

If my question came across as me accusing you of not working on anything I am sorry. I just didn't see anything suggesting otherwise. I just wanted to confirm with someone that things are happening. I do look forward to any update you have planned. And I have no problem waiting as, usually, the longer it takes for something to be made the more care and detail went into it.
Thank you for your time.

i see commits on this project in my feed all the time

The reason not much is happening on this repo specifically is that like on PC things are being seperated into another mod with this one mainly becoming an add-on to that in the future. You'll see the main stuff on the MultiplayerCore.Quest repo

Okay, thank you for the information.

Has been updated

oops


Inspector tweaks for CSM and removing an implicit varying from shadowsFragmentFunctions.fx

Allow for editing shadow max Z in CSM inspector when camera is using an infinite far plane (i.e. camera.maxZ = 0)
Add 4K shadow map option to CSM inspector
Replace implicit usages vPositionW with a function parameter (worldPos) which allows shadowsFragmentFunctions.fx to be included in shaders that don't have a vPositionW varying.
Other minor cleanup in shadowsFragmentFunctions.fx

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

Snapshot stored with reference name:
refs/pull/14376/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14376/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14376/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14376/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14376/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14376/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14376/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14376/merge#BCU1XR#0


Sc-4587 : Logout feature Implementation

Type of change

[x] bug fix
[x] new feature
[ ] documentation
[ ] other (describe)

Scope of changes
Sc-4587: implement the logout feature but for now we don't handle the expiration date. Also, work on a few improvements of not found component
Acceptance criteria
If you already have an account, please log in by going to /auth/login or register by going to /auth/register.
Open the console and check the cookie in the application tab.
You can see that the user cookie is being deleted when you open the page auth/logout. However, in reality, we will call the method auth0Logout of the hook useCustomAuth, which will redirect the user to auth0, and then log them out once everything has been completed.
Make sure the variable environment is set. The .env.sample file contains all the auth0 test environments that can be used.
Author checklist

[ ] I have manually tested the change and/or added automation in the form of unit tests or integration tests
[ ]  I have updated the dependencies list
[ ]  I have recompiled and included new protocol buffers to reflect changes I made
[ ]  I have added new test fixtures as needed to support added tests
[x]  I have moved the associated ShortCut or Jira ticket to "Ready for Review"

Reviewer(s) checklist

[ ] Any new user-facing content that has been added for this PR has been QA'ed to ensure correct grammar, spelling, and understandability.

Codecov Report

Merging #514 (91a63b4) into main (82fb4e2) will increase coverage by 2.39%.
The diff coverage is n/a.

:exclamation: Current head 91a63b4 differs from pull request most recent head e87be7a. Consider uploading reports for the commit e87be7a to get more accurate results

@@            Coverage Diff             @@
##             main     #514      +/-   ##
==========================================
+ Coverage   46.89%   49.29%   +2.39%
==========================================
  Files         234       82     -152
  Lines       11701     9336    -2365
  Branches      608        0     -608
==========================================
- Hits         5487     4602     -885
+ Misses       5249     3942    -1307
+ Partials      965      792     -173

Impacted Files
Coverage Δ

...com/trisacrypto/directory/pkg/bff/config/config.go
66.66% <0.00%> (-7.41%)
:arrow_down:

...github.com/trisacrypto/directory/pkg/bff/server.go
67.15% <0.00%> (-3.58%)
:arrow_down:

web/gds-admin-ui/src/redux/sagas.js

web/gds-admin-ui/src/hooks/useUser.js

...sicDetails/components/NameIdentifiersFieldArray.js

.../gds-admin-ui/src/components/AddressTypeOptions.js

...tails/components/TrisaImplementationDetailsForm.js

web/gds-admin-ui/src/pages/app/dashboard/Status.js

web/gds-admin-ui/src/helpers/api/utils.js

web/gds-admin-ui/src/layouts/Footer.js

... and 144 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 82fb4e2...e87be7a. Read the comment docs.


DeleteVariable

Hello,
Thanks for the recent update.
I've noticed this earlier too but I forgot to report/ask.
Cloud.Storage.DeleteVariable("key");
This doesn't seems to work, although Cloud.Storage.DeleteAll () works ok. Is this a bug or am I doing something wrong?
--
Environment

CloudOnce v2.4.0
Unity3D v5.6.3
Build platform: Editor

After a quick test, it appears to be working (at least in the Unity editor). Perhaps we are expecting the method to do different things?
Could you describe the expected behavior, and what specifically is not working?

Thanks for the reply.
This is a clean project just with cloudonce.
I have currency int var "coins", it's set to 3 (both localy & on cloud) and there is a text on screen in Update() which shows current value.
When I call Cloud.Storage.DeleteAll (), it's value is set to 0. and next time I run the app (after removing it first) it's still 0.
When I call Cloud.Storage.DeleteVariable("coins"), value on screen stays at 3 and  also next time I run the app (after removing it first) it's still 3 so the key/value is still there.
Could it be specific to this value type?

Ah, I see. The problem there is that the DeleteVariable method is not intended to be used with a variable that is in use. I can see that this is not very clear. It's meant to be used to remove specific unused data in existing cloud data, like if you stop using a key, and you want to remove it from your users data.
It sounds like the method you want is the PersistentCurrency.Reset method. However this can only really be accessed if you set up your cloud variables manually (described in the Advanced implementation section of the Getting Started guide).
To make this more practical I can add a ResetVariable("key") method to Cloud.Storage. Does this sound like it would solve your issue?
Btw the reason DeleteAll worked, is that that method also resets all variables that are in use.

Ah thanks for the explanation.
Yes, that would solve it but nothing urgent, I can also use the reset method.
Btw, on another subject.
As my next game will support tvos I'm currently checking if enabling tvos something I can do. I've added UNITY_TVOS everywhere it has UNITY_IOS and enabled plugins for tvos. So far it seems to work, I've got game center login dislayed and it seems to load cloud values. But it cannot read from existing ios key.
What I'd like to ask is, are there any ios specific sections in the plugin files? (..m)

Great, then I'll add it the next time I'm working on the plug-in. I'll leave this issue open to remind me.
To my knowledge, nothing in the plugin files should be iOS specific. It would be cool though if you manage to get it working!

That would be great, thanks.
And it's working on Apply TV :) I can confirm cloud save is working as expected and KVS syncs between iOS & tvOS both ways. Gamecenter login also works but I haven't tested all the functions yet such as player info, leaderboard overlay, etc.
As for the hack, all I did was:

Search & replace "UNITY_IOS" => "(UNITY_IOS || UNITY_TVOS)"
Mark all 4 files as TVOS compatible in unity file settings.

I'll update the post once I'm confident there are no issues.

That's awesome, thank you for the update!

it works! :) Closing this as I'll post details in other Apple TV issue...


Update troubleshooting links to docs.ros.org

This updates the two troubleshooting links to docs.ros.org. The redirection from index.ros.org to docs.ros.org didn't keep the #tag.
I also used ros_environment to be able to link to the right version of the docs, but if it's too overkill I can remove the second commit.
Signed-off-by: Christophe Bedard<EMAIL_ADDRESS>

@christophebedard I think our conversation kind of drifted above.  Since there doesn't seem to be strong opinion for or against the exec_depend, my suggestion is that we omit it for now.  If this turns out to be a problem later, we can always add it back in.  Does that make sense?

Since there doesn't seem to be strong opinion for or against the exec_depend, my suggestion is that we omit it for now. If this turns out to be a problem later, we can always add it back in. Does that make sense?

Yep! I removed the exec_depend in a20798392c61447dbaedae0154781ea37e6ce9ac

I'll suggest to run CI on this, just to make sure that there are no tests that expected the old string somehow.  Otherwise this looks good to me.

Test failures all seem to be unrelated.

Yeah, those are in the nightlies and are fixed elsewhere.

Linux
Linux-aarch64
macOS
Windows


Feat/cli/express

Summary of change
Add linter. Remove unused deps. Use vite-node to serve
Test plan

[x] If added a new boilerplate

I tested the new boilerplate by running the CLI locally
I tested that the boilerplate is created and works correctly when using command line flags (--recipe=... for example)

[ ] If added a new recipe, frontend or backend

I tested that the newly added option is usable by passing command line flags (--recipe=... for example)

Checklist for important updates

[x] Changelog has been updated
[x] Changes to the version if needed

In package.json
In package-lock.json
In lib/ts/version.ts

[x] Had run npm run build-pretty
[x] Had installed and ran the pre-commit hook
[ ] If added a new recipe, I also modified types to include the new recipe in Recipe and allRecipes
[ ] If added a new frontend, I also modified types to include the new frontend in SupportedFrontends and allFrontends if required
[ ] If added a new backend, I also modified types to include the new backend in SupportedBackends and allBackends

Do you want me to squash any (or all) of these commits?


add package_experiments.js and modify the gruntfile.js

added package_experiments.js  to handle the Petri experiments from the project.

In general I think we should have just one task which is responsible for validating the experiments and merging them into js and json files. I don't see much value in the merge-json task and I think both tasks should be merged into a single task. Also, I don't think all this code should be inside wix-gruntfile - it should be exported to a grunt plugin (like https://github.com/shahata/grunt-json-angular-translate) and it should obviously have tests which are currently missing.


Fix a typo on line 70

Proposed Semver Increment Bump: [MAJOR/MINOR/PATCH]
None
Changes Proposed in this Pull Request (List new items in CHANGELOG.MD)
Changed unsuported to unsupported

Mind changing the version increment to none and updating changelog.MD

Gotcha

I think we're good to merge


Document MSRV policy

Currently the goal is:

1.x will have the same MSRV.
When bumping the MSRV we bump 1.x to 1.x+1

One thing to consider is dependencies and their MSRV bumps. For I think we should just describe the issue in the policy and see if it becomes a problem and tackle it then (similar to what we did with the log crate in v0.8).
Related https://github.com/tokio-rs/mio/issues/1732

One thing to consider is dependencies and their MSRV bumps.

Tokio's current policy here is that for the latest version, we require all dependencies to have a matching MSRV, and we will remove the dependency or bump the MSRV if a dependency increases it past that.
Tokio's current policy for LTS releases is that we try to do the same, but that we don't guarantee it:

Note that although we try to avoid the situation where a dependency transitively increases the MSRV of Tokio, we do not guarantee that this does not happen. However, every minor release will have some set of versions of dependencies that works with the MSRV of that minor release.


fix: listing table and columns were executed with null category because it specify database name instead of category name.

Although getTables / getPrimaryKeys / getColumns expect catalog name, TableIdentifier::getDatabase() doesn't return catalog name. And getDatabase() returns null, so getTables, etc scan all catalogs unfortunatey.
In a user's environment, an error cropped up.  So I need this fix.

@yas-okadatech
The changes seem correct to me. I am creating a similar PR, although the problem I want to fix is different.
I would assume that this PR is an attempt to reduce the differences in the code,
the code of current main branch was copied from output-jdbc, so it can be largely deleted.
Please refer to this PR.


deploy-all target should take more than one ESX and more than 2 VMS

Currently our deployment script installs VIB only to one ESX and plugin on 2 VMs. Our deployment target should takes care multiple input.

Some test scenarios are being commented out due to testbed limitation. We need to collect all the detailed requirements here and decide what's exactly needs to be done.

We need to first figure out how many test scenarios are being blocked by this. If there are a lot, we should mark this as mustfix, as it impacts code coverage.


Attempting to integrate an OpenRPC spec into our docs site

This required a bit of fiddling, configuring, and customizing some packages. Hopefully, it all builds and compiles OK 🤞🏻
Essentially, I've made an openrpc.json spec for the Soroban-RPC specification to live within. This is then used to generate the RPC pages, using the @metamask/docusaurus-openrpc package. I've swizzzled the method page from that package into a theme file for our own customizations. I've also included a patch to the @metamask/open-rpc-docs-react package that will remove any irrelevant stuff about ethereum, and include a better code examples section.

Preview is available here:http://soroban-docs-pr533.previews.kube001.services.stellar-ops.com/

Looks like the patch for @metamask/open-rpc-docs-react didn't apply... 🤔 I'ma have to look into that...

Something went wrong with PR preview build please check

Preview is available here:http://soroban-docs-pr533.previews.kube001.services.stellar-ops.com/

To see what is happening, I set up a new series of pages that can be found here: https://soroban-docs-pr533.previews.kube001.services.stellar-ops.com/rpc. I'm particularly happy with how the "methods" pages have turned out, like this one: https://soroban-docs-pr533.previews.kube001.services.stellar-ops.com/rpc/methods/getEvents


Mod more fits the category of "Client and server" instead of "Client" on Modrinth.

This mod requires both the client and server to have it installed for it to work.
The "Client" label is for mods you can use on servers regardless of if they have it installed or not.
Also-- absolutely love the textures and models for this mod. Talented stuff!

Oh, thank you very much for clarifying it! I'm really new to this community and still learning the ropes so I appreciate it a lot!


[Docs] Broken links of sample applications

Current State
There are a few links that are broken due to current changes in the docs on support for service-mesh-page
e.g. https://docs.meshery.io/service-meshes/adapters/istio

Desired State
You can find the links from here https://docs.meshery.io/guides/sample-apps
Add the correct links.

Contributor Guides and Resources

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

I will take this issue

Sure @iambami, go ahead! thanks.

Can I take this issue?


Add links to repos that use the wrappers

Perhaps as a section in the README like
## Used by

1. tool the first
2. tool the second

Ref: #2

We already have this in README.md.


Add example of using Spanner arrays in a query

Implements #200

Sean, can you add an example for an array column as well?
e.g.    cities ARRAY<STRING(1000)>

@hostirosti I could do so, but does that substantially change the client API?   My purpose wasn't to demonstrate Spanner so much as the curious and unexpected machinations necessary to interact with it using the golang client.  The crux of that, really, being the []spanner.NullString type.
I don't expect that would be any different with an in-table array.   I haven't tried, though.  If there is a significant difference, it should definitely be illustrated.

On the other hand, I wouldn't want anyone to think that this contrived example was an indication that for something as simple as this you should use a separate table rather than an in-table array.  I just don't want to use some terribly complicated example for illustration.

Fair enough... just need to contrive something else for an in-table array... colors! :)


[warning] Calendar

Each child in an array or iterator should have a unique "key" prop.

fixed by https://github.com/SFantasy/mirana/commit/95590fde929dc7e2cede1a48d67995291b3b1d21


build: run tests against view engine and ivy

We currently only run tests against Ivy due to the
commit that switched the --define=compile flag.
This means that the ivy_test and local_browsers job
are doing the same. Instead we should change ivy_test to
run tests against View Engine. Same applies for the cronjobs.

With this PR, it's also possible to switch between Ivy and View Engine easily by just passing --define=compile=legacy. I'm not 100% happy about the postinstall patch, but our setup is special since we do all of this in Bazel, and by default the pre-render NodeJS target should run with Ivy.. there is no easy way to hack/overwrite the module resolution like we can in browsers.
Since it's temporary and reasonable, I think it's fine like that.


Fetch upstream from the official repository

Hi! I don't see that private keys initialization is handled. Your keys init function should be provided here - link.
Moreover, please double check that your adapter features are provided in all necessary places by searching exising adapter keyword. (I would rely on "Ethereum" for i.g.)

Looks good. I think we need a couple of more tests for:

Consuls signature collection & submission for Gravity contract.
Oracles signature collection & submission for Nebula contract.
Data extraction flow (test covers oracle node behaviour).


Spanish special characters in edit results in failing to save the new information

Describe the bug
When you enter some special (spanish in this case) in title or description, the edit operation goes through successfully in the UI but the change is not actually getting saved.
To Reproduce
Steps to reproduce the behavior:

Go to an asset you own
Click on Edit
Enter something like 'Básico, Básico Adultos Formal, Modalidad General, Técnico Profesional, Académicos y Artes' in the Title or Description field
Save
Nothing changed (the UI doesn't reflect any error, the console either)

Expected behavior
To have my change saved or to see an error message
Screenshots
https://user-images.githubusercontent.com/13741335/158864412-e97ca7e5-904b-4018-bd6f-c1b662e0a972.mp4
Desktop (please complete the following information):

OS: MacOS
Browser Brave
Version 1.36.109

I don't think this is specifically related to special characters.
I tried right now, without special characters and I got the same behavior. I think the problem is that the asset is not refreshed after the metadata edit because, after a refresh, the data is updated.

Most likely fixed with https://github.com/oceanprotocol/market/issues/1212#issuecomment-1079041423, needs a retest

yeah closes also this issue 🎉 , tested them both


Changed analogy to be consistent with image

Changed accuracy/precision analogy from basketball to archery to be consistent with image

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.


Compilation error in framework, bad mixin application

Error: 'package:flutter/src/widgets/routes.dart': error: line 283 pos 1: cannot generate an implicit mixin application constructor forwarding to a super class constructor with optional parameters; add a constructor without optional parameters to class 'TransitionRoute' that redirects to the constructor with optional parameters and invoke it via super from a constructor of the class extending the mixin application
abstract class LocalHistoryRoute extends Route {
(I'm trying to add Dart_CompileAll before generating a snapshot.)

@Hixie

We use this code regularly and the analyzer isn't complaining... I'm not sure I understand what's wrong here.

The code seems fine. ModalRoute extends TransitionRoute with LocalHistoryRoute. TransitionRoute has two constructors, one with named optionals and one named one with explict arguments. ModalRoute calls the explicit constructor. This code seems to work fine, and the analyzer is ok with it. Can you elaborate on what change you would want to see?

@hausner @crelier

I do not see anything wrong with the code either. The workaround explained in the error message seems to be correctly implemented (the error location pointing to the mixin class is confusing, but this has nothing to do with the problem).
I am not sure how to reproduce this error on my machine, but here is a code snippet illustrating the restriction and its workaround:
class S {
S({x});
// S.workaround(x) : this(x: x);
}
class M { }
class C extends S with M {
// C({x}) : super.workaround(x);
}
main() {
new C();
}
You get a similar error message unless you uncomment the 2 lines implementing the workaround and then the error disappears. I have no idea why the error remains in the flutter sources.

@rmacnak-google Do you still see this?

I see a different error now, but I don't think this is important any more.

Thanks for the follow-up!


feat: add custom extractor for blisterreview.com

This pull request adds a custom parser for blisterreview.com. The default extractor does not always include all sections of the articles. The custom extractor includes all article text, bulleted lists, and images (because I feel they are important for their articles since they are gear reviews).
I have tested many articles using thew preview script and builds and tests are passing.

@jhotmann It looks like you're capturing the comments in addition to the content. You might need to either tighten up the selector or look into the options for cleaning content.

@adampash fixed the selectors so the comments are no longer included. Also while I was at it, I noticed the image captions were missing so those are now included as well.


Clarify serializer example

(AN: my misinterpretation of this one line cost me at least 4 hours in debugging time. I'd love to spare others from my fate!)
By calling the serializer PersonSerializer, there's an implicit assumption made by the user that this is the same serializer that should govern the list/detail views (if they exist) of the Person model, but this clearly isn't the case.
By changing the name of the serializer to PersonSearchSerializer (as a tutorial I found did), it becomes immediately clear that the Haystack serializer is not supposed to be the same serializer as the serializer for the model throughout the rest of the API.

Coverage remained the same at 92.235% when pulling 29bb388c694b719973e5f0c5ec3708e81dd22bcc on cameronblandford:develop into ba432f2a974b79c15dfc3e64e206afefbd5a7cad on inonit:develop.

Oh, sorry to have wasted your time... Thanks a lot for bringing this up, and fixing it 👍

You've saved much more by providing this library in the first place :)


Release date

Hi!
Thanks for a great article! I'm interesting in it and want to try some ideas.
When you are planning to release the framework? Can you provide some estimation of release date?

Releasing today or tomorrow. Enjoy!
I.
From: Mikhail Trofimov<EMAIL_ADDRESS>Sent: Thursday, January 12, 2017 5:40 AM
To: NVlabs/GA3C
Cc: Subscribed
Subject: [NVlabs/GA3C] Release date (#1)
Hi!
Thanks for a great article! I'm interesting in it and want to try some ideas.
When you are planning to release the framework? Can you provide some estimation of release date?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/NVlabs/GA3C/issues/1, or mute the threadhttps://github.com/notifications/unsubscribe-auth/APNcGvIk8MiHWej8b0tzLISYC04Ytst-ks5rRi09gaJpZM4Lhwpm.

This email message is for the sole use of the intended recipient(s) and may contain
confidential information.  Any unauthorized review, use, disclosure or distribution
is prohibited.  If you are not the intended recipient, please contact the sender by
reply email and destroy all copies of the original message.


Allow StatefulSets scaling with Keda

Currently charts keda manifest will work only if deploymentMode is set to Deployment.
Add support for StatefulSets scaling.

fixed in: 9ad4f00e7e82178691311bcb2ab91aaa90d5f1ae


DEPRECATION: ember-views.render-double-modify

DEPRECATION: You modified (-join-classes "ember-view" "ef-textarea" (-normalize-class "errorState" errorState activeClass="ef-error" inactiveClass=undefined)) twice in a single render. This was unreliable in Ember 1.x and will be removed in Ember 3.0 [deprecation id: ember-views.render-double-modify]

This was introduced by https://github.com/ember-forge/ember-forge-ui/commit/1beda3fabf0455ef15bc8cedca13b9134dc07a77 and https://github.com/ember-forge/ember-forge-ui-bootstrap4/commit/1a741b667a763464eb2aa2d31d319305df4f7d87
Further explanation is available at https://github.com/ember-forge/ember-forge-ui/issues/152

Likely resolved but need to confirm


Add validation and test for borrowingLimit when cohort is nil

What type of PR is this?
/kind feature

What this PR does / why we need it:
When cohort is nil, borrowingLimit should be nil too.
After add a webhook validation, some tests failed:
integration-test
unit
Which issue(s) this PR fixes:

#1496
#1385
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Add validation for clusterQueue: when cohort is empty, borrowingLimit must be nil.

/cc @kerthcet

/ok-to-test

/remove-kind failing-test
/kind cleanup

/remove-kind cleanup
/kind feature
As we're adding validations for borrowingLimit.

Can you update the release note:
Add validation for clusterQueue: when cohort is empty, borrowingLimit must be nil.

Can you update the release note:
Add validation for clusterQueue: when cohort is empty, borrowingLimit must be nil.

Done.

/kind api-change


API calls seem to be broken.

Trading 212 is returning a 404 when sending a "Market Order". I'm going to assume that the API has changed in some way. Does anyone have any info on where the addresses/API code for the PUTs came from? I can see it in the site code, and there appears to be no API documents.

Thanks for reporting this bug. I will investigate as soon as possible.


API documents need cleaning up

The API docs have seen quite a bit of improvement since I began reviewing gollum, but still need some tidying up. For instance, there is a bogus "PHOENIX Spectrum" header at the top of the class list.
Review link: https://github.com/openjournals/joss-reviews/issues/6601

The most recent commits should have fixed the extra headers appearing in the documentation pages, instead replaced by module-specific headers for PHOENIX, Sonora, etc. Are there any other issues that stand out in the documentation?


payment mode in payment made only include cash

payment mode in payment made under purchase module only include one mode- cash

This issue is fixed !


Inference on conversation.

Hello, great work.
I had two questions:

what sent_id in the sample input file  supposed to refer to??

If I want to make an inference for dialogue like tc genre, what should be the conversation format ??

Sent_id means index of the sentence which the token belongs to. For instance, in "Hello . This is dog speaking ." you will get [0, 0, 1, 1, 1, 1, 1].

What exactly do you mean by conversation format?

I mean If I have a conversation suck as :
person_1: Hey, how are you?
person_2: I'm fine, how what your day?
.
.
.
.
and so on
examples like that would be in  telephone conversations: tc in ontonote
if I want to use the model to do coref resolution on dialogue like that what should be their format (for example unique tokens around the speakers and so on).
if there is no particular format would you provide an example for such a case?

{
        "document_id": "tc_mydoc_001",
        "cased_words": ["Hey", ",", "how", "are", "you", "?", "I", "'m", "fine", ",", "how", "was", "your", "day", "?"],
        "sent_id": [0, 0, 0, 0, 0, 0, 1, 1, 1, 1, 1, 1, 1, 1, 1],
        "speaker": ["#1", "#1", "#1", "#1", "#1", "#1", "#2", "#2", "#2", "#2", "#2", "#2", "#2", "#2", "#2"]
}

Something like that?

that's exactly what I was asking thanks

You're welcome! You can also try with punctuation and see if it works better :)


torq: drop

Removing torq because upstream went closed source.
Change made in agreement with the maintainers: @mmilata, @prusnak
https://github.com/NixOS/nixpkgs/pull/358383
#ZurichZHF
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Please do manual backport

Thank you!


regression-test add regression tests for query profile and insert profile

Proposed changes
Issue Number: close #16540
Problem summary
There have been no regression tests for profile
Describe your changes.
add 2 regression tests for insert profile and query profile
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[x] Is this PR support rollback (If NO, please explain WHY)

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

have added a test for show query profile stmt in test_query_profile.

run all

run buildall

run buildall

run buildall

run buildall

run buildall


Appium test fixes

@PeterNgTr I'd like to open up your PR #1564 once again, as it appeared that all Appium tests were disabled and they actually didn't run :sob:
So let's try to enable them once again, and see if they pass.
Also, I didn't understand what should we do with current swipe* actions? I mean you implemented swipe via performTouchAction, however, there is no transition plan for current actions and API is not compatible.
Should we deprecate current swipe*?
How should they be replaced with newer API?

@DavertMik fixed appium tests here . -> https://github.com/Codeception/CodeceptJS/pull/1609


Static Code Analyzer : Logging Practice with use of system output stream

remove outputs or wrap with
#if DEBUG
#endif

Addressed in PR #509


Adopt apiextension.k8s.io & admissionregistration.k8s.io v1 API

The yaml in this repo is still using beta APIs that can switch to v1.

related: https://github.com/knative/serving/issues/9494

note: - eventing & serving have switched in previous releases

/lifecycle frozen

/lifecycle frozen

/assign @knative/eventing-wg-leads

/assign @knative/eventing-wg-leads

I just spot checked rabbitmq / natss and they are using both crd v1. rabbitmq is using admissionregistration v1/v1beta1, natss only v1beta1. Wonder if we should create an issue in each repo for these.

I just spot checked rabbitmq / natss and they are using both crd v1. rabbitmq is using admissionregistration v1/v1beta1, natss only v1beta1. Wonder if we should create an issue in each repo for these.

yeah feel free to do whatever - just raising awareness. If you make the separate issues feel free to close this one

yeah feel free to do whatever - just raising awareness. If you make the separate issues feel free to close this one

/close
This repo is being archived and there's no more code here.


When installing it it crashes the whole Home Assistant for an extended period

When I install this it crashes the whole Home Assistant and keeps it offline for extended periods of time. When I removed it it was fine then as soon as I went to configure it it crashed again.

I have no idea why it might be crashing for you. I am not able to reproduce it. I also don't have time to try to debug your particular use case, so I'm closing this issue.


Spaces in filenames

Hey guys, I decided to give Emacs a chance and installed Intero.
Since I'm absolutely new to it, I just tried opening a random file from a (stack enabled) project I have and nothing seems to work. Moving around the buffer shows a message saying "Couldn't guess that module name. Does it exist?", autocompletion does not work, type inspection doesn't either.
After trying to load the module in the REPL (with C-c C-l), I think I might know what's wrong. So, I have a special thing that has broken so many projects, it's kind of baffling. My home directory has a space character in it. I suspect this project is yet another victim. This is what happens when I try to load it:
Starting:
  stack ghci --with-ghc intero "--docker-run-args=--interactive=true --tty=false" --no-load --no-build --ghci-options -odir=c:/Users/VITORC~1/AppData/Local/Temp/intero4620cWq --ghci-options -hidir=c:/Users/VITORC~1/AppData/Local/Temp/intero4620cWq hetris
Intero 0.1.13 (GHC 7.10.3)
Type :intro and press enter for an introduction of the standard commands.

target `Coimbra\AppData\Local\Temp\intero-script4620pgw' is not a module name or a source file
Prelude> target `C:\Users\Vitor' is not a module name or a source file
Prelude>

My home directory is called "Vitor Coimbra", so anything inside it probably won't open correctly.
Edit: If it helps, I reported a similar issue to the stack project some time ago (https://github.com/commercialhaskell/stack/issues/1401).

We can fix that readily in the elisp. I'll get to it tomorrow!
On 12 Jun 2016 17:39, "Vitor Coimbra de Oliveira"<EMAIL_ADDRESS>wrote:

Hey guys, I decided to give Emacs a chance and installed Intero.
Since I'm absolutely new to it, I just tried opening a random file from a
(stack enabled) project I have and nothing seems to work. Moving around the
buffer shows a message saying "Couldn't guess that module name. Does it
exist?", autocompletion does not work, type inspection doesn't either.
After trying to load the module in the REPL (with C-c C-l), I think I
might know what's wrong. So, I have a special thing that has broken so many
projects, it's kind of baffling. My home directory has a space character in
it, which I suspect this project is yet another victim. This is what
happens when I try to load it:
Starting:
stack ghci --with-ghc intero "--docker-run-args=--interactive=true --tty=false" --no-load --no-build --ghci-options -odir=c:/Users/VITORC~1/AppData/Local/Temp/intero4620cWq --ghci-options -hidir=c:/Users/VITORC~1/AppData/Local/Temp/intero4620cWq hetris
Intero 0.1.13 (GHC 7.10.3)
Type :intro and press enter for an introduction of the standard commands.
target Coimbra\AppData\Local\Temp\intero-script4620pgw' is not a module name or a source file Prelude> target C:\Users\Vitor' is not a module name or a source file
Prelude>
My home directory is called "Vitor Coimbra", so anything inside it
probably won't open correctly.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/commercialhaskell/intero/issues/109, or mute the
thread
https://github.com/notifications/unsubscribe/AAArCyNOKcWgnERdkcnaNbRbv7Z5bd3pks5qLCgagaJpZM4Iz0aJ
.

I looked into this; it's not because of the Elisp; it's passing the filenames properly. It's stack ghci itself. Please refer to that issue: https://github.com/commercialhaskell/stack/issues/2266 For future updates. I am going to work on fixing in stack.

Hey Chris, once again, thank you for your work. Fixing that stack issue still didn't make this work for me, I'm still having this issue. Did I miss something (like updating intero as well) or should updating stack have been enough to make this work?

Did you update to stack as of today?

Yes, I did it about half an hour ago.


Dev Builds: Unable to write to User Settings because sync.enable is not a registered configuration.

When I got the latest from master on my dev machine and launch ADS, I'm met with the following error:
'Unable to write to User Settings because sync.enable is not a registered configuration.'

Looking at vscode commits, there is at least one commit that changes "sync.enable". @anthonydresser just as FYI master may be sync'ed at an unfortunate spot at the moment 😞

I have another vscode merge I'll verify its fix in that and close if it is.

Should be fixed


Not working with XCode 7.1?

Was just playing around with this and it seems, that it will not work under 7.1?
Is that correct?
It works before unter 7.0.1

Same thing here ... Not working with xcode 7.1 =(

I use Xcode 7.1, all fine.

Not working with xcode 7.1

Is working in Xcode 7.1, iOS 9.1, El Capitan 10.11.1

Is working with "com.apple.mobilecal" but not with my app

@AdamZikmund did you correct configure shortcuts in plis file? (https://developer.apple.com/library/prerelease/ios/documentation/UserExperience/Conceptual/Adopting3DTouchOniPhone/3DTouchAPIs.html#//apple_ref/doc/uid/TP40016543-CH4-SW1)

I followed your guide 1/1 and it doesn't work for me. Meaning when I run last command
echo 'com.apple.mobilecal' | nc <IP_ADDRESS> 8000
Nothing happens. Latest Xcode.

Works for me on 7.1, El Capitan 10.11.1, iPhone 6S Simulator.

Getting:
HTTP/1.1 400 Bad Request
Content-Length: 30
Content-Type: text/plain
after echo 'com.apple.mobilecal' | nc <IP_ADDRESS> 8000

Doesn't work for me either. xCode 7.1.1, El Capitan

My solution is simple: I bought an iPhone 6S and testing on it ;)

@UMAD you need to use iPhone 6s ou 6s plus in Simulator to works.

#18 fixes this.


Clean up content model fields that are duplicated or don't show up from Contentful

As a content creator, it can be hard to know which fields will show up on the website. Sometimes description is duplicated, sometimes body shows up, other times it doesn't.
We might consider cleaning up our content models accordingly, and/or indicating for content users what does show up on the site?

This has been resolved for now by hiding irrelevant fields.


Have better conditions

/kind proposal
Goals
Improve conditions to improve the usability of the operator.
Detailed Description
We should improve the conditions in CAPHV. There are the following tasks to do:
Tasks

[x] Change rate limit condition to the same as in CAPH
[ ] Add condition that no available device has been found
[ ] printcolumns with message and reason like in CAPH
[ ] conditions Summary like in CAPH
[ ] Add condition for SSH key not found
[ ] Add condition for server off
[ ] Add conditions for TargetClusterReadyCondition
[ ] Add condition for wrong API token

Add condition that no available device has been found

isn't it the reason for it https://github.com/hivelocity/cluster-api-provider-hivelocity/blob/37f0a5563e72b3e717ad23a987db08f03af6671b/api/v1alpha1/conditions_const.go#L25 ?

Add condition for server off

do you mean by this?
https://github.com/hivelocity/cluster-api-provider-hivelocity/blob/fc8e3899b87a7246b6fbeeeabf634af96bbc54bc/pkg/services/hivelocity/client/hvclient.go#L47

Add conditions for TargetClusterReadyCondition

I see a https://github.com/hivelocity/cluster-api-provider-hivelocity/blob/fc8e3899b87a7246b6fbeeeabf634af96bbc54bc/api/v1alpha1/conditions_const.go#L40 is that same we are talking about?


Add Maintainers Guidelines

Signed-off-by: Kuhrt, Tracy A<EMAIL_ADDRESS>

Looks good to me.  Thanks Tracy!

I think you have the approvals needed for merge


Return errors when printing tables/etc

What was changed

Printing of Tables/JSON now returns errors and is handled through urfave/cli error handler.
Why?

Standardizes error handling.

You forgot to change PrintItems itself.

it just required this PR to be merged first https://github.com/temporalio/tctl-kit/pull/26


Add Integration tests for getSubscriptionThrottlingPolicies endpoint

This PR will add test cases for getSubscriptionThrottlingPolicies endpoint

merged https://github.com/wso2/carbon-apimgt/pull/9971. update the carbon.apimgt version ones the release is done


tests/add rudimentary unit test

refactor(function): Add path as argument with default
test: Adds test for export watering function
chore: configure repo
chore: Remove unused deps
refactor: Remove unused imports
ci: Run tests on ci
chore: Add devcontainer

@Lisa-Stubert PR is ready for review


isomorphic-git can't push in webcontainer

Describe the bug
The following shell command run fine in Node 16/14 on an Ubuntu machine, but fail on WebContainers
/tmp/test-dir
> npx isomorphic-git push --corsProxy='https://cors-proxy.myproxy.app'<EMAIL_ADDRESS>success Install finished in 2.27s

One or more branches were not updated:
  - refs/heads/main: unpacker error
GitPushError: One or more branches were not updated:
  - refs/heads/main: unpacker error
    at _push (/home/.turbo/npx/LOzMwT/node_modules/isomorphic-git/index.cjs:11731:11)
    at async Object.push (/home/.turbo/npx/LOzMwT/node_modules/isomorphic-git/index.cjs:11809:12)
    at async eval (/home/.turbo/npx/LOzMwT/node_modules/isomorphic-git/cli.cjs:14:20) {
  caller: 'git.push',
  code: 'GitPushError',
  data: {
    prettyDetails: '\n  - refs/heads/main: unpacker error',
    result: {
      ok: false,
      error: 'unpack-objects abnormal exit',
      refs: { 'refs/heads/main': { ok: false, error: 'unpacker error' } },
	  headers: {
	      'cache-control': 'no-cache',
	      'content-type': 'application/x-git-receive-pack-result',
	      date: 'Mon, 28 Mar 2022 16:14:39 GMT',
	      server: 'nginx/1.21.0',
	      'transfer-encoding': 'chunked'
	  }
    }
  }
}

Everything else seems to work fine. The same error occurs when running from a script instead of the CLI.
The issue seems to be some incompatibility with WebContainer FS, HTTP, or jsh.
Link to the blitz that caused the error
https://stackblitz.com/edit/node-b7uvun
Steps to reproduce

make a commit
Attempt to push the commit with npx isomorphic-git push --corsProxy='https://cors.isomorphic-git.org'<EMAIL_ADDRESS>
Expected behavior
This is the result of a successful push using the exact same commands in Node 14 and Node 16 Ubuntu.
npx: installed 24 in 2.736s
{
  "ok": true,
  "refs": {
    "refs/heads/main": {
      "ok": true,
      "error": ""
    }
  },
  "headers": {
    "server": "nginx/1.21.0",
    "date": "Mon, 28 Mar 2022 16:27:45 GMT",
    "content-type": "application/x-git-receive-pack-result",
    "transfer-encoding": "chunked",
    "connection": "close",
    "access-control-allow-origin": "*",
    "access-control-expose-headers": "accept-ranges,age,cache-control,content-length,content-language,content-type,date,etag,expires,last-modified,location,pragma,server,transfer-encoding,vary,x-github-request-id,x-redirected-url",
    "cache-control": "no-cache"
  }
}

Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots
No response
Platform
Node: WebContainer Node 14
isomorphic-git: 1.17.0
Chrome: 99
Additional context
Related: https://github.com/isomorphic-git/isomorphic-git/issues/1523

Hey @rayfoss, thanks for reporting this issue.
After digging into it a bit deeper, I found out that we don't handle binary payloads correctly. I will fix this and get back to you when it's released. Can't give an estimate when it will be released, but should be one of the following days.

Has the fix been released? As of Hash = 31afbfcfe1b0985e8cbdb27d44e16766166a118e I'm still getting a webcontainer unique error.
https://stackblitz.com/edit/node-jdru2t?file=index.mjs
{
  ok: false,
  error: 'index-pack abnormal exit',
  refs: { 'refs/heads/master': { ok: false, error: 'failed' } },
  headers: {
    'cache-control': 'no-cache, max-age=0, must-revalidate',
    'content-type': 'application/x-git-receive-pack-result',
    date: 'Tue, 05 Apr 2022 16:21:40 GMT',
    expires: 'Fri, 01 Jan 1980 00:00:00 GMT',
    pragma: 'no-cache',
    server: 'nginx/1.21.0',
    'transfer-encoding': 'chunked',
    vary: 'Accept-Encoding',
    'x-github-request-id': 'E2C2:8041:12F2DA:22CBA2:624C6C94'
  }

Hey there 👋
No the fix is not deployed yet. We'll let you know once it's on production.

This issue should be fixed now! Thanks for your patience 🙏 .


affects_stability_of can't make plan

{'message': {'query_graph': {'nodes': {'a': {'category': 'biolink:NamedThing',
     'id': 'NCBIGENE:18671'},
    'b': {'category': 'biolink:ChemicalSubstance'}},
   'edges': {'ab': {'subject': 'a',
     'object': 'b',
     'predicate': 'biolink:affects_stability_of'}}}}}

There's data in automat/ctd.
"Could not find a plan to traverse query graph"
This may be fixed with a rebuild of the registry, it's possible that this is a new predicate ctd...

Same is true of ctd predicate biolink:decreases_mutation_rate_of
{'message': {'query_graph': {'nodes': {'a': {'category': 'biolink:ChemicalSubstance',
     'id': 'CHEBI:8069'},
    'b': {'category': 'biolink:Gene'}},
   'edges': {'ab': {'subject': 'a',
     'object': 'b',
     'predicate': 'biolink:decreases_mutation_rate_of'}}}}}

Same is true of ctd predicate biolink:decreases_mutation_rate_of
{'message': {'query_graph': {'nodes': {'a': {'category': 'biolink:ChemicalSubstance',
     'id': 'CHEBI:8069'},
    'b': {'category': 'biolink:Gene'}},
   'edges': {'ab': {'subject': 'a',
     'object': 'b',
     'predicate': 'biolink:decreases_mutation_rate_of'}}}}}

The first query above appears to now make a call to automat/ctd, but no data is returned. Sending this to automat/ctd directly returns no results:
{
    "message": {
        "query_graph": {
            "nodes": {
                "a": {
                    "category": "biolink:Gene",
                    "id": "NCBIGENE:18671"
                },
                "b": {
                    "category": "biolink:ChemicalSubstance"
                }
            },
            "edges": {
                "ab": {
                    "subject": "a",
                    "object": "b",
                    "predicate": "biolink:affects_stability_of"
                }
            }
        }
    }
}

The second appears fixed as of #83.

OK, sounds like this one can be closed then - I'll check on the CTD side


Future development of pymee

Hi,
I'm planning some bigger changes to pymee in the near future:

support more message types (error, user, settings)
use enum classes in const.py
refactor code so HA integration does not need to use private properties

Do you still want to  be kept in the loop and do it via Pull requests -> release by you or shall we do it the same way as with the HA integration?

Hi, I can not guarantee that I will check GitHub regularly and merge PRs in a time frame that is acceptable to you. Feel free to continue development in your fork. Alternatively, I could also add you as a contributor to this repo so you can merge PRs yourself.

Let's try the second option. That way it still stays your application - you did the most part of the work on it.

Thank you, just tested a pull-request to develop branch to see if it works.


fix: add iam doc link to aws prerequisites

Spotted that the AWS Prerequisites section had a (docs) line implying a link to the AWS docs site, but no URL provided.
Have updated so it uses the same URL as on our main docs website

Wouldn't it be better to link to https://docs.kubefirst.io/aws/quick-start/install/cli#aws-prerequisites as we point there, but we also have additional information on how to set IAM when using the CLI?

@fharper makes sense - have updated


Update mockito-core to 3.11.1

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

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

labels: test-library-update, semver-minor

Superseded by #187.


Update docs for the Paris upgrade

Things that need to be changed:

Data is posted on L1 in ~10 seconds, updated from ~15: https://docs.etherlink.com/#its-fast -- addressed by https://github.com/etherlinkcom/docs/pull/64

Not aware of any other updates needed here -- closing.


Postgres: Information schema add user defined type

@a8m I didn't merge in the unit test changes because those nulls are expected. The only time udt will not be null is when using a type created with CREATE TYPE. I have added another unit test to cover this case and added an integration test.
The pr will now allow migrations with custom schema types whem the type does not change

@a8m I didn't merge in the unit test changes because those nulls are expected. The only time udt will not be null is when using a type created with CREATE TYPE. I have added another unit test to cover this case and added an integration test.
The pr will now allow migrations with custom schema types when the type does not change

I just checked it on PostgreSQL 10/11/12 and I got this:

@a8m nothing else to say except that I am completely wrong. Will update the unit tests

Thanks for adding this @cliedeman 😃


Support GitHub workflow badge for private repos

:clipboard: Description

The GitHub workflow badge does not support private repos, even with a PAT with access to them.
If I understand the code correctly, the current GithubActionsWorkflowStatus badge service uses the public url for Workflow Status badges to fetch the workflow status and converts it to a Shields badge. That would explain why the status of private repos results in a repo or workflow not found.
As said in the GitHub docs:

Workflow badges in a private repository are not accessible externally, so you won't be able to embed them or link to them from an external site.

To support this I think the GitHub API needs to be used instead.

The history behind why this badge doesn't use the API is long and spread across many issues, but we did at one point migrate this to use the API instead of scraping GitHub's native badge.
Unfortunately, this introduced several bugs that we weren't able to solve using the data/endpoints exposed by the API so we switched back to scraping the native badge. The key issue is https://github.com/badges/shields/issues/8733 (this was one of the bugs introduced by switching to the API as the data source) and to solve it we picked option 3 from https://github.com/badges/shields/issues/8733#issuecomment-1356274451 . We accepted that the tradeoff of that solution is that users who self host would not be able to use it for private repos, but it is a known limitation rather than an unexpected bug.
If you want to read the whole saga that lead to this, the relevant issues and PRs are:

https://github.com/badges/shields/issues/4587
https://github.com/badges/shields/issues/8146
https://github.com/badges/shields/pull/8475
https://github.com/badges/shields/issues/8671
https://github.com/badges/shields/issues/8736
https://github.com/badges/shields/issues/8733
https://github.com/badges/shields/pull/8738

I haven't really looked at https://github.com/badges/shields/pull/9190 but I suspect you're probably introducing the same bugs I introduced in https://github.com/badges/shields/pull/8475 when I tried to migrate this before and then reverted in https://github.com/badges/shields/pull/8738.

Looking at the issues more closely I understand that using the API has some problems outside of our control that affect the performance required for this badge. In that case I think we indeed have no choice but to use the SVG scraper.

I should have searched more thoroughly before I created this issue

No worries. As I said, the history of this is spread across multiple discussions so there isn't really a single search result that would have given all the background on this.

Looking at the issues more closely I understand that using the API has some problems outside of our control that affect the performance required for this badge

Yeah pretty much. It is not that it is impossible to implement the correct behaviour using the data exposed by the API, but we found it was not really possible to do it inside the timeframe we need to render a badge.

I'll close this issue and the PR, thanks for following up on it :pray:.


Improve performance by using FadeTransition

When changing opacity in animations, it is much more performant to use FadeTransition instead of a re-building Opacity. See “Performance considerations for opacity animation” in https://api.flutter.dev/flutter/widgets/Opacity-class.html.
I also made changes to animate_do_fades and animate_do_fadeouts to use SlideTransition instead of Transform.translate. This lets us get rid of AnimatedBuilder altogether.
This could be replicated in other files as well, but I ran out of time.

Friendly ping.

Hey, I'm actually seeing that the performance improvement isn't all that  — I was unwittingly comparing apples to oranges. Closing this, at least until I have an actual performance improvement.


API name cannot have a empty space

Describe your problem(s)
as of now, it is not possible to create an API name with empty space in the publisher.
Sometimes API names consist of two words and the only option is to use the Camile style with is not very esthetic.
Describe your solution
Can we have an API and addition parameters like 'display name' and use that in DevPortal?
The name will be still used for uniqueness but display name would be used in DevPortal

Update 27/10/2023

Checked the feasibility of including spaces to the API name and Changing the API provider.
Created feasibility doc[1].
There were some issues happened in both of these tasks, which were explained in this doc [1].
On 20/10/2023 had a meeting with Sanjeewa, Nandika, Dushani and Thilini regarding this issues. And suggested me to keep looking for the solutions for these issues.
For the search issues when we are search in publisher portal when including the spaces in api name, we are using "%20" instead of search in the backend.
To api provider change manually, these things needs to done.

Change the Api provider artifact in the registry for that API.
Change the api provider path, by renaming/changing the name of the api provider root folder.
Change the api provider name in the AM_API table in the db.

The various problems  have arose by using steps.

The APIs did not appeared in the publisher portal as well dev portal.
The total api count in the publisher portal was wrong.

After having several discussions with Prasanna we have found that, there is method written in the registry, registry.move() . By using this method we can easily change the path. For example

registry.move("/_system/governance/apimgt/applicationdata/provider/admin/PizzaShackAPI/1.0.0/api",
                "/_system/governance/apimgt/applicationdata/provider/sahan/PizzaShackAPI/1.0.0/api");

to
registry.move("/_system/governance/apimgt/applicationdata/provider/admin/PizzaShackAPI/1.0.0/swagger.json",
                "/_system/governance/apimgt/applicationdata/provider/sahan/PizzaShackAPI/1.0.0/swagger.json");

After using this method, all of the issues occurred when changing api provider name were disappeared.
Therefore i concluded that these are tasks feasible to continue.
Next week I am hoping to implement these features.
Started to create the design doc.[2]
[1] https://docs.google.com/document/d/1GfA81SP1JWfQGWyNsWtFz0Gv2BDTvlxIovFp4f8kHRw/edit
[2] https://docs.google.com/document/d/1xOo3vGUzTEm2A46UthkWA3xsjVtSgziREc-ebrpAJPE/edit

Update 10/11/2023

When checking the feasibility of including spaces to the API name, Tharika mentioned me to check with APICTL imports and exports when there is api name with with spaces.
This was previously disallowed by this PR: https://github.com/wso2/carbon-apimgt/pull/8305/files
But we can't find the codelines which were changed  by this PR, in the 4.2 branch and the master branch.
Checked the APICTL imports and exports when there is api name with with spaces.
When exporting we need to include the api name in spaces.
apictl import api -f "./Test API 123" -e production
When importing we can have \ symbol between space or use "".
Ex:-
./apictl import api -f "dev/Test API 123_1.0.0.zip" -e dev
./apictl import api -f dev/Test\ API\ 123_1.0.0.zip -e dev
Started to implement the api provider name change feature
Added the method changeApiProvider() in the RegistryPersistenceImpl.java, In this method the logic is
1.) First move the registry path to the new api provider
2.) Then change the API provider artifact in the new registry provider path
There was an issue occured when testing the new implementation, the apis doesn't display properly in the publisher portal and the devportal.
But in the debug mode this error wasn't occured.  I think the reason for this issue is, after moving to registry to new api provider, it needs some seconds to reindex, then only we can change the API provider artifact. I have put the thread.sleep(5000) in the middle of these 1 and 2 steps, then this issue wasn't occured.
This issue was discussed with Arshadh and Prasanna. There were several solutions have tested but all of these were failed. Tharindu mentioned that we do not need to change the path to change the api provider name. Currently testing this new approach.
This week I am on annual leave, next week hoping to conclude the design and start to implement the design in the new approach.

Change Api Provider PR: https://github.com/wso2/carbon-apimgt/pull/12189
Enable to create APIs with spaces: https://github.com/wso2/carbon-apimgt/pull/12187
UI PR: https://github.com/wso2/apim-apps/pull/519
Doc PR: https://github.com/wso2/docs-apim/pull/7467
Intergration Test PR: https://github.com/wso2/product-apim/pull/13304

Update 18/12/2023

Tested the reasons for the failures.
The test failures occurred because the newly added scopes were not included in the tenant.confs within the product-apim repo.

Need to merge the 3 PRs after getting the review comments resolved and approved - https://github.com/wso2/product-apim/pull/13304, https://github.com/wso2/apim-apps/pull/519, https://github.com/wso2/carbon-apimgt/pull/12200

Do we have a plan to release the patch for the 4.2 version?


Transparency loss when encoding GIF format with GD driver

Describe the bug
In the case of an image with (semi) transparency, e.g. a PNG format, the transparent areas are lost if the output format is GIF format and the size is changed. This happens only with GD driver.
Code Example
$image = ImageManager::gd()
    ->read($path)
    ->resize(300)
    ->toGif();

Expected behavior
Fully transparent areas should be retained in GIF output.
Images
Input

Output

Environment (please complete the following information):

PHP Version: PHP 8.3
OS: Mac OS
Intervention Image Version: 3.7.2
GD or Imagick: GD

Fixed in 9f5e0624d4b8d073f58f8bfef71ba8363d9fb688.


[TACHYON-660] Fix method names in CapacityUsageIntegrationTest

Change-Id: Ic426207296a7d8b074c588b857aebd1e38e45a4f

Can one of the admins verify this patch?

Jenkins, test this please.

Jenkins, ok to test.

Thanks!


IMPROVEMENT - Shaka Streaming Error Handling

Have you read the FAQ and checked for duplicate open issues?:
Yes
What version of Shaka Player are you using?:
2.4.3
Can you reproduce the issue with our latest release version?:
Yes
Can you reproduce the issue with the latest code from master?:
Yes
Are you using the demo app or your own custom app?:
Tried on both
If custom app, can you reproduce the issue using our demo app?:
Yes
What browser and OS are you using?:
Chrome 68
What are the manifest and license server URIs?:
Unable to share, but we can use any on the demo app to test this
What did you do?
Simulated 1001, 1002 and 1003 errors by proxying and overwriting stream responses with 401s
What did you expect to happen?
Similar to the native video players, example Safari's player, I expected Shaka player to attempt to use all available tracks before giving up and erroring out unrecoverably
What actually happened?
I feel like Shaka player errors out too easily. I am aware that the current streaming failure callback retries indefinitely for Live and not at all for VOD, however, it doesn't retry with other tracks.
For example, if there is something wrong with a track with of a particular bitrate, then the player should attempt with the next most appropriate bitrate. This should be the case for both Live and VOD.
Suggestions
I tried overriding the failure callback to manually selectVariantTrack and then retryStreaming. This works to switch and retry but even if the manually selected bitrate works fine, the ABR kicks in and tries again with the original failed bitrate (assuming that was the most appropriate one). Disabling the ABR doesn't make sense since the ABR could still potentially have 6-7 tracks to play with besides the failed one. Instead the ABR should remove tracks that have retried and failed a certain number of times (can be defined in the streaming or abr configuration) and only if it has tried and failed all the different tracks should it then throw some sort of unrecoverable error so that the player can surface up an error message to the user.
Please :)
This is something I need quite urgently, if it is something that needs time, in the meantime can we add something to the abr configuration to ignore tracks with certain bitrates? So that when a bitrate fails consistently, we can update the configuration to tell the abr to ignore it for that session. (Atm we only have the option to set a max and min bitrate)
Thanks again for everything Shaka has helped us with!

@joeyparrish could you please schedule this and/or advise on a workaround if need be

What kinds of errors would exist for a single track?  If it is a 401/403 error, it would affect every URL at that domain; if it was a 404, that seems like either bad content or a transient error that might be fixed by waiting; 5XX errors are also likely to affect every stream.  Also, if there are 2 audio streams and 5 video streams, that means we will need to exhaust the retry parameters on 10 attempts before we fail to play, that is a lot of network requests to wait for a failure.
For your application, you could implement a custom AbrManager.  You could subclass our implementation and filter out any variants you don't want to play.

@TheModMaker I get requests all the time from other teams in my org to improve player resiliency. Often the org wants to go to extreme (and IMO unreasonable) lengths to ensure the player makes every possible attempt to play something before producing an error. This is ostensibly to improve user experience, but mostly has to do with keeping the error rate down. There is an engineer who sits next to me whose sole job is to investigate error rates and video start time on web clients. I am sympathetic to @sandeep-mirpuri 's plight; demand for these types of features is strong, even if the engineering basis for them is weak.

In a nutshell, this request is for us to adapt between tracks as a way to avoid playback errors.  Is that right?

@joeyparrish yes that is the gist of it @TheModMaker @chrisfillmore when the number of pieces to get the content becomes many the potential of particular tracks erroring out, where others don’t, gets higher. We ingest VOD content from our customers or ingest and convert live streams directly from broadcasters into VOD content. And our app makes all of these available. There are often server side ads inserted into the VOD content. All our content is on multiple origins which are behind Akamai. So there are so many places where something can go wrong and a particular track might not get written out properly or when inserting ads there may be an issue. While I agree the chances are few and we need to be fixing the conversion or ingestion algorithms that are screwing up, in the meantime we need to make sure that streaming doesn’t fail for a user on VOD or LIVE if it so happens that one track didn’t get written to the origin correctly
@TheModMaker thanks for your advice on extending shakas abr manager. I was able to work that out and hack something together. So I am not in as much of a sticky situation as 3 days ago. But I feel like a more robust solution within shakas abr manager would be more desirable in the long run than my hack.
And it just can’t be said enough, thanks to everyone working on Shaka! It is still by far one of the best players and very usable and customisable.

@joeyparrish  The Google ExoPlayer for Android implements similar behaviour.  In the event of a 404 error it will retry on the same fragment for a fixed retry count. Once retries are exhausted it will move down to the next bitrate and so on.  ExoPlayer temporarily blacklists any representation that causes a the max retries to be exceeded (excluding the lowest bitrate representation), to avoid the ABR trying to select bitrates that are currently experiencing errors.

Okay, that sounds reasonable.  It will go onto the backlog for now, since our current milestones are very full.  I will also change the issue name to reflect the plan.  Thanks!


Kstar unable to parse plans with "s

When object names have "s in them, the planner works fine but Kstar errors out.

download pddl
Of course, one can say such strings are not allowed, but I feel that if the planner handles it then the package should be able to forward that information correctly.

Not sure I understand, do you want me to write to json "map \"new_york\" query" instead?

I am guessing I would need something like
#include
...
std::string name = operators[plan[0]].get_name()
name = std::regex_replace(name, std::regex("\""), "\\"");
...
added here:
https://github.com/IBM/kstar/blob/main/src/search/plan_manager.cc#L113C24-L113C53

do you want me to write to json "map "new_york" query" instead

No I think the problem is inside the package reading from the output of fast downward. Coz I am getting back an empty list of plans with the decoding error for the step map "new york" query
Try with the attached PDDL.

Okay, let me rephrase. A valid json would have an escape character before ". So, the json generated by K* is not a valid json. That's why it fails on reading.

I don't think thats the problem. "It" here is K* itself! The packages asks for the files directly as input so by then the escaping into text has already happened. The strings with "s then get passed all the way through to the planner as is. So the problem is at reading the sas plan output files when it is trying to generate the output JSON to return back (its not a case of an invalid JSON generated by K* coz it does not return the doc string it returns the JSON object).

There is no actual reading of sas plan files, the code I referred to above is dumping the plans into JSON. The action names are strings, they have the " inside, but without escape characters, and when written the way they are, the JSON does not have the escape characters as well, which makes the JSON invalid. I did the dump to JSON myself instead of using a JSON library, maybe such libraries do know how to handle the string with " correctly. Or I can just do the ad-hoc fix.

I see, up to you how / if you want to handle it. I am transforming the names before sending anyway but in general you might want to cover it for other users.

If this is not an issue for nl2flow, then maybe the issue should be opened on the K* git?
I have never experienced or asked about it until now, so I assume this is not urgent.

Ok closing this and logging it in kstar for posterity.