Cannot change data dir location from ~/.fn to somewhere outside of my home folder

Hi everyone,
I'm running into an issue when trying to start the fn server. I have manually installed fn by downloading the binary from the GitHub release page and placing it into a local folder that is on my PATH. This works fine:
❯ fn --version
fn version 0.6.26

However, I cannot start the fn server:
❯ fn start --log-level DEBUG
2023/08/07 08:35:20 ¡¡¡ 'fn start' should NOT be used for PRODUCTION !!! see https://github.com/fnproject/fn-helm/
docker: Error response from daemon: error while creating mount source path '/<redacted>/home/pfaion/.fn/iofs': mkdir /<redacted>/home/pfaion/.fn: permission denied.
2023/08/07 08:35:20 Error: processed finished with error exit status 126

I am working on a machine where my home directory is mounted via AFS with Kerberos authentication. I'm not entirely sure, but it might be that whatever happens when I use fn start doesn't have access to my home folder, since it doesn't use Kerberos. I would be fine using a different directory for the fn configuration, outside of my home folder. But I couldn't find an option to configure this.
Is this a known issue and/or do you have any suggestions for how to work around it?

There seem to be traces for such an option in the code, but I'm not sure if this is usable?
https://github.com/fnproject/cli/blob/06e5e0de619012b06c2dbdb86156de4386d43f74/commands/start.go#L72-L76
I don't know go and I couldn't figure out how to use something like a --data-dir argument. It would always give me a usage error.

Looks like the cli argument was dropped in a re-organization commit, I assume by mistake?
https://github.com/fnproject/cli/commit/f83f29239a87dd30de7cb00bf75eb3c5ba0efa9a#diff-63a7ad8e7bd042aa25c04ff627b2d29e0a8a949bb563e9467ae69e1b22d150b9L39-L42

I'd create a PR myself, but even the contribution "workflow" is broken. The docs mention that I should sign a CLA, but the link is broken. See https://github.com/fnproject/cli/issues/672

Hi we're no longer involved with the project, sorry!


A question about Dorefa-net

Hi,
I'm a little confused about equation 10 in the paper，"DoReFa-Net: Training Low Bitwidth Convolutional Neural Networks with Low Bitwidth Gradients". According to the formula, r0's derivative on r1 is 1-tanh2(r0), but in other open source such as BMXNet, where the derivative is 1, which one is correct?
Look forward to your reply and thanks for this wonderful toolbox!

I don't understand your question. Could you explain "According to the formula, r0's derivative on r1 is 1-tanh2(r0)"

Sorry, I didn't explain it clearly.
My question is how to calculate the derivatives (∂ro/∂ri) in equation 10. In other words, how to calculate the derivatives (∂Wb/∂W), Wb is the weight after quantization and W is the origin float-point weight.
My result is tanh(W)'==(1-tanh(W) * tanh(W)), but i see the other result in other realization.

It can be computed by standard chain rule.


Added functionality to select a chat item in the conversationsAdapter…

In AtlasConversationAdapter added code that allows sets the activated to true when a user clicks on a recycler view item the adapter tracks the selected position so that the recycler view gets deactivated when it is used for a new item, and activated when the selected item gets put back in to a recycler view.
Added a default selection method in AtlasConversationsRecyclerView and the AtlasConversationAdapter so that we can programatically select the first item (index 0)in the adapter  if needed.

@Dan2anderson as far as I can tell, this code marks an item as "selected" upon clicking, which opens the messages view, and then when the user comes back to the conversations list, the item continues to be in the "selected" state.
I'm not entirely sure what the use case is, esp when you consider that this change would affect all applications that use Atlas. I don't see a compelling case to accept the PR yet.
I'm going to close this PR, but in case you feel this needs to be in Atlas, please do the following:

Update your branch by rebasing it off develop to ensure there are no conflicts.
Make the PR against develop, since that allows us to fit it into our release workflow
Add an explanation of the use case you envision, perhaps with some screenshots?

Thanks,
Archit


ImgBot savings

@edent As requested in https://github.com/edent/SuperTinyIcons/issues/478#issuecomment-723659800, this is what ImgBot has distilled. Each of the included files is just a few bytes smaller (mostly whitespace), so I don't know how welcome that is.  Also, I've included a custom .imgbotconfig which specifically excludes the reference folder.  Let me know what you think! 🙇🏾♂

I'll go through when I can & check for what savings other than whitespace, but why insist on such?  It's much easier to check before/after if both are with/without such, &, @ these sizes, removing whitespace tends to be significant.

I don't think I was clear enough. A \n takes up the same number of bytes as a space. So we use newlines for readability. That also makes it easier to see what a diff has done.
I've just pushed some changes which removed Windows style newlines from some files, and trailing newlines from the end of some files.
I'd be interested to see whether imgbot makes any file savings against these newer, smaller files.
Thanks for your work on this, it is appreciated.

Eyeballing it, it looks like it got some missed leading 0.→. & similar.  I'll have ImgBot regenerate against the repo's current iteration asap.

Replaced by #485


[Mobile] : Allow mention

The use case is that I want to mention a user using @ keyword in quill editor as we do with other social media apps and show up the overlay for possible users, similar to tagging.
The first challenge is to trigger an event when a keyword is hit. I found that this can be done by listening to the quillController.changes stream. Not sure if there is any better way of doing it.
The second challenge is to show an overlay just below the cursor where the keyword was detected. I used LayerLink over QuillEditor but it does not seem to be working.
The third challenge is to get the text after the keyword. For example, If I type @burhanrashid52 I need a way to get text burhanrashid52.
If this is something not possible in the current lib then let me know how can I achieve it and if this is something the library is looking for then maybe I can contribute back with a PR.

Something similar to https://github.com/quill-mention/quill-mention


MM-20787 Remove ref from ConfirmModal

The previous version of this uses the ref to get the state of the checkbox instead of just using a controlled component. This changes that and adds some tests.
Ticket Link
https://mattermost.atlassian.net/browse/MM-20787

/update-branch

/update-branch

Creating a new SpinWick test server using Mattermost Cloud.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-4380.test.mattermost.cloud

Account Type
Username
Password

Admin
sysadmin
Sys@dmin123

User
user-1
User-1@123

@hmhealey
I'm not sure I understand the last qa step listed in the jira issue
You're in Out of Office mode and you attempt to send a message
I placed one test user in OOO status (sysadmin), but not seeing this warning when sending a message.
Can you add a step or clarify how to trigger this warning/confirmation? Thanks

Test server destroyed


[api] first draft version api blueprint document for implemented API

File blueprint.apib documents implemented API in API Blueprint format.
We can test API doc by dredd, will need more time to investigate how to setup CI build properly, this PR only documents how to run dredd on local machine.
We will continue to improve the document details, adding descriptions to all data structures or fields later when we finalize everything; at current stage, this PR only setup API specs and data structures that match server response.

/land


⚠ Bofrost.fr has degraded performance

In 9a32e0f, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 2417 ms

Resolved: Bofrost.fr performance has improved in e50477d.


⚠ MorethanTV has degraded performance

In af6ade9, MorethanTV ($MTV) experienced degraded performance:

HTTP code: 500
Response time: 1216 ms

Resolved: MorethanTV performance has improved in 70c6685 after 7 minutes.


Snapshot testing

This might be a tall order. I am wondering how hard it would be to extend golden-tests so that it can automatically update the expected-output when needed. Currently I need to manually update the expected output whenever it changes.

Usually the purpose of golden tests are to assert the output of something doesn't change, and to require human intervention when it does. E.g. I use it for (among other things) asserting that given an example program that the correct types are always inferred. If I ever change my type inference in a way that those expected types change I'd expect an error, then I'd verify whether the new output is expected or not, and if so copy-paste the new output from the terminal, or otherwise fix the type inference until it gave me the old results again.
Perhaps I'm missing something or not understanding though. Can you elaborate more on your use case?

Hi, sorry for the late response. I use goldentests for the same thing. The main reason I asked is because of laziness when it comes to manual copy-pasting 😅 . It's not close to being crucial, so if no one else experiences the same problem, I don't think it's worth implementing.

Hi, sorry for the late response. I use goldentests for the same thing. The main reason I asked is because of laziness when it comes to manual copy-pasting 😅 . It's not close to being crucial, so if no one else experiences the same problem, I don't think it's worth implementing.

I think to solve the "only automatically update the golden-tests sometimes" problem would require some CLI (y/n) confirmation interaction from the user so they can specify whether they want it updated or not. I'm not sure is possible while running cargo test, so for now I'll leave out this feature unless more people want it and I can hopefully come up with a better design.

I think to solve the "only automatically update the golden-tests sometimes" problem would require some CLI (y/n) confirmation interaction from the user so they can specify whether they want it updated or not. I'm not sure is possible while running cargo test, so for now I'll leave out this feature unless more people want it and I can hopefully come up with a better design.


Fix accessibility issues in Home Page

This ticket is a part of #150 EPIC.
The fixes that should be done under the banner of this ticket are:

[x] Fix duplicate H1 tags on home page (there can be only one H1 per page)
[x] Use empty ALT tags for course cards on home page by setting alt="". The thumbnails are purely decorative and no alt tag would be preferable to the repetitive ones we have now.

Closed via #160 and #168 (Partially)


py3 Library Errors on 20.5

The ‘json’ library is not available in python2 on 20.5.x vManages and the tool recommends to try python3 instead. If you do this, the ‘Queue’ import fails. This can be fixed by replacing ‘import Queue’ with ‘import queue as Queue’.

JIRA TIcket:https://jira-eng-sjc4.cisco.com/jira/browse/SDWANAURA-74


react-dom Over preact-render-to-string

What does this change?
This switches from using preact-render-to-string to react-dom/server for the renderToString function.
We use a mix of React and Preact across the department, and common practice in Preact codebases is to treat React as the default and alias to preact-compat. Therefore I think it's best to also treat React as the default in our libraries, and let the consumers decide whether they want to alias that to Preact.
This will also mean that we can remove an awkward workaround in Apps-Rendering, where we've tried to alias in the other direction in order to accommodate atoms-rendering. Unfortunately this workaround has started to cause us problems and we think that updating atoms-rendering to conform to the React API may solve this.
How to test
Checks on this PR pass and atoms continue to work in DCR and AR.
How can we measure success?
The webpack upgrade on AR becomes unblocked.
Have we considered potential risks?
Possibility of changes being required in DCR, although given that DCR already aliases react-dom to Preact it should work fine.

