Silence Output

Would be nice if all output to stdout could be silenced.

Hi, @StephenThomson. Is this still an issue? Just wanted to make sure you didn't close this by accident.

Hi Greg,
Yes this is still an issue but I closed it because I forked to fix it myself and didn't want you to waste effort.
Steve

Ok, thanks! I'm going to reopen this so I don't forget about it, but since you've made a comment, we'll remember that you're working on it.
If you haven't already, you'll need to sign a CLA before we can merge any of your changes in.

Closed by @StephenThomson in #21


Updated package.json file

Updated package.json file to include new dependencies and fix minor bugs. here a little description of change made.                                -   "start": "node index.js": This adds a script to start the application by running the index.js file with Node.js.                  -  "dev": "nodemon index.js": This adds a script to start the application in development mode using Nodemon, which automatically restarts the server when changes are made to the code.

"build": "react-scripts build": This adds a script to build the React application for production. This script uses react-scripts, a package that provides a build script for React apps.
"test": "echo "Error: no test specified" && exit 1": This adds a script to run tests. This script currently just outputs an error message, but you could add actual tests to it if you wanted to.

hopefully you guys enjoy the new feature as we work hard to bring it in thanks you for my teammat who help me a lot

man it take like eternety

hi sir i slept all day my bad XD wait

No worry thanks you a lot I love the beauty of your code seriously well done. do you wish for me to shred the jspkg I change as I'm sorry I read the Telegram sheet after I install, but I fix a lot of problem
regarding the native
Envoyé depuis Proton Mail mobile
-------- Message d'origine --------
Le 16 avr. 2023 17 h 51, MrsUnkown a écrit :

hi sir i slept all day my bad XD wait
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you modified the open/close state.Message ID: @.***>

Hey I read your assay on our Instagram look at this charity it great cause that will save us all one day https://ai4good.org/ I hope you really going to donate. By the way I have hard time staying they application they only thing it miss me it at the end of your video you change app icon éxtension do I have to do that.if so can you explain to me why it only makes 3 month I'm coding and rally good at it it come narrower to me but stuff like why you have to give .exe to make run your app I have difficulty finding information about this. And by the way all the change I made in the resperatory are more to mix some people it a beautiful code you do but a really dangerous one before I start coding I was analyst at computer research threat and destroy. If you only knew what you achieve anyway I put the PHP solution on my resperatory because if you try once to build it whit PHP after that the JSON.pkg will ever be able to come back whit native and recycle jsnode anyway people who follow your instruction will still be able to run the program but for me the people who are the most dangerous his them who are stress whit time and debt anyway can you please help me out starting the program because when I I change to .exe do I have to write test.exe or I can't write whatever .Exe anyway thanks you for your patient. For your to do list look at my stuff I have really cool thing you can merge whit your tool that will make me the number one clipper stealer in the world your not far anyway take care thanks you taking time answering me. Don't try to install any of my resperatory I came the code a lot so they are super hard to install apart my crypto clipper that will be useful for you as you will add three more token you can get and that clîpper his really easy to v coffee and meet whit your we almost have same structure anyway bye
Envoyé depuis Proton Mail mobile
-------- Message d'origine --------
Le 16 avr. 2023 17 h 54, doener(ium) a écrit :

Merged #2 into main.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you modified the open/close state.Message ID: @.***>


0.2.3 Mod ERROR

Describe the Bug
Mod crashes and cannot enter the game afterwards
To Reproduce
When I enter the game and use the shortcut key to open LtnManager, the game reports an error and crashes
Save file & Username
The game save is relatively large, I saved the error code, please check
Log file
Error while running event LtnManager::on_configuration_changed
LuaGuiElement API call when LuaGuiElement was invalid.
stack traceback:
	__LtnManager__/scripts/gui/main.lua:375: in function 'close'
	__LtnManager__/control.lua:88: in function <__LtnManager__/control.lua:77>
stack traceback:
	[C]: in function '__newindex'
	__LtnManager__/scripts/gui/main.lua:375: in function 'close'
	__LtnManager__/control.lua:88: in function <__LtnManager__/control.lua:77>

The bug report template specifically states that I still need a save file, even if it is large! I cannot fix an error if I cannot reproduce it, and often times the erroring save file is the only way for me to do that!
If it is too large for GitHub, please upload it to Google Drive or Dropbox and post a link.

After closing the MOD, I can enter the saved game normally.
After entering, I created a new save file and started MOD again to read it normally.
The saved file of the attachment is the content automatically saved by the game, which cannot be read after trying before sending
google drive link:
https://drive.google.com/file/d/1_TIj9GyY8jqOPSeckB9Zjye_pPs0j_bb/view?usp=sharing

This should be fixed with the most recent version. Clicking the LTNM button will cause the GUI to be re-built if the mod detects that it was invalidated. Please give it a try!

Please try the latest version, I believe that it should fix this issue.

tks you😀


folder renaming and reorgnization

Description
This PR is used to clean the folder naming convention to be consistent with industry.
Issues
n/a
Type of change
List the type of change like below. Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[x] Breaking change (fix or feature that would break existing design and interface)

Dependencies
n/a
Tests
n/a

@chensuyue  looks like the folder naming change impacts the preci. please check it
@jfding can you please review this change?


Fix mkdocs build command in the README

In later versions of Python3, while trying to build mkdocs, we see the following error
wso2@mariangela en % mkdocs serve --dirtyreload
Traceback (most recent call last):
  File "/usr/local/bin/mkdocs", line 5, in <module>
    from mkdocs.__main__ import cli
  File "/usr/local/lib/python3.11/site-packages/mkdocs/__main__.py", line 14, in <module>
    from mkdocs import config
  File "/usr/local/lib/python3.11/site-packages/mkdocs/config/__init__.py", line 2, in <module>
    from mkdocs.config.defaults import DEFAULT_SCHEMA
  File "/usr/local/lib/python3.11/site-packages/mkdocs/config/defaults.py", line 4, in <module>
    from mkdocs.config import config_options
  File "/usr/local/lib/python3.11/site-packages/mkdocs/config/config_options.py", line 5, in <module>
    from collections import Sequence, namedtuple
ImportError: cannot import name 'Sequence' from 'collections'<EMAIL_ADDRESS>

Fixed with https://github.com/wso2/docs-apim/pull/6739


Config script fails on desktop shortcut creation

This happens in Windows 10, see also here.
Workaround: create the shortcut manually (linking to executable in C:\Users\johan\AppData\Roaming\Python\Python38\Scripts).

Try this https://github.com/KBNLresearch/ipmlab/commit/0137a5ce1470d58b2850cf916d7554212c8ab5d8 (tested for --user install, needs testing for global install)

Made above changes:
https://github.com/KBNLresearch/iromlab/commit/9d17c963121b437a579c71de5dab2c4868cb2ee6
TODO: test on Windows machine (user + global install).


Create Mesherylight.png

Description
This PR fixes #
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Invalid image.


[Backport release/3.1.x] chore(changelog): new way to maintain the changelog

Backport 17c971b51a588d26c0c85dce11784ce516971024 from #11279.

@vm-001 CI is red.


Async containerd sync

Summary of changes:
Fixes: #2627
How to test (optional):
I restarted the daemon a few times and observed no problems. I'm hoping the runtime tests cover this codepath enough.

test-me-please


[Desktop][Mobile] Create post reset values

env: https://staging.fightpandemics.work/feed
Steps:

Go to Feed
Add new post
Add post Title, text, tags and the visibility of the post
Click Request Help and then come back to Offering Help
All values were reset except post visibility. Why we reset all values except visibility?

Second question: Do we have a max size for text? Should we have it?
User can drag and expand text area, no max set.

Regarding the text area, what do you think about this approach below, @galeza ?

Hi @Joeu
I like this approach. I would ask @manualzuru and @Yu-Alexa-Zhao about it.

I agree with you @Joeu here! Cleaning the tab value can be frustrating to users indeed

So is it okay for me to proceed with the suggestion? @Yu-Alexa-Zhao , @galeza


Custom domain error: Request Envelope is invalid.

When adding a custom domain to my static web app, I am getting the following error:

Request Envelope is invalid. Please ensure the custom domain is not linked to another site.

I know that the domain name has propagated, because that would cause a different message.
I have successfully added custom domain names to other static web apps.
Static web app: victorious-smoke-00203d10f.azurestaticapps.net
Customer domain: www.kryptryx.dev
I know that it has work for this particular domain previously when I was testing, and I know that when I deleted the test static web app, I did NOT manually delete the custom domain before hand. Is it possible this has resulted in some data inconsistency?

Hey @Kryptryx,
Yes looks like something got corrupted, we will take a look!

Tried again today, and it worked!
Either you fixed something in the data, or it repaired itself.
Issue closed.


Change verification order during packaging.

Once we support packaging workspaces with dependencies, dependency packages need to be built before anything is verified. In addition to a little refactoring, this commit reorders the console messages so that package metadata (archive size, etc.) is reported before verification results.
As suggested on #13947  this, splits out the first commit (which has a lot of test output churn) as a separate PR.

@bors r+

:pushpin: Commit c0287bec8d42f89cc17cf305bcfdfb69719d64a1 has been approved by epage
It is now in the queue for this repository.

:hourglass: Testing commit c0287bec8d42f89cc17cf305bcfdfb69719d64a1 with merge a1f47ec3f7cd076986f1bfcd7061f2e8cb1a726e...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing a1f47ec3f7cd076986f1bfcd7061f2e8cb1a726e to master...


OSL-1249 Adding build pipeline for yaml linter

Task linked: OSL-1249 finish integration of mend.io into CI pipelines

@DaniilEC  could you please check the code and approve it in case you think it looks good


FINERACT-1981: Introduce Interest should not be calculated on past due principal amount on API

This P.R. introduces a new flag on loan and loan product.

@janez89 Can you please attach some related test cases?

Set value on loan product, fetch value from loan product
Loan inherit it from loan product, fetch value from loan... etc


zoom being ignored on cropper rerender/update

Suppose you have
<ImageCrop
          image={this.props.image}
          cropHeight={this.props.height}
          cropWidth={this.props.width}
          zoom={this.state.zoom}
        />

Changing the zoom in the state via .setState does not control the zooming of the cropper. For example I would like to have an "update" action in which I change the image source and I reset the zoom to 0. It works for changing the image source, but providing a new zoom doesn't seem to do anything, the new image is zoomed in as the old image was before updating.

Please provide full component. i can see that zoom is from the state of the current component as the rest are props.
This is a bug in your component/state.

Hi, it is true that the zoom is from the state and the rest are props, why is it a bug? I can't provide the full component, but I will provide a watered down version of it with the essential parts:
Basically when componentWillReceiveProps gets called I can see I have a new image. I also call set state, although maybe I could have zoom as part of the props as well, should behave the same in my opinion though.
When rendering with a new image I can see that the state's zoom is once again 0 and since the cropper is "recreated" after that I expect it to take zoom 0 (thus reset whatever zoom was done previously).
On another note it seems that setting the initial zoom to another value (100 for example) still renders the image as if zoom 0 was provided. Maybe I'm missing something here. It happens on iOS 11.
type State = { ... }
type Props = { ... }
class CropTool extends React.Component<State, Props> {
  state = { zoom: 0 };

