How to use angular-meteor-auth from NPM?

I gradually want to upgrade our current meteor (1.2) app to ES6 modules and I hope to use as much from NPM instead of atmosphere as possible.
I'm testing out some strategies, but I can't seem to make angular-meteor-auth from NPM working.
Here's a little test repo:
https://github.com/sebakerckhof/ngmeteor13test
If I install angular-meteor-auth from npm and import it like https://github.com/sebakerckhof/ngmeteor13test/blob/master/imports/ui/app/app.component.js#L4
I get an error from the angular-meteor-auth code about this.autorun not being defined.

@Urigo Any idea?

@sebakerckhof do you have a stacktrace to the this.autorun not define message?

@mxab For some reason the developer tools doesn't give me a stack trace. However, the error originates from this line: https://github.com/Urigo/angular-meteor-auth/blob/master/dist/angular-meteor-auth.js#L90
Anyway, you don't need to do anything special to trigger it. If you just clone & run the repo mentioned above, you should see it directly when you navigate to the running app.

Actually, if I manually set a breakpoint I can get a stacktrace:
$$Auth (angular-meteor-auth.js:90)
(anonymous function) (angular-meteor.js:1756)
meteorInstall.node_modules.angular-meteor.dist.angular-meteor.js.angular.module.service._construct (angular-meteor.js:1755)
(anonymous function) (angular-meteor.js:1738)
meteorInstall.node_modules.angular-meteor.dist.angular-meteor.js.angular.module.service.mixin (angular-meteor.js:1737)
(anonymous function) (angular-meteor-auth.js:204)
invoke (modules.js?hash…1ba96295c:5415)
(anonymous function) (modules.js?hash…1ba96295c:5223)
forEach (modules.js?hash…1ba96295c:1018)
createInjector (modules.js?hash…1ba96295c:5223)
doBootstrap (modules.js?hash…1ba96295c:2455)
bootstrap (modules.js?hash…1ba96295c:2476)
onReady (main.js:7)
fire (jquery.js:3143)
fireWith (jquery.js:3255)
ready (jquery.js:3467)
completed (jquery.js:3498)

@sebakerckhof looks like we are using it here: http://www.angular-meteor.com/tutorials/whatsapp/ionic/authentication
does it happen there as well?

like, the bundle from bower works for you?

@Urigo Well that's bower in a webpack project... I'm talking about a Meteor project where I'm trying to move away from atmosphere packages as much as possible towards npm packages...
Please see the example repo: https://github.com/sebakerckhof/ngmeteor13test

Over here https://github.com/sebakerckhof/ngmeteor13test/blob/master/imports/ui/app/app.component.js#L31 you need to load angular-meteor before loading angular-meteor-auth since angular-meteor-auth is dependent on some angular-meteor internals.

@DAB0mB Thanks, that indeed seems to solve the issue. However, shouldn't the module dependencies be so that the order in which I put them doesn't matter? I mean if angular-meteor-auth specifies all the dependencies it needs.

@sebakerckhof It actually makes sense because there are some run blocks which should be invoked in the right order. Like, you can't load angular modules without loading angular first, am I right? Same thing for angular-meteor and angular-meteor-auth, an extension can't be applied without the core library being loaded first. Anyways if that's the issue, I don't find it critical, but whatever @Urigo decides.

Okay. I thought angular invoked the run blocks bottum up acvording to the dependency graph, but apparently not. Closing as it's a user error.


Comparison is buggy

When comparing -1 to a number between -1.00001 and -1.99999999 it says that the second number is larger than the first for values larger than or equal to -2.0 as the second number it works correctly

I think i found the issue, i'll try to provide a fix for this in the next few days

Thank you for your fix. I merged your pull request.


Android TextInput Overflow hides start of line content

Description
This issue is a repro of https://github.com/facebook/react-native/issues/25045 which was closed as stale but continues to occur in React Native 0.64.3 at the least (Expo 44). I'll update here when snacks move to Expo 45 or when I have a second to build out a local 0.66 repro.
If someone has a 0.66 project handy they can quickly overflow a TextInput on android and confirm I'd appreciate it. Given the previous issue I fully expect it will.
Version
0.64.3
Output of npx react-native info
System:
    OS: macOS 12.3.1
    CPU: (10) x64 Apple M1 Max
    Memory: 1.02 GB / 64.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 14.18.1 - ~/.nvm/versions/node/v14.18.1/bin/node
    Yarn: 1.22.17 - /opt/homebrew/bin/yarn
    npm: 6.14.15 - ~/.nvm/versions/node/v14.18.1/bin/npm
    Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
  Managers:
    CocoaPods: 1.11.2 - /opt/homebrew/bin/pod
  SDKs:
    iOS SDK:
      Platforms: DriverKit 21.4, iOS 15.4, macOS 12.3, tvOS 15.4, watchOS 8.5
    Android SDK: Not Found
  IDEs:
    Android Studio: 2020.3 AI-203.7717.56.2031.7935034
    Xcode: 13.3.1/13E500a - /usr/bin/xcodebuild
  Languages:
    Java: javac 17 - /opt/homebrew/opt/openjdk/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: 17.0.2 => 17.0.2
    react-native: 0.64.3 => 0.64.3
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Steps to reproduce
Create a sample TextInput in your code.
Type a lengthy string in it.
First few letters are truncated
Actual Result:
Displaying the text at the end of the text field
Expected Result:
Display the text from beginning
Snack, code example, screenshot, or link to a repository
https://snack.expo.dev/@kavla/android-textinput-overflow

i have same problem. it seems that a vertically fixed space has been allocated

Updated the snack to Expo 47
React Native 0.70.5
Issue continues to repro.
https://snack.expo.dev/@kavla/android-textinput-overflow


WR-675 Added sitemap module and its related configurations

Added sitemap module and its related configurations to meet the acceptance criteria.

Looks good to me.


Fix release bug

https://github.com/Azure/sdk-release-request/issues/3613

/check-enforcer override


Ticket validator icon is confusing

Describe the bug
The icon for amenity=ticket_validator is the generic vending machine icon (temaki-vending_machine). This is confusing, as ticket validators aren't vending machines, but are commonly placed near transit ticket and other (sometimes generic) vending machines.
To Reproduce
Place a ticket validator next to a vending machine.
Expected behavior

Reuse the existing ticket vending machine icon (temaki-vending_tickets) for ticket validators, this is not ideal but in my opinion still better than the current icon, or
Add a new icon for ticket validators.

Reuse the existing ticket vending machine icon (temaki-vending_tickets) for ticket validators, this is not ideal but in my opinion still better than the current icon, or

would reduce confusion with generic vending machine, increase confusion with machine selling tickets, not sure which is worse
trivial to change - I did it in  change ticket_validator icon to ticket #1388  - you can test it there and follow "Checklist and Test-Documentation Template" to make it more likely to merge
if on testing it is not great idea - please comment there so I can close the PR

Closed by https://github.com/openstreetmap/id-tagging-schema/pull/1388


[BUG] test_autoscaler import error

Patch incoming
ImportError while importing test module '/usr/src/swarm_cloud/tests/structs/test_autoscaler.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
/usr/local/lib/python3.9/importlib/__init__.py:127: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
test_autoscaler.py:4: in <module>
    from pytest import patch
E   ImportError: cannot import name 'patch' from 'pytest' (/usr/local/lib/python3.9/site-packages/pytest/__init__.py)

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

This can be closed


[bug] <I can't able to connect walletconnect on desktop with my mobile>

Is there an existing issue for this?

[X] I have searched the existing issues

RainbowKit Version
^0.4.1
wagmi Version
^0.5.6
Current Behavior
It shows the QR code of walletconnect wallet but can't connect after scanning from mobile.
Expected Behavior
No response
Steps To Reproduce
No response
Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
No response
Anything else?
Minimal Reproducible In Private I can't able to share. so please cooperate.

Also, it through 81 warnings at the start of different packages like:
Failed to parse source map from '~/node_modules/json-rpc-engine/src/mergeMiddleware.ts' file: Error: ENOENT: no such file or directory, open '~/node_modules/json-rpc-engine/src/mergeMiddleware.ts'
Failed to parse source map from<EMAIL_ADDRESS>file: Error: ENOENT: no such file or directory, open<EMAIL_ADDRESS>My Project is in React.Js.

This issue has been temporarily closed due to the lack of a minimal reproducible example. Please upload a minimal reproducible example (you can use this template) and then we will reopen this issue. :)


Update to version 1.1.3 and rdkit 2022.03, 2022.09

Updated with only rdkit 2021.09, 2022.03, 2022.09

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