A quick note on getting this to work with DCR:
With build configurations unchanged, you'll get an error:
Module not found: Error: Package path ./compat/server is not exported from package /dotcom-rendering/node_modules/@guardian/atoms-rendering/node_modules/preact

I believe this is because of how the alias is set-up in DCR's webpack config (webpack/frontend.js):
'react-dom': 'preact/compat'
Where react-dom/server is replaced with preact/compat/server which does not resolve.
The solution I found to this issue is to add a line before the existing react-dom alias:
'react-dom/server': 'preact/compat'

There may be another format for updating this which doesn't require adding an explicit alias; however this also ensures it doesn't cause issues anywhere else in the build.

Good point Olly. We'd best release this as a major version.


angular meteor 1.0.0-rc4 conflicts with meteor-job-collection somehow

Hi,
I just tested the new angular package and now the meteor job collection throws an error
'The global definition of Mongo.Collection has changed since the job-collection package was loaded. Please ensure that any packages that redefine Mongo.Collection are loaded before job-collection.'

I guess this is a related issue
https://github.com/vsivsi/meteor-job-collection/issues/58
I cannot really see where some Mongo.Collection shadowing is happening, maybe in one of the dependencies does it

A very simple workaround for now is to make sure that the angular packages is listed before the job collection in the packages file
angular
...
vsivsi:job-collection

@mxab can you check if that happens by one of the package dependencies?

I think I had the same thing with his files package..
I'm closing unless someone would like to investigate this package

Hi I have the same issue, It seems that angular package use dburles:mongo-collection-instances as depency
<EMAIL_ADDRESS>'client'); // For getCollectionByName
PackageFile
the workaround to list angular before the job collection in the packages file and to add 'Mongo.Collection.prototype.constructor = Mongo.Collection;' before the new new JobCollection( ... ) doesn't fix the issue

@Urigo The new API doesn't use this package. It's just for legacy code. Should we do something?

FYI, this also causes issues with vsivsi:file-collection which is a very nice package for people wanting to use gridFS and not take on CollectionFS. It is also a very well maintained package compared with CollectionFS.


Backend Environments tab not appearing

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

App Id
arn:aws:amplify:us-west-1:009611472941:apps/d94a0xc1cxak4
Region
us-west-1
Amplify Console feature
Backend builds
Describe the bug
The tab called "Backend environments" on the main page of the Amplify console page is missing.

Expected behavior
The "Backend environments" tab should be present.
Reproduction steps

Launch "Front-End Developer" course from the "Getting Started with AWS" page.
Follow course until Module 2 "Initialize a Local Amplify App".
Third step "Initialize the Amplify app" instructs to navigate to "Backend environments" tab in Amplify Console. But it's not there.

Build Settings
version: 1
frontend:
  phases:
    preBuild:
      commands:
        - npm install
    build:
      commands:
        - npm run build
  artifacts:
    baseDirectory: build
    files:
      - '**/*'
  cache:
    paths:
      - node_modules/**/*

Additional information
No response

Having this issue as of 8/7/2021. Can't believe there's no fix for this yet!

Also having this issue