  componentWillReceiveProps(nextProps) {
    console.log('componentdidupdate', nextProps);
    if (this.props.image !== nextProps.image) {
      console.log('setting state');
      this.setState({ zoom: 0 });
    }
  }
  onBtnPressed = () => {
      return this.cropper.crop()
        .then((base64Data) => {
          // etc
        });
  };

  cropper: { crop: () => Promise<any> };

  render() {
    console.log('rerendering. state, cropper = ', this.state, this.cropper);
    let cropper = null;
    if (this.props.image) {
      cropper =
        (<ImageCrop
          ref={(c) => { this.cropper = c; }}
          zoom={this.state.zoom}
          image={this.props.image}
          cropHeight={this.props.height}
          cropWidth={this.props.width}
          panToMove
          pinchToZoom
        />);
    }
    return (
      <View>
        {cropper}
      </View>
    );
  }
}

I dont have the time to debug this for you.
It works fine in the simple example, so it is somehow in your component.

hi again, I have written an even simpler example that shows this buggy behaviour, plus a possible, yet not pretty, workaround. I will post it here:
import * as React from 'react';
import { View, Button } from 'react-native';
import EStyleSheet from 'react-native-extended-stylesheet';
import { ImageCrop } from 'react-native-image-cropper';

const styles = EStyleSheet.create({
  container: {
    height: 500,
    width: '100%',
    alignItems: 'center'
  },
});

class Cropper extends React.Component {
  // initial state
  state = { zoom: 0 };

  resetZoom = () => {
    this.setState({zoom: 0});
  };

  render() {
    return (
      <View style={styles.container}>
        <ImageCrop
          zoom={this.state.zoom}
          image={"https://i.imgur.com/tCatS2c.jpg"}
          cropHeight={300}
          cropWidth={400}
        />
        <Button
          title="Reset zoom"
          onPress={this.resetZoom}
        />
      </View>
    );
  }
}

export default Cropper;

Zooming normally works with touch behaviour, but when trying to reset the zoom via the button it doesn't do anything, no warnings etc. The only way I was able to achieve this was by forcefully re-rendering the ImageCrop component.
I would give the component a key={someId} and would change that  whenever I intend to reset the zoom, like for example in the resetZoom method. I think it is an antipattern though, RN complains with warnings that I should not do that.
also, @stoffern unfortunately I did not find the example in this repository that you mentioned to work as expected.

did you solve this?

hi @stoffern
I only was able to solve it via the workaround involving giving a key to the component. the key would change when I want to reset the zoom, which would achieve that by re-rendering the whole ImageCrop component.

well that is just a workaround..

then I conclude I wasn't able to solve it

will look into it when i have the time..
If you can provide an example repo that is not working, it would be nice


A Simple Case Study using Wage Data from 2015

New jupyter notebook from this link.

[ ] Do we need to do all this procedure in order to get beta_hat and the residuals?

Should it not be faster if we just use the sklearn package and then after choosing the right parameters rerun the regression and project Y_hat and the residuals?

[ ] I think there is a huge difference in the regression of the flexible models using ridge regression. Do you have any idea why is that?


add CITATION.cff

This draft PR adds a CITATION.cff file. This file gets used by GitHub to populate a little "Cite This Repository" button and is used by Zenodo to populate archive metadata.  Please help by filling out your info in it including ORCID if you have one and affiliation.  See Kristina's entry for an example of formatting.  You do not need to include your email, as I think it makes sense to only include one.
Questions/comments:

The only allowed citation types are "software" or "data"
BUT, we can use a preferred-citation: that can be of any type, e.g. "book".  That will override the software citation for GitHub's "cite this" button, but I don't think Zenodo uses that information.  Maybe keeping the type as software is fine.
I'm not sure if it is most appropriate to use your current affiliation or you affiliation at the time of contribution.  Would love opinions on this
Authors are in order of: Kristina, authors from OSF citation, new members in order of joining the group.  Happy to take suggestions on a different order.  I figure leader first, every one else in order of contribution is probably fine?
We can include a version: in CITATION.cff but for this book it seemed more appropriate to just update the date-released every time we archive a snapshot of the repo.

Still TODO:

[ ] Add citation info to _quarto.yml

After this PR is merged, I'll link to Zenodo, make a release, and then add the new DOI to CITATION.cff and _quarto.yml

Thanks for doing this!

Can't the type be 'manual'? https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-citation-files#citing-something-other-than-software
Regarding authorship:

I agree it makes sense for Kristina to be first author as she has taken leadership of the group and is responsible for how it works.
I don't think it makes sense to keep the original order (me then alphabetical) and then switch to the order of joining the group; I like the idea of order of joining the group. I'd propose [Kristina, order of the group, David]. Maybe add a comment to the CFF with the protocol.

Can't the type be 'manual'?

Not according to cffr::cff_validate() or the cffinit tool

I don't think it makes sense to keep the original order (me then alphabetical) and then switch to the order of joining the group; I like the idea of order of joining the group. I'd propose [Kristina, order of joining group, David].

Sounds great to me, but I don't know the order.  Could you suggest a change in the PR?

plz add a comment to the CFF with the protocol + link to this PR

Good idea.  I'll add a note to the message: field as well as a comment linking to this discussion.

PR

type: manual is only valid for preferred-citation, not the main citation type where only software and data are allowed. don't think that preferred-citation is used for the primary citation metadata by Zenodo the way it is by GitHub. Not sure though.

I'll check in with folks at standup and try to get this merged today or tomorrow and I'll open a follow-up issue for adding metadata to _quarto.yml


feat(api-client): display which auth is required on a request

closes #3613

@cameronrohani will replace the icon later

sorry @hwkr I just augmented ComboBox again instead of pulling it out 😳

great one! how would we feel about something like the following? always felt that it could be enhanced since i don't see a use case with twice the same auth would there be?

cc @cameronrohani

great one! how would we feel about something like the following? always felt that it could be enhanced since i don't see a use case with twice the same auth would there be?
cc @cameronrohani

I like it @amritk, can we do this?

oh btw there will be 3 categories now: required auth, available auth, add new auth

Closing with a new one coming soon!


Running into local issue when running yarn build

Hey, love this package! I'm running the project locally on my server and everything works flawlessly until I ran yarn build. Here is output

yarn build Output
$ yarn build
yarn run v1.22.19
$ npm-run-all build-db check-db build-tracker build-geo build-app
$ npm-run-all copy-db-files build-db-client
$ node scripts/copy-db-files.js
Database type detected: postgresql
Copied /home/openrss/umami/db/postgresql to /home/openrss/umami/prisma
$ prisma generate
Environment variables loaded from .env
Prisma schema loaded from prisma/schema.prisma

✔ Generated Prisma Client (4.5.0 | library) to ./node_modules/@prisma/client in 193ms
You can now start using Prisma Client in your code. Reference: https://pris.ly/d/client
<code>
import { PrismaClient } from '@prisma/client'
const prisma = new PrismaClient()
</code>
$ node scripts/check-db.js
✓ DATABASE_URL is defined.
✓ Database connection successful.
✓ Database tables found.
Environment variables loaded from .env
Prisma schema loaded from prisma/schema.prisma
Datasource "db": PostgreSQL database "umami", schema "public" at "localhost:5432"

4 migrations found in prisma/migrations

Database schema is up to date!

✓ Database is up to date.
$ rollup -c rollup.tracker.config.js
/bin/sh: 1: rollup: not found
error Command failed with exit code 127.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
ERROR: "build-tracker" exited with 127.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Any thoughts?
$ node -v
v18.12.1
$ yarn -v
1.22.19

@briancao sorry to @ message you, but should this issue be labeled a bug? It doesn't seem to let me classify it. It's preventing me from using the software. Thanks for any information.

Not a problem. I'm using node v16.14 to dev locally, and our test site is on v18.9.0, so it shouldn't be a problem.
I'll have to take a deeper look.

@briancao it appears that the reason I'm running into errors like /bin/sh: 1: rollup: not found error is because I have NODE_ENV env var is set to production on my server, so none of the packages in devDependencies are being installed when running yarn install.
I must have NODE_ENV set to production on my server for other unrelated node processes, so I can't remove it, unfortunately.  Would you be willing to accept a PR that moves these devDependencies to dependencies? After all, they probably should be dependencies anyway, because they're required to run the project on production level servers.

Glad you were able to solve the problem.  I had no luck on my end trying to reproduce the issue, but I'll create a ticket on our end to look into issues with NODE_ENV further.
No PR is necessary at this time.

I think you're misunderstanding. i didnt solve the problem. I just now know why the issue exists.
You can easily reproduce the issue by cloning the repo of this project. setting NODE_ENV to production and attempting to run yarn install and then yarn build per the instructions.
Not sure why this issue was closed.

My mistake. Re-opening.

According to https://github.com/yarnpkg/yarn/issues/1830 it looks like you can simply run
$ NODE_ENV=development yarn install

I'll use that but that sets the NODE_ENV to a value that is not applicable to the environment and will cause negative side effects. https://github.com/yarnpkg/yarn/issues/1830#issuecomment-260520242 from a yarn contributor says in that same thread:

If you need your dev packages in production promote them to the dependencies group.

Why are packages that are needed in the production server declared as devDependencies? Moving them to dependencies is the less risky approach here. I also offered to open a PR to do this if you're not able.

Actually, I think I'll just hold off on using this project and just use plausible instead. Setting NODE_ENV to development in a production environment could have unintended side effects. Suggesting it as a solution makes me think there's a possible misunderstanding of how the Node ecosystem works and makes me a little nervous about the viability of the package. Anyway, thanks for your help!


Fixed build error and updated to swift 5

For whatever reason, none of the other pull requests would build for me when I tried to integrate them with my project. For this reason I updated the framework to Swift 5 myself and, after verifying it was working, are submitting a pull request.
I also tagged the new release and uploaded a zip file of the built framework to the release, allowing Carthage to just download the prebuilt framework instead of compiling it.

Closing since merge of Swift 5 upgrade


tflite runtime error with depthwise conv2D

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Custom code
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows/Ubuntu
TensorFlow installed from (source or binary): Tried source and binary
Python version: 3.6
CUDA/cuDNN version: 10/7.3
GPU model and memory: RTX 2070

Describe the current behavior
I run into the following error when trying to run a tflite model.
RuntimeError: tensorflow/lite/kernels/depthwise_conv.cc:104 params->depth_multiplier * SizeOfDimension(input, 3) != SizeOfDimension(filter, 3) (0 != 6)Node number 7 (DEPTHWISE_CONV_2D) failed to prepare.
The code runs fine normally, just not after exporting to tflite.
https://www.tensorflow.org/lite/guide/ops_compatibility -> States that the op is supported.
I know it has been used for the MobileNet variants before.
I specified rate=[1,1] and the input kernel is constant (tf.constant).
This error has also been reported in the comments of https://github.com/tensorflow/tensorflow/issues/20798
I have tried with tf v1.12.0 and v2.0.
Describe the expected behavior
In my case, the depth multiplier should be 1 and the size of the input channel dimension should be 6.
I'm not sure why it says it is 0?
Code to reproduce the issue
I will provide a small snippet of code to reproduce the error, if it is not well known.