For any CI failing because of this kind of error:
RuntimeError: Solver could not find solution.Mamba failed to solve:
 - rdkit 2021.09.*
 - pip
 - python 3.10.* *_cpython
 - boost >=1.67
 - python
 - boost-cpp >=1.67

with channels:

The reported errors are:
- Encountered problems while solving:
-   - package rdkit-2021.09.1-py37h9710f2f_0 requires python_abi 3.7.* *_cp37m, but none of the providers can be installed
-

you should exclude them in the recipe using the skip: true selector.

@ljn917 do you want to keep building for rdkit 2021.09? Or do you think we can just remove it and keep only the latest two versions of rdkit?

@hadim I think we can remove rdkit 2021.09 because it has been built for 1.1.2 before. There is really no real changes between 1.1.2 and 1.1.3, and people are request python 3.10 which 2021.09 does not support.

LGTM for me @ljn917, ping me if ready to merge on your side.
Do you want to be added as a maintainer?

@hadim It is good to me too. Glad to co-maintain this package.

Thanks!


Update to Julia 1.0

Addresses #25 through a number of code changes. All tests passing without dependency warnings in the 0.7 and 1.0 releases. Additionally, functionality has been added to save schema and table data to file.

@roll please tag this release v0.1

@loleg
Thanks. Done

Tag should follow semver ie v0.1.0

It's better to install attobot previously before tagging so it will automatically create a PR
at https://github.com/JuliaLang/METADATA.jl/pulls


Send modal

A MODAL
TO SEND MONIES
IT'S A DRAFT
IT MAY WORK
IT MAY BE REKT
NO ONE KNOWS

I suck at haikus