Also having this issue >:(

Hi, I have a solution for this:
In the root directory of your project run:
amplify init
And follow the prompts.
That tutorial needs updating for a lot of things. I've reached out to 3 people so far who apparently work on Amplify offering to change their docs for them (for free, mind you) just so that devs don't get hung up on these issues and they have yet to give me the time of day. 😅
You'll most likely at some point also run into the issue that the backend build container that AWS provides by default is restricted to Node 12 as the amplify CLI automatically switches to it no matter what as one of the integrated build steps when using the default build container.
Incredibly frustrating, and for that you'll have to use a custom docker image for your builds and point to it in your build settings.

Hi Guy,
After running amplify init, should I run amplify add api and amplify push to get the backend environments tab?
For me after running amplify initand the push the code to my connected branch, the backend environment tab is still not visible.

I've done absolutely nothing in the past few months and today when I visit AWS Amplify, I see the "Backend environments" tab is now present.
¯\(ツ)/¯


Add alternate constructor to Polygon and PolygonCoordinates

Adds alternate constructors to Polygon and PolygonCoordinates that make it possible to construct a Mongo Polygon given a Polygon from some other library with a similar API.
Posting this PR to demonstrate what it would be nice to have, but this was a breaking change if consuming the driver using Groovy as can be seen in a few tests I changed. I would assume this exact solution won't work, so open to any ideas.
Basically just want to be able to convert a Polygon with this type of API to the Java Driver's Polygon:
class Coordinate {
    double[] getCoords();
}

class Polygon {
    List<Coordinate> getExterior();
    List<List<Coordinate>> getHoles();
}

which currently is not possible as converting the above Polygon holes to the Java Driver's would require instantiating and populating a List<Position>[], which using conventional Java isn't allowed because of the compile error: "Cannot create a generic array of List". And I know using newInstance() I can make an array that will work, but just looking for some way to avoid using reflection if possible.

JAVA-4098

Added a small commit to address the compiler warning, as well as @since Javadoc annotations for the new constructors.
Despite the Groovy breakage, I think this is ok.  It preserves binary compatibility, even if it does not source compatibility.


Issue in npm install

Hi I was trying use this in my project.
I installed it by npm install.
Installation went fine. On npm run I am getting this  error
ERROR in ./node_modules/vue-final-modal/lib/VueFinalModal.vue 1:0
Module parse failed: Unexpected token (1:0)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders

|   <div
|     v-if="ssr || visible"
@ ./node_modules/vue-final-modal/lib/index.js 1:0-39 3:29-34
@ ./index.js
@ multi ./index.js
I just added
import { VueFinalModal } from 'vue-final-modal/lib' in my code and start getting this error.
Is there any prerequisite  for this..?

Hi @shikharani16

ERROR in ./node_modules/vue-final-modal/lib/VueFinalModal.vue 1:0
Module parse failed: Unexpected token (1:0)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders

I guess this means thath Webpack don't know what to do with .vue file type.

Try add vue-loader to your webpack config

This might be the same issue https://forum.vuejs.org/t/app-vue-module-parse-failed-unexpected-token-1-0/54125

Try use umd build for you project:

import { VueFinalModal } from 'vue-final-modal'

Vue.component('VueFinalModal', VueFinalModal)

HI ,
This error is gone . I did Step 2.
import { VueFinalModal } from 'vue-final-modal'
Vue.component('VueFinalModal', VueFinalModal)
Still It doesn't seems to be working.. It comes all over places. I think somehow  It is not able to Understand Css. I was following this https://codepen.io/hunterliu1003/pen/RwaQmo. We need to define classes for the size..? or w ecan mention height/ width somewhere.

Hi,

Still It doesn't seems to be working.. It comes all over places. I think somehow It is not able to Understand Css. I was following this https://codepen.io/hunterliu1003/pen/RwaQmo. We need to define classes for the size..? or w ecan mention height/ width somewhere.

The codepen link above was broken.
Vue Final Modal was a renderless component, so you have to define all style by yourself.
I wrote an plain css example without using tailwindCSS
https://codepen.io/hunterliu1003/pen/ZEWoYeE

Hi,
I followed your example and It is working almost fine. I see a Overlap from last page on Modal Dialog.
So this needs to be handled by classes what we are providing..? For example  ( modal,modal-content ) in https://codepen.io/hunterliu1003/pen/ZEWoYeE .
I tried using z-index property of vue-final-modal but that doesn't seems to be working.
I am attaching the screenshot of my problem. The tab coming over Modal or of DOJO framework.
As we are partially migrating application to Vue.js

Hi @shikharani16,

I followed your example and It is working almost fine. I see a Overlap from last page on Modal Dialog.
So this needs to be handled by classes what we are providing..? For example ( modal,modal-content ) in https://codepen.io/hunterliu1003/pen/ZEWoYeE .

You can use overlay-class property to overwrite the overlap style.
If you don't want to display overlay, you can set hide-overlay property true to hide.

I tried using z-index property of vue-final-modal but that doesn't seems to be working.
I am attaching the screenshot of my problem. The tab coming over Modal are of DOJO framework.
As we are partially migrating application to Vue.js

There might be a lot of situations cause z-index not work.
You can try set attach property to body, and then the modal will be append to body element.
<vue-final-modal attach="body">
    ...
</vue-final-modal>

Thank you very much for all the help It worked with  set hide-overlay property true to hide.
It is looking all good


install.sh not working on RHEL 8.4

OS Details
NAME="Red Hat Enterprise Linux"
VERSION="8.4 (Ootpa)"
ID="rhel"
ID_LIKE="fedora"
VERSION_ID="8.4"
PLATFORM_ID="platform:el8"
PRETTY_NAME="Red Hat Enterprise Linux 8.4 (Ootpa)"
Issue
I'm attempting to install on an AWS EC2 instance running RHEL 8.4. I successfully ran
curl -sfL https://raw.githubusercontent.com/redhat-et/microshift/main/install.sh > install.sh. However, when executing install.sh, it gets stuck (hangs) on line 167 (register_subs) below:
# Script execution
get_distro
get_arch
if [ $DISTRO = "rhel" ]; then
    register_subs
fi

Within the function register_subs(), the command sudo subscription-manager status seems to be the one that's hanging. When I run this command in my shell myself, it displays
[root@ip-166-28-35-54 ec2-user]# sudo subscription-manager status
+-------------------------------------------+
   System Status Details
+-------------------------------------------+
Overall Status: Unknown

and hangs until I CTRL+C.

i don't have a rhel to test right now but it looks there are previous subscription-manager hang incidents.
As a workaround, you might try add a timeout command for this line

Thanks @rootfs, but even something like SOMEVAR=$(sudo timeout 3 subscription-manager status) fails to set SOMEVAR because I think when timeout kills the command it also cancels the setting of the variable

that's fine, the line below takes care of that, but again it runs subscription-manager, which will hang...
i suggest you fix the hang issue first, maybe this one can help

/assign @cooktheryan


[Bug]: Query fails to run when it contains filter condition derived from table default search value

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When table default search text is set and the connected query filters data based on this search text, then query fails to run
Steps To Reproduce

Generate a CRUD page
On the table widget, set a default search text
Confirm the query uses this filter condition
Run the query and observe filter condition does not work

https://www.loom.com/share/f65291a649d041edb8b0b5f1ca6b2a6c
Public Sample App
No response
Version
Cloud

@ramsaptami in the query i see you are searching in id column, where as kutch is a name. Can you check by changing the where condition to use name instead of id and let me know if this is occurring?


How can I pause the running task and later resume  it or retry the failed task even if it exceed the Retry times？

When I use Tekton v0.25.0 , I met some scenes in actual use.
scene 1 :  I make up  a Pipeline with several Task , one of the Pipeline's Task runs failed .  I do not want to replay  the succeed Task , only want to restart the failed Task.
I know  Tekton has a feature with keyboard retry which  can retry the failed Task.  But we still met a problem that if the Task's failure time exceed the retry time I set , how can I restart the Task  without replay the former succeed Task？
scene 2 ：I make up  a Pipeline  with several Task , when the Pipeline runs to a Task，out of some reason ,  I want's to pause the Task to do some manual intervention operation  after that  I want to continue the Pipeline to go on the following Task .
For example ,   Before I execute automated testing  , I must deploy my service and the service  must be available .  I want to pause the Pipeline  until the Deployed service is available  ，and then I send a signal to resume  the Pipeline's running ， so that the automated testing Task can get  correct results.
Is there any solution to resolve the scene I mentioned above?
Hope for help,  Thanks a lot

TEP-0123 - Specifying on-demand-retry in a pipelineTask addresses the first scene (use case) mentioned here.


support a min version in the metadata

fail to install if running cassandra version doesn't meet the version requirements specified in the metadata

https://datastax-oss.atlassian.net/browse/PYTHON-301


Update zh_cn.json

Here they are!  :D

Hey, I've change the config system yet again and added 2 new config options. If your willing, please update your translations to match the newest en_us in the major_refactor branch. This should be the last time.


Comprehensive tool documentation: PSAT, CLI How To, Site analysis

Feature Request
Description:
Provide comprehensive and effective tool documentation related to how to use PSAT's extension and CLI capabilities, as well as the application of both tools to the analysis of sites.
User Story:
As a developer/user I want access to clear documentation enabling me to easily find the information I need to fully leverage the PSAT's capabilities.
Acceptance Criteria:

[ ] PSAT and CLI's how tos
[ ] Site Analysis methodology
[ ] Warning on starting analyses on fresh Chrome instances to avoid issue with previously set cookies.
[ ] Special cases related to the debugging of cookie scenarios which may not be part of PSAT feature set (e.g. CORS)
[ ] One-click commands for setting evaluation environment

@gagan0123 Please provide details to be included in cases such as the CORS scenario.

@fellyph This is the issue encompassing PSAT How To.

@amedina about the following items:
1.PSAT and CLI's how tos we have this section are you ok with it or do you want to update it?

 Warning on starting analyses on fresh Chrome instances to avoid issues with previously set cookies.

I can update the first paragraph at Evaluation Evironment
To evaluate the impact of the Privacy Sandbox changes on Chrome, you now have two options for configuring your evaluation environment. They have a complete view of the impacts of cookie-related changes.

You will need a brand-new Chrome instance with no extensions and no other cookies, so before starting tests, have an instance of Chrome free of extensions and without external cookies.
- To clean the cookies from your browser, access: `chrome://settings/clearBrowserData`
- To disable the non-related extension for testing, access: `chrome://extensions/`

Creating a new Chrome profile for testing will help with this process. [Here at Chrome Documentation](https://support.google.com/chrome/answer/2364824?hl=en&co=GENIE.Platform%3DDesktop), you can get more information about it.

 One-click commands for setting evaluation environment

Is it this solution with a terminal command accomplished the on-click solution:

@fellyph These sections are not at the level of quality needed. Currently the How To section just mentions the features, but there is no further guidance. Not the working in the issue title.
'

Is it this solution with a terminal command accomplished the on-click solution:

It seems you did not complete writing this part. What do you mean?

@amedina It was a question related to the CLI command solving the task related to a one-click solution.

Updates are now available in the Wiki, closing this issue.


The learning rate plot in Comet is not the expected one

Hi! I've been trying the comet.ml integration and I must say this has been a great addition to the framework. 🙌
I wanted to exploit it to keep track of the learning rate updates, but the lr being plot is not the one that I expected, especially when trying the learning_rate_warmup_epochs option, which I set to 6 as suggested. The learning rate that is plot on comet is the one set in learning_rate, and it's constant for the first epochs.
Could this be related to this error?
COMET ERROR: Failed to extract parameters from Optimizer.init()
To Reproduce

Setup comet
Set  learning_rate_warmup_epochs option to 6

Expected behavior
I expected to see the lr increase in the first 6 epochs, reach the lr set in learning_rate, and eventually decrease, as I set also reduce_learning_rate_on_plateau .
Actual behavior
The lr is equal to the set learning_rate in the first epochs, and eventually decreases due to reduce_learning_rate_on_plateau .

@lucaventurini We'd be glad to take a look at that over on https://github.com/comet-ml/issue-tracking/issues
And glad you find it useful!

@dsblank let me know if you need help on my side.

@lucaventurini  @dsblank where you able to solve the issue?

@Lothiraldan you have some insight into the current status? (I'll be back from vacation next week).

@lucaventurini Did you solve your problem? I didn't see an issue in the comet.ml tracker.

@lucaventurini was the problem solved? Can you please try with the latest version on master?


MIPS address setting macroses

For now Ghidra doesn't apply lui/addiu pairs, and sets reference only at the moment of the address using. But it will be great to find all references to some address, that was created from two parts.
Is it possible to add such functionality?

There is a setting in the MipsConstantRefernenceAnalyzer - Mark dual instruction references that should do what you want.  It is off by default.


Migration to add auto_update_facets to the customlists table

Description
Migration to add auto_update_facets to the customlists table

Motivation and Context
Forgot to add the migration script for the auto-update-facets column in customlists.

How Has This Been Tested?
Unit tests run.
Manually ran migrate_database and found the column has been created.

Checklist

[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.

I merged this immediately to avoid any broken dev/dependabot builds.


Handle exceptions while fetching offsets

Addresses https://github.com/apache/incubator-druid/issues/7600.
Summary: If the supervisor sees an exception while fetching an offset, this method exists immediately, thereby disregarding other partitions. This PR addresses this bug by handling any exceptions encountered while fetching offsets.

hey @shivtools it seems the tests are failing due to checkstyle violation, could you please check that [ERROR] /home/travis/build/apache/incubator-druid/indexing-service/src/main/java/org/apache/druid/indexing/seekablestream/supervisor/SeekableStreamSupervisor.java:2446:9: '}' at column 9 should be alone on a line. [RightCurly]


Run checks against a single group, single entry, or selection of entries

We can hook into context menus for the groups listbox and the entries listbox. We can also get the selected group and the selection of entries. We can add a separator and a submenu like we do in "Tools" for all items, then only run the checks for the selected group, or selected entries.
This would require changes to BaseChecker to accept a PwGroup, but then the entire database check just passes database.RootGroup to that method:
BaseChecker.cs
public async Task<List<BreachedEntry>> CheckDatabase(bool expireEntries, bool oldEntriesOnly, bool ignoreDeleted, IProgress<ProgressItem> progressIndicator)
{
    return await CheckGroup(passwordDatabase.RootGroup, expireEntries, oldEntriesOnly, ignoreDeleted, progressIndicator);
}

public abstract Task<List<BreachedEntry>> CheckGroup(PwGroup group, bool expireEntries, bool oldEntriesOnly, bool ignoreDeleted, IProgress<ProgressItem> progressIndicator);

If this is something you are interested in, I can make a pull request.

I use KeePass to store lots of different stuff, not just passwords, so running this takes easily 2.5-3x as long as it should because it checks every group instead of just the group with actual internet passwords. I'd really like to see the ability to perform checks on individual groups added.


Tree traversal in MATLAB

All code sections are included. Accidentally deleted some lines during debugging and added them back in.

Accidentally pushed to the wrong branch, fixing now

Okay that should be fixed

I'm not a fan of cell arrays either, but it seems it's the only way to get type-agnostic tree structures.
I'll work on the updates now.

That should be good to go!


Advise use of protobuf extension

Closes #800.
Should we suggest this in other packages which support grpc, and in google-cloud-php as well?

Good question, it seems reasonable to have this mentioned in other areas as well.

@dwsupplee @michaelbausor I updated all the other components to suggest protobuf and grpc, as appropriate.


BSC - NFT not fetching

BSC - NFT not fetching  properly.  0x08f2443EC6Dd59E47Dd17631EfCe5b99d54718C5

@hboon  any update ?

Will check. Sorry, missed it previously.

ok


Filter Widgets Data only by numeric fields

This is relevant for all widgets (But I think this can be done in our infra level)
Nowadays, when filtering Widgets data, we use location_info (within RequestParams).
The problem is that the location_info dict ALWAYS contains both strings (like street1_hebrew or road_segment_name) and numbers (street1, road_segment_id)
Note: these strings are used both in filtering of data AND when creating various titles.
We don't want to "harm" the titles functionality BUT We want to make sure to filter data ONLY by numeric fields:
Sub-urban: by road_segment_id (and road1 when necessary, for example when comparing the whole road's segments like in TopRoadSegmentsAccidentsPerKmWidget
Urban: by yishuv_symbol and street number (data is filtered both in street1 and street2 - with an "or" between them)
Current FE queries as as follows:
Sub-urban: https://www.anyway.co.il/api/infographics-data?road_segment_id=900020&years_ago=5
Urban: https://www.anyway.co.il/api/infographics-data?lang=he&street1_hebrew=אבן גבירול&yishuv_name=תל אביב -יפו&years_ago=5
We want to be compatible for now, meaning these queries should continue working.
After creating an API query for street by yishuv_symbol and street number, I'll make sure it's implemented it in FE, and we'll be able to remove query by Hebrew names.
Note 2: After checking, street1 param from location_info  is of python type <class 'sqlalchemy.util._collections.result'> and not int, perhaps this should be changed in order for this to work? (or are we taking the first value from this collection? Not sure why this is implemented in this way but I believe you'll be able to solve this one quite easily.
Note 3: I suggest that in the 2 DB tables where address exist (I think it's only markers_hebrew, involved_markers_hebrew) we'll join the street and yishuv_name using the joins in db_views.py file (I elaborated a bit more on this in the cities issue: https://github.com/data-for-change/anyway/issues/2665)

@ziv17  as discussed attaching production Widgets list

@ziv17 you are right, no effect on the result.
My intention was possible changing the FE query for infographics-data to be by street/city id rather than names. Nowadays it's by name). For example this one

If it's all done and no other additions, we can close this one :)


Stopped running post action

Current dependencies on/for this PR:

main

PR #12   👈

This comment was auto-generated by Graphite.


feat: enforce recreation (force updating) objects when operator versi…

…on changes
What's changed and what's your intention?
PLEASE DO NOT LEAVE THIS EMPTY !!!
Please explain IN DETAIL what the changes are in this PR and why they are needed:

Before the change, the EnableForceUpdate feature gate won't work even if there's a conflict when the object is created by the same version of operator. Though the feature was intended to address inconsistency after upgrading operator, it's now getting confusing as https://github.com/risingwavelabs/risingwave-operator/pull/727 is merged.
The PR changes the behaviour to let operator automatically address inconsistency problems on operator version changes, and do force updates when the feature gate is on.

Checklist

[ ] I have written the necessary docs and comments
[ ] I have added necessary unit tests and integration tests

Refer to a related PR or issue link (optional)

A patch release will be created once the PR's merged.


Prefect Block Resource: Type error in docs

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request.
Please do not leave +1 or me too comments, they generate extra noise for issue followers and do not help prioritize the request.
If you are interested in working on this issue or have submitted a pull request, please leave a comment.

Attribute in Prefect Block Resource
Data Type Difference
In the documentation it states that the type for Attribute Data -> String, Sensitive
This simply is not true, when I built it, it was of type map(string).
Godspeed and happy developing

Hey @scondo-prof, thanks for reaching out.
This field is a bit unique - it's technically a schema.StringAttribute, but it [has a CustomType of jsontypes.NormalizedType. This is what helps us unmarshal user-provided content into a format that the API recognizes. Under the hood, we unmarshal it into map[string]interface.

This simply is not true, when I built it, it was of type map(string).

Could you show an example of the configuration you were using so I can see what specific example you have in mind? This field was a bit weird to work with since it had to be so flexible, so I'm curious to see if we could use map here (I don't think so, because that means you have to be able to predict every potential key in the Terraform provider code - which is just not very reasonable for Blocks since there are so many different kinds of them).

Thank you for the response mitchell:
main.tf
terraform {
  required_providers {
    prefect = {
      source = "prefecthq/prefect"
    }
  }
}

provider "prefect" {
  account_id   = var.prefectAccountId
  workspace_id = var.prefectWorkspaceId
  api_key      = var.prefectApiKey
}

resource "prefect_block" "prefectBlock" {
  data      = jsonencode(var.prefectBlockData)
  name      = "${var.resourceName}-prefect-block"
  type_slug = var.prefectBlockTypeSlug
}

variables.tf
variable "prefectAccountId" {
  type    = string
  default = "9759ff6b-6f30-404f-92f3-02c593f6f7aa"
}

variable "prefectWorkspaceId" {
  type    = string
  default = "1733fd7b-18ad-4093-a0a1-4f128be45747"
}

variable "prefectApiKey" {
  type      = string
  sensitive = true
}

#Dependent on prefectBlockTypeSlug, use "prefect block type inspect <slug>" to see the required format
variable "prefectBlockData" {
  type = map(string)
}

variable "resourceName" {
  type = string
}

variable "prefectBlockTypeSlug" {
  type = string
  validation {
    condition     = contains(["aws-credentials", "aws-secret", "azure", "azure-blob-storage-container", "azure-blob-storage-credentials", "azure-container-instance-credentials", "azure-container-instance-job", "azure-cosmos-db-credentials", "azureml-credentials", "bigquery-warehouse", "bitbucket-credentials", "bitbucket-repository", "database-credentials", "databricks-credentials", "date-time", "discord-webhook", "docker-container", "docker-host", "docker-registry", "docker-registry-credentials", "ecs-task", "email", "email-server-credentials", "cloud-run-job", "gcp-credentials", "gcs", "gcs-bucket", "gcpsecret", "github", "github-credentials", "github-repository", "gitlab-credentials", "gitlab-repository", "json", "kubernetes-cluster-config", "kubernetes-credentials", "kubernetes-job", "lambda-function", "local-file-system", "mattermost-webhook", "ms-teams-webhook", "minio-credentials", "modal-credentials", "opsgenie-webhook", "pager-duty-webhook", "process", "remote-file-system", "s3", "s3-bucket", "smb", "sqlalchemy-connector", "secret", "sendgrid-email", "shell-operation", "slack-credentials", "slack-webhook", "snowflake-connector", "snowflake-credentials", "string", "twilio-sms", "vertex-ai-custom-training-job", "webhook", "dbt-cli-bigquery-target-configs", "dbt-cli-global-configs", "dbt-cli-postgres-target-configs", "dbt-cli-profile", "dbt-cli-snowflake-target-configs", "dbt-cli-target-configs", "dbt-cloud-credentials", "dbt-core-operation"], var.prefectBlockTypeSlug)
    error_message = "Variable prefectBlockTypeSlug must be one of the following values: : aws-credentials, aws-secret, azure, azure-blob-storage-container, azure-blob-storage-credentials, azure-container-instance-credentials, azure-container-instance-job, azure-cosmos-db-credentials, azureml-credentials, bigquery-warehouse, bitbucket-credentials, bitbucket-repository, database-credentials, databricks-credentials, date-time, discord-webhook, docker-container, docker-host, docker-registry, docker-registry-credentials, ecs-task, email, email-server-credentials, cloud-run-job, gcp-credentials, gcs, gcs-bucket, gcpsecret, github, github-credentials, github-repository, gitlab-credentials, gitlab-repository, json, kubernetes-cluster-config, kubernetes-credentials, kubernetes-job, lambda-function, local-file-system, mattermost-webhook, ms-teams-webhook, minio-credentials, modal-credentials, opsgenie-webhook, pager-duty-webhook, process, remote-file-system, s3, s3-bucket, smb, sqlalchemy-connector, secret, sendgrid-email, shell-operation, slack-credentials, slack-webhook, snowflake-connector, snowflake-credentials, string, twilio-sms, vertex-ai-custom-training-job, webhook, dbt-cli-bigquery-target-configs, dbt-cli-global-configs, dbt-cli-postgres-target-configs, dbt-cli-profile, dbt-cli-snowflake-target-configs, dbt-cli-target-configs, dbt-cloud-credentials, dbt-core-operation"
  }
}

###outputs.tf
output "prefectBlockId" {
  value = prefect_block.prefectBlock.id
}

output "prefectBlockAccountId" {
  value = prefect_block.prefectBlock.account_id
}

output "prefectBlockWorkspaceId" {
  value = prefect_block.prefectBlock.workspace_id
}

###test.tfvars
prefectApiKey = "<api-key>"

prefectBlockData = {
    value = "test-data"
}

resourceName = "test"

prefectBlockTypeSlug = "secret"

This results in the image:

Below is the example of variables.tf being modified and .tfvars being modified for the data attribute to be of type string:
###variables.tf
variable "prefectAccountId" {
  type    = string
  default = "9759ff6b-6f30-404f-92f3-02c593f6f7aa"
}

variable "prefectWorkspaceId" {
  type    = string
  default = "1733fd7b-18ad-4093-a0a1-4f128be45747"
}

variable "prefectApiKey" {
  type      = string
  sensitive = true
}

#Dependent on prefectBlockTypeSlug, use "prefect block type inspect <slug>" to see the required format
variable "prefectBlockData" {
  type = string
}

variable "resourceName" {
  type = string
}

variable "prefectBlockTypeSlug" {
  type = string
  validation {
    condition     = contains(["aws-credentials", "aws-secret", "azure", "azure-blob-storage-container", "azure-blob-storage-credentials", "azure-container-instance-credentials", "azure-container-instance-job", "azure-cosmos-db-credentials", "azureml-credentials", "bigquery-warehouse", "bitbucket-credentials", "bitbucket-repository", "database-credentials", "databricks-credentials", "date-time", "discord-webhook", "docker-container", "docker-host", "docker-registry", "docker-registry-credentials", "ecs-task", "email", "email-server-credentials", "cloud-run-job", "gcp-credentials", "gcs", "gcs-bucket", "gcpsecret", "github", "github-credentials", "github-repository", "gitlab-credentials", "gitlab-repository", "json", "kubernetes-cluster-config", "kubernetes-credentials", "kubernetes-job", "lambda-function", "local-file-system", "mattermost-webhook", "ms-teams-webhook", "minio-credentials", "modal-credentials", "opsgenie-webhook", "pager-duty-webhook", "process", "remote-file-system", "s3", "s3-bucket", "smb", "sqlalchemy-connector", "secret", "sendgrid-email", "shell-operation", "slack-credentials", "slack-webhook", "snowflake-connector", "snowflake-credentials", "string", "twilio-sms", "vertex-ai-custom-training-job", "webhook", "dbt-cli-bigquery-target-configs", "dbt-cli-global-configs", "dbt-cli-postgres-target-configs", "dbt-cli-profile", "dbt-cli-snowflake-target-configs", "dbt-cli-target-configs", "dbt-cloud-credentials", "dbt-core-operation"], var.prefectBlockTypeSlug)
    error_message = "Variable prefectBlockTypeSlug must be one of the following values: : aws-credentials, aws-secret, azure, azure-blob-storage-container, azure-blob-storage-credentials, azure-container-instance-credentials, azure-container-instance-job, azure-cosmos-db-credentials, azureml-credentials, bigquery-warehouse, bitbucket-credentials, bitbucket-repository, database-credentials, databricks-credentials, date-time, discord-webhook, docker-container, docker-host, docker-registry, docker-registry-credentials, ecs-task, email, email-server-credentials, cloud-run-job, gcp-credentials, gcs, gcs-bucket, gcpsecret, github, github-credentials, github-repository, gitlab-credentials, gitlab-repository, json, kubernetes-cluster-config, kubernetes-credentials, kubernetes-job, lambda-function, local-file-system, mattermost-webhook, ms-teams-webhook, minio-credentials, modal-credentials, opsgenie-webhook, pager-duty-webhook, process, remote-file-system, s3, s3-bucket, smb, sqlalchemy-connector, secret, sendgrid-email, shell-operation, slack-credentials, slack-webhook, snowflake-connector, snowflake-credentials, string, twilio-sms, vertex-ai-custom-training-job, webhook, dbt-cli-bigquery-target-configs, dbt-cli-global-configs, dbt-cli-postgres-target-configs, dbt-cli-profile, dbt-cli-snowflake-target-configs, dbt-cli-target-configs, dbt-cloud-credentials, dbt-core-operation"
  }
}

###test.tfvars
prefectApiKey = "<api-key>"

prefectBlockData = <<-EOF
    {
    value = "test-data"
}
EOF

resourceName = "test"

prefectBlockTypeSlug = "secret"

Resulted in this error:

I also tried a non multiline for the .tfvars:
###test.tfvars
prefectApiKey = "<api-key>"

prefectBlockData = "{value = \"test-data\"}"

resourceName = "test"

prefectBlockTypeSlug = "secret"

which resulted in this error:

I look forward to your response.
Best,
Scott

Thanks for the extra context @scondo-prof, that helps a lot. I opened https://github.com/PrefectHQ/terraform-provider-prefect/pull/257 to make this clearer. Could you take a look and see if that helps?

Mitch,
First, I would like to apologize for calling you Mitchell in my response.
Second, that would be incredibly helpful!
I have another question Mitch unrelated to my confusion in the documentation,
Will there be a Terraform Resource for Account API Keys developed?
I know that there is one for ServiceAccount which in turn gives an API key with a permission set, but that requires to be on a paid version of Prefect. As a developer whom cant afford that at the current moment, I was wondering if you and your fellow incredible devs were thinking about creating that resource? For people whom are lower on the totem pole and are utilizing the lowest tier of Prefect, this Terraform Resource would truly help with credential rotation to keep an environment as secure as possible.
Best,
Scott

No worries at all! I go by either name.
We'll get that PR merged, just need to fix a CI configuration drift that's making acceptance tests fail.
To your other question: this sounds very reasonable, and we definitely plan to (and already do) support APIs that are not specific to Prefect Cloud (see https://github.com/PrefectHQ/terraform-provider-prefect?tab=readme-ov-file#supported-objects).
Let me open a separate issue and we can get on the same page there.

The data field description is now updated at https://registry.terraform.io/providers/PrefectHQ/prefect/latest/docs/resources/block#required.


Add Enumerator methods

Add the common Enumerator methods in addition to those from Enumerable.

each_with_index
each_with_object
with_index
with_obj

The type exists and is used for many important methods, such as Array#each, and the common with_index adapter. each_with_object and with_obj can be added later.


Oopsy: E7N Initial

There aren't a lot of mistakes to be made in this encounter either. (Or, more properly, there are only a LOT of variations of a few categories.)
I'm really not happy with the handling for Dark's/Light's Course. There's gotta be a better way than spending five(!) triggers on two mechanics. However, I'm hesitant to move any of that functionality in-line since we tend to keep things separated elsewhere in the repository. Improvements to this mess would be welcome. (Also, anything we do here "should" be a suitable drop-in for Savage once the ability IDs are switched out.)
Another thing that could use improvement, as I mentioned in-line, is that I don't like the regex soup I used to check for damage events on Course casts. Should I have used the regex: Regexes.ability() construction here instead? Not sure how that all interacts with Oopsy. Or did we want literally a trigger per ID?
I'm also planning to improve this by having a check for each player's position when the stun from Away With Thee wears off. Unlike most "can take you off the platform" mechanics, it (seems to me it) should be definite whether the player's teleport left them on the platform or not. (I need to test further, but I think the platform coordinates are 80,80-120,120, with origin at northwest off the platform. Or I could have been totally misreading the log lines when I checked last night. It was late, and Daylight Saving is horrible.) Maybe someday the log lines can tell us "fell off" directly. A human can dream...

Actually. Re-reading the OopsyReadme, would Away With Thee and similar I-fell-off mechanics be appropriate uses for the deathReason property? It wouldn't be perfect, of course, but it would be way better than nothing.

Okay, from the More Than One times I ran that through, things seem to have triggered correctly. I'm not at all sure the modification I made to OopsyRaidsy.js was  entirely correct/complete, but at least there aren't any console errors. Thanks so much for pointing me to the changes needed there, the new setup is way nicer than a random regex string.

I think ideally somebody tracks down the network data for falling off the edge/ death wall and we could add that universally.

Fair enough. I've been experimenting with deathReason for E8N for the exact same purpose, and it hasn't been going well, so I may just ignore that for now and get that pull request put together.


added a neon button.I followed all the instructions,hoping it will work this time

Issue:

Great work @OuissalFX thank you!
@all-contributors please add @OuissalFX for code


Test metrics

Overview
This PR adds the same suite of unit tests for FBetaMeasureMultiLabel as exist for its parent class FBetaMeasure.

@MichalMalyska There is a block of code in the parent class (see here) that was dropped from our FBetaMeasureMultiLabel. It looks like it handles the situation where no document is labeled for a given class. Do you have any idea how this translates to the multi-label case? I don't really understand the concept of a "bin" here (several of their variables are named *_bins). If we can solve that, I can write the remaining 6 unit tests.

bincount just sums up the examples, but it has the edgecase that if input is all 0s then it returns an empty tensor. I don't really get their implementation without digging in too deep but it seems to just try to cover edge cases where predicitions are all zero or labels are all zero.

I am going to merge this as is because it works (and has test coverage) for our intentend use case. I will open seperate issues to make this code work with masking/batches with no gold labels.


Add labom2iformation.txt

@dkuzulu Please provide us with:

the school official website URL
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

@dkuzulu I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for the understanding!


[ENH] -- Option to exclude model_extra from repr

Initial Checks

[X] I have searched Google & GitHub for similar requests and couldn't find anything
[X] I have read and followed the docs and still think this feature is missing

Description
The model_config attribute gives us the ability to "allow", "ignore", or "forbid" extra values, but I'd like to hide them similar to how Field(..., repr=False) can hide required/expected/known fields. My current approach overrides the model's __repr_args__ method and excludes the logic around pydantic_extra:
    def __repr_args__(self) -> _repr.ReprArgs:
        for k, v in self.__dict__.items():
            field = self.model_fields.get(k)
            if field and field.repr:
                yield k, v

-       # `__pydantic_extra__` can fail to be set if the model is not yet fully initialized.
-       # This can happen if a `ValidationError` is raised during initialization and the instance's
-       # repr is generated as part of the exception handling. Therefore, we use `getattr` here
-       # with a fallback, even though the type hints indicate the attribute will always be present.
-       try:
-           pydantic_extra = object.__getattribute__(self, '__pydantic_extra__')
-       except AttributeError:
-           pydantic_extra = None
-
-       if pydantic_extra is not None:
-           yield from ((k, v) for k, v in pydantic_extra.items())
        yield from ((k, getattr(self, k)) for k, v in self.model_computed_fields.items() if v.repr)

But this seems hacky. I'd like to propose a new setting for the model_config, "extra_repr", and wrap the pydantic_extra logic in __repr_args__ with a conditional:
    def __repr_args__(self) -> _repr.ReprArgs:
        for k, v in self.__dict__.items():
            field = self.model_fields.get(k)
            if field and field.repr:
                yield k, v

+       if self.model_config.get("extra_repr", True):
+           # `__pydantic_extra__` can fail to be set if the model is not yet fully initialized.
+           # This can happen if a `ValidationError` is raised during initialization and the instance's
+           # repr is generated as part of the exception handling. Therefore, we use `getattr` here
+           # with a fallback, even though the type hints indicate the attribute will always be present.
+           try:
+               pydantic_extra = object.__getattribute__(self, '__pydantic_extra__')
+           except AttributeError:
+               pydantic_extra = None

+           if pydantic_extra is not None:
+               yield from ((k, v) for k, v in pydantic_extra.items())
+
        yield from ((k, getattr(self, k)) for k, v in self.model_computed_fields.items() if v.repr)

Minimal example with expected output after proposed changes:
from pydantic.config import ConfigDict
from pydantic.main import BaseModel

class Foo(BaseModel):

    model_config = ConfigDict(extra="allow", extra_repr=False)

    name: str

foo = Foo(name="Bar", Baz="Spam", Eggs="Spam", More="Spam")
print(repr(foo))
# >>> Foo(name='Bar')

print(foo.model_extra)  # No effect.
# >>> {'Baz': 'Spam', 'Eggs': 'Spam', 'More': 'Spam'}

print(foo.model_dump())  # No effect.
# >>> {'name': 'Bar', 'Baz': 'Spam', 'Eggs': 'Spam', 'More': 'Spam'}

As you can see, this keeps the Foo representation clean/succinct without removing the model_extra.
Affected Components

[ ] Compatibility between releases
[ ] Data validation/parsing
[ ] Data serialization - .model_dump() and .model_dump_json()
[ ] JSON Schema
[ ] Dataclasses
[x] Model Config
[ ] Field Types - adding or changing a particular data type
[ ] Function validation decorator
[ ] Generic Models
[X] Other Model behaviour - model_construct(), pickling, private attributes, ORM mode
[ ] Plugins and integration with other tools - mypy, FastAPI, python-devtools, Hypothesis, VS Code, PyCharm, etc.

Hi @it176131,
Thanks for the feature request.
This reminds me a bit of https://github.com/pydantic/pydantic/issues/9415.
On one hand, I think having a config setting that makes repr modification easier would be helpful - especially for some common cases related to extra, defaults, etc.
At the same time, I haven't seen too much demand for this, and I think the approach where you override the __repr_args__ function makes sense for your case in the short term.
Curious to hear other ideas from folks re what API might make the most sense in model_config going forward.

I'll also note, I think we can close either this issue or the other one + merge them into one feature request once we've ironed out some of the details.

Hi @it176131,
Thanks for the feature request.
This reminds me a bit of #9415.
On one hand, I think having a config setting that makes repr modification easier would be helpful - especially for some common cases related to extra, defaults, etc.
At the same time, I haven't seen too much demand for this, and I think the approach where you override the __repr_args__ function makes sense for your case in the short term.
Curious to hear other ideas from folks re what API might make the most sense in model_config going forward.

After reading over #9415 again I do see some similarities. Combing over my searches, #1831, #9712, and #3192 pop out as well.

I'll also note, I think we can close either this issue or the other one + merge them into one feature request once we've ironed out some of the details.

I'd be open to combining requests and collaborating on a PR.

@it176131,
Sounds great. Let me chat with the open source team tomorrow morning about potential APIs for this, then I'll get back to you with our thoughts :).

Yeah honestly, I think we'd lean more on the side of customizing __repr_args__ in user code for now.
That being said, if you have a compelling proposal for a way to add support for this, I'd be happy to review a PR :).

Yeah honestly, I think we'd lean more on the side of customizing __repr_args__ in user code for now.
That being said, if you have a compelling proposal for a way to add support for this, I'd be happy to review a PR :).

Sure thing! I'll link back to this issue.

@sydney-runkle should we close this as not planned?


SecureVarFile not used on force-unlock anymore

A defect got introduced where the secure var file is no longer used for force-unlock (if available) which will cause the task to attempt to use the AZ cli which may not be present.

@zanawar fix for this has been published to marketplace with 0.6.6. Thanks for reporting this and the fix contribution!

@zanawar fix for this has been published to marketplace with 0.6.6. Thanks for reporting this and the fix contribution!


Google Compute Platform (GCP) Runner.

This runner runs MLBoxes on GCP. In the current implementation, the following is assumed:

User has account on GCP and knows project ID and zone.
User has downloaded the Service Account JSON Key File.
User has added a public key to the GCP project.
User knows what the instance name will be used, and updated the .ssh/config file. It can initially contain some fake IP address. For instance:Host gcp-f1-micro
   HostName SOME_FAKE_IP_ADDRESS
   User GCP_USER_LOGIN
   IdentityFile PATH_TO_PRIVATE_KEY

[WARNING] Since compute instance IP addresses do not have to be static, the GCP runner can update .ssh/config file.
[WARNING] Creating instance will cost some money. In the examples, f1-micro instances are used that are free (if there's one instance of this type).
[WARNING] The GCP runner DOES NOT stop and delete the created instance.

Configure phase
The following logic is currently implemented:

Check if SSH config is available for the compute instance.
Connect to GCP.
If instance does not exist, create it.
If instance is not running, start it.
If IP has changed, update the .ssh/config file.
Configure compute instance (install docker, pip and virtualenv).
Call SSHRunner::configure.

Run phase
GCP runner does not implement run phase - it calls SSH runner instead.

can we make the platform file compliant with the existing platform schema (or maybe update the platform schema to adapt to this change)?

have fields specifying schema type and version
the "platform" field is used differently in docker and k8s runners, where the name and version of the platform is provided. I think for gcp we will still need those info (i.e. name and version of gcp) in addition to the reference to the ssh config. One suggestion is to provide the reference to "ssh.yaml" in a separate field whose name should indicate the reference to a sub-platform.

@sub-mod @xyhuang This is the final version I used for recording the videos. I also added a documentation page. I think it's OK to add it for now and then unify platform configs later.

lgtm


Flat View and Live File Filter Box

What feature or improvement do you think would benefit Files?
Hello Files Team,
I'm proposing two features that would greatly enhance the utility of the Files file manager: Flat View and a Live Filter field.
Flat View: This feature would allow users to display all files from a selected folder and its subfolders in a single, flattened list. Having used this feature in Dopus for decades, I am convinced that it is incredibly useful for navigating and finding files in complex, nested directory structures.
Live Filter Field: Coupled with the Flat View feature, a Live Filter field enables users to dynamically filter the displayed files based on partial filename matches or specific file extensions. This functionality is essential for quickly locating files among hundreds or thousands of files in large directories, particularly while in Flat View.
These two features would significantly boost the efficiency of file searching and management, especially for users dealing with extensive folder hierarchies.
Below is an animated screenshot from Microsoft Access showing the principle of a Live Filter Box (I believe you have to click on the small triangle at the top right of the screenshot for the animation to run).
Thank you for considering these additions to Files.

Requirements
Develop Flat View: Implement a feature for displaying all files in a folder and its subfolders in a flat list.
Integrate Live Filter: Add a dynamic filter box for file name and/or file extension-based filtering.
UI Update: Modify the user interface to incorporate Flat View and Live Filter.
Optimize Performance: Ensure efficient handling of large directories.
Files Version
<IP_ADDRESS>
Windows Version
10.0.19045.3803
Comments
No response

Thank you for the feedback, we're planning support for a filter feature in #14391 and a flat view in #14319 (you can already do something similar with libraries).

Thank you for the feedback, we're planning support for a filter feature in #14391 and a flat view in #14319 (you can already do something similar with libraries).

Thank you, looking forward to it

If you have any design ideas on how this can be implemented either in addition or as part of the existing search feature, please us know in #14391.


port [libffi] not work ASM [x64-windows]

win64.S
c1: fatal error C1356: unable to find mspdbcore.dll
Microsoft (R) Macro Assembler (x64) Version 14.14.26429.4
Copyright (C) Microsoft Corporation.  All rights reserved.

 Assembling: win64.asm
win64.asm(1) : error A2088:END directive required at end of file

not CL.EXE, must ML64.EXE
edit CMakeLists.txt
project(libffi ASM C)
work x64
project(libffi C)
enable_language(ASM_MASM)

discussion https://gitlab.kitware.com/cmake/cmake/issues/17532
error win64.asm(1) : error A2088:END directive required at end of file

no find win64.asm

p.s. ver 3.2.1 - 3.3 https://github.com/libffi/libffi/releases
p.p.s Fork that uses the Cmake build system https://github.com/newlawrence/Libffi

Windows x86 - Microsoft Visual C++ Compiler
Windows x86-64 - Microsoft Visual C++ Compiler

Hi @Voskrese ,
libffi:x64-windows built successful on my VM.
Here is log:

PS F:\vcpkg> .\vcpkg.exe install libffi:x64-windows
The following packages will be built and installed:
libffi[core]:x64-windows
Starting package 1/1: libffi:x64-windows
Building package libffi[core]:x64-windows...
-- Using cached F:/vcpkg/downloads/libffi-libffi-v3.1.tar.gz
-- Using source at F:/vcpkg/buildtrees/libffi/src/v3.1-206c189062
-- Applying patch F:/vcpkg/ports/libffi/export-global-data.patch
-- Applying patch failed. This is expected if this patch was previously applied.
-- Configuring x64-windows
-- Building x64-windows-dbg
-- Building x64-windows-rel
-- Performing post-build validation
-- Performing post-build validation done
Building package libffi[core]:x64-windows... done
Installing package libffi[core]:x64-windows...
Installing package libffi[core]:x64-windows... done
Elapsed time for package libffi:x64-windows: 20.13 s

Total elapsed time: 20.13 s

hmm... yes, work
Not clear what was the problem apparently with the environment


Certs expired

Just checked this out right now, and it appears that the webpack.js.org certs have expired. We probably want to get that renewed ASAP, but I assume that is with the js.org folks?

https://github.com/js-org/js.org/issues/14

It should be fixed now.


Recipe: Cache CSS and fonts

Serve stale files for CSS and fonts on load
Fetch new version into cache (on install or on fetch?)

Fonts probably didn't update but just CSS, should the service worker only invalidate changed files?

Refresh CSS in client when updates are available

Similar to https://github.com/phamann/embrace-the-network/tree/master/src/stale-while-revalidate

Taking a shot at the caching strategy here …
CSS assets = CSS and media referenced from CSS with low rate of updates (fonts, icons and spritesheets)

Cold cache

SW is installed
CSS assets added to cache in onfetch

Warm cache

CSS assets served from cache via onfetch
Check if a an update request is needed, could be

JSON file that references files and hashes/last-modified
Cache age < configured time (say 6h)

If cache needs an updated

Cache updated fetched assets
Notify frontend of update CSS
Client refreshes CSS files

@lodr what do you think is the best strategy here?

I was thinking about this. I agree on cold-cache strategy. Assuming Request is completely implemented, we can check type and destination along with content-type to identify fonts and CSS.
Fir the first version, the warm cache strategy @digitarald is suggesting coudl work but the problem I see is that an update in a CSS file could imply an update in the dependant HTML. This is something we should look after. Once determined if the new CSS / font is requires, cahing and sending a signal to the client to perform a reload in a controlled way seems to be the best.

Assuming Request is completely implemented, we can check type and destination along with content-type to identify fonts and CSS.

Request is complete aside from cache. Checking content-type would a good way to determine CSS

Fir the first version, the warm cache strategy @digitarald is suggesting coudl work but the problem I see is that an update in a CSS file could imply an update in the dependant HTML. This is something we should look after.

CSS updates are less critical though and the flash-of-mismatching-css/html should be only a short one as the sw should trigger an update within a few hundred ms after page load. Does that make sense?

Yes, it should be short and ultimately, the SW don't cause the reload, it just notifies the client and it's the client which is in charge of reloading the page. What we should put in the recipe is how to reload CSS only (without realoading the entire HTML) by adding a bumping query-parameter that the service worker ignores when looking for the new CSS.
Deciding when to update is more complicated. How did you plan to do? With a cache-age strategy, we should annotate last cache time somewhere (in a custom header inside the response, for instance) then use it to compare with present time to see if another request is needed.
The problem with a JSON strategy is you need to download the JSON file each time to see if something changed.
An alternative combining hashes and cache-age is as follows:
0. When cold-caching, annotate caching time in the response as a header.

When warm-caching, retrieve the resource and read the caching time and ETag.
Compare with now. If max cache-age is surpased, perform a HEAD request and get the ETag.
Compare with former ETag, if they differ, perform the complete FETCH on the resource and recache.

What do you think @digitarald ?

Hey guys, good discussion going on here!
I implemented a first pass at this in #63 that I think is pretty informative as a recipe. I wanted to avoid needing any server-side logic and tried to make the example as self-explanatory as possible. Please try running it in Chrome and let me know if I succeeded. ISTM that a lot of the points brought up in the discussion here would be totally relevant to production code but maybe could be glossed over in the recipe?
Let me know what you think!

When reviewing the PR from @TimAbraldes , I was supponsing some use case but I think I'm not getting the point of this recipe. @digitarald do you mind to explain it to me a little bit more? What is the use case and what is the solution overview? Thanks!


github: Rename COPYING.LIB to LICENSE.md

Github will identify the license in specifically named files,
COPYING.lib is really not one of them nor is it a common practice.
Fixes #708
Signed-off-by: Sean V Kelley<EMAIL_ADDRESS>

Great! Now it's Apache. But this definite a github's bug, it can link to the COPYING.lib but it can't detect the license type ;).
thanks for the patch


Version 2.0 Rewrite

Version 2.0 Rewrite
PRs

#677
#713
#715
#716
#718
#719
#721
#723
#726
#731
#736
#740

Summary:
This PR is for a full rewrite of Dynamoose for version 2.0. This PR is being created very early in the process to gather feedback from the community about the path forward on this project.
If you would like to contribute to this PR please feel free to create a branch off of the 2.0 branch and create a PR to merge into that same branch. Keep in mind the goal is to keep PRs and changes very small. It is highly recommended that you keep PRs small as I will be very strict about the type of PRs that will be accepted (especially in the beginning stages), and a lot of PRs will likely be closed without being merged, so please keep PRs small so you don't waste your own time.
Next Steps:

[x] Figure out how to get Should allow creating instance of Model unit test passing, and allowing to create an instance of an instance of a class
[x] Refactor Model code to be more clean (not having those nested functions in the way they are)
[x] Write Schema tests
[ ] Look into changing dynamoose.aws.ddb & dynamoose.aws.converter both set & revert functions to use same system as Model.defaults where the user can just set a new value and we can access it.
[x] Write tests for 3c408addfdf7e4703708d622605107f2cf24a20a
[x] Currently we are setting Document.Model = this inside of our Model initializer. This will cause issues if we create multiple models, since Document will always look at the last model created. We should add unit tests around this, and fix it.
[x] Model.get
[x] Add behavior to wait for table to be active (init to be done) before running any actions on the Model.
[x] Ensure Document conforms to Schemas (don't save values that don't exist in schema, don't put values that exist in DynamoDB but not in schema in Document)
[x] Scan

[x] Write documentation
[x] Write unit tests

[x] Add support for default values in Schema
[x] Add support for validation functions in Schema
[x] Add support for required properties in Schema
[x] Add support for enum properties in Schema
[x] Add support for forceDefault properties in Schema
[x] Add support for saveUnknown property in Schemas
[x] Add support for indexes

[x] Write unit tests
[x] Manual test
[x] Write documentation

[x] Query

[x] Add Query Support
[x] Write documentation
[x] Write unit tests
[x] Change query.where method to not be alias of query.filter

[x] Model.create

[x] Add support for overwrite option for document.save method
[x] Add Model.create method

[x] Timestamps

[x] Add support for date schema types

[ ] Model.update

[ ] Add support for set if_not_exists
[x] Add test for type mismatch with schema
[x] Add test for custom type (date)
[x] Add support for validation
[x] Add support for enums
[x] Add support for required when deleting properties
[x] Add support for default when deleting properties
[x] Add support for forceDefault
[x] Add support for updating updatedAt timestamp
[ ] Add support for setting to not update updatedAt timestamp when updating

[x] Add support for following schema types/ensure they work:

[x] String Sets (need to add more documentation for sets as well)
[x] Number Sets
[x] Buffers
[x] Buffer Sets
[x] Objects
[x] Arrays

[x] Test support for saveUnknown with sets
[ ] Ensure support for enum for lists, and sets works correctly

[ ] Ensure this works for Model.update when both setting, and appending to an array or set

[x] TTL for documents (#721)

[x] Support Date attributes with storage as seconds as opposed to miliseconds
[x] Add support for expires.returnExpiredItems

[x] Test model capacity

[x] Number
[x] Object
[x] On demand

[ ] Allow for updating model

[x] Capacity
[ ] Indexes

[x] dynamoose.local (#713)
[x] Model.getTableReq (#715)
[x] Document.originalItem (#716)
[x] Model.delete (#718)
[x] Document.delete (#719)
[x] Transaction support (#723)
[x] Add support for removing property without using default value. Maybe we can make something like dynamoose.undefined which would be a symbol. If that is used, it won't use the default value for the property. This can be a replacement for expires.defaultExpires. (#726)
[x] There is currently an issue where if we have an attribute that is an Object type (ex. address), and we have an array of strings for the saveUnknown setting (ex. saveUnknown: ["address"]), the attributes within that object don't get saved. I'm not sure if we want to add a saveUnknown option to the attribute settings, or if we want to allow something like saveUnknown: ["address.*"]. (#731)
[ ] There is currently a difference when updating tables and models vs Dynamoose v1.x. This should be corrected.
[ ] Ensure putting number in place of date works correctly for saving document
[x] set schema attribute setting (#736)
[x] get schema attribute setting (#736)
[ ] Batch get
[ ] Batch delete
[ ] Batch put
[ ] Rethink dynamoose.undefined based on comments in #726
[ ] Test to ensure when updating (setting) an existing Object within document, that if there are required properties within that object, that errors are thrown if we are replacing the entire object and those properties don't exist

Long term steps:

[ ] Fix website (update to support v2.0)
[ ] Model.statics

Not a fan of how this is currently implemented. You set the static methods on the schema, and then call the methods on the Model, which feels really messy. I'm can't think of a better way to do this off the top of my head tho since JavaScript doesn't have support for observing when a property is set.

[ ] Add debug support
[ ] There is an issue where when we do JSON.stringify(test.schema) throughout the test suite, it prints {}, we need to figure out how to fix this
[ ] Consider requiring new keyword for everything and moving to classes instead of constructor functions. This would really help with consistency and also catching bugs in users codebases, because of how internal behaviors work under the hood.
[ ] Figure out what reserved keywords means and implement it

Goals:

100% code coverage with automated tests that cover as many edge cases as can possibly be thought of
Better documentation, everything should be documented as it's being written and created
Fix a lot of pesky long lingering bugs in the codebase (ex. #669)
Better parity with Mongoose
Taking advantage of ES6 and beyond features from top to bottom

This is not always possible. Especially when it comes to classes. https://stackoverflow.com/q/58864709/894067

Have the codebase be more sustainable to adopt more features and functionality in the future, for example things like TypeScript, and new JavaScript features

Other Notes:

I assume that v2.0 will be a step backwards from Dynamoose v1.x at the beginning, for example I don't believe the following items don't make it into v2.0:

Plugins
Populate
TypeScript type file

This PR description will change frequently as work is done on this project, so check back often for updates
v2.0 will include MAJOR breaking changes, it will be likely that we will encourage all users to test their entire Dynamoose stack from top to bottom due to the amount of breaking changes that will be introduced

Process:
At a certain point once basic functionality is stable (retrieving & saving items), we will be publishing prerelease versions to NPM to gather feedback from the community.
News/Announcements:

February 16th 2020 - Beta 1 of Version 2.0 has been released. I'm really looking for a lot of feedback as devs integrate the beta version. If you have a smooth or difficult experience, I want to hear about it all. Feel free to comment below or contact me.
November 14th 2019 - We are looking for submissions of unit or integration tests to be run on features or functionality to be prioritized for version 2.0. These tests should be very through and cover as many edge cases as possible. If you have reported bugs in Dynamoose v1.x that still haven't been resolved (or have), this is a great opportunity to have your issues (or features) be prioritized for v2.0. In order to participate, please submit a pull request with your tests added. This PR should be branched off of and merged into the 2.0 branch.

Internal Notes:

It'd be cool for Model.populate to support passing another Model in as the type of a Schema property, that would basically create a reference to that other Model.

Questions:

Currently Mongoose uses a factory function (no new keyword) for mongoose.model and a constructor function for mongoose.Schema (new keyword). Is there any opposition to standardizing this and making all of them constructor functions for Dynamoose v2.0? Or does anyone see any reasoning for why mongoose.model should be a factory function while mongoose.Schema should be a constructor function for the purposes of Dynamoose?

Thank you for your excellent library, I has been using the version 2.0 and created the
NestJS Dynamoose Module. I also created a demo Serverless Starter Project which utilized those libraries. Thank for your great work again, it make my life so much easier :)

@fishcharlie Great work! Really looking forward for v2 and will try it out :) What do you think about using decorators for everything? Like TypeORM or dynamo-types does?
@hardyscc I just started a new Nest app with DynamoDB and saw your comment randomly, thats awesome! Could you try to get you module into the Nest docs for all others to come?

@MickL I just released this nest module in very short time to full-fill my need, still have many missing pieces. I am waiting Dynamoose 2.0 to come out first and will definitely notify the nest community soon. thanks :)

Thank you for your excellent library, I has been using the version 2.0 and created the
NestJS Dynamoose Module. I also created a demo Serverless Starter Project which utilized those libraries. Thank for your great work again, it make my life so much easier :)

@hardyscc Awesome!!! Glad it worked well for you. If you have any other feedback or suggestions, please let me know.

@fishcharlie Great work! Really looking forward for v2 and will try it out together with the NestJS module :) What do you think about using TypeScript decorators for everything? Like TypeORM or dynamo-types does? I wonder why v2 is not made in TypeScript in general? :)

@MickL I'm the main contributor to Dynamoose at this point, and I have never used TypeScript to date. It is on my list of things to work on. Maintainability is a massive priority for Dynamoose v2. I'm unwilling at this point to jump into a new technology on this project when I don't feel comfortable and confident with it. I believe Dynamoose v2 really prepares for an easy migration to refactoring to use TypeScript sometime soon. It just comes down to my level of prioritization, and I have chosen to prioritize things like documentation, test coverage, bug fixes, and code maintainability, over TypeScript. I believe that is reflected in the work I have done with v2.
As mentioned in the Goals section above Have the codebase be more sustainable to adopt more features and functionality in the future, for example things like TypeScript, and new JavaScript features. So TypeScript is 100% on my radar, and others have mentioned this as well. I'm confident it'll happen at some point (hopefully soon), but I don't know when and can't really give an ETA at this point.
Hopefully that helps clear up that question and gives some insight into my thinking regarding this.

Sounds good and I can totally understand especially when it comes to documentation, test coverage and bug fixes.

https://github.com/dynamoose/dynamoose/releases/tag/v2.0.0


Crash when reading file with > 65535 rows

I have a yxdb file that I cannot share (proprietary) that crashes when reading on record 65535. A hugely suspicious number. It crashed trying to read a string value, with an overflow (iirc... sorry my bad).
The file is not corrupt. I generated it from an AMP/e2 file by saving it as a "regular" AlterYXDB file. Further I can use teh AlterYX Designer to export the file contents as CSV just fine.
The file contents are a handful of strings, a few doubles, and a spatial object.
I tried reading the file using the Python library and it too crashes (did not write out the record number).
Any thoughts on why it might be crashing at record 65535?

It looks like the spatial index might be causing issues. I was able to create a crash using a file I generated. I will need to dig into the file format a bit to figure out how to handle the spatial index, correctly.
In the meantime, you should be able to generate a file that will work with the current version of this package by checking the 'No Spatial Index' checkbox on the output tool:

So glad you were able to repro!
For my needs, for this specific file, exporting to .csv was a good work around. Thanks though!
Do you think the issue could occur in any file with a spatial index? For a different task I've used the library to read files with spatial data and grabbed the geojson without issue. Only 700 records though.


Cannot login with new LDAP account when auto-registration disabled (request invitation code)

Same issue as #1 but using snap instead of docker
Server Setup Information:
Wekan version: 1.82
Operating System: Ubuntu 18.04

Problem description:
When Self-registration is disabled, account existing the ldap, but not on Wekan (never logged-in) cannot login as an invitation-code is requested.
Dec 05 10:24:35 srv-wekan wekan.wekan[7379]: [ERROR] Error creating user { Dec 05 10:24:35 srv-wekan wekan.wekan[7379]:   "isClientSafe": true, Dec 05 10:24:35 srv-wekan wekan.wekan[7379]:   "error": "error-invitation-code-blank", Dec 05 10:24:35 srv-wekan wekan.wekan[7379]:   "reason": "The invitation code is required", Dec 05 10:24:35 srv-wekan wekan.wekan[7379]:   "message": "The invitation code is required [error-invitation-code-blank]", Dec 05 10:24:35 srv-wekan wekan.wekan[7379]:   "errorType": "Meteor.Error"

Yes, it did. First install was on the 25th of october (I don't know what stable version was out there at that time)

I testet with docker version. It worked with v1.75. It does not work anymore with v1.82 or newer.
I cannot test versions between 1.75 and 1.82 because there are no tags between these two versions https://hub.docker.com/r/wekanteam/wekan/tags/

Quay has all tags: quay.io/wekan/wekan
Also see ChangeLog at the time you mentioned.
If you find in what Wekan version changes did break it, it makes fixing easier.

I testet more versions with docker. With versions 1.76 to 1.78 LDAP login with new users still work.
v1.81 is missing "Authentication method" drop down, so no LDAP auth can be testet.
With v1.82 logs say:
[ERROR] Error creating user {
  "isClientSafe": true,
  "error": "error-invitation-code-blank",
  "reason": "The invitation code is required",
  "message": "The invitation code is required [error-invitation-code-blank]",
  "errorType": "Meteor.Error"
}

@rnck
You should have drop down visible with this setting in newest Wekan v1.84:
- LDAP_ENABLE=true

Can you test is there still bug in Wekan v1.84 with this setting?

@xet7 Already testet "latest" version as of today. Now again with image: quay.io/wekan/wekan:v1.84. Dropdown is visible. Login does not work. Same error message as in v1.82:
[ERROR] Error creating user {
  "isClientSafe": true,
  "error": "error-invitation-code-blank",
  "reason": "The invitation code is required",
  "message": "The invitation code is required [error-invitation-code-blank]",
  "errorType": "Meteor.Error"
}

I did all my tests with same docker-compose.yml and changed only the image tags.
Please let me know if you need more information.

@rnck
Thanks! I or @Akuket will look at this sometime.

Saw that there was an update (1.85) but the issue remains. Any idea when it'll be fixed ? Sorry to be in a rush, but I've got users waiting to use wekan though ldap...


Investigate ResizeObserver Errors

Bug Report
We're seeing "ResizeObserver loop limit exceeded" errors pretty frequently in our New Relic logs.  It's occurred on the following page https://engineering.cerner.com/terra-ui/components/terra-form-input/form-input/input. We need to see if there's a problem and how to fix it.

We are not able to reproduce this issue yet. but we have fixed similar error across various components which uses resizeObserver polyfill by introducing animation. frames below are the links.
https://github.com/cerner/terra-framework/pull/212
https://github.com/cerner/terra-clinical/issues/549
https://github.com/cerner/terra-core/pull/1658
The two components which are left out without these changes are terra-application-navigation and terra-infinite-list. so these two components are being used (mainly the terra-application-navigation) by dev-site during load. so i am suspecting this might be causing the issue and introduction of animationframe for both components might fix this error. @ryanthemanuel  your thoughts ?


drumgizmo: 0.9.6 -> <IP_ADDRESS>

Updated Drumgizmo to latest version
Added myself as maintainer

Excellent, thanks @nico202!


libp2p-in-browser Example is not working

See the published version at https://github.com/libp2p/js-libp2p/tree/master/examples/libp2p-in-the-browser. It throws

We are doing some libp2p infrastructure inventory and it looks like the webrtc signaling server wasn't clearly documented. We'll work on getting that back up and documented appropriately.

The server is back up. I ran the example and everything is running as expected.

Thank you @jacobheun :D


When I use accelerate and trying to create the Accelerator object , there always been wrong

System Info
- `Accelerate` version: 0.16.0
- Platform: Linux-4.15.0-189-generic-x86_64-with-glibc2.27
- Python version: 3.9.13
- Numpy version: 1.22.3
- PyTorch version (GPU?): 1.12.1 (False)
- `Accelerate` default config:
        - compute_environment: LOCAL_MACHINE
        - distributed_type: FSDP
        - mixed_precision: no
        - use_cpu: False
        - dynamo_backend: NO
        - num_processes: 8
        - machine_rank: 0
        - num_machines: 1
        - rdzv_backend: static
        - same_network: True
        - main_training_function: main
        - deepspeed_config: {}
        - fsdp_config: {'fsdp_auto_wrap_policy': 'TRANSFORMER_BASED_WRAP', 'fsdp_backward_prefetch_policy': 'BACKWARD_PRE', 'fsdp_offload_params': True, 'fsdp_sharding_strategy': 1, 'fsdp_state_dict_type': 'LOCAL_STATE_DICT', 'fsdp_transformer_layer_cls_to_wrap': 'berry_TEST_layer'}
        - megatron_lm_config: {}
        - downcast_bf16: no

Information

[ ] The official example scripts
[X] My own modified scripts

Tasks

[ ] One of the scripts in the examples/ folder of Accelerate or an officially supported no_trainer script in the examples folder of the transformers repo (such as run_no_trainer_glue.py)
[ ] My own task or dataset (give details below)

Reproduction
All the code is here, and I has installed the NCLL in my computer
from accelerate import Accelerator

accelerator = Accelerator()

All the debug infomation is here

Traceback (most recent call last):
  File "/home/ubuntu/Desktop/dev/练手/翻译/code/123.py", line 3, in <module>
    accelerator = Accelerator()
  File "/home/ubuntu/anaconda3/envs/pytorch39/lib/python3.9/site-packages/accelerate/accelerator.py", line 323, in __init__
    self.state = AcceleratorState(
  File "/home/ubuntu/anaconda3/envs/pytorch39/lib/python3.9/site-packages/accelerate/state.py", line 162, in __init__
    torch.distributed.init_process_group(backend="nccl", **kwargs)
  File "/home/ubuntu/anaconda3/envs/pytorch39/lib/python3.9/site-packages/torch/distributed/distributed_c10d.py", line 602, in init_process_group
    default_pg = _new_process_group_helper(
  File "/home/ubuntu/anaconda3/envs/pytorch39/lib/python3.9/site-packages/torch/distributed/distributed_c10d.py", line 727, in _new_process_group_helper
    raise RuntimeError("Distributed package doesn't have NCCL " "built in")
RuntimeError: Distributed package doesn't have NCCL built in
ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: 1) local_rank: 0 (pid: 47775) of binary: /home/ubuntu/anaconda3/envs/pytorch39/bin/python3.9

Maybe this isn't a 'bug', but I have stock here for one day and I haven't find useful infomation on Google or Github. I'm a newbie， if this issue bother you, I will delete this issue, please let me know.
Expected behavior
Just want this code run successful.

I will try this command line , and thank you.
My operating system is ubuntu 18.04 , pure linux.
I can't figure out why the NCCL can't be found, and I has run the hugging face before , cuda can be used .

You need to install the cuda enabled pytorch version (I think), that's what this part means here as part of your accelerate env: - PyTorch version (GPU?): 1.12.1 (False)
(I think).
You can try:
pip install light-the-torch
ltt install torch torchvision -U

to grab the right wheels
Can you tell me a bit more about your operating system? E.g. is this running on WSL? Windows? Or pure linux?

I will try this command line , and thank you.
My operating system is ubuntu 18.04 , pure linux.
I can't figure out why the NCCL can't be found, and I has run the hugging face before , cuda can be used .

You need to install the cuda enabled pytorch version (I think), that's what this part means here as part of your accelerate env: - PyTorch version (GPU?): 1.12.1 (False)
(I think).
You can try:
pip install light-the-torch
ltt install torch torchvision -U

to grab the right wheels
Can you tell me a bit more about your operating system? E.g. is this running on WSL? Windows? Or pure linux?

Thank you !! Problem has been solved.


Use golangci-lint and resolve all found issues

ISSUE TYPE

Feature Pull Request

SUMMARY

Use golangci-lint as a runner for multiple linters
Parallelize CI steps
Fix all issues reported by linter

NOTE:

All issues were resolved, even these reported by disabled linters because of Go 1.18 incompatibility; I changed go 1.18 to go 1.17 for a while in go.mod to do that, as we don't use generics yet).
I tried to resolve the issues in a minimal way, to make as little changes as possible. Sometimes that wasn't possible.

Resolves https://github.com/infracloudio/botkube/issues/588
TESTING
Tested as usual with Slack, Discord, and Mattermost at the same time.
You can use the following image for testing purposes: docker.io/pkosiec/botkube:golangci-lint-changes.
For example, run:
cat > /tmp/values.yaml << ENDOFFILE
communications:
  slack:
    enabled: true
    token: token
    channel: botkube-test
  discord:
    enabled: true
    botid: "id"
    token: token
    channel: "id"
config:
  settings:
    clustername: sample
    kubectl:
      enabled: true
  enabled: true
image:
  registry: docker.io
  repository: pkosiec/botkube
  tag: golangci-lint-changes
ENDOFFILE

helm install botkube --namespace botkube --create-namespace -f /tmp/values.yaml ./helm/botkube --wait

Thank you, @PrasadG193 and @sanketsudake for review 🙇
I needed to rebase the PR to format one file introduced in previous PR (as I configured goimports formatting following the @sanketsudake's suggestion). Here are the changes since your last review: https://github.com/infracloudio/botkube/pull/593/commits/03b65bd100d708199021dd065931f783e3f0c6ad
I also tested it as usual with Discord, Slack, and Mattermost. Here's the updated image: pkosiec/botkube:golangci-lint-changes-v2.
Let me know what you think 🙂 If it looks good to you, please make the build job not required, and after merging this please turn on Lint Go code, Run tests, Lint Helm chart, Scan repository for vulnerabilities as required. Thank you!

Thank you @mszostok 🙇 , I resolved all the issues and tested the image once again. Here it is: docker.io/pkosiec/botkube:golangci-lint-changes-v3

Rebased this PR once again to make sure mergify will be able to merge this 🙂 Anyway, the required build job would need to be made optional in branch protection (https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/defining-the-mergeability-of-pull-requests/managing-a-branch-protection-rule), as the jobs changed their names and there's no build job anymore. @PrasadG193 if you could change this that would be great. Cheers!

Thank you @PrasadG193, I pushed the fixes, hopefully now everything's fine 🤞 Could you please check the latest commit https://github.com/infracloudio/botkube/pull/593/commits/165284805d22f194db3bd0710313143a66d94c2e with changes?

@Mergifyio refresh