Any update on this issue? I have a similar problem with running depthwise convolutions on a Raspberry Pi.

Please provide code to reproduce. Thanks!

@roymiles Please update as per the above comment and let us know if this issue still persists ? Thanks!

My apologies, I haven't been working with tflite for a while and I do not have the code to reproduce this problem.

@roymiles thanks for the update ,please let us know if you want to move this issue to closed status ? Thanks !


Restarting animation upon getting into view

Hey, thanks for the package, fantastic work!
Do you have any ideas regarding the possibility to restart the animation each time the child widget fully gets back into the view (after being scrolled off the screen, for example)?

Hi, thank you ! I'm glad you like it :)
Could you provide me with a reproducible example ? I can't reproduce the animation not restarting in a ListView.

No answer for a year, I'm closing the issue.


MatAutoComplete throws error if clicked outside dropdown when open.

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

Go to https://www.matblazor.com/AutocompleteList
Click on Car demo.
Display car dropdown panel
click outside autocomplete
check the browser console for the exception.

Expected behavior
No exception and the drop closed.

Kind of follow up:
I found that the issue is with CustomStringSelector.  It should have a null check before being called.
To fixed the demo, I changed the lambda to a normal method and do a null check and if null I return null.
          private string CustomSel(Car item)           {             if (item is null )             {               return null;             }             return item.Name;           }
This is a workaround, but if the component did the null check it would keep the code cleaner.


Manage time - No information who is working on the card, until stoping the timer

It would be nice if we could see who is working on a task, now until the user stops the timer there is no way to know who is working on the task.

This function would be also beneficial in a case when somebody forgets to stop the timer.

Hi @MatejSz,
Great suggestion. I'll look into this next week and implement this. Sounds like a great addition. I'll reference the merge request in this issue and ask for feedback from you (showing screenshots) before I'll release it.

Hi @MatejSz,
The functionality is now ready for 2.2
See the following screenshot: https://i.imgur.com/6ZnZOOk.png
Also added the ability to click on any running tracking's and have the ability to stop them: https://i.imgur.com/vZSBnuH.png

Super @danniehansen .
Looking forward.

@MatejSz Sorry for the delay. It's now been released with https://github.com/danniehansen/activity-timer/releases/tag/2.2.0

Thank you @danniehansen
Test OK


Feature Request: Optional key used to derive the StackSet name

Is your feature request related to a problem? Please describe.
Currently, the name key of each block in the resource section of the manifest file is used to derive the name of the CloudFormation StackSet.  Changing the name key will result in deleting the current StackSet and creating a new one.  This blocks our ability to refactor our manifest file, particularly for the blocks which builds critical infrastructure resources like our TransitGateway.
Describe the feature you'd like
I'd like to be able to directly specify or override the name used in deriving the resulting CloudFormation StackSet.  A possible solution would be to offer the use of an optional key, say stackset_name, which would then be used for naming the CloudFormation StackSet.  This would allow us refactor our manifest file, ensuring the StackSet remains undisturbed, by:

copying existing blocks
provide a new name key
use the stackset_name key to keep the previous value of name

Additional context
This could be viewed as a "lightweight" way to address some of what I raised in #88.  Refactoring a given block into two blocks would allow us to clone the underlying templates, and adjust the mix of OUs/accounts the block applies to, thereby providing us a way to roll out changes to incremental portions of our environment.
Note I do not view this as a "proper" solution to #88 as the feature being requested ere would essentially amount to humans cut-n-copying code/changes as a means of "promotion", which is not a proper SDLC.

Hey @cacack, thanks for detailing your use case for me. If I understand correctly, the feature as described is to have CfCT implement a SDLC within its deployment process. Is that correct, or have I missed the goal of what you're trying to accomplish?
CfCT was designed as a deployment tool for customizations in CT managed accounts, but isn't opinionated about the development/test/release process prior to that deployment, and that's by design because customers' release workflows differ so greatly. For example, one configuration to accomplish is to have a development, staging, and prod deployment of CfCT in separate accounts to promote tested infrastructure/code accordingly while keeping each environment suited to the relevant release stage. We recommend a route similar to that because it avoids a stateful nature for the CfCT deployment and keeps things clean and separated.
Let me know if that setup works for you, and please correct me if I've misunderstood the use case you have in mind.

Hi @cacack you should try to use this:
https://pypi.org/project/aws-control-tower-manifest-builder/
it builds your manifest file based on the data you have in the metadata of your template. it derives the name from the template file name.

Thanks @gabrielbac !  I will definitely take a look at that.  We did end up pivoting away from CfCT and back to our roots with Terraform (homegrown but borrowing from Account Factory for Terraform).


fix the default case, if we guess that we have structured post data, actually parse it.

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/7333/


Unable to modify dictionary-like private attributes

Initial Checks

[X] I confirm that I'm using Pydantic V2 installed directly from the main branch, or equivalent

Description
In v2 I cannot update a dictionary private attribute in __init_subclass__ , instead I receive a TypeError: TypeError: 'ModelPrivateAttr' object does not support item assignment.
This was a regression in behavior from v1 where the code worked.
Example Code
from pydantic import BaseModel

class Model(BaseModel):
    _registry: dict[str,str] = {}

    def __init_subclass__(cls):
        cls._registry[cls.__name__] = cls

class SubModel(Model):
    pass

### Python, Pydantic & OS Version