Should be good now (minus BTC transactions failing, but that's a live-common issue)


Rule to check for unused local variables

Nice use of the new start/end functions.
GlobalVarChecker also has a similar list of variable names to ignore.  Your's is more complete.  We could pull that out to a common utility function or even the config at some point.

The GlobalVarChecker is doing it in a nice way as well I'll refactor it.

I think this is better - any thought? All tests pass.

Looks great.  thanks!


Builds broken on macOS

lineno 48
7z x -so "$DL/$PYTHON_PKG" Python_Framework.pkg/Payload | zcat | \     cpio -i -D "$PACKAGE/Frameworks/Python.framework"
Extracting Python framework...
Brief Usage:
  List:    cpio -it < archive
  Extract: cpio -i < archive
  Create:  cpio -o < filenames > archive
  Help:    cpio --help

Building on M1 mac w/ USE_CLANG=1

We don't really support builds under macOS. There is instead a test.sh these days that you can use to run the installer directly, as long as you have Python3 installed.


refactor: move createFromString and resource from Prism Http to CLI

This PR is a proposal of moving the methods createClientFromResource and createClientFromSpec from the prism-http package to prism-cli package
The main rationale behind this is that the dependencies of such feature (json-ref-resolver, json-ref-readers) have an hard dependency on the fs package that makes prism-http a little bit harder to embed in the browser (you need to configure web pack to mock the fs module to empty)

This is technically a breaking change since we're removing a function although we have never "marketed" our client API — I'm summoning @philsturgeon to get his opinion.
If we're going to take this as a breaking change (Prism 4), then I'd take the occasion to also

Change our artefacts directory from dist to lib — it makes more sense and does not make you feel bad when importing stuff // cc @marcelltoth — we had such discussion some time ago
Rename some of the client methods to something that makes a little bit more sense

Closes #1008
Closes stoplightio/elements/issues/261

Since we're going to ship this as a breaking change I think I will tweak a little bit the function names (the current one might not exactly make that much sense) as well as maybe rearrange the parameters. I'll look into this later today and put something online.

Change our artefacts directory from dist to lib — it makes more sense and does not make you feel bad when importing stuff // cc @marcelltoth — we had such discussion some time ago

Just my two cents here. I think if we are going to need a major bump we should take the time to really define what our public API is. Merely renaming the directory to lib and saying it's OK to import from there is disastrous in my opinion. From that point on changing any export in any file will technically be a major break. I'm sure http\dist\mocker\negotiator\internalHelpers shouldn't be part of a public library distribution. But probably http-server/getHttpConfigFromRequest neither.
Let's take a step back, decide what the goal is for each of the packages (in terms of external use), and what are the key APIs we'd like to offer to the public to achieve them.
Then put these files into somewhere (lib folder, index.ts export / top level files / whatever we decide on), keep the rest where it is, and clearly document what is part of our API and what is not.

@marcelltoth I understand where you're coming from and I would love to do that, but I do not think we can now. The public JS API will probably be done (if done) once we'll gather from more feedback from hosted Prism, Mocking Tab usage and all these product using the package.
On the other hand, we need to make Prism Http browser-compatible as soon as possible since it's blocking some stuff we're doing — and I took the occasion of the breaking change to do propose some other reorganisation — among these the idea of using lib directory as output instead of dist because it just makes us feel sick. I'm fine deferring this if it's an issue for anybody though.

@XVincentX Then I propose to keep it that way and import from dist as before.
That pushes the "risk" to the consuming packages if they import from dist. Then we sort it out in the next major version, and everyone's happy.
However if we publish a major version where everything is public, how are we going to support that? Do we release a new major every 2 days? Do we break semver? In my eyes either of these are worse than just keeping it in dist and surviving with those ugly imports until we have the time to create an actual long term solution.

We'll keep the dist directory as it is at the moment — I am fine deferring the discussion when we'll have the time.
That said, we still need this change for our platform and so apparently this change is unenviable unless we publish this as a 4 beta and use it internally, which might be ok with me and with platform.

This is not even more important since the bundle of prism-http is big.

@marcelltoth
I'll address all the comments here:

Publicly teaching users that it is OK to import stuff from /dist is not great. It'd be OK if we bundled stuff but as we are not, this makes it very difficult to define correct semver versioning.

That is very correct, and it was a mistake, I admit it. We aren't going to release this breaking change soon; most likely we'll be still on 3.x for a while. The idea is to accumulate all the breaking changes (included your very good ideas on defining a public API) and then release. So at the moment, no harms has been done

importing a package called cli feels weird. A cli feels like it is intended to be an end-user thing that you run, not something you use in other packages, right?

It is a bit weird, but I really do not see any good reason to create a prism-utils package. If we can find another part where we can paste this, I'm all for it. Maybe http-spec? I have no clue

Is /src/ in the bundle? Should it?

Eheheh ok this is complicated — essentially is a trick to use TypeScript in VSCode without having to compile the code first. I agree that's not great, but

It's in the tests — not shipped code
We can fix that by exporting stuff from the main package (which unfortunately though has some side effects, so not viable right now)


separate and group graphs on the aws vm page

grou disks, separate the others..

maybe do cpu and networking as open boxes and collapse by default the disks just for the optical sake of it?

done


Updated free-programming-books-subjects.md

Added a free PDF of Object Oriented Programming in C++
What does this PR do?
Add resource(s)
For resources
Description
Why is this valuable (or not)?
This is a great book on oops that I myself refer to.
How do we know it's really free?
You don't need an email id to access it
For book lists, is it a book? For course lists, is it a course? etc.
Its a book
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

In general, we do not link to file sharing sites, or file hosting sites such as google docs. or dropbox.


Deploying static files to AWS S3 (with Cloudfront)

Hey guys, just read few posts on copying static files to S3 CDN. Idea is to copy static files to AWS S3 after the build, so CDN will pick them up with assetPrefix without even hitting the production sever (which only role is to do SSR, without serving the static assets). I generally made it to work but this is kind of cumbersome solution:
(... install awscli and jq)
- NEXT_BUILD_ID=$( cat ./.next/BUILD_ID )
- aws s3 cp ./.next/bundles/pages s3://BUCKET_NAME/_next/${NEXT_BUILD_ID}/page --recursive --acl "public-read"
- NEXT_APP_ID=$( cat ./.next/build-stats.json | jq '."app.js".hash' -r )
- aws s3 cp ./.next s3://BUCKET_NAME/_next/${NEXT_APP_ID} --recursive --exclude "*dist/*" --exclude "*bundles/*" --acl "public-read"

To explain: as far as I understand, static bundles are in .next/bundles folder, and they are prefixed with BUILD_ID, but app.js and manifest and other files are in root of the .next dir and prefixed with hash from build-stats.json.
So, in short: NEXT_BUILD_ID is prefix for static assets, and NEXT_APP_ID is prefix for app.js, manifest and other root files (and you need jq to extract the hash from json file).
This works, however, things get even more messy since I am building a Docker container, so I need to upload those files to S3 directly from the container, after the build is done.
One proposal to solution would be to have consistent BUILD_ID, when you run consequent builds without file changes (as in build-stats.json). So you can run build on some CI server, then build the container (and BUILD_ID will be in sync, since files are not changed).
Is there any more elegant solution for this problem? Just to remind, I don't want to bother server with static files at all.
UPDATE: Here's my final solution (if there is not anything better) for guys who use AWS ECS, and want cloudfront:

Build next in production mode
Copy files to S3 as explained above
When building docker container just COPY files and run yarn install, don't run next production build in docker container

This way you save some time by not running next build in the container, and the versioning will be in sync. If you have any questions feel free to ask!
Thanks in advance!

I have next + cloudfront + s3 working on multiple projects. Please, feel free to tweet to me @ statisticsftw and I can help you get it all set up... at some point in the future, I'll write a blog post about it, but in the mean time, happy to help :)

Hey all - I wrote this up what I did to get it working with S3 + cloudfront here: https://gist.github.com/rbalicki2/30e8ee5fb5bc2018923a06c5ea5e3ea5

I have successfully deployed a dynamic React app to S3 through this guide: https://www.fullstackreact.com/articles/deploying-a-react-app-to-s3/
Is there not something equivalent you can do with Next?

@dotkas nextjs has a server thing, so you need something like ec2 to run this server. Other solution is to explore export static next app and upload it to s3. I think you are confused what "dynamic" means.

Hi, I have a legacy system that I'm trying to port to nextjs. The earlier system did the following using gulp tasks:

look at each file in a single /dist folder

which I want to now work on both /.next and /static

figure out if it has been updated or not
for the updated ones create a manifest of new file names with unique ids
upload those files on s3 and manage cloudfront with the unique names
write a manifest file json on the file system
the server will then use that manifest with a helper function to require filenames
with this approach we

didn't have to upload hundreds of files every time, only the updated ones
kept an archive of all the old files

Sharing below portions of code and output to better illustrate the process. Here is the portion of the gulpfile that did the work
const RevAll = require('gulp-rev-all');
const awspublish = require('gulp-awspublish');
const cloudfront = require('gulp-cloudfront');

const deployToCDN = () => {
  const awsCreds = JSON.parse(
    fs.readFileSync('app/config/aws-creds.json', 'utf8')
  );
  const headers = { 'Cache-Control': 'max-age=86400, no-transform, public' };
  const publisher = awspublish.create(awsCreds);

  return gulp
    .src('./app/public/dist/**/*')
    .pipe(
      rename(dirpath => {
        dirpath.dirname = 'dist/' + dirpath.dirname;
      })
    )
    .pipe(
      RevAll.revision({
        includeFilesInManifest: ['.js', '.css', '.png', '.jpg', '.ico', '.wav']
      })
    )
    .pipe(awspublish.gzip())
    .pipe(publisher.publish(headers))
    .pipe(publisher.cache())
    .pipe(awspublish.reporter())
    .pipe(cloudfront(awsCreds))
    .pipe(RevAll.manifestFile())
    .pipe(gulp.dest('./app/config'));
};

Here is sample portion of the manifest file it generated
{
  "dist/css/desk.lp.1.min.css": "dist/css/desk.lp.1.min.a134392a.css",
  "dist/css/minimal.min.css": "dist/css/minimal.min.c68d5e57.css",
  "dist/css/mob.lp.1.min.css": "dist/css/mob.lp.1.min.c442674c.css",
  "dist/images/avatar.png": "dist/images/avatar.1a061ccf.png"
}

Here is the helper function server used
const manifestPath = JSON.parse(
  fs.readFileSync('app/config/rev-manifest.json', 'utf8')
);
export const asset = env => {
  return assetPath => {
    const cdnBase = '//llalalalalala.cloudfront.net';
    if (env === 'production') {
      const cdnPath = `${cdnBase}/${
        manifestPath[`/dist/${assetPath}`.slice(1)]
      }`;
      return cdnPath;
    }
    return assetPath;
  };
};

// sample usage inside a server rendered pug file
asset('/images/avatar.png')

I am hoping to recreate the same functionality in nextjs now. As you can see it deals with more than just the upload of files on s3. Can anyone guide me to a possible solution?


[Hub Generated] Review request for Microsoft.Cache to add version stable/2021-03-01

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When you are targeting to deploy new service/feature to public regions? March 2021
When you expect to publish swagger? ASAP
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[ ] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of “no breaking changes
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[x] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[x] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

To address the following (2) code changes in the Swagger, I created a breaking change intake work item: https://msazure.visualstudio.com/One/_workitems/edit/9272990
(1): Modifying operation Id (correcting to Noun_Verb format):
BEFORE: "operationId": "Get_OperationStatus"
AFTER:   "operationId": "OperationsStatus_Get"
(2): Changing case (correcting to camelCase):
BEFORE: "x-ms-code-generation-settings": { "name": "redisenterpriseClient" }
AFTER:   "x-ms-code-generation-settings": { "name": "redisEnterpriseClient" }

@greenms Does this also fix the 'GetMethod' method naming we were noticing in the SDK the other day?

@greenms Does this also fix the 'GetMethod' method naming we were noticing in the SDK the other day?

Yes, changing "Get_OperationStatus" to "OperationsStatus_Get" fixes this.

To address the following (3) code changes in the Swagger, I created a breaking change intake work item: https://msazure.visualstudio.com/One/_workitems/edit/9272990
(1): Modifying operation Id (correcting to Noun_Verb format):
BEFORE: "operationId": "Get_OperationStatus"
AFTER: "operationId": "OperationsStatus_Get"
(2): Changing name of client class in SDK to meet conventions across Azure:
BEFORE: "x-ms-code-generation-settings": { "name": "redisenterpriseClient" }
AFTER: "title": "RedisEnterpriseManagementClient" (the x-ms-code-generation-setting extension is being deprecated)
(3): Modifying operation Id (correcting for consistency with Databases listBy operationId):
BEFORE: "operationId": "PrivateLinkResources_ListByRedisEnterpriseCache"
AFTER: "operationId": "PrivateLinkResources_ListByCluster"

/azp run


Avoid cmake_external for likely transitive dependencies

Title: cmake_external-style dependencies in Bazel universe are hard to consume transitively as they have a completely distinct reference style.
Description: Even though cmake_external-style dependencies are very tempting, they are rather painful to use transitively, imagine Envoy itself is a submodule of project Parent, say Parent depends on another Bazel-native Child library, say Child depends on c-ares. Now since Envoy already has that, it'd make sense to use the same version when building Child and Envoy. Issue is that Bazel-native and cmake_external use different reference style (//external:<lib> vs @<lib>) so now Child needs to have something like:
config_setting(
    name = "c_ares_via_cmake",
    values = {"define": "c_ares_via_cmake=enabled"},
)

and do a conditional dependency
select({
               "//conditions:default": [
                   "@com_github_c_ares_c_ares//:ares",
               ],
               "@child//:c_ares_via_cmake": [
                   "//external:ares",
               ],
           })

For every dependency basically. So not only we now need to support
def child_dependencies(
        omit_curl = False,
        omit_com_github_c_ares_c_ares = False,
        omit_boringssl = False,
        omit_net_zlib = False,
...

But also Bazel config options for each on top of that so they can be used as a cmake_external dependency.
Note that omit_xxx and the reverse FQDN dependency naming are not part of Bazel standard/conventions and are rather a best practice in the community / answer to aligning transitive dependency versions.
Example of this is TensorFlow which uses locally-authored BUILD files for most dependencies https://github.com/tensorflow/tensorflow/tree/master/third_party.

@troshko111 if an external dependency has a native Bazel build, we will use this directly. We explicitly are not in the business though of writing and maintaining BUILD files for external dependencies that don't have this, unless they are totally trivial (e.g. a simple header-only library). We also don't want to vendor/snapshot some random BUILD file, as that way lies bit rot and maintenance overhead. This was a project-level decision we made long ago.
I think TensorFlow is an anti-pattern in this regard; it's hard to imagine a useful build tool that tells you that you need to go ahead and write a build system for every transitive dependency that you have. At the end of the day, we're in the business of building proxies, not writing and maintaining build systems for other projects.
The issues around cmake_external and naming should be taken up with the Bazel team; I think we all want to see Bazel succeed at working well with external dependencies, many of which are quite content with an autoconf or cmake build.
Please reach out to @irengrig at https://github.com/bazelbuild/bazel for further discussion.
CC @jmillikin-stripe @lizan

if an external dependency has a native Bazel build, we will use this directly.

Taking this one step further, what if said dependency has its own Bazel-native dependency, already present in Envoy? Surely we'd want to reuse it making sure we don't compile independent version statically linked together. Precisely this situation has got me starting this conversation. I've been toying with a submoduled Envoy and some custom extensions which rely on first/third party Bazel-native dependencies, said dependencies have their own dependencies like rapidjson, spdlog, curl, zlib, the usual suspects. When including the top level dependency, I load and call a function like project_dependencies() and pass various skip_xxx flags so it does not load rapidjson, curl and so on, only the ones missing in Envoy. This is rather error-prone as the names must match (reverse-FQDN) and with the addition of cmake_external now each of them has to be select-ed via a config setting since the dependency string has two different formats, the @ or the //external.
So even though in this case a Bazel-native dependency was added, it only works as long as the common dependencies have matching names and is severely complicated if any non-Bazel-native ones are present (especially cmake_native).
Example above would be something like:
RootProject

Envoy submodule
Custom envoy extension code
DependencyA

DependencyA has 10 direct dependencies like json libs, curl, and so on. Some have BUILD files written for them (header libs), some more complex. Many are already defined in Envoy. When compiling RootProject it is desired to have only a single version of any given library statically linked into the resulting binary for obvious reasons. DependencyA defines a commonly used function dependency_a_dependencies accepting various omit_xxx bools so we don't copy-paste all transitive dependencies into RootProject WORKSPACE file and have an ability to skip the ones we already declare. This is rather cumberstone but seems to be the best one can do with Bazel atm. It works alright but making sure the names align, cmake_external are explicitly supported via config settings, it all adds up and makes the experience a bit rough.

I think TensorFlow is an anti-pattern in this regard; it's hard to imagine a useful build tool that tells you that you need to go ahead and write a build system for every transitive dependency that you have. At the end of the day, we're in the business of building proxies, not writing and maintaining build systems for other projects.

I personally agree with you and this is not an issue with Envoy per se, it's really the build system shortcoming, I wonder why TF went this way but seems to be working fine for them so far. They would be better to talk to various pros/cons but it seemed to me they wanted more control over dependencies of their dependencies, like in #8140, if you add curl as a direct dependency and you already have zlib, it's not even obvious curl needs one and can reuse what you have as all you do is call cmake_external and Bazel makes it happen. With a catch that that cmake could have compiled code you have, a different version of it worst case. These are hard to catch and can result in nasty crashes should objects get shared between different versions.

The issues around cmake_external and naming should be taken up with the Bazel team; I think we all want to see Bazel succeed at working well with external dependencies, many of which are quite content with an autoconf or cmake build.

Thanks I'll try to start a discussion at Bazel, I probably should have in the first place.

Issue is that Bazel-native and cmake_external use different reference style (//external: vs @) so now Child needs to have something like:

Just bind in your WORKSPACE? Many project_dependencies() checks if there's already a dependency with existing_rules().keys(), so you don't really need a select based on that.

https://github.com/bazelbuild/bazel/issues/9325
@htuch feel free to close in favor of the linked issue, I'm still curios what you think of cmake_external blackbox problem potentially compiling something you already have compiled, this would not be a problem should all dependencies have a BUILD file. Maintaining a BUILD file is however very expensive and not feasible after some project size threshold.

Just bind in your WORKSPACE? Many project_dependencies() checks if there's already a dependency with existing_rules().keys(), so you don't really need a select based on that.

Bind is basically getting deprecated is it not? Also AFAIK one cannot rebind a cmake_external dependency i a way it can be referenced using @dependency syntax unconditionally?

Bind is basically getting deprecated is it not? Also AFAIK one cannot rebind a cmake_external dependency i a way it can be referenced using @dependency syntax unconditionally?

Right it is not recommended though no alternative yet (without nested workspace, etc). With bind you can reference using //external:dependency syntax unconditionally.

@lizan yeah, but the issue is that this is viral IMHO; you basically force bind style on anyone who depends on the project. I think there is a major gap from a usability perspective unless the whole world adapts bind naming rather than direct repository naming. I think that if cmake_external could synthesize a pseudo repository with arbitrary name, that might get us to a good place.


[CHERRYPICK] Reset master state before recovering from a checkpoint

Also the bug fix for the bug introduced by resetting master state.

@aaudiber @gpang Cherry-pick this bug fix to branch-1.8

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/1895/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/1903/
Test FAILed.

@yuzhu can you fix compilation? It looks like it's due to differences in configuration between 1.8 and master

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/1905/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/1910/
Test PASSed.


Get peers list and peer info by IP

Hello. Are there any plans to add methods to API to get information about peers in configurations? It should be nice to get by API more information)
We want to provide this info to thirdparty code by API in our service)

Hi! Yes sorry I forgot to add to the doc:
GET /api/getWireguardConfigurationInfo?configurationName=

oh, thanks) It works