```Text
pydantic version: 2.0.1
        pydantic-core version: 2.0.2 release build profile
                 install path: <redacted>
               python version: 3.11.1 (tags/v3.11.1:a7a450f, Dec  6 2022, 19:58:39) [MSC v.1934 64 bit (AMD64)]
                     platform: Windows-10-10.0.19045-SP0
     optional deps. installed: ['typing-extensions']

Selected Assignee: @davidhewitt

It looks like you want a ClassVar, rather than a PrivateAttr:
from typing import ClassVar

from pydantic import BaseModel

class Model(BaseModel):
    _registry: ClassVar[dict[str,str]] = {}

    def __init_subclass__(cls):
        cls._registry[cls.__name__] = cls

class SubModel(Model):
    pass

Annotations whose names start with a leading _ are automatically converted to ModelPrivateAttrs, but that can be prevented by annotating as a ClassVar, and I think that's what you want here.
I'm going to close this issue but please let us know if the above change doesn't fix your problem.


BaseCollectiveExecutor::StartAbort Invalid argument: Upper bound check fail is reported with CollectiveAllReduceStrategy

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):  CentOS Linux release 7.4.1708 (Core)
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: No
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): ('v1.13.1-0-g6612da8951', '1.13.1')
Python version: 2.7
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: 10.0
GPU model and memory: Tesla P100, 16G

You can collect some of this information using our environment capture script
You can also obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the current behavior
When running transformer within the tensor2tensor library with CollectiveAllReduceStrategy via little customized change for distributed training in t2t only, it reported below error:
INFO:tensorflow:Graph was finalized.
WARNING:tensorflow:From /home/xh/tf-1.13.1/lib/python2.7/site-packages/tensorflow/python/training/saver.py:1266: checkpoint_exists (from tensorflow.python.training.checkpoint_management) is deprecated and will be removed in a future version.
Instructions for updating:
Use standard file APIs to check for files with this prefix.
INFO:tensorflow:Restoring parameters from /home/xh/datasets/t2t_train_enzh_multi-gpu-ok/translate_enzh_wmt32k/transformer-transformer_base/model.ckpt-179000
2019-03-14 15:30:36.585213: I tensorflow/core/distributed_runtime/master_session.cc:1192] Start master session 4c2e055e28f41c2f with config: device_filters: "/job:worker/task:0" device_filters: "/job:worker/task:0" gpu_options { per_process_gpu_memory_fraction: 0.95 } allow_soft_placement: true graph_options { optimizer_options { global_jit_level: OFF } rewrite_options { scoped_allocator_optimization: ON scoped_allocator_opts { enable_op: "CollectiveReduce" enable_op: "CollectiveReduce" enable_op: "CollectiveReduce" enable_op: "CollectiveReduce" } } } experimental { collective_group_leader: "/job:worker/replica:0/task:0" }
WARNING:tensorflow:From /home/xh/tf-1.13.1/lib/python2.7/site-packages/tensorflow/python/training/saver.py:1070: get_checkpoint_mtimes (from tensorflow.python.training.checkpoint_management) is deprecated and will be removed in a future version.
Instructions for updating:
Use standard file utilities to get mtimes.
INFO:tensorflow:Running local_init_op.
INFO:tensorflow:Done running local_init_op.
INFO:tensorflow:Initialize strategy
INFO:tensorflow:Saving checkpoints for 179000 into /home/xh/datasets/t2t_train_enzh_multi-gpu-ok/translate_enzh_wmt32k/transformer-transformer_base/model.ckpt.
2019-03-14 15:32:18.009875: I tensorflow/stream_executor/dso_loader.cc:152] successfully opened CUDA library libcublas.so.10.0 locally
2019-03-14 15:32:28.447121: I tensorflow/core/kernels/data/shuffle_dataset_op.cc:101] Filling up shuffle buffer (this may take a while): 387 of 512
2019-03-14 15:32:31.537519: I tensorflow/core/kernels/data/shuffle_dataset_op.cc:140] Shuffle buffer filled.
2019-03-14 15:32:31.695275: W tensorflow/core/common_runtime/base_collective_executor.cc:203] BaseCollectiveExecutor::StartAbort Invalid argument: Upper bound check fail for input 1 from node training/gradients/transformer/parallel_0_5/transformer/transformer/symbol_modality_32610_512_1/softmax/concat_grad/Slice_7 to node scoped_allocator_concat_232 input bounds = [0x7fb060a84f00, 0x7fb060e7ff00] backing_tensor bounds = [0x7faf8da62700, 0x7faf8fe35700]
[[{{node scoped_allocator_concat_232}}]]
2019-03-14 15:32:31.708208: W tensorflow/core/common_runtime/base_collective_executor.cc:203] BaseCollectiveExecutor::StartAbort Invalid argument: Upper bound check fail for input 1 from node training/gradients/transformer/parallel_0_5/transformer/transformer/symbol_modality_32610_512_1/softmax/concat_grad/Slice_7 to node scoped_allocator_concat_232 input bounds = [0x7fb060a84f00, 0x7fb060e7ff00] backing_tensor bounds = [0x7faf8da62700, 0x7faf8fe35700]
[[{{node scoped_allocator_concat_232}}]]
[[{{node _send_add_1_0}}]]
InvalidArgumentError: InvalidA...ntError()
Describe the expected behavior
What does above error log mean, and is there any way to fix it? thanks.
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

Also seeing this bug when using tf.contrib.distribute.CollectiveAllReduceStrategy, any information on what it means?

Any update on this issue? Even I am facing the exact same error. @isaacnoble @jvishnuvardhan

Ping @poxvoculi; looks like a ScopeAllocator issue.

@poxvoculi by any chance did you get time to look into this issue? @byronyi any workaround you can suggest for this?

Pinging @dubey too

Even I'm facing the same issue. Do update us as soon as you find a solution.

Even I'm facing the same with tensorflow version=1.11 Python=3.6

Thanks for filing the issue.  589fb0dfdb fixed a similar issue in ScopedAllocator.  Can you try running with a nightly build and let us know if you still see this bug?  If yes, please also post instructions on how to reproduce.

Thanks for filing the issue. 589fb0dfdb fixed a similar issue in ScopedAllocator. Can you try running with a nightly build and let us know if you still see this bug? If yes, please also post instructions on how to reproduce.

Hi @dubey ,
I have a constraint where I need to work on conda tensorflow v1.11
Following are the steps to reproduce the error with tensorflow v1.11

Run the worker1.ipynb on node 1
worker1.zip

Run the worker2.ipynb on node 2
worker2.zip

Note: Do modify the TF_Config variable according to your cluster specs in both the jupyter files
You will get errors https://gist.github.com/vishald527/5c464addba66188625ea0088853db970 and https://gist.github.com/vishald527/4c06f908651cae57b8a9f72996ea834d on node 1 and 2 respectively.
After making the changes as suggested in https://github.com/tensorflow/tensorflow/commit/f10b00558de87020554c9c0512537dab96dba918 re-run the jupyter files.
Now you will get this https://gist.github.com/vishald527/616c1dc32f20d8226d712f4552b025fa error on node 1 and this https://gist.github.com/vishald527/d47696e61c3f31f4d3063ff48dffaea7 log on node 2.
Post making the changes suggested here https://github.com/tensorflow/tensorflow/commit/e692dda4c8b199555e2fa32132a7784e0893c870 re-run the jupyter files.
Now you will get this https://gist.github.com/vishald527/b1a0d7cb436f70902594dbd742701794 error on node 1 and this https://gist.github.com/vishald527/6dc8b8719b4a67e0ee6dabd3a61ad779 on node 2.
We even tried building tensorflow v1.11 after making the changes you had suggested here https://github.com/tensorflow/tensorflow/commit/589fb0dfdb694d6318784e1523672fc2833b736d, but we are again getting the same Upper/Lower bound check fail error on both nodes.
Request you to please suggest what fixes are to be done in tensorflow v1.11 to get rid of this error and get CollectiveAllReduceStrategy up and running.

@vishald527 I am not a member of the TF team but I believe they do not accept bugfix for experimental features, a.k.a. those in the contrib folder.
CollectiveAllReduceStrategy was an experimental feature in r1.11, so you probably need to cherry pick the bugfix patches, and backport them to r1.11. I am not sure how hard it is though.
Please correct me if I am wrong.

@byronyi got your point but how to get a patch for this particular error? i am getting this error in v1.11 and the person who raised this issue got the same error in v1.13.
The patch for this error which @dubey had mentioned above didn't work.
Also, do we say that currently tensorflow does not support a stable multi node gpu based synchronous distributed training strategy at all?

@byronyi got your point but how to get a patch for this particular error? i am getting this error in v1.11 and the person who raised this issue got the same error in v1.13.
The patch for this error which @dubey had mentioned above didn't work.

I do not know. I would suggest you to upgrade your TF version.

Also, do we say that currently tensorflow does not support a stable multi node gpu based synchronous distributed training strategy at all?

It depends on your definition to "stable". It was an experimental feature in r1.11, it is still an experimental feature right now. Use it at your own risk.

I do not know. I would suggest you to upgrade your TF version.

TF v1.13 already has the same error and TF v2.0 does not have this strategy..so not sure which TF version you are suggesting to upgrade.

It depends on your definition to "stable". It was an experimental feature in r1.11, it is still an experimental feature right now. Use it at your own risk.

By stable I meant a strategy which would at least start synchronous distributed training on a multiple node gpu based system. Currently we are in a situation where we don't have a single strategy which even starts distributed training on multiple nodes synchronously. Please do correct me if I am wrong.
Also is there any method you are aware of by which we can run distributed training on gpu based multiple node system?

TF v1.13 already has the same error and TF v2.0 does not have this strategy..so not sure which TF version you are suggesting to upgrade.

CollectiveAllReduce is exposed as tf.distribute.experimental.MultiWorkerMirroredStrategy in TF 2.0. See here for details.

Also is there any method you are aware of by which we can run distributed training on gpu based multiple node system?

Apart from MultiWorkerMirroredStrategy, you could try ParameterServerStrategy as well. They are both in the tf.distribute.experimental subpackage.

CollectiveAllReduce is exposed as tf.distribute.experimental.MultiWorkerMirroredStrategy in TF 2.0. See here for details.

TF 2.0 i guess requires CUDA 10 and I have CUDA 9.2 installed. Upgrading CUDA is not an option for me right now and that is why I need help on fixing this reported issue for TF 1.11/1.13. Any pointers on who might be able to help resolving this issue?

Apart from MultiWorkerMirroredStrategy, you could try ParameterServerStrategy as well. They are both in the tf.distribute.experimental subpackage.

ParameterServerStrategy is an asynchronous technique. I am looking for a synchronous one.

TF 2.0 i guess requires CUDA 10 and I have CUDA 9.2 installed. Upgrading CUDA is not an option for me right now and that is why I need help on fixing this reported issue for TF 1.11/1.13. Any pointers on who might be able to help resolving this issue?

You could recompile your TF using CUDA 9.2.

You could recompile your TF using CUDA 9.2.

Will try that too.
@dubey any luck on finding the cause and fix for this issue?

Apologies for the delay, this was a bit tricky to debug and fix.  I just submitted a3d2ee6 which should fix the bounds check failure.  Let us know if it works for you.

@dubey do I need to make both the code changes you had given and then try or only the latest one?

I'd recommend patching both changes.

@dubey
I used a recent built(6 days ago) nightly-gpu docker image, this error still exists

@royxue thanks for the update.  Can you post the error message you get with the recent image, as well as instructions to reproduce the error?

I will try run again with the most recent docker image, and get back to you the result

@dubey Here is the logs.
I slightly modified the TF object detection. And run the collective allreduce strategy with 4 workers (1 gpu per worker) on k8s.
2019-04-24 03:09:04.405555: W tensorflow/core/common_runtime/base_collective_executor.cc:215] BaseCollectiveExecutor::StartAbort Invalid argument: Upper bound check fail for input 1 from node gradients/AddN_20 to node scoped_allocator_concat_100_16 input bounds = [0x7fdbfcfb0f00, 0x7fdbfcff8f00] backing_tensor bounds = [0x7fdbea25fd00, 0x7fdbea2f7900]
	 [[{{node scoped_allocator_concat_100_16}}]]
	 [[global_norm/global_norm_G6542]]
2019-04-24 03:09:04.405608: E tensorflow/core/common_runtime/ring_alg.cc:279] Aborting RingReduce with Invalid argument: Upper bound check fail for input 1 from node gradients/AddN_20 to node scoped_allocator_concat_100_16 input bounds = [0x7fdbfcfb0f00, 0x7fdbfcff8f00] backing_tensor bounds = [0x7fdbea25fd00, 0x7fdbea2f7900]
	 [[{{node scoped_allocator_concat_100_16}}]]
	 [[global_norm/global_norm_G6542]]
2019-04-24 03:09:04.405617: W tensorflow/core/common_runtime/base_collective_executor.cc:215] BaseCollectiveExecutor::StartAbort Invalid argument: Upper bound check fail for input 1 from node gradients/AddN_20 to node scoped_allocator_concat_100_16 input bounds = [0x7fdbfcfb0f00, 0x7fdbfcff8f00] backing_tensor bounds = [0x7fdbea25fd00, 0x7fdbea2f7900]
	 [[{{node scoped_allocator_concat_100_16}}]]
	 [[global_norm/global_norm_G6542]]
2019-04-24 03:09:04.405728: E tensorflow/core/common_runtime/ring_alg.cc:279] Aborting RingReduce with Invalid argument: Upper bound check fail for input 1 from node gradients/AddN_20 to node scoped_allocator_concat_100_16 input bounds = [0x7fdbfcfb0f00, 0x7fdbfcff8f00] backing_tensor bounds = [0x7fdbea25fd00, 0x7fdbea2f7900]
	 [[{{node scoped_allocator_concat_100_16}}]]
	 [[global_norm/global_norm_G6542]]
2019-04-24 03:09:04.405741: W tensorflow/core/common_runtime/base_collective_executor.cc:215] BaseCollectiveExecutor::StartAbort Invalid argument: Upper bound check fail for input 1 from node gradients/AddN_20 to node scoped_allocator_concat_100_16 input bounds = [0x7fdbfcfb0f00, 0x7fdbfcff8f00] backing_tensor bounds = [0x7fdbea25fd00, 0x7fdbea2f7900]
	 [[{{node scoped_allocator_concat_100_16}}]]
	 [[global_norm/global_norm_G6542]]
2019-04-24 03:09:04.405823: W tensorflow/core/framework/op_kernel.cc:1455] OP_REQUIRES failed at collective_ops.cc:223 : Invalid argument: Upper bound check fail for input 1 from node gradients/AddN_20 to node scoped_allocator_concat_100_16 input bounds = [0x7fdbfcfb0f00, 0x7fdbfcff8f00] backing_tensor bounds = [0x7fdbea25fd00, 0x7fdbea2f7900]
	 [[{{node scoped_allocator_concat_100_16}}]]
	 [[global_norm/global_norm_G6542]]
2019-04-24 03:09:04.405885: W tensorflow/core/framework/op_kernel.cc:1455] OP_REQUIRES failed at collective_ops.cc:223 : Invalid argument: Upper bound check fail for input 1 from node gradients/AddN_20 to node scoped_allocator_concat_100_16 input bounds = [0x7fdbfcfb0f00, 0x7fdbfcff8f00] backing_tensor bounds = [0x7fdbea25fd00, 0x7fdbea2f7900]
	 [[{{node scoped_allocator_concat_100_16}}]]
	 [[global_norm/global_norm_G6542]]
2019-04-24 03:09:04.408029: W tensorflow/core/common_runtime/base_collective_executor.cc:215] BaseCollectiveExecutor::StartAbort Invalid argument: Upper bound check fail for input 1 from node gradients/AddN_20 to node scoped_allocator_concat_100_16 input bounds = [0x7fdbfcfb0f00, 0x7fdbfcff8f00] backing_tensor bounds = [0x7fdbea25fd00, 0x7fdbea2f7900]
	 [[{{node scoped_allocator_concat_100_16}}]]
Traceback (most recent call last):
  File "/models/object_detection/model_main.py", line 127, in <module>
    tf.app.run()
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/platform/app.py", line 40, in run
    _run(main=main, argv=argv, flags_parser=_parse_flags_tolerate_undef)
  File "/usr/local/lib/python2.7/dist-packages/absl/app.py", line 300, in run
    _run_main(main, args)
  File "/usr/local/lib/python2.7/dist-packages/absl/app.py", line 251, in _run_main
    sys.exit(main(argv))
  File "/models/object_detection/model_main.py", line 123, in main
    tf.estimator.train_and_evaluate(estimator, train_spec, eval_specs[0])
  File "/usr/local/lib/python2.7/dist-packages/tensorflow_estimator/python/estimator/training.py", line 464, in train_and_evaluate
    estimator, train_spec, eval_spec, _TrainingExecutor)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/distribute/estimator_training.py", line 290, in train_and_evaluate
    session_config=run_config.session_config)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/distribute/distribute_coordinator.py", line 853, in run_distribute_coordinator
    task_id, session_config, rpc_layer)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/distribute/distribute_coordinator.py", line 360, in _run_single_worker
    return worker_fn(strategy)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/distribute/estimator_training.py", line 252, in _worker_fn
    hooks=hooks)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow_estimator/python/estimator/estimator.py", line 362, in train
    loss = self._train_model(input_fn, hooks, saving_listeners)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow_estimator/python/estimator/estimator.py", line 1152, in _train_model
    return self._train_model_distributed(input_fn, hooks, saving_listeners)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow_estimator/python/estimator/estimator.py", line 1215, in _train_model_distributed
    self._config._train_distribute, input_fn, hooks, saving_listeners)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow_estimator/python/estimator/estimator.py", line 1325, in _actual_train_model_distributed
    saving_listeners)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow_estimator/python/estimator/estimator.py", line 1402, in _train_with_estimator_spec
    estimator_spec, worker_hooks, saving_listeners)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow_estimator/python/estimator/estimator.py", line 1348, in _train_with_estimator_spec_distributed
    _, loss = mon_sess.run([estimator_spec.train_op, estimator_spec.loss])
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/monitored_session.py", line 754, in run
    run_metadata=run_metadata)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/monitored_session.py", line 1252, in run
    run_metadata=run_metadata)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/monitored_session.py", line 1353, in run
    raise six.reraise(*original_exc_info)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/monitored_session.py", line 1338, in run
    return self._sess.run(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/monitored_session.py", line 1411, in run
    run_metadata=run_metadata)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/monitored_session.py", line 1169, in run
    return self._sess.run(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 948, in run
    run_metadata_ptr)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1171, in _run
    feed_dict_tensor, options, run_metadata)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1348, in _do_run
    run_metadata)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1368, in _do_call
    raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.InvalidArgumentError: Upper bound check fail for input 1 from node gradients/AddN_20 to node scoped_allocator_concat_100_16 input bounds = [0x7fdbfcfb0f00, 0x7fdbfcff8f00] backing_tensor bounds = [0x7fdbea25fd00, 0x7fdbea2f7900]
	 [[{{node scoped_allocator_concat_100_16}}]]
	 [[global_norm/global_norm_G6542]]

Can you provide a bare minimum test case that reproduces this error?
For some context, the scoped allocator is a static optimization that converts multiple collective all-reduce ops in to a single all-reduce.  In this optimization, we pre-allocate a buffer that is large enough to hold all the inputs to the all reduce.  We add an attribute to each OpKernel whose output takes part in this all-reduce indicating that the output should be assigned from a slice of this pre-allocated buffer.  The bounds check failure is a runtime error that indicates that the OpKernel somehow circumvented this process and used a different buffer.
a3d2ee6 fixed the case when an OpKernel calls set_output instead of allocate_output.
In this example, it looks like the input to the all-reduce comes from an AddN op.  I tried a small test case which fuses 2 ops using the scoped allocator whose inputs are both AddN, but my test case passed.

It’s not AddN, the node name is common for gradients that feed into training ops (GradientDescent etc.)
Would it be possible for this to be an interfering case between scoped allocator and some other graph rewrites?

@dubey
I was checking the solution yesterdays. If I started training from a checkpoint, this error happened. But is I started training from scratch this error will not happen. However the training process will hang up after the graph is initialized. Im thinking this is may caused by the conflicts between slim.argscope(object detection api used) and distributed strategy. As mentioned, the strategy do not support deprecated slim api.

Is this resolved or is it still an issue?

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

Closing due to lack of recent activity. Please update the issue when new information becomes available, and we will open a new issue. Thanks!


Azure App Service Deploy Task - Could not find the End of Central Directory Record

The task has been running for several weeks, but suddenly stopped with the following error:
2017-08-14T09:32:31.2397851Z ##[section]Starting: Azure App Service Deploy: xxxx
2017-08-14T09:32:31.2410405Z ==============================================================================
2017-08-14T09:32:31.2410405Z Task         : Azure App Service Deploy
2017-08-14T09:32:31.2410405Z Description  : Update Azure Web App Services, Web App On Linux , Function Apps, Mobile Apps using Web Deploy / Kudu REST APIs
2017-08-14T09:32:31.2410405Z Version      : 3.3.13
2017-08-14T09:32:31.2410405Z Author       : Microsoft Corporation
2017-08-14T09:32:31.2410405Z Help         : More Information
2017-08-14T09:32:31.2410405Z ==============================================================================
2017-08-14T09:32:34.3473460Z Got connection details for Azure App Service:'xxxx'
2017-08-14T09:32:36.3641276Z ##[error]Error: Could not find the End of Central Directory Record

@larserikfinholt ,  can you please set system.debug variable to true, re-run the release and share the logs?

@larserikfinholt , can you please share the logs for investigation?

Closing this thread as there is no response from customer.
Please re-open the thread, if required and provide Debug logs too.

2017-12-27T21:49:47.5916921Z Got connection details for Azure App Service:'sia-reconvener-ppe'
2017-12-27T21:49:47.5917759Z ##[debug]Performing the deployment of webapp.
2017-12-27T21:49:47.5919180Z ##[debug]Finding files matching input: d:\a\r1\a\SIA_Reconvener_CI\dist
2017-12-27T21:49:47.5921520Z ##[debug]set reconvenerUrl=http://sia-reconvener-ppe.azurewebsites.net
2017-12-27T21:49:47.5939404Z ##[debug]Processed: ##vso[task.setvariable variable=reconvenerUrl;secret=false;]http://sia-reconvener-ppe.azurewebsites.net
2017-12-27T21:49:47.9778782Z ##[debug]Azure web app is available.
2017-12-27T21:49:50.9798699Z ##[debug]Initiated deployment via kudu service for webapp package : d:\a\r1\a\SIA_Reconvener_CI\dist
2017-12-27T21:49:50.9801189Z ##[debug]Requesting for Auth Token: https://login.windows.net/********/oauth2/token/
2017-12-27T21:49:51.1269028Z ##[debug]Requesting Azure App Service web config Details: https://management.azure.com/subscriptions/a83b1dd0-478c-4fe4-af6e-e71d94058e4a/resourceGroups/sia-reconvener-dev/providers/Microsoft.Web/sites/sia-reconvener-ppe/config/web?api-version=2016-08-01
2017-12-27T21:49:52.2177180Z ##[error]Failed to deploy web package to App Service.
2017-12-27T21:49:52.2177464Z ##[debug]Processed: ##vso[task.issue type=error;]Failed to deploy web package to App Service.
2017-12-27T21:49:52.2178102Z ##[debug]task result: Failed
2017-12-27T21:49:52.2189576Z ##[error]Error: Error: Could not find the End of Central Directory Record
2017-12-27T21:49:52.2190009Z ##[debug]Processed: ##vso[task.issue type=error;]Error: Error: Could not find the End of Central Directory Record
2017-12-27T21:49:52.2213204Z ##[debug]Processed: ##vso[task.complete result=Failed;]Error: Error: Could not find the End of Central Directory Record
2017-12-27T21:49:52.2213714Z ##[debug]Application Insights is not configured for the App Service sia-reconvener-ppe. Skipping adding release annotation.
2017-12-27T21:49:52.2214094Z ##[debug]Web App Publish Kudu URL: sia-reconvener-ppe.scm.azurewebsites.net:443
2017-12-27T21:49:52.2214406Z ##[debug]build.sourceVersionAuthor=undefined
2017-12-27T21:49:52.2214712Z ##[debug]build.requestedfor=Strand McCutchen
2017-12-27T21:49:52.2215019Z ##[debug]build.buildUri=vstfs:///Build/Build/8853
2017-12-27T21:49:52.2215347Z ##[debug]release.releaseUri=vstfs:///ReleaseManagement/Release/2137
2017-12-27T21:49:52.2215923Z ##[debug]build.buildId=8853
2017-12-27T21:49:52.2216226Z ##[debug]release.releaseId=2137
2017-12-27T21:49:52.2216507Z ##[debug]build.buildNumber=8853
2017-12-27T21:49:52.2217044Z ##[debug]release.releaseName=Release-10
2017-12-27T21:49:52.2217387Z ##[debug]system.TeamFoundationCollectionUri=https://azsrels.visualstudio.com/
2017-12-27T21:49:52.2217732Z ##[debug]system.teamProjectId=1ab87a75-b7f1-450e-a912-2bb3bafd591d
2017-12-27T21:49:52.2218087Z ##[debug]build.sourceVersion=dafc699e50fe62e141a9d28edc2579e6dc900200
2017-12-27T21:49:52.2218425Z ##[debug]build.repository.name=SIA_Reconvener
2017-12-27T21:49:52.2218746Z ##[debug]build.repository.provider=TfsGit
2017-12-27T21:49:52.2219055Z ##[debug]build.buildUri=vstfs:///Build/Build/8853
2017-12-27T21:49:52.2219399Z ##[debug]release.releaseUri=vstfs:///ReleaseManagement/Release/2137
2017-12-27T21:49:52.2219853Z ##[debug]build.buildId=8853
2017-12-27T21:49:52.2220290Z ##[debug]release.releaseId=2137
2017-12-27T21:49:53.5667906Z Successfully updated deployment History at https://sia-reconvener-ppe.scm.azurewebsites.net/api/deployments/21371514411392213
2017-12-27T21:49:53.5803809Z ##[section]Finishing: Azure App Service Deploy: sia-reconvener-ppe
2017-12-27T21:49:53.5862532Z ##[section]Finishing: Release

After a bit of pain I discovered that my app was lacking a web.config file. I believe this may be the underlying cause of the original poster's issue. I will investigate adding error handling which indicates that this file is not getting loaded in the environment, and the user may be able to resolve the issue by following the instructions here: https://github.com/projectkudu/kudu/wiki/Using-a-custom-web.config-for-Node-apps

@strand , the issue is with compressing the folder to a zip file prior deployment and not with web.config file. Are you using private agent or Hosted agent - (windows / Linux) ?

I was working with Linux initially, and tried switching to Windows, then successfully deployed to Windows. I was not using zip to get a successful build, instead once a build is successful, my release process retrieves a git repo. If you would like I could spend some time writing out complete reproduction steps later this week.


Add support for azure resource manager service connections using managed identity

Hello,
I currently have a pipeline running fine with a SPN on a self hosted agent.
I would like to switch to a managed identity (VM)
I setup a new Service Connection in azure Devops, and got a simple powershell pipeline to work to validate the managed identity is fine.
Now with TerraformCLI@0, terraform init fails with this error :
##[error]Terraform backend initialization for AzureRM only support service principal authorization
(nothing more useful when I activate TF_LOG TRACE)
Does TerraformCLI@0 support managed identity ?
For info : I have also tried using MS's TerraformTaskV1@0
I get an error too, but different, where I think it is "badly" trying to authenticate will a null identifier instead of using MSI
(there is "-backend-config=arm_client_id=null -backend-config=arm_client_secret=null" in the command line....)
Failed to get existing workspaces: Error retrieving keys for Storage Account "xxx": azure.BearerAuthorizer#WithAuthorization: Failed to refresh the Token for request to https://management.azure.com/subscriptions/xxx/resourceGroups/xxx/providers/Microsoft.Storage/storageAccounts/xxx/listKeys?api-version=2016-01-01: StatusCode=400 -- Original Error: adal: Refresh request failed. Status Code = '400'. Response body: {"error":"unauthorized_client","error_description":"AADSTS700016: Application with identifier 'null' was not found in the directory '***'. This can happen if the application has not been installed by the administrator of the tenant or consented to by any user in the tenant. You may have sent your authentication request to the wrong tenant.\r\nTrace ID: 6274265d-18cf-4f75-b5c4-3fa164bc1b00\r\nCorrelation ID: d27e928c-76b4-4ad0-a4cc-be07b3480d6a\r\nTimestamp: 2021-05-19 23:09:07Z","error_codes":[700016],"timestamp":"2021-05-19 23:09:07Z","trace_id":"6274265d-18cf-4f75-b5c4-3fa164bc1b00","correlation_id":"d27e928c-76b4-4ad0-a4cc-be07b3480d6a","error_uri":"https://login.microsoftonline.com/error?code=700016"}
Any help or guidance would be appreciated, maybe I'm missing something obvious ?...
Regards,
Olivier Beau

@Olivier-Beau, unfortunately the task does not support managed identity. Its is explicitly coded to only allow for SPN which is why you are seeing that error. I imagine this would be a fairly simple change to add support for this so would be a good candidate for contribution from the community.

Edited the title to frame this as an ask to support managed identity service connections. Added to prioritized as this will probably be helpful for many not wanting to manage spn creds.

Ok, so it is a new feature request.
I'll be happy to contribute by being part of the testing

@tplive no one is currently working on this that I know of. If you would like to start on this, please assign yourself to the issue and move to the "In Progress" state on the assigned project.

@charleszipp hmm, for some reason it seems I'm unable to assign myself on the issue...

I'm sorry to announce that we were unable to get the resources needed to work on this issue. Please release it for someone else to work on. Again, I'm sorry for holding up the issue, I was sure we would be able to get it done.

No problem @tplive! Thanks for letting me know.

Hi @jaredfholgate, is this now resolved with your PR?

Hi @jaredfholgate, is this now resolved with your PR?

Hi. Yes, I added support for managed identity too. Thanks

@Olivier-Beau , @tplive is this working for you now?

No further responses.  This issue should be resolved.


v2.8.1 missing top left 3 bar ☰ menu button

After upgraded to v2.8.1, I can no longer see the top left ☰ menu button.  Please advise.

This seems more like a possible bug. Would you be able to update the description with the following template?
Was the previous version before 2.8.0?
<!--------- For bugs and general issues --------->
**Setup**
- Rancher version:
- Rancher UI Extensions:
- Browser type & version:

**Describe the bug**
<!--A clear and concise description of what the bug is.-->

**To Reproduce**
<!--Steps to reproduce the behavior-->

**Result**

**Expected Result**
<!--A clear and concise description of what you expected to happen.-->

**Screenshots**
<!-- If applicable, add screenshots to help explain your problem and/or errors in the browser's dev console -->

**Additional context**
<!--Add any other context about the problem here. -->

I upgrade from 2.7.x to 2.8.0 then to 2.8.1.Please review my info below.  Thanks.PatrickOn Jan 29, 2024, at 12:43 AM, Richard Cox @.***> wrote:
This seems more like a possible bug. Would you be able to update the description with the following template?
Was the previous version before 2.8.0?

Setup

Rancher version:
v2.8.1- Rancher UI Extensions:
no extension- Browser type & version:
Chrome v121.0.6167.85Firefox v115.7.0esr
Describe the bug

Missing top left 3 bar menu icon/button.  Cannot click and access Rancher configuration page and other menu items.To Reproduce

Login and 3 bar never show up.Result
no 3 bar menu at top leftExpected Result

see 3 bar menuScreenshots

Additional context

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Could you see the menu when you upgraded to 2.8.0, and it's only after upgrading to 2.8.1 you don't see it?
Can you provide some screenshots, and are there any errors in the browser's console?
Do you have any downstream clusters?

No 3 bar menu when upgraded to v2.8.0 either.  So, upgraded to v2.8.1 and
hope to see it in that version.  The error in console is this:
client.js:793 TypeError: Cannot read properties of undefined (reading
'fontSize')
at a.globalBannerSettings (TopLevelMenu.vue:79:1)
at e.get (vue.runtime.esm.js:3446:33)
at e.evaluate (vue.runtime.esm.js:3547:27)
at a.globalBannerSettings (vue.runtime.esm.js:5537:25)
at a.sideMenuStyle (TopLevelMenu.vue:63:1)
at e.get (vue.runtime.esm.js:3446:33)
at e.evaluate (vue.runtime.esm.js:3547:27)
at a.sideMenuStyle (vue.runtime.esm.js:5537:25)
at a.Bt (TopLevelMenu.vue:1:344)
at Tn.e._render (vue.runtime.esm.js:2684:28)
Please see the screenshot in my last email.
Patrick
On Tue, Jan 30, 2024 at 1:06 AM Richard Cox @.***>
wrote:

Could you see the menu when you upgraded to 2.8.0, and it's only after
upgrading to 2.8.1 you don't see it?
Can you provide some screenshots, and are there any errors in the
browser's console?
Do you have any downstream clusters?
—
Reply to this email directly, view it on GitHub
https://github.com/rancher/dashboard/issues/10332#issuecomment-1916374352,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANNMHM6YHMAHC3FATNIUHO3YRCZYRAVCNFSM6AAAAABCK7LF66VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJWGM3TIMZVGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Does the image below show up?PatrickOn Jan 30, 2024, at 9:46 AM, Richard Cox @.***> wrote:
Unfortunately there's no image.
This looks like it might be the same as #10140 though.
You might be able to work around this by going to /v3/settings/ui-banners and Delete the value there. Going back to rancher should then work (after refreshing the page). This does mean though you would need to apply any custom banners again.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Happy to report that after resetting the top banner and reload the page, the top left 3 bar menu reappears.  You may close the issue.  Thank you.PatrickOn Jan 30, 2024, at 1:00 PM, Patrick Leung @.> wrote:Does the image below show up?<image001.jpg>PatrickOn Jan 30, 2024, at 9:46 AM, Richard Cox @.> wrote:
Unfortunately there's no image.
This looks like it might be the same as #10140 though.
You might be able to work around this by going to /v3/settings/ui-banners and Delete the value there. Going back to rancher should then work (after refreshing the page). This does mean though you would need to apply any custom banners again.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Closing as requested, covered by https://github.com/rancher/dashboard/issues/10140. The fix is targetted for 2.9.0 and under consideration for backporting to 2.8.x


Reduce repository number

This appears to be a repository
with 8 meta, config or meta files
for 1 actual source file.
Is this necessary?
The only repository using this source file
I found was orbit-db, so I would propose
putting the file into the repo and making
the publishing dance easier.

Search: https://github.com/search?q=org%3Aorbitdb+orbit-db-pubsub&type=code


Separate Group's subcommands and commands in help

Coming from the Argparse background, I would appreciate having a possibility to differentiate between additional sub-commands (sub-groups) and commands in help page to increase its clarity when comes to big CLI programs.
Example how I see the help page to look like:
Usage: toggl_runner.py [OPTIONS] COMMAND [ARGS]...

Options:
  -q, --quiet    don't print anything
  --help         Show this message and exit.

Sub-Commands:
  clients          clients management
  projects         projects management
  users            users management
  workspaces       workspaces management

Commands:
  add  adds finished time entry

Still interested in this? I've just added the possibility of organizing subcommands of a Group in multiple (titled) help sections in an external package called cloup. I originally wrote the package to support only "option groups".
I opened a PR on my own repository because I'd like to receive some feedback: https://github.com/janLuke/cloup/pull/1. Also, I invite click mainteiners to give a look and tell me if they would be interested in merging this feature into click.
In cloup you can manually organize your commands in arbitrary sections, but it's easy to write a reusable decorator that monkey-patch a cloup.Group to do exactly what you want automatically:
import click
import cloup

# Re-usable decorator
def group(*args, **kwargs):
    """ Returns a monkey-patched cloup.Group with two "default section":
    one for MultiCommand's and one for simple Command's. """
    SUB_COMMANDS_SECTION = cloup.GroupSection.sorted('Sub-commands')
    COMMANDS_SECTION = cloup.GroupSection.sorted('Commands')

    def get_section(cmd):
        if isinstance(cmd, click.MultiCommand):
            return SUB_COMMANDS_SECTION
        return COMMANDS_SECTION

    def decorator(f):
        # Pass "sections" to control the order of sections
        grp = cloup.group(*args, sections=[SUB_COMMANDS_SECTION, COMMANDS_SECTION], **kwargs)(f)

        def add_command(cmd, section=None):
            if section is None:
                section = get_section(cmd)
            cloup.Group.add_command(grp, cmd, section=section)

        grp.add_command = add_command
        return grp

    return decorator

""" ===========================================================
Use the decorator to create your root group, then use it as normal.
Below, subcommands are cloup.Command/Group but they don't need to be,
they can be normal click.Command/Group
"""

@group()
def main():
    pass

# Use main.add_command to add commands defined in this or other modules
def f(): return 0
main.add_command(cloup.group('clients', help='clients management')(f))
main.add_command(cloup.group('projects', help='projects management')(f))
main.add_command(cloup.command('add', help='add an entry')(f))

# You can also use @main.group() and @main.command() in this module
@main.command(help='remove an entry')
def rm():
    return 0

@main.group('users', help='users management')
def users():
    return 0

if __name__ == '__main__':
    main()

This will produce the following help:
Usage: scratch_5.py [OPTIONS] COMMAND [ARGS]...

Options:
  --help  Show this message and exit.

Sub-commands:
  clients   clients management
  projects  projects management
  users     users management

Commands:
  add       add an entry
  rm        remove an entry

Going to say use cloup for this.


Filter error on adding calibration input to robyn_inputs

Project Robyn
Describe issue
Error in filter():
i In argument: get(date_var) >= temp$liftStartDate.
Caused by error in temp$liftStartDate:
! $ operator is invalid for atomic vectors
Run rlang::last_trace() to see where the error occurred.
Receiving this error on the following code
calibration_input <- data. Frame(
channel = c("display_s","sp_s"),
liftStartDate = c("2022-04-01","2022-04-01"),
liftEndDate = c("2022-05-01","2022-05-01"),
liftAbs = c(74314,1234),
spend = c(50888,56478),
confidence = c(0.8,0.8),
metric = c("revenue","revenue"),
calibration_scope = c("immediate","immediate")
)
InputCollect <- robyn_inputs(InputCollect = InputCollect, calibration_input = calibration_input)
Environment & Robyn version

packageVersion("Robyn")
[1] ‘<IP_ADDRESS>00’
R.version$version.string
[1] "R version 4.1.3 (2022-03-10)"

Hi! I've noticed a couple of things here:

I guess it's a typo but data. Frame(... should be data.frame(...
I see in your sample data that the dependent variable es named "revenue" and you're setting "revenue_i" as metric. Is this intended? It should have popped out an explanatory error when checking the input.
I can't seem to replicate the error. Would you mind sharing the code importing this CSV and all the steps I can follow to replicate this issue please?

This is the reproducible example I tested with:
# Import the CSV data
df <- read.csv("~/Desktop/Sample_input.csv")
df$date_i <- as.Date(df$date_i, tryFormats = "%d-%m-%Y")
# Dummy hyperparameters for the sake of the checks
hyperparameters <- list(
  sp_s_alphas = c(0.5, 3),
  sp_s_gammas = c(0.3, 1),
  sp_s_thetas = c(0, 0.3),
  display_s_alphas = c(0.5, 3),
  display_s_gammas = c(0.3, 1),
  display_s_thetas = c(0.1, 0.4),
  train_size = c(0.5, 0.8)
)
# Create InputCollect with no calibration data
InputCollect <- robyn_inputs(
  df, dep_var = "revenue_i", dep_var_type = "revenue",
  paid_media_spends = c("display_s", "sp_s"),
  adstock = "geometric",
  hyperparameters = hyperparameters)
calibration_input <- data.frame(
  channel = c("display_s","sp_s"),
  liftStartDate = c("2022-04-01","2022-04-01"),
  liftEndDate = c("2022-05-01","2022-05-01"),
  liftAbs = c(74314,1234),
  spend = c(50888,56478),
  confidence = c(0.8,0.8),
  metric = c("revenue_i","revenue_i"),
  calibration_scope = c("immediate","immediate")
)
# Add calibration data to InputCollect
InputCollect <- robyn_inputs(
  InputCollect = InputCollect,
  calibration_input = calibration_input)

Warning messages:
1: In check_calibration(dt_input = InputCollect$dt_input, date_var = InputCollect$date_var,  :
  Your calibration's spend (50,888) for display_s between 2022-04-01 and 2022-05-01 does not match your dt_input spend (~508.9K). Please, check again your dates or split your media inputs into separate media channels.
2: In check_calibration(dt_input = InputCollect$dt_input, date_var = InputCollect$date_var,  :
  Your calibration's spend (56,478) for sp_s between 2022-04-01 and 2022-05-01 does not match your dt_input spend (~668.3K). Please, check again your dates or split your media inputs into separate media channels.


Rename invoke to invokeMethod for ActorProxy

We have recently renamed invokeActorMethod() to invoke in ActorProxy, after discussions within maintainers, we decided to rename it to invokeMethod() instead. Please, rename it.
Parent: dapr/dapr#2535
Related: #409

/cc @arghya88 We discussed the renaming again and we decided to keep invokeMethod() for service invocation but rename invoke() to invokeMethod() for ActorProxy to keep consistency in naming.

/assign


Fix gradient of U+1F643 Upside-Down Face

The upside-down face emoji (🙃) is a vertically flipped version of the slightly smiling face emoji (🙂).  The Noto version of this emoji is shaded on top as if the light source were also flipped.  This produces an inconsistent visual effect and looks like a careless error.
This PR fixes the gradient to match that of https://github.com/googlei18n/noto-emoji/blob/master/svg/emoji_u1f642.svg.

I signed it!

@dougfelt Any comment?

My apologies, I forgot to follow up on this.  Thanks for calling this problem to our attention.
Our policy for images is to make fixes internally and then generate .png and .svg when we update the image data.  We plan to fix this when we next refresh.  Unfortunately this means a fix won't be available from us until we do so, and I can't say when that will be.

Heads up that this is fixed - thanks for surfacing this @ngaya-ll !
https://theoutline.com/post/4832/a-lot-goes-into-your-emoji-update-google-android


Update OpenJ9 JDK17 exclude list

The following PRs are applied to the OpenJ9 JDK17 exclude list:

#2188
#2192
~#2193~
#2215

Signed-off-by: Babneet Singh<EMAIL_ADDRESS>

fyi @pshipton @smlambert @llxia

fyi @pshipton @smlambert @llxia

Trying a single platform to ensure no unexpected failure are introduced. Note there are some expected failures (which should be resolved shortly when the associated PRs under review are merged).
https://ci.eclipse.org/openj9/view/Test/job/Grinder/1483/

The following are failing with these changes, which is undesirable. @babsingh am I testing the wrong type of build? I used the last jdk acceptance build which doesn't have OpenJDK MethodHandles enabled.
java/foreign/TestAdaptVarHandles.java.TestAdaptVarHandles
java/foreign/TestAddressHandle.java.TestAddressHandle
java/foreign/TestArrays.java.TestArrays
java/foreign/TestLayouts.java.TestLayouts
java/foreign/TestMemoryAccess.java.TestMemoryAccess
java/foreign/TestMemoryAccessStatics.java.TestMemoryAccessStatics
java/foreign/TestMemoryAlignment.java.TestMemoryAlignment
java/foreign/TestMemoryCopy.java.TestMemoryCopy
java/foreign/TestMemoryHandleAsUnsigned.java.TestMemoryHandleAsUnsigned
java/foreign/TestMismatch.java.TestMismatch
java/foreign/TestRebase.java.TestRebase
java/foreign/TestSlices.java.TestSlices
java/foreign/TestTypeAccess.java.TestTypeAccess
java/foreign/TestVarHandleCombinators.java.TestVarHandleCombinators
java/util/stream/test/org/openjdk/tests/java/util/stream/SpliteratorTest.java.SpliteratorTest

The above failures are seen because https://github.com/ibmruntimes/openj9-openjdk-jdk/pull/248 is unmerged. For JDK17, OJDK MHs should be enabled by default. So, https://github.com/ibmruntimes/openj9-openjdk-jdk/pull/248 does not need to be merged at this point. To address the above failures, I have reverted the changes related to #2193.

lgtm


Add conda-forge package?

Hello, I just wanted to ask if you'd be interested in a conda-forge package of MNE-HFO? I could build one.
Asking because I'm right now looking at packages that would be nice to include in the MNE installers.
Will the latest release work with MNE-Python 0.24 and 1.0?

Hmm idk if this package is ready yet. Is Conda forge easy to add? If so sure. If not, might not be worth the effort yet.


Stabilize 'attr_literals' feature.

RFC Issue: https://github.com/rust-lang/rfcs/pull/1559
Tracking Issue: https://github.com/rust-lang/rust/issues/34981
Depends on https://github.com/rust-lang/rust/pull/53044.

This has been stabilized in Rust. \o/
💟 Thanks for updating the reference alongside writing the RFC.


Fixed tuple empty issue

Coverage decreased (-0.03%) to 86.425% when pulling 605afde643c9fc09d0268f7c16d4121bac88bdaf on encoresky:master into b2d92feeeb0de93f8aebed83750dd5a30fd02d0e on softvar:master.

Thanks @vishai12345 ,
This is fixed in the latest version 2.0.0


brew install hipchat-cli

Would be nice for OS X.

+1

+1

This should be closed, it's way over 1 year old

We don't have plans to support brew install for this cli.


Add note for using PlantUML with MarkBind and Jekyll

Fixes #88
Added a note (panel) with a link to MarkBind's user guide on using PlantUML with MarkBind.

A good, simple addition to the guide that allows easy reference for readers, especially for MarkBind. Maybe we could leave it as a third sub-point rather than a dropdown for ease of access instead? Or we could rename the title to "How to use PlantUML with Jekyll and MarkBind"? Readers might miss the dropdown if they don't read it carefully.

Thanks for this PR @mfjkri
I translated this to a smaller change in https://github.com/se-edu/guides/commit/dc5c4977f5604af82aad577f853c43344bde0fa4


[Index] Write empty record files

When indexing an empty Swift file, we get a unit file that doesn’t have any record dependency declared on the source file. Because of this, we always assume that the empty file has an out of date index store entry and re-index it on project open. Write empty record files to fix this.
rdar://128711594

@swift-ci Please smoke test

@swift-ci Please test

@swift-ci Please smoke test


Zoom further in than level 19

I'd like users to be able to zoom further even if the tiles blur as they have to place markers to a high degree of accuracy.


OnSelectTemplate does not fire on iOS

I have a the following xaml:
`<local:StyledCarouselView
x:Class="MyPackSmall.DeliverCarouselView"
...
AnimateTransition="false"
IsSwipingEnabled="true"
ItemsSource="{Binding Items}">
<local:StyledCarouselView.ItemTemplate>
	<local:DeliverViewTemplateSelector/>