comment to close) /api/getWireguardConfigurationInfo?configurationName= where name is configuration name works perfect. Thank you


Unable to add items when this code is running in openshift

When running in openshift either the button or the routes may be messed up.
One can see the prepopulate data is added
https://github.com/weshayutin/todolist-mariadb-go/blob/main/todolist.go#L165
However a user can not add items through the UI.
It is possible to shell into the todolist container and post a new item.
This all works fine when running todolist.go locally against a containerized mariadb.

fixed in latest push


✨ Support iosSimulatorArm64 Kotlin Native target

📝 Description
We would like to support the iosSimulatorArm64 Kotlin Native target.
See the targets supported by Kotlin Native for more details about it.

✅ Checklist

[ ] Add the target in our Multiplatform Gradle plugin, add the target in the integration and the delivery workflows, add badge in README and update the unreleased changelog.
[ ] Close this issue as completed and update tracking ones if present.

We should attach a discussion to this issue before implementing it.

This issue will be resolved by #621.


requirements.txt: update dependencies so they work with Python 3.13

Previous PR (couldn't reopen): https://github.com/Dasharo/open-source-firmware-validation/pull/597
Works with:
telnetlib @ git+https://github.com/3mdeb/telnetlib@b85e3b5e6068eb3441cb21badb5a266301839e61

Should probably also work with https://pypi.org/project/telnetlib-313-and-up/
Quick test:
 robot -L TRACE -v config:qemu -v rte_ip:<IP_ADDRESS> -v snipeit:no dasharo-compatibility/network-boot.robot
==============================================================================
Network-Boot
==============================================================================
PXE001.001 Dasharo Network Boot is available :: This test aims to ... | PASS |
------------------------------------------------------------------------------
PXE002.001 Dasharo network boot menu boot options order is correct... | PASS |
------------------------------------------------------------------------------
PXE003.001 Autoboot option is available and works correctly :: Thi... | PASS |
------------------------------------------------------------------------------

Thanks @m-iwanicki !
I have not upgraded Fedora yet, anyone willing to confirm this?

I have upgraded in the meantime and tested in the same way as here: https://github.com/Dasharo/open-source-firmware-validation/pull/615


Send text at start support

In iTerm I use
tmux ls && read tmux_session && tmux attach -t ${tmux_session:-default} || tmux new -s ${tmux_session:-default}

for "Send text at start" to ensure I always start in tmux. Is it possible to do so in Alacritty? I haven't found anything like it so maybe it's something that is open to contriubtion.

You can achieve this using the “program” config. Start bash or zsh with -c as first argument and your snippet as second argument.

Cool I figure it would be something like that. Thanks :)


URL references in Authorization code grant flow diagram

In the Authorization code grant flow diagram above, step 2 states "http://login.windows.net/<tenant>" for the AAD authentication URL and step 3 states "http://<baseUri> for the Resource URL. The prefix for each of these should be https, so that there is no question about whether it is an encrypted connection. It is indeed an encrypted connection.

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

ID: 6ba7bfc1-7c23-5662-8f2c-b41b19fb9070
Version Independent ID: 19eb0a4a-1594-8f8b-054c-2c851df17e8b
Content: Service endpoints overview - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/dev-itpro/data-entities/services-home-page.md
Service: dynamics-ax-platform
Product: ****
Technology: ****
GitHub Login: @Sunil-Garg
Microsoft Alias: sunilg

@tomhig, thanks for making us aware of this!
@Sunil-Garg, please update the illustration.

Bug 369845 has been logged to track this issue.

Thanks, the diagram is updated.

The diagram was updated in PR 6261 and will be published soon.


Refactor Stringifying in DB-Collection

A common "parse, and then stringify" pattern was repeated everywhere.
This commit extracts this pattern into a helper function.

beauteous. in fact this is what I was doing to return copies of the data - I wonder if there's a better way?

Totes! We could use Ember.copy().


Add message about ldconfig

Checklist

[x] I have read the How to Contribute.
[x] I have run yarn build after adding my changes without getting any errors.