</local:StyledCarouselView.ItemTemplate>

</local:StyledCarouselView>`
OnSelectTemplate fires on Android but not on iOS. Hence no views are shown on iOS. Any idea as to why?
I'm using version 5.0.2, upgrading from 4.4.6 to see if that was the problem.
Forms: 2.5.0.

Never mind.


added windows support

fixes:  #6

Summary by CodeRabbit

New Features

Enhanced development environment with expanded permissions for system-level and network access.
Improved server startup logic with OS-specific feedback and conditional tunnel management.

Bug Fixes

Ensured proper server lifecycle management and execution flow.

@ekinertac
Thank you for your contribution!
looks like windows does not use wrangler anymore?
If so, please add instruction message for windows user!
Cheers

@ekinertac
Hi! Please check my comments!

Close this because I added a flag to disable cloudflared.
Thanks for your work @ekinertac


[PrintPreview]Resizing figure does not work properly

See https://github.com/vasole/pymca/issues/383
The first resizing operation (dragging to bottom-right corner of a figure) works fine, then the results seem to become random.
This can be replicated using examples/printPreview.py

It worked in PyQt4. The problem seems specific to Qt5.


Updated Version No Longer Compatible with DarkerVision Module

Prior to updating today, Dancing Lights played nicely with DarkerVision, allowing even flickering dim lights within a character's dim vision radius to appear as bright light.  After the update, even when using a standard light with its Dancing Lights check box unticked, DarkerVision no longer works if the Dancing Lights module is enabled.
Without updated Dancing Lights module enabled:

With updated Dancing Lights module enabled:

Is there a way I can roll back my update locally and install version 1.51 again?  I have a game in two days for which I'd love to have both fire flicker and more proper darkvision enabled.

Hi Dumbhuman,
In the latest DL version, we both override the same method, and the way I've done it clobbers his module a bit.
I've added support for the module in 2.0.4


Malformed SQL output

there appears to be an AS missing in the FROM statement in the query being generated by join monster 2.0.15:
SELECT
  "article"."id" AS "id",
  "article"."permalink" AS "permalink",
  "article"."page_title" AS "title",
  "article"."published_at" AS "published",
  "article"."updated_at" AS "updated",
  "article"."section" AS "section",
  "article"."category" AS "category"
FROM posts "article"
WHERE "article"."permalink" = "my-article-url"

which in turn is throwing the following error:
{
  "errors": [
    {
      "message": "SELECT\n  \"article\".\"id\" AS \"id\",\n  \"article\".\"permalink\" AS \"permalink\",\n  \"article\".\"page_title\" AS \"title\",\n  \"article\".\"published_at\" AS \"published\",\n  \"article\".\"updated_at\" AS \"updated\",\n  \"article\".\"section\" AS \"section\",\n  \"article\".\"category\" AS \"category\"\nFROM posts \"article\"\nWHERE \"article\".\"permalink\" = \"nylon-guys-lust-for-life-june-july-2017\" - You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '.\"id\" AS \"id\",\n  \"article\".\"permalink\" AS \"permalink\",\n  \"article\".\"page_title\" ' at line 1",
      "locations": [
        {
          "line": 2,
          "column": 3
        }
      ],
      "path": [
        "article"
      ]
    }
  ],
  "data": {
    "article": null
  }
}

my object:
module.exports = new GraphQLObjectType({
  name: 'Article',
  sqlTable: 'posts',
  uniqueKey: 'id',
  fields: () => ({
    permalink: {
      type: GraphQLString,
      sqlColumn: 'permalink'
    },
    title: {
      type: GraphQLString,
      sqlColumn: 'page_title'
    },
    published: {
      type: GraphQLString,
      sqlColumn: 'published_at'
    },
    updated: {
      type: GraphQLString,
      sqlColumn: 'updated_at'
    },
    section: {
      type: GraphQLString,
      sqlColumn: 'section'
    },
    category: {
      type: GraphQLString,
      sqlColumn: 'category'
    },
    [...]
})

actually never mind that's not where it's breaking, this is where:
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '."id" AS "id",
  "article"."permalink" AS "permalink",
  "article"."page_title" ' at line 2

i've tried both the mysql and mysql2 node adapters.

i just added this and now it's fine
sql = sql.replace(/\"/g, '')


Can I change default AVAudioSessionCategoryOptions?

I want to set mixWithOthers of AVAudioSessionCategoryOptions, but sdk set without options.
If I set options by myself, SDK override this.

@yukitoto At present, our existing API does not support allow configuring audio session category options explicitly. Internally we are discussing providing better audio APIs. However, In the meantime, you can work around the problem and set AVAudioSessionCategoryOptions  by using our CallKit APIs. Our AVPlayerExample how to configure audio session using Video SDK's CallKit APIs.
I have created a branch for you on AVPlayerExample 2.0.0-preview which demonstrates how to set AVAudioSessionCategoryOptions to mixWithOthers while using the Video SDK.
See this commit - https://github.com/ptankTwilio/video-quickstart-objc/commit/5cfbef977f0a46f69250f81901e3a0c69583bf8b
Let me know if you have any questions.

@yukitoto hopefully you were able to resolve this issue with the TVIDefaultAudioDevice APIs. I'm going to close this issue, but please re-open if you need more help with this.


admonition icon flip

What do you think @chrisjsewell? BTW - if you think there are better icon glyphs to use for the other admonitions, let me know and we can customize those a bit more so that everything isn't a !

Codecov Report

Merging #98 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #98   +/-   ##
=======================================
  Coverage   91.28%   91.28%
=======================================
  Files           2        2
  Lines         241      241
=======================================
  Hits          220      220
  Misses         21       21

Flag
Coverage Δ

#pytests
91.28% <ø> (ø)

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 85fc278...38e0c54. Read the comment docs.

Yeh cool, these are the icon semantics I came up with so far:
.title-icon-information .admonition-title:before {
    content: "\f05a" /* info-circle */
}
.title-icon-tip .admonition-title:before {
    content: "\f0eb" /* lightbulb */
}
.title-icon-important .admonition-title:before {
    content: "\f06a" /* exclamation-circle */
}
.title-icon-warning .admonition-title:before {
    content: "\f071" /* exclamation-triangle */
}
.title-icon-troubleshoot .admonition-title:before {
    content: "\f0ad"  /* wrench */
}
.title-icon-read-more .admonition-title:before {
    content: "\f08e"  /* external-link */
}

So maybe:

note: info-circle
tip: lightbulb
hint: lightbulb
seealso: external-link (or eye)
attention: exclamation-circle
caution: exclamation-circle
danger: exclamation-triangle
error: exclamation-triangle (window-close seems to be the closest to a cross)

nice - I added another icon, though realized that unfortunately we can't use the full vocabulary of icons, because the pydata sphinx theme doesn't distinguish between some of the directives when it translates to bootstrap classes, so I think this may also need an upstream there as well

I opened up https://github.com/pandas-dev/pydata-sphinx-theme/issues/181 to track that


All images missing

Hi! I have no idea if the following is related with #21 and / or #22, since my Nextcloud resides on https://mydomain.tld/nc/. Which, from what I know, is perfectly valid.
While the web app works as expected, I can't see any images on Android. There's a short animation (small circles, bigger circles), and after that only empty space where the pictures should be. The category names are missing as well, but they might just be invisible in white on white. All other text is there, so by guessing or choosing the first entry (must be "all"), I can find every recipe. The recipes are missing their title picture as well.
I'm using version 0.3.5.

I can confirm the same problem on my S10 using the same URL style: https://example.com/nc

Hmm this is weird.
Could one of you create a test account for me so i can check what exactly goes wrong?
You could send me the credentials over the matrix chat of nextcloud-cookbook or by mail to<EMAIL_ADDRESS>This would be the easiest way for me to debug this issue!

I created you an account. I joined the matrix chat as "Tom", just text me if any help is required.

Figured it out. Seems to be a problem with the url path /nc
I fixed it now and am currently building the release version that should fix this problem!

Figured it out. Seems to be a problem with the url path /nc
I fixed it now and am currently building the release version that should fix this problem!

I just updated to 0.36 - still no images, url is .../foldername/ and i use LineageOS 17 + afwall

I just updated to 0.36 - still no images, url is .../foldername/ and i use LineageOS 17 + afwall

@renarena Hmm this is not what i had hoped ;(
Could you maybe also provide me with a test account to check what is going wrong?

@renarena Hmm this is not what i had hoped ;(
Could you maybe also provide me with a test account to check what is going wrong?

Thank you! With 0.3.6, loading images ~works~ worked until I updated the Nextcloud app (→ #36). I can remember seeing all images, although some (or all? – no way to test it anymore) in low quality.

Thank you! With 0.3.6, loading images ~works~ worked until I updated the Nextcloud app (→ #36). I can remember seeing all images, although some (or all? – no way to test it anymore) in low quality.

Hi Teifun2, and thanks a lot for your work on this project. I didn't want to bog down the reviews on Google Play, so I searched around trying to find you. On my Android phone (Mi A3) the issue persists, but since the last update I also see this as the main screen when I open the app:

I can search for individual recipes, despite of the error, but they still don't show any images:

Information that may also be helpful:

Some of my recipes aren't in English but in Greek.
Could this mean that the app maybe having issues with folder/file naming?
I'm running Nextcloud 20.0.5
Cookbook version is 0.7.9

Let me know if there's anything I can do to help. Thanks again!

Hi Teifun2, and thanks a lot for your work on this project. I didn't want to bog down the reviews on Google Play, so I searched around trying to find you. On my Android phone (Mi A3) the issue persists, but since the last update I also see this as the main screen when I open the app:

I can search for individual recipes, despite of the error, but they still don't show any images:

Information that may also be helpful:

Some of my recipes aren't in English but in Greek.
Could this mean that the app maybe having issues with folder/file naming?
I'm running Nextcloud 20.0.5
Cookbook version is 0.7.9

Let me know if there's anything I can do to help. Thanks again!

I created a new version which should fix this issue. It will take some hours before it will appear in the play store, alternatively you can directly install the apk which is attached on github under releases.
Can you please check and confirm if this solved your issues? @hoyalu @katseassavvas @renarena

I created a new version which should fix this issue. It will take some hours before it will appear in the play store, alternatively you can directly install the apk which is attached on github under releases.
Can you please check and confirm if this solved your issues? @hoyalu @katseassavvas @renarena

Unfortunately, this is not the solution for my case. After starting the app, I get
(de) "Fehler beim laden (sic!) der Kategorien: Bad state: No element"
≈ (en) "Error while / with loading categories: Bad state: No element"
Perhaps the app can't deal with my not providing an image for every category / recipe?

Unfortunately, this is not the solution for my case. After starting the app, I get
(de) "Fehler beim laden (sic!) der Kategorien: Bad state: No element"
≈ (en) "Error while / with loading categories: Bad state: No element"
Perhaps the app can't deal with my not providing an image for every category / recipe?

I see the same "No element" behavior @hoyalu is describing, but now the app loads the image for a recipe if I search it with a recipe name.

I see the same "No element" behavior @hoyalu is describing, but now the app loads the image for a recipe if I search it with a recipe name.

with nextcloud 19.01 it works now for me

with nextcloud 19.01 it works now for me

it still works after updating to V20.5 after clearing cache

it still works after updating to V20.5 after clearing cache

Can somone provide me a test account that shows this behaviour<EMAIL_ADDRESS>

Can somone provide me a test account that shows this behaviour<EMAIL_ADDRESS>

Yes, I'm willing. How should I go about doing that?

Yes, I'm willing. How should I go about doing that?

Just create a new account on your cloud with some. Test recipes that show the same behaviour with the app. Then send me login data to my email address. I will check the problem and you can delete the account after my analysis!

Just create a new account on your cloud with some. Test recipes that show the same behaviour with the app. Then send me login data to my email address. I will check the problem and you can delete the account after my analysis!

Just sent the info. Thanks!

Just sent the info. Thanks!

Following...  Same issue with mobile app version 0.3.7, NC version 20.0.5, NC app version 0.7.10.

Following...  Same issue with mobile app version 0.3.7, NC version 20.0.5, NC app version 0.7.10.

@katseassavvas With your account i can now see what the problem is! In earlier versions of the cookbook plugin i was able to get all un-categorized recipes with  index.php/apps/cookbook/category/*  In the newer version i need to get index.php/apps/cookbook/category/_ . I have to find out when this change happened to make sure it works also with the old version!
But its my bad that there is no "graceful" degradation!

@katseassavvas With your account i can now see what the problem is! In earlier versions of the cookbook plugin i was able to get all un-categorized recipes with  index.php/apps/cookbook/category/*  In the newer version i need to get index.php/apps/cookbook/category/_ . I have to find out when this change happened to make sure it works also with the old version!
But its my bad that there is no "graceful" degradation!

I'm glad you pinpointed the problem. Should I keep the account?

I'm glad you pinpointed the problem. Should I keep the account?

@katseassavvas it would be good if you could leave it until I have a working fix (so I can properly test it!) should be ready in 1-2 days I hope!

@katseassavvas it would be good if you could leave it until I have a working fix (so I can properly test it!) should be ready in 1-2 days I hope!

@Teifun2 no pressure, I'm happy to help.

@Teifun2 no pressure, I'm happy to help.

@Teifun2 it's working as intended, and it will be of great use to me. Thank you very much for the work you're doing! I'm deleting the account.

@Teifun2 it's working as intended, and it will be of great use to me. Thank you very much for the work you're doing! I'm deleting the account.

Version 0.3.8 + Nextcloud 20.0.5 + NC App 0.7.10 work like a charm for me. Thanks a lot! @Teifun2
I hope you all agree with my closing this issue as solved. @ttcc-gh ?

Version 0.3.8 + Nextcloud 20.0.5 + NC App 0.7.10 work like a charm for me. Thanks a lot! @Teifun2
I hope you all agree with my closing this issue as solved. @ttcc-gh ?