Updating documentation or Bugfix
I wanted to run a preview node using systemd. I followed the exact instructions, but the systemd error output showed an error. After research I found this can be remedied by simply running ldconfig after setup.

@NickJorens are you finalizing this?

@NickJorens are you finalizing this?

I'm sorry, but I wasn't planning on following up on this.

OK then @katomm we have to drop this, because the advice offered is only intermittently applicable & not without some explanatory context.


US to EU conversion failed Error: Varient conflit

Having US Technisat PQ unit sw478, I patched it to accept any metainfo with toolbox, but I can't update to EU version and it says: Error: varient conflit
Medium: Device doesn't support this (MainUnit: varient)

MST2_US_VW_PQ_P0478T.zip

attach metainfo2.txt from SD

No the firmware was never updated, it's the factory one

No it’s the unit has never been updated it’s factory firmware 0478
Le mar. 21 mars 2023 à 06:12, lprot @.***> a écrit :

Closed #240 https://github.com/olli991/mib-std2-pq-zr-toolbox/issues/240
as completed.
—
Reply to this email directly, view it on GitHub
https://github.com/olli991/mib-std2-pq-zr-toolbox/issues/240#event-8801479517,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A56BUKBVAQ4HTM54PTNPQL3W5E2FTANCNFSM6AAAAAAWBWMBOA
.
You are receiving this because you authored the thread.Message ID:
@.***
com>

Please attach metainfo2.txt you're getting the error with.

OK I solved thank you very much


Provide headers that many older packages expect.

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


Permission Error on CodeUnit Create Demonstration Data with template bcsandbox-master:base

Hi,
With a fresh new VM created with template bcinsider.azurecr.io/bcsandbox-master:base, the web/nav client report the same error on first connexion with admin account :

You do not have the following permissions on CodeUnit Create Demonstration Data: Execute.

Thanks for your help,
Xavier.

This is because the default license on the base docker images isn't up to date.
You should specify your partner license when starting the container, then the problem should go away.
Will try to get it fixed asap.

Thanks Freddy.
Dummy question : does I have a possibility to upgrade the license inside the container/docker ? (instead of recreating the VM from scratch)

It's ok ; license upgraded with CSide client and container restarted.

You also have the NavContainerHelper PowerShell prompt, which where you will find a function called Import-NavContainerLicense.


Setup code coverage reporting

When I open a PR, I would like to get a report on whether the changes will increase or decrease the current code coverage, so we can keep an eye on test coverage during the review.
I think https://github.com/apps/codecov and https://about.codecov.io/ are good solutions to look into for that.

Coveralls seems like another good option that works similarly to Codecov (but the reports aren't quite as beautiful)

Yeah, that also looks nice. Want to take another look if codecov lets you in and otherwise set up coveralls here?


Crash on 'update' non-resolved async component

Vue version
3.4.5
Link to minimal reproduction
https://play.vuejs.org/#eNqVVVtv2jAU/itHeUmQIGnX7YVBVTb1YdNuWrunZQ8ZOYCbxI5sB1oh/vuO7TgkpVs7iaqc+3cun9kHi7qOtw0G02CmlpLVGhTqpr5MOatqITW8F1UNKykqCOPECMY97Mx7uGlUjVzhGCSu4ND6OqeULwVXGgp8gLmxR2ERjrw2z3Tm1eb70SL4j5o0SNZoBPNL2KccIElAb5gia/kAm6ymsjlkK40SGGeaZSWoFgwlVaLcYhumBOwQdhnT0HDNSsqDBA6ArSAicPE2KxuqNp9DWJyHI1cPLEJvA4vxFXUFcEg5fTzWTyLLMX8aK32oFlSC6pIPbpFTyAoy9cCXsKR5Ck4qHyLZeo2S8bXxsQkqzLjJkdFmfHOkbvvLTWCvAQvfgpslbp+0SRI0VnVJAyUJYOZXZiWS7Y6nxTwNKFUawNR0SpL5R+JVaRskhe/VKO3g3ZqsxX0lS+KKJL0qs6QDEIwDrWhyK7aO75TgdHl2YGlghsFKlF9rzWiyaTD1a0iDrCzF7qPVadng2OuXG1wWT+jv1L3RpcE3GhRKOoSgs+lMrlE78/XNF7yn752xEnlTkvc/jN/N6BuD0bm9a3hOsHt+Fu0HSxBa5a26vtc0CN+UAWqPyPqnAVHFLOBvrR/hXsQX/vhoip6Lp9SVPXb6veQDZr7t+SzMJQ5Y3mnOHdfJ1/y5c6+lqBUdWo4rxvGbkaK9MRu0xRRu6Ib5euw15oQGysNomA8rIqVPd02Cin6G7uDCMYS9Kwt/daFdV5GnXFvOZIsGQTbmWPRIC684UqPNMTtOZEp8oOO2PccF3XaPC66yK9jiHXnuTFoCucCWRpZgfQ0RxSLqc2M4++d2+7l9Vk52+7/7Gg4psy8lxx2Qf8UURtL0ShBuWYWi0ZEcw/nZ2dnIhjy/yP7mWsxPb64bZIfnJQvL2fZysVjAfl8cDrPEiKeTdWObVFn96N1xhiHpaJCO3ButazVNkmXOKSzHkm1lzFEnvK6SK3JLpPlFqXCSi+rqIn4dv6H6SvfVMapq8luKHb1ElKT3UJh4+zzJiaQfM5TmGXlZ2Udh/dKPTCflu0fk8AeG+bnk
Steps to reproduce

Open the playground

What is expected?
The text is AAA data2, and there is no error message.
What is actually happening?
The text is AAA data, and an error is shown.
System Info
No response
Any additional comments?
It used to work in preview of pr #7290 .
The playground of the commit shows the result correctly.
Probably it is also regressed by this commit

A note on why we can't just do initialVNode.el = placeholder.el:
In component's render effect on initial mount, we use the presence of el to serve as a check for adopted server-rendered DOM. If el is present, it will perform hydration instead of client mount. This is why the previous fix broke Nuxt in ecosystem-ci.


Add description and initial guidance for DOTNET_HOST_PATH usage

Summary
Based on conversations in https://github.com/dotnet/runtime/issues/88754, https://github.com/dotnet/roslyn/pull/69186, and other places, we need documentation about this environment variable and its purpose. This is a draft of that documentation - though we may also need a dedicated page for "How should I locate the correct dotnet binary?" for application developers.

Internal previews

📄 File
🔗 Preview link

docs/core/tools/dotnet-environment-variables.md
.NET environment variables

@MarcoRossignoli who was solving a very similar problem for dotnet test.

though we may also need a dedicated page for "How should I locate the correct dotnet binary?"

@baronfel in the test repo we have the alg to find out correct dotnet muxer https://github.com/microsoft/vstest/blob/main/src/Microsoft.TestPlatform.CoreUtilities/Helpers/DotnetHostHelper.cs#L121
maybe we could think to ship something open like the msbuildlocator for devs.


Throw on attempt to retrieve a branch node

This will almost certainly be the most common rookie mistake. It will be really helpful to provide an error message if an attempt is made to retrieve a branch note. Unfortunately we can't do anything on an attempt retrieve a branch node on the server. Would love to discuss any ideas here.
@trxcllnt @michaelbpaulson

I believe we went against this a long time ago.


Another Flutter 1.12 BUG -

The webview_flutter: ^0.3.16 was working before but now it throws the exception Trying to embed a platform view but the PrerollContext does not support embedding. All the configurations are correct. Flutter 1.12 has been a real pain and extremely buggy. It is breaking a lot of code.

Hi @azamsharp
could you please run
flutter clean
flutter pub cache repair
If the issue persist please provide your updated
flutter doctor -v and flutter run --verbose
together with a a minimal code sample to reproduce the problem
thank you

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of 'flutter doctor -v'  please.
All system setups can be slightly different so its always better to open new issues and reference related issues.


Provide meaningful error messages to end user. Remove CompletionStage…

… from api and let quarkus determine correct thread pool. Fixes #580

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.

Error message now provides meaningful information

Rebased, and merged.
Thanks for your contribution !


"dataset.get_ids_by_features" in examples

Hi Tal et al.,
It seems there has been an update in the structure of the "dataset" class. I found the following line in the example (in neurosynth_demo.py) didn't work any more.
ids = dataset.get_ids_by_expression('emo* &~ (reward* | pain*)', threshold=0.001)
I made it work by adding "feature_table" like this:
ids = dataset.feature_table.get_ids_by_expression('emo* &~ (reward* | pain*)', threshold=0.001)
So it seems the example (neurosynth_demo.py) needs an update. Or did I something wrong?
Thanks,
Wani

Tutorial definitely needs an update, just ran into this.

Will try to get to it when I can... otherwise, a PR with a fix would be much appreciated!

It looks like this a bit bigger of a bug?
These all don't work:
dataset.get_feature_names()
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-27-0a18cdcd8bea> in <module>()
----> 1 dataset.get_feature_names()

/opt/quarantine/anaconda/4.1.1/build/lib/python2.7/site-packages/neurosynth/base/dataset.pyc in get_feature_names(self, features)
    378             return self.feature_table.get_ordered_names(features)
    379         else:
--> 380             return self.feature_table.feature_names
    381
    382     def get_feature_counts(self, threshold=0.001):

AttributeError: 'Dataset' object has no attribute 'feature_table'

dataset.get_feature_counts()
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-29-6358a94d669b> in <module>()
----> 1 dataset.get_feature_counts()

/opt/quarantine/anaconda/4.1.1/build/lib/python2.7/site-packages/neurosynth/base/dataset.pyc in get_feature_counts(self, threshold)
    383         """ Returns a dictionary, where the keys are the feature names
    384         and the values are the number of studies tagged with the feature. """
--> 385         counts = np.sum(self.get_feature_data() >= threshold, 0)
    386         return dict(zip(self.get_feature_names(), list(counts)))
    387

/opt/quarantine/anaconda/4.1.1/build/lib/python2.7/site-packages/neurosynth/base/dataset.pyc in get_feature_data(self, ids, **kwargs)
    369     def get_feature_data(self, ids=None, **kwargs):
    370         """ A convenience wrapper for FeatureTable.get_image_data(). """
--> 371         return self.feature_table.get_feature_data(ids, **kwargs)
    372
    373     def get_feature_names(self, features=None):

AttributeError: 'Dataset' object has no attribute 'feature_table'

I am also facing a problem with "dataset.get_ids_by_features". Whenever I type this in Ipython,  it says dataset object has no attribute "dataset.get_ids_by_features". I din't get any satisfactory justification for this after searching.

I haven't used neurosynth in a looong time, but I'd guess that the example is old? if you want to use the updated code (in master branch of the repo) I think you might check out this function?
https://github.com/neurosynth/neurosynth/blob/master/neurosynth/base/dataset.py#L636
If you are doing this in ipython, you should be able to type dataset. [TAB] and use tab completion to see the functions that are there.

Thank you Vanessa !
I would try out your suggestion and see what happens.
Regards,
Sujata
Thanks and regards
Sujata Sinha
On Thu, Feb 9, 2017 at 10:08 AM, Vanessa Sochat<EMAIL_ADDRESS>wrote:

I haven't used neurosynth in a looong time, but I'd guess that the example
is old? if you want to use the updated code (in master branch of the repo)
I think you might check out this function?
https://github.com/neurosynth/neurosynth/blob/master/
neurosynth/base/dataset.py#L636
If you are doing this in ipython, you should be able to type dataset.
[TAB] and use tab completion to see the functions that are there.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/neurosynth/neurosynth/issues/66#issuecomment-278545387,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARC7QfHMCAnVH8FNDjVoWRFOAhpzo3ufks5raphZgaJpZM4ITgLy
.


Web App crashes when trying to upload new program to InventorHub.

Behavior:
Web App is able to connect to the Hub. I can start the interactive shell and execute Python code. However, when trying to upload a new Python program, by pressing the "Play-Button", the app crashes. Side note: it crashes most of the time. Some times the upload works and the Python code executes on the Hub.
The enironment works fine for me on Windows (11), but I am currently switching to Fedora Linux where the Software does not work properly any more. Could you kindly assist! Bellow you find further details - if you'd need to know anything else, let me know!
Many thanks!!
Eduard
Hub: InventorHub (flashed with PyBricks Firmware on Windows)
System: Fedora 40 (Web App executes on that system)
Browser: Google Chrom Version 126.0.6478.114 (Official Build) (64-Bit)
Error Text: ChunkLoadError
Stack Trace:
ChunkLoadError
at n.f.j (main.1bf60dfd.js:2:1662589)
at main.1bf60dfd.js:2:1658083
at Array.reduce ()
at n.e (main.1bf60dfd.js:2:1658061)
at 6938.beefce17.chunk.js:1:4061
at N (main.1bf60dfd.js:2:1380738)
at Al (main.1bf60dfd.js:2:1348027)
at wc (main.1bf60dfd.js:2:1336659)
at bc (main.1bf60dfd.js:2:1336631)
at ic (main.1bf60dfd.js:2:1331653)
fs @ main.1bf60dfd.js:2
n.callback @ main.1bf60dfd.js:2
Bi @ main.1bf60dfd.js:2
El @ main.1bf60dfd.js:2
wl @ main.1bf60dfd.js:2
bl @ main.1bf60dfd.js:2
(anonymous) @ main.1bf60dfd.js:2
_c @ main.1bf60dfd.js:2
ic @ main.1bf60dfd.js:2
_ @ main.1bf60dfd.js:2
O @ main.1bf60dfd.js:2
Then after some time:
Failed to load resource: net::ERR_CONNECTION_RESET
An unknown error occurred when fetching the script.
TypeError: Failed to register a ServiceWorker for scope ('https://code.pybricks.com/') with script ('https://code.pybricks.com/service-worker.js'): An unknown error occurred when fetching the script.
(anonymous) @ main.1bf60dfd.js:2
pyodide.asm.js:1

Thanks for reporting! Can you share the program that you're trying to run?

Certainly, it's just a quickly coded snake game using the lights of the Inventor Hub and the Lego Controller (88010). Sorry, it is a bit messy and full of debug statements. The interesting thing: I continued working sending code to my Hub, but I am getting the problem much less often now.
from pybricks.hubs import InventorHub
from pybricks.pupdevices import Remote, Motor, ColorSensor, UltrasonicSensor
from pybricks.parameters import Button, Color, Direction, Port, Side, Stop
from pybricks.robotics import DriveBase
from pybricks.tools import wait, StopWatch, Matrix
from urandom import randint

WIDTH = 5
HEIGHT = 5
SNAKE_HEAD = 1
SNAKE_BODY = 2
DOT = 3
SPEED = 1000
WAIT_BUTTON_PRESSED = 10

# initaliziere Hub und Fernbedienung
hub = InventorHub()
my_remote = Remote()

# Initialize the game state
snake = [(0, 0)]
direction = (0, 1)
dot = (randint(0, WIDTH-1), randint(0, HEIGHT-1))

# Function to draw the game state
def draw(hub, row, col):
    brightness = 0

    if (row, col) in snake:
        if (row, col) == snake[0]:
            # Draw snake head
            brightness = 100
        else:
            # Draw snake body
            brightness = 40
    elif (row, col) == dot:
        # Draw dot
        brightness = 80

    hub.display.pixel(row, col, brightness)

def beep(hub):
    hub.speaker.beep()

beep(hub)
# Get user input
# Check which buttons are pressed.
pressed = my_remote.buttons.pressed()

# Main game loop
while True:
    # Draw the game state
    for row in range(HEIGHT):
        for col in range(WIDTH):
            draw(hub, row, col)

    pressed = my_remote.buttons.pressed()
    steps = int(SPEED/WAIT_BUTTON_PRESSED) + 1
    for i in range(steps):
        if len(pressed) == 0:
            pressed = my_remote.buttons.pressed()
        wait(WAIT_BUTTON_PRESSED)

    # Show the result.
    if Button.RIGHT_MINUS in pressed: # right
        direction = (0, 1)
        print("right")
    elif Button.RIGHT_PLUS in pressed: # left
        direction = (0, -1)
        print("left")
    elif Button.LEFT_PLUS in pressed: # up
        direction = (-1, 0)
        print("up")
    elif Button.LEFT_MINUS in pressed: # down
        direction = (1, 0)
        print("down")

    # Move the snake
    head = snake[0]
    new_head = (head[0] + direction[0], head[1] + direction[1])
    snake.insert(0, new_head)

    # Check if the snake has collided with the wall or itself
    if (
        new_head[0] < 0 or new_head[0] >= HEIGHT or
        new_head[1] < 0 or new_head[1] >= WIDTH or
        new_head in snake[1:]
    ):
        print("Game Over!")
        beep(hub)
        wait(100)
        beep(hub)
        break

    # Check if the snake has caught the dot
    if new_head == dot:
        # Generate a new dot
        dot = (randint(0, WIDTH-1), randint(0, HEIGHT-1))
    else:
        # Remove the tail of the snake
        snake.pop()

SystemExit()

Strangly, it didn't occur anymore - for 30' now.

Maybe strange idea:
Could there be in the current environment more radio interference?
WiFi and Bluetooth can use the same frequencies.

Hmm, actually not that strange.
Lego uses Bluethooth LE, right? BLE could be quite susceptible to radio interference,  and indeed, I had at least 7-8 devices running over WiFi. Thus, this makes total sense.
However, as far as I know are most of them in the 5Gz radio spectrum and not in 2,4Gz anymore, where Bluethooth is sending.

ChunkLoadError

This error has to do with loading .js files that contain part of the web app implementation.  It has nothing to do with Bluetooth.
There was probably a temporary network problem that went away.

For the curious: if you peek behind the scenes you can see the actual .chunk.js files. Likely one of this did not properly download the first time.

@dlech you are right, the issue is not related to the bluetooth connection. I've been running the web app for couple of times and  "This error has to do with loading .js files that contain part of the web app implementation". The problem is that the web site itself seems to have performance issues. When connecting over a browser directly to "https://code.pybricks.com/" it is getting obvious. The application does crash because timeouts occure while fetching component code or similar from the server.
At first it only happened when uploading code to the hub but now I have the problem quite often, even when starting the application. The screen goes white, sometimes the app recovers but very often the "ChunkLoadError" occurs.

Does it persist if you open the page in a new profile or private window? Or likewise, delete the site data and reload?
I wonder if it could continue to fail if it didn't load correctly the first time.
Once loaded properly, it should remain accessible offline.

Yes, this persists if I am clearing the cache as well as using a private window - I've checked the network requests in the developer tool bar. Just for code.pybricks.com every request takes ages to download. My connection to the web seems  fine otherwise.
And once everything is downloaded everything works properly.

Thanks for sharing that. Maybe we should switch to a different host.
Are you able to share which country your accessing it from? I might be able to ask our hosting provider why certain regions are experiencing slow service.
It might also help a lot if we made packaged up the docs so it would be requesting one zip file instead hundreds of small files.

Hi guys,
I have to apologize! The problem occurred due to my network setup. Strangely, only some domains were slow when being accessed. Long story short: I have fixed everything and the connection to PyBricks works perfectly fine now, and thus, the PWA.
Thanks for your support!
Eduard

No worries! I am glad it is working now. Thank you for reporting.


Fix strict prototypes of Clang 15

Add missing voids to C functions.

Coverage remained the same at 99.64% when pulling 9f9a3f985b28c72b5ecbd26c42efea063af261ff on offa:clang15 into 2342647c93f6f5e0b30ca21001449dcb55a0c276 on cpputest:master.


Bazel seems incompatible with how we version things

Principle of bazel seems to be that the binaries are only a function of the source code. But ours aren't - the dm binary also encodes in it the version of the server as a function of the git environment. Under bazel, if the dm binary isn't recompiled because the code hasn't changed, that can block changes to the server code being released (or even tested?) because the version of the server installed by, say dm cluster init depends on the version (docker image tag) baked into the dm client.
Either:

figure out how to tell bazel that there's an additional build input of the version (this will presumably make the caching less effective)
stop using bazel and go back to simple containerised builds with go caching
change how the client determines which version of the server to install somehow

"Making the caching correct" isn't "Making the caching less effective" at all :-)
The version changes come in via version_status.sh - I'd have thought that Bazel was aware that changes to the output of this script should cause a rebuild of everything that refers to variables extract from it. So perhaps the problem is in how it knows what depends on what.
cmd/dm/BUILD.bazel says:
go_library(
    name = "go_default_library",
    srcs = ["dotmesh.go"],
    importpath = "github.com/dotmesh-io/dotmesh/cmd/dm",
    visibility = ["//visibility:private"],
    deps = [
        "//cmd/dm/pkg/commands:go_default_library",
        "//vendor/github.com/opentracing/opentracing-go:go_default_library",
        "//vendor/github.com/openzipkin/zipkin-go-opentracing:go_default_library",
    ],
)

go_binary(
    name = "dm",
    embed = [":go_default_library"],
    visibility = ["//visibility:public"],
    x_defs = {"main.clientVersion": "{VERSION}",
    "main.dockerTag": "{DOCKERTAG}"},
    static = "on"
)

I'd expect it, when DOCKERTAG changes in the output of version_status.sh, to not rebuild the client (as it's built into a library), but to relink it as the x_defs inputs to the go_binary declaration have changed.
Is this a bug in x_defs, not being correctly declared as a meaningful input to the build process? I found the implementation but I don't know how detecting changes in the build process is meant to work in Bazel, so who knows?

This seems like a duplicate of https://github.com/dotmesh-io/ops/issues/85 - moving to there as @Godley's working on a fix


Update leveraged long symbol

lETH => LETH

Pull Request Test Coverage Report for Build 8423

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 8408:
0.0%

Covered Lines:
1474

Relevant Lines:
1474

💛  - Coveralls


Ombi Request Integration

Description
Ombi is a media request application, similar to Jellyseer/Overseer but it supports Jellyfin, Plex, and Emby. For those of us that run both Jellyfin and Plex (or any combination of multiple media servers), it makes sense to have a single request system so Ombi is preferred over Jellyseer/Overseer. I would like to see similar features to the other request systems that Homarr currently supports, including widget.
Priority
Medium (Would be very useful)

Have you tried to use ombi but select jr / or in the integration field ? It might just work is Ombi is a superset of these

I'll give it a try but I doubt it will work. JS/OS have similar codebases but Ombi is completely unique.

I tried as you suggested @ajnart. I configured Ombi as an app with both Jellyseer and Overseer integrations, and used the API key from Ombi. I get an error on the Media Request widget: "No requests found. Please ensure that you've configured your apps correctly."
I didn't think it would work. Ombi is not like Jellyseer/Overseer, similar to how the *arr apps use the same foundation/code base. It was worth a shot, though!

I just saw this. Thank you for trying @sirjmann92 , this stays open if someone wants to look further into it.

Yea I'd love for this to be added also

+1 for ombi integration

Thanks again for your interest. As noted in #1993, we are no longer implementing any big changes in the old code base. However, 1.0 is coming along nicely and should be available relatively soon. See https://github.com/ajnart/homarr/issues/1993#issuecomment-2362948861
Thanks for your understanding

@manuel-rw - This request wasn't specific to a version. However, if I need to submit a new request for this somewhere so that it can be considered in the new version, just let me know.

Thank you but at this time, a new request is not necessary. We will provide more information on 1.0 soon


Feature Request: Support installing additional plugins

I see that you can override basePlugins in the CR yaml by specifying spec.master.plugins with something like this:
apiVersion: virtuslab.com/v1alpha1
kind: Jenkins
metadata:
  name: example
spec:
  master:
   image: jenkins/jenkins:lts
   plugins:
     configuration-as-code:1.4:
     - configuration-as-code-support:1.4
...

However, this either uses the default plugins OR the ones specified:
https://github.com/VirtusLab/jenkins-operator/blob/a275ed62d5a3e01ef5ed5e333ce6860b84a8f05f/pkg/controller/jenkins/jenkins_controller.go#L221-L224
As far as I understand, we have two options if we want to install additional plugins or update plugins:

Override spec.master.plugins with a complete list of plugins and dependencies.  This makes it difficult to maintain if jenkins-operator has a new release that uses a new set of base plugins.
Write custom .groovy scripts that install the needed plugins.  I believe this is the current recommendation in the README.

Let me know if there is another option I'm missing.
My feature request is to add a new option spec.master.additionalPlugins that would just be merged into the default set of plugins.  This would make managing plugins a little bit simpler compared to either option I listed, since only the updated or new plugins would need to be specified.

@justyns Thanks for the feedback.
Any change to CR  under spec.maser causes restart of Jenkins master pod. Plugins installed by groovy script will have the latest version and Jenkins will restart inside a pod.
Plugins required by jenkins-operator should be separated from user plugins. I'm adding this to my TODO list.

@justyns Could you submit this issue in new repo https://github.com/jenkinsci/kubernetes-operator?

@justyns Could you submit this issue in new repo https://github.com/jenkinsci/kubernetes-operator?

Sure!  I just copied this to https://github.com/jenkinsci/kubernetes-operator/issues/1 if you want to re-add your comment as well.
I'll close this one for now, congrats on the new home for the project!


fix: run cli after installed

What kind of change does this PR introduce?
bug fix, we already use right logic for running, just don't use it after webpack-cli installation
fixes #11836
Did you add tests for your changes?
No, I don't find tests for CLI, I think we should add them
Does this PR introduce a breaking change?
No
What needs to be documented once your changes are merged?
No

Thanks

I beg for your explanation.
Actually, the error is accoured by isInstalled function, but seemed then‘s callback (require('webpack-cli')) make this catched error output.
The reason behind this phenomena is ?
I attempt to do nothint in then's callback, error missing.
I attempt to return true in IsInstalled function always, but use require('webpack-cli') in then's callback ,  error missing.
It looks like  require function make prev error output?

@yinsang We use require('webpack-cli') so node load main file and it was wrong, now we use bin file, so no problems


New dev server setup

New dev server setup:

Migrated from capistrano-shoryuken to restarting via runit's sv commands
Clean up of code related to old dev / demo servers
Fixed typo issue in MAILGUN configuration

well played


Footer copyright is not updated

Describe the bug
The copyright information is not updated for 2019.
To Reproduce
Steps to reproduce the behavior:

Update the footer in the code.

Expected behavior
The copyright should be updated as © 2019 Copyright: techOus Organization instead of © 2018 Copyright: techOus Organization
Desktop (please complete the following information):

OS: Ubuntu
Browser: Chrome

I am working on this issue.

Thank you.


Hp bar not properly substracting below 0

Too lazy for pr so here we go:
Fix is:
  get hpPercent() {
    let hpHolder = this.currentHp;
    if(hpHolder < 0) {
      hpHolder = 0;
    }

    return Math.round((100 * hpHolder) / this.maxHp);
  }

Before it calculated the following:
return Math.round((100 * this.currentHp) / this.maxHp);

maxHp = 30
If I calculate ((100 * -1) / 30) = (-100 / 30) = -3.33%
The bossbar only shows values between 0-100%.
Math 101

I was not aware of systems having hp below 0, what’s the system in question

The system in question is 3.5e by Rughalt.

As people usually don't use the death-saving rules for npc's/bosses just ignoring the whole yada yada and setting it to 0 seemed appropriate.
Great fix tho, thank you 👍


[#38] [Backend] As a user I can login with an OAuth provider

Close #38
What happened 👀

Add API endpoint to login with Google
Store session in a cookie
Store user in the database
Use the New app Directory (Beta) of Next.js 13 for rendering files

Insight 📝
Reuse old structure Next.js code
This PR mostly reuse the code from PR #2 for convenience, since the major changes of Next.js from v12 to v13 are mainly around the rendering, the client side.
API routing

In earlier versions of Next.js(before 13.2), when you created a new application, you would get a pages/api directory within the application file structure. They are called API routes.
The new Next.js v13.2 release has added a Route Handler feature to Next applications. The apis now will stay in app/api instead of pages/api. And also added support for dynamic route imports. This addition means that pages can be loaded on demand, ultimately improving performance and reducing initial load times.

(Source: https://blog.logrocket.com/using-next-js-route-handlers/)
And,

Route Handler are the equivalent of API Routes inside the pages directory meaning you do not need to use API Routes and Route Handlers together.

(Source: https://nextjs.org/docs/app/building-your-application/routing/router-handlers#api-reference)
Since the benefits of the new Route Handler are mainly around helping us render better, which is not something we need yet in this PR(which only focuses on sign-in an user), and also because there seem to be conflicts around using Route Handler & next-connect & passport(read here), I will still use theAPI Routes in this PR(which also means that I will still use the old structure pages/api). I will use the Route Handlers with the existing API Routes in the later PRs that relating to rendering.
Proof Of Work 📹

Login flow:

https://github.com/nimblehq/js-framework-benchmark/assets/76901736/413f39a8-0657-41ae-87b6-157ed54a281e

Stored Cookie:

While re-using existing code is a good idea, I am unsure if we should still keep using next-connect with Next.JS 13.
I used that package as 1) passport is THE battle-tested oAuth library in the Node community, and 2) API routes in Next.JS 12 (with the Pages router) were quite "poor" (we had to check on the HTTP verb, e.g., `if req == 'POST').
However, Next.JS 13 (with the App router) has better server-side components, including routes handlers. So we can do defining GET and POST endpoints with function GET() {} and function POST() {}. So I believe we can do more backend logic similarly to what we do with Express (see this article to do oAuth without even passport).
I am not against using https://next-auth.js.org/ if it is easier (e.g, easier to test) and better (e.g., more secure, bug-free). I want to make sure we understand our options.

Yes I see.
I just updated the PR description. What do you think about this?
And there's also this one thing. When using passport authenticate, we are using it as authService.authenticate()(req, res), which needs both request and response, but since the syntax of Route Handlers is export async function GET(request: Request) {}, we don't have the response here. Do you know if there are any other ways for us to do this? The Nextjs and it's ecosystem are still new to me so I appreciate your help a lot:D.

@tednguyendev The primary issue is that this implementation uses an outdated version of next-connect. That will not do. We should always use the latest version of libraries for security and maintenance purposes. The secondary issue is that this POC does not leverage the latest features of Next.JS 13, namely the App Router. That will also not do since the goal of the POC is to understand if Next.JS can be a good solution to build full-stack applications.
So you need to revisit the implementation to address the above concerns: 1) not using an outdated library, and 2) using the new App router. You can try to do it with Passport first and if you cannot make it work then fall back to NextAuth.

Got it! I will reimplement this and will update you later. Thank you! 😁


"Limitierung Personenzahl bei Zertifikats-Scan" not mentioned in blog

Where to find the issue

https://www.coronawarn.app/en/blog/2021-12-20-cwa-2-15/ "Corona-Warn-App simplifies certificate checks when booking tickets"
https://www.coronawarn.app/de/blog/2021-12-20-cwa-2-15/ "Corona-Warn-App vereinfacht die Zertifikatsprüfungen bei der Ticketbuchung"

Describe the issue
The Twitter announcement of CWA 2.15 https://twitter.com/coronawarnapp/status/1472840234006155269

includes the item:

"Limitierung Personenzahl bei Zertifikats-Scan"

This is not mentioned in the blog announcement for CWA 2.15 on  https://www.coronawarn.app/de/blog/2021-12-20-cwa-2-15/ "Corona-Warn-App vereinfacht die Zertifikatsprüfungen bei der Ticketbuchung".
Suggested change
Add information regarding "Limitierung Personenzahl bei Zertifikats-Scan" to CWA 2.15 English and German announcement blogs.

@dsarkar
There hasn't been any response here so far and the functionality is not documented in the blog or the FAQs.
Perhaps it would be better now to just create a new FAQ article?

@MikeMcC399 Sure.

@dsarkar
I understood that you agreed there should be an FAQ about limiting the number of certificates which can be scanned, however nothing has been written so far.
Will you follow up?

I doubt that the blog will be updated reg. this now, 4 months after the version has been released. I suggest to close this issue.

@Ein-Tim

I doubt that the blog will be updated reg. this now, 4 months after the version has been released. I suggest to close this issue.

I agree. The issue was only left open because @dsarkar indicated there would be a new FAQ. Judging by lack of reported issues on this subject it seems that an FAQ is not necessary.
Closing because the request is now outdated.


Feat: Add support for MotherDuck

@eakmanrq I'm not quite sure how multi-catalog works with MotherDuck, so I'd rather defer implementation until there's actual demand for it.


Toast Removal

Let's say you have a Materialize.toast that shows "Loading...". The code would be: Materialize.toast("Loading...")
Now, since the toast doesn't have a timeout, the toast doesn't disappear. You want to make the toast disappear in another function.
I would love to see a Materialize.toast().remove(), to remove all toasts, or maybe another function to remove the current one.

If that can help you a bit you can do that
// toast takes (message, thetime, theclass), if you set second param (time) to null toast will never dismiss automatically
Materialize.toast($message, null, 'offline-toast');
$('.offline-toast').remove();

@NitroBAY thats a nice workaround, but i really want every toast to have a default timeout (unless specified) and the next toast will only show after the previous one timed out.

I wouldn't like a default timeout, and in any way add such a feature would be compatibility breaking.

@NitroBAY according to materialize documentation you can pass the third parameter as className
Materialize.toast(message, displayLength, className, completeCallback);
So To remove the toast you can use the below method I use to dismiss Toast
Materialize.dismissToast = function(className){
  $("#toast-container ." + className).remove();
}

The above method will remove the toast from toastContainer.

Also checkout pull request https://github.com/Dogfalo/materialize/issues/1317

@smitthakkar96 I know you can pass a className, I used className in my first example to show @FoxInFlame how you can add a className to remove one or many toast (depending if you defined one class per toast or one class per all toasts)

@smitthakkar96 maybe you wanted to talk to talk @someoneelse

@NitroBAY woops didn't see that

Closed due inactivity